desihub / desisurvey

Code for desi survey planning and implementation
BSD 3-Clause "New" or "Revised" License
2 stars 7 forks source link

add options to override config file and tiles file #62

Closed sbailey closed 7 years ago

sbailey commented 7 years ago

This PR adds several features useful for small scale functional testing:

Other changes along the way:

A companion update to surveysim will add the --config-file option there and update the tutorial instructions.

I suggest making tag 0.9.0 after this is merged (version file was already update but tag never made...)

dkirkby commented 7 years ago

I did make the 0.9.0 tag at the appropriate time, but forgot to push it (now fixed).

dkirkby commented 7 years ago

I see you edited changes.rst assuming that I didn't make the tag, so please update that.

dkirkby commented 7 years ago

Let's pull the docs at the head of rules.yaml and now rules-layers.yaml into a separate file (README.md?) so it isn't duplicated and easier to maintain.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.1%) to 38.163% when pulling 40f3432791a2d57243f3febe1dd77912a2a3341a on config into 9674ae960e3f71d18e72a397c82c51e51b6f29ca on master.

sbailey commented 7 years ago
coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.2%) to 38.148% when pulling 4ef6ad70a7e98937b2a67ec13e53f4df5478830a on config into 9674ae960e3f71d18e72a397c82c51e51b6f29ca on master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+18.6%) to 56.874% when pulling df848e8d9ba52ae939ba6753475e9fb36005156d on config into 9674ae960e3f71d18e72a397c82c51e51b6f29ca on master.

sbailey commented 7 years ago

postfacto comment: the new unit tests run fast when using a desimodel test data subset, but are too slow when running on desimodel svn trunk data with all tiles. I will fix that now.