This pull request addresses the following API changes for third party dependencies:
matplotlib >=3.9.1 now uses colorbar._draw_all() instead of color_bar.draw_all()
numpy >= 2.0.0 changed the behavior of np.ogrid() to return a tuple instead of a list. Address this by manually casting ogrid output to a list
numpy >= 2.0.0 changed the behavior of np.array(['0', '0', '1']).astype(bool) to return all True instead of [False, False, True]. Address it by casting to int and comparing with == 1
numpy >= 2.0.0 removed newbyteorder() as an array method. Recommended change is arr.view(arr.dtype.newbyteorder()).byteswap()
This was tested with numpy v1.26.0 and v2.0.1 to ensure backwards compatibility. All unit tests passed in both numpy versions.
I added a new unit test tests/missions/fermi/test_plot.py to help catch future matplotlib API changes.
I reduced the relative tolerance on two unit tests inside tests/missions/fermi/gbm/dol/test_legacy_functions.py down to 1e-2 (1%). This was the tolerance I originally intended for them. We didn't catch them in an earlier round of testing. They cropped up here because float32 support is handled a little different on my new mac.
Not sure why some of my older commits from Aug 6 & 7 appear in the commit log. They were already added to main in PR #38.
This pull request addresses the following API changes for third party dependencies:
colorbar._draw_all()
instead ofcolor_bar.draw_all()
np.ogrid()
to return a tuple instead of a list. Address this by manually casting ogrid output to a listnewbyteorder()
as an array method. Recommended change isarr.view(arr.dtype.newbyteorder()).byteswap()
This was tested with numpy v1.26.0 and v2.0.1 to ensure backwards compatibility. All unit tests passed in both numpy versions.
I added a new unit test
tests/missions/fermi/test_plot.py
to help catch future matplotlib API changes.I reduced the relative tolerance on two unit tests inside
tests/missions/fermi/gbm/dol/test_legacy_functions.py
down to 1e-2 (1%). This was the tolerance I originally intended for them. We didn't catch them in an earlier round of testing. They cropped up here because float32 support is handled a little different on my new mac.Not sure why some of my older commits from Aug 6 & 7 appear in the commit log. They were already added to main in PR #38.