search

boostorg / graph

Boost.org graph module
http://boost.org/libs/graph
319 stars 204 forks source link

Make comments match actual values of the random graph's number of ver… #339

Closed svengato closed 1 year ago

svengato commented 1 year ago

…tices and probability of edge connection

svengato commented 1 year ago

I noticed the mismatches while working on pull request #315. The changes are only to comments and should not require testing.

jeremy-murphy commented 1 year ago

Even better might be to remove the numbers from the comments altogether?

E.g.:

    // Generate random graph with N vertices and probability P
    // of edge connection.
    static const size_t N = 20;
    static const double P = 0.1;

or

    // Generate random graph
    static const size_t N = 20; // Number of vertices
    static const double P = 0.1; // Probability of edge between two vertices
jeremy-murphy commented 1 year ago

Btw, there is a way to tell the CI not to run, which would be appropriate for these changes, by including some magic text in the commit message. I don't remember exactly what it is... instructions are probably on GitHub somewhere.

jeremy-murphy commented 1 year ago

Thanks again!

svengato commented 1 year ago

Btw, there is a way to tell the CI not to run, [...]

Added to my Things to learn list. Thanks -