vloux / ProteoRE

GNU General Public License v3.0
2 stars 5 forks source link

ClusterProfiler: Graphical output layout #98

Closed yvandenb closed 5 years ago

yvandenb commented 6 years ago

Graphical display: layout to be improved (overlapping text, frame with reduced size, see below...) - Flo will discuss with Sam...

image

combesf commented 6 years ago

To be done (reminder) : to specify the "showCategory" parameter for the dotplot function for the clusterProfiler graphics

combesf commented 6 years ago

Done for the "showCategory" argument (committed on GitHub). Caution : set to 10 (categories to be showed). Would be better to be settable ?

NguyenLien commented 6 years ago

Updated on dev instance.

yvandenb commented 6 years ago

Thx Lien => will test

Would be better to be settable ? Ideally yes, this means that we have to define a range of categories: from 10 to X ?? what would be the max value according to what you've experienced ? Default value set at 10 should be OK to me...

combesf commented 6 years ago

let's 1st try with 10, and see ...

yvandenb commented 6 years ago

OK thx both of you, let's keep it like this for the moment...I propose to discuss this together along with the others enhancements posted related to 'ClusterProfiler', namely:

yvandenb commented 6 years ago

Meanwhile Lien, please could you

  1. modify the .xml so that the input required by default option is "Input file containing your identifiers"
  2. stilll in the .xml, in the user doc section parag "input required" replace "You can copy/paste these identifiers or supply a tabular file (.csv, .tsv, .txt, .tab) where there are contained." by "two modes are allowed: either by supplying a tabular file (.csv, .tsv, .txt, .tab) including your IDs (identifiers) or by copy/pasting your IDs (separated by a space).
  3. once done deployed this version on dev and proteore.org...and let me know ;-) Thx
yvandenb commented 6 years ago

@Lien : did you deployed the updated version on proteore.org ?

NguyenLien commented 6 years ago

@yvandenb I've just updated the dev one. By the way, I'll do the same (default input option is from file) for all components.

yvandenb commented 6 years ago

:+1: Let me know once done...

NguyenLien commented 6 years ago

@yvandenb they are all updated now in dev and proteore.org.

yvandenb commented 6 years ago

As a reminder: graphical display and size-adjustment => layout to be improved (overlapping text, frame with reduced size, see below...) - Flo is on charge with Sam...

yvandenb commented 6 years ago

Flo regarde ce qu'elle peut ré-utiliser du code de Sam qui a ré-ecrit la fonction de display dot & bar-plot en utilisant ggplot2 et HighCharts

yvandenb commented 5 years ago

Display graphics à retester par Flo/Yves

yvandenb commented 5 years ago

Cela désormais OK sur cet aspect graphical layout - je ferme l'issue