Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed some bugs with ST #108

Closed
wants to merge 7 commits into from
Closed

Fixed some bugs with ST #108

wants to merge 7 commits into from

Conversation

Sylphar
Copy link
Contributor

@Sylphar Sylphar commented Jan 15, 2025

Notably, when the chatbot answered in multiple paragraphs, the script crashed due to multiple <p>\ elements being found... I'm honestly surprised no one forwarded this to me before, huh.
Also fixed the test scripts, and removed the warnings because they make users scared

Never encountered those with my test bot Testy... they cause strict mode violation: locator(".mes.last_mes .mes_text p") resolving to multiple elements, crashing MonikAI. This should fix it ?
Just allows the script to act as if it were still in the main directory, just an operation to bring the working directory one subfolder up.
+ same logic change as main.py
See TestAdvST, also just remove all common warnings completely they're annoying and not very useful
See TestAdvST
They're just annoying and useless for most users. So, I removed UserWarnings and FutureWarnings. The people who'd be interested in these info probably know how to open the file and see for themselves that they have been suppressed.
Small, easy to distinguish voice file that works with the test scripts. That is all.
@Sylphar Sylphar closed this Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant