usnistgov / REFPROP-wrappers

Wrappers around NIST REFPROP for languages such as Python, MATLAB, etc.
195 stars 127 forks source link

Mathematica Wrapper Units Patch for Mathematica 13.2 #503

Closed henningjp closed 1 year ago

henningjp commented 1 year ago

Description of the Change

Mathematica 13.2 introduced some changes in unit handling that created errors in the Mathematica Wrapper. These have been patched including:

Benefits

Eliminates error message when loading the RefpropLink wrapper paclet, and when converting between Rankine and other temperature units. Also deprecates the affine temperature scaling variable definitions for °F and °C.

Possible Drawbacks

Temperatures with affine units must now be entered using Mathematica's Quantity[ ] function.

    T = Quantity[72.0, "DegreesFahrenheit"]
    T2 = Quantity[22.0, "DegreesCelsius"]

Also, extensive documentation still uses the provided °F and °C unit variables. However, an upcoming PR to address #502 will correct this misuse of °F and °C in the overhauled documentation files.

Verification Process

Changes were rather simple and can be viewed in the file changes. However, documentation examples were re-run and did not show any errors.

Applicable Issues

Closes #495

ianhbell commented 1 year ago

This one is good too, right?

henningjp commented 1 year ago

Yep. Simple patches.

ianhbell commented 1 year ago

Thank you!