Repository navigation
feat(bazi): expose native reverse lookup with shared core clocks - #17
Conversation
|
@codex review C++ core-first review is finished: core PR #86 is merged and synchronized to the exact public SHA pinned here. Please review the complete Python diff, especially the append-only private capsule prefix/capability checks, ownership/GIL snapshot, same-runtime forwarding, precise/historical reverse boundaries, and isolated source builds. Before pushing I reviewed the full diff and rebuilt the real extensions in the local venv. All 250 tests pass with local fixtures; sources plus the new reverse tests are Pyright-clean. New regressions cover all chart clocks/rat modes, half-open clipping/merging, physical Jie equality floors, assigned year-500 midnight, equivalent noncanonical Split-JD inputs, errors/closed contexts and independent-context concurrency. Legacy-core BaZi build also succeeds. There is no new tag/version/publication, and this PR should remain open for review. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Swish! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Summary
BaziContext.reverse_lookup_at_ut1(...) -> (candidates, result_flags)and a support query. Return immutable candidate records for maximal matching half-open UT1 intervals. All four pillars are required; fatal errors use the existing exception hierarchy.next_pillar_transitionfor reverse interval boundaries (leave the existing Ziwei point-search helper unchanged), complete shared-clock operator shims, and delegate Split-JD comparison to the base core so noncanonical equivalent dates work.e233c8fadc4754f8a234757841f5a1ad5334bc32and its independently downloaded SHA-256e33401e4cd03bb0e7d93511768b4b36222c58724af65ea655cd8c96253d4d5cd. Isolated CI builds exercise reverse lookup, not just old APIs.Validation
Review order / scope
C++ core PR https://github.com/RedSC1/taiyin-ephemeris-dev/pull/86 is merged and its public snapshot is available, so wrapper review can now proceed. Dart companion PR: RedSC1/dart-ephemeris#37.
No package version bump, tag, publication, or new release workflow was triggered. Keep this PR open for review; do not merge or release yet.