[Solved] Somes problems with sf.Event

[gaulois94]

It's me again

Well, today I have fix somes of my problems and only one survive. When I launch my script (test.py : see src.zip), only one time on two the script doesn't throw a segmentation fault. After many and many test, I found the problem :

TypeError: 'sfml.window.ResizeEvent' object is not iterable

When I do (on EventManager.update() ) :

for event in window.events:
    some code....

At the beginning, the window is resize. If the first test doesn't yield an error, I can resize my window, here isn't the problem : the problem is that an event can't be a sf.ResizeEvent for the first (and only the first) test I think (I'm not sur for 100% ).

with GDB, I have this errors on the traceback (always one time on two) :

Program received signal SIGSEGV, Segmentation fault.
PyObject_GetAttrString (v=0x6d00000005750000, name=0x7ffff1d244c8 "on_resize") at Objects/object.c:819
819     Objects/object.c: Aucun fichier ou dossier de ce type.
 

You can find the source on the attachment (the most importante in this sources are the property and the updates functions only). Maybe the problem is that my guiml.Window inherite from sf.RenderWindow, I don't know... (Indeed, with a sf.RenderWindow, I don't have this problem).

Thank you and sorry for desagrements .

[attachment deleted by admin]

[Sonkun]

Hello

Which script am I suppose to run ? I guess you use Python 3.3.

I tried test.py and test2.py, both run fine and I'm able to resize the window without getting a segmentation fault.

[gaulois94]

test2.py works fine (it don't use others modules). The problem is test.py : it yield a segmentation fault 1 test all of 2 tests, and I don't know why... And yes I use python3.3

[Sonkun]

It works fine on my computer, with python3.3 :/ It must be the ArchLinux package script which doesn't install pySFML properly.

[gaulois94]

Maybe. I have a little script who realize the same problem !

#!/usr/bin/python3
#-*-coding:utf-8-*-

import sfml as sf

class Window(sf.RenderWindow):
        def __init__(self, MODE, STRING):
                sf.RenderWindow.__init__(self, MODE, STRING)

window = Window(sf.VideoMode(800,600,32),"teste")
window.framerate_limit = 60
event = EventManager(window)

check = sf.CircleShape(20)
check.position = sf.Vector2()

while window.is_open:
        for event in window.events:
                pass
        window.clear()
        window.draw(check)
        window.display()

note that if you resize the window, the view is not resize correctly (see where de Circle is and where it should be)

Thank you .

[Sonkun]

Thanks for reporting this issue

When you subclass sf.RenderWindow, its C-level constructor seem not to be called and causes a segmentation fault when the Python object is destroyed probably because it's trying to deallocate an unallocated resource (since the constructor didn't allocate it).

[gaulois94]

Sorry for insisting, but can you fix this problem soon ?

[Sonkun]

Sorry, I tried to solve it one week ago and I just couldn't reproduce the issue... :/ So I guess I messed up somewhere, probably with git and its merge/rebase mechanism.

Can you provide a minimal code which reproduces the error (the one you provided uses EventManager which isn't implemented) ?

Thanks

[gaulois94]

#!/usr/bin/python3
#-*-coding:utf-8-*-

import sfml as sf

class Window(sf.RenderWindow):
    def __init__(self, MODE, STRING):
        sf.RenderWindow.__init__(self, MODE, STRING)

window = Window(sf.VideoMode(800,600,32),"teste")
window.framerate_limit = 60

check = sf.CircleShape(20)
check.position = sf.Vector2()

while window.is_open:
    for event in window.events:
        pass

    window.clear()
    window.draw(check)
    window.display()

Sorry, I created a EventManager but don't use it. Maybe is my last sfml version the problem (I used the official 2.0).

Sorry again .

[Sonkun]

Again I couldn't reproduce the issue, let me fresh install.. I'll tell you tonight.

[Sonkun]

Sorry it took so long... here's the workaround:

class Window2(sf.RenderWindow):
    def __init__(self, MODE, STRING):
        sf.RenderWindow.__init__(self, MODE, STRING)

window = Window2(sf.VideoMode(800,600,32),"teste")
window.framerate_limit = 60

Rename class Window to something else.

I'll fix it later.

There's also a hidden bug in your code:

for event in window.events:
     pass
 

This won't work on Python3.2 (only this version, 2.x and 3.3 do well) because you do nothing with the generator. This is a Cython bug. Add something inside the loop and it will work.

[gaulois94]

Sorry, for me "pass" works with generator, but may it's because I use python 3.3. I tried this code :

#!/usr/bin/python3
#-*-coding:utf-8-*-

import sfml as sf

class Window2(sf.RenderWindow):
        def __init__(self, MODE, STRING):
                sf.RenderWindow.__init__(self, MODE, STRING)
window = Window2(sf.VideoMode(800,600,32),"teste")
window.framerate_limit = 60

check = sf.CircleShape(20)
check.position = sf.Vector2()

while window.is_open:
        for event in window.events:
                print("ok")

        window.clear()
        window.draw(check)
        window.display()
 

but the conclusion is the same : I have an error of segmentation....

sorry for the problem.

[Sonkun]

Weird. Renaming its name has solved everything on my computer... :/ Actually, when a Window2 object is destroyed, since it inherits from sf.Window, the sf.Window destructor is called and deletes the internal C++ object IF the class name is "Window". The original purpose of that is to separate the sf.Window and sf.RenderWindow destructor behavior, the first destroying sf::Window and the second destroying sf::RenderWindow.

I need to rework the inheritance implementation (which isn't straight forward, no documentation or official how-to). One of the remaining task before the next version (1.4) is the investigation of the usage of smart C++ pointer which might  be a solution to that issue.

[gaulois94]

Here you can find screenshots who explain the problem :
As you can see, I have many errors of segmentation. With a resize, the circle is not resized :


Thank you .

[gaulois94]

Maybe with this code you can find the problem :

#!/usr/bin/python3
#-*-coding:utf-8-*-

import sfml as sf

class Window2(sf.RenderWindow):
        def __init__(self, MODE, STRING):
                sf.RenderWindow.__init__(self, MODE, STRING)
window = Window2(sf.VideoMode(800,600,32),"teste")
window.framerate_limit = 60

check = sf.CircleShape(20)
check.position = sf.Vector2()

while window.is_open:
        for event in window.events:
                if type(event) is sf.CloseEvent:
                        window.close()
                if type(event) is sf.ResizeEvent:
                        print(1)

        window.clear()
        window.draw(check)
        window.display()
 

I tried with the lateste sfml git version and with the official 2.0 version. I use your git repository and python3

and the error are :

1Traceback (most recent call last):
  File "./test.py", line 20, in <module>
    print(1)
AttributeError: 'str' object has no attribute 'on_resize'
 

or

1Traceback (most recent call last):
  File "./test.py", line 20, in <module>
    print(1)
SystemError: null argument to internal routine
 

Sorry for problems .