search

illinois-ceesd / mirgecom

MIRGE-Com is the workhorse simulation application for the Center for Exascale-Enabled Scramjet Design at the University of Illinois.
Other
11 stars 19 forks source link

Replace `configurate` with `dict.get`? #879

Open majosm opened 1 year ago

majosm commented 1 year ago

configurate in simutil seems to mostly be duplicating the behavior of the existing dict.get(). This issue is a reminder to look into whether configurate is actually needed.

majosm commented 1 year ago

Might need to handle https://stackoverflow.com/questions/30458977/yaml-loads-5e-6-as-string-and-not-a-number in read_and_distribute_yaml_data in order to avoid having to cast when retrieving from the dict.