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

Fix use of a non-integer index in split_path_inout. #3271

Closed

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented Jul 17, 2014

This fixes warnings like:

/usr/lib64/python3.3/site-packages/matplotlib/bezier.py:294: DeprecationWarning: using a non-integer number instead of an integer will result in an error in the future
  path_out = Path(concat([verts_right, path.vertices[i:]]),

because i += len(ctl_points) / 2 is a float. I assume that ctl_points is guaranteed to be divisible by two here.

@tacaswell
Copy link
Member

There are a couple other (unmerged) PRs from @jenshnielsen addressing similar warnings, can you check if this conflicts with them?

@QuLogic
Copy link
Member Author

QuLogic commented Jul 17, 2014

Ah, looks like this (and more) is covered by #3241.

@QuLogic QuLogic closed this Jul 17, 2014
@QuLogic QuLogic deleted the fix-bezier-deprecationwarning branch July 17, 2014 19:27
@tacaswell
Copy link
Member

Thank you anyway! Sorry for the duplicated effort.

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.

None yet

2 participants