labofsahil/github-event-dataset-2012 huggingface dataset:
It isn't tied to anything, it is the default value for a configuration for
a specific implementation of a AuthenticationModule.

If I renamed RestAuthModule to AGDefaultAuthModule (and
AGDefaultAuthConfig) would that make a difference?

Tokens in rest can be done dozens of ways.

Summers Pittman
>>Phone:404 941 4698
>>Java is my crack.




On Fri, Nov 9, 2012 at 8:41 AM, Matthias Wessendorf <
notifications@github.com> wrote:

> In
> src/org/aerogear/android/authentication/impl/RestAuthenticationConfig.java:
>
> > @@ -0,0 +1,19 @@
> > +
> > +package org.aerogear.android.authentication.impl;
> > +
> > +import org.aerogear.android.authentication.AuthenticationConfig;
> > +
> > +
> > +public final class RestAuthenticationConfig extends AuthenticationConfig {
> > +
> > +    private String tokenHeaderName = "Auth-Token";
>
> Well, even if the "Auth-Token" name is only one used in REST, I do assume
> that other future transports may have a "token header name", so it would be
> not optimal to tie it to a "rest specific" config class.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/aerogear/aerogear-android/pull/31/files#r2083017>.
>
>
Why were class variables necessary here?  I'd avoid class variables if possible.
И почему ты считаешь, что это ошибочная ситуация?
Закрывать ресурсы надо в finally.
The expression "runs from localhost" is ambiguous. Are you saying that the developer needs to run the test from a web server?
Can you elaborate on why Captionator can't load captions from the file system?
自然に考えると、0 (1未満)だったら消してよいという条件になりそうな気がするのですが、glossary_source_num が 1 のときにも delete_glossary しちゃっても大丈夫なんでしょうか？
もし、この後の処理で削除したら 0 になるからこれで大丈夫ということなら、delete_glossary_source をしてから上記の処理を行なった方が素直ですかねぃ。
I'm not sure how to do this.  Github actually updated this pull request automatically when I pushed these other commits.
we don't have that in bidpom 
https://github.com/mozilla/bidpom/blob/01c0a43d9ab7c2526ebc88a2bf58c409fcdcab9c/pages/sign_in.py#L36

the default  is to login with the default account from credentials :  <code>def login_with_user(self, user = "default"):</code>
It is not RPI only. On arm we just use ARCH=arm. Also android is treated that way.
-1 on deleting on GETs
Пытался минифицировать. Переделаю в typeof val === 'undefined'.
I'm glad you asked, I definitely need a second opinion.

It returns ``null` because I expect this code path is never executed. Here goes my reasoning: 

``getFile`` is  private and only called by ``fromEditorInput``, which is also private and only called by ``fromEditor``. Method``fromEditor`` expects to receive a fully initialized ``TemplateEditor``, and hence its underlying ``IEditorInput`` should already be attached to a system file that has a "scala.html" extension.
Furthermore, note that even if the former implementation used ``Option``, I believe the above was already assumed, e.g., https://github.com/scala-ide/scala-ide-play2/pull/27/files#L4L29

That said, I think you are right and I should not return ``null``, it's much better to throw an exception in this case. If you agree, I'll correct this.
This can be written more compactly using a comprehension

```
yield dict((k, v.rvs() if hasattr(v, "rvs") else v[rnd.randint(len(v))] for k, v in items)
```
`ndarray.resize` tries to move the underlying buffer, just like `list.append` or C++'s `std::vector<T>::push_back` do. When there are multiple references to the array's buffer, that fails. The error checking code inserted by Cython (try `cython -a`) gets such a reference and holds it until after the `resize` is done, causing it to fail every time.

`np.resize` circumvents to issue because it always copies. This is ugly, but it'll have to do for the time being.
actually I think I want this merged in with as little api changes as possible. We can still do the rename afterwards.
delete that.
We don't need to use `advance_iterator`; Python 2.6 has the Py3K [`next`](http://docs.python.org/library/functions.html#next).
standard import should be first :)
you could do

R = y.copy()
if coef_init is not None:
    R -= np.dot(X, coef_init)
you should prefix the function with test_ otherwise it's not run by nosetests

coverage is currently very low
если использовать JMSSecurityExtraBundle, то тут этого не должно быть
You are right, I probably did not see complains in the log...
Accepted and modified.
Are properties images and repeatCount supported on BB?  What about blob and file object support?
Please use int instead for consistency and clarify, unless there's a reason I'm not seeing.
Agreed, but I have created a new issue for handling nested superfiles.  HPCC-7821.  I'm still working in this area... i.e. adding package support.
Related to prev. comment really.. shouldn't this be something like version >= 1 really?
I know @ZER0 talked with folks from UX who are really into limiting width of widgets and have some special UI for toolbars, so it might be useful to sync with them before encouraging something like this. @ZER0 knows more about it so he should be able to provide a better feedback here.
Do we no longer need the onmouseout?
la c'est pareil.

Je préfére que tu mettes un
```html
<div class="spacer"></div>
```
avec
```css
.spacer { height: 20xp }
```
Que tu puisses réutiliser ailleurs
parameterize these values.
I am not swallowing any exception , if you are not able to see full view of file then click on view full code
Can you comment on the difference between this and the existing parseURL API function?
Why wouldn't we use the sender's DB? If other users of MC/MC-SAAS are using multiple databases, presumably it's good that we're actually checking what database to use.
The quoted strings are an artifact of variable interpolation in public/stylesheets/sass/application.scss
why are we using `/var/tmp` instead of `/tmp`?
I thought about using Metric but it didn't seem to be the right context:

one entity -> many checks
one check -> many agents
one agent -> many metrics
one metric -> many (data) points
Trying to keep comments length smaller make code more readable. I think no more than 100 columns would be right.
And use print_good to warn about the path where the file has been downloaded, adding after store_loot something like
<pre>
print_good("File stored in: #{loot_path.to_s}")
</pre>
Github uses https for everything now, so we'll want to keep that in there. Good catch though!
good call. done
I think that's fine, as long as we update the DEPENDS.txt
Ah, I never ran into that with my use of the class, but I haven't been creating intermediates. Adding that would probably be a good idea, though it'll be rather inefficient.
Need header here as per code writing guidelines: http://udig.refractions.net/confluence/display/ADMIN/About+this+Project
I changed it to items to make it slightly more universal. Seen as its now a publicly accessible template now i thought it would be a good idea to name its variables in a way that didn't restrict it to a specific function in the site.
Change `CompositePrimitive` to use the same terminology, and add the breaking change to CHANGES.md.
Since this also works in 2D and columbus view, I want to suggest the name `pickGround` or `pickSurface`, but `pickEllipsoid` is OK since it implies picking an ellipsoid surface, i.e., this same function will not work for terrain.
This is probably something for the future, but ISO 8601 allows intervals to be specified in different ways, such as "2007-03-01T13:00:00Z/P1Y2M10DT2H30M" and "2007-12-14T13:30/15:30".  We should support those, too, eventually.
Does this function have mixed tabs and spaces?  See https://github.com/AnalyticalGraphicsInc/cesium/blob/polylineCollection/Source/Scene/Polyline.js
It would be awesome if we could read this information from the tilemapresource.xml file.
We're not already at that point?  Show me an example.  It should be trivial to get there, right?
Why dont u delete and recreate the container ?
Absolutely, I simply missed this.
split path args
"forward" should probably a constant
They're not implemented in Android. I was under the impression that they weren't working correctly in iOS, but subsequent testing suggests I was wrong. I'll  update the support here.
I'm thinking it might be a good idea to alias window to a local "global" variable to save bytes and speed up scope resolution. If you declared "global" in the touchStart() function, then remove the var i = 0 from the for loop and just add them to the top.
Mobile Web doesn't support music player.
1 blank line is enough.
I'll add this part to throw the ValueError exception as well
Would this test have failed outright previously or just timed out?
Changed in 0a77f9c9010af977a9c43dea274eb51ceaf995d2.
You should be able to pass the hash in as an argument to the constructor instead of using marshal_load.
this line is dead code as you return
Это те самые яндексовые практики по наименованию классов? Давай, может, остальное тоже переведём тогда?

И ещё околосабжевый вопрос: для не-русской аудитории не будет выглядеть дико?
"...how much to sell a `<var>furniture(1)</var>` that just arrived..."

maybe "how much to <code><b>charge for</b></code>", or "at what price he should sell"?
@andralex: No, no, the fact that `Not` returns a template _is_ important. It's its sole purpose – the user instantiates `Not` to avoid having to write his own template.

Compare this to IFTI, where the user doesn't even need to know that there is a template. Note that because of this, I even explicitly mentioned IFTI in my »algorithm«. Ha! :P

Which names in Phobos would need changing in your opinion (besides `staticMap`, which is a misnomer)?
Why not make this part of the template constraint?
This leads to bad alignment. If there are two branches to an if, both should be either inline or on next line.
IIRC `real` might be implemented as a `double` on some platforms, maybe that should be taken into account?
Why the cast? It wasn't necessary before.
Since it uses `#gsub` and not `#gsub!`, body will always be returned as is. Also there's a typo in github.
It's a constant.
readXLinkConnectorsFromRegistry
deleting this class was an accident. I will add it again.
"real" data objects should always be serializable
ok, i'll change it
Why not `this.getDrugNodeID`?
Tolerant? Do you mean what level of precision do we care about?
the EventListener namespace should only contain the listeners. This has nothing to do in a subnamespace of EventListener
updated the tests. git merge does have some quirks there for example this one :(
Next four lines are kind of ugly, but I couldn't figure out a better way to express: "By default, use 443 or 80 as port for nginx depending on https set to true or false. But if the port was set to something else as normal or override attribute, then use the port that was set there."
Plain string interpolation in the attributes file didn't work for me, as https flag always seemed to be false, even it was set to true es override.
The requirement for specifying a configuration reader and an initializer breaks the intent and initial point of using mixins.

If non-static configuration is a requirement then moving to an inheritance based approach might be necessary to clearly define what the client is required to implement, in the case of inheritence it would be `super(config)`.

> Please be aware: a change like this breaks compatibility with existing clients of this library
It's an atom, not a variable.  It's used by the "object is properly formatted" step to call back into the cmdbs:validated method.
Yeah.  Is that on current Splash or Splash 2011 or what?

On Wed, Oct 31, 2012 at 11:05 PM, alexcole <notifications@github.com> wrote:

> In esp/esp/program/controllers/lottery.py:
>
> > @@ -178,6 +188,11 @@ def initialize(self):
> >              for (student_id,section_id) in interest_regs:
> >                  self.ranks[self.student_indices[student_id],self.section_indices[section_id]] = ESPUser.getRankInClass(student_id,self.parent_classes[self.section_indices[section_id]])
> >
> > +        # Set student utility weights. Counts number of classes that students selected. Used only for computing the overall_utility stat
> > +        for i in range(self.num_students):
> > +            self.student_utility_weights[i]+= sum(self.interest[i, :])
> > +            self.student_utility_weights[i]+= sum([sum(self.priority[priority_level][i,:]) for priority_level in range(self.priority_limit)])
>
> Woo hoo! So 4.4 is the equivalent of every student getting 20 hours of
> interested classes (or 13 hours of priority classes), which is both
> reasonable and pretty impressive.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/learning-unlimited/ESP-Website/pull/427/files#r2001087>.
>
>
It hasn't been merged yet. I'll move it out as soon as it has been merged and tested.
In this conditional you should check that locale's length is >= 2, otherwise you'll get array index out of bounds exceptions sometimes.
How about reducing the number of declarations to make the code easier to follow. 

```d
auto len = MultiByteToWideChar(
    CP_UTF8, 0, name.ptr, name.length, null, 0);
if (!len) return null;

auto buf = cast(wchar_t*)alloca((4+len+1) * wchar_t.sizeof);
if (buf is null) return null;

len = MultiByteToWideChar(
    CP_UTF8, 0, name.ptr, name.length, buf + 4, len);
if (!len) return null;

buf[0 .. 4] = r"\\?\"w; // prefix for extended-length path
buf[4 + len] = '\0';
```
"one OF the other" (should be OR). I don't think this sentence is necessary though, since people should know what mutually exclusive means.
When considering consistency in the framework we should compare apples-to-apples whenever possible; so compare this to other transformers not channel adapters. With transformers we typically do follow the "x-to-y-transformer" naming convention. Even if we decide not to use that convention, the transformer at least needs to indicate *direction* (even the channel adapter does that by using "inbound" or "outbound"). An example transformer pair that avoids "x-to-y" but indicates direction would be: payload-serializing-transformer/payload-deserializing-transformer. If we named this "syslog-transformer", what would we do when we add a transformer that converts TO syslog.

FWIW, we do already have prototypes where we use the SyslogTransformer and then convert to a new Tuple type.
Did we agree to remove this completely? 

It's ok, so long as they supply a dedicated T.S; shouldn't we just document the issues?
There are a lot of similar lines of code throughout the project...
How about them into `IntegrationNamespaceUtils`?
Just that the javadoc makes a distinction between message headers and message properties...


__Message Headers__

The JMSCorrelationID header field is used for linking one message with another. It typically links a reply message with its requesting message.

JMSCorrelationID can hold a provider-specific message ID, an application-specific String object, or a provider-native byte[] value.

__Message Properties__

A Message object contains a built-in facility for supporting application-defined property values. In effect, this provides a mechanism for adding application-specific header fields to a message.

...
use $jenkinsGroupRun->getJenkinsRuns() instead
I think `_.filter` would probably be a better choice than `_.compact(_.map(...))`.  It reduces both time and space complexity.
Should this be checking for status 200? It seems to return status 200 locally, with {"result":true} and the push notification does get sent. Using 201 makes this return a false error.
todo later: should contain a real documentation block rather than the method name
The entire document is about version |version|, so can this simply be "OMERO.server"?
We should probably scope this to the admin class
My bad ! Every time i commit i have to sanitize code as i really can't stand the coding style used by 99.99 % of developers. Sorry i forgot some but i fixed the ones you pointed out.
This is because the tests use 'origin' and 'destination' fixtures repetitively, causing table name clashing. I first thought we could use separate new 'atomic_origin/destination' and 'locked_origin/destination' tables by adding them to fixtures. Or, could drop all origin/destination suffixed tables after each test. But it seemed a lot of overhead for simple task. So using `sleep` here.
Decision: check in Mongo if the UA comes from a UDP client.
true, most curious thing here is the if (Map* map = GetTarget()->GetMap()) though :>
Should probably check for equality between `createdAt` and `updatedAt`.
The concept of converters is to convert request *attributes* to objects. I'm not sure that converting other "parts" of the request is something we want to support (cookies is probably out of question I think).
The original didn't use $translationFileList ?
I would rather put this in the "Notes" section of the docstring.
Hey, sorry for the late thoughts: for as much I had suggested `DONT_OVERRIDE` myself, and yesterday I thought `DONT_INTERCEPT` was awesome, today putting my hat of API user on I find it awkward to need to write my custom interceptor with double negations.

Looking into `org.hibernate.search.test.interceptor.IndexWhenPublishedInterceptor.onAdd(Blog)`
I think that the
   
`   public IndexingOperationType onAdd(Blog entity) {
		if (entity.getStatus() == BlogStatus.PUBLISHED) {
			return IndexingOperationType.DONT_INTERCEPT;
		}
		return IndexingOperationType.SKIP;
	}`

It would be far more readable as `IndexingOperationType.APPLY_DEFAULT` or just `APPLY` (as much as I like `SKIP`).
WDYT ?

I realize it requires a shift in point of view, but this is imho the point of view of who's writing such an interceptor implementation; our discussion of yesterday seemed to focus on the framework implementation..
please add a canGrowSync to the ImapFolderSyncer that returns true.
Sure, they can be done in a callback.  I think in an earlier draft of the patch there was some asynchronous stuff happening that benefited from the split that got streamlined out of existence.
Can you add a parseInt here around textContent?  The string gets safely coerced to a number when you divide it by 14 in the getEstimate callback, but it's not super hygienic since most other operations would footgun.
I updated this based on another committer doing so. If that's not cool, just ignore it.
the return statement should be on the next line
Can you please quickly investigate if xstream can be configured to skip this field if it is null? I don't want it to appear in xml so that packages exported with 1.1 can still be imported to 1.0.10. It may require writing some Converter. Don't spend too much time on this. It's not a must.
Pair the `interpret` and `assert` lines (i.e., move the blank line), and align the comments.
You have an orig file here from merging
Shouldn't this be existsSync ?
Why not just return the condition
what licenses are those packages of?