Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Added a bulk data filter, with tests #33

Merged
merged 2 commits into from Jun 9, 2017
Merged

Added a bulk data filter, with tests #33

merged 2 commits into from Jun 9, 2017

Conversation

KarlSjostrand
Copy link
Contributor

No description provided.

@KarlSjostrand KarlSjostrand self-assigned this Jun 4, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.0%) to 88.414% when pulling 7d797b5 on feature/bulkdatafilter into 55b0a43 on develop.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.0%) to 88.414% when pulling 2780442 on feature/bulkdatafilter into 55b0a43 on develop.

fredrik-exini
fredrik-exini approved these changes Jun 8, 2017
View reviewed changes
Copy link

@fredrik-exini fredrik-exini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not familiar with the reason for not removing pixeldata in sequences and wave data outside sequences but it looks good.

@KarlSjostrand KarlSjostrand merged commit 82302dc into develop Jun 9, 2017
@KarlSjostrand KarlSjostrand deleted the feature/bulkdatafilter branch June 9, 2017 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fredrik-exini fredrik-exini fredrik-exini approved these changes

Assignees

@KarlSjostrand KarlSjostrand

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@KarlSjostrand @coveralls @fredrik-exini