-
Notifications
You must be signed in to change notification settings - Fork 837
Conversation
looks good, just have one question. At some point we may want to use a file/CSV writer so that we don't buffer everything in memory, but for now should be fine. SonarCloud is complaining about duplicate lines, but it's unclear to me as to why. |
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.
Can you please re-target to develop
branch. I don't believe there should be anything related to active learning here.
@wbreza unfortunately I started this from the Active Learning branch so if I re-target to develop it will contains all the commits from that branch too. I think we have to first approve the AL PR. If this is urgent of course we could just copy the changes manually on a new branch but is this really so urgent ? |
cb71bae
to
d6db509
Compare
3b8d7fe
to
976bf0b
Compare
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
author Wallace Breza <wallace@breza.me> 1556573964 -0700 committer Elizabeth Halper <elhalper@microsoft.com> 1566326008 -0700 parent e4ee4cd author Wallace Breza <wallace@breza.me> 1556573964 -0700 committer Elizabeth Halper <elhalper@microsoft.com> 1566326007 -0700 parent e4ee4cd author Wallace Breza <wallace@breza.me> 1556573964 -0700 committer Elizabeth Halper <elhalper@microsoft.com> 1566326004 -0700 parent e4ee4cd author Wallace Breza <wallace@breza.me> 1556573964 -0700 committer Elizabeth Halper <elhalper@microsoft.com> 1566325986 -0700 parent e4ee4cd author Wallace Breza <wallace@breza.me> 1556573964 -0700 committer Elizabeth Halper <elhalper@microsoft.com> 1566325973 -0700 Release 2.1.0 (microsoft#790) Updates package version and changelog for 2.1.0 release changed line refactor: Remove editor footer Dummy commit to kick off build again doc: Add bug & feature templates (microsoft#780) Adds bug and feature github templates Create CODE_OF_CONDUCT.md (microsoft#779) Adds code of conduct docs: updates to readme and changelog (microsoft#781) Minor updates and corrections to the main readme and changelog. fix: Fix display of tag color picker (microsoft#782) Resolves issue of tag color picker not being shown on alt-click or color-click + edit button. Also adds several tests for increased test coverage of tagInput.tsx feat: Add CSV Exporter (microsoft#757) Adds CSV export provider fix: change method for alloc string to buffer (microsoft#777) String.length is not appropriate for calculating buffer size when non-alphabet letter is included in content. Change the method Buffer.alloc to Buffer.from as directed by the nodejs document. fix: Updates backwards compat & fixes cntk export image bug (microsoft#789) Fixes an issue where the images exported out of a video file were missing file extension for video projects. Release 2.1.0 (microsoft#790) Updates package version and changelog for 2.1.0 release changed line rebase upstream dev
Adds CSV export provider to VoTT utilizing
json2csv
npm package