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

[BUG]: Increase Param Value String Length to 6000 to Match MLFlow Client #1392

Open
vinayan3 opened this issue Aug 23, 2024 · 0 comments
Open
Labels
bug Something isn't working

Comments

@vinayan3
Copy link

Describe the bug

The ML FLow SDK states the following for log_param method

value – Parameter value, but will be string-ified if not. All built-in backend stores support values up to length 6000, but some may support larger values.

The database backend only supports up to 500. See https://github.com/G-Research/fasttrackml/blob/main/pkg/database/model.go#L134

Environment

  • FastTrackML Version: v0.6.0
  • Operating System: All
  • Database: SQLite & Postgres
  • Artifact Storage: N/A

Steps To Reproduce

The python example can be ran and with this param

    mlflow.log_param("param1", "a"*1000)

Expected Behavior

The param should be logged

Additional Context (Optional)

No response

@vinayan3 vinayan3 added the bug Something isn't working label Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant