search

trilinos / Trilinos

Primary repository for the Trilinos Project
https://trilinos.org/
Other
1.19k stars 565 forks source link

Ifpack2: SupportGraph preconditioner unlikely to compile #6616

Open kddevin opened 4 years ago

kddevin commented 4 years ago

Bug Report

@trilinos/ifpack2

Description

The SupportGraph preconditioner in Ifpack2 appears to be untested and is unlikely to compile. It uses functions (e.g., CrsMatrix->getGlobalNumDiags) that have been deprecated and removed from Tpetra.

The code is contained within ifdefs indicating it is experimental, so this issue has no impact on applications. Resolving this issue is very low priority.

Caveat emptor.

@egboman

github-actions[bot] commented 2 years ago

This issue has had no activity for 365 days and is marked for closure. It will be closed after an additional 30 days of inactivity. If you would like to keep this issue open please add a comment and/or remove the MARKED_FOR_CLOSURE label. If this issue should be kept open even with no activity beyond the time limits you can add the label DO_NOT_AUTOCLOSE. If it is ok for this issue to be closed, feel free to go ahead and close it. Please do not add any comments or change any labels or otherwise touch this issue unless your intention is to reset the inactivity counter for an additional year.

srajama1 commented 2 years ago

I believe this has to be fixed, or marked deprectated or since it is experimental just removed for better long term maintenance of Ifpack2.

github-actions[bot] commented 1 year ago

This issue has had no activity for 365 days and is marked for closure. It will be closed after an additional 30 days of inactivity. If you would like to keep this issue open please add a comment and/or remove the MARKED_FOR_CLOSURE label. If this issue should be kept open even with no activity beyond the time limits you can add the label DO_NOT_AUTOCLOSE. If it is ok for this issue to be closed, feel free to go ahead and close it. Please do not add any comments or change any labels or otherwise touch this issue unless your intention is to reset the inactivity counter for an additional year.