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

CLI tutorials #642

Merged
merged 35 commits into from
Jul 21, 2022
Merged

CLI tutorials #642

merged 35 commits into from
Jul 21, 2022

Conversation

cholmes
Copy link
Member

@cholmes cholmes commented Jul 20, 2022

This PR contains three tutorial docs.

  • cli-tutorial.md has all the main commands for orders and data API's, and is mostly complete (except for subscriptions, but may just add that when we get convenience methods)
  • cli-plus-tutorial.md has a number of more advanced workflows. There's more work to do, but is maybe half complete, with valuable stuff in there, and warnings on the state of things.
  • cli-concepts.md intends to help educate people a bit on core concepts we use, with links to other resources for them to learn more. Only one real section is fleshed out, but it's still a valuable one, so I'm leaning towards including.

One thing I'm less sure of is if markdown in this directory automatically shows up in the docs? Or do I need to include the files in some table of contents somewhere? Help appreciated on this, and I'll also try to dig in more on it.

Closes #595

@cholmes cholmes added this to the 2.0b1: Core 'MVP' milestone Jul 20, 2022
@cholmes cholmes changed the title Cli tutorial CLI tutorials Jul 20, 2022
Copy link
Contributor

@jreiberkyle jreiberkyle left a comment

Choose a reason for hiding this comment

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

First off, I learned a lot reading through this. Thank you for putting together such a concise and helpful guide. I did notice that our story on the - input is inconsistent. I would like to see if we can get more consistent. Right now I am pretty sure the dash is not required for any required arguments that are JSON. so e.g. orders create doesn't require the dash (ref). data search ([ref](@click.argument("filter", type=types.JSON(), default="-", required=False))) and data search-create don't require the dash either.

awesome kepler.gl library from the commandline. Useful for visualization of large amounts of
geojson.
* **[GeoJSON.io](https://geojson.io/)** - Simple tool to do editing of geojson, useful for creating
AOI's. Integrates with github, but doesn't seem to work so well.
Copy link
Contributor

Choose a reason for hiding this comment

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

This statement makes me curious. In what ways does it not seem to work well?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, it's the integration with github that's not working well. I'll clarify it. If I hit 'save' on geojson.io (like to gist or to github) I get an authentication failure, and can't seem to actually hook up my github account. This used to work before. If it worked then geojson.io would be a lot better for the command-line flows. I kinda work around it with the gh cli usage. Curious if you're able to 'save' - if I just got into some weird state, or if it's basically broken for everyone.


The 'permission filter' is set to true by default, since most people want to search only for data they have access to
and are able to download. But if you'd like to just get search Planet's catalog and get a sense of what is out there
you can set the permission filter to false:
Copy link
Contributor

Choose a reason for hiding this comment

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

how does this work? They have to have an api key to do anything, right? And if they have an api key then what is the limitation on their download access?

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe I have this wrong? Yes, they need an API key, but I was thinking if 'permission filter' was set to true then they'd only see data api results in the AOI that they have purchased access to. And if they wanted to search outside then they'd need to set it to false?

Though now I'm questioning - how does the permission filter work? What is the effect of setting it to true or false?

This is one of those times it'd be nice to easily test the non-planeteer experience.

Copy link
Contributor

Choose a reason for hiding this comment

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

hmm yeah I agree it would be helpful to be able to test different scenarios. @sarasafavi is there any way for devrel to get a set of tester API keys?

```

Or get the 5 latest cloud free images in an area and create an order that clips to that area, using
[geometry.geojson](https://gist.github.com/cholmes/378d050a263ae433ddbbb91c3439994b) from above:
Copy link
Contributor

Choose a reason for hiding this comment

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

would be cool to host this on the sdk repo

Copy link
Member Author

Choose a reason for hiding this comment

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

Would be happy to, what would the advantage be?

Copy link
Contributor

Choose a reason for hiding this comment

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

just seems more permanent and maybe professional than your personal GH account. Like having an office number with a secretary that answers instead of a cellphone? IDK 😆

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm halfway done with this, but it'll be better when I merge to main, so I'm going to just do another PR once this one is merged to finish it off.

cholmes and others added 3 commits July 20, 2022 07:11
Co-authored-by: Jennifer Reiber Kyle <[email protected]>
Co-authored-by: Jennifer Reiber Kyle <[email protected]>
Co-authored-by: Jennifer Reiber Kyle <[email protected]>
@cholmes
Copy link
Member Author

cholmes commented Jul 21, 2022

Merging this in - future updates will just be done as new PR's

@cholmes cholmes merged commit 12a7dbc into main Jul 21, 2022
@cholmes cholmes deleted the cli-tutorial branch July 21, 2022 21:44
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.

Add tutorials for CLI
2 participants