We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ankush pointed out that we're passing in the model parser id instead of the model id, which will cause serialize() to crash: https://github.com/lastmile-ai/aiconfig/blob/main/extensions/HuggingFace/python/src/aiconfig_extension_hugging_face/local_inference/text_2_image.py#L229
serialize()
Instead we should be passing in the model from the data param and using that: https://github.com/lastmile-ai/aiconfig/blob/main/python/src/aiconfig/default_parsers/openai.py#L86
We should also be building automated testing for serialize (similar to Dall-E) so that we don't run into this issue again
The text was updated successfully, but these errors were encountered:
Similarly, should support the "model" param arg for deserialize(): #883 (comment)
Sorry, something went wrong.
Note that ASR call supports model param: #883 (comment)
model
No branches or pull requests
Ankush pointed out that we're passing in the model parser id instead of the model id, which will cause
serialize()
to crash: https://github.com/lastmile-ai/aiconfig/blob/main/extensions/HuggingFace/python/src/aiconfig_extension_hugging_face/local_inference/text_2_image.py#L229Instead we should be passing in the model from the data param and using that: https://github.com/lastmile-ai/aiconfig/blob/main/python/src/aiconfig/default_parsers/openai.py#L86
We should also be building automated testing for serialize (similar to Dall-E) so that we don't run into this issue again
The text was updated successfully, but these errors were encountered: