-
-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add "Highest Measurement" field to log approval page #245
Comments
Maybe easiest way is to use the code we have for Databot (Python). rob Regards, On Sun, Apr 17, 2016 at 7:25 PM, seanbonner notifications@github.com
|
There is supposed to be another github request to embed the Databot diagnostics into the drive import and show these on the tab Process Log - that includes peak and error counts, etc. Pieter
|
Bumping to next given @nokton's request in slack today |
@eitoball's fix is looking good on prod: |
Looks like there's some bugs with this. Check this drive: https://api.safecast.org/en-US/bgeigie_imports/37388 Says highest reading is 141, but mousing over the red spots on the map shows readings over 300. Just saw several like this... |
I am sorry for taking so long to respond. Anyway,
The highest "logged" CPM value is
Yes, there are. For example, dialog says "logged" CPM value (Log CPM) is Do you want to show highest CPM value that is adjusted with "dead-time correction", or highest "logged" CPM value? It seems that it is related to #706. @Frangible could comment if I am wrong. |
Right now we have:
Filename
Number Of Lines
Number Of Measurements
I'd like to add to that
Highest Measurement
Reason being, with the map only it's too easy to miss a few very high readings on a log where 99.9% of the readings are within normal scope which leads to accidental approval regularly. If we can show "Highest Measurement" then the moderator can quickly see if there is reason to look closer at the log before approving or rejecting.
The text was updated successfully, but these errors were encountered: