labofsahil/github-event-dataset-2012 huggingface dataset:
A definite *-1* on including codenames here. Since the Windows build, we don't release that often anymore, but as soon as that gets going again, weekly releases shouldn't be too uncommon.
Why can't we use `compat_str` instead of this?
这条出错记录没必要，也不对，insert+direct模式也可以不配置preaccess的handler。
What's the true user name to run Tengine? www or nobody?
Is this reported as a metric elsewhere? Seems like something we'd like to know.
Richard Yao's question reminded me that I wanted to ask about your handling of iSCSI naming in general. That lead me to look at this code...

This whole function's objective seems wrong. From what I understand, the date does not represent the current date, but the date when you first controlled the domain. While the former is okay as far as uniqueness is concerned, the latter is *at least* the convention. That's why ZFS uses: "iqn.1986-03.com.sun".

Is it possible to generate names using Sun's system (such that the behavior would match the same system running Solaris)?
In markdown source code examples indented with four spaces. Plus one space after *, so it should be indented with five. All other is fine. Thanks. And sorry for pain :) If you want to check how your documentation looks like, goto: http://jsdoc.info/sdrdis/jugglingdb/abstract-class.html#class/belongsTo
seems like all these could be refactored out into their own class vs having branching in all of these methods.
'path' is just the appended url. if you look at **client.conf** you'll see that 'server' is the section and 'path' is on the of the saved strings (**/katello** or **/headpin**)
And, for instance, maximal depth of 3 or 4 for the recursion.
I agree.  Trying to find a better solution, I just checked with the source code to the `file` command, and this is exactly what it does.  We may have to do something like "try it as a bzip2 file, and if that raises an exception, fall back to pure ascii".
When writing to a file (including `sys.stdout`) Python will always convert the `'\n'` character to the appropriate line separator for the host OS, unless you explicitly tell it otherwise (such as by opening the file in binary mode).
You are right, should be ``1.50.0``.
I found this one and it explains what I was missing here: https://bitcointalk.org/index.php?topic=101514.0 thanks for your further explanation, too.
Doh I was right the first time, diffs can be confusing sometimes.  With your patch if $salt is true the lines executed will be 
```php
$salt = Configure::read('Security.salt') . $string;
$string = $salt . $string;
```
and $string will effectively become `$string = Configure::read('Security.salt') . $string .  $string;`
Another 4/5 mixup.
@markstory 

Do we still need these `if (!defined(LOG_*))` ?  I'd like to remove them, since they seem to be implemented for > 5.2 afaik.
Well I wouldn't expect nth-of-type to work in Internet Explorer. Can you run a test and confirm that?
what about ``CONTAINS`` to check that the field contains a given value (for fields holding an array of a collection obviously)
You can easily get rid of this else. =)
Downside of `posix=False` is how backslash is treated.

```python
In [1]: import shlex

In [2]: shlex.split(r'a \ b')
Out[2]: ['a', ' b']

In [3]: shlex.split(r'a \ b', posix=False)
Out[3]: ['a', '\\', 'b']
```

I prefer the current version of `%run` (using default `poisx` value).
It is intentional, because the code path for `git_diff_blobs` leaves repo unset (since there is no point in trying to look up the blob "filenames" in the attributes list).

That being said, looking over that code path, I think there is probably a better way to structure this. I'll reexamine the way that `git_diff_blobs` uses this code and see if I can make it a little more obvious.
My other concern is premature optimization that makes the code harder to read.
I'd probably stick the model name in there, too. e.g. "wiki_document:related_docs:%s"
`os.path.splitext` returns a tuple of ("prefix", "extension with dot").
So shouldn't this be:

```
prefix, __ = os.path.splitext(a_file_name.split[1])
```

Also, what's with the `a_file_name.split[1]`? Isn't `a_file_name` a string? If so, `a_file_name.split` is a function.
I don't think you want to delete this from my repo, or else sidekiq.org will stop working.
you really don't need to nest these tries
Hmmm. . . . this reminds me that I always wanted to add a MimeType parser to android.util and never got around to it. Sucks to me. I'd be tempted to just require that the Content-Type header contains "charset=UTF-8" as a substring. That's dirty and bad but much more sane than supporting arbitrary content types.
Your code is not identical with my. `preg_match()` checks that string after - is not empty.

I leave it as it is, with additional space after - in pcre:

    if (preg_match('/^- .+/', $this->currentLine)) {
It's fact the other way around. The visual grid produces less rounding errors than the span-columns() mixin.
I'm not sure if we can assume that error codes and messages are in ASCII.
good question, should not be in here
This looks fine in FF and Chrome on linux. If it looks bad on Windows/Mac, we can revert this change.
ok, the checkin redirect is still screwed, but the main post add works now.
If you use `attributedString` only when `italicFont` is true, then move this condition on top of this block and encapsulate whole `attributedString` creation into this `if`.
Is this ```0 +```  necessary?
I think VBD_epoch_end needs to be in VM_poweroff and VM_reboot but not VM_shutdown -- if you look at VM_suspend it calls VM_shutdown. Probably we should rename "VM_shutdown" to something more domain-specific to avoid this kind of confusion in future!
Oh you're returning "I". Can we put that in quotes (can use \ to escape single quotes or just use double)?
kheyli mohem noist vali kolan position hato ba nib va baclass benevis :D
论文里是不应该出现字体的变换吧，类似的问题之前讨论过，在模板里就删掉了其他不应该用到的字体
即使个别人有这种需求，我很怀疑这个比例到底有多少，1/100有么？如果有，也应该是他们自己去解决这个问题。
为了那样极少数人，在给所有人看的文档里面加那么一大段无关的内容，太不值得了
Предлагаю метод поиска ближайшего события по значению вытащить в коллекцию `projectEvents` - там ему самое место. В дальнейшем можно там будет оптимизировать алгоритм поиска.
This does not work on whatever version of commandhelper I'm running, but apparently works with you. I still recommend using /!, rather than #.
Update will be enroute.
Commands calls ensureEval to get around that problem.  I can put it one level lower in getReferenceKey here, which might make more sense because it only matters when you match with a LazyBs.
But according to the tests, this does get the ordering right, no?
These imports are actually for namespacing (so you dont have to understand where certain things live)
Blue was never working, removed it...
No, will remove this.
is both really necessary, should it be enough to go with synchronized?
Is that acceptable? If not, we could introduce a `_moveToTop` method that returns the boolean result, and have the regular `moveToTop` call that without returning anything, while using `_moveToTop` internally.
Wow. I'm not sure we understood each other.

Here we're not testing the same behavior from one test run to another, sounds bad...
We should have 2 tests where we **know** what we're testing. And where we're testing **both** behaviors each time.
Could you have 'Spanish' disabled? Now we have no locale for it.
Maybe, it would be good to update https://hg.mozilla.org/mozilla-central/file/08d435dedc7f/dom/network/interfaces/nsIDOMMobileConnection.idl#l198 and include some comment regarding the invocation of the onerror function of the DOMRequest in case anything goes wrong such as at https://hg.mozilla.org/mozilla-central/file/08d435dedc7f/dom/network/interfaces/nsIDOMMobileConnection.idl#l198 Or is there any other place where this information is avaiable since https://wiki.mozilla.org/WebAPI/WebMobileConnection is not updated YET? :-)
Take care! Probably you will change the hash before rendering threads properly...
As my remembered, they'd like to restrict the wallpaper selection at first page.
So maybe we could only listen to `contextmenu` at landing page/first page.
The landing page don't have to have an editing function because app icons won't live there. How's this idea?
Why do you not use the UTF-8 character?
They're just a guess.  The camera on unagi has a max resolution of 3 MP, but we'll probably be using it at about 2MP.

My point with these limits is to try to prevent OOM errors if the user puts big images from another camera on the SD card.

We can adjust the values as needed, but I need to put some upper limit in.  A 5 megapixel image requires 20Mb of memory to decode and display (or even to generate a thumbnail) So its pretty expensive.
need to return here?
The point is that to check if the content is bigger than the available space we have to make the comparisons:
   (primaryInfoMainNodeWidth + manyContactsNodeWidth + entryCountNodeWidth) > availableSpaceNormalMode)
and
   (primaryInfoMainNodeWidth + manyContactsNodeWidth + entryCountNodeWidth) > availableSpaceEditMode)
and these comparisons are easier in pixels since the data returned by getComputedStyle() are in pixels. Of course we could do them in rems but we would need to make the conversion for each specific case and not once as we are doing right now.
On the other instead of hard coding 10 we use the same getComputedStyle() to get the font size of the body:
   var defaultFontSize = window.getComputedStyle(document.body, null).getPropertyValue('font-size');
I am glad to know you are planning that :) Reading your code would learn a lot than reading code of others.

I'll say we can keep this pull req as-is and move to your library in the future if we are not in such short schedule ...
Sidenote: QCoreApplication::arguments().value(0) does have the correct application path. It'd be really ugly to rely on that here, though..
Good example :) 1 "highway" key and 4 value. What is the logic? We are missing value and including only tag?
yeah ... don't know what got into me here :)
Good catch, I should have had one for both.  This one recreates the problem (calling the other method, grouping by hour).  I just pushed up a second test that calls the new method (shows the fix).
really awesome! :-)
This is ugly... But I suppose this has something to do with PostgreSQL, hasn´t it?
this function seems pretty fragile. For example you're not escaping single quotes. 

Also, couldn't this whole thing could be done with a regexp?

something like:

`.replace(ESCAPER, '\\$1')`

I'm a little nervous about accepting this pull request.
I think you actually need CTuple, not Any here. The issue is if someone counts multiple fields (like age, height combinations) the arity will be 2, but the default TupleGetter has arity 1 (singleGetter). By using the CTuple type, scalding just passes you the original cascading tuple.
Ok now that I looked at the test case case more carefully, I realized what you are talking about. This behavior really sucks, except its probably not going to affect most cases.

I was mostly confused as to what aligned means. I thought that by misaligned you meant that field positions within a given tuple are misaligned, not that the initial values of a non-existing field is aligned with the first tuple on a grouping.
No problem, I'll fix it myself before merging in a moment!
Here for consistency. Previous version passed it.
Making each component monitor their own memory usage seems unnecessary when we have a process monitoring system (god) already. Furthermore, you rely on god to revive this AppServer after its failed, so it seems as though the proper fix is to just write a god config file with memory thresholds set in it and leave it to god.

Additionally, since god is built for this (and not our AppServer), it has termination conditions that are a lot smarter than this. The condition here is "kill this AppServer if it exceeds X memory one time, depending on a coin flip", which makes a lot less sense than what god would do, "kill this AppServer if it exceeds X memory 3/5 times in a row".
Space these imports properly.
Good point. In this case, it's really that indexing (by either rows or columns) doesn't preserve column groups. There are probably other cases where this is true. We need to decide how far to push support for grouped columns. I'll add an issue.
Without renaming the function we fix the whole issue -- also with announcements and categories. Now those will still be broken.
Change to ArchiveToGoogleStorage -- "Gs" is not self-explanatory.
I'm pretty sure the event listeners are registered in Sburb.initialize. There shouldn't be any race condition.
Does this default to verifying or not verifying the hostname?
This 'floppies' variant seems unused.
this is a bit strange, it will still pass non-fns.

`hasCallback = typeof callback is 'function'`?
I'll merge and fix this myself ;)
hmmm shouldn't this be implemented in StatusMessage, and then post could just do a no-op?
We will need to make this configurable.  The issue is that while this is the correct behavior (in terms of the assertion/check), there is existing code which is using duplicate node names already, and this fix will break that code.
find_each
'path' typo
Yeah, I think so. I saw a few other uses of 'article' around the place as well.

On a more general but related note, I noticed a few things like

```
article figure figcaption cite {
    font-size: 10px;
    font-style: normal;
    text-transform: uppercase;
}
```

These bug me because, there's every chance you'll one day want to re-use those text properties on other arbitrary elements, not just cites in that context. And the right to left selector parse will be slow in some browsers. I'd be more at ease with '.small-uppercase' than these. :)
Add defgeneric from process-module.lisp
it does appear that a removal method was unwanted since the weak ref is used..  I'm not sure forcing a GC would be much better, and we still couldn't remove the conditional statement I think.
As we decided during the weekly meeting today, we can safely assume we don't add any check if a module is suppose to work at least for `Firefox`, `Fennec` and `Thunderbird`.
document this function
`resolve` returns strings, shouldn't we free them too?
jwcrypto 0.2.0 is published, we can update to that.
Good news everyone! I think that `object_responds_to?` is unused as of b599a42e68738c1b7e04e855735cd368283775f6.

There's no reference to it in any of the rspec repos currently (except its definition):

```
[rspec-dev] grep -R object_responds_to\? repos
repos/rspec-mocks/lib/rspec/mocks/method_double.rb:      def object_responds_to?(method_name)
```

I'm going to bias towards removing it unless someone lets me know that I'm missing something.
oh, dont throw strings: http://www.devthought.com/2011/12/22/a-string-is-not-an-error/
Ah sorry - thought it was extending something :)
@Maks3w which package is not official?
Yeah, that's a very good point. I wasn't incredibly happy with how this one turned out which is why I wanted the review. I'll roll some of that in.
why this file is here?
Oh yeah, also, coding standards.
should be fixed in latest commit
Can you add descriptions what these are? the variable names are very indescriptive.
`new` removes duplicated keys, can we use `from_enum` instead? Thanks!
We could probably have `Enum.to_list`
Stijn De Weirdt
That's just because building is performed in install dir.

But I agree, it might make more sense to build in make, and leave make_install empty. Will change.
I can create test-cases easier this way. For example a gzip with only a GCC dependency.
LOL, my first approach was like that =D But you edited comment and suggested something like this. Changed anyway =)
s/an/the/
No, that's exactly what I did.
See the above two comments; trailing slash shouldn't be needed.
Dunno, its consistent with the indentation in the rest of the file. I'd like to keep it as is or change the entire file itself. For a nit, probably not worth it.
I think this stuff was used with the event hooks that supported "canplayall"
I've never used arrays with `eq`, but just trying it now the SQL produces this error:

````ruby
ActiveRecord::StatementInvalid Exception: SQLite3::SQLException: near ",": syntax error: SELECT  DISTINCT "translated_articles".id FROM "translated_articles" LEFT OUTER JOIN "translated_article_translations" ON "translated_article_translations"."translated_article_id" = "translated_articles"."id" WHERE "translated_article_translations"."locale" = 'en', 'en' AND "translated_article_translations"."slug" = 'a-title' LIMIT 1
````
This is unsound if offset>offset', as there will be uninitialized RGB frames between offset' and offset. A minimum would be better.
не працює. KnpZendCacheBundle вимагає вручну прописані пакети zendframework/zend-cache версії 2.0.0beta1 та zendframework/zend-filter 2.0.0beta1

без них помилка:
Your requirements could not be solved to an installable set of packages.

        Problems:
                - Problem caused by:
                        - Installation of package "knplabs/knp-zend-cache-bundle" with constraint [] was requested. Satisfiable by packages [knplabs/knp-zend-cache-bundle-9999999-dev].
                        - Package "knplabs/knp-zend-cache-bundle-9999999-dev" contains the rule knplabs/knp-zend-cache-bundle requires zendframework/zend-cache (== 2.0.0.0-beta1). No package satisfies this dependency.

наскільки я зрозумів, на KnpZendCacheBundle багато чого зав"язано, тому я не виключав цей бандл
If you are fine with a potential mismatch, let's just leave the code as is. It is probably not worth the extra complexity.
section.textContent will throw an error when section is null - why did you move it out of the test for section?
According to your chart, ```C1``` is mapped to Term for DICT and ENCYC, which suggests this would be an entry. I'm not sure I'd count on many applications to export this correctly, though.
That works (with a colon at the end), unless we want to make it clearer that this isn't a deficiency on Zotero's part.
@adrn: I see this as a useful functionality as well, but there have been long discussions on that on `NDData` and `Spectrum1D`. I would recommend - for now - raising an exception. In addition,  we write an email out to the community how to (in general - be it `quantities` or `NDData` like objects) solve this issue for arithmetic. As I see it - there's four options (left, right, center (si) and not at all). Who wants to write that email (It's very wide ranging - I think it should be to astropy and not dev)? In all cases we need to make clear that the result still knows about the unit and is not a float (that might be confusing to people who have not seen the units package).
This may have a big impact on other parts of the Sip-Creator. Have you thought it all through?
это на итальянском?
Hi David,

There are only few combinations of number-value; the purpose of this line
is just to know the set of class values for each classifier (since it is
possible to process more than one classifier at a time), most of
classifiers are binary and the remaining have a small set of class values.
Thus at the end the quantity of combinations of number-value really depends
on how many classifiers are processed at a time, and using FEW give us a
max number of 10,000 that I guess is enough i.e. it can process up to 5000
binary classifiers.

Regards
Victor



On Fri, Jun 1, 2012 at 6:14 PM, David Bayliss <
reply@reply.github.com
> wrote:

> > @@ -45,18 +44,61 @@ EXPORT Compare(DATASET(Types.DiscreteField)
> Dep,DATASET(l_result) Computed) := M
> >       SHARED J := JOIN(Computed,Dep,LEFT.id=RIGHT.id AND
> LEFT.number=RIGHT.number,notediff(LEFT,RIGHT));
> >       // Shows which classes were modeled as which classes
> >       EXPORT Raw :=
> TABLE(J,{classifier,c_actual,c_modeled,score,score_delta,sole_result,Cnt :=
> COUNT(GROUP)},classifier,c_actual,c_modeled,score,score_delta,sole_result,MERGE);
> > -     EXPORT CrossAssignments :=
> TABLE(J,{classifier,c_actual,c_modeled,Cnt :=
> COUNT(GROUP)},classifier,c_actual,c_modeled,FEW);
> > -     EXPORT PrecisionByClass := TABLE(J,{classifier,c_actual, Precision
> := AVE(GROUP,IF(c_actual=c_modeled,100,0))},classifier,c_actual,FEW);
> > -     EXPORT HeadLine := TABLE(J,{classifier, Precision :=
> AVE(GROUP,IF(c_actual=c_modeled,100,0))},classifier,FEW);
> > +     // Building the Confusion Matrix
> > +     SHARED ConfMatrix_Rec := RECORD
> > +             Types.t_FieldNumber classifier; // The classifier in
> question (value of 'number' on outcome data)
> > +             Types.t_Discrete c_actual;                      // The
> value of c provided
> > +             Types.t_Discrete c_modeled;                     // The
> value produced by the classifier
> > +             Types.t_FieldNumber Cnt:=0;                     // Number
> of occurences
> > +     END;
> > +     SHARED class_cnt := TABLE(Dep,{classifier:= number, c_actual:=
> value, Cnt:= COUNT(GROUP)},number, value); // Looking for class values
>
> How many combinations of number/value are we expecting - previously we
> were using ,FEW which is much faster (if it works). Failing that using a
> ,MERGE will prevent skew issues
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hpcc-systems/ecl-ml/pull/120/files#r918004
>
See https://github.com/openlabs/poweremail/commit/a8569d351a5c4fc4e2482aa3b0eec4149ee3df37#L0L952
You commented on the wrong commit but point still valid - will fix.
Should jake still be a dependency?
If you mean that we can only set a role for a user/group on a piece of content, discussion, etc. and not on a user/group, then the validator should be checking that resourceUuid isn't a user either.
I think _canManage only needs to be invoked if the content is private. Can this logic be revised to avoid the unnecessary permission check?

It may be very common that _canManage will be a "negative" check, especially if most of the content is public. As we know, currently those checks are significantly more time-consuming v.s. "positive" checks so we should avoid them if possible.
These messages are only used by ``ScalaJdiEventDispatcher`` (internally), maybe it's worth encapsulating them, i.e.,

```scala
object ScalaJdiEventDispatcher {
  private case class SetActorFor(actor: Actor, request: EventRequest)
  private case class UnsetActorFor(request: EventRequest)
}
```

And then you would import them inside ``ScalaJdiEventDispatcher`` (private members in an object are always *visible* for the companion class).
On Mon, Jul 9, 2012 at 1:38 PM, Iulian Dragos
<reply@reply.github.com>
wrote:
>> +    }
>> +
>> +  private def shownSettings(settings: Settings, filter: Settings#Setting => Boolean): Seq[(Settings#Setting, Option[String])] =
>> +    for (
>> +      box <- IDESettings.shownSettings(settings);
>> +      setting <- box.userSettings; if filter(setting)
>> +    ) yield {
>> +      val value = Trim(storage.getString(SettingConverterUtil.convertNameToProperty(setting.name)))
>> +      (setting, value)
>> +    }
>> +
>> + def scalacArguments: Seq[String] = {
>> +   import ScalaPlugin.{defaultScalaSettings => settings}
>> +   val encArgs = encoding.toSeq flatMap (Seq("-encoding", _))
>> +   val shownArgs = shownSettings(settings, _ => true) flatMap {
>> +     case (setting, Some(value)) if value.equalsIgnoreCase("true") => Some(setting.name)
>
> What if there are non-boolean settings?
Does eclipse allow for non-boolean to be configured in its UI? If so,
I should change it indeed.
>
> Maybe it's better to use `Setting.unparse`, as it's done in the [sbtintegration.ScalaSbtCompiler](https://github.com/scala-ide/scala-ide/blob/master/org.scala-ide.sdt.core/src/scala/tools/eclipse/buildmanager/sbtintegration/ScalaSbtCompiler.scala)
>
> Also, if this is Sbt specific, consider moving the code in the `sbtintegration` package, maybe where the previous links point to.
It's not sbt-specific, it produces command-line args.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/scala-ide/scala-ide/pull/144/files#r1121040
An [Exit](http://www.scala-lang.org/api/current/index.html#scala.actors.Exit) already exist in the standard library, maybe better to use that one?
But then how would we use UrlMatcher?  I suppose our subclass, which we'll need anyway, would have to implement Request Matcher and subcall match() from matchRequest().  I'm OK with that if you are, I suppose.
Vestigial.  Will remove.
Well yeah, there's some work to be done in Exception handling. So far, I just wanted to support PrettyPrint and output the serialization into a Writer. But you're right, I'll keep working on this branch.
I think xhaggi reformatted the whole lot (probably automated tool). Just stick to whatever he's using which I think is tabs, the main thing is that its consistent.
If Pig isn't working in local mode, we shouldn't keep dead code in the master branch. Definitely hang on to it for the future but clear it out of this version.
Since you have the "fix" already, this argument is pointless, but ... the code doesn't break, it just doesn't (or didn't) support protocol-less URIs. No example was given using such an URL, so that was fine without specifically mentioning and/or bolding it.
Хм, и правда, буду знать. Я бы и не догадался попробовать, так как ни в C, ни в C++, такой фокус точно бы не прошел. Спасибо за совет.
В любом случае, для вас нужно только сделать решение поставленной задачи, поиск хорошего алгоритма - это "бонусная" задача. Ей можете заморочиться, если действительно хочется придать "красоты" этой задаче.
Примите во внимание ещё тот факт, что замена ячейки на саму себя не будет считаться заменой вовсе.
Это все понятно, просто вы сравниваете указатели, а не сами объекты. Так что мой вопрос был о том, действительно ли это работает.

Не знал, что в Objective C перегружены операторы сравнения для (NSString *). Это странно.
Или дело не в операторе, а в оптимизации хранения неизменяемых строк.
If you make a change locally, and then re-push your change, it'll update this pull request! :boom: :sparkles:
Ge snapt mijn punt ondertussen wel al wellicht :p
叫*seed*会不会好一点？
How can I do that? Tryed 

```php
<?php

 ->arrayNode('path_patterns')
    ->defaultValue(array())
    ->children()
// ...
```
but this is not works
one line space between methods, two between functions.
shouldn't it be uV^2, etc., as these are variances?
please use JSLint to fix the style of the code.
remove this if statement and use assertion instead. ```test.strictEqual(expectedValue, actualValue, "<message>")```
It does not, removed and amended
would be nice to add a sanity check that there is patient with a matching person id.

Also ensure that if you add or edit anything that is defined on person e.g edit or add a name, it gets saved to the db along with the patient, this will ensure that the eviction of the person object in the dao didn't result in lose of changes in properties directly defined on person
I originally said "+1 to both comments", but thinking about it more, I think we should leave it as <T extends BaseOpenmrsData>. Because, imagine there's some implementation of an OpenmrsData that does _not_ extend BaseOpenmrsData. Would we really expect our base implementation of the DAO to work on it? It won't necessarily have correctly-named properties...
Likely. Did most of my edit in vim. First time tossing this together (and doing collab on github), so happy to re-edit and re-submit.
typo: Its should It's (It is)
typo: "purpose build" should be "purpose-built"
This is a nice setting to have, but I'm always trying to keep the number of settings as few as possible. I think that with the new `ACCOUNT_PROFILE_FORM` (which is great), we can easily just subclass the profile form, and define our own key order in that. It's a couple extra steps, but it means the field order setting isn't strictly required.
If we're just going to override a parameter, why not just do it in `app/config/config.yml`?

Also, we should note that if you want to do something beyond just overriding a parameter - like adding a method call - that it must be done as a compiler pass. We already have some docs on the compiler pass, so we can just link to it :).
I prefer to put a file comment before this line, something like:

    // src/Acme/TestBundle/Tests/Form/Type/TestedTypeTest.php
I think I fixed this and other one way sends to self.  Could you explain why this is important?
Yeah where condition must be there. My mistake. 

Sent from iPhone

On 09-Mar-2012, at 3:05 AM, Ryan Bigg<reply@reply.github.com> wrote:

>>         def associate_user
>> -          return unless current_user and current_order
>> -          current_order.associate_user!(current_user)
>> +          return unless current_user and current_order and not current_user == current_order.user
>> +          # get the last incompleted order
>> +          order = current_user.orders(:order => "updated_at").select { |item| item.completed_at.nil? }.last
> 
> Why couldn't you use `current_user.orders.where("completed_at IS NOT NULL").order("updated_at").last` here?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/spree/spree/pull/1231/files#r536069
@tuncer I will follow your advise to re-generate a pull request later. Thank you.
Nop, we're still thinking about it, and it seems that finally depending on twig is not a good idea as it will create a big phar at the end.
Probably want to change the case of this to OpenURL or OpenUrl.
Sounds good! Updating PR
using the "\" is windows specific, not sure if we should change this
That's not entirely correct. The drone will stop doing flips at 30% and refuse takeoff  below 20%.
It is not good practice to use `defn` do define helper functions, as the definitions will end up in the top level. Using `let` is better, though a bit ugly some times.
Yeah cool! I like it as well! So the overloaded method is not exposed on the interface then?
in which situation can I receive an update and have the pending CH not null?
Do you intend to rename the method? Otherwise just rename the todo :)
Why do you need to add the commit ?
Manik, yes change is there as well as I wanted to get as much feedback for this one and the other one. I'll adjust infinispan/infinispan#1322 after this one is integrated. There is also a big warning on 1322 not to integrate it yet.
These ternary expressions are unnecessary, since invoking methods against `nil` returns `NULL`/`nil`/`0`, which is what you're doing here anyways.
The `with`s here and below don't feel Objective-C-like. How about `-diffOldTree:newTree:inRepository:options:`?
I think libgit2 is wrong. It should be `const char *` and I see that it is not currently.
Made the change and pushed it to this pull request.
What do you mean with "occured"?
Variables should usually be declared at the start of a function, rather than inside of a block (JS has no concept of block scope, only function scope).
Whops, thanks
Don't sweat it bro. I should have done a test installation during review. Adding it to the checklist.
Maybe add that placeholders should not be pre-escaped for a particular target format.
These preconditions are added back later in a future pull request, slightly changed.
Instead of creating a variable we can just return true or false when appropriate.
please add curly braces around first for loop and add an empty line after the for loops end
增加了异常处理过滤器。
Same deal here; breaks Pro.
"Cache pool configuration" -> "The cache pool configuration"
I don't understand why cache pools can't be resized -- the only thing it seems to me you can't change is the name?  I ask because it seems to me you'd have to add code to keep this from working?
Intentional, but intended to be removed as MySQL support is introduced (the reversal of order of implementation left this incomplete).
We're also not doing this. Can you give an example where a word would be preceded by TeX markup?
Maybe make these specific to PostgreSQL? Looks like we had the check in migration 0005, but forgot in 0006:

https://github.com/caktus/django-pagelets/blob/develop/pagelets/migrations/0005_add_sequences.py

Might not make a difference (not sure how sequences differ between backends), but at least it'll be labeled for Postgres.
do you actually need the ToList here?
System.Linq.Enumerable is extension methods for IEnumerable<T>, and Dictionary<K, V> implements IEnumerable<KeyValuePair<K, V>>
Just a rename happened on this to `phpunit.xml` (bypassing git for renaming is bad, and I feel bad) nothing new has happened. Do you have a specific setup that requires the .dist ? PHPUnit first looks for `phpunit.xml` and then for `phpunit.xml.dist` thats why I changed it.
I think the message is inaccurate, the message from the lang files is the following:

    JController::getName(): Cannot get or parse class name.
just https://github.com/liip/LiipMonitorBundle/pull/15/files#L0R151 vs. https://github.com/liip/LiipMonitorBundle/pull/15/files#L0R163 .. not a big deal.
Sinceramente já nem me lembro xD.
no empty lines after ifs pls
Better, but a bit too much now. :-0

```
var post = $('<li/>').addClass('post').data('post', self).append(
    post_options
).append(
    $('<span/>').addClass('author').text(self.author.username)
).append(
    $('<p/>').addClass('content').text(self.message)
).append(
    $('<span/>').addClass('date').text(formatPostDate(self.created_time))
);
```
But you will not have dict's anymore in the database. Uahaha!

So please, simplify this.
Then add id field? :-) Simple.

So also URL is not good, because it is only an index in the list. If we will support deleting comments some day, this will not work anymore.

But here I didn't like that you was doing URL manually. So you hard-coded the URL. What if REST interface would not be under `/api/` anymore? This code would break!
Constant string, so use `'` around "other".

Use `!=` instead of `is not 'other'`.
This happens during deserialization when using DataContractSerializer. DataContractSerializer creates an unintialized instance - changedSubject and changingSubject are both null. Then when a property is deserialized and set and it uses RaiseAndSetIfChanged we run into the issue. You can repro it easily using the following code:

    class Program
    {
        [DataContract]
        public class Model : ReactiveUI.ReactiveObject
        {
            [IgnoreDataMember]
            private int _MyProperty;
            [DataMember]
            public int MyProperty
            {
                get { return _MyProperty; }
                set { this.RaiseAndSetIfChanged(t => t.MyProperty, value); }
            }
        }

        static void Main(string[] args)
        {
            var model = new Model() { MyProperty = 10 };
            var serializer = new DataContractSerializer(typeof(Model));
            using (var stream = new MemoryStream())
            {
                serializer.WriteObject(stream, model);
                stream.Position = 0;
                var deserialized = (Model)serializer.ReadObject(stream);
            }
        }
    }
An example is as follows:

		#ifdef DESKTOP
		uniform mat4 MatrixTransform;
		#else
		uniform highp mat4 MatrixTransform;
		#endif
		
		#ifdef DESKTOP
		attribute vec4 Position;
		attribute vec4 Color;
		attribute vec2 TextureCoordinate;
		
		varying vec4 Diffuse;
		varying vec2 TexCoord;
		
		#else
		attribute highp vec4 Position;
		attribute lowp vec4 Color;
		attribute mediump vec2 TextureCoordinate;
		
		varying lowp vec4 Diffuse;
		varying mediump vec2 TexCoord;
		#endif
I would use assertions only to check for internal errors.
since _newsLog is a std::map you can just insert with ```_newsLog[id] = log;```
Did you mean to remove this line?  From the code change above, it should probably just be updated to expect 'finder'
small optimization :)
`Copyright 2010-2012` Should be `Copyright 2012`.
You didn't make this file in 2010, did you?
Is this for debugging? If so, call "debug" instead of "warn".
Yes i think we should avoid a sinter call in the case of we have only one key, redis is faster as returning the wanted set instead of doing a sinter (seeing the time complexity O(N*M), N is the cardinality, M the number of keys, here 1, but O(N*1) is bigger than O(1) :)
shouldn't we explicitely call a function here?
Should it be 0deg here? Prefixed starts from east, so -90deg = north, unprefixed here starts north, so 180deg = south?
Can you change this argument to take a shared ptr  as well? It would then be clear at call site who takes ownership of the memory.
yule_walker needs to stay in the module reference
Looks like there's a fair amount of cleanup/DRYing that can be done in this file regarding the paths where we get and put stuff.
Do we need that log here?
The prefix is only added when it's not there. 
'''
if (NullChecker.isNotNullish(sTargetHomeCommunityId)) {
            if (!(sTargetHomeCommunityId.startsWith("urn:oid:"))) {
                sTargetHomeCommunityId = "urn:oid:" + sTargetHomeCommunityId;
            }
            request.getAdhocQuery().setHome(sTargetHomeCommunityId);
        }
        return request;
I moved the css left calculation down here after the element was appended to the body as it had no dimensions until it was added to the document tree.
Right.  You can't put them inside the class definition.  Then you would probably want them to be `static const`, in which case you would need to initialize them in the .cpp file.
I just converted to multi-line.
I like the notification at the top of the file.  +!
Throwing an exception now in the code that replaces this this: a JerseyClient filter in BrooklynCommand
there are 2 spaces indentation instead of 4
`is_numeric` will return true if it is a numeric string, but we want all strings to stay strings.  Putting it first ensures this.  `is_numeric` could also be replaced with something else, but I wasn't sure at the time which functions would cover everything. (there is no is_numeric that only returns true for numeric types).
I figured out and added it.
We're using pegdown, so you need to adjust your links to be `[url](url)` for no good reason.
Why was the killing process changed from SIGKILL to INT?

if the process is hung we might not get out of it with such a soft method as INT.
Workaround for BN_bn2mpi reading/writing out of bounds

When OpenSSL's BN_bn2mpi is passed a buffer of size 4, valgrind
reports reading/writing one byte past it. I am unable to find
evidence of this behaviour in BN_bn2mpi's source code, so it may
be a spurious warning. However, this change is harmless, as only
the bignum with value 0 results in an mpi serialization of size 4.
Yes i can.
Awesome change.
Cred ca e mai OK sa apelam de aici celalalt constructor cu valorile 0.
Changed to _INITIAL_LABEL_SEQUENCE_NUMBER
Для доступа к приватным методам, или не надо?
По заданию надо экранировать символы. Чтобы в результате получилась строка, которую можно вставить в java-код.
{ не переносится
It's supposed to be .* = % 

.* is regular expression
% is JDBC syntax for the same thing.

* is only used in file name globbing which is not what was intended here.
Isn't it JBoss Data Grid? Why the word Server?
Do we need this line?
~~#This is done to mimic a realistic workflow~~
add whitespaces between operators
It was two days since I was able to come up with the PR this time (it was the first time and I also had to discuss it with Lars). Normaly It takes only a few minutes between the sync and the PR creation. So it should not be a problem normally. But its not so a good idea to write some fast changing string into these paces, because translators than have to re-translate this string every time it changes. And that goes for ALL languages at all change.

This would also be a problem if we did not use Transifex at all, because the addon.xml file a translator working is never the freshest one.

So my suggestion is to use a not so frequently changing text in the addon.xml file summary, description, disclaimer fields.
Is it possible to use a string and more-readable date format here?
> I think n_jobs > 1 is avoided in examples for windows (to be confirmed)

confirmed
You're very welcome. Well it got more superficial in the end as I got a bit tired ;)
Btw I'm leaving Vienna in ~2 Weeks. Wanna go for a drink before?

Oh and if you like I'd appreciate if you looked at the 5 lines  of code I deleted here: #707 ;)
nk is filled with 1 or I am missing something
>
> If there is no *trick* that allows you to get the transformed value while
> fitting for free, there is no use in defining this implementation. The
> naive fit_transform as sequence of self.fit and self.transform is already
> defined in TransformerMixin.
>
>
@vene Somehow if I don't define it, I get a failed test in
sklearn.tests.test_common.test_transformers when I make. Basically the
default one outputs the wrong shape.


> —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/1200/files#r1760595>.
>
>
is there no non-anonymous role that is always present that we could use as default instead? to me its the wrong way round to be permissive.
I think such helper function itself does not save anything. The important problem is where resolveProperties should be called.
Pass the size in the constructor.
As I've said in another comment, getting actually parses the JSON string, so it's just an optimisation parsing it just once, instead for each getter. I'm not sure what, if any, the performance impact would be to do it the normal way.
Should either delete or reinstate.  If dictionaries were split from datasets it should be reinstated.  I'll invetigate.
I disagree. But let's focus on the views.py first.
I had to make a phonegap plugin just to get one java variable...I'm unhappy about it too
Tämä samanlainen kuiten MAIN_Start, mutta tarkoitettu tyhjentämään muistia pelin jälkeen tai mahdollisen kriittisen virhetilanteen jälkeen. Kutsuu pelien XXXX_End() funktiota, ei välttämättä tarvitsisi jos mitään tyhjennettävää ei ole. Tämän jälkeen ajaa kaikki normaalit tyhjennysloitsut.
is this data on all env? you should use the shared data so that it will run on multiple env. this is going to only run successfully against the env these creds are good for.
I'm trying that now and don't get an error or any noticeable problems, but
then again, entity_value also didn't cause an error :)



On Tue, May 1, 2012 at 11:23 AM, Simon Charette <
reply@reply.github.com
> wrote:

> > @@ -271,7 +271,7 @@ def _matches_filters(self, entity, filters):
> >                  entity_value = entity[field.column]
> >
> >                  if entity_value is None:
> > -                    if isinstance(value, (datetime.datetime,
> datetime.date,
> > +                    if isinstance(entity_value, (datetime.datetime,
> datetime.date,
>
> Can you try using `lookup_value` instead of `value` or `entity_value`?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/django-nonrel/djangotoolbox/pull/27/files#r756361
>
It gets converted to a string at some point. To see why look at https://github.com/django-nonrel/djangotoolbox/pull/24#issuecomment-4708787

I know, it's not idea, but it's the cleanest solution I've found.
But this works because these are .jsp files. which we'll need to fix if we change to Ruby

<%-- <td>
    {{#if waitlist}}
       Y
    {{/if}}
</td> --%>
are the @ really needed here? Can this be implemented without the @?
Has this been tested in Linux or Mac? Why did you decide to remove the OS check?
Shouldn't be in PKGCONFIG
More idiomatic ruby would be:
```ruby
datastore['UNIT_ID_FROM'].upto(datastore['UNIT_ID_TO']) do |counter|
  ...
end
```
It's svn related and can be deleted. Not needed anymore.
Good. once we get started enforcing it we should pull in some of the most common violations against this guideline into this document. Also whoever is responsible for code reviews needs to be really well versed in the apple recommendations so we have consistent format.
see above. valueerror will be raised (eventually, when this change lands!) when destructuring can't take care of more than one value.

Lookups that should be cached are always one keyword to one value. We don't care about any lookups that have multiple lookup keys.
I think `true` and `false` are fine as CSS classes. You could make it a little more explicit:

```
<div class="sub_records {{#if showExpanded}}showExpanded{{/if}}">
...
.sub_records:not(.showExpanded) { display: none; }
```
I couldn't decide whether or not to force it to open in a new window.  I noticed other off-site links did not open in a new window by default so I followed that behavior.
Still curious about the above question:

> When will the camera not have a frustum?
We can't really do this, right?  What if they are rendering to a separate framebuffer.  Perhaps it should be:
```
command.framebuffer = defaultValue(command.framebuffer, fb);
```
And we shouldn't modify their `command`.
Same comment for `openlayers.html` below.
I believe this initial `near < far` really means if there are any non-culled commands, right?  Perhaps we should make it a variable right above the loop so we can name it something sensible; otherwise, its meaning is non-obvious to most folks (even me if I am wrong about this).  Given that this is the heart-and-soul of the rendering engine, this code needs to be beautiful.  In a crazy edge case, I suppose the test should be `<=`, but perhaps `near !== Number.MAX_VALUE` is most precise.
Unlike every other file in Cesium, this file is *not* an AMD module, instead, it's a plain JS script intended to be used when creating a `Worker`.  It's actually a special Worker script, because it has the magical ability to transform itself into any other worker, specified as an AMD module.  In order to load the AMD module containing the real worker, and more importantly, its dependencies, we need an AMD loader, but we can't assume anything about the location of the bootstrapper script in order to find one, and Workers can't access anything about their parent browser.  So, we include RequireJS inline so that we always have it available, in all of the cases we need to handle: 

* separate source files in development loaded in the browser using either RequireJS or the Dojo loader
* Dojo layers pre-built by the Dojo build process
* no AMD loader in the browser, worker modules are combined into one file by the combine step

There are smaller AMD loaders, but RequireJS works correctly from within a Web Worker, and I already know how it works.  

TL;DR: it's complicated, and this is the best solution I could find.
You'll want to change this to `var PolylineCollection = function() {`

 See the new [Documentation wiki](https://github.com/AnalyticalGraphicsInc/cesium/wiki/Documentation-Best-Practices) to see some of the changes that went into master while this branch was being developed.
I am talking about auths in Payflow and data captured by Litle. Therefore, we don't need to add this test.
What's the meaning of index 0 here? Could we add either a symbolic constant, e.g. OutputChannel[Channels.Stdout] or a wrapping property (writer.StdOutChannel) that gives us the right index?
This implementation would lead to memory leaks and possible crashes. Please take a look at how we do [remove method on view ](https://github.com/appcelerator/titanium_mobile/blob/master/iphone/Classes/TiViewProxy.m#L191-251) and maybe optimize that code for `removeAllChildren` method.
Ah, didn't quite pick up that this is an example...it really needs to be changed then. IMO, writing bad code for the sake of consistency isn't a worthy tradeoff, especially since it might encourage people to write bad code in other places too.
Currently only supported on the Ti.XML.Element in IOS
Boolean on Mobile Web, not number.
You probably don't need the spaces inside the pre tag.
Minor nit, but don't wrap lines at 80 characters.  We have wide displays now :)  Look at the Hadoop source for an example of why not do do this...
Well, other languages have this function also in their path module (like Python: os.path.expandvars) and you normally just need it when dealing with paths. I see this function as an addition to expandTilde.
In basic, we should initialize `result` by default image. Binary move() destruct `result` in first, and user-defined destructor will run over `result`.
To fi bug 8055, we should fill `result` with valid memory image.
But, if T doesn't have elaborate destructor, we can avoid initializing of result`. I'll add this.
s/non existent/non-existent/