Skip to content
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.

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

Update to Processing 4 beta 3 CI pipeline #1019

Merged
merged 9 commits into from
Feb 1, 2022

Conversation

retiutut
Copy link
Member

@retiutut retiutut commented Dec 8, 2021

The purpose of this PR is to update the entire GUI to use Processing 4 and OpenJDK 11 (aka Java 11). Processing 4 is now in the beta stage and is much more stable than it was a year ago. I have been testing this periodically since the first Processing 4 release, and it seems it is finally time to make this change for the GUI.

@retiutut retiutut requested review from Andrey1994 and removed request for Andrey1994 December 8, 2021 17:38
@retiutut
Copy link
Member Author

retiutut commented Dec 8, 2021

@Andrey1994 Can you upload the latest BrainFlow jar that supports all 3 OS and Java 11 to this branch?

@retiutut retiutut added this to the 5.1.0 milestone Dec 8, 2021
@Andrey1994
Copy link
Contributor

link to the latest jar to try https://github.com/brainflow-dev/brainflow/releases/download/4.8.2/brainflow-jar-with-dependencies.jar

@retiutut
Copy link
Member Author

#1025

@retiutut retiutut changed the title Update to Processing 4 beta 2 CI pipeline Update to Processing 4 beta 3 CI pipeline Jan 21, 2022
@retiutut
Copy link
Member Author

retiutut commented Jan 22, 2022

@Andrey1994 I have tested each board, playback mode, synthetic mode, and the base processing.serial.Serial class for Cyton dongle coms.

I did not test the WiFi shield as it is discontinued at this time and probably works also.

Processing 4 beta 3 uses Java 17 and JavaFX 17, whereas this PR began with Processing 4 beta 2 which uses Java 11. I hope this should be ok with forward compatibility.

@retiutut retiutut merged commit de50fa7 into development Feb 1, 2022
@retiutut retiutut deleted the update-processing4beta2 branch April 28, 2022 18:03
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