Add use_conda_env param and Fix mysql command bugs#41
Add use_conda_env param and Fix mysql command bugs#41rainoffallingstar wants to merge 2 commits intoPoisonAlien:masterfrom
Conversation
…and Update mysql commands to mariadb
R/trackplot.R
Outdated
There was a problem hiding this comment.
Hi,
what is the use_conda argument in this context? I guess it should be the environment name, but the main argument seems to be TRUE or FALSE.
There was a problem hiding this comment.
Hi!
Indeed, the use_conda argument should be designed to accept the name of the conda environment. It is not appropriate to simply name it use_conda, as this could potentially mislead users. I suggest renaming it to use_conda_env to avoid confusion and to clearly communicate the purpose of the argument.
I believe that bwtool is much easier to install via conda. I think this improvement would make this package more robust.
R/trackplot.R
Outdated
There was a problem hiding this comment.
I am using MariaDB on Arch Linux, and the mysql package is being deprecated. It is expected to be removed in a future release, and it's suggested to use mariadb instead. By the way, MariaDB is a fork of MySQL , which is originated from the same source but has better performents and easy to install.
There was a problem hiding this comment.
Aye, by the way, I've retained the original mysql command in a separate commit.
There was a problem hiding this comment.
Yup. I will review them soon.
R/trackplot.R
Outdated
There was a problem hiding this comment.
Again, use_conda shouldn't be the environment name instead of T/F.
…meter to use_conda_env for clarity.
Uh oh!
There was an error while loading. Please reload this page.