labofsahil/github-event-dataset-2012 huggingface dataset:
On Tue, Sep 25, 2012 at 06:11:28PM -0700, Satrajit Ghosh wrote:
> hi @GaelVaroquaux : could you please take a look at this function and see how it might relate to:

hi @satra. I believe that the link is that generating diffusion maps by
iterations of the diffusion process creates finite-time diffusions, that
eventually converge to the equilibrium, or limit diffusion maps. The limit
diffusion maps can be obtained either by a power iteration, as done when
iterations single step diffusions, or by considering the associated
eigenvalue problem. That eigenvalue problem can then be solved using
arpack, in which case it relies on power iterations once again, but is
hidden.

Does that make sens?
The only thing that needs to be in the try/except is the view_init, it seems to me.
please the alphabetical order
Should be `static` and at the top with the other constants.
@erynofwales it's just notes to myself, to it will be ripped out.  I need to do doc strings/block comments better  :D
That does not automatically make it right.
Well the response is the same, but true the request is smaller, but I suspect the savings are greatly outweighed by the benefits of WSDL generated client stub - @afishbeck
thasnk well soptted
Yes, I guess so.
I'm a little unsure when 'critical' is flagged.
What you don't want to happen, is for another consumer with priority over HDD, to force out rows every time one is added.. that would mean an almost infinite # of phases as it spills table and fails to filter out much of anything.
I think you should rename the function checkAndTrimWorkunit, drop the 'inputname' parameter (which is always "Workunit ID") and remove the strieq test here
That's a bug.. I'll issue a pull request to fix in a sec.
indentation looks a bit odd here?
If you're going to print the error, why not print it inside the loop, rather than setting a flag and continuing even if there is an error and the program will bail?
Probably can't happen, but if unsubscribe() called again..
should change be set to 0 here
We need a test for no filter enabled.
boş satır pampa
I think your name should be listed there. Don't get me wrong.
@ntalbott Yes, that's a sensible assumption. I'll change it.
Was attempting to follow a strategy suggested by DASPRiD. However appear not possible. have removed from file
My motivation was following rails asset pipeline conventions app/assets should be only the assets that your application itself creates, while vendor/assets should be anything from an outside source/library.
This should read currentSiteName.
I'm still not convinced that this should be an Integer instead of a string...
Add descriptive name for LETTER
both will do same and also there is no difference performance wise
do you need to pass the driver? doesn't the page already know the driver from the constructor? If it doesn't please set it so that we don't have to pass it around.
It's not that the registration parameterization can't be done, it's that it has to be done by registering new users with diff time. Doing this by hand takes a lot of time but with the automation it's much faster. So breakup the data using a @dataprovider with the user and time pairs. 

you can use my login test as an example on how to use the @dataprovider annotation or here is a site online has has a good  example: http://functionaltestautomation.blogspot.com/2009/10/dataprovider-data-driven-testing-with.html
Interesting - I saw no reference to an @default identifier in the API explorer, but I do see it referenced at 
https://developers.google.com/google-apps/tasks/v1/reference/tasks/list
Would be nice to save a call - will test.
Please add docstrings to the methods of this object.
Denis, please argue, why do you think so?
I had to add an extra bit of code to support this properly (not tested since refactor :$)

i added an init to the class to set self.server to None so it can be tested in clean
I have fix that
Fixed, sorry.

I'm not sure about where to put the "bundles.show.infos.unknown" key though...
Need to mark this so the EMF code generator does not clobber your work :-) May need to add a javadoc so people understand what you are doing here .... // return Label + "Folder"
updated, committed, and re-based
I know I'm not suppose to nitpick local variable names, but why is `oldMaterialTemplate` an "old" material template?  Just because it is in the cache (or not) doesn't make it old.  When I see old in a variable name, I think the code is going to restore it later (get old viewport, set new viewport, render, set old viewport).  Maybe it's just me.
Should this really return a `Cartesian2 `?  It seems like we should introduce an interval type like we have `czm_raySegment` in GLSL:
```
struct czm_raySegment
{
    float start;
    float stop;
};
```
@mramato is there already something we should use?
It's not right now, but it will be required in the very near future.  Right now there is an issue with CZML with embedded image links.  Essentially the links are always relative to the server and not to the location of the CZML file the image is linked from (which is the needed behavior).  This is probably the first thing I will fix after this pull request goes in.
Is there a reason this just isn't `undefined` by default?  Do we not allow undefined text to mean no text?
Just in case you didn't understand, this color is actually NOT the color of a wheel, is the color of stroke of the wheel when a mouse move over the wheel.

In the spirit of your answer "Why not?", can I make such assumption?

* Should we make the size of wheels (including width, height, radius) configurable? "Why not?"
* Should we make the color a Vertex2D widget configurable? "Why not?"
* Should we make the size of a Vertex2D widget configurable? "Why not?"
* Should we make the size of highlighted a Vertex2D widget configurable? "Why not?"
* Should we make the color of the stroke of the chassis configurable? "Why not?"
* Should we make the color and height of 3D chassis configurable? "Why not?"
* Should we make the color of PointLight and AmbientLight all configurable? "Why not?"

For the other things that I am going to add:
* Should we make the color, size, position of the remove button for a wheel all configurable? "Why not?"
* Should we make the color, size, position of the rotation widget for a wheel all confgurable? "Why not?"
* Should we make the color, thickness of a 3D wheel configurable? "Why not?"

These were just thing off the top of my head that I said "a meeting to discuss about what we think users would care enough to configure". But if you don't think we need one, and think these "Why not?" assumptions are valid, then I would need your help in designing a new UI, because the current UI are not going to accommodate so many configurations.
It seems like the main difference that the toJSON call serves beyond the JSONObject constructor is that instead of a npe being thrown when the argument is null, a null is returned so it can be checked.  While there wasn't a null check there previously, can we go ahead and add one please?
Shouldn't this be [self.disabledString length] since length isn't a property but a method? Not trying to be pedantic :)
Will this work for translation? I thought this bit of l10n had to be handled in the DB.
This needs to be changed now that we have multiple hypervisor support
-1 for depending on rackspace-shared-utils. it gets updated too frequently.
String.isEmpty() is added in java6 and it only handles length() == 0, no trim, and  Its an instance method so you need a string first.
Ya que esto:

lang = file_manager.get_file_extension(editorWidget.ID)
key = settings.EXTENSIONS.get(lang, 'python')
comment_wildcard = settings.SYNTAX[key].get('multiline_comment', None)

Se esta pidiendo en el if y en el else, podria estar antes del if asi se pide una sola vez.
how about using `size_t` throughout instead of all this casting
"hand load by hand".

Sync settings.

Battery & Data Manager
does this really have to a bundle?
Also done, these were *your* names though. ;)
These test setups could use some comments. For example, I see we're adding a field to the original class, but to do so we need to create a static - couldn't we just use a static and have our category selectors set those? I don't think there's anything wrong with these, but they're rather complex and without any comments they'll be difficult to maintain.
Same thing with `this`
Please clean up this class.
Comments are missing?
hmm, on second thoughts @ahodgen , this isn't a typo!

It's just that we determine error type based on the string output of the program (at least my `kinit` command doesn't give error messages). Can you revert this "correction"? Otherwise I can if you don't want too. :)
These three lines can be merged into a single line  to fit into the designated widith.
yes, it's coming up, I just wanted this in the open.
hmmm... that might work, I'll try it.
Confirmed: removing classes works as previously, and the divs do not appear, if Javascript is disabled.
The PHP engine is lazy-loading helpers so they need to be public services.
the word started can be removed
Внутренности неймспейса отбиваются табом.
Seems the sass `append` function should suffice. Am I missing something?
Should be `.off()`
agreed. fixed.
Missing space after `if` and before `{`
Need to protect replacing a real lock registry

    this.lockRegistry = this.lockRegistry instanceof PassThruLockRegistry ? new Default.... : this.lockRegistry
And it would also break current functionality where expression that evaluates to null would trigger an exception
That I plan to do, once we settle down.
Hey Iacopo - is there a reason you'd like that better?

WIthout putting any thought into it, and considering I haven't seriously
looked at the code for 6 months, I don't really care which is used.


On Tue, Nov 27, 2012 at 7:43 AM, Iacopo Spalletti
<notifications@github.com>wrote:

> In userena/views.py:
>
> > @@ -128,6 +128,12 @@ def signup(request, signup_form=SignupForm,
> >              # A new signed user should logout the old one.
> >              if request.user.is_authenticated():
> >                  logout(request)
> > +
> > +            if settings.USERENA_SIGNIN_AFTER_SIGNUP \
> > +            and not settings.USERENA_ACTIVATION_REQUIRED:
> > +                user = authenticate(username=user.username, password=request.POST['password1'])
>
> Why using the password to authenticate:
>
> user = authenticate(username=user.username, password=request.POST['password1'])
>
> instead of:
>
> user = authenticate(identification=user.username, check_password=False)
>
> like in activate view?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bread-and-pepper/django-userena/pull/200/files#r2236667>.
>
>
I would not comment these out :)
Drop in 'n go, no?
Resources are getting set at "db". If we implement above change, chef tries to search for all above four new resource within "db_postgres" and gives error "NoMethodError: undefined method `privilege'" while setting "preset" ( Same will happen with other 3 resources ) .
I tried to implement this, but I'm having a hard time trying to figure out what goes where. If I put $groups in params.pp as follows will it create a dependency cycle?

```
# Groups                                                  
$groups = $foreman_proxy::dns ? {
  true  => [$foreman_proxy::puppet_group, $dns_group],
  false => $foreman_proxy::puppet_group,
}
```

And should it be override-able by the user?

Another thing I just thought of was the that it's not guaranteed the group exists. Only in config.pp is foreman_proxy::proxydns included. Maybe puppet-dns should be rewritten and split so foreman_proxy::install can include dns::install?
I'm not a security expert but I think that CBC is safer (http://www.adayinthelifeof.nl/2010/12/08/encryption-operating-modes-ecb-vs-cbc/).
and by the way, ECB don't need an initialisation vector so this line is useless
you can just call :symbol.to_s to convert a symbol to a string
This needs to be called something other than FunctionType to avoid potential name-collisions and general confusion,
Some broken links to address in another PR.
Again, please don't modify all the z's until the spelling choice has been made.
Yes there aresome.
You can use ``:mailinglist:`ome-users <ome-users/>` `` extlink. Will add that to the README as I thought it was there.
I think i18n will be tedious to type.  Maybe bind this to "msg"?
Is there a need for this empty link?
the nuget package and the assembly name have to be suffixed with ."Fody"

this was a design decision to improve the performance of assembly scanning at compile time. it allows me to on target nugets and assemblies ending in ".Fody" rather than scann all .dlls in Packages
You should use ``if`` instead of ``elseif`` as the previous ``if`` returns.
And it should be outside the loop checking for the different possible getters.

and finally, the space should be between ``if`` and its brace, not between ``isset`` and its brace
if you want to drop a line you should drop after "{"
it is good practice in Obj C to drop on "," but in JS we never drop on ","

For readability too, we try not to waterfall functions too much (ie pass though lots and lots of functions)
the problem with anonymous function is that if the code breaks and you get a call stack you can never trace the function that has the error because it has no name.

Where possible, to avoid the above ^, create separate functions.
Maven 2: Maven Harder
It should only flatten once here. Set the shallow flag.
I'd say add something after that line like:

    You can <a href="lanyrd.com/2012/a-barcamp-near-london/schedule/">add your session</a> to the schedule, and <a href="http://lanyrd.com/2012/a-barcamp-near-london/coverage/">share your photos, blog posts and other coverage</a> there too. Also don't forget to use the hashtag #bcb2012!

I'm not too sure about that last line with the hashtag, but I think that first one will encourage people to put stuff up.
Why are we using a priority queue to keep track of this?
is there a specific django command that could check if the object already exist in database? or do I have to build a logic manually to compare the variable?

(for checking if AnswerSet object already exist in database)
I wonder if we should start a new line here so that the location is on a second line? The wrapping seen in your example screenshots doesn't look very good to me.
Couldn't these methods be commend up and the check-sum function virtual?
Seems like this should be GetLazyPropertyFetchMode(path) for consistency.
I think in this case, we want to "reset" the folder list and try again. I say "reset" in scare-quotes because we shouldn't *actually* reset them entirely, since that would imply deleting all the locally-synced messages.
An implementation Proxy class wouldn't actually "use" this method. The "on" prefix here actually specifies that Kroll will call this method directly when the value of `hasListeners` has changed.
"true if the file is verified as read-only, false otherwise"
Just a thought - but for future samples we could use the *org.springintegration* base-package.
You can definitely define your own exceptions - it just might not be possible to pass them across WinRT boundaries.  In that case, if you want to detect them later on, you should define your own and then a WrapExceptionForWinRT static method that transforms the exception into something you can rethrow with a useful callstack.
@geihsler @samnang I think it's OK to reach into article views for article specific partials. The shared folder really exists for partials which don't have a home and are truly "shared" with every page like the footer and navigation sections.

Oh a related note, I don't know if displaying the full article partial in the search results is the best option. Especially when we start to display articles, people, and activities. However for right now just use the articles partial until we start pulling in more records into the search page.
conn.isolation_level returns an integer, so `else ""` is confusing things.

I think #331 is a slightly better solution -- we shouldn't add the `'iso_level'` key if we don't know what isolation level we're in.
Make Saxon an **Optional Dependency**. Particular when testing/running against other XQuery implementations (BaseX in that example), I experienced very strange error behavior.
Sure. The `boto` library I've used in the data-gathering repo would be useful.
what if there was a folder inside the extension that had more JavaScript files in it?
what is the point of this line? you are returning an empty value?
I think doing the if (blackberry) check earlier makes sense. I guess it doesn't matter whether the blackberry namespace is created by WebWorks or not? (I'm referring to environments where blackberry.location exists but WebWorks isn't supported)
because of the strange console which is loaded - i can send commands to the openengsb process, but i don't get any response, and i'm also not 100% sure if the commands are accepted - you told me about a parameter to turn this off, we should find it and use it, afterwards it should work smoothly
Do we need the <p> tag?
Why does it need Namespace on the end of the name. I think it's pretty clear that it is a namespace when it's used somewhere.
Pas d'accord. Modification sémantique : "normal  [...] jobs" => régulier. Le terme régulier porte plutôt une sémantique de répétition.