labofsahil/github-event-dataset-2012 huggingface dataset:
updated template loading

I adjusted the template loading feature located on line 1253 to use the WordPress locate_template function instead of the TEMPLATEPATH constant as locate_template supports child themes, where the TEMPLATE path constant does not. This allows for a BuddyPress site to use the normal theme structure of a child theme instead of needing to place a template file in the BuddyPress plugin directory. This would also affect anyone building a child theme of the standard WordPress themes and certain frameworks etc.

Example:
output of TEMPLATEPATH in a BuddyPress site would be similar to: <pre>/var/www/wp-content/plugins/buddypress/bp-themes/bp-default</pre>

Where in most cases a theme developer would be building a theme sourced from <pre>/var/www/wp-content/themes/my-cool-theme</pre>

Thanks!
Updated with proper true/false value in second call to "locate_template"
Use applicationFrame instead of using testing for a status bar

It's recommended in the Apple docs.
Bump version for cancel staged payments change
Features/sponsor packages
Fix small typos in community page
Немножко кучка)
Updating the production URL
*_from_user_id を編集することで他人のアカウントからほぼ全てのアクションを実行できる問題

HTML側で form element の *_from_user_id を書き換えることで簡単に他人にアクションを起こさせることが可能なコードになっていました。
仕組みが詳しくわからなかったので取り急ぎ before_filter に current_user かどうかをチェックして、ダメならルートにリダイレクトする処理を入れました。
Default connector fix

This is a minor bugfix for setting default connectors in Wordpress 3.3. Enjoy! :)
please merge my files to master,Thanks!
Collaborative work made on BeerCode meeting using TDD.
Remove decorators for hash elements
Merge Datalog fix und readme.txt in Quad_Mini Datalog
Development

Started moving the class, this is more of a test pull to see how git deals with out of sync merges.
Test fixes 

portal/permissions/webcontent
Updated AdobeTV to use available MP4s

 - Also resolved issue in AbsoluteRadio to ensure no audio URLs are selected
Fix for blocktrans syntax change breaking debug_toolbar in Django 1.2

Commit 2870dcdf320a7b0864cd59e775111a2618d3f1ec changed blocktrans in templates/debug_toolbar/panels/templates.htm to use the compact syntax introduced in Django 1.3.  This commit changes it back so blocktrans works in Django 1.2 as well as 1.3 and 1.4
Package structure

Fleshing out the package structure 

 * Core - components we can use independent of WebForms/MVC project structure
 * MVC3.Logic - contains controllers and model classes
 * MVC3.Web - contains content and views
 * MVC3.All - uses previous packages to scaffold an empty MVC3 project
 * MVC4.Logic - contains controllers and model classes
 * MVC4.Web - contains content and views (obsolete?)
 * MVC4.All - uses previous packages to scaffold an empty MVC4 project

TODO:

- do we want to style the site in a certain way?
- do we want to squeeze in a new package for jQuery/KendoUI?
user_stream returns all actions if user is not following any actors

ActionManager.user has no code path to handle the case where follow_gfks is empty, which causes it to return all public actions when the user has not followed any actors.

I changed ActionManager.user to return a EmptyGFKQuerySet if follow_gfks is empty. I also had to adjust the stream decorator, because Django returns a list, not a QuerySet, when you slice an EmptyQuerySet. Includes a simple regression test.
Улучшение скрытия выделенного

Если выделенное находится в пределах темы или сообщения и не содержит переноса строки, используется `#words`, если содержит `#exp`. Если находится за пределами сообщения/темы, используется `#exph`. Стоит заметить, что если выделение затрагивает сразу несколько постов, то релевантный спелл будет добавлен, однако ничего не скроет. Думаю, что проверка того, что выделение находится в пределах одного поста является излишней.
Update :public to :public_folder

I receved this message after pushing to heroku which prompted this
change.

2012-01-09T15:58:24+00:00 app[web.1]: :public is no longer used to avoid
overloading Module#public, use :public_folder instead
Mbappou yvan

Bonjour Monsieur,

Je me permets de vous écrire car j'ai une erreur lorsque je lance ma spec s_auth_spec.rb et je ne sais pas comment la résoudre.

Pourriez vous m'éclairer svp?

Cordialement,
Yvan MBAPPOU
Support for simulator builds for PIM and sensor extensions

Fix for issue #359
Support for simulator builds for PIM and sensor extensions
Use addOrderBy instead of orderBy in ProxyQuery

This update allows to add a priority sort order, like this :

```
public function createQuery($context = 'list')
{
    $query = parent::createQuery($context);

     if ('list' === $context) {
        $query->orderBy('c.myField');
     }

    return $query;
}
```

What do you think ?
update to regex filter to allow filtering on empty strings
PLFM-1578 Add a API to get storage items per entity
Fixing your shitty code style, bro

hehehehe <3
Fixed my message lists
Rowend/1 readme file and folder structure

readme files and folder structure
Fix issue with Acceptance.Windows test

Using Suite Teardown within Acceptance.Open and Close test suite, which comes right before the Acceptance.Windows test, and calling "Close All Browsers" we can resolve the failing windows test. In investigating the cause of failure, the tests passed if ran isolated (using the command "$ python run_tests.py python *ff --suite='Acceptance.Windows'"). Further investigation found the commenting out all the "Close Browser" keywords within the Open and Close test would also fix the issue (but expectantly cause some of those test to fail). It is unclear why calling "Close All Browsers" works while the less inclusive "Close Browser" fails or at least cause an issue latter on.

Although this fixes the problem I am troubled by the test isolation issue and I am still unsure why exactly this fixes the problem.
Update README to remove the now unnecessary (and harmful) step from iOS setup

Readme change suggested in davejohnson#22
Relecture du chapitre 7.
I like changes like this.
Do you think it's safer to err on the side of avoiding http calls?
Deine Mutter ist besser! :P
I'll move it if other changes are to be made to this pull request.
And the helpers are where? :)
Glad this issue is now resolved. Back to work!

![](http://i.imgur.com/PcTuf.jpg)
It would be nice if we could format the whole ember and I could use the hook, since I always mess up formatting somewhere. :P
I think you're right, yea. Though, since the image has already been decoded, it doesn't do us any harm to always return the bigger one (other than the extra memory usage when it's not necessary, but the previous behavior was to always hold the 100% image anyway).

Rather than continuing and messing with the refcounts, any issue with simply replacing smaller textures with bigger ones when they're loaded, then we don't need this test at all?
Is this still open?
Ok, I filed an issue in the tracker. I'll get right on it. Thanks for the info.
I have built in a custom Twig function to construct SLIR resize URLs.
Usage is
```html
<!-- resize_path(image_url, max_width, max_height, ratio) -->
<img src="{{ resize_path(image.url, 150, 150, '1:1') }}" alt="{{ image.title }}">
```
Should hoe-rubygems be listed twice or is this a typo?
Just reverted; shouldn't have happened anyway. Thanks for the hint.
https://github.com/dredhorse/DwarfForge/blob/master/etc/plugin.yml#L6
For performance reasons the compatibility layer is only available in the debian branch which is based on the stable version.

When I made the change I checked by code inspection, that ADODB should deal gracefully with the syntax in case a DB doesn't support it. IIRC the extra syntax should be ignored in that case. I didn't check it against a real pg installation, though. IIRC this exact syntax was already supported in the code in other places of xmlschema and I simply copied it over as the code didn't allow for re-use. I think pg allows indexes on CLOBS without the extension. That's why I assumed that it works on all PKP platforms. A test against a pg installation would of course be helpful. I assume that you'll do one as part of your release process.

I still agree, though, that a solution that does not require us to modify ADODB would be much preferrable.
These look good, we will wait for some feedback, archtaku was also working on this. I am also going to mark them as "experimental", we need to develop a standard way of saying when a module/state's interface is still highly subject to change
Problem is that state is recorded for the recording identified by the streamURL:
15:29:51 T:139733288060672   DEBUG: DoWork - Saving file state for video item http://xbmc:xbmc@10.22.22.22:9981/dvrfile/68

Works great before this commit:
15:25:22 T:140220807796704   DEBUG: MIKKLE: CGUIWindowPVRRecordings - GetResumeString - itemPath: http://xbmc:xbmc@10.22.22.22:9981/dvrfile/68

But after this commit, state is still saved to the recording identified by streamURL, but the resume functionality queries for FileNameAndPath:
15:25:22 T:140220807796704   DEBUG: MIKKLE: CGUIWindowPVRRecordings - GetResumeString - itemPath: http://xbmc:xbmc@10.22.22.22:9981/dvrfile/68
is this really what you want to happen?  if no deployment mode is set, seems like this gives a semi-cryptic error message?
Wouter, loop je exam dingen nog even na... ik kan er geen een maken omdat veel links niet kloppen enzo...
this is breaking MesoX's patch, https://github.com/arcemu/arcemu/pull/20
at zije final, styd se. :P
Yup, that would be great; I was waiting on the request until I tested it, but those lines do work by themselves, so I'm sure its safe.
Ok testing this now, was clearly the traceback that caused that issue but now to see if it actually works :P
It doesn't use inspect, but it still does not 1) allow a function with keyword arguments, and 2) disambiguate between a "Hessian product" function's vector input and additional inputs to a regular hess function.
There are two Chinese versions, the Traditional Chinese and Simplified Chinese (for historical reasons, Traditional Chinese use in the Republic of China,Hong Kong, Macao. Simplified Chinese use in the People's Republic of China and Singapore)
This change and the ones after it on Feb 29 are actually from March 9th. VMware time sync issues on my Ubuntu machine...
I think it's pretty subjective.  I guess the question is: "Do we want to be, today, as HTML5 supportive as possible?"  If the answer to this is yes, than I say change it.   If we aren't really concerned about it, especially since there is 0 difference to the end user right now, than I say leave it.
Hi, I'm Nov Matake, fb_graph developer.
I just found several developers are commenting out this part.

Can you explain actual issue here?
I'd like to fix this part, but don't get what's wrong..
Cool, thank you!

A couple things:

- please add definitions so that "line" is the default and doesn't have to be specified
- this really isn't useful unless the heights of the bars are related to the data in some obvious way...right now I'm not sure what they mean

Do you have any ideas for how we can generalize our design to support things like multiple plots on the same axes, or more options (color, etc.)?
I've got some other work with Vector2D and unit tests for both classes I would like to pull at the same time.  It should all be ready shortly.
там сделана нормализация координат и приведение m_caster->UpdateGroundPositionZ(x,y,z); - z приводится "к земле". это не совсем правильно, но пришедшая от клиента точка прицеливания, сидящая в m_destZ может быть ниже реальной поверхности земли. а этого делать нельзя, поскольку можем словить краш и сервера и клиентов. надо как-то выкручиваться - считать Z с учетом траектории.
здесь еще достаточно много надо сделать, но не это. Reset() вызовется сам для уже отинициализированного элемента если он станет верхним по приоритету (см. метод в ActionInfo).
No if a bug is fixed in 2.0/develop we don't commit directly to 2.1/develop because 2.1/develop follows 2.0/develop. So every few days we just merge 2.0/develop in and then 2.1/develop is all up to date.
Should be `c - 'A' + 10;`
I think there should be at least an config option where you can deactivate the login info if you don't want to expose the OwnerEmail to the Rest of the world ;-)
Some of the code was ported from Java, if you check the latest commit this class was converted to Scala there.
You can to try.. That's no work..
So, i just need to save the current session in temp array and push a new key, and save it.
This commit essentialy decreased performance of the websockets by, at least, 20%:

>>> timeit.timeit('bytearray("1234567890")')
0.20886757960613522

>>> timeit.timeit('array.array("B","1234567890")', 'import array')
0.2745079673895674

I'm working on some optimizations and will provide alternative way to support python 2.5 without degrading performance on 2.6+
removes the capturing group.
Yep, testing on Chrome/Windows confirms this: Backspace on http://snake.busyflow.com/ goes back in history, Backspace on http://terminal.jcubic.pl/ erases characters.
могу ошибаться, но помойму на pay_type стоит валидация на inclusion
Здесь наверное лучше использовать ту константу и взять первый из списка pay_type

ну и email уже лучше писать как email, то есть с @ (на всякий случай)
http://crbug.com/105960
typo, will remove
Any particular reason you have this function defined in a header file?
Why not use `{{ folder_create_form.name }}` here?
Oops, forgot the link:
http://code.google.com/p/cfwheels/issues/detail?id=350
Fixed in 5c623a5d39c
This is equally confusing ...
`all_files` coerces to a list and doesn't return a QS, but I need to filter it.
I don't see where you test the actual form. you should be testing that you can actually submit a device type from the view. start here: https://github.com/mozilla/zamboni/blob/master/apps/devhub/tests/test_views_edit.py#L417
Like. :)
All statements are wrong: you can only publish a version with draft status.
Le 27 févr. 2012 14:42, "Manuel Pichler" <
reply@reply.github.com>
a écrit :

> The doc comment for the BadStateException is misleading. Does it mean that
> I can only create a draft from a published version or does it mean that I
> can create a draft from every version that is not in draft state itself,
> which would mean STATUS_PUBLISHED and STATUS_ARCHIVED?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ezsystems/ezp-next/commit/ca4186b69659fd79b2c0456d873f8903ebed211a#commitcomment-1014906
>
maybe:
return new HashSet<>(nodes)
instead, since there are methods to add / remove nodes?
Thanks.  I took the widths out of the TD's because they inherit the width from the first TD.
I was trying to set the quality percentage for a carrierwave mounted uploader in rails. It kept throwing "can't convert Fixnum into String". This code fixed the problem.
@AlexeyProkhin: This broke D1/Tango, see #69.
it's 2012 :)
This looks about right.  I'd recommend a minimal unit test, like so:

- Annotate a class (not an interface) with an `@Component` annotation.
- Create another class with an `@Component` annotation and an `@Autowired` on the first one.
- Put some metrics-related annotations on the class.
- Create a Spring XML configuration that uses a `<context:component-scan ...>` to scan the package the two components are in and then whatever metrics configuration elements you like with the `proxy-target-class` set to true.
- If the context loads, the test passes.
Done thanks
I'll add the scope, but it shouldn't matter - I only added the testng dependency so that IDEA would pick it up. The compilation is all done in Ant.
Checked.
There is no chance the recent_log will live longer than the penge object, and if it does it's a bug, since installing a monitor will change the behaviour of the log in a way that is probably not wanted by other object holding a reference to it.

I'll anyway uninstall the monitor on disposal.
priv->recent_log is already unreferenced in _dispose, though.
The symbolic-ref call makes it so branch names are preferred over tags. When I put it back in, I get 'master' instead of the tag name associated with that commit.
Hi,

I wanted to be able to download binary data, using code like this:
```javascript
var http           = require('restler'),
	fs           = require('fs'),
	filename = 'google.gif',
	url          = 'http://www.google.com/images/logo_sm.gif';
http.get(url).on('response', function(response){
	var writeStream = fs.createWriteStream(filename);
	response.pipe(writeStream);	
})
```
but it's not possible because of the binary encoding at line https://github.com/adriengibrat/restler/blob/master/lib/restler.js#L126
Read me Test file
Lord knows I love me some consistent quoting, but was there a rationale for switching to single quotes everywhere aside from a desire for tidiness?
hmm this seems to have issues with PHP 5.4
http://travis-ci.org/#!/liip/LiipThemeBundle/builds/501228
2 exclam. marks added
Hi,
The only issues is that you are using the new security model in older scripts, but this is fine. After your next revision, I will completely convert three of the scripts above to the new security model (I then pass it back to you for some testing before committing it to sourceforge since the changes to convert to the new security model will be rather extensive).
thanks,
-brady
If possible, please use one of these in the title. Then the issue is automatically closed after the pull request gets merged.

    fixes #xxx
    fixed #xxx
    fix #xxx
    closes #xxx
    close #xxx
    closed #xxx
All tests are still passing, with 11 pending.
So it looks like `$this->cofig()->get('routes')` isn't implemented yet, @hafriedlander ? ;-)
een specifieke reden waarom der zoveel veranderingen in uw config file zijn?
# This README needs more Curly Braces#
I'm actually just practicing with git. This change made sense to me, but perhaps I'm not familiar enough with the project. I'll be deleting my fork after I finish playing around.
Oh, cool! It's funny: I wrote this 9 years ago and tried to keep it generic. It clearly took a decade, but it seems the DNA visualisation is finally going to get used ;-)
你可以在issue中提问。呵呵

博客的html就写在blog的collection中啊 { author: 'author', html: 'very long html', date: date}

一条纪录的上限是 16M，你可以直接存在一条document中存入html
Hahaha I was just answering your email :)  This is just some old code I was working on during the holidays it is currently broken, plus I managed to screw up the development branch and now have to revert two commits that we from the zipfile support branch.
I prefer to keep separate default theming with 5% stepping (which is default on CM7) than the extra feature of having 1% stepping. So it is ok to have it on 2 separate commits
dunno, i read (urls/index-refresh) better than (rest/index-refresh)
Cheers, it is just that replay.winner_known returned false, so I thought it may not of been implemented yet.
When a second letter is uppercase, then the first letter is not capitalized. It's Java convention.
This change fixes the issue, but it is strange that the authentication was without a user in the first place.
An investigation might be necessary.
Sorry, I'm pretty new to github, can i branch from this version? I can't find how.
No, it doesn't upload anything, neither does it download image.
This TODO can be dropped now, thanks.
Looks good, can you please move all that code to a separate file to keep the Default.ps1 more clean?
im revert commit

eye perfect work and not fall

http://www.imageup.ru/delete/863102/0c473fd54a7affc8e486f07368f69079
My apologies.  I didn't understand the original index function.

In this implementation, it supposed to return a persons finished, active, and shelved goals, for population on the left column.

Perhaps it would be better named my_goals, allowing index to be freed up for another part of the site.

On thing I try and do to make maintenance easier is make sure that all code is used.  Any code not used is commented, as an indication that it doesn't have to be maintained or may not be maintained.  I believe this was my thinking in commenting those methods.
The providers are in SpiffyDoctrineORM. This just provides common functionality between all modules and adapters, paginators, etc for those who want to implement their own ORM\ODM solution.
oh I see, the unicodehelper has those tests
on 2nd thoughts, ASP_NET is better than ASPDOTNET
Why do you create the RequestContext upfront?

When a JSON response is created, the requestcontext object is created but not used at all, which is exactly opposit to the commit text...
Ryan I am a little embarrassed to say I didn't recognize your CoffeeScript as code and thought you were saying that those dependencies were required for each module to get backbone thus the arrow. 

To spare anyone from the same ignorance. Here is the JS shim. :-)

``` javascript

// backbone.js shim

define(['vendor/underscore', 'vendor/jquery', 'order!vendor/backbone'], function(){
    return Backbone;
});

```

Sorry dude.
this is where i fixed that bug
Please try `mencoder_subcp = utf-8` in PMS.conf.
shit forgot to fix that in the patch, this fixes commit c444587ff93b3601c9d3bf3fde797e9ad814f300
When a client makes an authorization request, I want them to be able to pass various information through to the authorize handler. For example, if the client app knows the user's email address, they might want to pass that through so that my application can pre-fill that on the login form. The initial authorization redirect strips any parameters that were passed through during the authorization generation, effectively preventing me from allowing my app to offer auth flow improvements to the client application.
Agreed the order of instructions was messed up.  thanks.
This have nothing to do with auto return type. It is very simple how it works. It just uses type of expression passed to return(s), or void. Nothing mysterious here.

    delegate (double a) { return a+1; }

is actually 

    delegate auto(double a) { return a+1; }

My question was that 
    delegate (a) { return a+1; }
    delegate auto(a) { return a+1; }
looks to be polymorphic function/delegate, just like

    delegate (T)(T a) { return a+1; }

But it is incorrect also (there is no such things like delegate templates.

So I do not really understand how a=>a+1; or delegate (a) { return a+1; } is correct in example with bar39!(a => a + 1)(3);

Similary what with other examples without argument types or multiple arguments?

    (a) => (a+1); // is this correct?
    (double a, double b) => (a>b); // should be correct, right?
    (a,b) => (a>b);  // is this correct ?



As of =>, you are correct, it is distinct from >=. But still -> will make me fell me and my brain much better (it is more obvious, it is different from =>). => have oviously some advantages, like being similar to mathemtical inference symbol, but -> on the other hand is more similar to function mapping symbol. Which is exactly what we want here.
@lithium3141 Err... how does that work?
What about the error for integers that can only be decimal but start with a `0`? `09` should cause the compiler to say that decimal literals mustn't begin with `0`, right?
agree, fixed.
Oh also, don't forget to add information about your file to the content.php and headerSwitch.php. In headerSwitch, you'll probably just want to make a case that doesn't include anything just so you don't have the default home search bar and login info.
This default p.stroke() and p.fill() gave me a headache for over two weeks!
Because I'm calling size() after setup() (e.g. if the device rotates and I have to adapt the size of the sketch), the sketch lost the style settings set in setup(). While the problem and the solution sound very simple and logic, It took me some time to find that out...
You're right. The io_service constructor is deprecated. Thnx!
You don't need this, you don't attempt to print the msg body here.
boot.img 52
No sweat, it was a fun one to track down :)
I was just wondering how to implement a CSS decorator here. :purple_heart:
'raise' will print a scary backtrace that users tend to complain (often because they don't understand it).  Because of that, we prefer to avoid using 'raise'.  Instead, a simple 'print_error()' and 'return' should do the trick.
"Unlike backup tool, which uses the cm prefix to an existing lunch combo
variable, this flag can apply to devices not currently supported by CM.
The supported devices are manufactured by HTC, so instead of verifying
a cm prefix, the htc_ component is used to trigger the append."

^ it uses the htc_ part of your TARGET_PRODUCT choice to call it.
For CM7? Users will definitely have to wipe data...
Right now, the API doesn't have any way of calling functions, instance methods, or closures.

Perhaps it would be better to pass an entire callable value as the value of the array?  In the case of static method calls on the same class it would introduce a little redundancy, but it would let you to more with the TemplateGlobalProvider interface.

    return array(
        'HasPerm' => array('Permission', 'check'),
        'TimesTwo' => function($ra) { return $ra * 2; }
    );
Have no idea. I don't have any noises.
El viernes en la tarde Felipe decidió que por ahora, cómo no se ha definido un index para candidator, la raíz redirigiera a /elections pasando antes siempre por un login.
You know you're now setting the stack up twice.  Once here, and once in __jumpMain()
This one will reset the stack pointer after main() has pushed stuff on the old stack, effectively discarding it.
(consider using __attribute__((OS_main)) for main())
Fixed the alt attribute.

I didn't put the next event cause I wasn't sure of the lineup and stuff.  I've altered the text to say we've got speakers lined up, maybe we can do a proper announcement in a few days?
Você está certo =)
Valeu!
What was the reason for reverting this? I never saw anything mentioned in Gerrit about it causing issues.
I have been looking at a lot of Samsung kernels like Ilarian and Kwibo
One thing I noticed that some of them reference the /drivers/param but do not have it
I am thinking maybe this is something Samsung is not releasing
The folder does exist in my Prevail kernel
I am not sure if this is going to help anybody with anything, I just thought it was interesting

https://github.com/hroark13/prevail_kernel_non_booting/tree/master/drivers/param

Once again, thank you for all your help
no worries - done
This empty line is not necessary
Does this work if it's placed above the variable initialization?
Sounds good. I'll post a message to @cas-dev to jump start the discussion on this issue.
So we can do this with this patch:

````patch
diff --git a/pyop2/backends.py b/pyop2/backends.py
index 7ce6e09..a8adf1b 100644
--- a/pyop2/backends.py
+++ b/pyop2/backends.py
@@ -113,10 +113,13 @@ class _BackendSelector(type):
         # Invoke the constructor with the arguments given
         return t(*args, **kwargs)
 
-class _BackendSelectorWithH5(_BackendSelector):
-    """Metaclass to create a class that will have a fromhdf5 classmethod"""
     def fromhdf5(cls, *args, **kwargs):
-        return cls._backend.__dict__[cls.__name__].fromhdf5(*args, **kwargs)
+        try:
+            return cls._backend.__dict__[cls.__name__].fromhdf5(*args, **kwargs)
+        except AttributeError as e:
+            from warnings import warn
+            warn("op2 object %s does not implement fromhdf5 method" % cls.__name__)
+            raise e
 
 def get_backend():
     """Get the OP2 backend"""
diff --git a/pyop2/op2.py b/pyop2/op2.py
index 62e564c..771733c 100644
--- a/pyop2/op2.py
+++ b/pyop2/op2.py
@@ -83,22 +83,22 @@ class Kernel(base.Kernel):
     __metaclass__ = backends._BackendSelector
 
 class Set(base.Set):
-    __metaclass__ = backends._BackendSelectorWithH5
+    __metaclass__ = backends._BackendSelector
 
 class Dat(base.Dat):
-    __metaclass__ = backends._BackendSelectorWithH5
+    __metaclass__ = backends._BackendSelector
 
 class Mat(base.Mat):
     __metaclass__ = backends._BackendSelector
 
 class Const(base.Const):
-    __metaclass__ = backends._BackendSelectorWithH5
+    __metaclass__ = backends._BackendSelector
 
 class Global(base.Global):
     __metaclass__ = backends._BackendSelector
 
 class Map(base.Map):
-    __metaclass__ = backends._BackendSelectorWithH5
+    __metaclass__ = backends._BackendSelector
 
 class Sparsity(base.Sparsity):
     __metaclass__ = backends._BackendSelector
````

I.e. hook the `fromhdf5` classmethod on the `_BackendSelector` metaclass and raise an error if calling it from the wrong object.
I see a change to the calling convention, but not corresponding update to the user-facing documentation.
This is new, right? (plz 2 document if so)

In any case:
![trollface](http://t1.gstatic.com/images?q=tbn:ANd9GcScIRwt3BLaDWnP6M3vF-kto2AZGZLEPouZDRf0AiEzZRtzujFBsqf_ls39)
might add a comment that this is not actually returned
Yes, exactly, once that's done I'm merging this one into master!
I don't know what is the best way to represent points in time in Akka, so  
I just used Deadline for now. Is there a nicer way?
OK. good luck :-)
No, it is strictly local.
I think this is a non-issue, I'm more scared about `nodeFor`.
Will the version numbers above be replaced automagically? (bad memory)
Why is this unaligned now?
Shouldn't this be in some other reference.conf? It's not part of the official akka-remote user API right?
compression level could be a command line option with this as a default :)
hmm, personally I don't like functions that change things like this.   I'd prefer the 'on_same_domain' was set explicitly eg

`data_dict['resource']['on_same_domain'] = some_function_that_returns_a_bool()`

that may be me ckan has lots of this stuff makes it hard to debug especially as python dicts are mutable so the return isn't strictkly needed
This is wrong because add_listener returns the SSLServer object, but the TCPServer object should be.
`{`少了空格？
(Not that we have any tests for this yet.. just for consistency)
it is meant to be

On Wed, Dec 19, 2012 at 10:31 AM, Filipe Nepomuceno <
notifications@github.com> wrote:

> In
> library/src/main/java/net/sourceforge/cilib/pso/multiswarm/CooperativeMultiswarmIterationStrategy.java:
>
> > @@ -90,13 +90,18 @@ public void performIteration(MultiPopulationBasedAlgorithm algorithm) {
> >                          assignDataPatternsToParticle((CentroidHolder) particleWithContext.getCandidateSolution(), table);
> >                          particleWithContext.calculateFitness();
> >
> > -                        if(particleWithContext.getFitness().getValue() < particleWithContext.getBestFitness().getValue()) {
> > -                            particleWithContext.getProperties().put(EntityType.Particle.BEST_POSITION, particleWithContext.getPosition());
> > -                            particleWithContext.getProperties().put(EntityType.Particle.BEST_FITNESS, particleWithContext.getFitness());
> > +                        if(particleWithContext.getFitness().compareTo(particleWithContext.getBestFitness()) > 0) {
> > +                            particleWithContext.getProperties().put(EntityType.Particle.BEST_POSITION, particleWithContext.getPosition()).getClone();
>
> I think the getClone() should be inside the bracket e.g.
> getFitness().getClone()) and getPosition().getClone())
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cilib/cilib/pull/86/files#r2460607>.
>
>



-- 
Regards,
Kristina Georgieva
MSc Computer Science
CILib Research Assistant
This is something for cvmfs_check, I don't think it should be tested here
Using the ${PARAMETER:=} would be better.  The parameter comes with a default that can be overwritten
is it? they seem equally safe to me. [citation needed]
is there a reason why we test for `!= 'rename'` and not just `== 'change'`? Judging from the docs only those two events are available and checking for the actual one we want might be more clear.
You mean it should be moved after optimizeInlineFilter() or after addLoopToAll() (or getPriority())?

And I'm guessing this also counts for public functions, however getPriority() is declared after protected functions instead of before them.
Hmm, didn't think of it that way. That actually seems better. I'll add that and also GCD parallelization solver support to make things snappier.
That it's better to abstract away the notification? :smile:
Style: missing space.
The F looks strange :) Also I'm missing your name there :)
ненужный закомментированный код лучше убрать
Они везде так делают, наверное, так можно
Or pass the method name and use `send`. Unsure of how it will affect performance.
Does this show up elsewhere?   Seems a useful method, but I could be wrong.
Both `copyInto` and `copyTo` are too close to the existing `copyToArray` and `copyToBuffer`, and connote copying the contents into a pre-allocated collection.

How about `build? It sort of gels with the parameter name, no?
I left this in because we will want this code as soon as we put in draw_grid and draw_limb into composite map.
I guess 2to3 changes this automatically, because I tested in Python 3 and it works.
On Sun, Jun 10, 2012 at 8:41 PM, Matthew Rocklin
<reply@reply.github.com>
wrote:
>> @@ -3079,3 +3098,19 @@ def ode_separable(eq, func, order, match):
>>      return Eq(C.Integral(r['m2']['coeff']*r['m2'][r['y']]/r['m1'][r['y']],
>>          (r['y'], None, f(x))), C.Integral(-r['m1']['coeff']*r['m1'][x]/
>>          r['m2'][x], x)+C1)
>> +
>> +
>> +@vectorize(0)
>> +def C_to_IntConst(expr):
>> +    """Transforms symbols starting with a `C` into IntConst instances.
>
> What happens if I have a symbol like "Conductance"?

This function could just get a pattern (perhaps regex) that can be
used to match the target symbols to give more control over this
couldn't it? Or perhaps the function would have all but the desired
symbols masked off with dummies and the target symbols changed to
symbols starting with "C"...I'm not sure what the best way to go on
this is.
Well, I think the goal is simulating the standard behaviour, whereby `0 + 0 + 0 -> 0` and `1 * 1 * 1 -> 1`, for example.  This plays nicely with the intuitive perception of how identities combine.  I think one should keep in mind that removing identities is essentially an expression-oriented operation, and empty expressions won't really work.
Ok, thanks.

On 29.03.2012 17:38, Christopher Smith wrote:
>>           denom_others = []
>> -        newargs = list(args[:])
>> +        newargs, numer_others = expr.func(*args).args_cnc()
>
> On Thu, Mar 29, 2012 at 9:38 PM, Tom Bachmann<
> reply@reply.github.com
>> wrote:
>
>>>           denom_others = []
>>> -        newargs = list(args[:])
>>> +        newargs, numer_others = expr.func(*args).args_cnc()
>>
>> Are you allowed in python to grow a list you are iterating over?
>>
>>
> yes...this is a beginner's gotcha, too, since if you append or modify the
> list while iterating you may get unexpected results:
>
> ```
>>>> l=range(10)
>>>> for li in l:
> ...  print li
> ...  jnk = l.pop(0)
> ...
> 0
> 2
> 4
> 6
> 8
> ```
>
> A for loop like `for x in list: print x` is essentially sugar for
>
> ```
> i = 0
> while i<  len(list):
>      x = list[i]
>      i += 1
>      print x
> ```
>
> So whatever you do to the list in the loop, the next iteration will simply
> look at the next item in the dynamic list:
>
> ```
>>>> for li in list:
> ...   print li
> ...   list.reverse()
> ...
> 0
> 8
> 2
> 6
> 4
> 4
> 6
> 2
> 8
> 0
> ```
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/1187/files#r621884
>
No, this is related to the sort of thing I mentioned on the mailing list.  The hints really have to sometimes be applied multiple times to ensure that everything happens all the way.  

Actually, I think this may need to include a check for expand_multinomial too.  I'll look into it.
Is this the "right" way to use factory girl, or is there an equivalent of `fixtures :all`?
sorry. forgot to commit my changes for `knitr-lang.md`
Yep, that makes tutorial a bit more complex. Let's stick with just `password`.
Yii uses TABs.
you can get entity manager with just a single line of code `$em = \ServiceUtil::get('doctrine')->getManager();`
I'm not thrilled about this method name.

Perhaps, go with getConfigForLines, setConfigForLine, setConfigForLines? (slightly more readable)
Gratuitous change here (looks like you removed the file-final newline).
Perhaps some explanation on this and surrounding lines?
I could be wrong, but from what I can see, it seems like this line should be moved back to one indentation level less, or the user will get a `NameError` when they specify the --config option on the command line.
Stick with 'User', not a good idea to translate class names.
I thought `import *` was bad?
I take that all back. This wasn't working before (which confused me) but it is now. I've removed the reflection bit. I think this was due to classpath oddities with certain converters that I had moved to another package.
по мойму эта строка упадет
Well just a minor thing you might wanna rename this to @is_discussion (and same on the method above)
We should be able to use Ordering.natural().reverse() instead of using Comparators.inverse().
printjson not required.
abhor parenthesis
Nit 1: PEP8 will yell at a line over 75 (or is it 72?) characters long. Break this line up, maybe like so:

    initial = {'based_on': based_on_rev.id, 'comment': '',
               'show_toc': based_on_rev.show_toc}
yep - good suggestion. Fixed.
`private` should be `protected` everywhere
Should commit message be changed then?
Why is sql interface under driver namespace?
I hate regex but you're right: "The dot (.) metacharacter matches any single character except newline (\)."
There shuld not be a new line added here.
I'll write up some guidelines from the conventions we've established throughout adding these initial events at some point later today and put them in the events request sticky topic (verb-noun, not naming after functions/classes/files, when to use _ etc.)
We do this as well in the extension acp (functional) test as there is no other way to do it. I can probably make a temp directory of phpBB for other types of testing, but this sort of stuff still must be done for functional tests. Is that alright or not?
When we run on controllers this will use the 404 not found exception.
I think Accepted is the last stage? Or maybe closed (not a status, something else)?
Hartsock would know exactly what ought to show up.
I'm arbitrarily picking the first successful handshaked client for time sync purposes. Only on disconnect or error will a new client be selected.
could you be more explicit here? it probably is sufficient to add a comment which describes that it is set to null when the runner is ready.
I think it might be helpful to create an enum

CONFIRMATION_PAGE = 0;
..
BMI_PAGE = 4;

See: http://stackoverflow.com/questions/287903/enums-in-javascript . I also like the idea of capturing pages within a values object (as shown in example two), which allows you to associate a number with each page and also be able to output a string.
Superfluous sentence?
That would be great. Perhaps we could start a folder full of install instructions for specific iris versions on specific os/architectures. Something along the lines of:

file: ```ubuntu-12.04.rst```:

```
Iris 0.9
^^^^^^^^^^^

sudo apt-get install ...

```

I do not propose that this information will be able to be kept up-to-date, instead prospective users can find the instructions for a OS similar to theirs and hopefully make some more progress than if they didn't have this information. Hopefully the more users, the more people will contribute to keeping this information fresh.
Ah, ok, it was to separate the *hh* code from the *aux* code, but no problem.
Not me. 

I think the ```source_proj``` should be a CRS not a Projection, so should use ```as_cartopy_crs()```. I certainly don't think it should be a PlateCarree projection. I'd also like to see all sign of ```cube.coord_system()``` gone as at this point you have x and y coords so can get their coord system directly. However, I've done a similar thing in project, where I've assumed a WGS84 Geodetic coordinate system if one is not defined on the lat long coords. I've put out a warning though and felt decidedly uneasy about the whole thing.

Following on from this, the logic lower down is flawed if a projection is carried out. The modulus will no longer be valid (adding 360 to something in metres is clearly wrong). In fact the units aren't updated by the projection so the circular attribute is no longer valid.
Fixed in gh-1126
Am I understanding correctly that the subprocess' stderr is being redirected to the parent process' stdout? Would it be more consistent to redirect it to the parent's stderr?
You do not fallback to ['-'] here. Why not?
Don't leave trash comments.
I guess this is spurious. "costs and prices" in plural is fine by me.
Why use private names when you use it outside of that module?
Sure. I plan it as a further changes. Should it be done in this PR?
This doesn't seem related to Bitbucket service.
1) In a near future I want to remove also this patch.
2) How can you manage the matching lines before and after the effective line with the closed parenthesis alone ?
Why? 

`name='name' name='name'`?
I think this will cause issues. Try testing with qsa off. A leading space in Sizzle is treated differently than a leading space in QSA.
I advise against `uint32_t i = args[1].getAs<int32_t>();`. This indeed gives the possibility that the user may specify negative values. (Actually, since I have increased the warning level, this should also produce a warning with gcc!) If a value is NULL, `getAs()` will raise an exception.
The identity function is a good default for the reducer. I don't know what would be a useful default for the mapper.
Also should be have a QUERY_RE constant perhaps?
Domain name _is_ not valid
Would it not perhaps be better to have `MacOS.clang_version` or whatever anyway; it's a better more transparent as to what is being searched for.
Please don't use URL shorteners
I would spell this

```rb
as_framework? ? prefix+"Frameworks/Python.framework/Versions/2.7/lib" : lib
```
Heuuuu... Le nouveau nom de l'image est pas celui du dessus? Il y a eu un "git mv" dans le commit 0a84901384a01b09be9e6271b0025f15771c7842 (branch 0.2/dev/chelendil/data)
Would it perhaps not be better to have a single callback that calls all the pieces? Just figuring out what each of the these callbacks is being called with is a lot of running around the code base.
colons ?
We get a `RiakLink` object if the last phase in the mapreduce is a link phase or if we have no non-input phases. If we put our own map or reduce phases in, we don't get `RiakLink` objects out because the client libraries don't know that the results are key. This is basically silliness in the client libraries, because you can't tell them they've guessed wrong.
Having the README in the folder with the example makes the documentation immediately clear when someone is browsing through the examples either using github or when inspecting the folders after a checkout. Let's leave it like this for now, and once we add the information to the wiki, we can simplify the README and link to the proper wiki page.
OK. De toute façon, c'est déjà suffisamment abscons pour que seules les personnes qui comprennent les termes soient les seules susceptibles d'être intéressées par ce passage.
I believe you could get it from the routes state.
This is unnecessary -- should be a method on the `meetup` object (`meetup.next_event`).
Might be worth checking the RECT values before casting to avoid overflow.
So we're logging all failures, but throwing on the last one?   I think I like it.
Why AssertionError, and why not wrap e?
Why moved this to be a ToolButton ?
I noticed now you habdled this in terminated callback.
Comments with multiline support can be like this I think:

'comment': {
	pattern: /(^|[^\\])(\'\'\'[\w\W]*?\'\'\'|\#.*?(\r?\n|$))/g,
	lookbehind: true
},
In an earlier implementation it was optional but now it's not and I didn't update this code. Good catch!

-----Original Message-----
From: Peter Flynn [mailto:reply@reply.github.com] 
Sent: Thursday, May 24, 2012 2:08 PM
To: Ty Voliter
Subject: Re: [brackets] Menu and Command and Shortcut Extensibility implementation (#936)

> +        this._isEnabledFn = isEnabledFn;
> +
> +    }
> +
> +    /** @return {Command} */
> +    Command.prototype.getID = function () {
> +        return this._id;
> +    };
> +
> +    /**
> +     * Executes the command. Additional arguments are passed to the executing function
> +     *
> +     * @return {$.Promise} a jQuery promise that will be resolved when the command completes.
> +     */
> +    Command.prototype.execute = function () {
> +        if (this._commandFn) {

The ctor arguments above imply commandFn isn't optional, and register() below throws if it's null, so is this check unneeded?

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/936/files#r878115
Add parentheses to clarify operation order.
The original English string has a space at the end and a number is appended to this string. So do you need a space also? just checking...
It seems like this detects the case where the attr value is empty.  Why do want to short-circuit in that case and return an empty tagInfo?  Wouldn't it be more correct/useful to still run the code below and return info about the associated attr name & tag name?

I think it'd also be good to add a comment in the code describing (a) what case this is detecting (since it took a bit of reading to figure it out), and (b) why we want to return empty info for that case.
Check with C++ expert this is a good way to initialize a class in allocated memory
q, t, l, c? I can't even guess what these are =).
no, this test literally is for xxx.equals(xxx); This is correct. all other cases are further down.
Maybe blank here (and below in mangleJavaConstantName) should be an IllegalArgumentException?
Format is TODO(#{YOUR_USERNAME}). We need to be able to trace this back to the author.
maybe * @include OpenLayers/Layer.js too (to be double checked)
The typedef is unnecessary
For future extension and easier usage I recomed to allow passing Map instead of list of arguments. It is easier to extend it and also allow to set only part you want ( so if 100 for max is fine, you just set current progress ).
Eh, no. You can just write here in the comments. It is just few lines. I am interested only in the `LanguageField` definition.
Why empty line?
Don't be be sorry. You are learning! ;-)

But I have to think of a way of explaining all this things which I explain to one to everybody. Hmm. Because others will for example not learn about this nothing. We should learn on our mistakes, but how to tell all of them this. We will really have to start going over all pull requests, probably. Hmm.
Please order them alphabetically. And combine into one line:

```
from django import forms
from django.contrib.auth import forms as auth_forms, models as auth_models
```
move comment plz
make this multi-lined please
is this Fn even necessary?
any way we can meld `userHandler` and `searchHandler` into one function. The logic seems pretty similar and you can get a reference the to element it was triggered on through the event that gets passed in.
You probably don't need some much code here to run this test and have it finish decisively.

Since the popcorn code is created using `popcorn-wrapper::generatePopcornString` eventually, why not just test with that instead? I believe you can get at it just by using `media.popcornString`. Maybe just come up with some tricky text to put in a footnote plugin text field and make sure it gets escaped properly on export.

You might have to wait until #242 lands, since it skips constructing the player here and just writes out the string instead.
Should this not also have the `butter-btn` class?
this object literal and the one below should be multi-lined
From a code-cleanliness, decoupling-friendly perspective, referencing `Popcorn` specifically for its `nop` function here doesn't seem right. Defining a `nop` for Butter, or even just this module sounds like a good option. I can only assume that this snuck around the lint "implied globals" check because we made Popcorn a friendly global name, but it still seems rather arbitrary to include it for its `nop` purpose here.