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

Update phoenix-aces data download. #131

Merged
merged 14 commits into from
Mar 14, 2019
Merged

Update phoenix-aces data download. #131

merged 14 commits into from
Mar 14, 2019

Conversation

jason-neal
Copy link
Owner

Use Starfish's downloading function for phoenix-aces spectra, involves caching.

Move download scripts to scripts/download.

Starfish-develop/Starfish#108

Repository owner deleted a comment Mar 12, 2019
Repository owner deleted a comment Mar 12, 2019
@coveralls
Copy link

coveralls commented Mar 12, 2019

Coverage Status

Coverage remained the same at 83.213% when pulling 98f4b8a on ftpphoenix into f000c8d on develop.

Repository owner deleted a comment Mar 13, 2019
Repository owner deleted a comment Mar 13, 2019
Repository owner deleted a comment Mar 13, 2019
Repository owner deleted a comment Mar 13, 2019
Repository owner deleted a comment Mar 13, 2019
Repository owner deleted a comment Mar 13, 2019
@jason-neal jason-neal merged commit 21902eb into develop Mar 14, 2019
@jason-neal jason-neal deleted the ftpphoenix branch March 14, 2019 10:03
jason-neal added a commit that referenced this pull request Apr 7, 2019
* Use http for phoenix data

* Move download scripts to scripts/download.

Create new download script using new Starfish functions.

Cache data on CI services

* Fix starfish #egg

* Fix download_test_aces.py Starfish downloaded takes a iterable of iterables.

* Fix appveyor escape characters in paths \\.

* fix ps_download_eniric_data.ps1

 Add Python/Scripts path to appveryor config.


Former-commit-id: 5d57495b073cfd4296c6aeff9005ba06f3244556 [formerly c342965d6de0c6a646199e615452f6a886817878]
Former-commit-id: 81c66a359e70623314f13ae6b78ab8ad5b5a6de0
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

2 participants