Tuple is good for now.

On Saturday, June 29, 2013, Eugene Fang wrote:

> In datafeeds/datafeed_usfirst.py:
>
> > @@ -146,8 +110,18 @@ def getEventTeams(self, year, first_eid):
> >          Returns a list of team_numbers attending a particular Event
> >          """
> >          if type(year) is not int: raise TypeError("year must be an integer")
> > -        url = self.EVENT_TEAMS_URL_PATTERN % (first_eid, self.getSessionKey(year))
> > -        teams = self.parse(url, UsfirstEventTeamsParser)
> > +
> > +        teams = []
> > +        page = 0
> > +        while True:
> > +          url = self.EVENT_TEAMS_URL_PATTERN % (first_eid)
> > +          if page != 0:
> > +            url += '&page=%s' % page
> > +          partial_teams, last_page = self.parse(url, UsfirstEventTeamsParser)
>
> What if we update all of them to return the tuple:
> (models, more_pages)
>
> Or do you prefer the dict?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gregmarra/the-blue-alliance/pull/658/files#r4953315>
> .
>