- changed status to open
- removed comment
Multipole HDF5 file truncation with multiple variables
Previously a "first_time" variable was used to determine if truncation should happen. When decomposing multiple variables, this logic is incorrect. We now store the first_time information per output file.
I have tested the patch.
Keyword: Multipole
Comments (7)
-
reporter -
- removed comment
The patch looks good.
I would use set<...> instead of map<...,bool>. I would also move changing the map (set) into the then branch of the if statement.
-
reporter - removed comment
I agree with you about the changes. However I don't have time right now to make and test them. Since the patch works, and the suggested changes are only aesthetic, I would like to apply the patch to fix the bug right now, and the aesthetic changes can be fixed later. We can leave the ticket open so that we don't forget. Do you agree?
-
- removed comment
Agreed.
-
reporter - removed comment
The patch has been committed (r76).
-
- changed status to resolved
- removed comment
Closing the ticket since the original problem has been solved.
-
- changed status to closed
- edited description
- Log in to comment