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.

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

Help listruns() default to the current catalog in use. #474

Merged
merged 2 commits into from
Jan 13, 2021
Merged

Conversation

prjemian
Copy link
Contributor

FIX #468

@prjemian prjemian added this to the 1.4.0 milestone Jan 13, 2021
@prjemian
Copy link
Contributor Author

Raising the priority since we are seeing this as a problem with some beamlines using the intake-style catalogs.

@prjemian prjemian changed the title listruns() can make better choice of default catalog Help listruns() default to the current catalog in use. Jan 13, 2021
@prjemian prjemian merged commit 5b0e75b into main Jan 13, 2021
@prjemian prjemian deleted the 468-listruns branch January 13, 2021 18:58
@lgtm-com
Copy link

lgtm-com bot commented Jan 13, 2021

This pull request introduces 2 alerts when merging 0aa134b into f14240d - view on LGTM.com

new alerts:

  • 2 for Unused local variable

prjemian added a commit that referenced this pull request Jan 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

listruns() should get db from session if possible
1 participant