labofsahil/github-event-dataset-2012 huggingface dataset:
shoudln't you be validating the values too ?
Can you please also add a test for reconnecting after all subscriptions cease and then a new one is added?
这个命名有点怪 =_=
pardon my nitpicking but I would put the method arg before iir_params in the function signature since iir_params is only relevant if method is not fft.
if npad is independant of self you should put the default in the function signature
Or not. I just looked it up.
you should describe their goal (figuring whether the cache is still fresh by checking the resources used to build it) instead of just saying ``for later reference``
you can remove this line now ;)
正常化 -> 规范化
频率都改成速率吧
Oh, good catch. I like namespacing the constraints, the same way we did `Clearance::PasswordStrategies`.
Doesn't look like awesome_print is used. Can be removed?
What about $stopCommand to be more explicit?

And what about adding another property called $command which stores the command name?
ok, lo ponemos en el sprint 10

On Thu, Aug 2, 2012 at 4:36 PM, Alpha
<reply@reply.github.com>
wrote:
>> @@ -170,6 +170,7 @@
>>      <Content Include="Content\themes\base\jquery.ui.theme.css" />
>>      <Content Include="Content\website_green_logo.png" />
>>      <Content Include="Global.asax" />
>> +    <Content Include="RavenNotReachable.htm" />
>
> HTTP 500
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CommonJobs/CommonJobs/pull/146/files#r1296804
a little optimization here. Any columns that are already the broadest type (ASCIIString, presumably) need not have their values checked again.
jak wyżej bez 1
Finish your work here. I'll see this later.
Please remove code if unused
Ok, deal, so it is createDistributedTaskBuilder(Callable callable);
So ctx.isOriginLocal() doesn't work after all?
Made it easier to search for that output in the logs. I'll change to LC.
I can't do this because i getting 
`RuntimeException' with message 'Accessed request service outside of request scope. Try moving that call to a before handler or controller.`.

Do you have other solution for this ?
they don't interfere. setUp is called before each test, so each test receives a new filter
Oh wow. Yes, good eye. Well spotted. `np.repeat` is another option.
I'll add these.  I've never been a huge fan of that, but it's sometimes required, and of course is used a lot throughout the codebase.  We can remove them here if we clean that style up globally later.
Super-mega nit. Capitalize the 'c' s/click/Click/
really? the last listed message is the last received text message? My mental model would more think the last received text message would be the first listed
This is just word to word translation. I means when I translate something, I understand what I read, then I just re explain the same message with the other language.
Writting in french "Différents préprocessurs CSS ont de différentes caractéristiques" is just weird. "Les différents pré-processeurs CSS existant offrent des fonctionnalités et des syntaxes différentes" seems a lot better.
"Vos conventions doivent être étendues" : étendues should be "adapté"
про стрелочки. spawnDb - не особо в глаза бросается.
I would add:
@include http://www.pouet.net/*
+ 1 about using it with a space: `$b = (bool) $a;`
Can we bundle this require with the one above?
While in the area, can you remove the unnecessary variable `data_url`?
@bp323 I'll change it, I just thought we decided to only change the lines that really needed to change.
Why is there no space between the "Exception" and the "{"?
Oh sorry, mea culpa! Naomi fixed this in https://github.com/silverstripe/sapphire/pull/450
WebKit doesn't support fractional pixels so this will round down to `15px`.
These are changes to correct column-store recno handling for implicit records.
How about making the return type a bit more specific; maybe use ``ExplodeValidator|InArrayValidator``?
Is it possible to add a ``@param`` annotation for parameter ``$value`` which defines a more specific range of types; something like ``string|array``, or ``null|MyObject``? Please also check line 61.
Are you sure that the property "setPluralRule" exists?
Are you sure you want to override the parameter ``$predicate``? This often makes code less readable. I also found this on lines 253, 255, and 258.
异常信息放到info里撒。
Actually, perhaps I should just eliminate any return values from this function, as they're not often used.
That looks like a dog butt.
This is a really weird place to put simplecov. Why not at the top of your spec_helper?

Just added simplecov to the Realty app, here's the pull request, if you want to see how it was done there. https://github.com/verticalbrands/realty/pull/91/files
Könntest du noch dokumentieren wie das Model genau funktioniert? Ich hab gesehen, dass man da ein einzelnes Bier übergibt? Wird dann für jenes Bier eine Bewertung berechnet? Wie wird genau die passendste Bewertung aus allen möglichen Bieren gefunden?
man könnte die traversierung vielleicht so "injecten" damit man zu Usern und Passwörtern kommt. So als Idee auf den Ersten Blick.
Natürlich gibts noch Hürden wie type Checks von den nodes...aber wir sollten da nochmals ein Auge draufwerfen.
no. why would i use a font that supported evil?
Since the issue here is the batch cookie operation I think this return change should be threated in another PR.
The ´$.cookie(name, value)´ returns the string representing the cookie and it is used in a few tests, I don't think that is a good idea either, the plugin should return undefined if there is no documented feature specifying what it should return...
Mmm, why read from source? Can't you rebuild the source code by printing the `Tree`?
The problem we were having is that whenever the cache would clear, it wouldn't be able to find the string template on the filesystem the first time it was ran. By not using the current environment, this doesn't happen. But we don't have the custom filters and functions.

I could get the filters and functions out of the current environment and put them in the new environment. Unless there is an option I can set on a cloned version of the current environment to actually load the string the first time it runs after a cache clear. I tried setCache(false), and a few more with out any luck.

