Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Enhance remove_false_nodes Function: Retain Line Attributes and Expose aggfunc Parameter. #682

Closed
wants to merge 3 commits into from

Conversation

speco29
Copy link

@speco29 speco29 commented Feb 13, 2025

Fixes #447

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 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.
@martinfleis
Copy link
Member

Hi, sorry. This function has been moved to https://github.com/uscuni/neatnet where the aggregation has been implemented already.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

keep line attributes in remove_false_nodes
2 participants