-
Notifications
You must be signed in to change notification settings - Fork 21
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
parallel load with no attributes #253
Conversation
jrs65
commented
Aug 16, 2023
- feat hints to override
- fix(memh5): error with distributed loading of files with datasets in groups
- ffirst
- refactor(memh5): move no selections sentinel to empty list to simplify
d94f17d
to
eca92ff
Compare
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.
I think you can go ahead and merge this. I couldn't find anything wrong with this, but can't say that I have fully understood everything.
For my benefit, maybe you can explain
- In
_copy_from_file
, what would you expect the difference to be betweenkey
anditem.name
? - What's the benefit of passing
fname
rather thanh5group
toMPIArray.from_file
?
This extends the hints argument to a dictionary to allow it to force distributed loading of datasets with no specific `__memh5_distributed` markers.
eca92ff
to
8523abf
Compare
8523abf
to
41496df
Compare
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.
Are the failed run-tests caused by something upstream?
This swaps from using the pytest-lazy-fixture plugin to the pytest-lazy-fixtures plugin which supports the underlying changes in pytest 8.
Also update the ruff config to avoid deprecated usage.