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

Need much more tests #261

Closed
2 of 3 tasks
henry0312 opened this issue Jan 25, 2017 · 11 comments
Closed
2 of 3 tasks

Need much more tests #261

henry0312 opened this issue Jan 25, 2017 · 11 comments

Comments

@henry0312
Copy link
Contributor

henry0312 commented Jan 25, 2017

It's necessary to write much more tests to keep the quality of LightGBM.
I can contribute to python-package.

TODO:

  • Tests for C++
  • Tests for Python
  • Tests for R
@chivee
Copy link
Collaborator

chivee commented Jan 26, 2017

That will be great! Please help us on the test, and leveraging travis for a better release strategies. we are planing a better versioning for the release, so that the package users can stick to the stable version

@wxchan
Copy link
Contributor

wxchan commented Jan 27, 2017

Adding some tests on multi-class and lambdarank tasks will be great.

@henry0312
Copy link
Contributor Author

I start with python-package from today :)

@Laurae2
Copy link
Contributor

Laurae2 commented Apr 18, 2017

@henry0312 tests are completed for Python I think. For R we need to setup Travis for it.

@henry0312
Copy link
Contributor Author

@Laurae2 no, the current tests doesn't cover all of the python-package.

@wxchan I'd like to use nose. What do you think about nose?

@henry0312
Copy link
Contributor Author

I'll go with nose

@b1shal711
Copy link

i can help with c++ tests

@StrikerRUS
Copy link
Collaborator

@b1shal711 It will be great!

@StrikerRUS
Copy link
Collaborator

Closed in favor of being in #2302. We decided to keep all feature requests in one place.

Welcome to contribute this feature! Please re-open this issue (or post a comment if you are not a topic starter) if you are actively working on implementing this feature.

@StrikerRUS
Copy link
Collaborator

@jameslamb Thanks to all your hard work, I believe we can tick Tests for R now in the original comment. WDYT?

@jameslamb
Copy link
Collaborator

I think so, yes! I think the R tests are in a much better place than they were when this issue was opened

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants