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

Expose device information in medic-android so we can store it against monthly telemetry readings #64

Closed
SCdF opened this issue Oct 11, 2018 · 6 comments
Labels
Analytics Type: Improvement Make something better

Comments

@SCdF
Copy link
Contributor

SCdF commented Oct 11, 2018

It would be really helpful to be able to add hardware-specific information into our monthly telemetry documents that you can't find out from the browser.

Off the top of my head that could be:

  • Phone make and model
  • android major version and patch level
  • Max and free disk space
  • RAM, CPU, GPU (unless that's covered by make and model? Probably best to be explicit)
  • Network capacity re: their sim card (this is may be hard because we might just catch them at a bad time or whatever. It would have to be a static truth that was representative regardless of when we compiled it).

This would be exposed to medic-webapp via the JS bridge.

Code links to follow telemetry merge.

@garethbowen garethbowen transferred this issue from medic/cht-core Nov 20, 2018
@garethbowen
Copy link
Member

Telemetry is merged, so let's get this added!

@garethbowen garethbowen added the Type: Improvement Make something better label Mar 18, 2019
@kennsippell
Copy link
Member

From medic/cht-core#5540, some memory metrics for consideration are:

  • Runtime.maxMemory
  • ActivityManager.getMemoryClass
  • MemoryInfo.availMemory
  • MemoryInfo.threshold
  • Chrome's performance.memory (although they are quantized)

@tookam
Copy link
Contributor

tookam commented Nov 13, 2019

Ready for AT, medic-android: 64-expose-device-info-to-webapp, cht-core: 64-expose-device-info-to-webapp

@ngaruko ngaruko self-assigned this Nov 13, 2019
@ngaruko
Copy link
Contributor

ngaruko commented Nov 21, 2019

LGTM (storing device info)
image
The CPU info looks somehow inaccurate though. @wtekeu ?

"cpuInfo": {
          "cores": 0
        }

The downSpeed == upSpeed also looks suspicious !

    "network": {
        "downSpeed": 1048576,
        "upSpeed": 1048576
      }

@tookam
Copy link
Contributor

tookam commented Nov 22, 2019

@ngaruko I changed the way I get the number of cores, please give it a try.

Also, downSpeed == upSpeed == 1048576 because they correspond to the max link speed available for your device and connection type. You are probably using wifi of type AC.

@ngaruko
Copy link
Contributor

ngaruko commented Nov 24, 2019

Looks better. Thanks @wtekeu
image

ngaruko pushed a commit to medic/cht-core that referenced this issue Nov 24, 2019
ngaruko pushed a commit that referenced this issue Nov 24, 2019
* Expose device info to webapp via bridge

#64
@ngaruko ngaruko closed this as completed Nov 24, 2019
@ngaruko ngaruko removed their assignment Nov 24, 2019
nijanthanscs pushed a commit to livinggoods/medic-android that referenced this issue Dec 30, 2020
* Expose device info to webapp via bridge

medic#64
nijanthanscs pushed a commit to livinggoods/medic-android that referenced this issue Dec 30, 2020
* Expose device info to webapp via bridge

medic#64
nijanthanscs pushed a commit to livinggoods/medic-android that referenced this issue Dec 30, 2020
* Expose device info to webapp via bridge

medic#64
derickl pushed a commit that referenced this issue Jan 21, 2021
* Expose device info to webapp via bridge

#64
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Analytics Type: Improvement Make something better
Projects
None yet
Development

No branches or pull requests

6 participants