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
Add typing to write_binary_recording
#2794
Add typing to write_binary_recording
#2794
Conversation
for more information, see https://pre-commit.ci
@@ -110,8 +110,7 @@ def write_binary_recording( | |||
file_path_list = [add_suffix(file_path, ["raw", "bin", "dat"]) for file_path in file_path_list] | |||
|
|||
dtype = dtype if dtype is not None else recording.get_dtype() | |||
cast_unsigned = False | |||
if auto_cast_uint: | |||
if auto_cast_uint: # TODO should we deprecate this given that we have `unsigned_to_signed`? |
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.
Yes!
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.
@h-mayorquin can you add it to this PR?
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.
Done.
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.
I also improved the docstrings a little bit.
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.
The docstring looks nicer to me :)
Co-authored-by: Alessio Buccino <alejoe9187@gmail.com>
Working on the tutorial. Small PR with improvement.