labofsahil/github-event-dataset-2012 huggingface dataset:
well i was allready busy trying myself but seems when you make a change you cant start the server anymore.

the knock knock joke and loads of other stuff can be added, meaning of life quote and such things..
Hi Pedro, looks cool!  

You're working on a rather old fork. I patched sth up specifically for your colleague these days that you'd probably want to include by rebasing to the current Eonblast/Emysql master. There is a difference that might affect you: there are more separate wait queues for each pool now.

Cheers,
Henning
POST тут ключевой момент
agreed. Retry would need backoff semantics anyways.
Thanks for that patch, I struggeld with this bug nearly always when I worked with acrylamid!
e.g. `2,fr` will call the function twice.
Hmmm ... fail :)
Ok, I'll remove this check and submit a pull request on Monday. 



On Jan 22, 2012, at 4:49 AM, Vilém Kurz <reply@reply.github.com> wrote:

> prior to this change validateItem check happened in send method only, implemented on almost all sharers individually. If validate item was not OK, function returned and all sharing process stopped silently. As sharer retained self, there was a memory leak. To avoid a memory leak for this path in code, I decided to add validateItem check to share method too, prior to [self retain].
> 
> Now we do not retain self, and this check can be safely removed from the share method.
> 
> Just realized, that these two particular sharers have overriden validateItem method, which does not allow a picture without title (or status). So the problem you mentioned can have only those sharers with overriden validateItem method.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/ShareKit/ShareKit/commit/f285a015f533fff8161d210cbe99d7b7eae9aa3a#commitcomment-889602
Dude, Love this! this is the kind of thing that changes the game. thanks Phil.
maybe some how for map Threads no?
Is .app.pid.2 extraneous?
Just to avoid future problems, there's no need of pushing to the global repository unfinished models. 
The workflow is meant to be:

Work on whatever you like on your own repository, once its finished and bulletproof, pull request to rgoraya wc_master master branch.

The main problem is that pushing unfinished stuff here will lead other people to try to improve your code and that effort becomes fruitless if the code gets changed again (+ tests that wont work etc...). That just happened to me today when I was going to run the tests but the code suddenly changed. It's good but try to avoid this!

Peace !
is this ready to be localized? if not, change this to `loc('Device Support:')`
Yaaaayyyy!!!! I proposed this to you a while back but you weren't too into it. What changed your mind?
Why does escaping start at true?
can you remove the system.out calls?
My server keeps hanging up after a couple of hours with the latest bukkit.
It gives me read timed out and nothing else.

I'm nut sure if its bukkit or one of the plugins.
Plugins: Essentials, Groupmanager with bridge, EssentialsSpawn, Lockette and my own plugin.
jaina and sylvana not scripted... event not work
Good! Let's keep going a little bit further before we take this to the docs (but not too much further so that nobody beats us to it). I think Usage can be broken down into sub-sections:

 * Storing configuration parameters
 * Storing / instantiating objects (should show a simple class example and an example with an argument - use a parameter for that argument)
 * Dependency objects (show how different objects can depend on each other)

Also, should we show how to turn a ContainerBuilder into an actual dumped container?
Pranav, I assume we don't need this anymore because the RazorGenerator compiles the views? I just want to make sure we're still going to get view build errors when running Build-Solution, so we catch them early.
Any ideas fr testing? :)
Aren't we leaking the NSXMLParser instance?
I know, it was rethorical.
That's annoying that Guard doesn't specify it as a dependency.
fixed in a new commit, thanks.
Bonjour!  Tests are failing due to there not being a lexer provided here.  If you don't mind not having syntax highlighting, add this after line 928:

    lexer: Text only

If you do want syntax highlighting, you will have to implement a Scilab lexer for [Pygments](https://bitbucket.org/birkenfeld/pygments-main).
merge with TC-in sunday
I'm honestly split here... Raw variables give me a bad feeling, and I
usually trusty feelings. An injected configuration object would be the
right alternative.

The idea behind the 2nd configuration file is to make it safe to create a
more complex configuration (cluster events) without impacting the standard
index.php.
parece que lo hace bien..
Fixed in db90254d. Thank you.
I have been testing UDB / YTDB for 3 years on high loaded realms and tons of bugreports and feedback. But anyway, use whatever you want.
Странно почему то npc=35005 ( npc_toc5_announcer ) молчит. Или это только у меня?
We can delete this method
why do you want to remove the paragraph symbols? I think they are visual anchors and should not be removed, this is very useful to point to specific parts within the documentation
Yeah, I was planning on adding it to the dump_stuck task.
Other than the global (IT BURNS) r+ from me.  You should get a review from fwenzel too
ugh.  yeah, will do.
Please look at my last commit to fix the issue of the array_push($reqdfields, "ssid") missing the ; at the end of the line and yes I tested this code but when I added it to the commit over looked it.  SOOOOO PLEASE pull the fix.
I'm not quite sure how I made such an obvious mistake
Should be named "Canvas.js" (with uppercase "C")
why creating a link to github here instead of linking to the API doc ?
You should start putting newlines at the end of files.
Oh crap, it converted the signs to UTF-8 ~_~
@ahocevar Don't forget the release notes in https://github.com/openlayers/openlayers/blob/master/notes/2.12.md :-)
could do `SUFFIX = '_' + SUFFIX if suffix else ''`
Might be cache, but Ill double check.
Do you know if the http implementation in php handles redirections etc so that the php code will not see it?
Do you prefer my preg_match match version or your explode verison? we should probably do `if(count($http_result) < 2 || $http_result != "200")` also.
No problem.  Thanks to you folks for working on RoundhousE which does **exactly** what we need it to do!
any possibility this could result in resources held in `source` to not go out of scope normally and therefore memory usage to bloat?

* leaving no stone unturned :) 

/cc @yhahn @kkaefer
Rss 호출 시에 RssContentController에서 rssViewer로 이름 되어있는 CustomRssView빈을 view로 사용하기 위해 BeanNameViewResolver 를 사용하기 때문에 추가 한다. 

RSS Call TEST : http://localhost:8080/SpringMongoWEB/rssfeed (use browser) 이클립스에서는 xml형식으로 출력되지 않는다. 브라우저에서 테스트 하면 제대로 보임
It is the initial number of results.  It looks unnecessary.  Any access to a result higher than this works fine.
Yeah, I mean they only added it so every Permissions handler does things the same way, why WOULDN'T PEX overwrite it?
https://github.com/t3hk0d3/PermissionsEx/blob/master/src/main/java/ru/tehkode/permissions/bukkit/BukkitPermissions.java#L120
https://github.com/t3hk0d3/PermissionsEx/blob/master/src/main/java/ru/tehkode/permissions/bukkit/superperms/PermissiblePEX.java#L62
https://github.com/t3hk0d3/PermissionsEx/blob/master/src/main/java/ru/tehkode/permissions/bukkit/superperms/PermissibleInjector.java#L31
Blink needs pathfinding and that is not implemented.
When I tried to run 'ant coverage', I got an exception: "no such file to load -- ant/tasks/raketasks"... Do you know what I'm still missing?
doesn't xvfb always install "xvfb-run" ?
Maybe that might work
Basic usage
=========

When using JNI one typically uses javah to generate a C header containing the constants and function declarations for classes that contain native methods.

This patch lets SBT run javah to generate the header file for a given set of classes.

Setting `jniClasses in Android`  specifies the classes with native methods for which C headers should be generated.

The make environment variable `SBT_MANAGED_JNI_INCLUDE` can be used to refer to the directory containing the generated header files.


```make
# Android.mk
LOCAL_C_INCLUDES += \
        ... \
        $(SBT_MANAGED_JNI_INCLUDE)
```
This violates the principle of DRY: The plugin version is already specified in the top-level pom and repeating it here makes it harder to upgrade to a new version in the future.
No, it didn't make any difference.
Please do! Thanks for looking at the fork!

Sincerely,
Zachary Waldowski
Sent from my iPhone

On Feb 14, 2012, at 11:00 AM, rolinster<reply@reply.github.com> wrote:

> I just downloaded this fork and I am getting crashes on both the simulator and iOS device when I run the demo application.  I have Xcode 4.3 with iOS 5.1.  The demo runs for the most part and then crashes usually within a minute.  I can send you the crash logs.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/zwaldowski/MBProgressHUD/commit/0fd4e4692577104f80a3f1f32ac06920dd8c2895#commitcomment-965083
Not a fan of this fall-through (albeit clever) in this context; is there a reason why this was chosen over:
data[key]["duration"] = matches[3] || matches[4] || matches[5];
I am getting an error compiling. It looks to me to be with this commit.
'Unknown type name "MKNetworkOperationState'; did you mean 'MKNetworkOperation'?
nah there nothing at all wrong with that line... its win! :)
Nevermind I thought you were testing solely for attribute existence here not for a behavior logic tree.
You can define routing rules with HTTP verbs (GET, POST etc)
$route['test/test'] = "welcome/test";
$route['{GET}product/(:num)'] = "welcome/product_view/$1";
$route['{POST}product/(:num)'] = "welcome/product_edit/$1";

You can go on with classic rules like:
$route['test/test'] = "welcome/test";
In this case the HTTP verb is not important. The rule is based only with the uri.

If you want define the uri and the HTTP verb you can write:
$route['{GET}product/(:num)'] = "welcome/product_view/$1";
This rule is valid only for GET http method and uri like this product/22
If you have product/22 but with POST http method this rule don't match.

This is useful when you write RESTFul API (same uri but different HTTP verbs)

Help me to undersand if you like this patch or help me to test it :)
Thanks
thanks for breaking logout() into a separate method
This breaks when using mongoid
These are essentially every element that you would want the highlight to appear, if you want to add more its really easy
language-c-quote won't build on GHC 7.4.1 even with these changes because th-lift doesn't yet work with template-haskell 2.7 (th-lift is required by haskell-src-meta). Once haskell-src-meta and th-lift have been updated for 7.4.1, I'll push an update for language-c-quote.
Wohoo.
Hi alexo, thanks for your offer! I would love to create an Eclipse plug-in for all the cool things in wro4j, but I just don't have enough time for starting another project at the moment. I'm only working on jshint-eclipse on a few weekend hours, while almost all of my time goes into the Eclipse RAP project.

However, if you're happy with jshint-eclipse plug-in, why not making it the first part of it? Eclipse is all about modules, so instead of putting all the processors into a single plug-in, it will make more sense to have separate plug-ins, that can even share code and settings. When there's nothing left to do in jshint-eclipse, I might start working on the next thing, e.g cssLint, but no promises.

So, if there's anything I could do in the jshint-eclipse plug-in to make it more useful for your project, let me know!
ЕФГ видимо специально сделал всем назло. Ни iFrame, ни html5 (из-за неправильно настроенного сервера) нормально не работают с https.

`www.0chan.ru : server does not support RFC 5746, see CVE-2009-3555`
That may explain it then, it may not fill in all the right bits if you're using the web interface. Anything like that just on github would be on your profile page.
On the subject of validation... I don't think an image should be required. Just because it leaves the possibilities more open to people doing sound art, and people whose project might be in their fairly early stages won't be forced to upload an irrelevant picture "just because".  Since it's just the submission, it might be nice to know what's coming, but not necessary in the same way as the other fields are.
Only one empty line between class methods
It looks like you're right about count() : https://docs.djangoproject.com/en/1.3/ref/models/querysets/#methods-that-do-not-return-querysets

What if we add this to the top of the function instead:
```python
try:
    meeting_times[0]
except Exception:
    return False
```

As for ordered, we can make sure that we never call .order_by('id') in any of the functions that call cannotSchedule() (or any of the functions that call those functions, etc.)
and this applet doesnt exists
mental note: always do a diff to see if mpei decides to remove it!
Depois ainda vou completar esta spec com mais cenários, preciso dormir.
Pushed.  It was a pure error; other languages that do any() will fail if the array is empty, and it makes mathematical sense.
Maybe it should be fixed then?
Done - though allen has other updates.
$target was the correct terminology here. Please change it back.
weird spacings
Use scala.util.Try instead of Either[Throwable, X]
Should clarify what kind of properties we're talking about here.
I've copied that from somewhere, but makes no sense here indeed.
Just wrap in a RemoteTransportException and let the caller decide if it should be logged or not
I'll just fix this within the wip for docs here, and also do the reference.conf config FIXMEs.
Apache Camel Messages consist of headers, a body, and attachments. Attachments were never implemented in version 1.x of akka-camel, or as part of the rewrite so far.
I picked it up because there was an old ticket from Martin.

http://camel.apache.org/maven/current/camel-core/apidocs/org/apache/camel/Message.html

the attachments in the apache camel version (mutable version) of the message uses DataHandler as type for the attachments. These things are mutable so we can't just take them over to the akka.camel.CamelMessage class.
Perhaps a link to the docs on lifecycles?
Why does this need to be an inner method?
is the recursion here on purpose?
does that logging still make sense or should it be somewhere else?
I was wondering: is there the possibility that these two functions (`intersection` and `difference`) are called with lists containing duplicate items? If not, then using sets' methods like `.difference()`, `.symmetric_difference()` and `.intesection()` would be way more faster, as those are built-in.
Let me see if I follow the logic of this: sphere_eval is a function that takes a Sphere as its input and returns a number. Here, we initialize the Sphere object with a single vector: theta, phi = x[0], x[1] and it tells us the value of this function in that location in spherical coordinates? In what follows, you get back a 'peak' of this function for every seed that we placed? That seems good. One things to think about is boundary conditions for theta and phi: what happens here when theta, phi go out of their domain (for example, above 2*pi for theta). The Sphere __init__ doesn't seem to check for that, but maybe it should?
I kept this from this from earlier. I think the logic is that 20 out of 1000ish is small. I personally sometimes use different units, so that my b-values are on the order of 1-3, it which case it would be quite large yes. Is that what you mean?
Should schema-1.4 be changed? There's a new schema-1.5 below.
Like that you just make 'x' a list when it's not so your code can stay clean below where you actually test the regex.
Can we do these two calls in one atomic operation by making it $lte?
This line does nothing. The publish_section is retrieved because a subsection is created, but in this case the section is retrieved and nothing is done with it.
Fixed and posted for review.  Not sure if it is used, but it isn't tested.
hmm, I don't think it's a good fit here.  best use is substitute for super when super won't work (see http://erniemiller.org/2011/02/03/when-to-use-alias_method_chain/)  Isn't it just obfuscation here?
You will need to change this line as it violates style requirements.  Maybe you can use here doc instead?
Lets make this

def long_fn_name(arg1,
<tab> <tab> <tab> arg2...
I actually mean `yield len(1 for _ in values)`. This example is 2 kinds of broken.
The meaning is different.
So for now, we leave them both.
Rather than the _Workspace_Allocate, pre-allocate the entire set up front based on maximum number of key/thread pairs and then get one from a chain here.

On line 46, there is a {. Is it matched and we are missing the code? Or is this the formatting issue gedare is pointing to.
Rather than relying on the caller to specify the right tileset for the tile, I would prefer to have `Tileset::addTile` take care of creating the tile. So that this becomes:

    tileset->addTile(id, QPixmap())
I guess something like `qWarning() << "Error while reading map:\n" << reader.errorString();` will do.
I have copied the text from the method description.
So how about making the `$page` parameter default to `null` here and then only use it when calling this function from the `get_post()` action? That then allows us to set `$_GET['page']` as discussed in the Laravel  forums only when necessary.
This should be surrounded by braces, or else be on the same line as the `if`.
The current syntax works with 2.x and 3.x.
Right, I see what you mean now. Never mind my previous comment.
I think you need to get correct buf size by using `_SC_GETPW_R_SIZE_MAX` syscall()
missing empty line ;)
We shouldn't use a composite buffer - it's known to perform pretty bad and it will make hotspot confused because of one more ChannelBuffer implementation to consider.
We should first check if ChannelBuffer.hasArray() returns true. If not we need to byte copy it. Otherwise this may break with direct buffers.
I'llf fix the name, but comparing two maps does not make much sense.
That's my understanding too.
This needs to be changed to 7 for the backport, right?
ValueError might be more appropriate here:

>exception ValueError

>Raised when a built-in operation or function receives an argument that has the right type but an inappropriate value, and the situation is not described by a more precise exception such as IndexError.
§3.6.2 of the ANSI C standard (ANSI X3J11/88-090), declarations have to go before statements in a block.
http://php.net/credits.php
And a whole crapload of administration utilities.
Mirrors might not be using PHP 5.3. It's probably best not to use this anonymous function.
закомментированный код не нужен
А его вообще везде убрать, или только в этом случае?
const не по стайлгайду
Do you add it to every project you work on?
@gazay Yeah, I like your blog post about it. :beer:
What about ModelNaming or something more similar to what we have? @josevalim ?
Do we really need to assert this? I think the call above is only to reset the database state. I would remove the assertion and the reset inside the `ensure`
Here you could use f2py's templating:

The syntax <_arg=arg1\,arg2,\0,arg,\2> means the same as <_arg=arg1\,arg2,arg1\,arg2,arg,arg> -- one of the slashes escapes the comma, and apparently the syntax \0, \1, \2, ... can be used (but IMO shouldn't be because it's confusing) to refer to other values in the expansion.

<ftype> :: <_arg>

expands to

real :: arg1,arg2
double precision :: arg1, arg2
complex :: arg
double complex :: arg
you should use separate slots for the user object, and for the class that's used to read it
hence 
  (user :initarg :user)
   ...
  (user-cls :initarg :user-cls :initform gh-user)
Cool, can do. In Google open source code I recall being told not to include names.
Is there a reason that these were removed?
"allows provides" -> "provides"
`for m in unify(p1, expr, {}):`?  An extra variable wants an extra attention cell :-)
I'd rather expand this as "Yield all partitions of the sequence ``l`` into ``k`` bins." and leave a blank line after this introductory sentence.
This should be in bin/
We sure we want these values to be public in the sample code that will be posted for the world to see? I dont think we should have this code available for people to see with the keys that map to an RZ AppBlade test project.
```js
|| Buffer.isBuffer(this.source)
Yep, that's correct. Moving it back one indentation level.
Yeah, you're right. They're left over from earlier iterations.
simply push another commit to the same branch
What if the simulation is in the "decision phase" when you build the panel?
As with cut, mkdir does not know --parents, but -p in both Linux and FreeBSD systems.
should translate it in English
What happens if the action type is a report? Can that ever happen in this scenario?
I think you can release the parens with commas. 

The two-bit values, `00`, `01`, `10`, and `11`, represent one of the four quadrants of the system.
Strange indentation. Tabs?
endif and the block of content enclosed in it should be indented
Added function for error. Message is more for aesthetic purposes.
$row is not documented.
I'll need to rename the key to EMAIL_INVALID_EMAIL then I believe.
Typehinting for the `$options` parameter, also lacking docblock for this parameter.
Instead of redefining $result and only adding one new/changed element, just do `$result['avatar_type'] = $driver;` The other elements are already defined.
Yes I have, if you want it removed I don't mind doing that. But per the coding guidelines it should have one there.
IIRC This is only called from the UCP. I'll have to double check that, but unless I'm wrong, only the current user deleting himself will call this and therefore needs to be logged out.
Second function has been added.
@p What do you suggest? `phpBB/includes/cache/sql_driver/interface.php` instead?
Can you document the API here? My understanding based on past conversations:

1) nil means the check does not take targets
2) Empty array means the host has no targets available for this check
for which

```the product gives ``g``: ``g = Permutation.rmul(*f)`` where f[i] is one of the permutations of coset[G.base[i]] for i in range(len(G.base))```

The sanity of any re-write can be checked against this example:

```
>>> G
PermutationGroup([
    Permutation(8)(1, 3, 2),
    Permutation(8)(1, 5, 4),
    Permutation(8)(1, 2)])
>>> c=G.stabilizer_cosets(); p = Perm(c[1][2])*Perm(c[1][3])*Perm(c[2][1]); p
Permutation(8)(1, 2, 5)
>>> PermutationGroup([Perm(i) for i in G.coset_factor(p)])
PermutationGroup([
    Permutation(8),
    Permutation(8)(1, 2),
    Permutation(8)(2, 5, 3),
    Permutation(8)(3, 5, 4),
    Permutation(8)(4, 5)])
>>> Perm.rmul(*_.args)
Permutation(8)(1, 2, 5)
>>> G.base
[1, 2, 3, 4]

```
This change seems dangerous to me, though I'm still trying to work out how.  However, since this code will be going away (soon?), let's just ship it.
yes, I know, I said as much this morning.. :)

./node_modules and ./examples/**/node_modules are _excluded_ and everything else is allowed
Так лучше не делать. Для этого есть в jquery .click()
dominictarr's implementation of timestamp is almost the exact asme as your `uuid` function except your one is called `uuid`
Since none of the attributes in the attribute set reference each other, the 'rec' here is unnecessary. But I'm about to suggest a change that will do such recursive referencing, so you might want to wait to remove this.
provider lower case
This is a different test, and it may not be appropriate, for instance if another process cleared the whole Theano compile dir (the new test would return False even if the key is versioned).
Suggestion:

```python
    if key_data.keys and list(key_data.keys)[0]:
```

Rest looks good to me.
После `return` можно `else` не писать, экономишь на одном уровне вложенности
!!@config.amd
Hmm, I would've expected the catchall clause to return `true` here.  Does this mean no documents are replicated with the default settings?
should be called raptor_term_turtle_write
Should be self.sql_alter_column_not_null, I think.
So, API idea:

```python
class Meta:
    auto_register = False```

causes the model not to go in the default `AppCache`, then we can have a `cache.load_models(*classes)` as @akaariai suggests. This seems sufficient to make this patch work?
No dot!
maybe add a \s in front so it does not match `frameword_version = "1.2.3"`
That sounds like a bug. I'll take a look.
Should I also kill overloads like

```
[S when String?(S)]
forceinline overload UInt8(s:S) = ...
```

?
To resolve the ticket, this changes are needed
Sorry for responding later. My register mail is gmail, and i can't receive the mail from it in office.
The dp_ids should not be empty. Its initial value is null (the root node, or the nodes whose ancestor nodes are continuous), after a node's best split is a discrete feature, then an element will be added into this array. Thanks.
And if SystemInformation is for longer period than one function invocation (i.e., multiple SELECT func()), found entry should be rechecked if it is still valid.
Would an auto detect of this be possible, or will it just be some random serial port so you never know which one?
You should only need to refresh after indexing--not after deleting.
My mom taught me to use two space spaces when I was a child, or she would take away my cigarettes.
If they're not necessary, can you clear out these autogenerated comments?
Not saying my code is awesome, that statement referred to the RFC ;)
this came with the pull, but cannot see it in the latest file.
This freezes a post-processed copy of pymacs.el as per the packager configuration.  For @VERSION@, this is no problem.  For @PYTHON@ and @DEFADVICE_OK@, it might be more of a problem, as users installing the package might be using another Python (for the first case) or use Mac OS (for the second case).
I don't know if github is showing me a cached version or what, but this still `return_Value` and not `return_value`
Though now that I'm back in the code, I'm wondering if we should go the: "Perigee bright stars not being checked, no rad zone info available" route or if I should just hold all ERs to the 3 bright guide star standard in the absence of radmon history.  I'm going with the "Not checked" plan now.
Changed to 1 star(s) in 32c10ec
Save it for the Python version.
Probably better to use the path methods here. For example, in theory it could get a path with a trailing slash (though not sure if that'll actually happen).

    Path.GetFolderName(path)

Or something like that should do the trick. :)
I trust devs to go look up what volatile means.  I'm more talking about indicating the thread access pattern we're using that makes the volatile declaration necessary.
I was thinking of it.
If I did use TemporaryFolder  rule, then tested two rule. Maybe harder to isolate bugs if happen.

Should i definitely use TemporaryFolder ?
Why not delete this one, as well?
Replace `Method.toString` with `{@link Method#toString()}`
Seems good, Thanks for the advices
hm? it's about __FormConfig__Builder
Also... is this the same regex as in main.js?  Should share code if so.
Ctrl-Alt-H is "Hide Others" in OSX. That was our first choice, but it was taken... Ctrl-Shift-H is "Hide Sidebar", which arguably could be changed to something else.
Ok, so just for my own edification: the recent change that was the reverse of this diff (changing closeAll -> menu.close) was wrong because Menu does not have a close() method (only ContextMenu does).  The only API for programmatically closing top-level menus is closeAll().
I can't remember why the mousemove handler is installed this high up in the DOM tree. Why not on only the resizer itself?
Isn't this one called only from inside `dbg-node.js`?
Can you put this in the `.form-search` css instead?
Yes please - see also #16.
Make that `{%- if post.sticky %}<span class="new">Sticky!</span>{% else %}` (space after 'sticky').
what about putting this in the `default:` case of the switch (switches without default, you know...)
If not propagated, explain at least what has happened here. Exceptions are used to describe (most of the time) known brehaviour that sould not happen in normal conditions. The causes should be known and described in a message. By the way, don't catch Exception instances. You MUST try to be more accurate in the catch statement.
Removed from `default_error_messages`.
Can you put these two endpoints under /history? They don't make sense in the context of the app as a whole, for me.
Indentation looks wrong here too.
can you explain the text!dialog part of this, I don't get what it's doing
Can you fix this spacing?
Think it's worth it to save a bit of computing power here and only call `buildDataObject` once? It gets called when compiling the JSON object, and through `backupData` right afterward. Not sure if that would produce harder-to-manage code.
This indent is odd.
trackEvent's (apostrophe)
Let's just remove this completely.
@nvcVincent: See this? I thought we discuss this before? 

@nvcDeb: So we change this directly? but if we test it in XAMPP, it will show error in our localhost.
https://github.com/widmogrod/zf2-assetic-module didn't use the latest dev-master of https://github.com/kriswallsmith/assetic, which had an UglifyJS filter that I needed to use, so I cloned the zf2-assetic-module repo, and updated composer.json to accept any version of assetic.
吧session_sticky的相关部分挪到HTTP_UPSTREAM_LEAST_CONN_SHARED 之下吧
Rename dur to something that also tells us the unit type
Whoops, how did I miss that..
debug message? If so remove.
Okay, nevertheless I still think it best to match both versions given that hey should be identical. What do you think? @jk0 @hunter
nit: make sure that mozContacts exists.
Alex, Luis only has one head tag. He does have multiple header tags but I think this is correct. There should be allowed to have multiple header tags. Why not? In fact your story states it.
oops, I forgot to actually check with @demitri  and @adrn on this :) Since it's out now, though, @astrofrog or @taldcroft or @wkerzendorf, do any of you have opinions on this, also?   Right now, `__str__` comes from `__repr__`, so should we leave that or have it do what this (`string`) function does with default options?  And should `string` be renamed to `format`? (I would say yes, because otherwise it might get confused with `__str__`.)
It's actually a case of the documentation being out-of-date.  This used to be a factory function, but it was later changed to a metaclass-created class (but saying just "class" here is sufficient.)
Oh, I missed that this was a separate module.  Yeah, I guess it's less important then.  I probably would have given it a leading underscore, but if it's not in the "public" API (e.g. in the docs or at the `units` base level), then it's not really a big deal.  

Now that I understand this, though, I  think that `def_ptype` or something like it should be "public" (and in the docs), though.  No reason to not allow users to define additional physical types if they really want to (given that the machinery is already there).
It should be sufficient here too loop over `locals()`--no need to get the module object from `sys.modules`.
That said, I might also do this as metaclasses too, for the `TimeFormat` and `TimeDeltaFormat` classes.  That has the added advantage that any other module can define additional formats, and they'll be registered as soon as that module is imported.
All optional/conditional parameters should be written as below:

if params[:display_name]
  lb.display_name = params[:display_name]
end
The point is "increase priority" and "requires no fee" should, in my opinion, be independent notions. Somebody might want to bump up the priority of transactions from the Bitcoin Faucet a little because they like the idea of giving newbies a good bitcoin experience, but don't want to crowd out other high-priority/low-fee transactions.
And is this piece of the code necessary in non-fJustCheck mode?
Isn't a "var" keyword missing here?
Can we add it when we need it? I don't like things being there for the sake of it just in case we're going to need it. Bring it back when it's being used.
For the toy project I was using to implement this, the camelize function was turning location_id into locationId, if I remember correctly. I suppose I could have adjusted the property names on my iOS models. The shop I work at doesn't really standardize its naming conventions. Some of us use camel case and others use snake case.  Most of us snake it. Feel free to change it back, though!
You should be good just with NSYearForWeekOfYearCalendarUnit | NSWeekOfYearCalendarUnit
Please fix
should i add contact info? i have an email i could give.
I'd rather not add my name yet if thats OK
It would be possible, but it would require either the server to keep track of what (each of) the client(s) knows about, and workout whether it should send a snapshot along with a new image; or it just always sends a snapshot.

The first seems like a lot of hassle, and restricts the options the client has in terms of memory management (ie. if it's not using an image in the UI right now, it can't dispose of it even if its taking a lot of memory, because maybe the server will ask for it again later)

The second is fine for in-process and client-side loaded images, but inefficient for true over-the-wire communication of binary data.

So however its spelled we need lazy loading.

I guess you could imagine some sort of process where the client creates a placeholder to bootstrap the creation (ie. "I need object 'foo', but I don't have it!  Quick!  Here's a placeholder that will ask the server for a snapshot and build the actual object...")  The problem with this is what API the placeholder implements, because at the point we're asking for an object, all we know is its ID, we don't even know what class we'll be getting in the end...
Are those constants from some predefined list?
Let's add an `or None` here, so that we get None when they're not in an alliance.
Better to have 'alliance' be None if no alliance?
I buy that .. will you make the change?

On 27 March 2012 17:10, Thomas Kluyver <
reply@reply.github.com
> wrote:

> > @@ -403,6 +416,16 @@ def _greedy_changed(self, name, old, new):
> >          When 0: nothing will be excluded.
> >          """
> >      )
> > +    limit_to__all__ = Enum((0,1), default_value=0, config=True,
>
> An enum of (0,1) sounds a lot like a bool. Let's use a CBool  here, in
> case we want to add another level in the future (C for casting - tries to
> convert things to boolean).
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/1528/files#r609827
>
Just a heads up that this will conflict with my changes to `RMagicError` in #1864. I'd just use a simple `ValueError` here, and save `RMagicError` for errors actually running the R code.
sorry i don't understand what you mean.  we aren't testing   `input.equals(*anything*)`.

the only way it would be one line is if you replaced my inputAssociatedNetwork with the value i assigned it -- but then you exceed the line length so it comes back to being two lines.  if only java had the `?.` operator ... as in `return checkNotNull(input, "ipaddress").getAssociatedNetworkId()?.equals(networkId)`
is there an either/or here?  ex. objectID or name?  if objectID is not set, how does someone retrieve a blob?
I will move the code to super classes then

Will making changes in parsers according to later API versions break any existing code?
I am planning to change parsers according to newer API version
This makes sense. I've created an additional module to read the file.
Question though: right now I'm setting the path to the keystore file as the @Credential property (e.g, oauth.credential) since there is really no generic "password" (PKCS12 stores need a pass but others don't). Is this aligned with the common practice?
how is this file becoming present?  target/* shouldn't be in git
So the behavior is correct but the documentation is wrong. Will fix!
Please switch this to the AMO api before making the page live
Is necessary send this in other commit?
One thing you could do is just pull all the votes here, not only the helpful ones. Then figure out which replies have more helpful than unhelpful votes with some logic in python.
should this be self.binary?
Cool! Yeah, this is what I did for the middleware where the old external classes needed the old config stuff. Smart!
Logging imho be in the scala execution engine not the Java one as the scala one is also called directly.
Coding style: the last parenthesis must go to the next line :)
We need to keep this code for BC.
I also think that the second argument should be an instance of `InputInterface` instead of an array.
According the [specs](http://dev.w3.org/html5/spec/single-page.html#the-pattern-attribute) the pattern attribute always contains an anchored regexp, you should use `/^(.)\^(.*)\$\1$/'` which is probably good enough.
you should add ``= array()`` so that it is an array, not ``null``
`private[this] val ..`
I tried that first, but it wasn't working so I tried this as a last resort and it worked. I've made some other changes since then so it might not be necessary now. I'll change it and see what happens.
no need to cast to NSString
I think you have misunderstood my suggestion... as neither of your objections are true.  
The view holding the from and to fields (i called it fromToView, but it is not an outlet currently) is not moving, so my line: 

quietTime ? fromToView.frame.origin.y + fromToView.frame.size.height : fromToView.frame.origin.y, 

should be exactly the same as your line:

quietTime ? 200 : 161,

just more readable.. and less prone to breakage.  I am applying the "magic constants considered harmful" rule : )
no need for `else` as mentioned above
don't use camel casing with python.  (I know the agent mixes camelCase, but there's no reason to add to it)
Does this `P` has any special meaning ? maybe :P ?
Idem, add Box
Anyway the problem seems way deeper. I am trying to come up with a general input validation scheme that will work for ALL algorithms. I am so glad we generalized multithreading, I know I will be glad if I generalize this too.

I am going for something that will "tag" an image as {GR, GRA, RGB, RGBA, INVALID} where GR means grayscale and GRA means grayscale + alpha. I imagine this to be one single function that can later be used for all algorithms and put an end to this input validation and alpha cases nightmare.

Just to provoke some ideas, Chris how would you "tag" the spectral images you talk about? Do you see other possible tags I miss in that list?

All the algorithms I will do work with grayscale images, just the histograms functions take RGB. So having a general "classification" for images will help me a lot. I will propose some scheme, but just to give you a glimpse... What is a grayscale image? Is it an image with two channels and alpha channel specified? What about one channel and that marked as alpha, would you still tag that image as grayscale?
Despite the variable name, I think this can actually be a CRS. Would you mind looking into that?
you should put it in the doc instead
5 mb free. memcache. Oldest get's pushed out. 
issues is gist are live. you can't cache them forever. 
We'll have to make more intelligents/parametrable decorators.
(trim trailing whitespace)
Can you remove this inline style?
Looks good. Is there a unit test we should add to make sure this does not break again?
I'd call this onInstallSuccess :-)

But that's really nitting.
Could you add a data-l10n-id attribute, and add that id to the locales?
why should a `<h1>…<sup>` element be linked to a “favorite” star???
need a line break after `,`.
is `options[i]` an `[Object HTMLOptionElement]`?
nit: anonymous
Have you removed this line on purpose?
Completely subjective, but shorter and softer is what I was going for :)
It does, on status bar, a fix delivered by @lissyx in #1565 :)
importt -> import ?
UX Requirement. They want to add main cities in every Timezone (as Mac OS, or Windows). I'm gonna talk with UX Team for getting the whole list.
Fixing it... ;-)
OK. Looks like you have kept the icons on purpose.
Yes. They always had a gap, and I think it’s a nice feature (avoids typos).
`bodyText`
Does the site always have at least one binding?
Could we invert this? Not set the builder.Port in the constructor?
"for generating code" or "to generate code"
Nice @filipewl I will open a bug to do this :grimacing:
Advances should be superfluous, the buffer is read-only right?
It would be better to raise an exception if all ports are taken.
That can be done for "nearest" and "uniform".  Let me implement that as a separate test function.  And I'll also implement geometry regression test.
The original decision was made because Youtube Encoding suggests aac-LC audio encoding, which I was going to use the faac element for, but it's in bad-multiverse, so I opted not to use it.  I also couldn't decide on the appropriate element to adjust audio quality in the meantime.  As mentioned above, flvmux is not compatible with vorbisenc.
What I meant here is that the Config instance used to test the other settings do not include the logger settings.
Nope, generics do not allow passing in those through the list interface. See it as follows: How can you tell from an empty array of Object elements what type to cast to? There is no way to obtain the T type from the list genericly.
Huh.... Actually I just included the astropy.css from the main astropy repo so that the docs would build correctly....
FWIW, I tried a sequential nodetool repair a couple of weeks ago to bootstrap a new Cassandra node and it didn't look like it worked.  After several hours, the new server still had not streamed any content from other servers.  A regular (non-sequential) bootstrap worked just fine.
Would it not make more sense to raise a `config.ConfigError` in the constructor, if this is empty? Rather than constructing all the packets, just to drop them on the floor?
But this function looks exactly like `_full`.

One thing that might make it clearer what is going on, is if each function had a brief docstring that described what it did. Having would probably make it clearer how to factor the code.
This should probably have a more descriptive name, 'mkdir' is a well known name.
What's the usage of these two interfaces?
Surround c-style strings by QLatin1Literal constructor for better perfomance
Make qt.phonon to be available to found KDE's phonon, project must know nothing about this crap
Has already been defined. Return immediately and continue parsing the rest of the extensions, rather than wasting time reassigning the variable.
My review comment was wrong, in Python this needs to be 0xFFFFFFFF instead of -1.
Why projects_limit: 3 ??
I've already commented on your fork, so one more time did it here.
project.branch_names
As discussed - perf test this Vs. the setInterval viewport top lookup.
toggle for the overlay view
This is already imported at the top of the page
OK. , I propose:

```C
    if (DecodeTeredo(tv, dtv, p, p->payload, p->payload_len, pq) == 1) {
        /* Here we have a Teredo packet and don't need to handle app
         * layer */
        FlowHandlePacket(tv, p);
        return;
    }
```
I can't change the password on the server until the changes in the code are made. Please remove it from your copy. I removed it from the repository copy. If you can't, we should get the changes into production ASAP.
Last sentence is not clear, I don't understand it.
I'd prefer waiting for such asynchronous file API to be here before changing the layout of this code. It should be straightforward to do later.
Nit: is `content-permissions` not `content-permission`
I like this approach.  I think we should decrease the noise of the event stream, and fix bugs like this before opening it up
Is the intention between L239 and 243 to only update dialogTemplatesData if templates.generate returns a value? Is it possible for templates.generate to return a value on one run but not a subsequent one?

Would this be the case if a template is valid the first time it is compiled, a developer edits it into an invalid state, and an attempt is made to recompile?
Doing this here is exactly the suggestion I was going to make.  Nice.
__whimboo__ wrote
> So this is like a template? Wouldn't it be better to have it as a global variable in that file with a specific name?

Template? Template for what?
Should I move `treeView()` out of `paneTitle()` and add it an `Array[]` parameter? And use like `tree.view = new treeView(paneTitle.data);` ? (see `sort.js` at [MDN](https://developer.mozilla.org/en/Sorting_and_filtering_a_custom_tree_view "Sorting a custom tree view"))

> I know that this file doesn't really obey the coding style but please keep it as best as possible, e.g. indentation, body of object member methods.

Should I file a separete issue about that or a just a commit here would be enough?
Should be #@param string [String] the debug message

the name of the parameter comes before the type.
This will blow up real good when someone adds a new action, but didn't notice this hidden bit of invisible global state.
You should have a comment about why this is done - you and I know, but in six months I promise I will have forgotten. :)
This should be

    [result addObject:block([self.array objectAtIndex:i])];

if I am not mistaken
strange, new-line character? I am in linux, ok, let me redo it again.
Damn. What the hell? Kind of bike shedding here but at least reuse the memory slot by declaring `i` before.
Than where ?
Are both `bbox_polygon` and `bounds_polygon` needed?
I think that this model class must be in models.py
Yes, will apply recent @ezimuel patch
This should be called with each key value as argument
Do a positive match:

if (null==annotations) {
  initialize
}

return annotations
I'm not sure about the naming convenion for this.
You are not handling the string case after all...
Method signature states `ElementInterface|string`, but then you handle also `array`...
I'm thinking there's no good reason not to typehint the first three arguments?
Oops, ignore forgot to undo this :)
New PR for the formatting issue: #543
In my world, lower case starts all instance variables and methods.  Uppercase is used for types.   

Thus, apple is an instance of the type Apple.
Oops. I didn't mean to add the test. I'll close and resubmit
Needs a `root` prop, and add in the comment from the top level editorconfig file
calling this method 
get() seems confusing.
Shouldn't it be something like
create?
Should probably use `LOGGER` here.
The opening parenthesis of a multi-line function call must be the last content on the line.
Does PSR-2 really asks for 2 end of line at the end of the file ?
This is broken if someone defines ``cascade={"persist", "all"}``. You should use ``|=``
Why is a require statement here? Seems out of place.
It is the opposite. Enum works on everything that implements the
`Enum.Iterator.t` protocol.
We can allow people to type `Enum.t` for convenience, but it should proxy
to the `Enum.Iterator.t`. For instance, if we update the iterator to
support new types, those changes would be automatically reflected on
`Enum.t`.
Sure, i can change these all back if you want. 

The reason i went with this style is that it seemed to be the style used inside the docs. (see http://docs.python.org/library/stdtypes.html#mapping-types-dict for example).
I agree with Itkovian on this.
fix typo: "actual"
Last style issue: http_parser uses `type*`, not `type *` (i.e. no space).
why not hashCode() ?
please remove underscore before language
Ok, then you shouldn't make a new file. It is inconsistent.
Nope. I should be able to chose which user is getting the profile picture based on the arguments of the function, not who self.mozillian_client is logged in as.
Technically you spelt Cache wrong here.
This won't lint - `options` is already declared as formal param
It is an object whenever they are using HTML Video/Audio.
@kellyredding - do these TODO's still need to be done?
actually, not used yet
We need this to use hoistEither, which is pretty damn useful in the ```loginUser``` function.
https://github.com/theforeman/foreman/commit/a89c40ce147b83fe3398438dfea56ad70278db11