-
Notifications
You must be signed in to change notification settings - Fork 7
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
Initial implementation #1
Comments
It might make sense to use (perhaps optionally) AstroPy's http://astropy.readthedocs.org/en/latest/utils/#file-downloads |
This comment contains and example JSON snippet for data driven tests planetarypy/planetaryimage#10 (comment) |
Start off by implementing the simplest {
"1p190678905erp64kcp2600l8c1.img": {
"url": "http://pds-geosciences.wustl.edu/mer/mer1-m-pancam-2-edr-sci-v1/mer1pc_0xxx/data/sol0704/1p190678905erp64kcp2600l8c1.img",
"instrument": "Pancam",
"opens": "True"
},
} Then write the downloader, After the When including |
Follow the
You may use click to implement your CLI if you want, though it may not be necessary at this point since we are not taking any arguments or options yet. |
Done! Looks great! |
The intent of this package is to provide a framework to systematically list and test as many planetary data types as possible. Not only should these products be enumerated, but their test values (e.g. label entries, pixel values) should also be stored in this module.
We have started a list of Missions, Instruments, products and our level of support here:
http://planetaryimage.readthedocs.org/en/latest/supported_planetary_image_types.html
I envision initial implementation of this package to be an extension of the
get_test_data.py
anddata.json
code I wrote forgdal_pds
:We would extend
data.json
to define additional keys to store expected test results and this file would be used as a source for acceptance testing for every mission product supported by PlanetaryPy. Ultimately it is impractical to include mission data in the main testsuite for thePlanetaryPy
software, this module is intended to bridge that gap.The text was updated successfully, but these errors were encountered: