Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DM-41878: Implement RemoteButler.get() backed by a single FileDatastore #912

Merged
merged 9 commits into from
Dec 8, 2023

Tweak comments per review

d718a09
Select commit
Loading
Failed to load commit list.
Merged

DM-41878: Implement RemoteButler.get() backed by a single FileDatastore #912

Tweak comments per review
d718a09
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 7, 2023 in 1s

91.83% of diff hit (target 87.50%)

View this Pull Request on Codecov

91.83% of diff hit (target 87.50%)

Annotations

Check warning on line 201 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L201 

Added line #L201 was not covered by tests

Check warning on line 212 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L212 

Added line #L212 was not covered by tests

Check warning on line 278 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L277-L278 

Added lines #L277 - L278 were not covered by tests

Check warning on line 285 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L285 

Added line #L285 was not covered by tests

Check warning on line 290 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L290 

Added line #L290 was not covered by tests

Check warning on line 292 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L292 

Added line #L292 was not covered by tests

Check warning on line 328 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L328 

Added line #L328 was not covered by tests

Check warning on line 412 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L412 

Added line #L412 was not covered by tests

Check warning on line 438 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L438 

Added line #L438 was not covered by tests

Check warning on line 445 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L445 

Added line #L445 was not covered by tests

Check warning on line 513 in python/lsst/daf/butler/datastores/file_datastore/get.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

python/lsst/daf/butler/datastores/file_datastore/get.py#L513 

Added line #L513 was not covered by tests
View more details on Codecov