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

Warning on the website #936

Closed
raycite opened this issue Sep 28, 2024 · 4 comments
Closed

Warning on the website #936

raycite opened this issue Sep 28, 2024 · 4 comments

Comments

@raycite
Copy link

raycite commented Sep 28, 2024

Description

Screenshot from 2024-09-28 12-42-47

Way to reproduce

https://fury.gl/latest/auto_examples/01_introductory/viz_arrow.html#sphx-glr-auto-examples-01-introductory-viz-arrow-py

@WassCodeur WassCodeur self-assigned this Sep 28, 2024
@wasserth
Copy link
Contributor

wasserth commented Oct 9, 2024

Is there any solution to this? These warnings are quite annoying and I did not manage to find out how to call the functions differently to avoid the warnings. The tutorial / examples apparently have also not been updated to the new function definitions.

@skoudoro
Copy link
Contributor

Hi @wasserth,

it seems we missed some update on our tutorials. I will. update that this Friday.

Now we follow pep 3102 : https://peps.python.org/pep-3102/

You just need to be explicit with all function arguments to silent the warnings.

in this specific case:
fury.window.record(scene, out_path="viz_arrow.png", size=(600, 600)) should be
fury.window.record(scene=scene, out_path="viz_arrow.png", size=(600, 600)).

I hope it helps to update your code and avoid those warnings.
Note that those warnings will become an error after several realeases.

@wasserth
Copy link
Contributor

Thanks for the explanations. This resolved the warnings! And thanks a lot for this great library!

@skoudoro
Copy link
Contributor

fixed by #939, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants