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 a modified version of the Generic Cal Endcap with conical cutout #341

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

Conversation

tmadlener
Copy link
Contributor

@tmadlener tmadlener commented Jun 11, 2024

BEGINRELEASENOTES

  • Add a modified version of the Generic Calorimeter Endcap with conical cutout

ENDRELEASENOTES

Bringing more things back from the muon collider fork. @madbaron @bartosik-hep FYI

@andresailer
Copy link
Contributor

This is not without a cutout, but with a conical cutout?

Can't we add this as an alternative option to the other driver instead?

@tmadlener tmadlener changed the title Add a modified version of the Generic Cal Endcap without cutout Add a modified version of the Generic Cal Endcap with conical cutout Jun 13, 2024
@bartosik-hep
Copy link
Contributor

@andresailer From the driver point of view - yes, it's easy to include the 2nd radius of the cutout as an extra parameter, which falls back to the same radius if not defined.
The problem is that we already have a frozen geometry that uses o2_v01 driver in it, so if we change the driver version it will make our compact detector description incompatible with older releases.

@andresailer
Copy link
Contributor

The problem is that we already have a frozen geometry that uses o2_v01 driver in it, so if we change the driver version it will make our compact detector description incompatible with older releases.

Can you elaborate on your package structure? Where do your XML files and drivers live for the moment? Do you have them in your own fork of k4geo, or some other package?

@bartosik-hep
Copy link
Contributor

The XML files of the frozen geometry I mentioned are currently collected here, and it uses all the standard drivers except for the one added in this PR.

@andresailer
Copy link
Contributor

And how do you install that package along with a newer k4geo package? Because that is what you would need to do if you need backward compatibility? Or do you just copy the XML folders?

@tmadlener
Copy link
Contributor Author

I think the main idea was to do the driver(s) first, because the XML files can in principle also be put somewhere else / read from somewhere else. Then the XML files would come in a separate PR to not overload this one too much.

@madbaron
Copy link

For now the xmls are just copied over manually.

As Thomas notes above, in a few months we'll make a separate PR with the frozen 3 TeV geometry (MuColl_v1) and the two 10 TeV detector concepts.

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.

4 participants