Setting seed does not return same set of colours!

[EthanSK]

If I set count = 5, and generate the colours twice with same seed, they give a different list of colours!

[davidmerfield]

Hello Ethan, thank you for this report.

Which version of the library are you using (you can find this in your copy of this repo's package.json file).

And what seed input are you using? It would be great if you could share your full usage of the script, e.g.

randomColor({count: 5, seed: 'foo'});

[sanyavanya]

I'm having the same issue. If I just reload the page, the color stays the same, but if I change anything in the code (even if it's not related to the color generating options), the color changes every time, despite passing the same seed. And it doesn't seem to matter whether I pass the seed as an argument or hard-code it. I have a small util that goes like this:

const randomColor = require('randomcolor');
export const generate = (length, seed) => randomColor({
    count: length,
    luminosity: 'bright',
    hue: 'random',
    seed: seed
});

export const generateOne = () => randomColor({
    luminosity: 'bright',
    hue: 'random',
    seed: '1'
});

But even if I call randomColor directly in my components, the problem doesn't go away. Modifying any surrounding code breaks same-color generation from seed.

[sanyavanya]

I think it only breaks on hot reload. If I refresh the page manually, the seed seems to generate consistent colors every time. My project is on Vue 2.

[codebycarlos]

I was having the same issue with React. The solution was to memoize the result so that randomcolor only runs once on re-renders.

i.e.. instead of

const colors = randomColor({
                count: 1,
                seed: 'hello',
               });

try

const colors = useMemo(
  () =>
    randomColor({
      count: 1,
      seed: 'hello',
    }),
  []
);

Edit: Scrap that. Even with memoisation I was still running into issues. I then tried your version @sanyavanya, but kept getting:

So I did two things that fixed it (at least for React).

1. Always pass the reset option as true to randomColor.
2. Store the results in state with the seed as a key so it can be retrieved for later instead of re-generating.

Would be interesting to create a hook version of randomcolor...

[ecl1ps]

Yeah, seeding doesn't work correctly. The output changes even when only count is changed or when it is called multiple times. Latest version 0.6.2

import randomColor from 'randomcolor';

export function generateRandomColors(count) {
  return randomColor({
    count,
    seed: 42,
  });
}

const count = process.argv[2] ? parseInt(process.argv[2], 10) : 0;
const colors = generateRandomColors(count);
const colors2 = generateRandomColors(count);

colors.forEach((color) => console.log(color));
console.log('---');
colors2.forEach((color) => console.log(color));

> node .\color.mjs 1
#e00c08
---
#ed8609

> node .\color.mjs 2
#ed8609
#d8bf43
---
#efc909
#352f8e