LSSTDESC / rail

Top level "umbrella" package for RAIL
MIT License
5 stars 3 forks source link

BPZ demo data in smoke test #112

Closed OliviaLynn closed 7 months ago

OliviaLynn commented 7 months ago

Bug report Need to add commands to smoke test for getting bpz data from NERSC

Otherwise, the newly moved in bpz notebooks fail:

Before submitting Please check the following:

sschmidt23 commented 7 months ago

I think the commands are in the notebook, we just have them commented out by default because we didn't like the idea of having a notebook download data files when people just do a run all cells on a demo. We can talk about whether it's ok to just uncomment the lines grabbing the extra files, if it's a blocker for the smoke tests I'd say we just do that.