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 index.qmd #113

Closed
wants to merge 2 commits into from
Closed

Update index.qmd #113

wants to merge 2 commits into from

Conversation

hfu
Copy link

@hfu hfu commented Apr 9, 2024

Fix typo. COPS -> COPC.

(Hi! Many thanks for your great guide!)

wildintellect and others added 2 commits April 8, 2024 08:19
* Clarify text about cloud opt allowing partial reads (cloudnativegeo#102)

* Clean up imports

* Add back newline

* Adding writing COGs with Python notebook (cloudnativegeo#106)

* Adding writing COGs with Python notebook

* Addressing review comments

* Add overview generation and set NoData values

* Add description for overviews and predictors

* update nodata value to -32768.0

* add in-text citation

* update navigation link

* Adding link to Project  Pythia Kerchunk Cookbook (cloudnativegeo#108)

* Clarify text about cloud opt allowing partial reads (cloudnativegeo#102) (cloudnativegeo#103)

* adding ref to kerchunk cookbook

* Add notebook for LAS to COPC Conversion (cloudnativegeo#109)

* Add notebook for LAS to COPC Conversion

* Adding cli based access info

* Addressing review comments

* update navigation link for COPC notebook

* Minor updates for consistency in COPC full form

* Update environment file

* update pdal reader for copc

* update copc:true check for validation

---------

Co-authored-by: j08lue <[email protected]>
Co-authored-by: Kyle Barron <[email protected]>
Co-authored-by: Rajat Shinde <[email protected]>
Co-authored-by: rsignell <[email protected]>
Fix typo. COPS -> COPC.
Copy link
Collaborator

@j08lue j08lue left a comment

Choose a reason for hiding this comment

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

Thanks @hfu!

@kylebarron kylebarron changed the base branch from main to staging April 10, 2024 14:24
@kylebarron kylebarron mentioned this pull request Apr 10, 2024
@kylebarron
Copy link
Member

Thanks!

@hfu it's slightly confusing but we need PRs to be made against staging, not main. I switched the branch but now it's including an extra commit. Since this is a small typo fix, I made a new pr in #114 so that you don't have to go back and fix the commit history.

@hfu
Copy link
Author

hfu commented Apr 11, 2024

Thank you @kylebarron for guidance and including the fix to the correct branch!

Let me close this issue with thanks.

@hfu hfu closed this Apr 11, 2024
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.

4 participants