labofsahil/github-event-dataset-2012 huggingface dataset:
try (User.single ~ Permission.single *).map(_.groupBy(_._1))

On Thu, Jan 5, 2012 at 10:28 PM, recalcitrant <
reply@reply.github.com
> wrote:

> I have 2 tables user and permission. A user may have more than one
> permission so I came up with this:
>
> select * from user u
> join permission p on u.permission_id = p.id
>
> Now I want to retreive a User + List[Permission]
>
> with the user being of type:
>
> case class User(id: Pk[Long], Seq[Permission])
>
> User.single ~ ( Permission.single * )
>
> The compiler throws this error:
> type mismatch; found : anorm.ResultSetParser[List[models.auth.Permission]]
> required: anorm.RowParser[?]
>
> This had worked before the refactoring, now it does not anymore.
>
> Please also see my post here:
>
> http://groups.google.com/group/play-framework/browse_thread/thread/ca758fa370a5fc1f
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/playframework/Play20/commit/f35fd305d98038ab0fe3f8076368ee7f8d90015b#commitcomment-841760
>



-- 
www.sadekdrobi.com
ʎdoɹʇuǝ
ich hab durch aus ne flat, aber t-offline darf bei uns nicht mehr freischalten. hätte sonst dsl 32k aber ... nunja niedersachsen halt. bekommt nie wat auffe reihe ^^
El portal, y la cuenta de twitter, porque estaba viendo que la url que se encuentra en planeta Linux me manda de nuevo a planeta linux:

portal: http://logicalbricks.com
twitter: @LogicalBricks 

Gracias por la paciencia damog.
Sorry, you are wrong here ,-)

First, the enums are initialized. Than, the static map get's constructed. But because in the constructor of the enums the map is accessed the map is null at this point of time....

If you are still sceptical please try to start c:geo withou the null check here ,-)
Hmm... it does look nicer. I put the points there to differentiate between when there was a level change on the leaderboard and when there was just no data (and the line is straight between a couple days, say with a month in between).
Lol, look back one more commit and you will see that the original didn't have them. I added them then removed them in back to back commits. :P I thought thats what needed to be done, but after thinking about it decided I didn't want to break it.
I think it just means that **bar** is missing a `@nomodel` annotation
Voltage control and cpufreq source from N.E.A.K kernel and Siyah kernel
why it's not parametrized here ? use TimeStep ?
Thanks. Fixed.

On 14 February 2012 05:12, Panajev
<reply@reply.github.com>
wrote:
> userData_ could be a reference to the same instance userData points at.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cocos2d/cocos2d-iphone/commit/5a16b6e8dbc17a833c58757c63ea78c5ca7101ac#commitcomment-964484
Good catch.  Completely forgot about it.
SQL_GRAMMAR_CATEGORIES is covered by the "standard" SQLStateConversionDelegate.

SQLStateConversionDelegate has one extra value in  SQL_GRAMMAR_CATEGORIES, "2A". 

I'm guessing that the values were copied from an earlier version of the converter and that "2A" was added later. Anyone know if that's the case?

If "2A" specifically should not be included for Cache71Dialect, then SQL_GRAMMAR_CATEGORIES should be added back in here.
Building can be configured.
Sugestie de nume pentru funcție: `unflatten_with_defaults`. Ai putea să o pui chiar pe clasa `Person`. Also, ești sigur că merită să-i dai argumentul `defaults`? Nu pare să-l folosească nimeni.
At creation of the character the server crash
missing #include caused no compile errors, but a nice crash on win64 when the returned pointer was CDQE'd to fit into an int. (yay)
I think it is correct to keep looping.
Once a heap is dead it doesn't matter if you release it - unless it is linked 2^31 times which is very unlikely!
In the mean time another thread might have created a heap and added it - so we want to match that if possible.
This also shouldn't be *that* time critical.
This doesn't look right. Used to only apply to single parameter allocates, now will ignore the activity id on all allocates ?
Why move this (we really should come to a consensus about whether protected go at top or bottom, then stick with it....)
**FIXME:** having to detect the client _always_ reduces performance.
It should be detected _if the transforms need it_.
Как тут сделать `this`? Просто `compare` не катит.
```haskell
Tasteometer.hs:20:62:
    Ambiguous occurrence `compare'
    It could refer to either `Network.Lastfm.Tasteometer.compare',
                             defined at Tasteometer.hs:12:1
                          or `Prelude.compare', imported from Prelude
```
Database is for SkyFireEMU_rebase or SkyFireEMU?
this comment form kihyeon.
Now only "Frostmourne hungers!" remains to implement ^^
As expected it's 3 times slower and it starts to be slow at 1 million of samples:

```
>>> from sklearn.metrics import precision_recall_fscore_support, precision_score, recall_score, f1_score
>>> y_true = np.random.randint(3, size=int(1e6))
>>> y_pred = np.random.randint(3, size=int(1e6))
>>> %timeit precision_score(y_true, y_pred), recall_score(y_true, y_pred), f1_score(y_true, y_pred)
1 loops, best of 3: 1.72 s per loop
>>> %timeit precision_recall_fscore_support(y_true, y_pred)
1 loops, best of 3: 567 ms per loop
```

It's also more verbose to import and call the 3 functions separately.
Hmm, I guess my OCaml/Haskell roots show. Renamed in a04681bcb85fb85b8845b37a245bd7cd1203f44d.
This is repeated down below on line 62.  The one down below is the one that will be used.  This one should be removed.
Whichever suits your fancy :) surprise me. Haha :P
Hi rvanvelzen, I can't find any other way of contacting you, so commenting here;
Are you still using NNF for anything? I'm trying to get feedback from anybody using NNF but have had nothing from anybody for the last several versions. I feel like I'm just adding features just for feature's sake, and that's not a good path to take. (Camen Design forum doesn't use or need 99% of the features I've added to NNF). I could really do with some advice on what's really needed for deployment of NNF and if there's anything you're desperately missing from it (or something missing that's preventing you from using it). Thanks for your massive help on NNF thus far, you've really helped spur things on thus far.

If you want to contact me privately, e-mail kroc@camendesign.com
Commit erratum : Revert "[thederekjay] use __raw variants of readl/writel"
I had a problem with this changes. 
After the screen rotation, the views are going to be mixed.  If I am changeing the orientation on the 3th view, they are getting mixed up. 
For example: 
before: View1....View2....View3 
after: View3...View2...View1 

after changed back this optimalization code, evereything is fine.
@lucasas Hey, sorry just noticed this message. It looks like you got something similar committed though so I'm guessing this isn't needed?
I need to say that you forget `root = true` at the top of the file.
Можно аргументы передавать по const&, тогда не будет лишних копирований. Почему бы не сделать оператор константным? Зачем возвращать по ссылке временный объект, его в момент использования его уже нет.
On Feb 6, 2012, at 11:00 AM, Peter Kleiweg wrote:
> Het gebruik van Accept is alleen bedoeld voor content negotiation. Dat is hier niet van toepassing omdat de client precies weet wat ie hebben moet van de server.

We kunnen dezelfde argumenten nog eens herhalen.

Dit is volgens de standaard, de gang van zaken in de meeste frameworks, en aanbevolen wijze van het selecteren van een mime-type in RESTful applicaties. Als we in de toekomst JSON toevoegen (bijv. voor een web interface), dan werkt een 'xml' parameter sowieso niet meer.

Mijn besluit staat vast: toekomstige versies zullen de 'Accept' header gebruiken.
Just thought about this further: should we not have an action 'search for' as opposed to a new proxy object?

 was told off a while ago for naming the Web Search Module objects 'Search Google (UK)' because objects in the 1st pane should be 'items' and not 'actions'.

I'm thinking it should read like this:

Current Web Page ⇥ Search Site For... ⇥ TEXT

What do you think?
Never mind, I eventually figured out that `grid.data('urlFolderTemplate')` was the JS accessor for this.  However, we should probably be a bit more explicit in commenting JS files with the data attributes that they use.  Document the frontend/backend API in the code.
So the question is should this CDI enabled `ContraintValidatorFactory` be injectable:

- and defined in the standard (CDI specific)
- and defined in the standard (as a generic DI feature)
- but be implementation specific (Weld)

And if defined in the standard, should we offer a qualifier?
Совсем нет нужды подключать скрипт количества комментариев перед каждым выводом этого количества.
Достаточно подключить его один раз, где-нибудь внизу страницы - дальше он сам найдет все ссылки с якорем `#disqus_thread` и вставит количество комментариев туда.
Missing an `echo` ...
Most of this commit is adding support for LDAP group name cleaning, and related configuration options. Upgrading to Grails 1.3.7 (stable), adding a "finished loading" statement to the doWithSpring closure, and adding a unit test(broken as of this commit) to check the new code.
@Mook feel free to chime in http://bugzilla.songbirdnest.com/show_bug.cgi?id=25472
I've submitted a patch for it:
http://review.cyanogenmod.com/#change,13093
Ditto re: single quotes.
Nope, I was thinking exactly the same. Let me know if I can help with the implementation.
Ah, well, this is not actually Java code; people are supposed to know that Akka is written in Scala ;-) Will add one here, though.
perhaps place those in separate 
   
   "xxx " taggedAs LongRunningTest in {
perhaps we should mention that my dream blew up in the face of error handling, which would be less nice when using lazy vals.
`with ImplicitSender with BeforeAndAfter` not used, remove
I didn't validate the quality of the existing tests, but I agree that that looks very much like a PoisonPill. I can change that
@viktorklang seconded
mmmm, over-engineering
just to clarify, the way these are usually used is, destinations of files are constructed like `${DESTDIR}${PREFIX}/target`
Account is now taken as both the right name, and the module methods for the "session card", which is always CardWithAccount+*account.
Please take a look at a newer version of this comment: https://github.com/angri/nhlib/commit/e0687b9a619e0604667ae88632c37e5f5a7cab6a
This code sample would now cause AmbiguousResolutionException (and no longer makes sense).
```
parameter `qgrid_size`
```
I is just unacceptable for a user to wait 15 seconds.
I know that backwards compatibility means a big deal and I could probably write a Interface for that, so messing up the whole plugin interface architecture.

However, this was the easiest way to do.
Additionally, when this is part of not-yet-released CKAN 2.0, some changes for extensions could be affordable.
Btw. it would be just renaming or creating a new method that call the original function. So, it's not a big deal.
I'm not going to lie, what I'm about to say is entirely my OCD speaking: the indentation is off from the rest of them. I know PEP-8 says not to indent like that in the first place, but since it is at least maintain the straight line  :)
I'm just thinking that I'd say "JavaVM.framework" to indicate the library I want, I wouldn't explicitly add a trailing slash.
Hmm, missing merge deltas here, I'll have to fix that.
Seguindo a mesma linha, seria interessante mudar também o `:fee_payer` para `:fee_payer_login`.
why not just creating the object ?
What is `createMap` being called for ?
Can you remove these?
> this feels quite costly?

That is why it is a lazy val. I have to check the original dispatch code,  
but I think I took this from there. If there is a better way I am happy to  
implement it, though.
Shouldn't `fixed-width($pad-to, $context)` simply be `$pad-to`? I don't see a `fixed-width()` function anywhere, or any need for it.
how about `return $this->hasSourceCache[$name] = true;`
Cache the result of Date.now() in a variable?
Since I put the use of `Watchdog` inside the `if`, I can get rid of `beginTimer` and simply make the constructor initialize it.
this is the money line
Can we create a separate test and keep the original test as is?
лучше делать перенос после какого-нибудь параметра. например, так:

Id EditorViewScene::pasteEdge(EdgeData const &edgeData
		, QHash<Id, Id> const &copiedIds, QPointF const &offset)
QDebug надо убрать
Ah, roger.
Sorry about this. I fixed this another one.
1) In my testing we needed to explicitly call `layout` or none will be rendered, not sure why. 
2) Didn't try not explicitly calling render, when I take those lines out, it works fine...good catch. 

I can kill the presenter, going with the cmd+f option. I would eventually like to pass it a url `/users/5/whatever/3` and have it give me all the routes it matches since we can't Cmd+F for that, but for now i'm looking to get this out the door. I can experiment with filters/layouts in my free time on sextant, and just port over the most relevant/used features. 

Unfortunately `_routes` is a different class than expected by the inspector, final version will now look like this:

```ruby
  def routes
    inspector = Rails::Application::RouteInspector.new
    @info     = inspector.format(Rails.application.routes.routes).join("\n")
  end
```
@jeremy good point. I decided to omit fractional seconds from default value because they are optional. Do you think it'll be better to add them?
Nevermind. It is necessary.
I think this is overly specific, ExecutionContext shouldn't be tied into Try.

def bind[T, R](f: T => R): ExecutionContext = this
lines 379-388 simplify, as nearly as I'm willing to eyeball, to

    type MethodSeq = Seq[u.MethodSymbol]
    def select(alts: MethodSeq, filters: Seq[MethodSeq => MethodSeq]) = filters.foldLeft(alts)((a, f) => f(a))
Is this a sneak preview of an upcoming commit or am I missing something?
Time sync error.   You response appears first, but didn't show up when I
was making mine.
Please don't ever write ".length == 0".  It's "isEmpty".  It doesn't matter if you know it's short, it's just a really bad habit.

The second two conditions appear to be

    macroDef.paramss == List(Nil)
That Paul Phillips, always giving the benefit of the doubt...
Checking if something is a string is normally done with ``isinstance(window, basestring)``, this doesn't work for unicode strings.
yes! This code is used to set  the `active` class so that the corresponding menu appears to be selected in the admin section. The logic so far was to check if the name of the controller was in the url. It worked great until I added a way to edit a country.

The reason why it did not work is that `State`  is a nested resource under `Country`. Hence the word `country` appears in the url for states as well in line 16 in `_configuration_menu.html.erb:`
`admin_country_states_path(Spree::Config[:default_country_id])`

As a results both country and state  menus were selected when clicking in country instead of just one. 

I am not sure what's the best way to do that. I am checking if the url ends with the controller name. If yes it's all right and the class can be set. It works for almost all the menus except for a few such as 
` <%= configurations_sidebar_menu_item t(:tax_settings), edit_admin_tax_settings_path %>` where the url ends with edit. That's why there is the second conditions. It's a bit of a hack and I am open to any suggestion :)
Just checked in admin/orders backend and that doesn't seem to error out. Order emails are associated with `order.email`, not `order.user`
I liked this one better the other way (though I am biased here).
Newline after the `"""`. Also, the first sentence should fit in one line and be a paragraph on its own (a "short description").
While the first of the possible output is better, the second is worse.
We don't have numpy as a dependency, this needs to be removed.
Agree with you. I think it will be better to separate this commit by two. Thank's for advice.
My original soc plan was to have me implement multiple algorithms wherever possible. I think this was one of the cases where I found time to do so. Its the same thing. Maybe I should name it accordingly?
Maybe add an integration example as well? Not too important.
the parentheses are not needed, in fact it may return a tuple...but maybe not
bu sekilde kullanmaya gerek yok. tooltipComponentManager'in var oldugunu assume edebilirsin.

daha iyisi, TooltipComponentManager newable bir sinif olabilir. Bunun icin ```goog.addSingletonGetter``` kullanmalisiniz. tart.ui.ComponentManager da buna gecmeli.
what to do if it's not set ?

woudl it be better (Yii style) to check here the getIsSecureConnection() ?
As a plSimpleModifier it's only allowed to have one target.
See https://github.com/H-uru/Plasma/blob/master/Sources/Plasma/PubUtilLib/plModifier/plSimpleModifier.cpp#L92-L96
Yeah sure but since no routes are specified in the example it should show what's generated by default.
you can use UUID for a bigger hash
http://docs.oracle.com/javase/6/docs/api/java/util/UUID.html#nameUUIDFromBytes%28byte[]%29
Is this a performance issue? I'm sure we could cook something up.
I guess you can work around it like this:

```c
errno = 0;
kd = kvm_open(...);
if (kd == NULL && errno == 0) errno = EINVAL;
```
Can you remind me why we check that the 'checked' attribute is set to 'checked'?
Is this println() statement still needed?
why do we have commit=True default value if it raises an error?
Ah true. I have yet to integrate jshint with emacs. Will do.
I think thats a good idea. This is implying that both 'for' and 'do' need to return a boolean.
No space after include
The method. Anyway it was checked 2 days ago (forgot to delete this)
Should probably add a comment explaining what these blocks do. Same for the stuff in the previous and next file.
The driver name is used for displaying the selected avatar driver's settings in the ucp and acp pages where avatars can be set. Javascript is used in order to only display the currently selected avatar driver by default. If javascript is disabled it'll display all of them. This will make the setting page more readable for the user, especially if more avatar drivers are added.

Calling CSS classes or IDs i.e. "avatar_driver_local" is, from my point of view", definitely a better way to go than "59bc8b79abe28b4d2e3b4a719c8d5c9a".
Were line endings changed here? Please check the entire diff.
Or possibly an avatar driver's avatar template name.
Which part looks wrong to you? I could also move that part out of this function into the files where we request it from the user.
But it only duplicates the code a couple of times then
Yeah, I was the one who changed all "e-mail" to "email" in one of patches and here I absolutelly forgot about it :). Fixed.
Why is cache test interacting with global $cache?
@igorw is it possible to specify a Symfony component as a service? If so, that would make this much easier; otherwise, I have to somehow get the HttpKernel object injected for this method.
I need an answer here please...
Not identified as a query building event.
See other comments about waiting until tests are ported...  and punting for now...
this should just be zmq.green.Poller.  If the implementation does not make it available with this name, that must be fixed as well.
What's wrong with `mirror://` URLs?
Good spot. I've added a `points` argument to `from_coord()`.
#7 says *level_height* over *model_level_number*.
Here, *model_level_number* has been chosen over *level_height*.
Is this not incorrect?
Get it. Disable countable for generic keys, make it true always. Thanks.
コミットに含めるのが普通ですね。
自分用ということであれば.gitディレクトリの下に書きませう。
Apart from SN, this method should also pass information about MAC addesses when available.
To here are still spaces. Everything else is tabs.
you could just use rm_rf here to get rid of the ENOENT.
I like GlobalClientPlugin
I agree.. I'll add some messages in later after my exam and ask in #seneca what people think. I agree that it *should* have an error message but I'd say we are a little pressed for time at the moment, heh.
This logic duplicated in ReportMojo, so it would be better to find a way to reuse it instead of duplication.
Have you considered the case where dp_ids is an empty array?
Totally nit-picky, but the imports should be more like this:

```
import json
import os

from django import test

from nose.tools import eq_

from fjord import browsers
```

http://mozweb.readthedocs.org/en/latest/coding.html#import-statements

I occasionally screw this up, so if you see me mess up import ordering, kick me.
Instead of this single line:

```ruby
        if !MacOS.clt_installed? and python_prefix.start_with? '/System/Library'
          # For Xcode-only systems, the headers of system's python are inside of Xcode
          args << "-DPYTHON_INCLUDE_DIR='#{MacOS.sdk_path}/System/Library/Frameworks/Python.framework/Versions/2.7/Headers'" 
        else
          args << "-DPYTHON_INCLUDE_DIR='#{python_prefix}/Headers'"
        end
```

then it builds on all four different options: (homebrew-python, system-python) x (clt-only, xcode-only)

As a side note: I am about to make a proposal to organize all the python related stuff in one central place, so that we don't have to repeat writing `which_python` and related over and over again. This code here in the vtk formula is a good starting base for it.
Untested, but this should work: `prefix.install Dir["{#bin}/*.app"]`
Yeah ty. Classical copy paste problem. I ran the tests, they worked fine. I will fix this
"Вернётесь к своей исходной задаче и продолжите работать над ней", наверное, будет лучше.
Оригинал: "Switch back to your original story and continue working"
Another untraceable log message and callback with no error.
What I'm suggesting is that _you_ might want those hooks there in 90% of the cases, but that's specific to your workflow.

`wp scaffold post-type` should be concerned just with registering the post type, not with whatever custom functionality you might want to add on top.
Can you sort the imports alphabetically, or have your IDE do it?  If you don't do it now, the next person to edit the file will probably have it automatically done by their IDE.
Not quite analogous since `name` here is not a test name but something like `testName(test.Class)`, but I agree that that positioning looks better and will change accordingly.
depending on the application and what you're doing in the shutdown callback this might be a potential issue,
because we're not flagging that the callback has occurred. After invoking `fn()` we might want to `fn = function(){}`
so that when "job complete" etc still figure you're not getting the callback invoked a bunch more, though this might
not be an issue for most if people are just exiting etc
Please add private methods below the public ones so that the public interface is more obvious.
You also need to call unescape() after calling decodeURI to fix the remaining issue.
What about inline editors? See comments below.
I think the check _is_ needed. Line 142 (after your latest commits) passes a `forceTriangle` var, which defaults to null. With your latest changes the triangle disappears when changing the sidebar width.
unused, can be deleted, as it doesn't significantly add to projectHorizontal imo.
this sentence is somewhat redundant with the next, maybe just remove this one and the empty line
That's what I thought you meant. Then you have
display = CaseDisplay(report)
instead of 
display = report.CaseDisplay()
+1 for only including a non-minified version.

Minified versions suck for code diving, are going to end up making me write a bash function wrapper for grep so I don't have to look up the flag for truncating long lines, and probably aren't going to have a huge impact on performance.  If you can afford to worry about the performance benefit of removing a low fraction of the size of your JS files (they're probably gzipped when sent anyway), then you can probably afford to set up an asset pipeline that abstracts it away for you and bundles groups of files that are used together, which is really half the benefit of minification.
So am I correct in thinking that creating new ACLs based on {DAV:}authenticated, {DAV:}unauthenticated  is currently completely broken? :)

The ACL method has seen virtually no use :P so there's been little real-world testing.
Is this acrually what you want to do ? The value assigned to end is unused here. Should it be selectedNbClass = end; ?
Why do we need so many parameters ?
That's not exactly what the method does. And you must precise that it is (at best) a String representation of an URI.
I agree that it is good that you can see to which element the class applies, but I do not agree that it is smart to reuse classes for different styling. One class, one style.

So you have two options: everywhere rework the CSS to be like that. Or do not use that.
I haven't put that there, must have been auto generated by pycharm while i was testing something.
I removed this because our templating jade files cause this to actually not be used at all anymore.
use `equal` for plain string comparisons
Instead of capturing the first bit in a new group, try using a non-capturing group, like this: `(?:/|\\s)`.

http://stackoverflow.com/questions/3378773/can-i-use-an-or-in-regex-without-capturing-whats-enclosed

Then, you only have to change this line and not update all the indices below.
updateScrollBar
void换行
Since you only accept one form of config (dict) this method is unneeded.
I think regular RPM file should be ok.
did you mean to remove this space? not a big deal but it looks like line 22/20 has a space
Do we still need UNARCHIVED_FINISH ?
`check_call_git` will throw an error if this fails, and if fetching from origin fails we have a big problem.
Cube needs 2 mandatory fields : type and time, and one optional field : data

$time can be retrieve from $record.

$record['context'] is used for the optional $data.

Since there's no way to set $type directly in the $record, I set it in $record['context'] too, so $record['context'] always contains at least the type field. If it contains more fields, they will be passed to $data, else $data will be null.

Cube is a bit different, because it needs specific fields to sort the datas. It's not a pure logging system, but more of a metric tool. Instead of sending a string, and then let the logging tool parse the datas to extract the pertinent bits like the typical logging tool, Cube request a pre-formatted json object.
Yes, I believe you are correct. 
But dont thiink it's a problem here since it's already resolved and cannot be changed any more. The external code cannot resolve/reject it and callbacks have already been called.
So I doubt checking the repo is really that expensive, but what are your thoughts on moving this into the settings file so it would be loaded once and cached?

I don't know how it's implemented internally, but if it's (unnecessarily) hitting the filesystem with every API call, then it might be worth doing.

Thoughts?
Corrected to be "Should be at least two children in a union"

On Fri, Sep 21, 2012 at 5:28 PM, Paul A Martel <notifications@github.com>wrote:

> In src/frontend/org/voltdb/plannodes/UnionPlanNode.java:
>
> > @@ -36,8 +54,69 @@ public PlanNodeType getPlanNodeType() {
> >      @Override
> >      public void resolveColumnIndexes()
> >      {
> > -        // This node doesn't actually exist
> > -        assert(false);
> > +        // Should be at least two selects in a join
>
> "join" is not the right word, here.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/alexeevm/voltdb/pull/3/files#r1665954>.
>
>
Maybe also add a note discouraging the use of 'entry_points'?  Obviously it's needed for the releaser, but my understanding is that this is not going to be at all supported by `packaging`/`distutils2`, right?
If you want I'll add a pull request to this to implement this in a metaclass :)
`(1 / (u.cm * u.cm))` is a Quantity, `u.cm ** -2` is a Unit. That's because any operations between numbers and units returns a Quantity. I guess I could add a check to the Units to see if the number is one, and if so return a Unit?
Is there no way of asking windows for *the* startup folder, in a version-agnostic way? Seems hard to believe one would need guesswork like this.
Need to re-think LDAP authentication.
just curious why the <tt> tags
here I would set a color variable and have a single PPGeDrawText call instead of 3.
When using shaders, OpenGL ignores the enabled-ness of GL_TEXTURE_2D - if the shader samples the texture, there's a texture, if not, there's not. So delete this please, and the glDisable call too.
I'd side with the version using `__future__`, because `codeop._features` is undocumented, so it could theoretically be removed or renamed in a future version of Python.
Note there's also a more specific trait for this sort of thing - `DottedObjectName`.
This still suggests some coupling of kernels and notebooks.  Kernels do not need to be associated with a notebook, do they?
Have we tested this on Linux? I seem to remember finding that it made a mess of all the colour codes, which was why the `unicode_to_str` was needed. But I might be wrong about that, it was some time ago.
this code is weird and brittle.

What are we doing here? waiting for a condition if its true we wait X of not we wait Y. What is the goal?
In a discussion with @peff, @nulltoken suggested that perhaps the API could take a flag that specifies whether to interpret strings as pathspecs or as literal paths.
I went one better and removed it entirely. It's presentational anyway, so now it's in the CSS.
ok, yeah, I grok this now.  the update goes under the hood to make the change directly in the db, and self was being made out of date by that.  yeah, this makes sense.  thanks!
you can move this to the line above
I indeed use tabs everywhere.
Oh yeah, missed that. Thanks@
The exception message should add information, otherwise we can just create an exception without a message at all.
This file is getting pretty large.  For clarity, it might be worth splitting out this method and register_default_callbacks into their own files/classes.  Perhaps configuration/{built_in_param_parsers,default_callbacks}.rb?
BTW the corresponding Unix command is `gunzip`, which might be a more familiar term (or not).
There's no standard for having this rendered so I'm not sure what do to here about it.
When I've created custom panels I've made them look/act like the distribution ones so I don't have any example for a custom panel.

I'd rewrite the things in there but I think that it wouldn't be the right way of doing it as I'd do more damage that good for existing panels in this case.
in 2.4 ? It cannot as 2.3 is the LTS
Can you add the missing parts of the phpdoc? (same for the setter)
I can see why this is here for BC, I think, but it still means that using or extending UrlGenerator means you need to have the request context methods, even if you null them out.  That feels icky.  I don't have a better solution to suggest, but it still feels icky.
hangover from an experiment with `ParameterBag` to display 'em ... will remove it.
This is not needed. ``$groups[$q][] = $value;`` is allowed even when the key ``$q`` does not exist yet (reading an unknown key is forbidden, not writing)
The entries are already quoted inside the loop just above.
So, this is actually correct.
me likey. good call, much better that we have hooks called for all worker shutdowns, intended and unintended
module Ownerships
Could this be called after the block? This way it could be called few times, not just once.
Remove this line instead of commenting out
going to add sorting by date once the posts are added, just to be safe
Well, maybe I can. The truth is that this was the minimal patch I could do. In the past I could NOT use the Service.ycp API because I had to use different access to SuSEfirewall2.
Ah ok, see the most recent PR there. 

#
Zachary Heller


On Thursday, June 21, 2012 at 10:33 AM, Alex Sharp wrote:

> > @@ -0,0 +1,19 @@
> > +require 'spec_helper'
> > +
> > +begin
> > + Exceptional
> > +rescue NameError
> > + module Exceptional;end
> > +end
> > 
> 
> 
> Right, but the test needs to be performed against the library itself -- this code catches an exception because the exceptional library isn't getting loaded, and thus throwing a NameError when the `Exceptional` name is encountered. You can replace this with simply `require 'exceptional'` (assuming it's added to the gemfile) and everything should work correctly.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/zaarly/failsafe/pull/3/files#r1027698
> 
>
变量尽量统一管理，在上层定义。
I suggest using a function that calculate the binomial C_n^p, either in boost
http://www.boost.org/doc/libs/1_46_1/libs/math/doc/sf_and_dist/html/math_toolkit/special/factorials/sf_binomial.html
or a hand made one in math.
Does try/except/else work in python 2.4?
This avoid checking a bunch of runtime-varying type IDs for identifying like-wise types. I've explained this more extensively now.
Remove extra white space line
+= 1 is actually the most performant way to increment in javascript apparently
3 lignes au dessous
Это будет забавно смотреться на production :]
Don't forget to fix `ts` files...
You can actually do this using vanilla JS and childNodes attribute and get rid of all this string manipulations and if statements.
`(hash[1] || '').replace` might be more "JavaScripty". This just looks like python :)
Tu as du commiter le répertoire et ce fichier par erreur
Sure, I'll take care of that.
Yes, if possible.

Also you should move the test resources into the same package as the tests in src/main, and move the test classes to the same package that has the test resources.
I think you actually mean a different arquillian protocol. You should be able to just use the @OverProtocol annotation to use a different protocol for a specific deployment without having to create a whole new artifact.
Oops, I just realized I didn't put `"returnValue": true` in the manifest O_o
According to @kk1fff, when browser app is running OOP, the event is caught by system app;
when browser app is running inproc, the event is caught by browser app.
Still need to confirm when my browser part implementation completes, I will unmark this first.
I'd say 'application manager', since it launches and kills apps, and may do more.
the localization supports plural rules, you don't need to/shouldn't be doing this here yourself.  (Other languages have much more complicated pluralizatin rules.)  If you look on trunk, kaze actually already updated the deletion prompt to do this, and we already have a lot of other pluralization examples.
It's probably not the best name for the function because we use it also on the first run (independent of the event), but I didn't wanted to create next function for abstract it.
if you don't set up a `content`, `after` and `before` becomes not aviable and does not get any kind if style from the CSS
nit: too much spaces
Give names to anonymous functions (seekUp, seekDown, cancelSeek)
Won't this **always** be equal to the `length` variable?
Dough! The rebase didn't work really well.

Don't worry, let me send you a pull request to your branch to fix it :) (but probably I'll do tomorrow)

Thanks!
UX want the message surrounded by quotations, so we need the " inside the ellipsized text
We should add a comment here explaining that these strings don't need to be localized because they are specific protocol names and so there is no need to translate or benefit from translating them.
I now generate the ids in Webapps.jsm at startup, since I need to merge two apps lists.
I misunderstood the usage of element:last-child :P
At first I used the last-child CSS on the list parent, which looks like #views-list:last-child, and nothing happened.
After I saw your comment, I realized that I should use the last-child CSS on li elements, li:last-child works.
So what you said was 100% correct!
Also I used the same way on tiles view, works fine too, thanks.
Should put msbuild on path, but we can add that logic in a later commit
Conform to what of the OS?  The typical path separator?

It's a property in a properties file - using commas to denote multiple property values in the same property is more conventional.
These last three tests all use identical callbacks. Maybe should be just one function?
I think it would be cleaner to do:

FileSystem.get(config).exists(filePath)
Extract this and the following if statement into its own method.
id => rowId
Basically CF_Container is class from php-clouldfiles/rackspace: https://github.com/rackspace/php-cloudfiles/blob/master/cloudfiles.php#L917 and we use alias: https://github.com/KnpLabs/Gaufrette/blob/master/src/Gaufrette/Adapter/RackspaceCloudfiles.php#L7

BTW I see that build is broken too I will check this PR later :)
>I'm not quite sure what I would need this for. Why would I want to create a function when I can have just the object as is?

Because you get something like `o.type  = postID[0] if (postID[1] is 'submenu')` which makes no sense.
thread expansion could do the same. instead of cancelling requests, just make the button unclickable
Because I'm trying to save annotated objects, not annotated configs?
Instead of commenting this out, I think you should introduce an option to run in solo-only mode which defaults to false.

That way you can send a pull request back to the maintainer and have a better chance of them requesting it.
It was my first implementation too. But than I thought about other possibilities and stuff and moved from this implementation to more flexible - current - way. In the end the flexibility was not used so it can be changed back. @lmr  what do you think, should I change it in this commit or keep it working and we'll see how it evolves?
@pepsiman You indeed can update the description using setText dynamically during the execution of your test.
Probably you can also update self.description, and self.descriptionDone in your start method. At that time, your renderables are updated.
This is way cool :)
You're right. I'm going to change this. Thanks for the suggestion.
Why should we place wirteable files to 'App Data Directory' ? Is it because users need publish their app to desktop?
should add this logic to the model or maybe we should even break creating a status message to some sort of factory object //cc @denniscollective
The whole idea of this is that each page only gets requested once, because the atWho callback can be called multiple times while the request par a particular page is on the wire. So the next call will request the same page again because after your change the page only gets incremented *after* request has a response.

This way even if atWho were to issue another request before the previous one returned, the new request will already get the next batch of users.

You'll see the difference if you add `per_page: 1` to the `params` and look at your browsers network logs. With you have multiple requests for each page with your changes. With the old code you'll see each page only requested once.
form is not ajax so loader is useful here. am i right?
Is plain `Login` not clear enough?  Seems like `Login` is fine as you're already in the context of the branded app.  But I don't know exactly what this is used for.
Please change this line back. We have it the other way so it can be run by an automated testing system that only understands npm (not mocha). If you want to run tests easily, you can type "npm test" instead of typing out the full node_modules/.bin thing
What is this for? Explain in Javadoc.
is this a reload? I want to be sure that we're testing the websocket push
item.context isn't an array so reduce doesn't seem to be an option here unfortunately.
I don't think such method is relevant for a loader. It is ok to have any internal utility function. 
But it looks very odd to import such method from require('@loader').
this is also a field we don't want to compare - test fails when I run it against a database on a different host
Actually, this is not very good: there is no real mapping between the Windows DACL to the simple Unix permissions model: you can go from Unix to Windows, but not vice-versa, without loss of data.

Ultimately we need to fix ACL handling in the code, and that may end up with changes to File - but this is the best short-term compromise.  It is also the Windows team approved way.
Why catch these exceptions and try again? This seems like this is trying to cover up some other problem in the logic of the program.
It was quite interesting to refactor this code, take a look at my [commit](https://github.com/restebanez/agifog/commit/abcf1d265a11d38a3d0cf670ead0b74fb15c7863)
I feel that styles belong elsewhere. Really liked the idea of include_style <style_class>. Let's discuss it.
Please rename this to `tilecoord` to follow the TileCloud's conventions (no CamelCase).
@dgasparri ping
Could you refactor this? Is hard to read a double Ternary use
brake in wrong line
Why return an array here, rather than exposing the underlying object? Such methods have saved me more than once...
@param above @author tags
I believe so, however Microsoft.Dynamic is not building for the v2 profiles either. I need to revert that other commit again to test it out.


Keith Rome
Senior Consultant and Architect
MCPD-EAD, MCSD, MCDBA, MCTS-WPF, MCTS-TFS, MCTS-WSS
Wintellect | 770.617.4016 | krome@wintellect.com
www.wintellect.com


-----Original Message-----
From: slide [mailto:reply@reply.github.com] 
Sent: Tuesday, April 24, 2012 11:16 AM
To: Keith Rome
Subject: Re: [main] Silverlight 4/5 build fixes (#67)

> @@ -89,8 +88,7 @@ private enum ModuleCodeType {  module, or raises 
> ResourceImportError if it wasn't found."
>              )]
>          public object load_module(CodeContext /*!*/ context, string fullname) {
> -            dynamic sys = context.LanguageContext.SystemState;
> -            PythonDictionary modules = sys.modules;
> +            var modules = 
> + (PythonDictionary)context.LanguageContext.SystemState.__dict__.get("
> + modules");

Does this still build under the NET20 profile in the solution?

---
Reply to this email directly or view it on GitHub:
https://github.com/IronLanguages/main/pull/67/files#r726662
It should be: 0 item**s**

The jQuery example is btw correct: `return count === 1 ? word : word + 's';`
Would really like an overview about the differences between YUI and Mojito and how they correlate to each other in that blog post ;)
why do you check for $name? anonymous form controls should be reset as well.
please remove the spaces inside braces for function calls
It is true that the webserver can do all of this on its own: etags, expiry dates, compression. The main advantage of replicating this functionality within php is that we are less dependent on webserver configuration, so we make deployment easier.

As for the 10-years expiry: I guess that it is ok as long as there is no change in the js/css files that are served via the combo. This forces the css/js developer tho change the name of css/js files every time they patch them. While this might happen automatically whenever yui release a new version, I am thinking also about developers wanting to use the combo to load their own stuff
I'll put that in, but not now :-p
It would. Which outcome do you prefer? I don't mind the user being rejected.
Ah, the <div> tags were cruft that didn't add anything to the visual. The two <dd>'s are a visual change to make the appearance have a second line. 

What would you recommend to use a second line via css?
It's has the `<label>` that you asked where it went. Its in a weird order to look like bram's mockup.
Instead of doing this use `self.client`. `self.mozillian_client` is a ready to go logged in mozillian.
Let's make a completely separate unit test for this and do a test for `this instanceof Popcorn` in each of the known callbacks: `_setup`, `_teardown`, `start`, `end` and we'll add `_update` later when that lands.
This should be in its own ticket and PR. Also, please review readyStates before filing: http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#ready-states

readyState 1 doesn't know where to start playing a video, it's too soon
semicolon here plz
Just use blank nrpe plugin (to use nameserver in resolv.conf )
I these run today, since they get picked up by the functional test task.
maybe I misunderstood, but I'm not sure this is right, as the id can contain a name.

e.g. for hosts controller, the id is the hostname, and for operatingsystem, it would be a numerical id.
I'm fine with merging if the IE disease is limited to the separate module and we document the behavior seen without the code above thoroughly.
Sounds fine. The thing here is, what we probably haven't told you yet, the status of the testrun is always the one of the last execution. So you can't differentiate between the different platforms/branches. So I would simply remove that part and only refer to the dashboard results.
It's possible to have valid HTML that's missing a body tag, so I think we may want to get rid of /html/body here.
I'm working on getting that feature branch together right now, and I'll move this method into navigation.rb.
I have to admit that I don't know. I took this from the current logic which chooses the audio stream with the most channels / best codec (see https://github.com/xbmc/xbmc/pull/700/files#L1L640).
Ok, that is what I thought. Thanks for clarifying.
Can this be broken down into more than 1 line?
A space after the hashes would be nice (on all of the titles), as that's what I'm using in the other articles.
You mean that fetching the node as an php object to get his object_id is a bit overkill, right ? If yes, there's certainly a better way to do that. Adding a (light) `getObjectIDByNodeID` method to `eZContentTreeNode` could be a solution
We should take the apply function here and move it into Raven_Util

Once that's done, we should take the coercion step and move it into Raven_Client, and have it coerce data before it encodes to json, and before it passes it into data processors.

This will guarantee processors have a much easier time dealing with data.

This would then take the usage of apply here and obviously call out to the new util which applies the function recursively
I removed this line and changed the download link (wget) to the original 
one.
We don't want ot use this anymore, use the long/verbose version we do for nondestructive
Extra should be removed as well.
Do we let anonymous users comment?
might make sense to do `var row = rows[0];` first, so we don't have to use rows[0] more than once.
This is probably retrieve commentS
I'm not sure this paragraph adds too much information. Maybe it's not needed?
are the line breaks here intended?
Make sure that the "down" migration works. Make sure the test db gets migrated.
We'll probably need to make this optional somehow. I imagine not all users will want to link against curl (thinking embedded stuff here). But we'll see.
Согласен, задание простейшее. Ну да что поделать, привычка смотреть в будущее издалека. Кто знает какое второе задание нас ждет :-)
Теперь вижу, да.
Этот метод, безусловно, рабочий, но на процентах близким к 100 — получается очень много лишних итераций. Можно сделать без повторов
а может тогда как-то так:
```obj-c
- (char*) randomCharacter {
    static char *alphabet = "ATGC";
    return &alphabet[arc4random() % 4];
}
```
еще быстрее будет :)
Ah, completely missed the other test.
Is the double-backslash intended, or was it broken before? :)
Esta funcion es la nueva, se llama antes de empezar el Looping (solo una vez).
Esto seria luego de la transicion "ENTERING"