Skip to content

Conversation

@NathanMOlson
Copy link

@NathanMOlson NathanMOlson commented Aug 12, 2025

This PR gets OpenSfM to compile in the Docker Ubuntu 24.04 environment.

This codebase has diverged substantially from the upstream mapillary version. I'd like to move them closer together, but I think it is best to make that a follow-on effort.

The CI workflow fails. It also fails in the currently-used 357 branch.

@NathanMOlson NathanMOlson marked this pull request as draft August 13, 2025 01:18
…mentation ID of each observation. This allows it to pass the unit test "HasIOFileConsistency", which ensure that now information is lost when a track is written to disk and restored. Note that this increases the size of the tracks file by 40%,
… and segmentation ID of each observation. This allows it to pass the unit test "HasIOFileConsistency", which ensure that now information is lost when a track is written to disk and restored. Note that this increases the size of the tracks file by 40%,"

This reverts commit 3d54bc3.
…n ID and instance ID are recovered from the file (they are not stored in the current OpenDroneMap file version). This is specific to OpenDroneMap, my guess is that this is a speed/size performance optimization.
@NathanMOlson NathanMOlson marked this pull request as ready for review August 13, 2025 19:08
@smathermather
Copy link

The CI workflow fails. It also fails in the currently-used 357 branch.

Thanks for saving me the check.

This PR gets OpenSfM to compile in the Docker Ubuntu 24.04 environment.

This codebase has diverged substantially from the upstream mapillary version. I'd like to move them closer together, but I think it is best to make that a follow-on effort.

So this mainly is intended to take the existing forked OpenSfM and ensure it builds with a follow-on for pulling in latest. I wonder if @YanNoun would have tips on that process. Regardless, sounds like a reasonable division.

This reverts commit 31eb1ee.

The reverted commit breaks unit tests in ODM.
This reverts commit d0d5889.

Apparently this revert is also needed to pass unit tests.
@smathermather smathermather merged commit c532843 into OpenDroneMap:ODM Aug 18, 2025
0 of 3 checks passed
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

Successfully merging this pull request may close these issues.

3 participants