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

Add services to tests and correct small text error #26

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

mhbuehler
Copy link
Owner

Description

Fix for failing tests per PR feedback and small text fix

Issues

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

N/A

Tests

N/A

Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.com>
Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.com>
@mhbuehler mhbuehler merged commit aa5a5ac into mmqna-audio-query Dec 5, 2024
1 check failed
dmsuehir added a commit that referenced this pull request Dec 16, 2024
* Add var for LVM model

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* README updates and tests

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Build, compose, and doc updates for audio query (#12)

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* validated, updated tests

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* added one more curl test for audio

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* fixed typo

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* reverted git clone command

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* added ASR test

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* fixed command with backslashes

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

---------

Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.com>
Signed-off-by: okhleif-IL <omar.khleif@intel.com>
Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* MMQnA doc update correcting ASR and whisper image names (#24)

* MMQnA doc update correcting ASR and whisper image names

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Add image tags

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

---------

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Integrate audio query into UI (#22)

* Enabled audio query functionality in the MultimodalQnA UI

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

* Temporarily redirect clones for tests

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

* Doc update and set MAX_IMAGES to default to 1 in compose.yaml

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* README update to add a note about MAX_IMAGES

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Add services to tests and correct small text error (#26)

* Add services to tests and correct small text error

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

* Revert unintended changes

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

---------

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

* Fixed build.yaml inconsistency

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

* Moving note about MAX_IMAGES due to review feedback:

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Update repo clones for E2E tests

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

* Moved Audio Query Gateway changes to multimodalqna.py (#29)

* Moved gateway changes to multimodalqna.py

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* reverted port changes

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* addressed review comments

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* reverted print statement

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

---------

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* Removed proxies arg from translation request (#31)

* Moved gateway changes to multimodalqna.py

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* reverted port changes

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* addressed review comments

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* reverted print statement

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* removed proxies

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

---------

Signed-off-by: okhleif-IL <omar.khleif@intel.com>

* Add MAX_IMAGES to docs

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Moved gateway updates for image query to GenAIExamples

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Fix initial input format for TextImageDoc

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Fix merge conflict

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Fix merge conflict

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Add Gaudi tests

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Temporary update to use dev branch for test

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

* Temporary change for dockerfile to use dev fork/branch for GenAIComps

Signed-off-by: dmsuehir <dina.s.jones@intel.com>

---------

Signed-off-by: dmsuehir <dina.s.jones@intel.com>
Signed-off-by: Melanie Buehler <melanie.h.buehler@intel.com>
Signed-off-by: okhleif-IL <omar.khleif@intel.com>
Co-authored-by: Omar Khleif <omar.khleif@intel.com>
Co-authored-by: Melanie Hart Buehler <melanie.h.buehler@intel.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Abolfazl Shahbazi <12436063+ashahba@users.noreply.github.com>
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