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.

Sign up for GitHub

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

Jump to bottom

CU-8695x1dy9: Changes for running predictions async in a bg process #214

Merged
merged 16 commits into from
Nov 29, 2024
Merged

CU-8695x1dy9: Changes for running predictions async in a bg process #214

merged 16 commits into from
Nov 29, 2024

Conversation

tomolopolis
Copy link
Member

These changes enable:

  • running models in background process,
  • run process_tasks in separate docker compose service
  • add a dialog workflow to inspect running state of bg_process
  • add an API to cancel running process and directly go in to annotate a project.

Bug fixes

  • CSRF_TRUSTED_ORIGINS empty env var wasn't being processed correctly to not raise a warning, now fixed
  • Model_cacheing - new env var to clear down models in the module level cache. This is set to 2 by default

@tomolopolis
CU-8695x1dy9: runs model in background process, run process_tasks in…
30ebe9a 
… separate docker compose service, dialog workflow to inspect running state of bg_process and API to cancel running process and directly go in to annotate a project.
@tomolopolis
Copy link
Member Author

Task linked: CU-8695x1dy9 ability to load predictions for all documents in a background process

tomolopolis and others added 15 commits October 3, 2024 13:07
@tomolopolis
CU-8695xcr4k: fix styling of meta-annos for both model pack pred scen…
e452d64 
…ario and defaulting scenario of standard cdb / vocab project.
@tomolopolis
Merge pull request #215 from CogStack/meta-anno-click-event-bug
47fa649 
CU-8695xcr4k: fix UI bug of meta-annos
@tomolopolis
CU-86961hrm5: Fix bubbling up of keydown events when typing numbers i…
29d9eb8 
…nto concept picker
@tomolopolis
Merge pull request #216 from CogStack/meta-anno-click-event-bug
f6d0a76 
CU-86961hrm5: Fix bubbling up of keydown events when typing numbers into concept picker
@tomolopolis
bump to latest release
6be8ab2
@yangnoahlin
fix: vocab_file should be saved before loading
461af16
@tomolopolis
Merge pull request #217 from yangnoahlin/master
d1865f6 
fix: vocab_file should be saved before loading
@tomolopolis
CU-8696ent8k: vue3 upgrade and migration from bootstrap-vue UI framew…
3d76a5d 
…ork to vuetify3.

Bootstrap-vue doest not support vue3 hence the upgrade. Look and feel is mostly similar, as bootstrap is still used throughout.
@tomolopolis
Merge pull request #220 from CogStack/vue3-upgrade
7c49911 
CU-8696ent8k: major version upgrade to frontend frameworks: vue3 , bootrstrap 5, vuetify3 from bootstrap-vue
@tomolopolis
CU-8695x1dy9: runs model in background process, run process_tasks in …
5d6a3fd 
…separate docker compose service, dialog workflow to inspect running state of bg_process and API to cancel running process and directly go in to annotate a project.
@tomolopolis
Merge remote-tracking branch 'origin/bg-process-preds' into bg-proces…
84f186b 
…s-preds

# Conflicts:
#	docker-compose-prod.yml
#	docker-compose.yml
#	webapp/frontend/src/components/common/ProjectList.vue
@tomolopolis
CU-8696ent8k: fix bg process running cancellation and into project entry
9bc32e6
@tomolopolis
CU-8696ent8k: fix tooltip for concept search
7b9169e
@tomolopolis
CU-8696ent8k: increase timeout, and ensure database locked errors do …
8093d3d 
…not occur during DB reads and bg_process tasks are writing to he DB.
@tomolopolis
CU-8696ent8k: make prep_docs AnnotationEntity addition atomic.
92ac80f
adam-sutton-1992
adam-sutton-1992 approved these changes Nov 29, 2024
View reviewed changes
Copy link
Collaborator

@adam-sutton-1992 adam-sutton-1992 left a comment

Choose a reason for hiding this comment

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

Approved, with acknowledgement that SQLlite might need to be changed for a better suited database due to locking of read/write commands.

@tomolopolis tomolopolis merged commit 3a9337f into master Nov 29, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adam-sutton-1992 adam-sutton-1992 adam-sutton-1992 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tomolopolis @adam-sutton-1992 @yangnoahlin