This PR implements a few changes that make running the test suite with py.test a cinch:

1. Test file names were all changed to test_*.py
2. Test files adjusted to add the package to the PATH (sys.path.append("../bowshock/"))
3. The __init__.py file handles all the import statements

Here are the tweaks in action on a vanilla Precise32 box (though I didn't record ALL of the tests running haha):

This slight adjustment will fix #4.

HOWEVER, I suggest NOT merging this into the master branch just yet. It seems that the dim parameter causes some kind of error in the API endpoint:

$ curl "http://api.data.gov/nasa/planetary/earth/imagery?lon=100.75&lat=1.5&date=2015-07-04&cloud_score=True&api_key=DEMO_KEY"
{"date": "2015-06-15T03:28:16", "url": "https://earthengine.googleapis.com/api/thumb?thumbid=4c3e852191f53f5653732b91feae7efd&token=d9967e5c873d725ba22f4314728a63ab", "cloud_score": 1.0, "id": "LC8_L1T_TOA/LC81270592015166LGN00"}

Compare the nice output from above to the nasty output below (when the dim parameter is included):

$ curl "http://api.data.gov/nasa/planetary/earth/imagery?lon=100.75&lat=1.5&dim=0.025&date=2015-07-04&cloud_score=True&api_key=DEMO_KEY"
{"message": "Admins have been notified.", "error": "unsupported operand type(s) for /: 'unicode' and 'int'"}

Argh -- that does not look good!

On the NASA API page, they list Jason Duley ([email protected]) as the contact. Should we reach out to Jason to see if we have uncovered a bug in this endpoint?

Cool project!

I wanted to kick the tires a bit, but I'm having a really hard time running the test suite. With Python 3.4.1 (via Anaconda) on Windows 8.1, I get an ImportError on the helpers module.

Figured I was being dumb or it was a Windows problem, so I spun up Vagrant's Precise32 box (Python 2.7.3) to give it a whirl. Unfortunately, that blamed bowshock.

Can you tell what I'm doing wrong?

Thanks! Dan

GitHub changed the way Markdown headings are parsed, so this change fixes it.

See bryant1410/readmesfix for more information.

Tackles bryant1410/readmesfix#1

Small change introduced with this PR. It seems that the NASA documentation for the Earth, APOD, Patents and Earth Temperature Anomalies APIs has changed -- the updated links point to NASA's examples.

This PR implements a few changes that make running the test suite with py.test a cinch:

1. Test file names were all changed to test_*.py
2. Test files adjusted to add the package to the PATH (sys.path.append("../bowshock/"))
3. The __init__.py file handles all the import statements

Here are the tweaks in action on a vanilla Precise32 box (though I didn't record ALL of the tests running haha):

It looks like there was a small mistake in the definition of the adress method:

https://github.com/emirozer/bowshock/blob/master/bowshock/temperature_anomalies.py#L12

This results in a 404:

By converting from adress to address (two ds), the server response is as expected (even though I can't quite figure out how to form addresses for success haha):

When using earth.imagery, the dim parameter is not added to the URL even if it is specified.

>>> from bowshock import earth
>>> lon = 100.75
>>> lat = 1.5
>>> dim = 0.025
>>> date = "2015-07-04"
>>> cloud_score = True
>>> r_without_dim = earth.imagery(lon=lon, lat=lat, date=date, cloud_score=cloud_score)
# ... print stuff
>>> r_with_dim = earth.imagery(lon=lon, lat=lat, dim=dim, date=date, cloud_score=cloud_score)
# ... print stuff
>>> r_without_dim.url == r.with_dim.url
True

(here's a screenshot of the above in action)

It looks like this issue comes from a mised += in the earth module:

https://github.com/emirozer/bowshock/blob/master/bowshock/earth.py#L61

Since ingenology/MAAS isn't available anymore, use the more precise Mars InSight Mission weather API. https://mars.nasa.gov/insight/weather/ https://github.com/gablau/mars-insight-weather-node https://mars.nasa.gov/rss/api/?feed=weather&category=insight&feedtype=json&ver=1.0