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

cannot delete comet 12P/Pons-Brooks via SSO editor #3693

Closed
axd1967 opened this issue Mar 27, 2024 · 6 comments
Closed

cannot delete comet 12P/Pons-Brooks via SSO editor #3693

axd1967 opened this issue Mar 27, 2024 · 6 comments
Labels
state: outdated Outdated problems state: waiting feedback Waiting for user feedback

Comments

@axd1967
Copy link
Contributor

axd1967 commented Mar 27, 2024

Expected Behaviour

(My) comet catalog contains two instances of this comet:

"12P/Pons-Brooks"
"12P/Pons-Brooks (2024)"

Actual Behaviour

Cannot delete the first instance

Steps to reproduce

  1. Go F4 > SSO > Solar System Editor > Solar System

  2. look for 12P/Pons-Brooks (not the "(2024) version
    image

  3. click "Remove"

  4. quit Stellarium

  5. open Stellarium

BUG: removed comet reappears.

System

  • Stellarium version:
    Stellarium 24.4+
    Version 24.4.8-0de39da [master]
    Based on Qt 5.12.8

  • Operating system:

  • Operating System: Kubuntu 20.04
    KDE Plasma Version: 5.18.8
    KDE Frameworks Version: 5.68.0
    Qt Version: 5.12.8
    Kernel Version: 5.4.0-174-generic
    OS Type: 64-bit
    Processors: 8 × Intel® Core™ i7-10510U CPU @ 1.80GHz
    Memory: 15.3 GiB of RAM

  • Graphics Card: <Manufacturer (likely Intel, NVidia, AMD?), Model (HD, Geforce, Radeon..., with model number), driver version?>

  • Screen type (if applicable): Resolution, HighDPI, scaling

Logfile

If possible, attach the logfile log.txt from your user data directory. Look into the Guide for its location.

Workaround

Manually remove the object from ssystem_minor.ini.

@alex-w alex-w added this to Needs triage in Plugin: Solar System Editor via automation Mar 27, 2024
@gzotti
Copy link
Member

gzotti commented Mar 27, 2024

The first entry is of outdated format. Maybe some parser cannot identify it. Yes, just delete it manually. Like all ssystem_minor.ini files before 23.2 (?) have an entry "comet_orbit" about which current versions complain. Ignore the complaint and use the file with older versions, or use different files.

Is that outdated entry included in 24.1?

@axd1967
Copy link
Contributor Author

axd1967 commented Mar 27, 2024

Note: this is technically not a bug, yet a user annoyance, because both comet orbit data versions nearly overlap, making it difficult to read their labels.

image

Data files that use older formats should come with upgrade routines to deal with this kind of issues.
The software pattern is e.g. use a switch case that progresses through all older versions, successively rewriting / upgrading the format until the latest format is reached.

Is that outdated entry included in 24.1?

I removed the entry, can't tell, sorry.

@gzotti
Copy link
Member

gzotti commented Mar 27, 2024

If the new version would just modify the file, it would not be usable with older versions which may be installed. At some point new functionality like format changes which allow better distinction between multiple apparitions of periodical comets creates little annoyances for a little handful of users. I cannot say what caused the non-deletion.

@alex-w
Copy link
Member

alex-w commented Mar 28, 2024

Version 24.4.8-0de39da [master]

You're using the actual version for the first two weeks of January, but the code in the plugin was changed already - please check the latest master

@alex-w alex-w added the state: waiting feedback Waiting for user feedback label Mar 29, 2024
Copy link

Hello @axd1967!

We really need your feedback.

@alex-w alex-w added the state: outdated Outdated problems label Jun 4, 2024
Copy link

github-actions bot commented Jun 4, 2024

This report is outdated, such, the problematic code replaced by new tool or feature...

@github-actions github-actions bot closed this as completed Jun 4, 2024
Plugin: Solar System Editor automation moved this from Needs triage to Done Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state: outdated Outdated problems state: waiting feedback Waiting for user feedback
Development

No branches or pull requests

3 participants