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

Correct density of multiple materials #56868

Merged
merged 2 commits into from
Apr 20, 2022

Conversation

mx-moonling
Copy link
Contributor

@mx-moonling mx-moonling commented Apr 17, 2022

Summary

Content "Corrects the density of steel and wood"

Purpose of change

All variations of steels have a density value of 8, apart from older version of steel, which had a density of 30. Wood also had a density of 1.26, which is higher than all but a few rare types of wood.

Describe the solution

Changed the density of basic steel to 8, to correspond with the other forms of steel. This shouldn't affect anything in terms of balance. Also changes the density of wood to 0.6, based on an approximate average of various common woods. 0.6 g/cm3 is the value used for weight estimate of the wooden beam.
https://cedarstripkayak.wordpress.com/lumber-selection/162-2/

Describe alternatives you've considered

A different value for wood density.

Testing

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Apr 17, 2022
@mx-moonling mx-moonling changed the title Correct density of steel Correct density of multiple materials Apr 17, 2022
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Apr 17, 2022
@kevingranade kevingranade merged commit 25bd69f into CleverRaven:master Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants