-
Notifications
You must be signed in to change notification settings - Fork 32
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
Workflows failing for DataSets with multiple input files #32
Comments
@DamienIrving - would you be able to check the failing workflow into cwsl-workflows repo? Maybe we could open a development branch - it would be good to see your exact workflow to repeat the bug. |
@DamienIrving - actually, don't worry. I can replicate this. Sorry about this - I really thought this was covered by a unit test. I'm on it - it is a recent regression from something I did. |
@captainceramic Ok. If you change your mind let me know - I'm happy to push my workflows to cwsl-workflows if need be. |
This included - Checking for valid combinations of attributes instead of hash values - Using the ArgumentCreator as an iterator (__iter__) - Removing confusing logger debugging statements - Continuing iteration instead of returning None when no matching inputs are found for an output file.
@DamienIrving - I've pushed some code that should fix the simplest version of this bug (the one where models and institutes are getting mixed up). Can you have a go and see if it fixes it in your workflow? The second, more complex issue is around "mapping" constraints and is issue #14 - Would you be able to post your desired workflow (the one with the arithmetic comparisons of two datasets) under that issue? I think it could be a good test case for it. If you can get the multiple inputs going I'll close this issue. |
As noted in #35, I think I just found a bug with the handling of multiple input files in ensemble operations (i.e. where you want to pass all the files at once for constraints that your are overwriting). The bug happens when I run a workflow where |
I can reproduce this - looking at it now. |
I think this is a combination of two things -
|
See #30
@DamienIrving has a workflow that is failing whenever it is run with multiple input models.
The text was updated successfully, but these errors were encountered: