-
Notifications
You must be signed in to change notification settings - Fork 276
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
update owls_subfind frontend and add some format flexibility #3325
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.
LGTM, thank you!
@@ -4,9 +4,12 @@ | |||
from yt.utilities.io_handler import BaseIOHandler | |||
from yt.utilities.on_demand_imports import _h5py as h5py | |||
|
|||
_pos_names = ["CenterOfMass", "CentreOfMass"] |
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 almost suggested changing this to be a tuple, and then I realized we may actually want it to be mutable.
@brittonsmith can you rebase this on main? Should fix the unrelated test failures. |
These are failing with no old answers. What do we have to do update that? |
PR Summary
Resolves Issue #3307.
This updates the owls_subfind frontend to yt-4.0 io. I've also added some flexibility for specific file attributes and naming conventions to account for a variant of this frontend that was identified on the slack channel. This seems to work for the sample data I have for that.
PR Checklist