-
Notifications
You must be signed in to change notification settings - Fork 237
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.
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
Check for mapping conflicts for Files and Directories in the actual PathMapper #4888
base: master
Are you sure you want to change the base?
Conversation
…athMapper This should fix #4864's problem by making the CWL `_2` name generation actually go on to `_3`, `_4`, etc. It should also make it apply to directories, by applying it before we actually do the file or directory specific logic, at the point where we generate the target path. It should also make it properly distinguish between re-mapping the same thing and mapping a different thing with the same name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for looking at this @adamnovak. Looks like this PR is moving code around from cwltool.process.stage_files
; Maybe it would be better to leave that out and adjust toilStageFiles
to use cwltool.process.relocateOutputs
? I'm happy to add docs and/or refactor that function to be more useful to toil-cwl-runner.
I'm not pulling any new code from I don't feel confident in my ability to refactor |
This should fix #4864's problem by making the CWL
_2
name generation actually go on to_3
,_4
, etc.It should also make it apply to directories, by applying it before we actually do the file or directory specific logic, at the point where we generate the target path.
It should also make it properly distinguish between re-mapping the same thing and mapping a different thing with the same name.
Changelog Entry
To be copied to the draft changelog by merger:
toil-cwl-runner
can now handle more than 2 files with the same name, or any number of directories with the same name.Reviewer Checklist
issues/XXXX-fix-the-thing
in the Toil repo, or from an external repo.camelCase
that want to be insnake_case
.docs/running/{cliOptions,cwl,wdl}.rst
Merger Checklist