search

Nesvilab / FragPipe

A cross-platform Graphical User Interface (GUI) for running MSFragger and Philosopher - powered pipeline for comprehensive analysis of shotgun proteomics data
http://fragpipe.nesvilab.org
Other
175 stars 37 forks source link

possible fix for #1652 #1653

Closed guoci closed 18 hours ago

guoci commented 2 days ago

possible fix for #1652

fcyu commented 2 days ago

Thanks @guoci , but it doesn't work

Animation

Best,

Fengchao

guoci commented 2 days ago

Pasting the path doesn't work, but using mouse navigation does. I will check if that can be fixed.

fcyu commented 2 days ago

That's correct. But this is what Matt wants: type or paste the path, and then select the files.

Best,

Fengchao

guoci commented 1 day ago

This new commit should allow users to type or paste paths.

fcyu commented 1 day ago

Thanks @guoci , it cause some errors when clicking "this PC" and "Desktop".

Animation

Best,

Fengchao

fcyu commented 1 day ago

Thanks @guoci , but the new commit broke the file chooser in the other buttons.

As you can see in the following gif, it can't go into .d because of https://github.com/Nesvilab/FragPipe/pull/1653/files#diff-23d6af456b2afdf3296b61117f8c4824b8c497351f63e93db00e6d4cdf9b8058R114. And after double-clicking, it load *.d for the database file, which it should not because the extension doesn't follow the FASTA file rules.

I guess you need to override isTraversable only when the file chooser is for the LC-MS files.

Animation

Best,

Fengchao