Under New Management #17
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
new cog buff for under new management: makes cogs immune to "instant" damaging gags (sound, drop, throw, squirt)
updating list of contributors to reflect current names of individuals who have made a commit to the project
unm changes for throw org
UNM seems to be removing reinforced plating, actual code stays for now but i'll probably remove it at some point in the future
no longer relevant
making org function more efficient
i think the only trace of it is the function in calc.rs now
Extremely early, but updating the version info + changelog now so i wont forget later
removing reference to cog plating
ok now im like 98% sure the function in calc is the last reference
hopefully improved code readability im suuuper tired rn so i cant tell if this is actually better but hopefully it is lol
bumping dependencies
bump version in pyproject because apparently i forgot that i had to do that as well
adjust where things are in cargo
still a wip. will need to revise some functions for it to work as it does in the unm test build.
adding the new balance changes
negative defense at least appears to function how it does ingame currently, it does not yet appear on any of the indicators for level or history, but at minimum functions like other features in the calculator. both types of defense can coexist, but i don't actually know what this behavior would look like ingame since it does not exist yet, but i think it may be possible at some point in the future.
this is effectively done the same way as old v2 calculations were done, i think i should change this in the case that calculations like this become exceedingly more complex at some point in the future.
slight change of the format of calculation history + it now uses a struct rather than just strings. this should make adding to it easier in the future.
updating maturin
adding new defense values in clump rebalance
i forgor
want to try having CI in the calculator, would be cool
fixing readme name in pyproject
openssl seems to be causing some issues with the linux builds, let's see if adding it here fixes them
same as last commit; trying to fix this
now all we need is custom gags
woohoo
...and replace all of its references with widgets.App. really, the only thing missing now is the update checker, which is dubiously useufl anyways. i don't know if anyone actually uses it lol
[skip ci]
adding ubuntu to the list of executables in the readme
there are now better ways to run the calculator from the command line when downloaded as a package: you can run tt-damage-calculator or tt_damage_calculator instead of requiring a python3 -m first
Adding alternate installation method if a user does not want to install with pipx for whatever reason.
requirements.txt contained a list of build requirements rather than requirements to run the program; these are better left in documentation i think.
not sure why that was below keybinds
+ use new pyo3 module format
…ybinds not yet implemented to load, but will be soon maybe
currently there's just a print line, but a messagebox will be more helpful to users that do not run the calculator from their terminal.
it kind of feels unneeded. maybe i'll reimplement it if there's significant outcry about it but it feels like it's currently largely unused.
i really should remember to update this alongside the rest of the program
that's kind of a silly mistake to make oops lol
i thiiink i meant glibc instead
for development-specific requirements
to build ubuntu exe in a venv instead- required by debian-based distros after bookworm 12
unnecessary [no ci]
for the record you can also install tkinter on openbsd using: or something like that [no ci]
include AUR instructions [no ci]
remove patchelf from dependencies, you should actually install your distro's version of patchelf instead.
|
under new management tomorrow.... no review needed i'm merging this NOW |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
add any necessary changes needed for the "Under New Management" update.