Clean up SpeedTest output, avoid div-by-0#8340
Merged
mcspr merged 3 commits intoesp8266:masterfrom Oct 16, 2021
Merged
Conversation
Use a formatting function to give more human-readable output formats for flash bandwidth. When the test starts and ends in less than one millisecond, report as "Infinite".
TD-er
reviewed
Oct 16, 2021
TD-er
reviewed
Oct 16, 2021
mcspr
approved these changes
Oct 16, 2021
hasenradball
pushed a commit
to hasenradball/Arduino
that referenced
this pull request
Nov 18, 2024
Use a formatting function to give more human-readable output formats for flash bandwidth. When the test starts and ends in less than one millisecond, report as "Infinite". Sample output: ```` Creating 512KB file, may take a while... ==> Time to write 512KB in 256b chunks = 6641 milliseconds ==> Created file size = 524288 Reading 512KB file sequentially in 256b chunks ==> Time to read 512KB sequentially in 256b chunks = 211 milliseconds = 2.48 MB/s Reading 512KB file MISALIGNED in flash and RAM sequentially in 256b chunks ==> Time to read 512KB sequentially MISALIGNED in flash and RAM in 256b chunks = 212 milliseconds = 2.47 MB/s Reading 512KB file in reverse by 256b chunks ==> Time to read 512KB in reverse in 256b chunks = 367 milliseconds = 1.43 MB/s Writing 64K file in 1-byte chunks ==> Time to write 64KB in 1b chunks = 1249 milliseconds = 52.47 KB/s Reading 64K file in 1-byte chunks ==> Time to read 64KB in 1b chunks = 296 milliseconds = 221.41 KB/s ````
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Use a formatting function to give more human-readable output formats
for flash bandwidth. When the test starts and ends in less than one
millisecond, report as "Infinite".
Fixes #8339
Sample output: