Skip to content
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

import numpy before dlopen to prevent segfault #928

Merged
merged 1 commit into from Mar 5, 2017

Conversation

albanD
Copy link
Collaborator

@albanD albanD commented Mar 5, 2017

Please see this tensorflow issue for the reason why this fixes #786

@@ -31,6 +31,10 @@
# automatically filled by the dynamic loader.
import os as _dl_flags

# numpy *must* be imported before the call to setdlopenflags() or there is
# risk that later c modules will segfault when importing numpy
import numpy as np

This comment was marked as off-topic.

This comment was marked as off-topic.

@soumith soumith merged commit ac9245a into pytorch:master Mar 5, 2017
@albanD albanD deleted the segfault_scipy branch March 5, 2017 20:09
jjsjann123 pushed a commit to jjsjann123/pytorch that referenced this pull request Aug 5, 2021
…h#928)

Outdated due to 
- added invariance in computeAt csarofeen#838
- the change to barrier sync allowing block broadcast/reduce to be placed in conditional code
- persistent buffers being considered on inputs
jjsjann123 pushed a commit to jjsjann123/pytorch that referenced this pull request Aug 5, 2021
Undoes some of the changes of pytorch#928 as layer norm half was failing. This just doesn't run computeWithOutputs on inputs that aren't inputs to the reduction.
zhuhong61 pushed a commit to zhuhong61/pytorch that referenced this pull request Jun 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Importing scipy.misc after torch causes segfault
3 participants