атрибут href должен быть у всех ссылок


Could you comment on this a bit more? The code indeed disables logging through the log observer. But this is actually what the Yireo_DisableLog module is meant to do. Are you using this module for a different purpose?


для единообразия тут тоже лучше ссылками сделать.


Requesting a change of geometry for a non-moderator is a client error.
What about returning an error instead of silently skipping the geometry?


a co się stało z podstroną comments?


comment on issue


Nope, the behavior was simply unpredictable and might have differed according to the platform (windows vs. linux).

Nonetheless, it was quite likely that `resize(3)` preserved the content of the vector, even though it is not assured - of course.


Now using r73 which closes #71


Sometimes when contributors don't have enough expirience with git - they can commit into their fork master branch. So I this you should add advices if something goes wrong and `git push origin master` errors


No integer but "no more than two effective digits" imo. For example 15000 BTS for LTM grading but not 15769 BTS for LTM upgrading. Another example is 0.1 BTS for price feeding but not 1 BTS for price feeding.


I have reduced that fee to 0 since it can't be used to spam the network and if it was higher, why would anyone pay for it to have it expire before the expiration date


Better to change to "Provided schema contains fields that are not  a part of riak bucket: [list-of-fields]"


> I do not see a point of having text inside a note
For multiline content. We could imagine embedded a two line copyright for instance.

About note being before description ..ok I forgot the xml schema, I thought it was looking better after.
In fact, I don't even know why we need an order here.

About 2. ... ok so this was not implemented like that.


Better to change to "Provided schema contains fields that are not a part of riak bucket: [list-of-fields]"


That would make sense, yes.


I'm definitely not seeing it, if you can do it with less or more concise code please be my guest and submit a pull request as I'd like to know what you're thinking


This is a one time operation and our blockchain is the ONLY one that has this feature.  So why not profit from it!
Do you really think that $1 is to high to setup a recurring payment? Could you name a use case where $1 would be too high?


@radubrehar Do you think that stating the props is necessary?


@rbruce1: What's up with the tcsp? I can't find it referenced anywhere else in the code.


Here we need to use same values as in the frontend application. We need to check with @javierlopezdeancos


Thanks for the credits @mvaneijgen but I have nothing to do with this :)


Oh sorry about that, I did miss it - this is a tough one.

My thinking is, if the alert area covers two districts then that is a single alert targeting both districts for the same event. So the ACK will be bound to the alert message, regardless of their jurisdiction.

If there are two separate checklists specific to each district, then we would have to handle that in the training and SOP as to which one they would select; either one or both. Hence, we can send both checklists in the same alert message, I suppose.


Great, thanks for the quick fix :+1:


This variable should be defined as a project property, not in Java code


TeamCity Ppconsul :: Build (Linux) [Build 84](https://136.243.151.173:4433/viewLog.html?buildId=699&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


Will do ASAP


.btn-green всё ещё нехорошо - если дизайнер цвет изменит, будут проблемы, назови как-нибудь .btn-success или .btn-accept  или .btn-dark.   Тут в макетов не так много типов кнопок получается - красные, они вроде как самые стандартные, прозрачные  - на карточках категорий, белые, в пагинации, фильтрах, и вот эти две - зелёная и белая с зеленой обводкой. Я бы их так назвал:
красные - .btn .btn-standard  или .btn-default  прозрачные .btn .btn-transparent, белые .btn .btn-light, зелёные .btn .btn-dark  и белые с зелёной обводкой  .btn .btn-bordered-light, или же .btn .btn-bright (переводится как яркий)


It appears that both [HexChat](https://github.com/hexchat/hexchat/blob/4362085847f359ed13df6f8f488a06eb52ecd767/src/common/proto-irc.c#L51 ) and [ZNC](https://github.com/znc/znc/blob/1f226d2adeb2cfa08fa54d360c110c16fd361f18/src/IRCSock.cpp#L1184 ) (when acting as client) take the same approach of unconditionally trying CAP LS.  ZNC [additionally notes that error 451](https://github.com/znc/znc/blob/1f226d2adeb2cfa08fa54d360c110c16fd361f18/src/IRCSock.cpp#L1184 ) should be returned by servers that don't support CAP LS.  Right now [in Quassel](https://github.com/quassel/quassel/blob/17311326016f3654fd33a656b2a0c4ac95776db6/src/core/eventstringifier.cpp#L127 ) that would just print an error message in the status buffer yet still carry on.


Thanks for looking. I'll leave it in then.


You call establishSession() if you want your client to be bound to separate DataChannel on server (i.e. have dedicated server context). Cayenne also allows you to have shared sessions, i.e. when multiple clients share single DataChannel (context) on server, which is called "shared session". To be able to call sendMessage you first need to establish session using either one of "establishSession" or "establishSharedSession" methods depending on your needs.


yep this is what i meant, even 15500 bts i think could work.
Also 0.1 is good, but not 0.0989 or 0.01143

But maybe is hard do this with a script...there are several cases...
I don't think it would be difficult to manually fix them


Paste some word said by @BhuzOr in Telegram channel here:
>at the same time i can understand that as right now, we miss some feature to incentives to upgrade if the fee is too high, so we can state that the fee will be higher and probably lock at 150$(or whatever) BUT for the time beeing we run a "promotion" and keep it lower for the next couple of months
and we can also state that we will try to incentivize the upgrade as mush as we can
so in the future we will try maybe to reserve some op to ltm only


I agree on the LTM-only feature,
but I do not agree on having the *claim operation be more expensive.
The withdraw permission feature is something that you offer someone else so they can get hold on their money easily without requesting it. Pretty much like paying rent for your apartment. Why would the guy that gives you the apartment pay money to get paid? It is like being asked $5 to be able to withdraw your money from a cash machine.

Oh .. btw, this feature is probably going to be used alot for credit card integration where you setup a card that can withdraw from your account. If you have this a high fee, so will credit card payments have a higher fee.


why do we want to report fatal error when connecting a system database? looks not a good error info for end user. And from the error information, looks the process exits unexpected. is there a way to improve this?


TeamCity Ppconsul :: Build (Linux) [Build 84](https://136.243.151.173:4433/viewLog.html?buildId=699&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 103 Build time: 00:02:03


> Staging should always be either the exact same commit as master, or an older commit from master.

Good to know! :+1:

I was a little rushing. Now staging and master are the same.


There is no point using `instance_exec` since every code in this method is executed in the context of the object  `GrimmRecord`


Now, every member of the team except Echelon has gone to the "Sickrage" repo. Making sickragetv de facto , the unofficial :)


.product-discount нужно чтобы везде был блочным элементом, так что просто b - не подходит. <div class="product-discount"> и внутри если хочешь можешь <b> оставить, но я бы не стал.


Likely related to this commit, I get this kind of errors when trying to build test/triqs/gfs/multivar/CMakeFiles/mpi.dir/mpi.cpp.o

mpi.hpp:85:29: error: cannot initialize a member subobject of type 'MPI_Op' (aka 'int') with
      an rvalue of type 'nullptr_t'
   return {a, c, root, all, nullptr};


тут лучше тег <del> использовать, класс тот же оставить.


Missing the new line at the end of the file :P


> Imo we can keep the basic fee low but price_per_kbyte fee a little higher than other operations. By now 0.1/0.01 looks fair, but I prefer higher price.

Agreed. However, I would like to encourage development of businesses around this feature and not discourage it already by high fees. Once it gets abused, we can surely increase the fee, but until then, quite some development work in the backend (index custom operations and searches etc.) need to be implemented to really make use of this operation

> We can make this an LTM-only operation.

Agreed


Thanks! Addressed in PR #44.


Send a PR back for this?


Addressed in PR #44.


@rbruce1:
This looks OK to me, maybe with the exception of the TCSP which is added, but referenced nowhere else in the code (i.e. not when writing "twisslike" and "sigmasettings", like it is done for the rest of the collimators you added). Is this intentional?

Also, why when looking for collimators, why not either have a default setting for nsig (0.0?) or an "else" which catches TC-elements that are not recognized?

Meanwhile, I'll go ahead and merge master on top of this.


NOIS KODE! MUCH UGHLY, WERI RUBY :3


у тебя есть кнопка
В закладки
она в макете внешне выглядит иначе, а классы у них одинаковые - не проканает. В сообщении выше я описал подробно, как можно было разные кнопки назвать.


@jaap sorry, I am so used to @ mention people via Slack that I now type it every where and my colleague is also Jaap. Again sorry for the misconception.


How about this? Even more FP and cleaner.

```js
var retrieveArguments = require('retrieve-arguments'),
    minimist = require('minimist'),
    R = require('ramda');

module.exports = function (gulp, processArgv, callbackName) {
  var mergeCmdArgs = R.compose(
    R.merge(R.__, minimist(processArgv)),
    R.objOf(callbackName || "callback")
  );

  var wrappedTask = function (taskName, taskDependencies, taskDefinition) {
    if (!taskDefinition && R.is(Function, taskDependencies)) {
      taskDefinition = taskDependencies;
      taskDependencies = undefined;
    }
    taskDefinition = taskDefinition || function () {};

    var wrappedTaskFunction = function(originalCallbackFunction) {
      var cmdArgsMap = mergeCmdArgs(originalCallbackFunction);

      return R.compose(
        function(args) { return taskDefinition.apply(gulp, args); },
        R.map(R.prop(R.__, cmdArgsMap)),
        retrieveArguments
      )(taskDefinition);
    };

    return gulp.task.call(gulp, taskName, taskDependencies || [], wrappedTaskFunction);
  };
  return R.merge(R.clone(gulp),{task: wrappedTask});
};

```


I don't understand what is the meaning of this commit, "use 100 characters"


full-width самый полезный класс


>>Avoid fractions: although a $1 fee makes some sense, a same fee of 251.12321 BTS makes no sense when showing on the fee schedule.

>so either you get convenience in USD amounts and not in BTS amounts, or vise versa. Since most people do understand USD denoted values and many businesses have approach me about this "issue", I though targeting a USD value makes more sense. Also, except for the crypto crowd. The goal is to have end users that pay their fee in USD (or other currencies, but not BTS).

We won't get convenience in USD amounts, since price of BTS/USD keeps changing every minute (same result applied to other currencies). Unless we implement some USD-pegged fee algorithm in core. So for now, we'd better keep fees in BTS looks better.


pretty hard to follow this test. Maybe a small comment saying what its trying to test


The Mage_Model_Log_Visitor is overwritten in Yireo_DisableLog_Model_Rewrite_Log_Visitor. There, the overriding construct method causes the logging disable.

So the fix this module provides is in the overwrite for "log/visitor". This class is used by the observer that acts on the events: "controller_action_predispatch" and "controller_action_postdispatch". So the disabling of these events cancels out the fix.


I am using phpstorm and in settings the encoding is set to UTF8 let me try by reverting changes


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 45](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=747&buildTypeId=Xbim_XbimWebUi_XbimWebUi) is now running


@ahmadnassri should be `docker run -p 9042:9042 -d --name cassandra cassandra:2.2.4`


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 45](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=747&buildTypeId=Xbim_XbimWebUi_XbimWebUi) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:19


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 46](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=748&buildTypeId=Xbim_XbimWebUi_XbimWebUi) is now running


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 46](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=748&buildTypeId=Xbim_XbimWebUi_XbimWebUi) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:10


HUNTER_PACKAGE_SOURCE_DIR is introduced in hunter_download. But the pattern '${HUNTER_SELF}/cmake/projects/${HUNTER_PACKAGE_NAME}' will also be used in project specific cmake files which calls the hunter_download function. Defining the variable there would help the schemes but not scripts which are used in project specific cmake files. Or I am missing something? I am also little bit confused about the scopes here. In either ways I can also propose a solution in different direction. What if we mandate a common cmake file which needs to be included in every project specific cmake file? This file can nullify the lack of functionality by the use of hunter_add_package as macro, like all the common defines we can add it in this file. What do you think?


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


Follow-up in 9f196835cd6fe41bb25eb6ba14e7b63d08150575


that's something else to handle, like in neko. When try to catch a String, we should also check for a dynamic bytes.


This fails to build on buildbot:
`checking build system type... x86_64-unknown-haiku
checking host system type... x86_64-unknown-haiku
checking for a BSD-compatible install... /bin/install -c
checking whether build environment is sane... configure: error: newly created file is older than distributed files!
Check your system clock
Command '['bash', '-c', '. /wrapper-script']' returned non-zero exit status `


I reckon all that description would be useful in the javadocs. Does calling establishSession() twice reset the session (perhaps create a new cookie value)? Can you switch from shared to non-shared and back?

I'm still wrapping my head around why you'd want a shared session. I guess it could be useful if it read-only.


Favour use of class instance variables over class variables. Check out this [http://stackoverflow.com/questions/15773552/ruby-class-instance-variable-vs-class-variable](link)


@michaelklishin i probably should've verified this on our first discussion. but okay. leaving this as is :+1:


тебе нужно будет делать фон этим блочкам, так что они не могут быть строчными <b> - нужно их сделать подходящим блочным тегом. Например <div>. Или, возможно, один из них является ссылкой на страницу закладок, а другой - ссылкой на страницу корзины. Тогда их надо сделать ссылками.


Ha! I definitely agree that having lots of unit test coverage is a good idea.

When I asked where they came from, I meant did you (a) translate some/all of the integration/regression tests; (b) write them from scratch yourself by inspecting the implementation and/or looking at test coverage; or (c) something else? That's what I think it would be useful to explain in the commit note.


Тебе нужно будет делать фон этим блочкам, так что они не могут быть строчными < b> - нужно их сделать подходящим блочным тегом. Например < div>. Или, возможно, один из них является ссылкой на страницу закладок, а другой - ссылкой на страницу корзины. Тогда их надо сделать ссылками.


Maybe something like this:
```js
      try {
        var paths = yield readdir(node);
      } catch (e) {
        console.warn(`Access to ${node} from ResourceTree denied. (${e})`);
      }

      yield paths.map(child => co(function * () {
        child = path.join(node, child);
        const rpath = relative(path);

        const excluded = exclude.some(rx => rx.test(rpath));
        if (excluded) return;

        if (cache[child]) return;
        cache[child] = true;

        try {
          const stat = yield stat(child);
        } catch (e) {
          console.warn(`Access to ${child} from ResourceTree denied. (${e})`);
        }

        if (stat.isDirectory()) folders.push(child);
        if (stat.isFile() && include.some(rx => rx.test(rpath))) list.push(child);
      }));
```
I don't test this code.


Having added all these unit tests, did you consider removing some of the integration/regression tests?


Hey Temptin,
your idea of using one main and showing the GUI only if there are no arguments sounds good. I was thinking of making it a cmake option, but this is better.
Your other feedback is useful too. I will take a look at improving it when I get time. I mostly did it to learn how to use qt.
I had seen the issue with the call duration when it was reported and read some of your comments. I might look into that at some point later for fun as well. Is it ok to use the test accounts you have in resources/test_databases for testing?


> if the alert area covers two districts then that is a single alert targeting both districts for the same event. So the ACK will be bound to the alert message, regardless of their jurisdiction.

I was imagining it to be common that a single Alert covers multiple districts, and we would want to see an ACK per district...not just 1 for the alert?
Therefore if someone is responsbile for multiple districts then they should ACK for both.


В учебных целях предлагаю отказаться от сторонних приложений для пагинации, реализовать собственными силами, чтобы поработать руками с QuerySet'ами.


error on space after comma.


2016-02-04 9:28 GMT+01:00 James O. Coplien <notifications@github.com>:

>
> > Den 04/02/2016 kl. 07.58 skrev Rune Funch Søltoft <
> notifications@github.com>:
> >
> > Default initialisation to any thing not just null which is why I argue
> for the ability to void default initialisation altogether when possible.
> The programmer would be the oracle that knows when a meaning full
> initialisation can be made
>
> Altogether?
>
> Great — try it out and send me a pull request. I think I can prove that
> your vision is NP-hard. You can’t guarantee that the programmer has
> initialized something before use without really draconian rules.
>
> If I look at the current bulk of trygve code, very few of the megatype
> constructs have any constructors at all. Your wish, it seems, would bloat
> the typical program.

The only programs that wouldn't be valid, would be those that call a
constructor explicitly when that particular constructor is not defined. but
the class has one or more constructors declared explicitly. I'm arguing for
the programmer to be able to say "The only valid instantiation of objects
of this class, is if you provide these arguments" and what I understand is
that the it leads to problems if we provide the programmer the ability to
express those constraints. I have no problem with classes having default
constructors. My beef is with them being required to have one to the extend
that even when the programmer has made it clear that there should be none
the compiler will still create one


Yep, in a later commit, some of the integration and regression tests are removed, many of them seemed to be just checking data, which this unit test is trying to cover.


Стандартная красная кнопка, так что можно ей добавить .btn .btn-standart в дополнение к её классу .products-all


Please squash this commit with other fixes in `node-debug`


"adrianbroher committed on Jul 1, 2015" -> clearing out the backlog...?


Calling establishSession() second time will create second server session and return back new RemoteSession object, so you will have two session and can use both of them (assuming you have created two ROPConnector instances and configured them with different RemoteSession objects). You can have two RemoteSession objects, one for shared and one for non-shared session, and use them independently.


А где текст кнопки?)
И это стандартная красная кнопка, так что можно ей добавить .btn .btn-standart и на этом её стилизация завершится. А класс promo-slide-button будет использоваться только если js-ом как-то.


why hard code it?


Why we need this?


Well, the name is kind of unclear, and I was in a rush to get this in before the next crash so I didn't do my usual `git blame`. :|


There is no handling of the buf == NULL. Propose to pass by ref. And seems it can be even const. I see no check for null of the ids too. Maybe it worth to pass it by ref too.


на мой вкус - это должно быть заголовком. Хотя бы h3


Why error?


minor nitpick. could you listen on port `0` instead and then in the listen callback you can get the port used by doing `server.address().port`?


"Provided schema has nothing about  the following fields returned by query: [list-of-fields]"


I don't think the stateful/stateless distinction is too important. Randomly bailing out in the middle of a command could lead to unpredictable behavior, too.


DEBUG


,hvnvbkghmcbn


> clearing out the backlog...?

Pretty much. There are still some commits (around 70) left.


return


above


You made my day :)
1 evening with ramdaJS is not enough

On Thu, Feb 4, 2016 at 11:26 AM Christoph Hermann <notifications@github.com>
wrote:

> How about this? Even more FP and cleaner.
>
> var retrieveArguments = require('retrieve-arguments'),
>     minimist = require('minimist'),
>     R = require('ramda');
> module.exports = function (gulp, processArgv, callbackName) {
>   var mergeCmdArgs = R.compose(
>     R.merge(R.__, minimist(processArgv)),
>     R.objOf(callbackName || "callback")
>   );
>
>   var wrappedTask = function (taskName, taskDependencies, taskDefinition) {
>     if (!taskDefinition && R.is(Function, taskDependencies)) {
>       taskDefinition = taskDependencies;
>       taskDependencies = undefined;
>     }
>     taskDefinition = taskDefinition || function () {};
>
>
>     var wrappedTaskFunction = function(originalCallbackFunction) {
>       var cmdArgsMap = mergeCmdArgs(originalCallbackFunction);
>
>       return R.compose(
>         function(args) { return taskDefinition.apply(gulp, args); },
>         R.map(R.prop(R.__, cmdArgsMap)),
>         retrieveArguments
>       )(taskDefinition);
>     };
>
>
>
>     return gulp.task.call(gulp, taskName, taskDependencies || [], wrappedTaskFunction);
>   };
>   return R.merge(R.clone(gulp),{task: wrappedTask});
> };
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/przemcio/gulp-param/commit/888a8fc76b91d55567bd184ded4512ca9d6a5e78#commitcomment-15881916>
> .
>


первый < b class="product-price"> - надо заменить на < del class="product-price">
второй < b> - на < div class="product-discount">
иначе будет неудобно задавать фоновый цвет, т.к. у строчных элементов нет вертикальных отступов.


тоже самое


utp does not support half-open mode so i'm not we can actually get this behaivor. i'll investigate further though


good, I'll steal this doc


@lujun9972 昂实现的比较拙计来着，所以一时半伙也稳定不下来，等补完测试再完善可能才会写怎么写来着-。-、
话说我看了下因为 @kuangdash  的思路似乎和其他的博客系统确实还是有很多地方不一样的，blog-admin为了一致性肯定只会实现一小部分大家都有的功能，所以 ego 自己实现一套 post-manager 也是有价值的、


Seems time is only for reading. Propose to pass by const ref (at the moment there is no check for the null pointer). Use c++ cast.


Why not use the fail result value instead of additional outcome parameter?


Атрибут href потерялсяя
Это стандартная красная кнопка, я бы её сделал .btn .btn-standard


@lecopzer : I applied your changes to validate test programs via `make check` in commit 0aa5099d5f03f5ef894a55285183520fd49ed729.


tab-content


Virtual destructor or at least protected.


In the new constructor you forgot to allocate memory for index. Also, do you really need the first (empty) constructor?


Did you see https://travis-ci.org/mathjax/MathJax-node/builds/102695482?

It would be good to always do a PR so we can get alerted to build failures.


id = "tab-1"


Tnx for the fix.


I don't think we need to do this here, llvm will state the error just as clearly


Do we have time for doxugen?


тут delivery, guarantee, credit не нужны - их нужно навесить не на кнопки открытия табов, а на сами табы, по этим классам фоновые картинки будем к ним крепить.


can you please replace that to `<intsafe.h>` instead ?


id = "tab-2" class="guarantee"


Этот элемент тоже является элементом выпадающего select


id = "tab-3" class="credit"


Diffing this vs. master shows that the only real change brought by this branch is adding the TCLX and TCLD collimators. The TCSP's came with the previous change: https://github.com/SixTrack/SixTrack/commit/41e8e6dd5ad15343a7952943e30f57b4fd06c238

The diff:
```
11:43:37 kyrsjo@pcbe15843 SixTrack $ git diff master
diff --git a/SixTrack/sixtrack.s b/SixTrack/sixtrack.s
index 0507515..188ac41 100755
--- a/SixTrack/sixtrack.s
+++ b/SixTrack/sixtrack.s
@@ -29605,7 +29605,9 @@ C     Convert r(1), r(2) from U(0,1) -> rvec0 as Gaussian with cutoff mcut (#sig
                elseif(bez(myix)(1:4).eq.'TCLP' .or.                     &
      &                 bez(myix)(1:4).eq.'tclp' .or.                    &
      &                 bez(myix)(1:4).eq.'TCL.' .or.                    &
-     &                 bez(myix)(1:4).eq.'tcl.') then
+     &                 bez(myix)(1:4).eq.'tcl.'.or.                     &
+     &                 bez(myix)(1:4).eq.'TCLX' .or.                    &
+     &                 bez(myix)(1:4).eq.'tclx') then
                   nsig = nsig_tclp
      &                 bez(myix)(1:4).eq.'tcli') then
@@ -29615,7 +29617,9 @@ C     Convert r(1), r(2) from U(0,1) -> rvec0 as Gaussian with cutoff mcut (#sig
                   nsig = nsig_tcxrp
 !     TW 04/2008 ---- start adding TCRYO
                elseif(bez(myix)(1:5).eq.'TCRYO' .or.                    &
-     &                 bez(myix)(1:5).eq.'tcryo') then
+     &                 bez(myix)(1:5).eq.'tcryo'.or.                    &
+     &                 bez(myix)(1:5).eq.'TCLD.' .or.                   &
+     &                 bez(myix)(1:5).eq.'tcld.') then
                   nsig = nsig_tcryo
 !     TW 04/2008 ---- end adding TCRYO
                elseif(bez(myix)(1:3).eq.'COL' .or.                      &
@@ -30172,7 +30176,9 @@ C     Convert r(1), r(2) from U(0,1) -> rvec0 as Gaussian with cutoff mcut (#sig
             elseif(bez(myix)(1:4).eq.'TCLP' .or.                        &
      &             bez(myix)(1:4).eq.'tclp' .or.                        &
      &             bez(myix)(1:4).eq.'TCL.' .or.                        &
-     &             bez(myix)(1:4).eq.'tcl.') then
+     &             bez(myix)(1:4).eq.'tcl.'.or.                         &
+     &             bez(myix)(1:4).eq.'TCLX' .or.                        &
+     &             bez(myix)(1:4).eq.'tclx') then
               nsig = nsig_tclp
             elseif(bez(myix)(1:4).eq.'TCLI' .or.                        &
      &             bez(myix)(1:4).eq.'tcli') then
@@ -30181,7 +30187,9 @@ C     Convert r(1), r(2) from U(0,1) -> rvec0 as Gaussian with cutoff mcut (#sig
      &             bez(myix)(1:4).eq.'tcxr') then
               nsig = nsig_tcxrp
             elseif(bez(myix)(1:5).eq.'TCRYO' .or.                       &
-     &             bez(myix)(1:5).eq.'tcryo') then
+     &             bez(myix)(1:5).eq.'tcryo'.or.
+     &             bez(myix)(1:5).eq.'TCLD.' .or.                       &
+     &             bez(myix)(1:5).eq.'tcld.') then
               nsig = nsig_tcryo
             elseif(bez(myix)(1:3).eq.'COL' .or.                         &
      &             bez(myix)(1:3).eq.'col') then
```

@rbruce1: I'll open a pull request (which will show ~the same diff as here, i.e. your branch+master vs. master). Will then kill some extra spaces etc., bump the version number, and merge.


NP :)


Это будет фоном сделано, а не Img, этот тег не нужен


This is terrible! (not really)


Did you read [mostly-adequate-guide](https://drboolean.gitbooks.io/mostly-adequate-guide/)?
It's awesome. I learned a lot from that book.


Это отлично!)


BOOOO


Подробнее о_компании


gl


여기에 댓글을 작성할 수 있다는 말이다


OK I'll do it later


@OmgImAlexis yes, actually I wanted to use JS with the json.dumps({'result': 'redirect'}), but it doesn't redirect me despite of what I've done in ajaxEpSearch.js. I must confess that I'm a bit lost about this.


Cao


여기에도 해 보고


그냥 전체적인 코드 리뷰를 할 수도 있다


I think it should be without the `..` - it's not clear to me what the relative path means here


only one more overload of NetConnection is getting used see ClientDataSource#getConnection which won't reflect the same. ClientDataSource is used for XA connections which is rarely used. Probably you would like to handle that overload version too post this merge.


Rest looks okay other than one comment above. Please merge.


Можно либо `<span class="icecream-price-P"></span>` - а ему потом назначить фоном иконку рубля, либо саму цену обернуть в span с классом и ему назначить ::after с фоновой картинкой рубля


This was fixed in https://github.com/phoenixframework/phoenix/commit/f433d3df162d38818b4730068e3aceda6022d12b - leave a comment if you find anything I missed.


How come? Was just getting used to the variable width font for results after the refactoring broke that option and I was too lazy to fix it ;)


This fails to build on buildbot:
`Initialized empty Git repository in /boot/home/builder/x86_64/media-gfx/pngcrush/work-1.8.0/sources/pngcrush-1.8.0-nolib/.git/
Applying patchset "/boot/home/builder/x86_64/media-gfx/pngcrush/patches/pngcrush-1.8.0.patchset" ...
Patch format detection failed.
Command '['git', 'am', '--ignore-whitespace', '-3', '--keep-cr', '/boot/home/builder/x86_64/media-gfx/pngcrush/patches/pngcrush-1.8.0.patchset']' returned non-zero exit status 1`


using threadscope as a fallback seems to be pretty dangerous considering we moved everything to async in v6. Shouldn't we throw an exception when resolving `InstancePerUnitOfWork` scope objects without a named scope?


shouldn't this be with `get`?


post to github


Картинку тоже оберни в ссылку, возможно человеку будет проще тыкнуть на картинку мороженого чем на его название


Biztos, hogy jó importot használsz? Nashorn?
import jdk.nashorn.internal.ir.annotations.Immutable


Y si es 1 que pasa? Quedaría 0 y seguiría existiendo la key en el mapa.


Всё отлично! Пару правок и делай pull request


new


Task 1,2,3


Well theoretically if something breaks here, the file will remain there. Also `mktemp` is not safe and 2 processes can return the same file...


eww


formatting :)


Лучше сделать 2 отдельных метода - для верхних и для нижних мест, т.к. вызов вида resource.all_places(type)[0] не очень понятен


use `get` here in these places


It could be just me, but I think the following is a little easier to understand:
```py
#! /usr/bin/python2.7

import json
import time
import threading
import ev3dev.ev3 as ev3

# Log motor readings at requested intervals
class LogThread(threading.Thread):
    def __init__(self, interval, motor):
        super(LogThread, self).__init__()

        self.interval = interval
        self.motor    = motor
        self.done     = threading.Event()

    def run(self):
        tic = time.clock()
        self.results = []
        while not self.done.isSet():
            time.sleep(self.interval)
            self.results.append((
                    time.clock() - tic,
                    (self.motor.speed, self.motor.position, self.motor.duty_cycle)
                    ))

    def join(self, timeout=None):
        self.done.set()
        super(LogThread, self).join(timeout)

test = json.loads( open( './run-direct.json' ).read() )

# Iterate over commands in the control file
def commands():
    for d in test['data']:
        for c in d['commands']:
            yield c

port     = test['data'][0]['port']
interval = test['meta']['interval']
max_time = test['meta']['max_time']

print(port)

# Send the specified commands to the motor at the specified time points,
# while logging the motor attributes.
motor = ev3.Motor(port)
log = LogThread(interval * 1e-3, motor)

start = time.clock()
end   = start + max_time * 1e-3

log.start()

for c in commands():
    then = start + c['time'] * 1e-3
    while time.clock() < then: pass

    for a in c['attributes']:
        for key,val in a.items():
            print("    ", key, " = ", val)
            setattr(motor, key, val)

while time.clock() < end: pass
log.join()

print('data = [')
for r in log.results:
    print(r, ',')
print(']')
```


Me llama mucho la atención que es lo que devuelve esto jaja


This will break if port is something like `foo`. We'll need to do something more like:
```python
try:
    self["port"] = int(self["port"])
except (ValueError, TypeError):
    raise usage.UsageError(...)
```
Then we also won't need to convert port to an integer in `makeService` since we'll already have done it.


Dear Sir,

I have done all the thing to making video using nodejs but there is proble to play the video in mozilla the problem
1. video unable to seek at the time of playing the video
2.unable to replay the video also.
 Issue in Google Chrome
3. video play but seek not working at first time. All the things working like seek, play, pause or time updation everything working in chrome but  seek is not working at first time of play the video

Pls help us to solve this issue sir ....Its urgent


should this be abstract then?


As against having multiple if/else statement, you can split this method into multiple functions eg
```ruby
  def self.primary_key_query(type = false)
    "PRIMARY KEY AUTOINCREMENT" if type
  end
  def self.autoincrement_query(type = false)
    "AUTOINCREMENT" if type
  end
  def self.nullable_query(type = true)
    "NOT NULL" unless type
  end
  def self.type_query(type)
    type.to_s
  end
```
and call them with `properties << send("#{type}_query")` in create_table method.


Acá no debería agregarse el address?


not a Scala way...
```scala
val isView = table match {
  case _: MaterializedViewMetadata => true
  case _ => false
}
```


The link to `/check-state-pension` seems to respond with a 404 Not Found.


formatting


@sk89q's command framework?


Сделай 2 метода top_places и bottom_places, это гораздо легче будет читать и понимать


Поправь замечание


Thanks

No, I haven't read it ... yet :)

On Thu, Feb 4, 2016 at 11:49 AM Christoph Hermann <notifications@github.com>
wrote:

> Did you read mostly-adequate-guide
> <https://drboolean.gitbooks.io/mostly-adequate-guide/>?
> It's awesome. I learned a lot from that book.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/przemcio/gulp-param/commit/888a8fc76b91d55567bd184ded4512ca9d6a5e78#commitcomment-15882412>
> .
>


@samny, @danpaulsmith Hey guys, what is the status of this fix? BTW angular has [isString](https://docs.angularjs.org/api/ng/function/angular.isString) method.


seems like a bad assumption that it's a MatrixError? could you add a comment to point this out so I remember it when I come to deal with errors here?


Since we already have a `withdrawal_period_sec` field when creating the withdraw permission, we can charge dynamic "claiming" fee based on that, so less fee for daily payments every time but higher fee for monthly payments every time. However it could be gamed if user creates a permission just before every payment time point. Anyway a percentage based fee structure is ideal.
Note: all features said in this comment need development.


Why are these two lines repeated??? I wrote them just once...


Great!
Congratulations on the quick fix!


That will be the URL to the new calculator when DWP launch it (maybe later today?)


@nickdaugherty just showing this feature


Are you sure these `HandleScope` aren't needed?  The fact that they are declared and never explicitly accessed does not mean that they aren't used.  As per the documentation:

> A stack-allocated class that governs a number of local handles. After a handle scope has been created, all local handles will be allocated within that handle scope until either the handle scope is deleted or another handle scope is created. If there is already a handle scope and a new one is created, all allocations will take place in the new handle scope until it is deleted. After that, new handles will again be allocated in the original handle scope.
>
> After the handle scope of a local handle has been deleted the garbage collector will no longer track the object stored in the handle and may deallocate it. The behavior of accessing a handle for which the handle scope has been deleted is undefined.

By removing these, you might be affecting the scope of some internal objects in ways you did not intend, and it will be very difficult to detect such behavioral changes without memory measurement tools.


hey @mgruner, It looks like they moved `panda*` from `bin` to `share/perl6/site/bin`.
You should be able to find it in your `/usr/local/Cellar/rakudo-star/2016.01/share/perl6/site/bin`.


I've just tested it again and I noticed those things:
* Until the certificate has been generated a domain musn't have a SSL redirect (otherwise the token can't be fetched from Let's encrypt) - You may want to add a corresponding note in the README.
* In the future, it would be great if there would be something like an overview of the certificates with information like expiration date, etc.
* Side Note: Apparently, a subdomain inherits the settings "iswildcardomain" as well as "wwwserveralias" from its parent domain. This may result into unwanted www subject alt names for subdomains... IMO, those settings shouldn't be inherited.

Thanks again!


**Code Quality - Nitpick**

You don't need to pad the brackets used in functions, loop declarations or even conditional statements.
i.e
```js
for ( var i = 0; i < noDups.length; i++ ) {
```
should be
```
```js
for (var i = 0; i < noDups.length; i++)  {
```


Why closed, you need to check the record against CrossRef again.


do we need both a colon and a dash?

Timeshifting: - hh:mm:ss


It's fixed and and built into /dist last time I checked.


`Handle` itself is not deprecated.  Could you tell us which versions of `node.js` you built and tested this against?  We want to be sure not to break older versions, such as 0.10.x and 0.12.x while at the same time also supporting the latest build.


there's no ```pr``` target in this Makefile


will be interesting to see what this one is going to look like.


Hi Roberto,

Thanks, I was thinking to use this with Summer15GS + 76 DR. Will this be problem ?

Thanks,
Yuta


doesn't this mean a child container won't bind itself because it will always find the "root" container?


var $car = array('Make' => 'brand', 'Model' => 'model', 'Model_year' => 'year', 'Engine_type' => 'engine');


Thank you very much for reporting!


@DMips Will the sign be greater than or less than. It should be less than.


```
$key = $nodename->childNodes[0]->nodeValue;
$car_details[$this->car[$key]] = $nodename->childNodes[1]->nodeValue;
```


**Code Quality - Nitpick**

You seem to pass in a chain of argument into different functions in lines _20_ to _26_.
If each individual is not used separately as is the case, you could have a single variable receive the returns of all the functions like so.

```js
var result =  removeDuplicates(removeStopWords(removePunctuation(data)));
```

You could also process them in a single function, if you only use them together, like so:

```js
var result = refineData(data);
```

In the above, `refineData` takes care of all processing.


Probably too late to convert the code, but for streamlining RPC like communication you may want to check the use of Thrift to automatically generating the communication code:
http://www.yarp.it/thrift_tutorial_simple.html
http://www.yarp.it/thrift_tutorial.html


@nomeata Can you elaborate? There's currently no bound on `containers`, and besides `transformers-compat` doesn't depend on it.


Do you want to order the queryset here?


I changed my mind. `withdraw_permission_delete` should be encouraged since it releases resources. I propose 0 fee.

In technical:
Witness_node need to maintain a list of all withdraw_permissions to improve performance of "claim"/"update"/"cancel" operations. Witness_node can be lazy to remove expired withdraw_permissions (do it in maintenance interval and when someone claims) for better performance.


delete


+1 for the log class. Maybe use a small helper for the "time * 1e-3" or maybe specify it in seconds in the data file already.


**Code Quality - Nitpick**

The functions that are required by your `Index` class should be private and located in the class, so they cannot be altered by external code, or used independently, as it could compromise the proper operation of your class.


Is it in your repo? I found this code in the dist `_.contains(target.className.split(" ")` [angularjs-dropdown-multiselect.min.js](https://raw.githubusercontent.com/dotansimha/angularjs-dropdown-multiselect/master/dist/angularjs-dropdown-multiselect.min.js)


I think it is slightly better to use `%bcond_with`, as in https://github.com/yast/yast-core/pull/105/files#diff-5ada80d590698e14cfda5a88dce9f4c9


I moved it to Generic.xaml, But also noticed that when it comes to Icons the user might want to have them mirrored in RTL (for example for arrow icons) So I used TemplateBinding.


Сделал. Можешь посмотреть у меня последнюю версию. Только я не понял как часть твоего pull request'а замерджить, потому заодно и E= исправил.


Для div с классом content-info тоже нужен флоат!


Тебе получается pull request надо откатить и у меня последнюю версию взять


@MiroCillik jeste se divam na ty sleep. Je tech 5 sec spolehlivejch? aby to pak random nefailovalo. Maji to nekde popsane to chovani?


Where does the magic number of `30` come from? Can it at least be extracted into a descriptive constant on the calculator or perhaps a separate intention-revealing calculator method should be introduced.


**Code Quality - Critical**

The indentation used in your code base is inconsistent. You use 4 spaces in this script and 2 in your `../src/inverted-index.js`.

You should keep it consistent at 2 spaces (not tabs), as is recommended in most style guide.


A http redirect won't do anything at all with js. You need to get the URL you want to redirect to and then just use something like `window.location('http://google.com');` to redirect.


In all the other documents we seem to use the `$` sign instead of a hash. Could we use it here as well?


item-photo-main этот элемент не надо флоатить
он хорошо стоит и флоатов)


Hello, do you think we could symlink those so that they are in /usr/local/bin too?


`/*updateWnd=*/true`


Did you mean `which scopes are available`? And I'd prefer `what they do`.


`CopyPixelBuffer` is implemented at the top of this very commit: https://github.com/mono/maccore/commit/0168e84007c8657bc73d864c3fd85d4d0450109e#diff-b7f510e517c88eb74b4d32596490ca7aR38


**Cide Quality - Nitpick**

You should use strict javascript by adding `'use strict';` to the outermost closure of the script.
This keeps your code base standard with little to no ambiguity is allowed by mediocre Javascript.


@PhilipGarnero Any specific reason to keep this line ?


`using public key authentication`?


This is strange. Are these substituted by RPM build, like other percent macros in the spec file? Where are they defined.


Can you please do changes as separate commits? This commit should've been two: one adding const correctness, another refactoring DistanceToLine.


Is there a comma missing after `Additionally`?
`is given for`?


Maybe rephrase that slightly to `not only the meta data [...] but also the file content is extracted.`


Why is this not an inline method of Vector ?


In fact, perhaps we can extract a method which encapsulates both conditions in this line, e.g.

```
<% if calculator.state_pension_forecast_available? %>
```


<Compile Include="RegExTest.cs" /> this file was not included in commit. I've removed it. I see that ReplaceTests.cs was there instead so I've added it.


I'm wondering if clearAttributes() should really reset the value mask to fully active? Feels like it should either retain topology or set everything to inactive


**Code Quality - Nitpick**

You should resolve all the lint issues raised by JSHint. There are there to guide your code base to quality.


`that everyone with access to the system description automatically has access [...]`?


You realise that this isn't a good idea..... booleans should be treated as Boolean, and not a string.... especially a string value in a specific language...... this will break Booleans for users working in any language but English; it will break any formulae that check the cell value type, etc


We recently decided to require unconditional sudo rights, so that description doesn't fit anymore.


@dzimine done. thanks.


I think this commit is causing a problem:

```
[ INFO] [1454583202.938320216]: Pan tilt sweep started -> acquire data
cloud_merge: /usr/include/boost/smart_ptr/shared_ptr.hpp:653: typename boost::detail::sp_member_access<T>::type boost::shared_ptr<T>::operator->() const [with T = pcl::PointCloud<pcl::PointXYZRGB>; typename boo
st::detail::sp_member_access<T>::type = pcl::PointCloud<pcl::PointXYZRGB>*]: Assertion `px != 0' failed.
[cloud_merge-1] process has died [pid 27773, exit code -6, cmd /home/strands/chris_ws_metric/devel/lib/cloud_merge/cloud_merge __name:=cloud_merge __log:=/home/strands/.ros/log/bad8798e-c8eb-11e5-8558-00032d225
87c/cloud_merge-1.log].
log file: /home/strands/.ros/log/bad8798e-c8eb-11e5-8558-00032d22587c/cloud_merge-1*.log
```

It works fine when this commit is reverted.


This seems like a purely cosmetic change. Probably it shouldn't be part of this commit.


that would be great, just need to find out how to symlink another bin


private constructor is missing


I think I'd rather say that the system description directory is made readable only for the user. Whether the root user or other users with sudo rights can still access it is out of scope.


I understand that it's not easy to test, but it would be great to have a UT


issue #607


`By default, all [...]`?


No need to change the problem came from my code


`[...] in the home directory of the user running machinery.


Same rotation issue on Lumia 550 with Win10 Mobile


i'm just learning do this ...i'll need more braim... but  I'm trying..


:smile: :+1:


`There is a [...] file in each description directory`


If you look on realisation of `console.log` you'll see something like that:
```
Console.prototype.log = function() {
  util.format.apply(this, arguments);
};
```
Now we replace `util.format` by our func and we expect that `console.log` will be in previous frame.
So in this place we need `Error.stackTraceLimit = 2`;


Az osztó lehet negatív?


IMO This testcase is too many test. Could you make `moveTo` module that it has many test?


Do we need to declare these handlers in this header file? (I know we previously did, but I'm not sure we need to. At least, uVisor builds fine if we don't declare these here.)


There is no explanation in the commit message why ~230 lines of `StatePensionAgeTest` integration test have been commented out in this commit.

Also I think it would be preferable to delete the lines, rather than comment them out. They can always be retrieved from git if it turns out they are needed in the future. Currently these lines remain commented out even after the final commit of this PR.


Should this be 2016 only? If not, at least include 2016.


112796613_DesicDehum (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2059 of 2059 tests passed, 1 test warnings)

 * 7 tests had: AUD diffs.
 * 1 test had: Table small diffs.

![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)


How is that section related to the security implications? I think we should explain the security implications of the web server and esp. the public option, but not approach it from the presentational pov.


Can we rename `DateOfBirth` into `Birthdate`? Shorter and cleaner.


Seems like we need to sync usage and setting this field


Add debug log with set value


This line bothers me a little, but it’s not really your fault :) Guess I should add an optional strip-html step to conform to the spec. And if that’s not enabled, the HTML should be left in place, in my opinion.

Is there any chance that you could do me a big favour and add this too, now that you are familiar with the codebase and have everything set up? It should be pretty straight-forward if we use a third-party module to strip HTML.


JS translations - woop!


Personally, I would prefer the key will `nil` value is dropped instead of raise a warning.

In the `[]=` method, when assigned a `nil` to the key, the key is deleted from `@header`, which means `nil` is acceptable value. Otherwise, I would think `[key]= nil` to raise a warning as the init method.

``` ruby
# https://github.com/ruby/ruby/blob/3e92b635fb5422207b7bbdc924e292e51e21f040/lib/net/http/header.rb#L37
  def []=(key, val)
    unless val
      @header.delete key.downcase
      return val
    end
    @header[key.downcase] = [val]
  end
```


May be add logs for each case where handler is `null`?


**Code Quality - Nitpick**

Your variables should be fully descriptive. Abbreviations are good but the shouldn't compromise the hints that the variables names should present.


Make install is failing, it's missing the mandir directory creation before the install command

mkdir -p $(mandir)


Különös számomra, hogy ugyanabban a listában egyszerre számokat és string-eket is tárolsz.


How can I get the new version with this bug solved? Currently I am using plotly 2.0.16 and R says this is the last available version. Thanks!


Coudl we move it down after all return `ifs`?


Sorry, I meant `transformers`.


For compatibility with Python 3, you should use:
```
from __future__ import division
```
at the very top, and replace all occurrences of `A / float(B)` with `A / B`


perchè hai fatto sta faccina ?


This closes #42?


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


copying the result file
test1_coreos_VM_creation (integration.component.test_coreos.TestDeployVmWithCoreosTemplate) ... === TestName: test1_coreos_VM_creation | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 1 test in 1049.034s

OK


A két nulla ebben a sorban ugyanazt reprezentálja: [[(0): -1], [], 0]

A -1 ebben a sorban egy speciális érték ami csak azért van ott, hogy a "repetitionStart >= 0" feltételt ne teljesítse. Ha a "def previousStep = steps[step.next]" egy Optional-t jelentene (vagy neadjisten nullable-t), akkor ezt az egész speciális értékes és "[(0): -1]" -es történetet ki lehetne küszöbölni.


ciao luca


I think we should ensure that there are two Genders at least. And add static properties like `Male` and `Female` for convenience.


sucaaaa


Wow, meglepően hosszú a ciklus ahhoz képest, hogy milyen kicsi szám az osztó.


forteee


Thanks - I forgot to do this. I'll pull it down and push to the main repo now.


Try `devtools::install_github("ropensci/plotly") `. I believe the latest version is 2.4.1


Very good work! Will you add some (more) barrier mountains in Kalimdor?


:D


The ACK, simply serves as a feedback from the user acknowledging receipt of the message. So I think it is independent of their jurisdictional responsibilities.


it would be better simple.
```js
indexToMove = indexToMove + (indexToMove  < 0 ? 1:-1) * panel.count  ;
```


Missed to remove the decorator in line 45?


merge


Yes, please look at the ajaxEpSearch, that's why I've done without success


My point is that an on-page script might want to overwrite this callback to catch this event, in which case you won't be able to catch the error event on `browser` anymore (even if handling the error by the on-page script fails).


Thanks royr2. It works! I just had to install Rtools from https://cran.r-project.org/bin/windows/Rtools/ as well.


Egy általános megjegyzés: a megoldásod imperatív stílusú. Számomra mostanában érdekesebb témát jelent hasonló algoritmusokat funkcionális stílusban megoldani.


After the refactor, this function is no longer used and can be removed.


See http://en.cppreference.com/w/cpp/language/auto for details.


Dot missing after
> Behold `request!` and `request_ref!`


The intention was that `NextSimulationStep()` should calculate the simulation state that would result after `timeStep` seconds. How do you intend to structure it without this parameter?


Good spot


Good point. I need an object that is contextual to that async scope. Any ideas?


Nice work. Could you make a branch for this task and make pull requests to merge it into master? We could then do nice things like easy reviewing of code without looking at intermediate changes and tie closing a task to the pull request.


tajpo


The problem (I think) with kyotocabinet is that it's GPLv3 whereas qdbm is LGPLv2.1 with a similar spec. Among other things it drops sparse files, tho:

> a database file created by QDBM depends on byte order of the processor, to do with it, utilities to dump data in format which is independent to byte orders are provided.

Complete spec http://fallabs.com/qdbm/spex.html


Ya lo hace mediante el "checkMultiBox" en NpcManager.java


LGTM


added assertions on returned message statuses


Testing....


Test comment.


Thanks. As a cast, that means we always expect that type, and anything else is a bug; is that correct? If so, should this function take that type and push any casting responsibility up to the caller?


Copy: The comma between "State Pension age" and "Pension Credit qualifying age" seems ungrammatical to me. I'd have thought "and" or "and/or" would be more grammatical.


Please use `‹` and `›` not `<` and `>` to delimit parameters. Again we may rely on this when creating an editor, taking `‹…›` as a placeholder the user should fill in, while `<` and `>` are regular mathematical symbols and part of usual program text. Perhaps we should add a `forbidden` check to the build instructions, so that Travis can catch and report this kind of thing.


This ports will be open in monitoring and KVM also?


will add an implementation after your proposed PR (https://github.com/fvn-linagora/james-project/pull/1) has been merged


Hi Mark,
I apologise if making mistake, but is there any better way to get boolean cell value from reader as string 'true' : 'false' and not '1' or 'empty'?Here I found the same question.
http://stackoverflow.com/questions/18515150/cells-with-boolean-values-true-and-false-are-not-retrieved-using-phpexcel

Thanks in advance.

      From: Mark Baker <notifications@github.com>
 To: ivmikulic/PHPExcel <PHPExcel@noreply.github.com>
 Sent: Thursday, February 4, 2016 12:19 PM
 Subject: Re: [PHPExcel] print boolean cell value as raw string (9735243)

You realise that this isn't a good idea..... booleans should be treated as Boolean, and not a string.... especially a string value in a specific language...... this will break Booleans for users working in any language but English; it will break any formulae that check the cell value type, etc—
Reply to this email directly or view it on GitHub.


> I just moved an existing hack to another location, so I didn't expect any regression. I am right ?

No, in the past Object.assign wasn't needed because the object wasn't immutable.

> Do you think you could propose a patch to jsdom so that we can drop the hack here?

jsdom is behaving correctly there though, the object should be immutable. And changing implementation details requires that you use a jsdom implementation detail via idlUtils (at the moment). The API is in for an overhaul, but even then I'm not sure we'd add an interface to change random properties on event.

But I just noticed: if you use the MessageEvent constructor (you can find it on window), the second parameter allows you to add an object which you can provide bubbles and cancelable to initialise the event. I think you should also be able to provide data there so you don't have to modify it after the fact.


Typo: untility -> utility


imho "liste" is not the best name for that


Would it be possible to guarantee/document some specifics of the timeout behaviour? It would be useful to know the effective precision of the timeout (e.g. us, ms, s), if the actual delay time is strictly >= `timeout`, or <= `timeout`, or if this behaviour is different on different platforms.


Do we need #getSourceSection as well as #getEncapsulatingSourceSection? I have a feeling that from the client perspective only #getEncapsulatingSourceSection makes sense!

If the performance isn't affected, then I'd like to make #getSourceSection protected and #getEncapsulatingSourceSection final. Clients outside of languages would call just #getEncapsulatingSourceSection.


:clap:


+ FAB-placeholder.


Mise à jour année


@steven-borthwick I have updated the form styling. We already had the styling for text boxes, I just added styling for checkboxes. This is not published yet, but is merged with Master. Next time you make a pull request the changes will be integrated. Let me know if you have any questions.


It's asynchrony... sometimes things feel weird ;-)


shouldn't this be complemented with `jshint +W100`?


étrange comme bug, il intervient avec quel navigateur ?


ratio of coaches


lol what's this?


I think perhaps tone down the second sentence slightly.


Yes, ideally we would have 3 options:

- Strip `<` and `>` like you already did
- Escape `<` and `>` like as I added
- Remove full tags

The last options might come in two flavours as some might want to remove what is inside a tag like <script> or <pre> while others want to preserve that. Ideas?


- I'll have a look regarding the redirect, I tested it with ssl_redirect aktivated and it worked flawless
- A certificate overview is an idea, but with 0 priority currently.
- The settings are inherited, this happens since forever :P, so the certificate needs to be created as it is configured, I don't see this changing at all.


If a user gets this message, how will they know what to do? Is `cider-retest` a command?


113008903Provide-a-better-warning-message-for-a-triangular-window (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2057 of 2057 tests passed, 0 test warnings)

![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)


hhhhhhhh


What about a public version of ` pwg.getInfos` ?
`pwg.session.getStatus`  is fine too


Perhaps a bit clearer to change last sentence to: "If an unregistered setting is specified, the node will reject the setting during startup and fail to start."


Alexei


following arduino/Arduino#4501 , it could be interesting to add a check for file ownership like
```
user=`whoami`
owner=`find avrdude_bin -user $user`
script="chmod +x \"`dirname \"$0\"`/avrdude_bin\""
if [ x$owner == x ]; then
/usr/bin/osascript -e 'do shell script "$script" with administrator privileges'
else
$script
fi
```
need to test in on real OSX, based on http://stackoverflow.com/questions/10283062/getting-sudo-to-ask-for-password-via-the-gui


Even better, why not make this a `y-or-n` prompt and then actually run retest for them? :-)


I am not sure about this, this break the style on pre material phones by imposing the white icon on these phones.


да, заработало, но вот что не пойму..
Получается после подписи у меня в запросе

> <xades:SigningTime>2016-02-04T11:50:54.019+03:00</xades:SigningTime>

Но от них ответ приходит с

> <xades:SigningTime>2016-02-04T14:50:55.422+03:00</xades:SigningTime>

Где правда?
Или это уже косяки прокси класса..


Testing


fixed ;-)


не считаю это хорошим решением


Why would it? I used the same drawable redirection mechanism you used for other drawables as well.


Infact both old and new should work in hindsight.


Will the first option ever make sense? I can’t really see the use case for it. Agreed about the last option (although I think GitHub ate a part of your line :) )


#ifdef or #ifdefined()


А что тебя смущает? Ты отправил в 54 секунды, они тебе в 55 секунд ответ. То что они тебе отправляют они тоже ведь сообщения свои подписывают.


Есть хороший тег <nobr>


неразрывный пробел гораздо более правильнее решает проблему с висящими предлогом, так как если изменится размер выделенного блока, не будет насильного переноса, но связь со следующим словом сохранится


>The ACK, simply serves as a feedback from the user acknowledging receipt of the message. So I think it is independent of their jurisdictional responsibilities.

The way I see it is that the people in the EOC are not really aware of individuals, they are aware of locations.
They would like to see that each location has ACKed, or which one(s) if not.
Sure, we could just have a list of individuals on the Dashboard with the Green/Red 'responded or not' statuses, but this seems far less meaningful to me than having it be locations.


心凉，感觉你朝只能吃枣药丸了。
然后，非常感谢一直以来的陪伴，祝安好。


@4re
I think you might have to update the autogen and Makefile stuff..
libfftw3f-3.lib is no longer used, libfftw3-3.lib (double precision fftw) is now required..


обрати внимание на часы и временную зону


Should be fixed now thanks


Haha, well at least now we know GitHub's policy ;)

The first option does not make much sense to me, but if we remove that it's a breaking change.

Let's see what GitHub does with `<pre>` and `<script>` content:

<script>
alert('hello');
</script>

<pre>
some code
</pre>


@sergeylunev он проприетарный, не входит в стандарт и не рекомендуется к использованию)


Is this not indented intentionally or is it a mistake? When I indent the line, tests don't pass.

This gets compiled as

```js
runner.hooks[eventType + "Hooks"][0](sentData, function() {});
done();
```


This code is actually redundant. The class "toggle-add" is not used anywhere.


Interesting technique to replace the contents of a Sequence. (i.e. anything I would have come up would have been more complex)


Of course, we can make this transparent: user ACKs only for all jurisdictions & this ACKs for all locations...my reason to allow this to be distinct is to be clear that they are taking responsibility for each location...e.g. imagine the scenrio where the DB hasn't yet been updated & someone who was responsible for 2 districts is now only responsible for one since the other district has found a budget & managed to recruit someone. They should only ACK for their district until/unless they (or the EOC) can notify this other person. These may appear to be extreme edge-cases but to me they're exactly the kind of thing that will happen sooner rather than later.


Oops!  I didn't see the link that you included in the commit description.  Now that I have seen that, this all makes sense.  This is indeed a good change.  Sorry for the noise :-)


@sergeylunev про <nobr> не знал, спасибо.


In practice the `_generalTracksHp` should be the same for all eras (unless we rename the `generalTracks` collection), but ok, this way the code looks more generic.


@MadMed677 http://www.html-5.com/css/css-styles/css-style-properties/#white-space-property — ещё таким можно пользоваться.


You can drop `cms.untracked.bool` here if you want. The parameter is defined in `Validation.RecoTrack.MultiTrackValidator_cfi.multiTrackValidator`.


Типограф предлагает следующий вариант
```html
<p>РЕШАЕМ ЗАДАЧИ БИЗНЕСА С ПОМОЩЬЮ <nobr>ВЕБ-ТЕХНОЛОГИЙ</nobr></p>
```


I think it should throw an exception when resolving an instance which is bound in InstancePerUnitOfWork outside that scope?


you have made a spelling mistake!!


`publicIpAdress` should be `publicIpAddress`. This occurs in a few places.


Agreed. I doubt there will be many guarantees left in an intersection of five platforms, but I will look into it when this is merged. The documentation above should be considered a stub only. If the intersection is empty, I'll add platform-specific remarks (cf. glfwCreateWindow).


This one looks like a mixture of fastSim and phase1Pixel modifications applied for fastSim era. FastSim should have only the `dodEdxPlots` set, while the `label` change should be for phase1Pixel.


How does this work?


Thanks for suggested, but I put this directly on my local copy


Hi @isabellams this fix was not for label rotation, but for labels when the map canvas is rotated. What's your issue with labels rotation? Anyway questions like this are better suited for the users or developers mailing lists.


@sergeylunev к сожалению, фраза `ВЕБ-ТЕХНОЛОГИЙ` слишком длинная, и на ~ 300px просто не влезает на экран. Поэтому `<nobr>` решил не ставить


I had a look and successfully requested & installed a certificate with ssl_redirect enabled, seems like Let's Encrypt follows redirects ;)


Issue #576


CC'ing @bridiver  for review.


@luiges90 你看我的代码有问题吗？怎么事件不发生了，好像运行建筑，判断第一次false后，checkcondition为false


TeamCity pull requests :: netty [Build 1212](https://garage.netty.io/teamcity/viewLog.html?buildId=5413&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6409, ignored: 23 Build time: 00:43:10


what are you learning to do, in this case @abobrinha ?

I wonder how many moms in the world ever went to their kids github to post any comment! ;P


в принципе, это необязательно писать. практически во всех браузерах цвет по умолчанию будет черный


You may actually want to encourage usage of --upgrade here. If someone (like me) has already installed _some_ version of theano, current commend will not update them (as it only expects theano to exist), leading to strange errors with missing modules.


Stilig!


LOL


oops! I was relying on
autocomplete to minimize
typos and ended up cloning
the typo :D


It's intentional, a play on exclamation sign at the end of the macro name. Not sure if it's clear enough.


Apparently not in use, please remove.


Érthető, így viszont sokkal egyszerűbb a kód. Tettem típust a LL-nek, hogy látszódjon, hogy szándékos :)


So there are multiple possible levels of InstancePerUnitOfWork with the containers:
1. In the root scope (a normal `IContainer`)
2. In a child scope (after calling `container.BuildChildContainer()`)
3. In a nested child scope after repeating `2` any number of times

Ninject doesn't have a concept of child scopes, only different contextual scopes defined by objects.

It also appears that Ninject somehow shares configuration across instances of `StandardKernel`'s, so as soon as I take the base level context object out (currently the `StandardScopeCallbacks.Thread` object), then when the next set of acceptance tests start running they seem to configure bindings while the cleanup is still happening on the previous test and we occasionally end up with failing tests because a binding that should be there was cleaned up as part of the previous test.

TL;DR: The instance could be defined in the `IKernel`, but another instance of `IKernel` could mess with that binding unless we have an object to tie it to the actual context it's defined in.


> But the pattern '${HUNTER_SELF}/cmake/projects/${HUNTER_PACKAGE_NAME}' will also be used in project specific cmake files which calls the hunter_download function

Not in user's file which do `hunter_download` call (which is called by `hunter_add_package`) but in schemes, which are configured in the same scope, hence will be "visible" by `@...@` magic.

In short it will look like this:

```cmake
function(hunter_download)
  # ...
  set(HUNTER_PACKAGE_STUFF_DIR "${HUNTER_SELF}/cmake/projects/${HUNTER_PACKAGE_NAME}")

  configure_file("${scheme_in}" "${sheme_out}" @ONLY)
  # ...
endfunction()
```

```cmake
# scheme_in template file
set(local_script "@HUNTER_PACKAGE_STUFF_DIR@/scripts/run-me.cmake")

ExternalProject_Add(
  # ...
  COMMAND "${CMAKE_COMMAND}" -P "${local_script}"
  # ...
```


@timbussmann I am of course more than open to any suggestions :smile:


Same as the TaskReleased and TaskClaimed, passing the error message as parameter.


the number was taken from the c++ library. And no, there is no calibration value available in the EEPROM.

http://icwiki.physik.uni-bonn.de/svn/ModularReadoutSystem/host/trunk/HL/GPAC/HL_GPAC.h
...
#define CURRENT_LIMIT_GAIN  20  // hack -> calibrate!
...
Different boards probably have different calibration constants!


Да ты прав. Воспроизводится. Ща..


Boa!


ну по идее семантики этим тегом размечают то что "помечено на удаление"


sdfsfdsfdsdfsdf sdfssssssssssssss


Could you make this into a final class with a private constructor as off good practice for creating a constant class.


那一個判斷FALSE?


typo here


Missing capital letter to surname


ouch...


This commit breaks the ability to select a RabbitMQ version with $version. Reverting it will return use of that ability.


113008903Provide-a-better-warning-message-for-a-triangular-window (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2057 of 2057 tests passed, 0 test warnings)

![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)


Do you still  have the 1.7.21.6 version. That one would be compatible to the stable reelvdr.


associate ;)


Yes, please do.  Since they'd be below other new entries, I think the risk of introducing conflicts is minimal, and if it does, they'd be easy enough to resolve.


update


## TODOs
- [ ] Light intensity should decay linearly (or quadratically) over time, not suddenly disappear (optionally see iTween library for modifying properties over time)
- [ ] Rename "rigidBody" to "rigidbody" for consistency with other classes
- [ ] Create a new "PlayerFlare" class that allows the user to launch a flare upon key press
- [ ] Create a "wiggle" on the flare using sine/cosine function


in magbox.cpp and musescore.cpp it might be better to do this in a Default case of that switch


good


This does aboslutely nothing right now, so it should be removed.
NB I wasn't suggesting a 2nd setting...I was advocating for enhancing the  previous setting in a transparent way.
Moving beyond the simple True/False to True/False/<channel name>


Reminder: Add HomeRecentlyAddedInfoAlt to ovveriddes


As long as you guarantee that ```key``` is never ```null```, you don't have to catch ```NullPointerException```.  If you can guarantee that it isn't empty, you also don't have to catch ```IllegalArgumentException```.


How can you send a static map via twitter?
I assume you mean a URL which points to a static map?


I'd rather see a ```null``` return value indicate a missing property or invalid expansion than an exception.  Constructing exceptions is an expensive process.


good point...


Yes, this makes the multi-message slightly less likely to hit :)


And yes, ive figured out the emotes to github already :smiling_imp:


потому что у тебя получилась не _картинка в ссылке_, а _ссылка на картинку_


Just remove the ```public``` keyword to hide this class from other packages.


мне кажется тут английская маленькая L, а надо большую I (И)


сейчас, кажется, все ок!


Looks good to me. The JDK 7 code was:

```
JNIEXPORT void JNICALL
Java_java_lang_UNIXProcess_destroyProcess(JNIEnv *env, jobject junk, jint pid)
{
    kill(pid, SIGTERM);
}
```

and JDK 8 code is:

```
JNIEXPORT void JNICALL
Java_java_lang_UNIXProcess_destroyProcess(JNIEnv *env,
                                          jobject junk,
                                          jint pid,
                                          jboolean force)
{
    int sig = (force == JNI_TRUE) ? SIGKILL : SIGTERM;
    kill(pid, sig);
}
```

So, the value of `false` being sent is also correct.


idCount
это называется Camel Case


There's no local state here, so this method can be ```static```.


Looks good to me, +1


@OHerasym, are needed spaces here and 2 lines before?


magic number


@luiges90 public bool checkConditions(Architecture a)


Ok, I'll look into this more deeply when I have time. But honestly I will not if there's no hope that my PR are merged. Do you know if that's because it's too hacky or because @assaf don't have time enough?

Another consideration is that I couldn't upgrade jsdom to version 7.x. because of XHR. But I suspect there are other hacks in zombie which are useless / harmfull with latest jsdom. Would you like to help sorting this out?

Thanks for your comments anyway :-)


если ты сделаешь итерацию по массиву от 1 до массив.size то тебе не нужны будут магические числа


то же самое, что с картинкой выше


does this work ? You'll need to inherit multilib-build to have MULTILIB_USEDEP defined I think

also, please make it depend on >=media-sound/jack-audio-connection-kit-0.121.3-r1 to ensure multilib flags are defined in jack itself too (in case user is on an old system and hasn't updated it)


There's already a PR pending to fix this, thanks.


This will need to be a privileged action, otherwise there could be problems when modules are lazily loaded (unless it can be shown that all possible incoming call paths are already privileged, which might well be the case).


Could you describe the issues that you had with that?

After I catch up with myself, I'll send a PR to move to Container Travis which does not have a Trusty variant (yet?).

If the built in version is ill in a certain aspect, we'll need to roll our own. Does perhaps PERL have a similar user mode package management to Python (CPAN)?


как хочешь. лучше не прописывай, сделаем их через CSS


hi


Yeah :D


This will cause reset on REQUEST_RESET_NONE, too.


Definitely!


update


See below about privileged actions - also, this value probably ought to be cached in a static field anyway.  Or just use ```File.separatorChar``` instead.


Unrelated change? (Should be probably in a different/separate commit.)


just put proaudio project there I'd say (same as jack1), I don't expect you'll have anything to maintain for the virtual :)


Do not rely on implicit enum-> int conversion. Compare to REQUEST_RESET_NONE


@bogdanpetru let's call this `notifyResize`. Do you think it's a better name?


typo


There is a typo in the commit message (backtick at the end of the first line).


Compare != REQUEST_RESET_NONE please.


Well, LOCAL_PATH is $(call my-dir). You get typo again


@nllong Wow. Is this what the Dockerites decided was the thing to do for Java 8? Thanks so much for chasing this down!


épp észrevettem én is :)


I tested this. And it works. Any plans to merge this soon?


[ConfigPersist] is what causes settings to be saved and loaded to the config, you can't remove this!  What problems are you having?  I'm guessing there's another place that handles column stuff that you are missing.


@radubrehar Also for the component ?
`ResizeNotifier -> NotifyResize`


oké, próbáld te is megoldani :)


ok...did that: https://github.com/ericfont/MuseScore/commit/14e1bd67f5387a8bc98e85950064afe41777e0e9

now I'll squash and make PR.


Usually non-copyable means that if we will try to copy non-copyable class. then compilation will fail... I don't think we need this. I think it's the job of the owner of the pimpl to prevent copy-and-assing. so I propose to drop this template. Plus I don't think we have use cases for this.


Does it throws the first _bad_ field, or does it throws a list of _bad_ fields?


Очепятка) Декоративное.


meggondolom még, kösz, nagyon jó hozzászólások :)


Reminder: Re-add <visible>!ControlGroup(9004).HasFocus()</visible> to playliststats


Csábított a gondolat, hogy megoldjam én is funkcionálisan, Scala-ban, de ma ebédidőben nem volt időm. Talán később.


Closes #22


cc @AlbertoFuente @makotot @charbelrami for new translations. Thanks


не понял, префразируй


done


@malirod we have no `static_assert` so I cannot prevent this code compilation. If you insist we can remove this assigner, but we have such cases. For example `RWLock`


I don't think either of these are quite right, as CMake has some specific features for this.
1. There's a variable `CMAKE_COMPILER_IS_GNUCXX` that should be used instead of `CMAKE_CXX_COMPILER_ID`
2. There's also `CMAKE_CXX_STANDARD` and `CMAKE_CXX_STANDARD_REQUIRED` which is supposed to handle differences between compilers

So a simple, unconditional:
```
set(CMAKE_CXX_STANDARD 14)
```
should do. I tried this on MSVC and G++ v5.2.1 but I don't have any earlier compilers to test.

Finally - do we need C++14 or is C++11 enough, since I think C++14 compliance is still partial (for MSVC at least, though I'm prepared to be wrong about that!)


ааа прочита ниже


I am having difficulty starting your node-sones-http-api, error below, any help would be appreciated.
node server.js

module.js:340
    throw err;
          ^
Error: Cannot find module 'sonos-discovery'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/home/rob/node-sonos-http-api/server.js:4:22)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)

npm ERR! sonos-http-api@0.17.1 start: `node server.js`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the sonos-http-api@0.17.1 start script.
npm ERR! This is most likely a problem with the sonos-http-api package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node server.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls sonos-http-api
npm ERR! There is likely additional logging output above.
npm ERR! System Linux 4.2.0-16-generic
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "start"
npm ERR! cwd /home/rob/node-sonos-http-api
npm ERR! node -v v0.10.25
npm ERR! npm -v 1.4.21
npm ERR! code ELIFECYCLE
npm WARN This failure might be due to the use of legacy binary "node"
npm WARN For further explanations, please read
/usr/share/doc/nodejs/README.Debian

npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/rob/node-sonos-http-api/npm-debug.log
npm ERR! not ok code 0


@bogdanpetru ok, we should be constant: either `resizeNotifier` and `ResizeNotifier` or `notifyResize` and `NotifyResize`. Which do you think it's better? PS: the repo&module name is `notify-resize`, maybe it's good to favor this?


Either that or I forgot to allow --params to key a string with key value pairs in it :p


Thanks! @alexsaveliev warned me about gofmt, but incidentally i forgot to run it.


Hi

Finally my editor now recognize characters after I change the encoding to ISO 8859-1 https://github.com/practiceprovider/openemr/commit/0d82bf7fa397fb94d09f1520a165c3b09391017b


Eine Exception wäre möglich bei `getCountry()` und ` getNumber()`


Until further notice `OMIT_V2_CODE` must not be set.


I think it's redundant. Without static asserts it's behavior is not expected.  RWLock is already non-copyable.


Desde el mismo PC:

IP Player 1 --> 127.0.0.1-10.0.1.253|10.0.1.1|77.241.120.1|10.0.0.36|10.175.242.77|
IP Player 2 --> 127.0.0.1-10.0.1.253|10.0.1.1|213.248.93.21|77.241.120.1|10.0.0.36|


Hi, could you please tell me which is RadioQ? Thanks!!!


hi, no problem

two additional thing,which I didn't verify till now:
if you take in a table form element type = Urlfile, then you get an error when building the module
if you take in a table fieldtype = URL, then you get an error when installing the module (check mysql.sql: as I know, there is no datatyp URL in mysql)


It looks like this is the commit which should have been checkpoint 7.  Code looks good though, good work!


Why this parameter is not documented in sonar.properties ?


From what I can tell, they don't interact.  I also contemplated just using a shadow socket instead of all of the individual fields, but the OS can actually change the values, e.g. when you call setsockopt on Linux with a buffer size, it doubles it, and that doubled value is what's returned if you then call getsockopt for that same buffer size, so I can't reliably use a shadow socket as the store for these properties.  In the end this seemed like a reasonable compromise.


Solucionado:

`int boxCount = _addressManager.get(address);
			if (boxCount > 1)
			{
				_addressManager.put(address, boxCount - 1);
			}
			else
			{
				_addressManager.remove(address);
			}`


I am not sure if adding this option to all coap servers created by soletta is the right way to go. In commit comment you are stating that this will add this option to coap sockets only for well known ports. But you are adding for all sockets. Won't this affect other users of coap servers, as LWM2M, for example?
Or it is really expected to have this flag set for everyone?


Actually, @saleem-latif made this change in one his [PR SOL-1266](https://github.com/edx/edx-platform/pull/10075/files) as he had added this to support his change [HERE](https://github.com/edx/edx-platform/blob/master/cms/templates/js/certificate-details.underscore#L40)
`display_number_with_default`return `display_coursenumber` if present else returns `course number`. @saleem-latif kindly share your thoughts on it if I am missing something.


Corriste el task que te mandé?

Nicolás Kipreos
---

2016-02-04 0:25 GMT-03:00 Juan Pablo Baeza <notifications@github.com>:

> Esta wea nos esta dejando la caga, al parecer como no existen
> money_sources asociados se cae y no hace busquedas, no agrega nuevos items
> ni genera los pagos. Es normal que no hayan money_sources asociados para
> las ventas nuevas.
> Hay que ver como manejarlo pero no cacho como hacelo en la vista...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nkipreos/pushvendor/commit/a84c7f599c6327d55d979546fdf4520b7c64e149#commitcomment-15876135>
> .
>


Pay attention to swig!
To my on swig starting from 3.0.7 version is errors in created enygma.py
https://github.com/Taapat/vuplus-fulan-openpli-oe-core/commit/e3f44f138a8f01fc225dd9e4a163f5e0d56f813f
For example it no create working eDVBFrontendParameters() in enigma.py


I handle a list, but I didn't test the behaviour of Jackson


100644 → 100755 ??


@Halama nikde som to popsane nenasiel. Pozeram, ze to APIcko vrati ID tej notifikace, takze to prerobim na cykly.


@AMikhniev , same codestyle in all file.


You are not 100% correct. If there is no element that matches in vector that matches meta->key, 'r' will not be overridden . But anyway, I didn't understand what is the reason of this code, so it is better to recheck that.


@Mpdreamz this also needs to be applied in `WriteAsync()`


That was the thing. On the build I submitted, StoryGame was deleted. For all intents and purposes, it should not even be in that branch.

Sent from my iPhone

> On Feb 3, 2016, at 8:38 PM, Jessica Lohse <notifications@github.com> wrote:
>
> Fix errors thrown by Travis
>
> @jtmcginnis was there a reason you removed the scene from the MapView? If so, the methods in StoryGame won't work because they still access it.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/spring-studio-2016/storygame/commit/bf7f0372467ae5e2a453c4cd18ed22cf25a717db


Are both conditions required?
Are there any cases when 1st condition (`Visibilities.isPrivate(...)`) is true,
but the 2nd (`getVisibilityAccessFlag(descriptor) != Opcodes.ACC_PRIVATE`) is false?


-Added std input
-enhanced boolean logic (and, or are now working)
-enhanced while loop parsing -> other brace like structures are allowed in each other
-added some words


:+1: ty for catching that


Removed.


Test name might be a bit misleading, since parts are not empty.


@TomokoKuroki Upload your trainer's form code to pastebin or pastie, so I can take a look at it.
@duke12468 TomokoKuroki is correct, you would need a valid korean SSN to create an account. That being stated, there are many online services that may assist you in such -- lookup nexon korea account creation for more info.


Stimmt bei getNumber könnte der String null sein, aber was soll bei getCountry sein? Das Array und die Map, die ich benutze sind auf jeden Fall initialisiert. Die können nie null sein.


no thanks


Done.


Sorry for the error. I fixed it with your suggestion! Thanks!


I know :-) But I get tired of remembering that JSON does not so I just do it anyways


ahh yes , thanks :)


Great stuff, congrats!


The format of name="html_howto" and name="html_about" looks kind of messed up. Is that an Issue of Weblate? Best wishes & thank you very much for contributing!!


I think it's time for us to have a separate `cider-test` keymap. This would both free up some keybinding and simplify the lives of the users in terms of test discovery.


I have been speaking with codesnake, he will submit a PR with a different patch that leads to the same behavior (working on both Linux and Android), I have tested it on Android and with 1080p it works fine, I have no 4k tv to try here


Good thought!


Propose to use there the word "fix". With meaning that this is planned fix of the defect. And use word "hotfix" in sense which described in #L19


Why not simply reverse the branches and drop the `not`?


hello


Same here - `if not` usually doesn't read as nice as `if`.


I thought it was no additional information (just the company name) as the default, That is why I chose 0 , My oversight. Thanks


k keep rocking


cool


From the docs I read `done`  will handle the error.


Muestra un botón que, al hacer click sobre él, permite borrar dicha entrada del listado.


I think it's best to favour `seq.el` functions over functions like `delq` and `mapcar`, as their usage makes the code more clear (especially to Clojure devs).


There could be a lot more added, But I thought I would be getting into over kill 8^)


@OHerasym, search in both docs links by next patter "<a href". This is http link format. Replace it with markdown.


Φίλε έχεις διαθέσιμη και την MPI εκδοχή του κώδικα; Ευχαριστώ


Yes, +1 for the Log class - I am an old school procedural programmer, and Lua opened me up to the concept of generators that yield values - so making everything an object in Python is still new to me.

I put up the skeleton of the code hoping this would happen, which is that a more Pythonic way would emerge. I still need to add the capability to pull commands in temporal order and apply them to multiple motor ports, and then extend the concept to sensors.

Again, the subtle beauty of "everything is a file" helps us to create test methods and suites that have general application.


Isn't failing better than non-passing?


Please, release the reference count in `storaged_linux_glusterfs_brick_object_dispose ()` (as per [Object destruction](https://developer.gnome.org/gobject/stable/howto-gobject-destruction.html)).


I separated them into individual tests.


It would actually be quite easy to keep track of the tile source count, to avoid the count calculation altogether. @tschaub, you may want to cherry pick 5e766e4 from my [`tilesource-count` branch](https://github.com/ahocevar/ol3/tree/tilesource-count).


Evitamos duplicar código (todavía no explicaremos cómo hacer 'refactoring' y crear clases)


Removed.


`(concat ns "/" var)` is repeated a few times - it should be a function I guess.


My copy have this code:

`$t = 1;
$modversion['tables'][$t] = 'tdmcreate_settings'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_modules'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_tables'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_fields'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_languages'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_fieldtype'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_fieldattributes'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_fieldnull'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_fieldkey'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_fieldelements'; ++$t;
$modversion['tables'][$t] = 'tdmcreate_morefiles';
unset($t);`


Just for my information, why deploying the JDBC driver in deploy dir is no more required ?


It doesn't add any value.  The delegate ctor will log enter and exit around the work actually being done... this just logs an extra enter and exit for no good reason.


Not sure for this, v8 has changed the restart frame api https://github.com/v8/v8/commit/84f8e1af76e06f645256055387cdd2f5902917cf and it crashes in my environment. Now I'm using a custom built version of node and this commit aims to fit for it. Maybe I should not include this commit in the PR :P


You are running a very old version of node and npm, that is probably why. Install at least node 4+ and npm 2+ and try the instructions again.


I'm working since yesterday to refactor the files UserXoopsVersion.php


small text correction:

Win/Linux:
Alt+F4 = exits
F5 = reloads


Обычно на той же строке `else` пишут.


Obtiene el parámetro **ID** de la url (http://localhost:9000/**ID**=NNNNN)


Sorry, no. This is still broken.

Line 508 in S3/FileLists.py
        ## check md5
        compare_md5 = 'md5' in cfg.sync_checks
        # Multipart-uploaded files don't have a valid md5 sum - it ends with "...-nn"
        if compare_md5:
            if (src_remote == True and '-' in src_list[file]['md5']) or (dst_remote == True and '-' in dst_list[file]['md5']):
                compare_md5 = False
                info(u"disabled md5 check for %s" % file)

Therefore, sync will still ignore the md5 sums.


@ishantvivek storage_list still being use in 6.0


Failing or erring = non-passing. It's a bit pedantic, granted. I could change this if you'd rather use the term "fail" less specifically.


what


Thanks, I'll check it out on Rosie.


@malirod, since the issue you have mentioned are not critial one propso to create github ticket for this. Since I can't change this code in the nearest feature


References #385


is there anything else left to do, or can it pulled in?


> two additional thing,which I didn't verify till now:
> if you take in a table form element type = Urlfile, then you get an error when building the module
> if you take in a table fieldtype = URL, then you get an error when installing the module (check mysql.sql: as I know, there is no datatyp URL in mysql)

This SqlFile.php files should be checked and rebuilt better


Tiens? Le paramètre a disparu.
C'est voulu ou ça fait partie des tests pour débugger passport?


I don't think you need the `exit_callback`, that's just useful for getting the status code of the started process. The `output_done_callback` is called when the started process closes `stdout` and `stderr` and is all that you need here.

(A process can close stdout/stderr and continue to run and a process could start a child process and exit. This is why these different callbacks exist. You only care about "end of output" here.)


@tvsonar typo in `cannon`


Class SlackMessage is missing a docblock.


SlackMessage::__construct is missing a docblock.


SlackMessage::getPayload is missing a docblock.


Class HipchatMessage is missing a docblock.


HipchatMessage::__construct is missing a docblock.


HipchatMessage::getPayload is missing a docblock.


I think it would be much clearer if it was split onto 4 separate lines:

```
  The calculator gives an estimate. The exact amount the Child Maintenance Service calculates might be different.
  <% if paying_or_receiving == 'receive' %>
    For example, if the paying parent also supports other children, the amount you receive may be less than this.
  <% end %>
```

This change would not affect the whitespace visible in the HTML.


Woho, thanks! :)


It's best if you actually read the cell value as a Boolean and handle any display formatting in your own code logic.... If MS Excel stores the value of a cell as a Boolean, it interprets that through the MS Excel GUI, not by actually changing the value and datatype of the cell. Doing a var_dump() of the value would show it as a Boolean, and indicate whether the value is true or false; echoing the value in PHP is rather different, as outlined in the question that you've linked.

From your userland code, you can do something like
```php
$cellValue = $objPHPExcel->getActiveSheet()->getCell('A1')->getValue();
if (is_bool($cellValue)) {
    echo 'Cell A1 contains a Boolean value of ', ($cellValue ? 'TRUE' : 'FALSE'), PHP_EOL;
}
```
if you want to display a word like `true` or `false`


Should be 0x43210008, is not it?


IMO this is the wrong header. It should be `<string.h>` (or even better `<cstring>` and use the functions with the `std` prefix).


Could you make this so that a new command is only started after the old one finishes?
For example like this:
```lua
local start_timer, timer_callback
start_timer = function()
    gears.timer.start_new(timeout, timer_callback)
end
timer_callback = function()
    easy_async_spawn(command, function(stdout, stderr)
        start_timer()
        callback(w, stdout, stderr)
    end)
end

-- Start the first update
timer_callback()
```


The changes to this line feel as if they're unrelated to the rest of the changes in this commit. It would be nicer to split them out into a separate commit.


Ok, now it's clear to me! Originally, I was thinking that the EOC would simply list all those who have not ACK-ed and then attempt contact them in another way of resend those users an alert again.

Right, so I see the other angle. The example might be a certain village has 3 CERT members who take turns being responsible for receiving the alerts and then activating their community protocols. Only the person designated to be on duty would need to acknowledge for their village and not all 3 of them.

Good one! I was oblivious to this idea,.


@pestOO, I don't quite understand your comment. But I'll try to guess. You propose to use the return just int result and signal error with -1 as it's done in actually socket api. The problem in difference in uch api on win and linux. On linux it returns ssize_t, on windows the result is just int. And there is no ssize_t type on windows. So to unify return param for both was used such approach with bool result. Actually this logic was originally introduced in Pipes quite long ago. So to unify interfaces of the Pipe ans sockets and  get rid of ssize_t I've used the same approach. Personally I think that it's more that enough to have `int32_t`  as the result type on all platforms. Since usual tcp buffer is 4096. But I think this should be done for both Popes and Sockets at the same time


Yes.. anything other than SafeFreeSslCredentials is a bug. However, we couldn't figure out a simple way to fix calling code. In Windows case, the same SafeFreeCredentials is used for SslStream and NegotiateStream. In case of Unix, the SafeFreeCredentials is of two types.


@pestOO, please double check. there is **already** virtual dtor


canol oil?


As far as I understand `b` string prefix does nothing on python 2. Is this condition necessary?


Yes, I've already mentioned it to Sebastien (and I'm working on it - I'd like to be able to reproduce the original bug...).


I think just having the plot is fine, no need to deal with R objects and possible confidentiality issues that way.


@malirod , f0a7213, Done


Ok.


I'm wondering... should we use some `Logger` here?


driver = Selenium::WebDriver.for :chrome
Wait = new.Wait::Selenium::WebDriver.send_keys(:ID => 'login-box')


@pestOO, If you really insist I'll add doc to the methods. But since there is not strict rule to write doc and we are limited in time I propose to skip this.


This overrides default behavior for https://www.python.org/dev/peps/pep-0476/
which gives the following error when using unverified certificates
winrm.exceptions.WinRMTransportError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program **does not** include `main()` function or "ifmain" statement (it should not include these per my instructions) (1 pt)
- [ ] Program functions as expected (1 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program correctly demonstrates the primary reason to use `main()` function and "ifmain" statement (2 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program includes a function to accomplish 1, 2, & 3 (1 pt)
- [ ] Function returns unmodified `π` and prints it (1 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program includes a function to approximate *e* (1 pt)
- [ ] Program returns the approximate value of *e* and prints it (1 pt)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program is structured and functions as requested (2 pts)


I like this. We should make sure we do it everywhere we can.


//AN arduino controller for a car pre-heater that converts a RC ON/OFF signal to a 15m pulse that resets if the battery voltage drops to low or the RC is set to OFF.


Do we really want to do this? We've never had `LocalHost` as a class in the GPI before so we should think before adding it.


@vishnumohanm Hmm... I wasn't aware of that. We're not using 2.6 any more in production and I don't use Vagrant.
The whole point of pipstrap is to make sure you get pip8 installed. If you can comment out the `python tools/pipstrap.py` and replace it with `pip install -U pip`, does the rest of things work?


You are welcome, Håkan! :-)


Using the term “reducer” here is wrong — it has a very specific meaning in Juttle. I’d name the message e.g. `RT-POINTS-OUT-OF-ORDER` and the parameter `proc` (which would be consistent with other similar messages).