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.

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

fix: check ocrread results before getting image extent #296

Merged

Conversation

stew-ro
Copy link
Contributor

@stew-ro stew-ro commented Jun 1, 2020

No description provided.

@stew-ro stew-ro linked an issue Jun 1, 2020 that may be closed by this pull request
@stew-ro stew-ro requested a review from alex-krasn June 1, 2020 21:15
@stew-ro stew-ro requested review from kunzms and tianxiangs June 1, 2020 21:23
@stew-ro stew-ro merged commit 61dba02 into master Jun 1, 2020
@stew-ro stew-ro deleted the stew-ro/check-ocrreadresults-before-getting-image-exent branch June 1, 2020 21:32
alex-krasn added a commit that referenced this pull request Jun 25, 2020
* feature: shortcuts icon toolbar (#288)

* fix: fix mime check bug for jpeg/jpg and tiff (#291)

* doc: update version to 2.0-2760166

* Update CHANGELOG.md

* feat: Add better error message for CORS  (#289)

* feat: refactor CORS error message

* refactor: remove unuse import and console

* refactor: fix Spanish message and add multilineMessage function in Alert

* refactor: change number to bullet points

* add div when return multiline message

* style:remove extra space

Co-authored-by: kunzheng <58841788+kunzms@users.noreply.github.com>

* fix: check ocrread results before getting image extent (#296)

* feat: enable download analyze script (#304)

* migrating from npm to yarn (#305)

* migrating from npm to yarn

* added Typescript

* updated azure-pipeliens.yml

* style:  deletes run in scripts

* Update CHANGELOG.md

* migrating to Fluent UI (#306)

Co-authored-by: kunzheng <58841788+kunzms@users.noreply.github.com>

* feat: add description - how to delete info (#292)

* feat: add description - how to delete info

* style: new line

* style: removes extra line

* fix: correction of text

* Update CHANGELOG.md

* Update CHANGELOG.md

* Update statusBar.tsx

* refactor: use yarn in dockerfile (#309)

* refactor: use yarn in dockerfile

* refactor: upgrade node to 12.x

* feature: keep prediction in UI (#285)

* feature: we're keepin prediction in UI now

* feat: add protection for additional hidden elements for skipButton

* bugfix: - delete forgoten changes

* rerender on train model change + refactor

* fix: forgotten log , and comment

* fix: now rerender when setting changes and on new project

* refactor: delete unnecessary helper file

* fix: add  this.props.project check

Co-authored-by: stew-ro <60453211+stew-ro@users.noreply.github.com>

* fix: fix spelling correction for string match (#318)

* fix: capitalize python in analyze page (#320)

* fix: capitalize python in analyze page

* Alex krasn/docs update readme keyboard shortcuts (#313)

* fix:  deletes trailing whitespaces

* docs: added icon and updated description

* refactor: rephrasing and icon on new line

* feat: enable re-run OCR (#297)

* feat: enable re-run OCR (draft)

* add rerun OCR for single file functionality

* added tooltip

* add icon and changed position

* added - rerun OCR for all documents in project

* fix: remove idents

* feat: enable re-run OCR (draft)

* add rerun OCR for single file functionality

* added tooltip

* add icon and changed position

* added - rerun OCR for all documents in project

* refactor: delete redundant local state and rename

* fix: indent

* style: change submenu text

* docs: added manual test runbook

* style: idents

* refactor: rename dropdown class name

* renaming: re-run.. to run, files to documents..

* fix: style and grammar

* fix: run on not visited

* docs: new Test Runbook revision

* style: fix wording and spacing

* refactor: use anonymous wrapper function for onclick

Co-authored-by: kunzheng <58841788+kunzms@users.noreply.github.com>
Co-authored-by: Robert Stewart (eXcell <v-stewro@microsoft.com>

* feature: localization for canvas commandbar items #319 (#323)

* feat: localization all canvas commandbar items

* fix: labe in Layers - has to be plural

* style: - extra line

* fix: quick reordering tags (#322)

* fix: added debounce for quick reordering tags

* docs: added use case

* refactor: incresed dedounce interval so its not affect when reordering slowly

* decrease debounce time to 3000ms

* style: - indent

* add: show prediction component for both (receipts, predict) routes

* fixing icons + add alternative receipts icon

* + badge "Preview"

* Update README.md

* fix: error on copy APIKey undefined

* increase visibility of higlied values, separation valueNames and values ,misc styles fixes

* disable sidebar links

* fix: sigle quotes -> double

* fix: trailing whitespaces  + spelling "separator"

* redirect to receipt page + now showing python script download

* hide disabled links

* deletes prepended currency units

* fix: accessibility

* misc: styling and accessibility

Co-authored-by: kunzheng <58841788+kunzms@users.noreply.github.com>
Co-authored-by: tianxiang <60452964+tianxiangs@users.noreply.github.com>
Co-authored-by: stew-ro <60453211+stew-ro@users.noreply.github.com>
Co-authored-by: Robert Stewart (eXcell <v-stewro@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FOTT bug report: Cannot read property 'width' of undefined
3 participants