-
Notifications
You must be signed in to change notification settings - Fork 137
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
ParFORM redefine bug with more than one expressions #525
Comments
There are 3 basic strategies to handle
The current implementation of ParFORM tries to adopt option 3 for efficiency while at the same time trying to resemble TFORM code to share some variables and code (see also 23285ec). As a result(?), it gathers redefined variables after each expression is processed, and broadcasts them at the end of the module. But, with more than one expression, the information gathered after finishing an expression that indicates which variables are redefined (stored in A natural solution would be to move the broadcast to the end of processing each expression, rather than the end of the module. It may be an option to gather redefined variables at the end of the module (and then broadcast them), but then the meaning of |
Redefined variables must be broadcast after processing each expression. Otherwise, the information collected in AC.inputnumbers is wiped out at the beginning of processing the next expression. This fixes vermaseren#525.
Minimized example from the ParFORM issue in #524.
mpirun -np 3 parform
freezes with the following input:The text was updated successfully, but these errors were encountered: