Skip to content
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

Documentation validation #509

Merged
merged 2 commits into from
Dec 11, 2023
Merged

Documentation validation #509

merged 2 commits into from
Dec 11, 2023

Conversation

olivroy
Copy link
Collaborator

@olivroy olivroy commented Dec 8, 2023

I checked that no deprecated functions were present.

I added a note in NEWS for myself, but think it is worth including here for reference.

If I understood correctly, create_random_graph() is deprecated in favor of create_graph() + add_gnm_graph(), add_growing_graph(), add_islands_graph(), add_smallworld_graph() for the creation of randomized graphs

Fix #296
Fix #423

@rich-iannone
Copy link
Owner

This is all correct. Thanks for making all of these fixes for consistency in the docs!

@rich-iannone
Copy link
Owner

Are you able to merge yourself? If so, please go ahead at your leisure (the settings don't really indicate what the case is on your side).

@olivroy olivroy merged commit a1ce939 into rich-iannone:main Dec 11, 2023
9 checks passed
@olivroy olivroy deleted the docs branch December 11, 2023 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

create_random_graph mentioned in docs but not found? Provide link from create_node_df to node_aes
2 participants