labofsahil/github-event-dataset-2012 huggingface dataset:
Just tested this, and it fails when the imported file is in a subdirectory.

	break in node_modules/less/lib/less/parser.js:205
	203         debugger;
	204         if (e.filename && env.filename && (e.filename !== env.filename)) {
	205             return parser.imports.contents[basename(e.filename)];
	206         } else {
	207             return input;
	debug> repl
	Press Ctrl + C to leave debug repl
	> basename(e.filename)
	'colors.less'
	> parser.imports.contents
	{ 'test/colors.less': 'body {\n    color: #330033;\n};\n' }
Thinking on this some more, as I will continue to do for a few days :) I agree that decoupling the route callable from the app is good, but in your example you're moving the coupling from the callable to the hook. The decoupling should be in both directions. The callable in your example does not know about Slim because it simply returns an array. However, Slim must know about your callable because it decodes the returned response data in the hook callback. Ideally, the route callable should return an object that does not require Slim but Slim will know how to handle it in all situations.

What if the route callable can return a response object? This object would replace the existing response object. This also seems obvious: the callable is returning its response. If you choose to echo content instead, that content will be appended to the existing response object.

    $app->get('/foo', function () {
        return new Response('Foo!');
    });

The response object is responsible for its status, headers, and body. Status and header management may be gotten for free by extending Slim_Http_Response. The response body is more tricky to manage. The current `Slim_Http_Response` class is tightly coupled to its assumption that the response body is a string; this affects the `__construct()`, `write()` and `body()` methods. If I can change the `Slim_Http_Response` class to allow easier extension, this may become a viable solution. An example:

    $app->get('/foo.json', function () {
        return new JsonResponse(array(
            'name' => 'Danny'
        ));
    });

The JSON response object would store the data array internally. My only assumption is that it implements a `finalize()` method that returns an array of (int)status, (iterable)headers, and (string)body. But does this inaccurately mix concerns between the request and the view? A bit, yes. Is this a problem? I'm not sure. Either way, we now have complete decoupling between Slim and the callable because both speak a common interface (the response object) without knowing about each other.

Thoughts?

Josh
I already have and replied. ;)
This commit is only for convenience. To compile with clang (I assume) one has to export  CXX=clang and CFLAG=-pthread.
With this patch the cmake argument '-Dcompiler=clang' would switch to a clang build without the exports.

But looking at it again, 'cross-compiler' seems to be the wrong description in case of clang.
Is there a reason why .page-header class was totally removed from bootstrap.css and moved to docs.css?
It means we cannot use it anymore?
Thanks
While you can "hack the system" to build Lua or C, we would appreciate if you specify `language: erlang` in your .travis.yml to move your builds to machines that are not as heavily loaded as Ruby ones. Not having `language:` key is the same as `language: ruby`. Thank you for understanding.
Sadly this won't work. The Persistent Vector takes a bunch of arguments that are used internally. So we need to use a construction function.
Actually the error message refers in fact to the key which was used to generate the signature.
Not sure how to best explain this.
I guess we could set up a separate CI builder that didn't clone vendor and tried to run tests after a ``pip install -r requirements/prod.txt``
Sure!

BTW, I've tried a some different approaches with this new boottool, and I notice that it resulted in a lack of overall uniformity.

Hopefully this will all be fixed and coherent at v3.
Just tried to merge this in to the new master, but is conflicting quite nastily. I've given it a go - but couldnt get it working. 

Any chance of a search-v3 ? :)
add a all field
These both do very similar things. Can this be reduced to one code block? Something like `if (element.type === 'checkbox' || element.checked) {` etc? Should save a bit of wire size.
yes, do it like you want. Its only important, that somthing is reminding the user, that the field is required before submitting the form
The name of this function is pretty ambiguous. I might recommend something more like `_expand_single_key`, but I'm not sure you'll even still want a separate function once you've fixed `_split_keys_on_dots`.
I would :+1: `if present?` personally, and I think we prefer that in the code as well. 

btw: love all your commits! :)

On 23/02/2012, at 1:59 AM, Kirill Lashuk wrote:

> @joshk, I think that's a matter of taste, but I can change it if you want. I'm so used to unless-blank? that if-present? makes me stop and think for a while =)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/travis-ci/travis-ci/commit/901819abf66b43445a9e8aab89f02ad811db3f11#commitcomment-998853
It doesn't seem worth keeping ProcessBuilder around just to display the command.  You have the command anyways, in the 'line' constructor argument. What about keeping (a copy of) that around if it's necessary?
See commit 26fb6bb8f4b9727eb6a20d7b81a1e9ca01eb12a8 which made the change you have just made, and then ff89e757049a5abe21c8e6934275c2565ed58771 which reversed it. Now you have changed it back. We have flip-flopped at least three times. Once you have made sure, put comments all over the code so that nobody ever changes it back! (What if I were to unsubscribe from my commit notifications...? ;)
The indentation of that block seems odd. Did you use tabs instead of spaces?
this should be use_xinetd
Depends on what you want to do... If you just want plain openid auth, then the old version is perfect. If you want to use the "storage" part of openid, too, then maybe I could need some help fixing some bugs around the code...
Greets 
MMo
I guess I would also initialize the timer variable either to `millis()` or `millis() + 30000`.
I can help you with the details of the frostmourne, michalpolko. Send me PM if you like that I explain you the phase ;).
Solo comentar que cuando corrijas lo que te pedí en el otro commit, actualices también los archivos de este.
Hi @jpvasquez , you really solved the issue. Now reloading works good. Thanks for sharing.
improved :-) [b85415f2eaa5740f8e151cce16e772269cc07a36](https://github.com/koffeinfrei/dotfiles/commit/b85415f2eaa5740f8e151cce16e772269cc07a36)
Hello, how has the removal of the file locking worked out? Have you noticed any problems on high traffic sites?
Thanks,
Mike
Doesn't follow coding conventions, please no one-line method definitions
Is "ng:app" needed? You call angular.bootstrap() later on this page.
It's all good, something I was messing around with, thought I had that file on gitignore
增加文件模式说明
due to this modification, the scrolling by dragging the knob wasnt possible anymore. You introduced the `this`keyword into the function, but in this case the `this` refers to the `Slider` instance and hence the Slider's element. Solution is provided in #14.
Yeah that first one is actually the reason I wrote it in the first place. I really need to go on a quest to get everyone to set up reasonable Addressable dependencies.
Missing `id="wdn_dependents"`
Why not just "mkey_list"?
For the Licence I prefer to keep only BSD , currently rendering this long link is :( . Please have a look at http://packagist.org/packages/Aura/Autoload
Looks like I need to add support for this in CssMin..
What version of RubyGems? have you updated to RubyGems 1.8.15?

Does your installation have libyaml installed? Is Psych working?

I recommend instead of passive-aggressive comments (what you call snarky), help us resolve it so others don't suffer.

Here is the Issues page for RubyGems: http://github.com/rubygems/rubygems/issues
Ik heb da gisteren naar u doorgestuurd, het enigste wat er stond (ergens tussen die regels) was: "Library not found!" of iets dergelijks.
 Maar ik ga vandaag wel niet al te veel hier meer aan doen, want ik moet beginnen voor mijn mondeling morgen :)> Date: Mon, 30 Jan 2012 01:16:19 -0800
> From: reply@reply.github.com
> To: glenn_daneels@hotmail.com
> Subject: Re: [confine-dist] OMF feeds is now used. network config adjusted and hostname file adjusted. (0c2add8)
> 
> Welek fout exact? Want ik heb precies geen (duidelijke) foutmeling gekrege
> 
> Op 30 januari 2012 10:10 schreef Glenn Daneels <
> reply@reply.github.com
> > het volgende:
> 
> > Omdat met de laatste versie ik een fout kreeg ivm met "Library not found".
> > De fout die ik doorstuurde naar u. Ik zal het vandaag nog is proberen hem
> > te compilen want het kon idd iets zijn met eglibc.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> > https://github.com/gdaneels/confine-dist/commit/0c2add8ff38c39a9d6d05128e5f3bab64b6106f3#commitcomment-913547
> >
> 
> 
> 
> -- 
> To understand recursion, you must first understand recursion.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/gdaneels/confine-dist/commit/0c2add8ff38c39a9d6d05128e5f3bab64b6106f3#commitcomment-913564
please; sorry; you need to add a double space prior to that "-" in order to achieve an indendation. Sorry I didn't realize it when aa8ae835400bc09d13552304e10f653a58cd6dd5. Thanks for your care.
I think your comment about the scale parameter is a bit misleading. I know what you mean, but it implies that you can increase the size to 1280x1280, which you can't. The maximum div and image size is 640x640 regardless of what scale is set to. It looks like the server accepts sizes above the max, but returns a chopped image limited to 640, which the browser will then stretch to fit the div. This may be ok for some display purposes, but is no good for vector overlays, as they will appear to be in the wrong place on the distorted map.

Thanks for this example, though; it's ideal for use in smartphones, which only have a small viewport anyway.
are we sure this isn't used anywhere?
Thanks. I added them for the next UDB release.
yeah!
But its ready for dual boot things... lol
@jonathantneal e.g. Using ems for margin/padding of the element itself.
...FWIW, this looks like the same information as in the statistics controller.  This suggests to me that the information should really be put somewhere centralized.  I'm not sure where... perhaps in a model or a class or something.  But then you could do something like:

```ruby
EolStatistic.all(:select => data_required[:content_partners].join(', '))
```
The first found thread disagrees with you: http://ubuntuforums.org/showthread.php?t=397871
I respect licensing.
You don't need parens in the outermost method call on each line.

assert_equal Byte.zero, Byte.zero
Did you test the triple === with 'confirm'? Why take the highest eq check?
Well, I didn't want the case where it should have been the first but was the second to crash.  Especially since they are semantically equivalent.

It's a to many property with a single object not in an array that was crashing.  Perhaps not the perfect format, but also not a good idea to just crash.  Especially if someone pushes an update to a file over the internet.  That would break in the field.

On Feb 23, 2012, at 3:15 PM, Magical Panda Software wrote:

> You know, you can set this value in the userInfo dictionary:
> 
> mappedKeyName: chestDropRate.name
> 
> for a single value....but still, the to-one and to-many code is already there....not sure why it's missing that even though you changed the import to update...
> 
> On Feb 24, 2012, at 12:13 AM, Brian Doig wrote:
> 
>> "chestDropRate" : [
>>   {
>>     "name" : "foo",
>>     "sequence" : 0,
>>     "value" : 0
>>   }
>> ]
>> 
>> works but
>> 
>> "chestDropRate" : {
>>     "name" : "foo",
>>     "sequence" : 0,
>>     "value" : 0
>>   }
>> 
>> throws an exception because it tries to iterate over the keys of a NSDictionary which returns NSStrings which then throw a selector not recognized.
>> 
>> On Feb 23, 2012, at 3:09 PM, Magical Panda Software wrote:
>> 
>>> Can you give me an example of how this crashes? I'd ideally like to make a test case for this, but I'm also curious since this hasn't crashed here for me in a while...
>>> 
>>> --- 
>>> Reply to this email directly or view it on GitHub:
>>> https://github.com/BrianDoig/MagicalRecord/commit/32cf9d23363e137917b1ce89c5f85af336bc09e5#commitcomment-1004870
>> 
>> --- 
>> Reply to this email directly or view it on GitHub:
>> https://github.com/BrianDoig/MagicalRecord/commit/32cf9d23363e137917b1ce89c5f85af336bc09e5#commitcomment-1004883
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/BrianDoig/MagicalRecord/commit/32cf9d23363e137917b1ce89c5f85af336bc09e5#commitcomment-1004886
This commit is wrong. 3kids is correct.

@Ka0z there are no spells being cast at 3:26 in the video. There are no interrupted spells before it, so I assume you are referring to spell cast after it. There are two spells that get interrupted close enough to 3:26 that you could mean.
There is Frostbolt that gets interrupted by Blind at 3:31 and Frostbolt that gets cancelled by movement at 3:34.
I also added a bm/ directory to krypt, although they measure functionality that's implemented in krypt-core(-jruby). But since they may be used by both I also thought it better to put them in the "master" repo. What do you think?

PS: Did you try the benchmarks yet? I liked what I saw :)
I was referring more to the ``if(foo) { return true; } else { return false; }`` construct, which should usually be replaced by ``return foo;``
This isn't significant enough of an issue to warrant a release all on its own and making too many releases tends to cause gem conflicts when multiple gems that depend on addressable have conflicting version specifiers. So... no, not yet.
you just foolish . please give me some interesting information
this commit made turrets useless :/
looking into it
Ah cool, it runs off my mac branch right now?

