Skip to content

[Draft] Library Rewrite with API accessibility in mind #1

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
Draft
wants to merge 160 commits into
base: master
Choose a base branch
from
Draft

[Draft] Library Rewrite with API accessibility in mind #1

wants to merge 160 commits into from

Conversation

@JustArion
Copy link
Owner

@JustArion JustArion commented Oct 11, 2024 •
edited
Loading

WIP

JustArion added 30 commits April 9, 2024 19:13
@JustArion
Branch Commit
21e31e7 
- Added the ClangSharp binding generation to the repo
- Added some unit tests
- Rewriting under Dawn.CorsairSDK.Rewrite
@JustArion
Added binaries to Git LFS
4f315da
@JustArion
Log Level Asterisk
c75c3ac
@JustArion
Update .editorconfig
a59cc3d
@JustArion
Don't set colors that are already the same color
50da517
@JustArion
Malding on trying to implement xy modulation
8e311f5
@JustArion
Delete src/Testing directory
f09139a
@JustArion
Revert to Wave Function
71d3527
@JustArion
Effect Receipt & Bugfix
468da8c 
- EffectReceipt implementation (Task + IDisposable in 1)
- Fix: Non-Infinite was animating infinitely
@JustArion
Tests use the latest language version
06f3c69
@JustArion
Implemented some tests, and as a result, fixed some bugs
fa86a17
@JustArion
Build now follows the [*] log style
b9328d4
@JustArion
Comments -> Labels
2a295cb
@JustArion
Refactored some logs to their appropriate places
cabb420
@JustArion
Added repo analytics for Master branch
15cb2b8
@JustArion
Added support for the keyboard layout enums
4c70b19 
The layouts were previously displayed as integers.
@JustArion
Device Layout enums
ce70e5c
@JustArion
Don't set effects if there's 100% no visual change
2e8d72e
@JustArion
Update
b26da28
@JustArion
Device specific properties added / improved on
6cdfe6b
@JustArion
Battery Level Change from method -> property fix
4d75b18
@JustArion
BatteryLevel is updated based on config
a417fda
@JustArion
We should maybe do that to the other properties too, since those stat…
e9b8743 
…es can change
@JustArion
Welcome Back Explosion Migrated to Rewrite
5505765
@JustArion
SDK Resolver uses App Directory as the first check
20cba68
@JustArion
Random Color
b93e37c
@JustArion
Sample Remote Control Host Implemented
64c5b0e
@JustArion
Sample Remote Control Client Implemented
eb5433d
@JustArion
SLN addition
9eb3712
@JustArion
Switch to spans for unmanaged copies and reads
3b198b5
@JustArion
Added IsVertical & Added Color customization to progress animation op…
42f6aab 
…tions
@JustArion
Pulses and Flashes can now use Smooth Linear Interpolation (Slerp) in…
831346c 
…stead of Linear Interpolation (Lerp)

Fix: Added default Wave Modulation
@JustArion
Expose Device in CorsairSDK.KeyboardLighting
e9b08cd
@JustArion
Move ColorEffects helper to ColorUtility
00a76fd
@JustArion
Effect Receipt is now a class
a523e6a
@JustArion
Add Device to contract
39b0b1f
@JustArion
Improve internal state tracking for device control
5d38724
@JustArion
Improve Animation extensibility
cc3bee9
@JustArion
Added additional Utility Class
687d52e
@JustArion
WIP push for Progress Animation (non functional)
ff3a760
@JustArion
Set Log Level for Test script to minimal
01d5a56
@JustArion
Refactor for Keyboard Key namings & Bugfix on keyboard zones
b4be654 
KeyboardKey -> KeyboardKeyState (It contains the color of the respective keyboard key)
KeyboardKeys -> KeyboardKey

---

Bugfix: Keyboard Zones
Zones were hardcoded keys. Now they're somewhat more dynamic
@JustArion
Update Tests
a4e0139
@JustArion
Refactor GetDeviceAs<T> to GetDevice<T>
f37cd6e
@JustArion
Allow initialization of LightingInterop for a device even if there's …
f066a7e 
…no Leds associated with that device
@JustArion
Fix NRE
7066fe9
@JustArion
Added G-Keys and refactored the naming style of Keyboard Keys from UP…
8483865 
…PERCASE to PascalCase
@JustArion
Refactor
ec8a8d4
@JustArion
Create DomainImplOf.cs
1bb0a19
@JustArion
Add new attribute
dba1287
@JustArion
Added LedGroup (From Corsair)
218d6cb
@JustArion
Core Performance Improvement
41525ee 
The commit is an API change for the sake of big O performance.
From roughly O(n * m) -> O(n)
@JustArion
Update readme.md
619ab38
@JustArion
Include other access levels
68f83c8
@JustArion
Rename ScanAnimation to WaveAnimation to be more aligned with iCUE te…
03f0631 
…rminology
@JustArion
Update ProgressAnimation.cs
44bfec5
@JustArion
Improve thread safety to receipts
2ff7ad3
@JustArion
Remove accessibility modifier requirement
9651a8b
@JustArion
We don't permit Java style brackets here...
d035e0a
@JustArion
Expose for tests
348663f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@JustArion JustArion

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JustArion