Skip to content
Snippets Groups Projects

Further move bits from AnalysisConf to the DaVinci configurable

Merged Eduardo Rodrigues requested to merge erodrigu-analysisconf into master

Depends on Analysis!713 (merged).

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
  • Eduardo Rodrigues resolved all threads

    resolved all threads

  • added 1 commit

    • 862e5aac - Move AnalysisConf().initSequence to DaVinci

    Compare with previous version

  • Eduardo Rodrigues resolved all threads

    resolved all threads

  • added 1 commit

    • 369a493c - Fix trivial indentation issue

    Compare with previous version

  • @pkoppenb, if this all shows no issues tomorrow then I will be able to make a final removal from AnalysisConf -> DaVinci and get rid of AnalysisConf completely. This will hopefully help us "see" some structure to think about for the future.

  • Patrick Koppenburg resolved all threads

    resolved all threads

  • Edited by Software for LHCb
  • added 1 commit

    • 65511ac1 - Move import of FlavourTaggingConf to higher up, where needed

    Compare with previous version

  • OK, all seems fine so far apart from a silly mistake - the necessary import statement is now higher up, where needed.

  • added 1 commit

    • a41bcfa7 - Reordering of a couple of import statements

    Compare with previous version

  • Patrick Koppenburg
  • Patrick Koppenburg resolved all threads

    resolved all threads

  • added 1 commit

    Compare with previous version

  • Eduardo Rodrigues resolved all threads

    resolved all threads

  • Seems this and the related PR in Analysis are good to go, @pkoppenb?

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading