Skip to content

expose cmab prediction endpoint in url template - #378

Merged
Matjaz Pirnovar (Mat001) merged 2 commits into
masterfrom
mpirnovar-cmab-expose-pred-endpoint-fssdk-12011
Nov 7, 2025
Merged

Matjaz Pirnovar (Mat001) merged 2 commits into
masterfrom
mpirnovar-cmab-expose-pred-endpoint-fssdk-12011

Conversation

@Mat001

Copy link
Copy Markdown
Contributor

Summary

Exposing optional custom prediction endpoint in URL template when client is instantiated. For the purpose of proxy or testing with a different endpoint.

Test plan

  • new unit tests
  • existing ones pass

Issues

…inting error

- Changed initialize method to use keyword arguments instead of positional arguments
- Updated all call sites to use keyword arguments
- Fixes Metrics/ParameterLists offense (too many optional parameters)
- All tests passing (11 examples, 0 failures)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good.

@Mat001
Matjaz Pirnovar (Mat001) merged commit 7e2ccf2 into master Nov 7, 2025
10 checks passed
@Mat001
Matjaz Pirnovar (Mat001) deleted the mpirnovar-cmab-expose-pred-endpoint-fssdk-12011 branch November 7, 2025 18:32
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