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 compilation of prototypes #1249

Merged
merged 1 commit into from
Aug 20, 2024
Merged

Fix compilation of prototypes #1249

merged 1 commit into from
Aug 20, 2024

Conversation

lpsaavedra
Copy link
Collaborator

@lpsaavedra lpsaavedra commented Aug 19, 2024

Description

The prototypes were not compiling due to minor changes on the vtk compression level and interpolation via solution transfer.

Solution

The calls were updated according to current deal.II documentation.

Miscellaneous (will be removed when merged)

We should probably add the prototypes to the CI to ensure compilation. I will do it in a follow PR if @blaisb agrees.

Checklist (will be removed when merged)

See this page for more information about the pull request process.

Code related list:

  • The branch is rebased onto master
  • Code is indented with indent-all and .prm files (examples and tests) with prm-indent

Pull request related list:

  • Labels are applied
  • There are at least 2 reviewers (or 1 if small feature) excluding the responsible for the merge
  • The PR description is cleaned and ready for merge

@blaisb blaisb merged commit ed30045 into master Aug 20, 2024
9 of 10 checks passed
@blaisb blaisb deleted the fix_prototypes branch August 20, 2024 02:15
blaisb pushed a commit that referenced this pull request Aug 20, 2024
Description
Add prototypes compilation to ensure that they are all up to date. Follow up on PR #1249.
M-Badri pushed a commit to M-Badri/lethe that referenced this pull request Sep 29, 2024
Description
The prototypes were not compiling due to minor changes on the vtk compression level and interpolation via solution transfer.

Solution
The calls were updated according to current deal.II documentation.

Former-commit-id: ed30045
M-Badri pushed a commit to M-Badri/lethe that referenced this pull request Sep 29, 2024
Description
Add prototypes compilation to ensure that they are all up to date. Follow up on PR chaos-polymtl#1249.

Former-commit-id: f7e869a
blaisb pushed a commit that referenced this pull request Sep 30, 2024
Description
The prototypes were not compiling due to minor changes on the vtk compression level and interpolation via solution transfer.

Solution
The calls were updated according to current deal.II documentation.

Former-commit-id: ed30045
blaisb pushed a commit that referenced this pull request Sep 30, 2024
Description
Add prototypes compilation to ensure that they are all up to date. Follow up on PR #1249.

Former-commit-id: f7e869a
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.

3 participants