Skip to content

Uh60#42

Merged
JustinPorter88 merged 8 commits intodevelopfrom
uh60
Feb 5, 2026
Merged

Uh60#42
JustinPorter88 merged 8 commits intodevelopfrom
uh60

Conversation

@ptrbortolotti
Copy link
Contributor

This PR:

  • Adds back the blade of the UH-60, including some scripts that had been removed because unused with WT blades. Note that the blade runs as long as no balancing weight is used. I haven't been able to include those in the models as the code errors out. I also went back to https://gitlab.lrz.de/HTMWTUM/SONATA and the example there, but I get a different error
  • Update the org name to NLRWindSystems
  • Limit the tests to fewer stations to cut CI waiting times

sigma11 = applied_loads[0] / area

tol = 1e-4
tol = 1e-1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Flagging this so I remember. I need to go check if this test is still sensitive enough to catch errors in the code with the new test tolerance (i.e., verify the test still fails when I break the code).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't the test that I was most concerned about, so this change is okay in that regard.

Looks like the failed CI run is this one: https://github.com/NLRWindSystems/SONATA/actions/runs/21037290795/job/60488705197

That fails for sigma33, so I am going to push a commit where the tolerance is still loosened on that component, but remains at 1e-4*sigma11 for the sigma11 stresses (right now 10% error is accepted for sigma11 on this simple beam in tension, which seems a bit loose).

@JustinPorter88 JustinPorter88 merged commit 709f19b into develop Feb 5, 2026
14 checks passed
@JustinPorter88 JustinPorter88 deleted the uh60 branch February 5, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants