Skip to content

Fix for issue #31#32

Open
sandroos wants to merge 1 commit into
masterfrom
corsair_restart_fix
Open

Fix for issue #31#32
sandroos wants to merge 1 commit into
masterfrom
corsair_restart_fix

Conversation

@sandroos

Copy link
Copy Markdown
Contributor

Corsair restarts were failing because multi IO units were returning
incorrect byte size for particle arrays.

Corsair restarts were failing because multi IO units were returning
incorrect byte size for particle arrays.
@rjarvinen

Copy link
Copy Markdown
Member

I was able to run and restart example_particle corsair case with this patch.

@sandroos

sandroos commented Feb 5, 2018

Copy link
Copy Markdown
Contributor Author

Great! So if you're encountering segfaults in restarted distributed Hybrid runs, chances are that there is a bug in hybrid code.

@rjarvinen

Copy link
Copy Markdown
Member

I will explore the issue further once we have access to Voima again. But so far the patch looks fine to me!

@sandroos

sandroos commented Feb 7, 2018

Copy link
Copy Markdown
Contributor Author

I'm waiting for Vlasiator people to verify that the branch is good.

@ykempf

ykempf commented Apr 4, 2018

Copy link
Copy Markdown
Contributor

Oh, apparently I commented only on Flowdock. This failed for us in Vlasiator.

@sandroos

sandroos commented Apr 4, 2018

Copy link
Copy Markdown
Contributor Author

Hi, I saw the comment, haven't fixed it yet.

@ykempf

ykempf commented Apr 5, 2018

Copy link
Copy Markdown
Contributor

No problem, I was just making sure. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants