labofsahil/github-event-dataset-2012 huggingface dataset:
я бы рекоменовал деструктор сразу после конструктора помещать в самом начале. а потом все остальное уже
Extended the sortingDelegate with two more optional delegate methods:

`+- (BOOL)GMGridView:(GMGridView *)gridView shouldAllowMovingCell:(GMGridViewCell *)view atIndex:(NSInteger)index;` is called when a move is about to start. If this method returns `NO`, the move won't take place.

`+- (BOOL)GMGridView:(GMGridView *)gridView shouldAllowMovingCell:(GMGridViewCell *)view toIndex:(NSInteger)index;` is called when an item is moved to a new position in the grid. If this method returns `NO`, the insertion won't take place.
Because that's the sum of the data over that period.
I didn't notice this. Is it live? Does it work?
こ、これはw
Okay. Thanks. :-)
Good catch.  If you filtered by category:5 it would fail. I'll fix.
![jferris](http://g.images.memegenerator.net/instances/500x/14390606.jpg)
You might look at how IRC handles multiple servers on the same network. I do think we'll have to scale it at some point. Unfortunately I don't have much experience in setting that kind of thing up.

How much will the database end up affecting things? I can't imagine it's getting hit *that* hard right now. What's being stored?
These also look good to me, although I vaguely remember being promised cat-themed cards.
Obviously enough, `m_posLookup` and/or `h` are bad.

The changes @Venugh has done are pointless; he hasn't changed anything.
Expression `NULL == 0` always evaluates to `1`, both in C and C++.
When working with pointers, `0` and `NULL` is completely equal both in C and C++. When working with integers, the only difference between C and C++0 is that in C you can't write `int i = NULL;`
`if (a)` translates to the following: `if (a != 0)` (`a` is of any type). And `a != 0` evaluates to `true`, if `a` is non-zero (less than zero or greater than zero), and evaluates to `false` if `a` is zero. As I said, there's completely no difference here between `0` and `NULL`.
jQuery does camel case, but not the other transformations, for example:

    camalCase('data-x') === 'x' 

Checkout these two fiddles, completely the same code, first one with jqLite, the second one with jQuery:
http://jsfiddle.net/vojtajina/XUfRV/2/
http://jsfiddle.net/vojtajina/XUfRV/3/

So my point was, that we need to extract these special transformations out of jqLite, so that it works the same with real jQuery.
Lol you know it buddy! How's the py treatin you these days?

On Mar 8, 2012, at 4:01 PM, Ramon Quezada
<reply@reply.github.com>
wrote:

> the py?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/robhurring/dotfiles/commit/88cac2e0ed4544d313430edfd4f941a5b5e5e59f#commitcomment-1062943
`strip` is redundant here due to the first line
This means I added the newline; it wasn't there before ^_^
Changing transliteration rules is non-negligible - if people are using this to slugify their urls, then when they update Lithium that means that there's a chance their URLs will change.

I'm not a big fan of this.
Still testing. Source compiles, but no boot. Cleanly re-importing sources.
Why would you need this constructor?
Tenés razón, gracias.
Okay cool - thanks!
Inline structures again
This is the generic javascript footer code, the only things that get changed are $PIWIK_URL for piwik installation dir, and $IDSITE which is created when the install script is run.
cpanm, which is just a module installer, also runs the tests when it installs a module. It detects Makefile.PL and Build.pl, which should cover quite a lot of packages.
not sure why this would be true..
should be logged warning at least if encountered I think.
First thought - yes
Second thought - it may depend. Perhaps the result is checked for validity for some other reason.
After digging in the code, I did not find conflicting usage, so I'll do that.
i had no idea switchClass was a *thing,* but i guess it's also good i never used it either.
The term "checkout" is never defined.
I is already done in another commit with @PreDestroy.
``word.size - 1`` => ``word.size - 2`` ?

You can remove ``(word.size > 2) &`` because the for loop will have zero iterations if that test fails anyway.
On Mon, Feb 13, 2012 at 5:22 PM, Alexis Roche
<reply@reply.github.com>
wrote:
> I don't really see when this append_diag function is useful in practice. Could the doc string suggest a real use case?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/matthew-brett/nibabel/commit/cf288c31afb928e26523bb75547ef70ea63d92ba#commitcomment-962863

Haha - yes - it is used in nipy in constructing - say - 6x6 affines
from 4x4 affines.   I'd like to leave it in affines, for nipy
convenience, unless you think that's a very bad idea.  I'll update the
docstring, thanks.
Oop.   Missing var here.
y el import de esto??!?!?! ;) jejeje
I think every method that's using `wrapNative` has the same arguments signature so that means you can avoid `.apply` and use `.call`.
The theory is simple, but in practice it's not easy.

