Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • dumux dumux
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 104
    • Issues 104
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 92
    • Merge requests 92
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • dumux-repositoriesdumux-repositories
  • dumuxdumux
  • Merge requests
  • !1510

Feature/multidomain auto bind

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Dennis Gläser requested to merge feature/multidomain-auto-bind into master Feb 27, 2019
  • Overview 53
  • Commits 10
  • Pipelines 0
  • Changes 40

I posed this MR into the box-facet-coupling branch so that the diff is easier to read. If we wait for !1492 (merged), which itself has to wait for !1431 (merged), then we can merge this into master at one point.

@timok, @kweis, maybe you can have a look at this. If we want to keep this, we would have to:

  • deprecate old interfaces
  • change all tests where cm now receives grid variables
  • create an issue to remove the deprecated interfaces in all facet coupling managers and the poroelasticcouplingmanager until version 3.2 release (edit Timo: don't need an issue for that)
  • update reference to test_md_facet_tracer_box (velocity is included now, which tests the MD vel output)
  • Put a note in the changelog that scvfs now have to implement neighbor()

Fixes #619.

Edited Sep 30, 2020 by Kilian Weishaupt
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: feature/multidomain-auto-bind