Skip to content

Conversation

@smcguire-cmu
Copy link
Contributor

@smcguire-cmu smcguire-cmu commented Dec 4, 2025

Solves #956

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.50%. Comparing base (49881cc) to head (7b18869).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1155   +/-   ##
=======================================
  Coverage   96.50%   96.50%           
=======================================
  Files          45       45           
  Lines        2750     2750           
=======================================
  Hits         2654     2654           
  Misses         96       96           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

Before [49881cc] After [53e461a] Ratio Benchmark (Parameter)
721±5ms 748±30ms 1.04 benchmarks.time_open_many_columns_default
569±3ms 593±30ms 1.04 benchmarks.time_open_many_columns_list
6.89±0.01s 7.00±0.02s 1.02 benchmarks.time_lazy_crossmatch_many_columns_all_suffixes
3.30±0.02s 3.37±0.01s 1.02 benchmarks.time_open_many_columns_all
5.53±0.01s 5.58±0.02s 1.01 benchmarks.time_create_large_catalog
829±4ms 835±8ms 1.01 benchmarks.time_create_midsize_catalog
70.6±0.8ms 71.2±0.9ms 1.01 benchmarks.time_kdtree_crossmatch
6.97±0.02s 7.03±0.03s 1.01 benchmarks.time_lazy_crossmatch_many_columns_overlapping_suffixes
16.8±0.03s 16.7±0s 1 benchmarks.time_save_big_catalog
21.8±0.3ms 21.6±0.1ms 0.99 benchmarks.time_box_filter_on_partition

Click here to view all benchmarks.

@smcguire-cmu smcguire-cmu changed the title add explode doc Add explode lightcurves doc page Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants