labofsahil/github-event-dataset-2012 huggingface dataset:
Lucene 3.6, simple test, more phonetic encoders

Updating to Lucene Phonetic 3.6 obsoletes org.elasticsearch.index.analysis.PhoneticFilter and org.elasticsearch.index.analysis.DoubleMetaPhoneFilter.

I added a simple test that loads all token filters into the factory by a yml settings file and checks if the factory is an instance of PhoneticTokenFilterFactory.

Nysiis will be in commons-codec 1.7, see https://issues.apache.org/jira/browse/CODEC-63

HaasePhonetik is my favorite for german name phonetic encoding, it comes with an alternative KoelnerPhonetik implementation to commons-codec ColognePhonetic. I want to donate the implementation. Thanks!

Jörg
Small fix for an issue

This is a simple fix for this issue: https://github.com/ddeboer/DdeboerSalesforceMapperBundle/issues/3

By the way, thanks for the nice bundle, I would appreciate it if you included this fix. There might also be more contributions from me in the future.
Note to let users about broken package since NeCTAR upgrade
FLUID-4630: Small adjustment to indenting of lists.

@cindyli, one more small tweak.
Fixed typo. Javascript file name should probably be main.js instead of main.css...
Fix send audio file via bluetooth

also fix some formatting

Tested on CMX Nexus S and CM7 Motorora Defy+,works well :-)
Gles2

Some small bugs where arguments where in the wrong order.
Added some question and update chinese version

English version:
added some question
added my name to contributors list

Chinese version:
added some missed translated sentence,
added translate of new question
added my name to contributors list
Screenshot: http://i.imgur.com/S6AJ7.png

Here’s the original comment in text form (as deleted from http://mathiasbynens.be/notes/safe-css-hacks#comments):

---------

This link resolved about All IE browser compatibilty issue for Windows Only. I would like to share browser issues for Apple MAC. 

```
/*\*/ @import "hacks-ie-win.css"; /**/
/*\*//*/ @import "hacks-ie-mac.css"; /**/
```

Firefox Browser (MAC)

```
<script type="text/javascript">

var csstype="external" //Specify type of CSS to use. "Inline" or "external"
var mac_externalcss='/templates/yoursite/css/style_mac.css' //if "external", specify Mac css file here
```

Safari  Browser (Windows)

```
<?php $browser = $_SERVER['HTTP_USER_AGENT']; ?>  
<?php if (strstr($browser, "Safari")) { ?>  
<link rel="stylesheet"  href="<?php echo $this->baseurl ?>/templates/yoursite/css/style_safari.css"/>
<?php }?>
```
I ran into this with `WidgetButtons` too and had to step through `Attribute`'s code to figure out what was going on because it wasn't behaving as I initially expected.

@lsmith Your suggestion of testing whether a getter's return value is an Object seems like it could lead to even _more_ confusion about how getters work with sub-attribute values.
@PeterJohnson Please let me know if you see any issues with this patch. Here's why I think it's correct:

The section flags are read here:

    flags_override = yasm_dir_helper(object, vp, line, help, NELEMS(help),
                                     &data, yasm_dir_helper_valparam_warn);

The only fields of `data` that are read in the following code are `data.f_segname` and `data.align`. `f_segname` is used to look up the section object (using `yasm_object_get_general`), so it will match `msd`'s name no matter if it's new or not. So the only relevant value that's used is `align`. If the alignment matches the previous alignment value, there's no point in printing a warning about it being ignored. If the value is different, the warning is still printed.

I checked locally that this program still warns:

    section .rodata align=32
    section .rodata align=16

while this doesn't:

    section .rodata align=16
    section .rodata align=16
It shouldn't be too hard to add translation parameters to FormValidator, OTOH.
Ahh, I see.  The pref is for new items only.  There are also buttons to initiate a lookup for all current items and cancel lookup for all current items.  So it's pretty easy for a user to re-scan the old items if they need to.
Самой модели похоже нет в репозитории
This is a *test* comment.
Done: http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=34790
Thanks for suggestion!
Example does not run because of the typo - should be **function** instead of *functon*
* 1.3.19
@ry, @koichik: Are`_flush()` and `_buffer()` still necessary? It looks like duplicated functionality from `net.Socket.write()` and libuv.
By limiting the API I really meant making internal logic complex and not transparent, this then affects the API. See Kris Kowal Q design notes, everything is plain and simple until we reach `promise.send` addition, then everything is rewritten so it's more complex and not that extensible anymore, at least that was my impression.
I think this line has to be in the `included` block for the code not to be racy?
@sferik I like your style!
Hello @piscisaureus would you mind comment why this implementation is better? Is faster? is more stable?

(just curious as we face issues around `stat()` on Ruby)

Thank you.
I think that HTTPS doesn't buy you much here.  If we do HTTPS I'd like to
switch every repo over to HTTPS.

On 26 January 2012 23:34, Anders Nawroth <
reply@reply.github.com
> wrote:

> The url with https in front will just redirect to http anyhow. Maybe
> remove the "s"?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/neo4j/manual/commit/79e460482a12853983b9b250c1fb45b854641135#commitcomment-906320
>



-- 
Julian Simpson
The Build Doctor Ltd.
http://www.build-doctor.com
julian@build-doctor.com
(+44) 207 183 0323
Or nevermind, you're just doing 1-line ```if``` statements and confusing me :P
А с каким  словарём юзать?
I think overwriting the setContent() method makes more sense.

Nothing ensures the data set with setContent will be converted to json
@jaberg?
Correct twisted.python.log do not have debug() there for I want to use msg like the rest of the code in ldaptor for example ldapserver.py.
You can use do_QueryInterface() to get the return value directly:

*_retval = do_QueryInterface(newWindow);

or maybe you need:

nsCOMPtr<nsIWebBrowserChrome> thing = doQueryInterface(newWindow);
_retval = thing;
Great work @wicz. Most of the issues are really minor. Thanks for working on this. It will be really helpful for future meetings.
removed after finding the option in configure :)
line 58 should be deleted, btw
EINVAL has a value of 22 which might cause problems when sending data of length 22. Why not returning a negative value here.
yes, +1 on `getUtility(ISiteRoot)` from me as well.  i was actually thinking about it for a second or two, but figured i'd go with the `getSite` call as mentioned in some tweet given in pretty much direct response to liz's talk.  but i don't have a very strong opinion on it… as long as the tests pass ;)
took me a second to mentally process this, but now I think I get it. :)
Ah, yeah. That's probably a good idea. Which methods do you think we need as intrinsics?
This isn't necessary, but it is for the xor case and this is (very) slightly more efficient.
<h1><i> This failed so hard </h1></i>
Why did you remove this command? There are several people which need this: https://github.com/ddollar/foreman/issues/113
The file is always:
DELETE FROM ... WHERE guid IN (SELECT guid FROM ... WHERE ...);
INSERT INTO ...

You have to transfer requests to remove the very beginning of the file before the variable definition.
now problem with can't leave persits. player may correctly leave only over driver seat (corrected only seat change now). but in my tests passengers in 1&2 seats on demolisher not may control of movement (see SetClientControl(m_pBase, 1) for his. 1 - not may move). may be some diffirents again?
why? We push in targetUnitMap (*itr), not  (*itr)->GetCharmerOrOwnerPlayerOrPlayerItself(), origin code exclude npc and pets
another thing against OP bm hunters is https://github.com/gc/mangos/commit/87a7abd8adac6b70684f91d50a94f090c14e37dc and https://github.com/gc/mangos/commit/e42821557ea41dc156a53bd39ac13d2f649b2e1d , which will reduce their pets dmg by like 30% :p Im going to try rewrite it with using scaling data now
The indentation is all off here.
i've had this in my code for a year

  https://gist.github.com/1657500

using 'inspect' in the conditions is much saner that 'is_a?(Array)' as it handles funky nested objects.  also

  Model.not_found!(conditions)

is extremely useful for triggering 404s in apps.  more generally mongoid needs to provide factories for exceptions because doing

  raise Mongoid::Some::Error::That::Will::Move::Namspaces.new(custom_message)

is brittle
can you add test for that?
Could you do this by adding a filter to process_request, line 222?
`Addons.delete` already blacklists the GUID.
@billycao lol maybe i should have used better naming convention by prepending each member variable with <code>m_</code> or something like that
You could always abuse #include to pull common C code into a separate file. Anyway, it's not a big deal for just one function.
Why set defaults that cause lag?
Not sure whether it should be moved into `Travis::Mailer.setup`.
Right, the most recent "LIKE '%b%'" version this digs up and those correlation scripts can deal with is some Firefox 4 beta - not really helpful nowadays, and we actually have a number of basically or completely empty correlation reports generated each day due to the ancient versions this all selects there.
Need to test on 10.6 and probably 10.5, unless we can point to a changelog item that explains why this works now.
well in fact it depend which user got the required environment variables 

MONITORING-FR
AvatarDavid GUENAULT 
Membre de l'équipe Monitoring-fr 
secrétaire de l'association 
MONITORING-FR



----- Mail original -----

De: "Sébastien Coavoux" <reply@reply.github.com> 
À: "David GUENAULT" <dguenault@monitoring-fr.org> 
Envoyé: Lundi 6 Février 2012 12:48:33 
Objet: Re: [shinken] Fixing #158. Please, @david-guenault look at the comments with "Sugg : " ;) (165e4a3) 

Well, why not :) I know nothing about oracle, but the condition "ORACLE_HOME not set" seems to be enough to say that oracle is not installed :) 

--- 
Reply to this email directly or view it on GitHub: 
https://github.com/naparuba/shinken/commit/165e4a34d38be8efaef71a244edb005b2a6727d3#commitcomment-936264
I cherry-picked this commit.
@malleor Not the same author ;)
Will Mead
Damn HTML scrapers... what's the purpose of the c-u here?
不过为毛c-string就要乱码呢..
Fixes
Caused by: java.lang.NullPointerException
	at me.darazo.ancasino.listeners.AnPlayerListener.onPlayerInteract(AnPlayerListener.java:34)
	... 18 more
I'm not sure if it's a good idea to change the semantic (return *all* user's builds) of the existing method. Shouldn't we better introduce a new method with maybe some paging semantics?
yeah!  the reason I actually made this change is that when doing:

    resources = require('static-resources.js');

In code that uses static-resources.js, it *actually fucking changed* resources *inside* the module.  This is not supposed to be possible in my understanding of nodejs.  

Rather than yak shaving to deepen my understanding of nodejs, I made this change.  So it was less about fiddle the aesthetics of your code, and more about working around crazy shit.
I'm not entirely sure this is the best place for this fn. Currently no code in overtone.sc/* links out to overtone.studio/* However, if people wished to convert their buffers (in code that didn't use the studio functionality) to wavetable format then, they would have to include a studio ns. Unless you're thinking about breaking out of this tradition, I think that all *core* fns should be in sc/*. overtone.sc.buffer seemed a very natural place for this functionality.
ProTip™: Use `src="//example.org/bar"`
This breaks import functionality.  Maybe because the import div no longer exists?

Discovered using git bisect
Wouldn't have been a problem if the client didn't make so many assumptions.
Human readable method and field names? Party!
Could you remove those changes from the pull request?
Noticed a small problem with this.  This `conf.py` is imported by affiliated packages, which won't have `astropy.css` in their `docs/_static` directory.  We'll want to either somehow make `astropy.css` installed and available to affiliated packages, or move this parameter to the astropy-local `docs/conf.py` file.
that sound wrong. can you revert it? if the original didn't work in IE after the compiler change then there is something wrong with the compiler
Important to note that this commit is 4 months old...

...but that's changing because, internally, django returns locales with lower-case letters, or at least does when we use the `django.utils.translation.get_language` function to determine the current locale. So we use the lower-case version in our internal storage of locales (as in this case where we're setting the arguments for the profile edit form).
> Activation des logs
I would think a patch is for fixing a bug in the current release, but I always thought the -... was always for alpha, beta, and rc stuff

0.1.0-beta1
0.1.0-rc1

etc.
You could also do this in your `.vimrc`, like so:

    let g:no_turbux_mappings=1
    nmap <leader>rT <Plug>SendTestToTmux
    nmap <leader>rt <Plug>SendFocusedTestToTmux
а должно быть?
ok but do you think it's good enough?
Yes sorry I was free coding in github and didn't catch it.  Great framework btw, thanks for the hard work.

Sent from my iPad

On Feb 4, 2012, at 1:58 AM, Nicolas Ruflin<reply@reply.github.com> wrote:

> I assume this should be $this->_took instead of totalHits?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/yellowandy/Elastica/commit/43105b81ae840d2290e4589662612a396e2279ea#commitcomment-932827
Because of browser quirks, using vclick for navigation like this can result in what seem like double events. Click is safer in this case, even if it's a bit slower.
hmm, autoescaping is kinda the most desirable feature of Twig :(
Dear  Freek de Kruijf :
      It is me Mahroos Ali from Yemen .I have two quations If you could help me :
          Would it be usfull in case of openSUSE 11.1?
          Could you please give me a link to all of CPAN modules witch has been shown in your koha wiki?
Are we not worried about errors from aio_write and aio_fsync? It looks like EAGAIN might happen, and EFBIG could occur on 32-bit systems.
forward because you want to preserve sender.
**Update:** Never mind, [WebKit uses `darkGray`](http://trac.webkit.org/browser/trunk/Source/WebCore/css/html.css#L505) which [converts to](http://css.coloratum.com/#darkgray) `#a9a9a9`.
../game/libgame.a(Object.cpp.o): In function `WorldObject::IsWithinLOSInMap(WorldObject const*) const':
Object.cpp:(.text+0x1ed2): undefined reference to `Map::IsInDynLOS(float, float, float, float, float, float)'
../game/libgame.a(Object.cpp.o): In function `WorldObject::MovePositionToFirstCollision(Position&, float, float)':
Object.cpp:(.text+0x327a): undefined reference to `Map::IsInDynLOS(float, float, float, float, float, float)'
../game/libgame.a(Object.cpp.o): In function `WorldObject::IsWithinLOS(float, float, float) const':
Object.cpp:(.text+0x1c8c): undefined reference to `Map::IsInDynLOS(float, float, float, float, float, float)'
../game/libgame.a(Object.cpp.o): In function `WorldObject::IsWithinLOSInMap(WorldObject const*) const':
Object.cpp:(.text+0x1f3b): undefined reference to `Map::IsInDynLOS(float, float, float, float, float, float)'
collect2: ld returned 1 exit status

Linux ubuntu 64bit
No, I think that project is completed, until anybody find bug  :-)
This call to `event.getFormattedMessage()` does not actually do what you expect in this case. To expand the pattern layout here, you need to use <a href="http://logback.qos.ch/apidocs/ch/qos/logback/classic/PatternLayout.html#doLayout(ch.qos.logback.classic.spi.ILoggingEvent)">`Layout.doLayout(event)`</a> to get the result.
I don't believe you are supposed to use Monitor priority for anything that might change the event.  This applies to the block break event and a few others.
Shouldn't there be a 

	GROUP BY
		pt.type

to prevent returning duplicate types?

(Background: I'm updating the Page Select Box field to use the new Manager and don't want to use fetchAvailablePageTypes() to only get the actually used types.)
Sorry for the messy commit. My text editor is set to remove trailing whitespace on save. Any change we could run pinking shears on this?
Just use sender()->property("stat").toInt();
DNS resolution is not working.
It's java.lang.Character.
Get rid of these place-holders. Only use one blank line to separate methods.
i like InnoDB more :D
does it fix this https://github.com/TrinityCore/TrinityCore/issues/3433? bcs for me it does not work
<pre>Do that and I will hunt you down. DB shall not be changed.</pre>
:trollface:
already fixed, need push the pull requests ^^
pero esto funcionaba bien no?
WOW，Juan 兄好勤快，看到已经完成第二章了，特来支持一下，新年快乐哈。
thanks to: Kingarus
Ouch ^ but I see where that is coming from.
The test cases look good and the feature seems to work.

I just tested the pre-0.9 versions of the Vim, Emacs, and Gedit plugins with `indent_style = tab` and `indent_size = tab` and they seem to work appropriately (they ignore `indent_size` instead of raising errors).  Can you check whether any of the other plugins work correctly with 0.9?

I've also updated the Gedit plugin to work with `indent_size = tab`.
Hehe, random. :-)
I'm happy with this part, but there's no need to include my name. (I'm in the commit logs, which should be good enough for anybody trying to find the creator.)
I don't see any reason to do this. Just use `@location the whole way through.
Nice! Thanks for this. Much easier than their given widget. FYI: This works with jQuery 1.4.2.
`isValid` should indicate that `NULL` is an acceptable value, and if provided the pointer's object is set to YES/NO accordingly
Seems that some header files were forgotten to add to git....
Áhh csak idegelt hogy mindig megkérdezte de akkor majd rányomkodok h igen :) de am ez miért baj ?
next view is loaded and store in parent's children property
Dernière modif,j'ai été obliger d'enlever le \ dans l'\acteur j'ai juste mis l'acteur et sa passe.

Merci pour ton job
Crois tu qu'on pourrais faire la même chose en demandant par exemple l'horoscope ou le programme Tv et ca nous répond en mettant les bon liens biensur? 

Cordialement

Le 13 févr. 2012 à 17:59, "Ronan Pozzi" <reply@reply.github.com> a écrit :

> voila la nouvelle version est sur github
> merci
> 
> Le 13 févr. 2012 à 17:53, Doriginal a écrit :
> 
>> Ok d'accord bon bah fonctionne nikel alors a part se petit truc.
>> Ah oui corrige aussi "Quand est sortit" par "Quand est sorti" Sinon il comprend pas ;)
>> 
>> Cordialement
>> 
>> Le 13 févr. 2012 à 17:44, "Ronan Pozzi" <reply@reply.github.com> a écrit :
>> 
>>> oui ça c'est un problème lié à SiriProxy et non au plugin.
>>> 
>>> 
>>> Le 13 févr. 2012 à 17:37, Doriginal a écrit :
>>> 
>>>> Sa marche mieux lorsqu'on fait un requête il repond et la deuxième il répond pas,il faut fermer Siri et le relancer
>>>> 
>>>> Cordialement
>>>> 
>>>> Le 13 févr. 2012 à 17:11, "Ronan Pozzi" <reply@reply.github.com> a écrit :
>>>> 
>>>>> Bonjour, au niveau de logs qu'elle est la réponse de siriproxy quand vous exécutez votre demande ?
>>>>> 
>>>>> Envoyé de mon iPhone
>>>>> 
>>>>> Le 13 févr. 2012 à 16:19, Doriginal <reply@reply.github.com> a écrit :
>>>>> 
>>>>>> Bonjour,
>>>>>> J'ai installer votre version française de imdb plugin mais il ne reconnais rien lorsque je lui demande quelque chose, quel que sois la question, Dois-je voir, quand est sorti, qui a joué.....
>>>>>> 
>>>>>> --- 
>>>>>> Reply to this email directly or view it on GitHub:
>>>>>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960594
>>>>> 
>>>>> --- 
>>>>> Reply to this email directly or view it on GitHub:
>>>>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960763
>>>> 
>>>> --- 
>>>> Reply to this email directly or view it on GitHub:
>>>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960871
>>> 
>>> --- 
>>> Reply to this email directly or view it on GitHub:
>>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960890
>> 
>> --- 
>> Reply to this email directly or view it on GitHub:
>> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960918
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960940
``` ruby
1.9.3p0 :007 > "крутой тэг становится пустым".gsub(/[^a-zA-Z0-9]/, '')
 => "" 
```

Non ASCII tag names become empty strings, and when we use chained tagged_with we got collisions on table aliases:

``` ruby
User.tagged_with('круто', :on => :tags).tagged_with(['не очень'], :on => :skills).all #Mysql2::Error: Not unique table/alias
```
@gabebw yes, it is.
k ill move it back and push it
S0... 2.0.2 overwrote these changes when I pushed from EGit.
@sebzur I'm not that familiar with Git myself but if it can be done in mercurial without forking again it can be done with Git.
I'll give it a try and let you know how it can be done.
fixed by chmod -R a+w log ;)
-----Metals are finished.
yep, I forgot about this :(
The problem with loading classes in parser is property placeholders. The value of the 'containerClass' might actually be #{..} ${..},etc., and it won't get resolved until later. That is why we should start adopting the pattern of Parser -> FactoryBean -> Bean (and we already do it a lot for ServiceActivators, Transformers etc.). Since in the FB the actual values are guaranteed to be resolved the above logic would be safe and 'kosher'. 

Also wrt to the comment in the catch clause. If this code were to stand than yes, I would say it should result in parser error
das war quark
This is more readable than the original code and it is possible that a template could be assigned to a node without actually enabling the confirmation email checkbox. Gets my vote.
Why do we have the <p> tag?
> So you flattenize method and their parameter / return value right?

Right. The general idea was to have one node instance in the path representing the parameter or return value which either hosts the violated constraint or is traversed (annotated with `@Value`). 

If we're talking about return value validation, `getMethodDescriptor()` or `getGetConstructorDescriptor()` returns the concerned method/constructor, while `getParameterDescriptor()` returns `null`. If we're talking about parameter value validation, `getMethodDescriptor()` or `getGetConstructorDescriptor()` returns the concerned method/constructor and `getParameterDescriptor()` returns the concerned parameter.
Yes I think we should improve here. Maybe something like

```
Node {
  boolean is Method();
  MethodNode asMethodNode();
  //or <T extends Node> T as(Class<T extends Node>);
}
```

```
Node {
  boolean is Method();
  MethodNode asMethodNode()
}
```
This target.ret address looks pretty high.  Is it a system dll?
de fato pegou, estranho é q acho q tinha testado isso... anyway, form ficou bem mais feio q o anterior... e aí senhores? voltemos a apenas o simpleform?
Why do you need libsurfaceflinger.so?
Where is libsurfaceflinger.so from? From CM7

Why don't you use the gralloc, copybit etc. from cm9?
Where did you get the gralloc, copybit, etc source you have in your github repo?

Why didn't you submit the changes to SurfaceTexture.cpp upstream? Are they only for this implementation and it doesn't make sense for others?

There is no history in any of your github repositories which makes it hard to understand what you did.
i dont remember exactly but i think it was because of stability issues...

On Sun, Jan 29, 2012 at 4:22 PM, adridu59 <
reply@reply.github.com
> wrote:

> Why you did revert this ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/DooMLoRD/Xperia-2011-Kernel-2.6.32.9/commit/8cc51967e89e2483469126d2745f96ff038cd01d#commitcomment-911924
>
On 01/15/2012 03:42 AM, Janek Bevendorff wrote:
> You could set up a virtual host on your machine for static.getnightingale.com and then map this domain to your local loopback 127.0.0.1 using your /etc/hosts.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nightingale-media-player/nightingale-website/commit/6e2ae6db2fe3eae212691d4b69900fb8756070a9#commitcomment-868469
Even easier, he could just use the local file path and edit it in the 
html before he pushes it...
No, this feature was stated in Opera changelog, but actually not working, at least on my setup.
you already create an ActorSystem in constructor, and this shadows `system` in TestKit
It's not published for sbt 0.11.3, or is it somewhere?
_.address == address

?
Because there were two identical sections. http://doc.akka.io/docs/akka/snapshot/cluster/cluster.html#singleton-cluster
`context.actorFor` should be the same thing, can you please change in ClusterDaemon.scala and ClusterHeartbeat.scala also
you're closing over "self" here.
I hope so ;-) and as far as I have tested it does.
It's supposed to publish the stashed event, and store it in publishedLeaderChanged.
Do you see that it does anything else?
Having a separate ActorSystem for all DSL usage is not that useful: if the DSL is nice, people will want to use it for their real systems. Therefore, every action of the DSL must take an `implicit system` and this special thing must go.
consider adding descriptive messages to all these
It’s not problematic here, since this `self` name is only visible within the `Throttler` trait, but I agree that it should be changed to `this: Actor` as a matter of style.
fyi, I have a better solution for this, HeapMemory will be plain 
`case class HeapMemory(used: Long, committed: Long, max: Option[Long])`

