Skip to content

feat: added emulator detection (#1021)#1022

Open
mkopcins wants to merge 1 commit intorelease/0.7from
@mkopcins/emulator_detection
Open

feat: added emulator detection (#1021)#1022
mkopcins wants to merge 1 commit intorelease/0.7from
@mkopcins/emulator_detection

Conversation

@mkopcins
Copy link
Copy Markdown
Collaborator

@mkopcins mkopcins commented Mar 27, 2026

Added emulator/simulator detection for logging purposes

Introduces a breaking change?

  • Yes
  • No

Type of change

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation update (improves or adds clarity to existing documentation)
  • Other (chores, tests, code style improvements etc.)

Tested on

  • iOS
  • Android

Testing instructions

  1. Launch any example app (i.e. apps/computer-vision)
  2. Connect to Ru machine via ssh, then cd telemetry && docker compose logs --follow api
  3. Download any model (make sure the model is not currently downloaded)
  4. You should see a log looking something like this:
api-1  | /downloads {
api-1  |   modelName: 'efficientnet-v2-s-quantized',
api-1  |   countryCode: 'US',
api-1  |   isEmulator: true,
api-1  |   libVersion: '0.9.0'
api-1  | }

Added emulator/simulator detection for logging purposes

- [ ] Yes
- [x] No

- [ ] Bug fix (change which fixes an issue)
- [x] New feature (change which adds functionality)
- [ ] Documentation update (improves or adds clarity to existing
documentation)
- [ ] Other (chores, tests, code style improvements etc.)

- [x] iOS
- [x] Android

1. Launch any example app (i.e. apps/computer-vision)
2. Connect to Ru machine via ssh, then `cd telemetry && docker compose
logs --follow api`
3. Download any model (make sure the model is not currently downloaded)
4. You should see a log looking something like this:
```
api-1  | /downloads {
api-1  |   modelName: 'efficientnet-v2-s-quantized',
api-1  |   countryCode: 'US',
api-1  |   isEmulator: true,
api-1  |   libVersion: '0.9.0'
api-1  | }
```

<!-- Add screenshots here, if applicable -->

<!-- Link related issues here using #issue-number -->

- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [ ] My changes generate no new warnings

<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->

---------

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
@mkopcins mkopcins requested a review from chmjkb March 27, 2026 15:47
Copy link
Copy Markdown
Collaborator

@chmjkb chmjkb left a comment

Choose a reason for hiding this comment

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

The code looks good, I'll test it out when I have access to a real Android device, do we need this for telemetry?

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.

2 participants