Fatigue #185
Merged
Fatigue #185
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
c92bffe9b Merge pull request #383 from WISDEM/bug_relglob 5c9f4bc16 resetting truth values 818c79756 fix bug that was sending relative member z-coordinates to water load function instead of global z-coordinates git-subtree-dir: WISDEM git-subtree-split: c92bffe9b6467535a39547a3cbb2821f13fac2a1
1670f0305 fixing bug inherited from OpenMDAO output a370981ad equalizing connections for monopile & jacket to fix cost bugs in jacket b3aa1fa56 Merge pull request #384 from WISDEM/bug_post 464d62a10 incrementing version 3deeb12a2 correcting some variable references after changes git-subtree-dir: WISDEM git-subtree-split: 1670f03053cf106709dc8be5fff20e025a365c89
…calling fatigue postprocessing in additional DLCs
31afc3e48 adding fatigue param based on 10.1016/j.proeng.2013.12.061 eqn 5 4f10a5547 Merge pull request #386 from WISDEM/drive_constr d63cdfb3b fixing tests 355588953 add more constraints about deflection and non-torque rotation angle for generator rotor and stator attachments git-subtree-dir: WISDEM git-subtree-split: 31afc3e48724e4b5090070a03908264ea7ba0c9e
gbarter
requested review from
johnjasa and
dzalkind
and removed request for
johnjasa
dzalkind
reviewed
Aug 16, 2022
| ult_stress=inputs['tower_ultstress'][0], | ||
| S_intercept=inputs['tower_wohlerA'][0]) | ||
| ult_stress=1e-3*inputs['tower_ultstress'][0], | ||
| S_intercept=1e-3*inputs['tower_wohlerA'][0]) |
There was a problem hiding this comment.
Does this have to do with the ElastoDyn vs. SubDyn kN vs. N?
There was a problem hiding this comment.
Oh, I see the comment now.
dzalkind
approved these changes
Aug 16, 2022
|
It looks like we're covered on the elastodyn vs. subdyn thing because get_tower_loading and get_monopile_loading use the loads that come from elastodyn and subdyn, respectively. Merge whenever you're ready. |
|
Yes, we have get_tower and get_monopile loading, but now they also give consistent (and correct) units. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
Better support fatigue calculations of steel structures and account for unit differences in subdyn vs elastodyn.
Type of change
Testing