labofsahil/github-event-dataset-2012 huggingface dataset:
Rewrite of showImages

Rewrote most of the `showImages` module so that all of the site specific implementation is in a self contained sub module for ease of adding and removal.

This includes the gallery browser. If you merge this you *should* generate your own tumblr api key. And copy the then new sprites for the gallery browser to the main spritesheet. (Sprites currently hosted at http://i.imgur.com/VyouB.png, soruce svg/png files can be provided)

It may be a good idea to have a separate expando button image for galleries.

I did not re-implement flicker integration because I am not comfortable with the html scraping method.

I did not implement imgclean integration because it appears to me that they would prefer that the images are not directly linked to (they imply this in the FAQ and they did some minor work to prevent access to it) as well as the html scraping issue.

I did reimplement picsarus but there do not seem to be any posts from it on reddit.
removed system calls and glob function in FileManager

removed glob functions call, system function call and rsync call.
Replaced with symfony2 Finder and Filesystem classes
Add PHP 5.4 support

This fixes the problem that DokuWiki with the repo plugin fails silently without any error message in PHP 5.4.
Add support to load the map asynchronously

Hi,

I have added a few changes to been able to load maps asynchronously, it works for me.
Update russian translation

Added indexing
Sync with latest English version
Adjusted config to improve translated output
Octopussys revenge
Link to Aeolus nightly builds
Remove Cloud Foundry references
Fix: An instance 0x685f030 of class LocationFetcher was deallocated while key value observers were still registered with it.

An instance 0x685f030 of class LocationFetcher was deallocated while key value observers were still registered with it.

Observation info was leaked, and may even become mistakenly attached to some other object.
Set a breakpoint on NSKVODeallocateBreak to stop here in the debugger.

Here's the current observation info:
<NSKeyValueObservationInfo 0x6e44970> (
<NSKeyValueObservance 0x6e4a120: Observer: 0x6848520, Key path: fetchingLocation, Options: <New: NO, Old: NO, Prior: NO> Context: 0x0, Property: 0x6e44650>
)

The OnScreenButtonsViewController was observing its locationFetcher's fetchingLocation property and was zeroing out the reference to the locationFetcher in its dealloc method before removing the observer.

Changed the order of the calls to make the warning go away. All's now nice and dandy. :)
A little fix and changed Paste to waitress

It seems Pyramid uses waitress now.

I also had to test if `id` is in matchdict.
replace petclinic with petclinic-simple

replace petclinic with petclinic-simple
updates for 1.4.2

compiles against mc 1.4.2, and latest BC git
fix broken gps

due to missing assets, gps was broken in the last few nightlies.
this fixes the issue by correctly adding the missing files to leo-vendor-blobs.mk

furthermore this removes a duplicate entry in aforementioned file.
use binary search instead of plain do-while to accelerate the function. ...

...( It improves the speed significantly when maxFontPixels is large )
Issue 56

This simplified things for me and existing bindings on the change event that I had on the original checkbox still work.
Automatic load of older posts on profile does not work

Automatic load of older posts on profile does not work. It is because in https://github.com/phpBB-Social-Network/phpBB-Social-Network/blob/develop/root/socialnet/js/m.userstatus.js#L570 there is
```js
$('.sn-ap-getMore').size()
```
what should be rewritten to
```js
$('.sn-us-getMore').size()
```

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4421
Start using IMailHost utility to get more flexibility while sending mails
Update rubygems before installing dependencies

REE and JRuby ship with RubyGems 1.8.9 that are not compatible with Rails 3.2. We [recommend](http://about.travis-ci.org/docs/user/languages/ruby/)
people to do this because older versions of Rails seem to work just as well with 1.8.15.
Fix get_alphabet() example in readme

Hi,

I noticed one of your examples was wrong and fixed it.

Best,
John.
Closure webworks.js to prevent require leaking.
fix address content overflow
Глава 4 - "За Пределами Структур Данных"

4 глава и мелкие правки за ее пределами
Looking to limit the max SMS and MMS default size. 500 and 100 are big a...

...nd I think users would prefer to keep the default levels lower to save time when backing up and restoring SMS and MMS messages after a data wipe.
Allowing multiple instances

Addition of instance_allow_multiple(), returning true.
rowend/4-admin-catalog

Translate to spanish the admin catalog folder
Add basic Rotation settings UI

Hey,

I like your initiative to create at CM9 settings UI. I'm porting some stuff from earlier CM versions to v9 and since there is no official settings UI yet I use a fork of yours. First contribution is a *basic* UI for

http://review.cyanogenmod.com/11310    (please +1 it :))

Initially I'm gonna focus on porting the actual mods, making the UI pretty is lower prio. Feel free to help me by pulling in my changes to your code and convert them into UIs that doesn't make your eyes bleed :) The important thing is to get most of the old mods in and keep all settings in one place until they are in the official repos.
We need to call trim() function
Technically, we should be using the maximum sequence within the current press rather than the global maximum. Also, I'm starting to use even numbers only for sequence values, because that allows us to move any item between two others (giving it an odd number) and then resequence -- see FeatureDAO. Previously we were using floats and adding halves (0.5) rather than odd numbers, but that's wasteful.
Well, I use traditional shellcommand abook --mutt-query & regexp solution, which outputs one line per email. So it is not a problem. But I suppose the abook/addressbook parsing method would do like you say. It needs to handle a list there (OR a single address without any comma, which—if I understood correctly—is not a valid list according to the current config parser).

Hm, well at least the abook program doesn't display an entry which is missing name, or where name is empty. I tried that. But name being just a space is OK.
The bigest problem is that the merge is the wrong way around. You merged master into your dev branch. That mean you replaces history on github with your dev branch. Ie, my commits from yesterday can not be seen easily anymore (that's why this diff here shows the changes from them).

You must always merge your feature branch, into a prestine copy of master if you are to push anything.


I've said it before... We must find a way to block these types of merges. They are note so bad when there is only a few commits difference, but they are a mess when the base commit is far back and we loose ability to view history for master on github.
Should be fine. yet, it will need a bit of testing to make sure.

Try also to change your NPC and monster colors in the setup->colors to see whether it's effective.

Thanks for the fix!
@zephod seems like the standard dataset listing template snippet didn't work for you here. Can you expand a bit on the TODO you've added. It would be nice to keep this common so we don't get inconsistent look and feel for same thing across the site.
The hd-trailers.net stuff is handled differently but yea, we should most likely change the wording. Not sure if it's going to happen now as it would require all translators to come up with a new one.
Wow, didn't realise anyone was actually interested in this project!

I'm having a little trouble understanding what's been changed here - has each file been copied into frameworks/php-mvc? I don't really understand the point of that.
which is why Fabien made it part of the Symfony2 CS (which are applied in Behat) :)
Keep the braces on separate lines, to keep it stringent (even if there's no body).
提个建议，希望作者在本地修改的时候能够降低commit的频率。goagent的更新我在关注...可每次都更新信息都太小太碎且对主要功能影响不大，所以提出这个建议。
或者希望作者能在本地push到github之前利用git把多次提交记录合并一下。
^_^
Is it guaranteed that "size" >= 4 here?
Thanks, I saw the commits go by. Will check them out in the next day or two.
And it was mostly implemented with my baby sleeping on my chest! :)

`rb-inotify` & `rb-fchange` should be easy to add now!
Since you have S available you might as well just return S.Zero.
I looked through the fourier transforms examples (since this was the shortest file) and added explanations as to what's happening. (you may wish to "run all" again since my output looks quite hideous ^^)
https://gist.github.com/1627344

I'll look through the other two later.
вот здесь хорошо видно преимущество гемов для форм.
Базовая форма занимала 32 строчки, теперь порядка 8!!!
Jenkins is not amused, *you will feel his botty wrath!* http://swarm.jquery.org/job/983/
Reading stty documentation I discovered that it is possibile to use RTS signal in hardware to realize the handshake with external device. In RS485 this signal not exist, but sometimes is used to switch between transmission and receiving data.
This is the case on device that I use but unfortunally not works.
In other cases I hope that this can help.

Obviusly if RTS_HW is used, the function to set and reset RTS bit is not more used.
Could your explain me the principle behind hardware RTS (or give links to documentation), please ?
MODBUS_RTU_RTS _** could be considered software only so don't need to check for RTS_HW ?
Correction, I'm just being stupid and wasting our precious time. I'd use `paste` there too... dammit, why don't I just keep my mouth shut...
It was like pain in ass doing it, I've spent like 3 weeks on it and still wasn't complete. But thanks you finished it.
Also removed `make clean`... It's too harmful for a command...
But that interface is empty and seems to be in the wrong file (src/Cache/Filter.php).
Yeah. Need to shorten that line.
hmm... what about the "Make set" button being the last and named "More elements" [warning: just an idea, i may change my mind]
you mind renaming ServerIp to just Ip?  There doesn't seem to be any info Server specific in that object
How about this
String path = "/var/data/stuff/xyz.dat";
String base = "/var/data";
String relative = new File(base).toURI().relativize(new File(path).toURI()).getPath();
// relative == "stuff/xyz.dat"
Heh — whereas my approach is exactly the opposite. :) I would *only* write low-level tests for `urllib3` because I am not interested in whether any particular web server, whether Apache or nginx or wsgiref or eventlet, can correctly read HTTP. I just want to see the HTTP text that `urllib3` produces; see it react properly to HTTP text that comes back; and, when modifying the library and running the tests, I want to know immediately if I have done something that makes even the slightest unintentional change to the requests that are being issued, whether some particular web server would care about them or not.
ty undergarun this update solved a lot of crashes from my server
I get this:

```
/home/jeff/src/julia2/julia/lib/libarpack.so: undefined symbol: dzasum_
```

I don't see what `-rpath` is doing, since if we use our own blas it is still statically linked. Also, rpath needs to be set on the executable. But, if we do that it seems it will tie it to a particular path and binary distributions won't work.
Estaría chido colocar sus apps en un repo distinto o branch, así sería mucho mas cool… ;-)
Is there any way to fix this from github? It takes like 5 minutes to find it in my git repo and navigate to all the different sites for the pull request.
Ya I was and mine was off wow wotlk icon the w letter with the frost and ice over it super cooler then that gay ass mangos icon
http://pastebin.com/k7Mmgdfj
У меня дефольтно стоит, + на сервере время GTM 0
1. дельфи есть подо все. но нативного кода он не дает нигде уже лет 10.
2. сейчас вы скомпилили либу x64 в .NET псевдокоде. сделайте сначала нативную, тогда можно будет и о скорости говорить... но ведь не выйдет - там код на страницы завязан...
3. прилинкуйте к x64 коду любой мемманаджер, скомпиленный (как у вас) в x86 или псевдокоде - экономия памяти будет та же, если не больше.
мне все эти менеджеры глубоко пофиг. однако кажется, что вас кто-то надул :)
PS я пишу в VI.
good, one crush in oPvP founded. need check on if (!player) return in start of this method
Seems fine.  We'll properly handle "sync" XHRs in the future.
Is not that is faster. Is that it works for matrices that are not squares.
Is basically the best solution you can get, but sometimes if you do that
product and compare it to the identity that solution can seem quite far.
In case of a square matrix I'm pretty sure it gives same answer as
matrix_inverse but it might be slower and it does not get optimized into a
solve.

On Fri, Jan 13, 2012 at 3:25 PM, nouiz <
reply@reply.github.com
> wrote:

> You tell as close as possible. There is a typo in possible. Also, I
> suppose from the name pseudo-inverse that it is an approximation. So can
> you tell that if we need a more precisse result you can use the inverse
> that is slower? Or I mis understood something?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/pascanur/Theano/commit/3301d8d83ad24181358684cc0dcd7a01c13ebd26#commitcomment-866070
>
compile error right here... "	18	IntelliSense: Operatortypes are not compatible ("char" und "const char *").	c:\users\sharq\desktop\solidcore\src\server\game\server\protocol\handlers\movementhandler.cpp	153	88	game"

i already fixed alot of build errors in this patch but im stuck on this one... cant figure it out as i really cant see any errors or typos...
I was thinking about that, I've made a PR.  Please note the documentation is wrong at php.net for the interface, I've an open ticket on it.
damo must not have pulled the latest?
Add some whitespace padding around src
Bah. It breaks to many poorly written tests. I cleaned it up with getattr though.
it was out of the django docs so I think you have to use the i in total. I think the total gets messed with. hmm, maybe it's for deleted forms which means my check below might be redundant.
Added your commits, ran tests, bumped version number and released into wild.
r+wc works as advertised! I like!
I'm sure django does something there, but the thought of starting at / worried me. Presumably theres some prefixes that get added so /etc/passwd isn't possible for example?
pero es que haga lo que haga explota el assert_difference sin sleeps....
Hehe, ville testa ifall man kunde även göra @oleander (personreferenser) i commit-meddelandena. Vilket jag antar funkar iom du blev notifierad. :)
Commit 5f65ea58b6fc1000b5bfdfdc3a5dcb632dbc3ae2 fixes this.
I don't like having this in production code, but I do like that you've guarded it fairly well.

You also mentioned in IRC that this stall isn't in prod code, so I'm probably looking at a stale commit.

Good job writing tests, this is a really trick thing to test.
oh right hahahha .. duh
Of course it doesn't, I'm not done yet ;)
If you expect something yo work, please use the master branch :)

2012/3/1 queeup <
reply@reply.github.com
>

> script/subliminal bot working anymore.
> ```AttributeError: 'module' object has no attribute 'Subliminal'```
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Diaoul/subliminal/commit/28b99ad37445425202fd86ba80fe8ab2973ac487#commitcomment-1031711
>
Why not change the namespace of official plugins away from org.spout, and remove the crazy exceptions?
All widgets, by default, aren't drawable.

That changes the meaning too, I think. Actually every widget has its own Canvas as tshirtman explained, you can use  it to draw by adding graphics instructions to it but by default  it doesn't contain any graphics instruction, except for widgets like Button, label. 

In the interest of keeping things consistent,  "graphics instructions" seems a better fit than "drawing mechanism" as "graphics instructions" are used later in the documentation to refer to drawing instructions like Rect/Color... .
with open(...) as fp:
        chunks = fp.read()

would be nicer
@honarkhah Thank you for this.

I have updated the release/0.5 branch with another value in the /install/language/english/install_lang.php

If you could update your fork and then make the changes in the release/0.5 branch of your fork and then submit the pull request that would be great as I would like this to go in the next release.

Thanks
set user define method interceptor.
ACE有保存功能么？
Hm. What if i equip a bunch of arrows? Won't this get counted only one time?
@Harshavardhana, I don't want to be rude, but taking a copyright on a code you haven't wrote..., can't you be satisfied with you name added to the contributers list ? (I understand you want to help out, but leave the credit for Andrew that wrote the code)
I'm working on implementing frozen modules and such, which needs marshaling support. Currently the only place the marshaling was available was in the marshal module.
@nfedyashev Спасибо!
commenting on this line!
I like how you used the background image in the banner.  Do you know what the "background-repeat" does?
thanks for the catch, @andrewjstone. merged.
here if self is NULL it will return -1 even if other is NULL as well.

what about adding
g_return_val_if_fail (self != NULL && other != NULL, 0);

as first check? it very depends on what you think it's best, but what can happen is that you'll order something while that something is not existing. Might it be a problem?
님좀 짱인듯. good.. kkk.
Are the other commits in this branch needed for #177 or something else?
Actually, I had switched to tabs while doing this update, but after the pull request.

I think I'll close this one, remove the jar, and file a new pull request.
SlimJim :-D "fail" is JSTD specific API.
Added a note into comment message.
OHRLY?!!?!! :-p

I mean a new pull request needs to be made as this one is done, and every pull request needs a use-case and an explanation.
here's a few ways to break out as well: https://gist.github.com/1764684

They run fine in the the Console or Scratchpad.
Huh, will you look at that.

As far as `_` is concerned, I think it's safe to use if you're not using it for gettext (or similar). Unfortunately, there's no other non-letter character that's a valid Python identifier by itself. I guess I could recommend using any single character, which are probably not recommended as variable names anyway (except in the case of counters).
I still think Service Builder is an internal tool that should not be mentioned here. If in the future we decide to stop using Service Builder and generate the SOAP API in some other way, the users of this API won't care about it. I would change the beginning of the sentence above to:

Liferay's servces also provide acess via *Simple Object Access Protocol*....
Not sure I like this yet.  The dynamic lookup of attributes should be outside the inner loop.

The iteration with chain() is an idea, but the attribute lookup can be eliminated.  e.g.

     TreeCursor.prototype.nUp = function(i) {
         return this.chain(this.up, i);
     };

     TreeCursor.prototype.chain = function(f, i) {
         var n = this;
         while(i--) { n = f.call(n); } 
         return n;
     };

     // ...


Similar idea for the canChain work.  Rather than use the name of the function as an argument, use the function itself.  The up-side is that this approach should be able to avoid all the string slicing.

Do you want to try your hand at it first?
This can be:

    bin.install 'ecm', 'unecm'
Naturally, the same goes for projects.
There is no such file as travis/model any longer. It causes an exception on
travis-hub staging, that I deployed a while ago with fresh travis-core.

Besides that my modem suddenly died and I'm almost totally disconnected
from the Internet now =(
Spiffy thing we're doing in Funtoo, we've got * and ~* keywords now. So this one would probably be ~*, unstable for all.
Why would you add this?
the old way was correct
Not really related to this commit, but there's a bug that crashes the deamon when the logfile gets too big, not sure if it's fixed yet.
I might be missing something critical here, but for my uses case (sending syslog messages to the local syslogd) the majority of this method is causing problems for me. Here are a few items that I just don't get...

1) If syslogd is listening on localhost:514 how would this socket ever bind?
2) If you want to send messages why would you bind and listen on a port?
3) What's the point of binding and not handling 'message' events?
Makes much more sense. If we're printing, probably better to make a print_anagrams method.
Can i blame you a bit Nay? :)
Yes. "Wild Dogs" and that's it.
-DELETE FROM`creature_addon` WHERE `guid` in (133434,1 33435);
+DELETE FROM`creature_addon` WHERE `guid` in (133434,133435);

??
Stop trolling! Fuuuuuuuuuuu
Did you guys actually read what i said? use SMART_EVENT_GO_STATE_CHANGE with param1 = 2, it will be triggered when the go is activated, aka, at the end of the cast
1. We need spec/test for krypt itself (Ruby) that implements krypt API using krypt-core API, and spec for krypt-core API which is implemented in C and Java. And we're searching the best location for the latter, spec for krypt-core API. Am I understanding correctly? Then yes, krypt can be a place for the spec for krypt-core API. I think we should split krypt-core API spec from krypt API's one, such as spec/krypt and spec/krypt-core, then we can easily refer krypt-core API spec from krypt-core (C) and krypt-core-jruby. And spec/krypt-core/asn1 would be one of a source of deliberative for the grant. Can reorganize spec dir of krypt?

2. For krypt-core-jruby, I'll wait your work. After I give a try to create C-ext coverage build harness, I'll try to improve spec coverage of krypt-core and krypt-core-jruby. (https://github.com/emboss/krypt-core/commit/9c1565bc3b78536e4c9889fa2ea4f566e0e925eb)

3. I'll give this one-day try first.
Hmm, I don't know if that really qualifies as the same problem. At least the current behaviour is better than it was before. But I'm thinking we probably want to allow `### comment ### identifier`... @jashkenas: any insight?
Darsyk
Could we just replace these two lines with

    NSMutableArray *fileObjects = [[[QSLibrarian sharedInstance] arrayForType:QSFilePathType] mutableCopy];

and then use that for the rest of the method?
return Arrays.asList(this);
And more one. Why at db included non blizzlike patches like teleporter and pvp-seller?
@xuhdev do you think this documentation should go in the `editorconfig.h` or `editorconfig` man page (or both)?
El error en el desplegado de los bloques de noticias estaba en el archivo modules/article.php por alguna razón a partir del nuke 8.2 los suprimieron.
Is this font size actually working for you?  18pt seemed to work better for me, but it's not ideal.
>Much like how the 'auto' return type mysteriously only works on templated functions.
What do you mean? This compiles:
auto blah() { return 3; }
My bad. Jag tyckte också att det var lite konstigt, men jag gick efter bilden `Hemsidan/Produktion/Hemsida-live-v0.2_top.jpg`. Jag missade att headern fanns med i `Hemsidan/Produktion/Hemsida-live-v0.2.jpg`.
any reason to recreate the socket each time through the loop?
lol yeah, I typed it quickly :P but I don't know how to change it now :P
The form is doing a post back to itself.
If you add the checkbox you should see ```cache=off``` in the post variables inside function ```Ht_Generate_List```
splitEvery (length testPairs `div` clients + 1) testPairs
Why this change? Isn't changing the license something for the original author (if it's allowed at all, though limiting the available licenses might be ok, not sure)?
not needed dude, we build it
should't we use this.$el.html modelView.el rather than append? When i use composite view and invoke reRender or render again ... it append the view again to the DOM, shoudn't replace the view?
Wow. So many comments.
Fork it from cm github, then cherry-pick your commits or add your
repository as a remote, then fetch n merge it

And yes - setup remotes properly

Git@github.com:sgt7/xyz.git

Use git remote -v to check
On Feb 3, 2012 11:32 PM, "Scott Brady" <
reply@reply.github.com>
wrote:

> Thanks for adding that, can you make our frameworks/base and settings
> read+write.  When I moved them from my github it revoked the permissions.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/sgt7/android_packages_apps_Phone/commit/59f7eaaf203b634ce39f6a0625ebcfda4198d77a#commitcomment-931174
>
It was only supposed to match images that are larger than the limit in both dimensions, to avoid pestering people who were uploading scrollers and the like. More of a sanity check for people uploading *really* big images who may not be aware of it.
Are you aware that this only works in a flat wiki, i.e. you would need to do some kind of recursion for namespaces (and subnamespaces, ...)
Oops, sorry, i didnt see, Lol

will change ;)
When the corresponding framework change is merged in the other repo, you need to update the Framework git submodule so that your change is included.
This also seems expensive and not always necessary.  I think it would be better to have this logic in the public_ip_address method of server to be called when needed instead of always called as this does.
hey @gvalkov , I'll try to create the user site test problems you were having and see if there's a workaround.
Most init function can be written like this:
node.addChild(ADM.createNode(BWidgetRegistry[node.getType()].zones[0].allow[0]));

So we can implement in the base list widget and extending widgets don't have to write the init function
$.rib.confirm should take a callback function as its parameter or it will still use the old confirm function and we don't get UI improvement. Besides, I still think it's confusing to ask user about if he/she'd like to delete an unused file.
Shouldn't this be: `this.getClass()`?
In this case yes. We more or less consistently use `super` in other places in the code so I thought I'd adapt.
just thought the JS could actually strip "Controller" before sending the request as well.
ooops, forgot to rewrite this one :+1:
Btw, what's the reason for not connecting right here?
I'm a bit sceptic about this symbolic operator `-`
It's not in symmetri with `:+`
I'd simply name this `remove`
when using one parameter per line it's preferred to start the first one on a new line
no not really, you can start seed2 and seed3 before seed1, but they will not be "active", until they have been able to join another seed node (seed1). They will retry the join procedure.
So a possible startup scenario is:
1. seed2 started, doesn't get any ack from seed1 or seed3
2. seed3 started, doesn't get any ack from seed1 or seed3 (seed2 doesn't reply)
3. seed1 is started and joins itself
4. seed2 retries the join procedure and gets an ack from seed1, and then joins to seed1
5. seed3 retries the join procedure and gets acks from seed2 first, and then joins to seed2
This is the actual example  ....
Alright. Stay tuned.
I'll review again in the near future after I take a bit of a break.
If it derives from ODFModel, then I believe it is?
Can probably just remove these; it's a private method and anyone getting this far will know what they are working with.
Sure, I'll take any excuse to move code out of this file.
I am a little worried about this interface name. Maybe we should call it "GenericCapable" or "Genericable", I am not so sure if this is the exact term.
To rename and/or change the type of a setting you have to create a SettingsUpgrader and put it in the UPGRADERS map with the new version number as the key.
Ah, right. In that case I might move the whole discussion of distribution strategy into the PredisCluster-specific section. And briefly point to how RedisCluster takes care of it with ASK/MOVED.
This assignment looks useless as next line also sets this property.
The restriction is that with the patch "*" is hard-coded.
This change would prevent hovers over icon buttons from highlighting unless they were the active member of a enyo.Group.
allright, what integer will come back if the symbol was found?
Perhaps we should just start adding useful stuff like this to string from luvit?
I don't know what default would be the best, are people used to the new Merge that have been there for months?
False in that case would maintain the new way of merging that is flattening the list.
Looks like the line should be broken, it runs off the right on the github display.
This should also be added to all language files.
По сталгайду геттеры называют просто edges()
нотация именования полей с m имеет смысл только для закрытых полей, а для открытых только мешается. касаемо геттеров-сеттеров вопрос неясный, но конструкторы (по умолчанию и с параметрами) этой штуке сделать нужно точно. да и превратить в класс, видимо, тоже
I don't understand.  You mean the visibility of the constant?
If we want to make it faster this is making a little slower:

```
              user     system      total        real
block     1.680000   0.000000   1.680000 (  1.677796)
sym to    1.840000   0.000000   1.840000 (  1.850888)
[1.9.3-p194 placar-admin (master)]$ ruby bm.rb 
              user     system      total        real
block     1.680000   0.000000   1.680000 (  1.680283)
sym to    1.870000   0.000000   1.870000 (  1.873345)
[1.9.3-p194 placar-admin (master)]$ ruby bm.rb 
              user     system      total        real
block     1.700000   0.000000   1.700000 (  1.707954)
sym to    1.860000   0.000000   1.860000 (  1.868167)
[1.9.3-p194 placar-admin (master)]$ ruby bm.rb 
              user     system      total        real
block     1.860000   0.000000   1.860000 (  1.874244)
sym to    1.930000   0.000000   1.930000 (  1.933034)
[1.9.3-p194 placar-admin (master)]$ ruby bm.rb 
              user     system      total        real
block     1.680000   0.010000   1.690000 (  1.677366)
sym to    1.850000   0.000000   1.850000 (  1.857757)
```
This call is not right. the call should be

```ruby
link_to("Hello", hash_for, :remote => true)
```
probably shouldn't mess with other CHANGELOG lines.
I think we can just change this slightly, how about:

`If you already draw these, you must change these, => If you already draw such routes, you must change them,`
By my benchmark test separate iteration is faster than my example about 1.5 times :)
We should not require rails here
Good catch. Fixing now and trying to update a little.
I thought it made sense to make the bullet points talk about alternatives to the way that we do it, rather than lumping them all together.
Ok, I'll do that. I'll probably also need to check what happens when there are multiple throws annotations of the same type. What semantics would be reasonable here?
let's not let people get sloppy about this...it fails in master if you try to iterate over non-sets:

```
>>> set.union(*[set(range(3))])
set([0, 1, 2])
>>> set.union(*[(range(3))])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: descriptor 'union' requires a 'set' object but received a 'list'
```
On 10.04.2012 21:06, Sergiu Ivanov wrote:
>> +        remove = set()
>> +        for (a, b, c) in permutations(S, 3):
>> +            A = sdm_LM(a)
>> +            B = sdm_LM(b)
>> +            C = sdm_LM(c)
>> +            if len(set([A[0], B[0], C[0]])) != 1 or not h in [a, b, c]:
>> +                continue
>> +            if monomial_divides(B[1:], monomial_lcm(A[1:], C[1:])):
>> +                remove.add((tuple(a), tuple(c)))
>> +        return [(f, g) for (f, g) in P if (h not in [f, g]) or \
>> +                    ((tuple(f), tuple(g)) not in remove and \
>> +                     (tuple(g), tuple(f)) not in remove)]
>> +
>> +    while P:
>> +        # TODO better data structures!!!
>> +        #print len(P), len(S)
>
> This looks like debugging output and should be removed, I think.
>


Small amounts of helpful debug print statements exist, in commented-out 
form, throughout sympy [and I did not create most of them :D]. I 
personally think it's good to have them.
I have added doctest for all user-facing methods.
(It would be a good idea to correct this before working on the API changes.)
I think you want `rank//v + 1` -- py 3 related, perhaps?
This is as per WolframAlpha. When you put the above into WolframAlpha, it takes 'a' as one argument, 'b+c' as the other, and expands like '(x+y)**0.3' . The other way to do it would be through recursion. That is, expand each bracket of (b+c) as well.
NEVAR!
Let me give a better example of what I'm trying to avoid. Let's say that there was a company that, for whatever reason, continues to develop with CakePHP 1.3 for the time being. The company hires a developer who doesn't know CakePHP. In order to learn it, that developer reads through the 1.3 version of the blog tutorial. The developer arrives at [the conclusion](http://book.cakephp.org/1.3/en/view/1542/Conclusion), which suggests that they read the API. That link points to http://api.cakephp.org/ instead of http://api13.cakephp.org/ , so they could potentially be led astray.

See the problem? And also, like I said before, the very same page that I edited has a link to the API further down on the page. I used that same link.
Perhaps I missed something, but is there a `default` host in your example?
I'm a bit confused here... isn't the getRelativeEffect() derived from the MultivariateNormalSummary? If so, why are we testing against the QuantileSummary (which could be slightly different)?
да. было бы неплохо.
I think "currently accepted" would be clearer than "previously accepted."
I agree it doesn't make sense to display both for every test. Instead I think it would make sense to display measurements for tests and modules as a whole where the test is just the test itself (without setup/teardown) and module is everything during the execution of a module (naturally including all tests and the setup/teardown).

Since the setup/teardown is generally the same, it makes the data more accurate and generally useful I believe.
Why did you add the library if you are not using it?
Do you know whether it's safe for multiple stores to share a DatabaseFacade? (It might be - I haven't investigated)
:method:`db.printSlaveReplicationInfo()`
wiki says: center

I think the docs themselves need to be updated to reflect this point?
redundant to below?
move up to the parent condition? avoid having too many tested if blocks :)
> (I know it will work without them, but according to my knowledge, quotes should be used if it is string)

By the CSS selector rules, one must use the quotes in order to the selector to work. Anyway, this is a selector specially made for jQuery, so it may follow other rules. I made a quick search in the jQuery's manual, I didn't find anything about that specific part. Probably I'd have to search deeper.
I still need to find out what this file is actually for from @p
Unused now, applies to the other files too.
We always have jquery, there is no need to check for it.
If this is to go into the core, it should have its own (probably just "akismet.php") language file, and should not be in the mods directory.
Will do that
This is a link to another comment, github just shortens it down to the PR number
as @bantu mentioned for the previous one, lowercase. Consider using $forum_ids instead?
The DBAL has no knowledge of the column types it is inserting to. Neither is it in general possible to tell the difference between a string and a binary string in PHP. This leaves only one solution: You have to call decode/encode manually on all binaries, which is what is happening here.
A small piece of hungarian notation. It's good for marking class members.
Looks like it does. I'm sending them in unescaped and the request looks like this:

http://en.wikipedia.org/w/api.php?action=query&format=json&callback=test&prop=extracts|images&imlimit=10&exlimit=1&redirects=exintro=&iwurl=**&titles=Anas%20acuta**&exchars=275&callback=jQuery17103386705983430147_1346880064553&_=1346880081564

