Skip to content
Snippets Groups Projects

Improve run time of convertToLinearMesh

Merged Tom Fischer requested to merge TomFischer/ogs:AccelerateConvertToLinearMesh2 into master
All threads resolved!
  • Substitute geometrical search by a mapping that is calculated by known information before creating the linear elements
  • Store the result of the expensive to calculate getNumberOfBaseNodes() once and re-use it

Update

Forget to mention: Run time for a test example from @joergbuchwald reduced from 5400 s to 0.5 s.

Edited by Tom Fischer

Merge request reports

Merged by Lars BilkeLars Bilke 2 years ago (Dec 15, 2022 8:13pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Tom Fischer added 27 commits

    added 27 commits

    • bbc5bdea...5f228887 - 23 commits from branch ogs:master
    • bd5a2b9e - Create nodes map; use in convert
    • 55cf01f9 - [MeL/MeshEditing] Mark base nodes in ConvertToLinearMesh
    • 8a160f4c - [MeL/MeshEditing] Create the mapping efficient
    • 3ff9c676 - [MeL/MeshEditing] Extract getNumberOfBaseNodes() outside of loop

    Compare with previous version

  • Dmitri Naumov resolved all threads

    resolved all threads

  • Dmitri Naumov approved this merge request

    approved this merge request

  • Dmitri Naumov added 11 commits

    added 11 commits

    • 3ff9c676...376112b5 - 7 commits from branch ogs:master
    • c93ae288 - Create nodes map; use in convert
    • f723d51d - [MeL/MeshEditing] Mark base nodes in ConvertToLinearMesh
    • 3e32b812 - [MeL/MeshEditing] Create the mapping efficient
    • c54b2f82 - [MeL/MeshEditing] Extract getNumberOfBaseNodes() outside of loop

    Compare with previous version

  • Dmitri Naumov enabled an automatic merge when the pipeline for 62cdb036 succeeds

    enabled an automatic merge when the pipeline for 62cdb036 succeeds

  • joergbuchwald approved this merge request

    approved this merge request

  • Thanks, it works well :)

  • Lars Bilke canceled the automatic merge

    canceled the automatic merge

  • Lars Bilke mentioned in commit 72a3e51e

    mentioned in commit 72a3e51e

  • merged

  • Please register or sign in to reply
    Loading