search

chengqianma / sdenet

MIT License
0 stars 0 forks source link

Missing unit tests? No continuous integration? #5

Open dacb opened 4 years ago

dacb commented 4 years ago

I couldn't find the unit tests or a continuous integration configuration file.

chengqianma commented 4 years ago

No continuous integration (travis CI): We already add the .travis.yml. Since we need GPU to run our program. So it always shows cuda runtime error. image

Unit tests: Add some tests for all api For the model structure part(model module), we can only test the output info which we test it in our test cases. It is hard to find a way to test the intermedia model layers because we don't know what is the desired output.