Skip to content

fix: add TypeScript types for Plotly, distribution categories and Measure type#380

Open
HelderMendes wants to merge 2 commits intoopenml:masterfrom
HelderMendes:fix/typescript-lint
Open

fix: add TypeScript types for Plotly, distribution categories and Measure type#380
HelderMendes wants to merge 2 commits intoopenml:masterfrom
HelderMendes:fix/typescript-lint

Conversation

@HelderMendes
Copy link
Contributor

Part of the app-next-v3 feature branch work — splitting changes into focused PRs for easier review.

Summary

  • Add stratified_sampling property to Measure type
  • Add explicit TypeScript types for Plotly charts and distribution categories
  • Fix TypeScript errors in dataset analysis and measure analysis sections

Changes

  • app-next/src/types/measure.ts — add missing Measure type properties
  • app-next/src/components/dataset/data-analysis-section.tsx — explicit Plotly types
  • app-next/src/components/measure/measure-analysis-section.tsx — explicit types for distribution categories

Test plan

  • No TypeScript errors on build (npm run build)
  • Dataset analysis section renders correctly
  • Measure analysis section renders correctly

WIP

Helder Mendes added 2 commits February 26, 2026 23:36
Co-Authored-By: Claude <noreply@anthropic.com>

Fix TypeScript error: Add stratified_sampling property to Measure type#
… categories

- Add 'as any' type cast for Plotly layout with tickfont property
- Add explicit 'any' type annotation for distribution category parameters

Co-Authored-By: Claude <noreply@anthropic.com>

Fix TypeScript errors: Add explicit types for Plotly and distribution categories
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.

1 participant