Skip to content
This repository was archived by the owner on Jul 27, 2022. It is now read-only.

Added event display for npz event files with endcaps#2

Merged
pdeperio merged 4 commits intoWatChMaL:masterfrom
ABlairJamieson:master
Oct 16, 2020
Merged

Added event display for npz event files with endcaps#2
pdeperio merged 4 commits intoWatChMaL:masterfrom
ABlairJamieson:master

Conversation

@ABlairJamieson
Copy link
Contributor

Features:

  • give geometry file, event file, and event to display as command line arguments
  • if files not give, open file dialogs to select the geometry file or event file
  • three main windows: Charge, Time, or Charge vs Time selected by pressing button
  • button for next event
  • text entry to set event number
  • button to go to next event (or event set in text entry)
  • button and text entry to change z-axis range
  • toolbar to zoom and pan in plot

It runs a bit slow, but works.
Screen Shot 2020-06-17 at 5 31 00 PM

Merge pull request #1 from ABlairJamieson/master
@ABlairJamieson
Copy link
Contributor Author

I added a 3d display as well now.
Screen Shot 2020-06-20 at 10 29 25 PM

@calum-macdonald33
Copy link

@ABlairJamieson would it be possible to retool this to accept the h5 file formats we use in training? It seems straightforward, though we might need to add the 'digi_hits_pmt' variable somehow.

@ABlairJamieson
Copy link
Contributor Author

Anyone going to merge this in?

@pdeperio
Copy link
Contributor

was previous comment by Calum addressed? @calum-macdonald33 @nickwp anything else required before merging?

@nickwp
Copy link
Collaborator

nickwp commented Oct 16, 2020

While it would be nice if this could use the h5 format as Calum suggested, only the npz format actually contains the truth information for showing particle tracks, so changing format would limit the features already included. So I think it's fine to merge now and in future could be nice to add functionality for (limited) display of events from h5 files.

@pdeperio pdeperio merged commit f0d3aa3 into WatChMaL:master Oct 16, 2020
@pdeperio
Copy link
Contributor

does this need to be ported to the new https://github.com/WatChMaL/WatChMaL framework?

@ABlairJamieson
Copy link
Contributor Author

ABlairJamieson commented Oct 16, 2020 via email

@nickwp
Copy link
Collaborator

nickwp commented Oct 16, 2020

Sounds good. Feel free to take a look here to see how we're loading h5 file data in our ML framework, in case you want to reuse this code:
https://github.com/WatChMaL/WatChMaL/blob/master/watchmal/dataset/h5_dataset.py

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants