-
Notifications
You must be signed in to change notification settings - Fork 191
Migrate to pydantic >= 2.0 #613
New issue
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
Changes from 28 commits
24aabf8
62bd489
8801689
78cfaed
51542ed
b254425
945176a
6ca96f4
5cfe829
917eda7
c968b58
c1dc7e2
42aa111
6f43d74
46efb9b
3045008
d4a4d26
055d1ce
df9f4e9
a55e527
3d3e444
bc5a52f
32a1c68
cdb3de8
3e22780
eba80fa
a8ae34b
84e225d
5a8c31f
47d932c
da4d798
3cd9d38
d3593eb
19004cc
b2d95c8
ad080ed
6df0176
13bfad3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Vectorrent marked this conversation as resolved.
Show resolved
Hide resolved
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
services: | ||
lab: | ||
command: pytest -s -vv /home/hivemind/tests/test_dht_schema.py | ||
# command: pytest -s -vv /home/hivemind/tests | ||
tty: true | ||
stdin_open: true | ||
build: . | ||
volumes: | ||
- ./:/home/hivemind | ||
# environment: | ||
# HIVEMIND_LOGLEVEL: DEBUG |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
transformers~=4.6 | ||
datasets~=1.5 | ||
torch_optimizer==0.1.0 | ||
wandb==0.10.26 | ||
wandb==0.17.1 | ||
Vectorrent marked this conversation as resolved.
Show resolved
Hide resolved
|
||
sentencepiece | ||
requests | ||
nltk==3.6.7 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,5 +12,5 @@ configargparse>=1.2.3 | |
py-multihash>=0.2.3 | ||
multiaddr @ git+https://github.com/multiformats/py-multiaddr.git@e01dbd38f2c0464c0f78b556691d655265018cce | ||
cryptography>=3.4.6 | ||
pydantic>=1.8.1,<2.0 | ||
pydantic>=2.5.3 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This version is quite recent, can you provide any reasoning behind how it was chosen? Maybe we can bump to just 2.0? Ideally, we should even keep backwards compatibility with older versions There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I chose this version because it was the highest possible version I could use, which was still compatible with all of the tests. Remember: the whole reason we're upgrading Pydantic is because this old version has been conflicting with other dependencies, in other projects. We can revert to 2.0.0 if you still want me to do that, though 2.5.3 seems to be working fine. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. IMO, If the code is still compatible with pydantic v1 it might be worth it to just do |
||
packaging>=20.9 |
Uh oh!
There was an error while loading. Please reload this page.