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

fixes setting number of cores for assignment with positive integer #231

Merged
merged 1 commit into from
Mar 7, 2021

Conversation

janzill
Copy link
Contributor

@janzill janzill commented Mar 6, 2021

Addresses #230

@pedrocamargo pedrocamargo merged commit 5106cff into master Mar 7, 2021
@pedrocamargo pedrocamargo deleted the janzill/fixes_set_num_cores branch March 7, 2021 03:58
pedrocamargo pushed a commit that referenced this pull request Mar 12, 2021
* stepsize calculation fix, needs to be cythonised
* adds tolerance for root finding and tiny step to avoid stalling for zero stepsize
* use a fraction of the msa stepsize when line search fails
* stepsize tolerance heuristic, needs real-world instance testing
* simple cython stepsize calc
* use partial application to feed in non-stepsize dependent term into optimizer
* fixes setting number of cores for assignment with positive integer (#231)
* gets cost indexed as supernet
* Adds new version to documentation index
pedrocamargo added a commit that referenced this pull request Mar 12, 2021
* stepsize calculation fix, needs to be cythonised
* adds tolerance for root finding and tiny step to avoid stalling for zero stepsize
* use a fraction of the msa stepsize when line search fails
* stepsize tolerance heuristic, needs real-world instance testing
* simple cython stepsize calc
* use partial application to feed in non-stepsize dependent term into optimizer
* fixes setting number of cores for assignment with positive integer (#231)
* gets cost indexed as supernet
* Adds new version to documentation index
MAINLY-authored-by: Jan <jan.christoph.zill@gmail.com>
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