labofsahil/github-event-dataset-2012 huggingface dataset:
No, you understand correctly. I'm working on Eternizer, an old style module, so that way  cannot work for you, I think.

So indeed it's very strange that updateSchema does not work.
On Wed, Feb 8, 2012 at 12:53 AM, Anatol
<reply@reply.github.com>
wrote:
> BTW on linux you just need CAP_SYS_CHROOT capability to be able to do chroot(). IMHO it is safer than using suid.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gittup/tup/commit/3d7f7cf8018f0b23e6676d2790f2cc8487117c3d#commitcomment-944138

It also needs to call mount() in order to mount the /dev file-system,
so it would need CAP_SYS_ADMIN as well. Or is there a better way to do
that?
Did you remove this line by mistake ?
Dmitry, I'm about to release 1.4.4 version. I think there is no reason to rush implementing ruby less css engine support. It will, most probably, go to the next release (1.4.5). Is it ok with you? Or you would like to postone the 1.4.4 in order to make this new processor part of that release?
For me, significantly-less space consumption and almost instant cloning after first clone is really beneficial. But, I think harrah should decide on it.
Or you just do a TTAS
Well then we need more opinions, because I think that is a stupid reason to not use the interface that is already in place. As I said before, distinguishing between the two different formats is very simple... its not like we are returning two different array formats, but rather a string vs an array.
Ah it was wrong and now it is correct. Was coming from ApiDoc where it was still incorrect :-) Great. TY.
Well, engines tend to rely on rails internals and I feel like watching files to update routes is a common pattern. This class is part of ActiveSupport and changed its initializer without any deprecation notice. I just think this is going to break a fair number of engines (and named two). There is not a good way to detect which version of the file people are using either... I am just looking for an easier path for engines to be able to support multiple versions of rails.

Jeff

On Jan 17, 2012, at 3:16 PM, José Valim wrote:

> Sorry, but I don't understand exactly what do you mean. This class is pretty much internal to Rails. The documentation is only for people that wishes to implement their own file updater. In other words, this implementation is internal to Rails, but the contract used by the reloader should be stable from 3.2 on. Maybe we should add such a note to this class docs?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/rails/rails/commit/80256abb39332dd49996b909d6f0413a15291a90#commitcomment-876271
sure!  I'll revert. :-)
They require it.  toIndexedSeq doesn't because IndexedSeq is covariant.
ok... we have a first try (i guess i've inspired you yesterday with the pink avl applet :))
it is in some ways better than the old one; but
1. there is too small difference between NORMAL and DELETE
2. we will need to increase radius so that the numbers fit in the nodes
Installed this with cabal and my code works again against llvm 3.0.
https://github.com/vitek/cython/commit/19bb02e714e9a154c29250c487bbe8e58e1daae0

I don't understand your concern about non-literal case. If default isn't  literal its value is stored in a special defaults struct and it's much more easy than literal case. See
http://trac.cython.org/cython_trac/ticket/674
and
https://github.com/cython/cython/commit/8c0f27d039c44d8dab9e0882145b22fcc27d1e08
@tahg - Any chance to encourage you to take this one step further?
`/give <player> <item>[ <amount> [<data>[ <damage>]]]`

That way potions could be given directly with a single/simple command...
You you comment on this?
if offsetWidth is 0 the element is hidden from the user's view and cannot be focused in oldIE.  Some other browsers disregard this but it doesn't make sense as far as I can tell.

What is the useful purpose of focusing a non-visible element?
Woohoo <3
--
Romain Chossart



On Fri, Feb 17, 2012 at 10:25 AM, Pierre
<reply@reply.github.com>
wrote:
> Yes it's supported in tmux now :p
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/piec/dotfiles/commit/8f49fed38f603071cba7df1231c946d5855afe13#commitcomment-977404
You should rebase and force push this commet to make sure it's not in the repository any longer. :)
m_ naming convention?
Is it a correct way to append CFLAG for ./configure?
and that's the problem. Warden is too inestable. Disable it
Профайлер забыл, мать его!
Effects of comment setactiveobjectstate ?
http://pastebin.com/9BwHQyWP

about 300kb :D
https://github.com/Radicalmangos/Radical/commit/5ec001a2464660a13c28e8927bbbfb5944fe9f5e
I've to add the netbeans project folder on .gitignore.

Thanks for reporting it.

On Jan 22, 2012, at 2:07 AM, John Du Hart
<reply@reply.github.com>
wrote:

> This commit added a bunch of temporary files.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/rainphp/raintpl/commit/ea85517b0f5d4614a374563d9db8fdf8208ff69f#commitcomment-889491
Uh, really? The status fetch was not there before 1.6?
That would mean I have to code some switch (like you mentioned) or make i.e. xml-rpc needing s9y 1.6 
at least, as it is relying on the status fetch, too.. Mmhh..
the issue is fixed in ae7e0049
When you say, "make will use the, now working, implicit rules," what do you mean by "now working?" Was there an issue in Make with implicit rules that was recently fixed?
This docstring desperately needs an example. I'm having a hard time figuring out exactly what it does otherwise :)
коммит мессаджи должны быть более содержательны
Ooh, I hadn't seen the use of anonymous/anonymous before, this is pretty cool. I shall be making this the standard fallback option. Thanks for making me aware of that little detail. :)
the failure mode when `npm test` is run but phantomjs is not installed is a test failure that says "PhantomJS binary is in path and executable failed".  Is this sufficiently clear, or should we do more?
Is the view locked down at all with this? I can still go to the setup permissions URLs etc.
Oh, I see what happened. We both removed the elevator/shoot classes and committed that to the repository.

Hmm, I'm not actually sure what the best thing to do is in that situation...

(Anyway, I'm leaving now. 8pm, still at MHS, hw, etc. I'll sort it out tomorrow.)
Obviously we may build and provide several versions of composer.phar for different compression methods, but this look ugly for me. 
Maybe its "phar" extension related bug? As I see, there are no capability to combine several compressed phar's i one, and then load that version of it, wich PHP supports...
Саша что это? Что за комит такой 'dapi' ? Если это те изменения что были у меня - то почему ты их просто не вмержил?
Whoopsie - that should be arg3, not 2.
Yo dawg, I heard you liked to import code that you're installing before you install it.
the spelltimers and creatured id / spellids are correct ?
Thanks!

On Mon, Jan 23, 2012 at 3:41 AM, Steve Lindsay
<reply@reply.github.com>
wrote:
> Not sure if Daniel will get these messages, you might be better off commenting in the merge comments:
> http://github.com/MayDaniel/clj-mail/commit/d529ddef6234e9a970d41b29d8c21293a6782032
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/MayDaniel/clj-mail/commit/0e77c9b3f770a5703e6a101aa09bcf3a27599a9a#commitcomment-891846
Why you use element counting instead of checking s.empty first?
`i` might be overflow when s is much long range.
Or, this is a mixin declaration:

``` css
.grid-column {
  float: left;
  margin-right: 10px;
}
```

extend with a prefix `!`:

``` scss
@for $g from 1 through 4 {
  .span-#{$g} {
     !@extend .grid-column;
     width: 40px * $g + 10px * ($g - 1);
   }
}
```
Should I change the name of `_bytes` to `bytes` then, since the whole point of putting the underscore was to make it private, correct?
why adding a use statement if you use the FQCN after that ?
pk tu as fait deux commit différents avec le même titre ?

et puis est-ce que tu as tester ton code ? Pcq normalement tu n'as pas le droit d'envoyer du texte au client avant de le rediriger ...
Yeah, it basically does two things:

 - checks if the variable exists
 - does `return (bool) $var;`

... but the current logic checks if the value is exactly boolean `TRUE` and I'm not sure if that's intentional, so I keep it.
By "explode" do you mean it makes for a cool explosion effect, or that it causes a division by zero and breaks the simulation?
Wouldn't it be better to call `setDepthTest( true )` and `setDepthWrite( true )` first thing inside WebGLRenderer's render()?
cc @alteredq
Yup. Be sure to pull every time you start working on anything, as things will get shuffled around a lot at this stage.
I don't get your second line.  What do you mean, and what's an example?
Lioncash kirjoitti 6.2.2012 kello 7.45:

> I need input on this one. Should VGMStream be put into this project. I think it should, in my opinion because it does play back some formats from games.

I have nothing against adding more playback engines, provided clean rules (preferably content scan) can be used to robustly and in crash-free way make positive identification of the file format. (In particular: no assumptions of minimum file data length, and no array out of bound errors because of blindly following offsets in a file.)

You may have noticed that I have basically moved the file scanning to be suffix based, in sense that file is only given to the plugins that claim to recognize its suffix. After suffix identification, plugin is requested to analyze the file and if it does not report a MusicInfo instance as result of the scan, it will move to the next plugin in list.

Earlier version of DroidSound had a configuration option to always perform the content scan, but I have removed it. Consequently there might be some misnamed files we could play but do not because the right plugin is not tried against them. More robust context detection might be welcome, so I'll probably change the API around a bit and use content detection immediately ignoring the filename if it seems that content detection with that plugin will work in a robust way. For HivelyPlugin and GMEPlugin at least there seem to be magic strings we can use. On the other hand, Amiga formats suck, so they must always be based on filenames. (Even UADE itself internally uses filename to choose the player.)

Metadata extraction would be best done via native code, but UADE and VICE probably will use java-side code no matter what: UADE because it can't give any metadata info other than selected player and things like that, so we need to explicitly support important formats' metadata extraction manually, and VICE plugin already has robust SID metadata parsing, so no need to do anything on native side to cover that.

-- 
Antti
Since this is a different WSDL, I would prefer this be a separate resource and not part of virtualserver. This should be f5_virtual_address instead.
In my opinion, 30 minutes seems like a little bit too large of a window, but 5 minutes is too small. I'd meet in the middle and say 12-15 minutes personally.
GodPackage、SPPackage、StandardPackage、WindPackage、WisdomPackage、YitianPackage、YJCMPackage，这几个的LANG翻译文件的编码还是Unisiode格式，我在创意群里更改为UTF-8的编码了，你可以去下载后直接替换原件就成，别更新了，更新的话好像不能解决编码错误乱码的情况。
Add `/usr/local/transmission/bin` to PATH in `/etc/profile`
This can be done easily by advanced users IMO

But, why not.
To help out, here's an error log on the failed upload:
2012-03-09 16:01:39 ==> Uploading document: age_related_ebv_dlbcl.1.pdf
2012-03-09 16:01:39 ==> CouchDB enabled for storing documents.
2012-03-09 16:01:39 ==> Failed to Sote document content to CouchDB.
2012-03-09 16:01:39 ==> Document ID:
2012-03-09 16:01:39 ==> Array
(
    [0] => openemr
    [1] => 11age_related_ebv_dlbcl.1.pdf_2012-_03-_09_16_01_39
    [2] => 1
    [3] =>
)

2012-03-09 16:01:39 ==> Uploading document: age_related_ebv_dlbcl.1.pdf
2012-03-09 16:01:39 ==> CouchDB enabled for storing documents.
2012-03-09 16:01:39 ==> Failed to Sote document content to CouchDB.
2012-03-09 16:01:39 ==> Document ID:
2012-03-09 16:01:39 ==> Array
(
    [0] => openemr
    [1] => 11age_related_ebv_dlbcl.1.pdf_2012-_03-_09_16_01_39
    [2] => 1
    [3] =>
)

2012-03-09 16:01:39 ==> Failed storing document to CouchDB.
2012-03-09 16:01:39 ==> Document to be stored in HardDisk.


2012-03-09 16:12:15 ==> Uploading document: brady_sros_1_23_12.pdf
2012-03-09 16:12:15 ==> CouchDB enabled for storing documents.
2012-03-09 16:12:15 ==> Failed to Sote document content to CouchDB.
2012-03-09 16:12:15 ==> Document ID:
2012-03-09 16:12:15 ==> Array
(
    [0] => openemr
    [1] => 11brady_sros_1_23_12.pdf_2012-_03-_09_16_12_15
    [2] => 1
    [3] =>
)

2012-03-09 16:12:15 ==> Uploading document: brady_sros_1_23_12.pdf
2012-03-09 16:12:15 ==> CouchDB enabled for storing documents.
2012-03-09 16:12:15 ==> Failed to Sote document content to CouchDB.
2012-03-09 16:12:15 ==> Document ID:
2012-03-09 16:12:15 ==> Array
(
    [0] => openemr
    [1] => 11brady_sros_1_23_12.pdf_2012-_03-_09_16_12_15
    [2] => 1
    [3] =>
)

2012-03-09 16:12:15 ==> Failed storing document to CouchDB.
2012-03-09 16:12:15 ==> Document to be stored in HardDisk.
хмм ишледмикин7
We won't reject your patch because of it, but would prefer that you stick to our conventions. In this case they're not explicitly defined, but easily inferred from looking at existing code.
I completely agree the unapply(_).get is not really intuitive...
shouldn't it be ``to track`` ?
Using the event property handler is much faster for attaching a single event handler to an object
Yeah, that's right. 

Unfortunately, although I thought I had it all worked out - having the `highlight_modified_tabs` rule in the theme caused new side effects for users with a fresh / reset ST2 install. For those users, the dirty file indicator would not show up at all regardless of the tab close button setting. This was because the `highlight_modified_tabs` setting has been removed from the base settings which meant it was always false.

I've had another crack at it though, and just committed a tweak that I think should so make it work for everyone's settings. Hopefully that'll be it ;)

It might be worth letting Jon know that there's a use case for keeping `highlight_modified_tabs` around. Although the setting is still working at the moment, he may be looking at turning it off completely if it's no longer in the base settings.
I think User#current_company should return the instance, not the ID.  It only exists for the explicit purpose of scoping queries, having to find the Company everywhere is awfully WET :-D
WTF? <_<
Haven't pushed that change yet. I'll try and send it out tonight
I am building up a set of feature tests for expand-region, to document the wanted behavior of the expand-region function and make sure no features are broken later.

I'm also considering writing up some standards for expansions - like that it is best if expansions have both inner and outer versions.
Em 21-01-2012 00:54, Gregory Brown escreveu:
> Been meaning to do this, but it'd be better to organize within a namespaced folder... i.e:
> (...)
> I'd expect `lib/turing_tarpit.rb` to use `require_relative` to load all the files in the `lib/turing_tarping/` folder. Definitely would accept a pull request for that if you don't mind doing it.

Done!

--
@renato_bill
Renato Zannon
Steve, let's change this back to just `element.scrollTop = previousScrollTop;` IE8 otherwise changes the scrollTop if you select the second item.
I love clean ups. It filters out all these copy&paste noobs :=)
What is "Foo Bar dir:"?
2012-02-18 22:02:16 addToGossipMenus: GossipMenu 9821 not found
2012-02-18 22:02:16 Not handled grouped condition, SourceGroup 9821
Crsah? oO
I was reversing SMSG_AUCTION_OWNER_NOTIFICATION last night and this gist seems correct.
drop this variable, no longer used
2012_01_05_06_world_spel_script_names.sql -> 2012_01_05_06_world_spell_script_names.sql
I really like and apreciate the work this people do in special with instances, but I do not have database for this core, I consider this people (Broodwirm team and in special zwerg) and I am not really their friend then I have not ask for a database they use, bit is trinitycore database + their updates. I work woth this core but I try to do my best updating with what they do.
I don't see it. What's wrong with #1186? We still have a newline added after the herecomment, just not in the lexer. Though, that's how my original fix for #1186 worked... why did it change? Looking into it now.
:+1:

it's definitely better now!
Looks good :-)
I'm not the smartest guy ever, but I had trouble understanding this example and had to read the tests to see what's going on. 

I think part of the problem is that you're still initializing with .new; it'd be clearer with a different method.

Also, maybe you could include a line or two about why you'd want this. I'm having trouble picturing when I'd use this.
Updated
Waiting for review
Thanks! I've added these changes to the SVN repo on Google Code.
http://code.google.com/p/simpleinvoices/source/detail?r=3871
Mostly concerned with this change. This would exclude annotations from equality checks, which could have unpredictable results.
I'm guessing you're running PHP with error_reporting set to E_ALL? Try running it with E_ALL ^ E_NOTICE to alleviate some of these annoying notices. If we were to initialize these variables though, we don't want to set them to empty string. And we need to set all of the possible instance variables for this object.
btw you got one error in mastery system will fix it later :P
About haml: I think, it can be used only if every developer will agree to move to haml. If not, it better to stay with erb.

But maybe erb2haml can be useful as erb checking tool, because, actually, it breaks ror-e tests and site :) And it should not break anything in ideal situation.
Done e5296a5738bd27603652b578fed3d5e5416bcadc
I'm getting bad with spelling in commits >.>
Sorry, übersehen ;-)
what i basically does is create a binary image with set values to give more
emphasis on the illuminated strip removes any noise (small blocks that were
picked up during the threshold process) fill the shape it gets from that
process and thats where you get the pixles for your program

Rajath

On Wed, Jan 25, 2012 at 10:16 PM, Rajath Shetty
<shetty.rajath53@gmail.com>wrote:

> you could with get height and get left i think i still have not gotten to
> test this out we ran out of time at school
>
>
> On Wed, Jan 25, 2012 at 10:09 PM, MichaelStevens <
> reply@reply.github.com
> > wrote:
>
>> How can this work? If we get a rectangle we can't get the two different
>> sizes of the sides.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/erhs-53-hackers/VisionTech/commit/6ca04764da6f9afef20ade26fc8183a124090f62#commitcomment-902911
>>
>
>
(.*[%]) est différent de .*[%]
car du coup dans match[1].second tu te retrouve avec 04d.dpx et non pas avec %04d.dpx...
Après pour supporter le %5d, je pense que je dois modifier ca après:
try
{
matched = match[1].second;
matched.erase( 2 , matched.size()-2);
patternWidth = boost::lexical_cast<int>( matched );
}
catch( ...)
{
matched = match[1].second;
matched.erase( 1 , matched.size()-1);
patternWidth = boost::lexical_cast<int>( matched );
}
une autre idée ?
What happens if the column name has a "." in it, e.g. "Author.FirstName"?
Yeah, I see that. What I meant is it adds *more* coupling :_(

I always go back & forth about mixins and where to put the state. I guess my thinking now is, `matchers` really belongs in the Commands module - all the manipulation of them is done there, it's an implementation detail for Application. Likewise with `callbacks`. All Application knows about callbacks is the `trigger_callbacks` method. So why does the state have to reside in Application? Why not in Commands?

OTOH, Application *itself* needs to apply the extensions onto the controller. Application knows about Controller, Commands doesn't care a bit about Controller. So my mental model is, everything about extensions should be in the Application really, or at least...not in the Commands. (Maybe could be in a separate `ControllerPlugins` mixin for alternative Application classes to use, but that seems theoretical at this point... especially when you could do `Controller.include(MyModule)` if you had to. ) 

Sorry to belabor the point, but it hits on a question I often come up against -  the question of mixins and where to put the state.
To compare, here's with the U8150's pre-compiled binary:

```bash
<7>[215, WifiService] [   87.015586] bcm_wlan_power_on: WL_REG_ON voltage level successed
<7>[215, WifiService] [   87.016731] bcm_wlan_power_on: EXT_POR_L successed to pull up
<4>[215, WifiService] [   87.169879] set_polling: no need to enable polling for slot 2 
<4>[215, WifiService] [   87.170238] ## Calling platform_driver_register
<4>[215, WifiService] [   87.170513] ## wifi_probe
<4>[215, WifiService] [   87.174768] 
<4>[215, WifiService] [   87.174788] Dongle Host Driver, version 4.218.223.1
<6>[12, kmmcd] [   87.381218] mmc1: new SDIO card at address 0001
<6>[12, kmmcd] [   87.389666] board_support_bcm_wifi:<WIFI> wifi_is_bcm = 1, msm_wakeup_wifi_gpio = 17
<4>[12, kmmcd] [   87.389703] dhd_customer_oob_irq_map: customer specific Host GPIO number is (17)
<4>[12, kmmcd] [   87.389923] alloc static buf at ccec0000!
<4>[12, kmmcd] [   87.394141] DHD: dongle ram size is set to 294912(orig 294912)
<4>[12, kmmcd] [   87.691488] dhd_preinit_ioctls: use MAC address in nvram 00:90:4c:14:43:01
<4>[12, kmmcd] [   87.695206] Firmware version = wl0: Jun 25 2010 10:03:30 version 4.218.223.2
<4>[12, kmmcd] [   87.702091] dhd_preinit_ioctls: enable 3 wire BT COEX.
<4>[12, kmmcd] [   87.839406] dhd_preinit_ioctls: Enable BT low request, btc_flags1 = 1
<4>[12, kmmcd] [   87.855818] wlan0: Broadcom Dongle Host Driver mac=00:90:4c:14:43:01
<4>[12, kmmcd] [   87.856253] Enter wl_control_wl_start 
<4>[12, kmmcd] [   87.856286] Exited wl_control_wl_start
```
The problem is in this line (262), I have tried every which way, but the only thing I can get a checkbox to do is reflect the status of 'cache results', I can't get changes to the textbox to save.
Write actual logic. Check other methods e.g. *getAll()* for reference.
goto (or return) considered harmful?
Penses tu que ce morceau pourrait être mis dans un html.twig et qu'on puisse le récupérer avec le code $scriptJS= $this->renderView(" .... ")  ? Cela permettrait de séparer vue et contrôleur.
Typo: this should be `scale` instead.
https://github.com/ProjectSkyfire/SkyFireEMU_430/commit/a830ec823a2f17131d4f803004e60fddc72b43a2?w=1 unhides it
It might be more efficient, but it relies on another assumption and isn't parallel to the ignore sets we use in the later hooks, so I didn't do it.  I did the obvious StringBuilder thing.
I'm sure it is right. Use the ratio rule. You need to convert from the 'real' mode to the pixel mode.
and here (not declared too)

V4L2_CID_CAMERA_AUTO_FOCUS_RESULT_FIRST
What happens here really? Why do we need to pass the exception message and the exception itself? Looking at the _Log_ interface I see:

	@Message(id = 12, value = "Unable to create annotation for configured constraint: %s.")
	ValidationException unableToCreateAnnotationForConfiguredConstraint(String message, @Cause RuntimeException e);

How will the final _ValidationException_ look like and which message will it have (The one from @Message or the one from the parameter passed to it)? I guess I just don't know what JBoss logging does here.
Т.е. чтобы сделать глобальную переменную.

Предложение:
fork fn d = do
    mvar <- newEmptyMVar
    forkIO $ fn d >> putMVar mvar ()
    return mvar

waitForThreads = sequence_ .  takeMVar

runTest pairs = do
    ...
    threads <- mapM (runFn runPut) pairs
    waitForThreads threads
    ...

Но это так, замечание в сторону.
Gostei em! isso ajudo bastante no teste! mas pq o primeiro atributo é "nil"?
I agree but this is probably more a result of a deficiency in my test suite than anything else. It should have caught this change and failed. Notice there are no test suite changes in this commit.
512 KB read_ahead is laggy for me. Even 256 KB seems to high, 128 KB seems to be the best all-around value for all SD cards. I have an Imation class 10 16 GB microSD here.
After back-porting it and testing it again I found some issues with the SF2 test suite that makes ~15 tests fail. They call ->getMockForAbstractClass with ($classname, array());  and it fatals. 

Not sure why that didn't show up on my initial tests. I'll investigate

Given those issues maybe its not good to do release that change.( At least not before 3.7 comes around.)
Torch on star-common
Well, if you think that'll be fine just do it and push - I don't really remember all the reasoning...haha
@rsjtdrjgfuzkfg ...isn't this where you're going to change the profile folder name?
https://github.com/MarcWeber/vim-addon-manager-known-repositories/commit/b90c0765ac0b755c234fd3e1690cbb43b0b32eee
Missing whitespace around concat operator.
Use the proper copyright character, not a c in parentheses.
May need to rerun the appraisal tasks on your machine if you want the paths to still be yours
Well, if you're happy with the information here, then that's fine. I was thinking of some better way to link this to hazard results.

Anyway, we'll probably have to revisit this when we add risk logic trees (so we can differentiate the risk and hazard tree paths).
Perfect. Thanks for the clarification :)
I think it should be part of the interface, because it defines the behavior at the contract level that default username attribute is valued as such. Lower impls and other areas of the code can then do comparisons based on this constant at the contract level as they see fit. 

The code is still backwards compatibles for `null` values, such that null values while accepted are translated back to `(default)`. This I think better indicates what the username attribute retrieval mechanism is both at the config level as well as the UI in svc registry.
this is confusing.  It should just return null
created ticket #1926
How do you provide this from Java?
I think that piece of code is deceptive. Can we make it more readable?
I think this is redundant, just do: message.attachments get name
Given that the person realizes that this solution is just specific out of JIT-feature-add...
No, I mean if we make it an implicit EC then we already cover the base of actors calling Ask, and there's an implicit from AS => MD (EC) so if you have a system lying around you can obtain it from there.
HOWEVER, if we go the SIP-14 route with Havoc's proposal we might not even need the EC at all.
Debug residue, removing
no, what makes you ask that?
If I didn't miss anything, then the `getInstance` pattern should hold for all singletons, including Kill, GetRoutees, and some others I'm currently forgetting. Differences between docs and code will then have to be settled by changing the code ;-)
Remove replyTo here
How is any of this connected to moving packs?
And do these need to be hashes (the regular expression stuff).  Seems this could be much simplified.
√ (new symbol to indicate nothing in conversation requires attention pre-merge)
s/an/the/

Apologies for the grammatical corrections.
Yeah - I think that's already in place: 

https://github.com/nipy/dipy/commit/ceadea99f787b8df8aa43155db1c664380cc5e2d

Was that what you meant?
So the only test of validity really is whether the bvecs represent unit vectors?
Maybe it would be more explicit to test if template_path is a directory (os.path.isdir or something similar)?
Generally I prefer to call `e.preventDefault()` instead of returning `false` from a click handler as it is more explicit.

Returning false is the equivalent of calling:
```js
e.preventDefault();
e.stopPropergation();
```
This stops the click event bubbling up the chain which could be useful to other functions such as watching click events on the page for analytics or something.
This looks insanely inefficient. One request per call. We can happen to write just a few bytes at a time. 

Does dav/http need to know number of total bytes to write up front? If not, the mult_perform syntax like for read aught to be used and send data as we go along, closing connection when done.
Я имел ввиду 13 строку
Its no longer "brute force" because we know when to stop :)

Since we didnt invent any other optimization strategies, we can call this "optimize_reserve_pool" or something
this will never happen.
DatabaseObject -> IRESTfulResponse
Can this "legally" happy, ie. are there race conditions in which self.children might be 1 shorter than the expected number if someone tried to use this to kill all workers?

Does it even make sense to pass in num_workers_to_kill == len(self.children)? I guess the zygote will exit then since it's worker count goes to zero.

Would it be simpler to just set num_workers_to_kill to max(num_workers_to_kill, len(self.children)) and not treat this as an error?
this sounds like an interesting message, may want to be documented here
The braces are needed because it contains two statements
Thank you for moving this.
Why is this cast even needed? Just use change the type of the passed-in view to be `TUIPopoverBackgroundView`.
... to clear all state.
+1 Binary operators are very undidactic.

Also, I think you forgot to decrease the value of remaining.
extra environment what? variables?
also add to nipype/interfaces/setup.py
Technically this just checks that `m.choice` calls *some* `self` random function that affects the output; really we want to test that it *only* calls `self` random functions. But obviously it catches this particular bug, so this is finicky. I think the ideal tests for this kind of thing though look like:
```
for i in xrange(3):
  np.random.seed(i)
  assert_equal(RandomState(1234).choice(10, ...), [0, 3, 5, 2, ..., 7])
```
i.e. scrambling the global random state around while checking that a fixed seed continues to produce the same fixed output, and where this output is specified explicitly so that we can compare this version of numpy's .choice method to all the other versions we've shipped.

This kind of test is relevant for all the random functions though, arguably a separate project.
Just commit a fix and push it. You only need to use the --force option if you rebase it.
сейчас по теме - он сработает. но только на верхний цикл
там (name, newvalue), а тут  (oldvalue, newvalue) - вот в чём разница, если я правильно понимаю вопрос.
ok, agree :P
@jeremy Oops, good eye. I was thinking of it in terms of `as_json` when the comment is about the end result.  Thanks.
Weird. @jeremy thoughts?
I do not have time to make any changes now. Take the patch as is and fix the symbol/string thing as you wish, or wait until I have time to update the code.
Agree, sounds better, we should also change it in the gem to keep both synced
Yeah sounds good. Can remove the `It's possible to write the unicode route` to something like `Just write the unicode route`, or similar :).
Please remove the timestamp columns. They are not populated when creating the join model.
Should not be `Enable`?
I don't really see how we could do that. The two are completely separate things. 
RootImportTree would have to take expr1 and test it for errors as well.
Let's set a good example for other aborters and come up with a message.
It will throw TimeoutException and thus fail the test. I was looking for a right value to put there and 5 sounded good between you wait too long to see the failure and it will "never" be hit by unfair scheduling.

I have quite a few of these and they did not fail by now. Also, the computation in the test probably takes less than a millisecond. If you need to wait 5 sec for 1 millisec, then that machine is in trouble.
I added it and changed all the existing docstrings too
Perhaps more explicit:  ``# Frobenius norm (the default)``.
The above can be simplified to And(Not(y), Not(z), x).
This shouldn't be necessary. `hash` should come down from `Basic`
I tried to do, but it was not so obvious to me what to put into the basis. I have only three slots in the basis
but a few more function-terms which could be put there. Maybe we should discuss this on irc. But I won't be
there tonight.
This will be wrapped oddly. I would recommend either doing

```
raise ValueError(
    'first part of string '
    'next part')
```

or

```
from sympy.utilities.misc import filldedent
...
raise ValueError(filldedent(
    '''your sting goes here and can
    continue however you like
    until your final line.'''))
```

I'm not looking for this elsewhere, but the same comment applies to any text messages that are emitted with raising an error.
No, I need the compare. I just realized that the RGS algorithms make some ordering assumptions.


Failed example:
    a.partition
Expected:
    {{3}, {1, 2}}
Got:
    {{1, 2}, {3}}
> The class you suggest would be to Permutation as MatrixExpr is to Matrix

I'm completely fine with the idea of such a class

> At the simplest, it would have to know nothing about Permutation, except maybe how to reduce to one when given specific numeric input

I anticipate that this task will take me at least 16 hours.  Even very simple pull requests on topics on which I am already familiar tend to take at least six to write up, test, doctest, satisfy all interested parties, merge, etc...

At this stage my priorities do not allow me to devote 16 extra hours to this.  PermutationMatrix has been living in some of my branches for a while.  I was moving it around and decided I would make a branch off of master to ease the eventual colossal merge that's building up.  I'm willing to devote around 2-3 more hours on this PR in case there are serious issues.   If that doesn't work then I'll just wait until I'm no longer under thesis pressure (June/September) to reintroduce this pull request.  Let me know what you would like done.
The problem is not here. Just change the `while newargs` to `for arg in newargs` and remove `arg = newargs.pop()`. If you do that then even if a gamma had a nc in it -- and I don't know if it could -- the simplifications will still happen. The pop() is reversing the arguments:

```
>> l  = range(3)
>> while l:
..   print l.pop()
..
2
1
0

```
Why this form of output? Why not `Matrix([[x + x*t + 0*t**2] , [y + 0*t + 0*t**2]])`
see above regarding `plstring` vs `const char*`
These added variables aren't actually used and need to be removed.
Der eingeloggt User bekommst du durch $this->userService->Get();
this is broken if you pass a callback looking like ``array($object, $method)`` instead of a closure
If we allow the variable to be null, this needs additional checks. Possibly they exist in EqualsUtil or similar.
Hm.. I disagree, but I suppose this is a jQuery convention? Functions are keywords like `if`, `else`, `switch`, `new`, `delete` etc. They are naturally followed by a space for clarity, consistency and for distinction from invocations. I don't see why that would be different between names and anonymous functions (Douglas Crockford's conventions also follow this). But I suppose this is already set in stone. I'll amend it :)
Move this back to the other show then. I don't think we even need this here.
It's the SQL ``count``.


On Nov 16, 2012, at 6:38 PM, Sam Kleinman wrote:

> In source/reference/simple-aggregation.txt:
> 
> > @@ -0,0 +1,41 @@
> > +=======================================
> > +Simple Aggregation Methods and Commands
> > +=======================================
> > +
> > +.. default-domain:: mongodb
> > +
> > +In addition to the :doc:`aggregation
> > +framework</applications/aggregation>` and :term:`map-reduce`, MongoDB
> > +provides the following methods and commands to perform aggregation:
> > +
> > +Count
> > +-----
> > +
> > +MongoDB offers the following command and methods to provide ``count``
> not sure what the literalized count refers to.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
the $within what?
It looks like the only spot where we set this field is in the migration. The `is_template` field [gets auto-updated as part of the `save()` method](https://github.com/mozilla/kuma/blob/master/apps/wiki/models.py#L933), which should be done for this new field too.
sorry for the late response, i wasn't tracing this.

line 24 has been used for internally, and it is safe to ignore this. Whole batch_base/request/response has been built for internal usage by one of developer in our company.
Tu devrais utiliser l'attribut *margin* sur l'élément précédent plutôt qu'un `div`. Pour simplifier encore le code, tu pourrais créer une classe css nommée quelque chose comme *bottom-space*:

    .bottom-space {
      margin-bottom: 10px;
    }

En assignant cette classe au `div` précédent, tu obtiendrais le même résultat avec moins d'éléments HTML.
Well not easily, but i fixed it
Please add parameter and return value documentation.
Should take "options" as the only argument.
I removed them
Sorry we can't do a sleep in test, it makes running tests so slow that it gets annoying and people will stop doing so. We need to come up with some other solution here.
Spaces after // please.
fixed in e9ab04b
ma gandeam ca pare mai clar pentru cineva care se uita pe cod
Use ```Deallocate()``` here.
Yeah... Dunno what that was.
it was the only way i could find to replace only the first occurrence of the path without requiring extra parameters.  this pull request addresses a bug when he the the virtual directory name ('css','js') is the same as the extension of the resource being requested.  the regular expression was making too many replacements.
I misunderstood - you were talking about the checks in the constructor of the derived coordinate. I'll add an error if the units of delta/reference are inconsistent or if they are not compatible with the CF canonical unit.
номер строки, кстати, меня совершенно не интересует
Add test data with signed bytes (hex 80 or greater) and multiple bytes.
It would be better to assign a local variable to m_first -- or to "[]" if m_first is null and not change the value of m_first (and the same for m_second).
Minor point:
Both arguments NULL is not a special case.
Any null should actually cause the return of a NULL Boolean (currently not representable as a boolean NValue).
Short-term, it may be better to always return getFalse() -- this will correctly fail "NULL LIKE NULL" tests but
incorrectly pass "NULL NOT LIKE NULL" tests.
When owner=None, it's never used. Just use proper if statement and do the check only once.
Remove "Fill me in" comments please
I am perfectly happy with that.
I have found myself in the past moving to read and write especially for interface names,
as long as this is consistant and well communicated, this is good.

Although it might trip us up when working between MrSparkles and Lighthouse.
input objects
First I had run my tests again the mojombo/grit branch, not against the gitlab one. I added that line in the previous commit myself. I guess upon merging the tests were not run. I needed to fix this to make the tests pass in the gitlabhq branch. The gitlabhq defines more remotes than the mojombo one. HTH.
The `newtype` target could be evaluated as an expression (like variant or record bodies) instead of being restricted to a pattern.
+1 dmethvin!
Great, let's merge!
regex abuse.

just do: `process.platform.match === "win32"`

also excessive parens
Yeah, I think it's our option to support COLLATE in this function and I don't like it.  As it's been not aware of it, I think it's not urgent.
No, I don't think it's really important at the moment. But if the featureset table is already created by the user, does it have to be empty? If not, there could be collisions.
Should we also set `_shaderHandle = -1;` here?
This won't work; $PWD is wherever `brew test` is invoked from.
Would it make more sense to try and handle this globally, i.e. by adding `-KU` to our shebang lines and the places where we invoke the interpreter directly?
> Don't use sed to modify files - we provide an inreplace method to help
> with this.

Okay, fixing. Will post an update.


>
> Ideally though, since you're the author, maybe the changes could be made
> in the software itself in some way so we don't have to patch it at
> buildtime?

I would do this, if there was a build time, but it's actually a shell
script, and I really hate runtime detection, preferring instead to
patch like this on second class platforms like Mac.
It seems wrong to have a file committed that has a hard coded path on the original committer's machine...
This chunk of command code needs to be dried up.  None of these commands are any different except the text in the message.  Suggest:

    def stop(cartridge)
      control_operation(:stop, cartridge)
      results { say "...." }
      0
    end

    protected
       def control_operation(method, cart)
            .....
            rest_cartridge.send(method)
        end
rolling_kurt only takes DataFrame but nanops wants ndarrays no?

On Sat, Mar 24, 2012 at 11:27 AM, Wes McKinney <
reply@reply.github.com
> wrote:

> > +        if level is not None:
> > +            return self._agg_by_level('kurt', level=level,
> skipna=skipna)
> > +
> > +        if not skipna:
> > +            if isnull(self).any():
> > +                return np.nan
> > +            ser = self
> > +        else:
> > +            ser = self.dropna()
> > +
> > +        if ser.count() == 0:
> > +            return np.nan
> > +
> > +        from pandas.stats.moments import rolling_kurt
> > +        kurt = rolling_kurt(ser, ser.count(), ser.count())
> > +        return kurt.values[-1]
>
> shouldn't all this go in nanops? There's duplicated logic here
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/pydata/pandas/pull/932/files#r600205
>
I would recommend writing this a bit differently:

```ruby
coerced.each_with_object({}) do |(key, value), hash|
  hash[coerce_key(key)] = coerce_value(value)
end
```
麻烦使用中文括号（）。命令用反引号引起来。

因为我下文中用的是`which R`，这里有点不一致，当然`whereis R`也不是不行，稍显复杂了一点，只是为了找R的可执行程序位置。
You can make further commits; they're taken automatically in the pull request.
I'd like to try to convey that it will set the value of a directive with name `key` to the value of a keypath found at `data-#{key}` on the node. I like `directiveAccessor` (more than `viewDirective`) because it indicates where the keypath is found (in a directive) and where you can find the value (using `get` cause it's an accessor). It still isn't very clear. @jamesmacaulay or @rlanglois you got a good idea for a name for this?
Not my change, go with master?
Why are you changing .gitignore in a PR for whitelisting?
Heh, missed it because I was testing the preference off. Fixed.
Are we sure that our version of Chromium still doesn't support `onload` events for `<link>`? I've been Googling around and haven't found a definitive answer, but it might be worth testing.
No for PHP since it shows up as HTML.
Nit: needs a comma after the word "selection" to make this easier to read
Hmm...not that I can think of. Fixed.
Should we also rename `this.$lastColor` and  the `.last_color` CSS class?
Should this be "right-resizer" ?
Here I'd use an icon, these are special symbols.
a tab snuck in here ;) you might want to run the astyle thing to find any others
is anything here actionable?
The end result is a string in format "yyyy-mm-dd hh24:mm:ss".  Added a comment in the code to make it clearer - the main thing we need to validate here is that no two waves are scheduled with the same timestamp.
again, this should eventually be moved out of HQ core
should this be mark_safe?
Something is wrong here.. I would just call $path, $parentPath instead, so it's a bit clearer.

