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

Doc improvement suggestions, mostly Raster #372

Open
dshean opened this issue Jan 26, 2024 · 0 comments
Open

Doc improvement suggestions, mostly Raster #372

dshean opened this issue Jan 26, 2024 · 0 comments

Comments

@dshean
Copy link
Member

dshean commented Jan 26, 2024

Just reviewing, https://slideruleearth.io/web/rtd/user_guide/GeoRaster.html and jotting down some notes for discussion and potential improvement.

First of all, Is there a reason this section is called is "GeoRaster" instead of "Raster support"?

The Asset directory on the raster page includes both point cloud and raster sources. I think this table should only include raster datasets, or should point to a more centralized table. We have a separate table here: https://slideruleearth.io/web/rtd/user_guide/SlideRule.html#assets.

Also, some links to external data sources from the table would be useful, so users could read more about what they're sampling. Though this does require more maintenace to avoid dead links.

The sampling doc (including the example) could use some review. We might show a more generic form for the sampling request with appropriate syntax highlight to show required vs. optional user input...

"samples" : {
"output_column_name1": {"asset": "[asset_name_from_table]", [sampling_parameters]}, 
"output_column_name2": {"asset": "[asset_name_from_table]", [sampling_parameters]},
...}

Is there a reason the raster sampling parameters are split off in the core module doc? https://slideruleearth.io/web/rtd/user_guide/SlideRule.html#raster-sampling. This Sampling Parameters should at least have a section header here, so users don't miss it.

I wonder if it might make sense for us to define a default output column name and default sampling parameters for each asset (based on expected science use case for each dataset), so new users could just specify the desired asset to sample and get results.

It would be useful to have a dedicated subsection of the Raster doc for the ArcticDEM/REMA products, like HLS section, explaining nuances of mosaic vs. strips and what to expect. I think several of the raster datasets listed in the asset table require some notes (and potentially recommended/default sampling parameters), so maybe we just have a section toward the end of the Raster doc for "Notes for Specific Raster Assets" or something with subheadings for each dataset, instead of "Special Case"

Also, I'm not sure GeoParquet deserves a top-level section in our User's guide. This could go under a more generic "Supported Formats" section or something. This is in line with discussion of our last call around recommendations for reproducibility and documenting I/O options.

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

1 participant