Skip to content
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

weather probe not working when air quality is included #308

Open
sc00n opened this issue Jan 16, 2023 · 3 comments
Open

weather probe not working when air quality is included #308

sc00n opened this issue Jan 16, 2023 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@sc00n
Copy link
Contributor

sc00n commented Jan 16, 2023

Weather seems to work ok when I use all sensors excluding air quality. Once I add air quality I don't get anything anymore from weather.

code (with sampling every 10s. In real experiment the sampling will obviously be at a lower frequency).

 AirQualityService airService =
      AirQualityService(apiKey: APIAIR);
      protocol.addConnectedDevice(airService);

      protocol.addTriggeredTask(
          IntervalTrigger(period: Duration(seconds: 10)),
          BackgroundTask()
            ..addMeasure(Measure(type: ContextSamplingPackage.AIR_QUALITY)),
          airService);


WeatherService weatherService =
      WeatherService(apiKey: APIWEATHER);
      protocol.addConnectedDevice(weatherService);
      
      protocol.addTriggeredTask(
          IntervalTrigger(period: Duration(seconds: 10)),
          BackgroundTask()
            ..addMeasure(Measure(type: ContextSamplingPackage.WEATHER)),
          weatherService);
@bardram
Copy link
Contributor

bardram commented Jan 18, 2023

Strange - will investigate.

@bardram bardram self-assigned this Jan 18, 2023
@bardram bardram added the bug Something isn't working label Jan 18, 2023
@bardram
Copy link
Contributor

bardram commented Feb 13, 2023

I cannot reproduce this error. When I configure a protocol to sample both weather and air quality on a regular basis (IntervalTrigger) it seems to work fine.

Two things to note:

  • make sure that you're on the latest v. 0.40.12 version of CAMS (previously there was a bug in the IntervalTrigger)
  • check you weather API key - if you're using the CACHET api key (ending in 844a) it is on a regular basis blocked due to over-use

@bardram bardram added the invalid This doesn't seem right label Feb 13, 2023
bardram added a commit that referenced this issue Feb 13, 2023
@bardram bardram linked a pull request Feb 13, 2023 that will close this issue
bardram added a commit that referenced this issue Feb 13, 2023
…-when-air-quality-is-included

investigation of #308
@bardram
Copy link
Contributor

bardram commented Dec 18, 2023

This is still a problem...

image

@bardram bardram removed the invalid This doesn't seem right label Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants