0xDeCA10B icon indicating copy to clipboard operation
0xDeCA10B copied to clipboard

[simulation] Use normalization like in the demo.

Open juharris opened this issue 2 years ago • 2 comments

In the demo, normalization is done before updating the dense perceptron and dense nearest centroid classifiers, but the simulation doesn't use normalization.

Normalization is used to avoid submissions with large values that would corrupt the model. Using normalized vectors might be bad for these models but it seems like a necessary guard to have on-chain.

juharris avatar Nov 30 '21 20:11 juharris

Can you describe more about this Issue?

iamhardikat11 avatar Mar 07 '22 05:03 iamhardikat11

We enforce that data in normalized before adding in DensePerceptron.sol and some other classes in the demo folder but we don't normalize data before processing it in the simulation.

juharris avatar Mar 07 '22 14:03 juharris