Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add pm1 and pm10 fields to key map #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aijayadams
Copy link

@aijayadams aijayadams commented Feb 20, 2021

My BlueAir unit returns additional PM fields that arn't mapped in the transform and throw a key error when being processed. Units that don't have the additional fields should not be impacted by their inclusion in the transform map.

Blue Air:
'compatibility': 'classic_280i', 'timezone': 'America/Los_Angeles', 'model': '1.0.9', 'firmware': '1.1.38'

Example API Response:

{'uuid': 'XXXXXXXXXXXXXXXX',
 'start': 1613787478,
 'end': 1613787478,
 'sensors': ['time',
  'pm',
  'pm1',
  'pm10',
  'voc',
  'co2',
  'tmp',
  'hum',
  'allpollu'],
 'units': ['s', 'ugm3', 'ugm3', 'ugm3', 'ppb', 'ppm', 'C', 'pc', '%'],
 'datapoints': [[1613787478,
   0.0,
   0.0,
   0.0,
   296,
   772,
   19.656,
   55.324,
   24.428572]]}

@aijayadams aijayadams marked this pull request as ready for review February 20, 2021 02:30
@zacstewart
Copy link

I came here to make the same PR. @thedjinn I'd love to see this merged

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants