Skip to content

Feat 2cores#301

Open
nitin710 wants to merge 16 commits intomasterfrom
feat-2cores
Open

Feat 2cores#301
nitin710 wants to merge 16 commits intomasterfrom
feat-2cores

Conversation

@nitin710
Copy link
Collaborator

@nitin710 nitin710 commented Mar 9, 2024

Description

  • Data acquisition and Data processing now split into 2 tasks on 2 cores
  • Core 1
    • Data acquisition (read sensors)
    • Main loop
    • IDLE task (RTOS default. performs cleanup)
  • Core 2
    • update
    • Wifi
    • IDLE task (RTOS default. performs cleanup)

Requirements

  • None

Issues Referenced

  • None

Documentation update

  • This is a purely backend change. No user facing documentation needs updating.

Notes for Reviewer

  • None

Testing

Results

  • Synopsis of results, details if required.
  • See doc RTOS-2 core data collection timing analysis (emotibit) for timing analysis

Feature Tests

  • ❌ Short Button Press
  • ❌ Long Button Press
  • EmotiBit data acquisition time period accuracy with Streaming+recording
  • EmotiBit data acquisition time period accuracy with WiFi off+recording is
  • ToDo: Run a data integrity test
  • ToDo: catalog update loop timing

Shared files

Checklist to allow merge

  • All dependent repositories used were on branch master
  • Firmware
    • Set testingMode to TestingMode::NONE
    • Set const bool DIGITAL_WRITE_DEBUG = false (if set true while testing)
    • Update version in EmotiBit.h
    • Update library.properties to the correct version (should match EmotiBit.h)
  • doxygen style comments included for new code snippets
  • Required documentation updated

Screenshots:

@nitin710
Copy link
Collaborator Author

This PR is blocked because:

  1. Short/long button press seem to crash the ESP core.
  2. Seems like it has something to do with GPIO toggles during sleep raising an unhandled exception somewhere.

@nitin710 nitin710 added this to the Power Optimization-ESP32 milestone Dec 4, 2025
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.

1 participant