Skip to content

Update hypergraph_matrix.py#682

Merged
kaiser-dan merged 1 commit intoxgi-org:mainfrom
GavinAnderberg:patch-1
Feb 26, 2026
Merged

Update hypergraph_matrix.py#682
kaiser-dan merged 1 commit intoxgi-org:mainfrom
GavinAnderberg:patch-1

Conversation

@GavinAnderberg
Copy link
Copy Markdown
Contributor

Fixed documentation

Try running the function on your own. It seems like the example def. is more accurate to the behavior than the rest of the docstring.

Fixed documentation
Copy link
Copy Markdown
Collaborator

@kaiser-dan kaiser-dan left a comment

Choose a reason for hiding this comment

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

Hey sorry it took so long to get to reviewing this!

A concise, helpful change that does indeed correct an inconsistency in the function default kwargs and the docstring. Seems great to me, thank you for the contribution @GavinAnderberg !

I'm new to the xgi maintainer team so I am not going to merge this yet without checking with the more senior members, but I've marked it approved.

@kaiser-dan kaiser-dan added the documentation Improvements or additions to documentation label Feb 26, 2026
@kaiser-dan kaiser-dan merged commit a962e0e into xgi-org:main Feb 26, 2026
3 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants