machinelearning icon indicating copy to clipboard operation
machinelearning copied to clipboard

CodeGenerator throws a bad error when StablePackageVersion isn't set

Open eerhardt opened this issue 6 years ago • 0 comments

With the addition of https://github.com/dotnet/machinelearning/pull/4391, we added two new string properties to CodeGeneratorSettings:

  • StablePackageVersion
  • UnstablePackageVersion

These properties are required to be set, if you don't code gen fails. But even worse, it fails in a way that isn't immediately obvious to what you did wrong.

We should make these properties required when creating a CodeGeneratorSettings object. That way callers of CodeGen know they need to set it.

eerhardt avatar Oct 31 '19 19:10 eerhardt