Skip to content

feat: OnePlus 9R benchmark + install script fixes for clean installs#37

Merged
s1dd4rth merged 1 commit intomainfrom
feat/op9r-benchmark-fixes
Apr 16, 2026
Merged

feat: OnePlus 9R benchmark + install script fixes for clean installs#37
s1dd4rth merged 1 commit intomainfrom
feat/op9r-benchmark-fixes

Conversation

@s1dd4rth
Copy link
Copy Markdown
Owner

Summary

Three things discovered during the OnePlus 9R clean-install demo recording:

  1. OnePlus 9R benchmarkqwen2.5:3b at 6.19 tok/s warm (SD870, 11.2 GB RAM). Second real-device data point. README preflight table + model recommendations updated.

  2. install-ollama.sh: missing nodejs-ltsolladroid new fails on fresh devices because Node.js isn't installed. Added to the pkg install list.

  3. start-ollama.sh: silent exit on some devicesip -4 -o addr show wlan0 fails on the OnePlus 9R, and set -eo pipefail kills the entire script silently before any output. Added || true so it falls through to LOCAL_IP="unknown".

185 tests pass. shellcheck clean.

🤖 Generated with Claude Code

Three things in one PR — all discovered during the clean-install
demo recording on a second device.

1. OnePlus 9R benchmark: qwen2.5:3b at 6.19 tok/s warm on
   Snapdragon 870 / 11.2 GB RAM. Second real-device data point
   alongside the LG G8 (SD855 / qwen2.5:1.5b / 7.38 tok/s).
   README preflight table and model recommendations updated with
   both tested devices.

2. install-ollama.sh: add nodejs-lts to pkg install. Without it,
   `olladroid new` fails with "node is not installed" on a fresh
   device. Caught on the OnePlus 9R clean install.

3. start-ollama.sh: add `|| true` to the `ip -4 -o addr show wlan0`
   pipeline. Without it, `set -eo pipefail` silently kills the
   entire script on devices where the ip command fails (OnePlus 9R
   names the interface differently or restricts netlink sockets).
   The script now falls through to LOCAL_IP="unknown" and the
   server still starts.

185 tests pass. shellcheck clean.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@s1dd4rth s1dd4rth merged commit cdefa63 into main Apr 16, 2026
7 checks passed
@s1dd4rth s1dd4rth deleted the feat/op9r-benchmark-fixes branch April 16, 2026 08:51
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.

1 participant