Skip to content

General update: cleaning up the README and fixes #339, #367#529

Merged
ZigRazor merged 13 commits intoZigRazor:masterfrom
brghash:update
Sep 20, 2025
Merged

General update: cleaning up the README and fixes #339, #367#529
ZigRazor merged 13 commits intoZigRazor:masterfrom
brghash:update

Conversation

@brghash
Copy link
Copy Markdown
Contributor

@brghash brghash commented Sep 18, 2025

Summary of changes

  • Implemented the ability to use non-extractable types
  • Improved getNodeSet and getAdjList efficiency
  • Tided up the codebase, especially the README

Why

The main motivation for this PR was to improve the experience a new user has. I have personally found the README too cluttered and providing extra information that I do not require such as the implementation of the algorithms. The README should provide a point of entry into the codebase, showing the features, not providing explanations for everything, this is the role of the website and documentation. I have added in an example into the README and moved items around so as not to obscure the core information.

Related issue

Notes for reviewers

  • Please check the README to confirm changes

Testing

  • Added removed connectivity test
  • All tests were passed

Copy link
Copy Markdown
Owner

@ZigRazor ZigRazor left a comment

Choose a reason for hiding this comment

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

It does not build, as you can see from CI/CD

@brghash
Copy link
Copy Markdown
Contributor Author

brghash commented Sep 19, 2025

I expect that the failure of the windows build is due to #530 which was present in prior versions of CXXGraph for me. I will investigate this further, soon.

@ZigRazor ZigRazor merged commit 4c46755 into ZigRazor:master Sep 20, 2025
12 of 20 checks passed
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.

Use Adjacency Matrix to populate the nodeSet. Support default serialization for pointer types

2 participants