✨ Support json (dict) as parameter type
#985
Conversation
Signed-off-by: Muhammed Hussein Karimi <[email protected]>
Signed-off-by: Muhammed Hussein Karimi <[email protected]>
Signed-off-by: Muhammed Hussein Karimi <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
Signed-off-by: Muhammed Hussein Karimi <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Signed-off-by: Muhammed Hussein Karimi <[email protected]>
mhkarimi1383
changed the title
This comment was marked as outdated.
This comment was marked as outdated.
Signed-off-by: Muhammed Hussein Karimi <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
|
@svlandeg it's ok to ignore coverage on newly created class? |
|
No, in general we really need all the code in the main modules to be fully tested 🙏 |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
I have added tests for newly created class coverage is 100% again :) |
|
If changes are Ok so far I can work on Pydantic support, Or move Pydantic support in another PR? |
|
HI @mhkarimi1383! We haven't yet been able to review this PR in detail. When we do, we'll update here. No need to ping individual maintainers in the meantime 😉 In terms of adding more functionality, it's always best to open separate PRs for distinct functionality. It makes the review process much easier if one PR deals with one "atomic" change. Do have a look at existing PRs - if I recall correctly there's already a few PRs open with proposals for Pydantic support. |
|
Thanks again for this PR @mhkarimi1383! I'm going to start reviewing it in detail and will push some changes directly to your branch as I'm working on this. I'll put this PR in draft while I do so. |
This comment was marked as outdated.
This comment was marked as outdated.
svlandeg
changed the title
json (dict) as parameter type
📝 Docs previewLast commit d8b145d at: https://c94a95d7.typertiangolo.pages.dev Modified Pages |
|
I have updated the description to not include bytes support |
tests/test_tutorial/test_parameter_types/test_dict/test_tutorial001.py
Outdated
Show resolved
Hide resolved
tests/test_tutorial/test_parameter_types/test_dict/test_tutorial001_an.py
Outdated
Show resolved
Hide resolved
github-actions
bot
added
the
conflicts
This comment was marked as resolved.
This comment was marked as resolved.
github-actions
bot
added
conflicts
This comment was marked as resolved.
This comment was marked as resolved.
|
@svlandeg if this will not get merged I will close it... |
github-actions
bot
removed
the
conflicts
3 participants
Hi
Sometimes there is a need of decoding entire value as an json object (useful when some params are too dynamic or getting that option in a prompt). I wanted to also add pydantic support (for both single option as json and all of the options in a single model) there and also use orjson (by checking if that's installed as an optional dependency), If that's Ok.