Skip to content

Conversation

@meganrm
Copy link
Contributor

@meganrm meganrm commented Jul 30, 2025

Time Estimate or Size

xsmall

Problem

when the education app switched trajectories there are certain conditions where the 3D trajectory doesn't render.

Solution

made sure to clear all the agents when clearing a file

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

@meganrm meganrm requested a review from a team as a code owner July 30, 2025 00:43
@meganrm meganrm requested review from ShrimpCryptid and toloudis and removed request for a team July 30, 2025 00:43
@github-actions
Copy link

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 23.07% 2583 / 11196
🔵 Statements 23.07% 2583 / 11196
🔵 Functions 25% 149 / 596
🔵 Branches 80.41% 349 / 434
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/visGeometry/index.ts 7.98% 100% 0% 7.98% 116-124, 202-295, 298-299, 302-315, 323-332, 336-351, 354-363, 365-380, 383-504, 507-512, 516-532, 535-544, 547-548, 551-552, 556-567, 571-574, 578-598, 601-603, 606-609, 613-645, 648-650, 653-655, 658-675, 678-679, 682-683, 686-695, 698-699, 702-725, 728-729, 732-734, 737-739, 742-743, 746-749, 752-780, 783-807, 812-819, 823-833, 836-866, 869-902, 905-931, 934-939, 942-943, 946-947, 951-991, 994-1106, 1110-1117, 1120-1136, 1139-1144, 1147-1153, 1156-1167, 1173-1177, 1180-1186, 1189-1191, 1194-1257, 1260-1263, 1266-1400, 1403-1413, 1416-1440, 1443-1445, 1448-1456, 1460-1464, 1467-1487, 1490-1527, 1530-1559, 1565-1707, 1710-1788, 1791-1796, 1800-1830, 1833-1840, 1843-1844, 1847-1878, 1881-1884, 1887-1895, 1898-1899, 1902-1906, 1911-1921, 1924-1945, 1948-1949
Generated in workflow #616 for commit 2831cd1 by the Vitest Coverage Report Action

@meganrm meganrm merged commit 80d0546 into main Aug 1, 2025
6 checks passed
@meganrm meganrm deleted the fix/switching-trajectories branch August 1, 2025 00:29
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.

4 participants