Introduce component key for all SDK parameters

[aakrem]

The code here and here requires refactoring.
At some point it will be very challenging to build sophisticated ui components (A component that includes a InputParam and TextParam )
The reason of having that code is the way sdk returns the x-parameter, which is based on data types; boolean string integer...

As these parameters are mainly definitions that will be used in ui one solution could be is to add another component key in all SDK parameters. And of course remove the logic behind type.

SDK Code

class MultipleChoiceParam(str):
    def __new__(cls, default: str = None, choices: List[str] = None): component: str = None):
        if options is None:
            options = {}
   ...
    @classmethod
    def __modify_schema__(cls, field_schema: dict[str, Any]):
        field_schema.update(
            {
                "x-parameter": "choice",
                "type": "string",
                "component": component,
            }
        )

LLM App Code

param = MultipleChoiceParam(default="gpt-3.5-turbo", options=provider_models, component="model")

Then in the playground what we can do is to display sdk parameters per component instead of if-elses based on data type.

[mmabrouk]

@aakrem The issue is currently not clear at all (although I totally agree that there is a problematic). I am not able to understand your proposal.

Can you please explain the following:

The code here and here requires refactoring.

Can you please provide some context?

At some point it will be very challenging to build sophisticated ui components (A component that includes a textfield and a )

a what? Can you please provide more examples of UI components that we would like to provide in the future, and where do you see the current solution failing?

param = GroupedMultipleChoiceParam(default="gpt-3.5-turbo", options=provider_models, component="model")

This line comes out of nowhere. What do you mean here? Maybe a before and after example?

[mmabrouk]

I am converting this to discussion