Also in the case of 'beforeCreateFile' a parent path is passed, and in the case of 'beforeWriteContent' the object path is passed.

I would suggest writing a unittest first to confirm this is broken, and only then fix the code :)
please stick to 1.0.x version
Autoformat ?
The #region-{id}'s could be styled in the javascript rather than css, to allow for dynamically-generated id's.
Exactly this is the difference. Once it is an empty placeholder where JavaScript code replaces the content with new one. In your case there is something inside which get replaced with new content and we lose JavaScript. I don't know what browser does in such case, but I do not want to disk if it is not necessary.
Why both warning and italics? I would remove `*` around, no?

Also, you probably mean *site root folder* (applications do not have `./manage.py`).
I am not sure extending mongoengine User is this trivial. I've spent sever days on this matter and tried numerous combinations and at the end chose this one. 

The whole User from models.py is just a copy/paste from mongoengine auth.py (with some custom fields added) which seems to me plausible short to mid-term solution. In the long run I expect django to implement non-relational sources as well. 

I don't know what to do here... Do you want me to implement your notations in (slightly changed mongoengine) User because as I said this is just a ripoff of mongoengine auth.py.
I don't know whether completely exiting is a good idea. I think it should mention the error, but keep running.
`for ( i = 0; i < anchorTags.length; i++ ) {` -> `for ( i = 0, l = anchorTags.length; i < l; i++ ) {`

You will have to create the `l` variable where you create `i` as well.
Can you change this to a console.error and an early return?
This is interesting, but might not be necessary since `savedData` is already a JSON blob with some project data in it. As a result, `load` would be redundant unless the data on the server changed.

It's assumed that the data loaded here is actually *from* cornfield already.
Uhhh, this should just be "../index.html"
on desktop or just regular mobile browser, there is no need to close the window in any case. if it's an application -- maybe.

please use B2G-preprocessor directives to open dialog box and close window only for B2G platform
data.result and data.findPrevious must be validated before passing it to the updateControlState function

also we can add a check similar to the supportsIntegratedFind
The Salt Minion now maintains a *proc* directory in the Salt cachedir, the *proc*

Maybe? Since this refers to a single cachedir, it should refer to the minion in singular form.
Why not just override _new_conn directly?
I would personally prefer to see this as a `@classmethod` on `Url`: this will make it super obvious how a `Url` is constructed by reading the source of `Url`… Then use something like `parse_url = Url.parse_url` for backwards compatibility.
What does ssnm stand for? Worth expanding into the full name or at least describe in comment?
```js
if (configs.paths[path] !== undefined) return;

if (FS.existsSync(PATH.join(path, '.borschik'))) {
   // ...
}
```

Минус уровень.
Ugh....no, this should not go to code. Please keep it clean. Initialize empty is fine, bug keeping such hacky commented code is not good ( especially when it looks like regex in ruby :)
There is no such tag in a repo. You should:

1. Reference a commit here instead
`s.source   = { :git => 'https://github.com/farcaller/iphone-pinentry.git', 
                         :commit => '1d0f17e4d746881d3136c2933c43d694a9b3396f '}`
2. Ask the author to tag a versions. [Template](https://github.com/CocoaPods/Specs/wiki/%22Please-add-semantic-version-tags%22-issue-template).
As discussed in IM, there are a few ways to do this. Not sure which is the best practice. It also seems to me that not all of this stuff should live at the bottom of layers/models.py
Its another way how to write singletons. I prefer it because its more flexible:

* instances for testing can be easily created
* when needed singleton pattern can be easily abandoned
* other approaches don't feel clean (module and its class methods or Singleton from stdlib or stick it as class methods to RpmGemTest)

I know these reasons are not much strong here in this patch, but I am used to write all singletons this way, because you often discover later that you need more instances of that singleton.
Which line was a copy/paste fail?  The comment?
+    raise HttpErrors::BadRequest, _("New templates can be imported only into the '%s' environment") % "Library" if not @environment.library?

Added a "the" in the sentence. :)
I'll work on adding doc strings for tests in future development but for now just let this slide.