Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Stream.sample is not affected by processors #84

Closed
roll opened this issue Sep 13, 2016 · 0 comments · Fixed by #85
Closed

Stream.sample is not affected by processors #84

roll opened this issue Sep 13, 2016 · 0 comments · Fixed by #85
Assignees
Labels
Milestone

Comments

@roll
Copy link
Member

roll commented Sep 13, 2016

Overview

Expected behavior:

    # Processors
    def only_first_row(extended_rows):
        for number, header, row in extended_rows:
            if number == 1:
                yield (number, header, row)

    # Get table
    table = topen('data/table.csv', post_parse=[only_first_row])

    # Make assertions
    assert table.sample == ['id', 'name'] # for now will return all table rows
@roll roll added this to the tabulator-v1 milestone Sep 13, 2016
@roll roll self-assigned this Sep 13, 2016
@roll roll added review and removed current labels Sep 13, 2016
@roll roll closed this as completed in #85 Sep 13, 2016
@roll roll removed the review label Sep 13, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant