Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Some postprocessors need ghosting layers for FV #19534

Open
GiudGiud opened this issue Dec 3, 2021 · 0 comments
Open

Some postprocessors need ghosting layers for FV #19534

GiudGiud opened this issue Dec 3, 2021 · 0 comments
Labels
C: Framework P: normal A defect affecting operation with a low possibility of significantly affects. T: defect An anomaly, which is anything that deviates from expectations.
Projects
FY22 NEAMS MP - TA Support

Comments

@GiudGiud
Copy link
Contributor

GiudGiud commented Dec 3, 2021 •
edited

Bug Description

SideDiffusiveFluxXXX for example.
Takes a gradient on a boundary, needs to ghost

Steps to Reproduce

Seg fault with 2 mpi ranks when using one of those on the middle of the domain

This only seg faults if you dont have kernels that provide the required ghosting.
For example if you are solving diffusion for that variable, then taking a diffusive flux is fine

Impact

bug
@GiudGiud GiudGiud added T: defect An anomaly, which is anything that deviates from expectations. P: normal A defect affecting operation with a low possibility of significantly affects. labels Dec 3, 2021
@GiudGiud GiudGiud added this to To do in FY22 NEAMS MP - CRAB via automation Dec 3, 2021
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 3, 2021
@GiudGiud
Add ghosting to pass the boundary test for block material
c4e0589 
Ideally, we would use a postprocessor that currently does not have an issue with insufficient
ghosting when used on its own, see idaholab#19534. But I dont think there is one that will trigger
the ghosting we want. I could admittedly make a new one, which would be a rework of SideAverageMaterialProperty,
but it could make that one confusing.
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 3, 2021
@GiudGiud
Add ghosting to pass the boundary test for block material
d34ec95 
Ideally, we would use a postprocessor that currently does not have an issue with insufficient
ghosting when used on its own, see idaholab#19534. But I dont think there is one that will trigger
the ghosting we want. I could admittedly make a new one, which would be a rework of SideAverageMaterialProperty,
but it could make that one confusing.
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 4, 2021
@GiudGiud
Add ghosting to pass the boundary test for block material
760c5bb 
Ideally, we would use a postprocessor that currently does not have an issue with insufficient
ghosting when used on its own, see idaholab#19534. But I dont think there is one that will trigger
the ghosting we want. I could admittedly make a new one, which would be a rework of SideAverageMaterialProperty,
but it could make that one confusing.

Use MatpropName and FunctorName instead of strings for parameters for the AD converters
Refs idaholab#19420
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 5, 2021
@GiudGiud
Add ghosting to pass the boundary test for block material
6d7aae5 
Ideally, we would use a postprocessor that currently does not have an issue with insufficient
ghosting when used on its own, see idaholab#19534. But I dont think there is one that will trigger
the ghosting we want. I could admittedly make a new one, which would be a rework of SideAverageMaterialProperty,
but it could make that one confusing.

Use MatpropName and FunctorName instead of strings for parameters for the AD converters
Refs idaholab#19420

Use interface diffusive flux instead of side to use both sides
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 5, 2021
@GiudGiud
Add ghosting to pass the boundary test for block material
409f1d5 
Ideally, we would use a postprocessor that currently does not have an issue with insufficient
ghosting when used on its own, see idaholab#19534. But I dont think there is one that will trigger
the ghosting we want. I could admittedly make a new one, which would be a rework of SideAverageMaterialProperty,
but it could make that one confusing.

Use MatpropName and FunctorName instead of strings for parameters for the AD converters
Refs idaholab#19420

Use interface diffusive flux instead of side to use both sides

regold
@permcody permcody added this to To do in FY22 NEAMS MP - TA Support via automation Feb 1, 2022
@permcody permcody removed this from To do in FY22 NEAMS MP - CRAB Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: Framework P: normal A defect affecting operation with a low possibility of significantly affects. T: defect An anomaly, which is anything that deviates from expectations.
Projects
No open projects
FY22 NEAMS MP - TA Support
To do
Milestone
No milestone
Development

No branches or pull requests
2 participants
@loganharbour @GiudGiud