TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6529](https://teamcity.mesosphere.io/viewLog.html?buildId=79943&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 174 Build time: 00:06:23
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Perhaps you should rename this function.

Please write documentation for this function.


Weshalb ist das notwendig?


@SanychBY вроде нормально, посмотрим что дальше будет


firefox


yes, but I don't want to place into rmi-interface. It seems to me that such a function belong to stdlib, but I had no time to discuss it yet with Ilya. I'll write a proper comment.


:) haha
You pushed this to master!

> On 24 Nov 2015, at 11:44, lazedo <notifications@github.com> wrote:
>
> just merge this :)
>
> —
> Reply to this email directly or view it on GitHub.
>


asdasdasd


Good catch, thanks!


BIP112 implementation proposes to use `(1U << 10)`.


"Modular programming"


Tu as utulisé du code que j'avais écrit, donc je partage la responsabilité ;)


I know, sorry about it. Usually I'm trying to avoid it but at times changes are tangled too much making an effort of separating it really difficult.


aah. sorry, now I misunderstood you:

> I was seeing this

I thought that it's what you wanted to write :sweat_smile:


``int64`` isn't there anymore, is it?


See the build log
[here](https://travis-ci.org/Feldspar/feldspar-compiler/jobs/92736102).
It seems to have something to do with `Typeable`.

/ Emil

Den 2015-11-24 kl. 11:58, skrev Peter A. Jonsson:
> I'm the one who added larger tuples and we're still on 7.6 here. How do the tests fail with them?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Feldspar/feldspar-compiler/commit/e6747e0fc3c75f45cf6066aa5ea23679c0692316#commitcomment-14585257
>


It still says "buttom" here.


Why have we used "given answer" instead of "given_answer"?


why have all func bodies to use ifdef CROSS_CHANNEL?
maybe better include/exclude whole file.cc if cross-channel detected?


Ok I will have a look at that.


Can we change "given_answer" to "wrong_answer" to make it clearer what's happening?


this broke the installation. because setup.py could not find pixelated.adapter.errors.
fxed in 760ce63


I like that convention
stefan

On 24-11-2015 11:50, Kasper Marstal wrote:

Should we use name TInterface to be consistent with ITK template names (e.g. TOutputImage)?

—
Reply to this email directly or view it on GitHub<https://github.com/kaspermarstal/SuperElastix/commit/bedae73480fcdac7e2737f6bc7568c320c3c0c4e#commitcomment-14585103>.

--
Stefan Klein
+31 10 7043442
http://www.bigr.nl/people/StefanKlein


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6530](https://teamcity.mesosphere.io/viewLog.html?buildId=79945&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 174 Build time: 00:06:11
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


What's wrong with it? It is a representative record of a client, which is somewhat active, because we can check the aliveness of it.


Just to keep note, I think @otaviobp saw issues on this cast.


What is req 6?


more love


The intent of the PR was to standardize the project on one style of indentation (not necessarily my own, though naturally that was my first suggestion), since it's not ideal having several different styles, and since 2-spaces are hard to read for some.


![pie chart](http://i.imgur.com/LLkJqqV.png)


As far as I remember from my calculations, that matrix was applying rotations in reverse order, i.e., ROLL 90, PITCH 68 and YAW 293.

Even `libraries/AP_Math/examples/rotations/rotations.cpp` was showing that the rotation `ROTATION_YAW_293_PITCH_68_ROLL_90` was wrong.


wow, you're right that *is* hacky


![valores por grupo](http://i.imgur.com/MavJD83.png)

![scatter plot](http://i.imgur.com/8cmaiAP.png)


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


I know this branch doesnt work yet thats why i havent merged it, i am editing alot of the code on my local drive and implimenting redstone power. It would be easier this way then the blocks only have to check if the blocks next to them have redstone power.
```php
if($block->getPower() > 0) {
//Do Stuff
}
```
Or something like this:
```php
class BLOCK extends EXTENDER implements redstonePower {
//Block Code
}
```
where the ``` redstonePower ``` would do all the checking for the block.


What binary gibberish? And where is this initial 17 second delay...?


In your proposal, we can miss the important features. In case the one who is testing his changes will miss one of the functionality, he will never know that he breaks the someone else code.

In my proposal, the skipped tests will be marked SKIPPED and developer will be informed about that.
As an example,
Peer-direct code doesn't compile in and I don't have a clue if and how many peer-direct tests I'm skipping and breaking.


Apenas deixando igual ao original, já que ele permanece em inglês não vejo motivos de alterar, assim quando for decidido uma tradução fica mais fácil de mudar.


What I am saying is that the intended rotation may be right but with the wrong name.


How perceptive of you.


Someone please verify the "Sai" link


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


103643052-FiniteDifferenceFluxOutput (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1963 of 1972 tests passed)

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


I don't think that macro is supported:

```
[   44s] + echo '_smp_ncpus_max macro expands to '
[   44s] _smp_ncpus_max macro expands to
```


Maybe start using itkException?


That's not a problem. It's just an internal flag, it doesn't need to be consistent across codebases. We can just assign different bit flags when merging.


Okay... Thanks! Just letting you know you can see what causes errors in Travis btw


Isn't that too restrictive?


Owen was using `%jobs` and that works just fine!


i see. with define in every function is not very readable.
maybe bit fat warning during configure stating that disabling X tests is good enough?


Propose 15 -> sizeof(time_buf)


If I write:  `cat $TRANSLATOR $TEMPLATE | mustache > $PLOT`  I get this error:
`Syntax: mustache <view> <template>`

I have changed the line to:

`mustache $TRANSLATOR $TEMPLATE > $PLOT` which seems simpler.


Hier passt etwas nicht, da zwei mal die selbe Condition abgefragt wird.


not sure, but maybe ```-1``` are valid values? I think you could use ```nil``` if you don't want to set the values.


Wozu ist der variables-Parameter gedacht?


`Accounts.findUserByEmail()` simply calls `Accounts._findUserByQuery({email: email})`, but you're right, using the public API here is cleaner.


This check won't work as intended if the user sets a `_correctAnswer` of `0`. It's probably best if we explicitly check for both `undefined` and `""`.

See also line 411.


I am talking because I think honami is C not D


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


Is this roughly equivalent to 'static-files.fbp', right? Here treats more options, but there one can enable/disable while running. Anyway, my point is that if both did exactly the same, it would be a nice showcase of why fbp rocks =D


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


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


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


Please add comment " // namepace" after the brace. As it's done for the other namespaces.


Why you wrap the component in a div?


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=15)


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


done in 324a886


Can we handle multiple inputs at this stage? For example multiple metrics? Perhaps we should discuss this in the next sprint meeting.


add more bio


Oh, yes.


Nice!


`M-x untabify`


This won't fit the issue


Hebrew is in! \o/

Thanks for merging!


Erro meu, tinha esquecido que a palavra Status fazia parte da língua portuguesa. Revertido.


Standard-formatet skal være func.name, ...
Bruk av prop skal kun brukes sjeldent for konsistent design. Endringene var
for at endringene i __getitem__ skulle fungere konsistens, det vil si
func["name"] ikke func.prop["name"]. Sistnevnte burde forsåvidt fortsatt
fungere fint.

Jonathan

2015-11-24 10:43 GMT+01:00 geirpa <notifications@github.com>:

> Jeg ser att du på de andre stedene har brukt func.name, var.name. Er det
> en grunn til at du bruker struct.prop["name"] i stedet for struct.name?
>
> Ps. Skriver dette mest for å teste :) ettersom begge ting gir samme
> resultat?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jonathf/matlab2cpp/commit/94e888498f821c58f25e1aa3cbfcf024fb3707e3#commitcomment-14583480>
> .
>


Works with cabal-install, not out of the box with stack (but can probably be configured to).


Any() => Determines whether any element of a sequence satisfies a condition.

Not sure that's neccessary. You just want to check if there's any Role Assignments in the Role Assignments list of the Security Object.


Det ser bra ut!


heh, though that may be ridiculous, I *was* thinking of adding configurable sound / window flash alerts (`\a`)...


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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6527](https://teamcity.mesosphere.io/viewLog.html?buildId=79939&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:07


TeamCity TripleA :: UnitTest [Build 195](http://188.177.191.210/viewLog.html?buildId=265&buildTypeId=TripleA_UnitTest&guest=1) is now running


sdfasdfasdfasdfasdfasdf


Sorry, that's your responsibility to merge, not mine. Throwing tons of forked code over the fence into github doesn't help a yota. Don't expect anybody to pick up the pieces. Too bad you decided to fork & benefit but not collaborate & share with the open source community. You are driving your q.wiki car over a well-paved road that others have built ... but you don't collaborate on road-building, just car-building.


TeamCity TripleA :: UnitTest [Build 195](http://188.177.191.210/viewLog.html?buildId=265&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 126 Build time: 00:00:32


`phpDoc/master` represents the class docs of the current development version in the `master` branch, so the class docs actually need to be updated on every commit pushed to `master`. The problem is that this could lead to race conditions when pushing multiple commits in quick succession.


sorry!! I was totally wrong, I was thinking in the Count() method, not the Count property. ops, sorry!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6539](https://teamcity.mesosphere.io/viewLog.html?buildId=79954&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:05:47


TeamCity ReOpenLDAP :: devel [Build 205](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1219&buildTypeId=ReOpenLDAP_Devel&guest=1) is now running


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=16)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6542](https://teamcity.mesosphere.io/viewLog.html?buildId=79957&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:01:13


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


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


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


Is this still used?


This one is missing:

 +  assert_template_format("$(- 10000000000 5000000000)", "5000000000");


otherwise it's ok, so once you add this, I think we can merge it.


hhh


Your example does not work in any JsonPath implementation I have found. My
interpretation of the "spec" has always been that there should only be one
filter statement inside the square brackets. So yes it is intentional that
it does not work. Unfortunately I was not aware that it worked in previous
versions. This goes for the other issue as well. If I had known this the
current 2.1.0 release would  have been a 3.0.0 release.

The tests running in 2.1.0 are for the greater part an extension of the
test in 2.0.0. Some changes has been made to old tests because they where
wrong, or considered bugs. It is obvious that the grammar needs to be
improved and a lot of test rewritten. The error messages also need some
love.

The syntax for inline filter is [?( @.bar == 'x' || @.bar == 'y' )]
or
The syntax for inline filter is [?( (@.bar == 'x') || (@.bar == 'y') )]

Sorry about the incompatibilities

On Tue, Nov 24, 2015 at 11:17 AM, jochenberger <notifications@github.com>
wrote:

> More incompatibilities: $.foo[?(@.bar == 'x') || ?(@.bar == 'y')] does
> not work anymore, Is that intentional?
> The error message is not very helpful: Could not parse token starting at
> position 5. Expected ?, ', 0-9, *. Position 5 of what?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jayway/JsonPath/commit/1a72fc078deb16995e323442bfb681bd715ce45a#commitcomment-14584387>
> .
>


Currently filename and permalink are independent, and order is just something I added to make sure stuff is sorted as we want, and not as Jekyll decides. Using 100,200,300… for order numbers gives us some breathing room in case we need to add more content, but I don't love the idea either, so we're on the same boat there : )


I don't see where to download this from website? Is there a link?


OK, let's keep it if you think it is useful.


This can be done as a single write by using writeFile with the [`mode`](https://nodejs.org/api/fs.html#fs_fs_writefile_file_data_options_callback) option.
Also, generally speaking, there's no reason to use `chmodSync` instead of `chmod` since `denodeify` already transforms the callback-based function to a promise-based one, making async work easier :)


Hey dude,

swipe improved administrator.  Has save buttons, places to add blurb, and it is now a resizeable independent window.  We need to nail the structure of a set of builder factory implimentations and I think we should prototype some objects for our tests, so they are super reusable.


wow


Missing space `&& (!`? :)


:p


This should be `.then( () => chmodSync(authFile, 0o600)` or `.then(chmodSync.bind(null, authFile, 0o600)`, otherwise `chmodSync` will get called immediately after `writeFile` is called and not necessarily once the file is actually written to disk.

Anyway, as per previous comment, we should either use the `mode` option or use the async version of  `chmod`


In my app I've got special I18n translate hack to catch missing translations.
I needed empty locales


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


I'm reading configure output in case of "make failure" after, are you?


it's not even built yet, watch https://nodejs.org/en/blog/ for release announcement for this


wait, what ? ;D


package com.manisha.miniproject;

import android.app.Activity;
import android.app.AlertDialog;
import android.app.DatePickerDialog;
import android.app.ProgressDialog;
import android.content.Context;
import android.content.DialogInterface;
import android.content.Intent;
import android.location.Address;
import android.location.Geocoder;
import android.support.annotation.NonNull;
import android.support.v4.app.FragmentActivity;
import android.os.Bundle;
import android.util.Log;

import com.google.android.gms.maps.GoogleMap;
import com.google.android.gms.maps.OnMapReadyCallback;
import com.google.android.gms.maps.SupportMapFragment;
import com.google.android.gms.maps.model.LatLng;
import com.google.android.gms.maps.model.MarkerOptions;

import java.io.IOException;
import java.util.Calendar;
import java.util.List;
import java.util.Locale;

public class MapsActivity extends FragmentActivity implements OnMapReadyCallback {

    private GoogleMap mMap;

    SupportMapFragment mapFragment;

    String source,destination;
    String sourceStationCode,destinationSationCode;
    String date;
    int count=0;

    private AlertDialog alertDialog;
    ProgressDialog PD;
    ProgressDialog PD2;

    Context context;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_maps);
        // Obtain the SupportMapFragment and get notified when the map is ready to be used.
        SupportMapFragment mapFragment = (SupportMapFragment) getSupportFragmentManager()
                .findFragmentById(R.id.map);
        mapFragment.getMapAsync(this);
    }

    @Override
    public void onMapReady(GoogleMap googleMap) {
        mMap = googleMap;

        mMap.setOnMapClickListener(new GoogleMap.OnMapClickListener() {
            @Override
            public void onMapClick(LatLng point) {
                mMap.addMarker(new MarkerOptions()
                        .position(new LatLng(point.latitude, point.longitude))
                        .title("Marker"));
                count = count + 1;
                getCityName(point);
            }
        });

    }
    public void getCityName(LatLng point){

        List<Address> addresses = null;
        try {
            Geocoder geocoder = new Geocoder(this, Locale.getDefault());
            addresses = geocoder.getFromLocation(point.latitude, point.longitude, 1);

            if(count==1){
                source=addresses.get(0).getLocality();

            }else{
                destination=addresses.get(0).getLocality();

                // fetchTrains();
            }

        } catch (IOException e) {
            e.printStackTrace();
            Log.e("error ", "error");
        }

    }
}


Hello!
Thanks for the PR - I've added a couple of comment. I believe the cleaner solution would be to have the write directly set permission correctly, since it would
a) simplify the code
b) completely remove the vulnerability window caused by the file being first written and later on `chmod`ed


后面不用那样重复echo……在for块中?>后继续html，遇到php变量再开标签就可以
如果允许短标签还能有<?=var?>这样的语法表示<?php echo var; ?>，就是不知道目标环境是否允许


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6547](https://teamcity.mesosphere.io/viewLog.html?buildId=79966&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:00:47


Formatting is wrong


A co jak userpanel nie jest aktywny (tj. userpanel_dir nie jest ustawiony lub katalogu userpanel nie istnieje)?


Looks great :+1:


right
-or-
maybe configure.ac will generate empty test which will just show:

cross-channel: SKIPPED

but the code will remain w/o ifdef for every func?


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


批量添加了而已……不过确实改了过来后比较好看，有时间就改过来（其实已经脑子抽地改过了class的双引号来适应……然后已经心好累了……


It means it's restricted to servers in the same security group. It's not OK to put unsecured swarm masters on the internet.


This is breaking the test-project. Gives an error `Cannot find module 'elemental'`. Removing this line fixes it


:beers:


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


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


Done


This commit is breaking the test project due to missing modules


In your example you  do linear scaling. But we need an 1/x scaling.


Kiedy będzie poprawka?


Muy bueno


You cannot use jQuery feature, angular's jqLite don't support presudo class selector in find method


这个……当然是一律使用/了吧……


This was for the unit tests. Changed here and using a stub in the tests instead.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6551](https://teamcity.mesosphere.io/viewLog.html?buildId=79978&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:00:31


all the useful bits
OUCH


习惯了- -还想了好一会为什么没起作用……


@eparejatobes note that I removed this alias. I think it's doesn't change much, if you have to write `#or[...]`, but then it's not ambiguous with the predicates `Or`


Well, this would be a shame. :cry:


I have made the proper fixes to Cmakelists; readme and added unittest++.
It's now compiling over ubuntu & windows just fine. Please let me know if
you have run into any other trouble.

On Thu, Nov 19, 2015 at 2:50 PM, sheepbao <notifications@github.com> wrote:

> Thanks and I have modified the file named CMakeLists.txt like that:
>
> cmake_minimum_required(VERSION 2.8)
> PROJECT( OpenFaceCpp )
> find_package( OpenCV REQUIRED )
>
> include(${PROJECT_SOURCE_DIR}/Lib/dlib-18.18/dlib/cmake/)
> #find_package(unittest-cpp REQUIRED)
> include_directories(${PROJECT_SOURCE_DIR}/Lib/unittest-cpp/)
> link_directories(${PROJECT_SOURCE_DIR}/Lib/unittest-cpp/builds/)
> add_executable( OpenFaceCpp NativeDLib.cpp )
>
> SET(GCC_COVERAGE_COMPILE_FLAGS "-fdiagnostics-color -std=c++11")
> add_definitions(${GCC_COVERAGE_COMPILE_FLAGS})
> SET( CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${GCC_COVERAGE_COMPILE_FLAGS}" )
>
> TARGET_LINK_LIBRARIES(OpenFaceCpp dlib)
> TARGET_LINK_LIBRARIES(OpenFaceCpp unittest-cpp)
> target_link_libraries( OpenFaceCpp ${OpenCV_LIBS} )
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aybassiouny/OpenFaceCpp/commit/44f8cd7984e0e80deb1a9ecd1525fc11bc54ecf9#commitcomment-14498625>
> .
>


// blog -> post


I saw the `apscheduler` but it seems a bit too much for our needs.
What do you think of the simple mechanism I proposed?
There is no index on the token table, I will add one.


I thought that cross repo merging was a github thing. (I do care about the history hence the `git log --stat` in the message).
Ok I think I fixed it now. Although this commit is still here the original commits are now part of the repo (for example [here](https://github.com/sramsay64/python-iview/commit/709965aaf98ff5afe3429e49fad49f94c98e9f05)). Anyway thanks for the suggestion.


Parseのキーがあがっちゃってますけど、大丈夫ですか？
@ri0007


Why do you need to have this option ? what is the use case ? in which case you do not want the backend rebuilt ?
This is the constructor here, do you have to reconstruct the Operator ? do you use the same backend for the several operator (this would be ill advised and done with great care) ?
Moreover this option  `rebuild_op` does not belong in backend but in operator. It should be renamed as `rebuild_backend` or something like that.


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


Wouldn't it make more sense to drop all trailing / instead of the last one?


@jeanminet
J'aurai bien voulu qu'on garde le design des drivers sans dev_mem quelque part. Il me semblait qu'on voulai mettre la version dev_mem plutot dans le repertoire Lase public pour des raisons pedagogiques et qu'on garde la version sans dev_mem dans lase internal.

Globalement je pense qu'il faudrait deux dossiers drivers/core: un avec et un sans dev_mem et le driver global lase.py charge l'un ou l'autre selon la configuration de KServer.


Added to https://github.com/c2corg/v6_api/issues/70


I added my 3 cents to the issue.


Created the simple constant.


// Весь этот блок header вынести перед article в div или section.
// По смыслу к данному посту и постам в общем он никакого отношения не имеет. А относится к блогу в общем


Because it is part of the server.


Ohh, sounds feasible, I totally forgot about configure.ac abilities.


// section -> header . В данном случае это является шапкой поста c заголовком и т.д..


@jeanminet

Desole !! J'ai cru que c'etait le repo prive !!!

Oublie mon message precedent !!!

J'ai pas fini mon cafe c pour ca ;)


@inampaki The `topic` and `notificationKey` keys are not supposed to be arrays. Your code does not support those cases.


わかりました。確認しました。


I need this method for test: https://github.com/Soullivaneuh/composer-versions-check/blob/9993e7f026e99f206a82fb8935b5253c30d3ceca/tests/VersionsCheckPluginTest.php#L55

How can I handle it?


@tk-bm マージします。


That makes more sense, I just quickly dived in to change this to shut the javadoc generator up. I'll change it in a later commit.


what happens when you set it to undefined?


// Обернуть в div для контента. Например post__content или как удобно, чтоб было понятно его содержимое.
// Все section убираем. И все классы то же. Тут нужно понять что этот контент добавляется обычными контент менеджерами, например через админку, и если ему придется его так же заверстывать и проставлять классы чтобы все правильно отображалось, то это будет боль :)
        // Тут уже стилизация тегов


//blog-footer вынести отдельным блоком после article. Это отдельный блок и к статье никакого отношения не имеет. А имеет к навигации по блогу.


Added!


No reason. It's still a draft (not in readme). In my local code I've already changed that.
If you want to use the decorator just copy paste and do necessary changes.
New version will be uploaded soon


hehe


Я обычно использую один `.container` на страницу. Но, возможно, я не прав. Можешь обосновать подобное использование контейниров?


Good catch.


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


shouldn't you be calling ``$plugin->activate($this->composer, $this->io);`` in your test instead, as this is what you are testing ?


Ah ok. I'll wait for the new version.


But for the mapping between the CO and the internal range we need linear
scaling?

The different scaling variants should be expressed explicitly in the code to
become more obvious.

On 11/24/2015 12:52 PM, Daniel Schürmann wrote:
>
> In your example you do linear scaling. But we need an 1/x scaling.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/daschuer/mixxx/commit/cc8381aed4b3c18b102c4fb7c7029c54a36f20d8#commitcomment-14586297>.
>


Usage instruction of the script would be handy. `%_QT_PORT%` is the env var or cmd param?
I don't like that the script logic depends on some outer status (QT installation). What if I will have QT in the system and will want to build win native port? Also I don't like the hardcoded paths to the VCs. Seems it should be possible to init dev env vars by just VC versions.
I propose to make the usage more straight forward. To pass target as the cmd arg.
E.g. `make_ssl_lib.cmd <win_qt | win_native>`. (and write the usage on invalid input)


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=18)


please merge it, i have met the same issues , and the patch solved that


:+1:

The idea before was to show the text explaining what happens when you select or unselect folders as soon as the checkboxes are visible.  but it's true that since the user can cancel it might not be necessary


Loving the commit message. ;-)


Nice


Sorry that this is bothering you so much. I can simply not wait weeks and months to include simple changes. Also, my UseCases may differ from yours, which would result in unwanted merge requests for you.

Concerning this commit, I can simply not see the need, nor the responsibility for a corresponding change in upstream, nor what is upsetting you. Feel free to contact me directly.


Rather than having two params `note` and `notes` can't we just use `note` and make it accept a list? Or make `notes` an alias. Seems a bit odd to me in the implementation above.


Yup I just did a `gd lua_examples/onewire-ds18b20.lua` and saw that there were differences (`gd` is just my alias for `git diff master dev --`) and saw that there were differences.  Having looked at the code in detail, I see that the `master` is version 1 and `dev` is version 2 where you've recoded the implementation a little more elegantly.  So yes this is closed.  Thanks very much @nickandrew.


@danzilio I had to revert this when trying to roll 0.4.0 out the .config method is not found....this is on Hiera 1.3.4 - haven't had time to test it with 2.0+ but in the mean time this fixes the functionality


this is all fixed on my #102


Hi Alex,
I wanted to ask, not sure if thats the right place.
How can I get every response's content? Obiously, creating RequestHandler and using OnResourceResponse method doesn't work. All you can see in there is response headers.


Thanks for this, but it's a little tricky to follow the change a.t.m since I think you have your editor set to represent tabs as 4 spaces and all your changes appear shifted left by 4. I think it's best practice to ensure tabs are always shown as 8 spaces, but that's not the same thing as configuring the editor to indent code by 4 spaces. I think it would be easiest to take another look at this once the indentation is consistent.


Menuda máquina en GIjazo!


Please, check this commit from jassir

https://github.com/SAOSP-M/platform_frameworks_base/commits/eecaff810dbe5e679f0c4611e466a9006ab33617/services/core/java/com/android/server/pm/DefaultPermissionGrantPolicy.java

Maybe something will help you with calendar permissions.

Thanks. Only i want to help you.


Thank you. That fixes a scope issue as docker-storage.erb was defined by service.pp but most of the attributes were only in init.pp


yes.
however the text in LOG(INFO) should be more explicit (using ApplyInverse...)


minOccurs maxOccurs?


There could still be forward `break`s from within a `loop` to targets outside the `loop`, so it's not so trivially reduced to just sections with forward branches.


in https://www.microsoft.com/typography/otspec/recom.htm

> 'head' Table
> Although historical usage of the fontRevision value is varied, the recommended use of the field is to set it as a Fixed 16.16 value, and to report it rounded and zero-padded to three fractional decimal places. Examples: Decimal 1.5 is set as 0x00018000 and is reported as “1.500”; decimal 1.001 is set as 0x00010041 and is reported as “1.001”. All data required. If the font has been compressed with Agfa MicroType Compression, this must be indicated in the flags field of the 'head' table.

so I believe it is safe to assume that the minor version number will be less than 1000. Besides, in your current code there is no way to set the head revision to, say, `1.001`. Since `versionMinor` is an `int`, it cannot be padded with zeros, so it will always end up as `1.100`.

I suggest to revert this commit and go back to the old ufo2fdk behaviour.


I construct a preconditioner that is stored into the singelton.
The singleton stores the pair (preconditioner, backend)
If the backend is rebuild, I loss the preconditioner.
I do not have to reconstruct the operator, but as long as the operator rebuild the backend, that make the associated preconditioner unusable.


hy, i know they are ok but somebody named these sup nets as power nets .
But yes I was a little stupid to mistaken vcc to vss


Thank you very much for this!  Could we push this version to npmjs.org?


I don't think this is the right place to fix things. `self.pos` is not the absolute position in the line, but it's actually the position inside the current node the cursor is at. This change might avoid seeing some of the if statements below that fix things like images.


Thanks Mike!


The HTTP_ prefix get's added automatically. I thought that's done by add_header. But I have not found anything about that in it's documentation


No problem. I am just merging the working forks, trying to understand why they exists. If you find out that this is still an issue, please talk :)

Good luck there on your work.
Thanks.


Crossdomain is not really needed, try without it :)


Same here use json instead


This is where we actually have to do the change. It might just be enough to remove this if-statement all together, so that when hitting a `<return>` we don't stop.


See the correction I did right after I read your comment ;-)


Yes, I know that those expressions were never meant to work, but they used to work in 2.0.0. I'll see what I can do about translating the expressions on the fly.


Note there is ongoing work to include the 220R pulldown which is currently a placement option explicitly. You will want to watch the repo for an incoming pull request. I've also restructrued the folders as there is soon more activity here.


also it broke dialog positioning


В прнципе, можно падейтить скрипт.
Прочесал все борды. Некоторые мелочи можно было конечно починить, но вносить десяток лишних доменов для борд с сотней постов за полгода как-то лень.
https://github.com/SthephanShinkufag/Dollchan-Extension-Tools/wiki/Supported


I don't think there's any need to run `mvn verify` at this stage, as no sources have been added for testing.


That seems to work...


Ok 👍 I did not see that part XD


wait crap how was this even compiling


Removed the 'exec' form, because this seems to break the cache -- resulting in slow iterative builds.


You can simply use ```<input type="date">```, see:
![dateinput](https://cloud.githubusercontent.com/assets/2339512/11366477/6c7de10e-92e8-11e5-823e-18d167ab7089.png)


Из вопиющего, с чем аще не разобрался - не кликаются кнопки на панели в Phutaba-досках.

https://02ch.in/b/
http://ernstchan.com/b


// Этот article тут избыточен.
 // https://webref.ru/html/article  - "Элемент <article> задаёт содержание сайта вроде новости, статьи, записи блога, форума и др."


Removed exec form, as it seems to be breaking the cache for me.


// Откомментировал в post.html


// Класс пост уже занят в post.html :)


The formula is too long, right? I will try to split it up.

> But for the mapping between the CO and the internal range we need linear scaling?

We still use the original CO value internally. This commit just changes the range from 1 .. 8 to 1 .. 1000.
The mapping between the slider and the CO has changed from linear to 1/x. This results into a nice setting curve.


// Ссылку пропустила. Как же пользователь попадет на страницу с постом


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


what is this shit?


I don't think there's any need for this to be exposed outside of the network, good to show that only ports 5000 and 5001 are exposed on this network in a demo.


// для ссылок класс


Also you can add a line note


Put this part into separate class


Bind mounts with `build` aren't portable when running with swarm + compose, right?


Removed bind mount to get this working with swarm.


I'm pretty sure `self.failureResultOf` will do what you need here, instead of `_trapError`


Алсо, нульчан-like досок оказалось довольно много. Так что вынесение их в отдельный класс вполне оправдано.

http://0chan.cc/b/
http://www.0-chan.ru/b/
http://claire.ws/sg/
http://post0chan.ru/b/
http://www.2chan.su/b/


IMO, it's easier to understand the test without ```style="width: 256px; height: 256px;"```


it's useless to apply M_yy->zero(); It's automaticaly done by petsc when ksp-use-initial-guess-nonzero is set to false.
I think that all new lines added can be remove  (and also the previous M_yy->zero(); ).


I seem to have the same problem, any news about this?


Note: I hit this (unrelated) issue while coding the examples; I could pull this out for a separate PR, but I do believe it's the right thing.


½


Correctoins to packaging


Ни на одной из досок нет конфликтов с капчей!
С тинибордом все довольно печально. Традиционно, на половине его досок невозможен постинг.


A shortcut (escape) can be used too, to unselect tiles...


When refactoring this change to use the peep spawns structure.


Zend_Version::VERSION constant is incorrect (1.12.17dev) for this release tag.. :(


@robertmesserle - i think we need to revert this commit (and reopen the associated issues). This has significant side affects.


Hm... I think that we can expect the developers to be aware of the defaults. Also, they should validate their files against the schema while developing. But if you really think that this could help us, I can bring them back.


So if the fluid is not registered as flammable it can ignite? That doesnt seem right...


I see you created #1439 :+1:


Can you remove the kernel version? Let me know if you don't want to respin the patch, and I'll commit those changes by hand.


..also incorrect in README.md


It roughly matches its name for 321 intrinsic ordered rotations per my testing.


I would have preferred RDF to just have nodes that can have 0..n names (say as literal values of rdf:value statements) but as I read the RDF abstract syntax IRIs are node in the graph alongside blank nodes and literals. Am I missing something?


Yep. I only discovered this when I tried to compile for iOS again and it would error out. The `controller.microGamepad` is just not defined in the iOS libs :/


This is NOT on/off type of parameter. This is a float-point parameter with arbitrary values between 0 and 1. It can have a value of 0.4, 0.578 or whatever else!


I love you, this fixed my code completion and will make work much much faster.


I would not change this in a patch release as it will remove existing content if the template has not been manually adjusted. Same applies to other sections.


Did not remove close-element.js, as it may be used somewhere else.


Това имах предвид:

``` ruby
module WordCounter
  describe FileParser do
    describe '#parse' do
      it 'exists' do
        expect(FileParser).to respond_to(:parse)
      end
    end
  end
end
```


Roger that. I'm setting my git client to enforce the line widths.


Perhaps it should simply be renamed ROTATION_SOLO_COMPASS? It is useless for anyone else.


idem pour le nom


@bsstoner Do we care about transition when it disappears? I just discovered it's possible to do it if you add a one-direction delay on `visibility` - in one direction only.

Hope you don't mind me adding commits to your PR :smile:


@adrianschroeter I would prefer to do such a change in a separate commit. But I can take care of that.


Je n'aime pas le nom, ça ne set rien du tout ici, set ce serait si tu passer l'event comme argument et que ça retournait un event avec l'adresse (ce qui serait, je pense, moins bien). Return (pas réfléchi, y'a mieux à trouver) semble déjà mieux.


oohh asynchronous state machines ;-)

Reminds me of the Agni (is that the right name) code I wrote for Agent of Ascent!!


DeltaPhi ;)


@vincentchabannes Indeed.


ça marche ça "===space! " ?


spaces missing


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6541](https://teamcity.mesosphere.io/viewLog.html?buildId=79956&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout Build time: 01:00:07


lel xD :+1:


@teresamorais


space missing


newName ?


:( It looks like markdown does not support links in code blocks  ...yet


vague comme nom, j'ai du mal à capter tout ce qu'elle fait...


Hi Lars,
I'd be glad if I could integrate your changes into the Cuckoo code base.
Is that okay with you?
Thanks
Carsten


What are al these (and following lines)?


Shouldn't this be `args[0]` (the encoding passed to `merged.write()` below) instead of the literal `'utf-8'`?


Good catch Adolfo!


Pourquoi waitForSearchBar est ici ? Tu devrais pas plutot avoir une fonction initializeMap avec deux fonctions dedans? initializeSomethingWithTime et waitSearchBar ?


It will remove the markup but not the information, which is added above instead.


It will be great to have a space after comma.


Để clear() cho nó xóa hết đi chứ sao xóa mỗi username thế?


Ahh! I'll add some test coverage around this. We'll want to add Hiera as a development dependency and test with 1.X and 2.X. In the meantime, I just looked and `Hiera::Config[setting]` will work with both!


Exactly, which has no real reason to change in a patch release and will change the existing markup and styling of the page, right?


能加个怎么调的例子在上面吗？ 照顾一下不会php的，:-D


@zeroZshadow u welcome!


this translate doesn't work but if delete 47 line (days.day) this translate will work. I understand that days are the same keys for views and controllers so i18n is confused but I don't now how to solve this problem


In Laravel, there is something we call [Route groups](http://laravel.com/docs/5.1/routing#route-groups). So put these into route groups, right now, it's just too WET.


google also doesn't now )))


I would move these options to a `.clang-format` file in the root folder so that IDEs can use these options. The gulp plugin can also handle the configuration via a file.


Please review these changes.

These changes may have affected the build for https://github.com/mysteryjeans/android_device_lge_d856. A cm-12.1 build for d856 now shows no SIM card even after switching off the airplane mode. I think that this change impacts the code such that D856 is now appearing as a CDMA device and not as a GSM/LTE device. The build uploaded by mysteryjeans on 151121 (before these changes) does not have a similar issue.


Great one !
I wanted to do it myself cause it was kinda annoying.
Well done :)


@ciampix Thanks a lot. It is great !
And I am sorry that my commit https://github.com/KiCad/kicad-doc/commit/99fe7beb0ee5ae504559bd8ab8da85dd086f3d12 did not care about .dia files.


tu devrais extraire une méthode pour for {} d'autant plus qu'exactement le mm bout de code est utilité plus tard dans le fichier


@Jorch95 @santimarrone Esto pone la imagen de fondo, mucho mas simple que usar lo de `Graphics`, hay que aprovechar que estamos usando `JFrame` en vez de `Frame` y usar el `setContentPane`


How do we get here if location service is started?


Hola Jose,

Gracias por todas las colaboraciones hasta el momento. Creo que son
realmente útiles para lograr la calidad mínima con la que me gustaría hacer
el release oficial de timegrid.

Te comento que estoy trabajando en un gran refactor de la lógica de
reservas, y que este proceso implica un replanteo importante de las capas
de servicio para reservas. Esto hasta ahora no implicaría cambios en los
modelos ni la estructura DB.

En caso que estés interesado, estás invitado a participar por chat o
collabedit para diagramar en alto nivel cómo sería esta lógica. Al margen,
intentaré que salga algo de documentación al respecto ya que es parte de la
funcionalidad core de la aplicación.

Saludos!

On Sun, Nov 22, 2015 at 4:48 AM, Jose V Herrera <notifications@github.com>
wrote:

> @alariva <https://github.com/alariva> cheers!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/josevh/timegrid/commit/8a3fb83b87dbb78ec598208a38592c1946f2fbfe#commitcomment-14545794>
> .
>

--
*Ariel Vallese*
*alariva.com <http://alariva.com>*
UTN | Facultad Regional Buenos Aires
IT Consulting & Web Development
Barcelona, Spain
+34 651 464218

*PRIVACY AND CONFIDENTIALITY NOTICE*The information contained in this email
and any other attachments are for the use of the named recipient only. This
email and any attachment are confidential and may contain information which
is subject to professional privilege. If you receive this email in error
please contact us immediately and delete the email and any attachment
without making or retaining any copies and without disclosing the contents
to any third party.


TeamCity TripleA :: UnitTest [Build 196](http://188.177.191.210/viewLog.html?buildId=266&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 196](http://188.177.191.210/viewLog.html?buildId=266&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Compilation error: Software\TripleA\CashRegister.Program\CashRegister.Program.csproj (new); number of tests is too small: 0 (new) Build time: 00:00:08


not → now?


We had a pretty long discussion about this. The difference between the 2 syntaxes is not where they are resolved from, but how they are identified. So we deliberately left 'external' out of the syntax.
At the moment, a component can have 2 different identifiers (either or both):

- `ProjectComponentIdentifier`: project-path + component-name
- `ModuleComponentIdentifier`: group-name + module-name

These have `ProjectDependencySpec` and `ModuleDependencySpec` respectively. The fundamental difference between a `ProjectDependencySpec` and `ModuleDependencySpec` is how they identify the depended-on component, not where the component will be resolved from.


Looks good.  Why use the old gensym?


extrais une méthode pour la déclaration de newName, et le mieux serait que la fonction prenne en argument $scope.time et renvoie newName, comme ça la ou tu en as besoin tu fais var maVar = function() et y'a pas de side effect, ce qui est plus facile à tester et aide le raisonnement.


TeamCity TripleA :: UnitTest [Build 197](http://188.177.191.210/viewLog.html?buildId=267&buildTypeId=TripleA_UnitTest&guest=1) is now running


yes


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


Declaring variables in the middle of scope is against C standart


Interesting, I'm cool with that as long as it doesn't feel slow. When I close a modal I typically just want it gone and a transition can be annoying.


I would recommend using `NuSpec.ReferenceGenerator` to add/update the dependencies as they can change over time and the tool will ensure they're up-to-date.


TeamCity TripleA :: UnitTest [Build 197](http://188.177.191.210/viewLog.html?buildId=267&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 126 Build time: 00:00:32


crane is not synonym, it's component of pulp


this should show an elo gain with pondering, but we can't use pondering in fishtest (yet)


yep


I'm not sure about that, but should it be log_XXX and not fprintf?


@josevalim You basicly rewrite all of this :)
Thank you! Will do my best to get lesson from this fixes :+1:


Doh... Thanks!


I'll fix it. I just though it's ok since these declarations are all around the code.


line 48 is where you can configure 'v1 upstream registry', will elaborate


Woups, it's a remaining of the old code I tried...


why not sending a PR to upstream?


TeamCity TripleA :: UnitTest [Build 198](http://188.177.191.210/viewLog.html?buildId=268&buildTypeId=TripleA_UnitTest&guest=1) is now running


nice


TeamCity TripleA :: UnitTest [Build 198](http://188.177.191.210/viewLog.html?buildId=268&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 126 Build time: 00:00:31


I agree. This printf is completely removed in the following commit of this PR. This is just unnecessary output that adds an extra mess.


Please edit the grammtical blunder on your commit message


I have the same issue on my machine after the last update


That's not true. Using `Enum.slice` was a nice touch and the whole idea itself was great. :) Thank you!


do you not also need to add the repeaters app?


Hmm, fabric says it can be string or list of strings. I thing we should allow multiple arguments here...


Why are you using `asset` and `secure_asset` for everything?.


Please remove this docblock


`ids = ids.concat(data.pve.filter(daily => daily.level.max === 80).map(daily => daily.id));`


TeamCity TripleA :: UnitTest [Build 199](http://188.177.191.210/viewLog.html?buildId=269&buildTypeId=TripleA_UnitTest&guest=1) is now running


Please update description


Thank you Aleksej,

It conflicts.
I will manually apply your patch.


TeamCity TripleA :: UnitTest [Build 199](http://188.177.191.210/viewLog.html?buildId=269&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 126 Build time: 00:00:32


Consider making `distDir` a config option of this plugin (with the default value being a function that returns `context.distDir`). This will allow more flexibility for consumers of the plugin.


I know, there is already an issue for that feature request.


Let css files be grouped with css files


LGTM


Please add quotes around the number, currently this is parsed as float which leads to
```
    - Rendering SLS 'base:logstash' failed: Jinja variable 'float object' has no attribute 'startswith'
```


Example here: https://github.com/ember-cli-deploy/ember-cli-deploy-json-config/blob/master/index.js#L29


i would add to the wiki full path to this, so we know where all our data lives

https://maggie.lt.informatik.tu-darmstadt.de/foswiki/bin/view/Langtech/VWData


This commit is generated automatically.
Look at the source code tree to check if there are any mistakes.


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


Oh yea, you are right.


ah ok


Hmm, the number of arguments keep growing. How would you feel about passing just `quiet` and the rest as `env` dictionary? (the same in `_setup_environment`...)


Do the merge locally on your git, just instead of resolving conflicts, commit a modified README file while merging. Simple.


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


no since the views are now in a couchapp which is already included


TeamCity TripleA :: UnitTest [Build 200](http://188.177.191.210/viewLog.html?buildId=270&buildTypeId=TripleA_UnitTest&guest=1) is now running


Getting close.. One more thing: The default for Option.DEFAULT_PATH_LEAF_TO_NULL is false, that changes the behavior too. For the Goessner example, 2.0.0 used to return `[ null, null, "0-553-21311-3", "0-395-19395-8" ]` for `$.store.book.*.isbn` with no special options set, 2.1.0 returns `[ "0-553-21311-3", "0-395-19395-8" ]`.


TeamCity TripleA :: UnitTest [Build 200](http://188.177.191.210/viewLog.html?buildId=270&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 126 Build time: 00:00:31


To indicate which requester config owns the HttpClient. That way we can tell which requester has been configured to use an insecure client and alert users.


Already done.


hmm, good question. I did not know this was not the right place.

On Tue, Nov 24, 2015 at 7:48 AM, Asif Saifuddin Auvi <
notifications@github.com> wrote:

> why not sending a PR to upstream?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/h4ck3rm1k3/pythoscope/commit/dfa85a8f45a89a87c58f990357b8884f87bd86dd#commitcomment-14587479>
> .
>

--
James Michael DuPont
Kansas Linux Fest http://kansaslinuxfest.us
Free/Libre Open Source and Open Knowledge Association of Kansas
http://openkansas.us
Member of Free Libre Open Source Software Kosova http://www.flossk.org
Saving Wikipedia(tm) articles from deletion http://SpeedyDeletion.wikia.com


In this case, we want to do some validation on the urls.

Perhaps we want the thumbnail_url field to be generated from other values in the object, so we don't end up with valid arbitrary url's.


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


1. It will be great to document it.
2. Please prefer IB... format of variables over XX_IB.
3. We are in the process of name change from core-direct to be cross-channel. No need to go and change old code, however new code will be called CC instead of CD.


Rather than addin another bool here, i'd prefer if we'd make this a UnitDependency parameter, so that the caller specifies either UNIT_REQUIRES or UNIT_BINDSTO here.

(Of course, unit_add_node_link() should downgrade the dep to UNIT_WANTS even then)


the github maintainer accepts PR I saw :)


This will work when the repository is new, but not when an existing repository cache is outdated, since the first step will not trigger a change.


I am _STRONGLY_ opposed to this approach. This is exactly the kind of flexibility that can potentially open the door for security vulnerabilities.


works but appendLine is prettier


Do you use configChanges for detecting screen rotation?
If it's true then you are making a big mistake.
For example if I change language on my device activity will be restarted.


don't we need to revert this value when we're done?


чего то ты перемудрил. Зачем const перед {}


Why comment it out and not remove it if it's not needed?


don't we need any order here?


please remove comments, method name explains it well


@debonair You should also change the types of the methods which use this value, and add a non-nullable constraint to the database for this column.


To make th code clearer, you could have done :
```c++
 		bool isAllocated = (iterFilename != _inputFilenames.end());
 		if(isAllocated)
 		{
 			// get existing InputFile
 			const size_t filenameIndex = std::distance( _inputFilenames.begin(), iterFilename);
			inputFile = _inputFiles.at(filenameIndex).first;
 		}
 		else
 		{
 			// create new InputFile
 			inputFile = new avtranscoder::InputFile( filename );

 			// display file properties
 			std::cout << *inputFile;

 			// add to list of filename
 			_inputFilenames.push_back(filename);
 		}
		_inputFiles.push_back(std::make_pair(inputFile, isAllocated));
```
But that's good enough yet ! ;)


Fantastiskt, ser till att föra det vidare till projektets andra utvecklare!


Do you need this function?
If yes, please add call to it, if not, please remove it.


rename to: radiologyOrder radiology order corresponding to the obs


rename to: obs the obs to be changed


TeamCity TripleA :: UnitTest [Build 201](http://188.177.191.210/viewLog.html?buildId=271&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 201](http://188.177.191.210/viewLog.html?buildId=271&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Compilation error: Software\TripleA\CashRegister.Test.Unit\CashRegister.Test.Unit.csproj (new); number of tests is too small: 0 (new) Build time: 00:00:05


Why you pass instance of DatabaseHelper to that GetDataFromHtmlManager instead of creating new inside GetDataFromHtmlManager?


I think that's fine, sure.


I prefer to see log_XXX, but it is up to you.


This is not enough because this plugin is not on the plugin manager.

I deal with reflection class for the moment.


Yep, sure.


link please, maybe update the readme to help the next person?

On Tue, Nov 24, 2015 at 7:58 AM, Asif Saifuddin Auvi <
notifications@github.com> wrote:

> the github maintainer accepts PR I saw :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/h4ck3rm1k3/pythoscope/commit/dfa85a8f45a89a87c58f990357b8884f87bd86dd#commitcomment-14587735>
> .
>

--
James Michael DuPont
Kansas Linux Fest http://kansaslinuxfest.us
Free/Libre Open Source and Open Knowledge Association of Kansas
http://openkansas.us
Member of Free Libre Open Source Software Kosova http://www.flossk.org
Saving Wikipedia(tm) articles from deletion http://SpeedyDeletion.wikia.com


Well it might be worth considering printing these values here, rather than polluting the `EC2InstanceWrapper` with avocado `stream`. Then the `EC2InstanceWrapper` would be completely generic/independent of avocado...


Why does this change need to be made now? It doesn't seem ideal as it's introducing code that seems unrelated to the thing being tested.


прокомментируй мою пожалста.


Will add index. As for the question: in the kernel, `block` is the only sequencing construct, so any `<expr>*` desugars into that.


please check that all these callocs succeeded.


Yes, but have a proper solution to handle this on documentation would be better IMHO, especially since you warn user about performance degradation.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 56](https://teamcity.labkey.org/viewLog.html?buildId=287245&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 13 (9 new), passed: 34 Build time: 01:17:35
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


tested it, works! branch can be merged from my side.


Make sure git is also installed


That's because feldspar-compiler uses base-orphans, which implements Typeable instances for large tuples. I have changed feldspar-language to use the same library, see feldspar-language@793e0db4452a9f9388c2a75ab02d26fbf40ec5af.


Did you mean "on" or "no"?


Hmm, with the plugins base growing, I'm wondering if we should consider some standard way of specifying requirements per-plugin, eg. "avocado/core/plugins/${plugin_name}_requirements.txt"` and include it in case of `make install-requirements`. (also include it in rpm/deb builds using mapping would be super-cool)


"--OO--" I like it)))
By the way what is it?


https://github.com/mkwiatkowski/pythoscope/commits/master

and see the issues the answer and encourage contribution :dancer:


@nellag ㅇㅅㅇ (?)


link na non-public jiru bych do komentu spis nedaval :) akorat  bych ho dal do popisu pull requestu


@dsvoronin, я бы рекомендовал указать в явном виде, что это alias, те
`"$ref": "#/definitions/alias"`


This does not seems right, duplicate of L#681-683


Shouldn't this be ":" instead of "="?


why not to transfer this variable to constant field


From the docs.
"If a listener is added for this exception, the default action (which is to print a stack trace and exit) will not occur. "
"It is not safe to resume normal operation after uncaughtException. If you do use it, restart your application after every unhandled exception!"
So we should `console.log` the error .`message` and the `.stack` trace - then call `process.exit(2)`.
This is especially important here, because the MsgFlo connection has no error recovery handling.


Follow the format of: back-log, to-do, for-review, done. Each card represents a requirement, which may be in the form of a requirement or a component. Currently, you are listing tasks instead of requirement.

Design concept seems sound and small enough in scope to achieve within the time constraint.


regex


Never do this! You have just lost the original stack trace for this message, which make it hard to understand what is going on.


 - решил, что проверять контент не совсем правильно и лучше проверять линк
 - не нашел можно ли как-то прописать метод "делит", поэтому оставил только название линка и путь


We could add an opt-in (default: false) `--ignore-exceptions` that would not process.exit - but that is an advanced usage option...


Step 3 is phrased more like a requirement in the title :+1: . However, the checklist also seems more like requirements and not implementation tasks.


올리는거 되는지 확인


I will test this. In the future I will follow the code formatting you use.


@Crotalus please catch me in slack at your nearest opportunity.


Hmm, that's a fair point. I looked at adding this for consistency, but it looks cleaner without it. Reverting...


- решил перед удалением проверять есть ли на странице содержимое, которое должно исчезнуть... возможно это излишне...


I'm not absolutely sure what causes it, but without quotes around {{ versionstring }} & when logstash.version=2.0, the version ends up being evaluted as 62.0 on salt 2015.5.5
```
        pkg:
            |_
              ----------
              name:
                  logstash
            |_
              ----------
              version:
                  62.0
```


I think you should split that helper:
1) database helper which would work with your database file(create, update)
2) several helpers which would work with database tables

but it's just my opinion


wow


Copy paste detected ;)


Definitely "on". Reproduced at http://httpbin.org/forms/post with Chrome and Firefox.


You never described what is so special about your use cases nor is it guessable by plain code diffs.

This is one change of a hundred+ of recent changes to plugins forked by modell aachen with weird changes all over the place. Some of them render these plugins modell-aachen specific so that these changes will never ever make it into an upstream release. Frankly, this is bad software design on your end. Coming up with a clean solution is a prerequisit for any open source code contribution.

"Can't wait weeks and months": At *least* try to file a task at foswiki.org and provide a patch. So how can you know in advance? Nor was there any discussion in the community about all these changes. Maybe you have a valid point there. Maybe you fixed some serious issue. Who knows.

Some of those changes are already fit to be added upstream, e.g. part of recent SolrPlugin hanges. Alas, you decided to keep those improvements to yourself. Too bad.

Sure, going by community is the longer way. But that's the game. In the end going by community almost alway means higher code quality.

Is open source so difficult to understand?


Yes, the comment should be removed :-). If that's how it used to work, but changed, then the change was the source of this bug.


please remove assertion since this one has nothing to do with what happens in method populateModelAndView. if you really insist on checking that the mock is correct, please move it to //given. I would favor removing it.


Из названия неясно, для чего нужен этот класс. Он что-то парсит? Что означает "парсить"?


Apparently this works when I specify either vpc_name or vpc_id.
Nevertheless it should work without these because the security group gets created but no rules are populated on the inbound. Most people will have only one VPC anyways and having to specify the vpc mandatory makes things quite inflexible.


Yes. I wanted something that could give me a faster feedback at the moment I was developing the test though.

One thing to note is that the wrappers are just a convenience for me (it binds ec2's classes together so that I have less upkeep work to do), so the fact that they take stream as a parameter is not that big of a deal IMHO.

If we must keep it generic, we could replace stream with a generic print/report function. Then when we instantiate the `EC2InstanceWrapper` object we pass `stream.notify` as the print/report function.


This one overshadows user code, did you forget to exclude it from the release?


If you want to notify your activity that downloading complete you need to use callback instead of passing your Activity class.
Or use EventBus for example.


Why not solve this problem together?


TeamCity ReOpenLDAP :: devel [Build 205](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1219&buildTypeId=ReOpenLDAP_Devel&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:44:23


what's the point of this NULL check ?


Переменные обычно записываются маленькими буквами. Аббревиатуры тоже. Почитай http://www.oracle.com/technetwork/java/codeconventions-150003.pdf


Is there any documentation for this? e.g. the best way to get SID and Tenant IDs


You are right of course ;)

On Tue, Nov 24, 2015 at 2:20 PM, Dan Korostelev <notifications@github.com>
wrote:

> This one overshadows user code, did you forget to exclude it from the
> release?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/back2dos/closure/commit/eea6614bfb72cc56ed57063baaf2176066c8e69f#commitcomment-14588292>
> .
>


pravda, pravda, díky


It doesn't make sense to add marker from result again to result.
Here you need to add markers for all joints, i.e.
BOOST_FOREACH(std::string joint, rm->getJointModelNames())


fixed in 06dd977cc5812f1f320b42d87911bb02f740d1bb


Isnt execute_tunning mandatory in the driver? instead of this change the proper interface should have been added to dw_mmc-k3.


change these two to MBeanException


103643052-FiniteDifferenceFluxOutput (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1963 of 1972 tests passed)

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


splitUrl - плохое название для переменной. Не очень понятно, что содержит в себе переменная. Я бы предположил, что она содержит URL с разбиением. Лучше было бы назвать её urlSplit или parameterValuePairs, например.


Why two callbacks here? Either one of them seems sufficient for the tests to pass?


Did you test this on TLWR710? thats why i split this into basic and more


No, forget that last comment, it's something else that behaves differently now. Still investigating.


support for python users?


When the repository has not just been added via apt_repository then line 19 will cause an "apt-get update"  about every half an hour.

I like that better than running apt-get update each and every time.


I think the callbacks should probably [move to the top of the file](https://github.com/bbatsov/rails-style-guide#macro-style-methods) with the other activerecord callbacks https://github.com/mysociety/alaveteli/blob/develop/app/models/info_request.rb#L114


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


This method name should be in camel case to respect the Java conventions.


String parameter содержит не только параметр, но и его значение. Нужно поменять название.


Can we just use the named route instead anyway?


Thank you, could you please release a 6.0.1, it silently breaks installs.


Sounds reasonable.


Yep, that's how I've implemented it - `notes:` is just an alias for `note:` :-)


splitParameters - параметры разбиения, судя по названию.


Copy paste of what exactly? Is there a problem?


Why the param is called localName and not just name?


RedHat-based distros do not need this prefix.


Is this check really necessary now you have changed the iterator to throw an exception if the Uid is not valid.


emitMarkersChanged should be only called by a controller, when the number, type, etc. of markers have changed.
In response to this signal, ControllerDisplay::updateMarkers() will collect markers from controllers (by calling getXXXMarkers()).
For the joint controller, the markers only change when the robot model has changed.


Add this value to string.xml and use it from there.


done - thanks


> Create a new Edition presenter for logic shared between old world
(placeholders) and new world (content items)
Rename old Edition presenter to EditionPlaceholder

This description seems a bit off to me. The placeholders don't really relate to the old world.


So simply
  uids.add(iter.iterate())
should be sufficient


 I'm afraid we might know what is going on.

First of all I'm assuming that you redacted warning message and {filename}
was really some chunk file name.

I'll describe what the problem was. Chunk object (in memory) could be
reused if we had to create new chunk with old id. But the problem occurred
when chunk was in old format but new one was in new format. Then we have
old object with new format. This was causing reporting old positions of crc
and data in chunk file. Now with the fix new (correct) values are being
reported. But the problem is that those positions are different than the
data were previously written to.

The only problem I have is that I think that old version of chunkserver
should also write this message during the scan.
It's possible that it doesn't only if there are two files for the same
chunk id. Then old version wouldn't report error but new one would.
But it should remove one of the files and then again old version on start
should also write error message (the same as new one).

On Tue, Nov 24, 2015 at 11:53 AM, Dmitry Smirnov <notifications@github.com>
wrote:

> Yes, with both commits (this and e61487d
> <https://github.com/lizardfs/lizardfs/commit/e61487d00eba5b79db5f516062ed9ff867061169>).
> Error is logged during scan, I did not see it during normal operations yet.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lizardfs/lizardfs/commit/89e339a50fa10566e4c0309d49006c2c26c700c2#commitcomment-14585165>
> .
>


Remark:the javadoc did indicate how errors are passed to the caller - but before you counter that I agree an exception is superior to a non localized return string.


I don't see a need for such a function.


Could you please release a patch version ? This is blocking a feature I am developing.


allSignedUp is a function, not sure what effect this has here.


I wonder if it is desired to suppress exceptions for all handle! calls. It seems to me that exceptions throw upon handle! calls made by the user should NOT be suppressed. As it is now the user has no way of knowing if a handle! failed.


Found the problem byte_3 problem a few minutes ago. I already thought about a hardware problem... But after fixing this, everything runs fine.
@TerryE
* 20 mSec is far too long but was working in my case (surprisingly). Arduino code uses 1mSec - 1.1 mSec. The datasheet says at least 1 mSec: https://www.adafruit.com/datasheets/Digital%20humidity%20and%20temperature%20sensor%20AM2302.pdf
* your proposed code above for translating the DHT22 answer is working (tested on my device). But take care, there is one incorrect line left `bytes[j] = bytes[j] + mask` (the s are missing)
* some timings are realized differently in the arduino code in comparison to this lib. But it seems to work (at least for me).


It would be useful to get an explanation in the commit message here.


Right you are! Sorry I was being slow...


Два твоих метода делают одно и то же на двух разных входных параметрах. Почитай про параметризованные тесты (JUnit Parametrized Tests) и сделай так, чтобы у тебя остался только один метод, который запустился бы два раза с двумя разными наборами входных данных.


이거 왜 없어졌지?


Коммент-то этот можно удалить уже )


This works great for the icon thing. But you're not able to do anything else on other elements. In my example I removed the last paragraph. And maybe someone would like to change the background-color or border-color individually?


I think, the `g_ptr_array_add` is more understandable at the call site.


Нужен тест на сам этот класс. Тебе нужно протестировать, что ты создаёшь правильную результирующую строку.


cv


Thanks , it work well now.
I have execute some cmd:
$ ./OpenFaceCpp wangluodan.jpg
/home/bao/bao/program/c++/openfacecpp/NativeDLib.cpp:247:1: error: Failure in getlargestBB: Unhandled exception: Unable to open file: sample.jpg
FAILURE: 1 out of 4 tests failed (1 failures).
Test time: 1.65 seconds.

$ cp wangluodan.jpg sample.jpg

$ time ./OpenFaceCpp wangluodan.jpg
Success: 4 tests passed.
Test time: 2.38 seconds.

real	0m4.134s
user	0m4.016s
sys	0m0.120s

only aligned one image

$ time ./myOpenFaceCpp wangluodan.jpg

real	0m1.809s
user	0m1.716s
sys	0m0.064s

I don't know why the program need those file named "mean.csv" and "shape_predictor_68_face_landmarks.dat",
What knowledge should I learn to understand them?
I think the time program took is too long , and how can I make it fast?


Ah, got it:
Json
```json
{
  "book": [
    {
      "category": "fiction",
      "author": {"first": "Evelyn"},
    },
    {
      "category": "fiction",
      "author": {"first": "Herman", "last": "Melville"},
    }
  ]
}
```
JsonPath: `$.book[?(@.category == 'fiction')].author.last`
2.0.0 result: com.jayway.jsonpath.PathNotFoundException: No results for path: $['book'][0]['author']['last']
2.1.0 result: `[ "Melville" ]`
That's probably caused by the addition of the `REQUIRE_PROPERTIES` option defaulting to `false`.


Doesn't the iterator end up with null as the last entry? I think it usually does?


Well I'm always reluctant to forcing behavior by default, isn't there a way to disable gpg check when adding the repo as is in YumBackend? That would be IMO better.

btw:

```
       --force-yes
           Force yes; this is a dangerous option that will cause apt to
           continue without prompting if it is doing something potentially
           harmful. It should not be used except in very special situations.
           Using force-yes can potentially destroy your system! Configuration
           Item: APT::Get::force-yes.
```

Anyway if there is no other way around, then we'd have to accept it, or extend the API of `insecure=True`, but that would be IMO unnecessary overhead.


My bad just seen 1.1.0 was not published (or deleted), works fine to me, thank you ;)


Hmm, that is an option at the MBean level if you wanted? The main change I wanted was to the Iterator. We can catch the iterator issue and return "error" if that is the current behaviour? Can you point to a part of the code where the response is returned so I know the right text to use. Also, we can't do that for probe() as its a void.


I'd say `ClientOrSessionHolder` or just `ClientOrSession`


it's minor


There need to be getAllGroupMembers.


Alternatively you can just not print anything in the wrapper and do the printing here. Anyway I don't have any strong preferences, it can stay as it is.


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 Total|||||


CI systemd hooks enable running systemd in an OCI runc/docker container


Perhaps but I think we should have some tests to show this new behaviour
works as expected?

On Tue, Nov 24, 2015 at 1:34 PM, Tom Jenkinson <notifications@github.com>
wrote:

> Doesn't the iterator end up with null as the last entry? I think it
> usually does?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tomjenkinson/narayana/commit/f037fa87f2b638e00115ab9def1a1c5371f71d83#commitcomment-14588656>
> .
>

--
Michael Musgrove
Transactions Team
e: mmusgrov@redhat.com
t: +44 191 243 0870

Registered in England and Wales under Company Registration No. 03798903
Directors: Michael Cunningham (US), Paul Hickey (Ireland), Matt Parson
(US), Charles Peters (US)

Michael Cunningham (US), Charles Peters (US), Matt Parson (US), Michael
O'Neill(Ireland)


Since you can run this hook without docker in runc, no reason to require it


I explicitly removed the joint_name_property_ from your previous commit.
What should be stored in this variable?
All of the joints should create their markers.


Back in my young years when I was part of Mercury Team I was told modifying DOM is the last thing we want to do. Therefore, I suggest reverting your changes and moving the logic you've added in the template to [the right place](https://github.com/Wikia/mercury/blob/ADEN-2598-close-button-counter/front/scripts/main/components/LightboxWrapperComponent.ts#L34) in the component :) Let me know if you want some help from myself here :japanese_ogre:


Is there going to be an update that includes the newest volatility profiles?


ここの変数`new`だとなんだか分からないので、`follow`とかにしますか！


`invalid valid` (two instances)


lgtm


There need to be getAllGroupMembers. And also all other between this one and the last before this one.


Could you factor out the embedded do-while loop into a function?
What do you think about `_extract_enterprise_id`?


common logic: moving clients to other daemon


Да, согласна! Глупо написала, ведь по логике бутстрап - container создан для того чтобы просто сделать общую оболочку с конкретными отступами от краев окна браузера. Исправлю


This broke location auto-detect on my site - pressing 'Autodetect and improve location on save' would no longer geolocate the address.  Tracked it down to this commit in version 0.3.13 and up.
If I delete the above lines of code it works again.

Could this be removed/tweaked?


This commit has many purposes :
* Fix bug introduced by my previous commit 25e3d4daaa481d772d0709673b4048d121fb41b3 : The script wal looping twice over the same file.
* Improves output of the script : the origin folder is added one the first call of installdir. So all output can precise the relative path (to CUKOO_ROOT) of files installed/skipped
* The argument `--monitor` (install monitor binaries) is taken in count. Previously you could only install those binaries with `--all` option.
* Fix a bug which occured on monitor installation : l55 : a symlink is considered as a directory by `os.path.isdir(srcpath)` [Python doc](https://docs.python.org/2/library/os.path.html#os.path.isdir)
     * to get the bug type `./utils/community.py -b 2.0 -f -a` with community.py version before this commit.


If this should ensure that an update is possible, it also needs to verify that `role.Threshold == 1`.


updated 4 fix compilation in cygwin


@automata Did you test this? commander is sometimes a bit tricky to get to parse right


not sure if this might bite uns in the ass with flow, because i had instanced where get_class gave me crap, but i think as long as we completely excempt fluid from the proxy builder it should be fine


We need at least `self.line -= 1` here. Otherwise things get weird when using Up/Down afterwards.


Presumable this broke the tests on Windows. Please close the file properly.


I will add the registers themselves. I already have them in the address table, just forgot to tell you. Also I'll add a version of the glib_board_info_uhal.py which incorporates daq registers info and provides a way to force the daq link up.


Done here 960000


TeamCity TripleA :: UnitTest [Build 202](http://188.177.191.210/viewLog.html?buildId=272&buildTypeId=TripleA_UnitTest&guest=1) is now running


Why not move this constant to a file in `resources/`?


In this commit, you're doing a lot of more things that what stated in the commit subject.


care to elaborate how you can compare IDs, please? (`unit._id > self.parent.tags['latest']._id`)


:+1:  this is visually pleasing


We can simplify this as well by just doing `self.line += 1` and removing all other lines (including `return`). So just delete Lines 124-126.


TeamCity TripleA :: UnitTest [Build 202](http://188.177.191.210/viewLog.html?buildId=272&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 140 Build time: 00:00:32


Makes perfect sense. Thanks, done.


unfortunately that doesn't work. I tried it before but for some reason it barfed.


This commit is empty. So I think this is the result of a bad rebase, isn't it?


Any reason to limit this to 4k instead of something much bigger?


TeamCity TripleA :: UnitTest [Build 203](http://188.177.191.210/viewLog.html?buildId=273&buildTypeId=TripleA_UnitTest&guest=1) is now running


One side note: the logic with closeButtonHidden was added not by me.


TeamCity TripleA :: UnitTest [Build 203](http://188.177.191.210/viewLog.html?buildId=273&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 140 Build time: 00:00:31


Could you document here which section of the spec is not implemented? Thank you.


Checked on commander docs, but hey, maybe it's better to add test for it first.


Oh wow @ldoktor, I've just realized that there's no `boto3` rpm package on Fedora yet. It seems that this plugin will have to wait to get into avocado, unless we package boto3 ourselves and ship it in our internal repos :(

(I'm using `boto3` from pip, BTW).


Not sure about caching because profile could be removed(mean freed) from another thread


Definitely an important fix. :+1:


tu ne dois absolument pas inclure des tests dans un commit


Where is that stated?


I'd create issue


MULTIPLICATOR == somewhat archaic, in English, the more common term is MULTIPLIER


agree


ハッシュで渡しているので、valueの配列になおさないと、キーも含まれてしましました…


Factoring this out is a nice cleanup, however this function is a bit difficult to read: after a sequence of … `== $error` there is suddenly a `!= http.StatusOK`, the most common case. Move the `StatusOK` case to the top of the function perhaps?


Could you swap the operands in the latest condition? It will help to increase the readability.


:smile:


we use google style. I beilive here 4 space is VS artifacts


tu ne dois pas envoyer un commit "work in progress" sur une branche master


Could be a constant


Agreed, fixed in 0ed660dddcbcb9b75e79ec6fa8c8dc846d061821


commit vide ?


Please remove the variable declaration on line 4 too.


I would suggest to use `MakeAbsolutePath` here


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


Could we avoid magic number here? I assume 0x7F is the latest ASCII char, isn't it?


Copypaste!


@maelee DWMR (done with my review). Nice job here. Keep it up!


Hey,

Doesn't build for me ... probably only works in combination with your kernel since there is no official CM klimtlte kernel, correct?


Would be quite interesting to have this test a RubySpec as well :smiley:


May I suggest instead of assigning and `is rw` here, to have a method `Val::Sub::Builtin.inject-parameterlist` that does the assigning to `$!parameterlist` inside the class? Should be as simple as

    method inject-parameterlist($!parameterlist) {}


but your tests is currently covering both the PluginManager and your own plugin class. Covering the PluginManager does not make sense


Is this meant to be here (see line 41?)


Hold for PR#277.

Most of these configuration entries will disappear as soon as PR#277 is merged. -- That's the whole point of that PR.


@sbrunner This is unfortunately very dangerous to introduce pointer to original QgsVectorLayer and has to be avoided.

The correct approach would be that a custom feature filter provider stores any data that it may need. This change may otherwise introduce race conditions and crashes - and others may mistakenly try to use mLayer for more functionality, leading to even more problems.

Please could you update QgsFeatureFilterProvider class so that it does not use QgsVectorLayer directly? There is mLayerID member variable in QgsMapLayerRenderer which may be used to identify a layer if necessary.


Yes, it should be `top`. Fixed in ac4a7f9


@pekohl DWMR. Nice work here. Press ahead!


We need the logging for the output of the survey results - is this causing a problem?


Can you publish this fix on npm?


from BeiYuu


And I cannot set `REQUIRE_PROPERTIES` to `true` because that breaks the scanpath token (No results for path: $['book'][0]['book']).


Let's use the `https://` protocol here.


Its ok.


If we have an exception we throw - there is no return value...


Sorry - makes perfect sense.


les noms de variables doivent avoir une signification


Well - that's not a reason to keep this block :-) I don't see its use...


Can we remove it?


In present term please. (i.e. "Add Revision class to access specific revisions" instead of "Added Revision class...") :-)


Ну так у тебя id commentable  содержится не в поле id, а в answer_id и question_id соответственно.


I was halfway through defending the first point then realised that I may have made a mistake relating to the fact that I initially had a (supposedly unreachable) uniqueness validation on `url_title` which meant that 'save!` failed without the `before_validation` check (but with just the one callback, `save` would have unexpectedly thrown an error), but that's now been withdrawn so `before_validate` may be surplus to requirements (I will check that assumption presently)


Affects #552


Sorry I got a bit confused: it makes perfect sense to leave info or not? Yes sure can add aborted.


I have just pushed two followups:

0b6e0f4da09e073c7445e37e213469d9e70cb562 value-pairs: set token_start at the beginning of the loop in vp_walker_split_name_to_tokens()
c00f7ce270aa66717ddb51a77e7cfca9c0719e1c value-pairs: extract vp_walker_skip_sdata_enterprise_id()


Not all browsers support date input field.

http://caniuse.com/#feat=input-datetime


rtf standard


So we can add back the test of 15-tuples to our test suite?


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


What are those systems booted up with? Note that nspawn works well in many cases without machined, that's what rkt does after all. Just specify --register=no?


My bad, fixed.


adding library for oracle 11g


share my blog bojaper.com tweetriat.com hussainiatsearch.com


hmm, what's the rationale here? why would those not run in qemu?


TeamCity TripleA :: UnitTest [Build 204](http://188.177.191.210/viewLog.html?buildId=274&buildTypeId=TripleA_UnitTest&guest=1) is now running


@jgarboza813 DWMR. Keep pressing ahead!


TeamCity TripleA :: UnitTest [Build 204](http://188.177.191.210/viewLog.html?buildId=274&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 140 Build time: 00:00:33
### Failed tests
```
CashRegister.Test.Unit.dll: CashRegister.Test.Unit.ViewModels.SalesViewModelUnitTest.AbortCommand_SalesController_ClearOrderCalled: <no details avaliable>

CashRegister.Test.Unit.dll: CashRegister.Test.Unit.ViewModels.SalesViewModelUnitTest.PaymentCommand_SalesControllerStart_StartPaymentCalled: <no details avaliable>

```


A few suggestions:

1. I think the special handling of single-element lists is superfluous.

2. I was confused by the comment
`;; We don't use '-sort' because there's no need to copy argument.`
To make it clearer, you may change it to
`;; We don't use '-sort' because there's no need to protect the temporary list against modification.`

3. The docstring and the variable naming should be reworked.
Maybe the function should be renamed to `-sort-by`. Here's a start:

        (defun -sort-by (fn comparator list)
          "Sort LIST using comparison keys that are generated by mapping the list with FN.
          The keys are compared with COMPARATOR. ..."


**Code Quality: Nitpick**

Don't leave commented out code within your codebase.


hmm... this shouldn't be part of this PR :-)


Thanks. Next up is Simplenote.vim.


Oh, I thought this was too specific a case since the general feature is
being able to pass any class any time, but perhaps the Class and
cls.singleton_class cases are interesting enough to have their own specs?


// FIXME: Move this to another class as there is a lot more we should do here


// FIXME: There has to be a way to do this without converting to an array and then back to a collection


syntaktisk sukker: #each work her og indentering?


It will only return "true", you cannot get the trust password value back since we have it hashed.


Integer.valueOf() not needed, Integer is returned by obs.getId()


This requires better documentation. Language of instruction ... what instruction?


same, Integer.valueOf() not needed, Integer is returned by obs.getId()


Where is this `token_split_array` used?


These are the files needed be OpenFace itself - probably just face
templates.

In order to make it (way) faster it needs to be compiled in Release instead
of debug (probably using  cmake -DCMAKE_BUILD_TYPE=Release but I have not
tested this yet.  I will add this in the next commit.

On Tue, Nov 24, 2015 at 3:34 PM, sheepbao <notifications@github.com> wrote:

> Thanks , it work well now.
> I have execute some cmd:
> $ ./OpenFaceCpp wangluodan.jpg
> /home/bao/bao/program/c++/openfacecpp/NativeDLib.cpp:247:1: error: Failure
> in getlargestBB: Unhandled exception: Unable to open file: sample.jpg
> FAILURE: 1 out of 4 tests failed (1 failures).
> Test time: 1.65 seconds.
>
> $ cp wangluodan.jpg sample.jpg
>
> $ time ./OpenFaceCpp wangluodan.jpg
> Success: 4 tests passed.
> Test time: 2.38 seconds.
>
> real 0m4.134s
> user 0m4.016s
> sys 0m0.120s
>
> only aligned one image
>
> $ time ./myOpenFaceCpp wangluodan.jpg
>
> real 0m1.809s
> user 0m1.716s
> sys 0m0.064s
>
> I don't know why the program need those file named "mean.csv" and
> "shape_predictor_68_face_landmarks.dat",
> What knowledge should I learn to understand them?
> I think the time program took is too long , and how can I make it fast?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aybassiouny/OpenFaceCpp/commit/44f8cd7984e0e80deb1a9ecd1525fc11bc54ecf9#commitcomment-14588646>
> .
>


well, actually it returns the slightly odd looking "%!s(bool=true)". I'll send a patch to have it show up as "TRUE" instead.


I often have this dilemma - a spec makes sense for Truffle, as for example it exercises int and long code paths, but not other implementation has two representations of Fixnum like that so it doesn't make sense for anyone else.


**Code Quality: Nitpick**

You don't need to have this line of code. Remove all occurences of commented out code within your codebase.


I made suggestions depending to the usages


I think we can build it and make `EC2` plugin standalone (like html plugin) and disable it on RPM build on Fedora/RHEL.


The reason I merged develop into my branch was because otherwise I could not push on the server (due to code standards issues from poetry module)


**Code Quality: Nitpick**

You also don't need this line of code.


I don't see the point of `1* ` here, but I also see the same is done elsewhere in this code file so we can fix that separately.


Nice refact!

[will you send a new PR for the other layers or you want to push those patches onto this branch? I mean eg.: moving vpstack to a separate file, etc...]


@Annih
Hi, when do you plan to release it as new version on chef supermarket?


Nice!


I'd extract regex to avoid compiling it each time


Awesome!


Because we don't have network there. It's a separate issue we're trying to solve after this. For now, at least a basic bootup check is better than nothing.


@timgraham I refactored the code as you asked to avoid repetitions. the `_varchar_operator_index_sql` method is responsible for the creation of the statement that creates that index.


What's the rationale for using isNull ?


when i open xgboost.sln with visual studio it says it is not a valid solution file. any ideas?


Add "Apply" to translation on Transifex


Good point - I will add this line


Okay! (I meant the language the course was instructed in.)


OK


There's still SCROLBOX (with one 'L') in this code


Should be possible, but the the tests fail on 7.6, see https://travis-ci.org/Feldspar/feldspar-compiler/jobs/92736101

Could you test it on your machine? I do not have a working 7.6 installation ATM.


/api


Wouldn't it be straightforward to replace the featurefilterprovider with an implementation that sets the filter on the renderer instead of providing this callback?


利用Socket类和ServerSocket类编写一个C/S程序，实现C/S通信。客户端向服务器端发送Time命令，服务器端接受到该字符串后将服务器端当前时间返回给客户端；客户端向服务器端发送Exit命令，服务器端向客户端返回“Bye”后退出；服务器端和客户端都需要打印出接受到的消息和发出的命令。


I think it was that in changing the behaviour of `url_title` on save, I'd caused the spec to fail as the original test assumed that the title would match that of the original fixture (which IIRC is too long). The new code is forcing the issue, but probably not in a helpful way.


la función de getNeuron retorna una promesa (return $http)then(function(success}{}, function(error){})) y el success asignas lo que te devolvió... usa activate() (guía)


See line 954, above, and commit be56a3829b83015848f0f4d8976b7c1c43874f73. It's there already.


**Code Quality: Nitpick**

You need to remove all these trailing whitespace. Not needed here.


ok


Right, since it's been two months, but try to prevent it from happening in the future. It makes the reworking more work and in the meanwhile the code is less clear.


How about this writing style for `PoliticalDivision` model
```python
 pd_arta = mommy.make(pd, name="Arta",
                      parent=mommy.make(pd, name="Epirus",
                      parent=mommy.make(pd, name="Greece")))
```


**Code Quality: Nitpick**

This file should be renamed.


`AUTH_TOKEN_IGNORE_PORTLETS` is a wrong one, we need `portlet.add.default.resource.check.whitelist`.

Beside that, the tracker for `AUTH_TOKEN_IGNORE_ACTIONS` is no longer important, we don't have almost any struts actions and moving towards MVCPortlet, which, on the other hand, is missing support for actions to be whitelisted.

Also the reset is wrong, we cannot reset AuthTokenWhitelist from HookHotDeployListener because now it's filled from OSGi and not only by PropsValues.

I need to fix it myself, sorry I thought it's in better state :(


**Code Quality: Nitpick**

Any reason why this space is here?


What´s the use of this file, please specify


Review indentation, http://www.php-fig.org/psr/psr-2/


Very strange... And dangerous... Are you sure? Usually singleton is created once (lazily or at startup) and live until the app end...


It's not really clear what the distinct "theora" and "theoralib" switches do, as they have the same description.


`--register=no` solves that [problem](https://github.com/systemd/systemd/pull/1991)
I can add `--register=no` and remove `[[ -e /lib/systemd/systemd-machined ]]`.
Does it make sense?


Looks like this change should be extracted from this commit


Undocumented method!!!
http://www.php-fig.org/psr/psr-2/


Undocumented class and methods!! http://www.php-fig.org/psr/psr-2/


Are you scarry?))) Our singleton has public method called `destroy` so you have no any guarntees


I didn't split the files, at least not yet.

I didn't want to push on it too far without feedback, so I wanted to get this merged first.

I would get back to value-pairs later, especially as I want to speed it up. Performance of value-pairs is a real world issue (see posts by @faxm0dem, also I think the QA team also found perf issues with $(format-json) and probably the biggest issue lies in value-pairs.


See #1


Probably it should be in another commit


I'm not sure what the question is.  For me I'm using C++.  All the tests work correctly.  Just when I build from my project I get the compile problem.