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

Fold athletics skill into cardio #53619

Closed
Inglonias opened this issue Dec 20, 2021 · 10 comments
Closed

Fold athletics skill into cardio #53619

Inglonias opened this issue Dec 20, 2021 · 10 comments
Labels
<Enhancement / Feature> New features, or enhancements on existing Mechanics: Effects / Skills / Stats Effects / Skills / Stats stale Closed for lack of activity, but still valid. <Suggestion / Discussion> Talk it out before implementing

Comments

@Inglonias
Copy link
Contributor

Inglonias commented Dec 20, 2021

Is your feature request related to a problem? Please describe.

The athletics skill is basically useless right now, consigned only to swimming. In addition, we also have a cardio system, separate from the athletics skill, but serving the same function for stamina when running

Solution you would like.

Fold these two systems together. Either make the cardio system visible somehow through the athletics skill, or, more likely, remove the athletics skill completely and use cardio for both swimming and running. More visibility on the cardio and stamina system would be nice too.

Describe alternatives you have considered.

I mean, we don't have to do this, but it would be kind of neat.

Additional context

No response

@Inglonias
Copy link
Contributor Author

Actually, it would be really cool if we could use the cardio system to have an "effective" athletics skill, similar to how we use encumbrance to have an "effective" dodge skill.

@ghost
Copy link

ghost commented Dec 20, 2021

Atheltics is already a factor in cardio.

const int athletics_mod = get_skill_level( skill_swimming ) * 10;
int final_cardio_fitness = bmr / 2 + athletics_mod + health_effect + trait_mod + prof_mod + cardio_acc_mod;

https://github.com/CleverRaven/Cataclysm-DDA/blob/master/src/character.cpp#L6317

@nexusmrsep
Copy link
Contributor

Every skill has two characteristics: it has to be trainable somehow and it must serve a purpose somehow.
Athletics as a skill is problematic in a current form as it conceptually overlaps strength (stat) and cardio (discrete mechanic), but finally cannot decide what it realy is.

So we actually should start from answering the question: what is athletics?

  1. If the answer is: muscle mass and muscle strength, then the overlap is more with strength then cardio, and it would affect strength based activities, perhaps introducing a sort of stats-from-skills "mod" but for a singular skill and singular stat.
  2. If the answer is: muscle fitness for prolonged activity, then it overlaps cardio more then strength, and it might be used by for example reducing stamina cost of actions (example: lets say every tile you run eats flat 20 stamina, but it could be: 20 - athletics)

Problem with first approach is that as of now the game balance imply static stats, set in stone at character creation. It was already discussed at least few times how many months one would need to spend in the gym to get 1 point of strength in game units. Thus it is doubtful that strength-through-athletics would be approved, although if the skill levels were in a certain ratio to the "power" of 1 point of strength, then who knows? But then what's the point for the skill that has negligible effect?

Problem with the second approach is that cardio system is (I guess) a go-to system to govern stamina from the players perspective (answer to question: what can I do to get my stamina management better). Stamina's influence on game balance and gameplay is governed by 3 factors: stamina pool (max stamina), stamina regen ratio, stamina consumption ratio - if any skill, mechanic, trait, mutation touches one of these - the in game effect is apparent. Cardio's final effect defines max stamina (pool). Theoretically this leaves stamina regen and stamina consumption as possible targets of athletics. But then again: overlap - you already gain cardio/stamina by activity, so why gain athletics/stamina simultaneously - as that would do literally the same?

So in fact there are arguments to remove athletics altogether - lets not forget that it was migrated from the swimming skill.

Actually, it would be really cool if we could use the cardio system to have an "effective" athletics skill, similar to how we use encumbrance to have an "effective" dodge skill.

They are too far apart - rather, after removing athletics, cardio fitness should be exposed to the player in some form - there is no reason to why is should be hidden - you know if you are capable of running 50 meters or a marathon. If there are concerns about giving exact quantified measurements - give the info in blocks similarly to how weight is described.

@wapcaplet wapcaplet added <Enhancement / Feature> New features, or enhancements on existing <Suggestion / Discussion> Talk it out before implementing Mechanics: Effects / Skills / Stats Effects / Skills / Stats labels Dec 20, 2021
@Photoloss
Copy link
Contributor

Cardio's final effect defines max stamina (pool).

Does cardio not affect stamina regen at all? The mutation modifier for "maximum" stamina grants just as much of a regen boost as the actual regen modifier!

And stamina consumption would need more granularity for arms/legs, burst/endurance, strength/speed etc. With the current UI that wouldn't really work with Proficiencies let alone Skills.

@Terrorforge
Copy link
Contributor

Does cardio not affect stamina regen at all? The mutation modifier for "maximum" stamina grants just as much of a regen boost as the actual regen modifier!

It does. According to the original spec, regen is 5 + [Cardio/100]. And anecdotally, my 200cm Fast Metabolism characters (BMR ca. 3.4k) have no problems regenerating stamina even through a gas mask.

@wapcaplet
Copy link
Contributor

Cardio and stamina are in a pretty bugged condition right now (see #53603) - the max stamina modifier is definitely broken, and none of the size-related and cardio-related mutations are working correctly with respect to stamina and stamina regen. So you can probably ignore most of the current behavior of those factors.

@nexusmrsep
Copy link
Contributor

Thus definitely calling for a dedicated test suite.

@stale
Copy link

stale bot commented Apr 19, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not 'bump' or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

@stale stale bot added the stale Closed for lack of activity, but still valid. label Apr 19, 2022
@l29ah
Copy link
Contributor

l29ah commented May 29, 2022

Tested the athletics impact on stamina on today's master: it only adds about 0.5% to max stamina pool size per skill level, so much less than one sprinted tile :/

@stale stale bot removed the stale Closed for lack of activity, but still valid. label May 29, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Dec 6, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not bump or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

@github-actions github-actions bot added the stale Closed for lack of activity, but still valid. label Dec 6, 2022
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jan 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Enhancement / Feature> New features, or enhancements on existing Mechanics: Effects / Skills / Stats Effects / Skills / Stats stale Closed for lack of activity, but still valid. <Suggestion / Discussion> Talk it out before implementing
Projects
None yet
Development

No branches or pull requests

6 participants