labofsahil/github-event-dataset-2012 huggingface dataset:
Pull Req: Always whitelist blackberry.event

* Make blackberry.event always available even if the developer did not whitelist it
* Make sure blackberry.event gets loaded before any other extensions get loaded

Framework Pull Request: blackberry-webworks/BB10-WebWorks-Framework#37 <-- this pull request to be closed AFTER the framework pull request is closed, the Framework submodule needs to point to the right commit

Docs Pull Request: blackberry-webworks/WebWorks-API-Docs#118
Added backgroundColor Property and Podspec file

I added a `backgroundColor` property to SJNotificationViewController to change its background color regardless of its notification level. The `backgroundColor` property also adds the ability for users the give the notification different colors per view controller that SJNotificationViewController is used in. I updated the README with the documentation on the new feature.

I also added a Podspec file so that this project can be used with [CocoaPods](http://cocoapods.org) its like Bundler for Objective-C. Check out this [tutorial on CocoaPods](http://nsscreencast.com/episodes/5-cocoapods). I thought this a great project so why not add it to CocoaPods? 

Once you accept the pull request you can remove the second `s.author` line and `s.homepage` in the podspec file.
Adds the following utility methods to simplexml.class.php

- matches: allows you to provide a pattern (preg) to match the current node against.
- starts_with: determines whether the current node starts with a given value.
- ends_with: determines whether the current node ends with a given value.

Simple additions; however, makes a lot of code that uses the SDK much more concise and fluent. For example, the is now possible:

$object->Key->ends_with('/');
Process message and process icon editor
Simulation refactor

Take a look at the examples (carinsurance) and tests to see the impact of the changes
Localization API

This change introduces the new localization API. In the new localization API the UI language setting is disjoint from the regional settings. The new API allows developers to retrieve and listen to changes in the UI language and region.
Remove csrf_protection in FilterTypeGuesser.php

The filter guesser force disabling of csrf_protection for integers. This is already done in ``Builder\DatagridBuild::getBaseDatagrid``.

This PR is required for sonata-project/SonataAdminBundle#725.
rma, solr
Bootstrap 2.0 compatibility changes

I'm not sure you should pull this directly, but it includes a lot of small tweaks to add bootstrap 2.0 compatibility.  I haven't done extensive testing.  I've just been making sure everything works on my own forms.

Hopefully this can at least serve as a guide for a few of the changes you need to make.

Keith
Add MojoShader and GLSLOptimizer libs for HLSL stuff

Still need to compile stuff for Linux

MojoShader diff is at http://pastebin.com/WkBxxbK5, for posterity
Devise render_with_scope and Devise::Controllers::InternalHelpers was removed on version 2.0.0

Devise `render_with_scope` and `Devise::Controllers::InternalHelpers`  was removed on version 2.0.0
Add LogEvent value class
separate rake task to add in geolocated data

Now to seed the db run
rake db:seed
rake db:seed_geolocated
Bugfixes. Remove cruft that was specific to Team One

Thanks to @shennyg for the tip
Validación del formulario en lado cliente.
Slovenian translations added

Slovenian translations added. Thank you and Kind Regards.
Fixed PhpStorm name for better search

Modified 'Php Storm' => 'PhpStorm'
Issues/39/enhance install doc

Minor tweaks to the README.markdown file to complement the upcoming blog post.
[SWITCHYARD-676] - Upgraded to use 0.4.0.OS1 version

https://issues.jboss.org/browse/SWITCHYARD-676
Update Joomla/Joomla/resources/ServiceConfiguration.cscfg
Important changes to support more environments / setups.
Split the exercises into separate projects
hostdaemon aktivitaetsdiagramm hinzugefuegt
added PickupIMObject action to expose picking up of segmented/recognized...

... interactive marker objects

(cherry-picked from fuerte-with-groovy-vis branch)
Auto away and logout
Separate D1/D2 writeln/writefln and string/char[] distinction

These are additional fixes for issue 4550, which can be closed after merging this. http://d.puremagic.com/issues/show_bug.cgi?id=4550
teenage mutant ninja turtles
Don't worry about this for the moment -- but don't write any new code that uses inline styles either :)
Good point- removed that line
I was in bed last night thinking about this commit, and expecting this to be your comment. ;)

You absolutely have a point, but in some applications (like the one that triggered this change), that will lead to (in this example) thousands of empty cache files on this, which is not desired either.

Maybe a better solution is to add a third boolean parameter to cached(), allowing empty results to be cached yes/no, and default yes, thus reverting to the original behaviour?
DOI capitalized.
looks good to me but I agree that this patch should be split into two.
this looks really nice indeed. a couple of comments:

1. this feature should be configurable. someone might prefer the old behaviour and should be able to easily switch to that,
even if we make this here default.

2. i would rather have `Thread.get_authors` return a list of pairs `realname, address`. it just feels more natural than a string that represents all nicely. So: why not add a new new method `Thread.get_authors_string` or something, that
accepts a list of addresses `replace_with_me` or so that defaults to `settings.get_addresses`..
also, this method should accept a boolean `replace_me` to overide the config option whether or not this is done.

3. how easy do you thing one could switch/revert this at a later point if the bindings eventually provide the discussed
"middle layer"?
probably want withCombinedPrivs
This looks like a copy-paste mistake in the docs. Should probably be different than the `Mixed` case.
lol gg
bransfor wrote:
> Like in fmin_tnc, doesn't splitting f, g = func(x, *args) into two function call double the number of function evaluations in cases where func returns a tuple of f and g?

Yes.
Drunk coding is win
Since $dir now holds the full path for each loop, we don't need to keep track of $path any more.
Oh, I forgot about the :reaping_frequency. But now the config for use within PostgreSQLAdapter doesn't have symbolized keys any more. Is this intended?
BTW I see that there is already a 0.2.8 version here https://github.com/avalanche123/Imagine/tags
Cool. Noticed it just before my commit and changed the name of the booleans list to match yours. Hopefully this does the trick, it worked in all my test cases with compound matchers in top.sls.
Don't use anonymous unions.
Is it better to use

    table.sl-break {
    }
Seems like the correct solution would be to force the symlink to be created in FormulaInstaller, if I understand this bug correctly.
"The goals for this project <i>are</i> to investigate ..."
Absolutely! Sorry about that.
i got the .exe file to work
I think we just need a space here, or am I missing something?
This is where i try to make some repo
This provides for some pretty strange animation behavior when the console is at its minimum height. Especially when the console is being shown again, the input area goes down and then up again.

It does this because the minheight attribute is set before the animation is done, so the entire tab is "pushing down" the input area.
Ok,

Then I should simply remove the enum value, I guess?

Also, would it be ok with the change requested?
Si si lo baja , lo agregue de mas . igual dejalo
This only targets xlarge-screens and not large-screens, I think?
It took me a while to even parse what "specfun" meant. Having fun speculatively? Functions paid for on spec?
I'm doing some test with: https://gist.github.com/1921669 
(cf. http://www.blender.org/documentation/blender_python_api_2_61_release/bpy.ops.object.html#bpy.ops.object.transform_apply or http://www.blender.org/documentation/blender_python_api_2_61_release/bpy.ops.object.html#bpy.ops.object.visual_transform_apply )

but it doesnt look to work as expected
Yay! Now to get rid of jre ;)
we might want to print a warning  that this mode is not recommended, or deprecate it all together in version 1.7 and only support javascript api-less mode for share... no reason to support disabling the api once we are running our javascript on the page and there is no longer the jQuery dependency.
Great, I'll test it, thanks :D
DAS was es? omg :)
I prefer try: iter(x) and catching the exception, but it seems you want to do something special in the case of dicts?
Why not throw an error?
I haven't run any tests so far. And I have only targeted `IServiceBus.Publish( .. )` to my transport draft. There are better protocols in the guide to ZMQ, so I suggest that we all read all of it before continuing down this path.
this seems a nature of a create command vs a spec..
Does that mean that people running tests will have to tweak their test configuration in order to get meaningful test results?
Finally works ;-)
hm... in your server players have 4 level of privilegies? also possible reload over CLI/SOAP.
Would it be better then to make my rules look and behave like the
implicit ones?
Well -- I'm not sure if ONLY 1.5.1 will do --  but some later ones
(including current stable) introduce a bug, 1.5.1 seems the latest
that works.

On 1/28/12, David Cox
<reply@reply.github.com>
wrote:
> Can you elaborate a bit on what the pyparsing bug is?  Do you mean it
> requires *at least* 1.5.1, or exactly that version?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/yamins81/genson/commit/46351605b68dd72953a4615ed325d2af15bae8e8#commitcomment-911327
>

-- 
Sent from my mobile device
I think you need Sublime Text and good music to find some kind of failures :D
pra que serve o xml log4j.xml
There's a self.env reference here, but the class is actually inheriting from object, so this will throw an AttributeError. We need to change the inheritance, see above
If you look closer to Makefile CCFLAGS are passed to build line.
Shouldn't the session endpoint take precedence over one defined in settings?
I noticed on my mac that spaces seemed to be used instead of tabs even when they were set.  Not sure how that happened either.
I think the regex should be changed to something like
```"(find|show|where).* (nearest|nearby|closest) (.*)"```
so it doesn't matter whether you say "Find nearest Starbucks" or "Find the nearest Starbucks".
I don't mind nuking `.bytesWritten` altogether, it was a daft idea to start with.
Aren't we assuming that the DOM is updated at this point (but it may not be on some computers)?  That would be an argument for just adding the div in with the other html code above.  Or at least moving this line down to right before the container is used.
why this particular add-on? `self.addons[0]` should be just fine, right?
This was already fixed in "Use correct path when opening file in tests" pull request (https://github.com/kennethreitz/requests/pull/432)
arg, I think I have a regression due to my rebasing.

Indeed, I'll have to insert the itemInstance here instead. As the itemInstances are unique per player's inventory, it'll be fine once fixed.

Thanks for the lookup. :)
is it a big deal?
I think throwing is better here. But can we also solve the root problem of having v1 and v2 so tightly coupled?
Consider making hex_digits a constant frozen string, and referring to it here.

`HEX_DIGITS = '0123456789ABCDEF'.freeze`
Nice comment!

Should I prefer `type(...) is ...` to `isinstance(..., ...)`?
worked for me!
There might be small difference or may be not, but there is also another aspect, consistency. ;-)
Added support files for Accelerometer Simulation for iPhone Simulator and Unimotion
Ref: http://code.google.com/p/accelerometer-simulator/wiki/Home
Ref: https://github.com/gmarik/unimotion
This looks great and it seems to do the job. However, there are some things about this code that is making making me uneasy. The background images are weirdly defined as a background to a div in a [different spot](https://github.com/mozilla/mozillians/blob/79aa24f52edc52d598c3361bd1584d51cc3421d0/media/css/main.css#L397). Though this is not your fault, putting a link in a different spot only makes it more confusing.  Lets use an image tag for the images instead of this .imglink class. 

Btw don't worry about rebasing your changes till after you're done. We can try working through that together when we get to that point.
Would be interesting if the exporting was done automatically. 

`moo.define` could wrap a regular define, and `if (!has('amd')) exports to `Core.global` automatically?
No, I checked it my version 2.0.1 and this commit already there. I mean I checked default.py and I saw it. Updated few days ago but just realise because I restart not often. I don't know you have 2.0.2. Look here http://addons.openelec.tv/2.0/ION/i386/addons.xml
OK, I'll do that.


________________________________
 From: Ron Evans <reply@reply.github.com>
To: Omar Patel <omarpatel123@yahoo.com> 
Sent: Tuesday, February 21, 2012 10:40 AM
Subject: Re: [kidsruby] lesson 6 (cd1ed4c)
 
No, you can just remove that reference and link.

On Tue, Feb 21, 2012 at 10:35 AM, Omar Patel <
reply@reply.github.com
> wrote:

> I actually haven't created a lesson 7, yet. Is that required before I push
> the lesson? I just wanted it in Github, because I wanted to minimize WIP.
>
>
> ________________________________
> From: Ron Evans <reply@reply.github.com>
> To: Omar Patel <omarpatel123@yahoo.com>
> Sent: Tuesday, February 21, 2012 8:52 AM
> Subject: Re: [kidsruby] lesson 6 (cd1ed4c)
>
> Is there a lesson 7?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/hybridgroup/kidsruby/commit/cd1ed4c18f9b148bfcb52f1a6219d1c4f781c3b8#commitcomment-992067
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/hybridgroup/kidsruby/commit/cd1ed4c18f9b148bfcb52f1a6219d1c4f781c3b8#commitcomment-992520
>



-- 
Ron Evans
The Hybrid Group
http://hybridgroup.com
p. 310-916-9924
m. 310-597-1013

--- 
Reply to this email directly or view it on GitHub:
https://github.com/hybridgroup/kidsruby/commit/cd1ed4c18f9b148bfcb52f1a6219d1c4f781c3b8#commitcomment-992545
我都看了，OK啊，你在我的那个修改上面再把它们修改的更通顺点儿就是了。
Less files that @import all variables and mixins into itself or contain plain css without variables or mixins will render as a css file. Like the file below utilities.less. This file is included in the main styles.css - but when compiling less you can direct the outputs of files separately. I had thought to include a folder that a users could direct this into - but it makes total sense to remove it and just not output them.
As usual, thanks for your work lextm.

I’ve been performing some tests based on your last commits.

There is a big improvement on false click events, especially when scrolling (thanks to movement detection I guess?)

One downside however: touch-over-click is less accurate. Touch events are not always handled, especially when fingers were already on the mouse when tapping the mouse. 

A "close the current application" movement would be very practical.
I think this could be done by storing the partial match on the view (this.partialMatch) or something along those lines; if nothing else, best to store it on irc. It's bad practice to clutter up the global namespace.
「メソッドは実行される際に」と言い換えれば句点をなくすことができると思うのですがいかがでしょうか？
See: http://stackoverflow.com/questions/104184/is-it-safe-to-get-values-from-a-java-util-hashmap-from-multiple-threads-no-modi
why is the stop icon different from the original icon ? (and it's a little bit blurry on the bottom side of the white cube)
It is better if a TargetDirectory.forTest( getClass() ....) was used here
Wtf??? Para que lo mapeas con el slice? No entiendo... (se para que sirve el slice, pero no para que haces esto...
Hm. It works without js, i didnt test what happens if it tries to call the nonexisting functions.
I have committed my recent changes as well as C89 incompatibilities fix to googlecode
Let me know if there are still issues with your compiler.
Also, I noticed you needed to modify some includes with the path of .h files, why is it needed ? 
Isn't it possible to configure Makefile so they look for included files in given directories (-I option in gcc) ?
Please write more information in the commit message, after "[ENH] .... info here ... " :)
How could the Id be null? Can we reproduce?
Plugins may need to be updated for compatibility with this method:
    replace all instances of "ArrayList<DLNAMediaSubtitle>" with instances of "DLNAMediaSubtitleList"

I think it could be beneficial to wrap several ArrayLists, one for each type of subtitle, into this class. It would simplify code when searching for specific types/languages of subtitles, especially in more complex cases where a renderer supports only a partial set of the available subtitles.

It may also be possible to determine the highest priority/default subtitle in real-time while subs are being added to the collection. This should reduce the [subtitles code in Player.java] (https://github.com/ps3mediaserver/ps3mediaserver/blob/master/src/main/java/net/pms/encoders/Player.java#L188), and simplify the implementation of subtitle selection logic in [Request.java] (https://github.com/ps3mediaserver/ps3mediaserver/blob/master/src/main/java/net/pms/network/Request.java#L211)/[RequestV2.java] (https://github.com/ps3mediaserver/ps3mediaserver/blob/master/src/main/java/net/pms/network/RequestV2.java#L293).
It's more disappointing how long it takes @Subv to add a simple Typeid check :)
1. Aokromes Video is cata.
2. Aokromes Video shows Npc behavior, we have given you shitloads of proofs that shows that it should NOT work the same way on players. So either add a fucking Typeid check or just remove it completely ;)
Aokromes' non-stop request... blame him :D
I did rebase to recent master... 09d58df8fcfe7ecd58a7f2f59fbbc5039944c1d8
Once I changed the class name compiled without problems
I am on 
Apache Maven 3.0.3 (r1075438; 2011-02-28 18:31:09+0100)
Java version: 1.6.0_29, vendor: Sun Microsystems Inc.
Thanks for doing that. Could you update the README with a list of what to
include and maybe an example of how you've used it? I've tried to include
everything I think that should be included but it doesn't work for me.
Thanks.

On Thu, Feb 2, 2012 at 8:13 AM, iwiznia <
reply@reply.github.com
> wrote:

> Hi! It's divided for better organization, just like the Ext framework
> files. It uses the ext loader to load all the files or to compile a
> minimized version of the app.
> I just enabled the issues on the repo.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/iwiznia/Ext.ux.Exporter/commit/7ea64bb505a29833d6803e65e67f7f4c66723678#commitcomment-926603
>
Shouldn't be the name an option too ?
No it means:
``True`` for verbose (capital T) or
``true`` for silent
Damnit I knew I'd forget at some point. Added it in now haha
Otro reporte: está la abominacion de putricidio puesta.., lo gracioso es que se puede coger, bosses piñatas con esto, esperemos que no llevase mucho tiempo puesto
@iElectric, nice, much cleaner without all the nasty workarounds!
I considered doing that, but I didn't want a number like 3500000 because it gets hard to read and I considered doing some dividing and rounding/truncating to get 3.4MB or something like that.

That being said, I did want to explicitly point to *where* a developer can find the max value such that the error message would be:
less than or equal to 3.4MB (GridFS.MAX_CHUNKSIZE).
Cojo so mean.
I forgot about that spell ;P
thx for info
Thanks. Duh...
Thanks for the quick reply.

I think one possible cause of the error would have been if you were compiling using a pre-Honeycomb SDK.
I will change this back and see how it goes. If you still have trouble after my change, please do let me know.

-Flavio
Yeah, no worries @auzigog.

I updated my edit just now, since I just pushed that change. What do you think? It removes the nasty copy-paste issue (which you rightly brought up) and has minimal code repetition, and could probably be reduced further.
Prevents NaN from streaming for unknown metrics.
why librasdioif.ko?
what's the used of it?
This line looks misaligned
last block here should be part of the validate at the end since its doing cross parameter validation
Silly you, you forgot to push the block stuff. Can you do it please ? http://puu.sh/jV2Q
I was hopping that this file will generate empty macros for Main, Ap and Fbw, thus it would be compatible with both rotorcraft and fixed wing (in the strange case were you don't want telemetry ???). But I have not really tested it.
If you choose this file to work with some test firmware that doesn't use the telemetry, then it doesn't really matter what process you have in this file.
I would just like to reiterate kgerst's comment. Thank you for all the work on this plugin. My team looks forward to the 1.5 release.
Those default-locales shouldn't be there, you know :).

While the CSL 1.0 schema currently permits this, I want to make it as clear as possible that (at least currently) dependent styles are just bookmarks to independent styles. Setting the default-locale in dependent styles should have no effect on the style output, so we should probably just leave them out, because they can potentially conflict with the value on the independent parent.
I agree that this would be better to do in the BootStrap.
This is what Rainer referred to in the mailing list.
The [implementation](https://github.com/D-Programming-Language/druntime/commit/fdd189b990bbb960715949913b0a128031aa9526#L1R235) says only ```@trusted```.
May I use this example for my library ( https://github.com/Gottox/socket.io-java-client ) too?
yes, of course, i agree that there are exceptions how colours should be used. in general UI design, specially layout designs, colours can be used more freely (the google example you brought up). but what comes to the program output, and specially output that user should pay attention, colours have bigger role. they need to be catchy, indicate that attention is needed.

what comes about the colour coding, traffic lights, android's (probably iphone too?) answer to call / reject applet and mac OS X window buttons follow the same colour pattern: green = safe operation, red = pay attenion, danger. although only these three examples were delivered, these colour codings are widely used in many other products too.

> I would ask, why, after all, we are highlighting the "Accepted" status of solutions (whatever color it is) in an assignment grading system?

well, actually i would go vice versa; show (red) colour if something goes wrong. therefore, if a student is disoriented, incomplete judging/error is easier to spot even by accident. if compiling goes well, no extra attention is needed (= no colours).
i have made a pull requests .... as 2 months has passed from them, may be they are not enough...

Lucky  i finally figured out how to push gerrit review (http://review.cyanogenmod.com/Documentation/user-upload.html)

so 
 http://review.cyanogenmod.com/11427
Promise of this process to complete
It's OK :smiley:
I think you should use the `get_installed_distributions` wrapper, which has the local_only option you can use, instead of working_set directly.
in the case where you're in a virtualenv that exposes global pkgs, this will parse and use *all* dependency_links even we your using  `--local`

I don't see any case you want dependency_links from global pkgs to help you find versions for your local pkgs?
the _dirty flag and 'lazy' _buffer accessors are actually doing (2) and (3).

(4) might be tricky: if we limit it to a Dat setter that should be ok, but else we have to intercept modification of the a numpy.ndarray...
This has a couple implications in other parts of the code base (notably, how we handle node names), so I'm going to leave this as a TODO for now. But, I agree.
Why declare `__slots__`? We aren't exactly starved for space and we don't do this anywhere else.
I suspect "i.e." (*id est*, "that is") would be better than "e.g." (*exempli gratia*, "for example") here. :)
this is racy: child creation could check flag before this and create child after line 90 is finished; I think this is one of the few places where I’d use a conventional Lock
Exactly that logic/reason should have been present in a comment for that code...
Added a new test case for individual settings.
It should be in it's own module, so people who do not use it don't need the added weight.
"bytes" is mutable.
On Sat, Dec 8, 2012 at 12:17 AM, Viktor Klang (√)
<notifications@github.com>wrote:

> In akka-camel/src/test/scala/akka/camel/ProducerFeatureTest.scala:
>
> > +    override def oneway = true
> > +
> > +    var lastSender: Option[ActorRef] = None
> > +    var lastMessage: Option[String] = None
> > +    def endpointUri = uri
> > +
> > +    override def transformOutgoingMessage(msg: Any) = msg match {
> > +      case msg: CamelMessage ⇒ if (upper) msg.mapBody {
> > +        body: String ⇒
> > +          if (body == "err") throw new Exception("Crash!")
> > +          val upperMsg = body.toUpperCase
> > +          lastSender = Some(sender)
> > +          lastMessage = Some(upperMsg)
> > +      }
> > +      else msg
> > +    }
>
> How can this be right?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/akka/akka/pull/933/files#r2355013>.
>
>

Its maybe a bit of a strange test but I want to see that I can capture a
sender at an ok message (the reason for the test, the sender ref did not
reflect correctly), then crash the actor with a bad message and then send
the last message to the last sender, if I receive a msg on the last sender
(the test actor ref) then it works.

-- 
Raymond Roestenburg

code: http://github.com/RayRoestenburg
blog: http://roestenburg.agilesquad.com
twtr: @RayRoestenburg
book: http://manning.com/roestenburg
yes, all other events should just be published to the eventStream
authentication is also much simplified in the internal APIs, and uses one that we should be able to use with any auth provider with small adapter classes at the most.
Remaining parts merged into object_content, not much left now.
This is what you should find out when you're implementing this feature ;-)
better writing, thanks :)
I didn't seen it before, but if standing is standard, then... can you rename to DefaultHeader or BaseHeader / HeaderBase ? Check at others classname to see how the default classes are named
What kind of thing would have a .shape and not a dtype?   Should that unexpected thing not raise an error?
Is that "za" a typo?
No, this is the point: parent are the immediately upper level in the tree hierarchy while roots are the remote ancestors. I needed the distinction to make the algorithm run properly.
I didn't put it on the dev list/ standup because it is something that is not of general interest. I'll have a think about it and get back with a suggestion.
Again as a newbie, it would be awesome to have a docblock here so I can know specifically what these options are.
Also since this method is so independent, it would probably be nice for it to have its own unit test.
Simpler to just use log.exception, which if in an except block will do the traceback handling for you. Not worth changing, just a heads up.
Need to set auth=no on Fedora 17+ as well.
both the path setting and the Card.setting parts are needed.  Without doing Card.setting, edits to the setting card content won't take effect.  without path_setting, url will break on non-root wagns.
Isn't range 0.0 <=  timesize <= m_messageQueue.GetMaxTimeSize()?   If so, then a proper percentage calculation (mirroring what is done for IsDataBased elements) should be:

```c
return min(100, MathUtils::round_int(100.0 *  * timesize / m_messageQueue.GetMaxTimeSize()));
```
IMO you want to use m_basePath here for most cases, and only if it's the same as item.GetVideoInfoTag()->m_strFileNameAndPath use URIUtils::GetDirectory()
You could just have an array of usable fields, and use `in_array`, that way, you can add more fields without having to change the logic.

```php
<? $fields = array('h1_blurb', 'meta_description' /*, ...*/  );
```
Have you convinced yourself that we only want pre-filters, and no one is going to want post-filters ever? It doesn't make sense for efficiency reasons, but I can imagine a command having functionality that's not easily available in Python. Maybe a program written in Java that converts JSON to typedbytes or some such?
Like this:

~~~ruby
require 'tire'

Tire.index('articles').delete

class Article

  include Tire::Model::Persistence

  property :title
  property :size,       :as => proc { title.size }
  property :created_at, :default => lambda { Time.now }

end


p Article.create(title: 'Hello').attributes,

  Article.index.refresh,

  "Size: %s" % [ Article.first.size ]
~~~
you are absolutely right I was just reading about it, I used the code form a 3.2.2. I will change it now.
Use UriBuilder instead of string concat (see DatasourcesResource.getDatasources()):

URI link = UriBuilder.fromPath("/service").path(service.getName());

or even better when ServiceResource will be defined:

URI link = UriBuilder.fromPath("/").path(ServiceResource.class).build(service.getName());
OK. Everything that gets allocated from the Workspace needs to be accounted for in the calculations of memory usage in confdefs.h. That is a complicated file and I suggest asking for help on rtems-devel. Either that or use malloc().
You should remove the blank line.
os.path.join is safer
I found the console.log useful for debugging, but you are right that
consistently concise is easier to read.

On Mon, Apr 16, 2012 at 11:24 PM, Ben Noordhuis <
reply@reply.github.com
> wrote:

> > @@ -49,6 +49,20 @@ is supposed to return an Array with 2 entries:
> >  Which ends up looking something like:
> >  `[[substr1, substr2, ...], originalsubstring]`.
> >
> > +Example:
> > +
> > +    function completer(line) {
> > +      var completions = '.help .error .exit .quit .q'.split(' ')
> > +      var hits = completions.filter(function(c) {
> > +        if (c.indexOf(line) == 0) {
> > +          // console.log('Found: ' + c);
> > +          return c;
> > +        }
> > +      });
>
> This could be rewritten as `var hits = completions.filter(function(c) {
> return c.indexOf(line) == 0 })` right? Shorter is better in examples.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/joyent/node/pull/3084/files#r688955
>
osArch.equals(osArchCheck) obviates the need for the null check.
After sitting on `promise._deferred` for a day or two, now, I'm preferring it more strongly over closing over the deferred.  Yes, it violates the strict privacy called for in the contract, but it has always been more pragmatic to use gentleman's privates. Security can be managed at the YUI sandbox level.

I still really don't like including progress in `then`. Maybe because of language pedantry.  This really causes turmoil between my want to preserve convention, and my want to fix what seems obviously broken. Bleh, to go back in time and argue my case :(
To add, as part of renaming all host to hostPort (where relevant) within code, this was the only part which I did not change - and left as spark.hostname : even though it is actually a hostport now.
I was not sure if this was an exposed environment variable which can be set by users customizing spark execution : cant for yarn, but was not sure for other deployment modes.

If that is not the case and this system property is not part of exposed interface of spark, I would love to remove all references of spark.hostname and replace with spark.hostport !
the reason I did this was that the FeatureEditPopup passes information that we need as well:

https://github.com/opengeo/gxp/blob/master/src/script/widgets/FeatureEditPopup.js#L284

So not all of the config is supplied by the user/developer
Can we remove the parenthesis here in the hash access pls?
The assertion in rails master looks better, please send a PR to sprockets-rails updating this
Is not possible, tried and tests failed. The default `autosave` behaviour breaks.
Isn't with Homebrew ok to just do gem install pg ?
I unwrap one-liners everywhere where they have more (or much more) then 80 symbols. Just like this style)
Thanks, @rafaelfranca.  I updated the implementation to delegate to super.
```ruby
response = (_, headers, body = @app.call(env))
```
+10 seconds
This should be annotated with `@tailrec` now.
consider `List.map2(args, params){(arg, param) => if (...) () => arg els arg}`
That's what I tried first but it didn't work:

    In [4]: from scipy.fftpack import rfftn
    ------------------------------------------------------------
    Traceback (most recent call last):
      File "<ipython console>", line 1, in <module>
    ImportError: cannot import name rfftn

Maybe this was fixed in a later version?
"us that" --> "us is that"
or view()?
Do these lines ever do anything? Because _eval_nseries should not return log(x) terms if logx is passed.
This doesn't actually work on iterables, only on lists, tuples, and other classes that mimic either's interface.
Thanks, added it.
I have seen that the function above (resettodefault) does not use return. What is correct?

```return $this->redirect(System::getHomepageUrl());```

or 

```$this->redirect(System::getHomepageUrl());```
Does that mean the DisplayPythonOutput function would be disabled, or that this call should be disabled? It doesn't look disabled here...
I'd be glad to. However, I'm new to github and still learning the UI. How do I edit my version of the file?
Das würde ich nicht machen, da nicht alle Methoden eine Transaktion brauchen. Somit wäre das eine massive Resource-Verschwendung. Dazu kommt, dass wenn ich eine bestimme Exception abfangen möchte, dass das dann nicht mehr möglich wäre!
Please move the assignment outside of the if.
Looking at this code again, I realize it should use uv_strlcpy() instead of strncpy() because it won't always zero terminate the output buffer now. No matter, we can fix that up in a separate commit.
Are those extra lines needed by the new style guide?
"f"?  Did this code even run? :)
That's a fair point - apart from the code you'd need to write, the DB lookups would be quite fine grained and would potentially lead to a low cache hit rate.
What would be the knock-on effect of changing this ruleset (should you ever decide to)?
Yes, I'll add in some tests for common printable ASCII.
Toss in a note that all modes read from the nearest matching members, but "nearest" is optimized for latency above primariness / secondariness?
this needs to be reprhased a bit.
the distinction between collection and index here is perhaps too narrow?
J'ai essaye de creer un block reutilisable pour generer une separation. Ca revient a notre discussion sur les pattern de code. Je trouvais qu'inclure un div etait plus clair qu'une classe, mais il y a peut-être des désavantages auxquels je n'ai pas pense. Peut-être que ça ne se "fait" pas aussi.
I propose that events modifying sql should have "sql" in their name.

This event does not "get" anything from what I can tell.
Should probably be `FilesystemIterator::NEW_CURRENT_AND_KEY | FilesystemIterator::FOLLOW_SYMLINKS`.
Will change to: get_extension_metadata_manager
Empty if block? o_O
Low, but not entirely unlikely.
delete "class"
I have been unable to test update due to http://tracker.phpbb.com/browse/PHPBB3-11086
这个已经被 pry-nav 依赖了的情况下就不用再指定了
`or` behaves slightly differently than `||` and lots of rubyists (myself included) recommend against its use, due to some gotchas.  (I think github's ruby style guide even forbids its use).  Your parens ensure there's no gotchas here, but you may want to switch to `||` to be more inline with standard ruby style.
Зачем парсить второй раз конфиг? Он уже точно парсился где-то для работы с базой
Typo fixed^
The project and environments could easily be retrieved in the view model itself - it's view logics after all.

Only the parsing of the id is controller logics.
return new ViewModel(array('project_id' => $this->params()->fromRoute('id');
Here is more info about caching TPL stuffs.  http://bradwilson.typepad.com/blog/2012/04/tpl-and-servers-pt4.html
In some ch9 video I saw someone telling TaskCompletionSource<int> is optimized in .net 4.5 but in this blog post Brad Wilson uses TaskCompletionSource<AsyncVoid>.
please don't use formatting at this stage, unless absolutely necessary. prefer to stick with sencha defaults when doing development, and to do theming (across all modules in the application) as a consistent effort by creating CSS tags.
We don't want to remove all these, should stay
not a big deal... but revert change from this file if possible - just added unneeded whitespace
The same above goes for provider and location here
This image is so nice, and impressive, I'd love to see it in the gallery.
"If you do not have VMWare installed, install VMWare player (free):"
`InferShapeTester`, for consistency
I rearranged this method to be more readable while I was debugging it, but I didn't change its functionality.
Ditto the comment from the other index, and other to the other two.
Why not serialize the catalog version?
Don't describe WHAT are you doing.
Please descibre WHY are you doing :)

e.g. data from get_context_data contains names instead of id's - settings for comparison
if sn was duplicated, the message should be shown on the screen
I guess these have to be started first or else `lager:stop_trace` will error?
Alex: It makes sense to have them in the same method, as often they share the same namespace, and they all cross-enforce (for example, a UNIQUE constraint will also implicitly be an index). The name is perhaps the worst thing, but I didn't want get_constraints_and_indexes

Anssi: There probably is room for code reuse here, in particular I'd want to hang get_indexes off of the new get_constraints (which is that method which returns multicolumn indexes/constraints). Is there a particular reason you prefer named tuples for the return value?
It was kept open to get an example from the guy. Who didn't give any.

But regardless, your example is wrong not because of that ticket, but because you're matching /page//, not /page/, here, in the current code. It should always match the current code, not something we might do later on.
I'm not changing any of the calls (except skipping `getAttribute("type")` when `elem.type !== "text"`), so I don't think this can have any negative consequences.
Just to be uniform. 
This is a piece of code copy from main. Why d_1 and d_2 are quoted, and x are not quoted? I make them uniform. 
CREATE FUNCTION MADLIB_SCHEMA.fisher_f_cdf(
    x DOUBLE PRECISION,
    "d_1" DOUBLE PRECISION,
    "d_2" DOUBLE PRECISION
) RETURNS DOUBLE PRECISION
AS 'MODULE_PATHNAME'
LANGUAGE C
IMMUTABLE STRICT;
Sorry, maybe it's not very clear. As mention in jump array. In the old code, as we will add 1 to the feature value, the lmc_fval should be (2 -1) = 1. In the new code, lmc_fval is the feature value which leads to the left most child node. Thanks.
Yes, but in that pull request it is discussed that it causes buggy behaviour.
Module level constants or settings?
So, first, I never heard the rule explained like that, now things make a lot more sense.

Second, the reason I did this is because a new linter I had was complaining about it, but that linter is being more annoying than it is worth, so it is going away.
I guess this needs to be updated now!
Does this mean if I want to deparallelize the build I have to set MAKEFLAGS? That seems counterintuitive to me; I'm not running `make`.
This is the place where, for uncached users, we mush search for the user in the directory server
Well I think I wrote that initially to indicate that it returns the new page of the reddit site that the user is connected to. I think that's pretty obvious though, and thus not needed.
commands is deprecated: http://docs.python.org/2/library/commands.html
Excellent suggestion. I've added a note about this being a legacy route.
Not needed, use shelljs
'access_token' and 'api_key' are too generic name. Imagine there is a similar mechanism for Google added and someone would like to have both configured – this would be impossible if both mechanism used 'api_key' property for their service.

I suggest renaming those to 'facebook_access_token' and 'facebook_api_key'.
Again, can we use the TemporaryFolder Rule?
Why not use a TemporaryFolder (with rule) as the parent, and another TemporaryFolder as the child?
Thanks for factoring this into a new file! :-)
need a space between "if" and "(". I think JSLint would catch that, so be sure JSLint is turned on.
Origin is WebKit: http://svn.webkit.org/repository/webkit/trunk/Source/WebCore/inspector/Inspector.json
(the link can be found in jsdoc.rb)

That is all the info, I have.
That's a huge can o' worms! If we localize that string, we have to localize them all. Can we go with this for now and then deal with it if it comes up?
yes, hyphens in arg names seems to work. I ran all of the unit test and smoke test, and also played around with it. It looks like bootstrap is not consistent on naming because I can see mixin name with both lower-case-plus-hyphens and also camelCase.
Do think we should add assertions in any of these tests about whether the TextRange has or hasn't lost sync after the edit?  I know the goal of these tests isn't to test TextRange itself, but we do want the implementation of delete line to preserve the TextRange sync in certain cases, and it might be worth testing that...
Any reason not to use `$(".codehint-menu.open")` instead?
General comment: these errors about ids being missing from the map or ids colliding should probably include the id in the exception string.  Makes reading error reports / stack traces easier.  (This also applies to some of the error cases in KeyBindingManager).
As I mentioned in my comment, mu4e depends on mu, you must install mu for mu4e to be available.  I'm not sure this package is really suitable for el-get because of this fact.
trailing comma doesn't parse as JSON, please fix
can you plz run 'mvn license:format" on your source tree? This line should read

 * Copyright (C) 2012 Facebook, Inc.

Also, the header should not be a java doc. :-) mvn license:format corrects all that.
is there a particular reason to lock these in so exactly?
These two calls to parser.next() are strange... can you explain that a bit ?
Is not this now simply beautiful? ;-)
Yes this also works without '()'.
One disabled state to rule them all.
Change the argument order to `butter, media`
No? What should it be?
As I said in an earlier ticket, this text is no good.  We won't file a bug per feedback submission, since a lot of them will be noise vs. things we can actually file.  It's better not to setup incorrect expectations here.
unrelated change, breaks preprocessor
Why don't we use the same technique for development and production too?
You make a good point.  The method encapsulates more than just 'check if it exists', it also takes on the responsibility of setting view state, and that involves more than one bit of instance data.  So I think I would go with setting the state in this additional method, but try to name the method so it's clear what it does.  Maybe something like `set_pir_merge_view_state!`?  The exclamation point is useful to indicate that the method's purpose is to modify the state of this object.

Also, on the subject of exclamation points, if `@deployment.get_medistrano_pir` raises an exception if the PIR doesn't exist, then it should really be called `@deployment.get_medistrano_pir!` to follow the Ruby naming conventions.  Similar to the difference between `save` and `save!` on a model.
No. The switch is to prefer bundle plugins but the way it works is that it lays down then overlays may add plugins (that are added to sonatype-work plugin repository, plugins that are scanned and removed if they exist in bundle
Im pretty sure find raises the ActiveRecord::RecordNotFound even though it's not a bang method.
Okay, this makes sense I guess, but then measure would need to be added in the tools section as well, otherwise adding the dependency does not make sense.
此条重复
On 08/14/2012 02:06 PM, Jason E. Rist wrote:
> In src/app/views/systems/_subscriptions.html.haml:
>
>>            - until_time = @system.compliant_until
>>            = until_time ? _("Subscriptions are Current Until %s") % format_time(until_time) : _("Subscriptions are Current")
>> +        - elsif @system.compliance_color == 'yellow'
>> +          = _("Insufficient Subscriptions for System Compliance")
>
> I suggest "Subscriptions are not Current due to insufficient subscriptions"
>

How about "Insufficient subscriptions are attached to this system"

Attached is a branding approved word.

-- bk
Good to know!  fixing
Just calll this method with this instead:
```Nurse.create_nurse_invites!(unit, shift)```
its more natural =]
These are not needed. Use the user's home .gitignore file for user specific ignore preferences. 

Per project .gitignore should only include project specific types.
Ok, so by this point, have all of the parameters been resolved? Does each invoker need to resolve their own Json parameters or are they all expected to use the extension method? I have some fixes I want to make to this parameter resolution logic, what does that mean for this?
So this is the only piece of Code in the WHOLE system that depends on the HubDescriptor having a type.
Please avoid whitespace changes unrelated to subject of patch.
Should this be updated to use new-style string formatting?  I thought we were going for using that consistently in any new code.  I could be mistaken though.
Yes, that should be the way to take here.
I think this will still have problems when REQUEST_URI contains a query string parameter with the domain and also has the full domain.  For example: `http://example.com/some/path?url=http://cakephp.org`.
this is being done on the Tempfile.  Intent is to make sure the file is closed and removed.  Agree, this may be unnecessary, testing.
Oh, aha, I missed that we were flattening the lists.  My bad. :p
Right, good idea. Thanks.
Does this change mean we really should be on ` ~> 2.11.0`? was this syntax valid in 2.10.0?
@arrbee Fixed. I also took the liberty to replace the hardcoded `a/` and `b/` by `DIFF_SRC_PREFIX_DEFAULT` and `DIFF_DST_PREFIX_DEFAULT`. Does this fit you?
Heuh... Did you?
It's not incorrect, but it's always good practice to use interface (Map in this case) then specific implementation for variable type.
Why not combine these?:

```
    qs = (Forum.objects.filter(is_listed=True).select_related('last_post', 'last_post__author')
```
in testing, an OSError leaked out at this point from the bowels of this filesystem code.  Since this is legacy code destined for replacement in January, I'll not spend the energy tracking it down.  i just want to be robust enough to not bring down an app with an unexpected error
this should default to 5
I'll check and straighten it out as part of the changes we discussed in your email.
The $parent variable is not needed
It  is a BC break because it changes the signature of the interfaces
What if you mock a TableMap?

--
William Durand | http://www.williamdurand.fr



On Wed, Dec 5, 2012 at 9:20 PM, woodspire <notifications@github.com> wrote:

> In src/Symfony/Bridge/Propel1/Tests/Fixtures/ColumnMap.php:
>
> > @@ -0,0 +1,61 @@
> > +<?php
>
> No we cannot. ColumnMap expect to be pass a TableMap, an we do not have a
> table class in the test procedure.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/symfony/symfony/pull/6150/files#r2326252>.
>
>
This is fine. What would be the convention with urls included to this main url. Should the names in those url-view mappings contain the name of the package to avoid overlapping names for multiple views?
This TODO can be left for now
Good job. 
You can also use jQuery to hide and show elements and it is very easy.
please use propery notation: `self.mapPinImageView = nil`
please write this so each should is in it's own spec. That way if 3/4 fail you can see those in the same test run.
False if path caching is **not**(?) allowed...
不要省略 {}
@ebryn, that is the point where i am not sure if it could be any better way to make it.