Skip to content
Snippets Groups Projects

Feature/setposition

Merged Timo Koch requested to merge feature/setposition into master

Implement a setPosition method for vertices. This branch is based on the feature/growth-revised, so !2 (merged) should to be dealt with first.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 1419 1419
    1420 1420 return true;
    1421 1421 }
    1422
    1423 // change a vertex' position on all levels
  • 34
    35 Dune::VTKWriter<typename Grid::LeafGridView > writer(grid->leafGridView(), Dune::VTK::nonconforming);
    36 writer.write("setposition_0");
    37
    38 // move all vertices
    39 for (const auto& vertex : vertices(grid->leafGridView()))
    40 grid->setPosition(vertex, getPosition(vertex->geometry().center()));
    41
    42 writer.write("setposition_1");
    43
    44 grid->globalRefine(-1);
    45
    46 writer.write("setposition_2");
    47
    48 gridcheck(*grid);
    49 }
  • Timo Koch Added 2 commits:

    Added 2 commits:

    • dfcf7344 - [fix] Use corner instead of center to get the vertex coords
    • d374d7cc - [test] Add missing 0 return code
  • Timo Koch Added 1 commit:

    Added 1 commit:

    • 0fa0b3ae - [setposition] Make the assumption that only leaf vertices can be moved
  • Author Owner

    commit 0fa0b3ae makes it more clear for the user IMO.

  • It does. Can you merge the the entire patch set into a single commit? Then I'll accept the merge request. Thanks.

  • Timo Koch Added 1 commit:

    Added 1 commit:

    • ef6e2a8e - [foamgrid] Implement a setPosition method for vertices
  • Author Owner

    ok everything in one commit

  • Oliver Sander mentioned in commit 7792992d

    mentioned in commit 7792992d

  • Oliver Sander Status changed to merged

    Status changed to merged

  • Please register or sign in to reply
    Loading