Skip to content

feat(balance): improve airship air drag#8219

Merged
chaosvolt merged 4 commits intocataclysmbn:mainfrom
WishDuck:airshipdrag
Feb 28, 2026
Merged

feat(balance): improve airship air drag#8219
chaosvolt merged 4 commits intocataclysmbn:mainfrom
WishDuck:airshipdrag

Conversation

@WishDuck
Copy link
Collaborator

@WishDuck WishDuck commented Feb 21, 2026

Purpose of change (The Why)

In #7153 I did not know how to add airship drag well so didn't add a great version
Come today when drag was being polished I received a suggestion

@WishDuck Sorta related, the reference area when getting the drag coeff for airships should be taken as the square of the cube root of the airship's lift bag volume. Submurged bodies like boats use the wetted surface area, not the wetted volume like the code is using

Describe the solution (The How)

Add airship drag coeff like retagin suggests

Describe alternatives you've considered

Keep balloons without drag
Somehow use the normal system instead of retagins suggestion

Testing

CRIT blimp went up
Old blimp stayed about the same
Scales correctly with blimp balloon size

Additional context

I thought eepy brain could get this right unlike things like

ITTC 1957 puts a suggested approximation of 0.075/ (log10(Reynolds number) -2)^2
Reynolds number is (Velocity * Longest_hull_dimension * fluid_density (water is 1)) / water_dynamic_viscosity (just take it at 1.0016 mPa*s)

Eepy brain would make so many mistakies

Checklist

Mandatory

@WishDuck WishDuck marked this pull request as draft February 21, 2026 07:59
@github-actions github-actions bot added the src changes related to source code. label Feb 21, 2026
@WishDuck WishDuck changed the title feat(balance): add airship air drag feat(balance): improve airship air drag Feb 21, 2026
@WishDuck WishDuck marked this pull request as ready for review February 21, 2026 21:22
@chaosvolt chaosvolt merged commit e0d5eb9 into cataclysmbn:main Feb 28, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

src changes related to source code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants