Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Dem variable cell weight for load balancing #1446

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

OGaboriault
Copy link
Collaborator

@OGaboriault OGaboriault commented Mar 4, 2025

Description

When load balancing, the base weight of every cell was set at 1000. This feature allows the cell weight to vary depending on its barycenter position by using a parsed function. This way, the cell weight could vary over space an time.

This will be useful when dealing with static dem problem, where you can only load balance once at the start on the simulation to keep track of the tangential overlap. Cells where we know the particle are going to be could be assign a bigger weight then those where no particle will be present.

This PR is in WIP for now. I just want to see if the test are failling in dealii 9.6 on the CI .

Testing

Test need to be implemented.

Documentation

cell weight function is a dim function outputing a scalar value. This function will be evaluated at every load balancer for every cell barycenter.

Checklist (will be removed when merged)

See this page for more information about the pull request process.

Code related list:

  • All in-code documentation related to this PR is up to date (Doxygen format)
  • Copyright headers are present and up to date
  • Lethe documentation is up to date
  • New feature has unit test(s) (preferred) or application test(s), and restart files are in the generator folder
  • The branch is rebased onto master
  • Changelog (CHANGELOG.md) is up to date
  • Code is indented with indent-all and .prm files (examples and tests) with prm-indent

Pull request related list:

  • Labels are applied
  • There are at least 2 reviewers (or 1 if small feature) excluding the responsible for the merge
  • If this PR closes an issue or is related to a project, it is linked in the "Projects" or "Development" section
  • If the fix is temporary, an issue is opened
  • The PR description is cleaned and ready for merge

@OGaboriault OGaboriault added Enhancement New feature or request WIP When a PR is open but not ready for review labels Mar 4, 2025
@OGaboriault OGaboriault self-assigned this Mar 4, 2025
@OGaboriault OGaboriault force-pushed the dem_variable_cell_weight_for_load_balancing branch from 7b619f6 to 42f80d9 Compare March 4, 2025 16:54
@OGaboriault OGaboriault force-pushed the dem_variable_cell_weight_for_load_balancing branch from 42f80d9 to d3ada7f Compare March 4, 2025 16:54
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Enhancement New feature or request WIP When a PR is open but not ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant