Luis M. B. Varona
Luis M. B. Varona
> Indeed the formatting of `References` is all over the place. We do recommend using Chicago Manual of Style https://networkx.org/documentation/latest/developer/contribute.html#adding-references but we don't really enforce it on older docstrings. I'm...
I don't know how easy it is to automate a foolproof format check, but we can certainly create a script that checks for some basic Chicago-style indicators (placement of all...
@amcandio Your thoughts on next steps?
> > [@amcandio](https://github.com/amcandio) Your thoughts on next steps? > > I'd try to automate it somehow, otherwise it's going to be a huge manual effort for both coding and reviewing...
@amcandio @dschult, just want to know what other changes (if any) I should make besides the (not yet added) docstrings. Alternatively, if @gulshan-123's PR is already satisfactory, I totally don't...
@dschult I've made some of the changes in this commit: ``` Remove is_PEG alias for is_chordal; update triangulation Removed the is_perfect_elimination_graph alias for is_chordal, opting to add a sentence to...
@dschult, perhaps you are right regarding `weight`/`weights`—I'll change it back. As for changing `i` to `k` in the loop, I refactored the logic so that we're iterating forward in `i`...
@dschult, I've made the requested changes. Additionally, I noticed some inconsistencies in the way `References` sections were formatted as I went to add one to `is_chordal`, so my obsessive Political...
@dschult @amcandio I am, subject to your approval, done with the PR and ready to merge. Let me know if you have any further comments and/or would like me to...
@amcandio @dschult I've made the last of the required changes (I hope, at least). Let me know if anything further is needed before merging, and thanks again for the feedback!...