Skip to content

Conversation

@danielru
Copy link
Member

Minor changes and fixed parameters. Now runs PFASST with 16 processors - alas, ultimately the last process needs 16 iterations to converge.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you need this fixed at 16? The thing is that for FT we need to keep this free, otherwise we cannot estimate the overhead..

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, you can set this to whatever value you want. The only problem is that with too few iterations, PFASST can become unstable over the course of the simulation.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, so PFASST will reasonably converge with the residual criterion?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general, yes. You might have to tweak the tolerance a bit, PFASST takes quite long to reach very fine residuals. I tried to go for something between 1e-5 and 1e-6. The residual after the first iteration is already 1e-4, so larger values do not make much sense.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What would be a reasonable tolerance to get approx. 10-20 iterations per block, then? Does this change during the simulation or is it more or less constant?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try 1e-6 or 1e-7... for 16 processors, this might already give you quite a lot of iterations. It seems not to change too much over the simulation, but I did not take a systematic look at it.

pancetta added a commit that referenced this pull request Oct 15, 2015
@pancetta pancetta merged commit b9690ac into Parallel-in-Time:resilience_hard_faults Oct 15, 2015
@danielru danielru deleted the resilience/minor_tweaks branch October 15, 2015 14:10
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.

2 participants