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

Data paths should be tied to CLI for Resnet app #2184

Open
benson31 opened this issue Feb 1, 2023 · 0 comments · May be fixed by #2192
Open

Data paths should be tied to CLI for Resnet app #2184

benson31 opened this issue Feb 1, 2023 · 0 comments · May be fixed by #2192

Comments

@benson31
Copy link
Collaborator

benson31 commented Feb 1, 2023

The resnet.py app is too restrictive about its data paths. In particular, if a user at LC without lbann group access or at another institution wants to run it, they must modify Python code. Normally I'd say this is fine (it's only Python), but unfortunately for us, there's now precedent -- at the very least, the Cosmoflow app allows data paths to be overridden on the command line. So we should make this more pervasive.

@benson31 benson31 assigned benson31 and unassigned benson31 Feb 9, 2023
@benson31 benson31 linked a pull request Feb 14, 2023 that will close this issue
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.

1 participant