Skip to content

Conversation

@yaeldekel
Copy link

Fixes #794 .
In order to use the pipeline API, users will be required to call the Initialize() method in order to load the assemblies that contain the relevant entry points.

@yaeldekel yaeldekel requested review from Zruty0 and abgoswam August 31, 2018 21:27
Copy link
Contributor

@Zruty0 Zruty0 left a comment

Choose a reason for hiding this comment

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

:shipit:

namespace Microsoft.ML.Transforms.TensorFlow
{
internal partial class TensorFlowUtils
public static class TensorFlowUtils
Copy link
Member

@abgoswam abgoswam Aug 31, 2018

Choose a reason for hiding this comment

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

TensorFlowUtils [](start = 24, length = 15)

so in the unit test TensorFlowTransformCifarLearningPipelineTest I need to call TensorFlowUtils.Initialize() before creating the pipeline ? #Resolved

Copy link
Author

Choose a reason for hiding this comment

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

I think it would be useful, since the unit test serves as an example of how to use the transform in a learning pipeline.


In reply to: 214484316 [](ancestors = 214484316)

Copy link
Member

@abgoswam abgoswam left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link
Contributor

@TomFinley TomFinley left a comment

Choose a reason for hiding this comment

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

A proud moment for us all. 😃

@yaeldekel yaeldekel force-pushed the loadimageanalytics branch 3 times, most recently from 7f0b75b to 02f7d3c Compare September 4, 2018 17:08
@yaeldekel yaeldekel merged commit 1e26350 into dotnet:master Sep 4, 2018
@yaeldekel yaeldekel deleted the loadimageanalytics branch September 4, 2018 19:28
@ghost ghost locked as resolved and limited conversation to collaborators Mar 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants