-
Notifications
You must be signed in to change notification settings - Fork 4
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
As an air quality expert I would like to update the automatic processing of the low-cost sensors of A22 #27
Comments
- Refactored code to only download data that is actually needed (both stations and datatypes) - Directly load CSV instead of requiring translation in JSON via tool - Add new calibration coefficients - Modify NO2 formula, maintain old formula for old records (new field in processorParameters.csv)
@rcavaliere I've done the requested changes, and fixed some things:
Additional changes:
These changes should prevent future issues when other non-related environment stations get added |
@clezag wonderful! |
- Refactored code to only download data that is actually needed (both stations and datatypes) - Directly load CSV instead of requiring translation in JSON via tool - Add new calibration coefficients - Modify NO2 formula, maintain old formula for old records (new field in processorParameters.csv)
@rcavaliere Still struggling a bit with this issue. I've seen that the averages elaboration requires a certain amount of data points to be present within the 1h time frames. Currently it's 16 records / h. Now both in production and test, the data seems to come in quite irregularly with 5, 10 or even 20 minute intervals between points. Just for testing purposes, I've lowered the number of records / hour to 5 in testing, and some stations now update correctly. Is this parameter of records/hour something documented and intended? |
@clezag yes this control was intended to avoid the elaboration of intervals in which the instrument did not work well, and I would not change this. But the elaboration should be configured in a way, that it should up to the point in which new measurements are available. So, it could happen that certain stations have more uptodate elaborations, and other that stopped at the time in which the last measurements is available. We should ensure that this situation is guaranteed |
@rcavaliere Yes, I think I've fixed this now. It should skip periods without data or without sufficient data quality. Still, all the new stations don't have the necessary update frequency, it seems to be a systemic issue. They update every 5 minutes at best. I think there is a mistake in the configuration, currently it's set up to sync stations every minute, but data only every 5. |
@clezag If I understand well, there are issues in how the Data Collector work. Right? It if is the case, try to fix it so that all stations provide correctly real-time data. Please be aware that some sensors (i.e. the ones for which we have updated the calibration coefficients / equation) are at present offline. |
@rcavaliere turns out both test and production were using the same credentials, clientID and topic. I've configured a different clientId for testing, which means both test and production should now receive each message separately without interference. I've noticed some other things though:
|
@clezag ah this could the trigger of all the issues! |
@rcavaliere Just writing down what we've decided in our meeting:
|
@rcavaliere I think most of the pipeline is working now. The processing elaboration returns 0 for most up to date stations, which happens when the calculated value is below 0. I'm not sure if this is because of outdated parameters or wrong formula. I think we have to verify on a case per case basis. |
@clezag thanks for you work in this sprint. Yes, the processing seems not be stable, but this is something BrennerLEC partners are still working on. Currently we do no have a stable situation since the active sensors do not have calibration coefficients calculated for the processing, and the other sensors which are not active because they are being installed again on the highway have the coefficients computed. I would say, let's close this issue and then open another one again if we still need to fix something |
Released in production |
- Refactored code to only download data that is actually needed (both stations and datatypes) - Directly load CSV instead of requiring translation in JSON via tool - Add new calibration coefficients - Modify NO2 formula, maintain old formula for old records (new field in processorParameters.csv)
The current processing formula for NO2 is: NO2=a0+a1NO2raw^2+a2NO2^raw+a3O3raw^0.1+a4Tint^4
The new formula is now: NO2=a0+a1NO2raw^2+a2NO2^raw+a3*O3raw+a4*Tint^4
This is due to the fact that there are new sensors now, which perform better with this kind of calibration. To do list here:
The text was updated successfully, but these errors were encountered: