labofsahil/github-event-dataset-2012 huggingface dataset:
Use a full key path in the error messages instead of a single key.

A new method, `key_path` is used for errors inside of the check_property method and methods called from within that method. This means that check_object for additional keys and dependencies needed extra logic to display the key hierarchy as they have not yet entered check_property, where the key_path is modified.

This helps issues where there are identical keys in different structures and you don't know which one is funky. It also tells you which item in the array is messed up as well.

`test_object > foos > 2 > description may only be 1024 characters long`
LPS-25393

Hi Daniel,

I've restructured this commit a little bit. Please check it and let me know if you have any questions or considerations.

But let me explain why I did those:

- First I've left the search for the Specification-Version at the first place. The reason is that this has been served the portal for several systems, so I wouldn't remove that. So my assumption is that most systems will stop here.

- Then if needed we will fall back to the Implementation-Version. again if it can be found here the execution will return to the calling environment at this stop.

- then we will fallback to the fedora style packaging (which is not quite common though).

the reason I've added the returns is that if we have a value we will save some ifs. well I'm just thinking now that we shouldn't use the validator since we are sure that the GetterUtil won't return with null. but can we rely on that class' behavior? Never mind.

Please check out the changes and let me know your thoughts you can refuse to go with this solution :) It's just an idea.

Máté
Automatically create sparse indexes in unique and null fields

When you allow null in a unique field, you expect to be able to introduce several nulls in that column field. This code generates the sparse indexes on those fields automatically. Otherwise you can forget to create the indexes and get bitten by it in the future.
Syncing tablet with master
4 typos and errors corrected
Adding DockSpawn
Add local-infile option to mysql client

Be default the newer versions disable the local-infile command in the mysql client, causing the import step to fail with an error message.
Fixed time zone from BST to UTC for creation of all nodes. [2/6]



 .../views/barclamp/clouderamanager/nodes.html.haml |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
Replace transaction[Start|End] with [start|end]Transaction
Pulling Assets from Dropbox 20120625

#	Code - Group - ACJN/
#	Meeting Notes/
#	Prototypes, Wireframes, and Assets/
#	Tools/
#	html template/
Build std.net.curl and documentation on Windows (second attempt)

Another attempt at enable std.net.curl on Windows.  This is actually identical to the last one but comes after some discussion.

[Under Walter's advice](http://forum.dlang.org/post/jijhcs$e04$1@digitalmars.com) I've prepared a separate download containing cURL for Windows users. Phobos unit tests will break for anyone who does not have cURL on their machine unfortunately (and this includes the auto-tester).

You can find my packaging of it here: [curl-7.24.0-dmd-win32.zip](http://gnuk.net/d/curl-7.24.0-dmd-win32.zip).  It is dynamically linked which affords users easier use of the LGPL libraries this particular build of cURL uses.  Alternatively you can use the version of cURL that was offered during the std.net.curl review which has fewer libraries it depends on (just SSL): [libcurl_7.21.7.zip](http://gool.googlecode.com/files/libcurl_7.21.7.zip)

@braddr Can we get cURL installed on the autotester machine?  This pull request shouldn't be merged until this happens.

I have two more pull requests related to this which eases finding/downloading this cURL packaging for Windows users (one for the website and one for the installer).
Redo and returning true if undo/redo did anything.

I added two features that we needed for our app.

The larger one is redo.  It doesn't pop states off the stack, but instead keeps an index to the next state that should be returned by undo.  If a store() is called while the index is in the middle of the stack, it truncates the array to remove the remain states that should no longer be accessible.

Because store() is usually called before an action is performed on the model, there are generally some unsaved changes to the model when you call undo.  These unsaved changes are stored in a temporary attributes variable so they can be redone (redo'ed?).

I also return true if an undo() or redo() call actually applies attributes to the model. If nothing is applied, it returns false.
Trading API

I needed to use the GetCategories method for better product searching so I went ahead and implement that method...
Fixed dependencies and bumped version to 0.1.3
Merge from unstable (move to cordova 2.3)
Support for @ReadOnly annotation

It's now possible to annotate properties with @ReadOnly so that the values are not set during deserialization. However, the property values can still be serialized.

Closes issue #85
LPS-22179
LPS-18364 AssetRenderer fails when embbeded in custom portlet due to inc...

...orrect class loader
LPS-24865

LPS-24865 and LPS-24619 and LPS-24921
Added support for customizing JsonSerializerSettings
Added getGontainer() method ...

Added getGontainer() method so DoctrineContainer should be accessible through
Zend_Controller_Front::getInstance()->getParam("bootstrap")->getPluginResource("doctrine");
fix structure and rm non translated strings

cleaning a mess
C++ MPL Implementation

This is an esoteric implementation in C++ using template metaprogramming. It isn't how you would *want* to ever do this, but there are some pretty simple examples of C++ MPL, so it might be a good example for people wanting to learn C++ MPL. I wouldn't be chaffed if you didn't accept this, since it isn't how one should write real software, but I thought it was at least worth a pull request ;-)
upgrade metabox code to 0.8

Upgraded metabox code to 0.8 to incorporate those bug fixes and changes
Use self.tags_as (from the super class) instead of self._tags_as_string....

...

They do the same thing.
Pull Request for Connection Pool Scaffold

Hello OPOWER!

After speaking with Donnie Small, this is my finished code for the Connection Pool Homework.

Enjoy -
Samuel Reed
Use locale from Request for Symfony 2.1.0-DEV
Adds Wall Street Journal

Heere they are!
修改3.4一处错别字和4.1一处代码错误

如题
reinserted code for placing objects
Fix for live BBC iPlayer streams.

Hi,

Looks like the BBC have recently tweaked the config for the flash player on their live streams and set the drawing mode of the flash player to direct which lead to all kind of problems for Plex. This fixes that by resetting the draw mode to opaque.

More could be done with the live streams as the live streams are really now a 4 hour live buffer, meaning it's possible to seek back and forth. Some other day maybe...
some thoughts

color variables, and todolist reformatting. what do you think?
Update DeviceSettings/res/values-hu/arrays.xml

I'd like you to merge, this translation. Thank you!
New collective contributor.

Add rudaporto as a new collective contributor.
Fixed left menu submenu position on small resolution screens.
Use the new related_content field
Pull Req - Fixes issue #68 [webworksready event on BBOS/PlayBook OS]

Fixes issue #68 [deviceready event on BBOS/PlayBook OS]

When testing, please create an event listener for webworksready (see below) in your webworks application. You will then want to package this application using Tumbler, Snarf and BB10. For all three platforms, the webworksready event should fire.

<script type="text/javascript">
	document.addEventListener('webworksready', function () {
        alert("deviceready event fired");
    });
    </script>
Py review

I like this library. Overall, the python looks good - just making a tweaks to make it more idiomatic.

btw, if you find yourself doing much more python, take a look at PEP 8 (http://www.python.org/dev/peps/pep-0008/) and pylint (http://www.logilab.org/857).
Performance optimization

Loaded all the roles from the roleprovider for each element in the iteration
Add active_model_serializers 0.5.0 support
Handle popup closing for LVL layers and Leaflet popups

This seems to do the trick!
Implement DynamoDB DAO for hierarchical data
Allow OPTIONS request from configured clients

If you're making a call to your app from another domain, OPTIONS preflight request is made before your actual call to ask if your app allows that domain origin and accept X-WSSE headers. (http://www.w3.org/TR/cors/)
In this PR I've forced WSSE listener to answer correctly on that OPTIONS request.
Dev :  Upgraded the subsref method
Small UX change

In the instructions, I just added "cursor:text" so labels appear more invisible when hovering over inputs.

In my own usage, I defined a single class for inline labels (<label class="inline"...) so that text cursors don't appear on all labels and I still have the option to use top- and left-aligned labels.

Thanks for this.
session state in an HttpHandler enabled

Hi, I tried to follow this tutorial and was stuck for a while in an error documented exactly like in this question: http://stackoverflow.com/questions/9771526/object-reference-not-set-to-an-instance-of-an-object-httphandler-session so for the other people following this tutorial step by step (and copy/pasting from here) it would be quite a good idea to correct the handler part as explained in the stackoverflow anser: public class FacebookLogin : IHttpHandler, System.Web.SessionState.IRequiresSessionState (doing so, for me it worked)
Fixes #124 - setup.py now correctly sets templates as package_data
In response to Issue #148 Wrap cmb.js inside namespace object to prevent conflicts

This patch just wraps all the functions in cmb.js inside the CMBMeta object to prevent conflicts with other plugins using similar functions.  For instance we were declaring some of our variables in the global namespace and it conflicted with the Woo Themes Media Upload js.
Mbappou yvan

Bonjour Monsieur,

Je fais face à un problème depuis hier lorsque je lance les tests sur mon rack auth_api_spec.rb

Voici le message d'erreur :

Failure/Error: post '/sessions' , params={"login"=>"ok","password"=>"ok","backup_url"=>"","message"=>""}
     NoMethodError:
       undefined method `password=' for #<User id: nil, login: "ok">
     # ./auth_api.rb:57:in `call'
     # ./auth_api_spec.rb:18:in `block (3 levels) in <top (required)>'


et je ne comprends vraiment pas comment le résoudre.

Pourriez-vous m'apporter un peu d'aide?

Merci d'avance!
timob-8170: Titanium Mobile Template: Template.MasterDetail: needs a back button on the new opened window

https://jira.appcelerator.org/browse/TIMOB-8170
Fixed erroneous mention of the `@` token in lex.dd, and updated windows.dd example code

 * lex.dd had an example which said `@` is an invalid token, this example is now D1 only. I couldn't think of a good replacement for it because D now uses the entire visible ASCII range in its tokens.
 * windows.dd had some outdated code, the example now compiles as-is.

I bundled the two unrelated fixes together as they are both very small.

Fixes bugs [#6734](http://d.puremagic.com/issues/show_bug.cgi?id=6734) and [#5261](http://d.puremagic.com/issues/show_bug.cgi?id=5261).
s/propertys/properties
s/able/available
Hm i think it is quite handy to be able to register and then connect without retyping my password.
Probably already fixed -- but should be __('...') or at least AppLocale::translate.
あれ？インデント変じゃね？
https://github.com/ambi/xyzzy/commit/153afc7fc6fbf6a3c1ba2789e12efe523d66e92d#L0R626
但多数人还是喜欢自然风格的换行啊。
I think a more concise way to do that would be to just call a predefined rake task within another rake file like this:

```ruby
Rake::Task['setup:generate_secret_token'].invoke
```
"akka.remote.RemoteActorRefProvider"?
Something like "os.path.separator" should be available, I'll look into that when I got time
@josevalim, but for assets you need to configure nginx too, but file are gziped by default.
Thanks for the fixes. I've merged into master.
Should this go in the constraint on opCmp?
Problem with github content provider, f.e. https://gist.github.com/1cecc21e80ebc0658951
I agree that this adds yet more complication. But I couldn't think a better way to do this without code duplication.
Good catch. :-)
used same style as previously, will fix
- Please rename js variable email_text to invoice_email_text
- Let invoice file name be <id>.<format> eg. 34.html or 43.pdf  not invoice_<id>.<format>
Was this really the time for this? It seems you couldn't have picked a worse time.
i'm using it on my machine withoiut problems. What is your rebar version?
Its even less than that - the server sends the client 3 numbers: the exp level costs, nothing more (everything else on the UI is just window-dressing generated by the client).  The client uses the 3 numbers to decide which choices the user can make (any higher than their current level isn't selectable).  If the user applies the enchant, all that comes down is the index of which of the 3 choices offered was made.  The server then uses the level associated with that number, along with the item type, to create a random set of enchants, which is what we let you tailor with the second event.

Logically, since you can tailor whatever DOES get applied, you can do whatever you want to do during the first event - if you want to make a set of tailed enchants, and then apply those when-and-if the users opts to use them, your free to do so when handling the first event (just make and save them during the first one, doing whatever interaction with the user you choose, and then replace the generated ones during the second event).  Us doing that sort of thing gets beyond our 'minimum change' goals - the plugin can apply this logic as well as we ever could, and the limitations of the standard interaction between the client and the server restricts what we could accomplish anyway.
Novy bootstrap vyzera famozne. A gratulujem ku integracii Debuggeru do Configurator-u. Toto som vzdy chcel ! Problem co skor, ci vajce alebo sliepka vyrieseny.
Agreed.  I've been planning to address this in a refactor.
Hey David. I saw the multi-hPubs you have down at the Baker issue#20. but i don't know weahter it work or not. do you have anytime to help me out of it.
Yeah, I don't agree with Oleg that that is the least agreed upon, since the
left (sic) distributive law and left catch are actually far more
contentious, but the m >> mzero = mzero law relies on no irreversible
side-effects and precludes anything transforming IO from being a monadplus.

On Mon, Feb 13, 2012 at 11:27 AM, Jason Zaugg <
reply@reply.github.com
> wrote:

> D'oh, I read 'is' as 'at'.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/scalaz/scalaz/commit/d71abed072f271dbcecd159d3fdaac516e49ebb6#commitcomment-960827
>
the original change broke the pytables tests since it didn't take into account that date objects do not have microsecond information.  Tests all pass with above changes.  Please let me know if I broke your guidelines for submitting patches

Thanks!
Sounds good, amended to the last commit, my git log is a little fluctuating tody :-)
https://github.com/m-stein/genode/blob/d134a9b63ae7a0f15e0d0c26a2eb6cd3927d4843/base/include/util/register.h
Ok in all seriousness that name is probably ok.  But I always try to write that the data in the file is, on the first line, just in case someone is looking at it later and forgets.
Fehlt da was?
There's still "1.f" used here.
Fixes bug when auth/orm assumed primary key was id instead of using the pk() function.
"Different activities use different UI threads so one handler for all may not work."

I don't think so:

http://stackoverflow.com/questions/3290361/android-is-each-ui-thread-in-an-activity-a-separate-threads


Dne 19.1.2012 17:56, Fedor Vlasov napsal(a):
> Hello Vladislav!
>
> Fantastic modifications, I like the way you do it! Things that I always wanted but never could implement myself.
>
> Unfortunately I can't merge it because you put them all as a single commit. I want to have a clear history so I prefer small commits. Small commits would be easy to understand and evaluate but one big commit is really hard :-) It affects almost every single piece of code.
>
> Here are some comments from me:
>
> 1. You say it may crash when used in fragments. I know that's true because some people reported me about that. But I was never able to reproduce it myself. Can you advice me a clear steps to reproduce the issue?
> The way you fix it - using handler - perfectly OK. But. One handler instance per ImageLoader. I usually reuse one ImageLoader instance in all activities in application to get better cache usage. Different activities use different UI threads so one handler for all may not work. What do you think? I expected to have one handler in each PhotoToLoad task object. Not sure about that. That's why I wanted to reproduce the issue before making any fixes.
>
> 2. You use imageView.setTag. I used it before but then replaced with imageViews map. Because people may want to use setTag() in their own code and that will break ImageLoader logic.
>
> 3. You use lastModified to clean cache. Very good but unfortunately I'm not sure I understand your logic correctly. I expect lastModified to be updated when I read file from cache. This way I can identify files that were not accessed for long time. But in your implementation you update lastModified in getBitmapFromWeb. Are you sure about that?
>
> 4. You update lastModified in getBitmapFromWeb. But getBitmapFromWeb is only called when we failed to get image from cache so there's no file. Makes no sense updating lastModified. I guess I just misunderstood your logic.
> Anyway I would be happy to implement something like that. Are you sure it's safe. Is "touch" always fast operation or it can be locked for some time if file system is locked by other operations?
>
> 5. In my implementation images are never modified that's why I never refresh images I already have in cache. It's a good thing to do if you need it but I don't want this code in my repository because I want my sample to be as clear as possible so that everyone can understand it and reuse it. Some specific things may be outside of public repo.
>
> Anyway very inspiring commit. Thank you! Don't you mind if I reuse some of your ideas?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gingo/LazyList/commit/e9e2c8310f49768d78aad21bb6a29b831d4ef2f4#commitcomment-882752

-- 
Vladislav Skoumal

+420606722715
skoumal@skoumal.net
And 0.3.6 fixing one failing test.
Useful trick for this is to use <code>\`npm bin\`/coffee-script</code> as that'll return the current npm path
What happened to those umlauts?
it is you the core dev, up to you what to change or not. ;-)
Please add a limit for this. I think `8.kilobytes` is more than enough.
Page类的static method表示的是和Page这些东西有关的工具函数
#924
I prefer "and not self.identified" whenever checking if something is false. It looks better imo since checking if something is true is already just "and self.identified". It really doesn't matter all that much though.
Thank you, @seanlilmateus. I will try to fix it later
这类GTK项目基本都是用gtk-doc的阿，
像gtk，glib，eog，gedit，
+1. After ~24 hrs online - lags. Online very small - 30-60. I reboot mangos every day on 3:00 am.
no. in IsWithinDistInMap also added summ of ObjectBoundingRadius() for both units.
thx for info ;) Ill check this check as well ;]
dammit, thanks
Probably needs java resolution detection for default settings (and Appearance global override), a toggle for switching modes, and a user config for preference.
oh, that could be done
Haha nice man, good job :D
I agree, I won't be implementing the whole factory pattern but I think having the ability to extend the View class for instance is a good thing. For you as a dev I don't think the way it works now is more complicated than before, using instances means you don't have to worry about namespaces as much (especially in templates, and no more "use"). It's a bit annoying having to type `$this->app` instead of `App::` but that's about it.
I would not have thought of having the user pass in an instance into the configuration, but I kind of like this. It seems more flexible.
The comment should have been: Added obfuscation test stub for issue #587
I'm sorry to be a bother, but my last name ought to be spelled 'Tenney'. Thanks!
I initially tried clojure.contrib.base64/encode but it was failing in my use case. I was using it to base64 encode some binary data read from Active Directory. When I actually used the data (displaying images), the contrib version didn't work whereas using the internal Sun class did. I realize this is not an ideal solution but it's working for now until I can find out more.
Hum.... not sure about this one. I should add this case :)

Jean
Le 25 janv. 2012 00:14, "Sven Nierlein" <
reply@reply.github.com>
a crit :
>
> 7x24 should still be logged at least once a day. Maybe on logrotation.
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/naparuba/shinken/commit/17706831d6cfa96814cdc54d20712335780a8098#commitcomment-898325
Rallaz,

there is more work to be done for the TEXT entity, for example we also need to implement all the alignment. Please give me a bit of time to implement that and I will send you a pull request for verification
PREFIX=${prefix} is likely better for things like meta-micro
Trying to merge this with multirepo3, I get conflicts in ``process/buildrequest.py``, and ``schedulers/base.py``. So I think we should merge this now, with the exception of those two files. We can then fix them up after the rest of multirepo lands.
i do, thanks
...I'm surprised that nothing caught this error sooner.
обычные комментарии к коду можно писать как обычные комментарии, doxygen'у все равно плевать на них
oh l'autre.
Excellent point on the BC.  If someone had written a subclass of `FormMixin` that relied on `get_form` always providing a class this would change that behavior.  I'll bring it up on [the ticket][] as well.

[the ticket]: https://code.djangoproject.com/ticket/17557
В сборку "Все в одном" не получится - там принципиально один файл, нужные адаптеры будут вставляться в него механизмом сборки.

В "простой класс",  можно приложить отдельные файлы с адаптерами, нет проблем. Как включать, не очень принципиально. Можно в документации написать, что при использовании классов нужно подключать свой собственный автолод и не писать явно инклуды.
Hello people. I realy like Moment.js, it's lightweight and has great homepage with rigid documentation. But I find it laking one method for geting the number of days in the current month, so here is my contribuion. Hope you wish to include this in master.
The identation here looks a bit irritating to me. But maybe it is only my point of view
Maybe too in-depth.  I don't know that we need to document how to install Python or Pip.  Linking to the virtualenv page might be good, but a better solution is to refactor so we don't need to run in virtualenv.
I think don't bother with the `assert_equal [], ` part of this line. Maybe I'm missing something, though (not super familiar with this testing framework).
Probably because you don't have any location providers enabled (I added some code to check for that, but it's still buggy).
great lengths you have gone to make joke, well done sir I had a hearty chuckle
I'll try and I'll give you a feedback :D
i like this. i felt plugins should be loaded from CLI too.
You can add a default case to your switch for error handling, instead of throwing an exception.
FYI, I've forked in GitHub my own development branch of your MM plugins to
try and fix these. I've found numerous places where fixes are needed, and
some of them are already on their way. I've committed the fixes into my own
local repository, but not pushed them into GitHub yet.

Would it help you if I push my changes into GitHub and you can review them
and maybe take into your own builds if you see that they fix things? Just
trying to help you in your work...my GitHub username is "etsija", you will
find my fixes from there, as soon as I push the fixes.

    BR, Jyrki



2012/1/18 Rob Nelson <
reply@reply.github.com
>

> I'll look into it.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/N3X15/MinecartMania/commit/f0c4b20744c65c9f19c6348612df7e1a38e2798b#commitcomment-876773
>
Из-за правки CMakeLists.txt теперь не будут определятся HAVE_IFADDRS_H, HAVE_MALLOC_TRIM которые используются в коде
于 2012-2-25 0:16, liu fengyun 写道:
> You can add *.class to .gitignore, only upload source code.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sleepcry/Busecretary/commit/0c534e41ae9884a24c7fd3fc54356d4be0208842#commitcomment-1008170
i tried to add bin/ and gen/,but it didn`t work

-- 
Long Live and Prosper
Yours
Liuq
Sorry my mistake. You are right, I missed the previous increment.
Haha - `php index.php -s install gravatar_helper
чем обоснуешь использование id формы для доступа к элементу? лучшее решение?
Это почему ?
I agree with the above, but we can still put the test to see if CMAKE_C_COMPILER is an MPI compiler before, for 2 reasons:
 * respects the principle of least astonishment ( *"I set the compiler to be an MPI compiler so why don't you take it?*")
 * easier to set - 2 cmake variables instead of 4.

Another side note: I would like to be able to define MPI_LIBRARIES and MPI_INCLUDE_DIR directly and avoid the use of an MPI compiler altogether. Is this possible with the current FindMPI provided by CMake?

Yes, the Qt creator issue is unrelated and probably a PATH issue.
I'll test it on my university next week!
We can remove the Rails 3.2 Gemfile and use the global Gemfile
Also, just noticed a lurking trailing space
I think it makes sense to move this logic into the claim class itself rather than having it in gen_x12.
hey thanks! but how could you put the first `cond` clause on the same line?! blasphemy.
ну, я не работал еще никогда с этим псевдоклассом (lang), попробовать интересно, конечно. и как оно в плане производительности по сравнению с js.
в целом, идея мне нравится.
I Gary, this problem was left over from Sendu and it was my first attempt a purging the backstack. I have since written a better method for this and popped in into the WinPhoneKit.Navigation project on my account.
No direct linking of the files viz /home/hardik... understand how URL_ROOT is used. Refer to base.html for more help
oh, I see, you're talking about when cachify_css() is invoked after a cachified resource is changed.  I think a stat check for mtime and size is a good solution.  the inode will be in ram anyway. ?
not only i think that what done before with code reorganization is wrong
thx for revert.
why the update of the core?
Dangit. My fix looks like it works on clojure 1.3 as expected, but not 1.2. CLJ-855 is killing me!
Thanks SirTyler.
This used to be a method of a class, until it made more sense to make it a function. Self should be "coll" now.
I suggest you verify whether the translations are properly picked up or not; there seems to have been some issue with i18ndude which made me to forcefully insert the translation domain. 

If you see missing translations in the future this might be the cause, but I did not investigate the issue too deeply.
This is here to check that adding the shuffle did indeed change the sort order as we expected.  It makes more sense without the debugging line above.
(It caught a couple of pre-exsiting minor bugs in the meta processing...)
it will be better if we put this"(UI_USER_INTERFACE_IDIOM() == UIUserInterfaceIdiomPad)" in prefix.pch file and define a constant which can be used across the application without calling idiom repeatedly

ex:

#define ipad (UI_USER_INTERFACE_IDIOM() != UIUserInterfaceIdiomPad) which sets ipad value to true, which can be used across the application.
If we use the node instead of the node id in the SvgNodeTabEventHandler, we would not need to look up the node again (and might be able to easily move the method later).
but this line adds the class and never removes it – did you actually test this code and does it work as expected?
na página do
the blue background doesn't contrast as well. i also liked the orange bar better too. but that's just my opinion
cherry-picked from issue11 branch for deployment
This method is used by `Decks` to return last updated time to a client. This method is not public because class `LastUpdateTimeHandler` is not public. BTW, I’m not sure, all package-visible members of this class can be made public with the same result. Maybe it is better to do this?
Error has occured when I use `brunch new` and brunch-extensions@0.2.0 at my local. When @0.1.0, it was specified "postinstall" and there are no error by `brunch new`. Why do you specify "prepublish" now?
guava has a checkArgument method that can eliminate the if statement here and below
No, the ZTE update did that.
new stdClass();
I would use the description of the image as the "title" attribute:

                if fullimage is not None:
                    # Check to see if the alt / title tags need setting
                    title = aq_acquire(fullimage, 'Title')()
                    if 'alt' not in attributes:
                        attributes['alt'] = title
                    if 'title' not in attributes:
                        attributes['title'] = description or title

I think it is better, and title is given if a description is missing
shouldn't we be prefixing those with `wl-`? So they don't collide when we e.g. use a plug-in?
Thx, Timm. I'll never get used to this coding stlye. :/
额。 :em06
String s2 = (cType != null) ? cType.getBaseType().getName() : null;

which then sends it back into a null ptr since what it's pointing to is XSSimpleType not XSComplexType
tcsh was giving me trouble about not properly escaping `\n` newline characters, so I thought this was a pretty straightforward solution.
Ah - opps if/then. Ville commented on a redhat bug but I did, so I've got a bit more to fix. Saw Ville's ssh-add now that I know where the right repo is. Ville's _parse_help won't work as at least version OpenSSH_5.8p2 didn't support ssh-add --help.
Surely the leading backslash is wrong.
BTW: just noticed... 

the "CommonRaw" demos work fine.. (they just dump to the framebuffer, even in an xorg environment..)...

Regards,
Du solltest sonderzeichen verbieten und den gildennamen auf ... 16? zeichen begrenzen
Um comentário que se estende por várias linhas começa com a seqüência de caracteres /* e continua até ser encontrada a seqüência de caracteres */, que marcam o fim do comentário. Ao encontrar a seqüência /* o interpretador de JavaScript procura imediatamente a seqüência de finalização */, continuando aí a interpretação do código e ignorando o que está no meio.
stranger commit
I think this docstring should start immediately after the function definition.
Hey Ben,

sorry didn't get the point...what do you mean with tv shows? The script currently supports only movies?!?!
This procedure only is possible if your documentary is categorized as "movie"...

cheers,
mad-max
Just curious - why are we removing this one `else if` here? Is this check encompassed in the `if` statement on the next line?
I think the _LOCAL_SUFFIX  needs to be set first before others.
fixed, thanks

On Sat, Feb 18, 2012 at 2:19 AM, Martin <
reply@reply.github.com
> wrote:

> result is not set, so it will result in an error.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Manromen/script.TraktUtilities/commit/e9eea90aad90594bc73493e13374e3dc3954aef6#commitcomment-978347
>
yeah, i have search for this, and never saw on other kernel source too, and i think you're right, when i try to disable this, i feel less lag before i try disable, nice trick, you're great
 thanks for your answer :)
I agree that, in general situations, the red color should be used with caution in the UI, and indicate exceptions / errors per se. But there is no silver bullet. When we narrow down the discussion to the context of online judging, and to where the "Accepted" messages appear, I would ask, why, after all, we are highlighting the "Accepted" status of solutions (whatever color it is)? IMHO, the acceptance status is the very piece of information that a student desires most after submitting a solution. And it seems to me that the #1 priority, in this situation, is to ensure that the "Accepted" status stands out from all rest information. Red FG over white BG is a good combination in terms of contrast, green FG over white BG is less eye-catchy according to the "Colors-UI" article you suggest. Concerning its symbolism or implied meanings, using the red color to indicate the "excitement" or "noteworthy" of an accepted solution is also justified.
Agreed. +1 sha sum.
I'm trying to clean up my local additions. I had long ago added this plugin to the sources on my hdd, but never pushed it back up to github. Just now I synced back with upstream and this was present, but commented out. I'm attempting to upgrade to the latest, but keep the same setup that I had previous.
See https://github.com/J5/rhc/pull/19.
Would it make any difference if I turned it into a fabfile instead? It's just for convenience after all.
Where did you define this class?
Minor adjustment. Shouldn't this be `return test != null`?

I realize `return false;` is still required which is annoying of Java, but this would reduce 3 lines down to 1.
okay, removing it
No, there can never be any replacement. We check for existence a couple of lines above.
ok, cool, this is the bit you mean?

```scala
def getCallerName(clazz: Class[_]): String = {
    val s = Thread.currentThread.getStackTrace map (_.getClassName) drop 1 dropWhile (_ matches ".*AkkaSpec.?$")
    val reduced = s.lastIndexWhere(_ == clazz.getName) match {
      case -1 ⇒ s
      case z  ⇒ s drop (z + 1)
    }
    reduced.head.replaceFirst(""".*\.""", "").replaceAll("[^a-zA-Z_0-9]", "_")
  }
```

Is there any profiling that shows it is faster to use the Thread instead of the Throwable?
There is no conclusive answer, as described in the "Conclusions" section below. The question is there for search engine hits.
case _ => would work quite well here
neither … nor …
I'm not sure, but I thought we said that the akka.test.tags.exclude should add to the defaults, and not override it?

The use case was that you just wanted to exclude some more tests, and not all of a sudden start running tests that you didn't know were excluded. If you only want to run a certain set of tests, then you should use akka.test.tags.only.
What happens if this is negative?
Nice that it is still tracking the rename with the changes.  I seem to get that broken with git occasionally.

This is the big part of this refactor, getting all the packs -> merge to sets correct.
By the way - does the Gaussian version look OK to you?

On Mon, Jun 25, 2012 at 3:45 PM, Ariel Rokem <arokem@gmail.com> wrote:
> Yeah - I am starting to think that using the Rice distribution from
> scipy.stats will be the most satisfying solution. Will keep working on
> this.
>
> On Mon, Jun 25, 2012 at 3:37 PM, MrBago
> <reply@reply.github.com>
> wrote:
>>>
>>> +    # SNR = var(signal)/var(noise) => var(noise) = var(signal)/SNR:
>>> +    p_noise = np.mean(p_signal/snr)
>>> +
>>> +    if noise_type == 'gaussian':
>>> +        # Generate the noise with the correct standard deviation, averaged over
>>> +        # all the voxels and with the right shape:
>>> +        noise = np.random.randn(*vol.shape) * (np.sqrt(p_noise))
>>> +    elif noise_type == 'rician':
>>> +        # To generate rician noise, we add two IID Gaussian noise sources in
>>> +        # the complex domain and combine them together:
>>> +        noise1 = np.random.randn(*vol.shape)
>>
>> I think we need to distinguish between var(noise-sample) and var(noise-distribution). These two will be very close for a large noise-sample, but at the end of the day, var(noise-sample) is only an estimate of var(noise-distribution) and when noise-sample is small, it's a pretty bad estimate. I believe is usually defined as `SNR = mean(signal) / sqrt(var(noise-distribution))`. We can either hard code the analytical form for var(rice-distribution) for use scipy.stats either should work.
>>
>> Bago
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/nipy/dipy/pull/26/files#r1046764
https://github.com/srkunze/ckanext-semantic/blob/master/ckanext/semantic/theme/templates/package/search.html
The extension uses them as CKAN should still work without JS.
A middle-ground might be to remove the indirection where it looks up the user name in a dict to get the apikey, so use positional params but do `action_post('user_create', data, self.apikeys['random_key'], status=403)`, it's just nice if the args to action_post match up with what you need to call an action  function so anyone who's seen the action functions can understand it right away without having to study the test module code (and also action_post could be reused across test modules). But not a big deal
No need to highlight "bound" here, you're not differentiating these consumers from other consumers mentioned elsewhere in this documentation.
could simplify this as ```if kwargs.get('note') is not None:```
rescinded.  we will, but only within a given request?
Indeed it is, I didn't modify it from the original snippet in `load_mrjob_conf()`. Will change.
在外层（函数开始部分）已经有定义了box变量，根据ER的编码规范，所有变量统一在函数开始处定义，不需要在for循环中重复使用var关键字
Would it make more sense to just test for `@result` in the first condition?
why this change? afaik `-lutil` is required on linux.
more copy paste stuff : commodityion -> commodity production
This will insert the pasteboard text twice.
http://www.youtube.com/watch?v=tKdcjJoXeEY
The output array has 0-size. So nothing needed there. But that was the problem that is fixed by changing that while loop, because it actually did one inner loop in that case before.
Ок, исправлю, только подожду пока побольше исправлений наберется)
плюс на другую строку
I think you can `dup` only in the `else` clause, since `options.merge` already creates a "duped" object.
Yes, @carlosantoniodasilva is right. I prefer to leave it private
It is true. Maybe I'm looking in the wrong place ;)
This is creating many intermediate strings, maybe you could change the if above to concatenate to a buffer instead concatenating everything at the end.
It does :) except it should be "perform_deliveries".
Should not be `test_caching_stylesheet_include_tag_with_absolute_uri`?
Have you changed this? Or this is the correct fix?
Yeah, sorry, it's not very clear: (2) is implied by the inclusion of one box in the other (ownership relation) 
I'll make it clear in the next pull request.
Hmm, I forgot that I changed that. I can change it back, it's really not a problem.
You need raw strings now (because of TeX backslashes):
replace the tsrating `"""` by `r"""`.
I'd also recommend clearer names for parameters.  In any case, you are advised to document them in the docstring.
I will reintroduce the previous behavior by default, and the option to have the new one.
Please use Function.lazy instead of Function.lowDelay(0), it's more optimized for lazy call of function in some browsers.
in PhpStorm that looks good. I use spaces... try to fix it.
Reader dont have to jump if method named right (exposes its intent). I can rename it to getAccessMode() for more readability and shortness. mkdir() and getAccessMode() I think must be in separate methods because its more error prone to duplicate that code (you can assume that this algorithm used once, and forget to look for copy of it) and makes class code shorter (more readable).
So from looking around is it a simple replacement of Doctrine_Record with Doctrine_Entity? 
Also, I can update the code to return an array. In fact this is now done and pushed
Is it intentional that you dropped the initial "test." from these paths? I see you added @relative; do we want separate test cases for relative (without 'test.') and non-relative (with 'test.')?
Sorry I don't mean to jump on @joshaber's territory here, but the indentation here looks like it's spaces and not tabs.
I think you mean "in the :program:`mongod` process, rather than your application code." rather than "on the server code." 

   Use the ``distanceMultipler`` option to the :dbcommand:`geoNear` to convert the results of
   the  :dbcommand:`geoNear` operation in the :program:`mongod` process rather than your 
   application code. See the :ref:`distance multiplier <>` section for more information.
Just filling this with nitpicks... This template could probably live at `wiki/templates/admin/wiki/document/change_list.html` and be closer to the rest of the wiki app.
hmm do you need to attach it to the window object? why not just reference as the global var on line 4?
Maybe we should use exceptions for this kind of stuff?
Why even put all the stuff about navigating to install/index.php? It will automatically redirect you there if you visit the board root. So we could just simply say:
"Navigate in your web browser to your phpBB3 installation and you will be automatically redirected to the installation wizard. You can also manually reach the installation page by adding <code>install/</code> or <code>install/index.php</code> to the end of the URL."
Ah, sorry, I did not understand what truncated text is for...ignore my note.
If phpbb already requires pcre with unicode properties, which it might - please investigate, then none of mbstring stuff is needed here.
This should tell whether or not the currently installed phpBB version is compatible with the MOD. In other words, if it is not compatible, say so.
@Kami @ddispaltro and I talked about this and decided on:

agents/monitoring/lua -> agents/monitoring/default
agents/monitoring/lua/init.lua -> agents/monitoring/init.lua
agents/monitoring/lua/monitoring-agent.lua -> agents/monitoring/default/init.lua

And create agents/monitoring/collector.

That way things map to the entry point/zip name.
why rely on a white space after the minutes? Leave it to the render/templates, remove from here.
jruby 还是加到 allow_failures，经常超时或者莫名其秒地就 fail 了。只需要定期检查并解决 jruby 的兼容问题。
Certainly makes sense. Especially since I can imagine apps wanting to pass around HttpReqs Witt their own app data. I'll post an issue on iterIO but I suggest we leave it for later...
Off hand, I can't recall. I don't see any reason it shouldn't be 25 like you guys pointed out.
Nope, the commit moves the two lines after the CGContextScaleCTM call. The indentation looks good to my in Xcode. Seems to be an issue of using tabs instead of 4 spaces.
чет я забыл про статистику по дням) 
слушай, а так ли нужна статистика по часам?
и если нужна, зачем делать insert_update? по идее просто update будет работать быстрее, т.к. инсерт сначала пытается вставить, и только при ошибке делает апдейт, получается в 2 раза больше нагрузка
просто смотри как можно, есть таблица с общим количеством показов/кликов баннеров, нода по интервалу делает апдейт к ней. и есть отдельная таблица с статистикой(по моему дневной будет достаточно? ну или по часам). и вот эту таблицу статистики уже раз в час/день заполняет пхп скрипт запускаемый кроном.
Stp surveille ton langage !
The location is a steam folder, so it follows that it wouldn't work for non-steam
All test names should have a capital S
Should probably be a public read only property.
As per previous comment by @rhattersley
Surface pressure rather than orography.
Fix it, thanks.
A comment on top of this function explaining how it works would be nice. Otherwise looks good to me.
yes, indeed :)
Good catch. In the future, it's better to separate behavior-preserving and behavior-modifying commits, as it's hard to review something where both what is computed and how it's computed is changed at the same time, especially when the change is large.
The string arguments should be passed by `const &` since they don't need to be copied.
should be `parseFloat( .css("width") )`, correct! Good catch @staabm
as long as we are rewriting this, can we get better variable names than `i`, `r`, `n`, and `l` ?
palloc/palloc0 never return NULL.
Isn't DISTINCT ON slow?  Tried GROUP BY instead?
Why can't we separate the float[] into separate variables so that we aren't lossy on the datatype.  We shouldn't be casting the bigint count(*) to a float8.
On Windows, NODE_PATH worked out better to support both the dev bundle and the npm installer.
This doesn't handle KeyError. What happens if the locale doesn't exist in product_details.languages?
Should be `"#{bin}/wemux"`, to make sure it uses the Homebrew version.
This can be `name = ARGV.next if ARGV.include? '--set-name'`, and it will be nil if the conditional is false (same goes for --set-version above).
Ça me semble ok. Dans quel cas est ce qu'on a des entité sans tiles?
Pourquoi supprimé les entités puis les rajouter? Et pas seulement mettre à jour la liste?
Supprime plutot
I suspect `pyflakes` will frown at this.
I personally feel more documentation is better. There absolutely needs to be a comment about the user_agent, what existed before isn't really sufficient.

The matter of the site_name I think is important. You are correct that most people won't connect to any other site than reddit.com, however, the documentation then isn't for them. It's for those that want to know how to connect to their local instance. The code should explain that.

I agree the code is readable, nevertheless, this documentation is automatically added to the functions on the readthedocs page, and it should be clear form there what these arguments do.
Как-то не так всё равно вышло :) Предлагаю так:
"Так как ветка, которую мы слили, указывала на коммит, являющийся прямым родителем коммита, на котором мы сейчас находимся, Git сдвинул указатель вперёд. Иными словами, когда вы пытаетесь слить один коммит с другим таким, которого можно достигнуть проследовав по истории первого коммита, Git поступает проще, просто перемещая указатель вперёд, так как нет расходящихся изменений, которые нужно было бы сливать воедино."
I've mentioned this and added a test for it.
missing doc-string
Could you please split cleanup into a separate commit?
Fixed in next push.


On Sun, Dec 2, 2012 at 4:36 PM, Reuven Lazarus <notifications@github.com>wrote:

> In muss/commands/social.py:
>
> > +            if message:
> > +                firstchar = message[0]
> > +                if firstchar in [":", ";"]:
> > +                    message = message[1:]
> > +                    if firstchar is ":":
> > +                        message = " " + message
> > +                    target.send("Tell: {}{}".format(player, message))
> > +                    player.send("To {}: {}{}".format(target, player, message))
> > +                else:
> > +                    target.send("{} tells you: {}".format(player, message))
> > +                    player.send("You tell {}: {}".format(target, message))
> > +                with authority_of(SYSTEM):
> > +                    player.last_told = target
> > +            else:
> > +                # This is a hack to force handler to do the error reporting,
> > +                # since we don't have a parse token for "non-empty string."
>
> Would it be cleaner to just start having a parse token for non-empty
> strings? I suspect we'll want one anyway.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rlazarus/MUSS/pull/60/files#r2288564>.
>
>
Doh!  I think you may be right.  I was trying to avoid passing the buck here, but passing null is an expected behaviour and the code I fixed in the other PR already handles all the cases of instantiation (by adapter loading or reflection, as appropriate).  Thanks for catching this.
@dsaff 
This is an excellent book
http://www.amazon.com/Java-Concurrency-Practice-Brian-Goetz/dp/0321349601

, and the key specification JSR-133 
http://jcp.org/en/jsr/detail?id=133

, and the papers writen by Doug Lee (i would say the father of java concurrency)
http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html
http://g.oswego.edu/