LSSTDESC / imSim

GalSim based Rubin Observatory image simulation package
https://lsstdesc.org/imSim
BSD 3-Clause "New" or "Revised" License
36 stars 15 forks source link

Check spelling/validity of confirmation file entries. #341

Closed cwwalter closed 1 year ago

cwwalter commented 1 year ago

Currently, a misspelled/wrong yaml file entry will just be ignored. This has caused some confusion and mistakes by users during testing.

@rmjarvis told us in a meeting that there is defined a straightforward way to check if entries are proper/well formed.

We should do this if we can.

rmjarvis commented 1 year ago

This is the function that checks that all parameters are valid. It's automatic if using GetAllValues, but we can run it manually if everything is parsed manually.
Here is an example of how to use it in practice.

cwwalter commented 1 year ago

Classes missing parameter check linting now are:

We need to convert these to catch input mistakes.

cwwalter commented 1 year ago

Closed by #387.