Skip to content

fix: Flores200 data reading issue#179

Merged
tfburns merged 3 commits intomainfrom
flores_fix
Feb 13, 2026
Merged

fix: Flores200 data reading issue#179
tfburns merged 3 commits intomainfrom
flores_fix

Conversation

@prabhuteja12
Copy link
Contributor

@prabhuteja12 prabhuteja12 commented Feb 12, 2026

PR Checklist

  • Use descriptive commit messages.
  • Provide tests for your changes.
  • Update any related documentation and include any relevant screenshots.
  • Check if changes need to be made to docs (README or any guides in /docs/).

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Flores200 issue is fixed by directly accessing the parquet files for each subject in the fixed hf commit id.

Added/updated tests?

  • Yes
  • No, and this is why: Correctness checked by ensuring existing tests pass.
  • I need help with writing tests

@prabhuteja12 prabhuteja12 marked this pull request as ready for review February 12, 2026 10:49
Copy link
Contributor

@fsschneider fsschneider left a comment

Choose a reason for hiding this comment

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

LGTM!

@tfburns tfburns merged commit 9bf3155 into main Feb 13, 2026
36 of 37 checks passed
@tfburns tfburns deleted the flores_fix branch February 13, 2026 08:58
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