-
Notifications
You must be signed in to change notification settings - Fork 312
PyVista Jupyter notebook visualizer #852
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
base: master
Are you sure you want to change the base?
Conversation
@Victorwithak It seems that your python case is not in the test set. Your need a CMakeLists.txt file added so that this will be tested before merging. You can check how other python based test cases. Such as https://github.com/Xiangyu-Hu/SPHinXsys/tree/master/tests/test_python_interface/test_2d_dambreak_python , Also, your case better to be in test_python_interface folder together with opther python based tests. |
"metadata": {}, | ||
"outputs": [], | ||
"source": [ | ||
"# %pip install pyvista vtk" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that we need really install the libraries here.
…alizer/Jupyter_Visualizer/test.txt
…alizer/pybind_tool/test.txt
I've finalised the visualizer. I have added the two options of .gif (cross section and full) and some .vtp files to test the jupyter notebook on. I hope I placed it in the correct directory.
I will probably update it this week, since I want to add flow charts to it, but that will come next week.