Skip to content

Add Star SD analysis

Bruce Joseph Gallop requested to merge dev_bgallop_AddStarStrobeDelayAna into devel

Rebase of !489 (closed), should pass CI. Started adding tests as well, plus some other comments.

Things to resolve:

  • Should it be StarStrobeDelayAnalysis or StarStrobeDelayFitter (file name, class name, registration name)
  • Where does the update to FE configuration (with the new SD) happen? Probably should be in this class?
  • I would like to remove StarJsonData usage, but I haven't looked through every output. For instance it might be useful to put defects there?

@elebouli @arnaez

Edited by Bruce Joseph Gallop

Merge request reports