added "phase transition properties" to pyrrhotite in mines16 data file

[cardinalgeo]

I added the "phase transition properties" (m_phase_trans_props) to the pyrrhotite entry in the mines16 data file. It seems that the pyrrhotite entry is the only one requiring correction. In this case—and in many cases in other data files—temperature is the only property tabulated, so the info stored in m_phase_trans_props is redundant wrt what is already stored in the entry. However, both ThermoFun and Reaktoro throw errors (see screenshot for ThermoFun attached) when trying to calculate thermo. properties for pyrrhotite at temperatures above its phase transitions, so it seems this redundant info is still required. In the future, would it be possible to take the temperature from limitsTP when m_phase_trans_props is not specified? This would reduce redundant info in the data file in the case that only phase transition temperature would be included in m_phase_trans_props (i.e., dS, dH, dV, dT/dP of transition are unknown/zero).

[cardinalgeo]

Also, I made the above change in the thermofun repository rather than the thermohubclient repository, as I couldn't find the mines16 data file in the latter. Does thermohubclient use the mines16 datafile stored in the thermofun repository?

[gdmiron]

There is no connection between these files and the thermohub database. But your issues gave me an idea to keep a github repo as a connection where people can suggest fixes :). I also have ti add in the CI that when a commit to master is executed the database in thermohub should be updated.

In any case I have update the data in ThermoHub.