labofsahil/github-event-dataset-2012 huggingface dataset:
lalg1 Zusammefassung hinzugefügt, algd1 Anpassungen
Master

Made messages list private, and added respective methods.  
Changed default byte values in addMessageForm to 00
removed resizing from addMessageForm
Add minimum stability to beta as 2.1 is in beta
Replaced similarity nodes by styled edges

Hi,

I tried your add-on and really liked the idea, so I took a few minutes to play a bit with it.

I removed the similarity nodes and replaced them with bigger edges with a tooltip, it better conveys the relation between the passwords.

I wanted to add a key, and a border to the visualization so you had visual feedback on what is happenning when nodes go out of the viewport, but I feel a bit too lazy today :)

The code might be ugly as I am really not used to javascript and I might have added stuff that do nothing (and forgotten to remove them).

Best,
Charles.
Beatification

Read the comments in the diff view
Changed file headers to standard as described in Reconfingurable Equiple...

Updated the file headers to the standard as described in the Reconfigurable Equiplets PTD.
Update some values in - res/values-iw/strings.xml
Small typo fix for cod igniter completions

Hi, I found a minor bug with the db->delete completion, closing } was missing meaning the completion didn't work.

Not sure if you want pull-requests for fixes, so feel free to ignore, I have fixed in this fork though.

Thanks.
Added fields for first_name, last_name, company, and phone.  Also added ...

...default data for initial insert. Brings migrations script inline with sql script.
OAuth documentation for github

This is the same documentation as published on the developer site. 
However, it clearly indicates that the developer site must be used as the primary source of information.
Although this gives good performance, let's try using minimum requiremen...

...ts for framebuffer for LCD,  HDMI (lets try 720p for less framebuffer).  Plus let's blame Viewsonic for giving us only 512 :)
Tested creation of new corpora and switching between them
Pull Req: Next refactor build multi targets

Refactoring of code that builds BARs for multiple targets
Also fixes GHI: #20 - "BAR packaging complete" message appears even though there is a Native Packager exception
Keep the server running if the bug isnt fixed...

This will close the connection to the person with the bug, and hopefully continue running...
Ma partie du gros fichier, lignes 1 à 850
Add github ribbon

Adds a Github badge to all pages of the website for street cred.  [Here's what it looks like.](http://i.imgur.com/3ZFqJ.png)

There are [other color options available](https://github.com/blog/273-github-ribbons).

It could attract more contributions.  I won't mind if this pull is rejected. I was mainly trying to learn how the site is generated.
Don't show border below last field or if there is only one field.

Also stops the border-bottom style from being applied to nested tables (IE with tinymce).
The point is that dat.gui can attach to it - it can only attach to
properties. We have getters for the most functionality like
X.object._normals can be requested by X.object.normals()



On Fri, Feb 17, 2012 at 10:28 AM, Nicolas Rannou
<reply@reply.github.com>
wrote:
> @haehn Shouldn't we use "getters" for those local variables then? (rather than static text?)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/xtk/X/commit/6454937530b8c3c633c7796ee7e8dafdfdb05cc6#commitcomment-978379
SGTM

Evan Winslow
http://twitter.com/evanwinslow



On Wed, Jan 25, 2012 at 10:02 AM, Brett Profitt <
reply@reply.github.com
> wrote:

> Agreed, but see discussion at http://trac.elgg.org/ticket/4311 and
> http://trac.elgg.org/ticket/4327
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Elgg/Elgg/commit/2e499d0f1e62267105dbe1ff2457cd6098765983#commitcomment-901284
>
Should this code be removed? (I may be commenting on an old commit.)
Hi, Isn't something like Dir.: %s, Vit.: %i %s would fit ? Just asking. If not we can change to the shortest you suggested.
are you a wizard
ok agreed.
sorry to be so picky: i like the feature, but it should be possible to do this without having to construct this dict every time you want to compute this index integer, which is done for every header in the message if i see this correctly.

why not read the setting in `HeadersList._build_lines` at the very end, and sort the constructed headerlist in one go according to the setting list? sth like:

```
newlist = []
for key in setting.get('sort_order'):
    if key in headerslist:
        newlist.append( # key value, as constructed widgets
        remove pair from headerslist
# add rest of the headerslit to the bottom of newlist
return newlist
```
To ale může být BC break, pokud se využívá generovaná hodnota ID ve vlastním javascriptovém kódu.
I was thinking about dialog. but hime-setup is not a "dialog", it is a utility to setup hime, create words, and import data. seeing the meaning of this program, it is quite reasonable to think it is a "utility" rather than a "dialog".

besides, according to your link (thanks for the link), " _NET_WM_WINDOW_TYPE_UTILITY indicates a small persistent utility window, such as a palette or toolbox. It is distinct from type TOOLBAR because it does not correspond to a toolbar torn off from the main application. It's distinct from type DIALOG because it isn't a transient dialog, the user will probably keep it open while they're working. Windows of this type may set the WM_TRANSIENT_FOR hint indicating the main application window." it really should not be a dialog.

for the two reasons, utility should be the best choice.
Curious to know, any reason why the last newline?
Ich habe die Einbindung noch mal ein wenig verändert, damit die Rückwärtskompatibilität erhalten bleibt:

* Standardmäßig ist nur MooTools aktiv
* jQuery wird vor MooTools geladen, damit sich `$` in jedem Fall auf MooTools bezieht.

Damit laufen alle bisherigen Skripte ohne Anpassung weiter. Übrigens zum Verständnis: Es würde vollkommen ausreichen, wenn nur jQuery im "no-conflict"-Modus liefe! Denn MooTools wäre ja dann das Skript, mit dem es "no more conflicts" gäbe :)

Ich habe die Core-MooTools-Skripte nur der Vollständigkeit halber auch angepasst.
Yes, it is ready.


On 1/23/12 1:37 PM, "Mark Grondona"
<reply+c-893807-894bbb5c697bb4ad3a290fdc7a88b978ed929cc6-743906@reply.github
.com> wrote:

> Nice. Is this ready to pull into the chaos slurm branch?
> 
> Unfortunately, looking upstream (in schedmd/slurm-2.3 repo) it appears
> a bunch of this has already been merged, in a lot of separate commits?
> 
> mark
> 
> On Mon, Jan 23, 2012 at 8:57 PM, pdesr <
> reply@reply.github.com
>> wrote:
> 
>> Mark,
>> 
>> Don walked me through the squash procedure this morning.
>> We discussed it, and because all changes are relevant
>> and none stand alone, we made it as a single commit.
>> 
>> It can be found under:
>> 
>> https://github.com/pdesr/slurm/commits/sbatch-export-file
>> 
>> Phil
>> 
>> ---
>> Reply to this email directly or view it on GitHub:
>> 
>> https://github.com/pdesr/slurm/commit/9ff604a8c5c4e15659f878c7e1a6f1802851597
>> 7#commitcomment-893668
>> 
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/pdesr/slurm/commit/9ff604a8c5c4e15659f878c7e1a6f18028515977
> #commitcomment-893807
Yesterday I came up with using *TemplateThisParameter*.
Yes, a huge performance penalty as Stage() always writes the index to disk, which I don't need as long as I am not about to commit (every 60 seconds) see Flush()
Just checked the benchmark.  Does not seem affected by the change.  That iterator seems to be only used during compactions.
It actually proved a lot simpler to implement call/cc than I first expected. Every function application generates a continuation (which is used by the subsequent ret instruction), and all that ccc has to do is put that current continuation onto the value stack. Then, all that was needed was to just provide a mechanism for applying a continuation with an argument, which basically meant restoring the continuation and then placing the value that was applied to it in the value register (making it look as though ccc had returned with the value applied to the continuation).

Of course, the pesky protect function (which is essentially Scheme's dynamic-wind) creates complications. Instead of just applying the continuation as before, I now have to go up the continuation list and execute all the registered protect functions until I get to the continuation I'm supposed to apply. I've heard of the Hanson-Lamping algorithm for implementing dynamic-wind which is supposed to be more efficient but I've so far only seen vague references to an unpublished manuscript. *shrugs*
Looks like this commit caused a regression in Firefox 3.6 of all places. @rwldrn can you take a look?

http://jsfiddle.net/79gBB/1/
You're missing 
```case ENTITY_SHOOT_BOW
            return new EventExecutor() {
                public void execute(Listener listener, Event event) {
                    ((EntityListener) listener).onEntityShootBow((EntityShootBowEvent) event);
                }
            };```
in JavaPluginLoader. Otherwise there was no need to add it to PlayerListener.
Dear Charles,

Up to you to do what you think best both in terms of code clarity and 
performances. The best place where to put the if (Platform.IS_OPENVMS) 
remains the first lines of GetUnixPath.getUnixPath(filename).

However if such an addition is used and as I wrote you that junit does 
not seem to correctly work under OpenVMS, you will not be able to write 
test cases testing VMS file syntaxes running on both Linux and Windows. 
This may be bring a hole in your wish to run JRuby as green as possible 
as you told me you wish for JRuby.

Best,
Philippe

Le 15/02/2012 20:10, Charles Oliver Nutter a écrit :
> Can we put a platform check in here and not even bother calling the VMS logic? Use org.jruby.platform.Platform.IS_OPENVMS.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Vouters/jruby/commit/56f8cf54b83142114bc810beff8a2edae34132ee#commitcomment-970791
>

-- 
Philippe Vouters (Fontainebleau/France)
URL: http://vouters.dyndns.org/
Esto no funciona porque el motor de Haml no sabe qué es javascript_include_tag
But mappers and modders have for a long time included *some* version in the filename in order to purposefully not overwrite older versions.
Mazze, controlla di avere la versione che faccia così...
I think you're right about the source of the problem. I don't know offhand if giving x a default value would have any negative consequences. If you want to give it a shot, first run the battery of tests in `inst/tests/visual-dotplot.r` and save them to PDF or PNGs. Then make the change, re-run the tests, and compare to see if there are any differences. If it all checks out, it would be a good idea to add another test that checks the case where `binaxis="y"` and no `x` mapping is specified.
Some consistency in the naming would be nice here -- if you're calling the parameter probe_multiplier, should probably call this probeMultiplier too.
The incremental search behavior is a little strange:

1. Searching backward with ? pages to the end of the buffer even if the position had been somewhere else before searching.
2. If the last match is in view, less rings the bell on every subsequent character typed, even when the resulting search string still matches.
3. Deleting a character from the search string:
  1. Rings the bell
  2. Briefly displays extra characters at the end of the search string
  3. Deleting the last remaining character of the search string leaves matches for that character highlighted.
  4. After deleting the last remaining character of the search string, two more deletes rather than one are required to cancel the search.
Why is it required to set `QtTesting_INSTALL_DIR` ? It should be sufficient to set `QtTesting_DIR` and then have `find_package(QtTesting...)` loading QtTestingConfig.cmake ...
同上，也要改
Убери мейн и лучше создай тесты в тестовом пакете
Confirmed. Not sure if it's the database or some random horrible updating from the core. Regular platform works 100%, and I even see the other platforms before slipstream, but after I use it, everything disappears.
Without really knowing a great deal about inline functions, how is removing static and keeping the function in the source for Lion better than allowing Lion to provide it's own version?
Normally, I think installing into an etc based app-defaults directory is  
the right thing to do, because it suggests the editability of these files.  
We do have to be careful that our work doesn't overwrite the normal  
app-defaults files of any programs that might be named the same though.  
It's a little more complicated if we are going to assume that we are  
installing into a system directory, but being able to choose where the  
app-defaults files go separate from the rest of the shared code is  
probably a very good thing. If we do that, we need to make sure that  
scripts like dtsession and others that may be setting or controlling the  
resource directories are appropriately modified.

-- 
Aaron W. Hsu | arcfide@sacrideo.us | http://www.sacrideo.us
לֵ֤ב חֲכָמִים֙ בְּבֵ֣ית אֵ֔בֶל וְלֵ֥ב כְּסִילִ֖ים בְּבֵ֥ית שִׂמְחָֽה
Delete "the"
Just FYI, I was able to transition from Socket.IO over to BrowserChannel without too much trouble at all. Have been testing for a day or two now and haven't found any bugs.
At first, this is what I had done. But then, I needed to return this, because:

this is how it was implemented before in the TrackEffectAdded, pitivi/undo/effect.py, to pass this to tl_obj_track_obj:

https://github.com/kyotobay/pitivi/blob/undo_redo/pitivi/undo/effect.py

l. 117 and l.192
Это меня глючит, или в мастере mmap ? oO
This fixed #1466, and probably fixed a few other things, too.
Does not work. Hs been reverted. If you want a real fix, that is more longtem look at http://trac.xbmc.org/ticket/12768
Thanks @tshinnic, fixed!
logging in?  It's only one line here - it just didn't work because whoever wrote the test didn't pass in the right credentials.  Not sure how abstracting it further would help.
Creating unnecessary garbage by creating an empty array to get at Array.prototype.slice.

Should just do slice = Array.prototype.slice at the top with get and set.
How about something like this instead:

```php
$this->load($this->uid ?: $this->database->lastInsertId());
```

Line 66 seems to make sure $values is _never_ false. Am I missing something here?
для булевых значений используй true и false
Don't include this line.
Wouldn't it be better to implement `visit(ExistsCondition)`? That way you can also evaluate stuff such as `print(exists x else "no X")`. This way you don't have to do anything special in the if statement
Shrug.. I suspected that we might have a fair few cases where we have arguments over whether we should "improve" the engine or provide the same behaviour as the original, so this flag allows us to avoid anoying people by forcing a choice on them. (Though we can still enforce a preferred option by the default).

Previously in other engines, these have mainly been bug fixes and fairly uncontroversial i.e. very few fans/players want the crash/gfx foulup of the original, so this kind of boolean is not warranted, but it gets trickier when you are making more "aesthetic" decisions, which some fans want the new and improved and others want to play it as the original (Yes, I know they _could_ use DOSBox for that, but you know my point) so you need to provide a config flag for this.

A good example where this has been done previously is the SCI dithering/undithering config option.

Basically I would prefer this as a config option, but this is up for discussion.
Is there any special reason why this isn't using src.width() and src.height()?
The network graph does look wacky.  I did branch from master and it does appear to have the right parent.  Odd.
Good to go now. It must have been the cached files. Thanks for the tips, guys.
one space after the `=`
About as easily as I missed it. =)
I haven't reviewed the code, but we should only pass user id to web service calls when there is a scenario to use the web service method and pass in a different id than the caller.  Even if we allow that, we may have a config that only allows admins or managers to do such thing, so there may be some authentication / authorization involved.
default behavior kept
I think you need to check for `response.status === "success"` here
one of those.
Added 'usrname' as new value in $data, so it can be accesed later in mtxcsmo.php
These should be passed as separate args:

    system "./configure", "--prefix=#{prefix}", ...
Are you looking into it? I will have to fix it in the next couple of days anyway, might as well submit by fix. Unless you beat me to it :)
In what context is "kosher" used, anyways? In english I know it's another way of basically saying "okay". If that is how it is used, neither kosher or halal would work in swedish localization, and a new translation should be found.
True. (ha) But I thought leaving that code there for the future will be useful.
This is how github social features work. People with access to the project get notifications when commits are made, comments are added, etc. We've also setup an additional email hook so that an email is sent to productx-dev when these same events occur.

 @jeff-at-artefact maybe we don't really need the redundent email hooks that send mail to productx-dev? I think we all get email directly (and IRC if you are in the room) about check-ins, comments, etc. Something to consider as a low-pri thing.
Just a few lines ahead you can find
.nav-tabs > li > a,
.nav-pills > li > a {
  ...
  line-height: 14px; // keeps the overall height an even number
}

I think ist is useful to remove the line-height there and put it in a new selector:

.nav-pills > li > a {
  line-height: 14px; // keeps the overall height an even number
}

What do you think?
i say you two do not use this to chat!!
 2012-1-8 1:44"bigeagle" <
reply@reply.github.com
>д

> Do not send Chinese characters via email!
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/xdlinux/xdproject/commit/d967864e37647aea72f333bccb30c191c360d124#commitcomment-847164
>
调了一下代码格式。然后把一个条件移动到子查询内了。
Aldo, why are you bundling the opentok library instead of using the opentok gem? http://rubygems.org/gems/opentok
[Constructor(DOMString type, optional CloseEventInit eventInitDict)]
interface CloseEvent : Event {
  readonly attribute boolean wasClean;
  readonly attribute unsigned short code;
  readonly attribute DOMString reason;
};

dictionary CloseEventInit : EventInit {
  boolean wasClean;
  unsigned short code;
  DOMString reason;
};
+1 to this approach.  I believe it makes the API cleaner and more true to the name of ArchiveEventHandler#handle, and we can still get at the handled event via ArchiveEvent.
These:
http://paste2.org/p/1885619
http://paste2.org/p/1885647
I think they are connected ... and lead to one problem there ... All Venugh was trying to do is to remove possibility function checks both Null and 0 cases which can simply happenbecause code is programmed by many programmers and someone could define something different so this function (according to me) was just to make sure this is not the problem ...
FYI: when writing doc comments, please use proper capitalization and punctuation. This would look confusing when compiled to HTML.
eitanb :

You are right, my mistake. Sorry for the noise

Boaz
`Qt`, not `WX`
Yeah. I feel the same. I tried to minimize the code duplication to one method (`timestamp_attributes_for_table`). I was thinking in extract these methods in another module in include in the both places, but the context is different.
`Thus` ?
This needs to be FIXED!
Great job ! Thanks
I know, I thought doing this might be a bit more intuitive
Are you saying that I should remove the test completely or make the test better?

Bill

On Jan 13, 2012, at 5:14 PM, Adam Vandenberg
<reply@reply.github.com>
wrote:

> Don't really need this if this is all we're testing.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/billmoritz/homebrew/commit/8fad406ab8ae94825f811730d63f2b9991eb0cb7#commitcomment-866321
Actually 1.0/0.6 is correct

I'll pull and use git rebase --interactive to undo this
just noticed this, will have to figure out a way to build up the symlinks from more than one recipe.
According to the dev guide: http://developer.android.com/guide/developing/projects/projects-eclipse.html
"Under Target, select an Android target to be used as the project's Build Target. The Build Target specifies which Android platform you'd like your application built against.
Select the lowest platform with which your application is compatible."
This is different as you mentioned as well with the property for minSdkVersion
"Enter a Min SDK Version. This is an integer that indicates the minimum API Level required to properly run your application. Entering this here automatically sets the minSdkVersion attribute in the <uses-sdk> of your Android Manifest file. If you're unsure of the appropriate API Level to use, copy the API Level listed for the Build Target you selected in the Target tab.".
I think thought that both should be the lowest version that the app should run against. in our case 1.5 (android-3)
Sidenote: I think the whole confusion came from that fact that I am using Intellij Idea Community version (free) for development in which I don't remember finding all the settings for android development as exists Eclipse. I will take a look at the options again but I am suspecting that this is the case.
Was it meant so, that the files in /fixtures are the ones we're testing with jasmine against?
Looks good now, thanks.
This is not as easy. All the old comments which are using StringProperty should be migrated to TextProperty first and then we can apply this.
Nope, i didn't know that :)

Tnx!

Martijn

Op 6-1-2012 13:58, jpf91 schreef:
> You probably already know this, but in case you don't:
>
> All libev watchers have a 'void* data' field, which can be used to store an arbitrary pointer. You could store your clientConnection* in there.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/MartijnB/FlashPolleD/commit/3e07c2aff8b41ef581f3362e1910d9d6cd5ad834#commitcomment-843593
@blafoo I noticed it just looking at your commit.
I haven't seen that page before, but if I'm on there I will skew your map and reveal myself as not as a software developer in real life (unlike everyone else) ;)
Moved to c#
oh, this is pretty nice :-)
I agree with KingCrunch, but with a different point of view. Suffixes and prefixes are project-centric, therefore PSR-1 should not cover this. The project must decide if it's going to use a certain prefix or suffix. This is obviously a personal preference.
Problem with Github. Sorry
The compile. Since I packed the Inveotry listener function into the Player listener (possible with the new Bukkit Listever system)
Yup, it works. Just ran through a nice `clean install` with no issues.
@darcyparker what does makeprg actually get set to on windows without your modifications? I want to know why single quotes arent working as this may have implications for other syntax checkers on windows.

I currently dont have a windows machine - otherwise i'd get in and do it myself
Not if you make an integration test with `WriteLocalCounterBulkTest ` see https://github.com/framiere/rt-analytics/blob/master/processor/src/test/java/org/openspaces/bigdata/processor/WriteLocalCountBulkTest.java
I'm not entirely sure how to interpret that--given that there's no existing code to make this simpler right now, do you want me to change anything?
michalpolko... I will gather a few friend and test it all...u will have a full report soon inoff.
我的问题在于我的html中包含了很多单引号和双引号,如何在字符串中再包含这些引号和斜杠.我在mongo  sh里试过,单双引号不能嵌套相同的引号.-- Sent from my HP Pre3On 2012-3-9 01:35, Lin Gui <reply@reply.github.com> wrote: document是指一条记录, mongodb的说法, 类似于sql数据库里面的一行.html就是字符串类型啊, 作为字符串存入数据库即可--- Reply to this email directly or view it on GitHub:https://github.com/guileen/node-mongoskin/commit/d65cf6b9d91f575f58d3d040cdf9b9183da8f7be#commitcomment-1062058
Oh fail.
Ruby 1.9.3 + Rails 3.2.1 - variable body were nil, response.body is ok.
He hasn't chosen yet. You should ask him to hurry and find a good graphics library.
well on Dalaran Sewers you can use charge from the bottom to the top of the thingy (lol idk how to explain you should know i think).

and

on The Ring of Valor if you use charge if someone is standing on the things that go in the air then you fall down that hole.
Yes- immortal if negative

Sent via the InterWebs

On 20 Jan 2012, at 12:53, tristantarrant<reply@reply.github.com> wrote:

> Are values of lifespan < 1 valid ?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/maniksurtani/infinispan/commit/f53958a265dc3dcc8ccd0617c1146d90712ff8c1#commitcomment-885035
<moment of silence for the passing of mosinstall />
It doesn't have any errors anymore. Well, not console errors.   Can't create an arena, though.

Check into this: https://github.com/Betterverse/MobArena/issues/1
Why do you comment old code?
No problem. Ip is there!
Flags3 & Flags4 -> float
How about blog post about 'minimal ORM'? Or at least can you post update on http://code.google.com/p/dapper-dot-net/issues/detail?id=6
So, just run it. It's suggestions looks reasonable.
Need to mach both _something_type_ and _type_ columns

```ruby
scope.match(/(^type|_type)$/)
```
how can I download this one?
Probably some override marker would be enough. Mostly I wanted to save others from wondering about all those empty methods and the current "double" conversion handling -- updated those comments not to say the obvious and to make it clearer that we're countering some SQL-specific conversions done by Django here.

Later on it would be best to actually get rid of those `value_to_db_*` / `convert_values` methods and move their code, as well as `Field.get_db_prep_lookup`, to `BaseDatabaseOperations.value_for/from_db` or something similar, so it would be easier to override. I can think of two possible solutions here:

* Require `get_db_prep_save/lookup` to call super method at the end and call back-end conversions from there; quite bad because it would require updating many custom fields;
* Make back-ends only call conversions on `DatabaseOperations` and call the field methods from it; also not so nice because it would require changes to all SQL back-ends (albeit probably small changes).
Louis, in working on finder we learned that we had to account for more than Chinese with this, it is actually about 7-8 languages including Japanese and Thai. I think I'll break it out into a separate method the way you have, and if you would like me to send a pull request for the others I'm happy to do that. We also need to handle the different ways language may be recorded for this in that in many instances language is attached to individual items and they need to .
Bonjour,
J'ai installer votre version française de imdb plugin mais il ne reconnais rien lorsque je lui demande quelque chose, quel que sois la question, Dois-je voir, quand est sorti, qui a joué.....
I did not create this code.  Just copied from CM7
rnewman: can you verify that calling back about these errors is the right thing to do?
This comment says "single web role", but it's actually multiple roles.
Edowar, have you ever got this error, I have tried almost everything:
In file included from frameworks/base/include/ui/PixelFormat.h:33,
                 from frameworks/base/include/surfaceflinger/ISurface.h:28,
                 from frameworks/base/media/libstagefright/AwesomePlayer.cpp:48:
hardware/libhardware/include/hardware/hardware.h:24:29: error: system/graphics.h: No such file or directory
target thumb C++: libstagefright <= frameworks/base/media/libstagefright/WAVExtractor.cpp
target thumb C++: libstagefright <= frameworks/base/media/libstagefright/avc_utils.cpp
In file included from frameworks/base/include/surfaceflinger/ISurface.h:28,
                 from frameworks/base/media/libstagefright/AwesomePlayer.cpp:48:
frameworks/base/include/ui/PixelFormat.h:61: error: 'HAL_PIXEL_FORMAT_RGBA_8888' was not declared in this scope
frameworks/base/include/ui/PixelFormat.h:62: error: 'HAL_PIXEL_FORMAT_RGBX_8888' was not declared in this scope
frameworks/base/include/ui/PixelFormat.h:63: error: 'HAL_PIXEL_FORMAT_RGB_888' was not declared in this scope
frameworks/base/include/ui/PixelFormat.h:64: error: 'HAL_PIXEL_FORMAT_RGB_565' was not declared in this scope
frameworks/base/include/ui/PixelFormat.h:65: error: 'HAL_PIXEL_FORMAT_BGRA_8888' was not declared in this scope
frameworks/base/include/ui/PixelFormat.h:66: error: 'HAL_PIXEL_FORMAT_RGBA_5551' was not declared in this scope
frameworks/base/include/ui/PixelFormat.h:67: error: 'HAL_PIXEL_FORMAT_RGBA_4444' was not declared in this scope
In file included from frameworks/base/media/libstagefright/AwesomePlayer.cpp:48:
frameworks/base/include/surfaceflinger/ISurface.h:84: error: 'HAL_TRANSFORM_ROT_90' was not declared in this scope
frameworks/base/include/surfaceflinger/ISurface.h:85: error: 'HAL_TRANSFORM_ROT_180' was not declared in this scope
frameworks/base/include/surfaceflinger/ISurface.h:86: error: 'HAL_TRANSFORM_ROT_270' was not declared in this scope
make: *** [out/target/product/triumph/obj/SHARED_LIBRARIES/libstagefright_intermediates/AwesomePlayer.o] Error 1
make: *** Waiting for unfinished jobs....
target thumb C++: libstagefright <= frameworks/base/media/libstagefright/string.cpp
target thumb C++: libbinder <= frameworks/base/libs/binder/Binder.cpp
Did you update your .config with the new kernel parameters? The change shouldn't have altered the code being used by the kernel at all. Basically, I found out that the patches from Angstrom had been integrated into the upstream git branch and were no longer needed. I'm currently using the beaglebone_uclibc_defconfig for my builds and it seems to work fine. Maybe give that a try?
Syntax Error here!
Why did you choose such a name?
Als je HTTP access control implementeert op de server kan je in de browser
onder de same-origin policy uitkomen. Ook voor request headers:
https://developer.mozilla.org/En/HTTP_access_control#Preflighted_requests

2012/2/5 Peter Kleiweg <
reply@reply.github.com
>

> Je wilt een webapplicatie voor alpinocorpus maken. De accept-headers
> worden gezet door de browser waarin de applicatie draait, niet door de
> applicatie zelf. De enige mogelijkheid, voor zover ik weet, om die headers
> te veranderen in de applicatie, is vanuit JavaScript, met een
> XMLHttpRequest-object, maar om beveiligingsredenen is dat alleen toegestaan
> als het request naar dezelfde server gaat als waar het script vandaan komt.
> Dus.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/rug-compling/alpinocorpus-python/commit/49d790323010f5aef151d057048b6e9ed90703bf#commitcomment-934715
>
*panels* instead of *pannels* nop ?
OK @rgrp - thanks for the info.
this is wrong. You cannot ensure you will support 3.6.9 (which matches your constraint). You need an upper bound (you could use 2.1.* which is equivalent to ``>=2.1-dev,<2.2-dev``)
Ah I'm sorry I've fixed the formatting and the whole file looks like changed :/
ok all right
что за "а"?
if this is in the abstract, will it get run for each class that extends it? (sorry, I can't recall)
I get it now and I don't see a better solution handling UI concern.

I don't see any new test in the pull request, I think such a major change deserves it : can you add some test(s) to verify the behaviour with username set to :
 - null
 - "(default)"
 - a real attribute name ?
This looks good.  But can we make these service instead of svc ?
We want to allow passing `None` for `data` of a `Dat` because that's required for temporary `Dat`s. Accessing the data should then fail i.e. we could cause the `data` property to raise an exception in that case? Want me to fix that?
This looks like the function will accept something other than AoS data - if this is the case, the name `_aos_to_soa` is a bit misleading.
However, you may or may not want to close over the context
I feel that there is a word missing between "testing" and "is", like "library"/"kit" or otherwise.
Why was this moved out here?
Ah, no It don't. There is a missing deserialization step there  
(deliberately) which will be included when I split the protocol buffer  
definition of payloads.
since you use .isDefined combinded with .get

    sslHandler match {
      case Some(h) =>  // h
      case None =>
That's strange.  Can you check somehow which script you are calling there?   Maybe I should add that as output diagnostics...

What happens if you call:

    nipy_diagnose /path/to/source/nipy/testing/funcional.nii.gz 

Do you get the ``npz`` file?
I don't know what to do with this method. Remove, or leave (maybe it will be useful in future?)
When testing the remaining issue, I could see that something was up with these messages.  dep.name should be renamed by the time it get here, but I think it isn't.

Seems like it would be some sort of ordering issue, but nothing around that changed.
Both would make sense and I was waiting for more stability before going into that kind of optimization
what if reqTime < lastTime?
No need to cast. Just return "";
why are you importing this here?
encapsulate (private or package private).
Consisten naming: heapOfElders or eldersHeap or even a more descriptive name
The tsoclient is used outside of this class. It should be private and made accessible through a getter maybe. With this change some tests fail due to a NPE
need? no, could have? yes
Okay, I'll revert this back. It simplified the check in master.py That's why I defaulted this to 0.
Hmm, this is totally unacceptable for performance reasons. Converting the QPixmap to a QImage and then making another copy just to tweak the alignment. Instead, this should be implemented by the `TileDelegate` in `tilesetview.cpp`.
Here perhaps 'MakeHardlinks'
>
> There will be loss of precision here, because doubleValStr came from a
> double, and there is not a distinct double value for every long.
>
Yes, but there is no long in javascript anyway!   So the precision was lost
at the JS eval and not in the above code. :)


> However, none of this matters because it looks like Errai's JSON parser
> (server-side) presently treats all JSON numbers as doubles. GWT's JSON
> parser (client-side) delegates to JSON.parse() or eval(), so it's all
> doubles there too. If there was more precision on the wire, it will be lost
> in either case.
>
> Given all of that, what about just returning o.isNumber().doubleValue()
> unconditionally? It's simpler and shouldn't result in any loss of precision.
>
Makes sense - but could be annoying to work with, ie requiring a conversion
later.

My code was inspired by SmartGWT's way of doing this 'guess' conversion.
 I'm still not sure I like that approach either.  There is an element of
surprise/bugs.  But in practice you should probably be binding your JSON
value to a specific Java type - rather than relying on the ObjectMarshaller
to guess the time.

                if (objType == 'number') {
                    if(obj.toString().indexOf('.') == -1) {
                        if(obj <= @java.lang.Integer::MAX_VALUE && obj >=
@java.lang.Integer::MIN_VALUE) {
                            return
@client.utils.JSOHelper::toInteger(I)(obj);
                        } else {
                          return @client.utils.JSOHelper::toLong(D)(obj);
                        }
                    } else {
                        if(obj <= @java.lang.Float::MAX_VALUE) {
                            return @client.utils.JSOHelper::toFloat(F)(obj);
                        } else {
                            return
@client.utils.JSOHelper::toDouble(D)(obj);
                        }
                    }
                }
well, keeping commented code clutters the code. This, it does not help reviewing the code as the diff does not show what has been removed (as it is not really removed)
This seems like a terrible place to put this animation. Surely you want to animate on show rather than setting the `contentViewController`.
Why not just override `windowDidBecomeKey` and `windowDidResignKey`?
That's alright- either way works, and I'll be doing a clean up branch soon,
so all the code will have standardized style after these pull requests are
merged.
yes, but i left it because it might also be handy in in C# context if the user wants to define messages as a space separated string... should i still remove it?
Okay. Will do.
Is there a need for both this and handle_create? Could probably nix one of them.
why not final ?
add exists=True to all file/directory outputs
STYLE: Rename AFNICommandOutputSpec
вот тут еще надо пнуть что-то типа connectToPort() у этого EdgeElement'а, а то сейчас линк красным рисуется при вставке. но для этого надо иметь доступ к этому еджу, и откуда его взять, пока не понятно
I think that is better to use the return of `system` instead of the `$?`
Can we get some performance numbers here and how you derived them?   We still need a good set of performance tests, but I still like to see some backing to claims like this.   Note:  I can help you spin up Ye ole http:// github. com/ jsuereth/sperformance if you want.
Nice. I knew about that (not why, only that it exists) but forgot. #1824.
Can type refs with non-empty args return `true` from `tp.isHigherKinded`?
Given that the expressed limit is clearly stated in the class’ scaladoc comment, I don’t think it is worth calculating them at some other place and giving them names (which would also slow down the creation of every single Duration instance). And they wouldn’t really get any less magic by moving them into the companion to give them names, because then you’d have to search for their definition instead seeing a value which many programmers will intuitively associate with the right thing (granted, I also don’t know the digits by heart for all of them, but they are easily verified).

So, long story short, the best I can think of is adding comments saying how they were calculated.
I think the documentation in the `TypeSymbol` should be pulled up to this level and merged with these comments.
I have no idea. I followed the wrapper for *gees and it has these extraneous parameters as well AFAICT. I assumed it had something to do with the external function call since in that case select it takes at most two parameters.
This needs to be rephrased such that the summary fits on one line. The rest has to be added to the next paragraph.
Should this be called from Bus.unregister()?
```

>>> S('0.[142857]')
1/7
```
this is for the cases such as  (I)\**(I+2),(-1)\**I and1\**I which are not checked by <code>ask(Q.real(expr.exp),assumptions) and ask(Q.real(expr.base),assumptions) </code>even though they are real
when you use `&` in `if()`, it can almost always be replaced by `&&`
Yes, a reference link(s) on the bottom of the page would make more sense   :D
Changelog line should be added since bug was introduced in 1.1.11.
Follow Yii code style:
- remove extra spaces around operator keywords.
The guidStr in the return is being used as a constructor for a std::wstring, so saving it off to a temporary is not necessary.  Even if you leave the wstring return type, you can just call return on the .ToWchar() method, and the wstring will be constructed before the plStringBuffer<wchar_t> is destructed.
А для чего здесь эти строки?
Why do we call both of these in a row? Is there a case where we call one and not the other?
One day we'll all be using pre-processed CSS, and we can leave this madness behind...
ja, das wars! Deshalb hat auch Navigator verrückt gespielt.
I think it would be good to make all of these utility functions public, but I agree that they shouldn't be camel case.
Looking into it
Perhaps we could simplify by checking the value for noproxy once inside this method (the code from line 33 above)? It always gets passed anyway, this helps minimize the amount of stuff that needs changed. Thoughts?
Nice catch. :flushed:
One final thought here… sorry I missed it before. I don't think this needs to be declared in the header. The fact that it uses a custom window is an implementation detail. I would put the `@interface` into `RBLPopover.m`.
They are easier on the eyes :smile:
It actually seems strange to me that this needs any styling at all.

`text-decoration: underline` is default link style

Font size and family should probably be whatever the surrounding text is, no?

`cursor: pointer` is unnecessary if you add an `href="#"` to the anchor
`display: block` and `float: right` seem to work in concert, but would do better left out.
Aborting on EMFILE is undesirable.
Assignment after declaration. Also needs to handle a -1 return value (because of e.g. EMFILE).
it is fixed.
Except in idiomatic cases, I'm generally not a fan of really short variable or ID names.  In this case ````tft```` doesn't seems to have a lot of meaning.  I'd prefer something a bit more meaningful.  Ditto for frb.  This probably means that a lot of these lines will have to go onto multiple lines, which is fine.  Just line up the hash keys and it'll look great and be more readable.
This is part of the other issue. Don't try to solve this here.
The reason why the original code did not use variables was to make the scenario more readable - when you factor this out into variables, it becomes less explicit as to what the test data is. So whilst in general your approach is more maintainable, it has the disadvantage of being less readable.
This works, but the usual Windows/DOS command is "md".  "mkdir" is the Unix/Linux command and also works on Windows.  So, idiomatically, "md" is better for Windows.  You can also specify the driver letter in the command, so "md C:\data" (etc.) might be a better example.
No, If-Match and etag is best left optional.

It makes sense to use for a manual editing scenario - ie. the app pulls down the current content, time passes while user edits and saves, app PUTs new content & detects conflict.

However, it doesn't make sense for a content push scenario - ie. generate HTML from a pile of markdown, PUT all the HTML files. You don't really care about conflicts in this case, and probably want to just pave right over whatever's there.

If the API required the etag, then a doc update script would first need to download every affected doc to acquire etags or maybe retain them from the last update run. Generating them locally is a bad idea, and not guaranteed to work. That's a lot of work & API calls, if all you want to do is clobber read-only MDN pages with HTML from github-maintained source.
I am +1 on using a framework (I personally have been using backbone). But I realize that might be a bit of work for you to convert this now and you probably have to ship. Something to think about next time around. And if this starts getting way out of hand, then you can think about converting.

Also, sorry for landing here at random and being negative with "no me gusta." :-)
This should be done in the package task(s) and/or a develop script as I mentioned, not when extension manager is initialized.

Inline html is also not acceptable. There should be a template index.htm (that is version controlled, i.e. not gitignored) that is copied into the desired place.
What is the use case for setting a name? Is the name the same thing as avatar type? If yes what does it mean to either set the name at all or change it?
Please check if this language key already exists elswhere, I'm pretty sure it does. Please pick a unique name.
http://www.postgresql.org/docs/8.2/static/multibyte.html#MULTIBYTE-TRANSLATION-TABLE Documentation mentions that standard SQL syntax can be used.
Casting an array to int here.
Have to ensure this array is not getting too big. Either by deleting in a loop or by keeping a limit on the select ids for this cron run.
Will add that.
Its not creating it though. If anything more extraction. As its extracting/getting it from the metadata file.
I don't think the white space should be removed here.. Check out the lines which follow. They too have whitespace after every break.
Should probably return an array with the user_id as the array key so you can simply use that instead of the get_user() method below.
This was copy-pasted from tidy_cache, it should be changed in that file too.
metadata.
var scope =[]
_.each(... function(i){scope.push("'Scope:" + i + "')};
scope = scope.join(',');

close enough?
Actually, incomplete code, missed that. Whoops.

Fixed in e3344e2 

![](https://a248.e.akamai.net/camo.github.com/b09586ab81d8ef8b3a9984c1bb650212cf1fa636/687474703a2f2f692e696d6775722e636f6d2f41444b6a702e676966)
Please replace with: `cp_number = Player.objects.filter(race__can_play=True).count()`
We don't want to pollute our source directory with CMake cache, binaries, etc.
Well, first. This should add the policies category to the integrity of the result label. But moreover, to do that, we don't technically _need_ it to be TCB. In particular, if the policy was forced to implement this it could look like (a different name to reflect that it's not TCB, but...):
    
    toDocumentTCB :: (DatabasePolicy pol, DCRecord a) => pol -> DCLabeled a -> DC (DCLabeled (Document DCLabel))
    ...
    toDocumentTCB (GitstarPolicy _ priv _) lr = do
      lcur <- getLabel
      lres <- lostar priv lcur (labelOf lr)
      toLabeled lr $ do
        setLabelP priv lres
        unlabel lr >>= return . toDocument >>= label lres
It should probably also have a check if we are currently in a game (admins may view reported/global messages from play game screen, and hence wouldn't be in a game)
cache this.settings. it's accessed multiple times within each iteration of the loop and never changes.
Composition of URLs should *not* use libpath.  If we did, we'd have URLs like http://server/foo\bar\baz on windows, which is clearly wrong.
is this logic correct? ( !isNaN(x) --> If true, means that x *is* a number.)
screener would be the default here, but the listeners below should go into the actual toolbar file (otherwise we would have to add them in manually into each module)
We shouldn't change this.
Yes, src shouldn't be in the path for this to work correctly on our server. (as ROOT of our server corresponds to src)
Yeah, will put comment for now. This getFields business happens a lot -- ideally there would be an extjs function for the model that would get a field by name...but there doesn't seem to be one. Maybe in the next version of extjs
is this json is created by sending request to server ??
The usage of bounded cell comparisons might help clarify the desirable properties.

But I'd hope that we end up with a well-defined, self-consistent ordering:
 - `a < b` if and only if `not b < a`
 - `a <= b` if and only if `(a < b) or (a == b)`
 - `a < b` if and only if `b > a`
 - etc.
I think that now you can do:
cost.zeros_like()

but cost.ones_like() don't exist yet. Creating it would allow to remove the import circular dependency problem. If you don't do it, I'll do it after your PR is merged. This will also make theano more consistent in that interface.
The rest of the pull request is just me fixing gradients that were wrong and adding connection_pattern to the ops that need it.
is total_elements and total_size even used?
The CSVLoader should be generic. It sounds like this method is application specific. Maybe this method shouldn't be in the CSVLoader?

You can also connect to the database and get the item IDs by selecting from the item table.
It's there to prevent users from being able to call the constructor (practically).
I'm going to pass on improving this today, partly out of the interest of time, but also because I kinda like the pure pass-through approach it takes.
The name of some of this stuff seems less than straightforward. Why not something like "canLeverageIndexedExpression"? Binding does't seem like the right word. Nitpicky. Apologies.
Don't include status and message in the return value. Just return None.
heyhey, italian language. Could you switch them to English? Thanks.
Loadable promise types through plugins?!?  That would be AWESOME!!!