labofsahil/github-event-dataset-2012 huggingface dataset:
topの追加っすね。
This function should be available to all plugins, certainly hg needs it is well, see #66.
Apparently np++ screws it up... hmm
http://www.abload.de/image.php?img=99q5khty7okkr.png

And a Question...
-> Why you change "object" to "regObject"???
Is this ok?
this will pass if we are on the home page and we expect some other page 
ex: 
we are on https://addons-dev.allizom.org/en-US/
we expect to be on https://addons-dev.allizom.org/en-US/firefox/themes/
stimmt, schokolade und blog würde ich auch tauschen!
Per our in-office discussion, libPkt provides a split_srcname function in the C libraries, and it looks like it's been passed along to the Python interfaces as well. It's best to use the built-in functions for consistency.
We'll be using the Thrift interface from a Scala service which uses Netty and Akka (thus we don't want to be blocking on any of those threads and we want our architecture to be as asynchronous as possible - it's particularly write heavy).

For now (ie. next few weeks) we'll just be using it as a stand-alone search engine. One of the main motivations was to keep our deployment simple for now (I don't want to have to add Django and the other dependencies to our production environment when we won't really need them for now).

In the future, I'll probably be implementing more of a service layer so we can internally manage instances/scaling. Depending on the direction we take - I'll probably also be looking at making the indexer more distributed if it isn't already (think Akka style - where the indexer can be split into many JVM instances talking to each other). I haven't really looked into this at the current stage.
also looking at this I did the pull request for the wrong commit , this one has a typo in the path, https://github.com/bholtsclaw/pressflow-6-plus/commit/9ec5e33105d01c092764c132e4c63b6f7816aaec is what your gonna want to pull in.
Add line 19: Fix for drop-down menu work with jquery validate
Have you checked the new fw /  nvram?
Added to list for v3.x source migrations - see #42
subsequent commit https://github.com/dharmsheta/subversion-plugin/commit/bef806db927be4625226cb904b3b8b8073c561f0 is mending this auto import improvements:)
J'ai simplement déplacé le calcul des dates.
C'est un chgt impactant pour Solucom.

Bonne journée,
Corentin POUHET-BRUNERIE
On Feb 17, 2012 7:49 AM, "Kevin Deldycke" <
reply@reply.github.com>
wrote:

> Is it me or there is no changes there ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Smile-SA/smile_matrix_widget/commit/8207ff6dec16d63bffb36f0f3e1099da16a4bfbb#commitcomment-976911
>
`writeToFile:atomically:` uses the XML property list format which is much slower to read/write than the binary format. See `NSPropertyListSerialization`
So great news !
"Add a way to use the Symfony2 DIC.."

(better english IMO)
В любом случае, скорость - приоритет. Ежели оно быстро, и в то же время не портит иные функции скрипта - то енто то, что дохтур прописал.
Спасибо за труды, батенька, родина вас не забудът!
This repo shows up as the only legit-looking hit for a Google search on "iterm2 wombat color".  I love the wombat theme... if you've actually ported it to iTerm2 I'm sure others would like to use it as well.
What is the difference between you device/motorola/triumph files compared to Ikarosdev's own?
Oops: commits abbrevs don't work across repos, of course. That commit is

https://github.com/JuliaLang/julia/commit/df0f84b02bc1a1bd85053e383be0659fcf04b5bd
Jo schau ich mir mal an, hast du noch sowas wie ICQ oder jabber von smag 
laufen ?
mein jabber auf smag läuft jedenfalls noch
mach sonst mal an ...
On 01/25/2012 09:24 PM, Robert Schönthal wrote:
> du musst nach dem merge noch das travis icon in der README anpassen
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/digitalkaoz/BlageConnectBundle/commit/181562a56b116295ad56d376a08d596396db4dca#commitcomment-901774
>


-- 
http://blog.monofone.de
I agree with you on this point but _log.throwSomething_ means to me that the method will throw something.
Cool! I was planning to do this once, it makes configuration much friendlier.

It would be nice if the INI file can also be specified as an argument to server.py...
ok

Em 3 de maro de 2012 00:11, Douglas Camata <
reply@reply.github.com
> escreveu:

> No sei porque colocamos o formtastic. Pra mim, deveria ficar s o
> simple_form mesmo.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/douglascamata/Rick-Rocks-Academics/commit/b396bd442f09c38af613c01ccc7f882c823f3e43#commitcomment-1038144
>



-- 
Pedro Henrique Passalini Soares
Graduando em Cincia da Computao e Informatica - UENF
nope. some files are going to be deleted. today i have reworked the inputreader.cpp....i've compiled with no problem but i don't have time to test it. tomorrow i will try if my changes are goods or not :)
I see... thanks !
Out of curiosity what dies the output of: puts Rails.configuration.database_configuration.to_yaml look like on your rails console? 
Is it identical to: puts ActiveRecord::Base.configurations.to_yaml ?
If you are not using the default configurations how did you configure your rails?
Other than those the only way would be to parse database.yml directly AFAIK.
we need a redundant showBootMessage(CharSequence, boolean) for times when we don't know the package name in the calling class
Happy New year tooo...
I spelled "repetition" incorrectly :(
Check https://github.com/fog/fog/blob/master/lib/fog/aws/models/compute/servers.rb#L58 or https://github.com/fog/fog/blob/master/lib/fog/aws/models/compute/images.rb#L47. Used the same approach.
It's more code than Y.Parallel and does more.  Better that IO, et al, are refactored to output transactions based on promises. In which case, Y.when would be preferable for that/those specific use case(s).  If the seed has a specific use case, then it's fine for Y.Parallel to also exist. Things like grouping event subscriptions could go either way (not "transaction" related, per se, but could be considered to be, based on the events as independent/parallel steps in an operation).
This seems reasonable, although I don't think it has any direct bearing on what we were discussing re. shifter (except that this will make precompilation slightly more efficient). The question of whether precompilation should be possible without requiring YUI at all is still a separate issue, and I still think the answer to that is no.
the formatting seems to have been altered or does it just look wrong here?
Can't we just have one site.xml (a dream maybe) ?
have you checked that the text uses the right quotes? ;-)
What do you mean with 'this' ? The callback? the exchange? at timeout? 
Looking at the camelk code, if the exchange is transacted, the callback is always called. Otherwise the processor implementation is responsible for calling the callback. I can't immediately find an issue.
I'd probably collapse these two lines
same answer as for the scala Echo
This needs serious renaming
why not private[akka] for the whole ActorSystemImpl?
How do you know that no one is using it?
Is this the correct behavior? Or this was forgotten to be implemented? Btw, it was IntelliJ who discovered :)
All of the above is for userstamp and should be in develop already.
this should be "===" (triple)
Can you please elaborate? I still do not see the need for this, LPCM is not an encoded/passthrough format.
Thought about it some more; I like it.
perhaps the word "and" shouldn't be here?
currently everything coming out of the cache is frozen, I believe.  Do you know what makes something readonly?
I believe, as of 0a4ba5144300ccd58ad5673ce936d2040dd82077, everything passed to window._console is also logged to the browser's console.
I found the comment useful.
I thought I already had this, obviously it can/should be promoted upstream
delete. also collapse multiple blank lines below.
Although this property is only ever `readonly`, it's good practice to still note its memory management (`strong` in this case). It seems like it should also be `nonatomic`.
This conditional doesn't match the description in the comment above.
Oops! I meant to fix that!
It seems the diff code does not work properly on indentation when there are preprocessor statements involved.
Why set the include path?
3.10
и : тоже на другую строку
Дописал комментарий, что это в работе с моделями всплывает.
Ok. I think we can remove the `symbolize_keys` call. But we should change to a `dup` call since this method is modifying the options hash. We also need to write a test to this behavior. Something like this:

```ruby
def test_truncate_doesnt_modify_the_options_hash
options = {  length: 10 }
truncate("some text", options)
assert_equal options, { length: 10 }
end
```
@bogdan that's slightly different - it forces a cache miss so it will still write the newly rendered fragment. This will not cache the fragment at all.
Actually, I might be over thinking this. Are you asking for:

```ruby
Emailer.queue == QueueA
WelcomeMailer.queue == QueueA
```

or

```ruby
Emailer.queue == QueueA
WelcomeMailer.queue == QueueB
```
Are you sure that we can remove this one?
Why not `case _: IndexedSeq[A] =>` ?
Of course I have to retract some of that. The funny thing is that this represents me unfixing a bug; the use of isPackageObjectClass fell out of my restoring the old implementation to see if it fixed the cycle (it did) but then I also restored some of its bugs.
I don't like this style of extractor; I think it should return Some(tree) when it's a different tree, and None otherwise. Otherwise you learn nothing from the fact that it matched. I have a work in progress with a bunch of extractors along these lines: https://github.com/paulp/scala/blob/value-class-extractors/src/compiler/scala/tools/nsc/Global.scala#L1118
I'd rewrite this if I could figure out what it does without too much brain work; instead I'll just ask you to comment it more.
particle -> RigidBody
Yes, I could fix Pow so that Pow (10, -50, evaluate=False) to print :
              50
        10
instead of 
100000000000000000000000000000000000000000000000000,
but on going through the code in core/power.py, I found that it had been done on purpose, since it was explicitly mentioned in one of the comments. (though I couldn't figure out why).

As @smichr said, since this is for display purposes only, it shouldn't pose any problems.
already fixing it
Yes, that's right. This check does not provide the real formalism used to form the matrix. 
Therefore the correct verification will be implemented here. Thanks.
So I may have been wrong about this. What I was thinking was, say you have an outer product ``JzKet(1, 1) * JzBra(1, 1)``. If you take the trace in the Jz basis, ``Sum( JzBra(1, m) * JzKet(1, 1) * JzBra(1, 1) * JzKet(1, m), (m, -1, 1))`` ,it's clearly just 1. What I was thinking is that if you evaluated the trace in a different basis, e.g. the Jx basis ``Sum( JxBra(1, m) * JzKet(1, 1) * JzBra(1, 1) * JxKet(1, m), (m, -1, 1))``, it would be different, but since you use the rotation operator to evaluate those inner products and the rotation operator is unitary, it doesn't matter if you do it in a different basis like this. At least in spin cases, I couldn't think of anything that wouldn't be handled by just evaluating this inner product.
but you can turn all kinds of things into lists, like:

```python
>>> list('test')
['t', 'e', 's', 't']
```

There are all kinds of sequences (lists, tuples, sets, etc) and all will turn into lists. So checking whether something is a list doesn't help that much. Your function shouldn't care whether something is a list. It just needs a group of 1 to N particles and bodies. You can group them in multiple ways. It should check for particles and bodies but not worry too much about how you pass them in.
Should be sorted according to number. Message is better be something like the following:

~~~
Enh #967: Commands from YII_CONSOLE_COMMANDS environment variable are now always added to yiic console application (schmunk)
~~~
This was already done in `resolveNameID()`.
I made it more general.
We should standardize the way we attach and detach events. Perhaps this can be abstracted into Abstract. So, Tip.on( 'blur' ) would be equivalent to $element.on( 'blur.lu.Tip' ). Because of how jquery does its name-spacing $element.off('blur') would still remove the event. Thoughts?
Just wondering if the requirement of a nodal based interpolation field for pressure is just because when the pressure is applied over a face, it is decomposed down to the nodal equivalent forces? Also is this pressure bc only for elasticity examples or for fluid/laplace etc.
ne doesn't seem like the correct index here. Don't we want a nodeIdx?
oh juste a convenience :) I'll move the whole thing on the store.
yes i guess that's obvious… ^^
requiring additional (a majority?)  members off the set confirm the write operation.
That are in the pools, available for use.  Does not include connections that are currently being used as those are not in the pool.
let's say "he format for for describing the parameters of connections between applications and MongoDB instances in the offical MongoDB drivers." rather than "the format and options for creating connections."
The :dbcommand:`geoSearch` is the only way to return results using the haystack index: find and geoNear cannot access the haystack index. You must specify both the coordinate and other field to geoSearch, which takes the following prototypical form:
allowing `style` scares me - too open for security exploits? e.g., http://html5sec.org/#9

how vital is it?
Aha got it. thnx!
And why is format modified by this function?
What is $ for in that regexp?
$ug_id_list
, then?
Am menționat mai sus, preferabil să dispară parametrul ăsta. :)
What do you think of reversing it? Rather than `case ''` falling through to `case 'rad'` we let `'rad'` fail through to default.
Good catch.  However, `store.getResources()` returns a list of resources, so you might need to `[0]` on it.  Also, I think the `'asset-xml-crossdomain'` argument might instead need to be `{id: 'asset-xml-crossdomain'}`.  (Said another way: I think I rather bungled the previous version of the code :disappointed:  .)
This test isn't right.  Since you registered s1 with POLLOUT, poller.poll() will always return immediately as s1 can always send.  This should only test should only register s2.
Нет, code style - 4 пробела. У тебя стоит табы
needs remove_reference on Type and Cardinal
We're just using the VS defaults. Try performing an [Edit] -> [Advanced] -> [Format Document] on it
This entire method is _huge_ and a bit daunting if you're not up to date with the code. Perhaps it could be refactored into helper methods with descriptive names so even OWIN-noobs like me can follow along :P
Can we just delete all instances of 'caste' from our system?
Is this about "Users" or adding a drug? The naming is confusing (for model and view, also). Please rename for clarity, unless I'm misunderstanding your intent.

store: Ext.create('RaxaEmr.Pharmacy.store.Users', {
	storeId: 'drugStore' <<-- why would a store of Users be called the Drug store?
 }),
You are catching all errors inside handle_facts_packages too here.
components don't have priorities
`getDependecies` -> `getDependencies `.
I think this is mostly spacing issues...
I managed to install in postgres then create new posts using this change:

```diff

diff --git a/Plugin/Install/Controller/InstallController.php b/Plugin/Install/Controller/InstallController.php
index 4e219df..51ca416 100644
--- a/Plugin/Install/Controller/InstallController.php
+++ b/Plugin/Install/Controller/InstallController.php
@@ -219,6 +219,9 @@ class InstallController extends Controller {
 						$modelObject->save();
 					}
 				}
+				if ($modelObject->name !== 'I18n') {
+					$this->_fixSequence($modelObject);
+				}
 			}
 
 			$this->redirect(array('action' => 'adminuser'));
```

I would need to revisit the I18nModel in CroogoTranslateBehavior and see wether it's still required?
not exactly related to this pull request, but setModelType should probably be protected.
I would much rather have no comments than unhelpful/bad comments.  Since we require comments, we just as well require good comments.
There are a couple of rst bugs in this file and also a bit of non-pep8 code.
All `k`s are known in advance, and the `v` are all url-encoded in advance
You're right. I'm adding `respond_to :xml` back.
Still don't think it should be part of the public API. It's a functional programming library and this is a utility in an implementation detail.
Challenge accepted!

On Thu, Apr 5, 2012 at 10:04 PM, Mike Sherov <
reply@reply.github.com
> wrote:

> > +    },
> > +    jshint: {
> > +      options: {
> > +        evil: true,
> > +        browser: true,
> > +        wsh: true,
> > +        eqnull: true,
> > +        expr: true,
> > +        curly: true,
> > +        trailing: true,
> > +        undef: true,
> > +        smarttabs: true,
> > +        predef: [
> > +          "define",
> > +          "DOMParser",
> > +          "__dirname"
>
> enjoy the vacation!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jquery/jquery/pull/729/files#r653698
>
The above is inside of the `testRemoveClass` function that gets used a couple of times throughout this module. Inside of that function `QUnit.reset();` is being used a couple of times prior to my updates. Should we refactor that entire function to remove all instances of `QUnit.reset()`? Or am I just using it incorrectly?
Oh crap, I just noticed that was you Corey! Thanks for chiming in.
Nothing changed, this is the Linux part from above (the red lines that were removed are here).
Let me commit a fix.

Basically...  if I get an unknown platform touch id... i'll ignore the touch.  2 minutes.
The diff from osg is slightly different and didn't work for me so we have to go with the gist. I added a comment.
Well, those sequences aren't utf-8 (the last one even has an embedded null!). I suppose we have to compile them with `/n` to disable multibyte interpretation under 1.9.
Remove the `ohai`s; if this information is absolutely essential, it should be in the caveats.
See comment above but I don't think this makes success any clearer - the additional indirection doesn't improve readability over a constant, and using no value is superior to any value.
Bad things. I'll log a warning and discard the message in that case.
maybe just rewrite this as

```
module.exports = require('rc') ('bower', {
  //Bower Defaults
  directory: 'components', 
  json:      'component.json',
  endpoint:  'https://bower.herokuapp.com'
});
```
"By eye", this algorithm now looks correct - test cases?
Fixed with another commit.
Should we check to make sure that mechanism.prototype is present before trying to instantiate a random object?
Yeah but object arguments fuck with the jit and inlining, and why add another hop to the prototype chain when this is the only place its referenced? Empty classes strike me as code smells
Similar to the CSS issue: if file contains two functions with the same name, you can only get to the first one.  I think a TODO or bug is fine for now though.
Should just be this.editor.setSelection()
Nit (only bother fixing if you're going to make a new commit anyway): missing a space in "lookupfails"
Change "require" to "RequireJS"
Note small cleanup here: using one() in place of on() with immediate off().
Nit: if we're ok shortening to "there's" in the long form message below, we might as well do it here too
I mean: NO, you are right, we don't need to escape this.
i think this will propagate changes in the original to the copies, which is fine but should be known behavior
NULL seems to be more SQL-ish. E.g., in §10.9, "General rules", viii), the [draft for SQL:2011](http://www.wiscorp.com/SQLStandards.html) specifies:
> If VAR_POP or VAR_SAMP is specified, then let S1 be the sum of values in the column of TXA, and S2 be the sum of the squares of the values in the column of TXA. [...]
> 2)	If VAR_SAMP is specified, then
>   Case: A)	If N is 1 (one), then the result is the null value.
>   [...]
Please add some constant declarations so it's easier to decipher the params in the GET invoc
I have already merged with current revision.
I am not saying that the user should see any difference? I am just saying that the logout URL to which user goes once it click the "Logout" link should be different based on the way the user has logged into our site, no? Why we then have the facebook logout view in the first place? It seem to me there is no need for that anyway? It does not do anything more than a normal logout view, no? So maybe we could simply remove this view and always use just the normal/standard logout view?
Yes, but it would be a lot of work for me to remember/bookmark everything. It would be best if students would present to other students mistakes they made and what they learned from them. To offload me a bit.
same regex as above plz
I think I actually added in functionality to do this, so should be an easy fix
Ah I have a fix for this for round brackets, just have to apply the same logic here
Why are you removing all the units?
The javadoc should be inherited from `org.sonatype.nexus.configuration.application.ApplicationConfiguration` that does documents the dir creation.

But agreed, it should be handled somehow with the log, even if we did not handle it so far.
since this is an abnormal way to shutdown, shouldn't this be warn?
Please remove the underscores and use camelize :)
这个为什么要调大呢？
`lat`/`lon` +1 me too ponies
Nope, and in fact in many cases it definitely does not (for FK4 by default, for example).  I fixed it to remove that part.
What are `dbs`? Also, this is the first time you use `RequiredDs` and `DAM`. It's not immediately obvious what these mean. After looking over later code, I'm guessing "required distances" and "dictionary adjacency matrix" but a comment would be helpful here.
Good catch :) Thanks
That's one thing I didn't want, as I find it very hard to read if you change that much.
You're right, my editors automatic of docblock. I'll fix that.
This is a picky thing to comment about, but for future reference, technically, the first curly brace should be one line above this one, and if it's empty, they should both be there.. http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Function_Declarations_and_Definitions#Function_Declarations_and_Definitions
wouldn't it be better to prevent the default when the return value is ``false`` instead of doing it when ``true`` is returned ? The interface of your event system looks like the DOM event system so it would be logical to use the same way to prevent the default too.
Great to see thorough testing for all the new controller methods. :)

I'm an advocate of using mocks more heavily than this in controller specs, rather than using the real objects, but again, this is pretty subjectives. Plenty of excellent rails developers use real objects here.
I would like to see a comment explaining lines 14-20 in a general manner. What they do? What are the characters like \<\<?
No need to have this forward declaration when the real declaration is right above in the same file.
Yeah, let's leave it... Might as well start getting into a py3 mindset, sooner or later we'll be moving over.
That's just my implementation changing faster than my docstrings.  I changed my mind about how this should be done, and I will update the docstring to match.
super.setupContext and use the existing variables as opposed to making cloudstackEc2Client. ex. you don't need any feature from cloudstack subclass to execute this test.
Would that protect the create/delete functions from being passed `GIT_BRANCH_LOCAL | GIT_BRANCH_REMOTE`?
Both fixed in 5a4eb9e

Thanks!
ok, killing the old one wfm.
**WARNING** Doing this forces you to do the two-step push to production. If you want to skip that, don't make this change.
It looks like you are always stripping the locale. If so, I think this will break for non en-US redirects. Like if there is `/es/redirecto` redirecting to `/es/redirecto-` and you strip the locale, I may end up in `/en-US/redirecto-1` which would be a 404.

I didn't try this but at least we need some tests for redirects that aren't en-US.
As per the note above, this could just be ``var trigger_id = $(this).find(...``.

Also, missing semicolons on this line, as well as lines 138 and 141.
I don't like these magic Postrgres-specific tricks when you can use ansii sql with `SELECT count(*) from {{tablename}}`
Shouldn't that be 
```
from socorro.app.generic_app import main
```

Otherwise you might get some odd behaviour if fetch_transform_save_app.py starts to mess with that `start()` function.
why not urlparse.urljoin ?
Should `this` be `i`?
More `dict()` sadness
If it  needs to be implemented, we can move this to a separate interface.
це функціонал попереднього аплоаду, він не використовується з VichUpload
хіба що залишити його замість неймера. Але тоді це вже не торт. Можливо я краще на основі цього коду зроблю неймер, який буде повертати назву картинки під назву спонсора?
Typo here, it's `Ldap\Ldap`
btw, the element on which you could add the class to need to add it only once is ``#main``
totally agree with @stof + this actual solution wouldn't work with custom panel
I'm fine with `reset()`.
I would like the process builder to be injected by DIC and use it for the creation of multiple processes. Doing so, mocking a process builder would be easier.
No, as if the template is not found (see line 85), it will fallback to ExceptionHandler.
it is not only in the template. It is also used by the JS code implemented by users with the existing attribute name.
It's the only way to know it.
BTW, it seems this option is only used by OCI8 users ; ubuntu does not provide PHP with this option which seems quite exotic to distro packages... maybe RedHat ?
Please keep the signature on one line
I think this should `xsd:string` (to allow `%lifetime%`)
Good point regarding single assertions.

I guess I would like a new test - something like the above test in the context of multiple emails to make sure the parsing works properly.

For example, I would like to make sure the three created invitations are unique and appropriate (event.owner, guest1, guest1), not duplicates of each other.
nit: we need a charset declaration for `ł` to work ;)
You shouldn't use our internal ids to refer to the source type. Instead, specify "source_type": "HAProxy" and let the backend resolve the id
Did you mean to leave this second `downcase` call in place?
See comment about this in node--ding-event.tpl.php in this PR.
@Sunight  `directory `稍后可以放去 AppContext 之类的地方，估计还会再用到，不急改可以以后再说。
Using 'socks5' or something similar would be much more useful than just 'q'.
KEYSTORE_PROVIDER_ARGUMENT is supposed to be used in combination with KEYSTORE_PROVIDER. It contains a String that will be passed in to the provider constructor when instantiating the provider class.
I had tried to get the global $swarmContext before, but when I call it from getSwarmUAIndex it is not set. 

I could easily be doing something wrong though, heres a basic gist

https://gist.github.com/3928735
В PostgreSQL создание поля как "PRIMARY KEY" и так создает unique-индекс. А с этим выходит что у таблицы два идентичных индекса:

```
Indexes:
    "pk_user_remarks" PRIMARY KEY, btree (id)
    "user_remarks_uniq_idx" UNIQUE, btree (id)
...
```
Remove prefix plx!
Why do you need to reset this._timerSMenuScreenOff first?
What is the default action that you need to prevent here?  I don't understand what this is for.

Also, please remove the console.log() before landing.
if {
 ...
} else {
 ...
}
This method will reload the contacts list in background, in case they have changed. I think we do need keep calling it in order to paint contacts created from another app (from the Dialer iframe for example).
That doesn't work in Keyboard app don't you think?
@vingtetun Agree. Any idea? Could we add a special permission as 'homescreen' in order to identify univocally which app is the homescreen?
This could be https://bugzilla.mozilla.org/show_bug.cgi?id=769254, which is blocked on a Windows-only heisenleak that I didn't cause but my patch tickles.  I'm desperately trying to figure it out.
True that, didn't think of third party scripts, could just drop the curly braces ofcourse
Is ``if !!task_block` needed over just `if task_block`?
~~~
永続ジョブは周期的な基準にそって実行されるよう設定できます。
~~~

→

~~~
saved ジョブは周期的な基準にそって実行されるよう設定できます。
~~~
~~~
+また、テンポラリジョブや saved jobs の作成方法、その履歴検索についても理解していなければなりません。
~~~

→

~~~
また、テンポラリジョブや saved ジョブ の作成方法、その履歴検索についても理解していなければなりません。
~~~
~~~
シンプルに valuesUrl 属性を <option> に加えて下さい。
~~~

~~~
シンプルに `valuesUrl` 属性を `<option>` に加えて下さい。
~~~
I'd better use IoC container here.
I removed that FIXME line in a later commit already. But, I'd like to use full_hydrate() and I am working on that already.
Hmm, which version of mpl are you on. The website [suggests that it was added in 1.0](http://matplotlib.sourceforge.net/api/api_changes.html) (it's a bit cryptic, though, so I could be reading that wrong).
thanks; didn't know about require
This calls scope.get twice.
Also, we need to call it in GroupBuilder, which doesn't have the RichPipe implicits.  We could put them in there, but despite my admiration for the power of implicits, I also like to constrain them to where they are essential for usability.
I believe this is the clear proper behavior.
GET is not an acceptable method for this preset, so it will return 405 and not delete the preset.
POST will match and return the preset
Any further cal should return 404. I could add that in the testcase
I did benchmark.  It seemed to make no difference or even speed it up slightly.  I think it is because `this` is a fast lookup even when `with` is used, and I make use of `this` in the function.
you're totally right, I fixed it.
Removed it since I'm note sure it's useful.
```php
switch_whitespace_()
```
why is this commented out here?
Would it make sense to use `repourl` for this? I'm not familiar with hg, but for git, it would be easy to reuse a single workdir for all pollers.
This method is already defined by the attr_accessor method right?
How about add 

    assert(factor != 0)

before doing dividing?
You mean the `__construct` part in the docblock?

Happy to oblidge. Do you guys have `phpcs` rules to make it easier since I am not familiar with the coding standard?
``$this->process = $process ?: new ProcessExecutor()``
I guess you're right: https://github.com/jquery/jquery/blob/master/src/event.js#L781

I'll give it a try soon.
`%r{(:(\S+):)}.freeze`
If you are making it a sidebar, also break that down into two span6's, or span8+span4 (which the Project page uses). And add the outer row.
How about doing it at YAML.load, time? If YAML thinks there are multiple documents, present an error?
oops I meant the test on line 586
@lloyd - @ozten pointed out in another PR that he was not comfortable with not using translations for en-US - does that hold here?
not all of this code exported is run in production.  there are some comments in the code that are disconcerting when you don't know that (it took me a while to figure out that `verifyChain` is only used by our own unit tests.  The comment that concerned me was `for now we only do our own browserid.org keys`.

not a regression, shouldn't block the party.
this spacing looks funny.  (nit)
@lloyd, when does this get invoked?  When can sampling be enabled but there not be currentData?  It looks like currentData is created as soon as this module is run, and cleared only when session context comes in.
Typo: appearance
We have a getter called 'count'. We should always make sure to use it even for the class' own methods.
The name of the preference.
More node.js style goodness can be found here: http://nodeguide.com/style.html
My editor is a troll, sry. :trollface:
These two lines (and another two further down) are pep8 violations. The maximum line length is 79, not 80.
Agreed, but timestamp based update/delete is a part of requirements.

For my opinion, will be better allow only change/add records via import and add separate option to override all phonebook content with CSV.

Also we can add second step for import. On first step user select and upload CSV file, system will perform update/creation and collect list of records that not present in file.
On second step user can review list of all records for deletion and confirm or reject all/individual records. 
But, i think, all users will be disappointed to review a batch of numbers each time when he want import few new  records.

So, can we use implement add/override mode, or should i implement two steps import?
Fix indenting here (2 spaces, not 4). Add comments for why we nuke the margins.
Isn't SLE 9 out of support anyway?
Ah, mfilka is right. I overlook it. One more reason to change it. So my suggestion after this comment and your explanation comment should lead to one extracted method
```
/* comment explaining old persistent devices */
boolean isOldPersistentDevice( string interface)
{
  return issubstring(interface, "-bus-") || issubstring(interface, "-bus-");
}

global map< string, string> getUdevFromIfaceName( string interface)
{
   if(  !isOldPersistentIntercface ( interface ))
   ...
}
```

So now it is clear what happen there. That you translate only devices that is in old persistent format.
Should be review with the Stdlib\ErrorHandler API IMHO.
Same comment as previous two.
i tried adding them in an #instance_exec block, but it didn't work out ... it was late and i had bigger fish to fry ... so i began moving to other things ...

i dunno if i can get rid of the ghost methods... either i'll find some time on this iteration, or one of us will on the next iteration...
lessondate - in GMT, in_time_frame - you use IST. BTW, what about summer time, maybe we need some kind of setup?
yaay :+1:
$idlePeriod = getParameter(...min_score_diff). Something wrong here.
I had this in mind too. Problem is with existant form type classes. That would have to be modified.
Yeah, you did. Forget this.
Yes, seems to be added when dev'ing in 2012.
The problem with that is when Dart come out with breaking changes it might break this app. I also know @addyosmani would prefer apps to be hackable offline.
If that is preferred then I'll happily move it there. I generally prefer to keep business logic (and thus model handling) as much as possible in the controller and I consider the `is_maintainer` variable to be business logic (tbh: if I were religious about this I'd add a method `isMaintainer` to the User entity that receives a Package and use that.).
`require('some/plugin!{"arbitraryData":"isValidHere"}')`
it works..

```php
<?php
var_dump(strpos(null, '\\'));
//bool(false)
```

Any special reason for change it ?
Are you really going over 120 chars here?
I'm not sure, let's chat on IRC.
all the 4 locator above this comment are broken.
s/ "ol"/ "ul"
do we use all these locators?
I think we should have a FIXME: or TODO: at the top explaining why this test is commented out.
Fine.  You win this one.
can't you just do `ok_(participants.exists())`?
That's a good point; I hadn't thought about the side effects.  I'll patch that up.

On Monday, July 16, 2012 at 5:43 PM, Jeff Dean wrote:

> > + private :ids
> > +
> > + def reset_ids
> > + ids.clear
> > + end
> > +
> > + private :reset_ids
> > +
> > + def has_unique_id?(record)
> > + if ids.include?(record.id)
> > + raise IdError.new("Duplicate Id found for record #{record.attributes}")
> > + else
> > + ids.add(record.id)
> > + end
> > + end
> > +
> > 
> 
> 
> I merged this, so I don't think it's a blocker, but I'm not a fan of the `has_unique_id?` method. I would expect that to be a side-effect-free method that would return a true or a false. Instead, it either raises an error, or adds to a set. For clarity, I would rename that method or split it into different parts: one that answers the question, one that adds to the set or raises an error if it's already there.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/zilkey/active_hash/pull/50/files#r1173630
> 
>
This was moved over to <audio> tag. It makes more sense there.
Yeah, I knew this would have to change once the puppet stuff was done; it's like that because it worked when testing.
It should be `@throws` (not `@throw`) :)
This button label should be "Capture Data", right? To differentiate between it and "Upgrade ThinkUp"?
Might be good to set `var RESET_VECTOR = 0xFFFC` someplace to make this more clear?
We have one. https://github.com/pculture/vidscraper/pull/3/files#L9R39

I didn't propagate it here because I wasn't working on feeds at all, so I wanted to do the minimum of work to those tests to be absolutely sure I didn't break anything.
Use protected instead of private? Also for methods below.
This seems slower. Why?
Has now been removed in master
This will always be true
First, the description of this parameter isn't clear to me. Can you reword?

Second, I'm assuming it's the document the comment is associated with. If yes, per a comment (no pun intended) I made to @nicolaasmatthijs, wouldn't you want to design this now to easily accommodate what has previously been called annotations later?

E.g. here we're assuming the target of a comment is only and always a document (specified by a document id). But an annotation has a target that is some piece of content within a document. The distinction between them is artificial.

In the annotation models I've looked at, you treat the target as an object; see e.g. http://www.openannotation.org/spec/core/#Target.

I guess that leaves two choices:

1. continue as is, and later add a `target` parameter whose value is an object (to accommodate annotations).
2. change this now per 1, where the `target` object only has one parameter (`contentID`).

There's a third option, which I think is probably a bad idea: continue to treat comments and annotations as conceptually distinct.
After further inspection, I see that this may no longer be needed.
UrlMatcher only works with 2.2 and i moved the fixing after matching into ConfigurableUrlMatcher to keep the NestedMatcher clean - its not its job to do this. ok @Crell?
`columns` is not defined...
Это было действительно так необходимо? Нельзя ли проще обеспечить замену X процентов цепочки ДНК?
@voleg ок, ещё раз повторю про программирование: вам дали задание написать метод, при котором последовательность будет изменена строго на х%. Проверить легко, не так ли? Выводим последовательность до изменения, выводим после - и считаем, на сколько изменилась последовательность. Если не строго на заданные x% - всё, провал.
Попробовал для 1000  и 10000 элементов и мутцацией 100%, сложность стремится к n^2 при увеличенни числа элементов, но для 100 элементов и 10% мутации итераций будет всего 10. Подумаю на досуге, как улучшить алгоритм, но для меня этот вариант самый очевидный. После сдачи проекта, посмотрю решения сокурсников.
Тут я могу ошибаться, так как не сильно профи в ObjC, но из логики работы с другими языками выходит так:
Если вы написали в собственном init какую-то хитрую логику инициализации, которую нужно выполнять во всех инициализаторах — тогда конечно да, нужно вызывать собственный init, чтоб он проделал эту работу. Если же своего перегруженного init нет — то что [self init], что [super init] — вызов будет один. 
Я думаю что логичней вызывать таки [self init], на случай если вы потом допишете таки свой собственный init с какой-то логикой — не надо будет переделывать все остальные. 

Но все это при условии что ObjectiveC использует такую же логику в ООП как другие языки.
Better would be "Automaatne Tööpink"
typo: if user is not ACTIVATED, redirect to home
If I'm understanding this right it will replace the current (functional) text email with just the generic default. Why would you do that instead of just flipping the order of calls and leaving html2text around?
Good catch. I made some changes and this should no longer be an issue.
@tclem @Haacked I've created #183 to cope with the lack of proper exceptions issue
damn it. I fear I spent too much time with this, after some while you stop
seeing things....


On Sun, Dec 16, 2012 at 10:30 PM, Alexandre Gramfort <
notifications@github.com> wrote:

> In mne/viz.py:
>
> > @@ -1109,6 +1114,8 @@ def plot_image_epochs(epochs, picks, sigma=0.3, vmin=None,
> >          (data.shape[1] == len(times))
> >      show : bool
> >          Show or not the figure at the end
>
> units is not documented :)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/221/files#r2430689>.
>
>
Same with I/O functions for forward solutions here, and other things (basically distribute copies of the file I/O functions to their respective areas).
> As an alternative to this function, I suggest we add the attributes "lh_vertno", "rh_vertno" and "rh_data", "lh_data", which will make it easy for the user to access the data and, e.g., pass it to PySurfer. WDYT?

I like that
nit: does it need that # at the end?
I still don't understand why we need dump.html, popup.html and popup.js. Please explain here or we can discuss F2F.
Usually what people do is only use "exports" when it is defined, rather than creating a dummy exports symbol for non-CommonJS environments, which strikes me as a bit of a hack. See background.js comment for more details.
How come these had to change?
I think it might make more sense to do a getMap() function that returns the map and gives you access to it, rather then dynamically bring down all of the methods.  Thoughts?
~~~
return $this->_modelName===null ? get_class($this) : $this->_modelName;
~~~
this docstring needs lots of editing
Хехе. Няшненько.
Для этих целей лучше `currentIndex` использовать. И в предыдущей функции тоже.
А QObject проинициализировать?
@AlphaGit @matias78 En realidad, si hay tareas no priorizadas, no sabemos que está antes y que después de la línea de fin de proyecto, ya que si estas tareas se priorizan, tal vez alguna quedará después de la línea y otras antes, estas últimas seguramente arrastrarán a otras tareas hacía abajo de la línea. 

Yo me imaginaba algo mucho más dinámico y fácil de calcular esto de la línea de fin de proyecto, pero parecer que deberíamos tomarnos un tiempo en cada sprint para revisar el Product Backlog.
You should move this up to the Assertion interface.
Next time don't mess the entire file up if you're adding/modifying just two lines - completely ruins the point of a diff if every line changes.
Please declare these route in a single controller collection, like `TaskManager`
Please remove comments
Yes. Its a hack... 

Sent from my iPhone

On Mar 14, 2012, at 23:55, "Yosef Dinerstein" <reply@reply.github.com> wrote:

>> @@ -1,15 +1,23 @@
>> -# wintools - Some windows tools for node.js #
> 
> Jus copied this from your own version in role repository, that looked to me more updated :)
> 
> ________________________________________
> From: Elad Ben-Israel [reply@reply.github.com]
> Sent: Wednesday, March 14, 2012 10:36 PM
> To: Yosef Dinerstein
> Subject: Re: [node-wintools] Parse XML output of WMIC. (#1)
> 
>> @@ -1,15 +1,23 @@
>> -# wintools - Some windows tools for node.js #
> 
> The double `#` also works BTW...
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/anodejs/node-wintools/pull/1/files#r559384
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/anodejs/node-wintools/pull/1/files#r560322
>
The text field content can be changed manually, but this is not stored in the settings due to a missing listener.
Stack trace printing is not allowed. Please use Log.e(...) with the utils.Log implementation of c:geo.
why is that, it would be confusing.
The source of the message is technically the server from which the message originated from. I used to populate this (I think) with an IP address of the remote host. The destination I leave blank because technically the destination of the response is the current connection handling the request. In the future I may put something useful there, but in the meantime the destination is kept empty.

The assigning of an empty string is unnecessary, as default-constructed strings usually use small-string optimization and will hold 16 bytes (or some similar amount) of strings. Remove that as you don't really need to do it anyway.
Inside the catch, print out that you got an invalid content length value.
Again, why remove the license?
Well not by itself, no.
I see the root_key step is concerned with getting the 'first' key of the list that isn't 'links'.
I have to complain that dictionaries are unordered, and so this is only working for us in the case that we have a dictionary with up to two keys, one of which could be 'links'.
I'd rather pluck 'links' out on its own, then count the keys that aren't 'links' and complain if that's not exactly 1, and then assign root_key.
should this be added to the config as well?
So you say lets have:

interface MapperLifecycle {
 void onPreExecute(Mapper <KIn,VIn,KOut,VOut> task);
 void onPostExecute(Mapper <KIn,VIn,KOut,VOut> task);
}

interface ReducerLifecycle {
 void onPreExecute(Reducer <KOut,VOut> task);
 void onPostExecute(Reducer <KOut,VOut> task);
}


interface DistributedCallableLifecycle {
<T> void onPreExecute(Callable<T> task);
<T> void onPostExecute(Callable<T> task);
}

Yes, this could be an option. We can fine tune callbacks per interface if needed. Let me see how this could be implemented and what is the impact.
Thanks for noticing this. I have to rethink if synchronizing those _modifications_ collections are really need now. This was done before I wrote the code that blocks transactions in StateTransferInterceptor. Now I think it is safe to read modifications without danger of getting a corrupt collection so these changes could be undone.
This is why I said to use ``git mv`` (which stages the renaming directly), not ``mv``
I think mostly convention and old C habits - we don't pass structs directly as parameters in libgit2 APIs.
This should be `static`.
privateを明記してください。
I'll probably want to remove this again once we've added default site creation to the `alchemy:install` and `alchemy:upgrade` tasks.
Yes -- this file is autogenerated from the git history -- which unfortunately does have these typos, but c'est la vie.  Probably best to just not touch this file at all.  I should probably put a comment at the top that this file is autogenerated so as to not to waste the time of helpful people like @kdavies4 in the future ;)
Missing indents
We should add <code>_locator</code> after the name. This also applies to the below names.
okay, that makes sense
Два часа не мог понять что имеется ввиду стрелка :) Конечно исправлю.
Then create two separate constants for that.
Like:
CREATION_METHOD_NAME_2_X
CREATION_METHOD_NAME_3_X
don't redefine a variable with the same name
I really think we should be clear and simple with this. There should be only one way to do something. It's either `clientlink` or `client_link`, `auth_code` or `auth_token`, etc. I think here that flexibility  will just make things confusing. Personally, I like the all lower-case values with underscores
typo: should be shallow
.ss-ui-button may be the better class to reference for consitsency
fix formatting, add standard Newscoop header for source code files.
what about sorting in db and returning right after array_map e.g. return array_map(...)
Have a look at line 1113-1118 of this file. If the file is not posted, an error message will be given to the user.
On Wed, Aug 1, 2012 at 7:52 AM, Michael Cahill <
reply@reply.github.com
> wrote:

> > @@ -159,6 +159,10 @@
> >       ((strncmp((str), (pre), strlen(pre)) == 0) ?                    \
> >           ((str) += strlen(pre), 1) : 0)
> >
> > +/* Check if a string matches a byte string of len bytes. */
> > +#define      WT_STRING_MATCH(str, bytes, len)
>      \
> > +     (strncmp(str, bytes, len) == 0 && (str)[(len)] == '\0')
>
> This could also replace `__wt_config_strcmp`.


Sure, done.

--keith

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Keith Bostic
keith.bostic@wiredtiger.com
The restore command - If you want to execute this command, go like this:

/etc/init.d/minecraft restore
Are you sure ``$mockStatement`` has the correct type as ``object<PHPUnit_Framework_MockObject_MockObject>`` is not a sub-type of ``object<Zend\Db\Adapter\Driver\Sqlsrv\Statement>``?