TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 629](https://teamcity.labkey.org/viewLog.html?buildId=272307&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:07:49
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

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

```


Should faraday go in the development group, as it's not need for running the tests?


> I use flex from Cygwin distribution.

Which is, as far as I know, intended to generate code to be built with the Cygwin libraries - i.e., code for UN\*X.  Try WinFlexBison's version instead.


is this for Importing LC records only?  Might want to say.


@normanmaurer do you think the fix is related to #4225 ? I could _imagine_ a scenario where leftover packets would grow the pool :)


Quick question-when I attempt to run this tool I get the following message:

[PENETRATOR] No device specified, please choose from the list below:

Choose device ID:

Nothing is displayed apart from that, where should I look for the device ID and am I doing anything wrong?


@robwalch @donato This is the error handler in html5 provider. The error handler triggers JWPLAYER_MEDIA_ERROR, which is handled in controller.js to change the model state to ERROR, but the provider's state does not change to error.


"[Your parents] will probably keep using their [iPhone] 4s even though it is two generations behind the update"

I think this goes back to the idea of working and non-working. If a technology gets a change, but the user finds that change too confusing, the technology will have stopped working for them.


I would expect a "doc" folder to have documentation about this code.  Maybe rename to "import" folder?


I haven't tested it, I didn't even know it existed, I just remember that you mentioned the idea. I don't know if I have a suitable renderer either (I've bought myself an old PS3 for testing, but I've yet to see any thumbnails).


I am sorry if I didn't comment on this previously. Look at our [coding style guide](https://github.com/qbittorrent/qBittorrent/blob/master/CODING_GUIDELINES.md#1-curly-braces). Opening braces of functions go to their on line.


.arg(online ? tr("ONLINE") : tr("OFFLINE"))


4967-SZRH-spm (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1854 of 1872 tests passed)

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


same here


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.971.bff4e0530e1b1b23a445c97eb0ad91b36f5ac7af](http://build.palaso.org/viewLog.html?buildId=63275&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 514, ignored: 6 Build time: 00:03:25


There may be a type error here.  value is declared to be 8 bits wide, but it's being treated as a 32-bit value.  I think the implicit case to 32 bits is occurring after the shift, but I don't know for sure.  Regardless, I think the code will be more clear if value is declared to be a 32-bit value.


helpers.h has a BIT macro that could be useful here.


Ah okay; I'll exclude this commit from the PR then


Opinion: If relative file paths should be encouraged then there ought to be a documented requirement about the directory to which they are relevant. I chose to assume that the paths are relative to the top-level directory of this node module because that's the only way to maintain backwards-compatability with the known installations of this package.

I think that paths should be relative to `CONFIG_DIR`, and `CONFIG_DIR` should be outside of the node module in a production instance. For example, I think this would be a good layout to recommend:

    .
    ├── data
    │   ├── keys
    │   │   ├── 0ab022e6-cc91-458d-9ba7-7a0717fe4fdf
    │   │   └── another_example_key
    │   └── proxies
    │       ├── proxy1.json
    │       └── proxy2.json
    ├── node_modules
    │   ├── some-other-package
    │   ├── node-libs-browser
    │   └── oauth_reverse_proxy
    └── start.sh


Chunyang Xu <notifications@github.com> writes:

> I'm using the zenburn theme (its helm face customs), this added line makes it looks odd:
> 2015-09-
>
> its original look:
>
> 2015-09-
>
> I don't know how this happened and how to fix it, any idea?

I have reverted for now, will see this later.

Thanks.

--
Thierry
https://emacs-helm.github.io/helm/


Remove this, too opinionated, there may not be a 'timestamp' in the body, and even if it's a default... still opinionated about format.


Not needed without datetime


These types of comments aren't really needed.


Docstrings underneath each of those though should be.


Hmmm. In that case, would I have to similarly add these lines to the csproj for Microsoft.DotNet.Build.Tasks? I tried it and was assaulted with an enormous number of dll/function/class missing errors, so I'm not too sure about that.


Ah, I see. Sorry for the confusion.


PLEASE update pip archive. it still contains such buggy version...


What is the purpose of the double-negation?


You could try it like this also,

```ruby
def create
  @advertisement = Advertisement.new(advertisement_params)
  ...
end

def advertisement_params
  params[:advertisement].permit(:copy, :price, :title)
end
```


Replace the defines with "const unsigned int" constants , please (for type-safety reasons).


LGTM


And since you'll do edits anyway please change this too:
```c++
logger->addMessage(tr("qBittorrent is trying to listen on interface %1 port: %2", "e.g: qBittorrent is trying to listen on interface 192.168.0.1 port: TCP/6881").arg(ipString).arg(port), Log::INFO);
```


This commit may not be necessary as I think the GA settings end up taking precedence as [seen here] (https://segment.zendesk.com/agent/tickets/34238). I think customers need to ensure that they check the non-interaction box in their GA settings. If the customer confirms that checking the GA non-interaction box fixes the issue, we should probably remove this commit / code.


no this is a great way to do it, you've discovered a programming technique called recursion.


It's called XML, son. :) Though I admit that I did originally just stick in HTML thinking it would work! It totally didn't. But found https://wiki.duraspace.org/display/DSDOC5x/XMLUI+Configuration+and+Customization#XMLUIConfigurationandCustomization-CustomizingtheNewsDocument and was like DUH! XML!


For 1), I think the idea would be to compute per-frame figures by aggregating the per-draw figures.

But for 2) there isn't much alternative indeed.

The problem with multiple passes within a single glretrace process is that it's not bullet proof: we maintain several things in globals, and they not all get reset after the replaying.  Even if we cleanup all globals, there's still the risk that some global state in the graphics APIs we use cause behavior change.

Rather than doing multiple passes inside glretrace, it might be more robust to run glretrace process multiple times (you could even have glretrace spawn itself for each pass.)


Added English name for language and find_what_looking_for:


And you didn't fix this.
You should do
```c++
connect(&m_networkManager, SIGNAL(onlineStateChanged(bool)), SLOT(onlineStateChanged(bool)));
```


On 15-09-2015 11:28, Gustavo Lima Chaves wrote:
> That would be better. Aren't default values obligatory too?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/edersondisouza/soletta/commit/82e202da8a45c74526cd6772b69bb86729d8defe#commitcomment-13238836>.
>
Not really, they will default (defaultception).


Photovoltaics (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Cool!


This stuff should not be in here.


mysql_ is required for adoptConnection(). That said, adoptConnection() itself is usually a bad idea.


Sorry the correct one is:
```c++
connect(&m_networkManager, SIGNAL(onlineStateChanged(bool)), SLOT(networkOnlineStateChanged(bool)));
```


<another test comment  --   3>


that said, I'm surprised by mysqli here


This code is compatible with Python 3.


Will do, I plan on updating this tonight after I spec out my intent more formally.


Nice commit message :dango:


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - i386-Windows-7-VisualStudio-12: OK (1871 of 1872 tests passed)

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


Doesn't this need a `new`?


Oops...


@jef-n I'm wondering if the auto release scripts are working correctly - shouldn't 2.8.4 have been tagged yesterday? Or did we just miss out by a couple of hours/minutes/seconds?


Photovoltaics (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


This commented out line was likely unintentionally committed.


The first time it is called from the outside. It may happen.


Please remove the trailing whitespaces here.


@martinb3 thank you!


:+1: Hey, I learned something!


Already fixed in a later commit ;)


I am a nice comment


DOH!  done.


Many thanks!  This did it for me.  Did you submit a pull-request on fiestah?


Acredito que fica melhor diminuir o número de caracteres das variáveis.
Fica melhor assim: VelMediaParcial = VMP


This PR is no longer necessary.


Absolutely earth-shattering. :+1:


@romilrobtsenkov  tegin kodutöö valmis mõtlesin natsa hasseldada :D


jep. isch ufem view-expense ezt au druff :)


@rafaeljusto this is beautiful, thank you so much for doing the work! I will let you know when I've given it a try.


test


01-js-intro-and-jquery/Topic 1: JavaScript Intro and jQuery/js/script.js

+1! :+1:


se vc souber o tamanho do array de dados (da lista gerada pela query), fica facil

```
#!/bin/bash

query1=(web03 web04 web05 web06 web07)
query2=(web13 web14 web15 web16 web17)

for i in {0..5}
do
        echo ${query1[$i]}
        echo ${query2[$i]}
done
```


I was thinking that this would be defined by the backend (if it has a threshold for displaying an 'about to expire icon' already) or by the addon and put on the addon's wiki page.

Is it just the name that's the sticking point, (say if I changed it to bExpiresWithin48hrs would you be happy) or do you just not like the concept as a whole?


OK will do.


Database name still needed in `table_exists()`.


Photovoltaics (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


TeamCity pull requests :: netty [Build 207](https://garage.netty.io/teamcity/viewLog.html?buildId=1382&buildTypeId=pulls_netty&guest=1) is now running


Thank you!  This was causing me a lot of grief with the following error:

multiple dex files define Landroid/support/annotation/AnimRes


What about NPCs that are (legitimately) dead at the start of the game? With this change, I think you'll see them fall down when you enter the cell, rather than already lying dead to begin with, as they should.


I like merging into 'build' script. That way we will be running all of this as part of the build and rebuild.

How about merging all these under 'test' script and leave 'build' just as it is(just run 'test').
```
"utest": "foreach abacus node_modules/:name npm test && coverage",
"itest": "export THROTTLE=1 && foreach itest node_modules/:name npm run itest --",
"test": "npm run utest && npm run itest && npm start && sleep 5s && npm run demo && npm run perf && npm stop",
```


Nice work man, and welcome to 100 Commit Club! Keep it going.


I'm going through and adding explicit string bounds to cut down on the risk of this kind of thing. Thanks again for the reviews, and I'll have something a bit less fiddly soon!


yeah, I'll do that :3 Thanks!


Flex for Windows produces same output.


TeamCity pull requests :: netty [Build 208](https://garage.netty.io/teamcity/viewLog.html?buildId=1383&buildTypeId=pulls_netty&guest=1) is now running


OK, I will do it tomorrow. Thanks!


Thanks :smile:


Zeile 948 sollte leer sein.


Ok. I was following the convention [in the tutorial](http://ocaml.org/learn/tutorials/modules.html), but I'm happy to follow local style here.


This is my first repository for my class on data science. I hope to complete all 9 classes. This is not going to be easy.


Zeile 982 entfernen.


Thanks @msecret. Since I really should have a local copy of this running anyway, let me try to do that using these instructions. Will report back!


Tole raje naredi brez zanke `for`:
```R
konstruktorji <- unlist(sapply(f, function(x) x$Konstruktorji))
ITA <- unlist(sapply(f, function(x) x$ITA))
JPN <- unlist(sapply(f, function(x) x$JPN))
Leto <- unlist(sapply(1:length(f), function(i) rep(2005+i, nrow(f[[i]]))))
```


1.6.0 and 1.7.0 missing on npm


Why does this succeed where the other fails? Perhaps there's really an issue with `elem_is_visible_with_wait`?


fix to #321 PR


Assuming **Intel** CPU.


Relevant and attainable feature.
Direct to the point description.


Clear system state description.
Testable steps.


WTF??????

You may do ads on www.flyspray.org, but not on the software itself and turning it into adware
after all the work of volunteers done this year to shake out bugs.

If you do not revert this from github master I just leave and am forced to go my own way!

I'm really pissed off. What a pity.


GUI scenarios described using high level actions.
Platform independent scenarios.


Failure and success scenarios complement each other, giving a better understanding of system and GUI overall behavior.


Hehe. Yes there is a TERMINOLOGY for it :p just to be pro LOLLL


Fair enough. Done.


@jordanmendler No, this is not how opensource software should be.

Why are you doing this? I will personally remove this if I must. Please explain.


This a copy of my second comment I am trying to make a branch


:-1: Adding this is out of scope of what this change should be, which is a refactoring without functional change.


simplificação das variáveis feita


Why is this bumped?


Če želiš pretvoriti imena v oznake, bo najlažje narediti faktor in prebrati ustrezne številske vrednosti:
```R
motorOznake <- as.numeric(factor(konstruktorji))
```
Oznake bodo privzeto dodeljene po abecednem vrstnem redu. Če bi želela določiti vrstni red, bi ga lahko določila s parametrom `levels` pri funkciji `factor` (podobno kot pri urejenih faktorjih).


105 is a Invalid number
#define V4L2_CID_INPUT_1                (V4L2_CID_BASE+100)
#define V4L2_CID_INPUT_2                (V4L2_CID_BASE+101)
#define V4L2_CID_INPUT_3                (V4L2_CID_BASE+102)
#define V4L2_CID_INPUT_4                (V4L2_CID_BASE+103)
#define V4L2_CID_INPUT_5                (V4L2_CID_BASE+104)


Great, thanks for fixing my late commits!


Why did you remove {}?

```
$ echo foo{bar,baz}
foobar foobaz
```


This Index shouldn't start with 100?
#define V4L2_CID_INPUT_1                (V4L2_CID_BASE+100)
#define V4L2_CID_INPUT_2                (V4L2_CID_BASE+101)
#define V4L2_CID_INPUT_3                (V4L2_CID_BASE+102)
#define V4L2_CID_INPUT_4                (V4L2_CID_BASE+103)
#define V4L2_CID_INPUT_5                (V4L2_CID_BASE+104)


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1871 of 1871 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)


dptest - could be considered as end to end test - or system test with all servers at running state.

so may be  stest or e2etest sounds better?


If using Mocha, you should be able to write this as
```ruby
FilesApi.stubs(:max_file_size).returns(5)
FilesApi.stubs(:max_app_size).returns(10)
```
It's working for me over in #3925 - see [this article](https://yarorb.wordpress.com/2007/11/26/mocks-and-stubs-in-ruby-on-rails-the-mocha-solution/) for more examples.


Argh.  Not again.


👌👀👌👀👌👀👌👀👌👀 good shit go౦ԁ sHit👌 thats ✔ some good👌👌shit right👌👌th 👌 ere👌👌👌 right✔there ✔✔if i do ƽaү so my selｆ 💯 i say so 💯 thats what im talking about right there right there (chorus: ʳᶦᵍʰᵗ ᵗʰᵉʳᵉ) mMMMMᎷМ💯 👌👌 👌НO0ОଠＯOOＯOОଠଠOoooᵒᵒᵒᵒᵒᵒᵒᵒᵒ👌 👌👌 👌 💯 👌 👀 👀 👀 👌👌Good shit


4967-SZRH-spm (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (802 of 802 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-20.5%25-green.svg)


No. You have terrible coding habits that would not be up to the standard that we have in the read-me. I know I made this.

I do not feel the need to argue over something as petty as this because the PR that you used for your argument is also dated march 12. You do not have the date of the original commit.

*insert quote about me asking for credit*

Oopsies, I figured we would learn to both look at the date something was created before calling somebody out falsely.


Then resetting these values should be unnecessary - Mocha stubs only last for the scope they are defined in.


Why a separate error, why not simply append to the error message?


> Not again

Sorry, did we previously discuss this?  Did I previously give you conflicting advice?


As the code is not it should actually be `gamma ~ Flat` we could make it `Bernoulli(0.5)` but we would just need to add this to the log posterior evaluation.


test is the standard npm keyword for running your tests (see https://docs.npmjs.com/cli/test), so we can't really rename that one. Also, most Node projects map test to running their unit tests.

A typical Node dev / test workflow is like this:
```
# Run the unit tests, note that you don't even need to say npm run test here as test
# is a builtin npm command
npm install
npm test
```

Some popular projects (see Express, request, graphql, etc) have introduced test-ci, test-cover, etc for other things (using - instead of _ as a separator), but they typically keep these commands short as developers are expected to type them all day long.

I could sympathize with the idea of making ```npm test``` run both the unit and integration tests as our integration tests run fast, and that would get us in the habit of running all the tests all the time, which is a good habit IMO. That would give us the following:

```
# Run all the tests
npm test

# Run the unit tests only
npm run unit
# or
npm run unit-only

# Run the integration tests
npm run integration
# or
npm run integration-only
```

as opposed to the current:
```
# Run the unit tests
npm test

# Run the integration tests
npm run itest
```

... which after writing all this, still looks simpler to me :)


So we will have two different ways of doing the exact same thing? Are these methods just to make the library look more like Rx.Net?


Is there any way to prevent it from getting copied to rootfs-complete? I don't want jwm_config in my puplet.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I think this duplication here is rather confusing. Better append the message to the previous one


Other than append vs separate error, looks good to me


I can reproduce it on two computers and also with default settings.


If we want prior of `Bernoulli(0.5)` we should make this
``logpdf(MvNormal(X * (beta0 .* gamma), 1.0), y) + sum([logpdf(Bernoulli(0.5),gamma[i]) for i in 1:p])``


Jobs tables should not be considered in ERDs.  Jobs tables start with a ~.  The regexp should not match jobs tables.


I debugged this, making sure it gets unmounted properly. The element is totally new each time. That is indeed a strange browser behavior.


Variáveis atualizadas e variável "MinAtual" adicionada.


After looking, I'm not even sure it would help out at all. I'd have to reimplement and even test to find out - and given that this is tested and works, it doesn't seem worth it. I'd be open to checking it out if you think there's enough value, though.


embarrassing typo is what caused all the problems


Codepot workshop ? :)


very strange. I also wondered if there isn't a focus property. autofocus seems to be something different.


Looks like testRegions() is testing the vertex version against itself. Is this what you intended to do?


I have just noticed that Clang 3.6 is not able to compile the new code:

```
In file included from plugin-menu.cc:20:
In file included from ./libaudqt.h:25:
In file included from ../libaudcore/objects.h:23:
../libaudcore/templates.h:170:60: error: chosen constructor is explicit in
      copy-initialization
        { return new (loc) T {std::forward<Args> (args) ...}; }
                                                           ^
../libaudcore/templates.h:180:46: note: in instantiation of function template
      specialization 'aud::construct_base<audqt::ItemData,
      true>::make<audqt::MenuItem>' requested here
        return construct_base<T, aggregate>::make (loc, std::forward<Arg...
                                             ^
../libaudcore/index.h:152:37: note: in instantiation of function template
      specialization 'aud::construct<audqt::ItemData>::make<audqt::MenuItem>'
      requested here
        return * aud::construct<T>::make (IndexBase::insert (-1, sizeof (T)),
                                    ^
plugin-menu.cc:70:29: note: in instantiation of function template specialization
      'Index<audqt::ItemData>::append<audqt::MenuItem>' requested here
    auto & item = items[id].append (MenuCommand ({name, icon}, func));
                            ^
../libaudcore/objects.h:62:24: note: constructor declared here
    explicit constexpr SmartPtr (T * ptr = nullptr) :
                       ^
plugin-menu.cc:33:23: note: in implicit initialization of field 'action' with
      omitted initializer
    SmartPtr<QAction> action;
                      ^
1 error generated.
Failed to compile plugin-menu.cc (lib)!
```


> There are many implementation of the pcap interface

Yes, pcap has the notion of modules; currently, it only supports modules compiled into libpcap/WinPcap, but at some point we should probably support dynamically-loadable modules (as long as there aren't security issues; we don't want code running with elevated privileges loading untrusted code).

There's the "main module", which uses the native packet capture mechanism on UN\*Xes (pcap-bpf,c, pcap-linux.c, pcap-pf.c, pcap-snoop.c, pcap-dlpi.c, etc.) and uses the WinPcap driver and packet.dll on Windows (pcap-win32.c).

Then there are additional modules, which use whatever mechanism they use.

For libpcap, some people wanted to be able to build it with one or more of those modules as the *only* module, with the "main module" omitted.  The modules we currently support for that are those for DAG cards, Septel cards, and CSPI (formerly Myricom) cards; the modules in question work only on a limited set of UN\*Xes (Linux and, for some, FreeBSD), so those aren't options for WinPcap.

Packet.dll has its *own* support for multiple mechanisms - the WinPcap NPF driver, AirPcap, and some other mechanisms.  That's how AirPcap support currently works in WinPcap - packet.dll handles it.

So:

BPF (packet-bpf.c) and packet.dll (pcap-win32.c) are both "main modules", so it makes no sense to have them both in pcap - either you're running on a system with BPF (\*BSD, OS X, AIX, Solaris 11) or you're running on Windows.  (There's also BPF the *filtering mechanism*, which is supported by BPF the packet capturing mechanism as well as by Linux's sockets, Tru64 UNIX's packetfilter, and WinPcap's NPF driver, but that's different.)

The same applies to pcap-linux.c, pcap-dlpi.c, pcap-snoop.c, etc. - they're main modules so there is no reason to have one of them *and* pcap-win32. in the same pcap or, for that matter, to have two of them *other* than pcap-win32.c in the same pcap (for example, you wouldn't have pcap-linux.c and pcap-bpf.c in the same pcap).

CSPI [seems to indicate that their sniffer cards support OS X and Windows as well as Linux](http://www.cspi.com/en/products/packet-capture), so it's conceivable that pcap-snf.c could be included in WinPcap.  It might require some changes, as the SNF API might have some differences between Windows and UN\*X, but, if we have a version of pcap-snf.c that compiles on Windows with CSPI's software, we could arrange to have it and pcap-win32.c in the same WinPcap - as well as, if somebody cares, building an SNF-only WinPcap.  That would require compiling WinPcap from source, on a machine with CSPI's libraries installed.

AirPcap is currently handled by packet32.dll, so you *can* have it in WinPcap along with packet32.dll.  Eventually, I'd like to make AirPcap a module like the others, so that you don't go through packet32.dll to access it, in which case WinPcap could be built with AirPcap support as well as NPF support, and that's how the default binary distribution would be built (along with TurboCap support).


Yes, you're right, I'll change this


@afilipovich Hi Aleh.  We ( @mbusha and I) found a bug in your google safe browsing library on github (gglsbl) related to an extreme edge case.  In firefox, the URL ‘http://09' resolves to www.09.com (although it does not resolve correctly on Chrome).  However, if I try to lookup ‘http://09' in gglsbl using the lookup_url method, it crashes on line 345 in protocol.py with the error ‘gaierror: [Errno -2] Name or service not known’.  This is being caused by the assumption that ’09’ is an IPv4 address, which it is not.

There are probably a number of solutions to this issue, but here’s a patch that works for us:

diff --git a/gglsbl/protocol.py b/gglsbl/protocol.py
index 7a35d7d..a387488 100644
--- a/gglsbl/protocol.py
+++ b/gglsbl/protocol.py
@@ -342,9 +342,15 @@ class URL(object):
        host = host.strip('.')
        host = re.sub(r'\.+', '.', host).lower()
        if host.isdigit():
-            host = socket.gethostbyname(host)
+           try:
+                host = socket.gethostbyname(host)
+           except:
+               pass
        if host.startswith('0x') and '.' not in host:
-            host = socket.gethostbyname(host)
+           try:
+                host = socket.gethostbyname(host)
+           except:
+               pass
        if path == '':
            path = '/'
        quoted_path = quote(path)

Hope this helps.


ACK


Alphabetical


Why did you remove these?


@darkalor Huh? There was a bug?


I was thinking instead we'd put the appId into DeleteStats for the grain, and that way we can count the number of demo grains created of each app.

If we want to track which app an appdemo user started off with, I think the way we want to do that is by recording it at the time the appdemo user is created -- we have the information at that time. Otherwise in the case where an appdemo user uses multiple apps you're going to end up recording an arbitrary one here, not necessarily the one they started with.


The -dev packages are not Depends, they should only be Build-Depends.


wouldn't say the exact same thing, but more specific things with a simpler api. much like how we have `observable.subscribe(on: Event<E>)` and `observable.subscribeNext(element: E)`


This doesn't seem to work, possibly because the tests in `shared/test` do not use Mocha. Here is the error:
`NoMethodError: undefined method `stubs' for FilesApi:Class`


This comment applies to our choice of lastActive, which you moved up, so the comment should move too.


small issue with coding style... the "else" should be in the same line above.


Accepted by hashicorp.


Sounds good. I'm not sure why I made this change


note (d-1)/4; will be rounded down since it's using integers...
so if x=(d-1)/4
d=(1 to 4) x=0
d=(5 to 8) x=1
etc..

I don't think that is your intention...


Model architecture is better now! But code is still hard to read/understand. I see bad usages of backbone router, it is really helpful to restore #hash data! (Lets say I run search by value, I would like to bookmark it in my browser), router must be responsible for state recovery and drawing all views with initialized data. Ex: I bookmarked page with sorted list by numbers or by letters. When I click on this bookmarked page later, I wish to have order in which I bookmarked. I see a few typical renders in routers, it is not enough. Please investigate Routers deeper.
Also render method! I see that you start using backbone the same way I did :) lol :) But it is not correct usage! render methods should be invoked after validation, validation of a model should provide error instance with error messages. Once validation is done, render method of a view should be called again. So render is responsible for initialized render and subsequent renders during validation fails. I wish you to use Backbone.Model validate method for your models.
Don't panic! There is a lot of things I need to investigate in your implementation, I feel that I miss something, so it is just an impression of a first glance during 5 mins. Don't overload yourself with localstorage and etc. Baby steps first, then we will see :)


??? 👌👀👌👀👌👀👌👀👌👀 good shit go౦ԁ sHit👌 thats ✔ some good👌👌shit right👌👌th 👌 ere👌👌👌 right✔there ✔✔if i do ƽaү so my selｆ 💯 i say so 💯 thats what im talking about right there right there (chorus: ʳᶦᵍʰᵗ ᵗʰᵉʳᵉ) mMMMMᎷМ💯 👌👌 👌НO0ОଠＯOOＯOОଠଠOoooᵒᵒᵒᵒᵒᵒᵒᵒᵒ👌 👌👌 👌 💯 👌 👀 👀 👀 👌👌Good shit


This is in order to check if the flag `GPIO_OUT` is set in `flags`, the double-negation will return us `1` or `0` instead of a decimal value. In the `GPIO_OUT` case is useless because this flag uses the first bit. I let this way in order to maintain consistency but I can remove it if you think it would be better


Whoops! should be fixed now.


This is my second attempt to o a repo/branch/commit


Should `data` be `result`?


Lets not mix up XFF with remoteAddress, as the two are fairly different. Instead, how about setting a header like `x-client-ip` ?


Hast du auf Handy probiert? bei mir Funktioniert das auf den Handy nicht bzw schaut doof aus da er nicht rüber slidet


necesito las coordenadas del mapa, para uan cosilla k estoy probando como intro
he intentado buscarlo con el google earth pero no hay manera el único río k veo no localizo la zona k escogiste.
a ver si me paso por el ts k llevo unos días algo pachucho
1salu2


Sweet puns :+1:  Very descriptive and a good overview of the week!


For posterity:

Just tried this branch out and it resolved the H15 errors we were seeing on Heroku, closing the socket if the client doesn't reply within some time frame makes sense as well.

Would love to see this in the next version of phoenix, thanks again for taking the time to cut the branch @chrismccord


I feel ignored here


only 1.6.0


This can potentially be simplified if we set an `x-rb-client-ip` header on the way in.


This commit is supposed to be on a branch but branch is not an option on split


It has to do with the object type and how these object types deal with state changes


You guys need different avatars than at work.
![](http://i.imgur.com/IW7Zf8g.gif)


@krousey, just out of curious, what good does the {} do?


derrrrrp


Does `onError(myError)` do anything at all different than `on(.Error(myError))`? It looks to me that the new command just removes some parens and a dot (which I'd call a good thing.) Maybe the latter should be deprecated, or does it provide functionality that the former doesn't?


Typo: "UnStage"


Looks like it's there to me now. Might be a stale cache?


TeamCity pull requests :: netty [Build 207](https://garage.netty.io/teamcity/viewLog.html?buildId=1382&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5600, ignored: 21 Build time: 00:35:04


At least we have different usernames


Woah! every time I refresh I alternately get 1.6.0 and 1.7.0! Weird!


Likewise, "UnStage"


ObjexxFCL_Update (mjwitte) - x86_64-MacOS-10.9-clang: OK (1866 of 1866 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)


Thanks @tsegismont !


Maybe we could actually make this a more generic header match instead. Something like

```yaml
security:
  - x-header-match:
      x-rb-client-ip: internal
```


:) please let me now which lake. i need some radios.


As a user, I would personally prefer that this be `buildTxt`, with a default of false. I haven't seen this file be used anywhere really. The [first result](https://www.google.com/search?q=rjs+build.txt) when searching about it is how to remove it. I feel it should be opt-in. Among other things, it surprised me when it suddenly showed up when switching from a single module build using `name` to multiple bundles with `modules`.


this is causing problems


Looks reasonable.


@jhiesey I see it [here](https://www.npmjs.com/package/stream-http)


hi


Thanks for the inspiration @islemaster , it hadn't occurred to me to use the test framework instead of using language features directly. Looking through the docs for Minitest I found the `stub` method I was looking for.


At one point it ran a task. Maybe it doesn't anymore?


saw approval from hmmr
at https://github.com/basho/riak_core/commit/8a2d55d5ff47fd7c4d82a05cfdffbbeaf505b5e6


merging basho/riak_core/type_ready_checks_ddl = 8a2d55d5 into borshop-integration-779-type_ready_checks_ddl


basho/riak_core/type_ready_checks_ddl = 8a2d55d5 merged ok, testing candidate = 2162045e


Twitter


Добро пожаловать в модуль experiments ;)


I believe there may well be.


Are you kidding me?) Your change affects all classes `.mdl-button--fab.mdl-button--colored` in the page.


Since this will already be a breaking change, I think I might like it if we also switched from `data` to `body`. Body seems to be the more commonly used term (shelf uses it, http package uses it). Thoughts @trentgrover-wf @maxwellpeterson-wf @dustinlessard-wf ?


If you wouldn't have changed all the white spaces, would have written a meaningful long description and split this into atomic commits, then I would be interested in having the missing getline() replacement as well as Android support in the mainline source.
Think about it. If this sounds appealing, then rework this stuff and send us a pull request.


@lyndonbartlett88 DWMR. More fine work. Thank you.


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1871 of 1872 tests passed)

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


Is there a good place to put that snippet in the commit message? Could we show it off in the `strings` notebook, or something? We could also show how to print all the step objects for a given model. (That can sometimes be useful for diffing if it's a hassle to get both models in one script, too. Then you can just print out each model to a different file and diff the files.)


I'm using what Boost was available from Debian which I believe is 1.55 or 1.57, if I need a different version then please let me know. If you don't want to populate this thread with a conversation I'm available at scott@twedev.com


Yeah, that's where I sometimes saw each version. I think I broke npm by publishing 1.6.0 and 1.7.0 in backwards order.

Anyway I tried to publish 1.7.1 with no changes, and now I alternately get 1.6.0 and 1.7.1. Hopefully it's just different versions cached on different machines serving npmjs.org and will stabilize to 1.7.1 soon.


Yeah I was going to release it last night but was too tired. Anyway I'll move it into a branch until the new version is ready.


Fine by me.


Alternatively, we could consider generating a systemd `.mount` unit for this, and then systemd would take care of maintaining its state.


Well, the value stored inside the unit was always just `unsigned`, not `unit32_t`, hence all the misalignment now. I'm tempted to just turn it back to `unsigned`.


I like it, but not as an icon for the app. I prefer those material icons a bit more flat, you know, without those soft shades. But if I may I night see a usage of this for the login screen.


HomeWork


You made some mistakes there buddy.


I understand now, thanks.


Is it `new_settings.c_lflag &= ~(ECHO | ICANON);` or `tcsetattr(0, TCSANOW, &new_settings);`?


lol (commit message)


In this case try to find a solution, the thing is that the script must have
a chance to execute, it can't execute here.

2015-09-15 23:08 GMT+02:00 scrawl <notifications@github.com>:

> What about NPCs that are (legitimately) dead at the start of the game?
> With this change, I think you'll see them fall down when you enter the
> cell, rather than already lying dead to begin with, as they should.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zelurker/openmw/commit/2e8301831c3b6a7e3725427d539c252f4dca223d#commitcomment-13247927>
> .
>


Sure, I guess that could work. I'll test it tomorrow.


This brings up a good discussion point. To me, it seems like the expected behavior with headers is that it should not completely override headers that may have been set previously, but instead be merged with those headers. It's definitely a corner case...

```dart
WRequest request = new WRequest()
  ..headers = {'content-type': 'application/json', 'x-xsrf-token': csrfToken}
  ..uri = ...;

request.get(headers: {'content-type': 'text/plain', 'x-custom': 'value'});
```

Which set of headers should the above request produce:

**Merged**
```
content-type: 'text/plain';
x-xsrf-token: csrfToken;
x-custom: value;
```

**One or the other**
```
content-type: 'text/plain';
x-custom: value;
```


Set `HPDF_REAL_LEN=15` to allocate 16-byte buffer in `HPDF_FToA()`.


Looks good. Are we ready for 1.0?


To confirm my understatement, it flushes the buffer to the standard output (screen)?


Should alphabetize all of these optional params


FYI, you don't need the return unless it actually returns something real. Don't bother changing at this point, though.


Probably, but I'd like to have someone else test it at least first.


What about becoming the official Polish translator for scanmem/GameConqueror?

If merging this commit with the next commit (the rename), improving the patch descriptions and sending us the result as a pull request, then there would be a good chance for this being accepted into the mainline scanmem. Think about it!


nice one, thanks


I found this comment confusing. It sounds like we will add Status to all versions, but we actually only add it to v1, which is for backward compatibility purpose.

Maybe we should explicitly say adding Status here is for backward compatibility.


I could see it being fairly common that tests would want to make an request expectation without caring about the headers, which would most intuitively be done by calling `expect()` without providing a value for the optional headers param. In that case, this headers check should be skipped if `e.headers` is null. Thoughts on that usage?


Could you elaborate a bit about why this is needed?
Git commits should always contain a meaningful long description. I'd like to know if this could be relevant for the mainline. Thanks!


Broke app


As I mentioned in the original API 4.0.0 PR, where you already suggested this change, I think this is not correct. Values < PVR_TIMER_NO_EPG_UID are not allowed. Kodi core will not create values less than PVR_TIMER_NO_EPG_UID. Thus, you only need to check for > PVR_TIMER_NO_EPG_UID.


Perhaps this is a good demo of why bitwise operators are confusing — you've replaced the bitwise operators with a non-bitwise comparison, but the logic is inverted from what was there before. :)

We need to change `!==` to `===` now. :)


Note from chat with @ptaylor:

No need for `innerSub` here, since the innerSubscriber is also technically the subscription in our implementation, we can just use that.


If you publish them in reverse order, then the special tag "latest" will point at the older version (the last to be packaged), unless you used the `--tag` option to give it a different tag.

npm uses the latest tag to determine what version to install if you didn't provide any version or tags yourself. At any time, you can update what version tags point at, for example:

```
npm dist-tag add stream-http@1.7.0 latest
```


C# needs boolean values, so I'm guess that I have to compare `cx` as `cx > 0`?

[aka] What is `cx && cy && cx` supposed to give? I know that the statement is true when an int is higher than 0 though.


Well, seems fine to do this on Plone 4.3 as well. I have backported https://github.com/plone/plone.app.controlpanel/commit/9c24fee660f8fa9a764f798912085f413efdc145 to p.a.controlpanel branch 2.2.x and updated the tests.


@terinjokes Thanks! good to know


Looks good to me.  +1


@borshop: retry


Possible alternative:
```dart
Iterable matchingExpectations = _expectations.where((e) {
  bool methodMatches = e.method == request.method;
  bool uriMatches = e.uri == request.uri;
  bool headersMatch;
  if (e.headers == null) {
    // Ignore headers check if expectation didn't specify.
    headersMatch = true;
  } else if (e.headers.isEmpty) {
    headersMatch = request.headers.isEmpty;
  } else {
    headersMatch = mapEquality.equals(e.headers, request.headers);
  }
  return methodMatches && uriMatches && headeresMatch;
});
```


Shouldn't it be the other way around here?


When I tried to run rake after this change, things were really broken.  Did anyone else have this problem?


I get a compilation error (Can't cast int to byte*) on this since in C# we're re-assigning a variable with the `|=` bit-wise assignment.
So yeah I'm just wondering what variable is affected here.


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


But "unsigned" is incorrect, since it's documented to be 32bit wide...


Why did you delete v3.3.3? IMO, you should keep 3.3.3 changelog and just add 3.3.4 changelog.


poderia remover  o .idea do projeto  (gitignore)? atrapalha o diff,  e não commitar direto em master ? ou pelo menos gerar uma tag da versão corrente do rubygems ?


Does `!!` affects recursive parentheses? If not then lol wat.

Although I'll have to check for an implementation or some sort in C# for calloc. (Or make a global variable if I can)


@dhowden this won't work for all the examples in https://github.com/advancedlogic/GoOse/issues/7 though. The two key ones are schema.org and the one you've done already. We probably want a cascading priority


So nics.*.gateways only work for IPv4 gateway? How can we set IPv6 gateways?


TeamCity pull requests :: netty [Build 209](https://garage.netty.io/teamcity/viewLog.html?buildId=1384&buildTypeId=pulls_netty&guest=1) is now running


Your schema.rb changes are a bit disconcerting.... Can you revert those?


> Flex for Windows produces same output.

I.e., WinFlexBison produces output that doesn't compile on Windows without

    #define isatty _isatty
    #define fileno _fileno

That's bogus.

OK, for now, we add the #defines.  If there isn't already a bug filed against WinFlexBison, I'll file one.

If we end up *requiring* a sufficiently recent version of Flex for libpcap, we can just add a "non-interactive lexical analyzer" directive to keep it from generating the code that uses those functions to see if it's reading from a tty.


Hopefully `sizeof()` will work for this case.


Not that this matters too much functionally wrt to this change, but it's redundant to have the touch as part of the shell task, if you already touch the file in the file task.  I suggest removing the touch command as part of the shell task.


@jhiesey Don't forget to commit to prevent conflicts


Oops.. forgot the long description.

On Ubuntu 14.10 (mate-desktop) gameconqueror fails to run/gives some xauthority errors.
Reproduced this on two separate systems, both Ubuntu 14.10/mate.
The above change allows gameconqueror to run on these systems.

If I can get access to either machine I'll reproduce the error and update this.


Thing here is that int* isn't an array, but it's being used as one, which is (obviously) odd to me.

Or I misunderstood something here.


102867486-FixRectangleShapeTest-5173 (mjwitte) - i386-Windows-7-VisualStudio-12: Tests Failed

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


I'm afraid the changes will conflict with the work that I started on the SourceSet branch, but we will see...


Just uploaded test build 3415.


Looks pretty good. The only thing that doesn't seem to work is in the sample file. The 'addData' button doesn't add data on my machine. Chrome gives me an error "Cannot read property 'format' of undefined on lime-time-scale.html:147"


I think this should be read in the context of this being v1/register.go. Other versions would need to have similar lines.


Ah yeah.  Kinda forgot about those buttons there. haha


I've updated that base URL. You can try giving it another shot.


You just witnessed the simplistic beauty of arrow functions :D


I would not include a space after comma in the type signature.


The Ctrl-K hotkey description under Window->Dynamic hotkeys could perhaps be updated?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 630](https://teamcity.labkey.org/viewLog.html?buildId=272317&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 31, passed: 9 Build time: 00:24:06
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

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


Yes, you're right.


:+1: dummy column idea!


Thanks! Then also rebasing this to latest upstream/master and sending us a pull request makes sense. Once in the mainline you don't have to maintain this change yourself any more.


'itest'  and 'integration' ,on its own, sound ambiguous.  How about having the following:

```
# Run the unit tests
npm test

# Run the integration tests
npm run test-integration
```


TeamCity pull requests :: netty [Build 208](https://garage.netty.io/teamcity/viewLog.html?buildId=1383&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5600, ignored: 21; compilation error: Compiler (new) Build time: 01:11:20


TeamCity pull requests :: netty [Build 210](https://garage.netty.io/teamcity/viewLog.html?buildId=1385&buildTypeId=pulls_netty&guest=1) is now running


@xhaggi any comments, for instance regarding the naming?


Hmm, I think I prefer a more Unixy model separating code and state (on Linux systems, the oauth_reverse_proxy module will likely be in /usr/local/lib or the like while configuration data is better location in /var/state or something).


The query does not normally return deleted objects. Google cloud storage does not guarantee strong consistency between DELETE and LIST operations. So it could happen that a LIST operation returns an object that was deleted right before.This check is to filter out such objects, in case that were to happen (I have never experienced this in practice, but better safe than sorry)


So is something like

    #if defined(_MSC_VER)
    #  if _MSC_VER < 1600
    // MSVC prior to MSVC++ 10.0 (Visual Studio 2010)
    #    define snprintf  _snprintf
    #    define vsnprintf _vsnprintf
    #  else
    // MSVC++ 10.0 (Visual Studio 2010) and later
    #    define strdup    _strdup
    #    define isatty    _isatty
    #    define fileno    _fileno
    #  endif
    #endif

sufficient?  I.e., is it the case that:

* prior to VS 2010, the headers define `strdup`, `isatty`, and `fileno`, and there are `_snprintf()` and `_vsnprintf()` functions, but no #defines to map `snprintf` to `_snprintf` and `vsnprintf` to `_vsnprintf`;
* in VS 2010 and later, the headers *don't* define `strdup`, `isatty`, and `fileno`, but *do* define `snprintf` and `vsnprintf`?

I.e., do VS 2010 and later

* define `snprinf()` and `vsnprintf()` because they're trying to support part of C99, and C99 added them;
* don't define `strdup`, `isatty`, and `fileno`, because they're trying to do a better job of (at least by default) not polluting the namespace with POSIXisms?


Ah, sounds good. Could you add a comment describing this in the code so we have it for posterity.


Remove all the "Flag" prefixes from the method names, please. "Is" in combination with boolean return is sufficient and used all over the place in Kodi code.


Maybe you and @zjx20 work together, provide really cool getline replacement and Android support and then there is a good chance that we can include this into the mainline as well.


@smo-key Don't commit directly to staging.


the `on(event: Event<E>)` is a useful api to expose to consolidate logic that might send something more dynamic than an explicit next element or explicit next error, the more specific `onNext` & `onError` apis make it easier for when you do know specifically what type you want to send


Good call, I will fix this


Because if `properties[:ruby_default]` is `false` ie: it is a boolean, that guard will never be true and the `hsh[attribute] ...` will never be called.


Are these really status flags? I don't think so.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1871 of 1871 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)


@jonathaningram one quick thing i see here, meta have an array of vals, not a single val (to support delimited, etc)


@Oksisane Normally, yes. For comments, I don't see the issue. If we were REALLY going to be *strict* with this issue, we would use a shared `feature-comments` branch.


Why are these methods not inline?


102966118-HeatRecoveryBypass-4568 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1866 of 1869 tests passed)

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


@mish15 I'll try an example with array of vals too


Don't trust the npm website to show the latest version since they use
caching. You can use "npm show package-name" or "npm show package-name
versions" instead.
On Tue, Sep 15, 2015 at 3:12 PM Bogdan Chadkin <notifications@github.com>
wrote:

> @jhiesey <https://github.com/jhiesey> Don't forget to commit to prevent
> conflicts
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhiesey/stream-http/commit/43d1322dd619c20d249862f99cf73420c3cdd7ed#commitcomment-13248979>
> .
>


I deleted a comment I left in this function earlier because I kept reading it over and over and realised my comment was wrong. However, something just smells odd in here. I may be way off base, but it seems strange we're looping over list of keys, copying them to a new location, then (because of where the append is happening), deleting all but the one copied in the final iteration, then we're iterating the original list of keys and attempting to delete all those objects. Finally after doing all that, we do one final copy of sourcePath to destPath.

There's probably something I'm just not understanding... this just really stuck out to me.


Again, not a huge deal, but `destroyPersistentStoreAtURL` does (a faster version of) `performBlockAndWait` on its own. If you lift the `#available` outside, it's more clear what the legacy path is.


Add a message "pager is null"


Any chance you'll be able to go back to using the official jms/serializer 1.0.0 ... we are looking into use jms/serializer in an application but would hate to be stuck on this forked version here.


Use the https://msdn.microsoft.com/en-us/library/dd381412(v=vs.108).aspx here. Include the image.


This doesn't seem to be valid C++.  Some older compilers permit it, but GCC 4.6.4 and 4.9.3 both reject it with:

```
error: extra qualification ‘Outer::’ on member ‘foo’ [-fpermissive]
```

And clang 3.5.2 says:

```
error: extra qualification on member 'foo'
```


ok - Nino and I had discussed this, and agreed that it would be better, but you're correct.


Change of interface.


The copies are only deleted if a copy operation fails. In that case it tries to cleanup by removing the copies before exiting the function. The "dest" of the last attempt is not deleted, because its Copy operation failed, so it should not be there.


There is no provider state. ERROR is an internal state that is mapped to IDLE in the public API to maintain compatibility with JW 6 which did not have a state of ERROR.


I know about how it has to be done according to the SOA (but the service layer can be done differently, not only as in UserManager of FOSUserBundle). The only problem: we have a zend1 framework at work and the latest experience with symfony2 I had was a half-year ago. So I must have forgotten some symfony tips&tricks during that half of the year. I always keep in mind, that other developers will read my code in future when it's a production code and a real teamwork, not a recruitment task which is usually used to weed out total dumbasses before the technical interview.

It's the third evening I spent to write a code for a people with whom I don't even got acquainted. There was no communication about the project you're looking developer for, conditions, team, development methodology. Absolutely nothing. Only "please do this task for us to pass to the interview" and 3 evenings spent on it. I'm not getting paid for these evenings and ofc I want to finish this job as quickly as possible.

P.S. I've done everything you wanted. See the next commit. That's all. What should we do next? Is this enough?


ObjexxFCL_Update (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1865 of 1865 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)


#Wow

This is huge code how did you coded this much


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2308 of 2344 tests passed)

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


Go with the original - Model objects are the parts of the application that implement the logic for the application's data domain. Often, model objects retrieve and store model state in a database.


Thanks


By listening to the 'asq-role-changed', elements can do extra things when role has been changed.


I have tried various ways to simulate disabled localStorage in testing, but no dice (teehee). So far the only way I know is too manually turn off on browser. Please advise if you know any tricks here.


Hello


Wrap this in `unmodifiableFuture()`


unless you're recommending global installs of uglify (ugh) or adding node_modules/.bin/ to $PATH, calling node_modules/.bin/uglifyjs is the better practice here. Same goes for browserify but that isn't in package.json even.


I also did not find any non-abusive way to test the unhandled exceptions here. Please advise if you know ways to test it well.


I love a good breakfast


This is a test repo


how did you do that


This should be a rocket-chip parameter that defaults to false


i am now just clicking random buttons


This line is replaced because it seems like a risk for stack overflow.


more typically in MVC apps is route data, not qs. I would stick with the original.


I added a travis build and that use case causes a crash to the test program in travis. That may be arch specific as I cannot reproduce it locally. Please check.

https://travis-ci.org/OpenSC/engine_pkcs11/jobs/80534079


It seems that in Polymer the handlers of observers have their arguments in different order.
See here https://goo.gl/u10Ane


'on ie'
:sob:
can't we just give them every reason to NOT USE ie?


I'm mostly concerned that this version is also being bumped in upstream and if we keep bumping it we'll clash or end up in a world of confusion.
E.g. upstream is at 4.0.2 here https://github.com/cloudfoundry/vcap-common/blob/master/lib/cf/version.rb

I wish this lib/stackato/* code had never been added to vcap-common and if possible I'd like to see this code moved out of here as part of the refactoring to get back closer to vanilla vcap-common. Are these functions used in multiple places?


What exception is actually thrown here? This should be one of `RuntimeException`, 'IOException`, 'IOException | RuntimeException`


Pretty sure there are still some IE versions out there that will throw an error on console.error.


for understand descriptive/figure https://tr.pinterest.com/hyperplane/


Is that possible/easy to keep the `add_to_*` functions for back-compat (and make them deprecated)?


Just realized this line is not right. Fix coming up.


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


All auto changes. Not needed, but it's more work to exclude all that is not needed. The important part is that it is stable.


@jonboulle Should we drop the old indexes? Or it's dropped when you drop the table?


Hey, thanks so much for your updates.

This is just my thought...If the signingin is not configured manually, the application will still crash.

Moving google-services.json into the /app folder might not be sufficient enough, since signingConfigs  won't be added into [gradle file](https://github.com/googlesamples/google-services/blob/master/android/signin/app/build.gradle) automatically.

Thanks for your time and patience.


@louist Good call. I'd test, but I'm not familiar at all with the inner workings of Kiwi.


If this exception can be `IOException`, this should check `!(e instanceof QueryAbortedException)`


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

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


I assumed they are dropped with the table.... I suppose I should confirm.


LGTM!


Keep in mind that your tests are really going to be run in phantomjs... might be something different there.


Fixed.


Das muss weg


TeamCity pull requests :: netty [Build 209](https://garage.netty.io/teamcity/viewLog.html?buildId=1384&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5600, ignored: 21; compilation error: Compiler Build time: 00:35:23


Chöntsch du ned de glich formatter benutze? Isch recht müehsam usezfinde, was sich änderet so


These booleans somehow feel not "good" to me.

One problem I see is, because they are optional, you cannot build any functionality on top of "false", because this value is ambiguous. You never know whether false means "not supported" or really "false" (e.g. not recording). I cannot implement a "Is not currently recording" icon (!bRecordingInProgress), for instance.

OTOH, bExpiresNever and bExpiresSoon are both related to expiration of the recording and are mutual exclusive. For me it feels more natural to have an enum for the differnt mutual exclusive expiration states. Putting a value UNKNOWN = 0 into this enum, solves the ambiguity problem I just described, btw.

Unfortunately, you cannot put bRecordingInProgress into this enum, as it is not really related to expiration. Both bRecordingInProgress and bNeverExpires can be (logically) true at the same time!

Last, "soon" is to weak to be of any real end user value. Consider an environment with multiple PVR clients, all defining other concrete values for "soon". Now consider, that you're driving a "expires soon" icon from that new boolean in the window with the list of recordings of all clients. Lets farther on assume, all recordings from all clients at one certain point in time show the "expires soon" icon, but some signal "12 hours" with this, others "30 days". Would you understand that you have much time for some of the entries, but almost no time for others, while all do show exactly the same icon? Would this be of any real value for you? To answer your question, yes, something concrete like bExpiresWithinXYZhrs would be okay for me.


The buffer should stay an INT because of the Air Conditioner issues with long code strings.  They are many that are above the 255 limit of unsigned char.


4967-SZRH-spm (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1338 of 1338 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.23%25-green.svg)


Yes, I agree, doing it that way would be more ideal, however TARGET_OS_IOS didn't exist before the iOS9.1/tvOS9 SDKs.

Supporting watchOS in InterfaCSS would be nice of course, but it would be even nicer if watchOS supported UIKit ;)


Why not map them blindly to `Long`?

```
literalArguments.stream()
    .map(literal -> literal instanceof Number? ((Number) literal).longValue() : literal)
    ...
```


Thank you, @fippo! I installed browserify with "npm install browserify" and then invoked "make BROWSERIFY=./node_modules/.bin/browserify" before. Now that uglifyjs was added, I wanted to have uglifyjs in ./node_modules/.bin/ as well so that's why I added a make variable for it as well. (I don't really known anything about these things though so I accept I may be totally off-course here.)


Het din jshint bem build kein fehler gä vo wege unused parameter, wo das vorher testet hesch vorem commite?


That works for me too.

Each developer eventually finds the workflow that works best for her/him. Usually, after doing ```npm test``` repeatedly to run the unit tests for the modules I'm working on, I tend to do a ```git clean -df``` and a full build from scratch like the Travis build anyway, as I want to run the whole npm install, babel gen, unit tests, integration tests etc again to make sure I'm not missing anything.


BTW, I don't think we want to model the int arguments for types as "literal" arguments. The latter are a hack to model row types and we should get rid of them.


Das isch nice, denn hemmer es binding bis id direktive ine :+1:


unused code


@wfbuhl The lines you commented out conflict with some of the changes @Nigusse was making to the sizing of low temperature radiant systems. Can the two of you figure out what the lines should be?


Thanks for the fast fix. :+1:


As per comments at https://github.com/cloudfoundry-incubator/cf-abacus/commit/90d4a6906033556a3824602aa7df1c08afbec180#commitcomment-13249105

we will update the code to have the following

```
     # Run all integration tests
     "test-integration": "export THROTTLE=1 && foreach itest node_modules/:name npm run itest -- && npm start && sleep 5s && npm run demo && npm run perf && npm stop",
    "build-ci": "npm run build && npm run test-integration"
```


I looked there too. It is possible to disable but result is per session, so not a good option either.


These functions are used in dea, cloud_controller, and fence/fence.  And there's no overlap between our logyard code and the vcap code -- the cloudevents code does load vcap/logging.rb and vcap/logging/logger.rb. In either case, +1 on moving the stackato-specific common code to our own repository.


Do you mean wait sometimes returns after 100ms no matter what value timeout is?


 You can unit-test the application without having to run the controllers in an ASP.NET process, which makes unit testing fast and flexible.

That's not correct. You unit test controller methods. MVC was designed to enable testability without requiring dependencies on a Web server (IIS), on a database, or on external classes.


@shivamprakash, where is ``log`` defined? Seems like this would raise an Exception


@N1kw re: Looks like testRegions() is testing the vertex version against itself. Is this what you intended to do?

Yes, this last piece is supposed to ensure FK consistency within vertex


Merged, thank you.


No, we don't register it in api/experimental/v1/register.go. It's only registered in api/v1/register.go and api/register.go. So It worth leaving a note here :)


According to developer.mozilla.org, log and error are both supported starting IE8.
https://developer.mozilla.org/en-US/docs/Web/API/Console/error
https://developer.mozilla.org/en-US/docs/Web/API/Console/log

It doesn't seem to matter as long as I am not using string substitution.


Hmmm - 6 and 1/2 dozen I guess here.


I would be very interested in doing this.  I have been busy on my own project that is not ready to open source yet.  If there is interest in Android and getline replacement, then I am glad to do it.
On that note I already have the build files for Android.  I just never completed the bash build script to go along with it.  I will look at this again this week and make sure to put the source in development on my repository.


I believe `${WORKDIR}` could end up using spaces and needs quoting, i.e. `"${WORKDIR}"`.


This is equivalent to `$*`. The curly braces are just a habit I got into when using bash variables as it is less ambiguous.


New additions should got to testing, first.


Also the confusing nature of them, which may be somewhat relevant to a project for public consumption. I like em for one liners, but I was confused here at first as well.

I suppose we'll all get used to it though.


JsLint told me to do it. :p


leave this line in, it gives the header a background to contrast the text if the image loads slowly


Yeah - I figured when I saw it... looks like a good move.


Got it, thanks!


revert this change, the margin is set by the container


OK, yes, perhaps I'm confused.


use `em`s or `rem`s here


not sure why this was changed, or why this selector is that way


This should be set before the loop.

It is also not necessary to have the same code block below again.


remove this `}` as it's preventing the other `.header-*` classes from inheriting


I'm proposing to roll back this change with pull request #271. In my testing, `xpcall()` is returning the wrong arguments, so if you do something like:

```lua
local mod = require("myModule")
mod.someFunctionCall()
```

...it fails because mod isn't the actual module. Also, the way you've formatted the call, `base(modname)` executes and the result is passed to `xpcall()`. If there is an error in the module script, it will happen before `xpcall()` even gets called.


this is npm lore -- only passed down verbally :-)
global installs or in general installations of browserify/uglify outside node_modules/.bin tend to cause version conflicts at some point. And debugging those is not fun.


remove this line, height is set by text margins


"creating tables with data"


Fixes #1, #6, #8


remove this hover effect, it's automatically set by bootstrap if the primary brand was changed


The commit message could use clean-up. There are many things in that got squashed away.


this should be changed back to `.kickback` for proper color


This should probably be something like
```java
throw new PrestoException(INTERNAL_ERROR, "ConnectorMetadata beginCreateTable() is implemented but commitCreateTable() is not");


I don't agree. Imagine I updated Lodash. Would that be considered a breaking change? No because it's an implementation detail. Likewise, updating DnD Core is an implementation detail. In fact even DnD Core's API hasn't changed—just its implementation.

The fact that there was a bug has nothing to do with semver—it's just what happens when you write software. :-) Hopefully the bug is fixed now.


This looks like the start of clustered isvcs - did you mean to include it in this PR?


need to revert these changes, should be identical to original file here lines 51-60


```java
throw new PrestoException(INTERNAL_ERROR, "ConnectorMetadata beginInsert() is implemented without commitInsert()");


Fixed.


Is there any other place where we do this mapping for the Hive connector? Maybe we should consolidate.


I wonder if it might be clearer to have separate structs for the monitoring and event-handling modes.


This needs release notes for the SPI change.


This commit looks good


:+1: thanks!


This commit caused a lot of silent panics for us. Why would Managed VM users want to use the metadata server instead of the `appengine` package?


The Bernoulli(0.5) priors were left out because they're constants (0.5) in the posterior.  Would this be clearer if the prior in the comment was (equivalently) rewritten as

```julia
##   gamma ~ DiscreteUniform(0, 1)
```

???


 @zjx20 If you see this, this getline has a bug.  Some arguments or options do not work.  For example setting multiple values and hexadecimal "set 0,1,2,7=0x32'
I think the problem is commas.


I like it! I would be agree with something like `$CONFIG_DIR = /var/myApplication/proxies.d`, configuring keystores to be in `/var/myApplication/keys` or anywhere else, and SSL certs being in `/etc/ssl` or anywhere else.

All of these options are already possible if an oauth_reverse_proxy instance uses config files with absolute paths.

However, in the unit tests here I noticed that config files use paths that are relative with respect to the top-level directory of the oauth_reverse_proxy node module. Therefore, here in the source code I'm resolving paths relative to the top-level directory.


:+1: Thanks a lot!


@jonboulle nvm. I think they are removed https://github.com/cznic/ql/blob/c30917914189c97ebfd85019f736908424883339/stmt.go#L538


Hi, I seem to be having a problem connecting to the internet from the vpn.
I'm not sure why this is, can anyone help?


**valid-jsdoc**: Missing JSDoc @returns for function.


this is an upper bound so will not include the final release of 0.4.0 http://docs.julialang.org/en/latest/manual/packages/#man-package-requirements


Can you verify the BIOS shows "Sys Temp" and not "System Temperature" or "SYS Temp" or something else?  I like to make sure the strings show up identically to what's in the BIOS.  Thanks!


@derekperkins There's no reason not to use the metadata server when running on compute engine. Please file an issue about the panics.


cc @broady


102966118-HeatRecoveryBypass-4568 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1866 of 1869 tests passed)

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


@chobberoni we do support non-broadcasted SSID, right?


Is it necessary to close the connection here to prevent a file descriptor leak?


Yeah I didn't feel like looking for the actual names ;P


Is the metadata server faster or better in some way? I don't understand the purpose behind eliminating the ability to use something that was previously supported.


Looks good (pending "Sys Temp" verification comment).  Please note I'll supplement this pull req with a commit that fixes the alphabetical order of the X7DVL/X7DVL-3/X7DCL entries in boardlist, in addition to updating doc/SUPPORTED.  Thank you so much!


I would put extensible and pluggable framework last- we don't want most customers replacing the view engine, and that's not a compelling reason to use our MVC implementation.  Make DI it's own bullet.


Yeah that one's not supposed to be there at all.


Using AppEngineTokenSource complicates usage for Managed VM users.

Please discuss on a GitHub issue -- it's hard to track comments on a commit thread.


as variáveis hora é minuto são inteiros


101280704_lowTempRad (nealkruis) - x86_64-MacOS-10.9-clang: OK (1853 of 1868 tests passed)

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


Actually these should be setoid morphisms now…


URLs don't have file extensions


Default value should be order here


TeamCity pull requests :: netty [Build 211](https://garage.netty.io/teamcity/viewLog.html?buildId=1386&buildTypeId=pulls_netty&guest=1) is now running


The `find ... -exec rm -rf ....` statements seem more complicated than needed. If the delete a single folder (checked two ebuilds), why not use plain `rm -R ......`, instead?


@bricesanchez Thanks for the input!  Could you please provide some more details of what  was broken so that I can reproduce the error.  Maybe a snippet from the survey DSL using detail_info and an explanation of what action you took on the webpage that didn't work.


Please avoid making `README` changes until you are extremely experienced with `Reel`


I believe `app-arch/unzip` is missing in `DEPEND`.


Thanks.


The parameter comments here really help to make the code more readable.


Yes len should be int (allow sending longer signals) and hz should be int (allow potential use of 455kHz for sending B&O signals)


A bit scary perl one-liner...Has what can go wrong here scenarios been tested to prevent fstab corruption?


as per my other comment, leave len & hz as int


Bug type : Damaging a t2 mob breaks t2 weapons really fast
Location : Everywhere you can fight a mob
Reproduction : Fight a mob with a t2 (stone) weapon/item and it start to get damaged extremely fast, around 5 hits until it hits 10%.
Additional notes : This does not happen a lot, but it does happen numerous times to me, I repair my stone polearm/sword with an anvil, then i go farm some t2, and it gets to really low durability in roughly 5-6 hits, (I re-log and it the durability is still almost broken).


len should be int


If MSTest creates the test class once but runs the TestInitialize method once for each test (which I think it does), then previously each test would run with a different node request ID, but now they'll all have an ID of zero.  Does this matter?


Hi Mike. Could you please explain the purpose of this commit. In particular, why did you have to switch to using `tee`?


Hi, those two tests although they were touching into Core Data they were no longer needed after we removed the preventNSUndefinedKeyExceptionAttack feature correct? Is this a reminder to add extra Core Data related tests or did I do something wrong?


yes @kennethlimcp we do support it


The `t` in front of `${ENVVAL_OF_THIS_WEEK}` obviously does not belong there. Please remove it.


published/RHCSA Series--Part 04--Editing Text Files with Nano and Vim or Analyzing text with grep and regexps.md

"两个 ZZ" 这里应该改为 “两个大 Z”


I guess a better name would be `capitalize_name` ;)


I'm not sure why, but this doesn't seem to create `include/graphqlparser/c`. Here's the structure created in my test:

```
$ find install
install
install/include
install/include/graphqlparser
install/include/graphqlparser/AstNode.h
install/include/graphqlparser/GraphQLParser.h
install/include/graphqlparser/JsonVisitor.h
install/include/graphqlparser/lexer.h
install/include/graphqlparser/location.hh
install/include/graphqlparser/position.hh
install/include/graphqlparser/stack.hh
install/include/graphqlparser/syntaxdefs.h
install/lib
install/lib/libgraphqlparser.dylib
```


testing


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 631](https://teamcity.labkey.org/viewLog.html?buildId=272328&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33 (1 new), passed: 9 Build time: 00:23:33
### Failed tests
```
org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

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


You can use Assembly.Load to load the embedded common library or also use linked file in visual studio under a new project (Quasar.Common) as you stated above.


Oh! Really good.

I'm addicted to tomato too!!!


You don't really need to use `map!` here, `map` would work just fine for the array you get from `name.split` is something you don't intend to keep and `.join` is chained onto it within the same computation.


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


Calling Dispose on the test class within a test feels really wrong to me.  It's probably no worse than calling TearDown  was though... ¯\_(ツ)_/¯


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1854 of 1872 tests passed)

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


db = new PouchDB(dbs.name);

here 'fixes' the test


Thanks guys. Should all be fixed now.


“复制 M 行” 这里或许应该为 “复制第 M 行”，上一行类似。


这里多了一个空行。


Sorry about that, I don't have much experience with these Lua APIs. It seemed to solve my original problem but it's definitely not fully correct. I'll try to fix those issues and re-send it.


Even with access to the [docs](https://github.com/elastic/elasticsearch/blob/master/docs/plugins/delete-by-query.asciidoc), [source](https://github.com/elastic/elasticsearch/tree/master/plugins/delete-by-query), and Elasticsearch itself, it's difficult to make sure that these checks are right in all cases. I ran some tests with Elasticsearch with some infinite bash loops to cause version conflicts and missing documents, and these checks seem to be consistent with the responses I got from the official Delete by Query plugin.

Let me know how watertight this should be.


msmfas


这个我加回去


These might be moved to the initializer


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I'd recommend using references instead of pointers. Instead of...
    std::ifstream \*in;
do...
    std::ifstream& in;
Then instead of referencing it's value like this...
    \*in
you can simply do this...
    in
The dereferencing operator \* is understood.


Oh I see, that makes sense. No `gamma ~ Bernoulli(0.5)` makes sense.


@XinyuHou
Yes. The problem is, the timer retry is easy if you have a Stopwatch object, but much uglier if you only have struct timeval to work with.

I can write a bunch of code to do it, but it might be hard to read and possibly regression prone... let me know what you think.


Is there a difference between:

```
Key.(if_ is_xen) x y
```
and
```
match Key.(get target) with
| `Xen -> x
| _ -> y
```

?


Most innovative piece of code I've seen yet


ok


The goal is to be able to eventually set IPv6 gateways here, too. Since in.ndpd(1M) will overwrite the routes, though, as you point out in #475, I've made it IPv4 only until myself or someone else addresses this.


Two things:
1) The first returns a `Key.value`, the second do not
2) The first is not going to return nonsense if it's executed before parsing of the command line, which is the case of the `libraries` and `packages`.

`get` is still there simply for mirage's legacy, I would not exposed such function otherwise. The good way is to use `if_impl` over the relevant configurable instead.


Thanks. Fixed.


TeamCity pull requests :: netty [Build 210](https://garage.netty.io/teamcity/viewLog.html?buildId=1385&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5600, ignored: 21 Build time: 01:11:20


This is fine.


:joy:


Photovoltaics (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


I think there is an error with these login buttons.  Shouldn't the code be:

`<blaze-template name="loginButtons"></blaze-template>`
instead of <meteor-include>?

Thanks for a great tutorial and an AMAZING meteor plugin!


next time you update sdks, can you put the shortSHA in the commit description?


toggle between to do list and completed list works for one item so far.


@bartonjs @AtsushiKan @stephentoub PTAL.

This is separate from the original purpose of the PR and so the commit won't be squashed.
Let me know if this method is acceptable to unblock the original S.N.Security PR.


OCD much?


lines 284 through 657 are not your work, and credit should be given for work you did not do


Looks like post build was required after all http://forums.pcsx2.net/Thread-PCSX2-Executable-not-included-with-new-github-builds


Got it! Thanks :))


Might be slightly cleaner as `for i in 0..radii_values.len()`


I just put up a PR to fix that test: #3261


Thanks! :)


This is the piece where I had discussions with other folks during the past few days. I wouldn't call it a consensus, but there wasn't any objections to the sign (read only) vs hash (read/write) approach as a reasonable starting point. Per your comments, switch and bait attacks are still possible and will be addressed next.

One question came up regarding the use of dm-verity since it checks integrity at the block/blob level as opposed to specific files. This may be advantageous from a performance perspective, but I am assuming we don't have a read-only system partition vs. user partition model in Ostro as is the case in Android? Is this correct? For Ostro both signed and unsigned files can reside in the same block/blob.

+1


I presume this is mostly whitespace?  same nag as on last PR


@MaxXor How would you suppose we go about a plugin system without being able to dynamically load a library? :(
And there are several fun tricks to embed these references as resources. :)


Best error message ever! :laughing:


should be egrep


Photovoltaics (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


since selector isn't a string, find the component name somehow and make that the key on the helpers object, pretty sure that's the issue


The new logo for the Mongoose programming language.


TeamCity pull requests :: netty [Build 211](https://garage.netty.io/teamcity/viewLog.html?buildId=1386&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5601, ignored: 21 Build time: 00:35:24


Optimized it further


Okay, fair enough. Will patch.


a bit....and?


And then?


and thhheeeennnnnn???


I wonder if it's necessary to explicitly close the connection here as well.


We'll figure it out.


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (802 of 802 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-20.74%25-green.svg)


Done in commit 14cfa6a0fc097929dd04f232f4bec63e0cd1f2f5


There appears to be a parse error (line 118). Did checking qual well-formedness overlap with changes to the parser?


@synhershko why is `if (!string.IsNullOrEmpty(id))` required?  can a document ever have a null or empty id?


lines 133-135
lines 144-177
lines 284- 657
The lines mentioned in this comment are not your work, and credit should be given for work you did not do


with an exclusion of linux and modifying to not use `Pkg.test` you might actually be able to test 0.2 on OSX Travis


This has always been a topic of controversy. The MAS version of 1Pwd has the bundle identifier `com.agilebits.onepassword-osx`, and I think you're running the non-MAS version right?

Perhaps it should be a string to a custom defined icon that has both types listed, although I don't know if it works here?


@EmJay276 Magst du deinem Code ein paar Kommentare spendieren, die erklären, was su da tust? Ich kann mir gerade nicht erklären, was `rowId`, `codeId` für IDs darstellen sollen und wie der magische ERP Code zu verstehen ist... Dann kann ich versuchen, das in ein Skript zu mergen...


@jhass really nice :)


This doesn't work for me. My existing code did, I'm assuming it's a different between 1Pwd MAS/non-MAS? (I'm running 1Pwd 5.3)


Ah, so much for commit noise... this was supposed to go to the fork and be a pull request, but I had changed it so "git push" goes here. I guess I leave it for now...


Do we need to add x support and ppcle support here?
rnetboot service could only be used for ppc64 BE


@mostaphaRoudsari ,

Sorry for responding so late and I am very glad to see that you are excited about this.  There were a bunch of bugs in the version that I first committed here and I probably should have first put it into WIP (I don't think anyone would have gotten the loads to sum to a value close to 0).  I have tested it in a bunch of cases and it seems good enough now and so, if you do not mind, I am going to keep it in the "0" section of the 9 | Energy tab (sorry for the mental lapse with "-1").

To go down the list of issues:

1) I have put together an example file that you can find here:
https://app.box.com/s/9lgdmgcsxzhhhrfal38okjsccmoh57uw
For now, it seems best to view the energy balance with the bar chart and with stacked values like this:
![fullenergybalance](https://cloud.githubusercontent.com/assets/5567574/9866761/8ee782e0-5b34-11e5-9f30-480bfe354141.JPG)

The example file shows you how to do this.  In the future, I hope that we can get some cooler-looking sankey visualizations.

2) I think that our first benchmark with energy balances will be to accurately describe the energy balance over the whole model (or whole building).  This includes what I have just done as well as being able to specify helpful/harmful gains/losses, as well as a separation of envelope losses between roof, walls, floors, etc.  After we have described this model-wide balance, we can then move to describing what happens on the inter-zone-level over long periods of time including sankey diagrams that show flow between zones as well as flow in/out of the model.  Once we have this nailed down, we can build sankey diagrams that show this energy flow on an hour-by-hour time scale, engaging the storage mechanisms of the model more.  Finally, we can then build sankey diagrams that separate out the flow between individual surfaces.  Let me know if this general plan sounds good.

3) Upon your suggestion, I re-arranged the inputs to match the order of the outputs of the Read EP Result component and I changed the names of the inputs to match this component.  I had my own logic for how these terms were arranged (going from typical largest heat gain to typical largest heat loss) but I agree that, so long as the terms are output from the energy balance component in this rough order that makes sense for the visualization components downstream, the inputs should be made as simple for users as possible.

4) This said, I am against having only the input of HBzones and resultFile for this component as I feel users should have the freedom to remove energy balance terms that they are not interested in from the output.  I have found this freedom very useful so far and I feel others should enjoy it as well.  I might be willing to write a function that gives a comment bubble if the component thinks that there is a term for the energy balance missing but I don't want to stop people from getting an output if they do not have a certain term.  Lastly, it seems counter-productive to have the user import results from the CSV file twice if they want to both construct the energy balance and bring in the results for other charts/graphs/analyses.  I know that this means a bit more time setting things up and connecting components together but I feel that this time will pay off if they only have to import the results once for each model run.

5) ... and finally, for the big issue: is our energy balance actually balanced.  From my tests so far, I have come to accept that it is not possible to get a balance precisely equal to 0 out of EnergyPlus since the calculation is not steady-state and has thermal mass in it.  I have found that, even when I set all of the constructions to be no-mass materials, there is still the mass of the air in the zone that can cause a bit of storage.  This said, I have been able to get down this "storage" term to be very small in relation to the other energy balance terms to the point that I now think that I am account for the whole energy balance.  You should see what I mean in the example file that I posted.  Let me know if you disagree.

-Chris


Good job :+1:


I don't fully understand your question here on whether" semiotic power will yield to micropolitical"? I understand the concerns as it relates to the difference between what basically amounts to a patient intake/ history sheet vs a piece of pro-life propaganda, but I don't see how this leads to a larger debate on semiotic vs. micropolitical power.
I see this more as an instance of an underfunded organization printing their own products on cheaper paper with a cheaper technology vs. a government organization that has made pro-life actions a high priority allocating more funding towards that area and using a "flashier" medium to showcase their materials.
I think that if we wanted to follow Haas' footsteps we could read a lot into the formats of the two documents, but I don't think that analysis is neceessary or even important.  I feel like it's almost a given that an internal document used by the clinic and placed in a filing cabinet after the patient leaves is going to be less elaborate and more simply composed than something that is handed out and is meant to sway people's opinions. One is a form of documentation and the other could be classified as a piece of marketing collateral. Both serve very different purposes and are therefore going to appear differently. Maybe I'm just very dense, but I don't really understand why Haas felt this was particularly notable or interesting.


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1875 of 1875 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)


Please use Validator.isPhoneNumber(phoneNumber) or Validator.isDigit(phoneNumber)


git remote add origin https://git.gitbook.com/miumiu33/3-12-rwd-.git


Yeah,  it seems like some people are still running the 1.7 version of the jvm on their local machines, when I build locally it changes it back to 1.8 for me. I guess people just need to update their java installations. I had Matt do it yesterday but I must have missed it in Connors branch.

On Sep 14, 2015, 9:58 PM, at 9:58 PM, colegarien <notifications@github.com> wrote:
>This seems to make the app use jvm 1.7 instead of 1.8?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/colegarien/SDDTermProject/commit/528dc3358c02ad578fb102d3de929787276659a7#commitcomment-13227856


101280704_lowTempRad (Nigusse) - x86_64-MacOS-10.9-clang: OK (1851 of 1866 tests passed)

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


We should generate the SAS uri of the source blob, and copy it to the destination container.
If so, users don't need to set the permissions of the container `stemcell` in the default storage account.


ok, i think i got those back in there


Build the Service Node Diksless Image


The default output of copycds has no diskless service image now. Do we need to rewrite this part?


@ZhangYiJiang I'm going to revert this commit, because in some browser the url generated is "http://coursemology.org//www.google.com/jsapi"


* Code refactoring: main.c has been split into 32 separate modules, grouped by category
* No new feature.

Link: http://aldostools.org/temp/test/webMAN_MOD_1.43.04.rar


ok


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I am pretty sure adding static do not change anything.
The static is not needed in C++, constants like this have internal linkage.
http://stackoverflow.com/a/177781


@vinicius73  @andreoav  ok


There are no absolute rules about these things.  But a unit test module is named after the module it covers, generally.  Some test coverage overlaps into testing other modules.  Clear comments and good names help disentangle the connections.


It is as same as precreate table and containers.


RegistrationValidator.java L#212 minor: use a meaningful method name, for example: getAge()


we need to define "valid" clearly. Since returns valid revision ... valid revision is... Or you probably can just remove valid


逆や！commentの親はmy_thread！

```
@my_thread.comments.new(params.require(:comment).permit(:text).merge(user_id: @user.id))
```
カラム名textってあんま良くないな。text型もあるし、何より何のtextなのかわかりにくいので。


WatchableKV is KV that can be watched.


102687702-Schedule-Unit-Tests (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


has_many ならcomments では？ comment で行けるっけ？


何するcontroller?


view でSQL発行はNG


It looks like "canola" was misspelled.


Wow, kind of shows how old our code base is: 2.9 - > 2.57 is a big leap :) Thanks!

We'll need to test this PR carefully to make sure the upgrade doesn't affect other plugins. Plus, for non 200 response, we will change your code so the plugin just fails silently.  This way Iframely can try and fall back to generic oEmbed endpoint via auto-discovery when Twitter rate-limits or oauth tokens expire etc.

We'll merge it in couple weeks, just in time for the next Iframely version.


- vâng, có vẻ như lỗi là do tất cả các lệnh lưu tile xuống DB thì có lệnh không được thực hiện(các tile được load từ db lên, update sau đó bị loại ra khỏi lru nhưng không lưu xuống DB khi gọi hàm lru , trong khi tile cũ chưa được update trong db vẫn còn ). Em tưởng gọi hàm store thì nó tự chaining lưu xuống DB ở chỗ hàm backupToDbOtherBR rồi.
 -  Lúc em in ra thử thứ tự chaining chỗ backupToDbOtherBR thì khi hàm `onMouseClick_removeMarker` chạy thì in ra không đúng trình tự chaining.  Nó chỉ in ra 2 lệnh
`console.log("testing: ---------------------------------------before ");` và `testing: after`
mà nó không in ra lệnh nào của cái return promise ở giữa 2 lệnh trên cả
)

- Hình như nó rơi vào trường hợp lỗi hôm trước, giống như kiểu em chuyển đoạn code
```javascript
prev = prev.then(function(response) {
            console.log("in here", item);
            return p;
        }).then(function(response) {
            console.log("end------------", response);
        }).catch(function(err) {
            console.log("err-------", err);
        });
```
từ hàm `removeMarker` sang hàm `onMouseClick_removeMarker` thì nó mới chaining đúng


it looks like companyId is not used


Like to try this tonight! :)


In the old code (before the block change) we could allocate a single, reasonably sized, buffer, but now we don't know how the temporary buffer will be.  So instead, I read one entry at a time and rely on the fact that the input stream is buffered internally.


This is to make it match `DoubleStream`.


I do not have an objection - we should probably attach it to the Encoder
interface (subset of codec).

On Wed, Sep 9, 2015 at 5:44 PM, Chao Xu <notifications@github.com> wrote:

> this
> <https://github.com/kubernetes/kubernetes/blob/master/pkg/runtime/codec.go#L71>
> is the actual struct that implement Codec.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/caesarxuchao/kubernetes/commit/9c5ace530b4389e9ba47f3d2f93081b023191fd2#commitcomment-13147041>
> .
>

--
Clayton Coleman | Lead Engineer, OpenShift


Pude subir algo??