Skip to content
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

Explore where pydantic might help with UX #2860

Open
fselmo opened this issue Mar 6, 2023 · 4 comments
Open

Explore where pydantic might help with UX #2860

fselmo opened this issue Mar 6, 2023 · 4 comments

Comments

@fselmo
Copy link
Collaborator

fselmo commented Mar 6, 2023

Explore the idea of using pydantic dependency to better validate request object types and their params. This could help with field validation and error messages. There may be some UX improvements that are worth exploring there.

@e3243eric
Copy link
Contributor

e3243eric commented Mar 7, 2023

I think this idea is great for runtime type checking. The objects need to inherit from BaseModel of pydantic to be parsed by pydantic. Does this mean we need to update all the classes in types.py?

@XiaoHuiHui233
Copy link

XiaoHuiHui233 commented Mar 7, 2023

I'm very much looking forward to this feature. In my work, I encapsulate a bunch of pydantic Models outside of web3.py to represent some on-chain data structures. This is extremely friendly to the serialization. It can check various forms of data structure input and serialize them into different outputs in different situations. If web3.py can provide this capability, it will greatly reduce the workload in serialization, which is very effective for encapsulating high-level web interfaces and other tasks.

But actually there are some problems here. In my work, I use orjson as json serialization tool. But it was developed in Rust and converted to Python with pyo3. In orjson, integer data exceeding 64 bits cannot be processed, but most integer types in web3.py (such as Wei) will definitely exceed this limit. This is not a big problem in the json standard library (although it does not conform to the json specification). And pydantic uses the json standard library by default, this cannot be counted as a problem with pydantic. But the json standard library is notoriously slow, so I still think it's going to be a problem.

Also, pydantic V2 is expected. V2 provides more powerful capabilities that can save a lot of development costs and redundant code. It will be more efficient due to also using Rust for refactoring. Since Rust's json library is used (rather than relying on other Python json libraries), it is expected to solve the json problem just mentioned. However, there is no release version of V2 yet, and it is still in the testing process, and some fatal bugs will be found in actual use. In its documentation, the expectation for the official release of V2 is 2023Q1. The discussion on this aspect may have to wait for the official release of V2.

@fselmo
Copy link
Collaborator Author

fselmo commented Mar 7, 2023

Nice to see some discussion on this already!

@e3243eric I opened this as more of an exploratory "research" issue to track pros, cons, possible improvements, etc. There's not a real void to be filled with a PR quite yet but happy to see the interest coming in for it!

@XiaoHuiHui233, thanks for getting the conversation started and providing some solid input here. Very much appreciated.

Looks like some benchmarking could be useful. It might be nice to benchmark on a subset of calls / types to compare against the current state of things. Yeah, looks like V2 release is slated for Q1 of 2023 - will be good to keep an eye out for it. I don't quite have the bandwidth to dive in here at the moment but being as this would likely be part of the next major version of web3.py (v7), I think this is a good time to start the conversation and see how it evolves until then.

@DefiDebauchery
Copy link
Contributor

DefiDebauchery commented Mar 13, 2023

That's funny, I was literally thinking about this earlier but was worried about pushback on adding another dependency.

Would the idea be that this replaces AttributeDicts into structured request and response classes based on submitted RPCEndpoint?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants