labofsahil/github-event-dataset-2012 huggingface dataset:
Cancel old jobs when a new patch is uploaded

When a new patch set is posted, building the previous patch set
is no longer valuable. This causes an enormous extra load on the
build servers. If someone rapidly posts several revisions of a
patch, it could cause enormous delays and wasted resources in
Jenkins.

If others feel a need to make this an option, I can, but for the
environments I have seen, this is always desired behavior.
Issue39

had problems with activerecord.. mostly cause of my inexperience with it :P
fixed dependency injection

It seems there's a copy/paste error in the service definition of the orm manager classes. After upgrading i've been getting

```
[Semantical Error] line 0, col 66 near 'publicationDateStart': Error: Class Application\Sonata\NewsBundle\Entity\Tag has no field or association named publicationDateStart 
```

As the PostAdminManager used the Tag class to construct it's query.
fixed validation groups admin -> Profile

Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: -
Todo: -
Develop

CSS fix for project pages of Utopies Réalistes
centreing the #results div

this should centre the results div
Дз3 фильтрация.

Потерялся на стадии выборки по значениям полей (94 строка).
Общая идея: 1 единственная функция, в которую передаётся наш массив
событий и флаги выборки(предшевствующие/последующие или
прокоменнтированные/непрокомментированные). Вместе с флагами передаётся
хеш со значениями, по которым формируется выборка.
Выборка производится по данным, которые были переданы.
Instead of [Your Name]; fetch the site name. Standard practice.

**Warning**: Test before merge.
Allow jasmine to run in browser

```javascript
var invoke, _;
var __slice = Array.prototype.slice;

if (typeof _ === "undefined" || _ === null) _ = require('underscore')
```

Because ```_``` is declared locally in line one above, line 4 above will always be true, which means ```require``` will be called, and ```window.require``` doesn't exist.  I removed the ```_``` declaration so it falls back to the outer scope and picks up the previously declared ```_```.
Next fix63 accessuri nourn

The fix related to this bug fix is blackberry/BB10-Webworks-Packager#63

Made the packager fail if the access uri attribute does not specify a URN.
add doc about localizeddate
Checkboxes

blahbl;ah
Added blank content template

I've added a blank copy of the content app template matching the whiteboard version.
cambios bc
Fix for Issue #153

support new admin static files layout introduced in django 1.4
Multi-language obituary and chat messages.

This changes only the chat and obiturary messages to use UTF8.
Renames the test app's to WebWorks Test app-en and WebWorks Test app-fr.

grep and sed  for the win
and Jeff says it does **not** work 

```
egrep -lRZ "French application name" . | xargs -0  sed -i -e 's/French\ application\ name/WebWorks\ Test\ App-fr/g'
```
Added virtualenv configration

This way users can quickly setup the python project, without it conflicting with other projects, etc..
radio com1 and com2 implemented

The new version works great. No crashes during startup and shutdown anymore.
API key used for remote import

Used the same code as in checkin to be able to use the API key.
Added other sports to example
Add support for SOAP API

Pass through SOAPAction header, necessary when proxying requests to the
SOAP API.
added option to preserve object acl on copy. 

turning on the option causes two additional api calls per object (one to get acl, one to set acl)
INTSAMPLES-61 Changes, use LinkedHashMap to demonstrate LRU concept
README 部分的修改

* README 中增加关于 Arch 上如何调试相关信息
* `Rakefile` 增加关于 `public/` 目录的测试
adding nothing but joke questions
I would prefer to see this done via a switch() statement, with cases for each type and each "getBest..." method called by its full name. Otherwise grepping the code for getBest... won't match this case. Otherwise, I'd rather use a proper inheritance structure for this, but that's a lot more work.
It is already
The AS guide lists the differences among the two. Is there anything outdated? See http://guides.rubyonrails.org/active_support_core_extensions.html#wrapping
This is not enough, the dictionnary will not be escaped properly
There are bug here. Resource url will change in each WEB commit with style modifications. Same for .js assets.
I almost totally agree with jackpoz, there's just one thing that I disagree with.
If you think it's OK to add bugs, then it's not just Arcemu that's not a place for you, it's software development that's not for you.
What if you have to develop the software of some vehicle, like aircraft and you add bugs there and the plane crashes because of your bug? What if you develop software for medical uses ( like for surgery ) and your bug ends up killing or worse maiming someone irrecoverably?
Yes MesoX is right in that bugs do happen, but that doesn't mean that it's OK to add bugs!
We are supposed to focus on quality control, NOT damage control!
Is that better than brunchPluginType: 'javascript' ?
brunchPlugin: yes looks weird....
Some sort of convention over configuration would make this nicer
@tenderlove I wouldn't revert - I'm pretty sure everyone of these errors is due to passing nil parameters to url helpers so all we are doing is exposing existing bugs which previously would've linked to an index page.

What's unhelpful though is that there's no backtrace to the error which makes it look as though it's a recognition error rather than a generation error - ideally it would show up as a `TemplateError` which would give some idea where the problem was.
@irrationalfab done
Hi,

> What happens if the hv drivers are also included through a bootincluded kmp?
> Which .ko file take precedence?

That's not a kiwi question. It's a kernel question. imho if the
driver is installed in weak-updates or updates the one there will
be preferred

Regards,
Marcus
-- 
 Public Key available
 gpg --keyserver gpg-keyserver.de --recv-keys 0xCCE3C6A2
 -------------------------------------------------------
 Marcus Schäfer (Res. & Dev.)   SUSE LINUX Products GmbH
 Tel: 0911-740 53 0             Maxfeldstrasse 5
 FAX: 0911-740 53 479           D-90409 Nürnberg
 GF:  Jeff Hawn,Jennifer Guild, Felix Imendörffer                  
 HRB: 21284 (AG Nürnberg)       Germany
 http://www.suse.de
 -------------------------------------------------------
warning: string literal in condition
Was that last clause intended to be a file existence check?
put that onto the same line as the rest of the method signature, no?
ah bah oui! ca ne m'avait pas traverser l'esprit, mais bien évidemment!
How well does messing with the client params work for mulit-threaded requests on the same HTTP client?

If the same client is used for multiple requests to different hosts with different proxy settings, changing the shared client settings in the middle of requests appears like a bad idea.

If the client instance is only used for comm with one host, then it might be easier to just set up the client properly with the desired proxy or none right from the start.
Thanks, Peter, well spotted. Obviously I did the simplest possible thing to fix the particular issue, and didn't have a test involving .. tokens.

Running LuaMacro should be pretty straightforward; clone and make a forwarding script 'lua /path/to/luam $*" on your path.  The only trickery is that luam adjusts the module path so that it can find the libraries next to it, and I'm not _entirely_ sure whether a symlink would work properly (an alias would be another way of doing this)
inspiration for the rm / unset function composition comes from [remi's shash project](https://github.com/remi/shash)
rsa da ani tebja ignoriruet :)
a sam mangos sdox, tam nechevo nebudetj menjatsa, kstatji Smoo otpravilsa na otdih.
Should this be charwidth(c)?
Unbalanced parens.  How about this instead?:

"...and will be the default in the next release.  A serious warning will be issued if the new C3 linearization is different from the old `linearization specified in the DRM <http...>`_."
Гм... вроде же уже говорили. На кой хрен это в ядре, если каждый ГМ может сделать себе коротенький макрос и не парить никому мозг? Или идем на поводу у анацефалов с вовжп?
@peller This is what I came up with, for now.  But in the future, it would be nice to use async `require()` (see `getHelper()` in **metadata.js**).
I didn't understand why this if code block
shouldn't it be ``$session->getFlashBag() instanceof AutoExpireFlashBag`` ?
kjghjk.kj90
This was a leftover artifact from an earlier version of the code. Using PTHREAD_MUTEX_RECURSIVE is not only of no use (as this mutex is only waited on by the pevents code and never left blocking at any point, nor used recursively), but also generally a bad coding practice when used in combination with conditional variables because pthread conditional waits are not aware of the recursion state of the mutex. 

More info about this is available here: http://neosmart.net/blog/2011/waitformultipleobjects-and-win32-events-for-linux-and-read-write-locks-for-windows/#comment-644113
i traced that code back to http://bugs.dojotoolkit.org/changeset/24866/dojo which references the initial ticket to introduce dojox/mvc so there's nothing to help give us some background on the code.  either @edchat or rahul wrote it so i guess we'd have to find out from them directly why it ignores `null` and `undefined`
I think suggesting a prompt reload when loading and unloading major stuff is a reasonable thing to do.
I had bundled a cli with it, but reading the instructions I've split to send two commits (the name is riva). otherwise I dont know how to test a library in brew's context: I could do a piece of code to compile, put and get a value and even then it wouldn't be right as a software that depends on it is best as a review IMO. Any suggestion ? Can I send another pull request with the cli and then create a cyclic dependency as to test leveldb you need riva, and to have riva you need leveldb ? Or I coult otool the lib for the deps + some symbols... need a help here :D
проверку на то, что тип этого объекта ==3 (студия) лучше засунь сразу в метод get_studio_profile_resume (imho)
Hello Tony,

it seems to me that [the line](https://github.com/tonytomov/jqGrid/blob/master/js/grid.inlinedit.js#L273) with the second call of `sucret = o.successfunc.call($t, res);` should be removed because the call is already in the previous line.

Regards
Oleg
I'm sure about the day, which has no leading zero. I read some discussions.
As for the hour, I'm not quite sure about it. There were less discussions, and it seems not so strict (like the day).

My only doubt concerns the overall readability of a vertical list, for example:

1.01.2012 6:45
12.01.2012 18:20
3.02.2012 20:01
vs.
01.01.2012 06:45
12.01.2012 18:20
03.02.2012 20:01

I find the second list nicer to look at, read from.
I was using exactly 3.1.3. Without requiring globally everything worked like a dream. But out of the box controllers were logged as `ActiveSupport::Configurable::Configuration`. More, ActiveRecord SQL tracing was logged as `Rails`.

I'll try to take my time and work out the demonstrating app for you. The original code is commercial project, so I can't just share it to demonstrate.
you can use `el.hidden = true`
We felt that no one actually used it.  I guess we were wrong.  Additionally, it is completely derived from the contentID which is still available for easy copy an pasting.
MR BOOMBASTIC
SUPER ELASTIC
/cc @sarahmei
this causes some problems ... see #2627
The hello-gl target (line 5) requires "-lm" as well as the hello-gl-dummy
The engine calling cancel just mean the engine needs to track every one of its tasks.  That *might* be hard.  Could just be a detail that shakes out.  We could also have user data on the item and have a `filter` kind of thing that can run over the queue and clean out or run items at a specific time.  That'd be potentially harmful to other engines, but it should be rare.  Just the kind of thing to think about.

The plugin thing *sounds* cleaner.  I guess the default engine also needs such things, so maybe it should always be there.  I don't have a very strong opinion on this at the moment.
I know the code allows it, but I would think a nice api would be one that allowed any object that responds to `#call` to be passed in.
I see you rearranged the logic of creative inventory clicks. :P And some other little style changes.
I've made it a general principal to just never use foldl. Hasn't bit me yet!

You gonna fix it, or shall I?
Hi!
Could you tell me where do you get the hash and how does it make the composer works again?

Thank you
ive added every char update but it crashes on loading characters
```ruby
@comment = Comment.new unless (@page.country? || @page.city?)
```
totally, but I'm thinking to my fellow developers who are not as smart as us :p
aaaaand we're green! http://travis-ci.org/#!/UnionOfRAD/lithium/builds/666425 :)
Perhaps we should consider '@silent' to not define a ruleset? That might also mean it can no longer be called '@silent'.
You should take a look at https://docs.djangoproject.com/en/dev/topics/templates/ and look at block.super

Rather than this style of 'extrascript' block, you just call {{ block.super }} which allows you to 'add' to blocks rather than override them.
Oh boy...

So, we probably want to expand this to include "is face down", "is day side down / night side up" and "is flipped"... then again, not sure how much we care about the distinction and how much that borders on rules knowledge
Sure, but how do you manually set those lexicons ? :)
I added some views... will keep working around a nice look...
Stupid Kohanna and it's strict settings
The other registry key in the report is preferable, HKEY_LOCAL_MACHINE\SOFTWARE\SONOV\Shaiya

The long ID in the uninstall key's name isn't universal, so it has a lower chance of working on different systems
Sobre lo primero, no, no lo probé, lo voy a probar.

Sobre lo segundo, originalmente tenía más datos en el índice, pero me estoy basando en la especificación que nos dio Gabriel de esta búsqueda rápida.
I am using dir local variables to ignore different directories in different
projects.

https://github.com/ahobson/dot-emacs/blob/master/ahobson/my-dir-local.el

Since the variable is buffer local, I need it referenced before the ack
buffer is created.  If you use make-variable-buffer-local, will that keep
the defaults for all buffers?

--drew

On Mon, Feb 13, 2012 at 1:39 PM, Sudish Joseph <
reply@reply.github.com
> wrote:

> Cool, first pull request. :-)
>
> We can remove local-ack-arguments with make-variable-buffer-local right
> after the defvar at the top instead.  Since there're already other
> buffer-locals, that makes the most sense to me.
>
> What are you planning to use this for, if I may ask?  Is it to change the
> arguments on a per-search basis?  That'd be cool.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ahobson/ack-mode.el/commit/846ed95e35ca5ced0406f6dbb2d41f6ebe6d1276#commitcomment-961268
>
These all need to be escaped by the php htmlspecialchars function. There are shortcut functions in library/htmlspecialchars.inc.php. The two functions are text(escape normal text) and attr(escape text within html attributes). So, all these items should be changed to text(), for example:
```<td>'.text($RA['messageID'].'</td>```

For better explanation of the security model and using these functions see here:
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan
Have been tested and works fine, however needs another review. to be continued. 

gdk.threads_enter() gtk.gdk.threads_leave() functions must be always called if there is any direct or indirect kind of access to _model.
Unless this will crash, a disparity between the internal view of the GtkTreeView, and the GtkTreeModel will occur, as described in gtk reference. 
This generally means that the model has changed without letting the view know.
:D i am not sleeping :)) But i think you are sleeping :D copex = retriman :D ?
The reason I bring it up is because I don't know if it'll end up breaking queries, or what the performance hit is on doing this vs. a TOP style query.
Sorry, yes. Running build publish-local fails with the error above.
Yes .. in fact there was another patch that fixed the other instances of this .. [in src/surface.lisp](https://github.com/mathrick/cl-cairo2/commit/b552a571f551fcb0d0f75198d528da0c53dee460)
shouldn't it use ``:class:`` here ?
Change this to point to https://github.com/bottlepy/bottle-extras - I forgot to fix that URL when I did my Redis plugin. ;)
oops sorry... thanks for fixing
I agree with Paul. Even though `else return null` seems more readable, the braces rule adds consistency. Consistency is very important and it's the focus of PSR-1.
Why is it important to expose it to global scope?
These params don't match with the actual parameters.
Just for clarity the Parenthesis were not a part of the syntax.
What's the best way to do this?  Is it with a property like this or in update() or updateWidgets()?
Thanks, James.  I've included these changes on master.
Now you guys are disagreeing with each other ;-) is it a `c.l.Character` or a `j.l.Character`?
Why not `expandfeatures[i].onclick = toggleFeatureExpansion;`
Sorry, still figuring out how to use Vim :/
# Homework 3

| **Part**  | **Points** |
|-----------|------------|
| base      |  1 |
| advanced | 0 |
| **Total** |  1 |

Well done on the homework.  Everything looks fine in both the output and the code.  The only thing I can think to comment on is your variable names.  Specifically, you should just reuse variables, maybe just `text` instead of `text1 - text6`.  That said, even I think I'm nitpicking.  Only other criticism is I could go for math that we didn't go over in class.

Anyways, it seems you get how strings work.  You may want to look at python's string formating.  You can find more information on it [here](http://docs.python.org/library/string.html#formatstrings).
ここ、コンフリクトが残ってますね
Sorry, it's my mistake.
But, I think that it needs CFMakeCollectable() instead of CFRelease().

Thanks!
For some reason the SourceSpellId in the quest_template database is different to the spell ID in the code in the above dalaran.cpp which means that this quest isn't completable anymore. If anyone is having issues with this quest where they get teleported out of the opposing faction's area with the buff on this should sort it out:

```SQL
UPDATE `quest_template` SET `SourceSpellId`=70971 WHERE `Id`=24451 LIMIT 1;
UPDATE `quest_template` SET `SourceSpellId`=70973 WHERE `Id`=20439 LIMIT 1;
```
I think the better way is `t('flash.blablabla')`
why not injecting the document manager directly ?
Very sorry, I committed this to the wrong branch (I had the master open)
In my opinion, this should NOT be part of the codebase.  This is best explained via documentation!
Forgetting to comment this out will result in apps getting rejected from the app store!
because line endings are changed when changing it in 1) linux 2) windows inside the qt creator.
As you guess, I wrote this in vim :P
De qualquer modo, eu ainda acho a afirmação bem forte. Seria interessante dar um parágrafo sobre o assunto com algumas referências.
Volvieron las cuentas corrientes?
we can drop this thing alltogether
This is a comment on a commit.
... similar questions of the above.
Should this "index" be a constant/enum/type-checked?
In the setUp function.  Doing it by hand at the beginning of each test method was only necessary because these tests were in the wrong place.
Did you want the arguments to line up here?
Comment is incorrect in two ways.

1. It won't be inserted; it will be inserted or the existing row with that GUID will be updated.
2. This method returns true if the record is “new”, not if the insertion succeeded.
Hey, cool! :-)

Updated the OSX binaries accordingly to SB20 in [r1019](http://code.google.com/p/ps3mediaserver/source/detail?r=1019).
Thanks!
You may want two allocate functions - one to allocate a block you know is fixed size, the other to allocate a variable size bock - which I'm not sure you want to be affected by these other block sizes.
tilelint.enable is a new input parameter, right? Add a local variable to save it?
No hay que hacerse primero el replace y luego el striptags????
On Sat, Feb 4, 2012 at 6:59 PM, David Golden <
reply@reply.github.com
> wrote:

> * In a configure/make/test/install cycle, I do think that most of the
> arguments should be given in the configure stage, even if they apply to
> later actions.  E.g. you specify "-Dprefix" to a configuration script,
> which creates the Makefile, even though the prefix arg isn't used until
> "make install"
> * I'm not sure historically why it wasn't done that way in Module::Build
> and whether it was thoughtful design or just happenstance.  CPAN.pm allows
> users to specify options at various stages (e.g. suggesting '--uninst=1'
> for an 'install' argument) so maybe that's why.
>

It was really just the general idea of config options that successively get
narrower in their scope.  Something like having both a /etc/zshrc and a
$HOME/.zshrc .  It's useful for something like 'Build test', where you may
want to run the 'test' phase many times with lots of different parameters
passed.  It's also not too hard to imagine running 'Build install' several
times with different parameters each time, for instance if you're
installing from a mother machine to lots of daughter machines where the lib
directories are all on NFS mounts or something.

The only theoretical trouble with that (not considering whatever practical
troubles you seem to have here) is name collisions, but in practice that
seems not to be a big problem.  What I really wanted to do was have very
tight parameter checking (something like Class::Container, which I am
extremely proud of but never really able to explain :-) but never did it.

 -Ken
https://github.com/seyhunak/twitter-bootstrap-rails/pull/43
Anyway keep it,but i'm not sure in python env we are going to need it.
article doesn't sound good in a manual; rather section/part
lol.. yeah, imma like looking through this code :P
Fixed config parameter loading.

Removed web-app folder (not needed for a plugin)
It should not be there.
I believe there's a typo on line 214 of socialregistration/views.py 'erorr' should probably be 'error'?
I didn't mean that kind of protection. Anyways I can put it back just for reference and statistics. 
I'm my min as I thing I can  find useless code and dump it in favor of stability and robustness. So the assistant fell into that loophole. I can see now that it has some use in relating friends and devices. Although new are generated every week :-) (to keep in mind) 

On 28 Ιαν 2012, at 0:07, thpryrchn<reply@reply.github.com> wrote:

> Yes, But the server status already doesn't show all of it unless they are logged in as Admin. So it can't be spoofed, and banned.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jimmykane/The-Three-Little-Pigs-Siri-Proxy/commit/4a13114d5d780d788dd93233905161e322f42792#commitcomment-909896
Why did you do that change? @SB_APPNAME@ is the same as Nightingale I think?
This is a great idea - it means the constructor won't show as uncovered.
yes, geneId == 0 corresponds to a 'no gene'
This seems to be a specific local file that should not be merged with the master branch :).
Removed, in latest commit.
Why add the `*.project` to the ignores?
how costly is this for a distributed cache? what affect does it have on other actions on the cache?
I didn't see the other ones call init.  Also, if this is spring configured, shouldn't init called via Spring.
@lizlux Would you mind reindexing these lines into a separate protected method that will return the result of the $this->sendSelfRequest method for this call? That way, we can better write unit tests for this logic.
Probably want backticks or boldface on these
Possibly, but we'll see. It depends on where we want the non-guarantees to live for the two fields raised in the previous line comment.
Backwards message compatibility. At decoding I assume the old "akka"  
protocol if this is not set. But I guess, as it seems now, this could be  
required.
good point, I'll shut up now
2 closures created when isEmpty has already guarded this is a bit too wasteful.
yes, it's not actorFor that is a problem, but `node(master)` uses test conductor

it's a pitty that it's not possible to tag the whole Spec and have scalatest to ignore it completely instead of always creating an instance of it -- annotations not invented here syndrome
no not with a producer, the producer has an implementation of onReceive itself (and its final). if you want to change you behavior you have to use the onTransformOutgoingMessage, onTransformResponse, onRouteResponse methods
`val endpointId = Iterator from 0`
okay, will add bounds to breadth of printed tree; zipper is beside the point because a String is what is needed in the end
Can it be both terminating and normal? Otherwise I'd recommend just having the isTerminating (and perhaps implement the isNormal as a final method that returns !isTerminating. To avoid the true-true or false-false case.
yeah, otherwise it'll trip people up
Yes, definitely! This is mentioned in a TODO comment at another place, and  
this is something I will fix. I have to change the protobuf definition  
though, and I wanted to postpone it after everyone says OK to the current  
architecture.
this regex literal won't work in IE
I know you didn't add this, but is object_id actually valuable?
I added an optional 'name' parameter so you ca get a prettier output when using AR classes.
Same, don't remove the dot lookup optimization.
http://wiki.python.org/moin/PythonSpeed/PerformanceTips#Avoiding_dots...
Don't understand. You are calling ToString on any non-string value and you special case string which ToString call is no-op anyway
@johnmartin No, you are right. For later version there is an on event: http://backbonejs.org/#Events-on.
Totally a personal thing, I'd have started that with "By comparison, " to tie it into the previous example introduction about how those ones are generic to all repositories.
I don't see where the card is actually updated.
Not sure I understand the link between the name of the function and the code in it
Currently, m_filePos (in the "bytes wriiten" sense) is always = uiBufSize.

Conceptually,m_filePos being the number of bytes actually wriiten upstream, it is the one to use.
if you just use find, and check whether the return iterator isn't m_groupSelectedItems.end(), then you can save one search
я просто про пробел, до  {
I'm going to take the blame for this one, but let's use the standard python way of doing this and remove this dependency:
http://stackoverflow.com/questions/969285/how-do-i-translate-a-iso-8601-datetime-string-into-a-python-datetime-object
My bad then :)
Why not this?

```ruby

  if @result
    return
  elsif now > time_to_stop
    @result = TIMED_OUT
    return
  end
```
Should this be scoped to a specific `user_id`?
accept is just active now.  I think the more usable API needs to be in account.rb.  We will then need to adapt the code in account.rb to an external auth api, Warden or OmniAuth (leaning towards the latter now)
Please indent with four spaces.  (Most aqsis code is currently indented with tabs, but new code I write is indented with spaces.  Generally speaking, please try to match the indentation in the existing files if you're making changes here and there.)
Please put a space between the `if` and `(`. Also, don't start the `{` on a new line for if conditions.

Also,  note that you will need to add the same check in `isometricrenderer.cpp`.
when does this.swiping get reset to undefined/0 or does it matter?
Why not simplier:

```php
$name = $name->getName();
```
As you overwrite this `$name` variable either ? (same below)
Yeah I considered a struct, I tend to lean on the object side of the fence whenever possible when coding cocoa though.
As above, I don't think we should offer this feature.
Thanks, merged in zzak/node@08fa918
If it's a synchronous function, you don't have to make a callback, just return the result. Callbacks are supposed to run on the next tick of the event loop anyway.
Actually, the limit of 32 arguments is gone; the integer argument of the start_message function takes care of that.
There will be a SctpNotification for bindAddress/undbindAddress operations from SctpNotificationHandler, I think firing a user event is redundant here. See io.netty.channel.socket.SctpNotificationHandler,
replace with (here and elsewhere) .. whenever an out_file is specified, we want to create an absolute path to it to help with workflows.

```
os.path.abspath(self.inputs.out_file)
```
Bug: extra leading space
I don't agree.  The user may want to capture the log for a piece of code by enabling/disabling around it and fetching the log afterwards.
had to do `.try(:empty?)` to handle the case properly.
Style: no newline padding within the test block
I had some comments in the previous PR. I was curious if this should inherit from `Union` instead.  This may or may not be a good idea; just a thought.
> So is the only way to use your work by having TextPlot or matplot
> installed? You haven't changed anything about how pyglet gets used
> (i.e. it's not one of the new supported backends)?


You are correct. pyglet is not supported in my code. The old pyglet
plotters are still available: `from sympy import Plot`. However there
is a deprecation warning (pygletplot will not be removed, however it
will have a new import path: `from sympy.plotting.pygletplot import
Plot`). The old pyglet plotters provide superior experience for 3D
plots (for now).

To use my code you need to do `from sympy import plot` (no capital
letters). If matplotlib is not installed it will use ascii plots. If
matplotlib is installed it will use it instead. If matplotlib is NOT
the current unreleased development branch the 3D plots will fail due
to bug in their coloring routines (this is not checked for in my
code).
I'm referring to the "can is" ...
Do we have any other examples that can serve as good testcases?
I think the "groups" argument should accept arbitrary iterables (i.e. tuples, lists, sets, FiniteSets, whatever).
I am not really into the topic; however, wouldn't it be nice to actually have subprocesses created by default?  From what I understand, this behaviour should be rather transparent to the user anyway.
SymPy even includes a tool to strip trailing whitespace for you, it's called `strip_whitespace`; you should find it in `bin` in the source tree.  It's pretty simple to use and should do the trick when you've got more than one line with trailing whitespace.  If you use Emacs, you can also do `M-x delete-trailing-whitespace` in your buffer.
I think, that otherwise it is possible to change lens or file_path during agent's instance life cycle and it is not expected usage for gloves' agents.
不是放一行，而是用if-else结构，不要用ifelse()函数。
http://php.net/manual/en/wrappers.php.php

>Note: A stream opened with php://input can only be read once; the stream does not support seek operations. However, depending on the SAPI implementation, it may be possible to open another php://input stream and restart reading. This is only possible if the request body data has been saved. Typically, this is the case for POST requests, but not other request methods, such as PUT or PROPFIND.
Any blog posts or articles on the subject would actually be quite welcome.  I can't recall reading anything on it; I've been going mostly from instinct.
The title underline is a little short. They should match the amount of characters in the title (otherwise it triggers a sphinx warning).
extra space in here
You might wanna rename this partial to something more consistent with what we have been doing, like single_question
Having pgrep will be better then will change it to pgrep after some more research.
More descriptive `$is_multipart`?
Should be maybe "check if there are any user-selectable languages" since 1) you are not checking the count and 2) surely there is at least one language at all times.
@pquerna Limiting it to just a filename does make it simpler and more straight-forward though.
Depending on a platform only one of the definitions will be substituted by #define ... right?
Can you change $default here to $sectorID.
じゃあしゃあないな。
Good point regarding dependency.
Seems desirable to be completely independent of the real grib api.
Ok, but just to be clear: if in this function we can tell that ``labels.shape[0]`` is constant and equal to 1, then are we guaranteed that ``T.arange(labels.shape[0])`` is constant-folded into 0?

If that's the case then I think the docstring should say something like: "If the shape of labels is available and is a constant equal to 1, then T.arange(labels.shape[0]) must have been constant-folded into 0 for this function to return True.".
Should the comments tell Null cost? I'm not sure.
strtok is not thread-safe. Please use strsep instead.
I guess it's possible to use `super` instead of `read_attribute(:email_at_notices)`.
This module does not appear to be covered by any tests.
* if文の(indexPath.row == today - 1)については、line:53に合わせて(indexPath.row + 1 == today) とするのが良いでしょう
DOH! You are correct, good sir. A fool is me. I'll patch that.
This sucks to read
That's pretty much how the code went prior to the change so I gather it's smaller gzipped as it is now.
Which cases does this fix?
fixed, remove all quotes.
This look ok? https://github.com/possibilities/meteor/commit/8bd27f992f06b097258da21dd26ac0efb642d4fd
We could even consider mounting static on /static and just keeping / of limits.
Probably should indent these three lines 4 spaces.
Should we also have some tests for MtechUssdResponse (especially the nav part that isn't exercised by the transport)?
Why `self.__class__` instead of `type(self)` here?
I vaguely prefer `[msg]` to `msg,` for this kind of thing. The comma is easy to miss when reading.
Inconsistency in hexadecimal convention with the other flags on purpose?
@fat done
This isn't relevant to this pull request (and I should have mentioned it a very long time ago), but I totally don't buy the need for an entire dev-dependency template engine just to add a couple requires...
I believe min_upload_date is supported here too to track/pass back in, at least that's what the old locker code did, so that if they're following lots of people it won't re-page through all of them every run :)
I hadn't thought of that, I find the indentation in haml really easy to follow. It's the repeated `classname`s here that bug me, and the indentation is harder to follow with all the lines that have to begin with hyphens. It's also just that I'm used to reading haml and understanding how it maps to the dom. With this, it seems like we're trying to abstract the dom away, but we can't abstract away it completely.
I think this could be:

    enc = item.thumbnail && dojo.isArray(item.thumbnail) ? item.thumbnail[1] : item.enclosure || item.content;

I think @rmurphey would encourage us not to do variable assignment within a conditional whenever possible.
Oh, I see it now. I had totally missed the `>`...

Nice catch btw. I've changed it and it was returning the list of attributes instead of empty for that case, so I'll commit a fix with the changes.
OK. I wasn't seeing that behavior when selecting entire lines with inline comments.
Actually, I misspoke...it would bail if any of the components was unparseable (not if any of them were not a percentage), whereas now I'll go ahead and convert any components I can. The functional difference isn't really important since in either case it will end up being an invalid color value...it just simplifies the code to be able to treat each component independently (since I don't have to have three separate vars, or wait till I've parsed all of them before deciding whether to actually do the conversion).
Ok, comment retracted.
1. Every word here should be capitalized
2. Can probably shorten the labels a bit.  Maybe just remove the word "one"?
Tricky. Can this be refactored into 2 runs() blocks instead?
Could we use a Deferred here instead? It would be resolved when extension loading is complete. It would also handle the case where the client attaches a done() listener after brackets.bracketsReady is true.
Tiltのほうでは引数はfileとして受けとる
Sinatraはenb, hamlなどのメソッド名が渡される
Ok, I don't get it still. What can I change to not have false positives?
what happens if we cachebust?
The name getUser is seriously misleading, a getOwnerId would be much better.
Please do not commit this file.
Looks good, can you add a check of `Character.isUpperCase(timeString.charAt(0))` here as well, that way additional `CharSequence` objects are only allocated when necessary.
Probably you should also display message for Facebook login?
Please stay formal.
You can combine those two imports.
Oh, I think it's great to get a 400 error in that case. It doesn't look like you need a special exception to do that though, since the 404 only happens on a ValueError. Is that right?
I considered that, but figured it'd result in many superfluous calls to `uuid4()` and while it seems fast, it's a bit of extra work. Though this way it's two saves for every new user, which is likely the most common. Hmm.
Remove the commented out span if it's not being used.
Perhaps `DEFAULT_FONT_COLOR`?
Is this last `a` class correct? Might it be redundant or can it have a better name?
I think these 2 lines should be bumped to the left 2 spaces
This whole block is in a verbatim block on https://github.com/humphd/butter/tree/t2340, but I don't think it's supposed to be.
I hate when tests aren't work
VETO.

There was a purpose to this being a lazy compare. If `_popcornOptions` doesn't have the `start` attribute at all, this will catch that, and `null !== undefined`. Perhaps it should be `undefined` instead.
Nevermind. I don't see why it wouldn't.
The only danger I see here is if we ever add a query string to our template genearation (e.g., ?autoplay=1) then this will double-add the ? character.  Probably not a big deal, but something to consider.
We also don't need this because volo will create the cache manifest for you.
Dam, I had a huge explanation of what happened, etc, but the wrong keystroke made it all go away.

So, here's the short version, the only actual difference between my code and @giantlock's code is that his `SaltReqTimeoutError` exception shows the proper message as mine got lost when I fixed the issue.

```diff
diff --git a/salt/payload.py b/salt/payload.py
index 4336aca..03369af 100644
--- a/salt/payload.py
+++ b/salt/payload.py
@@ -142,11 +142,11 @@ class SREQ(object):
         self.poller.register(self.socket, zmq.POLLIN)
         tried = 0
         while True:
-            polled = self.poller.poll(timeout * 1000)
-            if polled:
-                break
-            elif tried >= tries:
-                raise SaltReqTimeoutError('Waited {0} seconds'.format(timeout))
+            sockets = self.poller.poll(timeout*1000)
+            if sockets:
+                break  # we have got result
+            if tried >= tries:
+                raise SaltReqTimeoutError('Waited {0} seconds'.format(timeout * tried))  # but still have no results, so it's enough
             tried += 1
         try:
             return self.serial.loads(self.socket.recv())
```

@bclermont you should enable travis-ci on your repo's administration, under hooks, that way you keep track if the code properly build's and passes tests.
I thought about other options including using inspect/etc. to try to determine if getresponse supported the buffering flag. I ran an additional test with a larger number of requests and noticed no difference between the exception handling case and caching the failure so just removed the caching.
This hostname should not be in the open source version
CSS Sprite算是性能优化的一种方式，移动到高性能部分吧。
Dude. Your name goes first, Tim.
And redcarpet was not in our composes, right?

Pls consult with mirek if we need to do any changes in Koji (like untag a package or something). Nice work!

ACK
This doesn't look intentional.  Re-writing the gpl2 are we? ;)
I would use the name of the library instead of the github url
Yes, this is the (/signalr/hubs)  So it gets registered on page load and never again.

However, if the dev decides to create their own proxies via $.hubConnection we register the onstart when they decide to do hubconnection.createProxy.
Similarly, here ``from ...table import Column, MaskedColumn, Table`` ?
Decoding and then implicitly encoding back again using `locale.getdefaultencoding`?  I don't get it, and it could fail on some international environments.  Can't we just to `stdout[:40]`?  Or at least specify the encoding, a la: `stdout.decode('utf-8')[:40].encode('ascii')`  (I think ascii is ok here since we can assume its a githash).
Should be `LambdaCDM(H0=70, Om0=0.3, Ode0=0.7, Ok0=0)`
Please use "&Copy Signature" here.
Have you really tested that this works? I remember having quite some difficulties getting askFee to return a value over the thread boundary, and ended up with a pointer as the only option. AFAIK references are considered read-only by Qt.
of course, no. comment is misleading.
Moving this into `DCell::Server` was my primary goal. 
That way it is optionally spawned based on configuration.
This should be set to NULL in the default constructor to avoid segfaults.
missing phpdoc for all methods
Its really a problem w/ old github codepaths. Most of the new stuff uses a case-insensitive "primary alias".
Sure, done in 72ce098266260bd810d8f4b7d1eddac67f8caffc.
Ah, tasty feature, I like this! :-)
Mixing logical and binary & without parenthesis is scary, I've been bitten before :)
Please change to:

else if ((buttons & CTRL_UP) && saveDialogInfo.menuListSelection > 0) {

and similarly in other places.
Be careful not to add empty trailing spaces like this.
Adding the `keyring_loaded` check is great, but `keyring.set_password` can still raise a `keyring.backend.PasswordSetError` exception, so I'd wrap it in a try/except.  Also, might be good to add a config variable or something so it only uses keyring/prints the error message if people want to use the keyring (can have it default on but to always have ir print "keyring library not found" could get annoying and indicate an error that really isn't.)
Ah, right.  I missed the ',' in the first join, sorry.
We need to have our fallback import here (and in interactiveshell.py), as `nested()` is gone in Python 3. Once we drop support for 2.6, we can use multiple contexts in a single `with` statement anyway, so we won't need it.
return Iterables.tryFind(ImagePredicates.withId(id)).orNull()
remove sporadic extra newlines
Didn't mean to, bad style.
There's a bug here by using secure_required, it breaks the CSRF check (even if using csrf_exempt). I fixed this on a separate branch for the marketplace url, and is being verified.

So don't use secure_required here, and add it as a decorator on the actual view (I'll comment down there too).
This is to make sure the browser is something we know about before we blindly add it to the <select/>

This is to fix an issue where passing a garbage browser in the fragment identifier caused a JS error. For example:

http://localhost:8000/en-US/kb/flash-113-crashes#os=mac&browser=garbage would crash js without this commit.
When is there a risk of `$diff` not existing? I tried to create a revision that only changed the tags, so there was no diff, but even then a "diff" was shown that didn't have any changes.
This is NOT safe. The order of `.iteritems()` and `.keys()` is unpredictable. It's rare that it occurs but it can't be trusted. I would just use:

```python
return dict((n, open(dumps_dict[n], 'rb').read()) for n in dumps_dict))
```
RICKY!
s/undefined/unused/
I don't understand this :)
Phil,

just removed the whitespace and updated my pull request
We already support `TypedFile` which is a `File` + `MimeType`. Let's just stick with that. I'll make all the related changes in this pull in another.
Should be `integer` :)
Why not close the stream here?
@Baachi The existing logic is also using ``<``
What do you think about the constants values @stof ? Do you think we should keep boolean to keep BC as I did first ?
I think cause otherwise exception is not throwed, but I will check it today again... I think It worked cause expectedException did not have "@"
I'm trying to RESTify the routes file, which means that they are, or will be, pretty much fixed.  /users/2 will always be the page about bob, whether that's Application.about(2) or Users.about(2) or the timeline, wherever that is.
big blocks like this are usually indentation or untabify, but I the original looks alright.  I don't know.
Yes, please! (See code copying comment above)
don't need this declaration here, it'll find it below
The apply_edition call now does not seem to get any dbg value passed in.
You can get it out of the __context: 

    let dbg = Context.string_of_task __context in
This is a vagary resulting from the following:

Production `yammer.com` accounts need to hit a *different* API endpoint than the `staging.yammer.com` accounts.

As such, these aren't *environment-specific*, rather they are *account type-specific*

There are places where we can combine use of the `.env` variables, though, which is helpful feedback.

Point taken on renaming `HOST` though.
Oops missed the z channel, that would be {GR, GRA, GRZ, GRAZ, RGB, RGBA, RGBZ, RGBAZ, INVALID}.

It makes sense for a grayscale image to have alpha or z channel attached.

The first four are grayscale images and the second four are color images. For some algorithms the math is different for grayscale and color. I would then have:

// call here the function to tag the image

if (color) { }
else if (grayscale) { }
else { }

You see where I am going? I feel uncomfortable thinking about "images with 5,6,... channels", I feel like we need to "classify" valid images for which we can apply our operations. We can even have some class called GENERAL if you will, but let's at least define it.
Oh, I see what you mean.  We presume that if the name of the file is the name of a format (or extension), then they are merely asking for an ImageInput that will read that kind of file, not that it is itself the filename. 

I'm willing to say for now, "If you have a TIFF file that is named simply "exr", you get exactly the kind of confusion you deserve."
Ah yes. Even travis told me that.
Ik vind het prima.
do we still need this cast?
The flask.jsonify() function returns a response object that sets the headers correctly and turns its arguments into JSON data. What else would a decorator do?
Well, not sure about collections but for models, backbone should be doing that automatically. Also we chose to list all available attributes or options on views and collections since it makes the code easier to read.
the gravatars werent fitting very well in breadcrumbs on the activity feed. it probably should have been height to begin with.
Im waiting 30 seconds for iris couch to make the new corpus, this means 30 seconds the user sits at a dashboard with no datums in the middle, and wonders what is going on. instead i made the user welcome modal hide much later, and they can actually read the green alert saying we are preparing their dashboard. @trisapeace @hisakonog @mecathcart please let me know how this feels interms of user experience...
Can you file a JIRA? Thanks!
OK so here we *do* want `function( event )`?  c/d
nit -> 'should handle empty developer object properly' ?

Not a big fan of "2" ;)
I just started seeing this yesterday, too. What's going on is marionette is dying on b2g desktop. I don't know why yet or why it just started happening yesterday. I'm hoping it's something I did by accident.
You don't need `this` here.
That's the cross-process mediastorage work.
(non blocking comment)

L10n issue: one should not expect charging note always come after the level in any human language.
L49 on top of everything:

    if (navigator.requestWakeLock) {
      console.warn('WakeLock API is not available.');
      return;
    }
`var DaysOfWeek;`
@fabi1cazenave  This has been done willfully, this `label` triggers the focus in another `input`, the real one to perform the search.
Sorry for only just getting to this. I think all we actually need to do to fix this is to change this 1 to a 0 (and change the assertion to 0 as below).