sacs-epfl / decentralizepy

A decentralized learning research framework
MIT License
24 stars 18 forks source link

Fixed seeding for data partitionning #12

Closed dimiarbre closed 6 months ago

dimiarbre commented 6 months ago

Fix of #11 with regards to data partition.

For validation sets, a seed cannot be provided to np.random.choices() (see the documentation), but reproducibility should be handled by the following line: https://github.com/sacs-epfl/decentralizepy/blob/170fc9eb1bf31d0559cf57917a311c3b575b1e16/src/decentralizepy/node/Node.py#L207

rishi-s8 commented 6 months ago

Looks good, merging.