Skip to content
Snippets Groups Projects

Add test for VP halves alignment

Merged Biljana Mitreska requested to merge upgrade-fix-2019 into master
  • also update version of Phys dependency
Edited by Rosen Matev

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
  • Sascha Stahl
  • Sascha Stahl
  • Biljana Mitreska added 3 commits

    added 3 commits

    Compare with previous version

  • @bimitres, @rmatev, @smalde, Thank you for the notice, I was already following this MR. At first glance, everything looks fine to me, but due to the little overlap time I had with Giulio, I'm not quite sure what would be the appropriate checks to be performed here, besides the nightlies. Would you have any suggestions?

    • Resolved by Rosen Matev

      The type of checks we are initially looking for are for functionality - i.e to test that all parts of the alignment algorithms can continue to run if there are any changes to Rec etc. This will help the alignment package keep up with latest changes. The second level of tests would also be to check the output of the algorithm such that one can check that the results do not go haywire. I think it will be easier for the some of tests to be proposed by the developers especially as their work progresses, however it would be useful if you can be involved. I know that @sstahl and @rmatev had some ideas too.

  • Rosen Matev added 7 commits

    added 7 commits

    Compare with previous version

  • @bimitres I merged the current master as this branch was quite behind and then I fixed a small issue to follow up on Rec!1604 (merged) .

    The test passes for me locally!

    I'll add some more comments today

  • Rosen Matev unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Rosen Matev changed title from WIP: Change of Rec version and added VP Halves alignment test to Add test for VP halves alignment

    changed title from WIP: Change of Rec version and added VP Halves alignment test to Add test for VP halves alignment

  • Rosen Matev changed the description

    changed the description

  • Great, I did not have that issue when making the merge request it's a rather new change.

  • Edited by Software for LHCb
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading