Skip to content

Comments

feat: Model to track node states and has_ended flag#94

Merged
yuechao-qin merged 1 commit intomasterfrom
ycq/count_execution_summary
Feb 21, 2026
Merged

feat: Model to track node states and has_ended flag#94
yuechao-qin merged 1 commit intomasterfrom
ycq/count_execution_summary

Conversation

@yuechao-qin
Copy link
Collaborator

@yuechao-qin yuechao-qin commented Feb 17, 2026

TL;DR

Added an ExecutionSummary class to track execution state with comprehensive test coverage.

What changed?

  • Created a new ExecutionSummary dataclass in api_server_sql.py that tracks:
    • Total number of nodes
    • Number of ended nodes
    • Whether the entire execution has ended
  • Added a count_node_status method to update counters based on node status
  • Created a new test file test_api_server_sql.py.

How to test?

Run the new test suite:

pytest tests/test_api_server_sql.py

Why make this change?

This will be used in upstream PRs to calculate execution summary for runs and root nodes.

@yuechao-qin yuechao-qin force-pushed the ycq/count_execution_summary branch from 2dab4d8 to b6873b2 Compare February 17, 2026 22:03
@yuechao-qin yuechao-qin force-pushed the ycq/count_execution_summary branch from b6873b2 to b18d382 Compare February 20, 2026 04:27
@yuechao-qin yuechao-qin requested a review from Ark-kun February 20, 2026 04:33
@yuechao-qin yuechao-qin merged commit 546f75f into master Feb 21, 2026
2 checks passed
Copy link
Collaborator Author

Merge activity

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