lincc-frameworks / tdastro

MIT License
4 stars 0 forks source link

PhysicalModel.get_band_fluxes(..., filters=...) #144

Closed hombit closed 1 month ago

hombit commented 1 month ago

This changes the interface of the method to get one extra argument and to always return an array

Fixes #138

Change Description

Solution Description

Code Quality

Project-Specific Pull Request Checklists

Bug Fix Checklist

New Feature Checklist

Documentation Change Checklist

Build/CI Change Checklist

Other Change Checklist

github-actions[bot] commented 1 month ago
Before [215228d1] After [dce23edf] Ratio Benchmark (Parameter)
7.02±0.06ms 7.06±0.04ms 1.01 benchmarks.time_chained_evaluate
1.38±0.01s 1.39±0.02s 1.01 benchmarks.time_x1_from_hostmass

Click here to view all benchmarks.