Skip to content

Fix for lvm-dependent tests and clear textbox#14

Merged
mhbuehler merged 1 commit intomelanie/mm-rag-enhancedfrom
melanie/fix_tests
Nov 6, 2024
Merged

Fix for lvm-dependent tests and clear textbox#14
mhbuehler merged 1 commit intomelanie/mm-rag-enhancedfrom
melanie/fix_tests

Conversation

@mhbuehler
Copy link
Owner

Description

Reordered the new lvm-dependent tests so they execute last and fixed a text display error.

Issues

MultimodalQnA Audio & Image Enhancements RFC

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Dependencies

No new dependencies

Tests

Checked that the tests pass when the lvm service is ready. If it doesn't pass in OPEA's CICD then we might have to remove the tests.

Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.com>
Copy link
Collaborator

@dmsuehir dmsuehir left a comment

Choose a reason for hiding this comment

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

LGTM

@mhbuehler mhbuehler merged commit 359b6f8 into melanie/mm-rag-enhanced Nov 6, 2024
@mhbuehler mhbuehler deleted the melanie/fix_tests branch November 6, 2024 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants