Closed cakiki closed 1 year ago
I think the current docstring for construct_timed_document_frequency is copied over from construct_document_frequency: https://github.com/TutteInstitute/vectorizers/blob/master/vectorizers/preprocessing.py#L40-L56 and might need adjusting.
construct_timed_document_frequency
construct_document_frequency
"timed" here probably refers to a sequence of (token, timestamp) tuples. I'm guessing both the docstring and the first parameter need to be changed? I'll send a tentative PR.
(token, timestamp)
I think the current docstring for
construct_timed_document_frequencyis copied over fromconstruct_document_frequency: https://github.com/TutteInstitute/vectorizers/blob/master/vectorizers/preprocessing.py#L40-L56 and might need adjusting."timed" here probably refers to a sequence of
(token, timestamp)tuples. I'm guessing both the docstring and the first parameter need to be changed? I'll send a tentative PR.