TEST: Initial regression tests #91
Conversation
|
I'd start looking here about how the injection process works: https://github.com/simonsobs/sotrplib/blob/main/sotrplib/sims/source_injector.py#L42 |
ColmTalbot
force-pushed
the
initial-regression-tests
branch
from
55f6d5b to
c7f75ac
Compare
|
It looks like there's a mismatch in the flux units at some stage because the flux ratios are all coming out at 1000. 
|
|
This is currently taking about 10 minutes to run in the CI. Are we happy with that runtime? |
|
@JBorrow @axf295 it looks like I can't officially request reviews. My current plan is to add a follow up after this is merged that will do some automatic tracking of the uploaded artifacts (post a comment on PRs and possibly send some kind of notification if there's a significant regression on main). |
|
Any idea why the new workload is not running in the CI? |
|
Oh, wait, it did. Never mind. |
|
I think you can merge main and that should fix the flux mismatch |
3 participants
This is some initial regression tests. There are some things that are not included yet (mostly in using the results, they are just uplaoded as artifacts.)
Remaining questions:
Based on #86