Silvano Cerza
Silvano Cerza
We reverted PR #5933 with #5980 as `isort` and `black` were conflicting. Reopening.
This is an `arduino-builder` issue right? If that's the case am gonna move it into that repo and probably we'll never fix it. The `arduino-builder` is deprecated, unmaintained, old and...
Nice idea, I didn't think about it. I guess performance won't change but the binary size seems to be cut quite a bit, at least on Linux where I tested....
@masci This is not solved, the change to `PromptBuilder` is just a workaround, even though it still a sensible change for the Component. Ideally we could fix this in `Pipeline.run()`...
This is actually intended. We made the decision to make all `PromptBuilder` inputs as optional to support some use cases like self correcting loops in a Pipeline. If the inputs...
> I would prefer that we can have an optional parameter during `PromptBuilder` initialization to tell whether to make (some of) the jinja inputs mandatory though (this also avoids breaking...
Awesome! I'll be on the lookout for the updated PR, feel free to ping me directly if you want. 💪
Closing as fixed by #7553.
Nice! PR looks great, there's some linting issues that need to be fixed though. I see `pylint`, `mypy` and `black` failing, should be easy fixes. You can run those quite...