Fix unit tests

[NathanMOlson]

Changes to get all unit tests to compile. There should be no functional change.

This is my first step toward syncing with mapillary/main.

[smathermather]

Thanks! Will be useful to have tests working first as the first step.

Any reason this is tied to 357? Is that to ensure tests work again for 357?

[NathanMOlson]

Any reason this is tied to 357? Is that to ensure tests work again for 357?

Because as far as I can tell, 357 is the latest known working version.

[smathermather]

Got it. Exactly the clarification I needed.. Happy to change the base branch to a branch off 357. Do you have a preferred name for that new branch?

Just created a branch to merge it into, as 357 would be live for anyone installing right now.

[NathanMOlson]

Just created a branch to merge it into, as 357 would be live for anyone installing right now.

Actually, ODM master is currently using OpenSfM 355 branch, so it should be safe to modify 357.

This situation is why I think we should use git hashes (or git tags) instead of branch names to specify dependency versions in ODM.