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

add DeepFlavourV01 in CMSSW-DNN C-based format #8

Merged
merged 1 commit into from Aug 4, 2017

Conversation

pablodecm
Copy link
Contributor

This PR adds the same model than #7 but in a new format which is compatible with the latest CMSSW C-based interface of Tensorflow developed by @riga and which will be used in cms-sw/cmssw#19893.

The files contained in this PR contain the latest model for the DeepFlavour tagger, trained for jets up to eta=2.5, based on the CMS Phase 1 detector design, and on simulation of 80 million jets from ttbar and QCD events.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 3, 2017

A new Pull Request was created by @pablodecm (Pablo de Castro) for branch master.

@cmsbuild, @smuzaffar, @mrodozov can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.

external issue cms-sw/cmsdist#3335

@slava77
Copy link

slava77 commented Aug 3, 2017

@pablodecm please remove the other format version after this is merged and the code in CMSSW PR is updated.

@pablodecm
Copy link
Contributor Author

@slava77 I will do another PR with the old format removal then

@mrodozov
Copy link
Contributor

mrodozov commented Aug 4, 2017

Follow the tests here cms-sw/cmsdist#3336

@smuzaffar
Copy link
Contributor

this should be part of next 93X IB (2017-08-07-2330)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants