Skip to content
This repository has been archived by the owner on Sep 8, 2024. It is now read-only.

Possible race condition in SkillGUI class when removing a screen #3013

Closed
krisgesling opened this issue Oct 20, 2021 · 3 comments
Closed

Possible race condition in SkillGUI class when removing a screen #3013

krisgesling opened this issue Oct 20, 2021 · 3 comments

Comments

@krisgesling
Copy link
Contributor

Describe the bug
There have been multiple reports of unexpected behaviour when removing pages from a GUI namespace.

Most recently in the Pairing Skill:
MycroftAI/skill-pairing#71 (comment)

As it's beyond the scope of the Pairing Skill it's been moved here.

To Reproduce
Run the GUI stress tester Skill: skills-gui-stress-test.aiix.zip

To run the test: "run stress test one"

Expected behavior
Adding and removing pages should operate consistently and be performed in the order in which they are requested.

@forslund
Copy link
Collaborator

forslund commented Nov 7, 2021

Was this resolved through #3022?

@krisgesling
Copy link
Contributor Author

Not sure, I haven't tested it deeply yet.

@forslund
Copy link
Collaborator

forslund commented Sep 8, 2024

Closing Issue since we're archiving the repo

@forslund forslund closed this as completed Sep 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants