GatePorts in Switch

Trace

Debug for GatePorts in Switch

That graphs was wrong : OutPort with ObjRef type was connected to a GatePort without any reason. Now they run

New Switch tests with Gates

It was possible to create more than one link from OutPorts of a Switch to the same InPort of the EndSwitch

In RemoveInPort, if the OutPort was a SwitchPort and if it is not connected, it must be changed to be a InLinePort

Debug : Links between SwitchNodes and EndSwitchNodes was masked.

After the destroy of a note we call UnValid instead of IsValid() and there is no error even if the graph is not valid

Call of ValidSwitchs during validation step

Traces

Debug for SwitchNodes in SomeDataReady ( see comments)
Traces and Comments
Debug : some CORBA types was missing for InLinesNodes in DataReady_ExecuteAction
Debugs for SwitchNodes (see comments)

Valid change of signature : bool argument that means that we have to execute CheckLoop/CheckSwitch or not.
If there is an error at the creation of a link in LoadXml, we continue to load the graph (see comments)
In Valid(), we check now the switches of the graph

IsValid change of signature : bool argument that means that we have to execute CheckLoop/CheckSwitch or not.

Method for CheckSwitch feature

Traces
Methods and fields for CheckSwitch feature

Debug in RemoveInPort (Kind field)

Debug in RemoveOutPort
New methods for CheckSwitch feature

Debugs of ReNameNode (see comments)
Debugs of RemoveNode (see Comments)
PAL7990
Debug in AddLink (see Comments)
New ValidSwitchs method

CheckSwitch step during validation of graphs

Value of GatePorts

CheckSwitch during the validation of graphs

There was a bug in the fori node (3 output arguments in Initfori function instead of 2).

New tests of SIGSEGV management

Many new tests of Switches

*** empty log message ***

Many tests of Switch Control and behavior : Validity before execution and control of SwitchPorts at execution time

Ir is now possible to have an Input Value inside a loop and execution take care now of that behavior (before there was a hangup).

New tests for Switch behavior and Control

*** empty log message ***

Different steps of an edition of a graph

This commit was generated by cvs2git to create branch
'JR_ASV_2_1_0_deb_with_KERNEL_Head'.

Cherrypick from master 2005-01-28 12:14:58 UTC asv <asv@opencascade.com> 'New example graphs from Jean Rahuel.  Most of them test correct exectution of Switch nodes.':
    examples/GraphGOTO1.py
    examples/GraphGOTO1.xml
    examples/GraphGOTOAborted.py
    examples/GraphGOTOAborted.xml
    examples/GraphLoopSwitchOfSwitch.py
    examples/GraphLoopSwitchOfSwitch.xml
    examples/GraphLoopSwitchOfSwitchNOTValid.py
    examples/GraphLoopSwitchOfSwitchNOTValid.xml
    examples/GraphLoopSwitchs.py
    examples/GraphLoopSwitchs.xml
    examples/GraphSwitchCheckBranch.py
    examples/GraphSwitchCheckBranch.xml
    examples/GraphSwitchCheckBranch1.py
    examples/GraphSwitchCheckBranch1.xml
    examples/GraphSwitchCheckDefault1NOTValid.py
    examples/GraphSwitchCheckDefault1NOTValid.xml
    examples/GraphSwitchCheckDefault2NOTValid.py
    examples/GraphSwitchCheckDefault2NOTValid.xml
    examples/GraphSwitchCheckDefault2NOTValid1.py
    examples/GraphSwitchCheckDefault2NOTValid1.xml
    examples/GraphSwitchCheckDefault2NOTValid2.py
    examples/GraphSwitchCheckDefault2NOTValid2.xml
    examples/GraphSwitchOutputNOTValid.py
    examples/GraphSwitchOutputNOTValid.xml

Modifications from last integration into PAL CVS were taken from there and merged into this version.  It was a bug with incorrect order of libraries.  Please, see comment to PAL7687 for details.

Modifications from last integration into PAL CVS were taken from there and merged into this version.  Nothing is lost!

A modification caused by change of SALOMEDS::Tool interface (accept std::string parameter instead of CORBA::String)

Changes were made to make the code compilable with head version of KERNEL.  It is compilable and working: execution of InLine nodes works.
Control of execution of Inline nodes - does not work (Kill, Suspend, Resume), because it was implemented using signals, that were caught and handled in Container.

Minor changes in various source files to remove compilation warnings (unused variables, non void function reaches end of control of execution, etc..)

Correct Thread placing if any node was deleted
in Full view.

Fix the following cases:
1) closing dataflow window in Table view leads to exception,
2) node's deletion in Full view and then switching to Table
   view leads to exception,
3) remove unnecessary COUTs and MESSAGEs.

A lot of unneccessary "cout"-s and MESSAGE()-s were removed.

That graph was wrong : 2 branches of the SwitchNode were executed at the same time ; that is not allowed. It is now corrected : there is only 2 brnaches now.

Execution of graphs with GOTONodes

Allow to create links to/from "default" ports of switch node in LinkBuilder::canCreateEngine() function.

A bug was fixed: paste of inline node with EMPTY function raised exception.  Now it is fixed.

Traces

Debug : an OutPort connected to a Gate may only be connected again to a Gate of an other node
Debug : RemoveLink of a link to an EndSwitchNode was wrong

Graphs for tests of Switchs

Loop with negative values

More than one branch of a switch is activated, the SwitchNode is aborted

If more than one branch of a switch is activated, the SwitchNode is aborted

The final values of InPorts of EndSwitchNodes in a loop may be wrong

Traces

InitLoop and DoLoop parameters names

Graph with Switch with two branches and in a Loop

Graph with C++ Components loaded in the SuperVisionContainer

A small bug was fixed: when library is not accessible (library file not found, etc.), the Library dialog with empty list of library nodes was still shown.  Now a new method Library::CanImport() is introduced.  It is called BEFORE showing the dialog, if it returns false - dialog is not shown.

"InitLoop" port name changed to "DoLoop".  This change was implemented in the Engine's code as well.  "DoLoop" is the correct name of "hidden" port of Loops.

The convert obj_ref to different types example was modified.  It is prohibited now to convert obj_ref to any type exept string.

The following bug was fixed:
for Loop node, removal of an Input Port automatically removes the corresponding Output Port.  So for Loops the oldPorts list (removed before creation of new ports) should be filtered to include only Input Ports.

Modifications for automatically rename EndControl node then
StartControl node's name is changed.

A constructor and destructor added.
A bug was fixed: replacement of ALL occurances of function name in the node's code, NOT only in "def Func_Name():" line.

A constructor with parent argument added, the default static creation function takes QAD_Desktop as a parent.

Creation of "read only" graph by GraphE() function was modified.  try construct added as it is done in "normal" graph creation in Graph() function.

replacement "on the fly" (during the call) of "InitLoop" port by "DoLoop" port is commented out.  Please, see the comment in the code (use diff with previous version).

Copy/Paste ports and nodes functionality was transfered to SUPERVGUI_Clipboard class.  All references to the functions in various classes were modified to reflect this change.

Gates were not properly handled in Loops ; now they are
Input Values of ports were allowd but were not properly handled : now they are
See GraphInputLoop in examples
See GraphLoopGates and GraphLoopsGates in example

Test with Loop(s) and with Inputs and Outputs inside the loop and only with links between Gates

Test with a Loop and with Inputs and Outputs inside the loop

PAL7120 : debug of destroy of Loop/EndLoop Nodes ; destroy of Switch/EndSwitch Nodes.

Kill of InLineNodes error (except LoopNodes)

Point 2.3 of "SUPERVISOR: Current state - bugs/improvements":
disable popup menu items "Copy Node" and "Paste Port"
if Loop or Switch node has been copied and then delete
before paste operation.

Point 2.3 of "SUPERVISOR: current status - bugs/improvements" :
correcting naming of the "InLine" nodes during copy/paste.

Point 2.3 of "SUPERVISOR: Current state - bugs/improvements" :
enable popup menu item "Copy Node" on EndLoop and EndSwitch nodes,
disable popup menu items "Paste Port" and "Paste Node" if port and node,
which has been copied, were deleted respectively.

The internal predefined port of a LOOP node was renamed from "InitLoop" to "DoLoop" in the Supervisor Engine.  This change influenced all graphs that contain Loops - they must be changed, all references to "InitLoop" port must be replaced with "DoLoop" port in PY and XML files.

Methods ReRun(), ReRunAt(), ReStart(), ReStartAt() were removed from CNode's interface.  These methods are called in GraphSyracuse* python scripts - decided to remove them with the corresponding XML graphs.

Error with incorrect popup items ("Edit Function" appeard even for Factory nodes) is fixed.

Methods ReStart(), ReRun(), ReStartAt(), ReRunAt() were removed, since they were removed from SUPERV.idl

Remove some warnings from compilation process.

Implementation of point 2.3. from
"SUPERVISOR: Current status - bugs/impovements"
document : Copy/paste nodes, ports.

New messages for Library functionality.

Import and Export of InLine nodes to Library (external file) is implemented.  This integration has all functionality done.

An improvement for adding a library node: it will use addNode() function, so creation of a PythonEditPane and LibDlg must take Service::myX, myY as arguments for future modification (on Add node command).

A new functionality: Import/Export Inline nodes to Library is implemented.  CanvasNode modification: add a popup command to call "Export to Library" on an InLine node.

Adding a Node to viewport is unified: one common method addNode() is created in Service to be called form all places where Engine's nodes are created (Paste, Library, etc.)

Messages are modified as usual: new messages were added.

IsEqual() is added to Link_Impl

An improvement on link creation: if a link already exists between the ports, then on new link creation it is removed, and new link is created.  If link already exists - is important check for Input Gate ports, which can accept many incoming links.

Point 2.6 from "SUPERVISOR: Current status - bugs/improvements" :
implementation of "Fit within rectangle" functionality.

An improvement on link creation: if a link already exists between the ports, then on new link creation it is removed, and new link is created.  If link already exists - is important check for Input Gate ports, which can accept many incoming links.

indentation of Python code in Python editor pane is implemented.

Point 2.12 from "SUPERVISOR: Current status - bugs/improvements"
document : Move segment of link.

The previous version had syntax error on line 1293, now it was fixed.

Execution Control and Loops

Execution Control and Loops

Point 2.8 of "SUPERVISOR: Current status - bugs/improvements":
Adding node to graph window with taking into account zoom factor.

PAL7385 (Bugs and Improvements p2.5) was fixed.  Output Gate ports added to Loop and EndLoop nodes.

1. PAL6170, "Bugs and Improvements" p.2.21: calling of dataflow->Editing() added inside each of edition operations (see list of edition operations in comments of PAL6170).  Also removed "Set Value" command from popup of CanvasPort in during execution (in Suspend).

2. removed ReRun(), ReStart() methods from Main as deprecated (p.1.20 of "Bugs and Improvements").

Removed use of deprecated constants from SUPERV namespace: ReRunEvent, ReStartEvent, ReRunState, ReStartState.

The following methods and constants were removed as never being used:
OutNode: Run(3 params), ReRun(), ReStart()
DataFlow: Run(3 params),
InNode: ReStart()
FiniteStateMachine: ReRunnedState.
Also removed use of deprecated constants from SUPERV namespace: ReRunEvent, ReStartEvent, ReRunState, ReStartState.

a new private method run() was introduced instead of using CNode_Impl::Run() which was removed.  run() is now called from both Run() ("Execute graph") and Start() ("Execute graph step-by-step") to remove code dublication.

Run(), ReRun(), ReRunAt(), ReStart(), ReStartAt() methods were removed.  ReXXX methods were removed from IDL as never being called.  Run() methods was removed, its functionality (basic calling of Executor->Run() ) was moved to Graph_Impl with modification.