-
Notifications
You must be signed in to change notification settings - Fork 4
major mods for cmdline params, one script for desktop/android, etc. … #3
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
base: main
Are you sure you want to change the base?
Conversation
…un_test.py is now the main test
| "--iterations", | ||
| type=int, | ||
| default=3, | ||
| help="Number of iterations", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, these arguments are a good addition.
|
@jesup I've pushed this PR to a branch in case you need it directly in the future. |
Also updated the site list
Docs updated to reflect android/desktop capabilities
…scripts into main
Added these options: -specify output path (defaults to "browsertime-results") -specify geckodriver path -specify browsertime path -precache cache shaders when WebRender is specified to avoid first-compile hit -option to restart adb between variants (workaround to adb file descriptor leak)
…un_test.py is now the main test
|
FYI, I've updated this with all my changes (mitm support, etc). It's merged with your current main, and base tests work. lots of irrelevant stuff is in there; focus on run_test.py, desktop.sh, reload.sh, variants_fission.py, perf.sh |
…run_test.py is now the main test