labofsahil/github-event-dataset-2012 huggingface dataset:
Added gloss-free icons to iOS 4 and iTunes Connect

Hey Gina,

As you mentioned on your blog post, the App Store displays gloss despite your plist saying otherwise. I decided to fix it for you. Here's what was causing it:

Your plist had UIPrerenderedIcon defined inside a key that was introduced in iOS 5. Prior versions of iOS, as well as iTunes Connect, look for the old key, which is UIPrerenderedIcon at the root of the plist. My commit literally just adds that key to the plist. I'm sorry I didn't have a more awesome solution.

Best of luck with the app. :)
Solved some minor problems and lossless optimized all images

1) Made navigation links accessible over the "aside" section
(the navigation links are not accessible when they intersect with the "aside" section (http://imgur.com/kMBCB), because they slide underneath it (http://i.imgur.com/2LDP2.png))

2) Fixed weird bottom border for "MORE" button
(http://i.imgur.com/fNpB0.png <- this appears in some browsers, e.g.: Firefox, Opera for Ubuntu)

3) Lossless optimized all images
[Lighthouse-65] Fixing display issues with the flickr widget
Fix user creation part of RPM installation + README.redhat

Fixed a bug where user creation would fail if group plex allready existed.

Fixed README.redhat to include DLNA/UPNP ports in iptables documentation.
Fixed How To Become following site change
Notification email styling

Added icons to network status box in email. I have used images hosted on sharedearth server instead of inline images.
Cannot run generator because of wrong main class in jar

The fully qualified name of AssertionGeneratorLauncher class was wrong. There was missing `cli` package in this name.
come user-comments were changed
Add several vector operations to SPPoint

This change adds a handful of convenient vector operations to SPPoint:

- negate
- rotateBy
- dot
- angleBetween
- lengthSq
- isZero
Update Controller/HelperController.php

subject in missing when route get-form-field-element is called.

Needed to created custom query from configureFormFields.
regions plugin and stuff

fixed region userlist duplication bug, added regions reload command, and notify user if there are no regions when using the list command.

and also I formatted the document for BackupManager along with fixing a bug in the Decompress method.
Updated build orb
Jellybean

Attempt to pull elginsk8r branch to aaraya1516 branch
A build variant for Matlab files to publish a script to html.

Using shift-cmd-P, two new build options are available for Matlab:
1) Publish (create a html page using Matlab's publish feature)
2) Open published file (open in default browser).

Hope you find this useful!
Backend refactor

1. Refactor the default items backend into a class that can easily be reused for custom backends instead of having to rewrite the whole function.
2. Make the default backend take multiple section relationships into consideration, like #11
3. Respect the section hierarchy when grabbing items
SUnit。

よろしくお願いします。
quick fix:
Improved shownotes are improved

I updated the shownotes of Binärgewitter Talk #2 - Syncender Podwal and added a few links...
I thought you maybe want to merge my small changes into your version.
![](http://ponyfac.es/sites/pony/faces/675f8c7d.png)
Yep, good catch.  Thanks.
*Fleeing Movement Generator Z toleration...
Ok, I'll change this.
ah: ih you're playing around in this widget anyway: we want customizable threadlines at some point:
https://github.com/pazz/alot/issues/23
Thanks for the fixes. Definitely suggests to me we should be using jslint on a more regular basis!
Well, IMHO master should be next version. And already released versions should be in their own branches for backporting bugfixes etc (and they are now - release-2.0 and release-0.9.x).
http://developer.gnome.org/hig-book/3.0/windows-utility.html.en#preference-windows

this is an example to use "utility" in your link. I have tried my commit in
either lxde, xfce, openbox and i3wm, but I cannot see any bug.  the
document does not show any conflict either. I need a screenshot to show the
bug.

On Sun, Feb 5, 2012 at 4:08 AM, Tommy Tsim <
reply@reply.github.com
> wrote:

> @taylorchu
> Read this, http://developer.gnome.org/hig-book/3.0/windows-utility.html.en,
> if you want to understand the behaviour on GNOME3.
> However, the exception stated in the second paragraph seems not to be
> applicable in my experience.
>
> You are responsible to provide a solution ( i.e. indicating that
> "main_window" is an utility window which need not obey particular rules and
> sub-settings windows could be "raised above the application when the
> application window itself is selected from the window list" ) before your
> commit applied without setting(s).
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/caleb-/hime/commit/8beaa68a98a67a254ff9d4976272c0473eb18214#commitcomment-933874
>
Ich bin dafür, MooTools definitiv zu behalten, und von mir aus könnten wir jQuery auch komplett außen vor lassen ;). Also, es gibt verschiedene Meinungen, und ich fände es äußerst schade, wenn Contao MooTools fallen lassen würde, nur weil jQuery populärer ist. Ich möchte hier jetzt keine Diskussion darüber beginnen, welches Framework besser ist, da soll sich jeder selber seine Meinung zu bilden. Aber wenn jQuery als Option existiert, sollte MooTools keinesfalls vergessen werden.
Awesome.  Thanks!
It's just a matter of timing. I agree on by default is ideal, but we can't go back and fix already released versions of scipy and other projects. So how about off for 1.7, advertising this in the release notes, and then for 1.8 turn it on. 

Scipy is also not the only project that uses NoseTester.
OK, I fixed this by removing the `n` and merging the best parts of the rand and randn docstrings for this.
:+1: @holman.

but who knows how many apps have this issue and don't even realize it. :(
Already "reverted" by [this commit](http://github.com/spree/spree/commit/472cb51f0059f76d1ee198d45bf0624d62c8c3ac)
@ruli I think we should leverage `spl_autoload_register()` and I will come up with few examples of how it can be done

I think it can be implemented independently. 

Another thing to factor here is versioning of the api and loading the right version of the class automatically based on the request which makes it even more challenging and interesting :)
You're bein inconsistent about using `opts` and `params` to name your hash arguments. In general, `opts` was a popular name a while ago, but it turns out that they rarely represent optional parameters. But they *do* always represent `params` and that word is common in modern frameworks like Rails and Sinatra, so I'd recommend using it universally except when a context-specific name would be better. So please don't use `opts` to name hash arguments in Newman :)
In that case, wouldn't it make more sense to change the function signature to:

    html_to_unicode(html_body_str, content_type_header=None, ...
I had already made one a month ago, but thanks.

[github.com/svnpenn/bm/blob/gh-pages/yt.js](http://github.com/svnpenn/bm/blob/gh-pages/yt.js)
What is this commit fixing? No comments anywhere.
That behaviour seems confusing to me, and I'm not sure what problem it solves.  Do you have a data example?
It is not added, I considered adding it but wasn't sure and didn't want to add that to the API w/o talking to you.
ne, ich find die zu überdimensioniert. hab jetzt einfach per css das extern bild vorgesetzt.
parentheses aren't required here
I think porting an the old comp is the way to go ... the history is there etc. So I will open ticket to remove this function - weight and then Weight really are the same thing.
western plaguelands and alterac mountains quest texts are comming!
Confirmed to work with visibility increased though somehow the slipstreams stopped working now, any idea why?
Can you make this into a function?
foreach (file)
{
readfile(file);
}

I just read a book that offers several arguments about why you should do this if it is possible. I saw this same thing in a few other places too.
There is no source-file-name standard.
http://www.parashift.com/c++-faq-lite/coding-standards.html#faq-27.8

I think, for this point, it is better to propose a new option which overloads the defaut settings.
We have issues with delay since we started using R2, that was like end July 2011, it is exactly the same issue as described above. With 20+ hours uptime we got delay, with only +-100 players online, enabling grid unloading solved the issue for us (although that solution is discarded), with grid unloading enabled the server could easily reach 2 weeks+ uptime.
They are buried in the abyss of devhub https://github.com/mozilla/zamboni/blob/db84f31949f5e74f1c0de63b5842c294fb9dd83a/apps/devhub/tests/test_views.py#L2678
On further review... This code puts the new column on the CLAIMS table, while the upgrade.sql script puts the new column on the x12_partner_id.  I am pretty sure that the column should be on x12_partners and not claims, but zh, can you confirm?
(I tested after using the upgrade.sql script, didn't test with an empty database.sql)
That's work ok.. What you think to do?
Anywhere.. you any time get a good syntax for xhtml vaild... no?
Yep. In this case at least we always have a valid host by default. Not great but a sensible default. 

Alexis Lê-Quôc (@alq)

On Feb 14, 2012, at 3:07, FabriceOllivier<reply@reply.github.com> wrote:

> Ok. Just keep in my mind the host the agent is running on is probably not often the host the metric is recorded for.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/DataDog/dd-agent/commit/5612742f210906297ccc22806ba9574cfd3074dd#commitcomment-963583
Ok, it was worth a try :)
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/jmcantrell/Code/dotfiles-vim/vim/bundle/ultisnips/plugin/UltiSnips/__init__.py", line 12, in <module>
    from UltiSnips.compatibility import as_unicode, byte2col
ImportError: cannot import name byte2col
Added support for floats and refusal to support negative numbers.

The calling convention has to be a little bit different, unless you're willing to always load & initialise IRONCLAD:*PRNG*, in which case STRONG-RANDOM could have an identical convention. In order for prime generation and other things I'd like to add, Ironclad probably needs to have an internal PRNG anyway--do you have any strong feelings one way or the other?
Actually, I take it back: we have to keep modifyRef atomic, since it's used by `register`. Since the release map may be shared by multiple threads, we have a race condition without atomicity.
FFFFFFFFFUUUUUUUUUUUUUUUU
Ok, I'll throw in a Property to set NCrunch to false if its not defined. Thanks for the reminder.
compat.views, discovery.views
No problem.  Do you want brackets (android style):

if (*argv) {
    argv++;
}

....or not (linux style)?
remove "base62" until implemented
And I don't think that Component's have a click event by default.
I should have fixed this bug, so in theory you can remove this code.
Because this is what bugs look like :) thanks, fixed!
Dmitry?
I don't have the documentation handy, this /sets/ the font style not adds to it, right?
params.sort.map {|k,v| "#{k}=#{v}" }.join("&")
#include?
Remove these comments and add todo issues in the master repo.
All type information should be retrieved using the `Symbol`. They hold the correct information, while the tree contains what the user wrote (meaning there is no type inference results, etc). So I would do:

    val sym = defDef.symbol
    sym.name
    sym.info.paramTypes
the purpose of the 0.01 interval was to run at maximum speed but always yield execution so that other events can be handled
with the 40ms interval I guess constraint checking will be much slower (on large models)
what was the reason for you to change this?
I don't know if it's possible to implement those methods with intended behavior. Therefore all delegates to the uptimeMilis(). 
Reasonably they should delegate to elapsedRealtime() which is assumed to be time since boot. I'm not sure if it's possible to implement counting time in deep sleep, that's why all are the same.
As all those methods are the same, should I copy-paste tests for each?
go for it :-)
Maybe the name just confuses me. I don't know. I'm tired...
See also issue Issue #11975, which appears related. Answers such as .89% and 8.9% are regarded as valid when actual answer is 89%
While the standard order for an Exception's parameters are

    this(string msg, string file = __FILE__, size_t line = __LINE__, Throwable next = null)

if you have very many lines where you're chaining exceptions, I'd suggest adding another overload that's

    this(string msg, Throwable nex, string file = __FILE__, size_t line = __LINE__)

And actually, it looks like `Exception` has both, so exceptions in general probably should. The first is more important, since `enforceEx` uses it, and in my experience, having to pass a specific file and line (or at least line) to an exception's constructor is more common than having to chain exceptions (most frequently for unit testing), but the second definitely looks to be of benefit in this case.
curly bracket on the next line
Fix proposal for Jira GPDOJO-12
Done. https://github.com/editorconfig/editorconfig/commit/9690b96c863fe0b9e93b58f9513c9ae3f5411021
Well done.  :D
A previous pull request (for this bug) implemented it as follows (https://github.com/jeffora/MiniProfiler/commit/45604b2abdd89f7a139dcb25b74c6d0fd7db6220):

    profiler.HasSqlTimings |= externalProfiler.HasSqlTimings;
    profiler.HasDuplicateSqlTimings |= externalProfiler.HasDuplicateSqlTimings;

It seems like this could be more efficient (for a large timing hierarchy).
This fix is not working in case the date format is for example dd/mm/yy:
_postGenerate is called at the end of Centurion_Form_Model_Abstract constructor, but at this time $this->getDateFormat return NULL because the setDateFormat is called after the creation of the form.
why `raise` then `rescue` then re`raise`? is this just a design pattern for handling custom exceptions?
Is this not automagically encoded when the request gets made?
what fix this? :)
I suppose using stop in production (I assume you're using shutdown hooks?) is the polite thing to do.

The use case for async start/stop is the ability to wait for complete start/stop in a different thread than the one kicking it off. This might be a little speculative, but it "feels" right to me. It's possible to do other things while the server is starting/stopping so I guess a very minor speed gain is the use case.

Any thoughts on what the proposed change might break?
Glad to see you have brought back the lang files. It makes things much easier this way, don’t you agree?

Just one thing: I notice the Spanish file has got some French in it, e.g the “[FindImageMenu]” and “[InformationsMenu]” sections.

Also, one suggestion: would it make more sense to have the name of the language in the language itself: “Français”, “Español”, “Deutsch”? Easier for speakers of those languages to recognize.
Only two variables in the insert, so recommend using binding.
Where is the module?

Isn't there a better way to share common code? The `CommonCode.require('..')` statement looks a bit ugly :)
what about q http://old.wowhead.com/quest=995
handled now with quest_start_script and kinda not works so well.
will be moved to scripts?
Interesting impact from this commit  _(from what i can tell)_ ...

![](http://paulirish.com/i/41b2f0.png)
Coding standards aren't met here. And the else {} is unnecessary, as you can just return true.
It seems that your files have CRLF and my push converted those line break to LF, right? I'll revert and try to make a push with unmodified line breaks later today.
Actually, the "implicit configuration" (as from the <a href="http://tony19.github.com/logback-android/doc/1.0.0-4/ch/qos/logback/classic/android/BasicLogcatConfigurator.html">```BasicLogcatConfigurator```</a>) would be:

```xml
<configuration>
  <appender name="LOGCAT"
            class="ch.qos.logback.classic.android.LogcatAppender" >
       <tagEncoder>
           <pattern>%logger{0}</pattern>
       </tagEncoder>
       <encoder>
           <pattern>%msg</pattern>
       </encoder>
   </appender>
   <root level="DEBUG" >
      <appender-ref ref="LOGCAT" />
   </root>
</configuration>
```

except the ```tagEncoder``` result is limited to 22 characters plus a star to indicate truncation (```%logger{0}``` has no length limit in logback, but <a href="http://tony19.github.com/logback-android/doc/1.0.0-4/ch/qos/logback/classic/android/LogcatAppender.html">```LogcatAppender```</a> limits the length internally). For example, if the ```tagEncoder``` evaluated to ```foo.foo.foo.foo.foo.bar.Test```, the tag seen in Logcat would be: ```foo.foo.foo.foo.foo.ba*```. Note that ```%logger{23}``` yields more useful results (in this case: ```f.f.foo.foo.bar.Test```).
Sorry, https://github.com/TrinityCore/TrinityCore/issues/5450
It isn't caught because this conversion is very late on just as the code is being generated.  The optimizer that removes them won't be called..

The reason the conversion is very late on is that it is much easier to optimize sort(shuffle)->sort than sort(group(sort(group)))) (and other more significant optimizations) since grouping adds lots of complications.
Fixed grammar/typing error, refined info
with this commit, sindragosa air phase is bugged
there isn't any ice tomb spawned, 

can you check it please?
30 minutes to find that typo <3
only change property if it doesn't already have that value
Where is `STRICT_PROSCRIBED` defined? It's not mentioned anywhere else in this file.
Nice, has this option been around for a while? would it still work with older versions of the SDK?
Yeah it's a bit hacky, makes sense to at least have a function. I can revise this. I'm not sure why we don't have something like this in facter or a similar function in stdlib.
Oh, thanks, I just got this problem
These version numbers mean **NOTHING** :rage1:
This lives at github.com/django-debug-toolbar/django-debug-toolbar now.
I have to look through this in more detail, but one lock might indeed make sense.
the db that has many different things I want to know what the db used to use does not exceed or original
You've got an extra body tag here. It should probably be in the layout as well.
Oh... how much I'll enjoy that :) Thank you!
Known bug: Harvest Soul on normal mode isn't always properly cast, sometimes Lich King doesnt channel this spell and player isnt teleported into Frostmourne. I have no idea why this is happens, no errors or anything. Works when I'm casting the spell as player :/
Yes, computercraft always stores the computers in the folder "world", no matter what the world is actually called, as of now, so that default works
It was long-standing artifact with cbpp item merging which seems to be fixed now, though not entirely vanilla behaviour. Vanilla creepers will destroy items dropped earlier (usual scenario - get killed by a skeleton, a hissing creeper comes along and destroys your items). Only exception is for items dropped when you die in that explosion.

btw, this fix introduced another bug - there is no explosion sound/particles emited at all, creeper just disappears. This is purely cosmetic, block damage is done and items are kept.
I think this must be: Don't show program if NoDisplay attribute is true.
nothing was recorded when I use do tag_list.add or tag_list.remove
@wwood: see 03ad606dd643db4231f9d729dbdf20e6066dc74b for an alternative approach for dealing with sub-uri in JS code.
Comment for class .right
You wrote:
""I've just re-written your commit to tv-shows as i own more tv-show documentaries then movie docus but the behaviour should be the same""

Do you get the recently added populated with tv shows?
As mike said, this is not possible atm
I don't think we can remove the second match case.  We could put it last, to de-emphasize it.  As for handling `{rear=[a]}`, there would be a small savings in avoiding the size computation, splitting and reversal, as a could be returned with the empty dequeue.  It might be worth it for some applications, but it wouldn't hurt to check its cost.  If you're interested in doing this, have a look at the `benchsuite/` directory
Is the a good reason to use FTP for downloads these days? HTTP should generally produce less problems with proxies…
This issue (pattern matching) will be solved by

https://jira.springsource.org/browse/INT-2417
Thanks for the comment;   I have now updated the code to use two different keys!
I do think so, if I recall correctly, the old version was running on ep.io.
If u mean 4chon.net, then it rly has, but node are some different.
Function will be look like

function getImgName(post) {
	var name = $x('.//p[@class="fileinfo"]/span', post);
	return name ? name.textContent : '';
}
My database.yml connection name is not one of dev/test/production, which causes errors when fetching. Is there a method to find the connection_config without resorting to one of the standard environments?
Quarx2k:

Thanks, now it is ok.
As hinted by @cirwin on #735  you should go through `_pry_.input` here.
this logger.debug line gives us the info you want but in order to get down to DEBUG logging, you have to use `-vv`, not just `-v` (which sets INFO logging).  how about we change this line to `logger.info` so it only takes a single `-v`, and leave the default behavior the way it is?
I would merge that.   @pfmoore asked about this in the mailing list the other day as well.
It's definitely not limited to profile info (there are further microformats for that) - but it does make sense to limit it to external URLs, you're right.
Int it will be
ok didnt know that one yet cool
yes, I'm aware of that, and as far as I can tell that should be alright. 
The purpose of wrapped messages (e.g. Broadcast) is when sending to the routees. Messages to `self` is Resize, AutoReceiveMessages, Terminate or other *administrative* messages, i.e. not wrapped.
I don't think there was any deep thought into that, and you can also see that it was handled differently if it was one destination or many.
Also, assigning vars inside a transformation?
header line here?
No you are both wrong, it is `roles`.
This class should go: anything not otherwise known is queued. This way you can use the throttler transparently in front of any other actor; otherwise the source of the messages would need to be adapted.
Looks flaky, I'd prefer try actor.systemInvoke(msg) or try { actor systemInvoke msg }
The way 
```php
$image_original_name = Kohana::find_file('media', $this->image_src, FALSE);
```
not finds files in `DOCROOT . 'media'` dir
I'm getting a crasher here! After debugging I found out for some reason the inline asm locations are using the local variable address, instead of it's value, so it tries to write to the function frame and corrupts the stack. Here's a dump of the debugging disassembly:

54B90C44  mov         dword ptr [reg_ptr],edi  
54B90C47  mov         dword ptr [ebp-10h],esi  
54B90C4A  mov         dword ptr [ebp-0Ch],ebx  
54B90C4D  mov         dword ptr [stack_guard],edx  
54B90C50  mov         dword ptr [ebp-4],ecx  
54B90C53  mov         dword ptr [ebp],eax  
54B90C56  mov         dword ptr [ebp+4],ebp  
54B90C59  mov         dword ptr [start],esp  

Patch:

#ifdef _MSC_VER
	ARCH_STORE_REGS (cur_thread_regs);
#else
	ARCH_STORE_REGS (ptr);
#endif
Same code, probably needs removing from both.
I'll just blame my son again and fix it. :)
use headOption?
OK I implemented using ACTION_SELECT_ITEM instead of ACTION_MOVE_UP/DOWN so ppl can give it a try. For me it's hard to say which would be more intuitive as I've used the up/down implementation for a few months now and I'm very used to it. Using select certainly makes the general navigation through the list of filters more consistent as every up/down press brings you to another filter. But I don't know if I would have intuitively pressed Select/Enter to get to the other slider.
you need to sort the group after the import
See m_messageQueue.SetMaxTimeSize
пробелами конкантенацию и категория другая
echo Yii::t('yupe', 'Количество отображаемых эллементов:') . '<br />';
Да нет, я потом посмотрел, он private, он не будет доступен. Ты просто возьми за основу как они делали mkdir и добавь нужное, например chmod.
This test will only match the IPs that are the beginning of a network, it doesn't make sense.
Давай первую, это проще. И с неё на вторую перейти проще, чем обратно.
According to the documentation of [std::string::assign](http://www.cplusplus.com/reference/string/string/assign/) this method should replace the content of the string object. Therefore you don't need to assign it to `res_host` afterwards.

`res_host.assign(conf_val, 1, conf_val.length()-1);` should also work
indentation needed here too
Oh, I'll fix that in a bit too.
Bound functions are particularly horrific in V8 for some reason. http://jsperf.com/argument-slicing/2
The `.map(Number)` isn't necessary, strictly speaking, but makes it slightly more robust against weird inputs.  Perhaps it should just fallback to null if there's a NaN.
This could probably be an Akka actor instead, to save a thread; did you look into that or was there a reason not to do it?
I don't know if this check actually does much, but it is simplifying things too much. You could maybe argue that giving only one index is not a good idea for this function, but `Ellipsis`, `newaxis/None` (even if its not supported in current master with fancy indexing) and boolean arrays will distort this check.
That could also just be written

```
log = []
for i in range(100):
    alloc_log_enable()
    result = my_func()
    alloc_log_disable()
    log += alloc_log_fetch()
    other_computation(result)
```

Slightly less efficient to keep appending to a list like that, but memory overhead always creates time overhead, no-one will be enabling this and then expecting maximum speed while it's turned on.
Maybe "from the list below".
A code snippet showing how to use this with a form field would be very useful.
en0 - OSX ethernet
en1 - OSX wifi
пустую строчку ставить перед этой
Чем лучше?
as we only accepts two arguments I think that is better to use they explicitly here.
I mean:

```ruby
serialized = (options.delete(:serialized) { true }) ? :serialized : :unserialized
```
fix capitalization "Ver más"
Yes, but here I check that the something is what I expect... so it's a bit more specific. I prefer to leave it here, unless you or others insist.
I think the version number in the top-level `setup.py` of the revision you base your pull request on is a good choice. The value there is always the next version number.

I think you cannot know with certainty beforehand if it's too late to next release or not. Some dates are listed in the roadmap in Trac, but these may slip. So hopefully, the people doing the merge will remember to update these if the PR is merged after the revision is made.
And Ransack here too.
I love that you included a docs page with the first PR.  We should encourage this more in the future.
Yes, conditions are booleans, so it doesn't even make sense to talk about their commutativity. Also, multiplication always "distributes" over Piecewise (i.e. `Piecewise((x, A), (y, True)) * z` is always equal to `Piecewise((x*z, A), (y*z, True))`) so it's clear that conditions have no direct effect on commutativity.
> So if this function makes no guarantees, when should it be used? It seems to me like it would be more useful if it made at least some kind of guarantee, even if that means that some rings have to raise NotImpementedError.  But I don't know where it's used, so I may be wrong.
>

It is used by the QuotientRingDomain to "canonize" representations. The 
way this domain is implemented, the only effect this function has is on 
performance and readability of output. (That is, if we work e.g. modulo 
X**10, then canonize could drop terms larger than that. Everything will 
still work if it doesn't, except that the storage requirements could 
blow up (if we accumulate higher and higher order terms), and that 
printing something like X**105 + 37*X**40 instead of 0 is a bit silly.)
re nc-objects...see the docstrings in #1204
what problem are you fixing here?
typo Itegral
Is there a reason to keep these imports here and not make them module-wide?
These are small symbolic matrices, not large numerical ones, so the asymptotical complexity of a numerical algorithm is not a good measure. I doubt that there will be any speed benefit here. On the other hand it is harder to read it.
The generic glom rule turns out not to be sufficiently effective. It doesn't handle things like `Add(x, Mul(-1, x))`. My hope with `glom_MatAdd` is that it would lead me to a more effective general version of `glom`. I think I effectively need to make `counts` here a parameter of glom. This is something I'll need to think more about. `glom` is a potentially important rule.
there are no < or > in the attribute selector.
just came up with this situation and I am not sure if this is expected behavior (before my change it did not refresh)...
There's a tab character here, but we should just be using spaces for indentation.
ok, I've merged the `render` and `stream` method: d9e1736
This should probably be namespaced inside Netrc to avoid collisions.
Why not move this to the caller and get rid of the builder argument?
nit: DeleteCriticalSection() the critical section when it fails?
The kvm_* functions don't set errno.
doesSpaceExist?
Any reason the pending_diffs table isn't also analyzed after recreating indexes?
The documents created by the database profiler have the following fields. Each profiling report will have a different subset of these fields depending on the operation.
If you make this function non-static, everything else should follow.
Shouldn't censor_text be run before truncate? How do we handle this in other places? How does censor_text work on the last word if it gets cut off partially?
overlay = (typeof options.overlay !== 'undefined') ? options.overlay : true,
Possibly that is how the other tags are handled?
Keep "marked read"?
custom_dsn -> odbc_dsn?
removed in be0a529... leftover
No it doesn't `function propogateEvents(toClass, fromClass, eventNames)`
if action != 'login': return. No need for one more indent level.
Done in 4dc0c02
Hmm, I'm not a fan of dropping the log level on the client like this, need that visibility. Dropping the log level on the server does make sense w/ a high volume of metric checks. Having several sensu-server's makes this less of an issue for us.
Can we make the message include the element that first fails (or at least its index) and check that here ?
Ah I see.

This project uses tabs rather than 4 spaces, please correct that and push to this branch. Cheers.
I’m ok with either regex. They’re both not as explicit as they could be, but they’ll do the job
садистик сказал что они должны переходить в удаленный статус
The leaky only wanted hand tools, so it only deals with hand tools
On May 7, 2012 3:22 PM, "James Clarke" <
reply@reply.github.com>
wrote:

> > @@ -0,0 +1,36 @@
> > +package org.bukkit.event.player;
> > +
> > +import org.bukkit.entity.Player;
> > +import org.bukkit.event.HandlerList;
> > +import org.bukkit.inventory.ItemStack;
> > +
> > +/**
> > + * Fired when a player's item breaks upon breaking a block
>
> Should it not include interacting e.g. Flint and Steel?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Bukkit/Bukkit/pull/599/files#r784468
>
good idea to borrow/pay tribute to Wes' work!
I would recommend doing this through a Raxa repository. So once you have pushed the changes to your own fork, can you request ownership of the repo to be given to Raxa??
Need some whitespace above this to separate builtins from 3rd party modules.
Inconsistent use of empty lines in this function. I think they help, but please be consistent, e.g. separate out dim coords, aux coords and aux factories.
操作符前后应该有空格。
class op -> Op class

I would say, "to verify that no Ops of that type appear in the shape-optimized graph."

Also couldn't this be accomplished without passing it as a parameter and just inspecting `self.__class__`?
maybe just change description a bit
Remove that function instead and fix it there:
<pre>
ack-grep build_url
cmdb/views_changes.py
24:from ralph.util.views import build_url
140:            'jira_url': build_url(settings.ISSUETRACKERS['default']['URL'], 'browse'),
192:            'jira_url': build_url(settings.ISSUETRACKERS['default']['URL'], 'browse'),
212:            'jira_url': build_url(settings.ISSUETRACKERS['default']['URL'], 'browse'),

cmdb/views.py
37:from ralph.util.views import build_url
383:            'jira_url': build_url(settings.ISSUETRACKERS['default']['URL'], 'browse'),
</pre>
Yeah since I added the other test, we can probably just remove the comments.
Not very clear as to the use of this function.

Sounds like it returns an array of strings, each of these strings being a concatenated list of the properties that were requested, separated by a specified string
Yeah, I'm still in the pattern of using dicts, and that function would have to build them up as dicts first anyway before converting to namedtuples (as it relies on the mutability during the process), so I'd rather keep them as dicts without a serious reason to change, and in the interests of not rewriting a good chunk of the code :)
as before ...
Could you add tests for all the other views?
Like that : 
```php
        $guzzleRequest = $this->getClient()->createRequest(
            $request->getMethod(),
            $request->getUri(),
            array_merge($this->headers, $headers),
            in_array($request->getMethod(), array('GET','HEAD')) ?
                null :
                null !== $request->getContent() ?
                    $request->getContent() :
                    $request->getParameters()
        );
```

or

```php

        $body = null;
        if (!in_array($request->getMethod(), array('GET','HEAD')))
            $body = (null !== $request->getContent()) ? $request->getContent() : $request->getParameters();

        $guzzleRequest = $this->getClient()->createRequest(
            $request->getMethod(),
            $request->getUri(),
            array_merge($this->headers, $headers),
            $body
        );
```
This may be part of team 3's test suite, but since we don't have a test to cover this. Thank you.
What about a relatively positioned element at 0,0?
> I'm sincerely lost here: what are the actual arguments for putting this in the default ajax module?

It uses ajax functionality. It's essentially the synchronous form of $.getScript. I think the bullet points you make above are consistent with this being a synchronous method, there's no need for global events or a callback if the caller gets control afterwards. Sorry to post a short reply to this but I'm not sure what other arguments to give. Again it would be great to get some perspectives from other team members.
I know it's not an issue, really... but drop the trailing comma
Because it is proper Markdown syntax. Not all Markdown implementations will convert raw links into anchors without the brackets.
I'd love to make the decision automatically inside this function.
I agree. But I did not want to make this decision on my own. This also relates to how we want to structure the source code in files once we have many tasks and many algo -- are we still keeping the same convention as in the regress module, where we kept all algos for the same task in a single cpp & hpp file.
The placeholder text needs to be gettext'd.
Need a versioned URL, otherwise the formula will break every time this archive is updated.
Cette orientation doit disparaitre, on a une belle duplication sinon
Did that so I could test that the timeout option is set for the Faraday instance
`0/1/2 star(s) brighter` will work.
nice. You can put this all on one line per Square's improved style guide if you like.

  if (binding instanceof SingletonBinding) throw new AssertionError();

You don't need messages for assertion errors. They never happen! And the line number in the stacktrace is enough.
nit: be consistent about generics and casting? (always use `Class<?>`)
You're right, I made the change but it seems that I made a mess of the commits since this is the first time I've used the pull requests on github.
A switch statement probably would have been better here. :/
this commit actually has nothing to do with fixing the device build, just squeezed a lil cleanup in to this pull request
Gotcha.  It seems like your earlier idea of having the _return_ value of handleSelect() trigger the re-popup would avoid this problem.  But I'm ok with the current solution too.
Use .toggleClass("resizing", true) instead.
Agree with cursor, but not tag info as it is not needed for other types of code hints like js hints or css hints.
yes please :D
Comma at the end of the line.
"are registered"
`$.each(layers, function(i,o) {`
Hmm... I did not code that (but that is not an excuse) so hopefully this is already tested well. Cannot do tests myself as I do not have FB.
Why removing this line? Leave it.
Can this fail?  What do you do if it does?
You can simplify these as well.  Just do `splitTime[ splitTime.length - 2 ] | 0` and minutes will be a number, and then you can drop the check for `if ( minutes )` and simply do the multiplication, since 0 * 60 will be 0.  Same deal for hours.
I agree, if there was no error, we can assume success. Nothing else needs to be passed.
Remove code and make comment more readable, e.g. calling error() would generate UnknownError
`pageNumer` will look better here (instead of `params.page`)
A
```python
if count == retries:
```
is needed here, otherwise on the last retry the program will say __Retrying...__ and then exit.
This commit is part of pull https://github.com/sonatype/nexus/pull/672 that was prepped yesterday for V5.
Today morning I made two new pull requests on https://github.com/sonatype/restlet1x and preparing V6.

So yes, with V6, this pull will need update. See
https://github.com/sonatype/nexus/pull/672#issuecomment-10652981
Explain when and why you would use this class
Rewrite as a single list comprehension `cmds = [...]`.
hanooz ham be in niyaze ?
Move this code to a method say run() or something sothat you can test run in your testcase
It seems that part of the line was unintentionally lost during some other changes, so I was reverting it what it was before.
BEAST MODE.
the connection should be lazy on first write IMO. Otherwise, you will open useless connection when nothing is logged (when the handler is wrapped in a FingersCrossedHandler or when you set the level to CRITICAL for instance)
Removed.

On Fri, Sep 21, 2012 at 5:27 PM, Paul A Martel <notifications@github.com>wrote:

> In src/ee/executors/unionexecutor.cpp:
>
> > +        boost::unordered_set<TableTuple, TableTupleHasher, TableTupleEqualityChecker> m_tuples;
> > +    };
> > +
> > +    boost::shared_ptr<SetOperator> SetOperator::getSetOperator(
> UnionPlanNode* node) {
> > +        UnionType unionType = node->getUnionType();
> > +        if (unionType == UNION_TYPE_UNION_ALL) {
> > +            return boost::shared_ptr<SetOperator>(
> > +                new UnionAllSetOperator(node->getInputTables(), node->getOutputTable()));
> > +        } else if (unionType == UNION_TYPE_UNION) {
> > +            return boost::shared_ptr<SetOperator>(
> > +                new UnionSetOperator(node->getInputTables(), node->getOutputTable()));
> > +        } else {
> > +            VOLT_ERROR("Unsupported tuple set operation '%d'.", unionType);
> > +            return boost::shared_ptr<SetOperator>();
> > +        }
> > +        return boost::shared_ptr<SetOperator>();
>
> These null returns are redundant. This one is unreachable as it stands.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/alexeevm/voltdb/pull/3/files#r1665945>.
>
>
Most of the documentation below this is sort of "narrative" form, and as such may belong better in the .rst documentation instead of the docstrings. What do you think about moving this there (except perhaps a pargraph or two of abbreviated examples)?
Good point - I've fixed this!
Perhaps this should be "Subtract datasets from each other"? Also, I wonder if there's any other method name we can use here that would make it less ambiguous whether it's "self - other" or "other - self"?  E.g. ``data1.minus(data2)``?  I'm not sure I like that method name, but it's less ambiguous which one is being negated...
Reverted to buffer and will keep using this ^^.
ApplicationHelper can easily become a catch all for lots of unrelated methods. This method seems to relate to notifications. Perhaps a NotificationHelper would be good.
Happy to re-architect that way - I was wanting to re-use your code from the Qt and Wx implementations more-or-less as-is, but I will not argue that it results in a convoluted code-path.

I'll re-write client_validators appropriately.
I believe a normal function here is faster than the generator (as we only need one .get())
It does hash the file which could be replicated by adding a similar function to: http://www.tornadoweb.org/documentation/_modules/tornado/web.html#StaticFileHandler.make_static_url 
to the jinja2 parser environment, but it wasn't clear to me if this was necessary.
What will this do with files containing non-ascii characters? I suspect we may need to use `io.open` and specify utf-8, but it's worth checking what it does.
OK, we should document the fact that the users needs to call the show method.  Not sure where to put that documentation.  I agree that this is a good idea though.
I assume the constructor on that is Ok to be protected (otherwise the builder pattern then seems somewhat redundant) - see respun commit.
Revert this file :)
Wat. We have a `git_filebuf` object which was designed for this. Pass it the `TEMP` flag and it'll take care of finding a place on the filesystem to create the file. You don't have to manage file creation at all, and cleaning up the filebuffer will remove the temporary file.
The Great Leak, Leakamount Pictures (R)
Could you condense the bracers in the previous line to follow our indenting style?
We can store the layer in a QVariant inside each item see http://qt-project.org/doc/qt-4.8/qcombobox.html#addItem
it looks like this script wouldn't be able to set the output path without first specifying b2g_root.. maybe these could be options? See scripts/bootstrap-mac.sh around line 38 for an example:

https://github.com/mozilla-b2g/B2G/blob/master/scripts/bootstrap-mac.sh#L38
have deleted the empty option - as we already have 'selected' on the United States one
Yeah, I wrote this test before fixing (adding the PasswordDisabledHasher fake thing). It dies in a 500 without it. It shouldn't 302 either, because that means the user was logged in.
Hmm, they don't. Is wrapping in a parens and breaking at the `.` preferred?
since you passed it through the view context, this is actually in the context, you don't need to pass this `third_party_libs` around. you should be able to access it fine without passing it as an argument.
This can just be

```javascript
$(".inbox_items_container input[type='checkbox']").live("change", handleGlobalButtons);
```
won't you get all classes on the first iteration with $classes ?
Since its the array key that is returned, the return type can only be `string|integer`.
Yes sure, I didn't mean to rush you :)
It looks like a big hack.
That is all clear. You are not answering my question.
@Tobion I have updated the PHPDoc but it seems there is other PHPDoc which have the same issue.
No need to make a new pull request. If you push any changes to your master repo, they appear in this pull request.
You changed the indentation here. :(
SublimeText fails to add newlines to the end of files (Vim does this automatically).

No functional impact but it is a style issue.

We had a hilarious 37-email exchange on the topic which I won't forward to you out of kindness.
I kinda like the idea of the generic table, but the value couldn't be typed in that case (it would need to be a table with two "string" columns, for we may need to store date, id, text in the value), so it would defeat a bit the consistency.
Maybe a table RudderIntProperties for storing int values ? Or maybe only RudderProperties, and we bet that we'll store only int ?
What do you think of 

logger.debug("""Deleting report with SQL 2 queries: [[
                             | delete from reportsTable  where executionTimeStamp < '%s'
                             |]] 
                             | where reportsTable are RudderSysEvents and ArchivedRudderSysEvents
                             |]]""".stripMargin.format(date.toString("yyyy-MM-dd HH:mm"))
Actually I think you can get git to warn about "whitespace errors".  Check out the core.whitespace config setting for example (git help config).
Let's go with #119.
nit, maybe:

variant.[flagName] ?  so:  variant.enable_debug_mode?
Are there any other? Would be great to clarify this further.
that's the only one for Bitcask, but I wanted to leave it open, in case
there was something that I didn't know about for leveldb, or a future
backend.  right now, I don't think that there is.

On Tue, Oct 23, 2012 at 11:26 AM, Sebastian Cohnen <notifications@github.com
> wrote:

> In source/riak/tutorials/choosing-a-backend/Multi.md:
>
> > @@ -86,6 +86,11 @@ When using Multi Backend, it is important that the sum of all backend memory
> >  use is at 50% or less. Three backends each set to use 50% of available memory
> >  would cause problems.</div>
> >
> > +<div class="note"><div class="title">Multi-Backend settings</div>
> > +Certain settings, such as Bitcask's `merge_window`, are set per-node,
>
> Are there any other? Would be great to clarify this further.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/basho/basho_docs/pull/44/files#r1919620>.
>
>
sorry misread the patch. Thanks for the feedback
js-ки лучше просто удалять
Done :)
I also changed the zoom level so that one selects it instead of guessing what should be a good input.
I'm not so familiar with the gallery code base so I'm probably being a bit too cautious here, since this widget can be included on many pages through templates, etc. I just wanted to be sure it wouldn't break the page if someone had attached a property called item in some custom form that happened to include the search bar.
@mecathcart do you have any idea where this line came from ? i think its a typo..
You really need custom describe handler? Generic one does not cut it?
By the way, while we are at this, the revertUpdateToRuntime() method of this class needs to be looked at too.
the name of the test seems wrong, since it's no longer actually testing a key violation, rather a query on a nonexistent table
`null` might be a better default, as the name itself is rather ambigous
`for book, chapter, verse, text in module.all_verses_in_chapter(book, chapter):`
спрингом не управляется, но можно волшебными пассами через соответствующий контекст (
Why the duplication in id + class?
It seems that 'hidden' attribute can not be applied for div with 'display: table'.
semicolons are not really needed
Wouldn't it be better to have this variable definition with the other buttons ?
Yes, I was thinking about it. However, in `system` there is no `screen.css`, that's why I've added two 'classes' to both files instead of creating a new one file... Do you think is better to create a new file called `/screen/screen.css` and add this styles there? Thanks!
Why do you need all this trickery? Can't you simply focus the iframe when it is showned?
Remove this useless check
Eeek, I don't even know how that semicolon got there. I'll fix it up and push an amended commit.
I'm all for adding a second method to handle the `IO`, but since this method returns a URI to be used elsewhere to perform the download, we need to return the S3 URL. Returning the GitHub API url as the test indicates is not good enough because that can't be passed to another system and used without authenticating to GitHub for private repositories. It's a worthless method.
Sure, let's rename it to try to keep any Azure references from public Kudu, even though it does have a dependency on azure-cli for generation.
See previous comment about using a hash here to clean this up.
See if it's easy to move this into the ``#attached``
I'm sorry but I don't get it.