AliceVision icon indicating copy to clipboard operation
AliceVision copied to clipboard

Update INSTALL.md to suggest installing Mosek 6 instead of Mosek 5

Open Christopher-Hayes opened this issue 3 years ago • 2 comments

Description

The INSTALL.md installation steps assume you have Mosek 6 installed, but the list of dependencies at the top of the document list Mosek 5.

I installed Mosek 5 and later ran into cmake errors which seem to suggest AliceVision prefers Mosek 6. This comment from a GitHub issue references an identical cmake error when using Mosek 8 intead of Mosek 6. Installing Mosek 5 instead of 6 seems to cause a similar problem.

Features list

  • [X] Update documentation

Update: I installed Mosek 6 and got the same error, so Mosek 5 might still be fine, but I'll leave this PR in the interest of consistency.

Christopher-Hayes avatar Jun 04 '21 03:06 Christopher-Hayes

Thanks. So do you confirm that you need the cmake patch from #411 to make it work with Mosek?

simogasp avatar Jun 04 '21 07:06 simogasp

Thanks. So do you confirm that you need the cmake patch from #411 to make it work with Mosek?

I attempted that patch, and it resolved the cmake error. However, I got an error when while running make, I suspect it's the same error that the OP from that issue also received. It seemed related to an env variable for getting the Mosek license. I got a trial version license, not sure if that is causing unknown bugs. If it's useful, I can open an issue with the error I was getting after using that patch.

Regardless, I just ended up removing Mosek completely and continued to build Meshroom without Mosek, which I was able to do without issue.

I'm running Ubuntu 20.04 btw, so potentially a Linux-related issue.

Christopher-Hayes avatar Jun 04 '21 07:06 Christopher-Hayes

Cannot be merge due to conflicts and I cannot update the original code as there is no corresponding fork.

fabiencastan avatar Sep 27 '22 20:09 fabiencastan