Skip to content

Conversation

@RaoulLuque
Copy link
Member

@RaoulLuque RaoulLuque commented May 31, 2025

This PR updates the docs.rs by mainly replacing the existing example by several other ones, which go into more detail. It also restructures the main docs while leaving most of the information the same by adding an overview section.

The goal of this is to offer a more smooth introduction into petgraph, by increasing the clarity of the docs and adding a usage section to help people that are new to rust get started. With a similar intent, more examples are added to show off some of petgraphs' features.

Among other things, this resolves #804.

For a preview of the new docs, just run

cargo doc --open

after cloning this PR branch.

Note that for the revamp I took inspiration from the documentation style of several of @BurntSushi 's crates.

Copy link
Collaborator

@starovoid starovoid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for your heroic work in writing documentation and examples! I'm only offering a few minor fixes.

@starovoid starovoid added this to the 0.8.2 milestone May 31, 2025
RaoulLuque and others added 2 commits May 31, 2025 21:33
Co-authored-by: Egor Starovoitov <52821033+starovoid@users.noreply.github.com>
@RaoulLuque RaoulLuque requested a review from starovoid May 31, 2025 21:47
@starovoid starovoid added this pull request to the merge queue Jun 1, 2025
Merged via the queue into petgraph:master with commit f5f791c Jun 1, 2025
14 checks passed
@github-actions github-actions bot mentioned this pull request May 31, 2025
@RaoulLuque RaoulLuque mentioned this pull request Jun 1, 2025
github-merge-queue bot pushed a commit that referenced this pull request Jun 6, 2025
## 🤖 New release

* `petgraph`: 0.8.1 -> 0.8.2 (✓ API compatible changes)

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.8.2](https://github.com/petgraph/petgraph/compare/petgraph@v0.8.1...petgraph@v0.8.2)
- 2025-06-06

### Bug Fixes

- Ford Fulkerson sometimes Panics on StableGraphs
([#793](#793))
- Run Maximal Cliques Quickcheck only on Digraphs which are symmetrical
([#800](#800))
- Run Steiner Tree Quickcheck on the connected components to properly
support disconnected graphs
([#801](#801))
- Quickcheck random01 function only outputs 0
([#798](#798))

### Documentation

- Specify that Acyclic::try_udpate_edge may add an edge
([#770](#770))
- Update remove_node doc comment in graphmap.rs
([#663](#663))
- Add examples to minimum spanning tree functions
([#808](#808))
- Minimal typo fix in comments
([#803](#803))
- Update docs.rs ([#807](#807))
- Add note about `StableGraph::edge_indices` behaviour
([#812](#812))
- Clarification of references to nodes and V (refresh #358)
([#814](#814))
- Fix link and mention Dfs and Bfs as special case in examples
([#816](#816))
- Unify algo docs
([#815](#815))

### New Features

- *(parser)* allow parsing graphs from Dot/Graphviz files
([#653](#653))
- Implement `DataMap` for `GraphMap` graphs
([#776](#776))
- Add Johnson's algorithm
([#741](#741))
- Add algorithm to find bridge edges
([#590](#590))

### Performance

- Reuse queue allocation in `maximum_matching` main loop
([#817](#817))

### Refactor

- Fix new clippy warnings
([#791](#791))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: starovoid <prototyperailgun@gmail.com>
github-merge-queue bot pushed a commit that referenced this pull request Jun 9, 2025
…823)

This is a minor change which just fixes some problems I just noticed,
which were introduced in the previous docs revamp #807.

Namely:
- In the README, the name of the examples section has to be updated from
`example` to `examples`
- In the Example about `Using algorithms on graphs`, the `visit` module
does not enjoy the same typesetting as the other modules.
RaoulLuque added a commit to RaoulLuque/petgraph that referenced this pull request Jun 18, 2025
This PR updates the docs.rs by mainly replacing the existing example by
several other ones, which go into more detail. It also restructures the
main docs while leaving most of the information the same by adding an
overview section.

The goal of this is to offer a more smooth introduction into petgraph,
by increasing the clarity of the docs and adding a usage section to help
people that are new to rust get started. With a similar intent, more
examples are added to show off some of petgraphs' features.

Among other things, this resolves petgraph#804. 

For a preview of the new docs, just run
```
cargo doc --open
```
after cloning this PR branch.

Note that for the revamp I took inspiration from the documentation style
of several of [@BurntSushi 's crates](https://github.com/burntsushi).

---------

Co-authored-by: Egor Starovoitov <52821033+starovoid@users.noreply.github.com>
RaoulLuque pushed a commit to RaoulLuque/petgraph that referenced this pull request Jun 18, 2025
## 🤖 New release

* `petgraph`: 0.8.1 -> 0.8.2 (✓ API compatible changes)

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.8.2](https://github.com/petgraph/petgraph/compare/petgraph@v0.8.1...petgraph@v0.8.2)
- 2025-06-06

### Bug Fixes

- Ford Fulkerson sometimes Panics on StableGraphs
([petgraph#793](petgraph#793))
- Run Maximal Cliques Quickcheck only on Digraphs which are symmetrical
([petgraph#800](petgraph#800))
- Run Steiner Tree Quickcheck on the connected components to properly
support disconnected graphs
([petgraph#801](petgraph#801))
- Quickcheck random01 function only outputs 0
([petgraph#798](petgraph#798))

### Documentation

- Specify that Acyclic::try_udpate_edge may add an edge
([petgraph#770](petgraph#770))
- Update remove_node doc comment in graphmap.rs
([petgraph#663](petgraph#663))
- Add examples to minimum spanning tree functions
([petgraph#808](petgraph#808))
- Minimal typo fix in comments
([petgraph#803](petgraph#803))
- Update docs.rs ([petgraph#807](petgraph#807))
- Add note about `StableGraph::edge_indices` behaviour
([petgraph#812](petgraph#812))
- Clarification of references to nodes and V (refresh petgraph#358)
([petgraph#814](petgraph#814))
- Fix link and mention Dfs and Bfs as special case in examples
([petgraph#816](petgraph#816))
- Unify algo docs
([petgraph#815](petgraph#815))

### New Features

- *(parser)* allow parsing graphs from Dot/Graphviz files
([petgraph#653](petgraph#653))
- Implement `DataMap` for `GraphMap` graphs
([petgraph#776](petgraph#776))
- Add Johnson's algorithm
([petgraph#741](petgraph#741))
- Add algorithm to find bridge edges
([petgraph#590](petgraph#590))

### Performance

- Reuse queue allocation in `maximum_matching` main loop
([petgraph#817](petgraph#817))

### Refactor

- Fix new clippy warnings
([petgraph#791](petgraph#791))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: starovoid <prototyperailgun@gmail.com>
RaoulLuque added a commit to RaoulLuque/petgraph that referenced this pull request Jun 18, 2025
…etgraph#823)

This is a minor change which just fixes some problems I just noticed,
which were introduced in the previous docs revamp petgraph#807.

Namely:
- In the README, the name of the examples section has to be updated from
`example` to `examples`
- In the Example about `Using algorithms on graphs`, the `visit` module
does not enjoy the same typesetting as the other modules.
github-merge-queue bot pushed a commit that referenced this pull request Jun 19, 2025
This PR resolves #766.

With the [new docs and
README](#807), using cargo
readme as suggested in the issue seems unsuited, since the templating
does not support including parts of the lib docs in the README.
RaoulLuque added a commit to RaoulLuque/petgraph that referenced this pull request Sep 21, 2025
This PR resolves petgraph#766.

With the [new docs and
README](petgraph#807), using cargo
readme as suggested in the issue seems unsuited, since the templating
does not support including parts of the lib docs in the README.
@RaoulLuque RaoulLuque deleted the update_docs_rs branch October 1, 2025 16:10
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.

Fix example in docs.rs example

2 participants