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

Application updates for v6.1.2 release #67

Merged
merged 2 commits into from
Jan 16, 2025
Merged

Conversation

dforbush2
Copy link
Contributor

@dforbush2 dforbush2 commented Sep 11, 2024

Adding one fix per commit
Listing broken library links for future reference.

  1. Controls MPC: exactly library body block + excitation force "GoTo" block to feed "From" in MPC.
  2. WECCCOMP MPC: same as 1).

The MOST simulation is failing checks as compared to logged data, and does not have any broken library links.
In all observed cases these errors were relatively small but larger than the specified tolerance. I suggest we merge now and revisit/revise these failing tests. I believe this could be the result of the initial optimization steps converging differently.

@kmruehl
Copy link
Contributor

kmruehl commented Sep 12, 2024

@dforbush2 thank you, I'll merge this shortly and we can look into what's going on with the MOST cases

@dforbush2 dforbush2 marked this pull request as draft September 12, 2024 20:01
@dforbush2
Copy link
Contributor Author

This PR is for merge once full directional spectra is working in all applications cases. It is not currently.
I will make another that SHOULD be merged after WEC-Sim Dev is reverted to 6.1 release state to fix the failing controls and WECCCOMP cases.

@kmruehl kmruehl changed the title Application error fixing with v6.1 updates Application updates for v6.2 release Sep 16, 2024
@kmruehl
Copy link
Contributor

kmruehl commented Oct 31, 2024

@dforbush2 what's the status of this PR? It's currently a draft

@dforbush2 dforbush2 marked this pull request as ready for review November 1, 2024 21:26
@kmruehl
Copy link
Contributor

kmruehl commented Nov 27, 2024

@kmruehl will merge this PR using the updated SLX files from @dforbush2's fork once WEC-Sim/WEC-Sim#1332 is merged

@akeeste
Copy link
Contributor

akeeste commented Jan 15, 2025

@dforbush2 what's the status here? Are there any other changes in this PR besides updating slx files with broken library links?

@akeeste akeeste changed the title Application updates for v6.2 release Application updates for v6.1.2 release Jan 15, 2025
@akeeste
Copy link
Contributor

akeeste commented Jan 15, 2025

Seems like this needs one more update for broken links with the main in our latest release. I made these changes and can push to this branch

@akeeste akeeste self-requested a review January 15, 2025 16:53
Copy link
Contributor

@akeeste akeeste left a comment

Choose a reason for hiding this comment

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

This should be updated and working now. Approved for a merge once the tests pass

@akeeste akeeste merged commit 546d4d2 into WEC-Sim:dev Jan 16, 2025
8 of 10 checks passed
@akeeste akeeste mentioned this pull request Jan 16, 2025
akeeste added a commit that referenced this pull request Jan 16, 2025
* try to update libraries again

* fix simscapeinstrumentationlogging error

---------

Co-authored-by: akeeste <[email protected]>
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.

3 participants