pombreda / pgreloaded

Automatically exported from code.google.com/p/pgreloaded
Other
0 stars 0 forks source link

Inability to create array2d #9

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
Trying to create array2d from any surface

What is the expected output? What do you see instead?
Expected output: an array with the copy of surface contents.
Output:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.2/dist-packages/pygame2/sdlext/numpysurfarray.py", line 77, in array2d
    pitchdiff = sformat.pitch - width * bpp
AttributeError: 'video.PixelFormat' object has no attribute 'pitch'

What version of the product are you using? On what operating system?
pygame2 SDL12 branch, 26:8196387fc5b5 changeset on Debian Wheezy with 
Linux 3.2.0-2-amd64 #1 SMP.
Please provide any additional information below.

Original issue reported on code.google.com by alex0pla...@gmail.com on 26 May 2012 at 2:23

GoogleCodeExporter commented 9 years ago
Fixed in rev. 29:c3ca743de87c. Please note that the fix was not tested (and one 
a side note: the numeric/numpy array functionality was never tested 
thoroughly), so feedback would be appreciated, if there's more to do.

Original comment by marcusvonappen@googlemail.com on 26 May 2012 at 5:44

GoogleCodeExporter commented 9 years ago
I'm sorry, but you didn't fixed the SDL12 branch. Bug still resides.

Original comment by alex0pla...@gmail.com on 27 May 2012 at 10:28

GoogleCodeExporter commented 9 years ago
...or have you not pushed your changes? I cannot find the abovementioned 
changeset.

Original comment by alex0pla...@gmail.com on 27 May 2012 at 10:33

GoogleCodeExporter commented 9 years ago
Did you use the correct repo? The branch was moved into

http://code.google.com/p/pgreloaded/source/checkout?repo=sdl12

in which the fix was pushed:

http://code.google.com/p/pgreloaded/source/list?repo=sdl12

Original comment by marcusvonappen@googlemail.com on 27 May 2012 at 10:43

GoogleCodeExporter commented 9 years ago
Oh, I'm sorry. I used the SDL12 branch of original repo. Now I see.

Original comment by alex0pla...@gmail.com on 27 May 2012 at 4:24