Skip to content

Fix cycle count for buckminsterfullerene example #58

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.

Sign up for GitHub

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

Merged
pckroon merged 1 commit into from
Jan 13, 2026
Merged

Fix cycle count for buckminsterfullerene example #58

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ blogposts with a decent writeup on the subject can be found [here][depth_first].
Despite how simple it seems at a glance, here be dragons lurking in the details.
The main problem is that in order to determine whether an edge is aromatic,
you need to generate *all* simple cycles. To illustrate why this is a problem,
let's look at buckminsterfullerene: it contains a staggering *15,024,073*
let's look at buckminsterfullerene: it contains a staggering *374,237,206*
cycles! It's really not feasible to investigate all of those separately.

Therefor we approximate the aromaticity for ring systems that are too large. By
Expand Down
Loading