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

Error stating backup directory path is invalid #12

Closed
sfregosi opened this issue Apr 28, 2021 · 13 comments
Closed

Error stating backup directory path is invalid #12

sfregosi opened this issue Apr 28, 2021 · 13 comments

Comments

@sfregosi
Copy link

Hi @felipeam86

Thanks for creating this and sharing it. I was able to install easily but am running into an error with actually trying to download data.

OSError: [Errno 22] Invalid argument: 'c:\users\selene\downloads\2021-04-28T12:52:38+00:00_6687511285.fit'

It states this error no matter what I specify as the backup directory or what other options I do or do not include. Any thoughts?

Thanks,
Selene
garpy_Screenshot 2021-04-28 131646

@felipeam86
Copy link
Owner

Hi @sfregosi

Thank you for giving garpy a try and reporting the bug.

  1. Is this the first time you try garpy or have you successfully used it before?
  2. Can you please do garpy download without any arguments and share the screenshot?

I'll find some time soon to switch to Windows and give it a try on my side.

Cheers

@sfregosi
Copy link
Author

  1. This is the first time I've tried it. I also tried installing/running on my desktop rather than my laptop and no luck there...but both are Windows. I don't have access to a non-windows machine, unfortunately.
  2. Attached the output just running garpy download

I searched that error and I think it might have to do with the file name 2021-04-28T12:52:38 containing ":" in the filename? I'm ok with installing and running things in python but relatively new to it.

image

@felipeam86
Copy link
Owner

@sfregosi thanks for that. I have just run the test suite on a windows machine and they are failing with the same error you are getting.

This has nothing to do with your Python skills, I guess my code has never worked in Windows and you are the first brave Windows user to report that. Thank you very much! I'll take a look at this as soon as I get some time this week and try to make the code compatible with Windows.

@sfregosi
Copy link
Author

Hi @felipeam86,
Happy to be the first brave Windows user! Thanks for looking into it and I look forward to see what you find.
Selene

@felipeam86
Copy link
Owner

Found the problem! pull request #13 will fix this. I will make a new release before the end of the week.

Thanks again for reporting, now the code is compatible with Linux, Mac and Windows!!!!

@felipeam86
Copy link
Owner

Thanks again @sfregosi for reporting the issue. I have done a new release of garpy. Thanks to you, garpy is now guaranteed compatible with Windos and Mac OS also!

Can you please run pip install -U garpy and try again?

Pull request #13 solved this.

@sfregosi
Copy link
Author

Hi @felipeam86 I was able to install the new version, but now I'm getting an authentication error (I swear I'm typing in my password right! Tested it with -u username -p password to be sure...just don't want to post that screenshot here). Any thoughts?
Thanks!
image

@felipeam86
Copy link
Owner

Hi @sfregosi

Yes, this is a new issue. Garmin has strengthened their anti-bot measures. I already find a fix. I'll do a release with the fix asap

@felipeam86
Copy link
Owner

Ok!! @sfregosi I did a new release. Can you please run pip install -U garpy and try again?

This time should work 🤞

Thanks for reporting the issues

@Dave-DigiMim
Copy link

Thanks also fixed my issue!

@sfregosi
Copy link
Author

Hi @felipeam86 I'm still running into an authentication error :(

image

@Dave-DigiMim You got it working on windows? Would you mind sharing a screenshot of your steps? Thanks!

@felipeam86
Copy link
Owner

Hi @sfregosi

Garmin has yet again strengthened their security and are blocking the authentication procedure. See issue #19 for a follow up on a resolution.

This has nothing to do with Windows :)

Thanks again for your interest on garpy

@felipeam86
Copy link
Owner

Ok, @sfregosi you chose the worst moment to start using garpy, sorry for that. Garmin is getting strict !!!

New release with a fix. Hopefully this fix will be more robust

Can you please run pip install -U garpy and try again?

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

3 participants