Free cookie consent management tool by TermsFeed Policy Generator

Opened 16 years ago

Closed 16 years ago

Last modified 14 years ago

#211 closed defect (done)

Full operator-graphs are stored multiple times in the database even though most of it is static anyway

Reported by: gkronber Owned by: gkronber
Priority: medium Milestone: HeuristicLab 3.3.0
Component: CEDMA.Core Version: 3.1
Keywords: Cc:

Description


Change History (17)

comment:1 Changed 16 years ago by gkronber

  • Status changed from new to assigned

comment:2 Changed 16 years ago by gkronber

  • Summary changed from Full operator-graphs are stored multiple times in the database even most of it is static anyway to Full operator-graphs are stored multiple times in the database even though most of it is static anyway

comment:3 Changed 16 years ago by gkronber

added a table for operators in the CEDMA database with r416

comment:4 Changed 16 years ago by gkronber

created a new type of operator library that is connected to the CEDMA db and a view for it with r417

comment:5 Changed 16 years ago by gkronber

added class OperatorLink that allows to create CombinedOperators in the DatabaseOperatorLibrary which use other operators from the library (not via a copy but via a link) with r418

comment:6 Changed 16 years ago by gkronber

  • Resolution set to fixed
  • Status changed from assigned to closed

worked on patching the operator-graph of an agent before execution with r419

comment:7 Changed 16 years ago by gkronber

  • Resolution fixed deleted
  • Status changed from closed to reopened

ticket was closed unintentionally in a hurry

comment:8 Changed 16 years ago by gkronber

  • Implemented downloading of linked operators from the database in AgentList (to allow view/edit of the whole operator graph)
  • Implemented downloading and patching of linked operators in RunScheduler to prepare the operator-graph for execution (OperatorLink operator can't be executed)

with r420

(there is some code duplication because the patching is very similar in AgentList DatabaseOperatorLibrary and RunScheduler. This needs some more work)

comment:9 Changed 16 years ago by gkronber

  • Status changed from reopened to new

comment:10 Changed 16 years ago by gkronber

  • Status changed from new to assigned

comment:11 Changed 16 years ago by gkronber

worked on this issue with r491:493

Patching of all operator links works correctly in test runs. However now all operator-links are patched on the first sweep. This means that sub-agents are already completely expanded (again fully expanded operator-graphs are stored multiple times in the DB unnecessarily). This issue needs to be fixed next.

comment:12 Changed 16 years ago by gkronber

fixed the bug in the run scheduler that caused full expansion of all operator links with r497.

However this solution is just barely working for the test example. We probably need a better solution for this.

comment:13 Changed 16 years ago by gkronber

There is still a bug where an operator-link is not patched before a view is created. Either make the operator-link patch itself or make sure all links are patched before the root operator is displayed.

comment:14 Changed 16 years ago by gkronber

  • Resolution set to fixed
  • Status changed from assigned to closed

r513: Fixed the problem that sometimes an operator link was not patched before a view was created (which resulted in a NullException) by adding a property for the source database in OperatorLink! and updating this property recursively whenever an operator is loaded from the CEDMA DB.

The method for recursively setting the property is in static class OperatorLinkPatcher because the code is needed in Agent, DatabaseOperatorLibrary and OperatorLink.

Closing this ticket because with the help of OperatorLinks it's easy to define the static parts of operator-graphs separately in the server side operator library and link them into the algorithm.

comment:15 Changed 16 years ago by swagner

  • Milestone changed from 3.1 to Iteration 0

Milestone 3.1 deleted

comment:16 Changed 14 years ago by swagner

  • Milestone changed from Iteration 0 to Current

Milestone Iteration 0 deleted

comment:11 Changed 14 years ago by swagner

  • Milestone changed from Current to HeuristicLab 3.3.0

Milestone Current deleted

Note: See TracTickets for help on using tickets.