search

PieterMostert / Lipgloss

Python app for creating glaze recipes
GNU General Public License v3.0
5 stars 4 forks source link

Other restrictions #2

Closed PieterMostert closed 7 years ago

PieterMostert commented 7 years ago

The script breaks down when it tries to interpret the normalization of any 'other restrictions' that have been selected. At the moment, the only 'other restrictions' are SiO2:Al2O3, LOI, Cost and Total clay. The first has the normalization lp_var['mole_Al2O3'], while the rest have the normalization 0.01*lp_var['total_ingredients']

When we implement the ability for users to customise 'other restrictions', it will probably be best for them to select either oxide weight, oxide mole, or ingredient weight, and then fill in the coefficients by which these variables should be multiplied, before summing them to give the numerator. Likewise for the denominator, except that here we should also include the various 'total' variables. Even if we don't include customisable 'other restrictions' in the first release, we should encode the numerators and denominators of the default 'other variables' in a manner that reflects the way we want to customise them.

PieterMostert commented 7 years ago

The 'other restrictions' have now been increased to include 'KNaO UMF' and 'KNaO % mol'. The normalizations of 'LOI', 'Cost' and 'Total clay' are now just lp_var['total_ingredients'].