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

Masked data iterator #168

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Bam4d
Copy link
Contributor

@Bam4d Bam4d commented Dec 19, 2015

Fixed history, added a test, removed old pull request.

I have found this useful for NLP tasks in recurrent neural networks where there are inputs that should be ignored within input windows, (no 'UNKOWN') class etc.

@yinyinl
Copy link
Contributor

yinyinl commented Jan 21, 2016

We are looking into bringing this in. Thanks for adding the test! Have you thought about subclassing it from the data iterator or adding mask as an optional input, to make data iterator handle both case (and re-use what is being done in there)?

@wei-v-wang
Copy link
Contributor

Thank you @Bam4d It has been quite a while and sorry for the delay.
Could you please resolve the conflicts?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants