-
Notifications
You must be signed in to change notification settings - Fork 4
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
TODO: in updateDBWithHits default, properly screen hits against database/collection info #15
Comments
Whoa, that was quite scary... let me know how it goes. |
I don't understand how updateDBWithHits works currently. It seems that the default behavior was changed from getting hits according to the hitids attribute, to boto.mturk.connection.search_hits(). It seems like this should almost never work. Am I missing something? |
Need to pass the list of hits. When not passed, this will get ALL the existing hits Amazon has stored for the our account. The advantage of using search_hits() is that it uses Amazon as a global store for the hits, as opposed to the local machine. However, this means we need to filter the hits to only take those destined for the collection of interest. Thus, here is a todo: @hahong it might be a good idea if we added the destination database/collection for each hit to the data sent to Amazon by the task JS, and then cause the updateDBWithHits function to continue to use search_hits as a default (when list of hits is not passed), but automatically screen the hits against the desired database/collection. |
Commit b98ac93 should do this. It works as follows:
|
OK sorry for freaking out... something weird seems to have happened with updateDBWithHITS, as I called it and I got a bunch of AccepTimes from June. Not sure what happened yet.
The text was updated successfully, but these errors were encountered: