Issue911

Title Do not store TaskProxy in Pattern(Collection)Information
Priority wish Status reviewing
Superseder Nosy List jendrik, malte, silvan
Assigned To silvan Keywords
Optional summary
Pull request: https://github.com/aibasel/downward/pull/10

Created on 2019-03-21.16:19:41 by silvan, last changed by silvan.

Summary
Pull request: https://github.com/aibasel/downward/pull/10
Messages
msg8736 (view) Author: jendrik Date: 2019-03-21.16:50:18
I had the same question. Maybe this needs another offline discussion before we
act here.
msg8735 (view) Author: silvan Date: 2019-03-21.16:42:14
I understand this concern. We could also not do this here but in issue906, at
the cost of a larger diff that would possibly be reviewed/tested only at the
next sprint (would also be fine with me).
msg8734 (view) Author: malte Date: 2019-03-21.16:40:45
I had a quick look. My only question is whether this change actually goes into
the direction we want. I'm not saying it doesn't, but I'm not sure if we already
have a clear plan what the eventual data organization between these classes
should be. So I want to avoid a scenario where we change things back and forth.
msg8732 (view) Author: jendrik Date: 2019-03-21.16:36:16
I'll have a look.
msg8731 (view) Author: silvan Date: 2019-03-21.16:34:23
I opened a pull-request here:
https://bitbucket.org/SilvanS/fd-dev/pull-requests/47/issue911/diff
msg8726 (view) Author: silvan Date: 2019-03-21.16:19:40
As another preparation for issue906, where we would like to keep PDBs and their
patterns at the same place in PatternCollectionInformation, using
PatternInformation objects, we should get rid of task proxies that are currently
stored there just for commodity. Since this induces a diff at many files, I'd
like to do this here in a separate step.
History
Date User Action Args
2020-07-24 11:29:39silvansetsummary: Pull request: https://github.com/aibasel/downward/pull/10
2019-03-21 16:50:18jendriksetmessages: + msg8736
2019-03-21 16:42:14silvansetmessages: + msg8735
2019-03-21 16:40:45maltesetmessages: + msg8734
2019-03-21 16:36:16jendriksetmessages: + msg8732
2019-03-21 16:34:23silvansetstatus: chatting -> reviewing
messages: + msg8731
2019-03-21 16:19:41silvancreate