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

Parallel ParticleSwarm #1125

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

Parallel ParticleSwarm #1125

wants to merge 6 commits into from

Conversation

sgaure
Copy link

@sgaure sgaure commented Jan 20, 2025

#1124

This PR enhances ParticleSwarm with a new keyword parallel::Bool = false. If set to true, the objective function will be called with a vector to fill in, and a matrix of column vectors as inputs. Thus, the user may choose to evaluate the swarm in parallel with whatever means.

@sgaure
Copy link
Author

sgaure commented Jan 20, 2025

Btw, I forgot to mention. The commits also contain a change which allows some upper bounds to be equal to the lower bounds. This can be used as a simple way to keep some variables constant.

@pkofod
Copy link
Member

pkofod commented Jan 23, 2025

I see, yes I also implemented something similar in NLSolvers.jl. Let me review it. Thanks

@sgaure
Copy link
Author

sgaure commented Jan 29, 2025

The test failures seem to be unrelated. I came to think of it, should the new parallel keyword be called batched instead? It's not implementing anything parallel.

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

Successfully merging this pull request may close these issues.

2 participants