git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 2/2] gitweb: Show trailing slash when listing tree entry in tree listing

From
LTLuben Tuikov <ltuikov@yahoo.com>
Date
Oct 10, 2006, 06:21 UTC
Message-ID
<20061010062126.46664.qmail@web31810.mail.mud.yahoo.com>
In-Reply-To
<20061009205551.GO20017@pasky.or.cz>
--- Petr Baudis <pasky@suse.cz> wrote:
Show 25 quoted lines
> I was looking into accesslogs of repo.or.cz for something and noticed
> that I see unusually large number of blame requests. That of course
> attracted my curiosity and I came to conclusion that what I'm seeing is
> not just my personal whim but we have serious usability problem here.
> 
> I'm unfortunately not sure when the update of repo.or.cz gitweb which
> dropped the blob/tree links happenned, so the following _is_ somewhat
> dubious, but I think it's quite telling anyway.
> 
> I have three samples (logfiles) available: #2 almost certainly when the
> blob link was still there, #1 covering the switch and some time before
> and after, and #0 certainly when the blob link was not there anymore,
> but unfortunately spanning only one or two days.
> 
> This is the count of actions invoked from the tree, commit and
> commitdiff view (using the referer information):
> 
>     blame  blob   total requests containing 'a='
> #2  1      18     264
> #1  31     23     399
> #0  4      6      50
> 
> The disparation between #2 and #1,#0 is quite apparent. If we want more
> exact results, I will let #0 accumulate data for a week and then revert
> the removal of the links and start another sample.
Oh, my, oh, my.

Anyone can come up with any "statistic" to convince anyone of anything. It's the American way! (to financial success)

I mean, I can even give you a mathematical series which I can add in certain orders to give me any number I want... Which is not at all intutitive.

Anyway, the "confused" link clearly says "blame". I'm not sure why your people were trying to think and figure it out, as opposed to simply clicking on the file name itself. It is the most intuitive thing to do as I mentioned in my previous email.

Did you do any demographics on your clickers? What is their background? Did you try to calculate how statistically correct your sample is and if the clickers represent the general computer population out there? What is your sampling error?

I can hardly accept this "statistic" as a proof to "reintroduce the redundant links".

But I give up.

If you guys want the redundant links back in, so be it -- submit a patch.

Then let's fortify gitweb, because we can. Lets add links and more redundancy to fortify the user interface so that all and any possibilities are covered. And as soon as a NEW git facility is introduced, then we'll add 10 or 20 more links to gitweb for this just one, single new facility. Then with each new single facility if it is related to any other, the number of links would grow exponentially. Then after such and such time has passed, let's look at the code. Then let's ask if someone has left maintaining the then gitweb jungle. More importantly, let's ask if someone has left _using_ it. (That would be the ripe time to start afresh with gitweb2.perl.)

As long as job is being done and the patches are flowing in, and more and more code is introduced, albeit redundantly, we shouldn't care what the people who use this every day want or care for.

Hey, let's add more links!
     Luben
P.S. I'll go and collect some statistics now.
Previous: Andreas EricssonNext: Jeff King
Message 44 of 54 in “gitweb: Do not print "log" and "shortlog" redundantly in commit view”
  1. gitweb: Do not print "log" and "shortlog" redundantly in commit viewLuben Tuikov, Oct 5, 2006
  2. Jakub NarebskiOct 6, 2006
  3. Petr BaudisOct 6, 2006
  4. Luben TuikovOct 6, 2006
  5. Petr BaudisOct 7, 2006
  6. Jakub NarebskiOct 7, 2006
  7. Petr BaudisOct 7, 2006
  8. 1/2 gitweb: Show snapshot links for tree entries in tree listingPetr Baudis, Oct 7, 2006
  9. Luben TuikovOct 7, 2006
  10. Petr BaudisOct 7, 2006
  11. Luben TuikovOct 7, 2006
  12. Petr BaudisOct 7, 2006
  13. Junio C HamanoOct 7, 2006
  14. Luben TuikovOct 8, 2006
  15. 2/2 gitweb: Show trailing slash when listing tree entry in tree listingPetr Baudis, Oct 7, 2006
  16. Luben TuikovOct 7, 2006
  17. Jakub NarebskiOct 7, 2006
  18. Petr BaudisOct 7, 2006
  19. Jakub NarebskiOct 7, 2006
  20. Petr BaudisOct 7, 2006
  21. Jakub NarebskiOct 7, 2006
  22. Petr BaudisOct 9, 2006
  23. Junio C HamanoOct 9, 2006
  24. Junio C HamanoOct 10, 2006
  25. Luben TuikovOct 10, 2006
  26. Junio C HamanoOct 10, 2006
  27. Luben TuikovOct 10, 2006
  28. Jeff KingOct 10, 2006
  29. Luben TuikovOct 10, 2006
  30. Jeff KingOct 10, 2006
  31. Jakub NarebskiOct 10, 2006
  32. Luben TuikovOct 10, 2006
  33. Junio C HamanoOct 10, 2006
  34. Jakub NarebskiOct 10, 2006
  35. Luben TuikovOct 10, 2006
  36. Jakub NarebskiOct 10, 2006
  37. Luben TuikovOct 10, 2006
  38. Luben TuikovOct 10, 2006
  39. Jakub NarebskiOct 10, 2006
  40. Luben TuikovOct 10, 2006
  41. Jakub NarebskiOct 10, 2006
  42. Jakub NarebskiOct 10, 2006
  43. Andreas EricssonOct 11, 2006
  44. Luben TuikovOct 10, 2006
  45. Jeff KingOct 10, 2006
  46. Andreas EricssonOct 10, 2006
  47. Josef WeidendorferOct 10, 2006
  48. Luben TuikovOct 10, 2006
  49. Josef WeidendorferOct 10, 2006
  50. Luben TuikovOct 10, 2006
  51. Junio C HamanoOct 10, 2006
  52. A Large Angry SCMOct 7, 2006
  53. Junio C HamanoOct 7, 2006
  54. Luben TuikovOct 8, 2006

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.