-
Notifications
You must be signed in to change notification settings - Fork 337
Document serializable and non-serializable types in Mesop state #1327
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
Conversation
Co-authored-by: richard-to <[email protected]>
…xample Co-authored-by: richard-to <[email protected]>
Co-authored-by: richard-to <[email protected]>
Co-authored-by: richard-to <[email protected]>
Co-authored-by: richard-to <[email protected]>
…improve proto extraction example Co-authored-by: richard-to <[email protected]>
Co-authored-by: richard-to <[email protected]>
… replace troubleshooting with workarounds Co-authored-by: richard-to <[email protected]>
richard-to
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…py arrays, and enums Co-authored-by: richard-to <[email protected]>
Added Known Issues section in 7cbbe89 with warnings about:
Each includes a brief description and workaround suggestion. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
docs/guides/state-management.md
Outdated
|
|
||
| Be aware of these known serialization limitations: | ||
|
|
||
| - **Dataclass inheritance**: Classes that inherit from other dataclasses may not serialize correctly. See [issue #659](https://github.com/mesop-dev/mesop/issues/659). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The info here is not exactly correct. The descriptions you mentioned do no match the issue descriptions. Let's just link to the issue. Just use bulleted list and post the issues in order
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Simplified to just a bulleted list of issue links in the requested order in c86a80a.
…ions Co-authored-by: richard-to <[email protected]>
Add docs on what is / is not serializable in Mesop state
This PR addresses the issue where users frequently encounter serialization issues with Mesop state by adding comprehensive documentation.
✅ All Feedback Addressed:
📝 Summary of Changes:
Added comprehensive "Serialization" section to
/docs/guides/state-management.md:✓ Serializable Types (14 types documented)
int,float,str,boollist,dict,set(excluding tuples)datetime.datetime,datetime.datebytespandas.DataFrame,pydantic.BaseModel,UploadedFile✓ Non-Serializable Types (shortened)
✓ Workarounds for Unsupported Types (3 strategies)
✓ Known Issues
Each workaround includes complete working examples.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.