labofsahil/github-event-dataset-2012 huggingface dataset:
Rather than having a bunch of conditional code everywhere checking $VERBOSE, I factored that out into a discreet_echo() function that takes two parameters, loudstr and softstr, where the second is optional.  The loudstr is displayed when VERBOSE is set, and the softstr, if set, is displayed when VERBOSE is unset.
I see ... 

In fact, the things to do are ...

* Add install rules to both PythonQt and QtTesting
* Install this two projects as regular CMake project. See https://github.com/Slicer/Slicer/blob/master/CMake/SlicerBlockInstallCMakeProjects.cmake
a test or figuring out why [these tests](https://github.com/mozilla/zamboni/blob/master/mkt/site/tests/test_views.py#L8) pass would be baller
Does this mean that unary & is going to be a general operator? Or will it be specific to ccall syntax?
@Th0ar 哦，我之前没细看，他是把审核前后放到一个action里边了是吧。。那样的确不行
*sondern ermöglichen auch das Gruppieren von Werten durch Collections und die Verarbeitung von Funktionen.*
ändern in?
*sondern ermöglichen auch das Darstellen beliebig langer Tabellen durch Gruppieren von Werten in Collections und die Verarbeitung von Funktionen.*
Ziel: Genau verdeutlichen, wie man den Begriff der *Collections* in unserem Fall einordnen kann.
fancy trick!
Surely the `return static::log($exception);` on line 58 prevents the `return true;` from ever happening?
what is signing-off for?
let me know if you run into any issues...I am waiting on jenkins but @cvan had some issues earlier with the migration which we think are related to an earlier pull he did - some of this will be moved to constants later anyway as @andymckay suggested
能否加个桌面通知啊，上午好不容易刷到6张票，一看超时了，因为在公司不能用耳机啊。
I wonder, would it be better to add this to SpecFlow?  

The easy way to approach this would be for SpecFlow to allow .feature OR .sfeature files.  The sfeature feature can be mostly ignored, except for some "Using SpecFlow with Sharepoint" article that explains that .sfeature is allowed as well.

Otherwise, most of everything I see in the commit can be added to SpecFlow itself.
I hope so . Although it should not break something for you :-)
I figured I'd match the name of the existing field.  We could change them both though.
Before releasing conduit 0.3 I guess it would be nice to release minor versions of these packages with an upper bound on `conduit < 0.3`, otherwise it's going to be a long dependency hell until everyone upgrades.
Wait, what? Die Pixelkoordinaten dürfen nur zwischen 0 und 255 sein? <mfw:futuramaFry>
Eclipse stripped these as they're not being used :/
If I understand correctly product_details_json provides a list of languages that firefox supports ans not a list of countries. Or at least I couldn't find a list of countries somehow. Can you please provide me more detailed instructions?
:sparkles: :cake: :sparkles:
does [this](https://github.com/robhudson/zamboni/blob/19babee/mkt/developers/templates/developers/payments/paypal-setup.html#L56) need to be hidden for owners only as well?
Thanks for moving it :)
Could do. Something like this...

Awe.array = function(a) {
  var aweray = { array: a };
  aweray.forEach = function() {
    this.array.forEach.apply(this.array, arguments);
  }
  aweray.someOtherFuturisticArrayThing = ...

  return aweray;
}
Not a submodule, clearly a non versioned file.
Why is this property now a string, while the `BinaryFileCreateStruct` still uses an instance of `ContentType` for the content type?
Are there no authorization checks in this service? I am missing some `@throws UnauthorizedException` annotations :)
I approve of this change.
Ironically enough, this failed when I ran the tests on it because you have perversely constructed the test such that it generates trailing whitespace without the source file itself having any trailing whitespace.  My very first trailing whitespace autofix is foiled by the instigator of said autofix.
@feildmaster of course I did, but I am a bit confused still.
Понял тебя.
Oh, I am sorry that I misunderstood, thank you very much Couchu
valuable time to reply I.

Once again, thank you for giving us such good works!
Sparkle would be so sweet
Hmm I don't know really - I'd have to try to replicate it.
This should be getParent.
Those are internal debug leftovers... should be removed
Merci! That truly documents what I already thought about the '>' doing. 
I purely wasn't aware of its name to search on my own. ;-)
Hmm when I try to repo sync I get an error:


error: vendor/cm/: CyanogenMod/android_vendor_cm checkout 8c67daebf8913d439b1b4738c0ca2f8512591495
I did lol
reddit cookie?
hmm really?
Creo que acá van 2 símbolos ##
Mike, is this quickstart good to go, or does it need to be changed use postgresql?
[Here is a script](https://gist.github.com/1861361) to print a histogram of the low-order bytes of tile position map coordinates.  Out of 77 MIRAX slides with tile position maps, I have found a grand total of zero low-order bytes containing anything other than 0 or 1.
Build configs to x86 && amd64
Right. I need to warm up in the morning :)
嗯。。。忘了，得加上
Hey, thanks for the note, that's interesting. I was playing around with this, and I think this particular bug results from the fact that I have Clojure 1.3.0 as a dev dependency, not a full dependency. I had this break on another travis build, actually. (None of my other :dev-dependencies pull in Clojure, and I can't make it a dep in this case).

I'm going to push a new version now with 1.2.1 to test this theory. Thanks for the feedback, I'm loving travis-ci!
是否考虑过：原文两行加在一起似乎也不长，为何拼接一处？为何不在第二个 %s 之后断句？

这是因为 %s 要用相应分支名替换，分支名的长度不定，可能会导致字串过长。所以这两句请忠实原文。
Seems excessive.  How about a new bogus keyword, "suggested", or
something.  That would be easy to filter out, and make things obvious
to human readers.

--Adam

On Thu, Jan 12, 2012 at 2:19 PM, Strife Onizuka
<reply@reply.github.com>
wrote:
> Found the bug, it's a variable trashing problem (I suspected this was the cause of several of these). LSL Const and LSL Constant use variables of the same name (the wiki templates grew organically, they could do with being redesigned from the ground up, but would require a huge amount of work in both designing and porting the articles... so it's not going to happen).
>
> To fix the feature suggestion problem. You have to explicitly declare the signatures for feature suggestions (very likely they will contain an empty signature section). Maybe I should fork the Function template for feature suggestions.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/BlindWanderer/lslint/commit/06977c8cefa28b76ef8ae7d9d16a6c0d30f1f01a#commitcomment-862704
There is a version of getWorkunitFactory(ISecManager *secmgr, ISecUser *secuser) that decides whether to use the secure version or not.
What is the Open Medical Billing Network?
@samueltardieu *Ooops* Without ```equals()``` and ```hashCode()``` the ```Set``` can't work normally as expected. Both methods are missing. Luckily this set is "protected" by the set of geocodes so it works by accident as expected.
Don't tell me you didn't know this one already.
Could, yes, but there really isn't a reason to downcast here.
Thanks! It's a little thing, but it's bugged me for a while... I finally  
had time to investigate it.

On Tue, 28 Feb 2012 01:35:53 +0800, Robin Morawetz  
<reply@reply.github.com>  
wrote:

> Great Work!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/alienmind-org/BlackICE/commit/30f1f0c868828de70860793f173fded6f2572233#commitcomment-1016124
That's why I was wondering if it should be fixed in all the `er/mark-*` commands.  If we decide this is better it would be nice to have this documented and maybe even an `er--trim-space` function to clean off the space (both at mark and point) that could be called in the `er/mark-functions` if desired.  I'm open to either.  The only thing I can think of is that it might be nice to not have the white space trimmed off--except for when there is another `er/mark-*` command which differs only by white space.  I suppose it could be called in a mode-specific way.  I think a relatively easy fix for this is to add

                             ;; Either the start or end differs by more than whitespace
                             (or (string-match (concat "[^" er--space-str "]")
                                               (buffer-substring-no-properties
                                                (point) start))
                                 (string-match (concat "[^" er--space-str "]")
                                               (buffer-substring-no-properties
                                                (mark) end)))

to the appropriate spot in `er/expand-region`.  It seems a little bit hacky the way I created the regex from `er--space-str`, especially if `er--space-str` ever has `[]^-` etc. in it.  Fortunately, I don't see that happening anytime soon. :-)  I wish we could use syntax tables without having to worry whether `\n` is whitespace or comment terminator.  That has to be a problem someone has solved before.
Give it a try first, there's bound to be some issues...
On Mar 3, 2012 8:37 AM, "Gavin King" <
reply@reply.github.com>
wrote:

> so should I disable them now?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ceylon/ceylon-compiler/commit/93f4d165b765ffef50af60ff880799d71e10d5c5#commitcomment-1038450
>
See my recent pull request, I disabled 'play sounds at start' and that's why I didn't notice this.
Hi Ian,

Grischa already explained what happened yesterday. I'm still not sure what
I did wrong, and I hope I didn't cause too much trouble ...

Actually, what I was *trying* to do was commit a fix for the dashboard,
i.e. also display comments with status 'confirm' (add. to 'pending').

I'd say, yes, push your fix. I can add my changes later if necessary.

 - Matthias

Am 3. Januar 2012 11:23 schrieb Ian <
reply@reply.github.com
>:

> brockhaus pushed to master at s9y/additional_plugins about 12 hours ago
> master is now 6a826e2
>
> Grischa, can you please tell me/us what this means?
> Is it a general set back to a status before (as last commit is the sync
> 5 days before)? Why does this not show up in the commit list, but does
> in the general news feed?
>
> As long as we are learning altogether, it might be nice to have some
> more detailed descriptions in these cases. ( at least for me ;-) )
>
> @mattsches
> I found a bug in the dashboard, shall I just drop that in now?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/s9y/additional_plugins/commit/3241bcb3156a9cc193adff968f09ba039bb85cb0#commitcomment-833147
>
In the JSON response. It's not yet documented nor finished.
Ha sorry it's a french sentence, it means "Foo Bar says:"
Foo Bar is the name of the persona i set up in our jenkins...
To make it dynamic it should be a call to getName (or id?) and "dit: " removed or replace by "says: " i think
shoule remove GetSessionDbcLocale. only need DBLocale in cata.
This also changes the BimServer revision, against which the plugin runs. Please indicate this in the commit message. Seems like r1683 upwards.
Any chance of being able to navigate it with keyboard controls? :)
Nice.  I need a recovery day from my frantic 0.6 release.  I'll try to merge this (see how much things have changed in the past 2 days), and then attempt a 0.11.0/1 release.   Note:  I don't have a huge amount of time to support 0.11.0/1 issues.  Can I assume that you'll feed me bug-fixes if you encounter issues?
The stable releases happen on Marmalade. Adventurous people can choose to track Github instead.
Thanks for your suggestions. :) You are right - I'll fix that together with a refactoring of this prototype. This is more a proof of concept.
`return isDeckNameCorrect();`
இதைவிட சின்ன bug fix எழுத ஒருத்தன் பொறந்து வரணும், நீ கலக்குடா அருண்மொழி :) This is called Self Appriciation :P
In that case, you caused a bug, because the menu was definitely adjusted. All the jobs are scattered now =/
No problem, I don't want to urge you ;)

I was just wondering if, by changing this behaviour, I was exposed to big problems in the future.
Just saying this if you remember you encountered this problematic in the past (that would argue for the DateTimeZone.UTC conversion), I'm interested to know your use case to eventually prevent mines for unexpected behaviour :)
So on my system this is what happens:
$ cat /proc/sys/fs/file-max
798224

May be there was a limit on older kernels but programs like database would really require at least hundreds of files open at any time.
Also sockets are also mapped using file descriptors. Servers would require lots of file descriptors to service clients. Of course this restriction would not be worrisome for all because servers can choose to use a fork like model in place of async io.
WTF? Here is a note.
I think your code prints to much data for successful automatic grading. It should only print final answer
Do I need to rewrite the tests so
Or do I need to keep it in order to fit the password_test?
missing commit
it's not working correctly clock for korea omnia2
it's solve can not booting issue on korea omnia2
by aslan
Hi,

This is only done in test cases to inject on an already-instantiated object (the test case itself)

Usually, you will get a Template via a constructor (or an @Inject method)

for example:

```
@Inject
public MyCoolObject(@Named("my_template") Template myTemplate) {
    this.myTemplate = myTemplate;
}
```
was there a problem with this? this code line actually worked for me and produced the right path.
Thanks for catching the fatal errors in the map libraries. I let Pete know that they fix issues that we actually just discovered earlier today.

The change on line 105 will break for data sources that are not KML and don't implement getDescription(), so it will need to be reverted it if we merge all the changes here. If necessary the change needs to go in KMLPlacemark, which currently takes either <address> or <snippet> elements as the subtitle. If we are going to support using the description as the subtitle in KML, we should consider emulating Google Earth by using <snippet> if it exists and the first two lines of <description> otherwise (https://developers.google.com/kml/documentation/kmlreference#snippet).
I think this was a mistake.

We originally made the changes before pulling the latest code. Then we ran "git stash", "git pull", and "git stash pop", and expected that it would do the correct thing. It didn't. It seems that it instead kept many of the things that were removed or changed in the new code. So, for example, it removed "from esp.admin import admin_site, autodiscover", which you added recently.

So the Django logout view was probably removed very recently, but added back by accident by the merge that happened after "git stash pop". I'll go fix that.

It looks like "r'^set_csrf_token'" was changed to "r'set_csrf_token'". I'll fix that as well.
Thanks for your contribution. Congratulations on your first pull request!

-- Daniel
Did you expect it not to work properly?
I refer you to my unit-test program:
https://github.com/mikael-s-persson/ReaK/blob/master/src/ReaK/ctrl/path_planning/test_vp_tree.cpp
that compares the performance of the linear search versus the DVP tree with 2-arity and 4-arity, as well as for a batch insertion of the vertices versus an incremental insertion of the vertices. The results can be found here:
https://github.com/mikael-s-persson/ReaK/blob/master/src/ReaK/ctrl/path_planning/test_vp_tree_results.dat
In which I think are in micro-seconds (time for one query) (and "-f" stands for the "fresh" DVP trees, that is, batch insertion).

The performance of my DVP-tree code is definitely not perfect (I still have some memory locality issues which explains the not-quite log-time performance) but the performance is reasonable compared to linear search. Also, there are other options to consider, like approximate search methods (like random-restarting a best-first search). And, when applying to your source-patch query, you might want to consider techniques which employ the fact that once you have found a patch to in-paint one pixel, it is likely that pixels close to it will also find the same patch (or one near to it) for their own in-painting.
I fixed the indentation. What wrong with the tabs?
CSE 143 is probably printing off the assignments so they can grade it.  

I meant "It's also fine to wrap it to *two* lines, right?"

2 longer lines is better than 3 tiny lines I think.
Yes, but I think it would make it more complicated to read and maintain, since that condition would need to be repeated in the current elseif and else clauses:

    // See if the string ends with a percentage
    if (!($value > 0 && $value <= 1) && substr(trim($value), -1) == '%') {
      // Strip the percent sign
      $number = rtrim(trim($value), '%');
      if (!($number > 0 && $number <= 100)) {
        form_set_error($field_name_prefix . 'percentage', t('Please enter a valid percentage for Percentage of Donations win condition. You can either enter it as a percentage (e.g., 45%) or a decimal value (e.g., 0.45).'));
      }
    }
    // Not a number
    elseif(!($value > 0 && $value <= 1)) {
      form_set_error($field_name_prefix . 'percentage', t('Please enter a valid percentage for Percentage of Donations win condition. You can either enter it as a percentage (e.g., 45%) or a decimal value (e.g., 0.45).'));
    }
fine. I will update the README tonight.
unfortunately i dunno how to remove it i'm a noob with github.....how can i remove it?
looks reasonable to me
It works, but the impact on battery consumption was very small. I tried to slow down a bit more, by setting MMC_VDD_165_195, but then the WiFi became unstable.
hahahahhhaa....

sure, add to manifest
Great, yup.  Although can we just do this...

    class OnTheFlySerializer(self.__class__):
        fields = info

Seems to work for me.
You're right -it just makes it more complicated. ill work on it tonight
Вроде же есть обновленная версия add_tool, не?
Hi,

Here is my test result:

When I am browsing in Modern Aqua theme it shows nothing in 'Last Viewed' item. But when I changed it to default 'classic' theme and then I came back to the ModernAqua theme it shows the list whatever I browsed in the 'Classic' theme. Again I browing in the Modern Aqua theme & it is not showing anything.

Older versions(5.7) it was just working fine for me.


Thx
Ravi
[buildId reference](https://bdsc.webapps.blackberry.com/native/documentation/com.qnx.doc.native_sdk.devguide/com.qnx.doc.native_sdk.devguide/topic/r_barfile_dtd_ref_buildid.html)
There's no need for this extra carriage return. Let's remove it.
In case of an error, it's better to return false so that the caller knows what happened.
There is no way to deprecate this gracefully. Just delete it -- the data model is completely different.
I really do despise these global functions...
We typically use constructor injection for required dependencies.
I think it was assumed that all comments were accounted for given the  3-4 days in between the feedback and upcoming fixes. We could certainly carry on the discussion to apply any further suggestions and fixes though.
len(self._dim) is 1, so reversing it won't make a difference.

I think it's easier to see what's going on with the .T call, which takes care of reversing the shape properly for us irrespective of the dimensions anyway.
I also thought about this, but we are dispatching messages in several places and several distinct ways, all entitled to receive the one appropriate name …
Fixed. I replaced the queue with a `LinkedBlockingDeque`, leading to a dramatic simplification of `receive`.
Why do we want/need all these debugs?
I don't understand what you mean, or what you think is the problem. Please elaborate.

When would it not be good that each node is monitored by 5 other nodes? It will of course not be more than number of members - 1. Why should it be proportional to number of nodes in cluster? That doesn't scale. 

/Patrik

8 okt 2012 kl. 18:17 skrev Viktor Klang (√) <notifications@github.com>:

> In akka-cluster/src/main/resources/reference.conf:
> 
> > @@ -78,6 +78,10 @@ akka {
> >        # how often should the node send out heartbeats?
> >        heartbeat-interval = 1s
> >  
> > +      # Number of member nodes that each member will send heartbeat messages to,
> > +      # i.e. each node will be monitored by this number of other nodes.
> > +      monitored-by-nr-of-members = 5
> What if you do not know the grand total of nodes when you boot the app?
> 
> —
> Reply to this email directly or view it on GitHub.
>
that's what I meant: it is not documented in the stdlib right now, I'll have to submit a pull req to change that.

But I just now rediscovered common/duration.rst which I'll fix.
Would you mind adjusting these 2 checks to do an `=== true`? Just to be explicit (and certain that we're getting what we expect!).
oh, right

I guess I missed one.
You can add into a list: <li>
Do not see why we need to delete the above, apart from the missing unquoteing.   Why do we we need a fixed list of filter names at all?
Feels a bit dangerous to reinstantiate this per task. The renderer carries information on both the sync and publish reports and keeps state of their rendering progress. This will probably work, but feels weird to use one class for what is now data in two entirely separate tasks without using a single instance to store that data cumulatively across the tasks. If for some reason we ever tried to have the renderer do a finally summary, it would look like it should have the sync data populated but it really won't since the object is thrown away per task. It's misleading to new devs since by definition, it looks like a single renderer will be used for both sync and publish.
Do we want this as info or debug?
Hmm, it does occur to me that the inject is a bit fragile.  if you have a nil result from fetch, then the next "injection" will break.
doesn't need to list all types, does it?
Pleas honor XBMC coding conventions: put {} on their own lines.
эта колонка не нужна
Die hele if() zou ik iets mee doen. Eigenlijk zou de max identation 2 moeten zijn. Als je vaker gaat indenten gaat er iets mis.
What is this monitoring collecion?
Please document it in
https://github.com/NAMD/pypln/wiki/Document-Store-Design
Hrm, yeah 'logger' is really a term used for objects that log errors, or output some information about the runtime of a program - they dont generate useful datapoints like you want here.

I would just narrow this down to "graphing tools for EMRio that use matplotlib"
You need some tests for `main()`. This would be a good place to use mocks; you basically just need to verify that the `main()` functions for the sub-commands are called with the right arguments.
Yeah, I can see that. I guess I'm saying that I want to keep this code path heavily travelled for now.

I think the clean way to do this would be to pass the steps in as an argument to the runner class's constructor rather than patching the runner's attributes. I'd feel better about this if we had a well-documented dictionary-like format for representing steps. It's not bad for the runner to represent steps internally like `['MR', 'R']` but it seems a little weird for this to be the canonical way of passing that information around.

I guess what I'm saying is that I'd like to do the steps optimization, support for other languages, and deeper support for the Java side of Hadoop all together. Not that we need to have all the code written before we release, but that we at least think through all these things and spec out the steps format.
That particular key `'featurepopup-map'` is not defined in the example-utility file. Could you please add a short description alongside some example code there?
Can we use both TLS and SSL?
Using Zlib requires that. Calling deflate/inflate must be done from
the same process which created the zlib streams (because of them being
ports). I was actually stuck for a good while as of that. The shell
experiments worked OK, while the program itself didn't, though the API
access was exactly the same. It's quite bad the doc doesn't mention
it.

2012/6/29 Krzysiek Goj
<reply@reply.github.com>:
>>      wait_until_closed(State#state.socket),
>>      {stop, normal, ok, State}.
>>
>> +handle_cast({send_compressed, Zout, Elem}, State) ->
>
> Why do we do it as cast, not in sender's process as we do the other sends?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/goj/lxmppc/pull/3/files#r1077420
Empty block after if statement is unnecessary. Could you change it?
Text field?  Shouldn't it be a text area?  Confirm it with Vivek
I don't think public API documentation should refer to private members.
I fixed this in the latest revision
I'm not sure how that answers my question.
What was the point of this method? Seems like it was there to trigger animations at a certain time? That would be pretty nifty.
Actually, that's why I mentioned it. This patch changes the context that `options.createConnection()` runs in from the global object to the agent object.
你这些代码是写在哪里的啊？你确定不是应该写在构造函数里面？
This is a TypeErro
Personally I would go for commitMessage then.
I can also see applications where we would only be asking for commit messages, but not having all the other versioned stuff in the UI.
replace the script name with ``%prog``
doxygen-комментарий нужно
Закомментированный код лучше либо убрать, либо пометить, почему он остается. Иначе через месяц никто уже не вспомнит, почему он закомметирован.
Class & ID names are usually hyphens(`class=table-stripped`) & sometimes camelbased(`class=tableStripped`) too. But seldom underscores(`class=table_stripped`) are used.
Updated to remove the "Austin Style" return
There is a trailing whitespace here.

And I think we can do this:

```ruby
key = key.respond_to?(:to_sym) ? key.to_sym : key
```

instead of use `rescue`.
Not sure if it is. For a while after Lion was released trying to run `gem install pg` would cause the following error:
`missing required architecture x86_64 in file for architecture x86_64`

And that was resolved by pointing the pg gem to the actual configuration from Homebrew. 

I've since added that `--with-pg-config` flag to my system-wide bundler settings as a precaution.

I'll blast my Homebrew install on my home machine tonight and remove that bundler setting to see if it blows up on Mountain Lion as it did, but unless the pg gem was fixed to account for that, that certainly risks being an issue for Lion users.
Yes. I agree with you. Will do.
ditto for the other test names that talk about `User` settings
yeah it was originally there. I think that maybe the code was changed and not the comment.
Sorry for being so nit-picky, but I didn't notice that before. I believe we could be consistent and use `CentOS` everywhere, the 3 versions are different. Can you fix that please?
Can remove the extra spaces around the args
I'm fine with the `||` too.
Maybe we should change `stale_target?` to check for that?

/cc @tenderlove @jonleighton
What do you think about "rebind" instead of "bind".
Sorry I'm late to this party, but this seems pretty suboptimal to me. If I understand the motivation being Stream evaluating one too many elements, that can be addressed without inflicting an iterator on it.
```
  override /*SeqLike*/
  def lengthCompare(len: Int): Int = {
    @tailrec def loop(i: Int, xs: Repr): Int = {
      if (i == len)
        if (xs.isEmpty) 0 else 1
      else if (xs.isEmpty)
        -1
      else
        loop(i + 1, xs.tail)
    }
    if (len < 0) 1
    else loop(0, this)
  }
```
"can there be an instance of A with B?" asks the opposite question as the method, but I thought it was apparent from context.  (Isn't there always a negation missing somewhere? Daring comment!)
it's okay to leave this for a future commit, but it would be great to restructure this method so we don't need these returns
Why this check? In what case could it be a mark?
I have added the get_point_coordinates method to facilitate the working of a ReferenceFrame as a Co-ordinate Frame. This is simply because, coordinates of a point with respect to the origin is something that I deal with a lot while working with 'frames' in general. As you suggested, I will change the implementation to use dot-product.
Was clearer before.
You mean I am computing the ecart twice for no good reason?

Thank you very much so far!
I'll avoid `all` as this is already a list function, `notFound` is more idiomatic than `not_found`, but the name is good.
this comment is wrong; it is not about r2swf
Need a link here explaining what exactly it does and what are all these `$2a$`.
Bundle would be suffix using the tryCreatePrintWriter abstract medhod
I'm not sure how i feel about this.
A good "!!!".
This doesn't match the line below, they should match.
Mit dem erstellen einer eigenen Transaction-Class wollte ich da nur das drauf haben, das auch wirklich zu einer Transaktion gehört.

Dieses Objekt habe ich eingeführt, um keine verschachtelten Transaktionen mehr zu haben.
(In der Doctrine Doku wird explizit davon abgeraten!)
are we missing some more tests for some other more complex cases?
Thanks. I switched to a critical section.
Params does not have .has_key method.
what are you trying to achive? to see if this param has value other then nil? if so you can define it as:
   Params.string('exist_cd', nil, 'exist_path')
Then ask:
   if not Params['exist_cd'].nil?
prob should have just copy pasted. Going to go do that now.
This needs to be changed for `self.collection.aggregates.sync()` (as `.aggregates` replaces `.blobs` in this case)
I don't know that this is a "feature" so much as a behavior. 

sometimes the metaphor of a circular buffer is useful for explaining capped collections
Might be nice to have a `result` version of this too.
OMG! So this means I have to change this in all the files? If so, is there any other way of doing it?
In most cases then there still shouldn't be a clash if the columns are named properly. In those cases where there is a clash it is IMO cleaner to use an alias.
"You should not" is perfectly correct...


On 4 May 2012, at 07:36, Oleg Pudeyev
<reply@reply.github.com>
wrote:

>> @@ -502,7 +532,7 @@
>>    <p>In SQL statements mixing single and double quotes is partly allowed (following the guidelines listed here about SQL formatting), else one should try to only use one method - mostly single quotes.</p>
>>
>>    <h4>Commas after every array element:</h4>
>> -    <p>If an array is defined with each element on its own line, you still have to modify the previous line to add a comma when appending a new element. PHP allows for trailing (useless) commas in array definitions. These should always be used so each element including the comma can be appended with a single line</p>
>> +    <p>If an array is defined with each element on its own line, you still have to modify the previous line to add a comma when appending a new element. PHP allows for trailing (useless) commas in array definitions. These should always be used so each element including the comma can be appended with a single line. In JavaScript, you should not use the trailing comma, as IE doesn't like it.</p>
>
>> EDIT: In fact, maybe also last sentence should be rewritten, because "should" is not proper verb. You cannot use trailing comma, because of IE.
>
> Correct.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/phpbb/phpbb3/pull/779/files#r772606
That is why I sad "in a subsequent PR" ;)
This no longer works with return to query strings.
Well this conditional block is never satisfied for sphinx i.e if statement on line 294 is never true for sphinx. This conditional block is executed for other search backends when all the keywords are less then min word length or greater than max word length, sphinx doesn't check for this condition before searching.
Call $(this) once.
Isn't this correct: utf8_htmlspecialchars(utf8_substr(htmlspecialchars_decode($last_post_subject), 0, 30));
as far as I can see this can be removed
Not sure of the proper jsdoc style for doing alternative signatures.

Took this approach from http://usejsdoc.org/tags-param.html under the heading **Multiple types**
I think the `Y.mojito.models` lookup is wrong.  When using sharedYUIInstance, that will contain -all- the models in the app (including those in other mojits).  Instead, the check should use `instance.models[modelName]`.
ba tavajoh be inke `#in` id hastesh chera enghadr pichoondi ?
masalan `.top_menu #in` kafi nabood ?
nakone chand ta id `in` darim ?!!!
хм... ну у тебя на странице units если нет баннеров тоже ошибка)
this takes a lot of arguemnts. Would it be better to take, say, 2 arguments: providerSent and config ... or just config, which is a dictionary of all the possible fields. 

Will these fields ever change later? Will be annoying to update method calls that have 8+ arguments!
Test diagonal with different axis then the default too. (offset, axis1, axis2) (1,0,1), (-1,0,1), (0,1,0), (-2,1,2).
Good catch. We basically just need to reproduce the MIT license, with the
author's name subbed in for the copyright notice, in that file somewhere
(probably right above the code).


On Mon, Dec 3, 2012 at 9:07 AM, Olivier Delalleau
<notifications@github.com>wrote:

> In theano/misc/ordered_set.py:
>
> > +try:
> > +    from collections import MutableSet
> > +except ImportError:
> > +    # Python 2.4
> > +    pass
> > +from theano.gof.python25 import OrderedDict
> > +import types
> > +
> > +def check_deterministic(iterable, known_deterministic):
> > +    if not isinstance(iterable, (list, tuple, OrderedSet)):
> > +        if not isinstance(iterable, types.GeneratorType) and \
> > +                known_deterministic:
> > +            raise TypeError((type(iterable), known_deterministic))
> > +
> > +if MutableSet is not None:
> > +    # From http://code.activestate.com/recipes/576694/
>
> I'm not an expert on the topic, but don't we need to do something about
> the license here? Apparently it's "MIT license", but I couldn't easily find
> its exact formulation.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Theano/Theano/pull/1107/files#r2292545>.
>
> [image: Web Bug from
> https://github.com/notifications/beacon/V4WeLnGyQaVRVkc2bSJLzmSyzDafggi5JdoKbHu6EwsnmF2vP4WcSp3_0cLH1DMS.gif]
This doesn't check if the data is the same as before. Could the data be truncated somehow?
This looks like it A) is overly complicated and goes too deep and/or B) could miss in some cases? Why not just:

        if is_sequence(item):
            for subitem in _flatten(item):
                yield subitem

In particular, you shouldn't be flattening subitem if it is not a sequence.
Good. I replace comments with your suggestions.
Good call. I added more logic where nil doesn't work to accommodate.
Why?
If device is in the, as I called it, GUI Mode, then we receive all required information without explicit requests.
Shouldn't this decorator live in `django.contrib.auth` somewhere instead of here?
Use `get_field_by_name('username')[0]`, `get_field` is O(n) whereas `get_field_by_name` is O(1). No joke.
Why don't we just use model.cid here?
The download of eigen has been changed in madlib/madlib/master, could you rebase on it?
List if the only way we want to trigger batch behavior? No chance of tuples ending up as input for example?
The template has portions that, depending on if a given form is happy or sad, fills in the appropriate text. In particular, if you look at https://github.com/mythmon/fjord/blob/2862df/fjord/feedback/templates/feedback/feedback-form.html, it fills the text server side.

The feedback page exists as 3 `<article>`s. One is the introduction, with the happy and sad button. The other two are the happy and sad forms. JS does the animation to bring the right one into view based on what you click in the intro, but they all exist in the HTML from the start without JS.
Have to use a smarter check to determine this directory; it won't always be python2.7 (e.g. the system python on Snow Leopard is 2.6). grep for "which_python" in the formula directory for examples.
setTimeout( notify_others_of_editing, ActiveOptions.interval);
oops, I'll add that.
der type war jetzt erstmal für closures nötig.. die leider - weil anonym - bei der timing ermittlung aussen vorbleiben. gäbs nicht-anonyme callable objects auch?
less rage is fine is guess ;)
I think the test should take account of the `use_session_stack` option instead of being removed.
But I guess that's the reason it's commented instead of being completely deleted :)
Mac下的R会忽略`~/.Renviron`吗？
I don't think you need regex here. `relativePath = taggedRelativePath.substr(Zotero.Attachments.BASE_PATH_PLACEHOLDER.length)` should be sufficient and preferable for performance reasons.
Version lock this for crypto, por favor
Orientation should still be locked  in portrait.
On the Mac, Cmd-Up/Down are used for go to beginning/end of doc. I can't tell if there's a standard Mac shortcut for scroll line up/down, but Sublime uses Ctrl-Alt-Up/Down and TextMate uses Ctrl-Alt-Cmd-Up/Down (which seems like a lot of modifiers :)) I suggest Ctrl-Alt-Up/Down.

Is Ctrl-Up/Down standard on Windows?
For the sake of efficiency, first check that suffixPos (and prefixPos when updated) are not null.
Ok, leaving it there seems reasonable -- could we just doc that it has to be treated as a read-only view?
Google translate returned "Wow ... looks like {APP_NAME} still does not work in browsers". Maybe the translation is bad? But maybe this can be worded more gently saying that it Brackets doesn't work in browsers "yet". If the translation is correct, it makes us sound slow. ;)
Gack, no. Fixed. Sorry Jason. Probably my worst pull request ever.
How are menu item groups defined and positioned? I don't see any API functions for them here.
Shouldn't this happen after ``triggerHandler``? Looks like we're firing the event passing the same editor as the current and previous arguments.
Underscore added and tests updated.
Use double-quotes (see note above).
please add a check for options.name, it might not be defined
Is this comment really needed?

Also, you need to put the cleanup inside a try/finally block, otherwise the cleanup is not called on errors.
Doh! Good catch. Here's a commit that fixes that.
*adds support*? Or defines?
Yep :D.
we can just set flag to `true` on line 214 instead of having this else statement as it breaks and sets it to false
Let's stick to span since everyone knows what it is and what it stands for. If `<i>` disappears from spec, or becomes something else, it won't harm us. It's only a few more characters :P.
Remove, not needed
But at the moment this change would mean that my logging on my local system would stop working.

