Skip to content

Revert "FIX #2562: send "dimensions" to Nvidia embedding NIM for variable-dimension models" - #2564

Open
amorton wants to merge 1 commit into
mainfrom
revert-2561-fix/nvidia-embedding-dimensions
Open

Revert "FIX #2562: send "dimensions" to Nvidia embedding NIM for variable-dimension models"#2564
amorton wants to merge 1 commit into
mainfrom
revert-2561-fix/nvidia-embedding-dimensions

Conversation

@amorton

@amorton amorton commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Reverts #2561

while there was some code things that can be improved, the main problem if this will break the emebdding gateway unless we rebuild it which we do not want to do right now.

@amorton
amorton requested a review from a team as a code owner August 24, 2026 23:13
@github-actions

Copy link
Copy Markdown
Contributor

📉 Unit Test Coverage Delta vs Main Branch

Metric Value
Main Branch 53.44%
This PR 53.40%
Delta 🔴 -0.04%
⚠️ Coverage decreased

@github-actions

Copy link
Copy Markdown
Contributor

Unit Test Coverage Report

Overall Project 53.4% 🍏
Files changed 100% 🍏

File Coverage
NvidiaEmbeddingProvider.java 94.51% 🍏
EmbeddingProvider.java 73.89% 🍏

@github-actions

Copy link
Copy Markdown
Contributor

📈 Integration Test Coverage Delta vs Main Branch (dse69-it)

Metric Value
Main Branch 71.44%
This PR 71.45%
Delta 🟢 +0.01%
✅ Coverage improved!

@github-actions

Copy link
Copy Markdown
Contributor

Integration Test Coverage Report (dse69-it)

Overall Project 71.45% -0.02% 🍏
Files changed 0%

File Coverage
EmbeddingProvider.java 44.52% 🍏
NvidiaEmbeddingProvider.java 14.84% -11.54%

@github-actions

Copy link
Copy Markdown
Contributor

📈 Integration Test Coverage Delta vs Main Branch (hcd-it)

Metric Value
Main Branch 72.76%
This PR 72.76%
Delta 🟢 +0.01%
✅ Coverage improved!

@github-actions

Copy link
Copy Markdown
Contributor

Integration Test Coverage Report (hcd-it)

Overall Project 72.76% -0.02% 🍏
Files changed 0%

File Coverage
EmbeddingProvider.java 44.52% 🍏
NvidiaEmbeddingProvider.java 14.84% -11.54%

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