Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration of topic branches #100

Closed
mulhearn opened this issue Nov 20, 2015 · 12 comments
Closed

Integration of topic branches #100

mulhearn opened this issue Nov 20, 2015 · 12 comments

Comments

@mulhearn
Copy link
Member

The various topic branches have been combined into an integration branch for 80x

Meanwhile, we are developing a working recipe for TSG studies here

Active developer branches brought into sync periodically:

Deprecated branches with no further expected developer, already integrated are:

Not yet integrated:

@mulhearn
Copy link
Member Author

Noting conflicts from merging stage2 and global dev branches:

        DataFormats/FEDRawData/test/DumpFEDRawDataProduct.cc
        EventFilter/L1TRawToDigi/src/implementations_stage2/CaloTowerUnpacker.cc
        EventFilter/L1TRawToDigi/src/implementations_stage2/GTSetup.cc
        EventFilter/L1TRawToDigi/src/implementations_stage2/GlobalAlgBlkUnpacker.cc
        EventFilter/L1TRawToDigi/src/implementations_stage2/MPUnpacker.cc
        EventFilter/L1TRawToDigi/src/implementations_stage2/MuonPacker.cc
        EventFilter/L1TRawToDigi/src/implementations_stage2/MuonUnpacker.cc
        EventFilter/L1TRawToDigi/utils/unpackData-CaloStage2.py
        L1Trigger/L1TCalorimeter/macros/compHwEmu.C
        L1Trigger/L1TCalorimeter/plugins/L1TStage2CaloAnalyzer.cc
        L1Trigger/L1TCalorimeter/python/caloStage2Params_cfi.py
        L1Trigger/L1TCalorimeter/python/l1tStage2CaloAnalyzer_cfi.py
        L1Trigger/L1TCalorimeter/test/runEmulator-CaloStage2.py

I kept the MuonPacker/MuonUnpacker from muon branch.
I kept the GTSetup/GlobalAlgBlkUnpacker from global branch.
Everything else I used version in Stage-2 branch.

@mulhearn
Copy link
Member Author

L1TMuon/data should already be in an integration build. L1TCalorimeter/data PR is here: cms-data/L1Trigger-L1TCalorimeter#5

@mulhearn
Copy link
Member Author

Noting that we should add in cms-sw#12176 as well, which will conflict with stage-2 updates to CaloParams. OK, this is merged in now, for the record, the conflicts were:

    CondFormats/L1TObjects/interface/CaloParams.h
    L1Trigger/L1TCalorimeter/interface/Stage2Layer2TauAlgorithmFirmware.h
    L1Trigger/L1TCalorimeter/plugins/L1TStage2Layer2Producer.cc
    L1Trigger/L1TCalorimeter/src/firmware/Stage2Layer2DemuxSumsAlgoFirmwareImp1.cc
    L1Trigger/L1TCalorimeter/src/firmware/Stage2Layer2EGammaAlgorithmFirmwareImp1.cc
    L1Trigger/L1TCalorimeter/src/firmware/Stage2Layer2EtSumAlgorithmFirmwareImp1.cc
    L1Trigger/L1TCalorimeter/src/firmware/Stage2Layer2JetAlgorithmFirmwareImp1.cc
    L1Trigger/L1TCalorimeter/src/firmware/Stage2Layer2JetSumAlgorithmFirmwareImp1.cc
    L1Trigger/L1TCalorimeter/src/firmware/Stage2Layer2TauAlgorithmFirmwareImp1.cc
    L1Trigger/L1TCalorimeter/src/firmware/Stage2TowerDecompressAlgorithmFirmwareImp1.cc

These were straightforward to fix by simply keeping the stage2 version of new algorithm but adding Alex swapout of CaloParams for CaloParamsHelper, and moving a few new things from Stage2 into the helper class.

@mulhearn
Copy link
Member Author

For future reference, I bet if I had tried

git merge -Xignore-space-change

that would have gone even easier... given @richard-cms's penchant for whitespace changes ;-)

@mulhearn
Copy link
Member Author

OK, the 80x integration branch is ready.

@mulhearn
Copy link
Member Author

The 76X integration branch is also ready.

@mulhearn
Copy link
Member Author

Looks like something went wrong with the standard sequences integration... missing files. Erasing history of that commit for now in the integration branches.... Fixed.

@mulhearn
Copy link
Member Author

Noting this PR which should be merged into integration branch too:
#105

@mulhearn
Copy link
Member Author

mulhearn commented Jan 5, 2016

Because of the many updates in the dev recipe branch, was easiest to make new integration branches. The 76x version is done and fully up to date with dev recipe branch, but the 80X branch is still in works.

@mulhearn
Copy link
Member Author

mulhearn commented Jan 5, 2016

The 80x integration branch is core dumping when I run our unit tests... investigating. It also missed the L1Trigger/HardwareValidation package. Need to talk to consumers of this one....

@mulhearn
Copy link
Member Author

mulhearn commented Jan 5, 2016

OK, 80x integration branch is now working. Only thing missing wrt 76x is the updates related to L1Trigger/HardwareValidation. Investigating those now.

@mulhearn
Copy link
Member Author

mulhearn commented Feb 2, 2016

Lately the workflow has changed to updating dev-recipes first, then into integration branch via "git rebase --onto"... so this page is obsolete: status of topic branches is maintained in dev-recipes topic. Closing this issue.

@mulhearn mulhearn closed this as completed Feb 2, 2016
@mulhearn mulhearn removed the URGENT label Feb 2, 2016
rekovic pushed a commit that referenced this issue Apr 27, 2017
panoskatsoulis pushed a commit to panoskatsoulis/cmssw that referenced this issue Dec 10, 2018
panoskatsoulis pushed a commit to panoskatsoulis/cmssw that referenced this issue Dec 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant