VTDA-Group / superphot-plus

Superphot+, upgraded version of Superphot that uses nested sampling
MIT License
10 stars 0 forks source link

hotfix to allow old LCs to be read #161

Closed kdesoto-astro closed 1 year ago

kdesoto-astro commented 1 year ago

Change Description

Code Quality

codecov[bot] commented 1 year ago

Codecov Report

Merging #161 (2c309cf) into main (e3e188d) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 2c309cf differs from pull request most recent head cc2d102. Consider uploading reports for the commit cc2d102 to get more accurate results

@@           Coverage Diff           @@
##             main     #161   +/-   ##
=======================================
  Coverage   75.70%   75.70%           
=======================================
  Files          36       36           
  Lines        2914     2914           
=======================================
  Hits         2206     2206           
  Misses        708      708           
Files Changed Coverage Δ
src/superphot_plus/lightcurve.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

github-actions[bot] commented 1 year ago
Before After Ratio Method
[e3e188d1] [9c604f56]
9.81±0.02s 12.6±1s ~1.28 inference_suite.InferenceSuite.time_numpyro_nuts_inference
8.37±0s 8.49±0.01s 1.01 sampling_suite.SamplingSuite.time_svi_single_file
4.30±0.04s 4.35±0s 1.01 sampling_suite.SamplingSuite.time_dynesty_single_file
124±1ms 126±0.9ms 1.01 sampling_suite.SamplingSuite.time_iminuit_single_file
12.6±1s 12.6±1s 1.00 sampling_suite.SamplingSuite.time_nuts_single_file
174±0.4ms 174±0.2ms 1.00 sampling_suite.SamplingSuite.time_licu_mcmc_ceres_single_file
8.47±0s 8.41±0.04s 0.99 inference_suite.InferenceSuite.time_numpyro_svi_inference
9.19±0.06ms 9.09±0.1ms 0.99 sampling_suite.SamplingSuite.time_licu_ceres_single_file
4.61±0.2s 4.54±0.2s 0.98 inference_suite.InferenceSuite.time_dynesty_inference

Click here to view all benchmarks.

github-actions[bot] commented 1 year ago
Before After Ratio Method
[e3e188d1] [f424f664]
13.1±1s 16.2±0.07s ~1.24 inference_suite.InferenceSuite.time_numpyro_nuts_inference
8.68±0.02s 8.79±0.03s 1.01 inference_suite.InferenceSuite.time_numpyro_svi_inference
9.31±0.02ms 9.32±0.03ms 1.00 sampling_suite.SamplingSuite.time_licu_ceres_single_file
175±0.3ms 175±0.2ms 1.00 sampling_suite.SamplingSuite.time_licu_mcmc_ceres_single_file
8.72±0.02s 8.68±0.04s 1.00 sampling_suite.SamplingSuite.time_svi_single_file
4.71±0.1s 4.66±0.07s 0.99 inference_suite.InferenceSuite.time_dynesty_inference
127±0.8ms 124±1ms 0.98 sampling_suite.SamplingSuite.time_iminuit_single_file
4.51±0.01s 4.36±0.05s 0.97 sampling_suite.SamplingSuite.time_dynesty_single_file
16.2±0.02s 10.2±0.03s 0.63 sampling_suite.SamplingSuite.time_nuts_single_file

Click here to view all benchmarks.