This pull request consolidates all /calc commands into two new files: d.calc.ts and g.calc.ts. Additionally, the autocomplete functionality has been modified to ensure it correctly supports the benzo calculation command.
Added some validation checks as well:
Implemented comprehensive validation checks for all parameters across the /calc subcommands.
Ensured that these validation checks prevent user inputs that could lead to unintended responses or errors.
This is for issue #776.
Let me know if anything needs changing. I aimed to be semi-consistent with the way things were done elsewhere in the source code, but I'm sure some improvements could still be made.
IMPORTANT: I tested it and everything should work however I would appreciate it if someone would try out the nasal command and verify that the math still checks out. It should, I didn't touch that logic beyond moving it into g.calc.ts, but I don't want that to go through without someone who understands how to use it testing it since it's rather important that these provide accurate information.
LunaUrsa
commented
1 month ago
This pull request consolidates all /calc commands into two new files: d.calc.ts and g.calc.ts. Additionally, the autocomplete functionality has been modified to ensure it correctly supports the benzo calculation command.
Added some validation checks as well:
This is for issue #776.
Let me know if anything needs changing. I aimed to be semi-consistent with the way things were done elsewhere in the source code, but I'm sure some improvements could still be made.
IMPORTANT: I tested it and everything should work however I would appreciate it if someone would try out the nasal command and verify that the math still checks out. It should, I didn't touch that logic beyond moving it into g.calc.ts, but I don't want that to go through without someone who understands how to use it testing it since it's rather important that these provide accurate information.