labofsahil/github-event-dataset-2012 huggingface dataset:
I agree about them being importers.  I must missed the base class name.  How does MappedImporterBase sound?
Missing a `;` here -- it might be worth running the new code through jslint
Can you chain this with the line above?
No problem. After merge. Issue created #8.
@nsfmc What do you think about just getting rid of the whole boolean passing thing, keeping the event listener and binding, but changing this to a simple testMode check like the following:

			if (testMode) {

				// testMode automatically renders the next problem --
				// integrated mode will trigger its own event when ready.
				jQuery( Khan ).trigger("renderNextProblem");

			}
>but this lead to massive code duplication

No duplication needed.
Just create two naked asm thunks that pass over to sigsegv_userspace_handler.
This is also the right place to add the missing stack alignment.
This template should have something at the head of it describing the variables it expects to be available.  e.g.

<!--
  expected bindings:
  encounterIndex - integer for the number of OBR this is in the line of OBRs.  Goes into OBR.1
  encounter - the current Encounter object that this is acting on
-->
In the DefaultTransactionalResourceSynchronization.beforeCommit, processResourceAfterCommit and afterCompletion
The one essential piece of info a place should have is that is located somewhere.  That is what my thought was.  Also previously I've had problems with geospatial queries in Postgres when the fields are null.  This may not be the case anymore, but I've always avoided nullable point fields since then especially since it is the most important field in the model.
`CompaniesHouse` should really be a module, not a class.
En esta redacción parece que será el framework quien hará la agrupación cuando lo correcto es `generalmente agruparás`
処理の種類が変わるので一行開けてほしいです。
`if subdomains[subdomain]?` could also be `if subdomain of subdomains`. Makes it more readable. This works, too, of course.
Can you use sprintf for string concatenation ?
variables should be camel cased
Done.

On Feb 29, 2012, at 2:50 PM, Justin D'Arcangelo wrote:

>> @@ -0,0 +1,4 @@
>> +jQuery ->
>> +  $("a[rel=popover]").popover()
>> +  $(".tooltip").tooltip()
>> +  $("a[rel=tooltip]").tooltip()
> 
> I'm not gonna block on this, but you should combine these selectors into one line:
> $('.tooltip, a[rel="tooltip"]').tooltip()
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/entropillc/bootstrapped/pull/5/files#r503217
this looks like it was an accident ... go ahead and delete this line.
this javadoc isn't necessary, since it extends an abstract already-declared method.  Instead use `@see PatientSnapshot#consume(Obs)`.
How will a blank subject be represented in this table?
I think 
app_context[name] = context[name] || Hash.new
would be more clear

app_context[name] = app_context
looks kind of weird.
same here needs the version/acct#
Good, can you just indent only one more than `super`?

```
super(EmbeddedDocumentField, self).__init__(
    to=embedded,
    attribute=attribute,
    null=null,
    full=True,
    help_text=help_text,
)
OK, developer of that code was looking at this [implementation](https://github.com/andresdouglas/django-tastypie-nonrel/blob/master/tastypie_nonrel/fields.py). I don't understand the rename of the field, though. So, I think your name (EmbeddedIndexedListField) is better. So this should be renamed.

Also documentation should be improved. EmbeddedListField does allow manipulation of list elements, only list elements are accessed by _id. And EmbeddedIndexedListField by index. I think for both there are good use cases. And now I agree with `pk` being the index. Although I am afraid that this will still break URIs if it is backed by SortedListField. In any case  EmbeddedIndexedListField has a problem of having non-persistent URIs. Or they are, if you forget about internal _id,but see index as API id of the element.

What do you think?
Won't the code at line 72 return (0) if count(//docblock/tag[@name=$marker]) is 0?
Use `from django.utils import six`
Good point, moved it there. I recently learned how to make stuff private in objective-c, must use it more often :)
extra spaces in method sigs from String suffix removal
This interpolate flag is now broken, along with calc/calcn. They used to have different behaviors (interp vs non). This was also the case for seti/set and tmp/tmpn. Also, set and tmpn set their content to strings as that what it should. Currently, there is no interpolation or definition for their content.
I think constants for sucess and bad sync key are reasonable here.  It's a minor thing given the presence of comments, of course.
When movie is not available width and height are 0
`cancelled` -> `canceled` (also in API)
Is this because this is where the webview will look for them? Is this true on all iOS versions?
I prefer this to be a local variable, and then assign that to the global variable: only one modification of shared state instead of many.
```ruby

    unsubscribed.each do |email|
      user_record = User.where(:email => email).first
      if user_record
        user_record.disable 
      else
        warn "No record for #{email}"
      end

      api.delete_user(email)
    end
```
Won't this end up being triggered before setContentView() is called in the subclass? If a receiver somewhere is relying on this event to determine that the layout of the main activity has been drawn and tries to access a resource from that layout, it will fail.
Sounds reasonable, I don't have any better suggestion in any case.

LGTM
Sure, I will put the comment same as what you suggested here!!
This can be taken out.
i'm not sure why this this here? are you always assuming that the lock was initialized by the base class? this should be handled by the class that originally created the lock
if this is unused it should be removed
Maybe package scope?
Why not use the homepage method? Seems like your duplicating here
Should probably be added at the end of list near `__traits`  Also, can you add `__vector` to the list while you're there anyway? Seems that one was forgotten.