Skip to content

Update FA and switch to poetry #2

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

Merged
merged 3 commits into from
Sep 8, 2021
Merged

Conversation

zStupan
Copy link
Collaborator

@zStupan zStupan commented Sep 8, 2021

Summary

  • Converted to poetry dependency manager
  • Sped up the FA algorithm using numpy
  • Changed FireflyAlgorithm to a function, since the refactored class contained only two methods, one of which was init.

@firefly-cpp
Copy link
Owner

Awesome! It is great to have this project in poetry.

What about the results of current implementation? Are results obtained by this package
similar to the results produced by the original implementation or niapy implementation of FA?

@firefly-cpp firefly-cpp merged commit 333c405 into firefly-cpp:master Sep 8, 2021
@zStupan
Copy link
Collaborator Author

zStupan commented Sep 8, 2021

The results should be the same as the ones produced by the NiaPy implementation.

@zStupan zStupan deleted the update branch September 15, 2021 18:32
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