Skip to content

fix: handle dtype coercion when editing cells (closes #231)#232

Open
vikask011 wants to merge 2 commits intoc2siorg:mainfrom
vikask011:fix/change-cell-value-dtype-coercion
Open

fix: handle dtype coercion when editing cells (closes #231)#232
vikask011 wants to merge 2 commits intoc2siorg:mainfrom
vikask011:fix/change-cell-value-dtype-coercion

Conversation

@vikask011
Copy link
Copy Markdown

Description

Resolves a bug where updating cell values in columns with predefined data types (int, float, bool) results in a validation error. This occurred because values sent from the frontend are always strings, leading to incompatibility when directly assigning them to typed pandas columns.

The fix introduces proper type conversion before updating the DataFrame, ensuring compatibility with the column’s dtype and preventing runtime errors.

Closes #231

Type of Change

Bug fix

Testing

Tested with datasets containing integer, float, and boolean columns.
Modified cell values through the UI and verified successful updates without backend errors.
Confirmed no regressions in existing functionality and ensured existing tests pass.

Checklist

Code follows the project's style guidelines
Self-review performed
No new warnings generated
Tests pass locally

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 26, 2026

PR Review

Squash

Your PR has 2 commits. Please squash into a single commit.

How to fix

git fetch origin
git rebase -i origin/main   # mark all but first commit as "squash"
git push --force-with-lease

This comment updates automatically on each push.

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.

[Bug] Editing cells fails for numeric and boolean columns due to type mismatch

1 participant