-
-
Notifications
You must be signed in to change notification settings - Fork 531
[#2656 & #2654] Complete rewrite of the multi-level wind UI #2738
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
Merged
Conversation
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
Now the blue does not match the highlight color of the table, but I prefer this coloring scheme
Member
Author
|
We received feedback from TRF that it was unclear whether the altitude was referenced to ground or to mean sea level. Some users also wanted AGL, others wanted MSL. Therefore, I added a commit to allow you to select MSL or AGL referencing: Screen.Recording.2025-03-10.at.00.05.02.mov |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR fixes #2656 and fixes #2654 by completely rewriting the UI of the multi-level wind input. Previously, I attempted to put everything in a
JTable. However, this solution was not flexible and required some hacks, which caused issues in units and decimal rounding. The new solution is to place the input in a dedicated dialog, which includes the wind profile visualization by default.The multi-level panel in the sim dialog contains an overview of the wind levels:

Demo:
Screen.Recording.2025-03-09.at.00.02.28.mp4
Notable changes:
And a bunch of other improvements.
This PR also adds a new "Wind direction" flight data type. It didn't make sense before multi-level wind to include this (it remained constant), but for multi-level winds it can change throughout the flight.