I'm assuming I should scrape out the escape() and unescape() functions?
old code should be deleted, not commented. it can be found again in GitHub history, if needed.

also you should delete the line above too,
```
var topbar = Ext.getCmp("topbar");
```
since it does nothing now.
can you move this store out of the view? ultimately, it will draw from some database -- won't be simple or hard-coded
if you dont want the message, please delete. no need to comment - it's in the source control history
Lines 658-659:
```
raise _ReferenceError("The file(s) {{filenames}} don't contain"
                      " field(s) for {!r}.".format(arg.name))
```
Not very helpful to the user, seeing a literal `{filenames}` doesn't clarify the issue.
this should be in the config
For clarity, let's say

```
x = scalar()
y = cast(x, 'int'))
z = x + x * y
```

The gradient of z with respect to y should be x. Previously it would have been 0.
The gradient of z with respect to x will be 1+y. This is the same as before. The path to x through the cast doesn't contribute anything to the gradient on x because dy/dx = 0.
This gradient is correct almost everywhere. In cases where adding any epsilon to x should change the value of y by 1, the gradient should be undefined, but we ignore that, the same way as we ignore the discontinuity at the center of the absolute value function.
I don't like the description of indices. I think it is the row index of the corresponding element.

I don't like the description of indptr. I think it tell where start the i col in data/indices field.
Would be more accurate/indicative if defaulted to an INVALID/-1 value since 0 could be misinterpreted (if only by humans) as a legal param index value?
See comments on QueryPlanner.
This file just got twice as long. Really, turning off counting seems of little value for larger keys. I'd say at least don't bother supporting both choices for the generic keys.
This is retarded, since it will no longer complain if you forget the semicolon.

You might as well get rid of the if() altogether, since every compiler in the world will remove it anyways.

Last, but definitely not least; With the if() it's impossible to use safely after 'else' without remembering to add the curly braces on all 'else' clauses.

This change is downright dangerous and could well lead to bugs that are extremely hard to find.

Consider this code:

if(x > 3)
  my_free(foo);
else
  do_something_else();


With your macro, the code will compile nicely, but the 'else' clause will belong to the if() statement in the macro, and so will never be executed.
But doesn't mean the same thing.
did some text get lost here?  there's a period then a fragment
Does this also need the mozTransitionEnd handler?  I saw you added it in the enyo diff.
We just need to modify `config/application.rb` to have the same lines as every new rails 3.2 app:

```ruby
if defined?(Bundler)
  Bundler.require(*Rails.groups(:assets => %w(development test)))
end
```
I agree with you. I have just made the output_type a mandatory argument.

2012/9/13 Lars Butler <notifications@github.com>

> In openquake/db/models.py:
>
> > @@ -1011,6 +1051,21 @@ class Meta:
> >          db_table = 'uiapi\".\"oq_job_profile'
> >
> >
> > +class OutputManager(djm.Manager):
> > +    """
> > +    Manager class to filter and create Output objects
> > +    """
> > +    def create_output(self, job, display_name, output_type="hazard_curve"):
>
> Why is the default output_type="hazard_curve"? For a generic create_outputmethod, this seems strange.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gem/oq-engine/pull/874/files#r1598867>.
>
>
We should use camelCase on all variables. Can you move this to a protected static variable on the class itself because it will never change?
Yea... and if PR discussions was stored in git repositories too... it would be visible that this one was actually the first one :)
Which function is responsible for the discretization?  I'm looking for that in existing code, but it's fairly difficult for me.
oh shit!, i pushed more than i meant to, i'm gonna fix and repush (forced)
Not a bad idea, so I tried now...

If I add the decorator outermost (above the `@dbus.service.method`), the functions are no longer discoverable/introspectable from D-Bus, but they seem to still work. If I add the decorator innermost (below the dbus decorator), the dbus decorator fails because it uses the `inspect` module to look at the function arguments, which is no longer where it expects.
Hi, I updated the original commit. The rootcause is that clang does not support unqualified lookups in c++ templates (also see: http://clang.llvm.org/compatibility.html#dep_lookup)
Good call! I'm new to this - should I change it and submit a new pull request?
Rather than doing all of this, it would be simpler to do

```rb
cd 'src' do
  system "make"
  bin.install 'web100clt'
end

cd 'doc' do
  man1.install 'web100clt.man' => 'web100clt.1'
end
```
when Array
  v.each .... deep_clease if == hash
Can you use this to attach the lookup checkbox in the `MusicSetupVBox`?  Then you wouldn't need to look at the checkbox in the Music tab popup.
Is this too long:

"""Raise client side exception(s) if errors are contained in the API request response."""
Nechci vám do toho moc vstupovat, jen: Lukáši, Java a superstriktní? Cheche :-).
@daniel-uzunu: you should use ``'ws'`` here since it's the correct protocol abbreviation for websockets (actually used when constructing urls). Also, these should probably be lowercase
@staabm Da ich das hier gerade sehe, sollten wir die default Einheit bei getFormattedTime() auf Millisekunden setzen? Ich glaube, die braucht man deutlich häufiger als Sekunden.
You want to make it still default to the expression when a name is not provided.

```js
}, (name || expression + ""));
```
JUnit style is never to import *.
One challenge is that a user might pass an empty array, rather than null, to either includedCategories or excludedCategories.  I think the easiest way to handle this is to replace nulls with empty arrays in the CategoryFilter constructor, and then we can just test .length == 0 anywhere we currently test for null, and we don't have to worry about accidental NPEs on loops.  Make sense?
Do you need $ here?
These are duplicated in the mixin. Seems like it should be DRY'd out.
Remove "File"
I think these are the same as $selectionBase, $hueBase and $opacitySelector, respectively
@redmunds See https://github.com/adobe/brackets/pull/1820
Is there a reason not to just let callers call `new DirtyDocumentTracker()` directly?
I hear this is no longer true.
It is, but `__repr__` is supposed to return a string that can be used to recreate the object, so it should use the same name.
Yeah, added a test in the reminders app to make sure it works (it makes sure that an old version of the couch document can't acquire the lock).
You don't need `:features` as langtool has autoload cookies.
Move this into closeDialog so it happens right after you click.
What about this? https://developer.mozilla.org/en/DOM/window.setTimeout#Minimum_delay_and_timeout_nesting

I just picked `10` because it's ... yeah. Probably should be `0`.
Could you create function, e.g. isValidURL(), in case if other user will be found? Two things, I would add: 1. check if url is null/undefined; 2. add ftp to the list of the protocols.
```
enum TimeInterval {
    INTERVAL_0(0),
    INTERVAL_3(3),
    ...;

    public int days;

    public TimeInterval(int days) {
        this.days = days;
    }
}
```
大写首字母吧
We really need a better way to get the value of `L_tmpnam` rather than hardcoding 1024. This could lead to security and/or corruption issues at a later date. I don't know of a good way to do this though. :-(
You are changing A LOT in katello.yml template, please test katello-upgrade (from Katello 1.1 to your build). What you need to test in particuler is if katello will start with the old configuration file, because this is what katello-upgrade do. It redeploys using puppet at the very end of the upgrade process. Test it.
I agree with Mirek. We definitely not want to show this to all crackers in the world. Let's force them to use nmap and other diagnosis tools ;-) This should be really available only to administrator, not even users (as a multi-teanent app this is also dangerous).

Also, I don't like of spawning two processes for each request. If you absolutely must spawn a process for this kind of information (and I am pretty sure you can get all the info in pure Ruby in this case), do it in an initializer into a constant.
Abusing this spot to ask a general question - I was under the impression were trying to consistently wrap params in parenthesis across the application now?
This would be clearer as a switch statement.
Here is the PR: taldcroft/astropy#3 - @mdboom, does that seem like an acceptable solution?
I would add here a boolean parameter to be able to desactivate printing the progress bar... users of the API might not want their standard output written to.  Another possibility would be to take the output stream instead of a boolean toggle.  I would then consider making silence the default.
You are right, it's fine ... just looked weird because of the do { above.
I had that in mind too, don't ask me why I didn't do it. Perhaps I like rebasing so much :D.
if (model && model->getOptionsModel())
This should just be boolean in my opinion. NULL is a default value for the boolean param. NULL is not a variable type in PHP.
This might change behavior if something inside chaco was listening to changes in the items of the list and not to the list trait. I doubt that is the case, since this is a tool, just checking...
Rather than maintaining a weak dictionary of callback, just a `Signal` from `enal.core.signaling`. That's what they were made for :)
TODO: /* I/O redirection tokens */

The C-part is however not to require any comments as we suppose the user to know how to use `yacc(1)`. The `yytext` variable is a string that matches the regular expression while `yylval` is basically the way to access the union defined in the grammar file.
Um, yeah, <_<
Oops, sorry!  Though it would apply to folks who code in Helvetica ( I know of at least one).
Don't wanna be nitpicky, but, heuh, yeah... Brackets in the same line, please. :p
can the select helper support the same too?


can we add a unit test for it ?
please move this tests out of this file to the other one.

Not sure when this file was split up ( since I did not see a PR, @jairodemorais )

There is no grammar changes relates to $idx, hence
There is no need to check whether `NaN` is bound in a Lexical Environment.
Yes, yes it will (and it does)
or this one.
Ah, I figured out that I can tag on `#desktop` to "fix" that behavior. Fixed below.
I don't get this. Can you explain this please?
Was this a mistake? I.e. to include in this patch.
TENSO
![tenso](https://f.cloud.github.com/assets/118430/9329/72ce07e4-44a5-11e2-8e89-63094a1b3340.gif)
Maybe each cache type should create its own subclass of Caches which implements this check (with the same type check in the Caches superclass). That way we can more clearly separate instantiation and management over decisions for keeping the caches or not.
Is this line really useful? It seems unused to me... Remove it if needed.
At the ruby side, we should use enum types, which are represented as strings. So it should be `t.string faithfulness` etc.
This is inconsistent, sometimes it is called `abandonment_callback` and other times it is `quit_check_callback`
@salmane for this I would suggest using
````$route['default_controller']                = 'blog';````

But I doubt that's what most people would want for a default install.
You can call it save() or whatever you like. I think replacing the MY_Model methods can end up being rather confusing.
is the .* necessary?  i've never seen the files be named anything other than pom.xml
Filed at #105
Треба написати тест на цей метод. Протестити декілька тест-кейсів:

* коли приходить один елемент
* коли приходить масив з одного елемента
* коли приходить пустий масив
* коли приходить масив з декількох елементів
* може ще якісь придумаєш

В цьому методі лоадер додає класи з фікстурами в масив, нам потрібно закинути фікстури, а потім витянути їх і перевірити чи всі закинулись.