Skip to content

Conversation

@jeremykubica
Copy link
Collaborator

This notebook is part of the same series started in #620, demonstrating how users can wrap their own packages for use in LightCurveLynx

@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 Nov 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.59%. Comparing base (1b733bc) to head (13a9fc9).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #622   +/-   ##
=======================================
  Coverage   94.59%   94.59%           
=======================================
  Files          62       62           
  Lines        5417     5425    +8     
=======================================
+ Hits         5124     5132    +8     
  Misses        293      293           

☔ 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 Nov 25, 2025

Before [27dabff] After [f6b2800] Ratio Benchmark (Parameter)
122±7μs 128±3μs 1.05 benchmarks.TimeSuite.time_evaluate_simple_linear_wavelength_model
5.17±0.1ms 5.25±0.08ms 1.02 benchmarks.TimeSuite.time_lightcurve_source
1.02±0s 1.04±0.01s 1.02 benchmarks.TimeSuite.time_make_x1_from_hostmass
7.59±0.07ms 7.66±0.1ms 1.01 benchmarks.TimeSuite.time_apply_white_noise
26.9±0.3ms 27.3±0.4ms 1.01 benchmarks.TimeSuite.time_make_and_evaluate_static_sed
30.1±0.3μs 30.4±0.5μs 1.01 benchmarks.TimeSuite.time_make_simple_linear_wavelength_model
48.3±0.8ms 48.3±0.8ms 1 benchmarks.TimeSuite.time_additive_multi_model_source
556±4μs 557±10μs 1 benchmarks.TimeSuite.time_apply_passbands
19.5±0.2ms 19.5±0.2ms 1 benchmarks.TimeSuite.time_chained_evaluate_sed
9.43±0.6ms 9.40±0.3ms 1 benchmarks.TimeSuite.time_evaluate_salt3_model

Click here to view all benchmarks.

@jeremykubica jeremykubica merged commit 7ed6681 into main Nov 25, 2025
8 checks passed
@jeremykubica jeremykubica deleted the wrapping_redback branch November 25, 2025 18:46
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.

3 participants