Что это за хуйня


you did promise to rermove those files didn't you?


https://github.com/golang/go/wiki/CodeReviewComments#variable-names
"Variable names in Go should be short rather than long. This is especially true for local variables with limited scope. Prefer c to lineCount. Prefer i to sliceIndex."


Remove first `\` ;)


Looks great overall! Just a few minor nitpicks.


unneeded line now we have pytest


unneeded main method now we have pytest


@dietmarw Do you know if there is something like %var supported in OMEdit for variables?


TeamCity Civic Assembly :: Build all [Build 39](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1050&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


can you elaborate on why you reverted these changes please?


Are you sure? I've been using these readers lately (SparsePCReader and LibSVMBinaryReader) without getting an exception.
DSSMReader is deprecated.


Funny thing is that they have it as TODO one line below :-). So this could get :+1: even from D. upstream.


TeamCity Civic Assembly :: Build all [Build 39](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1050&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new); exit code 123) Build time: 00:00:22


TeamCity Civic Assembly :: Build all [Build 40](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1052&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


This is no longer needed, can you remove?


Yes. I tested both php files and could not replicate any issues.


That's really extreme, but anyway, I'll check that.


@ssumpf : please check the authorships and dates of all new files again. Apart from the comments above I'm fine with the commit. Sorry for the inconvenience!


Sorry if this is stupid question, but why not just inherit DragonBase${Anything} from EventEmitter? especially since this is a class too. I don't think "fragile base class" applies here.
Other possibility is use decorators for mixing in, there's beautiful library for that: https://github.com/jayphelps/core-decorators.js#mixin-alias-mixins


yep, apologies it shouldn't have been in. Now removed,


commit message `create` -> `Create`


what the fuck


This should have been in the previous commit


@ssumpf I see, you need it now, but please rename it to invalidate_tlb_by_pid so it is consistent with the ARM implementation.


No, for something like that you need `DynamicSelect`. You can not refer to a variable via `%var` since which value should be displayed of the changing variable. That's also why it is not part of the specification (chapter 18.6.5.5 Text).


you don't use the module argument at all! Why have it?


good


Fixed here and in all other places. Thanks.


Thanks, was able to run a full delorean build :+1:


please close the bracket in the line above (I had that in my diff btw. :D)


jeeeej! :+1:  :D


An animation isn't a Timer; it won't reset the value to the start value when running is switch to true.


I'd like to reiterate that this may require #ifdefs as I've only tested this on the Teensy 3.2.


I think you should use the same line as line 8 here so the example really assures that it doesn't do any whitespace mangliung but actually cares about the imports


Should we really clear from if an error occurs? If we clear from, we will lose items not copied yet.


looking good


494-501 and 507-513 is the same function copied&pasted twice, please refactor


OK - thanks.


Again, a tweak for "user psychology".  The workflow should be that users never delete their bt objects.  We are only giving them a delet function because it is annoying having "mistakes" hanging around.  So I don't want to give the impression that yamls are being backed up or anything.

People are used to there being a "trash" folder from windows, mac and linux.  So let's create at startup a hidden directory under xpdUser, ``xpdUser/.trash``.  We can put the deleted yamls in there.

Then I think we should go back to ``delete`` rather than ``remove`` as it is more final sounding.

The workflow would never be ``bt.delete(b.get(3))``.  It would either be
```
bt.list()
bt.delete(3)
```
or it would be
```
bt.list()
delete(bt.get(3))
```
I can see arguments either way.  In the latter version the workflow is always
```
bt.list()
do_something_with(bt.get(3))
```
and now the "do something" is deleting.  Also, there is no asymmetry between creating and deleting objects.  In the former approach we have a ``bt.delete()`` method but we don't have a ``bt.create()`` method.

So many of the kind of "let's use logic to make things consistent" arguments argue for ``delete(bt.get(3))``.  But I somehow feel on the other hand a gut feeling that it will make more sense to users that they can ``bt.get`` an object but they can also ``bt.delete`` it, even if they can't ``bt.create`` it.  Another slight hesitation here is that, as already mentioned, I don't want users deleting objects.  I don't want them to think the workflow is to create,use,delete, so the delete method is a kind of hidden method, there if they need it but not really advertised.  One way to do that is to actually not make it on the same level in the API as ``get``.  They are supposed to use ``get`` and ``list`` perhaps 20 or 100 times per day, but ``delete`` hardly ever.

We also have a problem with the ``delete`` method that it will definitely change around the position in the list of all the other objects, which I am sure will cause confusion.  I am already sensing that it is helpful to people if things don't change their position in the list too much (or at all).  In that case, the behavior should probably be ``hide`` rather than ``delete``, so the yaml file stays there, but is not displayed when the the list is returned.  This might be a bit clumsy to implement.

Decision: let's move this issue/pull request to v1.0 and wait for users to request it before we deliver it.


@ssumpf : well you can define it in base-hw anyway, but for all I care leave it in this state. But then you should think of moving e.g.: the Cpu_state class to base consequently too, as we also define that for all architectures within base.


Sorry, my fault, done.


Now I'm lost at all: inputstream.mpd is for me like sprintf and has some VAArgs, wich can be set and wich cannot be set.
This is my personal meaning and the only reasonable concept for this issue here. Assume you have an sprintf where you are forced to put always 32 parameters regardless if you know that only two placeholders...

But I'm relaxed on this issue, if you want to make them fixed, this is no problem for me (because I know)
Maybe there will be questions from other video addon developers in the future why something has to be passed wich is not present. Then we / you have to answer it. I'm honest: I'm to lazy for it. Better removing one line of code than answering 20 mails why it is like it is.....


Removed.


Anyoung,

My fork is behind, how can I update it directly from github ?

Thank you


@m-stein: Alright, I will remove it and hope your memory is zeroed out. In my opinion, it is good to start with a clean register set (think covert channels), but that is just my opinion.


Good luck with LTC!


That's a good point - `is_` is kind of redundant thanks to the `?` at the end of the method name.  So maybe `cluster_remote_subset?` or `remote_subset_of_cluster?` is better.


Rather change this to

````C++
    _dev->set_speed(AP_HAL::Device::SPEED_HIGH);
    _dev->get_semaphore()->give();

    if (tries == 5) {
        hal.console->println("Failed to boot MPU6000 5 times");
        return false;
    }

    return true;
````
Because it makes easier to read but this can be fixed later.


Merged and deployed.


I don't know, but it seems Android Studio is acting up for me. It builds, then fails the builds.
I've reverted completely till I can figure what's wrong and make the upgrade successfully.


Could you please remove this pacage from the pins file, too?


What you can do is, in screenShooterGradient:
onVisibleChanged: if(visible) {
  screenShooterGradient.opacity = 1;
}
and leave the rest as you've written


close #22


annoyingly it can't be done for line charts

I'm thinking for charts with more than about 5 lines we could suppress the google charts legend and make our own widget - we have full information about the text and allocation of colours so it's totally doable


@bpilorget I had already updated the file, but not signaled that in this issue. Or is there something else I need to do?


alright this test it totally weird, I am aware that I wrote the same kind of test for `coala-ci` but I don't understand why the hell it works.


issue #360


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


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


Could we extract this to something like 'isRelevantProjectChange(IJavaProject)?
It would be inline with the other method and could be reused in AbstractJavaProjectsState, too.
Also isAttachmentChangeOnly wouldn't need to be exposed.


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


LGTM


Not related to this change, but why does this actually has to be done in a `finally` branch? In case of an exception we will anyways get a new actor-instance, right?


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 146](https://teamcity.labkey.org/viewLog.html?buildId=302823&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 10 (3 new), passed: 40 Build time: 01:31:54
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

```


I will close this pull request because it has conflicts with the current ionize's repository status.
Btw, this mod is added to the master.


Hee hee. Just making sure you're paying attention. :-)


I eposter er det viktig å være obs på at whitespace rendres as-is. Så for å få ønsket formatering må du dessverre legge alt på ei linje. Sendgrid!


Could you put a comment about why changing the semaphore lock from blocking to non-blocking?


Does it not make sense for this to be a static method?


It seems that this can be removed.


@ehmry thanks for your groundwork! I just try to limit patches to a port if possible. Moreover, I thought it's best to cover the library internal "libc" headers in a private include directory, that's why I've changed it like this.


I removed the "format":"uid" from the schema to make it more permissive. This caused the consumer.uuid column to be a varchar(4096). Should we manually make it smaller?


nullptr


Yep. It's now the only user of the label. I'll change it.


The view "admin/views/article/list" includes this :
	$select_views_class = is_null($all_article_types) ? 'w180' : 'w110';

So defining $all_article_types to array() is not correct.


Go back


David, it more sounds like any invocation of pick_ipc_buffer() passes in a "max" that is too large for you?


I am not used to calling global variables with capital letters. They could be mistaken for macros.


Этот код потом был удалён


Unit test in catkin work space is `-catkin_make run_test`

Check the catkin [wiki](http://docs.ros.org/hydro/api/catkin/html/howto/format2/run_tests.html)


Toto nie je najlepsi sposob ako customizovat subory vo sylednku (ak by ich pri rekompilacii prepisal nejaky balik atd)
Najlepsie je vytvorit overlay a povedat buildrootu kde je, on ho potom vzdy pekne prida:
https://buildroot.org/downloads/manual/manual.html#rootfs-custom


И этот весь хлам с листенерами тоже видимо закомментить надо и тудушку ляпнуть.


Where's reject bro?


@ehmry moreover, hardly influenced by cnuke, I was thinking about removing the hard coupling of libjitterentropy, and use a '/dev/random' file from the VFS instead, so we can either use jitterentropy as a VFS-plugin, or something different. But this is something we can change, when a new randomness source is available.


Here they obviously can't be mistaken for macros but I find this coding style a bit weird. Is it something you've seen used in C++ projects?


es6 bro?


Same comment.


We try to avoid using `overrideDerivation` in nixpkgs. Rather, why not just enable it in the package itself by default?


My bad, had the wrong Livre branch.


unack, wrong indentation, `or` should be aligned like all other continuations


You need to add "chai-as-promised" to project dependencies. http://chaijs.com/plugins/chai-as-promised

Then you can just return the Promise from "before" block and:

1) Have no need to call "done"
2) Have a meaningful message if the promise is rejected.

Refer to my older tests for an example.


And trigger_ip_interrupt and init_cpu_local as well.


TeamCity Civic Assembly :: Build all [Build 38](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1049&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new); exit code 1 Build time: 00:21:21


here too and probably in other cases


Код постоянно эволюционировал, всё старое барахло в конце было выпилено и заменено на новое, модное молодёжное.


@skalk: Okay. A short `find base/src/lib/startup/spec` shows:
./x86_64
./x86_64/crt0.s
./x86_32
./x86_32/crt0.s
./riscv
./riscv/crt0.s
./arm
./arm/crt0.s

So, why move it?

Does the Cpu_state thing belong to this issue? And don't see it here.


*scenario


that's a neat one :) like it


Тудушка, кстати, есть вверху


Why do we give out cwd ? wouldnt that be the same as `path` ?


We should somewhere in this header document why the PIC functionality isn't implemented (yet?).


@ssumpf no it isn't, you are right. I was reading it commit-wise, and didn't noticed it is filled with life in a later commit. Sorry for bothering you.


ок, дальше в коммитах?


Invocation service does not need to be aware of client, does it?


Oh, sorry,  haven't noticed this commit. I think this should be squashed into the first one.


TeamCity Civic Assembly :: Build all [Build 40](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1052&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 171 (171 new), passed: 1, ignored: 4 Build time: 00:21:36
### Failed tests
```
Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blog categories tests: Should delete blog category new blog category 1455273667375: <no details avaliable>

Blogs tests: Should navigate to Blog page: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

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


doesn't this need `valid=False`? :/


I'd love to see the Register framework here :)


Could we put in the full form here ? It'd be easier to understand

`changr_directory` isnt too long


@susumakusu the error suggests that your `/home/ElunaTrinityWotlk/src/LuaEngine` folder is empty and means you likely did not use the `git submodule init` and `git submodule update` commands. Try reading the installation steps: https://github.com/ElunaLuaEngine/Eluna/blob/master/docs/INSTALL.md


Why is this in Invocation service and not in the client class?

This creates an unnecessary coupling between the components.


The two directory names can have different length.


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


Да. Вкратце, изменения в коде старого загрузчика можно пропустить, чтобы не тратить время.


`git commit --amend` next time ;)


Его наверх к импорту остальных jni импортов надо


oops, debug string


this looks all wrong, you're replaceing a lot of Invalid invocations with Valid without setting valid=False


The test is good


unack that is


@deathbaba, подскажи где посмотреть правильный порядок импортов?


Мы вроде using проедпочитаем typedef-ам. @deathbaba, подскажешь так ли это?


You don't use allocated GID at all from what I see. You need to add `extraGroups`, mention it as the default group of the user, and do `chown -R ${pdnsdUser}:${pdnsdGroup}`.


ack


Думаю, прикручу к FOpenBaseFile/FOpenFile тогда.


Hm, this is the kind of thing our users will bump into a lot I guess. The original code was actually saying what the user intention was: take all components with sources, and do something with it. Now the rule's inputs are less well-defined, and it's also confusing for the user why one works and the other one doesn't.

A better approach would be to support the user intention in a better way here, say:

```groovy
    @Mutate
     void configurePrefixHeaderGenerationTasks(
         TaskContainer tasks,
         @All Collection<SourceComponentSpec> components
     ) { ... }
```

I mean, in this case we have `@BinaryTasks` as well (although they don't support non-build tasks ATM). Maybe we could generalize them in the future into:

```groovy
    @Tasks
     void configurePrefixHeaderGenerationTasks(
         TaskFactory<Task> taskFactory,
         SourceComponentSpec component
     ) { ... }
```

In any case, addressing top-level containers via `ModelMap<SomethingSomething>` doesn't look like an extensible way to write plugins.


That's my new repository: https://github.com/Gael-de-Sailly/lib_settings that allows to save per-world settings in a single file, `world.conf`.
That will not be merged until libsettings will be documented on github and on the forum. That was precisely made to test libsettings, and it works, so I saved it as a commit, but I don't expect to merge it now, as long as libsettings is not documented on the forum.
And it's optional.


Surprised by this version number v1.0.3-RC2. Seven days ago it was v1.2.0-RC5


Я так понял по коду, что каждый юзает по вкусу.


This seems to be throwing errors on Appveyor and travis


Still buggy.


unack


Здесь тоже static


Sorry, I didn't mean you should remove the initial line.
"Gets the value of an _optional_ property of an object."


И вроде TCallback принято сейчас писать у нас. Опять же - @deathbaba подтверди плиз)


idem.
"Gets the value of an _defined_ property of an object."


idem.
"Gets the value of a _required_ property of an object."


unack, those changes don't belong in this commit!


Default values should be zero, so no need to initialize private members to zero. Actually you taught me that :-)


other than that it's a good one


Sorry, I forgot the '.' between 'self' and variable ...
I just have performed new commit to fix it !

Bertrand

----- Mail original -----
De: "Pierre Raybaut" <notifications@github.com>
À: "PierreRaybaut/guiqwt" <guiqwt@noreply.github.com>
Cc: "berrosse" <bertrand.rosse@free.fr>
Envoyé: Vendredi 12 Février 2016 09:12:13
Objet: Re: [guiqwt] Update curve.py (dcd1f63)

@berrosse :
I'm not following: it seems that you have introduced the AttributeError bug instead of fixing it ( self_x and self_y are now used but never defined).

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


That didn't work. That however did give me another idea which seems to work on repeat screenshots :) Just not sure how this is performance wise? It basically resets the opacity to 1 when it reaches the end stage of the animation (where opacity = 0) and it will also set visible to false again.
onOpacityChanged: if(screenShooterGradient.opacity === 0) {
        screenShooterGradient.opacity = 1;
        screenShooterGradient.visible = false;
    }


yes, old yellow accent was totally unusable with lighter theme. This is the yellow palette from material design.

![material yellow](https://cloud.githubusercontent.com/assets/10454741/13005236/c7b118c6-d180-11e5-9be9-8e7931cfb8e2.jpg)


Я тут напоролся на неприятную штуку. static у methodId нельзя использовать, если метод ищется не по классу, а у конкретного объекта. Объекты релизующие определённый интерфейс, могут быть разного класса. И их methodId`s будут разные.


Можно через scopedlocalref наш


Michael, the dataHandler should also change the sampleTime to 120000 for Lora


one id that doing several effects


Using a HashMap to store an identifier + object is not advisable.
To workaround this Google is exposing it's own implementation: SparseArray.
They also expose an Long variant called LongSparseArray:
 - http://developer.android.com/reference/android/support/v4/util/LongSparseArray.html


@srisankethu please add `Fixes <full_url_to_issue>` to the git commit message.
[Here](https://github.com/coala-analyzer/coala/commit/4d7f12cc16ed4846af1bc05be2ac1289af078892) is an example


Should use C++11 initializers here.


## Put comments here that give an overall description of what your
## functions do

## Write a short comment describing this function

makeCacheMatrix <- function(x = matrix()) {

}

## Write a short comment describing this function

cacheSolve <- function(x, ...) {
        ## Return a matrix that is the inverse of 'x'
}


suhlas :+1:


Sorry typo error


The Cpu_state thing is it is used for Thread_state which is used potentially by any Genode application. However, the simple question is if we provide RISCV-specific code in a generic repo when not having any other use case than base-hw. Whatever we decide for we should apply it to all RISCV-specific code.


unack, that's a little complicated... can we have this split up into multiple arguments?


@sbillinge, don't put such confusing instructions like "installation instructions" in a file like this


don't give several meanings to one variable


@ssumpf: Nobody forces you to implement a library within base that is only used in base-hw, namely startup library for riscv. The argument you are using to do so is: for all other architectures the startup library implementation is located within base (of course it is because more than one kernel grounds on these architectures). So if we follow your argumentation (and as I said for all I care we can do so) you should move the public Cpu_state class to base too, following the same argumentation. A short ``find repos -name cpu_state.h`` shows:
./repos/base/include/spec/arm/cpu/cpu_state.h
./repos/base/include/spec/x86_32/cpu/cpu_state.h
./repos/base/include/spec/x86_64/cpu/cpu_state.h


I guess the content rendered inside the ```elseif``` suggests this is only for Bulgarian residents (Bulgarian ID or resident permit), so not resident of UK would be incorrect as it would also be shown to residents of other countries.


@m-stein: Honestly: Either you change it, or leave it. Your decision, but otherwise this goes too far and is not constructive.


otherwise good


je to jedno


Again needs `Fixes`. Hence - unack


commit message `ConteextManager` typo


unack


What sort of version string does this function really create? Can you provide an example?


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


This is great!


SafeGssCredHandle.Create can return null handle in case username is empty (ie. default credentials case). So guard this with if(!empty(username)). Accordingly, in ReleaseHandle, check for non-null _credential before DangerousRelease


@ljwatson when you're back can you have a look at this?


no toto este uplne nechapem, cakal by som, ze tam musim dat pr + 1, aby zobral aj ten uplne posledny riadok


how about this, is it better?


This could have a better name?


I think you misunderstood. I did not suggest to make them "fixed" but pre-fix the key with an expression related to the addon. The goad is to only pass information to the addon that to supposed to be seen by the addon.


Included X12Partner class https://github.com/practiceprovider/openemr/commit/225d5c505639906c4aa9555a01695742d2872527


For units, I would rather go with siunitx.


Probably it could be as it doesn't use `$this`. Now we are during discussion about core architecture and I think that you shouldn't rely on this method yet :wink:


yes, that is better.  I can pull it in now.  I think the users will be happy with that level of documentation


@sils1297 I tried automating it. It is unusual that my tests still worked after this


these should be the lua keywords.


@sils1297 You want the test gone?


There is a lot of `meat` intead of `meta`. Please fix it :)


ack


As you wish, but siunitx is heavier than this (low impact) macro :)
On 02/12/2016 12:09 PM, Bruno Pagani wrote:
> For units, I would rather go with siunitx.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/M2-AAIS/Poly_ET8/commit/6ebdc2db7c6524e5c67a08cfa3f0e737c1dde09a#commitcomment-16057433>.
>


Aha. i see. thats why my tests passed


init0: revision


Should be 1.3.0 because new feature is added this week


Implemented the following functions based on Ozo's initial work (Forgive my Python).

1. Ability to request for specific number of people e.g http://default-url/api/v1/faker/people/6 - returns six fake people
2. Ability to request for specific number of people e.g http://default-url/api/v1/faker/emails - returns all fake emails
3. Ability to request for specific number of people e.g http://default-url/api/v1/faker/people/6 - returns six fake people

**See screen shots **
http://default-url/api/v1/faker/people/5
![screen shot 2016-02-12 at 12 00 19 pm](https://cloud.githubusercontent.com/assets/4568634/13005273/29813b62-d181-11e5-8477-41e6214bc9e5.png)

http://default-url/api/v1/faker/emails

![screen shot 2016-02-12 at 12 00 38 pm](https://cloud.githubusercontent.com/assets/4568634/13005296/5955fad0-d181-11e5-805b-caa298111fab.png)

http://default-url/api/v1/faker/emails/3

![screen shot 2016-02-12 at 12 00 52 pm](https://cloud.githubusercontent.com/assets/4568634/13005307/641cc778-d181-11e5-8434-71de62f6088a.png)


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


But cleaner, especially regarding spacing.


Thanks for merging my pull requests. Sorry for the extra work, I wasn't quite aware of your coding conventions.

And I didn't find EntersBattlefieldAllTriggeredAbility, which really simplifies stuff (I only saw EntersBattlefieldTriggeredAbility for some reason).


Why did you delete hibernate? It is still used in your code!


ack, neat. I hope this covers most cases


TeamCity Device Info :: Tests [Build 35](http://82.35.147.206:8111/viewLog.html?buildId=80&buildTypeId=DI_Test) is now running


ack,


Well, I re-read the documentation and it seems I was wrong, I'm switching to it :)
On 02/12/2016 12:10 PM, Contact - cphyc.me wrote:
> As you wish, but siunitx is heavier than this (low impact) macro :)
> On 02/12/2016 12:09 PM, Bruno Pagani wrote:
> > For units, I would rather go with siunitx.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/M2-AAIS/Poly_ET8/commit/6ebdc2db7c6524e5c67a08cfa3f0e737c1dde09a#commitcomment-16057433>.
> >
>


unack


I got an error when compiling last sources on this line :
/tmp/yaourt-tmp-paul/aur-ring-kde-git/src/ring-kde/src/mainwindow.cpp:497:39: erreur: ‘class Media::Video’ has no member named ‘sourceModel’
       m_pVideoDW->setSourceModel(vid->sourceModel());
                                       ^


Mycomment for this


It would be nice to split the line in 2.


I'd love an own commit per bear


Well, when I had to rollback the line 361 to `if (row.uid) $(domrow).data('uid', String(row.uid));` because when I have all email rows selected, I can't unselect a single row. It feels like an **double click**, it get checked again. But at the background, the row is unselected by roundcube.


Actually, the right fix is *not* to overwrite the packet in the first place; if that's being done because a checksum calculation requires it, the checksum calculation needs to be fixed.


yep


## Put comments here that give an overall description of what your
## functions do

## Write a short comment describing this function

makeCacheMatrix <- function(x = matrix()) {

}

## Write a short comment describing this function

cacheSolve <- function(x, ...) {
        ## Return a matrix that is the inverse of 'x'
}


@skalk: Sounds reasonable, all I am trying to avoid is to make a special case out of the riscv thing and keep it generic (and I knew you would point the above out).


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


Three classes made.
1.	matrices
2.	matrixMultiply
3.	matrixTest
matrices:
•	Class asks for dimensions of the matrix. For ease restriction is made for the square matrix.
•	After asking for dimensions matrix is initialized by generating random numbers (0-100).
•	Matrices are also evaluated according to current time and strassen time.
•	In the end normal time elapsed and strassen time elapsed are displayed.
matrixMultiply:
•	This class simply multiplies the given matrices imported from the class matrices and applies the multiplication rules before multiplying.
•	A resultant matrix is generated as a result.
matrixTest:
I was not able to build unit test for the program. A simple junit testing class is made which only checks the dimensions of the matrix.


Actually, I think you would keep it if you variabilize username and password (with no default) in order to make it configurable in exo.properties.

In the same way, you could make the "/platform/api-access" group variabilized (though it is less essential)


TeamCity Device Info :: Tests [Build 35](http://82.35.147.206:8111/viewLog.html?buildId=80&buildTypeId=DI_Test) outcome was **SUCCESS**
Summary: Running Build time: 00:02:16


I think, coverd channels are no issue here because a kernel object that contains such a state is part of a corresponding core object. If I understand it right, core objects should be zeroed out (_clear_ds).


And potentially .reset could be a shorter syntax to avoid `std::.........>`


OneMoreCommentSeeIfYouCanReadIt


Got it

On Friday, February 12, 2016, pcorpet <notifications@github.com> wrote:

> https://golang.org/doc/effective_go.html#names
> " the visibility of a name outside a package is determined by whether its
> first character is upper case."
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pcorpet/hub/commit/c85826a685afaec3df40c352e80bf553d541196f#commitcomment-16056501>
> .
>


Ah, ok, yes - this sounds good! addonname.license (=inputstream.mpd.lincense)?


Now check this


@m-stein: Again, I will remove it.


TeamCity Civic Assembly :: Build all [Build 41](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1060&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


But what about c vs count ? :)

On Friday, February 12, 2016, pcorpet <notifications@github.com> wrote:

> https://github.com/golang/go/wiki/CodeReviewComments#variable-names
> "Variable names in Go should be short rather than long. This is especially
> true for local variables with limited scope. Prefer c to lineCount. Prefer
> i to sliceIndex."
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pcorpet/hub/commit/c85826a685afaec3df40c352e80bf553d541196f#commitcomment-16056704>
> .
>


Where is my middle or say second comment haaan?? Not right github you eat my second comment.


Yeah, less diff noise, done.


Du modifierar pEffect samtidigt som den blir läst ifrån, fixar i en ny commit.


again .reset could be nicer.


@minecrafter Thanks for clarifying.


:open_mouth: that's really trap. :wink: I can't believe someone has to think about that. I've made an (obious?) fix in b75347563d7d1eedc8719a8561fbeb88b94c015e so we won't have to bother with that.


FYI: http://www.lua.org/manual/5.3/manual.html#3.4.8


@martinklepsch is there a GH issue created in schema for this?


You won't get 256 color bars yet because the `colors` setting is set to zero in column mode.


Exactly. I did raise a PR which fixed this, but instead of getting merged, just some of the change was taken and manually added.


```
  1) FollowupsController POST create updates the status for successful followup sends
     Failure/Error: expect(request.reload.get_last_public_response_event.calculated_state)

       expected: "waiting_clarification"
            got: nil

       (compared using ==)
     # ./spec/controllers/followups_controller_spec.rb:235:in `block (3 levels) in <top (required)>'

Finished in 18.65 seconds (files took 26.87 seconds to load)
28 examples, 1 failure

Failed examples:

rspec ./spec/controllers/followups_controller_spec.rb:229 # FollowupsController POST create updates the status for successful followup sends
```


👍
Can you push it on your potato server?


change to "ignores 'at' when" otehrwise the commend is hard to read


I used wrong template MVC, so I decided to rewrite my program code. Also I think my program will be work quickly without Hibernate. I think, I have to use  only pure JDBC.

I refactored my code yesterday, and do not done commit changes.


Fix: arkenthera/ChiikaIssues#7


TeamCity Civic Assembly :: Build all [Build 41](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1060&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 165 (1 new), passed: 7, ignored: 4 Build time: 00:07:02
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455276050237: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

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


Is this target doing the right thing?

Take an example NuGet package that it creates: `AutoFixture.Idioms.3.40.nupkg`. While the package version is *3.40*, it's AutoFixture dependency is on *2.2*.

I can't say that I understand why the version isn't *2.2* as well...


That's a left-over from an older implementation that required `eventHandling` to be `false` in `preRestart`. This is obsolete now, will change that.


@SubJunk No, except that I'm unsure if you pushed to crowding after merging the opensubtitle PR with changes to ```message.properties```. Even though the changes are small, the translators ideally should have some days to react.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1378](http://teamcity.monogame.net/viewLog.html?buildId=14837&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependencies failed: 2 Build time: 00:00:00


Alternatively, you can move `_gateIsOpen = true;` line in place after cycle


Can you revert this commit? Snap camera buggy denoise not work and sometime blinked blue


What happen to deep sleep so ? o: we get around 15-20% of deep sleep in a night !


This modification is not fully appropriate.. We only display in Sigmah title window the x.y part of the version number (2.0, 2.1...) and not the z part (no 2.0.2, etc.). So this is the reason why the version number was entered at two different places.


Script error occurred if the account is closed


yes


Export audio_effects.h for userland to use


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


LOTM


Is it too late to rename this? AFAICS it's no longer a fold, it's a "deep" map with open recursion.
I'm not sure whether an actual `fold` (with an accumulator) would be useful (at any rate it's better not to include it until a use case is found).


The following doc(s) have been deployed to the [948terminu](http://948terminu-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://948terminu-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://712addcl-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


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


The following doc(s) have been deployed to the [823wpredi](http://823wpredi-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://823wpredi-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://1001coreu-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [418automat](http://418automat-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://418automat-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://980d8dir-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Or perhaps just rewrite that routine to have one switch statement instead of two, with each case having its own local variables for the components of the radiotap field.


It can be possible if will be added new cast functions to CAST_FUNC_REPLACEMENT_NEEDED collection without adding this functions to CAST_FUNC_REPLACEMENT_FROM_... collections.


`XmlOutputPath`, `HtmlOutputPath`, and `ToolPath`.


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


The following doc(s) have been deployed to the [980d8dir](http://980d8dir-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://980d8dir-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://418automat-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [1001coreu](http://1001coreu-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://1001coreu-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://823wpredi-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [712addcl](http://712addcl-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://712addcl-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://948terminu-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [1161myphpa](http://1161myphpa-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://1161myphpa-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://938d8inst-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


script error occurred if the account is closed


That would defeat the point of the method. The summons are supposed to acquire targets on the other side if possible.


Done. Could you please test my fixup so I don't have to build the RISCV toolchain?


We have to sit down and implement proper coloring, so we can remove that hardcoded setting :)


beware with installing `0.6.8 `
https://github.com/typings/typings/issues/200


I'd prefer no test files. With the previous solution, I couldn't find a way to _prevent_ the NUnit test runner from creating a result file, which is the reason the script subsequently deletes it.

If we can prevent the generation of files, that would make the script simpler.


The following doc(s) have been deployed to the [938d8inst](http://938d8inst-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://938d8inst-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://1124envco-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


I might be missing something here but we dont seem to be extracting the key here, so this message could be misleading? Maybe generating instead of extracting?


Hey,
I got following error while accessing schema of any resource:

> Reverse for 'api_get_schema' with arguments '()' and keyword arguments


@gdi2290 mentioned that he left the "-loader" suffixes because that would make it easier for people who are new to webpack to see what's going on and what webpack is doing.


Yep, I checked and thanks for suggesting them. I didn't know about them. However, it seems that for the commands where I'll need output for comparison, `shell=True` is the only option with `check_output`. I think I will have to stick with `Popen` for those cases.


I'd prefer `enumerateTempNuSpecFiles ()`.


"This fixes caching when using relative paths." it should have been. And the other typo.


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


Ребята решили перейти на using. Мне это не очень нравится, но чтобы было меньше хаоса, в новом коде пускай будет using.
В стандарте прописано TCallback.


The following doc(s) have been deployed to the [composer](http://composer-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://composer-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [circle](http://circle-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://circle-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


@abbradar thanks, I was confused how the group id should be used. Updated.


i removed the segmented control which was placed as part of the nav bar items and re added where the titile should present


В плюсовом коде у нас сверху вниз идут самые зависимые библиотеки к самым независимым (сверху map, снизу std). Еще ниже идет 3party.
В вашем случае можно либо не заморачиваться, либо вставлять инклуды jni части в самом верху.


I'd put it in the form's constructor (after the super call)


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


The following doc(s) have been deployed to the [d8rc2](http://d8rc2-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://d8rc2-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


let's reverse the if/else? (i.e. `if` instead of `if not`)


The following doc(s) have been deployed to the [debugfront](http://debugfront-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://debugfront-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Well spotted !
https://github.com/poparteu/openMVG/commit/18a91013b12abf07cfb0a7a7e1f733b9dc4029cf


The following doc(s) have been deployed to the [correcthos](http://correcthos-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://correcthos-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [1167custom](http://1167custom-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://1167custom-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://d8configm-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Modern Art UI Project


makes sense :) thx!


Заменю на using


sizer requirements


The following doc(s) have been deployed to the [pantheonle](http://pantheonle-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://pantheonle-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [gravityfor](http://gravityfor-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://gravityfor-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Ah, I see that now.


Can you move doris and richard down towards the bottom, as they're 'functional' NPCs (you can talk to them).


Can you fix this indentation? All of the x's should line up for all of the spawned NPCs grouped together


:heart:


The following doc(s) have been deployed to the [replacessl](http://replacessl-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://replacessl-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [d8configm](http://d8configm-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://d8configm-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://1161myphpa-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [calevansso](http://calevansso-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/continuous-integration-solutions](http://calevansso-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://1127phpmya-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Realised I hadn't set the request's status prior to posting to the action. This makes the spec pass:

```diff
diff --git a/spec/controllers/followups_controller_spec.rb b/spec/controllers/followups_controller_spec.rb
index c1bad75..d46c1d1 100644
--- a/spec/controllers/followups_controller_spec.rb
+++ b/spec/controllers/followups_controller_spec.rb
@@ -227,6 +227,8 @@ describe FollowupsController do
     end

     it "updates the status for successful followup sends" do
+      request.set_described_state('waiting_clarification')
+
       post :create, :outgoing_message => dummy_message,
                     :request_id => request.id,
                     :incoming_message_id => message_id
```


I think there might a bug. The install button is indeed always avalialble for "Latest", but in this case clicking on it does checkout the latest commit, although using git directly I can see that a fetch was done, and the commit is in origin/master (locally). So it appears it is not checking out the latest commit.

Removing path: /home/miguel/.local/share/SuperCollider/downloaded-quarks/ModalityTests
Installing ModalityTests
WARNING: Tag not found: Quark: ModalityTests[1.0.0] e09b619de7371aeb06fd1612ab7356f04a71cd1
 Possible tags: [  ]
WARNING: Modality not found
Installing UnitTesting
UnitTesting installed
Adding path: /home/miguel/.local/share/SuperCollider/downloaded-quarks/ModalityTests
ModalityTests installed

 e09b619de7371aeb06fd1612ab7356f04a71cd1 is the latest commit, but through git I can see that it is still locally in the one before.


OK.


No, I am not. :smile:
I just noticed it and thought method might be useful...
"Core architecture discussions" sounds like big changes expected...Is it related to the responsive UI?


The following doc(s) have been deployed to the [alertfors](http://alertfors-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://alertfors-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [arigoldpa](http://arigoldpa-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://arigoldpa-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


And OK.


The following doc(s) have been deployed to the [ttrowellpa](http://ttrowellpa-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://ttrowellpa-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Hey jetis, we have moved the project to https://github.com/tradable/tradable-embed-core

We thought that no one had started looking at it yet, sorry about that


Но тип, на который ссылается алиас тоже назван не по стандарту)


OK, no questions on these! :smile:


Change <string.h> to <cstring> and add <string> headers


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


The following doc(s) have been deployed to the [drush_error](http://drush_error-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://drush_error-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


`#include <string.h>` -> `#include <cstring>`


……Muh……

A reasonable commit message should be:
```
Add /rpgi version, close #53
```

By the way, watch your NOEOL.


init1: reply


mudar essa parte do codigo


The following doc(s) have been deployed to the [1020addpa](http://1020addpa-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://1020addpa-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The basic rule: the further from its declaration that a name is used, the more descriptive the name must be.

It's used on 4 lines, so I don't think it's worth detailing it more than that. On the opposite naming it would push a (Go) engineer to try to understand why it's named as such. Here it's just a very temp var that is used very locally.


hold


Add headers for std::string and c_str() method


Why do you need typedef int64_t and uint64_t? Wouldn't it be better to use the standard header? `#include <inttypes.h>`


+1 works like a charm here as well :)


Is it generating, or is it merely extracting the public key from the private key?  I'm not clear on this point.


The test case in CTSRD-CHERI/llvm#142 is now generating sensible code, so this is probably safe to try again.


`boss_gothik::boss_gothikAI::_gateIsOpen` change should not affect the value of `npc_gothik_minion_baseAI::_gateIsOpen`. Or I dont understand how it works :smile:


As you're mentioning there, mainly because Google wants me to. I can see your point though.


They can in fact be removed, which I will do.


При создании топика значение Published должно быть установлено в true по умолчанию.


The following doc(s) have been deployed to the [sukottokun](http://sukottokun-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://sukottokun-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


do you want to extend the prefix automagically to make sure that it is corectly defined?
In addon.xml it is listitemkeys="license" but internal and from video addon it must be [addonname].license ??


The following doc(s) have been deployed to the [hotfixred](http://hotfixred-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://hotfixred-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


too much spacing


qwerty


The following doc(s) have been deployed to the [rsyncdeplo](http://rsyncdeplo-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://rsyncdeplo-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The alternative would have been to limit the number allowed in the inspector :)

@reduz Is there a valid reason to limit the number of particules to 1024, or would 4096 be ok?


Add comment here


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


Once again


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


True. However, infinite loop.

OpenGate -> ACTION_GATE_OPENED -> no target found -> EnterEvadeMode -> ACTION_MINION_EVADE -> OpenGate etc.


We use it to build the Audio HAL on CM.. i dont think it will make any difference in kernel BY this commit


Fixed. Thx!


Можешь и его переименовать.

> On Feb 12, 2016, at 14:46, Alexander Marchuk <notifications@github.com> wrote:
>
> Но тип, на который ссылается алиас тоже назван не по стандарту)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/trashkalmar/omim/commit/84655140651e147d24e6cfd3ed7e7db928fd5d58#commitcomment-16058121>.
>


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


I haven't found a way to *not* generate the result file. Even setting it to null or empty does not help unfortunately.


The following doc(s) have been deployed to the [controlling](http://controlling-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://controlling-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Premier commit d'un grand projet !


@tsegismont could this prop please be moved to the top level pom and upgraded to 0.3.5.Final that is compatible with Accounts 2.0.21.Final used in https://github.com/hawkular/hawkular-metrics/pull/455 ?


The following doc(s) have been deployed to the [calevansso](http://calevansso-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://calevansso-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Не, не буду. Он по всему проекту используется.


Hi @ktommy, thanks for your contribution.
Would you mind to add some minor error handling in case `dos2unix` is not installed.. ?
Thanks in advance.
~david


Ок


The following doc(s) have been deployed to the [ttrowellfz](http://ttrowellfz-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://ttrowellfz-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [aeligature](http://aeligature-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://aeligature-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


not sure if understood this correctly. what do you suggest?


И как ActionBar работает? вы setSupportActionbar зачем закомментировали?


Hi @hamsterksu,

Thank you for your contribution on load balancer it is a very important contribution on RestComm. I would like to comment on  some modifications you added so to make sure that everything is working properly.
* logs_collect.sh :  As you can see, it is been added to the restcomm main bin path (/opt/Restcomm-JBoss-AS7/bin/restcomm/logs_collect.sh) for two main reasons:
  *  We need every script to be functional even with minimum docker RUN. So even if someone tries to run "logs_collect" script without defining any environment variables using basic command everything works properly.
  * As you suggested we need to move to a more central configuration process using  a main "installer.conf"
   that is actually included at the main RestComm-Core project, so not so many ```sed``` modifications will be necessary. That is the reason that is included at ```bin/restcomm/logs_collect.sh``` so eventually can be moved at the main RestComm binary ZIP file.

* About ```dockercleanup.sh```  please update your working branch the script is included at the project now. (*You were right I missed to add it to the main branch).

* About ```$RESTCOMM_LOGS``` it is a new flag added at the project so we don't use static/hard coded paths (User can set the pathfor the logs). Please see [here] (https://github.com/RestComm/Restcomm-Docker/blob/master/scripts/restcomm_env_locally.sh#L5).

* For the same reason ```docker_do.sh``` needs to be generic and able to run properly for what ever path user is configuring. That is the reason you see that we modifying that script at the docker configuration scripts. But again when we refactor the project using a main.conf file, everything will be more clear, but until then we must keep setup as less static as possible.

Please revise and repeat your PR:

* Dont need to add  ```RUN mkdir -p /var/log/restcomm``` path is automatically created from [here](https://github.com/RestComm/Restcomm-Docker/blob/master/scripts/restcomm_conf.sh#L23).

* Path  ```/opt/Restcomm-JBoss-AS7/bin/restcomm/logs_collect.sh``` is Ok, because is where is going to be, when refactoring takes place.

* directory ```/opt`` is created [here](https://github.com/RestComm/Restcomm-Docker/blob/master/scripts/restcomm_toolsconf.sh#L49), so no need to add it at the dockerfile.

* Scripts in ```etc/my_init.d/```` path are executed in alphabetical order (on the name), so please make sure that the script you added for the configuration of the load balancer is executed at the correct order.

*One last thing, maybe we can disable when running RestComm-docker the configuration of the LB if ```LOAD_BALANCERS``` flag is not set, so maybe it will be good to always disable ```/config-load-balancer.sh```, @deruelle?

Best Regards.


@brusch regarding the \Pimcore\Date() class, is the plan to phase these out too in place of DateTime or eventually have \Pimcore\Date extend DateTime?


There is almost the same tier-down code at LibRadosTwoPoolsECPP::TierDown thus you probably don't need another one here


Really. You're right


`automounter.sh` sa pusti aj pri odobrani, z manualu:
```
The special characters have the meaning:
	@ Run after creating the device.
	$ Run before removing the device.
	* Run both after creating and before removing the device.
```

Ale kedze iba 'potichu' failne, tak je to beriem ok...


The following doc(s) have been deployed to the [featuretes](http://featuretes-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://featuretes-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [1061wpcac](http://1061wpcac-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://1061wpcac-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [removeempt](http://removeempt-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://removeempt-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


если при устновке тулбара ругается на ошибку что тулбар в данной теме уже существует добавьте в стили вот это     <style name="AppTheme" parent="Theme.AppCompat.Light.DarkActionBar">
        <!-- Customize your theme here. -->
        <item name="colorPrimary">@color/primary</item>
        <item name="colorPrimaryDark">@color/primary_dark</item>
        <item name="colorAccent">@color/colorAccent</item>
    </style>

    <style name="AppTheme.NoActionBar">
        <item name="windowActionBar">false</item>
        <item name="windowNoTitle">true</item>
    </style> а в манифест добавьте вот это         android:theme="@style/AppTheme.NoActionBar">


That is odd, I'll look into it.


Instead of checking for specific platforms, I'd recommend check for features. It might be that future versions of given platform will have support for the currently missing feature.


do I have to write listitemkeys="inputstream.mpd.license" in addon.xml ? Or do I have to write listitemkeys="license" and you modify it to inputstream.mpd.license inside kodi while parsing the addon.xml?

The latter case has the benefit that there is no chance to define listitemkeys without the prefix


Ten find nie je spravny. Bash expandne to /media/*.mp3  a da to find-u na cmdline, ktory sa potom pokusi vypisat vsetko co najde s tymi cestami (cize to co mu dal bash). Vysledok je teda to iste, ako keby si spravil len `mpg123 /media/*.mp3`, t.j. nehlada v podadresaroch....


Are your sure for id ;)

What did you think if we make a tab for each plugin ? I make a test and a pull request :)


Can you use `isinstance` instead of these try and excepts? I think it would make the code quite a bit clearer and the line comments would  not be needed then.


Hmm, i see too many problems if we let Pimcore\Date extend DateTime, ... because if you're upgrading an existing project you will have to check your entire code.
Or is there an easy way how to do so without breaking everything? :)


The following doc(s) have been deployed to the [1167custom](http://1167custom-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://1167custom-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Hi, I'm running mono **4.2.1** on Linux and it is throwing *NotImplementedException* for `IPv4Mask` property.

Why is that? **Mono 4.2.1** does have it implemented, doesn't it?


Embarrassing indeed!


Changed in a later commit


BTW the version.org.hawkular.commons property in hawkular-metrics-bus should be removed after you move `version.org.hawkular.commons` to the top level pom.xml


看


看看嘛


Don;t see any logs


Are we sure on every target architecture that a `sizeof(bool) == 1`?


Concerning derivatives, I would consider http://www.ctan.org/tex-archive/macros/latex/contrib/esdiff, and for differential elements (dx alone), if this package can’t do it (didn’t test behaviour with empty second element), \DeclareMathOperator{\d}{d}.


тоже как вариант исспользования счетчика фрагментов в бекстеке но проще и правильнее вот так
public void onBackPressed() {
        BaseFragment findingFragment = (BaseFragment) getSupportFragmentManager().findFragmentById(R.id.main_frame);
        if (findingFragment != null && findingFragment instanceof ProfileFragment) {
            getSupportFragmentManager().popBackStack(null, FragmentManager.POP_BACK_STACK_INCLUSIVE);
        }

        if (drawerLayout.isDrawerOpen(Gravity.LEFT)) {
            drawerLayout.closeDrawer(Gravity.LEFT);
        } else {
            super.onBackPressed();
        }

    }


@developius I've been using [notify.js](https://github.com/alexgibson/notify.js).

It doesn't have the IE icon support, but that's fine with me.


@nipunbatra did you mean to commit all the building metadata yaml files? I thought they were supposed to be created dynamically by the converter...


The following doc(s) have been deployed to the [update_php_](http://update_php_-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://update_php_-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


It is not explicitly said that the -1 value is returned on failure, but we are testing on push_properties that the returned value is -1 on error. :dancer:


Case of 48 is hidden for all user roles except for retailer and wholesaler. Please check and confirm it on dev site.


The following doc(s) have been deployed to the [revert1081](http://revert1081-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://revert1081-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Опять - по имени коммита не ясно, что именно ты изменила и зачем


You are so good :)


у сообщения должно быть id по условию


I love it :100:


Yeah I can take a look today. One thing I am suspicious of is the implementation of Async.async, which you are using to create these effects. I'm curious if it propagates exceptions correctly. This is probably worth a unit test.

Other than that I'll have to dig in.


Did you take a look into boosts circle buffer?
http://www.boost.org/doc/libs/1_60_0/doc/html/circular_buffer.html


The following doc(s) have been deployed to the [rsyncstati](http://rsyncstati-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://rsyncstati-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [highlighth](http://highlighth-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://highlighth-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


怎么把 Alphabet 官网给删了
还有 Github的Amazonaws
还有 check.googlezip.net 的注释
为什么注释？ 请看这个[commit](https://github.com/racaljk/hosts/commit/b4036358fa07e1a3eea995f804d2dc522e5f2172)下的讨论。会出现500错误的


не исспользуйте повторяющиеся переменные суть у них одна и та же (значения одинаковые ) исспользуйте просто nav_rait_size


The following doc(s) have been deployed to the [edu](http://edu-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://edu-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Could you use `escape_latex` instead of `str` to make it a bit safer?


I think but I'm not sure, that `talloc_zero_array` here will be safer in case of failure and the process of checking by the caller


It is actually intentional. There are many packages for units out there, e.g.:
https://github.com/hgrecco/pint/blob/master/pint/quantity.py#L152
https://github.com/hgrecco/pint/blob/master/pint/quantity.py#L204

That said, I havent actually tried this with `pint` so if you prefer an explicit `isinstance` check I can definitely change the `try/except` blocks into that.


Ресурсы называйте в одном формате чтобы не было чередований _ и .


sure, the latter


TeamCity Device Info :: Tests [Build 36](http://82.35.147.206:8111/viewLog.html?buildId=82&buildTypeId=DI_Test) is now running


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


That would be called very often during the fade-out animation... Can you try my suggestion without the  if(visible) condition?


не добавляйте временные системный файлы под версионный контроль


The following doc(s) have been deployed to the [seturivia](http://seturivia-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://seturivia-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


The following doc(s) have been deployed to the [sort](http://sort-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://sort-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Will do!


The following doc(s) have been deployed to the [slashzerop](http://slashzerop-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://slashzerop-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


This change rendered my system unbootable because the right values were not in place, and I ended up with a grub.cfg that had no $kernel or $initrd entries at all.


If I understand `man ssh_config` correctly:

> **BatchMode**
             If set to “yes”, passphrase/password querying will be disabled.
             This option is useful in scripts and other batch jobs where no
             user is present to supply the password.  The argument must be
             “yes” or “no”.  The default is “no”.

setting `BatchMode=yes` will result in ssh don't prompting for passwords, so for password protected keys it won't be possible to provide git with the password to decipher the private key. I could verify from the command line that this setting breaks the `git ls-remote` command.


:+1:
Great idea: "if (Modifier.isFinal(field.getModifiers())) return null;" !!!


viem ako brat command line parametre, mozem to takto dorobit


As a matter of fact, it is not odd, but an omission on my part. The `NuGetPrepare` build target is missing. I will add it back in.


TeamCity Device Info :: Tests [Build 36](http://82.35.147.206:8111/viewLog.html?buildId=82&buildTypeId=DI_Test) outcome was **SUCCESS**
Summary: Tests passed: 1 Build time: 00:04:18


I thought about that too. For 2d Particles the max is 1024, but for 3d its 4096 in the inspector, so I assumed the intention was to rise the amount for 3d...


а если не первый запуск то что? фрагмент создавать не будете? если конфигурация устройства поменяется что произойдет?


Great stuff! Loving the concept here. Mobius strip really _is_ a twist! XD
Nanotechnology is a good angle - should definitely be able to do something with this.

You've also nailed the players/creators thing; it's really central to the concept.


The following doc(s) have been deployed to the [importerfi](http://importerfi-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://importerfi-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


TeamCity Device Info :: Tests [Build 37](http://82.35.147.206:8111/viewLog.html?buildId=83&buildTypeId=DI_Test) is now running


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


The following doc(s) have been deployed to the [sucuriguid](http://sucuriguid-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://sucuriguid-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Add a .gitignore to fix this.
https://github.com/rpng/vins/blob/vins_new/.gitignore


We are checking once a week. ``time.sleep()`` could much larger.


This is a public repo, please make it private.
This code cannot be public.


No problem. It's for sure a thing of pratice to know and find all existing framework classes.


Unfortunately it gets lost, as it does not fit our data model. I tried to map it to our samples but the numbers do not add up. :(


change


There is still a typo here on operatingsystem


Maybe just make the font-size bigger? The purple text doesn't look fantastic.


TeamCity Device Info :: Tests [Build 37](http://82.35.147.206:8111/viewLog.html?buildId=83&buildTypeId=DI_Test) outcome was **SUCCESS**
Summary: Tests passed: 1 Build time: 00:02:31


It's not directly related as responsive UI is just the case of HTML and CSS, not PHP code behind. But we'd like to clean up some PHP and try to make it more "current".


Idea is to get rid of this `sys.exit`. Reason being, if there are multiple Vagrantfiles changing between two SHAs, we should check all before exiting. So, I was planning something similar as to what you suggested.


Hm if the max is 1024 for 2D, then I don't think it was meant for 3D to accept even more than that. 3D particles are likely way more GPU intensive than 2D ones.


Ack, it was for local system.


thanks Jun Omae for the code review, most comments are fixed in commit:
https://github.com/walty8/trac/commit/85081cba5cb487f87bff23c9b4ebec33fbdba510

here is a complete patch regarding to #12171:
https://github.com/walty8/trac/compare/2b290dc5227b1d828184ce93767a2489dfcb3003...walty8:trunk

but I am not sure how to fix the problem of formatter.req, it's the original code from the plugin. Should I create a new request? Some hints would be appreciated.

Thanks!


Why is this significant?


değişiklik yaptım


reack


I was hoping that you were going to say that, but wanted to be sure :)
Yes I can foresee many headaches if Pimcore\Date were to extend DateTime!


The following doc(s) have been deployed to the [wpnonetwo](http://wpnonetwo-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://wpnonetwo-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


storage/app/public follows more conventions / sits better alongside other systems that have to do the same sort of thing.


Hmm okay that is a fair point. I think it would be best to just support `isinstance` for now and add support for other quantity libraries explicitly later on. That way we can find out if stuff breaks for other packages.


OK


Are you assuming this won't fail, based on the documented error cases?


We should use random user/group names in every testcase to avoid any issues if one of the testcase failed inbetween.


There is a range command in siunitx. I think it can handle exponent ranges too.


And switch on in Netbeans -> Options -> Editor -> OnSave -> Remove Unused Imports


As `#8` described here: http://www.toptal.com/php/10-most-common-mistakes-php-programmers-make


``class`` is redefined here. Also what does ``ModeAdmin`` mean?


The Mobius strip thing started off as a silly play on words, but the
post-justification is that it allows a variety of eco-systems - very sunny
in areas where the Sun is overhead, gloomy places where the Sun is just
below the horizon, and scary places which are always pitch dark and full of
monsters :)

On Fri, Feb 12, 2016 at 12:15 PM, Fjool <notifications@github.com> wrote:

> Great stuff! Loving the concept here. Mobius strip really *is* a twist! XD
> Nanotechnology is a good angle - should definitely be able to do something
> with this.
>
> You've also nailed the players/creators thing; it's really central to the
> concept.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Fjool/SuperMOO/commit/2fa3004b38056882df39a100c646e11916a1cd96#commitcomment-16058639>
> .
>


@pradeepto Yep there's a good scope for improvement. This is still a WIP so there'll be some crap. Definitely I'll try to reduce it to as little as possible. Thanks for the feedback so far. :smiley:


*Very* minor thing: I think non-expanded tabs should go between `void` and the name of the function (if I understood the coding style correctly)


Please be consistend with ``'`` and ``"``.


Oh, please import quantities in the function itself, just like numpy.


Good point


reack


Could you put some tabs here so `ret` matches the variable name column?


The following doc(s) have been deployed to the [unsupported](http://unsupported-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://unsupported-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Missing `[boltDB]` prefix


I don't like your variable naming. All of them seem to have an extra `_` at the end of it which seems unnecessary. Checkout the boost circular buffer, which will have a better implementation then anything we can implement.

Also please see what I originally implemented and compare to see if the KLT is implemented correctly. I did not do RANSAC. You need to start moving your things into the main code base.


The following doc(s) have been deployed to the [ttrowellwp](http://ttrowellwp-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://ttrowellwp-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


Fix the Y position of popups.

The syntax of the previous commit was wrong.
outerHeigh -> outerHeight


This looks a bit weird tbh, got a reference?


aaa


채팅 메시지 어댑터 수정. 오류 해결. 성공적.


Many thanks for doing this, it's now in use in http://lcc.org.uk/pages/consultation-map, allowing a sensible definition of 'staleness' (currently set as 6 months since latest activity) and latest activity to be noted in the popups on the map.


xcvcxvxcvxcv


Top top


Thanks, Peter!


@nk0leg you also have `QFile` creating at #L348


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


- [ ] APサーバに依存するため、


xcxzcxzc


If you find how to do it, I'm in!

On 02/12/2016 01:21 PM, Bruno Pagani wrote:
> There is a range command in siunitx. I think it can handle exponent
> ranges too.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/M2-AAIS/Poly_ET8/commit/4580dfc87f8d31fc9ce67149a2f53ca9f14a7669#commitcomment-16058730>.
>


@nk0leg don't we need to do the same for `QFileInfo`?


@carlaschroder they are good questions and for us it is now about refining the replies. How about placing this as an issue for 9.0.1 or 9.1. @MTRichards roadmap? @bboule thoughts from GCX?


zxzx


well, let's see what @reduz has to say. if that's the case we revert this and fix it the other way around


The following doc(s) have been deployed to the [slashzerop](http://slashzerop-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/developing-directly-with-sftp-mode](http://slashzerop-static-docs.pantheon.io/docs/articles/sites/code/developing-directly-with-sftp-mode)


@nk0leg I think `return 0u` would be more clear


Try \SIrange{e45}{e48}{\erg\per\s}.


Einwandfrei! Danke!!


Seems fragile.

There are two basename() implementations, libgen.h is POSIX version and it *modifies* the path argument. The function  exec_shell() does not expect this behaviour, because you really don't want to modify string from getenv() (and it's const char in the code).

It would be probably better to add strdup() to the code to make it portable to all basename() variants.


<3 you guys!


I don't quite follow why this is better.


Looks like a bad merge.  I somehow missed the PR that introduced this changes.


There is something of a spacing issue in here. Not very important, but my OCD just kicked in. :wink:


I meant to commit that this is for week 3 and NOT 4.

Thanks,
Spencer


`def each(&block : Cookie ->)` actually, no?


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


I'm pretty sure @peterbourgon would have something to say about stuttering here, but I can't think of anything better...


this how sinatra send cookies, it really no significant, but why not?


@SubJunk no objection too.


@pmlopes this is refactored in a later commit and over there, I check if a valid time was set (through a regular expression) as I don't know what timezone the build/test server has.


Seems that was chrome issue and it is fixed in latest chrome builds, PR may be closed


Adiciona arquivo removido na documentação: https://github.com/redelivre/delibera/wiki/Temas:-Estrutura-e-Seleção#exemplo-de-template-para-listar-pautas


this fix methods like `compact_map` on enumerable. (Cookie -> need to test it)


xcsdfc


Lines 570+571 walk through bouquets.tv and bouquets.radio and thus already catch userbouquet.favourites.tv and userbouquet.favourites.radio


Nope, it does "10⁴⁵ to 10⁴⁸ erg/s" instead of a nice '10⁴⁵-10⁴⁸erg/s'.
On 02/12/2016 01:25 PM, Bruno Pagani wrote:
> Try \SIrange{e45}{e48}{\erg\per\s}.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/M2-AAIS/Poly_ET8/commit/4580dfc87f8d31fc9ce67149a2f53ca9f14a7669#commitcomment-16058831>.
>


@ChrisBarker-NOAA Should we revert this file to a prior state?


Should say `Error loading ring data:`


Obsoleted by [SharedContextEvaluator](https://github.com/openshift/origin/pull/7128/files#diff-79e69f4d80514aff7b67bf664779b3fdR18).


Nope. For opening an issue I at least wanted to run macroexpand on the form so maintainers know what's going on (and I do first of all) but I didn't get to that yet.


to use this method, outside from constructor


Thanks!


tento subor nema execute permission, takze mdev-u sa ho nepodari pustit


Nah, this is OK. There is some precedent, e.g. `time.Time`.


The question is if we still need these extra fallbacks for at* functions. It would be nice to remove it and use everywhere what provides libc...

(yes, irrelevant issue for this patch, just brainstorming:-)


This was work on feature #6


obavam sa, ze SUBSYSTEM=usb matchuje este aj nejake intermediate zariadenia co tam povznikaju, nie len ten finalny block device... ale tak tie sa asi nepodari  mountnut.. ;)


Ниже пока не смотри, я вроде бы придумала, как это переписать.


Oh, I see... :-)


Movi o código do shortcode da timeline, para a timeline, ao invés de ficar junto com os outro shortcodes como sugerido. Realizei alguns testes e ela pareceu funcionar corretamente quando coloco o shortcode em uma página.


I think should be `echo "<td class='".$row_color."'><a href='".FORUM."viewthread.php?thread_id=".$data['thread_id']."&pid=".$data['post_id']."#post_".$data['post_id']."'>".$data['thread_subject']."</a></td>\n";`


This work is licensed under a Creative Commons Attribution-ShareAlike 3.0 Unported License.


If you want to make it public API, please add docs.


https://www.ietf.org/rfc/rfc0822.txt
5. DATE AND TIME SPECIFICATION

```
zone = "UT" / "GMT" ; Universal Time
; North American : UT
/ "EST" / "EDT" ; Eastern: - 5/ - 4
/ "CST" / "CDT" ; Central: - 6/ - 5
/ "MST" / "MDT" ; Mountain: - 7/ - 6
/ "PST" / "PDT" ; Pacific: - 8/ - 7
/ 1ALPHA ; Military: Z = UT;
; A:-1; (J not used)
; M:-12; N:+1; Y:+12
/ ( ("+" / "-") 4DIGIT ) ; Local differential
; hours+min. (HHMM)
```


ack


I figured it out, you didn't call GenerateDisplayItems for the active group. I'll merge it now.


TeamCity Civic Assembly :: Build all [Build 42](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1066&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


I think it's redundant because:
it's problem not reproduced for another cases
and the logic of work of SDL don't assumes empty dataset in this places


text.Replace("A", " "); use string.empty instead " "


- [ ] これなんなの？となりそうですし、本題ではないと思うので、omitしてしまってよいと思います。


We can reuse the code from VNCTabView.GetIPAddressForSSH() here, by moving it into a Vm property (in XenAPI-Extensions), for example VM.IPAddressForSSH. And then we can use this property here and in VNCTabView.


Chose the title attribute from gh-pages branch


Interesting, thanks. I'll expect people violating that RFC by putting whatever weird timezone there is into it, but we'll wait for that "bug" report then ;)


A question for my edification - if we persist the range grant and then die before the `defer`red ring update to the requestor, presumably when we come back up the requestor will learn of the grant in due course via random gossip?


Cancella il file di log e mettilo in .gitignore


please file a bug about this spelling error


Ano, dokonca ten finalny uz ma subsystem block :)
```
mdev[437]: 00:08:14.351676 ACTION:add SUBSYSTEM:usb DEVNAME:bus/usb/001/003 DEVPATH:/devices/pci0000:00/0000:00:0d.0/usb1/1-2
mdev[438]: 00:08:14.360885 ACTION:add SUBSYSTEM:usb DEVNAME:(null) DEVPATH:/devices/pci0000:00/0000:00:0d.0/usb1/1-2/1-2:1.0
...
mdev[447]: 00:08:15.677167 ACTION:add SUBSYSTEM:block DEVNAME:sdb DEVPATH:/devices/pci0000:00/0000:00:0d.0/usb1/1-2/1-2:1.0/host2/target2:0:0/2:0:0:0/block/sdb
```

;)


u dont need to create 2 intanse of Random


I know that feeling, bro.


Should this be turned into an exception instead?


this->viewports->at


Well, because it's noise in the commit history, with no explanation attached somebody at some point might wonder why this way and not another way.


vedo che fai sempre MY_SERVER.url + MY_SERVER.port etc...se questa cosa la mettessimo dentro MY_SERVER direttamente? in modo da usarla senza definirla tutte le volte?


@anosach-luxoft  I think it's return already defined variable is sufficiently transparent


ack


Это выражение работает впустую. Пайчарм даже должен был об этом сказать.


u can use  { get; private  set; }


Yes.  But it would make more sense to persist before sending the message.


Pourquoi pas salut ruguais ?


Yes, probably just one literal node with specializations would be the same. And to be honest, i just followed the previous logic.


all good


could perhaps start with

```javascript
var item = `me._itemsContainer[options.type]`
```
to make code more readable.


не то, проверь что будет


TeamCity Civic Assembly :: Build all [Build 42](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1066&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 165 (1 new), passed: 7, ignored: 4 Build time: 00:06:34
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455280770287: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

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


nice work. tricky business! :)

i say we merge these two after fixing comments, and bugtest properly afterwards.


I've installed **4.2.2.30** and still get *NotImplementedException*


I think we should probably hard fail at this point - thoughts?


Using `viewports` here instead of `this->viewports` because `this->viewports` is const and `clear_descriptors()` is a non-const function.


That's real software craftsmanship!


wow how did I miss we had that feature? Who did that?


I prefer SOL_NULL_CHECK() since it will log the error


@felixge Yes, this was most definitely an oversight, I'm sorry this happened. @shime please make changes via PRs so we can avoid such mistakes in the future. Thanks!


so is good file valid or not now?


Delete empty line


Document


I prefer SOL_INT_CHECK


`virtual ~MatchingBase (void) = default;`


Right. I will correct it.

For the optimization issue, no, they cannot be mutated. I may do something so the reads are cached. Not sure if it is worth to do it before there is a concrete case of usage that need to be optimized.


a teda, mal by byt niekde krajsie ako /hraj.sh ;)


Remove


@sils1297 I just revamped the earlier tests. Not adding/removing tests cases


See http://flysystem.thephpleague.com/api/ about visibility. For local, it sets the permissions, for AWS if its visible from an url etc.


ack though


Remove?


ng-maxlength="100" and in message 128


Is there a reason to mandate the `data.db` suffix? Why not allow the caller to specify the full path?


In-class initializers?


Just checked in locally. It is still valid.
This test seems kinda bad ... filing bug


Question: Why are we removing this test?

`readOnly` isn't something that we're supposed to test?


@hamsterksu,

Thank you very much, I will create a branch before merging at the repository, so we can test that everything is working properly.

I will send you here the  docker pull command so you can test as well the container with LB.

BR.


seems like a strategy pattern to me, please extract


see a3d9d6a

________________________________

This e-mail and any attachment(s) are intended only for the recipient(s) named above and others who have been specifically authorized to receive them. They may contain confidential information. If you are not the intended recipient, please do not read this email or its attachment(s). Furthermore, you are hereby notified that any dissemination, distribution or copying of this e-mail and any attachment(s) is strictly prohibited. If you have received this e-mail in error, please immediately notify the sender by replying to this e-mail and then delete this e-mail and any attachment(s) or copies thereof from your system. Thank you.


긴급소식입니다.
넥서스6p쪽에서
https://github.com/multirom-nexus6p?tab=repositories
멀티롬을성공했다고한거같습니디. 어느정도호환될걸예상해봅니다


reply


No luck: http://i.imgur.com/bXhh8hn.png

`"Error running ['zfs', 'create', '-b', '4096', '-o', 'primarycache=metadata', '-o', 'checksum=off', '-o', 'com.sun:auto-snapshot=false', 'antergos_eccn/home']: '-s' and '-b' can only be used when creating a volume\nusage:\n\tcreate [-p] [-o property=value] ... <filesystem>\n\tcreate [-ps] [-b blocksize] [-o property=value] ... -V <size> <volume>\n\nFor the property list, run: zfs set|get\n\nFor the delegated permission list, run: zfs allow|unallow"`

Logfile: http://pastebin.com/raw/7ULAPDWu


TeamCity Civic Assembly :: Build all [Build 43](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1070&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Writes data to a local named pipe may be splited in wrong place

[1455020732] Error: External command failed -> PROCESS_SERVICE_CHECK_RESULT;db1;mysql_
[1455020732] External command error: Command failed
[1455020732] External command error: Malformed command

sent command:
[1455020732] PROCESS_SERVICE_CHECK_RESULT;db1;mysql_backup;0;OK


You are right with the regexp, i could have added something like that. I forgot that SOM is not St. Also the implementation just cover a subset of the St literal arrays. Nevertheless, I do not think there is another syntax for literal arrays.


I'm guessing this doesn't belong to this commit?


Hay una constante definida en WebKeys.RESOURCE_BUNDLE_LOADER que se podría usar aquí.


Leftover refactoring from the previous version, but I think it can stay.


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


Wrap this in a `timeout`?


sip... ha sido vaguería

era para que pudiéramos debatir sobre la solución


@marcosnils To support testReadonly. We need to know cluster node id. but to know cluster node id we should use "cluster nodes" command for compatibility.

and old "cluster slots" command don't support cluster node id
(see http://redis.io/commands/cluster-slots)

To avoid compatibility problem I removed testReadonly.
(if we supposed "cluster slots" has cluster node id, it can be fail with old version redis cluster")
What do you think?


See quintel/etdataset@ f193ed5d4c1150f3cb3adfa581c4f80a97d9d731


I love this comment :D


* znamena, ze sa ten skript pusti aj pri pridani aj pri odobrani zariadenia... takze pri odobrani sa automounter.sh  pokusi mount-nut a pustit find na neexistujuci disk... ale tak vypise to asi par errorov ale nic zle by sa stat nemalo ;)


> I guess the real question here is do we care? I reset my rebase branch to run more tests and this is one of the things that was failing right off the bat

No, we don't.  Pick https://github.com/kubernetes/kubernetes/pull/20818


Если это интерфейс - да, могут быть разными, но тут же конкретный класс. Впрочем - может действительно измениться код на интерфейс и тогда будет нельзя статически кешировать. Нужно тогда ниже
static jmethodID methodID = jni::GetMethodID(env, *listener.get(), "onProgress", "(I)V");
поправить на нестатик.


>  But it would make more sense to persist before sending the message.

I'm not at all familiar with IPAM, but it looks to me like we are - `GrantRangeToHost` merely adjusts our local data structure, we then `persistRing` before the `defer allocSendRingUpdate(to)` sends the unicast message to the requestor?


I guess this doesn't belong to this commit


@charsyam there's a `JedisClusterTestUtil.getNodeId();` that we're using everywhere in `JedisClusterTest`. Can't we use that function?


Ah you already fixed it :)


nice!


I'm wondering if there's a hook in transformime or jupyter-js-output-area (jupyter-js-notebook) for adding classes to nodes reactively. Or if the best practice is to do it imperatively in the jupyter-js-service callback somehow.

@rgbkrk @jasongrout, any suggestions?


TeamCity Civic Assembly :: Build all [Build 43](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1070&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 165 (165 new), passed: 7, ignored: 4 Build time: 00:06:48
### Failed tests
```
Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blog categories tests: Should delete blog category new blog category 1455281737694: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

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


@piuccio Just a note on this, it's important not to change the ordering of the fields within the thrift definition: https://github.com/guardian/recommendations/blob/master/thrift.md

If you need to add a new field to the thrift definition it should be added to the end.


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


Конкретно сюда передаётся именно объект. onProgress уже поправил в одном из последних коммитов.


May I ask why you're disabling yasm?

Btw, nice work so far! I was actually planning on incorporating osx today, because I finally have access to a mac :stuck_out_tongue_winking_eye:


Why bellow and not above the super call?


good idea btw


That's the same message I got. Here's the build log of the last (failed) build before I removed Coverity:
https://ci.appveyor.com/project/mganss/htmlsanitizer/build/1.0.48

I then simply gave up.

Here's an earlier successful build with VS 2013:
https://ci.appveyor.com/project/mganss/htmlsanitizer/build/1.0.30


@ThomasZumsteg thanks for taking this on.

A few things will need to be changed before this can be merged.

1. It looks like you might have been working with an outdated copy of the master repo.  For example, the tests have all been renamed to `*.spec.js`, rather than `*_spec.js`.

2. The exercise should only have one `example.js`, so `example-2.js` should be removed.

3. Finally, we like to see all changes squashed in to one commit if possible.

I am happy to help you with any of the above steps if that would make it easier.  If you have any questions or need anything at all, just let me know.  Thanks!


this line should be done correctly in the previous commit in the first place!


because the field is not available before the super call (it's still unbound i think)


@marcosnils Oh, Thanks. I will look into it. :) Thanks.


Нет, пайчарм ничего здесь не пишет...


This fails to type-check on my machine with an `Unknown assertion failed`


It's not an exception. I updated the commit to return false in that case instead.