-
Notifications
You must be signed in to change notification settings - Fork 40
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
Changing hub height generates unexpected error #84
Comments
This was a good stress test- thanks. There was a subtle bug on the WISDEM side in rescaling the tower, but this should be fixed now. Do a |
yonghoonlee
pushed a commit
to yonghoonlee/WEIS
that referenced
this issue
Feb 15, 2021
1c927a973 Merge pull request WISDEM#192 from WISDEM/f/DV e34bb7c19 enumerate component_id 9ccde772e change opt variables to design variables f2fc9dc5a resolving relative import issue identified in python 3.6 and numpy 3.16 by conda forge d9681f958 fixing new warnings on environment yml a9e2db591 including forgotten __init 3e8672387 trying to get around conda-forge obstacles f1c10d913 Merge branch 'master' into develop cfccd0c80 Merge pull request WISDEM#188 from WISDEM/postprocess_gb ea5bd0f76 minor typo fix 47672563b update docs for improved postprocessing 9cd61370e ensure some of the postprocessing script is included in a test 5cd9a34f5 ennable loading of prior results and plotting 6268d274e cleanup ahead of release 0e9e7383c Merge pull request WISDEM#187 from johnjasa/update_post_docs f79d6c4cd Merge branch 'develop' of https://github.com/WISDEM/WISDEM into f/clean_mod_options 49e5e1f78 Updated compare_designs docs 70db44525 Merge pull request WISDEM#186 from WISDEM/f/clean_mod_options e94601db1 remove 1P and 3P constraint option when there is no rotor 83806f275 enable tower modes away from 1P and 3P modes as a constraint 6f2013fb6 finished doc update for modeling and analysis options files 795e8accc Create python-package-conda.yml 4f5ea4f0b GitHub Actions on wisdem 052ccae2a Merge pull request WISDEM#123 from WISDEM/develop f644b813c restore landbosse defaults 0891c39cc make consistent with simpler AirfoilPreppy reader 6a98511ff removing very old assemblies that are causing confusion c376da38c connecting some more landbosse inputs to ivc for the user e752a0d93 syncing with AirfoilPreppy repo bf751e470 better landbosse connection f57cae245 enforcing ints where needed 5d7f14b92 removing pandas warnings afb017c5b syncing with other branches 9996c8147 syncing to changes in other branches 5bdc45ff8 connecting wake_loss_factor too 5bd976ce4 fixing missing link with FCR in plant_financese 893d7fafe restore tutorial to working even though it will be superceded soon 1dea960d9 adding inits to library for conda 644ab87fe Merge branch 'develop' of github.com:WISDEM/WISDEM into develop 8a3a4c28a Merge pull request WISDEM#92 from nikhar-abbas/develop 69dc0f1e5 trying for more robust testing bff7ff333 trying for more robust testing af9374801 Merge pull request WISDEM#112 from WISDEM/landbosse-second-integration d3781f2c3 adding shapely to list of dependencies 6e5f7303c more robust finding of library directory a6accf9f2 adding sympy to the list of dependencies b79c3b1f7 adding missing init files d6d1a968a LandBOSSE second integration faaa61f0c LandBOSSE second integration ed24d5e89 LandBOSSE second integration cb2972e2a LandBOSSE second integration 143dddd02 LandBOSSE second integration 56261f46a Revert "LandBOSSE second integration" 308ae6332 LandBOSSE second integration f62ba8b06 LandBOSSE second integration 0f7f2348f LandBOSSE second integration a1728463b LandBOSSE second integration a4cc73f79 LandBOSSE second integration e1891098e LandBOSSE second integration 3e65cbb4b LandBOSSE second integration efef90b71 LandBOSSE second integration 0c976e66b LandBOSSE second integration e44a0f7a2 LandBOSSE second integration f4f365324 LandBOSSE second integration cd32b5d0d LandBOSSE second integration 6c5103da0 LandBOSSE second integration e4e2fe72c LandBOSSE second integration 17c954bc5 LandBOSSE second integration eeb5450bc LandBOSSE second integration. c53020987 LandBOSSE second integration a25c3c415 LandBOSSE second integration 90c271eb8 LandBOSSE second integration abae5e858 Merge branch 'develop' into landbosse-second-integration 7b5d52780 LandBOSSE second integration e34c2128e LandBOSSE second integration ab4f7ff67 adding forgotten library elements 24c222b49 Merge branch 'develop' of github.com:WISDEM/WISDEM into develop e65cb7d52 upgrading to ORBIT v0.4.3 60aba74c7 LandBOSSE second integration. 0d92556ab LandBOSSE second integration 46f33e5a3 LandBOSSE second integration 5b745ea6e LandBOSSE second integration 52c2456c7 LandBOSSE second integration e828370f5 LandBOSSE second integration f789ecd8f correct bug introduced in prior commit c747d8c06 update optimization example 08de8d00a LandBOSSE second integration 567c4d3fc LandBOSSE second integration 97df8077c LandBOSSE second integration dff8f7787 syncing with tower variables 4a7df5b44 Merge branch 'develop' into landbosse-second-integration 3272c27da Merge branch 'develop' into landbosse-second-integration 3ed18afb3 LandBOSSE second integration cf4b5ba87 LandBOSSE second integration a16e3afcc LandBOSSE second integration 64df76de9 LandBOSSE second integration 3be25a453 LandBOSSE second integration 09246de0e LandBOSSE second integration ca3b9b7cb LandBOSSE second integration 9e7f39bac LandBOSSE second integration c831a6aa9 LandBOSSE second integration b08f70cdd LandBOSSE second integration dd316d9f8 LandBOSSE second integration 516ba7681 LandBOSSE second integration 7fb9ae64f LandBOSSE second integration 73eef8d02 LandBOSSE second integration 6717fca7d Second LandBOSSE integration 2c37e34a4 Second LandBOSSE integration. a27d26be5 LandBOSSE second integration 7b15e7b97 LandeBOSSE second integration f76f37b19 LandBOSSE second integration e5d3b746e LandBOSSE Second integration 1eb8d2609 LandBOSSE second integration d4f1090db minor code cleanup 6fd620690 use top level flag instead of testing on the indevarcomps 55ce4b3b3 cleaned up prob.model mods to be before setup and allow variables to be set afterwards 26a14ce2b LandBOSSE second integration 6ef482871 Second LandBOSSE integration df92b94af better support user deactivating calculationi of region 2.5 d730c6c1e LandBOSSE second integration a4d1df785 incorporating latest ORBIT 0.4.2 de3c6bc09 LandBOSSE second integration 1fe95f370 LandBOSSE second integration e13674049 LandBOSSE second integration 30515a3d7 Issue WISDEM#98 032650f31 Issues WISDEM#98 WISDEM#100 1ff0a88f2 Issue WISDEM#98 bb1b80087 Issue WISDEM#96 6f6c08628 Issue WISDEM#96 ea5861670 Issue WISDEM#96 7b9f1f718 Issue WISDEM#95 835e26c27 Issue WISDEM#95 404c2fd02 switch to pytest driver for travis 3c13daf22 improving test connections 143279559 adding this directory 2e76f7849 running tests that return correct error code aab603da6 elevating new suction pile option to monopile assemblies c1a8aed40 LandBOSSE second integration 4ff75565b LandBOSSE second integration f590c2a0b LandBOSSE second integration a75806562 LandBOSSE second integration 9b1fbc8ae LandBOSSE second integration ee35ea2fd LandBOSSE second integration 90e3aaad7 LandBOSSE second integration 5720b34ff LandBOSSE second integration a2b3e8486 LandBOSSE second integration c6f34d399 LandBOSSE second integration 83d763ec2 LandBOSSE second integration 392c32e5d LandBOSSE second integration c7fe04ca6 LandBOSSE second integration 98b8b0eab LandBOSSE Second integration 1014942b3 Merge pull request WISDEM#93 from WISDEM/pile_sizing 28ebeaaed LandBOSSE second integration 9f691fe2f LandBOSSE Second integration 55aa44f52 Seocnd LandBOSSE Integration ea1fe3f16 LandBOSSE second integration d9d96a1ec LandBOSSE second integration b2a892de2 LandBOSSE second integration f4395bf5a LandBOSSE second integration c24ffb9a6 Delete XlsxFileOperations.py 8832021b3 LandBOSSE second integration b66af8ec0 LandBOSSE second integration 752b31fe9 adding suctionpile depth as ratio to diameter option 77a2e7aef LandBOSSE second integration d8f37b774 Second LandBOSSE integration b962f5504 LandBOSSE second integration b987f0037 LandBOSSE second integration 72e02abbb LandBOSSE second integration 69706fae7 Update .gitignore 576c725cd connecting tower cost 6ce7e526e improving cost model hookups 847466687 removing dependcy on numpy financial 69689a95a renaming for pytest improvement 6263a27f5 fixed failing test ace47ecc0 hand-merging new regulation trajectory 724a7bf9c incremented version number 8a770daef Merge branch 'develop' for ROSCO compatibility 3b4f1def7 fixing the test setup 1f67e53bd fixing the test setup 945a946db adding to test suite f488f32db this is complete 51b786d3f almost done with new version and testing 5c7626e01 initial testing shows 45 percent improvement in speed 83e696ac8 Include parameter descriptions a57a0babb stashing in this branch 9e93357c7 fixing orbit test issues fad3c6860 fixing ORBIT installation aac60112e error handling for importing ROSCO tools for read/write of the DISCON_in file 7d4cc389c Merge pull request WISDEM#86 from nikhar-abbas/develop 1a547a4ce Merge branch 'develop' into develop 504198f70 make sure flags are passed to ROSCO as integers 2446dea0e Import ROSCO d39ba5300 Update write for ROSCO toolbox version 2.0 - move reader to ROSCO toolbox 827150fc7 cleanup of import statements a1266b99b more robust file reading consistent with python3 c0edb327c adding test to ensure semi still runs 02a5dd4fd correct agent for agents eaf489083 correct agent for agents 4810e1c10 correct markdown tab/spaces a24af3a9b updating install intructions with marmot 7d4fcac39 updating install intructions with marmot 6693282bd Merge pull request WISDEM#81 from WISDEM/tower_bugs 746838ef3 Merge pull request WISDEM#84 from WISDEM/update-to-orbit-0.4.1 465e5cb14 Merge pull request WISDEM#83 from WISDEM/update-to-orbit-0.4.1 8aaba8a70 orbit updated to 0.4.1 89972140e marmot added to setup for orbit 0.4.1 9b7bdb784 Updating orbit library to reflect 0.4.1 changes. 14cd3e984 Updated wisdem/orbit API to 0.4.1 ec46f65d8 bug fixes primarily to turbsim iec runs 5286d2c6f bug fix to pyIECWind, larger grid setting 8f0b4b3b8 casegen_iec flag for larger grid, intended for floating 577d6a6d6 moordyn bug fixes, working 227593733 moordyn read/write fix 2ebb11293 added moordyn read/write eced1fb44 fixed bug in wind loads only being wired up for monopiles fb0aeea0e fixed two small bugs- tower top deflection was not wired up and foundation height is now used for sea floor z-value 31acc2197 make sure all tests run ahead of incremental release 2a428cf57 increasing cargo capacity of future feeder for scaling studies 311d44c8b Merge pull request WISDEM#72 from WISDEM/orbit_integration 5fca2eb3e error handling to for loading the IEA 15mw, differences in how the hub diameter is defined 499c00593 error handling to for loading the IEA 15mw, differences in how the hub diameter is defined 59631c3fa Merge pull request WISDEM#71 from WISDEM/dev-stall_delay 7298258ff added optional stall delay correction to interpolated spanwise airfoil polars, changed drag stall delay model in airfoilprep to remove asymptote at 90 deg git-subtree-dir: WISDEM git-subtree-split: 1c927a9734ac2ccf00b853eb50da7957fa0bd2d2
dzalkind
added a commit
to nikhar-abbas/WEIS
that referenced
this issue
Dec 20, 2021
50f686dd2 Open Loop Control (WISDEM#98) f57ee4553 Merge pull request WISDEM#97 from nikhar-abbas/b/listcheck 6f16a9da6 update listcheck method for numpy arrays aa888a1fc FOCAL Updates (WISDEM#64) de1a7f8e3 another typo! 9650a58e8 fix install typo 08a8c9f67 Merge pull request WISDEM#89 from nikhar-abbas/develop feb50489a Merge remote-tracking branch 'upstream/main' into develop a402e1652 bump version number 2821f729e RAAW Controller (WISDEM#84) 277b637a6 Fix Docs and Minimum Rotor/Generator Speed (WISDEM#83) 6f254299b Update CMakeLsits for VS and ifort compilation (WISDEM#85) 52418867f Merge pull request WISDEM#87 from nikhar-abbas/fix_tests 4d5adb1d9 cleanup, use setuptools b7f2592bc Combine ROSCO and the ROSCO_toolbox (WISDEM#54) git-subtree-dir: ROSCO git-subtree-split: 50f686dd2661756c057f98ade0db1ee5b6e9ece3
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
For the example included in 06_IEA-15-240-RWT, changing hub height generates unexpected error.
When I changed hub height from 150 to 145, or 140, or other value, I see error messages attached below.
Steps to reproduce issue
Please provide a minimum working example (MWE) if possible
python weis_driver.py
Current behavior
Output:
Expected behavior
Code versions
List versions only if relevant
The text was updated successfully, but these errors were encountered: