Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • dumux dumux
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 100
    • Issues 100
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 87
    • Merge requests 87
  • 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
  • Issues
  • #872
Closed
Open
Issue created Apr 22, 2020 by Timo Koch@timokOwner

Deprecation warnings from Dune::Hybrid::size

At some points, e.g. in the multidomain assembler, we use for Dune::MultiTypeBlockMatrix/Dune::MultiTypeBlockVector m:

using namespace Dune::Hybrid;
forEach(m, [](auto& row){
   ...
});

This accesses Dune::MultiTypeBlockMatrix::size which is deprecated since before dune 2.7 and causes the warning. Unfortunately this means we need to use a bit more verbose syntax:

using namespace Dune::Hybrid;
forEach(integralRange(M::N()), [](auto& i){
   auto& row = m[i];
   ...
});

should work.

A proposal to allow ::N() to de deduced from a range passed to Dune::Hybrid::forEach (https://gitlab.dune-project.org/core/dune-common/-/merge_requests/803) has been rejected for good reasons. The might be something like rows(m) in the future... but that doesn't help now.

Edited Apr 22, 2020 by Timo Koch
Assignee
Assign to
Time tracking