labofsahil/github-event-dataset-2012 huggingface dataset:
I'm not sure it's possible to avoid using the $_FILES array here as it is just used as a global all over the upload process. The code could probably be reworked to pass around an array instead but it will require changes in many other places, not just here.
The entire file should be wrapped into `(function($) { ... })(jQuery)`, see for example core.js.

Should phpbb_enable_tz_dates be invoked on domcontentloaded?
Fixed by the rebase I just made
Maybe add a log to the ACP log saying that it failed and why?
It is a changelog for phpBB 3.x (same everywhere on that file).

Either that, or it is a changelog for 3.1.x and all the 3.0.x stuff needs to be removed.
@brunoais I'm a native English speaker and I have not heard wording like that. IMO this is much better: "not supposed to be".
I'm not sure about this, but running the query all the time is probably the safer way.
Can be merged into the previous `if`.
``//<![CDATA[`` in new line, i.e.:

```<script type="text/javascript">
//<![CDATA[```
I do not like the in_array here. Something like `isset($this->unlimited_reading_forums[$row['forum_id']])` would be better. Look at how `function get_moderator_approve_forums()` and `function is_moderator_approve_forum($forum_id)` work.
"cron override" seems like a better name in this case.
Should the form key be checked somewhere?
I would prefer a modifying event which can override rather than an overriding event which can modify. But other than this currently seems to make sense to me for some reason I do not have a justification.
According to my knowledge, .data works fine, for multi-word stuff (like "data-refresh"), however it is better to use "dataRefresh". To the addition, .data() tries to change return value to best variable type (ie. data-refresh="30" should return integer); .attr() returns string always.

Anyway, I fully respect usage of .attr. Could you, please, provide references showing problems with .data for me to learn more about it? Thanks!
What do you think about setting the actual error as the status?  I'm not sure what's possible here.
the monitoring module has a constants file to hold all the timeouts for easy tweaks later.
Shoot! It seems I did forget to push. On the bus ride home now. Will comment again when pushed.
Этому здесь не место. Если уж возвращаться доефолтную картинку, то делать через engine с соблюдением шаблона
Don't need the ```~```
It might be more informative to use:
```python
'Expected header trailing_length of {}, got {}.'.format(leading_length, trailing_length)
```
:-|
Hopefully it won't keep doing this..?!
文字要支持被用户自定义的方法格式化
Does it make sense to make DeviceInfo time-trackable, when Asset already is?
return ','.join(form.errors['__all__']) or 'Correct the errors.' if form.errors else ''
Здесь можно написать так:

```js
this._nodePrefix = this.__self.createId(this.level, this.item);
```
Minor: rather than the (verbose) question mark operator you could just compare the return value of the assignment against the NULL pointer:

x.haveselectxpathregion = (x.select_xpath_region = GetConstraintValue("select_xpath_region", pp, CF_SCALAR)) != NULL;
Please add the reference or simply remove the section.
I, for one, would rather have "info" level logging at a higher level of abstraction ("altering column", "creating table" etc). While it would be nice to be able to choose which SQL is printed and which isn't, ultimately SQL here is only needed for debugging.
Why remove the database check from the test? This is a valid and useful test.
There are 2 problems with using this query to filter assets:

1) It's slow, especially the way it is use (we perform the same query every time, and then slice it with a given offset. This is pretty inefficient. That's the reason why a smaller number of tasks is faster.
2) It doesn't work in the intended way for a region constraint which cross the international date line (the vertical line which runs from pole to pole at longitude 180 and -180).

nhlib's geometry library has solved this problem, and I think we should make use of it. It would probably be faster than making PostGIS do this work. We can discuss this in scrum tomorrow.
You don't really need this function if you named the additional argument in the lapply call
=> `ok( exec, "Click executed." );`


It was probably wrong before, but like I said about, we can clean them all up as we go :)
`("div");`
I think http://www.postgresql.org/docs/9.1/static/collation.html explains this. You explicitly assigned collation "fr_FR" to dict.dict, so this is what will dominate and be passed to svec_sfv. If you also explicitly assigned a collation to ARRAY['0'] you would see an error.

So, all function arguments share the same collation, because the parser will error out in case of inconsistencies.
Probably fine for now, but eventually I would expect convex optimization to be completely subsumed by the abstraction layer. In that case, it will no longer be a 'module' but just an operation provided by the AL.
Sure - give me a minute and I'll push another revision (I'll actually build it before I push ;)
Shouldn't invent a version number - we use the original project's versioning.
Need to pass the installation prefix to the build system somewhere! (Think "\_where_ is this being installed?")
Pourquoi revenir sur mon commit sur ce point? J'aime pas traité les coordonnée un a un quand on fait pareil sur les deux. (qustion de point de vue peut être)
Right, this was early on in the refactoring. Added them to make sure that the dispatcher wasn't doing something special with only `ack`s
Premyslel jsem nad obema moznostma. Nakonec jsem se rozhodnul mit u
uzivatele jen seznam id kurzu. Informace ke kurzu pak mit zvlast a jeste
zvlast mit pro kazdeho uzivatele a kazdy kurz jeho progress. Souhlasim, ze
je to asi mene prehledne a o dost slozitejsi.

Take je pravda, ze jsem dal spis na svuj pocit, ze mit to vsechno v jednom
dokumentu by bylo hodne velke a mozne pomale. Podival jsem se ted na nejake
informace o mongu a max velikosti a ta je nyni nastavena na 16MB. Takze
priznavam, ze jsem podcenil mongo a precenil kolik textu a kolik submitu
bude nas uzivatel schopen odeslat.

Kdyby nas uzivatel byl prihlasen do 5 kurzu, v kazdem udelal 20 uloh, kazda
uloha by byl 50radkovy zdrojak (x80), ktery odeslal 10x nez sel dal, tak to
uz dela 4MB dat. To asi zvladnem, co? Neocekavam, ze by nam nekdo nahral
vice nez 16MB kodu, ale dobry je na to nezapomenout. Takze asi to vsechno
strcim do uzivatele.

David

Dne 27. srpna 2012 11:24 Jiří Maršík <notifications@github.com> napsal(a):

> In progma/users.coffee:
>
> > +      # User already exists.
> > +      if exists
> > +        return callback(new errors.UserAlreadyExistsError(), null)
> > +
> > +      # Create salt and password
> > +      salt = new Date().getTime()
> > +      passwordHash = exports.hashPassword(salt, password)
> > +
> > +      # Create new user.
> > +      user =
> > +        id: userId
> > +        displayName: displayName
> > +        salt: salt
> > +        password: passwordHash
> > +        email: email
> > +        courses: []
>
> Já tu Dejvovu strukturu teda původně pochopil tak, že v courses budou
> nějaká data popisující postup pro jednotlivé kurzy (tudíž včetně toho, jak
> řešil jejich jednotlivé lekce) a i věci, které se do dat o jedné lekci
> nehodí, např. kterou lekci měl v tomto kurzu naposledy otevřenou apod.
> Nemám zkušenosti s tím, jestli by placatější model nebyl výhodnější než
> tenhle hierarchický, což by třeba dávalo smysl (v databázi bys mohl chtít
> mít kolekci fakt, který nejsou nějak rigidně strukturálně propletený).
>
> Slovníková podoba pro courses, a jejich příp. vnořené lectures, by asi víc
> odpovídala tomu, že se často budeme chtít podívat na data o jednom
> konkrétním kurzu nebo lekci. Uspořádanost toho pole asi nebudeme využívat a
> když už bychom chtěli kurzy nějak řadit, tak by bylo asi lepší k nim to
> pořadí, např. timestamp, kdy do toho kurzu vstoupili, zapsat explicitně.
> Unikátní klíče pro kurzy a lekce v nich budeme mít taky, jelikož je stejně
> budem potřebovat na jinejch místech pro naše vlastní účely.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/progma/progma/pull/7/files#r1462905>.
>
>
When not specifying `EMACS`, I get this:

    usage: which [-as] program ...
Good point, that was for smoothly transitioning from FakeWeb and I meant to remove it.
That's what I tried at first but it turns out an element instance changes from round to round. If I just used the original element stored in a previous round, I'd encounter the missing TypeKind issue. The only way around that was to use the name to look up the element using the processing environment. This may be simplified by storing all module types as strings as done in the other processors.
I'd prefer to use the same exact $InjectAdapter that we're already generating. Just add a new 'injectStatics()' method on that?
Just a note: you can use the `ok` assertion for this

```coffeescript
 equals true, true
```
is equivalent to
```coffeescript
  ok true
```
. Really a matter of style though, and in the case where you are testing other things for falsey-ness then the way it is is ok with me.
I think it should be once per event, not once per handler. If we can't resolve this ambiguity through the API then I'll just remove the feature for `once`.
Yeah PG was getting pissed if they weren't there. But basically the idea is if a PG exception is thrown the failureCallback would be called, successcallback if the opposite.  We have no need of them right now, so that's why they are nothing.
Done. All specs are now recorded in an array under their (top-level) suite, with name/id/description. If there are messages or perf data, those are attached to the spec data object.
Why did you remove these items from the main menu? I think they should be in both places.

Note that it would be nice if these were disabled when the focus is not in an inline editor, but I think that's a separate issue.
Can you add a comment here explaining why it's commented out?
Verify that sessionProvider is non-null before dereferencing.
Le fait de changer cet argument change le WSDL, il faudrais verifier la rétrocompatibilité des clients
Again it **works** when I run `./bin/satis`. It's the same code.

So that's likely a technicallity and not a bug.
Ok, will do that going forward.
There's issues with these array_searches too. Every array_search is a reasonably heavy look-up.

Instead, you should refactor this to use uksort. This sorts by keys, which implies that instead of array_search, you can just do a standard array lookup.

Also, the most important part. with these additions I feel that the code is now non-obvious; especially in relation to the key lookups, and the magic scores (40000 + $position). So the documentation should explain *why* this is done.
The author line is fine of course.
Do we need to modify the link for that? Maybe we could generate the link using the normal createLink method but have a redirect happen in the controller in case the respective objective doesn't exist for the selected report type. This would also solve the issue of the method createLinkByTab contain a switch/case for each report type.
This is a bit cleaner IMO, and the `&&` is more appropriate:

```
if (!any(is.na(lims)) && lims[1] > lims[2])
```
controller 要用複數名稱
We should put `#butter-tray` or something more specific in front of these, right?
Nope! It never was. Just an object the core uses to scrape the page.
Oh hm... yeah, I suppose that works. I guess figuring out the pixel width of the bar compared to the inner size of the container and then do that only if it's less than 3 pixels apart would be too much ;P
just realized, that if `chomp()` returns `null` you are already doomed as it will NPE at `trim()`...
Should use 2 argument method now that only 2 {} exist (damn slf4j for not making these varargs already).
判断NULL，下面也是
Why is this here twice?
Под виндой будет другой сепаратор
We should probably create a "constant" for "open" states and "resolved" states. Then you can do:

    if self.state in RESOLVED_STATES:
        ...

Also, if the state is an open state, then I think we want to wipe the ``resolved`` field. That allows us to unresolve something and have it show up "correctly".
Hm, doesn't this parenthesis potentially mess with ddoc?
Apparently there is only one checksum stored per package in pulp, so we should be ok with ```pack.filename,pack.checksum.to_hash.values.first```
lets remove this from the PR.  we need to instead switch to a routine where we allow full UTF8 usernames in Katello.  If pulp can't support the above we should use DB IDs or some other scheme to store users in pulp
Vars are scoped to the function, not to the block containing them. Best practice is to put all vars at the top of the function to clearly indicate this:

function storeNotice() {
   var idx;

   for (idx=...

Adding the "var" as you did does scope idx to the storeNotice function (taking it out of global scope as it was previously).
flipping the order would break the current non-soy users that use fn. also it may be desired to support delegated rendering in soy? dunno
Frontend šalje backendu response, odnosno input od usera. Oba dvije klase služe iskljucivo kod rezolvanja evenata
upstart seems to miss the parameter for switching process owner.... that crap.
wrong indentation here. It should only be 4 spaces per level
typos: can not only save you a few creds, but also enormously reduce
I don't think this should necessarily be part of this PR. Not a big deal, but make sure Zach and Max actually want it before we merge it in. They'll be dealing more with the views.
I think it's more consistent to use C++ string handling everywhere instead of strlen() and friends. In most implementations, c_str doesn't perform a copy, but it simply returns a pointer to the internal buffer. Though it'd be better to use .data() if you don't rely on it being zero-terminated like in this case.
Does this safely close the current running client and restart it afterwards?
Ah yes, that should be "natural number"
This is pretty ugly. Why not just get the count once and switch on that?

```csharp
switch(extended.Count()) 
{
    case 0:
         return;
    case 1:
        commandName = extended.Single();
     default:
         throw new CommandAmbiguityException(extended);
}
```
将这个servlet块移到servlet部分。上面还有一个servlet-mapping
+ is external, dat lukt ook niet :-)
get rid of method's content
OK, got the '.' objects. The `list2env` isn't appropriate here, since the environment is already created by the `makeNamespace` function.
Why are you casting as an expression? A call would seem more natural to me given that there will only be one component.
@Yn5an3 basically the active thread is a global, and rescheduling swaps it.  RETURN() just sets a register on the global, so it affects the currently-active thread not the calling thread.  I hit the same thing too.

And these functions, if they do have to wait, will switch to some other running thread during the syscall.  It's happening already in the Mutex, Umd, and Sema funcs.

@hrydgard I was also thinking about this.  AFAICT none of the HLE functions ever need to reschedule inside the function.  I'm thinking it can just add a check for a reschedule flag after every syscall (outside the wrapper.)  And same with callbacks (it seems like some functions execute callbacks but don't reschedule.)

Then only the short number (or none?) of functions which swap to a specific thread (not sure, maybe start and rotate thread) would need special treatment.

-[Unknown]
Where is the old `static_url` function/macro we were using with tornado templates?  I looked but cannot find it.
OK.  Reread most of the comments going back sometime.  @takluyver suggested charenc as the name of the module, so:

1. utils/charenc.py
2. utils/encoding.py
3. utils/encodings.py
4. ??
I don't want to include pry here.
Thread.sleep = evil.  let's not call this directly as you cannot test it.
can we some unit tests that parse as buffer ( the same we discussed yday )
block contents should be indented another level.
That'd be fine. Mostly I want to make it easier to set up without understanding the details.

So:

* kitsune for the user
* password for the password
* db_kitsune for the database name (and test_db_kitsune for the test db)

Does that sound good?
Nice!

However, what happens when the list of topics is really long? Is there a limit to the cache key length?

Tuples of strings can be hashed. Maybe it's better to do `tuple(sorted([t.slug for t in topics])` and then md5 hash that?
At that point you are sure this will always be ``True``, because you check that ``params.signature`` was not ``False`` earlier. You can remove that ``if``.
I can't find ``mkt.webpay.urls``, is this magical?
What is the role of this variable?
Move to a `ThreadLocal`.
this will have a side effect when using the serializer several times
This id should be ``twig.form.renderer`` as it is not an extension. Same for the FormEngine
What if the last delimiter-character  is escaped? That would mean the regex is still invalid.
> But they can be both be used to achieve the same

See ConstraintViolationList add() and addAll() methods:

https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Validator/ConstraintViolationList.php#L63

You can probably achieve the same with add() than with addAll(), but addAll() is much more convenient for adding more than one item.
Yes, I think I forgot to update the README on this branch.
Ah, right. Thanks! :)
i had no idea i added white spaces
And here unless such anonymous RuntimeError has a special meaning.
mr @AliMD chanta soal daram az inja
alan age bekham masalan radius ro ham ezafe konam bayad injoori benevisam?
```css
vendors = webkit moz o ms official

border-radius()
transition()
  for vendor in vendors
    if vendor == official
      border-radius: arguments
      transition: arguments
    else
      -{vendor}-border-radius: arguments
      -{vendor}-transition: arguments
#content
  border-radius: 5px
  transition : 10
```

