lincc-frameworks / tape

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

fix divisions warnings #411

Closed dougbrn closed 5 months ago

dougbrn commented 5 months ago

Change Description

Resolves #410

Solution Description

Code Quality

Project-Specific Pull Request Checklists

Bug Fix Checklist

New Feature Checklist

Documentation Change Checklist

Build/CI Change Checklist

Other Change Checklist

review-notebook-app[bot] commented 5 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

github-actions[bot] commented 5 months ago
Before [60f12c36] After [8469c02d] Ratio Benchmark (Parameter)
35.7±2ms 36.3±1ms 1.02 benchmarks.time_prune_sync_workflow
36.0±1ms 34.6±1ms 0.96 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.78%. Comparing base (60f12c3) to head (d6cdc97).

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

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