seqrequester icon indicating copy to clipboard operation
seqrequester copied to clipboard

Can user provide seed to random generator during read simulation?

Open cjain7 opened this issue 3 years ago • 2 comments

It seems like the seed value is fixed for read simulation:

https://github.com/marbl/seqrequester/blob/31141c14d80fe0dde2766bcc3e622bf600e2bba4/src/seqrequester/simulate.H#L59

Is it possible to expose this parameter on the command line interface?

cjain7 avatar Jul 05 '22 04:07 cjain7

For my use, it would suffice if this seed is generated randomly. Maybe instead of seed value, a command line boolean option to randomly generate the seed will also work for us.

https://github.com/marbl/seqrequester/blob/31141c14d80fe0dde2766bcc3e622bf600e2bba4/src/seqrequester/simulate.C#L449

cjain7 avatar Jul 05 '22 04:07 cjain7

Hi Chirag!

Thanks for the gentle push here. I added -seed to everything that uses random values. It was, sadly, only lightly tested - and there was some major refactoring of command line processing done at the same time.

brianwalenz avatar Jul 15 '22 03:07 brianwalenz