Skip to content

Conversation

zhenrongliew
Copy link
Contributor

Worked with @kaihsin

Replaced custom serialization methods with standardized construction of SerializationUnit objects.

Keep handling Python builtins and IR objects in Serializer & Deserializer class.
Added runtime checks for supported serializable types for Python attributes and literals.
Cleaned up the serialization/__init__.py file.
Added new entry point for encode and decode methods to DialectGroup for program serialization and deserialization.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 14, 2025

PR Preview Action v1.6.2

🚀 View preview at
https://QuEraComputing.github.io/kirin/pr-preview/pr-532/

Built to branch gh-pages at 2025-10-20 22:42 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@codecov
Copy link

codecov bot commented Oct 14, 2025

@github-actions
Copy link
Contributor

github-actions bot commented Oct 14, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
11331 10092 89% 0% 🟢

New Files

File Coverage Status
src/kirin/serialization/core/_init_.py 100% 🟢
src/kirin/serialization/core/supportedtypes.py 100% 🟢
TOTAL 100% 🟢

Modified Files

File Coverage Status
src/kirin/dialects/func/attrs.py 82% 🟢
src/kirin/dialects/ilist/runtime.py 89% 🟢
src/kirin/interp/abc.py 89% 🟢
src/kirin/ir/attrs/py.py 89% 🟢
src/kirin/ir/attrs/types.py 89% 🟢
src/kirin/ir/dialect.py 85% 🟢
src/kirin/ir/group.py 91% 🟢
src/kirin/ir/method.py 95% 🟢
src/kirin/ir/nodes/block.py 80% 🟢
src/kirin/ir/nodes/region.py 77% 🟢
src/kirin/ir/nodes/stmt.py 85% 🟢
src/kirin/ir/ssa.py 83% 🟢
src/kirin/serialization/_init_.py 100% 🟢
src/kirin/serialization/base/_init_.py 100% 🟢
src/kirin/serialization/base/context.py 95% 🟢
src/kirin/serialization/base/deserializer.py 82% 🟢
src/kirin/serialization/base/serializer.py 80% 🟢
src/kirin/serialization/jsonserializer.py 97% 🟢
TOTAL 88% 🟢

updated for commit: 5b80bff by action🐍

Copy link
Collaborator

@kaihsin kaihsin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just some minor comment

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants