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

explore_data tutorial instrument_view with issue #508

Closed
jorobledo opened this issue Apr 29, 2024 · 3 comments · Fixed by #509
Closed

explore_data tutorial instrument_view with issue #508

jorobledo opened this issue Apr 29, 2024 · 3 comments · Fixed by #509

Comments

@jorobledo
Copy link

Hi all,

Was trying out scippneutron and following up with the tutorial 1_exploring-data.html and encountered two small issues which I worked around. I installed scippneutron by pip install scippneutron[all] in a new pyenv on Ubuntu.

  • When wanting to visualize instrument with scn.instrument_view() missing pythreejs dependency (installed it manually afterwards).
  • Then got an error on the plotting:
    image
    which was solved by modifying scene = fig.children[0].canvas.scene to scene = fig.canvas.scene in instrument_view.py.

Then it worked smoothly., but might as well report it.
Nice job btw.!

@jl-wynen
Copy link
Member

Thanks!

The second issue was actually fixed in #505. We should make a release...

@jl-wynen
Copy link
Member

Concerning the first point, the all extra is a bit misleading. If you want interactive (3d) plots in Jupyter, you need the interactive extra: pip install scippneutron[all, interactive]. I'm going to change this in the docs.

@nvaytet
Copy link
Member

nvaytet commented Apr 29, 2024

The second issue was actually fixed in #505. We should make a release...

I was waiting for #507 to make a release. @MridulS what's the status there?

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

Successfully merging a pull request may close this issue.

3 participants