serialize.py: np.float128 and np.complex256 are not implemented on some platforms about vivarium-core HOT 5 CLOSED

It looks like float64 is the default for numpy, so we'd only get np.float128 if someone specified that explicitly, which seems unlikely. We could remove these and note in the docs that we don't support those data types?

from vivarium-core.

who put those in and why? Can you check the git blame @cyrus-bio?

Maybe we can check if the type exists, and only use it if it does?

from vivarium-core.

From a cursory search it seems that most people just use np.float64 at most, or do something along the lines of:
if hasattr(np, 'float128'): do A else: do B

from vivarium-core.

I think I put those in. They just tell Vivarium that the numpy serializer should be used for objects of type np.float128. If someone does provide a np.float128 object after we remove the np.float128 reference from this file, nothing will break. Vivarium will just take slightly longer to find the right serializer (it will loop through the serializers instead of looking the right serializer up in a dictionary)

from vivarium-core.

Sounds like we can safely remove it then. @cyrus-bio -- want to open a pull request in vivarium-core?

from vivarium-core.