labofsahil/github-event-dataset-2012 huggingface dataset:
Use activity context instead of application context
Greg's Word Finder

I need Word Count Var and File Path var
CLOSE THIS

please use the other pull request from my dev branch
1.0

it seems that it works.

give me a notice
New Repo

Hi Mike,

Well my time looking after IceFilms was unexpectedly short lived :) I'm keeping an eye on what they're doing and will update the plugin if they manage to sort out their content / find another reliable file host.

Anyway, as the old saying goes, as one door closes, another opens, so I've spent the last day or so on a new plugin for Let Me Watch This / 1channel.ch. Think it's pretty much ready to go, so could you pull these changes in and make the plugin available.

Thanks.
Change github urls in instructions to match links above 

The links were changed but not the git urls in the setup steps
Remove some test elections
Use a static utility class

Instead of doing

```
new FluidPositionLock(movieClip, stage);
```

do


```
FluidPositionLocks.add(movieClip, stage);
```

which acts similarly to TweenMax's utility class.
Add examples and rename existing file to reflect changes to sta.sh api.

This adds files showing examples of how to move a sta.sh submission into a different folder and how to rename a sta.sh folder, as well as renaming the deviantart.pl file to more accurately reflect the functionality it utilitizes.
Feature/usermessages

This is for https://dev.imaginationforpeople.org/redmine/issues/746
There's still a bit of CSS to do. I'd do it but I can't get compass to work at the moment. What needs to be done:
* make avatar visible next to reply text area
* make message boxes expand vertically so that content doesn't overflows
Yes, thanks. :-)
Sorry I don't know actually. But you get the point right?
no, Tag began life as a Doctrine2 module, so a schema has always existed. I am trying to change it with this upgrade. If I use ``createSchema()`` I assume that will erase the old DB tables and replace with new, thus losing the data within. Obviously this is not what is desired :-) or am I misunderstanding again?
Actually, why does this need to be pure?
Fixed. To capture XML error I added a try-catch in the read() method of XMLReader in the processNextElement() method.
A kde vidíš BC?
my vote goes to "bug in the code" -- what good are they?
Yeah, that will work, though it'd be nice if we did it mipmap style, but that's something that can come later (i.e. we'll get a quality improvement in scaled down versions of textures)
Is it necessarily to parse io_options? I think you can let user write the complete string.
Why is `InventoryType` in the `org.bukkit.event.inventory` package? It looks more like a “general enum” so it should be placed in `org.bukkit.inventory` and most classes don't store an instance for an event.

Fabian
That's craziness.  I'm running GCC as well.  Can you point to the failed build?  Would love to see what flags it was using.
I have no idea, I just compiled them all and ran a test script.
Please Bruno confirm this should apply also to 3.0b and not only to 3.1
am I correct to assume that this change means that `globalLock.totalTime` will be `0` *unless* the global lock currently exists. I think it needs to say "amount of time time that the globalLock existed since the last time database started," but I'm confused by the actual data that this reports.
note to self:
-20 -> 20.
I think local_player is better then player_node. So i am fine with it.
You are assigning a size_t result to an int.  Better have a size_t bytes variable and assign to that.  Then make the check "if (bytes != sizeof(ent))" since that's what we wanted to read.  We probably don't care whether we got an error or EOF here -both are bad- and return KRB5_RC_IO_UNKNOWN (or a better error if you can find one) and be done.
This function seems  miscellaneous to me. Is this something you still want to use?
I comfirm I have the same problem
Have you looked at vampiric embrace?
Yeah, this occurred to me today and I knew it was the way to go. When implementing it I was amazed by how little code was required for so much functionality.
Currently, this is only useful for using the system BLAS and LAPACK on OS X. More stuff will come soon. 

Ah, I see I was using -rpath incorrectly. Will fix that too.

-viral



On 04-Mar-2012, at 2:09 AM, JeffBezanson wrote:

> I get this:
> 
> ```
> /home/jeff/src/julia2/julia/lib/libarpack.so: undefined symbol: dzasum_
> ```
> 
> I don't see what `-rpath` is doing, since if we use our own blas it is still statically linked. Also, rpath needs to be set on the executable. But, if we do that it seems it will tie it to a particular path and binary distributions won't work.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/commit/35595917556334a7282cae27fa7e7fa97a7cdf8e#commitcomment-1039256
Lo tomaremos en cuenta ;)
YUI().use('node', function (Y) {
    // Access DOM nodes.
    var oneElementById     = Y.one('#foo'),
        oneElementByName   = Y.one('body'),
        allElementsByClass = Y.all('.bar');

    // Create DOM nodes.
    var contentNode = Y.Node.create('<div>'),
        listNode    = Y.Node.create('<ul>'),
        footerNode  = Y.Node.create('<footer>');

    contentNode.setContent('<p>Node makes it easy to add content.</p>');
    listNode.insert('<li>Buy milk</li>');
    footerNode.prepend('<h2>Footer Content</h2>');

    // Manipulate DOM nodes.
    Y.all('.important').addClass('highlight');

    Y.one('#close-button').on('click', function () {
        contentNode.hide();
    });
});
The space around = rule does not a apply to named arguments.
no locked threads in this dumps. ergo - memory leak... may anyone check memory usage of mangos-worldd process (simple script)?
i think the DB is right, and thats a core bug
btw. the dummy goosip called GOSSIP_OPTION_BOT
не, по таким дампам ничего не скажешь... хотя бы -ggdb врубите, а то пусто. ну не шмогло оно с вехикла слезть при телепорте, а почему - ХЗ.
in current case my mistake - i change main virtual definition to () const, but forget change some overrides :)
semicolons if we need 'em
Thanks, this is *much* easier to read through.
https://github.com/fuel/core/pull/786
don't forget other prefixes. probably goes without saying though.
It's my first time to request.
I'm sorry if I made mistake in advance.

When one field is empty, a method for the field will be skipped now.
I want do something when the field is empty.
It looks like JS engines made some improvements on String concatenations: http://stackoverflow.com/questions/7299010/why-is-string-concatenation-faster-than-array-join
The more explicit jsperf test looks to be this one: http://jsperf.com/join-concat/2
It shows that every browsers but Safari are now faster with direct String concatenation
I've not made any further changes to the formatting yet.
Thanks. Will merge.
Le prof a dit que c'était mieux de coder avec un IDE...
If I deparse https://gist.github.com/1893513 I get this:

```bash
$ perl -MO=Deparse eval.pl
```
```perl
sub return_eval {
    use warnings;
    use strict 'refs';
    unless (return eval {
        do {
            die
        }
    }) {
        die 'does not get to here';
    }
}
use warnings;
use strict 'refs';
return_eval ;
eval.pl syntax OK
```

As you can see, the return is right in the condition, which causes the sub to exit earlier than we want.
I don't know how I missed this back when you sent it, but thanks man!
Per http://developer.mapquest.com/web/products/open/map the the aerial imagery for Japan is only available for "tracing" (whatever that means; I think for making derivative works freehand) so you might want to mention that in your source?
En vez de esto conviene directamente poner algo como *~ y nos ahorramos todos los demás problemas del git add .
hmm yeah good point. The disabled -> disabled mapping in the colours
should have given it away :)

On 01/02/2012, at 10:53 PM, techtangents
<reply@reply.github.com>
wrote:

> This can't work, surely. "newState" is map of job name -> color. newState.name will be the color of the job called "name".
>
> The way to exclude disabled jobs would be to omit them in the "massage" function.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/TheSpyder/fewtile/commit/fde4e8d5e26cd889592193bebb89bb6c8534b8ba#commitcomment-922333
But now this will fail on linux...
Not really, no.
Because that would result in the bug you mentioned.
We should do that when we actually call Navigate, that is the action, not
in the response to the action.

On Thu, Jan 26, 2012 at 2:30 PM, Fitzchak Yitzchaki <
reply@reply.github.com
> wrote:

> In the Frame control we handle any internal navigation, and IMO this is
> the place to decide whether to forward the navigation to a new tab.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/fitzchak1/ravendb/commit/c6e12fec50d2f9d4f2801a60229cd9b173f0b35a#commitcomment-903980
>
i think this not work in linux...in linux the separator of dir and file "\tmp\xxx"...only in windows is "C:/temp/"...
i guess path.basename(p, [ext]) is the solution...
some like this:
packageName = path.basename(packageFile);
packageName = packageName.substring(0,packageName.indexOf("."));
i need test this....
Ah, nice catch though your fix is a little off.  \n doesn't interpolate in single quoted strings.
Ok, tested it on the laptop, looks fine there, except my FPS drops down to 46 on stage1.
change this to `if device_form and device_form.is_valid()`
According to http://docs.python.org/whatsnew/2.4.html#pep-328-multi-line-imports :

"The PEP also proposes that all import statements be absolute imports, with a leading . character to indicate a relative import. This part of the PEP was not implemented for Python 2.4, but was completed for Python 2.5."
The error when len(s_ascii)%4 == 1 might be confusing. I suggest:

length = len(s_ascii)
assert length%4 != 1, "bad length %d" % (length,)
s_ascii += "="*{0:0, 2:2, 3:1}[length%4]
solid commit, 5 stars
`$vars['node']` will be set when preprocessing a content field even though it is not shown as part of the node view.

I've tried to add a few comments to where you started.

```php
if ($vars['node_type'] == 'wiki_page') {
  // So we only get here if we are preprocessing a field for a wiki page node
  
  $vars['wiki_gallery'] = (arg(2) == 'wiki_gallery') !== FALSE; //wiki_gallery is true for node/nid/wiki_gallery
  
  $vars['wiki_page'] = FALSE;
    if ((arg(0) == 'node') && is_int(arg(1)) && !arg(2)) {
      // path is node/nid
      $node = node_load(arg(1));
      $vars['wiki_page'] = $node->type == 'wiki_page'; // Content field is being shown on node/nid where the node is a wiki_page.
    }
)
```

The effect of the control statement should be:

Wiki gallery page `node\nid\wiki_gallery`: `$vars['wiki_gallery'] = true; $vars['wiki_page] = false;`
Wiki page `node\nid`: `$vars['wiki_page] = true; $vars['wiki_gallery'] = false;`
All other pages: `$vars['wiki_gallery'] = $vars['wiki_page] = false;`
@svizion sorry this isn't the correct branch.  I will be removing this branch and pushing a new one when I get the merge complete.
Then call it  `modifierTypes` or even just `modifiers`? You again slipped in an `attribute` prefix, but since this is already the `AttributeInfo` class, there is no information added by prefixing the member with `attribute`. You also need to keep it in plural form since it's a vector with multiple of them in it.
This should more closely mimic the DSL in Faces, especially the behaviour of trimming left-aligned whitespace used for indentation - but not layout - to make for nicer API.
;) Great commit!! JS the nicer way
It installs fine for me and npm definitely accepts this form. Are you using the latest version of npm?
@cedricbonhomme Thanks, filing a separate issue #438 for that.
1.9.3では動きませんでしたが、1.8.7だと動いた（rake specもspecからの呼び出しも）ので
$LOAD_PATHにカレントディレクトリが含まれなくなったためではないでしょうか？
実際にはトップディレクトリにファイルはあると思います
The icons are drawn by a friend of mine. I use them on many "medieval" -themed variants.
Though I agree that this and the `object.$methods` might not be the most pretty solution.
En esta parte asumi que los tipos estructurados se llaman por su propio nombre y no con "box pelicula". Falta por decidir como sera lo de los alias.
I suppose it's really an open issue, but the community wants a normalization algorithm that's not tied to JSON-LD.

We really needed to remove old normalization from this spec, and I wanted to replace it with something that basically just referenced the external normalization algorithm. Really, the result will be whatever it returns.
Marshal.dump and load don't work 100% of the time. According to ruby doc: Some objects cannot be dumped: if the objects to be dumped include bindings, procedure or method objects, instances of class IO, or singleton objects, a TypeError will be raised.

So... This is not 100% safe
ty..u2
I created issue #16 with some information on this failing test.

On 19 January 2012 14:33, Andreas Dreyfert <
reply@reply.github.com
> wrote:

> Hi Michael
>
> In test_analysis_symbolic.py you add "@unittest.skip("Skip failing test")"
> for test "test_symbolic_noise_kt_over_C" so
> I assume it fails for you. What version of Sympy do you have installed?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/michaelnt/pycircuit/commit/bdb396fbd9b2b329176e47c05d77940de7efe8c1#commitcomment-882250
>
Not sure what this does.  Should we do it for the Admin area too?
Have a look at xp-framework/rfc#236 - really awesome would be to define `Diagnostics::dump()` as extension method and then be able to write code such as `$return.dump();`. Unfortunately, this only works in XP language...
Good catch! Would you want to submit a pull-request for this, and get your name in the changelog? Else, one of us will jump on it.
lastmap is assumed to be non-null in this branch
Hm... so the webserver binds to another port than provided for the clients, so internat can be proxified... Basicaly a nice feature... However, we should in this case not change the console logs and the assetsStatus check, because it should check wether the sever is externaly accessable. Any arguments against that?
Doesn't this need an end?
Testing issues:
1. I was able to install CouchDB rather easily on ubuntu (and look at it via the ip_address:5984/_utils/ screen)
2. After saving the globals, it did create the database and view successully
3. When try to upload a document, it does not work. The log can not be created guessing some permission or trying to make a directory without creating the sub-directories. If I manually create the directories the log does work and simply states it was not successful.
4. It then copies it to the hard drive, but unable to see the docs, because the 'url' setting in the entry is not compliant (since you just use filename when using a couchdb only, rather than when harddisk is an option also). I actually think that invisibly copying the file to the hardrive as a "backup" method when only CouchDB is requested is a big flaw here. If the user wants only CouchDB and it does not work, then a error needs to be thrown that it is not working and the document should not be saved. The same strategy should be taken for if saving to CouchDB and the hard drive (if either one of these mechanisms fails, then an error should be thrown and the document should not be saved anywhere; otherwise the files will not bee the same). One could actually pose the question as to whether it makes sense to offer saving to both mechanisms because of this potential issue, so, I'd suggest either just save it to the hard drive or to CouchDB to avoid many confusing issues.
I care :)
Renamed to width, height, screenWidth, screenHeight.
check

On Fri, Feb 3, 2012 at 2:25 PM, Matt Zukowski <
reply@reply.github.com
> wrote:

> <br />, not <br>
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/colinmccann/EvoRoom3/commit/5552e8f5af5193fb8a050bacbf76cf63d4d203ff#commitcomment-931497
>



-- 
Colin
Yay for more granular data
This piece of code is not related to `AssociationKey` but rather live in your Voldemort specific dialect or a helper class somewhere.
Lol - good catch :)
The main reason was there was no spacing between the icon and text. In order to solve that, I needed to put a space between them (that will lead to an underscore effect of the link) or add special CSS rules to manage the margins. 

We have in XWiki, groups that contain (text + icon) where text links to an address and the icon links somewhere else (mostly RSS links) - in "Blog Categories" or "Quick Links" panels for example. 
But in this case we have just an address (so no need for an img). This way is kind of symmetric with the search results: where we have the page icon and the page link and you can click just on the text link and not on the page icon.

I don't know why here the RSS icon is put as an img and not as a background-image (since IMO the icon is just a mark for the rss link and not an action button on its own).
Same thing for single assignement: horizontal alignement will lead to read down the left values while ignoring the right keys. And of course it forces the developer to maintain the alignement.

    $an_arry = [
        'foo' => 'bar',
        'subarray' => [
            'baz' => 'dib',
            'zim' => 'gir',
        ],
        'irk' => 'doom',
    ];
Is that final dot intended to match any-character, or a literal dot in the yum output?
please move all add_filter() calls to after functions. also need no empty lines between the function comment block and the function declaration. this is required for the doc generator to work.
Разные языки дают разную максимальную длину текста.
http://www.ruby-doc.org/stdlib-1.9.3/libdoc/set/rdoc/Set.html bro
lol nice - gotta talk with u - gtalk?
This shouldn't be a different message.
Feel free to continue with `--strict` if you can implement it without major changes to Main
Branch with solutions working in Lion, not using C++x0 code
Thanks for the catch, johnduhart. Fixed in trunk with a settings regime similar to MediaWiki. Tomorrow we'll figure out how to expire the SID and token.
It looks like `rbenv` is installed to `/usr/local/bin/rbenv`. Can I write a patch for you here?
The source of this compile error is f09b5a6 not this commit.
Would you add some more tests including add_context_index with :wordlist option specified?
Is this a hack?
The RssFuturePostsKey should be private!
Could be "usando Sphinx, o gerador de documentação Python" or "usando o gerador de documentação Python Sphinx"
i refactored that yesterday to inject a repository rather than the document manager. (it need not be a doctrine repository, if you have orm you could still have one provider that multiplexes to the right repositories)

see https://github.com/symfony-cmf/ChainRoutingBundle/commit/12515f82b02c4a1ef230b05133ec89e03c1af2e0
if the prodution was in negative   say  -1000
it stil calculate how many hours to  be full (warehouse)
.DS_Store files in the root and img dirs.
I agree, which is why I limited it so those elements are not allowed to overflow. Content within any of those elements is allowed to scroll.

That said maybe some finessing could be done to the language to allow body scrolling. Some development styles may constrain the (ex http://shittyday.simurai.com/)
Since this commit the druntime tests have failed on all platforms.
I think this is tagged for the wrong issue; issue #582 relates to elevator routing.
Ah, nice one, thanks!
Will be properly implemented later.
Please add meeting ids on log...this way we can grep on the meeting id and display the complete flow.
Oops! Thanks for that!
I understand your concern but I have gotten a situation when 'notify' and 'success' URLs were called just several milliseconds apart so I was not able to set any flag between them. Also as the 'notify' process is running at the background I  don't think that 10 seconds delay will hurt anyone.
yes, I forgot to.
I definitely agree.  But there are odd scenarios where ST2 runs plugins slow because it is also doing other things, particularly I have noticed this during "on_load" on first start up and on "on_activated" when you have been out of ST2 for a while.  Something that would usually take no time to complete can exceed the time limit during these two scenarios.

There was heavy optimization on this before hand when I originally forked it (not to say it is impossible to do more).  The ability to cut down search distance is also in the settings file to keep things snappy.  If someone really wants to find two brackets very far away, it is going to take a bit though, so it must happen outside of these events.  Also, a lot of multi-select regions (which I do not restrict) will linearly add to the time for each added region.  I would be more concerned about the "alert" if using the plugin was also impacting performance, but I have been quick to reduce bothersome code when issues have been brought to my attention.

With that said, I am fully open to input on ways to optimize further and welcome pull requests :).  If you find the plugin is impacting performance, please do let me know.

By the way, big fan of your sidebar plugin.
??? no... this file is the legacy class, and its a java file, and there is no way to use board config flags in java :/
@infiro Note that before you do a pull with this, you'll want to move this file somewhere else, pull, then bring it back, in order to not lose your settings.
pourquoi 3 fichiers css différents?

Impossible de tout mettre dans un unique fichier css pour le bookmarklet? Je veux bien que charger le moins de css possible est bien, mais c'est relou cette duplication de fichiers de styles
I have already updated these old references.
@hafriedlander  Has get() been renamed to forClass(), or is that an oversight (phpdoc vs. implementation)?
why not `\tableofcontents`?
i dig the badge. looks great. the notifications were a little bothersome.
Exciting! I had been trying to get bootmenu to work with CM6, but for I could never get graphics to work after selecting normal boot. I really like the bootmenu approach to factor out a lot of stuff from the ROM (overclocking profiles etc).
@neillturner - I think you are spot on with required parameters vs an options hash. I think in this particular case the method definition is correct also. I think the change we need/want is on line 9, which should be changed from `'metadata' => {}` to something like `metadata => options[:metadata] || {}`. Otherwise any metadata that might be passed in would be disregarded, does that help clarify?
Glad to hear that works for you.  I'll close this then.
Not sure - this was there beforehand but I didn't notice anything odd in my testing (ie: the move handler was being called on touchmove). It may be outdated, but I'll leave the comment in there so if a bug report comes in at some stage, I know where to look.
"widgetView" should be paletteView
I think this should be a "mapdes@imperial" like address.  Rather than Graham.  This is not to disparage Graham.
So when someone searches for it, someone actually tackles that question.
Just being pessimistic ;). Read the barrier code in the TestConductor and as far as I can see it should be fine to reuse the names.
Are you sure about this one? I have tried some little things in the sample project (with the change above), and it never looked for 2.9 anywhere, always 2.9.1.
I'm not reading your mail...what do you mean no op?
Oh, that's even more wicked awesome.
this is a rather circumspect way of saying that the local node’s weight is replaced by the (rounded down) average, no?
I like to allow enough time to insure that all the data between the nodes is not going to do the wrong thing but I will reduce the time.
And pass _that_ ClassLoader into the ReflectiveDynamicAccess's constructor?
Why is this Java?
we could add a "scroll_distance" like in scroll view, that lets you scroll even if your over a button like @revolunet mentioned.

The min_move here is value that determines, how much the slide has to be dragged to the side in order for it to advance to the next slide on release (or if less, ease back so the current slide is in center again)...can we think of a better name?
easing_threshold ?
drag_threshold?
*_edge ?
There are a bunch of tab characters here instead of spaces which ruins the formatting of the file. Would you be able to fix this?
When I learned grammar (latin and greek!) we found out that there were
prepositions of <state> and prepositions of <motion>.

<At> is a preposition of <state>, (e.g. 'who want to position dMRI analysis
at the next level') whereas <to> is a preposition of <motion> (e.g. 'who
want to take dMRI analysis to the next level').

Ian


On 13 December 2012 16:34, Eleftherios Garyfallidis <
notifications@github.com> wrote:

> In doc/devel/intro.rst:
>
> > @@ -2,14 +2,13 @@
> >   Introduction
> >  ==============
> >
> > -dipy_ is in development mode at the moment.
> > -
> > -Please do join in the fun.
> > +Dipy_ is always seeking courageous scientists who want to take dMRI
> > +analysis at the next level. If you share the same vision and you are willing to share your code please let us know we will be happy to help.
>
> Google thinks otherwise "at the next level" 422M, "to the next level" 70M.
> Somehow, to seems nicer. This is what I initially wrote and then changed it
> to at. No worries, will ask Ian or Matthew who are native english speakers.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nipy/dipy/pull/103/files#r2409455>.
>
>
I believe it's "Descoteaux" (with an 's'). Wouldn't want to mispel E's new bosses name...
I think the large warning in the head should be enough. If something is wrong, CKAN won't start and complain. If you want, I can add an additional note box.
I like this. Helps make it clear to users outside of this module without having to resort to a single method called something like bind_or_unbind. Also gives an easy way to change the unbind implementation in the future.
This comment doesn't look to be valid anymore. The only imports after it are unused.
Having the section and the default values in the config would eliminate these nasty conditional blocks.
Ya, fully agree that looking through the actual printed text would suck as an approach.

The write command in okaara can accept an optional tag parameter. If the prompt is set to record tags, it will be kept in a list. The idea is that instead of comparing line by line, you can mark important line with a tag to indicate its usage. Pulp's render_* commands all pass in tags.

For instance, something like:

p.write('Name: Jay', tag='name')
p.write('Chat Nick: jdob', tag='chat')

Then in the unit test, instead of trying to match the exact "Name: Jay" string I'd match that the first tag is "name".

I've used that to make sure that the render_failure_message calls were made since it tags as it writes.
Some of these were resolved differently, and may be possible to remove them.  Let's address this later when we are cleaning out cruft and migrating all the extra fields and tables away.
value...c'est un peu trop generique comme nom...?!
General pet peeve: please use a global [`excludesfile`][1] for your IDE's litter so everyone's projects don't need to know about your personal choice of tools.

[1]: https://help.github.com/articles/ignoring-files
Retry should have a maximum then either move on or abort I think.
@Gravatronics  Which whitespace do you mean?
Yes if the child dies and we get the signal and then read the message from socket.
not a stupid question, i should have probably documented more.

for met to use these in a map, i had to define a comparator. i don't
necessarily care how they get compared, but one must be defined.

this typo is from a copy/replace from product to commodity. nice catch.



2012/6/20 Katy Huff <
reply@reply.github.com
>

> > +  /**
> > +     constructor
> > +     @param name the name of the commodity
> > +  */
> > + Commodity(std::string name) : name_(name) {};
> > +
> > +  /// the commodity's name
> > +  std::string name() const {return name_;}
> > +
> > + private:
> > +  /// the name of the commodity
> > +  std::string name_;
> > +};
> > +
> > +/**
> > +   a comparitor so that commoditys may be used in maps
>
> commodities (spelling). Also, what are the results of the comparison? What
> does it mean for a commodity name to be less than another commodity name?
> (is this a stupid question?)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cyclus/core/pull/194/files#r1020814
>
Cleaned up stray whitespace on this line as well as 611, 616, and 621.  (Sorry to be pedantic... it shows up red because of how I have vim configured, so I try to get rid of it.)
Shouldn't that be configured?
simply call ``twig_date_converter``. It already handles cloning the DateTime objects
This looks odd… wouldn't this log get called every time, whether the block completes or not?
Actually, this should be a non-issue, because both `resolve()` and `reject()` clear the subscriber lists, so both methods become no-ops.  Right, moving on.
This isn't 2.4 compatible. I'll fix it up.
Shouldn't this be a list of defaults?
and remove the completely unnecessary `0 or`…

Fixing that code would involve nuking it from orbit.
To simplify `gxp.plugins.WMSSource`, add

    featureinfo = featureinfo || {};

here.
Ah yes, of course. I’ve gotten so used to invisible-unless-activated scrollbars in 10.7 that I forgot that would happen.

I guess another solution would be responsive height-based CSS that stacks up the menu next to the author info. For example:

http://adrop.in/B6h

It requires a double-width sidebar, of course, so it affects the responsive width style changes, but it does deal with short but wide-ish windows. Maybe sleep will offer a better solution.
not sure but maybe it make sense to use `multiple_of` method from `ActiveSupport`? https://github.com/rails/rails/blob/master/activesupport/lib/active_support/core_ext/integer/multiple.rb
Do we need the `return`?
new CHANGELOG entries go to the top of the file.
there should be another test with `'QUERY_STRING' => "%C0%8Asea%C0%8Arch%C0%8A=foobar"`
Which leaves me wondering why the tests passed.
I could emit a warning, but the only way `pre == NoType` I can think of would be for ill-typed code, so they'll get an error about that anyway.
Pauli Virtanen a écrit :
> I think this doesn't work --- if `first_only` is True, the cached function value is never used.

You're right. In fact, in the original version (tested in #130) the
second and third elif clauses were swapped and it works that way. I'll
look into this again...
Is it possible to also show how `remove_idents` works?  That is, something like
```
>>> expr = Add(1, 2, 0, evaluate=False)
>>> remove_idents(expr)
1 + 2
```
I'm asking because I cannot get `Add(1, 2, 0, evaluate=False)` to show `1 + 2 + 0`; the zero gets dropped anyway.  If there is no way to keep the zero, maybe you could change the example and use `MatAdd` and `MatMul` instead?  (I suppose one can make them keep the corresponding unities by specifying something like `evaluate=False`?)
No.  This is showing what should be replacing the NotImplementedError.
This should be `det_LU`
"is needed"
The tensor symmetry might be specified using Young tableaux instead of generators; this class should deal with that
in the future.
Edgar Friendly
<reply@reply.github.com>
writes:

> isn't the grouping wrong for the ? in `\\(([^)]*)\\)?` i.e. doesn't
> the ? just apply to the final ), and not the whole parenthesized
> section?
>

No, '?' postfix operator should work with groups.
You are not using value of `$end` anywhere. Should be in the `substr()` when given, right?
Naming it just a `password` makes it a bit confusing, I think.
Does spork do you wany good if there's nothing in the prefork block?
what about a default value of null if the column is nullable and has no default?
Sadly, you're right I think. I'll take a slightly different approach but I still think that passing in the delta is the way to go, it's just a question of how to invoke the frame function that shares state. We'll see tho.
Do we use comma elsewhere for things list this? Wondering of a colon would feel more natural and classpathy
I'd imagine this to clash with the existing `stream` helper.
yuck - how could I detect this (special) case within my serialization so I can at least throw a NotImplementedError?
What else? :) There're no abort() calls in this function. Also, that's how it was before my patch.
@bnoordhuis Thanks, I got the point. You are right. This is more robust error handling. But I don't think UV_EINVAL is a proper error code but UV_UNKNOWN is in this case. 
I fixed it as below. How about it?
````
--- a/src/unix/netbsd.c
+++ b/src/unix/netbsd.c
@@ -173,6 +173,7 @@ uv_err_t uv_resident_set_memory(size_t* rss) {
   page_size = getpagesize();
   pid = getpid();

+  errno = 0;
   kd = kvm_open(NULL, NULL, NULL, KVM_NO_FILES, "kvm_open");

   if (kd == NULL) goto error;
@@ -188,7 +189,7 @@ uv_err_t uv_resident_set_memory(size_t* rss) {

 error:
   if (kd) kvm_close(kd);
-  return uv__new_sys_error(errno);
+  return uv__new_sys_error(errno ? errno : -1);
 }
````
JQuery is used in pretty much every page right?  Should we just include in the application.js file so that we always get it (the fewer overall JS files each page loads, the better right?)
After reviewing the JOOQ API, I think the `where` builder can accept a `Collection<Condition>`, which might be neater than the pattern used here. Having said that, I'd like to leave this as is for now, and potentially try that out on a different patch. We can always evolve our usage of the API as we go on.
invert first sentence
needs to be in the commands.txt file
this is where django admin docs said to put it.
haha, it is cool and all. I just think it is kind of tightly coupling the front end code to the backend structure. But if that is the way tasty pie is recommending, so be it. My experience with tasty pie has been with pretty basic queries so far.

You get to maintain it, your call :-)
Ah yes, color applies to groups directly. :)
This should probably at least mention that it's a bitfield which decides which bbcodes are defined in the template. Just "bitfield" is a bit generic.
I split up the two cron tasks and shortened this logic so it is no longer need to be split across lines.
Yes, this comment is outdated and will need to go.
The signature conditional directly above that this was copied from was wrong and has been removed. The auth requirement should be in modules table.
Camel case can also mean the version with the first letter capital. Maybe mixed case? Or clarify that the first letter is not capital.
Can you add some tests for timesync for forward and backwards adjustments?
This HttpResponseError doesn't look to ever be used. It is used in luvit-rackspace-monitoring-client though. Delete it?
please add 'ach-chall-def-big' to get_modifier
Only derived classes include OpenGLWindow, currently its Renderer class. Also OpenGLWindow.h has some types from Xlib.h, Xrandr.h.
As I've understood from here: http://sourceforge.net/mailarchive/forum.php?thread_name=4CA436C8.8050801%40astro.rug.nl&forum_name=doxygen-users

It does strips doxygen comments from \include and \code pieces.
I prefer to keep it cleaner.  If `store.config` is missing that's quite exceptional, and we should fail early.
If i look through the code there is never the assets/coffescripts folder. Only the assets/javascripts folder
Initialization of function-level static variables is not thread-safe. It is possible that variable will be initialized several times, or second thread can get uninitialized mutex. See http://homepage.ntlworld.com/jonathan.deboynepollard/FGA/function-scope-static-thread-safety-problems.html for some details.
Why not use heredocs?
Need to merge with upstream.
I don't want to touch this stuff for now. Later, if someone has a good benchmark, we could try seeing if changing the C code to have the strides hardcoded rather than declared as ints makes it faster or slower. (We can't always hardcode all of them because some of them are passed into the kernel)
pep8 calls for 2 spaces before this line, not 3.  I suggest running pep8 on this file again, it seems there are still at least a few violations left.
Как-то тестировать сборку таких файлов всё таки надо. Сейчас получился просто -1 тест.
ah, there's a "git revert" command that can be used to revert a series of commits.   I'll try that on my own tree.
@jdevera The condition should be `!empty(a:rtp)`, it's a mistake, sorry. I'll fix it.
Good catch @glennpratt . I also think it would be better to use uniqid instead of microtime.
It's more like

    helper = do
       a <- f
       b <- g
       c <- h a b

I agree that it's not obviously an improvement, but I don't really care since this code is going to be removed anyway.
I thought they were required as some sort of burnt offering to gzip, in order for it to smite a few more bytes.
<blockquote>
name.indexOf("background") === 0
</blockquote>
and it saves two bytes... done
Can't hardcode the compiler like this.

```rb
system ENV.cc, "fsbext.c", "-o", "fsbext"
```
Just `safe_system '7zr', 'x', @tarball_path` will save an extra fork-exec.
apply_publish and apply_consume are about whether a message is entering a particular worker or not and not about the route a message is being published or received on. The idea is that messages being consumed go through the middleware in the normal order and messages being published go through middleware in the reverse order (so that multiple middleware are consistently layered).
return early, reduces indentation

```javascript
if (!routerUtils.isValid(pathname)) {
    next(new RainError('The url is not valid!', RainError.ERROR_HTTP, 404));
    return;
}

// rest of the code here
```
Bitte beachte auch, dass ich die Exception page etwas verändert hatte, zum Beispiel den Klassennamen hinzugefügt. Also am besten nicht einfach meine Version in rex_error_handler durch deine aus rex_response ersetzen..
Good find of the unused imports, but could you just delete the lines please?
Capitalize first word, period at end.
Because the plugin stores your password in plaintext, added this to prevent accidental committing of this by anyone else forking the bot
In general these should be handled by the jsontoxml parser we're using, but it's shitty and buggy, and not taking care of &'s.. Doing the same for any of the other built-in entities (http://en.wikipedia.org/wiki/List_of_XML_and_HTML_character_entity_references#Predefined_entities_in_XML) could severely mess up the output, since this is being done on the actual XML document.. For example, replacing all < with &lt; would completely ruin the doc..
@dsaff 
pls see my latest commit (changed password for deployment to ftp://upload.sourceforge.net/incoming)
Yes, that's what we meant. Do you think `is(equalTo(...))` reads better?
No need to create a variable here — just have it return in the conditional (as with `getDuration`).
Sure, as long as it's a fairly simple change -- I think we should try to get this initial version of the colorpicker landed fairly soon so others can start testing it.
"if" should be "else if".
There was a recent refactoring that created LESS variables for all the z-indices we use so we don't have to make up arbitrary ones (they're calculated relative to each other, so they're more semantic than just arbitrary values). You should make one for this as well. See brackets_variables.less.
Nit: I think that aligning the equal signs makes lists such as this easier to grok at a glance.
Also $1 isn't my favourite
:+1: put this small change in a separate pull request having that in the .gitignore file would be good. Seperating out pull requests by feature also will make sure simple small changes like this don't get rejected if the larger pull request does not get accepted.
I think we could just edit the params hash in place. There's a delete method on hashes that may be of use -- this should only be 2 lines of code I think.
Maybe `.count()` is better to use if you want only to check for existence?
You can also hijack all other "useful" methods from Django User object in this way. ;-)
I am not sure why you repeat this again?
I do this on purpose, actually. It seemed like it was the only way to revert back to whatever it was before style was applied directly to the element, which means class-scope properties will work again.
I'm not sure we can rely on `classList` working in the absence of Butter for IE < v10: https://developer.mozilla.org/en/DOM/element.classList

Do we have a solution to fix that?
For consistency with kate's original design here, can you just split this into `mediaChanging` and `mediaReady`?
fix comment plz :)
`} else if` pretty please
You can completely skip passing `zoom` into this function now if you just call `scrubber.update` in the event listener and `update` functions below.
Same as  https://github.com/mozilla/butter/pull/1056/files#L1R261
Remove whitespace, please.
No. The Draggable hotspot is always there for the map.
Yeah, I can do that. Was thinking about doing that when writing the code, but felt lazy... :)
"specific object"
misuse of `parseInt` -- `this.width * 2` is not a string
How does this affect using the deprecated plugin?
Why was this changed to Data?
you have this going in quite a few places :) so going to say similar to 232 again for all these lines
Any reason not to use the `.one-line-ellipsis` class on these elements to achieve cross-browser support?
Why 'and' versus '&&' here?
didn't like my naming of the file huh? I'm insulted ;) how about camelCase?
Ok I get the context thing now, but.. I think this should be called `exception`, and then we should probably document this and make it a convention that logging an exception in monolog should ideally contain the exception object in the context array's "exception" key. That way we can adapt frameworks and such using it so that they do this, and we could detect that it's an exception and do some more stuff in some handlers. It would also be useful for the IntrospectionProcessor that could use the exception backtrace if it's available instead of crazy hackery like now.
Collapse into DAO.
I suspect UX will ended up with some panel transition for the music app. Instead of reusing `#title` for all the panel/modes, you should consider split it into `<div class="panel" id="...">`. It would result better CSS too :)

(this issue is non-blocking, can be done in a follow up)
|message| isn't declared.  This is dead and broken code.
Please catch more specific exception.
Personally, I've always found this `Coordinates` class confusing...it suggests to me something far more general than it actually is. To me, it would be the base class for all `Coordinate`-like objects, not a magic factory class.
can we please remove the commented out implementation?
That would make it impossible to first set a default and later addFormat(), imo its up to the developer to use a sane value, but if you think it should do the check i can add it
Why? Where did the injection go?
I was pretty sure this was used by one of the content providers.
Why trigger this event instead of calling ui.showHelp()?
Don't perform this query here, but add a callback-method to the DI-registration to execute (which executes this query) upon instantiation of this object
I wouldn't worry about trying to figure out branching in TFS 2008 - it's a different model, based on some [notes I gathered](https://gist.github.com/1473849) way back in the day
You can't return in a void function, you need to RETURN in that case (or make it return u32 or int, which should be perfectly fine for these functions as they don't block.

Also, again, please set your editor to use real tabs :P
The ElementTree comment was there in the gcalcli hosted on Google Code.  I removed that comment the other day when moving to github after some cleanup.  I assume @markgraf merged it on accident as the original patch was against the repo on Google Code.  No matter.  Going through this change now and merging soon.
Isn't `set()` now redundant here? It's already a set. I think these two lines can just be `words = sorted(words)`.
This function  (now named `oid_for_tree_path`) still needs some caring. You're waiting for Russell pushing `git_tree_resolve_path`?
I think this is much more readable now.
The `!remote_name` and `!merge_name` aren't needed, as the config API doesn't return NULL anymore. The others should be fine.
No, sorry. I guess the commonDialog holds a block on the main UI thread :-( That would go some way to explaining why the Firefox devs have been so keen to move away from relying on it over the past few years.

I don't think we can move KF.js onto a different thread either - it's so reliant on the networking and main event loop thread that even if we could somehow get it to function correctly, I'd expect random crashes at a later date.

There's a chance that the Electrolysis development project might lead to some improvements here IF it doesn't break KeeFox entirely and IF it ever actually happens (I've not heard anything about it for ages and the plaudits Firefox is currently getting for reduced memory usage vs the Chrome-esq multi-process approach might be going some way to reducing the priority of that work)... and IF Thunderbird actually implements any of the Electrolysis changes (Mozilla's recent announcement of a move to a "maintenance and security patching" mode for Thunderbird might place such a core modification out of scope). A lot of IFs :-(

How does Thunderbird behave when you cancel the login prompt? If it's like most HTTP Auth requests, it might automatically re-produce the commonDialog a couple of times so maybe a workaround would be to force the commonDialog to close after the user clicks a "refresh" button?

For Firefox that's probably not much better than just pressing refresh on the page but maybe the workflow in Thunderbird would mean it's slightly less inconvenient for the user?
media() tag
You should also check other modifiers. `ctrlKey`, `shiftKey` and `metKey`.
I like this idea. I'll switch to using that.
This scheme is unclear and confusing. I would expect something more along the lines of `/products/default_version?products=firefox,fennec` or `/products/:products/default_version`. Or better yet, I would expect the products API to return the default version as part of the json when it's hit without providing a version i.e. `/producs/Firefox/`
`open` will return a line generator so you can just:

```python
for line in open('sql/roles.sql'):
   ...
```
And right now expire needs to be called manually. But it's probably a good idea to handle that through a timer. Another problem is I don't want to add periodic timers, because they prevent the loop from exiting when there is no work left to be done. So maybe we need to do implicit expiry on lookup instead.
I didn't find conclusive documentation on how that's handled - if I don't return anything it'll go to the next error handler, and if I return false it'll fall back to the php error handler?
Unrelated to the PR, but I just thought to make this in sync with `getInitTime()`.
Yep sure i revert it as you wish
> Any returned value is ignored.

http://www.php.net/manual/en/iterator.next.php
Is it really worth it ?
As per the finnish and swedish weekdays, the days should not be capitalized in Norwegian, even when abbreviated. The correct string is
```
weekdaysMin : "sø_ma_ti_on_to_fr_lø".split("_"),
```

@ulmus phrased it so well, I thought I would steal it directly ;-)
maybe some others will chime in here, but I would personally rather see these named instead of introducing a bunch more chars to remember. Maybe `<% extend template.html %>` and `<% block name %>`
This is incorrect since graylog_web doesn't have anything to do with the application being deployed - if 
anything, graylog_web should be forced to be in its production env since we are a client of its functionality.
Can we remove the "nil false" from access token and set it to nil instead of 'EXPIRED'? That would eliminate the magic string.
as mentioned on IRC, I'll just abuse this PR to keep track of stuff I noticed around the code.
Во-первых в конструкторе её надо инициализировать. Странно что оно у тебя работает при обявлении в заголовках и в реализации
Please see the coding style guide https://sites.google.com/site/openimageio/coding-style-guide
Three blank lines between functions.
Opening brace for functions is on the next line.