Also, how do I check to see if the icon is included? (Maybe a .dmg is a zip file?)
-- 
Sent from my Android phone with K-9 Mail. Please excuse my brevity.

Maescool <reply@reply.github.com> wrote:

@dkowis added a mac slave to jenkins to build mac apps :)
whenever you update, you can see it on jenkins ;)
http://ci.maescool.be/job/Catacomb-Snatch-mac/

--- 
Reply to this email directly or view it on GitHub:
https://github.com/Maescool/Catacomb-Snatch/commit/f78d3e365cb5ffff8229462c33b9d8a1030d2ac3#commitcomment-1011400

!DSPAM:2,4f4949c8136001804284693!
There are several issues with sound...maybe someone should ask for help in the paulscode forum.
Got it :-)
Hey!
You need to build a zImage file.

from:
CONFIG_INITRAMFS_SOURCE="usr/galaxytab_initramfs.list"

to:
CONFIG_INITRAMFS_SOURCE=[path to your initramfs folder]

After, just build your kernel.

$ make -j4 ARCH=arm p1_cm9_defconfig
$ make -j4

Flash it using heimdall.
Ok, before it was just irritating... now it is getting stupid... what database are you using, a trinity 3.3.5? You SHOULD be using the latest skyfireDB.  It does not make any logical sense to add a bunch of 3.3.5 quests and no longer valid spells back into our conditions table when they have been removed from the game and/or dbcs... Also, do you test ANY of this stuff before you push it into master? I think if I was testing something (as you should be), I would have seen the 150+ startup errors this generates due to conditions not being properly aligned and or valid (caused by this commit). Seriously dude, if you aren't testing this stuff thoroughly, you DON"T NEED TO BE PUSHING TO MASTER. Put it in a branch or even a fork and test the stuff out first... just because it fixes something in TrinityCore, does NOT mean it automatically fixes anything in this project. Hell, it may not even be needed. ALL stuff ported from trinity needs to be put in a branch, examined, and thoroughly tested before pushing it into our  master. 

Example: quest 25444 
http://www.wowhead.com/quest=25444 - Obsolete. Removed in 4.0.1

Another Example "Aura condition has non existing spell (Id: 48085)"... Of course it is "non existing," that would be because it was removed in 4.0.3.

Some of us are working our asses off to make a solid and up to date database... Crap like this can easily destroy it fast. Please start paying attention to what you are doing... and please revert this crap before the bitching begins.
Yes. We could implement the new feature in the plugins now, I think.
@will-moore: Any idea how relevant this is to chris-allan/openmicroscopy@5b45928234a0933e3d931069237cc1a48db199d9? Is my commit just a different way of fixing the same problem?
Any way friend, we could start remembering old times, and also any consult you can askme too, always we are usefull.
Should be `Whether or not` instead of `Returns if`
I actually like Mike's suggestion. Get rid of the multiple settings and just have one. Using a string that represents a callable is definitely "djangoy" if not pythonic (FWIW if you did the subclassing, you'd *still* need to have a setting pointing to the right class, so that doesn't buy us anything).
Would you mind to write a little readme file to close #41?
Yeah, the other SIO commit: https://github.com/simone201/neak-kernel-sgs2/commit/3ea8cec4379c5ad14c44eb6d1785448ee0af9403
is fine, and is the same one as RedPill and Siyah is using. Hence why I'm asking Simone to only revert that particular commit.
The vertical position needs stay * 2. Otherwise the initial placement of the banner BG will not be centered vertically. To adjust the parallax effect amount change the offset multiplier (currently 220)
Сначала надо в контроллере передать альбом во вьюшку. По дефолту при просмотре картинки он не передается. 
Из сущности картинки его тоже взять нельзя - по дефолту не подтягивается, возможно будет добавлено.
Testing comments in your commits!
哦，好吧，不知道GitHub能不能合并仓库？算了，我还是重新造一个仓库好了
Andrew, any thoughts? I'd like to get this resolved as soon as possible. Should I just fork and maintain a separate package with my changes?
Let's keep this as <tt>android-sync</tt> rather than <tt>android-sync-parent</tt>.

Might reduce CI changes, and it's nice to keep alignment with the project name.
Maybe this should be done using the < (maybe >, not sure) operator instead of a subtraction?  I can also change the code to provide numerical date/time values (in seconds since the 1/1/1970 or something)... at the moment they are strings in ISO format, which I think can be sorted.
build.sh
- copies the generated kernel into the CM9 build tree, which I don't want (I bake my own upgrade.zip when I want)
- does not pass any arguments to "mka"
- I only build for my device, so the multibuild feature is not needed
- seems to blow away the previous build (at least by default) - I like crashed builds to continue from where they stopped,
  and while working on stuff, to rebuild only my changes (yeah, I see NO_RM...)
- for the cross compile flags I have a simple "armake" script in my path which calls make with whatever arguments I give it,
  after setting the relevant cross compile environment stuff. That way I use the traditional "armake oldconfig", "armake menuconfig",
  "armake -j 4" etc, without learning to cope with yet another way to build kernels

Anway, the .gitignore files are already there, and except for the two additional entries of this pull request, nicely cover anything a "normal" in tree kernel build generates.

Also when I desire to clean that up, there's always "armake mrproper"
A side comment. Not something to take so serious. I think the "alert" of "slow plugin" is a good reminder to find creative ways to optimize the code, not to hide it.
Regards
veranderingen:
- elke button is nu een game variabele.
- wanneer je op een button drukt en moet wachten op de server wordt de button op disabled gezet (automatch en join buttons nog niet)
- wanneer er geen connectie met de server gemaakt kan worden krijg je passend berichtje
> By the way, you cannot really differentiate overloaded methods

That's the rationale for returning `MethodDescriptor` and `ConstructorDescriptor`. Instead of creating new types for exposing a method/constructor signature via `Node` I thought it's a good idea to use what we already have for this, i.e. the descriptor types. We might also expose `PropertyDescriptor` to be consistent (although it's not strictly required as we have `getName()`).
Yes, and the iframe and other html injection stuff were included just for demoing purpose of the possibilties and/or possible vulnerabilties.  

We at Enersize use it, for now, just as an additional text field.
Não precisa, eu é que esqueci que existiam aquelas 2 primeiras linhas
But the code, for me, throws an InvalidArgumentException  from 

PHPUnit_Framework_MockObject_Generator::257

> throw PHPUnit_Util_InvalidArgumentHelper::factory(1, 'string');
stdint.h is not available in VS2008 but I verified and it is in VS2010 so this change is a Very Good Thing. Amazing work :) !
Not that I can think of.
Yeah, ```colnames(dec) = levels(esetForVariable[[fName, exact = TRUE]])``` is just setting the column names of dec. dec is put into thisFit under $boolupdn. It is copied from here to tab$Res in computeAnalytics(), but as far as I can see not used after that. It is also accessed within process.atlas.nc(), and written to the tab-delimited file it produces.
General assumption in programming is that ids are numeric. The ncdf file names used to have numeric ids in them - now they have accession. COuld you rename both vars to say expAcc and arrayDesignAcc ?
Let's follow PEP8 here and not add unneeded whitespace to group imports.
user-site should definitely not be considered local in a no-site-packages virtualenv, but it seems that with this code it would.
if you look at my diff,  "top level" requirements get (options.upgrade or options.upgrade_recursive), and dependencies get options.upgrade_recursive.  that's why I did the diff, so I could I show you how I was doing that.   It's not a simple pass-thru of the set property.

I understand your concern about footprint, but my thinking here is that if you have the information at the time of instantiation, go ahead and use it, and set a simple property.
Why do you put this line after writing device.json?
What happens if we have multiple successes ? Which handler name is kept in fact ? I think it may be somehow a random pickup...
Our convention is to only provide additional information that differs from the base.
yes that's what i mean.

The issue here is that we can't really forbid or detect it easily, and if we don't digest the kernel_name with the code we might end up with something wrong
Instead of this flag, could we check if lookup != underlying?
I can revert this change if you want, to me it was clearly dubious to call something "secure" and not use the SecureRandomSeedGenerator. I'll revert my change if you are sure that the original code was right. I'll also add some comments describing why it looks like it looks.
will fold the resolution into the LetItCrash(tm) approach
Also, Arrays are horrible.
Right now there's an implicit conversion from AS to MD: 


    private[akka] object MessageDispatcher {
      //…
      implicit def defaultDispatcher(implicit system: ActorSystem): MessageDispatcher = system.dispatcher
    }
y’aint gonna saturate no 100GB link like this ;-)
I used to just throw, but then I changed it to log an error. I can change it back.
Is this public API?
ok, I think I read this wrong.
I think I moved this to a set file
Uses the @user method directly rather than the Mailer method that it wraps.  The mailer method now unpacks the :subject and :message options from the Hash directly.  Simplifies the API.
I've modified it, hope that's what you meant
It is, i forgot to mention abort() which is also defined in stdlib.h
So using getPrototypeOf doesn't work. My objection was that I didn't see how it could be a better answer than {}, but either way:
- https://gist.github.com/2300868

