Fix CategoricalHashTransform to handle OutputKind "Key"

[ganik]

fixes #1939

[TomFinley]

Thanks @ganik! That was a pretty nasty bug.

[Ivanidzo4ka]

Can we have test?
I'm actually curios how I miss that case during testing....
Oh, I remember now.
From API point of view, that Key Output kind doesn't make much sense, you just apply hash transform, and you do nothing else, after that.
So this is why I remove that option from static extensions, but it remain in arguments class and entrypoints.

Still, can you add few more lines in already existing test to run this case as well?

[ganik]

Very good suggestion, once I added test I see that GetOutputSchema() needs to be changed as well otherwise it throws. Same issue exists for OneHotEncoding. I spent couple hours but couldnt figure out how to fix GetOutoutSchema. @ivmatan Could you pls take over this PR since I am OOF till end of Jan. Would be great if it makes into ML.NET 0.9 release

---

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

[Ivanidzo4ka]

closed in #2017