search

cms-sw / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
1.07k stars 4.29k forks source link

[DD4hep] [Very long-term goal] Move most of DetectorDescription/DDCMS to DD4hep repo #31051

Open cvuosalo opened 4 years ago

cvuosalo commented 4 years ago

Our plan is to eventually move most of the code in DetectorDescription/DDCMS to the DD4hep repository. Of course, some revision will be required to remove CMS-specific features and to make the code suitable for general use. Some small part of the code in this package probably cannot be moved because it represents the interface between DD4hep and CMSSW.

We would like to reduce the amount of code we have to support, so moving more code to DD4hep is desirable.

We can track progress in this issue.

cmsbuild commented 4 years ago

A new Issue was created by @cvuosalo Carl Vuosalo.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

cvuosalo commented 4 years ago

assign geometry

cmsbuild commented 4 years ago

New categories assigned: geometry

@Dr15Jones,@cvuosalo,@mdhildreth,@makortel,@ianna,@civanch you have been requested to review this Pull request/Issue and eventually sign? Thanks

cvuosalo commented 4 years ago

Filter and SpecParRegistry were added to DD4hep on 2-Jul-2020: https://github.com/AIDASoft/DD4hep/pull/675 https://github.com/AIDASoft/DD4hep/pull/682

cvuosalo commented 4 years ago

@ianna FYI

ianna commented 4 years ago

thanks, @cvuosalo !

ianna commented 4 years ago

@cvuosalo - work in progress, see my presentation https://indico.cern.ch/event/954409/

civanch commented 3 years ago

@cvuosalo , it is likely done?

cvuosalo commented 3 years ago

I think this goal is continuing. As we have time and can figure out how to make our code more generic, we can move more to DD4hep.