http://soft.vub.ac.be/~tvcutsem/proxies/ (wrongly) pointed me to http://wiki.ecmascript.org/doku.php?id=harmony:proxy_defaulthandler however from there you get to http://wiki.ecmascript.org/doku.php?id=harmony:direct_proxies, which looks like a solution, however in Firefox-current, Proxy is not a function.

Now my argument "So don't so that" has been slightly weakened by my memory-leak-finder script that does some tree walking, and ...
However even though I've actually seen this code fail, I'm still of the opinion that I don't yet care enough to add the ForwardingProxyHandler code from http://soft.vub.ac.be/~tvcutsem/proxies/. When we have a way this can fail in the real world, then maybe I'll care more.

Thoughts?
on 8.8.2012 19:49 Andrew Chen wrote:
>> @@ -370,19 +377,19 @@ http://k9mail.googlecode.com/
>>      <string name="account_setup_incoming_password_label">Salasana</string>
>>      <string name="account_setup_incoming_pop_server_label">POP3-palvelin</string>
>>      <string name="account_setup_incoming_imap_server_label">IMAP-palvelin</string>
>> -    <string name="account_setup_incoming_webdav_server_label">WebDAV (Exchange) palvelin</string>
>> +    <string name="account_setup_incoming_webdav_server_label">Exchange-palvelin</string>
>
> It is important to retain WebDAV as well as Exchange in this description.  Newer Exchange servers use ActiveSync, which we don't yet support.

This is fixed now.

Thanks for merging the translation!
Remove this comment please
2012/8/7 Gedare <notifications@github.com>

> In cpukit/posix/src/key.c:
>
> > @@ -53,4 +96,11 @@ void _POSIX_Key_Manager_initialization(void)
> >      NULL                        /* Proxy extraction support callout */
> >  #endif
> >    );
> > +
> > +  _RBTree_Initialize_empty(
> > +    &_POSIX_Keys_Rbtree,     /* the rbtree control block */
> > +    _POSIX_Keys_Rbtree_compare_function,
> > +                            /* the rbtree compare function */
> > +    true                    /* true if each rbtree node is unique */
> > +  );
>
> this header should be pulled in from including rbtree.h...
>
Does it mean I should change all header <rtems/score/rbtree.h> to
<rtems/rbtree.h>?

>  —
> Reply to this email directly or view it on GitHub<https://github.com/RTEMS/rtems/pull/3/files#r1321833>.
>
>



-- 
Best wishes!
Zhongwei Yao
For the above three functions, the bracket should be placed on the next line (or put them inline in the header file, which I usually do with such one-liners).

And there's a spurious space after `setAntiAliasing`.
This Regexp does not seems right.
Do you want year, or is it optional?
Between `[` and `]`, much less need to be escaped and `|` matches a literal `|`.
It seems that you added two times the "date" filter ...
This should be `readonly`, as it is only ever set once.
Oh sweet! Thanks for both tips.
Why not just offer a method to get an `CGAffineTransform` from an `NSAffineTransform`?
I prefer Cocoaland, thank you ver much! Much more readable, but I will make them constants.
May be wrap this in some method like `ToBuffer`?
looks like spaces are needed?
Throw IllegalArgumentException ?
On Tue, May 22, 2012 at 12:18 PM, Robert Kern
<reply@reply.github.com>
wrote:
> I think you are misunderstanding the semantics of `from mod import ...`. It does *not* do `import mod`. In most cases, doing `from mod import ...` will not put `mod` into the namespace.

Right, I know that :-). But I was misreading the bug report -- I
thought the claim was that "from numeric import *" was failing because
"numeric" had not yet been imported. In fact, like you say,
numpy/core/__init__.py is assuming that 'numeric' will appear in its
namespace when numeric gets imported. Which is confusing. Certainly it
confused me :-).

> it gets inserted into the package's namespace, as long as no one is doing anything slightly weird. We wouldn't have to change any other instance of `from mod import ...` outside of `__init__.py` files.

Yes, if that's the actual problem. See below.

> But sometimes slightly weird things do happen. Code analysis tools will often want to import modules directly and thus miss some details like making sure that when executing the `__init__.py` code, it should be as a package and not just another module. Now it's fair to say that those tools should be fixed to be more robust, however, I think that it's also fair to explicitly import the names that we need instead of relying on a quirk of CPython's import implementation. I don't even know if it's documented well anywhere. It might not work exactly the same way on other implementations.

The semantics of Python namespaces are pretty well nailed down; we
might well want to add explicit imports to make it less confusing, but
I wouldn't say that it's undocumented that the package and __init__
share a namespace.

Furthermore, it doesn't look like there's any hooking of the import
mechanism going on. AFAICT from the traceback, the sequence of events
is:

"pymacs.py" is a script that runs as a daemon for emacs to talk to.
Emacs starts up pymacs.py.

Emacs asks pymacs.py to do something like
eval("pyeval_load_helper('ropemacs', ...)")

pyeval_load_helper calls __import__("ropemacs"). Note that this is the
standard builtin __import__ function, and AFAICT there are no import
hooks registered.

Then ropemacs imports difflib, and difflib does "from collections
import namedtuple". But, a sys.path screwup causes this to be hijacked
by another "collections.py" module in the working dir (maybe
matplotlib.collections? who knows). This interloper module then
imports numpy. So I suspect ropemacs has nothing to do with this -- it
hasn't made it past line 4 of ropemacs.__init__ before the error
occurs. And even if numpy imported correctly, the code would still
crash, because this "collections.py" probably doesn't export
namedtuple.

OTOH, I tried doing eval("__import__('numpy')") in python 2.7, and it
worked fine. So I don't know why it doesn't work when pymacs calls
__import__.

I don't have any objection to adding import calls to make numpy's code
more explicit, but this seems to be a bug in either the Python
interpreter or perhaps pymacs, and should probably be filed
appropriately.
I am going to just remove it.  I copied it from the 1.6.0-notes.rst (I thought that the habit is to mention it in every 1.x.0 release).
Will static do instead of NPY_NO_EXPORT ? It doesn't look like this is in a header.
Ah, alright. Never mind :)
Please use { and } with if statements.
I know it's not you but naming out own stuff with language native keywords is never a god idea, here `list` will override the `list` type in current scope.

Poke @tricky21
оно тут точно нужно?
value.toString().isEmpty()
how would I create string keys with 1.9 syntax?
Hi @schneems! The postfix conditional is still there on line 27. This method was added because the fix would have resulted in this logic being duplicated. There shouldn't be any change in behavior since all the same checks are being applied. All the tests pass as well.
You can't, the new syntax can only be used for symbol keys
I agree with @exviva but we can make a separete PR to change the tests.
Aha! Now I got. Thanks to explain.
Ugh. That was left from before. I'll fix those.
In the future, please refrain from reformatting code unless you're actually making a meaningful change.
Oh sorry, I got my wires crossed.
i know it's a pain to submit several pull requests, but let's not couple the fate of sure-win cleanups and more ambitious refactorings
Modulo could be replaced with `util.Random.nextInt(array.length)`, as the other user suggested.
I agree. Maybe two littles make enough of worth?
Done.