Thoughts?
I'm not sure what you mean, but these are all actually included in GitHub flavored markdown (at least they should be). Here's the [original Markdown syntax][http://daringfireball.net/projects/markdown/syntax]

I think the biggest issue is that I used "tab" characters instead of 4 spaces.  I'll try to change it.

Github implements identation in two ways (needed to escape the character to get it to display correctly :P):

``` 
\``` 
This is my code block
Without identation
\``` 
```

```
    This is my code block
    With indentation
```
Both produce:

    This is my code block
    With(out) indentation.

If you insert that code block into the github flavored markdown it looks like the following:
_In a code block defined by \```:_

```
### This is an interesting case.

* Item 1
* Item 2
* Item 3
````
_In an indented code block:_

    ### This is an interesting case.

    * Item 1
    * Item 2
    * Item 3
I would change this to:
```
public function store($data, $id, $group = null, $wrkarounds = true)
```
just to be completely consistent with the parent.
This is a full sentence but the first line should not end in a period (since the sentence continues on the second line).
Ok, great, thanks.
@dbu I don't like seeing Travis failing because of a CS issue. It would make the build status weird. And simply displaying it without failing is useless as noody will go read successful builds.
However, you may want to start using @schmittjoh's [automated code review tool](http://jmsyst.com/blog/automated-code-reviews-for-PHP)
minor: You could also write

    task_result.status = case assignment[:AssignmentStatus]
    ...

(Both are fine with me, just an fyi in case you don't know.)
Improve word order. *vašem* -> *tvojem*. We are informal everywhere.
Because `horoscope[0]` is ugly.
Move this line before `author_link` line. So that it is in some normal order. (So you define delete and edit links, and then you combine that into options. Not that you have something completely else (author link) in-between.)
If I'm not mistaken it also has an unregistered user configured birthday to 1.1.1970
Fix spaces at the end of the line.
Delay is not a problem. Other users do not care or know that we are 10 seconds late (at worst) with information when user logged in. And for the user herself we will of course always show her as logged in. ;-) (Except when she will hide herself manually.)
`horoscopes?
These parameters are not relevant to a Classical PSHA calculation. If you run the old version of this config file through the engine, you'll get a bunch of `Ignoring parameter XXXX` warnings. I used this config file in some of my tests so I decided to clean up the file a bit.
since gen creation passes through symbols, this now works
Args section to doc?
What's up with the redirect to example.com? Is that just ignored? Seems like it could be brittle if we depend on that...
namespace einfügen
sollte man die nicht weg machen, oder sonst super(...) aufrufen?
Works for me. Merge it!


On Mar 28, 2012, at 9:01 AM, Scott Walker<reply@reply.github.com> wrote:

>> @@ -0,0 +1,40 @@
> 
> If you already have it in mind that you want to manage gemsets externally I can see that, but it actually does more than what you say.
> 
> 1. Creates  a gemset if you want one
> 2. Installs bundler & rails into that gemset
> 3. Runs rails new with the -T option specifically to prevent generation of test-unit garbage
> 4. Creates an .rvmrc for you to use the new gemset.
> 
> If you're concerned with having a pristine gem environment the wrapper helps make sure that happens.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/edgecase/pairing-config/pull/4/files#r614774
I think putting instance() on top makes it clearer that we are dealing with a singleton. pf::SwitchFactory is structured that way too.
Yes, let me page this back in.
Remove the blank line above this, indent the code back to the left margin all the way
thanks. Let us know one this is ready to go.
btw, you could skip map copy in the common case where no fields are set.
that's the current comment

>
// Unfold the string by replacing any breaking white space 
                // (meaning what splits the headers, which is \r\n) with a single 
                // space.
>
Yours :-) but it would require a code-search to rename the dependencies.I'm fine with that if you don't mind doing the work.
I don't see why this necessarily has to be a constant.
D'oh. Thanks for catching this.
Just wanted to make sure we weren't counting negative numbers as count data.
You mean --notest there, right?
nice, actually i think this should probably just return the elements and let the exercise dev insert them however rather than how this works, so 100% agreement
Open TODO
Would be faster to skip this and move the 2pi normalization out of the Info class and into the calling class, since that will scaleFlux again by `_flux`.  Better to just do one call of `result.scaleFlux(_flux_over_2pi)`.
In general, division is about 5-10x slower than multiplication.  So I have a habit of doing whatever I can to minimize divisions. 

However, I'm not actually sure if this is true of division by literals.  Probably not actually.  The compiler might actually convert the old version of this line into what I wrote automatically.  (If not, then it is slower.)
Comment: Eventually we should probably move this into C++ for speed, the list comprehension here will have quite large unnecessary overheads.
I copied that comment here to.  Seems worth having in both places.
Could keep the default yw=0, which means yw=xw.  So SBBox(0.28) would still mean a square pixel 0.28 arcsec on a side.
Explained on Discussion.
handy, fixed
Just to check...
Do you want to delete: `**/brooklyn*.log` (i.e. all brooklyn.log files in all sub-directories as well)?
Or will it always be in this basedir so no need to do it recursively?
I expect it's good as it is...
Little added value; no strong feelings. I like using ImmutableList when 
things aren't going to be modified but agree here it's probably pointless...

On 02/07/2012 10:59, Peter Veentjer wrote:
>> +            throw e.getCause();
>> +        } catch (Exception e) {
>> +            throw new IllegalStateException("execution failed, invocation error for ${summary}", e);
>> +        } finally {
>> +            mutexRelease.call();
>> +        }
>> +        if (log.isDebugEnabled()) {
>> +            log.debug("finished executing: {} - result code {}", summary, result);
>> +        }
>> +        if (!resultCodeCheck.apply(result))
>> +            throw new IllegalStateException("execution failed, invalid result ${result} for ${summary}");
>> +        return result;
>> +    }
>> +
>> +    public List<String> getLines() {
>> +        List<String> result = new LinkedList<String>();
> What would be the added value?
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/brooklyncentral/brooklyn/pull/182/files#r1083277
This I did not know. Much better - Thanks.
En effet c'rest with id. Pour ce truc je prefere les guillemets, j'tuilise l'italique pour parler du code plutot. Mais ca me rappeller que  je l'ai oublie sur d3TutoGraphContainer, je vais le rajouter!
You should be able to create this profile folder within the test and create the prefs.js with just the custom preferences. This would mean you don't need to check in a profile.
Why not add

```ruby
  a.approve   true
```

in the factory?
や、そのとおりです。そのとおりなんだけど、パラメーターとしてもらわなくても、設定したい UserGlossary の id だけあれば引いてこれる情報だから横着してただけなんで、そのように修正しますね
You don't return at the end of the function for an "int" function, but there are "return 0;" statements. Normally, you would just call this "void mongo..." and simply use "return;". Especially because the calling function never checks the return value anyway.
Hmm, good point. It would be nice to put these tests into the existing spec. RSpec rewards you if your spec files are named after the class they test. (Tools like Autospec and ZenTest work better, for example.)
Not sure what happened here.  These tests were all removed to the functional folder (as they use the test client).
As you wish. I usually don’t declare any variable when I only need them once.
So, I like this — it’s very clever. And `case` is clearer than `condp` — a definite win. Just one question — why use chars instead of string literals? I guess it’s a performance optimization — which makes sense, except this function is only called once. So it seems like readability is more important than this performance optimization.
please fix the indent for this too
Is it possible to make this just a bit more readeable (line break the array)? :)
Can you please fix the whitespace.
hossa, absolute URL
Linia aceasta probabil trebuia stearsa.
Ditto.  Move all this to the plugin I think.
I still don't like writing temporary files, and would rather see this in a fixture of some kind
I think the Pipeline should stay in the API package. In JS and iOS the _concrete_ Pipeline class is used to create/add Pipes.... Importing an impl class is a bit odd :) 

Also since we do not expect (at least to my current understanding) having 'multiple' Pipeline impls, I see no sense in abstracting this Pipeline contract out... I am not against including handy concrete classes in an API.
Если получили исключение, то хочется пробросить его дальше и завершить программу.
Какая-то длинная строка..
Is there a reason to disable animation?
updated to be consistent.
Does that work well on RLT?
there are some occasions when you have to do last minute loading of a class (circular references),
some occasions when you do it because the one method is the only place that uses it, the method is rarely used, and the file as a whole will load quicker without doing it globally
but if a class is used in multiple places in the file, it should definitely be imported at the top of the file.

in this case, it was my mistake to not tell you to put it at the top of the file
@willdurand what is the issue ?
Instead of `File.open(...) {}` use `FileUtils.touch 'tmp/always_restart.txt'` I find that more clear that you mean to create an empty file (or update the modification time if it exists already).
Maybe, "for running the examples" would be better.
Did you push it yet? I don't see it.
My point was less about having doctests (the coverage is great) than having some short simple example.
This seems a like a case where making up the input data is pretty artificial but
doing a transform on  ``[["spaghetti", "sirup"], ["kittler"]] `` is better than no example imho ;)
I've considered doing this but I would like to be able to handle the possibility of having a small portion of negative sample weights. Negative weights can affect the shapes of the distributions of course and can therefore be important if you want the tree to learn the correct shapes.
Is a function really needed for this: it is at most 2 lines of numpy. I tend to prefer inline such functionality rather than defining a vocabulary of simple functions that will be specific to your code.
Extra comma + PEP8 ?
Yes. post_status = 'auto-draft' is the default for new posts, which already have an ID and a post_author. I tested turning off and on force_guest_author as well.
On Sat, Sep 29, 2012 at 01:41:26PM -0700, Hara Kenji wrote:
> @adamdruppe , at least stop using hard tab. 

oh oops, I missed that one converting. My habit and editor both
use tab automatically so I type it that way then edit it later.
I think we should support it because bbndk supports it and it requires less work on our part since the bbdnk will validate it anyways. You can set the hostname using a hosts file or DNS. I'm not sure about changing the hostname on the device, but it's possible that there's one pre-defined that we don't know about.
In cases something went wrong default style will be used.
I think usually return unless customer; return unless order is cleaner. Not sure if that is considered 'the ruby way'
Dans le cas ou le document a été ajouté avec succès
Retourne la liste des objets (paniers et reportages) qui contiennent un document spécifique
`// The processor is not executed from a Jar`
> Use it on activities to set a custom layout
Crazy monkey patching. Turns out, making changes to UserChangeForm.base_fields['email'] get ignored. They need to happen after forms.ModelForm.__init__. So there it is!

Hacky, but actually fairly straightforward. Thank you Python object model!

I assign the 'username' data to the email.  This will trigger validation errors...
The size of this ArrayList is predictable. It's much cheaper in terms of object allocation to compute the final size and pass it to the constructor.
The comment I think.
NB: With these global settings, mean that the per component ulimits aren't strictly necessary.

It's my understanding that these global limits were put here, because of a problem with some distros' and pam.d configurations.
@pschwartz - can you elaborate ?
Some comment on the purpose of this transformer would be good
uh, maybe you get some inspiration e.g. here http://stackoverflow.com/questions/3758606/how-to-convert-byte-size-into-human-readable-format-in-java
Hmm, it looks like I was wrong.
1/ 
We need to do the chown *before* we check if cassandra is running. Otherwise it will run the ensure command before chown and try to start the server. The /etc/cassandra dir is still owned by root and not readable by the cassandra user.

2/
Because it's only `ensure => running` it won't try to restart the server if it is already running, not even if there is a config change.
This can be fixed by having subscribe  => File["cassandra.yaml", "cassandra-env.sh"], in the Service block
I think this resource will get executed each time `bin/apply.sh` is executed. As a result the `notify` suggests that the Cassandra service will get restarted each and every time `bin/apply.sh` is run, which may not be necessary.

Can we put the notify on the `cassandra.yaml` and `cassandra-env.sh` resources instead, and just ensure that the `Service['cassandra']` resource simply requires this exec. That way, I think the Cassandra service only gets restarted when it absolutely needs to (when a config changes)
Le code du jumbotron devrait aller dans un autre fichier.
What's the rational for this change?
You're right - I double checked and headers are allowed to have empty content. You learn something new every day.

However, these *specific* headers should never be empty. I don't want to go overboard here, this is a case where the service itself is sending malformed responses; if it wasn't needed to unblock a customer, I would prefer we do nothing and get the storage team to fix it rather than deal with situations where the server is giving us garbage without throwing some kind of exception.
Text ?
I hope my description for the language API call is correct. Kindly correct me if I am wrong. Thank you.
This `(vals (frequencies (map rank hand)))` seems like something you'd want to put in a helper function, since it is repeated multiple times in many functions.
Tässä voisi kutsua suoraan `str`-funktiota ilman `apply`n käyttämistä:

~~~clojure
(str (:name author) " (" (:birth-year author) " - " (:death-year author) ")"
~~~
Okay, so the change I'll be making is to email [settings.MANAGERS](https://docs.djangoproject.com/en/dev/ref/settings/#managers) by default from the mail functions, which is described as people who "should get broken-link notifications when SEND_BROKEN_LINK_EMAILS=True".
Does this code ever get executed?
I've been able to test successfully with shell sessions (bind and reverse payloads), but not with meterpreter sessions :\ 

Dhiru, where you able to run the post module with meterpreter sessions?
2000?
I personally prefer being explicit about `File.open` rather than just `open` since I'm always afraid someone's defined that somewhere else.
Isn't this a rather convoluted way of doing this?
Pure Python Version:                                                       74.9188730717
Python Version with moments functions in Cython:     1.10414004326
Cython Version:                                                                  1.04383802414
Why are you copying ranges? You're just iterating over it all over again below.
did I understood this correctly - read the file and assume it contains only:

    rvm 1.9.3

This is unfortunate assumption as there can be a lot more, better approach would be:

    ruby=`rvm rvmrc load . >/dev/null; rvm current`

but this goes to be equal to current ruby:

    set :rvm_ruby_string, ENV['GEM_HOME'].gsub(/.*\//,"")

which could be extracted to:

    set :rvm_ruby_string, :local

and `:local` => `ENV['GEM_HOME'].gsub(/.*\//,"")`
Fine then :)
Wrong level.
Perhaps I don't know the best practice, but why call `_isDirty` here when we access the property directly below to set it (a copy and paste from my code elsewhere?)?  It seems like an extra level of indirection that puts code in this function at different levels of abstraction.
No, that makes sense.
I didn't add a WMS layer to the Imagery example, but 2 of the 3 layers in the Imagery Layers example use WMS.  Is that sufficient?
Same comment for `_textureCoordinateOrigin`.
I was trying very hard not to break the "policy we all agreed", but I can't find a way to adhere to the policy while solving the problem there. Refactoring VertexDraggingHandler to inherit from DraggingHandler and make minor customization in VertexDraggingHandler is a very common object-oriented solution. I could do it in 5 min if I were doing it in Java or Ruby, but this took me more than an hour. Why? Because we agree to use mixin instead of classical inheritance, and I spent a lot of time hunting the web about how mixin could be implemented to call "super" method.

Blaming about breaking the policy won't help, man. Showing your solution that adhere to the policy helps. As I said before, I am happy to see other more elegant solutions.
I would invoke `sleep 5` here.  In my experience, invoking kill and start back-to-back without a sleep can be very iffy.
So then it is supported...it just isn't supported *well* :)
Just anchorPoint and animatedCenterPoint, although that's just because MW doesn't support them at all. I guess, technically speaking, they all should stay here and let the documentation for anchorPoint and animatedCenterPoint indicate that MW doesn't support them?
Not sure why we had the constant named this way before, but TiC.PROPERTY_SCROLLING_ENABLED seems more consistent with some of our other  constants
Conflict needs to be fixed.
Sorry, just realized this should be `mediaControlStyle` -- movieControlMode is deprecated. Might as well make it a link: [mediaControlStyle](Titanium.Media.VideoPlayer.mediaControlStyle).
Will cause a change in content size. Fire contentsWillChange. Same for the other padding methods.
Actually, this is somewhat confusing. Since I don't know what 38 and 40 are, I don't know what you're trying to do here.
Use the top-level query() instead of document.query().

Also, any reason for the two query styles? Could just be query('#grayscale') here as below.
that requires both net/http and uri already.
This check should also be above the var declarations since `value` is initialized to the value of `key`. If `key` is undefined (not supplied), then `_retrieve` retrieves the value associated with `undefined` which *could be* null (`undefined` is actually a valid key for storage). 

In short, if `key` is not supplied, then we should throw.
А, ну да, триггерить без неймспейстов.
An emerging idiom has it that templates that compute values are best expressed as CTFE-able functions whereas templates that deal only in types are best expressed as templates.
We do normally use this approach...
とおもったら、ちゃんとアクセサ定義してあったので、アクセサメソッド使いましょう。
https://github.com/fluent/fluentd/blob/master/lib/fluent/mixin.rb#L106
would be cool if we could at least change the code we're touching right now to constructor injection
oh yeah, i just saw that in my book last night.  i feel like i need to re-read the whole thing now that i have a bit of experience.

this lib was actually a great read in terms of understanding agents, state, watchers, etc.
```xml
<framework:config>
    <framework:templating hinclude-default-template="AcmeDemoBundle::hinclude.html.twig" />
</framework:config>
```
we shouldn't, I am not sure this was even added by me :(  Gosh, was this done by aliens? Martian? Removed.
This will re-create that array with each function call, so better "cache" it in a property to avoid that. Consider merging in the metalinkExtention function into this one.
Hi,

please re-review cufflinks easyconfig and easyblock. The patch itself is still in but is not used. This is for convenience, should we still be needing the patch.


On 10 Dec 2012, at 10:12, Jens Timmerman wrote:

> In easybuild/easyconfigs/o/Oases/Oases-0.2.08-goalf-1.1.0-no-OFED.eb:
> 
> > +
> > +name = 'Oases'
> > +version = '0.2.08'
> > +
> > +homepage = 'http://www.ebi.ac.uk/~zerbino/oases/'
> > +description = """De novo transcriptome assembler for very short reads"""
> > +
> > +toolchain = {'name':'goalf','version':'1.1.0-no-OFED'}
> > +toolchainopts = {'optarch':True, 'pic':True}
> > +
> > +velvet = 'Velvet'
> > +velvetver = '1.2.07'
> > +
> > +sources = [
> > +           '%s_%s.tgz' % (name.lower(),version),
> > +           '%s_%s.tgz' % (velvet.lower(), velvetver)
> 
> Well, parmetis does something similar, 
> we need to build met before doing the parmetis build,
> 
> If they are this tightly coupled they should be built in the same easyblock?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 

Cedric Laczny,
PhD Student
 
UNIVERSITÉ DU LUXEMBOURG
 
LUXEMBOURG CENTRE FOR SYSTEMS BIOMEDICINE
Campus Belval | House of Biomedicine
7, avenue des Hauts-Fourneaux
L-4362 Esch-sur-Alzette
T +352 46 66 44 6398
F +352 46 66 44 6949
cedric.laczny@uni.lu     http://lcsb.uni.lu
 

-----
This message is confidential and may contain privileged information. It is intended for the named recipient only. If you receive it in error please notify me and permanently delete the original message and any copies.
-----
@_total_count ||  @_original_array.count
This can be simplified:

```xml
<service id="jms_serializer.doctrine_serializer" parent="jms_serializer.serializer">
    <call method="setExclusionStrategy">
        <argument type="service" id="jms_serializer.doctrine_exclusion_strategy" />
    </call>
    <tag name="jms_serializer.serializer" />
</service>
```
I think we could use some smart default based on the method name.

```php
if ('get' === substr($method, 0, 3)) {
    $method = lcfirst(substr($method, 3));
}
Unnecessary `;` - Ask Chris suggested, we should run all these files through JSHint.
Since `GET` is the default, can we remove it?
Super glad there's a space here now.
On Sep 13, 2012, at 10:01 AM, Iain Buclaw <notifications@github.com> wrote:
> 
> Perfect example of what I was explaining - this assert throws for Windows (and this is the posix module :)

To be fair, Windows does support Posix both via Cygwin and whatever their Posix susbstem is called these days (used to be Interix).  This is why the platform-specific version blocks in Druntime all have version(Windows) … else version(Posix) in that order.


> I'd say put version (Posix): at the top of the file, I do believe that the same thing is done for the sys.osx and sys.windows modules (version (OSX): and version (Windows): respecitively).

I've considered it, but really, if you aren't on a Posix system then you shouldn't be importing core.sys.posix modules.
Yes; I know, but that's the wrong else; we also need an else on 

    if (messageOrPayload instanceof Map && !foundPayloadAnnotation && this.payloadExpression == null) {
80c. newlines exist for a reason.
This is generic so it should not say SHA (in fact neither should we!); it should say commit instead.
`.each`.
Actually, I just realized. That method would probably be better off in a CustomerHandler class.

The way I see it, GUI classes only interact with problem domain classes, which only interact with the handler classes, which interact with the data layer classes, then returning all the relevant data all the way up to the GUI so it can be shown.

-------------------------

So it'd be something like this:

GUI (view) -----> Customer (business) -----> CustomerHandler ------> CustomerDbAccess (data access layer)

CustomerDbAccess -----> CustomerHandler -----> Customer -----> GUI

-------------------------

So I hope that makes any sense at all. I'm trying to show the flow of data between the classes.
...I don't know. It's 2am and I'm finally tired, so my brain might not be fully functioning properly >:
We usually do `requirements.txt`.
Would this method of assignment be any more efficient than just using the dictionary later? I had originally done stuff like this (not this specific instance) with the intention of avoiding the dictionary lookup. Is that even remotely reasonable here?
This should be imported.
After eliminating a hole and adding its vertices to the returned list, `EliminateHole` removes the inner ring that was added from `innerRings`. Passing innerRings by reference makes it easy to do something like the following to simplify the polygon:
<pre>
while (innerRings.Count > 0)
{
       outerRing = PolygonAlgorithms.EliminateHole(outerRing, ref innerRings);
}
</pre>
the human should link to our facebook group, they can post help, or contact anyone in the community no?
It's a custom matcher that I added
1.9.3-p0-perf :004 > Array.new.methods - Set.new.methods
     => [:to_ary, :[], :[]=, :at, :fetch, :last, :concat, :push, :pop, :shift, :unshift, :insert, :each_index, :index, :rindex, :join, :reverse, :reverse!, :rotate, :rotate!, :sort!, :sort_by!, :values_at, :delete_at, :transpose, :fill, :slice, :slice!, :assoc, :rassoc, :*, :uniq, :uniq!, :compact, :compact!, :shuffle!, :shuffle, :sample, :permutation, :combination, :repeated_permutation, :repeated_combination, :product, :pack]

If we use any of these methods in the Array serializer, we are busted. :)
I think this can be removed and ``graph.rows`` directly used in the ``update``.
I just used the previous value here. And I also think connecting shouldn't take much longer. But what do you suggest? Adding another way to control the connect timeout?
if you use such abstraction layer like ORM, you should not look at such jokes like function calls :)
В clojure персистентный вектор 32 арный. Я решил сделать изменяемую арность а потом потестить, как быстрее.
Same query as above is this a pattern that we are following in documentation ? placing all constants available in each property ?
I guess we don't do it for other features, it just seems like an annoying issue to find as a developer that we can easily help with. @kwallis what are your feelings?
I am not sure if the packager currently has code that cares about the version (or required) or not. If there isn't code that cares about that, I will take out the hardcode.
Please add a `settings.js.sample`  with the `CLIENT_ID` and `CLIENT_KEY` fields blank.
Not sure, I dont think it was in spec. Thoughts?
noted, will change all pointer variables to use m_p
there are three fields - О Р П in Cyrillic. Not just present/expected.
A nitty-gritty comment.  Line 42-52, you could reduce the complexity of the if-else branches to one level like this:

if ( ! o instanceof LogEvent) {
    return false;
}

if (...) // check fields equals() {
    return true;
}

return false;

Or, you can let Eclipse generate the code for you.
Moved comments above method
See this line.
I will be removing this section. We already have a `quote data for sqlserver with literal 0x prefix` test in the column test file. BTW, before you say that the tests are spread out all over the place, remember that in many cases we mirror the test setup and organization of how ActiveRecord base is done. Another good example, the `test_column_names_are_escaped` in `base_test_sqlserver.rb` file. Either way, removing this.