Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tiny fixes in utils: documentation and generator #25

Merged
merged 1 commit into from
Oct 23, 2015
Merged

tiny fixes in utils: documentation and generator #25

merged 1 commit into from
Oct 23, 2015

Conversation

areski
Copy link
Contributor

@areski areski commented Sep 9, 2015

according to tests and online sources format is not faster but much more powerful, and this option looks cleaner too::

def get_generator(self):
    i = 0
    max_val = 10000
    while True:
        yield "%s/%s/%d/%d" % (self.prefix, self.uid, (i // max_val) + 1, (i % max_val) + 1)
        i += 1

@areski
Copy link
Contributor Author

areski commented Oct 23, 2015

Any comments?

gawel added a commit that referenced this pull request Oct 23, 2015
@gawel
Merge pull request #25 from areski/fix-utils
d5abdc0 
tiny fixes in utils: documentation and generator
@gawel gawel merged commit d5abdc0 into gawel:master Oct 23, 2015
@areski
Copy link
Contributor Author

areski commented Oct 23, 2015

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@areski @gawel