Skip to content

Conversation

ggdose
Copy link
Contributor

@ggdose ggdose commented Apr 14, 2025

Improve performance of Heat Flux Actor:

  • Avoid repetition of code between the mapper and the mesher
  • Reduce allocations

Old version:
image
This version to be pushed:
image

@ggdose ggdose requested a review from orso82 April 14, 2025 23:37
@orso82
Copy link
Member

orso82 commented Apr 14, 2025

❤️

@orso82
Copy link
Member

orso82 commented Apr 15, 2025

@ggdose can you please take a look at these conflicts?

@orso82
Copy link
Member

orso82 commented Apr 15, 2025

also some of the docstrings seem to be outdated based on your updates

@ggdose
Copy link
Contributor Author

ggdose commented Apr 24, 2025

Merged latest master and tested.
Ready to be merged!

Please remember that there is a branche with the same name in FUSE that needs to be merged together with this one

@ggdose
Copy link
Contributor Author

ggdose commented Apr 24, 2025

@ggdose
Copy link
Contributor Author

ggdose commented May 1, 2025

I will continue merging master into this branch, such that the PR can be merged at anytime

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