[INSTRM-2013] reported readTimes incorrect Created: 07/Jul/23 Updated: 28/Jul/23 Resolved: 28/Jul/23 |
|
| Status: | Done |
| Project: | Instrument control development |
| Component/s: | ics_hxActor |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Normal |
| Reporter: | cloomis | Assignee: | cloomis |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
The hxActor reports the expected times for a ramp as soon as it knows. That includes the time for a single read (which should be calculated from the ASIC times). That is somehow getting changed/miscalculated: 2023-07-06 07:18:14.245Z cmds 20 Actor.py:524 new cmd: ramp nread=113 visit=96389 exptype=object expectedExptime=1200.0 2023-07-06 07:18:18.760Z cmds 20 CommandLink.py:122 > 2 444 i readTimes=96389,2023-07-05T21:18:18.749665,2023-07-05T21:18:29.606715,10.857 2023-07-06 07:38:46.012Z cmds 20 Actor.py:524 new cmd: ramp finish exptime=1200.091 obstime=2023-07-05T21:18:42.394797 stopRamp 2023-07-06 07:38:57.717Z cmds 20 CommandLink.py:122 > 2 444 i readTime=42.3220 2023-07-06 07:39:23.960Z cmds 20 Actor.py:524 new cmd: ramp nread=31 visit=96390 exptype=object expectedExptime=1200.0 2023-07-06 07:39:27.987Z cmds 20 CommandLink.py:122 > 2 446 i readTimes=96390,2023-07-05T21:39:27.978713,2023-07-05T21:40:10.300723,42.322 |
| Comments |
| Comment by cloomis [ 07/Jul/23 ] |
|
This is calculated from the ASIC configuration registers. It turns out that the "numOutputs" register value changed from 32 to 4 between the two ramps. That register does not get written explicitly during normal operations, and all reads and writes are logged. I did, however, recently add a new call to the ASIC "command" to recalculate the internal ASIC state from those configuration registers. Did not want to, but did so both because another "geometry" register had been seen to change, and to try to make things more unquestionable while investigating the show ramps. |
| Comment by cloomis [ 28/Jul/23 ] |
|
Done as part of |