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

[WIP]Add test module to Gluon-Face #23

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[WIP]Add test module to Gluon-Face #23

wants to merge 5 commits into from

Conversation

PistonY
Copy link
Contributor

@PistonY PistonY commented Jun 17, 2019

I'm going to write a test module and create CI.
Any suggestions could write below.

Start working
@PistonY PistonY added CI CI enhancement New feature or request test_module test_module labels Jun 17, 2019
@PistonY PistonY self-assigned this Jun 17, 2019
@PistonY PistonY changed the title Add test module to Gluon-Face [WIP]Add test module to Gluon-Face Jun 17, 2019
@PistonY
Copy link
Contributor Author

PistonY commented Jun 17, 2019

For now, I want to use Gluon-cv style to write our test module but will use pytest as test runner.
CI maybe use Jenkins+Github cause we need test our datasets and dataloader.
The following module are planning to test.

basic_block
datasets
dataloader
loss func
metrics
model zoo

This would cost one or two week.

@PistonY PistonY requested review from haoxintong and removed request for haoxintong June 17, 2019 06:30
@haoxintong
Copy link
Contributor

I still want to use Green as unittest runner :D

@PistonY
Copy link
Contributor Author

PistonY commented Jun 20, 2019

It's almost same thing. /dog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI CI enhancement New feature or request test_module test_module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants