labofsahil/github-event-dataset-2012 huggingface dataset:
I just closed it. I don't see any new requests right now.
Somehow the comment-window swallowed some underscores, one more try:

(User.instance ~ Permission.instance *) map (_.groupBy(_._1) mapValues (_._2))
Why were these deleted, is there a replacement for a small sized indeterminate progress spinner?
Not the most constructive of changes ;)
It worked because I'm using a "all-in-one" build with one Ant task per project, and I have put the Ant tasks in the dependency order. But the wocompile task is using the same name for all installations, so it's not working at this level.
http://www.mmotop.org/emulator-repacks/8299-%5Bcataclysm%5D-deepshjirrepack-4-06-a.html
Kadootje is foutieve spelling. Het moet zijn : "cadeautje"
http://onzetaal.nl/taaladvies/advies/kado-cadeau
Diego,

if this was about changes to any of the language files, then we need to do this at the translation server located at : http://translate.librecad.org/
The reason I did this way is so that a theme author could declare their own `woocommerce_show_messages` to customize the HTML output but if you don't want that, then yes we can simply call `$woocommerce->show_messages();`. The alternative is to move the `show_messages` code to a template but my personal feeling is that the `Woocommerce` class shouldn't be dealing directly with HTML generation (the only other methods to generate HTML are `wp_head` and `output_inline_js`).
Great commit. I like it!
It seems there's something wrong with socket.io. On Opera, script fails with Undefined variable "io" when you initiate connection to actual server. Substituting your socket.io script advanced me to bug report alert, but that's it, nothing happens after that .
where else do 'list' and 'map' get used in their 'special' (vs. Pythonic) ways?
Sorry, couldn't find where Frame was imported from -- but I see now it's defined above as an inner class (why?), which my IDE freaked about.
Need to hack the way WordPress query. I open an issue for that (and I could reproduce the mechanism to avoid sorting by hand like it's done for the `Schedule` content).
I knew you couldn't keep up a serious conversion... disappointed (but not surprised).
It  doesn't need to be public.
my crappy questions finally brought this out :D
It's really not that hard to try executing your queries, is it? (not referring to you, Nayd)
Le echo un ojo a la validación de e-mail con html5, no lo conocía... La validación del mail con regex la encontré buscando en google - rails mail validation -, no me fijé en que fuera problemática...
Maybe try to use the "File.separator" property from "java.io.File" class, instead of trying to do it on your own. Also, this code doesn't support any other os, like any other Linux or BSD distribution. So, use "File.separator" instead.
Note: This alters the distance estimation to use ortigonal distance, as BS12 does not permit diagonal movement. It would actually be a massive downgrade if used for a bot that CAN move diagonally.
Why replace the long names with short names? Don't have you autocompletion ;)?
Looked at #187 - seems like either you forgot to uncomment it before commiting or you've done this intentionally. In the second case it's unclear why not add this as is. As far as I remember adding RewriteBase was a must for me to get symfony working with a standard ubuntu's apache configuration og virtual hosts (but that was more than a year ago).
Tested with Firestarter,won't work..
Going to check spell dbc as aspect of the fox works
Well done, thanks Gael and thanks continuous integration :)
Any idea when PHP_CodeSniffer 1.3.3 will be released? Thanks!
Hi Baylor,
Thnx for ur link. however, am facing one issue,  whereby i can't access edit/add through url like http://localhost/mvc/welcome/add. And i tried in submit form like this "<form id="login" action="welcome" method="post">"  but it shows "The requested URL /starkb/welcome was not found on this server." and if i leave submit form with action empty like this <form id="login" action=" " method="post"> it shows error msgs: 

The file _controller.php could not be found at
C:\wamp\www\mvc/app/controllers/_controller.php

Any advice, am using ur source code. Appreciate your advice. Thanks again
Good point, fixed in 726b9cac46e38c5a624841995474170e5aaa88fa  


Sent with Sparrow (http://www.sparrowmailapp.com/?sig)


On Tuesday, 28 February 2012 at 10:13 PM, Sam Minnée wrote:

> What happens if the column name has a "." in it, e.g. "Author.FirstName"?
>  
> ---  
> Reply to this email directly or view it on GitHub:
> https://github.com/silverstripe-big-o/sapphire/commit/3b72f598a2067ad64df38fd2084290eb2c756322#commitcomment-1023189
It would be nice if these were built-ins, I seem them being very common to sites.
XDA user rampv78 has build a HAL for gingerbread camera libraries that works. Except preview mode gives a error when allocating (frame)buffer
Os comentários com uma única linha começam com os caracteres //.
this is also done at boot later on in the initscript (see line 170). i'll see if it is necessary do do this at init or whether it is sufficient to do this later on.
I have a new working version had to make some changes to `static set<ofImage *> & all_images()`

Making a new pull request....
This function itself doesn't have any error handling.  So you'll have to do it yourself :-)
:D that's cheating!
What happens when you serve binaries like images or .zip files?
No, these get pruned in the later commit cleaning imports.  You can see I am working out the multi-commit git push workflow.
Need back slashes on line 35 and 36
Bad commit.  I shouldn't try to code when I am sober... the "missing" * is not needed!
Ohh, I see now, It is passed into the %s. Got it :-)
to explain, i deleted the files, thus git deleted the directory as it was empty, but it was still on my filesystem .. so silent bug...
Currently defined functions for the static busybox are only
	`[, [[, arp, ash, awk, base64, basename, bbconfig, blockdev, brctl,
	bunzip2, bzcat, bzip2, cal, cat, catv, chattr, chgrp, chmod, chown,
	chroot, clear, cmp, comm, cp, cpio, crond, crontab, cut, date, dc, dd,
	depmod, devmem, df, diff, dirname, dmesg, dnsd, dos2unix, du, echo, ed,
	egrep, env, expand, expr, false, fdisk, fgrep, find, flash_lock,
	flash_unlock, flashcp, flock, fold, free, freeramdisk, fsync, ftpget,
	ftpput, fuser, getopt, grep, groups, gunzip, gzip, halt, head, hexdump,
	id, ifconfig, insmod, install, iostat, ip, kill, killall, killall5,
	less, ln, losetup, ls, lsattr, lsmod, lsusb, lzcat, lzma, lzop,
	lzopcat, man, md5sum, mesg, mkdir, mkdosfs, mke2fs, mkfifo, mkfs.ext2,
	mkfs.vfat, mknod, mkswap, mktemp, modinfo, modprobe, more, mount,
	mountpoint, mpstat, mv, nanddump, nandwrite, nbd-client, nc, netstat,
	nice, nohup, nslookup, ntpd, od, patch, pgrep, pidof, ping,
	pipe_progress, pkill, pmap, poweroff, printenv, printf, ps, pstree,
	pwd, pwdx, rdev, readlink, realpath, reboot, renice, reset, resize,
	rev, rm, rmdir, rmmod, route, run-parts, rx, sed, seq, setconsole,
	setserial, setsid, sh, sha1sum, sha256sum, sha512sum, sleep, sort,
	split, stat, strings, stty, sum, swapoff, swapon, sync, sysctl, tac,
	tail, tar, tee, telnet, telnetd, test, tftp, tftpd, time, timeout, top,
	touch, tr, traceroute, true, ttysize, tune2fs, umount, uname,
	uncompress, unexpand, uniq, unix2dos, unlzma, unlzop, unxz, unzip,
	uptime, usleep, uudecode, uuencode, vi, watch, wc, wget, which, whoami,
	xargs, xz, xzcat, yes, zcat` I cant find the `file` function. I mean like `$ file ./whatisthis.rtf`
haem haem.. Tu as juste `bookmarklet` et `style` à mettre dans un if. Pas besoin de "dupliquer" la ligne ;)
Why do you force the session to start here?
Hast du das script eigentlich bei dir ausprobiert? Mich würde interessieren ob das so "ohne weiteres" auch auf MacOS geht
When you build with your setup with these HAL camera files, do you get errors such as this?

prebuilt/linux-x86/toolchain/arm-linux-androideabi-4.4.x/bin/../lib/gcc/arm-linux-androideabi/4.4.3/../../../../arm-linux-androideabi/bin/ld: out/target/product/triumph/obj/SHARED_LIBRARIES/camera.msm7x30_intermediates/cameraHal.o: in function qcamera_device_open(hw_module_t const*, char const*, hw_device_t**):hardware/qcom/camera/cameraHal.cpp:589: error: undefined reference to 'openCameraHardware'

If so, how did you fix it?  Thanks for your help.
this update doesn't work for mac yet :S
Ah good point. Maybe better to use YUI? :)
```javascript
Y.Object.owns(options, 'index') ? options.index : this._findIndex(model)
```
I didn't found anything better either, since the way to take the value is a call to a function.

Anyway the use of a break into a loop in discouraged. I know it is hard, but you should use two whiles for this.
@ewinslow about the placeholder attribute; i prefer the placeholder, but that means dropping the onblur and onfocus functionality. I don't think random acts of HTML5 are the way to go. This should be anounced for a new release (like 1.9) and not for a patch as some browsers may lose functionality if we change it.

@ewinslow about the security, i think there is no issue but having a second look at the code there is also a $display_query variable that is never used in the code.
We should alter the test to not use a port that known software uses :-)
Because it can be fired from outside in example from some Ads code, can't it? :) Thanks for looking at it here is the changeset: https://github.com/Wikia/app/commit/68bb847360f02ddafcfb018e4722b013ae6e843b
The trouble with this class of complex verification is it, eventually, becomes an isoform for the full configuration application pass available in mcp.py. While this is spot-fixable, I believe there's an argument to be made in favor of making Tron's configurator transactional, so misconfiguration in any sense of the term doesn't bring the service down. 

For now: I'll see if I can provide a mapping. I expect this code to be redundant in the long term, though.
could be an import in the beginning instead, similar to the other things
The plan was to include a reference to http://en.wikipedia.org/wiki/Moving_average#Exponential_moving_average
It also mentions half-life, which is I think is a pretty well know thing for exponential decay, otherwise wikipedia is our friend
http://en.wikipedia.org/wiki/Exponential_decay#Half-life

I leave it open to @rkuhn and @drewhk to decide what is the best name of this. The reason I added `duration` was that I thought of the plain `half-life` to be the relative to the interval between observations, i.e. `collect-interval`, but here we wanted it to be configured in wall clock duration.
What would happen if `PublishChanges` only contained `newGossip`, since this holds `latestGossip` anyway?
There is a problem though, if aroundReceive depends on Actor state, then it
will blow up in ctor.
what scheduler? the system scheduler?
mailboxes-common depends on remote, I am no expert but that sounds like it should work for me?
Zermomq also depends on protobuf.
the reason was that Future.failed was only introduced after Viktor and I had discovered that it was missing, and given the relative development speeds this resulted in what you see now :-)
besides a coincidence wrt. method name, what does this buy us?
Do you think it would be useful to also have a setting ``deploy-on-own-node = on`` that can be set to ``off`` for master-worker scenario where all routees are remote?
Don't know either. Possibly not. I'm just feeling my way thru this for the first time. 

On Jul 3, 2012, at 6:03 PM, Matthew Rocklin<reply@reply.github.com> wrote:

>> @@ -0,0 +1,14 @@
>> +(ns hits.models.migration
>> +  (:require [clojure.java.jdbc :as sql]))
>> +
>> +(defn create-actions []
>> +  (sql/with-connection (System/getenv "DATABASE_URL")
>> +    (sql/create-table :actions
>> +                      [:id "varchar(40)"]
>> +                      [:action "varchar(4)"]
>> +                      [:path "varchar(256)"])))  ;; should be textfield?
> 
> Also do we have direct access to the database? For example can we create an index or can we manually affect the database through a postgresql console?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hits/hits/pull/5/files#r1097390
Should be: "dipy is always looking for" *or* "dipy is always seeking", but I don't think you can say "seeking for", but EINMNL
Std = RMS only under assumption of zero-mean noise.  Is this the case?
import * makes me ill

I know all the tests have this sort of crap but it is good not too continue with making life painful :)
I'm a bigger fan of loading defaults in to the config object before I use it for parsing, so that you don't have to check for the existence of keys
The docstring says it returns a list of UnbindReports, but this is creating a BindReport. Is this right? If it's not should add a unit test to verify the returned result in the exception case.
tested, fixed and broken several times :)
It's a one-liner. True DRC sat for four months well ahead of feature-freeze, and this got pushed in a day. Ditto for the normalization fix. It's a regression, true, but that was addressed by true DRC as nothing was happening with this. Now that it's in it may as well be right and replace the harder clamp which is really just a safety.
it sounds reasonable
Move padding and optimizeMemory to attributes?
Never mind, this is still necessary since the above checks against the case where both are missing.
OK cool now I got it.
I'm not sure: cvmfs stores 0 for all normal cases, whereas the file system will return 1
Maybe `null !== $limit`? No biggy though...
Why wasting resources on additional array ?
I think we should go back to using a boolean for the loop, except this should actually set it to `NO` at some point. Then we avoid a nasty `goto` and still terminate the loop appropriately.

Alternatively, this could be an `if` statement instead, and then `break` would exit the loop. Either way, we should do something that is not `goto`.
This would be much clearer as a couple of `if` statements.

These alpha values should be pulled out into constants.
Admin's need to see the letter of the dni, they need check the identity of paiment.
You could store an `$exit_code` var and use `break` to stop the loop, so, after loop, you could `exit` with `$exit_code`
assert.equal(typeof e._events, 'undefined');
On Dec 10, 2012, at 5:21 PM, Kris Kowal <notifications@github.com> wrote:

> Is there any other way to detect VertX or alternately to patch process.nextTick or the proposed standard setImmediate instead? This line will interfere with static dependency analysis for asynchronous CommonJS module loaders.
> 
I'm a relative JavaScript novice.  I first looked for a way to provide a global setTimeout() of my own for Q to use, but wasn't able to figure out how to do that, as Rhino's CommonJS loader seems to (rightly) completely sandbox `require()`'d modules.  I could look for the presence of a tell-tale class; something along the lines of 

    if (typeof org.vertx.java.core.Vertx !== 'undefined') { /* do Vert.x-specific stuff */ }

but then I'd still need to `require()` the vertx module.  That seems less direct, but would it suit your needs?
Why do this rather than just `"super"`?
@jonleighton right, it's better name, thanks
Same thoughts.
Keep just the revised screencast, remove the other?
Put a space before the `{` when it is a block, after it and before the `}`. Like this:

```ruby
pluralize(3, 'person', 'users'){ |count, word| "#{content_tag(:b, count)} #{content_tag(:span, word)}" }
```
Wrong indentation and typo in `without`
unrelated whitespace change, not sure how much it matters.
No, but I would prefer to keep it for the moment (it is already marked with a todo, so it should not be forgotten).

Probably it happens in the same occasion when "no more than 32 elements" happens, but when you have resolution of less than 32 elements and they are negative (eg plot x over (-2.-1) with number of samples = 5)
Ah. Yes, each file should be tested line by line exactly once in the main test body.
I wonder if there is an operator that would be appropriate for `TP(x, x)`?. `x**x`?, `x%x`?
Let `w = exp(I*pi/4)`. I believe `sqrt(pi)/(2*w) z**(-1/4) * (cosh(2*sqrt(z))*fresnelc(2*w*z**(1/4)/sqr(pi)) + I*sinh(2*sqrt(z))*fresnels(2*w/sqrt(pi)*z**(1/4)))`, sqrt(pi)/(2*w) z**(1/4) * (sinh(2*sqrt(z))*fresnelc(2*w*z**(1/4)/sqr(pi)) + I*cosh(2*sqrt(z))*fresnels(2*w/sqrt(pi)*z**(1/4)))` and `1` (the constant function 1) should make a good basis. Then the first row of M would be something like `[-1/4, 1, 1/4]`, the last row would be `[0, 0, 0]` and the middle row (as well as checking my first row) I leave to you :-).
I'm not familiar enough with the `Expr` core to answer this.
Where did this test go?
I don't know how it got deleted. Will fix it.
Yup, that's precisely my point.
fixed. cool!
I think this should be "class attribute" (without statement).
I think any more than this f57bfccbd3ad9ce2de0768cbca4f255249f93524 would be superficial DRYness.
In this case it doesn't matter because our Rakefile requires `bundler/setup` which makes this the same as running with `bundle exec`.
A failing cast will always throw an exception, so why this wrapper?
On 4/30/2012 4:36 PM, Jeff Balogh wrote:
>> @@ -90,7 +90,7 @@ def message_validator(request):
>>           return request.errors.add('body', 'body', msg)
>>   
>>       # These are the keys we'll accept for messages.
>> -    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId'
>> +    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId', 'cryptoBlock', 'site'
> Can't the client derive that from the URL of the queue sending the message? The queue address is included in the message format.
>
Perhaps I'm missing where that's defined, then.

Inside of ui.js (where the decryption occurs), I get:
{"data": {
     "queue": 
"http://localhost:5001/queue/d325bd4a8285fa59e6a43670901023b67f37565c/",
     "timestamp":"1335829728.772296",
     "body":{
         "body":"The chair is against the wall",
         "actionUrl":"http://www.youtube.com/watch?v=nQB4nAjZIdE",
         "title":"You have a message",
"cryptoBlock":"{\"iv\":\"NmFaNBzjdfY30++ntGUbuLJGqy5uFK43\",\"cipherText\":\"POEpNZwVeveWO3onCCRrApxni64lJC0JWJuJncXRpDaJmRnTNKUW80I5fyE2e86rSLEufKmN/cXT2fCtcYO4/qUDtk2IPMnLT1L7ymocbrYJEPCvBUC0qj4uUo8nAb6i63zV+K7yOqFKCMSoHKM+dXhLcyvytls6WHxtlwmJdtM=\",\"site\":\"localhost\",\"hmac\":\"753988389494ed4a194d1a91d8027a0332920552fdf16eba1372be1fab811156\"}"},
      "key":"0824c7d0931f11e198ac080027a3c8f0"}}

In this case the queue name points to the url returned by the 
check/request call, which does not contain the name of the originating 
site. Where is the originating site specified in this packet?

Thanks.
Use strchr() here.
I think that works ....
maybe <query document> as a parameter...

Certain query documents specifies the field and value pairs, but other's with operators do not.
remove expanded.
Okay, I thought the _if_ became clearer, I will undo these changes.
OK, I have changed that.
> I f.e. need it in my gallery MOD to update images of deleted users, so they are guest images.

This should probably be done before the user row is nuked.
Is this line supposed to do anything? Currently it calls an undefined function and then runs the code block. At least, that what it looks like to me.
I'ts probably slightly neater to call `sql_fetchfield` here, as you are essentially querying for a single field and not a complete row.
It's probably a good idea to do this.
Is this required for the current upload to work? If not, consider adding a cron job instead. See `includes/cron/`
The second two condition elements can be condensed into `!empty($cfg_array['phpbb_akismet_key'])`
Hmmm, I guess alignment is good, it just doesn't show up the same on here so that was what the question was about. I guess just leave it as it is.
Only GET requests supported?
Same thing here, add it to the $sql_ary array like the other things are, rather than separately.
{
return ...
}
shouldn't traceroute also be required at runtime?
Shouldn't it be `amount = parts.amount` instead of `unit`?
Зачем нужно было менять порт?
Well, i know this was already implemented, but this hardcoded incremental parameters could be removed and use another approach (dynamic one).
The log string will only be created when there will be an exception.

Arrays.toString(batch.toArray()) can simply be batch. I think I was just not sure if list will print.
Good catch. We should be writing our own implementation unless the license of online code samples is permissive enough. @shuro- could you clarify which licenses are compatible? (I.e. when are we SURE we can borrow code... And in every other case we should not do so as a default)
> Could we print a warning too, with warnings.warn()?

I discussed this with @esc24.  If I remember right, he said that warnings as a general approach have **not** been working well for highlighting load-time problems
  * if you *always* show it, this gets so noisy you stop noticing what matters
  * conversely, showing just the *first* one means you will then miss others, with potentially more important causes
  * .. and there just isn't any other obvious approach ?
You don't use ```e```. How about using the try, except, else construct?
Can you put the comment on the same indentation level as the rest of the code?
//Consider migrating/calling the code originally here "as it was" as core.validate()?
I thought that it was possible for OVERFLOW handling to reduce
activeNumOfSearchKeys, so it could be less than m_numOfSearchKeys.
If I am wrong, then I would say "yes, use the member instead of a redundant
local".

On Wed, Aug 15, 2012 at 10:02 PM, Xin Jia <notifications@github.com> wrote:

> In src/ee/executors/indexcountexecutor.cpp:
>
> > +                } else {
> > +                    throw e;
> > +                }
> > +            }
> > +            // if a EQ comparision is out of range, then return no tuples
> > +            else {
> > +                m_outputTable->insertTuple(tmptup);
> > +                return true;
> > +            }
> > +            break;
> > +        }
> > +    }
> > +    assert(activeNumOfSearchKeys > 0);
> > +    VOLT_TRACE("Search key after substitutions: '%s'", m_searchKey.debugNoHeader().c_str());
> > +
> > +    int activeNumOfEndKeys = -1;
>
> replaced with m_numOfEndkeys sounds reasonable. But the searchkey also has
> a activeNumOfSearchKeys originally, should I replace it with
> m_numOfSearchKeys or not?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/VoltDB/voltdb/pull/126/files#r1389360>.
>
>
please rewrite it using one big dict / list structure nad compare each other
`treeish` переименовать в `revision`, равный `HEAD` по умолчанию
How you determine which user's CVs you will fetch ?
The `content` property is moved to the `HttpResponse` subclass.
Problem is, currently you can't create a model without any cache if it contains references to other models. And, therein lies a danger: adding the model in to the copied cache is going to alter other models in that cache, too. But, the models themselves aren't copier. See models/fields/related.py RelatedField.contribute_to_class() for one place where such changes are done (through do_related_fields() -> contribute_to_related_class()).. I don't know if there are other such cascading changes caused by loading a model into cache. Parent models come to mind at least...

Preferably before commit, but at least before release we need a way to add a model into cache in a way that is guaranteed to not change the already existing models. 

Alternatively we need to deepcopy the app-cache contents, too. But, this is an ugly approach, it can be hard to see what exactly gets copied by the deepcopy, and we might have overridden some .deepcopy methods to not work nicely with this approach (Field.deepcopy comes to mind here).
"Worked example" is legitimate english; however if it's going to cause confusion, I'll give some thought to an alternate label.
WEBVTT_NOT_SUPPORTED basically means that an API function isn't implemented. Probably want an error like INVALID_TAG_NAME or something
*sigh*

jQuery overloaded it's trim, that's why. This part: `text == null ? ""` is non-standard and we're stuck with it.
I never object to readable code <3
Like chi_squared I am concerned about the assumption of what the input looks like for user callable functions.
Keepalive doesn't work on Windows and is unnecessary for Meteor to function.
My point was not to _not_ update existing fields but rather set each field separately so that we don't lost fields that had already been there and now have a null or undefined value.

But thinking about it some more, I understand that you introduced this change in response to this [comment](https://github.com/meteor/meteor/pull/464#issuecomment-10295074) but I just want to better understand this -- is it the case that google consistently doesn't supply a refresh token at any point other than initial login? Their [docs](https://developers.google.com/accounts/docs/OAuth2WebServer#handlingtheresponse) didn't seem to imply this?
Unfortunately no. They don't event support non-80 ports. 

So developers have to test this by setting a correct ROOT_URL and add a binding in the local host files.
Did you test this formula? Passing an array to depends_on raises an exception, causing Homebrew to die.
Ok, I'll  ask a versioned url.
No, the <4.3 versions are insufficient.  I'll push a version very soon that includes a clarifying comment.

I'm not fond of the implicit dependency pulling in autoconf for us, but I'm happy to play by the house rules.
Change this to be an argument before the key path and call it "name".
argument :name, 'Identifier for this key', ["-k", "--ssh key-filepath"], ['-i', '--identifier name']
def add(name, key)
The links in settings/help.php still goes to docu.
srid support is here:
if ($srid) $geom->setSRID($srid);
I just indent in blocks of 4 chars to the first position after the opening bracket, pep8 doesn't complain about it and I've turned all exclusions off. Happy to change it if it bothers you a lot though.
Is there a reason why `GET` `POST` and `PUT` are all the same behaviour? Would prefer if we'd stick one or at least make it configurable? Treating `GET` like a `POST` / `PUT` generally ends in sadness.
Možnosti testování webových aplikací jsou... malý. Když píšu testy pro nějaký výpočet, je popis toho, co má splňovat výsledek, většinou radikálně jiný, než postup, jak ho dosáhnout, a zároveň, 'ké to štěstí, jednodušší.

Test toho, jestli nový uživatel vzniknul, to je... to je otázka, na co se vlastně ptát. Přijde mi, že když se zeptám na to, jestli má jeho objekt nějaký vlastnosti (a opíšu proces jeho zřizování), ničeho nedosáhnu. Skoro mi přijde, že by se takovéhle testy mohly psát i automaticky ze zápisu imperativního kódu.

Naději bych viděl v popisech nějakých uživatelských cest. Tj. že řeknu "Franta přijde, zaregistruje se, udělá tohle a tohle a tohle a pak by se mu v tomhle pohledu mělo objevovat tohle a tohle".
This sets the timeout period - the docstring should indicate that
I think you may also need to set self._dc to None, since if more traces come in, reset will later be called in `self._dc`?

```
>>> from __future__ import print_function
>>> from twisted.internet import task
>>> c = task.Clock()
>>> d = c.callLater(0, lambda: print('what'))
>>> d.cancel()
>>> d.reset(5)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/cyli/.virtualenvs/reach/lib/python2.7/site-packages/twisted/internet/base.py", line 108, in reset
    raise error.AlreadyCancelled
twisted.internet.error.AlreadyCancelled: Tried to cancel an already-cancelled event.
```
Just for fun, this could be done as:

```
doc = dict.fromkeys(self.param, _('a wiki revision ID'))
doc.update(VWikiPage.param_docs(self))
return doc
```
This should be a separate commit because it's a separate logical change that has nothing to do with what's above. The ```l```/```link``` issue sounds like it can be squashed into one, but I defer to @spladug on that.
Dagger isn't the graph. In this code it's a factory for graphs. While I agree that ObjectGraph.get() was lousy, I don't think we need a Dagger class that has only one method. I'd prefer ObjectGraph.forModules() or something.
Rename arg to `id` to match here and above?
I wonder if we really need a new command here. What about terminating the pipe after disconnect and creating a new pipe after reconnection?
But this only appends "about:blank?" to already flattened links after the files have been downloaded, so I don't understand how the original location would affect it. Plus, your `repairRelativeLinks` already undoes `relativeLinkFix`, but only for `src` attributes. Am I missing something?
You're right, the behavior is consistent with other tools. In that case, I think it's fine the way it is.
I think there's a bug somewhere in the logic of this big if/else block. When I run block comment, sometimes the comment delimiters are included in the selection afterward and sometimes they're not:
* selection w/i a single line: both delims not selected
* selection from start of line to end of same line: leading delim selected if there's any whitespace to left, but trailing delim not selected
* selection from start of one line to start of next line (e.g. all of 1 line including its CR): both delims selected
* selection covering parts of 2 lines (start & end mid-line): both delims not selected again
I got confused at first. The palette of saturation (x) and brightness (y) is what we're manipulating with hRatio and vRatio right? Would this be easier to read by naming these saturation and brightness?
Let's use different commands for the two platforms. Regarding begin/end of document, it looks like the standard on Windows is to use Ctrl-Home and Ctrl-End, which we already support, so it's probably just a fluke that Ctrl-Up/Down are mapped that way in CodeMirror.
I didn't want to mess with ``de`` since it's not part of the sprint. I simply reformatted it to follow the new module format.
I know we don't have an official style policy for this, but I like to see the instance properties documented somewhere.  In some of our classes we use dummy prototype copies (assigned null) to hang JSDocs off -- that's one option.  (Ditto for the existing color, startBookmark & endBookmark if it's not too much trouble).
Since the code in Brackets won't overwrite existing files, it seems like we should give an error if the file already exists (otherwise it just silently fails to overwrite the existing file). Or maybe we should consider moving aside the existing file.
Good ideas -- will add
I think we prefer the built-in Array.forEach().  It also supports an index argument (they're just in the opposite order as jQuery's each()).
Done in 0fee443.
This looks like some old code that survived the heavy refactoring (i.e. deletion) in sif...
I did want to keep this `throw`, as a protective measure in case someone is adding redundant code.
Again, this seems bad.
It's kind of necessary to do this, actually. Some editors have it allllll figured out, so they just use a pre-compiled layout, but some need to build it at run-time. Passing `undefined` to `create()` is what's supposed to happen if you purposely leave out a pre-compiled layout.
haha thats sneaky.
This is just less confusing if you remove the `!` from the conditional and swap the values. That doesn't change the result, right?
Oh, I htink in order to use return false, it needs to be applied with 

elem.onclick = function() { return false; };
"actual" -> "actually"
Force of the habit, I'll change that
Can you switch to commons logging as it's in common/lib and so you don't have to include another JAR.
This is checking which one is larger (absolute value) to check the overflow flag. It wasn't setting correctly in edge cases before this. (two negative numbers causing sign wrap and not setting flag). This has worked for all my test cases.
Just from reading the name here, and with no return value, I'd expect ```verify_medistrano_pir_exists``` to do something like set up errors or raise an exception if the PIR doesn't exist.  Athough there's no exclamation mark, so I actually probably wouldn't expect an exception, but I would expect a return value to indicate whether the PIR exists.

Actually it does set up errors, but the significant thing that I wouldn't expect is that it sets the instance variable ```@enable_merge_iq_with_medistrano_pir_checkbox```, and I would prefer to see that made more explicit - maybe set in this controller action to a value returned by ```verify_medistrano_pir_exists```.

This is not much of a complaint, it's just a point about code clarity and arranging code so that it's as obvious as possible what their side-effects are.
space between the `if` and the `(`
This param does not exist
Vain Exceptionin nappaus
No it didn't change--it's good to have a canonical url for things. The canonical url (i.e. the stuff that matters) includes the id, but not the slug. However, the slug makes the url more descriptive. So across the site, it should use the one with the slug even though the slug part isn't required and plays no part in identifying the video in question.

Having or not having a trailing slash is uninteresting and doesn't affect the url really.

Having said that, I had two patterns in there because I needed to do some quick fix at some point. It's probably better done as a single pattern.
Why have this flag? Does the bindbackend have it?
is the usage of "N_" correct in this case?
Also, I don't see how packages could not be an array, since it's a return value of sort.
Why 2*CPU_COUNT? I would say CPU_COUNT + 1 rather. Not sure if this number represents sockets or cores, I guess cores.
Robin, testing is for WIMPS. (hhh, tests coming...)
It's pretty easy to understand what's happening, but I think a variable name like `template` might be better here? Or if `template` is a bad choice for some reason, then that means I'm confused, which is even more reason to pick a descriptive variable name, right? :-)
You're missing state here in the new payload.
After XXX seconds.
I think wrapping a nav in a section is unnecessary
@cdeil @iguananaut - Turns out you can control tab-completion in IPython just be overriding the builtin `__dir__` function of an object.  So I did that, and coordinates now support tab-completion with the convertable names just look you'd want.
Sure, I'll do it tonight after client work.
Okey doke :D
@stof :+1:  good idea
simulation()
I see @nulltoken did a [PR to libgit2 that might do this](https://github.com/libgit2/libgit2/pull/938)
WTF! We're not going GNU.
Dont we have a `Bunch` like class somewhere else?  I am find defining something like this for a single modules usage, but if it is used in more than one module, like this is, it should be in utils somewhere.  But I could have sworn we already have something like this - I know you like this usage pattern.
Fixed.  Thanks for the quick review!
Unless you are filtering other stuff, in which case:
```python
def convert_to_json(message):
    """
    converts input (as formatted by collectd) to json
    """
    return dict(filter(lambda i: i[0] in ['Severity', 'Time', 'Host'], [line.split(': ', 1) for line in message.splt('\n')]))
```
D'oh! Somehow I didn't know that.
Part of the idiom implies returning 0 even when the user returns an error code from the callback; the user is not expected to check the return code of the walk if they have already interrupted it. There are other parts of the API that work like this... Does this strike you as counter-intuitive?

Either way, I don't like the idea of the library throwing an error code based on an user error. We should find an intuitive way to handle this.
Do you need to import with context here as well?
The tests should either pass or get skipped without these three things. If that's not the case, that's a bug.

Further, this is covered already at the end of the document, so there's no need to cover it here.
it doesn't matter.  it is used only when CrashTime is missing from the original raw_crash.  Since raw_crash comes in with no time zone information, timestamp doesn't need the timezone information either.
@peterbe I just went ahead and removed all submodules, I don't really see the advantage to using them... I can readd that one if you'd like though, I don't have strong feelings about it. We're not really depending directly on the search service repo right now.
Also, I thought the new standard was pep8 like variable names like `failure_ok`.
What, no Harvard comma?  :-1:
The file is not updated if there are not other changes in the entity
https://github.com/dustin10/VichUploaderBundle/issues/8
btw, I don't think this makes much sense: the validator would still validate the object graph so it could add errors on these children. So it should be taken into account for ``isValid`` too.
booleans should be written lowercased
That's true. And no, I don't mind, of course ;)
I find the name weird. Using container for an object representing a single cached item looks wrong to me.
yeah, but then it should mention it is a Doctrine release :)
what about filters, functions and tests registered directly ? And why are you doing some discrimination by collection half of the extension points only ?
@Tobion  since a call to ```unset($this->normalizers[$option]);``` is contained in this function I feel that moving this check to ```all()``` could lead to confusion and possibly unexpected behavior later.
let's `log.info` both `create_pidfile` and `remove_tempfile`
case class it is.
whitespace issues here?
not required ... the pointer is instantiated that way
Avoid data-type in name (e.g. string).

Also single quotes here since we are not interpolating.
So can we change all of these back to just "guest@"?
```css

/* Slideshow section
-------------------------------------------------- */

```
Forgot me a brace.  Fixing it now
This is done afterwards on purpose for 2 reasons:
1) minimise any overhead before sending data to the icarus (though it's negligible)
2) the icarus will not start hashing until the write has completed
I'll try to use this instead. Its crucial here to interpret this as UTF-8 instead of defaultCharset() here, otherwise it would fail on both sides of the assertion and therefore never fail.
This does not mean prod is prod environment, you can have many production environments depends on the request source
I don't think it makes much sense to implement learning rate decay as a SearchDirection. Usually we will want to use both momentum and learning rate decay, so if both of those are SearchDirections we need to implement some system for composing two SearchDirections together.
Maybe also have your Makefile have this file start with a "hey, this is an automatically derived file from mozilla-b2g/caldav, don't be poking this directly.
var list = ele.children[1], is it the same?
okay, I remove it simply because the value will be checked again in system/js/bluetooth.js.
but I think ril.data need to be checked here, thanks.
This line will never be called, in line 275 you have a return
can we rename action1 and action2 by something more explicit? actionCancel and actionConfirm, that way the code could be more readable
Please remove the log! ;)
why did you change the font size & weight? We should keep the UX defaults here imho, as they’re defined in the shared/style stuff.
Yap, : P
And I added a function called <code>setItemImage</code> to ListView.
You already check it in line.268, I think there is no need to check one more time.
Not for now. Let's come back to composite metrics once we see how people are using this.
I wonder whether these additions were necessary. I mean, they seem very logical, so I wonder why they weren't necessary before.
Did you mean to comment this out?
Strip the generate part. Just `browserCompatibleDocumentParser`.
Same test the role exists and has the correct status

order.should. include(role)
order[role].pending?.should == true
Figure out how to do flashes
call it edts-debug-window-config. There might be more of them at some point in the future
I don't really like the lambda syntax here. I think it reads better as a `fn` form so that the param (and optionally the function itself) can get a descriptive name to clarify what's going on.
I only put a return if it actually returns something.
is it impossible to skip it ?
`if (` and `else if (` etc
The set should be final.
what happens if there are neither mainstream results nor external link results for a search term, but there are inside government results. I'm sure there will be some technical terms where this is the case.
Should this redirect to `/a-slug` instead of using the slug for the first part?  I'm not sure what the correct answer is as we currently have 2 URL's for the first part of a guide.
style nit pick: space between if and opening parenthesis
Could you add support for just `composer`? Because people tend to rename it when installing it at the system level.
this could use some cleanup
Thanks for the feedback guys. Actually what I meant was, which languages should I add translations for?
Building with chicken 4.7.0 and I'm getting an error here:

Syntax error: in `let' - pair expected

	(let (b (f (car a))))

	Expansion history:

	<syntax>	  (##core#begin (define-monad <writer> (lambda (a) (quasiquote ((unquote a)))) (lambda (a f) (let (b (f...
	<syntax>	  (define-monad <writer> (lambda (a) (quasiquote ((unquote a)))) (lambda (a f) (let (b (f (car a)))) (......
	<syntax>	  (lambda (a) (quasiquote ((unquote a))))
	<syntax>	  (##core#lambda (a) (quasiquote ((unquote a))))
	<syntax>	  (##core#begin (##sys#list a))
	<syntax>	  (##sys#list a)
	<syntax>	  (lambda (a f) (let (b (f (car a)))) (quasiquote ((unquote (car b)) unquote (append (cdr a) (cdr b)))...
	<syntax>	  (##core#lambda (a f) (let (b (f (car a)))) (quasiquote ((unquote (car b)) unquote (append (cdr a) (cdr...	<--


I can make it work with:

(let* ((b (f (car a))))
        `(,(car b) . ,(append (cdr a) (cdr b))))
Well, if you want the pretty URLs to stay you need to do an internal forward - if you do a redirect people will never notice and start copy-pasting those nice URLs?
Really? I'm sure you can write your plugin without this dependency.
please freeze the version.
Получается json ради json
I'm fairly new to Ruby on Rails and HAML... so just let me know if I do something stupid.
What is the point of removing the trailing '/'? This is not part of normalization of URIs.
Never deal in id values. Always assign the model to the relationship. it helps catch errors. In this case, your line should have been:

```python
e = Event(profile=profile, venue=venue, **event)
```

The use of the name `event` is confusing here. `event` sounds like an event object, but it's just a dictionary. Rename it to `eventdata`.
use an iterator here to get rid of `.at()`, if you'd like to avoid the long type name here just typedef it and use `::const_iterator`
TODO: check for the proper anchor to use
FYI: This generally should be experimental for new plugins
Para manipular entradas do chart favor usar os métodos genéricos de chart_data:

save_entry
edit_entry
get_entry
get_entry_item

Para evitar criar getters e setters na classe Patients para cada atributo de chart data (alergias, problemas, etc.) como estava sendo feito anteriormente.
Nit: We put spaces between operators and favor `===` even though it not always makes diff it's a good practice. So it will be

```js
if (typeof frame.onUnload === "function")
```
Same comment as above w.r.t. labels.
static/i18n is now completely generated, no files are checked into tree.
@zaach: Oh, I bet I know what happened. Did you manually create a test account, and put its email and password inside credentials.yaml? If you don't, the dialog will time out. Give that a shot and let me know if it's still broken.
I *really* like that we're moving forwarding out of the `browserid` process.  the more we can move out of that process, the more performace wins we'll get out of this.
If there is a stagedEmail and stagedPassword, re-authenticate the user using the stagedEmail and stagedPassword.
ahh, dang.  sorted() is what I was looking for!  I tried sort() which sorts in place and returns nothing...  thanks!
what's the difference between cron-batches small and large if we always run them together in our crontab?
This is simple and great.
Are these lines needed now?
Mind explaining this check in a comment?
it does not need to be rocket-fast, so you can leave only `json` import
Filed as #18042
The description strings need to be fixed. If a method raises an exception, the description string should be, for example:

`it "raises an ArgumentError when passed an invalid month"`

@dbussink please do not merge specs unless they conform to the style guide.
No space -- we intended the shortcut ternary here.
Use the PHP_VERSION constant.
Hi Marc,

Perhaps we can move the part inside the foreach() to its own method? That would allow for easier comprehension and extendability.
+ * @see http://nl3.php.net/manual/en/function.date.php
I'd still call "query" (both in the methods and the property/variable). There is no reason why we should use unknown abbreviations in the framework.
Shouldn't there be some assertions...?
I understand why you added phpMyAdmin and FrontPage but I don't think they belong here. It should only detect phpMyAdmin when you're actually on a phpMyAdmin page, and FrontPage when you're on a page that has been created with FrontPage.
[Fact] should not return async. It should wait for the async result and return void, so that code execution caused by the test is complete before the test completes.
Well, this is how it was done before (for the old History Item before I changed any code). I'll assume you want the try/catch block removed.
Maybe add a short docblock here with an example use case?
You would think that would be the case but its not. Just tried it in the simulator and on my device - iOS6
@jthomas Does your original version imports Dijit CSS including dijitOffScreen? My quick glance did not indicate so, but wanted to double-check with you.
``else`` should be on the same line than the closing curly brace of the corresponding ``if``
There should be a blank line between these, and all further occurrences.  The language input files are defined as having blank lines beween each item. (Normally these files are automatically generated, but if you're going to manually edit them, please put in the blank lines.)
to me it's not the job of a PHP script. With a well configured web server (apache2 + mod_deflate) it's done for free.
Etag and 304 responses are useless with an Expiry of 10 years.
it changed in the last 2 versions...
cplex_studio122.acad.linux-x86.bin     cplex_studio123.acad.linux-x86-64.bin  cplex_studio124.linux-x86-64.bin
Could we rename this to setVerbosity()?
I've used a lowercase `q` elsewhere, so can you change this to make it consistent?
@purcell I'm not sure if I can follow you.  If I understand the documentation of `vc-find-root` correctly this function gives me the root *directory* of the working copy, and that is not what I need.  I need the root *URL* of the repository, i.e. the URL from which the working copy was checked out.
Look at the comments for the above test, same comments apply here
cputime should be divided by 1000000 instead of 1000, because apparently pti_info times are expressed in nanoseconds, and we want milliseconds in process->cputime
Make this a separate .html file so you can do something like 
```
...
{% include "_form_buttons.html" %}
</form>
```
I'm running into a slight problem. I can't figure out how to handle unicode variation selectors and overlaying characters in javascript. For example

http://www.w3.org/TR/2000/WD-MathML2-20000328/isoamsn.html

lvnE has unicode U+2268 U+FE00. If I try typing "\u2268\ufe00" into firebug, I don't see a difference between characters with and without variation selector. That's OK if firefox cannot display that variation, but now if we look at napE it has a unicode U+2A70 U+0338. Typing "\u2a70\u0338" in firebug places the "Combining Long Solidus Overlay" character (U+0338) as the second character without overlaying it. Am I doing something wrong or is this a limitation in firefox?
Sorry, I didn't realize this change from your previous comment about the RIS.js change.  I've modified the translator now in the same way that APS.js was modified.