Conversation
…`url_override` - Updated `configure()` to prefer `url_override` over `url`, with fallback logic when only `url` is provided. - Added a deprecation warning when `url` is used. - Updated unit tests to cover `url_override` precedence and deprecation warning scenarios.
…igure()` scenarios - Renamed test methods to follow a structured naming pattern, clarifying the behavior being tested. - Updated docstrings to reflect test intent more precisely.
alexkuzmik
approved these changes
Apr 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Details
Source: Claude session 2, issue #1
The Python SDK configure() method accepts url= but the ~/.opik.config file uses the field name url_override. Calling opik.configure(url_override="https://...") raises TypeError: configure() got an unexpected keyword argument 'url_override'.
Steps to reproduce:
Expected:
Parameter naming should be consistent between the API and config file, or url_override should be accepted as an alias.
Severity:
Minor
Summary
Deprecates the
urlparameter in theconfigure()function and introducesurl_overrideas its replacement, logging a warning when the old parameter is used and ensuringurl_overridetakes precedence when both are supplied.Changes by Component
Python SDK
url_overrideparameter to theconfigure()function as the canonical replacement forurl.LOGGER.warning) whenurlis passed, directing users tourl_override.url_overridetakes precedence overurlwhen both are provided;urlis used as a fallback only whenurl_overrideis absent.urlas deprecated and document the newurl_overrideparameter.TestConfiguretest class covering all foururl/url_overridecombinations: precedence, deprecation warning presence, warning absence, and fallback forwarding.Files Changed
Change checklist
Issues
AI-WATERMARK
AI-WATERMARK: yes
Testing
Implemented unit tests
Documentation
Added related docstrings and log output (deprecation warnings)