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

GetModelNodes not compatible with multi-file saved models #2102

Closed
JakeRadMSFT opened this Issue Jan 10, 2019 · 2 comments

Comments

2 participants
@JakeRadMSFT
Copy link

JakeRadMSFT commented Jan 10, 2019

System information

  • OS version/distro: Windows
  • .NET Version (eg., dotnet --info): ML.Net 0.8.0

Issue

  • What did you do?
    Tried to call GetModelNodes on a Tensor Flow model that was successfully able to be loaded using ScoreTensorFlowModel.
  • What happened?
    Received Exception: Access Denied Error
  • What did you expect?
    Successfully load model and enumerate model nodes.

Source code / logs

I believe it's because it's trying to load my folder path as a file:

var bytes = File.ReadAllBytes(modelFile);

@JakeRadMSFT

This comment has been minimized.

Copy link

JakeRadMSFT commented Jan 10, 2019

@Ivanidzo4ka

This comment has been minimized.

Copy link
Member

Ivanidzo4ka commented Jan 10, 2019

So I think it tensor flow terminology it's called unfrozen model, and you are absolutely right, GetModelNodes was written before we start support it, and it hasn't been updated.

@Ivanidzo4ka Ivanidzo4ka added this to To Do in DNN and ONNX scoring in ML.NET via automation Jan 10, 2019

@Ivanidzo4ka Ivanidzo4ka self-assigned this Jan 10, 2019

DNN and ONNX scoring in ML.NET automation moved this from To Do to Done Jan 11, 2019

@Ivanidzo4ka Ivanidzo4ka added this to Done in v0.10 via automation Jan 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment