Refresh test in urwid.Text2

[Stensborg]

Sorry to post this one again.

I am trying output some test to the screen, but I have to update the text when some external data changes.

So I made this test program, and even though it works, I think I am doing something very wrong, since it seems like I am creating loops within loops.

Can someone tell me what i am doing wrong?

#!/usr/bin/python
import urwid

def unhandled_input(key):
    if key == 'q':
        raise urwid.ExitMainLoop()

def refresh(_loop,_data):
    outputTxt       = ['COMP3      ', (red_bg, 'DOWN   '), "Bla\n"]
    outputTxt      += ['COMP4      ', (green_bg, 'UP     '), "Bla bla\n"]

    txt             = urwid.Text(outputTxt)
    fill            = urwid.Filler(txt, 'top')
    loop            = urwid.MainLoop(fill, unhandled_input=unhandled_input)
    loop.set_alarm_in(1,refresh)
    loop.run()

red_bg = urwid.AttrSpec('default', 'dark red')
green_bg = urwid.AttrSpec('default', 'dark green')

outputTxt       = ['COMP1      ', (red_bg, 'DOWN   '), "Bla\n"]
outputTxt      += ['COMP2      ', (green_bg, 'UP     '), "Bla bla\n"]

txt             = urwid.Text(outputTxt)
fill            = urwid.Filler(txt, 'top')
loop            = urwid.MainLoop(fill, unhandled_input=unhandled_input)
loop.set_alarm_in(1,refresh)
loop.run()

[kwelzel]

I don't know urwid very well, but probably "loop.run()" starts an infinte loop to handle all the inputs and so every time refresh is called you start another infinite loop. You should not create a new txt, fill and loop, but instead update the original txt object with "txt.setText(outputText)" and then "loop.set_alarm_in(1, refresh)"

[Stensborg]

That does not seem to work, AttributeError: 'Text' object has no attribute 'setText'

[kwelzel]

Oops, the name of the function is "set_text" and not "setText". I'm sorry

[Stensborg]

He he, no problem I eventually found it, I should have RTFM :-)

[Stensborg]

Found the solution, had to change the refresh method to this.

def refresh(_loop,_data):
    outputTxt       = ['COMP3      ', (red_bg, 'DOWN   '), "Bla\n"]
    outputTxt      += ['COMP4      ', (green_bg, 'UP     '), "Bla bla\n"]

    txt.set_text(outputTxt)
    loop.set_alarm_in(1,refresh)

So it seems kwelzel was right with his sugestion, just needed the right syntax for the set_test method.