labofsahil/github-event-dataset-2012 huggingface dataset:
These are the informations we currently have at hand: {"lock_version":0,"category_id":null,"is_private":false,"fixed_version_id":null,"description":"Kiwi","rgt":2,"parent_id":null,"author_id":1,"tracker_id":1,"estimated_hours":null,"done_ratio":0,"updated_on":"2012/01/23 12:25:11 +0100","priority_id":4,"assigned_to_id":null,"start_date":"2012/01/23","subject":"Another test issue","status_id":1,"root_id":2,"project_id":1,"created_on":"2012/01/23 12:25:11 +0100","lft":1,"id":2,"due_date":null},

The last update would be helpful along with the status and possible the assignee, but it looks like we'll need to run something like a lookup to retrieve them from their ids.
А деплоишь куда? Это для проекта? Или для личного использования?
Может тебе доступ к серверу своему дать?
Oh, one more thing: without a Gemfile, I felt kinda bad not specifying which version of `rake` we were using. So I actually think I prefer this for that reason.
+ be
Ah I totally screwed up the syntax for `slice` thanks for bringing this issue up!
@asseym make sure to you update your database (removed this duplicate)
This is probably silly, but technically it may not be safe to call `NSSelectorFromString` outside of an autorelease pool. (I generally assume that API is guilty unless proven innocent for this.) It might make the code nicer to just have a `SEL deallocSEL = NSSelectorFromString(@"dealloc")` at the top that the rest of the code could use, and this block could capture.
Warum hast du diese Zeile ohne Absprache wieder zurückgeändert? Ich finde speziell diese Zeile mit UH-Prefix wesentlich verständlicher. Zudem hätte dann konsequenterweise auch Zeile 185ff. geändert werden müssen ... Der tatsächliche Code sieht anders aus, das weiß ich, aber das ist auch inkonsistent, weil die Datei immer noch UHMapLoader ... heißt. Ich bin daher dafür, die Änderungen rückgängig zu machen.
Needs to be:
...  'secret' => $this->AWSSecretKey));
(two close parens)
I not sure why you need to add the dependency of release plugin to BuildConfig.groovy as it is not required by the jquery-validation-ui plugin itself and the release plugin can be installed by using install-plugin command.

Please advice.
Thanks, issue is resolved.
It is added to the upgrade guide as part of this pull request
Sorry for the delay. I've just made a change to the Karelia repo. Is that enough to solve your symbolic link need? It seems a little cleaner to me than this code if that's OK>
If I remember correctly, you can't run that through the 'as_user' function because doing so loses the directory context. Whenever running something through the 'as_user' function, it defaults to the user's (in this case minecraft's) home directory. So now you're only copying directories in minecraft's home directory into the ramdisk.

I could be wrong, but I'm pretty sure the 'as_user' function works like that because I had problems in the past where I'd run a cd command through the as_user function, and the script wouldn't stay in that same directory unless you ran your other commands inside of the same instance of the 'as_user' function.
The above 10 lines are where its more than a cosmetic change. If there is any error, it would be here.
First of all, thanks for adding the Spanish translation!

However, I think the correct order would be the following because at least in Argentina weeks start on Sunday (domingo):

    weekdays : ["Domingo", "Lunes", "Martes", "Mi\u00E9rcoles", "Jueves", "Viernes", "S\u00E1bado"],
    short_weekdays : ["Dom", "Lun", "Mar", "Mie", "Jue", "Vie", "Sab"],
    very_short_weekdays : ["Do", "Lu", "Ma", "Mi", "Ju", "Vi", "Sa"],
    first_weekday : 0,
    weekends : [0,6]
That’s a LOT of ceremony in this case. Still want it?
To be able to check for JOIN before checking for UP.
see my comment below: this is replicating the supervisor’s fault-handling, so I’d simply throw an exception in this case.
Timeout is for ask
:-)
no, it's sum of the squares
.empty[T] rockar fett!!
Since you linked the ticket to this pull request, do you want me to stop recreating the pull request so it's a clean commit and instead add additional commits on top of it?
aesthetic comment.  I get why you put `redirect_id =` here, but I think in multi-line ifs like this it's actually a little easier to ready if we just stick those on the last line, even if that means repeating it two or three times.
Hm, my flake8 didn't say anything, but i think you are right, it's not the usual practice for naming functiont.
Hum - ``sum`` doesn't cast the same as numpy though:

<pre>
In [14]: type(np.float32(1) + np.float32(1))
Out[14]: numpy.float32

In [17]: type(sum([np.float32(1), np.float32(1)]))
Out[17]: numpy.float64
</pre>

...
Again the special masked arrays--why not just store a dense ndarray?
Not too sure about this even though its quite clever.  Would prefer different extension point then to use convention of action names.
if type, data_type where actually on table would be able to filter query better here meaning someone with lots of subscriptions would not be so slow.
Ok, I'll proceed with adding a second test that hits the DB.

Considering your points about integration testing, forgive a very brief rebuttal:

Separating integration tests from unit tests is the difference between unit tests taking 5+ minutes vs 200ms. Then we could let hudson run integration tests, and developers can still feel very confident pushing changes after a 200ms unit test run.

I also don't think it's fair to say that we can't verify anything without hitting the DB. We can verify exactly everything except the interaction with the DB.

I definitely see the desire to not change anything about the layout now, so we can punt this and chat about it another time if you like.
Rest looks good.
I swear I double checked before mentioning the test case. Still getting used to reviewing in github like this. :)
Should we document which ones do and do not? Or perhaps just say "if you try to cancel a task that does not support cancellation, this is the output you will see" and show an example.
Set this up on init
remove and add a new line for dav keeping old formatting. diff you should be 3 lines or so.
Please pull the else onto the same line as the if `{`

`} else {`
Sorry, I expected to see all the methods supported by a filesystem, with docstrings and `raise NotImplementedError`. It's verbose, but it makes it a lot easier for someone reading the code to understand what these filesystem objects do.
this may not work if you need to escape stuff in there ( \  "  ' )
This should be `raise`, not `return`.
Sekalian index.php-nya di taruh sini saja
persist: false,
readOnly: true
请解释一下你对`cache_control`的理解
The reason I had the floats here was that when the window was too small, these numbers would go from one row to like 3 rows. Would this problem come back by removing the floats ?
Unix

Mac OS, *BSD
please define a macro for this, such as CONST_STRLEN
Actually, I just documented and exposed a few `TUIControl` things to make the `TUIButton` function the way it does. No new `TUIControl` features (aside from system control tint notifications… which are for the buttons too.)
`weak` should not be used-- use `unsafe_unretained`.
This needs to be set for basically every deletion method.
the performance should not be the "key" point here as this should only be executed if we suspect the cpu bug here. So not quite often.. So If you guys thing nanoTime is the way to go I will be happy to change that ;)

I just would like to know what you think about the "workaround" in general ;)
* DOCTYPE is usually uppercase for bc, but html5 does not care either way.
* Missing <html lang="en"> ?
This RETURN seems not to free all temp structured previously allocated.
I don't get why not. If you have a syntax error your app won't actually work. So it's a matter of whether you'd like to start some zombie apache anyway.
Yes, it could.
Not sure about the default alias here. Presumably we need to have one? How about just calling it "subquery"?
@lunks, I saw that before you opened the pull request. Like I said, it's up to rails core to change the style after they merge it, if they even decide to merge it.
I copied from this line https://github.com/frodsan/rails/blob/2f81be178fd000956974d53ad265fffa58b50090/actionpack/test/controller/caching_test.rb#L48. I will look into it.
It took me a while to think of a good test case for this. (.we Tours)

Wunderground always passes back the full structure. Fields that are not present are empty strings.
Should this catch AppliedTypeTree too?
https://issues.scala-lang.org/browse/SI-6374
See the comment for CaseDef
I'm not super happy with the name, since it's user-facing. How about `scala.concurrent.context.minThreads`? Or even `scala.concurrent.minThreads` (How many of these properties are we going to expect with prefix `scala.concurrent`? Probably not that many, so no need for very fine-grained grouping.)
With @mrocklins changes to simplify it seems like it would be good to get this simplifying logic into boolean expressions. Otherwise, we end up with two ways of dealing with expressions, the SymPy way (with Symbols) and your way (with strings). So `simplify_logic` would become `_eval_simplify`; POSform would become `as_pos`; SOPform would become `as_sop`. The other functions would just be private functions -- I'm guessing they wouldn't be useful to anyone except to put a boolean expression into a simplified form.

I would really like to hear from anyone else, however, as to whether this should be pursued. (I think that simplification of boolean expressions makes sense.)
Maybe also test `id_A != id_B`
@flacjacket  I did not get your train of thought here. Could you please elaborate a little more(or IRC?). 

Could you also say what you meant by trace in the same basis.
Also, can you refer me to examples when inner_products = Tr(outerproduct) fails. I think I am  being *ignorant* of some behavior of states here.
If I can't rely on getting the first 20 bits right, mpmath and evalf have got serious problems...unless I am missing something. sqrt(3) is already an irrational; I don't see the problem with using 3.2. But do correct me if I'm missing something.
I implemented the multi line way.
On 13.04.2012 05:41, Christopher Smith wrote:
>> @@ -115,6 +116,41 @@ def from_PolynomialRing(K1, a, K0):
>>
>>               return K1(dict(zip(monoms, coeffs)))
>>
>> +    def get_field(self):
>> +        """Returns a field associated with `self`. """
>> +        from sympy.polys.domains import FractionField
>> +        return FractionField(self.dom, *self.gens)
>> +
>> +    def poly_ring(self, *gens):
>> +        """Returns a polynomial ring, i.e. `K[X]`. """
>> +        raise NotImplementedError('nested domains not allowed')
>> +
>> +    def frac_field(self, *gens):
>> +        """Returns a fraction field, i.e. `K(X)`. """
>> +        raise NotImplementedError('nested domains not allowed')
>
> just checking: is this error message suppose to be the same as for `poly_ring`?
>

I only moved this code around. I guess so.
In context, we know that we are working with a Partition, so perhaps we could follow the Matrix precedent and just call this `tolist`: `(a+b).tolist()`
my opinion is to drop the carets and use L1 and L2 or something, but this minimal latex is still pretty readable, but consistency is important too, so I'm in favor of dropping them.
On 13.04.2012 05:48, Christopher Smith wrote:
>> +    Create a generalized multivariate polynomial ring.
>> +
>> +    A generalized polynomial ring is defined by a ground field `K`, a set
>> +    of generators (typically `x_1, \dots, x_n`) and a monomial order `<`.
>> +    The monomial order can be global, local or mixed. In any case it induces
>> +    a total ordering on the monomials, and there exists for every (non-zero)
>> +    polynomial `f \n K[x_1, \dots, x_n]` a well-defined "leading monomial"
>> +    `LM(f) = LM(f,>)`. One can then define a multiplicative subset
>> +    `S = S_>  = \{f \in K[x_1, \dots, x_n] | LM(f) = 1\}`. The generalized
>> +    polynomial ring corresponding to the monomial order is
>> +    `R = S^{-1}K[x_1, \dots, x_n]`.
>> +
>> +    If `>` is a so-called global order, that is `1` is the smallest monomial,
>> +    then we just have `S = K` and `R = K[x_1, \dots, x_n]`.
>> +
>> +    A few examples may make this clearer.
>
> preppend the header?
>
> ```
> Examples
> ========
> ```
>

Done.
"It" --> "The polyhedral symmetry group" (either all caps (P.S.G.) or none)
Or how about `commutes = True/False/None` this follows SymPy's standard three value logic.
Apparently, you can just do `return type(it)([...])` here, as below, in `index`.
I've asked a collague for help on this change. The main problem is that; (I didn't check if it allways happens, but on all execution trials) this.view.render() returns an Element, but we pass it to  tart.dom.createElement, and it returns "string" back.  So we didn't alter the implementation but added a check for Elements and if the renderer method returns an Element, we choose not to retouch it with createElement. 

As I've mentioned before, due to time limitations we weren't able to spend too much time on our "desparate" seek for the proper solution. If you've a recommendation, I'm onto it.
goog.provide already does this; you can remove it safely :)
I'm wondering why, but I'm okay with that. Are `fetchEtag()` and `fetchLastModified()` fine as method names?
That's why I wrote that the user/developer can choose if he needs to clone his commands, but I guess at most of the times the command will be created just for this usage so it's not important that the command did change...

we jsut need to document that on the parameter description.
You're right, so if it doesn't find the default name that we specified, it throws an error or use the one that is provided by the user. I like that, even less code!
Out of curiosity, what would be a more correct solution?
spaces around the equals sign
Long lines should be wrapped to 80 columns.
In `errors.py`? There aren't `units` here. I'm adding the docstrings in `core.py` though.
It's so easy to enable URL generation that I can't even think of a good reason to not do that.
For consistency with other output, maybe put Result above Source.
OK, this POM looks good (modulo the migration dependency) - you just need to remove the version from the kernel/pom.xml
`if statistic && !statistic.empty?` ;)
Closing parenthesis shouldn't be there.
We've already been down the route of using fileinfo. It is not available on Windows by default:

http://www.php.net/manual/en/fileinfo.installation.php

>Windows users must include the bundled php_fileinfo.dll DLL file in php.ini to enable this extension.
Yes, but ticket also says about perserving search query. Where are changes for this?
`is None`
ah right. overwriting the class name might not be such a smart idea :)

what do you think about this: https://gist.github.com/3110173
i don't know if that would work for the dependencies, but it looks nice.
The method is not to 'return' result, it's to 'get' the postion. And what is returned is described in \return section.
Beware that scanners now are stored in `whois/record/scanners/`. You might want to merge master into your branch to get the latest changes.
`] lua_run_cl  print(VERSION)
151`
I'll use whatever the team feels is best practice/style.
Why do we need a yuiPrefix?
I agree that this would be an optimization.  However, almost every optimization comes at a cost to code readability/maintainability.  We should only optimize that which is truly a problem (based on real-world profiling numbers).

The above *is* less performant, but I'm not sure that it would show up on any measurements/profiling, since it is not called that much.  Also, it happens at server-start, which is less time-critical.  So, I intend to keep it as-is.
Docblocks, camelCase
I'd rephrase: not 'by default...', but 'we recommend setting...'
This is in the 0.9.x branch, so let's fix it in the master branch when we come to the next release. For what it's worth we do call ```close()``` everywhere else in the code base, and all the examples I can find on the netCDF4 python website do explicitly call close(). I don't think we are entirely blameless here.
It does make it sound like it was our mistake in the first place...
When you set a destroy_map, you are not obligated to work inplace.

This mean you could just have done not a not inplace version even when self.inplace is true. In fact, the not inplace is probably faster as the loop is in c and not in python.

You don't need to change this as if people want speed, they'll use the c_code. I just wanted to explain a little bit of theano internal.
The thing is, sometimes the optimization process does not find out that labels.shape[0] is actually 1, that can be the case for instance if ShapeFeature is disabled. In that case, T.arange(labels.shape[0]) will not be constant-folded. This case was taken care of in previously-existing code, below.
Some installation help.
Now that we've fetched these values, we don't need to refetch when calling planner.plan.
Better, we could fetch them earlier and make them QueryPlanner member initializers passed in once to the ctor and shared by the methods.
Again, a comment would be awesome, but the commit looks good.
Look up what "assign" means.
Well, if I'm not mistaken, the continual timeout means there is nothing to read on the SSH stream for now. So either network is very slow, or the command is not outputting anything...

I don't think there is anything abnormal here. If the connection shuts down, we'll got '' from the pipe and handle "usual" disconnection. 

All in all, I'm not sure how we could handle/test that. If you have any idea, I won't mind looking at that and try to fix the 'issue' :)

While I'm at it, I found out the test cases were not covering all the real life issues I encountered (especially because the read is now done by chunk and \r\n can happen at a chunk frontier) while finishing the patch. I'm not pythonist enough to be able to develop that kind of test case... should I file an Issue and we discuss that there ?
This looks racy; the refcount decrement is not atomic, and neither is the refcounting done normally by `Function`. I think it'd be cleaner if the `Function` belonged to the high-level `Thread` object and the low-level `platform` interface only dealt with `CCodePointer`s.
Clarity, correctness, and unit tests are what matters here. Size and performance are just noise in this case. 

I personally feel this is clear enough, but show me the unit tests.
Though most of us can read the `~name.indexOf` I think we prefer `name.indexOf() !== -1` for readability.

cc @rwldrn @dmethvin to confirm
The was corrected as well.  The argument names no longer use capital letters, and the dependent and independent variables are no longer referred to as columns.
Since automake is provided by Xcode <= 4.2, this needs to be conditional:

```rb
depends_on 'automake' => :build if MacOS.xcode_version.to_f >= 4.3
```
Can we use the diff directly from open-scene-graph's site rather than a gist?
> http://www.openscenegraph.org/projects/osg/changeset/12790/OpenSceneGraph/trunk/src/osgViewer/DarwinUtils.mm?format=diff&new=12790
Ah! Okay. I was just curious since if you use Enumerable#all? it will actually stop iterating the values when it hits the first non-truthy value:

```ruby
[2] pry(main)> a = [1,2,false,3]
=> [1, 2, false, 3]
[3] pry(main)> a.all? { |i| puts i; i }
1
2
false
=> false
[4] pry(main)> 
```
Hahahahah.  Summands.  Awesome.

Ok... addsTo it is.
However I accepted the pull request. We can change this later. I'm sorry that actually basic shapes are not working well.
You probably need to sort the list since self._intervals is a dict and its keys are unordered
I've copied your previous javadoc indentation to fix it. Done.
This will throw an exception for the following:

    tempFolder.newFolder("level1", level2");
    tempFolder.newFolder("level1", level2", "level3"); // exception thrown here

I think that `newFolder(String...)` should behave like `mkdir -p` in unix systems, and only throw an exception if the directory at the end of the path exists
What's commented out here?
In general, for all strings. We aren't crazy strict or anything, it has just seemed to be general unspoken consensus over the past ~6mo.
The way the comparators work, they have to be constant while the object is in the queue.

You must decide the distance to the player when the snapshot is added to the queue and then never change it.

This should still work reasonably, as long as the player isn't moving to quickly.

There needs to be some way that snapshots for a given chunk are never reordered.
No worries. I should have caught this sooner.
moved this async init into the root ``brackets.js``
Name in comment (resolveLocalFileSystemURL) doesn't match name in code (resolveNativeFileSystemPath).
Another syntax error (see above).
Afaik, Django is not needed to install oq-risklib
Ok, git thought there was a move but there wasn't. So the package name is just wrong
I guess we don't need this commented code anymore.
style nit: should be `} else {` on one line, right?
Check `ProcessFormView` first!
See https://github.com/mozilla/butter/pull/946/files#L1R30
Add a `.` at the end of the sentence. :)
add details argument (and property)
Move this out to a constant or even a parameter?
assigning the separate 'length' variable seems superfluous here.
I really want people to know where they're going url-wise. However, there's no limit to url sizes.

So If there's a description, I'd like to show the description plus something from the url in parentheses that indicates where the url goes to. Maybe something like this?:

    Slides for presentation (slideshare.net)

Or more?:

    Slides for presentation (slideshare.net/long-url-here-cut-off....)
"only" repeated twice
the wfs Parameter was needed for the feature editor to actually FIND the web-feature-service which is the service that the editor is using.
The user.is_authenticated was there because non authenticated users obviously shouldn't "see" this in the source-code.

You can try removing it, the version I was using (I think 1.1 RC 1) did not work without that line of code.
Noreplace variant is the one I would expect. But its a detail, any my personal opinion. I am fine with the current solution.
why not move this up into application_controller or a util class?
what is wrong on plain #!/usr/bin/python?
Will do that.

But I have one issue with loading :test group in :development environment. Now we also load "webmock" and when I try to start up development mode, I get: 

    WebMock::NetConnectNotAllowedError (Real HTTP connections are disabled. Unregistered request: HEAD http://aa.lan:9200/katello_user with headers {'Accept'=>'*/*; q=0.5, application/xml', 'Accept-Encoding'=>'gzip, deflate', 'User-Agent'=>'Ruby'}

Can you confirm? I guess I will remove this autoloading and put only those gems we need to have in development also in the test group. What are those? You mentioned parallel_tests and rspec-rails. Anything else?

Maybe it's not good strategy to put testing gems into development environment at all...
We had something similar in spacewalk, seemed to work well.

On Mon, Jun 25, 2012 at 10:06 AM, Miroslav Suchý <
reply@reply.github.com
> wrote:

> > @@ -8,7 +8,7 @@
> >      %a{ "href" => "#" }
> >        #{_("Terms of Use")}
> >    %li
> > -    %a{ "href" => "#" }
> > +    %a{ "href" => "http://webchat.freenode.net?channels=#katello",
> "target" => "_blank"}
>
> there is mentioned katello.org in #katello topic. But anything is better
> then empty link.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Katello/katello/pull/236/files#r1042120
>
Would HeartbeatsPerKeepAlive be too wordy of a name? I'm worried that people will assume that KeepAlive represents a time span in seconds without reading the docs.

Also, is there any reason we shouldn't just make this a uint autoprop instead of doing a runtime check to make sure it isn't set to a negative value?
Log something here as well.
Actually, it happened because I accidentally added {{expand language}} there.
This needs a LOCK on mempool.cs, imho. Even if it's somehow already safe because of the lock on cs_main, I'd prefer having it there, for when RPC locks get pushed down.
Please check.  It is absolutely good style, but if it make the GUI super sluggish because we're calling it 100 times per second when computing the list of transactions to show then performance trumps style (I have no idea if we are or not, but don't want people to start complaining that the 0.6.1 release is a lot slower than 0.6....)
Meh, it would be nice to have that, so it serves as a TODO.
If this is approved we would change the core usage to pass conditions array. If someone has overridden method in their extended class and get errors they can fix it. With debug off things should work without issues.
`or` should be `||`
Why is entity_tag being changed and not used? Should line 140 have 'entity_tag.include?' etc.?
do you care what the args are?
Right now a link to a directory gives a 404.  The files URLs don't provide directory browsing IIRC.
Yes, I guess you should put it in `try` / `finally` block.
Another place where I think we can keep 'enabled'
We should add these strings somewhere in the template so they can be localized, rather than in the JS. They could be inline text somewhere hidden with CSS, or stashed in data attributes.
I thought we agreed to go with `redis.errror` and then change them all in one fell swoop. No?
Yep. Also - in order to encode hmac keys you have to do something like this:

https://github.com/fwenzel/django-sha2/blob/master/test/django14/settings.py#L41
This seems like it'd be better in /etc/socorro/
Oops I take that back, we do already have /home/socorro/temp on processors in prod
Yes of course, that .1% makes all the difference ;) Honestly though, I used the same style rules here that is used elsewhere in the tabs for h4 headings so, we can leave it as is or drop the .1%
Is there value in having a title attribute if it's the same as the link text? Is that for accessibility?
ah, ok. Cool.
Soren, you were mentioned accidentally (my github name is "seth", but in other contexts I am also "sf"). Once mentioned, I'm not sure if you will forever be on this thread. If so, sorry about that. Will be closed shortly. Otherwise, complain to github. I've had the same thing happen with mentions of "seth".
This means there is no longer a visual indication that a field is required, right? That's not good. If this is broken then it needs to be fixed, not removed.
Class name should be uppercase `Factory`.
what if I pass ``NETWORK_PATH`` or ``RELATIVE_PATH`` here ? I would still get the behavior of ``ABSOLUTE_PATH``
this method should likely be removed. anything that cannot be done via prepending likely means that the given Bundle should be adjusted to handle merging of configuration.
IMO you should add `return $this;` too all `set*()` methods.
you probably mean 'escaped'
IIRC such upgrade notes existed when an 2.0 was in beta/RC phase.
also convetion is tu use camel case (`checkoutTime`)
Aha! Good stuff

On Wed, Nov 7, 2012 at 10:35 AM, Jessie A. Young
<notifications@github.com>wrote:

> In app/assets/javascripts/countdown_event_name.js:
>
> > @@ -9,8 +9,12 @@ $(document).ready(function(){
> >      characters_remaining = max - $(this).val().length;
> >
> >      $('.text-counter').text(characters_remaining);
> > -  });
> > -
> > -  $('#event_name').keyup();
>
> You're right: chars remaining count was not accurate on edit without this
> line so we changed the definition of characters_remaining (see line 3),
> which solved the problem.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/yammer/sched.do/pull/191/files#r2058897>.
>
>



-- 
@adarshp <http://twitter.com/adarshp>
developer at thoughtbot, inc <http://www.thoughtbot.com>.
313.454.1515 (m)
what's the difference from the above line `local = {}`?
De șters și de folosit rspec, la cele de mai jos la fel.
As I said above, one declairation is with arguments and one is without. It runs correctly on the current command helper release. (I don't know about future versions.) (assuming the /'#' bit works on your system as you says it does. If it complains with /'#', you told me to do that. [I still recommend /!])
Did you want two newlines after this log entry?  orac_say provides a newline at the end.
You can also use something like this: http://code.google.com/p/gonicetrace/ (I am the author, feel free to copy/paste code without citation if you want any of it)
You should use "span" tags for icons.
@kselden Do you have an idea how this can be done? I've tried looking at how the `toString` works - because it' prints an objects'  root - but I haven't had luck finding it out ...
I agree, I don't like this either. Especially since the private data might increase over time. 

During the early days of EJB client API implementation, we had planned an infrastructure to pass the private attachments separate from the invocation context data. However, due to some issues (which I don't remember now), we actually decided to keep it simpler and implement the way it's currently done. 

But your idea of using a specific key to pass along private data or something along those lines is a good idea. This will require some changes to the EJB client API but are doable and in fact will make this cleaner. I'll close this pull request for now and will do the necessary changes in the EJB client project before integrating it in AS7.
It's not allowable for that reason, no. 

That module is not part of the API of the server module. If a module needs that lib it should directly depend on it.
I don't really like this either, I think that playing around with the exceptions does not really make things much clearer, and has to potential to hide other problems.  

If a developer can't figure out that an "Address already in use " exception means they have something else listening on the port then I really doubt that anything we can log would help them out.  Having the big exception stack traces means that it is really apparent that startup has failed, clearing the stack trace like this PR does just makes it much harder to see the problem.
Use the completeStep(ResultHandler resultHandler) variant. The recursive variant is deprecated and shouldn't be used inside the AS codebase.
I would prefer there to be spaces around the question mark.
In my implementation I ensured it was origin based for simplicity (ala MobileNetwork etc)
Sorry, it's a number. corrected!
Seems like you could just make a mapping, e.g.

```var technologyNames = {
  lte: 'LTE',
  ehrpd: '4G',
  ...
}```
Changing back-size makes a 'repaint'
No dead code! :)
nit: you could also check and throw if the ini file does not exists
This may be the source of the problem (on the bot, assuming this works locally for you) there is some funky stuff going on with timing on the bot server and usually timeouts need to be bumped up a bit. Something is failing in a way that does not report errors. I can try out your PR locally to see whats going on.
UX Requirement! ;)
The naming is a bit confusing to me, I understand that is just an 'Ok' button to dismiss the dialog, but the syntax of the permissions show, is the first parameter a 'cancel' action.
remove leading `#` sign please
nit: `bindMozBrowserEvents` or `bindBrowserEvents`
I removed those lines as they are now done in system_dialog.js.
And `clear` is now called by system_dialog, when we call its `hide` method.
It's an issue of scope. Using ThreadUI avoid sometimes errors in scope, but we should use 'self', 'this', 'call' and 'bind' when neccesary. In this case 'self' is properly used.
Yep, we will do that possibility, just in case :)

Thanks for catching!
Shouldn't be this selector p > strong ?
Another magic constant ... why 60ms?
Please use `menuItem.dataset.state = 'unknown'` instead.
Remove this allocation.
Add a check before this line to see if home screen frame is not available

    if (!homeScreenWindow)
       return;
I think that it's good to have an ability to define separated place for default storage because storage api is the separated abstraction. 

Additionally this config used to pass nimbus storage path to supervisors in local mode. When we use storage API only relative paths are stored into ZooKeeper and, so, we can't get topology jars from supervisors in local mode without the knowledge where is nimbus temp directory located. 

If you don't like separated config, we can hide it from users and don't define it in the configs.
Putting the backtrace here will cause any API request to not properly parse an error. The response must be 100% JSON.
tab vs. space issue here, it seems.
saved ジョブ 

永続ジョブにした方がいいのかな、、:astonished:
Ясно. Возможно это и моя ошибка. Давайте сделаем так. Оставляйте как есть, а я чуть позже проверю и добавлю. Если без этого джема это работать не будет, я поговорю о возможности подключения этого джема, или другого способа решения проблемы. Огромное спасибо за старания! 

3 дек. 2012, в 11:41, Maxim Kulkin <notifications@github.com> написал(а):

> In config/locales/ru.yml:
> 
> >        no_shipping_methods_available: "Для указанного местоположения отсутствуют способы доставки, пожалуйста, смените адрес и попробуйте снова."
> >    errors_prohibited_this_record_from_being_saved:
> >      one: "1 ошибка не позволяет сохранить запись в базе"
> > -    other: "%{count} errors prohibited this record from being saved"
> > +    other: "%{count} ошибок не позволяет сохранить запись в базе"
> Ок, если Вы говорите, что так оно и будет работать, мне несложно поправить текст. Просто, в геме 'russian' там отдельно задается правило локализации (see https://github.com/yaroslav/russian/blob/master/lib/russian/locale/pluralization.rb#L21-L25).
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
Yeah totally, I can add in the `WNOHANG`. I do think plain `Process.waitpid` is fine here.
I have been playing around with Scalding to try solve this out, and the more I delved into the code the more I understood your initial rejection towards the 2 implicits I have written. I got to the conclusion that the only place I could deserialize the HBase Array[Byte] in time, is before the TupleGetter#get is being called in GeneratedConversions.

Well, I won't get into the details, but I need you to suggest how much discomfort this causes you:

```class HBaseJob(override val args: Args) extends Job(args) with HBaseConversions with FieldConversions```

... to save you the gory details. The end result is that HBase deserialization works (!) without modifications to the rest of Scalding infrastructure and especially the implicit type conversion mechanisms. To achieve this isolation I eventually needed to create an HBase "job" that inherits from HBaseConverstions - a class that holds an implementation that "decorates" the TupleGetter#get call

If you are cool with the whole HBaseJob concept than I would be more than happy to proceed ..
All the best,
Amit Mor
I took that idea from bottle. 11 is interrupt, which can give the change to trigger a clean shutdown. Bottle uses SIGNIT to its parent to kill its children processes.
The class already is neatly added in the firstChild so I guess.
A brain?
arg.. I'm working on metabuildbot to get the tests run on my branches..
In particular, that means we can use the decorator anywhere we have write a fake implementation.
Does this function need to call ignored_test.complete or done or something for whiskey's sake?
Additionally Look at RFC 5280 for the untrusted/scramble algorithm. Its pretty complicated to validate a peer cert chain.
There is a second part of the PR that is yet to come: `"sensio/foo-bundle": "@dev"` or `"dev-master"` or `"1.0.0@dev"` should allow dev packages for this one package, while leaving the rest in stable mode.
shouldn't this be changed to ``The Composer community`` or something like that ?
is it indented to use ``$l->getId()`` when the old code was using ``$l->getPackageId()`` ?
迷ったけどこのへんで。

```
mattr_accessor :data
self.data = Hash.new
data[:currencies] = ...
```

とやれば無理にclass_variable_getとかしなくていいと思うのだけど。
Do you apply settings without restarting of qutIM?
I think if you add a short inline comment here to explain (i.e. `// halt the program`) then it's good to merge.
lets chant in campfire?
There is no API exposed here. Nothing is exported out of cfx.js. Am I misunderstanding something?
Please prefer `===` unless type conversion is desired (as suggested by style guide)
Mind adding docs to all methods we have on this class?
I think we should rather whitelist the extensions we support rather than try to handle this with a blacklist.  

Many editors will leave file~ for example as backups we'll always play catchup, a whitelist would provide certainty and make it easy document and promotes unity across installs
I think we should not overthink this ;-). I mean the former behaviour of `exec` was like this: The first word is the binary and if the binary has spaces put it in double quotes. If the binary is not found with this simple rule `exec` returns nil. I never heard of anyone complaining about it ;-).

