UDP-Inout change for multiply cs_surface_deflections and loads/strain#162
Merged
ngoiz merged 2 commits intoImperialCollegeLondon:developfrom Jun 30, 2021
Eriklyy:dev_multi_cs_io
Merged
UDP-Inout change for multiply cs_surface_deflections and loads/strain#162ngoiz merged 2 commits intoImperialCollegeLondon:developfrom Eriklyy:dev_multi_cs_io
ngoiz merged 2 commits intoImperialCollegeLondon:developfrom
Eriklyy:dev_multi_cs_io
Conversation
… possible to have multply cs inputs. Also its possible to send loads and strains now as output. A simple hale aircraft with two cs_surfaces is used as example.
… possible to have multiply cs inputs. Also its possible to send loads and strains now as output. A simple hale aircraft with two cs_surfaces is used as example.
ngoiz
approved these changes
Jun 28, 2021
| logger.error(msg) | ||
| raise IndexError(msg) | ||
|
|
||
| #Had to add this part for for_pos and for_vel since they are arrays. |
Contributor
There was a problem hiding this comment.
Suggested change
| #Had to add this part for for_pos and for_vel since they are arrays. | |
| # Needed for for_pos and for_vel since they are arrays. |
Well caught, just editing the comment to make it a bit more impersonal :)
Contributor
|
I'll test this tomorrow (we have lost our automated testing for the moment) and if it all passes I'll merge into develop. Thanks again! |
Contributor
|
Everything runs. Many thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changed the inout_variables for the UDP connection, in a way that its possible to have multply cs inputs. Also it is possible to send loads and strains now as an output. A simple hale aircraft with two cs_surfaces is used as example.