TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 833](https://teamcity.labkey.org/viewLog.html?buildId=290368&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 31 (8 new), passed: 250 Build time: 01:29:36
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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


That will affect the has helper though - as that works based on name. I think that's really confusing? Not really sure what you mean by treating it the same.


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 834](https://teamcity.labkey.org/viewLog.html?buildId=290352&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30 (7 new), passed: 251 Build time: 01:29:03
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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


fast-forwarding end-to-end/timeseries to borshop-integration-1292-fix/az/test-for-ranges-spanning-exact-multiples-of-quanta = 42044460


saw approval from jonmeredith
at https://github.com/basho/riak_kv/commit/e16c9987f0ca5b8b94576145166a230a119ed060


merging basho/riak_kv/unit/jrd/test-quanta-boundary = e16c9987 into borshop-integration-1299-unit/jrd/test-quanta-boundary


merging basho/riak_kv/unit/jrd/test-quanta-boundary = e16c9987 into borshop-integration-1299-unit/jrd/test-quanta-boundary failed


Okay then :p


"The parsed information will then be passed our central module..."

Specifying the actual central module (in parenthesis) might be helpful.


I'm not sure if they are fixed -  perhaps they are auto negotiated during system startup.
Perhaps later I will change that (once I figure out how exactly it is working)


Updated. See https://github.com/systemd/systemd/pull/2108


I do not like CSS yet.


Slm chejori mishe  in haro download konam??


If you are interested in making the library better just do that. Accusing the people who wrote it has no merit especially when you know they are not present to reply, and you are using this code with no investment (time or money). As of why the mutex code was removed, was because this library is not thread safe by design. Adding a mutex lock for a single operation when nothing is thread safe is pointless.


Base addresses are taken from device tree, reg offsets stay here for now (we will see how rfc goes).


I will collect the number later today.


This doesn't work.  The problem is that `packages/newton` doesn't depend on the sky_engine, which confuses the `flutter test` command.  We can teach `flutter test` how to handle that case, but we haven't yet.  We usually run the tests for Newton in the `dart` command line tool, which is why the previous documentation said to use `pub run test`


nope


why are your documentation comments all *so* long


We're going to see issues encountered one way or another. I have just taken some of the error checking from the code and made it the developer's responsibility.

This isn't unprecedented since this is how mock data is generated in web environments. You create a script to run queries against the database itself. It decouples the test data generation totally from server/client/maintenance/job runner code which is what you want so that when the code for any of those changes, all of a sudden you don't need to change the data generation code to be able to test your other code.

And making sure that that developer knows what they are doing is not the code's responsibility, it's the developer's responsibility. We're not the clients, we shouldn't just know the "what it does" part, we should know the "how it does it" too, arguably better than anything else.


`s/working on the flutter package itself/working on the flutter repository itself/`

The `--flutter-repo` flag isn't just about the flutter package.  It's about the whole repo.  For example, if `flutter_sprites` had tests, it would run those tests too.


Should we add a description about how to run tests against a locally-built engine?
Also, should we add a description about how to run a single test?


In retrospect, this would have been easier and better to do with `git revert`. Oh well.


Excellent test coverage!
I'd like to see if we can use https://github.com/Legitcode/tests for most of our testing needs. Unless you really feel like testing it out, let's keep this suite using `TestUtils` directly, but bias towards using the more readable/usable `LegitCode` library if we can.


far out


What if error is nil? Same comment throughout.


Didn’t localize this one.


I'd advise for shorter tests for such simple classes, it's just a lot of work *and* when we want to change the class it's also much work. Especially since this class is merely a data container there's nothing to test, I *really* think you should work on shortening your stuff (docs and tests) a bit, you're overdoing them so much they actually get useless and hindering :/


ack


Doesn't this do exactly the same? We still don't catch these exceptions, so the result should be the same. https://media.ilexiconn.net/mi4C0dV.png


ok, I have changed it


@Zoschi there's gotta be a better way of doing this? ...


maybe squash to avoid that problems?
in PR history you are changing .travis.yml file a lot ;) after squash there would be change only once, so less chances to wrongly resolve the conflict


Oh OK thought it was arm..


Requiring the 2.91 version of Vte causes issues on Ubuntu 14.04 as it has only 2.90 available. I tried to change the serialcon.py:32 to be:

gi.require_version("Vte", "2.90")

and it works perfectly.


dpkg-deb: warning: deprecated compression type 'lzma'; use xz instead

Should it be ignored? Or is there any fix for this?


const is called "module root" but the string says "application root" - can you rename the string please?


Done!

On Tue, Dec 8, 2015 at 4:19 PM Gustavo <notifications@github.com> wrote:

> I think you can remove it for now. We can revisit this once more fields
> are added.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/googleads/aw-reporting/commit/9dd44ca8425ec7c0bd91cba69b7d531959dbd8d4#commitcomment-14858990>
> .
>
--

Allison Greene | Developer Advocate | allig@google.com | 201-913-0277


Thanks, we'll take a look at this and pull it in.  Cheers!


Having checked commons-ip-math's source, I have to say I'm unimpressed at best.  e.g. Ipv4 encapsulates a Long, while Ipv6 a BigInteger? The only way it could be worse is by using String in there.

Why was this weird library used instead of the existing and superior IP*Resource that are already present in this very repository?


I'm not sure this test is needed, looking at other strategy_test.go files in the code base.


yay


Thanks, I'll take this commit (with one small change).


Just edited the file.


nice


S3QL is able operate in a "subdir" - however, it calls it a "prefix".


`refresh_time` is a property of the provider, not the account.  Also, there should be two tests, one for each branch of the conditional:

````
return (self.account.expires_at - datetime.datetime.utcnow()).total_seconds() < self.refresh_time
````


Added Megan


nice. thanks.


no crosswalk, android 5.1, all android libs updated to latest


You are right.  That is a great catch, thanks!  I fixed it in 29e323ab7e33576c6dd4b270decf011b59210d35 in the `1.0-final` branch.


Yeah we should catch the exception. But before we didn't even check if we could cast. Which would let the jvm crash instead of crashing our program.


Hmm, stuff like "<HintPath>..\..\..\BikeSensorApp\" this should be removed at best.

While I do appreciate C#6 features several projects use VisualStudio 2013 where C#6 does not come out of the box.


not strictly necessary, but good to have in case we change some logic down there. Otherwise it'll be hard to debug.


So should hidden tags have a hash added to their name on the server so that hidden tags have to referenced with a # everywhere?

All this PR does is change the display of hidden tags in the autocomplete list so that it's possible to see/search/add them. Eg. if you wanted to add/create a "Needs Review" hidden tag from the editor you would type "#Needs Review", then if you went to the tags management screen it would be named "Needs Review" and be the hidden checkbox would be checked.

If we want to extend the # naming throughout then toggling the "mark as hidden" checkbox will also need to toggle the name on the server side so that naming is enforced.


By the way, just curious, what are you developing with glTF?


Yeah, I'm just exploring and progressing with it, and I'm probably continuing with this approach. It's more the "concept" itself of s3ql, that I didn't get at first: it's a file system, not an "explorer", like hubicfuse or Swift-explorer


This doesn't look like it would handle multiple flags being set very well.


Is there a reason that tests are being put in `server/__tests__` as opposed to the tests folder?


Thanks for the report. Care to send a pull-request?


I don't think having six capital letters in a row improves readability. In particular we're accumulating a lot of 'BB' here, and I would suggest we always use 'Block' instead (similarly getBlockState instead of getBBState).

Is there a reason this can't just be BlockState, defined in an anonymous namespace?

That might hurt grepability, but I think RLEBBState hurts readability which is more important.


Yes I'm still using the email, however my github filter had stolen it.


Also here `-Ddeploymentprofile=cloudhub` must be configured

It would be nice to have a more deep explanation of what means each parameter.
* **muleversion**: Configure the mule runtime to be used in cloudhub
* **cloudhuburl**: (all CTF parameters are in lowercase) Gives the capability of change the CloudHub instance, by default the CTF deploys to *http://anypoint.mulesoft.com/cloudhub/*
* **cloudhubworkertype**: Gives the capability of configure the amount of vcores for the Mule App to be generated. The available options are: **micro** (0.1vCores), **small** (0.2vCores), **medium** (1vCore), **large** (2vCores), **xLarger** (4vCores)
* **cloudhubtimeout**: Gives the capability of configure the amount of type to timeout in the CloudHub Mule app deployment


> S3QL is able operate in a "subdir" - however, it calls it a "prefix".

Sorry.


Yes, then you'll need to add it to `mutex_tryenter()`.


The hostname prefix must always be valid, otherwise an invalid hostname will be set in /etc/config/system before the config mode is run. Please don't recommend such broken hacks, a much better solution would be to replace the hostname field in the config mode to do something different.


Shouldn't this also be used for `as.data.frame.grouped_df()`?


Hm, can't we have default func which would send `SIGTERM`

```go
var DestructFn = func(ctx context.Context) bool {
  proc, err := os.FindProcess(os.Getpid())
  if err != nil {
      return false
  }

  if err := proc.Signal(syscall.SIGTERM); err != nil {
      return false
  }

  return true
}
```

and let users change it using:

```go
selfdestruct.DestructFn = func(ctx context.Context) bool {
 // they can do whatever else
}
```


I was thinking the IP aliases to be part of `ipamapi.IPAMData`


Really looking forward to the next release


I'm trying to fight optional args/funcs where possible. It's not idiomatic go.


Don't know what to tell you.

On Tue, Dec 8, 2015 at 3:02 PM, Reshama <notifications@github.com> wrote:

> hi @JordanGoodspeed <https://github.com/JordanGoodspeed>
> interesting, your emoji did not render.
> [image: :grin:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JordanGoodspeed/dsp/commit/36b491b2d3d92dbde2225a6bf79aee0f22b0df7e#commitcomment-14857078>
> .
>


The backbone group must be in `groups`, the `bandwith_limit` is on the same level as `groups`.


Nit, "MoreObjects." not necessary (already imported - here and in the other subclasses).


Severity	Code	Description	Project	File	Line	Suppression State
Error	CS1061	'GetChocolatey' does not contain a definition for 'ListCount' and no extension method 'ListCount' accepting a first argument of type 'GetChocolatey' could be found (are you missing a using directive or an assembly reference?)	ChocolateyGui


109649198_EvapCoolers (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (910 of 910 tests passed)

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


I don't really see a problem with Entenhausen, as the name doesn't matter at all, but if we really want to change it, I agree with @jplitza.


unack, this sentence contains *exactly* the same info as 124 and 125, drop one of them please


Nit, empty line.


I'll fix this after merging so as not to hold things up, but we definitely need to clean up before this rather than after.


Still crashes:

Error: Cannot pass parameter 2 by reference
File: /src/pocketmine/block/Door
Line: 214
Type: E_ERROR

Code:
[205]}
[206]}
[207]
[208]return $bb;
[209]   }
[210]
[211]   public function onUpdate($type){
[212]if($type === Level::BLOCK_UPDATE_NORMAL){
[213]if(!$this->isActivitedByRedstone()){
[214]$this->getLevel()->useItemOn($this, Item::AIR, 0);
[215]}
[216]$blockNorth = $this->getSide(2); //Gets the blocks around them
[217]$blockSouth = $this->getSide(3);
[218]$blockEast = $this->getSide(5);
[219]$blockWest = $this->getSide(4); //Make redstone activation
[220]if($this->getSide(0)->getId() === self::AIR){ //Replace with common break method
[221]$this->getLevel()->setBlock($this->getSide(0), new Air(), false);
[222]if($this->getSide(1) instanceof Door){
[223]$this->getLevel()->setBlock($this->getSide(1), new Air(), false);
[224]}


@pschiffe, is there a way MAXSMP can be determined from the system environment at build or run time?


how about just
```
Loads all settings inside the according coalang-files as markers.
```


Is there any corresponding _ng-model_ ?


It was pointed out by 岡本.


where is command_line freed?


Yea.. Its useItemOn. I'll just comment it out

> Am 08.12.2015 um 23:07 schrieb flaxues <notifications@github.com>:
>
> Still crashes:
>
> Error: Cannot pass parameter 2 by reference
> File: /src/pocketmine/block/Door
> Line: 214
> Type: E_ERROR
>
> Code:
> [205]}
> [206]}
> [207]
> [208]return $bb;
> [209] }
> [210]
> [211] public function onUpdate($type){
> [212]if($type === Level::BLOCK_UPDATE_NORMAL){
> [213]if(!$this->isActivitedByRedstone()){
> [214]$this->getLevel()->useItemOn($this, Item::AIR, 0);
> [215]}
> [216]$blockNorth = $this->getSide(2); //Gets the blocks around them
> [217]$blockSouth = $this->getSide(3);
> [218]$blockEast = $this->getSide(5);
> [219]$blockWest = $this->getSide(4); //Make redstone activation
> [220]if($this->getSide(0)->getId() === self::AIR){ //Replace with common break method
> [221]$this->getLevel()->setBlock($this->getSide(0), new Air(), false);
> [222]if($this->getSide(1) instanceof Door){
> [223]$this->getLevel()->setBlock($this->getSide(1), new Air(), false);
> [224]}
>
> —
> Reply to this email directly or view it on GitHub.
>


use `e.g.` instead of `for example` please


@eeue56 published as 2.3.0 after making this change ^


800 is much to high, 200 should be fine for small networks. Of course, the required upload depends on background noise, which depends on the size of the network... but let's hope we can get rid of batman-adv soon, so the background noise will be much lower.


it's quite obvious that CPP is C++ and PYTHON3 is python 3, remove the examples please


You're right, python -v gives all kinds of strange responses.  Thanks for
that.

On Tue, Dec 8, 2015 at 4:01 PM, Reshama <notifications@github.com> wrote:

> hi @JordanGoodspeed <https://github.com/JordanGoodspeed>
> good job! [image: :+1:]
>
> To see what version of python is running:
>
> $ python -V
> $ python --version
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JordanGoodspeed/dsp/commit/88b0a44612cc3c22605605e558726111c9eb5bf2#commitcomment-14858535>
> .
>


I don't understand your point here. in `getAltNameFromName` (i am renaming the getHipchatNameFromName function) I check that the user exists AND for a hipchat name, since in production nobody currently has hipchat names.

I also think it's a bad idea to complain more about finding a user in a blob that doesn't exist. That means if someone is deleted we would have to deal with the complaints for all older ciphertexts.

Since this is messaging hipchat, I wanted their hipchat name but since some users might never add a hipchat name I wanted to fallback to their real redoctober name.


I think it makes sense for a lot of them, so I'd definitely want to look into that.


@jrose-apple Regarding the devirtualization, it should be easy to have IRGen fall back to static dispatch if the class containing the method we're delegating to is in the same resilience domain, by checking IRGenModule::isResilient(). If @bitjammer doesn't get around to it, I will when I do ivar layout for classes with resilient field types.


don't do an example for lowering, it's a bit like:

```
Please pass an integer here (for example you can create this with int(your_value))
```


e.g.


remove trivial example


arregle eso que estaba malo


Gonna try to release at the end of the week.


It looks like _modalWindow_. I was using something similar in Java Swing where I defined custom Modal Dialogs


```
:raises FileNotFoundError: If the given docstyle is unavailable; from compatability module.
```


do you mean that the aliases would be provided by the ipam driver, or that the aliases should also be stored in the IPAMData ? In this prototype, I do not allocate the aliases as they may have been already allocated (e.g. in the case of a VIP) and may not be in a network managed in docker.


Do you still need to check `config.attachement` here - you set it to `{}` if it is not present anyway?


In other words, to have them as explicit first class UI options.
For example the upcoming `--ip` (and `--ip6`) option can specify one or many comma separated ip addresses.
Client code will put the first address as the preferred one, and subsequent ones (if any) in the Aliases in a new `IPAMEndpointConfig`.


Prediction API v1.5 has been deprecated, and the latest version v1.6. Maybe you could create a sample for v1.6?


_Bool, eh?


There was a misspelling here, the word was Ressusitação instead of Ressuscitação.


hey, i can comment stuff?


Static import


This commit was moved off master and onto https://github.com/tomhanderson/zf-apigility-doctrine/tree/hotfix/zend-hydrator.  This was done before an official release was made.


Prediction API v1.5 has been deprecated, and the latest version is v1.6. Maybe you could create a sample for v1.6 instead?


This commit was moved off master and onto https://github.com/tomhanderson/zf-apigility-doctrine/tree/hotfix/zend-hydrator. This was done before an official release was made.


On Tue, Dec 08, 2015 at 08:59:46AM -0800, Dmitrii Pasechnik wrote:
>    There are more pedestrian concepts (basically, simple OO) of Knowledge
>    representation in Sage than categories; things like groups., matrix.,
>    graphs., which are quite useful in practice, and which Sage-aware
>    people take for granted, but for outsiders might be a revelation. They
>    ought to be covered...

That's totally right; in fact the categories just form the "abstract
classes" chunk of the above, and I usually start my talk along the
lines of:

http://combinat.sagemath.org/doc/thematic_tutorials/demo-modelling-mathematics.html#demo-modelling-mathematics

So, let's fix the title I suggested on the page, as it's
misleading. Maybe «Modeling mathematics in SageMath»?

Cheers,
				Nicolas


Good catch here. I noticed this but had not gotten around to hunting down the issue causing it.


that's an awful lot of code marked "static inline"... What's the point?


Not sure if astyle would handle those "ternary trees" properly. Didn't see any examples in the code.
It would be safer (while still readable) to create an array of pairs like `{{0,1,1,0,0,1,0,0}, 7}` and then iterate over this array until you find a matching key (the array with the same values).

It would be much more readable if the numbers on the right (I assume they represent directions) were values of an enum instead. And the conversion to rotation (`plaza_dir % 4`) was a function, like `direction_to_rotation` or something like that.


Could you add some more docs to this package?

I found it on godoc.org but it's not obvious how to use it, at least to me.

In my case, I want to decode images coming from an h264 video stream from a security camera.

What is a `nal` in https://godoc.org/github.com/opennota/h264#Decoder.Decode ?

Thanks!


So, are we planning to add C++ code to systemd, or what? ;-)


Same thing with the "ternary trees" here. Not sure if astyle likes those.


This is *too* late now; but can we do "fast forward only" (``--ff-only``) merges please?


Sarah - Myers! 'Contrary to common belief, we believe...' For shame! Change 'belief or 'believe' to a suitable synonym to avoid tautology. Call yourself an English teacher...


This could be `& Integer.MAX_VALUE`


that's right


Will PR some doc changes tomorrow


Closed Issue #4, closed Issue #8, and closed Issue #9


Remove?


Isn't error guaranteed from all these methods?


Shouldn't these be uppercase?


It is possible to have sufficient permission to overwrite a file but *not* have permission to chown it.  In that case, this change causes copySync to fail when it ought to succeed.  See #199


Maybe add a link here?


That looks rad.I'd like to get this merged as is, but I definitely want to check that out before I write too many more tests


I am thinking aliases will be specified by user in the `docker run --ip=[]` UI.

If the intent is to avoid libnetwork to declare failure when an allocation of an alias IP cannot succeed (address already allocated for another endpoint), then maybe this virtual IP can be passed as auxiliary address, pretty much as we do for network, and have libnetwork disregard an allocation failure on an auxiliary address ?


Does it say that somewhere?

Also, it looks like your indentation is off by one space.


Hehe.. i still did that deliberately. Just add the -s option to the usage-line..


hi @shermanash
Ok for Vim! :+1:
(I only know how to use Emacs.)


okay, but i think i have to try this way as well, I want to know about all possible solution, you can find this way in my commits :)


Do you have a pointer to the code for this feature ?


with default settings, astyle leaves ternary constructs like that exactly as I've formatted them.

changing the numbers on the right to an enum is a job for another patch. 4 and 8 entry `_nesw` arrays exist throughout the codebase. I'd probably like to convert them all to a templated class as part of my next effort touching this part of the code.


Feel free to close #784 once merged


Nah none of that is necessary imo. The hash, if added, automatically makes the tag hidden. That's it for magic. No need to hide or remove the hash, or enforce it for all hidden tags


hi @BryanBumgardner
We're on the same page!  :+1:

I use Emacs (it's the only one I know), and donating to Sublime is on my list!  I will add it to my:  "Before New Year's Resolution list"


I believe this is actually slower @kirillseva


Oh I thought I removed that already ...


Try on 453, okay?

> Am 08.12.2015 um 23:07 schrieb flaxues <notifications@github.com>:
>
> Still crashes:
>
> Error: Cannot pass parameter 2 by reference
> File: /src/pocketmine/block/Door
> Line: 214
> Type: E_ERROR
>
> Code:
> [205]}
> [206]}
> [207]
> [208]return $bb;
> [209] }
> [210]
> [211] public function onUpdate($type){
> [212]if($type === Level::BLOCK_UPDATE_NORMAL){
> [213]if(!$this->isActivitedByRedstone()){
> [214]$this->getLevel()->useItemOn($this, Item::AIR, 0);
> [215]}
> [216]$blockNorth = $this->getSide(2); //Gets the blocks around them
> [217]$blockSouth = $this->getSide(3);
> [218]$blockEast = $this->getSide(5);
> [219]$blockWest = $this->getSide(4); //Make redstone activation
> [220]if($this->getSide(0)->getId() === self::AIR){ //Replace with common break method
> [221]$this->getLevel()->setBlock($this->getSide(0), new Air(), false);
> [222]if($this->getSide(1) instanceof Door){
> [223]$this->getLevel()->setBlock($this->getSide(1), new Air(), false);
> [224]}
>
> —
> Reply to this email directly or view it on GitHub.
>


Le 2 décembre 2015 à 22:07, Bako kader <kaderbako7@gmail.com> a écrit :

> daccord..non je pensais que cetais ton numero car jai perdu mon cellulaire
> donc jai enregistrer le numero de ta femme pensant que cest la tienne je
> suis desolé..
>
> Le 2 décembre 2015 21:21, Alibel2011 <notifications@github.com> a écrit :
>
>> salut Kader,Je me rapelle pas cetait juste qu'il a demande apres toi puis
>> il a dit qu"on doit faire le formuaire et commencer au plus vite.Voici mon
>> numero 5142341615, Tu l"as normalement, priere de ne plus appeler ma
>> femme.bonne nuit.
>>
>> Date: Wed, 2 Dec 2015 17:40:22 -0800
>> From: notifications@github.com
>> To: tp_ordiplus@noreply.github.com
>> CC: alibel2011@hotmail.fr
>> Subject: Re: [tp_ordiplus] Ajout .gitignore (089dec2)
>>
>> salut.j'ai essayer de t'appeler mais rien.Que dois-je faire
>>
>>
>>
>> 2015-12-02 12:57 GMT-05:00 Alibel2011 <notifications@github.com>:
>>
>>
>>
>> > ajout gitignore
>>
>> >
>>
>> > —
>>
>> > Reply to this email directly or view it on GitHub
>>
>> > <
>> https://github.com/Alibel2011/tp_ordiplus/commit/089dec23af670ad787b4551ad4478e9616385b3c#commitcomment-14742441
>> >
>>
>> > .
>>
>>
>> >
>>
>>
>>
>> —
>> Reply to this email directly or view it on GitHub.
>>
>>
>>
>>
>>
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Alibel2011/tp_ordiplus/commit/089dec23af670ad787b4551ad4478e9616385b3c#commitcomment-14751900>
>> .
>>
>
>


Add `@SuppressWarnings("MarkerInterface")`


Yes it works now, I fixed it by putting it in front of yours, thanks.
On Dec 8, 2015 1:38 AM, "Flavius12" <notifications@github.com> wrote:

> Your code seems ok! It should work
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EvolSoft/PerWorldChat/commit/8cc446184eaf5a3bc0f4afd93f3465f72f74458c#commitcomment-14842581>
> .
>


I like to keep this, an external would now think "where's the compatability module?" and would search for hours^^


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


With regard to passing them as auxiliary, this would solve the problem when the aliases are part of the network passed in --net but what if they do not belong to this network ?


Great.. thank you! I probably pressed the wrong key during vim command-mode :D


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


Good but unrelated change


+1 on not catching KeyboardInterupt at this level.  The use case @danielvarga is a good one.  There is also a common pattern when using partial_fit which doesn't work well:

```
for x, y in network_streaming_data():
    net.partial_fit(x, y)
    # log status
```

In this case, `^C` does nothing.  `^C` twice does nothing.  I don't want to kill the whole process and lose the parameters and in memory cache.


Would be nice to make this a method.


@gribozavr @nadavrot A bit :cry: that my commit got reverted but :smile: that it helped spot an issue. Hurray for OSS :). Thanks again


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


Not yet, sorry.

Change would affect this function
https://github.com/docker/libnetwork/blob/master/endpoint.go#L782

It would now have to also process the `ep.iface.ipAliases` and reserve them with IPAM.


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


Async please


q


@bradfitz This is a fork. The original repo is at https://github.com/nareix/codec

NAL unit is an [network abstraction layer unit](https://en.wikipedia.org/wiki/Network_Abstraction_Layer).


does this make `this.agentProxy` redundant?


Yes ? Not sure I follow.. ROHost is only used for construction of a URL that can link to a redoctober instance. We could set it to example.com but the URLs that get send to hipchat (the URLs auto-populate some fields), will not work.

There are no requests sent to andrewbuss


@chuckwondo
Re-reviewing since your last submission.
Ok with Vim (I only know Emacs.)
PyCharm is great, and with lots of features! :+1:


lgtm


Nice :+1:


Still not sure I'm following how you want the hash to work, perhaps it would be easier to go over on Slack?


complete.


r is already initialized here, the "r = -errno" needs to be dropped..


Nevermind...it's async.


https://developer.apple.com/library/mac/documentation/Cocoa/Reference/Foundation/Classes/NSData_Class/#//apple_ref/occ/clm/NSData/dataWithContentsOfFile:options:error:

https://developer.apple.com/library/ios/documentation/Foundation/Reference/NSJSONSerialization_Class/#//apple_ref/occ/clm/NSJSONSerialization/JSONObjectWithData:options:error:

And I am personally guaranteeing the last one :)

But is it preferably to add a check for extra safety?


Done


Maybe the server name and priority should rather be on the RemoteDescriptorProvider interface than here. It is going to be the same for all the descriptors of the same descriptor provider I think?


Per #4010, maybe this should use `_type`?


Oh yes, I forgot (my own suggestion).


Hi @uramoty, nice to meet you too.

If you have an issue with one of the examples, please create an issue for it so we can properly track it: https://github.com/dropbox/dropbox-sdk-java/issues.

Thanks!


Why is it safe to ignore? When would this happen?


@mnav Thank you very much for merging my pull requests.  I don't blame you.  I just asked for the rationale of a design decision.  If you meant "I don't have time to write it properly" you should have written the truth, instead of blaming the design.

**I hereby declare my readiness to invest my time to improve the OpenSC project.**  In fact I am already working on it full time for a month or so (not only reading/testing the code, but also submitting fixes).  Would you be so kind to accept me as a project member?

You probably heard of my main project: https://www.stunnel.org/ (thread-safe since year 2000)


Does it make sense to set a new provider? A provider is a kind of factory, giving an object a new factory does not change anything because it has already been created.


Done, done


Is it clearer to define it as a variable or just inline `this._currentScale();`


I guess it should ```extends DescriptorProvider``` to show the affiliation?


Also, to decode a stream from an IP camera, you need an RTP protocol decoder, which isn't a part of this project. I wrote one myself, but never published it.


I think the previous way with `value` and a single line made this easier to read


Leave a comment saying that the error parameter will be non-nil if an error occurs. You can’t typically assume that.


he could ask us or `git grep` or look at the imports above, if he has an
IDE he has it in seconds anyway

2015-12-08 23:24 GMT+01:00 Mischa Krüger <notifications@github.com>:

> I like to keep this, an external would now think "where's the
> compatability module?" and would search for hours^^
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/0868cb6dd7da4d4d521cb13f3ed4b560422d7aa5#commitcomment-14860334>
> .
>


Can we toggle the star without waiting for pref save for smooth ui?


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


does it not work for you ? note that these are overload for char, so in utf8 that should never be an accentuated character, only and ascii one, 2 lines upper you got the unicode versions, that use the tow{upper,lower} functions and should give you the good result provided your locale is unicode.


Ups my bad. Thanks @arioch, I should have checked the most recent commits.

Btw the latest version available on https://forge.puppetlabs.com/arioch/redis is the 1.2 and doesn't have the fix. Could we tag a new version with the fix?

Thanks


Hi Patsy,

I added the sticky header. Please take a look.

Thanks,
Madhuri


Is it possible to use directive as a main tag like <modal></modal> is used here ?


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


leftover from a previous commit. turned it back into just ca.


Yes, it may be wrong, but my idea was that endpoint alias addresses must belong to one of the network's subnets.
If that makes sense, then the virtual IP should not be passed as alias address, rather via a specific virtual address field ?

Libnetwork needs to be aware of this address, it cannot be an opaque option sent down to the driver, because my understanding is that libnetwork osl package will program the alias addresses and this specific address in the sandbox.

Another question, is there any assumption about the virtual IP, like it is the same for all the endpoint in this network ?


Yup, that's what I thought too. Maybe I can squash first then rebase. Thanks anyways.


This is a good place to return.


This is too simple I fear... PTR RRs exist in two contexts: for doing reverse address lookups and for the DNS-SD discovery. In the former case they are unique, in the latter shared... This means you cannot derive this from the type alone...

Is the cache_flush bool you added to DnsResourceKey not actually the same thing as this?


You also need an RTSP client to initiate the stream.


That would help


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 835](https://teamcity.labkey.org/viewLog.html?buildId=290380&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 31 (9 new), passed: 250 Build time: 01:39:39
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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


Most of this needs removing.


Awesome, thanks - I forgot I added the `if` guard in there.


https://en.wikipedia.org/wiki/Flying_Spaghetti_Monster tis spaghetti monster?


https://github.com/CleverRaven/Cataclysm-DDA/blob/master/CONTRIBUTING.md
has this line:
```
astyle --style=1tbs --indent=spaces=4 --align-pointer=name --max-code-length=100 --break-after-logical --indent-classes --indent-switches --indent-preprocessor --indent-col1-comments --min-conditional-indent=0 --pad-oper --add-brackets --convert-tabs
```

If it works with those settings then it's OK, I guess.


I see. Yes. I worked around this problem for now.
If you see in https://github.com/jc-m/docker/commit/636aa789639419392ee3889cee1017ebf98608e5 I'm now just passing the value in a label to test. The placeholder for the value is in the genericOption, but could definitely be elsewhere if it makes more sense.


Hmm, isn't this leaking the old timeout_event_source object of that transaction?


BTW: I already started working on it: https://github.com/mtrojnar/libp11/commit/44183fadd654e57e7a7279c79f83d7606e516241 (it's an experimental code).


I can't read the output here well enough to verify that this is still correct. @nadavrot who would be a good person to review this patch?


Also `_calculateScaleStep` seems to only take 1 param?


I think create-user is good to be next to Create. Not that important


was ist das


No, you never import setuptools-scm at the setup.py top-level, just put a dict with the arguments for get-version into the use-scm-version value


Is this the change for initial minimum scale?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1109.d803e3325b8b20c04fcad9c56fb5dff6f6ca5ddd](http://build.palaso.org/viewLog.html?buildId=67039&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout; tests passed: 501, ignored: 4 Build time: 00:35:41


Good luck! :sparkles:


Filed https://bugs.swift.org/browse/SR-148 to fix this; @gottesmm or @trentxintong, mind taking a look?


@hamin , The swift standard library is special, and we can't allow performance regressions for small improvements to the readability of the code (there is an obvious tradeoff here).

I see two options moving forward:

1. Wait for the optimizer to improve and reapply this patch as is.
2. If we absolutely need to delete c-style for loops we can re-implement them as while loops.


Package Control was not displaying the message: `messages.json` was just erroneously named `message.json` (the “s” was missing).

Additionally, using a title in message files is unnecessary, as Package Control already adds one.


@eeckstein is the best person to review this code.


Am I missing something here? This loop never checks whether the cache entry matches any of the queries in the packet... Known answer suppression means you add the answers to your questions you already know to the message, but not everything you know...


:D


complete


:+1: consistency :) I recommend two things:

- [github cli](https://hub.github.com/)
- A *better* ``$HOME/.gitconfig`` e.g:

```#!bash
$ cat .gitconfig
[push]
    default = simple

[alias]
    co = checkout
    ci = commit
    st = status
    br = branch
    hist = log --pretty=format:\"%h %ad | %s%d [%an]\" --graph --date=short
    type = cat-file -t
    dump = cat-file -p
    pick = cherry-pick
    merge = merge --ff-only
```


Thank you! Inspired by you + @una :)


Given history is now at 1.15.0 why is this pinned at 1.13?


Yep, works great there. Only quibble is that it doesn't like it if I indent `-1;` to match the other constants, which is the case on one of the other such "trees". All it does is un-indent that last line, so not a big deal.


@maximevince, aside from the timers you've updated to a more proactive way to maintain the routing table for what I understand, to check the routing-table for outdated entries every second? But shouldn't there be another pico_timer_add in ```sixlowpan_ping_check``` as well?

My view on it was to only refresh a route when the host really needed it (checked in ```sixlowpan_rtable_find_via```), and not maintaining routes it doesn't really need so it could maybe sleep longer. The mechanism indeed seems to sometimes result in quite a lot of timer spawns. Didn't have the time to take a deeper look yet, though :-)

But this mechanism works better, but quite a lot of traffic, no?


I object to a global override. It would impact methods using `JSON.stringify` or `JSON.parse`, especially external packages.


@chriseidhof Can you please write a small test program and test that the performance of the test program under -O does not regress?  I'd love to see a simple tight loop that calls next()


Yes. Few reasons.

* It's always good idea to put tests closer to the source code (especially since all these are unit tests)
* I'm converting tests to BDD and ES2015 with this. I omit `test` directory for linting. But I can put new tests here and we can lint them.


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


@nadavrot I think it makes sense to improve the optimizer. And yes you're definitely right. Option 1 makes sense to me.


I'm afraid I do not. Despite the increased thread pool and inclusion of timeouts in `hyper:v0.7.0`, I couldn't get the UI to update promptly. If calico revisits this timeout issue, I think it will be handled by @caseydavenport


I actually looked at core.callLater before I commented to see if we
could just use this and was surprised it didn't already do thread
detection. But yes, changing core.callLater makes sense. The docstring
should be updated accordingly to mention that it can/should be used for
calls from threads other than the main thread.

Note that you don't need to use a lambda. CallAfter accepts args and
kwargs, so just: wx.CallAfter(wx.CallLater, blah, blah)

Also, I think the optimisation used for core.requestPump makes sense;
i.e. don't bother doing the CallAfter if we're already in the main thread.


lmfao I have no words

http://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSGettingStartedGuide/AWSCredentials.html


Fix riot/riot#1416
Now `tmpl.loopKeys` (for `each` expressions) supports spaces inside the key of the expression. Everything after ` in ` is assumed to be the key.


Ship It!
```
                              __    __    __
                             |==|  |==|  |==|
                           __|__|__|__|__|__|_
                        __|___________________|___
                     __|__[]__[]__[]__[]__[]__[]__|___
                    |............................o.../
                    \.............................../
               hjw_,~')_,~')_,~')_,~')_,~')_,~')_,~')/,~')_
```


How exactly do you expect this to compile on anyone else's computer?


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


this is redundant; fstabinfo quietly returns 1 if there are no entries in fstab matching the request


Saw that is passed as driver options.

If the other maintainers are ok with it, I would like to have a separate IPAMConfig for endpoints (like we have now for network creation) passed as `EdpointOption`, and correspondent IPAMData for driver CreateEndpoint()

Once we have that, the hard part is to come out with a proper abstraction for its fields. IP and IPAliases are perfect as generic enough and well known. Virtual IP, I don't know if it can fit as is. It should probably be part of a new category of addresses which would not go through `already allocated`, `be part of the subnets`, ... checks, but that will still be programmed by osl package on the sandbox interface.

I am now busy with moving the exposed ports and port mapping from Endpoint to Sandbox.
I should be able to start on this  end of this week or starting of next.


I don't know if it is a good idea to add it to preferences, at least client side....
Q: does backend save each preference in a SQL row and not as json? Then I would be fine with keeping this in prefs on backend. Otherwise I'd create a new SQL table.

2. On frontend I would recommend a seperate store for these to not abuse preferences storage


Done. Reset and re-rebase.


@jckarter I can see that you split your change to commits that touch code and other commits that change tests. Please don't do that. git commits should be atomic. We need to be able to bisect our git repo and be able to build the compiler at every single commit.


This comment doesn't look right anymore


@nadavrot These test changes are independent of any logic changes. I was updating the SIL tests individually to match SILGen's new patterns and uncover potential issues. Where there were issues uncovered, I updated the compiler simultaneously (see the AllocBoxToStack patch earlier).


https://github.com/spaceone/dotfiles/blob/master/gitconfig
oh, the github CLI thing seems nice.


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


Ah, I just wasn't clear on that. That makes sense. Tiny nit: RedOctber -> RedOctober


cool, thank you.


@jckarter okay. Thank you. :smile:


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


negedge reset_n like everywhere else?


TeamCity DnnPackager :: Continuos [Build 1.1.0-unstable.24](http://dazinator.ddns.net:81/viewLog.html?buildId=1462&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **FAILURE**
Summary: Compilation error: src\DnnPackager.TestModule\DnnPackager.TestModule.csproj Build time: 00:00:10


Why are you overriding this here? Should this be 2 secs also?


I'd suggest not putting logic in the template if you can avoid it.

I'd also suggest not making any exceptions for the test client.


this should close https://github.com/ministryofjustice/graphite-deploy/issues/16 once in prod


As per our discussion, please disregard my previous comment as this method does not modify state, thanks @msmirnoff !


As I already said the awk-script may looks to be a bit messy.. so perhaps I will put the whole stuff in a state-machine.


This makes an existing problem much worse. If a worker is not available in discovery at the moment the call is made (due to gc or being super busy on the CPU) we will fail queries. We should have higher tolerance for this. We can do this as a separate PR, but I would at least count the number of times this happens, so that we can monitor this.


Era en python mushasho


The intro message was changed a little bit along with some of my print() functions being deleted due to them no longer being needed for debugging purposes.  Messages were created for if the user guesses correctly or incorrectly and displays only if the game is about to continue through another iteration.  Otherwise, the game displays one of the two final messages before asking the player if they want to play again.  Also, disp_isnt_sec was replaced (in name only) by first_isnt_sec because i needed to compare lists in a different area of my code and thought the name was no longer fitting.


These look unused.


Maybe change this method name to ``fillInCompactRepresentation``?


I import `com.google.common.base.MoreObjects;` but not `MoreObjects.ToStringHelper` so I think `MoreObjects` is still needed.


I'd suggest calling this something other than "mock", since this isn't really a mock in the ordinary meaning.

I'd also suggest actually constructing the entire .../apis/testgroup/v1/ directory structure here.


This test wants to timeout. So I thought you would not like the test to wait for 2 seconds, so I restored the low timeout value for this one. Let me know if that was not the intent and I can revert.


Since I have both python 2 and 3 installed, I can look up the python 2 version with "python -V" but I have to look up the python 3 version with "python3 -V". Same goes with all python 3 commands.


the reason why this should work is that `isDifferentView` can only be true if `isChangingView` is true.  You can't get here and have a null `view`


With this, you can't deserialize.


At one point I was using this from Python for the Maya visualisers, but it ended up working out better to do that in C++ anyway.  So as far as I know I could just discard this commit if you don't think it's useful.

If you do think it's useful, then just warning Python folks seems reasonable to me.


LGTM.


I'd recommend making a real group in the sense of having all the directories. Just don't actually put the group in `pkg/apis/...`. Maybe put it here, `cmd/libs/go2idl/client-gen/testdata/apis/testgroup/v1` or some such.


@delivery approve


I think it makes sense to break out the `shouldDestroyView` logic as we only want to empty the region if there's currently a view in it, it's not the view we were showing and we're not preventing the destroy


I took this line out of the set-gen example, since it's not as useful as I wanted, and it makes verification hard.


nada a declarar


Think about doing this with `deactivate_project()` and `activate_project()`

Why? Because you might not get a value 'T/F' result from the POST. You can't assume that the client response in the form will make sense. So, interpret the form data in the `view.py`, and let that call `activate()` or `deactivate()`. Since state in this case is boolean...


When you fix the private constructor in the previous commit, this comment will be technically incorrect.


I guess I wouldn't expect to see this in an "unversioned" package, I'd expect to see it in a "mock" package (or whatever you end up calling it?)


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


Ok gotcha. LGTM

On Tue, Dec 8, 2015 at 2:56 PM -0800, "Ivan Kozlovic" <notifications@github.com> wrote:

This test wants to timeout. So I thought you would not like the test to wait for 2 seconds, so I restored the low timeout value for this one. Let me know if that was not the intent and I can revert.

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


Feel free to merge.

On Tue, Dec 8, 2015 at 2:56 PM -0800, "Ivan Kozlovic" <notifications@github.com> wrote:

This test wants to timeout. So I thought you would not like the test to wait for 2 seconds, so I restored the low timeout value for this one. Let me know if that was not the intent and I can revert.

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


Hmmm.... is it allowed to make a PR without a ticket?


I didn't expect, so I didn't not push it to master


Since this is new we can just drop this, no?


@mbreese so you suggesting to check the db first and reverse is_active?  and no relying entirely on post?


Can't do the uniqueness check here yet, but the rest of those formatting criteria should be easy to do.


j


lol. I've been getting the error:
`No local packages or download links found for multidipledispatch`

turns out it's typos.


Nice one.

Based on the latest Android App, I think that the following is a full list of commands. Not sure what all of them do yet - nor which TVs they work on :-)

Happy to make a PR if you'd like.

    NRC_30S_SKIP-ONOFF
    NRC_3D-ONOFF
    NRC_APPS-ONOFF
    NRC_ASPECT-ONOFF
    NRC_BLUE-ONOFF
    NRC_CANCEL-ONOFF
    NRC_CC-ONOFF
    NRC_CHAT_MODE-ONOFF
    NRC_CH_DOWN-ONOFF
    NRC_CHG_INPUT-ONOFF
    NRC_CHG_NETWORK-ONOFF
    NRC_CH_UP-ONOFF
    NRC_D0-ONOFF
    NRC_D11-ONOFF
    NRC_D12-ONOFF
    NRC_D1-ONOFF
    NRC_D2-ONOFF
    NRC_D3-ONOFF
    NRC_D4-ONOFF
    NRC_D5-ONOFF
    NRC_D6-ONOFF
    NRC_D7-ONOFF
    NRC_D8-ONOFF
    NRC_D9-ONOFF
    NRC_DIGA_CTL-ONOFF
    NRC_DISP_MODE-ONOFF
    NRC_DOWN-ONOFF
    NRC_ENTER-ONOFF
    NRC_EPG-ONOFF
    NRC_EZ_SYNC-ONOFF
    NRC_FAVORITE-ONOFF
    NRC_FF-ONOFF
    NRC_GAME-ONOFF
    NRC_GREEN-ONOFF
    NRC_GUIDE-ONOFF
    NRC_HOLD-ONOFF
    NRC_HOME-ONOFF
    NRC_INDEX-ONOFF
    NRC_INFO-ONOFF
    NRC_INTERNET-ONOFF
    NRC_LEFT-ONOFF
    NRC_MENU-ONOFF
    NRC_MPX-ONOFF
    NRC_MUTE-ONOFF
    NRC_NET_BS-ONOFF
    NRC_NET_CS-ONOFF
    NRC_NET_TD-ONOFF
    NRC_OFFTIMER-ONOFF
    NRC_PAUSE-ONOFF
    NRC_PICTAI-ONOFF
    NRC_PLAY-ONOFF
    NRC_P_NR-ONOFF
    NRC_PROG-ONOFF
    NRC_REC-ONOFF
    NRC_RED-ONOFF
    NRC_RETURN-ONOFF
    NRC_REW-ONOFF
    NRC_RIGHT-ONOFF
    NRC_R_SCREEN-ONOFF
    NRC_R_TUNE-ONOFF
    NRC_SAP-ONOFF
    NRC_SD_CARD-ONOFF
    NRC_SKIP_NEXT-ONOFF
    NRC_SKIP_PREV-ONOFF
    NRC_SPLIT-ONOFF
    NRC_STOP-ONOFF
    NRC_STTL-ONOFF
    NRC_SUBMENU-ONOFF
    NRC_SURROUND-ONOFF
    NRC_SWAP-ONOFF
    NRC_TEXT-ONOFF
    NRC_TV-ONOFF
    NRC_UP-ONOFF
    NRC_VIERA_LINK-ONOFF
    NRC_VOLDOWN-ONOFF
    NRC_VOLUP-ONOFF
    NRC_VTOOLS-ONOFF
    NRC_YELLOW-ONOFF


This could be ``for (int i = 63; in >= 0; --i, ...) { v.at(i) = ... }``.


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


complete


@smarnach  :+1: Looks good, and seems to work. There's something I don't understand though: how is `CircleCIParallelTestRunner` being selected for use? It seems like one needs to override `TEST_RUNNER` to have that value, but I don't see that anywhere in the commit. Is that being done in Project settings > Environment Variables, or is there something else I'm missing?


The dual plural here is a bit weird. `dns_server_names` perhaps?


LGTM.


Consider also doing this on L189?

`echo "One or more tests failed while running tests from '$fileNameWithoutExtension'.  Exit code $exitCode."`


@AndersHoglund
Wait more updated to come soon. Found some little bugs :D


Seems getVerticesToSimplicesMap has no docstring.


What is this file for?


No... I'm suggesting checking the return value from the form (in view.py below), and then calling activate() or deactivate() as required. It's just an idea. I haven't seen the html template for this yet, so I don't know what you're planning on returning. But the general rule of thumb is not to trust input from the client. But, like I said I'm not sure what you're thinking of doing here, so my comment might be premature.

Really - we don't need to worry about activating projects / promoting them / etc... yet. It's nice to have, and we need to *eventually* have it, but it's not the highest priority...


Thanks for this!


I don't expect, so I did not push it to master.

Sent from my iPhone

> On Dec 8, 2015, at 16:48, Andrew Sun <notifications@github.com> wrote:
>
> How exactly do you expect this to compile on anyone else's computer?
>
> ―
> Reply to this email directly or view it on GitHub.
>


@nkcsgexi Test? :-)


I think we need to move that to inside the media query, where we declare `position: fixed`.


Due to `tiled` being a default in TerriaJS (https://github.com/TerriaJS/terriajs/blob/master/lib/Models/WebMapServiceCatalogItem.js#L488) you don't actually need to set it for each layer anymore.


Just curious how You achived autobinding here... Could you explain, please?


The API doc actually refers to `dns_servers`, so I'm going to rename it to that.


@Dean177 We have deprecated parts of the `history` API, which causes warnings with the current implementation of react-router.


I think you might be on to something.


I removed the Autowired entry and cleaned up. Better solution, I think.

John

On 12/08/2015 04:55 AM, Erik Kemperman wrote:
>
> @jecollins <https://github.com/jecollins> Hi John, I was just looking
> into removing some of the many warnings Eclipse is giving me -- imho
> the Problems panel is currently slightly useless at the moment... But
> then I noticed this commit, which does the opposite of what I was
> thinking about doing. Can I ask what your rationale is here?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/erikkemperman/common/commit/86eca4966f89e8faa5cd29b18bb9a7536d55274a#commitcomment-14844138>.
>


I see, thanks!
On 8 Dec 2015 23:12, "Tim Dorr" <notifications@github.com> wrote:

> @Dean177 <https://github.com/Dean177> We have deprecated parts of the
> history API, which causes warnings with the current implementation of
> react-router.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rackt/react-router/commit/95111982dadcf4800b0ecd78a1195db0b6e6827c#commitcomment-14861326>
> .
>


You could probably make this global without offending anyone too much, and that way you wouldn't have to pass the daemon around more. I'm fine with it this way too, though.


No need to prepend ```"Node "``` as that is already included in the output of ```briefToString()```. And perhaps we can change the second string to ```" threw exception: "``` to make it more understandable?


LGTM.


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


Hmm, I'm a bit of an ocaml noob.  Should I be on a specific version or something?


Sure, I will add one on the SourceKit side.


Is `stream.end` called in the new revision? Are there any side-effects to not calling it?


109649198_EvapCoolers (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1448 of 1448 tests passed)

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


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


@bmackinney - I added the `DOMAIN_CURRENT_SITE` example to this doc so a single resource can be used for those migrating a network. Seems like an important difference between what users may have existing in their `wp-config.php`


:rocket:


Related to issue #43


The command is executed by the same user who is running the bot.
I don't see why there would be an issue restarting the bot via its daemon through this, although I'm unsure about updating, since that could potentially cause issues whenever you update the bot while it is running. If you've tested updating the bot while its running, and that works, then it should work through this.


This (and all the other changes to the README) is **awesome!** Do you plan for it to have a website or is this + some [exposure](https://github.com/elm-lang/projects) enough?


No idea to be honest, I'm still on OCaml 3.11 but most other devs are on some OCaml 4 version.


the reason why this should work is that isDifferentView can only be true if isChangingView is true. You can't get here and have a null view


https://parmarhh.files.wordpress.com/2012/04/move-fast-and-break-things1.jpeg%3Fw%3D640

(It's actually appropriate right now.)


If we're about to empty the region we want to end the feedback loop.  This listener is only for when the view is destroyed _not_ because of the region empty..  So we also want to do this if the view was only detached and not destroyed.


Merge issues. You should probably fix these.


yeah, I expect to do a bit of reshuffling of all that stuff when splitting things into more packages, the daemon package would then have Daemon as a global that everything else can use at which point having execPath be part of daemon would probably be okay, or it'd be another global of that package.


Merge issues.


Merge issues


LGTM (and arggh about the original code).


LGTM.


Merge issues (CTRL-F for '====', '>>>', and '<<<' in your code)


I still think show is cleaner by breaking out `renderView` and `attachView` and putting the particular flag logic inside them, but am open to alternatives.  They don't really correspond with the should show/destroy/empty stuff above


Check to see if the hotkey signature is PTHk and return eventNotHandledErr if it's not one of ours, rather than throwing an assertion exception.


That's fine w me too, but I wouldn't say it's passing a function arg isn't idiomatic, it makes sense. I would call the shutdownFn, then os.Exit()

Both are good w me tho.

> On Dec 8, 2015, at 5:05 PM, Vojtech Vitek <notifications@github.com> wrote:
>
> I'm trying to fight optional args/funcs where possible. It's not idiomatic go.
>
> —
> Reply to this email directly or view it on GitHub.
>


Perhaps this should be removed, but is always rendering a view on show a good thing?  If the view is already rendered, why do we want to render it again?  I can't come up with a use-case, but I assume if I'm at this point, showing an already rendered view.. it's probably because I don't want it re-rendered.


@sjgiri are we sure this takes the same value as `env.enable_captcha`


Merged


FYI you should make sure this runs and everything works before presenting.

Good luck, and let me know if you need anything! :+1:


@jhart-r7 is it possible to get some help with this Type error issue, the value of #{number} is a 0 not sure if its an int or a str, but I believe the table needs an int. I tried to use .to_i to help convert it to an int, but it does not seem to be working. I also dont know if its on the #{number} var or the #{channels} var. Is there any debugging I can do to help see where it breaks at?


you can't do this, content_url does not take in 4 params and will fail.


LGTM. Also arggh and LOL.


access app in http://localhost:8888/employee/struts2rest-1/111.xml  - to get the data in xml format.


Careful! This test isn't a duplicate. Its author just forgot to change its description. It should read "runs **post**install hook".


hi @yawitzd
Emacs all the way!  (It's the only editor I know.) :+1:


see 67eed50190c1555c4f95b3b62d665f6e128dc6eb


hi @same1009
Vim and Sublime, ok!  :+1:


LGTM.


Interesting, but `__attribute__((hot))` is not understood by the Visual C++ compiler, and breaks the build there.


@jamesallsup I'm hoping that you see my comment here. I'm in the process of integrating amazon pay into opencart 1.5.1.3 from 2.1.0.1 . Yeah I know why would I do that. Alot of legacy code so gotta stick with it. :) My question is I'm having an issue with the event subsystem. I installed Amazon Pay also on my dev version of 2.1. I'm getting an error on checkout because order.php file on addOrder is making a call to
$this->event->trigger('pre.order.add', $data); The problem is pre.order.add is not in the 2.1 event table. So I get a null exception. So when I attempt to port it to 1.5 its not going to work as you can imagine. My question is what are the action urls for the following events because they are not part of the amazon login and pay extensions when I installed: I need actions for the following events:

pre.order.add
post.order.add
pre.order.edit
post.order.edit
pre.order.delete
post.order.delete

I noticed that the Amazon Login / Pay extension ONLY installed the following events in MySQL
post.order.history.add   , action = 'total/voucher/send'
post.customer.logout   , action = 'module/amazon_login/logout'
post.customer.logout   , action = 'module/amazon_pay/logout'
post.order.edit, action , action = 'payment/amazon_login_pay/capture'
post.order.history.add , action = 'payment/amazon_login_pay/capture'

Are the above actions still supported??

thanks,
-K


:smile:


I think our coding convention is going to say that these should start with ``ZC_``.


But LGTM for the bsaic idea.


I prefer ZC as well since it's shorter.


> I'd also suggest actually constructing the entire .../apis/testgroup/v1/ directory structure here.

Do you mean everything including the install.go and conversions etc.?


These should probably be cached on an instance member, since re-using cached statements across connections is risky.


This patch works for me.

LGTM!


`redirectInfo` satisfies the contract for a location descriptor, so...


As much of it as is necessary to test the client generator.

On Tue, Dec 8, 2015 at 3:25 PM, Chao Xu <notifications@github.com> wrote:

> I'd also suggest actually constructing the entire .../apis/testgroup/v1/
> directory structure here.
>
> Do you mean everything including the install.go and conversions etc.?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/caesarxuchao/kubernetes/commit/55961ce7305e3404b8519ee43e015ef4339760fe#commitcomment-14861569>
> .
>


... we can just use the public `history.replace` API.


@talos-arch3y I suspect `channels_table.print` requires that all of the data in the table be a string.  This should work.  Something like:

```
data.each_with_index do |val, index|
  number = index
  channels = val[/[[:print:]]+)/]
  channels_table << [ number, channels ]
  channels_table.print
```


why this is only needed if its ipv4?


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


There is one functional difference here - this will actually run the matching on the redirected route.

Two concrete examples:

1. If you chain redirects (why would you do this?), this will now redirect you to the ultimate destination, rather than the first redirect in the chain.
2. If you redirect to a 404 (i.e. if your code has a bug), this will now hit the "no match" branch above.


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


Wait, why were the CERN ones removed? They are still functional.


easier to read with
``
        nameJoiner.add("shard_id")
                .add("shard_uuid")
                .add("node_ids")
                .add("bucket_number");
``


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


Yes, I know, I'm slowly trying to cleanup and make the unit test run again
under travis.

On Tue, Dec 8, 2015 at 3:03 PM, Hai Nguyen <notifications@github.com> wrote:

> lol. I've been getting the error:
> No local packages or download links found for multidipledispatch
>
> turns out it's typos.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chemlab/chemlab/commit/957173e0c3c76e39045e70a0329f19438aaf0d4e#commitcomment-14861179>
> .
>


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


Ah; I didn't think of deleted users. That makes sense.

AltName sounds like a better idea.

My suggestion is to either move the logic of defaulting to the redoctober name into getAltNameFromName so we don't need to return `found`, or remove the function entirely since it's only called here. I just feel like the interface is more complicated than it needs to be for this one usage.


I can't complain, 13°.


Note: This is not finished, but I wanted to push this up to master in order to keep a more complete version up compared to the previous version. Please refrain to push anything else that is incompleted.


Changes have been pushed to thunderbots.net


I'm not quite sure if adding to this map is sufficiently protected here. I.e. are these elements also _semilocal_.


HBIRE_vector_take2 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


can you try to write compat code for py2 and 3? (which is trivial for you to do with the help of 2to3). We should promote PY3 (which is better than py2).


109649198_EvapCoolers (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1978 of 1984 tests passed)

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


Successful, btw


TODO: Add solutions for deptinst1 and depinst2


Bitte keine Backup-Dateien anlegen und comitten. Dafür habe wir git ;)


It's a strange version. I wrote it in night. I will rewrite it later.


Nice fixes. But, on a 508 frames output:
Ctrl+Maj+Right jump to the 508th frame (good!), but at that point, (Ctrl+(Maj+))Right crashes. Better stay on the same frame.


I print the hit rate every 1000 queries, this is the last print i got by compiling the stdlib -O.
hit rate is 73.092297


Data Division:
For speaker classification purpose, training, validation and testing data is separated using the code - 'data_division'.


I've tried 2to3 to convert to py3 and works well for me.
https://github.com/hainm/chemlab/tree/py3


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


This last line prints out ~80 lines of s_client command line usage.


Method outSomething maybe not work. I didn't check it.


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

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


Should we really be linking to outside of the hack club eco system?


If that's all that is necessary, we could add a `SpectatorEntity` with a `SpectatorComponent` that does this inside `HandlePrepareNetCode`.

Is there a lazy way to turn this thread into an issue? :stuck_out_tongue:


the new mount 1 calls "Rift Runner"


There's the issue. Whatever I do the filters map will always be empty.  https://github.com/primefaces/primefaces/issues/53


Since you are taking this out of the consensus function, can you at least create a separate function that returns it from a given block and flags to disrupt consensus encapsulation a little bit less?


I think this should be `<html>` + `<head>` + `<body>`.


Hi,

can this project https://github.com/iMoreApps/ffmpeg-avplayer-for-ios-tvos be integrated to the
web browser? I tried to watch videos, however fullscreen videos do not work. Does it make sense to use this player? I have very little programming skills, so I am not sure how to implement this. If someone would integrate this, that would improve it and hopefully enable fullscreen mode for videos. Thanks in advance


Yes you can expect python 3 compatibility (that's another thing I'm doing).

On Tue, Dec 8, 2015 at 3:33 PM, Hai Nguyen <notifications@github.com> wrote:

> I've tried 2to3 to convert to py3 and works well for me.
> https://github.com/hainm/chemlab/tree/py3
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chemlab/chemlab/commit/957173e0c3c76e39045e70a0329f19438aaf0d4e#commitcomment-14861722>
> .
>


Do these all have to be list items?


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


Again, I think this would feel nicer to read in a short paragraph form


Note that the API jar does not include any resources. But our decomposition of jvm component isn't yet complete, so we depend on the finest-grained build item that we can. There are 3 choices of how to proceed:
- Go back to wiring the API jar task directly onto the various compile tasks.
- Introduce a separate build item for the compile output for each source set ('ClassDirectory'), so we can depend only on those build items
- Leave this as-is, and introduce the separate 'ClassDirectory' build outputs at a later stage.


Did you mean to link something here?


OK. No problem.
But please just commit the fixes. Try stay away from "un-commit re-commit" if possible. Just moving fwd is the best. But it's no biggie to reset and re-rebase either. As long as there are only a few files.


"img" should be either capitalised ("Img") or in code tags `img`


shouldn't this be a `not empty` check?


capitalise `src`


I already disliked returning int64_t instead of bool...


os.Exit sends SIGKILL (equivalent of doing kill -9 pid) and doesn't wait for anything.


Pranav, any edit I make I cannot push to any repository, I can't use git
bash or git desktop, I have to use the online text-editor.

On Tue, Dec 8, 2015 at 3:31 PM, Pranav Mathur <notifications@github.com>
wrote:

> Changes have been pushed to thunderbots.net
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Thunderbots5604/Thunderbots5604.github.io/commit/af91a04eaeb33e9dc12822b0b0b11dc91a8eda7e#commitcomment-14861675>
> .
>


mijin ya valimos


haha, no longer doing healthy food are we?
excluding the sunday breakfast surprise, it's still pretty good, right?
in-fact we'll buy a bunch of fruit for sunday too


I used Emacs eons ago and absolutely loved it, and I recently decided that
I want to learn it again. Do you have any recommendations for getting up to
speed on at least the basic key bindings?
ᐧ

On Tue, Dec 8, 2015 at 5:26 PM, Reshama <notifications@github.com> wrote:

> @chuckwondo <https://github.com/chuckwondo>
> Re-reviewing since your last submission.
> Ok with Vim (I only know Emacs.)
> PyCharm is great, and with lots of features! [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chuckwondo/dsp/commit/016a9a509832132af42787f8e52d41efa441ef10#commitcomment-14860396>
> .
>


The third argument, `true`, certainly widens the collection of pixels returned by the rasterizer.  Whether it is intended to -- and does -- solve the issue of returning partial pixels is to be determined.


Fix on the July release of the XRMToolBox


This will instead allow the user to select an app to install.


this API looks exactly the same as the parent's. If so, we don't have to define it again here.


As this is a pretty hot function I think `layout.triggerMethod` might be more reasonable.  I think the purpose behind `triggerMethodOn` was so that we could support views that do not utilize `triggerMethod`, but in this case I don't know of a way that we could get this far without the parent having `triggerMethod`  If it is possible than `getOption` and `getValue` also need to be addressed.


why does this return error, if it can never return an error?


why not use defer here like in the other places?


Same here. This API looks exactly the same as the parent's. If so, we don't have to define it again here.


Well burritos, and baguettes with fried stuff in them, while delicious and good quality aren't exactly known for their health benefits and nutritional value.

Fruit's a good shout.


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


To implement the ProviderConfigSetter interface https://github.com/ericchiang/go-oidc/blob/562ae81124e0ae327b444d609a597a35f3521027/oidc/provider.go#L63-L65


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


I'm curious, what is faster on average/expected input? Is it to check if strings are not sorted, then sort them. Or just to always sort the input without checking if it's sorted first.

Does anyone know?


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


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


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


I'm proposing to use the official vlc build which does not ship a `libvlccore.dylib`, only one with an soversion appended which you are not supposed to link against (and which CMake won't detect). Moreover we don't need it at all ...


do we not want to pass `this` after the event name?  I would think like `childViewEvents` these proxied triggers would need the context of which child triggered


LGTM


Definitely make sense indeed, thanks !


Might need to add a few things, but this is what I have so far.


let me try


since `aes.h` is a single header, I think the error message should say "header" instead of "headers".


The official build ships a bunch of libs that can also be provided by homebrew, thus I made the Formula keg_only.


tomahahahahahawk


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


Thank you!


mira, yo ya arregle lo del login que se muestre los datos cuando sea admin, continua con las otras cosas


I think you should let the providerConfig take care of protecting it's `config` resource.

In other words, I'd have a `Get` method matching your `Set` method which does the lock/unlock (and make chooseAuthMethod a function which takes a config rather than a method).

You'll also be holding onto the lock less time which is good as well.


Without removing the 64x64 icon from the list I'm getting this error:
```
Dec  8 15:07:06 74-80-249-47.ic2net.net iconutil[7771] <Error>: ImageIO: _CGImageDestinationCreateWithWriter capacity parameter (12) is too large for this file format (max is 10)
/Users/admin/sources/tomahawk/build/src/tomahawk/tomahawkSources.iconset:error: Failed to generate ICNS.
```


This should probably be `html5lib<1`. I assume the html5lib people are avoiding bumping to version 1 for a reason.


good point. My assumption was wrong. while testing i found this env variable is not set always.


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


fine http://i.imgur.com/zc3DGSk.gif


`rawDec` -> `rawWithDecorator`


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


I think it does, at least the test checks what view it was called on?


Typo here.


What's this file for?


Hmmm. https://travis-ci.org/laravel/framework/jobs/95691825


thanks. I got that out of https://github.com/NICTA/aremi-natmap/blob/master/datasources/aremi/common/standard_properties.ejs - worth removing from there too?


why this becomes a no-op?


This file doesn't exist under /images


@Optimusprimeums thats OK. I wish you to make the moving without any problems.


This is to determine if the race detector is enabled for TestProviderSyncRace. That test never errors, it only triggers the race detector, so it skips if the race detector isn't on.

There's no API to determine if the race detector is enabled, only a build tag you can detect with `+build !race` and `+build race`. I suppose rather than having this file, I could put TestProviderSyncRace into it's own file with a `+build race` tag at the top.


Yeah, that sounds a lot nicer.


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


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


This commit got duplicated on accident (see bc48fc63a9e0a1b9e065901797c4b907ec2c0e04)


[![Conception Synchromy](http://api.artsmia.org/images/120630/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120630)


I'm not sure how this tests the raciness - there's no concurrency that I see.


This commit does not necessitate pulling to keep validity, but it is recommended to avoid merge issues.


Yep, that's probably right.


Awesome, but sometimes people change their `/data/` directory with something else. It would be better if we can able to get that path from `config.php`. There is an `\OC::GetFromConfig` command I don't remember.


This is the start of the air


`getattr(cherrypy.request, 'girderRawResponse') is True:`

`girderRawResponse` is expected to exist.


Typo, read_fd_decode_safe**t**ly


Thanks man!


I do put commits for eventual review if needed. Some commits I did were just rubbish and confusing. Had to squash it


Hmm, can't remember from the top of my head, will find out tomorrow morning!


This stuff was quite complex. Too many special cases, but getting there


SyncProviderConfig is running a background process that writes to `Client.providerConfig`. I'll add a comment describing what's being tested.


dao ci yi you


I see.


Or ultimately it should be called by Bash and Bash only?


Alright


Do you mean angle of robot tilt, or angle off center of straight?


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


@JuanitoFatas Please take note.


With vcfeval, the default is GQ, but you just use --vcf-score-field to select whichever field you want (typically in our analyses we manually adjust depending on the pipeline and type of investigation we are running). You could even annotate the VCF with your own custom field and select that as the ranking attribute.

AVR is a score developed by RTG (if you think of VQSLOD as a machine-learned version of QUAL, then AVR is a machine-learned version of GQ, but where in our pipeline you typically use a pre-built model instead of having to run separate train/predict stages after variant calling). The machine learning technology is general enough that you could build custom predictive models given appropriate input VCF annotations and training data (e.g. predict other classes of variant).

The screencapture was showing a single raw set of calls made by the RTG singleton variant caller, just ranking the calls by different score fields.  Part of the goal of the screencast was to show that the rocplot tool can help you visualize the effects of filtering on the score field (e.g. by varying the slider and seeing how the metrics change). You could certainly plot "pre" and "post" filtering curves (although this is kind of redundant unless you are filtering on on some other field or a composite filter, since the curves would lie on top of each other).

Within bcbio, the question is how to handle the default score selection (and how to expose that to the user for customizing), and things like which set of curves to plot together. I'm not sure whether Brad envisions automatically generating a static ROC plot (e.g. using rtg rocplot to output a static image, or using some other graphing tool that does a prettier job), vs having the ROC curve data files available for interactive tinkering afterward.


Adicionado rotina de tratamento dos novos parâmetros `driver`  e `charset`!


where no example for look?  I need it ,Thank you !


Zheng is a ba-shit-ism writer (i.e. the worst side of bash) so I have changed it already.


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


Thanks @electrikjesus!


I thought of that and attempted it in the beginning, but it got complicated - so didn't want to do it. Because the `__init__` in Bears has args like `section, message_queue, etc` which I would have to inherit and the Bear writer would also have to write it though it wouldn't make sense to the bear writer much.

Also, This is more flexible - because the linter isn't coupled with a bear it can be local or global or etc. I found this to be a better choice overall.


Already issue with space in directory. This should be handled better.


Would be better to keep these as static variables (outside the init function) - as then the compiling of the regex wouldn't be done in every bears object but only once when the bear class is defined


You should change the variable type to LPCWSTR instead of casting away the const.


`const LPWSTR PhaseShortNames[]' -> `const LPCWSTR PhaseShortNames[]`


So that's where the fingerprint is (supposed to) contain. I believe this is a stupid choice of pack-of-info…


nit: remove empty newline


These results are awesome!


this should never happen, but every precondition should have a descriptive message.  Otherwise this just looks like any other null pointer exception.


- [ ] Should be specified a signId. Why changed this ?


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


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


- [ ] Please modify to use JOIN fetch (Currently, this mapping has N+1 problem)


How about both buttons at the top? Or at both ends? I was merely listing them in the order that I think people will use them. My concern is that you'll press back by mistake, get the modal, and have to scroll a few pages of screens before getting the "cancel" button.

Alternatively, you could put the screens in an independently scrolling list.

Also, please do include the extra wording in the "cancel" button. Cancel buttons by themselves are often confusing. (As Greg Wilson has noted, humans have trouble reasoning about negatives much more than about positives.)


could use the `transformDetail.getEntries()` method


Rollbacks an accidental change


Style consistency!


Presumably jspm should be removed from [package.json](https://github.com/Faithlife/generator-faithlife-app/blob/master/package.json#L19) as well?


And the [README](https://github.com/Faithlife/generator-faithlife-app/blob/master/README.md)?


Maybe i was too tired in last weekend, i didn't think about that. Forgot to import your changes after i reverted my version to CVS version. I did it step by step, and i thought it was faster :(
We hope this review can be done before the test production of run15 data. So i didn't really think about how to explain the details clearly. Apologize if i made much inconvenience.


Oh, Then I would just remove the additional 30% memory.
I am not sure about the exact size of memory I will need.
Then I think it would be better to let vector handle memory allocation by itself.


Style


No, it's covered by [previous check](https://github.com/rg3/youtube-dl/blob/3d24bbfbe433e3a2b2257096264bf4e846f6d889/youtube_dl/YoutubeDL.py#L1108-L1112).


I prefer to mark the end of  a function by  a ```return``` statment :P


`transformDetail.resetEmitter()`


We're adding two different listeners, both of which should receive ConnectionNotifications when events are fired.

The first is the current org.collectd.Connection, the second is the NotificationListener which was passed to the Connection during construction. Each one needs to handle / cleanup things differently.


Not needes here.


Also, the `run` would have to be overridden any way if the Bear needs section arguments


You can either do `newDstLang = pairs[curSrcLang][0];` at first or use `var newDstLang = undefined` and then `if(!newDstLang)`. `"default"` doesn't have much semantic meaning.


Ed*


only here..


`if(`, etc. instead of `if (` (keep consistent with rest of the file)


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


Links in html aren't the same as in markdown. I'll fix it.


I'm not sure... this code was written a long time ago (not sure it still merges)


actually, this code block isn't needed at all, and don't need to pass the terminalNodeEntriesMap around either. On line 69 you populate the map.


@tarsius This appears to be a breaking change (https://github.com/sigma/magit-gh-pulls/issues/67), should it be noted in the changelog?


Is this change going to remove Sauce build parameters support? I find this feature very useful as it enables me to select different browser for different builds


@dweiss any chance you could put out a maven release where the Open infixes still exist and are deprecated?

```java
@Deprecated
public class IntIntOpenHashMap extends IntIntHashMap {}
```

This breaking API change is causing some dependency issues for me because I use both elasticsearch and languagetool in the same project, which both depend on hppc.

I can make a fork and use a custom version of hppc, but I'd like to stay on maven central if possible, and it seems reasonable to me that hppc might have a release where both old and new names exist to make upgrading easier on users, so I thought I'd ask.

Thanks!


if there are next stages, need to reset the emitter here. Otherwise if we get to this node again, the previous entries will be re-emitted.


I know this so I wrote the documentation to remind myself about this.
And, by doing this I can avoid overloading this function for different kinds of vectors such as std::vector and deal::Vector.

If I move this class to deal.II one day, I would wrap it up.


Okay, I tried to do it but it didn't work. I think it's because it's in list form. sigh


nit: remove extra space


Good catch. I should clear each element of ```vector_data_ptr```.


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


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


i don't want to clear the errors map, maybe i can add a method there.


sadly this branch not working. I am working on cm-12.1-lte, but commit with adding ril brokes same way


Hey that one wasn't me :p


yeah, I do it when i'm too frustrated to notice


Up to this, I think ```types::global_of_index``` would be better.
Any way I just leave an entry here so I can change the type easily if I want.


Had to hop on a flight. Just pushed it up

On Monday, December 7, 2015, Fredrik Norén <notifications@github.com> wrote:

> Doesn't look like this was ever deployed to cargo
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/serde-rs/aster/commit/09b731d8bde6faf466f5845835a73c5351530d74#commitcomment-14837153>
> .
>


Beauty


Huh... did not realize that. I had just did a `sed` on it.


NEI issue should be fixed.  Weird issue with skope.


The coding style is not called "Mono Style" in official.
I recommend to use "Coding Guidelines of Mono".


delivers #5


use config


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (905 of 905 tests passed)

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


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


Agreed.  Where is DATACAT_CONFIG set exactly?  I see DATACATPATH. Is the
DATACAT_CONFIG area under CCB control?

   - Tom

On 12/7/2015 6:08 PM, James Chiang wrote:
>
> My preference would be to have the config file found via an
> environment variable:
>
> |config_path = os.environ['DATACAT_CONFIG'] client =
> datacat.client_from_config_file(config_path) |
>
> and have DATACAT_CONFIG set in the site-specific setup script.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lsst-camera-dh/harnessed-jobs/commit/b6973de0d58ed8e8bb23d91ef700cb47be603bd2#commitcomment-14837344>.
>


Yup, I used the wrong parameter.

Regards,
Bryan Drewery

> On Dec 7, 2015, at 17:05, Jan Beich <notifications@github.com> wrote:
>
> After this commit I get
>
> $ poudriere jail -cj test-amd64 -a amd64 -v 11.0-CURRENT                ~
> [00:00:00] ====>> Creating test-amd64 fs... done
> /poudriere/jails/test-amd64/fromftp/MANIFEST  100% of  670  B 1490 kBps 00m00s
> [00:00:00] ====>> Error: Empty MANIFEST file.
> [00:00:00] ====>> Error while creating jail, cleaning up.
> [00:00:00] ====>> Removing test-amd64 jail... done
> —
> Reply to this email directly or view it on GitHub.
>


I have renamed the tag name "ready for review".


ah, already in there. Thanks


no, you made it... stranger


I prefer to change the class template parameter into ```InternalValueType``` so it will be more descriptive and compatible with the new deal.II convention. The imperfect point is that the class template parameter is actually not that "internal" -- users can see it and set it.


java.io.OutputStream 的write方法第二个参数是偏移位置，第三个参数是长度。那override就错了。


this should be `.description-popup` not `class=description-popup`


It is prepared for sending this class to deal.II.
So users can inherit this class and redefine the algorithm for mapping data form old mesh to new mesh.


I appreciate comments like this especially when they're getting deleted when you figure out another way. :)


This is the part that you need to revise so it hook to assemble${variantName} instead


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


Yes thanks indeed!


to make this simpler, you can make an IdentityTransformation that sinks will use. ETLWorker could then pretend each sink is a transform, which would remove the need for a lot of this new logic.


I already said this elsewhere but I'll say it here too: I think it's because it's inside a list.


I was wondering why Libmacgpg wasn't a submodule and surprised to see that it was and now no longer is. What's the reason for not using submodule and instead having a nested clone?


o good point, can ignore this comment then.