JVET-O0756: HDRTools VTM software integration

The HDRTools software that works with the implementation can be found here: https://gitlab.com/standards/HDRTools/tree/0.19-dev

There will be an official release later this week.

Hi Dominik, I face issues while compiling it for VisualStudio/x64/Release in Win64 sys. Seems in Win32.H (from HDRLib/Inc), "# define open _open # define close _close"

interferes with "VideoIOYuv::open and VideoIOYuv::close()". Please let me know if I am missing something.

Hi Bappaditya, I pushed a minor change to the HDRTools/0.19-dev. That should resolve the issue. Can you try compiling again and let me know if you experience any more problems? Thank you

Hi Dominik, unfortunately the same issue occurs. Is it possible to provide a README file for all the steps required for linking and integrating for Win64/linux, to make sure everything is done correctly? Thanks!

added 1 commit

• 20246bdd - fixed issues with Compilation under Windows

Compare with previous version

added 1 commit

• 6e651103 - Made the code compliant with the softwares style guidelines

Compare with previous version

The build system is reporting some errors:

/Users/bboe201/jenkins/workspace/VVCSoftware_VTM_mr/source/App/EncoderApp/encmain.cpp:188:62: error: use of undeclared identifier 'metricTimeuser'
         ((endClock - startClock) * 1.0 / CLOCKS_PER_SEC) - (metricTimeuser/1000.0),
                                                             ^
/Users/bboe201/jenkins/workspace/VVCSoftware_VTM_mr/source/App/EncoderApp/encmain.cpp:191:10: error: use of undeclared identifier 'totalTime'
         totalTime / 1000.0);

added 2 commits

• a022d68f - Made the code compliant with the softwares style guidelines
• e1fe6df8 - Merge branch 'O0756-HDRTools-VTM-software-integration' of...

Compare with previous version

resolved all discussions

merged

mentioned in commit 1d09ab0e