egenn / rtemis

Advanced Machine Learning and Visualization
https://rtemis.org
GNU General Public License v3.0
140 stars 19 forks source link

Looks like Decom Broke #44

Closed abresler closed 1 year ago

abresler commented 1 year ago

rtemis::decom(x = mtcars, decom = "pca")

Error in if (!is.na(caller)) { : argument is of length zero)
abresler commented 1 year ago

Looks like one of the very recent commits may have caused it

egenn commented 1 year ago

Thanks, Alex! 0cee60df12d7a9d5aefcc739058573045869b09f fixes that. It was an update to the internal function used to write the messages with date-time and caller info. I forgot decom() existed, to be honest. Because it uses do.call(), it messes up the call stack.