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

#1778 - Make tokenization editable #2989

Merged
merged 13 commits into from
Apr 15, 2022

Conversation

reckart
Copy link
Member

@reckart reckart commented Apr 14, 2022

What's in the PR

  • Integrate BratVisualizer into BratSuggestionVisualizer since that was the only place it was used
  • Display sentences/tokens in the annotator views on the curation page if they are editable
  • If tokens/sentences are editable, run them through the merge process instead of obtaining them from the template
  • Make AnnotationPage and CurationPage code more similar
  • Create editor on CurationPage via the editor registry instead of instantiating it directly
  • When sentences are editable the use a line-oriented rendering in the curation view - otherwise a sentence-oriented view.
  • Add documentation for editable sentences
  • Disable changing anchoring mode and overlap mode for sentences / tokens
  • Disable coloring rules for sentences
  • Use a line-oriented editor instead of a sentence-oriented editor while the sentence layer is not read-only

How to test manually

  • Enable sentence editing and try out what works and what might not work - tell us about it: ui.sentence-layer-editable=true

Automatic testing

  • PR includes unit tests

Documentation

  • PR updates documentation

- Integrate BratVisualizer into BratSuggestionVisualizer since that was the only place it was used
- Display sentences/tokens in the annotator views on the curation page if they are editable
- If tokens/sentences are editable, run them through the merge process instead of obtaining them from the template
reckart added 12 commits April 14, 2022 22:07
- Make AnnotationPage and CurationPage code more similar
- Create editor on CurationPage via the editor registry instead of instantiating it directly
* main:
  #2987 - Error when pressing "add" when no user is selected
- Make AnnotationPage and CurationPage code more similar
- Create editor on CurationPage via the editor registry instead of instantiating it directly
- Remove duplicate NoOpPagingStrategy
- Rearrange rendering code in curation so that there is a better separation between pre-rendering and editor-specific serialization
- Reduce reliancy on CAS in serializers
- When sentences are editable the use a line-oriented rendering in the curation view - otherwise a sentence-oriented view.
- Move BratSuggestionVisualizer to other package
- Rename AnnotatorSegment to AnnotatorSegmentState
- Pull out curation unit overview into separate class
- Mark VDocument-related classes as serializable
- Slight cleaning up
- Add documentation for editable sentences
- Disable changing anchoring mode and overlap mode for sentences / tokens
- Disable coloring rules for sentences
- Use a line-oriented editor instead of a sentence-oriented editor while the sentence layer is not read-only
@reckart reckart added this to the 24.0 milestone Apr 15, 2022
@reckart reckart mentioned this pull request Apr 15, 2022
19 tasks
@reckart reckart merged commit 55c04c2 into main Apr 15, 2022
@reckart reckart deleted the feature/1778-Make-tokenization-editable branch April 15, 2022 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant