[Scstudio-devel] [ scstudio-Feature Requests-3530648 ] Error in rendering disconnected MSC

SourceForge Headquarters 225 Broadway Suite 1600 San Diego, CA 92101 +1 (858) 454-5900

Feature Requests item #3530648, was opened at 2012-05-29 13:01
Message generated for change (Comment added) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1046489&aid=3530648&group_id=219467

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Visio Interface
Group: None
Status: Open
Priority: 5
Private: No
Submitted By: Gilberto Tavan (pomate)
Assigned to: Matus  Madzin (madzin)
Summary: Error in rendering disconnected MSC

Initial Comment:
I've noticed SCstudio has a bug in import function, when rendering of disconnected MSC.
When a new instance of a node pops up, the messages are placed at the beginning of the .vsd file,
they should be always added after the last message.

Having multiple interfaces (IPaddresses) is quite common in SIP applications.

In the attachment an example of SIP Call Flow, where the rendering is incorrect.

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2012-06-22 16:52

Message:
A few comments on .mpr creation script pcap2z120.py, I know it may seem OoS
at first sight, but if current pcap2z120 algorithm will be inherited by the
new feature there are a few things to take into account:

1) pcap2z120.py  assumes the instances should be IP address only. 
In telco applications and especially in LTE it is quite common to have a
Point Code, i.e. a single number,  instead of an IP address as instance of
an MSC. 

2) It's also quite common to perform Network Name resolution of the IP
adresses in Wireshark in order to automatize the substitution of IP
addresses with Node Names, using host file. In this case an instance can be
represented by an alphanumeric string.

3) In telco application of MSC, i.e. Call Flows most of the time it is not
necessary to have a linearization proportional to time stamps, so it could
be an option.
The important thing is to be able to use vertical spacing (i.e. time
constraint) on new "disconnected" instances.
After importing with timestaps coordinates beautify function could
linearize vertical spacing as it does today, while other applications may
require to respect timestamps spacing.

4) Regarding  .mpr import function, that will have anyway a rendering
problem, the vertical spacing could be proportional to Message Number,
which is already available in .mpr file

PS: I was referring to current z120 import function, I would like to take a
look to a possible implementation of the concept expressed in bullet 4)
Thanks

----------------------------------------------------------------------

Comment By: Vojtech Rehak (xrehak)
Date: 2012-06-21 23:56

Message:
bug -> feature request

We will solve it as a special kind of Beautify that will take time
information into account.

Gilberto wrote:
> PS I've searched for import procedure in the source code, but I can't
find them easily.

The import procedure itself is in "trunk/src/data/pcap/". It reads a given
file and fills  the internal data structure.
It has not been implemented In the visio part yet, i.e. in
"trunk/src/view/visio/addon/document.cpp". 

----------------------------------------------------------------------

Comment By: Gilberto Tavan (pomate)
Date: 2012-06-09 07:20

Message:
Hi Matus,

Your interpretation is absolutely correct, sorry I could have been more
clear in my explanations.

Using .pcap timestamps as references would give a correct temporal order,
which would then render the MSC exactly as mscgen.exe does, but with the
great advantage of editing capabilities thanks to SCstudio.
The time constraint would solve the problem, if time constraint implies a
graphical spacing on the Y-axis.

The correct order and a constant spacing is more than enought in most MSC
Diagrams in my area of expertise  (which is Telecomunications) 

Talking about Z120 import  .mpr files produced by pcap2z120.py do not
contain timestamps, but I think that correct temporal order and constant
spacing could be easier to implement and have a better aesthetical result.

Thanks to you all for this unique and handy tool.

BR

Gilberto

PS I've searched for import procedure in the source code, but I can't find
them easily.

----------------------------------------------------------------------

Comment By: Matus  Madzin (madzin)
Date: 2012-06-03 09:14

Message:
Hi, 
Firstly I would like to thank Mr. Tavan for good report. Now, I replay to
xrehak's comment. 

Actually, I can confirm that it's basic feature of MSC, but I don't like
any of xrehak's suggestions. 

If I understand the problem correctly, Mr. Tavan wants a linearization over
all messages in an MSC diagram. To reach this functionality by clear
solution, I suggest to set time stamp of each message from pcap as an
absolute time constraint and create new feature of SCStudio to linearize
messages in MSC diagram if it's possible (when the diagram is created from
pcap file, it should be always possible).

I think, it would be correct to assign this ticket to feature request
category.

Matus 

----------------------------------------------------------------------

Comment By: Vojtech Rehak (xrehak)
Date: 2012-05-29 23:18

Message:
Actually, it is a basic feature of MSC that disconnected events on
different instances can be placed arbitrarily. 

I suggest two solutions:
- consider the interfaces of a single computer as standalone components and
add some messages between these interfaces representing internal
communication proceed within the computer; hence, the relevant messages
will not be disconnected any more. (Ok. this should be hardly achieved from
pcap. )
- consider the multiple interfaces as nicknames of the same computer and
make a single instance representing all of them together (Matus, is there
any possibility to specify this by parameters of pcap2z120.py?)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1046489&aid=3530648&group_id=219467