read pixel pushes an int to stack

[tonyforster]

read pixel rounds colour values to an int, thats 0,0,0 to 1,1,1 or a colour space of only 8 colours.

Colour values are complements of what I would expect, ie 101 for green where i would expect 010, black 111 rather than 000

I dont understand turtle sees, print turtle sees always gives me 0

[walter]

Hmm. Not at all the expected behavior.

You are running on an XO 1 or XO 1.5?

I've tested extensively on XO 1 (362py) and it all works as expected. (Also in sugar-jhbuild.)

I'll come up with a test .ta file so we can have a common ground for debugging.

[tonyforster]

running on xo1, os373pyg, ta94

[tonyforster]

more on print turtle sees
its not always 0, i just chanced a few 0's

set color turtle sees

red 0 30
orange 10 70
yellow 20 70
green 30 0
bu/gn 40 0
lt blu 50 0
mid blu60 0
dk blu 70 0
blu/vio80 30
vio 90 30

[walter]

ta project for testing Turtle Sees block

[walter]

output of test on XO 1 running 353py, TA v95 (prerelease)

[walter]

Bill,

Attached is a .ta file I used to test the Turtle Sees block as well as the output image from the test. My results are quite different from yours.

Please try using ​http://wiki.sugarlabs.org/images/3/37/TurtleArt-95.xo for your test.

thanks.

[walter]

output of read pixel test

[walter]

Also attached is the output of the read pixel test.

Note that since the color values in the palette are 8,8,8 bit rgb and the XO is 5,6,5, we'll not necessarily ever get an exact match. Currently I shift the red and blue values by 3 and the green values by 2. Perhaps I should change the palette to use the 5, 6, 5 colors so that we get an exact match (but this is only going to work for gray == 100 and shade == 5.

[tonyforster]

V95 fixes it
Should let Bernie know V93 is in the pyg release candidate

[tonyforster]

color != turtle sees
the error is quite big for shade !=50
havent tried gray
image attached

[tonyforster]

uploaded screenshot has the png in filename mangled, logged as ​http://bugs.sugarlabs.org/ticket/1638#comment:3

[walter]

I suppose that if shade or gray are not at 50, 100 respectfully, I could try to do a conversion before trying to map to the palette. (Originally I was going to ignore these cases as being inherently too noisy, but I'll test it.)

[walter]

will be included in c95 as per ​http://git.sugarlabs.org/projects/turtleart/repos/mainline/commits/16d7d2a54607c38c4539a23dc266d359c49f82a6