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

feat: auto_divisibility, expand_bbox_resolution, expand_bbox_backend #426

Merged
merged 1 commit into from
Jun 29, 2023

Conversation

dodamih
Copy link
Collaborator

@dodamih dodamih commented Jun 29, 2023

Note that the legacy expand_bbox is now called expand_bbox_processing - all the CUE files have been updated. Docstring will be part of the Docs PR.

@codecov
Copy link

codecov bot commented Jun 29, 2023

Codecov Report

Merging #426 (3ce2858) into main (dcf3bd7) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 3ce2858 differs from pull request most recent head 7961607. Consider uploading reports for the commit 7961607 to get more accurate results

@@            Coverage Diff            @@
##              main      #426   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          120       120           
  Lines         3573      3568    -5     
=========================================
- Hits          3573      3568    -5     
Impacted Files Coverage Δ
zetta_utils/geometry/bbox.py 100.00% <100.00%> (ø)

... and 6 files with indirect coverage changes

Copy link
Member

@supersergiy supersergiy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for patching all the specs!

@supersergiy supersergiy merged commit 0b102a8 into main Jun 29, 2023
13 checks passed
@supersergiy supersergiy deleted the dodam/nodivide branch August 1, 2023 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants