Skip to content

Fix have_negative() using correct constraint attributes#122

Closed
VaibhavSurolia wants to merge 1 commit into52North:mainfrom
VaibhavSurolia:fix-have-negative-bug
Closed

Fix have_negative() using correct constraint attributes#122
VaibhavSurolia wants to merge 1 commit into52North:mainfrom
VaibhavSurolia:fix-have-negative-bug

Conversation

@VaibhavSurolia
Copy link

@VaibhavSurolia VaibhavSurolia commented Jan 20, 2026

Issue

The method have_negative() referenced a non-existent attribute
neg_size, causing a runtime error, related to #118 issue.

Fix

Updated the method to correctly use neg_dis_size and
neg_cont_size.

Impact

Prevents AttributeError and ensures correct constraint detection.

Resolved AttributeError caused by invalid neg_size reference.
@kdemmich
Copy link
Collaborator

kdemmich commented Feb 2, 2026

Hi @VaibhavSurolia, thank you for your PR. We appreciate the time and effort that was put into it but, unfortunately, we have to decline the PR due to the limited added value to the tool.

@kdemmich kdemmich closed this Feb 2, 2026
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.

2 participants