labofsahil/github-event-dataset-2012 huggingface dataset:
I have some other plans with it and it's definitely not harmful to keep it in repo. I will fix the uname.
12 spaces? this is indented weirdly
Good work finding the source of this bug. It was driving me crazy in my projects.
Is Path the full physical path? If so, could get a bit ugly with the UNC, so ideally, repository relative would be best. But no big deal.
Ok, stuff discussed here should be solved by this commit:

https://github.com/alteredq/three.js/commit/025fb6243afed67811ee514e5dc11b382336c67c
No problem. 
Can you please explain how you and your workmates retrieve the storagePid, if it is not the General_Record_Folder.
I stuck into this in my actual project and could it only retrieve by extending your dispatcher like above.
Maybe my solution is not the right way - please share your experiences.

Cheers,

Daniel
Huzzah! I'm so glad to see Machinist is back to active :shipit:
AbstractStateMachine needs some kind of Event available at class level to define the default event_class. AsyncManager isn't available until services are instantiated, in which case it will use its subject's async manager. Maybe AsyncManagers need to have some kind of global singleton for situations like this? I'm not sure.
should this be changed in `media/js/zamboni`?
guess you can't
I don't think its necessarily to get the list of episodes when loading the series page, this should just load a list of Ints (season numbers), then the view would go over the list and pass them to season view which would render the season based on seriesID and season number.
`var FANCY = true` :D
Right, to be overly pedantic and get correct rounding I'm going for (pat.length * 5 + 2)/4
the why not also use the passed in argument along with the statement to generate the cache key? maybe i'm missing something here?
`codecs.EncodedFile` does not encode, it translates between encodings : you should use instead (line 1071)

```python
svgwriter = codecs.getwriter('utf-8')(filename)
```

Also note that the function `print_svgz` still uses `io.TextIOWrapper`
Fixed in 422d327.
`django` imports typically go above third-party-non-Django imports
Oh I see, but its more a fault of TomG, the maintainer of Blade...
Since other devices seem to have the source for libsurfaceflinger...
Why not ask him to release it??
Also, is there anyway that I can hack the surfaceflinger.so to make the screen normal??
ZTE Blade has flipped screen right...??
This one doesn't have to be merged, I have no idea why it's in this patch stack.
cat[1-8] are cache directories for man pages. This should be `(man+'cat1')`, typo'd before.
this is my first githut pro!~~
Plutôt que de checker en dur si le joueur est op, tu peux utiliser une option pour tes permission dans le plugin.yml

```` yaml
ma.permission.op:
    description: "Ma permission op"
    default: op
ma.permission.personne:
    description: "Ma permission pour personne"
    default: false
ma.permission.toutlemonde
    description: "Ma permission pour tout le monde"
    default: true
````

C'est toujours un truc en moins a mettre a jour si la methode isOp() venait a disparaitre/changer =).
へー、何このファイル。これ外してもメニューのAboutにバージョン番号とか出てました？
もしOKだったんなら外しちゃっていいかなぁ。
Yep simple as that :)
Thanks for the addition, now just share you test [code in the wiki](https://github.com/alixaxel/phunction/wiki/phunction_Net::OpenID). :D
This should go away as well.
I really hope I merged this right. But everything seems to work ok.
I realized (timezone). But I think that we should be the following code,  because we'll have a little time error .

```ruby
current_time = Time.current
Hash[all_timestamp_attributes_in_table.map do |column|
  [join_table[column], current_time]
end]
```
I think there is a closing } missing. also: coding standards put a space before/after the `else`
sorry about the confusion. i was using stylus-images as an example plugin to stylus. yes, stylus defines a paths variable that it passes to its plugin, stylus-images. it was my intention to add the same functionality to lumbar. i.e., paths is defined in lumbar and is passed to the plugins. since its not currently used by any of our plugins, i will delete it.
A yes, missed that. And now that you mention it, matches_patterns isn't available in Django 1.3:

https://github.com/miracle2k/webassets/commit/662bb60a0b4595e08ff85a5daeeb7c40476f802a

I suppose we would just do without the pattern matching in older Django versions, but still support the behavior? (who knows, maybe someone ports a CachedStaticFileStorage class to 1.3).
Хм, не знал.
Needs to be escaped. Can not use htmlspecialchars to escape javascript literals, and need to use addslashes:
```echo '   alert("'.addslashes(xl('Message Sent').'"));';```

For better explanation of the security model and using these functions see here (see exception in step 4):
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan
Enabling these drains breaks r0ketris here. IIRC, the join ack is no longer 
received by the r_player, so I get a timeout after selecting the game. 
Unfortunately I have only one r0ket available currently, so I can not test 
this again. I can probably lend another one next week, though.

Just after reverting, it does not work here either, which is why the usleeps 
are added with the next commit.

On Sunday 08 January 2012 17:48:38 you wrote:
> why?
> 
> we had errors without this on the sheevaplug (not on my notebook)
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ralfjung-e/tetrisVI/commit/f1b150b118bd0999ed47c64968923
> ad09b3077f4#commitcomment-847715
That makes sense. It would be great if it worked similar to factory loading.
放在单独命名空间下是指 ruby-china.org/api/xxx 这个意思么? 如果是, 那这些controller总是要复用的吧?
Looks solid, although two concerns:

1. `onContentHeightChange` in the Options block seems to be commented out
2. Wouldn't `readjustKnobPosition();` be better served if its functionality were merged into `reposition();`? I initially intended to use `reposition();` as a sort of "catch-all" function that both checks for correct placement of the scroller container & knob. Let me know your thoughts.
what's this?

This needs:
* proper license
* proper comments
* proper styling
2012/1/11 Yaroslav Halchenko
<reply@reply.github.com>:
> @d0ru: btw -- you aren't Teodor the one who ..

Yes
我也想问……
yeah, was not planning on committing this as is--just playing around a bit.
Don't forget the 16x and 24x sizes too
fantastic. :)
确实应该找个时间改改
No problem, thanks for your contribution.
Missing a comma ;)
=offers mike a coffee =
Why do you replace proper logging with System.out.println()?  You shouldn't have System.out.println() anywhere in production code IMHO.
Should this be `quotient.d.first`?
you absolutely vain removed the old code (and data in SQL), working with the GO. I and Insider42 many were working to make it all work just fine ...
Any reason for using ```time()``` over ```$timeline_action->getCreatedAt()->getTimestamp()``` ?
I'd like to know about any usecases where the sorting falls back to names of albums and tracks and the lack of "the " ignoring is noticeable. I'm open to changing it, but I don't know why we would other than consistency.
Typo here, WOLRD_UPD_ALL_WAYPOINT_DATA_WPGUID, I think you can figure out whats wrong.
@3kids read @e000 comment and watch @Aokromes video
Aah, there's the culprit. Mea culpa :)
Should there be some code in here? If not why the if ?
а почему для других связей не прописал?
@Flet why did this happen? wrong rollback?
&& is preffered, not AND... according to the CodeIgniter's style guide...
The sql for the conditions table is being reconfigured for skyfire database. Once completed, it will be put back in here
lolololololololol
Both of you calm down
Checked and agree
et là c'est mieux
Ah! this apparently happened only because of the variable. Interesting.
First impressions are that this isn't structured correctly.  Why should spillingsorter base and quicksortalgorithm share the same base class, but insertion sort not.
Are you absolutely sure you built correctly?

grep -R bcm4329_wlan arch/arm/boot/Image

As a test, I tried the firmware from your device, and I get these errors:

<4>[12, kmmcd] [11355.761797] DHD: dongle ram size is set to 294912(orig 294912)
<4>[12, kmmcd] [11356.844503] dhdsdio_htclk: HT Avail timeout (1000000): clkctl 0x50
<4>[12, kmmcd] [11356.855560] dhd_bus_start failed bus is not ready
<4>[12, kmmcd] [11356.855902] dhdsdio_probe: failed
<4>[12, kmmcd] [11357.854158] dhdsdio_htclk: HT Avail timeout (1000000): clkctl 0x50
<4>[12, kmmcd] [11358.854157] dhdsdio_htclk: HT Avail timeout (1000000): clkctl 0x50

Curiously, I tried the firmware pre-installed on my Ubuntu installation (/lib/firmware/brcm/bcm4329-fullmac-4.bin) - it's older than my version, but it works fine.
Commit will need to be squashed with previous ones
This can result in some cool stuff:

![NetherFort1](http://i.minus.com/i2NcWFg0aJ0dJ.png)

![NetherFort2](http://i.minus.com/iW2NHUDky9sTs.png)

Not sure it's even in our scope to pull, but I wrote it up and some people wanted to see it.
I have all our issues open in separate tabs right now, so if you want to delete this repo and remake it.. or whatever you want to do, just let me know :-)
I think debug message should only be displayed when the app set the log level to 'debug' (aka,  -d flag,  -Debug plugin,  sub debug in app, MYAPP_DEBUG or  CATALYST_DEBUG)
If we include Firefox why not IE?  Opera? Etc.  Maybe "Modern web browser with developer tools e.g. Chrome or Firefox with developer plugins"
Grrr... Corrected now. That one was leftover from the dependent style, of course, and I had it validated after removing it. Not sure how I put it back before pushing :-)
hum I guess the examples right after clarify it but I'd prefer to not bring confusion in the first place.
Duplicate function !
alt title is wrong
client_secret is not required for the refresh_token grant type:
http://tools.ietf.org/html/draft-ietf-oauth-v2-23#section-6
Good job on this - the column number for a header error seems pretty insignificant, and pretty labor-intensive to get the ocaml compiler to report correctly.
Whoops, I didn't realise the deprecated JS call had been replaced already. This is an actual fix to the JS call so that it works, but in this commit it is not actually replacing a deprecated function.
No I have tried that! It reads that as next object somehow and stalls the server in an infinite loop
Change 'not significant' to 'non-differentially expressed'?
I thought about that, but if we add an Indeterminate property, when its value is True, Fraction value getter/setter access doesn't have any sense. Should we throw InvalidOperationException() in this case?
Not better, but I found the try/catch blocks to be slightly messier. Here it's check/throw without any extra noise.
I guess this could be $this->wg
ok, yes that was what I was thinking when I proposed lazy. good
yes, when that's in and merged
is `for` correct?
to references `of` its parent,
Possible to remove a bit of duplication:

  if (currentRoutees ne consistentHashRoutees) {
      val rehash = consistentHashRoutees != currentRoutees
      consistentHashRoutees = currentRoutees
      if (rehash)
        consistentHash = ConsistentHash(currentRoutees, replicas)
    }
remoteNodes(n.address)
Isn't that last one going to be very expensive?
use address(victim) instead, or rely on the implicit conversion
It's also only used in one place, so doesn't need to be a def, just move it to where it is used.
As you can see, they are gone now, thanks for the help.
So what are we doing here?  I'd like the 'nicer' naming but if you are being stubbon then I can live with that.  we can always do a rename later.

Let me know what you think.
yes but I would need to trace all the places it is imported and I've sort of hit time constraints

feel free to do this though
This a actually part of the documentation http://docs.ckan.org/en/latest/writing-extensions.html#writing-a-plugin-interface
So, I would rather suggest, you change that first.
Same as above about how deleted bindings are handled.
Fixed (I agree)
ditch the second "to"
@adminnu 
я пока не вкурил как тут переписать свой пулл-реквест, потому - если в курсе - подскажи, буду благодарен (как добавлять файлы, уже понял)
Ik mis visibility (public, protected, private)
Can this be reformatted so there's less happening in the string generation?

```php
<?

// outside of the loop
$files_domain = (vf::$client->func_boilerplate['all']['files_domain'])?:'static.cravetickets.com';

// inside the loop
$file_type = array_search($_i['media_type'] . '/' . $_i['media_subtype'], $sky_content_type)?:'file';
$_item_html = sprintf(
    '<img width="%s" height="%s" src="http://%s/images/files-icons/%s.jpg',
    $gallery->width,
    $gallery->height,
    $files_domain,
    $file_type
);

```

I've become a fan of using the formatted string functions, it makes them easier to read.

Maybe for `$files_domain`, we should have a vf method that returns the current files_domain. Also, static.cravetickets.com should not be anywhere in this logic, this shouldn't work if there's no files domain specified in vfolder configuration.
I'd prefer to use `mock` instead of adding this kind of tracking code. `Mock.call_args` is intended for this kind of thing.

Fine if you don't want to add the dependency.
Мы выкачиваем ключи Танкера к себе в `i-tanker`. При этом нужно уметь «мапить» обращения к динамическим ключам Танкера на свои кейсеты.
This check is unnecessary. Please jusr s/request/response
#ifdef, and not sure about using the word "hit" here, probably just eliminate "hit". Same below.
@drak I don't think that will do what you think it will do.

I'd rather see this stay abstract.
Curious why did you choose to throw/1 versus returning {skip, string(), list()} here?
There is nothing about this in standart syntax description. It's rather design question than an implementation one.
`$options` should be added to keep constancy between all methods.
Actually, why do we need to do this check at all? When would we not want to remove the view from the map?
Okay, then cast the incoming view as id. There's no need for a second variable.
Why should direct calls be avoided? No such restriction is noted in the `highlighted` and `selected` properties.
Looks broke to me.
It's not clear where `isArray` is coming from here since it was defined in `deferred.js`, which will only be combined with `promise.js` at build time. This will throw off editor-based lint tools.
You needto use a different example for this. I was trying to generete this graph, but it has been running for over 24h now...
unsigned int?
возможно и хорошая практика, но в проекте ни разу не использовалась.
я за единообразие.
One thing I wanted to point out and get some confirmation on:

I reversed this assertion as I think it was incorrect.  'foo' casts to 0 which is a perfectly valid :zine_id absent  other validations suggesting that it is not,   so update_attributes should work just fine.
This assertion should probably go inside the constructor of `Instruction`.
What are the parameters after &info?
Oh, I meant np.squeeze(kpos), which does the same assuming kpos is shape (2, 1), which you say it is.
I think if we want to be really rigorous about restoring behavior, this should use try...finally or a context manager.  See http://blip.tv/pycon-us-videos-2009-2010-2011/pycon-2011-exhibition-of-atrocity-4899139 (the duck punching example).
I guess I didin't want *x to compare equal to Poly(2*x), but I just did a test and they don't so I can change that.
Probably not. This is one of the things that can be removed I think.
Manual cleanup on the store? I strongly believe this is useful.

Imagine I load some queries and then I know I will not use concerned records anymore. I want to be able to unload them.
renamed locally
if `DiffaConfig.ASCII_COLLATION` is a constant, why not use it instead of the string `ascii`?
All files need a license header .....
it's not an option

reprhase: 

To define the bounds of a query using :operator:`$center` you must specify

- foo

- bar
comment it out in the mean time?
Mostly because it's the standard API for ModelForm, and toggling the default, or requiring it to be specified explicitly, would break unaware code in ways that doesn't raise the NotImplementedError.
nit: Generate table of contents and TOC:
The message probably should be "incompatible database (engine)" instead of "version".
Yup, let's do that.
Why were these roles added?
We do not do this for the style paths with $phpbb_root_path, why would this be different?
Isn't the global cache supposed be a cache service instance?
Right, I overlooked the clearing.
The Style components string is no longer being used (it wasn't before this change either, but since I'm changing the style organization around, I figured I could remove it here).

As for Smilies, the tab behind the comma still appears in my files, so I know I didn't remove it. Maybe Git automatically removed it for me upon commit but it is still in my file, so I'm confused.
Will this return global $metadata, so that you do not need to fill it somewhere? I mean, should not this be rather:
`$metadata = $phpbb_extension_manager->get_meta_data($ext_name);`?
Empty line after imports.
How am I supposed to make typedef private? Or are you speaking about _padding? Not its not, its just a way to separate aligning field from others.
ah geez. that change needs to be made to 40+ files and the README. leaving that another PR for now.
Good call. Actually `RequiresSSL` and `RequiresHttps` are two ways of saying the same thing. So it should probably be `RequiresHttps`, for symmetry, or something like `Secure`
It's my carelessness. Now i fixed it.
Actually, we could remove the other Confusion* classes form the "public" API.
more commented code, avoid pushing this
if !$mime
Option的描述加上吧。
Minor nitpicking (I'll merge it anyway since it's still behaving reasonably -- I'm not asking you to change it, just to keep it in mind for similar situations in the future):
- Why raise a ValueError? An AssertionError would seem more appropriate.
- Instead of adding an extra boolean flag to detect if the exception was raised, you could have written:
```
try:
    grad(cost, x)
except ValueError:
    pass
else:
    raise AssertionError('...')
```
strip here is overkill. just use [:-2] and be done.
I know, but it's tied to the cache-as-global-state mechanic. This'll get fixed as all the other models code gets weaned off of it slowly.
This occurs in a few examples through-out the diff, but the references in the existing docs, as of 1.4 should be using ```from django.conf.urls import ...```, though I can't find the ticket that updated them.
From what I'm reading in this link we should 'unset' text in order to tell GIT not to convert line endings. If we were to put -text it would convert all line endings to the native OS upon check out which means that the tests for specific line endings like LF or CR on a Windows machine would now be using CRLF. 

We want to keep the integrity of the line endings upon check in and check out in the tests files to make sure we are actually testing what we say we are.
https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/String#Character_access says IE6 and IE7 have a problem with array-like access.
I don't think this should only be checked for the first data point.
This is only used for index_item_task and unindex_item_task, so it's probably better to put it just before where those tasks are defined.
Hmm, if you set `referrer` to `None` here, why do we still need the depth 2 wierdness? Shouldn't this request still be considered "depth 0" with no referrer?
This is  a `NameError` every time because `user_agents` is not defined. Maybe it belongs further down, and should be checking `self.batch_user_agents` instead?
Are these needed for the stable build, or just HEAD?

They also need to be conditional on `MacOS.xcode_version.to_f >= 4.3`, as older Xcode provides them.
Changing float: left; to clear: left; will prevent the sections from floating which is a waste of space on wide-screen monitors. What we want is a floating solution that supports both wide monitors and e.g. phones.
What does this do if `url` is `None`?
It's only called `qos_prefetch_count` in `FakeAMQP`, in `AMQP` it's called `prefetch_count`. I'll prefix it with `amqp_` instead.
Go ahead and update it and do a force push.
Yes. The previous line is at maximum length. I can see that it would be nice to have "are contained" in the summary line to make it absolutely clear and grammatically correct. But it's the above line or break it into more and that would still require a summary line.
log --> comment
I don't want to keep empty javadoc tags that add no information.
Any updates on weather or not objects affect speed here?
I'm still seeing problems on the right edge. It's fixed if you change "renameInputOffset.left" to "renameInputOffset.left + $renameInput.width()".
I'll try writing a script for creating a symlink.
Separate from in-browser branch.
Micro-optimization: %4 is the same as & 3
Same here. Do we want to delete it or not?
Yes! I mistyped. The variable is there unused.
You can always install with pip into virtualenv and then use pip freeze to get the requirements.txt. ;-)

I would not go for the latest trunk, but for release candidate. But also latest trunk would be good in other cases, where release candidate would not be already available.

So the idea is that we freeze the exact trunk version in requirements.txt, not just have the latest on time of install, because then we all will not have the same environment while they will develop Django further.
Done below now
Or was the intent to fire a `projectchanged` when they are changed?
Do we even need `_dropTarget` here, or can we just continue to use `_container`?
This is pretty gigantic. Any reason it has to be this large?
What response does this send?
Good catch!  will fix.
We can definitely write tests for this function now, we don't have to mock iwlist or anything.
additional access control system such as `SELinux`_ or `AppArmor`_ are blocking

It went from proper grammar to not :P
注意debug输入的格式，ngx_uint_t -> %ui, size_t -> %uz
没有恢复 cf ?
Oops, I completely forgot how @param works, I'll go back and change all of those tonight.  Good catch.
is Id[] needed?
where does this keystore_password come from?
FWIW, i've successfully used nodeunit.
e.g. https://github.com/Obvious/variants/blob/master/nodejs/tests/variants_test.js
I've added a comment below, where it's being done. I just take all the descriptors and add each instance multiple times to the dictionary, under different keys.
Minor point: We try to avoid `\` continuation lines.  You can use parentheses instead, for example:

```
            if ('network_error' in result and 
                 result['network_error'] is not None):
```
Heh, sorry Qt Creator did that...
Didn't we agree last time that the preferences dialog shows the settings currently active. Thus, as overridden by command line options etc? In that case, this change is not valid.
So all our threads are closed, even when for example we crash (thread unrelated)?
I don't think `null` is supposed to be mentioned as a type.
I've checked this and it is already tested? Or do you mean something else?

$this->assertTrue(Validation::mimeType(array('tmp_name' => $file), array('image/gif')));
$this->assertFalse(Validation::mimeType(array('tmp_name' => $file), array('image/png')));
Sure, on both $_cache and _cache() or just the variable?
You mean I should revert this changes?

Strangely compiling was fine in Development configuration but not in Deployment without the above changes. 'cellFrame' in the code is of Type NSRect but CGContextFillRect expects CGRect as the second parameter. And for drawInRect it's the other way around.
I think require or include without '_once' makes more sense because modules can be included more than one times.
This should be precompiled (in `__init__`) and saved as `self.docstring_arguments_re` or similar.
I believe you support seconds somewhere else, at least I remember reading something around those lines. Its probably not too much extra, so go ahead and add seconds support.
To make it more difficult to miss returning a value, maybe you want to change some of these functions to use the strucutre:

```
function x() {
  var result = null;
  if (foo) {
    ...
    result = [];
    ...
} else {
    ...
    result = this._properties;
    ...
  }
  return result;
}
"always"? :speak_no_evil:
I've been told that readdir() is "probably" threadsafe, so I'll take my chances with it for now.
Is there anything that can be done to document the properties in this global object? Not urgent, just may be helpful for people to know what the options are being used for- especially the debugging ones.
Yay for less divs! The two lines below can now move back two spaces.
These are all the same with some minor differences. nose allows you to write a test generator which generates a bunch of tests which would reduce this code pretty significantly, but maintain the distinctness of the tests.

http://readthedocs.org/docs/nose/en/latest/writing_tests.html#test-generators
I've had no problem with booleans and psycopg so far, and as unit tests seem to run just fine I suppose this is not a problem.
`if not profile.groups.filter(groupuser__group=group).exists()` is one line less, although not much less verbose :)
Just use curly braces and @wraithan will be a happy camper
Oh, I just got that may be for the tests. In this case, we probably should have a way to not check the certs locally, when doing that for tests, still doing it when we test the actual server.
I knew something looked wrong with that line.

-- 
Kris Maglione

All religions are ancient monuments to superstitions, ignorance,
ferocity; and modern religions are only ancient follies rejuvenated.
	--Baron d'Holbach
Is this really any more clear than the generic: "invalid field 'run_list'"?
This could be removed to reduce complexity. But I wanted to free the buffer before PostProcessTexture() etc are called.
Probably should remove all references to AbstractBundle.
Does this really belong the the `Iterators` ?
Do you really need a define call here ? and what if another file already defined ``BCRYPT_PREFIX`` ?
Mhh .. not sure how to properly do it, since @fabpot said: "This is wrong as the method behavior is totally different from the parent. Why not set the content disposition without overriding this method?".
I would add a ``return $value`` after the foreach loop instead of changing the next condition to make it check arrays too. It would be more readable IMO as you would see directly what the array handling would be
This should be removed. Guessing "null" does not help.
This is a normal member variable. See also `getMethod()` and `setMethod()`.
`|null`, int -> name ?
it is needed as you code is broken in case of a null value
I saw you updated the bridge composer.json (but with the wrong lower bound as 1.9.0 is also buggy) but you forgot the main one
Ok, it may be returned as NSNumber. It depend if it's in JSON `"123"` or `123`. Set breakpoint in that method and test in in console by typing:
```
po [utc class]
```
(this will print class of utc)

If it's NSNumber change all utc types from NSString to NSNumber.
Test code is code. Rules that apply to production code also apply to test code.

I create lots of private methods in my specs. I also create classes in `spec/support` for shared behavior.

Here are a couple examples of methods I wrote to improve readability and reduce duplication: https://gist.github.com/424dd0c829dd0d9e9c96
chance is 11 out of 1000 actually? 11 values (0,1,2,3,4,5,6,7,8,9,10) match the condition
This command is declared twice,  and cancels both declarations.
how about renaming this to ``child_plugin_instances`` to make it more explicit what this holds?
Made getOffsets and parseCss local methods. The others are used in unit tests.
Can't you simply do a |else| to your if block?
On alomost cases, this is a good idea.
But WPS available and no encryption AP was shown as "Secured by WPS".
Because "security === empty" means both "no encryption" or "old API".
This causes same bug to a comment of @mrbkap (https://bugzilla.mozilla.org/show_bug.cgi?id=791506#c12).

And on previous API, isWpsAvailable() always returns false. So all WPS functionality doesn't work.
"ME TOO!"
nit : add a name to your anonymous function ( i wonder if bot.io can have a rule for that!)
Can you use `addEventListener` instead of `ontimeupdate`?
I am really not a fan of adding a click handler to the whole page.
I will likely replace this with a animation handler hack so this behaviour
will only be added to specific elements as they hit the dom. (As seen in x-tags and other libs)
I agree with @arcturus that the case is not going to happen often. The issue would only occur when changing the SIM card with the Contacts app open. 

I would suggest re-checking SIM status every time Settings section is going to be open. That's an easy workaround that will fix 99% of cases. The rest 1% can be solved closing Settings and opening it again, so I think is fair enough.
move the checking in line #51, #52 to here.
`if (!settings) return;`
good work. Don't forge to pass the linter and fix all lints
Peut-être, c'est une remarque que je ne sais plus qui m'a faite pendant que je le traduisait samedi, comme quoi lui et Goofy se posaient également la question.
I though we were using the span to not allow formatting text. Adding divs or directly ... what if I send you an sms with:

<script>alert(1)</script>
Would `Metric.objects.filter(name='addons.downloads.count').count()` be faster? Maybe not since it's only a single record but it's more to the point to me than doing `len`.
Whoops.  It looks like there are also some whitespace issues here.  Probably a tab/space thing again.  4 space, soft tabs please :-)
why is what necessary?
Same as with RepoInfo; we already have licenseToConfirm(). Why licenseAcceptanceNeeded?

If zypper actually needs a bool-test, separated from retrieving the license text, it should probably be 'bool hasLicenseToConfirm()' , so it fits the current naming schema.
@malthe -- between your input and that from @davisagli on plone-developers, I am getting sense that the IUUIDObject markers/adapters may be (while useful to me) less than broadly useful compared to the cost of additional complexity.  I expect to remove these commits with a reset from an amended pull request that is limited in scope to the serialization fix.  Thanks for the feedback.
Move assignName to inside CoGroupBuilder.schedule
Not really a true slice() without the offset, but whatever.... can change the name if we need to.
hmm tests fail for me:

```
/Users/tj/projects/axon/node_modules/should/lib/should.js:125
      throw new AssertionError({
            ^
AssertionError: expected '\u0000' to have a property 'cmd'
```
What doe the plusses signify here?
Consider validating each parameter passed in to this method - or at least their types.
Why the "AppScale" tag?
``new \EmptyIterator``, or even better, moving ``$files = iterator_to_array($iterator);`` above and doing ``$files = array()`` here
Aren't we catching `<b>Error: Duplicate file exists<br><br><a href=//boards.4chan.org/f/>Return</a></b>`?
Just noting a minor formatting issue here (VirtualBox won't end up in the itemized list).
it'd be nice to test the `-n` option too
Good questions. I'm mostly just playing with pull requests but we should address those dependences for real, too. I'll check.
The web UI should do that locally -- epoch times are easy to transmit, and cacheable since they're absolute.
Any reason this changed from `name`?
`Besides downloading`
All the changes above don't really seem necessary anymore after extracting the second loop.
No, but these instructions are for iTerm, not "Mac" in general.  e.g. if you used regular terminal, these instructions wouldn't make sense
Foursquareは友達と会ったり、新しいスポットを見つけたり、お金を節約できる楽しくて小さなスタートアップ企業です。Backbone Modelsは[トップページのマップ](https://foursquare.com/)や[リスト](https://foursquare.com/seriouseats/list/the-best-doughnuts-in-ny)のような非常に利用頻度の高い機能を供給するコアとなるJavaScirpt APIレイヤーとViewsで多く用いられています。
well, e.g. 1st one about "index.html" -- it would be quite a stretch to react on every missing file which could be misplaced/queried for a reason (e.g. file moved)
You can access href directly on the DOM object, therefore: 
nextLink.href = ?index=3
arg... sorry for this.
This will surely show up a bit everywhere. I will change my Eclipse 
settings to make sure the indentation the one you use: 2 spaces, is this 
correct?

-- 
Renaud Gaujoux
Computational Biology - University of Cape Town
South Africa


On 20/06/2012 03:26, hadley wickham wrote:
>> @@ -235,7 +235,13 @@ to_html.link<- function(x, package, ...) {
>>     } else {
>>       topic<- to_html.TEXT(x[[1]])
>>       label<- topic
>> -    t_package<- opt
>> +	parts<- str_match(opt, '([^:]+):(.*)')[1,]
> Could you please fix the indenting here?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hadley/staticdocs/pull/22/files#r1014979
* current_user.score_private = !current_user.score_private 
* use localization
The toString implementations weren't actually quite correct so I fixed those.
can we rename this? I get it confused with the calls to `window.getSelewction().addSelectionListener`
Naming is hard! Do you have suggestions ?
Don't mix the single quotes and double quotes. It seems this file uses double quotes so write "use strict" instead.

Also, put a new line between the license and the use strict to separate them (are conceptually two different things, totally unrelated) and make them more readable.
Very cool! This will be helpful for spotting updates that break xhr.
The use of `data_iter` here as a variable name is misleading.  There is an explicit return type for `execute()` called `iter`.  The return type specified in most of these methods is a `tuple` which is iterable but you could probably just call the variable `data` and that would be fine.  I think this started awhile back where some `exetcute()` statements were copied and used as templates, that made use of the `iter` return type and appropriated called the return value `data_iter`.
I am faaairly certain we're not planning on caring about views, just clicks. Those are a) easier to count and b) make more sense for people to pay for.
Note to self: `jsdoc` please!
Did you remove this because Travis will do `rake` by default, and that now works?
As a general rule it is dangerous to modify the code that is under test. It is very easy to start fooling yourself into thinking that everything works, but in reality all that happened was that the test code showed that another bit of test code did what was expected (or maybe never could have failed because of a bug in the tests).
OpenBSD has a continuous evolution release model. The release numbers change twice a year like clockwork and reflect the release process phases rather than revolutionary changes of the platform (which are intentionally kept to a minimum). The system probably doesn't change radically enough to merit a release number in the os name here.
This should really be `http://aleph.sagemath.org/static/embedded_sagecell.js` now.  We have the old URL working for backwards compatibility, but going forward, it should be served from the static directory.
no worries. ill add it back in after the merge
The description needs to be updated as well
i believe document is wrong here.

If you click on an link in a dropdown menu it will close now as it is bound at the same level as the data-api
Could you add here better description why we need clear defaults? it is not clear for me
You're using floats here, `gmp_strval()` will return only int part
No need to capture this to a class variable -- just capture to a local variable.
diracPage.js is a nightmare. Please refactor in a modular way.
Space after `:`

`toggleAll:false,` -> `toggleAll: false,`
all the bindings required for non-plugin use
Wouldn't it be better to use `ch3 = ch2 + peek(2)`, saving a call?
IMO, selecting `task-list` directly is better as you might change your `ul` for a `ol`, but that's not a big deal ^^
this should be ``of type`` to be consistent
is plugins flag a proposed standard for the loader plugins api?
@samiluukkonen Täs kohtaa pitäisi mun mielestä käyttää $varasto muuttujaa ku se on viilattu tuol ylempänä kuntoon ja ois tähän tarkoitukseen the one.
$tilino_triang != 0 muutos lisätty.
Nope, I have a production app up on Heroku and I have it set to `config.assets.compile = false`. The one that needs to be set to `true` that isn't normally is `config.serve_static_assets`

`assets.compile` tells it that the app can compile the assets at run time, that you don't need precompiled assets. You don't want to do that in production.

`serve_static_assets` tell rails who is going to serve up precompiled static assets. If you have a traditional web server then you set it to `false` so only the web server serves up the static files. On Heroku there isn't a traditional web server in front of your app, so you have to set it to `true`. This tell rails it can serve up the static assets itself.

Does that help explain it?
Empty line to separate import from Python API and local package imports?
Application is better than program.
CHUNK_SIZE here should be chunkSize since it isn't a constant
return appletsCurrentlyInPanel.indexOf(uuid) != -1; would be simpler.
What happens when the parameters aren't a string? Shouldn't we be passing in native types here from the json?
Since messages are being localized, I'm assuming there'll be a defined list of messages. It seems wasteful to repeatedly store the english in the database. Plus the string extraction won't work on this dynamic message right? Seems like it'd be better to hard code message choices and store the choice rather than the message.
Uber nit, but why are these spaced out like this?
Should be 'Open sans'
Can you do that inside the note block?
it works locally.
Is it worth testing the incorrect user case as well as no user?
Just to make sure I'm following, the addition of 0.5 here is just a small fudge factor to make the magnitude of previous days always longer than a 'next' day.
This should be

```ruby
assert mock.verify
```
`exports.sortMethods`?
stubにしたので必要なくなっていました・・・
:+1: スナイパー
Broadly speaking we've been leaving whatever the scaffolding generates, and not deleting it pre-emptively.  That may or may not be the right choice long term but right now, while lots of us are getting up to speed with rails, I for one am finding it handy to have everything there because it helps me understand what's available.  I could be talked round, but right now "delete unused files/bits of files generated by scaffolding" isn't something we're requiring in terms of code style.
Instead of indicating ``IsPrivate`` or ``IsGood``. These are values returned from the web call to get the inventory.
Seeing as \purple is defined on line 56, we probably do, just not in PURPLE form
(basically i defaulted to leaning on the side of security)
Change spelling to "et cetera"
There's supposed to be code in there to prevent saving those generic names. Please do file a bug if it's no longer working.
If this is a regular properties file, then non-ISO-8859-1 chars should be `\uXXXX` escaped (use `native2ascii` or a text editor like NetBeans’s which does this for you).
Why not just not subscribe if instrumenting is false?
Questionable. Not sure whether he wants to use it.
I see your point, but the test is working as it is. I also tried to add a RelativeLayout object to a LinearLayout parent object, and the test ran without failure. Just like this:
    public void getLayoutParams_shouldReturnRelativeLayoutParams() throws Exception {
    	LinearLayout linearLayout = new LinearLayout(null);
    	RelativeLayout relativeLayout = new RelativeLayout(null);
    	linearLayout.addView(relativeLayout);
        ViewGroup.LayoutParams layoutParams = relativeLayout.getLayoutParams();

        assertThat(layoutParams, instanceOf(RelativeLayout.LayoutParams.class));
    }

While if I replace RelativeLayout with LinearLayout in the assertThat statement, the test fails.
sure, i can buy that.
Did you intend to add this file to the root of the source repository?
The `fieldValue` logic is exactly the same, and it's the only method in `ScalaReferenceType`. That's why I thought it can be mixed in (inherited) by every thing that has fields, and maybe `invokeMethod` can be abstracted the same way. But I let you decide what's the best way to avoid code duplication, just my two cents.
Don't need this - console.log reports line numbers
These aren't really necessary. The convention in the rest of the code is to not store every option name in a constant.
Dit kan ook weg denk ik
Да, это так, по этому при передаче значения я беру остаток от деления на 100: [cell1 mutate:arc4random() % 100];
Конечно правильнее было бы деление внести внутрь метода.
Вся программа будет выполняться: 1597.0ms )))
Программируя, мы обязаны принимать во внимание множество факторов: рамки ТЗ, рамки типов данных, рамки памяти, рамки архитектуры, те рамки, в которые мы вписываем себя сами. Если мы эти рамки расширяем, то мы обязаны сделать всё по уму - иначе потом всё разрастётся и свести концы с концами будет сложно. Задавая изначально быструю возможность изменения количества элементов в последовательности, вы расширили свои рамки, и обязаны их прикрыть, обрабатывая этот факт в других методах.

Нет абсолютно ничего плохого, что в процессе обучения один подталкивает другого обращать внимание на эту тонкость, если с этим свыкнуться, то и дальше проблем будет намного меньше.

"Свободный университет" даёт возможность поучиться, и сделать это не в одиночку, а в компании других таких же, и я буду учиться и буду делиться своими знаниями, навыками и так далее. Если у вас нет времени учиться - это целиком и полностью ваше дело, всё правильно. Тут нет цензора, который будет следить за чистотой вашего кода, и гибкостью ваших решений. Тут есть знания, принципы и навыки, которыми делятся/которые получают участники этого проекта. Если бы этого не нужно было, то сделали бы автоматику, которая по заданным параметрам проверяла бы проекты студентов.
Если percent = 80, а DNA_SIZE=200, то по формуле получится (80/200)*100 = 40. А 80% от 200 будет 160.
i would lazy require this when the options.policyPort has been set.
This should be `s/left/right/`, yes?
This line is a noop, right? Just double checking, I can clean it up after I merge.
server: new client %q
same here. Better be explicit. For newcomers the environment vars practice can be confusing.
np.dot fails now if morph_mat is a sparse matrix. Use morph_mat.dot(X)
this should be a private attribute then `_projs`
How about `allowed_keys`? Validate is fine too, but I think it would be clearer to emphasize that we are checking a list of known keys and not doing any other validation.

The implementation looks fine. For small lists as `ValidKeys` is likely to be, the simple linear search of `lists:member/2` is going to beat fancier approaches like creating a set and such.
"two digit" vs "two-digit"
"It value" vs "Its value"
超过时
直接意译成
这时
吧
好像更明确一些
统计失败尝试次数的时间段。在这段时间中，服务器失败次数达到指定的尝试次数，服务器就被认为不可用。
So... How i can just send you some changes... Or I remove my pull request, and you merge manually ;-)
Eeks missing space.
Is this alginment not required anymore?
Либо forward declaration не нужен, либо инклюд 8 строками выше лишний.
Лучше таки возвращать значение event filter'а. Как говорит документация:

    Notice in the example above that unhandled events are passed to the base class's eventFilter() function, since the base class might have reimplemented eventFilter() for its own internal purposes.
Inappropriate when you're not using a tab
you can probably use (recur child-group new-node) here instead of  having the function call itself by name.  The difference is, recur does not consume more stack (and therefore can't cause StackOverflowError).  Also, recur can only be used in the tail position (meaning that it is the last thing evaluated).  But in this case, i think it is in the tail position.  worth a shot at least.
Just a small note, out-commented code should never need to be checked in when using a version control system.
If it's code that's saved because it used to work and it's nice to have in case things break, then this would be available in the commit history.
If it's code that's there because it's not finished yet but things around work, then this should be in a separate branch.
if sensitive is set to true, the check will be case sensitive
`routing_number`
I think this will need to be called only after the launch is successful on the other side (to ensure that we can rollback). Also, is this supposed to be vms_conn.migration_post_bless()? I prefer post_migration() to the migration_post_bless() used below.
Uhh no. We put some regex parsing in order for the user to pass arbitrary things like 1gb or 256MB. So it has to be a string (and the params.get stanza takes care of converting appropriately)

@dscannell I understand now why you want the special case
Was this actually never added?? :)
You should log the exception in all these cases.

This code could be shrunk with a local function, like this:
<pre><code>
def fail(status, ex, msg):
    LOG.error(msg)
    LOG.error(ex)
    self.update_db(task_status=status)
    raise ReddwarfError(message=msg)
...

try:
   ...
except Exception as e:
    fail(BE_DNS, "Error creating DNS entry for instance.")

</code></pre>
kill cache manager in a finally section please...
Same as below, the tx shouldn't have any modifications.
*`YES`

No space after `error:`
The difference is that a check here would warn on import of matplotlib, not just upon using the PGF backend.  Also, the check in the PGF backend falls back on xelatex if the value is invalid -- I'm worried that if someone sets this to "latex" (trying desperately to get this to work with classic latex, which of course it won't), they may become frustrated this it isn't "doing what was asked", rather than getting a message that what they want to do isn't supported.
I find this decscription confusing -- even as the person who wrote the original version of the projection code.  Why would an end user be using `matplotlib.projections.projection_factory`?  Maybe we need a code snippet before and after.  This description of the change should be at the level of things discussed in `doc/devel/add_new_projection.rst`, which doesn't touch at this level of detail.
Perhaps, I'm not familiar enough with TAW to know.  I copied this test from somewhere else, I forget now which test though, and assumed it was good enough.  At least it worked when I tried it with the fix and failed when I tried without the fix.
@gsilver 
The `outline: none;`

Where exactly did you see that in the lhnavigation rendering?
@gsilver can we change this to single quotes?
Perhaps change this to foreach($this as $item) as well?
Okay. I also personally don't mind just referring to it as `$(lastToggle)`, especially if we reduce the number of times we refer to it as per [this suggestion](#discussion_r1316156).
noted. :)
This file is gone in the most recent master branch.  I have switched things around so cucumber runs locally now; gone are the days of the test runner and pushing everything up to the test lab.
Actually, it's a github issue, it seems to omit the "<foo>" part while displaying the comment.
Are you sure that the method "buffer()" exists? Please also check line 214.
Any clue what this is for?
please keep it on the same line
Almost! (model, **error**, options)
增加array和null的测试用例。
It'll equal "1", not "true".
"values" is used by HPCCDriver.getPropertyInfo. Which is a Static method usually called by a client before connect-time.
I forgot to correct that part.
Why the preceding white-space?
This is good to point out (and is the same behavior as Factory Girl, I believe), but again, I'm not sure about the example. Something like `name=` seems like a simple `attr_accessor`...maybe an override that does more than (what I assume) merely assigns an ivar?
You should move this to `EmailConfirmationToken`, no?
I edited my comments above. Please read them on GitHub.
Very not good merge!
Don't have hard-coded URLs.
There is no need to define temporary variables. You could just populate context with them.
This was igypigi's idea, he will change it.
Its easier to select name for displaying :)
Can you please update (add) parameter doc?

In particular, I'm wondering what type `asset` is now (it used to be a `dict`, but now it's something else).