Skip to content
Snippets Groups Projects

Streamline digitizer functions

Merged Laurent Mercadier requested to merge bpt_digitizer into master
1 unresolved thread

@lleguy reported an error with this line:

run, ds = tb.load(8196, 181, fields=["XRD_MCP_SMALLraw"])

which showed that several updates of the digitizer functions were necessary.

  • Remove hard-coding of bunch pattern in some functions
  • Get the digitizer type based on classId. This solves the issue above, which occurred because of a wrong assignment of digitizer type based on the name.
  • Update mnemonics to allow digitizer peak extraction
  • Simplify get_digitizer_peaks by only allowing the processing of one mnemonic at a time.
  • Simplify arguments of get_peaks() function, disable loading of data and of bunch pattern table: these should be done prior to this function.
Edited by Laurent Mercadier

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading