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

Release Candidate 0.0.4 #54

Merged
merged 115 commits into from
May 21, 2021
Merged

Release Candidate 0.0.4 #54

merged 115 commits into from
May 21, 2021

Conversation

aburrell
Copy link
Member

@aburrell aburrell commented May 11, 2021

Description

Alpha release fully compatible with pysat 3.0.0+.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

How Has This Been Tested?

This has been tested by running examples and unit tests locally

Test Configuration

  • Operating system: OS X Mojave
  • Version number: Python 3.7
  • Any details about your local setup that are relevant: tested with pysat 3.0.0 and develop

Checklist:

  • Update the version number
  • Verify and update Zenodo information in .zeondo.json
  • doc tests pass locally
  • activate this branch on readthedocs (https://pysatspaceweather.readthedocs.io/en/rc_v0.0.4)
  • Upload the test distribution to the testPyPi server: https://test.pypi.org/project/pysatSpaceWeather/0.0.4/
  • Contacted developers and users about the future release
  • Performed integration tests with downstream packages (include list of these packages)
  • Create a wiki page with the release text (https://github.com/pysat/pysatSpaceWeather/wiki/v0.0.4)
  • After approval, confirm all changes from the RC pull are listed in the wiki release text.
  • 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.md, summarizing the changes

If this is a release PR, replace the first item of the above checklist with the
release checklist on the pysat wiki:
https://github.com/pysat/pysat/wiki/Checklist-for-Release

aburrell and others added 30 commits January 13, 2021 12:19
Reworded comment to be more accurate to current custom functionality.
Reorganized functions to not include functions that cannot be used as custom functions from a block of common custom functions.
ENH: Update to latest pysat
Updated the pysat branch used for Travis CI.
Update travis after pysat release.  Also added use of `before_install` for environment set up.
Updated tested python versions, removing support for 3.6 and adding support for 3.9.
Added a yaml file for GitHub actions.
Removed tests performed by GitHub actions.  Also simplified installation, just using pip and not conda.
Updated changelog, adding a section for the upcoming release.
Made PEP8 improvements.
Added a flake8 ignore flag for line breaks before operators.
Updated links that were being redirected.
Added summary of recent commits to the changelog.
Remove coveralls call from Travis, since this has been moved to GitHub actions.
Attempting to fix the GitHub actions coveralls call.
Replaced tab with spaces.
Fixed coveralls call by fixing pytest command and reverting to old style.  Simplified flake8 calls by separating PEP8 requirements and complexity calls.
Removed extra lines from GitHub actionos yaml.
Updated pysat in Travis CI and implemented GitHub actions for most tests.
aburrell and others added 2 commits May 17, 2021 08:46
Updated Dst tag to be informative, and easily allow for future additions of historic data from Kyoto.
Copy link
Collaborator

@rstoneback rstoneback left a comment

Choose a reason for hiding this comment

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

Still in progress but I have an outside appointment. Back mid-late afternoon.

pysatSpaceWeather/instruments/sw_f107.py Show resolved Hide resolved
docs/examples/ex_init.rst Show resolved Hide resolved
pysatSpaceWeather/instruments/ace_epam.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@rstoneback rstoneback left a comment

Choose a reason for hiding this comment

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

Sorry for the multiple mini-reviews.

pysatSpaceWeather/instruments/ace_epam.py Outdated Show resolved Hide resolved
pysatSpaceWeather/instruments/ace_epam.py Outdated Show resolved Hide resolved
pysatSpaceWeather/instruments/ace_epam.py Outdated Show resolved Hide resolved
Simplified code, removing unnecessary loops.

Co-authored-by: Russell Stoneback <rstoneba@utdallas.edu>
@aburrell aburrell requested a review from rstoneback May 18, 2021 12:05
aburrell and others added 3 commits May 18, 2021 08:07
Fixed the metadata re-assignment of the fill value.
If the requested file is not present on the server, raise a warning instead of an error.
Copy link
Collaborator

@rstoneback rstoneback left a comment

Choose a reason for hiding this comment

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

Thanks @aburrell. I did my best to be thorough and check out each of the supported datasets.

pysatSpaceWeather/instruments/ace_epam.py Show resolved Hide resolved
pysatSpaceWeather/instruments/ace_mag.py Show resolved Hide resolved
pysatSpaceWeather/instruments/ace_sis.py Show resolved Hide resolved
pysatSpaceWeather/instruments/ace_swepam.py Show resolved Hide resolved
pysatSpaceWeather/instruments/methods/ace.py Show resolved Hide resolved
pysatSpaceWeather/instruments/sw_f107.py Show resolved Hide resolved
pysatSpaceWeather/instruments/sw_f107.py Show resolved Hide resolved
pysatSpaceWeather/instruments/sw_f107.py Show resolved Hide resolved
pysatSpaceWeather/instruments/sw_kp.py Show resolved Hide resolved
pysatSpaceWeather/instruments/sw_kp.py Show resolved Hide resolved
Created a general preprocess routine to replace specific fill values with NaN.
Removed preprocess routine now in general.
Improved the examples and warnings in the F10.7 docstrings and applied the general preprocessing routine.
Improved the examples and warnings in the Kp docstrings and applied the general preprocessing routine.
Improved the examples and warnings in the ACE docstrings and applied the general preprocessing routine.
@aburrell aburrell requested a review from rstoneback May 19, 2021 12:02
rstoneback
rstoneback previously approved these changes May 20, 2021
Copy link
Member

@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.

Looks good! tested out the ace instruments locally, changes are working.

Fixed float check to identify both numpy floats and normal floats.
@aburrell aburrell requested a review from rstoneback May 20, 2021 13:05
@aburrell
Copy link
Member Author

Re-requesting a review from @rstoneback since I changed his code lines.

@aburrell aburrell dismissed rstoneback’s stale review May 21, 2021 11:45

Changed the code added by @rstoneback, so want his eye on the update.

Copy link
Collaborator

@rstoneback rstoneback left a comment

Choose a reason for hiding this comment

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

Looks good to me. I should have tested the statement that checking against np.floating worked for all float cases better.

@aburrell aburrell merged commit b403a14 into main May 21, 2021
@aburrell aburrell deleted the rc_v0.0.4 branch May 21, 2021 17:06
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.

None yet

3 participants