trendscenter / coinstac-regression-vbm

Coinstac Code for Decentralized Regression with Normal Equation on MCIC Data
0 stars 0 forks source link

Remove rows with null values in any columns #52

Closed hvgazula closed 1 year ago

hvgazula commented 1 year ago

Please provide a sample inputspec.json with null values in each of "string", "number" and "boolean" columns. I can use this to modify the code as needed.

pixelsaurus commented 1 year ago

You should be able to modify any value here for testing purposes: https://github.com/trendscenter/coinstac-regression-vbm/blob/master/test/inputspec.json#L6

hvgazula commented 1 year ago

@spanta28 I just spoke to @rssk and this filtering of rows will null values (with feedback to the user) in any columns is already being done on the UI. So, we don't see a need for handling this in the computation code again. Please close this issue if you agree.