labofsahil/github-event-dataset-2012 huggingface dataset:
Update comments

Update comments to describe the test.
add required authors to gem spec
Feature/multisites

Translations mis-creation fix for the data migration.
ARQ-823 Enabled SSH identity and StrictHostKeyChecking in arquillian.xml
Tests for the fixes of DFL-3676, Release task 17
Declare in META files that share/ is not to be indexed

then the pm's don't need to be modified
with the "package #comment\n  Mod::Name" hack.

This also avoids this [pod indexing issue](https://github.com/CPAN-API/metacpan-web/issues/609) in metacpan.
only allow the bottom pull if the last item is fully displayed

The widget would start the pull system when only a portion of the last element was displayed
Textarea Automatic Resize

Fixed Issue #41.
Ability to register custom template engines...

...via Jasmine::Headless.register_engine hook

In response to [issue #102](https://github.com/johnbintz/jasmine-headless-webkit/issues/102), I added the ability to register custom template engines not currently supported by default in JHW. The hook may be registered by placing ruby code in #{spec_dir}/helpers/spec_helper.rb. (Perhaps the location of this file can be configurable.)

For example, I am using the following to compile Handlebars assets via the `handlebar_assets` gem for JHW in my current project:

```ruby
# spec/javascripts/helpers/spec_helper.rb

require 'sprockets'
require 'handlebars_assets'

module Jasmine::Headless
  class HandlebarsTemplate < HandlebarsAssets::TiltHandlebars
    include Jasmine::Headless::FileChecker
    def evaluate(*args)
      if bad_format?(file)
        alert_bad_format(file)
        return ''
      end
      %{<script type="text/javascript">#{super}</script>}
    end
  end
end

Jasmine::Headless.register_engine '.hbs', Jasmine::Headless::HandlebarsTemplate
```
refactor & cleanup of backup compression

slightly more efficient, it doesn't copy to a byte array; it just copies from one stream to another.
Removed ui-resources from Framework into WP, adds Localization as well

as Font changing in UI layer.
Notification emails sent to each member

This is a bugfix for Issue 31 where the notification emails were being sent to only one member. The workflow instance can have many members attached to the transition, so each member needs to be notified.
SECOAUTH-176: Corrected InvalidClientException to use 401 status

Proposed updates for https://jira.springsource.org/browse/SECOAUTH-176 

NOTE: Please see my comment on the JIRA before merging
Fix problem if selected http mode and fix polkit
multiple engine support. renaming.
merged release-1.8 into develop
0.10.0 Getting Started Guide, with some outstanding updates
Minor spell correction

In autocomplete_spec.rb and path.rb autocomplete was spelled as cutocomplete corrected it to autocomplete
Create a Mixin for generic class-based views

Implemented a Mixin to replace the default django core paginator by the pure-pagination paginator.
Update fr.lproj/STLockscreenController.strings
Added blackboardJavaClient to build system
Added sequence checker and updated stream reconnect
Updated creekevent_ct to have unlimited promotional content references
Bug fix and formatting change to WP plugin

(Fix) Bad URL in iframe reference.
(New) If keyword search returns multiple projects, plugin will now randomly display one.
(New) Prettified readme a bit and changed to markdown format.
Release/winter

Should fix https://dev.imaginationforpeople.org/redmine/issues/811
Textfield Format Issue

Fixed Issue #17.
Removed jQuery as a dependancy

Works with the same result, but doesn't require a 93k library to achieve the polyfill.
Update Ukrainian/main/Mms.apk/res/values-uk/plurals.xml
Conditional Object Actions

Allow generator.yml to supply a conditional function to object actions, along with an "inverse" option on whether the object's conditional result should be inverse.  In the generator.yml, specify the condition like this:

```yaml
  object_actions:
    delete:
      condition:
        function: canDelete
        inverse: false
```
In this case, the generator's model object must contain a "canDelete" method:

```php
class ProductEntity 
{
      public function canDelete() 
      {
            return $this->countProductSales() == 0 ? true : false;
      }
}
```

The link in the object_actions.php.twig template should then be wrapped with: 

```twig
{% if action.conditionalFunction %}
	{% if action.conditionalInverse %}
		{{ echo_if (builder.ModelClass ~ "." ~ action.conditionalFunction ~ " == false") }}
	{% else %}
		{{ echo_if (builder.ModelClass ~ "." ~ action.conditionalFunction) }}
	{% endif %}
{% endif %}

###---- LINK HERE ------###

{% if action.conditionalFunction %}
	{{ echo_endif () }}
{% endif %}
```
Updated readme to be more readable ;)
(Missing label)
I know. But then I should have traces. Will check the last log as this time I go 60 instead of 75. Was not working either.
Fine to me (and will permit me to fix the server remembering at connection time more easily.)
Thanks for the review ! What toolchain are you using ? I'm using make
3.82 on Arch, and I had the same problem on latest Ubuntu.

Best,
Guillaume

2012/2/25 triffid
<reply@reply.github.com>:
> works for me with or without this, seems to make zero difference.
>
> The dependencies go like this:
>
> all > build > elf > applet/$(TARGET).elf > applet/$(TARGET).cpp, applet/core.a, Configuration.h
>
> and from the build log:
>
> CXX   ../../arduino-0022//hardware/arduino/cores/arduino/Print.o
> **CXX   applet/Marlin.o**
> CXX   MarlinSerial.o
>
> it seems to work it out automagically.
>
> If that fixes it for you, by all means send a pull request, it doesn't seem to break anything for me and being explicit is probably sensible :)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/iXce/Marlin/commit/adc108d79e30d178471ee65476eb0f6a27a73f95#commitcomment-1009888
增加了一个判断方法，判断某个投票有没有指定user参与过
This file should be renamed to "console.markdown", since that's all it really documents.  The API of console is based on the browsers, and is a bikeshed factory any time we even vaguely mention a change.
I'd be tempted to put a large try/catch in the timer callback as well, so
that anything done during the timer callback can't crash the service, just
error out.  hopefully that would fix #77.

On Mon, Jan 9, 2012 at 6:12 PM, David Wolff <
reply@reply.github.com
> wrote:

> Agreed.   I switched to Exception in the next commit.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/pgina/pgina/commit/be3242a537a3858d81e173836f554d6812060775#commitcomment-851799
>
This is wrong. The `abs(s**2) > 1` condition should be recognised to always hold, since `re(s) > 1`. I.e. the test should not be changed.

The laplace transform integral definition only converges for `re(s) > 1`. For other `s` the function has to be continued analytically, the answer you get depends on the path of continuation. (So the other piecewise branch not agreeing with alpha is ok, but it should not show up anyway.)

`atanh(1/s) == acoth(s)` explains the remaining difference.
I have Problems with the Makefile. I work with WIN7 64 , but i think, grep and ruby in the makefile are only defined in Linux.
I want to compile it, because my Dtepperdriver is a L298/L297 and he has only Full/Halfstepmode. And he is slower than the new Dreivers (allegor etc.). It would be nice, if i could chance the Parameters by Runtime commands in the first lines auf the G-Code files.

Can someone help me.
No reason, I was matching the var used in the block. I've pushed a change back to exc for all.
This change traps the ESC key for all sites, instead of just on Google's instant search. So the behavior on G sites should remain unchanged, but we'll also avoid problems like closing FB chat windows when quitting insert mode.
It would have been better to split this into separate commits like 1 for char[]->string, then bla.
Anyway, good job.
never mind that comment -- I see that you are using the variable to pull all but var-dependent factors into the coeff.
Nah, as long as you're referencing it within the file itself it works fine.
For that first one, I can agree. I prefer to avoid it when the resulting code is not ugly, though. But I can revert that if you want.
For the continue's in the "regular if"'s, however, I think the chain of if..else is cleaner, don't you?
Hi,
currently forum is down (network reconfiguration :| ), so I put my comments it here (I hope you don't mind):

"[quote="swick"]Did the same thing for linux. It should find the "Installed Path" in wine:
https://github.com/swick/openmw/commit/16c214a17af6c4506d89bad139a4d66e7423f180[/quote]

Idea inside this commit is good, but I have few comments about the code:
- would you change the code so it would not contain multiple returns?
- line.length() check could be moved from two places to just one, i.e
[code]
while (...)
{
  if (line.length() > 0) {
    if (line[0] == ...) {
    }
    else if () {
    }
  }
  
  ...
}
[/code]
- instead of line.length() you could use just line.empty()

- boost/filesystem/fstream.hpp header should be moved from linuxpath.hpp into linuxpath.cpp

- regkey comparision with "SOFTWARE\Wow..." and SOFTWARE\Bethesda..." could be simplified to
comparision with "\\\\Bethesda Softworks\\\\Morrowind"

- startPos, pos variables should have type std::string::size_type, and they should be moved to the scope of "if(key.compare("Installed Path") == 0)"

- do-while loop in if (key.compare(...) == 0) scope could be replaced with:
std::replace( installPath.begin(), installPath.end(), '\\', '/' ); //std::replace is in <algorithm> header
"
An alternate would be a single job that took in m_service_map - no point having 10 jobs firing at once fighting for critical sections
So I finally stopped commenting and went and tested.  This works fantastically.  I was totally confused myself about data/durability/damage/MaterialData/short/byte/everything.  Sorry for wasting everyone's time.
I am including straides in my project from this commit
`return None if not els else els[0]`

Too clever? ;)
cooli sach!
aber zwei sache: chasch bitte luege das kei orig files ufeladsch? --> has glöscht
chasch commit messages uf englisch mache? fändi cool wel den chönteds diä andere als bispielsprojekt aluege und vilciht chli besser navohlzie:)
还有别太拘泥于格式了， 到时候写个脚本一次全改过来
Where did you download the three20 frame work.  Because this particular version of three20 will not have any warnings on ios5.  However i guess its not merged with the main tree20.
Do note that this will change the thread names but I don't know of a good/easy way to pass different thread naming conventions down from the builder.
should be `instance if (instance is not None) else self.model()`

`if instance` can return False even if `instance is not None` (http://docs.python.org/reference/datamodel.html#object\.\_\_nonzero\_\_)
so should I empty the local folder, and then fetch the files from master ?

still a little lost with GIT, I am slow :D
Well, the code was updated. Did you install the new `yesod` executable?
Good job seraphinchaos. I think it's more readable to do this with spellid in database.

Futher don't forget to add a check in Spell::checkcast or something like that, because some auras needs to be casted only one time, like Fire! if I remember
return generic __class__ so a subclass will work properly
改成叫`wiki_allow_write`会不会好一点，名字也比较短
Синхронизируй доступ при помощи synchronize
One issue with aliases is that the method definition establishes the "official" name, which will be used in printing and stack traces etc. That makes me think `enqueue` should be the core name, with `unshift` as the alias for compatibility. People who know unshift don't seem likely to be confused by enqueue.
> Also, the current printf doesn't seem to generate very well-specialized code. It does an enormous number of function calls for everything. For %d, I would expect something like write(s, int2str(x)). Instead there is a function returning four values, then a branch handling the minus sign, then another function call. Then tests for NaN and Inf which should not be needed for an integer format.

The comparison to `int2str` is kind of bullshit because you're sweeping everything that the printf implementation needs to do under the rug of `int2str`. The only difference is that `int2str` writes to a string as a buffer.
will be on-wiki soon
Ich mach das direkt aus Sublime Text 2 raus :>
see d358c66
I've added that missing statement with this commit:
https://github.com/danmar/cppcheck/commit/0e791929b561e7b0bd77568948cc6c6d86f97ab2

Thanks again.
@djpowell, @ ptaoussanis, is there existing middleware that supports this, or would this require an update to some piece of ring? I've been looking around the template-reloading middleware and haven't found anything that makes use of ``::templates`.
extra space before a method
compress_assets says ``@taupe`` is undefined here and the styles on marketplace-dev are broken
These checks ensure that the upcoming memcpy's aren't going to access buffer 'b' out of its bounds, which is expected to be the TPM's maximum buffer size. There are, they were attahced to their bugzillas, but that were reported privately. I can send you offline.
Ah of course thank you!
This ID attribute and the next one are somewhat redundant, since the the URIs in the href attributes already provide a way to uniquely identify references to individual albums or tracks.
We actually have a b2g wide pref("browser.ignoreNativeFrameTextSelection", true).

But this will make life easier for all the testing done in Firefox :)
Oh SFML really isn't that bad. Its just their implementation but Laurent said he'll look into it and they had pull requests from me as well.
i think it's useful, i was curious to see whether we were getting titles in walkaround or not.
Michael, it seems we're opening files using the wrong function, both on this new function and in `sinkFile`/`sourceFile`.  Although `Filesystem.openFile` opens files in binary mode, it seems we're using `System.IO.openFile` which opens in text mode.  Is that correct?  Perhaps this bug was introduced when filesystem-conduit was splitted?
Je vois pas pourquoi. Pour moi Eclipse et Visual Studio sont à peu près kifkif niveau fonctionnalités, sauf que VS est mieux intégré. Pour l'avoir utilisé plusieurs années, je peux te dire qu'il est vraiment pas mal... Après il est payant et non portable, mais ce n'est pas ça qui le rend moins meilleur que les autres.
On Fri, 24 Feb 2012 07:33:32 -0800
Urs Reupke
<reply@reply.github.com>
wrote:

> @the-smith, I'm surprised to see the change for favored "High Level
> MA" from 13 to 12. Did we ship 3.0 with wrong XP cost?

     As far as I can see, yes, we did.  Scroll of the Monk, pg16 &
Scroll of Heroes pg 159 both agree that the cost for high-level Martial
Arts (when Martial Arts if Favored) is 12, not 13.  There's no change
to this number in Scroll of Errata.

				Scott
leave?
it shouldn't actually be there because i deleted it in line 34-34 ^
The order gets lost doing it that way.
java classes or interface methods can't be protected
We compared again numpy to assert we have the same behavior.

Can you add the assert about the second owndata in the struct?
The image of the error monitor I emailed you are examples of errors that have only been noticed on beamline OPI's.  Both Dave and myself have never seen them on our respective laptops (Dave = mac, me = mac and linux).
I'm not sure why such outdated sources were added in this commit:
https://github.com/diem-project/diem/commit/5adcc2a98d2a37870f43f9d2d34d4131fd43f21e
but upgrading UI may only make improvements for diem :)

It fixes the bug mentioned here: https://github.com/diem-project/diem/commit/2d3ae84c7894bae3502f610e11bb7c6384787cec#commitcomment-847907
JFilterOutput can be handled by the autoloader. We should remove this line again at the next oppertunity.
This makes me a little uncomfortable, and I know it's abstract, but it sounds like we're expecting low level functions to be invoked in a certain order, which makes things a bit brittle to changes / future optimizations.  If you're confident that the only codepath that could land us here *must* have defined `addressCache` and must have also populated `addressCache[email]`, then I'm r+
I've done one IE7 project here but it's fair to clarify support intentions
(i.e. most of our stuff will work on IE7), versus browsers that require
100% test success before a commit (not IE7). More on this as I add unit
tests.

On Thu, Feb 2, 2012 at 10:48 AM, shyam-habarakada <
reply@reply.github.com
> wrote:

> I dont think we need to support anything earlier than IE8, so lets not
> claim this here.
> Also, what about android and silk browsers? We can claim to add support
> for those in the near future.
>
> It would also be good to have some position on how we compliment (or don't
> conflict with) other popular libraries like jQuery, KO and Prototype. Even
> if we aren;t doing anything specifically, making a thoughtful statement on
> our position on these will help establish a path for us to take around
> these other libs.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/sambaker/awe-core/commit/b727f93a9c6410d187f1d527af8031077cb82f33#commitcomment-927343
>
either change this to `import Flag, Switch` or change all references of `Flag(...)` to `waffle.models.Flag`
You two seem to be in a constant competition between `.navbar-search .search-query ::-webkit-input-placeholder {` and `.navbar-search .search-query::-webkit-input-placeholder {`

(see [here](https://github.com/twitter/bootstrap/commit/d4659521386eab794eaf67f57e8762656aa28de7))
The MetadataStore interface is in Bukkit. The implementations are in CraftBukkit. The individual implementations are not in the public API because their details are not important to how the metadata framework operates and users of the framework never need to touch these classes. No sense polluting a public API with internal implementation classes.
Is there any reason not to close the inventory on same-world teleports? I believe that would wipe out a few dupe issues some plugins are having.
I think he means cancel it.
здесь должно быть просто else. elsif предполагает какое-то уловие после
I don't want any watermarking behavior here, it should be completely post-processing. I don't see what's wrong with adding the watermarking ops either at the front or at the back of each page's ops?
Hi Avin,

Can you please update the sensor to the lastest unstable from OpenNI?
This is the way TJ suggests doing it: https://github.com/visionmedia/versionator/blob/master/Makefile
I remain unconvinced that telling a user "any notes you create will not be saved to the server" is a good idea.

It seems better to just not allow them to create a note.
Hi! Thank you for your submission, it's very appreciated. I'll review it asap.
Might rename `concurrency_using` to `concurrency_method` or `concurrency_type`.
Because HTTP on Github is SmartHTTP :)

https://github.com/blog/642-smart-http-support
No thought behind it at all really other than I just wasn't ready for 0.3.6 yet.
Want to get some other fixes in the next patch release.
I guess this could technically be a patch and not a build, though
Note to self: double check this. Seems ok, but merits a double-check.
I think for this 4 lines a pullrequest is a lot of overhead
;-)
And there's the possibility to use another field (Integer or Float with "is counter" enabled), to track the downloads count.
Also Updated Contact page form to remove textholder and placed labels outside text boxes.
Is this like one space off?
I had some merge conflicts during this push. I hope everything went fine.
This will cause problems for users still on slf4j-api.  The current change directly propagates to the manifest. At least we need to make sure that Import-Package: org.slf4j does not require version 1.6. or higher. I'd rather stay on slf4j 1.5 or even 1.4 in org ops4j.base
This should be Array[message.reply_to].join(", ")

No test for this?
> The best solution is to use CSS to set the tab stop size:

I did see the commit.

> It's not a very good hack as it doesn't work properly with tabs that are not at the start of a line.

I know. Codemirror had the same problem. 

```
a	aa	aaa
aaa	aa	a
```

(The whitespace between a's is a tab) All the text needs to allign. With the hack it doesn't.

Do you plan to stay with Esprima js parser? (btw. it caused a nice rendering speed improvement.)
Is this really the solution for the merge-problem?
haha love that one !
I assert that line length is a style rule, not a best practice.
Thanks for reviewing. will take care of this
@Seldaek this is wrong. @fabpot asked to keep the case following the Sf2 standard for the bridge class: https://github.com/symfony/symfony/blob/master/src/Symfony/Bridge/Monolog/Handler/ChromePhpHandler.php
Squish parameters.
klojo
p and q need to be **primes !**
Sam as above. typeof member.blackswan === 'object'
That code is useless here, the stat is already %4
The hell with Teambuilder.pro? I didn't delete whole .pro and re-add it. I haven't touched Teambuilder.pro (It was edited when i added the pmsystem files, but that's automatic) ._.
ja dennoch scriptloader, cmake, sql alles drinne und niet nada geht. mein umbriss ist der einzige code der läuft
I'd rather say 

> but speeds up indexing and overall performance.

Not a biggie though
These issues have been resolved in the next commit.
Yes, I'll push it today
fixed on 1e246cb
Fair enough.  I'm using it and if I make significant-enough changes to make it worthwhile I'll push to merge it.  Thanks for the bit o' code :)
I played retail from release until last year -- LOS will only cancel non-channeled spells. The top post quote from @Ka0z is incorrect information from WoWWiki. Read further into the same paragraph (1st one) it was pulled from:

>For example, if you are on the ground, and you begin casting Smite or any such spell on a melee mob that is in an elevated shack, when that >mob comes at you, he will cross behind pillars, most likely, and at that point your cast will be canceled.

>For channeled spells, however, the mob may not be attacked one second or another, or perhaps the effect that the spell gives (such as Mind >Flay) may not proc, but the next tick where there is no obstruction during that span, it will continue.

Only direct damage spells completely stop casting due to LOS. Channeled spells have no cast time before firing, they fire instantly, and instead persist for the length of channeling.

See the WoWWiki guide page specific to [channeling](http://www.wowwiki.com/Channeling#Differences_to_spells_with_cast_time)

>In addition, they have only one line of sight check at the cast of the spell; as soon as the spell hits the target, the caster does not need to >maintain LoS in order to keep the channel active.
Swahili?
It's now possible to Cyclone on warrior under bladestorm :s
as tested it in official, he hasn't do any resize, it is database issue... as checked in whydb, his scale is ok, don't know, why this even has been added, talking about ThukTheDefiant
I love you (:
What's the point of this function (or conversely, what's the point of the macro) ?
This way, if the file is not found (and so an 404 status is already commited) the filter will just return
instead of failing with a `NullPointerException`.
Выдели сам вызов метода в отдельную функцию. Нахуй дергать понос 2 раза за одним и тем же
?
Anyone against merging this?
http://baystation12.net/forums/index.php/topic,3377.0.html
Thank you very much! Very much appreciated.
that makes sense. I'll try to pull this in soon. It breaks a couple unit tests that count number of characters, so I'll have to update those, but all for the better.
I see. A bit tough to do. I'll wait for Thibaud input on that.
trailing whitespace (/me whitespace police ;-))
musste auch länger gucken, bis ich gesehen hab, was verändert wurde, finds aber gut!!!
It would be good to have this logic in the config file, not code.
Yeah, need to test it there. Can you think of a more reliable method?
The only other I know of is to keep a map of weakref(thread) ->
socket, and occasionally look through threading.enumerate() to see
what's died. Scott says the Java driver does roughly this.

On Thu, Feb 23, 2012 at 4:17 PM, Bernie Hackett
<reply@reply.github.com>
wrote:
> Careful here. This may not be reclaimed quickly in Jython or PyPy.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ajdavis/mongo-python-driver/commit/72f0d84c83d049c76c50830bb4d39e16be1904a9#commitcomment-1004417
Evan, how about adding support for groups? I think it would play together nicely in river context
You're really close here. Two changes:

1. In if-statements we are comparing, so we should be using 2 equals
2. You're comparing the value from the select box to '#bor', but if you look in the value fields from the HTML, there is no #
Makes sense, thanks for the clarification Tom, very appreciated.

On 26 February 2012 11:49, Tom Giordano <
reply@reply.github.com
> wrote:

> blade is 6002 or 6102 I think. You could try backporting arcee's 2.6.35
> module to 2.6.32.
>
> The module provided by ZTE is compiled against the kernel source they
> provide, so it will load in kernels built with that source as long as all
> the related bits are configured correctly.
>
> On 26 February 2012 22:43, Ricardo Pinho <
> reply@reply.github.com
> > wrote:
>
> > It work well on the 2.6.35.x branch of the z71 kernel, but most use a
> > 2.6.32.x kernel because the .35 has problems with battery drains (around
> 3%
> > to 5% per hour). Blade uses AR6002 or the AR6003?
> >
> > I thought the source was needed to compile a module so we can link it to
> > the kernel so it actually loads it.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/TomGiordano/kernel_zte_blade/commit/f4c15a5f8363cbb5400445c2b43ea464b6930a18#commitcomment-1012232
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/TomGiordano/kernel_zte_blade/commit/f4c15a5f8363cbb5400445c2b43ea464b6930a18#commitcomment-1012245
>
Il titolo del commit è sbagliato. Si tratta di morituri 0.1.2.
Xaudiocopy non funziona più con morituri 0.1.1 che per qualche ragione non riesce a rippare con il profilo 'wav' e bisogna lasciare il profilo predefinito 'flac'.
I'm reasonably sure these imports are not used here. But tell me if you know of a case where this is not true.
If the `focused` argument is removed here, `draw_torrentlist_item` can no longer make a difference between torrent listing mode and torrent detail mode, where all torrent information should normally be displayed regardless of the main UI being in compact mode or not. See also line 1591, where `draw_torrentlist_item` is called as well (with the wrong argument count now).
I think comment is frivolous after the change we made to not need the pointer and such.
Nice work, hopefully with the few adjustments I suggested we can get this released soon!
Dont you have to exclude this from SILVERLIGHT?
Ahhh you are right. We probably need some test coverage on this code. But what I meant to do was:

```ruby
reading = Reading.where(user_id: user, readable_type: self.class, readable_id: id)

if reading.any?
  reading.first
else
  reading.create
end
```

Can you fix that and start adding tests for this feature?
Thanks for the patch! I didn't use it because you were missing the change to %aws_secret_key% as well, so I needed to go in and commit a change anyway. I thanked you in the commit note thought :)  Cheers! --mark
Looks like that would create some false positives as well, e.g. bin laden - though makes me think that there could be a separate (broader) trigger down the line around plugin discovery that could pop a try this line.
Hi Josh. In regard to books, I also liked "Little Black Book of Entrepreneurship": http://www.amazon.com/Little-Black-Entrepreneurship-Fernando-Trias/dp/1580089321

Best regards,
Daniel.
If all the generators are working too, do you want to request a pull? I think the readme should be updated as well, referring to it as the Rails 3.1 asset pipeline.
I have the following worlds,
OurCraftV5
OurCraftV5_nether
OurCraftV5_the_end
OurCraftV4
OurCraftV4_nether
OurCraftV3
OurCraftV3_nether

V3 and V4 are 1.7 and 1.8 worlds respectively, and have been pre-populated and have worldborder running on them, so that they no longer grow and not a lot of activity is going on with them, so they run from the physical drive.  Only OurCraftV5 and OurCraftV5_nether are loaded onto ram disk.
 
I could do this kind of split with your original script, by just leaving the worlds that are not loaded onto ram disk in $SERVER_DIR/$BUILD, but that means moving the directories around when a new $BUILD folder is created or doing manual symbolic links with each update.

The OurCraftV5 worlds are stored in /srv/minecraft/ramworlds and the rest of the worlds are stored in /srv/minecraft/worlds.  The way I have the script working is mc_link creates symbolic links for all worlds in /srv/minecraft/worlds and /srv/minecraft/ramworlds into $SERVER_DIR/$BUILD or if world-container is set into that directory.  Then when mcrd_load runs, it overrides those links just for the ram disk worlds and points them to $RD_DIR.

Now that the world-container setting seems to be working with multiverse, I could keep my physical disk worlds in a directory outside of $BUILD and have the ram disk world’s link into it instead.  I’ll have to test to see if CraftBukkit or Multiverse cares if world-container is not a sub-directory from craftbukkit.jar. 

For the small number of users that I have, using a ram disk for the main world is really over kill, but it does seem to help during chunk generation or when someone is rapidly digging sand.  I also figure might as well use the ram than having it sit idle.
"mas se as pessoas no leem as issues e comentam, acho q no tem o direito de reclamar." >> reclamar é uma forma de comentar, se todos estão no mesmo projeto todos tem direito de dar a opinião e reclamar. seu argumento é inválido!

" se elas no afetarem o projeto como um todo e nem quebrarem testes de outras pessoas,
no vejo o problema de serem instaladas." >> concordo plenamente. porém instalaram o Inherited_resource, formtastic. e acho que eles influenciam em TODOS os controllers/forms. novamente argumento inválido.
I solved the problem by adding an option for manual execution of the jasmine runner in jasminerice: https://github.com/bradphelan/jasminerice/pull/33

This can be used for any cases where the developer needs to control the exact time of execution ;-)
I'm not sure what you mean by "real" hardware MAC addresses? The value from /sys/module/g_android/parameters/serial_number is the base serial number unique to each device, and the stock rom appears to derive the BT and wifi mac addresses from the base address simply by incrementing it by 1 and 2 bytes, respectively.

I checked a dmesg log from my stock rom, and I can see the usb_serial announced (identical to the sysfs node above), and the wifi address is also listed, simply 1 byte incremented and formatted properly. The value is not the same as yours.

Perhaps there's a better way to do this (e.g., on the stock ROM, the dhd.ko driver appears to be using some ioctl to get the mac address, and I have no idea how BT finds it), but that's not really important anyway :). The only problem is that the script is a bit dumb and will not yield valid addresses when the final two digits are >0xFD.
Er, at the very least the conversion should be done in `cleaned_data`. I'm not sure if accessing the value directly on the form gives the converted values or not.
Perhaps we can have a `middle:` key for names that have a clear middle name. What do you suggest for multiple maiden names? (e.g., Eliza Roxy Snow Smith Young might be `{last: Snow, first: Eliza, middle: Roxy, additional: { Smith, Young}}` but I'm not entirely sure yet.)
This does not work in jQuery 1.8 for me.

This does: 
        my_textareas = this.filter("textarea"),
    result_textareas = result.filter('textarea');
eval is not a best approach for a moment.
That sounds much better - see 809ae9e544dea5d823b7d8fc04ff9d488af9f1cb
This commit causes strange bug.
In example: Given a media file named: _01. Smoke On The Water (Deep Purple Cover).flac_, this code will try to treat it as an album art image! ( _"SkImageDecoder::Factory returned null"_ in logcat).
Moreover, the media file is copied to albumthumbs cache directory...
I suggest a small change to the condition - file extension check:
```java
if (entry.equals("albumart.jpg")
  || ((entry.contains("cover") || entry.contains("front")) && entry.endsWith(".jpg")) {
```
Requiring itself ? Seems weird
Sounds great! I think in my playing with this, I'd rather see it pick one of the ambiguous matches, rather than raise an error. Since there isn't much danger or cost to setting the wrong one active (especially now that it displays which account is set active), picking the first of the ambiguous options seems more useful than raising an error. 

That way, if you know your system will always pick `sam` account when you type `t set active s`, you can learn to use that as a short cut, and be more explicit only when you need to.

I suppose this would be a problem if you have 2 similar accounts, one not yet authorized. If I have `readysetgoal` and `readysetgoals`, a typo might result in a surprising selection that'd be hard to quickly detect... but that seems like the minority case. 

Your call on how to balance the trade-offs!
I suggest we have this just be a . property. No need to expose it in the config, since it's private. As with the mousewheel flag, if it's only there until we're confident we've ironed out the kinks, then that seems sufficient. When we're confident it's working fine, there's no reason anyone would want to not-optimizeMemory.
Ugh, IO is so cryptic :(
Can be extend it from BCommonProperties.theme?
It has effect. It is very trivial and hard to distinguish. Please enlarge the display and check it.
So until hide is called, its seen?  Also, would screen readers see it?
Do we actually want require the same name for equality?
Why bother wrapping these functions in a class?
so it's a "squaredIntervalSum"? ;-)
Why this catchall?
I know. I'm older and wiser now. Actually it was not at all this split into two test classes that I was thinking of now. You want to test that transitions Joining and Up are notified to the registered listener. Use one listener, and inside that send something representing the notification to `testActorRef` and then afterwards use `expectMsg`. No need for latch. You don't need to change this, but it can be a technique to keep in mind.
what is recv?
do you mean `i`
whats the purpose of throwing this?
To clarify: What makes the facade ready to the second node before "done-2", but not for the others? Why does adding a new barrier solve this (apart from increasing the delay somewhat - but then why is it working for the second node)?
What about passing a "start = true" into init and remove "def start" and the flag altogether?
We use it for our testing of the failure detector.
I don't think it's necessary to expose that to users. The less api the marier.
I think it should be scoped to the failure detector (define inside it's companion obj).
Pretty simple parser API, no?  It returns an ordered set of pairs (hash), the first is the main card.

:key=>:name tells it what attribute to use for the hash keys, :element names the element we are matching: <card name="..." type=>... </card>
:substitute tells it an attribute it can use to replace the element.  These substitutions apply to the hash values and the result is to make the values back into the raw content (assumes an attribute has the original chunk text).  In :extras you can name additional attributes you want it to extract for you.  So, I get the type, the name and the original contents, just what we need for creates and updates.
Is this test still required since contains() accepts only mesh objects and it is already tested in PolygonEdgesTestCase? Does it cover anything more than that?
Here you are calling the method 'resample', but as far as I remember, this method does not assure you to get a line sampled in a number of points equal to num_points (because of corners getting cut and rounding effects). That's why in my prototype I used another method that exactly returns a line sampled in n points (https://github.com/monellid/HazardEngine/blob/master/geo.py#L199)
That's so pythonic, it's making me smile
I doubt you meant to leave these print statements in. :)
It'd be nice, but I can also expect people to understand that it's tricky to do so. Is there a way for me to manually go in and dork with the next run time?
we'll need to get the migration wiping in immediately.
why is this necessary?  if permission_rule_card's trunk is not a set, isn't something horribly wrong?
The word combinatorically (aka: combinatorially) has no place here.
In order to transform image data, please implement a special JAI OpImage. However, JAI already provides the "log" operation. See javax.media.jai.operator.LogDescriptor.
GeoExt -> GXM
This should be passed to the constructor. If not, it will be guessed and this has some overhead.
It definitely couldn't hurt. How about we get this in first, though, and then parallelization can always be added afterward (either immediately, or when someone finds it necessary)? Any new features should go into a new pull request, which will help ensure that this one gets merged (since it's not a moving target for review).
Style: we use license boilerplate like this:

```c++
// Copyright Joyent, Inc. and other Node contributors.
//
// Permission is hereby granted, free of charge, to any person obtaining a
// copy of this software and associated documentation files (the
// "Software"), to deal in the Software without restriction, including
// without limitation the rights to use, copy, modify, merge, publish,
// distribute, sublicense, and/or sell copies of the Software, and to permit
// persons to whom the Software is furnished to do so, subject to the
// following conditions:
//
// The above copyright notice and this permission notice shall be included
// in all copies or substantial portions of the Software.
//
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN
// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
// USE OR OTHER DEALINGS IN THE SOFTWARE.
```
Needs more detail. Only `prctl(PR_SET_NAME)` is limited to 16 bytes.
Suggestion:

Note: Events may be raised by several threads, such as the GUI thread and the audio thread.  If a subscriber method calls operations that must be executed in a particular thread, then the subscriber method is responsible for posting those calls to the appropriate synchronization context.
Only needed for callbacks; please remove for now.
Are you still working on these tests or did you mistakenly submit some empty ones?
Typo here. Use tabs instead of spaces.
Alright, thanks. I don't have access to my dev environment at the moment, so I'll have to fix what you commented on tomorrow or the day after.

Thanks again, Trustin :)
This "array" should be imported from the line `from multiarray import empty, dtype, array` in the same function. I'll fix it in the next patch.
Style, opening brace on preceding line. Same below.
yes, Will change it to expires.
Or actually, you can use the id `#routeTable`.
can you change it to numeric? 'numbers objects' is odd.
Do you need the transaction block to create only one table?
Since it is public we should remove the `#:nodoc:` and add real documentation to this method.
I don't think we need to have this in here, I believe that should be handled by the validator itself in this case. There's no need to extend `Errors` API with this.
On Wed, Apr 25, 2012 at 11:22 PM, Ronan Lamy
<reply@reply.github.com>
wrote:
>> @@ -438,8 +438,14 @@ def __init__(self, rules):
>>
>>  class InconsistentAssumptions(ValueError):
>>      def __str__(self):
>> +        from sympy.utilities.misc import filldedent
>
> This exception can be raised very early in the bootstrapping sequence of sympy. Importing from sympy.utilities cannot be assumed to be safe at this stage.

filldedent only depends on python built-ins -- does that matter?
Yep, better :)
On Sat, Jul 21, 2012 at 11:20 AM, Adriaan Moors
<reply@reply.github.com>
wrote:
>>        val targs = solvedTypes(
>> -        tvars, tparams, tparams map varianceInTypes(formals),
>> +        tvars, tparams, tparams map inferVariance,
>
> would it make sense to do the same in protoTypeArgs?
>
Good catch. I'll add this to the pullrequest. - Martin

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/scala/scala/pull/959/files#r1210653



-- 
Martin Odersky
Prof., EPFL and Chairman, Typesafe
PSED, 1015 Lausanne, Switzerland
Tel. EPFL: +41 21 693 6863
Tel. Typesafe: +41 21 691 4967
Great work! But I would add a couple more tests:

* the object is defined inside a `val` or a `def`.
* the object is defined inside an early def section
* the object is defined inside another object, and *both* need to use this trick
I think the idea is it shows it like a basic image rather than a plot? I don't know I just left it in.
Use automatic string concatenation, and that `'` is not special inside `""` strings:
```
raise ValueError("The number of elements in the permutations "
                        "don't match.")
```
Just make these comments part of the docstring.
*I would prefer fixing the parts of sympy that do stupidly naive recursions of the expression tree instead of continuing this.*

Sounds like a good plan. A fine example is the use of strings in args. It would be nice to have strings without making a String(Basic) object.
I have tried this but It is not giving correct results because we need to change the powers if tansec is detected to convert it into sincos.
I would not be able to do that it both the patterns are in a single loop. Please help me to get around it
Removing those lines in next PR
few -> a few

:black_square: cps stopping point
refactor notu duselim kendimize; 30. satirdaki this.element = ... ile bu if (!this.element) ... else .. kismi createElements metodu icinde yer almali.
Ah, you know ... "Be conservative with what you give and liberal with what you receive". I already reworked some of this filter's logic so it's complying to the RFC ;)
Please, try to keep consistency. You have space indented code and there are no trailing dots (look at the other methods of CHttpRequest).
I think it is okay as it is.
If we'd prefer to drop 2.5 support, we could just `warnings.catch_warnings()`. probably not worth it, though
were you going to validate the `type` against the `validTypes` array?
It's not possible to shuffle a little bit?
Kolman,

I need you help to resolve this issue.


Thanks,
Slava.

Git Hub Reference:
add 'lib' to all paths.

Did you try building the gem? Installing? Using?
Using gem means the gem itself and not the code directly.


-----Original Message-----
From: kolmanv [mailto:reply@reply.github.com] 
Sent: Thursday, May 17, 2012 8:09 PM
To: Slava
Subject: Re: [bbfs] File generator (#32)

> @@ -9,7 +9,10 @@ Gem::Specification.new do |s|
>    s.email       = 'kolmanv@gmail.com'
>    s.homepage    = 'http://github.com/kolmanv/bbfs'
>    s.files       = ['lib/file_utils.rb',
> -                   'lib/file_utils/file_utils.rb']
> +                   'lib/file_utils/file_utils.rb',
> +                   'file_utils/file_generator/file_generator.rb',

add 'lib' to all paths.

---
Reply to this email directly or view it on GitHub:
https://github.com/kolmanv/bbfs/pull/32/files#r839671
Still we may use boolean because all know what it means and inside we use TrueClass or FalseClass.
On reflection, the hint about ScanningParticipantRef is probably more effort than is needed at this point, so don't worry about it.
Same issue of returning batch of results vs iterating through a cursor to print results.
Just tried that with IE8 and seems to work. I guess jquery may provide some cross browser workarounds? 

The search doesn't seem to work as expected though for other reasons which i'm still investigating. I'll reply here with findings, meanwhile please don't merge
Why not use the "plural system" here.
Brackets (I'll stop commenting about this now: it happens a few times above this, too).

Yes, I am reading this PR backwards.
(10:48:12) nn-: i have a plan-minimum
(10:48:22) nn-: add PHPBB_NO_AUTOLOAD check
(10:48:29) nn-: if that envvar is set take the current else path
(10:48:39) nn-: then do PHPBB_AUTOLOAD check 
(10:48:44) nn-: else unconditionally require composer
octet-stream (with a dash), in my estimation.
If we're using one style in one place and another in another, then it is not a holy war.
This comment is irrelevant to this cron task (copy-paste issue?)
Depending on how the AUTHOR_* ones are displayed, (i.e. if it is "Author Information" and then all the info beneath it), you don't necessarily need to have "Author X", but just "X". For instance, instead of "Author Name" it would just be "Name", and since it would be under the "Author Information" header, it would be clear whose name it is.

But that's nothing big, just wanted to point it out.
Can you explain the algorithm you are using here? It is period minus a half second? Why is that?

Oh, maybe I get it: you don't want the check to run into the next check period. A comment is probably in order.
sigur ai spații și nu tab-uri aici?
Why not just null?
Где хоть какая-то секюрность?
yes it goes. This is corrected
same comment as above regarding CSS/formatting. Applies throughout.
Thanks for tidying things up. :smile:
Missing s in "feature".
Missing some words after "of"?
This new param doc doesn't jive with the method itself.
processor has data from db
I don't see this being common. The `vnode_vclocks` code is kinda bizarre in that it changes the actual request sent to a node based on that node's operating mode. Everything else in Riak sends a request to Riak, and then the Riak node determines what to do with it -- ie. legacy vs pipe m/r, legacy vs proxy vnode routing, etc.

If this ever comes up again down the road, then we should refactor into a general purpose utility func. Or, better yet, come up with a different approach that isn't based on RPC'ing to Riak everytime a client creates a connection. I just went this route because it mirrored the pre-existing RPC query of the app var.
Done. Function moved and made static since it's now local to effects.c.
Bummer. Right.
Right, but I think to be 1.1 you need to handle chunked.
`return false if page.nil?` to prevent `page.path` from raising a nil error. There have been a bunch of issues with nil pages so it's important to guard against that.
More ideas:

- `Guzzle\Service\Plugin\GlobalPluginAdderPlugin`
- `Guzzle\Service\Plugin\MultiServicePlugin`
- `Guzzle\Service\Plugin\MultiClientPlugin`
Leave this empty line in for readability.
no space there
it would've infinite looped when `elem` was actually at  0,0, because `testEl` would be 0,0 as well and it would try adding a test element to `testEl`, _ad infinitum_
How does the absolute check prevent infinite loops? Can you make a unit test for this as well?
p.s. I can probably head pretty far down this road without additional direction (though all input is welcome of course!)... held back until I got the go ahead from you.
Have you checked what will have preference of a mopidy folder in static_dir and the /mopidy mount point?
No... I think you should add comment, though. Otherwise it's not clear why you're using these flags.
Do we need to return here to avoid raising the exception or returning false? Or maybe `retry` rather than duplicating the `rmdir; true`?
You assign `S_FORCE_UNINSTALL` the value `true`, yet you are checking for `S_FORCE_UNINSTALLED` in acp_mods.html.
the `root`` variable is not used in this module
Good catch! Done, see 834dcd7b89a57436c7aa2cb68dbc041fa03a89e1.
The indentation below doesn't match our style of two spaces for tabs.