-
Notifications
You must be signed in to change notification settings - Fork 37
Open
Description
Describe the bug
If the user does not specify JOBFLOW_CONFIG_FILE as an environment variable, it's almost certainly due to a misconfiguration or because they forgot to set it / add it to their environment, not because they want to run with a MemoryStore.
Unfortunately, jobflow just happily runs (presumably using MemoryStore) and doesn't complain about the missing jobflow.yaml. This not only burns computer time with no output files written, but also results in user configuration because they don't know why things aren't being stored.
Suggested behavior:
- softer version - emit a warning when no JOBFLOW_CONFIG_FILE is set
- harder version - require a JOBFLOW_CONFIG_FILE and error / exit if not set, making it immediately apparent that the user should explicitly set a configuration file.
See also: materialsproject/atomate2#1423
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels