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

Add support for --bounds argument to clip AOI #46

Open
j9ac9k opened this issue Dec 13, 2022 · 3 comments
Open

Add support for --bounds argument to clip AOI #46

j9ac9k opened this issue Dec 13, 2022 · 3 comments
Assignees

Comments

@j9ac9k
Copy link
Collaborator

j9ac9k commented Dec 13, 2022

At present, codem will attempt to register an AOI to a foundation dataset, and attempt to identify key-point matches over the entirety of both inputs. This can lead to poor results when the AOI is of a significant area with many features. To address this issue, we propose adding a --bounds argument that would added, where the contents could be relayed to the respective pdal reader.

@j9ac9k j9ac9k self-assigned this Dec 13, 2022
@hobu
Copy link
Collaborator

hobu commented Dec 13, 2022

  • Add --bounds as a GeoJSON bounds
  • Add --polygon as a GeoJSON polygon
  • Add --ogr as an OGR VRT that users could control to select polygon(s) from readable datasources like shapefiles or whatever.

@j9ac9k
Copy link
Collaborator Author

j9ac9k commented Mar 10, 2023

@willforker also brought up an idea of supporting shapefiles (so bounds can be drawn from inside ArcGIS).

@j9ac9k
Copy link
Collaborator Author

j9ac9k commented Mar 27, 2023

Proposed Logic

    graph TD;
        B{Fnd CRS Defined?} -->|No| C(Prompt user for approximate <br>bounding box foundation <br>as background)
        B -->|Yes| G{AOI CRS Defined?}
        C --> E(Clip Foundation to Boudning Box +/- 20%)
        G -->|No| C
        G -->|Yes| H{Same As Foundation?}
        H -->|No| I{Transformation Exists or <br> Enough Known Info}
        I -->|No| C
        H -->|Yes| J(Compute Bounding Box <br> as Floating Dataset <br> +/- 50%)
        I -->|Yes| J
        J --> K(Clip Foundation <br>to Bounding Box)
Loading

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

2 participants