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

Create border polygon #1825

Open
lucadelu opened this issue Feb 18, 2025 · 4 comments · May be fixed by #1828
Open

Create border polygon #1825

lucadelu opened this issue Feb 18, 2025 · 4 comments · May be fixed by #1828

Comments

@lucadelu
Copy link

lucadelu commented Feb 18, 2025

During a course students told me that in CAD system to show correctly orthophoto it is needed a file with a polygon of the orthophoto's bounding box (just the extension). They also told me that proprietary software is creating this polygon and provide it with the image.

I searched a little bit but it looks like there is nothing like that right now, I can implement it, I was thinking to create a new function (and a new option in the cli) and add it after args.orthophoto_cutline,

What do you think?

@pierotofy
Copy link
Member

Is odm_georeferencing/odm_georeferenced_model.bounds.gpkg perhaps what you need?

@lucadelu
Copy link
Author

Is odm_georeferencing/odm_georeferenced_model.bounds.gpkg perhaps what you need?

No, that is the clipping area, I would like to have the square extend of the full tiff.

Image

odm_georeferencing/odm_georeferenced_model.bounds.gpkg is the yellow, instead I would like to have green.

@pierotofy
Copy link
Member

If one wants the extent, then it should be extracted from the GeoTIFF (odm_orthophoto.tif). It could be added as a post-processing step in the odm_orthophoto stage. I don't think we should add more flags to generate it (if it's fast), just generate the file and place it in odm_orthophoto/<file>

Just curious, which proprietary software are we talking about and what's the file format of the extent file? Strange that CAD programs don't handle GeoTIFF automatically.

@lucadelu
Copy link
Author

Image

this is the result of the code into #1828

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants