Skip to content

Commit

Permalink
fix comment
Browse files Browse the repository at this point in the history
  • Loading branch information
henrysky committed Jan 14, 2025
1 parent ae41132 commit e0c0fdb
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/core/modules/Planet.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1768,7 +1768,7 @@ QVector<const Planet*> Planet::getCandidatesForShadow() const
return res;
}

void Planet::computePosition(const StelObserver *observer, const double dateJDE, const Vec3d &aberrationPush) // heliocentric
void Planet::computePosition(const StelObserver *observer, const double dateJDE, const Vec3d &aberrationPush)
{
// Having hundreds of Minor Planets makes this very slow. Especially on transitions between locations (StelCore::moveObserverTo())
// it seems acceptable to disable position updates for minor bodies.
Expand Down Expand Up @@ -1797,7 +1797,7 @@ void Planet::computePosition(const StelObserver *observer, const double dateJDE,
this->aberrationPush=aberrationPush;
}

void Planet::computePosition(const double dateJDE, Vec3d &eclPosition, Vec3d &eclVelocity) const // heliocentric
void Planet::computePosition(const double dateJDE, Vec3d &eclPosition, Vec3d &eclVelocity) const
{
// if Sun, then position is always 0,0,0 because coordFunc is computing barycentric coordinates
if (this->getParent() == nullptr)
Expand Down
2 changes: 2 additions & 0 deletions src/core/modules/Planet.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -393,10 +393,12 @@ class Planet : public StelObject
double getRotObliquity(double JDE) const;

//! Compute the position and orbital velocity in the parent Planet coordinate system and set aberrationPush
//! In case of the Sun, it is the heliocentric position and velocity so zeros are set
//! Does not compute new position when dateJDE is less than deltaJDE away from lastJDE
//! You can add the aberrationPush value according to Edot*lightTime in Explanatory Supplement (2013) formula 7.55.
virtual void computePosition(const StelObserver *observer, const double dateJDE, const Vec3d &aberrationPush);
//! Compute the position and orbital velocity in the parent Planet coordinate system, and return them in eclPosition and eclVelocity
//! In case of the Sun, it is the heliocentric position and velocity so zeros are set
//! These may be preferred when we want to avoid setting the actual position (e.g., RTS computation)
virtual void computePosition(const double dateJDE, Vec3d &eclPosition, Vec3d &eclVelocity) const;

Expand Down

0 comments on commit e0c0fdb

Please sign in to comment.