Mailing-List: contact user-help@mahout.apache.org; run by ezmlm
Precedence: bulk
Reply-To: user@mahout.apache.org
Received-SPF: neutral (athena.apache.org: local policy)
MIME-Version: 1.0
In-Reply-To: 
 <CANx3uAh0qS=K6WiuXeTJ5r3-asMqm+60O4b10cbMD5=y_fSSMg@mail.gmail.com>
References: 
 <CANx3uAi1a3DCCbhDBxFkA-Omte9dQZpxep3Gw_iWvhJF3MD00g@mail.gmail.com>
	<CAJwFCa2h1PcP1bH-Z1U-0AJ08qVosEpyeKVVKeDPCysjiK331g@mail.gmail.com>
	<CANx3uAigbw2=eu=Q9b7e3gfu1hV_8zbYr75x-nJus4=TmfY5KA@mail.gmail.com>
	<CACYXym9HtYxfZJdKV5PXYSkUfW3e=z9vaYowZ+Jy=+RzT0qaxQ@mail.gmail.com>
	<CANx3uAgwK++jkZcEvatFjNihSsJ7z+3LT2WV-Opn=7d+3TscWg@mail.gmail.com>
	<CANx3uAjNkeiDZOiRXg50_bTj5gkN-jv10Yz-mOB1WD68LmYKAQ@mail.gmail.com>
	<CACYXym8Aax1-2jNscfA_8TkXrb5goPYMXpH+w-Xz6DsU=85O_w@mail.gmail.com>
	<CANx3uAhTJ_hTVWXtZhbYQqFQ2VhDXfM-hzx38oXZ3VCFrvgsUg@mail.gmail.com>
	<CACYXym-qP4S0vF0Qq1=X8_=swRUycYkOLw1vm1vv4gCZ-E7VWA@mail.gmail.com>
	<CANx3uAh0qS=K6WiuXeTJ5r3-asMqm+60O4b10cbMD5=y_fSSMg@mail.gmail.com>
Date: Mon, 14 Jan 2013 16:41:57 -0500
Message-ID: 
 <CANx3uAgA+JHEUJOoH+_QtdGa5bT+m21Ai1pXn4zr8a3JQd=y5Q@mail.gmail.com>
Subject: Re: understanding vectors
From: Koert Kuipers <koert@tresata.com>
To: user@mahout.apache.org
Content-Type: multipart/alternative; boundary=047d7b5d4498acc3f604d3468342

--047d7b5d4498acc3f604d3468342
Content-Type: text/plain; charset=ISO-8859-1

i probably wasn't clear before: i can not reproduce bug in trunk.

On Mon, Jan 14, 2013 at 4:40 PM, Koert Kuipers <koert@tresata.com> wrote:

