Argonne-National-Laboratory / Frhodo

Simulate experimental data and optimize chemical kinetics mechanisms with this GUI-based application
Other
30 stars 8 forks source link

Duplicate information in `Chemical_Mechanism` class #6

Open WardLT opened 2 years ago

WardLT commented 2 years ago

The type of uncertainty is stored both in the "limits" class and in the dictionary defining the bounds. Reducing it to only one location will simplify the code.

WardLT commented 2 years ago

It looks like the one in the bounds dictionary is set by the UI. Presumably that'll be the one that takes precedent.

WardLT commented 2 years ago

Wait, the parameter in Uncertainty doesn't seem to be used. Could we just get rid of it? 🤞🏼