GalSim-developers / GalSim

The modular galaxy image simulation toolkit. Documentation:
http://galsim-developers.github.io/GalSim/
Other
224 stars 105 forks source link

Add list (and dict, LookupTable) as a registered type for Eval #1220

Closed rmjarvis closed 1 year ago

rmjarvis commented 1 year ago

@jmeyers314 discovered a bug in the config parsing that prevented Eval types from being used to generate a list object, when parsed in the normal way. The oversight was that list (and dict and LookupTable) were not registered as valid types associated with the Eval type.

The workaround is to use None for the value_type rather than list, but this is inelegant and less type-safe.