Skip to content

Save XIDs on healthagent restart.#64

Merged
aditigaur4 merged 1 commit into
mainfrom
save_xid
Jun 10, 2026
Merged

Save XIDs on healthagent restart.#64
aditigaur4 merged 1 commit into
mainfrom
save_xid

Conversation

@aditigaur4

Copy link
Copy Markdown
Collaborator

Only discard XID's if they are older than the boot time.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds persistence for GPU XID history so XIDs survive healthagent restarts, while discarding stale history from before the last system boot.

Changes:

  • Persist xid_history to a JSON file under the healthagent run directory.
  • Load prior XID history on module startup, discarding history if it predates the last boot.
  • Trigger background saves when new XIDs are observed (policy callback and field polling).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread healthagent/gpu.py Outdated
Comment thread healthagent/gpu.py
Comment thread healthagent/gpu.py Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread healthagent/gpu.py Outdated
Only discard XID's if they are older than the boot time.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread healthagent/gpu.py
@aditigaur4 aditigaur4 merged commit 81d919f into main Jun 10, 2026
2 checks passed
@aditigaur4 aditigaur4 deleted the save_xid branch June 25, 2026 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants