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

Orders requests to support source_type for basemaps #838

Open
cholmes opened this issue Jan 23, 2023 · 4 comments
Open

Orders requests to support source_type for basemaps #838

cholmes opened this issue Jan 23, 2023 · 4 comments

Comments

@cholmes
Copy link
Member

cholmes commented Jan 23, 2023

Is your feature request related to a problem? Please describe.

Right now I can create a orders request for basemaps with the CLI, but I can't use planet orders request to help me form the request. I believe the situation is similar in the Python side - no 'helper' method.

Describe the solution you'd like

planet orders request --source-type basemaps or something similar, where it works with the relevant parameters. Likely will take a more concrete design proposal for exactly how it'll work, as it's a little bit different than standard catalog orders.

Describe alternatives you've considered

Writing JSON by hand. It works, but it doesn't give the same level of convenience as the rest of the CLI/API.

Additional context

https://developers.planet.com/apis/orders/basemaps/ is our new documentation on basemaps. It'd be awesome if we could eventually upgrade those docs to be CLI or at least Python SDK calls instead of raw python requests stuff.

@cholmes cholmes modified the milestones: 2.0b2: Iterative improvements, 2.0b3 Jan 23, 2023
@joferkington
Copy link
Contributor

joferkington commented Jan 25, 2023

It's also worth explicitly mentioning that v2 dropped all support for the basemaps API (v1 had full support for basemaps). Orders does not replace the basemaps API and basemaps API functionality and metadata is still needed to use basemap data (e.g. listing a time series and otherwise getting the information you need to use the orders API).

Similarly, NICFI customers cannot use the orders API for basemaps and must use the basemaps API for data download.

Orders support for basemaps in the python client is not sufficient to actually work with basemap data. This client still needs to support the basemap API for basic information.

@jreiberkyle jreiberkyle removed this from the 2.0b3 milestone Mar 13, 2023
@jreiberkyle jreiberkyle added this to the 2.0.1: Major improvements milestone Mar 29, 2023
@sgillies sgillies removed this from the 2.1: Major improvements milestone Jul 14, 2023
@tbarsballe
Copy link
Contributor

Duplicate of #752

@tbarsballe tbarsballe marked this as a duplicate of #752 Aug 22, 2024
@tbarsballe tbarsballe closed this as not planned Won't fix, can't repro, duplicate, stale Aug 22, 2024
@cholmes
Copy link
Member Author

cholmes commented Aug 22, 2024

Duplicate of #752

This is actually different from that one - this is for the planet orders request command to include source_type as an option, the other one is for planet orders list to support source_type as an input - so you can actually see your basemaps orders.

@tbarsballe
Copy link
Contributor

Ah, missed that. Reopening

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

5 participants