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

--scale flag not compiled in the windows binary #69

Open
cubantonystark opened this issue Apr 30, 2024 · 1 comment
Open

--scale flag not compiled in the windows binary #69

cubantonystark opened this issue Apr 30, 2024 · 1 comment

Comments

@cubantonystark
Copy link


First of all, thank you for taking the time to report an issue.

Before you continue, make sure you are in the right place. Please open an issue only to report faults and bugs. For questions and discussion please open a topic on http://community.opendronemap.org/c/opendronemap.

Please use the format below to report bugs and faults.


What is the problem?

--cale flag not available on windows binary. Throws error when trying to use it as its an unknown flag.

What should be the expected behavior?

Usage of the --scale flag shouldn't throw an error.

How can we reproduce this? THIS IS IMPORTANT. What steps did you do to trigger the problem? If this is an issue with processing a model, YOU MUST include a copy of your model uploaded on Google Drive or Dropbox (otherwise we cannot reproduce this) along with the command line parameters you used to process it.

Run the program with the --scale flag.

@Yologuy101
Copy link

I had the same issue but with the --error flag.

I compiled the source code by using
git clone https://github.com/OpenDroneMap/Obj2Tiles.git
cd Obj2Tiles
dotnet build -c Release

I noticed that all the versions on the releases page are outdated. None of the source codes would compile to have the --scale or --error tag (the readme file is also not updated in the source code and using the --version always shows 1.0.0). Only the current GitHub version contains support for the tags (--version still not correct).

The newest version also requires .NET 7 instead of .Net 6. (I might be wrong about this)

To get the Windows binary to use with ODM CLI and passing --error or --scale flag during the pipe, I did
dotnet publish -c Release -r win-x64 --self-contained -p:PublishSingleFile=true -p:IncludeAllContentForSelfExtract=true -nowarn:CS8618
and I replaced the existing Obj2Tiles.exe in the SuperBuild\install\bin with the newly compiled Obj2Tiles.

Hopefully this helps and may the dev update the release page with the correct binary and source code soon.

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