Skip to content

Vignette added and clean up - #76

Merged
KarinSchork merged 51 commits into
develfrom
devel_Julika
Aug 7, 2026
Merged

KarinSchork merged 51 commits into
develfrom
devel_Julika

Conversation

@biwenzel

Copy link
Copy Markdown
Collaborator

added vignette
clean up

Comment thread R/01_peptideInput.R Outdated
Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd
@@ -92,137 +128,202 @@ citation("bppg")

# Quick start to using `bppg`

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For quick start, wouldnt it make sense to use the wrapper functions to generate the graphs quite easily? And then in a next chapter maybe go step by step?

Also I think that visualizing one of the example graphs here would be cool. Then in the end we could show the result table for that graph for example.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

do you mean the result from calcCi or the last one?

Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd Outdated
Comment thread vignettes/bppg.Rmd Outdated

@KarinSchork KarinSchork left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have some suggestions for the main vignette, lets talk tomorrow :)

@KarinSchork
KarinSchork merged commit c9b5ad0 into devel Aug 7, 2026
6 checks 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.

2 participants