Skip to content

Conversation

@kygm
Copy link
Collaborator

@kygm kygm commented Nov 27, 2025

During our testing first deployment drone+BCI testing, @Yash272001 and I found that on attempted read of current_data_mode in GUI5.py, an error occurred. We fixed this by initializing the data mode in the init function of BrainwavesBackend.
I also included a launch.json configuration to be able to line by line debug any python file in vscode.

@kygm kygm requested review from 3C-SCSU and Yash272001 November 27, 2025 07:46
Copy link
Owner

@3C-SCSU 3C-SCSU left a comment

Choose a reason for hiding this comment

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

Code approval process -- 3 Steps review:
(1) Yes; (2) Maybe; (3) No; > {Number of files, and total lines changed:add/delete lines} [Reasoning]
Example:

  • Feedback >> (2) > {4: +153/-20} [ Code needs to explain what it does ]

Copy link
Owner

@3C-SCSU 3C-SCSU left a comment

Choose a reason for hiding this comment

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

  • Feedback >> (1) > {2: +16/-0} [ No conflict or unexpected changes. Code approved. ]

@3C-SCSU 3C-SCSU merged commit 5c48bba into 3C-SCSU:main Nov 28, 2025
@3C-SCSU
Copy link
Owner

3C-SCSU commented Nov 28, 2025

Ticket solved: #615

@3C-SCSU 3C-SCSU changed the title First Drone+BCI Deployment - Initialized data mode and added launch.json configuration for debugging Deployment: automate debugging log Nov 28, 2025
@3C-SCSU 3C-SCSU changed the title Deployment: automate debugging log #615 - Deployment: automate debugging log Nov 28, 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.

2 participants