-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Migration of datatree/ops.py -> datatree_ops.py #8976
Merged
TomNicholas
merged 12 commits into
pydata:main
from
flamingbear:mhs/DAS-2065/migrate-ops
May 2, 2024
Merged
Migration of datatree/ops.py -> datatree_ops.py #8976
TomNicholas
merged 12 commits into
pydata:main
from
flamingbear:mhs/DAS-2065/migrate-ops
May 2, 2024
Commits on Apr 26, 2024
-
DAS-2065: direct migration of datatree/ops.py -> datatree_ops.py
I considered wedging this into core/ops.py, but the datatree/ops.py stuff is kind of spread into core/ops.py and generated_aggregations.py.
Configuration menu - View commit details
-
Copy full SHA for b22a56e - Browse repository at this point
Copy the full SHA b22a56eView commit details -
Configuration menu - View commit details
-
Copy full SHA for d072845 - Browse repository at this point
Copy the full SHA d072845View commit details -
DAS-2065: Fix leading space in docstrings
These are the only docstring that have a leading space and that was causing problems injecting the map_over_subtree information in the Datatree doc strings.
Configuration menu - View commit details
-
Copy full SHA for 7d59d3a - Browse repository at this point
Copy the full SHA 7d59d3aView commit details -
DAS-2065: Puts the docstring addendum as second paragraph
This works on most of the docstrings. The DatasetOpsMixin functions (round, argsorg, conj and conjugate) have different format and this gets inserted after the name (which is non standard in most docs) but before the description.
Configuration menu - View commit details
-
Copy full SHA for 5954034 - Browse repository at this point
Copy the full SHA 5954034View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3a494ae - Browse repository at this point
Copy the full SHA 3a494aeView commit details -
DAS-2065: Additonal update to make the addendum a Note
Just syntactic sugar to make that work
Configuration menu - View commit details
-
Copy full SHA for 5859b5c - Browse repository at this point
Copy the full SHA 5859b5cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3575257 - Browse repository at this point
Copy the full SHA 3575257View commit details -
Configuration menu - View commit details
-
Copy full SHA for f80a466 - Browse repository at this point
Copy the full SHA f80a466View commit details
Commits on Apr 29, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 51e8f90 - Browse repository at this point
Copy the full SHA 51e8f90View commit details -
Configuration menu - View commit details
-
Copy full SHA for b5ccd64 - Browse repository at this point
Copy the full SHA b5ccd64View commit details
Commits on Apr 30, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 696cd3e - Browse repository at this point
Copy the full SHA 696cd3eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3f65927 - Browse repository at this point
Copy the full SHA 3f65927View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.