Skip to content

Dti toperiod tz #2390

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.

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

Closed
wants to merge 2 commits into from
Closed

Dti toperiod tz #2390

wants to merge 2 commits into from

Conversation

changhiskhan
Copy link
Contributor

No description provided.

@ghost ghost assigned wesm Nov 29, 2012
@wesm
Copy link
Member

wesm commented Nov 29, 2012

Sweet, thanks. I'll get on the Cython refactor today

@changhiskhan
Copy link
Contributor Author

hmm…failing on travis for 2x but works for 3x, and also works on 2.7.3 locally…

@ghost
Copy link

ghost commented Nov 29, 2012

#2380?

@changhiskhan
Copy link
Contributor Author

Not sure why that would work for me locally then.

@ghost
Copy link

ghost commented Nov 29, 2012

I had the same issue, there's a random.seed() call in the code, which when combined with
the travis image makes randint deterministic, and it just so happens that the seed triggers
the problem on travis everytime. that's my take on it, anyway.
I was able to reproduce intermittently locally, but it was somewhat rare.

@changhiskhan
Copy link
Contributor Author

got it. thanks.

@wesm wesm closed this Nov 29, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants