Issue1131

Title backport small changes from Scorpion
Priority wish Status resolved
Superseder Nosy List davidspeck, florian, jendrik, malte
Assigned To jendrik Keywords
Optional summary
PR: https://github.com/aibasel/downward/pull/207

Created on 2024-01-09.15:04:32 by jendrik, last changed by malte.

Summary
PR: https://github.com/aibasel/downward/pull/207
Messages
msg11717 (view) Author: malte Date: 2024-12-13.11:46:20
There was an unresolved aspect from the pull request (hidden in an "outdated conversation") that we wanted to discuss later but didn't get to. This was about not allocating memory for domain size 1. David will open a new issue on this.
msg11589 (view) Author: malte Date: 2024-04-04.11:15:11
I found this now and have some late comments (sorry!) that I posted on github.
msg11515 (view) Author: jendrik Date: 2024-01-10.13:20:49
Thanks for the review, Florian! I took care of your comments and merged.
msg11514 (view) Author: florian Date: 2024-01-09.15:56:20
I had a look. As I commented on Github, I think the change in the output should be documented in the commit message as it could break existing parsers. For the change regarding IntPacker, I suspect that this is the wrong place to fix but I'm not sure if I understood the issue.
msg11512 (view) Author: jendrik Date: 2024-01-09.15:04:32
I'd like to backport some minor fixes and QOL improvements from Scorpion to Fast Downward. Can someone have a look at the pull request, please?
History
Date User Action Args
2024-12-13 11:46:20maltesetnosy: + davidspeck
messages: + msg11717
2024-04-04 11:15:11maltesetmessages: + msg11589
2024-01-10 13:20:49jendriksetstatus: reviewing -> resolved
messages: + msg11515
2024-01-09 15:56:20floriansetnosy: + florian
messages: + msg11514
2024-01-09 15:04:32jendrikcreate