-
Notifications
You must be signed in to change notification settings - Fork 24
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
newer test class for passive elephant tests breaks parallelism in dask, in the case of brian2 backend. #230
Comments
This is only the case when the backend is brian2, and the model is adaptive exponential. I think the reason is the passive tests try to store the model/simulator as a class attribute, and the brian2 module is assigned as a class attribute, but it is not picklable. |
Which "passive test class" are you talking about? Potential solution (though maybe there is something that should be done in the module itself): All SciUnit objects have an attribute Don't worry about this right now. |
Yes, I found a work around. Brian2 adexp is currently the slowest backend however, so it will hinder long term performance |
This was fixed. I mistakenly stored a brian2 model using |
The new passive test class that replaces
generate_prediction
withget_result
, andgenerate_features
cannot be parallel mapped with dask, as it causes an error:"cannot pickle weak reference".
The passive test class cannot be pickled using the pickle class either, for the same reason, I believe.
The text was updated successfully, but these errors were encountered: