nx-guides icon indicating copy to clipboard operation
nx-guides copied to clipboard

Notebook Explaining Centrality Algorithms solving issue #5535 from networkx Issues

Open 0ddoes opened this issue 3 years ago • 5 comments

Sir, I tried to convert it to .md file. However the Output Figures are not shown in the file. Can you help me to correct it. It would be nice to have a contribution guide to nx_guides. 😅

0ddoes avatar Apr 20 '22 19:04 0ddoes

The output figures can't be embedded into the markdown notebook and that's fine! The notebook is supposed to be executed and create the figures using github actions, which is then pushed to the nx-guides website.

MridulS avatar Apr 21 '22 07:04 MridulS

Sir, there was an online converter for jupyter notebooks to markdown which I used. Is there a better way?

0ddoes avatar Apr 21 '22 13:04 0ddoes

Sir, here the errors are that it cannot recognise the import seaborn as sns and access the polblogs.gml file that I have added in the data folder. Subsequent errors are due to same issue. How can I actually correct this? Please let me know.

0ddoes avatar Apr 30 '22 07:04 0ddoes

@MridulS Can you please help me with how I can correct this?

0ddoes avatar May 05 '22 06:05 0ddoes

Can you please help me with how I can correct this?

The CI was showing two failures: one due the fact that the notebook was not included in the site anywhere, and another due to the fact that heading levels were skipped. These are common problems so I went ahead and pushed up fixes for each. That should at least clear up the current CI failures... we'll see if we hit more now :)

rossbar avatar May 12 '22 06:05 rossbar

@0ddoes we would like to show the reasons and explain the algorithms in nx-guides, not only use the networkx functions to get the results. So while doing analysis of datasets we should be also looking at the results of the function calls and explain them. I'll go ahead and close this PR. If you have the time and interest please feel free to submit a new pull request :)

MridulS avatar Mar 31 '23 08:03 MridulS