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

Pull requests not acted upon #8

Open
simsong opened this issue Oct 23, 2014 · 16 comments
Open

Pull requests not acted upon #8

simsong opened this issue Oct 23, 2014 · 16 comments

Comments

@simsong
Copy link

simsong commented Oct 23, 2014

Hi. Do you intend to act on the pull requests?

@wvengen
Copy link

wvengen commented Feb 9, 2017

If not, would there be interest in maintaining a community fork with some people?

@cjlin1
Copy link
Owner

cjlin1 commented Feb 9, 2017 via email

@wvengen
Copy link

wvengen commented Feb 13, 2017

@cjlin1 thank you for your reply! I think it's a very good idea to thoroughly test any changes touching the algorithms (even when 'just' parallelizing). Thank you for guarding that.

Given community interest (which I'm probing here as well), I'm hoping there may be some way to get community contributions integrated. I think there are some good improvements in the queue (#14, #15, #18, #28, #29, #30 and probably more), and until they are merged they won't see the light of day in distribution packages, nor will many users will be able to reap those benefits until then.

So my first question would be to @cjlin1: assuming that you see value in improvements by others than yourself and the machine learning group, and assuming that you'd like to guard proper working of the algorithms (and I, for one, very much welcome that), and assuming that you might not have that much time for maintaining this, is there anything we can do to have both?

Some ideas: a team of PR reviewers; a team of community maintainers that assigns issue labels and verifies whether changes are in the build system, platform support, usability, or the algorithms; or starting a semi-official community fork where maintainers agree to only include non-algorithmic changes, which you could merge at your convenience.

If you're not really in for this, perfectly fine, I understand that you have research to attend to. But if you'd welcome community involvement, I'd be happy to see if something can be worked out.

In any case, I'd love to hear if there are other users (and developers) who would be interested in getting this library forward - don't hesitate to 👍 or comment.

@simsong
Copy link
Author

simsong commented Feb 14, 2017

I submitted my pull request more than two years ago, and you only responded to it four days ago. I have since moved on to other projects. Sorry!

@simsong simsong closed this as completed Feb 14, 2017
@hjmjohnson
Copy link

@cjlin1 It is disappointing that these pull requests are not being addressed. Explicitly rejecting the pull requests would be more responsive.

If pull requests are not going to be considered, then perhaps you should add a comment to that effect so that other developers to not expend time trying to improve liblinear.

@simsong
Copy link
Author

simsong commented Jan 21, 2018

I think that we should create a community fork. I am willing to be one of the organizers. I would want to add a number of regression tests and unit tests so that we can be more assured that the requests are reliable.

@simsong simsong reopened this Jan 21, 2018
@simsong
Copy link
Author

simsong commented Jan 21, 2018

If we create a community fork, we will want to move over the issues as well.

@hjmjohnson
Copy link

It seems this may be the only option to develop a community the benefits from each other.

@simsong
Copy link
Author

simsong commented Jan 21, 2018

The other option is to have @cjlin1 make one of us an administrator of this repo.

@hjmjohnson
Copy link

hjmjohnson commented Jan 21, 2018

If @cjlin1 is willing to do that, it would be the most effective and most conducive to supporting the community here. I would recommend the @simsong is made an administrator.

@angleto
Copy link

angleto commented Jan 22, 2018

@simsong I made a fork and a refactoring of the project introducing CMake as build system to compile the project easily on Windows, and I introduced Openmp to parallelize some routine. I hope it is helpful, the fork is: https://github.com/angleto/liblinear

@simsong
Copy link
Author

simsong commented Jan 22, 2018

Great. Can you move over the issues?

@simsong
Copy link
Author

simsong commented Jan 22, 2018

This issue was moved to angleto#21

@simsong simsong closed this as completed Jan 22, 2018
@simsong simsong reopened this Jan 22, 2018
@simsong
Copy link
Author

simsong commented Jan 22, 2018

All open issues have been migrated. @angleto , will you migrate the pull requests?

@angleto
Copy link

angleto commented Jan 23, 2018

Excuse me @simsong, where should I migrate the pull request: which repository/branch?

@simsong
Copy link
Author

simsong commented Jan 23, 2018 via email

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

No branches or pull requests

5 participants