Skip to content

Add option to color flight track by altitude in Top View#2999

Open
KushDutta23 wants to merge 11 commits intoOpen-MSS:developfrom
KushDutta23:develop
Open

Add option to color flight track by altitude in Top View#2999
KushDutta23 wants to merge 11 commits intoOpen-MSS:developfrom
KushDutta23:develop

Conversation

@KushDutta23
Copy link

Summary

This PR adds a new option in the Top View Map Appearance settings to color-code the flight track based on altitude.

Changes

  • Added a checkbox "Color flight track by altitude" in the Top View settings dialog.
  • Implemented altitude-based color rendering using a matplotlib colormap.
  • Default behavior remains unchanged (single color line) when the option is disabled.
  • Added new setting key: "color_by_altitude".

Behavior

  • When enabled, the flight path is rendered with a gradient representing altitude variation.
  • When disabled, the flight path is displayed in the default single color.

Testing

  • Verified that the checkbox appears in the Map Appearance dialog.
  • Confirmed correct rendering when toggling the option.
  • Ensured no changes to existing functionality when disabled.

Closes #

Copy link
Member

@ReimarBauer ReimarBauer left a comment

Choose a reason for hiding this comment

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

Flight track coloring is usually used to represent different data points about the flight.
The legend explains what each color/symbol represents.

Both are missing

Refactor __init__ method to call super().__init__ with parameters.
Refactor constructor to use super() for initialization
Fix incorrect elif condition in mssautoplot causing unreachable else branch
Fix super() call in MFDatasetCommonDims constructor
Added cleanup_connections method to disconnect signals properly when the widget is closed. Refactored parent close handling to call cleanup_connections before emitting the signal.
Implement cleanup_connections and refactor parent close handling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants