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

Migrate to PlatformIO #18

Merged
merged 1 commit into from
Nov 8, 2022
Merged

Migrate to PlatformIO #18

merged 1 commit into from
Nov 8, 2022

Conversation

awawa-dev
Copy link
Owner

Major and very important upgrade of HyperSerialEsp8266. There are many reason that caused are migration from Arduino IDE to PlatformIO.

  • Much easier development
  • All needed boards and libraries are automatically resolved. The user doesn't need to install them manually.
  • Visual Studio Code is a much better editor
  • Support for unit testing
  • Github Action support. No more manual compilation of firmware with every release
  • single main HyperSerialEsp8266 source unit shared by all supported LED strips

Refactoring was primarily caused by latest v19 beta1 release, that includes some changes making old 'First LED enabled/disabled' version firmware obsolete. You can read more about it here: awawa-dev/HyperHDR#379 and you can disable/enable first LED in HyperHDR v19.

Also all floating point operations used for RGBW calibration were removed and integer operations were used instead . A special unit test guards the correctness of the new RGBW calculations so the backward compatibility is preserved. All related variables were encapsulated into separated structures to make the code more readable.

@awawa-dev awawa-dev merged commit 1399cf6 into master Nov 8, 2022
@awawa-dev awawa-dev deleted the platformio branch November 8, 2022 20:32
@awawa-dev awawa-dev mentioned this pull request Nov 27, 2022
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