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

Telescopic asterisms are broken #3315

Closed
alex-w opened this issue Jul 6, 2023 · 15 comments · Fixed by #3992
Closed

Telescopic asterisms are broken #3315

alex-w opened this issue Jul 6, 2023 · 15 comments · Fixed by #3992
Labels
bug Something likely wrong in the code importance: low Small problem, rarely visible, no crash state: published The fix has been published for testing in weekly binary package subsystem: skycultures The issue is related to skycultures of planetarium...
Milestone

Comments

@alex-w
Copy link
Member

alex-w commented Jul 6, 2023

Expected Behaviour

Render telescopic asterisms

Actual Behaviour

Telescopic asterisms cannot be created due problem for finding reference stars

System

  • Stellarium version: 23.2
  • Operating system: macOS 13.4.1

Logfile

log.txt

@alex-w alex-w added this to the 23.3 milestone Jul 6, 2023
@alex-w alex-w added the bug Something likely wrong in the code label Jul 6, 2023
@alex-w
Copy link
Member Author

alex-w commented Jul 7, 2023

I can't reproduce the issue on Windows :-/

@Atque
Copy link
Contributor

Atque commented Jul 7, 2023

I can, but only sometimes. Having run Stellarium for a while, changing sky cultures etc., the problem shows.

Okay, here's how I can reproduce on Windows 64-bit (Qt6):

  1. Start with Modern sky culture
  2. Show asterisms
  3. Change sky culture to a few others
  4. Return to Modern. Now the telescopic asterisms won't show.

This is an old problem, 2 or 3 years at least. I have meant to report it, but I have found no consistent way of reproducing it until now.

@gzotti
Copy link
Member

gzotti commented Jul 7, 2023

I agree it's old and intermittent. There were occasional lines in the logfile about "cannot load TA6" or such, but not always.

@alex-w
Copy link
Member Author

alex-w commented Jul 7, 2023

I see the effect with arm64 binaries only :(

@gzotti gzotti added the importance: low Small problem, rarely visible, no crash label Sep 12, 2023
@alex-w
Copy link
Member Author

alex-w commented Sep 14, 2023

@Atque I guess your Windows machine is a HiDPI-device - is it correct?

@Atque
Copy link
Contributor

Atque commented Sep 14, 2023

@Atque I guess your Windows machine is a HiDPI-device - is it correct?

2560x1440 @ 25", don't know if that qualifies as HiDPI?

@alex-w alex-w modified the milestones: 23.3, 24.1 Sep 14, 2023
@hmartinez82
Copy link
Contributor

hmartinez82 commented Oct 3, 2023

@Atque In Windows:

  • Right click in the desktop anywhere in the wallpaper
  • Select Display Settings
  • Scroll down to Scale & Input
    If Scale is any number above 100% then you do have High DPI. For instance, I have it as 200% in my main desktop and in my personal Windows on ARM laptop.

@Atque
Copy link
Contributor

Atque commented Oct 3, 2023

@hmartinez82 Okay, it says "125% (recommended)", so yeah, then I have an HiDPI monitor.

@10110111
Copy link
Contributor

I reproduce the problem as described in #3552, and here's what I found with the following test:

diff --git a/src/core/modules/Asterism.cpp b/src/core/modules/Asterism.cpp
index 52c480d0f2..3c9eafb0cf 100644
--- a/src/core/modules/Asterism.cpp
+++ b/src/core/modules/Asterism.cpp
@@ -114,6 +114,15 @@ bool Asterism::read(const QString& record, StarMgr *starMgr)
                     qWarning() << "Error in asterism" << abbreviation << "- can't find star with coordinates" << RA << "/" << DE;
                     return false;
                 }
+                if (RA == 3.6803 && DE == 89.1048)
+                { // XXX: debug
+                    const auto pos = s->getJ2000EquatorialPos(core);
+                    StelUtils::rectToSphe(&RA, &DE, pos);
+                    RA *= 12/M_PI;
+                    DE *= 180/M_PI;
+                    qDebug() << "Found a problematic star, actual RA:" << RA << ", DE:" << DE << ", vmag:" << s->getVMagnitude(core);
+                    qDebug().noquote() << "Info: -----------------------------------\n" << s->getInfoString(core);
+                }
                 break;
             }
         }

In my case I consistently get some stars missing, one of them is at RA == 3.6803 && DE == 89.1048 as the patch above tries to catch. I've caught it, and here's the relevant output (reformatted for readability):

Found a problematic star, actual RA: 3.67637 , DE: 89.1101 , vmag: 8.05
Info: -----------------------------------
<font color=#fefeff>
<h2>HIP 17195 - HD 14369 - SAO 460</h2>
Type: <b>star</b><br />
Magnitude: <b>8.05</b><br/>
Absolute Magnitude: 3.22<br />
Color Index (B-V): <b>0.41</b><br />
RA/Dec (J2000.0):     3h40m34.94s/+89°06'36.3"<br/>
RA/Dec (on date):    21h08m23.29s/+54°44'26.1"<br/>
HA/Dec:     6h12m15.14s/+54°44'26.1"  <br/>
Az./Alt.: +322°37'02.9"/-18°17'31.5"  <br/>
Gal. long./lat.: +123°36'02.8"/+26°28'05.9"<br/>
Supergal. long./lat.: +25°45'31.6"/+15°07'10.0"<br/>
Ecl. long./lat. (J2000.0): +88°45'29.0"/+65°49'31.5"<br/>
Ecl. long./lat. (on date): +355°09'52.9"/+64°53'20.7"<br/>
Ecliptic obliquity (on date): +23°26'18.0"<br/>
Mean Sidereal Time: 3h20m38.8s<br/>
Apparent Sidereal Time: 3h20m38.4s<br/>
Rise: 11h59m<br/>
Transit: 15h51m<br/>
Set: 19h44m<br/>
IAU Constellation: UMi<br/>
Distance: 301.44±0.81 ly<br />
Proper motion: 59.92 mas/yr towards 239.2°<br />
Proper motions by axes: -51.45 -30.71 (mas/yr)<br />
Parallax: 10.820±0.029 mas<br />
Spectral Type: F0<br />
Solar Az./Alt.: +196°46'45"/-46°44'13"<br/>
Lunar Az./Alt.: +59°54'12"/+6°02'40"
</font>

This same star, found via F3 as HIP17195, has the same vmag (which confirms it's the correct star), but a very different position and even J2000 RA/dec:

stellarium-011

Why could it have moved?

@alex-w
Copy link
Member Author

alex-w commented Dec 13, 2023

Aberration?

@alex-w
Copy link
Member Author

alex-w commented Dec 13, 2023

Stop! Decimal degrees and decimal hours are different of course

@gzotti
Copy link
Member

gzotti commented Dec 13, 2023

I did not study this code. Aberration causes upto 20 arcsec of positional change. Is the search radius large enough?

@10110111
Copy link
Contributor

Aberration?

Indeed, disabling aberration avoids the errors.

@10110111
Copy link
Contributor

Interestingly, the docs for StarMgr::searchAround (in the .cpp file only) say that it ignores aberration:

// Return a QList containing the stars located
// inside the limFov circle around position vv (in J2000 frame without aberration)
QList<StelObjectP > StarMgr::searchAround(const Vec3d& vv, double limFov, const StelCore* core) const

But what I observe is that aberration does affect the position being checked.

@alex-w alex-w removed this from the 24.1 milestone Mar 9, 2024
@alex-w alex-w added the subsystem: skycultures The issue is related to skycultures of planetarium... label Oct 13, 2024
@alex-w alex-w added this to the 25.1 milestone Dec 11, 2024
@alex-w alex-w added the state: in progress The problem is in process of solution... label Dec 12, 2024
@alex-w alex-w added state: published The fix has been published for testing in weekly binary package and removed state: in progress The problem is in process of solution... labels Dec 29, 2024
Copy link

Hello @alex-w!

Please check the fresh version (development snapshot) of Stellarium:
https://github.com/Stellarium/stellarium-data/releases/tag/weekly-snapshot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something likely wrong in the code importance: low Small problem, rarely visible, no crash state: published The fix has been published for testing in weekly binary package subsystem: skycultures The issue is related to skycultures of planetarium...
Development

Successfully merging a pull request may close this issue.

5 participants