jsss icon indicating copy to clipboard operation
jsss copied to clipboard

Is this a WCA-approved scramble?

Open zaynv opened this issue 8 years ago • 6 comments

If I do:

scramblers["333"].getRandomScramble().scramble_string

Will the scramble_string be a "random-state scramble" like they use at WCA? Thanks!

zaynv avatar Jul 28 '16 21:07 zaynv

Yes

coder13 avatar Dec 20 '16 17:12 coder13

Thanks! Would you consider publishing this module to npm? Something like:

var scrambler = require('scramblersuite')['333'];
scrambler.getRandomScramble().scramble_string;

zaynv avatar Dec 27 '16 22:12 zaynv

something similar already exists on npm. https://www.npmjs.com/package/scrambo is (should be) a direct, unofficial port of jsss to node.

Although to minimize repos to work on, I wonder if @lgarron might want this repo turned into a npm package to be published to npm?

coder13 avatar Dec 27 '16 23:12 coder13

It would be great if you guys would be willing to do that. You kind of lose out on some functionality if you use the scrambo package. For example, it doesn't allow you to use drawScramble(). I opened an issue there awhile back, but I'm not sure if he's still actively maintaining it. I was considering just forking his and creating another npm package that kept all the features, but if you guys would be interested in publishing an official version that would definitely be better.

saadq avatar Jul 03 '17 19:07 saadq

I'm unlikely to do this with jsss, because the code is messy and licensing is... not clear.

I'm planning to make a better alternative some day, but it's low-priority for me.

lgarron avatar Jul 04 '17 02:07 lgarron

Thanks for the reply, I guess for now I'll just create another unofficial npm package that retains the drawScramble functionality if that's okay with you guys.


Edit: Created scramby awhile back.

saadq avatar Jul 08 '17 21:07 saadq