Why is this interface has no any method/function?


Me equivoque en el titulo del comitt , ahora el admin puede controlar que usuarios estan autorizados para usar la intranet


`searchPattern`


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


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


`0.000` is a valid value, technically meaning `>0.000`.  However, we didn't have this error before (and we certainly did have values like `0.000` before. I'm not sure what's the best treatment here. Brute force  is to equate it to `0` and make it give `NA` instead of `NaN` so that the report compiles at all.  Thanks, I'll implement it on my next revision.


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


点菜菜单对话框


Okay, I see you are aware of the possibilities. BTW I thought that maybe a simple web formular to create dependent CSL style may be handy (and might be able to avoid some errors). What do you think?


It is not a copying of code, it is a moving of code.

The visitor syntax is not that unwieldy as our experience with BooleanFormulaVisitor in CPAchecker shows.
There is not more code duplication with implementing `visit` than with implementing the `is*` methods.

The `is*` methods are bad for implementors of solver packages: lots of single methods that get outdated, no clear overview of what exists and what needs to be changed, possibility to easily introduce errors like two contradicting methods returning `true` for the same formula instance.

The `is*` methods are bad for users: no way to have compile-time safety that all cases are handled, runtime safety needs to be coded extra and can easily be forgotten, no way to find out all relevant places where user code should be changed if an `is*` method is added.

The `visit` method is good for implementors of solver packages: a single place to read, understand, and change, can use `switch` for solvers where we can for example get the kind of a formula in form of an int like for Z3.

The visitor is good for users: can provide compile-time checks that all cases are implemented, even when cases are added, can if desired replace compile-time checks with runtime checks (just inherit from DefaultVisitor), encourages use of existing traversal algorithm instead of manually reimplementing it all the time.


Another rewrite:

Other unikernel implementations have made trade-offs different to those made by MirageOS.  Some excel at handling legacy applications by making the most of existing OS codebases rather than building clean-slate implementations.  Some target a wide array of possible environments, or environments complementary to those supported by Mirage currently.


Hi Jsha:

I found this update is not work.
Another package "golang.org/x/net/publicsuffix" (Line: 144) seems to override this package.
Would you please have a look?

Thanks,
rony


Nope not going on any PaaS just yet. VM only for now. Possibly containers in future.


Done


add gitter badge


点菜菜单对话框的头文件


Thanks for your reply - I deleted the comment a few minutes after I made it, figuring that instead of "hey I found a problem for you to fix" it would be better to say "hey I found a problem and I fixed it"... then I ran into a problem getting su to work on my AVD, so I couldn't test my fix.
I had updated chroot, using the menu option to do so, which starts a terminal and runs start-update. ExploitDB (and csv2sqlite.py) didn't get installed, so I did it manually. I saw the (paraphrased) code:
if(doFeedDb())...else  "Install exploitdb"

Since the return of doFeedDb() was being tested against for validation of the exploitdb package install, I thought it worth pointing out that the test would always come back True before it became a bug someone else would need to hunt down.

Is there a different, "preferred" method for updating chroot that would have included the new packages?

    On Tuesday, December 15, 2015 4:37 PM, 3boll <notifications@github.com> wrote:

 @smountjoy i cant find your comment, ill add a check, but now the exploit database comes with the chroot, so it is always installed.https://github.com/offensive-security/kali-nethunter/blob/2b7a2291a47a07d5028337dcd979edd4d219de81/nethunter-fs/build.sh#L106If you update the app for testing, remember to update the chroot also.my best, Jaime—
Reply to this email directly or view it on GitHub.


EconomizerIntegrationNew (lgu1234) - x86_64-MacOS-10.9-clang: OK (1942 of 1978 tests passed)

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


Why do we need to create a new API (even if it's private) for unit tests?


@ext Thanks. Fixed in be0c7d1ef082c4b1bf6534f0aa4fa9446ec73d31.


Comments in directives we cannot avoid; we might even actually want them, for instance if trying to check against a coding style guide that requests mentioning the  #if condition in a comment to the #endif directive.

Regarding UTF-8, it is the default encoding for XML if no encoding is specified in the prolog and no BOM is present at the start of the file, so UTF-8 characters should be fine. We 'only' need to worry about (ASCII) characters which could disrupt XML parsing.

It remains right however that the RE here will not match if comments are present in the directive, and that it would need to do some preprocessing on the directive. I might argue that in the specific case of Y2038, _TIME_BITS and _USE_TIME_BITS64, comments in the defines would not match the GLIBC coding style; but for the sake of genericity, I can add a method to the Python Directive class to produce a the directive string with any comments removed.

BTW: the coding style for GLIBC involves indenting directives by placing spaces between the initial "#" and the keyword. IIUC, that is not something cppcheck currently supports, right?


I noticed, too.


missing an `end` here


This commit forced update to correct typo


We will not know Insert is success or not.
Please change void to:
1- bool
2- ProductDto (include Id which is just created) for next step


mzia


> It is not a copying of code, it is a moving of code.

`getArity`/`getArg` definitely got copied from UnsafeFormulaManager. Or you suggest not having them at all?


org.springfox.ApplicationSwaggerConfig находится в модуле wf-base.
Может refresh сделать или один раз собрать проект, тогда wf-base прольется в wf-central


I would drop commented code from the PR


Where is the unit tests?


We will not know Update is success or not.
Please change void to:
1- bool
2- ProductDto (include Id which is just created) for next step


I rolled in your additions to the latest version of NewPing.  It's now version 1.7 and includes other patches not included in your copy.  Deleting your version or just pointing it to https://bitbucket.org/teckel12/arduino-new-ping/wiki/Home would probably be best at this point.


I have a date picker calendar that I use to allow users to select multiple dates using custom classes to style the selected dates. This change breaks that because the custom classes need to be updated as the page runs.

I have created a simple demo of this on Plunker using ui-bootstrap version 0.13.0. If I use version 0.13.1 or greater, the custom classes don't update. Is there anyway to make this kind of thing work in newer versions of ui-bootstrap?

http://plnkr.co/edit/wjJCCpzLaUUcau7Z4cgN?p=preview


Sure. It's not a high priority for me personally, though. We also don't get that many dependent style contributions.


i am not sure i understand. Can u give more concrete example?


We will not know Delete is success or not.
Please change void to bool


Argh. Will fix C++ to use str


relative path import is not suggested in Golang


Done.


This is quite vague. What are those conventions, are they written down somewhere?


Yep. I have added his name in some lines because I think it is not affected. I tried to translate closest meaning in Vietnamese. At least it fully and more easily understood than the translations before. I'm sure that. And if others don't like my name in translation, I'll remove them.


It only appears if __DEV__ is true. You can disable it by setting `console.disableYellowBox = true;`.


test


Something is wrong with the CSS now, @pwalsh, please copy the code again as I fixed it now, http://tarekamr.appspot.com/dataviz/godi2015 (Just change the FAVICON to OKFN)


I think we should not use other dependencies in these snippets


Pls remove unused code :-1:


oh.. ok.


Typo here, I think.


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1948 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.19%25-yellow.svg)


@kevinphelps, take a look at #4692.


s/projectFromServer/myProject


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

```


OK, case in point: consider `extractLiterals` in FormulaManagerView.

Currently it is:

```
  public Collection<BooleanFormula> extractLiterals(BooleanFormula f) {
    return myExtractAtoms(f, false /*splitArithEqualities not supported for literals */,
        new Predicate<BooleanFormula>() {
          @Override
          public boolean apply(BooleanFormula pInput) {
            return unsafeManager.isAtom(pInput)
                || booleanFormulaManager.isNot(pInput);
          }
        });
  }
```

But we think that ad-hoc checks are bad and should be replaced with visitors.
It becomes:

```
  public Collection<BooleanFormula> extractLiterals(BooleanFormula f) {
    f = applyTactic(f, Tactic.NNF);
    return myExtractAtoms(f, false /*splitArithEqualities not supported for literals */,
        new BooleanFormulaVisitor<Boolean> (this) {
          @Override
          protected Boolean visitTrue() {
            return true;
          }

          @Override
          protected Boolean visitFalse() {
            return true;
          }

          @Override
          protected Boolean visitAtom(BooleanFormula atom) {
            return true;
          }

          @Override
          protected Boolean visitNot(BooleanFormula operand) {
            return true;
          }

          @Override
          protected Boolean visitAnd(BooleanFormula... operands) {
            return false;
          }

          @Override
          protected Boolean visitOr(BooleanFormula... operand) {
            return false;
          }

          @Override
          protected Boolean visitEquivalence(BooleanFormula operand1,
              BooleanFormula operand2) {
            return false;
          }

          @Override
          protected Boolean visitImplication(BooleanFormula operand1,
              BooleanFormula operand2) {
            return false;
          }

          @Override
          protected Boolean visitIfThenElse(BooleanFormula condition,
              BooleanFormula thenFormula, BooleanFormula elseFormula) {
            return false;
          }
        });
  }
```

Surely you'll agree the syntax is quite bad?


Reviewed with Tony, who is very happy!


Why `System.lineSeparator()`?


should link to index route


Thanks Ja


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


Das ist gerade das, was ich gestern angesprochen habe: Eben NICHT die MongoDB ID als ID verwenden - über die MongoDB-ID haben wir keine Kontrolle. Falls sich diese verändert, fördern wie inkonsistenzen der verwendeten Zeichen innerhalb eines Einsatz. Bitte überdenke diese Änderung noch einmal.


Eigentlich ist diese Zeile der eigentliche Bugfix, sprich es fehlte die Zurücksetzung der Variable.


`setUp` is a normal method, not a class method.  Remove the decorator and change `cls` to `self`


J'ai enlevé les deux variables STARTOFTEXT et ENDOFTEXT car elles ne sont plus utilisées dans le code


Removed


Please, remove them as it really is not necessary to have your name there. Especially when it replaces important messages such as "Requires reboot". Credits (your name) is in the change log and it's enough.
You can claim your reward for providing translations in form of free PayPal transaction ID you can use for unlocking premium features. Thank you.


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (906 of 906 tests passed)

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


à la place d'avoir une boucle infini, je vérifie si l'itérateur à un élément suivant


html呢


I think it would be better to do portability another way, via CMake.

you added power specific assembler files to xtradb sources, this looks a bit weird.
Also  newly added enable_language(ASM) even if it is not generally required, only for ower


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.622.e103e71af89d7458ebc56cfbe69943b558dfae24](http://build.palaso.org/viewLog.html?buildId=67379&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:07:51


This can't be done using `Json::Value`?


The latest build causes TRCL to run smooth on the 8800 GTX. Last week is was unplayable on that card. There are a few flickering polygons here and there and some polygons seem to be slightly misaligned but apart from that it runs like on the 9600 GT.


nice~!! Thank you @icxbb-xx


Good catch – I've been utilising ES6 stuff and forgot to mention the Node v5 requirement!


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1007.e103e71af89d7458ebc56cfbe69943b558dfae24](http://build.palaso.org/viewLog.html?buildId=67375&buildTypeId=PalasoPrecise64v26Cont) outcome was **FAILURE**
Summary: Tests passed: 2096, ignored: 70; exit code 1 (new) Build time: 00:09:09


Now it should really be fixed... v1.5.2 out


I've just removed all of them. Sorry :'(.


Build is failing on this commit due to the removal of `devDocs`.


the gmod.SaveSave is added?


Sounds like a good approach for now.  I'm surprised it didn't throw the same warnings before.

In the meantime, consider what the value of the correlation should be if there's a zero in the denominator.  Probably it's the conventional NaN, but maybe it's something else in the context of these models.


https://nodejs.org/en/blog/npm/peer-dependencies/ please require react native as a peer dependency so we know which react native versions you support.


Is this check needed? An empty string will appear anyways, I think.


"whose host is set host to" -> "whose host is set to"


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

```


@JamesScottMiller this is an awesome program, jimmy!


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

```


Hmm, I think it's pretty clear. But we could add a note like this perhaps:

> This new NetworkError exception does not convey any information about the error(s) in _result_. Since errors are being muted, all information about the original errors is intentionally lost.

How does that sound? Any tweaks?


TMath already includes PI. In general you should avoid #define in c++ code because we can const double instead, which has much better compile checks.

Please use
TMath::Pi()
instead of defines.


Thanks for the proposition, it would be nice to have a limit condition, like after 10 redirections it stops ?


Of course you can write bad code in both cases.

Just use `DefaultBooleanFormulaVisitor`.

Your example proves my point: there is a semantic difference between the two code snippets, and it god obvious because the visitor forced you to think about all cases explicitly.


Thanks. Please, do the same for Marshmallow branch.


This method will block until the server thread is terminated.


Should we add implicit class FutureReaderOps[A](val rr: RequestReader[Future[A]]) extends AnyVal
and implicit class FUtureOptionReaderOps[A](val rr: RequestReader[Future[Option[A]]) extends AnyVal?


This finance related data should be displayed close to the other finance related data above.


These really should be unsigned. Newer forests will have them unsigned, it's only old forests that don't have this correct.


Pls remove this.Configuration.LazyLoadingEnabled = false;
because by default is false
https://msdn.microsoft.com/en-us/library/system.data.objects.objectcontextoptions.lazyloadingenabled(v=vs.100).aspx


quais ids foram atualizados ?


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


Nothing yet, but please stay tuned.

Mike

> On Dec 15, 2015, at 8:51 PM, Brian Gesiak <notifications@github.com> wrote:
>
> This is interesting. In Objective-C XCTest, unexpected test failures are caused by uncaught exceptions, right? What constitutes an unexpected error in Swift?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift-corelibs-xctest/commit/3bab9bd76d93ff6be71379d1beb84e6df12ad71d#commitcomment-15004672>.
>


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

```


> getArity/getArg definitely got copied from UnsafeFormulaManager. Or you suggest not having them at all?

I do not know yet whether `getArity` should stay in `UnsafeFormulaManager`. If it does, it should obviously not be copied to `Z3BooleanFormulaManager`. I was only talking about the `is*` methods.


All done :))). Sorry again :3.


This code is duplicated at lines 528-547. Is it 100% identical or just similar? In any case please create a function to avoid code duplication.


J'ai supprimé les conditions car les variables n'existent plus. j'applique directement les instructions contenus dans la condition inText


no longer useful?


With these 2 substitutions you can no longer set the title in the terminal. I made a pull request #416 to fix that.


Primarily off a hunch because buffer overflow crashes are usually an underlying issue. It made the build pass :-\


We will not know SaveChange is success or not
Pls change to bool


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.632.e103e71af89d7458ebc56cfbe69943b558dfae24](http://build.palaso.org/viewLog.html?buildId=67376&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests passed: 2096, ignored: 70; exit code 1 (new) Build time: 00:05:27


nice


This section about configuring TopicMassPhiScore is also duplicated  below. Please wrap it into a function...


Je n'ai plus besoin d'envoyer la variable STARTOFTEXT


Hi Federica, this is my written pdf, ipynb  and cvs couldn't be manually added you can find my data and notebook here
https://github.com/avigailvantu/newgettest/tree/master/Desktop/extracredit


This overridden constructor does nothing but call the parent's constructor, get rid of it


For server side, I use **Processing.runAlgorithm(alg, None, args)** and I need that some input has None value. So If we add **checkValidity** or **setDefaultValue**, I'll update this method.

@volaya, @m-kuhn and @michaelkirk: how we organize us ?
First merge #2541 ? Then add **CheckValidity** and **setDefaultValue** ? At last update **rumAlgorithm** ?


Je n'ai plus besoin d'envoyer la variable ENDOFTEXT


Can't you use `self._run.aim_and_fire` instead here?


Oops, looks like I made the mistake once and then copied and pasted


This constructor does nothing, get rid of it


period missing here


I didn't think it was that useful...


period missing after "Jack Zhu"


this was previously ignored, is this new error going to hurt us?


Looks like this has been "fixed" upstream for https://bugzilla.gnome.org/show_bug.cgi?id=756654, by removing the internal libyelcommon library itself (so now underlinking is not an issue). I suppose we could backport that patch instead of adding this downstream fix, but that aforementioned "fix" does not apply cleanly on top of our 3.18.1 tarball from Debian Testing, so I'm more inclined to leave our fix downstream while we don't upgrade to a newer version.


@stof thx also added


> Just use `DefaultBooleanFormulaVisitor`

Sorry I don't understand. Could you show an example? How would that make the snippet more concise?

> because the visitor forced you to think about all cases explicitly

Yes it does, at the cost of increasing the amount of code ten-fold.
Unfortunately, it's not OCaml where we could just do pattern matching...


I guess we don't need the note. I was thinking about "8.1.3.6 Runtime script errors"'s "report an error" algorithm which in step 6 goes in particular detail how things are masked, but none of that really applies here.


You're trying to create a run object without a trace file, which currently fails.  I'm working on creating a `BareTrace` class for this, but until then you have to copy a trace file here to construct the run object.


There is more code that proves my point: `Z3UnsafeFormulaManager.isAtom` is wrong. Like to guess why?

I found the bug by writing a proper implementation of the `visit` method.


Resolves issue with removing all feeds.. gotcha for users using import functionality.


Can you change this to `const` where possible? Otherwise awesome!


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

```


@TexusDark  You are correct. This will not change ANYTHING but the version number.

@thebigsmileXD Only the protocol matters for joining with a new MCPE version!


Isn't this the same as `!!name && name`?


@DixonD-git Ah, perfect.


looking not critical to me? want to reimplement form support first; so weekend


Oh that's easy.  Just revert the commit.


Looks good!


Also I think `name` expected to be a `string` and not `boolean`


I would remove 'also' and change "commands in command line" to "command in the command line"


Oh, hey MCPEpig! Haven't seen you in a while.


This is much better :)


Better:

```
for series, expected in zip(result, expected_result):
            assert_series_equal(series, expected)
```


Das ist ein guter Hinweis. Zur Lösung des Problems habe ich die equals-Methode der StockItem-Klasse überschrieben, was den Vorteil hat, unabhängig von der konkreten Implementierung des DAOs zu funktionieren.


this loses some information, is this intentional?


the first comment on GITHUB)


what is different when we change above to
get{return _context;}
?


> Like to guess why?

Because it doesn't consider `Z3_VAR_AST` or `Z3_NUMERAL_AST`?

OK but coming back to `extractLiteral`, can you give a version which doesn't occupy my entire screen?


Comment on the same line probably?


Foi feita uma reestruturação.
Possivelmente diversos...mas agora não está havendo debug no client.


Same here: comment on the same line?


The spec uses Hixie English, which not only allows singular they, it actually requires it with a "should". To my knowledge this is the only normative definition of English.


You're right it doesn't seem to matter. I wrote a quick script to check and up to 100000000 there are no differences at least :). There is probably a quick proof that could be done, although I am sadly out of practice and so have to resort to silly looping checks instead of proving it :(


Are you sure you need to pass the list of traces and the traceFormatter and change the Storage interface? Storage should not process the traces format. The contract for the send method is: I have these data (in string format) and I want you to store it.


Leftover.


Hmmm, this should work without the full path AFAIK


I'm having some issues with this. Is there a a net_ssl_config.h file somewhere?


why not error, here? Isn't the app unusable?


s/imports/import


Perfect, thanks!


The recursive query seems to involve a sequential scan of the table. For a table with 3.8M points it takes between 50-100s in my laptop, dependig on the maximum Z level.


Same comment as above


shouldn't this stay an error?


The example about the bike was supposed to refer to a toddler's bike. I guess that doesn't come through in the example though. But yeah, please feel free to propose more examples. We need them anyway.


You probably can use the Net class of NetStorage for part of this logic.


@JonathanMeans you'll need to re-configure (logic to generate that file on the fly just got added)


Guy's, I maybe over stepping a bit here, but the profiles directory names can be different than the ***-overlay and package-*** file names, which is what ends up as the file name on and inside the ISO.
For example my profile directory is named /netrunner/kde-netrunner but the files inside are rolling-overlay and packages-rolling, the ISO ends up named as netrunner-rolling-next-2016-01-x86_64.iso.


麻烦问一下，这个视频制作完成了，如何修改 秒拍这样的字样，谢谢


This doesn't change the performance but chooses tiles to consider more sensibly (the Z levels used depend on the table extents and the depth which is paremeterized as nz).


I think it is pretty obvious how to use DefaultBooleanFormulaVisitor:

```
  public Collection<BooleanFormula> extractLiterals(BooleanFormula f) {

    return myExtractAtoms(f, false /*splitArithEqualities not supported for literals */,
        new DefaultBooleanFormulaVisitor<Boolean> (this) {
          @Override
          protected Boolean visitDefault() {
            return false;
          }

          @Override
          protected Boolean visitAtom(BooleanFormula atom) {
            return true;
          }

          @Override
          protected Boolean visitNot(BooleanFormula operand) {
            return true;
          }
        });
  }
```

Yes, it is not the same amount of lines as before, but it avoids so many potential problems that I strongly prefer this variant.


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


:ok:


Turns out it's no longer relevant for upstream because libyelpcommon.la was removed.


Dieser Bereich des Source Codes hat sich zwischenzeitlich verändert.


The cast should be to `IterableInterval<?>`, not `IterableInterval<Object>`.


That is the alternative indeed. But for headers it might not be that of an issue. For source files you get probably issues with incorrect rebuilds. Is a failing rebuild of superbuild related to the cmake of superbuild file searching for other cmake files?


Nit. I like "For example, to  get the project we just created we can do the following:" a bit better.


Similarly, the cast needs to be `RandomAccessibleInterval<?>`.


@MelLain Have you tested this change?
I'm a bit concerned that we never do unit-tests for regularizers, even though it is very easy. Could you create a new file in artm-tests project, and perform a simple sequence:
- generate a `TopicSelectionThetaConfig` and then create `TopicSelectionTheta` via constructor
- generate a synthetic batch using TestMother (there are plenty of examples of this in artm-tests already)
- call CreateRegularizeThetaAgent on your regularizer, and verify that agent is not null
- generate some theta vector, and apply regularizer to that theta vector. Verify that the results are as expected, and hard-code expected values into Assert.


>> Like to guess why?
>
> Because it doesn't consider Z3_VAR_AST or Z3_NUMERAL_AST?

No. Next guess?

With the visit code, this bug would never have been introduced at all, and writing a proper `visit` makes this bug obvious, so it shows my point about `is*` being error prone and `visit` not.


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


Вообщем не страшно, я ее переделаю в более дружественную форму.


static const?


I think for the histogramming step we should not draw anything. To fill histograms you can use the ->Project() function instead of ->Draw(), and we should not set any canvas or drawing properties. These things should all be set at the plotting step instead.

Specifically, I don't think we should be saving any canvases in this step, instead we should only save histograms.


oh.. ok


Pls remove
because by default: variable of bool type is false


I'd give some thought to naming these methods `error` -> `logError`, or, importing the library such that instead of calling `error(args)`, you call `Logger.error(args)` or `Log.error(args)` or `Log.e(args)` (shadowing the library you wrap). A plain old `error()` makes it hard, at a glance, to tell if this function is just going to log, or actually raise and crash.


李玲侠


I'd say start with `setDefaultValue()` and `runAlgorithm()` in this PR, then merge #2541 into this PR and make sure the two are compatible. Not sure about `checkValidity()` afaics it's an optional added value which can easily be added later?

Thank you all for your work!


These strings too


s/list/a list


Thanks :)  I've updated `CHANGES.md` accordingly :)


test this please


Unfortunately this doesn't work for me. I get "buffer too small" in wxZipHeader::wxZipHeader, called from wxZipEntry::LoadExtraInfo. The file I'm using was created with Info-Zip 3.0 which I've been using for many years. If I comment out the LoadExtraInfo call, it works again. Do you think Info-Zip is at fault? What alternative should I be using on the command-line on Windows (called from an MSYS script)? Thanks.


For this one, the All Ideas page now shows icons for Likes, Backs, Team with a number next to each...

![image](https://cloud.githubusercontent.com/assets/5081281/11845444/ee497838-a3e1-11e5-86ef-d32d8587623c.png)

So the projection for getHeaders should still include team, and we should add `team: doc.team.length` back to the object that we're pushing to headers.


I don't like this. Cppcheck does remove comments and indentation by design.

Checkers for indentation and comments have been officially out of scope ever since I started the project.

Currently, if you write an addon that checks comments or indentation, you need to parse the source files manually.

If we would want to keep this information in general I don't think it's a good idea to put it in the directives.


Commit


+1
Did you notice that  `man curl_easy_setopt` states "That parameter can be a **long**, a **function pointer**, an **object pointer** or a **curl_off_t**, depending on what the specific option expects."? I mean, there are no other places using different types than the expected?


As I said elsewhere I don't like depending on other libraries for such small examples.


Need
`my $g_TOCEyeCatchBackup = "BACKUP_PART";`

at the top, ideally after
`my $g_TOCEyeCatch = "part";`


> Yes, it is not the same amount of lines as before, but it avoids so many potential problems that I strongly prefer this variant.

OK I'll re-write the code in `FormulaManagerView` with those.

> No. Next guess?

Because it doesn't consider equality in `NON_ATOMIC_OP_TYPES`? I think what I have mentioned is a bug as well.


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


I think that when ForeignKey attribute is removed, it still works ok
As convention of EF, it will be auto MAP


this is not true. In 3.0, there is no name anymore, other than the FQCN


I recommend using `#define FOO 1`, as that matches what  `-DFOO` on the CLI maps to, and if you happen to write `#if` rather than `#ifdef` stuff still works


ignore. you change it in the next commit.


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


Added more documentation in the comment block before the C++ as well as Python class.


If you change to check, you don't need pylint-check and pep8-check correct?


My email is dorien.drees@gmail.com


new pp forests should include the hiEvtAnalyzer, so you shouldn't have to worry about this difference on real data/MC.


Fix for #9


if anyone is wondering what this dumb commit is about, lust look away and don't worry about it! ;)   I was exploring some weird issue I was having with my SCSS file and why it wasn't responding.  I thought...incorrectly, that it had something to do with Haml and how SCSS might need to look differently with Haml...turns out that wasn't the case and my file was saved just outside of the correct folder and thus not activating.   I had to move the folder and return my SCSS file to it's original glory....just look away, there is nothing exciting to see here.  :sweat_smile:


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


Вообщем не страшно, я ее переделаю в более дружественную форму.

Ок. Просто у меня не очень производительный ноут и сборка даже одного проекта требует большого числа ресурсов, а netbeans и java-реализации очень прожорливы и мне чтобы лишний раз собрать проект приходится тратить уйму времени. А данная ошибка появилась тогда, когда я сделал слияние и намеревался уже выталкивать тестовый... поэтому не проверял соберется ли вместе с этой аннотацией проект. Думаю, что раскомментирую и запущу на сборку еще раз... И еще. просто не хотелось бросать лишнюю ошибку на гитхаб, чтобы затем отлавливать, поэтому и прописал в комментах к фиксу об изменении..


> I think it is pretty obvious how to use `DefaultBooleanFormulaVisitor`

HM I know why I haven't seen that --- my version of `DefaultFormulaVisitor` doesn't have `visitDefault`! Neither in `JavaSMT` nor in `CPAchecker`.


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


Hi Barry,

I guess Dennis want to change the licence of his software to GPL v2. Fine.

But he keep the copyright on his work and all committers keep the copyright on their own commit. Did you contact all commiters to midify the licence ?

Where did the discussion take place for the choice of the new licence ?


>> No. Next guess?
>
> Because it doesn't consider equality in NON_ATOMIC_OP_TYPES?

Yes, Z3_OP_IFF and Z3_OP_EQ are missing, but both are sometimes atoms and sometimes not, so there even needs to be more code added than just these two constants to NON_ATOMIC_OP_TYPES.

And I would bet that the next time such a case needs to be added, it would be forgotten again.

In `visit` we can just do a switch over all cases.

> I think what I have mentioned is a bug as well.

Maybe.


Hmm, I thought I'd fixed it. Fixed again, sorry.


what's the point of having the css files in the repo if you're calling the file from the CDN?

use the local minified file


unrelated white space change


Yes rawpt is super important :).


Forgive me if I'm saying something really stupid, as I'm not a Go programmer, but should these two variables be left here in `core/util`, as `oidExtensionRequest` is again declared in `ca/certificate-authority` in the latest commit?


Nit. Is this variable really necessary?


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 852](https://teamcity.labkey.org/viewLog.html?buildId=291861&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 24, passed: 223 Build time: 00:25:59
### 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
```


the srcfile seems somehow redundant to me.

In C++ this is a well known value.

In XML/Python this will be the same for all directives right?


This comment isn't true for our implementation.  In RE2, if it runs out of states, then it clears the cache and tries to keep going.  By default if it has to do that more than once, it falls back to NFA.  However, there is an option to say keep going anyway, which it can do so long as there is room for current and next states.  The equivalent RE2 test is basically testing that if you say keep going anyway, nothing crazy happens.  In your implementation, once it reaches its memory limit it falls back automatically to NFA.  So this just tests that fallback happens when it reaches the max states and no sooner and that results are correct.  Additionally, state limit is just an approximation of memory.  If you have a lot of alternation, states can become quite large despite there not being very many.  That means that we can't make as strong claims about handling oom conditions.


Please specify on all labels for which input they're bound to:

http://www.w3schools.com/tags/att_label_for.asp


Students/30425/Farcas S. Silviu Vlad/snake/src/Menu.java

this is the old version, I don't know why it was uploaded, just ignore from here on...


Evtl. ist erneutes klonen notwendig


Pls remove unused code


Broken Travis build: https://travis-ci.org/peacecorps/medlink/builds/96468512


I don't like that the result of this part of the test depends on the thing above having executed.  can you reset fallbackToNFAInvocations in between?


Also you have whitespace after your find command.  This looks fine with those changes, but there would need to be a lot of cleanup to make this work,


@feross FYI: You can also set the Opbeat credentials using environment variables if that's easier


I would tell Shakespeare he got it wrong, but I'm prepared to limit my linguistic pedantry to the living (who therefore have a probability of actually determining the language…).

Is there really a normative definition of en-??-x-hixie? I thought there was just an algorithm: "Ask Hixie what he thinks. If he says yes, you're good. If not, you're chaals" ;) (There is also globish, which I think has a normative definition, and is english…)

I only fell on this by accident. I do applaud the goal of de-gendering the spec, whether it ends up meeting my particular linguistic preferences or not. And at some point I'll probably do some work on that which is usefully collaborative... but not yet.:(


Excellent :+1:


We'll have to manually fix all the dependency issues :(


@feross This line needs to be at the very top of your index file - else Opbeat isn't able to hook into and monitor your application


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#f1o579oFSkaXBfV9cbLP4w


Is this being commented? could it be removed?


The difference is that we don't currently have any sort of "runtime usage" for regular libraries. So the dependency isn't any different, but the _resolution_ of these dependencies is very different.

@cbeams @bamboo  I think we might need to go back to the idea of splitting the stories for 1) "my tests compile with declared source-set dependencies" and 2) "my tests use the same dependencies at runtime". While this is less user-centric from a feature point of view, it makes a lot of sense from an implementation point of view. The first is unsponsored, simple and pretty much "done". The second is much more work, because it involves updating dependency management so that we can resolve a different set of artifacts when requesting the "runtime" of a library.

A very important benefit of this story split is that the second story is sponsored, and the first story is not.


Fair points. Any good idea on how to solve this? The default size is definitely to small on my linux box =/


Bootstrap has classes for error, use those.


Use a bootstrap class here. This will make it so that if we change themes, the colors will not be of :)


I think "out-dated" was not enough to describe such versions :flushed:


Good work and im happy to see u add fixes so fast, i dont know if u know that if you click my place ( load my rooms after created one ) it dont works, it give a error thanks c:


Since all of your cases call `system "clear"` before and `main_menu` after, you could put those outside of the case statement.


Wenn jemand z.B. auf dem Ozean GIB 0 ... SILBER macht, passiert das. Da kann man jetzt jedem Caller die Bürde auferlegen, das er auf r->land testet, ehe er rsetmoney aufruft, oder man stellt sich nicht so an, und wirft das Silber einfach weg (ohne assert). Letzteres scheint mir einfacher, und tut auch keinem weh. Ich habe das mal weniger strikt gemacht.
So ein assert ist eine ziemlich harte Aussage, wenn man mit asserts in production arbeitet. Das sagt "wenn dieser Fall eintritt, ist die beste User Experience, das wir abstürzen und nichts machen, weil die beste Alternative Datenverlust oder -korruption ist". Das ist hier aber nicht der Fall.


> wasn't done that way originally, but i can bring it up w/ josh...

What do you have to bring up with Josh? I don't think this affects anything from the product side - does it?


now you're calling both files! wtf


I think this if-block needs to go above the "removeInteraction" because I'm getting undefined errors:

```
TypeError: Cannot set property 'isInTeam' of undefined
    at ctrl.removeUserFromTeam (ideasView.js:490)
    at $mdDialog.show.then.$scope.status (ideasView.js:620)
    at processQueue (angular.js:14745)
    at angular.js:14761
    at Scope.$eval (angular.js:15989)
    at Scope.$digest (angular.js:15800)
    at angular.js:16028
    at completeOutstandingRequest (angular.js:5507)
    at angular.js:5784
```

And I still see this:

![image](https://cloud.githubusercontent.com/assets/5081281/11845924/33c25748-a3e4-11e5-8a18-2b3f11dda855.png)


Is the NOT_FOUND case handled in the spi layer? We usually return `null` for not found in this case.


Additionally can you also add

($eyeCatch ne $g_TOCEyeCatchBackup ) &&

To the following block line 463

#don't try to add the TOC, but need to update all other paritions
#Add if side matches (or not set) -- so if it isn't equal to other side
#Also add if sideless
if( ($eyeCatch ne $g_TOCEyeCatch ) &&
    ($eyeCatch ne $g_TOCEyeCatchBackup ) &&
    ($sideInfo ne $other_side ))


use let instead of var


/br


@feross I just looked at the start script in package.json and found that the index file isn't your main app file - seems like run.js is the main one? In that case I'd suggest adding the opbeat start call there


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


Yes, it's not necessary anymore. Give 1 minute and you won't see any comment.


This might cause trouble on mobile! In fact, I know it will.


It's very strange to have this be an `Object`

This should either have overloads to accept different input types (one of which delegates to the other) or just make it accept a Reader.


<nit> Are all of these usings necessary?  Consider 'Remove Unnecessary Usings'


Pls change void to:
1- bool
2- ProductDto include Id which is just created


Nope, sorry! I was wrong. I think you just have to pass the `$scope.userBack` into `ctrl.removeUserFromTeam()`


this should not be needed, you are using "pool-fair" as attribute name also for xa


Ce commit n'est pas nécessaire, ce sera automatique en version 1.7.1.


What will happen to punctuation? GT allows for suggestions on it.


Consider throwing a more specific exception such as an ArgumentException.


The same here, pls change to bool ( you can change to ProductDto)


May not be relevant now but we need to transfer context with suggestions.


Missing `serialVersionUID`


Pls change to bool


remove comment


Resolved merge conflict


Consider making the setters on these properties private.  It seems like this should be an immutable type.


same as above


Can remove spellcheck attrib, it will have no meaning.


spacing fix this so it all lines up nice.


Chang to booll or UserDto


May be able to use `modal-sm`.


:+1: :+1:


Pls remove unused code


Is this the preferred wrapping?


This file is not needed, just wildfly-*.xsd for newer versions


"Improve Apertium's translation"


Consider making the setter private here and on VersionDependencyInfoMap


Oh, it has overloads already, I read right past them.  Just make this accept a reader and move the logic into the overloads.


mas os ids dos items novos foram mudados ou são os mesmos? eu adicionei uns nomes no items.xml
dai ter que trocar seria bem chato, também peguei umas partes de mapas também


What happened to this indentation? (also in the following)


This makes the not-now / never buttons look like a gray link, more subtle.


"," is too much


:D

Nice catch, thanks

I-it's not like I didn'n see it or a-anything! I just wanted to check who'd notice it!


Thanks!


改行


Why have you addedd as comment? Is it a TODO? You should add the new parameter to tests


You're right. I will fix that.


same as above


can you add tests, that ensure that the length of the list, is as expected? also for IterableDict.

also some whitebox tests for the expected keys values in `td` would be nice.


I didn't realize this, but now we're duplicating code between `multisite_drupal_toolbox_form_contact_site_form_alter()` and `multisite_drupal_toolbox_form_contact_personal_form_alter()`. Maybe this is also solvable with a `hook_form_BASE_FORM_ID_alter()`, or we can factor out the code in a helper function that can be called by both form alters.

On the other hand it's probably OK to just keep it like this. It's maybe not worth the trouble.


What if they're suggesting that the word doesn't belong at all, i.e. it's extraneous to the translation? This might be a question for the linguists.


You don't need to set all of those properties on the button yourself -- that's what the jquery-ui `button` function itself does. I also refactored things to use jquery-style method chaining.


Awesome :)


Static import all of these


Good catch!


Hi,

I'm not able to reproduce this error,
when are you receiving the error? what Chrome version are you using?

:hand:
PD. They have changed the css again...
![](https://s3.amazonaws.com/colorslive/jpg_512x512/516016-Djw11UKPBuU65Kno.jpg)


why is css used in web api?
What for purpose?


It looks Ok


Looks much better so far but it may be worth re-organising the specs? You have one context for dealing with a black listed article and non black listed article. However the bottom group, at first glance, I wouldn't know what it is being black listed. Articles? Categories?


While you're at it: Can you change this to `1.2`?  The example above is only available since `1.1`.


This just makes them have to manually close the modal and is kind of obtrusive. Make it so the modal only stays open if there is an error and the error message can show up underneath the text input itself (see Bootstrap docs) so we don't need extra `div`s.


コメントアウトしてる分は消しても良いのでは？


Yes, will rework the Linux build script to download and build it as well, instead of relying on system packages, after this the version should be trivial to bump.


Nit. Is this variable really necessary?


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - x86_64-MacOS-10.9-clang: OK (1983 of 1983 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)


Looks good!


Then we don't need area any more. Are you happy to remove area in the faces ?


Well, I totally screwed up this copy-paste. Not sure whether it's worth two hours of my time to re-release just to fix the mistake though.


Ederson's warning is relevant. +1 for the PR anyway.


I approve of this message.


Done


Checking for directive indentation isn't in my scope. It is enough for me that code which indents directives does not cause a problem to cppcheck, and indeed it does not. I just mentioned GLIBC's indenting by the way, as the thought crossed my mind.

Re: comments -- do you agree that their removal be done in the Python addon rather than in the XML dump?


General comment:
- transformers for resource-adapter, data-source and xa-data-source are missing. You should add transformers discarding in case of old default and rejecting in all other cases.
- Test cases stressing transformers described in previous point are needed too

regards
S.


I don't understand what are you talking about.
Initialy there is no space before `99 + "% " + msg;`
`99% Message`

After first if, one space added:
` 99% Message`


`count` returns 1 if it is invoked with no arguments (i.e. count is 0). That means `if count $chars >/dev/null` works.

Though it'd be clearer to use `test -n "$chars"`.


Doesn't `https://www.googleapis.com/auth/cloud-platform` already contain `https://www.googleapis.com/auth/cloud-platform.read-only`?


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


So nice to hear you want to help @smountjoy,

About the chroot update:

In this case, is not the usual commit adding a little thing or a fix for something.
(We added a new whole option to the menu, and a new package to the chroot and the csv2sqlite script as we talked).

As you said:
> I had updated chroot, using the menu option to do so, which starts a terminal and runs start-update.

This is really the same that:

Opening kali shell in terminal and doing `apt-get update && apt-get upgrade`

    This only updates the installed packages not the chroot really.

So when i said update the chroot i wanted to say:
 >  Remove and install the chroot again from chroot manager. This is needed because the check-sums have changed. (this is needed every time we modify the chroot (Is a security measure).

----------

If you are going to develop Kali-Nethunter, my recommendation is:

Clone the repository *(newinstaller-fj branch)* and build the chroot, (you need to change the check-sums with your own in the app).
Once you have the chroot created, and the app updated with the check-sums of your chroot, you can move the zip to `/sdcard` and install from there.

This will speed up the development since there's no download needed. (In the chroot install dialog, select `install from sdcard` instead of downloading).

(Ill try to do a little write up about "Developing Kali-Nethunter")

Feel free to ask anything you need.

Jaime

----------
P.S.:
The **exploit-db package is not a dependency of kali-nethunter package** at the moment. Perhaps (@muts) maybe we should add it.


OK, now I understand the purpose too. It was a creative way to affect all node edit forms. Excellent job!

We have seen a whole bunch of different ways to deal with this use case in this issue :)


This particular comparison cannot work with the suffixing test from line 402, when taking into account Swift's String logic. Foundation's String comparison is done on decomposed forms, but Swift's should not. See https://bugs.swift.org/browse/SR-243


Right. There is an [issue](https://github.com/saltstack/salt/issues/28310) related to multiple values for `vgname`.


Seeing the usage of this method I find the name misleading.  This is not about initializing the `StyleEngine` but about binding it to a a QML engine.  So maybe `setQmlEngine` just is what it should be named?


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


You could just:
return $this->sendRequest('POST', '/batch'.$this->eventUrl, json_encode($data));


LGTM


Or have both overloads translate to the same class (File or path perhaps) and then open the reader internally.


I don't understand this change in particular. Also, please provide a better commit summary and description as well as a Signed-off-by: line.

Are upstream linux changes required to make this code work? Also, this is a big change w/o lots of orthogonal changes which should be separated into separate commits.


```java
future.thenRun((none) -> stateMachine.transitionToFinished())
        .exceptionally((throwable) -> fail(throwable));


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


I think this is an exception that someone might someday want to catch explicitly, so could you add a new exception class `RuntimeScriptException` for this to throw?


Course.pluck(:title, :title)
為什麼重複?


It could be factorized as an attribute in the <dump> root element (possibly even renaming that element <file>), e.g. <file name="whatever.c"><dump ...></dump>...</file>. This way, the XML content is self-sufficient and does not depend on the XML file name.


Use `span` (since GT does).


This I find a very misleading or confusing name.  Why is the style engine not in a valid state?  It seems what you encode here is "isFullyLoaded", right?


I think we need to add to comments what size is doing. How about following.

- Make size as uint64 instead of int64
- If size is 0, that means graph drivers can create fs/device of any size.
- If size is non-zero, then graph driver needs to make sure fs/device is of specificed
   size otherwise return an error to top layer.


Meh this is an ugly and brittle way to do `$node = menu_get_object()`, but that's out of scope for this issue. Carry on!


same here fro the `RuntimeScriptException`


komentarz 1


komentarz 2
bla bla


`RuntimeScriptException` here


some code review – I see you're using `var`s in some places. Can you replace them with `val`?


What about adding an utility method to `LocalResourceManagerHelper ` that does this?


indentation issue


Po co?


komentarz lini 2


Whoops commited to the wrong repo


komentarz lini 3


Why do we need this? Isn't `assertEquals` enough?


komentarz lini 4


komentarz lini 5


:thumbsup:


komentarz lini 6


komentarz lini 7


That's kind of what I've expected, as this sadly is version-bound code.
I don't know whether this will ever land. Sorry for this.


komentarz lini 8


komentarz lini 9


komentarz lini 10


podkomentarz


Could you please backport this to 4.0.x?


Would it make sense to commit the dummy app and set this an an initialiser on the dummy?


Great! I'll update the other repos. (Don't forget to set up the relevant ENV vars locally)


Why do tasks need the transaction manager? It appears not to be used and it doesn't look like there are any methods that a task should call.


ewrwer


asdwefwe


Remove all these redundant `else`


qewqewqdef


Isn't this already tested in `compareReadWriteFields`?


Static import


Whoops, worded it incorrectly. It's not all GET requests resulting in error 400, it's all non-tournament requests resulting in error 400.

Anyways, this is a critical fix. We should probably make a hotfix release for the API.


At your will: 5a6e2bbd01ea890cbebcf293c365e861c591f110.

This was from https://github.com/Netatalk/Netatalk/pull/11.


This should work. Intuitively, I feel like there should be a more elegant solution, but I certainly don't have a suggestion as of right now.


空白


Just wondering: is this the preferred wrapping?


nvm, you already did :-)


Awkward phrasing :\ Let's ask in IRC for some better phrasing.


第一個值是顯示用，第二個值是 Value，剛好都是 :title


nice


vfs and all the drivers need to check for size and if size is  non-zero, then return an error saying `Graphdriver foo does not support specifying size.`


doneです！


It should be removed from the loop and maybe added as real interupt driven function. Everything else is nonsense.

People who use the old API just need to call the function inside loop() and you are done. Place this note in the docs for newer IDEs and you are done. That doesnt break much,  also I dont think many people will use it.


Wouldn't it be preferable to display the exception message in the console output?


I don't think "Legacy" is a good name. Many connectors won't support transactions, so we won't ever remove the old interface, right? Maybe "simple" or "autocommit" as a better name.


Awesome! You're changing the word in the output. I like it.


nie zgadza


Typo. The online markdown editor doesn't let you diff before committing AFAICT.


Ive always been told not to commit generated files.

And if someone tries to regenerate the test that could blow away the initializer. jhawthors was :+1: for doing this in the specs


Why drop support for node 0.10.x?


yepp.


I kept both constructors (File and String) this way because the latest JS engine (nashorn in jdk8) can use the java.io.File information to resolve a relative path ( https://wiki.openjdk.java.net/display/Nashorn/Nashorn+extensions )

> Current executing script file, line, directory of the script can be accessed by these global (read-only) properties. This is useful in debug/tracing scripts. __DIR__ can be used like Unix 'dirname' command. Script can load script-file relative path resources using __DIR__.

If we use a java.io.Reader we will lose some information.

What do you think ?


Tôi nghĩ rằng bạn cần phải đổi tên một thư mục trên mobile_default mobile_nukeviet


Some notes:
1- Don't commit bin,obj folder. It's very difficult for review.
2- Please remove source code/file unused.
3- For Insert/Update/Delete, please return bool/Dto object so that we can know that our action is true/false.


It would be super cool here to display the context as well since we need it later with the word itself bolded. This is an extra though so you don't have to worry about it. It's here just so that I can remember for the future in case it doesn't happen now.


reverted later to keep same groups functionality


Why is this needed? It appears not to be used. If there is some reason in the execution engine that we need to do it in order not to block the thread, all the existing ones should be updated as well, since some can take a long time.


Addressed.


done です！


This looks weird, it doesn't feel right to use self.frame, as maybe we want to add insets for the selected / background views in the future. Also it's not clear why you do self.maximumValue +1.
Can you open an issue with description of the bugs, steps to reproduce and example if possible? Thanks


Let's keep existing values. You can expose touchEdgeInsets property in the public API, initialise to UIEdgeInsetsMake(-10., -10., -10., -10.) in setUpDefaultValues and then you can customise from outside.


This breaks example app, please modify it too or make it optional.


ハッシュスペース


It already has been. You can npm update.


Move `RBX_VERSION=2.5.2` down to line 14...


It doesn't look like this header has the standard header guard `#ifdef`.  It should have something like:
```C
#ifndef __DS3_UTIL__
#define __DS3_UTIL__
```


This will cause communication error cases (either if the node app fails to reach Catalyze's server, or if Catalyze's server goes down) to count as success - this will just pass the null reference one step down the line (since the response would not be in the form the client code would be expecting in the success case).


"culprit" :( Is this a standard word used in this context? If not, it would be awesome if we could switch it out for something. I know this is really nit-picky but the word doesn't have the greatest of connotations and feels out of place.


... and then update this line to read:
`echo "export RBX_VERSION=${RBX_VERSION}" > $IROOT/rbx-$RBX_VERSION.installed`


Can you revert changes in file permissions?


Shishir, have you taken care of all the comments given in last iteration? Last time dan mentioned to allow for syntax --storage-opt size:2g.


Ändere das im nächsten Commit. Das war mir schon bewusst, wusste aber nicht, wie weit Lasse mit dem Erzeugen der id ist.


Thank you very much for the contribution, having highlighted handlers is a good one :)


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

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


Additionally, this is in the `dist/` folder - the change should actually be made in the source file: https://github.com/catalyzeio/catalyze-js-sdk/blob/master/src/node-catalyze.js#L33

`npm install` after that should rebuild the dist files.


ハッシュ


Have begun writing documentation for parameterObject.R.
In initializeParameterObject, I raise the question of whether or not we should do more error checking on the values given. Also, Cedric has added a variable to ROC for priors, which he will need to document, or tell me how it ties in with the model. I’m also changing the style of the brackets since some R loop structures require the bracket on the same line we should just make a brackets in R match this formatting.

 Also, in the loop which either creates a new parameter based off the restart files or calls the model’s specific initialize function FONSE did not have an else which would read a restart file. I have now added this.


It breaks the python 2 compatibility, I'm on it.


Above seems to be white space only - please avoid whitespace changes it makes it harder to review


@ojlyytinen @mjgiarlo I see some changes in a generator here but I don't see anything in the release notes. Was there an upgrade path-related oversight??


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


Was


This fixes #97


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


Not sure why you're comma-separating the oc_component variable -- you can just use spaces and then get rid of the echo/sed step here.


@ids1024 : does this look OK to you? https://github.com/pazz/alot/tree/0.3.6-refactor-main


same here


No, more interrupt context callbacks are not a good direction for Arduino.  Even experts have a hard time correctly dealing with interrupt context when it comes to data sharing.  Arduino is filled with APIs, like String, which aren't interrupt safe.

More responsive event callbacks can and should be implemented by checking inside yield(), rather than after loop() completes.


Might be better to load this from a CDN, if available. I don't want to unneccesarily hit Github for things like this :wink:


If you like Linq, you could avoid the nested loop by utilizing SelectMany.
foreach (var dependencyInfo in dependencyGroup.Values.SelectMany(dependencies => dependencies))

If you use the Aggregate method you could do something like the following.
string projectsMessage = dependencyGroup.VersionDependencyInfoMap.Values
    .SelectMany(dependencies => dependencies)
    .Aggregate((message, dependencyInfo) => $"Version: {dependencyInfo.Version} Path: {dependencyInfo.ProjectPath} {Environment.NewLine}");
return $"Conflict for {dependencyGroup.DependencyName} in projects:{Environment.NewLine}{projectsMessage}{Environment.NewLine}";

Lastly it might be nice to order by the version.
string projectsMessage = dependencyGroup.VersionDependencyInfoMap.Values
    .SelectMany(dependencies => dependencies)
    .OrderBy(dependencyInfo => dependencyInfo.Version)
    .Aggregate((message, dependencyInfo) => $"Version: {dependencyInfo.Version} Path: {dependencyInfo.ProjectPath} {Environment.NewLine}");
return $"Conflict for {dependencyGroup.DependencyName} in projects:{Environment.NewLine}{projectsMessage}{Environment.NewLine}";


Never hardcode a URL. the PhenoTIps URLs are configurable (e.g. 'bin' could be absent, the database name could be present in multi-instance environments, etc. Always use the API to generate the URL. In JavaScript, that is
```new XWiki.Document("PhenoTips.ChartService").getURL('get', 'querystring')```


finally!


@rlaffers Hello. Create please an issue for this discussion


Beta to Retired........
+ "This version" ????
+ Where is Version 2?


what about using the `_count` reduce function? this way we don't have to load the entire list in memory and return a much larger payload. also i think we can be clever about caching here. essentially i think we can use quickcache to only cache the result if the count is greater than 0. though i may be overthinking this since users are in their own DB now so it may not even matter. @dannyroberts do you have any opinions? just worried that there'll be a fat load every time you open the deploy modal


another hardcoded URL


I'm confused, don't you have a code like this on 'sol-platform-linux-common.c'?


(the quickcache idea might not work)


哦！！我学到了。学敏牛！


Question -- why do you have both oc_info and oc_info_file ?  Looks like this would query the same information twice, wouldn't it?


Thanks for pointing this out. I am aware of the `private` keyword (and use it in [other libraries](https://github.com/inhabitedtype/ocaml-session/blob/master/cohttp/session_cohttp.mli#L62-L76)) but it never occurred to me to use it here. The one issue with using it here is that there's an intermediate record between the user and the diagram node that they would have to manually traverse. As long as that's the case I think it's best to leave it as is.


この改行isなに？


def get_people(lines)
  lines.map { |l| l.match(/\A(\w+)/)[0] }.uniq
end


哦！！学到了！牛鼻！


It is with files created outside of cygwin.  I do not have a solution for it, other than remembering to correct them, or create dummy file inside cygwin to later edit.


Is that last line not to much pasted?


lines = File.readlines(ARGV[0]).map(&:strip)


Why don't you try the max retries option to terraform to see if that works better than a loop?

 https://www.terraform.io/docs/providers/aws/


`data_` is *not* uninitialized in any of these constructors. It is default-initialized.


Maybe add a `todo` so that this can be easily tracked


this is toString not identity ;)


xD


these functions must be available on the server as well


I removed all calls to `numpy.random.randn` in that method, so I figured it was not needed any more.


I like the new message format.  Just had a few minor comments for you to consider.  LGTM


This strips out the retries whish account for network failures and similar. Was this intentional? if so, why?


@JawhnL5 john works for stride confirmed


That's the change I made for the password--I had to increase the `AMT_TIMEOUT`, too (to `10`).


I do this a bit differently: I install packages I want to tinker on with `cd path/to/alot && pip install -e .` and can than just run `alot`.


read writes over 'n' bytes so this isn't necessary


Closing. We already have apt-get update.


Above two lines are duplicates of https://github.com/anhowe/scratch/blob/master/mesos-marathon/parts/configure-mesos-cluster.sh#L232-L240

Why are they added here?

Why is the optional OMS registry now a default?


did you consider using in-memory streams instead of tempfiles (like php://memory )?


Looks good!


I'm not sure what you mean. Do you mean that it would be better to replace lines 121-123 with:

```python
prefix = None
if is_input_port:
    prefix = 'input_'
else:
    prefix = 'output_'
```

If so, do you think it's more readable? I got used to using the style without `else` - it's only advantage over the one above is that it is 2 lines shorter.


Could you do this same ordering by changing https://github.com/dimagi/commcare-hq/commit/21fda3df9dddd513d20858a5987d03c58539c90b#diff-007cbd344b264b88f44398660b42183aR142 into an `order_by('sql_product__name')`


probably want 'narrowed'


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


Oops!


Nope, it's the other way round!


Will do! Is this the sort of thing that can't be discussed on a mailing list?


Java is not needed on our agents.  We have provided an (undocumented and potentially unsupported) postinstall script hook for doing this kind of work. See https://github.com/anhowe/scratch/blob/master/mesos-marathon/parts/configure-mesos-cluster.sh#L232-L240


Drupal coding standards: Control structures should have opening parenthesis on same line. https://www.drupal.org/coding-standards#controlstruct


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


maybe 'until it will expire' instead of 'that it will expire'


probably should check if n > 0 before doing anything, also printf the 'n' value


Nice. Good old svn :)


Ha... I see the issue... but I don't like the fix. We just hid the details of sha2/3 in search parameters, but need to keep them in mismatch parameters, that ought to confuse users.
This is going to be trickier than I thought, probably with some custom logic in `makeChecks` to determine which mismatch needs to be applied based on the content of the search.


You can access this element from the event handler by
```
someHandler: function(event, instance) {
  var commentDiv = instance.$('.discussion-comment');
  ...
}
```

Please also convert the classnames used for programming to camelCase and have them start with a hyphen. that way we can distinguish them from the styling classes. In this example the class could be called '-discussionComment'.


indent


TeamCity pull requests :: netty [Build 892](https://garage.netty.io/teamcity/viewLog.html?buildId=4135&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:44


We should probably add a test that unsets some property.


This section seems to duplicate https://github.com/anhowe/scratch/blob/master/mesos-marathon/parts/configure-mesos-cluster.sh#L395-L401


Did you test that one? Seems a bit dangerous to me to reassign a `const` within a loop.


indent the brases


There is no need to check ```hm``` since ```tfw_http_conn_msg_free()``` already does it.


Изегай не латинских букв - даже в автосгенерированных строках


Why do you need this include?


IMHO this should've been done in another commit / branch. It has nothing to do with sass-lint.


One time too much.


Would you mind naming the `err` business? It took me some time to get what was happening and what the signature of createCache's next callback was. Something like:

```js
const isCacheHit = !err

var reader = isCacheHit
  ? fs.createReadStream(path)
  : undefined

var writer = !isCacheHit
  ? fs.createWriteStream(path)
  : undefined

next(isCacheHit, reader, writer)
```


That is true. The main reason that I'm doing retries is because of a bug in the AWS provider. I'll take a look at the max retries option, that might make things simpler.


fasfa


**Line is longer than allowed. (80 > 79)**
*LineLengthBear, severity 1, section `pythoncheck`.* *Related to file `/tmp/tmpupr80b3e/coalib/bearlib/languages/documentation/DocumentationExtraction.py`, line 121 column 80 to line 121 column 81*.


**Line is longer than allowed. (80 > 79)**
*LineLengthBear, severity 1, section `pythoncheck`.* *Related to file `/tmp/tmpupr80b3e/coalib/bearlib/languages/documentation/DocumentationExtraction.py`, line 180 column 80 to line 180 column 81*.


alright just saw the exists here, makes more sense as a name instead of isCacheHit. Still comforting that I understood the code above :smile:


whyy


这个有时候要注意numerical stability，最好adaptive normalize

On Tuesday, December 15, 2015, Xuemin Zhao <notifications@github.com> wrote:

> 上面那个页面第三个公式就是上面贴的公式哈
> 当然Var(x) = E[(X-μ)*(X-μ)] 肯定是对的，E(X^2) - (E(X))^2可以少个循环。。。
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wangkuiyi/phoenix/commit/cf266efe80f0c3406c7c09af6630b0edb9da180c#commitcomment-15003260>
> .
>


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: 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)


ddd


What is this line for?


Подберай плиз более осознанные названия для grop и artefactId

В твоем случае подошло бы edu.nc.saydumarov и music-events-searcher соответственно


This changes appears to kill the second-order convergence of uv in `swe2d/test_steady_state_basin_mms.py`.  I get about 1.5th order instead.


Yes, as you noticed the current stats counting and algorithm selection is a simplification of what is in RE2. Based on your comment, what is your suggestion then or maybe ideas for future improvements? We could certainly improve memory accounting. I wonder if simple fallback to NFA is enough. This is because even if we continue with DFA we can expect that it will run out of states again (since we process millions of similar rows). Therefore in my opinion we could directly switch to NFA or fail (because we don't want Java to run out of memory).


The joke was in the misspelling of regrets :P


Yeah, looks good and ```tfw_http_conn_msg_dropfree()``` is coherent with design of ```tfw_http_conn_msg_free()```.


Из названия класса непонятно что он делает


Typo: sensitive


@thomasvnoort I added these constraints to restrict types a bit. For example: we don't want to be able to use  `String` or `PGInt8` with  `.<=?`


LGTM! Maybe I would move the `cacheKey` function to the `cache.js` file. Seems like it belongs there.

Otherwise, besides that variable's name in the createCache function, looks fine.


This will only work if the screen ratio is 16:9, why not using the actual screen resolution instead?
I probably missed something but what was wrong with the original implementation?


Best use a ReactiveVar on the template instance for this. This requires every comment to have its own template instance though.


This docstring doesn't match the actual function args


sure


For what we know here we might be retrying. We should at least check that `retryable()` is false or mock everything as in the previous test.


Yes, this results in cannot represent a stream of type MEMORY as a File Descriptor.


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


Yeah, kind of. I don't like using `not`s if this is not necessary. I guess we could also make it shorter and define it as:
```
prefix = 'output_'
if is_input_port:
    prefix = 'input_'
```
I am pointing that out because I've found several other places with `not`s that are not that smooth to read.


keytool -exportcert -list -v \
-alias <your-key-name> -keystore <path-to-production-keystore>


@mariopeixoto Any plans to create an actual 0.13.3.2 release/tag? I'd like to integrate the deprecation warning fix, but referencing a non-released commit in our Gemfile is a tough sell where I'm at. Thanks!


This was checked in perforce but not in to github


alright just saw you moved it later! ignore me


Not quite the same, convert name to a bool (!!), if it's true, use name as a string, if false, set name to false (in which case the attribute will be ommitted by react)


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


that is an ex coming from http://doc.akka.io/docs/akka/2.0/intro/getting-started-first-scala.html,
2 vars are used, one to calculate the value of pi, the other the nb of iteration, recursive functions could be used to avoid that most likely.


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


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


```tfw_http_conn_msg_dropfree()``` is used in ```tfw_http_conn_cli_dropfree()``` only, so do we need to separate the functions? Isn't it better to merge the functions?


Same as above. When every comment has its own instance, no element ID should be required to access the element.


Описание сервлета здесь не нужно так как мы не используем gwt-rpc


missing a trailing dot.


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


The formatter gave me some interesting results here, will change manually here and below


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - 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)


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


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


Cut and paste errors fixed thanks.


OK, I'll update and rebase tomorrow, it's real late here


<5.14, not >5.14. Sigh...


Looks better when passing an options object. Gives more liberty on injecting other things like cookies.


Provide an adjustable font size when compiling or runtime?


static -> immutable?
machine -> container.


Missing reject when `res.on('error', reject)` and in https.get(...).on('error' reject)


Missing a catch in case something fails to return the failure in the request


ssssssh


Can be simplified to `export const ENABLE_HOT_LOADING   = argv['hot-loader']`, treated as boolean.


You probbaly could detail what mounted means. Something like:

> to be mounted in the Docker container. In other words, the directory containing the GitHub repo is mapped from my Windows directory to the docker container.


Do you mean the 1.3333 variable? That's a good point, you are correct, it will only work for 16:9. Using the original screen resolution only works when the user chooses the native resolution, it doesn't work when the user chooses a lower resolution.


`pwd` will work if you run from `in.relation.to/ `right?


I suggest you mention `really.long.lvalue[expression]` instead of just `reallyLongVariable`.


Indeed...


My formatter gave me this because the two parameters together are > 100 chars, but I can manually wrap it differently.


This is some kind of convention used in GraphViz's dot format. See example starting with words "As an example of HTML labels, the dot input" in [documentation for node shapes](http://www.graphviz.org/doc/info/shapes.html).

A tidbit is that the format used here to describe a node is not real HTML but something HTML-like.


https://stackoverflow.com/questions/5828037/cygwin-sets-file-permission-to-000
https://superuser.com/questions/680729/default-permission-for-new-file-cygwin-on-windows-7

Does any of these help?


Fixed.


As in previous commit, catch errors on http.get(...).on('error', ...) and res.on('error', ..


> what about ifdefs of machine dependency ? is there any other way to do the job ?

Do you mean the ifdef for SECTION_ATTRS? That is due to incompatibilities in the assembler between ELF and PE formats [1]. We need it because gas aligns new PE sections by default to 4 which causes 0 to be inserted after each block.

We should be able to use .p2align 2, 0xa0 to avoid the ifdefs and cause the assembler to fill in with newlines instead of zeros but this defines on the assumption above (that gas aligns by default to 4).

[1] https://sourceware.org/binutils/docs/as/Section.html


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

```


This reduce could just be `join('')`?


unack, waitwaitwait, we had a *long* discussion about you moving this out, it feels a bit like you're ignoring me!


Why would you set a hard path to the index.php file?

Just use
```
<a href="<?php echo Skin::makeInternalOrExternalUrl( wfMessage( 'helppage' )->inContentLanguage()->text() )?>">
```
and everything forming the link string  for the href happens automagically based on the MediaWiki:Helppage contents.


doh! :( fixed!


Catch errors here and respond!


OK, kinda tricky with style and className props as they mutate later, will end up with something like this:

```
let { className, style, ...constants } = this.props;
const { other, listed, props, ...props } = constants;
```


Nice commit :+1: :whale:


@vincentaudebert Trying to keep this link editor-agnostic because we don't all use Sublime. I added this plugin to our dotfiles repo instead: https://github.com/springload/dotfiles/commit/f2235db654202fc9f3ee549699d248533ddb844d.


I saw that's what the update script does - there are times after a reboot where Nethunter will open the terminal but not run the commands (same with kalimenu), so I'd run it by hand. When you said to update chroot, I'd thought you meant you were adding it to the kali-linux-nethunter metapackage but my device hadn't pulled it for some reason.

I've already got my own fork of newinstaller-fj, so I can install my own kernel (did I mention I'm running MMB29K with success?), but I'd thought the chroot was built in the master branch - now I'm looking at build.sh and seeing that I have more tracing to do. Thanks for letting me help, hopefully I'll have something other than a kernel config.
Is there an email address or something other than the github relay that I should use for replies or questions?

    On Wednesday, December 16, 2015 8:14 AM, 3boll <notifications@github.com> wrote:

 So nice to hear you want to help @smountjoy, About the chroot update:In this case, is not the usual commit adding a little thing or a fix for something.
(We added a new whole option to the menu, and a new package to the chroot and the csv2sqlite script as we talked).As you said:
I had updated chroot, using the menu option to do so, which starts a terminal and runs start-update.
This is really the same that: Opening kali shell in terminal and doing apt-get update && apt-get upgradeThis only updates the installed packages not the chroot really.
So when i said update the chroot i wanted to say:
Remove and install the chroot again from chroot manager. This is needed because the check-sums have changed. (this is needed every time we modify the chroot (Is a security measure).
If you are going to develop Kali-Nethunter, my recommendation is:Clone the repository (newinstaller-fj branch) and build the chroot, (you need to change the check-sums with your own in the app).
Once you have the chroot created, and the app updated with the check-sums of your chroot, you can move the zip to /sdcard and install from there. This will speed up the development since there's no download needed. (In the chroot install dialog, select install from sdcard instead of downloading). (Ill try to do a little write up about "Developing Kali-Nethunter")Feel free to ask anything you need.Jaime P.S.:
The exploit-db package is not a dependency of kali-nethunter package at the moment. Perhaps (@muts) maybe we should add it.—
Reply to this email directly or view it on GitHub.


I'd add,

> for more info you can check this docker-machine open issue https://github.com/docker/machine/pull/2100#issuecomment-159231155


I'll stop bothering you now, but catch errors here xD


#4407-Zone-Node-Name-not-unique-in-AirLoopHVACReturnPlenum-object---need-to-issue-error (mjwitte) - i386-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)


I'd leave it as a normal sentence, it feel weird to end the blog / discussion with such an red icon.


There's a generalization here that we could capture—if we've got a `DecodeRequest[A]` instance for any `A`, then it seems like we should be able to get `DecoderRequest` instances for `Option[A]`, `Seq[A]`, etc.—without having to hardcode these for each "atomic" type.


TODO: we should add that for readme too.


Good point. I fixed that and the other one.


That's weird I don't recall having issues with this, let me check again!


That's dead code -- removed!


Oh good! I didn't think of using choice_with_entry here but that was the obvious thing to do.


@darokel another lookie perhaps?


@hbetts completely agree... I've been swampted in my office :S Yeah, no excuses, I know :+1:

Thanks a lot! Let me send an update...


For ile gezip her P[i] için `malloc(sz)` yapman lazım.


Good idea -- done!


I think @aozarov suggested removing all but the absolutely necessary utility methods from `LocalResourceManagerHelper` in my previous PR.  Did I quote your opinion correctly @aozarov?


I'm pretty jet-lagged so maybe I'm missing something, but I don't understand the motivation for this method.


Thanks! :bow:


Birer double değil yukarıda hesapladığın `sz` bayt kadar yere ihtiyacın var.


Is this always the case?
```java
resourceManager.create(ProjectInfo.builder(myProjectId).name("CoolProject").build());
// Does this unset the name?
resourceManager.replace(ProjectInfo.builder(myProjectId).build());
// Or must I do
resourceManager.replace(ProjectInfo.builder(myProjectId).name(null).build());
```


Ok, I'll give you an example of what would happen with the old implementation:

If you chose "720" as your selected resolution, we would render in the video plugin at 960 x 720, which then gets stretched by android to 1920 x 1080.

With the way I have it now, if you choose "720" as your selected resolution, we would render in the video plugin at 1280 x 720 and then stretch in android to 1920 x 1080.

It's a minor visual quality boost when choosing lower resolutions.


@terinjokes what do you mean by `logLevel`? some consoles support it, but i think it is not part of the `console` API


ok


Okay, here's a more detailed explaination of the changes. Right now I'll touch on the evaluation changes, I can't recall what the horde error was, I'll find out later when I get home. Moreover it was some sort of assertion error(s).

The evaluation additions/changes in detail
In position.cpp a separate list of piece values were defined. Basically, this change is responsible for a lot of horde definitions everywhere where piece values are considered in the position. Along with piece values, exchange values were added for rook and minor pieces.

In addition to the piece values, In psqt.cpp for each piece, square values were added.

The rest of the individual changes mainly exist within evaluation.cpp

In evaluate_initiative(...) I added a block of code to give a higher score when black's pieces are penetrating through the horde. Hence why I put it in evaluate_initiative(...).

In evaluate_pieces()
The rook was changed to get a bonus from Rank4 and greater (before it was Rank5) usually good in horde to blockade pawns and prevent g and b pawns from advancing. Also, the rook gets a bonus for being the back rank when most of blacks pawns are gone. This was aimed to help control the horde and if possible allow the black queen cleanup the horde while the king and rook defend the back ranks.

The king gets a bonus for being the 2nd rank. This is in part with the rook to help defend the back rank. Also the king is more useful often in the "opening" and middle games when it can act as a "pawn" per se, and get out of the way.

Lastly in pawns.cpp
I removed most of the evaluation of the black pawns. For the most pawn in horde, black doesn't care too much about his pawn structure as long as black has space and therefore isn't cramped. As for the white pawns I added lines to hurt its score when it's structure gets too unbalanced with the number of pawns in each file. The pawns maintain most of the original score evaluations as regular chess.

More later...
I have to go for now. Thanks.


$('#previewImg') wont work for each() function


Hayır. Zaten çok büyük diziler için stack'ten yer alamazsın bu şekilde. Burada istenen şey, A, B, C ve D göstericilerini, X'in içerisinde ilgili yerlere atayarak X matrisini 4 bölüme bölmek. Sen yeni diziler oluşturup onları tekrar doldurmuşsun.


:(
Bon bah je m'y colle


Yukarıdaki yorum geçerli.


Credit to Ryan Maverick, cheers for the help in getting it onto a device!


AFAIK you can write it like:

```
const { other, listed, props, ...props } = this.props;
let { className, style } = this.props;
```


good catch, thanks


remove exclamation


From my perspective, using libucl from ports/packages (that this pull request enables) is more desirable than vendorizing libucl, and building it directly.  I am running a similar (but inferior) patch on my local tree.

In the FreeBSD context, one would create a versioned FreeBSD port of their Go software, and then adding a dependency on the libucl port (via the ports mechanism), which will track upstream libucl with FreeBSD-specific patches/build flags.

The vendorizing mechanism of Go doesn't track FreeBSD-specific patches that will likely be applied.


thanks asshole


This is going to need a lot of work to make it balanced and fun to play through. I may end up using thermal expansion or Ender-io crafting components.


EconomizerIntegrationNew (lgu1234) - i386-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)


Eksik parantez.


Koşulsuz for olur mu? Nerede duracak?


Interesting. I'll investigate. Note to myself: test-test-test


It sounds like this is the procedure:
ssh user@loginxx
# system responds with password prompt
Password: <user types in IdentiKey password>
# System contacts Duo on the backend,
# Duo contacts the used via phone (voice only?) or app
# User OKs the login on the app, or does something as instructed by the phone call (enter a code?)
# User logs in.
How should we describe Duo Push?  A mobile or desktop app?  Or a process that currently requires an Android or IOS app?  Is Push mobile-only as suggested by the Duo website?

If a user has to change from Push to phone call, how do they do that? It sounds like you pick your default method at Enrollment. Can a user revisit the enrollment site to change methods?  Is there a link for that?

Do you need to auth again via Duo from the same address for new connections within a reasonable amount of time?

We should also tell people what to expect at the helpdesk:
User goes to the helpdesk
Asks for "RC Duo Enrollment"
The agent gets the user's name, checks the users' ID
(Note that acceptable forms of ID at the IT Helpdesk are a University of Colorado Buff Onecard or a USA state-issued ID card or driver's license.  Contact Research Computing (mailto:) if you need to present another form of ID such as a passport.)
The agent takes the last four digits of the user's ID card number to verify the ID check and sends this information to Research Computing.  Research Computing will then (send the user a link or something via email?)

We need to flesh out the process so the ITSC Helpdesk people know what the full procedure is.
Users will ask "what does this do?" and "what happens next?"
Often users are just told by instructors or lab mates to just "do something" and not why, so the helpdesk people will need to field these questions.  We should also get the link to this page into the Helpdesk  wiki page.  I am not clear on the URL, is it https://www.rc.colorado.edu/support/user-guide/accounts.html ?


that's an interesting one:

firefox:
```
console.log("%s %d", "a", null)
-> undefined
a 0
```

chrome:

```
console.log("%s %d", "a", null)
a NaN
-> undefined
```

I chose the type conversion here because it feels more logic to me as JS developer, but not really sure about that


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

```


Tek bir double değil sz bayt yer istemen gerekiyor.


It doesn't look like solarService is defined anywhere. This breaks the smart phenotype functionality.


Interestingly, `php://temp` seems to work.


`CaseAccessors.get_case_ids_in_domain_by_owners()` gets the domain from the `domain` instance variable on the `CaseAccessors` instance, so it doesn't need to be passed directly. So perhaps `get_case_ids_in_domain_by_owners()` should just be called `get_case_ids_by_owners()`, since the domain is implicit. I was copy/pasting the function names from the dbaccessor functions they replaced.


Lol, true...the ternary operator is definitely not required.


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


hmmm, node 4.x:

```
> new Number(null)
[Number: 0]
```


The comment-text is saved as HTML in the DB. So for example linebreaks are replaced by '
' tags. We'd have to turn the HTML into plaintext again. I think that is not a good way. We should consider either storing plaintext or editing HTML.

What we do now:

1 Writing comment "Hello\nNice course"
2 Convert to HTML "Hello
Nice course"
3 Store to DB  "Hello
Nice course"
4 Display  "Hello
Nice course"
5 Edit comment "Hello
Nice course\nSee you soon"
6 Convert to HTML "Hello<br>Nice course
See you soon"
7 Store to DB "Hello
Nice course
See you soon"
8 Display on client "Hello
Nice course
See you soon"

What we would have to do

5 Convert to plaintext again "Hello\nNice course"
6 Edit comment "Hello\nNice course\nSee you soon"
7 Convert to HTML "Hello
Nice course
See you soon"
8 Store to DB "Hello
Nice course
See you soon"
9 Display on client "Hello
Nice course
See you soon"

What we could do instead (Richtext option)

1 Writing comment "Hello
Nice course"
2 Sanitize HTML "Hello
Nice course"
3 Store to DB  "Hello
Nice course"
4 Display  "Hello
Nice course"
5 Edit comment "Hello
Nice course
See you soon"
6 Sanitize HTML "Hello
Nice course
See you soon"
7 Store to DB "Hello
Nice course
See you soon"
8 Display on client "Hello
Nice course
See you soon"

What we could do instead (Plaintext option)

1 Writing comment "Hello\nNice course"
2 Store to DB  "Hello\nNice course"
3 Convert to HTML for display  "Hello
Nice course"
4 Edit comment "Hello\nNice course\nSee you soon"
5 Store to DB "Hello\nNice course\nSee you soon"
8 Display on client "Hello
Nice course
See you soon"