Skip to content

Replace Swiss Ephemeris stub and update provider messaging#6

Open
Jakedoes1111 wants to merge 3 commits intomainfrom
codex/complete-and-finalize-web-app-ey4crf
Open

Replace Swiss Ephemeris stub and update provider messaging#6
Jakedoes1111 wants to merge 3 commits intomainfrom
codex/complete-and-finalize-web-app-ey4crf

Conversation

@Jakedoes1111
Copy link
Owner

Summary

  • remove the Swiss Ephemeris stub aliases so the real module loads on the server and during tests, and add ambient typings for its API
  • refresh system dashboard copy, provider registry hints, and the provider health panel to reflect the bundled calculators and their bootstrap behaviour

Testing

  • npm test -- --run

https://chatgpt.com/codex/tasks/task_e_690c4e61d8ec832d8426ed0e00b1feff

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +86 to +107
const PLANETS: Array<{ id: string; body: Body }> = [
{ id: "Sun", body: Body.Sun },
{ id: "Earth", body: Body.Earth },
{ id: "Moon", body: Body.Moon },
{ id: "Mercury", body: Body.Mercury },
{ id: "Venus", body: Body.Venus },
{ id: "Mars", body: Body.Mars },
{ id: "Jupiter", body: Body.Jupiter },
{ id: "Saturn", body: Body.Saturn },
{ id: "Uranus", body: Body.Uranus },
{ id: "Neptune", body: Body.Neptune },
{ id: "Pluto", body: Body.Pluto },
];

const wrapDegrees = (value: number) => {
const mod = value % 360;
return mod < 0 ? mod + 360 : mod;
};

const computeLongitude = (date: DateTime, body: Body): number => {
const vector = GeoVector(body, MakeTime(date.toUTC().toJSDate()), true);
const ecliptic = Ecliptic(vector);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Avoid using geocentric Earth vector for gate calculation

The HumanDesignGateProvider iterates PLANETS including Body.Earth and feeds each entry directly to GeoVector. For Earth this call returns the observer’s own location (a zero vector), so computeLongitude always produces 0°, meaning the Earth gate is permanently resolved as Gate 1 and the G‑centre is marked defined for every chart. This causes type/authority derivations to be wrong for anyone whose G‑centre should be undefined (e.g. Reflectors), regardless of birth data. The provider should derive the Earth’s gate from the Sun’s longitude (180° opposite) or omit Earth entirely rather than asking Astronomy Engine for Earth’s geocentric coordinates.

Useful? React with 👍 / 👎.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant