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

Call wk-worker nuclei inferral job #5626

Merged
merged 7 commits into from
Jul 26, 2021
Merged

Call wk-worker nuclei inferral job #5626

merged 7 commits into from
Jul 26, 2021

Conversation

fm3
Copy link
Member

@fm3 fm3 commented Jul 19, 2021

  • Adds a route that calls the webknossos-worker infer_nuclei job if jobsEnabled
  • Shows the result of this job correctly in the job list
  • Note that this PR does not yet include UI to trigger this route, that will be follow-up issue Add UI to trigger nuclei inferral worker job #5631

Steps to test:

Issues:


@fm3 fm3 self-assigned this Jul 19, 2021
@fm3 fm3 changed the title [WIP] call wk-worker nuclei inferral job Call wk-worker nuclei inferral job Jul 22, 2021
@fm3 fm3 marked this pull request as ready for review July 22, 2021 07:28
@fm3 fm3 requested a review from youri-k July 22, 2021 07:30
Copy link
Contributor

@youri-k youri-k left a comment

Choose a reason for hiding this comment

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

LGTM :shipit:

@fm3 fm3 merged commit 6f91d7b into master Jul 26, 2021
@fm3 fm3 deleted the predict-nuclei-job branch July 26, 2021 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build job for nuclei inferral
2 participants