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

bump zndraw version #4993

Open
PythonFZ opened this issue Sep 12, 2024 · 4 comments
Open

bump zndraw version #4993

PythonFZ opened this issue Sep 12, 2024 · 4 comments

Comments

@PythonFZ
Copy link
Contributor

For the MLIP #498 tutorial we need https://github.com/zincware/ZnDraw/releases/tag/v0.4.7 to visualize the energies alongside the system.

@jngrad
Copy link
Member

jngrad commented Sep 12, 2024

We tried running 0.4.7 two weeks ago and it wasn't working properly. I don't remember the specifics.
@Frieda95 do you remember which issue we experienced?

@PythonFZ
Copy link
Contributor Author

Any updates on this?

@Frieda95
Copy link
Contributor

As far as I remember, ZnDraw wouldn't work if its running on more than one server. The visualisation panel shows a white empty screen. So every time user has to relaunch jupyter notebook to run a new tutorial that involves Zndraw, to ensure its proper functioning.

@PythonFZ
Copy link
Contributor Author

As far as I remember, ZnDraw wouldn't work if its running on more than one server.

Does this only happen with the newest release or isn't this also present in the version currently intended for the tutorial?
ZnDraw was not designed with running the actual server inside Notebooks. Paul did a great job realizing this but there are still some drawbacks. Ideally, the server is run in an independent process via zndraw --port XXXX and then the UI is accessed within the Notebook. In this setup zndraw can handle as many connections as required.

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

No branches or pull requests

3 participants