I just upgraded my rsyslog server and a few clients (for testing) from 8.11 to 8.15 last week. I came in this morning with multiple errors in my logs of the type:
```
Jan 11 13:01:02 syslog.err rsyslog_server rsyslogd-2078: unexpected GnuTLS error -50 in nsd_gtls.c:530: The request is invalid.  [v8.15.0 try http://www.rsyslog.com/e/2078 ]
Jan 11 13:01:02 syslog.err rsyslog_server rsyslogd-2089: netstream session 0x7f77a8190520 from 172.30.108.105 will be closed due to error  [v8.15.0 try http://www.rsyslog.com/e/2089 ]

Jan 11 13:02:02 syslog.err rsyslog_client rsyslogd-2078: unexpected GnuTLS error -53 in nsd_gtls.c:1628: Error in the push function.  [v8.11.0 try http://www.rsyslog.com/e/2078 ]
Jan 11 13:02:02 syslog.info rsyslog_client rsyslogd-2359: action 'action 8' resumed (module 'builtin:omfwd') [v8.11.0 try http://www.rsyslog.com/e/2359 ]
Jan 11 13:02:02 syslog.info rsyslog_client rsyslogd-2359: action 'action 8' resumed (module 'builtin:omfwd') [v8.11.0 try http://www.rsyslog.com/e/2359 ]
```
The above is with a 8.11 client, but the same occurs with a 8.15 client.
I _think_ - I will need more time without errors to be sure - I've narrowed the issues down to this commit - I reverted the changes and recompiled - errors cease to be thrown and connections don't seem to be closed. Prior to making a separate issue on this I figured I'd comment here first. I'm not a developer so my usefulness in terms of writing code will be lower. Let me know if I can help resolve this issue now present. in 8.15


Shouldn't I be allowed to change the Kind here as well.  Seems like I should be able to even if its uncommon.


oops


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.646.9f2d4b1ce532a7e4f4ac4a47107326035a6c8ae0](http://build.palaso.org/viewLog.html?buildId=67816&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:06:07


Any idea how to write a quick check for trailing whitespace / missing newline at end of file that we could add to the build script?


Why did you put everything in '''?

I played through (after taking out the '''s) and it worked like a charm.  I like the alphabet check.


Because the data objects hat constructors using classes of the realm dependency and therefore crashed if the developers did not use realm in their project.
So I had to separate them.


souns like a plan - will try tomorrow


Will change it.


I don't like `outVersion` being a string.  You're using it as a `GroupVersion`.  Followup issue please.


i think imageoptim runs mutliple optimizers to maximize the reduction


reuse saveUser(..) to avoid duplicated code


This doesn't really seem to throw the "StopProcessingException", is it needed here?


the list should be private


@ghost1227 can you not just check for EDD_Newsletter class instead of maintaining this list manually?


Good.  Even better to put this in its own test.


missing space -
',', $ppp['ports']


We (I) did not think about that when creating additional constructors in the "normal" data classes.


what how


Comment is wrong.


it same


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11206](https://teamcity.mesosphere.io/viewLog.html?buildId=102970&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:09:01


all is good


Can you PR a test against the master branch to verify this, please? :heart_eyes:


@penguinpowernz #L68 should actually say "When using this library with sails add the config bellow to your"

since the config information is bellow, but apart from that it's fine.


I think you should make Standard Name in this line italicized.


Done!

-Sam Fries
AIMS Group, LLNL

On 1/11/16 1:07 PM, "Charles Doutriaux" <notifications@github.com> wrote:

>Merge pull request #71 from UV-CDAT/fix_multi_squeeze
>
>Fixes issue that happens when multiple axes were squeezed
>
>@chaosphere2112 please retag uvcdat-2.4.0
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/UV-CDAT/VisTrails/commit/f17263683798f967cc725ad912148a
>1e0771f0fb


I'm tempted to squash this with the previous commit before we merge into `master`... If that sounds OK to you, just let me know when you're ready to merge it with master, and I'll do the `git rebase --interactive` magic and force a push to this branch. NBD, but it would help keep the history a little cleaner.


Wrong access permissions.


Please ignore this file. This file is treated as "draft" test file.
I've left a comment that one moves to test.sh file in app_tests directory.


I tried bluesleep on angler on shamu (with required change in DT node)
The bug got apparent. So, maybe shamu bluesleep driver may resolve angler issue.


Might be a bug, but I was not getting those numbers. Let me re-run.

Also part of the reason they are slow is because of the delegate code. No need to test that.


I'm going to switch branches so the Go version is the default, because Shell script was just a quick hacky solution. So besides compatibility with Go packaging, which I'm still getting used to, do you think it is good Go code? It is my first Go project, I'd be happy for any comments.


Координати запиши в вигляді `49.85`, `24.0167` – як числа, а не рядки. Має бути так само, як в 254-255 рядку.


use REST conform url mapping, for example: /users/{id}  method=put


Makes much more sense ;-))))


Have a question on this one.   Is the point to prevent no playback after a buffer stalls that fails to restart?  IMHO this should just be picked up by the scheduler.  Or is this some issue with video element not reporting state correctly and this is a fix to cover that issue?


Added a bio file


Is it seriously necessary to remove one avatar icon for the greeter screen just because it is an image file? It was 5KB in size!!
What's the plan here? Install a separate package manjaro-i3-avatar with one icon, or what? ;)


0 - 41,369 bytes
7 - 391,593 bytes


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11207](https://teamcity.mesosphere.io/viewLog.html?buildId=102973&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:07:30


does `hide=True` hide the test output? If so, I think you should set it to `False` and print it out to the console.

Even better, why don't your run the `test` task you defined below, instead of calling `run("nosetests")`?


When you abort the install, can you exit with a non-zero status code? E.g. `sys.exit(1)`. This might be a useful way for a different script to tell whether `invoke install` completed successfully.


Great way to test for uncommitted files! It just occurred to me that we also have to check whether files have been pushed to the remote.

You can use `git log origin/<branch>..<branch>` to check for local commits that haven't been pushed upstream. For example, `git log origin/master..master`.

Therefore, the git-related checks should be reordered:
1. Check if on `master`
2. Check if uncommitted files
3. Check if files haven't been pushed


nitpicky: please replace "changes" with "files" to be more explicit


Before the final confirmation, please add a step that reminds the user to issue a new release. The message should be: "Did you remember to create a new release and increment the version number?"


nitpicky: replace the message with "You're about to install a new version of the WikipediaBase server. This will affect all InfoLab machines"


Its in manjaro-icons package.
See desktop-settings repo.


Per the original PR (#527) 15-20%.


Meant all but Yusef


:+1:
Next time it would be better if you issue a PR :wink:


@oscaritoro Did both. Not sure how that first thing got in there. Might have happened when git auto merged.


Thanks!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11208](https://teamcity.mesosphere.io/viewLog.html?buildId=102979&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:20


I don't get what you mean by codegen loop. In the generated code, it appears in front of the inner most loop:
```julia
    function eval_UC_spline_1d(a,b,orders,C,S,V,Ad,dAd) # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 111: # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 76:
        M1 = orders[1] # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 77:
        start1 = a[1] # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 78:
        dinv1 = (orders[1] - 1.0) / (b[1] - a[1]) # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 112:
        N = size(S,1) # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 113:
        @fastmath @inbounds(@simd(for n = 1:N # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 115: # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 91:
                        x1 = S[n,1] # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 92:
                        u1 = (x1 - start1) * dinv1 # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 93:
                        i1 = floor(Int,u1) # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 94:
                        i1 = max(min(i1,M1 - 2),0) # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 95:
                        t1 = u1 - i1 # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 96:
                        tp1_1 = t1 * t1 * t1 # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 97:
                        tp1_2 = t1 * t1 # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 98:
                        tp1_3 = t1 # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 99:
                        tp1_4 = 1.0 # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 116: # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 19:
                        if t1 < 0 # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 20:
                            Phi_1_1 = dAd[1,4] * tp1_3 + Ad[1,4]
                            Phi_1_2 = dAd[2,4] * tp1_3 + Ad[2,4]
                            Phi_1_3 = dAd[3,4] * tp1_3 + Ad[3,4]
                            Phi_1_4 = dAd[4,4] * tp1_3 + Ad[4,4]
                        else  # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 21:
                            if t1 > 1 # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 22:
                                Phi_1_1 = (3 * Ad[1,1] + 2 * Ad[1,2] + Ad[1,3]) * (tp1_3 - 1) + (Ad[1,1] + Ad[1,2] + Ad[1,3] + Ad[1,4])
                                Phi_1_2 = (3 * Ad[2,1] + 2 * Ad[2,2] + Ad[2,3]) * (tp1_3 - 1) + (Ad[2,1] + Ad[2,2] + Ad[2,3] + Ad[2,4])
                                Phi_1_3 = (3 * Ad[3,1] + 2 * Ad[3,2] + Ad[3,3]) * (tp1_3 - 1) + (Ad[3,1] + Ad[3,2] + Ad[3,3] + Ad[3,4])
                                Phi_1_4 = (3 * Ad[4,1] + 2 * Ad[4,2] + Ad[4,3]) * (tp1_3 - 1) + (Ad[4,1] + Ad[4,2] + Ad[4,3] + Ad[4,4])
                            else  # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 24:
                                Phi_1_1 = Ad[1,1] * tp1_1 + Ad[1,2] * tp1_2 + Ad[1,3] * tp1_3 + Ad[1,4] * tp1_4
                                Phi_1_2 = Ad[2,1] * tp1_1 + Ad[2,2] * tp1_2 + Ad[2,3] * tp1_3 + Ad[2,4] * tp1_4
                                Phi_1_3 = Ad[3,1] * tp1_1 + Ad[3,2] * tp1_2 + Ad[3,3] * tp1_3 + Ad[3,4] * tp1_4
                                Phi_1_4 = Ad[4,1] * tp1_1 + Ad[4,2] * tp1_2 + Ad[4,3] * tp1_3 + Ad[4,4] * tp1_4
                            end
                        end # C:\Users\pablo\.julia\v0.4\splines\src\macros.jl, line 117:
                        V[n] = Phi_1_1 * C[i1 + 1] + Phi_1_2 * C[i1 + 2] + Phi_1_3 * C[i1 + 3] + Phi_1_4 * C[i1 + 4]
                    end))
    end
end
```


I believe this was fixed in earlier PR so need to fix the conflict here.


I niby dlaczego to jest ważne ?


Opened https://github.com/kubernetes/kubernetes/issues/19501 to have the
discussion

On Sun, Jan 10, 2016 at 6:33 PM, Chao Xu <notifications@github.com> wrote:

> Yes, I mean splitting out the removing of the check. I should had said I
> was not fully convinced that the error message made the API backwards
> incompatible, that's why I'd prefer you make it a separate PR and assign it
> to brendanburns, since he added the export options. Thanks.
>
> Btw, it will be 6 lines, because there are two of them.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/96a1674da2399623a53207421851d744d40f6aa2#commitcomment-15362145>
> .
>


I'm not sure about "Used by". Do we have to update this for every user? I feel instead that there should be a brief description of the standard discussing what it aims to solve/possible applications of it.


interesting.. Thanks @LloydW93


Generally yes.  In practice...


I don't think this is the correct way to make an AST_Object instance with an AST_ObjectProperty. Take a look at lib/parse.js:

https://github.com/avdg/UglifyJS2/blob/174404c0f37bf14aff23c41889e6bab1fd87c1d7/lib/parse.js#L1242-L1283

Isn't `arguments` present in both AST_Function and AST_Defun? Perhaps you want such functionality in the base class AST_Lambda?


m.Serializer belongs to the common API, but this is a very specific API group.  They happen to be the same today, I don't think they are guaranteed to be in the future (the serializer for the root API might not support protobuf, for instance).


comm 4


Removed. It was at one point


I like guessing algorithm


Same statement as above - I think they're subtly distinct - the scheme for the common API doesn't have to be the same as the scheme for each api group.


komentarz


kreatywny komentarz


I took a quick look at the parsers, and the only that would probably be affected would be objective-c.  Though, I don't know objc and I was too lazy to install the 130+ Mo GCC compiler, so I won't know.
I then think it's fine like this for the moment at least.

---
FWIW, and for possible future use if we end up wanting this anyway, here's an (untested) implementation of partial matching:
```C
static gboolean scope_matches(const gchar *full_scope, const gchar *tag_scope, langType lang)
{
	const gchar *full_p = &full_scope[strlen(full_scope)];
	const gchar *tag_p = &tag_scope[strlen(tag_scope)];
	const gchar *sep;
	const gchar *sep_p;

	while (*full_p == *tag_p && tag_p > tag_scope && full_p > full_scope)
	{
		full_p--;
		tag_p--;
	}
	if (tag_p != tag_scope) /* no match */
		return FALSE;
	if (full_p == full_scope) /* total match */
		return TRUE;

	/* suffix match, check for sep */
	sep = tm_tag_context_separator(lang);
	sep_p = &sep[strlen(sep)];
	while (*full_p == *sep_p && sep_p > sep && full_p > full_scope)
	{
		full_p--;
		sep_p--;
	}

	/* whether sep fully matched */
	return sep_p == sep;
}
```


bardziej kreatywny koment


@lamont-granquist do you have the answer by any chance ? :bow:


Yeah, I just have to train a bit git, branching, making PR...


inn kokm


On 11/01/2016 19:35, John MacFarlane wrote:
> I wasn't sure either, but by far the most common use for
> this will be replacing a node you don't want to keep around
> at all. I'm trying to minimize the need for users
> (particularly people using this API via bindings in a
> dynamic language) to worry about memory management.
> People may remember to free the memory used by the
> whole document but forget to free nodes they've replaced.

Good bindings for dynamic languages handle memory management automatically and
don't require users to call `cmark_free_node`. This also means that any
function that calls cmark_free_node can't be exposed in such bindings, because
it might destroy a node that's still referenced from the host language. At
least that's true for the Perl and Ruby bindings.


I think "Used in" would make more sense than "Used by". That is, instead of "Used by X, Y, Z" you'd say "Used in _Turtle Navigation_".


i jeszce jeden


create.jsp and edit.jsp look almost the same. merge them together.


Je comprend pas trop la, les jeux indique clairement le 4 deja existant, donc ça "CELL_GCM_TEXTURE_LINEAR_NEAREST" alors pourquoi en crée une valeur qui n’existe pas sur la PS3 ?
Si je comprend mal explique moi ^^


Should this be "packages" lowercase?


Python capital here?


Some IDEs can generate code when iterators recognized, which can be convenient for users of the ZooKeeper library.

But I guess declaring a clear interface is a better priority, so I'll switch to RemoteIterator


Just made two tiny notes. If you have thoughts, let me know, Britta, and then I'll merge?


AddNetAreaColumnToEnvelopeSummary (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2075 of 2081 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.71%25-yellow.svg)


TeamCity pull requests :: netty-tcnative [Build 33](https://garage.netty.io/teamcity/viewLog.html?buildId=4688&buildTypeId=pulls_netty_tcnative&guest=1) is now running


TeamCity pull requests :: netty-tcnative [Build 33](https://garage.netty.io/teamcity/viewLog.html?buildId=4688&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:42


Comments addressed in 6788fba
<https://github.com/kubernetes/kubernetes/commit/6788fba4ba6c45fa95dd6e25702dbf8057796868>
barring
the question about serializer.

On Mon, Jan 11, 2016 at 2:08 AM, Chao Xu <notifications@github.com> wrote:

> I've done my pass. Just a few minor comments.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/bfdc589070c1f6d989e7c1ce77770c9ffa42151e#commitcomment-15364724>
> .
>


Yes. But you don't want to install a collection of all the leftovers of several profiles in a community release... I'll have to add the avatars to the wallpaper packages then.


Sorry, I'm a noob with these FilterDelegates... are we sure this is the only "IsEqualTo" that we have to modify?


This is interesting, I hadn't seen this before! So is the effect here that `loadObjectData()` is executed after `init()` has finished?


Again, a noob with FilterDelegates, but this "isNull" logic doesn't seem complete to me. Can you explain how this one should work?


well, we got to have to compromise, or you volunteer to package them all individually. ^^


Hmm maybe I didn't 'git add' the main file, they worked for me. Thanks for
adding them!
On Jan 11, 2016 8:07 PM, "Florian Golemo" <notifications@github.com> wrote:

> you forgot to add the measures in the main script. So I just ran the data
> analysis and the new measures weren't in there... so got confused. But I'm
> adding them now and will push to the git.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lstout/dataCollector/commit/41fdf7c7fa30eff7892a963f43a5a3848627988c#commitcomment-15378119>
> .
>


It works now using MingW :tada: .I will try to finish my implementation the next days.


Moved it to install for now


not sure if you meant "This method also adds"


ack


I looked up how to make it look like code with the proper coloring and all that. They said to do that but it didnt work. Glad you like the code though :D


List forces other test behaviors.  This has changed from time to time as different use cases needed to be tested.


Reformat?


"Please, note that"


Most of the error list is added before, in fact I didn't understand them, either...I think it looks like the Linux Error Code or something...


PATFORM -> PLATFORM


It's removed because it assumes YAML decoding.  It should be added into runtime/codec.go if we need it.


If I'm understanding it correctly, the `Metacard.CONTENT_TYPE` shouldn't be any of the following types, and if it is, then it isn't registry.


no tests in this commit? I assume you add them later, I'd prefer having them right in this commit


Works well.


Reformat in different commit?


ACK


I merged this but don't understand what the 2471 at USD is about.


I might be able to get rid of this. I was using an example and I may have copied this when I didn't need to.


Nit: remove "simply"?


Should we have new line at end of file?


@valib What did you actually change?


Ok. :)


a doctest would fit well here


The kernel_fpu_{begin,end}() calls are only needed for the AVX/AVX2 versions of the code, so they should be moved into sha256_generic.c and then just #ifdef'd into no-ops for the generic version.  That avoids adding needless overhead to the non-HW optimized version.


Did you test what this returns? I think `sys_siglist[]` contains descriptive messages while `sys_signame[]` contains the signal name.
Link: http://www.manpagez.com/man/3/sys_siglist/


doctest would do nicely, less unittests, better documentation


AddNetAreaColumnToEnvelopeSummary (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2043 of 2049 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.71%25-yellow.svg)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11209](https://teamcity.mesosphere.io/viewLog.html?buildId=102985&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:30


same here


Comments in https://github.com/smarterclayton/kubernetes/commit/84d92cb6c1398acc291c216938998905e005fa49


@snopoke did you intend to add this to both the new form and the old form? it changed the couch behavior which led to needing to add this https://github.com/dimagi/commcare-hq/pull/9836/. it wasn't totally clear to me that it makes sense to set the `edited_on` property of the original form. I think you could make the case that it's more helpful than `None`, but it leads to pretty confusing behavior if you edit a form more than once (every doc in the edit history will have the date it was _replaced_ except the latest one which will have the date it was edited).


Thank you. I stopped the test and resubmitted with the code fix.


unack, needs work


was this just used for debugging?


What is the purpose of this change? The example alias file is intended to be customized per-project and contain not just the local aliases but also the dev, stage, prod, etc. aliases.


I'd actually have preferred the feature development in a separate branch, and a merge to master after we've settled on a stable version, instead of a multitude of fixes in master, which make backouts, etc. rather hard. Please, let's do that the next time.


SGTM


You don't need +build s390x here, it's implied by the file name.


This, I think, would be better as:

```
os.path.join(root_path, TEST_DATA_FILE)
```

And then make `TEST_DATA_FILE` a relative path.


If you don't get an error you are guaranteed to have written to the stream.  Original object may be mutated depending on what implementation you have.  Any behavior not reserved in the comment is preserved for implementations.


AddNetAreaColumnToEnvelopeSummary (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (949 of 949 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-26.64%25-green.svg)


If we want this to be like an application the directory should be chosen by the user. (But that's not important for now, it's easy to fix!)


Hi @carolinasim  I have updated the readme at https://github.com/ITKarel/ChromeDriver to include instructions.


Should we? It's not breaking checkstyle.


Should the ctor/checker check that they value set is a valid css <ident>?

(Doesn't need to be addressed now, I just forget where typed-om is doing checking).


Їбать нафіксив


Serializers can ignore into.  Codecs don't.  Legacy behavior was for there to be an internal version.  When we switch to versioned codecs, if I say "Decode into v1.Foo" but the object is "v2.Bar" I'm now responsible for dealing with that (depending on where you are in the stack).

Into and defaults are always hints.  You don't get errors, because you get back the info you need to decide whether it's an error (you have the returned object, and the decoded GVK)


@matejka9 , taky hint este:
ak das do commit messagu #4 tak sa to automaticky prelinkuje s danou issue na githube
a issue sa automaticky zavrie, ak tam das fixed #4


The addition of line 584 in GoddessData.cxx `datum->depth = det->GetDepth();` results in an indeterminate result as the order of the stack, dE, E1, E2 in siDets is not specified. Whichever detector in the sector is the last will set the depth.


Iss390xRLD should be IsS390xRLD


ditto


No support for choosing defaults for both today.


Maybe the confusion is that you're interpreting the inheritance here as inheriting from `GPConstraint` OR `LocallyApproximableConstraint`? I tend to think of multiple inheritance as AND -- is that wrong?

And if we mean AND, then I'm still confused -- if a `ConstraintSet` IS a `GPConstraint`, then I expect `as_posylt1` to behave.


reformat?


Should we get some java doc love here as to what this filter actually does? (Or what it should do?)


If you save the name + .py at the end github will color it like python code


Yes, it's the last of the methods.


Always been this way, mostly used in tests.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11210](https://teamcity.mesosphere.io/viewLog.html?buildId=102994&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:24


Already see, but i don't understand :/


Added in a follow up.  Define emptiness?


а то


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11211](https://teamcity.mesosphere.io/viewLog.html?buildId=102995&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:00:42


Have you done any performance comparison of the auto-vectorized version compared to the hand-written AVX2 assembly version?


This kind of stuff is impossible, you need to set the username inside the fetchUsername's callback using `.then()`


also impossible


Is there a reason to remove s390 here? I think keeping it here should make gccgo maintenance a little easier.


We pass a generic serializer into something that wraps it with a versioning codec.  Only one side of the codec supports the operation, and errors if you try it.

Will add the assert.


Ping #538


That enum value is for min filter and vlj feels that it's wrong to use that for mag filter.


Will you remove this and the other flags before sending upstream? If not, they need to be made obviously s390x specific I'd think...


is it really working fine with -fstrict-aliasing ?!
Did u also check if it can compile without errors if u add those flags?:

-Werror=unsafe-loop-optimizations \
-Werror=strict-aliasing


[Code samples Java.zip](https://github.com/Ashorter1988/My-Code-Samples-C-and-java/files/86429/Code.samples.Java.zip)


I'm not talking about compatibility here. Let me make a different example to illustrate my argument. Let's say you've got a `GoogleSearcher` class that can be used as follows:

```py
GoogleSearcher("search term").search()
# OR
g = GoogleSearcher()
g.set_term("search term")
g.search()
```

Now you decide you want to add caching. You could easily extend the initial class signature as follows

```py
GoogleSearcher("search term", cached=True).search()
# doesn't work:
g = GoogleSearcher(cached=True)  # throws "'term' is None while trying to look up cache"
```

What you've done is extended the original API with additional/optional orthogonal functionality. That functionality should have gone to a new class entirely, like `CachedGoogleSearcher`.

It's only a suggestion, please don't feel offended. I won't bother you any more with this bike shedding :-)


Good Job! :+1:


@cro This won't work


I guess the comment here should be updated.


automatically added this?


I can't seem to find any CMake documentation on this idea of namespaces for targets.  The closest I can find is a few commands documenting a `NAMESPACE` argument.


Rather than depending on __attribute__((always_inline)) to inline SHA256Transform() from a different file, it probably makes more sense to have SHA256Transform() be a static inline in this file, and then it will be built once for each set of compiler optimizations.  The CPP macros would replace only this function name, and the static keyword would avoid namespace conflicts.


See the "External Assembly References" section in the Contributions Guide.
http://dotnet.github.io/orleans/Contributing


yeah, ups.. I will address these, thanks...


I must admit, at first glance I really don't understand how these Query Filter tests work... I hope someone who does gives it a good review :)... I'll keep trying to figure them out.


OK, I'll just have it unlink oldnode without freeing.

+++ Nick Wellnhofer [Jan 11 16 13:19 ]:
>   On 11/01/2016 19:35, John MacFarlane wrote:
>   > I wasn't sure either, but by far the most common use for
>   > this will be replacing a node you don't want to keep around
>   > at all. I'm trying to minimize the need for users
>   > (particularly people using this API via bindings in a
>   > dynamic language) to worry about memory management.
>   > People may remember to free the memory used by the
>   > whole document but forget to free nodes they've replaced.
>   Good bindings for dynamic languages handle memory management
>   automatically and
>   don't require users to call `cmark_free_node`. This also means that any
>   function that calls cmark_free_node can't be exposed in such bindings,
>   because
>   it might destroy a node that's still referenced from the host language.
>   At
>   least that's true for the Perl and Ruby bindings.
>
>   —
>   Reply to this email directly or [1]view it on GitHub.
>
>References
>
>   1. https://github.com/jgm/cmark/commit/93843bd2fe8c91fe9e659c5b801f1a7d672dc223#commitcomment-15381413


Looks good.


Just tried eslint inside sublime... works fine, and seems like a worthwhile change. :+1:


Do we not need to indicate code entry point anymore?


@hdngr This has already been converted. Nothing more to do except you can run `webpack` to make sure you see something like this in your console:
```
Hash: 8e3cff90bb3b586b7b85
Version: webpack 1.12.9
Time: 542ms
           Asset     Size  Chunks             Chunk Names
  todo.bundle.js  2.07 kB       0  [emitted]  app
vendor.bundle.js   1.1 MB       1  [emitted]  vendor
   [0] ./app.js 293 bytes {0} [built]
   [0] multi vendor 28 bytes {1} [built]
   [3] ./scripts/controllers/main.js 390 bytes {0} [built]
   [4] ./scripts/controllers/todo.js 429 bytes {0} [built]
   [5] ./scripts/directives/todo.js 180 bytes {0} [built]
   [6] ./scripts/services/data.js 403 bytes {0} [built]
    + 2 hidden modules
```


2015 or even 2016 surely :-) (many new files have this)


Can't blame you. ;) I had to look somewhere else in the code for an example


alo


@leizhangcn

I've moved `caffe.managed` to this repo.


sure the player is op?


@ragesoss @bmathews pointed out that while [istanbul takes un-`require`d files into account](https://github.com/SBoudrias/gulp-istanbul/blob/master/index.js#L48), gulp-jsx-coverage doesn't appear to. I'll try filing another issue.


104923612_EnhanceReportingToGithub (lefticus) - x86_64-MacOS-10.9-clang: Tests Failed (1685 of 1955 tests passed, 222 test warnings)

 * 499 tests had: AUD diffs.
 * 452 tests had: RDD diffs.
 * 169 tests had: EIO diffs.
 * 236 tests had: Table big diffs.
 * 1 test had: DXF diffs.
 * 12 tests had: ESO small diffs.
 * 8 tests had: MTD diffs.
 * 4 tests had: MTR small diffs.
 * 4 tests had: ERR diffs.
 * 1 test had: BND diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 3 tests had: MDD diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-86.19%25-red.svg)


For now I'm just choosing parity with Encode.  I don't think we have a use case for it today, it's easy enough to change (now that it's centralized).


[my work .pdf.pdf](https://github.com/CrossRef/rest-api-doc/files/86433/my.work.pdf.pdf)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11214](https://teamcity.mesosphere.io/viewLog.html?buildId=102998&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:55


hahaha eu acabei de mudar as mesmas coisas aqui!!
Vou atualizar o arquivo daqui a pouco


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11212](https://teamcity.mesosphere.io/viewLog.html?buildId=102996&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:20


For now, except no one actually needs the other methods on typer with this.  Once I nuke conversion/scheme, I was going to address this.


Cool cool - so no need to require angular in each file?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11215](https://teamcity.mesosphere.io/viewLog.html?buildId=103000&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:11


> I think the reason this is wrapped was because on *BSD MAXNAMLEN is defined in <dirent.h>

:+1:

> Maybe we shouldn't reuse MAXNAMLEN though and come up with something else eventually?

Please see the newest commit. We now directly use `NAME_MAX` from `limits.h` on all platforms but Windows, where we define it ourselves to`_MAX_PATH`.

Thanks a lot for the reviews, they've been very helpful.


I guess this commented code should either explain why it is commented or be deleted?


[Uploading NIHMS66559.pdf…]()


I might have been to hasty applying #57. Colon is used to separate the function name from the module name, however __main__ is the module name not the function name. Will reopen #57.


Forket kode hører rigtigt til #61


Same comment as before ... I think `os.path.join()` is your friend here.


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 189](https://teamcity.labkey.org/viewLog.html?buildId=296238&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:35


em algum momento você vai ter que aprender a resolver conflitos, hehe...


I'm usually a fan of them, but I actually just brought it up because the github diff tool points it out with a red "not return" symbol that stood out to me...


@ulfworsoe, I don't see any open pull request to METADATA.jl


Forket kode hører rigtigt til #62


Not Power ISA


Comments in https://github.com/kubernetes/kubernetes/commit/b252127906cdb8bfc3b1ca7e6584807a28b83a4f


Development of python has moved to github
https://github.com/python
but docs.python.org is not up to date.

I'll fix that.


> Please see the newest commit. We now directly use NAME_MAX from limits.h on all platforms but Windows, where we define it ourselves to_MAX_PATH.

:+1:

> Thanks a lot for the reviews, they've been very helpful.

No problem.


yes. I don't see a good reason to restrict the types. Although I suppose
because you're instantiating the template bodies in the .cpp perhaps we
have to to guarantee linking...

On Mon, Jan 11, 2016 at 3:34 PM, Alexandros Tzannes <
notifications@github.com> wrote:

> what is overkill? static_assert is a static check. Do you mean it is too
> defensive to restrict this code to only work for these two types (Function
> & Expression)?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ufora/ufora/commit/1c05c959c1cd5d3ba999802807d78bc4c0a24691#commitcomment-15380252>
> .
>

--
-------------------------------------------------------------------------------------------
Braxton McKee                             CEO / Ufora
www.twitter.com/braxtonmckee       www.ufora.com


Actually, is this used at all? Doesn't look like it to me.


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 190](https://teamcity.labkey.org/viewLog.html?buildId=296240&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:12


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11213](https://teamcity.mesosphere.io/viewLog.html?buildId=102997&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 207 Build time: 00:08:46


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 190](https://teamcity.labkey.org/viewLog.html?buildId=296240&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:14


Ahh good point, I misread what it was doing.

The docs on `@simd` specify this as a condition:

>The loop body must be straight-line code. This is why @inbounds is currently needed for all array accesses. The compiler can sometimes turn short &&, ||, and ?: expressions into straight-line code, if it is safe to evaluate all operands unconditionally. Consider using ifelse() instead of ?: in the loop if it is safe to do so.

It looks like the two `if`, `else` there will violate this condition...


Nope. Only when you need it, similar to `express`.


Let's `export elections16_FLASK_SECRET_KEY` in workinprivate and then do something like:

```
secrets = app_config.get_secrets()
app.secret_key = secrets.get('FLASK_SECRET_KEY')
```


Zoeller how do you add words to the game?


looks like another great plugin from you, but why not start with gulp?


HI,
these changes are necessary to get your setup running with:

 ~/projects/github/lemur-docker $ docker-compose version
docker-compose version 1.5.2, build 7240ff3
docker-py version: 1.5.0
CPython version: 2.7.9
OpenSSL version: OpenSSL 1.0.1e 11 Feb 2013

 ~/projects/github/lemur-docker $ docker version
Client:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 13:12:04 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 13:12:04 UTC 2015
 OS/Arch:      linux/amd64

Best
Tom


If it's possible to roll-back/undo please do it, I can then re-do my
changes..
If it's not possible, well, I'm really sorry :( Will be more careful in the
future.

Andreas

On Mon, Jan 11, 2016 at 8:59 PM, Ugo Pattacini <notifications@github.com>
wrote:

> This merge really messed up the history...
> @adamian <https://github.com/adamian> please remember always to do git
> pull --rebase before merging [image: :exclamation:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robotology/wysiwyd/commit/f7f6e97e4b0d5886731a3632deef00e4c35fe35d#commitcomment-15380897>
> .
>


I know objective-c but I'm afraid the whole autocompletion won't work very well because method invocation looks like this:

```
[myObject myFunctionWithFirstParameter: @"123" andSecond: 1 andThird: foo];
```

which in C++ corresponds to

```
myObject.myFunctionWithFirstParameterAndSecondAndThird("123", 1, foo);
```

We'd need the autocompletion to start after [myObject<space> which doesn't happen plus some quick way to quickly get after the ":" in keyword parameters (XCode inserts there kind of placeholders to which you can quickly jump with tab).

I think we shouldn't worry much about objective-c - everyone using it will probably use XCode for it (including me).


Yeah, we're working that out.  Your first fix was failing on Ubuntu 14.04 because of the earlier version of Python there I think.


yeah, dude, I'm pretty good with R at this point, completed the coursera R
programming course, which was pretty difficult.  Now, I'm using R for more
real world applications.  I've been trying to get my old webcam working on
it and was looking to get a motion sensor and make a sensor detecting
camera, but don't have the sensor and haven't had much time the past two
weeks.

On Mon, Jan 11, 2016 at 12:16 AM, ericatallah <notifications@github.com>
wrote:

> Ya, all on terminal. Are you studying R? What did you use the raspberry
> pi for?
>
> On Mon, Jan 11, 2016 at 12:05 AM, Adrien Atallah <notifications@github.com
> >
> wrote:
>
> > there is an R class on codeschool.com looks legit, but probably short
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ericatallah/practice/commit/752a986ea0be720a86124fe1e3041a5c4b800dad#commitcomment-15365178
> >
>
> > .
> >
>
>
>
> --
> Eric Atallah
> Web Developer
>
> Cell: (714)595-5111
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ericatallah/practice/commit/752a986ea0be720a86124fe1e3041a5c4b800dad#commitcomment-15365256>
> .
>


NIce!


+1 :D


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11216](https://teamcity.mesosphere.io/viewLog.html?buildId=103003&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:07


10zm jest tutaj zbedne


Although the Wiki is more detailed on usage, for continuity it'd be recommended to not force the user to open an external link to see what can be done by default.

One of the 'cardinal sins' of any program is never take a user out of an environment to show information that can be shown inside of it.

A possibility is to add to the end of the PM: "Detailed usage info can be found..."


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rongpengzhang) - x86_64-MacOS-10.9-clang: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-56.61%25-red.svg)


kod staje się bardziej przejrzysty


add line to file1


Pokaźny kod! :)

Ja bym tylko trochę uprościł:
```javascript
var EidRuntimeException = require('./exceptions').EidRuntimeException;
var EidNullPointerException = require('./exceptions').EidNullPointerException;
var EidIllegalArgumentException = require('./exceptions').EidIllegalArgumentException;
var EidIllegalStateException = require('./exceptions').EidIllegalStateException;
var EidIndexOutOfBoundsException = require('./exceptions').EidIndexOutOfBoundsException;
```

na rzecz:

```javascript
var exceptions = require('./exceptions');
var EidRuntimeException = exceptions.EidRuntimeException;
var EidNullPointerException = exceptions.EidNullPointerException;
var EidIllegalArgumentException = exceptions.EidIllegalArgumentException;
var EidIllegalStateException = exceptions.EidIllegalStateException;
var EidIndexOutOfBoundsException = exceptions.EidIndexOutOfBoundsException;
```


Dobrze, zatwierdzam zmiane.


Hello World


Is there a unit test you could add?


nice


@Dev2-PracticeProvider Sounds like a good idea.
-brady


enrique


I love this example - it's clear how this would generalise to the CLI and a GUI.


Propertie creation will be re-examined.

About the arguments object, probably both AST_Function and AST_Defun. But looking forward to es6, we have to account the arrow functions which doesn't have its own this object and probably not even an arguments object. Need to look at es6 spec.


This is crap!


Jest to usprawnienie


@Rechi

thanks for that! ... during my tests with mixed nameservers (two IPv4 and one IPv6) configured in resolv.conf  I realised the use is not as straight forward as the original code suggests.

For instance IPv6 server set in resolv.conf at 3rd line (behind both IPv4) wasn't added to list at all - it looks the positions of nameservers within the array(s) are not sorted / maintained to start at index 0 (my test case was _res._u._ext.nscount6 = 1 with the respective IPv6 DNS record at _res._u._ext.nsaddrs[2]).

For some test runs it seemed like **_res._u._ext** (with sin6 size) being the physical storage for all records and _res.nsaddr_list being just recasted pointer into it, but later I faced test runs (after resolv.conf lines shuffle) with IPv4 and IPv6 records being stored behind the same index (in own respective structures).

At last I changed the code as this https://github.com/mk01/xbmc/commit/f282ddeb81850c95956357a3c2a2ab2b2f6d6410 . I hope you don't mind.

Perhaps if you can retest/confirm this strange findings / or even find a better solution, let me know.

mk


Nein leider noch nicht!


Got an email with this bug report--seems like it's some complicated package version + dependancy mess:

```
>  remake::make()
[  LOAD ]
[  READ ]                                      |  # loading sources
<  MAKE > all
[    OK ] VascularPlantFamilies
[ DLOAD ] source_data/Spermatophyta_Genera.csv |  downloader::download("http...
[  READ ]                                      |  # loading packages
Error in download_from_remake_target(target, store, quiet) :
  Downloading source_data/Spermatophyta_Genera.csv failed (from http://datadryad.org/bitstream/handle/10255/dryad.59001/Spermatophyta_Genera.csv?sequence=2) with message:
there is no package called ‘downloader’
```


this.client.options.access_key was renamed to this.client.options.accessKey


Hmm... may be I was wrong using shortest name... Seems not all of them are localized properly in FR/GER versions and english one is shortest. And even in NA version, Unveiling Potion: For All/Me seems to be more descriptive than Self-Confidence/False Modesty Potion. Not sure what name is used in-game.


@Nadahar I only avoid to create and use the not necessary ```Object cloned``` variable and checking if it is an instance of the DLNAMediaInfo. If any other changes are needed I am not convinced because we use clone only for creating the TRANSCODE folder and other virtual folders. In that case the "shallow copy" is enough because we need only to change the audio and subtitles and the rest of variables can be "shared".


Would it be possible to create a docker image and share with me so I can reproduce the issue you're seeing?


I was planning to force the tests to produce this directory but thanks! #love


Kein Problem. Kannst du das noch in dem Dokument erwähnen?


Does this same fix need to be applied to the other tasks?


Not docker--would LXC work?


The polar accelerations look almost the same, should I put them these two cases together with an if statement for the ydim case or leave them as they are for clarity?


меньше -> раньше


I'm somewhat doubtful of this patch as well, though passing spectests is certainly an important improvement.

Looking at #679, it appears the justification for this patch is that current Rakudo is slightly less accurate than other Complex number platforms.  However, instead of trying to address this issue in a fundamental fashion, this patch just comes up with a bunch of special cases so that the previous algorithm is called less often.  I suspect there are improved algorithms we should be using in general, instead of fixing special cases only.


see my other request ... I have make a little error :upssssss:


Make this a constant: MANUFACTURER_SAMSUNG


Используй также эту константу в SeriesController, вместо SINCE_YEAR (который можно удалить).


Spaces around the +"/"+


А в инглише - норм?


OK if I merge these changes into LGSL?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11217](https://teamcity.mesosphere.io/viewLog.html?buildId=103005&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:20


Надо переименовать `year.invalid` так как это не просто год, это год выпуска серии. Назови его как-нибудь типа `series.too-early-release-year`


What's wrong with global variables?


I don't see a very good reason for moving the SDM hack out of this file.
Actually we are 1.- introducing an awful xhr call which produces in SDM an ugly warning in console and might be eventually unsupported in future browsers, 2.- an extra request in SDM 3.- We leave a file in the distribution which could be confusing for final users, 4.- The JS code size to maintain is not smaller.


Don't do this...


Is there a reason for `display: block`? It is pushing text after the `<a>` onto a new line, e.g.,

> ![foreign_relations_of_the_united_states__1951__europe__political_and_economic_developments__volume_iv__part_2_-_historical_documents_-_office_of_the_historian](https://cloud.githubusercontent.com/assets/59118/12248089/25bb3eca-b885-11e5-8859-c4ecb2b3838e.png)

(from http://localhost:8080/exist/apps/hsg-shell/historicaldocuments/frus1951v04p2.)  If I remove `display: block` this issue is solved, but I don't want to cause unintended consequences if this property is there for a reason.


If you have to use the old CONTENT_CONVERSATIONS_URI (in TextManager), then don't overwrite it here. Add ?simple=true in the place you need it.


Можешь поправить, да. Типа not earlier than и все-такое.


_id is BaseColumns._ID


ill take ios from here


Do we have tests for this method? It may be good to avoid regressions.


and android


info page looks cool, you should focus on that


Don't have LXC setup. Any way I can remotely login to test server?


Does this mean we now have to add entries into config.txt to include these dtb's to get USB?


Did not test ingame ingame, but..

Before this was not right because for example if a one hand item was transmogrified to the main hand warglaive (as an example - probably exist some non legendary example like some fist weapons), it could be equipped to off hand and the model was wrong way around in your hand.
Has this restriction changed? If not then this is wrong and main and off hand models must not be allowed on one hand weapons.

Also seems guns return 26 and bow returns 15 (as inv type), which would make them not compatible and thus error out even if they were supposed to be transmogrifiable with eachother? https://trinitycore.atlassian.net/wiki/display/tc/item_template#item_template-InventoryType


I've released `0.7.9`, don't mind


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11218](https://teamcity.mesosphere.io/viewLog.html?buildId=103006&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:00


@alexcjohnson better?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11219](https://teamcity.mesosphere.io/viewLog.html?buildId=103007&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:20


Paths need to be corrected. Wrong at the moment.


No problems :) Also, I have some suggestions about the joystick configs (as I've been testing them, lately).
It seems CyanogenMod supports force feedback for controllers, an option appears in the Input settings menu to redirect system vibration to the controller force feedback, and it works. I tested it with a Logitech Rumblepad. So I guess you can enable all the "FF" HID configs. Other than that, right now Wiimotes don't work at all, because the driver asks for hidraw, I enabled it and it works. Sadly, only old Wiimotes work (the new models don't, they need the new bluetooth hidp driver from 3.10... or so it seems. No luck for us 3.4 owners :( http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=e73dcfbf061b524fe9aaef56cf3c2e234a45ec19 ). Also, merging the wiimote commits from the CM kernel enables Wiimote Classic Controller and Nunchuck extensions support: https://github.com/CyanogenMod/android_kernel_oneplus_msm8974/commits/cm-13.0/drivers/hid .
I also tested the two missing configs for XPAD (force feedback and LED control) with the Xbox 360 controller, and they work. The LED used to blink indefinitely earlier, and now it doesn't. Vibration is redirected correctly :)
Finally, it seems Android doesn't report the power status for bluetooth connected devices (it doesn't with the wiimote, even though the driver seems to support it) so the Wacom config is probably useless.
These are all the changes I made: http://pastebin.com/G3HuSzki


On 1/11/16 4:14 PM, Alvaro Morales wrote:
>
> Before the final confirmation, please add a step that reminds the user
> to issue a new release. The message should be: "Did you remember to
> create a new release and increment the version number?"
>

Is it sufficiently obvious where to how to create the new release and
increment the version number?  If so, then all's well.  If not, please
add a pointer to where this is documented.


Everything. Especially when they are not needed. Its bad programming practice in Python. I've never encountered a situation where global variables are needed.

Also following convention (PEP8) it's preferred to name variables, functions and methods using "snake case"


Du kannst den Kommentar ignorieren. Das Dokument wurde abgegeben


I'm personally happy to drop host sharding. It's a clutch invented to circumvent web browser's per-host connection limit, but nothing we need to care about in native.


I rewrote this stuff in 1.6 for ppc64x to use a NOFRAME flag rather than the "negative frame size suppresses frame" thing, and I at least think this makes more sense. Given the lack of backwards compatibility requirements, maybe you could use that flag for s390x?


oops. not quite. my bad


Also notice that `configure` method may not be the best option while using the gem under Rails apps. See Prezento's example: https://github.com/mezuro/prezento/blob/master/config/environments/production.rb#L83

Overall, nicely done!


Workshopped


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 930](https://teamcity.labkey.org/viewLog.html?buildId=296232&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 16 (1 new), passed: 248 Build time: 00:36:09
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDateBased: <no details avaliable>

##### there are 6 more failed tests, see build details
```


This line got squashed. I edited the commit message, and notice that new SHAs are generated because the commit message and the parent SHA are hashed into the commits SHA. So this commit and the following ones (I made minor commit message edits there too, however their SHAs would have changed even if  I hadn't done this due to their parent SHAs changing) have the same content (except one commit squashed or combined with this one) but slightly different log messages, and completely different commit IDs.


Like here technically the setProfile function should be called set_profile, and as its in a class python expect the first parameter to be called self. By setting @staticmethod it defines the function static.


Podrías evitar error...
if(foco === undefined)
 form.submit();
else
 foco.focus();


Yes, but if I remove the conditions (which are used to extrapolate) by
calling `create_function(d, "none")` instead of `create_function(d,
"natural")` in `csplines.jl`, I still don' t get any magical speedup, sniff.

On Mon, Jan 11, 2016 at 9:48 PM, Spencer Lyon <notifications@github.com>
wrote:

> Ahh good point, I misread what it was doing.
>
> The docs on @simd specify this as a condition:
>
> The loop body must be straight-line code. This is why @inbounds
> <https://github.com/inbounds> is currently needed for all array accesses.
> The compiler can sometimes turn short &&, ||, and ?: expressions into
> straight-line code, if it is safe to evaluate all operands unconditionally.
> Consider using ifelse() instead of ?: in the loop if it is safe to do so.
>
> It looks like the two if, else there will violate this condition...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EconForge/splines.jl/commit/c6751276f39c0d13192875ee7d143abfefac148b#commitcomment-15382106>
> .
>


I don't know what a what you mean by "name of classpath".


#testingOverrated
huehuehuehue


Hi.
Sorry for dumb question, but why we need to use rsync rather than regular ansible's [copy](http://docs.ansible.com/ansible/copy_module.html)?


What is the problem with having an icon in an ISO-profile? I don't see that.


This can be written as `self.sensorIsValid =  sensorType in SENSOR_TYPES`


haha oops, I guess you could submit this upstream now?


@alexcjohnson this should do the trick?


or I could do all


Arrow function documentation: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions


I don't understand why you're making this change.


The other options us CamelCase. So I think this should be targetClasspath.


cuz this took me like 20 mins


(I made some comments on this comment, then noticed I wasn't looking at the full pull request, so I deleted them)


@nficano I'm not entirely sure what is going on here, maybe add a comment?

So `sys.exit(0)` should raise the `SystemExit` exception always, allowing for all `finally: ` blocks to execute up the call stack. But here, you're catching it in the very next line and issuing `os._exit` which is basically `SIGKILL`, bypassing any cleanup by the interpreter.

If you want to call `os._exit`, maybe call it without the try block? Otherwise, I would remove the try except here so any of the callers can have a chance to handle cleanup


Does it matter that the call to `failure` was previously independent of the Core Data save?


Override is an irregular verb, the past participle is "overridden" ;)
I think this would sound better: "Default editor layout overridden."


What about our change for proper boolean handling according to YAML 2.0 spec?


Thanks!


-  it was inconsistent that some methods like put was not synchronized and confusing.
- I added that DefaultBucket is not thread safe so that the clients with multiple-threads can handle that.
-  AbstractManagedStateImpl was also performing additional synchronization on bucket. Actually getFromMemory can be done concurrently but getFromReaders/All on a single shouldn't be executed by multiple buckets.

I made this change because I saw concurrency issues while testing the DefaultBucket.


@Rusketh :+1:


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 931](https://teamcity.labkey.org/viewLog.html?buildId=296234&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 15 (4 new), passed: 248 Build time: 00:33:34
### Failed tests
```
org.labkey.test.tests.RedCapTest.testDateBased: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.premium.FooterTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.cas.SSOwithCASTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitFooter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

##### there are 5 more failed tests, see build details
```


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 931](https://teamcity.labkey.org/viewLog.html?buildId=296234&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 15 (4 new), passed: 248 Build time: 00:33:35
### Failed tests
```
org.labkey.test.tests.RedCapTest.testDateBased: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.premium.FooterTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.cas.SSOwithCASTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitFooter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

##### there are 5 more failed tests, see build details
```


No - they exist to allow you to choose which driver to use, but each system will always have a default.


@karasu You can rerun build for a transaction (a single webhook received from github) using the pink "refresh" arrow icon in the timeline. You can also build any package on demand by typing its name in the bottom left input box. (just making sure you were aware since I never really announce new features when I add them to the build server (I really should start :sweat_smile: LOL)


and this is 4.4 tree which most users won't be on.


As I read it, npm-publish-report isn't an independently meaningful grunt task — it looks like it's only there to output the current value of updatedPublishList, and it can't be executed from the command line. If that's correct, recommend changing npm-publish-report to a simple function.


@d00rman ^^ look there are two new tests


the .git dir grows rapidly, and if it constantly changed, the .git dir even grows faster.
Its better to have a dedicated repo for images and artwork and stuff.


But why would the only time it's nonremote be when putting in an item..?
On Mon, Jan 11, 2016 at 3:49 PM Crowe <notifications@github.com> wrote:

> player.inventory.markDirty() ? I have no idea otherwise.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/L0neKitsune/Botanical-Addons/commit/341a48dbf8dcf222a9bf14d288a52235d75ddaba#commitcomment-15380592>
> .
>


:+1: https://github.com/plotly/plotly.js/issues/173#issuecomment-170709056


Thats true, 31c5c1d1032cc33cba797c30e791f1a25c1346e1 used to point to: https://github.com/tangrams/yaml-cpp/commits/yaml-1.2.

@hjanetzek do you have a PR on the original yaml-cpp project for this fix? If not, maybe its good to file a PR there!


I had no idea... thanks!


Why is there `return` here? From what I can see, it's not necessary... Am I missing something?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11220](https://teamcity.mesosphere.io/viewLog.html?buildId=103011&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:06


True, I admit I don't (yet) fully understood how Python's class system works... too use to procedural scripting and mark-up languages.


s/dnode/object.

It looks like the zfeature_register() parameters have changed since this patch was written, even though GitHub doesn't think there is any conflict for this patch.  In particular, this feature should be marked `ZFEATURE_FLAG_READONLY_COMPAT` and `ZFEATURE_FLAG_PER_DATASET`.


@marcelhuberfoo sorry about that.  I didn't catch it initially because I only did a debug build.  Fix applied as:

b858767 Fix 'prevsnap property' build failure


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11222](https://teamcity.mesosphere.io/viewLog.html?buildId=103016&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:28


Oh, thanks. Problem is that in the transaction a kmozillahelper package appears (and it does not exist).

I think I managed to hang the build server again...


+1 Good call, should have updated that comment.


Good call. The origin repo has made other fixes as well, so a better
approach may be to rebase our fork onto the origin.

On Mon, Jan 11, 2016 at 5:18 PM, Varun <notifications@github.com> wrote:

> Thats true, 31c5c1d1032cc33cba797c30e791f1a25c1346e1 used to point to:
> https://github.com/tangrams/yaml-cpp/commits/yaml-1.2.
>
> @hjanetzek <https://github.com/hjanetzek> do you have a PR on the
> original yaml-cpp project for this fix? If not, maybe its good to file a PR
> there!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tangrams/tangram-es/commit/af3477b0483849f288a3e9778b7dd13615c6ffd4#commitcomment-15382787>
> .
>


Learn OOP, that's basically it. And also follow PEP8 and you'll be set. Its fine, I have a nice IDE which goes though it and fixes all the PEP8 style violations


Кажется, там еще нужно с порядком проверок разобраться. Посмотришь?


Hey @TomBaxter,

These test fixes are missing an important point: we want to assert that an exception definitely happened.  If the call to `.validate_v1_path` succeeds here, it'll just pass through.  Instead, success should be an error.  I've sent a fix for the bug you encountered to `aiohttpretty`, so if that's accepted you should be able to just revert this commit.


Should probably include the permission


@lemire yes that's better of course:

http://lagua.nl/uploadfiles/example.html
http://lagua.nl/uploadfiles/example.uncompressed.html


Ага


TeamCity DnnPackager :: Continuos [Build 117](http://dazinator.ddns.net:81/viewLog.html?buildId=1550&buildTypeId=DnnPackager_Continuos&guest=1) is now running


TeamCity DnnPackager :: Continuos [Build 2.0.0](http://dazinator.ddns.net:81/viewLog.html?buildId=1550&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


TeamCity DnnPackager :: Continuos [Build 118](http://dazinator.ddns.net:81/viewLog.html?buildId=1551&buildTypeId=DnnPackager_Continuos&guest=1) is now running


I think this will need a fix similar to https://go-review.googlesource.com/#/c/18449/


i remember we had this at some point already, and it was causing other trouble...?


TeamCity DnnPackager :: Continuos [Build 2.1.0-unstable.46](http://dazinator.ddns.net:81/viewLog.html?buildId=1551&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:08


Oh dear, the tangrams fork is based on my personal fork :\

I'll go update both of those haha


Let's rename this `stephistory`?


I think you can remove all references to `matrix`.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11221](https://teamcity.mesosphere.io/viewLog.html?buildId=103012&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:19


@unional please edit `atomts.ts` and not js :rose: Also I don't see how this is `batching` I think its only *debouncing to send the last version* (if this is true then its wrong ... but then again I might be wrong :rose:)


@jplock 3.4.7 was removed due to deadlock bug.


@McFanta this was just to show the code, I knew i had to update on the page


Hi.
Quite new here so sorry for this question:
I know how to run libnfc but I have no idea on how to install this patch ?
Writting the all block 0 is important to me.
Thanks !


Arrow usage in UglifyJS' harmony branch: https://github.com/mishoo/UglifyJS2/blob/1cd9a2df9a282fbe53301c3a1d447e1381bf59c4/lib/ast.js#L479


It will be good to have tangram folk directly of the original.


I have not tested. But are you sure you need to return this? It doesn't look right as its setting properties on the object class . I suspect that you are not instantiating a new object using 'new'. I wanted to keep this as close to .net / Object originated programming as possible, so went for a modal that required created new instances of the class when creating a site.


not needed


Just the github "download zip" link. DIdn't try a clone, can do...


discount_form is a context variable provided by OrderDetailView - can it be removed?


We need collaborators who can get our software installed on Windows Computers to generate leads.


this got removed from the other arches in the 1.6 cycle i think


No please keep the ascii, but change the program icon


While grunt-shell seems like a useful way to declaratively define simple shell commands, the definition for the shell task here is complex enough that it ends up in some uncomfortable middle ground between declarative and imperative code. I think it'd be cleaner to just push this work to be imperative code that calls child_process.exec() directly. Aside from all the Grunt option parsing, that looks like that's all grunt-shell is doing for us anyway.

That would allow everything like npm-addowners and npm-owner-add to be converted to simple functions.

The work of our npm-publish task feels significant enough to warrant factoring into a separate script loaded by grunt.loadTask().


Thanks for merging the latest branch. Saved me a load of testing. It has all been written blindly as i have did not get the time to work on this today


If so, there may be cascading effects that let you remove things in forms/organizer.py.


@djih you could have them send an email to our support at friends@segment.com and we can check their implementation


How are we going to get around that .-.
On Mon, Jan 11, 2016 at 5:19 PM Crowe <notifications@github.com> wrote:

> ¯\_(ツ)_/¯
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/L0neKitsune/Botanical-Addons/commit/341a48dbf8dcf222a9bf14d288a52235d75ddaba#commitcomment-15382820>
> .
>


Super dumb me. Don't know why I edited the dist files..

When `pendingDiff` exists, it updates the `newText` and not the `oldText`, i.e. the `parent.editText()` call will get the first `oldText` and the latest `newText`. The range could be a problem, as it can span multiple sections of the source. Currently the idea is to just get the top-left and bottom-right, as an early draft.


Creating a consolidated definition of our package list like this seems like a good idea.


heh or many other things, I guess this comment is a bit of a hostage to the future


As this is what all SP would expect. It feels wrong to me but mirrors the server side model


Ok, so given that we'll need to continue passing `TileID` and pixel ratio through, are you ok with keeping those as-is in this code and replacing `SourceInfo` with a URL template string?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11224](https://teamcity.mesosphere.io/viewLog.html?buildId=103027&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:00


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11223](https://teamcity.mesosphere.io/viewLog.html?buildId=103026&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:26


lgtm


Sorry - you're entirely right.  The packages are required for the remake part of the analysis.

Might be worth remake being able to work downloader being required as part of its `install_missing_packages` routine (see https://github.com/richfitz/remake/issues/77)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11225](https://teamcity.mesosphere.io/viewLog.html?buildId=103032&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:24


Also, not sure if the sliding window (shifting the 500ms when there are edits coming in) is a good idea or not.

I wrote it that way expecting user are typing continuously and `atom-typescript` will run only after the user stops.


Sorry lol.


> The range could be a problem, as it can span multiple sections of the source

Indeed. A simple solution would be to store the list of edits as an array[] and then send them as is to the backend and then just let the backend apply them in bulk :rose:


Tem um "<" extra aqui


@hankim813 Hi Han, they are part of the email/Zendesk ticket that you commented on.


@karasu check your forum chat


I'd forgotten that Hannes has also made some nice aesthetic changes on our fork and it might take a while to get those into the upstream repo, so keeping our own fork is the practical thing to do for the time being.


Looks good :+1:


Interesting, let me re-publish 3.4.6


tyvm


gofmt?


@lhoward, @parkera, @jckarter, I think this is the only change I wanted to make to the demangling for non-private types. Generics may be something else we want to tweak but I'm not going to worry about that right now.


This seems like a good thing to be sure of :-)


In general I think a much bigger refactoring is needed, IMO.  Do you have anything in plan?


That exception is thrown in the `RestResponse` constructor. If reddit's response does not contain a `Content-Type` header, then the class doesn't know how to handle it, so it throws an exception. I cannot reproduce this on my end. Is it still happening?


This is not ppc64...


I haven't figured that out yet.  Its a little past my modding experience.  Its probably going to need a packet handler since, we need to update the inventory on the server.  I still have no idea why what I was doing didn't work....the whole nonremote thing is a little troubling.  If we could figure that out it would be easier to fix.


:joy:


Capital F on force ?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11226](https://teamcity.mesosphere.io/viewLog.html?buildId=103033&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:09:32


We should never use a bare except.  This should catch whatever errors you expect to occur.


I wasn't really talking about getting the G value dynamically (although it would be cool), but a little local or upper level variable to store the value to give it better context. also @ljfa-ag 9.8/25 is not a *nice* fraction :P


Fixed #246


You can readd them in skel too I think. skel/icons


Do wywalenia.


Should/could this be sent upstream now?


Yeah, but if we're passed a remote world there's nothing we can do about
it. Packets, but how to?
On Mon, Jan 11, 2016 at 5:33 PM Crowe <notifications@github.com> wrote:

> I haven't figured that out yet. Its a little past my modding experience.
> Its probably going to need a packet handler since, we need to update the
> inventory on the server. I still have no idea why what I was doing didn't
> work....the whole nonremote thing is a little troubling. If we could figure
> that out it would be easier to fix.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/L0neKitsune/Botanical-Addons/commit/341a48dbf8dcf222a9bf14d288a52235d75ddaba#commitcomment-15383139>
> .
>


ULib.ucl, ucl not exist
[ERROR] addons/customcommands/lua/autorun/server/sv_dban.lua:22: attempt to index global 'ucl' (a nil value)


Thank for your super prompt reply, but it's still not working as expected.. it doesn't print "invalid" :-(
![radare2_nodup](https://cloud.githubusercontent.com/assets/9197985/12248786/f82e5672-b8bb-11e5-8ce4-fbe53301d20c.png)


@palmer-dabbelt


Good point, but I guess resolved now that MPB 0.4.1 is tagged


Current todo:
- [ ] Property creation
- [x] Share AST_Function and AST_Defun init_scope_vars function


Regarding splitting the changes, you'd have to be a bit creative and probably slightly unorthodox about it. I.e. looking at the set of changes here, one way you could do it is to introduce in one PR a StageBasedPostProcessor class alongside the PostProcessorStage class, leaving them temporarily unused. In the PR description you'd reference your work branch to show where you are heading, but you wouldn't introduce anything else. In a second PR you'd then remove the current PostProcessor and wire the new one in, alongside the individual PostProcessorStage implementations.

But that's just an idea: you could come up with something totally different and equally valid. Bottom line is: we are a small crew, the smaller the PRs the less daunting the review process is, the quicker the PR goes through.


@thatJavaNerd Yes it is still occurring for me. Did you try out the code from the commit? Maybe the way I implemented it was wrong


Here is where we "Describe the feature that you’re testing in plain English" - [Article by Eric Elliott](https://medium.com/javascript-scene/why-i-use-tape-instead-of-mocha-so-should-you-6aa105d8eaf4#.obzaz0v1o)


Please don't start docstrings with a blank line.  The style guide for docstrings is [PEP 257](https://www.python.org/dev/peps/pep-0257).


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11227](https://teamcity.mesosphere.io/viewLog.html?buildId=103036&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:55


Can you explain why this change fixed this test case? I don't see any extensions at all.


Should git ignore this .idea folder.


@lots0logs He went to bed.

<+karasu0> I must go to sleep, have to go to work tomorrow early in the morning. Good night and thanks for your contributions!


"countries" typo.  Also, please don't start a docstring with a space.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25200/testReport/) for this commit**


Use the gem `devise` for authentication. It manages everything for you mostly, including registrations, confirmations, etc. https://github.com/plataformatec/devise


@diasdavid Oh I didn't mean that row, there's another row a little bit below


If I recall correctly, you need to use 'b' when opening a CSV file on Windows.  So you'd need to have `open(DATABASE_PATH, "rb")`.


There should be a space after the `#`, and a period ending the sentence.


But that was also I typo I guess ;)


Seems reasonable to me.


Is it correct to have the '//' at the front of the string?


Sure, this crashes when SourceKit tries to print the type interface of "aName" which is a String. String certainly has extensions.


You can achieve a lot of functionality with gems. Use devise for authentication. There is also a gem for the booking system, play around with that. Also HAML, SASS are really handy - play around with those too.


Are you sure this should be like that? I don't think you can bump "msgfmt:ui@2.0.0-preview.7" below without bumping package version here.


Oh, right, of course. Thanks!


Here, i.e. row 75 `## Tooling on top of the Core + Exntetions`
----------------------------------------------------------------------^


Shouldn't this be `**`?  From the docs:

    private IPs are marked with '**' (two asterisks)


Pressurestat (lgu1234) - i386-Windows-7-VisualStudio-12: Tests Failed (1669 of 1986 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-84.04%25-red.svg)


@cmbirk is this right?  no saveUser() ?


um, there are other things we're storing in `SourceInfo` right now; where would they go?


Strings with regular expressions are almost always more readable as raw strings.  i.e. `r"..."`.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11228](https://teamcity.mesosphere.io/viewLog.html?buildId=103037&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:22


hmm ok


What is `X-Received`?


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-MacOS-10.9-clang: OK (2044 of 2045 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.95%25-yellow.svg)


`findall` typo.


@Wyn10 Oh, haha..ok, thanks for letting me know!


`append` typo.


benny, I just copy-pasted exactly what was on the d3 website. thanks for checking though!


> Do you have anything in plan?

Not at all... sorry :rose:


Holy cow!  Great work, Cameron!


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25201/testReport/) for this commit**


fix `:ok` case in `setup` except for `having` context.


@robinboehm but first... coffee! <3 an den nested lists muss ich noch ein bisschen arbeiten... :)


@lots0logs Join him in irc ;)


A+ magic getters are always confusing.


No problem. Just asking.  Don't want to try something that stumble on what you are planning to change/throw away. :)


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25202/testReport/) for this commit**


The documentation isn't that clear, but I think this is meant to be added in a `X-Relay-Countries` header.  `X-Spam-Relay-Country` is the example custom header that people could add if they wanted to.


Yes, but I'd expect to have at least one test which evaluates with `endsWithDot: true`


Shouldn't this be `XX`?  From the docs:

    IPs not found in database are marked with 'XX'


I am not sure why this is as well.


Agreed. Only for b/c should they be used.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11229](https://teamcity.mesosphere.io/viewLog.html?buildId=103038&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:45


On Mon, 11 Jan 2016 21:39 Richard Thomson <notifications@github.com> wrote:

> I can't seem to find any CMake documentation on this idea of namespaces
> for targets. The closest I can find is a few commands documenting a
> NAMESPACE argument.
>

I know! It's frustrating. A lot of the newest and coolest stuff in CMake is
barely documented. You just have to pick it up by reading between the lines
and looking at example code. Iirc the Qt5 libraries and Zlib use namespaced
targets. With time, more are adopting it.

>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11230](https://teamcity.mesosphere.io/viewLog.html?buildId=103039&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:18


saw approval from javajolt
at https://github.com/basho/riak_pb/commit/06b891031a860c605339e08049df161154d554db


merging basho/riak_pb/perf/ttb_newmsg_ts1.0_release_candidate_v2 = 06b89103 into borshop-integration-176-perf/ttb_newmsg_ts1.0_release_candidate_v2


basho/riak_pb/perf/ttb_newmsg_ts1.0_release_candidate_v2 = 06b89103 merged ok, testing candidate = 94eaaad8


Maybe a switch statement would be better?


I mean if you still want to remove the `SourceInfo` dependency from *TileData, replace it with a URL template string there. Not remove it entirely.


I've seen the X-Received header in messages. I thought that it should be considered too.


Okay, X-Relay-Countries will be used.


104923612_EnhanceReportingToGithub (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1691 of 1961 tests passed, 224 test warnings)

 * 502 tests had: AUD diffs.
 * 454 tests had: RDD diffs.
 * 169 tests had: EIO diffs.
 * 236 tests had: Table big diffs.
 * 1 test had: DXF diffs.
 * 12 tests had: ESO small diffs.
 * 8 tests had: MTD diffs.
 * 4 tests had: MTR small diffs.
 * 4 tests had: ERR diffs.
 * 1 test had: BND diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 3 tests had: MDD diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-86.23%25-red.svg)


Ok, thanks.


can you specify the language and put it in the right directory?


Don't worry about updating...


Very nice, this is the true essention of automate the boring stuff!


Wouldn't it be easier to just require pygeoip for this than implement IP lookup ourselves?  To use the plug-in you need to get the database anyway, so also needing to pip install a package doesn't seem like much additional work.

That means you can get a location like this:

    self.gi = pygeoip.GeoIP(DATABSE_NAME, pygeoip.MEMORY_CACHE)

    code = self.gi.country_code_by_addr(ip)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1201.af507631ecf13b1efa8f9a014ba5525241f6eec8](http://build.palaso.org/viewLog.html?buildId=67823&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout (new); tests failed: 1 (1 new), passed: 823, ignored: 7 Build time: 00:35:54
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


Could you paste the link that you're using here? Because when I do the Download ZIP thing, this file looks correct to me.


I think we should stick to the standard Received headers.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11231](https://teamcity.mesosphere.io/viewLog.html?buildId=103050&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:31


Not really necessary indeed, but I like the string to also say between min and max.


Não cara, isso é necessário pra rodar todos os testes... E pra que tirar um novo package só pra rodar o teste de integração? Que nem está pronto ainda....


If you say so.


`-static-libgcc` is an unrecognized option on Macs with clang:

```
clang: error: unsupported option '-static-libgcc'
```


`rel_path` could be renamed to `db_path` and this line could be put into `core/database.py` where `db_path` is used now.

Then this file would be constants only, with no logic and no need "not to touch"


E não precisa ignorar esse teste, ele tá passando.


LGTM


Look at the stuff I just pushed.  I have no idea if it will work and it hasn't been tested, so take it with a grain of salt.


> It's only a suggestion, please don't feel offended.

No problem, really. I'm happy to discuss the code.

> What you've done is extended the original API with additional/optional orthogonal functionality.

Ok, I think I've got this now.

It seems nobody cares about such orthogonality. Even Django core developers leave neat error messages for such cases.
https://github.com/django/django/blob/master/django/db/models/base.py#L855
And to be honest I don't worry about this problem too. I believe you will not try to render the message before initialisation. Finally you will get an error saying you doing something wrong if you even try to do this accidentally.

I prefer to keep things simple as much as possible without loss of flexibility. You probably know there is a bunch of similar projects. I started new one because I did not like another implementations. The goal of this project is to implement the templated mailing features using KISS and OOP at the same time. I use OOP because it works fine for huge projects for me. KISS probably does not require explanation. And of course it would be nice if a reusable library would be flexible as much as possible, because this makes it much easier to reuse it. I will be happy to hear any recommendations how to accomplish all three requirements at the same time in this project.

I also will be glad to hear about any potential problems, errors and bugs in this library. And I believe such things should be testable, i.e. there should be a way to write a test case.


Remove space between `round` and `(`. Functions must be connected to parentheses.


On my setup, there is no python2, only python, python2.6 and python2.7. I have no experience on this matter, but maybe this would be too specific to put the name of one of those last two versions?


yoohooo


meu calam, eu irei fazer os testes de integracao só estou deixando configurado, para quem for usar nao precisar configurar :D


AddNetAreaColumnToEnvelopeSummary (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1500 of 1500 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.67%25-green.svg)


meu calma*


eu rodeie le local e não está passando.


Remove space between `length` and `(`. The space between `if` and `(` is okay. Correct these throughout the script.


Probably worth adding some comments to this new method since this is not immediately obvious until you look at all the implementations.

Additionally, "replicateTo" sounds a bit like a command as opposed to just a filtering request.


Probably wanted to just remove this.


Thanks for this CCX brew! Which compilers for  C and FORTRAN are you using? Did you use something different from the ones recommended in the HOWTO of the official website?


Ta, mas porque não deixar dentro do package de testes junto com os unitários?


On 01/06/2016 11:12 AM, Ricardo de Almeida Gonzaga wrote:
> If we really want to initialize I2C, GPIO and PINMUX to always be
> performed during platform boot, I'm ok with the patch.
>
> @jeez <https://github.com/jeez> @aguedes <https://github.com/aguedes>
> any thoughts about it?

I'm fine with initializing those devices during platform initialization.


Given that this file doesn't change, not sure if we want it in this commit.


I now read this last comment with slow => so low... for a second there I wondered what speed had to do with it ;)


pq ele é de integração, ele deve ser rodado em um momento separado dos testes unitários, por diversas razões, nem sempre tu irá querer rodar os testes integrados, a medida que tu nao precisará subir o teu ambiente apra rodar eles, e também eles demoram  mais tornando custoso o build local se tu deixar eles na pasa default do maven de teste.


However if we want to use database elsewhere for some reason, we would have to define absolute path twice - and import os twice as much.


Minor but we often include `// Guard` with these guards.


closes #2


@ragesoss Here's a first pass. We killed the avatars and collapsed mainspace/userspace edits:

Legacy course display:

![screen shot 2016-01-11 at 2 59 19 pm](https://cloud.githubusercontent.com/assets/702327/12249457/bea0cca6-b874-11e5-8b6a-05211b104449.png)

Current course display:

![screen shot 2016-01-11 at 2 59 25 pm](https://cloud.githubusercontent.com/assets/702327/12249459/c337b8c4-b874-11e5-9115-2fe7284db8dc.png)


Since you have divided uninit and stop, and stop is called in flush to reduce power consumption, shouldn't clock be stopped there? Since Toshiba documentation does not consider stop/uninit but presents a single stop routine I guess this is our choice (if we have a way to restart clocks of course)


Is this debugging or did you want the logging around this assertion?  If we want to keep it here, we probably want to word it up a little more.


Pressurestat (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


do we need this still?


Great job. Another alternative here was to use an AND statement in the condition.
`if (i % 3 === 0 && i % 5 === 0)`


if you don't like the `if`/`else` logic, this could also work with separate class methods for each type. For example: `+[UIApplicationShortcutItem wmf_search]`, `+[UIApplicationShortcutItem wmf_random]`, etc.


I can just comment here sorry, but at line 340, if using a variable in the for loop why not declare and assign directly, or define val in the function scope?


Great. It should only show that data (and only send that data to the client) if the user is the instructor for that course, though (or an admin).


Esse sequence pode quebrar o CNPJ se criarmos mais de 9 itens.


php56-tokenizer too?


if it is, seems like we should try and handles this in `without` as it is barely ever used.


gene analysis pipeline is slow and dirty


does this effort include pruning all the namespace tostring code? Or is that orthogonal ?


@mdtusz @cldougl done.


Is this desirable? Seems like previously we would have made a colour based on a store's preference for$color-navbar-bg


Only show the real name?


Correct.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25207/testReport/) for this commit**


Overall great job. Nice use of variables of function and nice control of the Document object model.


I create an issue for that:
https://leap.se/code/issues/7788


Closes https://github.com/18F/before-you-ship/issues/66.


I was reluctant to add a dependency to geoip (geoip, geoip2, pygeo) because of the database they use (maxmind's geolite2 databases) for which I don't know the format of such databases, csv seemed to be better to me. The change is easy if you want to use pygeo, although the code is kind of simple.

Also I'm unsure about which module is better, pygeo or geoip2 (from maxmind) which seems to be updated regularly.


:clap: excellent! Thanks for adding the lexer! This will be a great template for future work too :smile:


ahh cool - I think it's a little more clear
plus I just like `function()` *way* more than `function ()`


Pressurestat (lgu1234) - Win64-Windows-7-VisualStudio-12: Tests Failed (1669 of 1986 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-84.04%25-red.svg)


This should remain 4.0.0


along the same lines as my previous comment, if you had specific class methods for each shortcut item, you could make some of them `nullable`, which would simplify the logic of adding the shortcut item only when `wmf_openArticleTitle` is available. In either case, `lastRead` (and the subtitle) should be never `nil` or an empty string. For example:

``` objc
NSMutableArray<UIApplicationShortcutItem*>* items = [NSMutableArray arrayWithObjects:/* persistent shortcut items */];
UIApplicationShortcutItem* continueReading = [UIApplicationShortcutItem wmf_continueReading];
[items wmf_safeAddObject:continueReading]; // only add if non-nil
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11232](https://teamcity.mesosphere.io/viewLog.html?buildId=103055&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:35


This should be System.AppContext


this is shit


Great job. Another alternative here was to use an AND statement in the condition.
`if (i % 3 === 0 && i % 5 === 0)`


You need to update the version of the ref\*.csproj to 4.1.0.0


Something to note, it appears `PINCache` supports back to iOS 5 in it's readme, but `NSMapTable` has only been available since iOS 6. Furthermore, one cannot simply check for the existence of the `NSMapTable` class, it seems that `NSMapTable` was a private class in iOS 5, and was made public with `weakObjectsHashTable` in iOS 6.

*Snippet from Flipboard iOS*
![screen shot 2016-01-11 at 3 09 36 pm](https://cloud.githubusercontent.com/assets/522951/12249614/ad27acbe-b875-11e5-80a1-1cac6df749c2.png)

So, you should check if the `NSMapTable` class exists and responds to the `weakObjectsHashTable` selector before allocating it to preserve compatibility back to iOS 5.


@cldougl

> plus I just like `function()` *way* more than `function ()`

me too. I think only @bpostlethwaite likes the latter more.


Nice! You could have added the `document.write` on your javascript and that would have save you adding all this extra lines. But great job.


On it.


Não, testes de integração rodam junto com testes unitários, embora eles serem mais custosos que unitários, são menos que testes funcionais. Quando você fala custoso, é demorar 1 minuto a mais na pipeline? Isso não é um teste custoso e outra coisa, bota na balança, custo x garantia de qualidade, o que é mais eficiente nesse contesto? Rodar 1 teste uma vez antes de um deploy e quebrar o sistema porque faz tempo que o teste n é rodado ou rodar o teste sempre para evitar que isso aconteça mesmo que demore 5 minutos a mais.

Ah, e na pipeline ele deve ser colocado em um job separado, assim como cada tipo de teste, mas pra isso é criado tarefas para rodar eles separados.

Já dei os motivos para você reverter isso! :+1:


How is the software supposed to know when it is a packet coming IN or OUT?


Same as above


Since theoretically SAB code can run on weird architectures outside the usual ARM/x86 group, add some correctness for fencing reasons.


Good start :+1:


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25208/testReport/) for this commit**


This disables SFTP for additional created users on your server. This does not affect your master sudo account, nor your root account. This only limits SFTP via 4747 connections to limited shell users. Moreover, users still have access to SSH via 4747.


Since the set_state function on the sensor can fail, but this is stateless, moving this after the other avoid starting the rx interface if sensor configuration fails


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rongpengzhang) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-56.59%25-red.svg)


Ah! Yes! That makes sense -- as we only shipped 4.0 versions of this!


Duh.


Done!


Here's the link:
https://github.com/nicholasmr/obblm/archive/master.zip

And the contents of header.php inside it:

```
else {
    $conn = mysql_up(defined('T_NO_TBL_CHK') ? !T_NO_TBL_CHK : true); # MySQL connect.
    setupGlobalVars(T_SETUP_GLOBAL_VARS__COMMON);
    require_once('modules/modsheader.php'); # Registration of modules.
    setupGlobalVars(T_SETUP_GLOBAL_VARS__POST_LOAD_MODULES);
}

/******************************
   Translate skills globally
******************************/
global $lng;
$lng->TranslateSkills();
```