Ino de Bruijn

Results 89 comments of Ino de Bruijn

Wasn't able to push to this fork somehow, so opened a new PR: https://github.com/cBioPortal/datahub/pull/2073

@byoo thanks for reaching out! @zainasir on our team will take a look at this Current plan is to create a cronjob in our k8s cluster that runs `mysqldump` and...

@haynescd let's see if this is sovled in the clickhouse implementation

@TJMKuijpers Good catch! Thanks for trying to come up with a solution to this I think the "Table" name here is unintuitive. How about we call it "Summary Statistics"? In...

@SURAJ-SHARMA27 this looks awesome - thank you! For the sorting, can we also sort by "# samples"? The value of the Y-axis basically This would be similar to e.g. how...

Amazing - nice work @SURAJ-SHARMA27 ! > Should the 'Sort By' menu be included in the 'Vertical Axis' section since it's using the values from vertical axis. @jjgao Good question...

@SURAJ-SHARMA27 Thanks for the fixes! I noticed the "Sort by" option is currently missing [here](https://deploy-preview-4960--cbioportalfrontend.netlify.app/study/plots?id=msk_impact_2017&plots_horz_selection=%7B%22dataType%22%3A%22clinical_attribute%22%2C%22selectedDataSourceOption%22%3A%22CANCER_TYPE%22%7D&plots_vert_selection=%7B%22dataType%22%3A%22clinical_attribute%22%2C%22selectedDataSourceOption%22%3A%22SPECIMEN_TYPE%22%7D&plots_coloring_selection=%7B%7D), do you see that? ![image](https://github.com/user-attachments/assets/8eaef471-5031-447e-b11d-c4e3db3a7272)

@SURAJ-SHARMA27 Just discussed this during the community call, people liked it! Some more feedback from the group: - When switching to percentage, the bars should sort by the percentage value...

> i) This means that when the number of samples is selected from the dropdown, the data should be sorted by the percentage value. @SURAJ-SHARMA27 correct > ii) Sorting will...

@olzhasmukayev Nice work! Can we enable it for Metastatic Sites as well?