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

Merge fea-sherlock feature branch into branch-23.11 #1359

Merged
merged 352 commits into from
Nov 22, 2023

Conversation

drobison00
Copy link
Contributor

Merge fea-sherlock feature branch into branch-23.11

dagardner-nv and others added 30 commits October 19, 2023 16:41
…ine-runnable

Improving the `CoroutineRunnable` class to use schedulers and support multiple in-flight messages
Co-authored-by: Michael Demoret <[email protected]>
…-nv/Morpheus into david-fea-sherlock-restructure
@drobison00 drobison00 added improvement Improvement to existing functionality DO NOT MERGE PR should not be merged; see PR for details sherlock Issues/PRs related to Sherlock workflows and components labels Nov 13, 2023
@drobison00 drobison00 added this to the 23.11 - Sherlock milestone Nov 13, 2023
dagardner-nv and others added 14 commits November 13, 2023 12:21
Updated agents pipeline README.md

Closes #1307 

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/nv-morpheus/Morpheus/blob/main/docs/source/developer_guide/contributing.md).
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - Bhargav Suryadevara (https://github.com/bsuryadevara)

Approvers:
  - Devin Robison (https://github.com/drobison00)

URL: #1352
Allows injecting an optional cache path to the WebScraperStage. If `cache_path` is not None, it uses the path as the path for the response caching system's sqllite database. If `cache_path` is None, the response caching system is bypassed.

Closes #1355

Authors:
  - Christopher Harris (https://github.com/cwharris)

Approvers:
  - Devin Robison (https://github.com/drobison00)

URL: #1358
Commits some models to LFS which were accidentally committed as source code.

Authors:
  - Christopher Harris (https://github.com/cwharris)
  - Devin Robison (https://github.com/drobison00)

Approvers:
  - Devin Robison (https://github.com/drobison00)
  - Bhargav Suryadevara (https://github.com/bsuryadevara)

URL: #1343
…end-vdb-upload

End to end test for vdb upload example
Added agents simple pipeline test 

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/nv-morpheus/Morpheus/blob/main/docs/source/developer_guide/contributing.md).
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - Bhargav Suryadevara (https://github.com/bsuryadevara)

Approvers:
  - Devin Robison (https://github.com/drobison00)

URL: #1363
Add missing milvus mark for two tests which require milvus
@drobison00 drobison00 removed the DO NOT MERGE PR should not be merged; see PR for details label Nov 17, 2023
@drobison00 drobison00 closed this Nov 17, 2023
@drobison00 drobison00 reopened this Nov 17, 2023
dagardner-nv and others added 4 commits November 17, 2023 16:20
…st-ci-1370

Draft: fix for test_simple_request
Closes 

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/nv-morpheus/Morpheus/blob/main/docs/source/developer_guide/contributing.md).
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - David Gardner (https://github.com/dagardner-nv)
  - Bhargav Suryadevara (https://github.com/bsuryadevara)
  - Devin Robison (https://github.com/drobison00)
  - Eli Fajardo (https://github.com/efajardo-nv)

Approvers:
  - Devin Robison (https://github.com/drobison00)

URL: #1374
Made the `enable_cache` accessible as a parameter

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/nv-morpheus/Morpheus/blob/main/docs/source/developer_guide/contributing.md).
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - Bhargav Suryadevara (https://github.com/bsuryadevara)

Approvers:
  - Devin Robison (https://github.com/drobison00)

URL: #1376
Copy link
Contributor

@mdemoret-nv mdemoret-nv left a comment

Choose a reason for hiding this comment

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

Approving to get merged into branch-23.11

@mdemoret-nv mdemoret-nv merged commit 4806119 into branch-23.11 Nov 22, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement to existing functionality non-breaking Non-breaking change sherlock Issues/PRs related to Sherlock workflows and components
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

6 participants