Skip to content

Issue482 no implicit#487

Merged
asamake merged 2 commits intodevelopfrom
issue482_no_implicit
Nov 2, 2020
Merged

Issue482 no implicit#487
asamake merged 2 commits intodevelopfrom
issue482_no_implicit

Conversation

@einola
Copy link
Copy Markdown
Member

@einola einola commented Nov 1, 2020

Remove the implicit solver and all connections with PETSc.

  • I would like to ask @asamake to aprove this pull request.

  • I'd like to ask @akorosov to create a new docker image without PETSc.

@einola einola requested a review from asamake November 1, 2020 10:08
@einola
Copy link
Copy Markdown
Member Author

einola commented Nov 1, 2020

Link to issue: #482

@einola
Copy link
Copy Markdown
Member Author

einola commented Nov 2, 2020

We cannot completely remove PETSc dependencies because we still use METIS from PETSc.

@asamake asamake closed this Nov 2, 2020
@asamake asamake reopened this Nov 2, 2020
@asamake
Copy link
Copy Markdown
Collaborator

asamake commented Nov 2, 2020

We cannot completely remove PETSc dependencies because we still use METIS from PETSc.

Hi Einar, I think it should be possible to use METIS installed from GMSH. This could avoid keeping PETSc only because of METIS.

@asamake asamake merged commit 4b4acaa into develop Nov 2, 2020
@einola einola deleted the issue482_no_implicit branch November 10, 2020 05:35
@einola
Copy link
Copy Markdown
Member Author

einola commented Nov 11, 2020

Hi Abdoulaye, It turns out that we already use METIS from GMSH, not PETSc like I thought. So I was wrong and after this merge we have no dependence on PETSc anymore.

@asamake
Copy link
Copy Markdown
Collaborator

asamake commented Nov 11, 2020

Hi Einar, ok super. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants