search

anmartinezs / polnet

Generates synthetic datasets for Cryo-Electron Tomography
Other
20 stars 9 forks source link

Add support for controlling random seeds #14

Closed kephale closed 1 month ago

kephale commented 5 months ago

It currently isn't possible to reproduce an exact run on polnet because the polnet.lrandom does not let the user seed the random number generator/use the same RNG for all random calls.

I'd be happy to take this on if ya'll are interested, but regardless would strongly advocate for this functionality.

anmartinezs commented 2 months ago

Sorry for late answer. This is a great idea, are you still interested into take this on?

kephale commented 2 months ago

I've started: https://github.com/anmartinezs/polnet/pull/16

Currently tests aren't passing. I haven't fully diagnosed the source yet.

kephale commented 1 month ago

Just checking in on this @anmartinezs. :)

anmartinezs commented 1 month ago

OK, we understand that you pull request is ready for being merged, we will handle this soon. Many thanks for your contribution.

kephale commented 1 month ago

Great, thank you!