Ticket #3463 (closed: fixed)
Reduce size of MuonAnalysis.cpp
Reported by: | Anders Markvardsen | Owned by: | Nick Draper |
---|---|---|---|
Priority: | minor | Milestone: | Release 2.6 |
Component: | Muon | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Mathieu Doucet |
Description (last modified by Nick Draper) (diff)
The code in
Split MuonAnalysis.cpp into smaller files.
For example move the plot functionality including methods such as plotGroup and plotPair into a separate file MuonAnalysisPlot.h/cpp.
Change History
comment:2 Changed 9 years ago by Robert Whitley
Refs #3463. Reduce the size of MuonAnalysis.cpp
Changeset: fafff54d7833fc0ad47b8cd23b9cf84d8a853312
comment:3 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 32 to Iteration 33
Moved to iteration 33 at iteration 32 code freeze
comment:4 Changed 9 years ago by Robert Whitley
Refs #3463. Reduce size of MuonAnalysis.cpp
Changeset: 539ec0399aa3576c5fcdb725d58c6affb585d3f7
comment:5 Changed 9 years ago by Robert Whitley
Refs #3463. Change recently moved functions to private where possible
Changeset: b98785dbb7c37da2b9a3dc8af5c01fac55df55b8
comment:6 Changed 9 years ago by Robert Whitley
Refs #3463. Remove the middle man (MuonAnalysis) when grouping fits.
Changeset: 02b6d5f1f288cee86044d8323f7f66397d676a54
comment:7 Changed 9 years ago by Robert Whitley
Refs #3463. Remove the middle man (MuonAnalysis) when grouping fits.
Changeset: 02b6d5f1f288cee86044d8323f7f66397d676a54
comment:8 Changed 9 years ago by Robert Whitley
Refs #3463. Remove trailing white spaces and take out unused code.
Changeset: 25280b432b8320baa80ced4a6f847255b3b29c06
comment:9 Changed 9 years ago by Robert Whitley
Refs #3463. No need to manually add workspace to FitPropertyBrowser.
Remove temp "_Raw" workspaces which are now added in MuonAnalysis.
Changeset: cfb74619d323559f679de4e14d94b97a29205de6
comment:10 Changed 9 years ago by Robert Whitley
Refs #3463. Remove trailing white spaces and take out unused code.
Changeset: 25280b432b8320baa80ced4a6f847255b3b29c06
comment:11 Changed 9 years ago by Robert Whitley
Refs #3463. No need to manually add workspace to FitPropertyBrowser.
Remove temp "_Raw" workspaces which are now added in MuonAnalysis.
Changeset: cfb74619d323559f679de4e14d94b97a29205de6
comment:12 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.1 to Release 2.2
Moved at end of release 2.1
comment:13 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.2 to Release 2.3
comment:14 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.3 to Release 2.4
Moved to milestone 2.4
comment:15 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.4 to Release 2.5
Moved at the code freeze for release 2.4
comment:16 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.5 to Release 2.6
comment:17 Changed 7 years ago by Nick Draper
- Owner changed from Robert Whitley to Nick Draper
- Status changed from new to accepted
comment:18 Changed 7 years ago by Nick Draper
- Status changed from accepted to verify
- Resolution set to fixed
There is nothing to test or merge. This is an old ticket that is being clered up. All commits were before the change in workflow and are already in Master.
comment:19 Changed 7 years ago by Mathieu Doucet
- Status changed from verify to verifying
- Tester set to Mathieu Doucet
comment:20 Changed 7 years ago by Mathieu Doucet
- Status changed from verifying to closed
Sounds good.
comment:22 Changed 7 years ago by Nick Draper
- Component changed from Framework to Muon
- Description modified (diff)
comment:23 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 4310
Bulk move of tickets to iteration 31 at the iteration 30 code freeze