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

Rename BoundaryFluxPP #26755

Closed
GiudGiud opened this issue Feb 6, 2024 · 3 comments
Closed

Rename BoundaryFluxPP #26755

GiudGiud opened this issue Feb 6, 2024 · 3 comments
Labels
C: Modules/RDG Tickets pertaining to the rdg module P: minor A defect that does not affect the accuracy of results. T: defect An anomaly, which is anything that deviates from expectations.

Comments

@GiudGiud
Copy link
Contributor

GiudGiud commented Feb 6, 2024 •
edited
Loading

Bug Description

It's specific to RDG. The name is way too general for what it can do.

Steps to Reproduce

Use it, see it not work on anything but RDG.

Impact

Confuse users, waste user support resources
See #26745
@GiudGiud GiudGiud added P: minor A defect that does not affect the accuracy of results. T: defect An anomaly, which is anything that deviates from expectations. labels Feb 6, 2024
@lindsayad
Copy link
Member

Does this come from a discussion?

@lindsayad lindsayad added the C: Modules/RDG Tickets pertaining to the rdg module label Feb 6, 2024
@GiudGiud
Copy link
Contributor Author

GiudGiud commented Feb 6, 2024

yup

@lindsayad
Copy link
Member

can you ref it here? it's annoying that github doesn't automaticially cross reference discussions

GiudGiud added a commit to GiudGiud/moose that referenced this issue Jun 16, 2024
@GiudGiud
Rename BoundaryFluxPP in RDG to be RDG-specific, closes idaholab#26755
65170b8
GiudGiud added a commit to GiudGiud/moose that referenced this issue Jun 16, 2024
@GiudGiud
Rename BFluxPP in inputs, refs idaholab#26755
c02efa3
@GiudGiud GiudGiud mentioned this issue Jun 16, 2024
Merged
GiudGiud added a commit to GiudGiud/moose that referenced this issue Jun 16, 2024
@GiudGiud
Rename BoundaryFluxPP in RDG to be RDG-specific, closes idaholab#26755
afc44e9
GiudGiud added a commit to GiudGiud/moose that referenced this issue Jun 16, 2024
@GiudGiud
Rename BFluxPP in inputs, refs idaholab#26755
234cde3
GiudGiud added a commit to GiudGiud/moose that referenced this issue Jun 17, 2024
@GiudGiud
Rename BFluxPP in inputs, refs idaholab#26755
305f175
abanki1 pushed a commit to abanki1/moose that referenced this issue Jul 11, 2024
@GiudGiud
Rename BoundaryFluxPP in RDG to be RDG-specific, closes idaholab#26755
6b7129a
abanki1 pushed a commit to abanki1/moose that referenced this issue Jul 11, 2024
@GiudGiud
Rename BFluxPP in inputs, refs idaholab#26755
6588a07
N4hom pushed a commit to N4hom/moose that referenced this issue Jul 30, 2024
@GiudGiud @N4hom
Rename BoundaryFluxPP in RDG to be RDG-specific, closes idaholab#26755
b46ba48
N4hom pushed a commit to N4hom/moose that referenced this issue Jul 30, 2024
@GiudGiud @N4hom
Rename BFluxPP in inputs, refs idaholab#26755
3e33313
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: Modules/RDG Tickets pertaining to the rdg module P: minor A defect that does not affect the accuracy of results. T: defect An anomaly, which is anything that deviates from expectations.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lindsayad @GiudGiud