-
Notifications
You must be signed in to change notification settings - Fork 18
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
Allow import/export from JSON #136
Comments
Format of what? The gef files, or the state we keep internally when we parsed it? |
The whole |
Right, there is room for optimization here. I think we should consider a naive For instance the columnar format of the |
Why not use pydantic's BaseModel which makes it easy with json() and parse_raw() |
Hi @tversteeg, this feature would be useful for some services I'm running. Ok if I put a PR in for this enhancement? |
Any PRs are greatly appreciated! Pydantic looks like a nice solution for this problem. |
The current format is not serializable, so it can't be non-trivially sent over a network connection. We need to add
Cpt.from_json()
andCpt.to_json()
methods (same for Borehole).The text was updated successfully, but these errors were encountered: