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

rmapy upload fails #3

Closed
faroit opened this issue Aug 29, 2022 · 4 comments
Closed

rmapy upload fails #3

faroit opened this issue Aug 29, 2022 · 4 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@faroit
Copy link

faroit commented Aug 29, 2022

next issue, some accounts can't upload with rmapy as mentioned here:

subutux/rmapy#31

@faroit
Copy link
Author

faroit commented Aug 29, 2022

maybe switch to https://github.com/juruen/rmapi would make it more stable?

@opal06
Copy link
Owner

opal06 commented Aug 29, 2022

This is the main issue atm. rmapi could be an alternative, but unfortunately it's written as an interactive application and written in Go, not as a python library. And while it seems that you can run commands non-interactively and one can run cli programs easily in python by using the subprocess module, this is not a really clean way to implement such a thing and comes with some problems, like no real return values beyond "success" (returns 0) or "fail" (returns 1).

I think the way to go is rather looking at how rmapi handles the new 1.5 reMarkable api and try to implement that for rmapy, either by making a pull request or forking it.

@opal06 opal06 added bug Something isn't working help wanted Extra attention is needed labels Aug 29, 2022
@opal06
Copy link
Owner

opal06 commented Sep 24, 2022

I have now experimented some more with python's subprocess module and found it more than capable enough to implement reMarkable cloud API access through rmapi. So I have rewritten the program to use rmapi which should fix issues related to the newer cloud API versions. You can find this version under the rmapi_workaround branch of this repo. I have updated the Readme to show how to change branches.

I have not come around to test this implementation thoroughly though, so please feel free to give it a try and report any bugs found, but be aware that it might not work perfectly yet. If you find any bugs, please open new issues, so it's easier to keep track of them. I will close this issue for now.

@opal06 opal06 closed this as completed Sep 24, 2022
@faroit
Copy link
Author

faroit commented Sep 24, 2022

@opal06 Thats great. I will give it a try asap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants