Skip to content

Conversation

@garrettwrong
Copy link
Collaborator

@garrettwrong garrettwrong commented Apr 20, 2020

I narrowed the windows failures down to loosening atol as below for consistence with other single precision tests. In double precision the test (test_simulation) passed comparison with the original reference result. I am not sure at what point this began to fail, perhaps someone else can comment. The tolerance is a little loose for my taste, but probably fair for singles broad cross-platform support

I scheduled a nightly build for master and develop in the azure-pipelines.yml. This should help us stay current with rolling azure/platform updates, which I think recently broke the pipelines for some time.

I could not find any (yaml) documentation on an automated retry for jobs that fail due to Azure platform issues. These happen occasionally, but more than they should. It seems we can only retry from UI. If it becomes detrimental, we can explore options again.

Removed WIP Draft status. after observed nightly test ran. I removed this branch name from the inclusions in the pipeline yaml (testing purposes) and rebased the commits.

@garrettwrong garrettwrong added bug Something isn't working enhancement New feature or request labels Apr 20, 2020
@garrettwrong garrettwrong requested a review from junchaoxia April 20, 2020 17:24
@garrettwrong garrettwrong self-assigned this Apr 20, 2020
@garrettwrong garrettwrong marked this pull request as ready for review April 21, 2020 12:14
Copy link
Collaborator

@janden janden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@garrettwrong garrettwrong merged commit fedc45d into develop Apr 24, 2020
@garrettwrong garrettwrong deleted the dbg_ci_windows branch May 6, 2020 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants