-
Notifications
You must be signed in to change notification settings - Fork 30
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
arXiv: cleanup parser code #274
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.
👍 except for minor comment
01d77be
to
aa033d7
Compare
* removing unused imports * other minor code improvements Signed-off-by: Thorsten Schwander <thorsten.schwander@gmail.com>
aad0256
to
6d38d51
Compare
@michamos any chance this will get merged sometime ? |
@@ -357,5 +346,5 @@ def source(self): | |||
@property | |||
def authors_and_collaborations(self): | |||
if not hasattr(self, '_authors_and_collaborations'): | |||
self._authors_and_collaborations = self._get_authors_and_collaborations(self, self.root) | |||
self._authors_and_collaborations = self._get_authors_and_collaborations(self.root) |
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.
Hi @tsgit, just noticed this PR was still open. This seems like a huge mistake that would cause an exception due to wrong number of arguments previously. Do you understand how it was working?
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.
Thanks for the IRL explanation!
@@ -357,5 +346,5 @@ def source(self): | |||
@property | |||
def authors_and_collaborations(self): | |||
if not hasattr(self, '_authors_and_collaborations'): | |||
self._authors_and_collaborations = self._get_authors_and_collaborations(self, self.root) | |||
self._authors_and_collaborations = self._get_authors_and_collaborations(self.root) |
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.
Thanks for the IRL explanation!
Signed-off-by: Thorsten Schwander thorsten.schwander@gmail.com
Description
code cleanup, removing unused imports, some whitespace and correcting an ill advised @staticmethod decorator
no functional changes
Related Issue
Motivation and Context
Checklist:
RFC
and look for it).