-
Notifications
You must be signed in to change notification settings - Fork 15
uploading CPO heat example #378
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
base: develop
Are you sure you want to change the base?
Conversation
Spell Check ReportCPOHeat.ipynb:
Checked 1 notebook(s). Found spelling errors in 1 file(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great result @FilipeFcp and showcase of transient thermal analysis!
-
I suggest a more formal title such as "Steady-state and transient thermal analysis for co-packaged optics"
-
Since there are quite a lot of warnings, explain why they are and why it's fine to ignore in this case?
-
In the temperature plot, plot the structure boundaries as well?
Also tagging @prashkh since he is interested in this model as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great and seems to match the paper reasonably well! I wonder if the difference could be coming from the insulating BCs along x and y, in reality there's probably some heat going out of there too?
Just one note: the schematic appears on the top of the first cell but at the end of that cell it says "A schematic of the simulated structure can be seen in the image below:"
Just out of curiosity I noted that the meshing and solver seem to take about the same time ~25 minutes each. Hopefully we'll improve on the meshing side!

:maxdepth: 1 | ||
|
||
../../HeatSolver | ||
../../CPOHeat |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't the usual policy for one notebook to go in one place only?
I'm not necessarily opposed to linking in multiple places just wondering.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work as always @FilipeFcp! A couple suggestions:
-
You might want to define the unstructured mesh in terms of one of the feature variables. Raw numbers might be more unclear as to why they were chosen.
-
For the two time_specs, you might want to deifne the time_step and total-time_steps variables at the beginning of cell 71 to have it match cell 79
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @FilipeFcp. This is an awesome notebook!
Is it possible to use MultiPhysicsMedium
instead of Medium
to run this notebook? I feel like it would make sense to use it.
Very nice notebook! I am sure a lot of people will be interested in this work. Thanks @FilipeFcp !! |
Thanks everyone for the comments. Sorry, it took me a while to implement; I was running some mesh tests, and the 2D transient simulation takes quite a long time to run. With a coarser mesh, the 2D transient simulation shows some strange dips, so I kept the first one. @momchil-flex, these are the boundaries that they report in the paper. I think the discrepancies are in the source definition. They won't report in volumetric power, and I didn't find it in the paper any information to directly translate it. |
I believe the results look like the paper, although I had to make some guesses on the material properties and heat power.
@momchil-flex, feel free to take a look if you are interested.