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

Examples folder missing from SDK package 4.0.48 #34

Open
eSonOfAnder opened this issue Sep 6, 2023 · 4 comments
Open

Examples folder missing from SDK package 4.0.48 #34

eSonOfAnder opened this issue Sep 6, 2023 · 4 comments

Comments

@eSonOfAnder
Copy link

As of 6-Sep 2023, the current version of the iCUE SDK package is 4.0.48. The included PDF documentation says that there should be an examples folder in this archive with a sample project, but this folder does not exist. If this was removed intentionally, the document should be updated to reflect that (and it should also be included in the release notes). The better solution would be to add the sample project back to the distributed SDK package, as documented.

@hasteagag
Copy link

3.0.464 appears to be the version publicly available on the corsair site. Also a note, the PDF in the docs in the latest release on github has the previous protocol version (15) so I would guess that is out of date should anything have changed. I am tempted to check if swapping the lib etc has any effect on the examples, but I havent yet

@hasteagag
Copy link

Well I had a look at the last commit, and it looks like a lot changed. Groups seem like a great addition, to categorize entire sections of a lighting setup (CorsairLedGroup) and the overall enum looks like it was dramatically simplified rather than a bunch of slocs of incremented Leds. I havent spotted it but it must be done as needed or more efficiently. So I think that would break many of the lighting examples out of the box, so thats probably why it was dropped. I suspect it will return soon.

@eSonOfAnder
Copy link
Author

Interesting, because I was not sure where to grab the SDK from, and if you do a Google search for it, you will end up here and not on the Corsair site. So if the "latest release" here is not actually the "latest release", then it should be labeled as such, or mentioned in the readme.md. The original bug, as reported, is still valid as is, because 4.0.48 is listed as the latest release, and it clearly is missing both what I reported, along with all of the undocumented changes mentioned above.

@hasteagag
Copy link

hotlink to corsair site for prior release for convenience: https://downloads.corsair.com/Files/CUE/CUESDK_3.0.464.zip

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

No branches or pull requests

2 participants