-
Notifications
You must be signed in to change notification settings - Fork 23
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
working towards py3 compatibility #22
base: master
Are you sure you want to change the base?
Conversation
manu-chroma
commented
Apr 2, 2017
- fixed imports
- print statements
@nmoya don't approve this yet. This is still WIP. |
@@ -170,24 +173,24 @@ def compute_message_proportions(self, list_of_messages, senders, root, contact): | |||
self.proportions["avg_words"] = {} | |||
for s in senders: | |||
self.proportions["avg_words"][s] = self.proportions["words"][s] / self.proportions["messages"][s] | |||
self.proportions["avg_words"]["ratio"] = self.proportions["avg_words"][root] / self.proportions["avg_words"][contact] | |||
self.proportions["avg_words"]["ratio"] = self.proportions["avg_words"][root] / self.proportions["avg_words"][ | |||
contact] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Funky line break
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
haha yeah. pycharm magic 😄
@manu-chroma so far so good with the changes, let me know when you update. |
8d0ab0e
to
71caeb8
Compare
- fixed imports - print statements
@nmoya Pls review. Python 3.x tests are still not passing, but I've made a bunch of improvements to the codebase. Also, rewritten the test file. |
args = "-f test/testChat2.txt --root Username{}".format(case).split() | ||
|
||
with captured_output() as (result, err): | ||
parser(args) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rather than using os.system
, better approach is to import the main package and pass args directly to it.
@manu-chroma Sorry for the delay, nice work. Can you remove the *.bak files from the PR? Other than that, LGTM. |