<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
El 06/09/2008 10:31 PM, David García Garzón escribió:
<blockquote cite="mid:200806100331.33219.dgarcia@iua.upf.edu"
type="cite">
<pre wrap="">Pau, i just commited the patch previous to the pos and sizes because it
modifies the network XML and further changes might be done with care because
they might break stored networks. We should be sure that the format is the
one to stay for a while.
The point is that the proposed patch stores the position and the size as
string:
boxPos="20,25" boxSize="400,20"
Is that what we want?
not x='20' y='25 height='20' widht='400'
or even position='20,25' size='400,20'?
</pre>
</blockquote>
As you want. Plus, maybe it can be stored in a different path (let's
say "box") instead "processing".<br>
<br>
About the structure, I was thinking about network scalability
implementation... How to do that? making "super" or "intra" networks? I
mean, intra:<br>
<tt><br>
<network><br>
<br>
<subnetwork id="subnetwork1"><br>
.<br>
.<br>
</</tt><tt>subnetwork><br>
<br>
</tt><tt> <subnetwork </tt><tt>id="subnetwork2"</tt><tt>><br>
.<br>
.<br>
</</tt><tt>subnetwork></tt><br>
<tt><br>
<subnetwork_connection><br>
<port_connection><br>
</port_connection><br>
<control_connection><br>
</control_connection><br>
.<br>
.<br>
</</tt><tt>subnetwork_connection</tt><tt>><br>
</network><br>
</tt><br>
Or super:<br>
<tt><br>
<supernetwork><br>
<network id="network1"><br>
</network><br>
</tt><tt> <network id="network2"><br>
</network><br>
</tt> .<br>
.<br>
<tt></supernetwork><br>
</tt><br>
If I start the implementation on NE maybe the "supernetwork" can be a
"ClamNetworkCanvas"...<br>
<br>
<blockquote cite="mid:200806100331.33219.dgarcia@iua.upf.edu"
type="cite">
<pre wrap="">I also guess that the procAttribMap to store such attributes is a too generic
tool that can become subject of abuse if it is in the api. We are expected to
have some kind of network configuration/attributes and i don't know if this
is missleading when we actually want pos and size. Just a feeling not a
conviction. In this part we are more in Pau's zone.
Natanael, you should also review the naming conventions in the new functions:
Some local variables starting with upper case and some CripAbrvOnNams.
The worst thing on abreviating words is that you never remember which letters
you removed. Modern editors have completition. You can also use the context,
if the only ones in your context to have positions and sizes are processings
you can remove the Proc and call them prositions and sizes.
</pre>
</blockquote>
Yep, I tried to improve that in the new patch.<br>
<br>
<br>
Regards,<br>
Natanael.<br>
<br>
<blockquote cite="mid:200806100331.33219.dgarcia@iua.upf.edu"
type="cite">
<pre wrap="">
David.
El Tuesday 10 June 2008 02:24:57 Natanael Olaiz va escriure:
</pre>
<blockquote type="cite">
<pre wrap="">El 06/09/2008 03:56 PM, Pau Arumí Albó escribió:
</pre>
<blockquote type="cite">
<pre wrap="">El dg 08 de 06 de 2008 a les 11:20 +0200, en/na David García Garzón va
escriure:
</pre>
<blockquote type="cite">
<pre wrap="">I would like to have a commit on the cut&paste patch, to accept this one
separately. (i want the c&p working :-))) )
Pau, i'll do the commit of the patch before the positions in 30 min if
you don't do it first.
</pre>
</blockquote>
<pre wrap="">I like the commited code. And like even more the pace of the
progress :-)
</pre>
</blockquote>
<pre wrap="">Thanks :)
</pre>
<blockquote type="cite">
<pre wrap="">Natanael, are there patches pending for revision?
</pre>
</blockquote>
<pre wrap="">Just the store and load of positions and sizes. I attached here an
updated version.
The major issue that I see with this code is the duplicated code for
sizes and positions. In a while I'll send another version which makes
all in one step.
Regards,
Natanael.
</pre>
<blockquote type="cite">
<pre wrap="">P
_______________________________________________
Clam-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Clam-devel@llistes.projectes.lafarga.org">Clam-devel@llistes.projectes.lafarga.org</a>
<a class="moz-txt-link-freetext" href="https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel">https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel</a>
</pre>
</blockquote>
</blockquote>
<pre wrap=""><!---->
</pre>
</blockquote>
<br>
</body>
</html>