Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Daniel/git actions py version #165

Merged
merged 46 commits into from
Aug 23, 2024
Merged

Conversation

betaBison
Copy link
Member

  • adds compatibility for Python 3.12 and removes compatibility for Python 3.8
  • updates functions to enable Python 3.12 compatibility
  • large expansion of tutorials and documentation

kanhereashwin and others added 30 commits February 1, 2024 11:32
This import was clashing with the interpolate from glp.navdata.operations causing issues when trying to use glp.interpolate, which incorrectly referred to the scipy.interpolate module instead of the glp.interpolate method
Try to add Python 3.12 support as a new action.
@betaBison betaBison requested a review from danineamati August 5, 2024 02:15
Copy link

codecov bot commented Aug 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.66%. Comparing base (a22343f) to head (aad7449).

Additional details and impacted files
@@            Coverage Diff             @@
##           v1.0.3     #165      +/-   ##
==========================================
- Coverage   99.66%   99.66%   -0.01%     
==========================================
  Files          27       27              
  Lines        3885     3883       -2     
==========================================
- Hits         3872     3870       -2     
  Misses         13       13              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

betaBison and others added 7 commits August 5, 2024 12:17
Revert the python requirements to ensure compatibility with Google Colab and related uses that require numpy 1.x.
numpy 1.x on MAC OS may not raise a singularity flag where Linux and Windows due. This issue appears to be addressed in numpy 2. Either way, this test should now be more in line with the goals of this test case (i.e., either (a) confirming that the matrix is singular and caught appropriately with nans in the DOP entries or (b) near singular even if the flag is not raised and the relevant DOP entries are non-physical).
Remove the test_singularity_dop print statements now that the tests are passing.
Copy link
Member

@danineamati danineamati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All updates are working across platforms and ready to be merged.

@danineamati danineamati merged commit 87428c4 into v1.0.3 Aug 23, 2024
18 checks passed
@betaBison betaBison deleted the daniel/git_actions_py_version branch September 23, 2024 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants