Skip to content

Release Mouse Buddy v3.0.2#1553

Closed
FunkybotsEvilTwin wants to merge 2 commits into
ReaTeam:masterfrom
FunkybotsEvilTwin:reapack.com_upload-1745772035218
Closed

Release Mouse Buddy v3.0.2#1553
FunkybotsEvilTwin wants to merge 2 commits into
ReaTeam:masterfrom
FunkybotsEvilTwin:reapack.com_upload-1745772035218

Conversation

@FunkybotsEvilTwin
Copy link
Copy Markdown

No description provided.

Comment thread Cursor/funkybot_Mouse Buddy.lua Outdated
Comment thread Cursor/funkybot_Mouse Buddy.lua Outdated
Comment thread Cursor/funkybot_Mouse Buddy.lua
…0.9.3 requirement

Let me know if I did any of this wrong. I'm pretty much a novice here. Builds and tested.
@FunkybotsEvilTwin
Copy link
Copy Markdown
Author

Hi @cfillion, please review the changes and let me know if good. I'm a relative novice here.

And to answer an earlier question: yes, ChatGPT helped with portions of the script. There's some hand-rolled code from ToursitKiller that is the basis of this script and I built upon with ChatGPT to help with scripting additional features on top of that and ReaImGui code. This is noted in the comments.

@cfillion
Copy link
Copy Markdown
Member

cfillion commented May 1, 2025

The indentation of the metadata header got messed up and there are 30 r.ImGui_* remaining (ignoring the first two which are OK)...

Blindly AI-generated scripts and code not submitted by the original author are generally not allowed here. Did Touristkiller give permission or a license for their part of the code to be used this way?

@cfillion cfillion closed this May 1, 2025
@FunkybotsEvilTwin
Copy link
Copy Markdown
Author

FunkybotsEvilTwin commented May 1, 2025 via email

@cfillion
Copy link
Copy Markdown
Member

cfillion commented May 1, 2025

To clarify, it's only when the AI's often subtly incorrect output is blindly trusted that it becomes a problem (just testing the script "it seems to work" is not enough: human reasoning on the code is required).

With proper understanding of the code and the AI artifacts cleaned up, a good AI-assisted ReaScript can be indistinguishable quality-wise from a 100% human one. Those are fine. Keyword being AI-assisted, not AI-generated.

(Current AI tech is unable to reason. It only predicts the most likely "next word" over and over.)

You'd be welcome to manually fix the AI's blunders and submit the correction.

get_mouse_context for example contains some pretty suspicious code... (BR_GetMouseCursorContext always returns lowercase strings, yet the code checks in case it might not be strings then converts to uppercase with string.upper, only to re-convert to lowercase immediately after but this time exhaustively using hard-coded values... except in the second native_ctx branch where it uses string.lower to do the same thing. Except for `segment = "TRACK" which stays upper-cased unless window is "tcp".)

@FunkybotsEvilTwin
Copy link
Copy Markdown
Author

FunkybotsEvilTwin commented May 1, 2025 via email

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