Skip to content

Analysis Side-Stats Bug Fix#228

Merged
DhritiKohli merged 1 commit intomainfrom
debug_analysis_sidie-stats
Jan 5, 2026
Merged

Analysis Side-Stats Bug Fix#228
DhritiKohli merged 1 commit intomainfrom
debug_analysis_sidie-stats

Conversation

@DhritiKohli
Copy link
Copy Markdown
Contributor

Added tmps in analysis-pipeline.json such that the average drops and misses could be depicted.

added tmps for the team object to work
@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 28, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
spot Error Error Dec 28, 2025 9:21pm
spot-rxzs Error Error Dec 28, 2025 9:21pm
spot-wnld Error Error Dec 28, 2025 9:21pm

Copy link
Copy Markdown
Contributor

@gcschmit gcschmit left a comment

Choose a reason for hiding this comment

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

Excellent debugging!

"type": "team",
"name": "sum",
"outputPath": "counts.totalAlgaeDrops",
"outputPath": "counts.dropAlgae",
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.

For a team, counts.dropAlgae isn't used anywhere; so, this block could be removed. Doesn't hurt to have it.

"type": "team",
"name": "sum",
"outputPath": "counts.totalCoralDrops",
"outputPath": "counts.dropCoral",
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.

For a team, counts.dropAlgae isn't used anywhere; so, this block could be removed. Doesn't hurt to have it.

@DhritiKohli DhritiKohli merged commit adfc1ae into main Jan 5, 2026
1 of 4 checks passed
@ACoderGuy1002 ACoderGuy1002 deleted the debug_analysis_sidie-stats branch March 12, 2026 13:42
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