A quick comment on FLUX_RED_TIMEDFLUSH support here.  Assuming r->timeout_msec has been set to zero via xzmalloc above, this statement would return a timer that will expire immediately.

But if I want to set a real timeout with flux_reduce_set_timeout_msec below at line number 88, I don't see a statement that will rearm this timer.

At 156, I see the timer get restarted with r->timer to support TIMEDFLUSH, I can't find a statement that sets the timer with the new timeout value.

I had problems with the current head so I patched my version to rearm the timer with the newly set timeout value.

I know this isn't a problem specifically associated with this PR, I thought I would pass this along in case this is a bug.


Nice,  since then I learned some new tricks for optimization :)


That should be more "direct":

```js
optionString = argString.split('=', 1)[0];
argExplicit = argString.slice(optionString.length + 1);
```


test comment


Ah, nice idea! I like it.

I'll add to #13 what I was thinking there. But for now, lets not make big changes to xmpp-client, until we have otr3 more or less ready


I like the patch, but I don't think this loop is necessary. Specifically, the issue existed because the status_map is initialized with values that override each other. The non-deterministic behaviour will happen only between two runs of the application, not between two calls to .abort(). That means even before this test would sometimes work and sometimes fail.

But the test is still very useful without a loop - getting all 8 exceptions "right" before would be extremely lucky!

Thanks for the fix :)


For a 12 hr vote it gives
59905143d 18:16:22


(so I'll be honest and admit that I didn't really understand what this commit is doing; e.g. I do not have an intuition about the follow-on effects from the two changes that added `commit_if_ok(|_| ...)` so its best that I handed this off to @nikomatsakis)


That's a cool idea :)


It might be interesting if we could see which Aaays have grenades.


My patch added the following after 90.

if (r->timer)
    flux_timer_watcher_destroy (r->timer);
r->timer = flux_timer_watcher_create(1E-3*r->timeout_msec, 0, timer_cb, r);

You may have a better way of course.


not sure if that's a good idea, but since you have to include so many files from your models folder you might want to do something like that http://stackoverflow.com/questions/735073/best-way-to-require-all-files-from-a-directory-in-ruby also that way if you add more classes you won't have to require_relative them

also i would make attr_accessor and uri method private and then you probably don't have to test your uri method since if it does not work correctly your get wouldn't work as well


@mpkrueger DWMR (done with my review). Hey Matt. I'm looking forward to working with you. Looks like you are about to jump into projects. I'd like to discuss a strategy for success, and fun. Have you setup a schedule yet?


Oh, definitely. We will have to develop entirely new hacks for Wayland.


thx


ha!


Currently the ou of the group is required. If it's not possible to change the value attribute of the html elements, a clean solution would be to rename the groups following the pattern of the "cn"s in ldap (e.g. "LG Karlsruhe") and modify the API to search for the cn. You might also just prefix an "LG ", if renaming the groups is too hard.


first comment, yeah that's a good idea. Idk how the github flow works cause aperantly rebasing screws everything up which goes against everything i've learned since i've been here lol.

As for the second one, I'm looking at it from a public api perspective. Someone may want the uri of my request. I think it's reasonable as a public api, even if it could avoid testing.


@mpkrueger DWMR. Your work here is good. Press on!


which in pip is xmpppy


When clarke was first reviewing my code, i got called out for not having my specs on the same commit i added stuff. I see it as, when you add a feature, you should also add proof that it works. In otherwords my application should work and have 100% coverage at any point in time. No commit should lack coverage or be broken.


You only need aliases if you want to be able to use that alias to import a facade/


```js
let p = androidBootstrap.start(appPackage, disableAndroidWatchers);
conn.emit("connect", 'connected!');
await p;
```


360 quaternions in angle axis format with axis being the X axis - [1 0 0]
And angles being 1 deg each.


please be careful with indentation. applies to every line.


space after if...


Typo "sfFinishAFter".  No biggie.


return $connection != null; or just $connection


The message associated with this commit was mis-labeled (it was accidentally committed as part of some other work) but I did want it committed, so I won't be manually reverting it.


return type. and type hinting for each param.


return type please... and space between ","


Thanks. I saw this on the interface. I will update the file soon. Some other words need to be refined because from the visual of the interface I know in what context they are used. It was not too bad for my first time using GIT...


); should come the before.


@psytronx DWMR. Press ahead.


space after if


seeing as `li <- pick $ choose (0, lc - 1)` where `lc = L.length ls` it should be fine.


@psytronx Smart to verify before taking action. Good.


I guess this is temporary solution?


return type.


Yes, the flaw is in the page.Render function in Hugo ... it should always try to use the template given as first argument.


isn't m_request public?


This code contaims code from bukkit . A license will be given tommorow and the used files are:
'''java
RegisteredListener.java
HandlerList.java
Event.java
PlayerEvent.java
Command.java
```
Credit will be given tommorow


I thought we are doing camel casing? m_loggedInUser


I've removed this check because setting a foreign key to the same value (root) isn't dangerous and this way the code is simpler. Maybe when the foreign key is allowed to be null and it contains a non-root id we could fill this with a null value.


Yeah, that'd be nice but there are two problems: it'd need trailing whitespace (which would fail tests), and we need to flush stdout with a newline.


these logic should move out of this class. you simply set the user to this Request class.


Descrição errada de commit. O que foi feito aqui é a mudança de alguns itens (TextureAtlas e Region), passando instância para classe pai.


Hummm ta errado isso hein


Minor tweak. To be consistent. `vtkMRMLNode::SafeDownCast` could be used.


let's discuss on this style. putting ); at the new line.


fix_intel_compile_linux (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1775 of 1776 tests passed)

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


you should just pass whatever you need in the function instead of th ewhole context. you just need DBConnection, right?


added second line


Still try the gunicorn command or another command?

Taylor French, B.Sc.

> On Jul 28, 2015, at 8:32 PM, Travis Jacobs <notifications@github.com> wrote:
>
> Add windows support to trendlinks.py as __main__
>
> Haven't tested on windows, but this might do it.
>
> Let me know if it works in #37 @tfrench93
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/travs/TReND-links/commit/4a36bcdaedaf1822d7b4404b363e15cad9fb279b


Will be fixing what you commented as soon as possible


Before adding get_command, I was testing the intro screen and needed it to show for several seconds. However, once I added get_command, this wasn't necessary. I forgot to remove it in the first commit.


We could possibly restrict them to just mutons and floaters.


EMOJIS! :tada:

(that is my only feedback.)

(jk. not really.)


@esteele I see you have meanwhile fixed the wrong version.  (And I have fixed the too short title underline.)

But do you know you can do this more easily using the `postrelease` command from zest.releaser?

Maybe you tried exactly that and something weird went wrong. I just want to make sure you are aware of the existence of that command. :-)


Unless I'm misunderstanding, making `rewrite_on` configurable doesn't do anything since we only look for `x_forwarded_proto`. Maybe we should rename `rewrite_on` to `allow_scheme_rewrites` and default it to false instead?


Address has already been sent out in an email to all the hackers by the time this happens.


Thanks


For the third cohort we guided them all down to the hackerspace. I think it would have been pretty disruptive to have 14 people standing around in the lobby. We also had people arrive at pretty erratic times (one person came two hours early on the first day in the third cohort).

Would it be alright if we leave as is?


Can we remove the sales pitch?


Again


Can we add something about what the service actually does to mitigate DDOS, etc.


@jxandery I've updated the README to be more expressive of things. Let me know what you think or what I could improve :smiley:


Dale

On Tue, 28 Jul 2015 19:29 Luis Perichon <notifications@github.com> wrote:

> okkk!!! hago un cherry pick y lo paso a master. vos cuando puedas tirate
> la nueva version!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dwaynemac/padma-assets/commit/c98088e58c981560634076a3972e70bd6a7d4f06#commitcomment-12408139>
> .
>


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (743 of 743 tests passed)

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


lxmcv;lxmcvl;xcv


OK, makes sense, thanks.


prese to fix line spacing here :100:


LGTM. I don't think we have the non-helper call versions of these implemented yet for ARM64


Bug: path_to_key  to  path_art


Hi,
Thanks for the quick reply.
Firstly, I am not too familiar with the Jenkins plugin architecture.

I dropped the jar file from the link (renamed it git.jar) into
 $JENKINS_HOME//plugins/git/WEB-INF/lib

After restarting jenkins I get an exception. Are there other steps that I need to follow to test this?

Thanks

Jenkins
javax.servlet.ServletException: org.apache.commons.jelly.JellyTagException: jar:file:/var/cache/jenkins/war/WEB-INF/lib/jenkins-core-1.620.jar!/lib/hudson/actions.jelly:39:70: <st:include> Error setting property 'from', exception - java.lang.NullPointerException
	at org.kohsuke.stapler.jelly.JellyFacet$1.dispatch(JellyFacet.java:103)
	at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
	at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
	at org.kohsuke.stapler.MetaClass$13.dispatch(MetaClass.java:411)
	at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
	at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
	at org.kohsuke.stapler.MetaClass$6.doDispatch(MetaClass.java:249)
	at org.kohsuke.stapler.NameBasedDispatcher.dispatch(NameBasedDispatcher.java:53)
	at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
	at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
	at org.kohsuke.stapler.Stapler.invoke(Stapler.java:649)
	at org.kohsuke.stapler.Stapler.service(Stapler.java:238)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:848)
	at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:686)
	at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1494)
	at hudson.util.PluginServletFilter$1.doFilter(PluginServletFilter.java:132)
	at com.smartcodeltd.jenkinsci.plugin.assetbundler.filters.LessCSS.doFilter(LessCSS.java:46)
	at hudson.util.PluginServletFilter$1.doFilter(PluginServletFilter.java:129)
	at hudson.util.PluginServletFilter.doFilter(PluginServletFilter.java:123)
	at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1482)
	at hudson.security.csrf.CrumbFilter.doFilter(CrumbFilter.java:49)
	at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1482)
	at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:84)
	at hudson.security.UnwrapSecurityExceptionFilter.doFilter(UnwrapSecurityExceptionFilter.java:51)
	at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
	at jenkins.security.ExceptionTranslationFilter.doFilter(ExceptionTranslationFilter.java:117)
	at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
	at org.acegisecurity.providers.anonymous.AnonymousProcessingFilter.doFilter(AnonymousProcessingFilter.java:125)
	at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
	at org.acegisecurity.ui.rememberme.RememberMeProcessingFilter.doFilter(RememberMeProcessingFilter.java:142)
	at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
	at org.acegisecurity.ui.AbstractProcessingFilter.doFilter(AbstractProcessingFilter.java:271)
	at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
	at jenkins.security.BasicHeaderProcessor.doFilter(BasicHeaderProcessor.java:93)
	at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
	at org.acegisecurity.context.HttpSessionContextIntegrationFilter.doFilter(HttpSessionContextIntegrationFilter.java:249)
	at hudson.security.HttpSessionContextIntegrationFilter2.doFilter(HttpSessionContextIntegrationFilter2.java:67)
	at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
	at hudson.security.ChainedServletFilter.doFilter(ChainedServletFilter.java:76)
	at hudson.security.HudsonFilter.doFilter(HudsonFilter.java:171)
	at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1482)
	at org.kohsuke.stapler.compression.CompressionFilter.doFilter(CompressionFilter.java:49)
	at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1482)
	at hudson.util.CharacterEncodingFilter.doFilter(CharacterEncodingFilter.java:81)
	at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1482)
	at org.kohsuke.stapler.DiagnosticThreadNameFilter.doFilter(DiagnosticThreadNameFilter.java:30)
	at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1474)
	at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:499)
	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:137)
	at org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:533)
	at org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:231)
	at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1086)
	at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:428)
	at org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:193)
	at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1020)
	at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:135)
	at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116)
	at org.eclipse.jetty.server.Server.handle(Server.java:366)
	at org.eclipse.jetty.server.AbstractHttpConnection.handleRequest(AbstractHttpConnection.java:489)
	at org.eclipse.jetty.server.AbstractHttpConnection.headerComplete(AbstractHttpConnection.java:949)
	at org.eclipse.jetty.server.AbstractHttpConnection$RequestHandler.headerComplete(AbstractHttpConnection.java:1011)
	at org.eclipse.jetty.http.HttpParser.parseNext(HttpParser.java:644)
	at org.eclipse.jetty.http.HttpParser.parseAvailable(HttpParser.java:235)
	at org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:82)
	at org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:668)
	at org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:52)
	at winstone.BoundedExecutorService$1.run(BoundedExecutorService.java:77)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
	at java.lang.Thread.run(Thread.java:745)
Caused by: org.apache.commons.jelly.JellyTagException: jar:file:/var/cache/jenkins/war/WEB-INF/lib/jenkins-core-1.620.jar!/lib/hudson/actions.jelly:39:70: <st:include> Error setting property 'from', exception - java.lang.NullPointerException
	at org.apache.commons.jelly.impl.TagScript.handleException(TagScript.java:726)
	at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:281)
	at org.apache.commons.jelly.TagSupport.invokeBody(TagSupport.java:161)
	at org.apache.commons.jelly.tags.core.ForEachTag.doTag(ForEachTag.java:150)
	at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:269)
	at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:95)
	at org.apache.commons.jelly.tags.core.CoreTagLibrary$2.run(CoreTagLibrary.java:105)
	at org.kohsuke.stapler.jelly.CallTagLibScript.run(CallTagLibScript.java:120)
	at org.apache.commons.jelly.tags.core.CoreTagLibrary$2.run(CoreTagLibrary.java:105)
	at org.kohsuke.stapler.jelly.JellyViewScript.run(JellyViewScript.java:95)
	at org.kohsuke.stapler.jelly.IncludeTag.doTag(IncludeTag.java:147)
	at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:269)
	at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:95)
	at org.kohsuke.stapler.jelly.CallTagLibScript$1.run(CallTagLibScript.java:99)
	at org.apache.commons.jelly.tags.define.InvokeBodyTag.doTag(InvokeBodyTag.java:91)
	at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:269)
	at org.kohsuke.stapler.jelly.ReallyStaticTagLibrary$1.run(ReallyStaticTagLibrary.java:99)
	at org.apache.commons.jelly.tags.core.CoreTagLibrary$2.run(CoreTagLibrary.java:105)
	at org.kohsuke.stapler.jelly.CallTagLibScript.run(CallTagLibScript.java:120)
	at org.kohsuke.stapler.jelly.CallTagLibScript$1.run(CallTagLibScript.java:99)
	at org.apache.commons.jelly.tags.define.InvokeBodyTag.doTag(InvokeBodyTag.java:91)
	at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:269)
	at org.apache.commons.jelly.tags.core.CoreTagLibrary$1.run(CoreTagLibrary.java:98)
	at org.apache.commons.jelly.tags.core.CoreTagLibrary$2.run(CoreTagLibrary.java:105)
	at org.kohsuke.stapler.jelly.CallTagLibScript.run(CallTagLibScript.java:120)
	at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:95)
	at org.apache.commons.jelly.tags.core.CoreTagLibrary$2.run(CoreTagLibrary.java:105)
	at org.kohsuke.stapler.jelly.JellyViewScript.run(JellyViewScript.java:95)
	at org.kohsuke.stapler.jelly.IncludeTag.doTag(IncludeTag.java:147)
	at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:269)
	at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:95)
	at org.kohsuke.stapler.jelly.CallTagLibScript$1.run(CallTagLibScript.java:99)
	at org.apache.commons.jelly.tags.define.InvokeBodyTag.doTag(InvokeBodyTag.java:91)
	at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:269)
	at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:95)
	at org.kohsuke.stapler.jelly.ReallyStaticTagLibrary$1.run(ReallyStaticTagLibrary.java:99)
	at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:95)
	at org.kohsuke.stapler.jelly.ReallyStaticTagLibrary$1.run(ReallyStaticTagLibrary.java:99)
	at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:95)
	at org.kohsuke.stapler.jelly.ReallyStaticTagLibrary$1.run(ReallyStaticTagLibrary.java:99)
	at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:95)
	at org.kohsuke.stapler.jelly.ReallyStaticTagLibrary$1.run(ReallyStaticTagLibrary.java:99)
	at org.apache.commons.jelly.impl.ScriptBlock.run(ScriptBlock.java:95)
	at org.apache.commons.jelly.tags.core.CoreTagLibrary$2.run(CoreTagLibrary.java:105)
	at org.kohsuke.stapler.jelly.CallTagLibScript.run(CallTagLibScript.java:120)
	at org.kohsuke.stapler.jelly.CompressTag.doTag(CompressTag.java:44)
	at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:269)
	at org.kohsuke.stapler.jelly.JellyViewScript.run(JellyViewScript.java:95)
	at org.kohsuke.stapler.jelly.DefaultScriptInvoker.invokeScript(DefaultScriptInvoker.java:63)
	at org.kohsuke.stapler.jelly.DefaultScriptInvoker.invokeScript(DefaultScriptInvoker.java:53)
	at org.kohsuke.stapler.jelly.JellyFacet$1.dispatch(JellyFacet.java:95)
	... 69 more
Caused by: java.lang.IllegalArgumentException: Error setting property 'from', exception - java.lang.NullPointerException
	at org.apache.commons.beanutils.ConvertingWrapDynaBean.set(ConvertingWrapDynaBean.java:74)
	at org.apache.commons.jelly.impl.TagScript.run(TagScript.java:265)
	... 118 more
Caused by: java.lang.NullPointerException
	at hudson.plugins.git.Revision.toString(Revision.java:117)
	at hudson.plugins.git.util.Build.toString(Build.java:91)
	at java.lang.String.valueOf(String.java:2849)
	at java.lang.StringBuilder.append(StringBuilder.java:128)
	at java.util.AbstractMap.toString(AbstractMap.java:523)
	at java.lang.String.valueOf(String.java:2849)
	at java.lang.StringBuilder.append(StringBuilder.java:128)
	at hudson.plugins.git.util.BuildData.toString(BuildData.java:242)
	at org.apache.commons.beanutils.BeanUtilsBean.copyProperty(BeanUtilsBean.java:353)
	at org.apache.commons.beanutils.BeanUtils.copyProperty(BeanUtils.java:160)
	at org.apache.commons.beanutils.ConvertingWrapDynaB


Bug: path_to_key to path_art


Is there a reason to use the "if ... endif" lines?  They are actually preventing SaxonCE from loading.


`year" -> "week"?  That's what line 163 suggests.


我一会看看……

发自我的 iPhone

> 在 2015年7月28日，23:19，解清山 <notifications@github.com> 写道：
>
> 酱紫，私信下拉刷新会。。。‘刷屏’
>
> —
> Reply to this email directly or view it on GitHub.
>


@danicampora thanks, fixed!


或者你pr一下也行

发自我的 iPhone

> 在 2015年7月28日，23:26，解清山 <notifications@github.com> 写道：
>
> if (bottom<oldBottom)
> mList.smoothScrollToPosition(ListView.FOCUS_DOWN);
>
> —
> Reply to this email directly or view it on GitHub.
>


You need to download the hpi file, not the jar file, then use the plugin manager "Advanced" tab to upload that hpi file to your Jenkins server.  Jenkins will handle the deployment, the unpacking, and all the other things necessary to use that hpi file.


I checked before doing it this way, we don't.  We always generate as BlkOpKindHelper.


Comment is out of date


GroundDomainUpdate (sushobhit) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Let's refactor this to a method with a more meaningful name, like `getDefaultPeekTranslation`


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 218](https://teamcity.labkey.org/viewLog.html?buildId=263263&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabkeyTrunk :: build Build time: 00:00:00


We specifically named them with `get` because `isPeekOnDismiss` doesn't really make grammatical sense. Please revert these.


Check the require_once in this file... require_once (__DIR__ . '/../core/Context.php');
you have a cyclic include.


ok


Should have a newline between the description and the params


Just to make sure I understand, you are talking of veth42 being managed by networkd? Then there is indeed a bug. The intention is to never exit the second loop until all the networkd managed interfaces that we are aware of (and thanks to the first loop veth42 will be one of these) have been fully configured. Looking at the code again, I see a possible race, I will post a PR.


This assignment stuff with the optional seems to work pretty well here.  Looks good.


LGTM


Do you want this branch to make it into release v1.11.0?
If not I'd like to make a release soon since it's been a while and there's been many new features since the last release.


Thanks! I sorted the lines in order of increasing size for readability.


:icecream:


A constructor initializer list should be used here and the arguments should be moved and not copied.


remember, this is global to the otr3 package. I'm not sure if using the name QueryMessage for something that has v2 in it is a good idea.


Naked new without returning `std::unique_ptr` or `std::shared_ptr`.


`origin` should be moved if it supports move semantics.


This opened up a can of worms and big change so I'm going to rewrite the last few commits to incorporate Howard's change.


`origin` should be moved if it supports move semantics. `original_text` should be moved.


Oh I missed comment this out

2015-07-28 18:34 GMT-05:00, Ola Bini <notifications@github.com>:
> remember, this is global to the otr3 package. I'm not sure if using the name
> QueryMessage for something that has v2 in it is a good idea.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/twstrike/otr3/commit/c0ecd32352b1ac1cfb56fb845524fb618c9e1d17#commitcomment-12409160


Yes, now fixed.


I found `idea.images.first` worked for me. Adding `.url` didn't return anything. Sorry I couldn't investigate it further.


`origin` should be moved if it supports move semantics.  `original_text` should be moved.


Thanks - I'll add you on my readme too ;)
I'm doing stuff a bit differently than you but it has been great inspiration.


origin should be moved if it supports move semantics. original_text should be moved.


Neat.  Is the use of `QRCode.propTypes` because `this.props` may be a subset?


getPeekSheetTranslation is called in others methods, it will refactor those calls as well. Ok?


Two spaces is and has always been the correct thing to use when your font
is fixed width, our fonts are fixed width.
The 'modern' change to using a single space is solely based on the use of
proportional-width fonts.


@jglick Indeed, my mistake.


its essential to always do a 'git status' before "git add's" otherwise its
all to easy to commit things that happen to be in your local directory tree.

On 28 July 2015 at 16:12, Peter David Carter <notifications@github.com>
wrote:

> Good tip, thanks :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hoegrammer/democracy/commit/3c9469c7815c4811c520acb267071a7581b9bc3e#commitcomment-12398613>
> .
>


Oh no, I meant just refactor the calculation out

```java
public float getPeekSheetTranslation() {
    return peek == 0 ? getDefaultPeekTranslation() : peek;
}

private float getDefaultPeekTranslation() {
    return hasFullHeightSheet() ? getHeight() / 3 : getSheetView().getHeight();
}
```


Oops


Awesome, I'm really close now. When I use

```shell
cmake -DCMAKE_BUILD_TYPE=Debug \
      -DCMAKE_INSTALL_PREFIX=$HOME/kodi \
      ..
```

everything works:

```
Install the project...
-- Install configuration: "Debug"
-- Installing: /home/garrett/kodi/lib/kodi/addons/game.netplay/game.netplay.so.1.0.0
-- Up-to-date: /home/garrett/kodi/lib/kodi/addons/game.netplay/game.netplay.so.15.0
-- Up-to-date: /home/garrett/kodi/lib/kodi/addons/game.netplay/game.netplay.so
-- Installing: /home/garrett/kodi/share/kodi/addons/game.netplay
-- Up-to-date: /home/garrett/kodi/share/kodi/addons/game.netplay/addon.xml
-- Installing: /home/garrett/kodi/share/kodi/addons/game.netplay/resources
-- Up-to-date: /home/garrett/kodi/share/kodi/addons/game.netplay/resources/settings.xml
-- Installing: /home/garrett/kodi/lib/kodi/addons/game.netplay/resources/libXBMC_addon-x86_64-linux.so
-- Installing: /home/garrett/kodi/lib/kodi/addons/game.netplay/resources/libKODI_game-x86_64-linux.so
```

but when I try to install to my kodi source repo:

```shell
cmake -DCMAKE_BUILD_TYPE=Debug \
      -DCMAKE_INSTALL_PREFIX=$HOME/workspace/xbmc/addons \
      -DCMAKE_PREFIX_PATH=$HOME/kodi \
      -DPACKAGE_ZIP=1 \
      ..
```

it looks like I need to provide another parameter to cmake:

```
Install the project...
-- Install configuration: "Debug"
-- Installing: /home/garrett/workspace/xbmc/addons/.//game.netplay
-- Up-to-date: /home/garrett/workspace/xbmc/addons/.//game.netplay/addon.xml
-- Installing: /home/garrett/workspace/xbmc/addons/.//game.netplay/resources
-- Up-to-date: /home/garrett/workspace/xbmc/addons/.//game.netplay/resources/settings.xml
-- Installing: /home/garrett/workspace/xbmc/addons/game.netplay/game.netplay.so.1.0.0
-- Up-to-date: /home/garrett/workspace/xbmc/addons/game.netplay/game.netplay.so.15.0
-- Up-to-date: /home/garrett/workspace/xbmc/addons/game.netplay/game.netplay.so
-- Installing: /home/garrett/kodi/lib/kodi/addons/game.netplay/resources/libXBMC_addon-x86_64-linux.so
-- Installing: /home/garrett/kodi/lib/kodi/addons/game.netplay/resources/libKODI_game-x86_64-linux.so
```

what do you think i'm missing?


I had to do a kind of hairy merge here w/ waldos latest commit, so apologies if I messed anything up


This causes mega-jank


Sure, when you jam it somewhere that it's not supposed to go...


Could just use `Option<selectors::PseudoElement>`


As I recall, the general idiom is to call `return Transactor::preflight(ctx)` here (rather than `return tesSUCCESS`).  If there's a good reason not to let the Transactor base class finish it would be good to leave a comment here.  Otherwise someone might think it's a bug.


Hi again,
Thanks for the info, I did that and the new plugin version does now show up.
However I still get the same exception after a few restarts. Are there other dependencies/steps required?


Hey Frank, moved this to the Vigor fork so we could use the Docker builder.


is there a strong motivation for switching from the word "illegal" to the word "invalid" here?


`return Transactor::preflight(ctx)`?


I like smashing things


oh, I see, #27288 ; let me go read that issue.


I've got some code laying around somewhere that does some 'dumb' testing on private constructors for final utility classes.  Seems like that might be useful to also include.  That might be going overboard but it certainly would hide even more boilerplate code that shouldn't be typed all over.  I'll dig that up after I digest all this.  So far loving all this.


trying to understand how this defaults to dasherize..


You can use this "arrow to the line" format instead, which can be slightly nicer to read:

```
//~^ ERROR cannot cast as `bool`
//~|   HELP compare with zero instead
```

(and likewise for the other cases below.)


I like the naming - very clear :+1:


Whoops. Sorry, should have caught this.


this commit looks fine to me.


CAralho, revisa isso!


Please move `removeCompletion` above the `if` statement so it doesn't need to be duplicated.


Do not commit that :)


Is the correct way to move the arguments simply to call move() on each? Should the ints also be moved, or only the non-primitive types?


using instead of the try finally?


We need to find a better way... :monkey:


The thing that's jumping out at me is that this would be a good opportunity to stop addressing languages by the English label.  It would be nice to decouple from this pattern completely, and just have a list of registered languages somewhere, indexed by ISO code.


Yes, that's the correct way to move them.  Only primitive types need to be moved.

The way it is now, this constructor takes these parameters by value, creating a copy.  Then we assign without moving them, creating another copy.  For things like `vector<string>`, that can be quite expensive.

For constructors, you want to take by value like you have here, but use `std::move` on the argument when initializing in a ctor initialization list.


Missing a comma.


@vhcandido issue #13. is correct??
print like this:
searching .....
Couldn't find many servers, would you like to continue? [Y/n] Y
searching .....


Created website.


Never mind. It was my mistake.


good


eagle eye


What you building man ?


note that `dct` is an exported function from base, bad practice to use it as a variable name


Would you mind #ifdef'ing out the body of these GCLog::Should* methods and the s_forcedGCInProgress variable? From what I can tell its functionally correct now, but #ifdefing out makes it more obvious to other readers that it will return FALSE.


This bug isn't in 2.1.x :wink: It was introduced with #643


build_fixes-20150727 (lefticus) - Win64-Windows-7-VisualStudio-12: OK (2207 of 2231 tests passed)

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


sffds


@yehudasa Thank you! I've updated the patch as your comments.


One interesting thing I find is get_params() actually doesn't return any other value than 0:
https://github.com/ceph/ceph/blob/master/src/rgw/rgw_rest.cc#L617-L625
Should we just remove the error check here


whoamiコマンドが正しく処理できました。


Moved to http://pmt.mcpe.me


@alienaut @meinac https://github.com/rubytr/ruby-tr/commit/d97fc265f981ad1da3e59cb35b51df91bbdd462a


add login test


Links edited


I guess I should stop using cfx then..


fixes #8


It looks like you were having some git issues and attempted a `git merge`?

Please make sure to manually resolve the merge but deleting all of those extra comments like

    ======
    >>>>>>> master
    <<<<<<<< HEAD

etc

These are are added by git to show which branch the code from to help resolve the merge


File Handling and Mails/Assignment20/FileHandling/DirectoriesAndFiles/Assignment20.cs

Avoid unnecessary spaces


No need to mention finally if not required


warning?


Give proper spaces.


File Handling and Mails/Assignment22/Assignment22/BinaryFile.cs

Make a generic catch also that takes Exception as parameter so as to handle any other exception that may arise.


+1
I will remember it.


File Handling and Mails/Assignment21/Assignment21/StudentDataHandler.cs

Follow Naming Conventions.


experimental area


add


+1
I will do it from next time


fdgf


Interesting, do you know if this occurred when building coreclr.dll or mscordaccore.dll? I'm guessing this is the __GlovalVal<T>::operator& which should only be showing up in mscordaccore.dll.

These routines don't need to be compiled into mscoredaccore (and if they ever executed there it would be a bug). Unless it causes refactoring chaos lets put this whole set of inlines under #ifndef DACCESS_COMPILE and restore the '.' operator.


SQLCommand can also be put in using.


Fixed The UI you proposed, made button mechanics work better, other changes. Test it out.


I think it's a statistical issue : p

TL;DR not quite random enough :)

The `_random_filename` function has only `26^5` combinations, we don't clear the folders used by the `PythonTest` user after tests, so these have been accumulating for quite some time. If I'm doing the right birthday paradox calculation (http://stats.stackexchange.com/questions/25211/simple-combination-probability-question-based-on-string-length-and-possible-char) we should only expect ~1500 trials until we're at a 10% change of having a clash

```python
>>> 0.5 + sqrt(0.25 - 2*(26**5)*log(0.9))
1582.7945535421156
```

If the python tests run 5 times (now only 4 though) for different combinations of settings, that means we only needed to have circle run tests ~375 times before having a 10% chance of a name clash. I'm going to increase the length of the string and have this test clean up after itself.


add_fixtures_to_VS_solution (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1260 of 1260 tests passed)

![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-69.95%25-green.svg)


Close the connection.


Dot Net Induction/File Handling And Mails/Assignment21/Assignment21/ReadingCSV.aspx

+1
ok


Dot Net Induction/File Handling And Mails/Assignment21/Assignment21/StudentDataHandler.cs

+1
I will remember it.


Dot Net Induction/File Handling And Mails/Assignment21/Assignment21/Utility.cs


What about just linking to the mapnik style guidelines?


File Handling and Mails/Assignment23/Assignment23/SendEmail.aspx.cs

Generic details can be fetched from Web.config and must be kept there.


build_fixes-20150727 (lefticus) - i386-Windows-7-VisualStudio-12: OK (2212 of 2231 tests passed)

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


Give proper names of IDs.


Please use an enum, not ``#define``. Also please rename the header to ``UnsupportedCallErrors.h``.

If you have time please find all other uses of ``exit()`` and ``_exit()`` and add appropriate enums for those and use them.


Probably not.


問題なく動きました。
できれば前のnameコマンドの際”名前の変更を全員に知らせる”部分などがありましたが、postメソッドに変えてもらえるといいと思います。任せます。


这个新版的buttons 。。。不知道哪儿有点问题不能用。。。（编译没问题，就是lcd不显示）


@motivatedHamster Can we use Postgres only for production (when we push to Heroku)? I want to use sqlite3 for dev and test since it's easier.


Hello devs ...
I cherrypicked qs customization and d necessary commits from this project and thanks for that ... But when i open the settings go to qs .. i get an fc of settings ... Can you help me to fix that bug ...


I assume the 'NetcdfResource' and other resource type classes be proxy classes and be inheriting from the 'BaseResource' class like the 'GenericResource' one class at a time to minimize migration issues.


TimeSeries-Fix (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2171 of 2228 tests passed)

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


:boom:


Can you give pointer to relevant part of OpenShift code? Not trying to make a change of it makes sense.  I can look tomorrow for the pointer as well.


For #113.


You know that `ter` will be `tesSUCCESS` here; you should just `return { tesSUCCESS, ... }`.


TimeSeries-Fix (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@bamboo3250 any idea what this is for?


update_sqlite_library (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1776 of 1776 tests passed)

![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)


how does this work???


Hi there. droppin by and checkin your code. looks neat, I guess this is Qt focused. If you want to write a stable IMG interface, please listen. IMG files are like Win32 virtual memory. It is an array of pages of size 2048 and you can allocate file data on it. You should even optimize IMG archives by sorting the data and group relevant things together, like TXD with relevant DFF file. The basic things are on gtamodding of course! Good luck!


Nice Done.


Cool glad that it did something :) its probably more consistent from needing less Garbage Collection.


update_sqlite_library (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@cicku
If you just want to upload a larger image to that page, you can use [bleachbit.svg](https://github.com/az0/bleachbit-misc/blob/master/bleachbit.svg) to scale to any size you want.

If you're suggesting adding larger image sizes to the `/usr/share/icons/hicolor/` directory, I've been thinking about suggesting that as well, since some application launchers use larger icons now (unity, gnome).

Which where you asking about? Does this solve your issue?


Yeah, long story short  it was late, I was feeling like I hadn't done anything all week, so I quickly wrote this shoddy code just to get out of my funk.

As shitty as it is, it does implement the algorithm in question (radix sort) which is all I cared about.


Please indent this one level. The weird indentation here makes it hard to visually understand what's going on. I think that my personal preference would be to just stuff the lambda in a local, and return the local:

    auto completion = [&view,&tx,flags,&config,j] ()
    {
        ...
    };

    return { tesSUCCESS, completion };

Question: why can't the apply callback not be a regular function, and we return a `std::bind` to it?


yet another testing


$ python glm_parser.py -i 5 -b 2 -e 21 -t 0 -p ~/penn-wsj-deps/ -a --learner=average_perceptron --fgen=english_1st_fgen --parser=ceisner

07/27/2015 06:39:33 PM INFO: Unlabeled accuracy: 0.885664463628 (41140, 46451)
07/27/2015 06:39:33 PM INFO: Unlabeled attachment accuracy: 0.890205077317 (43061, 48372)


as we discussed api naming, this is not the name we need, as user don't even have field Status, moreover, field with such name is used for another purposes


@troydai This message needs to be more generic. The other case where this can happen today is when the project is unloaded.


blah


Error con javax.crypto. Te funciona compilar usando el build.xml?


should there also be a "-n 1" here?


it is not gameId, in case when it is private game, it is a secret code, so I named 'gameId' and 'secretCode' as gameIdentifier. Maybe not the best name but it is called as it is said in method name, if you are going to rename field to more pretty name, rename both, field and method.


Thanks! The "#define _WINSOCK_DEPRECATED_NO_WARNINGS" in socketft.cpp is good, but disabling warning C4996 for winsock2.h doesn't do anything. It would have to be disabled for the caller of the deprecated functions, just tested with Visual C++ 2015.


<?php  ?>


Все-таки лучше записать результат `createConfigFile()` в переменную и передать ее в `createModelFile()`. Как-то это больше в джава-стиле.

Хотя в си подобное выглядело бы очень модно, но мой взгляд.


Можно инициализировать внутри `try`, и тогда не нужно инициализировать нуллом.


see issue #313


this log is never shown anymore when shit happens. When I tried to join private game with empty privateCode, I have seen the following line in logs: ""<< Cannot get game with empty gameId"" and I spent some time to understand, that this is related to empty privateCode) we should think how to improve it


Was this intended to be plaintext?


let's compare with the previous result that was above 90% UAS. do you know
which commit got the highest accuracy in the past.

On Tue, Jul 28, 2015, 12:52 PM Yizhou Wang <notifications@github.com> wrote:

> $ python glm_parser.py -i 5 -b 2 -e 21 -t 0 -p ~/penn-wsj-deps/ -a
> --learner=average_perceptron --fgen=english_1st_fgen --parser=ceisner
>
> 07/27/2015 06:39:33 PM INFO: Unlabeled accuracy: 0.885664463628 (41140,
> 46451)
>
> 07/27/2015 06:39:33 PM INFO: Unlabeled attachment accuracy: 0.890205077317
> (43061, 48372)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sfu-natlang/glm-parser/commit/641428b53d8c15056e1cf2ac74c2624f6d7c2677#commitcomment-12387564>
> .
>


Responsive Vim statusbar based on window size


Why F・G・D needs additional check?


you are throwing it)


Me funcionaba pero luego del push y el merg dejó de funcionar. Estoy
buscando solución.
El 28/7/2015 12:56 AM, "Raimil Cruz" <notifications@github.com> escribió:

> Error con javax.crypto. Te funciona compilar usando el build.xml?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/raimil-cruz/jgitplugin/commit/9a28e84f8eb8ac0e45af71512a262b9748b0a4a0#commitcomment-12387592>
> .
>


Hi. Don't know yet how git is working. Just been able to clone the repo locally by using the GUI of GITHUB. And changed the file to have french (FR).

How do I create the patch? I see buttons to create a pull request and another one to commit (Would not like that I would be able to commit anything without a active developer review first) Do I need to create a branche for this? I will need to look more how github is working...

Here is a link to the modified file from my web site:
http://www.clavet.org/files/URHO/EditorStrings.json


Trying to be a good Unix citizen!


maybe we should throw an exception as we do it in other cases when recurring operation happens?


What do you mean?


`http` rather than `https`.


Thanks! I will keep it here for now because I am coming back to this code


Acabo de hacer commit y push solucionado el problema. Basicamente adicioné javax-crypto.jar dentro de la carpeta libs y adicioné la dependencia al build.xml


This has now been tested by several people: none of them ever had a two-partition USB-stick automounted as USB.
Some relevant parts from a bootlog (first partition was exFAT):
EXT4-fs (sda1): mounted filesystem with ordered data mode. Opts: (null)
/etc/mdev/mdev-mount.sh: line 223: can't create /tmp/mdev-mount.log: nonexistent directory
UDF-fs: warning (device sdb1): udf_fill_super: No partition found (1)
mount: mounting /dev/sdb1 on /media/usb failed: Invalid argument
FUSE exfat 1.1.0
WARN: volume was not unmounted cleanly.
/etc/mdev/mdev-mount.sh: line 223: can't create /tmp/mdev-mount.log: nonexistent directory
EXT4-fs (sdb2): unable to read superblock
EXT4-fs (sdb2): unable to read superblock
isofs_fill_super: bread failed, dev=sdb2, iso_blknum=16, block=32
attempt to access beyond end of device
sdb2: rw=0, want=3, limit=2
UDF-fs: error (device sdb2): udf_read_tagged: read failed, block=256, location=256
UDF-fs: error (device sdb2): udf_read_tagged: read failed, block=512, location=512
UDF-fs: error (device sdb2): udf_read_tagged: read failed, block=256, location=256
UDF-fs: error (device sdb2): udf_read_tagged: read failed, block=512, location=512
UDF-fs: warning (device sdb2): udf_fill_super: No partition found (1)
mount: mounting /dev/sdb2 on /media/sdb2 failed: Invalid argument
FUSE exfat 1.1.0
ERROR: exFAT file system is not found.
/etc/mdev/mdev-mount.sh: line 223: can't create /tmp/mdev-mount.log: nonexistent directory
EXT4-fs (sdb5): recovery complete
EXT4-fs (sdb5): mounted filesystem with ordered data mode. Opts: (null)
/etc/mdev/mdev-mount.sh: line 223: can't create /tmp/mdev-mount.log: nonexistent directory

From another bootlog (first partition was FAT32):
EXT4-fs (sda1): mounted filesystem with ordered data mode. Opts: (null)
/etc/mdev/mdev-mount.sh: line 223: can't create /tmp/mdev-mount.log: nonexistent directory
/etc/mdev/mdev-mount.sh: line 223: can't create /tmp/mdev-mount.log: nonexistent directory
EXT4-fs (sdb2): unable to read superblock
EXT4-fs (sdb2): unable to read superblock
isofs_fill_super: bread failed, dev=sdb2, iso_blknum=16, block=32
attempt to access beyond end of device
sdb2: rw=0, want=3, limit=2
UDF-fs: error (device sdb2): udf_read_tagged: read failed, block=256, location=256
UDF-fs: error (device sdb2): udf_read_tagged: read failed, block=512, location=512
UDF-fs: error (device sdb2): udf_read_tagged: read failed, block=256, location=256
UDF-fs: error (device sdb2): udf_read_tagged: read failed, block=512, location=512
UDF-fs: warning (device sdb2): udf_fill_super: No partition found (1)
mount: mounting /dev/sdb2 on /media/sdb2 failed: Invalid argument
FUSE exfat 1.1.0
ERROR: exFAT file system is not found.
/etc/mdev/mdev-mount.sh: line 223: can't create /tmp/mdev-mount.log: nonexistent directory
EXT4-fs (sdb5): mounted filesystem with ordered data mode. Opts: (null)
/etc/mdev/mdev-mount.sh: line 223: can't create /tmp/mdev-mount.log: nonexistent directory

I really hope someone can look into this.


1. Ich hatte junit als *optional imported package* drin, weil ich dachte, das die Junit-Sachen eh in jeder eclipse Installation drin sind. Aber offensichtlich unterscheiden sich unsere Target-Plattformen. An dem Thema  sollten wir auch mal arbeiten, also Buildserver + Nexus-Repository + Eigene Targetplattform.

2. Wegen des Testens war ich mir auch nicht sicher. Mockito brauch ich nur zum Test, die sollten also nicht mit in bundle.jar gepackt werden. Bei eclipse builds kann ich den *Scope* der dependencies aber leider nicht unterscheiden, weshalb dort empfohlen wird die tests in separate Plugins auszulagern. Sonst sind die Testklassen ja auch mit in dem produktiven bundle.jar drin. Hmmm. Aber das macht mehr Aufwand.


> Returning `actor#ID` would require to keep a string member around.

Actually, what I really wanted is something like this in addition to `name()`:

```cpp
std::string description() const {
  return name() + "#"s + std::to_string(id());
}
```

This comes at no extra cost. Then it's up to the user to decide whether they want a per-actor or per-actor-instance tag. With SSO available on reasonable compilers, you may even get away without a memory allocation for concise actor names.


Thank you! I'm very happy to say I got add_player to work in my latest commit! However, I simplified a lot of the code because I was trying to debug an error, which ended up being a simple example of this:(http://stackoverflow.com/questions/14247732/python-syntaxerror-non-keyword-after-keyword-arg) in the __init__ parameters for the Transaction class.

To test the other functions I will undo some of the commenting out to make the program more flexible. I'm referring in particular to my use of a dictionary in line 184 to call different xml_builder functions based on the transaction type. Do you think this is a good way of doing this? I was trying to be a little more sophisticated than writing a bunch of if statements :grinning:

I know the code is perhaps a little less elegant with multiple build_xml functions. But I thought it was the easiest way to make the function calls in tests.py as simple as possible. For example, I wanted the parameters for add_player to be just the player code of the player you wish to add to your team. Right now you do have to also type in your team code and league code, but I envision those being moved to more of 'global' variables soon.

If there were only one build_xml function for all of add_player, pending_trade, edit waiver, etc... then the build_xml function would have tons of if statements because the input XML string is very different for each of them. That's my justification for splitting them up into more but simpler xml_builder functions.

Take care,


when will we get cm12.1 officially?


I should be able to start game when number of users more than minimum


update_sqlite_library (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (742 of 742 tests passed)

![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-7.11%25-green.svg)


Hi Falko, das ist in der Tat schwierig. cssName und minWidth sind nicht notwendig, wenn sie aber gesetzt werden, dann muss das im *create* sein. So gesehen sind sie immutable und mandatory aber mit dem default *null*. Ein späteres Setzen nach dem *create* ist nicht möglich. Und das wollte ich mit dem ctor ausdrücken.


you have changed Preconditions to Parameters but I can still say: I would rename this method, as noone understand, what parameters should I check, if I am passing both user related details and game related details. Maybe split this method to user validation check an game validation check?


boo


yooka


@snowdream Thx!


My oversight ... yes, `dct` can mean DCT transform ...


we have contstants for such things, and you are using it in this test case above)


Ahora todo ok. Lo raro es que esas clases son dependencias del core de java
y no debieran ser agregadas manualmente.En fin, muchas gracias

*Pedro Pablo Bustamante Barrera*Estudiante Ingeniería Civil en Computación
Universidad de Chile
Fono: +56976974143

2015-07-28 1:08 GMT-03:00 Raimil Cruz <notifications@github.com>:

> Acabo de hacer commit y push solucionado el problema. Basicamente adicioné
> javax-crypto.jar dentro de la carpeta libs y adicioné la dependencia al
> build.xml
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/raimil-cruz/jgitplugin/commit/9a28e84f8eb8ac0e45af71512a262b9748b0a4a0#commitcomment-12387679>
> .
>


@davidfowl the current text does merely hint at one possibility... but anyway what would the most generic comment be? I structured the second sentence around the only scenario I was told about.


git diff have too much change, but below almost not change


according to these conditions, when min players is 1, game should be started already)


Thanks for fixing it.  Golly it's so close I wonder how I noticed and why it bothered me.  You couldn't have moved it more then a few pixels.  I feel like I must be crazy.  I mean, It's clearly fixed, but the why I know is really puzzling me.


I have updated it. you have just added break in for cycle. But what is the difference with getGameByGameIdWithJoinedUser method?


from grouped `externalImportsMap` instead of externalImports


fetch node


Yes, the list monoid is horribly inefficient because it uses list concatenation.


@FWAJL
I made a mistake, and I can't fix it because I am in the middle of a large Issue.  This is the error:

       <?php if ($user->getAttribute(\Library\Enums\SessionKeys::UserRole) == 3): ?>

` -    <?php endif; ?>`
`     <figure class="logo"><img src="../Web/images/FWM_logo_only.png" alt="Mobile logo"/></figure>`
` +<?php endif; ?>`


build_fixes-20150727 (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2177 of 2231 tests passed)

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


For the record, this forcing of `lib` is also wrong for Fedora (and for Mageia and openSUSE and other similar distros, I believe) where shared libraries should be installed to `/usr/lib64` on x86_64. See https://github.com/roboptim/roboptim-core/pull/101#issuecomment-125431336 .


Yes, it's still relevant because all events are hold in memory.


Passing `IHashRouter` to a member of `STTx` violates levelization.


why did you drop all those tests?


Hi Leon,

I have some smaller comments on this commit.

Stefan


build_fixes-20150727 (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I think it's for displaying available rows while the page is still being generated.


Oh, good point. I just copy/pasted the URL, didn't think to check if fossies served over HTTPS.


I think we wanted to change this back to a function to harmonize with radius(), charge() etc.
Or make all these properties... will break lots of code but now would be a good time. Opinons?


@dukesrg thanks for your effort. Or just set it to "en-US" the first start.


Should work :). Sadly Fossies doesn't auto-redirect to SSL/TLS, so when we use `http` it actually serves `http`.


Fixed in d7d89b462d54c3a442cb9cc0476f81213e62c3b6!


Thanks Misty :)


Hello World!


Fascinante


Hi, the markers in lines 1, 872 and 1743, are GIT conflict marks?


@flavour I applied the filter on the resource here https://github.com/biplovbhandari/eden/commit/782563459960709ddcd30537b5f164dbf2e9762e#diff-59bf1eeed7c39491fc28b7a977ef8b0cR2050 Is that what you mean?

Consider for cyclone. A place 'X' near the cyclone the origin of Cyclone has 'high' priority of warning, than area 'Y' with 'low' priority. With the movement of Cyclone, area 'Y' has 'high' priority,  area 'X' with 'medium' or 'low' priority plus another area say 'Z' with 'low' priority (in future 'high' priority)


proposal:

var text = data.text ? data.text : $window.document.title;
urlString += 'text=' + encodeURIComponent(text);


Because AES128-CTR generate a cipher with the same length of the plain text.

From the spec:
> Note that counter mode does not change the length of the message, so no message padding needs to be done.


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2211 of 2231 tests passed)

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


Merging from upstream


But to be clear... the actual problem was to simply solve the "delete_node" issue.  And this is solved in O(1) time.


이렇게 하면 문제가 될 확률이 많아.
예를 들어. 지금처럼 body 밑에 스크립트를 넣으면 상관없지만, body위에 넣는 경우 정상적으로 동작하지 않을꺼야.


If it's possible, I think it would be better using `String` in "iot_util.h" which now should be initialized at construction but we can add functionality for it to update its contents.


This is `:initializers`


Function name should be UpperCamelCase according to coding convention.


잘했어~
근데 fadeIn 밖으로 빼서 함수를 만드는게 효과적인 것 같아.


This last arity should be deleted.


Delete the last arity.


Use `IOTJS_ASSERT()` macro.


Now, you are pulling from the `Actor::System` and getting running `Actor` instances, no longer threads.


Delete this arity.


No longer needed, not polling `Tasks` directly


궁금할 수 있는 부분이다.
수업시간에 같이 보자.


I think it's better to expand the the inline fields rather than skip them. For example, "typemeta" should be expand to "apiVersion" and "Kind". A user will need to specify apiVersion and Kind when writing a config for Pod, we will surprise them if our doc doesn't show these two fields as part of Pod.


Looting changed so much in WoD, not only in instances loot which are based on difficulty, but also on certain items that comes as a reward for quests, or similar. For example, despite it says is a 650 blue ilevel, you have a chance for it to be 675 epic when clicking to finish quest, get item or so. Of course not with everything, but this happens.


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2176 of 2231 tests passed)

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


Btw, same happens when getting new followers,the stats and info you see is a minimum, but randomly they can be better from start


@Macrofig Did you push this to master without any tests?  Related to #1775.


@bsaiken

See last commit under the issue.


Add libtirpc-devel, please. Includes files is not supposed to be in main package.
https://wiki.eng.vmware.com/Project_Discus/Adding_new_packages#Spec_file


Web Application and Exception Handling/Assignment15/Login.aspx


Yeah implemented it after you said that the build process should depend on the cpp/pyx files ^^ It worked fine (for me).
Didn't know there would be a problem after a checkout.


Web Application and Exception Handling/Assignment15/Optimus.master

-1
page names as mentioned in the assignment


Web Application and Exception Handling/Assignment15/techNews.aspx


It was done already. Thanks


-1
Assignment mentions to catch IOExceptions explicitly


-1
Names as mentioned in the assignment


이렇게 개발한건 의도하고 한거냐?
수업시간에 질문해라.


If the size of the buffer is not enought, {l}getxattr() just return -1. (does not return of the real size of the value.) So if the return is negative then we have to get the size of the value by calling {l}getxattr() with NULL pointer and size 0.
And if we allocate the same size of {l}getxattr(), we have to get again the size for the null-terminate string.

I push this on #752


ssdssdsa


이렇게 구현하는건 그리 좋은 방법이 아니라고 했는데 개선해보면 좋겠다.


А возвращать тоже внутри try? :)


jQuery쓰는 김에 다 jQuery을 사용해봐


domcontentloaded안에 넣지 않고 여기에 넣은 이유는 뭐니?


FWIW, you don't have to pull the text out just to get the [length](http://developer.android.com/reference/android/widget/TextView.html#length())


Nitpick: don't end `description` with a dot :-)


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This is awesome!


Thanks David, I actually checked GA and I suspected something like this. I will remove it.


This doesn't catch overflow or rounding problems.


"off" is correct per the standard. Is there a specific buggy Chrome version/platform where "off" doesn't work but "false" does? (Bizarre.) I had to change it back to "off" to properly suppress the built-in list in Chrome 44/Windows 8.1.


I was just looking at this again, and are you sure "cd .." is a possible command? We are only using what is in input[0], and when we split on spaces that would put  ".." in input[1]. If it is working my guess is that your regular cd function is doing the work. You should probably debug this call and see what is really happening, you may not even need a seperate api call for going back to a parent directory.


Replace with Destroy


@moy Yup after seeing your reply I've changed this and included %(colornext:..). But the only problem with colornext is that it will reset any previous color's if set, Is that ok or should I somehow save the last used color and use that again.


why is jquery commented out? should the user of this class define? then same should apply to other css/js (BOOSTRAP*)?


@donv just liked that due driver updates on master we still test against older versions (on 1-3-stable).

SQLite is special due Android - would be great to have a test target against SQLDroid and I also re-call there has been a hack or two to handle the 3.8 version compared to the previous one (3.7.2 has been used for a long time with AR-JDBC since the JDBC driver has seemed abandoned for a while) so it would be best if 1-3-stable did not break the older ones - if one moves the .jar there to support newer features its easier to miss/break support for 3.7 ...


why are these defined here?


you should create this link only if getCSS() returns non empty string. I think browser is ok with it, but it looks unprofessional when you look at the source.


SUT means System Under Test. We also use LUB for Library Under Test. Here you can write:
```php
use Hoa\Socket as LUT;
use Hoa\Socket\Transport as SUT;
```
and use them everywhere. Sorry, the contributor guide is under view and I am starting to write the documentation of `Hoa\Test`…


The leading `\` is not needed.


Note: Removed primary key as most fields need to and can be duplicated.


If you are creating a database prefer using use statement so that if you are not in your database then also your queries work.


What are you testing exactly?


Add Comments.


One test case per behavior please.


You can use single insert query for this.


You don't actually test exception code and message, just be ensured there is an exception is fine.


Good!


Again, 2 tests cases are required here: `case_exists` and `case_not_exists`.


same a above


What is the need of adding to all the tables. The better approach is use user_master


I think your shell tried to execute the ``` `Base.file` ``` part of your commit message as code - single quotes might help


offline일 때 동작하는 코드를 구현해보면 좋겠네


@Sutharsan sorry for this but I closed this issue for following reasons:
1) @todo Make location of the binary configurable.
This code is not required over here hence I have removed it. tcpdf library doesn't requires binary file.

2)  @todo Make header content configurable and @todo Make footer content configurable.
 I have not done addition over here because now we are providing users with option to call object of pdf library directly using which they can call header and footer themself. Also since the parameters which can be sent in `writeHtmlCell()` were too many hence I coudn't come up with a way to totally generalise it(also the way this library works compares to other library differs too much which makes generalisation even more difficult)
Thanks

Regards,
Alok Pandey


> doesn't even add an end `</fuckyou>` tag

this is why it was rejected


Patch written and submitted by Antonius32.


Could you also add a test for order query by cart Id?


:dancers:


I use `m_moveFinishedTriggers` as it's name, can't think of a better one and I also added a comment block.


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2203 of 2231 tests passed)

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


what if it happens to be "a read operation of 0 bytes"? and the `result` is negative?


I was a little bit confused myself when coming back to the controller after a few days so I'd definitely agree that the `before_action` isn't that clear. I might try the middle ground solution next time.


```javascript
this.props.onChange({
  ...this.props.todo,
  checked: !this.props.todo.checked
});


Thx


@cvrebert Sorry about that, fixed now.


Good to see the R2 project isn't dead.
Keep it up!


This is still very bad: this block should not be reached by non hosts, I think?

What is it about proxy connections that makes us get here?


... Mayyyyyybé?


Spring Data is based on CDI (JTA + CDI will work on JEE7, but on JEE6 we have to use EJBs for that).

So we have:
* `BuildRecordSetRepository` repository interface (from SPI)
* `BuildRecordSetRepositoryImpl` EJB implementation of the above
* `BuildRecordSetSpringRepository` Spring Repository (CDI) used by `BuildRecordSetRepositoryImpl`

The EJB bean sets transaction boundaries.

So in this situation you have to `@Inject BuildRecordSetRepository recordSetRepo;` and use `recordSetRepo.save` method.


lobbyComm:PeerDisconnected gets called invariably when a peer disconnects.


ok


please change coding standard ..


So shall I delete this `celluloid/group/unlocker.rb` ?


Also, I updated `GNUInstallDirs.cmake` 5 months ago from upstream (cf. the [differences](https://github.com/jrl-umi3218/jrl-cmakemodules/commit/e43e1e2740bf2f8fd3e1b70c214c116cfe2b622d)). Some of the changes are directly related to multi-arch (cf. the commit message I pasted when updating it), so this may help solve the problem.


I did not use the model objects because I assumed that using `@post.topic` would make an extra query to the database to fetch the topic and pass it to the helper but after testing it appears it does not. Thanks for pointing that out, it will save me a few keystrokes in the future!


Из соображений последовательности, в ограничении, если прописано PDF, XML, то тогда уже и DOC, XLS, а не так как сейчас  ('WORD', 'EXCEL', 'PDF', 'XML')


Looks like [this refactoring](https://github.com/openstreetmap/openstreetmap-website/commit/dbe165bbb3d6eda37d9579f1ac767599f9055607#diff-3ef1e2237f323b7d4dc1ce1fc11a5c20R13) broke it.  I suggest raising a ticket on their repo, I am not sure when I can get around to fixing it myself but it should be simple enough.


1) Fork engine on site
2) Install Git program https://msysgit.github.io/
3) Download your repository
```
git clone https://github.com/christianclavet/Urho3D.git
```
4) Modify it
5) Create patch
```
cd Urho3D
git add -A
git commit -m "A brief description of the patch"
git push origin master
```
6) Press "Pull request" on https://github.com/christianclavet/Urho3D


i am okay with this change though. just a little bit concerned with your intention to optimize for the negative code path based on a pessimistic hypothesis. in general, `-EIO` is not an expected return val.

this change by itself is correct and easy to understand. but i can image when more and more of them get merged, they do not improve the maintainability of ceph in long term, or help with the performance otherwise.

if the our error handling turns out to be a bottle neck, and we do need to improve the performance. it is a  signal that something goes terribly wrong somewhere else.

what do you think?


fsd


Not scaling though, i didnt touch that.


I though this is already done by https://github.com/sphereio/sphere-jvm-sdk/commit/294dfe2a871fa43e4a7421b8fde034d7304775ce#diff-1127f3a87c60c9fa2f3f35432570443eR89


While debugging tests I commented out that code. Yes, I agree will clean up that


이부분이 새로운 코드군요. 멋져요.


И я не вижу, где для groupkey, smtpkey внешние ключи созданы. И в таких ситуациях, когда в одной таблице несколько разных вещей хранится, лучше поля группировать префиксами или называть так, чтобы не попутать сполем из другой области. Например, emailgroupkey.


Thanks


yep, online right now. Brian noticed that libraries didn't work yesterday, so I fixed it


This should actually be a factory method, as the model factories should not have to read from the database and rely on the seeder to have run.


It would only make our main group checkable, not the other groups like Settings, special events. Please correct me if I am wrong!


What the fuck is this?


No it wasn't. I updated `config/puma.rb` to use it in e0a0e284e00b53ce346a8bb61dbfbfcbf93eee99.


Sorry to bump in.

Something similar also happened when giving yourself or someone an itemset.

For example, when using .additemset 1194 (mage tier 16) it hands out 4 different sets (as in ilevel 528, 540, 553, 566 )

Sorry to make a request here. You guys doing a great job!!

For now I would love to have

CMSG_DESTROY_ITEM
CMSG_AUTO_EQUIP_ITEM

enabled to as least delete stuff from the bag and equip items that were added.


Same issues as database assignment 2  use database query should be there


@shujingke
So should I integrate it with the existing codes under opencog/embodiment (e.g. replacing the old spacemap functions with new ones)? I tried to do this but after reading your comment It seems that I don't need to do that, just wait for the new embodiment code to integrate with..


Special Events is part of the main group. And all the items in Settings do not have a drawer on them. They open another activity with a Back Arrow. The only other item that needed to be checkable was the "Arrow", so I added a setCheckable(true); to that item individually.


Closes #89


Better naming convention followed as compared to assignment 2


@julienlancelot feedback applied. Thanks


LGTM


The table name is not clear


  var child = new ProtocolTreeNode("create", new[] { new KeyValue("subject", subject) }, new ProtocolTreeNode[] { participant });

Report CS0266 Error;


Jaccard doesn't deal with the requirement that recent events are more important very well. It requires (as you know) calculating the weights twice (which is slow) and some magic numbers (which are difficult to configure).

I came across an annoying edge case that pushed me over the edge. If a person has one event within the `recent_event_days` limit then that results in them being really strongly similar to anyone else with that event, much more than they should be. Which can make recommendations really bad. The only solution to this is to make the recent_event days large enough to ensure that most users will have more than one event within its limits, which kind of defeats the point of the configuration.

I will change the recent_event_days

I am almost happy with where I am at, in GER. I want to hit a stable external GER API, and an internal ESM API soon. As soon as I do, I will let you know and will help upgrade the rethinkdb ESM.


严重性	代码	说明	项目	文件	行
错误	CS0266	无法将类型“System.Collections.Generic.IEnumerable<WhatsAppApi.Helper.ProtocolTreeNode>”隐式转换为“WhatsAppApi.Helper.ProtocolTreeNode”。存在一个显式转换(是否缺少强制转换?)	WhatsAppApi	E:\谷歌浏览器下载\Chat-API-NET-master\Chat-API-NET-master\WhatsAppApi\WhatsApp.cs	453


makes shop broken

http://forge.prestashop.com/browse/PSCSX-6390


Why foreign key are added in the end rather we should have added foreign key constraint to the table


Look [here](http://c.learncodethehardway.org/book/ex4.html) to see the whole exercise and to see what valgrind actually is.


yep that's right, $this is not available in static method.
Fixed


use Single  insert with multiple values


makes shop broken
http://forge.prestashop.com/browse/PSCSX-6390


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.14](http://pixelated.design/build/viewLog.html?buildId=83&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 287 Build time: 00:00:09


Better approach is using joins instead of sub queries.
Identifying the main table and applying joins afterwards


asdad


qwe


OMG!!!


@fnk0 I am talking about this issue (https://code.google.com/p/android/issues/detail?id=175216). Not sure if this issue is already resolved in last update! My previous point is due to this issue only because we have a submenu for "Arrow" and with a different group ID.

So only one item across these 2 groups stays checked at a time?


It should indeed, I'll fix it


@viktorklang @rkuhn

This change breaks `scala-java8-compat`. How should we reconcile this?

https://github.com/scala/scala-java8-compat/blob/master/src/main/scala/scala/concurrent/java8/FutureConvertersImpl.scala#L75-L80


ip (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2177 of 2213 tests passed)

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


yes, now it is only `actor/unlocker` beside `actor/system`


I agree. I did this to maintain the current behavior. Maybe @mattt has an opinion if a "breaking" change is okay for this PR? (unfortunately mentioning ppl doesn't work..)


You're right, further investigation shows this will be an issue. Hence, I'll need to do some refactoring and push this to a future release, as we can easily overload setAttribute and do the required checks, including updating how model construction works.


@PareshMayani You are about that 2 items can be checked at the same time. But since we are checking the item when we create the activity we don't run into the problem of having multiple items checked.

Here's a screenshot of a subgroup item checked.

![screenshot](http://i.imgur.com/5a17FUY.jpg)


Takk! Beklager å ha rotet det til med commit & run igjen...


ok to push..


Initial commit


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2213 of 2231 tests passed)

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


Nice one


@fnk0 perfect. Good contribution (Y)


distributionUrl in the gradle-wrapper.properties should point to gradle-2.2.1-all.zip


ip (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


The correct answer to the problem would be roughly 26000. I understand your approach but you may not be doing everything till the end


You shouldn't have to compute your degree. It is a field in the `Polynomial` class, so a simple `getter` on that field should suffice


Fixed.


Careful on naming conventions


You're entirely right. I slipped up and got `BrowserStackLocal` in there.


@PareshMayani Thanks!


Why 20 and not `length`?


You've done great so far. Keep it up


*In fact the package works:*

root@backsrv:~# dpkg -i keepconf-1.2.00.deb
Selecting previously unselected package keepconf.
(Reading database ... 45884 files and directories currently installed.)
Preparing to unpack keepconf-1.2.00.deb ...
Unpacking keepconf (1.2.00) ...
Setting up keepconf (1.2.00) ...
Processing triggers for man-db (2.7.0.2-5) ...
root@backsrv:~# keepconf -V
keepconf version 1.2.00

*Most likely I used a corrupt package.*

*Now I have the following scenario: I want to make automated backups from a web server using a cron job. At the moment I can do it as follows:*

rsync -avz --delete  -e "ssh -i /root/rsync/mirror-rsync-key" backman@websrv.actf.co.mz:/var/www /backup/

*How to achieve the same using keepconf?*
*rgds*


I propose:

```
  /**
   * Returns a Scala Future that will be completed with the same value or
   * exception as the given CompletionStage when that completes. Transformations
   * of the returned Future are executed asynchronously as specified by the
   * ExecutionContext that is given to the combinator methods.
   *
   * @param cs The CompletionStage which may eventually supply the completion
   * for the returned Scala Future
   * @return a Scala Future that represents the CompletionStage's completion
   */
  def toScala[T](cs: CompletionStage[T]): Future[T] = {
    val p = Promise[T]()
    val bc = new BiConsumer[T, Throwable] {
      override def accept(v: T, e: Throwable): Unit = {
        if (e == null) p.complete(Success(v))
        else p.complete(Failure(e))
      }
    }
    cs whenComplete bc
    p.future
  }
```


why would you need this method? you have the field `degree` which you could simply return, right?


LGTM


sorry to come to bother you again I have a question, I've ridden server goes well but when I leave it working in the background works but to spend 24 hours the server automatically closes, despite the fact that only one user to connect try creating a demons but not with that work the server falls without any reason will be that you can help you appreciate for your time and help forgive the inconvenience


I can accept this approach as well. Instead of having your functions in a different class, you put them in the `Polynomial` class and so, whenever you want to multiply one polynomial with the other you will do something like `p1.plus(p2)` instead of doing `functions.plus(p1,p2);`
It's ok, it's quite alright, however, you may later find out for yourself that if you can trim down the size of a class, you will want to do that. But let's leave this for later


It is mainline driver. We want to have all drivers upstream.


@m-kuhn can you please check that the manual example using the bridge is still valid? https://github.com/qgis/QGIS-Documentation/blob/master/source/docs/pyqgis_developer_cookbook/loadproject.rst#id21


Again, I don't have anything to do with this project. So please don't quote me using at-tags.


This method breaks the rule which you started with. The previous (and subsequent) methods `plus`, `minus` etc. work on the current polynomial + another which is passed as a parameter. Wouldn't it make sense that this method should work only on the current polynomial? if yes, then do you truly need another `Polynomial` passed as a parameter?


Looks good. However, Could you rebase your work on master:HEAD ? I have changed how machine files are named. Thanks (and sorry for the delay)


I think you can use generics to get rid of this `rawtypes` warning, but since you didn't really know about them yet, it's quite ok


Step 5 is not neccessary.

Best regards,
YaoWT

On Tue, Jul 28, 2015 at 2:06 PM, 1vanK <notifications@github.com> wrote:

> 1) Fork engine on site
> 2) Install Git program https://msysgit.github.io/
> 3) Download your repository
>
> git clone https://github.com/christianclavet/Urho3D.git
>
> 4) Modify it
> 5) Create patch
>
> cd Urho3D
> git add -A
> git commit -m "A brief description of the patch"
> git push origin master
>
> 6) Press "Pull request" on https://github.com/christianclavet/Urho3D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/urho3d/Urho3D/commit/d00438c224c3bcc851fda74f87fb68396f39aecc#commitcomment-12388647>
> .
>


This looks much better, well done


I haven't started a full review yet. Just glancing stuff over for now and noted this issue. When you create a new directory to store files, need to create the new directory in the $GLOBALS['OE_SITE_DIR']/documents/ directory. Then you will be leveraging the security, upload privileges, and backup mechanism that already exists for the documents directory.


ping @weilu


Much better!


closes #21


This was nice :)


if your FS still use struct nameidata *nd
then yes, that is correct.
i think in code there are checks for this, if you above 3.5 or 3.6, then you already should have unsigned int flags pointers.
if not then nameidata pointers.


hhahahah


Thank, I will remove it from dotfiles.


typo


also closes #3


Oh boy... one of those remnants that until today never got implemented ^^

You might leave it in, just mentioning that `filter` is not yet supported in the API -.-


ok to go


It's escaping for brackets, may be should be done in tw too, not sure, ignore it :)


I changed it to vunit_hdl


\o/


I'd rather remove it then, not sure if having this in there makes any sense in its current form.


Why do we pass a variable that is not used in the function? Why don't we calculate the length inside the function?


Is it reasonable to check for `>0` if we calculate the length inside the function?


Just a general note on consistency: I usually prepend a verb here (e.g. `list revisions`), to distinguish it from POST and PATCH actions.

We can leave this as-is now. At a later date we will have to touch it anyway and rename it to something like "revisions by work package", because we add "revisions by repository". Lets hope that this day will come ^^


Nice catch, thanks! I will work on getting this fixed.

Right now I'm thinking of using [atexit](http://cmr.github.io/ioctl/ioctl/libc/funcs/c95/stdlib/fn.atexit.html) if that works or implement something like this here - https://github.com/rust-lang/rust/issues/11695

If you have some thoughts and ideas to share that would be great, thanks!


We shouldn't set `0` as a default value for any variables. Using special initial values (if not necessary) in order to indicate invalid state is always a bad practise.


ghfilter
- .*\.java
- *.java


Excuse my ignorance, but is there a way to do that without rolling back all the commits since then?


Didn't had any better solution than that actually.
Open for better solutions.


Is it possible also to parse number of room occupants and place it to separate field? It is usually part of name e.g. "room name (15)"


Explain more about the base case.


--OK


I think I prefer keeping jshint and eslint rules in the package.json file. Otherwise this starts to look like a parody of an open source JavaScript project really fast - two actual .js code files, but a million dotfiles and other framework / buildsystem files. :)


--ok


okokok


I think that the second argument should just be ~base. Otherwise you're guaranteed to nest at least once.

Also, explain at some point that recursive == nested within itself.


Why is this needed?


# This is a right test!


@EdTorbett Again, I don't have anything to do with this project. So please don't quote me using at-tags.


Ah, OK, I get it.  I'll look into testing 3.7.x and 3.8.x in parallell, and maybe against SQLDroid as well.


Когда таблица прописывается в скриптах, то к ней сразу рисуется и триггер на присвоение ИД. Его надо сделать.

Второе, такую логику делают следующим образом:

* задвигают триггер в конец последовательности
* обязательно исключают текущую запись

т.е. должно выглядеть так:
```
CREATE OR ALTER TRIGGER gd_biu_smtp FOR gd_smtp
  BEFORE INSERT OR UPDATE
  POSITION 32000
AS
BEGIN
  IF (NEW.principal = 1) THEN
    UPDATE gd_smtp SET gd_smtp.principal = 0 WHERE id <> NEW.id;
END
```


No es la version autoload es la version crear preguntas.


Title Case Preferred in Titles :)


哎哟喂。。狗眼  不是黄头发差评


Please be aware that you're commenting in a topic over a month old, and that nobody has mentioned you since.

I'm sorry for the original mistake in which I referenced you. If you want to stop receiving notifications (which only you can do) you need to unsubscribe from this topic (the button is at the bottom of the page on github) or from the project (the button is on the right on the project's main page).

Ed

—
Sent from Mailbox

On Tue, Jul 28, 2015 at 8:00 AM, Страхиња Радић <notifications@github.com>
wrote:

> @EdTorbett Again, I don't have anything to do with this project. So please don't quote me using at-tags.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/bbaumer/ansible-modules-core/commit/98845db47867c742a4c2b98b0acfe798776a0d2f#commitcomment-12389158


I think it's OK, but make it clear (e.g. with a comment, or perhaps in the documentation) that this limitation is known. You don't want to waste reviewer's time with people telling you that you don't reset the previous color.

That said, saving the last used color should not be so hard: whenever you encounter %(color:XXX), save XXX in a field of ref_formatting_state (initialize this field with GIT_COLOR_RESET, and don't reset it after displaying an atom). Then, use this field when you want to switch back to "the current color". Perhaps it's easy enough to be worth the trouble.


Good catch! It doesnt fail as it still matches the error message, as we just appended more text to the first draft.. I'll commit a fix shortly :smile:


thanks


commit


This is the block that commits chips from whatever text is entered when there are no suggestions. Since suggestions contain both existing users and possible addresses from the user's organization, the only time we won't display suggestions is when the user has started entering an email address from a domain that doesn't match the org. In that case, we should do nothing.


Not at this layer of the API. This essentially maps to xep45 example 6, and this does not contain the number of occupants. If you want to display that information you need to query every single room after retrieving the list. Also note that I believe that not all rooms allow you to get the number.

In the future, please use the forums for such questions. Thank you.


@flavour How do I provide permission to 'Subscription to email/sms/rss'?


This commit should not have changed anything for standalone
applications. It only exposes the bridge for usage in python plugins.

On 07/28/2015 08:40 AM, Alessandro Pasotti wrote:
>
> @m-kuhn <https://github.com/m-kuhn> can you please check that the
> manual example using the bridge is still valid?
> https://github.com/qgis/QGIS-Documentation/blob/master/source/docs/pyqgis_developer_cookbook/loadproject.rst#id21
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qgis/QGIS/commit/70fe622661852db61f66f8b973d1991259338af2#commitcomment-12388945>.
>


@EdTorbett @bbaumer Right. A simple edit of the commit message to include the correct author is out of the question then?


STOP HURTING MY EYES


Wydaje mi się, że lepiej użyć zmiennych bez `@` i przekazać id jako parametry do metody try_swap

Jeśli chodzi o obiekty firstPhoto i secondPhoto, zamiast pobierać je jeszcze raz można:

`firstPhoto.reload`


再过两天正式的CG就画出来了(X，先让金先辈辛苦下233


Related to this: Its also recommend to choose a meaningful name for conditional methods. The implementation shouldn't differ from its meaning. The method **should** return **true** per default to avoid confusion:

``` javascript
// bad
hasSomeState(x) {
    if (x.someState === false) {
        return true;
    }
    return false;
}

// good
hasSomeState(x) {
    return x.someState === true;
}
```


I am the only one to maintain this bsp-meta layer, I have time to support the lastest release but not all other branches... Please also note that "master" is the development branch, which means that it is the next yocto/oe release. As I never tested with this development branch, I cannot say that it is supported, also these branches can quickly break things or introduce API breaks...

- How is someone to know, a year from now, that master if your layer really means fido in others?
Because this person will read the README ?


Тогда размеры заголовков получаются такие же, как основной текст :(


Checker -- это интерфейс, т.е. мы должны создать или использовать тип, который будет реализовывать его. В нашем случае то, что возвращает Next, будет реализовывать метод Check() и соответственно сам интерфейс.  Помимо этого, то, что реализует Checker, должно возвращать две вещи: []*Problem, error. Сейчас у тебя возвращается только одна.


Из методов модели тестируется всего лишь destroy. Больше выполнено интеграционных тестов контроллера. Потому правильно будет убрать тест удаления сущности и переименовать класс в BooksControllerTest


Nie wiem, czy wykorzysujesz `seeds.rb` do testów, jeśli tak to lepiej wykorzystać fixtures albo tworzyć przykładowe dane za pomocą `let`:

```ruby
let(:photo_1) { FactoryGirl.create(:photo, :image => costam) } # Leniwe wykonanie
let!(:photo_2) { ... } # Wykona sie od razu
it "should work"
  photo_1
  photo_2

end
```


... и ты пытаешься запихнуть две возвращаемые величины в одну переменную :)


good! ++@yongbin


So that's why I should fix this bug, because when we wanna access the NodeWrapper from runtimes/runtime-cocos2d-js, we can't normally get the extended members from the intellisense, let me fix it, thank you guys @jareguo @jwu


I think we don't need the extra space before ``C_``


Mhhh good point, so I think that README.emmc and README would be enough (the first one for eMMC and the second one, like the existing one, for petitboot). What do you think ? Please also note that petitboot is not only for uSD but for other boot media (usb key, uSD, network...).

Could you rename README.firefly-eMMC to README.eMMC and make it more generic ? (it should work for almost all rk3288-based boards I think). You could of course, add a section for the firefly board in this file of course.

What do you think ?


I think you should use SPRT[-3, 1] for this patch, I don't see any reason to use SPRT[0, 4]


Ань, а ты можешь эти комменты убрать из паттерна? По git blame будет прекрасно вижно, кто что нагадил, а код они захламляют


Thought we were using scrollback? scrollback.io/vitacademics


Если делаешь к нему геттер, то объявляй его как
```
class Cockroach(private val _gene: Gene) {
```


We were? I thought that was for the Android app?


В тестах не следует использовать try ... catch. Правильнее ожидать появление какого-либо [исключесния](https://phpunit.de/manual/current/en/appendixes.annotations.html#appendixes.annotations.expectedException)
В данном случае нарушена логика. Т.к. сессия не должна содержать ошибку


Just:
if ($clickable)


new line


FYI this one works too. Took it from TS I think.
Blackbird88/OpenRA@8f3fc54


Nice...I see you got right of the `transform()` function.


Indent this properly.


watch out for except all, bad practice most of the times.


Nice...we might want to update the docs later to say `@return A logical value indicating if all columns are boolean`, to be in line with R's way of saying "True" or "False" (i.e. logical). It's an internal function anyway, doesn't matter too much, but I could see this being used as a helper function to other functions later.


Interesting implementation - I like it!


sorry... I may seem too tired, but what is it you want to do with this? what does "get relation data on many to many relationship" means?


You can use the `#{version}` here. Furthermore, the HTTPS is also available (`url "https://www.screamingfrog.co.uk/products/seo-spider/ScreamingFrogSEOSpider-#{version}.dmg"`)


What happens if you don't check `if (a_value)`? I'm concerned that it's not equivalent to the PP logic since it pushes everything in `@$val`.


The `check_boolean()` and its use are very elegant, I must say!


Булы - плохо. многобайт на биты информации. Лучше инты, там в случае чего их масками можно будет пофильтровать.


get : function(){
    TODOSync.get(function(json){
      console.log(json);
      var contextArray = [];
      for(var i=0; i<json.length; i++){
        context = { key : json[i].id, content : json[i].todo};
        contextArray.push(context);
      }
      $("#todo-list").append(this.build(contextArray));
    }.bind(this))
  }
해서 contextArray에 각각의 context를 넣고, 한 번에 template에 넣어서 build하는 방식으로 변경했습니다


Возвращаемый тип в этом случае можно не указывать


То же самое


Ok. I'll will register on PyPi once I've figured out if there are any
licensing issues.

Lars
On 28 Jul 2015 08:50, "kraigher" <notifications@github.com> wrote:

> I changed it to vunit_hdl
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LarsAsplund/vunit/commit/6c64f9cf02f07f8c6a271ba770118eb3e95c35d8#commitcomment-12389043>
> .
>


ты вызываешь его от val, поэтому тут лучше тоже явно указать, что это val


@Montellese
Thanks, thats working. We should document that in the wiki.


As I said, that's the official way to use the firefly board, so feel free to do so. Propose these changes directly into firefly.conf


Or if there are other issues... I'm not that familiar with PyPi.

Lars
On 28 Jul 2015 09:25, "Lars Asplund" <lars.anders.asplund@gmail.com> wrote:

> Ok. I'll will register on PyPi once I've figured out if there are any
> licensing issues.
>
> Lars
> On 28 Jul 2015 08:50, "kraigher" <notifications@github.com> wrote:
>
>> I changed it to vunit_hdl
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/LarsAsplund/vunit/commit/6c64f9cf02f07f8c6a271ba770118eb3e95c35d8#commitcomment-12389043>
>> .
>>
>


One import thing, I have recently changed how machine files are named. Could you rebase your work on master please ? firefly.conf is renamed to rk3288-firefly.conf for example


I'm not sure. This change is applied 1 year ago and there is no report about cronolog.


Please clean up.


@catterbu: I don't know how this function that accepts boolean can contain `validate_num_data()` function, which says:

```R
#' @title Validate that the arg is either a numeric data.frame or matrix
#' @description An internal function that raises an error if the argument is not
#' either a \emph{data.frame} or \emph{matrix}. Also, If all columns are not
#' numeric, it will raise an error.
#' @param d A \emph{data.frame} or \emph{matrix} whose class the function will
#' confirm.
#' @keywords internal
validate_num_data <- function(d) {
  classes <- c("data.frame", "matrix")
  types <- c("numeric", "integer")
  if (!(class(d) %in% classes)) {
    stop("The argument 'd' is not a data.frame or matrix.", call.=FALSE)
  }
  lapply(d,
         function(x) {
           if (!(class(x) %in% types)) {
             stop("The classes of the columns of 'd' are not all numeric.",
                  call.=FALSE)
           } else TRUE
         })
  lapply(d,
         function(x) {
           if (identical(unique(x), c(0, 1))) {
             warning("At least one column of 'd' contains only values 0 and 1.",
                     call.=FALSE)
           }
         })
```

I'm looking at the second `lapply`, which is supposed to stop the function if boolean data exists.

That function `validate_num_data()` is supposed to throw an error if the data is boolean...so not sure how you'd get to the part in `clust_boxplot()` where boolean data is plotted. Either your Travis test is about to fail, or you need to

1. Update your unit tests for both `validate_num_data()` and `clust_boxplot()`...this kind of discrepancy should be detected by your unit tests. In effect, now that you added the ability to plot boolean data, you should have created a unit test that plots boolean data. Given that your `validate_num_data()` is supposed to error if you try to do that, your unit test should fail, leading you to modify `validate_num_data()` to allow `clust_boxplot()` to accept boolean data. That was wordy...hopefully it made sense - just try to be helpful/useful.
2. Fix `validate_num_data()`


When i tried to get post categories, result like:

````
array(
    ...
    'categories' => array(
						array('post_id' => 1, 'category_id' => 3),
						array('post_id' => 1, 'category_id' => 7),
						array('post_id' => 1, 'category_id' => 9)
					),
    ....
)
````

With my `get_relate` option, result like with category data:

````
array(
    ...
    'categories' => array(
						array('id' => 3, 'name' => 'Category 3 name', ...),
						array('id' => 7, 'name' => 'Category 7 name', ...),
						array('id' => 9, 'name' => 'Category 9 name', ...)
					),
    ....
)
````


If you're talking about those painful "whitespace changes" commits, AFAIK it's MFF himself who requested them. So don't blame Mg for that.

The way you changed the structure of MFF to protect your things is wrong. What's next, Crabman will make his fishing mod a submodule to protect if from changes from another contributor ?  That's not fitting into a team.

If you want to be the sole developer of quests, then take it back as a generic minetest mod to **your** github home and let MFF either fork it or make it become a git submodule in his. This way you move the pain to cope with your changes from you to them without making the minetest-minetestforfun-server structure look weird on MFF's github home.


hi @kullii,

Thanks to follow this issue: http://forge.prestashop.com/browse/PSCSX-6390

Best regards,


new line


Made a PR that de-conditionalizes the dependencies and simplifies the formula: #42194. (Primarily to simplify it, but also because I'm curious about the IDN support and I'd only use the pidgin GUI.)


Use localization and do not hard-code german language here


@LarsAsplund we should not submit intill the TODO in setup.py is adressed. Polluting the parent folder of the package is not nice.


This change broke our CI system on jenkins. We are using it slightly differently to normal use case. We have P.R.s coming from different forks, that is the target and source branches are in different repositories. We use this plugin to trigger the build and set the branch build specification to */${targetBranch}. Then the first line of the build script we run a custom little script that actually fetches the remote repo and does the merge proper before building.

With this change, it tries to do a 'git refspec' on the source commit hash. But this does not exist in the target repository, so the git command fails and the plugin exits out before the build script even has a chance to do the merge proper.

I checked out 1.4.5 and built that with maven and deployed it to Jenkins, and it fixed our issue. But just wanted to give a heads up that some people are using the plugin in interesting ways where the source commit ID may not be present in the repo.


Setting the index to 0 unconditionally looks wrong. Should be first enabled room type like the code previously did, no?


reformat


Nakagawa-san

This is Kim.

Thank you for your kind explanation.

>I'm not sure. This change is applied 1 year ago and there is no report about cronolog.
We are not excluding our usage/environmental error.
And we will definitely report you all on finding any cause of the duplication.

Best Regards,
Kim.

From: Masahiro Nakagawa [mailto:notifications@github.com]
Sent: Tuesday, July 28, 2015 4:29 PM
To: fluent/fluentd
Cc: Kim, Jinyong | Atssa | MPD
Subject: Re: [fluentd] Change rotation handling with start/stop overwrap for high throughput (f64c2f4)

I'm not sure. This change is applied 1 year ago and there is no report about cronolog.

—
Reply to this email directly or view it on GitHub<https://github.com/fluent/fluentd/commit/f64c2f4c4df296acce0ddf4ef59e51506a31b36d#commitcomment-12389509>.


@Cadovvl с типом понятнее


add types


lgtm


¡Hola! AngelTDL
thanks for updating the Spanish files.
One question: Is it intended that you omit the additional letters like "..." and "( * . * )"


Agree. I will also be off computer for a few days so it will take some time
anyway.

Lars
On 28 Jul 2015 09:34, "kraigher" <notifications@github.com> wrote:

> @LarsAsplund <https://github.com/LarsAsplund> we should not submit intill
> the TODO in setup.py is adressed. Polluting the parent folder of the
> package is not nice.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LarsAsplund/vunit/commit/6c64f9cf02f07f8c6a271ba770118eb3e95c35d8#commitcomment-12389585>
> .
>