Skip to content

Conversation

@keagdo
Copy link
Contributor

@keagdo keagdo commented Mar 12, 2025

Watch for infinite loops... :| its super easy for glamourer to loop you, but also really easy to "stop" the loop but just disabling the rules 1 by one. No different than spam changing outfits. Doesn't actually get caught in an infinite loop in code. Only when you a loop is finished, it updates your looks (changing your race), then the rules get updated etc etc....

There's an extra window pop-up warning of this, and requiring users to "yield right to complain"

keagdo and others added 2 commits March 12, 2025 19:11
@semanticdiff-com
Copy link

semanticdiff-com bot commented Mar 12, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  DynamicBridge/Gui/GuiRules.cs  7% smaller
  DynamicBridge/Checkers/Races.cs  0% smaller
  DynamicBridge/Configuration/ApplyRule.cs  0% smaller
  DynamicBridge/Configuration/Config.cs  0% smaller
  DynamicBridge/DynamicBridge.cs  0% smaller
  DynamicBridge/Gui/GuiSettings.cs  0% smaller
  DynamicBridge/IPC/Glamourer/CursedActionManager.cs  0% smaller
  DynamicBridge/IPC/Glamourer/GlamourerManager.cs  0% smaller

@Limiana
Copy link
Collaborator

Limiana commented Mar 20, 2025

Thank you, I'll hold this for a bit because I want to make sure infinite loops can't happen, one way or another

@keagdo keagdo closed this Mar 28, 2025
@keagdo keagdo deleted the race-rule-selection branch March 28, 2025 19:25
@keagdo keagdo restored the race-rule-selection branch March 28, 2025 19:25
@Limiana
Copy link
Collaborator

Limiana commented Mar 28, 2025

Hello! Please keep it open, the PR will be merged after I can figure out how to prevent infinite loops

@Limiana Limiana reopened this Mar 28, 2025
@keagdo
Copy link
Contributor Author

keagdo commented Mar 28, 2025

yeah sorry i accidentally deleted the branch while deleting the other old ones! trying to figure out how to make the conflicts resolve! Didn't know how to open it again! Still learning collaborative git.

@keagdo
Copy link
Contributor Author

keagdo commented Apr 5, 2025

Assuming this change works, this should finally resolve Limiana/DynamicBridgeStandalone#32

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.

2 participants