This seems to reduce the leaks to only 2; with a:
```
examine --tool=depends  f:\MingW32\TDM-gcc\bin\gcc.exe
```
there are still these leaks:
```
Dumping objects ->
lib/examine_file.c(433) : {564} normal block at 0x005FD918, 24 bytes long.
 Data: <f:\MingW32\TDM-g> 66 3A 5C 4D 69 6E 67 57 33 32 5C 54 44 4D 2D 67
Object dump complete.
Detected memory leaks!
Dumping objects ->
lib/examine_file.c(433) : {564} normal block at 0x005FD918, 24 bytes long.
 Data: <f:\MingW32\TDM-g> 66 3A 5C 4D 69 6E 67 57 33 32 5C 54 44 4D 2D 67
Object dump complete.
```

*PS* With a build fresh from todays repo.


For posterity: "Whether".


This actually closes #2 :)


These should not be headings.


1. Код, который не нужен лишь усложняет использование. Такие методы ты можешь дописать в любой момент. Вы пишете продакшн проект, а не фреймворк или библиотеку. Дополнить и так далее можно в любой момент, это не противоречит обратной совместимости.
2. По поводу flow логгера и ресурсов я уже писала - не нужно держать ресурс открытым, когда это не необходимо.


2 spaces


for System/390


THIS is serious shit !

... and definitely a media attachment to show the huge speed difference between exploring with the GPAtlasRRT (single thread so far) and with Marching Cube (multi thread) or any other explicitation algorithm


@silentguardian Slowly trying to catch up :innocent:   While going through this one, I I don't see the actions for

  -	'listmainitem' => 'sportal_admin_menus_main_item_list',
  -	'addmainitem' => 'sportal_admin_menus_main_item_edit',
  -	'editmainitem' => 'sportal_admin_menus_main_item_edit',
 - 	'deletemainitem' => 'sportal_admin_menus_main_item_delete',

In this, or the subsequent two other menu module ones.  The functions for Custom Menu and Custom Item are in place but not the Mani Menu


а если он не закрыл старый файл и программа упала или он ее выключил то грусть.
а это вполне себе нормальный case

если ты пришел на практику спорить а не учиться, то смени группу.


What happened to these two lines?


Those are actually the rop chain lol. Not actual chinese characters.


this was needed because `pipeline_name` has a pattern on it in routes so generating a route directly was not possible (without this hack)


+1 :)


In Firefox, when we scroll the scroll bar of the pitch matrix, it does not drags the whole matrix as it used to do earlier, but now it has rendered the draggable matrix undraggable.


Sorry, I just used a solution I saw LibreSSL devs suggesting in other software that had similar issues.


Hi just letting you know generate-wallet.html.sig no longer validates. I had to checkout 657db38 to verify the commit.


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


滑动冲突代码示例


asdfa


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


Tell us when it gonna release.?


Cool then!


Build fix


logging.exception already prints the stacktrace IIRC


How about `XCTAssert((9 ^^ 999).isPowerOf(9))`? I guess it gonna overflow.


Please remove the comments.


Uploaded!


It's from Mean Girls. :)


Thanks!


There is [QString::sprintf](http://doc.qt.io/qt-4.8/qstring.html#sprintf). Quoting the documents:
```
Warning: We do not recommend using QString::sprintf() in new Qt code.
```

Only if other devs agrees, I can switch.


layer_size_t has been renamed cnn_size_t (because now this type is not merely used for layer size, but size of parameters).


Why you added this?

I can't find the utility since all main game scripts are pure JS and not angular.


Unfortunately this doesn't work, `GetContentScaleFactor()` always returns 1 before the window is realized, so I never see the new menu item.


I'll have to pull out the documents: http://doc.qt.io/qt-4.8/qstring.html#arg


Instead of 0 this should check for TM_ECODE_OK.


answer 4 should be `PRAME`. You might check to see where that gene name falls in your file.


Instead of saving all variables from production object, you can do like the  or research object : just save the production object (so game.production) and import just the needed production variables when loading the savegame.


Config


changed title


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=23)


The agree, the check for vname2 should be removed as a null value for vehicle name is ok to send to Livetrack24


adding new title


I used
e->pawnInterSpan = popcount<Full>(e->pawnFrontSpan[WHITE] & e->pawnFrontSpan[BLACK]);
but you are using
e->pawnInterSpan = popcount<Full>(e->pawnFrontSpan[WHITE] | e->pawnFrontSpan[BLACK]);
Is this intended ?


Consider storing emails as a non-versioned JSON object or something. The bots are roaming these waters. Spam ahoy!


Try to leave a comment.


Oh, never mind Robin, as mentioned in the commit msg, it was just an "oops,
let's not go there". Instead of hacking through the import gene list file
functionality, we just rely on the backend(s) such as Gemini & co.


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=24)


hey Prem, rather use `rabbit_amqqueue:list_down` and pass vhost when calling `rabbitmqctl` command


Right, fixed in 1e3b6684


sa


That will make it accept rsync options as well, which is why the `--` was put in place. What we want instead is to force completion of `cpv` to only allow files. I'll look into it and put a PR in place.


The image is kinda of blurry.


Need check n, because it can be negative, and it will allow allocate very big buffer.


This change leads to regressions when using `NonBlockingHashMap` standalone, i.e. without H2O server. In particular when using version `ai.h2o:h2o-core:3.8.0.3` of the library I got this stacktrace:
```java
java.lang.ArrayIndexOutOfBoundsException: 366
	at java.util.ArrayList.add(ArrayList.java:459)
	at water.util.Log.write0(Log.java:102)
	at water.util.Log.write(Log.java:94)
	at water.util.Log.log(Log.java:71)
	at water.util.Log.info(Log.java:62)
	at water.nbhm.NonBlockingHashMap$CHM.resize(NonBlockingHashMap.java:972)
	at water.nbhm.NonBlockingHashMap$CHM.access$200(NonBlockingHashMap.java:798)
	at water.nbhm.NonBlockingHashMap.putIfMatch(NonBlockingHashMap.java:702)
	at water.nbhm.NonBlockingHashMap.putIfMatch(NonBlockingHashMap.java:368)
	at water.nbhm.NonBlockingHashMap.put(NonBlockingHashMap.java:318)
```
From what I can tell by inspecting https://github.com/h2oai/h2o-3/blob/master/h2o-core/src/main/java/water/util/Log.java code it is not thread-safe.


changed username to mawize


I'm not sure what you mean here, can you elaborate?


Why deleting the `data/home-repos` file? I think the right place for hardcoded data like that is in separate file, not mixed with business code. Thoughts?


@collectiveaccess FYI! In case this ends up blowing up in our face you'll know where to look :-)


Is there a reason to put here an arbitrary timeout (I guess time depends on machine speed) instead of putting `refreshFeatured()` in the `server.listen` callback?


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 341](http://ci.infinispan.org/viewLog.html?buildId=36320&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for 3eff1c16fe04582b3f8a447249e97b4785b55c7a
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:01

Process exited with code 255


This should have an underscore instead of a hyphen.


Java/Ch 05. Bit Manipulation/Q5_01_Insertion/QuestionA.java

just some renames...


Java/Ch 05. Bit Manipulation/Q5_01_Insertion/QuestionB.java

the trick is that shifting right by a negative value will add the 1s to the front, not the end.
Could also be
```Groovy
int mask = ~(Integer.MIN_VALUE >> j - i >>> Integer.SIZE - j - 1);
```
but that seems more complicated.


Checking isCancelled() immediately and dropping using `Exceptions.onNextDropped(d)` if cancelled will :
- protect onNext that have been concurrently cancelled (close connection ?)
- force a volatile check on the current instance (cancelled field). This will refresh the dataBuffer reference for the current thread and make sure its not cached by JVM memory model.


I'll remove it, thanks!


can you try with that patch ? I'm at work, i can't test it

[exm_list.zip](https://github.com/vtorri/examine/files/144631/exm_list.zip)


Right now it seems to work fine... Compilation is OK, if all the things work as they should be, expect a release!


@JohannesPfeifer yes. The idea is to see exactly what we have modified in the version we got from elsewhere.


Fixed.


IsValidTarget() ????


@Alexpux, why did you remove the patch?


Added https://github.com/jruby/warbler/tree/v2.0.0.rc2


Hello Andrew.

TFooNoiseRemoval + noiseremovalmultichannel + audacity_noiseremoval + audacity_realfftf -->

Merged into --> [uos_dsp_NoiseRemoval.pas](https://github.com/fredvs/uos/blob/master/src/uos_dsp_noiseremoval.pas)
------------------------------------------------------------------------------------
paio_channelhelper + pa_ringbuffer + paio_utils + paio_types -->

Merged into -->[ uos_dsp_utils.pas](https://github.com/fredvs/uos/blob/master/src/uos_dsp_utils.pas)

Many thanks ;-)

Fre;D


Why not just mark second DATA struct member as optional without unions and deciders, like in all previous games?


You can still change the state to planned (it should be interpreted as Work Orders Created then)


Deployment::commands is missing a docblock.


It is accidental


Maybe because it not apply anymore?


scope


I guess you can just fix-up this commit. (squash)


Where does isCancelled() come from? I don't see that method anywhere.


I looked a bit into it. The EXADialect and PyODBCConnector are both 'competing' for initializing `supports_unicode_statements`.

EXADialect (421): `supports_unicode_statements = sys.platform != 'darwin'`
PyODBCConnector(21):
`if util.py2k:
        # PyODBC unicode is broken on UCS-4 builds
        supports_unicode = sys.maxunicode == 65535
        supports_unicode_statements = supports_unicode`
PyODBCConnector(139):
`self.supports_unicode_statements = (
            not util.py2k or
            (not self.freetds and not self.easysoft)
        )`

on darwin, the statement on line 21 evaluates to `True`, same for line 139 in the ìnitialize()` function as we are testing on py2k and neither use freetds nor easysoft as odbc implementation.

This is a quite fragile initialization of configuration variables. It seems likely that changing the inheritance order changes the outcome of the initialization. Not sure yet what breaks the tests `test_numeric_as_float` and `test_render_literal_numeric_asfloat`.


But it's needed. `npm` stopped working because the patch is missing.

I'll try to rebase the patch later today and send a PR.


이거 '^' 무시되니까 지워


good catch


There's so much going on in this commit that it's hard to be sure nothing has gone awry. Ideally it would've been split up into a number of smaller more focussed commits.

I'd suggest something like the following (updating the relevant tests in each commit to keep them passing):

1. Rename the `year-1415` and `year-1516` options and their associated outcomes, etc.
2. Re-purpose the original `what_year?` question as the `what_year_full_time?` question, but *without* renaming it i.e. just change its content, etc.
3. Add the new `what_year_part_time?` question and associated template, but don't include it in the "routing" logic.
4. Change the "routing" logic to include the `what_year_part_time?` question.
5. Rename the `what_year?` question to `what_year_full_time?`.


Backticks around “peerDependencies”?


private final for the two + getters


Backticks around “read”?


I think a blank line after `}` on a line on its own is a common part of many Java style guides.


Some items and with a period, some do not. Unify?


why did you do that ?


`readFrameSlot(Frame frame)` as in https://github.com/jruby/jruby/wiki/Developing-with-JRuby-Truffle#the-lazy-pattern would be even better and more consistent with our guidelines actually.


@JohannesPfeifer actually, forget it. I'm just going to get rid of `strjoin` altogether


minifying html may introduces unwanted side effects
see http://perfectionkills.com/experimenting-with-html-minifier/
i'll remove this feature at the moment...


Very minor: It looks as if you're restricting the number of characters per line in your commit note body to 50. The 50-character limit is only for the subject line, the body line limit is 72-characters. I think the short lines in your commit note body make the note harder to read.


@AliceWonderMiscreations Is the commit history relevant for the pull request? If not you may want to consider squashing the commits.


do all queued log messages get logged too ?


yeah


Oh can be deleted it was something I've played with, sorry!


Why change this from utf8 to ascii?


Done.


Also added GPIO.cleanup() on close


I never used `FactoryGirl` in `db/seeds.rb`. This is actually a great idea.


Yea, the progressive sampling, how cool is that !


>You'll be pleased to know that on the next generation of Cray machines, ALPS is being replaced with native slurm only.

Indeed, that'd be good. I hope they don't pretend to be something else than SLURM in this case, otherwise our native SLURM support might not work for them.


Yes, just numbers looked quite strange to me as usually it requires special symbols for argument formatting and i googled docs by myself too since i'm not familiar with QT


@kovayashi  コメントありがとう。

ちょっと自分だけの意見になってしまって、一般出来でなかったかもしれません。

いろいろ考えると、 確かにメソッド名を`round`とすることは正解ではなさそうでが、それでも`getRound`は良くないと思っています。

余裕で議論の余地があるので、考えを述べます。
コメントでも書いたように
get/setはgetter/setterの役割ではしっくりきます。
理由は

```java
User user = new User();
// 省略.
String name = user.getName();
```
というように、'Userインスタンスから、名前を取得する'
というのが明確だからです。

で、public staticなメソッドだと
```java
User.getName();
```
となるり、「誰の名前だよ」となるのかなと思っています。
なぜなら、クラス定義自体にnameのデータは存在しないはずだからです。

-------------

さて、そうなるとクラス設計に話が行きます。
(ここまで来るとキングに教えようとしていた範囲を超えてくるんだけど)

このクラスを四捨五入を返す為のstaticなクラスとします。
クラス名Rounding は名詞じゃないので, Rounderとします。
```
Rounder.getRound(args)
```

`四捨五入するやつから、四捨五入を取ってこい`

なんかよくわからないかなと思いました。

おそらく正解は
```
Rounder.calculation(args)
```
とか
```
Rounder.rounding(args)
```
が良いかなと思ってきました

ちなみに、今後この子たちをインスタンスクラス化しようと思っています。
そうなるとまた名前は変わってくるかもしれません。


I thought ```arg()``` could replace this behavior of ```sprintf()``` but if not then do not bother yourself with that.


You can remove prefix


public?


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


Yes, I wanted to somehow count the average pawn tempi in the position.

Now I realize that my metrics does not exactly count that. What I
would like
is in case of a passed pawn, the number of squares to the promotion
square,
and in case of an opposed pawn, the interspan squares divided by 2
because the
interspan squares are shared by the two players...

>

... /// ... \\\ ...


lgtm, but sooner or later we need a test


In WP we put spaces in brackets only around variables, not numbers or strings.


もしかしたらjavaの文化と相反してる気がしてきた。
まぁ各プロジェクトそれぞれのコーディングルールがあるからそれに準じれば良いって話ではあるんだけど


seems to be an [issue](https://github.com/travis-ci/travis-ci/issues/5036) in tarvis, will fix it anyways


This looks good. I think this provides a solid base for an initial implementation and we can tweak it if needed in the future.


Also i suspect that ```QString::sprinf``` is less preferable due to easily making mistakes with formatting :)


Can we make this configurable to keep the library generic?


So that's the purpose of it...


There is a bug in ```frameworks/base/core/jni/android_view_GLES20Canvas.cpp```. This code
```
if (property_get("ro.kernel.qemu", prop, NULL) == 0) {
```
does not do the right thing. It should be
```
if (property_get("ro.kernel.qemu", prop, NULL) == 0 || atoi(prop) == 0) {
```


This is nice! :+1:


Calculation for RMS was incorrect

npts was outside the sqrt, and should be inside. I changed the expected result in the unit test (I believe these values are hardcoded?)


Runaway space before `__At the moment`


Hi!
Just had a look to the whole thing and it looks quite promising :)
Following a current review on OpenStack, we are putting in place a docstring structure (for auto-generated doc) here: https://review.openstack.org/#/c/279084/. So can you do the same for your strategy?
I reckon it should be fairly similar to https://review.openstack.org/#/c/279084/8/watcher/decision_engine/strategy/strategies/basic_consolidation.py


Looks good.


Looks like this cant be done from github, time to learn about squashing commits it seems...


But please, can you change it?


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


@@var_name is a class variable
@var_name is an instance variable

doesn't matter, don't fix here, just be aware of the difference :smile:


@rco-odoo See with RCO why the group hidden computed fields are pre-fetched on view loading


What do you mean by squashing the commits?

What happened is after the first pull request I realized I had left of the ### after the [RPM] line, so I made this to add it.

I'm not familiar with the term squashing the commits.


Nice!


Is there a reason for writing the condition this way, instead of `isinstance(data, text_types)`? I find it much harder to understand the intention, with it written this way.


```shell

git submodule update --init --recursive


Here I said that because the component name is used in debug mode as the prefix for the hook name.


Ha sorry I meant subscriberComplete !


Nice, thanks.


Right now it is 6 commits with 2 merges for adding a file. This could be done in a single commit which is atomic. You may want to try the first result of https://www.google.de/search?q=git+squash+commits


Here's a pattern that I find useful:

```elisp
(defun counsel-eshell ()
  "Browse Eshell history."
  (interactive)
  (setq ivy-completion-beg (point))
  (setq ivy-completion-end (point))
  (ivy-read "Symbol name: "
            (delete-dups
             (ring-elements eshell-history-ring))
            :action #'ivy-completion-in-region-action))
```

`ivy-completion-in-region-action` will overwrite anything between `ivy-completion-beg` and `ivy-completion-end` and reset them.  This means that you can e.g. `counsel-eshell`, narrow to `cd` commands by entering `cd` in the minibuffer, and cycle the inserted command with C-M-n and C-M-p.  I think it's nice because you see the final result in place, while you're still able to change it.

Another note, I don't bind keys in hooks - a hook is run each time a mode starts, so those key bindings are an overhead. It's enough to bind the keys once.

Another note, I don't put lambdas in hooks - it's a pain to remove or modify them later. Instead, name the hook function, redefine it when you want with eval-defun, and use `revert-buffer` to call the new hook function from the hook.


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


Why is that?


maybe this should also impact https://github.com/georchestra/ansible/blob/master/roles/postgresql/tasks/main.yml#L2


cast not needed?


Yes you are right. My intention were to hold the save data as small as possible. This has two reasons:
a) Working on upgrades and adding new features is easier to handle with less save data as possible
b) Some browsers like opera have a limit of 3mb, firefox 5mb. So I wanted to take precautionary never to come to the limit ( excluded that there are libraries like lz-string to minify).

That were only two ideas I had for this decision. I know that writing down every single variable isn't the right thing, too.


You don't want to check private keys into github. You should delete this from the target resource if it is still there and remove it from github. You should use .gitignore to make sure you accidentally make these commits.


that is an awesome video!
Two stupid questions though:
1. can you extend the range of colours, so that doesn't seems to be just a
green blob. This affects directly the impression of some first-timer
watcher. Right now gives the idea that we have a rough modelling of the
uncertainty, while that's far to be true.
2. can you also produce a video with less points so that it also seems
faster, but it also look complete. If this is not possible we can easily
speed up the video, but it would be great to say 'you know what, you are
watching something in real time!'

On 24 February 2016 at 13:21, Federico <notifications@github.com> wrote:

> Yea, the progressive sampling, how cool is that !
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pacman-project/gaussian-object-modelling/commit/e118f7c38bfb95be1522de2d8f5af3757e354d8f#commitcomment-16296162>
> .
>

--
Claudio

Curiously enough, the only thing that went through the mind of the bowl of
petunias as it fell was "Oh no, not again".
Many people have speculated that if we knew exactly why the bowl of
petunias had thought that we would know a lot more about the nature of the
Universe than we do now.
D. Adams - Hitchhiker's guide to the galaxy


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


It's a branch (which is on GitHub). Unfortunately it segfaults on current Julia master due to a type inference bug.


This probably needs to go up in a separate Jira. In the end it has little to do with the concurrent access problem.


Those numbers are listed on the mta.info website.


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 342](http://ci.infinispan.org/viewLog.html?buildId=36321&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for ccecf359d7175d1255b27c1b9501183883736774
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:02

Process exited with code 255


Please fix the missing imports Jenkins is complaining about. We require those in all newly written code.


Just noticed this now but.. these look like awesome changes - would you mind opening a PR to get them back in to the main repo?


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 343](http://ci.infinispan.org/viewLog.html?buildId=36323&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for ccecf359d7175d1255b27c1b9501183883736774
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:01

Process exited with code 255


:+1: if this fixes the zlib problem. I looked into it a while back but didn't get very far.


@graimundo Will change to the global property version list in a future commit. Please merge as is.


Hi!

On top of unit tests, we should probably implement an integration test just to make sure it all works fine.
So you can have a look at https://github.com/icclab/watcher/blob/master/watcher_tempest_plugin/tests/scenario/test_execute_basic_optim.py which implements a Tempest test scenario with the ``basic_consolidation`` strategy.


@MarcoFalke okay, I don't have cli access to github right now, so I can't do that. Sorry.


made public


Alright
what does this actually mean?
`ro.kernel.qemu=0`
It should mean that device is not emulator


removed


xmlid in self.url?


```
User.find_by_login!(m)
```
throws an exception, otherwise I would need to catch it in the controller!


I'm not sure it is needed for a CreationMessage


Ok, gottit, thanks. `kf/loctrack` for the record.


should be @CheckForNull


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 344](http://ci.infinispan.org/viewLog.html?buildId=36325&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for ccecf359d7175d1255b27c1b9501183883736774
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:01:58

Process exited with code 255


**exnglish chars** should be **english chars**


any reason for this change? not that it really matters. (angle vs quote)


Java/Ch 01. Arrays and Strings/Q1_04_Palindrome_Permutation/QuestionC.java

:+1:

Since `mask` is used only once now, feel free to inline it


Could you type the key stronger?


That was my fault as I tried merging some patches into our private version.
But you should not use this version but the upstream version e.g. https://github.com/scality/Droplet


1) I can, but how ? adding a blue channel ?

2) I cant directly reduce the sampling density of marching cubes sampling (the first one that you saw), because otherwise it will not work (i would have to extend its code a lot, and i dont want to)
But i can remove the initial (slow) sampling and directly start with atlas with fast(sparse) sampling, then at the end when the object is reconstructed i can show the slow(accurate) sampling.

Suggestions?


Ah, I haven't though about the maximum size of local storage on some browsers, I may have to change the data saved ASAP.


@ldachary i was wrong. can we backup the pidfile before https://github.com/dachary/ceph/commit/ffb3f6a2827a6e9b02fe0f1da2739594aa2f559a#diff-e5e910b69c49ff23ffdbeb6e6137dac8R71 ? for example
```sh
cp $dir/osd.0.pid $dir/osd.0.pid.old
```
?
and `kill_daemon` will match the pid and kill it. and the `$dir/osd.0.pid` is left untouched. otherwise the daemon is still lingering.


this is the best explanation of tech debt : https://twitter.com/carnage4life/status/585458646680670208


Minor: Our APIs always receives a pointer to sol_network_link_addr. Should we stick to that?


to remove: we don't want to handle xmlid, just url in /static


@heller: Thanks!


That's what I saw in a few other binary addons but doesn't really matter.


I'm not sure te odd/even check is needed, or that it helps readability.


done


to remove


commit message?


Yup, it does. :rocket:


right. was just curious.merging..


missing wercker the best one.


Copied from outdated commit:
```
Why is this needed?
```
```
User.find_by_login!(m)
```
throws an exception, otherwise I would need to catch it in the controller!


(out of scope of this request) as revision is not null, no need for condition at line 38


I see I forgot to include patch for it.
If you use your own, you need to patch the file lib/gdi/picload.cpp
as below:
"
-	gft = DGifOpenFileName(filepara->file);
+	int GifLastError;
+	gft = DGifOpenFileName(filepara->file, &GifLastError);
"
if you use standard one, it will be corrected shortly


I'm not really sure about this one, All coap servers will reuse the addr?
What if I want to create a coap server that does not reuse the addr?


@santoshgi  Test case should create data and test on same data and at last delete the same data. This will ensure that db data is not touched and same as before running testcases.


I agree with @pietersv said. Is it possble to change this line as well?


or #private ;)


just some renames...


These are all used. NPM is the package manager, the libraries are based off of node. Protractor runs the E2E tests, Jasmine/Karma are for Unit Tests, Travis will be up once the tests are written, Browserify is how the JS is combined into the app.


很好。谢谢。


the trick is that shifting right by a negative value will add the 1s to the front, not the end.

Could have been
```Groovy
int mask = ~(Integer.MIN_VALUE >> j - i >>> Integer.SIZE - j - 1);
```
also, but that seems more complicated.


JUnit 4.11 is not a particular good release of JUnit.


Yes. We set ```ro.kernel.qemu=0``` to tell android that our device is not a emulator. But the code parse this setting like above just test that ```ro.kernel.qemu``` is set or not and does not check what its value is.


Bump.. I was forced to switch to dev-master because there is no tag. Please add 1.2.1 for us to have a non breaking fix. Being on dev-master is dangerous.


> is there a Phoenix callback that can be used to send emails here?
I believe Phoenix _has_ callbacks but I'm strongly against them unless you must have something that matters around the DB. Rails callsback provide enough pain that I avoid all of them.

>Or is it a custom function that's called from the Controller layer? Would that send email function be here or in a services layer?

Probably makes the most sense for this to happen in the controller. If for some reason we were to create a member directly in the console, an email wouldn't be necesary.


Fixed issue #5


OK


:sparkles:


made public.
But most of the comments on scoping were for classes designed on purpose with package local scope. I'll figure out eventually why this is such an issue to use this scope for internal (not meant to be reused elsewhere) classes.


>Models as Modules is a very interesting perspective. I'm not sure how I feel about that.

Moving forward there will be no "model" only "schemas". Phoenix really doesn't have "models" already, they're just weirdly named.


Is there a reason dojo and dijit were added to `dependencies` in package.json?  There's functionally no reason they're needed to build the theme, which is what the other package.json dependencies are for (though actually, I would think the grunt deps should be devDependencies anyway).


should be called only in debug mode


TeamCity geo-publisher :: GeoPublisher-viewer [Build 1](http://teamcity.idgis.nl/viewLog.html?buildId=13&buildTypeId=GeoPublisher_GeoPublisherViewer) is now running


int?


@hvelarde I did this to solve flake8 complains


Go Imports formatted this on save.


Corrections have been made appropriately. Thanks


Remove this line, please


This is not a class description; remove


кто так делает? имена таблиц надо делать


Did you implement this part of the spec:

> At this point the server will check that it has everything it needs for a valid message. In particular, that it has a valid “From” address, it has at least one address to send to, and all addresses in To/Cc/Bcc are valid email addresses. If it cannot send, it will reject the creation/update with an invalidProperties error. The Error object SHOULD contain a property called properties of type String[] that lists all the properties that were invalid. The object SHOULD also contain a description property of type String with a user-friendly description of the problems to present to the user.

In my understanding, valid email means "should contain one @"


`i` was never incremented so errors were always on line 2.


@vtorri Works fine. Except for 2 missing `;`.


I don't know about 1), but about 2), surely having a 1X speed video is much better...

Perhaps annotating the video can help to distract the viewer while the stuff is going on, to also help understand what's going on, phrases like:
"Note the speed of the GPAtlasRRT candidate point generation"

At the end, when the object is fully covered by the atlas
"Compare the speed of covering the full object with respect to the initial marching cube computation"

no more ideas for now, I need to speed up the writing
:fist: hasta la victoria siempre


Надо данные отдельно вынести


section titles can be created using markdown - so maybe
`### Section title`
or similar - check doxygen markdown


Если юникод возвращать - нужно тогда в jni обязательно учитывать и создавать строки из байтов. Или проверять что диапазон символов используемых не выходит за U+10000


Do you allow user to clone any branch ?
Thats what I understand.


@vkuchyn This method is not necessary.


Can you re-word this such that it's not a changelog anymore but documentation of what this constructor does and how?


removed


Alright


strange mailboxId, but... why not...


If `investment=True` and an input rule is used an error message is raised. Sometimes the in_max value exists but is set to 'inf' or even to 'None' or it does not exist but can exist. In all these cases the error is still raised. I think this should not be the case so I check for 'None' and 'inf' values before raising an error.

@simonhilpert Is it okay for you?


Remove - not relevant for documentation


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


indent?


Yeah I did a backspace too much there when correcting another typo ;)


Keep (but reword) this - it explains the mechanism of child / parent symbol resolution which is important top explain.


Remove


此句校对不准确


Move to doc of Initialize(), please.


EcolexSolr.extract handles pysolr.SolrError internally


增加一条


hi @michalslaski , are u guys going to create a 1.6.2 tag? I don't see it :)


This sounds good to me. Any strong feelings about whether to add the minimum good offset (e.g. +18) or be extra safe and add a much larger value (e.g. +600)? There is no difference in performance there.


These are all dependencies that came about with the project structure change (as a direct result of switching to browserify) and are now used in the project.


and  --> 和


//technologies instead of technogies


Yes, we are going to create 1.6.2 tag. We just waiting for final load tests results before doing that :)


Yea i will surely add annotations, what you saw was a test. Once i reached convergence.

Also what should i record for data (apart from the video):
- number of steps to reach goal
- what was the goal
- some measure accuracy of shape representation (no idea how i can do this)

Then record for n objects the stats above, with  random picking, single touch with atlas, sliding with atlas.

we should get a nice table out of it.


Good idea, I'll add.


Adicionando gitignore # Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio and Webstorm


Not sure if I like this, aren't there some builtin double-underscore methods Python has to make this easier to handle without needing a getter?


I suspect that the above in this implementation would not work, try it out:

```restorecon -v /var/run/libvirt/virtlogd-sock```

Instead it should, probably, be specified that this applies to sock_files:

```/var/run/libvirt/virtlogd-sock	-s gen_context(system_u:object_r:virtlogd_var_run_t,s0)```


Thanks for mentioning it! This has been fixed in the latest version.


I can't understand your opinion.
Why does only the 3rd system() require TRACKER_PREFIX ?
line 179:
    system("tracker-control -t");
line 189:
    system("tracker-control -t");
line 448:
            system(TRACKER_PREFIX "/bin/tracker-control -s");


Same question here.

Thanks


still not corrected for all cases:
https://en.wikipedia.org/wiki/Operator_associativity#Non-associative_operators
I checked and at least '>' '<' need parenthesis too (and probably '<=' '>=')
I'm trying to make the code more general -as it was done with precedence handling- ... but I'm stuck


warning? :)


Fuck if I know.


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 345](http://ci.infinispan.org/viewLog.html?buildId=36327&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for ccecf359d7175d1255b27c1b9501183883736774
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:01:57

Process exited with code 255


`git grep "tests/"` shows three locations you've missed changing the name


Nah, this was the old way the undo was done (it used setScale instead of the now used setTransformation). I should have removed that. Good catch.


@marceloboeira It's because you have it already compiled. Try `make clean` and then `make libcrystal`. If you do `make` or `make crystal` or some other targets, they will do `make libcrystal` automatically.


Which of these functions are actually part of the interface to other components?


TeamCity geo-publisher :: GeoPublisher-viewer [Build 1](http://teamcity.idgis.nl/viewLog.html?buildId=13&buildTypeId=GeoPublisher_GeoPublisherViewer) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:08:41


I know, but that only hides and not solves the underlying problem: `except: Exception:` is the same that `except:`.


试试功能


yes


TeamCity geo-publisher :: GeoPublisher-viewer [Build 2](http://teamcity.idgis.nl/viewLog.html?buildId=14&buildTypeId=GeoPublisher_GeoPublisherViewer) is now running


TeamCity geo-publisher :: GeoPublisher-viewer [Build 2](http://teamcity.idgis.nl/viewLog.html?buildId=14&buildTypeId=GeoPublisher_GeoPublisherViewer) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:03


I don't think we want a global assumption - much better to let code paths that happen to just see one thread, no matter how many there are, specialise on their own. I've switched to using the DSL here. It's a pity I can't see a way to hoist the thread guard to be once per method. This probably has extremely limited practical benefit anyway.


MVC is a logical construct though. What happened to the "fat model" concept?

Totally agreed on the Rails-callback-as-pain comment.


删除文件


I am not sure what you are trying to avoid. You can't use side="input" because this isn't implemented. No matter what. Even if in_max of the components exist ... So the error should be raised in any way, only if you implement the constraint it would make sense to change anything there.


Needless function call? Just save it once?


I would, probably, use instead:

```/usr/lib/systemd/system/*virtlogd.*	gen_context(system_u:object_r:virtlogd_unit_file_t,s0)```

Mainly for confined administration. Not sure if systemctl has the SELinux awareness i requested a while ago but the above would make it so that when you do ```systemctl edit virtlogd```, that it will create ```/etc/systemd/system/virtlogd.service.d``` also with type virtlogd_unit_file_t. Without this, a confined virtlogd admin would need to be able to write generic systemd unit files to be able to "edit" virtlogd units.


此句与下文有衔接，标点不对而已


that's not an issue, if you really mean package-private, just put @VisibleForTesting
anyway I don't see any good reason to have such restrictions on these classes


holo


I don't think it's important to tune this option now. Any argument either of us made would really just be guessing. All these numbers are arbitrary and what we need to do is run some serious code and do some machine learning to find the best combination for startup, for peak, and a balance between the two.


Ah ok, sorry. got the point.
"magic string" is used only in tests, but "production code" has a private constant. I could expose this value outside the class and use it into tests ;)


# 你好！


旨在


project specific :smile:


Now we have the HouseSchema twice - any reason for that?

Once in models.js and once in this file....


This could be static


`lipo -remove armv7 -output osxonly *.framework/Versions/Current/$file`

and replace `*.framework/Versions/Current/$file` with `osxonly`


@jhecking Seems you are heading to the finishing touches? Hope you didn't discover any bigger problems.


Can we hide this struct and make it private to the module and instead use a single id for communications? (I.e. how the open, read, write and close system calls work). In this way I think that the caller module would be much neater and simple.


styleci could take care of this automatically, the copyright can however only be added from within the StyleCi Interface.


It works just fine

# ls -alZ /var/run/libvirt/virtlogd-sock
srw-rw-rw-. 1 root root system_u:object_r:var_run_t:s0 0 Feb 24 08:34 /var/run/libvirt/virtlogd-sock
# restorecon -v /var/run/libvirt/virtlogd-sock
restorecon reset /run/libvirt/virtlogd-sock context system_u:object_r:var_run_t:s0->system_u:object_r:virtlogd_var_run_t:s0
# ls -alZ /var/run/libvirt/virtlogd-sock
srw-rw-rw-. 1 root root system_u:object_r:virtlogd_var_run_t:s0 0 Feb 24 08:34 /var/run/libvirt/virtlogd-sock


Any reason as to why this isn't `__init__`?


气体君，双卡机型卡二只能用２Ｇ网络，有考虑修复吗？


@XhmikosR Added!


If /var/run/libvirt is labeled type ```virtd_run_t``` then this interface does not allow the caller to traverse ```/var/run/libvirt``` to be able to get to the ```/var/run/libvirt/virtlogd-sock``` socket.

Instead consider:

```stream_connect_pattern($1, virtd_var_run_t, virtlogd_var_run_t, virtlogd_t)```


> 你好！

hello


Hi baidu,

I understand from this commit that you're using nested functions, am I right?

Could you give an example of dynamic template function?

Thanks a lot,

Alex


This class should be named with Abstract prefix like AbstractHandler, some frameworks (Zend, Symfony, ...)  uses this form to be readable.


As for 1) I did something similar to show my particles. I had 3 colour
channels and they were map accordingly to the weight of the hypothesis.
Basically I had 3 gaussians: one centred at 1.0 (red channel), on other at
0.5 (green) and the third at 0.0 (blue). All of them with the same variance
(0.2 if I remember correctly).
So if your hypothesis had probability 0.3 it would result with almost no
red, and almost same amount of green and blue.

As for the 2), I guess carlos' suggestion of having explaining text is a
great idea!
On 24 Feb 2016 13:44, "Carlos J. Rosales" <notifications@github.com> wrote:

> I don't know about 1), but about 2), surely having a 1X speed video is
> much better...
>
> Perhaps annotating the video can help to distract the viewer while the
> stuff is going on, to also help understand what's going on, phrases like:
> "Note the speed of the GPAtlasRRT candidate point generation"
>
> At the end, when the object is fully covered by the atlas
> "Compare the speed of covering the full object with respect to the initial
> marching cube computation"
>
> no more ideas for now, I need to speed up the writing
> [image: :fist:] hasta la victoria siempre
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pacman-project/gaussian-object-modelling/commit/e118f7c38bfb95be1522de2d8f5af3757e354d8f#commitcomment-16296706>
> .
>


Do we want to reload config on USR1?

Maybe we should do something like:

```go
switch sig {
case syscall.SIGUSR1:
    // dump
case syscall.SIGHUP:
    // reload config
}
```


actually, it's in the scope of another commit of this PR, so, I'll fix it
good catch


@vikashgupta2000 just curious...why is this commented?


paymentplan


Внимание лекция №2:
"Чистый код а) легко читать б) легко понимать в) легко выявлять ошибки г) легко поддерживать"
Пункт 2 твой логгер явно не проходит. Да и 3 тоже (так как один и тот же логгер всплывает в разных местах).
Работает он лишь в виду костыля с отлавливанием Exception везде где только можно.
Внимательно изучила принципы работы твоего логгера. Впредь будь внимательнее к ресурсам и исключениям.


whoops, forgot to change that


ошибка


I know the solution with the uuid is a bit messy, but it works. There are just to many ROI objects in Insight. I couldn't see another way of making an ROI object identifiable without iterating and comparing all shapes, etc. Open for a better/easier solution, if there is one.


This is written using sepolicy generate, so if it is redundant we should fix the tool


btw, the comment is correct, but it's really because of lines 73 and 74


documenta los params, fijate como lo he hecho yo en los otros ficheros


doc


@yuyichao does this look about right to you?


All functions prefixed with `esc_` will provide the user a way to control and manage the ESCs. Actually, the whole file was created for this. These functions will probably use other external functions to perform what they need to do.


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


它


This needs to clarify that it's the algorithm we're currently considering/favouring, without making any commitment to it being the final algorithm.


The only app that will ever connect to virtlogd is libvirtd, and that is already granted permission for /var/run/libvirt, so it only needs to be granted, but yeah, looking again at the args for stream_connect_pattern it should be changed as you suggest


Note that the contexts given to each Enter/Exit method is a class internal to `Generated.TypeCobolParser`.
This is not a problem for rules explicitely defined in `TypeCobol.g4`, but rules defined in `CobolCodeElements.g4` and not overriden would be expected to belong to `Generated.CobolCodeElementsParser` ...


This is not encouraged. Because the caller is not the source of the operation (its the target instead)

A solution would be to use type attributes.

I would also split out the stream_connect rules and put them in their own virtlogd_stream_connect() interface.


I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

See https://github.com/iron/iron/issues/417#issuecomment-170375012 for my opinion.


It's because every individual test is ran in a clean directory, which is created and set as the configuration directory in the setUp() method.


Shouldn't be related to this.


cc @jrdrg


actually, no
method apply from interface `Function` defines method `@Nullable` on return type


Didn't realise - fixed.


In case we hide the esc struct and make it private to the module, the init and free functions should also be private to the module and therefore not declared in the header.


Yes about the table... I'm thinking the results to be divided in two: that and the real trials... probably what you are doing now is slightly more important than the real experiments, which only serves for the demo, and to say that we are able to do it with a real robot.

> * some measure accuracy of shape representation (no idea how i can do this)

The sum of all function values of the ground truth mesh vertices seems fine to me. The lower, the more accurate the model, zero the perfection. What do you think?

---

How hard would it be to have the raycasting synthetic touch to be random over the surface and have the same metrics you mention? thinking of a naive approach to compare to...


An exemple of what partial class automatic code generation would look like, should `CodeElementBuilder` be able to be aggregated into `TypeCobolElementBuilder` ...


TeamCity geo-publisher :: GeoPublisher-viewer [Build 3](http://teamcity.idgis.nl/viewLog.html?buildId=15&buildTypeId=GeoPublisher_GeoPublisherViewer) is now running


Isn't this redundant, since the following while loop will not iterate if the buffer is empty?


TeamCity geo-publisher :: GeoPublisher-viewer [Build 4](http://teamcity.idgis.nl/viewLog.html?buildId=16&buildTypeId=GeoPublisher_GeoPublisherViewer) is now running


I'm not sure what you mean by using type attributes, can you elaborate


ok


TeamCity geo-publisher :: GeoPublisher-viewer [Build 3](http://teamcity.idgis.nl/viewLog.html?buildId=15&buildTypeId=GeoPublisher_GeoPublisherViewer) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:12


missing at least handling for '<' and '>'

```
(big_a < 6 < 7)
((big_a < 6) < 7)
(big_a < (6 < 7))
(5 < (6 < big_c))
```


A sugestão é somente usar um banco de grafos? Porque não o MongoDB ou a mescla deles?


"No newline at end of file", plus i suspect that this also requires usable of the virtlogd_t file descriptor:

if would probably consider the following:

"virtlogd_write_inherited_fifo_files()"

```
allow $1 virtlogd_t:fd use;
allow $1 virtlogd_t:fifo_file write_inherited_fifo_file_perms;
```


Hello Thomas,
It is a first and naive implementation of "first page" logic.
Could you please look at it and confirm that it is the right direction.
Thank you!
@tmorgner


@esoergel This gets used in [is_system_key][1], which gets used in [get_data_to_save][2]. I don't want these values ever to get written. Can I just add them here (looks nice and neat), or should I create a new list for them, and use them in [_validate_reserved_words][3] but no `is_system_key`?

  [1]: https://github.com/dimagi/commcare-hq/blob/7b4fd7751dca6afee9399a2958172861c5a2014f/corehq/apps/custom_data_fields/models.py#L25-L25
  [2]: https://github.com/dimagi/commcare-hq/blob/793295ca084c6e40bd37dc2cdd44f925ffa6a3b0/corehq/apps/custom_data_fields/edit_entity.py#L89-L89
  [3]: https://github.com/dimagi/commcare-hq/blob/7b4fd7751dca6afee9399a2958172861c5a2014f/corehq/apps/custom_data_fields/models.py#L17-L17


Is changed to static_cast


Included in my next commit. My bad!


Never mind. I think I just answered my own question.


Simply generating code as part of the same namespace as code generated for `CobolCodeElements.g4` doesn't do the trick.


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=25)


creationId <<an arbitrary string set by the client>> could be anything. So I guess no.
Ideally, we should always avoid primitive obsession but ... reasons !


FWIW, I have tested this policy locally, and I did not get any AVCs when QEMU wrote to the pipe FD it was passed. What action/usage would the  'virtlogd_t:fd use' rule correspond to ?


Okay my mistake then. I suppose when nothing is specified then it applies to anything


此处译为 “计算属性”


it is better to use array_merge


No worries. Feel free to do later.


Two reasons:
- I tried and it was really painful
- We decided to avoid mocks as often as possible, and to test with in memory implementations when available


> The sum of all function values of the ground truth mesh vertices seems fine to me. The lower, the more accurate the model, zero the perfection. What do you think?

That is what i was thinking too

> How hard would it be to have the raycasting synthetic touch to be random over the surface and have the same metrics you mention? thinking of a naive approach to compare to...

I already done that (that's a few lines of code), the raycasting is still the same and i plan to use it in all the 3 methods. (random-naive,  atlas single touch,  atlas sliding)  And produce data for all of three for n objects (for  which i need a mesh a priori)


Produk yang akan datang maksudnya gimana ya? Saya rasa gak perlu.

Saya mikirnya featured product semacam product yang paling laris/product pilihan, yang akan muncul di homepage.

Featured product kayaknya nanti aja dibuatnya. Perbaiki aja comment2 yang ada di branch ini supaya bisa di-merge ke master. Setelah itu baru kita buat yang lainnya.


We could do that. Anyways, the user will use the functions declared bellow and should not alter the struct by itself. Making this private and using ids will improve user experience but will also complicate things and maybe introduce bugs. I believe it's good to leave this struct public for now.


Yes, I agree.


Oh yes, that's a good idea.


I don't think so https://github.com/rails/sprockets-rails/blob/master/sprockets-rails.gemspec


ok, thank you


@PEMapModder Sorry, what


If this is for internal usage only then it is probably better to not use this interface. Since generally interfaces are for external usage.

This particular interface is not pretty either since it basically deals with two things, that should ideally be split out.


[11:13:13] [Server thread/INFO]: [CONSOLE: Stopping the server]
[11:13:13] [Server thread/INFO]: Disabling DevTools v1.11.0
[11:13:13] [Server thread/INFO]: Unloading level "world"
[11:13:16] [Server thread/INFO]: Stopping other threads
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
The build has been terminated


Final DB File Created.


I recommend throwing an error if the client asks for a viewtype that doesn't exist.


ok, will just inline the rules in the virt.te file then


Python (and most programs?) like a new line at the end of a file.
From Reddit:
https://www.reddit.com/r/Python/comments/1zjugg/question_why_does_pep8_recommend_leaving_a_blank/
"I think the proper way to think of it is not to think of it as a blank line at the end of the file - that blank line appears in text editors, but if you were to open the file in python or most programming languages and do readlines, you wouldn't see any blank line at the end. The newline would be the last character in the last line which would be the line before the apparent blank line."


I think it would be better to check for the request's response code rather than for a hardcoded string. Most probably API is responding with a `2xx` or a `4xx` code when the user is already registered.


Done. I'll go back and look at other nodes' lazy allocation that I've refactored as well.


>MVC is a logical construct though.

I'm not sure I understand what you're even saying here. Also the MVC pattern that Rails developers know is not what MVC was really meant to be.

> What happened to the "fat model" concept?

It lives on in lots of code bases that are terribly painful to work with :trollface:


aaa


Actually, most of the endstop that we have tested use a normally close configuration, with the signal pin pulled up to 5V, so the commit message is incorrect.

The code is still correct.


You probably aren't done yet, but docstrings would be helpful.  PyCharm makes them easy.


bbb


cool


@fdeurioste Hum? Es-tu sûr de cela ? Pourquoi ?

@elliott-iadvize tu confirmes que maintenant il ne faut plus le mot de passe chiffré ?


No because `esc_init()` and `esc_free()` are functions called by the user. The user creates a pointer to a `struct esc` and then uses `esc_init()` to initialize that ESC. Think of `esc_init()` as `open()`.
Also `esc_free()` is a cleanup function that releases any resources used by an ESC. Something like `close()`


TeamCity geo-publisher :: GeoPublisher-viewer [Build 4](http://teamcity.idgis.nl/viewLog.html?buildId=16&buildTypeId=GeoPublisher_GeoPublisherViewer) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:08:51


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=26)


You need to provide more detailed instructions on how to use your user interface. In particular, the interface calls for a Job File to be uploaded, but it is unclear what this should be. Please clarify.


C'est ce que je disais DONC !
On touche a la flexibilité de thématisation.
C'est ce que je disais on peux forcer les colonnes (mise en page) et l'admin mais tout le reste doit pouvoir être customisable a volonté.

Pour le fichier pages.php customisé, je peux essayer de faire cela DES que je rentre de deplacement


please review test


Я бы предложил форматирование всё таки оставить на клиенте.


Muito bom


comment


This is under code review right now. Docker guys would have a better approximate than me.
I would suggest to check with docker guys on #docker-dev irc.


Lets say that virtlogd_t needs to be able to read state of all virtlogd "clients"

Then you could create a new type attribute "virtlogd_client_type"

Then associate the following rules with the "virtlogd_client_type" type attribute as follows:

allow virtlogd_t virtlogd_client_type list_dir_perms;
allow virtlogd_t virtlogd_client_type read_file_perms;
allow virtlogd_t virtlogd_client_type read_lnk_file_perms;

The create a interface "virtlogd_client()" that will associate the calling type with the virtlogd_client_type attribute:

```
gen_require(` attribute virtlogd_client_type; ')
     typeattribute $1 virtlogd_client_type;
```

But if this is only used internally then this wouldnt need an interface in the first place and you could just add directly to the `te` file:
```
allow virtlogd_t virtd_t:dir list_dir_perms;
allow virtlogd_t virtd_t:file read_file_perms;
allow virtlogd_t virtd_t:lnk_file read_lnk_file_perms;
```


Interessante.


Arguments in your docstrings would be great.  PyCharm makes them easy.


@hvelarde it just copy the data if the ``id`` of both ``origin`` and ``target`` tile are the same.
Can you provide a case where it don't work? maybe create a new issue?


нужно добавить хотя бы method


`private`?


type="button" не отправляет форму, замени на type="submit"


Dit transformeert "eMbRaCe" naar "1embrace". Weet je zeker dat je de oorspronkelijke casing niet wil behouden? ;-)


There is not virtlogd_var_runt_t type directory (AFAIK)

Instead `manage_files_pattern(virtlogd_t, virt_var_run_t, virtlogd_var_run_t)` , would be more appropriate and efficient


TeamCity geo-publisher :: GeoPublisher-viewer [Build 5](http://teamcity.idgis.nl/viewLog.html?buildId=17&buildTypeId=GeoPublisher_GeoPublisherViewer) is now running


TeamCity geo-publisher :: GeoPublisher-viewer [Build 5](http://teamcity.idgis.nl/viewLog.html?buildId=17&buildTypeId=GeoPublisher_GeoPublisherViewer) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:10


When I talk about strong typing, I only mean something like ClientId.


I believe we don't need this anymore thanks to //@ngInject


这次的修改不涉及核心的处理。只是一个简单的目标计划。争取早日实现自己的个人博客！


Unless `vkDestroyInstance` (both the julia and the underlying C function) is never going to be used in a finalizer and the user (both high level and low level interface users) has to manually destroy the instance, this shouldn't be the constraint on the argument.


needs an undefined check


исправлено.


DOH!

Thanks *very* much for letting me know. Apparently I forgot to hit the “Sync” button yesterday morning when I posted a brand new version of the generator.

Re-download the entire software. The new version (will say “Updated Feb 23, 2016” in the header) should validate properly,
and includes some minor improvements.

- Canton

On Feb 24, 2016, at 5:47 AM, riberon <notifications@github.com> wrote:

> Hi just letting you know generate-wallet.html.sig no longer validates. I had to checkout 657db38 to verify the commit.
>
> —
> Reply to this email directly or view it on GitHub.
>

--------------------------------------------------------------------
Canton Becker
canton@gmail.com • (505) 570-0635 • http://cantonbecker.com


missing ]


Here it implies, that virtlogd maintains a directory with type virtlogd_var_run_t, but the file context specifications do not back this up.


@adaur


dfghgdgfgdh


TeamCity geo-publisher :: GeoPublisher-viewer [Build 6](http://teamcity.idgis.nl/viewLog.html?buildId=18&buildTypeId=GeoPublisher_GeoPublisherViewer) is now running


No, we don't want that. I have not realized that when USR1 was called the behavior destined to HUP was happening too. And the switch/case structure looks so much better for this situation, thanks a lot!


The code checks against the public suffix list to defend this sort of thing. `uk.com` is one of them. See `com.google.common.net.TldPatterns`


Ignore


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=27)


Are you sure about the ```"relabelfrom relabelto"``` here? Why is that. I can't imagine that this would be needed or even feasible?


Just a question:
Why was 'use strict'; moved out of the define function?
eslint complains about this and it seems uncommon


do you need static_cast<UnitAI*> for anything in particular ?


Можно оставить на клиентов, но тогда придется заморачиваться под каждую платформу отдельно, причем довольно много кода и разных случаев. Я уверен, что мы будем постоянно дорабатывать и тюнить информацию в подзаголовке в PPP, а это гораздо проще делать в одном месте.


vbmnmnfgkjjgghk


@jdubois Need to check it. For some dependencies the resolving is different. The result was different version in maven and gradle (when not defining a version) thats why more dependencies have an explicit version. But need to check it again.


unless Elixir is adding some magic, this isn't true. when your process is restarted, it will have forgotten the items in the cart


Прошу прощения. Не сразу понял что вы имели ввиду именно такой способ работы с файлом. Я думал про другой close(). Открывать, добавлять запись и потом закрывать файл действительно правильно. Собственно я и хотел сразу так делать. Но смутило меня постоянное открытие и закрытие лог файла. Что не очень показалось мне производительным решением.


This is the MI2 xml import utility.  Just a field to import the data to.  I assume that all standard fields were included by rote, but they may actually be getting used with this tool.  I cannot say.  I think it may be used as a continuous import utility for the vendor's client/claim data.


Hi Mariam!


I have no idea; that was almost 2 months ago.


Mental note: `websafe_*` return types before and after this commit.

    Python 2
    Python 3

    Before
    After
    Before
    After

    websafe_decode(b'Zm9v')
    str
    str
    N/A
    bytes

    websafe_decode(u'Zm9v')
    str
    str
    N/A
    bytes

    websafe_encode(b'foo')
    str
    unicode
    N/A
    str

    websafe_encode(u'foo')
    str
    unicode
    N/A
    str


Does virtlogd_t really need to ```write``` to its log files? Ideally it should open the log for "append" only. That would allow use to prevent "a compromised" virtlogd process from erasing the audit trail by using the "write" permission to delete entries from the log


Do you read the documents folder?


Changed to tuple so that it can be used as a default parameter.


Do you really use tracker?


You should investigate using the [`raises_kind`](https://github.com/jandecaluwe/myhdl/blob/master/myhdl/test/core/test_Simulation.py#L53).  You might want to ping @jck on IRC/gitter.  That would be a preferred method, if the error string is ever change it will be a maintenance pain to keep track of the strings in multiple spots.


Github github github


What's wrong with just `false` instead of `sizeof(T) == -1`?


Just checked https://github.com/rails/sprockets/blob/0de36bbee349c66c68656c82c5bc98fbc75b6fed/UPGRADING.md#es6-support :+1:


@HDouss We need it, after removeng - you will get UnusedPrivateMethod violation.


xcvbncbvbmvn


@john-yan Why is this option dependent on big endian?


I may be wrong here but i suspect that the above is redundant and that `logging_stream_connect_syslog(virtlogd_t)` is already part of `logging_send_syslog_msg(virtlogd_t)`


Конкретно в этом случае unicode не выходит за пределы, но момент важный для других случаев, спасибо!


Trying to remember why I added this cast. Not sure.


expectedClassOption.exists { expectedClass =>
  ...
}


We could just as easily use functions with different names like `get_XRPAmount` and `get_IOUAmount`. Then we wouldn't need the static assert, since the functions could never be misused. There really isn't much difference between `get<XRPAmount>` and `get_XRPAmount`.


static const.


ok


Fixed.


I agree.


What are the supported levels and where are they documented?


TeamCity geo-publisher :: GeoPublisher-viewer [Build 6](http://teamcity.idgis.nl/viewLog.html?buildId=18&buildTypeId=GeoPublisher_GeoPublisherViewer) outcome was **SUCCESS**
Summary: Running Build time: 00:06:03


> What does this call do since x-user-path is removed in the previous commit?

This function is only used for the thentos workaround " if not asbool(settings.get('adhocracy.validate_user_token', True)):  user_path = get_user_path(request)" .  We can remove it.


@enzy Why is this not inside of the `class Shapes`? Now you are not able to load multiple shape files.


.getOrElse(None)


Html2pdf issue resolved and now I am able to generate pdf with all checkboxes
https://github.com/practiceprovider/openemr/commit/74605a3dffca26ee79435cb7f6a2aac798936b77


replaced with guid-like strings


Yes, it's uncommon indeed. Nice catch.

We'll be taking a look at why that happened.

Thanks for your contribution.

Cheers.


looks ok


library/html2pdf/README_OpenEMR.txt all steps I did to make it work


If this is a public interface it needs a description


Это означало international ) Я выпилю.


This breaks compatibility with scripts out there. So please don't remove it without providing a grace period with proper warnings for users to adapt.


Is the variable even needed?


TeamCity geo-publisher :: GeoPublisher-viewer [Build 7](http://teamcity.idgis.nl/viewLog.html?buildId=19&buildTypeId=GeoPublisher_GeoPublisherViewer) is now running


Because I got an error about missing 4 bytes while dumping. I did not dig deeper. I expect we'll have to revisit this when the weight is actually used.


you are right


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 346](http://ci.infinispan.org/viewLog.html?buildId=36329&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for d121978f276deab477ac984276c79f0b63914b91
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:02:10

Process exited with code 255


I am testing calendar by updating smarty now at least I am able to see errors


Hej Joakim!

Håller på och försöker implementera  Confirmation page , men får  en Webbexception som säger "ServiceUnavailable" fast jag har rätt  SharedSecret.  Har du något  koll på varför är det så ...  När jag körde  Checkout då fanns det inga  problem...

Tack i Förhand...


TeamCity geo-publisher :: GeoPublisher-viewer [Build 7](http://teamcity.idgis.nl/viewLog.html?buildId=19&buildTypeId=GeoPublisher_GeoPublisherViewer) outcome was **SUCCESS**
Summary: Running Build time: 00:00:40


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 17.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -14,7 +14,7 @@
 object HelloWorld {
   def main(args: Array[String]): Unit = {
     println("Hello, world!")
-    var x = 10
+    var x = 10
   }
 }
 """.splitlines(keepends=True)
```


**Line is longer than allowed. (84 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `bears/scala/ScalaLintBear.py`, line 11 column 81 to line 11 column 85*.


**Line is longer than allowed. (98 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 25 column 81 to line 25 column 99*.


**Line is longer than allowed. (87 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 26 column 81 to line 26 column 88*.


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=28)


Well you guys are huge to us :)


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


Yes that is because fedora policy is very generous with allowing inheritance and usage of file descriptors all over the place (you can use the sesearch command to see if the virtlogd_t fd descriptor is already widely inheritable). Thus it probably is already allowed to use its fd. That is not a good thing though. Especially if this leads on reliance of that policy.


Because it calls exit on errors, which makes the entire script exit, which is what this patch is designed to avoid.
The more obvious alternative to this would be to use a plain subshell:

(sign_domain ${line}) || true

The problem with that is that set -e does not propagate into sub-shells, and even setting it again does not work.
This workarounds these issues, by spawning a sub-process via &, while keeping set -e in effect.


Fixed
Sorry, but had to use eclipse to fix these warnings and got indent mismatch as a consequence ;)


There is a PR for this: https://github.com/flot/flot/pull/1458


Why you would insert multiple shape sprites? This code is prevention from a multiple insertion of the same file.


There are server side changes, which will be making slight changes to the way it is currently now?
The server side changes will come after the OpenMRS merge.
So better not to spend time in fixing these which will anyway change later.
So commented.


I'm getting "TypeError: Map.map is undefined" only on the first call to reCenter,
I'm using it on a bootstrap modal, and call reCenter after "shown.bs.modal"


Ай-ай-ай!!! Что это за горизонтальное центрирование?! У тебя обычный text-center справляется. Удали и больше так не делай)


so this is what “assigning myself an issue” looks like @bjansen? :P


Do you still think that when this is only called from test code to make it easier to visually check test results?

(The reason I chose print rather than log is to avoid 100-char left indents for tag, file, function, line number, etc.)


Could be `static auto const logTable`...


rofl! Embrace moet gewoon altijd bovenaan! Maar 3mbrace was misschien nog toffer geweest..


Used here and not in `_validate_reserved_words` because `_validate_reserved_words` is also used by [`is_system_key`][1], which is used by [`get_data_to_save`][2], which potentially writes to fields in SYSTEM_FIELDS (even though the values remain the same).

  [1]: https://github.com/dimagi/commcare-hq/blob/c273fa5ba0ae43cc5979f1076349edf737a67710/corehq/apps/custom_data_fields/models.py#L26-L26
  [2]: https://github.com/dimagi/commcare-hq/blob/793295ca084c6e40bd37dc2cdd44f925ffa6a3b0/corehq/apps/custom_data_fields/edit_entity.py#L89-L89


@vkuchyn Ok. Let's keep it. Thanks.


Hmm, an `awk` snuck in here? Please revert this change. And when doing it, please amend your previous commit and force-push to your branch so we keep the history clean(er). If you want, while at it, feel free to squash the previous commits all into one.


The main goal of the virtlogd daemon taking over log file writing from QEMU, is to prevent a guest triggered denial of service on the host by writing so much data to the logs that it fills up the host filesystem. As such virtlogd does file rotation, so it inherently needs the ability to truncate the logs.


fixed


po co ten plik tutaj?


Решение в целом правильное, но лучше поправить:
1. Вместо print(" ") лучще использовать "\n"
2. "Каждый файл с решением задачи должен иметь имя task_N_V.py" см. https://askras.gitbooks.io/pythonintask/content/chapter_000.html


qqq


@kasia2504 proszę używaj wcięć tylko z 2 spacjami


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=29)


sure


end


created by 2016-02-24


**Line is longer than allowed. (98 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 25 column 81 to line 25 column 99*.


**Line is longer than allowed. (87 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 26 column 81 to line 26 column 88*.


`static`?


@mada1019 Pretty good already! But I'd suggest to rename e. g. `get_sensor_northeast` to `get_temperature_sensor_northeast`, so the sensor type is specified for all sensors as well.


если конфиги одинаковые во всех less файлах, опиши их в отдельном файле и делай import: http://lesscss.org/features/#import-directives-feature


Yeah, looks like it probably is redundant. Will test it to double check


Основной текст лучше выводить через вызов `print()`, а в `input()` сообщать только о нажатии Enter'а


Thanks for the tips.
On the key binding in the hook: I need it because if I bound the key in `eshell-mode-map` outside that hook I get `Symbol’s value as variable is void: eshell-mode-map`.


mwaaa


Why?


Yeah, that is bogus - copy+paste mistake from the virtd_t rule


Understood.


Hi! Wil there be support for sha2 512? I don't think current SSL certificates work with Sha2 256 anymore.


Choco uninstall doesn't remove them without `--force`


Regexp for path comparision saves a lot of trouble if you have dynamic subdomains (I have them a lot in my examples)


I have many shapes used by only one page. So I have the usual `shapes.svg` for the whole site and an extra `shapes-something.svg` for the single page.


sorry about that :)


Same as everything else in this file.


Nope, initially it was for https://github.com/ceylon/ceylon/issues/6032 :)


Yep, will change it


Yeah, that's a mistake


In the pawns.cpp loop, you can use something like
if (passed)
    e->pawnInterSpan +=14 - 2 * relative_rank(s);

and finish the interspace calculation as we currently do, using the &

But if we use &, in the start position, the average would be 2.

In the course of the game, we would end up most likely with a [0, 2] value which is rounded
in pawns.cpp. That cannot be very useful. Maybe change e->pawnInterSpan  for a single,
or do the normalisaton in evaluate.cpp or do a smaller normalisation.

===============================
So you view this as a tempi reserve...maybe no need to average then...or not so much.

Another view is to see the interspace as the common playground for both players.
Places where pieces can develop frontal attacks against pawns.

The side with the + score might be able to reshape the pawn structure to his advantage.
Either by provoking weaknesses, or my closing the gap (if advantageous)
or better control of the interspace.

Once the gap is completely closed, there is much less ways to improve.

here is an idea
imeasure = (# squares in the "& interspan"
 # squares in front of unopposed pawns
- # squares attacked by all pawns)  / x

where x might be 1, 2 or 4, would have to find out using a sample of many positions.
If we go in this direction, maybe we will not need the asymmetry anymore
and possibly also the # pawns (since imeasure will naturally go down when less pawns on board)..


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=30)


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


typo


@philips any objections?


`static`?


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 347](http://ci.infinispan.org/viewLog.html?buildId=36331&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for d121978f276deab477ac984276c79f0b63914b91
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:02:08

Process exited with code 255


@rimpey this should be pushed to the original PR. So just make the change in that branch, commit and `push origin branchname` again. Close this PR. As this is dependent on the code that has not yet been merged.


Ничего, бывает.
Все что укладывается в 10^8 операций - достаточно производительное решение для лабораторных:)


Compiled code


OK, that's the valid reason to move `spriteInserted` inside a class instance.


Шапка не соответствует требованиям.
Одним коммитом лучше делать одну задачу.
Файлы лучше называть строчными буквами


I think a C++ style comment is better since its less work to edit


The setup file ALWAYS needs to exist.


But why is the prevention from multiple insertion neccessary? Who would initialize `shapes` component twice for the same `data.url`?


Based on the input data, this looked to me like it was a bug in the test that was covered up by fact that unmapped reads simply did not have this flag set.  On the on the other hand, interpretation of ReadNegativeStrand seems dubious without mapping, though setting it in that case seems likely to be benign.


Please incorporate this fix so toshiba air cards will work :- http://www.rc-cam.com/forum/index.php?/topic/4058-marlin-reprap-with-toshiba-flashair-wifi-firmware-patch/


Looks to be code for an EMR thingie.


chodzi o htaccess ?

W dniu 24 lutego 2016 15:31 użytkownik Konrad Głowacki <
notifications@github.com> napisał:

> @kasia2504 <https://github.com/kasia2504> proszę używaj wcięć tylko z 2
> spacjami
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/webkrak/accounting-api/commit/9d3681ec6d676730afe035366cf7ae4ef5de3e65#commitcomment-16297964>
> .
>

--
Pozdrawiam
Katarzyna Sabat


LGTM. Any reason not to just use

    , bool verbose = true )


Hey, very nice, that you improve this hook!!

There is one thing I would like to have in this plugin, but I don't know how to solve it.

It would be really nice, If the plugin is only sending captchas, if Captcha9Kw service is not activated or the credits are empty. My problem is, that I couldn't call the get_credits funktion in Captcha9Kw plugin from this Pushnotification plugin.

Can you help me out?


@trociny can we check for the existence of `rbd` instead check for `uname`. i understand they are basically equivalent. but the former is easier to understand, imho. what do you think?


Just to remind me, strides in VIGRA don't take into account other dimensions. Correct?


This does not look like my code.  My copy of sl_eob_search.php is only 750 lines...and does not have this line at all.  This must be from the !INTEGRATED_AR sections that I chucked out of everything long long ago.  Of course it doesn't work, 'cause sql ledger doesn't.
[sl_eob_search.txt](https://github.com/bradymiller/openemr/files/144765/sl_eob_search.txt)


ok

W dniu 24 lutego 2016 15:31 użytkownik Konrad Głowacki <
notifications@github.com> napisał:

> @kasia2504 <https://github.com/kasia2504> proszę używaj wcięć tylko z 2
> spacjami
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/webkrak/accounting-api/commit/9d3681ec6d676730afe035366cf7ae4ef5de3e65#commitcomment-16297964>
> .
>

--
Pozdrawiam
Katarzyna Sabat


Python logging streams. This feature is for python logging streams so I'd expect those who use them to know them. But I can add it somewhere (eg. `...followed by LEVEL (DEBUG,INFO,WARNING,ERROR,CRITICAL)`)


The problem is that it doesn't work for cyg-get. It expects a cygwin.bat file.


This implements #5


Комментарии лучше писать осмысленными.


Install-bin creates a cygwin.exe file.


.htaccess m.in.: do późniejszej optymalizacji serwisu, plus do moich
pewnych ustawień na lokalu

2016-02-24 15:31 GMT+01:00 Konrad Głowacki <notifications@github.com>:

> po co ten plik tutaj?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/webkrak/accounting-api/commit/9d3681ec6d676730afe035366cf7ae4ef5de3e65#commitcomment-16297948>
> .
>

--
Pozdrawiam
Katarzyna Sabat


prefer using a std::unique_ptr here. Its faster and ```zipArchive``` isn't being shared.


Why not split it ?

```
<check level="warning"
 class="org.scalastyle.scalariform.MagicNumberChecker"
 enabled="true" />
```


Makes sense to me, but this causes three test suite failures: one for scp, and two for sftp. See the Travis test details link for more info.


formatting ;-)
`extends ConnectionStore {`


Ok. Maybe you are suggesting `ArgumentException` :)
I think `InvalidArgumentException` doesn't exist


You also need to remove anything that you put in outside of chocolatey folders, like the cygwin directories in binroot.


Комментарии лучше писать осмысленными


Since the class is called ViewNames then there is no need to name each constant with a View prefix.


Предположим, мы разнесли форматирование, звёзды вынесли в другую строчку на планшетах. Надо лезть и в ядро, и в клиент. Что заведомо сложнее, чем написать пару простеньких строк на каждом клиенте.


to sobie dodaj do .gitignore ;) nie potrzebne jest tutaj w repo


w sublime text mam ustawione 4 wcięcia - dopiero na githubie widać, że są
za duże

W dniu 24 lutego 2016 15:38 użytkownik Kasia <artika30@gmail.com> napisał:

> ok
>
> W dniu 24 lutego 2016 15:31 użytkownik Konrad Głowacki <
> notifications@github.com> napisał:
>
>> @kasia2504 <https://github.com/kasia2504> proszę używaj wcięć tylko z 2
>> spacjami
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/webkrak/accounting-api/commit/9d3681ec6d676730afe035366cf7ae4ef5de3e65#commitcomment-16297964>
>> .
>>
>
>
>
> --
> Pozdrawiam
> Katarzyna Sabat
>

--
Pozdrawiam
Katarzyna Sabat


This one looks better

https://youtu.be/wMcyDo2oTnw

maybe got lucky with the concavity.

I'll work on what is missing:
 - Other synth touches
 - RMSE to measure accuracy
 - Mesh refining, some of them are missing the bottom! and we need it cause it will make them convex


wystarczy 2 ;)


entonces esto que hacemos


Yes I thought about it. The problem is I can't define it in both places (at least I don't know of a way to do this in argparse).


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.61](http://build.clear-measure.com/viewLog.html?buildId=6883&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


@liggit this is a registry client factory passed to resource quota controller.

For now, I'm passing `c.Options.ServiceAccountConfig.MasterCA` in place of registry's certificate authority which is clearly wrong. Need to figure that out.


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 25.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -22,7 +22,7 @@
 config_file = """
 <scalastyle commentFilter="enabled">
     <name>Scalastyle standard configuration</name>
-    <check
+    <check
         level="warning"
         class="org.scalastyle.scalariform.MagicNumberChecker"
         enabled="true" />
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 26.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -23,7 +23,7 @@
 <scalastyle commentFilter="enabled">
     <name>Scalastyle standard configuration</name>
     <check
-        level="warning"
+        level="warning"
         class="org.scalastyle.scalariform.MagicNumberChecker"
         enabled="true" />
     <check
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 27.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -24,7 +24,7 @@
     <name>Scalastyle standard configuration</name>
     <check
         level="warning"
-        class="org.scalastyle.scalariform.MagicNumberChecker"
+        class="org.scalastyle.scalariform.MagicNumberChecker"
         enabled="true" />
     <check
         level="warning"
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 29.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -26,7 +26,7 @@
         level="warning"
         class="org.scalastyle.scalariform.MagicNumberChecker"
         enabled="true" />
-    <check
+    <check
         level="warning"
         class="org.scalastyle.file.FileTabChecker"
         enabled="true" />
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 30.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -27,7 +27,7 @@
         class="org.scalastyle.scalariform.MagicNumberChecker"
         enabled="true" />
     <check
-        level="warning"
+        level="warning"
         class="org.scalastyle.file.FileTabChecker"
         enabled="true" />
 </scalastyle>
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 31.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -28,7 +28,7 @@
         enabled="true" />
     <check
         level="warning"
-        class="org.scalastyle.file.FileTabChecker"
+        class="org.scalastyle.file.FileTabChecker"
         enabled="true" />
 </scalastyle>
 """.splitlines(keepends=True)
```


??? Tabular or Schema???


No it can't be done from GitHub. Would you mind doing it locally as this is the preferred option over creating new pull requests? (Make sure to re-open the pull before force pushing)


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=31)


related to #3


sortOffer ??


ok, w wersji na github dam 2,

2016-02-24 15:41 GMT+01:00 Konrad Głowacki <notifications@github.com>:

> wystarczy 2 ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/webkrak/accounting-api/commit/9d3681ec6d676730afe035366cf7ae4ef5de3e65#commitcomment-16298265>
> .
>

--
Pozdrawiam
Katarzyna Sabat


@liggit And here's the same for the resource quota controller (managing OpenShift resources).


I wasn't sure if that would cause problems for clients that expect the two parameter version or if they would have to be recompiled.


I am not really familiar with the matter but, i get the impression that this applies to "svirt_t", if this is true , then it should, probably, apply to all "svirt types". I think the policy provides interfaces that allow one to create new "svirt" types, instead of only svirt_t. Not sure what that type attribute identifier is but i suppose it would be something like "svirt_domain_type". in that case the above should probably apply to "svirt_domain_type" instead of just svirt_t.

Not sure though , and I am also fine with just "svirt_t". Just noticing it.


findBy ??


Collaborator:
A review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=32)


ok

2016-02-24 15:40 GMT+01:00 Konrad Głowacki <notifications@github.com>:

> to sobie dodaj do .gitignore ;) nie potrzebne jest tutaj w repo
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/webkrak/accounting-api/commit/9d3681ec6d676730afe035366cf7ae4ef5de3e65#commitcomment-16298245>
> .
>

--
Pozdrawiam
Katarzyna Sabat


Hi fraser1978.

I've updated the readme for the XBMC plugin at - should allow you to get started
https://github.com/kevjs1982/xbmcplugins/blob/master/README.md (Alas my Kodi/XBMC installs are a couple of years old so these haven't been tested with the latest ones).

The web-ui requires knowledge of setting up webservers and PHP which is a bit on the complex side.


I don't understand the question. Each axis has a stride that specifies how many array elements must be skipped in memory when the corresponding index is increased by one. So, the stride of the y axis typically depends on the shape of the x axis, but the stride of a singleton axis is in principle arbitrary. However, zero strides cause various problems (for example, one cannot create an iterator pair that represents an STL range of length one).


Hi,

This change breaks the classic ui which needs to read the status.dat file (from StatusDataWriter) in /var/cache/icinga2 . The web server (apache) needs to reads this data but since the file is is now 0600 and not owned by the web sever user (usually something like www-data vs nagios) it fails.

Do I need to create a ticket for this?

Bart


This shouldn't have been committed?


@PEMapModder check this? :)


how the "_" translation function can still work?


i can change it, but it'll be necessary one more layer to expose the socket. Tell me which real cases this would be bad ?
AFAIK iotivity also does it.


these should still be size_t


big tab stop


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.61](http://build.clear-measure.com/viewLog.html?buildId=6883&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 85, ignored: 13 Build time: 00:05:29


@tchaikov My another commit f9435bcf makes rbd built on all platforms but without linux kernel specific features (i.e. rbd will be available but missing `rbd map|unmap|showmapped`). I could check if `rbd unmap` were present, e.g. running `rbd help map`. Not sure it is better but will change if you think so.


it's not in your global gitignore config?


這裡為什麼不是用之前那個檢查 null 的方式？？ 那如果 actionbar 是null 就會 crash？


@PocketKiller I wrote it on my way home!


Shouldn't this be `isinstance(data, text_type)`? Where text_type as the same meaning as [six.text_type](https://pythonhosted.org/six/#six.text_type).

In other words why is it necessary to run `data.encode` when `data` is a `str` on Python 2?


http://www.cplusplus.com/reference/cstring/strncpy/

>destination and source shall not overlap (see memmove for a safer alternative when overlapping)

Btw. you change semantic here!


And what about anonymous classes @benzonico ? NPE.


separate if and parenthesis '(', add newline after if, so --target_idx; isn't inline.


nevermind


This could have just been a regular member function. And then we wouldn't have to rename variables like `cache_`


I would have put the return type on its own line


@hannesmuehleisen  ok to integrate this?


Почему сложнее? В описанном тобой случае нужно будет удалить кусочек текущего кода в ядре, и восстановить старый код форматирования в клиентах.
Или для удобства добавить int GetStars() в place_page::Info().
Но пока такая задача не стоит, UI получается вообще очень простым в разработке поддержке.


@eljefedelrodeodeljefe, yes, almost done. Pretty happy with the result. Thanks again for kick-starting this effort and for your significant contributions to the code base.


Newline before return;


Do we need the whole `Logs` or can we just pass a `Journal`?


that one looks really impressive....

> maybe got lucky with the concavity.

is it a matter of the target values one gives to the external points?

> RMSE

Yep, we should be using the language of regression analysis... https://en.wikipedia.org/wiki/Errors_and_residual

I think what I said was actually the SSE (sum of squared errors), but without the squared.

> Mesh refining, some of them are missing the bottom! and we need it cause it will make them convex

For now I think we can take only those with a bottom


extraneous semicolon


Start function block in new line.


Separate if and parenthesis.


the route /web/webclient/qweb shoudl also be removed


You should write your `if` statement as in `if... else`. If your case it would be

```
var thresholdRides = 12; // capture this in a meaningfully-named varible

if (numberRides > thresholdRides) {
  alert("Buy a monthly pass, why don't ya??")
} else {
  alert("Just pay per ride!")
}
```


Probably this default value should be endOfMonth


Just debating semantics -- totally understand your points. Thanks


OK, seems good. I don't think we need the print statements at all. But if we do have them, then they should be marked for translation. i.e. _("Done").


Love it - this is awesome. Can you add on src/CMakeLists.txt:37 a way to enable just that platform and I'll merge it.


[feature4 8](https://github.com/zapelin/collab-github-v4/commit/ff646f2b6bf102f200390cbe680c8e992cd57675)


extract constants for 5000 and 100 and reuse them


This should not be neccessary, and potentially harmful causing render loop to trigger more than neccessary.


Is `operator==` private? Does this mean only a `Step` can compare `Step` for equality?


I think it would ultimately be more efficient to pass by value and use std::move in the ctor. This would, at least, allow temporaries passed into the ctor to be efficiently moved.


no habrá un nombresito más genérico para `data`? `treeData` quedaría mejor. Pueden haber varios resolvers y luego no vamos a saber diferenciar entre datas


Put that dir into your .gitignore ;)


Don't do random things like this; you'll break stuff


Gulp kicks in to let me compile scss on the fly and use it while developing, 'cos i hate plain css so much.


`treeModel.rootId = treeData.meta.root_id;` podría ser


change the year to 2016


Surely this function must already exist, perhaps `accountHolds`?


We need to review this code. This should not be part of ExtensionMessageSource only. The infrastructure to manage lifecycle of a failing message source needs to be part of the runtime


Start the function block at new line.


You are totally correct, i was fiddling around in the wrong branch, it was fixed in a later commit though.


Yes, it seems tmp_ptr and line_data_buf may not be the same, as str_trim() takes a pointer to pointer.


Consider renaming the file and class name as PluginNames is not clear what it contains. This file is in a folder named MVC, which does not makes sense as it is not related to MVC in any way.


For the record, I had this error:

```
python /opt/range_repair.py -v -d -D myDC -H myHost -k myKS -c myCF --par --steps=10 --workers=4 --dry-run --logfile="/opt/range_repair.log"
Traceback (most recent call last):
  File "/opt/range_repair.py", line 400, in <module>
    main()
  File "/opt/range_repair.py", line 396, in main
    repair(options)
  File "/opt/range_repair.py", line 277, in repair
    tokens = Token_Container(options)
  File "/opt/range_repair.py", line 37, in __init__
    self.check_for_MD5_tokens()
  File "/opt/range_repair.py", line 74, in check_for_MD5_tokens
    if not self.ring_tokens[0] < 0:
IndexError: list index out of range
```

In debug I had the following output for all the nodes and tokens:

```
2016-02-24 07:52:31,663 - DEBUG - Discarding node/token x/-9219728190764740185
2016-02-24 07:52:31,663 - DEBUG - Discarding node/token x/-9216542687938394223
2016-02-24 07:52:31,663 - DEBUG - Discarding node/token x/-9214874184734139893
2016-02-24 07:52:31,663 - DEBUG - Discarding node/token x/-9213130010355246186
2016-02-24 07:52:31,663 - DEBUG - Discarding node/token x/-9203020322850175615
2016-02-24 07:52:31,663 - DEBUG - Discarding node/token x/-9202837115857321087
```


We don't need the AuthManager cos the UserData is inside the state. am I ok?


as a point of "code cleanliness" consider whether the copy and assignment operators need to exist or not. They should be removed if they aren't required.


Well I can do a special magic:

```diff
diff --git a/avocado/core/parser.py b/avocado/core/parser.py
index e759dfe..b0f11d8 100644
--- a/avocado/core/parser.py
+++ b/avocado/core/parser.py
@@ -61,6 +61,13 @@ class ArgumentParser(argparse.ArgumentParser):
     """

     def error(self, message):
+        if message in ("unrecognized arguments: -s",
+                       "unrecognized arguments: --silent"):
+            import logging
+            msg = ("Option -s|--silent was moved from 'avocado run -s' to "
+                   "'avocado -s run'. Please adjust your scripts.")
+            logging.getLogger("avocado.app").warning(msg)
+            return True
         msg = '%s: error: %s\n' % (self.prog, message)
         self.print_help(sys.stderr)
         self.exit(exit_codes.AVOCADO_FAIL, msg)
@@ -133,6 +140,7 @@ class Parser(object):
             msg = 'unrecognized arguments: %s' % ' '.join(extra)
             for sub in self.application._subparsers._actions:
                 if sub.dest == 'subcommand':
-                    sub.choices[self.args.subcommand].error(msg)
-
-            self.application.error(msg)
+                    if sub.choices[self.args.subcommand].error(msg):
+                        break
+            else:
+                self.application.error(msg)
```

But the problem is when silent is used, the output is not printed. So even the message is not displayed...


the problem is line_data_buf and tmp_ptr work on the same memory. As str_trim alters the ptr to the ptr which is again line_data_buf.


Then i suspect that the `connectto` permission may not be needed there either. Unless it stream connects internally which is not all that unusual.


Great,
this codes works. Thank you so much.
Do you have information that Gateway send some responses ? Can we send to gateway information for static ip ? Do you have information when we send a code for ControlUp that device before is off ?

Best Regards,


> is it a matter of the target values one gives to the external points?

Actually i dont know, I'm already scaling the external point targets, to their distance from the surface, It looks like that when you add any external point too inside the concavity, gp gets mad.

> I think what I said was actually the SSE (sum of squared errors), but without the squared.
I can produce MSE and RMSE easily (pcl has methods to do that), for SSE I would need some copy pasting of code, but it's still a matter of minutes. What do you prefer then ?

> For now I think we can take only those with a bottom
 AH, you wish it :D, ALL those with a bottom are CONCAVE :palm_tree:


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


No I don't, we also don't need network for the moment.


Yes.. this is redundant and can be removed.. the below while loop will not iterate and return 0, if the oa buffer is empty


You could have done something where you do not build but just warn if the required binaries are not in the search path ...
which genwqe_echo
if [ $? -ne 0 ]; then
    echo "Ensure that you have all the pieces together before you use this ..."
    exit 1
fi

Anyways, it is a testscript ...


You should report it as an Emacs bug. I don't use eshell, so I'm not that interested:)

> (defvar eshell-mode-map nil)

This isn't the way keymaps are defined these days.

> (defvar eshell-mode-map (make-sparse-keymap))

This is the recommended approach that allows to bind the keys just once, after `eshell` is loaded.


Is there some way to fire non-sandboxed applications from a sandbox? It is required for Steam support.


I understand. I mean I shouldn't remove the strncpy() completely, just replace it with memmove().


"In American English, label makes labeled and labeling, with one l. In Canada and in varieties of English from outside North America, the preferred forms are labelled and labelling." - http://grammarist.com/spelling/label/

Are there guidelines about the Rails documentation stating if American English has to be used?


esto es un GET