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

gbellmf documentation has parameters mixed up #115

Closed
wemily-amazon opened this issue Aug 2, 2016 · 3 comments
Closed

gbellmf documentation has parameters mixed up #115

wemily-amazon opened this issue Aug 2, 2016 · 3 comments

Comments

@wemily-amazon
Copy link

Docstring says that 'b' is the center, but 'c' is the center.

http://pythonhosted.org/scikit-fuzzy/api/skfuzzy.membership.html#skfuzzy.membership.gbellmf

@JDWarner
Copy link
Collaborator

JDWarner commented Aug 2, 2016

Good catch, thanks for the report! I'll get right on that.

@wemily-amazon
Copy link
Author

Thanks!

I also have gotten strange results, at least in the matplotlib figures, when giving parameters as integers where floats are expected, in this and other mf. It's not really a bug, just a gotcha.

@JDWarner
Copy link
Collaborator

JDWarner commented Aug 3, 2016

@wemily-amazon hmm, that sounds like it might well be a bug. Do you use Python 2.x or 3.x? Many of these functions use division, and if they use the simple / operator this behaves differently if the denominator is float or int (truncates the result if int) in Python 2. Under Python 3 everything is true division.

The fix is from __future__ import division at the top of files, which harmonizes this behavior under Python 2.

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

No branches or pull requests

2 participants