Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Expose Model Explorer's visualize_from_config#14005

Merged
zingo merged 2 commits into
pytorch:mainfrom
jmahbs:expose-model-explorer-api
Sep 10, 2025
Merged

Expose Model Explorer's visualize_from_config#14005
zingo merged 2 commits into
pytorch:mainfrom
jmahbs:expose-model-explorer-api

Conversation

@jmahbs

@jmahbs jmahbs commented Sep 5, 2025

Copy link
Copy Markdown
Contributor

This PR exposes the Model Explorer visualisation API, which provides the foundation to enable us to load custom file types and plugins.

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

Change-Id: I64a01aba413d9d448690da621d5f79b78ecec5a7
@jmahbs jmahbs requested a review from Gasoonjia as a code owner September 5, 2025 13:29
@pytorch-bot

pytorch-bot Bot commented Sep 5, 2025

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/14005

Note: Links to docs will display an error until the docs builds have been completed.

❌ 4 New Failures, 4 Cancelled Jobs

As of commit a104159 with merge base fbdee8f (image):

NEW FAILURES - The following jobs have failed:

CANCELLED JOBS - The following jobs were cancelled. Please retry:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla

meta-cla Bot commented Sep 5, 2025

Copy link
Copy Markdown

Hi @jmahbs!

Thank you for your pull request and welcome to our community.

Action Required

In order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you.

Process

In order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with CLA signed. The tagging process may take up to 1 hour after signing. Please give it that time before contacting us about it.

If you have received this in error or have any questions, please contact us at [email protected]. Thanks!

@jmahbs

jmahbs commented Sep 5, 2025

Copy link
Copy Markdown
Contributor Author

@pytorchbot label ciflow/trunk
@pytorchbot partner: arm
@pytorchbot release notes: arm

@pytorch-bot

pytorch-bot Bot commented Sep 5, 2025

Copy link
Copy Markdown

To add these label(s) (ciflow/trunk) to the PR, please first approve the workflows that are awaiting approval (scroll to the bottom of this page).

This helps ensure we don't trigger CI on this PR until it is actually authorized to do so. Please ping one of the reviewers if you do not have access to approve and run workflows.

@zingo zingo added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk labels Sep 5, 2025
@pytorch-bot

pytorch-bot Bot commented Sep 5, 2025

Copy link
Copy Markdown

To add the ciflow label ciflow/trunk please first approve the workflows that are awaiting approval (scroll to the bottom of this page).

This helps ensure we don't trigger CI on this PR until it is actually authorized to do so. Please ping one of the reviewers if you do not have access to approve and run workflows.

@pytorch-bot

pytorch-bot Bot commented Sep 5, 2025

Copy link
Copy Markdown

To add the ciflow label ciflow/trunk please first approve the workflows that are awaiting approval (scroll to the bottom of this page).

This helps ensure we don't trigger CI on this PR until it is actually authorized to do so. Please ping one of the reviewers if you do not have access to approve and run workflows.

@pytorch-bot pytorch-bot Bot removed the ciflow/trunk label Sep 5, 2025

@mergennachin mergennachin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

sure

@digantdesai digantdesai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please add rationale for the change in the PR summary. Thanks.

@zingo zingo modified the milestone: 1.0.0 Sep 9, 2025
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 9, 2025
@pytorch-bot

pytorch-bot Bot commented Sep 10, 2025

Copy link
Copy Markdown

To add the ciflow label ciflow/trunk please first approve the workflows that are awaiting approval (scroll to the bottom of this page).

This helps ensure we don't trigger CI on this PR until it is actually authorized to do so. Please ping one of the reviewers if you do not have access to approve and run workflows.

@zingo zingo added the release notes: none Do not include this in the release notes label Sep 10, 2025
@zingo

zingo commented Sep 10, 2025

Copy link
Copy Markdown
Collaborator

Test fails seems unrelated and appear on other PRs

@zingo zingo merged commit 4d6209b into pytorch:main Sep 10, 2025
450 of 463 checks passed
StrycekSimon pushed a commit to nxp-upstream/executorch that referenced this pull request Sep 23, 2025
This PR exposes the Model Explorer visualisation API, which provides the
foundation to enable us to load custom file types and plugins.
@jmahbs jmahbs deleted the expose-model-explorer-api branch September 30, 2025 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants