yocto-gl icon indicating copy to clipboard operation
yocto-gl copied to clipboard

[FR] More flexible X-axis for time series plots

Open vilim opened this issue 10 months ago • 8 comments

Willingness to contribute

Yes. I would be willing to contribute this feature with guidance from the MLflow community.

Proposal Summary

It would be very useful to have the per-epoch metrics also plotted vs epoch on the x-axis, not step. In general, it should be possible to have an arbitrary calculated expression from metrics for both x and y axes in plots.

Motivation

What is the use case for this feature?

Comparing models, tracking training.

Why is it currently difficult to achieve this use case?

It is only possible to select time or step as the x axis.

Details

No response

What component(s) does this bug affect?

  • [ ] area/artifacts: Artifact stores and artifact logging
  • [ ] area/build: Build and test infrastructure for MLflow
  • [ ] area/deployments: MLflow Deployments client APIs, server, and third-party Deployments integrations
  • [ ] area/docs: MLflow documentation pages
  • [ ] area/examples: Example code
  • [ ] area/model-registry: Model Registry service, APIs, and the fluent client calls for Model Registry
  • [ ] area/models: MLmodel format, model serialization/deserialization, flavors
  • [ ] area/recipes: Recipes, Recipe APIs, Recipe configs, Recipe Templates
  • [ ] area/projects: MLproject format, project running backends
  • [ ] area/scoring: MLflow Model server, model deployment tools, Spark UDFs
  • [ ] area/server-infra: MLflow Tracking server backend
  • [ ] area/tracking: Tracking Service, tracking client APIs, autologging

What interface(s) does this bug affect?

  • [X] area/uiux: Front-end, user experience, plotting, JavaScript, JavaScript dev server
  • [ ] area/docker: Docker use across MLflow's components, such as MLflow Projects and MLflow Models
  • [ ] area/sqlalchemy: Use of SQLAlchemy in the Tracking Service or Model Registry
  • [ ] area/windows: Windows support

What language(s) does this bug affect?

  • [ ] language/r: R APIs and clients
  • [ ] language/java: Java APIs and clients
  • [ ] language/new: Proposals for new client languages

What integration(s) does this bug affect?

  • [ ] integrations/azure: Azure and Azure ML integrations
  • [ ] integrations/sagemaker: SageMaker integrations
  • [ ] integrations/databricks: Databricks integrations

vilim avatar Apr 17 '24 09:04 vilim

Do we have any plans on supporting this feature? @daniellok-db @hubertzub-db

serena-ruan avatar Apr 17 '24 11:04 serena-ruan

@vilim thanks for filing in the issue!

to have an arbitrary calculated expression from metrics for both x and y axes in plots.

before we jump into that - can you provide a concrete example of how the resulting chart axis/values would be in that scenario so we can be sure we're all on the same page? 🙂

hubertzub-db avatar Apr 18 '24 06:04 hubertzub-db

An example would be e.g. weighted sum of accuracies over different classes

vilim avatar Apr 18 '24 08:04 vilim

@vilim ah got it! So there's an experimental feature of using metric values for X-axis already! It's disabled by default in MLflow UI code, but can be enabled by flipping this flag to true: https://github.com/mlflow/mlflow/blob/06ecbe720332b24b19c1c4b39f5068f782411823/mlflow/server/js/src/common/utils/FeatureUtils.ts#L34-L34

Please let know if this helps. Also CC @daniellok-db shall we enable it globally already?

hubertzub-db avatar Apr 18 '24 08:04 hubertzub-db

i think it's probably safe to enable, we can do it next time we update the UI!

daniellok-db avatar Apr 18 '24 23:04 daniellok-db

Great, thanks!

vilim avatar Apr 19 '24 07:04 vilim

@mlflow/mlflow-team Please assign a maintainer and start triaging this issue.

github-actions[bot] avatar Apr 25 '24 00:04 github-actions[bot]

Let's close it as soon as we enable phase3 features then

hubertzub-db avatar Apr 25 '24 05:04 hubertzub-db