I'll add sentence "See docstring of `from_dot()` function for more details" here.

However, note that the description placed in the docstring of this function is also slightly vague because it says "Among other things (...)." This has been done on purpose. Namely, the `SimplifiedDotGraph` is an internal structure with its (so far) only purpose to make defining graphs in unit tests easier. Because of this, I think that it's not that important to keep the documentation detailed. I don't like a too detailed documentation because it has a disadvantage that it easier for it to diverge from the actual implementation. In this case, if the reader wants to know what constructions are supported by this format, he can take a look at the unit tests.


Great!

Somehow this got tacked on to the (already-merged-and-deleted) `jogl-java3d` branch though.

So I cherry-picked it to master: 42cdbe4ddd263c67694d1cb1c69ee0f4581ffe15


@nairdo @azturner fixed in https://github.com/SparkDevNetwork/Rock/commit/45fc9c3b3a7b0f8e78fd1e048584ac6ba885839b


Yanlış. Sürekli sonuc değişkeninde birikiyor toplam, oysa her iç çarpım sonrası sıfırlanmalı `sonuc`.


This is not translatable.


`for` ile yapsaydın keşke.


Bunlar gösterici olmalı...


Thanks!


The 2nd link explains exactly what I am seeing. BUT it does not appear there is a good solution.


Koca Z matrisini, bir double'a atamak?


done


I'm getting an undefined index here, I think something went wrong :)


Diziler böyle toplanabiliyor mu C dilinde?


Assigning directly makes for cleaner code
```
var post.deletableByUser = mayDeletePost( currentUser, course, post);
```


проверка


Eşittir işaretinin solunda aritmetik işlem yapabiliyor muyuz?


Ok, I'll scan the code for such occurrences and remove `not` where it makes sense.


Hum, did you forget to add the normal.h file?


you either need to `unlink` the file or use `http://php.net/tmpfile`


`k * Bpitch` olacak.


Gotcha. Well, I'm happy with either removing this or fixing the crash, whichever you think is best.


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


This is duplicate logic. I think having a template per comment and then adding `mayDeletePost` as helper would be the cleanest solution here.


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


OK. (Do we have any sense for how much these derived instances increase compile time? I feel like it is nontrivial.)


`php://temp` uses a file backend for long content and php://memory is always in memory


Fixing now. Sorry about that.


OK.


`for` ile yapsaydın keşke.


It's my comment!


It's an internal function that I attempted to define right above the Console interface. The algorithm is the same for a lot of methods, with the exception of the `logLevel`, so I separated it.


Locally, I've changing "host: 80" to "host: 8888" on both my Ubuntu laptop and my Mac laptop. On Ubuntu, I already have something (Apache) listening on port 80 that I don't want to shut off. On Mac, I don't have anything listening on port 80 but I can't browse to the homepage. When I switch to 8888 it works fine on both platforms, I can see the app from <http://localhost:8888>


This bug is not in 4.0.x.


You also forget test on the scale argument.


so this still will produce file IO


I don't know much about the code here, but I dumped $node->children at the point where falseExpr is undefined, and it has false/true/cond keys but not falseExpr/trueExpr. Could it be you have different ways to express this node, or maybe it changed and you're using some old notation?


@rerb - I'm thinking we might want to accommodate for users who haven't filled in their first name since there are a lot of drupal users without first names. Username probably isn't a good option either. Maybe "Hi there!" if there's no first name or something like that... ask @monika-aashe. In fact, eventually we may want staff to be able to edit these easily, but we can wait on that for now.


Ho, nice, may be you can add a test? :-) Or change the existing one for argmin/argmax?


Will old strings stay in the platform? Can they be deleted if not used any more?


@Seldaek This should be good now.


![screenshot from 2015-12-16 19 12 53](https://cloud.githubusercontent.com/assets/7817276/11848171/bb39cf86-a429-11e5-9409-158bb652cea0.png)
![screenshot from 2015-12-16 19 12 10](https://cloud.githubusercontent.com/assets/7817276/11848185/c815a716-a429-11e5-83b9-9a327fddf326.png)
![screenshot from 2015-12-16 19 13 28](https://cloud.githubusercontent.com/assets/7817276/11848188/c9f193ba-a429-11e5-9cb4-d4a16cd3e3de.png)


wait... you split up `doccomment` in `doccomment` plus something else? IMO this is a naming issue, the same name shouldn't change its meanign!


Done.


Some issues with merge, bah


I know. :) It's just a proof of concept if it could work. I'm not sure I really want to make it a feature of vfsStream which is why I opened this PR to gather some feedback on the feature overall. If it becomes one the actual implementation will definitely take care of that - and of course will contain some tests to make sure the feature doesn't break in the future.


This is used in another section of the site still at the moment! But yes, we can delete strings that are no longer being used, so thanks for keeping an eye out for it :)


Ok I understand now, btw that's why I initialy defined the resolution option as a pourcentage of the actual game res using a seekbar. Anyway I'm okay with this, we just have to adapt the 1.33 factor to the real screen ratio.


Clearly a typo here


this one is also different in FF and Chrome:

FF:

```
console.log("%s %snewword", "duck")
-> undefined
duck newword
```

Chrome:

```
console.log("%s %snewword", "duck")
duck %snewword
-> undefined
```

I like printing the format specifier a bit more because it shows me if i forgot to pass an argument.

here is how it is handled in C:

```c
#include <stdio.h>

 int main()
{
   printf("%i plus %i equals %s.\n", 3, 2);
   return 0;
}

```

```
(18:19:17) [rocko@tequila-t61] ~ $ gcc -o foo foo.c
foo.c: In function ‘main’:
foo.c:5:4: warning: format ‘%s’ expects a matching ‘char *’ argument [-Wformat=]
    printf("%i plus %i equals %s.\n", 3, 2);
    ^
```

```
(18:19:27) [rocko@tequila-t61] ~ $ ./foo
3 plus 2 equals (null).
```


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


You could compare directly
```
return Template.instance().editing.get() === this._id;
```
instead of doing the `equals editing this._id` in the template.


keep the comment here, links can get broken over time.


z ve x hep aynı.


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1991 of 1993 tests passed)

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


@Seldaek What code were you running it on to get that?


Because we use std::swap which is defined in according C98 Standard in <algorithm>.
And we always have explicitly to add necessary headers.


This should not be necessary because the comment body is only shown when not editing.


Bir alt satırda yaptığın gibi
`double *P1 = malloc(sz)`
şeklinde yapacaktın.


OK.


Everything will get squashed later.


Sorry, should be "opengl" instead of "glsl" for these two enums, so:
shaderc_target_env_opengl
shaderc_target_env_opengl_compat

(And thanks for teaching me about compabitility mode!  That was a corner of the spec I hadn't yet explored.)


Maybe also add a confirmation message when there was no error.


@Seldaek  Also, which hash of php-ast are you using? I'm on https://github.com/nikic/php-ast/commit/51f2183 - trying again with the latest build


looks good.
I will check the SML part


Burda yapmışsın heap'ten yer almayı.


не нужно коммитить файлы специфичные для IDE


Yeah, but the thing is that `php://temp`  will not result in the error message as `php://memory` does. Although, if file IO happens it is in the hand of PHP, vfsStream wouldn't have to take care of that. My guess is that most files in such scenarios will be so small that almost always memory is used as it will be below the threshold on which it will switch to the real file backend. We could even use the `/maxmemory:NN` syntax to increase this threshold.


Yes, the second will overwrite the name and set it to `null` (what Resource Manager calls "update" is a replace (rather than a patch) using our terminology)


I'm not planning any revisions unless something comes up after my presentation tonight.


это IP сайта


EconomizerIntegrationNew (lgu1234) - x86_64-MacOS-10.9-clang: OK (1985 of 1987 tests passed)

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


This is not cool, bro. How is that “hausnumero” calculated? Think about modifying the `calculateHourLabels()` function and subsequently this test in future. How would you do that?


Haha. Looks legit.


F ve H alt matrisleri Y'nin içerisinden geldiği için Xpitc değil Ypitch kullanman lazım.


This feature is incomplete. I dropped it as soon as I realized that Travis CI does not have Windows build machines. pyovr is Windows-only at the moment, so what's the point?


OK.


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I kinda disagree with this enhancement. Play framework 2.4.x suggests using an injected MessagesApi object in the future: (https://www.playframework.com/documentation/2.4.x/Migration24, search for MessagesApi).

Actually, what I am planning in the future is to have that object injected in `AbstractJudgelsController`:

```
public abstract class AbstractJudgelsController {
    ...
    @Inject
    protected MessagesApi messagesApi;
    ...
}
```

and then passing around that object to the views.


+1, be sure you fix the build problems. (semaphore is failing)


// Presign does not have a payload, use S3 recommended value.
var payload = "UNSIGNED-PAYLOAD"


Looks good, but is better to delete than comment out the old code because is becoming harder to read.
Git history is there if someone wants to go back and check :wink:


6 değil 7 :)


лучше всякие пути, которые специфичны для твоего компьютера доставать из отдельного .property файла, и не коммитить его, а коммитить только файл *.property.etalon где есть только ключи, но не значения.


Will make correct indent


This now returns NULL - this breaks boblight addon - RenderCapture is only allocated once here:

https://github.com/xbmc/xbmc/blob/master/xbmc/interfaces/legacy/RenderCapture.h#L40

and know when you start up kodi it will be NULL and legacy/RenderCapture is not prepared for handling the NULL m_capture pointer ... also it never resets it to some valid rendercapture instance because its only doing it once in the c'tor


You would save yourself a lot of trouble when you cherry-pick things like this if you follow this simple process for committing:

```git commit --author="Author <email@address.com>" -m "[commit message]"```

This way authorship is kept in tact, and you're accurately displaying what's happening.


@robertkowalski that's implementation behavior.

> The **fprintf** function writes output to the stream pointed to by stream, under control
of the string pointed to by format that specifies how subsequent arguments are
converted for output. If there are insufficient arguments for the format, the behavior is
undefined. If the format is exhausted while arguments remain, the excess arguments are
evaluated (as always) but are otherwise ignored.

(And `printf` is just a special case of `fprintf`)


OK. (it's a bit annoying that UHC needs comp but whatever.)


Because index.php would redirect to the correct links for example if the user has the article path set as example.org/wiki/ then index.php would redirect to that. This is so the link is not hard coded. and we know that if someone changes index.php to a different name it wont work.


If you're only ever planning to have one language, then I'd recommend against using `<FormattedMessage>` as its inherently more complex. You could use a combination of the other components instead.

The core ideas behind the `defaultMessage` property of the Message Descriptor is that it will be extracted using `babel-plugin-react-intl` during the build, and the translation files `id: translatedMessage` will be fed into the `<IntlProvider>` at runtime via its `messages` prop. The `defaultMessage` in source also serves as a fallback in production in case the translations weren't loaded for whatever reason.

So my worry is that making it configurable would cause as easy out for people that would then bite them in production. It would probably be best if you would setup an issue or PR to further discuss this so it's visible to anyone else interested.


OK. Typo in commit message (regiser).


Comments are missing, add for each function and their purpose.


Alright, https://github.com/etsy/phan/commit/534b08753143b4022d8c4421655c3549f3875656 seems to be making CI pass. No idea why i'm seeing 'trueExpr' and you and Travis CI are seeing 'true'.


You would save yourself a lot of trouble when you cherry-pick things like this if you follow this simple process for committing:

```git commit --author="Author <email@address.com>" -m "[commit message]"```

This way authorship is kept in tact, and you're accurately displaying what's happening - especially when you're not changing anything. If you change something, then sure - mention where it came from - but don't act like you're doing anything other than cherry-picking - and as such, follow proper git / open source process.


There's a tension here between the Shaderc API concept of dialect and codegen semantics controlled by target env vs. the Glslang concept of "message rules".   I'm ok calling it coder's choice to decide which way to represent the concept internally.  So this is ok with me.
But future readers should watch out for the mismatch.

Secondly, there is another parameter to glslang's parse API (the 9th arg, a bool), "forwardCompatible" that determines whether the parser should issue an error for use of a deprecated feature, or various warnings for things like use of a reserved word as an ID.  So that *also* controls messaging behaviour of glslang.  Currently shaderc just turns that off by sending kNotForwardCompatible (==false) into that argument.

I'll file an issue to add a shaderc API to control that behaviour.


Right.  Will add necessary indents. Thanks.


Niye 16 ki?


Events get the instance passed as second parameter. you can repolace this line by adjusting the function parameters to

```
'click button.updateAnswer': function (event, instance) {
```

I try to use `instance` as variable name for the template instance everywhere, but some old code still uses `template`.


Plus <a href="<?php echo Skin::makeInternalOrExternalUrl( wfMessage( 'helppage' )->inContentLanguage()->text() )?>"> wont work since 'helppage'  message has a mediawiki link and a user would then need to override that. It is better to use a non-hard coded solution.


Provide a sample comment on the canonical request style being used.

```
// getCanonicalRequest generate a canonical request of style.
//
// canonicalRequest =
//  <HTTPMethod>\n
//  <CanonicalURI>\n
//  <CanonicalQueryString>\n
//  <CanonicalHeaders>\n
//  <SignedHeaders>\n
//  <HashedPayload>
//
```


maybe switch lines 121 and 120


还包含 good （精华）


this is duplicate logic to `click button.deletePost`. I think it should be possible to merge the two functions.


unack, I still dislike the naming here, you're changing the meaning of `doccomment` with this line, could you make it a new variable?


FWIW this is the right size for me, chewable.


TeamCity pull requests :: netty [Build 893](https://garage.netty.io/teamcity/viewLog.html?buildId=4136&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6229, ignored: 21 Build time: 00:41:48


didn't you want to use annotations btw.?


a return annotation would be nice IMO


Sorry, -1 primarily for the request to change "glsl" to "opengl" in the enum value names.
While you're at it, please elaborate the comment on the message_rules_ member varaible.


We are in the IRC channel #nethunter on freenode almost 24h/365.

Github is ok for quick things/bugtracking/reporting etc, think that everyone involved in the project reads this conversations. So is a good communication channel imo).

Hope to see you soon in the IRC so we can have more fluid conversations.

0/

Jaime.


There is no validation here so all logged-in users can delete all comments!


почему решил не указывать package где лежат твои REST-сервисы?


Agreed, I kinda missed that, will fix in next commit


done


@Zteve Give some thought to a bit of a `TestPageUtils` that mirrors the `PageUtils` and makes this a little bit nicer.  We could use a callback to build the response payloads based on page or something.


True, removed .read-only


tbh I would expand it right here: `line, column, comment = _extract_doc_comment(...)` and then return `None` for all of those if something's wrong


I'm guessing it might be the php-ast version. I'm on the last tag 0.1.1. I don't see what changed in the last few commits since 0.1.1 that would cause this though. But yes your last commit finally fixed it for me.


@barrydegraaff, @manens is right!

I didn't gave my consent to this and I'm actually not really keen on moving to a more restrictive license. Please recover the old license file. Thanks.


believe it or not, *here* I'm missing a very *brief* documentation comment with a description what the `simple` is, same for the two functions below


Pěkné:)


Shouldn't this be "inherits from `ActiveRecord::Base`"?


Please delete obsolete code right away. You can add the rationale in a commit message.


thanks


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (915 of 915 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-26.3%25-green.svg)


may decide it


Это самое главное изменение в этом коммите.


Made compatible with Java 6.
Switch statements with strings are Java 7+


Do we expect that this will actually end up being a useful hostname, or is it just kinda last-ditch "try something"?


The gave error as per my testing. If we remove this line, then in the 2nd onward session of the same connection if server restarts, Reconnection doesn't work... it gives error. Please check


@rldhont if you'd like to update the test cases as you go, I'd suggest first merging #2541. Otherwise, I'm happy to help with updating the test cases to reflect the new behavior.


Update this to support 0


No permission checks so all logged-in users can edit all comments.

Though because the userId is always set the comment would be updated to belong to the user that changed it :-)


I've updated them in the original spec


To continue the discussion, you'll find proposal code at the PR #2590 [Processing] Manage default value for parameter


Thx, Jaroslav.


I added a port mapping line to https://github.com/IQSS/dataverse-ansible/blob/master/README.md but can make it more prominent if you think that would be helpful?


I had already deleted this line in my commit. Did you merge my commit properly(or handle merge conflicts if any) ?


Needs documentation


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


Le 16/12/2015 06:02, Masatake YAMATO a écrit :
> In c.c, c parser doesn't reset AnonymousID.
> Do you think resetting it in each new input is better?
> (I don't have no opinion.)

I'm pretty sure it's better to reset it, because not doing so would mean
different tags being generated depending on the order files are
processed (or if they are processed all at once or separately).

Also, it doesn't make sense for CTags itself to give IDs unique to
something larger than a file as we don't have any control on which files
we are called on, so we can't guaranteed it's unique project-wide,
system-wide or something-else-wide.  So yeah, IMO better to have it
unique per parser run.

BTW, that's something we "fixed" in Geany because otherwise we get
ever-growing IDs as we don't reset the parser (so we parse everything as
if it was the same run); so that's another reason :)


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 114](https://teamcity.labkey.org/viewLog.html?buildId=291871&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:16


Most of the patches have been out there for quite some time, and I run them
in production environments.

On Wed, Dec 16, 2015 at 2:25 AM, Georg Schönberger <notifications@github.com
> wrote:

> OK,
> I would prefer your v2.8a branch. Do you consider it as stable?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/goochjj/pound/commit/98768806614cb611c147996c3ea86b135acdbf70#commitcomment-15006595>
> .
>


This change broke /home/postprocess/processEpisode.


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1989 of 1989 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)


Remove spurious semi-colon


I'd turn this if around: `if pos != -1: return ...` and then save one indentation level for the rest


Good point! Feel free to make the change.


Le 16/12/2015 06:10, Masatake YAMATO a écrit :
> I'm sorry what I wrote above is wrong.
> C parser doesn't generate a tag for anonymous struct.
> It is just refereed from a member of the struct.
>
> […]
>
> I think it is better to make a tag for __anon1.

Oh?  That's odd -- and Geany's c.c does generate the anonymous
enumeration tag.  But regardless, yes, I also think it's way better to
generate the real anonymous tag, otherwise we get "orphaned" children,
and miss some information.


Could we return an enum for this rather than an arbitrary string? (similar to SolarPanelState) Means the client doesn't have to figure out what the possible status messages are


"Returns" instead of "Grabs" to be consistent


Hmm, maybe.  Right now it's being put into a dictionary to aggregate by product id, which loses any ordering you had at first, so it'd take some restructuring of that.  That seems like it could get messy though, so I think I'll just keep as-is.


instead of :

memmove(private_key, seed, 32);
memmove(private_key + 32, public_key, 32);

you can replace it by one line :

memcpy(memcpy(private_key, seed, 32) + 32, public_key, 32);
and remove #include <string.h>


KRPC methods can return lists ;) Better to return List<String> to save the client parsing a csv


simpler, would it be, if this protocol inherited from `WMFTitleListDataSource`.  Then, conform to both protocols separately, our classes would not.

writing all my comments like this until SW comes out


It's not over yet.


Ah, yes, it says, "If you're interested in testing Dataverse locally using Vagrant, you'll want to clone this repository and edit the local port redirects if the http/https ports on your local machine are already in use."

In the case of my Ubuntu box `vagrant up` fails fast because of a port conflict on port 80, which is my cue to change the port. Failing fast is a very good thing! :)

On Mac it doesn't fail fast (I'm not running anything on port 80). It runs the whole playbook but then I can't see the homepage at <http://localhost> which bums me out. Maybe we can have someone else try it on Mac and report back if they see this too.


Return a ```List<String>``` (as above)


hey dude


No problem. It is helpful to me. Even I need to make these changes to run the program on my local machine. On my office desktop I have these files in a framework folder.


In this way bCNC.bat is obolete and could be removed.
I've just  make a test on a clean machine and it works by simply double click the bCNC.py.
@vlachoudis  do you prefer to let it for old users?


hey dude what's up


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1153.e60da012e6133bb1e9795ec8da33a569cda509cc](http://build.palaso.org/viewLog.html?buildId=67398&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout (new); tests failed: 1, passed: 774, ignored: 6 Build time: 00:35:38
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


If that's the case, why isn't it public?


Bla bla bla


Uppercase D


Uppercase A


do not move here


> All I see in this commit is some moving around of code and the renaming of the latter variable to Eigen3_DIR. Not sure whether that qualifies as copyrightable changes either.

I only added my name to make clear right away that the file contains changes.


If the value is plain text, how does it surface in the REQUEST object? As `{"body" : "a string"}`?


Glad you did this instead of pulling in a whole mock library for the moment. :)


Are you sure this should be `i*2` instead of just `i`?


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


Niye 3?


Can you add a bit to the README about what request mimetypes are handled and how they surface in the REQUEST object in the notebook?


Cift pointer olmayacaık. Zaten  yukarda `double *P[7]` yapmışsın, sonra şimdi tekrar double **p yapmıştın. Çok karıştırmışsın.


I noticed this and  #//// above. Looks like a C leftover or something. What is it?


Ooops... Sorry.. Wrong issue number >.< Meant issue #221


Could you please not use doccomment? I got it confused with document. :p
doc_comment is easier to read.


saw approval from lukebakken
at https://github.com/basho/riak-ruby-client/commit/b64dd0a61444bc3663ad45c31c19594d7e6bb86c


merging basho/riak-ruby-client/bk-gh257-empty-results-error = b64dd0a6 into borshop-integration-257-bk-gh257-empty-results-error


basho/riak-ruby-client/bk-gh257-empty-results-error = b64dd0a6 merged ok, testing candidate = 7c052343


Consider `(X/Y)/Z`.  Given nonnegative integers X, Y, Z, the exact result of `X/Y` will be `N + f`, where `N` is an integer and `0 <= f <= (Y-1/Y)` (since X is an integer).  Then `(X/Y)/Z = (N + f)/Z = N/Z + f/Z = (N' + f') + f/Z` where `N'` is an integer and `0 <= f' <= (Z-1/Z)`.  We cwant to ensure that `(f' + f/Z) < 1`. Expanding, `f' + f/Z <= (Z-1/Z) + (Y-1/YZ)` and so `f' + f/Z <= (YZ-1/YZ)`.  That will always be less than 1 *assuming X Y Z are nonnegative integers* (which they seem to be in this case).


Bu zaten yapılmıştı, sizden istenen bir şey değil.


Awesome. Sorry again for the breaks.


Changed it to return null if not found


This didn't seem to add anything to me.


Thanks!


Shouldn't we remove the line 'RMQConnectionPool.remove_connection(self._url)' as well? I know, i should have done this earlier :P


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


commit LGTM


Still don't understand how this is a _todo_


+ değil * olacak!


Any additional benifit of replacing self._connection.ioloop.start() with self.run() ?


Thanks :)


Ah, thanks.


I think the interface needs an "Init" or "Alloc" functions that can be called after playback has started.


OK.


I fail to see how this is related to Network either.
IMHO there is a gaping hole in the xdg categories where (amateur) radio / SDR  is concerned.
I have just in the last few minutes (before reading this) modified our (Mageia) spec for 2.4:

desktop-file-install \
    --remove-category='Network' \
    --add-category='AudioVideo' \
    --add-category='Audio' \
    --add-category='X-Mageia-CrossDesktop' \
    --dir=%{buildroot}%{_datadir}/applications \
    %{name}.desktop

I think Audio is more relevant than Network but neither are ideal, and the above brings it to the same place in the menu as most of our other Radio / SDR applications.

BTW the icon appears to be missing from the install in this version.


Fix #5


How about making the test a little more descriptive?
```java
@Rule
public final ExpectedException expectedException = ExpectedException.none();

    @Test
    public void noConnection() {
        expectedException.expect(IllegalStateException.class);
        expectedException.expectMessage("No connection provided");

        QSurvey survey = QSurvey.survey;
        SQLExpressions.select(survey.id).from(survey).fetch();
    }
```


No, we’re getting a proposal ready to post soon.

Mike

> On Dec 16, 2015, at 8:44 AM, Brian Gesiak <notifications@github.com> wrote:
>
> Will do! Is this the sort of thing that can't be discussed on a mailing list or evolution proposal?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift-corelibs-xctest/commit/3bab9bd76d93ff6be71379d1beb84e6df12ad71d#commitcomment-15019738>.
>


@manens , @dploeger I don't think putting an MIT license is going to help the Zimbra project, as people are even less pushed to publicize changes. And we already don't have enough people giving something back.

Looking at the mail below, I decided to change the license, but of course, that does not apply to all the code that was already committed before I changed it. That would mean everything is still MIT, but new changes are not. That is what I preferred.

Since I value what you both are saying, I will move this back to MIT, but I probably won't maintain or update this repo any more.

> From: "Dennis Plöger" < develop@dieploegers.de >
> Cc: "Barry De Graaff" < barrydg@zetalliance.org >, "Barry de Graaff" < info@barrydegraaff.tk >, "Zeta Alliance Founders" < founders@zetalliance.org >
> Sent: Thursday, 10 December, 2015 18:49:36
> Subject: Re: hello-js-zimbra-goodbye-zimbra
>
> Hi guys!
>
> Sorry. I will simply make Barry the admin of the organization and he can
> decide, what to do with the work. All my work is released under the
> MIT-license, so you're basically free to do what you want with it.


This comment should probably go above the belen = .. line since that is where -1 is set.


OK. (BTW, Cabal has always had this bug, but `removeDirectoryRecursive` follows symlinks. We probably shouldn't use it.)


Could you please create a git tag for this release? That eases packaging.


We have these instances for most other types in these modules already, and we do actually use them. The ones we could probably omit these days is the Read and Data instances.


#NinjaCommit


OK. (This has all been tested on UHC? Setting up the builder with UHC would be nice.)


"as a binary big endian 32-bit signed integer"


@abhinavjain241  Hey I tried installing it and I believe these are the following things you need to install additionally apart from the ones you have mentioned already .
- daemontools
- libiw-dev
- libsdl1.2-dev


See travis-ci/travis-ci#4771


@megrymo not 100% sure about this edit. lmk your thoughts


Probably a good idea, I will add this in.


In principle they all ought to be given their `Compiler` value, and not be given the `ProgramDB`. So I don't see UHC as unreasonable here (at least not now that I stopped it looking at the `PackageDescription` and `inplace :: Bool`).


payant ? il me semble que c'est gratuit discourse


OK.


was ist mit den constraints, tags, data (hab ich noch was vergessen?)


can you see http://localhost:8080? and which version of OS X? I wonder if System Integrity Protection is blocking ports <1024


s/bridge// here


Do you mean in the "Cadastral reference" help? Those are provinces that have a different website to look for it. Such websites  be mentioned in every language. (pe. people from Galicia moving to Araba but still accessing the form in Galician).


Why do we need this? Since the connection remains same here, i don't think its required. It was required in reconnection above since the connection there was changing.


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (915 of 915 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-26.1%25-green.svg)


I tried this a week ago and it broke camera. I will try again soon. Thanks!


There needs to be a comment explaining what the "recache" trick is, in the source code.


I'm afraid it has not. I've just tried to be careful to not actually change behaviour. I don't think it's critical for a release that goes along with GHC for it to be tested with UHC, we can pick up fixes for them for later releases.


a) Congratulations for your first test !

b) Please read section "Parameter Tweak" on
https://github.com/glinscott/fishtest/wiki/Creating-my-first-test

The proper SPRT 1 Elo for such test is 4 instead of the default 5 that you used.
But this would not have changed the test outcome.

c) Your test idea was original.

Doubled pawns in the center are not that bad in some positions,
unless they are isolated too, but isolated are handled separately.

You must also consider that some of the doubled penalty is handled in the PSQT
table, some indirectly in the Connect bonus (when a pawn is both phalanx and connected),
and some is also handled by the king pawn shelter.

Also, this doubled penalty had been tuned to death at long time controls.
But sometimes, hand tweaks can work too. But rarely.

d) Generally, before running a test, you can look at what others have tried on similar ideas.
It can give you ideas, and also a sense of what might work or not.

You can find out by clicking on the History button at the left
For example, a search on Doubled will return this
https://groups.google.com/forum/?fromgroups=#!searchin/fishcooking_results/Doubled

e) Good luck for your next tests !


Will do.


Should be FromResult(null), CompletedTask is for void.


Change of plans, this changes will remain since it need to be static.
Too many changes needed.
Still can't do static injection very well.


Thank You. Very nice point!


On Wed, Dec 16, 2015 at 09:19:03AM -0800, pbrunet wrote:
> Ho, nice, may be you can add a test? :-) Or change the existing one for argmin/argmax?
yes!


Regarding the links, lets them be untranslated. Indeed most of those FAQ pages does not have translation even the translated url already exists with the spanish text. One step at a time. Right now form strings are enough for the form to be published.

Besides, I don't know how did you managed translating FAQ before. I will ask Francesc.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1154.e60da012e6133bb1e9795ec8da33a569cda509cc](http://build.palaso.org/viewLog.html?buildId=67400&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 774, ignored: 6) Build time: 00:11:19
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/00d4d71b868039a8a485c75c52a24cf976cf3abb


merging basho/riak_kv/ts1.1/timeseries = 00d4d71b into borshop-integration-1309-ts1.1/timeseries


basho/riak_kv/ts1.1/timeseries = 00d4d71b merged ok, testing candidate = 2e7daca4


You are right, I went the quick way ;)


Two return statements here?


Nope. I guess I still am a derp. Will fix.


The function name is different because `SizeConsiderFunction` returns the actual consider function while `ElementaryAbelianConsider` is the consider function.


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 853](https://teamcity.labkey.org/viewLog.html?buildId=291873&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 24, passed: 223 Build time: 00:26:07
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.cas.SSOwithCASTest: <no details avaliable>

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

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

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


commit LGTM


I think this is not required since channel closes automatically when connection closes.


Aha! Let's get you clear on github flow at some point.


I have not looked at whether or not the mangling is ok or incorrect, but the commit message here needs to be improved if this is going to go into tree.

Specifically, the first line of the commit message should give at a glance what you are changing. If I read the commit with git log --oneline, I have no idea what you updated.

Here is what I would do.

----
[abi.rst] Fix incorrect mangling in substitution mangling documentation.

This commit fixes an incorrect mangling of a substitution. Before the commit, it was CS0_zoo. This is incorrect because .... Now it is the correct value CS0_3zoo.
----

See how it is much cleaner and clearer just from the commit message what I am doing?


@mikepetersonccv I just tested your fix on our prod test and it's nice and quick now too.


TeamCity YaccConstructor :: YC.Core General Build [Build 514](http://teamcity.codebetter.com/viewLog.html?buildId=204478&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Why you touch Utils class; That means another conflict for me :-1:


dev.sh is executed via bash


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


I know I've seen this before, but I can't understand why? We log.error with just message, but if DEBUG enabled we add Throwable to it. Why not just ```logger.error(message, t);```?


Лол))) автор действительно был в истерике когда писал)))


Might want to break these down into methods?


Лучше использовать обрамление выражения скобками вместо косой черты, если требуется длинное выражение:

```python
target_free_at_s = (rdkit_mol0.GetAtomWithIdx(target_point_second).GetTotalValence() -
                    rdkit_mol0.GetAtomWithIdx(target_point_second).GetExplicitValence())
```


Ok, so the question is: What does `theme=dark` do? Which background color settings are they settings, that I couldn't with single style commands?


Test? (No GHCJS support?)

I don't understand the getUserPackageDB implementation. Is this path really not calculated by Cabal today? (Is it code in cabal-install? Why wasn't it moved over?)

Do you actually ever return a directory? I don't think so, because for directory package DBs you just look at the binary cache file, right? (Maybe a remark that this function only picks up changes if you RECACHE).


id should just be id, not this.id (since it's assigned by SQL and not your Article constructor function)
```"data": [this.title, this.category, this.author, this.authorUrl, this.publishedOn, this.markdown, id]```


in the same connection nobody reconnects!!


OK.


@rajatchopra in the evolution of how we manage plugins, we need to answer how to pass kube-specific stuff down to plugins.  Do we really want to be building CNI plugins that only work on Kubernetes?  That seems to defeat the purpose..


actually basically this is after the merging.


Boşu boşuna tüm matrisi geziyorsun. Böyle yapmak yerine aşağıda toplarken bir değişkene toplayıp en son yerine atayabilirsin.


Javascript is just using this as the convenient place to store the information.   Just putting in the URL simplifies some what.  The GET variable is simply how we store it, it could be changed out to a # instead.

And regarding your comment on changing u to username is simple and can be done with just a few variable changes.


Zaten sz'yi yazarken sizeof ile çarpmışsın tekrar çarparak 8 kat fazla bellek istiyorsun.


I was suggesting to remove all the "helper" methods that did not go through the service API (and used the internal fake implementation state directly) as those will not be available when we switch
to the official fake (when available). I thhink the only exception was to provide a way to trigger a purge of deleted projects.

Also, some of those helper methods were mostly useful specifically to our tests.

It may be OK to have some helper methods *based* on the RPC however I doubt if `clearProjects` would be that useful outside our tests.


Sorry for the inconvenience, but I don't officially support installing via bower anymore.
However, since bower supports installing any git project, it's still possible, it just doesn't offer any convenience meta data as the `main` property.


since we are doing self._connection.ioloop.start() which is of the old connection.


Super cool, thanks! :monkey_face:


You're right this should be immutable


Thank You!!


F ve H matrisler Y'nin içerisinde, dolayısıyla Xpitch ile ilgili bir şey olmamalı..


You didn't, that test wasn't run explicitly, but
double r = K[pv][0] + log(d * 1.05) * log(mc * 1.05) / K[pv][1];
was an improvement over
double r = K[pv][0] + log(d) * log(mc * 1.05) / K[pv][1];

Anyway, I welcome such combos from time to time.


commit LGTM


That thing marks the beginning of a huge redirection, I can remove
it..  ///// \\\\\

It also looks like the elemental stones
http://static1.1.sqspcdn.com/static/f/545460/12328141/1306007590800/stoneslable1.jpg


Aynı şekilde A ve B matrislerinin toplanmasında YPitch'in işi yok.


@arunoda
Can't create issue on this rep, so I'll leave it here:
[Here](https://github.com/kadirahq/page.js/commit/3f404dd90ee6d806af046e41b2faa195d940e629?diff=unified#diff-50e3e6872ec225cd4caa566f1f15e89cR429) and [here](https://github.com/kadirahq/page.js/commit/3f404dd90ee6d806af046e41b2faa195d940e629?diff=unified#diff-168726dbe96b3ce427e7fedce31bb0bcR429) I'm getting this error:
```shell
Uncaught SecurityError: Failed to execute 'replaceState' on 'History': A history state object with URL 'http:' cannot be created in a document with origin 'http://localhost:3000'
```

Caused by URL with multiple slashes (empty URI parameters), like `http://localhost:3000//`, `http://localhost:3000///` and etc., unless first parameter is presented


int değil double olmalı!


great!


Again to not to loose any hanging connections or callbacks this is a safer option!!  Anyway by doing this we are not loosing anything!!


Thanks for the reply, mainly interested in me change current radio station or maybe toggle to another quest ;)


Bunu yapan `matrand()` fonksiyonu vardı yukarıda. Onu çağırmanız yeterliydi.


Getting too many "bad mass flow limits". Possibly all due to maxmassflowavail set to zero. Will track this down and do something. Also some odd diffs in results in HAMT files and HVACTemplateFanCoil file. Will check these out.


Matt, I would seriously consider moving it out of constructor and into _@OnScheduled_ method. Sort of "do all that if it wasn't done before" kind of thing. Just look at GetKafka as an example.
The main issue here is that this constructor will execute just because the NAR is sitting in the 'lib' dir and will be alive and monitoring things even if I am not using it at all in my flow.


So you just need to make a pull request to have these changes in your local machine.


Not really!! Just wanted to keep the same flow...Basically sugar coating!


The problem is that anything that extends filter creates a Nokogiri DocumentFragment, and by doing so, it calls `parse` on Nokogiri which does not handle certain string characters very well.

Notice here: http://docs.hhvm.com/hhvm/FAQ/faq#enabling-hack-mode-on-php-files-in-repo-mode

vs.

http://devbig429.prn1.facebook.com:8080/hhvm/FAQ/faq#enabling-hack-mode-on-php-files-in-repo-mode

But, look, the table does render: http://devbig429.prn1.facebook.com:8080/hhvm/configuration/INI-settings#supported-php-ini-settings

So, I basically want to create a Nokogiri document fragment where the string text is *unmodified*. Then I think this way works.


You should create a method that will just generate you an sobject with the data prepopulated.


@Ankush96 Add it in the readme then. :)


Typo: resturn/return

I don't think this should stop this patch from going in, but I generally have a poor opinion of manually remembering what places you have probed. Way easy to forget to that you probed a location, the resulting bug is difficult to find (some strange nondeterminism) and you're constantly playing whackamole.

Here's an alternate approach: make a new monad for tracking file dependencies, don't export a MonadIO instance, reimplement System.Directory with mandatory tracking. (Or maybe some people would say you should just instrument syscalls...)


all tests pass:
success: http://buildbot.bos1/builders/ruby-client-test/builds/429


OK, fine by me.


Done.


That URL will not be there in RMQConnectionPool since it did not stop till now!!


I know we can't get rid of them. But I marvel at how long it takes Cabal to build at -O, I suspect deriving has to do with it (since many of the slowest modules are defining big datatypes with lots of deriving) and I wonder if there isn't anything we can do about it. Not anything for this patchset, of course.


Should it be just ```Collections.unmodifiableList(descriptors);```?


which files a re opened? This leaves dirty stuff in the install :-/


We have a test that checks absence of such annotations


I think there's an issue with this function. My impressions is that as it is implemented now, it returns the minimum at the first iteration, no matter the outcome and whether self.compareMinima exists or not


Was ist `data`?


> Uh hm I didn't consider undocumented struct members private yet. While they have no description they still appear in the API documentation and are in the public parts of a struct (so removing them is an ABI/API concern), so for me they've met enough criteria to be generally public.

My main point here is that the structure *definition* for `GeanyBuildInfo` is currently fully private (`GEANY_PRIVATE`), so *no* well-behaving plugins can access any member.  And as you can also see, it's absent from the generated Doxygen, so we say people can't trust what's in it.

So IMO making it public now either mean
* we fix the API we unexpectedly broke.
* we introduce this API again (for no good reason).

And as we already broke it anyway, and nobody seem to have complained, I would think let's keep it out, wait and see.  @elextr do you happen to know why this would be public?


I get `/ZI is not supported` for VS2013 x64 builds. VS2013 x86 work.


This whole problem is solved if I just use HTML here instead of markdown. e.g., `<table><tr>`...

Then we can put it in the `MarkdownFilter` pipeline.

We can decide if trying to get the preMarkdownFilter thing is worth it.


OK (modulo comment on previous commit.)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1155.e60da012e6133bb1e9795ec8da33a569cda509cc](http://build.palaso.org/viewLog.html?buildId=67402&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 774, ignored: 6) Build time: 00:12:43
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


Wildfly опредляет их сам, если их указать, но не работает


The latter is essentially the approach we do take in cabal-install in the new branch. However, converting everything over in Cabal lib as well right now is a bit much. But I agree we ought to have a build monad at some point in Cabal, to take care of several things, including this.


Damn. This is due to latest hard rebase (after Core to Base renaming).


revert-5367-#5252_TinDiscrepancyNewVRFModel (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1979 of 1980 tests passed)

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


I would change this to a formatted string with the time specified in UTC for 2 reasons:

1. User readability.
2. There will be no debate on which timezone was used for computation.

I can implement this in my PR #223 for review.


It appears, that dash-newline starts a sequence entry.

Although http://www.yaml.org/spec/1.2/spec.html#id2759963 says
"Block sequences indicate each entry with a dash and space",
"Example 2.4.  Sequence of Mappings
(players’ statistics)" at the same section
does not contain any space characters before the newline, suggesting
that the space may be omitted in this case.

The more technical description
(http://www.yaml.org/spec/1.2/spec.html#id2797382)
says we expect a non-(ns-char) which
includes tab and space  and (somewhat unexpectedly)
CR, LF (and maybe some others)

```
  [184] 	c-l-block-seq-entry(n) 	::= 	“-” /* Not followed by an ns-char */
                                           s-l+block-indented(n,block-in)
```

where ns-char is http://www.yaml.org/spec/1.2/spec.html#ns-char

```
[34] 	ns-char 	::= 	nb-char - s-white
```

where nb-char (non-break char) does not include newline.

So at the end of the chain of negations:  `b-line-feed` and `b-carriage-return`
are accepted as well as space or tab as separator between the dash and
node.


I tried to do some overrides of `def html`, `def doc`, `def initialize` that are found here:

https://github.com/jch/html-pipeline/blob/master/lib/html/pipeline/filter.rb

but could not get the right combination,


tvle hnus fuj... 2x body v html dokumente :D zmaz to prosim ta


Server may restart and code will need to reconnect automatically


If it's not too much effort, a comment explaining why having functions in Programs is unavoidable, and why it's fine for ProgramDb to leave them out, would be nice. Mostly hoping you looked at this, figured out why that was the case, and can write it down.


Well, I use <http://localhost:8080> all the time on my Mac for Glassfish installed on the Mac directly. I wouldn't want to use the port for a demo instance in Vagrant. (I like being able to run `vagrant up` and continue working on the code.) So I wouldn't want Vagrant to try to use port 8080.

Good theory though about https://en.wikipedia.org/wiki/System_Integrity_Protection though it seems like it's part of El Capitan, which I haven't upgraded to yet. I'm on 10.10.5.


ten new line tam byt ma :D


The bug that this is a work around for should now be fixed.  The new version, however, is equivalent to what the compiler will generate at sufficiently high optimisation levels, so is probably better to keep in general.  This is probably one that should be pushed upstream (adding `const` would also be nice, while you're at it).


commit LGTM


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9604](https://teamcity.mesosphere.io/viewLog.html?buildId=93888&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:09


my bad, I confused `continue` with `pass`


That's why i am saying we should remove the line 'RMQConnectionPool.remove_connection(self._url)'


Testowy komentarz.


It works


I copied that from somewhere, I think it's so we only dump the stack trace in DEBUG


@fefifofum  oh yesss I just checked...cam is broken in stock app but its working in snapchat :/


Yeah - thats intentional. That protocol literally does not depend on inheritance of the other protocol. So it seems odd to make it inherit from it. You can also see where we may be adding other protocols here and you may want to mix and match them composition wise.


old connection! Connection here here is remaining same, not changing.


Also we need to follow this style across all libraries, any such offending function i.e undocumented should be called out and open a bug against it.


I suppose another option is to not use the `HTML::Pipeline` stuff at all and just make this a rogue, standalone filter that does string/regex-based processing on the raw text string. That way we control the string totally. Then we run it through the `MarkdownFilter`


OK. (I still think we should drop support for the other compilers, but maybe that's just me :) #2714


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1156.e60da012e6133bb1e9795ec8da33a569cda509cc](http://build.palaso.org/viewLog.html?buildId=67404&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1 (1 new), passed: 649, ignored: 5) Build time: 00:04:16
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData: <no details avaliable>

```


No GHCJS support yet, but should be similar. I've not tested GHCJS support in the new cabal-install branch either yet.

Re getUserPackageDB, no, currently neither Cabal nor cabal-install has to calculate the actual location of the user package db, it's always referred to by name, which translates into `--user` etc flags to ghc/ghc-pkg. Here to do the change tracking we must pierce that state hiding that ghc is doing.

Re recache, that's quite true, which matches up exactly with when ghc itself will notice the changes. Can certainly add a comment to this effect.


init() in init {} :)


yeap, thanks!


@jmunhoz does that mean that without aws4 working all these tests are not going to run?


I reversed the diff, thanks for catching this!


what is the process by which these plugins are built and pushed?


Rename `filename` to `item_name` since directories are copied as well.


commit LGTM


Nope there is one in the if condition...


Hi Barry,

@barrydegraaff You modifyed the licence and you also took the copyright without mentioning the original author of the software. I don't think copyright violation will help "Zeta" or Zimbra.

In the email you gave, I read you can do what you want if you respect the existing license/copyright.

And at the end, you can not modify this to GPL by yourself, GPLv2 is contaminating, That mean that the software that integrate a cummit licenced under GPLv2 becomes fully GPLv2. So your explaination on existing code is a non-sense IMHO.

Sorry that this will prevent you for comiting to this project.


TeamCity YaccConstructor :: YC.Core General Build [Build 514](http://teamcity.codebetter.com/viewLog.html?buildId=204478&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 284 Build time: 00:17:33


My bad...we are doing new connection.ioloop.start() and old one connection.ioloop.start() is still there so just want to ensure that old ioloop does not create any problem in case if they do...


Shouldn't `ly` be different from `1.0` for rectangular lattices?


@steve-manywho is this code still needed at all, or is it now redundant and can be removed?


Yes will do


I am wondering if we really need to do that (resetting class loader). The ```ExecuteScript.class.getClassLoader();``` is actually the execution context class loader, so by setting this new class loader as context class loader it's as if you executed _addURLs_ method on URLClassLoader. So what's the point of resetting it back? Or do you see some danger in having 'modulePath' to remain?


@mada1019 please note those changes, all sensor tests now work again


You commented on the wrong line and github hid that line and I thought you are suggesting one to add. Sorry!! We should definitely remove that live.


This hunk is undoing a previous hunk - accident?


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


@disprosium8  Should i throw an error when querying anything but exnodes ?
The reason it always returns empty for other fields is because `parent` is hard coded in `getRecParentNames` in db.py. Maybe i can parameterize that , though am not sure if we have any usecase for that.
I was initially planning to do it as a separate controller, but I needed the big argument parsing logic chunk present in the networkhandler and am not clear on how to move it to just the exnode handler.


Needing this means there is a more serious bug before. Having a duration with a value without unit defined should never happened.
I pushed a more complete fix.


It's a bit distressing we can't interrogate GHC for this info. And indeed, I don't see any way to get `versionedAppDir` or `resolvePackageConfig` from GHC on the command line. I'll submit a PR to mention that these are reimplemented in Cabal.


ok i retry


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1157.e60da012e6133bb1e9795ec8da33a569cda509cc](http://build.palaso.org/viewLog.html?buildId=67406&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 775, ignored: 6 Build time: 00:05:44


Yes this is a cut-and-paste error from InvokeScriptProcessor (which added other things). I will make the change


Here filePath is temporary file path with random name.


Could you solve it like this? I can build my own kernel on my PC, but on my server I get the same errors.


What's the standard here? Should I add preceding and trailing zeros?


In server restart too it is working fine.


Or should it be no placeholders? Obviously this isn't either.


your are missing app:, if you dontl add it it is not goign to be recognized when try to define your global variables


moje pytanie


moja odpowiedz


I [optimized](https://github.com/bcosca/fatfree-core/commit/b34ec65e1e3ee478d496f128f5a57259ade727b7) this a little since `is_callable` can return `TRUE` for strings as well.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1158.0c89048ac5ef60eb5d9863fee250f33967f99f41](http://build.palaso.org/viewLog.html?buildId=67407&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1 (1 new), passed: 611, ignored: 4) Build time: 00:02:29
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


is @formatter:on/off an eclipse setting?


No está protegido contra SQL Injection.


This was fixed and re-based back in to the original commit long ago.

What version of portage did you create this patch against?


@nucro looks good thanks


@threesided, Doesn't need any configuration, should be removed.


Removes lots of redundant headers, and at least one needed one using MS V120 compiler. I haven't tested any other combinations of platforms & compilers. System headers tend to be different so this will likely cause more problems.


Cheers!


Same comment as before about doing something substantial in the constructor of the processor


Even if this is a hook, this is an special hook because all fo this code is executed before start running all cucumber code, it is  abtter place to define it in env.rb file, beacuse your are defining there all relevant information for your execution


![hai arachnid](https://cloud.githubusercontent.com/assets/15968213/11850049/13234bea-a3f9-11e5-95fc-ed3fbdc90c00.png)


![hai mite](https://cloud.githubusercontent.com/assets/15968213/11850054/1c65983e-a3f9-11e5-906a-9aecea919491.png)


Maybe it will be better to remove this and next debug messages. IMO, it's overkill.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1159.0c89048ac5ef60eb5d9863fee250f33967f99f41](http://build.palaso.org/viewLog.html?buildId=67409&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1 (1 new), passed: 611, ignored: 4) Build time: 00:02:28
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData: <no details avaliable>

```


see commit: e571aa31dfc17c56052f21c7a1bbe2504563fd04 in an up to date checkout.

This was done before the merge into master


So does this send a message only when the sender is ignoring the target, or does this also send a message when the target ignores the sender? If it is the latter, which it appears to be, then I somehow am going to need to refactor Complete Softmute to capture the message beforehand which is currently not possible with your event handler at the lowest. (This would also circumvent your antispam checks though.)

However, it also appears I can just set the message empty in messages.yml so I can handle the message sending myself, which I'll do for now.


Do you really need a lock here? You have a single thread scheduler so it is essentially serial


Now I see where is the trick in this formula. X and Y are swapped. Nice!


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1160.0c89048ac5ef60eb5d9863fee250f33967f99f41](http://build.palaso.org/viewLog.html?buildId=67411&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 610, ignored: 4) Build time: 00:02:30
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData: <no details avaliable>

```


\o/ ;)


Corrected spelling in two places: in an error message and in a method name in the tests.


@steinar this change is great!  Not sure why the class loader behavior has changed.  I am going to open a PR for this and maybe Bela and shed some light on things.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1161.0c89048ac5ef60eb5d9863fee250f33967f99f41](http://build.palaso.org/viewLog.html?buildId=67413&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1 (1 new), passed: 600, ignored: 4) Build time: 00:02:12
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


commit LGTM


Классно, спасибо. Не буду, обещаю


@barrydegraaff I think that for this one too, @dploeger need to give the property of the source code or to modify the License of the code he own but it is not possible to modify to GPLv2 without that.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1162.0c89048ac5ef60eb5d9863fee250f33967f99f41](http://build.palaso.org/viewLog.html?buildId=67415&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 625, ignored: 4) Build time: 00:02:56
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


If `Contents` does not have to get everything from the header, maybe we don't even need this. I'll figure this out as part of #223.


EconomizerIntegrationNew (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1947 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-98.14%25-yellow.svg)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2965


Yes.


Maybe you're right. Need to check it.


@barrydegraaff this one too, @dploeger need to give the property of the source code or to modify the License of the code he own but it is not possible to modify to GPLv2 without that.


Short, sweet, and to the point!


harder to test and doesn't add something useful imo. I could just say if the first element is None then """, but then I could just return None for all, right?


If the original author was omitted, it was not omitted by my, it clearly stated  Copyright (c) 2013, Dennis Ploeger


Oh interesting, I didn't read that fully... I thought your indentation was off. =) Personally, I find ternary syntax to be more clear in this instance...

```
return this.range ?  this.options.getSelectionBarColor(this.scope.rzSliderModel, this.scope.rzSliderHigh) : this.options.getSelectionBarColor(this.scope.rzSliderModel);
```

Or even more succinct (my overall preference):

```
return this.options.getSelectionBarColor(this.scope.rzSliderModel, this.range ? this.scope.rzSliderHigh : null);
```


sure, haven't changed it from "outlining" the function^^


Was this notification was dropped intentionally?


huh? why that?


Why not just

```/lib /usr/lib
/lib64 /usr/lib
```

Does anyone actually use `lib32`?


I'm going with a pthread_mutex.  The condition variable solution would have been unnecessarily complicated.


you wanted annotations :)
since I've never written them I wanted to be consistent and rely more on documentation.
But if you like annotations more, I write them down^^


Yes. Now manual plugin update is invoked from the dialog, and the dialog shows the update status in a label.


@barrydegraaff Can you confirm that all commiters agreed to switch from GPLv3 to GPLv3 ?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1163.0c89048ac5ef60eb5d9863fee250f33967f99f41](http://build.palaso.org/viewLog.html?buildId=67417&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 775, ignored: 6 Build time: 00:05:34


I think I'll stick with sched_yield() here.  As you said, it's startup and shutdown processing, so performance isn't an issue.  And pthread_yield() is a gnu extension which wouldn't get us anything in this situation.


Oh, right, yeah. I see. Didn't notice you'd bound $_GET to a var, sorry!
I'll pull this in and change up the names myself.
Ta


@GovInTrenches should notes go in the readme? As a layperson it's hard to discern what to do with the **civic.json**.


Le problème vient dans un premier temps d'ici, nbWord vaut 11 alors qu'il n'y a que 10 mots dans le fichier.


JSON string, right?


Space is missing between when and (


Also, JSON string, right?


Space is missing.


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


You're right. Change to novaFolder.endsWith().


Design Guide for pervasive e-paper


Je le prends en compte.


OK I think it's done. I'll willing to try and speedup more code, if I
can understand it


TeamCity YaccConstructor :: YC.Core General Build [Build 515](http://teamcity.codebetter.com/viewLog.html?buildId=204479&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


such clean :)


TeamCity YaccConstructor :: YC.Core General Build [Build 515](http://teamcity.codebetter.com/viewLog.html?buildId=204479&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:01:13


ce om, nume de functii in romana :(


You may find it picky and we might be talking about milliseconds, but the length could also be determined *before* creating the HTML elements for each character.


This does not seem right to me.
The logical size of the L0 hole should be the data (not metadata) block size.


Perhaps we can add a helper function `localIndex :: FilePath -> Index` that'll just set `localCacheFile` to ``indexPath `replaceExtension` "cache"``.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 72](https://teamcity.labkey.org/viewLog.html?buildId=291890&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:15:55


@mikepetersonccv I need to do more testing because I am seeing in some cases a new error (The cast to value type 'System.Decimal' failed because the materialized value is null. Either the result type's generic parameter or the query must use a nullable type.) -- which may just be because I'm trying to deploy only the change in the 45fc9c3b3a7b0f8e78fd1e048584ac6ba885839b commit.


No contempla todos los casos!


> BTW the icon appears to be missing from the install in this version.

If you mean "make install" does not install it then yes, please see issue #311.


I'm curious - why are you disabling php-fpm caching by default?


@gunnarku Your are right. It shows white so I could not see it. Pls, disregard my comment.


IMO LINQ in this case will make the intent of the code less obvious. I do want to look at refactoring this file in general in a future PR though.


> On Dec 16, 2015, at 9:57 AM, Jordan Rose <notifications@github.com> wrote:
>
> If that's the case, why isn't it public?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/e9a2e1e1281cdb54d660ac1f0f04fbff83d0f807#commitcomment-15021491>.
>
It isn't because it didn't need to be (we had ++/--), but with the removal of those operators, we might have to reconsider that.


But then we should change all occurances of `/lib` troughout the policy to `/usr/lib`


Sounds good! Thank you!


:+1: :+1: :+1:


Yeah there is definitely some unnecessary ones here.


One caveat here -- you'd be better off exporting these symbols always and either throwing a runtime exception where the clock is unimplemented elsewhere, or just using a slower clock.

We made the same mistake with network re: IPv6 support being conditional and it took Johan *years* to fix it --- every downstream user of the package will have to put CPP in if they want to prefer these clocks and haddock will just show the list for whatever platform it was run on.


None for all makes more sense IMO


because startposition and endposition related together closely


We're also trying to nail down loaded state terminology in JS land @ https://github.com/mapbox/mapbox-gl-js/issues/1715. Lets have a cross-platform discussion in https://github.com/mapbox/mapbox-gl/issues/5


with a `r_io_sundo_push` at the beginning you make the same work, don't you? because in any case is executed with the same parameters


It's not very intuitive what's the difference between "local" and "global". The global one lives in `~/.cabal`, so it may be argued that it's actually local too. Maybe `Index | SandboxIndex`?


For this exact reason I miss CoffeeScript's existential operator "?" :
    $scope.cm?.refresh()


Ich habe jetzt den gesamten Code in ein Maven-Projekt mit log4j2 verfrachtet.
Man sollte es jetzt in Eclipse importieren und wieder pushen können.
(File -> Import -> Maven -> Check out Maven Projects from SCM)
Ich würde 'Florian' aber erst dann in 'master' mergen, wenn es auf allen IDEs funzt.

@BrackCurly Ein paar kleine Sachen habe ich bei deinem Aufbau nicht ganz verstanden,
können wir ja morgen klären.


I was not able to convince other of this but i still think we should instead create a `systemd_helper_exec_t` to for this and then allow `systemd` to run these without a type transition.

Eventually everything `/usr/lib/systemd/systemd-.*` should be targeted anyways, and i believe it is probably for the best if the generators get run in with systemds' permissions for simplicityb


Style typo: missing a space.


LGTM.


no its a github username


Love


fast-forwarding end-to-end/timeseries to borshop-integration-1309-ts1.1/timeseries = 2e7daca4


Style nitpick: switch cases should not be indented. (I also would put that `return false` on its own line, since not all cases are single-line. But that's not such a big deal.)


No contempla todos los casos!


Yep, I'll just remove everything after that.
On Dec 16, 2015 9:07 AM, "Spuds" <notifications@github.com> wrote:

> Could you also look at the old function parse_bbc ... a few lines into it
> it has
> return $parser->parseMessage($message, $smileys); so all code after that
> point is not reachable. I'm assuming that function is there for
> compatibility with 1.0 addons..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joshuaadickerson/Elkarte/commit/70ccc4883b3bb3b2d75807bb28226f0cb6738149#commitcomment-15014785>
> .
>


In fact we could leave out the location of the cache completely, and just define

``` haskell
cacheFile (LocalIndex{..})   = localCachePath `replaceExtension` "cache"
```

I didn't do this because I didn't know if this was too specific (but it would be ok in how this is currently used, I think).


Looking at M3.0 in /user/messageselect.php then $CFG->messaging is not the whole picture:

````
// If messaging is disabled on site, we can still allow users with capabilities to send emails instead.
if (empty($CFG->messaging)) {
    echo $OUTPUT->notification(get_string('messagingdisabled', 'message'));
}
````

thus the solution is incomplete because it does not take capabilities into account and thus would lead to a regression.


The namespace is what I found on how to handle singletons in TypeScript. Seems somewhat odd. Or should the context be handled during the activation of the extension?


This should be "Oscar's Bio"


No contempla todos los casos


@alvarofe Obviously yes :+1:


![hai arachnid](https://cloud.githubusercontent.com/assets/15968213/11850596/3fc5849e-a3fc-11e5-8dca-0b78220c51db.png)


This file should really be in `./lib`.


I'd call this `maybeRepoRemote`.


I agree, it's rather a quick fix for Swing for the time being. I hope that JavaFX will handle high resolution displays better than Swing.

See http://news.kynosarges.org/2015/06/29/javafx-dpi-scaling-fixed/


Where in California?


Yes my concern here is that the sentence:
> For example, if you create a project with `projectName` "some-project-name" and subsequently call replace using a `ProjectInfo` object that didn't set the `projectName`, then the server will unset the project's name.
Might suggest that `resourceManager.replace(ProjectInfo.builder(myProjectId).build());` unsets the name.


Este método no es correcto!


`SandboxIndex` is fine with me, but `Index` isn't particularly descriptive is it? `RepoIndex`?


> Should I add preceding and trailing zeros?

I prefer them yes, so write `-0.85` instead `-.85` for example. If you want you can also align the decimal sign.


I'm not completely sure I know what you're talking about here.


I just cloned master and added the patch.

The commit you refer to is fine. Things break, when refactoring happens in d22abc6a686eaf204cab31387928ba85605a729b, since it misses that one occasion of self.checkdir.
So it seems things are broken since portage 2.2.22.


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


AS I understand, it uses Apple style buttons, margins or colors of elements. you can do it on your own if you like. it doesn't set background color at AFAICT.

> On 16. 12. 2015, at 19:18, iBaa <notifications@github.com> wrote:
>
> Ok, so the question is: What does theme=dark do? Which background color settings are they settings, that I couldn't with separate style commands?
>
> —
> Reply to this email directly or view it on GitHub.
>


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


:D


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


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


Looks like the menus on the feed list aren't working. Can't get to preferences and can't get to individual interactions. The popover works, but selecting a menu item doesn't.


Array<Nothing>


Should there be asserts to make sure all the types and levels in the block pointers are properly set/the same or the block pointers are zeros ?


LGTM.


Someone care to explain this one to me please (via email, not git comments)?


Thanks. Putting all these things into --info would be best, minimises the number of calls we have to make to ghc when configuring. It's there in the --info for the global package db.


Voce pode usar ng-class pra definir qual classe css usar, e nas classes css voce deixa as cores.


Removing it from geanydata breaks the abi I'm afraid.

I can follow your reasoning for BuildInfo. What about others like AutoIndent?


halt template_data, im moment JSON. Also String


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1452 of 1453 tests passed)

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


division by 0 can occurs only when remisejs is 100 and there is already a test on it. I understand to have the if (remisejs == '') remisejs=0; to be sue to have remisejs defined.
But why introducing bpjs ? And why moving the $("input[name='price_ht']:first").val(price); into the if. If the remise percent is 100, then price_ht must be set to 0, isn't it ?

Can you describe the use case that was not working (which field you entered) and that need this change ?


:iphone:


Yes, `RepoIndex` is better.


Why are we testing for the prepared list being null when we're in commit? We should only be calling commit if there's something prepared, otherwise ... well ... what are we trying to commit in the first place?!


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


Yes, that'd be fine too.


If you remember we spoke a while ago about using `api/3.json` to check which themes are included. Since no other folders are included anymore, I guess I can use the folders themselves to get that info. Unless you want revert the exclusion of the api folder or think of another file where to store this info (e.g. `package.json`?)


Ok. (`repoRemote'` was by comparison to the--partial--`repoRemote` accessor, but I'm fine with either name.)


Mal tabulado!


The last check for stone is unnecessary, since stone passes for isOccluding.


Nombre mal escogido!


Looking here:
https://github.com/Zimbra-Community/pgp-zimlet/graphs/contributors

You can see 95% of the code is my own work, most of the rest is translations, the remaining part is very small. And can be easily replaced.


No estan contemplados todos los casos!


+1, this is much nicer.


Also this change was recommended by Zimbra.


Hmmm I think this is a question of preference... Your syntax is shorter but not clearer to me.

Anyway, what do you think of this new version?


@jef-n love it! Thanks!


Why does the registry warn about environment variables it doesn't recognize?


Code that converts map coords to paper coords.


No estan bien contemplados todos los casos!


Yes, that is correct


@blowdart Can you have a look?


лучше super просто.


That's what I get for only manually testing the new requests. :flushed:


I dont think we want that thing to happen. At least i did it on purpose

> On 16 Dec 2015, at 19:53, Álvaro Felipe Melchor <notifications@github.com> wrote:
>
> with a r_io_sundo_push at the beginning you make the same work, don't you? because in any case is executed with the same parameters
>
> —
> Reply to this email directly or view it on GitHub.
>


This now no longer makes sense since there is no jQuery plugin.


Okay; may or may not matter; I don't know.  Might as well let it be.  I doubt that build failure is deterministic (e.g. would always fail).


Выше просто `def self.print_format`, а тут прям `class << self`. Это идентичные конструкции, вторую используют когда нужно сразу много методов класса определить. Но вперемежку их точно лучше не использовать :)


yeah but you see that they directly change because of the line above?
`line, column, doccomment = doccomment`


Hmm, good point about the timezone. I would prefer the time to be kept in a "computationally friendly format"--meaning that, for example, if it is a `time.time` object you can format it any way you like using `strftime`, but if it is an already formatted string then you have to convert it back into a number (a process fraught with its own set of problems) before you can do anything with it.  Better to convert it on the way out.

How about a timezone-aware `datetime` object?


Or a UTC datetime, if you prefer.


El metodo no funciona conforme a las especificaciones!


Looks good!


I'm really happy that these issues get discussed. Representation and acknowledgement of diversity is certainly an issue in the artifacts themselves, but it is also a matter of the community's willingness to discuss the topic. And here I see open discussion and a willingness to consider change. That's awesome!


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1454 of 1455 tests passed)

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


Процедура вставки элементу в дерево.


> The Windows versions are coded without testing, so I hope I did not break something :)

Unfortunately, you did: https://gist.github.com/LB--/5eccbffdf6872408fbfc


EconomizerIntegrationNew (lgu1234) - i386-Windows-7-VisualStudio-12: OK (1990 of 1993 tests passed)

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


Auch wenn es in der Datenbank dann JSON ist, kann es hier ja anders gespeichert sein.


I'm not sure.  Certainly the case where every execution requires exactly one cache reset every time seems pretty unlikely.  And I'm not really sure how to gauge the likelihood of having one particularly nasty input among otherwise fine inputs that would lead to hitting the states limit. However, a case that may be more likely is where the combination of states for different inputs causes you to hit the cache limit even though any particular execution does not (and even though the inputs are similar in some ways, small variations in input can lead to some substantial differences in states).  That becomes a question of whether running on a clean cache for every several input rows is still faster than running the nfa.  Based on the results of running only one input in nfa vs. dfa, I believe it probably is.

Maybe a still somewhat simplified version of RE2 would be to reset the cache when you fail over to NFA.  You still end up with some things maybe unnecessarily running the NFA, but you don't end up switching everything to NFA if you don't need to, nor having one bad input spoil an otherwise efficient execution for the rest of the rows.  This of course also assumes that the extra overhead of repeatedly trying the dfa first for queries that do need to run on the nfa for everything repeatedly is comparatively less. I'd also be okay leaving todos as an improvement for the future, and seeing if we encounter an issue during benchmarking or in some real use case.

Agreed that better memory accounting would be a good improvement for the future.

Action item that is definitely for right now is updating the description of this test to reflect implementation and what it is actual testing.


Let me tag @ondrae who might be better to answer this -


I love this approach to naming z-indexes.


No se podian cambiar las cabeceras!!


test coverage fell, so this likely doesn't work the way I think it does, I'll need to investigate


No se podían cambiar las cabeceras!!!


right side


what kind of symbol is this???


Fix-allocation-for-feneSolarDelaySeq (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1981 of 1981 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)


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


Oops, I'm describing pre-1.2.2.0 `directory` behavior. (Which is in our support window, but I suppose is fading fast.)


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


@yuslepukhin No problem :-) Thank you for reading the incoming changes and helping to find bugs!


No se pedia construir este metodo!


Esto no debería ir en timeTick!


would it be possible to add a case in vim_test.js for this?


ja schon. Ich mein das mit der Datenbank ist nicht in stein gemeißelt, sollte sich eher nach der api richten. Bin nur rüber gegangen und habs nicht gesehen.


healing point take from health??


fair 'nuff


No contempla todos los casos!


Tak zwany Delegated Event.


Only in oSSL 0.9.8c   Should we version check??


yeah removing it would not really be possible without an ABI break…  maybe we could replace it with `void` pointer in the meantime.

> What about others like AutoIndent?

Not sure, ideally I would prefer not to add them to the API just for this.  So… not sure, in case of `AutoIndent`, as there is no public members after it typedefing it to `gint` or `guint` would surely be fine -- and it probably would work in all cases, but I'd have to double check the C standard (which might not even give specifics).
I guess ideally GIR wouldn't include it and so wouldn't care about not knowing the type.

BTW, if all members are exported, and GIR picks them all, this means they get public in the GIR, right?  this seems sub-optimal, although I don't have any sensible solution to suggest.


why not just do a `display:none`?

Curious what the `left: -99999em;` gets us


Qué ha pasado con este commit?


No contempla todos los casos!!


+    def take_damage(self, damage_points, other):
+        if self.health > damage_points:
+            self.health -= damage_points
+        else:
+            self.health = 0


I can change language in the switcher. But the interface doesn't change language even if I restart Sakia.


Ah shoot, sorry about that. If you're installing from npm, basing it off folders seems like the way to go. Or you can GET https://bootswatch.com/api/3.json.


1. Это вполне осмысленная проверка, но крайне неэффективная. Если у тебя кто-нибудь делает `movies.select(&:comedy?)` — то это 250 вызовов такого метода, и 250 раз на одном и тот же `@@array_of_genres` будет `.flatten.uniq` выполняться. Лучше проследить за его уникальностью в момент наполнения (или вообще использовать класс Set, который по своему смыслу — именно набор уникальных элементов)
2. Этот код всё ещё будет странно ввести себя в таких случаях:
```ruby
movie.comedyz # => z отпадёт при chop, и упс
movie.comedy?(15) # => уж не знаю, что имел в виду программист, но такой код работать не должен
# и даже...
movie.comedy = true # это читается как вызов метода `comedy=` с аргументом true, и визуально имеет смысл — а вот поведёт себя совершенно диким образом :)
```


Probably there so upstream can warn you if you mistype an env var you're expecting it to turn into a config setting.


TeamCity YaccConstructor :: YC.Core General Build [Build 516](http://teamcity.codebetter.com/viewLog.html?buildId=204481&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


This is not normal. Did you run `gen_translations.py` ?


fast-forwarding master to borshop-integration-257-bk-gh257-empty-results-error = 7c052343


Okay, I'll do that and try to make writers for those soon.


Yep, I did.


> white-space: nowrap;

Hadn't seen that before. Nice.


See #1261


No contempla todos los casos!


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


Why remove the count check? If user deletes all markers they will not be restored.


:dancer:


Hi Regelink

I can not help you, as I have no rpy2 at the moment.

Sry.


What is the size of the file `i18n_rc.py` in `src/sakia` ?


then switch 120 and 119...


This should be `array_shift`.


@alexischr ^

if `compile_only` is set (via `-c`), `as` should still be executed.


nit: looks very non-rustfmt-y way to format the arguments.


No contempla todos los casos!!


I just had an idea...
What if there was a link to a `.jpg`  mockup of `.top-bar` so that other devs quickly see what the styling is supposed to accomplish.


I don't have `i18n_rc.py` in `src/sakia`.


@hackmastera Yes! It should have been in the 6.4.0 release. :unamused:

Good find, @hackmastera. I'll tweak the release notes and write an email up.


actually, using `curry` is not so elegant here due to the new parameter order of `joinWithSeparator `


Yes, the `pwd` works from within the GitHub repo. I'm going to add this to the post too.


In `src` maybe ?


Ok, Rocky!
Got it; I will use this advice for my future tests!
Thnkas,
stockfishdeveloper


It'd be better to put this "head" inside SoQLAnalyzerHelper so that when it comes time to change to chained queries it's possible to change that and follow type errors to find the places it's used.


the old `minElement` in Swift 1 did not return an Optional - so the quick fix was a force-unwrap, but that's not something too advisable in educational material


I'm not crazy about properties implementing module. Can't we have a properties module that's separate? Why not move out your SingletonModule and push that into the injector?


Mine is ~ 360ko... It seems you are missing something here. can you post the logs of the `gen_translations.py` ?


@mfrederickson Is this a commit we want in 2.3.2?


maybe a new example should be thought of without using `toInt`


```bash
/home/moul/projects/sakia/build/res/ui/import_account.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/import_account_uic.py
/home/moul/projects/sakia/build/res/ui/transactions_tab.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/transactions_tab_uic.py
/home/moul/projects/sakia/build/res/ui/informations_tab.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/informations_tab_uic.py
/home/moul/projects/sakia/build/res/ui/contact.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/contact_uic.py
/home/moul/projects/sakia/build/res/ui/account_cfg.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/account_cfg_uic.py
/home/moul/projects/sakia/build/res/ui/member.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/member_uic.py
/home/moul/projects/sakia/build/res/ui/create_wallet.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/create_wallet_uic.py
/home/moul/projects/sakia/build/res/ui/preferences.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/preferences_uic.py
/home/moul/projects/sakia/build/res/ui/community_cfg.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/community_cfg_uic.py
/home/moul/projects/sakia/build/res/ui/currency_tab.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/currency_tab_uic.py
/home/moul/projects/sakia/build/res/ui/about.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/about_uic.py
/home/moul/projects/sakia/build/res/ui/toast.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/toast_uic.py
/home/moul/projects/sakia/build/res/ui/homescreen.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/homescreen_uic.py
/home/moul/projects/sakia/build/res/ui/wot_tab.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/wot_tab_uic.py
/home/moul/projects/sakia/build/res/ui/community_view.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/community_view_uic.py
/home/moul/projects/sakia/build/res/ui/identities_tab.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/identities_tab_uic.py
/home/moul/projects/sakia/build/res/ui/network_tab.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/network_tab_uic.py
/home/moul/projects/sakia/build/res/ui/wallets_tab.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/wallets_tab_uic.py
/home/moul/projects/sakia/build/res/ui/certification.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/certification_uic.py
/home/moul/projects/sakia/build/res/ui/password_asker.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/password_asker_uic.py
/home/moul/projects/sakia/build/res/ui/transfer.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/transfer_uic.py
/home/moul/projects/sakia/build/res/ui/mainwindow.ui >> /home/moul/projects/sakia/build/src/sakia/gen_resources/mainwindow_uic.py
/home/moul/projects/sakia/build/res/icons/icons.qrc >> /home/moul/projects/sakia/build/src/icons_rc.py
/home/moul/projects/sakia/build/res/i18n/ts/fr_FR.ts
Updating '/home/moul/projects/sakia/build/res/i18n/qm/fr_FR.qm'...
    Generated 296 translation(s) (253 finished and 43 unfinished)
    Ignored 40 untranslated source text(s)
/home/moul/projects/sakia/build/res/i18n/ts/fr_FR.ts >> /home/moul/projects/sakia/build/res/i18n/qm/fr_FR.qm
<RCC>
      <qresource prefix="i18n">
<file alias="fr_FR">qm/fr_FR.qm</file>
      </qresource>
    </RCC>

/home/moul/projects/sakia/build/res/i18n/langs-1450293507.qrc >> /home/moul/projects/sakia/build/src/i18n_rc.py
```


That should be
Bitte gebe so viele Informationen wie möglich an!
(My main language is German)


This is the run of `gen_resources.py`.


Or maybe `$command = implode(' ', array_map('escapeshellarg', $args));` might be better.


You're right, that's better. But I was wondering about how to report syntax errors without `provide`, anyway.


This step is correct for specific fields, but if you foe example need to validate a complete response of a json and avoid some "dinamic" values. You should keep your json parsed and perform a comparison between what you get form service and what your expected, and only fo those field that you expected you should check the values. That would be the way to perform that validation in a real scenario.


with the new `count` function in Swift 2 this could actually be written more simple as `$0.nuts.count`, but it would not feature the `|>` operator as much


```bash
Updating '/home/moul/projects/sakia/build/res/i18n/qm/fr_FR.qm'...
    Generated 296 translation(s) (253 finished and 43 unfinished)
    Ignored 40 untranslated source text(s)
/home/moul/projects/sakia/build/res/i18n/ts/fr_FR.ts >> /home/moul/projects/sakia/build/res/i18n/qm/fr_FR.qm
<RCC>
      <qresource prefix="i18n">
<file alias="fr_FR">qm/fr_FR.qm</file>
      </qresource>
    </RCC>

/home/moul/projects/sakia/build/res/i18n/langs-1450293654.qrc >> /home/moul/projects/sakia/build/src/i18n_rc.py
```


@tresf
The focus here is the new class aproach, I will change a lot of the code in tray manager soon.


Fixed.


Mal tabulado!


Yes, @manens. @barrydegraaff just added the Zeta-line. That's fine.

In the mail you cited, Barry. I really meant do what you want, BUT change the license. Sorry, I should've made that more clear.

I don't think, a broader license would mean fewer contributions. Only because you force people by license, doesn't mean that they will actually contribute their changes. I mostly contribute to software because of local changes I don't want to apply each time I do an update. ;-)


Comentario mejorable


broken build: idepix is included in the s3tbx pom but appears to be a work in progress or the poms haven't been pushed.


Yes, that's what the behavior will be.  When I meant "second" I should have specified that I was referring to this line:
```java
resourceManager.replace(ProjectInfo.builder(myProjectId).build());
```


I will push something tomorrow. The entire thing is a bit hacky ...


Fix-allocation-for-feneSolarDelaySeq (JasonGlazer) - 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)