labofsahil/github-event-dataset-2012 huggingface dataset:
Add support for Navigation Lists

Hi Piotr,

I added support for [Navigation Lists](http://twitter.github.com/bootstrap/components.html#navs).
Please have a look at the examples in the README. Just a quick one:

    bootstrap_navigation do |nav|
      nav.link_to "Nav1", "/link1", :active_nav => true
      nav.link_to "Nav2", "/link2"
    end

which results in the following html:

    <ul class="nav nav-tabs">
      <li class="active">
        <a href="/link1">Nav1</a>
      </li>
      <li>
        <a href="/link2">Nav2</a>
      </li>
    </ul>

I also added two minor changes:

- added color to rspec :)
- added redcarpet as a development dependency, otherwise `bundle exec rake yard` fails (at least under ruby 1.9.3).

Do you agree these changes?
Fix rubygems.org profiles URL

* wrong: https://rubygems.org/profile/profiles/okkez
* correct: https://rubygems.org/profiles/okkez

I fixed this problem.
"Promptly"
Review: Make empty instances statistic more accurate.

When we're down to just one layer and we determine it's empty and cull the whole group, we should count that layer, in addition to the group, in the statistics about how many empty things we culled.
LPS-25031
LPS-24633 After publishing an article with a translation, changes to the...

... article's structure are not localizable
when the route parameter is set but is the empty string, do not take it

Use case:

A form of a MenuItem Document is saved with the route field as empty value. It gets saved as an empty string. The Router checks the existence of the route parameter, even when is an empty string, and crash happens because is not a valid value.

A related fix would be to discover why form fields with empty strings are not saved as null in doctrine phpcr and fix it (in doctrine orm they get saved as null), but anyways, if the route parameter is an empty string it would be nice to fallback and try to get the route from the content parameter.
New Web Services sample script - replace Config Set with Content Type

Script to find all pages with a specified Configuration Set and apply a specified Content Type instead. Based on discussion here: http://help.hannonhill.com/discussions/how-do-i/1000-apply-content-type-to-all-subscribers-of-configuration-set
Attempting to fix nav highlighting issue

This may or may not work, again.
Changes to the ReadyNAS installer to make it work for ReadyNAS ARM

Modified the general installer to copy the right bzip2/bunzip2 binaries and silenced the install script so the Add-on updater on ReadyNAS ARM won't choke
HPBladeChassis 2.0.2

Hey Simon

I hope you are enjoying the holidays! As discussed I forked and created this. I am not quite sure if you want to merge it or keep it as is. Perhaps it is better to have two branches on the repo one for 2.5.2 and one for 4.1.1. Let me know what's the best way.

Regards
Kiarash
More tweaks and updates to keystone and glance, nova still needs love. [2/3]




 chef/cookbooks/keystone/providers/register.rb      |   12 +++++-----
 .../keystone/templates/default/keystone.conf.erb   |   22 +++++++++----------
 2 files changed, 16 insertions(+), 18 deletions(-)
Convert @memoized into a function
(fixes #2641)bring translation code up to date.
Fixes #51 Paginate tag missing mapping attribute

Included fix from fazlan for bug #51. The paginate tag now supports the mapping attribute.
Beshr Test
Here is a sample pull request

hi @mecathcart 

what i did:
* I added a class called Corpus.js and put some notes as well as some functions that i think the corpus will need.
got rid of unneeded service
Add string translations in Japanese

japanese translations for you
Updating italian translations.
Helper functions to replace standard browser popups

I have created replacements for alert(), confirm() and prompt() in dialog2, inspired by [jqDialog](https://github.com/knadh/jqdialog).
Implement story clone in iteration view.
Change output location of webplatform.js and i18n.js to the app root
UI bug fixes and behavior

Fixed bugs with UI login and initial update buddy list issues.
enhanced touchscreen configuration

this enhance don't remove jitter but add more info about touchscreen

src: http://forum.xda-developers.com/showpost.php?p=23201590&postcount=41
Add a final() method on View

This is a my first pull request on github. don't be rude :)
Bring in LED Settings

Credit: CM
Updating the title of my talk
pass extra args to the constroctor up to TestCase, the parent, so it can hand them off to subprocesses

pass the  arg to the constructor up to TestCase, which will keep track of it and hand it down to subprocesses once phpunit-parallel gets released

without this and my recent changes to phpunit, phpunit_selenium can never run with process isolation or parallelism using a public static $browsers array, which is a standard feature of phpunit_selenium
Add the two getDescendants() methods to the Java client
PLFM-1527: create local ACLs by inheritance; PLFM-1438: batch UserGroupHeader REST service
Fix updating of multiple interacts in the same session

We were previously over-zealous in rejecting messages.
Added some error checking upon hitting the OK button

To prevent invalid name and IDs without clicking on the name or ID fields
setAssetBasePath for TMXLoader and TSXLoader

This is the same as nicolasgramlich/AndEngine#146 but for GLES2.
Made some changes so it actually sounds better in spanish. There was a m...

...ix of formal and informal sentences.
Great presentation Ryan - spotted a few typos so fixed incase you wanted...

... use this at another conference :)
database mirroring support

Sql Server database mirroring support for dblib and odbc connections.

Mirror server could be specified in database.yml. In case of failed connection to primary server, adapter will try to connect to mirror.

Add :mirror section to your database.yml with mirror server specific connection params:

    :adapter => 'sqlserver',
    :mode => 'dblib',
    :username => 'rails',
    :password => '',
    :database => 'activerecord_unittest_mirroring',
    :dataserver => 'primary_server',
    :mirror
      :dataserver => 'mirror_sql_server'

For running test see ./MIRRORING_HOW_TO.md.
Updated Mms/Files/res/raw/sms_frequently_used_phrase

It look's like to be in russian because it's russian translate :)
indexファイルとライセンス、READMEを追加

