Skip to content

Make Nutils participant micro-nutils in the two-scale-heat-conduction tutorial faster#516

Merged
IshaanDesai merged 5 commits intoprecice:developfrom
IshaanDesai:simplify-two-scale-heat
Apr 8, 2024
Merged

Make Nutils participant micro-nutils in the two-scale-heat-conduction tutorial faster#516
IshaanDesai merged 5 commits intoprecice:developfrom
IshaanDesai:simplify-two-scale-heat

Conversation

@IshaanDesai
Copy link
Copy Markdown
Member

@IshaanDesai IshaanDesai commented Apr 8, 2024

This PR removes the initial correction calculation of the phase field in the micro-nutils participant of the two-scale-heat-conduction tutorial. This calculation was originally implemented so that the micro simulation will start with a very accurate grain description. However, if the micro structure (grain) is not represented extremely accurately, the resultant phase field and the average porosity will still yield sensible values. Therefore this calculation can be commented out to cut down the initialization time and make the micro-nutils participant faster.

Approximate runtime

  • On 4 processors: 6.5 minutes
  • On 2 processors: 10.5 minutes

@IshaanDesai IshaanDesai self-assigned this Apr 8, 2024
@IshaanDesai IshaanDesai marked this pull request as ready for review April 8, 2024 11:24
@IshaanDesai IshaanDesai changed the title Make Nutils participants in the two-scale-heat-conduction tutorial faster Make Nutils participant micro-nutils in the two-scale-heat-conduction tutorial faster Apr 8, 2024
@IshaanDesai IshaanDesai merged commit f4d2fef into precice:develop Apr 8, 2024
@IshaanDesai IshaanDesai deleted the simplify-two-scale-heat branch April 8, 2024 13:32
@MakisH
Copy link
Copy Markdown
Member

MakisH commented Apr 8, 2024

How long was it before?
Does the time hint in the README need to be updated?

@IshaanDesai
Copy link
Copy Markdown
Member Author

IshaanDesai commented Apr 8, 2024

How long was it before? Does the time hint in the README need to be updated?

Good point, I will update the README hint. Previous runtime was ~25 minutes.

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