Skip to content

Issues: InPreD/PRONTO

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.

Sign up for GitHub

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

11 Open 24 Closed
11 Open 24 Closed
Author
Filter by author
Loading
Label
Filter by label
Loading
Use alt + click/return to exclude labels
or ⇧ + click/return for logical OR
Projects
Filter by project
Loading
Milestones
Filter by milestone
Loading
Assignee
Filter by who’s assigned
Sort
Sort by
Newest Oldest Most commented Least commented Recently updated Least recently updated Best match
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀

Issues list

All the textbox6 related commands could go to a new function called something like set_layout_textbox_tumor_content. Similarly, commands related to other textboxes could be split to smaller functions with meaningful naming.
#47 opened Jun 25, 2024 by xiaoliz0
It would be nice to separate 'graphical' and 'functional' parts of the code to separate functions. I know this is quite some work, but I also believe it would be worth from the perspective of readability.
#46 opened Jun 25, 2024 by xiaoliz0
just a practical question, why do not impress sites get an empty string "" and impress sites Inclusion site. Would it be better to maybe clearly state that it is impress or not? But I am not sure how you handle this downstream, maybe it will change a lot.
#45 opened Jun 25, 2024 by xiaoliz0
I understand that these are all column names? Maybe they can be collected in an object?
#44 opened Jun 25, 2024 by xiaoliz0
Suggestion from Tina
#43 opened Jun 18, 2024 by xiaoliz0
Typing error for "target_cod_region"
#41 opened Jun 18, 2024 by xiaoliz0
Suggestion from Martin for file opening function
#40 opened Jun 18, 2024 by xiaoliz0
1
Extract the inclusion_site information from the code to config file.
#39 opened Jun 18, 2024 by xiaoliz0
1
Move "import" module from function to the top of script.
#38 opened Jun 18, 2024 by xiaoliz0
Tumor content number provided multiple times for one sample set
#37 opened May 3, 2024 by tinavisnovska
3
Considering switching over to python image with pip instead of the miniconda image
#27 opened Jan 31, 2024 by marrip
ProTip! Add no:assignee to see everything that’s not assigned.