> ok i can write unit test for trunk, but it will succeed, since this is
> fixed in trunk. ok?
>
> On Mon, Jan 14, 2013 at 4:35 PM, Jake Mannix <jake.mannix@gmail.com>wrote:
>
>> We don't have a way to add the code to 0.7, it would just go in trunk.
>>
>>
>> On Mon, Jan 14, 2013 at 1:19 PM, Koert Kuipers <koert@tresata.com> wrote:
>>
>> > since trunk behaves properly, where do you want unit test? in version
>> 0.7?
>> >
>> > On Mon, Jan 14, 2013 at 3:59 PM, Jake Mannix <jake.mannix@gmail.com>
>> > wrote:
>> >
>> > > I could have sworn we've seen and fixed this bug before, but I guess
>> not.
>> > >  Let's get a (failing) unit test in there, and then it should be an
>> easy
>> > > fix.
>> > >
>> > > I think nobody runs into this because it's really rare to want to
>> iterate
>> > > over all of the entries in a sparse vector - iterateNonZero() is used
>> > > almost exclusively for these.
>> > >
>> > >
>> > > On Mon, Jan 14, 2013 at 12:51 PM, Koert Kuipers <koert@tresata.com>
>> > wrote:
>> > >
>> > > > not the same behavior in trunk. SequentialAccessVector.iterator
>> seems
>> > to
>> > > > behave properly there.
>> > > >
>> > > > On Mon, Jan 14, 2013 at 3:07 PM, Koert Kuipers <koert@tresata.com>
>> > > wrote:
>> > > >
>> > > > > sorry yes i meant 0.7
>> > > > >
>> > > > > On Mon, Jan 14, 2013 at 12:58 PM, Jake Mannix <
>> jake.mannix@gmail.com
>> > > > >wrote:
>> > > > >
>> > > > >> I think you mean 0.7, right?  Can you see if you get the same
>> > behavior
>> > > > in
>> > > > >> svn trunk?
>> > > > >>
>> > > > >>
>> > > > >> On Mon, Jan 14, 2013 at 9:21 AM, Koert Kuipers <
>> koert@tresata.com>
>> > > > wrote:
>> > > > >>
>> > > > >> > i am using version mahout 7.0
>> > > > >> >
>> > > > >> > On Mon, Jan 14, 2013 at 10:15 AM, Ted Dunning <
>> > > ted.dunning@gmail.com>
>> > > > >> > wrote:
>> > > > >> >
>> > > > >> > > Which version are you using?
>> > > > >> > >
>> > > > >> > > (this misbehavior sounds familiar)
>> > > > >> > >
>> > > > >> > > iterator() should return all values.
>> > > > >> > >
>> > > > >> > > iterateNonZero() is allowed to skip zeros.
>> > > > >> > >
>> > > > >> > > On Mon, Jan 14, 2013 at 6:11 AM, Koert Kuipers <
>> > koert@tresata.com
>> > > >
>> > > > >> > wrote:
>> > > > >> > >
>> > > > >> > > > i am looking at the iterators for DenseVector,
>> > > > >> > RandomAcccessSparseVector
>> > > > >> > > > and SequentialAccessSparseVector.
>> > > > >> > > >
>> > > > >> > > > for both DenseVector and RandomAcccessSparseVector the
>> > iterator
>> > > > >> seems
>> > > > >> > to
>> > > > >> > > > return all values, including the missing zero values.
>> > > > >> > > > for SequentialAccessSparseVector the iterator also returns
>> all
>> > > > >> values,
>> > > > >> > > but
>> > > > >> > > > only up to the last non-missing value!
>> > > > >> > > >
>> > > > >> > > > is this by design? what is a vector iterator supposed to
>> > return
>> > > > >> > exactly?
>> > > > >> > > i
>> > > > >> > > > can't see a logical pattern/consistency. see examples
>> below.
>> > > > >> > > > thanks! koert
>> > > > >> > > >
>> > > > >> > > > scala> val x = new
>> > > > >> org.apache.mahout.math.RandomAccessSparseVector(5)
>> > > > >> > > > x: org.apache.mahout.math.RandomAccessSparseVector = {}
>> > > > >> > > >
>> > > > >> > > > scala> x.set(3, 1.0)
>> > > > >> > > >
>> > > > >> > > > scala> for (item <- x.iterator.asScala)
>> println((item.index,
>> > > > >> item.get))
>> > > > >> > > > (0,0.0)
>> > > > >> > > > (1,0.0)
>> > > > >> > > > (2,0.0)
>> > > > >> > > > (3,1.0)
>> > > > >> > > > (4,0.0)
>> > > > >> > > >
>> > > > >> > > > scala> val y = new
>> > > > >> > org.apache.mahout.math.SequentialAccessSparseVector(5)
>> > > > >> > > > y: org.apache.mahout.math.SequentialAccessSparseVector = {
>> > > > >> > > >
>> > > > >> > > > scala> y.set(3, 1.0)
>> > > > >> > > >
>> > > > >> > > > scala> for (item <- y.iterator.asScala)
>> println((item.index,
>> > > > >> item.get))
>> > > > >> > > > (0,0.0)
>> > > > >> > > > (1,0.0)
>> > > > >> > > > (2,0.0)
>> > > > >> > > > (3,1.0)
>> > > > >> > > >
>> > > > >> > >
>> > > > >> >
>> > > > >>
>> > > > >>
>> > > > >>
>> > > > >> --
>> > > > >>
>> > > > >>   -jake
>> > > > >>
>> > > > >
>> > > > >
>> > > >
>> > >
>> > >
>> > >
>> > > --
>> > >
>> > >   -jake
>> > >
>> >
>>
>>
>>
>> --
>>
>>   -jake
>>
>
>

--047d7b5d4498acc3f604d3468342--