Any objections to that? I'd go for a quick refactoring without a jira if no one objects.

--sebastian

On 11.02.2011 22:36, Ted Dunning wrote:
+10!!

On Fri, Feb 11, 2011 at 10:13 AM, Sebastian Schelter <[email protected]
<mailto:[email protected]>> wrote:

    Maybe we should rename them to something like dimension and
    initialCapacity then?

    --sebastian


    On 11.02.2011 18:49, Weishung Chung wrote:

        Thanks a lot for the explanation. This really helps me out :)

        On Fri, Feb 11, 2011 at 11:38 AM, Ted
        Dunning<[email protected] <mailto:[email protected]>>
          wrote:

            Argh!

            This is a really confusing API.

            Cardinality is the dimension of the vector.

            Size is the number of storage elements that you want to have
            in the vector
            initially, much in the style of ArrayList where you specify
            how many
            elements to pre-allocate.

            On Fri, Feb 11, 2011 at 8:33 AM, Weishung
            Chung<[email protected] <mailto:[email protected]>>
            wrote:

                Is cardinality the original size of the vector including
                zeros and size

            is

                the number of nonzeros in the vector?
                I am referring to

                  public RandomAccessSparseVector(int cardinality, int size)

                Thank you :)




Reply via email to