Skip to content

OnnxTransform: Make Create() method public, for use via Nuget package #1051

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
Closed
jignparm opened this issue Sep 26, 2018 · 1 comment
Closed

OnnxTransform: Make Create() method public, for use via Nuget package #1051

jignparm opened this issue Sep 26, 2018 · 1 comment
Assignees
@shmoradims
Labels
API Issues pertaining the friendly API

Comments

@jignparm
Copy link
Contributor

We need this method to be public, instead of private:
IDataView trans = OnnxTransform.Create(env, pixels, model_location, "Output", "Input" });
@shauheen shauheen added the API Issues pertaining the friendly API label Sep 26, 2018
@shauheen shauheen assigned shmoradims and unassigned jignparm Sep 26, 2018
shmoradims pushed a commit to shmoradims/machinelearning that referenced this issue Sep 27, 2018
(Fix issue dotnet#1051) Added public create methods
4a537e4
@shmoradims shmoradims mentioned this issue Sep 27, 2018
Merged
shmoradims pushed a commit that referenced this issue Sep 28, 2018
OnnxTransform: Fix 3 bugbash bugs (#1080)
8592d96 
* (Fix issue #1050) Renamed namespace to Microsoft.ML.Transforms to conform with TF transform.

* (Fix issue #1051) Added public create methods

* (Fixed issue 1053) IDV names don't have to be the same as OnnxModel node names.

* Added license header
@shmoradims
Copy link

Resolved by PR #1080

@ghost ghost locked as resolved and limited conversation to collaborators Mar 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@shmoradims shmoradims

Labels
API Issues pertaining the friendly API
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@shauheen @jignparm @shmoradims