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

Make developer mode activation readme more comprehensible. #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Stefan-Kosker
Copy link

@Stefan-Kosker Stefan-Kosker commented Sep 12, 2023

This one just makes the activation of developer mode a little bit easier :)

@icza
Copy link
Owner

icza commented Sep 12, 2023

While this makes it obviously clearer, for someone having developer experience, this makes it a "piece of cake". I wanted it to have a little challenge, while still having all the info in the help text.

I stopped developing and updating Scelight a long time ago, if I make a new release, I'll consider including this (or a variation of it). Thanks.

@icza
Copy link
Owner

icza commented Sep 12, 2023

Also note that googling the term "scelight developer challenge solution", the first result leads to the StackOverflow question containing the solution code.

@Stefan-Kosker
Copy link
Author

I totally understand what you mean. But consider, I first get the max number, took the hash and than did the bitshift & xor.

Challenging =/= not comprehensive. But thanks mate! We will stay in contact :) I am very interested in unpacking the information from hex and try to understand the way single information gets handled.

@Stefan-Kosker
Copy link
Author

Stefan-Kosker commented Sep 12, 2023

Also note that googling the term "scelight developer challenge solution", the first result leads to the StackOverflow question containing the solution code.

image
Please consider, google results are highly personalized :)

@icza
Copy link
Owner

icza commented Sep 12, 2023

Yes, I know, that's why I also checked in a private browsing tab, and it's the first result:
image

@icza
Copy link
Owner

icza commented Sep 12, 2023

And as to what's clear and not in the test: you made several mistakes by not being thoughtful or observant. E.g. the 8-bit vs 8-byte, shifting by a wrong number...

All I'm saying is that if you hadn't make these mistakes in the first place, you could've found the solution earlier, even without help. If you do everything right except calculating SHA checksum first, that's still just 2 attempts.

I agree that the wording is not the clearest it can be, but I also didn't want to make it a routine coding task.

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