For example, I've not been able to work out an easy way to parse the
version number from the first cell in last table.
Implements this pull request: https://github.com/PEXPlugins/ChatManager/pull/2
Updated max/min calculations for readability.
Yes, the original files do build correctly and the design meets timing.  However, if a version of ISE >12.1 is used (I'm using version 13.3), many of files are automatically regenerated.  The key Coregen files are the .xco files.
I'll keep these methods. I think they are worthwhile.
Oh exciting! Looking forward to this seeing the light of day :)
Any chance you'd contribute this to the ZF1 documentation? This is incredibly thorough!
Ah. apparently DBPool's .execute() doesn't act like a printf and accept parameterized arguments, so that's out. Looking at wrapping in sqlalchemy now.
mapM_ putStrLn сделает все само.
Yes, but only if !next_target.target.all_relative.  If relative, then these should be left at 0.
Very cool, thanks!  I only looked at the html output, not the latex / PDF, but it looks like a very useful documentation system!
Couldn't you just use the logical or? if (message.Contains("^IPGET ") || message.Contains("GETIP"))
A few observations on your wro4j configuration : 
* you should only add configuration for non-default values, no need to clutter the pom unnecessarely
* you should probably comment the maven-wro4j-plugin, since the minified versions are not used by default.
* In the comments, add a statement about how users should be using/installing m2e-wro4j in eclipse
* Now, making the assumption users will comply, you can remove the m2e profile and
* not generate files in your source directory, but use target/ instead. That'll simplify your configuration even more. See https://github.com/jbosstools/m2e-wro4j/blob/master/org.jboss.tools.m2e.wro4j.tests/projects/p1/pom.xml
Do this in a separate pull request next time; I didn't know you made it.
spotted that small bug, here's the fix
mhm... couldn't of done alot of it with out the others though, 
xD i guess i should start paying more attention to my msn instead of irc's
What about 'def __init__(self, types=[], message=None) as a constructor and dropping allowed_types above?  If we followed that convention would it allow people to extend the class for filetype specific validators in the future?
Which version of casbah is that against? I'll dig into this shortly.
On Jan 31, 2012 10:38 PM, "Richard Wallace" <
reply@reply.github.com>
wrote:

> Just tried this out.  Ran into this exception right away.
>
> app[ERROR]: java.lang.IllegalArgumentException: can't serialize class
> scala.collection.immutable.$colon$colon
> app[ERROR]:     at
> org.bson.BasicBSONEncoder._putObjectField(BasicBSONEncoder.java:270)
> app[ERROR]:     at
> org.bson.BasicBSONEncoder.putObject(BasicBSONEncoder.java:174)
> app[ERROR]:     at
> org.bson.BasicBSONEncoder._putObjectField(BasicBSONEncoder.java:226)
> app[ERROR]:     at
> org.bson.BasicBSONEncoder.putObject(BasicBSONEncoder.java:155)
> app[ERROR]:     at
> org.bson.BasicBSONEncoder.putObject(BasicBSONEncoder.java:120)
> app[ERROR]:     at
> com.mongodb.DefaultDBEncoder.writeObject(DefaultDBEncoder.java:27)
> app[ERROR]:     at com.mongodb.OutMessage.putObject(OutMessage.java:142)
> app[ERROR]:     at com.mongodb.OutMessage._appendQuery(OutMessage.java:85)
> app[ERROR]:     at com.mongodb.OutMessage.query(OutMessage.java:44)
> app[ERROR]:     at com.mongodb.OutMessage.query(OutMessage.java:38)
> app[ERROR]:     at
> com.mongodb.DBApiLayer$MyCollection.__find(DBApiLayer.java:303)
> app[ERROR]:     at com.mongodb.DBCursor._check(DBCursor.java:369)
> app[ERROR]:     at com.mongodb.DBCursor._hasNext(DBCursor.java:504)
> app[ERROR]:     at com.mongodb.DBCursor.hasNext(DBCursor.java:529)
> app[ERROR]:     at
> com.novus.salat.dao.SalatMongoCursorBase$class.hasNext(SalatMongoCursor.scala:45)
> app[ERROR]:     at
> com.novus.salat.dao.SalatMongoCursor.hasNext(SalatMongoCursor.scala:145)
> app[ERROR]:     at
> scala.collection.Iterator$class.foreach(Iterator.scala:660)
> app[ERROR]:     at
> com.novus.salat.dao.SalatMongoCursor.foreach(SalatMongoCursor.scala:145)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mongodb/mongo-java-driver/commit/e7b58d101d0307d0a3c10a2af92d1ce31784a043#commitcomment-921263
>
I thought it was possible to import it silently but when I couldn't get that to work, I just imported. Guess using np.array is better.
Hmmm I wonder if there's one of these for IntelliJ... Is this file useful? ie. should it be wiki'd but not versioned?
If you use the send_request_cgi() function, it'll set the set the content length automatically with set_content_len_header().
So to compile against Qt 4.8 with patches, we need to pass `--qt-4.8` to the build script? 30 minutes with 2 cores will mean 45-55 minutes extra in our case.

I decided to provision it after seeing Ember.js using it. I think for a week or so we will use this PPA. I will look into building our own .deb package of 1.4.1.

Thanks for letting us know that this version is outdated.
omg neubi das haben wir aber schon mal besser gesehen!!!!
There's really multiple ways of doing this :) 
* No JS SDK at all, redirect the top window to the dialog
* JS SDK, modal dialog, reload of app
When used as a canvas app then there is no need to read any cookie, the PHP-SDK will read the signed_request posted to it right??
In this case there is actually no reason to run `FB.init` at all :)
well, the commit message is kinda weird.

should read:
"fix broken gps due to missing propfiles"
@hijazi man what are you doing here, why!!!!!!!!!!!!!!
Sorry this just happens if I forget to sync before committing. I'll learn one day :p
May be something more descriptive.
I think that "System.Data.Linq.Binary" is found only in the .net frameworks 3.5, 4, but not in silverlight.
Ah good catch. Fixed 1b537f9
This file seems extraneous.
jquery wasn't getting included. i'll roll out an update in a sec. be sure to fetch the changes.
By the way, you cannot really differentiate overloaded methods today as the only returnt he method name in the Node and not the parameter list
Dette betyr egetnlig at om man vil, så kan man lage en funksjon som heter now:

function NOW() {
  return new SQLExpression('NOW()');
}

Så kan man skrive $this->date = NOW();

;P
P.S. remember to retain it ;)
Hi Tom,

