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

Feature/Issue 376 Update EnigmaReader Data Connector #380

Conversation

hydrosquall
Copy link
Contributor

Implements #376, and includes

  • Updating EnigmaReader
  • Updating tests
  • Updating docs
  • Updating the main data.py library.

The exact sunsetting date of app.enigma.io is not determined, but is coming soon.

@hydrosquall hydrosquall force-pushed the feature/Issue-376-update-enigma-connector branch from e21897b to a034f31 Compare August 5, 2017 18:43
@hydrosquall hydrosquall force-pushed the feature/Issue-376-update-enigma-connector branch from a034f31 to f79ef9f Compare August 5, 2017 19:01
@hydrosquall
Copy link
Contributor Author

@jreback , this is ready for review :)

@@ -202,15 +202,18 @@ Available expiry dates can be accessed from the ``expiry_dates`` property.
Enigma
======

Access datasets from `Enigma <https://app.enigma.io>`__,
Copy link
Contributor

Choose a reason for hiding this comment

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

add a note/warning that this changed in 0.6.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated on line 207

the world's largest repository of structured public data.

Datasets are unique identified by the ``uuid4`` at the end of a dataset's web address.
For example, the following code downloads USDA Food Recall Data from https://public.enigma.com/datasets/292129b0-1275-44c8-a6a3-2a0881f24fe1.
Copy link
Contributor

Choose a reason for hiding this comment

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

your https won't render, do like: `USDA Food Recal Data <https:.......>`__

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done :)!

@hydrosquall
Copy link
Contributor Author

@jreback , changes are complete! Let me know if you need anything else tweaked

@gliptak
Copy link
Contributor

gliptak commented Sep 8, 2017

@jreback Would you like to see other updates to this PR? Thanks

@hydrosquall
Copy link
Contributor Author

@davidastephens - would it be OK to role this PR into the next release?

@hydrosquall
Copy link
Contributor Author

Update: app.enigma.io has just been sunset, and points to public.enigma.com instead. As a result, the old EnigmaDatareader has stopped working, but the new EnigmaDatareader in this PR fixes that.

@davidastephens davidastephens merged commit 87dffc8 into pydata:master Oct 22, 2017
@davidastephens
Copy link
Member

Thanks @hydrosquall

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

4 participants