Technical changes: Delete and concurrency rework #94
AlUlkesh
announced in
Announcements
Replies: 2 comments
-
Great going, thanks! Will check this out and update. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
This started with trying to fix the delete functionality, which had been bandaid-fixed several times and still didn't work right in difficult to reproduce circumstances. And deleting the wrong file is always very bad.
While deep in debugging I noticed more problems with the code, especially related to synchronisation with Gradio, but also javascript functions running at the same time, that should not do that.
What's more, while fixing bit by bit, other parts started failing, that apparently depended on the other wrong code. I suspect these were put in as workarounds at some point.
After many many different attempts, I decided to completely rework the whole logic.
This current version seems very stable and responsive in my tests. Also it will make future changes much easier for me.
Some more details about what was changed:
Beta Was this translation helpful? Give feedback.
All reactions