8000 Adding link to cheatsheet in 19.1.3.2 & removed some verbose messages by ElySeraidarian · Pull Request #457 · microbiome/OMA · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Adding link to cheatsheet in 19.1.3.2 & removed some verbose messages #457

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
May 24, 2024

Conversation

ElySeraidarian
Copy link
Contributor

Fixing additionnal files that were added in last PR (#455)

@ElySeraidarian ElySeraidarian mentioned this pull request Apr 25, 2024
@ElySeraidarian ElySeraidarian linked an issue Apr 25, 2024 that may be closed by this pull request
@@ -294,7 +294,7 @@ var_explained <- e/sum(e[e>0])*100
This section has a cheatsheet for translating common functions in phyloseq
to TreeSE/mia with example code.

```{r}
```{r,message=FALSE, warning=FALSE, eval=FALSE}
Copy link
Member

Choose a reason for hiding this comment

8000 The reason will be displayed to describe this comment to others. Learn more.

add space in "r, message" here and elsewhere

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In fact, on the same go you could move the messages, warning and eval into quarto header in chunks of this PR

Like
#| message: false
#| warning: false
#| eval: false

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be better now (couldn't be able to do the way you said but pretty much the same I believe)

@TuomasBorman TuomasBorman merged commit 7b31332 into microbiome:devel May 24, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cheatsheet
3 participants
0