-
Notifications
You must be signed in to change notification settings - Fork 509
Update Visual Studio projects in dev-tc #3169
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
base: dev-tc
Are you sure you want to change the base?
Conversation
… load calculation. Hydrodyn doesn't need hydrostatic loads when calculating dYdu and perturbing velocities/accelerations. Resulted in 20% speedup.
…rID_None when entering the function. Therefore, each function doesn't need to set ErrMsg = "" at the beginning which reduces some overhead.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
|
|
||
| ErrStat = ErrID_None | ||
| ErrMsg = "" | ||
|
|
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.
Really interesting that this was a performance hit.
…l, have build_windows_executables.bat list executables that are built
This reverts commit 0589eac.
This PR is ready to merge
Feature or improvement description
This PR updates the Visual Studio project files in
vs-buildso OpenFAST and FAST.Farm build correctly. Another update will be required after mergingdevintodev-tc. This migrates the Fortran compiler to IFX, which has been out for a while and Intel doesn't make it easy to get back to IFORT. It also updates to Visual Studio 2022.Impacted areas of the software
Visual Studio solution and project files in
vs-build