[INSTRM-2035] Add xxxxTimes elapsed time keywords, starting with getVisit and updateTelStatus Created: 22/Jul/23  Updated: 24/Mar/24

Status: Open
Project: Instrument control development
Component/s: ics_gen2Actor
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Normal
Reporter: cloomis Assignee: cloomis
Resolution: Unresolved Votes: 0
Labels: EngRun
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

Add keywords declaring the timings of important commands/processes. For the case of the gen2actor, start with getVisit and updateTelStatus, since the underlying calls to the Gen2 reqframes and the opdb inserts have proven to occasionally be problematically slow.

I think the actor should declare when there is a problem (and not the alerts actor), and that the timing keywords can have multiple fields. For example:

getVisitTiming=status,reqframesTime,telStatusTime
    status                  = "OK" or a non-"OK" warning string
    reqframesTime   = the elapsed time for the Gen2 reqframes call
    telStatusTime     = the elapsed time for the tel_status, etc. insert. Maybe NaN if none.

STS could alert on the status string, and allow plotting whichever fields it deems important.

I'm actually proposing this a general mechanism. Think fpsIterationTimes with a handful of major segments (cobra moves, MCS times, calculation times). Or ccdReadTimes or h4RampTimes with at least the header construction and file writing times. Group all into an STS page, maybe?



 Comments   
Comment by arnaud.lefur [ 31/Aug/23 ]

a instrument-wide design needs to be discussed.

Generated at Sat Apr 19 05:19:40 JST 2025 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.