Skip to content
This repository was archived by the owner on Dec 10, 2019. It is now read-only.

Add convenience functions and example notebook for datasource CRUD #67

Merged

Conversation

aaronxsu
Copy link
Member

This PR:

  • adds some convenience functions for datasources' CRUD
  • adds an example notebook for it

How to test:

  • Point to spec in here https://raw.githubusercontent.com/raster-foundry/raster-foundry-api-spec/feature/asu/update-first-class-object-crud-spec/spec/spec.yml
  • Run the example notebook

This is the first part of #53

Copy link
Contributor

@jisantuc jisantuc left a comment

Choose a reason for hiding this comment

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

What do I need to set my host to when I create the client? localhost:9091 is throwing a swagger validation error, and I need to point to local for the changes in raster-foundry/raster-foundry#4105

"outputs": [],
"source": [
"import sys\n",
"sys.path.append('/path/to/raster-foundry-python-client')"
Copy link
Contributor

Choose a reason for hiding this comment

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

For user's who've pip installed, this won't be necessary, since pip will have put the library in their path. For users in development, if they've done scripts/update and are in the resulting virtualenv, it also won't be necessary, since that script installs rasterfoundry into the virtualenv. So this cell can be removed.

@aaronxsu
Copy link
Member Author

hey @jisantuc , api = API(refresh_token=refresh_token, host='localhost:9091', scheme='http') has been working for me. Please let me know if it still does not work.

Copy link
Contributor

@jisantuc jisantuc left a comment

Choose a reason for hiding this comment

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

Looks good 🎉

@aaronxsu aaronxsu merged commit b4f381c into develop Sep 27, 2018
@aaronxsu aaronxsu deleted the feature/asu/add-convenience-funcs-and-example-notebooks branch September 27, 2018 17:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants