Changes between Initial Version and Version 1 of Ticket #5485, comment 23


Ignore:
Timestamp:
2019-07-04T23:03:03Z (5 years ago)
Author:
Francesco Casella

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #5485, comment 23

    initial v1  
    1 I ran the same experiment with {{{s="dassl"}}} but I got the same leak. I'm not sure how @sjoelund.se got the output shown in comment:3, but it really seems we need to fix the C runtime first. Maybe the Kinsol leak has been fixed by the maker of Sundials in the meantime?
     1I ran the same experiment with {{{s="dassl"}}} but I got the same leak.
     2
     3I understand @sjoelund.se tried to "fix" the leak in [https://github.com/OpenModelica/OpenModelica/pull/193 PR 193], in a way that led to severe performance degradation, see comment:4, comment.5. I don't think this is acceptable.
     4
     5I would suggest that we
     6- fix the remaining issue in the runtime
     7- undo PR 193
     8- clearly identify the issue and contact the developers of Sundials to sort that out in the Kinsol source code