lincc-frameworks / tape

[Deprecated] Package for working with LSST time series data
https://tape.readthedocs.io
MIT License
12 stars 3 forks source link

add force flag to calc_nobs repartitioning #413

Closed dougbrn closed 5 months ago

dougbrn commented 5 months ago

Change Description

Resolves #405.

Solution Description

All this needed was the force=True flag to allow light partition expansion when the boundaries don't overlap completely.

Code Quality

Project-Specific Pull Request Checklists

Bug Fix Checklist

New Feature Checklist

Documentation Change Checklist

Build/CI Change Checklist

Other Change Checklist

github-actions[bot] commented 5 months ago
Before [750fe4b5] <v0.4.0> After [4f46bcd5] Ratio Benchmark (Parameter)
33.9±0.4ms 35.5±1ms 1.05 benchmarks.time_prune_sync_workflow
32.5±0.6ms 33.1±0.4ms 1.02 benchmarks.time_batch

Click here to view all benchmarks.

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.77%. Comparing base (750fe4b) to head (fffa3f6).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #413 +/- ## ======================================= Coverage 95.77% 95.77% ======================================= Files 25 25 Lines 1751 1751 ======================================= Hits 1677 1677 Misses 74 74 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.