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

update schema docs #2207

Merged
merged 1 commit into from
Nov 14, 2024
Merged

update schema docs #2207

merged 1 commit into from
Nov 14, 2024

Conversation

kostmo
Copy link
Member

@kostmo kostmo commented Nov 14, 2024

Updates some content in the JSON schema

@kostmo kostmo added the Z-Documentation This issue is about README or Wikis or Haddock or... label Nov 14, 2024
@kostmo kostmo requested a review from byorgey November 14, 2024 02:18
@kostmo kostmo marked this pull request as ready for review November 14, 2024 02:18
@kostmo kostmo added the merge me Trigger the merge process of the Pull request. label Nov 14, 2024
@mergify mergify bot merged commit 7b50426 into main Nov 14, 2024
4 checks passed
@mergify mergify bot deleted the docs/schema-update branch November 14, 2024 03:33
mergify bot pushed a commit that referenced this pull request Nov 17, 2024
# Prerequisites
* #2209
* #2207

# Overview
Certain hacks (e.g. #2115) were employed to better handle "transparency" when recognizing shapes.  However, these approaches are still limited in capability.

The "right" way to use Aho-Corasick with transparency is to break rows up into the contiguous segments separated by transparent cells.  The individual segments recognized by the automaton can then be matched against their expected position.
```
scripts/play.sh --scenario scenarios/Testing/1575-structure-recognizer/2201-piecewise-lines.yaml --autoplay
```
To view the internal shape recognition logs:
```
http://localhost:5357/recognize/log
```

# Other notable changes
* Use `Writer` monad for logging
* Remove all "entity masking" logic
* Update documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me Trigger the merge process of the Pull request. Z-Documentation This issue is about README or Wikis or Haddock or...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants