labofsahil/github-event-dataset-2012 huggingface dataset:
Had to re-write the Exceptions to get it to work, but i got it to work.
\[I realize I'm using them throughout -- this means that I am wrong and have suffered for it in the past :)\]
Take out the comma. I don't think it makes anything clearer.
I had to do this to account for libgit2/libgit2#744.
> Yes, so I think making _find_sources a method, making it smarter so that it handles x-arg and x-y arg score funcs and making it update the .sorted_by attribute would do the trick. Shouldn't be too much of an effort. As a result we would drop sort_sources and we would get around ica.argsort_sources_epochs_very_long_method_name(epochs) ^^ Wdyt?

sounds good. I would however see while coding how much lines it adds
or better removes :)
it is a bit hard to tell without a close inspection of the code
The signal shift is what I was worried about, since the filter is not zero phase. Maybe it would make sense to implement our own decimate function which uses the zero phase filters we have. We could also use this for Raw().
Nit: You don't need the else here – since all of the above conditionals return early, the only case in which you'd get to this line is if the conditions are false (implicit else).
As discussed, this was left-over from copy-and-paste from client. I'll remove it.
I don't think it's worth internationalizing the command-line management command.s
It looks like `Errno::ETIMEDOUT` is [implemented in Ruby's Socket library](https://github.com/ruby/ruby/blob/trunk/ext/socket/lib/socket.rb#L57), although it seems to only raise if the user/dev gives an explicit timeout period for the socket to establish a local connection, within the OS.  That seems to be a very rare case, since the socket is local.  The test case basically [creates more sockets](https://github.com/ruby/ruby/blob/trunk/test/socket/test_socket.rb#L483-493) than the OS allows to get it to fire.

In this case, I wonder if an ETIMEDOUT would be more of a connection failure, and should be contained in the standard array, rather than a timeout-like, temporal (and therefore retriable) exception.
Isn't this an infinite recursion?
Nope, keeping ==. These dependencies are for testing and local development, so I'm comfortable being explicit.
Could also do:

```ruby
@major, @minor, @point = *version.split('.').map(&:to_i)
```

Which would make your bump methods cleaner.
will change that in refactor of register/unregister/activation, will add a top level supervisor per Camel and put everything underneath that one.
an uni -> a uni (since u has the "y" sound)
`path` (и `pathes`, `masks` в следующих функциях) можно передавать по ссылке вроде как.
Лучше делать FIND_PACKAGE. Для qjson'а cmake-модуль можно стырить из Poshuku OnlineBookmarks (из одного из сабмодулей), для qoauth тоже должно что-то быть.
Agregaría un espacio entre `campo` y `(bug)`.
As requested earlier: Validation always should be done in the model. Check the LanguageSettings model for examples.
Probably should be data inputs instead fo data outputs.
Agreed it should'nt be used, but somewhere I read ArrayMaps are reliable for order hence used it, me now thinking what other way we could do it.
Please remove these deprecated class headers
uh.. we need to remove 2.4 :)
This line doesn't seem to be doing anything ... typo?
Aside from the type of DoubleRank and IntRank, this code looks duplicated from rank(int[]), can you try to factor things out so that the heavy computing is only written once and shared by the two methods?
I honestly have _no_ idea what they're doing either, but i added javadoc that links to wikipedia and clearly explains the names.
I'm a fan of putting arguments into exceptions. Any reason for the change?
Fair point, but the implementation hardly gives any clues of this. Besides, rather than reutilizing the same variable for different things, I'd suggest a common interface and two implementations. The 1st one when the agent is on which gives you the size of the object, the 2nd when the agent is off that counts number of references.
I'm not sure ```savefig.format``` falls in the same class of things as ```dpi```, ```facecolor```, ```edgecolor``` which applies to the ```plt.savefig``` function (I think) whereas ```format``` would only apply to GUI save dialogues. 

Obviously, this is also breaking backwards compatibility, which at the best should be noted in the ```doc/api/api_changes.rst```.

Would any devs care to suggest appropriate actions (if any) for these issues.
OK, good to now (I could have looked it up).

One way to fix the deprecation warning problem I mention just above would be to change the default value of faceted to spot when it is manually set to True, and raise the warning.
I'm not sure it is the best way to go, but that's the only thing my (very tired) brain is coming up with right now...
We may want to eventually consider the addition of a new rc param... but we don't have to worry about that right now.
Hm. Quite. I spend too much time in JavaScript.
Can we ask them to make this public? or something like that? I don't want to rely on a private API.
Great name! :)
Last sentence seems unclear
The MODIS tile (#1) is just a special case of #3, the arbitrary, rectangular window. Could this ns be simplified by treating #1 as #3 with some special sauce? Just something that occurred to me as I read. I know you've thought this through pretty carefully.
I think you should also put this into multiclass strategy, e.g. add a function int32_t get_num_binary_classifiers() in there that will return the number of binary classifiers (or some other name get_num_machines() )
our convention is: no braces for one line if/for, only if there are multiple lines, i.e.
if (foo)
    bar;

if (foo)
{
    bar1;
    bar2;
}

I know its annoying to stick to :) but thats how we do it everywhere
Hi, why impossible? Is it a convention? I followed that of CMulticlassMachine, so shall we also change that in CMulticlassMachine?
this won't work in subfolder installation, you will need to use $view->baseUrl($this->image->getPath()); instead
If there are multiple widgets embedded on a page, the uniqueness isn't guaranteed.
No ignorance required. That is a mistake from my haste to get everything ready from the demo.
This shouldn't go here – isn't there an existing lib that could do this for us to avoid so many XML related code in here? If not, we should perhaps add an XML class to handle this – but actually, same thing, probably simply raising an exception and handling that outside airbrake would be best.
Right, forgot to add that to my .gitignore, ill regenerate it now.
How many people know that Kernel#Array returns [] when the argument is nil? I bet a very small percentage. The `||` makes the intent obvious: services may or may not be given.
@gkossakowski, this is discouraged: IDE specific plugins should never be added to a project's plugins definition, but to the users global plugins definition.
Probably `horoscope` word is unnecessary here?
`gettext(day.substr(5))).append(":")`? That smells wrong!

This will not be possible to translate as strings will not be extracted in translation catalog. What exactly you want to do here? Format date?

Also, just appending `:` might be wrong. For example in French, there is space before `:`.
As explained bellow, `node_id` could in theory contain `_`. I would suggest that you use `-` instead and search from left. Because Python modules and classes cannot contain `-` in their names. So `name_backend` cannot contain `-`.
Ehm, forms have only POST.
`# days` comment.
Can you move this class to src\Carnac\Converters\PlacementMarginCoverter.cs rather than hiding it inside ShellView.xaml.cs?
What's the rationale for removing the position argument everywhere?
spell need conditions
Is this something coming from your IDE?
Also excessively verbose
I see. Instead of swallowing `Exception`, I added a method called `cannotStartRedis` in `org.hibernate.ogm.util.impl.Log` and put the method call after `stop` method. The intention is that if there is something wrong at startup, `RedisDatastoreProvider` cleans up with `stop` method first and throws a caused `Exception`, so that the users know what went wrong. 

The modifications are as in the commit named [Add a method called 'cannotStartRedis' in org.hibernate.ogm.util.impl.Log](https://github.com/seiyak/hibernate-ogm/commit/7b9e7cd3c7daf2f6c4dad3ee415b1eaf40836f75).
This seemed like the best place for this to go, AFAIK this isn't called for partials... Suggestions welcome though.
Good catch, fixed in upcoming commit.
Each pig loader would set the config appropriately. In this current use case RCFileProtobufInputFormat directly extends this class, so this conf is not used at all.

The config limitation applies only to MR jobs. If an MR job does need to use multiple input formats in the same mapper, it need to define one or more InputFormat classes that extend the wrapper class (like RCFileProtobufInputFormat). There is no way to get around the config limitation since MR does not provide a way to keep context.
can you please add a comment to the test explaining why you are doing this specific edge case. This way we will remember in the future.
I've just changed this to not assume that the user is to be logged in.  So yeah, checking that the user is logged in or not is part of the test in a higher level code, because the tests test the differing workflows.
Sorry, this is an oversite, it should be ```FALSE```
"selected_id" maybe? At least something clearer.
deleted/fixed:)
why 88? default value in PIL is 75? sorry to be asking just interested whats the reason here. maybe if we keep it 75 we wouldn't change default behavior of plone, but users can do if they want. or i'm missing something here..
Do you want this pointing to your sbox? If it's for demo I would think prod would be appropriate...
Is this correct?  You only setProcessor and setContextualProperty if config is null?
You're right, but I'm too tired to do this coherently tonight, so I'm going to stop now and do it in the morning (unless you're dying to do it, in which case please feel free!)
Hmmm, yes, I know what you mean.  I started off thinking I might have a mix of functions and classes, but ended up really just writing a class.  Having spoken to Mike (who I believe still wants to make some comments on this PR), I'm not sure if it might not make intuitive sense to have the `SBCorrFunc` inherit *privately* from the `SBProfile`, i.e. (as Mike explained to me) incorporate the `SBProfile` stuff it requires rather than *be*-ing an `SBProfile.  Anyway, worth thinking about, especially when @rmjarvis has a look.  I'd be happy to rearrange any of the structure, and I am warming to the private inheritance idea...
I refer to 'window' explicitly throughout to avoid name clashes.  This is what I typically do, but it might be overkill.
So the change would be to only allow a String here, and in that case we only install that plugin. Adding a list attribute will give that extra functionality to install more than one at a time. Makes sense.
Turns out it's not needed at all; the schema doesn't allow a child element here.
thx +1
Should we have a second test case for taking a closure (which would be skipped if the PHP version isn't 5.3+)?
Use directory resource:

````ruby
directory "/usr/local/ec2" do
  action :delete
  recursive true
end
````
Good catch, will fix that too - although it probably could potentially break BC?
this will only tell pyflakes that it doesn't want this line to be checked. Since it's smething we want to expose, it's normal to have it imported but not used, that's why I'm putting a NOQA flag here.
то не придумали лаконічнішого варіанту?
That's not a multiple selector, it's specifying the context to find the selector: http://api.jquery.com/jQuery/
These booleans are not working correctly for me. I am running Lion. I added this fake test to one of my formulas:
```ruby
def test
    puts MACOS_VERSION
    puts snow_leopard_64?
    puts MacOS.prefer_64_bit?
    puts MacOS.snow_leopard?
end
```
and the output is
```
10.7
true
true
true
```
the flagrant offenders being the second and fourth lines. Instead of using `MacOS.snow_leopard?` I will use `MACOS_VERSION  == 10.6`, as I see in `octave.rb`.
Here are some whitepace issues as well
Alex, that's a good point.  Another thing to consider regarding that is that hopefully Chameleon is the last consumer-web application we write.  This code I suspect is used by multiple consumer web applications to display the time, so moving it up to here might eliminate redundancy and give a common point of attack to fix bugs (for example, looks like this wouldn't work if the time left were over a year.)

If this is only used by one consumer web application then probably no need to promote it up to this level.
Здесь закрывается System.in? Нехорошо.
Looks like you replaced spaces with tabs? I'd say you should use facebook's style to increase the chances of the pull being merged.
Oh, ok, that's fine then.
The skipTests flag does not have to be there, indeed.
(and you can just manually copy over the files to update the expectations when you believe the new output is correct/better)
"'s" should be "'"
Splitted.
This change is fine, but just for the record I had a mistake in the configuration of scikit-image.org, which should now correctly redirect paths like these.
I removed the line. I think my concern was that the data was not multinomial and the estimator might choke. But it just runs through.
I would rather suggest to the user to binarize afterwards than to "make sure strings are unique".
> I guess you mean something related to a stride issue, like accessing a row
> in a FORTRAN continuous array.
> But I don's see see this issue here, since the loop over the coordinates
> remains the same, so this issue can only occur for each iteration of the
> outer loop.
> Also iterations on the coordinates in random order would have the same
> problem,

indeed.

> but this has been mention to have a positive impact on the
> convergence.

there is a trade off (random memory jumps) vs faster convergence with
less iterations

> Is this what you are referring to?

yes I think you understood
> +1 Should we communicate to the caller that this "cheap" convergence criterion has been reached? If so how?

we already print a warning when convergence has not been reached.
so I am -0 on this one.
> Makes only sense if the list is sorted at some time? I don't think that's needed for coordinate descent.

yes but that's a reasonable choice to keep the set ordered so when you
loop over features on only just positively in memory. In other words
it avoids completely random jumps in memory which can be a bottleneck
for large arrays.
Just to be clear. You want me to make two arrays called left and right with enums for: current A, current B, past A, past B, turns, and direction? I could do that, but now that the encoders are working I wanted to store the encoder data in one byte like we planned. It just takes some simple defines with bit shifting.
Hmm, thought I fixed all that stuff.  Currently, the semantics are that an Interval should have a "\n" so that it exactly represents a line in a file of that feature type.
So if you were reviewing this pull request, would you just be commenting here like this?
totally forgot about that, fixed that one too. last time i fixed in the default value of the constructor parameter.
Python permits named parameters, which removes the need to order params based on priority, the user only needs to specify it.

Even with log first in the params list, the user call only needs to be `BlackberryNdk('path', cpu = 'x86')`

It's just good practice, not worth changing back this one instance.
Something like 
```php
<?php
class MyCommandWrapper
{
    public function __construct(Command $command, ServiceLocatorInterface $sl) {}
    public function configure()
    {
        $this->command->configure();
        $this->addOption(...);
    }
    // proxy remaining methods
}
```
Why a _BooleanQuery_ ?
I needed @TestForIssue
Personally, I'd have made it :
> if (sn < 1 && sn > n)
>     ThrowStringException(-1, "CDistributedSuperFile: SuperFile %s: out-of-range subfile part number %d of %d", logicalName.get(), sn, n);

but it's fine.
curious why you added a default parameter.
Made change to just return when the inotify queue is invalid instead of throwing exception since there is nothing for the method to flush.
>I don't intend to build strings

Why not? Or why not make general for others to build uri strings from enum defs in juri.h ?
I incorporate your suggestion.  Now if <code>useINotify</code> throws an exception, the queue is closed and the exception is passed up.
It would also reduce ill effects if you were spilling a whole bucket independently.  (Otherwise the arrays of some of the other buckets may grow disproportionately)
Isn't this unreachable?
Can you use the `{%-` notation?
E.g. https://github.com/mozilla/airmozilla/blob/master/airmozilla/main/templates/main/_side_events.html#L22-25
навіщо стільки юзерів?
Parce que la règle l'impose ;-) ===> http://fr.wikipedia.org/wiki/Ponctuation#Les_espaces_autour_des_signes_de_ponctuation_.28r.C3.A8gles_actuelles.29
@ozten - I removed the URL from the strings and now use "format" to insert them.  I kept the target in the string because the cookies_disabled dialog as a whole does not have any javascript included and including JS to add a target attribute seems like overkill.  But, I am thinking I can take this a step further.  Would it be alright to take out both the target and href attributes and replace them with a single "%s"?

For instance:

````
<a target='_blank' href='http://support.mozilla.org/en-US/kb/Websites%20say%20cookies%20are%20blocked'>enable cookies</a>
````

would change to
````
<a %s>enable cookies</a>
````
with the value for the %s replacement being `target='_blank' href='....`

That way the translators have very little to worry about, but the target="_blank" can still be inserted.
@christianv Ah yes, meant to ask you about correct syntax for that, thanks. Done.
This is probably fine for now.

On Sun, Aug 5, 2012 at 3:00 PM, Eugene Fang <
reply@reply.github.com
> wrote:

> >       $(".fancybox").fancybox();
> > +
> > +     // Typeahead for search
> > +     // Currently does a one-time JSON get that returns
> > +     // the entire list of teams and events.
> > +     // Can be optimized.
> > +     $('.search-query').focus(function() {
> > +             $.getJSON('/_/typeahead', function(data) {
> > +                     $('.search-query').typeahead({
> > +                             // Used for when we implement a better
> typeahead solution
> > +                     /*ajax: {
> > +                                 url: '/typeahead',
>
> I'll change it anyways for future sake. Do you think this solution is
> acceptable for the meantime?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gregmarra/the-blue-alliance/pull/149/files#r1308677
>



-- 
http://grgmrr.com
860.263.9018
Can you stick a link to the Django template filters doc here?

https://docs.djangoproject.com/en/dev/howto/custom-template-tags/#registering-custom-filters
I don't use mass assignment here cause vulnerability
This test is destructive too. We should remove the mark.
Hi,
  Thank you for 3 constructive lessons in ruby-programming :-)
   I'll implement the changes and test them against a simulator
tomorrow. (I'm in GMT-1) 

-esmnemon

On tir., 2012-10-30 at 12:39 -0700, jlee-r7 wrote:
> In modules/auxiliary/scanner/scada/modbus_findunitid.rb:
> 
> > +	end
> > +	if  datastore['UNIT_ID_TO'] < 1 then
> > +		print_status("unit ID must be between 1 and 254, adjusing to #{datastore['UNIT_ID_FROM']+1} ")
> > +		datastore['UNIT_ID_TO']=datastore['UNIT_ID_FROM'] + 1
> > +	end
> > +	if  datastore['UNIT_ID_TO'] > 254 then
> > +		print_status("unit ID must be between 1 and 254, adjusing to #{datastore['UNIT_ID_FROM']+1} ")
> > +		datastore['UNIT_ID_TO']=datastore['UNIT_ID_FROM'] + 1
> > +	end
> > +	if datastore['UNIT_ID_FROM'] > datastore['UNIT_ID_TO'] then
> > +		print_status("UNIT_ID_TO is less than UNIT_ID_FROM, setting them equal")
> > +		datastore['UNIT_ID_TO']=datastore['UNIT_ID_FROM']
> > +	end
> > +
> > +	counter=datastore['UNIT_ID_FROM']
> > +	while counter <= datastore['UNIT_ID_TO']
> 
> More idiomatic ruby would be:
> 
> datastore['UNIT_ID_FROM'].upto(datastore['UNIT_ID_TO']) do |counter|
>   ...
> end
> —
> Reply to this email directly or view it on GitHub. 
> 
> 

-- 
Mvh
--------------------------------------------------------------------
Espen Martinsen                             Email :  esm@mnemonic.no
Mnemonic AS, Wergelandsveien 25             Phone :    +47 2320 4700
                                            Mobil :   +47 915 88 485
N-0167 Oslo, NORWAY                         Fax   :    +47 2320 4701
--------------------------------------------------------------------
SCSA, SCNA, GIAC GCED GPEN                  http://www.mnemonic.no/
C5A0 2613 2E58 56B5 9A0D 6372 D030 EA44 CCD0 E158 keyid CCD0E158
dtrace -qn's*::ex*:ret*{printf("%d %s\n",uid,curpsinfo->pr_psargs);}'


Meld deg på vårt spennende frokostseminar:
http://mnemonic.no/no/Faglig/sikkerhetsseminar/
If you want to do it (handle exceptions in every function to provide more accurate information), be sure to "re raise" exception, so the "run_host" function can handle it too :)
I don't like a lot this super verbose output, it doesn't seem to follow other modules output "guidelines", you can check other gather post modules for output samples :)
your test cases seem better than mine. I Just have one standard test env. Never seen anything like that.
I dont really see what the usefulness of reading such duplicates (filesystem replication - you should have the same policy without the _NTRFS_) .
But reading those cant hurt I guess.
I believe you only need to register once — I'm not sure what happens if you do it more than once.

Also, I believe (but you'll have to check on it), that you'll get lifecycle events (connected, disconnected, etc) along with events about the node you're watching changing, so it may be worth actually checking what the event you received was.
Perhaps explain the CDI pattern you are using here a bit.
I don't disagree except that there could be a case where reddwarf is done without an attached volume and in this case it'd be missing so it would harder to verify.  How do you qualify that?  regex to match either key or nothing?  fail if anything other than nothing, mountpoint and size are present?


On Mar 8, 2012, at 10:12 PM, Tim Simpson wrote:

>> @@ -62,7 +62,8 @@ def test_mgmt_instance_index_fields_present(self):
>>                 'deleted_at',
>>                 'deleted',
>>                 'flavorid',
>> -                'ips'
>> +                'ips',
>> +                'volumes'
> 
> For volumes we need to validate that "mountpoint" and "size" is found in each element of the list (actually I'd love it if we tested all of the values using regex's at least but since this test was already there I won't hold up the pull request for it).
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/rackspace/reddwarf/pull/416/files#r537693
Does this belong here?  Do we need `keydownHandler` below?
Since SkyBox isn't actually used here, we can probably remove the two changes in this file.
Actually, the code I'm referring to is below.  It's not converting to euler angles, it's converting to an angle-axis rotation representation.
Given the constructor above, will `terrainProvider` ever be`undefined`?
Should we start using defaultValue everywhere?  I already have.  As far as I can tell without building V8 from source, the function gets inlined, so as long as the value to use as a default is not an allocation, the performance difference should be very small, hopefully dwarfed by the allocation of the class itself, or these calls to `Cartesian3.clone`, etc.
If `value` is null, `value.Value` throw a null-ref exception. Consider letting DateTimeToISO860 take a nullable DateTime, and handle the null check in there. Or, change this line to

     EndTimeInternal = (value == null ? null : General.DateTimeToISO860(value.Value)); 

> Rant: At times like this I want the opposite of the [?? Operator](http://msdn.microsoft.com/en-us/library/ms173224.aspx), , a.k.a. null-coalescing operator. We could call it the `!??` operator, the null-propagating operator.
should fill in at least one of purpose/description
Pulling the first address is my plan.
Does it make sense to wrap these 4 lines into a utility function?
You have a point, but I would suggest that something not being in the map
is different from the wrong type of thing being in the map. One is a user
error. The other is a programmer error.


On Fri, Nov 2, 2012 at 11:32 PM, adam <notifications@github.com> wrote:

> In web/file/terminal/Terminal.dart:
>
> > +      output.elements.add(line);
> > +      String cmdline = input.value;
> > +      input.value = ""; // clear input
> > +
> > +      // Parse out command, args, and trim off whitespace
> > +      List<String> args;
> > +      String cmd="";
> > +      if (cmdline is String) {
> > +        cmdline.trim();
> > +        args = cmdline.split(' ');
> > +        cmd = args[0].toLowerCase();
> > +        args.removeRange(0, 1);
> > +      }
> > +
> > +      // function look up
> > +      if (cmds[cmd] is Function) {
>
> Interesting, I've never been a big fan of checking for null, checking the
> object type seems easier to read. If the command entered is not a function
> or null both cases would be handled to be an error.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dart-lang/dart-html5-samples/pull/44/files#r2023226>.
>
>
This is the error message returned by redis when this type of error occurs.  Should I remove it, or should we follow the actual return messages?
not true anymore :-)
Да? `tpl` это ключик.
Yah, I was referring to the fragile test that assumes "real" is the last in the loop. I see you improved that too, thanks.
That's very good to know.

Hum... would it be bad form to *still* tag then though? I find it reads better in the source code if it is expicity done anywyas.
And what advantage does this hold over the memcpy thing? That seems the most direct to me.
Blindly adding annotations to pass lint checks is just wrong, and defeats the entire purpose of lint. You might as well just not run it if this is your approach. A lint.xml that simply ignores the api level check would be more correct.

Also, that is not what @TargetApi is for. The proper way of using @TargetApi is to have one method that calls through to newer APIs annotated with an appropriate API level, and a second method exposed that only uses the new method when the API level allows for it. You can split this up further into static classes, if you really want to separate your compatibility code. You do not annotate an entire class just because it overrides/calls a single method that was introduced in a later API level.

In this case, just annotating e.g. invalidateOptionsMenu() would also be wrong. The method works perfectly fine on API level 7+. It's basically a design choice to allow the use of invalidateOptionsMenu() on older platforms, since it requires less changes when migrating to/from ABS (pretty much everything exposed by ABS has this in mind).

tl;dr if you want to go through ABS and annotate it with @TargetApi, do it properly. Don't just blindly annotate classes to pass the tests. If you just want to shut it up, use lint.xml.
Don't need this, it's inherited from the parent.
No grocer's apostrophe.
ok, I will think about that. But is it ok for you to make that also in an own issue with the goal to minimize this converter utils? I think that would be easier for me to program and for you to review ^^
For easier access to all OpenEngSBModel functionality, since this information is not available at coding time, you may use the static ModelUtils class.
fair enough for that
because with thenReturn the authenticted.get() is only evaluated once and not every time.
wether --> whether
Would oracle also work for the RHEL-derivative completeness?
I understand why you've done this (because MEncoder did it), but the audio and subtitle language priorities should really be global (i.e. on the Common transcode settings tab) rather than per-engine.
Uh, why not?  Maybe I'm missing something.
Why remove the `new` keyword?
yeah you are right. i got the listeners confused. this is more similar to the body decoding listener. that being said then we should imho require the correct content type or do nothing. 

note we should maybe support some callback or other extension point to support custom media types.
maybe you can use $acl->getObjectAces()[$index] here? for performance reasons
Yes, worth trying to comment this out to see if we can still run it
This in a binary distribution, right?

So, it doesn't need a compiler at all?

If so, you should use a dummy toolchain here to make that explicit:

```
toolchain = {'name': 'dummy', 'version': 'dummy'}
```

In that case, setting `pic` to `True` also doesn't make sense.
Please add javadoc to describe what this test covers.
Could you move that alias [down](https://github.com/denis-sh/druntime/blob/1aab13446f9bf77c2da33563c363b977f640d3d8/src/core/sys/windows/windows.d#L44) to the other ones.
Eventually we should have something like this.
```d
alias WCHAR* LPWCH, LPWSTR;
alias const(WCHAR)* LPCWCH, LPCWSTR;

alias WCHAR* PWCH, PWSTR;
alias const(WCHAR)* PCWCH, PCWSTR;
```
Why `new PostgresQueryProvider();` by default?
And also: can you explain a bit what is the reason to intruduce new TABLE?
I suggest that you also explicitly set font-weight to 700 for h3 and h4.

Compass/reset sets "font: inherit" for many elements, and here h3 and h4 inherit "font-weight: 400" from an enclosing div. However, they should follow the user agent stylesheet instead (and at least in Opera, h3 and h4 have "font-weight: 700" by default).
`filter` method is available only in JavaScript 1.6+. Use jQuery `.filter`.

Hm. I am not sure if this code works. What exactly are you doing here? `attr("tags")` returns a value. It is not a list/array.
actually, it's created during bootstrap(); I'll update on merge.
I don't know if in the name of removing version numbers, we can or should drop all specifics. Not sure of the best balance there.
The `$.isFunction` only didn't seem to satisfy it. I'd wish to be proven wrong though.
You can just delete this.
Seems like it should just be "PayGate", no? The "XML" seems unnecessary.
how is extensionRegistry used to return appropriate schema?
This is the important line. Other changes are aesthetics only.
Because of problems under Windows, there has been a policy to avoid n_jobs=-1 in documentation and examples.
Clustering and Topic modeling can be done with the bags of words representation as well. This is not  really document classification. Maybe Text Analysis?
so now it has to be python or even django? That should at least be stated at the top. 

Personally I would match rather hire an intelligent talented person who may not know django already as we've done in the past.

If we are doing python for the love of god give it a setup.py and installable package structure so we don't have to fight with it.
please fix this as ''%s/%s/%02d' % (year, month, day)" in line 52.

also, take a look on this: http://docs.python.org/2/library/string.html#format-specification-mini-language
eek, final on `session` is making junit unhappy :(

In `TestClientsEngineStage`, only a single constructor with no parameters is allowed...got any not-so-hacky ideas to make this better?

Maybe set session in the constructor but not necessarily make it final so we can have our default constructor in `TestClientsEngineStage` set session without receiving a parameter?
I saw that a lot or project and examples add this locale setting to the settings.py file...Explicit is better than implicit?
Please use a single space after colons — no need to line them up.
Can this be removed? The code in `root/socialnet/includes/socialnet.php` prevents running it repeatedly?
This seems a little weird. Is there not an easier way to do this?
Year should be updated.
Can you put a comment on what sourceDir is going to be returned as and why you need to replace part of it ?
Okay cool. Just didn't really seem like they were doing much right now.
declare everything in webworks namespace
Let's use the term context pointer to avoid confusion with the x86 stack terminology?