[Rtk-users] R: HNC reader

Thu Nov 21 11:37:48 EST 2019

Dear Simon,

I have forked RTK and pushed a commit <https://github.com/GabrieleBelotti/RTK/commit/cd8ef6dc31a987925949da01f4340810f3b85710>  which took all major changes from this  <https://github.com/SimonRit/RTK/commit/dc987fca11ae283b07547af19efcd26ad377a4da> commit and I hope I correctly added a RTK_USE_BUNZIP2 preprocessor option in rtkConfiguration.h, and in the HncImageIO files.
I hope this is fine, I can’t test it with my current ITK build, but if there’s something I missed or got wrong please let me know (I’m just a newbie).

Gabriele

Da: Simon Rit <simon.rit at creatis.insa-lyon.fr> 
Inviato: mercoledì 20 novembre 2019 11.37
A: gabriele.belotti.bergamo at gmail.com
Cc: rtk-users <rtk-users at public.kitware.com>
Oggetto: Re: [Rtk-users] HNC reader

I think I would first check if there is some code to bunzip on ITK's side (it's not very clear from a quick search in the repo).

If not, then I'll indeed create a cmake option (e.g., RTK_USE_BUNZIP2) and include the code when activated. You can avoid compiling the cxx files from cmake but you'll probably need a preprocessor option in rtkConfiguration.h <https://github.com/SimonRit/RTK/blob/master/rtkConfiguration.h.in>  (as e.g. RTK_USE_CUDA). 

Don't hesitate to submit a work in progress as a PR if you need some help.

Thanks!

Simon

On Wed, Nov 20, 2019 at 11:26 AM <gabriele.belotti.bergamo at gmail.com <mailto:gabriele.belotti.bergamo at gmail.com> > wrote:

Thanks Simon!
I’m able to retrieve those codes for a start and that’s good! Currently I’m using 1.4.0 so I don’t have to fiddle around too much.
How would you go for making the bzip2 dependency optional? Maybe creating an option flag in CMake and editing the reader accordingly?
Gabriele 

Da: Simon Rit <simon.rit at creatis.insa-lyon.fr> 
Inviato: mercoledì 20 novembre 2019 10.49
A: gabriele.belotti.bergamo at gmail.com <mailto:gabriele.belotti.bergamo at gmail.com> 
Cc: rtk-users <rtk-users at public.kitware.com <mailto:rtk-users at public.kitware.com> >
Oggetto: Re: [Rtk-users] HNC reader

Hi,

I think there's been a confusion on my side when writing the release notes of RTK 1.4. I merged a pull request from Andreas Gravgaard who implemented the Xim reader based on the Hnc reader. There is a  <https://github.com/SimonRit/RTK/commit/dc987fca11ae283b07547af19efcd26ad377a4da> commit which includes Hnc but this other  <https://github.com/SimonRit/RTK/commit/9a1754f6e9f7d0257702fc9b48be724c74750990> commit deleted it before merging.

See the related  <https://github.com/SimonRit/RTK/pull/72> PR and  <https://public.kitware.com/pipermail/rtk-users/2016-September/010334.html> mailing list conversation.

We can probably include the reader in RTK if we can make the bzip2 dependency optional using Geoff's code.

Sorry for the mess,

Simon

On Wed, Nov 20, 2019 at 10:11 AM < <mailto:gabriele.belotti.bergamo at gmail.com> gabriele.belotti.bergamo at gmail.com> wrote:

Dear RTK users,

There is mentions of a .hnc reader in the 1.4.0 release ( <https://www.openrtk.org/RTK/news/201802_release1.4.php> https://www.openrtk.org/RTK/news/201802_release1.4.php) but I can’t find it linked in any version I tried (1.4.0 and 2.0.0/1).
I think this may be a mistake so I wanted to raise the issue.

I hope this is useful for anyone other than me!

Gabriele

_______________________________________________
Rtk-users mailing list
 <mailto:Rtk-users at public.kitware.com> Rtk-users at public.kitware.com
 <https://public.kitware.com/mailman/listinfo/rtk-users> https://public.kitware.com/mailman/listinfo/rtk-users

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://public.kitware.com/pipermail/rtk-users/attachments/20191121/6814a6f9/attachment-0001.html>