Skip to content

Conversation

@tmrts
Copy link
Contributor

@tmrts tmrts commented Mar 6, 2016

This PR ports text module to python 3.5 and adds tests for it.

Note: I've intentionally didn't touch a random sampling doctest, since I don't have access to the random seed used with that particular test.

@norvig
Copy link
Collaborator

norvig commented Mar 6, 2016

Doctests that produce nondeterministic output like that should probably go into a .ipynb file, not a _test.py.

norvig added a commit that referenced this pull request Mar 6, 2016
Port and test text module for python 3.5
@norvig norvig merged commit b36947c into aimacode:master Mar 6, 2016
@tmrts
Copy link
Contributor Author

tmrts commented Mar 6, 2016

@norvig we can override the random module's RNG seed to get a deterministic output. I think there should be tests for them as well. Otherwise it's hard to find errors for those.

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