Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Add the missing seed argument for deterministic results. #351

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ice-tong
Copy link

@ice-tong ice-tong commented Jan 28, 2022

Summary

Add the missing seed argument to self.image_data_generator.get_random_transform for deterministic results.
Especially during multi-thread, we cannot set numpy.rand.seed out of the thread, this argument pass is important!

Related Issues

None

PR Overview

  • This PR requires new unit tests [y/n] (make sure tests are included)
  • This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
  • This PR is backwards compatible [y/n]
  • This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Add the missng ssed argument to `self.image_data_generator.get_random_transform` for  determistic results.
Especially during mutli-thread, we cannot set numpy.rand.seed out of the thread, this arugment pass is important!
@ice-tong ice-tong changed the title fix(NumpyArrayIterator): for determistic results Add the missing seed argument to self.image_data_generator.get_random_transform for deterministic results. Jan 28, 2022
@ice-tong ice-tong changed the title Add the missing seed argument to self.image_data_generator.get_random_transform for deterministic results. Add the missing seed argument for deterministic results. Jan 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant