Conversation
Signed-off-by: assagman <ahmetsercansagman@gmail.com>
Signed-off-by: assagman <ahmetsercansagman@gmail.com>
Add a pre-flight `ctx.Err()` check before running program steps to return a clear cancellation error instead of starting execution. Signed-off-by: assagman <ahmetsercansagman@gmail.com>
Signed-off-by: assagman <ahmetsercansagman@gmail.com>
add program execution tracing, signature validation and completions routing BREAKING CHANGE: Program.Forward now returns the last module’s prediction (no synthetic merged outputs). - Track per-step execution status, timings, inputs, predictions, and aggregated usage - Add signature compatibility checks (ValidateSignatures / AddModuleValidated) - Route `completions` via a CompletionsConsumer interface (MultiChainComparison opts in) - Update integration + unit tests to match new error messages and output semantics Signed-off-by: assagman <ahmetsercansagman@gmail.com>
Signed-off-by: assagman <ahmetsercansagman@gmail.com>
- Add `ForwardWithTrace` + `ProgramResult` to return both final prediction and full execution trace - Track executions by unique `ExecutionID`, retain last N runs, and expose lookup/getters for step outputs - Validate module outputs against signatures and avoid mutating stored traces via deep-copy helpers - Re-export tracing types/constants in `dsgo` and add `examples/program_tracing` + coverage tests Signed-off-by: assagman <ahmetsercansagman@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ForwardWithTraceandProgramResult, including per-step status, timings, inputs, predictions, and aggregated usage (cost/latency).ValidateSignatures/AddModuleValidated) and deep-copying stored traces to avoid accidental mutation.completionsare passed to the next module when it implements the consumer interface (notably MultiChainComparison).ctxcancellation with a clear error before starting program steps.dsgo, adds a runnableexamples/program_tracing, and updates tests to match new semantics/errors.Breaking Change
Program.Forwardnow returns the last module’s prediction only (no synthetic merged outputs). This may affect callers that relied on merged outputs from earlier steps.