-
Notifications
You must be signed in to change notification settings - Fork 7
friendrated.pl returns only books I have already read, gets ratings wrong #41
Comments
Did you use the original Goodscrapes.pm library or the version you patched from the San Kumar issue? Because his patch actually affects scraping of rating data and is obsolete. I added other code later. |
That seems to have been at least part of it -- after reverting to the original code I'm now getting books I haven't read. There are still books appearing that were not rated 4 or 5 stars by the users listed in the results, though, and I'm still getting some books that are on my read shelf when using |
Okay, need to ask for some more background before:
|
I happened to notice this first because the results were showing my sister as a "vote" on a book that I know she doesn't like. I went to the table view first and verified she had given it three stars. It's showing up as three stars on the book page too. I have since found a couple other examples of the same thing with other users.
Yes, I was not using the
Yes, confirm I see this message each time. |
I just ran the friendrated.pl script and randomly checked the hits. I'll take a closer look at this. If you like, it would be helpful if you tell me your Goodreads account so I can test against it. |
I'll shoot you an email with the specifics. I did try it both with a build from the cloned repo on my local machine and a build straight from github ( |
I just ran friendrated.pl with no arguments, and it returned a list of books in common with my friends/followers only -- specifically, books on my read shelf. In addition, it showed me some books as faved by my friends that they did not rate 4 or 5 stars.
Running with
-x read
in the arguments reduces the number of results by about 3/4 but still shows me books from my read shelf only (the-x <shelfname>
argument is meant to exclude books from the specified shelf).Running for one of my friends with the
-u <their id>
argument returns only books that she, I, and her followers have on our read shelves.The text was updated successfully, but these errors were encountered: