Skip to content

#198 - Enable SWC for Neurons Fixed - #171

Merged
ddelpiano merged 7 commits into
developmentfrom
feature/issue-198
Jul 1, 2025
Merged

#198 - Enable SWC for Neurons Fixed#171
ddelpiano merged 7 commits into
developmentfrom
feature/issue-198

Conversation

@jrmartin

Copy link
Copy Markdown
Collaborator

Fixes bug #198, enables SWC for Neurons.

@jrmartin
jrmartin requested a review from ddelpiano June 29, 2025 15:33
@jrmartin
jrmartin marked this pull request as ready for review June 29, 2025 15:33
@jrmartin
jrmartin requested a review from Copilot June 29, 2025 15:33

This comment was marked as outdated.

@jrmartin
jrmartin requested a review from Copilot June 29, 2025 15:43

This comment was marked as outdated.

…vas.js

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jrmartin
jrmartin requested a review from Copilot June 29, 2025 15:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request fixes bug #198 by enabling SWC support for Neurons.

  • Updated logic in the InstancesReducer to correctly locate and update a matching 3D object.
  • Expanded and refined the SWC file fetching and error handling in the ThreeDCanvas component.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
applications/virtual-fly-brain/client/src/reducers/InstancesReducer.js Changed the iteration method from forEach to find to correctly update the matching object's visibility.
applications/virtual-fly-brain/client/src/components/ThreeDCanvas.js Improved SWC file fetching logic including enhanced URL determination and error handling.

jrmartin and others added 2 commits June 29, 2025 08:46
@ddelpiano
ddelpiano merged commit dd54c8d into development Jul 1, 2025
@ddelpiano
ddelpiano deleted the feature/issue-198 branch September 10, 2025 10:12
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