bugs #tokazama-imagemeta about nifti.jl HOT 8 OPEN

Thanks for taking the time to look at all of this!

1. This seems reasonable to me.
2. This is definitely a typo, but I'm thinking about just reading straight to a dict and avoiding this exact function all together as I think it's unlikely enough people will be motivated to maintain a unique set of NIfTI like functions for a NIfTI header struct and ImageMeta. This is a great example, as a lot of software just defaults to qform or sform and ignores the other because it's too hard to figure out what everyone wants to do with it.
3. Yes, this will be fixed. I've just been lazy.
4. I think this may be related a work around I was trying to use to make CodecZlib work. It's suppose to be the fastest at gzip decompression right now, but it lacks a seek method for streams and that's really helpful for working around NIfTI's bizarre extension region. I'll probably just drop CodecZlib until someone creates a workable solution.

from nifti.jl.

@Tokazama: Any update on this? The design of this looks pretty good and I tested reading which seemed to work fine. Writing however not. I get ERROR: UndefVarError: scl_sclope not defined if I call niwrite.

from nifti.jl.

@Tokazama: I am trying to get the tests running but ran into several issues.

• First it seems that there are several files that are not checked in. Could you add them?
• Then I needed to comment in fileio.jl but I get LoadError: File{DataFormat{:UNKNOWN}}
• The load functions seem to never call niread. Isn't that what they are supposed to do?

from nifti.jl.

Sorry for the delayed response. I'll take a look at this over the course of today.

from nifti.jl.

I'm working on fixing the errors mentioned here and writing better unit testing. I'll likely have something by the end of the day for further review. I'm also going to write down some details related to design choice and small changes I'll probably start working on #16. None of these should be user facing so it's likely of little concern to most.

from nifti.jl.

Great. Isn't the branch already implementing #16? Or do you refer more to the standardized parameter naming within the ImageMetadata?

from nifti.jl.

It may be worth trying this all out again if your interested. You will need to explicitly add NIfTI to your File IO registry for now ( see the tests for how to do this). Im working on a couple more things to really stress test the new implementation and then it should be good to go.

from nifti.jl.

I currently do not have the time to review so go ahead with merging, once you are confident with the design. I would be very interested in learning how to generate a nifti file from scratch. You have one load/ save example but this does not help, if one has just an array at hand.

from nifti.jl.