search

tardis-sn / stardis

TARDIS stellar radiative transfer
https://tardis-sn.github.io/stardis/
11 stars 15 forks source link

add arg to the config_schema.yml file for gzipped and hook up in run_stardis #116

Closed jvshields closed 1 year ago

jvshields commented 1 year ago

Allows the config to take a gzipped argument in model and passes that to the model reader.

This allows the config to point to any plane-parallel model obtained from https://marcs.oreme.org/

codecov[bot] commented 1 year ago

Codecov Report

Merging #116 (faf3362) into main (437e371) will not change coverage. Report is 1 commits behind head on main. The diff coverage is n/a.

:exclamation: Current head faf3362 differs from pull request most recent head 679f54d. Consider uploading reports for the commit 679f54d to get more accurate results

@@           Coverage Diff           @@
##             main     #116   +/-   ##
=======================================
  Coverage   33.89%   33.89%           
=======================================
  Files          17       17           
  Lines         658      658           
=======================================
  Hits          223      223           
  Misses        435      435
Files Changed Coverage Δ
stardis/base.py 33.33% <ø> (ø)

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

tardis-bot commented 1 year ago

*beep* *bop* Hi human, I ran benchmarks as you asked comparing main (437e37112be20f05ea00027476b646fdb6cb0f5e) and the latest commit (679f54df9ac02da5c0d8ed9f07b0b99ed16a3710). Here are the logs produced by ASV. Results can also be downloaded as artifacts here. Significantly changed benchmarks:

```diff ```

All benchmarks:

```diff All benchmarks: before after ratio [437e3711] [679f54df] 15.1±0.07s 15.4±0.1s 1.03 run_stardis.BenchmarkRunStardis.time_run_stardis ```