Skip to content

Conversation

@czerwinskilukasz1
Copy link

No description provided.

@drzraf
Copy link

drzraf commented Apr 13, 2020

  • Is it possible to do this OpenCV update on top of Unbundle OpenCV #111 where I tried to unbundle OpenCv? (I think it's the way to go. Easier to update, easier to review & maintain)
  • Which OS was tested for compilation/runtime?

@czerwinskilukasz1
Copy link
Author

  • Is it possible to do this OpenCV update on top of Unbundle OpenCV #111 where I tried to unbundle OpenCv? (I think it's the way to go. Easier to update, easier to review & maintain)

I am not sure whether unbundling OpenCV is the right thing to do right now. This project is not updated on frequently basis, so it's much easier to run into compatibility errors with newer versions of OpenCV. I would leave it bundled for the time being.

  • Which OS was tested for compilation/runtime?

So far I tested on Windows.
I will also test on Linux (Ubuntu) and MacOS when Marcus preliminarily approves this PR.

@hudrima1, what do you think about my idea of upgrading OpenCV to 4.3.0?

@czerwinskilukasz1
Copy link
Author

@hsm4, would you have a while to let us know what you think?

@hudrima1 hudrima1 merged commit 3cccd1e into cpvrlab:master Apr 25, 2020
@hudrima1
Copy link
Contributor

Hi Lukasz,
I'm happy that someone takes time for some maintenance and improvements because we don't have time in our group for this project right now. We are busy with the SLProject project. We there use also several libraries among them is also OpenCV. This is a development project where we provide prebuilt versions of some libraries that we download with cmake.
What about the prebuilt executables for Windows, MacOS and Linux of ImagePlay?
Regards
Marcus

@czerwinskilukasz1
Copy link
Author

Thank you @hudrima1 for your appreciation and for merging my changes! :)
I could provide prebuild executables with my change for all 3 OS's.
Do you have any special requirements on how they should be built?
Where should I upload the executables?

@hudrima1
Copy link
Contributor

Hi Lucasz,
we are working almost exclusively on SLProject (https://github.com/cpvrlab/SLProject). It uses also OpenCV among other libraries. Some of them are prebuilt such as OpenCV because their build with Contribs for Win., Macos, Android and iOS is rather painful. We download these prebuilt libraries from an FTP of us. We do this all in cmake. You can see how we do that in the cmake file SLProject/cmake/DownloadPrebuilt/. I would prefer a similar way for ImagePlay before we to prebuilt executables. I just updated Qt and cloned ImagePlay and I can't build because the linked OpenCV libs are still on 3.1. I thought you migrated them to 4.3?
Regards
Marcus

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