Issue618

Title use function for formatting paper references
Priority wish Status resolved
Superseder Nosy List florian, jendrik, malte
Assigned To jendrik Keywords
Optional summary

Created on 2016-01-06.10:11:42 by jendrik, last changed by jendrik.

Messages
msg5081 (view) Author: jendrik Date: 2016-01-12.12:38:11
Merged and pushed to the private repo. Thanks for the review!
msg5080 (view) Author: florian Date: 2016-01-12.12:05:55
Looks good to me.
msg5079 (view) Author: jendrik Date: 2016-01-12.10:53:10
I finished the pull request. Could you have a look, please, Florian?
msg5078 (view) Author: florian Date: 2016-01-11.19:26:25
I'm fine with merging this, but I just merged issue625, which renames the
namespace Utils to utils, which will require some small changes (sorry).
msg5076 (view) Author: malte Date: 2016-01-11.16:57:19
I'll defer to Florian. :-) (IOW, please merge when the two of you are happy.)
msg5075 (view) Author: jendrik Date: 2016-01-11.16:19:44
I have started a branch for this issue. Can one of you guys have a look and 
tell me if you're ok with the function's signature, please?

https://bitbucket.org/jendrikseipp/downward/pull-requests/41

The Merge-and-shrink code repeats some references in multiple places. Should we 
add a new header merge_and_shrink/papers.h to avoid repetition?
msg5071 (view) Author: malte Date: 2016-01-09.18:30:41
I looked at the function and like it. Anyone want to assign this to themselves?

I guess the main part of the work is adapting the existing paper references in
the paper.
msg5034 (view) Author: jendrik Date: 2016-01-06.10:11:42
The CEGAR code (issue600) adds a function format_paper_reference() in 
cegar/additive_cartesian_heuristic.cc. We should move this function to a more 
central place and use it for formatting all paper references.
History
Date User Action Args
2016-01-12 12:38:11jendriksetstatus: reviewing -> resolved
messages: + msg5081
2016-01-12 12:05:55floriansetmessages: + msg5080
2016-01-12 10:53:15jendriksetstatus: chatting -> reviewing
2016-01-12 10:53:10jendriksetmessages: + msg5079
2016-01-11 19:26:25floriansetmessages: + msg5078
2016-01-11 16:57:19maltesetmessages: + msg5076
2016-01-11 16:19:44jendriksetmessages: + msg5075
2016-01-11 14:22:17jendriksetassignedto: jendrik
2016-01-09 18:30:41maltesetstatus: unread -> chatting
messages: + msg5071
2016-01-06 10:11:42jendrikcreate