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

Implement FunctionIC::gradient(const Point &) #3312

Closed
dschwen opened this issue Jun 6, 2014 · 2 comments
Closed

Implement FunctionIC::gradient(const Point &) #3312

dschwen opened this issue Jun 6, 2014 · 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

@dschwen
Copy link
Member

dschwen commented Jun 6, 2014

FunctionIC::gradient(const Point &) is not implemented, even though the Function class has the method. I don't see a reason why we shouldn't pass the gradient from function through.

The next step is using the FParser auto differentiation to compute the gradient in MooseParsedFunction::gradient(const Point &). That way we would correctly initialize variables using higher order base functions (e.g. THIRD order HERMITE).

@dschwen
Copy link
Member Author

dschwen commented Jun 6, 2014

MooseParsedGradFunction is a MooseParsedFunction with explicitly specified gradient components (as Function Parser expressions). This could/should probably be retired in favor of auto differentiation, unless there is a good reason to allow the user to specify a (potentially wrong) hand-made derivative. This will still be needed, as we cannot autodiff the additional variables which may depend on xyzt.

dschwen added a commit to dschwen/moose that referenced this issue Jun 10, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 10, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 10, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 10, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 12, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 12, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 12, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 12, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 23, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 23, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 23, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 30, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 30, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 30, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jun 30, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jul 1, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jul 1, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jul 1, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jul 1, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jul 1, 2014
dschwen added a commit to dschwen/moose that referenced this issue Jul 1, 2014
@dschwen
Copy link
Member Author

dschwen commented Jul 3, 2014

With the merged PR this should be done now.

@dschwen dschwen closed this as completed Jul 3, 2014
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

2 participants