Conversation
…tions as a dependency
Krastanov
left a comment
Krastanov
left a comment
There was a problem hiding this comment.
Thanks for looking into that, the help is really appreciated! I left a few requests for changes.
There was a problem hiding this comment.
Do we need this file? I would prefer to stick to the default unless there are specific reasons to avoid it.
| Documenter = "e30172f5-a6a5-5a46-863b-614d45cd2de4" | ||
| DocumenterCitations = "daee34ce-89f3-4625-b898-19384cb65244" |
There was a problem hiding this comment.
these should not be dependencies of the project as a whole, only of the docs
| ## References | ||
|
|
||
| ```@bibliography | ||
| ``` |
There was a problem hiding this comment.
This should probably be a more central page, as it will contain all references ever added, not just the references about centrality. I would suggest making a new page dedicated just to references.
|
I will mark this as a draft for now to keep my review queue more organized. Please do not hesitate to mark it back to not a draft whenever you have time to engage with it. |
3 participants
In this PR, I have used DocumenterCitations for citing sources of different algorithms used in Graphs.jl package. Currently, I have made changes only to
centralityalgorithm API. Once these changes look good, I will incorporate the changes to other files as well.