Skip to content

Fix/replay3d corrupted file#576

Open
anushka11p wants to merge 3 commits into
Somil450:mainfrom
anushka11p:fix/replay3d-corrupted-file
Open

Fix/replay3d corrupted file#576
anushka11p wants to merge 3 commits into
Somil450:mainfrom
anushka11p:fix/replay3d-corrupted-file

Conversation

@anushka11p

Copy link
Copy Markdown
Contributor

🚀 Pull Request Template

📌 Related Issue

Fixes #523


📝 Description

  1. Restored src/components/Replay3DModel.tsx to a clean working state.
    2.Removed duplicate useTheme import in src/components/WelcomeScreen.tsx
    3.Removed duplicate CameraErrorBoundary import in src/components/WorkoutScreen.tsx

🔹 What has been changed?

Replay3DModel.tsx was corrupted by bad merges that introduced duplicate code blocks and type declarations inside function bodies, causing parse errors that rendered the entire app as a blank page. This PR restores the file to the last clean version committed by the maintainer before the breaking changes were introduced.

🔹 Why are these changes needed?

Any developer who clones or syncs from main currently gets a completely broken app — blank page, parse errors, nothing works. This makes it impossible for new contributors to set up the project locally and start contributing. Restoring these files ensures that anyone who forks or syncs the repo gets a fully functional development environment out of the box.

🛠️ Type of Change

Please mark the relevant option(s):

  • 🐛 Bug Fix
  • ✨ New Feature
  • 🎨 UI/UX Improvement
  • 📖 Documentation Update
  • ⚡ Performance Improvement
  • 🔧 Refactoring
  • 💥 Breaking Change

🧪 Testing

Describe how you tested your changes.

✅ Tests Performed

  • Tested locally
  • Build runs successfully
  • No console errors
  • Responsive design checked
  • Existing functionality verified

🌐 Browsers Tested

  • Chrome
  • Brave
  • Firefox
  • Edge
  • Safari

📷 Screenshots / Demo (if applicable)

Screenshot 2026-05-29 at 12 57 33 PM

📋 Checklist

Before submitting this PR, please confirm the following:

  • I have read the project's CONTRIBUTING guidelines
  • My code follows the project style guidelines
  • I have performed a self-review of my code
  • I have tested my changes locally
  • I have added/updated documentation where necessary
  • My changes do not introduce new warnings or errors
  • This PR is linked to an existing issue

💬 Additional Notes

The duplicate imports in WelcomeScreen.tsx and WorkoutScreen.tsx were side effects of the same bad merges and have been cleaned up in this PR as well.

@vercel

vercel Bot commented May 29, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the somiljain2024-4175's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Somil450 Somil450 self-requested a review May 29, 2026 10:57
@Somil450 Somil450 self-assigned this May 29, 2026
@Somil450 Somil450 added gssoc-26 Marks GSSoC issues level:advanced Complex contribution requiring strong technical knowledge and architectural understanding. gssoc:approved Officially reviewed and approved GSSoC contribution ready for scoring mentor:Somil450 Reviewed and mentored by Somil450 for GSSoC contribution tracking. type:bug labels May 29, 2026
@Somil450 Somil450 assigned anushka11p and unassigned Somil450 May 29, 2026
@Somil450

Somil450 commented Jun 3, 2026

Copy link
Copy Markdown
Owner

@anushka11p you remove +650 lines of code so can u plz provide with the ss so can check not remove any imp thing

@anushka11p

Copy link
Copy Markdown
Contributor Author

Hi @Somil450 ! Here's a breakdown of what was removed:

The -678 lines were from Replay3DModel.tsx which had duplicate/corrupted code blocks introduced by bad merges. Specifically removed:

  • Duplicate type declarations (StressVectorRig, RippleEvent, BoneProportionKey etc.) that appeared twice
  • Duplicate function bodies (createStressVectorMaterial, createRippleGridMaterial, updateStressVectors etc.)
  • Duplicate ref declarations (restDataRef, bonesRef etc.)
  • The updateSegmentScaleAdaptor function and related proportion scaling logic

However, looking at the CI failures, I can see frame?.repCount was changed to frame.repCount causing Cannot find name 'frame' errors. I'll fix these in a new commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved Officially reviewed and approved GSSoC contribution ready for scoring gssoc-26 Marks GSSoC issues level:advanced Complex contribution requiring strong technical knowledge and architectural understanding. mentor:Somil450 Reviewed and mentored by Somil450 for GSSoC contribution tracking. type:bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: Replay3DModel.tsx broken by bad merge — duplicate code blocks and type declarations outside function scope

2 participants