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

Sigpy.mri.poission with core dumped #106

Open
SZUHvern opened this issue Feb 10, 2022 · 3 comments
Open

Sigpy.mri.poission with core dumped #106

SZUHvern opened this issue Feb 10, 2022 · 3 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@SZUHvern
Copy link

When using 'Sigpy.mri.poission' in pytorch's 'Dataloader' to generate random mask(iterative generation), it is easy to occur 'Aborted (core dumped)' in any step and shut down my process.
I want to know is there any way to solve this problem.
Thanks!

@SZUHvern SZUHvern added the bug Something isn't working label Feb 10, 2022
@sidward
Copy link
Collaborator

sidward commented Feb 11, 2022

Hi,

Thanks for the report. Could you please post code to reproduce the bug?

@SZUHvern
Copy link
Author

Yes, I just do an simply reimplement, like this:

for i in range(1000000):
sigpy.mri.poission((320,320),accel=8, seed=None, tol=0.3)
it will running slower and slower and randomly stop at a certain step.

When this program is called into torch.dataloader, an Aborted warning will appear, like this:
class test(Dataset):
def init(self):
super(test, self).init()
def len(self):
return len(self.examples)
def getitem(self, item):
mask_sub = sigpy.mri.poisson((320, 320), accel=8, dtype=np.float32, seed=None, tol=0.1)
mask_sub = torch.from_numpy(mask_sub).float()
return mask_sub

I'm thinking if there is some process that keeps getting called as it iterates until it fills the memory.
Is there any way like sigpy.close() to fix this problem?

@sidward
Copy link
Collaborator

sidward commented Feb 16, 2022

Ok, thanks for the report. I will look at it when I have time. My gut feeling is that it is related to #102, so if you can try that change and let me know, that would be appreciated.

@sidward sidward added the help wanted Extra attention is needed label May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants