Skip to content

Display dataset name instead of ID in experiments table#308

Merged
cristian-tamblay merged 3 commits into
developfrom
fix/experiments-table-dataset-name
Sep 25, 2025
Merged

Display dataset name instead of ID in experiments table#308
cristian-tamblay merged 3 commits into
developfrom
fix/experiments-table-dataset-name

Conversation

@Creylay
Copy link
Copy Markdown
Collaborator

@Creylay Creylay commented Sep 24, 2025

Summary

Improved user experience in the experiments table by displaying readable dataset names instead of numeric IDs. This enhancement makes it easier for users to identify which dataset is associated with each experiment without needing to memorize or cross-reference ID numbers.

Type of change

  • Frontend fix.

Changes

  • Enhanced experiments table display: Modified the dataset column to show dataset names instead of IDs using a valueFormatter that maps IDs to their corresponding names
  • Added dataset data integration: Implemented dataset fetching in the Experiments page to provide the necessary data for name mapping
  • Improved data flow: Added datasets as a prop to ExperimentsTable component with proper PropTypes validation
  • Robust fallback handling: Added fallback display for cases where dataset lookup fails, showing descriptive text with the ID

How to Test

  1. Navigate to experiments page:

    • Go to the experiments section of the application
    • Verify that the experiments table loads correctly
  2. Verify dataset name display:

    • Check that the "Dataset" column shows readable names (e.g., "Customer Analysis", "Sales Data") instead of numeric IDs
    • Confirm that all experiments show their associated dataset names properly
  3. Test fallback behavior:

    • If possible, create a scenario where a dataset ID doesn't have a corresponding name
    • Verify that it displays "Dataset ID: [number]" as fallback

Screenshots

...

Notes

...

@Creylay Creylay requested a review from Copilot September 24, 2025 20:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the user experience in the experiments table by displaying human-readable dataset names instead of numeric IDs, making it easier for users to identify datasets associated with each experiment.

  • Added dataset fetching functionality to the Experiments page
  • Modified the experiments table to display dataset names using a valueFormatter
  • Added proper PropTypes validation for the new datasets prop

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
DashAI/front/src/pages/experiments/Experiments.jsx Added dataset fetching logic and passed datasets to ExperimentsTable component
DashAI/front/src/components/experiments/ExperimentsTable.jsx Enhanced dataset column with valueFormatter to display names instead of IDs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread DashAI/front/src/pages/experiments/Experiments.jsx Outdated
Comment thread DashAI/front/src/components/experiments/ExperimentsTable.jsx Outdated
@cristian-tamblay cristian-tamblay merged commit bbeb00c into develop Sep 25, 2025
5 checks passed
@cristian-tamblay cristian-tamblay deleted the fix/experiments-table-dataset-name branch September 25, 2025 16: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.

3 participants