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

Updating the Contributing file #683

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

speco29
Copy link

@speco29 speco29 commented Feb 15, 2025

Fixes #633
Added the ruff and pre-commit in the contributing file.

Added the missing dependency
Key Changes:
Added retain_attributes and aggfunc parameters to the function signature.

Created a list new_attrs to store the aggregated attributes if retain_attributes is True.

Aggregated attributes using aggfunc and added them to new_attrs during the loop that processes connected components.

Combined final_df with the new attributes if retain_attributes is True.
Added the ruff and pre-commit in ontributing.
@martinfleis
Copy link
Member

please clean up the commit history here

@martinfleis
Copy link
Member

Try that again pls :)

@speco29
Copy link
Author

speco29 commented Feb 15, 2025

Try that again pls :)

Are you asking for the deletion of commits?

@martinfleis
Copy link
Member

The only change in this PR should relate to the contributing guide. You're now deleting two pages of user guide.

@speco29
Copy link
Author

speco29 commented Feb 16, 2025

ok, I apologize for the mess, these commits were created in different branch still they got added here, I don't know how -_-
I'm new to open source so right now I don't know how to clear the commit history, I'll look through it and will resolve it soon :)

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.

Update contributing formatting part
2 participants