Skip to content

Conversation

@karulis
Copy link

@karulis karulis commented Jan 9, 2019

Enabling(24h only) CAQI index calculation

Copy link
Owner

@hrbonz hrbonz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also provide a reference to the standard in the README? That helps with reviewing the code



class AQI(PiecewiseAQI):
"""Implementation of the EPA AQI algorithm.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix the name here

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Name or the comment?
Because class name(AQI) seems to be used in get_algo implementation?

return mod.AQI()

POLLUTANT_NO2_1H: Decimal('1.'),
},
'units': {
POLLUTANT_O3_1H: 'µg/m³',
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are those units correct? O3, CO, SO2, NO2 are traditionally measured in ppm/ppb

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(26, 50),
(51, 75),
(76, 100),
(101, 1000)],
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I should probably think about adding code to handle open scales like this one. Let me know if you have suggestions here.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No suggestions yet:)
Interesting thing is that I did not find algorithm for calculating CAQI in https://www.airqualitynow.eu/download/CITEAIR-Comparing_Urban_Air_Quality_across_Borders.pdf
But I checked output of Your algorithm and it matches official measurement in my city.

@karulis
Copy link
Author

karulis commented Jan 11, 2019

I have updated README with CAQI
Now I see there is even newer aqi promoted by EU - EAQI: http://confluence.hyds.es/display/A4CATDOC/European+Air+Quality+Index
If/when I will have time I will implement it as well

@djjudas21
Copy link

Hey, this thread is old but is there any chance of merging this? Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants