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

Get back the sites that are indexed to a comID #14

Closed
jkreft-usgs opened this issue Jun 10, 2016 · 8 comments
Closed

Get back the sites that are indexed to a comID #14

jkreft-usgs opened this issue Jun 10, 2016 · 8 comments
Milestone

Comments

@jkreft-usgs
Copy link

right now there is basically a bare json response here:
http://cida.usgs.gov/nldi/comid/23382345/

it would be valuable to be able to see the sites that are linked to that comID.

@dblodgett-usgs dblodgett-usgs added this to the V milestone Jun 19, 2020
@kkehl-usgs
Copy link
Contributor

Does "linked" mean we should upstream to find the next upstream site, and look downstream to find the next downstream site -- and then show those like next and previous? Or does "linked" mean something else?

@dblodgett-usgs
Copy link
Member

It just means those that are indexed to that COMID locally.

I could see doing a new path like:
https://labs.waterdata.usgs.gov/api/nldi/linked-data/comid/23382345/linked-features that would give back a geoJSON representation of all the features linked to that comid that would use the same schema as the result of navigation. e.g. https://labs.waterdata.usgs.gov/api/nldi/linked-data/comid/23382345/navigate/DM/nwissite but from all sources and only sites that are linked to the local catchment .

@jkreft-usgs
Copy link
Author

I think we should punt on this for the moment. There is not an immediate use case.

@dblodgett-usgs
Copy link
Member

I don't disagree. It does seem like a generally useful feature to discover other nearby features, but it's not critical. @kkehl-usgs have you gotten into it yet? If you have, and it's not going to be much more work, maybe we finish it, but otherwise, let's set this aside.

@kkehl-usgs
Copy link
Contributor

I started looking at it a bit yesterday. It looks like 2-3 days work. I'll hold off, then.

@dblodgett-usgs dblodgett-usgs removed this from the V milestone Aug 5, 2020
@dblodgett-usgs dblodgett-usgs added this to the Summer 2021 milestone Jun 22, 2021
@dblodgett-usgs
Copy link
Member

I want to put this in the context of a rudimentary html client for a given feature. Will work up a summary issue for that work and link it to this.

@dblodgett-usgs
Copy link
Member

superseded by #20

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

3 participants