Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add open_htj2k_support #9

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

osamu620
Copy link
Contributor

@osamu620 osamu620 commented May 24, 2022

related to #3

@palemieux
Copy link
Contributor

@osamu620 Does OpenHTJ2K require Cxx17 by any chance?

Compilation dies on #include <filesystem>

@osamu620
Copy link
Contributor Author

@osamu620 Does OpenHTJ2K require Cxx17 by any chance?

Compilation dies on #include <filesystem>

Yes. The threadpool class uses C++17 features.

@palemieux
Copy link
Contributor

@osamu620 Does OpenHTJ2K require Cxx17 by any chance?

Compilation dies on #include <filesystem>

Yes. The threadpool class uses C++17 features.

Which version of GCC should I use?

@palemieux
Copy link
Contributor

@osamu620 Does OpenHTJ2K require Cxx17 by any chance?

Compilation dies on #include <filesystem>

Yes. The threadpool class uses C++17 features.

Which version of GCC should I use?

It looks like GCC10 should work.

@osamu620
Copy link
Contributor Author

@palemieux
I have confirmed GCC10 (gcc-10 (Ubuntu 10.3.0-15ubuntu1) 10.3.0) works.

@palemieux
Copy link
Contributor

@osamu620 j2k_ht_ohtj2k fails on RGBA images

./build/libench j2k_ht_ohtj2k src/test/resources/images/rgba.png

@palemieux
Copy link
Contributor

@osamu620 Can you add j2k_ht_ohtj2k to the Python script:

f0aa58d

@michaeldsmith
Copy link
Collaborator

why not just add this line to CMakeLists.txt to require C++ 17 for all platforms?

set(CMAKE_CXX_STANDARD 17)

@palemieux
Copy link
Contributor

why not just add this line to CMakeLists.txt to require C++ 17 for all platforms?

Not all Linux distributions ship with a GCC toolchain that support C++ 17.

osamu620 and others added 2 commits May 25, 2022 17:52
@osamu620
Copy link
Contributor Author

@palemieux @michaeldsmith
A bug on RGBA images in OpenHTJ2K is fixed. And I have modified some C++17 or C++14-specific features. Now, C++11 compilers can compile OpenHTJ2K.

@palemieux
Copy link
Contributor

@osamu620 You might have forgotten to update the OpenHTJ2K submodule. Also, the library outputs Tile num x = 1, y = 1 repeatedly to stdout. How can the latter be suppressed?

- Remove decoder message that is not useful
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