-
-
Notifications
You must be signed in to change notification settings - Fork 2
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:rse.utils.urls:Cannot find endpoint
when running rse import --type google-sheets
#76
Comments
Row 11 of your spreadsheet is not a valid GitHub identifier: https://github.com/sarabsethi/audioset_soundscape_feats_sethi2019/tree/master/calc_audioset_feats That's why it says "cannot find endpoint" and shows that partial url. |
👍 confirmed that if I change row 11 to just |
"valid GitHub identifier" is defined as just the landing page of a repo? |
I got further this time but got a 403, rate limit exceeded 😭 Is that maybe cuz I've been running it over and over again?
|
Ah yes! So this is common whenever using the GitHub api in any context, if you export GITHUB_TOKEN=xxx (as a personal access token) it should increase your limit. It only happened because you ran the same thing many times (and there are a lot of rows!) A valid GitHub identifier matches:
And right now it's doing a very basic matching and using some split logic to derive the underlying name. Looking at this now, I could improve upon it - I'll open an issue for future me to work on :) The reason it's done simply is that (at least before this googlesheet addition) most GitHub urls came from places where they would always be in that format. Now that's not the case so likely my parser needs to account for that. |
Got it, thank you I think I can close this then
So just to make sure I understand the source of the error: |
Correct! And I opened an issue #77 so that you should be able to provide a poorly formatted GitHub URL and still be able to retrieve the metadata. Have your cake and eat it too :) 🍰 |
Description
I now have my copy of the Google sheet from @rhine3 set up so that I can start
import
running like so:that is, I don't get any errors about missing fields, i.e. incorrect column names, now that I removed the message in the hidden row 1, and renamed the first three columns
full sheet is here:
https://docs.google.com/spreadsheets/d/1Ba1MY4o5Sm1f08IekJcbxAtSjkDN71Z1RZ42kzrofJ0/edit?usp=sharing
However I do get an error now that I'm not sure how to fix, traceback below
Two notes:
What I Did
I can't see anything specifically different about the link that causes the crash
I notice that when I click on the link I get a redirect from GitHub?
But this happens for all the links in the rows above it too.
The text was updated successfully, but these errors were encountered: