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.

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

[Good First Issue]: Expose ov.saveModel() functionality #27092

Open
almilosz opened this issue Oct 16, 2024 · 2 comments
Open

[Good First Issue]: Expose ov.saveModel() functionality #27092

almilosz opened this issue Oct 16, 2024 · 2 comments
Assignees
Labels
category: JS API OpenVino JS API Bindings good first issue Good for newcomers no_stale Do not mark as stale

Comments

@almilosz
Copy link
Contributor

almilosz commented Oct 16, 2024

Context

OpenVINO works in Node.js environment!
We are looking for new contributors who can help with enabling C++ API methods
in JavaScript side.

First of all read Node.js API Contribution Guide.

Task Details

Expose ov::save_model() method.
From JavaScript side it will be ov.saveModel() method.
This method takes 3 arguments: model to save, path to output model file and bool value whether to compress floating point weights to FP16.

Useful links

What needs to be done?

  • Propose the place where the ov.saveModel() method should be implemented. It might be
    ./src/bindings/js/node/src/addon.cpp. Implement the method.
  • Test manually if the functionality is working with different models. Describe in the PR what models were tested in which formats.
  • Update TypeScript definitions in ./src/bindings/js/node/lib/addon.ts
  • Create unit test for added functionality using Node.js Test Runner. The logic might be similar to the Python tests for this functionality. Link to Python API tests

How to take this issue

To take this issue leave text: .take as the comment in this issue.
It will assign this issue to you automatically. Please, make sure that the issue has the status Contributors Needed and doesn't have another user in Assingnees.

Example Pull Requests

Resources

Contact points

@almilosz @vishniakov-nikolai

Please, mention us in this issue discussion if you have any questions.

Ticket

No response

@almilosz almilosz added good first issue Good for newcomers no_stale Do not mark as stale category: JS API OpenVino JS API Bindings labels Oct 16, 2024
@hub-bla
Copy link
Contributor

hub-bla commented Oct 19, 2024

.take

Copy link
Contributor

Thank you for looking into this issue! Please let us know if you have any questions or require any help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: JS API OpenVino JS API Bindings good first issue Good for newcomers no_stale Do not mark as stale
Projects
Status: Contributors Needed
Development

No branches or pull requests

2 participants