Skip to content

Comments

Prototype Implementation of the Argus Survey#732

Draft
jeremykubica wants to merge 14 commits intomainfrom
argus
Draft

Prototype Implementation of the Argus Survey#732
jeremykubica wants to merge 14 commits intomainfrom
argus

Conversation

@jeremykubica
Copy link
Collaborator

This PR is the initial implementation of the Argus survey wrapper based on the example files provided by the Argus team.

@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 Feb 13, 2026

Codecov Report

❌ Patch coverage is 98.11321% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.53%. Comparing base (de3b17f) to head (11f4479).

Files with missing lines Patch % Lines
src/lightcurvelynx/obstable/argus_obstable.py 98.11% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #732      +/-   ##
==========================================
+ Coverage   92.44%   92.53%   +0.09%     
==========================================
  Files          69       70       +1     
  Lines        6429     6535     +106     
==========================================
+ Hits         5943     6047     +104     
- Misses        486      488       +2     

☔ 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 Feb 13, 2026

Before [de3b17f] After [024bb23] Ratio Benchmark (Parameter)
10.4±0.3ms 10.5±0.4ms 1.02 benchmarks.TimeSuite.time_evaluate_salt3_model
1.16±0.01ms 1.17±0.01ms 1.01 benchmarks.TimeSuite.time_apply_passbands
671±10μs 679±10μs 1.01 benchmarks.TimeSuite.time_fnu_to_flam
71.2±1ms 71.6±0.8ms 1.01 benchmarks.TimeSuite.time_make_and_evaluate_static_sed
7.88±0.07ms 7.88±0.08ms 1 benchmarks.TimeSuite.time_apply_white_noise
23.2±0.1ms 23.3±0.1ms 1 benchmarks.TimeSuite.time_chained_evaluate_sed
7.60±0.06ms 7.62±0.09ms 1 benchmarks.TimeSuite.time_lightcurve_source
1.12±0s 1.12±0.02s 1 benchmarks.TimeSuite.time_make_x1_from_hostmass
120±1μs 120±2μs 1 benchmarks.TimeSuite.time_sample_x0_from_distmod
16.1±0.2μs 16.1±0.2μs 1 benchmarks.TimeSuite.time_sample_x1_from_hostmass

Click here to view all benchmarks.

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.

1 participant