-
Notifications
You must be signed in to change notification settings - Fork 118
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.
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
String features such as lenght, word count, character pos tracking #236
Conversation
Pull Request Test Coverage Report for Build 969712123
💛 - Coveralls |
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.
Overall looks good. Feedback is mostly around the char tracker and the behavior (eager creation of trackers) and the supported set of characters
Co-authored-by: Andy Dang <26821974+andyndang@users.noreply.github.com>
Co-authored-by: Andy Dang <26821974+andyndang@users.noreply.github.com>
Co-authored-by: Andy Dang <26821974+andyndang@users.noreply.github.com>
Co-authored-by: Andy Dang <26821974+andyndang@users.noreply.github.com>
Co-authored-by: Andy Dang <26821974+andyndang@users.noreply.github.com>
Co-authored-by: Andy Dang <26821974+andyndang@users.noreply.github.com>
Co-authored-by: Andy Dang <26821974+andyndang@users.noreply.github.com>
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.
Adding minor changes
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.
GitHub is confusing
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.
LGTM
…hylabs#236) * add character position tracker * add string attributes to flat summary * track unicode characters as NITL * add string notebook
Description
adding String features such as lenght, word count, character pos tracking
General Checklist
if it was a bug, test must cover it.