[DEECo] [d3scomp/JDEECo] faf15c: Fixed and marked some bugs in the knowledge manage...

[Jaroslav Keznikl]

As a reply to:

// BTW, why do you decompose the initial object on the first level of
// nesting? Why can't you just store the initial object as such?

This was a trade-off between being able to dynamically add/remove knowledge
fields (this way, it can be done at least at the top level - which is
sufficient for most of our use cases), and the possibility to employ
structured (class-based) knowledge values.
The other two options is either store the initial object  - which prevents
addition/removal (well, it does not exactly prevent addition/removal, but
the implementation would be a bit more complicated).
The last option is to store the knowledge only as a hierarchical
composition of java.util.Map, which prevents (or largely complicates) using
user-defined classes for structured knowledge values.

So this is a simple trade-off to cover both use cases with minimal effort
(while not sacrificing much of the conceptual correctness).

J



2014-06-29 22:29 GMT+02:00 GitHub <noreply at github.com>:

>   Branch: refs/heads/sim-bures
>   Home:   https://github.com/d3scomp/JDEECo
>   Commit: faf15c745943bdf4e10cfd0d3df3c75a14f34ee9
>
> https://github.com/d3scomp/JDEECo/commit/faf15c745943bdf4e10cfd0d3df3c75a14f34ee9
>   Author: Tomas Bures <bures at d3s.mff.cuni.cz>
>   Date:   2014-06-29 (Sun, 29 Jun 2014)
>
>   Changed paths:
>     M
> jdeeco-core/src/cz/cuni/mff/d3s/deeco/knowledge/BaseKnowledgeManager.java
>     M
> jdeeco-core/src/cz/cuni/mff/d3s/deeco/model/runtime/custom/PathNodeComponentIdExt.java
>     M
> jdeeco-core/src/cz/cuni/mff/d3s/deeco/model/runtime/custom/PathNodeMapKeyExt.java
>     M jdeeco-core/src/cz/cuni/mff/d3s/deeco/task/KnowledgePathHelper.java
>
>   Log Message:
>   -----------
>   Fixed and marked some bugs in the knowledge management. The performance
> of the knowledge manager improved.
>
>
>
> _______________________________________________
> DEECo mailing list
> DEECo at d3s.mff.cuni.cz
> https://d3s.mff.cuni.cz/mailman/listinfo/deeco
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://d3s.mff.cuni.cz/pipermail/deeco/attachments/20140703/7b834b7c/attachment.html>