Skip to content

Conversation

@corradodebari
Copy link
Member

This solves issue #198 since to set_profile is passed a float/int value. string conversion avoid issue in DBMS_CLOUD_AI execution

to set_profile is passed a float/int value. string conversion avoid issue in DBMS_CLOUD_AI  execution
@corradodebari corradodebari requested a review from gotsysdba June 25, 2025 13:13
@corradodebari corradodebari self-assigned this Jun 25, 2025
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jun 25, 2025
@gotsysdba
Copy link
Member

@corradodebari I was trying to duplicate this to write a test-case against it, but wasn't able. Can you give me example to reproduce to build the test against?

@corradodebari
Copy link
Member Author

Please, could you check if the attribute_value in :
def set_profile( conn: oracledb.Connection, profile_name: SelectAIProfileType, attribute_name: str, attribute_value: Union[str, list], ) -> None:
is a str/list type when used in binds?
I'm working currently with a db instance on docker with the documented process to enable select ai, but I don't think is related to. I've checked that the attribute_value, for token is Int, and for temperature arrives float, so I've got the error. I'll rebuild in parallel another env to see what's happen.

Copy link
Member

@gotsysdba gotsysdba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only impacts container image of ADB, but doesn't break ADB-S in OCI.

@corradodebari corradodebari merged commit 90cb55b into main Jun 30, 2025
2 checks passed
@corradodebari corradodebari deleted the 198-selectai-issue branch June 30, 2025 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants