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.

Sign up for GitHub

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

Jump to bottom

Implement hyperparameter search techniques in utils #4

Closed
ljvmiranda921 opened this issue Jul 24, 2017 · 5 comments
Closed

Implement hyperparameter search techniques in utils #4

ljvmiranda921 opened this issue Jul 24, 2017 · 5 comments
Labels
enhancement Feature requests help wanted Help is much appreciated

Comments

@ljvmiranda921
Copy link
Owner

No description provided.

@ljvmiranda921 ljvmiranda921 added enhancement Feature requests help wanted Help is much appreciated labels Jul 24, 2017
@SioKCronin
Copy link
Collaborator

I'd like to help with this. Is it ok if we add sklearn as a dependency to take advantage of their GridSearchCV and RandomizedSearchCV?

@ljvmiranda921
Copy link
Owner Author

Hi, sorry I haven't done any commits lately, currently on a trip. I am also thinking about that. But if we are to use scikit-learn's hyperparameter search tools, maybe we can just scrap this feature altogether? Unless we provide a more convenient API in using them then it is fine. What do you think?

@SioKCronin
Copy link
Collaborator

I compared scikit-learn's implementation of GridSearch and RandomizedSearch with Optunity's, and I think it should be straightforward to implement these here. I haven't found another PSO library that has this functionality, and I imagine people would be more apt to tune if it was built in. I can start with GridSearch and see how it goes?

@ljvmiranda921
Copy link
Owner Author

I see I see. Sure! Let's try GridSearch first and iterate from there.

Thanks a lot! @SioKCronin!

@SioKCronin
Copy link
Collaborator

Great. I should be able to get this going next week, and will reach out if I hit any snags.

ljvmiranda921 pushed a commit that referenced this issue Sep 3, 2017
Merge pull request #20 from SioKCronin/master
3c7631f 
Merging pull request #20 fixes a documentation error in pyswarms.single.LocalBestPSO
and adds a new feature as referenced in Issue #4 

Merged by: ljvmiranda921
E-mail: ljvmiranda@gmail.com
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Feature requests help wanted Help is much appreciated
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ljvmiranda921 @SioKCronin