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
If colors are set differently for the knots and control points via keyword arguments, these will be ignored if the color c is stated. Here is an example:
It seems that the function eval in make_showable keeps track of the results. I think that the key words are just passt without checks to the vedo interface. Maybe we should not use vedo-keywords as show_options. Use spline_c / spline_color / 'base_color` instead? The keywords are not really consistent. We use full snake-case for some keys but abbreviations for others.
If you use show_options to set options, it will show you as expected. To answer # Uses this instead : why? - as **kwargs are passed to gus.show, it overwrites all the options that's applicable to sampled objects. c is a very common option. We can stop processing show_options from kwargs to avoid confusion.
I know that c is applicable to points and meshes as well, which is exactly why I think we should rename the spline color options c, alpha, etc. to unique splinepy option names. Having duplicates is confusing imo, or we have to set vedo options more explicitly, with something like 'vedo_options=dict`
If colors are set differently for the knots and control points via keyword arguments, these will be ignored if the color
c
is stated. Here is an example:Results in:
![image](https://private-user-images.githubusercontent.com/63095162/287519719-277be64f-9178-4a6a-a948-097b6510c681.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQ0NDM2MDgsIm5iZiI6MTcyNDQ0MzMwOCwicGF0aCI6Ii82MzA5NTE2Mi8yODc1MTk3MTktMjc3YmU2NGYtOTE3OC00YTZhLWE5NDgtMDk3YjY1MTBjNjgxLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MjMlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODIzVDIwMDE0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWU2OWE4YjM4YWI2ODZkNWEyNWJmN2MzM2YzNjI0YTUwZDA3ZDgwMGVmNjMyMWM1YjdjM2Y5NmIxYWNjNDU5Y2UmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.SnqSkq7vH_0JnD5UD_6Z_nxIvhXu1o-3b-2h9J1oBOk)
The text was updated successfully, but these errors were encountered: