feat(api): add analysis context tracking #14928
Merged
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.
Overview
Closes https://opentrons.atlassian.net/browse/EXEC-399
Evaluate robot context as
In Analysis
when _analyze is running inopentrons.cli.analyze
Changelog
performance_metrics_data
directorytrack_analysis
factory decorator (I think that is the right name for it lol)opentrons.cli.analyze._analyze
withtrack_analysis
decoratorstorage_location
. If the passed Path wants to be a dir that needs to have a file defined or a direct file path, is an implementation detail.RobotContextTracker
define its own file nameTest Plan
performance_metrics_data
directory is getting createdReview requests
None
Risk assessment
Low, although this is wrapping production code, it is gated by a feature flag that defaults to false