Test fails when device is set to lower resolution

[Silverdimond]

Tested on Galaxy S23 Ultra, if the display is set to 1080p or 720p it fails to parse the resolution

Connected to dm3qxxx

Battery stats reset.
Testing browser firefox...
Traceback (most recent call last):
  File "/home/silver/browser-power-hour/browser-power-hour.py", line 262, in <module>
    test_app(device, app, args.iterations, args.speedometer_timeout)
  File "/home/silver/browser-power-hour/browser-power-hour.py", line 152, in test_app
    search_bar = get_coordinate_string(device, browser_data["searchbar"])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/silver/browser-power-hour/browser-power-hour.py", line 134, in get_coordinate_string
    height = int(dimensions[1])
             ^^^^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: '3088\nOverride size: 1080'

Seems to work fine at the native resolution

[Silverdimond]

I know this isnt related but firefox loves to nag with dialogs like these

oh and i killed the test when it started for chrome

it entered the url into duckduckgo lite (which i set as a simple homepage as the readme said)
maybe instead of using the brightness the screen could be turned off entirely (like scrcpy -S)

[mbestavros]

Whoops - the resolution bug should be fixed now. @Silverdimond try pulling again and re-running?

And yeah, nags are unfortunate... not sure if there's a way around them, though.

[Silverdimond]

Battery stats reset.
Testing browser firefox...
Traceback (most recent call last):
  File "/home/silver/browser-power-hour/browser-power-hour.py", line 262, in <module>
    test_app(device, app, args.iterations, args.speedometer_timeout)
  File "/home/silver/browser-power-hour/browser-power-hour.py", line 152, in test_app
    search_bar = get_coordinate_string(device, browser_data["searchbar"])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/silver/browser-power-hour/browser-power-hour.py", line 134, in get_coordinate_string
    height = int(dimensions[1])
             ^^^^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: '3088\n720'

I think you need to remove the native resolution as well as the newline after it, not just the text saying Override size:

This is what wm size returns in adb shell

dm3q:/ $ wm size
Physical size: 1440x3088
Override size: 720x1544