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

Add functionality to automatically mark a Process class as an invalid cache source if it contains a non_db port #5835

Open
sphuber opened this issue Dec 14, 2022 · 0 comments

Comments

@sphuber
Copy link
Contributor

sphuber commented Dec 14, 2022

The non_db keyword allows to define an InputPort in a process specification whose value will not be stored permanently in the database, but will just be available to the process itself during its lifetime. This probably means that reproducibility is violated and so instances of the process should not be valid cache sources. A Process class should automatically mark its nodes as invalid caches as soon as the spec contains a non_db port. We might need an override that allows a user to disable this behavior if they are convinced that the non_db port does not affect reproducibility.

Feature was requested by @giovannipizzi in a discussion during #5801

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

No branches or pull requests

1 participant