Skip to content

Update performance metrics for iPhone 13 Mini and Galaxy A56#386

Merged
HenryNdubuaku merged 2 commits intomainfrom
jakmro-patch-1
Feb 23, 2026
Merged

Update performance metrics for iPhone 13 Mini and Galaxy A56#386
HenryNdubuaku merged 2 commits intomainfrom
jakmro-patch-1

Conversation

@jakmro
Copy link
Copy Markdown
Collaborator

@jakmro jakmro commented Feb 23, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 23, 2026 04:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the benchmark performance table in the README by adding performance metrics for two mobile devices: iPhone 13 Mini and Galaxy A56. These metrics replace placeholder entries and provide data across three different model benchmarks (LFM2-350m, LFM2-VL-450m, and Moonshine-Base-67m).

Changes:

  • Added complete performance metrics for iPhone 13 Mini across all three benchmark models
  • Added complete performance metrics for Galaxy A56 across all three benchmark models

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

README.md Outdated
| iPad/Mac M1 | - | - | - |
| iPhone 13 Mini | - | - | - |
| Galaxy A56 | - | - | - |
| iPhone 13 Mini | 516tps/65tps (29MB RAM) | 0.3s/69tps (20MB RAM) | 0.7s/413tps (10MB RAM) |
Copy link

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

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

The RAM values for iPhone 13 Mini (29MB, 20MB, 10MB) appear unusually low compared to other devices in the same benchmark table. For instance, the iPad/Mac M2 shows 334MB, 146MB, and 201MB for the same workloads. Even accounting for device differences, the iPhone 13 Mini values seem suspiciously low - particularly the 10MB for Moonshine-Base-67m while achieving 413tps. Please verify these measurements are correct and not missing a digit or using incorrect units.

Suggested change
| iPhone 13 Mini | 516tps/65tps (29MB RAM) | 0.3s/69tps (20MB RAM) | 0.7s/413tps (10MB RAM) |
| iPhone 13 Mini | 516tps/65tps (290MB RAM) | 0.3s/69tps (200MB RAM) | 0.7s/413tps (100MB RAM) |

Copilot uses AI. Check for mistakes.
@HenryNdubuaku HenryNdubuaku merged commit bd484af into main Feb 23, 2026
0 of 2 checks passed
ncylich pushed a commit that referenced this pull request Feb 24, 2026
* Update performance metrics for iPhone 13 Mini and Galaxy A56

Signed-off-by: jakmro <[email protected]>

* Update benchmark table with device specifications

---------

Signed-off-by: jakmro <[email protected]>
Co-authored-by: Henry Ndubuaku <[email protected]>
cattermelon1234 pushed a commit to cattermelon1234/cactus that referenced this pull request Feb 28, 2026
…compute#386)

* Update performance metrics for iPhone 13 Mini and Galaxy A56

Signed-off-by: jakmro <[email protected]>

* Update benchmark table with device specifications

---------

Signed-off-by: jakmro <[email protected]>
Co-authored-by: Henry Ndubuaku <[email protected]>
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.

3 participants