On 05.04.2012 17:12, Christopher Smith wrote:
>> @@ -1438,3 +1438,19 @@ def pretty_print(expr, **settings):
>>       print pretty(expr, **settings)
>>
>>   pprint = pretty_print
>> +
>> +def pager_print(expr, **settings):
>> +    """Prints expr using the pager,  in pretty form.
>> +
>> +    This invokes a pager command using pydoc. Lines are not warpped
>
> warpped ->  wrapped
>
> delete double space after comma in first line of docstring
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/1192/files#r650931
>
Does this work for nested lists?
this one passes without simplify
I like it there for completeness, but will remove.
I could be making some other mistake, but I think you will fare better if you stay away from the string concatenation that you are doing:

```
minterm = ['' if i else "~" for i in minterm]
rv = []
for i in range(len(minterm)):
    rv.append(minterm[i])
    rv.append(variables[i]
return '&'.join(rv)
```
is it absolutely necessary to copy a file? it seems you only need its content, so you can probably just `readLines()` and pass the lines to `procTikzString()`
In my opinion the name `$unsafe` would better describe argument's purpose. `'js:'` should not be allowed when unsafe data passed into method — `$unsafe` is true. On the other hand if passed data is absolutely safe (`$unsafe` is false, default value), `'js:'` will work as expected.
Is \mkdir Doxygen syntax or something? Maybe something to mark that we're referring to a PHP function? Other Yii code doesn't have the `\` in `@see`.
missing @since tag
Missing `CActiveRecordCollection::getIsNewRecord()`/`CModelCollection::getIsNewRecord()`.
maybe the value should be greater than '0' here. 
what's the minimum width the bar should be allowed to be ?
Title field is required [1]. In case of its lack, Kotti applies url_normalizer to object of type _none (instead of string) [2] [3].

In old normalizer implementation (plone.i18n) there was a hack for this type of incorrect cases:

   if not isinstance (text, basestring):
      text = str (text)

This trick was used to conceal errors in other areas of code. In my opinion we should avoid this kinds of code.

This change fixes broken tests. Unfortunately this is not enough, as there are other problems too. Kotti is not prepared to handle incorrect forms and there is no test cases for this kinds of situations.

[1] https://github.com/Pylons/Kotti/blob/master/kotti/views/edit/content.py#L25
[2] https://github.com/Pylons/Kotti/blob/master/kotti/views/form.py#L147
[3] http://pastebin.com/hnVngVsZ
agree, but as far as the idioms of folsom goes, it is always `get_value`.
Hm, actually, I think this is rather silly code (my inexperience with 
JNA).  This should also work:

                 planesRef.getPointer().write(0, nativeBuffers, 0, 
nativeBuffers.length);

No intermediate array of pointers needed, as nativeBuffers is already 
exactly that... sorry:)

On 8/09/2012 14:20, Mark Lee wrote:
>
> In 
> src/main/java/uk/co/caprica/vlcj/player/direct/DefaultDirectMediaPlayer.java:
>
> >                   Logger.trace("lock");
> >                   // Acquire the single permit from the semaphore to ensure that the
> >                   // memory buffer is not trashed while display() is invoked
> >                   Logger.trace("acquire");
> >                   semaphore.acquireUninterruptibly();
> >                   Logger.trace("acquired");
> >  -                plane.setValue(nativeBuffer);
> >  +
> >  +                Pointer[] planes = new Pointer[bufferFormat.getPlaneCount()];
>
> This method is invoked every frame, is it possible to optimise this to 
> not keep re-allocating this pointer array? Would it still work if this 
> was created in the setup callback? I think it's pretty cheap to 
> allocate that array these days, but every scrap of performance counts 
> for video playback?
>
> Don't submit a new patch, just interested if you think it is worth 
> optimising that or not.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/caprica/vlcj/pull/93/files#r1560560>.
>
@adrn - sorry, I meant this *class* - typo on my part - under the hood it's effectively a generator function (although @demitri and others on the list thought it would be less confusing to implement it as a class, against my recommendation :)

So to clairfy,  *Right now* it only accepts angles, just because we haven't implemented anything else.  We might later modify it to accept ``Coordinates(x=1,y=2,z=3)`` and produce a `CartesianPoint`.  It's just that currently the only coordinates are in spherical systems, because that's the most useful for celestial coordinates.
`(poll_timeout % 1000)` ?
Hey @dtreiter can you respond to this comment?
What is binary collation?
What does `of` mean?
Is the chosen css still being used?
Is it done to achieve "high performance" or to reduce the window of exposure to catastrophic loss?
can we make sure that there's a ticket for the migration of this page?
Filed https://bugzilla.mozilla.org/show_bug.cgi?id=762097
We don't check if _.has() is defined even if "_" exists. 

So- technically a "_" can exist without .has() (or even, a _.has() that doesn't do what we expect it to do :P)

Shall we move has() into the TraceKit object itself?
Oh if we want to be notified if it's no longer failing, we'll want change here as well, no?
Or if that does not work, select all columns from the tables in the set having a non-null column_default in a single query, iterate over the result set in php matching against correct $sequence, etc. This will still be faster than 35 queries.
Added more asserts in 70b366b51ac28291719666f82a019ce3d17c60ff.
Are we supposed to always align this? :)
probably a dev mode option
In this case, put actually accepts two arguments: a hash describing the user-data associated with the object, and then a second optional set of parameters:

    # The job's user data only
    queue.put({:hello => "howdy"})
    # The job's user data, plus optionally setting the priority and a delay
    queue.put({:hello => "howdy"}, :priority => 5, :delay => 60)

How would Ruby interpret it if I were to say

    queue.put(:hello => "howdy")
Is there any reason to wrap this in a ZMQError, when it isn't one?  Also, I don't think this will even work, because the signatures probably don't match.  I would just let any error here raise.
walking through groupsort_indexer() and groupby(). Cool, if tricky! This line doesn't work if the column to split on includes NAs, though. Instead of the length of the pool, it'll have to be the number of unique dv.refs. Or maybe `length(dv.pool) + (findfirst(dv.refs,0) > 0 ? 1 : 0)`?
This change is not related to demosaicing. Suggest to move it out of there (at least move it to separate comment).
In that particular case the $context['can_moderate'] is useless...I just noticed is used only inside this function so it can be removed .
The allowedTo could be just passed as argument (lazy way O:)).
Shouldn't there be some data that satisfy the type? Like `0`?
Maybe a link() function to eliminate the duplicated code? Don't have strong feelings .. it's just a couple of lines.
Doesn't this just trade on bug for another? A node might fail in between the time it is at the initiator and the time it gets to the coordinator. While it is in flight there is nothing that will remove the failed site from the list of non-coordinating sites.
Here ``instance.dirty_fields['device']`` is enough.
one space less please
As much as it pains me to say it, we're going to want this for bolth Sir Bitifred VonCask and Levelenstein D Brown.
`snapshotArch()` -> `takeSnapshot()`?
Technically, the attribue must be ``getattr``-able first and then ``seattr``-able afterwards. A read-only attribute would not be settable via passing an argument to as_view. I suppose it's most accurate to say that setting an attribute via a keyword argument to as_view will only work if hasattr(YourView, 'keyword_argument_name') returns True and that attribute is settable via ``setattr``. ``getattr`` is not a sufficient condition because of the existence of read only properties. Can you have a Python object with a write-only attribute?
`and 'DummyCache' in`
Thanks. With that, I don't actually need a separate class, I can just use the existing FileStoragePermissions and decorate the test methods. Added commit 5f69e10.
These probably have to be TO-DONE before the patch lands. At the very least, make 'em into comments so they don't show up in the rendered output and make us look unprofessional :)
I don't think this line is needed anymore. @beberlei can you confirm ?
you should keep it in the try/catch block
With this change, the `GEOSGeometry` import is no longer needed; you can remove it.
The above lines will cause a merge failure.
This is a mistake and should only happen during test runs.
If you mean 'id = static_cast<size_t>(llvm::Intrinsic::ID(1));', then I'll have to cast it in 'new IntrinsicSymbol', so it won't be shorter
You are right. Done. Thanks very much.
Change to single quotes.
The first call is probably unnecessary. It's never going to be the case that the first is false and the second is true (other than race conditions), and you're not providing different errors in the two cases. So I would just leave it off.
... wat?
Add an explicit ``return None``?
Check the indentation here
Just use `(etc/'couchdb/default.d')` here---`etc` expands to `#{HOMEBREW_PREFIX}/etc` instead of `#{prefix}/etc` unlike other shorthand paths such as `lib` and `share`.
You can still copy/paste over these URLs just as easily as before.  Most of the time, there just will not be any need to.
`head` typically comes after `url` and `md5`, not before.
`try` can be used like a case statement in addition to the catch clauses. So you can combine the case statement with `DBResult` below into the body fo the try. That will be more idiomatic.
Is this file generally useful to others and should it be included in the repo?
I left them like that in order to preserve the logical structure, if you consider they should be shorter it's ok
I don't understand this logic.  If `$state[$_` is not defined then it would seem that either the backstop or history parsing must have failed somewhere.  Just using the last value of `$radmon_state` doesn't seem right.

I think that if there is any processing anomaly that results in an `undef` or even a `time_violation == 1`, then this routine should return `undef` for the `@radmon` value.  Any downstream processing that depends on `@radmon` should then throw an appropriate warning (e.g. "Perigee bright stars not being checked, no rad zone info available").  

The upshot is that if things don't go perfectly, assume the inputs are not valid.  Continue with starcheck but don't do any checks that depend on the invalid inputs.
Never check for keywords in the IQ id !! :D Keywords in ids are optional and it's possible to omit them to reduce bandwidth usage.
IMO, this method encapsulate noting. This means it have no value to be exist. What is the bad point of containing subscriberRepository.findByCustomerNo() two times in delete()?
Oops, pull in, cleanup after?

Sent from my Windows Phone
________________________________
From: Brendan Forster
Sent: 2/05/2012 6:49 AM
To: Jake Ginnivan
Subject: Re: [pretzel] Added ability for extensions to add command line arguments (#99)

>
>  namespace Pretzel.Logic.Extensibility.Extensions
>  {
>      [Export(typeof(IContentTransform))]
> -    public class WebSequenceDiagrams : IContentTransform
> +    public class WebSequenceDiagrams : IContentTransform, IHaveCommandLineArgs

Why is this necessary if it has an empty implementation?

---
Reply to this email directly or view it on GitHub:
https://github.com/Code52/pretzel/pull/99/files#r759092
Yeah, that's what I meant, it should be `if(action == Action.ADD || action == Action.UPDATE)` I think in the case of Solr. 

Throwing an exception is fine if action is missing though, yes.
Please remove braces.
It's a max of two right now, and I can imagine using three temporarily as there might be a shard migration happening, but it's never an unbounded array, that array is very explicit and small.
I don't know! I thought there was a decent reason originally when this first started but I sure cant think of any now :)

I'll upgrade it to memcache, yes!

On Nov 27, 2012, at 8:32 PM, Dave Smith <notifications@github.com> wrote:

> In lib/acl.js:
> 
> > @@ -115,12 +115,13 @@ exports.getAppAccountCount = function(appId, callback) {
> >    });
> >  };
> >  
> > -// given an updated auth object for a profile, make sure it has all the correct
> > -// tasks in the system
> > +// our own optional in-memory cache of apps since they are used often and change rarely
> >  var APPCACHE = {};
> Why not use memcache for this?
> 
> —
> Reply to this email directly or view it on GitHub.
>
This may be fine, but you should make sure this doesn't cause a flicker when the page loads. We were using the cached viewport width because getting the element width causes a reflow and repaint and thus a flicker. It may be ok, since I think we have everything hardware-accelerated where necessary, but you should make sure by checking on some devices.
I thought I saw the @jstree-sprite-size variable being used in the thirdparty/jstree folder, but it was in the brackets/src/styles/jsTreeTheme.less file. My mistake.
This doesn't work if you type `function foo() {`, hit return, and then hit right-brace--it doesn't unindent. I think you need something like this:

```javascript
                var nonWS = lineStr.search(/\S/);
                if (nonWS < 0 || nonWS >= cursor.ch) {
```

(that worked for me, but i haven't thought through all the cases--also, maybe `=== -1` is better than `< 0`)
Minor nit: The only global directive we need here is "brackets". All the others can be removed.
Internally in Bootstrap. I copied these values from their LESS files as reference points. They are not currently used by Brackets code.
Need @param descriptions
I talked to Zef about doing a refactoring that moves the language-specific parts of ext.language to another plugin, but we're not in a real hurry to tackle that problem right now. And we don't use treehugger in the snipper_completer since it's purely textual.
This seems weird, why did it change? Test name probably needs to be updated too.
It would be nice if this was sorted by group name
Setting `_key_kwargs` to dict() out here makes all instances share the same key_kwargs. Is that intended?
Thanks for catching that. I use `plot$plot_env`, but it should be `env`.
hmm.. this is interesting. Not sure why its referencing a different commit on the submodule. I'll check this out before I merge.
everything is the same as in current fields.py except:
    class LanguageField(mongoengine.StringField):
and
    super(mongoengine.StringField, self).__init__(*args, **kwargs)

in models.py
    language = fields.LanguageField()
This would not help. We will talk about this when we will be preparing production settings.
Use `LOGIN_REDIRECT_URL` instead of `''`!
Maybe this should be a dev dependency?
Might want to update the repo URL ;)
Why not use a select here?
Don't think the last comma is necessary.
Can you place some spacing around the `j` here?
The canvas is never appended to the DOM. It's simply used to convert the data to a URI.
I think it should be `include("/Login_Form/check_login.php");`
Aww... to_screen assert Unicode type. Thinking about a `u` function to wrap library functions that return differently on 2 and 3.
odd FQCN here given that its also imported.
Hm, removeRedundantPluginsInWorkDir? The name to me suggests that this method removes plugins from the bundle, but I see where you're coming from. removePluginsAlreadyPresentInBundle?
They have not.  Since this is in a branch currently, I had thought to tackle that later down the road.
Oh sorry, From our discussion yesterday I thought you were going ahead and merging this one request to master.  Carry on!
Yeah, point taken. I was on the fence about how to do this. On the one hand, dominant style seems to encourage `new Blah()` over `createBlah()`. On the other, what you said.
Shouldn't you use ``$record['formatted']`` here instead of ``$record`` ?
Both added to katana backlog
Instead of copying this code twice what about doing an helper an pass it a callback.
Is it possible to remove this variable from `keyboard.js` and put it into `jspinyin.js`. This is Chinese-specific stuff.
So just to be clear, for this release, the next version is '0.2', right?