Debug view occasionally lists representation coordinates out of order #41

New issue

Closed

opened 2025-01-31 08:40:11 +00:00 by Vectornaut · 3 comments

Vectornaut commented

2025-01-31 08:40:11 +00:00

Collaborator

For debugging, the outline currently shows each element's representation vector. Unfortunately, it sometimes shows the coordinates out of order.

This may be a bug in Sycamore's Indexed view, which I've inquired about on the Sycamore help forum. I have a workaround ready to pull-request whenever it's convenient. That pull request should commute with #34.

Reproduction

You can reproduce this bug in many ways. Here's one that works especially consistently.

Load the "General" test assembly
Set the Lorentz product between Deimos and Pollux to 1
Select Deimos and then focus the display
Tap W repeatedly while watching the coordinates of Pollux
- You'll occasionally see the last two coordinates switch places

For debugging, the outline currently shows each element's representation vector. Unfortunately, it sometimes shows the coordinates out of order. This may be a bug in Sycamore's `Indexed` view, which I've [inquired](https://discord.com/channels/820400041332179004/1334778981328752680/1334778981328752680) about on the Sycamore help forum. I have a workaround ready to pull-request whenever it's convenient. That pull request should commute with #34. #### Reproduction You can reproduce this bug in many ways. Here's one that works especially consistently. 1. Load the "General" test assembly 2. Set the Lorentz product between Deimos and Pollux to 1 3. Select Deimos and then focus the display 4. Tap **W** repeatedly while watching the coordinates of Pollux * *You'll occasionally see the last two coordinates switch places*

glen commented

2025-01-31 19:44:02 +00:00

Owner

With #34 merged, feel free to PR away.

Vectornaut commented

2025-01-31 19:52:17 +00:00

Author

Collaborator

Will do. The Sycamore maintainer responded to my forum post, so I'll follow up on that first in case there's a cleaner, simpler fix.

Vectornaut commented

2025-02-06 18:40:51 +00:00

Author

Collaborator

Based on my conversation with the Sycamore maintainer, it seems like this is indeed a bug in Indexed. I'll pull-request my workaround as soon as #43 is reviewed and merged.

Based on my conversation with the Sycamore maintainer, it seems like this is indeed a bug in `Indexed`. I'll pull-request my workaround as soon as #43 is reviewed and merged.