Skip to content

added associatedEntity field#4

Merged
Tushar-Naik merged 5 commits intoPhonePe:mainfrom
Priyanshu0109:add-chimera-key-to-filter
Oct 31, 2025
Merged

added associatedEntity field#4
Tushar-Naik merged 5 commits intoPhonePe:mainfrom
Priyanshu0109:add-chimera-key-to-filter

Conversation

@Priyanshu0109
Copy link

@Priyanshu0109 Priyanshu0109 commented Jul 22, 2025

This change introduces key-aware filtering capabilities into the core evaluation engine by passing the evaluation key down to the filter execution layer. This is achieved by adding a generic associatedEntity field to the ConditionEngine and ConditionalMatcher interfaces, which is then populated with the current evaluation key in BonsaiTree.

This enhancement is foundational for enabling new filter Uniform Sampling Filter.

Bonsai's evaluation engine is the only component with direct, real-time access to the full context required for these new filter. The current key is only available within the BonsaiTree.evaluate() call stack. Placing the logic in Bonsai is efficient, as it avoids a complex and slow process of passing context back and forth with the consuming application.

@Tushar-Naik
Copy link
Contributor

Lets rename all references of associatedEntity with entityMetadata (M). This terminology seems more apt.
Rest of the changes look good to go.

@Tushar-Naik Tushar-Naik merged commit b96e243 into PhonePe:main Oct 31, 2025
0 of 2 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.

2 participants