labofsahil/github-event-dataset-2012 huggingface dataset:
As I see it the  element that is used in the pages.mobile.regions.message.Message type of classes the as root element of the li html tag because the rest of the data is variable. 
The  locator that you are sugesting is a "a" tag.
Tag that is variable for each "message"
That is why I'm pointing to the li tag and after that searching for the body.
https://github.com/bebef1987/input-tests/compare/mobile_search#L2R22

Not saying that the locator you suggested will not work but the  I'm trying to replicate how the devs constructed the message and read it in a similar way 
https://github.com/mozilla/input.mozilla.org/blob/master/apps/search/templates/search/mobile/message_list.html#L4
ditto earlier comments - if it isn't POD, you can't have it as a global static, unfortunately
I presume that the default socket/conn timeouts are probably just fine for us here... but apache.org seems to be down so I can't see what the defaults are.
You can remove the commented variable that's not needed.
This huge chain of checks seems like something that should be wrapped into a method.
Agreed. It's going to make use of config variables in other modules a lot easier as well.
Should be en_CA.

:)
I'm fine with failing silently in this case when `err` is true, but I think we should at least log the error using `oae-logger`.
Ch. 5 i
Good point about stripping off the protocol part (I guess we could consider using some URL type for these).  Makes me wonder how this even works though; you'd think the basename would just give us “git:” or something, unless the filepath package thinks it's something else, like a Windows drive.  Hmm, need to do some poking.
The code.zikula.org part should be corrected later on of course, but that's just minor :-)
... no longer set the 'name' attribute if it's empty.
Не заметил "не" в описании. Сейчас исправлю.
Такой цикл имеет нелинейную сложность, т.е., например, при DNA_SIZE=100500 и percent=99 будет очень много лишних итераций.
Это я пропустил когда переименовывал переменные. Хотя XCode не заругался так как важно что бы совпадали типы передаваемого значения.  Поправлю в ближайшем комите.
спасибо, исправил
Было бы неплохо добавить проверку входящих параметров.
Using `is_a?` results in spec failures.
This is an amazing gitignore comment.
this was just a temporary fix to get rid of the password recovery flow which was breaking the app, will be returned later.
Shorter: `document.hash = 'blowme';`.
I deleted the how to build proto, seems unecessary with the protobuf.go code checked in.

I renamed them to add-server and add-client.

Well, both example/echo and example/remote:
* Name the server echo or remote (no server in the name)
* Have the server in the basedir
* Have the client in a subdir

So that's what I did.
 
So seems like either server and client should be in subdirs, or neither.  For browsing/simplicity seems best to have all 3 go files in the basedir.
Nice catch, done!
Yeah, you're right... high five.

On Sat, Jun 16, 2012 at 12:10 PM, Brian J Brennan <
reply@reply.github.com
> wrote:

> > @@ -0,0 +1,23 @@
> > +from django.core.cache import cache
> > +from django.http import HttpResponseBadRequest, HttpResponseForbidden
> > +import hashlib
> > +
> > +def throttle_view(func, methods=None, duration=15):
> > +    """If I was a better programmer, I would get this to work with
> passed arguments,
> > +    however - it's just not in the cards tonight. hence ...
> > +    TODO: make this thing actually take arguments
> > +    """
> > +    def inner(request, *args, **kwargs):
> > +        throttled_methods = methods if methods else ['POST', 'GET']
>
> This may show my python ignorance, but why couldn't you set
> `methods=['POST', 'GET']` in the signature and just have
>
> ```python
> throttled_methods = methods
> ```
> (or just directly use `methods`)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mozilla/webpagemaker/pull/434/files#r997778
>



-- 
@chmcavoy <http://twitter.com/chmcavoy>
http://lonelylion.com
Why <!-- NEW --> ? Remove if not needed.
You do it wrong with PowerMockito. See https://source.openmrs.org/changelog/OpenMRS/?cs=28239 for example.
That doesn't look like idiomatic c to me - the `unsigned` keyword is only valid for `char`, `short`, `int` and `long` types. size_t is always unsigned as far as I know (as opposed to ssize_t).
We should use `lock_strategy_enabled?` here.
This should be:

```
'PUT' if id is None else 'POST'
```

