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

RC V2.7.0 #93

Merged
merged 52 commits into from
Dec 31, 2024
Merged

RC V2.7.0 #93

merged 52 commits into from
Dec 31, 2024

Conversation

aburrell
Copy link
Owner

@aburrell aburrell commented Dec 27, 2024

Description

This is a release that updates the coefficients for IGRF-14 and also contains the new version of the C code, 2.7. Deprecated functions have been removed.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality
    to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Ran unit tests locally after updating them for new behaviour.

Test Configuration:

  • Operating system: OS X Ventura
  • Version number: Python 3.10
  • Any details about your local setup that are relevant: gcc 14.2.0

Checklist:

  • Update the version number
  • Verify and update Zenodo information in .zenodo.json
  • doc tests pass locally
  • activate this branch on readthedocs (see CI tests below)
  • Upload the test distribution to the testPyPi server: https://test.pypi.org/project/aacgmv2/2.7.0rc1/
  • Contacted developers and users about the future release
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • Add a note to Changelog.rst, summarising the changes
  • Add yourself to AUTHORS.rst and .zenodo.json

aburrell and others added 30 commits February 8, 2023 10:51
Added a RTD yaml file at the default location.
Moved all the doc requirements to the pyproject.toml file.
Added a summary of the changes to the changelog.
Fixed a bug where the old requirements file was used in the docs test.  Updated main test to import test requirements from pyproject.toml.
Updated call to napoleon sphinx extension.
Removed an extra period, introduced as a typo.
Added a version limit for the sphinx RTD theme.
Updated the AACGMV2 coefficient files for IGRF 14.
Updated the IGRF coefficient file to IGRF-14.
Transitioned more parts of the setup.cfg file into the pyproject.toml file.  Also updated the supported Python versions.
Updated the CI tests to:
- cover new Python versions,
- update the coveralls execution, and
- use the pyproject.toml for installation and reference.
Updated the Python version used in the documentation tests.
Updated the magmodel coefficient file.
Updated the init file to use importlib packages, get the version dynamically from the pyproject.toml, and change the IGRF coefficient file back to the magmodel file.
Updated the C source documentation: the README and release notes.
Updated the AACGMV2 headers for version 2.7.
Updated the C code to version 2.7.
Updated the README to reflect the new C version.
Removed the deprecated source file (routine now lives in rtime.c).
Removed bad lines and formatting from pyproject.toml.
Update the list of source files in the setup.py file.
Added instructions for updating the Python code for IGRF and new C versions.
Updated the documentation configuration file to use the pyproject.toml for version information.
Updated the supported Python versions in the docs.
Update the test values for the new AACGMV2 coefficients.
Update the documentation to include a list of expected tests that need to be updated with new coefficients.
Updated the changelog with changes, a new version  number, and a target release date.
Added a missing package for documentation configuration.
Updated my email in the code of conduct.
Fixed broken links and references.
Fixed the sublist formatting for the list of tests that need to be updated.
More fixes to the docstring style.
Remove an old keyword from the coverage specifications.
Added init files for the unit tests.
Be extra careful about imports when testing the environment.
Added a more detailed error message to a test.
Updated the module structure after adding init files.
Removed pytest flags that are no longer necessary.
Updated test logic to handle the different MacOS python support options.
Added the numpy version to the run names.
Fixed a typo in the numpy version variable name.
Removed the module containing the deprecated functions.
Removed the tests for deprecated functions and updated the structure tests.  Fixed bugs in the structure test docstrings.
Updated the documentation index depth.
Update the changelog to include changes made here.
Remove deprecated functions
Updated the version in preparation for release.
@aburrell aburrell added this to the v2.7.0 milestone Dec 27, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 12520790390

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.1%) to 93.73%

Totals Coverage Status
Change from base Build 4125745284: 1.1%
Covered Lines: 1095
Relevant Lines: 1165

💛 - Coveralls

@aburrell
Copy link
Owner Author

@dchaddock, @MITHaystack, @JouleCai, @indiajacksonphd, @sapols, @KCollins, @dchaddock, @mattkjames7, @mshumko, @HamSCI, @jklenzing, @aalekseevx, and @zemarchezi. Each of you have actively developed projects that depend on the most recent version of AACGMV2. If you would like, here's the release candidate you can test out. If you have any issues, feel free to let me know.

I am going to try and get this out before the new year to prevent any issues with realtime processing in AAGCMV2 coordinates due to the IGRF-14 update.

@jklenzing
Copy link

jklenzing commented Dec 30, 2024

Updated my RC tests in github actions here. Tests are successfully running with the RC as a dependency, however I am getting the following warning from aacgmv2: DeprecationWarning: path is deprecated. Use files() instead. Refer to https://importlib-resources.readthedocs.io/en/latest/using.html#migrating-from-legacy for migration advice. Not sure if this is expected in this release.

EDIT: just realized this is coming from importlib-resources when aacgmv2 is imported.

Copy link

@jklenzing jklenzing left a comment

Choose a reason for hiding this comment

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

Overall looks good. A few minor comments on things that could just be written up as issues for a future release.

Comment on lines +52 to +58
_os.path.split(str(resources.path(__package__,
'__init__.py').__enter__()))[0],
'aacgm_coeffs', 'aacgm_coeffs-14-')
IGRF_COEFFS = _os.path.join(
_os.path.split(str(resources.path(__package__,
'__init__.py').__enter__()))[0],
'magmodel_1590-2025.txt')

Choose a reason for hiding this comment

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

These are the lines that generate the Deprecation Warning. resources.path is replaced by resources.files. Probably worth an issue for the next update.

Copy link
Owner Author

Choose a reason for hiding this comment

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

#94


[aliases]
release = register clean --all sdist
version = 2.7.0

Choose a reason for hiding this comment

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

I think you could drop the duplicate metadata here since python 3.9 is the minimum supported version.

Copy link
Owner Author

Choose a reason for hiding this comment

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

#95

@aburrell aburrell requested a review from jklenzing December 31, 2024 14:27
@aburrell aburrell merged commit 2c25565 into main Dec 31, 2024
48 checks passed
@aburrell aburrell deleted the v2.7.0_rc branch December 31, 2024 15:24
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