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

Beyond first set of endpoints: related features, related actions, actionby, etc #58

Open
emiliom opened this issue Nov 23, 2017 · 2 comments

Comments

@emiliom
Copy link
Member

emiliom commented Nov 23, 2017

Looks like all the endpoints defined in the REST API design doc and targeted by issues in milestone 1 are now implemented! See @lsetiawan's test deployment at http://54.186.36.247:8001/v1/docs/ to test it out. Don is also working on a Jupyter notebook that demos some of the endpoints, for next week.

One clear need exposed by this initial implementation is the ability to request "related" entities, specially related features (eg, the site sampling feature holding the geospatial information, related to a specimen sampling feature of interest). Here is an example samplingfeaturedatasets request returning a rich response (overly rich, depending on the use case!) that does not include location (x-y coords) information or the ability to obtain it. It also doesn't have the ability to pull out the people and organizations related to the returned actions (ie, ActionBy and related entities).

Anyway, I'm laying this out just as a motivator for the next stage. Below I'll paste and briefly comment on some relevant discussions from PR #54:

(EMILIO) looking at this response and the corresponding endpoint description in the REST API Google doc brings up broader questions about what specifically should (and shouldn't?) be included in the response.

(JEFF) I think the approval of the response content and structure should go in a separate issue. I think all of the endpoints need to be reviewed now that we have initial implementations. And, as your discussions point out, this issue is tied to the Python API because the job of the REST API is to simply serialize the objects that are returned from the Python API.

@horsburgh, I don't think we should constraint ourselves to having the REST API be a 1:1 match to the Python API. They're operating in different settings and with different constraints.

But anyway, that's for the next phase of testing, discussion and dev.

Looping in @aufdenkampe, and @jkreft-usgs (since he's volunteered to provide feedback on the REST API!). Gobble gobble ...

@aufdenkampe
Copy link
Member

@emiliom, thanks for that noting that milestone, providing a status summary and describing the kinds of decisions we now face!

@emiliom
Copy link
Member Author

emiliom commented Nov 28, 2017

All: FYI, @lsetiawan has created a nice Jupyter notebook that flexes the current version of the ODM2 REST API, via the test deployment Don is managing. Give it a look. But beware that github will not render the interactive folium/leaflet map; to see those elements in action, either run the notebook yourself, or open it on nbviewer by going to this link.

Thanks, Don!

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

No branches or pull requests

2 participants