Skip to content

Commit

Permalink
Deprecate ModelDataV2.gpuExecutionTime (commaai#33607)
Browse files Browse the repository at this point in the history
Deprecate gpuExecutionTime
  • Loading branch information
fredyshox authored Sep 20, 2024
1 parent 4893809 commit 580388e
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
8 changes: 4 additions & 4 deletions cereal/log.capnp
Original file line number Diff line number Diff line change
Expand Up @@ -933,7 +933,6 @@ struct ModelDataV2 {
frameDropPerc @2 :Float32;
timestampEof @3 :UInt64;
modelExecutionTime @15 :Float32;
gpuExecutionTime @17 :Float32;
rawPredictions @16 :Data;

# predicted future position, orientation, etc..
Expand All @@ -960,12 +959,13 @@ struct ModelDataV2 {
# Model perceived motion
temporalPose @21 :Pose;

# e2e lateral planner
action @26: Action;

gpuExecutionTimeDEPRECATED @17 :Float32;
navEnabledDEPRECATED @22 :Bool;
locationMonoTimeDEPRECATED @24 :UInt64;

# e2e lateral planner
lateralPlannerSolutionDEPRECATED @25: LateralPlannerSolution;
action @26: Action;

struct LeadDataV2 {
prob @0 :Float32; # probability that car is your lead at time t
Expand Down
2 changes: 1 addition & 1 deletion tools/latencylogger/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ Frame ID: 1202
Execution finished 63.002552
modelV2 published 63.148747
modelV2.modelExecutionTime 23.62649142742157
modelV2.gpuExecutionTime 0.0
modelV2.gpuExecutionTimeDEPRECATED 0.0
plannerd
longitudinalPlan published 69.715999
longitudinalPlan.solverExecutionTime 0.5619999719783664
Expand Down
2 changes: 1 addition & 1 deletion tools/latencylogger/latency_logger.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
}
SERVICE_TO_DURATIONS = {
'camerad': ['processingTime'],
'modeld': ['modelExecutionTime', 'gpuExecutionTime'],
'modeld': ['modelExecutionTime', 'gpuExecutionTimeDEPRECATED'],
'plannerd': ['solverExecutionTime'],
}

Expand Down

0 comments on commit 580388e

Please sign in to comment.