-
Notifications
You must be signed in to change notification settings - Fork 50
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
Rename path argument to "dataset" in hdf5_lookup #775
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks Jakub! I'd still wait for a comment from @danielballan before merging it in case path
needs to be changed in any other places that I'm not aware of.
I agree the old name was confusing. This needs a SQL migration to update existing DataSource documents. |
To create a migration script:
add to the top of this list: Lines 7 to 17 in 15abbc9
In the migration file created above, you need to run an tiled/tiled/catalog/migrations/versions/e756b9381c14_add_structure_family_to_data_sources.py Lines 49 to 53 in 15abbc9
|
Checklist
Recent ophyd_async change renamed the internal HDF path parameter passed to the stream resource to
dataset
frompath
.Required to merge this in sync with bluesky/bluesky#1790.
See: https://github.com/bluesky/ophyd-async/blob/5ac94e283b170b603df42a7d5fe33fd7f2786524/src/ophyd_async/core/_hdf_dataset.py#L64