labofsahil/github-event-dataset-2012 huggingface dataset:
Yes, this is only because I wanted to cache the additional work I do here (parsing for `extends` blocks and compiling corresponding templates).
By the way, it should be done for `include` too.

Not sure the cache here is required anyway, it's just a matter of "executing regular expression" versus "another hash in memory".
Indeed, this is confusing. The reason is as follows: when the user presses the Run button (ButtonOnClickListener() in MeasurementCreationActivity.java) to execute the selected measurement, the value of the measurementTypeUnderEdit variable is used to select what measurement should be done. I need to have two types there so I know whether I should do an uplink or a downlink measurement
Result for this: Expected hit in cache.(missed = 0) expected:<1> but was:<0>
it was more the first part "uses the same code and file blocks described"..
@MilleTest : caused by trigger level = 1
HAVE_IRIX_IPV6 is never defined, there is no autoconf test for it, and no substitution target in include/config.h.in... So I assume IRIX can never have IPv6 support?
Didnt understand.
Damn it, got excited with the prospect of actually getting this thing running and forgot the rebase -i! :-/
Given that these scripts are one-liners with long name, just a shorter
command :D

Also, for maintenance, only one makefile with 4 targets.
Furthermore, as you insist to put in every project you make your scripts
instead of putting them in a ~/bin and add ~/bin in your path, a much
simpler command to duplicate them (one line cp).

Anyway...

:D

2012/1/17 denlab <
reply@reply.github.com
>

> why not, but what would be the advantage of a make file over a script that
> call the subscripts?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/denlab/w-import-gen/commit/6f9f9e0bc48542df7480ae36e26eb00f9fd9e8ce#commitcomment-873147
>
I definitely agree, although I'm too busy at the moment to volunteer myself to do it.

I appreciate the design ideas behind Apotomo and Cells, and as I continue to use it extensively in my project I am sure I'll be getting more involved in the documentation and code of the projects.

Thank you again for providing these projects, and for the insights to the design that your blog provides!

Warm regards,

Daniel Zollinger, New Zealand
Orlando, I totally need this for amon on node 0.6. Can you add a pull request for this to mark's repo and request a release?
Why the type check? Is there a better way to handle this?
I just meant that I was seeing an error like this when I refreshed so it was looking like I was getting essentially a 404:

```

Server Error in '/' Application.

Runtime Error

Description: An application error occurred on the server. The current custom error settings for this application prevent the details of the application error from being viewed remotely (for security reasons). It could, however, be viewed by browsers running on the local server machine. 
...
```

Looking at your URL, it looks like you're using nested lists for the two pages - is that right? Nested lists use the ```&ui-page=0-0``` convention in the hash and custom select menus will cause issues because that moves the # when it tacks on a ```#&ui-state=dialog``` to the URL so these are competing.

This is clearly a bug but if this works ok if you use a normal, multipage setup or individual pages then this will be low on our priority list. I *really* want to discourage people from using nested lists to embed full pages like this because it is just too fragile and there are a lot of constraints because we generate the pages (you can't customize the layout for example). Ive been trying to encourage people to either use a multipage setup or singel pages with the prefetch feature. Both will give you the same results, but in a better way.
(User.instance ~ Permission.instance
*).map(_.groupBy(_._1).toSeq.headOption.map {
   case (u, ps) => u.copy(permissions = ps.map(_._2).toSet)
})

On Fri, Jan 6, 2012 at 5:07 PM, recalcitrant <
reply@reply.github.com
> wrote:

> Very nice, almost there, now I am wondering how to get the User out of the
>
> anorm.ResultSetParser[Seq[models.auth.User]]
>
> so I can get the user as a singleOpt:
>
> val user = User.byId(42).getOrElse(...)
>
> def byId(id: Long): Option[User] = DB.withConnection(implicit connection =>
>  SQL(sql + "and ua.id = {id}" + order).on("id" ->
> id).as(User.withPermissions.singleOpt))
>
> val withPermissions = (User.instance ~ Permission.instance
> *).map(_.groupBy(_._1).toSeq.map {
>    case (u, ps) => u.copy(permissions = ps.map(_._2).toSet)
> })
>
> Thank you again for you time.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/playframework/Play20/commit/f35fd305d98038ab0fe3f8076368ee7f8d90015b#commitcomment-844047
>



-- 
www.sadekdrobi.com
ʎdoɹʇuǝ
I had it parse to null initially but it conflicted with some of the tests, did not try with empty strings though. Empty string to match nodejs' library would be better
why not use getter instead?

exposing raw property will allow a cancelled event to be uncancelled.
I don't think you need to make sure its long enough.. strncmp will just return that it doesn't match.
Ah, right. wireless and wired behave very differently here. Ok, operstate is the better check, then.
Контент блок хэлпер слева создает список доступных элементов для блокировки прописанных в CSS. Эта проверка для того, чтобы не ставился title к элементам этого списка. На сколько я это помню.
Yep.. there's 4 bytes missing there.. I'll fix it
Hi Tom,

Why do you want me to push someone else patch on the dev branch ?

Thibault
Le 18/02/2012 18:29, Thomas White a écrit :
> can you also push this to the dev branch, please?
>
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/LimeSurvey/LimeSurvey/commit/f495f51389775abb690e3c90c58f968e08ee537d#commitcomment-981324
Uh... did you actually really read the code to see what it changed? Only code in getMap() was changed, which is solely used by /f map.

https://github.com/MassiveCraft/Factions/commit/4b4d26ed29cb4f2e12837a70d06db73774fa6698#commitcomment-911012
li should be inside the if, so in case the condition is false there is no need for empty li tag.
does cassandra's style guide have a line length limit ? I'd stick to 100 characters otherwise
Tested in VoiceOver and JAWS 12 (IE 6/7/8, Firefox 4/5/6) with no problems whatsoever. Great stuff, guys.
Yeah, I forgot to take that out.  Sorry.

On Fri, Jan 27, 2012 at 10:14 PM, milt <
reply@reply.github.com
> wrote:

> Hey, did this month select ever actually get implemented? I'm inclined to
> take it out, as it doesn't seem to work, and is pretty useless. I can't
> even figure out where @month is coming from, it doesn't seem to be in the
> controller anywhere, AND YET IT RUNS
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/milt/redmine_scheduler/commit/ed9b1cd13538e334f88f75197e6672d859e4a81b#commitcomment-910422
>
Any reason why this was undone mpriour? Alyssa had asked to remove the unload prompt when leaving the page.
agreed - it shouldn't be necessary
Also, would really like to see a test case for this to make sure it does not regress later.
sp. likelihood not likelyhood
most of our tests clean the data they create. It helps on test reruns.

In this case you would delete the object you have created
When you say that COPY FROM is much faster than individual inserts,  did you remember to put a transaction block around your insert statements?   One transaction is going to be a *lot* faster than 50,000 transactions.    Without an explicit transaction block, each individual insert statement is a transaction.

Also,  did you test against an insert statement that inserted multiple rows at once?    This has the added benefit of requiring fewer round trips between the client and the backend.

COPY FROM should still be faster than these,  but I'm curious if you are doing a fair comparison here.
No problem – nothing to be sorry for! Really!
In master branch, go to extension preferences -> scripts -> click save on bottom -> switch to other extension tab (may be 'magic') -> go back to scripts -> list is duplicated twice -> save -> switch forth and back between extension tabs -> list is duplicated 4 times ->...

Remove comment in:
```//pos = rule.indexOf('##');
//if (pos !== -1 && rule.length > pos + 2) {
...
//}```
go to extension settings -> url-filters -> press save -> list is cleared
with how many ppl online? (average)
cezeta: No.
now that im rechecking
i get loads of these:

2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.
2012-01-08 20:30:19 ERROR: BattlegroundHandler: itrplayerstatus not found.

i hope that this doesnt affect everyone since i tested a beta verison of this fix
so do I in general, but I also dislike repeated tests... 

You could (without ill effect) remove cyclesNow variable altogether and simply call get_cycles_now twice if you wanted.
Maybe inline childUris and parentUris (they seem to be used only once).
hm.. did this break anything?
It looks like this gets added because of this commit: e76c078326e920a80e1bbc6264e7e00d1c942d49, which added a pdf file.  Did you pull the changes back down to your repository with `git pull` to get the `.gitignore` file on your local computer?
Intersting, I'm on Ubuntu 11.10 w/ Gnome Shell and haven't come across this issue, we have a number of devs on the the same distro.  @masteinhauser are you running Fedora Core 16, could you try installing this?
Great! So now it won't overwrite my translations when updating :)
Change this: either "...the stages key to..." or change the preposition "to" to "of" or "in"
Discutamos...
I changed the preamble to say "WTF, bitch?"
Mouuh , ser godt ud !! Roterende globus mangler :(
compiled with this new set of commits, charging works via nook charger, and status is displayed. However with a 500ma line (From pc) it does not show charging. Which is probably true with the screen on, but with the screen off it should be charging, and I can't confirm yet that it is or isn't. 

Hop on IRC some time eh? =P
Psst it wasn't Zeerix.
Sorry Jen :(
/**
 * Hello World from Ryan
 */
you need to remove hdmid from device_thriump.mk
deze functie moet na loadBarChart() komen (coding standards = alfabetisch rangschikken, ook al heeft da weinig zin ^^)
**Set**GlusterVolumeOptionVDSCommand
See https://github.com/D-Programming-Language/phobos/pull/392
which fixes the abs hijack issue.
Have you looked at the quickstart provided? If so, please provide me specific problems you are having.
https://github.com/onmyee/Xperia-X10-Kernel-2.6.29/commit/475e04296c6f05aa04d2ebd35dfdc5c9b70d31b3#diff-0
So what about breaking code that uses `valHooks.checkbox` for instance as you told me?
I think this plus the `@TranslateGroups` feature of BVAL-208 (?) will achieve what I am looking for
https://github.com/carz2/samsung-kernel-crespo/commit/417c2c58d447d68ad34998c10a9d85f514dbaeb3

this fixed it for me
You should fix this...
webcfullscreen.xul:    <key id="focusURLBar" disabled="true"/>
webcfullscreen.xul:    <key id="focusURLBar2" disabled="true"/>
webcnoaddressbar.xul:    <key id="focusURLBar" disabled="true"/>
webcnoaddressbar.xul:    <key id="focusURLBar2" disabled="true"/>
webconverger.xul:    <key id="focusURLBar" disabled="false"/>
webconverger.xul:    <key id="focusURLBar2" disabled="true"/>

I have it so CTRL+L is disabled in webcnoaddressbar since you can use
it to override the purpose of missing URL bar, so folks can't navigate
just anywhere. :)


CTRL+L should work fine in webconverger.
I think it would be fine to add passed-in fields to the default ones.
since the apache_request_headers function will work only if php was installed as apache mod ( http://php.net/manual/en/function.apache-request-headers.php check the change log ), i had a problem on the live server 
so i have change it a bit to check if its exist or not ..
if its not found use the getenv() function instead ..

am not sure if my code is right, but it solved my problem ..
check this commit ( https://github.com/linuxjuggler/codeigniter-restserver/commit/fc1d9a8022f4a0f558eb5ddd960e2571f2fb30a1 )
i tried to make a patch but i dont know how to do so , and the pull will take all my commits , and you just need this one .. sorry
Here's a commit containing the driver :
https://github.com/erasmux/hero-kernel-2.6.35/commit/c3cf7e34042f021dfbab9902197118cf428935de
I just thought of another solution to this problem.  If we do need to use `;` in a key/value pair in the future we could add quoting or escaping.  Thoughts on this versus removing comments at ends of lines?

Also, should we allow using `#` for commenting at the beginning of a line?  Currently this is allowed (due to inih).
Whoa, >= is way dangerous here. It's is expected that the interface to the ey-serverside gem changes with minor version bumps.
This means the code will now break on ILP64 (int=64bits) and LP32(int=16bits) (reference:http://www.unix.org/version2/whatsnew/lp64_wp.html). None of the major OS/compilers use either models but is there a way to detect datatype model at compile time and issue a warning for those two platforms ?
Editing isn't being friendly with my phone so feel free to submit another commit to fix it. it wasn't mentioned in that commits notes so I'm sure it wasn't intentional.
Может сделать как выше с tNum? Т.е. `(this.hash.match(/\d+/) || [])[0]`. А то где-то будет строка с номером, а где-то массив с единственным элементом - номером - возможны ошибки.
Why are you writing to "exportFile" instead of fileName?
@brettp I don't know why I didn't check `$_SESSION['last_forward_from']`
Ah. I know why. `JpaDaoSupport` is also deprecated. I'll reverse the changes, and then do a separate pull for that one.
That's basically what happens in ActionRun.start().  I think this false is being used as a token to say that the render failed when viewing the rendered command later from tronview, and to have something to serialize.  We could expand the string to include more information.
Can you not use awaitCond(..) for the 'isAvailable' expressions instead of the sleep?
yes, it contains the convergence flag, perhaps that is confusing
Yes, but only one will win. How can you avoid that the colliding threads  
do not create competing versions?
Ah, so much nicer. Great.
This breaks actor encapsulation by writing to an internal field from outside of the receive
no need for `val aref`
Yeah, I think we should just migrate the db and fix it once and for all, but that it a small detail and a simple change.
Not newly introduced, but the '*request+*to' thing should probably use traits, too. If your eventual solution to the contants thing above involves the (reasonable) assumption that there is already a *request card, it could be possible to reuse?
would `sum(not_none).dtype`, `reduce(np.add, not_none)` is very slow.
given the liberal usage of tertiary operators in line 186, i would have expected this conditional to be something like:

    result *= colOrder === 'asc'? 1 : -1;

:smirk:
have to note that these excursions took us way over the line count.
this is to expose mailboxSize which is protected in Actor
а че ты убрал это декоратор?
Sorry about the delay in getting to this. Should this be `< 6`?
Estas usando un diminutivo en esta linea, seria cuentaResultados o simplemente resultados
These constants should be factored out too.
This isn't a conventional pattern for `+initialize`. Please see my previous comment and make this match.
This addition should be in a different pull request, also.
In Guard we have all methods and constants documented, so we have superior API docs on rubydoc.info. The inline comment should be moved to the method comment. We also have always an empty comment line before the method:

```ruby
# Quiets tmux output
#
def turn_on(options = { })
```
OK, makes sense. Just add the test as I mentioned and it should be good to go.
Remove that log output, it would be issued at each http request.
Have you tested calling these from a third-party module? I think this won't work because of the funny way linking works for dynamically loaded Python modules, and these functions will need to be added to the API via the machinery in numpy/core/code_generators.
Who supplies this? Coming in a later commit?
Is the name curry OK?
Noticed this constant isn't qualified with `ActiveSupport::`
Just noticed, can you use the new hash style please? Thanks :heart:
We can use Hash#fetch
Ok, have split the asset processing part out into a separate `#process_assets` method that is called from `#compile`
fix translation "Ver más…"
Although I guess I'm looking at the "-" part, so that's not very useful.
I think none of the `*Ref` classes should be able to capture it, since the `@static` field is never a local.
Or did I miss some case where we lift a field to a `*Ref` class?
Ha ha, lot of good that will do you.

https://issues.scala-lang.org/browse/SI-6217

    package foo
    package _root_

    object Test {
      import _root_.scala.Option
    }
good point, you found a missing test case; your UPDATE is not enough, though, due to the `-0d` sickness it needs to use `compare` instead of `<`.

BTW: is Double (and Float?) the only example of a violation of the equals/compareTo contract on the JVM which actually ships as part of the platform?
Not all trees `representing types` are TypTrees
wow, super intuitive!
Shhhh! Don't spoil the surprise!
Can the offender explain him/herself? ;-)
Is there any particular reason why we are raising the exception. Repeated calls to to a vectorized_lambdify function will raise the exception when adaptive sampling.
Alternatively you could add the ```@cacheit``` decorator to this function. The caching should happen automatically.
> Yes, thanks. I think I get the general idea. It's a little like
> MatrixBase, isn't it?
>

Yes, indeed.
Chris,

I have made the changes you indicated and have made changes to address
the snags that sympy-bot kicked up.  I hope this is getting close.

Comer

On Sun, Apr 22, 2012 at 9:03 PM, Christopher Smith
<reply@reply.github.com>
wrote:
>>      M = Matrix(( (x,y,z),
>>                   (1,0,0),
>>                   (y,z,x) ))
>>
>>      assert M.det(method="bareis") == z**2 - x*y
>>      assert M.det(method="berkowitz") == z**2 - x*y
>> +
>> +
>> +    def test_det_LU_decomposition():
>> +        x, y, z = Symbol('x'), Symbol('y'), Symbol('z')
>
> I updated test1 with your changes and made a few more over that. Do
> they look ok? (You can see which are the changes I made by looking at
> https://github.com/smichr/sympy/commit/88e4fe014772e1a70b6446c868a85e527b704195
>
> /c
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/1253/files#r718483
Thanks! 

Still though you were right to ask for something better. Aaron brought up this point as well. I'll give it thought.
I have not changed the original text, only the formating and examples,
however you are right: While I am at it I can as well write the text
in a nicer manner.
> It was for a comment so that a type error wouldn't be triggered. I don't get an error if the lines are commented out. I do get an error if they are left uncommented. So at least on my machine this seems to work.

I'm still for showing the error message, since it does not disrupt the flow of reading the documentation (well, _this_ is a general unfounded statement, but I hope it's rather intuitive to sell as true :-) )

> What is the standard practice in this situation?

I'm not sure what the _standard_ practice is, but, for one of my own classes, I have found the following way to test for exceptions:

```
>>> Diagram(f, g, f * h * h)
Traceback (most recent call last):
...
ValueError: All generators must be expanded generators.
```

Basically, as the output of a statement which raises an error, one should quote the first and the last lines of the error message.

Personally, I find this more natural, since this actually tests for the exception and also looks more natural when reading the docs.  This is a question of taste, mostly, I'd say.  Note that, apparently, a lot of code in SymPy does the `>>> #` trick, so again, I'm not sure what the standard way to handle such situations is.
Ok, will make these changes and update the pull request soon.
That's http://code.google.com/p/sympy/issues/detail?id=1973 by the way.
Note that the delete statement is there from before, it was just moved from init() to processLogs()...
Shouldn't it be up to the client as to whether or not they'd like to display messages by the appblade sdk? iOS version doesn't do this, right?
acds
Können wir beginTransaction und commit auch in den ServiceWrapper mitnehmen?
Shouldn't this point to https://github.com/etsy/logster.git?
well, I just copy-pasted whatever was used in Route
why? the cap makes it less beautiful and simple, and also disagrees with the rest of the crumbs and toolbar. also, you also added unicode flakiness. I'd like to keep the placeholder to `search` the way I set it please
s/4xx Errors that occur that as result of/4xx Errors that occur as a result of/
A task for another day: test loading up some data to verify constraints (like fk and uk) - not just for this..
nit: leave the `:` here? It wasn't immediately clear to me that the hard cases were handled outside the `match` instead of unimplemented.
This would be more readable if the ternary were the other way round:

```php
'log_data'    => (sizeof($additional_data)) ? serialize($additional_data) : '',
```
The underscore after the other ones is because of the way those files are named.
I think this is here because I had added something here at one point and then (re)moved it but didn't remember to remove the comma. If I have to remove this, I can, otherwise I'll leave it.
then right here do `self.dev_name = params.details.target`
În loc de `if rule.enabled`, am putea modifica filter de mai sus, să fie: `filter(applies_on='login', enabled=True)`
Why are there two comments for the same function?  It should be one comment I think.
punting, since problem wasn't introduced by the resource store redesign.
I think you should use `__dirname` here. I think `@configuration.bin` for npm or homebrew installs would be `/usr/local/bin` rather than the realpath under Cellar.

``` coffee
libexecPath = "#{__dirname}/../libexec"
```
agreed, this change should not be included in the pull request.
> I see that the 8.6.8 release is on sf now. I'll certainly fix that.

Would it be possible to decouple the work on asciidoc from the other changes you've made in this pull request?


 > Curiously, ascidoc is written in python, so the only additional dependency is mercurial, right?

Plus autoconf.
Style: for readability, prefer not to use abbreviations ... HtWtID would be better as HeightWeightId
same goes for all files
Keep this as well.
move store out of view -- @nundys which database should we be using for pharmaceuticals?
That would put every node in v's dictionary, not just the nodes that act on v.
Remove ``to`` in ``they can easily to hide errors``
You have some misspellings. "tha", "candidate" must be "candidates"
Yeah, believe it or not, I just did an experiment going through and cleaning up all those parseFloats... it was -1 byte :-\. Although, I might have done something wrong.

I took your suggestion for right now, and I'm going to sleep on ways to make it a bit less obfuscated. Of course, give it a shot yourself in the meantime!
Yep, see boost/random/bernoulli_distribution.hpp.
But its going to throw some file exceptions because of the windows-only paths, no?
Actually, postproc is enabled by default now so we've removed the entry entirely - https://lists.ffmpeg.org/pipermail/ffmpeg-cvslog/2011-May/037246.html
I would rename this _check_niimg_methods
Don't support all these options - just 1 is sufficient.  This is a temporary thing and extra code paths are confusing
Translations are made in [Transifex](https://www.transifex.com/projects/p/owncloud/) and generated nightly.
shouldn't this use valo instead of val.valo?
Yes- this hasn't been a problem in the past as no one else was working on anything and even fewer people ran the tests. I'll _try_ to be more explicit about what I'm working on.
We can rename this function to `send_message` and add a `compose_message` with a deprecation warning. That is the recommended way of handling such changes.

What is the other breaking change you'd like to make?
Ok, now it works properly.
Thanks for clarifying that. (I've avoided maven for years, and now I'm mainly a Clojure guy, I can still manage to avoid it.)
I mean like this:

```php
if ( test for memcache ) {
    ...
} elseif ( test for APC ) {
    ...
} ...
```
No need to cast ss_len to socklen_t
Aw cool.
no worries. fixed
I think this comment is not required anymore.
It just spammed me with like 20 emails...  I'll do i tomorrow; working on
Graphics now.

Dan

On Sun, May 6, 2012 at 10:41 PM, Gareth G. Aye <
reply@reply.github.com
> wrote:

> > +}
> > +
> > +/* Variant : for right positioned triangle
> > +------------------------------------------ */
> > +
> > +.trip.right {
> > +     margin-right:30px;
> > +}
> > +.trip:before {
> > +     content:"";
> > +     position:absolute;
> > +     bottom:-20px; /* value = - border-top-width - border-bottom-width
> */
> > +     left:40px; /* controls horizontal position */
> > +    border-width:20px 20px 0;
> > +     border-style:solid;
> > +    border-color:#5a8f00 transparent;
>
> Can you just align/format this plz :)?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gaye/rideshare/pull/17/files#r779652
>
Maybe a better approach would be never to send anything through to superfeedr that points to localhost, as that's clearly never going to be appropriate.
The header has been changed
And check also for other fields.
What if user with such username already exists? Add TODO comment for that. (Or fix it.)
"in the dist/cornfield directory in order to run the server in production mode"
At first, it looks like `removeComma` is following a functional approach: not changing the input. But, it turns out that the `string` variable is probably not even required here if you're just manipulating `tempJSON` directly.

So, let's choose one or the other. Either return the morphed string from this function or remove the `string` variable and just manipulate `tempJSON` directly.
use a mixin here
Same, handlePositions
This is probably wrong, yes?
Can't believe I didn't notice this but you duplicated a variable here! Surprised lint doesn't complain.
scott's drawing algorithm assumes that it's drawing the entire timebar in one pass (look above, the inc is for the scrollWidth, not the canvas' width anymore). I translate by -scrollLeft to bring the part that should be drawn into the drawable part of the canvas. Getting rid of the translate calls here would require rewriting the drawing algorithm below, which is why I stuck with just translating the canvas and... manual culling haha
Yeah, that's a solid list.
make this line `return result;` after replacing ` if (operations == null) {` to `if (!operations) {`

the `result` will always contain at least one item
Looks like you meant to rename it to [playdoh.wsgi](https://github.com/mozilla/playdoh/blob/master/wsgi/playdoh.wsgi)?
Arguably, this could check CAP_SETUID/CAP_SETGID, but in practice - meh :) You could scrub the cap bits to be extra paranoid.
Similar comments to the above about 'there should be a new app'.  Now that we're referring to 'the app' in the step, I would expect that app model to be in an `@app` instance variable in this step, which would be set in the prior step where the app is introduced into the scenario.
Using `e.inspect` may not give you the best information about the error, but it's not a big deal.
No good reason. I just can't think in negative priorities I guess.
"    %V (shared), nginx dso version (%ui.%ui)" -> "    %V (shared, %ui.%ui)"
修正しておきます。
I'm pretty sure SiteNews.updated is always set. The field uses `auto_now`, so on each save the date is updated. The field doesn't allow null values, so that's another hint.
parameter name being used for tobeCommittedPaths and argument name doesn't match here...please fix to ensure code readability
Unused code, right? Just checking :-) ACK
Ma nisam odma bio skuzio da ste stavili pod type= "food resource" , pa po tome netreba dodatni resource type
got it I just changed that
Similarly, perhaps `MissingDataAssociationError`?

Also, can you add one-sentence docstrings that describe what these errors are for? Right now they appear as just blank doc pages.
I thought I had fixed this - @taldcroft, I'll check to see if I forgot to merge in my PR.
Why'd you drop this step?
Last rebase just added this comment.
Why is this inverted? Won't it hide all non-coinbase txns? O.o
I'll get this changed.
We shouldn't use the keyword swiss, because if we use it in France it become wrong. It can simply be:
all: "{coord_label} : {coord_x} {coord_y}<br />" +
Can you explain what it's supposed to fix?
I thought that latter was more clear and it doesn't rely on `plot` existing.
Unfortunately, we cannot add `lex(1)` [documentation](http://pubs.opengroup.org/onlinepubs/9699919799/utilities/lex.html) as a comment. Please refer to Extended Description section, the second paragraph.
`find.package(name)` ?
this should be a set of distinct hardware in the available nodes
We have recently switched to "CanCan" gem for performing authorizations. Could you try merging the latest code and see how we can use that common framework?
Isn't there a way to rather special case the `HEAD` in `add_ref()`?
Boom! **[He's dead, Jim](http://www.youtube.com/watch?v=MH7KYmGnj40)**
Can't this just do `criteria.exists?` with the above mentioned changes?
Good stuff, will do. I did not feel good about the code duplication anyhow.
yay for bringing this over :)
OK, that makes sense. For some reason i thought text_phrase wasn't even being used at all for any fields except the two listed. But I read the code again realized it applies text_phrase to all the fields.
Sure, I'll make the changes later for the permission.
This file needs updates as mentioned on the appropriate bug on Bugzilla. But it's a good start.
You get exactly the same with:

```python
for x, y in enumerate(fts_app.source_iterator()):
    ...
```

I wasn't sure so I wrote this:

```python
iterator = xrange(3)
for x, y in zip(xrange(1002), (a for a in iterator)):
    print x, y

iterator = xrange(3)
for x, y in enumerate(iterator):
    print x, y
```

and the output is

```
0 0
1 1
2 2
0 0
1 1
2 2
```
should this be `dest`?
I'm just going to backup the Marketplace CSS changes for now 
instead since my Marketplace install is broken and I can't test 
it properly.
Good catch. Don't know why I left that in there.
what about throwing an exception if the user tries to dump already existing constant names? reflection would be needed in that case, anyway, I'll rework on it if the feature is accepted ;)
you shoould allow injecting it, to allow using a different implementation
missing spaces around the comparison operator
let's wait for a review then, if the bug is still there that's probably not the good fix anymore as this is not the master branch
@beberlei?
The getter needs to be added in the interface too
ugh, I'll change this to os.path.join(self.management_dir, filename)
It turned out that the slowest part was this wrapEach function...
no space after ")", should be:

    + (BOOL)hasFeedVenues;
by convention autoCheckoutForRegion(CLRegion *)region

also your casing on Checkout does not match your casing for CheckIn or setCheckedOut.
I believe these `event`s can be built with `build_stubbed` in all cases.
Style: sort validations alphabetically by attribute name.
Will this automatically be cast as UTC or should we force it?

```ruby
Time.now.utc
```
oops, i meant to put that in the supervisor child subprocess environment field per http://supervisord.org/subprocess.html#subprocess-environment.
Worth changing them all then?
image location variables?
Do we need this? `{nodes|safe}` seemed enough for the template
The problem doing a `super` call to `create` in the implemented `create` is that it is going to be blocked on `p.wait()`. So after the super, your own code is going to be useless because the `subprocess` has ended (and everything was fine). We need to stop that `subprocess` in some way.

That's why we needed to rewrite all the code :sheep: .
py3k complains because of octal value, maybe we should use a constant from http://docs.python.org/dev/library/os.html#os.chmod
this is part of what needs to be checked for #431
https://issues.jboss.org/browse/PRODMGT-135
I meant, reported against the buggy component.
要。今天的工作不得完成。主要功能已经完哦，国际化和修饰界面一起做。
`gotAlarmListHandler` -> `callback`
Give some names to some anonymous functions.
is it needed here? Request is used to confirm dialogs like, do you want to uninstall xxx app?
I meant as it is done at https://hg.mozilla.org/mozilla-central/file/08d435dedc7f/dom/network/interfaces/nsIDOMMobileConnection.idl#l53
Nit: typo (“**Building** Blocks” rather than “Buildging Blocks”)
Maybe we can do a retry, or incremental retry, but I guess we don't need to tell the user we couldn't remove from the pending db a message that was sent correctly
We should use `settings.createLock()` now.  https://bugzilla.mozilla.org/show_bug.cgi?id=788561
This file looks like it is the main entry point to the app.  Is index.js a conventional name for that?  If not, and if that other calendar.js file is deleted, you could rename this to be calendar.js.
In principal, I believe that the idea is to have a text field in the landing page for searching anything on Internet. I don't know if the idea is a "search" web activity or for example a "navigate" web activity to a URI like this http://www.google.com/search?q=blabla Do you have more info about it @dcoloma ? But in any way, I think that we will need a web activity to implement this new feature.

thanks Ben
yes, that should work as the children array contains element nodes
I found a way to implement this. Pull request updated.
parenthesis are mandatory, always.
Should we compare .to_i or is there some reason for compare .to_s ?
Line up variable assignments with the three top ones, add an extra carrier return after the five has been declared.
You can just do (if (and file (stringp file))...
Code style: `n == s.length`
good idea, I changed it it in the code. Do you want me to regen the training data?
No, you didn't. This isn't a TODO, it's a "Note".
use the assertion helpers like the other BIFs, to report the name of the arg etc https://github.com/mythril/stylus/blob/1a5d157771e71a5003b61b57cc44104f8ba02089/lib/functions/index.js#L153
One problem with this is that the parsing sometimes goes several files deep, so if say firstRoomDialog.xml has an error, it logs an error for firstRoomDialog.xml then firstRoom.xml then openbound.xml. Maybe add a filename property to the error with the name of the file from the innermost Sburb.loadSerialFromXML and then suppress the other messages?
You should use the parseCharacterString method for this. That way keywords like "char" get interpreted meaningfully, as well as any future keywords that might be added.
Do we have something similar done? If we do, could you point me where it is, please?
If not, this is not a big dial, I just want to make the pattern recognizable.
nyuck nyuck nyuck
Extra points for using Math.pow()! ;)
looks better:

```coffeescript
method = if _.isObject controller.before
  'executeAction'
else
  'executeFilters'
```
is that a standard ?
This could also be used to rescue_from ActiveRecord::RecordNotFound,
which would allow to render the 404 page and use layouts, translations and what not...
I know this was like this before your changes, but we should avoid lazy static initialization.
I've tried to fix it, but not sure if it's the correct way to do logical 'or' in m4 macros.? Please have a look to Sty.c.m4. Thx
Oh, ok. Should probably reread the http spec =)
Ja das wäre einfacher :)
habe ich ganz vergessen dass es rtrim gibt !
Interesting.  So could they name the container after the type, or today's date, or?  What is the value proposition?
Would there need to be any checks to make sure it's valid, or let them suffer the consequences of a bad argument?  And if we check, what constitutes a bad argument?  Catching an error in calling the sprintf seems like a minimum.
O arquivo já tem o código, não sei porque estava dando problema
With this style of simple entries, the column order is nondeterministic. Unless JS uses order-preserving dictionaries.
Oh, whoops, I misread this. It's actually in the right order. Guess the ones below should be reordered. :) My bad.
If that would help I could do following instead:

```js
let defualts = {
  sandboxPrototype:  {},
  wantXrays: false,
}

override(options, settings)

if (options.sandbox)
  sandboxOptions.sameGroupAs = sandbox;

...
```
address_info should still return a type of "primary" here and not let this bleed through to the front end.
So, @6a68 and I decided that combining these make more sense because it reduces the cognitive load on the developers. Going to modify this in the morning.
Sounds like this new branch is missing a patch. BrowserID should be using well known to find the auth url. Up until this week, there was a special rule that address_info used a hardcoded /proxy/:email, but that is no longer the case.

Making sure your bigtent is up to date...
If you do curl -v -k https://yourhostname:yourport//.well-known/browserid you should see 

    "authentication": "/authentication",

If not, can you update and restart bigtent?
Uh, so the `Puppet::DSL::Context` is a proxy to a class named `Proxy` which is where the actual implementation lives. If I'm understanding that correctly, this class needs to be renamed. Is this maybe `Puppet::DSL::Actions`?
distutils isn't smart enough to see root_numpy/tests. You must explicitly add root_numpy.tests to this list...
Arff, my bad! I'll fix this now… anyway, who cares about darwin ;) No kidding: thank you and I'm sorry.
Is this `before` block really necessary?

Can't we just put it inside the `it` block? Since it's a single spec I think it's unnecessary to create a `before` block.
IMO the `framework` keyword is all you need here. Zend and ZF are present in description/name etc. The search will pick up on those already, and these really have nothing to do with what the package *is* or *does*. It could be zen, but I doubt zend is a state of being :) Of course the components themselves should have keywords about what they do more specifically.
`@param bool $wrap`
Thanks - actually, that whole conditional is worse than redundant because of the line just before it!  I've altered the commit to remove the conditional altogether.
fixed, thanks for that :)
Merging [464](https://github.com/pypa/pip/pull/464) would result in a conflict here (resolving it should be straightforward)
should not be on a new line. move this to the line above
if (is_array($dataValues) && count($dataValues) > 0) {
=>
if (!empty($dataValues)) {
Ah, ok, thanks for clarifying.
that catchall import error broke debugging on a regular basis
The _nominal_ tests fail, as if the nominal src package never got imported. For example

    ERROR in (test-nom-2) (core_deftype.clj:527)
    expected: (= (run* [q] (fresh [x y z] (nom-fresh [a] (nom-hash a q) (== q (clojure.core/seq (clojure.core/concat (clojure.core/list x) (clojure.core/list y))))))) (quote ((_0 _1))))
      actual: java.lang.IllegalArgumentException: No implementation of method: :runnable? of protocol: #'clojure.core.logic/IRunnable found for class: clojure.core.logic.nominal$_nom_hash_QMARK_$reify__3731
     at clojure.core$_cache_protocol_fn.invoke (core_deftype.clj:527)
        clojure.core.logic$eval6142$fn__6143$G__6133__6150.invoke (logic.clj:165)
        clojure.core.logic$cgoal$reify__8147.invoke (logic.clj:3016)
        clojure.core.logic.Substitutions.bind (logic.clj:1147)

In other cases, it's not doing nominal unification, for example:

    FAIL in (test-nom-4) (tests.clj:84)
    expected: (= (run* [q] (nom-fresh [a b] (== (nom-tie a q) (nom-tie b b)))) (quote (a_0)))
      actual: (not (= () (a_0)))

The test works when I run them in emacs in the repl, but fail with lein test. Some configuration issue?
some IDE don't recognize it. you should use ``float`` which is the proper name for the type in PHP (``is_double`` is only an alias for ``is_float``)
Replace `\n` with PHP_EOL
if you return false for both ``false`` and ``0``, there is no need to handle them separately.
This is unnecessary -- Rails automatically provides dynamic finders for you (e.g. `User.find_by_<attribute>`, `User.find_all_by_<attribute>`).
>
> You added a default status code of 200 in constructor and headers are not
> overwritten once they're set.
>
But... setting the property won't send the header, will it ? What am I
missing here ?

--
Bertrand Dunogier
Lead Engineer Extensions
eZ Systems AS | skype:bertrand-dunogier | http://www.ez.no/
---------------------------------------------
Freedom to Create Your Experience - Join us at this year's Conference
http://cologne2012.ez.no

eZ Publish Enterprise 4.7 Available
http://ez.no/About-eZ/Events-News/News/eZ-Publish-Enterprise-4.7
Tiez nezabudnime, ze eventualne chceme aj casy (pricom to vysielame vtedy ked aj datum aj cas sedi). Ale predpokladam ze to si odlozil na neskorsi commit.
maybe should we provide a setting to change this?
I think this query can be optimized to do the filtering on the db side instead of in the Python code. It doesn't matter right now since a video is in so few playlists, but it'll come up in several months.
add the fact that it preserves the order in the docstring. (and add unittest to make it clear)
http://www.simunova.com/downloads/mtl4/MTL-all-4.0.8878-Linux.tar.gz returns a 404
On Tue, Sep 11, 2012 at 12:12 PM, Kenneth Hoste
<notifications@github.com> wrote:
> Please fix these.

Done; I'm sending this message in case the inline feedback I have
appended is not visible...

please review.
@jsocol Is it ok to have the link here? I don't remember anything specific to this.
you should do this in __init__.py but not here.
sorry, thought i had removed that
I think this might read better if 'None' is returned instead of sys.maxint

If I'm reading it correctly, returning 'None' would be already be compatible with the `if next_day and prev_day` logic added in risk.py
:) Ya probably not.
From what I've read about drupal_static it doesn't need to be used in cases in which we are sure we don't need to reset the static variables during the request. 

We do need both variables to be static, this function does get called many times during menu_get_item(). I find it more obvious without it, but I'm not opposed to using it.
Ooh, very hand - fixed!
this is done because we don't care what the name of the artifact is - we just care about the version solution
Yes they should be internal.

I'm a bit unhappy about ->value being public currently. I may change that in the future too.
Having any public property is kind of a bad idea at the moment. If I update ->value manually, ->arr would be incorrect, and the opposite is true too.
get часто не пишут. 
count --- это считать. количество --- number
How would you check that? Or shall we throw the exception when we execute it? That should be much easier
Might be due to some meta programming in Rails...
Should `!hide` this comment.
Great catch. Fixed.
@bp323: This will probably conflict with your work, but I think this means that the getGlobalServer function can now be removed in favor of this,
This function should be dropped in favor of the Cassandra.constructUpsertCQL function
> Yes, you're right (right now it would be <nosymbol>, but I can turn that into <undefined>).

Right. But if the ``symbol`` is ``null``, then it would just be skipped. Furthermore, from a user-perspective I guess it would make more sense to have ``<undefined>`` (or ``<error>``), rather than ``<no symbol>``.

> The best way would be to have a nice pretty-printer and print the trees themselves (I don't understand why the original code uses only the symbol name, since you might have a more complicated expression that a simple reference to a symbol).

You lost me here :-) Would be great if you could give an example.
Add comma after second to last item
Rake's default behavior is to only ever run each task once. Are you sure this has the same effect as the original alias?
Didn't see the last comment until I had committed - it's EPG_TRACE now, but with an additional log prefix that allows for easy grepping.
Bleargh icky! But tests, so this shall pass, I guess?
the load_from class method can not call the private method process_config.
I have seen many similar blocks but this looks like it is going a bit far. Debugging in there is going to be tricky.

If wasn't perfect, but it spread this out.

Of the same block of code is used with some one-liner comments and a space above each it wouldnt be so tough.
Um ... that's not right.
наверное лучше исключение генерить в таких случаях
Создав свойство таким образом, вы должны понимать, что автоматически создадутся геттеры и сеттеры для данного свойства. Раз уж на то пошло, свойство нужно делать readonly, тогда создастся только геттер.
@slebedev при чем тут nil и сообщения? Это обычная практика, если в init если не проходит [super init] - возвращается nil. Просто в данном случае мы даже до [super init] не доходим - ибо незачем при неверных начальных условиях.
хоть бы ошибка какая выскочила, а то как-то нехорошо получается. Передаёшь ему значение - а он ничего не делает. Совсем ничего. Нипанятнааааа... (это я про отрицательные и значения > 100)
That's what `indexOf(path) === 0` is for. `replace(...)` will only replace the first occurrence and we only reach that point if there is a match at the beginning.
The "version", "branch", and "SHA" are generic enough that someone could add these in another group and cause a naming conflict. Do you think it's worth making these names a little more unique, or maybe keeping track of which group you're in? Or keep it agile and fix it when there's a problem?
No, there is also the next line, where the constructor is specified as the second-to-last argument to declare, rather than inside the hash.
common-dylan
Maybe we could just use HTML::Table here, and expose some of the options to the user with a config file?  Looks like the module takes the DBI handler directly: "(-data => \@) straight from DBI::selectall_arrayref".  That way you wouldn't have to worry about formatting too much.  I wonder if it makes sense that there's a fragment output as well as fully formed page, or maybe one that includes a jQuery datatable pagination and inline resorting and querying?
should it not be getOrElse(null) to make sure oozie deals with it correctly?
I'd actually been meaning to remove this method entirely. Any objections to doing that instead?
I like switching shake_id to shake in this situation
I think we should leave the logic or angular position outside of the plotting function ar at least be able to pass so kind of circular layout file. We could ship a layout for neuromag sensors and FS aparc.
You need to put all the logic in these if clauses inside one if clause since you are checking for the same thing in all the if clause