labofsahil/github-event-dataset-2012 huggingface dataset:
Fix code highlighting on _site/organizing-backbone-using-modules/index.html

Replace double quote with single quote.
Fix Travis

Currently failing: https://travis-ci.org/thoughtbot/factory_girl_rails/builds/3442038

Maybe this line from Factory Girl will fix it: https://github.com/thoughtbot/factory_girl/blob/master/.travis.yml#L7
Merged Agendas into application
A simple but useful example

Backups all cards of an organization to JSON files.
Add support for other boards, examples and keywords

Add support for Teensy (timer4) and Arduino Mega
Add example
Add keywords
Datbase work, reviews can now be added to the database
Arabic support
Add Cascades to page

Need to batch these operations a bit more rationally
LPS-27214 Organization Roles are not clickable on pop-up in User Information
Pulling the latest from master

Pulling the latest from master
Upgraded to SnakeYAML 1.10
Typo

Just fixing a typo
Fix #1019 by correcting the index for substring calls.

The calls to substring should not be adjusted (-1 or +1).
I tested and this code works for short descriptions matching following patterns:
`<image>`
`<image>Text`
`Text<image>Text`
`Text<image>Text<image>Text`
T xsd docos

https://issues.jboss.org/browse/ISPN-1733
https://issues.jboss.org/browse/ISPN-1734
https://issues.jboss.org/browse/ISPN-1735
Print CSS

See https://github.com/rossp/django-helpdesk/issues/91
Browser module

I moved Cookies/Internal.hs to Cookies.hs because the design is simpler and still solves the problem I was trying to solve by making Internal.hs in the first place.

Broke up receiveSetCookie into two composite functions because the Browser needed them. Exported the two functions so other people can re-implement.

This is intended to be a "first pass" and only implements a few useful browser-based features.

Any comments you have will be greatly appreciated.
Add Windows 2000 support

* Windows 2000 doesn't have RtlCaptureContext

`RtlCaptureContext` from `core.sys.windows.stacktrace` makes every D progrum unlaunchable on *Windows 2000* for a long time (see [Issue 6024](http://d.puremagic.com/issues/show_bug.cgi?id=6024)).
INT 2612: Support for MongoDB Inbound/Outbound adapters

INT 2612 looks to add support for Mongo DB adapters there are two sub tasks to this
  * Outbound adapters
  * Inbound adapters

Currently only outbound adapters are developed and the PR will be subsequently updated to include the inbound adapters too.

https://jira.springsource.org/browse/INT-2612
LPS-31808

If only one reminder is set, it is always set as the firstReminder so we don't need to check secondReminder.
Some Basic Test

I am on the way to writing more useful tests, but these simple ones got written on the way.

What is the preferred way to add a test-only dependency to dzil?
picture not returned for google oauth

Don't know if this is the case for all authentications...but no picture returned for newly created user during oauth request
Pgfixes rebased with tomcat7

upgraded cookbook 'app_tomcat' to support Tomcat version 7 with existing version 6, this includes db support for mysql and postgresql in three-tier.
optimize SymfonyRequirements

fix phpdoc of SymfonyRequirements
improve the requirement checks
- reordered the requirements to make more sense
- improve wording, e.g. a test message like "OK  Your project might not work properly ..." is very confusing when the requirement is fulfilled.
- made some tests conditional so that a requirement is only checked when necessary (otherwise it's misleading). E.g. only check Suhosin config, when Suhosin is installed.
- fixed compatibility with PHP 5.2
Minor rpc8e improvements

I've made some changes to the rpc8e collection of code and the like, hopefully good ones. The original test file -should- still compile and function fine, although admittedly I hadn't tested it against my last revision. I've also added a few new things that should help facilitate expansion of the library. I hadn't started integrating anything new because I'm not quite qualified to write anything big and new, like a MCFS filesystem driver, at least not without some guidance. Hopefully, however, you'll find these improvements worthwhile.
Fixing little bug in coffee

On our browser the button was switching back after it was set.
some event were fired twice on one mouse click
text-shadow not handling negative offsets

There was a bug where "text-shadow: -1px 0 black, 0 1px black, 1px 0 black, 0 -1px black" would only draw two shadows.  The negative numbers were turned into zero by the parsing routine.
Great rename (again)

Some phpdoc only changes that I'd thought had gotten into the last pull request
Removed the support for jQuery - we don't use it that way

Add the support for a noConflict methods
use Net::HTTP::Proxy and allow configuration of the proxy host and proxy port

Adds configuration options for proxy_host and proxy_port, and uses Net::HTTP::Proxy instead of Net::HTTP so that it works for apps that are behind a proxy server
upload with listener and checksum upload, item permissions
a couple of bug fixes, and some refactoring

This should fix issues 7 and 8.  Also some minor refactoring.
Add section for test-one to readme
Synchronized with the Cyberbotics svn

Now compatible with Webots 7 (beta)
Fix coffeescript compilation error in spec.coffee

2.2.0 has a much more sophisticated parser. Empty ifs are no longer
valid syntax. Use a no-op function to demonstrate validate coffeescript
parsing.
Create boot method

I've created an empty boot method just to comply with the Silex\ServiceProviderInterface. It works now with Siles v.1.0-DEV.
First stab at new function-based view decorators

First stab at improved decorators for function-based views, for discussion.

Basic idea:

```python
@api_view(['GET', 'POST'])
@renderer_classes([JSONRenderer])
def my_view(request): # request is now a DRF Request!
    # view logic here
    return Response({'data': 'woohoo'})
```

The order of the stacked decorators doesn't matter.

Throttling and permissions don't seem to work just yet (I can't make the tests pass, haven't investigated much more).

One thought for now: I think the `api_view` decorator taking a list of HTTP methods might be a bit odd. What do you think about:

```python
@api_view
@allowed_methods(['GET', 'POST'])
@renderer_classes([JSONRenderer])
def my_view(request):
    # etc...
```

Strictly, `api_view` wouldn't be necessary at all in this case (using any of the decorators is enough to wrap up your function) but it would make the very simplest case (a function that responds to any HTTP method) really neat:

```python
@api_view
def my_view(request):
    return Response({'data': 'woohoo'})
```
fix invalid escape sequences and references to $this->send_modifier

I've made two changes to your source. The first may mot affect running code at this point, but Zend Studio pints out that \u is an invalid escape sequence in double quotes (I presume because there is abiguity around whether \u is the escape or whether you mean to indicate a unicode character). Same for \n in regexes that are double quoted. The first patch just fixes those so Zend no longer warns about that code.

The second patch is for send_modifier() which Zend reports as undefined. From he context, I take it that you meant $this->send_modifier()

I hope you find these useful. I know I've enjoyed your library, so thanks for the good work.

-- 
Karl
dev1 changes to readme
Added semi-transparent shadows for the Mummy.

Added semi-transparent shadows for the Mummy.
Fix mapred_system's default value for pipe (must be a symbol)

If `mapred_system's`'s value in `attribtues/kv.rb` is a string it ends up as `"pipe"` in the config.

`console.log` will show the following:

~~~
[warning] <0.122.0> Unknown value for riak_kv:mapred_system:
   "pipe"
Defaulting to 'legacy'.
~~~

If the value is a symbol it'll become `pipe` which is the correct value.
Added support for exclusion of child elements, converted px to array called params

Due to the limited support for inline-block of older versions of IE I needed to add an exclusion for my '.clear' div. simple solution really just added a join to the array and dumped it into the .not() on the child iterator.
Fluid 4704: fully functioning mammals demo with GPII preferences

The mammals demo currently works with the development version of node server that reads the user preference from the local file.
Add 'Lives' page

Add 'Lives' page for Ustream.
Untitled

added newline option.
Changed crit setting
Collapse ImageThread into ProgressThread

Fixes #51
stream.publish attachment href
Fix an error with iframe

FIxing it
Fixed the example

Okay the example is now working properly. Hope you enjoyed your holiday.
fixing up system.prop for better 3g possible agps fix and commenting for...

... cleaning up lines that are just not neccessary
Fix a method attribute mismatch warning.

This shows up when compiling with `-Wmismatched-method-attributes`.
fix Issue 143,1161,1441,2225,2775,2830,6180

- Check protection whenever resolving Identifiers.

- Check all symbols not only declarations, this includes UDT.

- Protection peephole for template arguments so that
  they can be instatiated with private symbols.

requires: D-Programming-Language/phobos#413 (dawgfoto/phobos@a55c4bf)
added ian's video
keep locale default value in the i18n objects

patch for issue #86
Fix for Issue #110
Switchyard 568

https://issues.jboss.org/browse/SWITCHYARD-568
Better solr update error reporting

As discussed on IRC, this makes an immediate improvement for me but might make more sense to refactor into a common method on the base class. Perhaps do that when updating for 2.x?
[TIMOB-7457] Add deprecation warning - movieControlStyle/Mode

[TIMOB-7457](https://jira.appcelerator.org/browse/TIMOB-7457)
speedup unique impressions

When using the :unique option, then a ```impressions.count == 0```is used to validate uniqueness. This requires a full table scan every time (without appropriate indexes):

```SELECT COUNT(*) FROM `impressions````

 Using ```!impressions.exists?``` results in a much more efficient query (but even requires a full table scan when there is no existing impression):

```SELECT 1 FROM `impressions` LIMIT 1```
turned into a standalone ELPA-compliant package

I uploaded it to [marmalade as well](http://marmalade-repo.org/packages/color-theme-cobalt/0.0.2).  If you have an account on mamalade I can add you as an owner.  Thanks for the nice theme!
Update res/values-zh-rCN/strings.xml

fix:some build error
Making consumer threadsafe
Fixes for a couple of typos

Hello.

Thank you for releasing a stable version of the spec!  I noticed a few typos while reading through it today and figured I would take the opportunity to exercise the power of social coding and offer a pull request.

-Chris
Openengsb 1931 

based on 2190. strange blueprint exception vanished when doing it in its own bundle.
Fixed labeling

Wrapped fieldset in tabset for consistency and improved labeling.
Optimized Certificate SQL Query 

Removed timecreated subquery.
Added Pagination.
Added two constants used for pagination.

Hi Pukunui,
We had a client who had 10K+ certificates in a course and it was taking over 5+ minutes to list certificates. The query should now take  less than 4 seconds.

I also added pagination to the list all certificates page.

Tested on Moodle 2.2.3+

Thanks,
bunwich
Minor changes and resurrecting ping/pong benchmark.

I haven't committed in a long while so I thought I'd run this through you first, Edsko.
LPS-24786

http://issues.liferay.com/browse/LPS-24786

Moved the test to:
portalweb/portal/controlpanel/socialactivities/usecase/blogsentry
It works now...

Here's the changes you asked for, plus it works now.  Though occasionally you'll get a 408 timeout.
Requested fixes and PR to develop brance
Bug 2930

Fixing the issue with case in th table uniqueId
Di nunzio tom

Bonjour,

Voici mon projet. 

Bonne correction !

Tom Di Nunzio
Showing readme some love

Adding readme
You shouldn't mix joda-time and Calendar
We normally don't merge branches. I for myself just cherry-pick from my dev branch. But there are different approaches out there.
thanks for the reminder, will scrap the currently provided default argument which enables running without providing a Timeout instance
Agreed.  I'll put together a patch shortly.
NOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOO!
Nothing but test and whitespace changes. The tests are the clarify you can add packages
Oh. This was just missing an ```=``` because I'm retarded.
great, just a tip for future patches, keep cosmetic changes separate so its easy to review what has been changed, I will still accept this however as we discussed the changes as they were made :)
Write  a comment below the change bu welkang on Feb,8,2012
Ah ok got it thanks.
There is no difference - I got rid of "context" as part of pull request #126, and merged your changes.
Well it was a submodule. So if you ignore the submodule you end up deleting it just like this. This ended up getting pushed upstream and broken endpoint manager....for now. Will talk to Darren about a better way to do this after talking to Jort.
For Problem 2: Check out Ian's https://github.com/igrabes/Euler-Project/blob/master/euler2.rb

We noticed that his solution takes a long time, too, because the algorithm loops through each prior fib(1..n) each time it's called. For instance for fib(4), it runs fib(1), then fib(2), fib(3) to get to fib(4). And his line 5 fib(n-1) + fib(n-2) really means the algorithm runs fib(n) twice before it gets the sum.

Yours is getting the last two elements in the vals array. The parallel I see is that the algorithm has to run through 34 times before it reaches 4E6; each time getting the sum of last two elements in vals; so it might take about the same time as Ian's.

One way to get around printing 'sum' each time it loops through line 11 is to do this in a block and print only the 'last' sum:

line 11: yield sum # instead of printing the sum each loop
line 12: end
line 13
line 14: fibs = []
line 15: fib(4E6) { |f| fibs << f } # => array of fibonacci numbers' sum
line 16: print fibs.last # => should give you the last sum, which is the answer
Just curious - what's the purpose for having a date as well as the start and end times? 

I don't think ActiveRecord / ActiveSupport support having detached 'time' types - the time is always married to a date, IIRC. could possibly remove session_date and change the following columns to :start_time and :end_time. Would allow for multi-day events, if those ever happen, as well.
If you think it'll take >10 minutes, forget it.
Are you sure this legacy way is faster? I'm not convinced on that.
@Th0ar 看上面
It is neither a TODO nor a Hack.
Also fixed defaults interfering with original placement
We can do that if the non-core libraries are actually separate somehow; they are still built into the system image with everything else.
Yeah, right. add all numbers but return just the last element. :P
@wichert: do you know what caused that change of syntax and whether my change is correct?
Edoardo Prezioso wrote:
> Since it's a C problem, what about if I add 'bool isCcode = isC();' before loop and add 'isCcode' condition before checking 'using' string?
>
>    
C detection in cppcheck doesn't work well.  It was common in the 90's on 
unix systems to use the extension .C for C++ files but cppcheck gets 
that wrong and has no way to override it's mistake.  But this is a whole 
different issue that still remains unresolved.
Hmmm, you're right. Not sure why it wasn't blowing up then!

Pushing a quick fix shortly.
Not actually a logic correction but an optimization. So warning should be really fixed now.
This is optimized into a loop through tail-recursion, I think.
It would be very nice, if the default options are configurable. Because it would be very easy to fix backwards compatible breaks here.
don't copy&paste...
don't make 3 methods doing the same thing...
you can either make a procedure that moves the whole tree once it is "boxed"
or pass a parameter x,y - where the root should be
Would `$project_name.pdb` work here?
Windows says: "ImportError: No module named crypt"
As far as i understand it, Python does not come with the crypt module on Windows. What i found is this 3rd-party-module: http://www.voidspace.org.uk/python/modules.shtml#pycrypto
Do you think this should be recommended as the preferred module?
Thanks, It's great. I added one line at line 531 in script.js: 
    columnsStateCallback: null,

 But switch the grid  page, the columnsState will be lost.
FYI, item.GetVideoInfoTag()->GetPath() is using the exact same logic. Maybe you should use that one.
So in general, this is creating some weird issues with respect to incremental tree construction.  Some of these issues are apparent in the code required to make the parsing work.  If you instantiate XML nodes in code, you will be building the tree bottom-up.  However, in order to build an `Elem`, you need namespace information from the *parents* (which you haven't built yet).  So, the build process is this weird back and forth, bottom-up and top-down, depending on information.

Additionally, and maybe more significantly, this tangles scope structures with the (still-to-be-built) ancestor structure.  What happens if you build an ancestry that contradicts the scope structure?  What if that contradiction is shadowed by subsequent definitions so it doesn't *actually* affect the `Elem`?  (e.g. declare a scope ordering of `"foo" :: "bar" :: "foo" :: "baz"` but introduce namespaces in the ancestry in order as `"foo" :: "bar" :: "baz" :: "foo"`)

Do we really need to know the full scoping and shadowing that led to the specific `NamespaceBinding` for this particular `Elem`?  Why isn't a flat `Map`-like structure of the current scope sufficient?
Maybe it could be parameter to `object.add` / `object.remove`?

```javascript
var bakeTransform = true;

// 1
parent.remove( child, bakeTransform );
scene.add( child );

// 2
scene.remove( child );
parent.add( child, bakeTransform );
```

Or differently named variant of `add` / `remove`?

```javascript
// 1
parent.detach( child );
scene.add( child );

// 2
scene.remove( child );
parent.attach( child );
```
"Manage contact details"?
Hmm, could you try 1.2.2? I remember fixing something related to this.
as i said,the original author is Vlad and i modified it...dont know if signfinder did any changes,but i havent noticed any
The thing is, not every user of composer has gz enabled, so we can't gzip the phar, no matter whether the machine building the phar has gzip or not.
Yes, the oEmbed tags would only be output on an individual photo's page. Allowing a plugin to define an endpoint would be great as well.
Actually, it looks like now airbrake is just completely broken, and not being loaded in the middleware stack at all (Airbrake::Rack)

@MrZYX  Did you test this pull request with Airbrake before you pulled it in?
Yes, because FlowContainer extends Container, and Container just cares it's own size(to draw it's bg). So changes in it's content doesn't matter container's size.
This needs to be reverted.  JGroups requires the logical name of a channel to be unique both per node and within a server.
alternatively, you could do: `self.client.get(from_url, dict(userlang=to_locale), follow=True)`
shouldn't this be `IconUrl`?
Right, I figured it wouldn't cause a big problem to re-setup the test server (due to its idempotency), but it does add some additional processing.   I imagine the difference in the speed of the test suite wouldn't even be noticeable but it still a good practice not to add additional unnecessary processing.
I could be wrong but I believe that in the case of a Resque::Dirty exit this won't happen as the child forked process gets killed.
That ensure block will handle the case where the job actually pukes it's own exception though.
Not really, if you look through the Report there is more *.scr files, which isn't settings files but rather files required and some auto-generated by the game. same goes for the others I've added with *.scr files.
FullTextSession would not be able to extend both, so you would need to
still use delegation for the one you would not extend.
On Mar 8, 2012 1:20 PM, "Hardy Ferentschik" <
reply@reply.github.com>
wrote:
>
> Do you have to roll it up into one class? Could it not be two? Then we
would have a _DelgatingSession_ and a _DelegatingSessionImplementor_. It
think at least name wise it would be much clearer.
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/Sanne/hibernate-orm/commit/51eea2c551c315b1d9705775e4af0c162321b5d1#commitcomment-1058301
@xonecas I could not find this package in the npm repository. Is it in there under "disqus-client" or something else?
Friend, why have you revert TC change in Core/PacketIO: Prevented excessive WhoOpcode spam - solves possible crash/lag method?
No no no no no!

    "Damageable": 0

We use booleans for that kind of stuff!
Ah I had forgotten about all that; I haven't been using cfquery much. 
Thanks. 

I'll take care of it.
> Whenever You find some bug and You know how to fix it - go ahead, dont report, just fix it Yourself in our repo :)
And how about upload e. g. my fixeson our R2 repo?
IMO we need to say that we refused to serve the template too since otherwise when you read this you don't know the state of your system. The user needs to know it's safe I think.
I think "mongoReady" should probably be "waitingForMongo" instead.
Feel free to patch it.

In my team we like to just throw around comments like "Check out #542" in the chat and dont use anything else that would get in the way.
But I could understand it being annoying if you do.
The above two lines should be wrapped in "if (!push) { ... }", else they'll lead to an exception when starting a push.
I know the comments etc don't seem to be lines up correctly, they looked ok in Notepad++
Thanks. A test should also be failing for this one. Investigating.
Hrm, how does this help with duplicated keywords in existing setups? If the keywords have been already assigned duplicate entries how can we delete them?
This could also be written using only wiki markup in syntax 2.1
Hmm actually, the correction isn't what I wanted to accomplish. Oh well, doesn't matter anymore.
This change causes some notices:

> Notice: Undefined index: availability
> Notice: Undefined index: countries
Created https://github.com/HumbleSoftware/Flotr2/issues/30
The only place that I know of that has big lines is predefinedentities.py because it would be an awful mess if it wasn't. Even that, though, is going away in the coming months. If you see anything >80 chars, feel free to wrap it.
this needs to be set to n
Oh dear…
This is useful in the changes I've been making to how submit_video works - essentially, I want to be able to instantiate an incomplete video, use it as the starting instance for a ModelForm, then let the user fill in the blanks. You're right that I overlooked the use of from_vidscraper_video in livesearch. Thanks.
Yep, fixed that as well. I was waiting for a build on manager to finish to test that it works correctly first.
Why do you need to sort them?
Will do extensions - cheers.
its very complicated quest, thanks
> `/\\[0-7]/`

Also wrong. You need to allow `"\0"` `"\\1"` etc.
Removing a listener fires an event?  Or do you mean there should be a method to remove a ThrowableCaught from the list being maintained, and that should fire an event?
uninitialized constant ApplicationController::SessionHelper
can you help me
This method has to do with the state of the board not the rules themselves. This makes more sense as a public instance method on the board object.
Hey back!
Oh sorry, missed that this is on the 1.5 branch.
I believe the overhead of calling each of them separately is not big, even with millions of examples but I'm fine with keeping it public if you prefer.

The recent PR introduced a new parameter `average="weighted"` to `precision_recall_fscore_support`. You need to change it to `None` by default if you want the old behavior.
this uses a borwser id interface it should be moved there
Oh, yeah. That's right. It's not a well-formed program when it contains an octal literal in strict mode code. We'll have to use something that would still be well-formed but would cause a runtime error. I've got it! No need for backticks:

    ok !this

Bam. In case you forgot:

```
coffee> do -> !this
false
coffee> do -> "use strict"; !this
true
```
no definition in class declaration.
Can be either inline inline int getNum(int i);
with implementation under the class declaration
or implementation in .cxx file

getNum is not Slicer style compliant (should start with uppercase)
You might prefer a more explicit name:
e.g. GetNumberOfDICOMSeries()
(by reading the code I don't know what number it is.)
Looks good, although I'm not 100% sure about what I'm reading.  Not sure if @hafriedlander has any thoughts.
Move this call to inside the if condition by reversing the condition.
Change method name to *updateVolumeStatusInDb*
I think I found one issue with this: when the script tries to "merge" meshes into one objects (in case there are multiple meshes in a nif), we add a new vertex_color layer (names: "VertexColor, VertexColor.001, ...). There is a boundary, we cannot create more than 8 layers of the vertex colors. When the script hits that case, the method returns None object and there is an exceptions caught when the script tries to access b_vertcolorlayer. 

I got this case when importing my infamous cave object using the import_nif.py script from my script repo. Commit SHA: a5fcbbb7040590ca34a01fbafc2048cfc95d1b79
`notification` seemed pretty clear, why shorten it? If you're concerned about length, I'd rather see `n` here than `not`. "not" has meaning that you probably didn't intend (`isPossible.notification = not`? Not what? :) ).
thanks for the input uwe, i fixed this in 86faf95d2f0900b027238316baddfbbe93945737 now
The development server does not belong in code comments.
Also, sorry to be a pain. I am a bit overwhelmed with how widely used the Customizr is getting. And Radiance has become a popular theme so I am fixing it on a daily basis so that it plays nice with the Customizr. The Customizr has saved a lot of clients from leaving Shopify to another platform.
+1, this is the right fix. When you map an entity with a different name than a database table (i.e when overriding ModelBuilder entitites), we need to resolve to what gets set in ToTable() mapping. This code does exactly that.
Hurrah! This was my bug report :)
> add a context object to these methods

Do you think about some particular use case requiring such sort of context?

> let them accept a `ValidateGroups` annotation instance

I don't think I like that. If integrators aren't based on this annotation they'd need some way to instantiate it in order to pass it to `Validator`.

> or let BV's engine find and read `@ValidateGroups` annotations entirely but you don't seem fan of that approach

No, not really :) This should be the task of other technologies specifically targeted at controlling interceptors IMO.
FYI, if you surround the address with a <pre> tag, then line breaks won't be ignored.
git rm recovery/recovery_ui.c (or just comment out the line in BoardConfig.mk, it won't be used then :P
Might want to ask Kali. I put the question out there on his submission but
he may still be mad for being called on abuse of power.
On Feb 23, 2012 11:44 AM, "LorD ClockaN" <
reply@reply.github.com>
wrote:

> I tried that yesterday.. will try again
>
> also I tried compiling and what ever I do vendor/qcom doesn't get
> compiled... I have all the flags set..
>
> I do't get those libOMX files at all in the end of compilation
>
> any suggestion??
>
> On 02/23/2012 05:42 PM, 0002647302 wrote:
> > Removing the audio ifdef. The one you claimed fixed it.
> > On Feb 23, 2012 11:34 AM, "LorD ClockaN"<
> > reply@reply.github.com>
> > wrote:
> >
> >> so you are telling me that you have working mic with new drivers??
> >> you remember what commit fixed it?
> >>
> >> ---
> >> Reply to this email directly or view it on GitHub:
> >>
> >>
> https://github.com/TwistedPlayground/android_frameworks_base/commit/830100747ee6630fbedfda77c83e132d9fd90c81#commitcomment-1003173
> >>
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/TwistedPlayground/android_frameworks_base/commit/830100747ee6630fbedfda77c83e132d9fd90c81#commitcomment-1003232
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/TwistedPlayground/android_frameworks_base/commit/830100747ee6630fbedfda77c83e132d9fd90c81#commitcomment-1003248
>
Definitely gave a version error with Java 6
Why not `std::string::operator==()`?
As part of the "data access in YUI" refactor, we should determine if we want to allow different "rights" when this module is run in Node.js. That said, I totally agree with forcing `"GET"` now because that's all the `yql` module claims to support.
Use ```===```.
Ah yes, the DOM and all its glory :)

I just wanted to be sure there wasn't a suitable way without having to sniff the browser.
Reinstated these comments in ff53c9a
right; I wanted to keep the “keep ’em separated” semantics which we had previously; not sure who could possibly depend on that, though
There is only one child.
yes, the scenario is that the singleton cluster node sends join and then later receives a gossip, which includes him in the member ring
Would be nice to have a linter for stuff like this.
Well, adding another field in the class is not optimal at all.
And also you're saying that the Promises' callbacks should be executed on the default dispatcher, which is probably not what any user would expect.

just do:

Promise.successful(ch)(context.dispatcher) if it's only needed in one place,

or do:

import context.dispatcher

at the most appropriate places otherwise.

If that's the only place that needs
nooooooo please cant we have this somewhere else?
hmm, I actually find that kind-a nice …
File needs a bit of docs
```
:func:`dipy.reconst.dsi.half_to_full_qspace`
```
sorry to be unclear, ditto for FilePathValue(), and, of course, succeeding `if` statement.
Yea that was just noise when I was testing meant re-enable that.
4 space spacing, not 6.
I may be wrong, but isn't this call messing with the returned Inputstream ?
Yes, it is in quite a few places. Busy changing it now.

Also need tow ait for a response from Prof to change something else, so
don't merge yet after I have fixed this. I noticed a problem after the pull
request was sent.

On Thu, Oct 18, 2012 at 5:28 PM, Filipe Nepomuceno <notifications@github.com
> wrote:

> In
> library/src/main/java/net/sourceforge/cilib/pso/multiswarm/CooperativeMultiswarmIterationStrategy.java:
>
> > @@ -106,12 +106,17 @@ public void performIteration(MultiPopulationBasedAlgorithm algorithm) {
> >                          particleWithContext.calculateFitness();
> >
> >                          if(particleWithContext.getFitness().getValue() < particleWithContext.
> getBestFitness().getValue()) {
> > -                            particleWithContext.getProperties().put(EntityType.Particle.BEST_POSITION, particleWithContext.getPosition());
> > -                            particleWithContext.getProperties().put(EntityType.Particle.BEST_FITNESS, particleWithContext.getFitness());
> > +                            particleWithContext.getProperties().put(EntityType.Particle.BEST_POSITION, particleWithContext.getPosition()).getClone();
> > +                            particleWithContext.getProperties().put(EntityType.Particle.BEST_FITNESS, particleWithContext.getFitness()).getClone();
> >                          }
> >
> >                          if(particleWithContext.getBestFitness().getValue() < contextParticle.getFitness().getValue()) {
>
> and here
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cilib/cilib/pull/86/files#r1880006>.
>
>



-- 
Regards,
Kristina Georgieva
MSc Computer Science
CILib Research Assistant
I've added ```::setExtendedErrorHandling()``` just for that. Even if it throws the error it would be less expensive that before and it will require minimum changes to the existing code base.

I'll dig down this weekend more optimization possibilities using XHProf.
@galaxas0 just came across it: https://developer.apple.com/videos/wwdc/2012/?id=405
Or mayb, @maxgoedjen, you could make it a static inline C-function and have it return a CGRect.  That would certainly be faster than messaging.
ca bogais avec cette ligne...
Retire la classe btn-info. Pour les back et annuler, on met le style par défaut (gris).
Not obvious why this tx will fail. Maybe add a comment "Node still has relationships"
well, here is the question what to return if there is no definition. DiscoveredReflection throws Exception. Maybe this is better. Nevertheles, the approach should be the same.
We don't do method chaining in `ByteBuf` yet.  It should be part of other commit / pull request in my opinion.
Chris,

I still don't know where things go in your organizational scheme. I'll put it where ever you think it should go.

Hans

From: Chris Filo Gorgolewski <notifications@github.com<mailto:notifications@github.com>>
Reply-To: nipy/nipype <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Sunday, November 18, 2012 9:12 AM
To: nipy/nipype <nipype@noreply.github.com<mailto:nipype@noreply.github.com>>
Cc: Hans Johnson <hans.j.johnson@gmail.com<mailto:hans.j.johnson@gmail.com>>
Subject: Re: [nipype] Enh/update ants registration capabilities (#484)


In nipype/interfaces/ants/utils.py:

> +                            desc='image dimension (2 or 3)', position=1)
> +    out_postfix = traits.Str('_cmt', argstr='%s', usedefault=True, position=2,
> +                             desc=('Postfix that is prepended to all output '
> +                                   'files (default = _cmt)'))
> +    reference_image = File(argstr='-R %s', xor=['tightest_box'], position=3,
> +                                desc='reference image space that you wish to warp INTO')
> +    transformation_series = InputMultiPath(File(exists=True, copyfile=False), argstr='%s',
> +                             desc='transformation file(s) to be applied',
> +                             mandatory=True, copyfile=False)
> +    invert_affine = traits.List(traits.Int,
> +                    desc=('List of Affine transformations to invert. '
> +                          'E.g.: [1,4,5] inverts the 1st, 4th, and 5th Affines '
> +                          'found in transformation_series'))
> +
> +class ComposeMultiTransformOutputSpec(TraitedSpec):
> +    output_image = File(exists=True, desc='Warped image')


Shouldn't this be "warp/transformation file"?

—
Reply to this email directly or view it on GitHub<https://github.com/nipy/nipype/pull/484/files#r2161304>.



________________________________
Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged.  If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited.  Please reply to the sender that you have received the message in error, then delete it.  Thank you.
________________________________
Afaict, the second push is the important one. It's the first one that I was questioning. But, I could be wrong. Here's my logic:

1. Translate to the center of the screen
2. Scale and rotate from there
3. Push those changes down to bake them into the MV matrix so that other transformations won't be affected or vice-versa
4. Translate that point to appropriate texture-drawing space.

I thought step 3 was the necessary step, but maybe its just redundant. I'll have to actually try removing it to see if it screws up or not.
80 character limit on line length.
Oh, I see, you're actually setting them to None most of the time. But since you have to handle the NULL case anyway, can we just make that be the "unset" value?
typo - extra ) here
My pull req for coercion is still open - see https://github.com/buglabs/node-xml2json/pull/16. So, this breaks type coercion. Can you clarify?
это нужно для создания линков только в том случае, когда мы вытаскиваем из линкера в сам мастер, либо когда зажимаем правой кнопкой и в себя. этого в qreal/master просто не было. там линк улетал на небеса.
this is loaded already inside of ```rack/session/dalli```
@joshsusser I totally respect you *think* it is more readable *for your taste*. But we don't agree, and we do not want that cosmetic change in the middle of this PR that we otherwise like as a feature proposal. Could you please submit the patch without that change please?
Also, new CHANGELOG entries go to the top :up:
@rafaelfranca do you prefer an array containing the methods?

I don't feel good making a hash with 3 symbols and calling the `any?` and after that checking the method.

I think it's too much computation for a simple thing.

Anyway, that is not a "performance issue spot", I just want to be sure before changing :)
You're right. Testing it for class and STI type now.
I believe `setError` sets the type anyway.
...although that is not really sufficiently inclusive.  It's more like isImplementationArtifact.
Perhaps we should give a code example here of how to use `push` or whatever we want to call it.
I just checked what happens to `unapply`. The `UnApply` tree in a pattern has an `Apply` with an `Extractor.unapply` `Select`. Here the `Extractor` `Ident` simply has the same range position as the  `Extractor.unapply` `Select`. To me that also doesn't look right..

I'm not sure what to do, in my understanding of positions, both `apply` and `unapply` should be transparent positions and their underlying `Ident` or `Select` trees should have a range position. But if that prevents the IDE from hyperlinking directly to the definition of `apply`, then maybe we should just leave this as it is now.
backticks.
How about leaving h1.search-results-title with H6 and I will rework general semantics later. Really want to give green light on this pull request. The general semantics of all core theme needs to be reviewed anyway.
Looks good! Could you just add a few lines here to explain what is going on here and in the following lines?
And what if I want a `tuple` :). Google for "isinstance considered harmful". This would be better with a `try except` block: you try to use it as a list, when it fails with a `TypeError` you use it as an int. You leave al other errors to propagate.

```
In [1]: a = 1

In [2]: a[1]
TypeError: 'int' object has no attribute '__getitem__'

In [3]: [i for i in a]
TypeError: 'int' object is not iterable
```

Or you can check for an attribute: `hasattr(obj, '__getitem__')` or whatever.

Also I think that here you have duplication of code. Why not just transform your `int` into a `list` and aways use the list code.
>
> ok, makes sense.
>
>
I'll start reviewing your work this evening (in about 7 hours). If we don't
have power at that time, though, you won't see anything until we do.

Thanks for looking over the ode work. That brought my count down to zero at
https://github.com/sympy/sympy/pulls/smichr .
There currently isn't a way to get out basis states, since they're just ``|j,m>`` for ``m`` in ``(-j,j)``, whenever I needed basis states, I just generated them. There is an ``.basis`` property for the spin operators, but it is a string used for doing ``.rewrite`` when applying states in non-default bases. I'd thought about trying to dovetail something that would give basis states for discrete bases into Tomo's represent stuff when that was merged. There are ``matrix_element`` methods, but those are only defined for the default basis, so generalizing them to change of basis would be tricky.

If the ``.basis`` property could be worked into a Ket, that might be used to generate the diagonal elements and easily allow for a different basis to be defined by the kwargs for doing the trace. That would at least work for most cases without having to do anything with getting such functionality into the represent PR.
> Note that `FiniteSet` is derived from `Set`, and inherits `Set.sort_key`.  With the current version in `master`, I have those old hash-randomisation dependent sort keys back in #1429.
>

You are right. sort_key tries to give a deterministic key for sorting
(e.g. dict, and now set, are sorted by default_sort_key) so any
object's sort_key should be ordering its args if they can be
system-dependent. (That means that Add and Mul's sort_key should use
_sorted_args, too, I believe.)
You can redefine the str function to whatever you want. I'd say using the { } braces is prettier (even if you are just wrapping a FiniteSet).
> Yii convention says the prefix “$_” is used for any not public property, this means both “private” and “protected”.

CManyManyRelation would be the first place with a protected property with $_ then. Nowhere else exept in vendor directory will you find such a think.

If you think it should be protected then remove the underscore. But I am not sure if there is a proper use case for exteding this class.
correct. Oauth does not version.
I reverted my changes to JCC because something was broken in it
Put each clause on its own line, easier to read. E.g.:

```c
if ((handle->cf_flags & UV_FS_EVENT_RECURSIVE) == 0 &&
    pos != NULL &&
    pos != path + 1 &&
    *(pos - 1) != '\\')
```
Что это? :smile:
beware of trailing spaces :)
I wonder if this shouldn't attempt to cleanup the mess it created before exceptioning out?
has been is passive.
Just don't implement the perform function. It need the perform or c_code. When there is c_code, you don't need to implement the perform, but you can if you with to help debug. In this case, I would just don't do it.
Still must be escaped.
yes I know how they work, but I dont see a reason for a "all or nothing" in here.
FreeBSD has no pidof. It does, however, have pgrep.
How is that relevant to this PR?
Here you are dropping `extension_allowed()` which basically does what `phpbb_check_attach_extensions()` does. The first should be used.
Yep, `.push` please.
Indeed, will do both, so you can configure it if you want to, and it'll pick something uncommon otherwise.
There is a big problem with this approach: we don't save 'seen' activities into database, AFAIR.
Won't work. HTTP_REFERER is set by browser. Should've removed it. See commented cURL part for detecting redirection. But the browser will detect first.
Highcharts does allow it for 'non-commercial' use. What exactly that means for our purposes, I'm not sure. Highcharts isn't a huge deal -- I wanted it for plotting the histogram, but I'm sure I could find something else (like gRaphaël) if it becomes a problem.

The non-commercial license for highcharts: http://creativecommons.org/licenses/by-nc/3.0/legalcode
slight performance gain if we cache length before the loop.
I don't like a duplication of whole algorithm inside accuracy tests. If it is avoidable then I suggest to implement tests without it.
Well, virtualbox is using its own build system, but uses make for their kernel modules, so in fact the buildPhase was essentially just building the kernel modules, while virtualbox itself was built earlier in preBuild
is it necessary to call ```this.getContact().getStore()```? What is the difference between this and ```this.getContact().getStore().load()```, which you call in the ```refreshList``` method below?
Note that there is no need for NotInList2 here as you aren't using it....
This should just check for `GeogCS` or `RotatedGeogCS`. (If you need to swap the order of the spherical vs. cartesian code blocks to make it more obvious then feel free.)
Shouldn't it take into account offset / axis1 / axis2 in ``__eq__`` and ``__hash__``?
Why this one returns java SQL timestamp type while the others return Volt TimestampType?
Maybe this additional optional parameter is the correct to way to enable this feature, but I guess I am a little disappointed that this wasn't accomplished instead through the use of an alternative (trivial, I suspect) variant to normalizer.py -- I suspect that the alternative normalizer approach would be more efficient -- am I correct that the current fuzzylevel processing kicks in AFTER a full pass of the default normalizer, most of whose activity is wasted effort?

Put another way, my feeling at the time when I implemented the fuzzynormalizer was that it provided a little justification for the normalizer component architecture, even though I thought that it didn't fit the model particularly well (too much cut and paste required vs. the default normalizer.

In contrast, I thought that this "count only" normalizer was going to be a clear win -- replacing, rather than duplicating, most of the normalizer behavior, but instead this was done by other means. If this case isn't a win for having a normalizer component, then I'm beginning to feel like there is no such thing. That it's time to give up on the idea of "normalizer component option" and just have different fuzzylevel values. Actually, in that case, I would suggest fuzzylevel==0 be normal exact row matching, fuzzylevel==4 be fuzzier number matching, fuzzylevel==8 being count matching (leaving room in between for other possibilities).

Not that I think we need to hold up accepting this feature largely as implemented, but there should be a discussion about packaging.
You've defined an is_string utility method, would it be clearer to call that here and above rather than replicate its ugly try-except code?
No it is not. In the code everywhere it will continue to try to allocate memory even after it first failed and even though it knows all of it would have to be de-allocated because error already happened. I consider that to be badly written code and border-line bug.
graphs_three -> graphs_tree ?
forgot to remove "sub" here but no inline styles in the page itself so no really applied.
See above - use variable and break.
Read up on dynamic loading and will add this back in with some better documentation
Automatically close an file-like objects that were assigned as content, after the response has been iterated.
I think we need to have some way to close the file - either wrap it into something explicitly closing the file when fully read, or check for hasattr(._content, "close") in stream_content finally block.
Originally I didn't added the `<url>` since it's not used anywhere on Airbrake gem, but both you and @martinciu [requested](https://github.com/errbit/errbit/pull/228#commitcomment-1747560) that, so I guess that's a good idea to keep the API fully compatible with the gem.
Not sure I like the exclamation point here. Sounds like we are yelling at people to get off our lawn.
Why are we removing this line?
Using promise always would be preferrable actually, so +1.
@jaubourg why you always so hard on me all the time? I just trying to make a good contribution. Why you need two nearly identical PR? They will depend with each other. I'm sayin this pull is for solving the ticket and to make a small optimization, this is why i put make result in PR description. If you doubt in some of the changes, i can explain them.
I agree with you. The cmsketch's comment is list as following. 

/**
 *@brief <c>cmsketch</c> is a UDA that can be run on columns of type int8, 
 * or any column that can be cast to an int8.  It produces a base64 string
 * representing a CountMin sketch: a large array of counters that is intended 
 * to be passed into a UDF like <c>cmsketch_width_histogram</c> described below.
 */
CREATE AGGREGATE MADLIB_SCHEMA.cmsketch(/*+ column */ INT8)
(
    sfunc = MADLIB_SCHEMA.__cmsketch_int8_trans,
    stype = bytea, 
    finalfunc = MADLIB_SCHEMA.__cmsketch_base64_final,
		m4_ifdef(`__GREENPLUM__', `prefunc = MADLIB_SCHEMA.__cmsketch_merge,')
    initcond = ''
);

It seems that this function only supports int8.  I need more investigation whether this method supports other types. If it can, I think it could be better to open another jira to track this.
Don't use int.