feat(studio): Data sample panel#187
Conversation
Signed-off-by: Sean Teramae <steramae@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds BigInt-safe Parquet row serialization; shows a centered Spinner and early-error Banner in FileSamplingSnippet; adds DatasetSamplePanel with tests; integrates the panel into FilesetCard using ResizeablePanel; and tweaks route/container sizing and a test assertion. ChangesDataset file preview and sampling
Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
@htolentino-nvidia taking a look! |
Signed-off-by: Sean Teramae <steramae@nvidia.com>
Fixed by removing one scroll bar on the outer container (the inner data view still needs scroll bars) |



This PR adds a data sample to the bottom right panel in order to support quick previews of the data in the dataset card case.
This also slightly changes the panel layout in the FilesetCard component. It adds a Resizeable panel so users can choose which view to focus on (README or the dataset sample)
Summary by CodeRabbit
New Features
Bug Fixes
Improvements
Tests