dill 0.3.7 doesn't completely preserve the __dict__ or __module__ of subclasses pickled by value: see https://github.com/uqfoundation/dill/issues/612. This broke our mechanism for detecting when subclasses of OrientedPoint2D have already been transformed, as well as the pickle tests asserting equivalence of Scenic objects and their unpickled versions. This PR fixes the former by working around the bug, and silences the latter by adding a special case to areEquivalent to ignore wrong __module__ values when using dill 0.3.7 (hopefully this will be fixed in subsequent versions).
(I've also updated the URL used to download OpenSCAD: their website is having issues and they recommend getting the releases from GitHub in the mean time.)
dill0.3.7 doesn't completely preserve the__dict__or__module__of subclasses pickled by value: see https://github.com/uqfoundation/dill/issues/612. This broke our mechanism for detecting when subclasses ofOrientedPoint2Dhave already been transformed, as well as the pickle tests asserting equivalence of Scenic objects and their unpickled versions. This PR fixes the former by working around the bug, and silences the latter by adding a special case toareEquivalentto ignore wrong__module__values when usingdill0.3.7 (hopefully this will be fixed in subsequent versions).(I've also updated the URL used to download OpenSCAD: their website is having issues and they recommend getting the releases from GitHub in the mean time.)