In the long run I think having only one log4j JAR is better and just doing it in the Log4jConfigurationManager is better.
This test will only work for the `Ruby` driver. Please skip the test if a different driver is used (otherwise the test will fail).
use CONNECTION_POOL_TIMEOUT_DEFAULT = CONNECTION_POOL_KEEPALIVE_DEFAULT, if the indent is to make pool and keepalive timeouts equal,
value的序号不对？
Remove internal ref
Typically, helpers are intended for views; however, they can be included in a controller, if needed there also.  That said, if the intent is to share this code across multiple controllers and/or models, I'd suggest placing it under src/lib.  This is a location where we have several modules defined for similar reasons.
Nevermind, I just checked the subprocess documentation.
Should you also see Pediatrics?
I have to admit.. I'm also feeling the strain lol.. Its all good.. I
appreciate the feedback and will endevour to make all of the requested
fixes :) just need the time to 100% grok the monolog implementation.


On Fri, Nov 30, 2012 at 3:46 PM, Jordi Boggiano <notifications@github.com>wrote:

> In src/Monolog/Handler/PusherHandler.php:
>
> > +
> > +/*
> > + * This file is part of the Monolog package.
> > + *
> > + * (c) Jordi Boggiano <j.boggiano@seld.be>
> > + *
> > + * For the full copyright and license information, please view the LICENSE
> > + * file that was distributed with this source code.
> > + *
> > + * @author Ross Crawford-d'Heureuse <sendrossemail+monolog@gmail.com>
> > + */
> > +
> > +namespace Monolog\Handler;
> > +
> > +use Monolog\Logger;
> > +use Monolog\Formatter\JsonFormatter;
>
> Could you guys please stop sending useless comments? Unused use statements
> don't do any harm, and they will be removed next time I run the
> php-cs-fixer on the project. I have better things to do with my life than
> deleting a billion emails about stuff that php-cs-fixer handles
> transparently, and you should too.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Seldaek/monolog/pull/132/files#r2278370>.
>
>
This is no good. We're leaking details about the message payload here in the message bus itself. I think there's a better solution here.
You don't need ToString().
hace -> have
Should these things injected into the `locals()` namespace also be added to `__all__`?  (I'm not certain they should: I just don't want to miss it if so).
As both address and label are passed in, the code to construct the title could be moved to QRCodeDialog constructor
Won't this cause strict errors in 5.3+ as that parameter is used for the previous exception?
It does save you from accidentally doing assignment when you want equality.  `null = $foo` is an error, which is one feature of yoda conditions.
Information has no plural in english.
Why don't we call this class ProxyExceptionInterface ?
Why should it? It's just 1 word :)
Logging configuration like this should never be unconditional inside a function, nor should there be stdlib imports anywhere but the top of the file.  The ifmain block is an artifact from before the config system / Application object code was written, and should probably just be removed.  It is never used.
Sure, though I should note that the API hasn't changed.  Nobody should have ever explicitly called `%load_ext parallelmagic` at any point, as it's always been totally redundant with activate.  The only difference now is that activate is unnecessary as well, rather than being the only necessary step.
It is awkward to pass `IPython.config.cell_magic_highlight` here.  I still prefer that dict to be in the `Cell` class rather than config and the `auto_highlight` method should just it it rather than having it passed in as a method argument.
Why did you decide to make the call this way and not with the higher-level `run_line_magic` method?
Yes, well the point of this line is to prevent matching the current line you are executing (or any previous line), however just blindly looking for 'rep' or 'recall' in the sequence is pretty fragile.  Nonetheless I've included your suggestion and pushed the change.
Agree. I also suffered testng groups and got crazy trying to make them run the tests I wanted.
I haven't seen other providers use the xml based configuration, so I just configured the exclusion with ant path exclusions

Anyway, for me it is ok to make whatever changes are required to have the cleanest pom.xml possible!
`)  )< 0)` -> `)) < 0)`
I've noticed a trend of people not liking my indentation. :weary:
Can't you set controls on the element itself and not here? IIRC they don't show up until the video starts playing.

Also, from my testing, changing the poster doesn't seem to do anything since you call play immediately, which gets rid of the poster and shows the loading icon. Unless behavior in other browsers is different, you could probably get away with removing the other posters and data attributes and just use the first one.
Not sure if this is a rule or not, but when the opening { is on its own line, I like for the closing } to have its own as well, and be dedented.
Changed the tipWidth to 450 in order to fix the alignment of the arrow with the tip box.
Awww... I'm going to miss code like this!
Ah, never mind, I see what's up now. It looks through the top 60 to find 50 translatable documents.
Those latter files are from the Mozmill IDE which we are going to remove. See [bug 791625](https://bugzilla.mozilla.org/show_bug.cgi?id=791625).
This is odd. I figured maybe I wasn't compiling what I thought I was compiling - but I verified it, and the right source is getting pulled in, compiling, and passing tests.  I will make this change, but I'd like to understand why I'm seeing something different here.
Where are you building from? What version of Erlang?

I see this when I build from the chef_wm directory using `make`
What's this do?  The look and feel should be set once at startup.  Is this some hack to circumvent memory leaks in Swing or something?
Shouldn't this argument remain as "host"? It corresponds to the "interface" argument that can be passed on to the notebook.

(I remember there was some discussion about the interface argument, but I can't remember where. So, ignore this comment if this change is because of that discussion.)
I just removed this clase from test_extraction, placed in a common file utils.py, and called from test_extraction, and reused in text_extractionII.

test_extractionII is where i added a test for this new feature. As it implied to copy some code from slybot in order to test, and test_extraction has becoming very big, i preferred a separate test file for this one.
using keys is much more maintainable: you don't need to edit all translations when you fix a typo in the english version.
Please revert this change. We need this feature in the form type extensions.
If we find a non-deletable buffer, is it a good idea to just skip it and try to close the ones after it?
I would have stored the executable under a `Resources/bin` directory like done in some other components.
Btw, it does not return an integer as you can store anything in a ParameterBag (and the Request will generally store strings as $_GET and $_POST only contain strings). So the only possible fix is ``@return mixed``
With a new name and some docs, I'd prefer this be elevated to the fundamental public interface for the user-supplied node value materializer.  I think anytime you have a function that takes more than 1 arg in a public interface, its more clear to create a dedicated interface.  This affords an opportunity to choose a better name than apply if one exists and also choose a more apt name for the function interface.  I think the benefits this bestows users of the api often outweigh the inconvenience to the library author and those api users that might benefit from something like Functions.compose for example. 

Naming suggestion:
interface NodeDeserializer<T> {
  T deserialize(byte[] data, Stat stat);
}
出发 onend 方法？同上
This seems like a reasonable check, though I agree that it doesn't tell the whole story.

Any ideas on how to validate the angle? (actually -- we know that one is wrong right? We probe along one of the two input axes instead of the ellipse axis IIRC)

My vote would be to just to disable filtering and super-sample something like the "tube" test with a nice filter (like a renderer would do) and compare that to what texture filtering does. Something like Chris Foster's "swirl" test from aqsis would also be good (nice mix of anisotropies and angles).
Would you mind splitting this line please.
``from django.utils import simplejson`` for compatibility? or is this not needed anymore?
why not
```python
super(CMSTestCase, self)._urlconf_setup()
if hasattr(self, 'urls'):
    set_urlconf(self.urls)
```
Mark had some suggestions on the decorator, but will be committing flask.jsonify() soon :-)
So it is! Thought for sure it was a typo of `on` :)
En fait je ne me souviens pas avoir mis de règle sur les logins, donc là je ne sais pas si tu vas tous les prendre.. C'est sans doute plus simple de juste chercher " @" (avec un espace devant le a) et de s'arrêter à l'espace suivant. Là tu prends tout facilement.
Why are these actions run using SecurityActions?
It was requested by the jira (https://issues.jboss.org/browse/AS7-1189) to not display the stacktraces. I was not able to find a way to "hide" them other than reverting to blanking them out. Do you know of a better way of doing this?
Same for all calls to `.trigger()`.
Let's declare that after the mozSettings check. No need to assign something that will be never used in some cases.
// Deal with application uninstall event
    // if the application is being uninstalled, we ensure it stop running here.
@vingtetun Oup! here it is!