My latest patch now also allows single quotes but I really dont think we should mimic full CLI behaviour.
You know I have not, thats probably why you ask ;)
Valid point thou, I didn't do much thinking about windows up till now. 

I see several ways on how we (I) could fix this. 

* One option could be to check for indenting, 
* another would be to check if the next line is 'param = value' 
* or I could just check for windows and disable or use another character to split lines based on the OS type.
`{true, true}` is not the syntax you want. Just leave it off to get the defaults.
I like this new addition! Though you dropped the comments from ffi.js (minor I know)
This code is no longer used
I can tweak these if you want but u guys wanted me to be verbose :D anyway, this could be: `utils.forEach(this.tests, fn);`
we shouldn't need this line since it's defined above
Fair, and I had two other people code review it and not see that was still
a problem.

On Tue, Sep 25, 2012 at 6:11 PM, Stefan Zier <notifications@github.com>wrote:

> In src/main/java/com/hyperic/hudson/plugin/S3BucketPublisher.java:
>
> > @@ -165,6 +165,7 @@ public boolean perform(AbstractBuild<?, ?> build,
> >                  while (tries <= retries) {
> >                      try {
> >                          upload.waitForUploadResult();
> > +                        tries = retries;
>
> This doesn't appear to fix things -- the condition is tries <= retries,
> which will still be true if you set tries = retries.
>
> In fact, I'd recommend a 2nd boolean variable to make this more explicit:
>
> boolean success = false;
> while (!success && tries <= tries) {
> try {
> ....
> success = true;
> }
> }
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/weirded/hudson-s3/pull/2/files#r1691437>.
>
>
I would implement it as separate command - RefreshBoardCommand
CS, spacing & capitalization (common best practice = lowercase?)

$message = var_export($message, true);
First I tried to keep it consistent with the other tests that are initialized in this way.

Isn't that more convenient to start each test with a clean instance? If we wish to do it your way, there should be also a test that check if loading a new file set up all internals properly.
spaces around .
@jmikola I leave the decision to @weierophinney
Bad idea. This way I can with JS debugger to add closed catalogs if I'd like. It must be default: open=false
Ouch! :P I'm used to coding in allman style, nice catch
Oh, I didn't know about `__proto__`.

What would you do? Make a for loop copying all properties (recursively), or should we just pass the original settings object?
yah it does now. Wither Skeleton = Skeleton with metadata == 1
Done. Commit added to pull request.
use a modules section to avoid tons of bind entries
Can be either str or bool if we're using a single setting, so we need to keep the origin.
Tu as inversé les deux conditions en fait. as_file ça veut dire qu'on crée un fichier sur le disque. non?
it exists in all browser platforms we are targeting (ie9+).
ah right .. i guess it should then be done here https://github.com/doctrine/phpcr-odm/blob/master/lib/Doctrine/ODM/PHPCR/Mapping/Driver/AnnotationDriver.php
is it worth to implement here sending compressed output if client supporots it?
And etag/304 responses? Maybe could share some code with index_cluster...
`null` じゃなかった
On 7 November 2012 05:30, Justin Mayer <notifications@github.com> wrote:

> In pelican/generators.py:
>
> > @@ -156,10 +156,7 @@ def __init__(self, *args, **kwargs):
> >
> >      def generate_feeds(self, writer):
> >          """Generate the feeds from the current context, and output files."""
> > -        if self.settings.get('FEED_ATOM') is None \
> > -                and self.settings.get('FEED_RSS') is None:
> > -            return
> > -        elif self.settings.get('SITEURL') is '':
> > +        if self.settings.get('SITEURL') is '':
> >              logger.warning(
>
> There are solid use cases for serving feeds from a different
> domain/sub-domain than the site itself. For example:
>
>    1. Enabling support for external feed management services (e.g.,
>    FeedBurner)
>    2. Easier to migrate feed handling to another server, virtual machine,
>    or external service
>    3. Distributes load for sites with high traffic/subscribers
>    4. Potentially easier to analyze site logs
>
>
Thanks @justinmayer for this explanation.
Thanks @jawher for your hard work on this: I'll merge it soon.


>
>
>  —
> Reply to this email directly or view it on GitHub<https://github.com/getpelican/pelican/pull/551/files#r2051820>.
>
>
As you've defined depMgmt in the parent, no need to repeat it.
This is no longer right, as we deploy our own datasource. Perhaps it should say "ship with H2, an in memory database written in Java.".
I wouldn't mix primitiv and object based types and decide on one specific type. And I would keep the primitiv type if there isn't a good reason on why using Integer of int here.
Use 4 as the size here?
The switch to staticfiles is planned as part of moving this site over to the new playdoh layout, which sadly is being delayed by this feature. :(
Spelled "thunderbird" wrong
This is somewhat we discussed with Doug, as far as I could remember.
The API is not so terrible, so I think we could leave things as they are, or we can come up with a different implementation :)
this will break how we resolve puppetmasters names, not bad a thing to change the existing implementation, but needs to solve it before merging
When an instance changes power state, you need to update the database.  Turning the power on or off *is* editing an instance - you are changing fundamental attributes at that point in time.
This looks little paranoiac as ServiceDependencies code looks like it grants you that it return at least empty array. I think at least you should believe your internal code that you call. ( in case of external module it is something slightly different and defensive programing is fine )
can you rename this python-esque?
I don't think we can currently without asking for the datastream and seeing if we get an error. I'm not sure if fedora supports the ability to do anything else.
I was actually gonna ask about this black magic. I used it in a couple other places too but wasn't sure whether it was kosher.
Two syntax errors here.
Ah, gotcha. No, that's OK. Just the perils of reviewing code outside an IDE.
not a huge fan of this personally, I think if people have ./node_modules/.bin in their PATH that's all good but then we can just leave it as-is anyway, or if someone has strong feelings towards using the localized component(1) they can just point it straight there (I always do with mocha etc)
For the record, I believe the average joe would find this a supremely confusing syntax for creating a list, but then I've never been a fan of curly braces for lists to begin with. I'd want to be able to define a list like this:

mylist = [adam, bertil, caesar]

Commas optional, equals sign mandatory. I'm not even sure the commas should be optional. I'd be fine with quoting values with a comma in them.
I thought about this, and then thought we don't have that many repos, so it's a very lightweight query that then gets cached so only results in one hit on the DB.  Also, given that this page is only hit when a signed in user adds a repo, it's not going to be a high traffic query so I left it as it was.

I may be worth creating some sort of Redis list in the future of added repos for the future (for instance, some sort of search repo / autocomplete), but it's not needed now.
Format the footer. Defaults to an empty string if nothing has been configured
check if a footer should be added
Does this test pass even if we have the extra spaces, or was this change made by your IDE ?
@zacc , the commented out lines are the test cases that got condensed. the test cases they were condensed to are the second one and the last one in the file.
Prevent a possible loop
if (volume != volume) {
   volume_ = volume;
   emit volumeChanged();
}
Oh, just the `var that = { ... }` instead of `var that = {}; that.id = ...`
I'll add a comment about these alternatives and leave it as such for the moment. We can revisit this when we work more on step filters, like adding caching. Does that sound good?
There's been a large re-write of the javascript implementation which includes much nicer exception handling which i'm slowly working on pulling into jsonld-java in my own dev branch. I'll hopefully have time to finish this up in the next month or two.
I don't think Upload and Save For Later should be uppercased. Maybe in quotes if better - please also update confirm-license-text to mention that the quotes should match up with the buttons.
Ik heb het idee dat dit korter kan. De logica klinkt nogal onnodig complex. Je kan het iig doen als:


```js
if(this.standing && ((this.vel.x < 0 && this.accel.x > 0) || (this.vel.x > 0 && this.accel.x < 0))){
  this.vel.x = 0;
}
```

Dat is niet bepaald leesbaar, maar je kan het herschrijven als:

```js
condition1 = this.vel.x < 0 && this.accel.x > 0; //todo: verzin een betere naam voor condition1
condition2 = this.vel.x > 0 && this.accel.x < 0; //todo: verzin een betere naam voor condition2
if (this.standing && (condition1 || condition2)) {
  this.vel.x = 0;
}
```
Does this pass foodcritic FC001?
It looks like the attribute accessors should be strings rather than symbols (convention).
А имеет ли смысл чисто внутренний метод показывать "наружу", в родной для меня джаве это был бы обычный приватный метод с инкапсулированной логикой.
Вам не кажется, что здесь возможен случай, когда new ==NucleotidesEnumMax и тогда все свалиться в Exception метода replaceNucleotideAtIndex? 
Сравнивать значения текущего элемента массива и вновь сгенерированного надо, но зачем знак больше - не пойму? (if (new >= old))
Решение не соотвествует заданию по двум пунктам:
1) свойство DNA должно хранить непосредственно символы A T C G а не int. Думаю в этом была фишка задания. Студенты должны были научиться работать с NSString.
2) заполнять свойство DNA должен был именно метод init. Если я разботчик использующий Ваш класс Cell попоробую создать клетку, и при этом не вызову метод generateDNA ? У меня получиться клетка без ДНК что невозможно.
Честно говоря, после Java специально заставляю себя использовать конструкции Small Talk вида [адресат сообщение], а не объект.переменная. Имхо как раз для Obj C это более естественный формат.
Можете для тренировки попробовать использовать C-массив для mutateIndexArray (заполнить его от 0 до DNA_LEN - 1 и перемешать). Оптимальнее и по времени выполнения и по памяти.
Можно не использовать переменные-"обёртки", а сразу сравнивать соответствующие ячейки наших массивов.
The current version is 1.  We won't choke if you set `Version: 1.0`, but that doesn't mean you should do it.
Se voce nao vai tratar $postData nem $phaseId pode simplificar:

$model = new Application_Model_Phase($this->_request->getPost());
return $this->_mapper->save($model);
Oops, sorry about that one. I guess I should have tested more thoroughly.
The functions here are generally named after the structure they check, rather then the rule they are relevant to.
chmod or chown?
Alright, all good points :+1:
Do we still need these casts? I'm not sure what `self` is at this point.