search

ejwa / gitinspector

:bar_chart: The statistical analysis tool for git repositories
GNU General Public License v3.0
2.36k stars 328 forks source link

Inaccurate output #75

Closed ibm-ecm closed 8 years ago

ibm-ecm commented 8 years ago

It is entirely possible, I don't understand how stats are calculated. When I run 'git shortlog -sne', the number of commits vary WILDLY from when I run gitinspector (I am not specifying any time interval, trying to get stats for the entire lifetime of the branch). For the same user, git shortlog shows 317 commits while gitinspector shows 3. What am I missing?

adam-waldenberg commented 8 years ago

Hi.

Gitinspector also tries to merge authors independently from .mailmap. If you, for example, use different auhor names. between differen commits.

So:

Kalle kalle@kalle.com

is the same as:

kalle@kalle.com
ibm-ecm commented 8 years ago

ok I made sure there are no duplicate personas.

adam-waldenberg commented 8 years ago

Going ahead and marking as invalid. Please notify if there is an update on this issue.

ibm-ecm commented 8 years ago

Adam, why did you close as invalid? What info can I provide to help troubleshoot? As I said the git shortlog and gitinspector are wildly different.

adam-waldenberg commented 8 years ago

I'm fairly certain there's nothing wrong here. What happens if you do something like

gitinspector.py -f "**"

and compare that to the shortlog ?

Gitinspector filters out commits that are not in the list of defined file types that should be scanned. -f "**" will include them all (excluding binary commits). You can also pass "-l" to see all the file types detected by gitinspector.

Also; commits that are pure moves are never counted, as they don't consist of any code.

adam-waldenberg commented 8 years ago

Checking in to see if there's any news on this ? Did my suggestions help ?

ibm-ecm commented 8 years ago

sorry for the delay, I am traveling this week. Hope to try it out next week :-)

adam-waldenberg commented 8 years ago

Added a new entry to the FAQ concerning this. Hope it helps.

adam-waldenberg commented 8 years ago

As there has been no update for a while, I'm closing this as invalid.