Chronological Thread 

< Chronological >      < Thread >

---

Randall Britten wrote:
> Don't you think we should put a test in somewhere for this? (Perhaps a
> manual test for it is already in Litmus?).
>

It is already in Litmus, although we could probably do automated testing
of PCEnv too (it requires running Javascript as part of the tests, but
Mozilla do this so we could do this if we want to depend on having debug
versions of Mozilla available as part of build process - probably a
space problem for now on my VM, but once we get our testing platform I
have asked for much more space for the Win32 image).

Best regards,
Andrew

> -----Original Message-----
> From: automated-notifications-bounces at cellml.org
> [mailto:automated-notifications-bounces at cellml.org] On Behalf Of CellML
> Automated Notifications
> Sent: Wednesday, 19 September 2007 4:11 p.m.
> To: automated-notifications at cellml.org
> Subject: r1756 - pce/trunk/chrome/content/ui
>
> Author: amil082
> Date: 2007-09-19 16:10:36 +1200 (Wed, 19 Sep 2007)
> New Revision: 1756
>
> Modified:
> pce/trunk/chrome/content/ui/Menu.js
> Log:
> Fix export CSV function, which was been broken as a consequence of the move
> to the new
> CCGS.
>
> Modified: pce/trunk/chrome/content/ui/Menu.js
> ===================================================================
> --- pce/trunk/chrome/content/ui/Menu.js 2007-09-18 17:47:09 UTC (rev
> 1755)
> +++ pce/trunk/chrome/content/ui/Menu.js 2007-09-19 04:10:36 UTC (rev
> 1756)
> @@ -153,34 +153,48 @@
> function WriteCSVData(datacollector, cmodel, stream)
> {
> var header = '';
> - var varit = cmodel.codeInformation.iterateVariables();
> + var ctit = cmodel.codeInformation.iterateTargets();
> var vars = [];
> var i;
> - var v, bv;
> - while ((v = varit.nextVariable()) != null)
> - if (v.type == 0/* BOUND*/)
> - bv = v.source.name + '(' + v.source.unitsName + ')';
> - else if (v.type == 1/* CONSTANT */)
> + var ct, bv;
> + var rateCount = cmodel.codeInformation.rateIndexCount;
> +
> + while ((ct = ctit.nextComputationTarget()) != null)
> + if (ct.type == 0/* VARIABLE_OF_INTEGRATION */)
> + bv = ct.variable.name + '(' + ct.variable.unitsName + ')';
> + else if (ct.type == 1/* CONSTANT */)
> ;
> - else
> - vars[v.variableIndex] = v.source.name + '(' + v.source.unitsName +
> ')';
> - var vc = cmodel.codeInformation.variableCount;
> - for (i = 0; i < vc; i++)
> - {
> - if (header != '')
> - header = header + ',';
> - header = header + vars[i];
> - }
> - if (bv != null)
> - {
> - if (header != '')
> - header = header + ',';
> - header = header + bv;
> - }
> + else if (ct.type == 4/* FLOATING */)
> + ;
> + else // state variable or algebraic...
> + {
> + var isSV = (ct.type == 2/* STATE_VARIABLE */);
> + var isRate = (ct.degree > 0);
> + var idx = isSV ? 1 : (isRate ? rateCount + 1 : 2 * rateCount + 1);
> + idx += ct.assignedIndex;
> +
> + var primes = "";
> + if (isRate)
> + {
> + var x;
> + for (x = ct.degree; x > 0; x--)
> + primes = primes + "'";
> + }
> + else
> + primes = '(' + ct.variable.unitsName + ')';
> +
> + vars[idx] = ct.variable.name + primes;
> + }
> +
> + header = bv;
> + var vc = 2 * rateCount + 1 + cmodel.codeInformation.algebraicIndexCount;
> +
> + for (i = 1; i < vc; i++)
> + header += ',' + vars[i];
> +
> header = header + "\n";
> stream.write(header, header.length);
>
> - vc = vc + 1;
> try
> {
> var t = 0;
>
> _______________________________________________
> automated-notifications mailing list
> automated-notifications at cellml.org
> http://www.cellml.org/mailman/listinfo/automated-notifications
>
>

---

• [cellml-discussion] FW: r1756 - pce/trunk/chrome/content/ui, Andrew Miller, 09/19/2007

---

Archive powered by MHonArc 2.6.18.