labofsahil/github-event-dataset-2012 huggingface dataset:
1 language on object creation, empty tabs at end, can't remove default translation, CSS changed

* Does not display additional language tabs on object creation, only default language
* Empty language tabs go to the end
* Can't remove default translation
* CSS changed:
  * empty languages = color: #ccc
  * available but not currently selected = font-weight: normal
* added ```__unicode__``` method to BaseTranslationModel
LPS-25946 Instanceable portlet on removal from page does not clear up In...

...vokerPortlet cache causing memory leak
Update dependency of ganglia on apache [1/4]



 chef/cookbooks/ganglia/metadata.rb       |    8 ++++++++
 chef/cookbooks/ganglia/recipes/server.rb |    2 ++
 2 files changed, 10 insertions(+), 0 deletions(-)
LPS-24965
LPS-25549 Make website code more robust

Already reviewed by @juliocamarero
Clean up the unix mtime tests and fix parsing during leap years.

While fixing this issue that I just ran into with parsing during leap years, I attempted to clean up the time tests to avoid the situation that led to this failure in the first place, specifically sharing copy/paste logic between tests and code under test.
Animated Case View - master Branch
Added benjic as  Learner

Updated members list to reflect my involvement.
Added self to roster

Hey, just adding myself to the roster, looking to learn some stuff.

Thanks!
Run pydoc as a module. Python does not seem to ship with a pydoc script ...

...on all platforms.
Added Announcement LInk 

Please pull, I just cloned and added a link to the google groups annoucement link
Edu bootstrap migrate 0.9.0 code

Migrating `0.9.0` code for @rcoleman this is updated from my original [pull request](https://github.com/puppetlabs/puppetlabs-training-bootstrap/pull/32) . This should be the latest code from the point of migration: ee0f145fce55a3c5e111126c3a0731529cb4319c which has this parent 695279f736002f869fa3d4c2ccf44c24ffb6a273. @rcoleman should be good to go with the updates for 2.5.2 and  exercise updates from 7/20
INT-2431 Improve 'Dispatcher Has No Subscribers'

Add channel name to MessageDeliveryException

'Dispatcher has no subscribers'.

In a large integration flow, it can be difficult to track
down which subscribable channel has no subscribers.

This commit adds to the message text in the form

    [channel:someChannelName]

to the exception message. For example:

    "Dispatcher has no subscribers. [channel:myChannel]"

Also

    [amqp-channel:someAmqpChannelName]
    [jms-channel:someJMSChannelName]

This does not apply to redis-backed channels because
they use a BroadcastingDispatcher, which does not throw
'Dispatcher has no subscribers' exceptions.

@olegz @dturanski
Version number not updated

This patch lets setup.py pull the version number from musicbrainz.py.
That should prevent you from updating only one of them, next time you do a release. ;-)
Add Heroku deployment instructions to README

Also clarify `bundle install` options
Test Fixes

portal/dbupgrade/viewsampledatalatest/address
portal/dbupgrade/viewsampledatalatest/webcontent
INT-2355 Update Mockito to 1.9.0
Complete the image join between file

- There are commits i could not rebase for i had already pushed them (i made a mistake with magit).
That was a huge mistake

By omitting "base/" for the permissions
Moodle 23 stable

Dear Julian,

Here is my latest work on renderer.php - lots of changes to simplify things, make it more OO (less globals) and robust with underlying core code.  Fixed topic name / summary text in icons and sections when they don't exist and the code no longer makes them up and places in the database.  Added the icons (new 2.3 functionality) at the bottom of the sections when editing to add remove sections with AJAX - same as core course formats.

Things not working yet are the 'One page per section' and reusing the section_header function that is inherited from the parent class.  Plus a few old bits of functionality like 'get_title' that should really go.

I know this is another pull request, but thought you would rather spend two mins dealing with this than hours duplicating what I had already done.

Cheers,

Gareth
make 'em look alike
in call volume fix

Fix in call volume bug
Fix Submit button styles to be btn-primary instead of primary
Fixes #7

Fixes #7 by altering [uploaded|downloaded]_[packets|octets] columns.
Delete confirmation for long statuses is not shown properly

Modal confirm box to confirm status deletion does not resize if status text overflows. This makes long statuses unable to be deleted.

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4418
beggining of db anonymize feature (creates tables, copies old data and deletes)

in agilefant.db.export i added two files  Atablesmodifier and anonymizetestmain. main is for testing. First run will create anonyme tables and second run will delete them. Tables are NOT anonymized they contain copy of old data.
Develop

German Translation of getting_started guide!
For Issue #31: Remote site with not empty path in app root fails to follow links

Hi all,

I've written a fix with some tests for Issue #31.

I have all the other tests passing (although I did have to alter the Gemfile to update to rspec 2.8.0. the tests would not run in rspec 2.0.1 as per Gemfile. This isn't included in my commits.)

This change has allowed me to use capybara-mechanize to run tests on a PHP app in a subdirectory of a server.

Feedback appreciated, and many thanks for your work!

-Matt
maguro: Update NFC fw for maguro 4.0.3

Fixes tag reading. Before this, the only NFC functionality working in CM9 for maguro was Beam. Tags didn't register at all.
test!

tost!
scroll to added pagepart
get pics for a specified user_id
Tickets/issue 12 capitalized headers
Add deleteVersions code back into JavaClient

Already code reviewed on 11/12/12 by Dave, but accidentally merged out again afterwards.
Updated to use latest Web API RC Nuget packages
Added Bulgarian translation for several strings

* favourite_group
* other_group
SublimeVideo

This is a Kirby plugin to embed HTML5 videos using SublimeVideo
Update demo/controllers/admin.php

just trying this Auth library.
found that variable error, but realy nice work!! -- Jaume MK
compatibility with activerecord-postgres-hstore

Right now it is not possible to use these two gems together, this patch restores compatibility.
New : Added composer.json

Added composer.json file which will help this module for submission on proposed extension.silverstripe.org website. Please add more fields according to requirements.

For more information please visit at http://extension.openbees.org/instructions/

Todo  :

1 -  You  need to add "extra" field for snapshot, like
```json
{
    ......... 
    "extra":
    {
        "snapshot" : "http://example.com/example.jpg"
    }
}
 ```

This will help us to showcase your module in "New Module" section.
Use corrected function name from silverstripe-cms project

The CMS project had a function named "SiteTree::prepopuplate_permission_cache" which
should be "SiteTree::prepopulate_permission_cache".  Since another commit corrected
the function name in that project, this project needs to use the new name.

See https://github.com/silverstripe/silverstripe-cms/pull/51 for the master pull request where the function was actually renamed.
Added init.js to index.html

Added init.js to index.html. This should allow the live audio stream to play.
Changes for V1.1

Adding ProductVariationImport.csv file for sample CSV data.
re islandora 492
Fixed the world_converter, still were some errors in it.

Should now succesfully work.
German Translation

There my be some Translation errors, but this is just a first start
Remove unused variables, fix validation warnings

This commit removes two unused variables and fixes several JavaScript validation warnings.
Next

Fix configure script and lint
Fix for issue #14

That was a simple fix :-)
Update Ukrainian/main/MediaProvider.apk/res/values-uk/strings.xml
Caching culture lookup

Looks like wasteful processing on every hit to the site. This saves iterating those arrays so often.
Updated README - mostly just to have a link to the prebuilt downloads
exploration des égouts
@haehn Shouldn't we use "getters" for those local variables then? (rather than static text?)
For consistency -- generally speaking we're using:
return $json->getString();

...rather than...
echo $json->getString();
Im not sure, maybe this is temporally, because i'm making some changes on templates ...
sorry, i removed last time, because it changes colors at home, and it is more complicated to work
I'm not completely sure this is correct. The purpose of this is to use the `call` operator even when directly calling external functions, thus making it possible to do the type casting rules in the library.
I'm *not* the big fan. Eh eh, forgot a word, sorry.

And yeah, I'm talking about the `SpriteReference` init.
Cool kan dat ook met sickbeard en headphones?

Eventueel kan je ook nog couch (e.a.) de pidfile laten beheren met het --pidfile=/pad/naar/ commando, dan werken ze het pid ook zelf bij.
@rainyday +1

He could made really bad things - without making "any noize". Instead he just alarmed the REAL BIG SECURITY problem in a really kind way IMHO.
@pusewicz

![406 not acceptable](http://dl.dropbox.com/u/29972428/kwejk/http-status-codes-cats.jpg)
m_dTime の間違い
Opened pull request for libuv: https://github.com/joyent/libuv/pull/292
TODO fix the redundancy
@onesuper :)
Shouldn't this be rect.right < 0 and rect.bottom < 0?
Yeah, sorry about that. There wasn't one before and in the last commit I get rid of it again. I had put that there so I wouldn't forget to add those tests.
nope, this is safe
.sass example using it:
``
#content-whatever
-- @include span(4)
``
This got fixed in http://github.com/monlog/monlog/commit/5cef107519494769a2d6f4f2336c4be6566c6d8e
Thanks Ryan. I'm +1 on serving media, but will get at least one other core dev to review before merging.

My only reservation is with the existing Pinax approach here in light of the new contrib staticfiles. Since static() uses DEBUG to determine whether or not to include the url, I'm thinking we should either simply rely on DEBUG explicity for both of static and site media or make sure SERVE_MEDIA is respected for both if we want to continue supporting that.
Err, is that //* some sort of dryice thing?
We should be able to use `dojo.place` to place the section node at this point.  Also, for all node references, we should try to keep them postfixed with `Node`.
No, it's not a rails bug. It's a bug in the restler JSON-handling code that is triggered by interacting with rails rest services. The restler code blows up if data is not null, but contains no useful data, so this is just a trivial fix to prevent that

Realistically, this should be left to the JSON decoder itself to determine if valid JSON was provided, but that obviously doesn't happen nicely. Another option would be to put the JSON.parse in a try/catch, which would handle a wider array of possible deserialization issues.
ReportingService provides a set of reports it can generate on its own.

```java

public Path renderProvidedReport(String reportId, List<IReportData>, ClassLoader classLoader)
```
@anjonas Kom också ihåg att använda nya hash-syntaxen, alltså key: "value" istället för :key => "value"
We should just bump the default for all the boxes to the same memory footprint.  (IMO the point of this project is to maintain uniformity on all the images).
Thank you!

On Mon, Feb 6, 2012 at 7:32 AM, Eloy Durn <
reply@reply.github.com
> wrote:

> @escoz Looks good! Thanks :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/CocoaPods/Specs/commit/36a3b74dc14540bb8c16804a274340fbbff58ff3#commitcomment-936500
>
Untenrum!
Не, ну чего сразу обижаться? Я сделал просто чтобы команды вводились по-старому, вот и всё.
"Team" is capitalized but "the" isn't? bullocks!
The code removed by this commit shouldn't have happened; debugging Safari is a real pain sometimes. Helpful hint: if your CORS request isn't working and it requires credentials, verify that the domain is allowed cookies based on Safari's policy. You might need to subdomain the API.
Sure, is strange something work with out sql.
Very good job ;)

Please consider the following: 
-------------------------------------------

2. Your coding level has increased. I'm proud of you ;) keep it up. pretty fast you'll become a tough coder ;)

3. When you write a new method, please add PHPDOC comments to it, starting with `/**`

4. I'm not sure what is the last parameter $force doing, since nobody ever uses it and the name is not very self-explanatory. 
Renaming it into $addToVisitedList might help better understand. 
Readable, understandable code values lot more than short code, full of `$i`, `$j`, `$k` ;)

5. Since the only place in the project to use this method is in QNA, there is no good reason to move it out to the Helpers class. I know you have been thinking about the future, where visit counts may be required in another places, but there is a principle in software development saying : keep it as easy as possible and don't overcomplicate it before actually needed. You will always be able to move it out to helpers in a later stage.
For now it's best place would be as a method of the qnacontroller class itself. 

*on the bottom line:* I suggest moving this method to be part of qnaController class, making sure you give it a name *fully* describing it's *entire* functionality, because now it doesn't only check, but adds stuff. This means function's name is lying ;)

6. methods returning booleans should start with *is*. `isAdmin()`, `isViewable`, `isQuestionViewedEarlier`. same goes about variables
Are you aware that this option can cause routing loops?
only because of npm works really strange on travis vm
`all()` isn't necessary here. `Collection.objects.count()` works just fine.
Yes, i don't need to check that. I just needed a collection with a description and then wanted to test if setting a blank description was working or not.

I am unable to do the same by your method as well for some reason I haven't been able to figure out (https://github.com/vaidikkp/zamboni/commit/919d2c86430f0602350af2f63d076f87a50a6e8d#L0R451).

Getting the following error:
ValueError: invalid literal for int() with base 10: 'a'
Eye C wat U did there
No, this is not OK. This chooses language based on the browser settings but we want to chose the language based on user settings for our portal. It is true that also our own language settings will be based by default on browser settings, but if user overrides that, search in Google should follow.
This fixed a problem with Japanese characters for me. Thanks.
On Fri, 2 Mar 2012 00:11:42 -0800, Philippe DENIEL wrote:

 PD> I do not agree with this commit : FSAL_DIGEST_SIZE_HDLV* are not the same as sizeof(p_in_fsal_handle->data). 
 PD> The digest is (by definition) smaller than the FSAL_Handle

Yes. And therefore one should not copy random stuff off the end of
p_in_fsal_handle->data into a digest.

I've had random operations failing because the filehandles were changing.
In reference to #70

I hacked it locally to something like below and it now delivers messages when `ssl` is `true`.

    try {
        var _http = ssl ? require( 'https' ) : http;
        _http.get({
basically, everything is now under ./clojure instead of ./py and ./clj. This will help when we have full interop with python. Besides, the code just looks cleaner this way.
I believe $size query only match exact values, not ranges. You need to do it manually (adding a size field and do the bookeeping yourself) then you can query on a range.
dont add venughs things :P his c++ is awful :D he can  make crashes where normaly no one can be ...
i know (because i tested it), but i think this change does not fix the crash, in my results, the error is into the function "getTileAt", not here.
In Faye, every client can publish messages. A faulty or malicious app/visualization could seriously screw up boxxy if we don't limit publishing permissions.
I can remove those two methods ?
Good catch man !
That's just wrong search/replace I guess - inject on describe makes no sense.

Fixed: 6c4f1391bc10ebfd76bb39d3cb7f7b3b3bff4dd5
`# :nodoc:` this, it's a standard method and you're not doing anything surprising.
Don't forget to regenerate the thumbnails after pulling this commit with: 

```
rake paperclip:refresh:thumbnails CLASS=Picture
rake paperclip:refresh:thumbnails CLASS=Video
```
А какая разница?
错了是妖凌统
ay reason why you changed the name from delete?
You may fancy using `safe_columns.select(&:text?).collect(&:name)` (just saying :) )
Hm.. Yes, I supposed that someone would prefer to have an option and keep existing habit, so I'm agree to add an option.

I tried to imagine your case and first idea was to simply create new experimental branch, commit there interesting changes and stash other changes. We can easily create and remove short-living experimental branches, so I've never afraid to create one more temp commit. In addition this solution is more general: it allows to select changes that you want include to experiment from both tracked and untracked files (imagine more general case when you have few tracked and untracked files and you want to keep part of tracked and part of untracked files).

Could you suggest place and behaviour of a new option?
Should it affect all stash operations or manual stash only?
Will this option be set once or user should have it at hand and switch everytime (i.e. on stash form)?
yes of course, if there is uncertainty, better to be protected. This was only to ask, not determinant.
cc @addyosmani .. need to update this to use docElem or something
The makehtml one is as in MacPorts, the "pndriv.c" is mine. Granted, both could have been made with inreplace. However I just tested using /GIF and /PNG drivers and both fail... so I've some work to do on this...
I know it's old, but if you see in the line below.  is still used.
1. surround $user['id'] with the attr() function

2. surround $user['fname'],' ',$user['mname'],' ',$user['lname'] with the text() function
I'm in support of reverting all these `hasOwnProperty` checks.  I do think the above test is valuable, however.  We currently rely on calling `extend` with events and we test it nowhere.
This was capitalized, which was causing a lot of errors.
Also, should the adapters live in storage or extensions?

On Feb 28, 2012, at 5:33 PM, Nate Abele<reply@reply.github.com> wrote:

> Hm, I wonder if my last comment just disappeared into the ether. It's not showing anymore. Anyway, I suggest moving this to `li3_queue\storage\Queue`.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/UnionOfRAD/li3_queue/commit/00a5e21092b8b94d20bfa5c355b2523176487d27#commitcomment-1023466
the change is restored in the next commit.
We've had some difficulty understanding the documentation and source code on this and tried illustration so that we could talk about it to each other, we're still not sure if we have it right though. So far it has been a trial and error process. Would love some feedback from any of the contributors on this: http://stackoverflow.com/questions/9010351/understanding-jquery-mobile-swipe-configuration-variables
I have requested an account and it is awaiting approval.  I'll see what I can do about rspec-puppet tests.
+msgstr "nie jesteś zalogowany jako tamten użytkownik"
s/tamten/ten/

+msgstr "użyłeś tutaj znaku którego nie możemy obsłużyć"
s/znaku/znaku,/

+msgstr "By je wydać odwiedź [/gold](/gold) lub stronę twojego ulubionego użytkownika."
s/wydać/wydać,/

+msgstr "dzięki za danie konta gold!"
s/danie/podarowanie/ or s/danie/ofiarowanie/

IMHO we should be consistent and either use "reddit gold" or "konto gold" but not both.
・無駄な処理の省略
Thanks. Amazing theme! =)
if you take a loop i look through NUM_PAGES so the url such as http://hypem.com/popular/3 will get parsed as well for instance if NUM_PAGES is larger than 3.
No, it's already merged. We'll update it at some point when it lands in Django.
Being more specific makes the ruby script of jsDuck a lot faster.
Seems I have an issue with the firebird library that's giving me these issues. I'll attempt to resolve them so I can do some testing myself. In the mean time, how's this looking @philsturgeon @narfbg?
> Yes, the only option would be to have Spartacus pull every other night,
> which would result in longer "release cycles" for plugins.

Sorry, I don't understand you: Why would that be a solution? There would also be that dangerous time 
frame (the other night)..
this fixes nothing. overriden PrintError function will be never called
What is this for? It's hid's all select inputs.
for me stable core from date 16.12.2011 , other unstable :(
UPDATE `creature` SET `MovementType`=1 WHERE `spawndist`!=0 AND `MovementType`=2 AND `guid` IN (49123,49124,49125,49127,49128,49129,49143,49144,49145,49146,49150,49151,49154,49156,49157,49158,49167,49186,49187,49188,49318,49319,49664,49665,49751,49762,51398,51402,51422,51423,123148,127753,137490,137491,202314);

`spawndist`!=0 - copy paste or not?
Plucky now has a Methods constant for what needs to be delegated. We should probably use that in MM.

https://github.com/jnunemaker/plucky/blob/master/lib/plucky.rb#L14
It's cuz NULL == NULL or (0 == 0)
it's tatolly wrong here.
+    if (m_seasonalquests[eventId].empty())
+        return true;
Another utility of CommonsIO is to read a character stream into a `String`.
I don't understand this change... _GetInt32("Port", 3306) Already sets the value to 3306 if the Config option "Port" is not a number... (see _GetInt32) so why check it again? That Default value was included to avoid to have to do any check outside Settings function.
Project search revealed no other places where it was used
I think it is a better solution.  But it would either require addressing the problems with the FullTermParser or modifying the TermParentParser to return all parents and children (maybe ignoring special OBO relations).
I have not tracked down the tests that would have failed if they were not deleted or change in this commit, but I have a gut feeling theses tests are so white-box and contrived that a critical error was introduced in 2.3.2. No longer do your lazy attribute blocks allow you to talk to your model. Instead you now get:

```
NoMethodError: undefined method `killer_instance_method' for #<#<Class:0x10c1afa60>:0x10c1aeea8>
```
oO my bad
nothing has changed. diff is due to CoffeeScript compiler upgrade.
i am getting "Warning Your system does not support epoll"
This is not a dependency of Refinery, it's just in Refinery's Gemfile.

I think that a better solution would be good - i.e. once dependency we can add to these projects that handles all this for each project so that we don't need such verbose Gemfiles. Maybe @reset will comment?
:( Nothing to learn from you.... did you uploaded all the kernel again?

is there a way you can show every one what is really done.

Thanks.
For example: `"nextMask": "http://somesite.tld/pages/theme|10|.0"`
Thanks, I really should have had another look at what I had done, I need to stop relying on you to do that  :)
Sous windows, l'ajout me renvoie l'erreure suivante :
   "Column count doesn't match value count at row 1"
Listiterators of generics return their type
line 155, in start Process.start(self)

pickle.PicklingError: Can't pickle <type 'thread.lock'>: it's not found as thread.lock
394 INFO  [lr.lib.model_parser] [MainThread] Cannot get document type: 'str' object has no attribute 'value'
668 DEBUG [lr.model.resource_data_monitor] [MainThread] 

I'm looking into it some more. Thanks for the help.
> I am not much fond of the code that adds yet another trap that is hard to debug. If there will
There will not. Its a common design pattern I use for a reason
in all of my plugins.
And it is debuggable if it ever happens. A simple grep for the key is enough.
And if the caller cares about it he/she can add a copy easily()

sort, filter, .. they all change in place. So tell me why I should do it
differently.

I do no longer want to spend time on discussing such issues.

What should be fixed instead? Help me get an VCS based wiki up and
running because people write @ vim_use and irc all day:
- how do I configure Vim for latex, c/C++, PHP, python, ....

If we work on that we help people. If we discuss such small things
nobody cares about we don't (IMHO).

Or help me with a JS interface.

faithfully
Marc
~ PwnKernel 1.4 ~
I don't know about this line Chris, I think you're just taking credit ;)
You had some code here about returning, but honestly, exit never returns so we shouldn't consider that.
`db.close()` in the `finally{}` clause should also get called after the failure cases. Unless you mean it should be called before `session.abort()`?
test notitication
Can the store fail?
Format:  Please leave two lines before and after classes
This might fix #197
made it:)
i recomended to using my file manager source, and built them with ics tree
I merged and resolved everything. Don't know what is the issue here.. all alog are log and changes around them are added
Sent from HTC Desire HD running my IceColdSandwich

Twisted <reply@reply.github.com> wrote:

>DisplayHardware: Set MDP composition flag correctly
>Change-Id: Id0c7172fe8578f3eb26bbe10a448a96fee1d72cd
>commit 0179465b3aad2654f4b1eb3e6aecdb647a27c551 1 parent a22a7ae1d0
>Naseer Ahmed authored 9 days ago
>existz committed a day ago
>
>LorDClockaN does not maintain commit history when resolving merge conflicts and many of the patches available on aospX would require reverting the LOG -> ALOG changes. This was handled by allowing LorDClockaN to cherry-pick first.
>
>These changes were previously present in this project, but required reverting a change before they were added that would cause massive corruption. These commits need to be checked to ensure they contain all of the same information previously present.
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/PlaygroundICS/android_frameworks_base/commit/44baa2af121374f73d38a498c6ee600ef16eecaf#commitcomment-1029434
ok i have to say Im having some trouble working with these libs (Freimage, glew). Please tell me how I can build the project using them.
@ohadlevy @jeffmccune  in vSphere terminology,  'Create a VM' means creating a blank VM (without template),  'Clone ...' means cloning from a VM or Template.

So I think we should implement like this:
1) use server.create() to create a new blank VM
2) use server.clone() to create a new VM by cloning this server(or say VM)  ;  server.clone() will call connection.vm_clone().
3) use connection.vm_clone() to create a new VM by cloning the VM specified in the param

Thus, we need to keep the existing create() and modify server.clone()
Something like this should work too while being shorter:
```c++
if (!_bv) {
    _bv = new AABB<Point>(_sfc_pnts, { pnt_a, pnt_b, pnt_c });
} else {
    ...
}
```
That was my thoughts too..
This should be in either 3.6.1 or 3.7.0?
Better reword this:
Forbid changing the style of the first page to "Dialog"
It looks like there's some IDE auto-formatting funkiness going on. Please revert the indentation changes as well as the brace location changes.
Consider using function below from ``/includes/wikia/GlobalFunctions.php`` to shorten a snippet. Additional benefit is that it uses "localized" ellipsis.

```php
function wfShortenText($text, $chars = 25)
```
those two are not equvivalent, but I would say Class.forName is correct for our usage, but I think @bantonsson should confirm

http://stackoverflow.com/questions/8100376/class-forname-vs-classloader-loadclass-which-to-use-for-dynamic-loading
http://blog.bjhargrave.com/2007/07/why-do-classforname-and.html
If it should be fixed as a part of the PR there's no way to forget it?
Hi & thanks for the comment Victor,
Roland pointed out in another comment that this way we avoid problems if the refresh would take longer than the `retryInterval` (schedule the refresh when the previous is done, instead of having a hard refresh timebox): https://github.com/ktoso/akka/commit/150a6a83fbce2f762b6d30f4987b31a01483056b#commitcomment-1381816 I think I'll agree on that - maybe the name of `refreshEvery` should be changed to better reflect how (where) the delay is instead?
leftover from the revert, will remove, thanks!
yes, better.
first test run on jenkins was successful, which doesn't mean that it's fixed, but anyway a good start
I'll keep it running
I'll fix review comments also
well, more precisely it is a fresh name every time you use it, and those names are guaranteed not to be referable to … it boils down to what viktor said ;-) (sorry, couldn’t resist)
I suppose so :)

I think I improved it (pushed)
The point of activity_renderers is that it would be easy to add an extension point for plugins to register new renderer functions for new types of activity (or override the existing ones). I'm not sure about this approach, I can see that it removes a lot of repetitive functions and template files, but these were very simple, and they were done like that for a reason.
in python it is good practice to leave a comma after the last item in a list, dict as it makes adding new items easier - I know this is evil in JavaScript so may make you feel ill.  I don't mind if you don't change this but just so you are aware.
I think we need to extract utility code out of Rack::File into Rack::Utils for path depth checks and readability checks, then use that here.
The AR relation calls.  That's why this ended up in this branch.

It is really a very similar idea.  It doesn't fully initialize the objects yet, it may only have loaded id's or the fields requested or something.
why was this test removed? What is the new expected behavior?
There are 1-2 places that still want the individual artists - the UPnP server I noticed can serve the individual artist names right. This is pretty temporary as well, later it will be replaced by a more fully functional artist credit processor somewhere. It doesn't really want to live here long term.
Good question, maybe the AE guys know... @gnif, @DDDamian ?
I misinterpreted your question.

The discrepancies are entirely due to the different shells, or possible lack of a shell. I don't believe Hadoop Streaming uses a shell at all.

Maybe I should have the local runner use `shlex` to enforce the lack of a shell so the runners stay consistent. If someone wants a shell, they can use `bash_wrap`.
was thinking out loud.   on 2nd thought, It think it's not relevant.
This should never be the case. Please drop the if statement and prepend cls._host (or with its new name cls._url_base) always.
document the meaning for this typedef.
Maybe make this lager_message.
I prefer such formatting:

    override def write(payload: ByteString): Boolean =
      if (gremlinAdapter.shouldDropOutbound(wrappedHandle.remoteAddress)) true
      else wrappedHandle.write(payload)
Good idea! Is it visible in the profiler?
So if `jsonquery` begins with `'.'`, we add `''` as a key into `keys`, is this correct?
Not needed right now.  Please remove.
{
    else {
Documentation needs to go here.
I see the issue. package.json and npm-shrinkwrap.json are not using the same version of xml2json. So, this pull req gets the right xml2json if I use npm 1.1.12, but the wrong one if I use an older version of npm. This needs to be fixed.

Could you also broadcast an email to the team that we need to redo shrinkwrap whenever we change one of our modules - for instance, engine needs to updated when compiler changes.
I'd prefer a templates directory within rails and then use the standard layouts/application and info/:action - that way if we add anything other than InfoController it can use the same directory structure.
I think that `assert_nothing_raised` is not necessary, there's already an assertion inside the block, and "nothing raised" actually doesn't do any assertion. Let it just :boom: in case of an error.
Is JavaScript that will be sent to the client side
Can we just remove this exception?  The `path=` method doesn't enforce it, so I'm not sure why we're checking in the constructor.
Your contrition is deemed sufficient.
Yes. I tend to forget that because the architecture of reflection pushes you so hard to use the extractors.
typos: remove "of the" and the comma
You should update the README doc.
Should this trigger an exception?  I was thinking that it should just silently return, knowing that it shouldn't execute the method.
i imagine that mpl is using this very direct check for a reason. this way there is no confusion that you have write perms as you actually are writing a file. i would prefer to leave it as is.
I guess splitting it as assignments and then rewriting will be better because its too long an expression.
moved to iterables and merged already
sorry about this. WIll fix it.
I don't know how to reference the outdated diff discussion... it's on the discussion page of the PR and is the only outdated diff there.
Well it will return whatever you put into the tuples ^^. It should be an int, I guess.
Raising a general question then: Is `is_Mul` always strictly equivalent to `isinstance(x, Mul)` or does it have a loose meaning that a object acts like a `Mul`?
remove "the order of" after tradeoff?
Why not just use the built-in Python `__abs__`?
Not sure, but I'd think that the default should be `True` here.  Default value needs to be added to docs.
No biggie, this is why we review :)
I would replace this line and the next with destructuring:

```clojure
(let [[fname args] ..
      [params &  body] args]
  ..)
```
all databases.
need the leading / or else browsers thing `Project` is a protocol.
You may want to look at render() instead of render_to_response() .. it is a new function in Django 1.3 (maybe 1.4?) that makes the syntax a bit easier.
It seems that 3.1.3 has been released: http://static.springsource.org/spring/docs/3.1.x/changelog.txt
This function should take a single argument of "options".
This white space is not part of ticket, but it should be here. This file was generated by develop/create_schema_files.php, which added that space. I guess someone edited that file manually when adding reported_post_text, which resulted in missing space.
In that case the body of this while loop should be put into a function (that is class method) that takes the post_id as input.
You can do if 1=0 for the else path.
Split onto multiple lines, wrap with {}.
+1 for "Do not use..."
That scans weird. Can you rename err to status.
I don't like both setValid and setRaw methods, they should be replaced/renamed. Ok, I will document this.
да, сега проверих какво точно се случваше и ето какво:
когато се извличат директно с sql заявката, се взимат времената както са си записани в базата, а през клас методите - май се взима предвид разликата в часовата зона на сървъра и клиента.
Т.е. ако часовата зона на сървъра е UTC а на клиента EET (+0200), то при добавяне на нов запис, времената се записват в базата с 2 часа назад (т.е. в UTC). После когато се взимат през клас методите отново се трансформират до часовата зона на клиента.
Затова първо с sql заявката излизаха с 2 часа назад, после като смених зоната, се оправи. Т.е. като се зададе time_zone-a на сървъра, същата като на клиента, този проблем не се вижда.
No description :)
don't worry about the send_foo methods - you should only have to implement the very basic send/recv/close and get/setsockopt.  The rest are defined in zmq.core.pysocket, which should be moved somewhere that it is importable from both cffi and core.  That was much of the point of separating out the sugar methods from the .pyx file.
ты про таблицу или ключ?
Would it not be better to merge the "password" and "input-type" options into one. Then in the php code test for input-type and if not found fall back to the password check? Would make things more readable I think for the user.
uPortal does setter injection instead of reflection based injection to make sure things are more easily testable. Can you add a setter for MessageSource and annotate that?
I'll have to figure out how I want to handle this skidrow, and wether I want to even support paths for cracked versions of games...I recognize the legitimate use of cracks, I just have to figure out how to implement.
This code is taken directly from the referenced bug.
remove comment. adds nothing.
I don't disagree. The editor did this automagically....
is -> are
operation -> operations
There was a typo, the "with" shouldn't have been there. I have rewritten the comment to fix the typo and some more explanation.
It eliminates a warning about assigning to a method argument variable.
I have negative tests (like column mismatch, unsupported keywords in union, multiple distributed tables) in the planner.TestUnion (together with the positive cases). Do you want to keep them separately?
'%s' % foo is ridiculous and unnecessary
Considering that thre are 3 different versions of them needed, yes.
But there are `or ''` in the lines below you touched. It's trivial enough to be fixed right away. If you'd rather do that as a separate pull request, fine.
What is on line 38 in this file?
This used to be problematic on some platforms that didn't set their DNS/hostname properly, most often after DHCP (you'd get a splitbrain where one component saw the configured hostname, and another saw the DHCP/reverse-dns hostname). Something to watch for on our testing platforms.
'ornament'. i had to laugh at that one :)
useless brief and useless `@param`
Should `Model.definition` be caps to be more consistent here?
The `_set_container()` method is used by `content` and `streaming_content` setters. It ensures that content is assigned consistently as a list (for non-streaming) or an iterator (for streaming). It closes file-like objects after consuming them (for non-streaming), or adds them to `_iterator_content`, to be closed after a streaming response has been iterated.
I'd prefer `let` instead of instance variables for `app` and `err`
Yes, jslint says "Expected an identifier and instead saw 'default' (a reserved word)." But I am pretty sure this has no practical consequence in any browser that d3 supports.
Just because it's not the source, doesn't mean it shouldn't follow the style guide LN1-6
Eh. It's just me picking nits. There is a really long line - goes out of view and the a single character line *shrugs*
Afaik `grunt.file.read` does this verbose logging already.
That's a dangerous expression (see [L506](https://github.com/PaulBRamos/jquery/blob/6fb68d22f4b136f0b467cf060ea3286f8cc6fcce/src/manipulation.js#L506))... I'm not saying it's wrong, but you should throw a lot at this to test for robustness.
This is not good. It causes many selectors to go through Sizzle that should go through QSA. rgroups may be valid for Sizzle, but it is not valid for QSA. According to the selectors spec, divisions should only happen on an unescaped comma (but we can test for a comma within quotes). If it then fails QSA, it should catch the error and revert to Sizzle.
All the functions in dt.sql, dt_preproc.sql and dt_utility.sql are internal functions. Thus, we intentionally do not generate the document for them. For these API interfaces in c45.sql_in and rf.sql_in, we will generate documents.
If you happen to know the appropriate section in the C++ Standard re: whether the template qualifier is really necessary (according to the standard), please post it.
This file contains the API of random forest. It also calls the tree growing engine, which is at decision_tree.sql_in.

After this pull request is accepted, we will submit another pull request and separate decision_tree.sql_in into two files, namely c45.sql_in and tree_common.sql_in. c45.sql_in is for the API of c4.5. And tree_common.sql_in is for the tree growing engine.
True, that should be done.
okay.. made it a little more cleaner
Ew, inline styles. I don't actually know if this is a problem though, especially something this short.
This also seems really similar to the code above, so putting it in a base class, too, seems like a reasonable thing to do.
Because it's incomplete and there is no reason to jump to 2.50.
Don't need to shell out to get the prefix, just `#{prefix}`
this is just a small thing but to be consistent with other places and with the github markdown style.
Probably want to remove the timeout.
It would be greate to have 'application_name' in the Messages, much less confusing than the current "transport_name" for everything. It's slow me down when writing this dispatcher trying to figure out how to write correct test. 

For me, it's also link to the issue 215 (document the Message) in order to spot what is missing or confusing.
https://github.com/praekelt/vumi/issues/215
I was not sure how to get out of the Exception Handling? Let me know if you desire something else.
AHahahahahhahahahahaha :D
I think we should reuse the convention of using dicts. (we were planning to port pipelines too).

It's true that priorities won't add much in in this case, but it's not only priorities what the dict mechanism provides, but also being to disable specific components of the (presumably most sensible default). Priorioties wont' add much to this (as they don't to extensions) but don't harm and it could be useful if someone ever needs them. We should make sure internal code in an intuitive behaviour. Like pre_process being called in priority-order.
Drop 'Dagger' from ClassloadingDaggerPlugin?
You mean the `JavaWriter`? Awesome!
This doesn't need the extra function call. It would work equally well as

```javascript
var me = this;
this._detectWeb(translate, function(translate, itemType) {
    if(itemType) {
        translate.translate(me._libraryID, false);
    } else {
        me._queryGoogle();
    }
});
```

The pattern above is usually only necessary if a variable is changing in a loop and you'd like to include it in a closure.
I think you can delete this comment now.
It's hand written
No need for the trailing `?`
Delete log. (I think this is leftover from my mess)
Is this change still required? .resizing doesn't get applied as a descendant anymore.
Instead of overriding position "FIRST" and "LAST" and looking for "sectionMarker", I think it might be better to add new relative position constants something like "FIRST_IN_SECTION" and "LAST_IN_SECTION", and give normal IDs to the dividers. This seems like a simple API and also simpler code.
I see -- it was just the way the diff mapped the lines. Anyway, all parameters and return values need descriptions.
"global" not required here
You need to remove line 72, 77 - 83. Line 75 has to be moved to the event handler.
hm, not really. I'll remove it, thanks!
Isn't it odd to call a class method like this? You're relying on the underlying assumption that the first argument isn't a dependency
Put this on one line.
linq here?
`foreach(var angle in clip.angles.Where(a => !types.Contains(a.angleName))){
types.Add(angle.angleName);
}`

or even just:
`types.AddRange(clip.angles.Where(a => !types.Contains(a.angleName)));`
Same. It should be limited to possible values. You could also do our own field type too.
Extra spacing here to keep our style consistent please. It might be terrible, but at least it's (mostly) all the same.
no double-quotes around "0", should be 0
I think it's better we do it this way because it's a better user experience to have some sort of notification happen that the value they entered was bad VS the browser just switching it to the cap. Besides, we do this same thing in the twitter editor.
This will only work once. Can you increase the number at the end for each crash?
No, the DOMMouseScroll event doesn't pass back values that need to be modified. In other words, smooth scrolling out of the box.
Could we break this down onto multiple lines for readability?
nicer formatting to write as:

`numCallbacks ++;` :)
you can just reference this as a truthy since undefined will always be false, e.g. if (body.layout || body.layout.row || body.layout.col)
I was hoping we could get the abort() fixed at some point and just leave that in there.  Think it would be better to just call done() for now?
there is no `undefined` literal in JavaScript afaik
日志格式
Use the indicator "%v" ? ngx_http_variable_value_t is the same as ngx_variable_value_t.
Can put these sentences in separate lines. Will make it a bit easier to read
darzemn axe 48px e na 480px
why data is generated only for rootDirs[0]? Can you generate for all rootDirs and baseDirs?
You are right. I was too focused to getting the matrix scalar operations working through the AbstractMatrix code. Assignment should convert the to normal Matrix.
Do we plan to update the user create and user edit to support also setting of  'Default Org on Login'?
I would probably assign a variable here and then call `Bundler.require(*bundler_groups)`. That would seem cleaner to me and it would fit better with our style guide which recommends using parentheses for method calls that have arguments.
`simplest`
there should be a Conf variable already for the sites pipeline dir
More generally, can you make sure this file properly pass the style-test of `gjslint`?

https://developers.google.com/closure/utilities/docs/linter_howto
Here and elsewhere, the type shouldn't be in parentheses (I know it was already incorrect before this PR, but if it's easy to do, maybe it can be fixed here?)
bash isn't a csh variant in any practical sense :)  And it isn't necessary on bash.
Wikipedia is actually citing a textbook here.  The full citation is McCarthy, D. D.; Seidelmann, P. K. (2009). TIME—From Earth Rotation to Atomic Physics. Weinheim: Wiley-VCH Verlag GmbH & Co. KGaA. pp 218–9.

I think otherwise it's easiest to link to Wikipedia for this simply because it's not *going* anywhere.  Although it largely depends on the site, links to scientific publications tend to be a bit more ephemeral I've found...
This sort of un-metered, un-monitored spawning makes me uneasy.  I know one goal of these changes was to remove serialization on a `gen_server`, but it seems like finding the process with the large mailbox will be easier than sorting out thousands of throw away processes, while troubleshooting a node running low on resources. I suppose the key questions are, how often do we expect this function to be called, and how often might it be called in the worst case?
Didn't realize I was creating composite indexes here :-/. I'll refactor
into independent indexes...noob mistake.

*Nikhil Goel* | Vanderbilt University, 2013 | Computer Engineering,
Economics**

LinkedIn <http://www.linkedin.com/in/ngoel1> |* ngoel36@gmail.com | ( (256)
503-9183 | NikhilGoel.me <http://nikhilgoel.me/>



On Mon, Nov 19, 2012 at 1:54 PM, Luke Steensen <notifications@github.com>wrote:

> In db/migrate/20121114025229_create_users.rb:
>
> >        t.timestamps
> >      end
> > +    validates_uniqueness_of :name
> > +    validates_uniquness_of :email
> > +    validates_format_of :email, :with => /^([^@\s]+)@((?:[-a-z0-9]+\.)+
> [a-z]{2,})$/i
> > +
> > +    add_index :users, [:name, :first, :last, :email]
>
> What is your reasoning for this?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bitcast/bitcast/pull/11/files#r2171835>.
>
>
@Diapolo adding IMPLEMENT_RANDOMIZE_STACK to CreateThread would probably be best for another pull request.
I will create an additional commit for this, take a look and then we decide if we leave this as is or use the version after the commit.
you should keep ``@inheritdoc`` and write the phpdoc in the interface
The plugin only gets loaded when called inside the module javascript.  f.e. in locale.js:
`$('.dataGrid td.translationValue').inlineTextareaEdit({params});`