元ドキュメントと同一のライセンスを追加しました。
また、簡易な説明を書いたindex、READMEを追加しました。
Minor changes.

* Reply url is #/ not /#.
* Made "You have no new notices" message not appear if you have
messages.
* Commented out notification when it kept throwing security errors (not sure why).
upgraded to grails 2.0.0
Added sonar functionality
another handy post processing script example

this script generates a sane possible name for your screenshot,
allowing the user to override the guessed name via a zenity dialog.
Fixed changelog and version string
Add exception handling resolver and upgrade to use Tiles views
s/for/to/
This is just a syntax fix mentioned by a user. Not messing with the scripts logic ;)
Is it intended that you give `true` as minVersion? This is the same as

    (isset($package['minversion'])) ? isset($package['minversion']) : ''
"It is useless and can be easily implemented in user-land."

Implemented how? via .close() and then .listen() ?
This is the embarassing bit I need to get rid of - hardcoded to /relax/openssl/include
Noted.  Would you call it stable, or unstable, then?
Ueberhaupt ist die Ada-Aktivitaet auf Github doch sehr ueberschaubar.
First, Yay you are contributing! *celebrate*
Now to be a pedant, note that this assignment is happening every time the loop runs. This is almost sure to have no effect on the current game, but be aware when things are in loops.
Your additions were otherwise perfect. Keep it up!
Nice. thanks for coverage as well.
That seems to be working for me. Great job, thanks!!!
ok. Thanks for the clarification.
I think hyperexpand on top will be better because of the same reason
Typo, 'immediately'
I dont believe thats the issue deathmax.

public static bool CheckRangePermission(TSPlayer player, int x, int y, int range = 32)
{
if (Config.RangeChecks && ((Math.Abs(player.TileX - x) > 32) || (Math.Abs(player.TileY - y) > 32)))
{
return true;
}
return false;
}

That method is a little fucked.  It never uses size.
You really aren't supposed to be committing to the new API. The goal is to make plugins out of every TShock component, except the C# API.

"the server" == ?
Not afaik. I did some research on this back when I originally did it in DM last year. You're welcome to check and see if any other built-in (or methods supplied by Virtus::InstanceMethods) need to be excluded.
I'll include SQL queries. What exactly would you like me to test?
I removed this to reintroduce. This code did not work ever ) I will introduce choose components asap.This is the next feature i've planned.
In install-deps.sh.
Typo -- compatibel -> compatible
Patch looks fine now, but I think you should change this part of your commit message:

    - Changes done after o11c's code review.
    - Changes and simplifications following Bjorn's review x2.
    - Removed < > arrow signs fallbacks on bjorn's request.

The problem is that it describes changes you made to the patch, which are now irrelevant. The last line even mentions a fallback that wasn't there before the commit so it can't possibly be removed. The commit message should describe what the patch is doing, and who reviewed it. So I'd expect the above to be more like:

    - Replaced the < and > arrow signs with images

    Reviewed-by: o11c
    Reviewed-by: bjorn
Yes, but please comment a bit on the problem in your commit message. Took me a bit to check why passing the attack type to `handleAttack` was wrong.
I searched for it, too but I hadn't the time to find the value in the tA code for now.

I'll change this for (32)*30, to be sure.
What are coding conventions on this case?
No I hadn't, that's just a coincidence. I removed this since the only useful property which was needed for paging was totalRecords (coming from SearchResults.numberOfRecordsMatched), and I did not see any purpose for any of the other properties of the result object from the OpenLayers parser. But jsonData catches this use case perfectly as you pointed out.
@dcramer I think it should be djkombu here too.
Thanks, the `setuptools_git` is indeed helpful, although I will postpone making changes to the setup.py until I move to distutils2/packaging, which will be much more future-proof.
What is the idea behind the tempDirectory, I noticed that all the factory now have it has a ctor param ?
These aren't very useful, anyways, will probably delete.
It's probably better to abort playing the audio if the sample rate in the file is not available for direct playback, instead of forcing an incorrect rate here.
Oh.. I forgot to mention it.. I changed the "color" options of ls and grep to color=always instead of auto because I wanted to be able to see the colorized output when piping it through my pager (I have the -R option set up in my LESS variable so that I can see the colors).

There's also the "engange" funny command for listening to the star trek enterprise engine :P
Uh, good point, I don't actually know how this ended up in this commit. Something totally unrelated.
Проверка можно ли двигаться. Мотан же.
I don't know why but isn't working for me, when i click the editor i can't see the previous content.
I don't know how to use Eclipse debugger. I find those things confusing. The console in Chrome is better for me.
Yeah you are right. So we need this dereferenced sorting.

On Sat, 2012-01-07 at 16:10 -0800, Heiko Strathmann wrote:
> I am not sure, but I think it is not possible to overload the comparison operators for pointers, like this:
> TParameter* a; TParameter* b
> ...
> a<b // compares by name of dereferenced TParameter
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/karlnapf/shogun/commit/1239e9cf3ffdf4301269920d7351fa55c8283426#commitcomment-846912
> 

-- 
For the one fact about the future of which we can be certain is that it
will be utterly fantastic. -- Arthur C. Clarke, 1962
This is wrong. Negative values on the filter mean the filter should not be applied. If you look at `filter.c`, you'll see that we check there for OOM conditions.
sorry did not notice on "return false;"
Mmm, 
member.lastMap   = player->GetMapId(); might be a problem. I didnt test player pointer before. But when we can add group member who is offline??? Anyway few line after there is a test for player so it make sense that pointer can be null.
this commit is to long for a good overview
Ah, I got the point.  Thank you.
included your update in the hookbox repository. thanks.
Well, added to my 2k11 repo, this will go in s9y in a later commit.
same as `case class Tournament(players: List[Player]) extends {`
but I thought function names with leading underscore are already included. ?
980px?
That would work for me.  I can change my detection logic to support it, but I don't understand what the extra tables are for.

The module's path could possible be passed in the environment for the luaopen_*() C function, but wouldn't it need to be a new table and not the default?  I am not sure but I think the default environment for C functions is the global table?
hi !

It's not clear to me if I have to add those @Inject's
in SendCommandTest.java :

  @Inject
  private ProtocolSessionContext sessionContext;
  @Inject
  private ProtocolSessionContext protocolSessionContext1;
  @Inject
  private ProtocolSessionContext protocolSessionContext;



thanks !
So when some column is up and you standing on it you can use spells on it? and also use totems and traps from hunter on place where are elevators?
Ok, não tinha entendido como passar o controller.
that's what I get for not waiting for the whole directory to be copied before I commit... 3 separate commits to add this codemirror library
looks good- thanks for the adjustments!

r+
/?>?./,.mn
create .gitignore file to exclude binary :)
return make_link, nicole crème...
Since you are making this comparison so many times, why not make a function validate date, so if we figure out another thing we should check in the future, there will only be one place to change in the future instead of 6.
Укажи -> Укажите
or change the entire sentence to 
Выберите, что могут делать пользователи, обладающие этой ролью, из списка ниже.
fbfbe222714882be42b0d515f4b744442e413b70
`public_get` Is just an alias, see [this](https://github.com/derdesign/core-js/blob/master/lib/controller.js#L90)
What about the `head` of the `NonEmptyList`?
I am going to revert all these all they don't check the running mode.
it *should* be fine in theory since the page height would be the same but I totally agree it's safer to do it in both cases since we have the toggled messages and such
I'm confused by this. The description says that the children don't need to add up to amount of the parent column, but the example below looks exactly the same as the fixed grid example. Fixed grid uses 3+3 inside of a 6. Fluid uses 6+6 inside of a 12. They seem to function the same even the description says they are a bit different.
Wouhouuuu !
I've bee doing some tests locally and dind't manage to make it work. The problem here seems to be on the spells like 68282 wich triggers that spell with TARGER_95 too.

I haven't found a way of making a proper aproach for solving this. Any idea?
It looks like my commit changed the .classpath and .project files, each in one location. I think this is due to running slightly different versions of the android SDK. Not sure what to do about that...
Note: this algorithm is known as "flood-fill"

http://en.wikipedia.org/wiki/Flood_fill
any file:
#include all.h

all.h:
#include h1
#include h2
...
#include hN
@ugisozols https://twitter.com/#!/damirca/status/155637699246686208 this is why we had this code.
You guys should check out what [Apache Tika](http://tika.apache.org/) and [pantomime](https://github.com/michaelklishin/pantomime) do. They default to application/octet-stream.
2 제목도 한글로 바꾸는게 어떨까?
This gist is a specialized version of my [deep-indent-dwim advice](https://github.com/dgutov/dot-emacs/blob/master/progmodes.el#L151), which decides whether to use deep-indent depending on whether the first element after the opening bracket is on the same line.
You might want to take a look, I found it mighty useful.
Yes, we have ;)
IMO 1 line of code is cleaner that 3 to do the same thing.
The user has been added / already exists. Both is ok for us
Thanks for reporting. So far the integration has been tested with the "default" _computer-database_ examples where no errors occurred. Could you please provide a reproducible test case for your issue?
Could you explain why it was misleading? It seems there was more information before and now there's less.

The text was wrong (copy paste) but having something like "Changed document parent from [%s] to [%s]" is nicer IMO.
same question here re ct_id = 5.
I uncomment to load all the P/D from the group?
@bbatsov I think you missed a negation here, should be
     when (not (package-installed-p p)) do (return nil)
а что нужно было помещать?
I think that was a fail git comment :p
Translation: Removed before release :P
I don't know if you are missing something or not. I see that Dave had
review the code, but I don't know who pushed it.






On 1/31/12 8:36 PM, "Chase Brammer"
<reply+c-921258-d24f08c6c6390aabe7dfe63c70f00a407452a75f-375008@reply.githu
b.com> wrote:

>I don't see any place that redis is actually being connected to. That may
>be by design, which is fine.  Am I missing something though?
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/ifit/node-boilerplate/commit/99eecab15ea48a7eb6676272e0
>f43de715d9b049#commitcomment-921258
>
How so?  Wouldn't this just be a matter of adding another method to the plugin interface and implementation?

> Any reason this wasn't brought up earlier?

Speaking for me personally, the longer something is in review, the more that occurs to the reviewers.  That seems natural and healthy to me.
Could you fix the spelling on this?
"For more details about the project methodology ... "
I'm not sure if XML/Xml or HTTP/Http is more appropriate
But, I went ahead and merged anyway.
Thanks. Try this

https://github.com/evanmiller/ChicagoBoss/commit/82ea0d3f017fbc6cbb106ccb877aaeeaaccec890
This runs the default layout - is it used?
@Ze70r i dont think you understand what i wrote, i never been disrespecting, or do i misunderstood you now.

what i mean is, kandera for dev, better sooner then later, loads of nice fixes from her and i think she will be a great contribution for trinitycore.
UnresolvedAddressException is not part of the IOError hierarchy so its not caught by the existing catch. Sorry about the tab/space mismatch.
Yep. My only doubt was about 3.3.5 having unk uint32 and 4.x having unk uint64, so I used ReadValue. I can't name them differently whitout an "if" there.
4 is perfect, but soft tabs, not tabs.
Yeah, OK, let's merge it first. Maybe add a TODO in case someone else can get to it first if you're busy? But yeah... let's not drag out the pull request.
There is a bug. Accordingly documentation [API_SIGNATURE] calculate as: md5(token+skey).

    sig = md5(token+settings.API_SIGNATURE).hexdigest()
    params.update(id=settings.WIDGET_ID, sig=sig)
prepublish basically compiles `brunch-extensions` at my machine before it publishing to npm, so user doesn't need to compile it by himself.

What kind of error do you have?

`brunch new application` creates a valid app for me with `brunch-extensions` 0.2.0.
I'm testing the change this weekend and will merge it afterwards.
The local variable names are overly verbose for me.
I think 'stderr', and 'stdout' is fine if you add documentation to 
indicate their class (IO).
> I think the problem may be with font name. SSA styles use font titles like *Arial* not exact file paths.

Indeed, that is what it looks like.

> On linux systems we can grab list of registered fonts with `fc-list` command, but I don't know if it works on Mac. 

Yes, it works on OSX as well. `fc-list | grep Arial` returns a list of 13 fonts, e.g. this one:

    Arial:style=Normal,obyčejné,Standard,Κανονικά,Regular,Normaali,Normál,Normale,Standaard,Normalny,Обычный,Normálne,Navadno,thường,Arrunta

> Try to enter font title like Arial in GUI and also enable fontconfig to see if it makes any difference.

Didn't make any difference, so I took more drastic measures and hardcoded font options into the command line to see what would happen.
These are my conclusions after a lot of testing:

 * Subtitles only appear when `-subfont /System/Library/Frameworks/JavaVM.framework/Versions/1.5.0/Home/lib/fonts/LucidaSansRegular.ttf` is present.
 * When it is present, the font name used in `-ass-force-style FontName=` is ignored. The other options (Outline, Shadow and MarginV) are still being used to force the subtitling style, though.
 * Without `-subfont`, I tried dozes of different font names, with and without spaces (e.g. "Arial", "Lucida", "LucidaSansRegular", "Lucida Sans Regular" etc.). None of them made the subtitles appear.
 * The font path actually exists. Using `-subfont` with a non-existent path would not show subtitles.
Ok, before it was just irritating... now it is getting stupid... what database are you using, a trinity 3.3.5? You SHOULD be using the latest skyfireDB.  It does not make any logical sense to add a bunch of 3.3.5 quests and no longer valid spells back into our conditions table when they have been removed from the game and/or dbcs... Also, do you test ANY of this stuff before you push it into master? I think if I was testing something (as you should be), I would have seen the 150+ startup errors this generates due to conditions not being properly aligned and or valid (caused by this commit). Seriously dude, if you aren't testing this stuff thoroughly, you DON"T NEED TO BE PUSHING TO MASTER. Put it in a branch or even a fork and test the stuff out first... just because it fixes something in TrinityCore, does NOT mean it automatically fixes anything in this project. Hell, it may not even be needed. ALL stuff ported from trinity needs to be put in a branch, examined, and thoroughly tested before pushing it into our  master. 

Example: quest 25444 
http://www.wowhead.com/quest=25444 - Obsolete. Removed in 4.0.1

Another Example "Aura condition has non existing spell (Id: 48085)"... Of course it is "non existing," that would be because it was removed in 4.0.3.

Some of us are working our asses off to make a solid and up to date database... Crap like this can easily destroy it fast. Please start paying attention to what you are doing... and please revert this before the bitching begins.

NOTE: What is worse about this entire situation is that Alter made a special branch called "conditions" just for this work to be ported to... as it stands now, anyone that runs 2012_03_08_00_world_conditions.sql has completely fucked there conditions table. Anyone who doesn't run it after this commit has completely fucked their condition table... I personally am reverting to an earlier version of skyfire BEFORE you touched the conditions.
I think the default value should be dynamically linked to libc, but when packaging, we should statically link it
As you can't backport features to 3-0-stable branch we should fix the SimpleForm tests to work with Rails 3.0.x
Meaning to use string formatters on these anyway.
I'm not sure I understand this addition.
Yeah I noticed that. HTML Char codes are being put in as well. I originally did html: true because that rendered <br /> as new lines. We have to find a way to render those things appropriately without allowing for script injection.
.length = 1628 /*  203.5MB */,
Коментарий на русском это сильно :) он же развалится в другом редакторе скорее всего
+1 a poner una descripcion del TAD en el .h, sobre cual es la idea con que se origino, en este caso representar listas no sincronizadas.
En la respuesta de los ejercicios ponemos porque es mejor usando un decorate y listo :P
This removes support for Rails 3.1, which I wasn't planning on doing.
Ah yeah. Wasn't looking through all the rest. That makes sense. Don't do the hacky part. ;)
Actually, this is probably more of a problem with the discovery service than a bug in the client. :-/

The whole point of JSON schema using '$ref' as a value is that it wouldn't be something that would typically show up as a object key in JSON.
Don't mind, I'm french too :)

The rotation is done at line 83. All images are rotated here, but some of them are rotated of 0° ;)
Without underscores, I get the following error:

Traceback (most recent call last):
  File "tools/Preprocessor.py", line 477, in <module>
    main()
  File "tools/Preprocessor.py", line 462, in main
    pp.handleCommandLine(None, True)
  File "tools/Preprocessor.py", line 168, in handleCommandLine
    self.do_include(f)
  File "tools/Preprocessor.py", line 430, in do_include
    raise Preprocessor.Error(self, 'FILE_NOT_FOUND', str(args))
__main__.Error: ('', 0, 'FILE_NOT_FOUND', '/Users/marinasamuel/android-sync-project/android-sync/Sync')

When executing line 21: `$PREPROCESSOR $DISPLAYNAME_DEF $CLIENT_NAME.in > $CLIENT_NAME`
Preprocessor.py reads "Firefox" as the first argument and sees "Sync" as a second argument which is expected to be in the form of a file.

Wasn't sure what can be done within the script to avoid this.
Good stuff! You even left a trivial typo in so we can try out the "fixing nits" workflow :D

Please amend the commit, re-push and re-request, and I'll merge.

\o/
remove hardware/ti/omap3/liboverlay
Language.T("Result: ") 
will search exactly the quoted phrase in the language files. It's better to make it:
Language.T("Result") + ": "
In that way we can eventually reuse the translation of "Result".
Lol ok... are you not supporting your kernel anymore? Now that you're
monitoring my git ;)
On Feb 28, 2012 9:23 AM, "Steve Garon" <
reply@reply.github.com>
wrote:

> I agree with abev66. It actually does not improve performance at all.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mathkid95/linux_samsung_ics_real/commit/75373c84b9ade2cc5747f316fe7a886c7f09bb1c#commitcomment-1021273
>
You mean ```s.split```, right? Looks good other than that.
Queria fazer o filtro com "map" mas quando tentei ele retornava nil em alguns lugares do array... Se alguem souber como fazer com map do jeito correto, agradeço pela informação.
on p999 /data is on mmcblk07... seems like copy and paste from p990. lol.
This fix actually causes another problem if you are using an "options.check", for example: :less_than_or_equal and you receive a blank input value.

In that case, it generates a JS syntax error as it attempts to create the Function("return " + "" + "<=" + "10").
this example looks like it will fail again on retry?
By the way, I was using [FOSRestBundle's composer.json](https://github.com/FriendsOfSymfony/FOSRestBundle/blob/master/composer.json) as an example, so that likely needs to be fixed as well.
We should try to use multiples of the SCSS grid base variables if possible
so this thing will make camera to work now?
any more changes?
Uh

    self.add_field(**field)

Dovrebbe bastare
I always put a space after comma, in tuples and function arguments.
Here typo, too many `i` symbols
"input[type=text]" is too strict, ":input" will also cover textareas and selects.
Is there a verdict on this? Keep it or not?
I guess you forgot to undo it.
Use delete_data(). I don't know if it makes any difference, but it could in the future...
This won't work with MPI at all quite yet. `mat.createAIJWithArrays` will actually use `MatCreateMPIAIJWithSplitArrays` in the MPI case but requires 6 arrays to be passed in for the CSR (diagonal and off-diagonal parts). You may want to look at the [relevant petsc4py commit](https://bitbucket.org/fr710/petsc4py/changeset/7fcac2d03d96e1e2850e40a3c054ab32) and give your thoughts on that.
__DIR__ . '/' maybe then? :-)

Works for me from very start of PHP 5.3 (TM)

On 10/24/2012 07:02 PM, Robert Elwell wrote:
> In extensions/wikia/SpecialVersion/WikiaSpecialVersion.setup.php:
>
>> @@ -0,0 +1,33 @@
>> +<?php
>> +/**
>> + * Wikia Special:Version Extension
>> + *
>> + * @author Robert Elwell <robert(at)wikia-inc.com>
>> + */
>> +
>> +
>> +$app = F::app();
>> +$dir = dirname(__FILE__) . '/';
>> +
>> +/**
>> + * classes
>> + */
>> +$app->registerClass('WikiaSpecialVersion',				$dir . 'WikiaSpecialVersion.class.php');
>
> Actually, *DIR* doesn't seem to work. Going to stick with $dir.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wikia/app/pull/9/files#r1930982>.
>
I noticed that this isn't explicitly covered anymore. @federico-lox -- have you tested against this?
Nice catch! :-)
Is there any tests missing here compared to the public API exposed in the implementation(s)?
And the reason it dies from the Kill is due to the escalate?
Is it documented that Escalate on the /user will kill everything?
Thanks for the tip. I first used 0 as port in the system settings but then didn't found a non-reflection way of findig the exact port opened. This will work nice.
Why send the messages from inside the actor?
Perhaps two different tests, one sending from the outside and one from the inside (from constructor like this).
Then we're on the same page :-)

Alright, could you do me the favor of squashing the commits into 1 so it's easy to cherry-pick it into 2.0.4?

Great work!

√
I'm just thinking that I want the difference to be minimal.
`cluster down victim`
It shouldn't be possible to get a reference to an Extension instance until it has been initialized. Can you minimize it?
Hmm.. I'm not sure it's actually going to be a problem as-is in IE. You could paren for clarity: `!(/^\s*</).test(elements)`.
In any case, a good unit test suite will pick up any problems with old IE so working on that should be your next task to figure out.

If you want a full IE test suite then you can download and use these virtual machine images direct from Microsoft exactly for this purpose: http://www.microsoft.com/en-us/download/details.aspx?id=11575
I don't think we actually use many of these, do we?
I was thinking about your previous comment that we shouldn't let this fall through to Ruby since we've already detected the error in Haml. I think you're right, let's change this to raise a Haml::Error there and remove these engine tests.
may be my fault somehow ... will check more later but now fails for me with:

```
======================================================================
FAIL: nipy.tests.test_scripts.test_nipy_diagnose
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python2.6/dist-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/usr/lib/pymodules/python2.6/numpy/testing/decorators.py", line 146, in skipper_func
    return f(*args, **kwargs)
  File "/home/yoh/proj/nipy/nipy-suite/nipy/nipy/tests/test_scripts.py", line 78, in test_nipy_diagnose
    assert_true(isfile(out_fname))
AssertionError: 
    """Fail the test unless the expression is true."""
>>  if not False: raise self.failureException, None
```

on vectors_components_functional.npz  file check... also that tmp directory is not there by the time of failure (shouldn't it be there?)

here is debug output:

```
$> nosetests-2.6 -s -v --pdb --pdb-failures nipy/tests/test_scripts.py 
nose.config: INFO: Ignoring files matching ['^\\.', '^_', '^setup\\.py$']
nose.plugins.cover: INFO: Coverage report will include only packages: ['nipy']
/home/yoh/proj/nipy/nipy-suite/nipy/nipy/__init__.py:23: UserWarning: Running the tests from the install directory may trigger some failures
  warnings.warn('Running the tests from the install directory may '
nipy.tests.test_scripts.test_nipy_diagnose ... I:  /home/yoh/.tmp/tmptC7ENQ
> /usr/lib/python2.6/unittest.py(325)failUnless()
-> if not expr: raise self.failureException, msg
(Pdb) up
> /home/yoh/proj/nipy/nipy-suite/nipy/nipy/tests/test_scripts.py(78)test_nipy_diagnose()
-> assert_true(isfile(out_fname))
(Pdb) print out_fname
vectors_components_functional.npz
(Pdb) 
[3]  + 24354 suspended  nosetests-2.6 -s -v --pdb --pdb-failures nipy/tests/test_scripts.py
2 23346 ->20 [3].....................................:Wed 19 Dec 2012 04:53:16 PM EST:.
(git)novo:~nipy-suite/nipy[master]git
$> ls -l /home/yoh/.tmp/tmptC7ENQ
ls: cannot access /home/yoh/.tmp/tmptC7ENQ: No such file or directory
```
gettext here?
As an alternative solution maybe we could borrow something from haproxy:

http://haproxy.1wt.eu/download/1.2/doc/architecture.txt point 4.3 :

```
For this reason, a new hot reconfiguration mechanism has been introduced in
version 1.1.34. Its usage is very simple and works even in chrooted
environments with lowered privileges. The principle is very simple : upon
reception of a SIGTTOU signal, the proxy will stop listening to all the ports.
This will release the ports so that a new instance can be started. Existing
connections will not be broken at all. If the new instance fails to start,
then sending a SIGTTIN signal back to the original processes will restore
the listening ports. This is possible without any special privileges because
the sockets will not have been closed, so the bind() is still valid. Otherwise,
if the new process starts successfully, then sending a SIGUSR1 signal to the
old one ensures that it will exit as soon as its last session ends.
```

The way I understand it is that we could unbind old sockets (instead of closing them) and then new sockets on same tcp port can be started. I leave the decision to you. Maybe that way could lead to less code.
(dislike api, as per previous comments)
append waybill_user_filter
Jest zawsze zdefiniowane, ba na siłę w php tworzysz w else podstawienie false.
The one problem that this can run into is if your zk has lost and re-established its session, you actually won't exist in the list of `ordered_lock_children`, so it will return all of the children.

I'm not sure if there are going to be any cases where that matters, but it could cause something that would previously fail early (because everyone below you was gone), to fail late (only when all locks are gone).
I think the nicer way of doing this would be to call the option "function" instead of generator, and then automatically detect whether it's a plain test (foo_test()) or a generator (foo_test_()) and run the tests in the appropriate way. You could even allow the user to skip the final "_" and match the given string against the exported functions in the module - there will probably not be both foo_test() and foo_test_(), so just pick the matching function. Hell, you could say that the rebar user just gives the prefix, so you could say "test=foo" and automatically find foo_test() or foo_test_() or even foot_fetishist_test().
uhh, now we get again a mess of `#defines`...
It seems like overkill when it could be simplified so much though. Or better yet, just use that CGRect syntax instead of the method. It seems cleaner. To me that is....
Absolutely no need to synthesize properties anymore.
This `else` case could go into the `switch` above for `TUILayoutConstraintAttributeBounds` and `TUILayoutConstraintAttributeBoundsCenter`, which could both return early after setting the bounds. Then, this `if` statement becomes unnecessary (it would always just be setting the frame).
There's trailing whitespace. :)
You can search for tw in vim with `[ ]*$`
git diff can be configured with colors: `git config color.ui auto`. After that it should be easier to spot tw.
Why does this method need to be public?
we do not have a style guide, but can we split this up onto different lines. IMO it's more readable.
J'ai ajouté ces tests dans mes derniers commits.
I used http://api.nette.org/2.0/source-Caching.Storages.FileJournal.php.html#1166 without measuring. I'll measure it and if it shows as a significant improvement I will rewrite it to own method or switch to array_marge otherwise. Thanks for pointing out.
It is likely that there are no special tests to check type coercion in engine. We need to add one.

Secondly, could you explain if shrinkwrap is unable to process tarball references?
Sure will make the change.
You are right. These change seems fine.
I'm always torn: guides should have just the most useful stuff, not have a copy of everything in the API docs. Maybe there's a better balance to be struck here.
Not entirely sure, don't think I had a good reason. I can put to stdout instead...i'll make that change soon.
Ok, I didn't notice the other tests. I can't take a look right now to see if I missed anything, I'll report back later. Thanks @blakesmith.
I don't think there's a good reason to have a class for this; it's just data, no behaviour. And the data is only two values.

I would make your `find` method just be a method on `PredicateBuilder` that returns a 2 value array.
The `module` class symbol
That looks like (unnecessary?) double checking. superclazz will be validated at the end of validateParentClass, right?
one way to whack the mole more reliably and get more encapsulation to boot:
leave this intact and provide failing implementations of all Symbol methods in the StubSymbol subclasses

I'm figuring out whether this is doable. There's also risk with missing an override of course.
I also need to treat `\f` as a newline for consistency with `stripMargin`.
How much time is MSIL kicking into your builds?  Just curious...
Well, it looks like classfile annotations are pickled as well.

`class W extends scala.annotation.ClassfileAnnotation; @W trait X`

Decompilation shows:

```
@ScalaSignature(bytes="\006\001M1q!\001\002\021\002G\005QAA\001Y\025\005\031\021a\002\037f[B$\030PP\002\001'\t\001a\001\005\002\b\0315\t\001B\003\002\n\025\005!A.\0318h\025\005Y\021\001\0026bm\006L!!\004\005\003\r=\023'.Z2uQ\t\001q\002\005\002\021#5\t!!\003\002\023\005\t\tq\013")
public abstract interface X
{
}
```

Note there are no java annotations on X, so all the info is to be stored in the pickle.
No, we try to avoid ``from *`` imports (which is also what pylab does). The standard import conventions are listed at https://github.com/numpy/numpy/blob/master/doc/HOWTO_DOCUMENT.rst.txt

You only need one ``plt.show()`` call to display both figures. Here's how I think it should look

    >>> from scipy import signal
    >>> import matplotlib.pyplot as plt

    >>> s1 = signal.lti([1], [1, 1])
    >>> w, mag, phase = s1.bode()

    >>> plt.figure()
    >>> plt.semilogx(w, mag)    # bode magnitude plot
    >>> plt.figure()
    >>> plt.semilogx(w, phase)  # bode phase plot
    >>> plt.show()

I don't think it's too verbose, I prefer completeness over conciseness. Many functions have longer examples than this.
`ftpResponse` could also be undefined.
a dual return is used rather than a flag and a single return
This should probably be in SparseMatrix
just `map(C.FiniteSet, args)`?
Imported above
They are removed.
Ok then. I've thought that it's critical to follow consistency (to use empty instead of negation).
Hi Prasad, not sure what you mean here since BOUNDARY_CONDITION_IMPERMEABLE_WALL=19 is the maximum condition number?
How does this even come around? If the promo isn't present then it would cause this message to incorrectly appear.
This docstring would benefit from a read of http://sphinx.pocoo.org/markup/desc.html#info-field-lists.  At the moment this will come out as mush when put through Sphnix.  Also, I'm going to assume you didn't realise your editor was soft-wrapping the lines here.
I think the way you did it is the right way.
`that is is registered with the autoloader`
Technically, you only have one service. I would just remove the whole paragraph as it does not give any valuable information.
I think you should use smaller priorities.
In Symfony 2.1, the priority of the ProfilerListener is set to 1024 and it needs to run on all requests to avoid an error on kernel.response. This means avoiding that a listener with a higher priority stops the propagation or thorws an exception.
So using a priority lower than 1024 would make the integration of the Profiler into Silex easier.
I don't know if we should really support such a case, but this will blow up if multiple sheets get presented (which happens unintentionally sometimes). It'd probably make more sense to set this block on the sheet.
libuv internal. I'm 99% sure this won't compile with -Werror.
Измени роуты в соответствии с теми изменениями, что я описал выше
Do this function as private.
yes probably, will do
Double checking against master, it looks like it only ever has the value of null; in fact, its use was discontinued in b74d21c4e by the looks of it.
monitor the rate of replication by watching..
we usually alphabetize the rules inside each selector (i know totally crazy!), but it helps for scanning items when it gets large.
En fait les deux sont responsive. La difference, d'apres ma comprehension, est que dans le cas fluid l'aspect de la page est maintenu peu importe la taille de la page, tandis que dans le non-fluid les elements dans chaque row vont se superposer.

Vu que la page d'accueil contient des petits divs pour les services et produits, j'aimerais mieux qu'ils se superposent lorsqu'on rapetisse la page, plutôt que les rapetisser encore plus aussi.

Pour les pages avec un sidebar a gauche par contre, on doit avoir un fluid layout pour maintenir le sidebar. Les divs sont plus gros aussi alors ils resteront toujours lisibles.
Lowest I know of is 1s on Macs. What system has a lower one?

Anyway, you definitely don't want them to be equal, that's incorrect behavior (or at least consistent with incorrect behavior).
Reading the docblock I get the impression that this function takes user input and returns something that can be safely stuffed into single quotes and given to the database.

The pass-through of user data is not safe in such a situation.
Ensure $post_time is not in the future here?
This is the base sql driver. The generic sql driver extends the base sql driver and depends on a data cache which is correct. Any other sql driver may not depend on a data cache and thus this constructor should be in the generic driver.
You should explicitly declare `$`.
@unknownbliss I think that's what he means. As long as there is no key, enabled/disabled is irrelevent. But having it enabled already makes it a one step process to simply enter a key and it starts working. otherwise, you have to enter a key and hit enable.

I'm not entirely sure which way I would prefer. I suppose having it pre-enabled is the most user-friendly? *shrug* I guess it's fine as it is for now.
同意
BTW, a side benefit of this: ruby allows anything to be used in a rescue clause that responds to `===` and uses that to see if the exception matches.  The normal thing is to use class constants, but it gives you the power to use an object that inspects the message or whatever.  By leveraging ruby's built-in logic, it allows things to be more flexible here and work the same way as ruby itself does.
This was an incomplete commit BTW, look at the current master

Am Mittwoch, 19. Dezember 2012 schrieb muescha :

> In lib/assets-compiler.js:
>
> > @@ -27,16 +52,27 @@ AssetsCompiler.prototype.require = function(module) {
> >  };
> >
> >  /**
> > + * Precompiles assets in /app/assets/coffeescripts and /app/assets/stylesheets
>
> If i look through the code there is never the assets/coffescripts folder.
> Only the assets/javascripts folder
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/1602/compound/pull/250/files#r2469967>.
>
>


-- 

Mit freundlichen Grüßen,
Sascha Gehlich
Founder & CEO

Mobil: +49 (0)151 4190 2765
Mail: sascha@gehlich.us
Web: http://www.filshmedia.net

*FILSH Media GmbH*
Kortumstraße 26
44787 Bochum
Germany

Firmensitz: Bochum
Amtsgericht Bochum, HRB 14141
Geschäftsführung: Sascha Gehlich
suggest this should be ```private```, since the ```getId()``` method is already public.
Is it always wise to do this operation on the GPU? I imagine this sort of job might often be slower.
As pointed out, putting this inside a conditional "if node in node_to_children" might make for a slight speedup (I think this is one of those weird performance glitches that is fixed in Python 3). At any rate, I'd say it's a code clarity win too.
I get it and realize my typo for "\\N" other than "/N". Already fixed.
Simply return None.
Is there a reason we'd ever want this to be configurable?
Interchanges *class-based* and *class based*
(side note: Does this warrant a distinct note in the Django specific terminology for documentation?)
I guess it should go to the `assets` group.
A nice thing about having this trapping here is that if at any point during the require/setup process the user hits ^C the trap will fire. With it moved down below hitting ^C during that phase will generate a backtrace.
Should be split_criterion.
Yeah I am thinking we change this to `no_timeout` since it's misleading to call just `timeout` when passing nothing and taking arguments doesn't really have any affect.
I've checked a bunch of other servers and clients now, and every one of them just supports or uses "Alphabetical", so I'll let this follow the spec strictly for now.
This can just be

```rb
ENV['PYGTK_CODEGEN'] = which 'pygobject-codegen-2.0'
```
Don't move `head` around, the preferred layout is homepage, url, (version), sha1, newline, head, newline, devel block.

Remove the 1.7.1 version here, Homebrew can parse that one fine, and it triggers an `audit` warning otherwise.
`mode` reads better for sure.
expected and actual should be swapped
All dispatch methods that require redis start with `yield self._redis_d`, so it's safe enough. Just very ugly.
Maybe messages without a session_id should be complained about and logged (in case someone accidentally routes random outbound messages here)?
toString() is supposed to help the user debug, not the implementor. So ArrayList's toString is [a, b ] instead of "size=2, elements=[a, b, null, null, null]". The toString should be designed carefully, and make sense in a debugger.
I merged master and went thru the conflicts and saw your changes. Preserved whatever was necessary and made edits to suit the @options pattern being used in the plugins now.
Oops, it is a field.  My bad.
not indented
I added an exclusion for the altlassian plugin... hmm did I really delete /worlds ?
I assume all the changes here are just from updating the plugin? Were there any edits you needed to make?
What is the significance of this change?
If we’re going to translate *build* as *Version* everywhere, then you are right.
Doesn't seem like `link` is used anywhere, so we could just call the function.
"reset" sounds like it's being changed back to a previous state. How about updateDocumentTitle() or changeDocumentTitle() ?
@njx for reference the pull I thought I merged was #1487. It's now merged. Sorry for the mixup.
Functionally it’s a little strange, but it is showing two distinct hings. I wanted to show the use of localized string in JavaScript and HTML.
From: Jason San Jose [mailto:notifications@github.com]
Sent: Thursday, August 23, 2012 12:39 PM
To: adobe/brackets
Cc: Ty Voliter
Subject: Re: [brackets] Plugin localization example and documentation (#1424)


In src/extensions/disabled/LocalizationExample/main.js:

> +        Dialogs             = brackets.getModule("widgets/Dialogs");

> +

> +    // Load an html fragment using the require text plugin. Mustache will later

> +    // be used to localize some of the text

> +    var browserWrapperHtml  = require("text!htmlContent/sampleHTMLFragment.html");

> +

> +    // Load the string module for this plugin. Not this references to the strings.js

> +    // file next to the main.js fiel for this plugin. To access core brackets strings

> +    // you would call brackets.getModule("strings") instead

> +    var Strings             = require("strings");

> +

> +

> +    // This sample command first shows an alert passing in a localized

> +    // string in JavaScript then it shows a localized HTML dialog.

> +    function TestCommand() {

> +        alert(Strings.ALERT_MESSAGE);

Does this example need both an alert and a dialog?

—
Reply to this email directly or view it on GitHub<https://github.com/adobe/brackets/pull/1424/files#r1448392>.