predictive-technology-laboratory / sensus

A Cross-Platform System for Mobile Sensing
http://predictive-technology-laboratory.github.io/sensus
Apache License 2.0
37 stars 39 forks source link

[High Priority] Add Google Fit probe #785

Open Boukhechba opened 5 years ago

rd3fa commented 5 years ago

Reference HealthKit probes

Boukhechba commented 5 years ago

Yes, this is the Android version of HealthKit.

bzd3y commented 5 years ago

@Boukhechba Am I refactoring the healthkit probes to use base probes like all of the other probes? I.e. have a google fit bloodtype probe and the healthkit bloodtype probe both derive from a bloodtypeprobe?

Boukhechba commented 5 years ago

That's a good idea.

bzd3y commented 5 years ago

@Boukhechba Are the default poll sleep times for the iOS healthkit probes supposed to be int.MaxValue? Some of them are set to five days, while some are set to int.MaxValue milliseconds.

bzd3y commented 5 years ago

@Boukhechba Mehdi, could you explain the difference between the Sampling iOS HealthKit probes and normal polling probes?

Boukhechba commented 5 years ago

HealthKit probes read from the healthkit database. Conceptually, it's similar to the calendar probe. It's then not very important when you poll data, because you request it retrospectively. For instance if you poll every day, you will get data gathered in that data. However, normal polling probe will collect data related to that specific moment (e.g. current sound level).

bzd3y commented 4 years ago

I don't have anything to PR for this, but I think I figured out how to fix the problem preventing the Google Fit package from being installed as I mentioned in #977.

This is already fixed in #977 but if that isn't merged, it can be done separately by upgrading the Estimote.Indoor and Estimote.Proximity packages to a version that depends on the Xamarin.Android.Support packages version 28.0.0.0 or higher. Estimote.Android.Indoo version 2.5.4.3 and Estimote.Android.Proximity version 1.0.3.1 should work.