spgroup / groundhog

A framework for crawling GitHub projects and raw data and to extract metrics from them
http://spgroup.github.io/groundhog
GNU General Public License v2.0
15 stars 10 forks source link

To provide an answer to the question "How many projects use more than one programming language?" #45

Closed fernandocastor closed 11 years ago

fernandocastor commented 11 years ago

We need to implement and test the features required to use Groundhog to answer the question in the title of the issue. We then have to use it to actually answer the question.

jesusjackson commented 11 years ago

@gustavopinto @rodrigoalvesvieira can i use the ft-fetch-languages branch to solve this or you already closed this branch?

gustavopinto commented 11 years ago

I think it is still open. Go ahead! :-)

jesusjackson commented 11 years ago

@gustavopinto, thanks and one more thing, you commited to the master, is there a way to get clone your code the branch?

gustavopinto commented 11 years ago

Yes. Just go to the branch you want, and use git merge.

jesusjackson commented 11 years ago

Thanks!

jesusjackson commented 11 years ago

I'm having a real problem with this limit that we have. I have to get the repos, then get the projects, get the owner and finally get the languages to solve this question from the issue. Alone it consumes a lot of my hourly data limit. The solution is ready and working.

jesusjackson commented 11 years ago

Sorry, closed accidentally.

gustavopinto commented 11 years ago

@pork9 are you using authenticated requests?

rodrigoalvesvieira commented 11 years ago

You either performed some unnecessary, extra requests there or there's a problem with the tool. Because the project metadata (example) provides the user name and so there's absolutely no need to perform the request to a User resource in order to fetch the project's languages.

jesusjackson commented 11 years ago

@rodrigoalvesvieira ok, i'm not using User anymore and it's working. I'm going to push soon. @gustavopinto now i'm using it.

rodrigoalvesvieira commented 11 years ago

Great!

jesusjackson commented 11 years ago

Only problem is i couldn't merge the branch i intended to, is there a problem to push to master?

rodrigoalvesvieira commented 11 years ago

Is the code tested? If so, then I guess there's no problem. But preferably you'd push to a separate branch. What exactly is the problem?

jesusjackson commented 11 years ago

It's tested. The only real problem is the amount of requests because the amount of projects it's really huge.

jesusjackson commented 11 years ago

Now it's done. @fernandocastor Can you check it? So i can close and start doing something new?

jesusjackson commented 11 years ago

@rodrigoalvesvieira i found problem with your implemantation of fetchProjectLanguages. It seems that when the api data limit is reached you are not checking if you got the message. So you try to get a value that doesn't exist. Should i fix and push the solution or you want to check it out first?

rodrigoalvesvieira commented 11 years ago

Go ahead and fix it! :+1:

jesusjackson commented 11 years ago

@gustavopinto can i close this issue?

rodrigoalvesvieira commented 11 years ago

Go ahead! This was great!