ye soal dg inke ghaziye **vendor** chiye in vasat? man injoori bar dasht kardam ke ye array hastesh va mitoone har chizi bashe
Is `-init` ever called on a `NSSplitView`?
They are both about replacing files, but one is about replacing a compyled binary, which is unlikely to have any data the user cares about, whereas the other is about replacing a python source file, which the user may have edited, reformatted, etc.
@pelson - could you be more specific? I'm not a fan of this type of import, but the the namespaces in shapely are overly long and complex in my opinion. I also got tired of seeing polygon.Polygon and multipolygon.MultiPolygon as it seemed a little redundant.
I added the `require 'draper'` lines to the modules that use `Draper.validate_options`.  Again, I wobbled on this (why would someone be using `Draper::Decorator` without `Draper`), but the presence of `require 'active_support/core_ext/array/extract_options'` in `decorator.rb` was what tipped me in favor of making the requirement explicit.
This would come up again when I have get_layout, set_layout again
I need to reload server to set origin native-interface (it keep old interface until restart). If I don't do it, next tests will run with this changed native-interface.

----- Original Message -----
From: "Brian Stansberry" <reply@reply.github.com>
To: "Ondrej Lukas" <olukas@redhat.com>
Sent: Thursday, August 2, 2012 3:23:08 AM
Subject: Re: [jboss-as] JBPAPP-9506 [7.1] regression test (#2780)

> +
> +            // set up native-interface            
> +            op.get(OP).set(WRITE_ATTRIBUTE_OPERATION);
> +            op.get(OP_ADDR).add(CORE_SERVICE, MANAGEMENT);
> +            op.get(OP_ADDR).add(MANAGEMENT_INTERFACE, NATIVE_INTERFACE);
> +            op.get(NAME).set(SECURITY_REALM);
> +            op.get(VALUE).set(nativeInterfaceName);
> +            op.get(OPERATION_HEADERS).get(ALLOW_RESOURCE_SERVICE_RESTART).set(true);
> +            updates.add(op);
> +
> +            applyUpdates(managementClient.getControllerClient(), updates);
> +
> +            // restart server
> +            updates = new ArrayList<ModelNode>();
> +            op = new ModelNode();
> +            op.get(OP).set("reload");

Bartosz Baranowski tried to include reloads in integration tests for 7.1.2, but we found it screwed up the test suite on the client side (the reload would close connections and the clients wouldn't recover.) I don't know if this has been improved while I've been on PTO. I'll check into this next week when I'm back in my office.

Anyway, I don't think we should need to reload the server to clean up after this test -- if we do that's a sign that something isn't working properly.

---
Reply to this email directly or view it on GitHub:
https://github.com/jbossas/jboss-as/pull/2780/files#r1289434
You can do what's done at line 428 with ServerInventoryService. Have the DiscoveryOption.allowDiscovery() stuff done in a Service's start() method and have MSC inject the NetworkInterfaceBinding into that service.
At some point I want to go through an remove all the embedded persistence.xml files, and move them into the src dir next to the classes to be consistent with the rest of the descriptors, although we have so many already like this that one more is not an issue for now.
switched to AtomicInt.
Missing spaces, should be `.attr( "id" )`
See also http://docs.jquery.com/JQuery_Core_Style_Guidelines
That's probably a good idea. Are you making the suggestion for performance reasons or for code clarity?
Add the usual vim file header and you are good to go.

Bonus: remove the getter of `GridView.grid` and use real object property. That won't be null since we have deferred the Javascript loading in HTML.
(Just to make sure please do some test to verify the above statement)
As we’re mentioning `.filter` here, we probably should merge `tabs.css` and `filters.css` to avoid cross-block dependencies.
we have always the '/' character at the end of the origin URI?
I add some description and sync the returning result to contact api request result type.
and this white space
mocha.reporters.JSONMozTest = x ?
I *think* you can do something like this to make sure other existing CHLD handlers are not overwritten:

``` ruby
old = trap('CHLD') do
  old.call
end
```

I'm not sure we want to in this case though... since the other handler is probably also going to call waitpid and would block.
There is no outerjoin with joinWithTiny, but there is a left join.

Not clear to me that we need outerjoin though.
Hi Kilica,

Thanks for replay. You mean.
Should I change to if clause below 2 method?

$this->remove_install_folder
$this->rename_install_folder

Yoshi
What does it mean to "remove an application"? Make this simpler by just saying what it does.
Remove empty($entity->type).
It just seems like an unnecessary extra command for people to learn
My general policy these days is to leave master untouched. I use this workflow:
[fork origin on github]
git clone fork
[set up "origin" to be the main repo and "fork" to be your fork of it]
git branch dev
git checkout dev
[do stuff]
(git add if you added new files)
git commit -am "description"
(if there have been substantial changes to master and you want to test them:
git checkout master
git pull origin master
((you can now test the current master exactly how it is right here))
git checkout dev
git merge master)
git push fork dev (creates new remote branch "dev" if it didn't exist)
[in github submit a pull request from fork:dev to origin:master]

Where "origin" is the main development repository (e.g. WhatPumpkin/Sburb) and "fork" is yours (e.g. threonin/Sburb). I've also played around with the inverse where "origin" is your fork and "mothership" is the main dev repo. That one is a bit nicer because all the defaults have you talking to your fork perfectly fine, which is the most natural behaviour.
Wrong class name?
I will probably joing them into a single regexp since both are so "close" and it should provide a slight performance benefit
The original one redirect to /dev/null, I just move to CD_TO_APP_DIR.

I think it is related to cd hook message of RVM. 

My rvm won't tell such message in this case, but I don't know if there are other cases, that's why i still keep it.

Should I remove /dev/null ?
As a note to myself, we'll want to have a compile-time check to verify that `sizeof(struct spd) <= PAGE_SIZE`.
typo? (yes, being anal, sorry...)
So I looked at this a bit. This is a really cool and interesting idea! But unfortunately, I don't believe it could be made thread safe without applying a mutex lock around the entire socket write operation. So it would make Grocer less thread safe, because exceptions could potentially inherit the wrong "last" errors.

Based on that, I would like to continue with the existing implementation. Thoughts?
In principle it doesn't matter which value you choose. You even could choose "false" or "0".
It seems that this is inactive/dormant, but it is a bad idea for a distribution script to mess around deleting DNS-records from the DNS-zone. It seems that it deletes everything less NS and SOA records. But will it delete MX, CNAMES, A-records, TXT-records (like SPF's e.g)? It kills quite more than records the script created itself. If it deletes MX-records, e-mail for the domain will stop working
also, tryna is not a typo: http://www.urbandictionary.com/define.php?term=tryna
I think this may be better done during ```prepareForDraw``` if only to avoid an extra null check on every willDraw.

Also, just an aside, if this is a common solution to choosing the right vendor prefix we may want to extract it and make it generally available elsewhere, but I don't think that's necessary right now.
=> resource:///modules/devtools/gcli.js
Nit: We usually use `# H1` instead of

```
H1
==
```
Should either test or remove for now this and `url` & `tabs`  proprerties too.
I don't think so. `this` can be different than `obj`.
Some web frameworks do something similar to this: `Object.prototype.toString.call(someObject)`
@Gozala I think you missed this comment ^
OK, if we are for sure escaping it earlier on the client-side, then that is fine.

This was just my knee-jerk dirty-in, dirty-out behavior.
My two cents: `"enabled": false` kinda makes me think that you must have a `"enabled": true` to make it work at all. I'm inclined to use the rule of "the default should be short", which suggests `"disabled": true` for one side and nothing for the other.
confirmed that these strings extract correctly.
This docstring is now out of date :-)
@jeffweiss based on ticket 17129, it sounds like we want to log the unreadable file on the server?
should this be a higher error level than debug? this means it will silently fail unless ppl are running `puppet master --debug`, right?
I see why PEP8 said to do this, but it seems un-Django like. Can we disable a filter or something to make it look like normal?

e.g. https://docs.djangoproject.com/en/dev/topics/http/urls/#example
Isn't this incorrect semantics? How can a "stashed method" be asked if it is stashed?
Nicely found!
comma after 'helper'
No spaces after opening parentheses or before closing parentheses according to java coding standards.
IMO this should depend on that the tweet were sent succesfull or not.
Opening brace on the same line. Applies to multiple places.
You're right. I thought it was possible to leave a post entirely uncategorized, but I see now that the default category is automatically selected in that case.
extraneous + ?
onBeginTransaction is used when the transaction is just started. onCommit is used just before the commit. I think both could be useful, one before any changes are made, and one after all of them are made.
Then look for a file that matches the regex "cplex_studio%s*.bin" % ''.join(version.split('.')) ?
Please add a comment explaining this code
We've had some issue without this workaround (ctrl.$error.date = !valid;). I'll check what was the problem... Fixed for this time.
I don't see this file in the tests/fixtures directory? Or is that what triggers the error?
Sorry to sound like a broken record, just wanted to capture the conversation in this pull request as well.

If one link fails the loop will exit and the test doesn't provide coverage on the other links. Is there a different way to do this?
so i would need to pass in an email then to log in using it?
I was thinking of `user__username`, which is, because it's a required field, or `user__email`, which will obviously always have a value. We could get to those through callables in the `list_display` list, but if there's always going to be some clickable string, then there's no problem.
This is not a very meaningful default value. Perhaps something like "Popcorn.js LowerThird Plugin" would be a better fit.
This line should be like

if ( media.offsetHeight && media.style.height >= 390 ) {
  height = media.offsetHeight;
} else if ( ... ) {
this file still needs some work
Note: if we have a kwarg saying that the values are all positives (or negatives) we could use blas' `asum` here. We can also use a C version using sse (simple) for maximum througput!
Ok, so this is the real controller, it is not a view
I assume it's whitespace.  I guess it's sensitive to each space.
I agree. However, the way the Hosts model is written, the "certname" field is not written to when UUID=false - it's always empty. That's why my original patch failed on UUID=false.

Do we want to update the certname field, or shall I jump straight to implementing the new short-term tokens table?
i wonder if it makes sense to have updae_multiple_attributes helper then :)
I've only set it to edit those required by the smoke test, i don't want to overload the test.
How about module_load_include in hook_init() ?
also: 
Warning: XSLTProcessor::transformToDoc() [<a href='xsltprocessor.transformtodoc'>xsltprocessor.transformtodoc</a>]: No stylesheet associated to this object in fedora_repository_object_details_XSLT() (line 73 of /mnt/share/drupal6/sites/default/modules/islandora/ObjectDetails.inc).
Apparently this isn't quite working.  (*shakes fist at IE8*) Gotta re-write for all browsers consideration.
Do we need a `brew update` here? Or is it OK without
I like the debug logs, but I feel like this one might really pollute the logs.
can you update the PR and remove this change? build files should not get modified.
Also, we should talk to @haus et all about whether or not /usr/sbin is the right place for this.
Chnaged it. Thanks!
This can just be a regular query as well.
Should have braces.
Indexing manipulation has been moved down to the internal elasticsearch.js API.
Yea that's a good idea.
needs quotes on the key here as well.
The schema is only cached when the server is restarted. Removed this line.
Fixed.

On 6 Sep 2012, at 10:49, Simon Gaeremynck wrote:

> In node_modules/oae-tenants/lib/model.js:
>
> > @@ -12,7 +12,7 @@ module.exports.Tenant = function(alias, name,  
> description, port) {
> >      that.alias = alias;
> >      that.name = name;
> >      that.description = description;
> > -    that.port = port;
> > +    that.port = isNaN(port) ? parseInt(port) : port;
> parseInt should include the base.
> ie: parseInt(port, 10);
>
> —
> Reply to this email directly or view it on GitHub.
>
>
I think there's a lingering bug in here. Currently, when a user doesn't have access to a piece of content, canManage will return no error, it will just return `null, false`. This means that if a user no longer has access to a piece of content, he will still be able to delete his own comments.

First of all, we should write a test that validates this scenario. In terms of fixing the bug, I think it's best for canManage just to return an error when the user has no access to the content.
you don't use the logger
what can be wrong here?


  [Symfony\Component\Config\Definition\Exception\InvalidConfigurationException]               
  Unrecognized options "base_url" under "behat.extensions.behat_symfony2extension_extension"
als ik in enemy-health.js direct health wil opvragen, dan wild hij dat niet doen. dit is de workaround
die staat er nu tijdelijk tot dat we animaties gaan plaatsen in milestone2. dan kunnen we animaties koppelen aan de movement
Actually, now that I look at it again - why not just make those the same as the function defaults?
'options' and 'selected' should be `array()`, while 'extra' should be an empty string.
Использован неверный метод для сравнения, мы создаем два объекта NSString данный метод возвращает NSComparisonResult который равен 0(FALSE) если строки совпадут, в итоге находятся совпадения, а не отличия. Верный метод для сравнения двух строк isEqualToString.
Так в моем методе initDNA так и сделано: self = [super init].
подход неверный, в этом цикле может 5 раз подряд удалиться значение с индексом 0, например, соответственно в массиве останется больше значений, чем этого требуется для мутации
Пересмотрел первые лекции. Не увидел там рассказ про переменные класса. Про методы класса есть а вот про перменные не услышал.
Ммм, а может эти файлы тоже в .gitignore сунуть?
If you can live with two commit messages (PR + PR merge), I can fix that...
Don't we miss some break statements here to avoid falling through the cases?
I tested this today and this is not necessary anymore. I think I changed this before last fix in postgresql driver. This is related to problems with null values. I will revert this.
this should still be a test dependency
Whats the reasoning for this return?
to  be honest I am fine adding n_jobs to apply_inverse_epochs although it will conflict with @mluessi that wants to return a generator but I don't like much touching that much the MNE inverse code for a gain that is a bit unclear to me. I understand to you want to apply the DRY principle but here I think we can leave with a few duplicated lines to keep the steps of the computation clear.
I do! I haven't actually uploaded any changes yet :)
projs += mne.read_proj(proj_fname) ?
We do that in all component documents.
I don't think duplicating the same note twice is a good idea
a word on the get_class name of the object to attach the transformer to would be most helpful. Do y@all guys know what are the 2 different type of objects involved here? thanks
The problem was caused by `file.mkdir` and `file.mkfile`, both didn't exist. I think there is a problem with the File prototype getting lost somewhere.

Using `file.driver.mkdir` and `.mkfile` works, but you have to use an absolute path `query.path + query.name`.
@bnoordhuis Also, I can't seem to figure out how to get `node-ffi` compiled without using `-mimpure-text`. This is a minor thing since I'll just add it to node-ffi's gyp file, but I was just curious if in your compilation mastery your would know a better way: https://gist.github.com/8aef6813e593df23f3d4
`InputMultiPath(File(exists=True), ...)` should be sufficient
Тупой кдевелоп
`const auto&` :3
I apologize; I should have explained my reasoning. Your implementation requires me to know what inputs to pass into the `order_choice` method. I chose to simplify it to `since`, which infers that you pass a date into it. And if you're not entirely sure, then you can crack open the model and see that it takes a date parameter. Does this make sense?
^^ same as above.
All this HTML should be done by template engine
You guys are right. I overlooked a few things.
`ends` should be `end`
Yes, Arrays.asList() already creates a new ArrayList.  See http://bit.ly/NGqIJ8
This is an interceptor, which should be able to handle be able to consume both evictions and expirations via the visitX methods.

This is equivalent to using a listener, but listeners are more oriented at the user (and they're buggy).

So, I'd suggest getting rid of the cacheNotifier, and handling evictions and expirations, in corresponding visitEvictCommand and visitExpireCommand (if not present, it should probably created - did you already look into this?).
Could this not just inherit from ```FFMegBase``` - or make a real baseclass-mixin that is the superclass of both this and ```FFMegBase```?
I'm not sure this change (i.e. 5 lines) is necessary. It confuses the issue a little (i.e. there is more than one way to do something).
That's fine then
This test does not seem logic to me. Should this probably read

```javascript
t.eq(exc, undefined, 'maximize and minimize do not trigger an exception');
```

?
I think that the spaces after casting operator would look better: ```$b = (bool) $a;```
grrrr. I thought I removed that agin.... sorry about that. push coming
Gah no, my bad. Never mind.
All-caps variable names are reserved for constants.
I'm assuming this will need to be translated?
please use SG_MALLOC instead of new[]
Will it come back?
Says "setdb" rather than "selectsession"
There is no reason to use a template like this for Preconditions., as it already supports an optimized format expression that is only calculated fully when required.
Must inspect this (and Xi\\Tests as well). Must they be here?

Anyways, Filebanksta doesn't care about autoloading Services_. It's just an optional dependency and not the responsibility of Filelib to autoload. It should handle it's own autoload (maybe via a pull request with composer.json to php-zencoder?!?)
Maybe it's just me, but I'd rather make two separate sentences instead of using a semicolon.
Are you sure ``$collectionName`` has the correct type as "integer|string" is not a sub-type of "string"?
Is it possible to define the element type for the array of parameter ``$chars`` (using ``array<SomeType>``, or ``SomeType[]``)? Please also check line 244.
Should the return type not be "null" instead of "BinaryStream"?
This has to be added or everything will crash...
Incorrect title?
Where are the coming from? #1362 had a change just like that.
These also should be three different test methods.
Not cluster safe !
That stands for the global cost of the event :)
Perhaps something should be stated at the beginning about the motivation for the "advanced usage" section. e.g., one can also introduce their own string interpolators- otherwise you get the motivation kind of in the middle
Remove space before colon (here and everywhere).
Please apply a consistent formatting style, i.e. enclose the paths with "*".
Oops. Not quite complete...
Aha, thanks, I did not know about this feature. Is it possible to specify minimal patchlevels for rubies? E.g. furnace-avm2 is known to crash 1.9.2-p0 and IIRC p180.
Add `r` prefix to those regex strings before and after. I know, that there is no reason for it, but it would be easier to recognize regexes so.
Unused strings should be cleared. Especially if they have been made by you during developing of this pull request.
Don't just comment it out without adding a TODO comment or some other comment with a reason.
`showUnread` -> `showHiddenPosts`.

So, you should tell that those functions are working with posts, we have a lot of functions around. And it is not really unread, it is hidden.
`var i`!!!
Good point on the exception, it put me in the right direction. 
I can't use get_or_create because we only want to search for label with label_name=name. This:
    label, created = Label.objects.get_or_create(label_name=name, query_string=query_string)
searches for label.query_string=query_string also, which we don't want.
Moritz, just read your blog post. Cool.

Question: what is the idea behind having all exception classes declared in a central file and not locally?

cheers
tobi
Was just a method of sorting my work so that I could keep track of what I was doing, sorry.
I totally agree with that. I am thinking whether the calculation of the upper/lower limit (which is done in the `RandomNumericSequenceLimit class`) should be really separated or not. If not, then that repetitive Fixture Setup of the `context` wouldn't exist.
Since Substitute's For method is a static method, perhaps you may be able to use the [StaticMethod](https://github.com/AutoFixture/AutoFixture/blob/master/Src/AutoFixture/Kernel/StaticMethod.cs) instead. You may also have a look at the [FakeItEasyMethodQuery](https://github.com/AutoFixture/AutoFixture/blob/master/Src/AutoFakeItEasy/FakeItEasyMethodQuery.cs) class.
remove 1 emty line
Indeed there is an issue, but your fix is incorrect, we should check that $form->getConfig()->getType()->getParent()->getName() equals country, not that genemu_jquerychosen_country contains country :)
Better:

If this variable declaration occurs outside of a function, then the variable `x` exists on the global object.
In the words of Homer Simpson. Doh!!
Why the error supression?
Agreed and amended, via a helper function.
Yes, also returning None is not a Pythonic way to handle exceptions.
shell_out! will raise an error if an error happens, but shell_out won't.
Please remove those spaces.
pep-8 function names are lower case, although I'm not sure in this case
optimality condition "for gradients"
accLocated:
A method that retrieves the specified object's current screen location.
for each galaxy in each file?
I find it more intuitive to write:
```
y2[0] = V(0);
```
rather than using a static cast.  Also, it is more traditional to use `T` for a POD type, although obviously not required.
Forgot to tell about this one. In D a string with the length of 0 is the same as null string. So call code like this, that checks if a `String` is null, should be commented out.
I will rewrite it to variadic function.
> Is it clear now?

No it is not.  Can you provide an example in the wild of an object property that is callable but not a function and of wich would not be considered a "method" in the general sense?  Is there an implementation of javascript in the wild that lets you define custom callable objects that are not Functions?
Suggest shortening up the lines for readability:

    add F @ $10 on 03/15/12 renewing every 1 month

To:

    add F @ $10 on 03/15/12 renewing monthly
So this stops pyflakes from checking that file, right? Why would we not want that?
Nevertheless everywhere in the code empty tags are xml valid. I think we should stick to it.
This template is used by the template tag ``generate_notification_description`` that generates markup for a notification based on the notification class. I thought this would make it easier to display descriptions of notifications that might have different classes by simply providing a template for each notification class. 

I see you point however and removed both the template and the template tag.
We can probably remove this shared-ptr param now too and use m_weakRef instead, right?
Oh, and it's generally good practice now to use `'execfile' in prog_ns` rather than calling `has_key`.
this appears to be an empty file
Put this in a "#", we already have the flavor text.
The RestAdapter used in AdapterFactory is default visibility. If I move it from out of pipeline folder I'll have change RestAdapter visibility
This will set the hostname to the devstack branch (in case you are working with different branches, you will notice inmediatly which devstack branch is installed in the box
Да пробела
Что не нужно?
Должен быть указан правильный package. Корень - это папка src.
it is the scope. 

    public String getAccessToken(String mediaServiceScope) throws ServiceException, URISyntaxException {
`for` loop over `["hover", "active", "focus"]`?
If we're going to have plugins for this plugin, they might as well be able to access everything.
`PersonalProject` に絶対パスを返すメソッドがあると(`personal_project.expand_path`みたいな感じで使えると)よいのではないかと思いました！
Red Hat family is actually "rhel".

https://github.com/opscode/ohai/blob/master/lib/ohai/plugins/linux/platform.rb#L92-93
why is it necessary to reverse an array?
``np.hstack([np.repeat(np.arange(n_classes), step), np.repeat(n_classes - 1, n_samples - step * n_classes)]``

Ok, yours is slightly more readable ;)
>> +        if y.ndim == 1:
>
> Yes

I tried but this breaks a lot of tests as a lot of linear estimators
do not support 2D Y

I don't want to do this right now
done.

On Wed, Oct 3, 2012 at 3:06 AM, Gilles Louppe <notifications@github.com>wrote:

> In sklearn/rbm.py:
>
> > +        v: array-like, shape (n_samples, n_visibles)
> > +
> > +        Returns
> > +        -------
> > +        pseudo_likelihood: array-like, shape (n_samples,)
> > +        """
> > +        fe = self.free_energy(v)
> > +
> > +        v_ = v.copy()
> > +        i_ = self.random_state.randint(0, v.shape[1], v.shape[0])
> > +        v_[range(v.shape[0]), i_] = v_[range(v.shape[0]), i_] == 0
> > +        fe_ = self.free_energy(v_)
> > +
> > +        return v.shape[1] * np.log(self._sigmoid(fe_ - fe))
> > +
> > +    def fit(self, X, y=None, verbose=False):
>
> As Mathieu said, verbose should be a constructor parameter.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/1200/files#r1747215>.
>
>
Here and for all other parameters: one space before and after the `` : ``, also end the description with a full stop ``.``.
I was wondering about that... I will inherit more directly because it is no classifier.
I think Random Forests is more recognizable, which is the main reason I choose the name. I will have a look at Gilles' references and see what they call it.
This is a schema version bump. :(
Yes, I've tested in local that the pattern replaces multiple slashes correctly.
Regex engine searches *(?<=regex)* and *(?=regex)*, but doesn't forward actual matching point by them. 

lookbehind and lookahead pattern example is <a href="http://dlang.org/phobos/std_regex#replace">here</a>.
As discussed, it's in the same class so it should be possible to access.
Why not use a QMultiMap?
Bad line wraps.
it is, isn't it?
Straigh*t*
should be a const function.
Checking that newly added lines/files obey the rules for whitespace etc.

This file is just a copy of the standard pre-commit.sample that is installed by git by default.
typo: s/findint/findng

But actually, I don't understand the issue/reason for comment here.
Can you explain a bit more why it it ok for !findCluster ?
Hmm... Why did we let this happen?

It implies that if, say, `product` is `None` you don't have access to a variable in the template called `product`. Thankfully Jinja templates consider non-existant variables as False meaning you can do `{% if product %}...`

However, I think it's dangerous to go down that PHP-style way of doing things. It's better to have it there. So we should be able just change these to:
`data['product'] = getattr(request, 'product', None)`
там є метод для перевірки чи плагін активний
nit: missing `else`
I don't think it' actually true as observer service uses a weak references for listeners, so they can be gc-ed without explicit un-registration.
The problem with this is that status code returned by this decorator is not "method not allowed" (it's a redirect)! But "ok", I'll do the change.
Another Erlang pull #178 has an example of additional dependencies in a "lib" folder (https://github.com/SublimeLinter/SublimeLinter/pull/178/files#L2R14). Is this common? Something worth considering?
『然后就像进行第二次调用一样的动作』，这里似乎有什么错了。
I think java convention is wrong.. instead of linkUsers you should user usersLink etc etc..
Consider using new Data Providers.. look at other recently updated test cases.. sorry for not updating sooner.
Single quotes here too.
maybe use
$this->showWorld()
with
function showWorld(){
     return $this->World->showWorld();
}
This doesn't seem to be the right way/place to do this to me - not having a RequestContext doesn't make a user a bot...
All looks fine now (sorry for the delay, didn't realize you updated the PR) but not sure why you removed the core. It's not useful via the dsn but if you specify it as separate args I think using path + core can be easier to grasp for people than having to append the core to the path. Also it's documented as such right now so it's best if it keeps working :) You can add a ->defaultNull() though.
No particular reason other than to follow the pattern but I moved it inside cacheFileName.
Why did user_owns_usr_local get deleted?
Any old SSH server........ what does the check command say?
Maybe 'median' instead of 'mean' just to be more specific. Also, I think the backticks should be replaced with single quotes (both `mean` and `gaussian`) since these are strings, not variables.