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 DeepFlavour model in GraphDef format #10
Conversation
A new Pull Request was created by @pablodecm (Pablo de Castro) for branch master. @cmsbuild, @smuzaffar, @gudrutis, @mrodozov can you please review it and eventually sign? Thanks. external issue cms-sw/cmsdist#3572 |
The path is now consistent with the file added in cms-data/RecoBTag-Combined#10.
@mrodozov @smuzaffar |
Testing the changes added here: cms-data/RecoBTag-Combined#10
@slava77 , cms-sw/cmsdist#3573 is merged and should be available in 23h00 IBs tonight. |
On 11/14/17 12:57 PM, Malik Shahzad Muzaffar wrote:
@slava77 <https://github.com/slava77> , cms-sw/cmsdist#3573
<cms-sw/cmsdist#3573> is merged and should be
available in 23h00 IBs tonight.
Thank you.
… —
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEdcbjyFE9UZRimblSU7e6hVlq3xokwPks5s2YAjgaJpZM4Qb6Rl>.
|
The path is now consistent with the file added in cms-data/RecoBTag-Combined#10.
The path is now consistent with the file added in cms-data/RecoBTag-Combined#10.
The path is now consistent with the file added in cms-data/RecoBTag-Combined#10.
The path is now consistent with the file added in cms-data/RecoBTag-Combined#10.
Testing the changes added here: cms-data/RecoBTag-Combined#10
The path is now consistent with the file added in cms-data/RecoBTag-Combined#10.
The path is now consistent with the file added in cms-data/RecoBTag-Combined#10.
This adds the DeepFlavour model transformed to a constant GraphDef format, produced by using https://www.tensorflow.org/api_docs/python/tf/graph_util/convert_variables_to_constants, which reduces significantly the memory overhead of loading the model (from O(100) MB to O(1) MB ) which is crucial for integration in cms-sw/cmssw#19893.
The changes in pablodecm/cmssw#9 are awaiting to be pushed once this is merged and available in IBs and allow to use this model format directly.