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

fix(par): fix backtracking and residual norm calculation for certain cases in parallel #1780

Merged

Conversation

mjr-deltares
Copy link
Contributor

  • update petsc matrix before multiplication
  • backtracking now properly synchronized

this closes #1779

- backtracking now properly synchronized
@mjr-deltares mjr-deltares added the parallel Parallel capabilities label May 8, 2024
@mjr-deltares mjr-deltares self-assigned this May 8, 2024
@mjr-deltares
Copy link
Contributor Author

Hi @jdhughes-usgs , looks like this is going to be green. Feel free to merge if you agree with the changes, so you can try it out.

@jdhughes-usgs jdhughes-usgs merged commit e6202c0 into MODFLOW-USGS:develop May 8, 2024
18 checks passed
@mjr-deltares mjr-deltares deleted the fix-1779-par-l2norm branch May 8, 2024 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
parallel Parallel capabilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Residual L2 norm calculation incorrect for parallel simulation with Backtracking and/or PTC
2 participants