Brian Brian’s Comments (group member since Mar 30, 2011)


Brian’s comments from the Goodreads Developers group.

Showing 1-20 of 24
« previous 1

Aug 01, 2012 02:57PM

8095 By the way, I did some librarian cleanup on the particular examples you gave above that didn't do well on the search API, and it seems to have helped in most cases. Others might take 30 minutes or so for search indexing to catch up. So if you have a number of these and can post them to the librarians' group, we'll probably find that takes care of many of the problem books.
Jul 31, 2012 12:51AM

8095 Yes, it should be the app name. I don't see an app by that name, so I'm guessing that the text is not a link (which it should be, and should link to a page revealing what the app is).

You'll get this condition if the developer key has been revoked or if the API developer does not have a URL provided for the app. If you're the owner of the app in question, try adding a URL associated with your developer key.

So I looked into your authorized apps, and it looks like you've got an app created by this user: http://www.goodreads.com/user/show/11..., which kinda looks like you (evil twin? ;) ).

Just provide a URL for the app and it should display properly. Looks like all you're doing is testing things anyway, so should've been pretty clear which app that was. ;)
Jul 31, 2012 12:37AM

8095 i am interested, thanks. curious why you're always inserting author into searches. are these searches generated by your users or are you constructing these queries programmatically? have you tried dropping the author from search terms? because, yes, we definitely weight the title more than author.

do any of these look like 'dirty data'?...perhaps low quality books squatting on those exact titles, non-books, or even books that just need to be combined? please post to librarians' group if you're finding any of those conditions.
Jul 30, 2012 03:08PM

8095 We've had to put in some authorization to prevent some inappropriate use of our users' review content, so you don't get access to review text by default. So none of the APIs will allow access to those data. It appears some of our documentation may be out of date, apologies.
Jul 28, 2012 01:05PM

8095 There certainly are examples where our search isn't returning the book you're expecting. I think you may have to concede that the example you gave us was going to be a difficult one: the title was (I've updated the book's data now that you pointed it out) exactly the search string you gave. We certainly endeavor to improve our search algorithm, but in cases like this, the best recommendation I can give you is to post on the Librarians' Group: http://www.goodreads.com/group/show/2... to request someone clean up a particular book that looks funky (wrong title, misleading title, wrong author, author in title, needs to be combined with other editions, etc.).

Now that a few minutes have passed, my corrections to the book's data seem to have fixed the search you've provided.

I haven't seen this behavior very frequently, but you've found it happening 'often'...could you share some more examples? Are they usually in cases where you provide title and author as the query? See if the librarians can help out on your list, but if all the book data are correct, we'd certainly appreciate if you compiled a list of some examples so we can take a look at how to tweak the search algorithm parameters.
8095 Understand the request and it makes sense, though since the API does support all the functions you need to perform, I think it's unlikely we'll make that change soon. If lots of community support for this builds, we can take it under consideration.
Jul 17, 2012 09:13AM

8095 Not sure I get the picture of what your response is. can you give me a snippet of the XML? cause this is what i see for your to-read:

<user_shelf>
<book_count type="integer">26</book_count>
<description nil="true"/>
<display_fields/>
<exclusive_flag type="boolean">true</exclusive_flag>
<featured type="boolean">false</featured>
<id type="integer">...</id>
<name>to-read</name>
<order>a</order>
<per_page type="integer" nil="true"/>
<recommend_for type="boolean">true</recommend_for>
<sort>position</sort>
<sticky type="boolean" nil="true"/>
</user_shelf>

so there's a node for position and a separate one for order. yes, for order, 'a' is ascending, 'd' should be for descending and nil will be default (i would have thought default in our display is ascending, but not sure). i checked in the db as well, and your ordering is definitely set to ascending for to-read there, for what it's worth. not sure why the my books page would be showing it as descending, weird.

as for your currently-reading, yeah, if you haven't specifically set an order on the shelf, then you'll get nil for sort and order...in which case, i believe our UI defaults to date added, ascending. i don't imagine we'll insert those defaults into the response any time soon, but on the other hand, i don't imagine we'll be changing the defaults either, so you could potentially assume those on your end to be in line with the my books page.

that answer everything?
Jul 16, 2012 10:56AM

8095 should just be the one IP address you get from DNS lookup right now from your perspective. for right now we're always using the same hostname for API calls. do recognize of course that we might change IPs for whatever reason in the future and won't be giving heads ups.... ;)
Jul 11, 2012 12:26PM

8095 Not currently scheduled, but it is in our ticketing system. All I can say with any confidence is probably not in the next month.
Jul 06, 2012 05:11PM

8095 Should be released on Monday sometime.
Jul 06, 2012 09:15AM

8095 Thanks for the heads up, Ed!
Jul 05, 2012 11:37PM

8095 I've submitted a fix for code review, so should be released friday or monday. Thanks for finding this! Should also provide correct 'order' values as well.
Jul 05, 2012 04:45PM

8095 Your URL doesn't look quite right...the sample URL I see on the API documentation page is: http://www.goodreads.com/author/list/...

and i get a valid response.

In the link you posted, I see "author_url" in place of the author id, and id= as a url param. Those don't look right. You sure you took that url from the API doc page?

I think the url you might want to try is this one: http://www.goodreads.com/author/list/...

Can you check if you're still having the same problem? We did do a release between when you reported this and right now, so it's possible we fixed it....
Mar 27, 2012 05:30PM

8095 Daniel wrote: "I've released a version of my Goodreads Android App optimized for tablets:

https://play.google.com/store/apps/de...

It requires Android 3.2 or higher running on a tablet ..."


release on nook store?...
Feb 28, 2012 08:40PM

8095 That is absolutely possible...though you either:

- must be the user who first entered the book (and no one has changed your data afterwards)
- or must be a librarian (anyone can apply, part of the fun is finding out how ;) ). It's enough to have a copy of the book (or a record of a version you had at one point) and enter data from that to update an existing book.
Feb 28, 2012 05:50PM

8095 Are you a librarian by any chance? If you are, go to the librarian edits page and you'll see that the data came from Ingram. We can't forward a lot of data we get from them in our API due to our agreement with them. Format is unfortunately among those data.
NoCover image (6 new)
Feb 28, 2012 11:31AM

8095 Our agreements with data providers at times prevent us from sharing book images in our API, even when we can show them on the website and in our own mobile apps. That is unlikely to change. Some images we will eventually get directly from publishers or uploaded by users. Those images we can share images for.
Feb 22, 2012 04:56PM

8095 Here's a png version of the login button: http://d2owxupnsl35mn.cloudfront.net/...

I'll add it to those pages, but may not get release till tomorrow.

All the others look to be jpeg or png, right? Let us know if you need anything else.
New Topic API (3 new)
Nov 10, 2011 12:13AM

8095 Actually, yeah, I think it should work if you add a 'topic[folder_id]' parameter. Be sure that the folder is in the group or else you should get errors.

Please let me know if it works and I'll add it to the documentation....
Nov 04, 2011 04:57PM

8095 Sweet, thanks for the update, and happy coding! :)
« previous 1