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

Increased Flexibitility in Bounding #28

Closed
3 tasks done
TimothyRogers opened this issue Nov 4, 2021 · 1 comment
Closed
3 tasks done

Increased Flexibitility in Bounding #28

TimothyRogers opened this issue Nov 4, 2021 · 1 comment
Assignees
@TimothyRogers
Labels
Feature request New feature or request
Milestone
0.1.0

Comments

@TimothyRogers
Copy link
Collaborator

TimothyRogers commented Nov 4, 2021 •
edited
Loading

Currently all methods hardcode sticky bounds for the optimisation space. It would be better if this were instead a call to a user definable function (with defaults) which was a bounding method of the base class.

TODO:

  • Replace current sticky bounds calls with abstract call to self.apply_bounds(pop) -> pop
  • Provide default and example bounding strategies, first move sticky bounds to new pattern
  • Provide template class for bounding strategy
@TimothyRogers TimothyRogers mentioned this issue Nov 4, 2021
Closed
@MDCHAMP MDCHAMP added the Feature request New feature or request label Nov 4, 2021
@MDCHAMP MDCHAMP assigned MDCHAMP and TimothyRogers and unassigned MDCHAMP Nov 4, 2021
@MDCHAMP MDCHAMP added this to the 0.1.0 milestone Nov 4, 2021
@TimothyRogers TimothyRogers mentioned this issue Nov 7, 2021
Merged
@TimothyRogers
Copy link
Collaborator Author

Should be fixed in #29

@MDCHAMP MDCHAMP closed this as completed Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TimothyRogers TimothyRogers

Labels
Feature request New feature or request
Projects
None yet
Milestone
0.1.0
Development

No branches or pull requests
2 participants
@MDCHAMP @TimothyRogers