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

Update usage and schema documentation #70

Merged
merged 13 commits into from
Jun 25, 2024
Merged

Update usage and schema documentation #70

merged 13 commits into from
Jun 25, 2024

Conversation

kylebarron
Copy link
Collaborator

@kylebarron kylebarron commented Jun 24, 2024

Change list

  • Allow pystac.Item input wherever dict is allowed
  • Add doc page on schema considerations
  • Add usage doc page with high-level considerations for converting between JSON, dicts, Arrow, Parquet, Delta Lake

@kylebarron
Copy link
Collaborator Author

I published the docs (triggered docs CI manually) from this branch so we can view the rendered docs. We can delete the 0.5.1 version from the docs when we publish the 0.6 release version

docs/usage.md Outdated Show resolved Hide resolved

Except for the [legacy API](api/legacy.md), [Apache Arrow](https://arrow.apache.org/) is used as the in-memory interchange format between all formats. While some end-to-end helper functions are provided, the user can go through Arrow objects for maximal flexibility in the conversion process.

All functionality that goes through Arrow is currently exported via the `stac_geoparquet.arrow` namespace.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think that an example here (or on the index page?) from STAC -> geoparquet would be valuable.

We have the one currently at https://github.com/stac-utils/stac-geoparquet?tab=readme-ov-file#usage. Even an example using a simple pystac.Item() built locally would be good to have.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I added a simple example below

docs/usage.md Outdated Show resolved Hide resolved
stac_geoparquet/arrow/_batch.py Show resolved Hide resolved
docs/usage.md Outdated Show resolved Hide resolved
stac_geoparquet/arrow/_batch.py Outdated Show resolved Hide resolved
@kylebarron kylebarron merged commit cf07d13 into main Jun 25, 2024
1 check passed
@kylebarron kylebarron deleted the kyle/docs-usage branch June 25, 2024 15:54
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.

3 participants