Skip to content

Finalize cryo window - #74

Merged
ast0815 merged 9 commits into
masterfrom
jdk-fixed-cryo-window
Aug 11, 2025
Merged

Finalize cryo window#74
ast0815 merged 9 commits into
masterfrom
jdk-fixed-cryo-window

Conversation

@jdkio

@jdkio jdkio commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

Adds back the cryo window. Also combines fixes from #72 and #73 into a single PR to create the final cryo window design.

From #73,

Removed polyurethane foam material as it is no longer used in cryostat, also removed the auxparam that makes the window sensitive.

From #72,

Adds air to the cryo window. Doesn't try to remove the warm steel after the cryo window, but that can be easily achieved by increasing the dz of the air pocket.

dz = {CompositeWindow:dz}+Q('50mm')

I tried to do that and add a plywood membrane CompositeWindowMembrane, but I was unable to get it to work. This achieves the same thing, though it relies on the accidental coincidence that plywood and our low-density steel have similar densities.

Note that I removed overlaps by moving the cryo window slightly by about z=-0.5x4.765mm. I assume that's within the margins.

Here are the final images. They're almost identical to #72, but the cryo window is moved upstream a small amount. You need to flipbook for it to be obvious

Y vs Z Slice

cryo_yz

X vs Z Slice

cryo_xz

X vs Y Slice

cryo_xy

@jdkio
jdkio requested review from Cjjr2021 and ast0815 August 8, 2025 16:45
@github-actions

github-actions Bot commented Aug 8, 2025

Copy link
Copy Markdown

Geometry is ready for visual inspection at https://dune.github.io/dunendggd/

@Cjjr2021 Cjjr2021 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My only concern was the CompositeWindowMembrane volume added, but Jeffrey said that it is not doing anything in the code yet. So it all looks good from my end.

@jdkio

jdkio commented Aug 11, 2025

Copy link
Copy Markdown
Contributor Author

@ast0815 please merge it if you're happy and pass it on to Alex

Comment thread CHANGELOG.md
Comment thread duneggd/Config/ArgonCube/ArgonCubeCryostat.cfg
Co-authored-by: Lukas Koch <lukas.koch@uni-mainz.de>
@jdkio

jdkio commented Aug 11, 2025

Copy link
Copy Markdown
Contributor Author

@ast0815, are we good to merge? It's rerunning the checks due to the change log change

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.

3 participants