Skip to content

Change "model scale" node to "get model info" node #3090

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

Merged
merged 4 commits into from
Mar 12, 2025

Conversation

tumuyan
Copy link
Contributor

@tumuyan tumuyan commented Mar 6, 2025

Output additional model information, similar to #3089
The original "get model scale" node has been modified and renamed, but since the schema_id unchanged and the scale is set as output0, it will not affect any existing *.chn files contain the "get model scale" node.

image

@tumuyan tumuyan force-pushed the pr_get_model_info_node branch from fc56a07 to 14fac89 Compare March 6, 2025 14:18
@tumuyan tumuyan force-pushed the pr_get_model_info_node branch from 14fac89 to 9771715 Compare March 6, 2025 16:34
@tumuyan tumuyan force-pushed the pr_get_model_info_node branch from 370d5d9 to 792d527 Compare March 6, 2025 18:38
@tumuyan
Copy link
Contributor Author

tumuyan commented Mar 6, 2025

update

image

joeyballentine
joeyballentine previously approved these changes Mar 12, 2025
Copy link
Member

@joeyballentine joeyballentine left a comment

Choose a reason for hiding this comment

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

Looks good to me

@joeyballentine
Copy link
Member

Oops, I guess there's some issues

@joeyballentine joeyballentine merged commit 04a9459 into chaiNNer-org:main Mar 12, 2025
14 checks passed
clausqr pushed a commit to clausqr/chaiNNer that referenced this pull request Jun 25, 2025
* Change get_model_scale node to get_model_info node

* Add get_model_info node for onnx model

* fix get_model_info_node

* fix pyright
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.

2 participants