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
Test serialization of hp::DoFHandler #5496
Test serialization of hp::DoFHandler #5496
Conversation
All sorts of errors - it may simply be the time of day for you to actually sleep ;-) /run-tests |
include/deal.II/hp/dof_handler.h
Outdated
// some versions of gcc have trouble with loading vectors of | ||
// std::unique_ptr objects because std::unique_ptr does not | ||
// have a copy constructor. do it one level at a time | ||
unsigned int n_levels = levels.size(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const unsigned int n_levels
?
include/deal.II/hp/dof_handler.h
Outdated
// have a copy constructor. do it one level at a time | ||
unsigned int n_levels = levels.size(); | ||
ar &n_levels; | ||
for (unsigned int i = 0; i < levels.size(); ++i) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
< n_levels
include/deal.II/hp/dof_handler.h
Outdated
ar &has_children; | ||
|
||
// the same issue as above | ||
unsigned int n_has_children = has_children.size(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unsigned int n_has_children
?
include/deal.II/hp/dof_handler.h
Outdated
// the same issue as above | ||
unsigned int n_has_children = has_children.size(); | ||
ar &n_has_children; | ||
for (unsigned int i = 0; i < has_children.size(); ++i) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
< n_has_children
c358f5a
to
02b32ed
Compare
Hey, it was just the indentation and the randomly failing timeouts on the testing machine. But fair enough, sleep is always helpful. 😃 |
Again, only header timeouts:
|
Is there anything I should address? |
02b32ed
to
1dcbf8a
Compare
It turned out that we had the same issue with |
Fixes #4684.