Skip to content

Comments

Add unit test for MLflow tracking URI regression fix#694

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/sub-pr-693
Draft

Add unit test for MLflow tracking URI regression fix#694
Copilot wants to merge 2 commits intomainfrom
copilot/sub-pr-693

Conversation

Copy link
Contributor

Copilot AI commented Feb 11, 2026

Change Description

Adds regression test for the MLflow tracking directory fix from PR #693. The original fix ensured MLflow stores its backend in the root results directory rather than per-run subdirectories, but lacked test coverage.

Solution Description

Test Implementation

  • Mocks mlflow.set_tracking_uri to verify it receives file://<root_results_dir>/mlflow
  • Validates fix using existing loopback_hyrax fixture (both dataset variants)
  • Placed in tests/hyrax/test_train.py alongside existing training tests

Additional Changes

  • Added mlflow.db to .gitignore to exclude test-generated database files

Code Quality

  • I have read the Contribution Guide and agree to the Code of Conduct
  • My code follows the code style of this project
  • My code builds (or compiles) cleanly without any errors or warnings
  • My code contains relevant comments and necessary documentation

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Base automatically changed from mlflow-fixes to main February 11, 2026 23:56
Copilot AI changed the title [WIP] Update PR to address feedback on MLFlow Tracking Directory Add unit test for MLflow tracking URI regression fix Feb 12, 2026
Copilot AI requested a review from aritraghsh09 February 12, 2026 00:13
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