[Sugar-devel] Draft Proposal - improve & maintain 12 sugar activities

[James Cameron]

No worries.  I'm not a mentor this year, so your proposal is not
something I'll advise on.  My feedback was about Simulate, and the
normal workflow of the organisation.

On Sun, Apr 02, 2023 at 01:06:38PM +0530, Spandan Barve wrote:
> Thanks for the reply quozl, I understand now and this will also help me when
> porting simulate to python3 which is also a part of my proposal, I'll also
> specify sugargame in my proposal, thanks.
> 
> On Sat, Apr 1, 2023 at 4:54 PM James Cameron <[1]quozl at laptop.org> wrote:
> 
>     On Sat, Apr 01, 2023 at 04:21:59PM +0530, Spandan Barve wrote:
>     > Thanks for the feedback Ibiam.
>     > [...]
>     > > Simulate activity sometimes crashes Sugar, what do you mean by crashing
>     > Sugar?
>     > I was not able to reproduce it but I do have the error saved. by
>     > crash, I mean that the activity stops responding completely,
>     > including the stop button and pressing CTRL+Q, I had to go back to
>     > the journal and then remove the activity, after that I got an error
>     > in the terminal. 
>     > here is the error : [2]https://pastebin.com/1RcTvYHc 
> 
>     It's a bug.
> 
>     The error showed Python 2.7.  Simulate is yet to be ported to Python
>     3.  The traceback can be ignored until that port is done.
> 
>     Simulate also needs a port to later Sugargame.  During the port of
>     Sugargame a complete review of the event loop should be done, as
>     Sugargame changed substantially.
> 
>     Simulate contains many inner event loops, with calls to wait, update,
>     and tick, yet without checking for the stop button or Control/Q.
>     That's probably the bug you saw.
> 
>     Simulate also uses an unconventional exit method.
> 
>     The Sugargame sample code shows how to do it instead.
> 
>     Simulate window can also vanish but the system process persist, which
>     was first reported in 2018.
> 
>     [3]https://github.com/devAbnull/simulate_sugargame/issues/1
> 
>     I could not tell if you used git HEAD or the latest release of
>     Simulate, which predates git HEAD by some years.
> 
>     [4]http://activities.sugarlabs.org/en-US/sugar/addon/4778
> 
> References:
> 
> [1] mailto:quozl at laptop.org
> [2] https://pastebin.com/1RcTvYHc
> [3] https://github.com/devAbnull/simulate_sugargame/issues/1
> [4] http://activities.sugarlabs.org/en-US/sugar/addon/4778