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

Robot page exception in DatasetUtils where jointDataset.metaDict[property] is undefined #2148

Merged
merged 1 commit into from
Jun 27, 2023

Conversation

tongyu-microsoft
Copy link
Contributor

We were seeing two robot page exceptions where jointDataset.metaDict[property] is undefined and is not handled properly.

TypeError: TypeError: Cannot read properties of undefined (reading 'sortedCategoricalValues')
    at xa (../../../../node_modules/@responsible-ai/core-ui/core-ui.esm.js:5609:54)
    at pushMultilabelRowData (../../../../node_modules/@responsible-ai/core-ui/core-ui.esm.js:5499:8)
    at constructRows (../../../../node_modules/@responsible-ai/dataset-explorer/dataset-explorer.esm.js:2975:15)
    at new n (../../../../node_modules/@responsible-ai/dataset-explorer/dataset-explorer.esm.js:2830:27)
    at Bde (../../../../node_modules/react-dom/cjs/react-dom.production.min.js:135:171)
    at Mg (../../../../node_modules/react-dom/cjs/react-dom.production.min.js:181:145)
    at pi (../../../../node_modules/react-dom/cjs/react-dom.production.min.js:269:426)
    at ck (../../../../node_modules/react-dom/cjs/react-dom.production.min.js:250:346)
    at bk (../../../../node_modules/react-dom/cjs/react-dom.production.min.js:250:277)
    at ak (../../../../node_modules/react-dom/cjs/react-dom.production.min.js:250:137)

Description

Checklist

  • I have added screenshots above for all UI changes.
  • I have added e2e tests for all UI changes.
  • Documentation was updated if it was needed.

@tongyu-microsoft tongyu-microsoft changed the title Robot page exception Robot page exception in DatasetUtils where jointDataset.metaDict[property] is undefined Jun 26, 2023
@tongyu-microsoft tongyu-microsoft merged commit 9c5940b into main Jun 27, 2023
75 checks passed
@tongyu-microsoft tongyu-microsoft deleted the tongy/robotPage branch June 27, 2023 01:38
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.

4 participants