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][TEDD] improve TEDD tests to also run on CPU Docker image #6643
Conversation
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.
Changes look good to me.
Retriggering, due to some CI crash at build time, not related to this PR |
Also cc @zhiics @FrozenGene for reviews and merge if you are happy with this change |
* Amend regular expressions to match with what is being reported by CPU Docker image Graphviz * Fix typo on dependency checking function * Organise imports
Once this is merged, I'd consider it closes #6642. |
If possible, can we merge this? Just asking, so that I can fix my own CI. |
…che#6643) * Amend regular expressions to match with what is being reported by CPU Docker image Graphviz * Fix typo on dependency checking function * Organise imports
…che#6643) * Amend regular expressions to match with what is being reported by CPU Docker image Graphviz * Fix typo on dependency checking function * Organise imports
…che#6643) * Amend regular expressions to match with what is being reported by CPU Docker image Graphviz * Fix typo on dependency checking function * Organise imports
…che#6643) * Amend regular expressions to match with what is being reported by CPU Docker image Graphviz * Fix typo on dependency checking function * Organise imports
…che#6643) * Amend regular expressions to match with what is being reported by CPU Docker image Graphviz * Fix typo on dependency checking function * Organise imports
This change improves TEDD testing, so that is can run on
ci-cpu
Docker image. It currently fails if you try to run it on aci-cpu
image that hasgraphviz
installed. The test currently shows as "passing" in CI, but checks are not actually done.We still need to have a separate PR to install graphviz on
ci-cpu
, to make this test to actually do something. In this PR, we only will be able to obeserve that it doesn't break the current setup. Reported on #6642 (more context there).cc @yongfeng-nv @tqchen for reviews.