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

Upgrade Muon TF Unpackers #102

Open
mulhearn opened this issue Dec 3, 2015 · 6 comments
Open

Upgrade Muon TF Unpackers #102

mulhearn opened this issue Dec 3, 2015 · 6 comments

Comments

@mulhearn
Copy link
Member

mulhearn commented Dec 3, 2015

The BMTF unpacker is available but not yet in standard sequences.
The OMTF and EMTF unpackers are not yet available.

@mulhearn mulhearn added the URGENT label Dec 3, 2015
@mulhearn
Copy link
Member Author

@jpazzini does the TwinMux unpacker include the inputs to TwinMux?

Also, you've been added to the cms-l1t-offline organization... for now can you just put your code in its own dedicated branch like this:

git remote add cms-l1t-offline [email protected]:cms-l1t-offline/cmssw.git
git status # and note my current local branch
git push cms-l1t-offline (my current local branch):l1t-twinmux-dev-$CMSSW_VERSION

@jpazzini
Copy link

@mulhearn - what the TMunpacker does at the moment is to decode the information received from the TwinMux FEDs (according to the latest version of the TM firmware) into the standard L1MuDTChambPh(Th)Digi format.

Regarding the push to the 'cms-l1t-offline', I'm having some problems.
I'm on the local branch 'TwinMuxUnpacker_CMSSW_800pre4', and I've (successfully) added the remote as you suggested by doing

git remote add cms-l1t-offline [email protected]:cms-l1t-offline/cmssw.git

but if I try to push my branch to 'cms-l1t-offline' by doing

git push cms-l1t-offline TwinMuxUnpacker_CMSSW_800pre4:l1t-twinmux-dev-$CMSSW_VERSION

I got the following error:

ERROR: Permission to cms-l1t-offline/cmssw.git denied to jpazzini.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

@mulhearn
Copy link
Member Author

@jpazzini it seems you did not accept yet the invitation to join the cms-l1t-offline organization, and that is why you don't have the permissions...

About your answer, I guess I am asking does the latest version of the TM firmware include in the information it sends the inputs to Twin Mux as well as its outputs. I can't tell from your answer since the inputs and outputs use the same data format...

@jpazzini
Copy link

@mulhearn - thanks, you're right, I missed the email with the invitation.
The unpacker is supposed to send both the inputs and the outputs; however, currently the outputs are just a copy of the inputs as there is still no firmware implemented to build the "super-primitives".

@mulhearn
Copy link
Member Author

Thanks for the answer! In offline, its the inputs that we need most critically, since there is otherwise no reliable way to bitwise re-emulate the trigger primitives.

@jpazzini
Copy link

@mulhearn - I should make a minor fix (only affecting the text output of a WarningLog actually) to the TMunpaker package. Is it ok if I push it?

@mulhearn mulhearn removed the URGENT label Feb 18, 2016
rekovic pushed a commit that referenced this issue May 22, 2017
Propagate integration updates into the development branch
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

2 participants