It'sa good practice to use enums instead of hardcoded literals.


Sorry about that, still learning the tools VS 2015, GIT, tortoise Git, etc.
I used notepad++ to show me all the characters and replaced
"\r\n" with
"\n"
Now all the files I touched have "\n" and no "\r\n"


Hi Lukasz, I've finished the round 1. of review :)


сделал через require. Этот validate тоже тестить надо, так что наружу он должен торчать как-то


вот вопрос — оставлять ли className. Это по идее не BEMJSON


inspect/2, right?


I''m not sure that makes much sense in this context. Usually we throw console warnings to inform developers that they should update their code to account for API changes. In this case, however, they've already made the code change by updating this file. That means the only person the warning would benefit are users, and I don't think they're looking in the console for browser update advice.

As such I think the comment is sufficient to denote an older code path that should be removed once the browsers supporting it have been fully phased out.


Proverka


first


Can you explain more?


Doh! Thanks - have committed the other file too


а что за "vectpar"?


Updated to test


So far, we could change the 180s to turnAround, but the 30s were just a random number I put it because I don't know how we are going to turn.


Hi! It's never a good idea to post your api id and hash online. I believe there's also a warning on my.telegram.org when you create them.  Please consider removing them.


BEST :) . Don't forget to merge :)


RobotConfig robotConfig = robotSubsystems.getRobotConfig();
AutonomousConfig autonomousConfig = robotConfig.getAutonomousConfig();

travelLength = autonomousConfig.getTravelLength();
caution = autonomousConfig.getCaution();
... etc - so it is a single method call on an object vs repeating the full get() path to the object


lol great wisdom!


schoolname -> schoolName


then put the random number in a config and a property file.... just like the others.  we may change any of the values, and want all of them consistently stored.


This does not work for me. The jquery.dataTables.min.css should be included before the dataTables plugins.


If you mean the extra newlines everywhere, I agree that it looks a little odd to me but that's the three.js style and it's best to follow it.


There's one more typo on this line. Hint: --falvor


https://github.com/symfony/symfony/blob/master/UPGRADE-3.0.md#yaml

> Starting an unquoted string with @, `, |, or > leads to a ParseException.


@mark-rushakoff is there a case for converting this into an Add and taking it out the scope of the write lock?


I'm not sure what you mean about the indentation, but this code is in the `if (vrInput.getPose)` condition, so pose should be always be defined.


miss spelled the "canola" oil.


А как же группа, e-mail? В about всё?


KY32_W would be the correct style endpoint label for a child/bannered route


@gr2m I've added the logo and a text. Feel free to update it.


OMG! Thank you!


`n` might be better called number_bids, but more importantly, both number variables should be unsigned?


Actually in this case we should have had a preceding space as well to remain consistent with three.js style.


Not exactly. I have described what I'm doing in this DTO overhaul in the first of these commits: https://github.com/rithms/riot-api-java/commit/a9f76b18ae8c9d2f67c29eee9a3059132da48503

The main goals were to normalize some data types (e.g. champion IDs always being int), and find out which new stuff we might be missing and which old stuff we have doesn't exist anymore.
So far I have already found quite a few variables in various endpoints that we didn't support yet, aswell as some variables we have, that don't exist anymore in the official Api (probably leftovers from older versions of the endpoint?)

While I'm at this, I also clean up the files by ordering the variables and methods, but have a look at the linked commit message.


done :)


Can you introduce a macro for the 25 with an appropriate name? The magic number is otherwise a bit unmotivated and random.


#1 Closed.


I will try :). It's true it's not something that i look often. My bad ;)


Yes you're right. My bad. I will fix this.


@XhmikosR Would you mind giving this a review? https://github.com/gruntjs/grunt-contrib-internal/commit/e8a71001196c157487d66db5834bedc192a14403


I believe you'll still want to rename this to not include the 'InFreeRCT' bit, since, as stated previously, this code resides in the freerct program. :)


Looks okay. Let's not merge yet. Will merge with my branch which has some login endpoints first to test.


OK


@hsbt

the change `early proposed draft of a CoC for PostgreSQL` -> `early draft of the PostgreSQL CoC`
reverts the recent commit 1a72163c59579b9f35c306b4df088440d8ffb5bc from #1300. Was this intentional?


    The change to the global value of company-backends will have effect in other buffers .

Indeed you are right. I've update documentation.

    I'm not sure what you mean.

Maybe [this picture](https://github.com/proofit404/company-anaconda/issues/15#issuecomment-186555248) can clarify this a bit.


This is my first website.  I am learning through Treehouse.com.  My wife is pregnant so I wanted to create a website to showcase her pregnancy and the growth of the baby.  In the ABOUT page, javascript pulls from a website the current week's image and information by using an algorithm based on the current date and the expected due date to determine which week of pregnancy we are in.  PHP is used to send a Guess the Date game submission or a Student Loan Donation email.  I have yet to reach the PHP section in Treehouse so I had to read multiple forums and blogs in order to learn enough about PHP in order to get it to work.  So far everything seems to be working well and we are receiving game submissions and loan donation emails.  Website is responsive to fit Iphone, Iphone horizontal,  tablet and computer monitors.


@mark-rushakoff  was wondering if this could be taken out the write lock as per WriteMulti()? On the other hand being outside the write lock is not ideal as some updates to the value will get lost, especially if we use Set rather than Add to manipulate the counter. Perhaps we should move the update outside the write lock in both case, but do the updates with Add rather than Set?


I think you're reading it backwards. `deprecatedAPI` indicates that we're using the old API, so `!deprecatedAPI` means that the new calls are available.


That was one of the things addressed in a "comment incorporation" commit that was squashed onto the last one. The new name is simply "GetColor".


why do you assign gmTES5 to wbGameMode in this function call?


> Maybe this picture can clarify this a bit.

And using `:with` somehow makes a difference?


It's done in  https://github.com/kevincaradant/web-template/commit/79fe9b9544f038242ad2c86f130beb82d09e8ba8


just for clarity, you should define a local variable plugin: TPlugin and then assign it to TPlugin(PluginsList[modIndex]).  That will shorten this line of code so it (hopefully) doesn't need to be wrapped.


Not a big deal, but there appear to be two spaces between `PaletteColorIntensity` and `n`.


Yes, it's the best way to get page number for correct order because it has nice url format `/fairytail/vol55/464`. I'm going to parse chapter number from url for 3rd site too so it's better to move them out from here, but i'm new to android and java so don't how to do it correctly. I want it to try parse by url and in case of error, parse by name with this recognizer.


Very difficult assignment. Will not be continuing specialization.


**A note for future reference:**
I go against convention and like to put begin on the same line as the condition/loop.  That's how all code in my projects is formatted.  If it's not too painful, it'd be nice if PRs could follow that format as well.  :+1:


We know that the return type of the closure should coincide with the map's value type


ping @asgeirrr @don-mums
Sepsal jsem kód a osnovu. Aspoň půlku plánuju projet teď v pondělí v Brně (se třídama), takže to možná ještě poupravím.


@sseemayer Isn't this a bit to much? I would rather say 7 days is enough. Also there was the idea to limit Let's encrypt certs to 30 days if I remind the official blog posts correctly.


this is the important part, i.e. checking whether the value should be stored or not


Nevermind I thought it was a thing but It seems I'm wrong ;)


Value space of datatypes is defined by [XSD Schema](https://www.w3.org/TR/xmlschema-2/#value-space). For _float_ and _double_ they overlap, so would seem be be comparable. However, _decimal_ (and therefore _integer_ is defined differently, so I think that Integer("1") != Double("1.0"), but I could be wrong. @bendiken may have an opinion.


Hey! A long time has passed, but when you get a chance could you push 1.6.4 to Rubygems (or give me an idea of what needs to happen for this to be ready to push)? The [latest](https://rubygems.org/gems/treetop) at time of writing is still 1.6.3.

Thanks!


I usually print a tracker message when doing an operation that could potentially be large.  In the context of the TInitThread tracker messages are shown on the splash window.  So you could add here:
`Tracker.Write('Loading BSAs');`


Since there is only the one await in the last line of the method, you could remove the 'async' modifier and simply return the Task that is returned from _bus.SendAsync(). That would be more efficient. (Yes, this is a small knit).


не, просто `stringify` и `parse`, `classname` уже и так в названии пакета.
у babel главный API метод называется просто `transform`.


should this happen after BSAs are loaded, perhaps?  To keep the flow of the code a bit more logical?  Also, because this is an atomic operation (addition of a string to a list) it shouldn't have a tracker message.  The message would never be drawn on the screen because we'd hit the next tracker message before it could be drawn.


You can use "else" instead of "case NONE() then" to avoid additonal check of "optionNone()"


не, ты не понял — я имею в виду в объекте, который передается для `stringify`


он отлично тестировался и инлайново, косвенно


In the context of this function you sohuld be using Logger.Write instead of Tracker.Write.  In the context of the TInitThread tracker messages are only for printing to the Splash window, and aren't saved anywhere.


Here, we should keep this method async, and change the call to _context.SaveChanges() into a call to_context.SaveSaveChangesAsync();  If I read the rest of the method correctly, we should await that task before sending the notifications, to ensure that the removed volunteers have been removed from the DB.


ну, можем сделать инлайново, только тогда нужно будет в тестах ENV менять


No. Actually it make no difference - that what I mean saying "it will not help". But I completely forgot about global priority effect.


Here this could be a synchronous method that returns the task from _bus.SendAsync(). (In the above code where it returns early, simply return a completed task (which you can create using Task.FromResult(true); )


looks good.  I'd add a comment to explain what is happening here. at the top of this code block (so before assigned `sMyDocumentsPath`).  You'd want to saying something like `// set game ini path`


Díky Petře, začnu to studovat...


Ah, sorry about that. I shouldn't have left this here.


This should be a Task returning method, if it is calling an Async method. Async Void is Bad!


You may also append to list:
NOA   http://eida.gein.noa.gr


Could be synchronous Task returning method.


While writing an example table, I came across a "pagination" menu, which had anchors as direct descendents. Do you remember why `&& !this.context.isMenuChild` was originally added? Maybe `this.props.link` should override and create an anchor anyway, or we need to add an anchor to the div / add a click handler if link is true.


Looks good.  Code review done.

Thanks for the PR!  :smile:


lol


This confuses me. We are not awaiting the SendAsync async command, but we are creating a completed task below it. If there's something I'm missing, a comment as to why would be really useful.


Was missing set expandtab in vimrc
Fix files with :retab


develop (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


This assert should be fixed into a proper exception now


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: templates/getCheckbox.phtml


same here


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Tutorials/Linter_Bears.rst`, line 367.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Tutorials/Linter_Bears.rst
+++ b/docs/Users/Tutorials/Linter_Bears.rst
@@ -364,7 +364,7 @@
             :param max_line_length: Maximum number of characters for a line.
             :param tab_width:       Number of spaces per indent level.
             :param pep_ignore:      A list of errors/warnings to ignore.
-            :param pep_select:      A list of errors/warnings to exclusively
+            :param pep_select:      A list of errors/warnings to exclusively
                                     apply.
             """
             return self.lint(filename,
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Tutorials/Linter_Bears.rst`, line 376.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Tutorials/Linter_Bears.rst
+++ b/docs/Users/Tutorials/Linter_Bears.rst
@@ -373,7 +373,7 @@
                              select=pep_select,
                              max_line_length=max_line_length,
                              indent_size=tab_width)
-
+

 In the example above, things to note are:

```


iteration (evanweaver) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


from now on,  this would close all windows and eventually close X, if a there's an "exec some_terminal" in .xinitrc


iteration (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


@llopv, please check if commit https://github.com/P2Pvalue/teem/commit/3ea727c572cac73654d875221c86ba868d7f22da fixed the issue. The promise to wait is the session, and not SwellRT.


I believe "John Deeney should be higher up on the license"


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


Should Out.txt be part of the repository? It is generated by javac and the "make clean" target removes the file.
For similar test cases Out.txt is not in the repo and instead added to .gitignore.


icos


space before `(` missing


comma missing after `bears` before `can`


no comma here AFAIK


Hm, I think this should go into an own file and should be a bit more extensive, you should definitely explain that lint e.g. should return the new file content or how it works with stdin and stuff like that. Feel free to open a new bug for this.


Okay. Will make these changes.


Credit to Fenix for suggesting that there may still have been an issue with the previous fix to the population and infantry enrollment issue.


a fun one is to use unsafePerformIO at the top level to make the config seem pure --
```haskell
config :: String
config = unsafePerformIO $ getSqlServerCommand "db.cfg"
```

The config shouldn't be changing throughout the lifetime of the program so this is 'safe'.


Thought about it, IMHO makes sense there, too (sort-of a bugfix after all). Cherry-picked.


Done.


?? `this.type = null` then ??


yes, you can do this:

```
environment:
  matrix:
  - nodejs_version: 5

platform:
  - x86
  - x64

install:
  - ps: Install-Product node $env:nodejs_version $env:platform
```


Already fixed, could be discarded.


This is a MIND MAP drawing showing the architecture of the project. Like a brainstorm on a napkin, this is an evolving bird's eye view.


@himanshu188  Add your solution in the solution folder in palindromic crypt folder.Make one solution to problem 1 by including all the options into one solution.


[Week 3 Log.pdf](https://github.com/AmaroReynier97Portfolio/Amaro_Reynier_Portfolio/files/139789/Week.3.Log.pdf)


I followed the approach in the existing code - See the string "server" right above that is also only defined, then used once. One could think about refactoring this, but I didn't want to mix up refactoring and the IPv4 feature in one contribution.


I'd suggest to assign the variables directly, instead of in the `if` clause


I think the old loop looped backwards so the first location that is not null would get returned, which corresponds to the spawn priority. I would suggest to loop forward but to extract the handling for each `priority` into its own method—if it returns a Location (and not `null`), just return that. I can do it if the description is unclear, @DNx5
Otherwise looks great! :smile:


Here are the links for the resources I used in my discussion post about a famous quote from Bruce Lee.

Link: https://www.youtube.com/watch?v=FlMX6kc1bqA

Link: http://web.a.ebscohost.com.oclc.fullsail.edu:81/ehost/detail/detail?sid=e8bf4243-ae4d-418d-9241-c660d8b5723f%40sessionmgr4005&vid=0&hid=4114&bdata=JnNpdGU9ZWhvc3QtbGl2ZQ%3d%3d#AN=21987350&db=a9h


This has alot more swag.


May want to revert this commit to avoid changing the main readme.md


@anne-france , je sais pas ce que tu en penses de la manière de définir le type de on. Je penses qu'on peut faire mieu


It looks like campaign generation was already creating rows in the smtp table, but they were never used elsewhere. This db migration should import those campaign -> smtp relationships into the new table.


Got it. Going to update README.md shortly with instructions.


T A B S
A
B
S


Thanks for fixing all these issues. Tested on my Chromebook and it works well. (CPU usage is indeed very high)


https://github.com/ivanich/android_system_bt/commits/cm-13.0
not sure which exactly commit from caf fixes bt pairing


docker和Jenkins不是什么新东西了，两者结合也不是什么稀奇的事情，也已经有很多Jenkins和docker相结合的文章，此文仅为自己的一点心得实践，如有不对的地方，欢迎大家纠正。

先贴上大致的流程图，逐步说明：

代码-Git：
并没有什么好说明的，就是简单的使用了Git作为版本控制工具而已，通用使用规范不在细说。
此步的产出：
Git分支特定版本号

Git-自动构建、自动构建-代码包：
做法也很通用了，将project的Git钩子同Jenkins结合，达到特定分支有push时机触发自动构建，将代码包从Git拉取并打包为代码包。

此步产出：
打包好的代码包：project.tar.gz

代码包-Docker镜像
在此步中，我们为每个project提供特定的测试环境，并且在此环境中执行项目代码镜像打包操作。在此步中，需要提前准备几样东西：

测试环境：我们这里为一台干净的服务器（不要再问好奢侈，有钱就是任性），部署docker环境；
project的base镜像：对于一个成熟的项目，所依赖的环境是固定可知的，因此提前准备好其所依赖的base image是必要的。
如，我们一个项目的base image的Dockerfile:
FROM centos:liuyanglong
MAINTAINER liuyanglong "liuyanglong@xxxx.com"
MAINTAINER version "online"

USER root
ADD php.ini /home/work/local/php/etc/
ADD php-fpm.conf /home/work/local/php/etc/

ONBUILD ADD project-code.tar.gz  /home/work/
ONBUILD ENTRYPOINT ["supervisord", "-c", "/etc/supervisord.conf", "-n"]
注意最下面的两行ONBUILD
而在每一次Jenkins的构建时，要做的仅仅是将代码包传入，并且执行docker build即可，此时build所使用的Dockerfile的内容只有一行：

From this_project_image:base
而执行build时只会根据base image中的两行ONBUILD执行两个命令：

ADD project-code.tar.gz  /home/work/
ENTRYPOINT ["supervisord", "-c", "/etc/supervisord.conf", "-n"]
注意：此步仅仅在测试服务器做了docker build操作，并没有执行docker pull！！

镜像打包完毕后，此步并没有结束！！

调用脚本，根据此构件号的版本docker image创建对应的容器，脚本的输出为其对应的访问方式，供QA同学测试使用。
这样，每个构建好的版本都有对应的测试环境，且互不冲突！
鄙人的脚本地址为：https://github.com/Liuyanglong/docker-tools/blob/master/create_docker

此步的产出：

docker build成功的project image，如以构建号为image版本号，叫做： project_dev:530
此版本代码的测试环境地址，如：172.30.40.2
生成线上镜像
到上一步为止，测试构建环境已经结束，当QA同学确定要上线时，执行Jenkins的Promotion操作，这时触发 此步，将对应build版本对应的docker镜像推送到 私有docker registry。
所执行的操作自然为 ：

docker tag project_dev:530  docker-registry.xxxxx.com/xxxxxxx/project_name:version
docker push docker-registry.xxxxx.com/xxxxxxx/project_name:version
此步产出：
push好的线上镜像

AB上线
此为最后一步，同样是执行promotion操作后最后所执行的步骤，调用我们的内部接口，对线上应用执行AB上线，具体可参见文章：http://segmentfault.com/a/1190000002978115#articleHeader6

总结
上述就是我们在生产环境中的使用Jenkins和docker所构建的持续集成&自动部署的逻辑架构。也欢迎各位大大拍砖指教。


These changes look solid at first glance, though not being as familiar with Python as you, I'll need a bit of time to make sure I understand it all. I'll merge it in once I verify everything works as intended. This is so much more pythonic than my C-like way of handling everything.


No I meant how to do that but test the other versions with x86 only. Is
that possible?
On Feb 21, 2016 20:28, "Feodor Fitsner" <notifications@github.com> wrote:

> yes, you can do this:
>
> environment:
>   matrix:
>   - nodejs_version: 5
>
> platform:
>   - x86
>   - x64
>
> install:
>   - ps: Install-Product node $env:nodejs_version $env:platform
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/addyosmani/grunt-uncss/commit/ca2df0914f5429fe0b8342f8516a49955e356535#commitcomment-16228285>
> .
>


Thanks... I suggest this is indeed better...
Can you please remove the extra blank lines.... (like e.g. line 202,183 enz...)


Right now all interface_type are set to "SMTP". In the future this can be used to route campaigns through senders other than email library/SendWithTLS


Define all permutations then:

```
environment:
  matrix:
  - nodejs_version: 5
    platform: x64
  - nodejs_version: 5
    platform: x86
  - nodejs_version: 4
    platform: x86
    ...
```


Changing it back to LINEAR still makes it crash -

Context reset!
anv_device.c:405: FINISHME: Get correct values for VkPhysicalDeviceLimits
anv_descriptor_set.c:328: FINISHME: VkDescriptorPool is a stub
retroarch: anv_cmd_buffer.c:353: anv_cmd_buffer_emit_state_base_address: Assertion `!"unsupported gen\n"' failed.
Aborted (core dumped)


oh. ok.


On mobile, so I can't test. It looks like this will remove SQLite from the Python build. But SQLite is typically available to Python interpreters and is required to run the examples.

Is there a specific bug or feature this patch addresses?


https://github.com/jembi/openmrs-module-emtfrontend/commit/160b39991c39e6d4a2e3be096f9909ea12a5843b does the same thing :+1:


hi bjimmykim, I think you are on the right way of the answer. It is suppose to be a (mixed ) string of 40 characters with numbers from 0 to 9 and letters from A to F.


Why remove `refresh` key from the `kwargs`?


Nevermind, I didn't realize you could leave a space between a function and parentheses, `raw_input` and `()` (e.g. I've always seen it as `raw_input()`.


Could you check with a debugger what exactly the input is in CommandLineTools.checkText()? For me, it's `This is an test.\n\nThis is a test of of language tool.\n\nThis is is a test of language tool.\n`


Why swallow exceptions?


branch delete


There is no need for logging here. Throw RuntimeException with the message and original exception wrapped.


I will test it in the upcoming week :)


try


Why is the test disabled?


You are right, thanks! I'll try that later.
On Feb 21, 2016 20:44, "Feodor Fitsner" <notifications@github.com> wrote:

> Define all permutations then:
>
> environment:
>   matrix:
>   - nodejs_version: 5
>     platform: x64
>   - nodejs_version: 5
>     platform: x86
>   - nodejs_version: 4
>     platform: x86
>     ...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/addyosmani/grunt-uncss/commit/ca2df0914f5429fe0b8342f8516a49955e356535#commitcomment-16228345>
> .
>


Can you document what external setup is needed to run the tests?


Will do as soon as I'm back home. Do you agree in testing x64 for one node
version only?  If yes which one? I have push rights for internal so I can
push there and ping you.
On Feb 21, 2016 19:56, "Kyle Robinson Young" <notifications@github.com>
wrote:

> @XhmikosR <https://github.com/XhmikosR> Would you mind giving this a
> review? gruntjs/grunt-contrib-internal@e8a7100
> <https://github.com/gruntjs/grunt-contrib-internal/commit/e8a71001196c157487d66db5834bedc192a14403>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gruntjs/grunt-contrib-concat/commit/49536bb5dbc69525f0e502e7df3e85597ea2ca7f#commitcomment-16228081>
> .
>


Remove logging, same as commented above. Please check entire PR for this pattern.


I am not sure ledInit is called anywhere, but it should be the case


@jeffrobots
Didn't you change this in the last commit for #13? What will the final method name be?


Interesting - I'll have to double check my db. I didn't see any information about SMTP sessions stored in the database, but I might have been looking in the wrong spot.


All done. Please test and submit bugs if any 😁


Closing #2


Actually, neither of these is strictly correct, as neither of them gracefully handles exceptions that might be thrown by `self.anim._run` but that's another question...


also ich finds sexy


@BillWagner I tried that with `ConfigureAwait(false)` as it's a non-UI code. The issue was that `_bus.SendAsync(command);` is used to send a command which should return nothing, but it does return a value (not sure what for). Because of that, `ConfigureAwait(false)` is not possible. I suggest to do it the following way:
`await _bus.SendAsync(command).ConfigureAwait(false);`


ja macht mich total geil... meow :octocat:


How Are You?


:+1: fixed


Ugh. I didn't catch this in the diff. My raspberry pi git clone got behind by a couple of commits because I'm working on two different machines.

The method name will be get_water_level, but it's being pulled out and into the sensor models.

I need to stop using the graphical interface. It causes more issues than it's worth.


Молодец, что вынесла операции во вложенные функции. Сама векторизованное умножение, правда, можно было записать так
`sum(a + b for a, b in izip(row, col))`


Thank you


Не хватает одной проверки.


В таким случаях говорят, что у матриц несовместима форма ("incompatible shape")


Hmm this means that the MD5 hash of the password is `password`; a better example would be:
`CREATE ROLE demorole1 WITH LOGIN PASSWORD 'password1' CREATEDB`


Cleaned up most of the code required for operating the motors and placed it in a different package which can be imported by any Python script that wishes to operate the motors.


Thank you for catching this :)


Rendering to linear textures is not guaranteed to be supported. You can query for it, but it doesn't make sense to do so except for very specific cases.


These records should only be accessible to the appropriate authenticated user, but smtp passwords are currently stored in plaintext. Might want to work out a method for encrypting with user's key.


Same as here https://github.com/SeanFeldman/allReady/commit/210ef547308916fe2aa00f55c4f024b2c69be79e#commitcomment-16228400


Issues mentioned in this commit (72a3e3a7d919ce17fde13bb30b8bc27ef1d4c9cc):

* [Issue #9155](https://bugs.mtasa.com/view.php?id=9155)


Looks like the CI build failed on compiling the template: https://ci.appveyor.com/project/tidusjar/nzbdash/build/1.0.436/tests


@JonathanRosado, @AndreeaDora - for the sake of the test I suggest you will change the rest of the fields - this way we will know for sure that we are not reading the values from the same place


I run ghost 0.7.0, I still have the following issue running gulp :

```
 % sudo gulp build
[19:49:32] Using gulpfile /var/www/samblog/content/themes/Decode-for-Ghost/gulpfile.js
[19:49:32] Starting 'styles'...
[19:49:32] 'styles' errored after 60 ms
[19:49:32] ReferenceError: Promise is not defined
    at module.exports (/var/www/samblog/content/themes/Decode-for-Ghost/node_modules/pify/index.js:37:7)
    at Object.<anonymous> (/var/www/samblog/content/themes/Decode-for-Ghost/node_modules/read-cache/index.js:5:12)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/var/www/samblog/content/themes/Decode-for-Ghost/node_modules/postcss-import/lib/load-content.js:1:79)
    at Module._compile (module.js:456:26)
```


I just changed again, ledInit was not the right function name


@mikegee @rchavarria  I think the issue revealed here is actually a flaw in @mikegee 's solution  submission for the Triangle exercise: http://exercism.io/submissions/1a614864d2dc44bdb18e1be9f5f6970e

line 19 reads:

```
if (sides[1] === sides[2]) {
```

when it should read something like:

```
if (sides[0] === sides[1] || sides[1] === sides[2]) {
```

which is what the commenter @plinto said.  @rchavarria is correct in pointing out the two other tests that cover this case.  I think the two existing cases are probably useful BC in one case, the equal sides each have a measure less than the third side, but in the other case, the equal sides each have a greater measure.

Unless I'm also missing something @mikegee ?


We may need to break it up into multiple values.


I think this is the default already


True


you might not need the `.class` here, i.e.:
```Groovy
@ClosureParams(FirstParam)
```


Sorry, fixing this.
`return _bus.SendAsync(command).ConfigureAwait(false);`
won't work because we are not return a `Task` from `_bus.SendAsync()`, so switching to
`await _bus.SendAsync(command).ConfigureAwait(false)`


2016 :)?


2045-draw-dual-ducts (evanweaver) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


How would your function change if you used *args instead of **kwargs? Or both?


It is odd that _decimal_ and _integer_ are defined explicitly in terms of Real Numbers, while _float_ and _double_ are "non-zero numbers", plus the special +/-0, NaN, etc...

My reading was that all of these numbers are equivalent; but on [closer reading](https://www.w3.org/TR/xmlschema11-2/#equality) I think that's wrong:

> In the equality relation defined herein, values from different primitive data spaces are made artificially unequal even if they might otherwise be considered equal.  For example, there is a number two in the decimal datatype and a number two in the float datatype.  In the equality relation defined herein, these two values are considered unequal.  Other applications making use of these datatypes may choose to consider values such as these equal; nonetheless, in the equality relation defined herein, they are unequal.

It sounds like we __can__ choose to treat these values as equal, but we probably don't want to enforce this for data stores.

Perhaps the safer check is _decimal_ `"1.0"` & _integer_ `"1"`, since integer identity is explicitly defined in terms of decimal identity. What do you think of that?


Nice!


2045-draw-dual-ducts (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


у нас в ридми написано, что это "BEM Props", и `className` туда входит, иначе придётся костылять поддержку существующего класса во всех местах, где используется `stringify`.

`bemjson` по старой памяти тянется уже в котором файле, нужно от этого понятия как-то ловко избавиться.


Agreed. I thought there was a particular reason it was this way, which is why I didn't change it. I can put in a PR to update it, or combine it a pending PR I have. Thoughts?

cc: @petebacondarwin


A pleasure to see that regex used :+1:


Library


I leave here a spanish translation for AMO page.

[VimFX-AMO.txt](https://github.com/akhodakivskiy/VimFx/files/139809/VimFX-AMO.txt)


yeah. or call it (x,y) if that seems less strange


Är det dags att återinföra den här koden? Dvs nu när boten svarar asynkront.


You should consider reverting this commit. It makes local debugging impossible, as all links will be pointing to yoursite.github.io. It just cost me 2 hours of debugging as I was modifying my stylesheets/syntax.css file and didn't notice that my local site was still pulling the (old) css from github.io.


words.txt-- contains word which will be matched for segmentaton.


Är det dags att återinföra den här koden? Dvs nu när boten svarar på anrop asynkront.


s/Know what freenode is/Learn about freenode
s/Know who's using freenode/See who's using freenode


@borzamircea, @gmatoshp  - I suggest you will read all error messages from DB or from ErrorMassage file - this way you will have all the errors in one place, and will be logic to search for it and update it accordingly.


Can we not have a container called "jumbotron" please?


yeah that was a good old fashioned typo. meant to be `lambda state`. I think it's fixed in the latest develop branch. To-be-fixed tomorrow as well; I messed up shared history by accident =(


@aimablehirwa @anne-france

1. avant tout, pourquoi l'appelez-vous `on`? Summarize `by` add ... `as`. Je vois pas de `on`. Je mettrais simplement `getResultingType()` ou `getAggregatedType()`.

2. Ensuite, je suis pas sûr d'être d'accord avec `setTypeOfOn(...)`

3. P


Have you tested your trapz function by passing a function to it that takes multiple arguments?


Thanks for the edition and for updating the README as well!


Sounds good, should I still leave behind the "SimpleDualTyped* classes?  I think I need to as the existing SimpleForwarding* classes will be typed to a single delegate type since the Forwarding* classes will override the delegate functions already.


Answer to my own question: `$macro` is modified on _multiple_ levels, so the match string in `$_`, which is _local_ in every loop, need _not_ be present anymore. This is obviously the case if an option is concurrently used on different levels, like `%{-p:...}` with the embedded `%{-p*}` in this test.


I think this harks back to when these things were ES6 classes, where there is no ideal scenario...

```js
var MyDirective = ['dep1', 'dep2', class MyDirective {
  someProp1;
  someProp2;
  ...
  constructor(dep1, dep2) {
  }
  ...
}];
```

or

```js
class MyDirective {
  someProp1;
  someProp2;
  ...
  constructor(dep1, dep2) {
  }
  ...
}
MyDirective.$inject = ['dep1', 'dep2'];
```


некрасивый перенос


Поленился написать сообщение?


+1 I did not change back my fork because builds on main branch are still using old setup.


Fixes [#12503](https://code.mythtv.org/trac/ticket/12503)


Ой, извини, просто забыл


Completes wrappers for question based URLs.


This should be wrapped in either a `<text>` tag or a `<div>` and it should be End: @Model.BreachEnd


This might improve things a bit: https://github.com/ruby-rdf/rdf-spec/pull/61


This should be `Duration: @Model.Duration`


This should be` Percentage: @Model.Percentage`


those lines shouldn't be removed in this commit but in the next one, right?


This should be` Threshold Time: TimeThresholdSec: @Model.Threshold Time: TimeThresholdSec`


otherwise good


good except the thing with the last commit


ATTOW, and with the issue #29 at hand, it might be possible to amend `handle_macro_options()` with the construct already developed and "make things work". AFAIK, there are no conflicting uses of the same option in multilevel macros.


@cconklin

It looks like something in here is causing the current mask to be overwritten when you update a control object. For example, if I turn on the light and fan for plant 1, they come on properly. If I then attempt to turn on the light on plant 2, the light and fan for plant 1 turns off and plant 2's light comes on.

It seems like the issue might be with one instance's mask updates not being reflected in the other instances.


why not first get the tab width? You initialize the spacinghelper with 4, shouldn't you initialize it with tab_width?


Trying to figure this out. I do not see a pencil icon to edit. Well lets see if this will get us there.


This really needs a toggle, it's very confusing for younger, european players.


Please see this discussion also about the nomenclature: https://github.com/caolan/highland/issues/109


Yes.


Been following this project hoping for support for the mx master, any chance you can tell me the result of this test?


@sbenejam  This is not OK, empty string ??


Oh, yes!


video: https://www.youtube.com/watch?v=C6sl1h6yxno&feature=youtu.be

related with : https://github.com/TrinityCore/TrinityCore/issues/16647


Кавычки в док-стрингах оставляют висящими. Документация не совсем правильная.


Not OK ? @janosvitok please review before merging


Mainifest is not needed, Maven generates it automatically.


Starting with this commit, two of my hubs (hubzilla.nl/hubuno.org) are giving white screens when logged in. Correction: white screen in Firefox, error 500 in Chromium). Cleaning the store/[data]/smarty3/compiled directory didn't help. My developer hub (standalone) is oddly enough not affected.


groupgoto will call each player individually. so there would be no way of knowing if every member in the group was near at the same time


It may be but I have not considered every possible case to be sure it is safe to convert for every ophandler. Defining a per-ophandler flag was on my mind but I wanted to see more instances of the conversion first to understand how much sense it would make.


MIspelled "canola".


Good luck, I'm fan of reduction formula :)


I'm reverting this commit for several reasons.
1 - "It looks like master was committed to directly. No one should be
doing this unless the change is very small."
Yeah, it should be, I thought my commit was small, but never mind, you're right. I made a lot of changes but I minimized fixing the current way how the code works atm. but next time, I'll work on different branch.

2 - "I am Immediately suspicious when 8 files are changed with 163
additions, 296 deletions, and a 1 line commit log."
I believe you see a lot of people make some changes and have one line commit log. It's normal for me since I want git to save the current state of my code so I can get it back when I make something wrong. Also, since I usually make many changes and then commit, it's my habit, so I don't try to remember all small details I have done and list them all in the commit line. If you want have any question, you can leave any comment to my commit and ask for the reason.

3 - "Many of the files changed seem to have little to do with the commit
log."
Same for number 2. I tried not to modify much about current code. It still works like the way it is. The thing I modified is solely for the purpose of my code to work, and if you don't like my change, we can discuss about it.

4 - "Nearly everything in the agents folder calls draw with 2 parameters
when previously they did not and drew perfectly fine. What was the
reasoning behind this?"
I believe you have read and understood how Taylor's code work. I wanted to ask Taylor directly about this but still haven't got a chance. I don't know if you are aware of this but Knight.draw(), Skeleton.draw() and other agent objects never get called. You can try getting rid of all the draw function and see it still works. It's not a magic but because the Taylor's game engine doesn't call their draw function, but call the entity draw directly, which doesn't go through from each agent draw function. So the question is "why do we have a draw function for each object even though it's never called?" and the next question is "what if I want to modify the draw function, how can I do that? Am I supposed to do is go to the Entity object and make a change of it?" No, it will change all the way how the other objects drawing because other object have Entity as their field. Therefore, what I change is to make the Taylor's game engine call each agent draw directly and then, from the draw of each agent, it will call entity to draw. Since the entity.draw need 2 parameters, so I need to fix all other agents to take 2 parameters and pass them to entity.draw. If you have any concern, please leave a message after this.

5 - "Agents now receive a bunch of information most of them don't
require, and that all of them are able to access through their own game
engine reference. Same question: Why?" The reason for this is for my convenience. It doesn't affect other things. You may have known about design patterns when there are always some arguments in some methods but you can never use them. They are there for other object that needs them. I added to make it easy for me to access to playerAgent easily. Instead of using this.entity.game.playerAgent.entity.x or y and then this.entity.game.playerAgent.entity.width or height, I let the game engine pass all these information for me, so I can use it, and I believe other class can use it as an advantage in the future as well. Also, I see the entity appear twice in the example above, and I feel unsure about it even though it is correct. Of course I can do a long way like setting a variable to access all these info, but you know people have their own style. And for sure, If you don't like to receive a bunch of information, I can delete it and do the long way as you want.


Док-стринги оставляются внутри тела функции.


Ah, sorry guys, I wasn't being clear: The style itself isn't up for debate here (of course anyone's free to adapt it within their own projects afterwards).

Instead, the question was whether we should (a) start with a blank slate, (b) extend `eslint:recommended` or (c) base everything around JavaScript Standard¹ Style while overriding individual rules to fit [our](https://en.wikipedia.org/wiki/Pluralis_majestatis) preferences.

* (a) seems too minimalistic even for me
* (b) would mean limiting the number of rules to only protect against _common_ violations (carefully extending that list as code reviews uncover inconsistencies) in order to keep `.eslintrc` a manageable size
* (c) eliminates almost any leeway, ensuring consistency across the board - however, it does mean the rules themselves become a potential maintenance burden (as you kind of have to be aware of and understand [all the underlying rules](https://github.com/feross/eslint-config-standard/blob/master/eslintrc.json))

¹ a rather unfortunate name IMO - after all, it's aspirational at best


oops, good catch - thanks!


Здесь лучше воспользоваться распаковкой, чем индексироваться по `pair`.
`for char_a, char_b in izip(a, b)`
Хотя я бы назвал аргументы функции как `p_distance(seq1, seq2)` и сделал
`for a, b in izip(seq1, seq2)`


`if (...)` be an abomination unto the Lord


IDK what volley is.


develop (evanweaver) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-89.14%25-red.svg)


Зачем вы распаковываете, а потом обратно запаковываете объекты в список? Создание списка, кроме того, сравнительно дорогая константа.


-Oui, tous à fait d'accord avec vous pour les noms, on va changer cela.
-c'est une solution que j'ai mis pour l'instant, on cherche une meilleur
-setType.. est surtout pour setter le type de result pour l'aggregat max
-mais on cherche une autre solution


I'm wondering if any progress has yet been made on this. Would LOVE to see it happen. Would even be willing to contribute if it's still being pursued.


Такой код очень тяжело читается. Лучше вынести субрутины в отдельные вложенные функции. Здесь напрашиваются функции: получить строку по индексу, получить столбец по индексу, векторное произведение


I did briefly read it, but apparently too fast... Sorry.


Не понял, что это значит?


How's this my fault? Didn't you make all of that?


@denfromufa should consider adding this line to the config. The wheels will be available after the tests are completed.


s/modern responsive/modernly responsive
s/freenode.net/freenode


I believe this can be changed to

```clojure
;; Close the display if the close window button has been clicked or the gl-lock has been release programmatically
(or (Display/isCloseRequested) (not @gl-lock)))
```


@denfromufa Also noticed I force-pushed (and deleted) my comment from yesterday. This branch is what is kinda what i meant by squashing similar commits together.
Also had to maneuver around some of the git-fetches from the origin/develop and instead used a rebase ontop of it.  But makes the evolution of the changes easier (atleast for me) to follow.


este ficheiro tem um erro


FYI @lukasgraf
because all our Plone builds are failing.


This can just be
```clojure
(reset! gl-lock false)
```


I didn't account for that. I'll look into it.

> On Feb 21, 2016, at 2:22 PM, jeffrobots <notifications@github.com> wrote:
>
> @cconklin
>
> It looks like something in here is causing the current mask to be overwritten when you update a control object. For example, if I turn on the light and fan for plant 1, they come on properly. If I then attempt to turn on the light on plant 2, the light and fan for plant 1 turns off and plant 2's light comes on.
>
> It seems like the issue might be with one instance's mask updates not being reflected in the other instances.
>
> —
> Reply to this email directly or view it on GitHub.
>


@BillWagner I have gone through the comments and added a few modifications, including tests change that needed to be `async` and not `void`. My mumbling on `ConfigureAwait(false)` was not accurate because it would force signature change.


Great idea. I think it can be make a bit more simple by using gl-lock to control the logic.


*vormerk*


root@local:/var/www/midcamp-website/tests# bin/behat features/session_proposals.feature --tags midcamp-277
.F------------------------------

--- Failed hooks:

    AfterStep # FailureContext::handleFailure()
      Unable to access the response before visiting a page (Behat\Mink\Exception\DriverException)

--- Failed steps:

    And session content: # features/session_proposals.feature:50
      Catchable Fatal Error: Argument 1 passed to FeatureContext::midcampAfterNodeCreateScope() must be an instance of AfterNodeCreateScope, instance of Drupal\DrupalExtension\Hook\Scope\AfterNodeCreateScope given in features/bootstrap/FeatureContext.php line 283

1 scenario (1 failed)
32 steps (1 passed, 1 failed, 30 skipped)


@aimablehirwa à mon avis la meilleure solution c'est alors d'enlever `setType...` et d'aller vers `getAggregatedType(RelationType on)`, où `on` est le type de la relation sur laquelle porte le summarize. Max, par exemple, n'a qu'à retourner le type de l'attribut sur lequel il porte.


Это явно не так.


true, just changing to "feature" in this example is enough


@trws You may want to remove this duplicate :smile:


Thanks for the input.
Running gulp gives me a clean output (expected .htaccess file -- at least on 2 of my test boxes if that matters) is there anything i don't understand maybe? Feel free to educate me, that's my life's purpose!

I am not even using Apache, i though this could potentially benefit "new users".
I was still debating if i should pull request. If you think it could be of use, i do not mind you add it yourself. just let me know then

on all my servers with nginx i am caching all static files. Maybe i could edit the docs or add a working nginx config file, what do you think?


I asssume you can have A inherits B, B inherits C, hence you have multiple "parents"; perhaps better to say "super types".


I mentioned that in this comment that I added in https://github.com/JetBrains/intellij-community/commit/82f187e2d1926c4f1963a2f4eee85476adaa0a9b


Thanks for pointing that out:  Addressed in commit 37649d2


I've renamed the variable.  Unsigned isn't an issue since the numbers involved are always very small.


This looks right to me


That would work. :shipit:


I'm not familiar enough with the codebase to resolve it myself but if you comment/vote for the issue that is mentioned in the comment I left in the source code, maybe it'll get some attention.


Agreed. I won't add comments on the other similar notes. But I still agree. :shipit:


why not just change the file name and do .github/CONTRIBUTING.md ?


Library's src


Added comments


Thanks Ivan.

One thing I noticed is that at some cases !pv reduction = pv reduction.
Logically !pv reduction should be greater than pv reduction...


added comments


figure out border edges. find better way to upload tumblr data.


Guess you are right - the object type abstracts features, the different kinds of features are in the same namespace so a type check is enough. Few users will actually use an Object type directly, they will create a new type - so this error message is ok.


Why the lambda here ?


I also fixed style, and about it I added an additional comment.

I agree that the style was able to correct in the independent commit, but here not a lot of corrections, so I leave all in one commit.
I believe that if we if touched this file, then it would be good to change it according to PEP 0008 style guide for imports and empty rows.


You are absolutely right. Sometimes we need to make changes which are important for whole project. I really get pain when I try to start wakaama's client on dual stack sockets. But I guess, it's ok to break some conventions, when you are providing such improvements. Formally, we have a variable which contains server address. And we shouldn't to care about address family.


That's another way of doing it. This is just an example I think it's fine as is.


yes, we get to that later - separate issue. Was just curious if you thought about it.


OHMYGODALARWHATHAVEYOUDONE?

Copy-pasting from patch notes:
	[*]Hyperdrive FTL storage should multiply by ShipSize instead of Size, so that there is always a token amount of storage available.
	[*]Hyperdrive FTL generation/resupply should do the same.


Java? is this ported?


ok


Пустой контент можно не указывать


`string[]`?


не nicer


parse/test/test.cpp uses these


Fixes WooMinecraft/WooMinecraft#76


I'm afraid I will have no access to my machine till Wednesday. I'm
travelling right now. Please make a fix and I'll debug that later.
21 lut 2016 19:47 "Daniel Naber" <notifications@github.com> napisał(a):

> Could you check with a debugger what exactly the input is in
> CommandLineTools.checkText()? For me, it's This is an test.\n\nThis is a
> test of of language tool.\n\nThis is is a test of language tool.\n
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/languagetool-org/languagetool/commit/96962056aaf84fa2f4930c18dba0d17fda9d27ea#commitcomment-16228365>
> .
>


Что за пустой блок?


This is wrong!


Почему?


Why we even need a parent? A smash game like this doesn't need one.


Не оставляй код в коммите


Can you fix these various inconsistent continued indentations? I think this was the first one added, but there are a few more with later commits.


Is is really necessary to use fully-qualified names here?


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


Пустой контент


this is actually an issue in trunk, but probably noboby noticed due to the rather incomprehensible current display of supply ranges


Da sollten Anführungszeichen um „Abgelenkt“ sein.


Later I understood than LMR is much more than formula log * log which I tried to improve. I'm glad that you work hard on LMR because I believe that there is much elo in it.

Btw, I convinced myself that log * log is asimptotically the best for higher depths, but there is much room for improvements on lower depths.


Потому что всё ещё разрываешь выражение, которое без труда влезает в 80 символов.
```
mismatches_without_gaps = sum(char1 != char2 and char1 != "-" and char2 != "-"
                              for char1, char2 in izip(seq1, seq2))
```


:+1: I think it'll look nice in the attachment format.

What does the color signify? If it doesn't have a particular meaning, I'd leave it off: the content'll get the same gray stripe as unfurled links (which is probably appropriate).


Lol! Märchenstunde-Modus, das hat was.


I think it could be nicer if the check_define decorator had an optional (or required) message parameter so both decorators could be collapsed into one check_define call.


Теперь это ошибка.


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


Die Wirkung ‚Abgelenkt‘.. so?


It is all in how you name the test; not it talks about "parent resolved correctly"; resolution is a trait of the TypeParser not the ObjectType. If you instead phrasing it as examples that a derived object has a parent, and that an object type that redefines inherited features produces those definitions instead of the redefined then yeah. Here you are using the TypeParser to produce the ObjectType (I get that, but what is tested/examplified is unclear because of the it-title; you could just as well view this as a test of the TypeParser's ability to resolve as expected (almost since it is also testing collection of features).

Just change the title...


Not "`None` is not an orderable type", but "`NoneType()` and `int()` are unorderable"; a similar example:
```
>>> sorted([0, ""])
TypeError: unorderable types: str() < int()
```

In the ARPA file (and accordingly the script being discussed), *n*-grams are sorted in batches; for each batch, since *n* are the same (the code does not explicitly parse *n*, but `l[0] == "\\"` does effectively the same), and *n*-grams are supposed to be unique, provided all words are in `dic`, comparison should never exceed the *n*-gram boundary.

Therefore if we can assume that words in *n*-grams are all collected in `dic` (`dict_head.utf8` + `dict.full`, *ie* all those in `dict.utf8`), it is unnecessary to avoid `None`.  Now the question: do we assume all words to be collected in `dic` whenever we use this script?  If not, then a change would be necessary.


Came here to see that this was added. Thanks for doing this! :+1:


Did you mean to include this? This looks like debugging code.


This is the fix @zackhall


This is `bits_left` .


@chhapil, this change is stable. I think root cause of SODs was intelliactive, all interactive based govs don't like dorimanx kernel :(


tutaj już olać – nie oczekujesz, że ktoś będzie ten obiekt bezpośrednio inicjalizował, tylko że skorzysta z YoutubeApi

https://en.wikipedia.org/wiki/Facade_pattern


Includes zoom and minimap movement, as well as terrain identification.

I need to fix some errors with navigation, comment code, and add more texture assets since some terrain is sharing textures.

Also has support for edge mapping on the hex (so beaches appear around water, plains appear around edges of the forest, etc.


What does the 800 and 725 being multiplied here signify?  Why those 2 specific numbers?


d


Proper solution with working override toggle: https://github.com/gsstudios/initramfs3/commit/0b91251264750e38f741bbccda0b3fb538641382


jak brzmi wiadomość w razie błędu? np gdyby zwróciło 3 itemy albo gdybyś oczekiwał jednego?


This should be fixed now.


I'd stay away from modernizr as well, see for instance: http://stackoverflow.com/a/22238927
Not in the means of _always_ but more like use it for apps where you think it is required.


That should be "except detached" I think. ;-)


Не нужно менять a на span. Нужно добавить дополнительный класс, например disabled со свойствами:
pointer-events: none; /* делаем ссылку некликабельной */
cursor: default;  /* устанавливаем курсор в виде стрелки */
и поменять цвет фона, теста и т.д. по макету


@ChewyMoon put this back after


It checks GOOS. There are no tests that we can run, so the easiest solution is to not test the package at all.


good


right thing to do


Might it be worth adding the explicit check here, like in the actual implementation?

```c
top = duk_get_top(ctx);
if (top < nvalues) {
	DUK_ERROR_API((duk_hthread *) ctx, "not enough stack values for notify");
	return ret;  /* unreachable */
}
```


os.path.join would be more corss platform and stable


Yup, these ones run! do they not look like they can?


That really works?


So you just record it with rumblefighter open and then you play it as .exe?


Yeah, that'd produce a cleaner API error. I was a bit torn between doing that and just relying on `duk_pop_n()` throwing as it will throw an API error too (though with a different message) and there'd be less footprint. (I know it seems silly but I'm literally fighting for kilobytes for some targets :-). It's probably better to fix though.


May you like give me a little guide? @VyrusMaker


@PeyTy I wanted to use a special -D luajit flag to enable some extra jit features.  I put the bit fix there for now, but I think a better idea might be to check luajit/lua during run time, and choose the bit library then.


We need to exclude external code ie skin and vendor dirs


Thanks for pointing out this, I have corrected the same in repo and added test case also.

Can you please explain the requirement for feature callback in your commit. There is callback option when svg/path string is created. If callback for feature seems generic feature then I'll merge your commit.


do the rest of the functions in here need span counterparts?


I know this is supposedly fixed and broken again, and I know that @remram44 probably doesn't want these notifications. However, I did find the solution to the the problem @eternal-sorrow and @catox0 were having. You need to make the edit to line 173 where you add in the "native_str" bit, but you no longer need to import it on line 16. Had myself a bit of trouble getting things to work after trying a pip --upgrade pip.


cvxx


are these fixes? does the original game still have functionality?


Looks good


lololol
What are you doing?

On Sat, Feb 20, 2016 at 6:01 PM, Nathan Ringo <notifications@github.com>
wrote:

> Nononononooo, you use the Let's Encrypt client to get certificates.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ash-mita/LAUNCH-Physics/commit/223af3e2b9efe5794eac7d03743ee7cc7ddf8666#commitcomment-16224139>
> .
>


博主你效率真高啊   赞！！！


I see. But then it leaves the package in a kind of limbo state. In theory, the package works and is supported. In practice, its tests are completely disabled (even if new tests are added that are GOOS-agnostic), and it's listed as "partially supported".


What would you prefer?


为了打个点，你也是挺拼的


This is much better than my wording, however I think I'd personally prefer "A mechanism for..." over "Support for..."


oops


@dukesrg I used `TCHAR` to indicate the strings are for _FatFs_.


i made my own yt api key but looks  like  the q command  only works for links  and not  for titles.


In English?


it appears that simple.php and simple.php.save are the same are both needed?

Terry


lol someone probably spent a lot of time on that.


Should you do SQL Binding here?


description updated


Should you have a standard file header here containing the information for copy rite, author etc?


side_effect seems to be unused


Removal of all commented out debugging statements and code


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


Probably not required


THis was not done correctly


could you put ```['pylint', '--version']``` also into ```self.command_to_check_install``` ? we're monkeypatching script locations on startup, because ours are not on path


do you by any chance know how to get a full version of jithub for free? @VyrusMaker


POSIX standard requires newline at the end of file. Might break some unix tools otherwise, when used on this file, e.g. sed


testing code-diff comment


prolly unneeded? use the one from the baseclass ```RegexBasedDriver```


tri-state-key path is wrong for our device
https://github.com/dlhxr/android_device_oppo_common/commit/8d8bb2c810867f29e894ab52726d1b7e1d3bfcf4


Yeah, both of these can just become instances of `RegexBasedDriver`


zuado é que eu mandei o PR pro cara com o atalho certo e depois ele mudou pra um atalho cagado.

zueided


Comment: I am not sure about extract  . I think it will lead to confusion for those who make changes later. Would need some of the long time developer to weigh in on its use.


KLik "Download Zip" Untuk mendownload semua file.


Closes #28.


![Not bad](http://icdn4.digitaltrends.com/image/not-bad-obama-1500x1500.png)


“更新到 2.5.0 版本使我们代码中出现的 bug， 因为我们一开始就走错了路。” 其实这一句我没太理解 =。=


So with regards to the search method you cleaned up a bit too eagerly I think. You now actually lost type-safety for the search api. Doesn't matter for the main() method when it is called from the command line, but I would like the search to be callable from another Scala program. Now you have a method search and if the result is a CoordinateResult or ClassnameResult depends on the value of the string that is passed. If a caller tried to query for a class, he would get a Vector of SearchResult and pattern match and ignore the ClassnameResult case, because from the value he expects that to never happen. Better would be to have 2 search methods, as before. One that returns CoordinateResult and another one that returns ClassnameResult. Then the methods makes that clear.


Awesome!


Curious: What does the $$ indicate/do?


i dunno man, this was a pretty kickass plugin.


Removal of commented out code suggested


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.36.0](http://build01:8888/viewLog.html?buildId=14341&buildTypeId=OpenSourceLibraries_Sourcerer_CI) is now running


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.36.0](http://build01:8888/viewLog.html?buildId=14342&buildTypeId=OpenSourceLibraries_Yaclp_CI) is now running


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.43.0](http://build01:8888/viewLog.html?buildId=14343&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) is now running


ok


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.43.0](http://build01:8888/viewLog.html?buildId=14343&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:00:25


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.36.0](http://build01:8888/viewLog.html?buildId=14342&buildTypeId=OpenSourceLibraries_Yaclp_CI) outcome was **SUCCESS**
Summary: Tests passed: 20 Build time: 00:00:28


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.58.0](http://build01:8888/viewLog.html?buildId=14344&buildTypeId=BlogMonster_CI) is now running


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.41.0](http://build01:8888/viewLog.html?buildId=14345&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) is now running


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.58.0](http://build01:8888/viewLog.html?buildId=14344&buildTypeId=BlogMonster_CI) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 12 Build time: 00:00:17
### Failed tests
```
BlogMonster.Tests.dll: BlogMonster.Tests.WhenImportingPostsFromAnInstagramFeed.ThereShouldBeAtLeastOnePost: <no details avaliable>

```


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.41.0](http://build01:8888/viewLog.html?buildId=14345&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) outcome was **SUCCESS**
Summary: Tests passed: 21 Build time: 00:00:24


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.525.0](http://build01:8888/viewLog.html?buildId=14346&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.62.0](http://build01:8888/viewLog.html?buildId=14347&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) is now running


Could you please guide me properly on how to run BFS individually?
As it gives me an error saying function definitions not permitted this way :(


+ 4 пробела на отступ и уже не влазит (78+4)


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.36.0](http://build01:8888/viewLog.html?buildId=14341&buildTypeId=OpenSourceLibraries_Sourcerer_CI) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:01:17


Looks right


Think it is a matter of writing better titles for the tests.


Thanks, that is much better!


Test comment really


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.525.0](http://build01:8888/viewLog.html?buildId=14346&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:34


This is outdated! Use the input.php


As mentioned in https://github.com/AppFolioOnboarding/Boying_ImageUploader/pull/51#discussion_r53540756:

> right now, we have no test covering [this part](https://github.com/AppFolioOnboarding/Boying_ImageUploader/blob/03b7657d4eb13d1b68c64fe14883b1688785602b/app/assets/javascripts/images/index/sharing_modal.js#L18-L20) of your javascript.

This functional test only tests the server response for the not found case. It does not test the behavior of how the client handles that response (which is dictated by those lines of javascript I linked).


test on line 2


Samer,

Thanks for taking this onboard as it will make co-existence much easier but I do think we need the two lines after doing the dotenv load, at least on RPi Jessie and current node_module versions:

**line 12:	host = process.env.HOST || host;
line 13:	port = process.env.PORT || port;**

Note that the require("dotenv").load() command only updates the **process.env.HOST** and **process.env.PORT** variables but it is the **host** and **port** variables that are used in the subsequent app.listen(port, host, function) command. Hence the need to copy them across.

I did a fair bit of testing on this to ensure precedence was right. Try putting "PORT=3001" and "HOST=0.0.0.0" into .env and calling "node server.js". Without the above two lines you will still have an app.listen on host="127.0.0.1" and port="3000".

I am curious if you have a different setup and seeing different results on this.

Cheers, Pete


I'm not sure there's a need to have a conversational version of the `/esv` command.

On the other hand, what if you had Houston **_overhear_** Bible verses and unfurl them. You can use the [pericope](https://github.com/boblail/pericope) gem to match them (see the `Pericope::PERICOPE_PATTERN` regex).


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.62.0](http://build01:8888/viewLog.html?buildId=14347&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) outcome was **SUCCESS**
Summary: Tests passed: 3 Build time: 00:00:58


Die einfachen Anführungszeichen werden nur verwendet, wenn man schon innerhalb der Doppelten ist. Hier müssen also die Doppelten verwendet werden. Ich vermute mal stark, dass ist nicht die einzige Stelle in der Datei.


Um why?


Run yard :-D on some ruby code and try it out...


Delete this @outofyourmind


What system does this fail on? I think I have a fix, but I would like to test first.


Delete this please @outofyourmind


Might want to fix these constants to be 0, 1, 2, and 3 instead of just 0.


Thanks for pointing it out! I had already amended this yesterday - commit 136e119 is what is now running on tools labs.

So right now we're not displaying any links on lead section snippets. Do you have any suggestions for what we should use there, if anything?


Z  muss zu Y


придумать название переменной получше - и влезает.


The file should be HelloWorld.java please rename
Please send your own class


My mistake, this should be `platform.system()` :fearful:


where am i?


Thanks, but that won’t happen. Too much work for too little gain.


Daithi, I have set up a folder called 'router' per the tutorial, should i have left main.js, in the main folder?


Zeile 335 Z muss zu Y !!


Zeile 345 Z muss zu y


`sudo chown -R "$(id -u):$(id -g)" "$bg_dir` is more correct (The username might be different to the group name)


no changes


Why not `if platform.system() == "Darwin"`. This would be more safe and clean imho...


Zeile 348 Z muss zu y


Ты понимаешь, как это работает? Если нет, зачем мне это показываешь? Тем более что, это довольно уродливо.


Salut,
j'ai remarqué que tu avais mis le .idea dans ton git repository.
Tu es très attaché à l'interface graphique de intelliJ à ce que je vois.
Bises


Probably a decorator that you pass a tuple of strings, e.g. ("linux",), that does a conditional involving something like sys.platform.startswith(passed_tuple) and raises NotImplementedError or whatever based off that.


Surely it would work, you call GROUPGOTO on click of npc and then make the GROUPGOTO script so that each individual person is checked whether they are on the map and within range.

Example
```
<Enter Fox Cave/@enter>
<Check near/@near>

[@near]
#ACT
GROUPGOTO @nearby

[@nearby]
#IF
CHECKMAP $MAP
CHECKRANGE $X_COORD $Y_COORD 5
#ACT
LOCALMESSAGE "You are nearby"

[@enter]
#ACT
GROUPGOTO @checkarea

[@checkarea]
#IF
CHECKMAP FOX02
CHECKRANGE 200 200 5
#ACT
MAPMOVE
```


Plain and simple reason, it didn't work before this and did after :)

I'll try and open a pull request with the commit reverted and we'll see how Travis gets on.


Pour l'ajout d'une image j'ai du utiliser "image_tag" et j'ai du créer l'arborescence "public/assets/images" pour pouvoir placer mes images et les afficher.
Mais je pense que c'est complètement foireux comme solution


I didn't ask you to use inheritance. But it's good if you've learnt it yourself
Be ready to answer questions the next lesson


```
 // At factor 0, 10 time_to_cut is 10 turns. At factor 50, it's 5 turns, at 75 it's 2.5
     time_to_cut *= std::max( 25, 100 - factor );
     if( time_to_cut < 500 ) {
         time_to_cut = 500;
     }
```

I'm sorry if this is the wrong place, and there's a really good chance I simply don't have a good grasp of C++ or the source code; but the above comment makes it seem like with any `factor` greater than '0' the `time_to_cut` would be reduced. If I'm reading this right, at a minimum `time_to_cut` would be multiplied by 25, then further increased to 500 if less then 500.

I realize this isn't your code (at least I mean its not part of your edit) just wondering if it has any affects worth considering?


I would check for is_string() explicitly here and throw and exception in an else-branch for any other type.


it's faster to use !empty() instead of count()>0


Can we impove on the heading


I've seen some vimrc's and a lot of them that use neobundle, or yours even that uses dein... "wrapping" plugins configuration like:

```vim
if dein#tap('unite.vim') "{{{
  " The prefix key.
  nnoremap    [unite]   <Nop>
  xnoremap    [unite]   <Nop>
  nmap    ;u [unite]
  xmap    ;u [unite]

  nnoremap <silent> ;b
        \ :<C-u>Unite -buffer-name=build`tabpagenr()` -no-quit build<CR>
  …
endif```

It's strange, because if the plugin is added to dein, why I would want to check again if it exists? Or is it that dein loads the value between that "if … endif" only when the plugin is actually being loaded?


I'm trying to get this one done, so letting you guys know


The problem is related to integer overflow error occurring on some compilers when MIN_INT value is being parsed.


instead of this whole if, use:
```
return in_array($parsed['host'], $this->domains, true);
```


В общем, раз уж ты решил показать это моим глазам. Объясни мне подробно, как это работает, иначе я ставлю тебе -3 бала за это задание. Если успешно объяснишь, поставлю 6 баллов.


will update later, still in flux so there might be a lot more templating engines + components added


@macagua 'than my design days ' by ** than my design tastes **


Please create one more class "Mark" and use the array or list of marks
Provide the method for calculating average mark on this array


Is there the comment wrong or the module? I think this should be `group`, no?


I've Googled this and found similar stuff, but not this solution. I'm not sure why you're aliasing this function in that way. Why can't we use `django.contrib.auth.get_user_model()` directly in the same spots you call `_get_user_model()`?


Which ones specifically?


First Commit


s/are handled from 0 to project/are handled from Odoo project/g cc @macagua


s/salt/solved/g


Good catch with platform.system().
The reason why I'm testing against the presence of the compiled library, instead of conditioning on the platform, is that in the future the separate OSX makefiles may end up unified into the linux makefiles. The version of the script that I proposed should work correctly if the user has managed to build the linux makefiles on OSX.


Please create one more class "Group" and add the necessary properties (name - string, students - array or list of this objects, maybe curator  - teacher)


@hvanhovell Is this what you prefer?


*vorgemerkt*


Thank you!! That must have been a lot of work, I noted your side branch on this and it had to require extensive debugging in addition to the vast number of improvements.


s/rose I can't even use 02 to hire more/growth I can't even use Odoo to hire more/g


Yes, this is how it works.

Currently the best factor possible is 60. Factor can be negative too.


If the tests are using those just delete the tests. They are not maintained anyway.


'Freenode' should also be 'freenode', as it is lowercase.


s/simple dark now and growing your/start now to growth your/g


ОК, в общем я понимаю как это работает.
Тут всё основано на распаковке zip'ом матрицы, он распаковывает столбцы, а дальше, мы имея столбец и строку слепляем их через zip, прогоняем через стармап умножением и суммируем добавляя в строку


I think the expression should be s.Length


Ja, die Standardimplementierung nutzt das zwar nicht, aber man kann in einer eigenen Schemaklasse auch ein Array mit Kandidaten zurückliefern.


Should be openstreetmap.org, right?


that looks correct, I don't understand why it doesn't save the block


lol


This is my first DevOps project.


Think this override can be omitted now (we copy bootstrap css directly anyway)


As you know, I was the one who authored this commit, so, it would be nice to mention my name or say committed together with @optimumpr

Regards


Das minus da, muss der Strich werden: –


I'm sorry. I was working through SSH on my Raspberry Pi, and I needed to add this file to configure mod_wsgi. I spun up a separate branch to work on it, but I forgot to check it out before committing. Feel free to revert this commit if a principle is at stake.

Pushing directly to the master branch, that's a paddlin'.


Linking errors, as if the compiler stripped something it shouldn't have. When going back to -O2 it disappeared. It is for sure worthwhile to investigate but not high enough of a priority right now.

Error in question:
[ 75%] Linking CXX executable webbridge
cd /home/wouter/workspace/yocto/build-7401/tmp/work/core2-32-rdk-linux/webbridge/1.0+gitrAUTOINC+1cb45102ce-r0/build/WebBridge && /home/wouter/workspace/yocto/build-7401/tmp/sysroots/x86_64-linux/usr/bin/cmake -E cmake_link_script CMakeFiles/webbridge.dir/link.txt --verbose=1
/home/wouter/workspace/yocto/build-7401/tmp/sysroots/x86_64-linux/usr/bin/i686-rdk-linux/i686-rdk-linux-g++    -m32 -march=core2 -mtune=core2 -msse3 -mfpmath=sse  --sysroot=/home/wouter/workspace/yocto/build-7401/tmp/sysroots/7401  -Os -pipe -g -feliminate-unused-debug-types -fstack-protector-all -D_FORTIFY_SOURCE=2 -fvisibility-inlines-hidden  -m32 -march=core2 -mtune=core2 -msse3 -mfpmath=sse  --sysroot=/home/wouter/workspace/yocto/build-7401/tmp/sysroots/7401  -Os -pipe -g -feliminate-unused-debug-types -fstack-protector-all -D_FORTIFY_SOURCE=2 -fvisibility-inlines-hidden -Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed  -Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed CMakeFiles/webbridge.dir/MetaData.cpp.o CMakeFiles/webbridge.dir/WebBridge.cpp.o CMakeFiles/webbridge.dir/Controller.cpp.o CMakeFiles/webbridge.dir/WebBridgePluginServer.cpp.o  -o webbridge -rdynamic ../WebBridgeSupport/libwebbridgesupport.so.1.0 -lpthread -ldl -lgenerics -ltracing -lcryptalgo -lwebsocket -lrpc -Wl,-rpath,/home/wouter/workspace/yocto/build-7401/tmp/work/core2-32-rdk-linux/webbridge/1.0+gitrAUTOINC+1cb45102ce-r0/build/WebBridgeSupport:
../WebBridgeSupport/libwebbridgesupport.so.1.0: error: undefined reference to 'Solutions::WebBridge::IWebBridge::ID'
collect2: error: ld returned 1 exit status


A simple `or` should work as well, i.e.:

    key.anyNull && rowBuffer != null && (condition.isEmpty || rowBuffer.exists {
      (row: InternalRow) => boundCondition(joinedRow(current, row)))
    }

I think hotspot should be able to figure out that it can eliminate the exist clause when the condition is empty.


From 2PM yesterday to this morning, yes. This is  the second try. The first try broke quite a few things, and it had to be scrapped.


`header` is a little mysterious -- maybe `skip_header=True`? Also, `detectRow` at first sight I don't know what `detectRow` implements.


Sorry. Solved on new pull request

2016-02-21 20:26 GMT+01:00 Massimo Callegari <notifications@github.com>:

> @sbenejam <https://github.com/sbenejam> This is not OK, empty string ??
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcallegari/qlcplus/commit/8d54fa7eaa5a1fff16be353d5314dfc49ac88ff5#commitcomment-16228567>
> .
>


*vormerken*


Yeah, that looks better. Let me update it and re-run my tests.


I already pushed a fix for this. Holding it for UX changes in progress.


I can try to clean some of this up when I get some spare cycles, but yeah
mkdocs is pretty sensitive to these kinds of things. I agree that we should
have a larger discussion about docs. Perhaps a good project for one of the
part timers?

On Saturday, February 20, 2016, Randy Olson <notifications@github.com>
wrote:

> @bartleyn <https://github.com/bartleyn>: The docs issue largely stemmed
> from using a non-readthedocs theme. Guess we're stuck with this one for
> now.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rhiever/tpot/commit/14aee27560441e3db4ea05a0e49240438023e4ca#commitcomment-16223408>
> .
>

--
--------------------------------
Nathan Bartley
University of Chicago
Biological Sciences, A.B. 2013
Computer Science, S.M. 2015

Cell: (310)773-1145
Email: bartleyn@uchicago.edu
--------------------------


About the unification, it would be great to install the Python package in a more simple way, like the regular `pip install git+https://github.com/sambayless/monosat`. I think it is somehow easy to control a [`scons`](http://www.scons.org/) compiling process from a `setup.py` file.

I tried `scons` before but not the full integration with `setup.py`. I may try this process in the coming weeks and keep you posted on how it turns. It would be great if we could automate a full painless installation process.


XD


нет, в `zip` распаковываются не столбцы


People are saying it isn't, but I've gotta check LOL
I think you can license your new files you've extended though.


I reset all these tokens and passwords. Don't even try


Thanks @XhmikosR! Do you think x86 is more used over x64? I think only testing a different architecture on one node is a good idea but don't know how to configure that. I would like the code to hit both at least once, just in case something ends up using a native addon or has an architecture issue, we'd know.


Un `$this->trust($this->is('ADMIN') || $this->isClubMember($slug));` serait plus court et lisible (le UserBundle est le suel Bundle a ne pas avoir été refactoré avec ces methodes)


Neat! if you can make it work, I'm all for it (so long as it doesn't introduce extra complexity into the build process on my end).


Check issue #82 as I mentioned above - it contains detailed description of the problem.


Instead of creating es object again and again for each file, can we create one when we receive the request and pass it to this method? Maybe we can create es in judgeDir and pass it to other methods?


ty!


you can fall through here too, right? after all the offline state will be "OFFLINING" here in the normal case.


I just figured it out it's just that I'm not home yet.

I just don't know which version to test x64 on. Maybe 0.10 both? By default
x86 is used BTW.
On Feb 21, 2016 22:29, "Kyle Robinson Young" <notifications@github.com>
wrote:

> Thanks @XhmikosR <https://github.com/XhmikosR>! Do you think x86 is more
> used over x64? I think only testing a different architecture on one node is
> a good idea but don't know how to configure that. I would like the code to
> hit both at least once, just in case something ends up using a native addon
> or has an architecture issue, we'd know.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gruntjs/grunt-contrib-concat/commit/49536bb5dbc69525f0e502e7df3e85597ea2ca7f#commitcomment-16228974>
> .
>


Ce commentaire n'a pas lieu d'être :)


Checksum error in the encrypted file /storage/emulated/0/Download/CVE-2016-1287.7z. Corrupt file or wrong password.


This code is similar to the old version [resolve.c#L481](https://github.com/racket/racket/blob/master/racket/src/racket/src/resolve.c#L481), but I haven´t read all the details to be sure if it's better to change it too.


Thx


Sounds good to me! I very much appreciate your help! :D


Je pense que c'est plus propre si tu mets seulement `member` en paramètre et que la concaténation du nom se fasse dans l'alerte
Idem pour la fonction removeMember


This should be `totalTicks`, see: https://github.com/EA31337/MT-Formats/blob/master/fxt-405-refined.mqh


Fixed. I didn't update the addons with the newest wppost commits.


What about fixing the quick start guide to align with installing 3.6?
There is mixed content there for 3.4 & 3.5.

http://www.ovirt.org/documentation/quickstart/quickstart-guide/


The content of `result` here is constrained by XCCDF standard. It will be always one of: pass, fail, error, unknown, notapplicable, notchecked, notselected, informational, fixed.

Does it make sense to create a separate enum table for this? Or do you think it would be premature optimization?


Awesome!


Fix http://review.cyanogenmod.org/#/c/133777/


Should be using medbook:primary-collections which includes the schema.


Well done!


когда я делаю zip(*matrix) он превращает строки в столбцы


This is a property, not a field. You aren't assigning it, you are generating a new value on each call!


Oh, yeah, it does make the `SimpleDualType*` classes mostly useless. Yeah, I let's remove them.


You should add a version attribute here too. That might be what's causing you the  message mentioned in the discussion forum


Add I am a girl


Nice :) zajtra to vyskusame


Так и пиши. Потому что распаковка тут происходит строк. Зип уже, в свою очередь, сшивает элементы в строках в столбцы.


Если ты действительно погимаешь, как это работает, не хочешь убрать что-нибудь лишнее?


I'm really sorry - I'm still learning - I think I'm conflating time_to_cut with number of turns (Turns being 6 seconds) when I think it's actually action points? So that minimum 500 is referring to 500 action points or roughly ~5 turns, is that correct?


CacheLoader를 활용하지 않으려면 LoadingCache보다는 그냥 Cache를 쓰는게 좋지 않을까요?

<http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/cache/Cache.html>


Please use `type(exception) == StopIteration`.  `isinstance` is slow and people don't subclass `StopIteration`.


How should they be made consistnet?


Я говорил, что не надо импортировать так.


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


Correct, `System.Net.Http` has been replaced by `Microsoft.Net.Http.Client`
for DNX Core. I am currently working on a DNX Core application that will
eventually need Consul. I was hoping that what I currently have on my fork
would be enough to get it working, but a bit more is needed. Because of
changing priorities at work, I haven't had a chance to work on my fork
since. I am hoping that I can find time to get this working in DNX Core in
the next week or two. Don't worry, I'll open a pull request as soon as
things are working with DNX Core. Let me know if there is anyone else I
should be collaborating with on this project.

On Fri, Feb 19, 2016 at 4:57 PM, highlyunavailable <notifications@github.com
> wrote:

> Hi, I noticed you forked and made some changes to remove PushStreamContent
> and there was an issue filed related to that as well: PlayFab#41
> <https://github.com/PlayFab/consuldotnet/issues/41>
>
> What specifically is causing you not to be able to use this as currently
> written? Was PushStreamContent removed in DNX Core and that's what you're
> targeting?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nbahramzad/consuldotnet/commit/b0614a0b4634f8ab1f0b0cccaed0e7a88fb8f234#commitcomment-16216071>
> .
>


Hier sind die Tabs in Leerzeichen gewandelt worden, die hätten aber entfernt werden müssen für den Treppenstufeneffekt.


Awesome!


Hier ebenso.


Seriously? It makes a noticeable difference? Wow. Okay...


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


Well done!


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


Syntaxes/Symbol List - Heading.tmPreferences

Seems to be working without this change on my end. What exactly is the purpose of this additional space here?


.gitignore добавь к проету чтоб лишнего не было))


Great!


Wouldn't keep this version, but bumping the release, trigger an update?


@SalmonGER Habe eine neue Nummer und kann dich nicht mehr erreichen :(


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


Won't this leave the .zip file in place after?  Even in /tmp It grows the image.


Made cosmetic, formatting edits to Gary's text, to make lists clearer


I would have said `Math.exp(log * Math.LN2) - 1`


в оригинале результат выдавался через yield, ты на это намекаешь ?


:bowtie:


nice one!


ну, можно сказать и так


This is even better http://review.cyanogenmod.org/#/c/133778/


Ugh, yes, I had forgotten. I like to leave things unpublished for a couple of weeks to let folk try it first, but seven months is ridiculous. Thanks for the reminder.


What is the exact warning text when you don't have the suppression there?


independent set of eyes: Looks to me like all subs are correct


I guess this might not be the right way to do. The class should not be dependent on the config file. Or, you should pass EsCfg to the class's init method so that it can extract params from it.


mistohise i needed a quick name... This is a mockup still... Suggest another


Something like this would be OK:

```python
raise StopIteration("StopException interacts badly with generators "
                    "and cannot be raised into a Future")
```


not sure, I didn't want to touch what was working and just changed what needed to be made working.


@ItzQuiet +49 015904435827 Yolo.


хитрец :) а мы в своё время обходились всего несколькими элементами :)


did u just delete my name?


Yeah, yeah, still a mockup to get the concept of this kind of design based on dsamples


just to make the indentation more visible. With only one space, on my computer, the indentation is so small to be recognized.

vs


Exercicio 1 Resolvido


Unlock here. No need to use stats_copy. stats_ is only used by this function/thread, so it's safe. You do need to add a timer outside to see how long does it take to finish UpdateRoutes. This decides how fast our system can adapt to channel condition change.


Thanks. I did not see this until after submitting another fix for the same problem. I'll watch the Pull Requests more carefully in the future


тут яростное дублирование кода :)
кажется, что всё таки здесь стоило бы только считывать, а если ничего не вернулось - уже в другом месте брать дефолтные значения.. ну это может быть только кажется..


isEmpty()


Exercicio 2 Resolvido


@amir20 Simply because I'm running a command-line app and these messages appear when I haven't subscribed to these events. :)


@SanketDG what's this line for?


List


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


@pulyaevskiy I like the economy of `has` (I thought it was my code!), but I think it's probably better to imitate the `Map` interface here and spell it `containsKey`.  Thoughts?


Maybe this comment is something not required? We won't be having files ending with .sdbf


Тут не понял, поподробнее, пожалуйста


Is this method get_SDBF_files() required?


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

Первый раз вижу вот такой подход с созданием переменной типа  Exception :)


@hongyanli No. But Frank committed a file without your name just after you. Please do again! Get the file again. Sorry!


There's an extra space here before the close parenthesis of the if.


Do we need the SDK constraint?  I'm afraid I've forgotten what was introduced/broken after 1.8.


Ты тут кажется сделал просто нереально много раз inverse :) что-то мне подсказывает, что это не самая дешевая операция.


another extra space


An extra space (there are two spaces) before the {.


Согласен, но ведь суть одна и та же - невалидная строка, или выход за рамки допустимых значений, разве есть в таком случае смысл делать 4 разных вида исключений?


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.526.0](http://build01:8888/viewLog.html?buildId=14348&buildTypeId=ConfigInjector_CI) is now running


If this is a problem, then set a variable and call the SNMP function later. If not, there's no need for the FIXME.


Add lock() here when you change the routing table. Please update other UpdateRoutes function to match this.


> the indentation is so small to be recognized.

I'm guessing you use tabs?


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.526.0](http://build01:8888/viewLog.html?buildId=14348&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:34


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


Oh, that's for the list itself, wow! :)

LGTM :+1:


Красавчик, больше запятых


```js
GeneExpression.upsert({ // using medbook:primary-collections
  gene_label: doc.gene,
  sample_label: sample_label,
  "values.quantile_counts_log": {$exists: false},
}, {
  $set: {
    gene_label: doc.gene,
    sample_label: sample_label,
    collaborations: doc.Collaborations,
    // very specific set object so that other normalizations aren't rewritten
    "values.quantile_counts_log" : log2,
    "values.quantile_counts" : counts,
  }
});


how can I include headers in Threading folder from other headers in common project without this line ?


@scharffc Ah okay, I just did again. Thanks


should this be `smx.cassandra.component`?


Generic comment:-
No need to give space after variable name and ','. Use saveDir(this, dstdir, indexName): instead of saveDir(this , dstdir , indexName):. This applies to other areas of code as well....

Generally, the code should follow python PEP 8 standard: https://www.python.org/dev/peps/pep-0008/

Also, every fxn end and new fxn start should have 2 lines spacing

Spacing within fxns in a class in 1 line, I have learnt it from the syntax highlighting errors I used to receive when using an IDE.


I wont be able to contribute much until Wednesday afternoon. But I promise
I will get my part done before Thursday night.

On Sun, Feb 21, 2016 at 1:52 PM, Jasper Micah Davey <
notifications@github.com> wrote:

> All done. Please test and submit bugs if any 😁
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jasperdavey/CS490Project/commit/b0fd9a31242426bd20986f19405c17144f06b2af#commitcomment-16228393>
> .
>


Google closure compiler doesn't like the `new` keyword here. If you remove it and return `this` from the `reverseIter` function you'll be good.


Wow, I wasn't aware that you can add templates for the scaffold command. I may actually end up using it again ;)


@urmastalimaa do you mean something like this?

I haven't addressed other comments yet. Just tried to make it work with callbacks.


Since the files are not stored with .sdbf index, the coments looks confusing. Please correct me if I am wrong.


To go to the next line?


Thanks, fixed. Wanted to have a working example. Honestly I haven't paid
much attention to what's stated in agreements when created that test app
and hash ids...

On Sun, Feb 21, 2016 at 6:43 PM, Luca <notifications@github.com> wrote:

> Hi! It's never a good idea to post your api id and hash online. I believe
> there's also a warning on my.telegram.org when you create them. Please
> consider removing them.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vmagaziy/Telegram/commit/0410605410316fc4c8d9a90edab6c4cf0c24b10b#commitcomment-16227994>
> .
>


My mistake!  Thank you for catching it.  I have removed it from the repository and added it to the .gitignore file.


@hurdtech I love the idea of the Social Media Usage "guide/charter"! I think we should start with a framework for completing the charters so people can fill in the blanks as they make their way through, and potentially fork different versions (e.g., industries, job titles, etc.). Happy to hear your thoughts! I create these dummy files in the master branch to chat about the entries because I couldn't find a way to do so in the wiki.


Looks like this commit didn't make it into opencivicdata/pupa. Worth creating a PR?


`eccryptoverify.h` and `ecwrapper.h` don't exist. I also do not see them referenced anywhere else in this commit.


This check is for debug purposes, there is no need to make an expensive call here. And yes, `isinstance` can be very slow.


This commit is so hot right now.


<!-- PMABOT:TAB -->
This commit is using tab character for indentation instead of spaces, what is mandated by phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: templates/getDivForSliderEffect.phtml


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: templates/getDivForSliderEffect.phtml


The old algorithm to produce gaps/seams between bricks simply scaled the brick by a constant factor in every direction. This will cause gaps that are bigger in the direction in which the part is the longest, which results in weird and physically inaccurate gaps.

I modified it to use a different scaling factor for every direction, based on the dimensions of the brick in that direction, in order to produce gaps of equal size in every direction, regardless of how long the part is in that direction, and makes sure the import scale of the model has no effect on the relative size of the gaps.
For future reference: if the size of the gap should prove to be either too large or too small, simply change the gapWidth variable to keep things working in every direction. The value of 0.007 was chosen purely on sight for aesthetic reasons, and has no physical foundation.


Instead of name as saveDir(), can we use index_documents() or some better name? And instead of variable "dstdir", we can use "path" or "dir"?


removed the inconsistent whitespace


@urmastalimaa by proper serializing did you mean something like this or did you mean implementing Serializable (or smth else) interface and using `encode` method instead?


Are we using file_dest? I am not sure why we need it if we are indexing it to elasticsearch. Maybe by mistake?


нет, я имел ввиду не это.


delete


I think you want to do `Users.insert(currentUser)` because I don't think we want to do `user.user.userName` to access the `userName`


Let's encrypt currently uses [a 30 days grace period too](https://letsencrypt.org/howitworks/) (search for 30 days on the page). Since there is a chance that renewal will not work immediately (e.g. because of the current strict rate limits), it's a better idea to be conservative about this for now.


Same thing as above.


Hi, I've never used github so please make the change yourself :)
and thanks for the nice script! It helped me a lot


I guess you mean the kinda nasty branch? I can probably refactor that out, but it seemed the most convenient way to get it in there.

________________________________
From: Massimiliano Culpo
Sent: Sunday, February 21, 2016 11:43:03 AM
To: trws/spack
Cc: Scogland, Thomas Richard William
Subject: Re: [spack] llvm trunk version (5c8dd6c)

@trws<https://github.com/trws> You may want to remove this duplicate [:smile:]

—
Reply to this email directly or view it on GitHub<https://github.com/trws/spack/commit/5c8dd6c3c86400ac82b061ce45f58b543526bfff#commitcomment-16228656>.


This is wrong. We are opening the file but not closing it. We should use "with open(filename, "r") as sdhashFile:" like syntax where you don't have to worry since it will close it once it exits from it.


You just need the `<text>` tag here now, in the `else` statement :)