and I will remove those unapply things
Yey, sounds fun but wrong :)

#1 You ensure the integrity just 3 lines above  $attributes['value'] = (string) $value; so it is set
#2 One line lower you have && as the and operator, so unless you are breaking your own rules, I was right. Which one is it after-all?
#3 comparing 0 to the expression comes from older times, and it ensure that you're not asigning the value but comparing it, call it best practice if you will. Nevertheless I don't see it as an problem that would prevent someone from contributing.
should be @Vetoed
pourquoi des sauts de lignes ici ?
ccontrasts typo
this is the master version.
It seems a shame requiring Flask just to serve some files. If there are no fancy requirements, this can be done with standard library modules, see eg:

https://github.com/okfn/ckanext-spatial/blob/master/ckanext/spatial/tests/test_harvest.py#L31
https://github.com/okfn/ckanext-spatial/blob/master/ckanext/spatial/tests/xml_file_server.py
Good question, will remove it until all the templates for organization activity is done.
need a space before "importer"
In fact, doesn't work with val either ;) Need to use "def" if I want to call function I'm defining.
on a stream change, shouldn't we re-parse, even if we for the time being keep the old extradata?
if оборачивается скобками, внутри > 1 оператора
Weet ik niet? 755 een betere?
Пусть будет, я же говорю, предполагается, что она всегда на экране кейса уже заполнена.
Это я чтобы не забыть записал.
copyright/license header. also rename this file?
AFAIK, interfaces should be sorted by name.
first issue is simple
this path doesn't call reset
so after copy pressing a key inserts copied text too
The function `toAscii` was deprecated in Qt 5 because it doesn't always do what you'd think it does, since it depends on which codec is set for C strings. Please use `QString::toLatin1` instead (also below).
Use property syntax: `stack.count`.
Leave the copyright header in please.
You're not taking scope into account. Yes, it's used in a branch - but then a pointer to its contents is passed to initgroups().
Aha, finally a real-life use-case for this.

Some users will be so vindicated :)
What about `ch`? I prefer terse.
I know its 512 and unlikely, but strncpy?
This change changes the ABI/API! We can't alter the signature of a function that's marked NUMPY_API in a released version of numpy.

It looks like `PyArray_CreateSortedStridePerm` is only added in 1.7, which hasn't been released yet, so perhaps something could be worked out. But this will take a bit of thought, since this PR is too big a change to merge into 1.7 at this point in the release cycle...
The second `if` is part the body of the first `if`.
`feeds` config option needs to be documented.
aren't you supposed to free spki data structure?
please refer CODING_STANDARD.

your code need to be cleaned.
This line-break seems useless and there should be space before and after '=' .

