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.

Sign up for GitHub

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

Jump to bottom

Optimization cleanup to fix and conform to viz changes. #305

Merged
merged 68 commits into from
Sep 18, 2024
Merged

Optimization cleanup to fix and conform to viz changes. #305

merged 68 commits into from
Sep 18, 2024

Conversation

cfrontin
Copy link
Collaborator

@cfrontin cfrontin commented Aug 1, 2024

Purpose

Making some small bug fixes to the optimization example to conform with and unify some mutual changes from #301.

Type of change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (non-backwards-compatible fix or feature)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Documentation update
  • Maintenance update
  • Other (please describe)

Testing

Changes to load_OMsql tested against failing log SQL files from @mayankchetan and Sora:

/projects/weis/sryu/visualization_cases/1_raft_opt/log_opt.sql
/projects/weis/sryu/visualization_cases/0_control_opt/log_opt.sql

Checklist

Put an x in the boxes that apply.

  • I have run existing tests which pass locally with my changes
  • I have added new tests or examples that prove my fix is effective or that my feature works
  • I have added necessary documentation

@cfrontin
added updated optimization documentation... in progress.
a63fc24
@cfrontin
fixed rendering of optimization rst file
1f3e626
@cfrontin
add partial table to docs optimization page
0473af2
@cfrontin
update to optimization doc.
75c0d43
@cfrontin
Merge branch 'optimization' of github.com:cfrontin/WEIS into optimiza…
b511be7 
…tion
@cfrontin
analysis updates from kestrel
8d6edfd
@cfrontin
added some analysis stuff
d9f8c54
@cfrontin
Merge branch 'optimization' of github.com:cfrontin/WEIS into optimiza…
50f119a 
…tion
@cfrontin
Merge remote-tracking branch 'upstream/optimization' into optimization
f947bd8
@cfrontin
update to optimization docs
c856eb2
@cfrontin
Merge remote-tracking branch 'upstream/optimization' into optimization
6168918
@cfrontin
Merge remote-tracking branch 'upstream/optimization' into optimization
cf3a4cb
@cfrontin
recent kestrel changes
d24d05a
@cfrontin
stuff for local work
c408bbc
@cfrontin
viz toolbox simplfied for inclusion in weis tools
a117c14
@cfrontin
Merge remote-tracking branch 'upstream/optimization' into optimization
f5d9532
@cfrontin
moved viz_toolbox out of examples and refactored
f580177
@cfrontin
corrected accidental git deletion
040dfc5
@cfrontin
updates for kestrel push
f3aae71
@cfrontin
added processor edit for DE/GA
c7eb130
@cfrontin
updates to the viz toolbox and analysis notebook
289661b
@cfrontin
actually update analysis notebook
864f3ed
@cfrontin
actually update analysis notebook with more updates
a4be3a3
@cfrontin
Merge remote-tracking branch 'upstream/optimization' into optimization
a9a48b5
@cfrontin
Merge remote-tracking branch 'upstream/develop' into optimization
a7d67ca
@cfrontin
updated example 17, refactored utilities to match Mayank's files/stru…
377b28e 
…cture
@cfrontin
remove run script
f9d4cef
@cfrontin
Merge branch 'optimization' of github.com:cfrontin/WEIS into optimiza…
8dec7a7 
…tion
@cfrontin
more progress toward PR
ba21e16
@cfrontin
Merge branch 'optimization' of github.com:cfrontin/WEIS into optimiza…
bc1675b 
…tion
cfrontin and others added 16 commits July 24, 2024 13:55
@cfrontin
upenfast updates
573de4f
@cfrontin
removed the weis connections to member ids as om design variables
ee57127
@cfrontin
update openfast analysis
e53ab48
@cfrontin
updated raft analysis notebook
cf0abb1
@cfrontin
improvements to analysis script, some for viz tool update
d6c9fb9
@cfrontin
Merge remote-tracking branch 'upstream/develop' into optimization
424e196
@cfrontin
update to remove remove load_OMsql_temp
fa5e1f3
@cfrontin
rename new example file
f47454a
@cfrontin
updates to get parallel evolutionary methods to work
8fb16c8
@dzalkind
Remove platform_elem_memid discrete input
4262c32
@dzalkind
Add index back in
3f31086
@cfrontin
Merge branch 'optimization' of github.com:cfrontin/WEIS into optimiza…
3e7d853 
…tion
@dzalkind
Set PropPot only if floating
97d990e
@dzalkind
Merge remote-tracking branch 'cory/optimization' into optimization
65a5f93
@cfrontin
viz tools fixed, now using safe yaml loading for problem_vars
eee6ad2
@cfrontin
Merge branch 'optimization' of github.com:cfrontin/WEIS into optimiza…
26a9bfb 
…tion
@dzalkind dzalkind mentioned this pull request Aug 8, 2024
Merged
11 tasks
cfrontin
cfrontin commented Aug 9, 2024
View reviewed changes
examples/17_IEA22_Optimization/driver_weis_openfast_opt.py
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

lol

cfrontin
cfrontin commented Aug 9, 2024
View reviewed changes
Copy link
Collaborator Author

@cfrontin cfrontin left a comment

Choose a reason for hiding this comment

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

Looks great- the argv call is still my go-to way of working with the OF cases fwiw

@dzalkind dzalkind merged commit 397ba52 into WISDEM:develop Sep 18, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dzalkind dzalkind dzalkind approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cfrontin @dzalkind