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" <[email protected] > (mailto:[email protected])> > 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 <[email protected] > > > (mailto:[email protected]) (mailto: > > [email protected] (mailto:[email protected]))> > > > > > > > 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 <[email protected] > > > > > (mailto:[email protected]) (mailto: > > [email protected] (mailto:[email protected])) (mailto: > > > > [email protected] (mailto:[email protected]))> > > > > > > > > > > > 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 <[email protected] > > > > > > > (mailto:[email protected]) (mailto: > > [email protected] (mailto:[email protected])) (mailto: > > > > [email protected] (mailto:[email protected])) (mailto: > > > > > > [email protected] (mailto:[email protected]))> > > > > > > > > > > > > > > > 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 <[email protected] > > > > > > > > > (mailto:[email protected]) (mailto: > > [email protected] (mailto:[email protected])) (mailto: > > > > [email protected] (mailto:[email protected])) (mailto: > > > > > > [email protected] (mailto:[email protected]) > > > > > > > > )> > > > > > > > > > > > > > > > > > > > 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 < > > > > > > > > > > > [email protected] > > > > > > > > > > > (mailto:[email protected]) (mailto: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [email protected] (mailto:[email protected])) > > (mailto: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [email protected] (mailto:[email protected])) > > (mailto: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [email protected] > > > > > > (mailto:[email protected]) (mailto: > > [email protected] (mailto:[email protected]))) > > > > (mailto: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [email protected] > > > > > > > > (mailto:[email protected]) (mailto: > > [email protected] (mailto:[email protected])) > > (mailto: > > > > [email protected] (mailto:[email protected] > > > > > > > )))> > > > > > > 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 : [email protected] > > > > > > > > > > (mailto:[email protected]) (mailto: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [email protected] (mailto:[email protected]) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ) > > > > > > > > > > http://www.brondsema.net : personal > > > > > > > > > > http://www.splike.com : programming > > > > > > > > > > <>< > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
