-
Notifications
You must be signed in to change notification settings - Fork 156
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
#2312 - Add support for new external recommender v2 #2313
Draft
jcklie
wants to merge
77
commits into
main
Choose a base branch
from
feature/2312-external-recommender-v2
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Move old external recommender to v1 subpackage
- Add a v2 copy of the v1 external recommender
- Fix/Add autoconfig for external recommender
- Fix/Add autoconfig for external recommender
- Can check connection
- Remove old documents
- Can select classifiers from the server
- Move v1 tests to their own sub package
- Add FormatConverter - Add data classes - Started tests
- Start predicting
- Predicting works
Use exceptions in external recommender API
Use exceptions in external recommender API
WIP: Add more API methods
- Add more API implementations - Add more tests
- Start implementing training
- Add tests for two more API endpoints - Improved error messages
- Use simpler for classifier/model ids - Add status code check for listDocumentsInDataset
* main: (80 commits) #2505 - Slot features should be ignored when comparing annotations for the purpose of bulk-annotation #2505 - Slot features should be ignored when comparing annotations for the purpose of bulk-annotation #2286 - Upgrade dependencies #2491 - Bulk-deletion does not unselect annotation #2543 Show export progress only on export #2541 Cannot export project when curation started #2534 Add word conversion colab notebook No issue. Fix scope of the tocify dependencies. No issue. Fix layout on external search page. #2522 - Opening sidebar changes document No issue. Avoid error when switching from main to 20.x branch. #2176 - Make long relations nicer without arcs II Bump elasticsearch from 6.8.15 to 6.8.17 in /inception #2534 Add word conversion colab notebook [maven-release-plugin] prepare for next development iteration [maven-release-plugin] prepare release inception-20.1 #2523 - Reset filter on dynamic workload page #2176 - Make long relations nicer without arcs II #2522 - Opening sidebar changes document #2426 - Difference in labelling of annotation marker vs identifier lookup ...
* main: (94 commits) #4240 - Facilitate using DIAM serializer in isolation #4240 - Facilitate using DIAM serializer in isolation #4240 - Facilitate using DIAM serializer in isolation [maven-release-plugin] prepare for next development iteration [maven-release-plugin] prepare release inception-29.5 #4230 - Upgrade dependencies #4230 - Upgrade dependencies #4235 - Direct access-by-URL to sidebar curation mode sometimes does not work #4229 - Better project template selection #4229 - Better project template selection #4232 - Drop evaluation page #4229 - Better project template selection #4230 - Upgrade dependencies #4229 - Better project template selection No issue: Update system requirements and improve pointers to Docker-based deployment No issue: Call out to admins to look at the admin guide when reaching the installation section in the users guide. #4215 - Ability to specify GID and UID for the user used inside the Docker image [maven-release-plugin] prepare for next development iteration [maven-release-plugin] prepare release inception-29.4 #4226 - Display logout button on annotation page if dashboard access is disabled ... % Conflicts: % inception/inception-imls-external/pom.xml
* main: (68 commits) #4316 - Dissolve generic API module #4316 - Dissolve generic API module #4316 - Dissolve generic API module #4316 - Dissolve generic API module #4316 - Dissolve generic API module #4316 - Dissolve generic API module #4314 - Move project API to own module #4308 - Support for custom XML formats #4308 - Support for custom XML formats #4308 - Support for custom XML formats #4310 - Viewport tracking does not work well in RecogitoJS editor #4308 - Support for custom XML formats #4302 - Files imported as HTML do not show properly in HTML-based editor #4305 - New predictions message is shown even if no new predictions were generated #4304 - Log more startup messages through boot logger #4229 - Better project template selection #4298 - Recommender status says "Waiting for evaluation" for recommenders that cannot be evaluated #4297 - Popover always showing "tag not in tagset" even if feature has no tagset at all #4294 - Infer paragraph boundaries when displaying plain text files in HTML-based editors #4150 - Upgrade dependencies ...
* main: (359 commits) #5047 - Clean up layer detail UI a bit #4949 - Showing the start and end points of relations in left side bar No issue: Minor cleaning up #5043 - Ability to specify token breaking zones when calling tokenizer [maven-release-plugin] prepare for next development iteration [maven-release-plugin] prepare release inception-34.0-beta-6 #5009 - Better handling of stacked annotations with link features in curation #5009 - Better handling of stacked annotations with link features in curation [maven-release-plugin] prepare for next development iteration [maven-release-plugin] prepare release inception-34.0-beta-6 [maven-release-plugin] prepare for next development iteration [maven-release-plugin] prepare release inception-33.6 #5040 - Improve feature form tab navigation #5037 - Show fraction of annotators that chose a certain label in curation sidebar mode #5035 - NBSPs should not be treated as tokens #5031 - ChatGPT recommender fails because format is not a supported parameter #5029 - Duplicate lines on the about page #5027 - Add more CSP configurations #4753 - Entity linker should skip already linked concepts #5007 - Lazy details on suggestions for multi-value concept features fail rendering ...
* main: (30 commits) [maven-release-plugin] prepare for next development iteration [maven-release-plugin] prepare release inception-34.0 No issue: Mention MS SQL and PosgreSQL as experimental DB options No issue: Fix versions after merge #5043 - Ability to specify token breaking zones when calling tokenizer #5064 - Project template for PHI annotation #5071 - Better document which layers are supported by which formats #5061 - Multiple synchronous recommenders only the last one wins #5064 - Project template for PHI annotation #5068 - Show annotation sidebar by default #5033 - Ability to configure recommenders interactively on the annotation page #5066 - Increase default number of rows for brat editors #5056 - Ability to configure additional languages for knowledge bases #4909 - Upgrade dependencies #5056 - Ability to configure additional languages for knowledge bases #4909 - Upgrade dependencies No issue: Actually, document-metadata doesn't seem to be experimental after all... No issue: Set version to 35.0-SNAPSHOT [maven-release-plugin] prepare for next development iteration [maven-release-plugin] prepare release inception-33.7 ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What's in the PR
How to test manually
pip install -e ".[all]"
make inception_test
Automatic testing
Documentation