hypergraphx icon indicating copy to clipboard operation
hypergraphx copied to clipboard

No seed variable when initializing HyMMSBMSampler

Open larryzhang95 opened this issue 2 years ago • 1 comments

In the HyMMSBM Sampler class, there's a random number generator variable that takes a seed value, in the constructor:

self._rng: np.random.Generator = np.random.default_rng(seed)

The solution is as easy as adding another member variable to the constructor seed, which I did on my end. I can submit a pull request if desired. It worked properly after doing that.

Note that I installed this from source , so I used pip install -e .['all'] while in the hypergraphx directory.

larryzhang95 avatar Nov 20 '23 08:11 larryzhang95

Thank you for reporting this! Pull requests are welcome:)

FraLotito avatar Nov 20 '23 16:11 FraLotito