-
Notifications
You must be signed in to change notification settings - Fork 20
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
pypasta: config: change default filename for cluster output #50
Conversation
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.
Almost :-) Just one last simplification, then I'll integrate it. Thanks!
pypasta/Config.py
Outdated
cluster_dst = 'MBOX_RESULT' | ||
else: | ||
cluster_dst = 'PASTA_RESULT' | ||
cluster_dst = 'PATCH_GROUPS' |
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.
One last thing: You can use 'PATCH_GROUPS' directly below. There's no need for outsourcing it in a variable.
pypasta/Config.py
Outdated
cluster_dst = 'MBOX_RESULT' | ||
else: | ||
cluster_dst = 'PASTA_RESULT' | ||
cluster_dst = 'PATCH_GROUPS' | ||
self.f_clustering = path(cluster_dst) |
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.
Just use path('PATCH_GROUPS')
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.
yup makes sense
The default filename for both patch stack and mbox mode is same now Signed-off-by: Rohit Sarkar <rohitsarkar5398@gmail.com>
982327c
to
7b1c481
Compare
Thanks, applied! |
Ah crap, that was a bit too fast: Could you please forward the resources submodule? |
Sure! Will do once the PR is merged there. |
It is merged. |
This isnt merged yet |
The default filename for both patch stack and mbox mode is same now
Signed-off-by: Rohit Sarkar rohitsarkar5398@gmail.com