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

Elegent migration #12

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

misadows
Copy link

@misadows misadows commented Jun 30, 2016

Elegent migrated from 7.5.0

@jan-kaspar
Copy link
Collaborator

Czesc Michal, wait a moment. Elegent has become a standalone project: http://elegent.hepforge.org/ . So the version which was in 7.5.0 is obsolete. Now I can see several options.

  • Elegent is not ported at all. Obviously the fastest, but most likely not the best.
  • Elegent is included as an external tool to the CMSSW environment. This is the cleanest solution, but I have no idea what needs to be done.
  • A static copy of Elegent is integrated in CMSSW code. A compromise solution.
    Let's first decide how to proceed.

@misadows
Copy link
Author

misadows commented Jul 1, 2016

Ok, my mistake. I was told to move it to CMSSW. It is here CTPPS/cmssw@e5daf57

I agree that firstly we have to choose the best solution and then I will make changes

@jan-kaspar
Copy link
Collaborator

@misadows It is not your mistake - you didn't know all the details. Quickly discussing with Valentina, we think we should keep Elegent external from CMSSW. This is the typical case for event generators. What is then added to CMSSW is an interface to the generator.

Are you one of the students coming to CERN in few days? If yes, we can discuss here - it would be much easier.

@misadows
Copy link
Author

misadows commented Jul 1, 2016

@jan-kaspar Yes, I am coming to CERN on Monday so we will discuss it then.

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

Successfully merging this pull request may close these issues.

2 participants