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

Feature request : seperate satellite names for C-band and KU-band #204

Open
ahmedmoselhi opened this issue Jul 1, 2024 · 7 comments
Open

Comments

@ahmedmoselhi
Copy link

Like in oe-allience sattellites.xml I want the web importer to have an option to make seperate sat names for both C & ku band i.e: 1.0°W Thor 5/6/7 becomes Thor 5/6/7 (KU) and Thor 5/6/7 (C) while using internet import service from lyngsat or kingofsat

@DYefremov
Copy link
Owner

I'll look what can be done, but I can't promise a quick solution.

@DYefremov
Copy link
Owner

DYefremov commented Jul 2, 2024

I had a little time to experiment. :)
split_satellites

I haven't tested it much!

@ahmedmoselhi
Copy link
Author

ahmedmoselhi commented Jul 2, 2024

I had a little time to experiment. :) split_satellites

I haven't tested it much!

I tried using latest commit but can't find split by band option in ui .
Screenshot_20240702_084042

@ahmedmoselhi
Copy link
Author

ahmedmoselhi commented Jul 2, 2024

btw I'm on opensuse tumbleweed with kde 6.1 wayland as main desktop

@DYefremov
Copy link
Owner

Looks like I misunderstood you :) I made this option for importing satellites and not services! I'm not sure if this is a good option in this case, but we'll have to think...

@DYefremov
Copy link
Owner

DYefremov commented Jul 7, 2024

I have experimented a little more...
If you need names merging and separation by band in the satellites list at the startup, then I definitely won't do this option. This is due to the fact that transponder data is loaded only when necessary (lazy). Consequently, in order to divide satellites by band, it is necessary to download data for all transponders of all satellites, which is not good. If we are talking about combining names by sat position only, then this option is quite acceptable.

E.g so:
merge_sat_names

Оr, as an option, an additional button on the toolbar, by clicking on which you can combine satellites.

P.S. There may be other suggestions. I'll be happy to discuss...

@DYefremov
Copy link
Owner

Hmm, experimented with combining names in the sat list. In short, the disadvantages of implementing this feature for me are much greater than the advantages.:( E.g. Saving the last selected satellites, not to mention the need to seriously rework the logic when working with satellites. For now I'll leave it as it is. Perhaps there will be other options later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants