labofsahil/github-event-dataset-2012 huggingface dataset:
tiny tiny typo fix to this wonderful package documentation

Hi Gabriel,

your Pipes package is absolutely beautiful, many thanks and congrats for it !

You'll find attached a tiny typo fix.
Create dest folder if necessary

This one creates the destination directory before attempting to write the resulting js file. (Depends on mkdirp)
Also, I upgraded the UglifyJs version to the current latest in dependency list.
OAEWDGT-94 - Truncate review

Jira: https://jira.sakaiproject.org/browse/OAEWDGT-94
Possibility to choose list identifier route

This allows to choose which route to use to create identifiers links :

```$listMapper->addIdentifier('title', null, array('route' => 'show'))```

Edit is still the default route.
Улучшенное открытие треда в клавиатурной навигации в Firefox

Теперь:
1) Не требуется разрешать всплывающие окна
2) Если уже открыт таб с этим тредом, Firefox просто переключится на него (Scriptish).
.min_point() and .max_point() for Arc and BezierCurve

Creates bounding boxes drawn in tight to include only the points that are drawn by the respective shapes.

Pay special attention to assumptions in Arc, the upverter format spec wasn't especially clear so I just went for an informed guess.
Pull Req: Adds support for Card's methods and events
allow = sign in key-value pairs and trim value.

just a simple usability fix. i had to view source to figure out why my headers weren't being sent. this fix makes it a little easier for first-time users of the tool.
motify <p><p> to <p></p>
Upgraded to latest version of phonegap (1.4.1)

I was having a problem getting this to work on my Kindle Fire (works fine on other Android devices). I noticed that a newer version of phonegap was out, so I upgraded and thought it might be useful to the community. If I figure out how to fix it with Kindle Fire I'll send another pull request.
Code cleanup, more examples, documentation

I cleaned up some of the code, added a view to assist in some examples, added more examples, and added some documentation
Update Tests/Twig/Extension/EchoExtensionTest.php

#88
s/content-lengt/content-length/
Correct. It will only get here if there are contributors to show.
You don't get better trolling than this.
LOLWHOOPS
This won't build...
added a colon in a later commit.
This chunk is taken directly from the git version... which is fine, except that comments in the hg module now reference git behaviour. Also, the formatting is old and definitely not PEP8. Could you please clean up the comments and formatting as appropriate, and check to see whether the documented git directory behaviour makes sense for hg?
Here's the values implementation in cpython https://gist.github.com/1970445
Never mind, I made changes manually.  Thanks.
If you are going to null out m_children you should also free it!
Yep, officially the API is:
    def shift(self, periods, offset=None, **kwds):
I've been going with offset (not freq) everywhere, to keep consistent with prior naming choices.  Any issues here?
If I got it well, that's implementing the: "Either by creating one, or reusing an existing one." part using lua weak typing capacity, right?

It's amazing to see how you seem to be at ease with dealing with lua metatables and the registry. :)
It took me time to look through the logic of the insert() function because of my lack of knowledge there, but If I get it well, I guess you're also doing that to reduce the number of different stacks in the future, right?
Btw it's easier if you just create a simple form with a simple field in the test, then you remove all the overhead created by the choice type.
Is er een reden waarom je de validatie niet in de url zelf doet?
may not compile on some old VC/Win SDK versions.
name should be "ftext" to match the export types.ftext = new FormattedText()
already fixed :P
My apologies for the erronous commits, Git foobar on my end.
my favorite singleton pattern is enum :)
http://stackoverflow.com/questions/70689/efficient-way-to-implement-singleton-pattern-in-java
Shouldn't it still be version 2.0.9 since 2.0.9 haven't been tagget yet? If no, should it be v2.0.10 instead? :P
Example how VC 10 compile:

int result1 = int((in1 * in2) / 100.0f);

0000006d  fld         dword ptr [ebp-0Ch]
00000070  fld         dword ptr [ebp-8]
00000073  fmulp       st(1),st
>>>>00000075  fdiv        dword ptr ds:[03E02F90h]
0000007b  fstp        qword ptr [ebp-2Ch]
0000007e  movsd       xmm0,mmword ptr [ebp-2Ch]
00000083  cvttsd2si   eax,xmm0
00000087  mov         dword ptr [ebp-18h],eax

int result2 = int((in1 * in2) * 0.01f);

0000008a  fld         dword ptr [ebp-0Ch]
0000008d  fld         dword ptr [ebp-8]
00000090  fmulp       st(1),st
>>>>00000092  fmul        dword ptr ds:[03E02F98h]
00000098  fstp        qword ptr [ebp-2Ch]
0000009b  movsd       xmm0,mmword ptr [ebp-2Ch]
000000a0  cvttsd2si   eax,xmm0
000000a4  mov         dword ptr [ebp-14h],eax
mmm, something like that (linux shell):


mysql -u user -p mangos < *mangos*.sql

mysql -u user -p characters < *character*.sql

...
не только высаживанием вендоров, хватает квестовых штук, которые могут суммонить в неограниченных количествах
hmm... this not implemented at all.
@JonFerraiolo  Do we have precedent where a property in the Prop Palette sets child content of the tag?
I already know that, but I don't like to hide the null test logic into a base class. I tend to think that explicit is always better...
But still, I won't argue, thats fine ;)
Hm... I'm not sure we can maintain that. I don't know which pages are python or PHP (when I'm focusing on one page), and when we switch one I'd have to go and update all the links.

I'll make a helper that adds the lang for now, and eventually we can go back and add a bunch of reverse/url tags when we have most of our pages on Python.
just delete this class, it's getting its own refactoring
ammm entendi mal esto :
Estos eventos han sido realizados por WowRean sin cojer ningun dato de otros servidores, estos eventos han sido privatizados en nuestro repositorio publico para que no se puedan usar las scripts fuera de nuestro ambito personal.

Pero, si lo poneis en un repositorio publico, no lo privatizais.. XDDD
There really isn't a Psych-specific error that you can catch? :(
Yeah that's probably a better idea incase something isn't speaking HTTP correctly.
Looks like the code has shifted around, will apply manually.   Thanks!
It'd be nice if you port these improvements to master branch
I'm not a huge fan of this... it ties notation directly to the font file. Do we really need this here, and if so are there alternative approaches?
you can also try 

     deadbeef --quit 

to gracefully close `deadbeef`
this is good, but the commit includes only a new line addition.
If i'll merge it -. the only change in the code will be a new line ;)
true, just trying to minimize changes to tracks 2.1 to get it out of the door :-) I don't mind dropping older browsers, but when we are getting close!
To hide all the resize handles, change the ```getDisplay``` function (in ```"transform"```) to something like:

```javascript
getDisplay: function(feature) {
    return feature.attributes.role.match(/\S{2}-resize/) ? "none" : "";
}
```

Or to hide the north east and north west resize handles:

```javascript
getDisplay: function(feature) {
    return feature.attributes.role.match(/ne|nw-resize/) ? "none" : "";
}
```
I went with `ot` and `os` for now ... since the shift-key makes the rhythm strange. Then I realized that `o` might be mnemonic for **other** aswell, so **jump** is now in-buffer and **open** is other.
1. Now we follow the freedesktop Base Directory
Specification<http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html>,
QSettings supports it out of the box. And standards implementation supports
some features like atuo merging  global and user settings.
Using its own directory, will require additional code and some hacks with
Settings::setPath function.

2. With precompiled path, you can't change the path, without rebuilding. In
the new implementation path is "dynamic", and you can change
it, using $XDG_DATA_HOME and $XDG_DATA_DIRS environment varables.


2012/2/29 Jerome Leclanche <
reply@reply.github.com
>

> Why? What's wrong with leaving it configurable, but in .config/razor by
> default?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Razor-qt/razor-qt/commit/e564586c86583a7ed1797b5c9428adcfbb5e3329#commitcomment-1024027
>



-- 
Best regards,
Alexander.
最好用 attr_accessible 哦
The CMake dashboard:
http://open.cdash.org/index.php?project=CMake

The Qt4Deploy test on all those machines last night:
http://open.cdash.org/testSummary.php?project=1&name=Qt4Deploy&date=2012-02-24

It failed on some, mostly because there are multiple Qt installations and it used the wrong one, or the installation that was found can't be used.
I just pushed another commit to fix most of those:
http://cmake.org/gitweb?p=cmake.git;a=commit;h=52e827

From what I can tell, the few failures that will remain are not our fault.  For example, the ninja generator is still being developed, and they need to fix the generator to make the test pass.
I just lost The Game. :(
use new lambda syntax?
Hello,

I fully understand that your priorities right now are others than Dimscape development. I think is nice to have an applitude of zzStructures in Python, and this is my main interest.

Concerning video cells I understand this adds a lot of problems compared to static contents, but this can also serve as an interesting challenge. I'm more interested in the use of pictures as contents, which I assume will be easy -not so difficult as- to add when video cells issue has been solved. I have to add that I could'nt make 'video cells' working in my Ubuntu 11.10. In fact I started to configure on Ubuntu 11.04 and had a lot of problems with qt-phonon and/or kde-phonon, which stands incompatible versions with PyQt and PyKde. Fortunately -and to be honest surprisingly to me- upgrade to Ubuntu 11.10 'fixed' this problems with versions.

On the other hand I have to add that despite my interest in Dimscape my skills on programming are limited, and my capability to make apportations to the project are really minimum. Nevertheless I will study the code and hope can count on your help for further explanations. As I told you my priority would be to add cell cloning capability, in order to make Dimscape more fesible to hold more realistic databases. I hope the video cell compatibility don't make the task unaffordable to me.

Sincerely,
Miquel Joy
this.properties is better? or this.name = {};

this.name = {required: true, unique: true}
Please do fix it, and in general don't use the word versions of || && and !. They cause precedence problems.
не самое лучшее решение хранить общие js в модуле шопа, ведь на сайтах где шоп не нужен, многие могут его просто выкинуть
you could `filter`
I would use `render_xml`: https://github.com/mozilla/zamboni/blob/master/apps/api/views.py#L79
Completely forgot I had edited the README file through the web editor, my push failing left me wondering for minutes.
The situation is a bit convoluted and could use better documentation.

Basicly the field formatter uses [`theme('ting_search_collection')`](https://github.com/dingproject/ting/blob/master/modules/ting_reference/ting_reference.module#L151) which [wraps collections in `<li>`](https://github.com/dingproject/ting/blob/master/modules/ting_search/ting_search_collection.tpl.php).

To avoid references which are wrapped in `<li>` twice I needed to override the theming of the content field.
@sminnee This should've been rebased, its reverted in the following commit.
Lisez ce fichier !
Must of missed that :S
Actually this fix really belongs on master.
Document each variable
This can be an `if` of its own after you remove the one above.
This change was probably not related to the removal of the Field->version link.
This maybe shouldn't have been reverted?
seems ok to me
Az a baj a subsubsectionnal, hogy az alpontokat már így is subsubsectionba tesszük a latex fájlunkba, ezért minden osztályt külön feltüntetni feleslegesnek tűnik.
Image:
http://img268.imageshack.us/img268/7712/emeseney.png
post-facto r=dmose on the stuff we didn't pair on, assuming my inference in the comment is correct.
list(, $method) = ...
You can point to commits or add pull request and I will check it.
children is an out param, probably want to make it a pointer (if we are following that convention, that is)
Good call- I'll fix it in the morning.

On Feb 24, 2012, at 11:27 PM, Srdjan Pejic<reply@reply.github.com> wrote:

> How about:
> 
>    if %w(on true yes).include?(value)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/commondream/tconsole/commit/8887abf1df27324e4505306d4c8e55678797beea#commitcomment-1010266
hmm, i think this code was broken anyway?  backfill shouldn't reset the log like this anymore.  will fix that separately.
esto esta malo, lo arreglo apenas pueda D:
no it does not *bummer*
@bensie I wonder... How far from JRuby-compatible are things at present? As someone who knows nothing about the Carrierwave source nor Java or the JRuby internals, is it feasible for me to tackle this, or will it be too daunting? We want to use Carrierwave in an app which runs on both MRI and JRuby, so having Jruby-compatibility is required. Thanks!
The str part is a bug.

The style nit is to add an explicit 'else'.
@jancel I'm sorry if I came across as rude.  I meant to ask if it was test w/ PG in particular... and I should have checked the issues list.  Too many nights in a row of late night coding ;).  Thanks for the quick response and the original update/patch.
LoL thanks :)
How did this break mouse tracking? I have a better solution for the
_highlightsBy test and also modified CPButton to properly initialize these.
The mouse tracking is new though. Do you have an issue or group post to
link to?

On Mon, Feb 13, 2012 at 8:34 PM, Adam Radabaugh <adamradabaugh@gmail.com>wrote:

> Agreed.
>
>
> On Mon, Feb 13, 2012 at 6:52 PM, aparajita <
> reply@reply.github.com
> > wrote:
>
>> There are three problems:
>>
>> - His commit edd09ef06a (which you merged, doh!) broke CPButton and
>> CPPopUpButton mouse tracking and also broke updating of titles in
>> CPPopUpButton.
>>
>> - I don't like this test for `_highlightsBy == undefined` either, it
>> should *always* be set to something. That tells me that stuff is not
>> getting initialized correctly. Both _showsStateBy and _highlightsBy should
>> always be initialized, currently they are not.
>>
>> Personally I think edd09ef06a should be reverted until all of these can
>> be addressed properly. Not having proper mouse tracking and title updates
>> is a much bigger problem than not having the extra button types.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/aradabaugh/cappuccino/commit/915b1f3638c1ee5e71989570fdb3ac399aee220a#commitcomment-962924
>>
>
>
I've added a more informative assert output in 11922 - this should be much easier to track down now
Shouldn't lookup have an else branch to return a non-nil value? (This may be a dumb question, I haven't read the rest of the code yet.)
Yep, this code isn't even called any more AFAIK.
See my other note on checkbox label.
can you just move it into the post itself? with maruku you gotta put 
&nbsp; in between the script tags otherwise it converts it into an 
"empty" tag.

On 2/13/2012 10:48 AM, Seamus Abshere wrote:
> if it's a prob we could def take it off
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/brighterplanet/numbers/commit/2a731c12c5a349f950ac42d0f4b343d8b546fd8e#commitcomment-961308

-- 
Andy Rossmeissl
co-founder and product design director
Brighter Planet
aim: brighterplanetar
office: (802) 458-0441 x302
freenode: #brighterplanet
twitter: @rossmeissl
http://brighterplanet.com
This is a bit cleaner: `var elems = (content.cheerio) ? content : parse.eval(content);`
Though, that shouldn't matter when it runs as root.
Why did it block?
can be const, and have spaces before/after the {
zwerg.razorscale In a flight mode сasts in back
I'd be really keen for LaTeX to stay where it belongs - a distant memory of my PhD days!
Yes, that's correct.  If you're allowing connections from localhost, I figured you might as well connections from the machine to itself on the external IP address as well.

You could put the whitelist declaration into the master class as exported resource that each node collects.
I really like what you have done with adding the html report.  Any chance of adding in support for when using Scenario Outline and Examples?
this fixes #170
That is perfect.
formatted oddly, will fix
Just translate to russian whole name.
OK doing a pull request then.
Hi! I updated to this revision, but I noticed that Arrays.copyOf method is available since API level 9. So it doesn't work in froyo devices :( 
Will be API level 9 the min sdk version?
> good catch, I hope you did not waste too much time debugging this.

Actually, it was breaking a 'mem.cache' in some of our code for a long,
long while. I think that @agramfort kinda melted the cluster in Harvard
because of that :).
> I would better use `None` as a default marker.

+1
A klasa Scanner? Wydaje się być jeszcze lepsza.
Ah, good, I was trying to figure out why it never worked. Just pretend this commit never happened.
Там проверка в 39-42 строчках, в main. Я же правильно понимаю, что:
pow(x, 2, f) = f(x, x);
pow(x, 1, f) = x;
Так?

Тогда для степени в нуле нужно возвращать единицу. А для матрицы - единичную матрицу. А как реализовать такую функциональность в темплейтной функции, я не представляю, честно говоря.
还有这边月球不能叫卫星吧。。。又不会挡住攻击的。
Same comment as above.  Just copy the entire list of certs.  Much less code, and much less work for the server.
Have to update some stuff (seems like JFugue's needs some special notation for the notes) but thought it would feel cooler to hear the result :)
#54 is pretty meaningless in the context of the code. I sometimes work offline and push changes when I come online. There is no way for me to know what #54 is when I am offline. Same for a 3rd person reading the code: "Uhm! Where did #54 come from?"

 I try to write in a very self explanatory fashion:

    # skip past all but alias file of a NCBI multi-part database
    #
    # [elaborate a bit if necessary]
    # ...

This way I am describing what the following code does.
First commit.  Awwwwww yeah.
Please add headers for cshtml and javascript files
Yeah, for the record, you need the following kernel configuration options set:

    CONFIG_WIFI_CONTROL_FUNC=y
    CONFIG_WIFI_MEM_PREALLOC=y

Then it compiles fine.
Use a different SP name as per previous comment.
Just testing my request, is it been denied or accepted !!!!!!!!!!!!!
In case I wasn't clear about feature requests, go to my project page:

    https://github.com/kubitron/redmine_git_hosting/

Register a new issue, and click on the "Feature Request" label at the right before saving....
header.getValue() may be null. You should lift that up to under line 145 and check for null, just like a missing header.
It seems like you could lift out this whole try/performWait/catch recipe, turning each of these tests into a tiny snippet like:

<pre>
expectExceptionAfterDelay(SHORT_WAIT, inThread(performNotifyErrorRunnable());
assertNotifiedError();
</pre>
Before you take a look at my (upcoming) pull request, run these tests again. Note that they fail for the wrong reason. This is a common mistake when testing that something raises an error in Rspec. See if you can come up w/ your own implementation that fails "correctly" before you look at my pull request.
Err, the code here is correct or my assertion is it backward is correct? :)
You don't have a wiki yet... and what would be considered relevant for it? Would it have a specific purpose for project?
I'm really not sure why the case changed.  It used to require the capitalized word.  Makes me wonder if other issues have arisen due to case sensitivity.
I wasn't exactly sure where I should comment, so I picked here. I was looking for your MIUI ROM respiratory, so I could see and better understand, how you got MIUI working through your commits; I thought I seen it before, though I can't seem to find it now, maybe you deleted it. Any help?
`self.view.response` doesn't work if the view is a standard Django view.
Let me see how many bytes we save doing that. Newbies coming to yui3 grids from other grids have often told me they get confused with the `g` and `u` syntax. I was actually thinking a better naming convention would be:

`y-row`, `y-col-1-3`, `y-row-responsive`. I feel this would be less typing and easier to read semantically than `yui3-g`, `yui3-u-1-3`, `yui3-g-responsive`. What are your thoughts?

One of the reasons the GridBuilder exists is to allow people to customize these prefixes as they see fit.
Derp, I am bad at the alphabet
I believe best practice is to always `import actionrun` for that reason.   This is only being used as actionrun.Error right now.
Closed is for “we don’t have an underlying transport yet” while WaitActivity is for “we are waiting for activity on the underlying transport”. Incoming connections logically start out in the second stage, the first stage happens unbeknownst to us.
or some move-the-ROBERT-out massage … (done)
The patch comes out something like this: https://gist.github.com/2822571 
It's ugly in its own way.

Tricky bits include:
 - creating the instance may not run Actor's constructor (an existing instance can be returned)
 - receive can depend on `context` and `self`, e.g. `akka.event.LoggingReceive` does
 - handling restart of the actor

As you say, receive and aroundReceive will be called before the actor's constructor completes, which may be unexpected.

Also, while the above patch handles getting the same instance back on restart, it doesn't handle the case where a constructor is _never_ called (not even the first time). I don't know if that case is "allowed" but if it is, this approach can't work.

Side note: I think the original pull request has an issue that on restart the actor is not null inside become() during the second construction of the actor instance. So it probably uses receive/aroundReceive from the first instead of the newly-restarted instance. This is probably fixable by just setting actor to null during restart.

Side note 2: maybe it's too late at night but with either the original pull request approach or this patch, I wonder if behaviorStackPlaceHolder can just be Stack.empty; if we're using behaviorStackPlaceHolder then I think receiveMessage will NPE anyway because it tries to invoke actor.unhandled with a null actor? It seems to already be true that receiveMessage is never called unless there's an actor instance and thus an actual non-placeholder behavior. Don't know.
Note that immutable.Seq() returns List, not Vector.

Welcome to Scala version 2.10.0-RC1 (Java HotSpot(TM) 64-Bit Server VM, Java 1.7.0_10-ea).
Type in expressions to have them evaluated.
Type :help for more information.

scala> collection.immutable.Seq()
res0: scala.collection.immutable.Seq[Nothing] = List()
ah, yes, of course, my bad; will fix
or an AkkaException even
added links to assembla and roadmap
type_id stuff comes last, right?
In NHLIB the unit for PGA and SA  is g and not m/s2. You can get the standard acceleration of gravity from scipy:
from scipy.constants import g

and then convert to g in this way:
mean = np.log((10 ** (imean - 2.0)) / g)

Note that this conversion needs to be applied only if imt is PGA or SA but not PGV. The original equation gives PGV in cm/s which is already the unit used in NHLIB. Because of that the test or the test data needs probably to be updated.
They are certainly not stupid! I wasn't making myself clear, and there has been a confusion with what action means (controller action != logic layer action).
The one I don't think we need is `get_actions` (the hook point for p.IActions). This only is needed if you want to add an action to the logic layer (ie something like logic.get_action('resource_proxy') or http://ckan/api/action/resource_proxy).
So, keep the before_map (IRoutes) and the controller and just drop the get_actions method.
I was taking a minimal change approach to modifying the code to support loading general frameworks.  There are three ways a framework can be specified — `MyKit.framework/MyKit`, `MyKit.framework/Versions/Current/MyKit`, or `MyKit.framework/Versions/YYY/MyKit`.  This was the smallest change I could come up with that could address all three cases and leave the user in control.

The changes I submitted are enough to enable generic loading of a framework from `/Library/Frameworks` or from `~/Library/Frameworks`, where before an absolute path to the framework binary was required to bypass all the checks — So the change is an strict improvement, but there is certainly room for more.

The macholib of Python approaches this problem by returning a list of alternatives from the parallel to `mapLibraryName()` so that many different options can be used, and the loadLibrary could remain "MyKit" instead of "MyKit.framework".

Another option could be:

    if (libName.contains(".framework")) {
        if (libName.endsWith(".framework")) {
           // get <name> before ".framework", and construct a result like "<name>.framework/Versions/Current/<name>"
        } else return libName;
    }

This is quite literally the first Java code I've ever written, so complex string work in Java is a bit beyond my skills of mimicry.
if watermelon is renamed to grapefruit, then watermelon+seed is ALWAYS renamed to grapefruit+seed, regardless of whether update_references is set.  the +seed card is a "dependent", not a "referencer" in these terms.
createContext
I understand your concerns. The started flag is updated in a synchronized block and the zk state is a volatile enum. Furthermore, connected is only called from the OperationLoop actor. Yes the two conditions can be modified independently but this is not critical (only happen in stop).
That indeed does make sense yeah. Only thing we need to carefully look at is how we handle filenames in CURL::. Looking at its code it doesn't seem *entirely* straight forward, since we also use Get/SetFilename internally there. One thing we *could* do is add an extra bool option to CURL:Set/GetFilename to tell them to URL encode/decode or not?
The minimum would be SchemaTupleClassGenerator.isGeneratable(conf)
ideally, we have a configure(conf) step when the conf is parsed so that it happens (and is validated) once, and the code that reads the conf is centralized
Rename it to EJBType (better consistency with other EJB artifacts)
Try to remove trailing whitespace.
After the asterisk, there has to be a (single) whitespace (in all method documentations).
it's not tread safe only if colum.size() is 0 (the first time). 
I'll add the AtomicReference. I learned this one new today, I would have used a synchronized block.
I'll commit this, this evening.
Later you query the size of this member. If it has never been initialized to an empty vector and nothing has been pushed onto it path_stack_.size() may return a segfault.  That is often the case with these standard containers. This is probably worth checking/testing.
Yes, but that doesn't solve the url problem - laravel's pagination uses current url, so when you're on the /core/post/1 page the pagination links will look like: /core/post/1?page=1 etc. (instead of /core/topic/1?page=1).
These (and the above `valueWith*` methods) are somewhat concerning to me because they're not prefixed. I know that OS X doesn't use the `CG` convention at this time, but there's no guarantee that OS X will always use the `NS` variants of these types.

Calling @jspahrsummers for his opinion on this.
The first time it's not called, see the if check: if(feature.state === OpenLayers.State.DELETE && !this.commitMsgPrompt)

If I did not do this, the Delete request would not go off anymore. Couldn't put a finger behind it.
@cataphract updated and rebased the commit.  thanks
Any suggestions? What about

    send("#{store_attribute}=", HashWithIndifferentAccess.new) unless send(store_attribute).is_a?(HashWithIndifferentAccess)
    send(store_attribute)[key]
Ok, great, thanks :)
The last usage of this method was removed @ 8572ae667
Sure.

Test environment can create object with same id many times during test suite

Now I do:


``` ruby
  Rails.cache.fetch(object.id, :force => Rails.env.test?) do
    
  end
```

But using `:if` will be better.
Changing `stale_target?` makes sense. I think it can also be simply a nil check, rather than using `present?` ?
Do we intentionally skip more than two variants when issuing a warning? E.g. what should happen in case of:
```scala
class Bippy
class BIPPY
class bippy
```
My fault; I knew that was a lossy comment move when I did it.
I would just say “Twitter‘s” - there were many authors of Try.
In the interests of keeping the number of aliases to a minimum, I believe the standard for universe is "ru".
I think this will erroneously decrement the refcount of Py_None when the callback argument is none.
The problem with `A.I` is that it uses attribute syntax but can trigger expensive computations, so it doesn't follow the recommendation that properties should perform simple and fast operations. I didn't remove it but I think that we should not encourage its use.
Perhaps RigitBody and Particle should define no-op `potential_energy` that has the error message there.
I've rewritten this whole section.
Why should vlcj provide this implementation and not any of the others?
Ah, didn't mean to merge this (it's work in progress)
Didn't knew that $app['security.encoder.digest'] was available.
/bin/bash is not always the location for bash - definitely not on FreeBSD. See http://en.wikipedia.org/wiki/Shebang_(Unix)#Portability
Following through line by line, it looks like a selection of endpoint changes are expressed in the various transitions.  It would be nice to either have each transition isolated as a test case or cover it with a theory and explicit data points which might help to expose whether there are any holes in coverage.
Not needed here any more?
Blank lines in the end of the file.
Did you mean toggle?
Maybe this query should be broken into multiple lines.
Each set_config should have an isset check for safety.
I see that this was existing code.
Yes, all the template side stuff will need to be redone, it's really ugly and not valid at all, it is just there for an example of what I think it could be like. :)

If you'd like to help out with the template side let me know, it should be fairly simple once a design is brought up and agreed upon. It should probably be discussed in the RFC:
http://area51.phpbb.com/phpBB/viewtopic.php?f=108&t=42829
Why not on the top of the function body, like the other globals?
Personally I hate to white this->attr since it's obligatory in JavaScript.
Please specify this in INSTALL as well. Certain version is much more useful than 'C++11 capable compiler'.
I'm pretty sure that passing in `dbPriv` is not correct.
For one, we are effectively no longer checking if the current thread can write to the MP DB.
Second, an MP doesn't get to decide if they want to endorse a policy labeled field. If we don't pass in the dbPriv then the VC _can be_ forced to 
label fields before performing the labeled insert by e.g., calling an MP function that does this.
The motivation for the labeled insert/save is to allow a VC to insert an already labeled document that it could not  have labeled itself, but the labeling of this document should be done elsewhere (e.g., when converting labeled request to labeled document).
isn't it true that two plus two are four ?
the heck is this?
Each file (index.js and app/commands/test.js) have its own definition of what it means to "configure YUI".
Maybe say ", such as" or "e.g." instead of the horrible "like"? :)
change this location to <location_path environment_variable="steamuserdata" path="55230"/>
The 3 above tests are doing the same thing...
Also naming them according to what they test is more appropriate that if...then description
typo - 'of'  should be 'or'
I've considered it, but I'd rather this was part of a follow on PR looking at the extent and ideally calculating it based on the cube.

@pelson - please suggest something for the docstring.
I added that check, and 29/36 tests on theano/compile/tests/test_pfunc.py fail, because the compute map contains 0.
I'm not sure how to find out why and how it happens.
I think this is more safe, as concurrent write don't have all the behavior well defined I think. I didn't wanted to look into the defail, but I think only 1 of the write is correctly writed, but only if the write don't cross a cache line. I think that when you alloc memory, you receive memory that will have each element fix in one cache line, but I prefer to be safer in case this change in the future.
Hrm, I could pull out the `tag` field for this. Is the 5 an input technically? Is it listed in `make_node` (perhaps as a constant)?

I do run into problems as you predict. Specifically the constant folding optimization tries to actually run the op at compile time which, of course, hangs.
Note that you can't get wrong expressions, just wrong Exceptions, if you fail to implement a connection_pattern for an Op.
If people still think connection_pattern imposes too many mandatory requirements on Op developers, we could add a flag to tensor.grad to turn the exceptions off.
I'm not 100% sure what you're worried about, but I think the code should work. Chat me up?
And probably you would really want to check the ``content-type`` headers or prefix instead.
I'd rewrite this piece of code:

```php
$query .= (($foreignKey->hasOption('deferrable') && $foreignKey->getOption('deferrable') !== false) ? 'NOT ' : '') . 'DEFERRABLE';
$query .= 'INITIALLY ' . (($foreignKey->hasOption('deferred') && $foreignKey->getOption('deferred') !== false) ? 'DEFERRED' : 'IMMEDIATE');
```
Do we output a sensible error message (like e.g. `build` does) if not?
This seems overaggressive... shouldn't `build:*:+effects` include `css` rather than exclude `effects`?
Yep, this is light on documentation so far. The main point is that we have two hierarchies: The inheritance hierarchy and the containment hierarchy. A state A can **contain** other states B and C. In that case A is a "parent" of B and C. But all transition states also **inherit** from class `State`, i.e., have `State` as "base" class. Now the C++ idiom used here is the "curiously recurring template pattern", i.e., transition states derive from `State` using itself as template argument. This achieves static polymorphism and avoids run-time virtual-table lookups. (Benefits: Slight performance benefit plus more compile-time checking.)
Yes, just let me finish, ok? ;)
I was doing this to define the Pool interface for the possibility of non-uniformly-weighted edges. I can take it out if you like.
It's worth a comment about the intentions here. You're using flags for purposes other than their intended one.
These methods should be defined outside the class.

Moreover, the options need to be documented (in a `def options` method.)
Got it, thanks. Have fixed this now.
Thanks for your suggestion. I was not happy at all with my approach, but I could not find out the preferred way to use a compiler directly from the Formula Cookbook.
data.each_line
this.routes_request is a deferred/promise, so just ```this.routes_request.done(callback);``` should work
You could write this as:

    self.keyword_mappings = config['keyword_mappings'].items()
    self.transport_mappings = config['transport_mappings'].items()

which is shorter and (in my opinion, at least) easier to understand.
Shouldn't we set the Config instance's access token here? Otherwise it seems we have to make a new instance of Reddit in order to utilize the access token.
You are right on this one, so it should be a constant at the beginning of the file.
Inconsistent use of parentheses. Perhaps cleaner to just say `not (c.is_wiki_mod or wp.special)`.
The comment

    # If true the session based breadcrumb is used, otherwise the static breadcrumb will be used.

Should be here, where the user has to make a decision and must know why ;)
What I wanna do next is replace this pool with something smarter: 5 connections per address is weird!
Add a protected method called getDirectoryListingFormatString() that returns DIRECTORY_LISTING_FORMAT_STRING. This gives users a quick way to change the format without having to duplicate the entire class.
Wow, that is quite slower. I've actually noticed something else that I need to fix. Will update in a bit.
It would be cool to get rid of the splats here. Do you think you could pass arguments as an array to `fireWithContext` and just handle it as an array in there? Or is there a case where `fireWithContext` needs to actually be called with a splat?
I think it *should* actually be safe because of the last gsub, which should eliminate all left angle brackets from the input string.

This commit just prevents `<!-- foo <ul> bar -->` from becoming ` bar -->`.
The pull view just lacks context here. A few lines up is

```
resp.data[base]       = posts.data;
```
nitpick: 
'playpause' class -> 'play-pause'
'playpause' attachpoint -> 'playPause' or maybe 'playPauseBtn'

Also, the title "Play" attribute should be in the localization file.
Should we have a test case that tries to call removeBinding() with an empty keymap? Just to verify that it does not causes anything bad by calling it.
These "Menu Item synchronizing" tests seem a little less mutative (is that a word?), so it seems ok to me to leave this part as-is (all sharing the same menu item).
Question: Why is this wrapped in a function closure?
I followed the steps of https://github.com/ajaxorg/cloud9/blob/master/plugins-client/ext.codecomplete/snippet_completer.js
I guess it is OK to block the worker a little while ?!

Anyway, I'm moving the function to a common utility `ext/codecomplete/complete_util`
Sorry. I forget to mention it. 

Here you can see the different symbols for each prefix http://en.wikipedia.org/wiki/Milli-

In this specific case, they are:
m -> milli
u (actually should be μ) -> micro
n -> nano
p -> pico
f -> femto
a -> atto
z ->zepto
y -> yocto
Are we assuming IE9 > for tests?
are we going to be consistent with get_id or _id? Mike was pro get_id and Cory complained earlier that _id was being used.
Kill this. If there is a user_id key in the session hash, then it should be in the database. The only reason you'd ever lose it is if you change the database in which case you'd want to have the user log out.
Adressed in beee7e87. Thanks for the hint.
Not on the whole file, however... We must try to avoid modifying th whole lines for nothing, taking the risk to retrieve an unusable diff. That said, I don't know which strategy is used by github to display diffs...
I didn't know it was possile to match on `null`. Does this definitely work?
Just the copyright, not license.
Space between name and value (after `:`).
We only care about youtube? Not soundcloud or vimeo?
the default Tabzilla floats to the right ( which doesn't work for our layout ) and doesn't have a high enough z-index, so we need to override those two properties.
Ah, ok -- re: my comment up there, this would be fine, if we reduce the height to 0 or something with the `on` class.

If your aim with the visibility stuff is to create an interim solution before transitions are added, you might as well do `display: none` and `display:block;`.

In fact, left me refer to you to a set of tests someone did that is helpful when it comes to this stuff:
http://fvsch.com/code/transition-fade/
Not sure why this is a `for in` loop, but it doesn't need to be. Can you make this an index-based hot-loop (`for i = 0 ... length`)?
You're right - I didn't need it!
Also: seems like you use this form over and over again -- could split it out into a partial?
This is not strictly true - you can use the methods from Rack::Test (get, post, put, delete etc.) here if you want to go directly to the controller and not use the view, since these tests are set up to run the rack app in-process anyway.  But in Cucumber I would not go out of my way to test less of the application; going through the UI is probably better here, since the additional get request to view the 'new app' page is so quick.
shouldn't we document @throws NPE if principals null , or/and something like @param principals non-null identifying principals
Ja, har inte tänkt igenom det så noga, fast det känns som att den relationen bara är vettig i kurser där man har samma assistent för alla labbar. Fast så är det inte på alla kurser, och då känns det konstigt att det värdet finns. Jag ska fundera vidare lite, men nu vet du hur jag tänker.
delta is the difference of any change in the size of the document if applicable
This will return NULL when val = 0, which is wrong. Need to check strictly.
Possible to move this to the top so it's not lost at the bottom?
I would also vote to log info message instead of throwing it out. If we will be hunting an error and increase logging level, we wont see anything. Please log it, but not with the error level, but with info (warn is the default).
Perfect. I will add something in.
On Nov 26, 2012 8:31 AM, "Robin Kraft" <notifications@github.com> wrote:

> In src/clj/fossa/cdb.clj:
>
> > @@ -0,0 +1,40 @@
> > +(ns fossa.cdb
> > +  "This namespace provides CartoDB support."
> > +  (:use [cascalog.api]
> > +        [cartodb.core :as cdb]
> > +        [clojure.data.json :only (read-json)])
> > +  (:require [fossa.utils :as u]
> > +            [cascalog.ops :as c]
> > +            [clojure.java.io :as io]
> > +            [clojure.string :as s]))
> > +
> > +;; Slurps resources/creds.json for OAuth: {"key" "secret" "user" "password"}
> > +(def creds (read-json (slurp (io/resource "cdb-creds.json"))))
> > +
> > +;; Slurps resources/s3.json for Amazon S3: {"access-key" "secret-key"}
> > +(def s3-creds (read-json (slurp (io/resource "s3-creds.json"))))
>
> Yup you did, but I'd forgotten about it. Maybe good to include a note in
> the NS as well?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/MapofLife/fossa/pull/33/files#r2223994>.
>
>
See my comment above, about the guideline.
"Documentation" is redundant.
"It works" != "It's guaranteed to work"; I would be fairly disappointed if setting focus on a widget also brought the window into focus.
I prefer Diapolo's description also...
This line becomes tx.vout[0].scriptPubKey.SetDestination(script.GetID()); after #1357.
This matches the original submission, and saw no reason to change it.  Using "> 60000" makes us slightly more independent of whatever the next-proto-version is.  No strong objections to ">= 70000" though.
Messages with flag WARNING have "Warning" as title, same for error. If we
aren't using title to summarize the message we could leave it out entirely
and base it on the flags too. Another simplification of the interface.
Oh I missed that, thanks for the clarification.
This function should be protected, not private.
`NULL` must be written as `null` as per the coding standards.
`Sys.which` would be better.  On windows, you'll need to look for `git.exe`
Even if you want to keep these things in `IPython.config` the `auto_highlight` method should directly access that rather than having the config dict passed to it as an argument.
yes, that's perfect, thanks!
With an `is` check, infinite recursion is easy to trigger with a copy:

```python
class C(HasTraits):

    d = Dict()
    def _d_changed(self, name, old, new):
        self.d = new.copy()
```

So that doesn't seem like a good choice. Maybe it should be left as it is?
please remove all setters, as they hint at mutability, which can introduce subtle bugs
same with long code lines ;)
I have sent a poll request to the develop branch including the fix for all necessary files:
And @crynobone you where right that would be a problem I have solved this by updating the :
laravel/database/schema/table.php

by updating these method all should work great :

/**
	 * Add an auto-incrementing integer to the table.
	 *
	 * @param  string  $name
         * @param  int     $length
	 * @return Fluent
	 */
	public function increments($name,$length=10)
	{
		return $this->integer($name, true,$length);
	}


and :

/**
	 * Add an integer column to the table.
	 *
	 * @param  string  $name
	 * @param  bool    $increment
	 * @param  int     $length
	 * @return Fluent
	 */
	public function integer($name, $increment = false, $length = 10)
	{
		return $this->column(__FUNCTION__, compact('name', 'increment','length'));
	}


regards
You are totally right, i add it now.
OK going to rename to "setupdb.py", underscores in executables seem obnoxious to me
no, that changes the logic.  The existence of a 'None' percentage only effects the outcome of a matched rule.  If the line were to be moved up before the match logic, then all crashes would be rejected all the time.
This is a preference. 

I would create a new method called minmaxSelf which combines the functionality of maxSelf and minSelf. As I think that the cost of a 2 method calls is too big for the functionality they have.
Is it conceivable that the URI doesn't originate from the client?
Do I have do remove whitespaces before and after 
/**
 * Show a list of blog categories.
 * 
 * @author		PyroCMS Dev Team
 * @author		Stephen Cozart
 * @package 	PyroCMS\Core\Modules\Blog\Widgets
 */

and

class Widget_Blog_categories extends Widgets
{
        public $title = array(

???
Hmmm, I think it would be probably useful to pass `$loop` and `$signature` as arguments for the callback here, I can't think of any other possible (that is, potentially useful) argument after all.
I don't think Qt guarantees that receiver must be of type QWidget here? (http://qt-project.org/doc/qt-4.8/qcoreapplication.html#notify)
You need the remove the `@api` tag here (and the same goes for the other one).
Sorry, sorry :(.
Because it was a very simples change, I didn't tested it.
I forgot to use $this->
so, the line is :
return $this->generateUrl($route, $parameters, $absolute);


On Thu, Nov 1, 2012 at 10:58 AM, Christophe Coevoet <
notifications@github.com> wrote:

> In src/Symfony/Bundle/FrameworkBundle/Controller/Controller.php:
>
> > @@ -31,6 +31,16 @@
> >   */
> >  class Controller extends ContainerAware
> >  {
> > +    /** Generates a URL from the given parameters.
> > +    * If you use Twig Template Engine, you are familiar with
> > +    * its path function. So, you can use this method
> > +    * to generate a URL at a controller.
> > +    * @see generateUrl
> > +    */
> > +    public function path($route, $parameters = array(), $absolute = false) {
> > +       return generateUrl($route, $parameters, $absolute);
>
> This is totally broken. There is no generateUrl function
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/symfony/symfony/pull/5887/files#r2004669>.
>
>
you should use ``call_user_func`` to support any callable on PHP 5.3 and not only closures
ok -- where is the CHANGELOG file? I see only CHANGELOG-2.1.md and CHANGELOG-2.0.md
I mean line above :open_mouth:
What do you call a config file? Changing this `gh_values` will work, however it is not a config file. The bot already supports a config file in the `.sympy` directory.
大変(Railsの current_user の実装に)よくあるこの文だけど、個人的にこの文は左右に条件があって混乱するので避けた方がいい気はしてるんですよねえ。
あるいはUser の前で改行するだけでもだいぶ見やすい
I guess you can go ahead and merge in the regex fix.
is `opts.date` on purpose? or is it supposed to be `opts.datePrefix`? or both? (also, how about ISO8601 format?)
When tuples aren't natural, use sequence the statements instead. While correct, it isn't obvious to the reader that the order is br.readInt24, br.readByte).

Also, since these are side-effecting, they should be furnished with `()`'s:

```scala
val length = br.readInt24()
val seq = br.readByte()
```
How about `rand(20) == 0`? Does the same thing without a range.
Doc could be improved:

 "Compute the distance map for all accepted points."
`VALID_USERNAME.match(params[0].replace("*","").replace("?","")` Would that not work?
No, we don't. I needed them while test project was standalone. I've removed them.