CDL changes

[mwetter]

Based on TAG feedback, implement the following (non-backward) compatible changes:

• Remove Continuous.Add3 and Integer.Add3 blocks, and instead add an Integer.MultiSum with parameter Integer k[nu]=fill(1, nu) "Input gains"; (there is already a Continuous.MultiSum).
• Rename SignalRanker to Rank (or better, Sort) as all blocks operate on signals anyway. Sort may be better as it sorts the signal.
• Remove Continuous.Sum as this does the same as Continuous.MultiSum.
• Add Continuous.Round that rounds the input to n digits, and remove Continous.Truncation as this can be done with the Round block.
• Add Continuous.MultiMin and Continuous.MultiMax and remove the current MinMax. Reason: often one only needs either the min or the max, but not both.

[JayHuLBL]

@mwetter Following changes have been done in branch issue933_CDL_changes:

• removed Continuous.Add3, Integers.Add3;
• created Integers.MultiSum;
• renamed Continuous.SignalRanker to Continuous.Sort;
• removed Continuous.MinMax;
• added Continuous.MultiMin and Continuous.MultiMax;
• removed Continuous.Trunction ;
• added Continuous.Round;

Regarding Continuous.Sum and Continuous.MultiSum, they are different. As for Sum, its input can be a vector with the size being parameterized. The MultiSum receiving inputs by drawing lines and its number of inputs is changing with the connected line. We'd probably keep both blocks.

[mwetter]

I agree we should keep both, MultiSum and Sum. Then, blocks that automatically size the parameter nin should have the name Multi*. We are consistent in this respect with MultiSum and MultiAnd

[JayHuLBL]

I just realized that we can use MultiSum to replace Sum. I will update it now.

[mwetter]

Will the parameter `nin` be configured correctly if a vector is connected graphically?

…

On Thu, Sep 14, 2017 at 5:11 PM, Jianjun Hu ***@***.***> wrote: I just realized that we can use MultiSum to replace Sum. I will update it now. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#933 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABRHuDNjcPO74INISwDrxLxJd5bjL4-9ks5sicDNgaJpZM4PWtfB> .

[JayHuLBL]

Yes, the parameter `nin` will automatically be configured according to the vector size. On Thu, Sep 14, 2017 at 5:31 PM, Michael Wetter <notifications@github.com> wrote:

…

Will the parameter `nin` be configured correctly if a vector is connected graphically? On Thu, Sep 14, 2017 at 5:11 PM, Jianjun Hu ***@***.***> wrote: > I just realized that we can use MultiSum to replace Sum. I will update it > now. > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#933# issuecomment-329642225>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/ ABRHuDNjcPO74INISwDrxLxJd5bjL4-9ks5sicDNgaJpZM4PWtfB> > . > — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#933 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AWZ1UQIuWEnrZqCWQNgfufkmivpUtgwyks5sicVzgaJpZM4PWtfB> .

-- -------------------------------------- Jianjun Hu, PhD Simulation Research Group Lawrence Berkeley National Laboratory 1 Cyclotron Road, MS 90R3147 Berkeley, CA 94720 email: JianjunHu@lbl.gov <TSNouidui@lbl.gov> http://simulationresearch.lbl.gov ----------------------------------------