labofsahil/github-event-dataset-2012 huggingface dataset:
Bugfix for clearDestinationDatabase always true and new option --checkIdentityExists

- Bugfix for ClearDestinationDatabase. Option not set correctly and was always true.
- Added new option --checkidentityexists to avoid sql exception ([table] does not contain an identity column.) when reseeding on tables without identity column.
Express: Fixed bad status header

fixes #127
Allow to set header for apexrest function.

I believe it's pretty common to set HTTP headers when using REST. I added a parameter to the apexrest function to pass in a map of header names to head values. What do you think?

Signed-off-by: Alex Berg <chexxor@gmail.com>
Dev

Fixed typo causing clobbering of global variable 'o' in #_getPatternAmpm
Added zip.cfc

I tested actions zip, unzip, read, and I tested using the parameters. I had to add a couple things to base.cfc to get it to work for me. You might want to adjust my "for(var..." changes if you think those will cause scope leakage (my CF9 instance threw errors on those lines.)
added recipe for checking type of value is array
this line should probably be in another commit, as it's the one that does the magic
Nice fix. That's something I didn't think of.
谁叫你非的用button标记啊.....有空改
Does .detect run with Ruby 1.8.7?

Ideally we would like Fog bindings to with with 1.8.7 - 1.9.3 right?
These look good to me. Merge them but also commit to jruby/ruby repository?
can you please open a pull request?
Returns the shortest distance between self (an arbitrary Polygon) and o (a Point) or
the shortest distance between two convex Polygon objects.

```
Raises
======
NotImplementedError is raised in the case of two Polygons if either of them is not convex.
```
This commit doesn't work for me. There are now problems with file paths, e.g. if I want to run the Contao install tool I get:

```html
<title>Contao-Installtool - Contao Open Source CMS 2.11</title>
<base href="http://git.musicacademy.local/">
<link rel="stylesheet" href="TL_SCRIPT_URLplugins/stylect/css/stylect.css" media="all">
<link rel="stylesheet" href="TL_SCRIPT_URLsystem/themes/default/basic.css" media="all">
<link rel="stylesheet" href="TL_SCRIPT_URLsystem/themes/default/install.css" media="all">
<!--[if IE]><link rel="stylesheet" href="TL_SCRIPT_URLsystem/themes/default/iefixes.css"><![endif]-->
<script>var CONTAO_THEME="default",CONTAO_COLLAPSE="Bereich schließen",CONTAO_EXPAND="Bereich öffnen",CONTAO_LOADING="Die Daten werden geladen",CONTAO_SCRIPT_URL="TL_SCRIPT_URL",REQUEST_TOKEN="520757881bb20bcd2fb70ac1b9ac00f9";</script>
<script src="TL_PLUGINS_URLplugins/mootools/1.4.2/mootools-core.js"></script>
<script src="TL_PLUGINS_URLplugins/mootools/1.4.2/mootools-more.js"></script>

<script src="TL_SCRIPT_URLplugins/stylect/js/stylect.js"></script>
<script src="TL_SCRIPT_URLcontao/contao.js"></script>
<script src="TL_SCRIPT_URLsystem/themes/default/hover.js"></script>
<!--[if lt IE 9]><script src="TL_PLUGINS_URLplugins/html5shim/html5-3.js"></script><![endif]-->
```
Let's ditch TrackSharing and just call start_tracking() inline.
Well, I thought if rendering a frame is throwing exception we should instead just catch it higher to shut down Ember.
I think If 1 frame fails then all frames will fail.
Maybe it was a bad idea? Is there any reason to keep it?
I'm curious why this is named `_uiSetLabel()`, instead of just `_setLabel()`? But besides the naming, this could follow suite better with the idea behind `_getLabel()`. The way you have it allows for a `<span class="yui3-button"><input type="submit" value="Foo" /></span>` structure for a button, which I don't think people would expect to work, and might cause other issues.

So I think the `tagName` should be checked on the "parent":

```javascript
_setLabel: function (label) {
    var node    = this.getNode(),
        tagName = node.get('tagName').toLowerCase();

    if (tagName === 'input') {
        node.set('value', label);
        return label;
    }

    (node.one('.' + Button.CLASS_NAMES.LABEL) || node).set('text', label);

    return label;
},
```
From the point of view of the client application (for example Slicer), it shouldn't really matter if the project is retrieved from a build or installed tree. In fact, each project should be discovered using `find_package(<PROJECTNAME>)`.

This would work if the variable `<PROJECTNAME>_DIR` is defined. It should be set with either the build or install dir. And since if should work if either the install or build tree. Using just the build tree should be enough. 

Doing so would imply fixing the different CMake project we are working with.
The argument list in registerInstruction is wrong. Actually the syntax for the choice function isn't fully supported. As a possible workaround i suggest using "/SlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSlSl".
In PHP, I generally use something like this:

```php
<?php

$callback = isset($_GET['callback']) ? $_GET['callback'] : false;
header('Content-Type: ' . ($callback ? 'application/javascript' : 'application/json') . ';charset=UTF-8');

$data = array('some_key' => 'some_value');

echo ($callback ? $callback . '(' : '') . json_encode($data) . ($callback ? ')' : '');

?>
```

Note that you can use either `application/javascript` (the technically correct one) or `text/javascript` for JavaScript (anything works).
DP [i-k][j--k] indexing pattern
On 4 February 2012 10:27, Martin Dybdal
<reply@reply.github.com>
wrote:

> Kan man ikke logge på IRC fra Bruxelles?

Jo, men det gør vi ikke, da vi er svin.
Kan man ikke logge på IRC fra Bruxelles?
well done .
however, proofs of my stupidity will remain in previous commits... =)
could you add the `_build` directory to .gitignore and remove it from the commits? makes the code diffs clearer
Now using app.curl instead of child_process.exec directly.
The proper/eunit testing stuff really needs to be abstracted. They share almost everything except the tests they are running.
Shouldn't we also globally replace RS_SHUNNED with RS_REMOVED ?
I want to redo option handling to be smarter, allow name/values. Well, *I* specifically don't want to do the work per se, just want it to happen.
la dependencias... por favor!!!
This was reverted on purpose?
> how about writing it this way?

Does this also solve the problem you are seeing? I am running chrome 17 on linux.
Ik voorspel errors... de file moet ook met hoofdletter zijn (dit heb je waarschijnlijk lokaal ook) maar GIT is CaSe-InSeNsItIvE
Dus iedereen zal de file handmatig een hoofdletter moeten geven
seems to be issue here, busy testing latest on windows to see what errors pop up and with the latest I get this:

Traceback (most recent call last):
  File "bitHopper.py", line 317, in <module>
    main()
  File "bitHopper.py", line 237, in main
    bithopper_instance = BitHopper(options, config)
  File "bitHopper.py", line 86, in __init__
    self.workers = Workers.Workers(self)
  File "G:\c00w-bitHopper-7a05101\Workers.py", line 17, in __init__
    self.fd = open('worker.cfg', 'rw')
ValueError: Invalid mode ('rw')
here is the example:

```R

df <- data.frame(expand.grid(x=1:3, y=1:4), z = rnorm(3*4))

p1 <- ggplot(df, aes(x, y, fill = z)) + geom_raster() + 
  geom_point(colour = "red") + opts(title = "default", legend.position = "none") + xlim(0, 4) + ylim(0, 5)

p2 <- ggplot(df, aes(x, y, fill = z)) + geom_raster(hpad = 0, vpad = 0) + 
  geom_point(colour = "red") + opts(title = "pad=0", legend.position = "none") + xlim(0, 4) + ylim(0, 5)

p3 <- ggplot(df, aes(x, y, fill = z)) + geom_raster(hpad = 0.5, vpad = 0.5) + 
  geom_point(colour = "red") + opts(title = "pad=0.5", legend.position = "none") + xlim(0, 4) + ylim(0, 5)
  
p4 <- ggplot(df, aes(x, y, fill = z)) + geom_raster(hpad = 1, vpad = 1) + 
  geom_point(colour = "red") + opts(title = "pad=1", legend.position = "none") + xlim(0, 4) + ylim(0, 5)
```

output here:
https://skitch.com/e-kohske/g57x6/geom-raster-more
Curious, why do you qualify deprecated with 'scala.'?
Strange, it works for me in the emulator...what kind of tablet are you on?
Should i port that style correct to R2?
`fullClasspath in test` implies `compile in test`, so this shouldn't be necessary
Idéalement, le port sur lequel le serveur écoute en HTTPS devrait aussi se trouver dans la configuration et dans l'URL de redirection.

Ideally, the port the server listens on for HTTPS requests should also be in the configuration file and added to such redirects.
insertAsElementAfter() still very buggy.
ha, no problem. Those console.log() are pretty sneaky about showing up.
huch thx dir
Убрал, падать перестало. Спасибо :).
Надо ставить линукс...
Уберете этот лок в R2, или у себя в ветке special для фри убрать?
Found out its the Object.h part at https://github.com/mangosR2/mangos/commit/80512f92a172c9c6e06d276498670e3e331ecff8  bugging zeppelins and ships
<offtopic> If in my Server.log I read 2012-02-15 18:07:09 ERROR:WARDEN: RESULT MEM_CHECK fail CheckId 438 account Id 1000151722, what this mean? Dupe/wpe? Is there any place where look up this checkid?</offtopic>
Thanks for explanation. getStyle is fine.
`stdio.h` and `stdint.h` should included from `src/collectd.h`. Please add a guard around `kstat.h`, i.e.:

```c
#if HAVE_KSTAT_H
# include <kstat.h>
#endif
```
Crazy, you are correct.
I think this should use url_for to get the static directory.
хорошо бы `<form:select path="mode" items="${modes}"/>` поднять над textarea
Can you confirm the indentation for this? GitHub's review interface makes it look like it's only got 2 spaces, when I expect it should have 8. I suspect this is a tab. Please replace it with 8 spaces. Thanks.
Can you add a set of cucumber scenarios (e.g. https://github.com/projectblacklight/blacklight/blob/master/test_support/features/search_history.feature) for the case when a user isn't logged in? It looks like the case when there is a user is covered already. You might be able to even get away without writing new step definitions.

Thanks
t_version_id ? You forgot the $
Is there a test case to cover this line of code?
La formula para hallar los stats oficialmente es de la siguiente forma:
(BaseStat * nivel/50) + 5
It might appear useless but if this is failing something is broken. Makes it a good test not? :)
It didn't do anything, and W3C says that it behaves like a regular text field: http://dev.w3.org/html5/markup/input.search.html. Probably an extension added by someone to put in a clear field?
Great catch.  I'll write a test for it.

On Sun, Feb 5, 2012 at 6:00 PM, kprevas <
reply@reply.github.com
> wrote:

> "is not available"
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/carsongross/ronin/commit/155040afd73773b85846d09214034d6987d40ba4#commitcomment-935071
>
the point is I need raphael to run tests and as an assets for index.html (repo gh-pages and master are the same, easier to manage them)
Looks like whitespace changes. If you add a `?w=1` to the URL, it'll show the meaningful changes.
Also allowed user to filter projects by active/inactive from project overview screen
s/Here there/Here
s/this is//
Er, also forgot to mention that this will reduce the size of the XPI, too, since it won't include all the files specific to the website.
Another minor thing, 'unchanged' is spelled wrong here
They are unreliable. I'd just leave both of them out.
Just spotted you working on some interesting ideas in your master fork.

Before you go too far down this (master) track, I highly recommend reading over http://help.github.com/fork-a-repo/ and pay particular attention to the "Working with branches" subsection.  It makes Marius' integration work a lot easier when we submit pull requests via a branch instead of the master.

As it relates to openscad itself, have a look over https://github.com/openscad/openscad/blob/master/doc/testing.txt to see how to add basic tests for your new feature(s) while you're at it.  Since you're modifying an existing primitive (cylinder()) you'll need to pay closer attention to the tests to make sure your additions don't break expected default behaviour.

Looking forward to seeing what you come up with!

Andrew.
Also, on login, add_contact should be called, not update_contact. Weird stuff. Can you check what group they were in first? Because maybe a proper fix is changing the lines below your fix to


```
        # if we are in order by status, the contact was found and now is offline/online
        # delete contact from offline/online group and add to the oposite.
        if self.order_by_status and found:
            # y todavia no estoy en el grupo.
            if offline and group_found != self.offline_group:
                self.remove_contact(contact, group_found)
                self.add_contact(contact, self.offline_group)

            if online and group_found != self.online_group:
                self.remove_contact(contact, group_found)
                self.add_contact(contact, self.online_group)
```
genre tu fais des commit la veille du devoir d'UML ... confiant !
Why is the diff screwed up? 1572 deletions 1572 additions?
Hey @romac nice job on my homepage :D

j/k, actually I'm a cup of lamesauce for doing work as root on you know what server. Then again why is our root ssh key associated to your GitHub account? :x

Weird and weirder.
I'd say referrer check is redundant since it partly solves what CSRF does.
I'm a non-native speaker, maybe I just don't know :). Anyway, description is understandable. It was just to prevent a typo if it was one.
Да есть немного. С онлайн пользователями так и не разобрался, форуму нужна своя таблица online как минимум, а так вроде не плохо работает.
Please see this gist for usage https://gist.github.com/1613428
для тестовых целей
We should log this situation, so users can find why something is not working more easily. The first thing users do is they look into logs, so we should make this appear in logs.
Agree -- if I start using it again I'll pull the relevant bits.
Against magic numbers.
A method call that creates an object representing a new class and assigns this object to the constant Movie
The dev cycle was "Copy code to ListboxField to make it work properly with relations, find out its a pain in the ass, try to cram it into CheckboxSetField" ;)
I figured it was too much of a distraction to fix up ListboxField for the goal at hand, but done that now in 6e3ceefbb86b048bb9b12b414103fa578a8f2504, and reverted this commit. Also, I assume your envisioned MultiItemSelectField matches the existing ListboxField?
puddi puddi !
а как мы будем считать заблокированные urlfilter?
Brainstorming/ramblings:
Push the implementation into the Multicast client? Could the group information be part of the AddPeerSubscriptionMessage? Maybe sub-class the subscription message? Consume it on the other end and pass it on to existing code? This could eliminate the need to wait for a WorkerAvailable message before starting to use it. It would also remove a bit of setup, wouldn't need the UseGroupWorker, replace with group parameter in the subscribe methods. Replace UseMulticast with UseGroupMulticast?
Thought we worked out we didn't need that tl file?
What does this property do?  How/why should one set it?
Happens to the best of us ;)

Sent from my iPad

On 5 Jan 2012, at 19:15, Josh Goebel<reply@reply.github.com> wrote:

> Ah, duh. :)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/ianwhite/response_for/commit/81cde3e777bd9df8b0374dd30e08f0422f5a364e#commitcomment-841342
Cherry picked from 476b8d250d15bd0387d5f9fc57b5a257bd8f8ab3
yo ritchie!
I used to have almost the same code here. But decided it was probably easier to execute a shell command that returns the path of the executable rather than harding coding it in.

[check it out](https://github.com/mkhattab/emacs/blob/master/mustafa-util-func.el). I'm not sure if it's safe or the proper way, but who cares.
Introducing code this year should have 2012 copyright.
I didn't know we optimised named param calls when they are in the proper order.
Maybe playing with the suspendedDeliveryBehavior for the observer (in registerForNotifications) would help the blocking issue?
Nice! I like.
Ben, thanks for the detailed information. I was confused by the "(not)" in Steve's description of the change.
there's probably no need for this macro now (or never was any need) [[download|/tiddlers.wiki?download=tiddlywiki.html]] should do the job - customising the filename as the spacename seems like a nice to have.
:(

Fixed. Published 0.0.6.

:)
The only menu left that cant be navigated with keys is the options . But it is likely to change so often that theres no point in doing it yet.
Sorry it's taken so long for me to review this. I want to include something like this in the library, but I don't like the fact that the controller needs to know about cells (at least its implied by the delegate's method names).

I liked how now you only have access to the element in the methods, I think we just need to adjust the method names so it matches that. If I have time later today I'll include do this.

Thanks!
You can avoid making 2 calls (setCharacterEncoding, setContentType), setContentType will also set the character encoding.
Another option is doing setContentType("application/json") and a setCharacterEncoding(enc) after; that will render the "application/json; charset=%s" on the content type.
SkyFireEMU/src/server/game/Spells/Spell.cpp: In member function 'void Spell::prepare(const SpellCastTargets*, const AuraEffect*)':
SkyFireEMU/src/server/game/Spells/Spell.cpp:3021:12: error: expected identifier before '!' token
SkyFireEMU/src/server/game/Spells/Spell.cpp:3021:12: error: expected ';' before '!' token
`respond_with` will work here as well.
Yes, good idea.

Am 16.01.2012 02:01 schrieb "Nuno Job" <
reply@reply.github.com
>:

Do you think this makes it more readable?


---
Reply to this email directly or view it on GitHub:
https://github.com/dscape/node-clarinobjects/commit/eb290524edfa27879b196dea290a7fac1a5f9820#commitcomment-869528
Korrekt lösen!
Should this not be outside the loop?
Prob just "Hook for any code that needs to run after everything else in the view"
mhh maybe atm only a subset, we got to check in detail.
If your missing something, feel free to add it and make a PR or open an issue if you are not sure.
The goal should be to have everything available to be overridden via the view!
Also maybe we should change the namespace, so you only have one dict to override like:
```
    {{ form_row(form.select1, {'attr': {'class': 'span1'}, 'bootstrap': {'addon': {'icon': 'pencil'}, 'prefix': 'something'}}}) }}

```

i will have a detailed look intothat tomorrow
this doesn't make sense.
when the fselector scene is changed, you set the scene to the fselector ?!?!?!?
You know, in the future, you COULD send a pull request to let your upstream know you had modifications....  Not that it did any good when *I* did so to the movabletype account owner.  Or that time when I submitted all of those issues.  Pfft.

By the way, I wonder why you broke compatibility with MT 4 instead of simply providing a compatibility module?
For some reason the changes on HTMLHelper are not shown here, but they do exist
Please do not comment out -- drop. Then it's a clean merge. History of change is available in git log then.
Actually no, I think it's the test code that's wrong.
I did this originally, but it changes the public interface and breaks the symmetry with enablePlainHTTPConnectionManager, so I reverted it.  I'll push this as the final review comment so you can peel it off if you agree that it is not desirable.
@sminnee In this specific case, using OR connectives in a single statement.

```
$list->filter('Name.Endswith', $exts); // will produce AND connected statements
$list->filterAny('Name.Endswith', $exts); // one way to solve it, we already have DataQuery->whereAny()
$list->filter('Name.Endswith', $exts, DataList::OR); // another way, although that might become tricky in terms of arguments vs. flags
```

But more generally, two other options:
- Set global connective to OR rather than AND on a DataList instance. Already possible through SQLQuery->useConjunction()/useDisjunction(), but not exposed on DataList (possibly to keep the API clean?)
- Set grouped connectives. See Django API: https://docs.djangoproject.com/en/1.3/topics/db/queries/#complex-lookups-with-q-objects
I added `getSherlockActivity()` to all the fragment classes. It'll be pushed shortly.
A payment?
Note to self: ./repack_bootimg.pl ../arch/arm/boot/zImage ./ramdisk_galaxyr ../boot.img
"number" is geen bestaand datatype in PHP. int
The variable name "attr_string" does not follow the coding conventions of this project (using camelCase variable names)
NICE =)
omg i have search since 7 day , i have look UpdateFields.h and no UpdateData.h.
PS in UpdateFields.h CONTAINER_END  have 0x004A value and not 0x0003
No debería ir un url directamente apuntando a la vista? creo que debería haber una vista que decidiera a que elección redirigir, puesto que este es el comportamiento que el cliente quiere.
Good catch! cb6cbdc54577bd137d649ac3859d943806128567
so the atrix at stock only records 720p
not sure if that may be an issue because it seems to be set at 1080p
Let's remove 'Might want more details here?' and add more info in the future - if necessary
Hi @nvie , sorry for the late reply, been a hectic day.

Thanks for the changes! Small nitpick though, does ``enqueue_callable`` sound better than ``enqueue_call``? ``func`` is also not technically correct since we now accepts instance methods too, right?

I think ``enqueue`` is still the perfect API for this. What if do something like this to bridge the change to the more explicit model?

```
def enqueue(self, f, *args, **kwargs):
    if func in kwargs:
            self.enqueue_call()
    else:
            # ...
```

It's not 100% backwards compatible, but I think this would affect only a very minor number of users - those who use ``func`` as kwargs (or ``callable`` if we decide to call it that).

I feel like RQ is still in an early stage of development so some API changes may still be acceptable, if this would mean better API for future development.
Would a single word status be better? `"progress"`.
defaultValue should be true instead of "true"
Just for my information : the clearpass extension uses EHCache to store passwords ?
Alright, I'll just omit the fields in every case unless you know of a specific case where the field needs to be output.
Ok, I'll remove "in" (it's wrong), and change "lowest resolution" to "finest resolution"
I think the write-method should return a Future so that the logic inside the method could be extracted out, because all other transportconnectors will want to achieve the same behavior (sending to DeadLetters etc).
Why Vectors and not Sets?
say—just out of curiosity—did you have some debugging trouble inspiring this change?
Wow, this is definitely Dark Art.
yes, the order is not needed for this compare
What's the use-case?
It's needed in apache Camel land, but it should actually be private. will change.
behaviorStack is now moved to ActorCell
Oh, well. Sloppy docs are sloppy. I will improve the quality. Thanks!
I’m working on this (writing a preprocessor in our SBT build, seemed the easiest of a lot of crappy ways).
Maybe a short comment here that explains why we care about the uid?
Here you can write: "Supported tectonic region type is active shallow crust because the equations have been derived from data from Southern Europe, North Africa, and active areas of the Middle East", as explained in the 'Introduction', page 195.
Does this belong in the IO module?
Yeah, I'm probably the guilty party :)
Sounds good, thanks for the link!
I'll implement this and the tuple change as soon as I get home from work!
The comparison is getting quite complex; could you simplify the code into

    acceptable_length = @content_length && BUFSIZE >= @content_length 
    content = @io.read(acceptable_length ? @content_length : BUFSIZE)
start, end dates не нужны тут. Это для администрирования. Тут это не надо знать людям.
self.venture.get_kickstart_path() would work just fine.
It's slightly different. We could have a per "type" default (ie UDF defaults to on, or whatever), but this is a per *key* default. I suppose that these keys are pretty specific (ie won't be reused), so maybe that is the way to do it...
That stuff is already at the top, looks like a bad merge/diff to me.
I don't believe calling `QApplication::quit` is needed here, or did you experience otherwise?
After thinking about this some more I think the best way to handle this is to just call `backendEvents.emit("error", time_stamp, metrics, err)`. This will work with the existing backends for now, but provides a possibility to check for the error and handle it accordingly.
And as a matter of fact, I believe that could be shortened to the following: 

````
CGRect rect = (CGRect) {
    .size.width = 32, 
    .size.height = 32
};
````
as the origin will be initialized to zero point automatically if not provided. Anyway, it's irrelevant. :stuck_out_tongue_winking_eye:
Please document properties independently of each other.
Would it be simpler to make the name a property of the `RDD` itself?  It looks like you only fetch the names for RDDs that are members of `persistentRdds`, so you could just fetch the RDD then access its name.
done, except where it helps readability
API docs are not up to date.
это был не мой код - но поправлю
That causes the copy to be an instance of `FactKB` instead of `type(self)`, which isn't the expected behaviour.
I am not 100% sure, but that 274 should be part of the EXIF spec. PIL.ExifTags seems to exist only to help labelling EXIF constants (e.g., to display info about an image).  So I think the string value for that tag has a higher probability to be changed than the integer value in a later version of PIL (which could break the rotation fix code).
Oh didn't notice that part. I meant why refactoring `scala.collection.mutable` into `mutable`?
eigenvects are already sorted in master (after my changes) so I don't believe any of these changes in matrices or `test_matrices` are necessary.
Once again, I was replicating the presentation of "Kane". I have changed this now though. Thanks.
Oh, I guess a comma after "printing" makes this more clear.
I cannot see where I test it once more :-)

I have however done some reshuffling to `test_morphism` and dropped a few more repetitions.
Oh, this is probably just a moved file, sorry. Anyway, you may feel like correcting it while you are at it.
Actually, the checker supposedly does check for this, but for some reason, this file is not tested.  I also tried putting trailing whitespace on this file, and the code quality test still passes.
Yes, but multiplicity is known before the inner loop starts so it should be tested outside the inner loop.
You should include a file overview such as;

/**
 * @fileoverview Event Types.
 *
 */

(taken from goog.events.EventType)
`Всего {count} результат.|Всего {count} результата.|{count} результатов|{count} результата.`
We should not assume `controllerPath` is directly under `basePath`. People may not like the default directory structure and we offer the freedom to let people have their own directory structure. Now this change breaks this freedom.

Another problem is that this change assumes if a module class is namespaced, the corresponding controllers must be namespaced similarly. This is too big an assumption, IMO.

Namespacing module and controller classes is not a trivial thing. Let's think this way: in Yii's design, module and controller class names are part of the route (or user request). So by default, they are in the global namespace, unless we explicitly state their namespace in some way. 

Relying on the owner's namespace to decide controller's namespace is a big assumption, as I wrote above. 

One possible solution I can think of is to add a property named `controllerNamespace` to CWebModule.
Good question. Currently nothing is calling this explicitly. We should probably wire this into the Configuration frontend. This doesn't solve the inconsistent state that could occur if the actor is stopped in the meantime.....
Why would a NoopScalingStrategy ever be provisioning?  I'm confused...  Is this mixing test mocking concerns with a Noop strategy?
I don't like these previous/next relational reference. use hyperlinks/cross references, and describe the original purpose or query.
maybe rework a little
[0,0] = ``[0,0]``

replace throughout as needed
yes, command is $center

btw, I don't see this line in the current version... is this a comment on an earlier commit?
kill aspirational documentation
could we use something like https://github.com/django-mptt/django-mptt ?
indent after condition
That is not part of this branch. Should it also be changed?
thanks @naderman  @igorw  have fixed these things.
Function descriptions are meant to be readable in isolation. This description is not.
devolopment should be development
This should be defined in constants.php
What part about them?
"a user"
Why do you think that it doesn't belong in the avatar manager? The user or group data contains the content of the user's or group's database row in the users or groups table. In the users table all data is prepended with user_ while it's prepended with group_ in the groups table. In order to allow avatar drivers to handle avatars without having to handle this difference clean_row() is needed. It's not just helping, it's necessary. Therefore I think it needs to be part of the manager.
Unlikely to work with postgres.
Perfect format, I like it.
If the user sets the Audio instance to play before `prepareUserEvent` is called then the associated HTML5 audio element will not play. This touchstart event handler we add for touch-devices will cause the audio element to play then immediately pause (<- this is the broken? bit). If the programmer then calls `play` on the audio instance it will play correctly.

I'm not sure what the expected behaviour should be though. The touchstart event will respond to anything on the document. I'm sure the programmer doesn't want their audio to start playing at a random place. In the scenario when they called `play()` they would expect it to play immediately. If they haven't yet called `prepareUserEvent()` then we cannot achieve this anyway because of the iOS limitation. It's best, I think, for us to just fail (not play) and then work when they next call `play()` (after `prepareUserEvent` has done its stuff), rather than start playing as soon as some arbitrary touchstart event occurs.
see other comments about delinting tests...
ina hame ro 2ros karde bodam ke akharin commit to laptop khoda biamorzam bod :(
in akhare asan ye joore khasi my computer o baz mikard ingar midoones akharaye kareshe
так у тебя крон тоже будет 2 действия делать же...

> почему? вполне себе можно обновить. И запрос будет не такой уж и сложный.

покажи пример запроса пожалуйста, учитывая что у тебя в таблице записи общей суммы. причем у тебя 1 запрос должен проверить и обновить статус всех баннеров
does it really make sense for user to type assert_fast = False in order to cause a crash?
Again a random print.
Why DatesotreService is not set from the constructor as a constructor parameter ? JobRepositoryImpl( DatastoreService ds )
Yep I think you're right :-) but it's fine for now=)

With the process_opt_something a little aditional might be that sometimes our action is co dependent on multiple options which might be confusing. Well we can always refactor.

And damn so sorry about the non snake_case - that's the side effect when your studies force you to write C++ :-/
@ndbroadbent 

How about this method used in travis-ci?

https://github.com/travis-ci/travis-ci/commit/57948007abdbf059bf7c6a2f24cc0b22db2ba619

In this way we can still assets-related gems apart from all the others
From Apple's Dictionary app:

----

`informal` an indefinite small number: [ as pronoun ] : _he hoped she'd be better in **a couple of** days_ | _we got some eggs—would you like a couple?_ | [ as determiner ] : _just a couple more questions_

----

It may still be an American thing, but determiners are a concept in all English dialects. I'll change it just for sounding weird, though.
I think it would be nice to have type parameters introspection (somehow you proposed `[T, U] overload changeTypeInBox(#T, #U) = SymbolRoot(T)[U];`):
I think it should be UnpackType(V[Function[[A],[B]]) primitive which returns something like [V, [Function, A, B]].
Could you describe shortly where and how primitives defined in compiler? Right now I don't understand where they are and their interaction with Clay code.

Then the protocol would be `boxType` and  `typeInBox` with trivial default implementations and overloads if user needs support for types like MyArray[n: Int, A]. After this `sequences` may be patched to use more general `boxType` and  `typeInBox`.
i will start working on that, thanks!
Really like what you did there.
This one I get... it uses `slice` to cast array-like `arguments` as an *actual* array so it can take advantage of `join` to generate the almost useless selector parameter for pushStack.
Is None an acceptable value to return? Or should this be initialized to something else?
We currently do not have a code pretty printer; it would be good if you could roughly follow the existing style.
That error means that the formula is not actually using clang, but passing clang CFLAGS to gcc; the fix is to actually make it use clang (also `:all` is not a valid value for `build`).
Ah.  I read the wiki page and was confused by that.

In that case, is the use of `share` below correct?
shouldn't this use e_dbname? (the escaped version)
Here using `internalExecutor` is just an optimization, you're right it's fine to remove it here.

But `internalExecutor` doesn't exist because of this, this is just the one place where it's used when we already have an app executor. So it's also the one place it's specified explicitly instead of implicitly, because the app executor's presence makes the implicits ambiguous.

The reason `internalExecutor` exists is to cover all the spots where we call `onComplete`, `onSuccess`, etc. but _don't_ have an app executor. Look for example at the implementation of `def failed`, but there are a lot of places we use it.

The pattern is almost always:

    val p = Promise[T]

    onComplete {
        // whatever
    }
   
    p.future

The internal executor is just running some quick internal code that converts `Either[Throwable, T]` to `Throwable` (for example in `failed`), or whatever thing like that. Then the app callback if any will have its own executor.

So in most places `internalExecutor` is not just an optimization, it's needed to call `onComplete`. The alternative I tried first was to add an `onCompleteInternal` `onSuccessInternal` etc. that don't take an executor, to be implemented by `DefaultPromise`, `KeptPromise` by running the callback immediately... but it's more code, and ends up being equivalent.
Updated. Added a `README` as well.
> 
> > Find one thing worse than the other?
> 
> Le sigh. Is there a way to tell perltidy in a comment or something to
> leave some block of code alone?

Yes... but it kind of defeats the purpose in a way :-(

    #<<<  do not let perltidy touch this
       my @list = (1,
                   1, 1,
                   1, 2, 1,
                   1, 3, 3, 1,
                   1, 4, 6, 4, 1,);
    #>>>

For certain sections where geometry is quite important (such as the
above example) I find this valid. However...

> I am starting to wonder if this is worthwhile, if before every commit
> it makes changes that annoy…

Please don't think to hard about the challenge this brings at this
point in time.  Not being able to find *quite* the right combination of
flags or support from the tool is initially an annoyance, but one that
with time I find disappears. In my case it has been replaced by the
sheer, utter joy of just not worrying about the structure any more.  I
was not even always consistent with myself and that used to give me
grief!

The thing is, no one else is going to have exactly your style so
without a clear guide or perltidyrc you will end up (wanting to)
reformat inbound patches anyway. A standard also helps with
collaboration from the other side: there have been quite a few times
I've wanted to post an author a patch, but gave up after not wanting to
work with whatever style they had, and not believing they would accept
a mostly whitespace patch. A .perltidyrc just *removes* a whole bunch
of lame conversation and lets us talk about the semantics.

Please give it some time - I think the benefits are real.
Yep, I saw this come in but distracted by work (how dare they! ;))

I'm cool with those changes, that was based off some sample code on how to do it, and I just copied because when dealing with temp file names I thought it was safer to keep it, but makes sense.
these two were a cosmetic change.
i added docs to all variable. i hope this would help.

Cool thing and expected was the preformance.
I measured exec time in old tests, when added 1 sec delay, and it was same as in previous implementation -nothing broken. 
New test measurements can be easily calculated, and really match the reality.
E.g. two Threads for classes improved the prformance twice having same thread pool for methods.
Note the postfix if. Classes have a default `onlyObserve` setting, which we only want to override if the definition explicitly sets it to something different.
Should just have one other test for the `hasOne` or `belongsTo` case.
Should I be able to simply define a lazy accessor for 'html'? If that accessor returns `undefined` until a response comes back, then this expression will evaluate to false and 'waitForHTML' won't be set. Should I need to know about 'waitForHTML' and set it to true myself, or alternatively should I need to know about 'readyToRender' and redefine that myself? None of these options seem ideal, but we might be forced into one of them (or multiple view classes...)

(Furthermore, the second half of this expression seems to not really change anything, since if 'html' is defined already, we don't need to wait for it.)
ah it is below... well after I deleted the plugin reference again this came out :-(
I noticed a related bug while testing this:
1. use the following code: <input dirname="" accesskey="">
2. put cursor right before the "a" in access
3. use code hints and pick "type"

result:
<input dirname="" type=""="">
Updating these classes need to be done earlier so that on Windows the folder is sorted correctly when first created.
brackets.inBrowser was defined and used before my changes. Should I still change it to brackets.config.inBrowser?
This `state` field is very implicit: it's not published in the interface, and it seems to have (at least) three different states that include string literals, `null`, and maybe `undefined`.

The old debugger used to have an `active` field that was a boolean. You seem to have more states now, so I'd suggest either making constants for the states (so they're all collected in one place and can be easily discovered) or adding an accessor like `isActive()`. The latter probably covers most use cases for external modules?
as opposed to what? the underscore at the end is to avoid colliding with any existing case properties. i took my cue from 'computed_'
Looks like it was a remnant of previous attempts to keep the wells the same height.
I think this should probably live in appstore/partials/ unless sidebar is a thing that's going to be across the site (but then why would it live in the appstore app?)
see comment in reports/models.py about this
Space is cheap, but not *that* cheap :)
I'll take a shot at updated the whole section and improving my example. I've learned a  lot of things that you can do that we haven't documented.
You made it public, you have to document it ;-)  And even if you set it private because of this comment, I want some doc for this class now :-p
this is a performance disaster..
System imports come first.
We should specify useCapture here correct?
font-weight: 700
Do all our editors work this way?  What if it's not the first instance?  Why are we not doing this byId?
Somehow my comment didn't take in the last review.  This should match the check above, in case we ever add new input types:

input.type === "text" || input.type === "number" || input.type === "url"
Is it possible to remove this empty end() function?
We should probably drop our license header into all of the files your added.
this should be done in another ticket
That's my place holder for the image icon.
Wenn du auch hier den default fuer den doctype angibst, kanns du die backwards compat erhalten..
It works (I can verify this again if you like)..they define the struct inline so I cannot create an instance of one like that.
this function returns IO (), so why are you returning counter?
I'm assuming we're discussing this because we want to make this as easy to
read as possible; do you think using $(shell ...) instead of \`...\` is
worthwhile? I'm unused to bash/make readability conventions so I don't
really know what's considered unreadable.
On Jul 26, 2012 1:09 AM, "Keno Fischer" <
reply@reply.github.com>
wrote:

> > @@ -10,7 +10,7 @@ default:
> >       mkdir -p $(JULIAHOME)/deps/SuiteSparse-SYSTEM/lib
> >       cd $(JULIAHOME)/deps/SuiteSparse-SYSTEM/lib && \
> >       rm -f *.a && \
> > -     cp -f `find /lib /usr/lib /usr/local/lib $(subst -L,,$(LDFLAGS))
> -name libamd.a -o -name libcolamd.a -o -name libcholmod.a -o -name
> libumfpack.a 2>/dev/null` . && \
> > +     cp -f `find /lib /usr/lib /usr/local/lib \`eval
> $(JULIAHOME)/contrib/filterArgs.sh $(LDFLAGS)\` -name libamd.a -o -name
> libcolamd.a -o -name libcholmod.a -o -name libumfpack.a 2>/dev/null` . && \
>
> Well, the problem is that make also uses `$()`. However, what you can do
> is use `$(shell $(JULIAHOME)/contrib/filterArgs.sh $(LDFLAGS))` which is a
> make intrinsic.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/pull/1058/files#r1242100
>
Yop, it means relation between those two manifests. It's different way how to write `notify =>`
+1 for getting the username/password consistent across each environment, this had been bugging me recently
I really do not like copy'n'paste code form wild internet without thinking about license and updates.

Can you please create package inflectorpy, give it correct license and require it here?
@kamisama the channel is defined by the Logger instance. It's the Logger name you give in the constructor. It allows you to have multiple loggers that are used for specific purposes, while they use a same set of handlers. They give their name inside `$record['channel']`. Maybe you could use that by default, but allow overriding it via the context's type key or something.
This assertion does not test anything
No need to use start-stop-daemon if we are using upstart...
I like the idea of a `validate_shape` method that defaults to just doing nothing in the base class, but raises some error if it decides the shape is wrong (probably a `ValueError` to match other numpy shape mismatches).  Then in `BoolMask` or `SDError` that you have here you can just check if the shapes match.
虽然现在的area和faburen的data是entity，但是不倾向于记到这两个字段里，因为这两个字段目前只想记录跟这个ad相关的东西。目前看来area或者faburen都只是跟交易相关，而跟这个ad本身的关系不太大。
categories的确可以放进去，而且我之前自己写过一个放进去的版本。暂时去掉的原因是它暂时还不是entity，而且毕竟不是UGC的，我们自己还会把名字改来改去，感觉不够纯粹。
Can you move data_stream back to a local variable captured in the closure, as it was before I added the socket.on('close') stuff?
One windows "icon" embeds multiple sizes/color depths. An executable can embed multiple icons, for example the icons for different file types. However, Bitcoin-qt only has one icon defined in the .rc, so this doesn't matter here.
@laanwj As we want hex PRIszx is correct here?
You would know better. After taking a quick look, looks that way. I will update it now.
`Number` is the JavaScript type for numbers.
Typo `cancellable`
Module level constructs should be separated by 2 blank lines.
I do not think we can add such explanations as comments in the code, because we cannot keep explaining a utility we use.
I forgot to change it to 'tsv'.  Thank you.
typo: s/fc/ec/
There's a change in behaviour here if the user specifies an input variable that doesn't exist - before it would fail with a KeyError, now it will use None (or the octave equivalent) instead. I think failing is right, because it probably means the user mistyped something. It might take a couple of extra lines to get it to fail properly, though.
I don't think `caller` properly conveys what this is doing: that would imply it looks at the frame which calls this function, whereas its looking at the frame above that. `above` seems less ambiguous.

I agree that the more general function is probably sufficient for the limited use of this, though.
This is not how the code indicates that an option was set from the command-line. Are you just trying to distinguish how the value got set?  There may be a completely different way to achieve that, rather than using the "source" system.  That exists currently only for the --options reporting option; it really doesn't have meaningful value in any other way and could change due to that assumption in future.
I think `GIT_DIFF_NORMAL` wants to be capitalized since it is a constant.
We should probably create some kind of helper function which can give us the "basedir()" of a path. I imagine we're going to need that functionality a second time somewhere down the line.
The intro blurb is widowing the last word ("involved.") on its own line at desktop size. That's hard to avoid but maybe we can bring in the margins another column width: #partnerships #main-feature p { margin: 0 90px; }  It only needs it at desktop size, we could set it back to 40px for tablet.
The unit testing should clear out the database between tests. No need for these deletes.
Nice catch, I'll leave it as it was for now.
It seems the convention is to not have a space after the `#` for code that is commented out. See above on `#CEF_PRODUCT ...` for example.
However, verbiage should have a space after the `#`.
this stderr logging line doesn't have a counter part in the legacy collector ```.../socorro/collector/wsgicollector.py```.   Is this a debugging line that escaped deletion on submission?

If you really want to log the omission of extra minidumps, then rather than writing to stderr, you ought to use the logger.

```self.logger.debug('Removing %s from form', key)```

however, even that is somewhat inadequate, as it doesn't give any indication as to which crash submission has had an extra minidump omitted.  At this point in processing a crash, though, there is no ```crash_id```.  So I suggest moving lines 60 and 61 to a position above this for loop.  Then add the ```crash_id``` to the logging line.
I have a list of paths called 'dump_pathnames' - I'm only interested in the 2nd through the last one.  I have a function that will return a list of names based on a list of paths.  now I have two lists: names and paths.  I want a dict with names as keys and paths as values.  I use zip to combine the lists into a list of name/path tuples.  I then use dict constructor to make the list of tuples into a mapping of names to paths.
Good catch. Wrote some tests and updated. I think that's everything...
Rename to `enqueued`
Collapse these clauses since there is no difference in the follow on logic. So:

    {ok, _} ->
        delete_checksums(Checksums)
maybe pull the defaults into a defaults object?
Same as the one in step_3(), moved down below.
Now I could find the difference, '테' and '텐'.
Thanks!
In all honesty I think we should remove torches. Attempting things like this is just not good practice in bukkit.
oooh, and the RFCs:
http://www.ietf.org/rfc/rfc2045.txt
http://www.ietf.org/rfc/rfc2046
деталізуй селектор. наприклад ```.fos_user_registration_register .eye``` чи до форми його привяжи
The actual changelog should be ``Added the memory usage`` as it was not done at all in 2.1
@stof there is no such rule and the form component uses docblocks for private properties. Why should it hurt?
put the description as key of the array returned by the data provider instead. PHPUnit let you name the provided data and will then use this name to identify the failing test instead of dumping the provided data.
please indent on same level as first NSNumber