cubing / jsss

⛔️ Deprecated — use cubing.js instead! (was: Javascript Solver/Scrambler Suite)
42 stars 24 forks source link

Is this a WCA-approved scramble? #9

Open zaynv opened 8 years ago

zaynv commented 8 years ago

If I do:

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

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

coder13 commented 7 years ago

Yes

zaynv commented 7 years ago

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

var scrambler = require('scramblersuite')['333'];
scrambler.getRandomScramble().scramble_string;
coder13 commented 7 years ago

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?

saadq commented 7 years ago

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.

lgarron commented 7 years ago

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.

saadq commented 7 years ago

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.