Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[OV JS]: Expose ov.saveModel() functionality #27148

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

[OV JS]: Expose ov.saveModel() functionality #27148

wants to merge 3 commits into from
+119 −1

Conversation

hub-bla
Copy link
Contributor

@hub-bla hub-bla commented Oct 20, 2024

Details:

  • Created init_function to initialize functions directly in the addon.
  • Extended type_validation with Napi::Boolean
  • Exposed ov.saveModel
  • To make proper tests like in Python API I guess I have to wait for Model.getOps() to be exposed.

Tickets:

@hub-bla hub-bla requested a review from a team as a code owner October 20, 2024 09:42
@hub-bla hub-bla marked this pull request as draft October 20, 2024 09:42
@github-actions github-actions bot added the category: JS API OpenVino JS API Bindings label Oct 20, 2024
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
category: JS API OpenVino JS API Bindings ExternalPR External contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hub-bla @sys-openvino-ci