labofsahil/github-event-dataset-2012 huggingface dataset:
Can you send me .mdb you are trying to read (ideally with the steps how 
to build it myself)
> This breaks Cecil ability (and all the associated tooling) to read/write debugging symbols. e.g.
>
>
> Could not link assemblies: System.ArgumentException: An element with the same key already exists in the dictionary.
>    at System.Collections.Generic.Dictionary`2[System.Int32,Mono.CompilerServices.SymbolWriter.MethodEntry].Add (Int32 key, Mono.CompilerServices.SymbolWriter.MethodEntry value) [0x00163] in /Users/poupou/git/mono/mono/mcs/class/corlib/System.Collections.Generic/Dictionary.cs:450
>    at Mono.CompilerServices.SymbolWriter.MonoSymbolFile.read_methods () [0x00072] in /Users/poupou/git/mono/mono/mcs/class/Mono.CompilerServices.SymbolWriter/MonoSymbolFile.cs:594
>    at Mono.CompilerServices.SymbolWriter.MonoSymbolFile.GetMethodByToken (Int32 token) [0x00019] in /Users/poupou/git/mono/mono/mcs/class/Mono.CompilerServices.SymbolWriter/MonoSymbolFile.cs:608
>    at Mono.Cecil.Mdb.MdbReader.Read (Mono.Cecil.Cil.MethodBody body, Mono.Cecil.Cil.InstructionMapper mapper) [0x0000c] in /Users/poupou/git/mono/mono/mcs/class/Mono.Cecil.Mdb/Mono.Cecil.Mdb/MdbReader.cs:71
>    at Mono.Cecil.Cil.CodeReader.ReadMethodBody () [0x000a6] in /Users/poupou/git/mono/mono/mcs/class/Mono.Cecil/Mono.Cecil.Cil/CodeReader.cs:109
>    at Mono.Cecil.Cil.CodeReader.ReadMethodBody (Mono.Cecil.MethodDefinition method) [0x0001f] in /Users/poupou/git/mono/mono/mcs/class/Mono.Cecil/Mono.Cecil.Cil/CodeReader.cs:66
>    at Mono.Cecil.MetadataReader.ReadMethodBody (Mono.Cecil.MethodDefinition method) [0x00000] in /Users/poupou/git/mono/mono/mcs/class/Mono.Cecil/Mono.Cecil/AssemblyReader.cs:1983
>    at Mono.Cecil.MethodDefinition.<get_Body>m__34 (Mono.Cecil.MethodDefinition method, Mono.Cecil.MetadataReader reader) [0x00000] in /Users/poupou/git/mono/mono/mcs/class/Mono.Cecil/Mono.Cecil/MethodDefinition.cs:141
>    at Mono.Cecil.ModuleDefinition.Read[MethodDefinition,MethodBody] (Mono.Cecil.MethodDefinition item, System.Func`3 read) [0x00018] in /Users/poupou/git/mono/mono/mcs/class/Mono.Cecil/Mono.Cecil/ModuleDefinition.cs:815
>    at Mono.Cecil.MethodDefinition.get_Body () [0x00035] in /Users/poupou/git/mono/mono/mcs/class/Mono.Cecil/Mono.Cecil/MethodDefinition.cs:141
>    at Mono.Linker.Steps.MarkStep.ProcessMethod (Mono.Cecil.MethodDefinition method) [0x00163] in /Users/poupou/git/mono/mono/mcs/tools/linker/Mono.Linker.Steps/MarkStep.cs:817
> Failed to link /Developer/MonoTouch/Source/monomac/samples/AnimatingViews/bin/Debug/AnimatingViews.exe
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mono/mono/commit/70cc60e9572e3cd4ec69fd8c77340b1109eacbfb#commitcomment-981227
yea i think its safer to test for width >0..
Probably sane, but needs to be in a separate commit.
Bit of a :hankey: commit message really
1+ for lame
Because the `ajax_change` event is not limited to subpalette changes. Why not just leave it like it is? :)
On Tue, Jan 10, 2012 at 10:33 AM, Roland Kuhn <
reply@reply.github.com
> wrote:

> yes, sounds good, might be good to call that method on each check, though
> so that the Resizer has full dynamic control.
>

Yes, that's the plan.
Thanks for good suggestion.


>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/jboner/akka/commit/8b71bf5bea435cc0ef2caa68a3e6afa4ac01da77#commitcomment-852541
>



-- 

Patrik Nordwall
Typesafe <http://typesafe.com/> - Enterprise-Grade Scala from the Experts
Twitter: @patriknw
Yes; a simple boolean flag will resolve the issue just fine. Considering this was previous behavior, taking the route of less surprise is probably a good idea.
tam nahoře je <code>use</code> ne  <code>namespace</code>
It's impossible to search for a method when you are looking for "def
method" if it has self. at the front of it.

Problem? :)
I guess because Unicode isn't very sexy, it takes a while for people to notice any changes.
Timed out?! Oh noes, I broke GitHub!
No need to use partial here...
<3 @Dinnerbone
You need to update your iterator here. fMsgHandlers.erase(it++);
This test belongs in src/stdtest/tuple.rs - it won't be run here
you could store the `'stats.overview'` url in an `attr` and let the view accept `GET` parameters. then you could do

    $.getJSON(document.body.getAttribute('stats-url'), {'start': urlStart, 'end': urlEnd}, function(data) {
        $def.resolve(data);
    });
Huh? I didn't intentionally commit anything to vsvim.

Sent from my Windows Phone
________________________________
From: Jared Parsons
Sent: 2/24/2012 5:02 PM
To: Kevin Pilch-Bisson
Subject: Re: [VsVim] Fixed project file to build project (b3472ff)

Why did you remove BulkOperationsTest.cs and TokenizerTest.cs from the test project?

---
Reply to this email directly or view it on GitHub:
https://github.com/jaredpar/VsVim/commit/b3472ffc5319e644bda1d9c6b6dd51fe75ac5f39#commitcomment-1010002
You know, I just want to say that in retrospect this is pretty funny to look at.  I'm not sure what was worse, writing each character on it's own line for thousands of lines or that I was using hwords instead of bytes to store each character.
Hmm ok.  How about an RSS icon with a link on the top right of the table?
Google+ Hang out ;)
Can we pass CFLAGS in to make rather than patching?
;)

Oh, after playing with it for a while, I managed to completely screw up the moon transform.

I wonder if quaternion that pops up from `Matrix4.decompose` also has the same troubles like Euler angle coming from `Vector3.setRotationFromMatrix`. 

Just we don't have any method yet for converting quaternion into Euler angle. There is one here:

http://www.euclideanspace.com/maths/geometry/rotations/conversions/quaternionToEuler/index.htm

Though I somehow suspect with this conversion we would be back to similar problems.
Тоже лишняя обработка ошибки, если тут что то упало то надо валить весь тест
You have to manually inline `dup2` at the call site, since otherwise it will still need to do native code generation or at the very least updating the method cache. This gave me a significant speedup.
I think a line break is missing here; at least, GitHub doesn’t parse it correctly.
+1 for ``activate_translation(lang=None)`` in [amo/tests](https://github.com/mozilla/zamboni/blob/master/apps/amo/tests/__init__.py#L180)
@rjyo Oh, of course! =)
Ah gotcha, cool will pull that in.

WOM - means Weight of Money, its and indicator of which way the price may go.
I just did this because some of the other tests were set up that way. This test also fits the criteria of failing pre-fix and passing post-fix.
@pimvdb: Sprint4 has been discontinued, Sprint5 taking its place. I've made the change there now.
The commit message on this is ill-formatted. In addition, it's misleading since it doesn't say what this actually does on the first line.
True - sorry, with the break from coding at home last year, a recent update to Sublime Text 2 screwed my user config options. It's already fixed locally. FYI - after starting this class I realised it could just as easily be a standalone lib (probably more maintainable that way) and it would mean I don't need to repeat this process like I keep doing in my own work. Not sure what Composer's policy re new dependencies are however. Cleaning up what I'm doing with this elsewhere back into a simpler (and cleaner) single class include wouldn't be all that terrible anyway (see https://github.com/padraic/hasty for a WIP).
Please remove this string
Please create a pull request for this!
do you need to change https://github.com/vaidikkp/zamboni/blob/c9d62c/apps/editors/views.py#L499 as well?
In Python this is more idomatic as:

for i, ri in enumerate(rowv)

but because of the strange indexing, it would seem you then need to skip the first element.
In order to reproduce the error, put a breakpoint in function FormatCommand(), line "sb.AppendFormat("\t -> {0}{1} [{2}] = \"{3}\"\n"..." and cause an SQL exception (I executed a stored procedure that required an argument but forgot to pass the argument -- Query<>("EXEC sp")). You'll hit the breakpoint and crash.
I'd say this is more of an `IPSET_PARSE_ERROR` than an `IPSET_IO_ERROR`.  (The possibilities are defined at the top of include/ipset/bdd/nodes.h, which admittedly might not be the best place for it.)
No problem Dave. I'm diving into the gui now, so I will probably have some
questions for you soon.



On Sat, Jan 14, 2012 at 7:14 PM, Dave Ray <
reply@reply.github.com
> wrote:

> Thanks. I saw that happen once but hadn't looked into it yet.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/overtone/overtone/commit/823a6d29b51395e8ec3bc3d136f6bcd098a7d377#commitcomment-867976
>
Oops. :)  I threw that in there at some point b/c it seemed like the user's groups were getting cached and the following test was failing. But I can't get it to happen all the time... only very infrequently.
97 * 5 + (0)
@mbontemps told me they weren't :(
Yes, I tried and it works. Will update
1) Add a docstring saying what this is, perhaps a link to documentation where this logic comes from
2) call it interpret_ranger_cpu or interpret_amd64_pmc_cpu as I suspect this is pretty specific to ranger
how would the dirty state be reset to "false"?
if via the _setDirtyState method, then we should probably remove the underscore since it's public API
Yeap, I'm aware of this. I tried doing it by first collecting the entire text for a tag and then hyphenating. But run into quite a few issues. It is a big change from how the parsing is done now, so it represented a bigger change from the master branch and would be harder to maintain in the future.

In the end I opted to add the hyphenation in the collect characters, figured I'm max missing a hyphen every 1000 chars, can live with that. But it's a much simpler change from the way it's on master and thus easier to keep it tracking master in the future.

-- Guy

Sent from my Mac. 



On Jan 27, 2012, at 7:51 , Oliver Drobnik wrote:

> I've been meaning to tell you that there is a flow in doing this. With libxml you are not getting the entire block contained in a tag, but only until the next entity or max 1000 characters.
> 
> I think it might improve performance to always output an attributed string for an entire block (like the entire p) and possible construct it in a way that you can add the hyphenation wothout having to change the classes. Possibly with letting you pass a block or subclass the steing builder.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/gshaviv/DTCoreText/commit/6e8ae6cd34b67a2a72f906508b0cd7b59fb176fa#commitcomment-907032
Note that I had already fixed this in a more compatible way (i.e. with the req_ prefix as it should be!): https://github.com/nursix/ifrc/commit/b386a7af4da88751afa1c906f703a90d26245406
чёто меня вообще сломало полностью.
1. методы в конвенции - с маленькой буквы
2. надо иметь на каждый стат тогда филд приватный
3. для каждого филда - паблик геттер и сеттер (делаются автогенерацией в идее, см ctrl+n)
4. странная структура объектов. сттаы лучше вшить пока в плеера мб?
tite.
The URL was molested by a severe case of mavenitis ;)
Why create 'ta' just to save two extra letters and lose a whole lot of understandability in your code?
Hahahah esto no es una mecanica... deberia ser como "duelos" o gameplay o algo por el estilo.... fail para mi :)
LGTM

Can you make this change very clear in the release notes, as a lot of people use the pattern: WebServer server = create(..).add(...).start();
Bad news about Akonadi: "unviewable" messages still happen, but rarely. Looks like this is connected with filtering messages as I didn't see them in the "inbox" folder itself yet.
This keeps the given separator at the end of each chunk.

The native JavaScript command `split` removes the given separator.
If I want to keep it (like in the Ruby version), I need to use a lookbehind assertion but, because JavaScript does not support them, I use a lookahead assertion in a reversed string.

	>>> "foo#bar#baz".split("#")
	["foo", "bar", "baz"] // the separator is stripped.
	>>> "foo#bar#baz".split(/(?=#)/)
	["foo", "#bar", "#baz"] // with a lookahead assertion, each chunk begins with the separator instead of to end with it.

Finally:

	>>> _.reverse("foo#bar#baz").split(/(?=#)/)
	["zab", "#rab", "#oof"]
	>>> _.map(_.reverse("foo#bar#baz").split(/(?=#)/), _.reverse)
	["baz", "bar#", "foo#"]
	>>> _.map(_.reverse("foo#bar#baz").split(/(?=#)/), _.reverse).reverse()
	["foo#", "bar#", "baz"] // Yeah!
That's kind of icky. Probably best to write a script that runs all the tests.

At Mozilla we use django-nose and so it's a one-liner for running all the tests.
escape paying\_$var\_index
> I don't like the look of the three backticks together, but do appreciate that '$$$' isn't ideal.

IMO consistency would be useful (even important/crucial) here, so if you can bear it, that should trump personal taste.
@ahocevar I'm curious to know what Git command(s) you used to do this revert. When I want to revert a pull request I revert every commit making up the pull request (in the reverse order), and do not revert the final merge commit. I'm not sure what's appropriate and would be happy to share experience. Thanks.
und hier bitte mit match und möglichst nciht den ganzen satz so ist das nämlich zu Änderungsanfällig
This function will lowercase the rest of the string..
Not at this point, no.  This is the minimal change - the one that doesn't require major surgery to pass that data around. :(
Certainly.  I was responsible for testing the patch, and I thought it worked.  So it's not just you.

1.  Can't just edit an ipa, those things are cryptographically signed, and editing them will fail the install.  We need to edit the info.plist before the build process.
2.  There are potentially multiple plist files (one for each target), and picking the first one we find is not always correct
3.  Info.plist files sometimes contain spaces in their filenames so their names must be quoted in bash commands
4.  By appending the number to the version string each time, we run into a situation where we have runaway build nos (5.6.123.124.125.126.127...)
5.  Because Info.plist files can have varying names (abc-Info.plist, bcd-Info.plist) we must convert the xml version back to the appropriate file name, not just to a file with the exact name "Info.plist"

These are the ones I remember.
Due to the indentation issue, let me clarify that I simply added these few lines:

    case '=':
      $operator = '';
    break;
    case '<=':
    case '>=':
      return array(
        $model . '.' . $field . $operator . $values,
      );
    break;
This test is about check that the `hiera_hash()` function throws an error when it is not called with at least one argument. The function should fail the in the same manner whether Hiera loads zero or one hundred yaml files.
Really important to be able to quickly read through the timestamps when reading vitals of a patient, especially if several are taken of one patient in a day. Having to get the time by using mouseover will be an issue due to:
1. slow time to show up
2. some providers may even feel the need to write the times on a piece of paper while trending numerous vitals from one day
Suggest showing the entire timestamp without needing to drag over it.
Also, the use of the word 'At' is very difficult for translators and doesn't seem necessary.
Shoot - my bad.
Det kan väl vara trevligt att kunna markera [flera](http://twitter.github.com/bootstrap/javascript.html#buttons) (checkbox) state också?
Ok, changing.
FWIW, I think it would probably be fine to allow overriding.
I have a slight preference for the second one for the following reasons:

 - it doesn't keep the handle hanging
 - it doesn't rely on the garbage collector to free the libgit2 handle
Ah, yes I used the solution from `nebirhos.zsh-theme`. Instead of `if which rvm-prompt &> /dev/null; then` I used `if [ -e ~/.rvm/bin/rvm-prompt ]; then` and it works for me.

I do have RVM installed, and here are the plugins I'm using:

    plugins=(rails git git-flow textmate ruby brew gem mysql)

I think that the problem might be that `which rvm-prompt &> /dev/null` is returning false even if you have RVM installed, which is causing it to skip down to the rbenv section. Can you try using `if [ -e ~/.rvm/bin/rvm-prompt ]; then` and see if that works?
@ophian headers $Id$ expansion?
what use case are you looking to support?
same with this

```sql
UPDATE `smart_scripts` SET event_param2=7, action_type=15 WHERE entryorguid=18261 AND event_type=62;
```
Shouldn't this really be set as a much higher level default on a per-organisation basis? We set this in our site.pp where everything else inherits it, don't want a module to override that.
I'm hardly a Git expert, but I'll try :D
Ok!

On 1/24/12, Shlomi Izikovich
<reply@reply.github.com>
wrote:
> Probably just spacing. The method was changed in the previous commit as it
> had no try-catch clause.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/NServiceBus/NServiceBus/commit/c1793e9892f9f1b02de44ad2e811dcc87d1b3ce2#commitcomment-895127
>


-- 
http://andreasohlund.net
http://twitter.com/andreasohlund
# Homework 3

> Before I start grading the homework, I just want to point out that we will almost never be grading ahead (or at least I won't.)  If you aren't sure you have every requirement, send and email to `cs112@hampshire.edu` asking about it specifically and we can check it out for you then.  Otherwise we might just get to it after the deadline.

| **Part**  | **Points** |
|-----------|------------|
| base      |          1 |
| advanced  |          0 |
| hurting a TAs feelings     |  0.25 |
| **Total** |         1.25 |

Well done overall.  Step one was flawless and I genuinely had a moment of feeling insulted after that initial barrage.  Your code is clean overall.  Just a couple of notes so I feel like I am actually doing my job:

First of all, add some blank lines in your code to break up sections.  It makes it easier for me to read and to find problem areas in general.

```python
## There are a couple of things you can do differently with 

# original code 
n1 = raw_input("Number 1: ")
a1 = int(n1)

# you can reuse variables
n1 = raw_input("Number 1: ")
n1 = int(n1)

# you can write this as one line
n1 = int(raw_input("Number 1: "))
```

```python
# another example
total=i1+i2+i3
total1=str(sum)

# ...can be written...
total=i1+i2+i3
total=str(total)

# ...or
total = str(i1 + i2 + i3)
```

Since it looks like you handled this assignment easily, you should be able to understand string formatting.  You can find more information [here].(http://docs.python.org/library/string.html#formatstrings)

```python
# how you typed it
print "Prissybot: Hello there, "+name+"."

# with formatting
print "Prissybot: Hello there, %s." % name
```


Anyways, for hurting my feelings, I give you the [asterisk](http://youtu.be/JTVpxxzb2Dc) award

![award image](http://scienceblogs.com/pharyngula/upload/2006/10/asterisk.gif)
Has troubles when there are position:relative, in the element or its ancestors
I like the simplification of removing the return statement too.
pretty sweet patch will love to get it on the master branch, let me know if I can help with something.
Got a built .jar hanging around?
What does the new retention represent? Why the switch?

old:

    60:565920 => minute data for 393 days.

new:

    60:100800,900:6300 
      60:100800 => 60 seconds per data point for 100800 data points => 1 minute data for 70 days.
      900:6300 => 15 minutes per data point for 6300 data points => 15 minute data for 65.6 days.
This value should be passed as an option to FileUploader constructor rather than platform specific implementation like that.
Hi, the code looks good. Can you please provide a Unit Test, too ? There's no test so far for this use case and it would be good to have one. I'll merge afterwards.
`proto` is not initialized in this branch.
On Fri, Jan 20, 2012 at 5:52 PM, bobharner
<reply@reply.github.com>
wrote:

> Hey Massimo, I'm confused? Why the downgrade to 5.3?

Simply cause tapx heroku is based on 5.3 and not 5.3.1 ... I've
preferred to keep 5.3 other than excluding tapestry deps from tapx
heroku

-- 
Massimo
Veranderd voor consistentie.
Does really insight-6.8a contain gdb v7.3?
1. You document two parameters but the method only has one.
2. The return type is `bool`.
thanks, will update the code
Why do you ask for a access_token, instead of key, secret pair? I think it might be easier for user passing in just key and secret.
IT WORKS!!! 

Haha. After all this time. Thans sooooo much psyke. 

Now to get our kernel to 2.6.35....
Isn't this in 1.0.0?
Not CRUD actually but only retrieving filtered list. CRUD was added before this commit.
Should also change the android-maven-plugin settings.
Ah! That makes more sense. But why is my file being converted into the latter format then?
Huh? I get no test errors on OSX?
I will try to find out what the problem could be.
> I recompiled the same revision and it's working fine now. This has happened before too; something goes wrong with the compilation but it  doesn't abort. It never happened for me before using Maven, maybe it's unstable?

If PMS ran, it means the source code was compiled allright; so that part wasn't unstable. Thumbnails and broken playback seem to indicate missing binaries.
That should be visible in the size of the resulting PMS-setup.exe and in the logs.

> Maybe the external downloading doesn't properly verify the files?

Dunno, that should be pretty straightforward. Plus I think you would notice the missing output as it's a pretty long stretch of output on Windows during the build process.

It sounds more like NSIS not packaging the PMS-setup.exe properly.
Or the files not being where NSIS expects them to be at the time of packaging.
Or maybe Windows is locking files that Maven or NSIS would like to delete.

Next time you encounter a version with those problems, try to copy the Maven output and the debug.log.
They should contain hints as to what this unstability might be!
CSS3 standard versions should come after their vendor-prefixed equivalents.
sry :(
I just removed placeholder.jpg and it builds now.
> By the way - best NOT to include all PYC files in the repository.

If you look at the above commit you'll see that that's exactly the sort of thing it did: It deleted all PYC files and added them to .gitignore ;)
Also nicer (option 2). Something about ? : notation bugs me!
@Subv Sic 'Em! Rank 2 doesn't proc off.
And this latest change for the talent to count only 1 attack, doesn't work, the focus is still reduced for the entire aura duration.

This were the results when i tested.
if your deployement relies on the blacklist signal,  call it from your application's models.py. be warned you will never send messages to blacklisted users.
Yes, I have the unit tests already written as part of the iSynaptic library.  If you like, we could move a copy of the tests over to this project.
Si, pero acá estas dibujando un json y si no me equivoco esta tirando un error
That kind of compiled linker won't work with us 100%, the copied one from stock 2.1 is good - I use it in my aosp GB and CM9 roms
It is a feature, not an issue
In this function you build the shortcode as a string. I imagine it'd be cleaner to build a javascript object and then serialise it to a string somehow. It separates out the two ideas of building the data into one object and how to represent it.
Might want to wait until I get all my comments in. I'm still reading and pondering. :)
Use `Y.config.doc` instead of `document`.
"simplelist" should be "simple_list"
I think, we could additionally control here if possible timestamp is big enough, let's say, after the 2010 year.
actually should it call super in this instance?
it’s that mossy smell in the forest when the sun shines through the rainbow onto the freshly fallen rain drops?
I've done a publish-local to be able to verify and try the artifacts without any problems. That should have created the docs.

This file is also in test, if that makes a difference.
No, if the latch is open, then it allows messages to flow. If it is  
closed, it doesn't. A circuit breaker is the thingy that allows flow of  
things when closed :) But I can change the terminology.
That piece of code is not obvious what it's good for
I agree with Patrik reuse the same code for both and check instanceOf[InstantiationException] to determine the rethrow
stat: illegal option -- c
usage: stat [-FlLnqrsx] [-f format] [-t timefmt] [file ...]
./imgo: line 194: [: -eq: unary operator expected

MAC OS :-\
```python
if __name__ == "__main__":
    from numpy.testing import run_module_suite
    run_module_suite()
```

so that test file can be run separately
We should set up a skype or something to talk about this, It's not clear to me why you're using var(signal) / var(noise). Also it looks like you're implementation of the Gaussian noise scales the noise differently for each voxel. Even though there might be applications where this is useful, it's not physical for MRI. MRI noise generally has the same power over the whole volume.

Also, in the MRI literature there are two ways that noise is usually measured, either the mean signal to noise of diffusion weighted images or the mean signal to noise of b0(no diffusion weighting) images. I believe that the first is a more useful number, but because the SNR of dwi images depends on b-value some people report the second. I believe we should keep that in mind.
@domoritz this is looking better, but I'm still a bit worried about loading arbitrary resources like this. What happens if someone links to a 6GB file and CKAN tries to read it all? Is there any timeout option available for urlopen?
Maybe we should check the incoming headers for 'Content-Length'?

@tobes do you have any thoughts on this?

BTW I'm happy for you to use requests for this and add it to requirements if this makes live easier (it will probably do)
> Groups also enable you to assign roles and authorisation to members of the group - i.e. individuals can be given the right to publish datasets from a 
> particular organisation.

We could just copy-paste this into the orgs page :)
I'm now passing in the renderer and reusing it for all tasks in the task group. I cannot tell a difference between the output before the change and after, but as long as you're happy :P
Add delete here as an example too, I want to hammer that idea home as much as possible that a delete isn't instantaneous.
No way, they support icons. Replace my above comment to this one:

:+1:
```js
return d + "@" + data.dependencies[d]
```
I'll use =begin/=end in case we need them for something later.
Yup, agree
The `Server` and the `Connection` class are not the same namespace.
Isn't the second `if deprecated else ''` redundant?
it's all Quanming code, i thought it was mature, i will clean it now.
took a moment there, but I see it's recursive now... it could still return null, but only if no parent in your chain has a node.
Actually, LLVM 4.2 provides a niftier way to do this by defining the struct member like so:
```
(CGRect) {
    .origin.x = 0, 
    .origin.y = 0,
    .size.width = 32, 
    .size.height = 32
};
```
I don't feel really strongly about this, but maybe it makes more sense to return `nil` here? In either case, the return value should be documented, so callers know what to expect.
Really, because I was thinking just the opposite...
Also one here.
when you're talking 1,000,000 ops/s that's pretty negligible, especially for this function :s it's not exactly a hot one
It's never reassigned so use `function processImmediate() {` here.
Why not send the Content-Length?  I thought that was the preferred way since it allows keepalive to work and means you don't need chunking.
Ces lignes là devraient être dans le `<head>`. Pourquoi tu les as mis dans le body?
just a clarification 
 foreach(array('lang') as $k=>$v) ... (is_int($k) == true)
and
 foreach(array('lang'=>'cs') as $k=>$v) ... (is_int($k) == false)

Does it solve the issue? I dont quite get the comment you wrote... Thanks.
Thought it was just optimization, but I didn't think about the cost of ITER_RESET. Could do an explicit check before the loop instead to keep the optimization.
for this one i did not find a constant in the JCR API docs, but its listed here http://dev.day.com/docs/en/crx/current/administering/jmx-console.html
so it might be just jackrabbit specific
I'd actually just skip the try/catch here. Better to make the server die than print a bunch of stuff on STDERR but start up when it can't compile anything.
Please remove these modifications to the Gemfile.lock also.
Not sure I like the use of hard-coded HTML here. Would a `content_tag` not be better?
That could work, but it'd need to handle existing `Hash` instances too (convert them to indifferent access)
```ruby
ignore_none = options.fetch(:ignore_none) { num == :any }
```
I added a test here: 98de70e2e12cff98acbc38727e709ff99db331b6
N/m - synced with @tuncer
I'm talking about the system property, not the val.
Tricksy.   `breakCycle` more likely than past typer?
Great you like it. See the above dicussion w.r.t throwing exceptions.
Thanks, I did exactly as you proposed (move fromNow into FiniteDuration) and added a negative test so that `(x: Duration).fromNow` does not compile.
Matching? It's not a coordinated maneuver, it's just this transform:
```
val x = expr ; sideEffect() ; x
```
into
```
try expr finally sideEffect()
```
aside: mutexes are so much easier in C++ scope locking/unlocking. :-(
```

>>> sqrtdenest((sqrt(10 + 2*sqrt(6) + 2*sqrt(10) + 2*sqrt(15)))).expand()
sqrt(2) + sqrt(3) + sqrt(5)
```
comma after "nutzt"
>
> Is it possible to restore the output as it was (remove unnecessary space symbols)?

Done.
I agree. I've unfortunately lost access to the really mature functions `_print_Mul` and `_print_Add` because I removed the `Expr` dependence. I'll need to recreate these. This is a low priority for me right now. This is a regression. I think that it is a tolerable one.
Also, if you want to do these ask's, you need to include the ``assumptions`` argument.
On 13.04.2012 05:40, Christopher Smith wrote:
>>
>>       def __str__(self):
>> -        return str(self.dom) + '[' + ','.join(map(str, self.gens)) + ']'
>> +        orderstr = " order=" + str(self.order)
>
> ```
> s_order = str(self.order)
> orderstr = (" order=" + s_order) if s_order != self.default_order else ""
> ```

ah these pythonisms :-)
For some of the `assert False` statements it may be possible, however
there are some that are "undefined behavior" problems. For example,
what should happen if there are inconsistent equations? I can probably
use a `raises()` test. Anyway, you are right, I will try to minimize
the number of `assert False` statements.
`count_ops` gives, approximately, the size of an expression's tree:

```

>>> [(x, 0), (x, 0), (And(x, y), 0), (Or(And(x, y), x), 0)]
[(x, 0), (x, 0), (And(x, y), 0), (Or(And(x, y), x), 0)]

```

Hmmm...I thought the And(x, y) would at least give 1 op. Perhaps it should be replaced with just len(a.args).
Looks like something's wrong with formatting. Are you using tabs?
Strange formatting…
Clef, TimeSignature, KeySignature all don't have draw functions.
use b1
What do you mean? Is your change backward compatible?
Should be sqlalchemy.Column or just Column, but not sqla.Column.
Because that's what github give me by default. I don't like it too.
On Jul 18, 2012 9:24 AM, "Yuen-Chi Lian" <
reply@reply.github.com>
wrote:

> > @@ -14,6 +14,8 @@ Environment
> >
> >  Then, install the gems with:
> >
> > +    git clone https://github.com/Sinar/Kratos.git
>
> Am a better fan of git:// or ssh:// protocol, given the performance, and
> you don't have to deal with cert matters. Can I have some links (tried) to
> the actual reason why Git now recommends HTTPs by default?
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Sinar/Kratos/pull/1/files#r1184783
>
If you're going to use `with` you need to `from __future__ import with_statement` to maintain py2.5 compatibility
Unit test that demonstrates this, please.
in the buffer is left "200 another result", so next call to next_result returns: [200, "another result"] which is what we expect.

The idea of including EOL in result is that you could theoretically send an erroneous message to Cyc like this:

cyc.talk "invalid \n command"

and the result then will include "\n" in "500 {error description}" response. It works now, try it. (In previous implementation the result message was cut at first "\n".)

Since we cannot simply distinguish EOLs in response from result terminator, we must somehow sniff the data after "\n". The RESULT_MATCH is then used to check if data after "\n" is actually the next message.

Anyway this corner-case should happen rarely if at all.
let's cache the regex in the closure, like in exports.regexp
Верстку придется немного поменять. в rails 3 есть такая штука:

    render @events   

И все. Тебе остается тока сделать паршиалы `events/_event`  и `user/evenrs/_event`
missing semi-colon ;)
this isn't strictly true. master/slave replication isn't a strict subset of replica sets: slaves don't maintain their own oplog, and there are number of architectures you can get from master slave replication that you can't get from replica sets.. 

it's ok to admonish people to use replica sets.
Set? :)
Please capitalize PostgreSQL consistently everywhere.

Also please use or don't use terminating periods everywhere.
These should both be on the same line, as you have above.
this should be the same as `if (!empty($qm_ary[1]))`
I still would like `$lang_string = $user->lang($qm_ary[0]);` being moved into the function (its better for coming hooks)
Change wording to "Can view topics in forum"?
IMO, all three should be parsed. Also, aren't you supposed to add the flags together when passing them to generate_text_for_display, rather than using bitwise OR? (i.e. OPTION_FLAG_BBCODE + OPTION_FLAG_SMILIES)
Template var should be MD_DISPLAY_NAME
unrelated? there is no such language variable right now. maybe the function should use `$user->lang()`
hmm, this really should be in the schema changes and not here. We might want to get migrations merged first to make this easier. Or alternatively we need to look at how to get database_update to run two steps for one release. Or we just merge this now and split it up later when we have migrations.
This is roughly equivalent to is_enabled call but spelled differently.
Since it's static it does not matter much where it is, since there is no coupling. But phpBB mostly has functions for that kind of stuff, so maybe a `phpbb_avatar_normalize_row` would be appropriate. Not that I really care in this particular case.
should this go into pkg/monitoring like the rest of the packaging stuff?
Yeah, I just had a look at MySQL check. Will do.
ah good idea. i'll fix that up.

the propogateEvents function reemits the events to the ConnectionStream instance. Good idea, bad?
Nu șterge asta. În schimb, modifică emiterea de semnale să conțină action :)
GL_ARB_pixel_buffer_object was introduced in GLX 1.4 which is stable release already, I suppose this should be either a personal kludge. Also to stay safe we should provide a check for GLX version in CMakeLists.txt.
I had to add this because one of the tests fakes an HTTP request and emits each chunk as a string.
Yep, I've fixed all cases of this.
No need for slice as `map` works for `arguments` too.
not sure that's an improvement in terms of readability
These comments are very helpful, thanks.
abstract any such # to a variable, e.g. var TIMEOUT = 4000
can you forward me the file.??

On Fri, Jun 8, 2012 at 2:06 PM, shivamshah <
reply@reply.github.com
> wrote:

> > +                "links": "tok6-block-tok",
> > +                "id": 1
> > +            }, {
> > +                "uuid": "2283b157-fef7-4887-bea8-957c37a80796",
> > +                "display": "WEIGHT(KG)=45.0",
> > +                "links": "tok6-block-tok",
> > +                "id": 2
> > +            }, {
> > +                "uuid": "2283b155-fef7-4887-bea8-957c37a80496",
> > +                "display": "BMI(KG/M^2)=24.53",
> > +                "links": "tok6-block-tok",
> > +                "id": 3
> > +            }]
> > +        }]
> > +    }]
> > +}]
>
> no, john sent me a file and according to that i created this json file.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Raxa/Raxa-JSS/pull/73/files#r949627
>



-- 
Regards:
Vikas Singh
Is your webcam output a square image? It might be a good idea to get the dimensions of the webcam and scale down.
To be consistent with previous behaviour, this should really be `cm.coord_system('GeogCS') is not None`
"projection" -> "CRS" .... they're not always projections.
I make that *semi-demi*, actually. Like the quaver.
`self.schedule` is a defined method. We only overwrite it if there is another is provided. 

I used to put a function in here as a default argument. The issue here is that if you add a new function as a field then the object becomes unpicklable.
Гм, еще одна копипаста!  чем отличается код обработки issue от кода обработки комментария?... 

по-моему, ответ "ничем"
close is in unistd.h, g++ complains about it while compiling files using the timepps.h header.
This should be in cmdb/views.
level is pre-incr on purpose?
Логику по чтению директории вынести бы в отдельный метод
I can't figure out why this one is `public`.  It looks like all the mutations are executed by the server process.
Ah, right, I see - I think I was misreading the diff location. You're right that given the other change, this becomes a lot less expensive.
A `.. versionchanged:: 1.5` directive here would be very useful.
So, @reines: you are saying that this should be a 403 (because of the ambiguity)?
Done, I've squashed it into the original commit.
Yes, I like that idea. A map will make much more sense.
I opened a separate issue for this https://github.com/jacoco/jacoco/issues/59

I propose to implement this first on master and then merge it to here?
am I guaranteed `#qunit-fixture` is `position:static`?
I had that version as well but the broken unit tests made me wonder. The good thing about doing that would be it also fixes some hex-style strings like 12E4 that might get converted to numbers. 

One way to look at it is that we just keep dialing back the magic until we get a ticket that says, "You were supposed to convert this to a number!" We've never gotten one of those ever, but we could try.
I think we can adjust this logic to make this function more readable. There's something backwards if you're checking for invalid and continuing.
Should change the exception to atomac._a11y.ErrorUnsupported
Hi Kun,

Similar to my comment about the values of expected_crf_feature, could you explicitly document where these hard-coded values for crf_feature_test and expected_extraction came from since they are used to check against the MADlib crf output? Thanks!
the function ``` __create_training_instance ```  in ```decision_tree.sql_in```will do the job. After we get the ACC set, we will do the discretization for continuous features, and finally get the ACS set used to find the best splits. About the ACC and ACS set, please refer to the following description.

```
Attribute value and Class label Counts (ACC) set, which includes the following counts information for both continuous and discrete attribute. 
•	Counts of a combination of attribute value and class label occurs 
•	Counts of an attribute value occurs 
•	Counts of a class label occurs 
•	The number of records in the training table 
•	Is an attribute continuous or discrete

```

```
Attribute value and Class label Statistics (ACS) set, calculated based on the ACC set, which includes the following statistics information for both continuous and discrete attribute. 
•	For a <aij, ck> pair, the counts of  <aix, ck> pairs where aix is less/greater than aij, if ai is a continuous attribute 
•	For attribute value aij, the counts of aix where aix is less/greater than aij, if ai is a continuous attribute 
•	Counts of a combination of aij and ck occur, if ai is a discrete attribute 
•	Counts of a class label occurs 
•	Counts of an attribute value occurs, if the attribute is discrete 
•	The number of records in the training table 
•	Is an attribute continuous or discrete

NOTE: ai means the ith attribute, aij means the jth value of the ith attribute, and ck is the kth class label
```
Hi Kun, Where did these values come from?  Since they are used for the accuracy check, it would be important to document.
sorry for reviewing so fast ;)
That's part of it in fact, I need to move back those ids.
Do clients really need to care about Cpids?
Did jbalogh write everything in playdoh? He seems to show up all over the place lately.
Also, this should have more data:

```
log.exception('Error while live unindexing %s %d: %s', mapping_type, item_id, exc)
```
Is there a reason the url was changed?
I'm having trouble understanding these two lines. Maybe we can chat about it later. Was expecting to see a tail call here.
"while does not return" should be "while not returning"
Ist das nicht überflüssig, unser Autoloader enthält doch sowieso schon das "vendor"-Verzeichnis im Core und der kommt ja sowieso mit jeder Struktur klar.
DenormalizedViews are created from Thing objects and duplicate all the attributes so you don't have to go look up the original object. My thinking was that we may want to check that the object is _commited and not _dirty before allowing someone to create a DenormalizedView from it so it will be consistent.
I'll tell you what, I'll come up with a better strategy for the next added use-case.  Right now it's <20 lines and a pretty simple if/else in a loop.  But with more else-if's, we can push it to something table-driven or whatever best makes sense.
これ、envの中に PRODUCTIONとか定数作ろうって意味だったのです。
* コメントには//を使うようにしましょう。一般的に、/* */ はドキュメント用のコメントに使用します。
* XcodeはCommand + / でコメントアウト/インの切り替えができて便利
The packages have versions.
The junit.* packages have old version 3.8.2 that we are exporting.
Other bundle XYZ that imports this one junit bundle, and wants to import junit.* packages with version 3.8.2+ still can use. If the XYZ do not want to use these packages, then simply their import section Import-Package may omit junit.*.
That is what I'm suggesting :) I imagined we could just set it in the constructor and assume it exists everywhere... are there ever properties who's change events are never accessed and the lazy instantiation there allows for less objects to be made? I guess thats the tradeoff.
I don't  see the point of this.
I'll change second.css to point to third.css
This will lead to unexpected behavior. _enabledAgentNames becomes an alias for agents, so any time enableAgent() or disableAgent() is called, it will be changing the agents objects in addition to the _enabledAgentNames object. It would be better to iterate through the agents object and make sure all entries in the agents object have a corresponding entry in _enabledAgentNames.
Should doSequentially() be used here to enforce loading in a particular order (I guess testing in parallel is ok)?

That seems a bit wasteful, it would be nice to have something like loadInParallelThenExecuteSequentiallyInOriginalOrder().
Merge with master. This was changed from ``brackets-app`` to ``brackets-shell``
Nice suggestion! I also added Save in the new context menu.
Ah, I neglected to add them to the `updateSource` - they're there now
In node 0.8 there already is a native exists function on the fs object
Does the green block at the end of this line (and a few others) imply that your editor is adding whitespace to some lines?  Or cleaning the lines or trailing whitespace?
Refactored the function and added more comments.
we should clean up / abstract away these syntactic keys
As a side note, you mentioned in #127 about the use of `.bind()` all over the place: you'll find that its main use is not to assign `this` but to curry additional arguments to functions that need to be passed around, so we don't end up with callbacks within callbacks and it's easier to organise. For instance in this above line we're simply passing in the `writeFile` function as the callback argument to `SourceBuild.asString()` but we're injecting the first 2 arguments, `filename` and `callback` so they are preserved and `SourceBuild.asString()` is oblivious to them, plus we end up with smaller, more narrowly focused functions. It's the same as using [`async.apply()`](https://github.com/caolan/async#apply) (which you'll find a lot in the old CLI code).

There's only a few actual instantiatable  objects (in the traditional sense) in the code: `SourceBuild`, `SourcePackage` and `PackageDescriptor` (but that one will likely change) so they are the only places you'll find a `.bind(this)`. They also use `Object.create()` in preference to a classical prototypal setup with a constructor and the use of `new`.
Same as above. Use instanceof.
OK. Then use the same as it is in `LICENSE` file in the root of the repository.

```
# PiplMesh Slovenian translation
# Copyright (C) 2012 wlan slovenija, open wireless network of Slovenia
# This file is distributed under the same license as PiplMesh.
# Aljaz Prijatelj <aljaz.prijatelj@gmail.com>, 2012.
```

The last line is the same as `Last-Translator`, yes?
What's the purpose behind catching this KeyError just to raise your own?
I don't think we need this here, do we?
Same as above url comment
Change references to "Context Switcher" in this file to shrink.
I don't think we should delete the license
Try to make these comments look like others that already exist in the codebase if you can.
Also, missing a period at the end of "radio".
Also, does "title" get escaped?
If `width` is adjusted with `this.curMatrix[0] * this.matrix[0]` you don't have to do `* widthAdjust` here
Why `7`? Seems a little magic numbery, maybe either storing the value in a descriptive variable or writing a short comment before might be helpful. (Or it could be obvious if I were more familiar with code/results of the code).
Fixed in 5ccd3467.
I guess you mean the Webistrano::Deployer tests I wrote - they were pretty horrible.  That was a case like I mentioned above - I **was** having trouble testing a class without stubbing some of its methods, and that **was** and indication that the code **does** need to be refactored.  I just didn't want to invest the time to refactor it because I am sort of hoping it will get killed off before we really, unavoidably need to refactor it.  So please don't take that as a good example of how to write your tests :o)  Hopefully the other RSpec tests are much better examples.
Nevidím problém např. s *Přeloženo %i %%, %i řetězec*. Tohle potřebuje celkově vylepšit v originále, což v 1.6 udělám.
If there's only one video url, then we don't need to do the media:content thing. We only really need it if there are multiple video urls.

So for this section, there's only one enclosure, don't do the media:group/media:content thing.
i would recommend toLowerCanonic from misc.hh - it's more correct and also likely to be faster
Yes!  Partha is working on them.  Quite tricky to test some of this code.

On Wed, Jul 18, 2012 at 10:50 AM, Brad Buckingham
<reply@reply.github.com>
wrote:
>> @@ -0,0 +1,5 @@
>> +require 'spec_helper'
>> +
>> +describe ContentSearchController do
>
> I assume tests are coming? :)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Katello/katello/pull/327/files#r1188317
Apparently it takes lists as well as strings. I changed it in the most recent commit, though.
Change the name to ```GetDisconnectToken```.  The caller doesn't care that it's a GetOrAdd.
Sorry, I wasn't clear in my previous comment. One doesn't exist yet so it would be great if you can create it there.
Can this be broken out into another module function? It doesn't use anything in the lexical context. Also you could use `lists:foldr/3` in this iteration if you want to preserve the order (not strictly necessary).
no, it sets locale, no return needed