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.

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

Add display of sat names in rfplot and rffit #56

Merged
merged 10 commits into from
Nov 28, 2024

Conversation

MartinHerren
Copy link
Collaborator

Hej Cees !

This PR adds the sat names in rfplot (in the plot as well as in the list of ids) and rffit.

In the identified satellites list the name will always be shown in both rfplot and rffit when the name is available in the catalog.
In rfplot's plot, it is not shown by default. It can be activated either by

  • pressing the n key which toggles the display of the name on and off
  • giving the -n parameter at command line

Example of rfplot's window:
Screenshot from 2024-07-14 14-25-00

This PR is based on #54 (and thus also on #52 and #48) so should not be merged before they are merged.

@MartinHerren MartinHerren force-pushed the sat_names branch 2 times, most recently from 6a56f9c to 3f8b076 Compare November 24, 2024 20:22
@MartinHerren MartinHerren changed the base branch from master to dev November 26, 2024 09:52
MartinHerren and others added 10 commits November 26, 2024 17:45
Field is not provisioned yet.

Signed-off-by: Martin Herren (HB9FXX) <[email protected]>
Signed-off-by: Martin Herren (HB9FXX) <[email protected]>
Signed-off-by: Martin Herren (HB9FXX) <[email protected]>
Signed-off-by: Martin Herren (HB9FXX) <[email protected]>
Signed-off-by: Martin Herren (HB9FXX) <[email protected]>
Signed-off-by: Martin Herren (HB9FXX) <[email protected]>
@MartinHerren MartinHerren merged commit ad35a4d into cbassa:dev Nov 28, 2024
@MartinHerren MartinHerren deleted the sat_names branch November 28, 2024 20:59
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.

1 participant