-
Notifications
You must be signed in to change notification settings - Fork 3k
equeue: Added profiling reports to Travis #6378
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
Conversation
027a3f1
to
d828fc0
Compare
/morph build |
Build : SUCCESSBuild number : 1481 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 1126 |
- | | ||
CURR=$(grep -o '[0-9]\+ cycles' prof | awk '{sum += $1} END {print sum}') | ||
PREV=$(curl https://api.github.com/repos/$TRAVIS_REPO_SLUG/status/master \ | ||
| jq -re "select(.sha != \"$TRAVIS_COMMIT\") |
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.
jq: Command-line JSON parser.
Good to know.
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.
It's not on all Ubuntu distros yet, but Travis guarantees it's their machines.
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.
As long as Travis guarantees it 👍
Also, I take it that's one of the reasons why || echo 0
exists?
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.
Oh, that's a trick on my end. Since we're calculating a ratio below, if we fetch an old value of 0, we would divide by zero.
So for any errors, we just treat it as zero, and check it later.
Gonna wait on @adbridge |
Description
Adds equeue profiling reports to Travis. Just like the code size report for LittleFS and inbound Astyle warning reports. You should be able to see the report in the travis-ci/events job.
The equeue make prof also measures code size and size of important structures. But it's hard to report more than one number without cluttering up the Travis statuses.
Pull request type