Retirement of MooreAnalysis
FYI @erodrigu @rmatev @amathad
Decisions made in Analysis#52 (closed)
As discussed there, Analysis
is being retired and FunTuple
is being moved to DaVinci
. HltEfficiencyChecker
(the original use case for MooreAnalysis
) depends on FunTuple
, and so must therefore move to DaVinci
with it. Furthermore, since HltEfficiencyChecker
is the last remaining package/code in MooreAnalysis
, this project can be retired once the move is complete.
Work has been ongoing this week in preparation for the move. The remaining timeline of work required is:
-
Merge !134 (merged) (and dependent Moore MR; doc update) to close #33 (closed) and #44 (closed) (the last two open issues). -
Merge !131 (merged) and !132 (merged) to get them out of the way before the move. These two do not conflict, and are both ready to merge. They also should not conflict with !134 (merged).
(the above merges can all proceed in parallel/their order is not important as they do not conflict)
-
Rebase/update DaVinci!1005 (merged) to reflect above merges. -
Make MooreAnalysis MR to remove HltEfficiencyChecker. -
Remake !130 (closed) in DaVinci on top of DaVinci!1005 (merged), as !130 (closed) doesn't look like it will be ready to merge before HltEfficiencyChecker is moved to DaVinci. Close !130 (closed). At this point there'll be no open MRs to MooreAnalysis and no open issues apart from this one and the MR to remove HltEfficiencyChecker. -
Merge MooreAnalysis MR removing HltEfficiencyChecker, DaVinci moving MR and Moore!2812 (merged) (MR for docs changes) together, et voila. -
Retire MooreAnalysis (not sure what that means in practice for e.g. nightly builds, as I guess we don't actually delete the repo)
Edited by Ross John Hunter