Skip to content

Comments

Feat: add prototype for workshop-sensor-burst-analyzer#1071

Open
sebacodes wants to merge 9 commits intofreeCodeCamp:mainfrom
sebacodes:prototype/workshop-sensor-burst-analyzer
Open

Feat: add prototype for workshop-sensor-burst-analyzer#1071
sebacodes wants to merge 9 commits intofreeCodeCamp:mainfrom
sebacodes:prototype/workshop-sensor-burst-analyzer

Conversation

@sebacodes
Copy link

@sebacodes sebacodes commented Dec 12, 2025

Checklist:

relates to freeCodeCamp/freeCodeCamp#64114

@jdwilkin4
Copy link
Contributor

Is this ready for review?
Or is this still being worked on?

If it is ready, please bring it out of draft status

- Change summarizedData from array `[[],[],[]]` to object `{"min":[],"max":[],"avg":[]}`
  to match prototype specifications
- Round average values in summarizeSensorData for cleaner output
- Use object property notation (summarizedData.min.push, and so on.) instead of array indices
- Fix minor typos and improve comments
@sebacodes sebacodes marked this pull request as ready for review January 29, 2026 03:32
@sebacodes
Copy link
Author

Status changed, ready for review

Copy link
Contributor

@jdwilkin4 jdwilkin4 left a comment

Choose a reason for hiding this comment

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

I am not sure I understand the learning objectives in this workshop.

Is it to practice arrays?
working with loops?
something else?

where is this supposed to go in the curriculum?

I am going to tag Naomi to review this 👍🏾

- file do not belongs to the root directory
- file goes on fullstack-cert/js-projects/sensor-burst-analyzer/ directory but is not being use
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

2 participants