Otherwise it doesn't handle the `id=0` case.
In master, rtd.js references the `Search` object, which is unavailable on pages where instantsearch.js isn't included and causes a JavaScript error. This breaks the $(document).ready() chain, so I added the script back in.

The script appeared to have been removed in a previous commit as an attempt to fix a flash on page load, but then a later commit moved some JS files to the top of the page to fix the same problem, so I assumed that removing instantsearch.js had not been the ultimate fix.
Thanks, nice clarification.
Could `image?` work here?
"Listeners to" should be "Listeners of". They "listen to", but they are "listeners of".
I'm guessing that the dictionary is still an issue, no?
I struggled a bit with this one, but I think it's reasonable to expect interpolation keys, in a path, to be separated by non-word characters.
I didn't add them. I was using the niftygenerators gem that they were using in the railscast. So maybe the scaffolding command i used created test cases?
remote unused comments
Please wrap these lines. They are very hard to read ATM.
children foreach stop
raise polynomial to power, n
Клево, малаца.
Please add the tag 'title:' here.
we have to add something like

```php
else {
    $app['EM']->persist($session->getModuleId($moduleId)->setUpdated(new \DateTime()));
}
```
#heading
##heading
Ahhh... now I get what you mean. Yeah, that's certainly a good option (wish that I had thought of that at the begining).

I think I've factored it to the point where it makes sense as separate packages though.
im pretty sure we want to phase out the use of secondary contribs in docs. as when your pull gets merged in you would show up in the GH list of contributors.
maybe we should set this to an UNKNOWN state instead of ERROR
We can indeed.
Perhaps something a little more verbose? `mplDeprecationWarning`?
No, like with the startangle kwarg, we want this to be None.  I know there are several things already set to values in the call signatures, but that will likely change in the future to None anyway.
python2.5 again.
This is a PEP8 related change. I get this message when running the pep8 CLI.
Seems weird that you'd need to dereference *config here, none of the other plugins are doing so. Maps are reference objects by nature, you rarely need to use pointers to a map.
Just curious, what strategy should we use if <code>_empty_signature_title</code> is the only item in the results? It's unlikely that we'll bump into this particular scenario in our data set (and probably not worth solving but do you have a simple solution in mind)?
Hmm...I wonder if it's reliable to test against mozilla.com's title, as that title could change and break this test.

Is there something more reliable we could go off of to be more robust?
Shouldn't an existing tile be removed here in an ```else``` block?
You can just do a link like this:
`<http://output.chrissnell.com>`

You only need to use brackets when link text is different.
Where would I look to figure out what's changed from one wip to another? Could be helpful for figuring out why a version bump sometimes breaks things
adicionar um enter.
Doesn't this end up reading 4 bytes more than before if there are 0 entries. Will this work fine for Grim?
This function below isn't commented out.  Why did you remove it?
@bp323 missing space
grrr. i might add that this is not consistent to me. if we say that functions should be called like foo() and not foo () then since if() is a function as well it should be if() and not if () or? :)
no I will remove it would have been left in when I was trying something different
These need to be nullable as bills often die in committee.
Hmmm - creating repository.version but not using it....
Because I was told to. Don't remember who (either nils or igor) said to make sure the class implemented the interface. I guess that way we know that the class will have a handle() method when we call it on line 62?
Thanks for updating the test, it's helpful for reviewing!
Are you sure the variable ``$groups`` exists, or did you maybe mean ``$group``?
this entire query building is best accomplished with an array:

```javascript
var queryString = [];
// loop:
queryString.push(k + '="' + query[k] + '"');

queryString = queryString.join(",");
Break this line to keep 80-character line length.
I agree. 

I wait of @beberlei answer :)
which path ? you are getting the entity manager, not a path
you've got whitespace issues
`sender` is part of the hook payload.  Its the same as `pull.user` in this case though.
Makes sense. Basically we are memoizing embeddable. Nothing is going to change so this is legit and I can see how it would help.
Only use periods when there is a full sentence
Isso tens bem: o activity_id no Ticket.

Isto é a na tabela da Activity: faz sentido `category_id` para categorias (uma Activity só tem uma Category), mas tens várias Tags e vários Tickets.

Se reparares no modelo das actividades, não tens `tag_id` porque não tens só uma Tag e aqui passa-se o mesmo: não deves ter `ticket_id` porque podes ter mais do que um.
Ok, the question is whether it's worth it limiting the amount of data stored in the output variable above. This needs changes to the excute() function. I think it even makes sense, as we already specify time limit and memory limit in its arguments.
Maybe just to move this code to `Command#handle` method?
Aha, so it is always enough to request only authentication? Can you test that? Removing your permission for our application and try to login?
Hm. Instead of using bisect, couldn't you simply sort data over which you are looping in correct order? And then you do not have to search and insert. In fact you are reimplementing insertion sort here. A bit.

```
for column, order, name in sorted([(panel.column, panel.order, name) for name, panel in self.get_layouts(columns_count).items() if panel.column is not None]):
    assert column >= 0
    while len(panels) <= column:
        panels.append([])
    panels[column].append(name)
```
Programmer should not have to modify everything all around because of this. All this should be made decoupled.
No need for "title". This you can leave simply in the blocktrans content.

Also don't have newlines in blocktrans. It is really hard to translate. Put <span> directly inside of blocktrans. Yes, it is ugly, but ...
Maybe *Uporabnikov*?
no need to give the full constant path here, just use the relative path like in `notification`
I don't think RDP uses these same abbreviations.
This is to keep my local gemset files out of the repo
I'm not sure what you mean with post_params_dict.
But nobody wants to store the whole list in python, which can be an heavy trick (bandwidth and memory on the python side)
Ok you want to argument ? :)

    a store action at each first collection call is not costless

=> no, it's not, but doing same computations many times to get many objects, isn't costless either.We speak about redis, here to save cpu by using memory (and here we have an expire)

    this also means that each call of collection will almost check in redis if the temporary key exists, which another time is not costless

=> it's much much cheaper than computing many times the same collection, which not only make redis calls, but not as costless ones as a "exists". In limpyd we know we do a lot of queries to redis (simply launch a "monitor" in a redis-cli while running tests and you'll see what i mean). And the need for checking the key is already done in some case (sort)

    if you ask for the first element of a very huge set, you'll add this set again in RAM, and the store operation another time will not be costless

=> as for now, your code will use the exact same amount of RAM for doing the sort, freeing it just after. And making the sort (maybe heavy computations) many times, cost even more than just a store (simply store data in memory)

    there is also some implicit/explicit issue for me

=> in your use cases ? could you explain it please ?

    we also have to think about thread safety

=> hmm yes, but the whole limyd stuff isn't thread safe ;) and your actual code can cause the exact same problem with my store-option or not : 1000 process calling a collection will do the store, and use the memory (less time, yes, but still)

    saying "cache", we have to say "invalidation" also: what if some data stored in a temporary key changes?

=.> only a list of PKs is stored. This problem is the same with your actual code, an object in the collection could have been deleted between the moment the collection was created and the moment the pk was got

    it will really add complexity

=> but limpyd IS about complexity... to mask it to the user and provide useful tools to access its data on redis. And this feature would add a small amount of code compared to the whole collection stuff.
Same as before, should be `request_id`.
This is fine and preferred according to the style guide.
лишний файл - удалить
using exog or endog instead of variables would be clearer, I guess it's for exog variables. (or regressors if we want to avoid exog)

Is the definition of the marginal effect below correct for non-binary endog variables, e.g. Poisson or MNLogit
Fixed, variables renamed.
created [GATEWAY-3207](https://issues.connectopensource.org/browse/GATEWAY-3207)
I'm a little confused by this.  Would this four set of 'STOP' strings not want to be added to the `task_queue` before the processes start running?  I'm thinking of the case were the tasks started in lines 920-921 might be over before the interpreter gets to 925... Or am I missing something?
Sorry.  Never mind.  You did do this.
Good idea, will push a change along the lines you suggest...
haha, good spot thanks!
All these checks are ok.  But I think there are also performed at the C++ level, so they aren't really necessary.  You could just do:
```
sblist.append(galsim.Gaussian(flux1, sigma=sigma1, fwhm=fwhm1))
sblist.append(galsim.Gaussian(flux2, sigma=sigma2, fwhm=fwhm2))
```
Why two separate try-catch blocks?
I read this as guido-like substance.  I laughed.
self is not used?
Remove this space also
We always stretch and render on the CPU right now, sadly. As long it happens once it might be ok. After that we should be moving the texture around. Try it on the akami and make sure load and animation times aren't worse.
This change is not critical. Just a style preference.
This file is the vendor CSS - it is upstream SportsML generic CSS - we can probably make a sportsml-custom.css to put these into and figure out a way to include it on render...
Also full stop :)
This indent doesn't look right.
Don't hard-code paths - if you forget to remove it later, someone else will be buggered!
This block here determines if, when updated, it needs to add a zone connection (like if it is a door), or if it needs to make the zone update itself (added a wall splitting a zone)
NONONONONONONONONONONONO

Constants.TAG is backwards and against convention and just a bad BAD idea UNLESS you are doing logging IN the constants class.

http://developer.android.com/reference/android/util/Log.html
Short answer, that works just fine.

Longer answer, because this hides what you are doing and isn't the usual "style" of Android logging is suggested by Google/AOSP.
Upps .. kann ma passieren :smile: 
Soll ich dann auch noch alle anderen Parser anpassen?
Топографический много(>2) условий в if уже будет списком, поэтому их лучше их записать вертикально.
```
if (typeof gps !== "undefined"  && 
    typeof gps.x !== "undefined" && 
    typeof gps.y !== "undefined" && 
    typeof name === "string") {

}
```
Городить столько блоков try не надо. Достаточно перед входом в try объявить все переменные-ресурсы null, а в finally проверять на != null и делать последовательно close.
Тест запускать можно из IDE. Инициализация через методы с @BeforeClass/@AfterClass или для инстансных методов @Before/@After
Попробуй запустить java с параметром -Duser.language=ru
CopyOnWriteArrayList надо использовать, когда запись производится редко, а чтение - часто.
Надо проверять корректность тайм-зоны.
those exception will require mock server instance to test. a new issue filed to track it. https://github.com/WindowsAzure/azure-sdk-for-java-pr/issues/270
Can we pin this dependency please.
Can you sign off : https://github.com/xbmc/xbmc/pull/1380 so we are in sync
/cc @chris-allan since ``3000**2`` is the server-side logic currently in place.
Amended, thanks for the catch.
done + autopep8
@ndawe actually, I can do that - I have some minor changes that I will include in a PR.
@amueller Or simply, `n_iter` :)
I would name it cv_scores rather results
`A field` perhaps?
confusing diff as it makes it looks like the line $('base').attr('href', page.getCanonicalUrl()); was altered - please adjust to retain history of this line.
From the view of D programmer, a variable name `string` is not acceptable. Changing it is better.
This goes for all of the above.  I think we should introduce an D_ILP32 to maintain the pattern of always having else assert.

Also, same question regarding platform abi differences as above.
the build command shouldn't fail because an ip address was not supplied
is it possible to use an Enums?
Yes, I think that is the idea.
If this gets used by non-constants (by accident), just comparing the record types might produce wrong results. Shouldn't we assert if the expressions are not constants? Or at least restrict the pointer type to constant expressions.
Using different names for bundle files is mandatory with my changes, which open the door to concurrent generation of bundle files for updating the caches of different slave nodes.
Das ist peinlich Leute... :O
There would be some value in having them be configurable, but could address it later on.

I'm more concerned about the shift away from transport client.  What if you have a meta cluster with an authentication index that you want to use across multiple data clusters?
i dont get this `else` branch, if it's mustache hogan etc render partials, otherwise render partials?
I didn't understand this line. Should it be ``votes - (votes_up - votes_down)``?
That's same coding convention of PHP
What happened to the section element? All these `div`s below seem redundant...
Лучше не мешать JS c HTML и CSS, чтобы эти части приложения можно было поддерживать независимо. Поэтому мы навашиваем события в JS, шаблоны храним в HTML, а стили объекта максимально меняем CSS-классами.
```javascript
document.querySelector('#').addEventListener('click', preventDefault, false);
```

Постарайся поставить перед собой запрет на использование событий в атрибутах и рано или подздо поймешь профит ;-)
I have resolved the issue. Now it is working with various sets of data but still  throwing error after 17th loop. I had created static object for time that's why only one copy got generated in test run and loops were using the same copy of user email. Now I have changed with default.
I have used the jxl to fetch data from column with column name rather than using data provider because there are lots of data in list and I wanted to have all the registration list data in one sheet. jxl gives us flexibility to read the data with column name.
@shacker we're actually getting the right format back here with the right timezone. This way it gives us more flexibility in the front-end (e.g. when we need to change 09 to 9). So I would like to keep doing it like this whenever it's possible.

See https://github.com/ets-berkeley-edu/calcentral/blob/master/app/models/my_up_next.rb#L55
funktion alive? voi antaa sellaisenaan filterille `(filter alive? authors)`.
Just determines which path to use based on architecture then checks for its existence. Cleaned the whole lot up using Post::File. Sorry about all that not that familiar with the metasploit API.
Good points. Right now there is no notion of a redis_failover root node.
The configuration option that redis_failover currently supports is simply
the top-level znode for storing the redis nodes config. Nothing gets nested
under that, currently. However, by default the zk-groups gem will store the
group name under "/_zk/groups". So we will have:

/_zk/groups/redis_failover_client_presence
/_zk/groups/redis_failover_client_failover_ack

In the separate distributed_monitoring branch I've introduced a root
redis_failover path under which everything gets nested/based. At some point
in the future I'll nest it all under that.

On Tue, Aug 21, 2012 at 8:40 PM, Eric Lindvall <notifications@github.com>wrote:

> In lib/redis_failover/client.rb:
>
> > +    def join_presence_group
> > +      @presence_group_lock.
> synchronize do
> > +        return if @presence_group
> > +        @presence_group = ZK::Group.new(@zk, CLIENT_PRESENCE_GROUP)
> > +        @presence_group.create
> > +        @presence_group.join
> > +        logger.info("Successfully joined ZK group #{CLIENT_PRESENCE_GROUP}")
> > +      end
> > +    rescue => ex
> > +      logger.error("Failed to join ZK group #{CLIENT_PRESENCE_GROUP}: #{ex.inspect}")
> > +      logger.error(ex.backtrace.join("\n"))
> > +    end
> > +
> > +    # Joins the ZK failover ACK group.
> > +    def join_failover_ack_group
> > +      failover_ack_group = ZK::Group.new(@zk, CLIENT_FAILOVER_ACK_GROUP)
>
> Same with this one — it should be based on the redis_failover root node,
> right?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ryanlecompte/redis_failover/pull/31/files#r1431822>.
>
>
It's for scaling unsigned/positive integers, which can however be stored in a signed int type.
`factor * rows` and `factor * cols`? (Plus, `round`, `floor`, or `ceil`)
If the variable is always being used in the same way, it just makes more sense to me to have it defined that way. I guess part of the problem is that `_coeffs` doesn't have any specific meaning to me.

In any case, it's not really important to this PR. Merging.
No need to pass arguments to `super` here.
Good catch. I've removed it.
I don't think there's an Error class at this point. Since this is a command-line script, I would Trollop::die here instead
What does that have to do with this exception?  We throw on negative values.
delete the line instead of just commenting it out
good point will fix that.
There is a concern here I think regarding the integrity of the location data.  At which point is the location data provided so old that it is worse to use it in order to report data versus simply reporting that we are unable to report data?  I would argue that it is better to say we are unable to report data rather than use a location fix that may be days old.
type= is unecessary. Just do <script> to save bytes.
The extra tabGroup is not necessary because this.active will return false if tabGroup is not defined.
Extract the following into a variable: e[i].isDirectory ? 'folder' : 'file'

Then, use string interpolation to make this line more readable.
Fix import order (alphabetical)
we should start from the last tab (Categories) and go backward  or avoid checking the "Featured" tab  first because this is the default selected tab and the click has no effect
Let's leave it package for now and defer the decision of the best public API to later.
`assert(ix < bufx.length && ix > 0);`
"mais vous pouvez également configurer atoum vous-même" me parait mieux :)
XLinkConnector
@satoruhiga did you see this above comment?
If we're going to alias bm_reports, then we should have:
<pre>ORDER BY bmr.a_datetime DESC</pre>
Having spaces around the equals sign would make this a bit more readable. :)
Instead of doing `instanceof` consider adding a method to the annotation class?

`$param = $config->getParam($request)`
It feels like we could do with a way of sharing all of our standard breakpoint sizes rather than gradually replicating across all apps.
Why did we send logs to syslog for the test environment in the first place? IMO that only makes sense for production use, though I prefer using the Rails default of sending them to `#{RAILS_ROOT}/log/#{environment}.log` by default.

