-
Notifications
You must be signed in to change notification settings - Fork 29
Define monitoring variables for gemdaqmonitor package #334
Comments
This issue was maybe open as a test before the GitLab migration, although we can start sharing ideas. First, a question: what is the meaning of "Block" in the table? Is it a firmware block? If yes, some metrics don't have a direct corresponding block... Also, what do you think about generating the GEM monitor documentation? The firmware address tables proved to be very successful. The information from the table you suggest would be stored in the monitoring tables. The monitoring data flow I'm considering is the following:
The second part is controlled mainly through the monitoring tables. Path-specific files can be generated from the main monitoring table(s): JSON file for the Web GUI, C++ filters, ... The documentation would be automatically generated for the monitoring table(s). |
Yes, the first idea behind was example, however I tried to do it realistic :)
I meant logical monitoring block. In most of the cases it will correspond to FW block, but as you correctly noted, not all the time.
This is fine to me, however there should be the starting point. Such table as I request can be generated from address table given that all the required attributes are present there and filled accordingly. A caveat: we do not want to monitor everything, so certain selection has to be applied somehow. The data flow you propose sounds reasonable to me and provides a nice extension of the outcome of our discussion, I fully support it. |
Generating the monitoring table from the firmware address might be a good first step, but, as you wrote, it will have to be pruned down. I was thinking more about creating different monitoring XML table(s) which would be under On an organization point of view, the developers and release schemes are different between SW and FW. We are certainly going to have more flexibility and faster releases in software. On a technical point of view, the monitoring variables will probably require much more parameters than want is currently in the firmware address table. I'm not sure it is worth cluttering the firmware address table with many more monitoring-only attributes. Particularly since some metrics won't be firmware registers. |
Google sheet with table of variables (in progress still) all of these values are pulled from the the ctp7_module code (develop branch), the block being the name of the function call and the name being the names of keys in the returned map. I'll have to look into finding out what these values are, but this is at least a full list of what is available in the code now. |
Issue moved to the GitLab project. |
Brief summary of issue
Define a set of monitoring variables for gemdaqmonitor package in a following form:
This table should be added to the gemdaqmonitor documentation
Types of issue
Need-by date
Apr 15, 2020
The text was updated successfully, but these errors were encountered: