fix: nested types, use dataclass_wizard to deserialize/serialize types#21
fix: nested types, use dataclass_wizard to deserialize/serialize types#21
Conversation
|
wooo! |
template/prepare.sh
Outdated
|
|
||
| # fetch dataclass wizard | ||
| if [ ! -d dataclass-wizard-0.33.0 ]; then | ||
| curl -O https://files.pythonhosted.org/packages/0a/2a/6ae6638cd5ca919b73d393cdb4a4e53c8c3dcacc003a79d9480ecad46798/dataclass-wizard-0.33.0.tar.gz |
There was a problem hiding this comment.
We're already using uv in pyproject.toml to manage dependencies - is there a reason not to do that for these as well?
There was a problem hiding this comment.
No, I was just unfamiliar with the dep management. Does it look right in a545a0e ? I had to specify a narrow python version to make sure it will match with the PYTHONPATH in xtp.toml
There was a problem hiding this comment.
That looks good - We should also be able to detect the path using something like:
PYTHONPATH=$(uv run python3 -c "import sys; print(sys.path[-1])")To avoid requiring a specific python version - but since uv also manages the required python versions it might not be a huge deal either way.
There was a problem hiding this comment.
Thanks, I tried that out and can't seem to embed the command in the toml so maybe we keep the hardcoding for now?
There was a problem hiding this comment.
Yeah that works for me! If it comes up again we can figure out a more generic solution.
3 participants
Uh oh!
There was an error while loading. Please reload this page.