TommyJones / textmineR

An aid for text mining in R, with a syntax that should be familiar to experienced R users. Provides a wrapper for several topic models that take similarly-formatted input and give similarly-formatted output. Has additional functionality for analyzing and diagnostics for topic models.
Other
106 stars 34 forks source link

suppress verbose in textmineR::CreateDtm() #28

Closed sunsiyu closed 7 years ago

sunsiyu commented 7 years ago

Hi,

When using the function CreateDtm() in textmineR, the progress status verbose cannot be suppressed regardless of suppressWarnings(), suppressMessages(), invisible(), or try(silent = T) etc.

For strictly non-verbose environment, this is very undesirable. Could this be somehow modified in a way that leave the option of verbose to the user?

stackoverflow question here

TommyJones commented 7 years ago

Thanks for bringing this to my attention. I will have to add a verbose argument since the underlying function producing the output is text2vec::create_dtm. Shouldn't be an issue. I will add that to the next minor release. It will be up on CRAN sometime in March.

TommyJones commented 7 years ago

Yikes. Upon testing that doesn't do it because the source is in text2vec::create_vocabulary which does not have a verbose argument. I will have to dig deeper to see what can be done.

dselivanov commented 7 years ago

itoken has verbose argument

3 мар. 2017 г. 6:51 ДП пользователь "Tommy Jones" notifications@github.com написал:

Yikes. Upon testing that doesn't do it because the source is in text2vec::create_vocabulary which does not have a verbose argument. I will have to dig deeper to see what can be done.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/TommyJones/textmineR/issues/28#issuecomment-283850600, or mute the thread https://github.com/notifications/unsubscribe-auth/AE4u3b3at51thIZGSse11lZZppK3PRF6ks5rh4AjgaJpZM4MOjag .

TommyJones commented 7 years ago

It worked! Thank you @dselivanov. I'll close this issue once the next version is released to CRAN

TommyJones commented 7 years ago

Closing. Latest version includes this feature.