Skip to content

Commit

Permalink
Put the --import-file back
Browse files Browse the repository at this point in the history
Skyline needs the path to locate the LC-MS files not in the work dir.
Removed by e960094
  • Loading branch information
fcyu committed Oct 11, 2024
1 parent 2a09902 commit efd169d
Showing 1 changed file with 2 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -255,11 +255,8 @@ private static void runSkyline(String skylinePath, Path wd, String skylineVersio
writer.write("--full-scan-isolation-scheme=\"Results only\" ");
}

if (dataType.contentEquals("DIA") && overridePeakBounds) {
// lc-ms file import is automatic when reading peptide list, but needs the DIA files imported separated for DIA SSL
for (String s : lcmsFiles) {
writer.write("--import-file=" + s + " ");
}
for (String s : lcmsFiles) {
writer.write("--import-file=" + s + " ");
}

// todo: waiting for a fix (or clarification) from Brendan. Skyline does not allow this unless fasta has been imported
Expand Down

0 comments on commit efd169d

Please sign in to comment.