Skip to content

Conversation

@pegasas
Copy link

@pegasas pegasas commented May 5, 2020

No description provided.

@pegasas pegasas force-pushed the luna.ai branch 2 times, most recently from 12367d4 to 0a37b0b Compare May 10, 2020 12:51

public Guid AADApplicationId { get; set; }

public string AADApplicationSecrets { get; set; }
Copy link
Member

Choose a reason for hiding this comment

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

Better to use SecureString.

}

[Key]
[System.Text.Json.Serialization.JsonIgnore]
Copy link
Member

@bowgong bowgong Jul 23, 2020

Choose a reason for hiding this comment

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

Remove unnecessary namespace.
Same for below two.

GetABatchInferenceOperation,
ListAllInferenceOperationsByUser,
DeployRealTimePredictionEndpoint,
GetADeployOperationByEndpointIdUser,
Copy link
Member

@bowgong bowgong Jul 23, 2020

Choose a reason for hiding this comment

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

Naming looks strange, maybe GetDeploy... or GetOneDeploy....?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants