[Cuis-dev] Morphic glitches out/hangs when adding faulty code to drawOn method on morphs that step

[Alan Dao]

Hope I'm not annoying by bringing this up, but I pulled the latest changes,
5601 and 5602, and unfortunately it breaks my startup script for setting up
my workspace/browser windows. More specifically, my windows aren't
appearing and the background doesn't switch to the dark theme. When I
comment out "ChangeSet installNewUpdates.", my workspace/browser windows
appear as intended.

Could you take a look and see what's going wrong? I've attached my startup
script below. You can try commenting out "ChangeSet installNewUpdates." to
see the impact of the latest changes. I've been running my startup script
through this terminal command:

/Applications/Squeak.app/Contents/MacOS/Squeak
/Users/alandao/Cuis/Cuis-Smalltalk-Dev/Cuis6.0-5542.image -s
/Users/alandao/Scripts/init_workbench.st

Alternatively, I'd be ok with re-writing my script if the way I did it was
hacked up together, as long as there's a way to migrate this script over.
Any help would be appreciated!

Also, thanks for the suggestions to the step method. The package itself is
still in extremely early development, but once I get the UI to where I want
it, I'll look into optimizations.

Best,
Alan

On Mon, Dec 19, 2022 at 2:17 PM Juan Vuletich <juan at cuis.st> wrote:

> I just realized we're creating an exception handler for each morphic
> cycle, and that's not really needed. I just pushed an update that should
> have some positive impact on performance on slower hardware. Hopefully, I
> didn't add new annoyances.
>
> BTW, Alan, what you're doing in your step method, #add: and #removeFirst,
> are pretty expensive, and it can be made cheaper, by just changing which
> indexes you map to X coordinates.
>
> Cheers,
>
> On 12/19/2022 5:57 PM, Alan Dao via Cuis-dev wrote:
>
> Awesome, thanks for the update! Just pulled the latest changes and can
> confirm the fixes are working.
>
> Best,
> Alan
>
> On Mon, Dec 19, 2022 at 9:50 AM Juan Vuletich <juan at cuis.st> wrote:
>
>> Hi Alan,
>>
>> Hey, thanks! This really helped.
>>
>> I could reproduce the problems you saw, and I found a couple of subtle
>> bugs. Fixes are now at GitHub. Keep them coming!
>>
>> Thanks,
>>
>> On 12/18/2022 4:20 AM, Alan Dao via Cuis-dev wrote:
>> > Hey Juan,
>> >
>> > So for a mistake on my end, I didn't realize updates from Git had to
>> > be explicitly installed.. after installing the updates and doing your
>> > test-case, I can see there's no glitches.
>> >
>> > Unfortunately, my morph, BebopTransform, still hangs Morphic when I do
>> > the below test case. If you have spare time, I've attached my package,
>> > named Bebop, for you to reproduce. I apologize in advance if my morph
>> > is not implemented correctly.
>> >
>> > - Repos are pulled
>> > - Started fresh Cuis.
>> > - Installed updates.
>> > - Loaded package 'Bebop'
>> > - World / New morph... / Vector Graphics / BebopTransform
>> > - In BebopTransformGraphLine >> drawOn:, add "aCanvas drawString: 1.0
>> > at: -13 @ 58 font: nil color: Color brown."
>> >
>> > It looks like Morphic only hangs when we create BebopTransform and
>> > then modify drawOn: later on.
>> >
>> > Best,
>> > Alan
>> >
>>
>> --
>> Juan Vuletich
>> cuis.st
>> github.com/jvuletich
>> researchgate.net/profile/Juan-Vuletich
>> independent.academia.edu/JuanVuletich
>> patents.justia.com/inventor/juan-manuel-vuletich
>> linkedin.com/in/juan-vuletich-75611b3
>> twitter.com/JuanVuletich
>>
>>
>
> --
> Juan Vuletichcuis.stgithub.com/jvuletichresearchgate.net/profile/Juan-Vuletichindependent.academia.edu/JuanVuletichpatents.justia.com/inventor/juan-manuel-vuletichlinkedin.com/in/juan-vuletich-75611b3twitter.com/JuanVuletich
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20221219/64afdea6/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: init_workbench.st
Type: application/octet-stream
Size: 1733 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20221219/64afdea6/attachment.obj>