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.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I propose renaming the current Line.flip() and Line.flip_ip() methods to flip_endpoints() and flip_endpoints_ip(), respectively. The current names are confusing as they could imply that the entire line is flipped(along x or y axis), when in fact only the endpoints are being swapped. This naming convention are in conflict with potential future functions that flip the line along the x or y axes.
The new names better describe the functionality of the methods and will prevent confusion when using them in conjunction with other line flipping methods that may be added in the future.
The text was updated successfully, but these errors were encountered:
I propose renaming the current
Line.flip()
andLine.flip_ip()
methods toflip_endpoints()
andflip_endpoints_ip()
, respectively. The current names are confusing as they could imply that the entire line is flipped(along x or y axis), when in fact only the endpoints are being swapped. This naming convention are in conflict with potential future functions that flip the line along the x or y axes.The new names better describe the functionality of the methods and will prevent confusion when using them in conjunction with other line flipping methods that may be added in the future.
The text was updated successfully, but these errors were encountered: