Bug 485 - implement deserialization of normal/gaussian random variables
: implement deserialization of normal/gaussian random variables
Status: RESOLVED FIXED
: ns-3
simulation core
: ns-3-dev
: All All
: P5 normal
Assigned To:
:
: patch
:
:
  Show dependency treegraph
 
Reported: 2009-02-02 11:54 EDT by
Modified: 2009-03-03 06:25 EDT (History)


Attachments
patch (1.84 KB, patch)
2009-02-02 11:54 EDT, Gustavo J. A. M. Carneiro
Details | Diff
implement serialization also (5.79 KB, patch)
2009-02-25 06:22 EDT, Gustavo J. A. M. Carneiro
Details | Diff
rebased patch (3.91 KB, patch)
2009-02-25 07:46 EDT, Gustavo J. A. M. Carneiro
Details | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2009-02-02 11:54:03 EDT
Created an attachment (id=366) [details]
patch

like the summary says
------- Comment #1 From 2009-02-25 05:25:48 EDT -------
The patch looks good to me but, gustavo, could you please add serialization
support also ?
------- Comment #2 From 2009-02-25 06:22:05 EDT -------
Created an attachment (id=388) [details]
implement serialization also
------- Comment #3 From 2009-02-25 06:26:02 EDT -------
(In reply to comment #2)
> Created an attachment (id=388) [details] [details]
> implement serialization also
> 

Would you mind generate a final patch against the tip of ns-3-dev ? This patch
does not apply since the rng changes have been merged.
------- Comment #4 From 2009-02-25 07:46:13 EDT -------
Created an attachment (id=391) [details]
rebased patch
------- Comment #5 From 2009-02-25 07:56:20 EDT -------
ok for me. let's ask raj and michele for an ack.
------- Comment #6 From 2009-03-02 15:41:03 EDT -------
I don't see any problems with this patch.  Is this something that should be go
ahead and be added for 3.4 or wait until later?
------- Comment #7 From 2009-03-02 15:51:42 EDT -------
(In reply to comment #5)
> ok for me. let's ask raj and michele for an ack.
> 

ACK.

(In reply to comment #6)
> I don't see any problems with this patch.  Is this something that should be go
> ahead and be added for 3.4 or wait until later?
> 

My two cents is that there is no sense in allowing this to bitrot again if it
works as promised now.
------- Comment #8 From 2009-03-03 06:25:00 EDT -------
I'll take that as approval.  Committed.