Skip to content

New Wearable Sensing App Release #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

New Wearable Sensing App Release #3

wants to merge 6 commits into from

Conversation

@tab-cmd
Copy link
Collaborator

@tab-cmd tab-cmd commented Jul 23, 2025 •
edited
Loading

Contributions

  • Fixed compilation issues and moved to CMake
  • Removed unhelpful logs
  • Add check impedance and Analog reset feature
  • Updated GUI
newLSLGUI

Deprecations

  • We are no longer supporting other OS. This release is currently compatible with Windows only. We may add support in the future.

Coming soon

Release file

https://github.com/WearableSensing/LSL/releases/download/v1.1/WearableSensingLSL-Windows.7z

note We are using 7zip due to some false flags by Windows Defender and other antivirus software using zip files.

erizheng and others added 4 commits July 16, 2025 15:04
@erizheng @isaiahtheant
resolved changes
5f002da 
* added cmake readme

* gitignore + impedance button

* changed README

* start analog reset command

* implemented impedance with button

* both check/reset impedance

* Update dsi2lsl.c

* added pull request changes

---------

Co-authored-by: isaiahtheant <[email protected]>
Co-authored-by: Isaiah <[email protected]>
@erizheng @isaiahtheant @tab-cmd
Multithreading (#2)
335688a 
* added cmake readme

* gitignore + impedance button

* changed README

* start analog reset command

* implemented impedance with button

* both check/reset impedance

* Update dsi2lsl.c

* Impedance start/stop + threading

* Update README.md

* temp remove ui button

* Updates to README and cmake list

* pull request changes

* Remove dsi-api-v1.18.2 from tracking (should be ignored)

* updated pull request changes

* Removed constant impedance printing

* Update README.md

* Update README.md

* Update README.md

* Standardized + More comments

* Revert changes

---------

Co-authored-by: isaiahtheant <[email protected]>
Co-authored-by: Isaiah <[email protected]>
Co-authored-by: Tab Memmott <[email protected]>
@erizheng @tab-cmd
New release
dfb9268 
* added cmake readme

* gitignore + impedance button

* changed README

* start analog reset command

* implemented impedance with button

* both check/reset impedance

* Update dsi2lsl.c

* added pull request changes

Co-authored-by: tab-cmd <[email protected]>
Co-authored-by: isaiahtheant <[email protected]>
Co-authored-by: erizheng <[email protected]>

remove old README file
@isaiahtheant @erizheng
compile with mingw64 (#5)
52addd4 
---------

Co-authored-by: Eric Zheng <[email protected]>
@cboulay
Copy link

cboulay commented Aug 1, 2025

Hi Tab,
Can you please remove the .vscode folder? Typically it's good to add this to your global .gitignore but if you want to add it to this repo's .gitignore then that's fine too.

@cboulay
Copy link

cboulay commented Aug 1, 2025

Can you also check to see how difficult it would be to upgrade to Qt6? It's been out for over 4 years now and Qt5 is increasingly harder to find and install. In my experience, if you aren't using any niche features of Qt then the upgrade is pretty smooth -- there's typically only 1 or 2 small API changes in the main widgets.

@cboulay
Copy link

cboulay commented Aug 1, 2025

Actually, the Qt6 thing can be a separate PR. Let's not complicate this PR too much. Sorry about that.

@cboulay
Copy link

cboulay commented Aug 2, 2025

@tab-cmd , I invited you as a collaborator on this repo. I also checked out the WearableSensing-master fork from which this PR originates and pushed it here as the WearableSensing-master branch in case you want to redo the PR entirely in this repo.
The main reason I did that, however, is because I am going to create an issue that explains a bug and I want to link to source code using something recent.

isaiahtheant and others added 2 commits August 11, 2025 10:43
@isaiahtheant @erizheng @tab-cmd
Time jitter fix and random string fix (#6)
083e847 
* compile with mingw64

* Update CMakeLists.txt

* Delete mingwbuild directory

* fixed getRandomString issue with it using same String

* fixed getRandomString issue with it using same String

* Change push_sample

* Fixed time jitter

* fixed mistake

* document CHUNK_SIZE

* fix linting issues

* refactor

* clean up and add documentation

---------

Co-authored-by: Eric Zheng <[email protected]>
Co-authored-by: Tab Memmott <[email protected]>
@erizheng
Merge branch 'master' of https://github.com/WearableSensing/App-Weara…
effdb79 
…bleSensing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tab-cmd @cboulay @erizheng @isaiahtheant