Skip to content

Add air to cryo window - #72

Closed
jdkio wants to merge 4 commits into
masterfrom
jdkio-cryo-window-air
Closed

Add air to cryo window#72
jdkio wants to merge 4 commits into
masterfrom
jdkio-cryo-window-air

Conversation

@jdkio

@jdkio jdkio commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

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.

Here are slices of showing the window with air:

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 06:24
@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/

@ast0815

ast0815 commented Aug 8, 2025

Copy link
Copy Markdown
Member

Looks like this currently introduces a geometry overlap:

=== Overlaps for Default ===
 = Overlap ov00000: volAirBoxCompositeWindow extruded by: volAirBoxCompositeWindow/volCompositeWindow_0 ovlp=0.4765

@jdkio

jdkio commented Aug 8, 2025

Copy link
Copy Markdown
Contributor Author

Looks like this currently introduces a geometry overlap:

=== Overlaps for Default ===
 = Overlap ov00000: volAirBoxCompositeWindow extruded by: volAirBoxCompositeWindow/volCompositeWindow_0 ovlp=0.4765

I'm not sure what it's overlapping with. I made it slightly bigger than the window to see if that's the cause

@ast0815

ast0815 commented Aug 8, 2025

Copy link
Copy Markdown
Member

If you want to test this more rapidly, you can build the "tms_nosand" geometry locally and then use the overlap checking macro on that.

@jdkio jdkio mentioned this pull request Aug 8, 2025
@ast0815

ast0815 commented Aug 11, 2025

Copy link
Copy Markdown
Member

Incorporated into #74

@ast0815 ast0815 closed this Aug 11, 2025
@ast0815
ast0815 deleted the jdkio-cryo-window-air branch August 11, 2025 13:19
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