Accessing Serial Port on Windows Causes JRE Crash

[retiutut]

Description

OpenBCI/OpenBCI_GUI#674

Expected Behavior

Access Serial ports directly on Windows without the JRE crashing.

Current Behavior

Crashes and produces a log in the application directory. Happens on Standalone app and processing-java.

Steps to Reproduce

1. Download OpenBCI GUI v4.2.0 pre-release.
2. Open GUI, and hopefully you have some serial ports to populate the list (e.g. COM1, COM2, etc.)
3. In the control panel, click: Live(from Cyton)-->Serial(from Dongle)-->Manual > -->Any Radio Configuration Button

Your Environment

• Processing version: 4.0.0-alpha1
• Operating System and OS version: Windows 10 Pro - up to date
• Other information:

Possible Causes / Solutions

Probably crashing on the following line:

serial_output = new Serial(this, openBCI_portName, openBCI_baud); //open the com port

hs_err_pid11188.log
hs_err_pid2120.log

[retiutut]

Happens using the Processing-IDE and the example from the Processing docs also:

https://processing.org/reference/libraries/serial/Serial.html

[sampottinger]

Hey @retiutut thanks for posting this! I'll take a look. For folks following along at home, not reproducing on Mac but I'll give it a go on Windows in a minute.

[retiutut]

That's correct. Seem's to work just fine on Ubuntu Linux and OS X Catalina.

[sampottinger]

Was able to reproduce on Windows 10 (non JVM access violation!). We will dig in.

[sampottinger]

Looks like we need to update! https://github.com/java-native/jssc/releases/tag/v2.9.0 or https://github.com/akuhtz/spsw. I'll give it a shot. Just FYI... this fork seems like the active continuation of the now dead original.

[sampottinger]

See ... https://github.com/sampottinger/jssc. Looks like there's still some work to do here.

[benfry]

Thanks for looking into it, @sampottinger

[sampottinger]

Of course! A quick update... I think I sorted it on windows but have mac build issues. Hoping to have more soon.

[sampottinger]

Blocked on #73

[sampottinger]

Yay! See the JSSC migration branch. I've tested for Linux 64 bit, Windows 64 bit, and Mac.

[benfry]

Merged the fix from Sam for alpha 2.

[retiutut]

Awesome! Thanks!

[retiutut]

@benfry When will alpha 2 release be published?

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.