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

remote: handle -d ERROR #3219

Closed
ghost opened this issue Jan 22, 2020 · 10 comments
Closed

remote: handle -d ERROR #3219

ghost opened this issue Jan 22, 2020 · 10 comments
Labels
bug Did we break something? p2-medium Medium priority, should be done, but less important

Comments

@ghost
Copy link

ghost commented Jan 22, 2020

I encountered ERROR: unexpected error - expected str, bytes or os.PathLike object, not NoneType
when I run command dvc remote add -d tmpremote [a shared Windows drive] at default home directory of my environment.

The issue is gone when I run the same command at my git folder.

image

EDIT:

More context:

https://discordapp.com/channels/485586884165107732/485596304961962003/669661824190513162

@triage-new-issues triage-new-issues bot added the triage Needs to be triaged label Jan 22, 2020
@efiop
Copy link
Contributor

efiop commented Jan 23, 2020

Hi @qwljasmine !

I see that the path you are supplying is //something/dvc_tmp_repo wich doesn't look like a valid path for windows. Could you please use a proper windows path instead and see if that helps?

@efiop efiop added the awaiting response we are waiting for your reply, please respond! :) label Jan 23, 2020
@triage-new-issues triage-new-issues bot removed the triage Needs to be triaged label Jan 23, 2020
@efiop efiop added the bug Did we break something? label Jan 23, 2020
@ghost
Copy link
Author

ghost commented Jan 23, 2020

Hi @qwljasmine !

I see that the path you are supplying is //something/dvc_tmp_repo wich doesn't look like a valid path for windows. Could you please use a proper windows path instead and see if that helps?

Hi @efiop this is the path that my teammate shared to me and as I mentioned I am able to use it when I execute the command at my git directory. So I guess I am not able to try out another kind of path since that is the only path I have. Thanks

@shcheklein
Copy link
Member

... it also looks like no matter if that path is correct or not we have some strange behavior with that exception.

@efiop
Copy link
Contributor

efiop commented Jan 29, 2020

@qwljasmine sorry for the delay. Could you try removing the leading / and see if that helps? Though I think it will still fail, because you are using posix path on windows machine. Is your colleague running linux or mac?

@ghost
Copy link
Author

ghost commented Jan 30, 2020

@efiop removing the leading / can run. My colleague is running windows 10

@efiop
Copy link
Contributor

efiop commented Feb 1, 2020

@qwljasmine So / helped and now it works as expected? Is the issue resolved(except that we could indeed present it better)?

@shcheklein
Copy link
Member

@efiop I think the problem is that certain command line options combination leads to this unhandled properly exception "ERROR: unexpected error - expected str, bytes or os.PathLike object, not NoneType".

@efiop
Copy link
Contributor

efiop commented Feb 1, 2020

@shcheklein It is a problem (i've mentioned "except that we could indeed present it better"), but not the core problem. Just checking if the core problem is resolved or not.

@shcheklein
Copy link
Member

@efiop ah, yeah. It looks like it was from the Discord discussion. Not 100% sure though.

@jorgeorpinel jorgeorpinel changed the title dvc remote add -d ERROR remote: handle -d ERROR Feb 18, 2020
@efiop efiop added p2-medium Medium priority, should be done, but less important and removed awaiting response we are waiting for your reply, please respond! :) labels Feb 26, 2020
@efiop
Copy link
Contributor

efiop commented May 3, 2021

Closing as stale. The general "unexpected error" is a topic for a separate discussion.

@efiop efiop closed this as completed May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Did we break something? p2-medium Medium priority, should be done, but less important
Projects
None yet
Development

No branches or pull requests

2 participants