You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@allura.apache.org by Tim Van Steenburgh <tv...@gmail.com> on 2013/03/01 16:23:44 UTC
Re: User stats and organizations
Hey guys, I don't see a way to get the Statistics tool installed for an existing user. Am I missing something?
Tim
On Thursday, February 28, 2013 at 4:02 PM, Stefano Invernizzi wrote:
> Hi,
> Thanks for rebasing our branch! I moved user stats within the user project as
> a tool. Let us know if this implementation is ok.
> Regards,
>
> Stefano
>
>
> 2013/2/25 Tim Van Steenburgh <tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com)>
>
> > Cool, I just pushed a rebased copy of si/5453. Will look forward to your
> > next updates!
> >
> > --
> > Tim Van Steenburgh
> >
> >
> >
> > On Monday, February 25, 2013 at 4:27 PM, Stefano Invernizzi wrote:
> >
> > > Hi Tim,
> > >
> > > Yes, we know our branch is out of date. We would be very happy if you can
> > > rebase it, there's no problem about that!
> > > The idea of moving user stats within the user project is also very good.
> > >
> >
> > We
> > > will change and upload it according to your proposal as soon as possible.
> > > Thank you very much!
> > >
> > > Stefano.
> > >
> > >
> > >
> > > 2013/2/25 Tim Van Steenburgh <tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com) (mailto:
> > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com))>
> > >
> > > > Stefano & Simone,
> > > >
> > > > Hey guys, I started doing some more testing on user stats today, and I
> > > > have a couple questions.
> > > >
> > > > 1. Do you mind if I force-push a rebased-to-master copy of si/5453 to
> > > > origin? Your branch is quite out-of-date relative to master, and when I
> > > > rebased it there were several merge conflicts and other cleanup
> > > >
> > >
> > >
> >
> > necessary
> > > > to get the code running. You're welcome to do the rebase yourself of
> > > > course, but if you're not comfortable with that I'd be happy to do it.
> > > >
> > > > 2. I noticed that all the user stats links are rooted at /userstats. I
> > > > think it would be preferable to have the stats tool installed in the
> > > >
> > >
> >
> > user
> > > > projects instead. So, the url would end up looking something like
> > > > "/u/admin1/stats". What do you guys think of that idea?
> > > >
> > > > Tim Van Steenburgh
> > > >
> > > >
> > > > On Saturday, February 23, 2013 at 9:07 AM, Stefano Invernizzi wrote:
> > > >
> > > > > Yes, we fixed the issue in our code, it should be ok now. At the
> > moment,
> > > > > when I try to run all the tests I get some errors from other
> > > >
> > >
> >
> > packages. I
> > > > > tried to fix them, but I don't know how to do that, since they come
> > > >
> > >
> >
> > from
> > > > > modules and packages dealing with repositories, and they are
> > > >
> > >
> >
> > unrelated to
> > > > > our code.
> > > > > Thank you very much!
> > > > > Stefano
> > > > >
> > > > >
> > > > >
> > > > > 2013/2/21 Tim Van Steenburgh <tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com) (mailto:
> > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com)) (mailto:
> > > > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com))>
> > > > >
> > > > > > Hi Simone and Stefano,
> > > > > >
> > > > > > I pulled your si/5453 branch today to do some more testing. I
> > rebased
> > > > it
> > > > > > against master and kicked off the tests (`./run_tests` in the
> > > > >
> > > >
> > > >
> > >
> >
> > project
> > > > >
> > > >
> > > >
> > > > root)
> > > > > > and got some failures. If you guys can get those fixed up and
> > > > >
> > > >
> > > >
> > >
> >
> > pushed
> > > > >
> > > >
> > > >
> > > > back,
> > > > > > I'll try to wrap up the testing.
> > > > > >
> > > > > > Thanks!
> > > > > >
> > > > > > Tim Van Steenburgh
> > > > > >
> > > > > >
> > > > > > On Monday, February 18, 2013 at 2:26 PM, Simone Gatti wrote:
> > > > > >
> > > > > > > Dear Cory,
> > > > > > > what Dave said is correct, we refer to user stats.
> > > > > > > I'm sorry for this ambiguity, that we didn't noticed before your
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > comment.
> > > > > > > We welcome your *feedback on user stats*, since we have not yet
> > > > > >
> > > > >
> > > >
> > > >
> > > > received
> > > > > > > many comments yet.
> > > > > > >
> > > > > > > Simone
> > > > > > >
> > > > > > >
> > > > > > > 2013/2/18 Dave Brondsema <dave@brondsema.net (mailto:dave@brondsema.net) (mailto:
> > dave@brondsema.net (mailto:dave@brondsema.net)) (mailto:
> > > > dave@brondsema.net (mailto:dave@brondsema.net)
> > > > > > )>
> > > > > > >
> > > > > > > > Cory, I think Stefano is referring to user stats, which his
> > feature
> > > > > > branch
> > > > > > > > starts collecting, not user profile data (gender, location,
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > etc).
> > > > > > > >
> > > > > > > > But on the topic of user profiel data, we've had a least one
> > > > > > SourceForge
> > > > > > > > user
> > > > > > > > communicate to use that he/she thought the fields were
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > required. I
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > can
> > > > > > > > see how
> > > > > > > > this might be inferred since they're the first thing you see
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > on the
> > > > > > > > /auth/prefs/
> > > > > > > > form. We might consider labelling those optional, or putting
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > them
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > on a
> > > > > > > > separate
> > > > > > > > page from "subscriptions" and other sections on that page.
> > > > > > > >
> > > > > > > > -Dave
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > On 2/18/13 9:55 AM, Cory Johns wrote:
> > > > > > > > > Stefano,
> > > > > > > > >
> > > > > > > > > Could a user simply not fill in the personal info fields they
> > > > don't
> > > > > > wish
> > > > > > > > to
> > > > > > > > > share? What is the value of entering that info but then not
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > displaying
> > > > > > > >
> > > > > > > > it;
> > > > > > > > > to encourage users to enter it if only for our edification?
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > Thanks,
> > > > > > > > >
> > > > > > > > > Cory
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > On Sat, Feb 16, 2013 at 9:53 AM, Stefano Invernizzi <
> > > > > > > > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com) (mailto:
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com)) (mailto:
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com))
> > (mailto:
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com) (mailto:
> > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com)))>
> > > > wrote:
> > > > > > > > >
> > > > > > > > > > Dear all,
> > > > > > > > > >
> > > > > > > > > > I recently pushed some new commits allowing a single user
> > to
> > > > hide
> > > > > > his or
> > > > > > > > > > her personal statistics. I and Simone implemented it since
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > some
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > users
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > may
> > > > > > > > > > prefer not to show this data. In that case, data is still
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > available for
> > > > > > > > > > their personal use. However, if you think we should not
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > allow
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > users to
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > do
> > > > > > > > > > this, we can simply put it back as it was.
> > > > > > > > > > As usual, we hope to get some feedbacks from you about
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > this, as
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > well as
> > > > > > > > > > about the rest of submitted code.
> > > > > > > > > > It would be great for us if the code could be reviewed
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > and, if
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > you
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > think it
> > > > > > > > > > would be useful, included on the forge before we complete
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > the
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > thesis we
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > are
> > > > > > > > > > working on.
> > > > > > > > > > Thank you very much!
> > > > > > > > > > Stefano
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > --
> > > > > > > > Dave Brondsema : dave@brondsema.net (mailto:dave@brondsema.net
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > )
> > > > > > > > http://www.brondsema.net : personal
> > > > > > > > http://www.splike.com : programming
> > > > > > > > <><
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> >
>
>
>
Re: User stats and organizations
Posted by Dave Brondsema <da...@brondsema.net>.
On 3/1/13 12:36 PM, Tim Van Steenburgh wrote:
> I think there are 3 options:
>
> 1. Leave as-is, and write a script to install the tool on existing user projects.
> 2. Make the stats tool installable, but only for user projects.
This isn't an option now, but we do have a ticket for it, it'd be nice to have
https://sourceforge.net/p/allura/tickets/2286/
> 3. Install the stats tool on demand when necessary.
This should be doable by putting it in the "Anchored tools" feature. It's a per
neighborhood setting, so it could be added just for the user-neighborhood:
/u/_admin/overview Then it'll get installed on demand when a user-project is
visited.
>
> I think 1 and 3 are the best choices, and I'm leaning towards 1 b/c it'll keep the code cleaner. Let's wait and see if any one else chimes in with an opinion, and if not, we'll just go with that. In the meantime, I'll just continue testing with a new user instead of an existing one.
>
> Thanks!
>
> Tim
>
>
> On Friday, March 1, 2013 at 12:19 PM, Stefano Invernizzi wrote:
>
>> Yes, that's a good point. At the moment, the tool is automatically
>> installed when a new user is created, but it can't be manually installed,
>> therefore this is a problem we should solve. By changing the installable
>> flag, the problem is solved, but the tool can be plugged within other
>> projects too. I was wondering if it's possible to make a tool installable
>> within user projects only.
>>
>> Regards,
>> Stefano
>> Il giorno 01/mar/2013 16:24, "Tim Van Steenburgh" <tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com)>
>> ha scritto:
>>
>>> Hey guys, I don't see a way to get the Statistics tool installed for an
>>> existing user. Am I missing something?
>>>
>>> Tim
>>>
>>>
--
Dave Brondsema : dave@brondsema.net
http://www.brondsema.net : personal
http://www.splike.com : programming
<><
Re: User stats and organizations
Posted by Tim Van Steenburgh <tv...@gmail.com>.
I think there are 3 options:
1. Leave as-is, and write a script to install the tool on existing user projects.
2. Make the stats tool installable, but only for user projects.
3. Install the stats tool on demand when necessary.
I think 1 and 3 are the best choices, and I'm leaning towards 1 b/c it'll keep the code cleaner. Let's wait and see if any one else chimes in with an opinion, and if not, we'll just go with that. In the meantime, I'll just continue testing with a new user instead of an existing one.
Thanks!
Tim
On Friday, March 1, 2013 at 12:19 PM, Stefano Invernizzi wrote:
> Yes, that's a good point. At the moment, the tool is automatically
> installed when a new user is created, but it can't be manually installed,
> therefore this is a problem we should solve. By changing the installable
> flag, the problem is solved, but the tool can be plugged within other
> projects too. I was wondering if it's possible to make a tool installable
> within user projects only.
>
> Regards,
> Stefano
> Il giorno 01/mar/2013 16:24, "Tim Van Steenburgh" <tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com)>
> ha scritto:
>
> > Hey guys, I don't see a way to get the Statistics tool installed for an
> > existing user. Am I missing something?
> >
> > Tim
> >
> >
> > On Thursday, February 28, 2013 at 4:02 PM, Stefano Invernizzi wrote:
> >
> > > Hi,
> > > Thanks for rebasing our branch! I moved user stats within the user
> > >
> >
> > project as
> > > a tool. Let us know if this implementation is ok.
> > > Regards,
> > >
> > > Stefano
> > >
> > >
> > > 2013/2/25 Tim Van Steenburgh <tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com) (mailto:
> > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com))>
> > >
> > > > Cool, I just pushed a rebased copy of si/5453. Will look forward to
> > your
> > > > next updates!
> > > >
> > > > --
> > > > Tim Van Steenburgh
> > > >
> > > >
> > > >
> > > > On Monday, February 25, 2013 at 4:27 PM, Stefano Invernizzi wrote:
> > > >
> > > > > Hi Tim,
> > > > >
> > > > > Yes, we know our branch is out of date. We would be very happy if
> > you can
> > > > > rebase it, there's no problem about that!
> > > > > The idea of moving user stats within the user project is also very
> > > > >
> > > >
> > >
> >
> > good.
> > > > >
> > > >
> > > >
> > > > We
> > > > > will change and upload it according to your proposal as soon as
> > > >
> > > >
> > >
> >
> > possible.
> > > > > Thank you very much!
> > > > >
> > > > > Stefano.
> > > > >
> > > > >
> > > > >
> > > > > 2013/2/25 Tim Van Steenburgh <tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com) (mailto:
> > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com)) (mailto:
> > > > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com))>
> > > > >
> > > > > > Stefano & Simone,
> > > > > >
> > > > > > Hey guys, I started doing some more testing on user stats today,
> > and I
> > > > > > have a couple questions.
> > > > > >
> > > > > > 1. Do you mind if I force-push a rebased-to-master copy of si/5453
> > to
> > > > > > origin? Your branch is quite out-of-date relative to master, and
> > > > >
> > > >
> > >
> >
> > when I
> > > > > > rebased it there were several merge conflicts and other cleanup
> > > > >
> > > >
> > > >
> > > > necessary
> > > > > > to get the code running. You're welcome to do the rebase yourself
> > > > >
> > > >
> > > >
> > >
> >
> > of
> > > > > > course, but if you're not comfortable with that I'd be happy to do
> > > > >
> > > >
> > >
> >
> > it.
> > > > > >
> > > > > > 2. I noticed that all the user stats links are rooted at
> > /userstats. I
> > > > > > think it would be preferable to have the stats tool installed in
> > > > >
> > > >
> > >
> >
> > the
> > > > > >
> > > > >
> > > >
> > > >
> > > > user
> > > > > > projects instead. So, the url would end up looking something like
> > > > > > "/u/admin1/stats". What do you guys think of that idea?
> > > > > >
> > > > > > Tim Van Steenburgh
> > > > > >
> > > > > >
> > > > > > On Saturday, February 23, 2013 at 9:07 AM, Stefano Invernizzi
> > wrote:
> > > > > >
> > > > > > > Yes, we fixed the issue in our code, it should be ok now. At the
> > > > moment,
> > > > > > > when I try to run all the tests I get some errors from other
> > > > > >
> > > > >
> > > >
> > > >
> > > > packages. I
> > > > > > > tried to fix them, but I don't know how to do that, since they
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > come
> > > > > >
> > > > >
> > > >
> > > >
> > > > from
> > > > > > > modules and packages dealing with repositories, and they are
> > > > > >
> > > > >
> > > >
> > > >
> > > > unrelated to
> > > > > > > our code.
> > > > > > > Thank you very much!
> > > > > > > Stefano
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > 2013/2/21 Tim Van Steenburgh <tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com) (mailto:
> > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com)) (mailto:
> > > > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com)) (mailto:
> > > > > > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com))>
> > > > > > >
> > > > > > > > Hi Simone and Stefano,
> > > > > > > >
> > > > > > > > I pulled your si/5453 branch today to do some more testing. I
> > > > rebased
> > > > > > it
> > > > > > > > against master and kicked off the tests (`./run_tests` in the
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > project
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > root)
> > > > > > > > and got some failures. If you guys can get those fixed up and
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > pushed
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > back,
> > > > > > > > I'll try to wrap up the testing.
> > > > > > > >
> > > > > > > > Thanks!
> > > > > > > >
> > > > > > > > Tim Van Steenburgh
> > > > > > > >
> > > > > > > >
> > > > > > > > On Monday, February 18, 2013 at 2:26 PM, Simone Gatti wrote:
> > > > > > > >
> > > > > > > > > Dear Cory,
> > > > > > > > > what Dave said is correct, we refer to user stats.
> > > > > > > > > I'm sorry for this ambiguity, that we didn't noticed before
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > your
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > comment.
> > > > > > > > > We welcome your *feedback on user stats*, since we have not
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > yet
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > received
> > > > > > > > > many comments yet.
> > > > > > > > >
> > > > > > > > > Simone
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > 2013/2/18 Dave Brondsema <dave@brondsema.net (mailto:dave@brondsema.net) (mailto:
> > dave@brondsema.net (mailto:dave@brondsema.net)) (mailto:
> > > > dave@brondsema.net (mailto:dave@brondsema.net)) (mailto:
> > > > > > dave@brondsema.net (mailto:dave@brondsema.net)
> > > > > > > > )>
> > > > > > > > >
> > > > > > > > > > Cory, I think Stefano is referring to user stats, which his
> > > > feature
> > > > > > > > branch
> > > > > > > > > > starts collecting, not user profile data (gender, location,
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > etc).
> > > > > > > > > >
> > > > > > > > > > But on the topic of user profiel data, we've had a least
> > one
> > > > > > > > SourceForge
> > > > > > > > > > user
> > > > > > > > > > communicate to use that he/she thought the fields were
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > required. I
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > can
> > > > > > > > > > see how
> > > > > > > > > > this might be inferred since they're the first thing you
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > see
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > on the
> > > > > > > > > > /auth/prefs/
> > > > > > > > > > form. We might consider labelling those optional, or
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > putting
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > them
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > on a
> > > > > > > > > > separate
> > > > > > > > > > page from "subscriptions" and other sections on that page.
> > > > > > > > > >
> > > > > > > > > > -Dave
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > On 2/18/13 9:55 AM, Cory Johns wrote:
> > > > > > > > > > > Stefano,
> > > > > > > > > > >
> > > > > > > > > > > Could a user simply not fill in the personal info fields
> > they
> > > > > > don't
> > > > > > > > wish
> > > > > > > > > > to
> > > > > > > > > > > share? What is the value of entering that info but then
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > not
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > displaying
> > > > > > > > > >
> > > > > > > > > > it;
> > > > > > > > > > > to encourage users to enter it if only for our
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > edification?
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > Thanks,
> > > > > > > > > > >
> > > > > > > > > > > Cory
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > On Sat, Feb 16, 2013 at 9:53 AM, Stefano Invernizzi <
> > > > > > > > > > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com) (mailto:
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com)) (mailto:
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com))
> > (mailto:
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com) (mailto:
> > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com)))
> > > > (mailto:
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com) (mailto:
> > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com)) (mailto:
> > > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com
> > >
> >
> > )))>
> > > > > > wrote:
> > > > > > > > > > >
> > > > > > > > > > > > Dear all,
> > > > > > > > > > > >
> > > > > > > > > > > > I recently pushed some new commits allowing a single
> > user
> > > > to
> > > > > > hide
> > > > > > > > his or
> > > > > > > > > > > > her personal statistics. I and Simone implemented it
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > since
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > some
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > users
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > may
> > > > > > > > > > > > prefer not to show this data. In that case, data is
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > still
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > available for
> > > > > > > > > > > > their personal use. However, if you think we should not
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > allow
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > users to
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > do
> > > > > > > > > > > > this, we can simply put it back as it was.
> > > > > > > > > > > > As usual, we hope to get some feedbacks from you about
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > this, as
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > well as
> > > > > > > > > > > > about the rest of submitted code.
> > > > > > > > > > > > It would be great for us if the code could be reviewed
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > and, if
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > you
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > think it
> > > > > > > > > > > > would be useful, included on the forge before we
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > complete
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > the
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > thesis we
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > are
> > > > > > > > > > > > working on.
> > > > > > > > > > > > Thank you very much!
> > > > > > > > > > > > Stefano
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > --
> > > > > > > > > > Dave Brondsema : dave@brondsema.net (mailto:dave@brondsema.net) (mailto:
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> > dave@brondsema.net (mailto:dave@brondsema.net)
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > )
> > > > > > > > > > http://www.brondsema.net : personal
> > > > > > > > > > http://www.splike.com : programming
> > > > > > > > > > <><
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > >
> >
> >
>
>
>
Re: User stats and organizations
Posted by Stefano Invernizzi <st...@gmail.com>.
Yes, that's a good point. At the moment, the tool is automatically
installed when a new user is created, but it can't be manually installed,
therefore this is a problem we should solve. By changing the installable
flag, the problem is solved, but the tool can be plugged within other
projects too. I was wondering if it's possible to make a tool installable
within user projects only.
Regards,
Stefano
Il giorno 01/mar/2013 16:24, "Tim Van Steenburgh" <tv...@gmail.com>
ha scritto:
> Hey guys, I don't see a way to get the Statistics tool installed for an
> existing user. Am I missing something?
>
> Tim
>
>
> On Thursday, February 28, 2013 at 4:02 PM, Stefano Invernizzi wrote:
>
> > Hi,
> > Thanks for rebasing our branch! I moved user stats within the user
> project as
> > a tool. Let us know if this implementation is ok.
> > Regards,
> >
> > Stefano
> >
> >
> > 2013/2/25 Tim Van Steenburgh <tvansteenburgh@gmail.com (mailto:
> tvansteenburgh@gmail.com)>
> >
> > > Cool, I just pushed a rebased copy of si/5453. Will look forward to
> your
> > > next updates!
> > >
> > > --
> > > Tim Van Steenburgh
> > >
> > >
> > >
> > > On Monday, February 25, 2013 at 4:27 PM, Stefano Invernizzi wrote:
> > >
> > > > Hi Tim,
> > > >
> > > > Yes, we know our branch is out of date. We would be very happy if
> you can
> > > > rebase it, there's no problem about that!
> > > > The idea of moving user stats within the user project is also very
> good.
> > > >
> > >
> > > We
> > > > will change and upload it according to your proposal as soon as
> possible.
> > > > Thank you very much!
> > > >
> > > > Stefano.
> > > >
> > > >
> > > >
> > > > 2013/2/25 Tim Van Steenburgh <tvansteenburgh@gmail.com (mailto:
> tvansteenburgh@gmail.com) (mailto:
> > > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com))>
> > > >
> > > > > Stefano & Simone,
> > > > >
> > > > > Hey guys, I started doing some more testing on user stats today,
> and I
> > > > > have a couple questions.
> > > > >
> > > > > 1. Do you mind if I force-push a rebased-to-master copy of si/5453
> to
> > > > > origin? Your branch is quite out-of-date relative to master, and
> when I
> > > > > rebased it there were several merge conflicts and other cleanup
> > > > >
> > > >
> > > >
> > >
> > > necessary
> > > > > to get the code running. You're welcome to do the rebase yourself
> of
> > > > > course, but if you're not comfortable with that I'd be happy to do
> it.
> > > > >
> > > > > 2. I noticed that all the user stats links are rooted at
> /userstats. I
> > > > > think it would be preferable to have the stats tool installed in
> the
> > > > >
> > > >
> > >
> > > user
> > > > > projects instead. So, the url would end up looking something like
> > > > > "/u/admin1/stats". What do you guys think of that idea?
> > > > >
> > > > > Tim Van Steenburgh
> > > > >
> > > > >
> > > > > On Saturday, February 23, 2013 at 9:07 AM, Stefano Invernizzi
> wrote:
> > > > >
> > > > > > Yes, we fixed the issue in our code, it should be ok now. At the
> > > moment,
> > > > > > when I try to run all the tests I get some errors from other
> > > > >
> > > >
> > >
> > > packages. I
> > > > > > tried to fix them, but I don't know how to do that, since they
> come
> > > > >
> > > >
> > >
> > > from
> > > > > > modules and packages dealing with repositories, and they are
> > > > >
> > > >
> > >
> > > unrelated to
> > > > > > our code.
> > > > > > Thank you very much!
> > > > > > Stefano
> > > > > >
> > > > > >
> > > > > >
> > > > > > 2013/2/21 Tim Van Steenburgh <tvansteenburgh@gmail.com (mailto:
> tvansteenburgh@gmail.com) (mailto:
> > > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com)) (mailto:
> > > > > tvansteenburgh@gmail.com (mailto:tvansteenburgh@gmail.com))>
> > > > > >
> > > > > > > Hi Simone and Stefano,
> > > > > > >
> > > > > > > I pulled your si/5453 branch today to do some more testing. I
> > > rebased
> > > > > it
> > > > > > > against master and kicked off the tests (`./run_tests` in the
> > > > > >
> > > > >
> > > > >
> > > >
> > >
> > > project
> > > > > >
> > > > >
> > > > >
> > > > > root)
> > > > > > > and got some failures. If you guys can get those fixed up and
> > > > > >
> > > > >
> > > > >
> > > >
> > >
> > > pushed
> > > > > >
> > > > >
> > > > >
> > > > > back,
> > > > > > > I'll try to wrap up the testing.
> > > > > > >
> > > > > > > Thanks!
> > > > > > >
> > > > > > > Tim Van Steenburgh
> > > > > > >
> > > > > > >
> > > > > > > On Monday, February 18, 2013 at 2:26 PM, Simone Gatti wrote:
> > > > > > >
> > > > > > > > Dear Cory,
> > > > > > > > what Dave said is correct, we refer to user stats.
> > > > > > > > I'm sorry for this ambiguity, that we didn't noticed before
> your
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > comment.
> > > > > > > > We welcome your *feedback on user stats*, since we have not
> yet
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > received
> > > > > > > > many comments yet.
> > > > > > > >
> > > > > > > > Simone
> > > > > > > >
> > > > > > > >
> > > > > > > > 2013/2/18 Dave Brondsema <dave@brondsema.net (mailto:
> dave@brondsema.net) (mailto:
> > > dave@brondsema.net (mailto:dave@brondsema.net)) (mailto:
> > > > > dave@brondsema.net (mailto:dave@brondsema.net)
> > > > > > > )>
> > > > > > > >
> > > > > > > > > Cory, I think Stefano is referring to user stats, which his
> > > feature
> > > > > > > branch
> > > > > > > > > starts collecting, not user profile data (gender, location,
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > etc).
> > > > > > > > >
> > > > > > > > > But on the topic of user profiel data, we've had a least
> one
> > > > > > > SourceForge
> > > > > > > > > user
> > > > > > > > > communicate to use that he/she thought the fields were
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > required. I
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > can
> > > > > > > > > see how
> > > > > > > > > this might be inferred since they're the first thing you
> see
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > >
> > >
> > > on the
> > > > > > > > > /auth/prefs/
> > > > > > > > > form. We might consider labelling those optional, or
> putting
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > them
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > on a
> > > > > > > > > separate
> > > > > > > > > page from "subscriptions" and other sections on that page.
> > > > > > > > >
> > > > > > > > > -Dave
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > On 2/18/13 9:55 AM, Cory Johns wrote:
> > > > > > > > > > Stefano,
> > > > > > > > > >
> > > > > > > > > > Could a user simply not fill in the personal info fields
> they
> > > > > don't
> > > > > > > wish
> > > > > > > > > to
> > > > > > > > > > share? What is the value of entering that info but then
> not
> > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > displaying
> > > > > > > > >
> > > > > > > > > it;
> > > > > > > > > > to encourage users to enter it if only for our
> edification?
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > Thanks,
> > > > > > > > > >
> > > > > > > > > > Cory
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > On Sat, Feb 16, 2013 at 9:53 AM, Stefano Invernizzi <
> > > > > > > > > > stefano.invernizzi88@gmail.com (mailto:
> stefano.invernizzi88@gmail.com) (mailto:
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > >
> > >
> > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com))
> (mailto:
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > stefano.invernizzi88@gmail.com (mailto:
> stefano.invernizzi88@gmail.com))
> > > (mailto:
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > stefano.invernizzi88@gmail.com (mailto:
> stefano.invernizzi88@gmail.com) (mailto:
> > > stefano.invernizzi88@gmail.com (mailto:stefano.invernizzi88@gmail.com
> )))>
> > > > > wrote:
> > > > > > > > > >
> > > > > > > > > > > Dear all,
> > > > > > > > > > >
> > > > > > > > > > > I recently pushed some new commits allowing a single
> user
> > > to
> > > > > hide
> > > > > > > his or
> > > > > > > > > > > her personal statistics. I and Simone implemented it
> since
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > >
> > >
> > > some
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > users
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > may
> > > > > > > > > > > prefer not to show this data. In that case, data is
> still
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > available for
> > > > > > > > > > > their personal use. However, if you think we should not
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > allow
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > users to
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > do
> > > > > > > > > > > this, we can simply put it back as it was.
> > > > > > > > > > > As usual, we hope to get some feedbacks from you about
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > this, as
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > well as
> > > > > > > > > > > about the rest of submitted code.
> > > > > > > > > > > It would be great for us if the code could be reviewed
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > and, if
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > you
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > think it
> > > > > > > > > > > would be useful, included on the forge before we
> complete
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > >
> > >
> > > the
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > thesis we
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > are
> > > > > > > > > > > working on.
> > > > > > > > > > > Thank you very much!
> > > > > > > > > > > Stefano
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > --
> > > > > > > > > Dave Brondsema : dave@brondsema.net (mailto:
> dave@brondsema.net
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > )
> > > > > > > > > http://www.brondsema.net : personal
> > > > > > > > > http://www.splike.com : programming
> > > > > > > > > <><
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > >
> >
> >
> >
>
>
>