-
Notifications
You must be signed in to change notification settings - Fork 51
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
Conversation
@dforbush2 thank you, I'll merge this shortly and we can look into what's going on with the MOST cases |
This PR is for merge once full directional spectra is working in all applications cases. It is not currently. |
@dforbush2 what's the status of this PR? It's currently a draft |
@kmruehl will merge this PR using the updated SLX files from @dforbush2's fork once WEC-Sim/WEC-Sim#1332 is merged |
@dforbush2 what's the status here? Are there any other changes in this PR besides updating slx files with broken library links? |
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 |
There was a problem hiding this 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
* try to update libraries again * fix simscapeinstrumentationlogging error --------- Co-authored-by: akeeste <[email protected]>
Adding one fix per commit
Listing broken library links for future reference.
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.