Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Before:
With fix:
This pull request includes updates to various files to ensure proper lowercase conversion using the appropriate locale. The most important changes include updating the
toLowerCase()
method totoLocaleLowerCase()
inInsights.tsx
,getFilteredData.ts
, andModelAssessmentDashboard/Controls/ModelOverview/CohortStatsHeatmap.tsx
.Main changes:
libs/fairness/src/lib/Controls/Insights.tsx
: UpdatedtoLowerCase()
method totoLocaleLowerCase()
forthis.props.selectedMetric.title
andthis.props.selectedFairnessMetric.title
to ensure proper lowercase conversion using the appropriate locale. [1] [2]libs/interpret-vision/src/lib/VisionExplanationDashboard/utils/getFilteredData.ts
: UpdatedtoLowerCase()
method totoLocaleLowerCase()
to ensure proper lowercase conversion of labels and search value.libs/model-assessment/src/lib/ModelAssessmentDashboard/Controls/ModelOverview/CohortStatsHeatmap.tsx
: UpdatedtoLowerCase()
method totoLocaleLowerCase()
forthis.series.xAxis.categories[this.point.x]
to convert the category value to lowercase using the appropriate locale.GitHub Chat Prompt
Response snippet:
Checklist