Test batch single_lc with mean instead of stetsonj - Githubissues

lincc-frameworks / tape

[Deprecated] Package for working with LSST time series data

https://tape.readthedocs.io

MIT License

12 stars 3 forks source link

Test batch single_lc with mean instead of stetsonj #429

Closed wilsonbb closed 4 months ago

wilsonbb commented 4 months ago

Change Description

[ ] My PR includes a link to the issue that I am addressing

Solution Description

Code Quality

[ ] My code builds (or compiles) cleanly without any errors or warnings
[ ] My code contains relevant comments and necessary documentation

Project-Specific Pull Request Checklists

[ ] I have added a function that requires a sync_tables command, and have added the neccesary sync_tables call

Bug Fix Checklist

[ ] My fix includes a new test that breaks as a result of the bug (if possible)
[ ] My change includes a breaking change
- [ ] My change includes backwards compatibility and deprecation warnings (if possible)

New Feature Checklist

[ ] I have added or updated the docstrings associated with my feature using the NumPy docstring format
[ ] I have updated the tutorial to highlight my new feature (if appropriate)
[ ] I have added unit/End-to-End (E2E) test cases to cover my new feature
[ ] My change includes a breaking change
- [ ] My change includes backwards compatibility and deprecation warnings (if possible)

Documentation Change Checklist

[ ] Any updated docstrings use the NumPy docstring format

Build/CI Change Checklist

[ ] If required or optional dependencies have changed (including version numbers), I have updated the README to reflect this
[ ] If this is a new CI setup, I have added the associated badge to the README

Other Change Checklist

[ ] Any new or updated docstrings use the NumPy docstring format.
[ ] I have updated the tutorial to highlight my new feature (if appropriate)
[ ] I have added unit/End-to-End (E2E) test cases to cover any changes
[ ] My change includes a breaking change
- [ ] My change includes backwards compatibility and deprecation warnings (if possible)

github-actions[bot] commented 4 months ago

Before [89aa1169]	After [6f796700]	Ratio	Benchmark (Parameter)
34.2±0.3ms	34.4±0.08ms	1	benchmarks.time_batch
34.7±0.1ms	34.2±0.3ms	0.99	benchmarks.time_prune_sync_workflow

Click here to view all benchmarks.

wilsonbb commented 4 months ago

Closing debugging PR