after cleanup you could rebase this commit to one commit.  a test file needed too.
Why can't you just return a copy of object->postfix[0].c?
Would calling it a wrapper be more appropriate?
оставил
ну сейчас оно вот так:
![пример1](http://img835.imageshack.us/img835/8710/snapshot88.png)
а хочется как-то так:
![пример2](http://img838.imageshack.us/img838/3120/snapshot892.png)
или так:
![пример3](http://img577.imageshack.us/img577/9669/snapshot893.png)
ну чтобы линк не налезал на элемент
On 13 Aug 2012, at 15:56, Aaron Patterson wrote:


In activerecord/lib/active_record/relation/query_methods.rb:

> @@ -297,6 +297,19 @@ def where!(opts, *rest)
>        self
>      end
>
> +    # Join scope clauses using or
> +    #
> +    # The passed scope is joined:
> +    #
> +    #    User.where({ name: "Alice"}).or(User.where({ email: 'joe@example.com<mailto:joe@example.com>'}))
> +    #    # SELECT * FROM users WHERE name = 'Alice' OR email = 'joe@example.com<mailto:joe@example.com>'
> +    #
> +    def or(scope)
> +      left  = self.where_values.inject {|l,r| l.and(r)}
> +      right = scope.where_values.inject {|l,r| l.and(r)}
> +      self.unscoped.where(left.or(right))


Haven't tested, but would self.where_values = left.or(right) work?


Sadly not,  the assignment works, but then after the code has executed the assigned value is lost again :(

—
Reply to this email directly or view it on GitHub<https://github.com/rails/rails/pull/6817/files#r1362549>.




If you've received this email by mistake, we're sorry for bothering you. It may contain information that's confidential, so please delete it without sharing it. And if you let us know, we can try to stop it from happening again. Thank you. 

We may monitor any emails sent or received by us, or on our behalf. If we do, this will be in line with relevant law and our own policies.

Sage (UK) Limited. Registered in England at North Park, Newcastle upon Tyne, NE13 9AA. Registered number 1045967.
This include can be outside of the included hook. When you `extend ActiveSupport::Concern` they are lazily loaded.
Perhaps we should invert the `unless..else` block here, wdyt?
Ah, the pitfalls of copy-paste :-)
Fixed, thanks.
Generally we prefer commits to be squashed, so amend/force push.
Not possible on the model Project in question. Should I invert it to query on a `developers` relation instead? I guessed `sum` was trivial here so left it out.
You are right in that we don't use the argument again, though.
noDocTemplatesCache.getOrElse(aSym, new NoDocTemplateImpl(aSym, inTpl))
`templateParents` is now fixed, so that we can distinguish `T` and `T()` in parent type lists. Now there's an interesting question. Should the following be an error? (It's not an error at the moment)

```
trait T
class C extends T()
```
I borked the logic of the above, but the gist is stil sound (especially the 2-case match on mix)
Would it be possible to reduce duplication w.r.t `MultiStringSetting`?
You can view/edit the README, and all other files not under one of the real modules.
Interesting that you chose to do it through params than `@product.available_on` after the assignment.
I added another test to make sure this is triggered.
I'd rather just remove this.  You can get something very similar with http://about.travis-ci.org/docs/user/browser-extensions/, which looks nicer anyway.
Why don't you just use `isinstance(x, MatMul)` then?
I strongly object!

Please format the code properly. PEP8 is the standard we use.
I think you might just need to make https://github.com/sympy/sympy/blob/master/sympy/solvers/ode.py#L1462 look for something like `expr.atoms(IntConstant)`.
That's a very cool idea, thanks for spotting it!
done.

If I interpret this right (:D), thank you for your time again :-).

I hope pushing this over does not screw up this note.
otherwise it'll always append to the end.
Need committer name here.
These additions are not OK. The additions of the netbeans ignores and DS_Store should be added to your local git settings global ignore -  see http://help.github.com/ignore-files/
is this debug code?
Yes I did, thank you catching that.

Justin
On Jul 27, 2012 8:24 AM, "Ben Noordhuis" <
reply@reply.github.com>
wrote:

> > +AM_CONDITIONAL([Linux], [false])
> > +AM_CONDITIONAL([SunOS], [false])
> > +AM_CONDITIONAL([DragonFly], [false])
> > +AM_CONDITIONAL([FreeBSD], [false])
> > +AM_CONDITIONAL([NetBSD], [false])
> > +AM_CONDITIONAL([OpenBSD], [false])
> > +AM_CONDITIONAL([Cygwin], [false])
> > +
> > +AM_CONDITIONAL([UNIXLIKE], [true])
> > +#if the compiler is gcc add some extra options
> > +if test "x$GCC" = "xyes"; then
> > +   UV_CPPFLAGS+="-pedantic -Wextra -Wno-unused-parameter -D_GNU_SOURCE "
> > +fi
> > +
> > +#determine whether or not we need internal lib resources
> > +AC_CHECK_HEADERS([event.h],
>
> Did you mean ev.h? event.h is from libevent.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/joyent/libuv/pull/501/files#r1252396
>
Keep this here.
Those methods should be moved to the ORMInterface and that's what should be passed in the loader, not the ObjectManager directly because I don't see any reason to tie this to doctrine.
Should really say, we strongly recommend
Spaces after if.
Is there a reason you don't just name $tag_args as $location in the method signature? I don't see it being altered prior to this definition, and all this does is effectively renames the variable for clarity.
Please add a comment for each row explaining what it is testing.
Space between `if` and `(`.
This should be at the very top of the function. It doesn't affect behaviour or performance at all as they're all hoisted anyway.
Why not SELECT COUNT(user_id)?
style nit: _, ep
DOT is a tool for generating graphs. If DOXYGEN_DOT_FOUND = NOTFOUND, then there will be no graphs.
Yep, let's do that. So an explicit dev mode option? Or just a sphinx options setting that can be wiped for dev? Determining colour mode by ansi support is good.
Memo to either of us: replace `_find_first` with `findfirst` that's now in core.
Missing "this."
space is not used anywhere here -- I guess SurfaceDisk should handle it

also it might be longer but more 'coherent' with the rest to call it SurfaceDiskQueryEngine, since we do have some primitives as Sphere which just define the neighborhood, while QueryEngine's use them... and first I thought that it is such a primitive.
Yes, don't show uuid.
On Mon, Sep 10, 2012 at 10:34 PM, Matthew Rocklin
<notifications@github.com>wrote:

> In theano/tensor/tests/_test_mpi_roundtrip.py:
>
> > @@ -0,0 +1,48 @@
> > +# Run using
> > +# mpiexec -np 2 python _test_mpi_roundtrip.py
>
> Yes, that'd be nice. I do this on my system but there are a few issues
> that need to be handled first. I'm not sure what the standard solutions are
> to the following:
>
>    1. I need to specify the path to the file - are paths always going to
>    be from the theano root directory?
>
> __file__ and realpath can help here I think... I think it makes sense to
navigate from the theano root dir.


>    1. Does subprocess work on Windows?
>
>  No idea.

>
>    1. What happens if the tester doesn't have mpiexec or mpi4py?
>
> This is an important question, I don't know how to deal with it properly.
I see it as being similar to the GPU, in that you are encouraging the user
to type something like "compile me a function to execute the following
graph, knowing that I have mpi-enabled machines X, Y, and Z, and X and Y
have GPUs." I don't think that our current interface for device selection
on GPUs is very good, because it is not in fact possible to tell
theano.function(...) to use a particular device.  The reason things were
done the way they were done is that memory must be dispatched to either the
GPU or the CPU when the user calls shared(), before the graph to be
compiled has even been built.  I'm afraid that a slightly different
metaphor is necessary to support a better approach to memory allocation and
function compilation. That's a bigger question than belongs in this thread.
I mean:

"When it is allocated"

I'll update the comment.
pep8 calls for 2 new lines above function defs
Fix it in indexcountexecutor.h. These two parts are the legacies of IndexScanExecutor, same problem happens there. Should I also fix it in indexscanexecutor.h?
Even a 0-length string can match pattern '%'.
Model name has to be displayed even when there is no model.group!
escape(value)
Where this class is used ? :D
Cool. As long as we have them stashed somewhere to be re-added later.
Not really a comment, just wanted to say "sweeeet"
I did a little more investigation, and doing the no-pollute app-loading properly is much bigger job than what I can do before feature freeze. The basic problem is that doing it for a single model is somewhat easy (if ugly), but when you need to be able to load a set of models which have relations to each other things get complicated.

I think we should be able to have the .auto_register flag with the meaning that a model with .auto_register = False will not be loaded to cache, and will not alter other models for foreign keys or m2m tables. This could make the SQLite part less fragile. But, doing even this pre-freeze seems hard.
was the version of python-noq package, returned to python-oq.
This field should be declared with `null=True, blank=True` as well.
Are you volunteering to update this for every release?
First of all, this string is apart from madlib version, strictly speaking.  PGXN distinguishes the version string of "distribution" (the packaged zip file) and the actual module extension, which is described below "provide" section.

And I really want to automate the version string with Version.yml.  It may be possible with configure, but this file should be located in the top directory which is out of build/, and may make packaging process complicated.
`#{bin}/gdmap`
I set the value of build to 421 (as per my Xcode 4.4.1 installation on Mountain Lion). If I remove the fails_with block, the formula fails. Any suggestions?
good catch on timeout! re: the message, the handle_call expects 3 args and just passes that through to the db driver. otherwise we need 'duplicate' code just to handle a 2-arg message.
We'll host the API at api.p2pu.org so we don't need to prefix the URL with api/
Ah. I missed 'sent' being in `add_outbound_message_key()`. Agreed let's keep it for the transition and worry about it later if needed.
I think putting an `else` in there would make the fact that the `if` block returns less obvious. The actual semantics are "do some things, if the condition is met return, do more things".
Personally, I'd run it as:
```
%if isinstance(thing.editted, datetime):
    <time ...>
elif thing.editted:
    <em>*</em>

Regarding translation - the parameter to the `_()` function should always be a static string. So it should be something like:
```_('last edited %(timeago)s ago') % dict(timeago=timesince(thing.editted)```
I think this is clearer without the English labels on the links
It's bound to get confusing if people need to accept two licenses.  Is there any way to use maven, and avoid this extra license?
Can you try to make sure this doesn't change indentation?  Thanks.
when this error is passed back up the callback stack, it ends up getting ignored after the async foreach :/
I'd prefer spelling out $mobile-web instead of $mw. The var name is ridiculously long, but I think that's better than wondering "what does 'mw' mean?" Not a biggee, though.
F2 is usually a Windows only shortcut. Mac generally uses Enter key for rename shortcut.
Should use hostEditor.setSelection() instead
Should enabled be a property of a MenuItem or a property of a Command? Think of Undo as an example. It's only enabled when there's history. Should the CommandManager dispatch when a Command's enabled state changes or should the command modify it's MenuItem?
This is to make sure we only allow "select and continue" for url hinting. So we shouldn't remove it from here unless we want a general logic of any completion string with ``/`` at the end should not close the hint list, which I think is not right for other types of attribute value hints.
demütiger - LOL
Dies ist ein Anfang.
The option is `shift` (it's referencing `opts.shift`). The @param in `testPreventDefault` is the one that's actually incorrect (it says `shiftKey` instead of `shift`).
@nlanget 
Having the minimum SNR hard coded is dangerous ...  It should be a reloc option.
Micro-optimization: << 2 is the same as * 4

(i << 4) | 1 is the same as i*4 + 1

same with 2, 3
I tried a couple different methods to do this in CSS, but I couldn't get any to work.
Same as above, but here in particular it's confusing what dt ends up being with the parsing and unparsing. Is it supposed to be a datetime in "dddd-dd-dd dd:dd:dd" format? Internationalized human-readable format?
Please use the relative class name `Property`
give me an eclipse code formatter that does that and I use it.
Probably shouldn't be naming a variable 'obj'. This seems like it should be teams and the variable above something along the lines of teamsApiResponse.
Do you really need an array here ? Can't you use directly a List<Double> structure ? That may be cleaner, faster, and the memory footprint will be lighter...
Yeah I dig that too @mtodd, more expressive. I'll make the change.
current_dialog is aliased to current
UI has UI.current_dialog

Dialog has Dialog.current_dialog, also aliased to Dialog.current
Тут ее не положено писать почему-то
Please put this line back as it was, it was like that for a reason.
`for ( var i = 0; i < contentContainers.length; i++ ) {` -> `for ( var i = 0, l = contentContainers.length; i < l; i++ ) {`
This is actually there because some of our editors have elements in them already for layout reasons. This way we can force start/end to be at the top of the editor in whatever tab it's placed in.
I think this is wrong.  You might go on irc, have a discussion, and change your mind.  It would be better to maybe highlight the one they clicked with a color, or have a label beside that shows their choice.  I think you want to be able to change it.
Can `results.length` ever be less than 0?
Yeah, it's how the layout system works. `text!` will deliver a piece of text and not try to run it as code. Great for including HTML in your projects :D.
revert this line change
This code has been through the ringer on so many projects now I think I'm going to leave as is . If we can make it better for the future though that would be awesome!
This can probably be moved to its own method, that is only defined if the socket constants are defined (since this condition doesn't change over time).
The Windows information got promoted to the kernel selection branch level with the conversion to ```platform.uname()```. It cut out a lot of duplicate code.
Doesn't HTTPSConnectionPool, pass it into VerifiedHTTPSConnection?
This code is used in conjunction with [Charger](https://github.com/sonatype/sisu-charger) in Nexus Core that enables to "parallelize", end hence, greatly improve response times when Groups are involved in requests, see [NEXUS-2592](https://issues.sonatype.org/browse/NEXUS-2592).

This part was "done" but never tested and let into production (by default this feature is disabled, dormant), but you can enable it with "hidden" (read undocumented) switch. I did some preliminary tests, and stopped when I discovered similar bugs in HttpClient, that was since fixed (it was depleting sockets, as it was using pre-1.4 API as HttpClient 3.x we use is still 1.3 compatible).

This code above does the "parallel request" on behalf of given group, and if log would be inited as in your code, we could not differentiate what group was it, etc (all the requests would log to same log). But in general, I do agree with your statements, but in this very place, I do not feel it as "very problematic".
Or rather, this is chored.
This is using a write access url, please use the read only version.
I would like usage of d_searchType to be more readable - right now one has to remember what 1/2/3 mean.
Can this line be replaced with,
```ruby
 environment.systems_readable? || User.allowed_to([:systems_readable], :system_groups, self.system_groups_ids, self.organization, true) 
```
the "true" in the last param will return true if any of the passed in tags match. If this approach works , you will want to do this in the rest of the models to prevent a sql query per environment. If your argument was that systems model has no business understanding system groups model, you can add a method in system_group like
```ruby
def self.systems_readable(sys_group_ids, org)
    User.allowed_to([:systems_readable], :system_groups, sys_groups_ids, self.organization, true) 
end
```
or any variation of that  for each of the 'any' methods..
Not a typo - it was just string concatenation, I rewrote it to use @template.concat.
why not elif?
Don't use var when it isn't clear what type this is.
I'm not sure to understand the need for those. Does click/dblclick not enough?
The format and scale "properties" are implemented in `__getattr__` so are not quite suitable for lazyproperty (and anyway see following).

The `Time` object is *mostly* immutable - the `scale` and `format` and internal time values `jd1` and `jd2` cannot be changed by the user.  But currently the attributes `opt` (which is a dict of options) and `lat` and `lon` are open to change.  So I think I can do a Time instance cache (`self.cache`) which caches the output from `__getattr__` for format and scale outputs.  Then if `opt` or `lat` or `lon` get changed the cache gets cleared.
This and line 45 act on the local filesystem. Perhaps you could use rt:cmd instead?
Wow, this is so much stranger than I thought.

If I pass a string instead of an iolist to stream_cmd, the command still gets run, but a whole bunch of tests fail that weren't failing before.

Lots of ```undefined method `X' for #<PrebuiltTestServer:0x007f94bd8d2530>``` errors
Last rebase just adds this comment.
Why make it slower by not caching it? Adding the static Array idea to rebase.
The line just above that is 

    if (isset($this->info['extension'])) {
        return basename($this->name, '.' . $this->info['extension']);
    }

If there is nothing, it returns ''. `$this->name` is set from `basename()` in `__construct()` so will also be ''.

This would match better as you get consistent empty strings vs sometimes false, sometimes ''? People would not have been able to use $File->name() === false because of the line above, so I think it should be safe.
why using a property access whereas the code was using a getter previously ?
this method should have a full or no description :)
I was just as surprised as you, but Amber put a pre-save hook in my vimrc for python files :)
this is PHPCR specific: you can not simply add to the db, the parent object is required. so maybe the wording could be changed to add PHPCR to the message. ``... to create new objects with PHPCR.``.
I'm a bit confused about this: If it's creating a concreate collection, shouldn't this be in the CollectionInterface?
This looks a bit weird.  Are you trying to handle a race condition, with the double checking and throwing?
@takluyver,I didn't want to add a ```depth``` argument to ```caller_module_and_locals()`` without renaming the function. See if you like https://github.com/scottt/ipython/commit/a61f44535f6be5d6f844886e0854cb2152fac592