cubert-hyperspectral / cuvis.sdk

Apache License 2.0
14 stars 2 forks source link

Hypertools Submodule #23

Closed robertCubert closed 1 year ago

robertCubert commented 1 year ago

Open Points for Discussion:

robertCubert commented 1 year ago

feature/add-hypertools

antmedellin commented 1 year ago
nhanson2 commented 1 year ago

@antmedellin it might be useful to have a tutorial with images to show how a user might be expected to operate the interfaces. I am happy to test this on my Ubuntu machine.

arndcubert commented 1 year ago

@antmedellin The imreadmulti function reads multi-page TIFFS. By default, the cubes are exported as multi-channel-Tiffs. Can exports using the tiff_formatMultiPage option be imported?

robertCubert commented 1 year ago
robertCubert commented 1 year ago

Regarding the integration of the Hypertools submodule, as mentioned in #26 , I created a new repo Hypertools C++/CUDA Classification. In this repo we would suggest to add the Hypertools as well as the SDK as submodules. In this repo we then want to provide a cmake file that is looking for both of the submodules so that the user can link their project.

@antmedellin from your readme, I've seen that you would like the Hypertools to be agnostic to camera manufactures, which I think is a good idea. We would therefore suggest that cubert specific examples that use the Hypertools could be developed in our repo. What do you think about this approach?

antmedellin commented 1 year ago
robertCubert commented 1 year ago
antmedellin commented 1 year ago
robertCubert commented 1 year ago

@antmedellin The imreadmulti function reads multi-page TIFFS. By default, the cubes are exported as multi-channel-Tiffs. Can exports using the tiff_formatMultiPage option be imported?

that there should be an option for MultiPage as well. I haven't tried it myself though.

antmedellin commented 1 year ago

Just tested this example with 'args.format=cuvis::tiff_format_t::tiff_format_MultiPage;' for the multi-channel tiff and it worked. I was able to view the output file in Document Viewer on Linux in the desired format.

As a side note, it did not work on the cornfields dataset. I think the example requires the session_xxx_xxx_cube.tiff to work properly.