BigTiff support? about image-tiff HOT 6 CLOSED

This should be fixed by #83 (waiting for the release to close this issue)

from image-tiff.

Hi,

welcome to the Rust world. Don't worry about your skills, PR feedback is there for improving them (and making sure nothing too terrible gets merged without fixing it first, of course).

As for the code architecture, the byte you mention seems to be just a marker in the magic header right at the beginning of the file. As such it probably makes sense to have two separate Decoders internally, but I'd keep the public interface to just one. You could have a private field of type enum {Tiff(Decoder), BigTiff(BigDecoder)} and have the public API switch based on that when you really need completely different behavior for the two.

from image-tiff.

Previous comment suggests this is closed. If this is wrong, feel free to reopen.

from image-tiff.

It looks like only decoding is supported for BigTiff, is that right? Are there any plans to add encoding support too?

from image-tiff.

If you plan to do it yes there is 😄
I have no more time allocated to tiff improvement at work 😞 but my schemes can help you to implement it -> #47

from image-tiff.

Thanks for the quick reply! Your schemes look indeed useful.

(I don't know if I'll be able to work on this though, we are still not sure whether we need BigTiff support at all.)

from image-tiff.