Shouldn't the include-dir from crespo device be included? videodev2_samsung.h is not found which is in crespo include dir (see cgjones/android-device-crespo)
As written in the ChangeLog: "All lines, including those that are not executable, of such a file are counted as not executed."

Unfortunately, your observation that people have worked out to not whitelist problematic sourcefiles does not match mine. Instead of adding more and more workarounds I think it is best to not bother including uncovered files to provide acurate information on the number of executable lines they contain.
Keeping in mind that I've reverted the longterm merge (random reboots), I haven't noticed a huge difference so far. It may be placebo, but performance seems a little smoother.

I still need to port some of the commits from board-msm7x27.c that would be applicable to our board file (which is still based on the 6010 baseline).

As for your wireless - you should be able to load the older module if you fake the kernel version - and don't significantly change the kernel build configuration from stock. For example, if I enable tracing in the (stable Huawei) kernel in order to use powertop, my stock module would cause the kernel to reset. Thankfully I can use the Broadcom source ;).

Speaking of that, I'm sure that the existing bcm4329 source can work on your phone; bcm4319 is listed in parts of the source, indicating it's support. It's just a matter of building with the right flags.
Dear Mr. Tpruvot

I got some error due to my building. I saw your reply other guy this on my same error.
you said  see my complie.txt. Could you tell me where is your complie.txt file?

Many thanks
resync and try again :)
FYI, there's a PR open for this. Commenting there so context is maintained.
This looks weird. Are you sure that the Makefile should be in the same directory as the manifest? I would put it above.
will be fixed with issue #47 8) where is my initial comment... hmm...
Good catch. I fixed this in 944c99.
The maven-site-plugin is pulled automatically by maven as part of the default lifecycle and should available in the parent pom. We are not customizing any of the default params, so not overriding anything here. 

I was hoping to able to create the project site on github pages which would have been available at jasig.github.com/cas. Turns out, that's not possible for two reasons:

1. The github version of the maven-site-plugin has very poor support for multi module projects and does not support a staging site (required for relative submodule links to work on the site, etc)

2. the github api only allows 5000 API calls for authenticated requests, which is not enough to push CAS site artifacts over.

So I decided to do away with it and simply just polish up the site a bit more and improve the doc.
1 Loc > 4 :-)

    def gossip(): Unit = selectRandomNode(nodes.toIndexedSeq filterNot (_ == selfAddress)) foreach gossipTo
How do they know what timestamp to use?
yes, fine
cool, nice catch :)

I really think we should re-write this regex to just glob the path after `/topic/`, programatically split on `/`, and then validate the components as `topic` or `channel` via the built in method in the `nsq` package.

```go
var urlRegex = regexp.MustCompile(`^/topic/(.*)$`)
matches = urlRegex.FindStringSubmatch(req.URL.Path)
if len(matches) == 0 {
    http.NotFound(w, req)
    return
}
parts = strings.Split(matches[1], "/")
topic := parts[0]
if !nsq.IsValidTopicName(topic) {
   // error
}
if len(parts) == 2 {
    channel = parts[1]
    if !nsq.IsValidChannelName(channel) {
        // error
    }
}
```

either way, lets add a test case
These are related to the replacements for current_user and as_user that may have been on the User class?  Not sure what it was exactly before, but these are the class vars for the replacements.
step in the wrong direction.  less readable and less idiomatic.  you won't see this kind of thing in the rails code anywhere
Backwards compatibility. I understand what you mean, but this is how it was designed, we do not change the outcome because we think it's better, we do what the dev tells the function to do.
"analysis at the next level" => "analysis to the next level"
This has the disadvantage of being forced to use just one interface. Maybe it's best, when I introduce a IPackageController2 interface, that distinguished between before_view and before_search_view.
This is wrong we should be adding this functionality via the modules not as some top level function

the template should be using data-module=... to attach the functionality not using onsubmit=...
that's a shame..  I'll ditch this bit then
I draw a wrong picture now, wait for a moment.
Ik wil alle output geescaped hebben ;)
Kan je 'm niet als submodule toevoegen?
Indentation:

```
X = {
    'x': 1,
    'y': 2,
}
Is this really necessary ? I may be wrong, but I think the tests already start in the module path.
这里的空行去掉
For now I would like to not add this to the global namespace. It may be confusing to have a chmod that works on guest paths, but not on datastore paths
I discussed it with him.
That's a good solution. I pulled this, to keep these nitpicky things out of @gidden's way. Maybe that was premature. I can fix these things (when I get home from the lab) and make a little cleanup pull request out of it.
failureDe_te_ctorCreationLock
Same as above-- uses more than one variable "output".
Hurrr. A bad habit of mine.
Ok, can it just send request as last argument then?
Why don't you simply throw?
this should be

~~~
var defaults = {
      enabled: false,
      baseURL: "not-set",
      idSite: 0
},
~~~

since you are referencing `settings.baseUrl` and `settings.idSite` without the `piwiki` wrapper
This is something that the instructors for each course could write themselves to format their test output however they want, instead of needing to format it for each testing suite.
It looks like one major difference is that PyErr_PrintEx calls into sys.excepthook.  Perhaps that's what's wanted, to allow, e.g., pdb to be started on an exception in the tracing callbacks.
I also meant to add a padding-bottom here to make sure the headers still had breathing space before the bottom-border, but it seems like I forgot. I think I was using padding-bottom: 5px, however, looking at it again right now, I realized that the bottom padding has unintended consequences on the sidebar, so maybe it should got within the .page-content hierarchy.
Если нужен foreach, то только  в верхнем цикле(у деревьев нет метода, возвращающего контейнер с верхними элементами).
Как альтернатива:
int diagramIndex = 0;
	foreach(const QTreeWidget *editorTree, mEditorsTrees) {
		for (int j = 0; j < editorTree->topLevelItemCount(); j++) {
			const QTreeWidgetItem *topItem = editorTree->topLevelItem(j);
			if (topItem && topItem->isExpanded()) {
				mSettings->setValue("Diagram" + diagramIndex + topItem->text(0), 1);
			}
		}
		diagramIndex++;
	}
не знаю, что и лучше
Then you can't set up rack-cache yourself. Adding the middleware just because it's in your Gemfile is surprising.
Here we are wrapping and Hash:

```ruby
>> Array.wrap({key: "value"})
=> [{:key=>"value"}]
>> Array({key: "value"})
=> [[:key, "value"]]
>> Array({})
=> []
>> Array.wrap({})
=> [{}]
```
why not new hash syntax?
Why `*@args`?
Perfect :green_heart:
ah shoot, you're right.
`scala.Cloneable` is a subtype of `java.lang.Cloneable`. Both would work, I think it doesn't matter too much in the end: anyway the `@cloneable` annotation is deprecated now.
typo: `root`
Ah, interesting - I didn't know this fact.
-- 
Aleksandar Prokopec
LAMP, IC, EPFL

Sent from my Android phone with K-9 Mail. Please excuse my brevity.

Jason Zaugg <reply@reply.github.com> wrote:

> @@ -54,6 +55,7 @@ import scala.concurrent.ExecutionContext
> * @see [[http://docs.scala-lang.org/overviews/parallel-collections/configuration.html Configuring Parallel Collections]] section
> * on the parallel collection's guide for more information.
> */
> +@implicitNotFound(msg = "Cannot construct a parallel collection without a task support implementation in scope.")

>Maybe I would place an implicit default in the parallel package object, so that people who import collection.parallel._ get it, and people who just call par don't get a default. That way both schools of thought can be satisfied. :)

An implicit in the package object is actually part of the implicit scope of all types defined in the package.

---
Reply to this email directly or view it on GitHub:
https://github.com/scala/scala/pull/930/files#r1196647
Fixed in beb91c8a557c3a6e90173bf2c4a71414bd04824c.
I get this error while building html docs:

```
Warning, treated as error:
/.../sympy/diffgeom/diffgeom.py:docstring of sympy.diffgeom.CoordSystem.base_vectors:2: ERROR: Error in "rubric" directive:
no content permitted.

.. rubric:: Arguments

 - to_sys - another instance of `CoordSystem`
 - from_coords - list of symbols in terms of which `to_exprs` is given
 - to_exprs - list of the expressions of the new coordinate tuple
 - inverse - try to deduce and register the inverse transformation
 - fill_in_gaps - try to deduce other transformation that are made
```

It could be that is simply caused by the mismatch in indentation. But I also notice that none of the "Arguments" sections show up in the generated doc. So I would try this syntax instead:

```
    Parameters
    ==========

    to_sys
        Another instance of ``CoordSystem``.
    from_coords
       A list of symbols in terms of which ``to_exprs`` is given.
```
and so on.
Why not just arg.is_nonnegative?
should this commented code be deleted?
I think this is not a very serious problem, though.  I believe the two paradigms mix rather naturally here, without too much interference.  And, obviously, mixing cannot be avoided altogether, just because SymPy is object-oriented.
`MatMul(*args)` calls all of the rules again. The only way to change the AST without triggering the constructors is by `Basic.__new__(cls, *args)`
Ok, so this PR should be good to merge?
Might help to add "cd utils" before this, and possibly add "cd opencmissextras" before the svn co line.
Remove the unshift above with your patch and I'm happy to merge this.
My bad; one is passing the LoanReport constant and one passes the LoanAuditReport constant.
the message was incorrect
Hmm, that's a good point. And this also affects #524.
I would remove the `$app->raw('mailer') === $self->mailerFactory` part of this PR.
@mytholog  Where is this class ?
Give this field a better name.
If you don't need the file above, remove this too.
I kinda like the idea of having a volatile creationTime set to 0 and have withWaitMeasuredFromNow() just set that variable to the current time.
cross reference.
I know it's a lot of plumbing, but can we add a `_depth_test(6, 4)` ?
Il est certain q
If, by some reason, there's an option that is:
0b110.
The bitwise will not produce a real problem, but the sum will.
0b110 | 0b101 = 0b111
0b110 | 0b100 = 0b110

0b110 + 0b101 = 0b1011
0b110 + 0b100 = 0b1010

Anyway, such mistake is easily spotted (I hope).


On 03-06-2012 19:07, David King wrote:
>> @@ -227,7 +227,7 @@ function main($id, $mode)
>>   					'REPORTER_NAME'				=>  get_username_string('username', $report['user_id'], $report['username'], $report['user_colour']),
>>   					'U_VIEW_REPORTER_PROFILE'	=>  get_username_string('profile', $report['user_id'], $report['username'], $report['user_colour']),
>>
>> -					'POST_PREVIEW'			=>  bbcode_nl2br($report['reported_post_text']),
>> +					'POST_PREVIEW'			=>  generate_text_for_display($report['reported_post_text'], $report['reported_post_uid'], $report['reported_post_bitfield'], OPTION_FLAG_BBCODE | OPTION_FLAG_SMILIES, false),
> If it works, then I suppose it's fine. I was just going by what I've seen
> done elsewhere in the code. I am not completely familiar with all the
> bit-related stuff and will need to look into it eventually. For now, as
> long as it works it should be fine.
>
> On Sun, Jun 3, 2012 at 12:29 PM, brunoais<
> reply@reply.github.com
>> wrote:
>>> @@ -227,7 +227,7 @@ function main($id, $mode)
>>>                                        'REPORTER_NAME'
>>        =>  get_username_string('username', $report['user_id'],
>> $report['username'], $report['user_colour']),
>>>                                        'U_VIEW_REPORTER_PROFILE'       =>
>> get_username_string('profile', $report['user_id'], $report['username'],
>> $report['user_colour']),
>>> -                                     'POST_PREVIEW'                  =>
>> bbcode_nl2br($report['reported_post_text']),
>>> +                                     'POST_PREVIEW'                  =>
>> generate_text_for_display($report['reported_post_text'],
>> $report['reported_post_uid'], $report['reported_post_bitfield'],
>> OPTION_FLAG_BBCODE | OPTION_FLAG_SMILIES, false),
>>
>> That: OPTION_FLAG_BBCODE + OPTION_FLAG_SMILIES
>> Is only equivalent to:
>> OPTION_FLAG_BBCODE | OPTION_FLAG_SMILIES
>> because there don't have not shared bits.
>> I.e.
>> OPTION_FLAG_BBCODE = 001
>> OPTION_FLAG_SMILIES = 010
>> OPTION_FLAG_LINKS = 100
>>
>> The bitwise OR is the standard(?) (at least the most logical to me and
>> what's teached). I don't know about the specifics and which one is faster
>> but the bitwise OR is what makes more sense to me. If there's something in
>> the coding guidelines that I missed, I'll fix it.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/phpbb/phpbb3/pull/786/files#r919889
>>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/phpbb/phpbb3/pull/786/files#r919963
>
hum.... That's quite hard, ya know. What was made is quite obvious in the diff itself. I don't know how redundant it must be, actually
Should probably use the `sql_table_drop` method in `phpbb_db_tools`
All of it but prepending group_ or user_ get stripped.
These includes are needed, `functions.php` is always included in phpBB itself, as for the tests, the test file should have the included for those instances.
I've changed it here and in the phpBB emails
It is used. https://github.com/phpbb/phpbb3/pull/865/files#L1R109
The failing tests should be modified instead of bootstrap and also the mock i created should be used: https://github.com/nickvergessen/phpbb3/commit/815e8ef9e173e24bd6d3d99321cc68e6d44f742b
strip the carriage returns on windows?
mie mi se pare mai clar cu paranteze decât cu __setitem__. La fel și cu delitem, ai putea folosi direct del request.session[i].

Alții ce părere aveți?
would it be better to just set it to the empty string? I really don’t know, just asking
I think you can remove `max` and `round`. They seem not to be used anymore.
TypeError: attributes.matrix.clone is not a function :(
This subset of functionality could be factored out and shared with the common routine extracted to support the previous comment.
Docblocks, inspringen (gehele methode)
OK, will be on the lookout for this in the future.
Why do you need to set DEPMOD=? Why didn't we need that before?
typo -- common
well -- ConfusionMatrix I use whenever I want to 'grow' it in a script... *Error one is obviously also of use, so they should be in public API
My original concern was actually that it is a Confusion**Matrix** and Confusion itself felt "incomplete", but I guess having in mind that this would spit out a dataset. Matrix suffix indeed might be redundant enough
be careful about naming your vars things like ```obj``` (here) or ```element``` (as you did below).  Sometimes these are reserved keywords and you can overwrite their behavior. (```obj``` is OK, but ```Object``` would be a problem! same that ```element``` is OK but ```Element``` would be a problem.)
http://www.quackit.com/javascript/javascript_reserved_words.cfm
you could streamline this block of about 40 lines of code (535-568) if you just said something like

if arg in [PAGES]
  t.setTitle = arg.titleText
  b.setHidden(arg.shouldHideBackButton)
  Ext.getCmp('viewPort').setActiveItem(arg);
else
  throw an error for an invalid page name

and add the appropriate fields to your PAGES var.
Could be confusing keeping this comment here
It does. If you run tests from anywhere except the theano root directory this test will fail. This is posted as an open question up above. I'm not sure what the best way to do this is.
Remove first ``a``.
I cant make it without this parameter.
Missing test cases: patterns that include adjacent specials like '%%g', '%_a%', '%___c%', 'a_%b'.
You have to escape names and such if you generate HTML by hand.
please use google translate to handle english errors.
Maybe just: <code>type_ = int(values.get('type', 0))</code> will be enough?
:'<,'>!sort
s/defind/defined/
What about asynchronous exceptions (e.g. if someone tries to kill this thread).
It is still a warnings. But casting as `atexit((PFN) (uintptr_t) dtorLlvmFun);` is OK.
Use `#pragma clang diagnostic` to locally disable these warnings instead of disabling them everywhere.
For consistency we can't.  It must pass through the chunk of code that can read html attrs.
That is an excellent point, but probably out of scope for this batch of changes. I'll open a new ticket.
Or in other words: Is your script ready for inclusion? If not, how much work is it to polish it?
And now the problem is SRF_next throws C++ exception, it should not intersect with PG_TRY() block.
typo: tyoe-safety
And: The `handle_data` method is really just there to have a convenient entry point for the tests. All regular use will be through `handle_json`.
The name for python binding options are split between "--enable-python", "--with-python", just "--python" and a lone "--enable-python-bindings".

I tend to prefer shorter option names; maybe just "--enable-python" here instead?
This is outside both the class or an instance, though.
The block form mkdir doesn't remove the directory when it returns, does it?
Probably should allow an atom for `Cn` as well in case of a registered process. And since `gen_server:call` will crash in either case if `Cn` is not good, I'd be tempted to just remove this guard. In general, I'd expect some of the guard behavior at the edges of an api, but less so internally.
This would create a hard dependency on openssl support in php, using OC_Util::generate_random_bytes provides a fallback for it.
Idem, `$login` n'existe pas, c'est ici `$this->nickname`
Thanks for the docstring
the ease of grabbing strings for localization from source seems seems to be an odd reason for moving away from the well proven approach of keeping things together that work together ... eg documentation and code (see POD) ... so  keeping all the exceptions in a separate tree seems odd, especially when I think of adding new exceptions for my own classes, or would that work differently ?
Good catch - and very Rubyesque. ;-)
Conflicts with `import javax.lang.model.element.Element;`
I'm pretty sure the DASBiT style is camelCase for methods :)
Unless listeners are added and removed often, a CopyOnWriteArrayList should be much faster. The read operations for COWAL are non-blocking; in iterator() it relies on a single volatile read to ensure safety; after that, it's just indexing off an array. In contrast, the CLQ might need to skip over nulls in the queue.

For reference, here's the OpenJDK source code for COWAL and CLQ:

http://www.docjar.com/html/api/java/util/concurrent/CopyOnWriteArrayList.java.html
http://www.docjar.com/html/api/java/util/concurrent/ConcurrentLinkedQueue.java.html
Can you shorten this line?
Indeed. Didn't think going through albums would necessarily catch straight-to-wall posts, but now I believe it :)
Because it's a known small bounded size we want these to happen as fast as possible :)

The dal/pool code is the control point for how much parallel db activity we want, and these would ultimately be constrained by that.
Would be nice to stick these in an object and then iterate over it to set the properties--easier to read/less redundant code.
Do you still need the check to see if unload() exists before calling it?  I notice the check for load() existing is still present below...
Can we isolate the JSLint execution time vs. the DOM changes and repaint from resizeEditor()?
Instead of a lookup here, you could have a var in the closure to flag if #sidebar-resizer was already added to the DOM.
This is the same encoding function we use every where in Brackets so I'm not sure why it's an issue for this feature. Since this is just a bonus feature, can we add this issue to the followup user story?
https://trello.com/board/brackets/4f90a6d98f77505d7940ce88
So yea, make this 'PartialUpdate' :)
Why? We want support for messages? They do not work on MongoDB?
What does the Django setting reference documentation says about that. (Yes, it is commented in or out, depending which repository you look. Don't mind that.)
Yes, they kind of are. There was some thought process at the time, I'm sure.
Kann im [changelog](https://github.com/netcarver/textile/blob/2.5/CHANGELOG.textile) zu 2.5 nichts konkretes dazu finden.. 

Dafür lese ich gerade, daß textile 2.5.x PHP 5.0+ benötigt.. wie ist der aktuelle Status bei r4, ist PHP4 compat nachwievor Thema, oder hamwer das eh schon irgendwo/irgendwann gekippt?
`be` is the RSpec matcher to test that an object is not nil.  `be_true` tests that it is not nil or false.  So it's a slightly different expectation.

I don't actually particularly like the way it reads - `object.should be` would be better written as something like `object.should exist`.  Maybe 'exist' conflicts with something else.  This is the 'official' way to do it, anyway.
I'm not sure if we should follow the advice of rails style guide about rspec: "Use just one expectation per example"

https://github.com/bbatsov/rails-style-guide#rspec
Good point. I've changed it to this:
```
page.should have_css('#flash_notice', :text => "successfully created.")
```
The lines removed above is what broke the tests suite.
%ud -> %uA
Hm that makes sense to do this, but Rails does not use SQL CONSTRAINT, it only performs select. Therefore index is still not necessary if we have just few custom info record per each systems. But I have no problems with adding it, yeah.
Shall I assume this is ok?
I don't know as I have no idea of what the newrelic extension really provides. Do you have a link to the documentation of the extension ?
Address bar is a container for URL bar, I agree the naming is confusing. Have changed to URL bar and URL input, but eventually I think these names will change and will contain the word "awesome" ;)
Well, I think we've just decided to ditch this class anyway for 0.2, but your suggestins definitely apply for whatever we might bring in its place in 0.3 (`parse_coordinates`?)
uniqueQuery?
Perhaps I'm blind, where is the change here ^^?
`return $httpX === null ? false : $httpX;` is much more readable.
Oh that is a bit silly.
This is personal but I hate this type of construct. I think `&&` and `||` should used for boolean expressions only, and not for conditional executions.
CS issue here for the curly brace :)
sleekxmpp right there. Do you need this?
This should be /index.php as well.

It's a part of my PR though, just noting it here so I won't forget to fix it.
Could you also add a warning for this case?  We should be encouraging people to move to the new directory structure.
Ok, cool; I'll merge it in and we can play with it. Thanks!
The `CARET` state is interesting. When I was originally thinking about this problem, I was imagining more states.  For example: a `^` could put you in a `CARET` state, but then on the next character you would exit that into one of three states: `CARET_DIGITS` which you then stay in until a non-digit, `CARET_BRACE` which you then stay in until a matching close brace and then call a handler for the brace contents, or any other character which causes the caret to execute (i.e. move to parent) and go to the new state. Of course, you could break the `CARET_BRACE` state even further into `CARET_BRACE_SLASH` for searches, `CARET_BRACE_EMPTY` for `^{}` and `CARET_BRACE_TYPE`.

There is nothing wrong with keeping the number of states small, I guess. Essentially the `CARET_BRACE` state I was imagining is just covered by the broader `handle_caret_syntax()` function. This design does mean that you don't get to take advantage of the state machine for most of the syntax error detection and have to do more in the handler, but apart from that, I don't think it adds substantially to the code complexity.
ok, will change it to "regex".
On seconds thought you might be right :)
This should go down in line 73 or so.
If there are 1000 retries, this will perform 1000 round trips.  Just clear the sorted set in one go.  I'd suggest adding a `clear` method to the API objects.
it's odd to pass in null as the second param, you can just not pass it.
this  is a line comment
NITPICK: I'd rename to `update_sys_path` or maybe the even more verbose `add_root_parent_to_path` instead. "parent" by itself kind of implies the immediate parent of the task module or something.
здається це було для дебагу
Lovely search and replace. Fixed.
This is wrong. It will do the opposite of the previous code
I don't understand this change? Can you explain it? ping @eriksencosta.
You're not consistent here. Below you're throwing an InvalidArgumentException and here an Exception.
`;`, `,` and `=` must not be encoded  so the routing component can be used to generate path parameters. this feature is defined in the uri specification but I guess most people didn't know about that yet (me including until recently).
The other chars are only left unencoded for readability. But I think it's nice and would otherwise confuse people who for example specify a route like `/calcsum/{summand1}+{summand2}` that generates something like `/calcsum/x%2By`.
It has nothing to do with `option.user`. I think that this needs another name. How was the bot forced to use other repos before these changes?
remove this file from commit.
Ok, thanks.

I can merge it, just let me know.
use `!` inseted of ` == NO`
There wasn't one there before. I just cut, formatted and pasted.
Silky!
@JosephPeng 好的，我稍后就把挪过来
Anulează aceasta
It was, actually, my own change, from the Java interface delegation patch. I've sent both these requests simultaneously, so I couldn't rely on one's code in another, sorry. I always try to merge latest changes locally before sending the pull request.
Yes, it did in my previous tests.  We could maybe fix it but in my experience it's better to just have a strict format and stick to it.
Rather than hard-coding this into iv, I'm tempted to make a feature so that an app (like this) can query all the known extensions.  I think this is ok for now, but we'll put it on the list to clean that up so that every time we add a new format, we don't need to go edit this as well.
you could to this in one line with StringUtils.readFileToString(...) (Apache commons-io)
Well, I don't know what the InterfaceManager is. Would it be expensive to instantiate one each time? Does it need to preserve state across requests?
Here's the usage of ``warnings`` as noted above.
I'd prefer to see the NetworkUtils.formatPossibleIpv6Address done here rather than in DomainServerMain.
Don't miss the pending here (I did ;) ).
初始状态，不给用户分配服务器。用来测试这回新加的功能。
There are some indentation issues here
Note that the Action Menu building block (which I think the context menu is probably going to use for the time being) uses buttons directly inside a menu element, with no list https://github.com/mozilla-b2g/gaia/blob/master/shared/style/action_menu/index.html

You might want to change this to make styling easier.

Mind you, it also uses (form role="dialog") which doesn't seem appropriate here.
Can you clarify, please? because here I'm defining (registering to) a System Message and not a permission.
Sorry, I thought It was a button :S.
I'm using the name "open" for the analogous Camera->Gallery activity.  I think my open activity is disposition:inline, though. If this is disposition:window, then maybe using a different name makes more sense.
nit: 1 extra line
we won't needed if commented :)
Please make sure any changes here go to the upstream repo. And this is a formating output. 001 is of course treated as input.
I think something like this would be more readable:
```
handleEvent: function ph_handleEvent(evt) {
  if (evt.type !== 'message' || !evt.data)
    return;

  switch (evt.data.type) {
    case 'success':
       ...
       break;
    case 'error':
       ...
       break;
    default:
       this.showMessage(evt.data);
  }
}
```

Might be some typos in my code but you get the idea :)
I see what you mean by a boolean helper. I'll add that refactoring.
Works for me; I'm not a PostgreSQL expert anymore by any stretch (used to be, once upon a time, but since then...).  Just thought I'd ask...

On the BSON/JSON front, I'll have to defer to @lynxbat (it is his code).  I think the code to manage BSON came from the fact that MongoDB uses a BSON::OrderedHash to store data internally (thus the need to convert documents back to JSON when they are retrieved from MongoDB), but I'm not entirely convinced that the code that does this conversion (from a BSON::OrderedHash to a Ruby Hash) is still used.  Nick, can you clarify this?
fixed tests so that they're correct and using async.series in bb3217f
Why would you want to use bitwise or here?  Seems like you either want to use the status assigned above or use the `to_i` return value from `run`.  Bitwise-or is handy for bitmasks and such but I don't think you want to combine the two status values here, right?
Just another style suggestion:

```scala
try job ret job.task
catch {
  case x => job err x
}
```
It won't think it's a unit test, since it has arity 2. But I will change it anyway.
Also, re: Avi's first comment, I'm actually thinking of changing the joinWithTiny against the full sampledCounts to a joinWithSmaller, since I'm not so sure sampledCounts is that tiny.
Why render() and not view()?
Do this in the idiomatic Python way, using a `with` block.
done

On Fri, Nov 16, 2012 at 9:30 AM, Chris Bunch <notifications@github.com>wrote:

> In lib/god_interface.py:
>
> > @@ -0,0 +1,77 @@
> > +# Programmer: Navraj Chohan
>
> Add in your e-mail.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/AppScale/appscale/pull/99/files#r2154279>.
>
>



-- 
Navraj Chohan (Raj)
PhD Candidate @ UC Santa Barbara
http://cs.ucsb.edu/~nchohan
http://appscale.cs.ucsb.edu
http://nlake44.posterous.com
I don't see what code you see as duplicated.
Also AFAIK dblclick doesn't have a default action for links.
The `zen_ticket_status` variable name isn't clear. It seems like it's actually something like `zen_ticket_request_successful`
Yes and no, we need PredictableTransformComponent , just the line 62 are bad.
You're right. Cool.
Use `<label>`.
Feels like there should be a `iEquipmentDatabase.size() ` method.
prefer invoking internal functions like `_cd` over the user-facing ones (search the file for `_cd` for examples of usage). this avoids multiple-level wrappers, which would introduce for example nested try-catch blocks
Using a jinga template would mean that this logic live in the template, where it would be clear what it is doing.
Interpolate("%(SOME_JAVA_LIB_HOME)s") needs "prop:" in there, right?
isn't it more intuitive to call the joints and links pan and tilt instead of 1 and 2?
no need for persisted
This is also pretty out of sync with actual code now :(
This was originally copied verbatim from the original implementation. I've made some changes to make it more readable and renamed the function while keeping the behaviour the same. Basically all that matters is that the first listener that returns true or a string means the context matches, then that string becomes the new label for the menu item.
typo here, shouldn't say pushlog
Should this return a KeyCreated page object? The page that appears after clicking the create key button does appear to be a different page from the Create a Key page.
why not save the author to self.author as part of init, so that you won't have to specify it in go_to_author_page, just take it from self.author
What kind of performance game did this buy you?
I'm not sure `bool` is valid. We use to use `Boolean`
Ops, Phpstorm has automatic wrapping.....
As long as you raise instead, seems okay.
'because'
Same comment here as in the other specs, is this the correct version guard?
I reverted so that we'll actually have instances where we'll want to synthesize compensation events.  Because it can't use message information, it pretty much ends up just copying in the record-time state, but at least we can see it in action.  A solution for this problem is pretty much the next thing I want to look at, because at this point it's hobbling synthesis.
I think that you must move this test in the constructor, because the class is useless without this extension. Why do you test the function insteadof the extension is loaded?
Our CS allows it; we use it in several places where the functionality is an
implementation detail we want to call out as something to avoid modifying.

On Wednesday, November 21, 2012, Maks wrote:

> In library/Zend/Mvc/Router/Http/Segment.php:
>
> > @@ -25,6 +25,11 @@
> >  class Segment implements RouteInterface
> >  {
> >      /**
> > +     * @var array Cache for the encode output
> > +     */
> > +    private static $__cacheEncode = array();
>
> var name should not start with underscores
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/zendframework/zf2/pull/3045/files#r2198911>.
>
>


-- 
Matthew Weier O'Phinney
matthew@weierophinney.net
http://mwop.net/
Is that for security reasons ?
All the HTML, XML, CSS, JS specs defines it as case-insensitive.
Why converting each hex-escaped character to upper case than ?
когда ты пишешь Courier::Subscriber.where() у тебя не идет мгновенные поиск, созается просто объект к которму ты можешь продолжать навешивать дополнительные условия

лучше сделать так 

subscriber = Courier::Subscriber.where(user_id: current_user.id)
subscriber = subscriber.where(resource_id: resource.id, resource_type: resource.class.model_name) unless resource.nil?
subscriber = subscriber.first

в итоге меньше букав, потому что запрос в базу будет выполнен только на 3 строке
Given that we have setXxx methods for these, I don't think we need a constructor too. DI or SM can set it up correctly.
^ double `html`
Where is this used? We don't have any due date in our app spec.
yeah, the point was that you decide between getFileContent and getFileContentAsStream. though its not consistent with the setters, which might be annoying. i would be ok with a small BC break to clean this up. but the thing the current getContent does should not become public.
yeah .. imho we can just drop those. no reason to keep BC there.
maybe instead do instanceof check on $value and $value->xml?
Meant to say "big TODO" :).
It looks prettier to indent this.
I am not sure about this level of abstraction particularly not sure what benefit it brings. It's a fair bit more complexity for the same end result.
I'd be happy with just the 5 locators, it's readable and consistent with other projects.

Can I get a 2nd/more opinions?
"Picture only" is odd wording. How about "Only users with photos"?
Might be worth testing that city DOES NOT show up here, and likewise state and city for the country test.
can probably declare parts  up on #178 as well. Maybe name it something like `queryStringItem` just so we know what were using it for in the future.
Don't need extra space before function()

_queue.push(function() {
Use this instead.
```java
final TreeMap<Item, Integer> invMap = new TreeMap<Item, Integer>(new Comparator<Item>() {
	public int compare(Item i, Item ii) {
		return i.getWidgetChild().getChildIndex() - ii.getWidgetChild().getChildIndex();
	}
});
```
Should we tell the user what the limit is in the msg?
> The code you added will raise an exception with that example code.

Oops, I was misreading things. Please disregard this comment.
It seems like putting this on the scope config would be better for consolidating this, i.e.

```javascript
{
  ...
  "scope": {
    "size": "resource",
    "startTemplate": "asdf",
    "endTemplate": "asdf"
  }
}
```

Also is it weird that code is being output from this file where 99% of code is generated from file inputs? Files seem like they would provide easier maintenance if the scope templates a fairly complicated.
Because Eclipse. (The class is mentioned in JavaDoc, I'll remove import and use full name in the doc.)
Logical error here; we'll never run post delete, 'cause we've already returned.
nit: s/amount/number/
In both of the above, does the server return a success code?  For instance, a 204 or 200 or some such? We should probably match on the success so we don't accidentally return success on another 40x response that was not explicitly caught.
This will need to be changed to `/usr/share/java/mysql-connector-java.jar` which is where the `mysql-connector-java` package installs to. That location is a symlink to whichever specific version is installed.
错位了
`$('.passForm').submit(pad.savePassword)`!
А как вынести graph_->graph_[curr_node] в отедльную переменную если это unique_ptr?
非utf-8平台有乱码。一般先转换成unicode。