Utilize the power of JUnit! You can use `assertEquals` with `List`s as well! You don't need a new boolean variable or `for` loops.


actually, why $y + 1? wouldn't that be the block above?


Thanks :+1:  :smile:


I think these were still here to not break old data, i.e. if you have a long build-history with runs of the jacoco-plugin, it might fail to load older builds if these are removed and thus fail to show the history coverage trends.

So if we remove this, we also need to tell people in the changelog that older builds may be broken (I did not look how long these are set to @deprecated)


ditto


This may be used for some of the generated HTML files, did you check that they are all still working fine?


Not sure whether I understand your comment correctly. This is supposed to be the list of flags that can be applied to a regular expression. I think ideally we would have a type for regexp which could be applied to all cases where regexps are used.


This should have been two commits, I'm not sure what happened. Missing commit comment should have been "added ethers.c for systems without net/ethernet.h or netinet/ether.h".


dito, may be used from the HTML/generation side


Ah. That's it. No, that's fine. But I'd make that more explicit by renaming it to `DevMode`.
And please add a comment.

Sent from my iPhone

> On Jul 11, 2016, at 3:55 PM, Christian Felder <notifications@github.com> wrote:
>
> The Dockerfile must decide between:
> 1. running npm start for development
> 2. creating a static build and serving with nginx
>
> So, no it's not to test the setup. We can discuss if we want to have the ability to run a dev server, but I think what VSHN introduced makes sense.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


e363219c1ca805f962760e82a4d7ed5de7c287bd
參考replay.cpp改成在底層讀取檔案的時候分成win32和其他這兩種情形
我的想法是MBCS還是盡可能避免


통신사 WI-FI를 못쓰는 문제가 있군요......
자세한 설명 감사합니다


cool, thanks! We did freeze it, don't worry. I just stumbled upon it while updating.


As said above, removal of @Deprecated stuff might cause older builds to fail.
Chaning public to private might cause HTML generation to fail, you need to test thoroughly to ensure it does not cause issues, the unit-tests cannot verify this as they do not generate the HTML currently (might be a good idea to add some integraiton-tests to do this in the future)


미리 정의 된 테마를 사용하려면
Theme.AppCompat.Light.NoActionBar 를 부모 스타일로 잡아 Title을 없앨 수 있습니다.


Hey, do you guys know a fix for particles which you set invisible appearing pure black?


no problem... if you wish I can take a look into it probably tomorrow. Planning to release 0.5 soon so I will just revert it if it is not ready until then.


Did not see anything breaking on a quick look, however I probably do not know Jenkins plugin functionality well enough myself to do a proper review here...


Did you make this work in the end? I am also trying to get a setup going with two separate containers, one that runs Nominatim and one that hosts the data.


Oops! Invalid user. Effects of another console...


@Miigon QAQ


Here are no flags - to my understanding those are still missing here. It's more about the design and nesting of tags. Here we have one element which can occurr several times with different meanings directly under the main tag. In taskxml we always have a tag which collects all nested elements (e.g. files with nested files)


卧槽


I would suggest using `type` instead over `which`

    if ! type hg > /dev/null; then
      return
    fi

Explanation over at http://stackoverflow.com/a/7522726/1039350


什么情况


卧槽这是卧槽


I have a refactoring at hand where I renamed it to `SKIP_BUILD` and changed the default command of the container to `nginx`. And the docker compose, which we only use for dev sets the environment variable and runs `npm start`.

I don't like that the default behavior is dev / no build.
What do you thin about the `SKIP_BUILD` naming I have ready? Better? Or still want `DEV_MODE`? I'd prefer explaining what happens instead of your more abstract suggestion.


i'll keep you updated if i got REAL types working


Cuando refrescamos la ruta de perfil de usuario no la encuentra, porque esta ruta cuando entramos directamente o recargamos lo que hace es buscarla primeramente en el servidor y en el servidor no esta definida, si la definimos utilizando pagejs pero no del lado del servidor.


卧槽这是什么情况！


啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊无视这个commit啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊啊！！！！！！！！！！！！！！！！！！！！！！！！！！！


TeamCity OTA+ :: rvi_sota_server :: ci [Build 444](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7579&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


looks like, that I can merge this?!


I was thinking about pipes that have binary data passing through them when I wrote that `decode` bit. Actually I am not sure what type of data should be passing through a pipe, string or bytes?

For example, a command like `cat some_file | grep pattern`. What should `cat` push into the pipe if `some_file` is a binary file? I thought it should be `bytes` and then the `grep` command would decode the bytes into string again for pattern match?

You are right that these code are very suspicious. I am open to suggestions on how the string/bytes should be used across I/O boundaries.


Are there colors the same as Mischh is using for his raid frames?
Because I really like those :)

Gonna try em out!


Looks good to me


Or `$";password='{Constants.Password}'"`


Yes, ok. This file isn't responsible for running the dev server. So `SKIP_BUILD` is better.


@pp1336 Please don't push directly to master without PR/review! I will back this out in a minute.


But I'd still add a comment.


Actually I think Checkstyle is its own testtype at the moment. I can't remember whether we were planning to replace this by jartest. At Ostfalia we are using jartest for the SetlX language which is implemented via a jar. The idea for jartest is that one doesn't need lots of test-types which are basically all just a configuration file and a jar. Maybe we should declare it as an external resource. But I still think as long as the jar is available on the web and clearly identified by name and version, it is ok. JUnit is also only declared by name and version.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 445](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7581&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Noted! Thanks for the feedback


Yeah I see, but just using a jar an not providing it hinders exchange.


Debug Symbols created by CONFIG_DEBUG_INFO will be stripped out by final objdump.
Also some debug configurations disabled by this commit are triggered only when kernel panic.
Disable those features could give you a cleaner last_kmsg but won't bring lots of benefits to performance.

BTW: You should notice some noticeable power and thermal efficiency degradations caused by your previous commits.


FixDiffuseSolarWhenTransmitSch (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2229 of 2229 tests passed, 0 test warnings)

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


Updated the project with proper naming conventions.


Thanks for the note @l-hedgehog! We had this code somewhere else before as well, I'll check it out.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.745.b13ce9bad680df0e794c01e6489a3be1ebe33dc9](http://build.palaso.org/viewLog.html?buildId=74634&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3490, ignored: 126 Build time: 00:06:36


it's an array of domain host patterns.. with #606 it could make sense to rename


# 233


TeamCity OTA+ :: rvi_sota_server :: ci [Build 445](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7581&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Compilation error: Scala compiler (new) Build time: 00:03:32


厲害哦.gif


The left version is the correct version. For all CMake flags of IGTLink m_Content is correctly initialized. All messages should use this form. (left file)


@afiebig Good catch. Even better if you add a test case that would fail under the previous implementation and passes with the new, correct one.


I did push it last week, but for some reason it didn't apply.

I just tried restarting all dynos on Heroku and that didn't seem to have an effect either.

<img width="344" alt="screen shot 2016-07-11 at 09 52 28" src="https://cloud.githubusercontent.com/assets/1231749/16735040/38fd3fa4-474d-11e6-98b1-005986338916.png">


![lihaio](https://cloud.githubusercontent.com/assets/7542810/16735123/88509b2e-47ba-11e6-8eec-4bac68e1a372.png)
這個


@mintsoft Hehe, I'm so lazy ;)

Have I even used this...?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 446](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7585&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


I suggest defining [`msapplication-TileColor`](https://msdn.microsoft.com/en-us/library/dn255024(v=vs.85).aspx#msapplication-TileColor) and / or [`msapplication-navbutton-color`](https://msdn.microsoft.com/de-de/library/gg491732(v=vs.85).aspx#msapplication-navbutton-color) for Internet Explorer / Edge as well.


I suppose I could just `return entry` instead of using `entries[index]`


The previous code seems to be missing naming conventions. Followed them


Hi,
here an error, the code:
`$prev = count($versions) - 2;`
has to be:
`$prev = count($versions) - 1;`


TeamCity OTA+ :: rvi_sota_server :: ci [Build 444](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7579&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:10:58


哈哈哈哈


It was a count here, though I updated it in d7494ac to be more predictable.


@jesec I agree, may be there are some downgrate after undo Samsung cpufreq modifications.


looks good, thanks


hey hey! what an annoying API change.....


为啥啊QAQ


~不是挺好的吗2333


Just wondering why not pass all these keyword args directly?

However it's not a reason to stop a merge.


you already have `leftNearest` in the scope, no? It is declared at line 61


Apparently it is unnecessary, since you commented it out and then submitted it as working.


有的字体把这个字符显示在行顶端，所以很难辨析


TeamCity OTA+ :: rvi_sota_server :: ci [Build 446](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7585&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:52


@larsbrinkhoff Ahh, thank you, I didn't even look at the ANS FORTH dictionary for some reason when I did that.


Gradual infill sounds interesting, what is it? :open_mouth:


Overfits the training data. Validation accuracy is terrible. Needs regularization and learning rate decay.


This code is indented as if it is a part of `if` at line 72; actually it is not. Please indent code appropriately.


你这样让CMake情何以堪？


translations were only in the /interface/billing/edih_view.php.  Only the initial web page form labels really.  No translations in the /library/edihistory/*.php scripts.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 447](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7587&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


而且，不要一味追求大小写和文字看起来有多装b，NEWorldServer输入至少要按15键


@jdgonzales09 You might want to change you icloud pwd - if this is real.


Man this is fantastic!


Also, it's quite weird to mix names such as **bound** and **index** in Swift 3.

Having to use the **Swift** namespace on `Swift.min` and `Swift.max` methods, too.

Are you updating your proposal to Swift 3?


ocgcore应该是个可以独立输出为dll使用的库，不应该使用`#include "../gframe/bufferio.h"`
有什么理由非要避免WideCharToMultiByte


Yes, they are the same.


This code will be easier to read if you assign `var rightCandidate = allBrackets.right[key], leftCandidate = allBrackets.left[key]`


Thank you for reminding me.  I fixed this in a branch, but forgot to also fix it in the main release branch.


Hello i want join ur team @pdsouza im android developer i can support more devices with maru os and update source


@legoboy0215  Do u remember me XD


It kinda does what is says on the tin. BagelOrb can probably explain this better than me, but it gives you the option to decrease the amount of infill (halve it every time) in a certain number of steps where each step is a certain mm high. It decreases the infill the further it gets away from the top layer.


remove commented out code


@Daniel15 any tips for `Lists of Items` when using functional components?


@markine : Can we make this an altschool-specific fork to track there? Otherwise don't see this in our list of repos and have to hunt it down in your account.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 447](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7587&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests failed: 3 (3 new), passed: 176, ignored: 3 Build time: 00:05:12
### Failed tests
```
org.genivi.sota.resolver.test.ResolveResourceWordSpec.Resolve resource should support filtering by installed packages on VIN: <no details avaliable>

org.genivi.sota.resolver.test.ResolveResourceWordSpec.Resolve resource should support filtering by hardware components on VIN: <no details avaliable>

org.genivi.sota.resolver.test.ResolveResourceWordSpec.Resolve resource should return a string that the core server can parse: <no details avaliable>

```


FixDiffuseSolarWhenTransmitSch (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2263 of 2263 tests passed, 0 test warnings)

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


test commit


FixDiffuseSolarWhenTransmitSch (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


@rmosolgo Thanks for the answer :)


i think it is okay to use `allBrackets.left.length` instead of this constant -- just to decrease the number of places which should be updated when new bracket type appears


remove debug output


I guess this is for old versions still using clirr since we now use revapi. Note that revapi doesn't fail the build if no previous version exist so we shouldn't turn it off by default.


This commit mis-attributed to a different user.

Had a fake email address in my git config file. Apparently I wasn't the only one to ever have this idea. :)


This actually breaks backwards compatibility as the calling function will always pass in two arguments.


Стилизируй селекты через этот плагин http://dimox.github.io/jQueryFormStyler/demo/#select , не делай их списками


It SHOULD make it easier to add new mobs. I guess we'll see :)

http://fossils.enjin.com/
There's a donate button on there, I'm not sure if it still works though. I can't seem to access it without an account. The money goes to maintaining the server, we don't personally get any of it.


not sure if it makes any sense in our context, but still: if we can return msmatches then it perfectly makes sense  to return detected mismatch when cursor in the very beginning of the document


Exactly that. You'll see ;)


This pom are based on LTS.

Clirr does not fail anything either but it try to download and compare to something that does not exist which takes time. I guess it's the same with revapi.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 448](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7589&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


hmm in that case I'm not sure I agree about having it off by default. If it's off you can be sure that the majority of projects will not turn it on, thus removing an interesting check that we want to have as much as possible by default.


please test hipchat room notifications


well done!


Fixed via 902f06b

thanks @gsaponaro


@jesec thanks for your advice, I am trying to backport schedutil to improve power consumption...


can it be replaced with a single `console.log(pos)` ?


the problem here is saving the password clears the token, which breaks docker auth. because that token is required to connect to the CDK docker registry


@DevriesL Instead of 2b9e3428967ccf48dc42264f3ddb0245454ebd5c why do you not add a seperate governor from stock CAF 4.4 kernel (since samsungs modified version of interactive is based on it)


Why is this necessary? As far as I've seen, you are only using iteration and array access from these lists, both of which are defined for `ClipperLib::Paths` as well.


NTFS的檔名編碼是用UTF-16/UCS-2
以這點來說，replay,cpp用_wfopen()開啟的作法應該比較恰當


This does actually break the container. When I start a fresh installation with `make up` followed by `make create typo3`, it is not possible to enter the /app folder anymore (`make bash` will throw an error). This is because the folder has a different inode than before.

My suggestion is to delete only contents but not the app/ folder itself.

See #138.


It is almost impossible to do that with HMP in kernel.
You'd better replace HMP with EAS first.
(That's what I am working on)


Having it run does not change anything except that the build is slower.

There is only two things that would make people set it:
* if the build failed
* if it's documented in http://contrib.xwiki.org


TeamCity OTA+ :: rvi_sota_server :: ci [Build 448](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7589&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:05:12


Oops, sorry for this :)


first commit


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 174](http://teamcity.hq.judo/viewLog.html?buildId=45488&buildTypeId=Mobile_IOS_JudoKitDevelop) is now running


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 174](http://teamcity.hq.judo/viewLog.html?buildId=45488&buildTypeId=Mobile_IOS_JudoKitDevelop) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:03


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 175](http://teamcity.hq.judo/viewLog.html?buildId=45489&buildTypeId=Mobile_IOS_JudoKitDevelop) is now running


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 175](http://teamcity.hq.judo/viewLog.html?buildId=45489&buildTypeId=Mobile_IOS_JudoKitDevelop) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:02


I agree with you, but we may find a way to avoid those calls and improve that code. e.g. `clamped(to:)` could be used there.
For sure I am updating it. :-)


来自我的华为手机-------- 原始邮件 --------主题：Re: [facebook/react-native] Implemented automatic IP detection for iOS (8c29a52)发件人：Nathan Azaria 收件人：facebook/react-native 抄送：Ah, I think I get it.
setJsLocation uses the NSUserDefaults API, which means that unless it's cleared explicitly, the jsLocation will persist throughout different versions of the app inside the device.

So even though you removed the setJsLocation line, the location that has already been set will still be set.
To clear the actual jsLocation and use the IP detection, you can either call
[[RCTBundleURLProvider sharedSettings] setJsLocation:nil] or
[[RCTBundleURLProvider sharedSettings] resetToDefaults].

This might be something that we'll need to change to prevent confusion.

So what happened is that:
1. Automatic IP detection without NSAllowArbitraryLoads doesn't work
2. Automatic IP detection with NSAllowArbitraryLoads doesn't work
3. setJsLocation without NSAllowArbitraryLoads doesn't work
4. setJsLocation with NSAllowArbitraryLoads works

The reason for 3 and 4 is clear - it's because numerical IP address ATS exceptions doesn't work, so let's focus on 1 and 2. Is your device connected to the internet when you were testing, or was it just a local network?

—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.


- Database: ensure `replicate()` can also skip timestamps
  - [See code change from 4.2](https://github.com/laravel/framework/commit/79e0fef538b0fc16441420c118c996c78938b6cd#diff-d8e24a5815a26f7211e66db787be647f)
  - [See code change from 4.2](https://github.com/laravel/framework/commit/b4073331c49c74bf0df0f829e5338e280ec53955#diff-d8e24a5815a26f7211e66db787be647f)


"Having it run does not change anything except that the build is slower."

I don't understand. It makes it execute clirr/revapi! :) (vs not executing it)


7/11/16 @ 11:19
1163297005
PLEASEEEEEE


Hey man its because we forked the node-etcd client so I just wanted to be able to pass in our custom client.


Single Sign-Onかぁ、夢が広がるなぁ。


Validating uniqueness of email should probably happen on the Cyclist and Mechanic models to allow Mechanics to also have a Cyclist account.


asafsd


True, I'm not sure when they use it. Maybe it has something to do with private tabs or guest users or something...
I would need to study the code to understand this more.


因为只有残局会有这个问题，所以给single mode另外设置一个script reader


Note for future -- maybe let's not define associations (has many bookings) to models that don't exist yet. They can get added when the other model has been added.


Of course. This was need for debug


conservation-tests (rraustad) - x86_64-MacOS-10.9-clang: OK (2219 of 2222 tests passed, 10 test warnings)

 * 10 tests had: ERR diffs.
 * 3 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 3 tests had: Table big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.86%25-yellow.svg)


Typo! "presentce"


FixDiffuseSolarWhenTransmitSch (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1672 of 1672 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-71.01%25-green.svg)


@fchollet AtrousConv2D or AtrousConvolution2D?

I know it's a minor point and call me pedantic but keras will be stuck with this name once it's released. I like to make names consistent and easily guessable.


nit: add a space after `if`


Descartados cambios en formulario e imagenes


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 176](http://teamcity.hq.judo/viewLog.html?buildId=45490&buildTypeId=Mobile_IOS_JudoKitDevelop) is now running


@giuge qua pesca solo dentro /public?


Actually this breaks the docker auth. Even when the connection is based on basic auth. a token should be passed around. Here, it's always wiped out once the auth. happens


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 176](http://teamcity.hq.judo/viewLog.html?buildId=45490&buildTypeId=Mobile_IOS_JudoKitDevelop) outcome was **SUCCESS**
Summary: Tests passed: 65 Build time: 00:01:20


@jesec Maybe I need another way to prevent DVFS limit cpufreq, I am wondering this method may disable thermal engine in userspace layer...


I didn't find anything wrong with this other than the minor issues I noted.


Should these `any-wrap/c` changes, with the warnings for `#:opaque` predicates, be in the release? Or should the release always generate a warning, never an error, whether it's from `#:opaque` or not?


my bad, it's commit https://github.com/jbosstools/jbosstools-openshift/commit/67bcf6d3d2c70d02744e45e6875bc54a4d9bba1d#diff-63f0fe66ae00113b07727df6263af68cR306 that actually breaks everything


Am I the only one that gets kinda scared of commits this big? This makes bisecting or finding out where something went wrong a whole lot harder.


通过递归把字典或者数组里的NULL对象换为空字符串


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 177](http://teamcity.hq.judo/viewLog.html?buildId=45491&buildTypeId=Mobile_IOS_JudoKitDevelop) is now running


Would love to be able to move Visible Prefs under  a "Variables" page under "Content" so clients don't have to wade through a bunch of regular prefs.

great work BTW...


@viktorklang I feel like I'm being a bit dense, but could you elaborate on why `A => Option[() => B]` is necessary? I don't see how this is correct in modeling the current `PartialFunction` interface if `applyOrElse` is the preferred mechanism for evaluating a PF. From the perspective of a user of an API that takes a `PartialFunction`, it seems they have to not only concern themselves with the performance conundrum that I mentioned at the outset, but they now have to understand whether the effect is invoked via `isDefinedAt + apply` or  `applyOrElse`.

Also, I don't understand your comment:

> it will needlessly perform the work of the function only to test the validity of the parameter.

This seems wrong - it will perform just as much work as isDefinedAt does to test the validity of the parameter, at which point it will either return None or then perform the additional work necessary to produce the value to be contained in the Some.


This isn't a lab, it's a reading.


> Have I missed something?

It does not change anything if you did not configured it, it will fail to download the previous version and do nothing but it won't fail the build.


What do you guys think about letting price be a float (round to two decimal places if needed)? Probably won't see much of it but it might be strange if someone tried to type in something like "12.50" and it got denied.


This commit may could help you.
https://github.com/jesec/android_kernel_samsung_universal8890/commit/22b32635c46d0808d11248cdc6cc2b8484197713
I will continue working on EAS tomorrow.
(Have to sleep now)


No need to indent this `<title>` tag so far. Since it's a child of `<head>,` one tab is fine.


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 177](http://teamcity.hq.judo/viewLog.html?buildId=45491&buildTypeId=Mobile_IOS_JudoKitDevelop) outcome was **SUCCESS**
Summary: Tests passed: 65 Build time: 00:01:20


You know I can't do that without using `Invariant`.

Also, this totally needs connection string escaping--you didn't catch that. :smile:


I'm pretty sure this kind of string comparison starts breaking after double digits.  `RUBY_REVISION` should be used for this.

```
[flexirest]$ ruby -e 'puts RUBY_VERSION; puts RUBY_REVISION'
2.3.1
54768
```
```
[flexirest]$ ruby -e '(0..20).each{|x| x=%{#{x.to_s}.0}; puts %{"#{x}" < "2.0"? #{x < "2.0"}}}'
"0.0" < "2.0"? true
"1.0" < "2.0"? true
"2.0" < "2.0"? false
"3.0" < "2.0"? false
"4.0" < "2.0"? false
"5.0" < "2.0"? false
"6.0" < "2.0"? false
"7.0" < "2.0"? false
"8.0" < "2.0"? false
"9.0" < "2.0"? false
"10.0" < "2.0"? true
"11.0" < "2.0"? true
"12.0" < "2.0"? true
"13.0" < "2.0"? true
"14.0" < "2.0"? true
"15.0" < "2.0"? true
"16.0" < "2.0"? true
"17.0" < "2.0"? true
"18.0" < "2.0"? true
"19.0" < "2.0"? true
"20.0" < "2.0"? false
```


@jesec Thanks :D have a good night


Perfect. Thank you, Dongjun!


hmm I must be missing something... Let me rephrase my question: why don't we set up clirr/revapi to execute AND fail the build (otherwise it's not useful at all) in the parent POM?


Travis CI finally builds again: https://travis-ci.org/modelica/Modelica_DeviceDrivers/builds/143916990


I'm leaning towards the "only warnings" approach for the current release.


not sure why, but when using negotiated rates you need to provide a state. grab the latest changes, add a state (e.g. `var destination = new Address("Rockville", "MD", "20852", "US");`) and try again.

BTW, use something like fiddler to look at the actual request and response and it'll typically tell you the issue.


@RWOverdijk endless loop ??


Also, I'm pretty sure that the "build" (on RubyGems' servers?) reads this file only once, and that this file is _not_ executed on the client.  Not sure how to test it, though...


@nuttycom, one of course could implement `A => Option[B]` like you described, but that's not mandated by the type. One could easily implement it the wrong way. OTOH, using `A => Option[() => B]` clearly defines the rules for users and implementers. Ain't that what we want types to be?


So should I make a new pull request to the release branch, to have this commit plus another one that makes the normal `any-wrap/c` also use `on-opaque-display-warning`?


Good job.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.245.6a4bf3f2b507bf6e9ad503fcbbe180ede7e59162](http://build.palaso.org/viewLog.html?buildId=74643&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 949, ignored: 50 Build time: 00:37:06


I think `cmLoc` is not defined here.


Yes, that would be helpful.


Thanks for working.


I have no idea if you can make clirr fail when the previous version is missing. Also most of the contrib extensions don't have any public API so forcing to configure clirr is not very nice...


I can confirm that after #2027 my script is working.


TaskCluster: @cgsheeh does not have permission to trigger tasks.


LGTM +1.


I think the behavior of countDefined should rather be:

metric1, [ math.NaN(), 1, math.NaN() ]
metric2, [ math.NaN(), 1, 1 ]

countDefined(metric*) -> [ 0, 2, 1 ]


okay thanks :)


Esto es una prueba


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 178](http://teamcity.hq.judo/viewLog.html?buildId=45492&buildTypeId=Mobile_IOS_JudoKitDevelop) is now running


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 178](http://teamcity.hq.judo/viewLog.html?buildId=45492&buildTypeId=Mobile_IOS_JudoKitDevelop) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:03


It seems checking `has_tracemalloc` isn't sufficient. The user could've chosen a different source (e.g. `psutil`) via API.


You have to create your own context. It's as simple as `llvm::LLVMContext()` :)
(for other ideas, look at the implementation of `getGlobalContext()` in llvmhelpers.cpp


not the best idea to put a puts statement in the initializer


By adding methods like these, you implement collection interfaces:
https://docs.python.org/3/library/collections.abc.html

If your collection implements certain interfaces, tell the world by making it to inherit from the appropriate abstract base class(es).


Good use of a `<div>` tag here!

Since you're applying this only to the 'header' of your resume though, in the future, consider using a `<header>` tag.

Your resume looks great! There are a few suggestions I'd make on how to structure your HTML document so that your code's a little more readable and organized. At the bottom of this page, I included a version of your code with more html5 tags, and a more segmented layout. This kind of setup will help your code stay organized moving forward, and, when we get to CSS, will help you style your sections faster.

Come tap me on the shoulder before class and we can talk about why I suggested these changes.


maybe make a better name, good practice for when you're working on an app with others


watch spacing.


addressed.


thanks :)


Used dummy path 'path'


You can have multiple `@compute` modules, so this won't work well. I would make one DComputeCodeGenManager per target. (perhaps subclass it for each target type). Then in main.cpp you can loop over all targets.


[R/calc_nltt_stats_from_file.R:19:1:](https://github.com/richelbilderbeek/Cer2016/blob/fc0c28a5776b431b80b962eac5e46b6af8da1d03/R/calc_nltt_stats_from_file.R#L19) *style:* **lines should not be more than 80 characters.**
```r
  nppa <- Cer2016::extract_nppa(file) # number of number of posteriors per alignment
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/calc_nltt_stats_from_files.R:33:1:](https://github.com/richelbilderbeek/Cer2016/blob/fc0c28a5776b431b80b962eac5e46b6af8da1d03/R/calc_nltt_stats_from_files.R#L33) *style:* **lines should not be more than 80 characters.**
```r
    napst <- Cer2016::extract_napst(file) # number of alignments per species tree
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/calc_nltt_stats_from_files.R:34:1:](https://github.com/richelbilderbeek/Cer2016/blob/fc0c28a5776b431b80b962eac5e46b6af8da1d03/R/calc_nltt_stats_from_files.R#L34) *style:* **lines should not be more than 80 characters.**
```r
    nppa <- Cer2016::extract_nppa(file) # number of number of posteriors per alignment
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.746.b13ce9bad680df0e794c01e6489a3be1ebe33dc9](http://build.palaso.org/viewLog.html?buildId=74669&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3490, ignored: 126 Build time: 00:07:05


Was already not an issue any more.


"I have no idea if you can make clirr fail when the previous version is missing". This works for revapi. You make it fail on errors and it doesn't fail if the previous version is missing.

Just got an idea: with recent maven versions we can configure plugin properties in a custom lifecycle. Thus we should be able to skip clirr/revapi automatically when the packaging is XAR.

I'll explore this for platform.


move to uda.h/uda.cpp


ri parameter never used? (I can provide a patch if needed.)


This code doesn't exist any more and was indeed wrong.


plant-autosizing-gaps (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2208 of 2222 tests passed, 541 test warnings)

 * 553 tests had: AUD diffs.
 * 12 tests had: Table big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.37%25-yellow.svg)


Save uses asObject, which then also changes the state of the entity, resulting in different behavior. I haven't experienced an infinite loop with this yet, personally.


I believe a better option would be to provide the expected argument to `-i` and then remove the backup file.

Accepting to have `-e` interpreted in two fully different ways is really awkward.


conservation-tests (rraustad) - 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)


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 179](http://teamcity.hq.judo/viewLog.html?buildId=45493&buildTypeId=Mobile_IOS_JudoKitDevelop) is now running


I don't think so. Maybe you overlooked that infill_line_distance_here is also used in the `infill_comp` below?


I just realized that this scenario (installing Xposed on a ROM with Xposed included, then uninstalling to return to the pre-installed version) couldn't have worked anyway, as xposed.prop and XposedBridge.jar are removed unconditionally. I think I did this to avoid cluttering /system and /system/framework, but effectively it means that the pre-installed version would be broken afterwards. It also means that flashing only the uninstaller on such a ROM would remove exactly those two files. Xposed couldn't work afterwards, so it kind of works, but I would compare it to removing the launcher icon for an app, which only hides it as well.

I'm really not sure about this, but it has to be consistent. Either I create no_orig files for all files which didn't exist before and also implement a migration path for existing installations, or I get rid of it completely.


TeamCity Mobile :: iOS :: JudoKit (develop) [Build 179](http://teamcity.hq.judo/viewLog.html?buildId=45493&buildTypeId=Mobile_IOS_JudoKitDevelop) outcome was **SUCCESS**
Summary: Tests passed: 65 Build time: 00:01:06


1.7rc1 is out there, btw x)


One more console.log here


Actually this is OK on current beta Rust, just not on old ones.  See https://travis-ci.org/sourcefrog/conserve/builds/143770650.


@hseeberger interesting... of course, one could still implement `A => Option[() => B]` wrongly, so in this case you're purely adding overhead without actually increasing safety, as far as I can tell. This runs afoul of the same things that `Function1 <: PartialFunction` would - the type can add no semantic restrictions that the implementer is forced to adhere to, and so it's spurious complexity.


> "I have no idea if you can make clirr fail when the previous version is missing". This works for revapi. You make it fail on errors and it doesn't fail if the previous version is missing.

I really don't understand your point here, you are saying we should fail the build if the previous version is missing and what is good about revapi is that it does not fail when the previous version is failing.


this should return `static_cast<AtomBrowserContext*>(context)->original_context()`


:tada:


### Metagenomics Tools

[QIIME](http://qiime.org/install/virtual_box.html)

[Mothur](http://www.mothur.org/)

[MetAMOS](https://github.com/marbl/metAMOS)

[Fantom](http://www.sysbio.se/Fantom/)

[IMG/M](https://img.jgi.doe.gov/cgi-bin/m/main.cgi)

[iPlant] (https://pods.iplantcollaborative.org/wiki/display/WiNGS/Scalable+metagenomic+analysis+using+iPlant)

[Kraken](https://ccb.jhu.edu/software/kraken/)

[MEGAN6](http://ab.inf.uni-tuebingen.de/software/megan6/)

[Meta-IDBA](http://i.cs.hku.hk/~alse/hkubrg/projects/metaidba/)

[MetaVelvet](http://metavelvet.dna.bio.keio.ac.jp/)

[MG-RAST](https://metagenomics.anl.gov/)

Xander: https://github.com/rdpstaff/Xander_assembler

[AmphoraNet](http://pitgroup.org/amphoranet/)

[ClaMS](http://clams.jgi-psf.org/)

[Clark](http://clark.cs.ucr.edu/)

[CoMet-Universe](http://comet2.gobics.de/)

[EBI Metagenomics](https://www.ebi.ac.uk/metagenomics/)

[EstimateS](http://viceroy.eeb.uconn.edu/EstimateS/)

[E-vident](https://github.com/biocore/Evident/)

[FLASh](http://ccb.jhu.edu/software/FLASH/)

[Genometa](http://genomics1.mh-hannover.de/genometa/)

[Glimmer-MG](http://www.cbcb.umd.edu/software/glimmer-mg/)

[GOTTCHA](http://lanl-bioinformatics.github.io/GOTTCHA/)

[MetaABC](http://metaabc.iis.sinica.edu.tw/)

[METAGENassist](http://www.metagenassist.ca/METAGENassist/faces/Home.jsp)

[metagenomeSeq](http://www.cbcb.umd.edu/software/metagenomeSeq)

[metaMicrobesOnline](http://meta.microbesonline.org/)

[MetaPhlAn](http://huttenhower.sph.harvard.edu/metaphlan)

[MetaPhyler](http://metaphyler.cbcb.umd.edu/)

[Metarep](http://jcvi.org/metarep/)

[METAVIR](http://metavir-meb.univ-bpclermont.fr/)

[My Taxa](http://enve-omics.ce.gatech.edu/mytaxa/)\
[One Codex](https://www.onecodex.com/)

[Panda](https://github.com/neufeld/pandaseq)

[PEAR](http://www.exelixis-lab.org/pear)

[PhymmlBL](http://www.cbcb.umd.edu/software/phymm/)

[Real Time Metagenomics](http://edwards.sdsu.edu/RTMg/)

[Ribosomal Database Project](http://rdp.cme.msu.edu/misc/about.jsp)

[SMASHCommunity](http://www.bork.embl.de/software/smash/)

[Taxator-tk](https://omictools.com/taxator-tk-tool)

[VIROME](http://virome.dbi.udel.edu/)

[ZORRO](http://journals.plos.org/plosone/article?id=10.1371/journal.pone.0030288)


> I really don't understand your point here, you are saying we should fail the build if the previous version is missing and what is good about revapi is that it does not fail when the previous version is missing.

Quite the opposite. We should NOT fail the build if the previous version is missing (why would we want that?).


@melodyserra students may need more to understand the DOM here. Can you include a diagram or a more detailed breakdown and explain how it's different from HTML


TeamCity OTA+ :: rvi_sota_server :: ci [Build 449](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7590&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


TeamCity OTA+ :: rvi_sota_server :: ci [Build 450](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7591&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


> Quite the opposite. We should NOT fail the build if the previous version is missing (why would we want that?).

I said several times that CLIRR was NOT failing the build and that it was just slowing it down for nothing.


@melodyserra can you include an example of pseudocode here? I'm thinking a small activity might be beneficial too.


Has it ever worked?


Is the `cpointer-predicate-procedure?` commit (https://github.com/racket/racket/commit/8d698389e76d8a53baca59b0be9b5680dc3f2d75) in the release branch? This depends on that.


因為其他模式沒有使用set_script_reader()而是直接使用預設的函數default_script_reader()
這樣有可能變成進入一次單人模式之後讀取函數從此就被替換成單人模式的版本


OK with me


@viktorklang I do like the idea of 'match' being a totality-checking macro but it seems like it adds a corner case - if you defined `pf` elsewhere, how would  `_ match pf`  be desugared, just to `pf` without the totality check? Or would this be a compile-time error if `pf` were anything but a partial function literal?


@nuttycom, well, we can almost always fool the type system. But for me looking at `A => Option[() => B]` shows the intent of separating the "is defined at" check from the actual (possibly effectful) computation very clearly as opposed to `A => Option[B]` which conflates these concerns.


Rename this to "public-private-protected".


Off by default:
- + faster build times for the first version of a module
- - it must be explicitly turned on right after the first version is released, and it is very likely that people will forget to do that, unless it is automated in the release process

On by default:
- - slower build time for the first version of a module
- + will always do its work, deva don't need to remember to activate it

The downside of off by default is worse than just a slightly slower build.


LGTM, although I suggest adding a test that uses the new NoEscape field.


Architecturally, this calculation actually makes more sense on the server side, so that OEMs can easily modify the reporting engine and still get a sensible result here.

Let the parameter-entry in the parameter-xml have an extra boolean attribute "must-validate-on-server". When you calculate the dependencies on the server (ParameterXmlContentHandler class), you can set that flag on the element itself.

This way, if the server side validation rules change (they are pluggable, after all), we simply check a flag set by the server.


Change name just like in artifactId.


@eregon Fixed at 24ad16ba60c928dc6965ccb2abab778fe1fdfc54


Il est meilleur de demandé l'architecture étant pour éviter d'avoir un problème si on renomme les fichiers.


It lift up the lowest frequency actually, might need revert it


No, it isn't.


Test method names should follow 3 part convention (methodName_scenario_expectation).
i.e. `constructor_always_isPrivate()`


This should not be necessary at all.

For strictly validating parameters, the only time code like that would be triggered is if a upstream-parameter has changed and in that case you should get an updated parameter-xml document back. For non strict parameters, this validation rule makes no sense, as a user can make their own selection anyway.

The server is the correct place to handle this.


Sync'd to current master/HEAD (d73d32e) which includes your commit.

No segfault. Thanks.


> + will always do its work, devs don't need to remember to activate it

No it won't, if you don't configure anything you previous version is 7.4 because that's what you get from commons.


I just changed everything to this, but the "core dumped" error happened, so I think maybe it is safe to allocate memory for these constants.


Thanks Sergiu for summarizing it :)

FTR I've checked the latest version of Maven (3.3.9) and I don't think it can do what I wanted (i.e. turn off clirr/revapi for XAR packagings). What can be done is configuring the plugins executed for the various phases, see https://issues.apache.org/jira/browse/MNG-5805. So if we explicitly add the clirr/revapi plugin to the parent pom, we won't be able to exclude it.


Use
```
import java.lang.reflect.Modifier;
```

```
assertTrue(Modifier.isPublic(employee.getModifiers()))
```


closes #94


perhaps make a pull request upstream?


> No it won't, if you don't configure anything you previous version is 7.4 because that's what you get from commons.

Why is that a problem? It's good. And if the dev introduce some code change that makes clirr/revapi fail because it's comparing to 7.4 then they can update the version they compare to. I'd say it's better than not doing any comparison.

What would be nice would be to find some way to not execute clirr/revapi when the packaging is XAR just to win a few second for each build.


what?


O pdf para ser revisado está lá no git. Não resolvi as referências e outras
questões ainda, mas creio que o de mais é a seção de fundamentação estar um
pouco curta.

2016-07-03 21:06 GMT-03:00 Laércio Lima Pilla <notifications@github.com>:

> Como está o texto com a formatação da UFSC? Precisamos de pelo menos umas
> 15 páginas de conteúdo (desconsiderando capa e afins).
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/IgorKinGamer/proj-tcc/commit/df84c820f592a715486dec2dbe5d030ef1560b38#commitcomment-18109069>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AJdbV35L23r99vY5J48y90d3sOmy0Mfcks5qSE5sgaJpZM4JD-hE>
> .
>


> Why is that a problem? It's good. And if the dev introduce some code change that makes clirr/revapi fail because it's comparing to 7.4 then they can update the version they compare to. I'd say it's better than not doing any comparison.

How is that good that your 1.1 extension is compared to 7.4 ?


chac chua vay?


closes #93


gi day?


delete line


Ah, I've been meaning to bring this up. A lot of people really seem worried about this message. :confused:


@adrfer what do you think about this implementation? :-)

```swift
    subscript(clamping range: Range<Index>) -> SubSequence {
        let clamped = range.clamped(to: (startIndex ..< endIndex))
        return self[clamped]
    }
```


TeamCity OTA+ :: rvi_sota_server :: ci [Build 450](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7591&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:10:18


TeamCity OTA+ :: rvi_sota_server :: ci [Build 449](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7590&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:10:29


Juan is no longer a host :-( moved out of the area


Flip to Curiosity Media


ok so the issue is that we can't know the versioning used by an extension. Wow, I must really be tired because I've never understood this in the previous messages.

So Maven can easily find the last released version so clirr/revapi could be made to guess the last version easily. I need to check since it's possible that revapi already does this.


Ouch!


- Database: ensure `replicate()` can also skip timestamps
    - [See code change from 4.2](https://github.com/laravel/framework/commit/79e0fef538b0fc16441420c118c996c78938b6cd#diff-d8e24a5815a26f7211e66db787be647f)
    - [See code change from 4.2 ](https://github.com/laravel/framework/commit/b4073331c49c74bf0df0f829e5338e280ec53955#diff-d8e24a5815a26f7211e66db787be647f)


Hyun, here's a shorter version of your code with a little reformatting. Please let me know if this makes sense. If you have any questions, please come tap me on the shoulder before class :)

[hyun_an.html.zip](https://github.com/amberan77/resume/files/357520/hyun_an.html.zip)

Great job on the homework!
Zack


just checked and it should work fine with revapi, see http://revapi.org/modules/revapi-maven-plugin/check-mojo.html

> If you don't want to compare a different artifact than the one being built, specifying the just the old version is simpler way of specifying the old artifact. The default value is "RELEASE" meaning that the old version is the last released version of the artifact being built.
Default value is: RELEASE.


To use `-i` correctly, we need to know whether we have GNU `sed` or BSD `sed`. The BSD version expects a space after `-i`, while GNU doesn't like the space. So either we leave it the way it is, or we complicate things by detecting what version of `sed` we have, or we do not use `-i`.


Ok will do.

--
Thanks, Serafim Suhenky.

> 11 июля 2016 г., в 19:04, TomasM <notifications@github.com> написал(а):
>
> perhaps make a pull request upstream?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub <https://github.com/serafxxx/Python-ContextIO/commit/5cbd4655a878c24161fcf0c8cad039e10d646872#commitcomment-18197401>, or mute the thread <https://github.com/notifications/unsubscribe/ABOYnUlbciAOWD4ed-OR3Dk2SPdWniZRks5qUml6gaJpZM4JJhU3>.
>


@boite droid-nginx submitted to packagist. Merged this pr and tagged v1.11.0.
OCD patches appreciated 👍


And it works for clirr too, see http://www.mojohaus.org/clirr-maven-plugin/examples/specific-version.html


we will see what this does on performance. It shouldn't matter on windows and AWS, but for Linux and mac it may cause a bit of a slow down.


Above comment is a duplicate. You said the same in a different commit which had the same code, but a different indentation.


Typo


This is an incredibly small point, but there's no need for a line break here. To paraphrase Bob Ross, tidy html is happy html :)


get_catigories
Another typo


So the issue is now unsetting the value in the commons pom. Maybe it's possible with http://stackoverflow.com/questions/33251049/how-to-remove-a-plugin-configuration-defined-in-parent-pom-from-child-pom


*possition* hahahah


Please 3 lines.


👍 Did you add me to this to round robin the reviews?


not needed


I think this was from an on-call thing you got paged on a long time ago, haha


But I'm happy to take the review


packages are usually lowercase :)


def prime?(n)
  if n <= 1
    false
  elsif n == 2
    true
  else
    (2..n/2).none? { |i| n % i == 0}
  end
end


Actually I found a way to put CLIRR back in automatic mode.


Please use _ for the names of members of LabelData


FixDiffuseSolarWhenTransmitSch (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2229 of 2229 tests passed, 0 test warnings)

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


Gotcha. Well, looks simple enough.


#Home page
<!DOCTYPE html>

 <title>Bootstrap Example</title>
 <meta charset="utf-8">
 <meta name="viewport" content="width=device-width, initial-scale=1">
 <link rel="stylesheet" href="css/normalize.css">
 <link rel="stylesheet" href="css/custom01.css">
  <link rel="stylesheet" href="css/hoverpro.css">
 <link rel="stylesheet" href="http://maxcdn.bootstrapcdn.com/bootstrap/3.3.6/css/bootstrap.min.css">

My First Bootstrap Page

This is some text.

Mannaggia a Zio Carlo!!!

        <button type="button" class="navbar-toggle" data-toggle="collapse" data-target="#myNavbar">

        </button>
        WebSite Name

Start

Info

Brands

Awards

Contacts

Polkura Syrah 01

2012 – James Suckling 95

          2012 – Descorchados 94

          2011 – Wine Spectator 91 (Top 100)

          2011 – James Suckling 93

          2010 – Descorchados 93

Polkura Syrah 02

2012 – James Suckling 95

          2012 – Descorchados 94

          2011 – Wine Spectator 91 (Top 100)

          2011 – James Suckling 93

          2010 – Descorchados 93

Polkura Syrah 03

2012 – James Suckling 95

          2012 – Descorchados 94

          2011 – Wine Spectator 91 (Top 100)

          2011 – James Suckling 93

          2010 – Descorchados 93

Polkura Syrah 04

2012 – James Suckling 95

          2012 – Descorchados 94

          2011 – Wine Spectator 91 (Top 100)

          2011 – James Suckling 93

          2010 – Descorchados 93


make it a typed list instead
e.g.
```groovy
List<StaffChild>
```


you can refer to elements that can fit into a Staff by creating a marker interface and extending it, like:
```groovy
class Note extends StaffChild
```


You could take environments from a config file or .env
https://github.com/bkeepers/dotenv


I'm getting this error but I don't know how to fix it.  Where can I get the Julia development headers and install them?


items[0] == ???


NULL should be &scan_cache


`get` is too general in this case


feels a bit slow on my computer with animation on. but no, don't take out. looks great. let's figure out. but i will turn on and off as we go, thanks! same for below


Thank you DanikTheBoss. I will add you to the team.
I will mainly put up an issue saying `[Danik] Request - Do this` if I need help doing something in the files or if I find a bug and I want to see if we can fix it.


make it typed


same


`== 8`, meaning?


how can an `items` be a String :)?


that would be nice indeed :)


那其它模式只能都设置一下了
总之直接使用ocgapi来满足特殊需要应该是比较合理的


this is weird, why wold you need it :/


please extract these dubious parts to separate methods with explanatory names


It can be uncommented out.  I needed to figure out what the menu would look like in it's final iteration, so was commenting out all of that functionality to make sure the menu worked correctly.  I believe the events are also commented out, but if you need them, you can add them back in.  The new functionality needs to be added as soon as possible.


:+1: for `final`


too general


Could you please explain, why Text::Soundex is now mandatory? There are platforms where this needs an extra compiler to install...


* [The details which triggered help requests](https://systeme.lip6.fr/pipermail/cocci/2016-July/003432.html "Checking build dependencies for Menhir") might be a bit different around my use of a tool like "ocamlfind".
* By the way: You can omit the abbreviation "SF" in such commit messages.   ;-)


Woops. Thought I did add it. I'll  push it tonight when I get back home


I think the order is waiting for a server update in v1.0 tiles related to
normalizing National Forests.

_n

On Mon, Jul 11, 2016 at 9:19 AM, Geraldine Sarmiento <
notifications@github.com> wrote:

> feels a bit slow on my computer with animation on. but no, don't take out.
> looks great. let's figure out. but i will turn on and off as we go, thanks!
> same for below
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/tangrams/tron/commit/fb2ecb919ccc483b939492cf24fdaac44d20e819#commitcomment-18197663>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AA0EO_voTGp0OYvfN5X6MfPR0glOk3lnks5qUmz0gaJpZM4JImu5>
> .
>


split this into three lines + asserts.

First fine `aseTree->find(parentId)`, and `Q_ASSERT` that you found it.
second line is `baseTree->find(nodeId)`
last line is if it's there, is it in the parent


The usage of always_comb here is very odd. Why not just assign?


conservation-tests (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1112 of 1112 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-34.63%25-green.svg)


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


@hseeberger I guess this is another example of where `() => ???` and `??? => ()` are lousy for describing effects. I do see where you're coming from; if `() => A` is Scala's version of `IO a` then this kind of makes sense, but it means that you're forcing the assumption that a PF is going to do IO into every PF value.

These green eggs and ham aren't just weird-looking, they're infected with salmonella.


forgot to add node here


This is the reason why your app shown wrong movies when searching. In this block (`if searchActive`), you only set the title and didn't load the suitable poster.


make sure to increment the iterator.


The `;` is unnecessary in Swift.


`changeIt`


`- len(s) % 8` gives you the right padding length


Still not sure how to structure the path to my external .js file. StackOverflow should be able to help me. I read up on [shimming modules](https://webpack.github.io/docs/shimming-modules.html), but I still don't understand it completely.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.747.b13ce9bad680df0e794c01e6489a3be1ebe33dc9](http://build.palaso.org/viewLog.html?buildId=74714&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3490, ignored: 126 Build time: 00:05:47


If you do not have any special thing in the `override` method, you can remove it.


OK, although it works only for whitechars before and after "useful" text. My function solves white chars inside text as well and replaces it with space. Don't we need it this way?


No need for an `hr` here. The `head` section of your HTML is for code that defines things that we won't see in the body of our webpage. Things like our `title` (which defines what appears in our browser tag when our site is open), or metadata.

Right now, in your `head` section, you only currency need to worry about your `<title>` tag.


You can remove `(` and `)` here.


labelA = {};


Very small point, but there's no need for a line break here.


#5768-Example-files-which-show-discontinutity-in-air-or-water-flow-rate-through-fans-and-pumps (mjwitte) - x86_64-MacOS-10.9-clang: OK (2222 of 2222 tests passed, 0 test warnings)

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


Не присваивает значение...


You should not commit node_modules folder, just add the path to .gitignore file


please clarify why this is so (because we deleted label changes)


closes #17


You're wrong!


From the line 124 to 127, you can shorten it like this:
```Swift
  return searchActive ? filteredData.count : movies.count
```


not very creative!


Pluto is a dwarf planet, not a real planet


It is a dwarf planet


Comment about line 3.


Try to remove sample files and dead code, there is a hello world jade file


That's so not True.


Was this a good idea?


This is code review.


But when is someone going to clean up that big spot?


You could follow http://nvie.com/posts/a-successful-git-branching-model/ when working with git


Yes; those doc managers are essentially the same.


closes #4, #5


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.246.6a4bf3f2b507bf6e9ad503fcbbe180ede7e59162](http://build.palaso.org/viewLog.html?buildId=74675&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 949, ignored: 50 Build time: 00:38:02


Fixes #97


Indentation should be 2 or 4 (we use 2) but be consistent, always use 2 or 4 in all the files


Going by memory and the comment I wrote, what I seem to recall is a situation where if I didn't use mStarted, a package that built a path to a destination (like travel) would immediately end when the package was cycled and started again.

However removing both the `!mStarted` and the `mStarted = true;` here and giving some repeating travel and escort packages a test, I didn't have any problems, and it's not immediately clear to me from looking at the code what the exact intention was. This was my first main attempt at the code and although I'm still a programming newbie, I was a super newbie then and doing a lot of stuff by trial and error, so probably something changed that made mStarted unnecessary here, or for that matter it may never have been necessary in the first place.

Anyway I can't 100 percent say it's OK to take it out, but as I said the problem it is supposedly there to address doesn't seem to happen when it is removed, so feel free to remove it.


Only regarding location? I also have Fluencia listed as a Sponsor.


A change might also be a value/type change (Stationary and Value/Type updateFlag should be set, one or both of them). Ignore these.


GetChildAs<T> sounds fine. For components we have simply GetComponent<T>  but that's a bit different operation as it gets the first component of specified type in the node.

IMO static cast variety of this operation isn't necessary (since UI is typically not performance-critical with hundreds/thousands of element manipulations performed per frame) and could even be dangerous.


The race condition (actually it's not) happens after that 1s delay so that doesn't matter.


"dwarf" may be offensive. It's a small planet.


Same here! :smiling_imp:


Q_ASSERT move or deletion


Great job! Your resume looks good. Very good use of <ul> and <li> tags, and your tabbing is perfect. Keep that up!

I rewrote your code with some more specific tags that will help you write clean, maintainable code moving forward. This kind of formatting is especially important when we start to write CSS, as it will help you hone in on certain parts of your code quickly.

Please take a look at this and come tap me on the shoulder before class if you have any questions about these changes. You can also schedule office hours for tomorrow (tuesday) night and we'll go over these changes.


@mage2pro
This seems to have broken the checkout flow.
I now get the below error when clicking on "Place Order":

jquery.js:9666 POST [MY_MAGENTO_INSTANCE_URL]/magento2/rest/default/V1/dfe-checkout-com/75781a5fb308b971180a6e2e4bb23cf3/place-order 500 (Internal Server Error)


LOL. Bootrons. A projectile vomiting action.


you don't like the logging? I thought our users would enjoy seeing that... lol jk thx for cleaning up after me!


Ok thanks!!!! But keep in mind I'm a beginner.


Thanks for this fix. Maybe you should also change the version to 0.6.1?


@doom369 FYI


Update flags here


http://docs.oracle.com/javase/7/docs/platform/serialization/spec/input.html

> Any field of the object that does not appear in the stream is set to its default value. Values that appear in the stream, but not in the object, are discarded.

So this should all work, especially as the fields are not used anyway.


Yes, sure.


So semantically, we would call this variable @questions, as it has many of them.


increment iterator


Notice how you are using @questions here, but @question in your index method. If this is not working, that's why.


http://docs.oracle.com/javase/7/docs/platform/serialization/spec/input.html

> Any field of the object that does not appear in the stream is set to its default value. Values that appear in the stream, but not in the object, are discarded.

So this should all work, especially as the fields are not used anyway.

And I will try to add a test for HTML. In the meantime we can also skip this commit.


Don't think it's necessary to say "click the button below..." I would just say, "Reset your password below"
Thanks!


@mpirio Yep, I'll be pushing a new 0.6.1 release in just a minute :)


@lukehutch

> I tried switching the code to use NIO everywhere a few versions ago (including using ZipFileSystem / FileSystem, TreeWalker and Path), but I found that this incurred a significant overhead

Don't mix up NIO and NIO2. I am talking about NIO (Channel-based IO, which has been rock-stable for a long time). NIO2 have been introduced in Java 7 and was sane in most of it's aspects, but I recall, that ZipFileSystem was largely rewritten during Java 8. No wonder it worked out poorly for you. Either way, ZipFile is not the only (and by far not best) library for Zip extraction.

> Under the hood, since NIO's ZipFileSystem actually uses the "old-style" InputChanne;… then… NIO can't use interruptible reads.

You are wrong. There is no magic within NIO interruption. All channels, including those from `Channels#newChannel`, are inherently interruptible. JDK Thread code simply closes underlying stream/descriptor of every open channel whenever the Thread, using it, gets interrupted (see `AbstractInterruptibleChannel#implCloseChannel`). You can accomplish the same thing without NIO Channels by overriding Thread's interrupt() method, but that would be less flexible (won't work for custom Thread subclasses).

> So, on this point, I don't know what to do other than to continue using InputStreams. Is it a problem that a 16kb read can't be interrupted partway through, when we're talking about a difference in thread shutdown time of maybe a millisecond or two compared to shutting down the thread at the end of the read, once the interrupt status can be manually checked?

Not a big problem, if you check for cancellation frequently (but doing it too frequently is a performance bottleneck in itself). Btw, you should consider using bigger buffer sizes 16kb is too small for modern computers, even if you aim for level 1 CPU cache.

>  a few releases ago, in the first version of multithreaded scanning, I did use ExecutorService with Futures… I found that this incurred a cost of something like 15% greater total scan time relative to manually starting threads

It is not fair to compare performance of existing feature-heavy implementation with one, tailored for your specific purpose. If you were to do your testing with custom ExecutorService, precisely mirroring behavior of your current code, you would get correct results.

Each builtin executor have a lot of redundant parts (tracking it's own state, counting Threads, own BlockingQueue). There is yet another Queue in CompletionService… And you was not even using CompletionServices, Executors or Futures for their stated purposes.

You used *another* thread-safe Queue, effectively wasting the synchronization overhead of Executor Queue. You never returned any Futures to user (you may have as well never created them). Instead of keeping results within individual Threads/Futures until full completion, you eagerly shoved them into (another!) thread-safe Collection (makng the CompletionService redundant)… And now you complain, that using generic ThreadPoolExecutor incurred overhead. Sure, you have duplicated entire infrastructure of both of ExecutorsService and CompletionService within your own code, so using them only made you slower.

Speaking of redundant code, I highly recommend you to read [this article about (false) sharing](http://www.1024cores.net/home/lock-free-algorithms/false-sharing---false). It is rather enlightening in many ways.

TL; DR: don't eagerly store everything in thread-safe Collections just because you can. Either submit tasks to Executor ASAP or wait until you know full scope of work and split it between Threads in advance (thus eliminating the need for input Queue altogether). Don't collect outputs in thread-safe way each time you produce a tiny unit of work.

> Since I'm now adding interruption checks into my code, this is an earnest question: do you really assume that all library writers will add interruption checks to all methods that could be time-consuming (blocking or not), even if the methods provably terminate?

Nobody is forcing you to manually scatter interruption checks after every method call (this is both pointless and slow). But if you already do some things, like using thread-safe Queues (read up about "poison-pill" technique) and calling interruptible methods, it wouldn't hurt to use those facilities correctly, would it?

> Even an in-process debugger must be able to stop long-running code, surely?

Sure. Sometimes VMs just switch to interpretation mode during debugging. Sometimes debuggers embed interruption checks everywhere when JIT-ing code. Sometimes VM developers use nasty tricks, such as [using mprotect to strip permission from thread](https://jandemooij.nl/blog/2014/02/18/using-segfaults-to-interrupt-jit-code/), making the running code controllably segfault.


test


Recreating that bug: When I set everything to dashes, save that alarm, and then edit that alarm to be prorahn 0, it does that misfiring thing.


wew


thanks :)


Ah, gotcha. Well, I released v1.2 over the weekend that does nested namespaces w/ the separator, etc. Check it out and see if that doesn't clean things up for you.


It could be a value/type update. Make sure to ignore these.


In the future, try and avoid using `<br>` tags. You can accomplish this kind of spacing in a more streamlined way using CSS.


Thanks for adding the bold text. Looks like I missed that bit!


update flags


Strange. I wonder if it is a shell issue? That line just breaks for me since $PROJECT folder in /profiles doesn't exist yet.


This file is showing several errors. Mislabeled variable and missing semicolon here. Missing imports for the activated ability.


Yes, I will probably need help renaming stuff like
`getSkinName()`
to:
`getSkinId()`
in every file.


Missing a closing `</ul>` tag


TeamCity OTA+ :: rvi_sota_server :: ci [Build 451](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7607&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


These operators aren't equivalent at different places in the hierarchy, which means I think you probably want to make this a non-friend operator so that e.g. (ParamInfo&) |= (const VariableInfo &) fails to compile instead of just falling back on the "super" implementation.


only update the flag if move or stationary


在push的时候停在了Writing Objects，怀疑是大文件的问题


It worked for me


Typo


only in case of move


That is not the way parcellite works for clipboard content. Nor is what trim means in most programming languages. This commit only dealt with clipboard content and our config option for trimming the content. But it is what we want for our menu labels. That one I am working on now.


Might be a good idea to add a checksum to files that you download directly. See http://docs.ansible.com/ansible/get_url_module.html


тут видимо должна быть проверка на details.InitialOrder.PickupAddress != null, а не details.PickupAddress


можно просто отдавать false\true

return substr($ticket, 0, 3) === substr($ticket, 3);


if (isLuckyTicket($left)) {


getLuckyTicket`s`


Updating to current code.


$per`c`ent


Ditto for checksum on this one. @calvinmclean


I do indeed get a problem with cycling escort packages immediately ending if I remove this whole chunk here, (from f `(!mStarted && distance(pos.pos, dest) > 10)` to the `else if`), but it looks like the issue is resolved just by doing `distance(pos.pos, dest) > 10)` instead of calling `doesPathNeedRecalc`.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 451](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=7607&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests failed: 9 (9 new), passed: 170, ignored: 3 Build time: 00:08:14
### Failed tests
```
org.genivi.sota.core.DeviceUpdatesResourceSpec.install updates are forwarded to external resolver: <no details avaliable>

org.genivi.sota.core.DeviceUpdatesResourceSpec.GET update requests for a device returns a list of PendingUpdateResponse: <no details avaliable>

org.genivi.sota.resolver.test.ComponentResourceWordSpec.Component resource should add component on PUT /components/:partNumber { description }: <no details avaliable>

org.genivi.sota.resolver.test.PackageFilterResourceWordSpec.Package filter resource should be able to assign exisiting filters to existing packages: <no details avaliable>

org.genivi.sota.resolver.test.PackageFilterResourceWordSpec.Package filter resource should list existing package filters on GET requests: <no details avaliable>

org.genivi.sota.resolver.test.PackageFilterResourceWordSpec.Package filter resource should list packages associated to a filter on GET requests to /filters/:filterName/package: <no details avaliable>

org.genivi.sota.resolver.test.PackageFilterResourceWordSpec.Package filter resource should list filters associated to a package on GET requests to /packages/:pkgName/:pkgVersion/filter: <no details avaliable>

org.genivi.sota.resolver.test.PackageFilterResourceWordSpec.Package filter resource should delete package filters on DELETE requests: <no details avaliable>

org.genivi.sota.resolver.test.PackageFilterResourceWordSpec.Package filter resource should delete all package filters when a filter is deleted: <no details avaliable>

```


@luish Oh, I like it, it's quite elegant 👍

I'm still getting an issue with the `...`, though:

*Cannot convert call result type 'CountableClosedRange<_>' to expected type 'Range<_>'*

```swift
let array = [1, 2, 3]
XCTAssertEqual([3], array[clamping: 2 ... 4]) // FAILS
```

I found some discussions [here](http://stackoverflow.com/questions/38171488/swift-3-ranges-best-practices) and [here](http://stackoverflow.com/questions/37915972/overloads-for-exist-with-these-result-types-closedrangebound-countable) at StackOverflow regarding that as of Swift 3, `..<` and `...` produce different kinds of ranges.


conservation-tests (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2259 of 2262 tests passed, 10 test warnings)

 * 10 tests had: ERR diffs.
 * 3 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 3 tests had: Table big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.87%25-yellow.svg)


conservation-tests (rraustad) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


такой код лучше не использовать, а делать отступы в CSS (если нужен отступ)


$arr = [1, 2, 3, 4, 5, 6, 7];


English comments only. Replace array_sum and array_product with your own functions (use array_reduce).


Insert var before `prop = element.properties[name];`


I fixed it, thanks.


а чего удалил? или не нашли еще подходящего в png?


** * EXCITED * **


committed.


больше не предлагается опция pickup in store в estimate shipping (как было раньше)


Also LGTM.


Woot, thanks @stephentyrone !


Хинт у настройки "A value indicating whether "Pick Up in Store" option is enabled during checkout" лучше изменить на "A value indicating whether "Pick Up in Store" option is enabled during checkout. Please ensure that you have at least one active pickup point provider"


we had about 50 or so last year


You can also remove line 67 in aitravel.cpp and lines 71 and 103 in aiescort.cpp, which all read

`mStarted = false; // Reset to false so this package will build path again when repeating`

As you can see from the comment, this was intended to make the path rebuild when the package cycled, but seems to be unnecessary. Sorry for the mess!


DTO나 빌더를 사용해서 인자를 줄이던지해야겠네요. 방금전에 영화타입이 하나 더 늘었습니다. ㅋㅋㅋ


should be:

```
@return \Magento\Framework\Controller\Result\Redirect|\Magento\Framework\View\Result\Page
```


If anyone reads these comments, with improved annotation of all sorts of SVs in SnpEff 4.3 we're moving from bed file based prioritisation to gene name based prioritisation as SnpEff now correctly annotates genes at both breakpoints for all SVs (incl. Inv, Dup, Del). There will be less hard filtering and more soft filtering plus levels of prioritisation.


In face, the mStarted variable may be wholly unnecessary. Give me a moment.


@czhu12 @chriszhu1 I would really like to use some Slacknote data in the 310 project this term; any chance we can talk sometime?


Why not:
```C++
if (Flags & QUEST_FLAGS_RAID)
```
Confused...


Well, `A => Option[() => B]` allows things that `A => Option[B]` simply *forbids*, namely testing definedness without the side effects from apply (even if the function is defined), and that's what @viktorklang talks about.

I'll make up an example: imagine `apply` blocks, `isDefinedAt` doesn't, and you must call `isDefinedAt` in a context where you can't block, and call later `apply` in another thread.
In such a context, you clearly can't use `A => Option[B]`.

But though this use case is plausible, I made it up, and don't know any real one. I've looked a bit for this, or the use cases that @viktorklang mentioned, and failed to find actual examples—I'm not yet sure any real examples exist. Viktor could you provide one?

#### TL;DR. for what's below: I've looked for examples in Akka and have half-baked ones, not sure you want to read them 😄

For sure, Akka is full of uses of `isDefinedAt`, but they might all be avoidable: It seems many are followed by `apply` in the same context, hence they could be rephrased with `applyOrElse`—or at least, it's hard to see exactly what would break. Even when dequeuing messages from Actor inboxes (https://github.com/akka/akka/blob/65a9e0a0f9372d2e2ecb945c8a9dae129a438f77/akka-actor/src/main/scala/akka/actor/dsl/Inbox.scala#L104), I only see one potential issue with using `applyOrElse` — namely, you can't drop a message you're handling from the queue before running the handler, only after*. I imagine this is a problem if a message handler tries to fetch another message, but I don't know if this is legal and can't write an example off the top of my head.

*Why can't the actor drop the message from the queue before running the handler? I don't see how when I imagine the code. For concreteness, below's what I imagine.

I imagine using `applyOrElse` to process the queue using something like the following —
for each message in the queue, try applying the handler with applyOrElse, passing as failure case code recursing on one of the next message. If the handler succeeds, drop the message from the queue.
Here's that in Scalaish pseudocode, even though there's tons of things wrong with this:

```scala
var queue: List[Message]

def handleMessage(handler: PartialFunction[Message, Result]) = {
  def handleMessagesLoop(messages: List[Message]) =
    messages match {
      case msg :: rest =>
        (handler andThen {... /*drop msg from queue */... }).applyOrElse(msg, handleMessagesLoop(rest))
      case Nil => NoResult()
    }
handleMessagesLoop(queue)
}
```


Переименуй лучше в "New York store", а то это наззвание смотрится не очень после выбора (для демо-магазинов)


os_2_0_dependency_reduction_signals_slots (schrismartin) - x86_64-MacOS-10.10-clang: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


@mpirio FYI, 0.6.1 is now live on npm


допиши стили CSS для блока показа адреса (когда один магазин). надо показывать его посередине как и сам checkbox " Pickup" (и в LTR и в RTL). если будут сложности, то пусть Маринна доделает это прямо в твоей ветке


status: false?


I like Milk Shakes


Would it make sense to start `.get(,None)` for other ones of these as well?


I don't consider Frozen Yogurt as IceCream


Yeah, that's the intention. I have assumed that if the server were unable to send data to the client, then the netcat test would fail.

I don't check the NAT's forwarding tables at all - I guess these tests are more of the 'black-box' kind. It could definitely be worth adding unit tests to the C# code though.


Well, detecting cycles will be tedious, see e.g.: https://en.wikipedia.org/wiki/Tarjan%27s_strongly_connected_components_algorithm
I would keep the depth.


фразу "free shipping" надо убрать (из dropdownlist когда несколько точек, и просто текста когда одна точка), когда стоимость 0. лучше просто ничего не показывать


convenience:
editPassword
checkRememberPass
checkShowPass
btnLogin


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.247.6a4bf3f2b507bf6e9ad503fcbbe180ede7e59162](http://build.palaso.org/viewLog.html?buildId=74717&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 949, ignored: 50 Build time: 00:35:33


private


Rang en dur dans le code ?

Et il manque le fichier de conf sample, non ?


you're welcome @dacresni. How do I get in the slack? I'd like to help improve the `readme.md` next, but have some questions.


Please assist in making improvements


Can you clarify why the namespaces were changed from `System.IdentityModel` to `Microsoft.IdentityModel`? This seems to cause an issue when using `CreateToken(SecurityTokenDescriptor)` because of a type mismatch. My project is completely hosed because of the reference incompatibilities.

```cannot convert from 'System.IdentityModel.Tokens.SecurityTokenDescriptor' to 'Microsoft.IdentityModel.Tokens.SecurityTokenDescriptor'```

The code I've been referring to for implementing JWT is here: http://blog.asteropesystems.com/post/securing-web-api-requests-with-json-web-tokens. It's a bit over a year old but seems valid. I was under the impression that the newest namespace for JWT tokens is under `System.IdentityModel`. And I'm using .NET 4.5.1.

Am I missing something?


I do not know how to get to these comments the correct way.  i have to go through my EDIT history


It is good idea to use a separate class for SharedPreferences.
Take a look for [this one](https://goo.gl/DKfqJJ)


Yep, seems to be totally unneeded. As I recall I originally introduced mStarted to fix the starting time calculation for Wander packages, but it's no longer needed there because Wander now runs on a counter that doesn't start until the package is called, rather than setting its start time as soon as the NPC is loaded. (This was the original problem, and it meant that an NPC with wander later on the AI package list would think those later packages all started at the moment it was loaded by the player entering the cell.)

Please remove all instances and comments about mStarted, and thanks for bringing it to my attention.


Warning due to implicit conversion to bool


Thanks, makes sense. Don't worry about adding the checks on the tables
for the time being. It would be useful to remember this as a feature
idea however, for future work. Could you open an issue for an
enhancement on GitHub, and reference this comment's URL? That way the
issue could be contextualised by the code example we're discussing here.

On 2016-07-11 18:08, TMVector wrote:
> Yeah, that's the intention. I have assumed that if the server were
> unable to send data to the client, then the netcat test would fail.
>
> I don't check the NAT's forwarding tables at all - I guess these tests
> are more of the 'black-box' kind. It could definitely be worth adding
> unit tests to the C# code though.
>
> --
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub [1], or mute the
> thread [2].
>
> 	*
>
> Links:
> ------
> [1]
> https://github.com/niksu/pax/commit/3423b4d22d307d203fee9eb86a44c5fdc5c7427a#commitcomment-18198546
> [2]
> https://github.com/notifications/unsubscribe/ABDTTsJpOoYtPGE9bsDiJYgyeasFPjPkks5qUnh9gaJpZM4JJQV1

--
http://www.cl.cam.ac.uk/~ns441/


про CSS (предпоследний мой коммент) забудь. я забыл кеш сбросить


只要客户端加上相应请求头即可。postman一样可以


по картам гугл. я получаю в консоли предупреждение (не ошибку) - Google Maps API warning: NoApiKeys https://developers.google.com/maps/documentation/javascript/error-messages#no-api-keys

посмотри, что такое. какие могут быть проблемы. где их указывать? они платны?


explicit margins are bad


You have the right idea here, but, going forward, `<center>` probably isn't the best tag to use here. Center is actually an outdated tag that centers the content of the tags it contains. To center this text, a better approach would be to use a `<header>` tag inplace of `center` and then center the contents of that tag using CSS (we'll explain how to do this soon).

In my last comment on the page, I'll go more in-depth on why we want to use tags like `<header>` in places like this.


Thank you for clarification :+1:


Oh yes, you'd still need the second subscript method that receives a
ClosedRange
instead of a Range. Hopefully I'll find a better of doing that.

On Monday 11 July 2016, Adriano Ferreira <notifications@github.com> wrote:

> @luish <https://github.com/luish> Oh, I like it, it's quite elegant 👍
>
> I'm still getting an issue with the ..., though:
>
> *Cannot convert call result type 'CountableClosedRange<>' to expected type
> 'Range<>'*
>
> let array = [1, 2, 3]
> XCTAssertEqual([3], array[clamping: 2 ... 4]) // FAILS
>
> I found some discussions here
> <http://stackoverflow.com/questions/38171488/swift-3-ranges-best-practices>
> and here
> <http://stackoverflow.com/questions/37915972/overloads-for-exist-with-these-result-types-closedrangebound-countable>
> at StackOverflow regarding that as of Swift 3, ..< and ... produce
> different kinds of ranges.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/luish/swift-evolution/commit/1323181085de5de0f828cef567a318597cbd929d#commitcomment-18198306>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAIKlK--TOPMMKDgyHaIJ_XTI5v95WGiks5qUnUwgaJpZM4JFa81>
> .
>

--

---

*Luís Henrique Borges*
iOS Developer at IBM <http://ibm.com>
Dublin, Ireland - luish.github.com


Oh, I didn't notice the extension is in the store again. @wickwood told me in issue #44 that it got removed from the web store and then I dedided to reupload it again. And then I also removed the warning.

But I think I'll have to reintroduce the warning now that the extension is available again. But I might have to delete the entry I've just created in case the buyer asks me about it.

I'm also a bit concerned because the extension is still claiming it's open source and linking to this repository. But it might be useful on the other hand, because people will see the warning when clicking on the link.


Great job using a `<span>` tag! For best practices, the `<span>` tag should be placed inside of your `<h3>` however. Also, you're missing a closing </span> tag.


In this way, you also cannot have cycles, don't you? And if we don't allow cycles, then this is classic DFS - just keep track of objects under construction and don't let the same object be constructed twice. (Yo dawg I herd you like object construction..)


Oh ok... and then down the path I'll learn how the files work! I know array () $var and basic stuff


IOError: cannot identify image file 'python.png'


Noice, now I need to find a solution to the LWJGL code being in the objects, I'll make an issue


LGTM.


czemu taka zmiana?


Obtenemos del servidor el nombre del usuario, fotos y enlace a imagen de avatar

Definimos la ruta del lado del servidor que nos devuelva una pagina y esta página la va a requerir app.js que tiene toda la logica de la app

Primero se definen las rutas particulares y luego las rutas que son mas generales


It is not needed anymore. I am going to delete it in my following commit.


Oh, wait, it doesn't build xD I'll see what I can do!


need to add on the errors


Tests showed some problems with this change.

In nxdrive, _abspath_deduped is usually called with file name (variable "name") defined beforehand. For example:

```
    def duplicate_file(self, ref):
        parent = os.path.dirname(ref)
        name = os.path.basename(ref)
        locker = self.unlock_ref(parent, False)
        os_path, name = self._abspath_deduped(parent, name)
        try:
            shutil.copy(self._abspath(ref), os_path)
```
If the limit is set to 3, when the last duplication is called, name is set to "filename__2" beforehand. Then _abspath_deduped is called and does not return a valid new name (LimitExceededError is raised instead), so it presumes the predefined name (filename__2) and use it as deduped name, causing it to be overwritten.


Yes. If you use app-drawer-layout, the drawer will always be open when narrow. If you don't want this, either set `forceNarrow` on app-drawer-layout or use app-drawer without app-drawer-layouy.


@jjallaire I'm not sure which version I should test against here. I guess it does not have to be the very latest version.


This commit broke attribute selectors with values containing `#`, ex `a[href='/page#anchor']`


Minecraft does 90% of the stuff and XProtocol allows Minecraft to connect and XPocket triggers stuff on events sent from Minecraft


-I used a simple solution for now by reducing the delay on erase by 100ms. I may need to rethink the logic of chaining the transitions together.
-I also deleted some obsolete comments.


We thought it might be a shell discrepancy, since it worked for some people. I'm using Bash with no additions.

Does it work for you if you add `mkdir $TEMP_BUILD/profiles/$PROJECT` to the line above the `cp` command?


Seems you missed a pipe here (I'm now getting tr usage output in the middle of a run).
Also, shouldn't it be a comma that's being replaced?


Great work! Your resume looks fantastic. Great job planning out your page and using appropriate tags to style your HTML.

Make sure to indent your HTML only when necessary (i.e., when a tag is a direct child of its parent tag), and to remove as many unnecessary line breaks as you can before submitting your HW. The tidier your code is, the easier it will be to build on it!

I rewrote your code with some more specific tags that will help you write clean, maintainable code moving forward. This kind of formatting is especially important when we start to write CSS, as it will help you hone in on certain parts of your code quickly.

Please take a look at this and come tap me on the shoulder before class if you have any questions about the changes. You can also schedule office hours for tomorrow (tuesday) night and we'll go over them then!

Again, great work!
Zack


You forgot to copy the GLFW wrappers to jgloom-lwjgl


Note that some browsers like Chrome will fail to load the file unless it's served via HTTP.


@mailinglists35 yes, it looks fairly straight forward to extend ZFS_IOC_DATASET_LIST_NEXT in a similar way.


The traditional way to make the Chinese/Japanese/Korean fonts is to use a modified Texture Font Generator (still Windows-only, have never tested it or any other SM TFGs with Wine):
http://kki.ajworld.net/tfg_cjk.zip

To add other characters, you'd have to edit the source code and recompile...


conservation-tests (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1671 of 1671 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-71.0%25-green.svg)


You can merge the PR into your branch too. Instead of doing a kind if "cherry-picking".


line length


plant-autosizing-gaps (EnergyArchmage) - 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)


its not a warning


If we do this, we should at minimum leave a message where the textures were written, since it may be difficult for people who have a weird install to find their texture packs.


runs fine on my BSD against a server with SAN (DNS+IP)

For the rest: do I have to consult my optician?


I didn't intentionally leave the .pyc files in, but this is Python, which isn't compiled in any conventional way. Those .pyc files are automatically generated by Python (if you so choose) just to make initialization a little quicker when it's imported later.
Also, this commit's almost a year and a half old. pls


oops?


Resolves #1.


<h4> should be lowercase


The algorithm was


i didn't get the assignment on coursera,
what we have to do in that blank space.


It'll still render, but all tags should be written in lowercase for best practices.


(call me)


Danke


ref #495


Make sure you un-indent closing tags so that you can quickly see what children are contained within a parent tag. The `</ul>` above should be the same indentation at the opening `<ul>`


Make sure all tag names contain only lowercase letters.


plant-autosizing-gaps (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2248 of 2262 tests passed, 545 test warnings)

 * 557 tests had: AUD diffs.
 * 12 tests had: Table big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.38%25-yellow.svg)


Ha! Be careful, even if you delete it, each change you make will also appear in your commits.


I'm still working on it- maybe make a method `getIdentifier()` and override it in LWJGL?


Genisys I've seen them program and add blocks in item.php


Hehe, I see you've spotted it now. :)
I probably should've looked around the code a bit more before my original PR to spot these other similar cases anyway, consider my optician consulted!


Same goes for sponsor, should be Curiosity Media


Yeah, that's a good idea actually. I'll add it to the new code I just commited


Great work Dave! Your resume looks awesome. Very good job planning out your page, and styling with the limited HTML/CSS we've learned so far.

Make sure to indent your HTML using only one tab for each child element. Just as important, when you close a parent tag, make sure the closing tag aligns tabbing wise with it's opening tag. Finally, remove as many unnecessary line breaks as you can before submitting your HW. The tidier your code is, the easier it will be to build on it!

(i included examples of this kind of tabbing in the code below).

I rewrote your code with some more specific HTML5 tags that will help you write clean, maintainable code moving forward. This kind of formatting is especially important when we start to write CSS, as it'll help you hone in on certain parts of your code quickly.

Please take a look at this and come tap me on the shoulder before class if you have any questions about the changes. You can also schedule office hours for tomorrow (tuesday) night and we can over them then!

Again, great work!
Zack


Comment on commit


conservation-tests (rraustad) - i386-Windows-7-VisualStudio-12: OK (2225 of 2228 tests passed, 10 test warnings)

 * 10 tests had: ERR diffs.
 * 3 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 3 tests had: Table big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.87%25-yellow.svg)


сожержать
:smile:


seems we can remove Nel with this implementation.


this is not my patch and I'm no author.. I fucked up while adding cChangeID for the gerrit

this is proper patch with author


Hi Adam,
Thanks for your correction.
I see, the protocol directive lines for children classes were already moved to the message base... which makes the code cleaner.


HI Rev,

I using this latest version, just booted it up but i can't see any Parking markers, any idea's?


https://github.com/Download/bridalapp/issues/1145
https://github.com/Download/bridalapp/issues/1139


Accepting changes to accessNav readme.md


lgtm


Since this has ```TARGET_SELF``` it should be fine, but it won't hurt to check for (unitTarget).
Actually I would have used unitTarget instead of m_caster because of the target_self.

Sorry for late reply, but only now I got my hands on Spellwork.


Is the hardcoded file name here intended?


I looked around and I don't think there's a way to make the identifiers mandatory and still have the `@FunctionalInterface`. Making them static is just a simple improvement I forgot to add along the way.


I don't know exactly which version either but I'd just use a relatively
recent one (can't hurt to move people to later preview releases!).

On Mon, Jul 11, 2016 at 1:24 PM, Yihui Xie <notifications@github.com> wrote:

> @jjallaire <https://github.com/jjallaire> I'm not sure which version I
> should test against here. I guess it does not have to be the very latest
> version.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/rstudio/bookdown/commit/49283ae011fa1fd556c64c89a9cfb32d1b4e1313#commitcomment-18198828>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAGXxzE22IGBOzLDOtB8spwwrOlLF85Wks5qUnw-gaJpZM4JJl0H>
> .
>


Ok, thank you


It's definitely later than 1000. I'd just change it to 1200


Is this depending on the variant effect method used for SNVs? We're using VEP there.


That I need to do.


Actually pointed-by list should not be stored in the model. It's not something you can save to a file, it's rather a dynamic list meant to help the user :smiley:


Yeah @ireade,
I just noticed the bug while I was trying to migrate to ES6.
😞


Accept the invite please.


Might be a good idea to add a checksum to files that you download directly. See http://docs.ansible.com/ansible/get_url_module.html


Typo @calvinmclean


Let me know when the matrix-scalar subtract is pushed.


@willwach The error below was fixed in version 2.5.1. It would happen if the app process was not running at all (background or foreground) and a notification was received without a `NotificationExtenderService` setup.
```VB
java.lang.NullPointerException
at com.onesignal.NotificationBundleProcessor.ProcessFromGCMIntentService(NotificationBundleProcessor.java:57)
```

Add the full stacktrace here if you're seeing the issue still after this fix.


That's one of the reasons why we have to get rid of the "hard-coded" directory names :)


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


I realised something strange about this test case.  If I name the imported module properly, like
```agda

module Issue623.B where

S : Set
S = S
```
then the termination error will be report the function `B.S`, not `S`.  However, relative to the importing module, this module *should* be named `Issue623.B`.  I guess what is happening is that the module is read for itself and given name `B`.  If it succeeded to type check, the import would later complain that the imported module "can be accessed via several project roots".


Here is a comment


as an in-game host, or logged on a dedi?


Ooo interesting, I'll look into it in a few mins, I'm fixing up the tests now


I had wrote here that mStarted seems to be totally unneeded, but I guess it is serving a function, isn't it? Looks like it's keeping the path from being constantly recalculated while the distance is greater than 10. I wonder if we can't just make the path empty between package cycles. Then, if I'm not mistaken, we could just check if the path was empty before doing the initial path build instead of using mStarted.


it might be better to follow MNE-Python test script style here

you can do:
```
...
import mne

def test_ttest():
    """Test t-tests"""
    ...

mne.utils.run_tests_if_main()
```


Ignore the "Added lwjgl to lwjgl gradle" part, turns out that was just an Idea config


Hmm... It's impossible to load a conversation without any NPC options in BetonQuest, but I guess it would be better for new packages created in the editor. Or I can simply create new conversation with an empty default option, that would be good too.


It seems that the culprit is in `TypeChecking.Errors`:
```haskell
-- | Drops the filename component of the qualified name.
dropTopLevelModule :: QName -> QName
dropTopLevelModule (QName (MName ns) n) = QName (MName (drop 1 ns)) n
```
This does not work with hierarchical modules!


Thanks, I'll add that @julianpistorius


Was just something like, oh index access and it could be in the future, better check that :)