Skip to content

refactor: Run Update Mechanics (ENG-1296)#53

Merged
monoxgas merged 5 commits into
mainfrom
nick/eng-1296-sdk-runupdate-span-pacing
May 29, 2025
Merged

refactor: Run Update Mechanics (ENG-1296)#53
monoxgas merged 5 commits into
mainfrom
nick/eng-1296-sdk-runupdate-span-pacing

Conversation

@monoxgas
Copy link
Copy Markdown
Contributor

@monoxgas monoxgas commented May 15, 2025

  • Refactored run update mechanics
  • Extended data updates to include object, object schemas, inputs, and outputs
  • Improved frequency of automatic data updates

This should create a better experience when monitoring in-progress runs in the UI and querying their data with the API.


Generated Summary:

  • Updated error logging in util.py to avoid logging for missing objects and schemas in "pending" runs.
  • Modified the push_update method in Dreadnode to clarify its purpose and ensure data is pushed immediately if called within a run.
  • Introduced forced updates for run data to improve UI responsiveness by automatically sending updates when tasks complete.
  • Enhanced data handling in RunSpan to manage pending parameters, metrics, inputs, outputs, objects, and object schemas more effectively.
  • Implemented a strategy in RunSpan to check update frequency and handle large attributes separately during updates.
  • Updated documentation and method signatures for clarity, aligning them with new internal mechanics for maintaining state and updates.
  • These changes are expected to enhance the tracking of run data and improve overall system responsiveness, especially in long-running tasks or complex object management scenarios.

This summary was generated with ❤️ by rigging

@monoxgas monoxgas requested a review from briangreunke May 15, 2025 00:24
@monoxgas monoxgas merged commit 9ebf762 into main May 29, 2025
8 checks passed
@monoxgas monoxgas deleted the nick/eng-1296-sdk-runupdate-span-pacing branch May 29, 2025 18:47
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.

1 participant