-
Notifications
You must be signed in to change notification settings - Fork 15
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
Conversation
RC v2.6.3
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.
Read The Docs update
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.
IGRF-14 and C 2.7 update
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.
Pull Request Test Coverage Report for Build 12520790390Details
💛 - Coveralls |
@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. |
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: EDIT: just realized this is coming from |
There was a problem hiding this 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.
_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') |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
[aliases] | ||
release = register clean --all sdist | ||
version = 2.7.0 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
to not work as expected)
How Has This Been Tested?
Ran unit tests locally after updating them for new behaviour.
Test Configuration:
Checklist:
Changelog.rst
, summarising the changesAUTHORS.rst
and.zenodo.json