OpenDroneMap / Obj2Tiles

Converts OBJ files to OGC 3D tiles by performing splitting, decimation and conversion
GNU Affero General Public License v3.0
196 stars 58 forks source link

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

Open cubantonystark opened 2 months ago

cubantonystark commented 2 months ago

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 commented 2 weeks ago

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.