Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • dumux-course dumux-course
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 7
    • Issues 7
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • 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
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • dumux-repositoriesdumux-repositories
  • dumux-coursedumux-course
  • Issues
  • #21
Closed
Open
Issue created Sep 10, 2019 by Johannes Hommel@hommelMaintainer1 of 9 checklist items completed1/9 checklist items

Things that came up in Tunis

All issues that came up during the dumux course in Tunis or which I noticed:

  • exercises-properties: A "TODO" component might be added in neumann() to be consistent with other exercises

  • exercises-runtimeparams: update Readme.md. The default input file is now params.input, not "executablename.input", or more correctly, both are working by default.

  • exercise-coupling-ff-pm general: Many of the files do not contain "TODO" comments where changes need to be made in the fashion of the other exercises.

  • exercise-coupling-ff-pm (a: interface and b,c): warning about neumannFlux being deprecated, probably should be updated. Also, I guess that the tasks in the Readme.md should include removing the initial velocity field in initialAtPos().

  • exercise-coupling-ff-pm (b: models): the 2p2c model's indices.hh is references in the Readme.md. However, this does not seem to exist.

  • exercise-biomineralization: The line numbers in Readme.md do not match the current code. eps_ is added both for the upper and lower injection area boundary. A comment in Readme.md should be added that NH4 is ignored in the simplified biomineralization model and thus also the chemistry.

  • general comment: there is always the "property macro deprecation warning" when compiling any of the course exercises. Are the exercises free of the property macros?

  • general comment: The names of the exercises' executables could be more consistent, e.g.: ex_coupling_* vs exercisebiomin

  • general comment: deprecation warnings for gravity

Edited Sep 18, 2019 by Johannes Hommel
Assignee
Assign to
Time tracking