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
chore!: Hard-deprecate as_adjacency_matrix(edges = )
#1080
Conversation
Current Aviator status
This PR was merged using Aviator. See the real-time status of this PR on the Aviator webapp. Use the Aviator Chrome Extension to see the status of your PR within GitHub.
|
37f3c28
to
36fa375
Compare
@maelle: This also affects |
See the failing tests with this PR. |
32a53e1
to
b7f8bb6
Compare
as_adjacency_matrix(edges = )
as_adjacency_matrix(edges = )
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll run revdepchecks.
if (!sparse) { | ||
get.adjacency.dense(graph, type = type, attr = attr, weights = NULL, names = names) | ||
} else { | ||
if (sparse) { | ||
get.adjacency.sparse(graph, type = type, attr = attr, edges = edges, names = names) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Passing edges
now looks like dead code, but get.adjacency.sparse()
will be implemented in C anyway, perhaps no need to act.
#' @param edges `r lifecycle::badge("deprecated")` Logical scalar, whether to return the edge ids in the matrix. | ||
#' For non-existant edges zero is returned. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we just mention that this must be omitted?
if (!missing(edges)) { | ||
warning("The `edges` argument of `as_adjacency_matrix` is deprecated; it will be removed in igraph 1.4.0") | ||
if (!missing(edges) && isTRUE(edges)) { | ||
lifecycle::deprecate_stop("2.0.0", "as_adjacency_matrix(edges = )") | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a deprecate_soft()
if edges
is FALSE
?
59ad321
to
c85c8f0
Compare
revdepchecks look fine. |
c85c8f0
to
89a1d4b
Compare
Closes #815.