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

BUG: Revert integration to GitHub service for coveralls on MacOS #237

Merged
merged 12 commits into from
Aug 5, 2024
17 changes: 3 additions & 14 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,23 +52,19 @@ jobs:
python setup.py develop
cd ../pysatNASA
pip install .

- name: Install NEP29 dependencies
if: ${{ matrix.test_config == 'NEP29'}}
run: |
pip install numpy==${{ matrix.numpy_ver }}
pip install --upgrade-strategy only-if-needed .[test]

- name: Install standard dependencies
if: ${{ matrix.test_config == 'latest'}}
run: |
pip install .[test]

- name: Set up pysat
run: |
mkdir pysatData
python -c "import pysat; pysat.params['data_dirs'] = 'pysatData'"

- name: Test PEP8 compliance
run: flake8 . --count --select=D,E,F,H,W --show-source --statistics

Expand All @@ -86,7 +82,6 @@ jobs:
run: |
curl -sL https://coveralls.io/coveralls-linux.tar.gz | tar -xz
./coveralls report --parallel --repo-token=${{ secrets.COVERALLS_REPO_TOKEN }} --build-number ${{ github.run_number }}

- name: Coveralls Parallel (Windows)
if: startsWith(matrix.os, 'windows')
env:
Expand All @@ -95,19 +90,13 @@ jobs:
run: |
curl -L https://github.com/coverallsapp/coverage-reporter/releases/latest/download/coveralls-windows.exe -o coveralls.exe
./coveralls.exe report --parallel --repo-token=${{ secrets.COVERALLS_REPO_TOKEN }} --build-number ${{ github.run_number }}

- name: Coveralls Parallel (MacOS)
- name: Publish results to coveralls (MacOS)
if: startsWith(matrix.os, 'macos')
env:
COVERALLS_REPO_TOKEN: ${{ secrets.COVERALLS_REPO_TOKEN }}
COVERALLS_PARALLEL: true
run: |
brew tap coverallsapp/coveralls --quiet
brew install coveralls --quiet
ls -lh .coverage
ls -lh coverage.xml
coveralls report coverage.xml --format=cobertura --verbose --parallel --repo-token=${{ secrets.COVERALLS_REPO_TOKEN }} --build-number ${{ github.run_number }} --debug --dry-run

coveralls --rcfile=pyproject.toml --service=github
finish:
name: Finish Coverage Analysis
needs: build
Expand All @@ -120,4 +109,4 @@ jobs:
COVERALLS_PARALLEL: true
run: |
curl -sL https://coveralls.io/coveralls-linux.tar.gz | tar -xz
./coveralls done --build-number ${{ github.run_number }}
./coveralls done --build-number ${{ github.run_number }}
5 changes: 5 additions & 0 deletions .zenodo.json
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,11 @@
{
"name": "Spence, Carey",
"orcid": "0000-0001-8340-5625"
},
{
"affiliation": "Universities Space Research Association, Goddard Space Flight Center",
"name": "Govada, Aadarsh",
"orcid": "0009-0004-7873-5899"
}
]
}
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,13 @@ This project adheres to [Semantic Versioning](https://semver.org/).
* Allow graceful failure with no files in jhuapl load functions
* New window needs to be integer for calculate_imf_steadiness
* Fixed a bug where cdas_download may drop the requested end date file
* Reverted the coveralls integration to the GitHub service for MacOS runs
* Fixed a bug where cdas_list_remote_files errored without remote data
* Documentation
* Added example of how to export data for archival
* Updated documentation refs
* Add keywords to zenodo
* Fixed broken links
* Deprecations
* Deprecated '' tag for de2_vefi module, support moved to de2_vefimagb
* Maintenance
Expand Down
7 changes: 4 additions & 3 deletions pysatNASA/instruments/cnofs_vefi.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,10 @@
spacecraft is to enable an accurate V x B measurement along the spacecraft
trajectory. In order to provide an in-flight calibration of the magnetic field
data, we compare the most recent POMME model (the POtsdam Magnetic Model of the
Earth, https://geomag.us/models/pomme5.html) with the actual magnetometer
measurements to help determine a set of calibration parameters for the gains,
offsets, and non-orthogonality matrix of the sensor axes. The calibrated
Earth, https://geomag.colorado.edu/pomme-5-magnetic-model-of-the-earth.html)
with the actual magnetometer measurements to help determine a set of calibration
parameters for the gains, offsets, and non-orthogonality matrix of the sensor
axes. The calibrated
magnetic field measurements are provided in the data file here. The VEFI
magnetic field data file currently contains the following variables:
B_north Magnetic field in the north direction
Expand Down
Loading