-
Notifications
You must be signed in to change notification settings - Fork 2
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
Return codes #4
Comments
Given the discussion in #6, if we go with LAGr_Matrix, we should probably go with LAGr_Info. As you've pointed out, we can include algorithm-specific error messages and other LAGraph-specific functionality. I think this would be particularly useful given that the LAGr_Matrix being discussed is not opaque, and so we may need to notify the user of a lot more possible errors (e.g. "Hey, you told me this matrix was symmetric and it's not!"). |
I agree with you …. LAGr_info rather than GrB_info.
I really hope that LAGraph is successful as a standalone library and that most of our users will not really know (or care) that GraphBLAS is running behind the scenes.
…--Tim
From: Scott Kolodziej <notifications@github.com>
Reply-To: GraphBLAS/LAGraph-Working-Group <reply@reply.github.com>
Date: Wednesday, April 8, 2020 at 6:19 AM
To: GraphBLAS/LAGraph-Working-Group <LAGraph-Working-Group@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Subject: Re: [GraphBLAS/LAGraph-Working-Group] Return codes (#4)
Given the discussion in #6<#6>, if we go with LAGr_Matrix, we should probably go with LAGr_Info.
As you've pointed out, we can include algorithm-specific error messages and other LAGraph-specific functionality. I think this would be particularly useful given that the LAGr_Matrix being discussed is not opaque, and so we may need to notify the user of a lot more possible errors (e.g. "Hey, you told me this matrix was symmetric and it's not!").
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<#4 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AATVMEZLZ2IFKZVHIWG6F3TRLR2SRANCNFSM4LZC7HEQ>.
|
Do we use GrB_Info or have LAGr_Info that contains (possibly) a GrB_Info and a much richer set of information.
Is there an LAGr_error() that does analogous tasks to GrB_error() or something completely different.
The text was updated successfully, but these errors were encountered: