HTML5中新增的标签

脚本：
<template> 通过JavaScript在运行时实例化内容的容器

章节：
<section> 定义文档中的一个章节，一般来说会有包含一个 heading
<nav>定义一个含有多个超链接的区域，这个区域包含转到其他页面，或者页面内部其他部分的链接列表
<article> 表示文档、页面、应用或网站中的独立结构，其意在成为可独立分配的或可复用的结构
<aside> 定义和页面内容联度较低的内容--如果被删除，剩下的内容仍然很合理
<header> 定义页面或章节的头部。它经常包含logo、页面标题和导航性的目录
<footer> 定义页面或章节的尾部。它经常包含版权信息、法律信息链接和反馈建议用的地址
<main> 定义文档中主要或重要的独一无二的内容。一个文档中不能出现一个以上的<main>标签

组织内容：
<figure> 代表一个和文档有关的图例
<figcaption> 代表一个图例的说明

文字形式：
<data> 将一个指定内容和机器可读的翻译联系在一起。但如果内容是与time和date相关的，一定要使用<time>
<time> 代表日期和时间值；机器可读的等价形式通过datetime属性指定
<mark> 代表一段需要被高亮的引用文字，用来表示上下文的关联性
<ruby> 表示被ruby注释标记的文本，如中文汉字和它的拼音
<rt> 代表ruby注释，如中文拼音
<rp> 代表ruby注释两边的额外插入文本，用于在不支持ruby注释显示的浏览器中提供友好的注释显示
<bdi> 代表需要脱离父元素文本方向的一段文本。它允许嵌入一段不同或未知文本方向格式的文本
<wbr> 代表建议换行，当文本太长需要换行时将会在此处添加换行符

嵌入内容：
<embed> 代表一个嵌入的外部资源，如应用程序或交互内容，换句话说，就是一个插件。如<embed type="video/quicktime" src="movie.mov" width="640" height="480">
<video> 代表一段视频及其视频文件和字幕，并提供了播放视频的用户界面
<audio> 代表一段声音或音频流。 <audio> 元素可以包含多个音频资源， 这些音频资源可以使用 src 属性或者<source> 元素来进行描述
<source> 为<video>或<audio>这类媒体元素指定媒体源
<track> 为<video>或<audio>这类媒体元素指定文本轨道（字幕）
<canvas> 代表位图区域，可以通过脚本在它上面实时呈现图形，如图表、游戏绘图等
<svg> 定义一个嵌入式矢量图
<math> 定义一段数学公式

表单：
<datalist> 代表提供给其他控件的一组预定义选项
<keygen> 代表一个密钥对生成器控件
<output> 代表计算值
<meter> 代表滑动条
<progress> 代表进度条，用来显示一项任务的完成进度

交互元素：
<details> 代表一个用户可以（点击）获取额外信息或控件的小部件。与summary配合使用
<summary> 代表 <details> 元素的综述或标题
<menuitem> 代表一个用户可以点击的菜单项
<menu> 代表菜单，呈现了一组用户可执行或激活的命令。这既包含了可能出现在屏幕顶端的列表菜单，也包含了那些隐藏在按钮之下、当点击按钮后显示出来的文本菜单。


You can add it :)


No problems, regressions are the developments normal roadmap ^_^


In other words, the router should just tell our app that something has happened, then the controller decides what to do. This is a clean separation of concerns, and means we can test by spying on the eventSubscription


***Code Quality: Nitpick***

white-space declared multiple times, if white-space is declared multiple times the previous value is over-written. and won't be rendered in the dom


Recommend using Plek for this so we get the correct URLs outside of production.


We decide to return an Optional, and then it's logical that an empty input return an Optional.ermpty.
Antoine


Yes, This routine isn't used anywhere except this file.


I know but how is it related to the createMailboxSession feature ?  It leaks implementation details


Hi,

Nice job.

I did make a couple changes related to the adodb library (there were some differences between mysql and mysqli):
https://github.com/bradymiller/openemr/commit/ec3d43d0f07ebbddc17bf149c2bad7e69741c4b0

Here is the full commit, which I plan to ask the community to review before placing into the codebase:
https://github.com/bradymiller/openemr/commit/c35c91b1834acd70b35796bb9afcc643dacbe0b6

-brady


darn ;). ok.


I don't think you need a version check here, do you?  Adding and removing with undefined hooks should be fine.


Do we not still need versions of these tests?


if(1 == 1)


HPX_APPLICATION_STRING appears over 250 times in HPX, whereas HPX_APPLICATION_NAME is <10
Are both really needed?


Rather than "email" (on the presenter) we refer to this as the "subscription_url" elsewhere.


@bbatsov Mh, isn't this a global mode anyway (e.g. `:global t`)?  As far as I can see, `super-save-initialise` globally advises switching commands and adds to global hooks, doesn't it?  Thus, toggling this mode in one buffer would implicitly enable it for all other buffers as well, wouldn't it?


You are very welcome! :)


Shouldn't this be appended rather than being overwritten ?


how does it work?


ㄷㄷㄷㄷㄷ Rinity의 시작잉가


Are you 100% sure that this field is autoupdated on major changes?  The one in whitehall can be set manually.


With your reasoning, many inputs should give us many outputs ?


A specific example is the IdP  provided by http://unitedid.org/ which we are investigating as a replacement for openidp.org


They may be hardcoded but we could still use Plek so they work outside of production.


Sure thing


Hello,
I just want to minimize the modification.


OK, this discussion is the reason I wanted to do this PR. :)

So does it makes sense to introduce new UI event? Like `Donations.UnauthorizedRouteRequested`. `RouteController` should listen to this event and publish `RouteRequested` with `routeName: 'login'`?

There we have ping-ponging between router and route controller.

If we ditch the flow router triggers feature than route controller could do security checks which would be complicated.

Can you explain your suggestion in more detail?


> Putting only one check per line is a bit inconvenient.

The new PR adds support for compound expressions and chained operations in general.

> Casting integer literals around is annoying as hell...

The cast is not mandatory actually. But then you will get the warning for comparing integers of different signs. The previous version simply does the cast under the hood, and hides it from the user, which is not a good thing. Doing the cast implicitly has other problems as well.

> The output also seems to be a step back compared to the original code.

Have I changed the test output? I don't intent to.

> Also your equal_to for floats is just weird. What benefit do you see over epsilon comparison?

The original version is incorrect. `std::numeric_limits<T>::epsilon()` gives the machine epsilon, that is, the difference between 1.0 and the next value representable by the floating-point type `T`. So the base line is 1.0. Given a magnitude other than 1.0, the epsilon value should be scaled accordingly. I also get rid of `std::abs(x - y)` for subtraction leads to a precision loss.


Looks good. I'd change the media driver peer-config option to :onyx.messaging.aeron/media-driver-threading, and make it a keyword. Once that's done I'm happy to merge it.


Great! Thank you for reviewing! A fresh view on the code is always helpful! :)


return empty string if all is ok? Very bad :-(
Shouldn't the code look like
>>>
if ( is_wp_error($field_value)){
        if (defined('WP_DEBUG') && WP_DEBUG == true) {
            return sprintf("%s: %s",__('Error generating thumbnail, reason',$mf_domain),$field_value->get_error_message());
        } else {
            return "";
        }
    }
<<<


Hello!


we should probably have benchmark for read/write performance too.


Isnt it ml2_mech_drivers += ...... ?


This was already possible before, though. We've only added endpoints to delete users. I don't know if this should be featured.

Would it be worth featuring the new Skype sharing button as well as the new Social Menu theme tool instead?


Hi,
what about embedded linux that use OpenGLES, like Raspberry Pi / I.MX6?

IMHO the detection of GL support should be totally decoupled and not bound to operating system checks (where possible).
Maybe libepoxy can help: https://github.com/anholt/libepoxy


Looks like title could be fetched from `attributes`, `slug` could be derived from `base_path`.


Почему Night style? Там же раздел "Стиль карты"


***Code Quality: Nitpick***

float-left is declared multiple times for .col-1, .col-2, .col-3 ... col-7, css selectors, it will be a good idea if this properties are grouped under a css selector like so
```
[class*='col-'] {
    float:left
}
```
*NB: assuming it doesn't affect external libraries


lolol, I'm a git pro. revoked the token ;)


There is no need to do this only when the TypeError occurs. We should always remove NULL characters from the message.


I think it'd be best to order the list alphabetically, both for enhancements and bug fixes. It's easier to skim through.


This commit prevent Spark (spark-shell or any other spark-submit application, and probably TimelineClient) to start when one of the Namenode in HA mode is down.
See more details in the case I did open https://community.hortonworks.com/questions/9586/spark-in-yarn-with-namenode-ha.html
This part might be excluded is HDFS is running in HA mode.


The preferred form is `"\x00"`


тёмную


I don't see any NULL characters added to this message (`\x00`)


Запятая не нужна


After this code goes into the main codebase (guessing it will go in over the next couple days), then the next step will be testing OpenEMR on PHP7 to see what happens.
-brady


In that case, can you please have a look at https://github.com/pbatard/rufus/blob/master/res/localization/ChangeLog.txt#L18-L27?

These are the changes still needed to bring the Dutch translation up to the latest, as it is currently at version 1.0.18 and new messages have been introduced for 1.0.19. Please see [this](https://github.com/pbatard/rufus/wiki/Localization#Editing_an_existing_translation) if you need more info about editing an existing translation.


It's good that you added a functional test, but developers are required to add unit tests.


***Code Quality: Awesome***

Consistent use of shorthand css properties


Отступ


it was bug 59, not 60!


А где эта строка будет?


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


I didn't, but we should since it's much likely that some of potential users could use this version of Java.
About the Javadoc I recently opened an issue (see #18) explaining how to make it available with the latest version of Java. Normally everything's fine, so please let me know if you observe any pb.
We could also think of making it available on website.


@mateuszgorniak this could be caused by using branches, not sure yet...


:heart: this commit, but Hound is right about everything here ;)


Sounds good :)


No problem ;-) It's accepted anyway. I just wanted to let you know about it ;-)


To replace with PS one :)


This does not work! If you don't define any secondary storage (which happens 99% of the time), this will fail.

Will open an issue.


***Code Quality: Awesome***
Its a good practice to group style sheets section together using comments. and also separating sections with new lines


Hallo Kascper,
der Code benötigt eine dringende Überarbeitung. Die Dokumentation ist nicht vorhanden und der Code ist sehr schwer verständlich. Die Variablen sollen eine sinnvollen Namen bekommen und der Styleguide eingehalten werden. Zudem sollte der Commit eine sinnvolle Commitnachricht bekommen.


Probably all these changes of "post" to "call" are erroneous as they would undo https://github.com/Hostle/luci/commit/8bb749ecc3b5f7f836f744f0056e90ac78522926

Or are they intentional?


$request->isGet to $request->getIsGet()


Minor thing, but unnecessary white space change.


```
game = prepare_game()
game.player1.discard_hand()
statue = game.player1.give(ANIMATED_STATUE)
for target in (statue, game.player1.hero, game.player2.hero):
 game.player1.give(MOONFIRE).play(target=target)
cs = game.player1.give("EX1_392")
cs.play()
assert len(game.player1.hand) == 2
```


Could use a service interface, and an Adapter service decorator instead, to match what is done for adapter services.


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


In general you dont want to do that only for tt_content but for all tables that are referenced. So the last join should be left out.


is passwordhash still needed here?


Tag to be removed


probably unintentional change of post to call.


this results in a compiling error. :)
../src/smt/theory_seq.cpp:1243:9: error: 'tout' was not declared in this scope


в ios


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 201](https://teamcity.labkey.org/viewLog.html?buildId=296714&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:39


No, Intention for this was just to remove `MultiJSON` as dependency.
We can use `MultiJSON` if need it.


hey @dkayiwa
According to the [maven-resources-plugin documentation](https://maven.apache.org/plugins/maven-resources-plugin/), starting with version 2.3, this plugin uses the Maven Filtering shared component for filtering resources, and from [this ticket] (https://issues.apache.org/jira/browse/MSHARED-325), it looks like the maven-filtering plugin had issues and they were fixed in maven-filtering-1.3

Since we hadn't specified the version for the maven-resources-plugin, we could fix this by specifying a version below 2.3, but maven-resources-plugin prior to 2.4 is not supported by m2e, you have to use maven-resources-plugin version 2.4 or later.

I chose to set the version for maven-resources-plugin to 2.7 because this version forces you to set maven-filtering to 1.3 or higher.


Fair enough. Halogen is a bit overkill. Let's simplify!


Mein Commit, kein Plan warum Git den als dschmidl gepusht hat.... Username und Password hab ich ja eingegeben...


I know I'm late to the party here but this check is executed every time we clone a container and `btrfs_same_fs()` calls `SYSERROR()` on all occasions where it can fail. This will lead the user to see e.g. when he clones a container error messages like `lxc-copy: bdev/lxcbtrfs.c: btrfs_same_fs: 243 Inappropriate ioctl for device - BTRFS_IOC_FS_INFO /some/path` when it is not a btrfs fs.


I'll pass this, so you won't forget ;)


Couldn't we just assert the presence of the loader?
It seems unnecessary to add the 'privateComponent' class.


Nope… Not sure why PhpStorm didn't auto-remove it either…


// TODO: See if this can be removed when switching to the $request class as it use to work!


For any reason now the order moved back from "Firewall, Services, VPN" again to "Firewall, VPN, Services"...
I could not see any changes in the `order` value of Services and VPN, so I don't think there was an intentional change on that, but before that commit VPN and Services were swapped for me...


gui cho kinhnv


You could also call initStore with the initialState argument:
```javascript
initStore({authentication: {isAuthenticated: true}})
```


Without using `<seg>`, the matching in Southey was done like this:
>   `<anchor ana="start" corresp="#M1_p1_c65"/>seeing<anchor ana="end"/>`
>   `<anchor ana="start" type="add"/>and hearing,<anchor ana="end"/>`
>  ` <anchor ana="start" corresp="#M1_p1_c65"/>their religious purposes be overcome,<anchor ana="end"/>`

Montalvo's clause M1_p1_c65 is used in two different parts of Southey's translation. That was being displayed as it follows:

Southey | Montalvo | Word count rate | Analysis
---------- | -------------| ----------------| ----------------
seeing | con las vistas su sancto propósito dañan: | 6 | Match
and hearing, | | | Addition
their religious purposes be overcome, | con las vistas su sancto propósito dañan: | 0.4 | Match

That was problematic for a number of reasons, including misgiving data when counting the words.

By further segmenting Montalvo's clause, as you can see in the lines I'm commenting here, Southey's anchors can be better identified by using the `<seg>` IDs instead:
> `<anchor ana="start" corresp="#M1_p1_c65_1"/>seeing<anchor ana="end"/>`
> `<anchor ana="start" type="add"/>and hearing,<anchor ana="end"/>`
> `<anchor ana="start" corresp="#M1_p1_c65_2"/>their religious purposes be overcome,<anchor ana="end"/>`

And that would be displayed as:

Southey | Montalvo | Word count rate | Analysis
---------- | -------------| ----------------| ----------------
seeing | con las vistas | 2 | Match
and hearing, | | | Addition
their religious purposes be overcome, | su sancto propósito dañan: | -0.2 | Match

To sum up: there can never be two `<anchor>` in Southey with the same ID. If we need to repeat it, it means that we should go to Montalvo and add as many segmentations as needed to do a more accurate match. Luckily, it doesn't happen that often.


Вимкнуто


Увімкнуто


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


***Code Quality: Nitpick***

@media all covers screen for all devices, this defeats the purpose of setting media queries. a much better syntax would have been
```
@media only and (min-width: 320px) and (max-width: 600px)
```
Here is a  [post](https://css-tricks.com/snippets/css/media-queries-for-standard-devices/) that explains more on media queries


Thanks a lot for this contribution !


breaks CompressIntegrationTest, will make this an id


The string starts with `{` but has no closing `}` at the end.


The string starts with { but has no closing } at the end.


верни файл назад


этот тест должен быть в DefaultControllerTest


player player


Can we be sure it is always a StablePool? Is it the casting safe?


Is there a reason why not to validate the `packageId` here through `PackageIdValidator`?


x


I suggest that `should_default_to_snapshot()` simply take another argument `char  *bdevtype` and we check if it actually is `btrfs` and bail if not. `copy_storage()` calls `should_default_to_snapshot()` and itself takes an argument `bdevtype` so we can just hand it down to `should_default_to_snapshot()`. I have two trivial fixes for `lxcbtrfs.c` and `lxcoverlay.c` in line (but that bug me).  If you're ok with this I'll just combine my trivial fixes with this.


please keep naming convention: ANONS ON MOBILE SECTION


redundant


holy shit...my /usr


Whoops, good catch, will double-check


Nice job!


useless then ?


Uses comma to replace / in physical_nics/BOND/VLAN/BRIDGE


Perfect! Thanks!


ownership Social I guess


This does not match the pool type, which lacks `ha_host_failures_to_tolerate`.


cool @Piicksarn !!


resolved


Actually, that is the intended order, it might have been switched because of this? https://github.com/opnsense/core/commit/62e14f6ff9cbf365f5c60913d0d5418af8b52d75

I removed the <service> ordering in the IDS module, it shouldn't have been there in the first place.


what if user registers pre-defined or existing compressor?
I suggest throwing exceptions...


Since this example is for the whole doc: I recommend use a new line like:

Following example will be used in the whole doc to describe how to use confignetwork postscript.


Any reason to do this on the staging branch?


Reviewed-by: Nicolas Palix <nicolas.palix@imag.fr>


For the record: VPNs are a service, too, more important than the other ones, thus this order firewall, vpn, services. Sounds ok?


***Code Quality: Nitpick***

Use 3-character hexadecimal notation where possible. this line could have been better written like this
```
.color-dark-gray {
       color:#444
}
```
it makes it look more succinct


This change by @simonb breaks profiling in our case of statically linked binary. I can provide binary sample proving that. It seems that our case is looks like ".text is not in the final segment". Btw, what means "final"? By address range or what? Sorry for stupid questions.


I know that many of the above functions are coming from different, existing modules and `_deserialize_sdata()` is static, but do you think it would be worth it to unify naming a bit?


remove this line since from up down to read the doc, reader should know the next chapter will describe how to use confignetwork


Will be accesses through magic method. I think should be left as is.


:heart:


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


Zou jij doen toch?


This log file should not be committed.


Lets keep it in some `.config` file and use `${}` parameters.


This title is too long, recommend to be： Define nic attributes in nics table


This was updated in a subsequent commit (for a different reason).


Yeah, you're probably right. Just a force of habit... :-)


Ah, I add that field in a later commit - things got a bit out of sync in the rebasing.


Where are you define these properties?


Thanks, will fix.


Ditto.


plese keep naming convention: annon/loggedInUser+device+functionality
eg. annonUserOnMobile....


@KeizerDev nobody gets in trouble, the icons are licensed under [Creative Common Attribution 4.0 International License (CC-BY 4.0)](https://creativecommons.org/licenses/by/4.0/).


plese keep naming convention: annon/loggedInUser+device+functionality
eg. annonUserOnMobile...


+1 this


added some test fixtures


😷😷😷😵😵😵😲😲😲😲😟😟😟
😍😍😍😍😘😘😘😘😘

On Thu, Jan 14, 2016 at 2:32 PM, MohammadJafar MashhadiEbrahim <
notifications@github.com> wrote:

> Nice job!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sarahakhavan/OSProject/commit/8b4a7c8d844095082f1afa753bf718a48a3d6378#commitcomment-15446984>
> .
>


Where is this defined? why it can be only 8? is this macro used anywherE?


yes, that's correct.


We recently made contacts-fronted use links instead of details for organisations (see: https://github.com/alphagov/contacts-frontend/pull/41) and as part of that we had to stop using `abbreviation` because it's not part of the representation of an organisation you get from content store.  Will we have the same problem here?


Might want to check if the new Thorim boundary makes him evade immediately (if the platform is outside it), and exclude EVADE_REASON_BOUNDARY during platform phase if it does.


Please break the lines to make it easy to read:
~~~~
chdef cn1 nicdevice.bond0="eth2|eth3" \
                 nicdevice.bond0.1=bond0 \
                 nicdevice.bond0.2=bond0 \
                 nicdevice.br1=bond0.1 \
                 nicdevice.br2=bond0.2
~~~~


Currently "Stacked Series" and "100% Stacked Series" are located in the Functionality folder. I didn't put them inside an inner "Functionality"/"Stacked Series" folder as they should be placed inside Chart Types folder which should be further split.


Can `$('''#favScroll')` be made into a variable?


写的很好啊，不过可以另外建一个 webpack.config.test.js。我的refetch里有一些phantomjs的单元测试，https://github.com/Lobos/refetch。用了gulp-watch监听webpack打包出来的文件，可以持续进行测试。


It's hard to edit nics table, so I'd like to remove this part. use def or stanza should be enough.


```and node Windows nodes``` from the commit msg should be ```and non Windows nodes``` ?!


<header>


What does `Читеришь, гадёныш?` mean?


Instead of the verb "flow" we should use Forward:

```
ForwardWindowsAuthentication
ForwardClientCertificate
```


Build error fixed :yum:
`frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/policy/Clock.java:500: error: cannot find symbol setTypeface`


Should we check that `d.build.Spec.Source.BuildSecrets` is not `nil` before passing it?


Looks fine


LeetCode 做题记录 ——Togo.


should recommend using the 'def' command instead of tabedit command


It is a very good question!
i will delete this block


how is this different from MBED_ASSERT? why that one is not used


it will never be "nil" is it is list, so it is always initialized to empty list... in case no secrets are specified, the `for` down will be no-op


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


Great stuff will try this on a PI... if anyone has already done so please  shout


Don't we need to change `a`.


@malirod I don't agree with you. Profile concats all paths with application storage path to pass complete paths to components. So components do not need to care about paths correctness. If components will concat paths as you wish we have to pass additional parameters (e.g. application storage path, system storage path) to them. I don't like this way in our current realization. Besides, if do you say we have to rework similar logic for another components. @pestOO, @AGaliuzov what do you think about it?


^^ les designers me posent des questions seulement maintenant, merci pour le tractopelle :+1:


@Tiryoh, thanks. I need to validate your changes with @yohru, since he recently updated the Japanese translation.
Please note however that you should not increase the version (1.0.19 → 1.0.20) on your own. The version is something that comes from [this Changelog](https://github.com/pbatard/rufus/blob/master/res/localization/ChangeLog.txt#L18) and that I am the only one to change.

@yohru, can I please ask you to review this proposal and let me know if you are okay with it?


So, then why we're using if+for later, in the sti.go? (https://github.com/mfojtik/origin/commit/5f91cba58e31560bf971cf8b6a8735aa8be798cb#diff-b9627ec6ab0689e50e5fa3e5d958c1d3R145)


@php-coder that if is redundant, we can remove it


This part is not clear, please refine.

the network table set should be put before
~~~~
chdef cn1 -p postscripts=confignetwork
~~~~


I did not see the part that how use it
* os deployment
* updatenode

Also you can give out the output of your example for user to refer.


Chúc mừng


ありがとうございます。修正します。
Getterのみの自動実装プロパティってC#6.0からなんですね、ハマりました。
(悪いことにプロジェクトの詳細設定で言語バージョンを4.0にしても検出されないという…)


@malirod agree with you that for our purposes (we only use this wrapper in pipe_streamer_adapter) such interface is redundant, but it could be useful in future. As for me my approach is more consistent but I don't insist. @pestOO, @AGaliuzov please participate


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 128](https://teamcity.labkey.org/viewLog.html?buildId=296710&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7 (1 new), passed: 42 Build time: 01:22:53
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

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

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

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

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

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

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

```


@josepjaume This is a use case for Redux! Communication between siblings


exFat failed after this commit. Can't detect nor format the external sdcard. Please check.


This comment still applies. Can you add this back.


Commit


Ruby2.2+ Support patch author is charliesome .
Copied from as follows:
 https://github.com/tmm1/gctools/commit/b2dd2f9b06dbed8e673e895012be92aaf8842108
 https://github.com/tmm1/gctools/pull/14


We have such logging inside pipe wrapper implementation. I think it will be redundant


Comments are publicly visible :-D


@JGinn I understand this design is much simpler as it simplifies the metrics. At the same time, I think that showing string like `345 seconds` may not be very user-friendly for >= 60 seconds.

After some research I found this solution that:

1. would not make metrics worse, and
2. keep the view logic in at view-level, instead of in the model.

The templating system allows for Date, DateTime and Time objects (actually, any object) to be manipulated in the view. For example, returning the video time as a Time object, it would be possible to use
```erb
<%= time.strftime("%M:%S") %>
```

What do you think?


nie wiem czy to ma sens. Wyrzuciłbym sprawdzanie validacji do testu modelu. Jeśli nie jest valid to wiesz że się nie zapisze. Tutaj bym tylko sprawdził czy dowolny zły parametr sprawia że nie tworzy i renderuje 'new'


My bad. Missed an assertion. We can't throw an exception because user might want to update other information


It cherry-picked incorrect.
I added this later again but multirom deamon don't start with 5.0 kernel :( And no any logs and ideas about that.


Please could you also squash this commit into the one introducing the comment?


percentual isn't a work :wink: maybe you mean percentage.


wcięcia ;)


No, wait. You need to add mentioned parameters for `hibernate.cfg.xml` too


Of course, it must be easier than on cm-13.0, 5.0 and 5.1 audio/media parts are very similar.


WTF, why so much lines are changed? =\
ENCODING!..


Yes, this is compatible what we have in PE.
We've talked about the name of this module, but we haven't better name, have you?


@CytoDev I knew it already! :wink:


Checkmarx scan results 14 Jan 2016 11:34 UTC +00:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
4
33
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

ad030f7b673e3ef26e01d1ede8d18e7229e0908c	@macxil


「アプリありますよ」って言われたので、なぜアプリがあるのにわざわざサポートツールを作ったかを主張してみた。


Isn't this causing whitespace in the exception message?


Ok, but you also should minimize future issues for users.
I have test your expression `/(\w*)=(\w*)\(([\w-]*)\)/` and doesn't work for me with "user-test" username. But works with `"/(\w*)=(\d*)\(([^\s]*)\)/"`.
It's important because an invalid expression Pydio just lauch an error like "you are not allowed to access folder XXXX" and It's not true.
I think this regex should match all valid usernames possibilities (including ldap, pam auths) or Pydio should alert on bad username matched.


@dkayiwa Actually this test is invalid. This test is actually trying to update uuid field and throwing ConversionException. Do you have any idea how can we test this scenario?

But  the code is making sure the logged in user is same as the user you are trying to update.
https://github.com/openmrs/openmrs-module-webservices.rest/commit/72f71a6e31061ba89d8c03c0fd519945989b427d#diff-52c2c66b2873b085320eeec968635a04R152


добавь \n, пожалуйста


Hmm, I actually think we're best off ignoring the trigger and groups features (at least for this use-case) and consider how we would handle this is a first class `Space.ui.RouteController`. I think Arunoda initially wanted to keep Flow Router free from these types of concerns for this very reason. Triggers and groups are good when you don't have formal event-driven/flux architecture, but here we have a different scenario.

Maybe this will help: What would happen if we wanted to swap Flow with something else?


можно сразу this.value = startingValue;


Have you updated the download link as well? https://rhiot.gitbooks.io/rhiotdocumentation/content/cloudplatform/starting.html


Unused constructor and cause of warnings "...multiple default constructors specified".


и тогда здесь будет: this.value += +prompt('Введите число:', 0);


@aspiers actually, it got introduced in another PR...


How about this instead?  Given the intended functionality, making the strip optional is redundant:

diff --git a/lib/PPI/Xref.pm b/lib/PPI/Xref.pm
index 27b3aec..adccbd4 100644
--- a/lib/PPI/Xref.pm
+++ b/lib/PPI/Xref.pm
@@ -196,9 +196,15 @@ sub __close_open_package {
 # a volume name in pathnames, but the main idea is to safely split
 # the argument into a new directory name (possibly modified by
 # prepending the volume name as a directory), and the filename.
+# E.g.
+# '/a/b/c' -> ('/a/b', 'c')
+# 'c:/d/e' -> ('/c/d', 'e')
 sub __safe_dir_and_file {
     my ($self, $path) = @_;
     my ($vol, $dirs, $file) =  File::Spec->splitpath($path);
+    if ($^O eq 'MSWin32') {
+      $vol =~ s/:$//;  # splitpath() leaves the $vol as e.g. "c:"
+    }
     return (File::Spec->catdir(grep { length } ($vol, $dirs)), $file);
 }


// TODO: See if this can be removed when switching to the $request class as it use to work!


TODO: improve these regexs, using the following stolen from PHPCI (sorry Dan!)


Can I haz a space after the comma in the array?


хм... или я условие неправильно помню)) сейчас найду


Use the real tags here ie 2.8.14 for SS7 RAs see https://github.com/RestComm/jain-slee.ss7/tree/2.8.14
Do the same for HTTP and JDBC, don't use master it may make things unstable


if it is `loops` then should be `look` instead of `looks`, shouldn't it?


`payload` should be `data` btw.  Should we raise a separate issue for this?


resync


Without the `brew tap --repair` every CI build was failing. I presume it should be fixed with the addition of the `git clean` last night though?


Also, as far as I can tell, the description method will now return a message without a `name` field for messages, and without the action fields for Presence messages, should we not include that somehow?


E.g. for windows, this is not just path. This is special format, it's windows specific. Why profile should care about windows specific rules for the pipe names? In my mind the profile should just read the ini file and keep its representation in memory. There should be no logic at all.


yep of course ;-)


Why do we duplicate the task disposing mechanism? If it's been changed, it could cause a problems with consistency.
```javascript
task.abort();
task.removeAllListeners();
await browserSet.dispose();
```


I considered this, and it would be ideal, but I suspect that there would be
a challenge since the time within the video is stored as a number of
seconds (since this is what the YouTube api provides and requires), not a
time ruby object. However, if there is a simple way to cast it as a date,
this would be ideal.

On Thu, 14 Jan 2016, 11:30 Alfio Emanuele <notifications@github.com> wrote:

> @JGinn <https://github.com/JGinn> I understand this design is much
> simpler as it simplifies the metrics. At the same time, I think that
> showing string like 345 seconds may not be very user-friendly for >= 60
> seconds.
>
> After some research I found this solution that:
>
>    1. would not make metrics worse, and
>    2. keep the view logic in at view-level, instead of in the model.
>
> The templating system allows for Date, DateTime and Time objects
> (actually, any object) to be manipulated in the view. For example,
> returning the video time as a Time object, it would be possible to use
>
> <%= time.strftime("%M:%S") %>
>
> What do you think?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DAMS-Bale/flippd/commit/b16fa86ca75b8bf96fc9c99c639b4149bf3f713b#commitcomment-15447483>
> .
>


Thanks!

czw., 14.01.2016 o 12:44 użytkownik Greg. A. <notifications@github.com>
napisał:

> yep of course ;-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rhiot/rhiot/commit/6638bbb1f8f2b4ef7ca27a7b1fccd51acb3e9caf#commitcomment-15447708>
> .
>
--
Henryk Konsek
http://about.me/hekonsek


@aspiers squashed


хм, хотя вот задание:

Напишите функцию-конструктор Accumulator(startingValue). Объекты, которые она создает, должны хранить текущую сумму и прибавлять к ней то, что вводит посетитель.

Более формально, объект должен:

Хранить текущее значение в своём свойстве value. Начальное значение свойства value ставится конструктором равным startingValue.
Метод read() вызывает prompt, принимает число и прибавляет его к свойству value.
Таким образом, свойство value является текущей суммой всего, что ввел посетитель при вызовах метода read(), с учетом начального значения startingValue.


How does this ensure that any path that might be cached in a `StyleSet` attached to `pParent` (or one of `pParent`'s parents) is already up-to-date?


`brew tap --repair` is used to purge the old symlink files in `Formula` folder. Should them already be covered by `git clean`?


/r-


'provider' finde ich besser als 'provider_name'...


Oh wait, it should be possible to `Q_ASSERT(pParent == pItem)` inside the `if`, right?


Did you ran it on Tomcat?
Tomcat has his own servlet API - use it.


Sweet! Thanks @mikepetersonccv!


Nice catch here @collinsnji How did i miss this?


Arrrr - can't we just use scalariform or something :)
Haven't even read that one, sorry!


I have added a test script that demonstrates (and tests) my answer to question 2 above - lets call this running planemo around conda.

https://github.com/galaxyproject/planemo/blob/master/tests/scripts/conda_test.sh

If anyone really cares about the use case, feel free to extend the example in either of these directions:
 - Create a new tool example to with a test case that actually installs something useful in the environment
requires that something for the tool test to path.
 - Add the script as a tox test.
 - Add the script as a travis test.


@alfredodeza see https://jenkins.ceph.com/job/ceph-pr-commits/633/console for https://github.com/ceph/ceph/pull/7232,

```
ceph-build/ceph-pr-commits/build/test_commits.py::TestSignedOffByCommits::test_signed_off_by Running command: git log --no-merges master..master
PASSED
```

so both `$ghprbTargetBranch` and `$ghprbSourceBranch ` are "master", as the contributor created the pull request from his branch named `lzhng2000:master` targeting `ceph:master`. and  that's why the test always passes.

probably we should use
```
git log --no-merges origin/$ghprbTargetBranch..$ghprbActualCommit
```

or simply
```
git log --no-merges origin/$ghprbTargetBranch..HEAD
```

`origin/$ghprbTargetBranch`is used here, because i am not sure if the "master" contains the commits from the pr or not.


The same for jstl.


It's odd how Werror just pointed me to the same. As a workaround, I put `*version = 0` way more up locally, but I agree that it doesn't matter.


@anosach-luxoft, agree with @malirod these functionalty might be move into the pimpl


Use log4j or slf4j instead of this.


Hmm, thinking about this: How about this psedo code:
```
foreach(child in visual children or qobject children of item) {
  if (uiPathParent(child) == item) {
    doPropagatePathTo(child);
  }
}
```


OK


@anosach-luxoft , I think in future we will add this metod. Dont think that we really need stuff which we will not use


Add JavaDoc, which describes the logic that this impl of doGet does.


This last `false` is necessary as it disables the automatic reboot. Otherwise, the following EEPROM call to come back to NORMAL mode is never called.


Klopt, maar heb er even een comment bij geplaatst zodat ik het niet vergeet.


Oh no sorry, this was introduced in 2.1.0, my bad!


@JGinn sorry, I forgot to include it in the original comment, I edited the comment to include the code to return a Time object. Could you give a quick look at the comment via github?


Comment should be in English.


OK, looks good, but some minor changes should be done.


Yeah, it was used in this situation to overwrite the old Caskroom installation manpage. I thought `git clean` would work here but the CI builds are failing again after one of the last few changes pushed with the error before: https://travis-ci.org/Homebrew/homebrew-fuse/jobs/100983879


Hello @molinch & @daniel-luberda
Seems, that issue is still present in 2.0.3, I'm getting this on different devices running Android 4
 FFImageLoading.Drawables.SelfDisposingBitmapDrawable.SetIsDisplayed(Boolean isDisplayed):0
 FFImageLoading.Views.ManagedImageView.UpdateDrawableDisplayedState(Android.Graphics.Drawables.Drawable drawable, Boolean isDisplayed):0
 FFImageLoading.Views.ManagedImageView.SetImageDrawable(Android.Graphics.Drawables.Drawable drawable):0
 FFImageLoading.Work.ImageLoaderTask.SetImageDrawable(Android.Widget.ImageView imageView, Android.Graphics.Drawables.Drawable drawable, Boolean fadeIn):0
 FFImageLoading.Work.ImageLoaderTask+<LoadPlaceHolderAsync>c__async4+<LoadPlaceHolderAsync>c__AnonStoreyF.<>m__1():0
 FFImageLoading.Helpers.MainThreadDispatcher+<PostAsync>c__AnonStorey0.<>m__0():0

Is there any way to fix/overcome this?


Oh my, this is awesome :+1:


This method will create a new connection each time is called, please review the Singleton pattern


Was this repair that got things working again: https://github.com/Homebrew/homebrew-fuse/commit/4af98b4cbc411dda8a97b55c0d59486d120c87ca

Without those we run into:
```
Error: Cask 'osxfuse' definition is invalid: Bad header line: parse failed
```
Again. But I'm not sure why `git clean` wouldn't necessarily do the same job there.


@molinch We should modify current implementation to use ConcurrentDictionary


So you propose to pass application storage path to streamer adapter on linux?


Extended note: also restyled the buttons for idle workers slightly, were made weirdly before.


There are locks everywhere the dictionaries are used so I don't think it's needed.
@Mookker could you please create an issue and post your stacktrace there?


@Mookker This stack trace is not complete. What is the exception?


The CI error you referenced uses the old `.travis.yml` file. Therefore, it doesn't run `git clean`. Until it rebases against master, the error will stay.


The path is relative to the current page. When you click on Submit Event, then you go a level deeper and the relative path breaks.
You can use an absolute path to always start at the same location `src="/assets/DT-logo.svg"`.
It would also be nice if you could add a `alt="Ticket"` or something so we get some descriptive text instead of just the missing picture :)


hahaha


Ah, of course. Forgot that happens with Travis. I'll ask him to rebase again and see how it goes this time.


Is better to always use entire words for parameters, that makes code more legible.


Of course you can disable the xdebug in php.ini just comment it out with ";"

Regards


@daniel-luberda Message: [InvalidOperationException: Cannot redisplay this drawable, its resources have been disposed.]
@molinch
Will create an issue, than you.


This can be anything that you can pass to a ngDisabled directive. Like: "myModel-property == 'whatever'"


We actually now have a pixel value from the design. It says 20px. Do you want to change it?


缺少了 `lisp/magit.el`


Please add a main method that shows how will you use the classes you created.


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


This class is not in the repo (anymore).


***Code Quality: Critical***

Comment section in HTML should be descriptive of what the markup does. It should contain meaningful helper messages for example this could have been written like this

```
<!-- Contents -->
markup content goes here
<!-- /Contents  -->
```


I beg your pardon, sir


Why not to pass ref to profile? so the ctors of both pipes (win and posix) will be the same. The option I like less is to have different ctors for the posix (and pass app_storage_folder_) and win (without additional param).
There is another option. Add new param to the pipe ctor `pipe_name_prefix`. And pass on posix the app_storage_folder_ and on windows pass the constant `\\\\.\\pipe\\`


The class `SpecificAvroDeserializer` does not exist in the repo.


The class `SpecificAvroSerializer` does not exist in the repo.


I like the order "Firewall, VPN, Service", because it always used to be like that... :D
I most of the time navigate through the menu "blindly" and always navigated into the wrong menu... ;)


You're right.


***Code Quality: Nitpick***

For consistency there should also be an HTML comment tag that signifies end of the HTML grid like so

```
<!-- /Header -->
```


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


`return null`


***Code Quality: Nitpick***

For consistency there should also be an HTML comment tag that signifies end of the HTML grid like so

```
<!-- /Main Image -->
```


***Code Quality: Nitpick***

For consistency there should also be an HTML comment tag that signifies end of the HTML grid like so

```
<!-- /Featured Work -->
```


I have not yet found a way to share this single file, that I like. A submodule is too much, placing it globally makes packaging harder…

There are multiple Files that are nearly identical (args.py, config.py, some parts of connection.py/controlserver.py) which I'm not particularly happy with. If you have a good idea let's chat about it on irc.


I waited, waited, and waited.


What @hashier said!


Thread? Should be `Test`.


okay


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


Das kann man ja noch ändern, genauso wie der Name der Models bzw. der Tables.....


I think `prefixLengthFromSubnetMask` is a better name. Bit long but it's a rarely used feature.


Number 4 is fine, please remember to add exercise 2.


It returns a number of leading ones.


Dear Anna,

is there a difference between getCurrentTransformation and getJointPosition ?

Thanks,


Add please PotionItem class


But compile fails.


功能都已经实现，手机像素可能有现实上的问题


Did you fix it already? I wanted to finish with the race-detection algo before responding to you, but now I cannot reproduce the error. Instead I get a bunch of "Unsatisfied constraint for global ..." messages.


It is simpler and looks better, OTOH it leads to collisions with native function names...


Seems good


Wooops, messed up the count.


***Code Quality: Nitpick***

I'll advice you choose a  consistent style across the HTML file markup. This section could have been commented like so
```
<!-- Projects -->
```


Oh yeah, never mind.


Is convenient for legibility to keep uninplemented methods at the top of the file


@RMcGirr83 should I create a PR for changing the type in PHPDoc comment, because it is not the string now?


Ah yes, that's cleaner.


Fixed


Ah crap, if you wouldn't mind.  Thanks!


tack så mycket det värmer mvh


Please remember to format the code.


Just a notice that this will not be supported from Jan 22 and forward. https://github.com/rust-lang/crates.io/pull/213


Nice, please submit the rest of the exercises too.


You are welcome and I'm glad to help!


Thanks. I realise those weren't an issue with the present PR, but worth addressing.


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


Test comment


Is great that you coded more than one topic, but please remember to submit each topic in its own pull request.


CSF. skybox.html


Yeah..


I think the reason is the way DRF handles fields. All the fields are defined as class variables, as opposed to instance variables. When DRF introspects the class to discover the fields, it probably doesn't check the superclass, which is probably why mixins don't work.

I therefore tried the method recommended in drf-nested-routers about using NestedHyperlinkedRelatedFields. That had some specific assumptions about the model structure which was difficult to adapt to our case, but by following the same style, it seemed to work out ok:
https://github.com/galaxyproject/cloudlaunch/commit/040bd3c10dc729dec5c7b01bda95594998b2031a


The pseudo-code is really nice! But you confused me with your first two comments:

> How does this ensure that any path that might be cached in a StyleSet attached to pParent (or one of pParent's parents) is already up-to-date?

Why would you care?

> Oh wait, it should be possible to Q_ASSERT(pParent == pItem) inside the if, right?

No, I don't think so.


> helpers which always pass the code and info arguments

IIUC, the defaults 'guarded' against those arguments being undefined - the helpers merely pass them to the constructor. If nothing relies on this behaviour, then this should be safe, though.


Units here are only 'Gauss', and not 'Gauss per second', I believe.


Should do this for server-called transfer votes, too.


it looks good


Sorry if I don't fully understand how to install it in ASP.NET 5, but I guess things are not clear / documented.

http://getglimpse.com/Docs/Installing

I guess the steps to take for < ASP.NET 5 and ASP.NET 5 are different.

> Note: The Glimpse Core (Glimpse) package contains the Glimpse framework infrastructure, and will be installed as part of the other Glimpse packages; you shouldn't install it on its own.

And for ASP.NET 5 I think it should be exactly just the 'Core'? (I think I saw this in a ASP.NET Community Standup)
    Install-Package Glimpse -Pre

I added aspnetvnext as a source and installed "Glimpse.Agent.Dnx"

Now this package is missing, could that be the problem in his breaking changes issue?
"Microsoft.Extensions.Options.ConfigurationExtensions"

It would be nice if this part can be added to the Documentation > Installation

Thanks in advance!


I'm not sure why this change is necessary though, it only triggers with my supercluster tiles. Not sure what's going on. cc @ansis


Is better to keep variables not initialized and their value will be null unless you need a default value set.
In Strings is best practice to keep them null until used.


If you are happy to create a PR, I can merge this.


Unit here is only 'kPa', and not 'Kpa per second', I believe.


@barbosa

About the separator: I created a view that wraps time labels and the separator so we can hide all these components instead of setting labels text to "" and hiding just the separator.


No, but with the new commits, the error no longer occurs.


Ok, sorry I missed that when reading on my phone. This solution looks good!


prova a vedere se ti da qualche problema...potrebbe non funzione...se non funziona devo inserire un altro controllo.


OK, so how do we store multiuser metadata? access control lists? extended attributes?


I believe it should be 'username' => 'admin', to prevent "Could not parse for environment rp_env: Syntax error at ':'; expected '}'" error message


It seems to be a valid word?
http://www.dict.cc/?s=percentual


instill was the intention:

> gradually but firmly establish (an idea or attitude) in a person's mind.

Maybe it should say:

`-with the Thales Simulator to help instill the concepts of LMKs in the reader's mind.`


Ugh. Oh well, I guess it'll be safer to stick with a single version, less risk of new changes breaking something.


test comment


You could have used an Enum for the DB Types, thats a nice to have anyway, there is no need to fix that now.


```javascript
function(req, res, next) {
            req.on("end", function() {
                var meta = { // we can add any meta data we want
                    url: req.path,
                    method: req.method,
                    params: req.params,
                    licence_key: req.user.licence_key,
                    appid: req.user.appid,
                    user_agent: req.headers["user-agent"],
                    host: req.headers.host,
                    status: res.statusCode
                };
                log.info("============ request end ================");
                log.info(JSON.stringify(meta, null, 4));
                log.info("============ request end ================");
            });
            next();
        }
```


Hi officesec guy, did you submit this as a patch request?  I can't recall seeing it come through.


The First repository :rabbit:


A quick grep through the codebase did not reveal any calls relying on `JT-ERROR` being used as the default.


Tyhle returny bez returnu jsou fakt divočina:D Ale jinak kód mi přijde v pořádku... neumím CS natolik, abych tu našel něco špatně.


TeamCity Civic Assembly :: Build backend + frontend [Build 160](http://54.229.176.143/tc/viewLog.html?buildId=441&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


What do you think about something like this:
'''
LogMsgSerializer *self = NULL;
  Plugin *p = plugin_find(cfg, LL_CONTEXT_SERIALIZER, name);
  if (!p)
    {
      g_set_error(error, LOGMSG_SERIALIZER_ERROR, LOGMSG_SERIALIZER_NOT_FOUND, "Unknown serializer name \"%s\"", name);
    }
  else
    {
      self = plugin_construct(p, cfg, LL_CONTEXT_SERIALIZER, name);
    }
  return self;
'''


Merge branch 'spyder-ide/spyder/master' into developer 'master'


It's probably because of gcc5. Earlier gcc versions didn't see into called functions for initialization.


Kernel for exchange recovery tool can easily repair corrupt or inaccessible exchange server database as well as retrieve edb files and save into  RTF, MSG (Unicode/ANSI), HTML, EML, Text formats. If exchange server database is damaged or inaccessible due to any reason then fix it by using this tool. Read more http://www.nucleusdatarecovery.org/exchange-server-edb-recovery.html


On another hand I'm going to provide file interface with `Open`+`Close` and `Create` + `Delete`. Maybe it would be better to keep similar interface for pipe and file?


I don't think it is possible to import a module from itself. This file should not even define `__version__`. If someone wants to know the version, they should call the `get_version` function.


***Code Quality: Nitpick***
Inconsistent indetation between the <html> tag and <head> the head should be indented two or four spaces like so
```
<!DOCTYPE html>
<html>
    <head>
          <!-- Rest of the head content -->
    </head>
       <!-- Rest of the html content -->
<html>
```


如果我闲下来我试试去哈


Great work, I added a few comments but those are just nice to have, there is no need to fix the exercises.


I remove it to check output all the output should be a geojson. https://github.com/osmlab/osmlint/blob/09fef7af06c589c3192b21da40845659add6c82b/test/test.js#L51 , I saw you  add again ,  hmm,some tiles are empty and it throw null back on those tiles,  i saw it CLI.


or see here: http://dict.leo.org/ende/index_de.html#/search=prozentual&searchLoc=0&resultOrder=basic&multiwordShowSingle=on


On windows pipe is like a socket. And I think this is more correct. So I prefer not too mix file interface and pipe interface. On my opinion they are different.


Whats is happening? Has @marcheing worked on this?


@Rub21 - we don't have to check for geojson validity because turf.featurecollection will always return a geojson. It's not okay to write empty geojsons to the stdout. I've changed all the tests and added new fixtures.


I don't think that 'per second' applies here...


Коммит точно в мастер ветку?


iiuc, it would be if we would reverse the order between `take` and `drop`?


Sorry, typo: `dataOject`. It is also always set.
Maybe check for `if (!is_null($this->dataObject->get('activity_id')))`?


I'm not sure, that your proposal is better, than the original one.
I think, it is harder to understand.


It seems like you are still trying to run Galaxy inside of conda. I'd recommend focusing just on ``--conda_dependency_resolution`` and let planemo install Galaxy normally and outside of conda.


> > How does this ensure that any path that might be cached in a StyleSet attached to pParent (or one of pParent's parents) is already up-to-date?

> Why would you care?

Because `refreshPath` will find a cached, out-of-date path in `pParent`'s (or `pParent`'s parent's) StyleSet when traversing the path up, and the path change won't be propagated.

> > Oh wait, it should be possible to Q_ASSERT(pParent == pItem) inside the if, right?

> No, I don't think so.

Inside the innermost if? Like this:
```c++
  if (auto pItem = qobject_cast<QQuickItem*>(pRoot)) {
    for (auto pChildItem : pItem->childItems()) {
      auto pParent = pChildItem->parent();
      if (!pParent || !pParent->isWindowType()) {
        Q_ASSERT(pParent == pItem);
        doPropagatePathTo(pChildItem, visitedObjects);
      }
    }
  }
```
Why shouldn't this work? If `pChildItem` is in `pItem->childItems()`, then `pChildItem->parent() == pItem` should be true, right? Ah wait, no, `pChildItem->parentItem() == pItem` would be true. Does this mean that your current solution is incomplete? Anyway, it seems like this is one more reason to reuse `uiPathParent` here methinks...


I think __constructor is a good name, but it should be static


Oops- you are right.

v == 1

Should be the check. I will fix it on GitHub when I get home from work.
Thanks!

JEREMYSTOLL.ORG

> On Jan 13, 2016, at 9:39 PM, Sumeet Jain <notifications@github.com> wrote:
>
> Is v <= 1 the right check? Less than 1 would be zero, which means an element is not in the array at all.
>
> —
> Reply to this email directly or view it on GitHub.
>


fixes #19


We should be `strip`ping this string earlier, so if the user enters `" Jacob"` we don't put that space in all of the activities.


Lovely change. I'm a big fan of column 3


Also you have not needed condition. Much easier is just
int max = arr[0];
for (int i = 1; i < arr.length; ++i) {
    if (max < arr[i]) { max = arr[i];}
} That's all. But you each iteration compare 2 items and then max with 1 item.


RTC_CLOCKDIV is used together with the LF reference frequency to determine the frequency of the RTC counter (not necessarily 1Hz), and be able to calculate the counter value back to seconds. Not sure where this _INT thing comes from, will investigate.


LGTM


Yes I have.
This is the 'work in progress' result of a refactor of the metric configuration controllers. It was pushed into a branch because @danielkza preferred a remote branch to trading patches ^^


Should be MBED_ASSERT, will fix.


Why this change?


***Code Quality: Nitpick***

Avoid user agent detection as well as CSS "hacks" whenever possible —try a different approach first. browsers support css box-sizing property  out of the box


Dude, would this work with old versions of chrome? (is stream.getTracks()) supported? See my solution (sorry no link, writing from mobile phone).


TeamCity Civic Assembly :: Build backend + frontend [Build 160](http://54.229.176.143/tc/viewLog.html?buildId=441&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests passed: 1348, ignored: 8; exit code 1 Build time: 00:15:39


The PSR coding standards for this project state that this should be written:

```php
$responseWithJson
```


Oh, my...


Потому что настройка включает или отключает функцию приложения "Ночной режим"


Why is ssl verification disabled when a http_proxy is set? Seems like on oversight from a local dev setup.


do what you wish. It isn't a word I've ever heard or used.


Would we rather refactor the controllers on a separate branch as a specific story?


`is_numeric` should do it (or changing the DuplicateFinder to accept non-numerics and return false + adjustments in unit tests).


Thanks for the password dude :P


I don't thinks line is doing anything for you


package algo2015;

import java.util.Scanner;

class EuclidGCD {

	static int gcd(int x, int y){
		if(y==0)
			return x;
		else
			return gcd(y,x%y);
	}

	public static void main(String[] args){
		System.out.println("2つの整数の最大公約数を求めます");

		System.out.print("整数を入力：");
		int x = extracted().nextInt();
		System.out.print("整数を入力："); int y = extracted().nextInt();

		System.out.println("最大公約数は"+ gcd(x,y) +"です");
	}

	private static Scanner extracted() {
		return new Scanner(System.in);
	}
}


The reason of it is made pluggable was, we would like to make it possible to switch serializer (e.g. json, avro, anything). Later the user can select serializer for example in disk-buffer.


It doesn't work


Sorry I missed this earlier but this is going to cause problems, because friend names appear all the time in activity descriptions in accidental ways. Here are two:

`2016-01-14: **Dan** and I went to John's Diner.`
`2016-01-14: **Dan** and I visited the Johnson & Johnson company headquarters.`

In both of these cases, if we have a friend named `"John"` and try to rename him to `"Jacob"`, we'll end up messing up these descriptions.

The correct way to handle this is to have a regex that checks for leading and trailing asterisks with something like this:

```ruby
activity.description.gsub!(
  Regexp.new("(?<=\\*\\*)#{friend.name}(?=\\*\\*)"),
  new_name
)
```

That's starting to feel like it might be better as a method on `Activity`, like:

```ruby
def update_name(old_name, new_name)
  ...
end
```


I have the latest version include the "green" lines


fyi @matethurzo


Hi @luben, just wanted to ping you on this. What you'll see with this test is that the `com.romix.scala.serialization.kryo.ScalaCollectionSerializer` doesn't instantiate the `akka.util.ByteString$ByteString1C` here due to a lack of a no-arg constructor. the `fallbackInstantiator` is `org.objenesis.instantiator.sun.SunReflectionFactoryInstantiator`, but the object it creates does not in turn create the correct type when `Builder.result` is called.

This is beyond me what's going on and your help would be *greatly* appreciated as I've already buried a week of nonstop time in this. IJ won't let me step into `result`, so I'm having a problem understanding why it's choosing to not convert the `Vector` into a `akka.util.ByteString$ByteString1C`.

Could I beg you to take a look at this? :)


I think a string.Format is missing from here and the item below.


Having something like 0.2.* would work fine. The idea is to actually go with proper schematic version http://semver.org/ so 1.x would be fixes to the existing version and breaking changes comes in 2.x


Why is this method here if all it does is call another method with the same arguments? The `replace_old_name` method can be removed and its internals can just be in here.


Please consider closing L<...>


Thats my fault, I will correct it using only the single underscore conventions


Beginner's question: won't this return a boolean?


752 строка, то же самое?


package algo2015;

import java.util.HashMap;

public class MapSample {

	public static void main(String[] args){

		HashMap<String,String> map = new HashMap<String,String>();
		map.put("red","赤");
		map.put("blue","青");
		map.put("green","緑");
		map.put("green","緑");

		System.out.println(map.get("blue"));
		System.out.println("");

		for(String key:map.keySet()){
			System.out.println(key + "="+ map.get(key));
		}
		System.out.println("");

		map.remove("green");

		for(String key:map.keySet()){
			System.out.println(key +"="+map.get(key));
		}
	}
}


I actually prefer the named arguments that you used to have here, because it makes it more explicit which argument is which when calling the method.


I think the rule should be that casting within a package must be considered safe


Minor suggestion: I think it might look nicer if you change `NEWNAME` to `NEW_NAME`.


Please make use of LocationService instead of PersistenceService, incase if you want any additional functionality wrt Location please add it to LocationService itself


```
├─────────────────────────────────┼─────────────────────────────────┤
│ Total Unique Font Sizes         │ 20                              │
├─────────────────────────────────┼─────────────────────────────────┤
│ Unique Font Sizes               │ small                           │
│                                 │ x-large                         │
│                                 │ xx-small                        │
│                                 │ large                           │
│                                 │ x-small                         │
│                                 │ 0.4em                           │
│                                 │ 0.5em                           │
│                                 │ .5em                            │
│                                 │ 2em                             │
│                                 │ 2.4em                           │
│                                 │ 3.2em                           │
│                                 │ 5em                             │
│                                 │ 2rem                            │
│                                 │ 3rem                            │
│                                 │ 11px                            │
│                                 │ 12px                            │
│                                 │ 13px                            │
│                                 │ 90px                            │
│                                 │ 25%                             │
│                                 │ 30%                             │
├─────────────────────────────────┼─────────────────────────────────┤
│ Total Unique Font Families      │ 0                               │
```


No. Check this:

```
In [1]: a = None or 3

In [2]: a
Out[2]: 3

In [3]: a = 5 or 2

In [4]: a
Out[4]: 5
```

If the first argument is evaluated as `False` on python (e.g. empty list, empty string, None, False...) then the second argument will be assigned.


Еще одно, юзай полный вариант создания сервисов.


That can really reduce code. Nice!


fixed.


@renhwztetecs i think this is a denotation of an integer set between "up_from" and "up_thru", and it includes "up_from" but not "up_thru".


This was the result of trying to create a controller for tree metric configuration. The previous metric configuration needed a refactor, which would simplify this inheritance tree. Now, this refactor is maybe big enough to require its own story.


I had some freezes after new USB commits in adsrcp branch. Jfyi.


Use SLF4j and java exception multicatch


Summary after offline discussion: the current implementation is identical to the pseudo code variant. The latter is just much shorter, better readable and less error-prone. Will adapt the code accordingly.


Watch the caveats: 0 is also evaluated as False on python.


After first finish we should return...


OK, it's good that you caught up this problem.


You need to register your handles based on servlet init params in web.xml.
See method http://docs.oracle.com/javaee/5/api/javax/servlet/GenericServlet.html#init(javax.servlet.ServletConfig)


I think that you should use `"data_type": "rgb"` here, and set default accordingly, as `sol_rgb` members are actually of type `uint32_t`.


Is the test invalid or is it exposing a bug? :)


Okay convinced :smile:  Going to change it to `percentage` as this is more common.


Well, if that is the only thing holding you back, consider this alternative:

result &= (strlen(MESSAGE) == string->len);
result &= (strncmp(MESSAGE, string->str, string->len) == 0);

This is a proposal addressing functionality, not style. Without this, you
will miss catching NULL termination vs. garbage issues.

On Thu, Jan 14, 2016 at 1:33 PM, juhaszviktor <notifications@github.com>
wrote:

> I'm not sure, that your proposal is better, than the original one.
> I think, it is harder to understand.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/juhaszviktor/syslog-ng/commit/45f01241da7fbc0bd3e49056b4789ca1829beb01#commitcomment-15448567>
> .
>


Could we add a unit test for credentials missing?

I entered an incorrect path and received the following exception:
`NameError: global name 'FileNotFoundError' is not defined`

```
======================================================================
ERROR: test_should_be_able_to_make_a_connector (pandas.io.tests.test_gbq.TestGBQConnectorServiceAccountKeyPathIntegration)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/tony/pandas-gbq/pandas/io/tests/test_gbq.py", line 167, in setUp
    self.sut = gbq.GbqConnector(PROJECT_ID, private_key=PRIVATE_KEY_JSON_PATH)
  File "/home/tony/pandas-gbq/pandas/io/gbq.py", line 118, in __init__
    self.credentials = self.get_credentials()
  File "/home/tony/pandas-gbq/pandas/io/gbq.py", line 135, in get_credentials
    return self.get_service_account_credentials()
  File "/home/tony/pandas-gbq/pandas/io/gbq.py", line 175, in get_service_account_credentials
    except (FileNotFoundError, ValueError, KeyError, NotImplementedError):
NameError: global name 'FileNotFoundError' is not defined

----------------------------------------------------------------------
```


guess it's http://www.doxygen.nl/manual/docblocks.html#cppblock. and i don't think we need to change them.


You need to add logging in each action (via AOP for example)


Typo in the commit message, `throw`/`thrown`


Route makes it clearer to me.


@lmr thanks for catching the typo!


You should check if `tmp` value fits on `uint32_t` range before assigning. What if `tmp` is negative or very big? Probably it should not happen, but better safe than sorry. The same for all channels.


Which means that the warning is only appearing in some cases, right? You'll get it when the attachee is re-QObject-parented, but not when the StyleSet is attached to a *child* of an item that gets re-QObject-parented. Hmm, I guess that has to be good enough for now.


I'd change the order here, so it's the same as in SDL.Dialog.


According to the
https://docs.oracle.com/javase/specs/jls/se7/jls7.pdf
you have to change the package name


Does not look like clean code. It is not clear why you call handleFeelingSocial here even though it is correct.


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


Have you any idea?


@cliftonc do we need urls any more?  does this url make sense?


Just one minor observation, Jan. This commit does multiple things at once. It can be 5 separate commits each focused on a single change. Small commits are good, because developer's mind is focused on the single change and that generally helps to ensure everything is done well.


lgtm - @cliftonc can you advise on the urls?


Assert seems more correct here and below (I'd add a message)


It is not clean at all. It is really dirty. And I am not sure here if we are leaking Activity. :(
I am adding comments now. What else should I do?


What's the benefit here?


Indeed. Thanks for spotting, will fix.


Just remember to write more meaningful commit messages.

Also, for what I've seen of the code, the changes could be separated into several commits.


No, URLs are part of client / server so I suspect could be moved over to that project, surprised I didn't do that when I split them.


maybe just wrap it in a method "migrateFellingSocialAchievement" `activity.getAchievementActionHandler().handleFeelingSocial(numberOfTaggedOrganizers);`


1. brakowało "
2. testowałem też w innych miejscach i nie wszędzie jest 'trans("ID:")' i albo wszędzie zrobimy bez ID lub będzie trzeba dorobić tu if($params['description'])


That is my fault, I will correct it using only the single underscore conventions


I understand the reason behind your proposal and it is improved and correct
in your context of thinking.

Generally, I personally like it better if less variables, identifiers,
names and source lines are introduced in cases where this does not increase
the complexity of the remaining part. To each his own.

So all in all, for me, while this is not the most preferred form, it is an
acceptable compromise if you insist.

On Thu, Jan 14, 2016 at 1:26 PM, juhaszviktor <notifications@github.com>
wrote:

> What do you think about something like this:
> '''
> LogMsgSerializer *self = NULL;
> Plugin *p = plugin_find(cfg, LL_CONTEXT_SERIALIZER, name);
> if (!p)
> {
> g_set_error(error, LOGMSG_SERIALIZER_ERROR, LOGMSG_SERIALIZER_NOT_FOUND,
> "Unknown serializer name \"%s\"", name);
> }
> else
> {
> self = plugin_construct(p, cfg, LL_CONTEXT_SERIALIZER, name);
> }
> return self;
> '''
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/juhaszviktor/syslog-ng/commit/45f01241da7fbc0bd3e49056b4789ca1829beb01#commitcomment-15448445>
> .
>


@jpfr could you add a compile with lowest debug level to the one of CI-runs?


migrateFeelingSocialAchiement could be already called in line 49


forget about the last comment


Unfortunately I don't know that. At least 3 reported cases during last year, as I recall it, so it is not a gigantic number. If we remove the code and merchant using automatic-update there is always the risk that the payment method stop working. In those cases we could advise them how to counter the problem.


#4
`timeout`


Using org.jetbrains.kotlin.idea.caches.resolve.getModuleInfo and org.jetbrains.kotlin.fileClasses.javaFileFacadeFqName utilities in this function seems beneficial.


Hello Joseph,
the only difference is that getJointPosition returns a pose (position&quaternion) w.r.t. the world frame whereas getCurrentTransformation returns the same in the form of a transformation matrix.
Cheers,


add gitignore


@Stoux all hail SONARQUBE!


Please, put the curly bracket on the separate line.


I'm pretty sure automated votes already checks shuttle status.


typo, `below`


@richlv It would be probably better to define ``sfile`` beforehand so that we are checking the existence of the same sequence file (in Line #76).


Если не вводить переменную var a, тогда если присвоить (напрямую) this.value какое-то значение, оно начнет работать от этого значения (т.е. напрочь теряется смысл startingValue), а если использовать доп. переменную, то после вызова read - this.value вернется к значению  с учетом startingValue. Как-то так...


TeamCity Civic Assembly :: Build backend + frontend [Build 161](http://54.229.176.143/tc/viewLog.html?buildId=442&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


I think this needs a JPS plugin test


Moved above comment (with cleanup language) to the README.


Merged templates loginbutton and registerbutton, then refactored into different files


@alanliubj  应该不是，至少要绑定邮箱才可以指向  **torvalds**


@ldoktor OK, let me work on it!


@pentaho-nbaker  Missing license headers


That may be my personal way of seeing things, but I would keep this variable, because it makes it clearer that the following code uses same object in both conditional branches.
This is a proposal, not an obligatory change.


How about now. Git diff may look weird, please check the final source. :)


Why was this test removed? As far as I understand, this is still disallowed as binds `yield` inside of generator, but updated code allows this.


Correct. that is the issue I'm having.
I was thinking that if a json encoder (that you can pass to `json.dumps(object, cls=custom_encoder)` was provided, then I could write my own encoder for my custom classes and it would be a much better solution for the flask-profiler library than this hack I did to get it to work.


I receive the following exception when running unit tests. Is there another dependency?

```

======================================================================
ERROR: test_should_be_able_to_get_results_from_query (pandas.io.tests.test_gbq.TestGBQConnectorServiceAccountKeyPathIntegration)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/tony/pandas-gbq/pandas/io/tests/test_gbq.py", line 173, in setUp
    self.sut = gbq.GbqConnector(PROJECT_ID, private_key=PRIVATE_KEY_JSON_PATH)
  File "/home/tony/pandas-gbq/pandas/io/gbq.py", line 118, in __init__
    self.credentials = self.get_credentials()
  File "/home/tony/pandas-gbq/pandas/io/gbq.py", line 135, in get_credentials
    return self.get_service_account_credentials()
  File "/home/tony/pandas-gbq/pandas/io/gbq.py", line 164, in get_service_account_credentials
    json_key = json.load(f)
AttributeError: 'module' object has no attribute 'load'
```


Why remove solr and lets encrypt?
They are very usefull?


This declaration does not have any meaning here, the constant is used in wfnfont.cpp only (and declared there).


So, in other words, if it doesn't work, it's my fault, so I'll need a hug to feel better... lmao jk


Solr isn't removed, just an old variable.
LE will be implemented with an alternative client.


I added a fixed version for jquery. Try again but first delete the contents inside web/assets

https://github.com/chamilo/chamilo-lms/commit/14407f4c9615c7d23d4c4d93cc25f9f7aec6015b


Sorry, but I'm afraid I don't really understand this. Do you think it should be near the nvtable (e.g. in lib)?


ah ok :)


Why do this env.user.id all the time?  This does not make sense at all.  Everyone looking at it, will think he is responsible for it.


yes, you are correct.I was using 15.05 so you are right in all cases


I agree. Better revert, for now.


It is for my test. There is actually no official support for proxying in resources as far as I know.

If you know it is documented anymore, I'll add support for it.


I'm wondering if to set -U is the best solution, as it will force to create a group with the same name as the user, and add the user to this group.
if nothing is specified (-g, -N, and -U) it will follow the config files.
so if USERGROUPS_ENAB is set to yes it will create the group and assign it.
and if it's set to no it will follow /etc/default/useradd.


@HelenaSabel Thank you--this is perfect to help explain!


fix #67


Please propagate the returned `gboolean` error condition of `serialize_*`. This means a few more changes in helpers as well, and the `default:` branch.


alterando o readme


Should probably remove the sleep I added as well. Want to do that in this PR?

Otherwise this looks good to me. Thanks. I'd misinterpreted the problem. Strangely, it passed on my machine when I added the sleep.


I could have sworn that the PEP8 checker would've caught the misalignment above. ^


Hm... I looked into Google Benchmark doc and they don't say anything about reporting errors, so I think this might be fine... The only thing is that the benchmark will exit with status zero :/ I guess we'll need to bear with that.

Just one thing: could we print the message to `stderr`?


Please, follow the same style everywhere. If you leave brackets at the same line everywhere, there shouldn't be any exclusions- that would look much better. (that's for future)


I actually am not sure. Without the instantiation I was getting some errors deep in jsonobject and then saw the workaround right above. Didn't look closely at the root cause.


Might be related to this issue: https://bugs.freedesktop.org/show_bug.cgi?id=93703

But I don't know yet - will find out tonight.


I'm missing a `NULL` check on `addr`. I guess it's okay to not check `sa` because we consider the OOP object which this method operates on.


Solves tthbellcom/trckr#14 combined with b5f20b6


That was my first attempt, but I didn't found an approach that works. That was because this script builds the full command line, joining strings like "--optionName=$VARIABLE". Systemd is able to read variables but, if a variable is not setted the command line will have strings like --optionName=[EMPTY] throwing an error. That is because I've created two services, one that creates the command line as an environment variable and other that launch Jenkins using that variable.

https://github.com/amatas/ansible-jenkins/blob/master/tasks/install.yml#L26-L48

It's the same approach I found here:
https://github.com/coreos/docs/blob/master/os/using-environment-variables-in-systemd-units.md#environmentfile-directive


test change for commit


I will break in 80 columns. Thank you.


This is to edit the branch


From a benchmark perspective this looks good to me. There's just one comment regarding reporting errors.

P.S.: I didn't worry to look for the semantics of the VideoIn functions being called ;-)


TeamCity Civic Assembly :: Build backend + frontend [Build 161](http://54.229.176.143/tc/viewLog.html?buildId=442&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 1309, ignored: 8 Build time: 00:14:29
### Failed tests
```
venues_transit_stop.api.admin.v1.transit_stop.tests.WorkflowTestCase.test_state_filter_should_return_objects_in_draft_state: <no details avaliable>

```


As I commented above, I wasn't able to do that. If you know a way to do that with only one systemd service please share.


cc @adrianq


This type of logic needs to be in a public interface. Otherwise we break the TA interface, i.e. users cannot provide their own policies.

The better place to put this logic is inside the policy class. Something like
```c++
class DensePolicy {
  // ...
  template <typename P>
  using mult = typename mult_policy_trait< DensePolicy, P>::type;
};
```


What the reason to store index instead of value? :) It will work anyway - up to you


I think the `ITransactionSynchronizationTaskMergerService` was supposed to be implemented in application code. Now that we added our own merger inside OWSI-Core, there's no way applications can define their own.
Maybe `ITransactionSynchronizationTaskMergerService` should become `ITransactionSynchronizationTaskMerger`, and we should support multiple instances? I see no problem with multiple mergers being called successively.


> Bare ld doesn't need -nostdlib -nostartfiles since those are GCC flags.

-nostdlib can be found in ld manpage, at least. -nostdlib only searches specified paths for linking

> Lots of people say that using ld directly is bad and you should always use GCC. Pussies.

Haha, that was my thinking, too. Either way works, I guess.

> That __aeabi_memclr4 undefined reference is exactly what I'm getting.

So the issue is resolved?  We get the same errors if the linker script isn't at the end. I wonder why my system knew what to do with the linker script and yours didn't.

> I don't see ARM targets in your dir lib (maybe that's only linux/?),

There's only linux dir in lib.

>  By using ld we remove the gcc dependency.

That's a definite benefit. I just wasn't sure if that's possible at all. On arch, clang package even depends on gcc.


if choice.lower() == 'y':


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 66](https://teamcity.labkey.org/viewLog.html?buildId=296740&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 34 Build time: 01:04:26
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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


That's amusing...


atomic merge aborted.  Not prune I believe?  Just remove this line
Remove "atomic prune aborted"


Please, name your classes and functions more clear. If you return some value, then use prefix 'get'. In names I see a mess everywhere)


I see.  I do not see the logic behind that, however.  Wouldn't that slow down the runtime for the operation, as the first `order` method becomes needless?


I don't know why you need this.


We should just drop comprehension expressions altogether, as proposal was discarded.


I think `links` should be a list per the schema - [mdsol/ODMSchema](https://github.com/mdsol/ODMSchema/blob/master/ODM1.3.1/mdsol-extension.xsd#L342)


Perhaps just

raise ValueError("Please delete dangling images before running atomic migrate export")


Do you think a g_assert(addr) should be enough?


updatea


Is it a good idea to let the plug-in introduce a preference on how often to autosave?


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


This all has to be removed, no js gets added to mako files unless I say it absolutely needs to. If you're running into js errors this is where they're probably coming from as the page doesn't load everything in the "normal" order.


- [ ] `DefaultClassicDateFactory` ->  `ClassicDateFactory` (型の宣言はinterfaceで)


TeamCity Civic Assembly :: Build backend + frontend [Build 162](http://54.229.176.143/tc/viewLog.html?buildId=443&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Oops :blush:


Unit is only 'm/s^2', there's no need for the 'per second'.


TeamCity Civic Assembly :: Build backend + frontend [Build 162](http://54.229.176.143/tc/viewLog.html?buildId=443&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new); exit code 123 (new)) Build time: 00:00:44


Remove these, they're added via the main templating file.


Do You have https://github.com/CyanogenMod/android_frameworks_base/commit/dd2be59619c7b807a2ebd6ae951f45e73c9d0687?


Can you explain this commit? I don't geet what the problem was (Common was being included by all files via a compiler flag)


TeamCity Civic Assembly :: Build backend + frontend [Build 163](http://54.229.176.143/tc/viewLog.html?buildId=444&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Note that SQL diff above is misleading and looks like a bug in @github For reference check https://github.com/RestyaPlatform/board/blob/master/sql/restyaboard_with_empty_data.sql#L3040


this was done with the help of phs000424.v6.pht002743.v6.p1.c1.GTEx_Sample_Attributes.GRU.txt
if there is a library missing, it means the file did not give the library


This needs to be removed. Everything loads as a sub-template of `main.mako`.


添加了轮播图
![qq 20160114213141](https://cloud.githubusercontent.com/assets/8433954/12325646/a1620bc0-bb06-11e5-9c9e-7df0bcd65630.png)


Thanks @pgleeson :)


This should be "Compute the `constant out the front` in a more stable way.".


Fix-allocation-for-feneSolarDelaySeq (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


    (P ==> M), (S ==> M), neq(S,P), dt(bmint) |- (S ==> P), (Belief:Induction, Derive:AllowBackward)

    (M ==> P), (M ==> S), neq(S,P), dt(tminb) |- (S ==> P), (Belief:Abduction, Derive:AllowBackward)

    (P ==> M), (M ==> S), neq(S,P), dt(sumNeg) |- (S ==> P), (Belief:Exemplification, Derive:AllowBackward)

    (S ==> P), (P ==> S), neq(S,P), dt(avg) |- (S <=> P), (Belief:Intersection, Derive:AllowBackward)

...
    (S ==> M), ((&&,S,A..+) ==> M), dt(task) |- ((&&,A..+) ==> M), (Belief:DecomposeNegativePositivePositive, Order:ForAllSame, SequenceIntervals:FromBelief)

..
    A, B, after(forward), neq(A,B), notImplicationOrEquivalence(A), notImplicationOrEquivalence(B) |- (A ==> B), (Belief:Abduction)
    A, B, after(reverse), neq(A,B), notConjunction(A), notConjunction(B),  notImplicationOrEquivalence(A), notImplicationOrEquivalence(B) |- (A && B), (Belief:Intersection)


Mind using field identifiers?


/* special case: no address was stored */
:)


Could you remove this comment?


No need for the 'per second', just say that the humidity read is a percentage (%).


Added bases. Search option only for movie providers, not torrent, for them there is general Torrenter Search


Actually at some point this would be a great thing to add to both the `scala-js` and `scala-js-dom` build process, but noone's done this yet :)


updated indentation issues. not sure why it would say coverage is decreased...


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 90](https://teamcity.labkey.org/viewLog.html?buildId=296735&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 19 (1 new), passed: 274, muted: 1 Build time: 01:40:41
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.JUnitTest$RemoteTest.org.labkey.study.model.StudyManager$DatasetImportTestCase: <no details avaliable>

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

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

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

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

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

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


oohkay  got it.. buddy.. there are roms without these.. ZdrowyGosciu.... thanks for pointing that....
bw ur logo.. kitkat ??


This code path handles a case where user already exists. It seems to me that changing password of existing user to a random string, and then passing that to 'login' is at the very least inelegant, and possibly breaks username/password authorization? Would it not be better if cloud code could login as any user without a password?


Now it's going to fail on whitespace between keyword and semicolon, i.e. ```source  : whatever```
Make it ```E_CAPTURE_OPTIONAL( E_OPTIONAL_SPACE ":" E_OPTIONAL_SPACE )```


Please do not feel


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


But shouldn't this happen by default?  Could you provide a concrete example illustrating both cases: without and with

``` ruby
class Scope < Scope
```

That is, if `ApplicationPolicy` inherited from `Scope`, then shouldn't all of that inheritance be passed down onto `PostPolicy` as well?


You forgot `ic_launcher-web.png` :)


Can you move the assignment out of the `if`?


Add Thread.currentThread().interrupt() since other logic may be depending on Thread.currentThread().isInterrupted()


We need to take into account element-wise multiplication. Remember that the `MultExpr` represents both element-wise multiplication and contractions.
In the case of element-wise multiplication, a result tile is non-zero if the left- AND right-hand tiles are non-zero. In the case of tensor contraction, it is unlikely that the result is sparse, but it can happen if an entire row/column is zero. Furthermore, the Dense Summa would deadlock if one of the result tiles were zero. For correctness, the result MUST be sparse. In either case, Sparse should win, not dense.


To be honest I don't understand why strcmp is not good enough in this context. The MESSAGE is a predefined constant string, have to ended with '\0' char. The strcmp go while it doesn't found NULL character so the maximum value is the length of MESSAGE.


Is `getCI()->getASTContext().ExternalSource` now empty, after the `AutoLoadCB` protection?


Looks like there is some dodgy formatting here, could you fix?


TeamCity Civic Assembly :: Build backend + frontend [Build 163](http://54.229.176.143/tc/viewLog.html?buildId=444&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 20 (1 new), passed: 115, ignored: 4 Build time: 00:07:09
### Failed tests
```
Media type tests: Should not create new media type if cancelled: <no details avaliable>

Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

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


Of course I will correct the double underscore to single underscore


This approach looks good:

/usr/libexec/jenkins/jenkins: http://pkgs.fedoraproject.org/cgit/rpms/jenkins.git/tree/jenkins.init?h=f23

/etc/sysconfig/jenkins: http://pkgs.fedoraproject.org/cgit/rpms/jenkins.git/tree/jenkins-sysconfig?h=f23

/usr/lib/systemd/system/jenkins.service: http://pkgs.fedoraproject.org/cgit/rpms/jenkins.git/tree/jenkins.service?h=f23

There is the option of replacing/adding these files, or building a proper RPM with them for CentOS 7.


That needs to be an array, i.e.
`const char* argV[1] = {"cling"}`


This got some issue


Can you remove these commented out imports?


>Hm... I looked into Google Benchmark doc and they don't say anything about reporting errors, so I think this might be fine... The only thing is that the benchmark will exit with status zero :/ I guess we'll need to bear with that.

Yeah, I searched for it as well and didn't find it.

>Just one thing: could we print the message to stderr?

Sure (:


Cool, thanks for the review @guludo


Shall we declare `foo(int)` in `ChildInterp`? Then we exercise some other parts than calling `foo()`.


Oops, bad search I guess - hit the wrong line.

On Wednesday, January 13, 2016, Vikraman Choudhury <notifications@github.com>
wrote:

> It's not this one, it happens in simple_scavenge_block, and the loop is
> because of scavenge_loop. Anyway, I know how to handle MUT_VAR_s now.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iu-parfunc/ghc/commit/babfa8d07acc21cb55711d8e719ab26fa9c58135#commitcomment-15441160>
> .
>

--
Sent from Gmail Mobile


I am so proud of you, @XeR


Thanks @juliocamarero! I must have forgotten that :)


What are this additional checks inside the `if` preventing from? Isn't `isIdentifierStart` after `let` enough?


Why are the generated conversions removed here?


Fix-allocation-for-feneSolarDelaySeq (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

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


serait il pertinent d'utiliser plutôt cette lib (signalée récemment par Thomas P.) pour gérer les vérif de syntaxe de doi ?
https://github.com/regexps/doi-regex


Could we call this something more specific (both the class and filename), like CollectionAsserts?  Then the method names could be simplified to just Equal, EqualUnordered, etc.


HI


<?php
  echo "Hello World!";
?>


hello


these lines should be sorted alphabetically if possible


jaslfkj


Thanks for th PR!


You're casting a `guint32` difference to `gint`. I guess for the purpose of doing a deterministic `qsort`, the current one should work okay. From a coding perspective, I'd consider explicit checking of `==` and `<` via ternary operators or if's.


TeamCity pull requests :: netty [Build 1027](https://garage.netty.io/teamcity/viewLog.html?buildId=4736&buildTypeId=pulls_netty&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 5896, ignored: 22) Build time: 03:46:41


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


removed the file from .gitignore


added


Great, thanks


This has effect on the class. Move it to __init__() maybe.

Please also check the other classes you added for this.


In order to enable this behavior, the distributed evaluators need to be updated to handle the logic of mixed sparsity types. The binary evaluator should be relatively strait forward, but Summa is going to be much more involved.

Also, the tile interface need to be expanded to produce a dense tile of zeros, if you want to support casting from a sparse to a dense result.


Can you add your new option to the explanation comment?


It was just a made-up example, with `(…)` hiding some specific logic which would make the first call to `.order` necessary. For instance, you may want to retrieve the last five `Posts` by creation date, but want to display them from longest to shortest (custom order criterium). In that case, because the subset of posts in tiny (only 5 records), reordering like that wouldn't affect the overall performances at all.


shouldn't we check first that the box is there?


you already import numpy. Remove this line.


TeamCity Civic Assembly :: Build backend + frontend [Build 164](http://54.229.176.143/tc/viewLog.html?buildId=449&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


前の行と合わせて単に next_actions = choose_actions(responses) でいいと思います。


same thing


I’m going to look at this today but I disagree on the frequency. I saw this multiple times per chapter so we’re talking about hundreds of instances across four volumes. Is that going to distort our notion of Montalvo’s clauses, or is there going to be a way to recover that?

S

—
Stacey Triplette
Assistant Professor of Spanish and French
Humanities Division
University of Pittsburgh at Greensburg
Faculty Office Building 200
150 Finoli Drive
Greensburg, PA 15601

On Jan 14, 2016, at 8:20 AM, Elisa Beshero-Bondar <notifications@github.com<mailto:notifications@github.com>> wrote:

@HelenaSabel<https://github.com/HelenaSabel> Thank you--this is perfect to help explain!

—
Reply to this email directly or view it on GitHub<https://github.com/ebeshero/Amadis-in-Translation/commit/32a406b2b8de0e5575e3d4e3c0851693c45cb564#commitcomment-15449481>.


nitpick: Move one line above (keep the "from gi" lines in their own group)


Doesn't this have a similar problem because byte size is not devisible by 10?

Good catch though.  Thanks!


This doesn't look like a good idea. Despite wiki may say default is 0, the code may require default of -1 to see if user entered the field or not. Let me see...


It worked at the time of the change, but it also wasn't a completely fresh get.  Try rolling back to the previous entry.  There isn't anything in the updated version that I strictly rely upon.


Ah, I see.  Thanks.


$index will never be an integer here as it is explicitly assigned a string two lines up


Morning @Gary15, no qualms on doing that on my end. Want to put up a pull request with that change?


Do you think something similar:
```
if (handle_a == handle_b)
{
  return 0;
}
else
{
  return (handle_a < handle_b) ? -1 : 1;
}
```


StringComparer implements both of these interfaces... why are these special comparers needed rather than just using StringComparer.OrdinalIgnoreCase and StringComparer.Ordinal in the tests directly?