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

feat: improvements to running small #1909

Merged
merged 2 commits into from
Aug 20, 2023
Merged

Conversation

midgleyc
Copy link
Member

The way stats work in Small is that your black stats are capped at 1, but your level is uncapped.

Implement this by ignoring charsheet requests, trusting api.php for the stats, and adding a modifier that limits your stats to 1. This makes maximizer + numberology work as desired (i.e. maximizer treats percentage buffs as acting on 1, numberology gets your level correct).

Limit fullness and inebriety.

Skip parsing campground for now. I have HTMLs, I will get back to it later. Your housing never shows up -- it's just "rest in the dirt", but we could parse whether you have free rests left.

Don't count Long Winter's Nap if it's out of standard, which it is now.

@midgleyc midgleyc requested a review from a team as a code owner August 20, 2023 08:15
@codecov
Copy link

codecov bot commented Aug 20, 2023

Codecov Report

Merging #1909 (0d1a697) into main (db3e4c2) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1909      +/-   ##
============================================
- Coverage     36.37%   36.37%   -0.01%     
- Complexity    18831    18833       +2     
============================================
  Files          1083     1083              
  Lines        166619   166628       +9     
  Branches      35393    35397       +4     
============================================
+ Hits          60613    60615       +2     
- Misses        96122    96125       +3     
- Partials       9884     9888       +4     
Files Changed Coverage Δ
src/net/sourceforge/kolmafia/KoLCharacter.java 64.15% <0.00%> (-0.14%) ⬇️
...ourceforge/kolmafia/request/CampgroundRequest.java 56.39% <0.00%> (-0.12%) ⬇️
.../sourceforge/kolmafia/request/CharPaneRequest.java 53.48% <0.00%> (-0.11%) ⬇️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db3e4c2...0d1a697. Read the comment docs.

@jaadams5 jaadams5 merged commit 89601d4 into kolmafia:main Aug 20, 2023
5 checks passed
@midgleyc midgleyc deleted the running-small branch August 20, 2023 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants