re-export commonly-used functions from the matrix package

TommyJones / tidylda

Implements an algorithim for Latent Dirichlet Allocation using style conventions from the [tidyverse](https://style.tidyverse.org/) and [tidymodels](https://tidymodels.github.io/model-implementation-principles/index.html).

Other

41 stars 3 forks source link

re-export commonly-used functions from the matrix package #35

Closed TommyJones closed 4 years ago

TommyJones commented 4 years ago

at the least: t(), colSums(), and rowSums().

TommyJones commented 4 years ago

Not doing t() looks like that's covered in base. Either way, turned into a much bigger pain than expected for such a small thing... 4d2ba4ca1b8a90638aff5bd77964c8f80f93ce66

TommyJones commented 4 years ago

Undoing this with b010c6ddf74ea353519ac5d12ca057a67f43bdf0

Decided that this doesn't fit with the ideology of the package. Other packages are for working with DTMs etc.

Additionally, this created whining from R on loading package as rowSums etc. are also in base. Not sure why this doesn't happen with Matrix, but oh well. Per, above. Doesn't fit with tidylda's goals anyway.