If we are going to switch to the Rails default (which I am in huge favor of), we should remove these lines instead of commenting them out. And in a separate commit.
we talked about it this morning - is "" as a default output the sensible solution?
FYI, it could just be `@var ManagerRegistry`, as you imported the class.
в список инициализации засунь
From the Intel manual:

    NOTES:
    Leaf 07H main leaf (ECX = 0).
    IF leaf 07H is not supported, EAX=EBX=ECX=EDX=0
I don't think you meant to put this method here.

I am confused about the id cache. In the javadoc you talk about using tx Synch to remove the id (although the SpEL syntax looks wrong to me - you need to pass in a String for the messageId into the method.

However, in pollMessageFromGroup, you are removing the id from the cache immediately - it's not clear to me how the id will be excluded from the next query if it's immediately removed.
I don't understand this class. What additional value it provides on top of making a direct call to BF?
Got it, just replied
with ~~'temporary-file-suffix'~~ 'append' since __the__ append
Buggy DIC stuff or unwanted addition to the PR?
I think I'd check for a truthy value here instead of checking explicitly for `is not None`.
Title underline must match with title length
Please leave a blank line after `code-block`
Looking at code it's more like: `void|array`
This is basically what I did above. I require `factory_girl_rails` instead of `factory_girl`, as this seems to know where to look for the factories on its own.
Fun in rspec --- you could do `g.status.should_not match(re)`
The original intent for util-core package is actually the *same* original intent that the one at LinkedIn: to *not* have any dependencies with anything (but the logging framework). The problem being that since everything depends on util-core then everything drags their dependencies. So I am not ok adding those dependencies to util-core and would rather put them either in their own module (util-json) or add a java folder in the util-groovy module and move those dependencies there.
Add the shapefiles to the project, set them to Copy If Newer, and then simplify this to `"SampleShapefiles/ne_110m_admin_0_countries.shp"`.  Use forward slashes because either works on Windows and Unix hates backslashes, so forward slashes are more portable.
Also, when I try to run the tests, I get this:
`System.InvalidOperationException : The 'Microsoft.ACE.OLEDB.12.0' provider is not registered on the local machine.`
I assume it's because I need this: http://www.microsoft.com/en-us/download/details.aspx?id=13255
But that redistributable says it's not appropriate for use in a server-side application.  That sounds like a big limitation to me.
does django-categories really need to provide the admin/editor/grappelli_* templates if grapelli is installed?
this one was momentanely and I go to revert it.
Is it good to hardcode this here? 2 seconds might not be enough in all cases...
s/mac_re/MAC_RE/ i.e. naming nitpick on my part :-)
this is also overkill. it doesn't really make any sense in the context of the problem. you just lose. what are the stats? how many times you lost? again i don't think this gains us insight into applicants
Like here, we have to update these package names too!
Can this access token be kept in settings and then use that here .
Please add limited JSDoc here or some other comment that indicates that parentId is an opaque string of up to 64-characters.  The parentId !== '0' check has a JS code smell on its own that makes it look like something sketchy is happening because of coercion.
The NameError comes from 'name' not being in the ENV hash due to the user not giving the 'name=' argument on the CLI
show any soft input area when normally appropriate (when the user is navigating forward to your window).
This assert may do more harm than good. It's generally not good to assert things about the file system
You see this line is commented out, right? 

I assume you're talking about https://tickets.openmrs.org/browse/META-282 ?

I also assume we will want to change the "if (MetadataSharing.getInstance().isAddLocalMappings())" to something in pubsub when META-282 is done as well
For some reason I initially thought the two parameters are named v1 and v2 because they are named as such in the invert method. But they are not related at all, which I think makes this puzzler really tricky!

So, does anybody else think it should be made clear in the explanation that two sets of v1 & v2 parameters are not directly related?
I'm guessing this was accidentally included? :-)
Do you suggest we change the method to toast.show in the index file? For context menus it exposes just the methods like addItem and removeItem.
Can you add a comment about why these are here.
remove this catch, and let the test throw any exception needed.
Why is e.tab undefined?
When the private variables are only in the class and no "getter" is unsing it, then take the _ away. The _ is from java
I'm proud of what I have added, and would love to be listed as a contributor. :)
It doesn't seem like you ever call RefreshService.refresh(...) using STOP_GEOSPATIAL_INDEX.  Does that get called from an external module?