-
Notifications
You must be signed in to change notification settings - Fork 3k
TEST update : tests-mbed_hal-common_tickers_freq #6573
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
TEST update : tests-mbed_hal-common_tickers_freq #6573
Conversation
The changes looks ok. Only the modification of the host test can not be performed in this way. The host test is used by other tests which expects 5% tolerance. So I think host test should be modified, so the tolerance can be configured or we need a copy of it adapted to the 10% tolerance. |
9e881f4
to
05b61a0
Compare
As host part is not really need for my tests, I removed this part from this PR. |
/morph build |
@jeromecoutant What's the reason for migrating from |
Nvm, re-read the description. |
Build : SUCCESSBuild number : 1691 Triggering tests/morph test |
Exporter Build : FAILUREBuild number : 1323 |
Test : SUCCESSBuild number : 1485 |
/morph export-build |
Exporter Build : SUCCESSBuild number : 1329 |
Description
For me, tests-mbed_hal-common_tickers_freq needs few update:
Thx
@mprse @c1728p9 @bulislaw
Pull request type
[x] Fix
[ ] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change