-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Updating FileRecordStream docstrings #3545
Updating FileRecordStream docstrings #3545
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.
Done with review. There a couple of typos. And I have a more general question about some attributes that are not being used. Otherwise it looks good to me.
src/nupic/data/file_record_stream.py
Outdated
(line 1), type (line 2), and a special indicator (line 3). The special indicator | ||
can indicate that the field specifies a reset, is a sequence ID, or is a | ||
timestamp for the record. | ||
:class:`~.file_record_stream.FileRecordStream` is class that can read and write |
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.
FileRecordStream
is a class
src/nupic/data/file_record_stream.py
Outdated
The FileRecordStream class supports the context manager ('with' statement ) | ||
protocol. That means you con do: | ||
:class:`~.file_record_stream.FileRecordStream` supports the context manager | ||
(``with`` statement) protocol. That means you con do: |
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.
Typo: You can do
src/nupic/data/file_record_stream.py
Outdated
retval: a data row (a list or tuple) if available; None, if no more records | ||
in the table (End of Stream - EOS); empty sequence (list or tuple) | ||
when timing out while waiting for the next record. | ||
:param useCache: NOT USED |
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.
Maybe we should just get rid of this?
src/nupic/data/file_record_stream.py
Outdated
# input bookmark is not applicable in case of a file storage | ||
inputBookmark = inputBookmark | ||
:param record: a list of Python objects that will be string-ified | ||
:param inputBookmark: NOT USED |
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.
Can we rid of this?
src/nupic/data/file_record_stream.py
Outdated
Returns whether there are more records from current position. ``bookmark`` | ||
is not used in this implementation. | ||
|
||
:param bookmark: NOT USED |
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.
Same question as before, ca we remove it since it's not used.
@marionleborgne took all your feedback, thanks |
Fixes #3542
Blocked by #3541