Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lambda transform should be optionally randomisable #1542

Closed
wyli opened this issue Feb 3, 2021 · 2 comments · Fixed by #1546
Closed

lambda transform should be optionally randomisable #1542

wyli opened this issue Feb 3, 2021 · 2 comments · Fixed by #1546
Assignees

Comments

@wyli
Copy link
Contributor

wyli commented Feb 3, 2021

Is your feature request related to a problem? Please describe.
the lambda transform may have some random factors (such as using np.random), in that case it shouldn't be cached in a cache dataset. Ideally the lambda transform should support dynamic inheritance of the randomisable interface

@rijobro
Copy link
Contributor

rijobro commented Feb 3, 2021

Should there be a RandLambda transform such that the CacheDataset knows whether it's a random transformation or not?

@Nic-Ma Nic-Ma self-assigned this Feb 3, 2021
@Nic-Ma
Copy link
Contributor

Nic-Ma commented Feb 3, 2021

RandLambda sounds like a good solution, I will add it soon.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants