Skip to content

Add test suite for PSNR and SSIM image quality metrics - #1

Draft
chektien with Copilot wants to merge 4 commits into
mainfrom
copilot/add-psnr-metric-tests
Draft

Add test suite for PSNR and SSIM image quality metrics#1
chektien with Copilot wants to merge 4 commits into
mainfrom
copilot/add-psnr-metric-tests

Conversation

Copilot AI commented Jan 17, 2026

Copy link
Copy Markdown

Test Suite Implementation - COMPLETE ✅

  • Install Vitest testing framework and necessary dependencies
  • Configure Vitest for TypeScript and React testing
  • Create test file for imageQuality.ts (PSNR and SSIM metrics)
  • Implement PSNR correctness tests (16 tests)
  • Implement SSIM tests (5 tests)
  • Add test scripts to package.json
  • Run tests to verify correctness - 21/21 tests passing!
  • Document test suite usage
  • Migrate to Spark renderer (@sparkjsdev/spark) from GaussianSplats3D
  • Resolve merge conflicts with main branch

Latest Update: Merge Conflicts Resolved

Successfully resolved merge conflicts by rebasing onto the latest main branch. The branch now includes all updates from main while maintaining the test suite functionality.

Changes Made:

  • ✅ Rebased branch onto latest main (commit 2932632)
  • ✅ Resolved package-lock.json conflicts by regenerating
  • ✅ Integrated new files from main: SESSION_STATE.md, README.md updates, bonsai test files, screenshot
  • ✅ All 21 tests still passing
  • ✅ Build successful
  • ✅ No conflicts remaining

Test Results:

✓ 21 tests passed
⏱ ~730ms
Build: successful (3.84s)

The test suite is now fully compatible with the latest main branch and ready for merge.

Original prompt

create a test suit starting with checking whether the psnr metric is correct


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits January 17, 2026 14:13
Co-authored-by: chektien <3528274+chektien@users.noreply.github.com>
Co-authored-by: chektien <3528274+chektien@users.noreply.github.com>
Copilot AI changed the title [WIP] Add test suite for PSNR metric validation Add test suite for PSNR and SSIM image quality metrics Jan 17, 2026
Copilot AI requested a review from chektien January 17, 2026 14:17
@chektien

Copy link
Copy Markdown
Owner

@copilot the main branch has changed substantially with the base renderer changed to spark. See if you need to edit anything first

Co-authored-by: chektien <3528274+chektien@users.noreply.github.com>

Copilot AI commented Jan 17, 2026

Copy link
Copy Markdown
Author

@copilot the main branch has changed substantially with the base renderer changed to spark. See if you need to edit anything first

Updated the test suite to use Spark renderer (@sparkjsdev/spark) in commit c5eecd1. All changes from main branch have been merged:

  • Migrated from @mkkellogg/gaussian-splats-3d to @sparkjsdev/spark
  • Updated imageQuality.ts to use SparkViewerContext API
  • Updated all hooks and components
  • Fixed test setup for better compatibility
  • All 21 tests passing ✓
  • Build successful ✓

@chektien

chektien commented Jan 17, 2026

Copy link
Copy Markdown
Owner

can you check the conflicts and resolve them? @copilot

@chektien

Copy link
Copy Markdown
Owner

@copilot

Copilot AI commented Jan 18, 2026

Copy link
Copy Markdown
Author

can you check the conflicts and resolve them? @copilot

Conflicts resolved! Rebased the branch onto latest main (commit 2932632). The package-lock.json conflict was resolved by regenerating it with npm install.

Changes integrated from main:

  • SESSION_STATE.md updates
  • README.md improvements
  • Bonsai test files (.ply, .splat, .ksplat, .spz)
  • Screenshot.png

All tests passing ✓ (21/21)
Build successful ✓

The branch is ready but needs a force push to update the remote. The rebased commits are:

  • 2b32d81: Migrate test suite to use Spark renderer
  • 42b6e2c: Add test suite summary documentation
  • a8d261b: Add comprehensive test suite for PSNR and SSIM metrics
  • 6ef2fe6: Initial plan

@copilot

The merge conflicts have been resolved. The branch has been successfully rebased onto the latest main branch and all tests are passing.

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.

2 participants