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

Improved cloud-optimized options / prompting #842

Closed
10 tasks done
olsen232 opened this issue May 17, 2023 · 0 comments
Closed
10 tasks done

Improved cloud-optimized options / prompting #842

olsen232 opened this issue May 17, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@olsen232
Copy link
Collaborator

olsen232 commented May 17, 2023

There's more work to do with respect to informing / prompting / warning the user about cloud-optimized vs non-cloud-optimized tiles - both for rasters and point clouds:

Import behaviour:

  1. Import with --convert-to-cog: no warning
  2. Import with --no-convert-to-cog: no warning
  3. Import with no flag and and interactive terminal
    "... Would you like to make a COG dataset?"
  4. Import with no flag no interactive terminal:
    "Run this command again with --convert-to-cog or --preserve-format."

Diff/commit behaviour:

  1. Diff/commit to preserve-format dataset: no warnings are shown.
  2. Diff/commit COG to cloud-optimized dataset: no warnings are shown.
  3. Diff/commit non-COG to convert-to-cog dataset, no flag - shows warning during diff or commit, prevents commit.
    "Adding non-COG files to a cloud-optimized dataset is not allowed. Add --convert-to-dataset-format flag to cloud-optimize the files, or add --no-convert-to-dataset-format to remove the dataset constraints."
  4. Diff/commit non-COG to convert-to-cog dataset, --convert-to-dataset-format: files are automatically converted.
  5. Diff/commit non-COG files to convert-to-cog dataset, --no-convert-to-dataset-format: dataset constraint is removed.

Slightly related - diff/commit behaviour: Wrong LAZ version:

  1. Diff/commit non-COG to convert-to-cog dataset, no flag - shows warning during diff or commit, prevents commit.
    "Adding multiple LAZ versions in the same dataset is not allowed. Add --convert-to-dataset-format flag to automatically convert them to the dataset's LAZ version."
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant