Great progress. I laid out some minor adjustments in the comments above, and then we should be good for the functionality, just need the tests.


@brianbmcintosh DWMR. Nice work here. You are a code :hammer:


I tried to implement it, but I do think that:
* `WorldBounds` is related to the `World` - so it should stay within `world.go`
* `Cam.Setup()` should be called after `WorldBounds` is set - but especially after the user sets their `WorldBounds` for the first time (within `Game.Setup()`).

When setting the Camera on `Game.Open()`, as I've tried to do, will not create the nice 'default zoom location' we want - because `WorldBounds` isn't set yet.

I propose to leave (at least) `Cam.Setup()` here.


oh, yeah, `setExpression` isn't actually being executed at this point. thanks :)


I agree... Atleast 40+.4 is the best we can aim for... I hope maintainers will suggest something realistic.


Since we are using BTLE this is more compact than Protobuf. It's what Jamel & crew are using for other projects as well.


@alex-ab there is a WiP NIC driver for hw_zynq (#1667). Shouldn't we merge the work rather than disabling the test? What's your opinion @m-stein?


This needs to be moved to the right class.


ah this commit is only there so the next one (redness fix) gets applied cleanly.


seems weird that it's not documented – maybe it should be :/ i personally don't like this style of passing transforms/plugins, but this is one of the things people really like about webpack.


I prefer do not move these lines, because in some other changes (for KT-9299), it will be moved again here and changed.


This is a line added only in this branch?


Fixed in 3968f084a6fe26405fb33e2da2e3859cb191f710. p.Kill() is brutal (-9) and could be changed to p.Signal(SIGTERM)


@cehallen DWMR. It works :camera:


EFLAG_REPEATABLE  not working - EAI Dev Filter Log not display, because not have not the slightest mention of.
It does not work on any core, so that your words I can easily refute.


This was the latest working version I think...


Okay, I'll revisit this.  I think maybe just a link to https://github.com/dimagi/commcare-hq/blob/master/corehq/apps/userreports/README.md#aggregation will be enough.  When I was going through this, I thought of an aggregation column as a more advanced feature - like, "leave it alone unless you know you need it", so I was really confused when my report came out with just one row.


I'd extract `getInstance` to variable


Just curious, have you ever use this utility in real project?


I'd extract getInstance to variable


Okay, we'll go to step. This is going to be annoying :-)


Somehow, emitting an event just after destroying a tab containing a layout, was making the splitbar of other layouts contained in in other tabs, sticky. This fixed my issue. I have no idea how though.
Thank you.


This function probably should be merged into the setter for health.

The setter already checks to see if the health is < 0 whenever it is set (and kills the actor if appropriate), and on set we could easily update the text on the healthDisplay. This way we don't need all the checkHealth, takeDamage, etc functions (unless we need the functions for bonding with Unity, then we can deal).


cloud :3


see [configuring logging for a library](https://docs.python.org/2/howto/logging.html#library-config).

`logger = logging.getLogger(__name__)` at module level with default `NullHandler` is better suited for libraries. I don't really like adding extra arguments for a non-standard way to configure logging.


@chelmuth: Didn't noticed this. If #1667 is ready, of course we should rather merge the commits instead of disabling it. Let's see what @m-stein is saying.


hi darwin


Hi, this change has broken retries upon Glacier's Complete Multipart Upload request!
Indeed, for some debatable reasons, Amazon decided to implement this request with POST, though stating it is an idempotent operation...: http://docs.aws.amazon.com/amazonglacier/latest/dev/api-multipart-complete-upload.html
Worse, the new version of the invoke(HttpCommand) method will have a different behavior when the server returns an error (will repeat) and the request times out (won't repeat).


This check is superfluous. Whenever health is set, an identical set of code runs in the setter.


Is this the programmer's fault?


Option confusion - removed map


@lyndonbartlett88 This works and is perhaps the very best, one could also do this `[self.webView goBack]`


why hello


@lyndonbartlett88 DWMR. Ship code :truck:


I would prefers functions to be verbs actually.  What about `fit_GARCH` instead?  Any recommendation on conventions?

Going ahead, it may be good to build towards a `fit_model` function that calls the corresponding methods for different models.


Yep, this is correct. :+1:


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


Ah, that's why it wasn't loaded on travis. I was using an alias.


I FUCKING DID IT MOM


I really want this to stay, looks ugly


I'll do it.


@lyndonbartlett88 DWMR. Checkpoint and assignment 35 complete. Press ahead. :+1:


Wonderful!


文字化けしてしまう。。。


what about `instanceof`?


Someone else should see this: @keenanlang


hi


great work! feel free to submit PR!


You can just do `!!pli` instead here. You could even change `pli` to something like `is_chroma`, and the caller could just call the function with `od_codedquantizer_to_quantizer(enc->coded_quantizer[pli], !!pli)` but that's not particularly important.


Why do I not see this change reflected in the current file?

https://raw.githubusercontent.com/gramps-project/gramps/maintenance/gramps42/po/sk.po


some additional checking of the content would be nice.
* Is the escaping correct?
* Does (utf8) encoding work correctly?


> For instance: mix a grand-ma in a wheelchair will be turned into grand-ma in wheelchair

@ozdemirburak you're correct, we need to remove the `/g`.

Right now, without the `\b` after the `(a)` this is going to remove the leading `a` for any drink name that starts with an `a`, e.g. "absinthe".

`how to mix absinthe` -> `bsinthe`

If you add a test for a drink that starts with an `a` it should be failing, because the remainder of the name doesn't exist in the list of drinks.


sorry for late reply, been on vacation. Your work around is exactly what I was thinking myself. I'm not a java dev myself, but can do some things. What I dont understand, is why isnt jenkins reading the promotion configs when using the folders plugin or is it job of the promotions plugin? any luck with 1.633 ?


solo se non è gia un array !


``` QUOTE
sound entries are currently missing
```
Blizzard Do not voiced these bosses.


Fixes #529


Thanks! This seems to have fixed it. :)


I am new to Github, so I'm unsure where to post my files, hopefully this is an acceptable location.

[PrototypeComponents(unassembled).pdf](https://github.com/metanomy/trash_grabber/files/20955/PrototypeComponents.unassembled.pdf)

I am also unable to post STL files in this location.


Is it not necessary to use a dollar to call a variable?
- Ty, Dennis


Exactly Ezee! I'm not 100% happy with this code. When I start Jamtaba I'm experimenting with the idea of an audio graph. So, I create the class AudioNode, and AudioNodes are connectable. Many modern audio architecture are programmed in this way, and I really love this idea. In the future (is not a priority) I will change this code to work like a audio graph 100%. At moment somethings are a graph, other not. Using a graph we have many nodes connected, like:
LocalInputTrackNode -> VstPluginNode[Kontakt] -> VstPluginKontakt[Addictive Drums] -> PanNode -> GainNode ->  AudioOutputNode

This is idea is beautifull, flexible, easy to create routes, etc. But to use this scheme in Jamtaba we neet re-write many things, and this can introduce a lot of new bugs. So hard to change, hard to test, etc. But someday, someday .... :)


20 tests passing so far! Now time for break; and thankful to regex for once lol.


szerintem ez nem jo igy, de inkabb igy jo...


TeamCity DNN JavaScript Libraries :: CI [Build 106](http://teamcity.engagesoftware.com/viewLog.html?buildId=384&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 106](http://teamcity.engagesoftware.com/viewLog.html?buildId=384&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


What is the point of this immortal skin spell? It looks like another version of protection field?


@davidwilson2038 — sure, that makes sense. Would you like to see that change as part of this PR, or as a separate PR?


Proposed search engines:

sk https://seeks.ru/search?q=_dwb_search_submit_
sr https://searx.me/?q=_dwb_search_submit_

https://beniz.github.io/seeks/
https://github.com/asciimoo/searx


Not sure if the String check really belongs here. IMHO the code which calls can_modify_package? should be aware that it is a remote object. We may overlook other problems avoid the following exception.

But I do not have too large concerns if this is really needed. Would be interessting if it is just the logfile view so far ...


@asr back-ported this commit into the 2.4.2.4 branch in a3a78c79bcd21c381def8696ae3532c0f3e6711d.


Hi Philippe,

When read_all_run_pars is called on line 73 of pc_collect, it will try
to read *_init_pars in run.in instead of *_run_pars, if lstart is
.true., which is set on line 31 of pc_collect.

The problem came from the unfortunate scheme in subroutine read_namelist
in module sub.  In lines 6998-7002 in sub, the type 'init' will always
be selected in read_namelist if lstart = .true., even though it is
read_all_run_pars that calls it.

But by setting lstart = .false. and lrun = .true, read_all_init_pars on
line 69 of pc_collect will read param.nml for *_init_pars, while
read_all_run_pars on line 73 will read run.in for *_run_pars.  I believe
this is the correct and intended behavior for post-processing.

Sincerely,
Chao-Chin

On 10/26/2015 04:15 PM, Philippe Bourdin wrote:
>
> Hello Chao-Chin,
>
> In this case, "read_all_init_pars" is supposed to read the
> 'init_pars', not the 'run_pars'.
> Otherwise, I'm not sure if the 'init_pars' are read at all, if "lrun =
> .true." and "lstart = .false.".
> What exactly was the problem you had?
>
> Thanks and best greetings,
> Philippe.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pencil-code/pencil-code/commit/28eece2bd5c703c3a109bcda6f7faec2f9be6fb5#commitcomment-13986943>.
>

--
Chao-Chin Yang
Postdoctoral Fellow
Department of Astronomy and Theoretical Physics
Lund University
Sweden


Упростил немного установку цвета кнопок. Теперь вместо гаданий что поставить `fill` или `stroke` достаточно просто поставить `color: <цвет>;`. Правда для установки фона теперь недоступен `currentColor`, но невелика потеря.


trailing space


I've been using ruby version 2.2.3, so this change forcing ruby 2.2.2 broke my environment :-(.

But I understand why you might want a fixed environment, so I did this locally (I'm documenting this in case someone else needs the info):
rbenv install 2.2.2  # I'm using rbenv to manage Ruby versions
rbenv global 2.2.2
gem sources --add https://rubygems.org
gem install bundler
gem install rails
bundle install
rake db:migrate

Then this works again:
bin/rails s

I'm writing this here just to document it...


testing out scripts on new home


splitsville


FYI, this breaks the following path handler. The path ("tsv") looks like a sample type. The solution to this kind of url collision is to put the known paths before the variable paths. This also means we can't have an entity with the name of "tsv"


Maybe you mixed it with Project class? projects can be easily a String when using get_by_name. Package.get_by_project_and_name is only delivering a nil object if there is a chance that the package exists on the backend (and therefore remote) only. A single string without telling in which project the package is would not help much, no?


Looks great!


this is a change


Hi Johan,

can you tell when there will be a release including this fix? I tried cloning and building a snapshot of the current master branch 9d6d89 via mvn package, but the build fails due to tests in json-path.

json_path_supports_deserializing_input_stream_using_with_given_charset(com.jayway.restassured.path.json.JsonPathCharsetTest)  Time elapsed: 0.083 sec
 <<< FAILURE!
java.lang.AssertionError:
Expected: "Ç%#Õ÷"
     but: was "Ôé¼%#├Ñ├Â"
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
        at org.junit.Assert.assertThat(Assert.java:956)
        at org.junit.Assert.assertThat(Assert.java:923)
        at com.jayway.restassured.path.json.JsonPathCharsetTest.json_path_supports_deserializing_input_stream_using_with_given_charset(JsonPathCharset
Test.java:50)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:497)
        at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
        at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
        at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
        at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
        at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
        at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
        at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
        at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
        at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
        at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
        at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
        at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
        at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
        at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:252)
        at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:141)
        at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:112)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:497)
        at org.apache.maven.surefire.util.ReflectionUtils.invokeMethodWithArray(ReflectionUtils.java:189)
        at org.apache.maven.surefire.booter.ProviderFactory$ProviderProxy.invoke(ProviderFactory.java:165)
        at org.apache.maven.surefire.booter.ProviderFactory.invokeProvider(ProviderFactory.java:85)
        at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:115)
        at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:75)


This version was a pure mistake. Accidently, the entire code been overwritten by a different class. Should have to be undone. Unfortunately I couldn't find out how to eliminate this commit


This looks much better! `tostring` is redundant now, perhaps?


suggest?


This is a markdown flie


the side note in the spec may not respected here
1 The “addpids” or “delpids” parameters shall not be used in combination with the “pids” parameter in the same RTSP query


Yes, I had the same thought, that adding "--version" to all the executables we produce would make this even better. I just figured I would wait to revisit that in the future in order to move on to some higher priority things now.
--Matt

----- Original Message -----
From: "Andrew Melo" <notifications@github.com>
To: "accre/lstore-lio" <lstore-lio@noreply.github.com>
Cc: "Matthew Heller" <matthew.f.heller@accre.vanderbilt.edu>
Sent: Monday, October 26, 2015 10:30:55 AM
Subject: Re: [lstore-lio] Simplify getting the version of lib & executables (71bb96e)

Do we perhaps want to build on this PR and expose a "--version" flag to the lio command line options? Perhaps have it dump out the version of all of the projects it built against? Just a thought.

—
Reply to this email directly or view it on GitHub .


Hi @johanhaleby ,

can you tell when there will be a release including this fix? I tried cloning and building a snapshot of the current master branch 9d6d89 via mvn package, but the build fails due to tests in json-path.

```
json_path_supports_deserializing_input_stream_using_with_given_charset(com.jayway.restassured.path.json.JsonPathCharsetTest)  Time elapsed: 0.083 sec
 <<< FAILURE!
java.lang.AssertionError:
Expected: "Ç%#Õ÷"
     but: was "Ôé¼%#├Ñ├Â"
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
        at org.junit.Assert.assertThat(Assert.java:956)
        at org.junit.Assert.assertThat(Assert.java:923)
        at com.jayway.restassured.path.json.JsonPathCharsetTest.json_path_supports_deserializing_input_stream_using_with_given_charset(JsonPathCharset
Test.java:50)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:497)
        at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
        at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
        at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
        at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
        at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
        at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
        at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
        at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
        at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
        at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
        at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
        at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
        at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
        at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:252)
        at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:141)
        at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:112)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:497)
        at org.apache.maven.surefire.util.ReflectionUtils.invokeMethodWithArray(ReflectionUtils.java:189)
        at org.apache.maven.surefire.booter.ProviderFactory$ProviderProxy.invoke(ProviderFactory.java:165)
        at org.apache.maven.surefire.booter.ProviderFactory.invokeProvider(ProviderFactory.java:85)
        at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:115)
        at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:75)
```


@davidwilson2038 — I think we should rename `googleapis` in the driver to `google`. Would you like that change as part of this PR, or as a separate PR?


Number of fixations from "F" and "Oth" is not equal, therefore we cannot access indexes uniquely using "NFixations".


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


и тут


The problem with IOExceptions is that usually are thrown when the connection has been closed or there is some issue in the http channel. In most situations you can't know if a retry is correct or not, so the idempotent check makes perfect sense. WDYT? Mind opening a JIRA issue to track this?

We could move the idempotent logic to the default `ioRetryHandler` implementation, and let the Glacier API configure a custom one that did not take into account the method.


Hi @JuergenSimon. Normally I'd ask how I could easily see the differences between the original and your changes, but solution 2 (the more interesting one) looks like a substantial rewrite, so I'm not sure such a question would make sense. I do have some others though.

1. What happens here?

    ```js
var rsrc = NAMESPACE.resource('foo'):
rsrc.close();
// `rsrc.isClosed()` now returns `true`, but what is the state of the underlying resource at this point?
```

1. What would happen in the following scenario?

    ```js
var rsrc1_1 = NAMESPACE.resource('ref1');
var rsrc1_2 = NAMESPACE.resource('ref1');
rsrc1_1.close();
var rsrc1_3 = NAMESPACE.resource('ref1');
// What is the state of the world here? What *should* it be?
```

1. Are your tests easily automated?

Thanks, and we look forward to your responses!


> Note to self: change the first commit message when rebasing

Tip: a `squash!` prefix (`git commit --squash <commit-ish>`) will prompt for a revised message automatically!


ок


worrisome, but good to know


I set the package name to a string in the controller if ``Package.get_by_project_and_name`` returns nil. In the view we call ``  User.current.can_modify_package? @package `` in src/api/app/views/webui/package/_live_log_controls.html.erb which then fails. I could also check this in the view!


The characters are written in a UNICODE file where the executable is launched.


I can't tell exactly. There are some things that I want to include and fix in the next release that I haven't been able to fix yet. To build REST Assured manually you need UTF-8. But you can just use snapshot pre-built snapshot instead. Depend on version 2.6.1-SNAPSHOT after having added the following maven repo:

```xml
<repositories>
        <repository>
            <id>sonatype</id>
            <url>https://oss.sonatype.org/content/repositories/snapshots/</url>
            <snapshots />
        </repository>
</repositories>
```


Oh! Okay! No problem then I’ll just hook my model to the default db. Thank you, that would have been an issue lol =)

Sent from Windows Mail

From: Ernest Mombay
Sent: Monday, October 26, 2015 10:58 AM
To: OzzieSoft/SrPrjGrp2
Cc: kyle chaput

A connection string setting in web.config is for one DB. You have another one, so it's a new DB

—
Reply to this email directly or view it on GitHub.


Works for me. I want to take a stab at making the command line handling /
ini parsing less segfaulty. That can probably get rolled in.

--
It's dark in this basement.


@GreenLunar could you stop spamming everything with those projects?


use consts instead of literal numbers


@masterkk no worries, glad it works for you.


... `typeof x === 'ThenSuccess'`?!


Neat, thanks for the tip!


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


This should be `const gchar *` since it introduces a warning:
```
../pkg.c: In function ‘scan_dir’:
../pkg.c:175:18: warning: assignment discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
   while ((d_name = g_dir_read_name(dir)))
                  ^
```


that manager... up to no good.

Could you add an if-null check to unregister(tag) - even if HashMaps support null keys. The ones we'll use it the future probably won't (when we make the move to int-centric maps).


use `if...else`.


This version of the code is inefficient, at least in my eyes. I want to revisit it and redo it in a more efficient manner. Perhaps in a for loop?


I found your examlpe http://jsfiddle.net/zachpainter77/z8us2aej/ . I changed it so I could display paging and it worked until there were two pages and I added a new person via the dialog box. That's when the paging stopped working. I also noticed that there can never be more then 2 pages created.


Hi niksmac! Actually, my javascript knowledge is very limited. But I try assign, for example src="https://www.github.avatars/2.jpg" rather than src="../images/4.jpg", and nude.js doesn't work. Can you explain me how can I make remote url detection possible? You may give me some references if possible, also. This is a big issue for me to solve. I want to learn from you something, if you can help. Thanks in advance!..


@magelle plz add the if check for `enableTranslation` in the service js resolve part as it will cause error when i18n is disabled during generation


Yes I agree that is a need and part of the reason that I figured adding "--version" would not be a quick task.


Consider adding a new status here so we can display updated progress on the client.


"don't generally" isn't necessarily a good policy (#12832). all of the functions in this file (except the one you just added) do full validation of their arguments. the only reason the functions in this file should be called from codegen is to trigger the runtime check if the validity couldn't be statically determined.


If that does change, any idea what the failure mode would be? I don't really love another option flag, either, but I think it's worth doing that if this would be a difficult bug to notice and track down.

Thinking about it some more, this is probably fine as is...I'm guessing it'd either throw an error or wouldn't create bubbles properly, which is easy to notice and not a destructive bug.


@duff I wonder if this is a contributing factor to this: https://secure.helpscout.net/conversation/119445050/7572/


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


thanks for the information

On Monday, October 26, 2015, Assela Pathirana <notifications@github.com>
wrote:

> Hi,
>
> Thanks. Emitter: not planned at the moment, but lets see what we can do in
> the future.
>
> Assela
>
>
> Assela Pathirana,
> UNESCO-IHE Institute for Water Education
> T: +31(0)15 215 1854
> Westvest 7, 2611 AX Delft, The Netherlands
>
> On 22 October 2015 at 04:33, jermaine1ronquillo <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> wrote:
>
> > I'm looking forward to the development of this project sir. By the way,
> is
> > there any chance that epanet emitter will also have an API in python?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/asselapathirana/epanettools/commit/929c94a1cf7e558c522d1c996305d500fd6bdadf#commitcomment-13916832
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/asselapathirana/epanettools/commit/929c94a1cf7e558c522d1c996305d500fd6bdadf#commitcomment-13985279>
> .
>


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 33](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=364&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


La verdad es que ni lo había pensado, pero tienes toda la razón del mundo, en cuento pueda me pongo a ello, muchas gracias;)


I don't think you need to write so much about it. Just state that ExbiByte is maximum available with int64


are dreptate doamna teodora,la mine fara paranteze nu functioneaza😉

În data de 26 octombrie 2015, 14:26, FavianIoel <notifications@github.com>
a scris:

> Salut Teo , ai incercat sa vezi daca iti ruleaza codul ? Nu sunt sigur dar
> cred ca la comanda "print" tot timpul trebe sa pui stringul in paranteze :
> print("teodora");
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lmihai/fasttrackit_php2/commit/f07f7ebec7118d16447e19c629d95237f9f07bce#commitcomment-13982467>
> .
>


It depends on where you want to check if the package does really exist.  Because nil just says that there are remote projects. It does not say that the package exists there.

The only way to find out is asking the backend, which is relative expensive compared to a database check, since it is a HTTP call.
Question is what kind of error do you get when the package does not exist? Is it still understandable for the user?
One needs to make a difference between the source package exists not at all and no build result has been created yet IMHO.


This appears as a better solution ...
build(Ask, {oframe, Tabs, Def, Flags}, Parent, WinSizer, In0)
  when Ask =/= false ->
    DY  =  wxSystemSettings:getMetric ( ?wxSYS_SCREEN_Y )*5 div 6, %% don't take entire screen.
    1 =:= Def orelse error({default, 1}),
    buttons =:= proplists:get_value(style, Flags, buttons) orelse error(Flags),
    NB = wxNotebook:new(Parent, ?wxID_ANY),
    AddPage = fun({Title, Data}, In) ->
              Panel = wxScrolledWindow:new(NB,  [ ]),
              wxScrolledWindow:setScrollbars(Panel,-1,10,-1,10), % no horizontal
              wxWindow:setMaxSize(Panel, {-1, DY}),
		      case os:type() of
			  {win32,_} ->
			      BG = wxNotebook:getThemeBackgroundColour(NB),
			      wxPanel:setBackgroundColour(Panel, BG);
			  _ -> ignore
		      end,
		      Sizer  = wxBoxSizer:new(?wxVERTICAL),
		      Out = build(Ask, Data, Panel, Sizer, In),
              wxPanel:setSizerAndFit(Panel, Sizer),
		      wxNotebook:addPage(NB, Panel, Title),
		      Out
	      end,
    In = lists:foldl(AddPage, In0, Tabs),
    wxSizer:add(WinSizer, NB, [{proportion, 1},{flag, ?wxEXPAND}]),
    In;


De acuerdo, haré revert entonces.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 33](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=364&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:50


yes it does! i updated the changes


Since the bug happened when `mpi_grow()` was called, I think it would be good to add a test where a carry happens. For example, `mbedtls_mpi_add_mpi_inplace:16:"ffffffffffffffff":16:"01fffffffffffffffe"` should to it.


@daveismyname  you had said on previous comment
~~~
yes I was getting errors for failed logins using imap the logger was stopping me from being able to catch the exceptions in my script.
~~~
Could you instead use a custom try catch block? Also could you give some instructions of situations to use and how to use custom try catch blocks? Perhaps in a video.


This is the job done by shm, now i'm working on some issues with SMAP that cause panic's.

Thanks for review, i'll take into account your comments.


It's a binding on the inside of the loop (here the loop is driven by recursion). I wouldn't mind if the binding were inside the `(puthash ...)`, because then it's cached. But since it's on the outside, it'll add overhead to every single call, cached or not.


Sure, if you are not interested.

These projects are decentralized and privacy-driven, is this really bad?


merging


@spaceliang 这个我会找时间弄的~


Minor issue, but please align comments inside enum


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2380](https://teamcity.mesosphere.io/viewLog.html?buildId=64732&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:27


@st0012 cool! Great!

Probably it makes sense to optimize for the case with block, because it `inputs` is called mostly with block anyway.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2381](https://teamcity.mesosphere.io/viewLog.html?buildId=64733&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:02


I can't figure out how this differs.

Should the other composed `c[ad]+r` functions be rewritten as well?


@pksunkara @klokane it seems that in [RefractDataStructure:504](https://github.com/obihann/drafter/blob/0b1f2a644189d6a327f9c8a60431f1c12009e567/src/RefractDataStructure.cc#L504), while using a blueprint like this

```makdown
    + Attributes
        + users (array[User])
```

one of the ValueMember objects that is being passed contains the actual value ("pksunkara") and another one contains the reference to the type `User` which lets `InjectNestedTypeInfo` be called later on and the attributes be properly filled. However when we use the other mson

```markdown
    + Attributes (array[User])
```

we never get the association with `User`. Because of this the function `InjectNestedTypeInfo` is never called, and this is the problem.


OK ... seemed to need setScrollbars( )  call.
Your setMaxSize should have been on Panel ... not the NoteBook.
And yet ... need setSizeAndFit( )   ... not just setSizer.

With all this ... the  5 div 6  max size is not enforced unless the content demends it.  So that's better for sure. I'd do an amend after a bit more testing.


why did you remove absolute positioning?
what about plugins that now are using this metods?


I think this is slightly older pull. I had updated these fixes already.


it is better, when b is null, to fire userdata as empty object rather than null


Is this freed anywhere?


I'm sorry.


Now that you've eliminated one of the nested loops we don't need to do the second check for the command interrupt at https://github.com/jmaher/autophone/blob/9ee148950d3d4c0054a1c7da15e4dcdff0cd25d9/tests/talostest.py#L182

I think you could remove the

            if command and command['interrupt']:
                break
            elif not success:
                break

and move the break up to the end of the             if not success:
block.


opening an issue on this #22

I'm in favour of reverting this commit from 6.1.12 or at least doing some work to more clearly define what we do/don't support here. I think it would be much cleaner to simply drop all references to the Messaging system in one go if possible (or at least as much as possible). Either way I'll do some digging and see how bit the changes need to be to fix this.


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


тут объект обрисовки поля не нужно на самом деле в список добавлять, просто пока не придумал где это делать


Consider replacing try/catch with `error = record.get('error', '')`. We don't want to disregard chunk/mosaic messages in the absence of this field (although it *should* always exist).


Same comment as above.


ok


+
 | Nombre | Carnet |
 | :--- | ---: |
 | Jonnathan Luis Fernando Aldana Ruano | 0900-14-22577 |


Added vacusr.f file and vac...par files naming notation is as follows
src/vac_PPP_N_M_vGGg.f
par/vac_PPP_N_M_vGGg.par
or
par/vac_PPP_N_M_vGGg.par


@obihann it can be related to https://github.com/obihann/drafter/blob/0b1f2a644189d6a327f9c8a60431f1c12009e567/src/RefractDataStructure.cc#L832

Can you please check, if `ExpandRefract()` is invoked for second case
```
+ Attributes (array[User])
```


@justin808 I hadn't switched them yet, going to get that done today


Aha.


Personally I'd try to avoid mentioning JAVA_HOME in any set of instructions, it's a very outdated way of doing things and might actually cause conflicts. I'd think it's better to just say that you should check that your proper Java 7 installation is active by typing `java -version` and use your platform's appropriate way of changing the default if it's not.


Is there a change to the PR needed to get it accepted or do you just not
want sha256 right now?

On Sun, Oct 25, 2015, 10:51 AM Mike Thornton <notifications@github.com>
wrote:

> That's an old comment, but yes should be remove now since it's no longer
> true.
>
> have_sha256 checking is done early on. You won't have gotten this far if
> sha256 wasn't supported. Although it might make things a lot clearer if you
> did the following and just checked support_sha256 everywhere:
>
> local support_sha256 = have_hmac and have_sha256
>
> sha256 might be "experimental" but we use a similar patch to add it and
> it's worked great for us. Would be much better if upstream could support it
> if all the requirements are installed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dobesv/lua-resty-hawk/commit/b4c32c6b74558b14da769c43794d97d4bc373d6f#commitcomment-13972855>
> .
>


should bufereven passed to callback be recreated with BEV_OPT_CLOSE_ON_FREE flag? Or is it better to pass file descriptot to library user so one be able to decide how to work with it?


Use `ToscaType.MAP` instead of `"map"`.


Учитывая что стиль `color` наследуется, теперь нет смысла держать дополнительные классы для элементов `svg`. Теперь для указания цвета кнопки надо использовать класс самой кнопки.


And `ToscaType.STRING`.


I did this for the tutorial reasons..


Otherwise voiceover doesn't focus on the element and the tab cannot wrap-around


@zallek  I renamed it GoogleColumnChart, because GoogleChart it's already used by GoogleChart React Component.


Yes please! :+1: This has been bugging me as I have to create a new eclipse setup for each machine that I work on as eclipse setups are inherently not very portable. And then I have to remember not to commit them - and then have to unwind the setup each time I pull down some changes...

Can we also remove the .project, .pydevproject and .settings/ dir from the root of the project?


"a default name and version"


@justin808 exactly. I checked the docs because I couldn't imagine they wouldn't write it that way, sure enough that's how they wrote it!


hi

Give the urgency of this and your application can we meet from 4 to 5 in my office to discuss?

I like both 1 and 2.

-Francisco

On Oct 25, 2015, at 3:36 PM, Brian Cohn <notifications@github.com<mailto:notifications@github.com>> wrote:

@valerolab<https://github.com/valerolab> Which arguments do you recommend I make here:
1. a cost function can be evaluated across all solutions, to find patterns across different muscles
2. multiple cost functions can be simultaneously constrained, to test changing motor 'modes
3. anything else on this point?

—
Reply to this email directly or view it on GitHub<https://github.com/bcohn12/space/commit/f1748525e2b70631b73defd39b233f52da4c0453#commitcomment-13974036>.


All I can say is: Wow.


s/The detailed/Detailed/


sounds good

-Francisco

On Oct 25, 2015, at 3:12 PM, Brian Cohn <notifications@github.com<mailto:notifications@github.com>> wrote:

@valerolab<https://github.com/valerolab> should I add an and here?
"cost-agnostic and probabilistic"

—
Reply to this email directly or view it on GitHub<https://github.com/bcohn12/space/commit/138c5b5189bc7da3c6735a7b4109adf9a3701258#commitcomment-13973925>.


@justin808 Agreed! Thanks again for this tip!


I would focus on your NSF application today ;)

-Francisco

On Oct 25, 2015, at 3:12 PM, Brian Cohn <notifications@github.com<mailto:notifications@github.com>> wrote:

@valerolab<https://github.com/valerolab> should I add an and here?
"cost-agnostic and probabilistic"

—
Reply to this email directly or view it on GitHub<https://github.com/bcohn12/space/commit/138c5b5189bc7da3c6735a7b4109adf9a3701258#commitcomment-13973925>.


the comma-separated...


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


@justin808 Isn't that okay since it's a readme? It gets converted to HTML so I don't want to manually break it, right?


the in built?


@justin808 I already have both a link to the dev server and a link to hot module replacement on line 30. You want me to add it again?


Cool, thank you!


Thanks for your lightning-fast reply! ;-)
I totally agree with you: the idempotency-based change makes perfectly sense.
The actual problem is due to Glacier's approach using an non-idempotent HTTP verb to implement an idempotent operation...
Your proposition to move the idempotent logic to `ioRetryHandler` sounds good, I'll open a Jira issue.


@asr back-ported this commit into the 2.4.2.4 branch in 4e39e918ba880c29a3ea0320636bb4416545fab7.


wew


@asr back-ported this commit into the 2.4.2.4 branch in 35e795a430e6d22ac4b90c56b9b69a5ab584071d.


I would say that was a bug in `reinterpret`, which was not itself an intrinsic at the time. Traditionally, all uses of intrinsics had to be correct; they are not first-class functions and are not considered user-facing. The design is that checks should be done by dispatch or by julia code that prints a friendly error, and intrinsics are the low-level work protected by those checks.

For example, the code in intrinsics.cpp does very few checks, but it checks the number of arguments to an intrinsic and throws the error at compile time, since I saw this as more of an assertion failure. More checks are always good, so I think it's ok to have the checks in this file but they should maybe be assertions instead.


@klokane indeed it does, and in that function the `registry` looks to contain the type and value, however when we reach this block

```c++
        if (refract::IElement* expanded = expander.get()) {
            delete element;
            element = expanded;
        }
```

we never enter, and because of this return the same `element` that was passed in originally.


Since this is in `AC_CONFIG_FILES`, it's not necessary to add it to `EXTRA_DIST`.


@justin808 See the helper in `generator_spec` line 63. It's just boilerplate text I put into files when I'm simulating their prior existence to running the generator. It helps me ensure test for the difference between append, prepend, and overwriting the the file.


Line 16, typo, 應該是 add(Book $book) 不是 $books


Thanks, I'll test this out.


It does what you want, but doesn't reflect precisely the nature of the thing you're destructuring. You can't be certain there's one arg, before checking how many args there are. I realize this code works, but I don't think it's as clear as it could be.


Is it not GetBlockTime instead of GetMedianTimePast ?


@notspiff @wsnipex i really have no clue how this is even possible. For osx - building of spectrum works:

pulled from here:
https://github.com/afedchin/visualization.spectrum

build of waveform fails

pulled from here:

https://github.com/afedchin/visualization.waveform

The problem is that for waveform the glew dependency is not built. Thats why it can't find it and fails during final configuration step of the addon. But i don't see any difference between spectrum and waveform in the those 2 linked repos. The only difference is that in the repo-binary-addons for spectrum platforms.txt holds "all" and waveform has an explicit list of supported platforms. Any idea how this could be leading to not building the glew dependency?


This line is redundant - the rest looks good.


Is this branch ready to rock?


Best to put brackets around the ```return```


your banned


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


throttle function from https://remysharp.com/2010/07/21/throttling-function-calls


navigation aria label goes missing as it is rendered before accessibility is enabled and never re-rendered. this fixes the aria-label so that it is read out.


Attention, master est vraiment DEV et STABLE est 3.4 ... donc sorry pour avoir rate ce changement .. Je vais changer ca pour utiliser la meme syntaxe pour les deux. Je vais changer le format JSON encore avant release (qui n'est pas pour bientôt)


**You're


+1, except that bracket comment


Aqui es solo el comentario, no es aqui donde tenés que editarlo sino en el repositorio que descargaste. Acá sólamente son comentarios del código, podés hacer anotaciones y discusiones.


Instead of doing so, I suggest to check if (!result_set) after the loop calculating result.

Anyway, this condition can't happen, since at least a port must be connect otherwise we wouldn't receive a package.


I told you not to be a dick because you complain about such minor thing
that doesn't even matter while you have declaration of pointers all over
the place which are declared without being initialized to zero. If you
think that is good style, please shoot yourself.

On Mon, Oct 26, 2015 at 10:18 AM, Jo-Philipp Wich <notifications@github.com>
wrote:

> My understanding was that you want to get this stuff merged so I checked
> whether it fits the style of the other code. But forget about it, I won't
> bother you anymore.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mkschreder/juci-uhttpd/commit/75d7b42162ff4b603d880c9bd34cb80d39608702#commitcomment-13978780>
> .
>


El archivo completo tendría que quedar algo asi

| Nombre | Carnet |
| :--- | ---: |
| Jonnathan Luis Fernando Aldana Ruano | 0900-14-22577 |
| Mario Soto Ovalle | 900-15-16795 | $1600 |


Works for me!


What is the "hello" string?


btw why `qreal` instead of `qint64`?


@obihann so it is probably related to `ExpandVisitor`, but currently not able to say why.


@yurloc  It's to force the runtime exception. But I agree will be nice to have the stack that originate that.
Do u want to send a PR to fix that? Or I can also do that.

Thanks for the review!


Щикарно получается. Переведем в векторы даже небо, даже Аллаха!


@logbon72 - can you also add fix for README.md
it'd be nice to have people copy-paste the code from the project's github page and have it play automatically in browser
thanks again
neat patch


@anryshitko


No, `GetMedianTimePast` is preferred because it is not a free choice by the miner who included the transaction being spent. BIP 113 changes the other end of the interval for both nLockTime and nSequence to be GetMedianTimePast, which is a soft-fork change, but changing the start of the interval in that way would be a hard-fork change. So we start from the very beginning using `GetMedianTimePast`.


ok


I'm not sure I like this change. I prefer to have the getters be positive, and have to invert them when necessary. Make more sense that way IMO


```
[root@staging ansible]# ansible-playbook stage_resources.yml

PLAY [Prepare the staging resources] ******************************************

GATHERING FACTS ***************************************************************
ok: [localhost]

TASK: [Install required support packages] *************************************
ok: [localhost] => (item=createrepo,dhcp,docker,firefox,httpd,ipmitool,iptables-services,ntp,python-ipaddr,python-passlib,python-pip,python-xvfbwrapper,sshpass,unzip,vim,wget,xorg-x11-server-Xvfb,yum-utils)

TASK: [Install required pip-based support packages] ***************************
ok: [localhost] => (item=docker-py)
ok: [localhost] => (item=pip2pi)
ok: [localhost] => (item=selenium)

TASK: [Create the staging directories in /opt/autodeploy] *********************
ok: [localhost] => (item=/opt/autodeploy/projects)
ok: [localhost] => (item=/opt/autodeploy/resources)

TASK: [Create the resource directories in the staging directories] ************
changed: [localhost] => (item=docker)
changed: [localhost] => (item=ISO)
changed: [localhost] => (item=packages)
changed: [localhost] => (item=rpms)
changed: [localhost] => (item=scaleio)

TASK: [Download the private project repo from Git] ****************************
ok: [localhost]

TASK: [Download the Hanlon microkernel image] *********************************
changed: [localhost]

TASK: [Find the RHEL DVD ISO file url] ****************************************
ok: [localhost]

TASK: [Find the Red Hat KVM Guest Image file url] *****************************
ok: [localhost]

TASK: [Download the KVM Guest Image file] *************************************
changed: [localhost]

TASK: [Download the Red Hat RPMs] *********************************************
changed: [localhost]

TASK: [Create the local offline repository] ***********************************
changed: [localhost]

TASK: [Mirror docker-py and its dependencies] *********************************
changed: [localhost]

TASK: [Create the PyPi-compatible "simple" package] ***************************
changed: [localhost]

TASK: [Start and enable the docker service] ***********************************
ok: [localhost]

TASK: [Pull the project docker images from the docker registry] ***************
ok: [localhost] => (item={'image': 'mongo', 'name': 'hanlon-mongo'})
ok: [localhost] => (item={'image': 'cscdock/hanlon', 'name': 'hanlon-server'})
ok: [localhost] => (item={'image': 'cscdock/atftpd', 'name': 'hanlon-atftpd'})

TASK: [Save container image to usb drive] *************************************
changed: [localhost] => (item={'image': 'cscdock/hanlon', 'tar_file': 'cscdock_hanlon.tar'})
changed: [localhost] => (item={'image': 'cscdock/atftpd', 'tar_file': 'cscdock_atftpd.tar'})
changed: [localhost] => (item={'image': 'mongo', 'tar_file': 'mongo.tar'})

TASK: [Download ScaleIO files for ansible-scaleio project] ********************
changed: [localhost]

TASK: [Copy the downloaded projects to the usb drive] *************************
changed: [localhost] => (item=kragle)
ok: [localhost] => (item=wiley)
ok: [localhost] => (item=slimer)
ok: [localhost] => (item=ansible-scaleio)

TASK: [Copy the downloaded projects to the usb drive] *************************
changed: [localhost] => (item=docker)
changed: [localhost] => (item=ISO)
changed: [localhost] => (item=packages)
changed: [localhost] => (item=rpms)
changed: [localhost] => (item=scaleio)

PLAY RECAP ********************************************************************
localhost                  : ok=20   changed=11   unreachable=0    failed=0
```


Ты вручную что-ли подбираешь? Почти не отличить, если не зумить.


I'm afraid I don't understand the purpose of testing this at LTC.  The results at 3 and 7 threads have not been favorable.   Adding additional code or special rules is not the way to go IMHO.  It would also need to pass (0, 5).


Вручную в Illustrator'е рисую на основе растра.


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


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


You can just use Temp.CreateDirectory() without the using statement (see test below).


Рисуешь, а как потом цифири берешь?


https://github.com/afedchin/visualization.waveform/tree/master/depends
platforms should be osx and ios, not darwin.


@tomsoftware It would be a big help for me if you could tell me what the first values are in your code
```
unsigned int checksum = 0;

	for (int i = count; i > 0; i--)
	{
		unsigned int currentDWord = FR->readIntBE(4);
		checksum = ((checksum >> 31) | ((checksum << 1) ^ currentDWord));
	}
```
What is your first ```currentDWord```?
What is your first ```checksum``` (after the inital 0)?

This I could check against the java code


did this push just go straight to master @mistakevin? if yes, can you do PRs in the future? If this had been a PR, I would have had the change to ask some questions to see if we needed to re-deploy. the questions are:

- what does this fix?
- is it related to the bug I identify in https://github.com/OpenSourcePolicyCenter/webapp-public/pull/82 (the helpful 'Default: <value>' text does not work in CPI flags)
- do you think this change should trigger a push to heroku?


Path.Combine, Path.GetFileName and Path.GetDirectoryName would work here as well, since we are working with real paths that won't contain invalid characters.


@klokane thanks I'll look in that direction next.


`min/1` and `max/1` are currently private, since they are returned in the `%CIDR{}` struct (which is also a change).


Сохранить как -> SVG. Потом пропускаю через https://github.com/svg/svgo и копипащу оптимизированное содержимое в код.


OK I think this is the point where we start to say: this is smith and not waf. That means we can strip out tons of stuff that is waf specific, even including waflib/extras in that. Given that. I agree that everything you've removed should go. I would put docs/smith back though ;) Maybe we should remove this stuff from the repo? Any thoughts?


### [Codacy](https://www.codacy.com/app/Codacy/codacy-phpmd/commit?cid=16757579) detected an issue:
#### Message: `Avoid unused local variables such as '$unusedLocal'.`

#### Occurred on:
+ **Commit**:  a44f8c8862cb4bf3dea52526f60f36fe6db61a58
+ **File**:    [src/main/resources/docs/tests/unusedcode.php](https://github.com/codacy/codacy-phpmd/blob/a44f8c8862cb4bf3dea52526f60f36fe6db61a58/src/main/resources/docs/tests/unusedcode.php)
+ **LineNum**: [12](https://github.com/codacy/codacy-phpmd/blob/a44f8c8862cb4bf3dea52526f60f36fe6db61a58/src/main/resources/docs/tests/unusedcode.php#L12)
+ **Code**:    `        $unusedLocal = null;`

#### Currently on:
+ **Commit**:  316d2d62b6bd16ec997ef2e21134a2e6c45417b2
+ **File**:    [src/main/resources/docs/tests/unusedcode.php](https://github.com/codacy/codacy-phpmd/blob/316d2d62b6bd16ec997ef2e21134a2e6c45417b2/src/main/resources/docs/tests/unusedcode.php)
+ **LineNum**: [12](https://github.com/codacy/codacy-phpmd/blob/316d2d62b6bd16ec997ef2e21134a2e6c45417b2/src/main/resources/docs/tests/unusedcode.php#L12)


let's drop unnecessary parts


@SubJunk I didn't realize that the release schedule was time-driven (I took for granted that it was feature/bugfix driven). In my thinking it's important that a release is as complete as possible.

Regarding the translations things aren't that simple. The translators haven't gotten any notice about the changed strings. As I explained/discussed with @skeptical in b80aaffe47818e70a3530c6d0a7e0d2d6ac5581d, there's no way for the translators to know if a string is suddenly changed. We'd have to manually message everyone about which labels to retranslate in that case. The failure to take this into consideration is largely to blame for the current translations being so meaningless as they are for many languages. The english text has changed, but the other languages are unchanged - while the strings use have changed. That ends up in a complete mess over time, and is probably why I had to start at scratch with the Norwegian translation.

To prevent this dilemma it's my suggestion that we never change strings, but delete the old and create a new one. That way, it's easy to see that something's changed, and crowdin will even automatically PM the translators for new strings. But, nobody has even uploaded the changed ```messages.properties``` to crowdin (I certainly haven't since I've replaced the changed strings with new ones in #689), so the translators haven't had a chance to update the strings even if they had telephatic abilities. I didn't think that there would be a release before #689 was merged, so I hadn't anticipated this situation.

Regarding the version numbers there is an effort to standardize version numbering with [Semantic Versioning] (http://semver.org/). Many projects have switched to this in recent times, and it makes sense to use a standard because it makes version numbers have a meaning to non project members. The concept is explained many places, for example in this article: [Semantic Versioning: Why You Should Be Using it] (http://www.sitepoint.com/semantic-versioning-why-you-should-using/). I think this should be considered for UMS.


Whatever host you install this on, I can magically transform your filesystem into read-only state. :8ball:


А я думал, как ты так быстро в сокращенные записи все попереводил.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2384](https://teamcity.mesosphere.io/viewLog.html?buildId=64737&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:24


Je reste en mode texte, le json étant incomplet à la date où j'ai commencé mon dév.
Ok, bon courage pour la suite.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2385](https://teamcity.mesosphere.io/viewLog.html?buildId=64738&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:02


Clone this repository, then add a subdirectory called "stl", move the file into that subdirectory, then follow the instructions here to register the file into your local copy and then push them back up to Github: http://stackoverflow.com/questions/7036193/how-to-push-my-changes-back-to-the-source-code-in-git


Check indentation level.


Check on Android


Please ... this is closed.  See replacement pull request.


Done. Merged onto master branch at 15d06016c837d60551ef57ae2e1e7721c799edab


Touching on grove reminds me that the final temperature output should be a drange (as sensor has a min and max), but is on a eternal TODO list.


@ragesoss Thanks for finding this! That definitely gives me something to go on this morning.


Ah, good to know.


EFFFFFF


Yes. I needed to bring in the namespace to make it work, I changed it.


In fact I was having a similar issue as well and still didn't solve it. I wanted to implement nude.js as chrome extension to replace nude images with some other image in all pages. But, I was getting a CORS error (Cross-origin resource sharing) when scanning at some remote images. Some help would be very appreciated. Here is my repo:
https://github.com/trimkadriu/Nudity-Blocker-Chrome-Extension


It's not broken, I wanted it to behave like that.


Done.


Formatting


We should change the way we add the relations because for each row we do a sql query.

```
for(col in columns)
{
if()
{
call to database
}
}
```

We should think in add them in the select clause (more difficult but more efficient)


Do we have issue in tracker about it?


Changed _hasTabPosition to _isButtonRedirectionOn to help clarify the meaning of the variable


No, it shouldn't be now that I've added the second ctor param.


Consider handling the `STARTED` statuses as well.


What for?


I see that extension drivers are directly placed in ml2/extensions folder and not under drivers/extensions folder.

Does this work?

Is there any other out of tree example on where to place extension drivers for decomposed repositories like ours?


Great :+1: thanks!


These messages do not contain an `eventSource` field so this error occurs:
```
[ERROR] 2015-10-26 12:30:30,151 process Missing fields in message {u'status': u'STARTED', u'jobId': u'7', u'stage': u'chunk'}
Traceback (most recent call last):
  File "/opt/app/apps/workers/process.py", line 85, in handle_message
    event_source = record['eventSource']
KeyError: u'eventSource'
```


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1424 of 1424 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-70.42%25-green.svg)


This should be const. The assignment and double curlies should not be necessary.
static std::array<const char*, 5> const stateNames  {
    "disconnected", "connected", "syncing", "tracking", "full"};


Can you re-base this change on the current state of the upstream's uzem140 branch, which now uses CunningFellow's instruction pre-decoder and single-level switch statement inside avr8::exec() and submit as a PR? Then I'll do some benchmarking between having the update_hardware_fast() calls everywhere, or just in specific spots. With them everywhere, I was able to hit 225 MHz emulation speed using my Haswell-based i7, but I didn't get a comparable baseline of with them only in specific spots. Here is what my manual merge ended up looking like: https://github.com/Uzebox/uzebox/compare/uzem140...artcfox:merging-latest-timer-optimizations


This member is not necessary and can be removed.


Had to disable this temporarily because crashes.


Можешь потом все SVG файлы позакидывать в master/Images


Please remove extra line


You should probably pass `env` down to `update`.


This member is not necessary and can be removed. Use stateNames in its place instead.


Agreed, it is here by mistake.


Thanks. I have added comments on that.


It should be deleted.


I think you confused me. When you tell "infinite loop" you don't mean in programming terms, right?
You mean that each time you start qbt it notifies you about this kind of torrents being finished? That's the loop, that it continuously will notify you on every start. Correct?


Файлы кнопок панели? Потому что в файлах кнопок постов нет бэкграунда (я его рисовал отдельно).


Please remove this empty line removal from your diff


Should it be added to parent pom?


Я бы как минимум возвращал по умолчанию true.


I am not sure about it, though it seems @mglukhikh will fix it in near future.


What was the "other reason"?


Added ONLINE CAMEO functionality


Yes, of course. I'll add it.


Yes, then there is another array called alreadyBlacklistedTopics, which will determine the next call be true or false.


- Late-era Saxons wear Western clothes and headgear (historically informed; threads on the Paradox forums all confirm that Norse clothes for the Saxons don't make sense, and their fashion as shown in contemporary Anglo-Saxon manuscripts is much, much closer to Western graphical assets)
- Updated the monk and nun headgear code accordingly
- Dutch beard alignment moved to the left by one pixel


Hmm, then something should be changed in this rule:
- "The list cannot be scrolled down if the last known sith has no apprentice. The list cannot be scrolled up if the first known sith has no master. Use .css-button-disabled on the buttons to indicate this status."
Because without this rule your example can enter a state where no siths are displayed and up/down buttons no longer work.


Then there will be an infinity loop here.


I'll replace this (and other repos you've pointed) by a variable and whoever needs to run the script will point to the right place. Perhaps in the future there will be a public repo for this with soletta porting.


Woops I forgot to click save on Saturday


Лучше AnimationDirection


``DownloadConflictError`` is still referenced elsewhere in the file.


Nemo SonarQube analysis (http://nemo.sonarqube.org/dashboard/index/764069) build fails since this commit with error in jOOQ-meta-extensions module:
jOOQ-meta-extensions/src/main/java/org/jooq/util/vertabelo/VertabeloXMLDatabase.java:[90,7] error: VertabeloXMLDatabase is not abstract and does not override abstract method getDomains0() in AbstractDatabase


patch worked if I delete the closing bracket after the second value in line 67


@NullVoxPopuli Do you feel more comfortable with this?


Just to clarify for anyone else reading, we agreed that syntax is good for prototypes, but we'd want to have some more detailed discussions (with a wider group of people) before rolling it out to components themselves.

Int he meantime this is a good place to experiment with the BEM-inspired approach :+1:


I'm sorry about the confusion I made. Here goes detailed explanation. I have a few (less then 10) such torrents, and when I run qBt all of them are checked in a loop, qBt shows notification about finished torrent for each of them and this seems to be endless. qBt checks them again and again. I hope now it is clear what happens.


I'll remove mentions to proxy. Regarding the tarballs, I think there is a public repo for those, need to check.


I do not understand (any more) how Travis is intended to be used. I use this in over 12 free-software projects, each of them relying on a substantial amount of software only available on random PPAs. The situation now is that, I

1. Either need to do a substantial amount of work to install these deps from sources.
2. Or rely on a source-based packaging system (like PKGSRC) to install these deps,basically making ubuntu packaging system an almost useless layer.

How to do things right?


why?


@NullVoxPopuli I extracted this to its own method for clarity.


There's an extra full-stop that snuck in here :-)


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


The badge wasn't working anytime I looked, so the badge didn't render and I figured at least having a link worked. You're right that it doesn't auto-invite with just the link. Maybe check on that server and let me know - I can add it back if it's going to be something that will work.


Цвет кнопок ставить на белый?


Да можешь все что есть закинуть в какую-нибудь отдельную подпапку Images/svg. Просто мало ли, вдруг в будущем пригодится.


Ack.


В отдельных файлах ^


Added online cameo functionality


Can you provide a gif/video of my code where the UI is in a state where no siths are displayed, up/down buttons don't work, AND no network requests happen anymore? I haven't seen the invalid state happening. There are cases where you scroll and the list gets empty, but then there is a pending request for an actual sith, and it'll be put in the middle of the empty list.


Do we really need this calls here?


Как удобнее.


What's this guy for?


Please fix comment.


get the fuck on slack you whore


can you put a @return yardoc on these,

`@return [Array] [primary, included] the resource objects`

would make this complete

thanks!!!!


I do, see other comment :-)


Remove blank line.


Please move ``is_conflict = False`` to here from above -- defining variables in the narrowest scope in which they're needed tends to make the code easier to follow (and we only need ``is_conflict`` at line 693 I think).


SourceMaps support an optional `sourcesContent` field, which contains the content of each source file. It removes the source map's dependence on these ambiguous source URLs. The `sourcesContent` field is an array of strings, where `sourcesContent[i]` is the contents of `sources[i]`.

Almost every tool now supports generating source maps with this field defined (UglifyJS, TypeScript, etc.). It would be great if Istanbul preferred this field when present over files in the file system.


None of our business which methods you want to use.


@juddus ...


Why is this line duplicated in red?


Still there, I compact {proxyHost, proxyPort, proxyRefreshTime, maxRetries} to a blob as options, callback is the second parameter


This could use ``and`` rather than a nested ``if``.


Could drop io.js now.


Aww man, why didn't you make a new one? Now we *still* have to update DBs manually!


Ah, okay, I think I see what you're trying to do. You want to have this ELF_INTERPRETER_PATH to be set from the LStoreCommon.cmake. But, you only add the command-line define with at https://github.com/accre/lstore-lio/commit/71bb96e22b688e23e6f1c43356a02106dfb9bcd0#diff-44d137654d7b6ad9b59fa84c40be595aR89 within the UNIX NOT APPLE block.

At first glance, I believe you want to replace the above with

```c
#cmakedefine LSTORE_HANDLE_ELF
#ifdef LSTORE_HANDLE_ELF
const char elf_interpreter[] __attribute__((section(".interp"))) = "@ELF_INTERPRETER_PATH@";
#endif
```

Then add a corresponding change to the cmake:
```cmake
            # I think this has to be quoted to keep spaces from blowing it up .. maybe.
            set(ELF_INTERPRETER_PATH "${_ELF_INTERPRETER_PATH}")
            message(STATUS "ELF interpreter is ${_ELF_INTERPRETER_PATH}")
            set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -e print_${LSTORE_PROJECT_NAME}_version_and_exit")
            # Default exec bit policy for sharedlibs varies by OS and distro, force bits to be set
            set(CMAKE_INSTALL_SO_NO_EXE FALSE)
```


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


fixes for initial focus around accessibility usage instructions, page/menu load alert, first focus to navigation and skipping initial focus if user has already tabbed


Change to ``ConflictError``.


@joilson-cisne tecnicamente não é `compile` 8) . Pode ser `build` ou `transpile`.


I think checking `netloc` is safer. You can have urls like `//hostname/foo/bar' too


@joilson-cisne Não que esteja errado, mas existe uma forma mais elegante de fazer isso. ;)
Se você fosse escrever isto como HTML, como você passaria o valor do conteúdo?


ok, thanks.


... and change ``DownloadConflictError`` to ``ConflictError`` on line 701. Otherwise LGTM.


@udalov, what do you think about this paramter?


the unnecessary line


Some background on this diff:
Since v1, the highest firing crash in ads manager android was related to JS processing `touch end` events before `touch start`. This diff finally fixes that.

TLDR: android was supplying timestamps for `touch end` events that came before `touch start` events. ReactNative sorted these events by their timestamps, so JS got the `touch end` event before the `touch start` event of the same touch event, and crashed. The fix: generate our own timestamp when dispatching touch events.

Long dramatic story:
* Initially we weren't sorting the touch events at all, so we thought that was the cause of this crash, fixed it and called it a day.
* The fix only made the problem worse, and nobody knew why.
* We found some more inaccuracies in our event dispatcher and in some other components that might have caused this crash, so we fixed those and hoped that that would somehow fix this bug.
* The crash rate didn't change at all. What made matters worse, was that users started to report that madman was crashing 'after 2 or 3 clicks' or 'tapping the menu bar'. Nobody managed to repro any of this.
* Since we had no idea what was going on, we added whatever debug information we could get our hands on, force crash the app as soon as we see this problem, and ship that in a RC.
* So we did, and we started to see that indeed from the native side we were sending a `touch end` event that had no `touch start`. **WAT**.
* We added more debugging information and shipped that to the next RC. Now we saw that for each `touch end` event that would crash the app, there was a `touch start` event sitting next in the dispatch queue. And these touch events were ordered correctly because the timestamp for `touch end` was smaller than the one for `touch start`. **WAT**.
* The timestamps we were using for touch events were all coming from `MotionEvent#getEventTime()` supplied by android, since we thought those were the most reliable timestamps we could get. So we changed them to use the dispatch timestamp of the events instead. The problem immediately disappeared.

Lessons learned: question everything, especially android.


this change takes away the only advantage this weapon had compared to rifle class (mitigation is one thing which is usually useless, as there is only one race which has armour at all: androns)


Did this work?

It looks like you need to insert:
    @property
    def extension_alias(self):
        return self._supported_extension_alias


@mcostalba, can you pls suggest a suitable Time control for LTC. The draw rate suggests this will take forever at 60+0.4.
A game that lasts 100 moves adds another 40 seconds. That is doable for 1 thread tests. Not realistic for 24 threads with just 2 machines available.

@jhellis3 pls see Marco's comments above for testing this patch. I don't intend to add additional code or uci options as Marco suggested. just changing the coefficient of log from 3 --> 2.2.


== answer me


@joilson-cisne Normalmente a gente não dá commit dos bundles gerados. Além disso não é muito comum dar build para a mesma pasta, e sim para uma outra pasta - `build`, `dist` etc. Recomendo tirar o vilão e impedir que ele volte. 8)


I don't see the purpose of decrementing here. iteration is reset at the top of the loop.


This method can be omitted by using node's `util.inherits` and `util._extend`, please check:

https://gist.github.com/aynik/4a67c68ef9f6372c91ec


Can you also update these versions?


Another modification.


It's 1.5.5 now (afaik). Can you update it to whatever the latest version is?


Just being curious: I'd expect it to be `yield next()`. Why don't you call the method?


Done.


Add a space before and after "%".


Unrelated: looks like `module` should be replaced with `Xmodule`


As i mentioned in my comment in PR it maybe better to re-order onlyNamesCheckbox as other fields:
`Field name` `Field value`


Ah never mind. I understand the functions that go beyond order two are aliases to `cl-lib`.


Sorry bout that, just caught it this am, but you had it fixed already.


@moret Copiei o comentário diretamente da [documentação](https://babeljs.io/docs/usage/cli/). Ver a parte que fala sobre `--watch`.


This project (internalTest) was created as a copy of a sibiling project. I did'not look at the details of this file.


I'll investigate it.


@educostachaves não esqueça de dar uma olhada nisso 8)


I needed to implement it for a model I'm developing for my lecture. Since Cur already had a test suite, I implemented it here first.

--
Sent from my phoneamajig

> On Oct 26, 2015, at 12:36, "Max S. New" <notifications@github.com> wrote:
>
> What was the "other reason"?
>
> —
> Reply to this email directly or view it on GitHub.
>


looks good


It seems it is not necessary.


How about to test the accessing to internal members of class


Typo on purpose


@joilson-cisne hunf, este povo moda jovem do Babel, quem eles pensam que são? 8)))


Hi,

Short remark, I think your usage of SemaphoreSlim is wrong here. You seems to try to limit your outstanding requests number by using a semaphore but you're using the wrong constructor.
As we can read on [MSDN](https://msdn.microsoft.com/en-us/library/dd270887(v=vs.110).aspx) and [reference source](http://referencesource.microsoft.com/#mscorlib/system/threading/SemaphoreSlim.cs,ead13858a2ff9c7c,references) the constructor used here is defined to set the **initial count** instead of the **maximum count**.

This code only creates a semaphore with initial value at *maxOutstandingRequests* and never block anything.

You should use the constructor that take two parameters, both initial and maximum count:

```Cs
var semaphore = new SemaphoreSlim(0, maxOutstandingRequests);
```


Thanks for changing it, I think for now we can save on every command and if it later becomes a resource hog we can always add a cvar.


pszTotalInstallSize should be freed at the end.


They did before, but if you look at line 101 and line 108, I reversed their indecies for the specs to pass. I added a comment above each of those lines explaining this.


This solves the issue on add the element `data-filterable="false"` on a column header?


It is actually part of the readelf output and used in the regex on line 86


I use unittest-xml-reporting for functional testing that proceeds very long and outputs lots of logs with different severity to many handlers. This param lets test runner use my configured logger. And tracebacks are on the right places in logs.


GOoood !
Well , so i start to document myself about audio graph .
Available for windows 10 :
https://msdn.microsoft.com/en-us/library/windows/apps/mt203787.aspx

Because i am thinking at a chordRecognition node , eh eh !
^^


Is there a reason why this has to be called SNMPProvisioningAPI?

Can't this be just called PortProvisioningDriver?


Hmm, I only checked with 1 torrent, which display an initial notification and then nothing(until qbt restart). What happens normally to those torrents before my commit? In what final state do they go into? Paused/seeding/downloading/queued?


Thank you very much for the hint. Fixed with https://github.com/jOOQ/jOOQ/commit/e89a2a8c9e95bc2797a50f7de35a6d5e6d11bc79. Unfortunately, that project isn't being built by our nightly builds - only by our release builds. That's not good of course. We'll fix that.

Are you checking jOOQ for any specific reason (e.g. you're using it at Sonar Source) or do you do that with all of GitHub's Java repositories?


@ecoslado notice this controller is only for invoices and recurring invoices, not all the app.


P.S: I just added the missed discovery files. You can add it now:
> api_discovery: "https://geo.sv.rostock.de/citysdk/discovery.json"


Debug print?


:neutral_face: I must not be able to ctrl-f and spell "interpreter" properly. My bad.


This is still a side-channel: the time it takes to create a copy is probably as detectable as the difference between `mpi_lshift()` and `mpi_add_mpi()`. Anyway the rest of our bignum implementation is not constant-time, unless the few functions that are explicitly documented as such (we use blinding in some places to compensate for that). So I'd prefer to have the simplest implementation here (the one in your first patch looks good).


Сделал поправки .
Оставила метод findQuery, потому что он находит записи, удовлетворяющие запросу, которые потом посылаются в find, remove, где уже идут разные выводы сообщения.


I may be being overly critical here but bumping a major version of a dependency seems like it should be reserved for at least a minor version, not a patch version. I'm getting an NPM warning on Hapi because of this.


@UlfNorell please add documentation for this change in [future.txt](https://github.com/agda/agda-stdlib/blob/master/doc/release-notes/future.txt).


Could maybe also have the format string as its own parameter, seeing as these are all logging calls which will do the formatting as needed.


Don't hard code here.


Thanks for your comments.

I have some past experiences implementing inlining using machine learning -- stuff that I found difficult to incorporate into the paper, since those projects were never made public.

Suffice it to say that I don't want to presume any particular structure for how heuristics combine to make a recommendation, so the neuro-evolutionary approach resonates with me there. But adopting it also concerns me since there's a lot of art in making these things work well and I've got no experience here to fall back on. The one thing that counters this is that I really like the idea of projecting the result back onto a decision tree which hopefully makes logical sense.

The correlation comment is interesting. We likely will have too many features. In past inliners I've had as many as 30. That's a lot of weights to juggle. Running something like PCA may make sense, to try and winnow things down a bit.

I  disagree with you about the easy predictability of inlining. There are some obvious good cases but things get considerably harder if you want to both be aggressive to boost CQ while also keeping CS/TP under control.

Note this is an early draft -- after a bit more internal review I'll push this up to CoreCLR proper, hopefully in a few days. Once this happens it would be good for you to add your thoughts there since they'll be easier to find.


This should never happen. As in if the tcp connection to the proxy is not avail, you should panic.


Panic!


Simialry should be names SNMPDiscoveryDriver.

Can't this jsut be DiscoveryDriver?


cause the following error:
[26-Oct-2015 16:24:23 UTC] PHP Notice:  Undefined property: nzedb\processing\tv\TraktTv::$requestHeaders in /var/www/nZEDb/nzedb/processing/tv/TraktTv.php on line 33
[26-Oct-2015 16:24:23 UTC] PHP Stack trace:
[26-Oct-2015 16:24:23 UTC] PHP   1. {main}() /var/www/nZEDb/misc/update/nix/multiprocessing/.do_not_run/switch.php:0
[26-Oct-2015 16:24:23 UTC] PHP   2. nzedb\processing\PostProcess->processMovies() /var/www/nZEDb/misc/update/nix/multiprocessing/.do_not_run/switch.php:261
[26-Oct-2015 16:24:23 UTC] PHP   3. nzedb\Movie->processMovieReleases() /var/www/nZEDb/nzedb/processing/PostProcess.php:203
[26-Oct-2015 16:24:23 UTC] PHP   4. nzedb\processing\tv\TraktTv->__construct() /var/www/nZEDb/nzedb/Movie.php:1088


Please address a few comments.


This comment confuses me. I think something more like:
# Consider two or more missing measurements to be a failure.


@martijn00 When will the fixed nuspec be published to nuget.org?


I'll add it.


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


## Changes
- More portable & 8-bit code, And comments.
- Remove the non-reliable benchmark example (file `./examples/SpritzCipherBenchmark/SpritzCipherBenchmark.ino`).
- Rename the function `stream()` to `spritz_rand_byte()`.
- Change version in file `library.properties` from `0.1.2` to `0.1.3`.


I have 6 of such torrents. After reverting this commit (and before it, I guess) they are in "seeding" state.


For now, it can only be setup via the .ini


I can't test it at the moment, but I don't think that's the issue. `contactTest` supports testing across different worlds, as noted by the Bullet documentation: "The query object doesn't need to be part of the world." It's just to test any rigid body, even ones from other worlds, against a certain world.


[Deployment available online](https://s.brightspace.com/lib/ifrau/0.11.0/ifrau.js)


So this line should be in ApplicationController.


Thanks!


@nad please add documentation for this commit in [future.txt](https://github.com/agda/agda-stdlib/blob/master/doc/release-notes/future.txt).


@azul This PR seems to break the /users page.. can you take a look please?


thank you


keep-when and remove-when are defined in my fork of Apraxis: https://github.com/christianromney/apraxis/commit/33e21985bc105e3a1f69d3e6f7e0e70a126e7383 (PR opened).


What is the scope of `all_ids` now?


Why ssize_t ?


ex'DEE


These two add no value and clutter. I recommend in-lining "transitions" and "duration_us" in the one spot they are used.


@UlfNorell  please add documentation for this commit in [future.txt](https://github.com/agda/agda-stdlib/blob/master/doc/release-notes/future.txt).


Hi @maxts,

I have a couple questions:

* How can I easily see the changes you made to the original?

* Does it work? How would anyone other than you know?

* What happens in the following scenario?

    ```js
rsrc1_1 = NAMESPACE.resource('ref1');
rsrc1_2 = NAMESPACE.resource('ref1');
rsrc1_1.close();
// What is the state of the world here? What *should* it be?
```


We received your resume from StackExchange. Why not send us your portfolio now?


Not necessary. Pending this I say go.

Also next time, it would be helpful to open a pull request from a branch.

git checkout -b bugfix/my_bug_fix

or
git checkout -b feature/my_feature.

so we can iterate & squash empty changes like these.

THanks!


@catterbu I would change the name of the function - you're not generating a block dictionary every time, you're doing something more like `append_block_dict`. Actually maybe naming is not even worth it: I'm worried about three levels of functions after calling `corral()`. It's a lot to follow, logically. Since your function is only two lines long, I'm thinking we should keep that code in `_explode()` and maybe rename that function instead.


cm-12.1 branch -_-


Well, I think there's no problem accessing a const std::string from within the thread handler (since it's const, it won't do memory allocation) -- this is outside so there's no limitation on allocation.

Besides, 'sprintf' is unsafe in any context (to first order, there's no reason to use it, ever).  You can probably read up on the topic by googling 'sprintf bad'.


Je n'arrive pas a afficher l'image par defaut. erreur 404


get_buf_size will never return a negative number. Maybe you should change
int get_buf_size(size_t rows, size_t columns, size_t *out);


Here is what you see:

* A flashing rectangle outline indicates an update because of incoming data.
* A flashing cross indicates a discarded update (loop/cycle detection).
* A small flashing "dot" directly before the flashing cross shows the perpetrator of the duplicate update.

![animated-dataflow-1](https://cloud.githubusercontent.com/assets/5765308/10736116/8a7f65fe-7c0b-11e5-9aa0-6d611667d19c.gif)


Il resto è OK per me, su questa se ho ben capito sono gli aggiornamenti manuali, Su richiesta?


That's it, thanks !


in analyze_logcat, I think it would be ok to change:

            if not results["tcheck3"]:
                continue

            # calculate score
            data = results["tcheck3"]
            pageload_metric["tcheck3"] = median(data)
            pageload_metric['summary'] = geometric_mean(data)

to:

            if results["tcheck3"]:
                # calculate score
                data = results["tcheck3"]
                pageload_metric["tcheck3"] = median(data)
                pageload_metric['summary'] = geometric_mean(data)


I don't like string processing on paths, it's a common source for fragility. Can you replace the String that with the appropiate C# type? Something like [this](https://msdn.microsoft.com/en-us/library/system.io.fileinfo(v=vs.110).aspx) or [that](https://msdn.microsoft.com/en-us/library/system.io.file(v=vs.110).aspx) or sth else?


Outside of the resource function as resources may share an expensive
resource with the same ID, perhaps I should see it work in the context with
a series of assertions.

On Mon, Oct 26, 2015 at 7:00 PM, Matt Bogosian (formerly @mbogosian) <
notifications@github.com> wrote:

> What is the scope of all_ids now?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maxts/curiosity/commit/95f40bfe851e056935d71f607cec2019ac9a75e4#commitcomment-13989777>
> .
>


now can be in-lined, the local is unnecessary and not useful. If you are going to create a local here I recommend 'elapsed', which would be useful for debugging purposes.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - i386-Windows-7-VisualStudio-12: OK (1944 of 1960 tests passed)

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


Wait, in fact this is not important, counting convention is up to the user^^


@jtblin Fascinating, I've been running rack-cache for years in a threaded deployment. Any thoughts on what kind of errors could have occurred before this change?


not sure why this was done like this: addNode probably needs to be called with "true"


Leaks the string.


What is wrong is ``get_buf_size()``, then. Thanks.


Here, not zero on the end.


If there isn't a `homepage-url` won't Datomic complain that we are trying to store a `nil` value?


Also resolved issue #102


Hey @siddii, I didn't understand why did you wrapped other members with trustAsHtml, but published date is converted to numeric value again? why don't just wrap it as well?


here's how you can solve it: you need to have a proxy to have the image served from your domain -- unfortunately there is no other way due to HTML5 canvas' CORS restrictions.
but someone created a plugin to fetch a base64 representation of an image via JSONP and his server: http://stackoverflow.com/questions/7988202/how-to-use-canvas-to-modify-images-from-another-domain
so you probably don't need your own proxy server


Yes, this is on my TODO list of things to fix.


Please use the DateUtility class in dcs-shared-util to format date for consistency with other places in the code.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 34](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=365&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


Thanks!


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 34](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=365&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **SUCCESS**
Summary: Running Build time: 00:00:40


Changes made in this commit were amended in the commit following it. (Removed PendingIntent function)


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1944 of 1960 tests passed)

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


`.r-List` ?


loadXDR also needs to be updated below


Ugh Daniel, read the code. `assoc-when`


This renames a hook, are we going to have trouble with this? Perhaps we should have hook aliases? I think :page is a nice name, you think :pages is a nice name, we can have aliases like we do for Jekyll Assets hooks.


Ditto on the above.


Why not `Hash#fetch`?


From past experience this is slower than `define_method` with `send`, any reason why you opted for this?


why not use __builtin_frame_address on compilers that support it (gcc and llvm), as we have a warning every time this function is compiled?


in analyze_logcat, I think it would be ok to change:

            if not results["tcheck3"]:
                continue

            # calculate score
            data = results["tcheck3"]
            pageload_metric["tcheck3"] = median(data)
            pageload_metric['summary'] = geometric_mean(data)

to:

            if results["tcheck3"]:
                # calculate score
                data = results["tcheck3"]
                pageload_metric["tcheck3"] = median(data)
                pageload_metric['summary'] = geometric_mean(data)


Sure! We use duplicates here for front matter defaults so I figured it would be nice to be uniform. `:site` has stayed singular, but `:pages`, `:posts`, and `:documents` are all pluralized now. It gave me the flexibility, for `Document`s, to call `Jekyll::Hooks.trigger collection.label.to_sym, :hook_name, args` which I found powerful. This way, you can define hooks for your `:cookies` collection or whatever and have global `:documents` hooks for all Documents.


早点睡吧！


@tvsonar this mapping is duplicated in multiple places


As discussed, feels like the wrong place for this sort of thing. Reckon a test helper of some kind, perhaps used in conjunction with a test instance of the exchange?


I think this is an advantageous example of the need for Hooks on configuration, we could push this off of this class and into an internal hook and simplify this class since it's probably starting to get beastly.


We use that in different places and do need a convention. So far we're counting from 1 for line numbers because that fits to what users see in the editor.


Not sure, just copied from above


Hadn't thought of define_method last night in the trenches.


Estes campos não são específico para o itaú, no caso o boleto_especie
 C015 - Espécie do título , os valores que coloquei na seleção, realmente são do itaú, somente.


@tvsonar this seems duplicated in multiple places


This is pretty cool, not 100% sure of what it does(I also never used spim for assembly so that might be why)


:+1: let's do that in 3.1. can you show me how you'd like that done at some point?


This is a expected behavior
See this works
http://www.question2answer.org/qa/47326/updating-donut-theme-any-suggestions
But these are not
http://www.question2answer.org/qa/47326/updating-donut-theme-any-suggestions%
http://www.question2answer.org/qa/47326/updating-donut-theme-any-%suggestions


I was in the middle of typing that comment… I agree it feels a bit wrong.


Want to be able to trigger hooks for arbitrary collection names, like `posts`. https://github.com/jekyll/jekyll/commit/b89f943bf22bd7f36c8378791ca543e16a59f442#commitcomment-13990093


Is "FOO" a placeholder for a testing string?


I will do this as a separate PR, since this PR needs to be reworked to move region names into the provider rather than nested below it.


I guess, I'd skipped that line. No need to test the above code, it will provide the same results. I had to add my objects to the world because my gamemode tends to simulate objects. I'll try implementing contactTest to ColAndreas, if it works I'll get back to you.


[Deployment available online](https://s.brightspace.com/lib/ifrau/0.11.1/ifrau.js)


Digging down and cleaning up the Plant Model Template led me to a few different types of holes, notably those of a few (~5) pathways that should've been linked to subsystems, and a few missing reactions (it turns out that if an appropriate reaction was not available AraCyc for a given role, it's counter-part in MetaCyc would be used, but I originally missed this).

The end result is that the parent database for the new Model Template (PlantSEED_Roles.json) now has all the reactions it should have. NB: This currently also includes reactions that are unbalanced and will either not be added to the template, or fixed.


I'm not sure about this code.  I'll need to look at it in more detail later today.


Can we use hasattr() here?


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


ditto


Any way to configure this?

In raven Client.php in "sanitize()" on line 502 -> $data = Raven_Serializer::serialize($data);


Addressed recent review comments.


Ok. I reinstalled the sucker and cmd-R finally works and now SIP is disabled, but I am stuck in a log in/off loop. How do I get out from this? I installed SwitchResX and can't do anything because of this loop.


@catterbu Also, please write test cases to verify this functionality. With the fast iteration and adjustments we are likely going to have to make, we'll need to lean on those to make sure we don't screw something up last minute.


This commit is not complete. Proper setup of coverage.py is still needed in order to generate a test report.


agree.

@divan: what do you think?


in what condition GetState returns error?


lgtm


remove :


The documentation is pretty spotty unfortunately. This probably should be documented, but there's a lot of stuff I wouldn't want to document without some feedback and I've had limited success getting that.

I usually call the API methods, but I think it's good if stuff can alternatively be passed as options.


Nit: s/have/has here and line 561


This is the issue.


Should we just remove this?


I have actually considered the scenario, the limitation is that you
wouldn't dispose of a resource until you are finished with it, an
implementation to check for open resources could be engineered to prevent
the issue. Provided the script is small I haven't forked your repository
and checked if it works in Google Chrome console, for a bigger project
forking and merging would be a daily event (as well as writing series of
assertions).

Here are a couple of my previous works:
goptions.com
binaryoptions360.com

Both are live and highly profitable businesses.

Best regards, Maxim.

On Mon, Oct 26, 2015 at 7:02 PM, Matt Bogosian (formerly @mbogosian) <
notifications@github.com> wrote:

> Hi @maxts <https://github.com/maxts>,
>
> I have a couple questions:
>
>    -
>
>    How can I easily see the changes you made to the original?
>    -
>
>    Does it work? How would anyone other than you know?
>    -
>
>    What happens in the following scenario?
>
>    rsrc1_1 = NAMESPACE.resource('ref1');
>    rsrc1_2 = NAMESPACE.resource('ref1');
>    rsrc1_1.close();// What is the state of the world here? What *should* it be?
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maxts/curiosity/commit/95f40bfe851e056935d71f607cec2019ac9a75e4#commitcomment-13989812>
> .
>


It works now.
I don't know why, but Heroku changed dynos to paid plan (not free as it has been before).


Indeed, tks!


It will probably be some weeks before the next beta arrives.


deletePlot should return a `plot` not a `figure` ?


Thanks, @mekza.


Credo sia l'opzione "Controlla aggiornamenti ma permetti all’utente di scegliere se installarli" in Avanzate->Aggiornamenti.

Inizialmente avevo messo "chiedi", "richiesta" mi sembra un po' più formale.


证书是你自己签名的? wosign弄个免费的https证书嘛


yup


Yeah, you can actually use this syntax:

```js
browserify({
  // no options:
  plugin: [watchify, otherPlugin]
});

browserify({
  // nest arrays for options:
  plugin: [[watchify, {/*opts*/}], otherPlugin]
});
```

Unfortunately, the nested arrays syntax doesn't work for tranforms, you can't actually pass options like that. But it should. I'll put a PR together for that later.


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


[frau-jwt/host available on the CDN](https://s.brightspace.com/lib/frau-jwt/1.0.5/host.js)


Where's the parameter with the stream? ;)


@gavinking yes, looks like this works (2 for 2 in repro attempts before rebuild, 0 for 5 after.)


Don't we only make a new one for new releases?


What's `@context` stil ldoing here?


@sbeaver-netgate Bonus right-bracket in this line reported on forum https://forum.pfsense.org/index.php?topic=101390.0
PR https://github.com/pfsense/pfsense/pull/1988 to remove it.


I like! :+1:


We need the stream itself as a parameter, otherwise you have to keep it in your implementation as well...


Can we split the tasks @joseph-jalbert one for detect the errors and one to detect & fix. Something like gulp fix:php

Thanks.


Couple of questions:
- Why not check specifically for Nan if that is the problem?
- Does the collection view blow up if the offset is too big or if it is nan or both?


I think we should have an associated ``screen`` event fire in viewWillAppear so something shows up in the screens section on GA.

I also kind of prefer to have the event be in the controller too - just to keep the router dumber. Though another solution would be to have a protocol controllers can implement to have the router fire an event for them when they're shown?