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

Missing docstrings #469

Closed
5 tasks done
pkofod opened this issue Oct 1, 2017 · 5 comments
Closed
5 tasks done

Missing docstrings #469

pkofod opened this issue Oct 1, 2017 · 5 comments

Comments

@pkofod
Copy link
Member

pkofod commented Oct 1, 2017

We still need docstrings for:

  • Fminbox
  • NewtonTrustRegion
  • SimulatedAnnealing
  • ParticleSwarm
  • NelderMead

The other doc strings should be plenty inspiration if you're in doubt of what a doc string should contain for these types/constructors.

@alexandrebrilhante
Copy link
Contributor

Up for grabs? I can work on this.

@anriseth
Copy link
Contributor

anriseth commented Jul 7, 2018

Up for grabs? I can work on this.

Yes, that would be very helpful :))

@pkofod
Copy link
Member Author

pkofod commented Jul 15, 2018

@brilhana It would be super appreciated! You can look at the existing ones for inspiration (I'm in Greece so I'm not on Github too much, but please just post a PR and we'll review!

pkofod pushed a commit that referenced this issue Jul 22, 2018
* Added docstrings (#469)

* Removed empty lines
@ViralBShah
Copy link
Contributor

Seems like some of these are added. Worth updating the issue?

@pkofod
Copy link
Member Author

pkofod commented Dec 5, 2019

Yeah, it's actually just Fminbox. Should add that.

@pkofod pkofod closed this as completed Sep 4, 2020
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

4 participants