Skip to content

[AI-FSSDK] [FSSDK-12750] Use attribute id instead of key for CMAB prediction requests#648

Open
raju-opti wants to merge 1 commit into
masterfrom
ai/raju-opti/FSSDK-12750-cmab-attribute-id-fix
Open

[AI-FSSDK] [FSSDK-12750] Use attribute id instead of key for CMAB prediction requests#648
raju-opti wants to merge 1 commit into
masterfrom
ai/raju-opti/FSSDK-12750-cmab-attribute-id-fix

Conversation

@raju-opti

@raju-opti raju-opti commented Jul 1, 2026

Copy link
Copy Markdown

Summary

The CMAB service was using attribute keys instead of attribute IDs when building the filtered attribute map for prediction requests. The CMAB prediction endpoint expects attribute IDs in the JSON payload, so this change corrects the map key from the attribute's key to its ID.

Changes

  • Updated the filtered attribute map in CmabService to use attribute ID as the dictionary key instead of attribute key
  • Updated all related test assertions and cache hash computations to use attribute IDs
  • Fixed a pre-existing bug in MockProjectConfig where an attribute was incorrectly added to attributeIdMap instead of attributeKeyMap

Jira Ticket

FSSDK-12750

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage is 93.849%ai/raju-opti/FSSDK-12750-cmab-attribute-id-fix into master. No base build found for master.

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