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

feat: ocr(ios) #84

Merged
merged 20 commits into from
Feb 24, 2025
Merged

Conversation

NorbertKlockiewicz
Copy link
Contributor

@NorbertKlockiewicz NorbertKlockiewicz commented Jan 16, 2025

Description

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (improves or adds clarity to existing documentation)

Tested on

  • iOS
  • Android

Testing instructions

Screenshots

Related issues

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@NorbertKlockiewicz NorbertKlockiewicz marked this pull request as ready for review January 27, 2025 10:32
@NorbertKlockiewicz NorbertKlockiewicz changed the title feat: implementation of detector pre and post processing(ios) feat: ocr(ios) Jan 28, 2025
@NorbertKlockiewicz NorbertKlockiewicz added the enhancement New feature or request label Jan 29, 2025
@@ -71,6 +71,19 @@ const StyleTransfer = StyleTransferSpec
}
);

const OCRSpec = require('./NativeOCR').default;
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should also implement hookless api for OCR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it will be best to add it when this pr is merged to branch with hookles api already implemented

@NorbertKlockiewicz NorbertKlockiewicz changed the base branch from main to v0.3.0 February 13, 2025 14:58
@mkopcins mkopcins changed the base branch from 0.3.0 to main February 19, 2025 09:55
<!-- Provide a concise and descriptive summary of the changes
implemented in this PR. -->

- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] Documentation update (improves or adds clarity to existing
documentation)

- [ ] iOS
- [x] Android

<!-- Provide step-by-step instructions on how to test your changes.
Include setup details if necessary. -->

<!-- Add screenshots here, if applicable -->

<!-- Link related issues here using #issue-number -->

- [x] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [x] My changes generate no new warnings

<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->
@NorbertKlockiewicz NorbertKlockiewicz force-pushed the @norbertklockiewicz/ocr-implementation branch from 42cc379 to 9538fbf Compare February 24, 2025 12:40
@NorbertKlockiewicz NorbertKlockiewicz merged commit deffbe5 into main Feb 24, 2025
2 checks passed
@NorbertKlockiewicz NorbertKlockiewicz deleted the @norbertklockiewicz/ocr-implementation branch February 24, 2025 15:18
jakmro pushed a commit that referenced this pull request Mar 5, 2025
## Description
<!-- Provide a concise and descriptive summary of the changes
implemented in this PR. -->

### Type of change
- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] Documentation update (improves or adds clarity to existing
documentation)

### Tested on
- [x] iOS
- [x] Android

### Testing instructions
<!-- Provide step-by-step instructions on how to test your changes.
Include setup details if necessary. -->

### Screenshots
<!-- Add screenshots here, if applicable -->

### Related issues
<!-- Link related issues here using #issue-number -->

### Checklist
- [x] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [ ] My changes generate no new warnings

### Additional notes
<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants