-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
8 and 16 bit gray-scale (and others) tiff Encode outputs defective on Mac - two 1/2 x1/2 images #6421
Labels
Comments
chaishushan, you've been changing the go.image/tiff code lately, including grayscale images. Can you have a look at this? Owner changed to @nigeltao. |
nigeltao, I read the TIFF6.0 Spec again, and found: Page 39: SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64: Currently Predictor field is used only with LZW. If we fix the 2 bug, roundtrip.predictor.video-001.tiff and roundtrip.video-001-paletted.tiff will ok. |
The patch has been sent as https://golang.org/cl/13779043/ |
This issue was closed by revision golang/image@420d421. Status changed to Fixed. |
olt
pushed a commit
to olt/golang-x-image
that referenced
this issue
Jan 10, 2019
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
mrhyperbit23z0d
added a commit
to mrhyperbit23z0d/bhegde8
that referenced
this issue
Jun 6, 2022
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
GalaxyForcew
added a commit
to GalaxyForcew/A1bisshy
that referenced
this issue
Jun 6, 2022
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
yi-ge3
added a commit
to yi-ge3/wislie
that referenced
this issue
Jun 6, 2022
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
balloontmz6
added a commit
to balloontmz6/Likewise42l
that referenced
this issue
Jun 6, 2022
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
snapbakkhfbav
added a commit
to snapbakkhfbav/SayedBaladohr
that referenced
this issue
Oct 6, 2022
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
MiderWong5ddop
added a commit
to MiderWong5ddop/sidie88f
that referenced
this issue
Oct 7, 2022
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
rorypeckwnt4v
added a commit
to rorypeckwnt4v/LearnByBhanuPrataph
that referenced
this issue
Oct 7, 2022
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
egorovcharenko9
added a commit
to egorovcharenko9/RiceBIOC470z
that referenced
this issue
Oct 7, 2022
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
RafayGhafoorf
added a commit
to RafayGhafoorf/dustinsand8
that referenced
this issue
Oct 7, 2022
…ixel and non RGB ExtraSamples. TIFF6.0 Spec: Page 31: ExtraSamples can only included in RGB mode image. Page 39, SamplesPerPixel is usually 1 for bilevel, grayscale, and palette-color images. Page 64, Currently Predictor field is used only with LZW. Fixes golang/go#6421. R=bsiegert, nigeltao, gbulmer CC=golang-dev https://golang.org/cl/13779043
This issue was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
by gbulmer:
Attachments:
The text was updated successfully, but these errors were encountered: