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

Fix deflated off-road bicycle wheel recipe, plus set correct length and volume for wheel rims (#51264) #51292

Closed
wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Sep 1, 2021

Summary

Bugfixes "Fix deflated off-road bicycle tire recipe, plus set correct length and volume for tire rims"

Purpose of change

Fixes #51264

Describe the solution

Change deflated off-road bicycle wheel recipe to use correct rim
Change longest side and volume of wheel rims as follows (With thanks to @RandDenner for the calculations here):
Bicycle rim (27", 30mm wide) - 686mm, 11L
motorcycle (20", 60mm wide) - 508mm, 12L
medium steel (17", 190mm wide) - 432mm, 28L
wide steel (24", 240mm wide) - 610mm, 70L

Modified the volumes of deflated and inflated versions of tires based on these rims to line up

Describe alternatives you've considered

Suggestion by @actual-nh to create a wider rim for off-road bicycle wheels, will look at that in a future PR as well as current wheel volume

Testing

Loaded into base game and tested

Additional context

rims 1

rims 2

rims 3

rims 4

@actual-nh actual-nh added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Vehicles Vehicles, parts, mechanics & interactions labels Sep 1, 2021
@RandDenner
Copy link
Contributor

RandDenner commented Sep 1, 2021

Suggestion by @actual-nh to create a wider rim for off-road bicycle wheels, will look at that in a future PR as well as current wheel volume

Actually, in each case I used the largest potential dimensions for each rim. So the bicycle wheel I calculated IS an off-road size of 30mm width.
20mm is a normal bicycle rim width. This would drop the volume by 1/3 to 7.33L

@ghost ghost marked this pull request as ready for review September 4, 2021 13:26
@kevingranade
Copy link
Member

I don't know what happened here with the github infrastructure, can you just rebase this and push to see if we can get the tests running?

@ghost
Copy link
Author

ghost commented Sep 12, 2021

@kevingranade No problem, should have been rebased and pushed as requested

@ghost
Copy link
Author

ghost commented Sep 22, 2021

Closed due to resubmission

@ghost ghost closed this Sep 22, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Crafting / Construction / Recipes Includes: Uncrafting / Disassembling [JSON] Changes (can be) made in JSON Vehicles Vehicles, parts, mechanics & interactions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Off road bicycle wheels made using incorrect rims?
3 participants