-
Notifications
You must be signed in to change notification settings - Fork 26
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
literature: add record params #359
Conversation
2d6a14a
to
308f862
Compare
@@ -222,6 +223,7 @@ def add_author(self, author): | |||
|
|||
@filter_empty_parameters | |||
def make_author(self, full_name, | |||
record=None, |
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.
you should add it at the end, otherwise it breaks when using positional args, and update the docstring.
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.
but I hope nobody is calling it like that :)
def add_accelerator_experiment(self, legacy_name, record=None): | ||
"""Add legacy name in accelerator experiment. | ||
|
||
:type legacy_name: string |
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.
add record
to docstring.
}] | ||
|
||
|
||
def test_publication_info_public_note(): |
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.
wrong copy/paste?
308f862
to
29400e0
Compare
Enables adding record objects for various fields: * publication_info: conference_record, journal_record * accelerator_experiment: record * author: record
Enables adding record objects for various fields: