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.

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

deprecate or enhance AreaPostprocessor #4799

Closed
WilkAndy opened this issue Mar 10, 2015 · 2 comments
Closed

deprecate or enhance AreaPostprocessor #4799

WilkAndy opened this issue Mar 10, 2015 · 2 comments
Labels
C: Framework P: normal A defect affecting operation with a low possibility of significantly affects. T: task An enhancement to the software.

Comments

@WilkAndy
Copy link
Contributor

I want to calculate the integral of a function over a sideset. Therefore i want to either deprecate AreaPostprocessor and create a new FunctionIntegral, or to enhance AreaPostprocessor so that it takes an optional Function argument:

params.addParam("function", 1.0, "doco")

FunctionIntegral::computeQpIntegral()
{
return _func.value(_t, _q_point[_qp]);
}

Comments?

@friedmud
Copy link
Contributor

Please don't modify AreaPostprocessor - it does exactly what it's supposed to do and doesn't need to do anything different.

If you would like to create FunctionIntegral that inherits from SideIntegralPostprocessor... go for it

@friedmud
Copy link
Contributor

Actually - I suppose that would be FunctionSideIntegral

@permcody permcody added C: Framework T: task An enhancement to the software. P: normal A defect affecting operation with a low possibility of significantly affects. labels Mar 10, 2015
permcody added a commit that referenced this issue Mar 11, 2015
FunctionSideIntegral added.  Fixes #4799
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: Framework P: normal A defect affecting operation with a low possibility of significantly affects. T: task An enhancement to the software.
Projects
None yet
Development

No branches or pull requests

3 participants