Volcano_Plot #440
Merged
Volcano_Plot #440
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
release 0.1.3
Merge into Main
"New Mini Release"
* changed to show debug text again; to let user know what is going on; addresses #421 * Make things clearer in layout, incl. button design, placement, tip highlights and promoting more parallelism (#419, #413, #414) * changes and clean up to the docs to easen the retrieval of information * indicate that batch collection is optional, adding explicit help file - addresses #418 * making top links more obvious; attention no order change #415 as suggested * make cOmicsArt print prettier * Crash fix sig ana (#420) * add hiding of tabs to avoid usage of results; fixes #417 * fixed to work with native SumExp next to cOmicsArt created #404 (#425)
…hing aside from DESeq2 and DESeq2 not working on multi tabs
…ed to other display functionalities
|
Leave Options for multiple plots |
…nge). Volcanos now shown on top of each other with legend toggable
LeaSeep
approved these changes
Jan 30, 2025
There was a problem hiding this comment.
Major (maybe even a new issue? ):
- The app crashes if Testdata > Deseq> Sig Ana > select 1 condition DO -> adding a new condition DO => Crash
[1] "Start the Significance Analysis"
[1] "Updating data..."
Warning: Error in session$userData[[ns(paste0(name, button))]]$destroy: attempt to apply non-function
Minor:
I think I would put the options 'adj. p value threshold', 'LogFC thresholds', 'Select the anno, 'Show legen corrected / uncorrected' to the right of the plots and decrease font size of legend
Will fix 👍🏼
Will leave that as the functionality rn is not well designed (recomputation every time) and opened another issue regarding that. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The following was fixed/changed:
I have specifically NOT checked and/or worked on the getRCode and i dont think it should be a concern of this PR.
Things to further consider to further improve the volcano plots: