Add portfolio verdict transition ledger#179
Conversation
saagpatel
force-pushed
the
codex/integration-trend-ledger
branch
from
63c3b36 to
cab270d
Compare
saagpatel
changed the base branch from
codex/integration-scoring-hygiene
to
main
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cab270de6e
ℹ️ 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".
| if latest_index != len(observations) - 1: | ||
| status = "disappeared" | ||
| run_count = 1 | ||
| since_index = latest_index |
There was a problem hiding this comment.
When a repo is missing from the latest snapshot(s), this branch always reports run_count/consecutive_snapshots as 1 and sets since_generated_at to latest_index, which is the last snapshot where the repo was still present. For a history like [present, absent, absent], the ledger says the repo has been disappeared for one snapshot since the present date, understating the disappearance duration and giving consumers the wrong start date; the disappeared streak should be counted from the first absent snapshot after latest_index.
Useful? React with 👍 / 👎.
1 participant
Stack 2/7. Adds the verdict transition ledger and command-center movement summary. Depends on stack 1.