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

Crash in track fitting because of plotting #114

Open
coquelin77 opened this issue Apr 23, 2018 · 4 comments
Open

Crash in track fitting because of plotting #114

coquelin77 opened this issue Apr 23, 2018 · 4 comments
Labels

Comments

@coquelin77
Copy link

Plotting should never cause a crash. Traceback:

Traceback (most recent call last):
  File "/home/daniel/MasterThesis/testbeam_analysis/testbeam_analysis/gui/gui_widgets/worker.py", line 54, in work
    self.main_func(self.args)
  File "/home/daniel/MasterThesis/testbeam_analysis/testbeam_analysis/gui/gui_widgets/plotter.py", line 226, in _get_figs
    **self.kwargs[multi_plot_key])
  File "/home/daniel/MasterThesis/testbeam_analysis/testbeam_analysis/tools/plot_utils.py", line 930, in plot_events
    ax.set_zlim(np.amin(np.array(z)), np.amax(np.array(z)))
  File "/home/daniel/anaconda2/lib/python2.7/site-packages/mpl_toolkits/mplot3d/axes3d.py", line 732, in set_zlim3d
    bottom = self._validate_converted_limits(bottom, self.convert_zunits)
  File "/home/daniel/anaconda2/lib/python2.7/site-packages/matplotlib/axes/_base.py", line 3040, in _validate_converted_limits
    raise ValueError("Axis limits cannot be NaN or Inf")
ValueError: Axis limits cannot be NaN or Inf
@leloup314
Copy link
Member

leloup314 commented May 1, 2018

@coquelin77 Does the GUI actually crash or is the exception caught in the exceptions window? Usually any exception raised during plotting should be caught and handed through to the exception window.

@leloup314 leloup314 added invalid and removed bug labels May 4, 2018
@leloup314
Copy link
Member

I'm closing this since the GUI doesn't crash but behaves as expected and opens an exception window.

@DavidLP
Copy link
Collaborator

DavidLP commented May 4, 2018

It was not a GUI related bug, since the GUI behaved as expected, I agree. But it is a plotting bug that I implemented 2 years ago ;-).

@leloup314
Copy link
Member

Ah, okay, so I'm reopening this then but removing the gui label.

@leloup314 leloup314 reopened this May 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants