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

fluids: Remove ComputeL2Projection function #1257

Merged
merged 1 commit into from
Jul 18, 2023

Conversation

jrwrigh
Copy link
Collaborator

@jrwrigh jrwrigh commented Jul 13, 2023

  • It's superseded by using the NodalProjectionData struct + ApplyCeedOperator* + KSPSolve
  • Primary reason is to allow using ApplyCeedOperatorGlobalToGlobal, ApplyCeedOperatorLocalToGlobal, or ApplyCeedOperatorGlobalToLocal depending on what's the most efficient for a specific application.

@jrwrigh
Copy link
Collaborator Author

jrwrigh commented Jul 13, 2023

Not sure what's causing the ICC IFort test to fail (or the backend codecov), but everything else passes fine.

@jrwrigh jrwrigh force-pushed the jrwrigh/remove_l2_proj_func branch from ba9fe03 to 84d495d Compare July 14, 2023 17:12
@jrwrigh
Copy link
Collaborator Author

jrwrigh commented Jul 14, 2023

Well, I tried to run it again and the ICC IFort test still fails... Looks like it's just the IFort compiler that can't get installed on the container correctly.

Also the codecov backends even though I haven't changed anything on the backend...

Regardless, whoever is able to can go ahead and merge this.

- It's superseded by using `NodalProjectionData` + `ApplyCeedOperator*`
  + `KSPSolve`
@jrwrigh jrwrigh force-pushed the jrwrigh/remove_l2_proj_func branch from 84d495d to 9fcc926 Compare July 18, 2023 22:09
@jrwrigh jrwrigh merged commit d429f2b into main Jul 18, 2023
22 checks passed
@jrwrigh jrwrigh deleted the jrwrigh/remove_l2_proj_func branch July 18, 2023 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants