Skip to content

Conversation

@dforbush2
Copy link
Contributor

The cable tension bug fix from #1478 (thanks ttran18)

@dforbush2 dforbush2 self-assigned this May 30, 2025
@dforbush2 dforbush2 requested a review from akeeste May 30, 2025 16:17
@jtgrasb
Copy link
Contributor

jtgrasb commented May 30, 2025

This looks good and the tests are passing. Merging now.

@jtgrasb jtgrasb merged commit 321a1b2 into WEC-Sim:main May 30, 2025
10 checks passed
@kmruehl kmruehl requested review from jtgrasb and removed request for akeeste May 30, 2025 17:13
@kmruehl kmruehl assigned jtgrasb and unassigned dforbush2 May 30, 2025
@kmruehl kmruehl added Cable Class Cable Class (cableClass.m) Bug bug in WEC-Sim source, high priority labels May 30, 2025
kmruehl added a commit that referenced this pull request May 30, 2025
* Match PR 1478 on main (#1479)

* match PR 1478 changes on main

---------

Co-authored-by: jtgrasb <87095491+jtgrasb@users.noreply.github.com>
Co-authored-by: akeeste <akeeste@sandia.gov>

* Fixing Regression Tests for RM3 (#1473)

* Fixing Regression Tests for RM3

* Updating the correct _org.mat files

* resolving figure issues

---------

Co-authored-by: Ruehl <kmruehl@sandia.gov>

---------

Co-authored-by: dforbush2 <dforbus@sandia.gov>
Co-authored-by: jtgrasb <87095491+jtgrasb@users.noreply.github.com>
Co-authored-by: akeeste <akeeste@sandia.gov>
Co-authored-by: Jorge Leon <72461917+jleonqu@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug bug in WEC-Sim source, high priority Cable Class Cable Class (cableClass.m)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants