labofsahil/github-event-dataset-2012 huggingface dataset:
Overwrites default comparison operator to delegate to position's value

First thing that i've noticed when downloaded the Gem was that the default ".sort" method on collection was not returning list values order based on position_column defined.

IMHO, this should be the default behavior of any collection that implements act_as_list's behaviour.

Regards!
Initial take on the admin pages.

I have changed all the tabs for images as per the old UI.
I have added some table colours to make things a bit easier to see. I would like to make some of these corners rounded but I have not found out how yet.
add authorize endpoint
Add id attribute for fieldset (group), useful for group toggling.

Useful for toggling particular groups, for example:

$form->addRadioList('radio', 'Select', array(
1 => 'Group 1',
2 => 'Group 2'))
->addCondition(Form::EQUAL, 1)
->toggle('group1')
->addCondition(Form::EQUAL, 2)
->toggle('group2');

$form->addGroup('Group 1')->setOption('id', 'group1');
$form->addGroup('Group 2')->setOption('id', 'group2');

etc.
WebUI: Fix boolean enums
blackberry.app (partial) for BB10

Tagged fields supported in BB10, blackberry.app.exit() not tagged yet as it's not in feature branch yet
User Story: blackberry-webworks/BB10-Webworks-API#1
normpath no longer lives where it used to

This stops it working on Python 2.7
Several changes in your latest updates
[JENKINS-15197] Add support for Maven toolchains config files
Fix user can seemingly apply discount. Issue #141 

You can seemingly apply the discount by changing
the email to one that hasn't been used then changing it back to the correct
one. You won't be able to make the purchase but you can make the payment
amount change.

JavaScript fix to check when user changes email and revert discount.
Receipt Shortcode

See #495

Usage:

`[edd_receipt]`

Certain things can be turned off:

`[edd_receipt price="0" discount="0" products="0" date="0" payment_key="0" payment_method="0" payment_id="0" key="123123123"]`

If a purchase key isn't explicitly set, there are two options:

* Load from the current session (most recent payment)
* Load for the `?purchase_key` in the URL

### Todo

* Create a helper function to generate a "Receipt" link (`add_query_arg()` to success page)
* Something else I can't think of
Merges from edgarjs in order to make it work on Rails 3.1+

I have merged the last merges from edgarjs
It's now compatible with Rails 3.1+

When there was conflicts, I've used your version every time (was better in my opinion)
Don't format boolean facets as raw queries
increase bamcheck read length to work with long miseq reads
[TIMOB-6513] Added deployType and sessionId
Yes its not near its here

The Beastie Boy Demo :)
Fix issue with replication lag check and whitespace issues

Was trying to set up this check today and ran into a few issues.

First, there were some whitespace issues that was causing Python to choke.

Second, and not in here, was getting a `'module' object has no attribute 'version'` error for the `pymongo.version` checks.  I don't really use Python, so I just took out the version checks and it worked.  The pymongo docs say it should have a `pymongo.version`, but not sure why it didn't work.

And finally, the replication lag check was always returning a warning.  Found it was because the if statement was comparing a number against a string, while the check for the critical level was doing it right.
Destacar los eventos, la lista de correo, la cuenta de Twitter y agregar detalle a los enlaces externos en el homepage; Simplificar la página de eventos.
Use Travis-CI here for testing
Adds NuGet files
Update master

New Korean Translation arrays.xml
HV-481

Trying to wrap this up. After discussion via email I renamed the log method which return an exception to _getXYZException()_. 

For now I moved the log messages for _ValueFormatterMessageInterpolator_ back into the main _Log_ interface. Using the inner class approach you cannot make the class private and the generated class will be public either way. It is a really a tradeoff decision. I think for simplicity reasons we just go with what we have and maybe revisit it later.
Undo use of 1.9 Hash syntax

This seemed to be the only location that was using the 1.9 Hash syntax, so I undid that, and everything works fine on 1.8.
Adjusted IN/OUT CDArt animations

I've modified the start point of the IN animation and the end point of the OUT animation, because with the default settings I can see the cdart behind the cover.
Update github url in generator template to use https

Not a big change :). Thanks.
fix: ocamldoc of 3.12.1 can't parse "@raise [Exception]"

...but needs "@raise Exception".
  While I was at it I added a few @raise here and there where they
  were missing.
sync() should return a promise

Fixes #23

Note that this adds an implicit dependency on jQuery.Deferred.

Existing tests pass, and this works in my own project, but I've not created new tests that exercise these changes.
First spec

Watch it fail (red), write the implementation (green), and then refactor (if you see any opportunity to do so). Finally, write the next test (eg: for a single digit, "1") and let me know when that's done. That's a full red/green/refactor ping-pong.

Remember: Don't try to be too clever on your first try. Just write the simplest possible thing that could work.
LPS-21606

LPS-21606 Fix regression caused by LPS-23845
LPS-24562 - fixing security hole that allows webdav clients to access an...

...y files on server
LPS-24607
Need no locale in addition to nb and nn

It seems Norwegian language support isn't ideal in Android. There are two written forms of Norwegian, which each have their own ISO code: Norwegian bokmål (nb) and Norwegian Nynorsk (nn). Unfortunately "Norwegian" without any for specification also has an ISO code: no.

By law they are equal in Norway, but in practice, bokmål is far more prevalent and in use (80-90 % of written text is probably bokmål).

It seems both no and nb are used in Android: http://stackoverflow.com/questions/5153674/android-localization-question

The only practical way to solve this seems to be to have two copies of Norwegian bokmål, one under "nb" and one under "no". Symbolic links are not supported on FAT filesystems, and thus the app cant be moved to memory cards, string aliases doesn't seem to be practical as one would still need to make the string files containing aliases(?), and I don't know of any way to "prioritize" languages (if no "no", then use "nb").

I've added a readme file to all folders containing nb and no noting that they should be kept in sync.

The "nb" Android market text should probably also be copied to "no".
Updated schedule.html to remove Coda Hale's talk. :`(
various changes

- add security hooks
- move selfTest back to selftest
- rename main class to reflect jetty8
Added modman configuration file

Hi Stefan,

please consider adding this modman configuration file :)

Bye,

Fabrizio
drop null pointers in module info section
changes made during collective blackout
Fix compass_twitter_bootstrap_responsive.sass imports.

Corrects the corresponding sass file of compass_twitter_bootstrap_responsive.scss to account for the import fixes documented here: https://github.com/vwall/compass-twitter-bootstrap/commit/3e2434603edcbb7559c92c83565bac6ca2500890#stylesheets/_compass_twitter_bootstrap_responsive.scss
YouSee removed

A subscription or YouSee connection is required.
Moved to the YouSee Channel
Added BinaryTree and his tests
toro: fix vzw apk certs

this fixes the issues with some of the VZW apps (ie My Verizon reporting no data connection while on 3G)
Field Type term_field

CLEANED UP. SHOULD BE GOOD CODE NOW. IF NOT, LET ME KNOW
Translation of register.php

Hi,
I worked on the translations of the register.php.

Enjoy ;)
CoffeeScript brush

For your consideration; my CoffeeScript brush, including .coffee original.
Cheers!
move a recently created repo (not used yet) to a more generic one
Various improvements to the pagination helper

I've made several small changes to the pagination helper, and if you find any of them useful you're very welcome to merge them into the original. 

The commits are fairly atomic, so it should be possible to cherrypick only relevant changes.

I realize that not everyone will be using pagination in connection with a search string/parameter, but I've made it optional, so it'll silently "disappear" if not in use. And it should be easily adaptable to non-search situations where some other querystring needs to be passed from page to page.
Update WebInterface/install.php

Add 3 fields in table "keys": last_used / iPad3 / client_apple_account_id
Correction for avoiding the "siriproxy gentables" required before starting TTLP
[LOGTOOL-39] readResolve() should return java.lang.Object

Use java.lang.Object return type for readResolve() methods on message bundles.
Updated BBM functional tests to work with new BBMSP API behavior
File Downloader API

Hey guys,

Here is my next contribution. Let me know if all is good.
DAO for querying storage usage.

1) Note that this is a release patch.  So first of all it includes modifications on DELETE to avoid deadlocks.  This change was meant to be an minor improvement.  But I noticed today that the data migrator has been running much slower due to deadlocks on insertions into the new storage_location table.  There were already deadlocks on the string annotations table.  Now it is getting much worse with more deadlocks on the storage location table.  The data migrator has been running for 9 hours only 20% done. This prompts me to peel off the changes associated with StorageLocationDAOImpl into a release patch.

2) Storage DTOs get more consistent names.  A new DTO StorageLocationDimensionValue to capture the name-value pair as suggested by your last review.  Remove one storage usage DTO that is not going to be used.

3) Implement StorageLocationDAO for 3 queries of storage usage.

The remaining part of this feature implementation (i.e. controller/service/manager) will be in a separate change list on the develop branch.
show plugin dependencies on wiki

This change adds the plugins displays the plugins dependencies in the info box, this is specially interesting for users not able to download the plugins via UpdateCenter but manually.
Merge/ch
Facebook meta

Hello!

I added one simple method to enable the use of Facebook meta tags in HTML header.

I would be happy if you consider pulling it into your project.

Thanks a lot for your great work!

I use your plugin on the website www.popnews.com (one of the biggest French music webzine).

All the best,

Guillaume
Missing buildout options

Added the missing 'develop = .' to tell buildout to pick up the current package instead of searching PyPI.
prepare-release
Add examples and rename existing file to reflect changes to the sta.sh API

This adds a files showing examples of how to move a sta.sh submission into a different folder and how to rename a sta.sh folder, as well as renaming the deviantart.php file to more accurately reflect the functionality it utilitizes.
Add a Apache-2.0 form that is common on Java software
Correction of problem with hover over Browse button

Hello. 

I use jQuery 1.6.2.
I found problem with Browse button. 

When you have Browse button below currently visible part of page and you have to scroll down, then Browse button will stop reacting on hover events. The problem is in computation of 'top' coordinate.

rochejul's solution - from issue #2 was to replace (https://github.com/filamentgroup/jQuery-Custom-File-Input/issues/2#issuecomment-4301656):
'top': e.pageY - upload.offset().top - $(window).scrollTop() - 3
by:
'top': e.pageY - upload.offset().top - 20

This is correct and it works for me. Tested with Google Chrome 17, Firefox 9, Internet Explorer 9, Safari 5.1
Update Music/Files/res/values-ru/strings.xml

поправил собственные неточности
can specify a superclass to enable sharing of code between versions

Here are teh codez.
Fix: Prevent "Uncaught RangeError: Maximum call stack size exceeded"

I made this pull request including code from this: http://trentrichardson.com/2011/07/29/many-updates-to-timepicker/#comment-32649
Update docs for issue 8216 "CTFE unrelated pointer comparisons"

Also tidied up the CTFE pointer specs a bit.
Will use Apples default if regenerate_interval is not set in config, instead of crashing :)
Fix NotFoundException when using auto increment id strategy

The generated Doctrine ODM EditController can not handling numeric key.
If the document id assumes numeric. Convert it to integer.
This and the following commit - nothing was really changed, right ?
> This all seems to be working ok for me. The only thing I have a question about is why you dropped the config/initializers/send_file_options.rb file. Was it causing problems for you?


I excluded it because in the instructions it says to change it to 'apache'  so either it should be defaulted to 'apache' and that install step amended or we can just exclude it from git to match instructions.


On Jan 29, 2012, at 6:29 PM, Walter McGinnis wrote:

> This all seems to be working ok for me. The only thing I have a question about is why you dropped the config/initializers/send_file_options.rb file. Was it causing problems for you?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/noel/kete/commit/2c0a535b9847ae58dea82ba86c70e5e86fb29ac0#commitcomment-912968
For Articles and Issues, there's a specific check to ensure that $request->getJournal()->getId() == $pubObject->getJournalId().
For Galleys and Supplementary Files, it's assumed that $request->getJournal()->getId() will be appropriate.

If the check is deemed necessary for any of the entities, it should be coded for all of them. We've never had performance problems in this area before; if that's old code, it's probably a mistake.
Should use fieldLabel for label tags (maybe already done)
难道我是在统计行数吗？
On 05.01.2012 22:16, Harald Szekely wrote:
> I was searching for that commit, but the stupid search function of Git didn't return that ... :-(
>
> What exactly is broken? The transitions work here. What is the difference in his work? (i'm at home, can't diff that now)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/WoltLab/WCF/commit/4c6d3da093db961627a3219ddbabe682e06acf16#commitcomment-841736
Max added transitions for the right property:

```
.right .sidebar .collapsibleSidebarButton {
	background-image: url('../../icon/arrowRight1.svg'), url('../../icon/arrowRight1.svg');
	right: 250px;
	-webkit-transition: right .1s ease;
	-moz-transition: right .1s ease;
	-ms-transition: right .1s ease;
	-o-transition: right .1s ease;
	transition: right .1s ease;
}
```


You did not. I assume the sidebar will not fold correctly. I cannot test 
right now either.
Expected, nope.  That would imply that printf is not 0-padding but space-padding.  Don't tell me it's platform dependent somehow.  But I don't know where/how the manifest in question is being generated.

Try this on your platform:

printf "%016s" bippy

If that doesn't print 11 leading zeros then I guess it's back to the drawing board.
build failed due to this commit
Very nice.

Did you reported the issue upstream CMake ? I guess not easy to reproduce
...

If problem were occurring on windows7, some work has been done regarding
CMake/ExternalProject:

http://wiki.slicer.org/slicerWiki/index.php/Documentation/4.0/Developers/Build_Instructions/Prerequisites#Windows
  http://cmake.org/Bug/view.php?id=12957

https://qshare.queensu.ca/Users01/lasso/public/cmake-2.8.7.12957-win32-x86.exe

May be it could helpful in some way.

That said, in addition to URL, may be the MD5 param could be added. To
check the integrity of the downloaded archive.

Jc


On Thu, Mar 1, 2012 at 2:03 AM, Sascha Zelzer <
reply@reply.github.com
> wrote:

> Yeah :) We had lots of problems with our Windows cdash clients producing
> spurious "unable to unlink" errors when doing git clone. We now use
> tarballs for everything and don't do git clone anymore. First time in
> *months* that we have an almost green dashboard...
>
> On 03/01/2012 07:43 AM, Jean-Christophe Fillion-Robin wrote:
> > Very nice :) This is great that we can also pass down URL param.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/commontk/CTK/commit/de466419165d9ffc0cf2e34f514bded76ad1f29d#commitcomment-1029509
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/commontk/CTK/commit/de466419165d9ffc0cf2e34f514bded76ad1f29d#commitcomment-1029549
>



-- 
+1 919 869 8849
Good spot! That's probably it.
Laurent you normally just update the version number in package.json.
There's no need to redownload the tarball with each release.

2012/2/6 Laurent Kempé
<reply@reply.github.com>:
> Thanks @tombell for the link! Exactly the issue I have
>
> —
> Reply to this email directly or view it on GitHub.
> A tip: you can just write "fixes #1" or "closes #1" and GitHub'll auto-link the issue as well as close it for you.

Thanks!  Am still learning github (obviously :-) )

craig
@Monty You're welcome!
thanx! =)
We still have a namespace attribute that has to be unique (we use it in the
URLs).

-- pierre



On Tue, Jan 3, 2012 at 12:23 PM, Benjamin LAN SUN LUK <
reply@reply.github.com
> wrote:

> Then we should remove validations on namespace ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/voxe/voxe-web/commit/8eaaf2031521dc9ce42ded3e442221117e2f194f#commitcomment-834163
>
Hmm… I think `c` would have to be initialized as a `d3.map()`, too, if you want to be able to interpolate the __proto__ property? But I guess that's a pretty bogus use case, because I can't think of any situation where it would actually make sense to do so.
Can you please confirm this won't cause trouble? :)
It's not a fixed size *queue*, right, it's just that the number of concurrently running tasks is fixed?
This is probably a python bug, I wonder if it has been reported
Can you provide an example of when it's desired to add a delimiter in such a situation? Some examples of when it isn't is inserting the first element into an empty argument list `fn()`, array literal `[]` or object literal `{}`.
I find it often reads better and results in more concise code.  Fewer var declarations, for starters.  Yes, the debugging is a pain, but that's a problem for many common JS constructs.   Also, it forces me to consider other iteration methods.  This might actually work better as a map.  You'd never have to declare 'views' at all.
if you want `<details root URL>/statistics/` then you can define `'^statistics/$'` in [detail_urls](https://github.com/mozilla/zamboni/blob/31fda3f/apps/bandwagon/urls.py#L16)
I don't know ^^
its still crashing :\ at same assert
sry...werd langsam alt:)
best way would be to check other "similar" code to see how it's written, and use the same...

for example this should be all in one line "} else {"
:) :) :) over a year bug persists and no one point me...
Подскажите в чем проблема. Обновился до последний ревизии перераспаковал maps, vmaps, mmaps. заходим в яму сарона сервер упал, последние строчки в логах:
Creature (Entry: 31260 Guid: 7779265) enters grid[33,31]
Object 7779265 (Type: 3) is visible now for player 984. Distance = 362.817932
Segmentation fault

Удаляем нпц, краш пропадает. Есть и другие такие локации.
не надо грязи, год назад этих свистелок-перделок в ядре не было :)
this `<p>` isn't closed
this is crazy compact and hard to read.  please just use a regular if statement
I think this should be self.content_disposition, not self.cache_control.
ughh, you are right. I hate that the web interface doesn't give a simple overview of the files changed (or at least I haven't found it). OK, will do this later tonight
I put the wrong file into my previous commit, here's the change we need!
Sorry Horn :P
Hmmm... I'd somehow missed that slimmer was doing this filtering. It shouldn't be. I added an easy way to filter headers in the router last week so will try and work out how this came about and then (probably, unless someone presents a very good counter-argument) move the responsibility over.
@wagenet Although I do usually see `destroy()` called on objects in `teardown`, I can't say I've really seen the variable set to `null` (in the ember tests).

Playing around in the console shows that setting it back to `undefined` would actually be the original state. 

```javascript
var view;
// undefined
view = Ember.Object.create();
// Class
view.destroy();
// Class
// ...run loop...
view
// Class
view = undefined
// undefined
```

Is it a best practice to reset variables like that even though you're likely to reassign them in the next test?
With this I brought some Auth classes up into the application directory so we can modify our user model however we want.
Classes that are marked as `final` don't need to have their methods declared `final`.
> As for the double quotes for the `ENV[]`, Ruby doesn't care as long as its in an interpolation thing `#{}` :)

Learning something new every day, thank you!
I don't think you need to test that new_userlang is truthy... unless None is in AMO_LANGUAGES etc. which it isn't.
are you ready to rename this to "Themes"?
It would make the code a little bit too cryptic in this case, I think. upate_wrapper is good enough.
Small typo: Instalation ↣ Installation
Sry, I should have explained what I meant. "world.getHandle()", "(x << 4)" and  "(z << 4)" will be re-executed 256 times. The first one cannot be optimized away by the compiler (because the runtime world object's method might have side effects), and for the other two, I expect the Java compiler to be not good enough to optimize them out of the loop. The shifts might get eaten by modern CPU's pipeline, but the method call is heavy.
This last parameter is not tested before used. It throws error in many case.
No - if anybody wants to run our tests w/ nose, they should be able to.  By default, nose was doing some dumb discovery-related things, so I added this block.  Our tests still don't *require* nose or py.test to run, though.
Awesome. What about servers that use permissions and don't have any ops?
Can there be a bukkit.onupdate and bukkit.onbreak permissions that will alert players that have those permissions?
fixed the config binding and the auth line in the test.  ready for you to do the next one!
I have added a fix in [knp_components](https://github.com/KnpLabs/knp-components/blob/master/src/Knp/Component/Pager/Event/Subscriber/Paginate/Doctrine/ORM/Query/WhereInWalker.php#L96) if the bug is worth fixing, I guess not many people used query walkers that much for daily projects
this is causing the HierarchyTest to fail .. and probably its not the right approach anyway .. but without it .. the original collection is updated when the user does changes to the collection on the object and so changes cannot be detected properly.
Copy/paste was failing me pretty bad :(
this doesn't seem to be exposed anywhere
This looks really good.  Did you figure out how to get the menu on the right?
This is obviously wrong. Either the message is wrong, or the code, and I'm afraid that's the later. This means login+token authentication is broken now.
check this, you have a typo, ccontact instead of contact
For why? This was real useful
The LockManagerInterface::lock phpDoc says:

* @throws \PHPCR\InvalidItemStateException if this node has pending unsaved changes.

There was a misunderstanding from my part here I guess...
Dus een gebruiker kan alleen zijn eigen naam veranderen?  Dat is vreemd.
:D

I prefer two spaces in Ruby, but I've finally decided it just doesn't look right in JS.
Nit: Remove SortMethodsWith from the Javadoc. See http://www.oracle.com/technetwork/java/javase/documentation/index-137868.html#styleguide
I guess they could be static methods of OSDOp...
This does actually compile bootstrap2, plus our custom plugins.
*However*, if there's an error in an import, it now prints out the **entire** source tree to the console (in our case over 7000 lines). This makes the useful information very difficult to see. Is it really necessary to list the entire source?

meglio, the compiled output goes into the staging area at target/scala-2.9.1/resource_managed
利用するDatabaseに合わせて設定します。ここではお手軽にH2DBを使っています。
Thanks! That was it, I was using Titanium to link the resources.
No, but its the only way I can get the WINE folder to show up in the menu again.
this is useless here (and may lead to issues for maintenance as it will overwrite the param set when registering the TwigServiceProvider which is the one you will obviously look at to change things)
ありがとうの一覧・検索ページを /thankyous/ （アクションは index）として用意しました。
一覧時の per_page はとりあえず 10 にしています。

検索条件は「ユーザ名（の like 検索）」および「送った or 受け取った」です。
「送ったユーザが XXXX  かつ 受け取ったユーザが YYYY」という条件を作れずに断念した結果でもあります。
I think someone typo'd in your wheaties this morning.
Yes :-) I'am not interested in money but i think with thinks like Magmaw on trueblood or Guldemores Repack the Joke ends...

I think the Time will come that i'll release these Scripts but not yet.
if there is eventAboutToBePlayed shouldn't there be a eventPlayed ?
I made `er/mark-defun` for modes where `er/mark-outside-pairs` doesn't work, like `c++-mode`, and thought I would turn it on everywhere since it's always defined and that's how I found the bug.
done, updated on the pull request. df4f112e86ab12f9e4f7eb9c142d6fe62712a8b9
Who told you to apply this to your characters database?
Agreed. We may have a way to have more than two phase cycles on GE and store them all independently anyway...

J

Sent from my iPhone

On 22 Chwef 2012, at 14:46, "richardedden"<reply@reply.github.com> wrote:

> I don't like this hard code of 2 phase cycles... this code handles GE and others too - I think I'll add an if.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/cjohnevans/GannetMASTER/commit/80c5734358a30038dbe632f5c5f0434fb5ab29c2#commitcomment-996306
That's what I meant, and I knew there was a reason, just couldn't put my finger on it. ;)
If there is code duplication between assertElementEqualsFile and assertSvgRootElementEqualsFile, please extract it.
You will have to also remove "(FIGlet)" from Package Control as well.  This is how you mapped it on Package Control.  I would try and get it before wbond merges it.

```"ASCII-Decorator": "ASCII Decorator (FIGlet)"```
They key here was minimal impact to existing code, in most cases, we check IsDebugEnabled before rendering the strings. The callback versions were in the original Magnum logging abstraction, but I haven't added them here yet. This was meant to be a simple changeover at first, not designing yet another logging abstraction.
This quest wasn't working again. After some checking I noticed that there was an ID missing from one of the smart_script lines. This resolved the problem:

```SQL
UPDATE `smart_scripts` SET `event_param2`=3 WHERE `entryorguid`=18351 AND `source_type`=0 AND `id`=2 AND `link`=0 LIMIT 1;
```
Excellent work, thanks
django_statsd is independent of zamboni and doesn't have nose
Just did this because github was glitching
Yes, because the else part is not finished yet. :-)


Send from my PhoneBot

On 3 Mar 2012, at 08:24, Gabriel Filion<reply@reply.github.com> wrote:

> YAGNI
why did this contain jar-files?  please don't blowup the git-repo!
Maybe inline responseXml. It is always a good idea to inline a variable that is only assigned and immediately returned.
Since your are defining class JellyBean, all instances should answer true to: self.instance_of?(JellyBean)

because you appended !

it you be always false

and if one side of the AND condition is false...

well this line is always false!

you only want: @flavor != 'black licorice'

cheers
this one should not be needed as it is already registered in the Composer autoload
it could use at an optional build.
code is reviewed successful
I personally would prefer phrase “There are no flashcards…” to “You have no flashcards…”
One alternative that I am thinking would be reasonable if implementing this turns out to be a pain is to remove the checkbox from creation and always link back to the site when it is first created and then allow disabling it only after the creation has happened.

The only drawback of this is that it may require changing one screenshot in the documentation. Thoughts?
is this enough boops to satisfy you good sir?
Feels proper to me, fetching the response to get the `alias` feels easy enough.
Otherwise false.
Another thought:  using a cursor to implement a fold would also have the benefit of solving the type oid conundrum.
I actually had this code in there, but this will not work. The `worldtype` is stored in the NBT, thus making this param unnecessary. The environment is required because it is _not_ stored in the nbt. If you tell the worldcreator to load a normal world as a flat one, it will read the nbt and override you.
Marshmellow Fluff. http://humoroutcasts.com/wp-content/uploads/2011/05/marshmallow-fluff-entert0406-de.jpg
Please add jsDoc headers to every declaration - its unstructured enough as it is through entwine,
we at least need consistent documentation of behaviour :)
nice name, yes
how to enable..

```js
        // todo the following will display the share link when ready 
        // $('#sharelink').css({'display':'inline-block'});
```
スペースが足りない
I had to do the following so that the code will compile :

public void WriteLinqBinary(TextWriter writer, object linqBinaryValue)
{
#if !SILVERLIGHT	
        WriteRawString(writer, Convert.ToBase64String(((System.Data.Linq.Binary)linqBinaryValue).ToArray()));
#endif
}

We can't then use the System.Data.Linq assembly as it wasn't compiled against silverlight runtime. And its quite normal, since we are suposed
to do database calls on an web service.
In bbcodeAdd.tpl you missing the icon in line 3. The HTML is duplicate due to the way the Attribute-Adding is built.
`Theme.Holo.Light.NoActionBar` was introduced in Android 3.2. I would just use this instead to target all of Honeycomb without the additional special case:

```xml
<style name="Theme.ActionBar.InternalBase.Light" parent="@android:style/Theme.Holo.Light">
    <item name="android:windowActionBar">false</item>
</style>
```
Added a postrm script
@rcruz @adrianlee I noticed that the lib folder is missing from the build
Richtig, die anderen Hooks werden weiter unten in der entsprechenden aktivierungs und deaktivierungsfunktion nochmal zusätzlich ausgelagert. Von mir aus kann das auch raus und direkt in den jeweiligen Hooks gearbeitet werden. Was meint ihr?
This is invalid you should use `*` if you want to match any version.
Just download the latest version from github, and replace the  
buddypress-courseware directory with archive contents.

În data de Mon, 23 Jan 2012 05:40:03 +0200, aveangelus  
<reply@reply.github.com>  
a scris:

> how can i fix my plugin?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/scholarpress/buddypress-courseware/commit/c4fe685633ccc565f10bc2eac8efd6c35293abbb#commitcomment-891073
Судя по всему, должно быть UserLocation.
And this is why I wouldn't accept that story because I couldn't test it :)
kamarush, is this working for you ? I tried on my Neo and lulzactive_suspend makes phone unable to wake up back...
add this line

#include "wm8994_voodoo.h" 

into wm8994.h and it should work...

wm8994 has all the fmradio stuff in it.
I'm not sure if that was intended, but on 4chan it returns 'File1326230239.jpg-(100 KB, 555x475, filename.jpg)', same value as getImgInfo.
I think it was the first reason I saw them and I didn't immediately understand what they did. I've since found use of them elsewhere. I don't mind reverting this change.

I'll try to do this tomorrow.
@Nullable annotation is not consistent with 'assert bioEntityType != null'
We really need to document this better, I see this too much.
In fact, no. These values should never be null. I will remove the check.
Public API?
I'll try removing it ;-)
Is it intentional that if this throws an exception, the function may not have been applied to all children.
that was a strange link conversion, it starts on line 203, `diff(oldGossip, newGossip) foreach { event ⇒`
yes, postStop of parent is called after all postStop of children have run
Great, thanks again for the review, will address them ASAP.

On Mar 13, 2012, at 10:14 AM, viktorklang wrote:

>> +  // Needed for Java API usage
>> +  def this(exchangeParameters: ExchangeParameters, channelParameters: ChannelParameters) =
>> +    this(Option(exchangeParameters), None, Option(channelParameters), None)
>> +
>> +  // Needed for Java API usage
>> +  def this(exchangeParameters: ExchangeParameters, returnListener: ReturnListener, channelParameters: ChannelParameters) =
>> +    this(Option(exchangeParameters), Option(returnListener), Option(channelParameters), None)
>> +
>> +  // Needed for Java API usage
>> +  def this(exchangeParameters: ExchangeParameters, returnListener: ReturnListener, channelParameters: ChannelParameters, errorCallbackActor: ActorRef) =
>> +    this(Option(exchangeParameters), Option(returnListener), Option(channelParameters), Option(errorCallbackActor))
>> +
>> +  // Needed for Java API usage
>> +  def this(exchangeParameters: ExchangeParameters, channelParameters: ChannelParameters, errorCallbackActor: ActorRef) =
>> +    this(Option(exchangeParameters), None, Option(channelParameters), Option(errorCallbackActor))
>> +}
> 
> Stopped here for the day, will continue tomorrow :-)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/akka/akka/pull/370/files#r552733
IO workers
please put a FIXME in, this will work again as soon as we get a more recent scala version
List[ActorRef]() == Nil
How? I mean the question stated is "Should I use Dataflow or for-comprehensions?", and I don't see that answered in the code.
Change all the scheduled tasks.
double "a"
The difference between the first and second results is that ``sum`` adds to a Python 0.  For example:

<pre>
In [33]: type(sum([np.float32(1), np.float32(1)], np.bool_(0)))
Out[33]: numpy.float32
</pre>
We use a discrete peak finding to seed the non-linear search. The discrete
peak finding finds maximums by comparing neighbors on a grid (or gridish).
Although the old line is nasty try to avoid using ids as you limit yourself to one per page maybe not an issue with the data preview but generally not good to do.
I think the tests would be much more readable if these method calls looked like  `action_post('user_create', data=data, apikey=self.apikeys['random_key'], status=403)`. i.e. just make it a trivial wrapper for `self.app.post`. Just so I don't have to memorise action_post's param order or remember that it takes "random_key" and looks it up in self.apikeys.
_auto_manage_db() is called inside of main(). I made the separation looking forward to when there may more more things this script can do, so I wanted to separate the functionality into its own method. Currently the unit tests all call main(), so there aren't unit tests that specifically call _auto_manage_db(). Those tests could very well be changed to call _auto_manage_db() instead if you like, but my intention is that main() will call _auto_manage_db() by default unless the params specificy a different function (once we actually have some different functions).
Is this intentional? I know it has no impact on the test, but funnily enough in my language (catalan) ``passo`` means *I don't care* :)
I think you can remove the {}
This will break JSON-RPC clients retrieving the "runtime" property of a video as they expect a string. And it will also need changes in the JSON schema.
This causes all the SortByFoo methods which don't have a corresponding SORT_METHOD_FOO (and therefore aren't listed in the map in SortUtils) to appear as "None" in the smartplaylist editor dialog. Either add the missing SortByFoo to the map in SortUtils or keep CSmartPlaylistRule::GetLocalizedOrder() for now (as the map it uses is still present anyway).
These aren't ints anymore, right? You might just want to refer people to the cost definitions file instead of AWS site.
I think just `options.quietness` would be fine here. Either it's going to be `>= 1`, or it's going to be `0`, which is false-ish. Also, not putting spaces around your operators makes `pep8` cry.

Also, I see what you're doing with "quietness", but my preference would be to continue calling the option "quiet", in the interest of not having multiple similar names for the same concept. Also there's only one place in the code where you need to check its numerical value.
That should be `continue`, no? `break` gets us out of the whole `for` loop.
Really really stupid question, but is += implemented directly in MetricsGroups. If this is the case, this could be a val. 
This isn't functional, but at least it uses val...

I definitively have funtionalitis...
make them private
I feel like arguing either way about this particular conditional is super pedantic and not very meaningful.

On one side, `is not None` is an explicit "did the user specify a non-default value for this kwarg." Some theoretical MySQL adapter might use a lock_mode that both evaluates `False` and does some non-null locking action. On the other, removing 'is not None` removes 11 characters and some visual noise. In reality there is basically no difference.

See http://www.python.org/dev/peps/pep-0008/#programming-recommendations point 2, and http://effbot.org/zone/default-values.htm
Hell, a record might not even be the best idea, unless we make it opaque.
This is very nitpicky, and you can ignore it, but since this is supposed to be related to the SupplyDemandManager class and the name CycNegativeValueException is a pretty generic name, it might be worth calling this something like CycNegativeSupplyException .
erm, yeah. `type` is the way to go.
It's just like undefined behavior – just because it hasn't resulted in problems in testing so far doesn't mean that it's correct. In this case, it's conceivable that a font could have an ellipsis character which is longer than three other variable-width characters, so adding the ellipsis would actually make it longer than it should be.
Might as well use `ARRAY_SIZE(argv)` here while you're at it.
Is there any reason why you made the master remove the entry right away instead of waiting for a message from the worker? The latter seems simpler and safer unless the master will immediately need to check whether it's removed some particular dataset.
The validations are [in Compiler](https://github.com/nette/nette/pull/839/files#L0R159)
верните обратно QtCore
The same check to add the table name has to happen for each column name, otherwise the method would work differently for each case.
I think is better to use assertions here.
This breaks apps upgrading to Rails 4. They'd need to *migrate all data*.

This should act as a hash with indifferent access instead.
As Josh suggested, we need tests here which specifically deal with corner-cases such as adding a certain number of elements into the linked hash set, then removing them all and inserting elements again (on each insertion testing the consistency of the linked hash set).
The certain number of elements here should be: a single element, 2 elements, multiple elements.
If a Java client extends this trait (that is, interface) without extending `Runnable`, then a Scala compiler might later complain - but I haven't tested this out, just thought it might be an issue.

Well, since the only place where this is extended is internal to the framework, and custom execution contexts can only check if the runnables they are given are `OnCompleteRunnable`s, I don't think it plays a difference. So, LGTM.
Can this receive some documentation please? "Dummy" can mean all kinds of things, and elsewhere in the compiler doesn't mean what it does here (that is, "dummy type args", which are not placeholders the way that a pendingSuperCall is, nor is pendingSuperCall a distinguished object which exists only to satisfy the absence of the field like emptyValDef is.)

Plus as long as we're in here, "emptyValDef" is a horrible name. Let's call it "emptySelf".

It is not clear what isEmpty means on a Tree; it is not clear what isDummy means either.
you can use pararentheses:

```python
def blank():
    return (long line .................
        rest of long line)
```
sympify works just fine. I will replace compile_rule in my next PR.
Everything is all right in raw file (https://raw.github.com/resurtm/yii/efa1dfde52b0b3eabb645d1891aa1680240f59fc/framework/gii/generators/crud/templates/default/controller.php).

Here's screenshot from my IDE:
![](http://storage9.static.itmages.ru/i/12/0624/h_1340548958_6598489_f0e918cf33.png)
!!!! AWESOME !!!

I am not sure it's needed here but AWESOME!!!!
I agree with blampe that we should have some documentation about why a user would choose @setup vs @let_before.
Should be changed to 

public void setMeasurement(final StudyOutcomeMeasure<? extends Variable> som, final Arm a, final BasicMeasurement m);
Please add comments letting me know the purpose of methods
Why this line is needed?
yeah, let me try your way and see how it goes
`Interval` classes don't support null values, which we make heavy use of.
what about:

```
page_ids = []
for row in kc:
    page_ids.append(row[0])
iters.append(self._query("""
    SELECT * FROM pages
    WHERE page_id in (%s)
""", ",".join(page_ids))
```
?
either single quotes or double quotes but not both for consistency.
You mean, abused?
> Shouldn't censor_text be run before truncate?

No, we have an option to disable censoring, so that full, uncensored text is provided to user. Therefore all censoring needs to be run just before output.
If this does what I think it does it needs to be tested on postgres. Within a transaction, once a query fails subsequent queries will be automatically rejected.
Get rid of inline assignment here.
the for/id attribute combo is not need here as the input is inside the label.
`$user` doesn't describe what the variable contains. I'd suggest something like `$revision_user_id`
I'd like to see a test where the error is a retryable exception but the job has exhausted it's retries.
instance.base might be undefined ?
"action" is not declared or set within the scope;
I'd like to see us use options to set the Y log level and update all our logging calls to log at a level rather than this type of checking. So no if() here, just libutils.log('Removed: ' + shortDest, DEBUG); 

Maybe we need a design discussion on how to upgrade our logging so we can do a general pass.
It might be nice to also include a link to the docs (http://about.travis-ci.org/docs/user/gui-and-headless-browsers/). That might make maintenance easier in the future should the docs change.
Can you also add a '-' button to remove one item from the list?
text is wrong. 
* not a drug purchase order - this is Billing
* also, this is the ```onShutdown``` method, not the startup method
White space to correct.
基线值还是以0开始吧。不然最小的一个完全无法显示。用户设置为undefined的时候才从最小值开始
The test in the c_code is not needed. The dtype is tested by the TensorType.c_extract fct. So I'll do your change and remove the test in the c code.
политика партии поменялась) делаем не на STI, а на Cancan https://github.com/ryanb/cancan
убери все модели, оставь только User.
Every name on its own line.
`helpers.extend` would be cooler
Also, test data thematics must be inspired by the tv-series Jersey Shore, see api unit tests for details. No potatoes.
On second note, why isn't this failing? There is a test that I believe should have caught this, https://github.com/github/gollum/blob/master/test/test_app.rb#L190
One problem is that `[..xs]` will `copy` all the values in `xs`, so `move` capture does a copy, then a move. It makes more sense for me to have `lambdaPack` construct the lambda object in place of the lambda expression, and insert a call to `lambdaUnpack` into the lambda body to bind the captured variables:

```
foo(a, b) = x -> x + a + b;

// becomes

foo(a, b) = lambdaPack(#<lambda symbol>, a, b);

overload call(<closure>:<lambda symbol>, x) {
    forward a, b = lambdaUnpack(<closure>);
    return x + a + b;
}
```
Why lose the "exec" here? Also `$@` needs to remain quoted.
Yep why not. I don't know your plans for this module, as request headers are just a plain object.
Add reasonable 2-3 sentence descriptions for each command.
Apparement, je l'avais mis dans le git mais il n'est pas vraiment utile ici. Tu pourrais donc le supprimer aussi.
You need to end an inject with the object you want to pass into the next invocation. In this case, it'd be +rack+. However, you're not using the result of the inject, so use an each instead.
I'm not sure this is the right place to fix this. Have you seen what `asobject` returns when you have NaT values?
Trick to expose the pimpl methods as slots to be used with signals because the pimpl is not a QObject!!!!
You probably want to change this to `Float`.
What do you think about using `alias dup clone` instead of declaring the same message body twice?
I'd suggest to make jaidir a parameter as well, if this can be changed on fbsd (I believe it can)
英译中please
Can you line up the indentation?
If the user assigns the same index to multiple fields, these fields will have the same value (hoping that their types are equals if not an IllegalArgumentException will be thrown ). If he doesn't have enough fields there is no problem (the fParameters array will be upper than the number of annoted fields) and if the index is bigger than the length of the fParameters array, an indexOutOfBoundException will be thrown.
We can check if  the number of parameters and the number of annoted fields are equals and if the index value isn't upper than the length of the parameters array. If applicable an error should be thrown.
I went with an empty string to mirror the behavior of other getters in the feed model. I don't know why that convention exists. (At first I assumed it was to keep the type consistent, in which case I'd want to return `{}` here, but `_getImage` also returns an object on success.)
Not sure how much effort you want to put in this, but I would say "Move font stacks into variables that can easily be reused and overridden"
menuItem is unused here
In this case, the code actually seems wrong--if we had reverted to the previous valid color (by hitting the second `if` case above), it definitely seems like we shouldn't set `_this.hsv` to be `newColorObj.toHsv()`, since that will be some invalid value.
Instead of having two functions, how about allowing the first parameter to be a string or an array. If it is an array, do the optimized version. If it is a string, do the slower one.
I don't see a Promise here... is this comment vestigial?  (Same goes for _close())
Ditto -- this one now depends on both of the previous tests (e.g. for the expected value of $listItems.length)
This change reduces code redundancy!
JavaScript function names are case sensitive. Bonus: case sensitive string matching is faster!
It seems like this is done everywhere we call findNext() _except_ for the one call in setInitialSearchQuery().  So I wonder if we could just bake this into findNext() itself and either (a) have an optional arg that setInitialSearchQuery() can pass to suppress setting it, or (b) have setInitialSearchQuery() explicitly set the flag to false after it calls findNext().
I don't know any css attribute that has numbers, do you ?
This is not ui/navbar
that doesn't seem to set self._(function name)
Extra comma at the end of this line
Just make sure this is XHTML valid. Bare input fields I believe are not valid in `form`? Because those are hidden input fields you could just wrap them into `div` with some class which does (additionally) `display: none`, similar to how `csrf_token` is rendered.
Rerun XHTML online validator, and got warning that input and button should be in `<div>`so I put them inside a div. Now XHTML validation passes without warnings.
Of course test it with the browser afterwards. To see if it really proposes the correct language as initial value.
I'm mostly just curious as to why all of them are assigned email. Probably something I just don't know about our database info.
Same deal here.  Explain a bit.
It feels like this jump happens rather too early. If possible would like to keep the four columns until 600px and scale the images down to help with the fit.
Yeah, I wanted to do that but then I saw it's commented out. Anyway, doesn't hurt.