quinngroup / dr1dl-pyspark

Dictionary Learning in PySpark
Apache License 2.0
1 stars 1 forks source link

Added ceil function so that R can be used to index elements even if it is not an integer. #70

Closed arunbalachandran closed 6 years ago

arunbalachandran commented 6 years ago

I noticed that the R value used in the op_selectTopR function was passed R values which were percentages and sometimes if you have a small dictionary, the resulting R value turns out to be really small and this necessitates the addition of the 'ceil' function so that you can round of the fractional numbers to the nearest integers.

magsol commented 6 years ago

Perfect, great fix. Thanks!