Fix optional args toolcalling by aymeric-roucher · Pull Request #1538 · huggingface/smolagents (original) (raw)

I think there might be some confusion around the term "nullable" in this context.

We should clarify terminology: "nullable" can be interpreted in two ways:

• either that the argument can explicitly be set to None,
• or that it's optional in the sense that it has a default value and doesn't need to be passed at all.

I think that distinction is important when reasoning about how LLMs should construct tool arguments.