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

ENH: Added option to pass other parameters to optimize #1772

Merged
merged 1 commit into from Mar 21, 2018

Conversation

Projects
None yet
4 participants
@IanEisenberg
Copy link
Contributor

IanEisenberg commented Mar 20, 2018

I ran into the problem of needing to set bounds on parameters for a custom function I wrote (Weibull, with lapse rate), and this change makes that convenient within the normal psychopy framework.

@IanEisenberg IanEisenberg changed the title Added option to pass other parameters to optimize ENH: Added option to pass other parameters to optimize Mar 20, 2018

@coveralls

This comment has been minimized.

Copy link

coveralls commented Mar 20, 2018

Coverage Status

Coverage increased (+0.001%) to 51.768% when pulling 572e466 on IanEisenberg:optimize_flexibility into b8e9449 on psychopy:master.

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Mar 20, 2018

Codecov Report

Merging #1772 into master will decrease coverage by <.01%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1772      +/-   ##
==========================================
- Coverage   47.09%   47.09%   -0.01%     
==========================================
  Files         218      218              
  Lines       33282    33285       +3     
  Branches     5555     5556       +1     
==========================================
+ Hits        15674    15675       +1     
- Misses      16051    16053       +2     
  Partials     1557     1557
Impacted Files Coverage Δ
psychopy/data/fit.py 88.17% <33.33%> (-1.83%) ⬇️
psychopy/app/dialogs.py 27.68% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b8e9449...572e466. Read the comment docs.

@peircej peircej merged commit 23e264a into psychopy:master Mar 21, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
lgtm analysis: Python No alert changes
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.