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

When providing --ignoreFiles comma separated the code returns Attribute error. #96

Closed
EvDuijnhoven opened this issue Sep 18, 2019 · 3 comments
Assignees

Comments

@EvDuijnhoven
Copy link

The command below with the --ignoreFiles field set

gradient experiments run single node \
--name test \
--projectid 12345 \
--container paperspace/tensorflow-python \
--machineType V100 \
--command 'python main.py' \
--ignoreFiles "file1,file2,file3,folder1,folder2,folder3"

Returns the following error

file_paths = self._retrieve_file_paths(workspace_path, ignore_files)
File "/lib/python3.7/site-packages/gradient/workspace.py", line 59, in _retrieve_file_paths
    exclude += ignored_files.split(',')
AttributeError: 'list' object has no attribute 'split'
@mkulaczkowski
Copy link
Contributor

@EvDuijnhoven thank you for reporting this - may I ask what version of CLI you were using? you can check it by gradient version

@EvDuijnhoven
Copy link
Author

Hey @mkulaczkowski,

The version I am using is 0.3.1. I think a simple fix would be removing the .split(',') from this line because it seems that the split already occurs earlier in the code. Didn't look into the details though.

@BartoszCki
Copy link
Contributor

Hi there @EvDuijnhoven
New patch version (0.3.2) has just been released fixing this issue.

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

No branches or pull requests

4 participants