Skip to content
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

use pytest instead of nosetest #3524

Merged
merged 1 commit into from
Sep 8, 2019
Merged

Conversation

MarisaKirisame
Copy link
Contributor

nosetest is in maintenance mode, so we should switch to the more modern framework.

@MarisaKirisame
Copy link
Contributor Author

@tqchen if you add 'import topi' to test_codegen_llvm.py, and run it, it will failed. do you has any idea?

@kevinthesun
Copy link
Contributor

#3595 should fix your issue.

@MarisaKirisame MarisaKirisame force-pushed the pytest branch 4 times, most recently from d3c41c0 to eab51b9 Compare September 4, 2019 17:40
@MarisaKirisame
Copy link
Contributor Author

@tqchen @kevinthesun can you guys help review?

@MarisaKirisame MarisaKirisame force-pushed the pytest branch 2 times, most recently from ba8b31f to 11bc95d Compare September 5, 2019 15:34
update test script to use pytest instead of nose

fix mybfloat16

do

save

fix test

fix test
@tqchen tqchen merged commit 184fa48 into apache:master Sep 8, 2019
@tqchen
Copy link
Member

tqchen commented Sep 8, 2019

Thanks @MarisaKirisame

@MarisaKirisame MarisaKirisame deleted the pytest branch September 8, 2019 02:40
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 16, 2019
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 16, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants