Skip to content

Add descriptions GUI#61

Merged
ammarcsj merged 35 commits into
mainfrom
add_descriptions_GUI
Feb 27, 2025
Merged

Add descriptions GUI#61
ammarcsj merged 35 commits into
mainfrom
add_descriptions_GUI

Conversation

@ammarcsj
Copy link
Copy Markdown
Member

@ammarcsj ammarcsj commented Feb 23, 2025

Add instructions and test data download options to GUI

@ammarcsj ammarcsj changed the base branch from main to accumulate_nb_wins February 23, 2025 21:43
@ammarcsj ammarcsj marked this pull request as ready for review February 24, 2025 08:02
@ammarcsj ammarcsj requested a review from mschwoer February 24, 2025 08:02
Comment thread alphaquant/ui/dashboard_parts_run_pipeline.py
Comment thread alphaquant/ui/gui_textfields.py Outdated
Comment thread alphaquant/ui/gui_textfields.py Outdated
####

The data needs to be exported in the **normal long** format as .tsv or .csv file.
[Download example data](https://datashare.biochem.mpg.de/s/m1qR1hbz7lOyIzn/download)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this the shared data share?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this is the alphaquant data share, where I also have the paper data

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should we move at least the example data to the shared one?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

👍

Comment thread alphaquant/ui/gui_textfields.py Outdated
Comment thread alphaquant/__init__.py Outdated
Comment thread alphaquant/__init__.py
Base automatically changed from accumulate_nb_wins to main February 24, 2025 15:47
Copy link
Copy Markdown

@anna-charlotte anna-charlotte left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@mschwoer mschwoer left a comment

Choose a reason for hiding this comment

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

LGTM

width=700,
sizing_mode='fixed',
description='Enter the full path to your input file from MaxQuant, Spectronaut, or DIA-NN'
description=gui_textfields.Descriptions.tooltips['file_input']
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

good idea!

**DIA-NN:**
Provide the path to the DIANN "report.tsv" output table. The samplemap.tsv file must map the Run column.

**AlphaDIA:**
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

shall we put AlphaDIA first?

mkdir -p ${CONTENTS_FOLDER}/Frameworks/alphamap/data/

####
####Download all AlphaMap FASTA and CSV files from GitHub, which are needed for the further analyses. There is a lot of error checking to ensure that the files get actually added during the build
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

for windows, this is not required?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

windows does not have the connection issues, so one can download the required files for alphamap. However, it would be nice to align this, I opened a separate PR for this
#71

@ammarcsj ammarcsj merged commit 5ab5dc2 into main Feb 27, 2025
@ammarcsj ammarcsj deleted the add_descriptions_GUI branch February 27, 2025 16:37
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.

3 participants