I just ran pylint on this code, and it had this to say:

************* Module feature_selection_utils
C:  9, 0: Line too long (117/100) (line-too-long)
C: 11, 0: Line too long (107/100) (line-too-long)
C: 20, 0: Line too long (117/100) (line-too-long)
C: 50, 0: Line too long (114/100) (line-too-long)
C:  1, 0: Missing module docstring (missing-docstring)
C:  7, 0: Invalid function name "calculate_features_target_correlation" (invalid-name)
C: 24, 4: Invalid variable name "y" (invalid-name)
C: 29,12: Invalid variable name "f" (invalid-name)
C: 33,12: Invalid variable name "f" (invalid-name)
C: 38, 9: Invalid variable name "ax" (invalid-name)
W: 38, 4: Unused variable 'fig' (unused-variable)

It is funny that it calls things like y, f, and ax invalid variable names.

Any idea why it doesn't like the function name "calculate_features_target_correlation"?


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


@PEMapModder i see.. I fixed this in the "old" way in ClearSky some weeks ago. Maybe i'll copy this change by you to there to keep it.. unified? is that the correct word?


GroundDomainUpdate (mitchute) - Win64-Windows-7-VisualStudio-12: OK (2112 of 2119 tests passed, 549 test warnings)

 * 556 tests had: AUD diffs.
 * 6 tests had: EIO diffs.
 * 7 tests had: ESO big diffs.
 * 6 tests had: Table big diffs.
 * 2 tests had: ESO small diffs.
 * 1 test had: MTR big diffs.

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


112796613_DesicDehum (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2120 of 2123 tests passed, 1 test warnings)

 * 1 test had: AUD diffs.
 * 1 test had: Table small 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)


while doing some testing found an issue here that we forgot to set the vmm_type here. I will go ahead to fix it then.


This doesn't seem to be used anywhere in this commit. Also, the implementation seems problematic, as it will throw an error if the number is not an integer.


What CGI bug is fixed by this change?


SZVAV-Fan-Control (rraustad) - i386-Windows-7-VisualStudio-12: OK (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-100.0%25-green.svg)


We don't want to notify the removals here?


SZVAV-Fan-Control (rraustad) - Win64-Windows-7-VisualStudio-12: OK (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-100.0%25-green.svg)


Wonder if you also shouldn't unschedule ticks from here.


we actually don't need this as while doing retrieve_domains() for the old config file, if the vmm_type is vmware, we automatically insert one more openStack vmDomain. I will go ahead to make a fix on this then.


I did some more reading in the libcurl-errors(3) man page, and I noticed:

> CURLE_WRITE_ERROR (23)
> An error occurred when writing received data to a local file, or an error was returned to libcurl from a write callback.

The first case does not apply, but we do register write callback functions. They are set with CURLOPT_WRITEFUNCTION. My reading in the documentation for that option is: any value returned not equal to the data size (size*nmemb) will raise a CURLE_WRITE_ERROR. For example, In swupd_download_version_to_memory(), we may return 0 for an error; and in swupd_download_file(), we return -1 for errors, and this will also raise the error.


Line 115 and 112 are the same. I would move this up to 112 because `id` is a clearer name for this than `targetTft`


If you want to group these into a nice set of Intercom permissions see the bottom of this older article: http://www.balbuss.com/adding-new-group-permissions/


Good catch!!  I will fix now.


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


@ZoeBlumenfeld I'm not sure that I did this correctly since the changes aren't being reflected on the site!


Can do - and on line 30 I'd have to do this though: `service = goa.NewGraceful("", false)`, since `service = goa.NewGraceful("", false).(*goa.GracefulService)` seems to fail for me with non-interface errors. I'm not quite sure what that trailing `.(*goa.GracefulService)` added since I'm newer to Go?


Interestingly, `currentTimeMillis()` can be affected by various clock adjustments.
Please see:
http://stackoverflow.com/questions/1770010/how-do-i-measure-time-elapsed-in-java/1776053#1776053
http://stackoverflow.com/questions/238920/how-do-i-calculate-the-elapsed-time-of-an-event-in-java
http://www.techper.net/2008/08/10/systemcurrenttimemillis-systemnanotime-and-their-resolution/


LGTM


Looking great :+1: Let's :ship:  it!!


I suspect this may be losing some metadata or something. Might be better if it is possible to clone the fragment from source and use it in here. See if there is a `clone` or `copy` method on `TextContainer` or `Fragment` or something.
Also, looking at `getUnsegmentedContentCopy()`, maybe just calling that on the source would be enough since it looks like it makes a copy.


- [ ] fix `$no`


> What purpose does this addition serve?

There's a bug with Acts As Taggable ([see the "For MySQL" section](https://github.com/mbleigh/acts-as-taggable-on)) that causes the gem to break when using MySQL. Obviously we aren't using MySQL, but I included this for compatibility.


Was about to send the exact same PR, perfect timing @shicks !


Nice, zat' an unused field? :smile:


RIP *baned.php*


GroundDomainUpdate (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1562 of 1563 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-99.94%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.71%25-green.svg)


Should be unsigned char


ý em là sao?
Btw, anh đã sửa được lỗi send char qua cho xe lăn. Chỉ cần tăng baudrate (tốc độ) truyền lên là xong. Giờ mình full control được xe lăn rồi.


Aha! It used to have `.clone()` on here, but you removed it in this commit. That may be what started the source modification problems.


Looks like this is used in metrics.js but not in gateway.js. Revert commit @karrots when you get home.


FIX #1


$options->rcpt = '01012345678'; 이 맞는 것 같습니다.


If you replace the colon ( : ) with a slash ( / ) it will work (i just tried it).  so the url should be /v1/events/E1
And replace E1 with whatever the event is.  Same thing with all of the places with colons.


$options->start 으로 변경해 주는게 좋을 듯..
API 버전업이 되면서 s_start에서 start로 바뀜.


wow amazing


2097-EP-8-5 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


s_end도 end로 바뀜


I'm happy to help if you have any question :)


I'm trying to figure out what's the best behavior when a one of these detectable errors happens.  Should the OpenNlpToolkit throw an exception with a happy error message?  Log the error and then just let the operation fail (as it currently does)?  Something else?

I like adding better information about how to fix the problem (.properties file miessing, property key/value missing, model file itself missing).  I'm leaning towards changing it so that an exception is thrown instead of just `logger.error()` and continuing to try to use the file when it's obvious that it won't work.

What do you think?


trim하면 안되는 파라메터는 고려해야 하지 않을까요?
예로 문자내용입력 파라메터의 경우 의도적으로 앞이나 뒤쪽에 스페이스 혹은 줄내림을 입력할 수 있음


Nope -- turns out I was passing along the arguments to `console['method']` incorrectly. :innocent:


There's also generally no harm in always generating those files, even when packagekit is disabled, as long as the actual features are disabled.


Use !=


Which distro needs librt explicitly?


No need for internal quotes


Thanks @jnealtowns - I caught that sff.supported was the wrong test, but I thought we would get an error if not parsed.

@rohitmalaga : please change this as jeff suggests, specifically - put the code back (sorry) but change the test to sff.identified.  If you can't get to this shortly, let me know and I can do the change.  Either way, this is probably good enough for the OCP demo so I'm going to merge this now.


Hey Yuan, were you able to create your own branch, commit your python plot file to it and push it to the branch?


not sure about this i mean trigering a missile is missile a spell? did u tryed to update missile opcodes? for example NotifyMissileTrajectoryCollision


Just use `$PWD`


You may want to keep the same indent level with the following items.


FixAll support?  Or is that coming in a followup?


You may want to specify the input variable name.


the email though :V


Specify the input variable name.


Technically, i think you could just use LookypSymbols.  But this is nbd.


1. I don't know. Thoughts?
2. It needs some, yes. I'll try to get to that.
3. I could add a planner class config option that you could change, but if you need more than that, could you modify endpoint.psgi to add a planner in the `Attean::Endpoint->new` call? If so, I can add the planner as an attribute of the endpoint.


:clap: :100: for adding the integration test!


@jboz62 - if this is the change you were asking about in the email you sent, it looks fine.


113837367-Fanger-PMV-Equation-Bug (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1008 of 1008 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-29.11%25-green.svg)


111101900_MSHeatPump (Nigusse) - x86_64-MacOS-10.9-clang: OK (2117 of 2118 tests passed, 0 test warnings)

 * 1 test had: ESO big diffs.

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


Yep, created a follow up work item and linked it to this PR.

Sent from Outlook Mobile<https://aka.ms/blhgte>

On Wed, Feb 24, 2016 at 5:03 PM -0800, "CyrusNajmabadi" <notifications@github.com<mailto:notifications@github.com>> wrote:

FixAll support? Or is that coming in a followup?

-
Reply to this email directly or view it on GitHub<https://github.com/balajikris/roslyn/commit/30feedc3ad60b9cea327fa6a20f2c58fec04fb24#commitcomment-16314599>.


Hi Daniel,
what's the use case for this please? What's the string you need to be hashing?


lgtm


Yes, that’s it, thanks for checking.

> On Feb 24, 2016, at 8:05 PM, Andrew <notifications@github.com> wrote:
>
> @jboz62 <https://github.com/jboz62> - if this is the change you were asking about in the email you sent, it looks fine.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/meetinghouse/cms/commit/5f4020e331d1a4df92caf04d05d7dab4f65494d6#commitcomment-16314632>.
>


Passing this directory as a parameter makes a lot of sense IMO. For example, the name "mobilespec" is defined in the Python code, and the JS scripts only receive it as a parameter. It makes sense for the screenshot directory to do the same thing, and there should be one per builder.


This line indicates that we cannot include '.csv' in the input variables. You may want to clarify this at the beginning documentation


> Which of these fields need to be nullable?

Aren't all of the columns (except for the id) null?  Here's the output from .schema taggings:

`CREATE TABLE "taggings" ("id" INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL, "tag_id" integer, "taggable_id" integer, "taggable_type" varchar, "tagger_id" integer, "tagger_type" varchar, "context" varchar(128), "created_at" datetime);

CREATE UNIQUE INDEX "taggings_idx" ON "taggings" ("tag_id", "taggable_id", "taggable_type", "context", "tagger_id", "tagger_type");
CREATE INDEX "index_taggings_on_taggable_id_and_taggable_type_and_context" ON "taggings" ("taggable_id", "taggable_type", "context");`


That's the other possible fix, which is to remove the conditional around the recipes. They shouldn't be there in the first place as packagekit isn't needed to run the rules.

That said, I would consider it better form to not distribute these generated files as it would be best to have them created during the build rather than by the developer.


yes I also thought this doesnt feel the same as the rest of the operators. This patch was just a personal thing
because my mapping configurations act with some latency and sometimes I dont know If the operator got
triggered or not. In the case of :mark I think it is already too explicit to display a message, the operator
feels more invisible to me.


As far as I know, the variable is just the file name instead of the CSV file. A little confused on the comments.


Many-to-many (n:m) relationships have to be fixed. Also relationships like house -> character (1:n and not n:1).


Für was ist die exe und wie muss man die starten ?

MFG


Good catch. Will interactive rebase before merging up.


People are making carat hacks using zombie mode and they are making boss mode hacks that allow you to make a 2 round bm room. But they are daring to sell it for 55$ per month. Someone if you have. A better hack or something that comes close and are willing to share please add me on skype @halfthelife.
@neetVeritas
@VyrusMaker


Will update this once I merge with app_design.


um grande e trabalho superandos as es necessidade


Maybe if there was a way to indicate that a sign's content needs to be translated, but I'm not sure how that would work currently.


111101900_MSHeatPump (Nigusse) - 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)


I don't think this constructor is used anywhere. Also, I don't think we should be handling DECIMAL with missing precision/scale at this point in the code, since they should be syntactic sugar for DECIMAL with a specific precision/scale.


点点滴滴


Yeah I think that's right, and then can remove the `if (!mapped)` check in the tick callback as well.

In the future comment on the upstream issue! Still hoping I can land a version up there before we land all this
https://bugzilla.gnome.org/show_bug.cgi?id=762642


@mvwieringen Thanks, man.


Like it


Fix teh formatting issues


Rock on!  Thanks John

Jen Duncan
Sent from my iPhone

On Feb 24, 2016, at 7:57 PM, johnmcmahon <notifications@github.com<mailto:notifications@github.com>> wrote:

If you replace the colon ( : ) with a slash ( / ) it will work (i just tried it). so the url should be /v1/events/E1
And replace E1 with whatever the event is. Same thing with all of the places with colons.

—
Reply to this email directly or view it on GitHub<https://github.com/venicegeo/pzclient-sak/commit/08856f2113a534c3780894982344a88529ab0b2c#commitcomment-16314499>.


That was needed to cast back the interface but now that goa is using structs that is not necessary anymore.


errortag 扩展性不强，以后增加其他日志怎么办么。应该log就是一个json结构，接受几种日志的上传。其中关于dataitem的日志，是其中一类。
提供给对方判断tag 有效性的接口看起来也很怪啊，当面讨论吧。


Thanks! Looks great


make this final, as it's not supposed to be overridden by subclasses


This line needs to be removed


read it 。三记得review


Compilation - yes, but they also need to do preprocessing on device side and that's done using 'host compiler -E'


Yeah, feel free to upload a new patch there after these comments and I will comment :)


Some more lines?


Shouldn't it be `fuzzyR` instead of `fussyR` ?


Call this method `buildTypeParameters` or `makeTypeParameters`


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


With ".csv"


What issue was addressed by this commit?


Zz


Ben de bilmiyordum mantikli gelmeyen yerleri ogrenmeye calistim :) sen de anlamadigini sormaktan cekinme. Herseyi bir anda ogrenmen imkansiz ama yavas yavas sorular sorarak anlamaya calisabilirsin.


It looks like this could be package-private


111101900_MSHeatPump (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2155 of 2156 tests passed, 0 test warnings)

 * 1 test had: ESO big diffs.

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


aynen oyle, farkindaysan sorun yok.


This commit breaks jersey2-guice; see the comments here:
https://java.net/jira/browse/JERSEY-2551 (which, I can't verify is actually fixed)

https://github.com/Squarespace/jersey2-guice


It's happening :+1:  https://gist.github.com/JohnyDays/fdbab1f031f0218aee8d
I do have a question, is it safe to assume the same bundleURL for each platform?
That would probably simplify / make it faster. I'm leaning towards the safer, less perfomant option right now


you guys have a key in keyboard for that :O !!!!!

Thanks & Regards,
Deepu

On Thu, Feb 25, 2016 at 7:42 AM, Gaël Marziou <notifications@github.com>
wrote:

> µ would be fun though I'm not sure it's easy to find on all keyboards and
> also you may have hard time to capitalize it to respect java class naming
> conventions :D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhipster/generator-jhipster/commit/1436271a98e8b81b170f4869f4e5ff1bbe8201d2#commitcomment-16313053>
> .
>


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


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


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


delete


Very close, nice job! I think this looks a bit better:

```ruby
s = s.to_ascii if !options[:preserve_unicode_characters]
s = s.squish
s
```


Just add another commit with my suggested feedback and I'll accept it. :)


Why do we add leading 0s?


This method is a bit hard to follow. Can you explain what it's trying to do?


Thanks!


아~ 잘못 생각했습니다. 삭제하겠습니당~


What I mean by my comment is will there ever be an instance in the taggings table where `tag_id` is null? By default all rows in the database can be null. This value should have meaning, and where null isn't needed it's a good idea to prevent the field from being set to null.


Maybe call this POWERS_OF_TEN_TABLE_LENGTH


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


:relieved: Phew.


Why not the fluent method?


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


Hello again Kev,

That's fantastic, thanks for that! Excellent work mate, exactly what I
needed. Is there any plans to build on the catchup section (if its actually
theoretically possible) and maybe a record setting option? Also, is there
any way to add EPG and choose programmes as required?

I'm definitely going to consider looking at the webui stuff in the future
as I have some experience helping a friend build his website with some php
code. Its certainly an interesting subject for me.

I honestly highly commend you for your work on this, as after many searches
on google you seem to be the only person who is expanding on an already
lacking service. So hats off to ya pal!

Thanks again.

Regards,

Fraser

Hi fraser1978.

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

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

—
Reply to this email directly or view it on GitHub
<https://github.com/kevjs1982/eetv-webui/commit/8cd482dbbab10c7b93363c0af4355c6ef582628c#commitcomment-16298393>
.


Great. The "we're not using MySQL" part is what I wanted to double check.


Cool, thanks!


read it


113837367-Fanger-PMV-Equation-Bug (RKStrand) - x86_64-MacOS-10.9-clang: OK (2116 of 2116 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)


Yep, you are right, I don't know what I was thinking there, I used globbing but the `find` and `xargs` is totally unnecessary in that case.
Funnily the glob resolves to all the .babelrc files which are passed as arguments to find, since they are all files find will just "return" them again, so the command does actually work, but is not correct indeed.

This should already suffice: `rm node_modules/**/.babelrc`


어라 이거 상위에서도 땡겨올수있나요?


这个出来了？


我比較喜歡底下這個說法，也就是不加 null check or assert ，針對不會 null 的情形，就讓它 warning 就好，而且如果要加的話，專案裡有很多都有可能null，那有的 check 有的不理它，就態度不一致，如果都加的話，那專案就反而不 clean
了，my opinion
Warning is not an Error. And the warning which you are talking about says "it may produce", don't say 'it must produce'. So choice is yours. Either add null check or not

So, If you are sure that findViewById in your code will never be cause of NPE, then don't add the null check.


This looks good! I'm going to test and then merge :+1:


提交代码


You changed the convention for the position. What about the normal? In which reference frame is now?


Objectのところ良い関数名が思いつかないんですけど何か案はありますか(´・ω・｀)


:-D


Oh crap, the commit message is wrong.


これって、デフォルトでinitdbを実施したOSユーザと同じ名前のPostgreSQLスーパーユーザのアカウントが自動的に作られることを指していると思った(だから、敢えて「起動」を避けた)のですが、勘違いでしょうか。
もっとも、インスタンスを初期化したユーザと起動したユーザは同じだから、結果的にはどちらでも同じですが。


How about you do not break my workflow without offering any replacement?


If you don't use WRF positions any more, you should comment also lines 314...321 because there is no point in computing them if not used.


Actually, in this PR, the write lock is actually still needed.

The purpose of this PR is to show that the test is effective at detecting race conditions caused by concurrent reads and that, prior #5701, taking of the write lock was absolutely needed.

The fact that the same test now passes in #5701 demonstrates that the other changes in #5701 make the taking of the write lock for the purpose of protecting entries against concurrent reads is no longer necessary.

(It might be necessary for different reasons but if that is true, those need to be demonstrated).


111101900_MSHeatPump (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1010 of 1010 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-29.33%25-green.svg)


Should these fields ever be null?


Your patch will break some processes in ImageMagick.   A number of algorithms within ImageMagick require that the temporary file persists on disk in order to function properly.   In addition, the patch changes the behavior of the algorithm, that is, on Posix systems the file is removed immediately but the descriptor stays in memory.  However, under Windows you cannot unlink a file if the file descriptor has not been released.  Consistent behavior across different platforms is a requirement of ImageMagick.


111101900_MSHeatPump (Nigusse) - 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)


master added


yes ,master added


Changed this -> self fixing issues with call-operations


Here it is! Nevermind my previous comment. Awesome. :+1:


that's definitely a new line


Got it! Thanks for that.


read it


fc


Indeed! Fixed.


keep those bits DRY! 🙌🏻


No idea, [here](https://www.python.org/dev/peps/pep-0008/#function-names) says "Function names should be lowercase, with words separated by underscores as necessary to improve readability."


What happens when the password does not match the password confirmation?


Great tests! :+1:


`getParent`; see the [guide](https://github.com/jenkinsci/workflow-plugin/blob/master/COMPATIBILITY.md#plugin-developer-guide).


Yes!!! :D
Em 24/02/2016 12:06, "Felipe Bueno" <notifications@github.com> escreveu:

> [image: gif]
> <https://camo.githubusercontent.com/0281f6de495286ab434108ef35d573e20aebdb23/687474703a2f2f672e7265636f726469742e636f2f6564416f48304b6359372e676966>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sneerteam/sneer/commit/6a5e2b2a3814d93707e72a0fea5df3e85b5cd7fe#commitcomment-16299072>
> .
>


このクラスがコンソールからの呼び出し以外の用途を想定してませんでした。
public static定義は、最初に教わったものを書いているだけだと認識してましたので。

これ以上は確かに規約遵守ですね。
getあったらsetメソッドも必ず用意せいみたいなナンセンス規約は滅びればいいと思ってますが。


Is there a show action?


這邊應該不用定義？正確要對應到 user_id


I get the following error in Chrome: `Uncaught SyntaxError: Unexpected token [` and the blame links don't show up. Removing the surrounding square brackets alleviates the error message and I can see the blame links, but I don't know if that's exactly what you want to do.

Chrome: Version 48.0.2564.116 (64-bit)
Mac: Version 10.10.5


There is no equivalent to `getPublishersList` for a Pipeline project, since there is not a fixed list of publishers like there is for a freestyle project. You will have to refactor things so that this call is unnecessary, perhaps by adjusting the call chain to pass in more complete context.


Note on the commit for instructors, DELETE was only done to allow commenting on full code


Great job restricting the routes to only those being used!


May want to provide for ipywidgets to allow dynamic plots (do not necessarily need to address now)


One minor quibble `L == NULL` won't happen since `alarm_timer_common()` can only be called after on of the Lua has called `tmr` method which sets the alarm, but these all take an L argument so *L0* will have been set up at that point.


Should we check that an error message is displayed?


Good catch. :+1:


now we must wait for @Zaffy . Feel free to exclude this commit from your repo for now.


Yes. I've already corrected it


Excellent test!


I am skipping the version constants in my generator, should I emit them in this fashion?


@NathanSweet
Hi, I really need setIgnoreUnknownProperties method.
Please, release that.


Can you wrap all the operations in `RealmContext` with a `try {} catch() {}` thing? That way we can send this event.

Also, when defining failure strings, do this:
```java
public static class Failures {
    public static final String COULD_NOT_EAT_PIE = "COULD_NOT_EAT_PIE";
}
```

That way it's nicer on us.


Shouldn't this be a `Ref`?


Thank you. Fixed at https://github.com/fkanehiro/hrpsys-base/pull/958


Unfamiliar with this function. Isn't there also savefig() in matplotlib that allows saving as pdf?


Haha, sorry for the scare.

So, with those log utilities fixed, the local `bin` is functioning correctly and finishing the default flyfile in 400ms with `fly-xo` (which is basically `fly-eslint` + a little extra).

Compare that to fly@0.9.1's local `bin` completing in 1.3s :raised_hands:


shouldn't this be `resetTo` @Kureev


cool!


For problems that are removed from spoj.
Example - http://www.spoj.com/problems/SUM1/


add space before =


Include lines to update the template costs field


We should deprecate all non-prefixed macros we expose, especially the
syntax sugar ones here that have nothing to do with the API we vend.
Eventually we could move them to internal headers or if related to the
API being exposed, add prefixed versions that don't make me smash my
head on the desk debugging macro madness (I'm looking at you `geany`
macro and your friend `documents` too !!!) </rant>


Arguments of functions are clearly laid out in all cases, very clear!


I think 'float' is the proper terminology in python instead of 'double'...


Does this offer any help for using temporary  variables instead of trying to get values from block/try/switch ? This would clean things up a bit.


Vamos modularizar essa classe?!
Perceba que está classe está "God", você poderia modularizar essa classe, criando um objeto para trabalhar com as funções da "lista" de produtos.


Is there any reason to save to PDF only? Maybe allow the user to specify, pdf, png etc.?


This is the function that integrates...


Should we have a flash message indicating that the user was logged out?


You may also want to add a title on the graph.


Can you explain what this line is doing?


This is intentional; one element is already embedded in ```struct arena_s```.


Por convenção, nomes de variáveis inicia com minuscula...
Ex.:
        boolean existeProduto = true;
        Produto produo = new Produto(...);


@lsj4401 어때? 맞는거 같아? 궁금함 ㅋ


Removing the 'remotipart' gem resolved the issue for me.


Cuidado com o tamanho dos métodos!
Há métodos que estão um tanto quanto grande.
Teria como diminuir?


Don't we also need a favicon?


There should be an error message for that. I will add one for that.


That's crazy, wut? This is interesting :open_mouth:


Do you like this better than using a sessions resource for the routes?


It seems a bit odd to do the try catches on the non-boolean stuff. Should I do it anyway? For the Failures class, should that be in the RealmContext or somewhere else?


Because it flat-out removed the babel-compile time


The booleans don't mean much -- they're bogus anyway. The reason we want to do that is so we can broadcast to everyone what failures happened, just in case. It should still be in RealmContext.


So yes to all?


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


No. There is only a show page for showing a user, no action for it. So it will look like
```Ruby
     def show
     end
```
So, I remove it.


ok


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


Putting secret keys on the command line is a bad idea since it is visible in a process list and also often stored in command history. This is true for view key restores as well, but more serious for spend keys.

Better alternatives are to prompt for the keys (as is done for seeds by --restore-deterministic) or read them from a file.


ok!


I don't think visiting the `new_user_path` should result in destroying the session as that's not very intuitive.

What else could you do when a logged in user visits that path?


Yes, we can. But in integration test, I have checked those error messages, is that enough?


what does blah mean?


Yes, we can, I will add it.


Oops, I guess that's why you call it a sanity check. I'll fix that now.


- [ ] what would you like to eat?


Is this field used for anything?


I think I know what the problem is now, it's just a matter of figuring out the best way to fix it. I can walk through the trace with you, but basically it comes down to `perform` does what seems to be the right thing (in isolation), but when we follow up with the call to `constrainValue` things go bot because `perform` did too much work (and emitted something it shouldn't've emitted just yet).


Result
---
This is Double linked List Structure
Test :: Insert()
20 --> 30 --> 10
Test :: Append()
1 --> 2 --> 3
Test :: Remove()
1 --> 2 --> 3
1 --> 3
3


what do you want to eat?


113837367-Fanger-PMV-Equation-Bug (RKStrand) - i386-Windows-7-VisualStudio-12: OK (2122 of 2122 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)


Great private method. :+1:


I approve


I didn't test it, but I think it will not work on linux, because on mac bash loads ~/.bash_profile but on linux bash loads ~/.bashrc, maybe we should verify it.

@deniscostadsc ping me to test it later


Remember that controller tests should only invoke a single action.

How else can you set-up the environment prior to issuing the delete request?


OK.


So there is a situation that a logged in user happens to access the `new_user_path`, I will make the user logout(Although when a user is logged in, there is no signup link on page).

That is my initial thought, but I just find out it is a bug and actually I want:
```Ruby
    log_out if @user.present?
```


You forgot a couple mentions of this in gl_flats.cpp which causes it to not compile.
https://github.com/coelckers/gzdoom/blob/master/src/gl/scene/gl_flats.cpp#L663
There's 4 instances of CenterSpot(sector) still there.


JSのfor文でオブジェクトを回すときは動きが特殊なので注意


Build is working fine. I have some errors that I'm investigating:
* ignoremissingresources in pom is highlighted as an error - not sure it's needed ?
* when using jetty:run, I get 404 on static/* resources

In the long run, we should probably add a profile to build at compile time or not, which means:
* configure the web.xml (with or without the filter)
* other ? (remove wro4j dep ?)

You mentionned that this affect also formatter - not sure what needs to be done for that.


Why shouldn't it receive the message that as published by channel2 in line 1345 below?  That's the point of connection resume.


```
$ cat /tmp/test.py
import numpy as np
from hmmlearn import hmm
np.random.seed(42)

model = hmm.GaussianHMM(n_components=3, covariance_type="tied")
model.startprob_ = np.array([.3, .4, .3])
model.transmat_ = np.array([[.9, .1, .1],
                            [.1, .9, .1],
                            [.1, .1, .9]])
model.means_ = np.array([[0.], [2.], [4.]])
model.covars_ = np.array([[10.]])
X, Z = model.sample(10000)

from time import perf_counter
start = perf_counter()
fit = hmm.GaussianHMM(3, "full").fit(X)
print(perf_counter() - start)
print(fit.means_)
print(fit.covars_)
start = perf_counter()
fit = hmm.GaussianHMM(3, "tied").fit(X)
print(perf_counter() - start)
print(fit.means_)
print(fit.covars_)

$ python /tmp/test.py # master
2.128354109999691
[[ 1.04810061]
 [-1.71950131]
 [ 3.48208342]]
[[[ 6.27390103]]

 [[ 7.86157748]]

 [[ 7.44889504]]]
1.9765830049996111
[[ 3.33402637]
 [-1.76825803]
 [ 1.06985529]]
[[[ 7.30412682]]

 [[ 7.30412682]]

 [[ 7.30412682]]]

$ PYTHONPATH=src/hmmlearn python /tmp/test.py # The PR.
0.5280850550002469
[[ 1.04810061]
 [-1.71950131]
 [ 3.48208342]]
[[[ 6.27390103]]

 [[ 7.86157748]]

 [[ 7.44889504]]]
0.44080258200028766
[[ 3.33402637]
 [-1.76825803]
 [ 1.06985529]]
[[[ 7.30412682]]

 [[ 7.30412682]]

 [[ 7.30412682]]]
```


:laughing: fixed b641cb7d


@alonsogarciapablo sounds much better


removed ;)


Ok :+1:


Reproduced in 8.5.1:
Error: http://pix.my/ypPmyXCo?1456394596
Reason: in /var/opt/gitlab/nginx/conf/gitlab-http.conf was:
`server {
  listen *:443 ssl http2;
`
Fixed:
`server {
  listen *:443 ssl spdy;
`


Riirh


Ack, renamed to `TaskOpFactoryImpl`


Is this comment right?  I thought this simulates a lost connection, why is that effecting the connection state?  [RTN15a](http://docs.ably.io/client-lib-development-guide/features/#RTN15a) is really about succesfully restoring connection state, yet it seems you're testing not restoring connection state. I appreciate both conditions are important, but why are we testing only the failure to restore connection state here?


@ksgwr :+1:


prova commento


Initializing project


Connection ID would only be different if connection state could not be resumed, however I do think the primary goal for [RTN15a](http://docs.ably.io/client-lib-development-guide/features/#RTN15a) is to prove that connection state can be resumed.


As discussed, I'd rather not because this would divide updating the state


I think both `type` and `as` should be here. If `type` was changed, then we may need to preload a resource that previously we didn't need to. (e.g. type change from "foo/bar" to "image/png")


@matwey this seems to have slipped through. Improvements are made to the process but keep in mind that we don't have the ressources Django has.


It is!


thanks!


-# Getting Started
 -## Dependencies
 -Tools needed to run this app:
 -* `node` and `npm`
 -Once you have these, install the following as globals:
 -`npm install -g gulp karma karma-cli webpack`


It's good that you have removed the test data from this spec file: `spec/controllers/selfservice/offices_controller/create_spec.rb`


@ssured I have never done this before. I need to learn how to do PR.


Sorry, I see you have a PR for [RTN15d](https://github.com/ably/ably-ios/pull/249/files) which provides coverage for successful state resume, my mistake.


That would fail if `$aCurators` has zero elements. Although that should never happen, within the scope of the function it is a possible situation.

But I do agree that the line is a bit cryptic, so I can at least put a comment there explaining the purpose.


> I thought we were leaning towards omitting it

Ah, sorry - I was taking the "if cipherParams are given then implicitly encrypted = true" feedback wording a bit too literally :) (I was imagining a use-case where someone might set a cipher but want to temporarily disable encryption without having to re-set the cipher when re-enabling it -- but you're right, that's just YAGNI)


it is not expected that this function modifies its ```name``` parameter


Did you look at the license?

> fancyBox licensed under Creative Commons Attribution-NonCommercial 3.0 license.
You are free to use fancyBox for your personal or non-profit website projects.
You can get the author's permission to use fancyBox for commercial websites by paying a fee.

Not sure if our website counts as commercial website. But I guess it's not a good idea to go into this gray-zone exactly for the RMS page;-)


@jwgrenning Hi James, it would be nice if you could point us to some blog / book / documentation on how to use this new feature, like some simple example code.

I had a chance to do some contract testing a while back, where I replaced the onAssert() function with a mock, and a test would look something like:
```c++
TEST(stopw, assert_1) {
    /* Setup */
    mock().expectOneCall("onAssert")
          .withParameter("module", "Stopw")
          .withParameter("id", 1);
    /* Exercise */
    UT_STOPW_CTOR((QHsm*)0);
    /* Check */
    mock().checkExpectations();
    /* Cleanup */
    mock().clear();
}
```
Essentially, the test verifies that the assertion is triggered when it should be. There is no need to exit the test. So I am just wondering, how TEST_EXIT would be used in contract testing.

We could then also document this, along with other new features, on cpputest.github.io.

Thanks, Arnd


make this just reader, as you are not using the writer anywhere


... shit, that shouldn't have happened. I'm very sorry.
It worked for me because I introduced the typo with the merge request -.-


It's a shame we don't have `#ifdef` in Java ;)
Otherwise we could have some sort of "compatibility" layer around HotSpotVMConfig in Graal, with different implementations for jvmci-8 and jvmci-9...
Perhaps we can do something with annotation processors. Like generating a HotSpotVMConfigAccess class that contains getters for all fields Graal knows about, with a default implementation for the fields that don't exist. Unfortunately that would break the nice property of having a single graal.jar that would work on both 8 and 9.


> _It is practically impossible to teach good programming to students that have had a prior exposure to BASIC: as potential programmers they are mentally mutilated beyond hope of regeneration._
>
> Edsger W. Dijkstra

C'est comme ça et pis c'est tout. Oubliez deux minutes Java et valsez le Ruby comme Matz l'a conçu.

* http://ruby-for-beginners.rubymonstas.org/objects/predicates.html
* http://ruby-for-beginners.rubymonstas.org/writing_classes/attribute_readers.html
* http://ruby-for-beginners.rubymonstas.org/writing_classes/attribute_writers.html


Oops, something was wrong on my side


Any reason why this is in staging instead of master? It seems it only changes the non-default version and only on darwin, and it failed before this commit anyway...


Could we add support for setting these values through environment variables?


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


Schissdreck!


- [ ] check


@evemeier Schissdreck


Sorry, I was not aware that your German is that bad. :grin:


right, that is probably only useful in very simple corner cases, like `@Cacheable(key = "#id")` for your example, since only then would the user know what the "cache-level" key is and be able to infer the "storage-level" key out of it using `getDocumentId(cacheLevelKey)`.


I totally agree, much better to throw an exception than letting it just roll on without the resource which will cause an NPE or something else down the road.

It means adding "throws Exception" to the method signatures.

Do you think it better to throw the general Exception class with a detailed message, or do you think it better to create special exceptions, or maybe use FileNotFoundException ? (I lean towards the FileNotFoundException)


The way you'd do this programmatically anyway is via `Cache.get(id)`. You'd never access such low-level API as a user IMO.


Hallo,

die exe einfach durch die arma3server.exe ersetzen und damit starten.

Diese erhöht die Performance deines Servers.


Agree. Reverted https://github.com/yiisoft/yii2/commit/1e1baec6e6c3949740066623b1245216acad9419


Las asociaciones ponelas en el archivo donde definis Movies


The way you'd do this programmatically anyway is via Cache.get(id). You'd never access such low-level API as a user IMO.


Yeah, I've seen that happen as well. I could never reproduce it deterministically. It happened when using [move-panes](https://atom.io/packages/move-panes) in conjunction with nuclide (It would throw an exceptions while editors would loose their decoration and live inside the tab of another editor, like you could use shortcut to focus the different editors but their would share the same tab. Really weird.). I now switched to [pane-move-plus](https://atom.io/packages/pane-move-plus). Works so far.

The speed is really great. I remember a drop some time ago, but now it's perfect again. :clap:

Sry for not replying to your message, I was involved in lots of stuff lately. Also I don't use SQL so I wouldn't be able to test it sadly :disappointed:
Definitely looks slick and useful.


I see https://github.com/ably/ably-ios/blob/909c77f7d818fddce15646c12e647d21b0dfa257/ablySpec/TestUtilities.swift#L562-L569 does indeed change the connection key.  Please can we change that func name therefore to something like `simulateLostConnectionAndState` so that it's clear it's not just a lost connection which is resumable by default


What happens if the path that I provided is not correct ? I guess it will still go ahead and call the api. But this time with the path as the value to the options[:icon].  Ideally if the file does not exists then the method should raise an exception.


The supposed rule is to show the red dot unless both backup AND 2FA/email verified is complete.
return isTime && hasBalance && (!didBackup || !(has2FA || verifiedEmail));


So here is a working version https://play.rust-lang.org/?gist=92a767771c4fd75a7a51&version=stable

The error makes sense as ```&[...]``` is actually pointing to memory on the stack. The borrow checker needs to be sure that it out-lives the hash-map and it's defensive in this regard.


My bad! Thanks for "catching" this :+1:


This a test.


This checkin does not work.  Whilst it builds OK it throws JS errors on the Product Attributes page.  "ConditionString is not defined"


Son support de cours et maintenant supprimé de l'interweb. Merci de m'avoir rappelé de l'enlever après avoir effectué le transfert.


@JaviMerino I completely agree and this was the underlying problem in the `handle_duplicate_indices` in atleast some of the cases where data does not have duplicate indices by default


Das sieht besser aus im Logo :+1:


@loliee

:mount_fuji: :runner: :sparkles:  :sparkles: :sparkles:


Librou


Actually, the first error that led me to add this code was due to genuine duplication in indexing in a trace file. IMHO, We should let this check stay. This wont get called ever, only in the rare circumstances when the traces does have duplicate timestamps, especially in systems which have timer inaccuracies


what's that?
bindable, inject, computedFrom need their dependencies

generally might be good to align package.json with some newer plugins like aurelia-api


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


s


comment while editing


Peter -- Thanks for generalizing this fix. That's working great for my problem case.


Yup makes sense to me.


Alberto Jimenez Vazquez y Jose Antonio Perez Beltran.


in MSYS, when you write for example

ls /c/foo/bar.exe

there is an automatic translation of /c/foo/bar.exe to c:/foo/bar.exe

but no translation with ../foo/bar.exe or ./foo/bar.exe.

Is \foo\bar.exe a relative path in the console ?

I will check your example above, but `_exm_file_path_is_absolute("\\foo\\bar.exe")` should return false as it does not begin with a letter (see code of `_exm_file_path_is_absolute()`)


I would make sure that the path is resolved relative to the directory of the file. Right now, the path is resolved relatively to a the current working directory, which might be a different one.


> @Mumfrey Last I heard, we aren't adding everything and their uncle to Sponge, this counts as one of those in my opinion.

See was that so hard? A sensible rationale. Maybe next time include it in your comment to add context rather than just barking unsolicited orders at people.


Sorry, I'm not used to GitHub.


The source of the picture can be found in this link:

https://www.websequencediagrams.com/?lz=dGl0bGUgSm9iIGZsb3cKCnBhcnRpY2lwYW50IFVzZXIgSW50ZXJmYWNlAA4NV29ya2xvYWQgTWFuYWdlbWVudCBTeXN0ZW0ANQ1Db21wdXRlIEVsAB4FADIRZXIgTm9kAE0OU3RvcmFnACoKCm5vdGUgb3ZlcgCBAg86IFN1bWJpdCBqb2IKAIEeDiAtPgCBCBs6AIFrBSsgSW5wdXQgc2FuZGJveABcCwAeHFNlbGVjdCBiZXN0IENFCgCBbhogLT4AgW8QAFggAB4RU2NoZWR1bGUAgV0FAII9DwCBWwgAgjwHAIEwJAAiCVN0YXIAgjEGAIJeDwBEEVJlYWQgZmlsZXMgADUYRXhlY3V0ZSB0YXNrAIILBQCDTgcgLT4AgzwQOiBXcml0ZQAyH0ZpbmlzaACDUQUAQA8Agj8RT3V0AIM3DACCDBcAg2ocbG9nICsAOBAAhFgaQ29sAIN7BW8AdAUAg2AfAIUfEACBHw8&s=roundgreen


Seems good ! Rebuild oscillo zip and no crash anymore.


Mantappp


@DaleMcGrew @nf071590 @lisamcho I don't agree with having to leave commented code in the code base. Unless JSDocs are being used for comments, everything else should be removed. Having version control, you can always look at the history of that file and pull that code right back in when it is time. This is my personal opinion but the teams decision.


as you can see, nothing changed, because they were all the same, cuz I made something wrong on Git so I was forced to push this no-change commit :warning:


You are right, so instead of clean i'm adding errors  -1 :(


first try


We don't use Hibernate HBM2DDL for the database, we use Liquibase as it supports migration. In the default server config the database is initialized/updated. See http://keycloak.github.io/docs/userguide/keycloak-server/html/server-installation.html#d4e123 for details, it's the 'databaseSchema' properties that configures this.

We also don't set Hibernate properties directly in persistence.xml.


user.id would be more accurate I guess


I got rubocop failures when using `block.call`. Hence the `yield`


Yay!


We lost the `puts msg if $verbose`


I'm happy to include the logging changes you have added, but they should be debug, not warn.


Right, I also wanted to ask because I don't quite understand why the issue I opened was closed with no (for me) clear indication that a fix will be merged to master ?


I would prefer 'redirects to main page for new users' or 'redirects to main page for anonymous users' given that _nobody_ user is a User instance in OBS


I'd suggest adding a debug log statement and leaving the message in the exception as is.


We shouldn't display the message if `tries.zero?`


Any chance this fix could be back-ported to 1.8 LTS?


We don't use Hibernate for creating the schema, so this would have to be added to Liquibase changesets.


if you want to go this route, it might be better to use Ruby's `||=` notation so as to ensure `git rev-parse --show-toplevel` only gets evaluated once. This way you can also get rid of the `defined?` check.

```
require 'benchmark'

class Foo
  attr_accessor :top_level
end
foo = Foo.new

Benchmark.measure do
  10000.times do
    foo.top_level = `git rev-parse --show-toplevel`
  end
end
# => #<Benchmark::Tms:0x007fbaa22b22f8 @label="", @real=36.337381, @cstime=9.29, @cutime=17.47, @stime=3.82, @utime=0.7499999999999999, @total=31.33>

Benchmark.measure do
  10000.times do
    foo.top_level ||= `git rev-parse --show-toplevel`
  end
end
# => #<Benchmark::Tms:0x007fbaa24371a0 @label="", @real=0.000817, @cstime=0.0, @cutime=0.0, @stime=0.0, @utime=0.0, @total=0.0>
```


This is already fixed in 1.9.0.Final/master


**Nitpick - Code Quality**
Since we have a subject, you can have something like
```
it { is_expected.to respond_to :call }
```

You can check this out for more information on how to write better specs http://betterspecs.org/#subject


Hi lubna

I was using bitnami redmine installer for easy setup. Do you know download
link of bitnami's 3.0.2 ?
Could you provide additional information such as ruby version and platform
win or mac ?

thanks,
hhokawa
2016/02/25 17:47 "lubnan" <notifications@github.com>:

> Hi hhokawa777,
>
> I installed it on redmine 3.0.2.stable and its not working! but on the
> 3.0.3.stable its work well, What I can do to be run on our current version?
>
> thank you,
> best,
> Lubna
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/team888/redmine-assign_current_date-plugin/commit/3ce52d587b47d1e2fa5978479bedebb44cc86ad5#commitcomment-16319879>
> .
>


I still have a semantic problem : why a "compare" function would modify its arguments ? As I told you from the start, it would need to be pretty well documented to be understandable (by dummies like me...)

Why not call the the function "extended_method_func" ?
```
evhttp_set_extended_method_func()
evhttp_connection_set_extended_method_func()
```


With this change on an older codebase (SLE11SP4), the build fails:
unresolvable: nothing provides pkgconfig(glib-2.0)
I wonder what is the reason and if it can be fixed.


I actually came to the same conclusion after a while. This makes perfect sense.


same here. Given that we might not even retry, we shouldn't even login again.


OK, the 'logs' and 'toString' commits are rather my debug commits - I did not intend to push these to upstream.


Please, explain for what '.present?' is here for?


sounds good


Great, I should have rebased on Final and test with it (and check if the issues persist there).


Fix bug if $_SERVER["CONTENT_TYPE"] doesn't exist


merge


Sketch*


Wrong test case was accidentally removed. Fixed in https://github.com/yiisoft/yii2/commit/4dc771d0a0f092dc56d8dd002329fb0ed1022243


Please, what does "Cannot use the portable version of AssemblyLister in an end-program. Please add a reference to AssemblyLister via NuGet to your final Application." mean? It's not clear what are we supposed to do. There is no AssemblyLister NuGet in NuGet Package Manager.


Whoops indeed :grimacing:

Thanks, your solution to the query works. I had something nearly like that, but I see where I went wrong now.


OK, good to know. I'll check Liquibase, then. I was surprised by the lack of indexes when I've just connected KC to DB, and I spent 15 minutes waiting for realm remove (that realm had 50k users).


ok


Duplicates of index in a trace file shouldn't be a problem.  Pandas only barfs when you try to reindex, and we don't do that here.

I'll drop this patch and send a v2, but I'm almost sure we don't need the code.


cool


thanks


>  but _exm_file_path_is_absolute("\\foo\\bar.exe") should return false as it does not begin with
> a letter (see code of _exm_file_path_is_absolute())

I know, that's why I think that is a bug. How can a `\foo\bar.exe` *not* be absolute? Disregard the current drive.


Thanks for looking into that. I wasn't completely mistaken then, I thought perhaps I was missing some obvious array-to-slice-with ownership conversion, such as [1,2,3].to_slice_I_can_store_forever() or something I don't know to look for.


This can't be an interface since we need to be able to do:

```java
DocumentRevison rev = new DocumentRevision();
// ....
ds.createDocumentFromRevision(rev);
// ....
```


indent?


:thumbsup:
I was thinking "status of the database", but I agree that's quite confusing when there's a table called status.


whats this change about


**Nitpick - Code Quality**
`hash` is not an appropriate variable name. It does not tell us what it holds.


Hi hhokawa,
I restart the services 2 times then its work with me and waiting,

Thank you for your fast reply :)

Lubna


Ok


Gorgeous tileset.


Initial commit


Okay so this fixes the issue with blocking the ui, but now we will run into the following issue:

When initialising the ui we set the `current_tab = system-wide render metric`, but then in `redraw-ui` we try to initialise the GL tabs, so `pending_tab = GL render metric`, so we then do `current_tab->leave()`. But at this point `gputop_current_perf_stream == NULL`  so we end up not closing `current_tab`, so we stay in the system-wide view, and then when switching tabs we end up jumping across tabs since the switch will be relative to `pending_tab` even though we are still in `current_tab`. Fecking feck.


I would prefer if this was using DNS SRV records. Any organization that has this need should be sufficiently in control of their DNS servers that they should be able to create DNS SRV records. This kind of ad-hoc API implemented in this module is not something that I think should be encouraged


The staging branch is periodically merged to master.


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


Both matches.f.txt and matches.e.txt can be use for Incremental or Global pipeline.
Each pipeline expect just a given existing file:
- IncrementalSfM expect matches.f.txt to be present in the match dir
- GlobalSfM expect matches.e.txt to be present in the match dir
So you can rename the file and launch the global pipeline if you used -g f on mainComputeMatches.

BTW, I recommend to use -g f for Incremental and -g e for GlobalSfM.


**Nitpick - Code Functionality**
There were no tests written for the `with_value` method.


Thank you!
This resolve bug from my camera, day/night


Reverted this for now


I understand your point.
Will try the singleton implementation.


Not 100% but I think this one fixes a problem with html not being correctly displayed inside the description of the color field


I wrote a software rendering API in C++ last year for Computer Graphics Course. (answer branch)
I wish I can continue if I have time and STM32F429I (I lent to somebody else).....
Maybe I can update hackpad at this moment.


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/613/testReport/) for this commit**


es otro comentario


以下のif行を
@user.assign_attributes(user_params)
@user.assign_attributes(latitude: lat, longitude: lot)
if @user.update
にする


You can of course clone something if that is what you want. Or if you want them to persist over a function boundary the need to be stored on the heap (unless returned by value from the function)


You're absolutely right: fixed in https://github.com/mudge/re2/commit/8467456cfc983fa5955cdb196556a6d3a088e124.


ERROR: 'cast' is not followed by whitespace.


ERROR: 'sessions' hides a field.


ERROR: 'tabPos' hides a field.


ERROR: Missing a Javadoc comment.


ERROR: Name 'SEARCH' must match pattern '^[a-z]([_a-zA-Z0-9]+)*$'.


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/614/testReport/) for this commit**


ERROR: [PMD-CPD] 65 lines duplicated.
app/src/main/java/org/fossasia/openevent/activities/SessionDetailActivity.java


ERROR: [PMD-CPD] 15 lines duplicated.
app/src/main/java/org/fossasia/openevent/activities/SessionDetailActivity.java


WARNING: [SimpleDateFormat] To get local formatting use `getDateInstance()`, `getDateTimeInstance()`, or `getTimeInstance()`, or use `new SimpleDateFormat(String template, Locale locale)` with for example `Locale.US` for ASCII dates.
 Almost all callers should use `getDateInstance()`, `getDateTimeInstance()`, or `getTimeInstance()` to get a ready-made instance of SimpleDateFormat suitable for the user's locale. The main reason you'd create an instance this class directly is because you need to format/parse a specific machine-readable format, in which case you almost certainly want to explicitly ask for US to ensure that you get ASCII digits (rather than, say, Arabic digits).

Therefore, you should either use the form of the SimpleDateFormat constructor where you pass in an explicit locale, such as Locale.US, or use one of the get instance methods, or suppress this error if really know what you are doing.


WARNING: [UnusedAttribute] Attribute `fontFamily` is only used in API level 16 and higher (current min is 15)
 This check finds attributes set in XML files that were introduced in a version newer than the oldest version targeted by your application (with the `minSdkVersion` attribute).

This is not an error; the application will simply ignore the attribute. However, if the attribute is important to the appearance of functionality of your application, you should consider finding an alternative way to achieve the same result with only available attributes, and then you can optionally create a copy of the layout in a layout-vNN folder which will be used on API NN or higher where you can take advantage of the newer attribute.

Note: This check does not only apply to attributes. For example, some tags can be unused too, such as the new `<tag>` element in layouts introduced in API 21.


Watch for tabs - use spaces!


FATAL: [MissingTranslation] "`menu_schedule`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`filter_sessions`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`select_all`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`select_none`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`add_to_calendar`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


WARNING: [LogNotTimber] Using 'Log' instead of 'Timber'
 Since Timber is included in the project, it is likely that calls to Log should instead be going to Timber.


WARNING: [Overdraw] Possible overdraw: Root element paints background `@color/more_background` with a theme that also paints a background (inferred theme is `@style/AppTheme`)
 If you set a background drawable on a root view, then you should use a custom theme where the theme background is null. Otherwise, the theme background will be painted first, only to have your custom background completely cover it; this is called "overdraw".

NOTE: This detector relies on figuring out which layouts are associated with which activities based on scanning the Java code, and it's currently doing that using an inexact pattern matching algorithm. Therefore, it can incorrectly conclude which activity the layout is associated with and then wrongly complain that a background-theme is hidden.

If you want your custom background on multiple pages, then you should consider making a custom theme with your custom background and just using that theme instead of a root element background.

Of course it's possible that your custom drawable is translucent and you want it to be mixed with the background. However, you will get better performance if you pre-mix the background with your drawable and use that resulting image or color as a custom theme background instead.


WARNING: [Overdraw] Possible overdraw: Root element paints background `@android:color/white` with a theme that also paints a background (inferred theme is `@style/AppTheme`)
 If you set a background drawable on a root view, then you should use a custom theme where the theme background is null. Otherwise, the theme background will be painted first, only to have your custom background completely cover it; this is called "overdraw".

NOTE: This detector relies on figuring out which layouts are associated with which activities based on scanning the Java code, and it's currently doing that using an inexact pattern matching algorithm. Therefore, it can incorrectly conclude which activity the layout is associated with and then wrongly complain that a background-theme is hidden.

If you want your custom background on multiple pages, then you should consider making a custom theme with your custom background and just using that theme instead of a root element background.

Of course it's possible that your custom drawable is translucent and you want it to be mixed with the background. However, you will get better performance if you pre-mix the background with your drawable and use that resulting image or color as a custom theme background instead.


WARNING: [Overdraw] Possible overdraw: Root element paints background `@color/title_picker_bg` with a theme that also paints a background (inferred theme is `@style/AppTheme`)
 If you set a background drawable on a root view, then you should use a custom theme where the theme background is null. Otherwise, the theme background will be painted first, only to have your custom background completely cover it; this is called "overdraw".

NOTE: This detector relies on figuring out which layouts are associated with which activities based on scanning the Java code, and it's currently doing that using an inexact pattern matching algorithm. Therefore, it can incorrectly conclude which activity the layout is associated with and then wrongly complain that a background-theme is hidden.

If you want your custom background on multiple pages, then you should consider making a custom theme with your custom background and just using that theme instead of a root element background.

Of course it's possible that your custom drawable is translucent and you want it to be mixed with the background. However, you will get better performance if you pre-mix the background with your drawable and use that resulting image or color as a custom theme background instead.


WARNING: [UnusedResources] The resource `R.color.item_session_divider` appears to be unused
 Unused resources make applications larger and slow down builds.


WARNING: [AlwaysShowAction] Prefer "`ifRoom`" instead of "`always`"
 Using `showAsAction="always"` in menu XML, or `MenuItem.SHOW_AS_ACTION_ALWAYS` in Java code is usually a deviation from the user interface style guide.Use `ifRoom` or the corresponding `MenuItem.SHOW_AS_ACTION_IF_ROOM` instead.

If `always` is used sparingly there are usually no problems and behavior is roughly equivalent to `ifRoom` but with preference over other `ifRoom` items. Using it more than twice in the same menu is a bad idea.

This check looks for menu XML files that contain more than two `always` actions, or some `always` actions and no `ifRoom` actions. In Java code, it looks for projects that contain references to `MenuItem.SHOW_AS_ACTION_ALWAYS` and no references to `MenuItem.SHOW_AS_ACTION_IF_ROOM`.


WARNING: [ContentDescription] [Accessibility] Missing `contentDescription` attribute on image
 Non-textual widgets like ImageViews and ImageButtons should use the `contentDescription` attribute to specify a textual description of the widget such that screen readers and other accessibility tools can adequately describe the user interface.

Note that elements in application screens that are purely decorative and do not provide any content or enable a user action should not have accessibility content descriptions. In this case, just suppress the lint warning with a tools:ignore="ContentDescription" attribute.

Note that for text fields, you should not set both the `hint` and the `contentDescription` attributes since the hint will never be shown. Just set the `hint`. See http://developer.android.com/guide/topics/ui/accessibility/checklist.html#special-cases.


je pensais à deux dimensions, pour être à l'aise avec la syntaxe, pourquoi à une seule dimension ?
(Bonne idée les obstacles sinon)


We might need some reindexing in the BART code where we do manipulations with the indexing. It could potentially be a problem there


1) The normal will also be in local FoR, which is what I want. Right?
2) Thanks, I commented the WRF position calc out then.


very nice. it looks awesome.


Ok, I see. In this case, I don't need anything to outlive the function, I just expected to be able to do a hastable.insert with a array literal, that's all ;)


Why Gender.Male and Gender.Female are private?

this static constructor made me crazy when EnsureGenders in clean database caused to Male gender appears twice:

![image](https://cloud.githubusercontent.com/assets/15857369/13317479/62065360-dbc6-11e5-9c19-e5a50d75debf.png)

Maybe it would be better to make constructor private as well? to avoid possible Gender object creation...


Would be neat if we could test those per action.


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/615/testReport/) for this commit**


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17201](https://teamcity.mesosphere.io/viewLog.html?buildId=145772&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:02:20


the location is based on Facebook Graph search
https://graph.facebook.com/search
you will need graph access token
your enter the place to seacrh

this is my .net Code for this search

var reqparm = new NameValueCollection
            {
                {"q", SearchtxtPlace.Text},
                {"type", "place"},
                {"max_timestamp", DateTimeToUnixTimestamp(DateTime.Now.Subtract(new TimeSpan(0, 0, 1)))},
                {"access_token", GraphAccessToken},
                {"expires_in", AccessToken},
                //{"callback", "5184000"}
            };

            var responsebody = InstageamWebRequest.DoGET(GraphUrl, reqparm);

the response you get s the id of the place


Because is html compatible. Il we don't bundle the content we can use a js to load te content and for link tag when is not a style we use alternate.
No.


Still getting similar issue when running "Git status"
Error detected while processing function <SNR>44_Git:
line   11:
E499: Empty file name for '%' or '#', only works with ":p:h":     tabedit %


@m-stein: What do you think about this simplifcation? We tested it on Muen and all Genode scenarios work as before.


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/616/testReport/) for this commit**


Yeah get on it.


OK, let's leave the 'logs' and 'toString' commits then. We're at a stage now that we don't want to introduce non-required changes.


oh, working games so soon... good job :smile: , is the api good so far?


it is an idea but "client_templates" is not clearer, but for me it's <kifkif>


Great!


Auch.. How much did the indexes improve things?


Yeah but in this case it inserts a ```borrowed reference``` to array literals which means there has to be another owner and the owner needs to out live the hash table.


Unfortunately no. New features are never backpatched to maintenance releases.


it's a xml content...


Shouldn't that also be in the "describe 'GET show'" block?  I think in this case I would handle set_group before_filter like a private method (=something that get tested as part of show action).


Do we still need South?


It takes time to get used to this and I had/have my fair share of fights with the borrow checker but in general it turns out that code becomes better when I restructure it so it works as it should.


Start


Can we lose this logger?


with the shoulda matchers unfortunately not possible, I tried it!


Stupid think... because in this case you must re-render each time, and parse and translate to don't have a lot of attachment... The content is splitted into separate js module.


Yep, I got to that when I saw the DocumentTranslationBundleFactory that unregisters the bundle from the component manager when the document is deleted. Sorry for the noise.


Prepatch, i have plans to move this part to src/server/scripts/PassiveAnticheat when making PR :)


AppVeyor is cool!

But it seems that they has only Windows Server 2012 machine, which is ok for building, but for running tests it will be better to use npcap-driver on various systems.
Unfortunatelly, I didn't found anything with various Windows version (although I was googling only couple of minutes).


Sure, but that's handled in `stats/grammar.py`.

Note that I'm not getting rid of `handle_duplicate_lines()`, I'm just removing its call in `Constraints.py`


good idea :)


Will this fail the build or just log a warning?


You're welcome! Replay is amazing! Thank you!!


`LinuxCapabilitiesSet` is a interface, so think that we can use: `if _, ok := val.(types.LinuxCapabilitiesSet); ok`  ?
`Set` method from interface returns `LinuxCapability` slice, so I'm not sure that, we have to cast to string or maybe good idea is to use caps of `LinuxCapability` instead of `strings`


I'm Blue dabedidabedei


Woop!


Ты не все добавил в вроект, что указано в задании

> Ради учебного эффекта, добавим в проект (не забыть про bower.json):
>
> underscore конкретной версии 1.8.3
> backbone самую свежую версию, но с фиксированной мажорной 1
> react версией большей чем 1.1.0
> angular любую версию


So couldn't the hashtable take ownership instead, if I had declared it differently?

I agree that in general the borrow checker is beneficial, and makes you think twice about why that wouldn't work, and you often realize you made a mistake. But sometimes it's overly conservative. That'll improve over time, I'm sure.


Makes sense, but when the user opens the wallet it's not immediately known if 2FA / email verification is complete. So you have to make sure it doesn't flash in the split second (or longer) while it's still loading that.

That doesn't seem to be an issue right now.


is better use `LinuxCapability` type ?


hihi


It can take ownership if you send in the data by value instead. now the hash map takes &[u32] which is borrowed data.


safasf


false


sdfasf


no


ok


GCC/MingGW do, but some odd compilers not. I don't know what compiler will be tested next :D


`Binary | String`


DuplicateSoundMemを削除した理由は何でしょうか？


僕としてはDxLibの関数名も残しておきたいのです。


So we basically have two ways of making a decision wether we want SSL for a container or not, right? We have the labels on the container and we add another ENVIRONMENT variable to say wether we want to you or not use the label?!


Hi dukesrg.. Quite some time not seeing you and b1l1s.
````
CakeFW and Decrypt9 have the Multi-EmuNAND feature, i dunno if rxTools should have that as well.
I personally use Decrypt9 for decrytion, making rxTools a solely CFW launcher and patcher for me.
Maybe I should create multi-emunand for J/E/U regions and just use CakeFW instead..
````
~~Recreating similar things is a waste for elite. Decrypt9 does quite better now. Are these too much?~~


sorry.


I've fixed this now.


getCircuitData (in get_and_set.html) is not working any more. I guess it has something to do with mixing up $workspace and $workArea...


i'm not sure what happens if this.repository.count() returns 0 (which is a perfectly fine count)


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


pull request is made.


Thanks for recommending it to me!

However, I don't know how to import my code signing key to the CI. (also I don't know how to upload a file to the CI server? or importing a key to the CI Windows machine. On my laptop, it needs GUI operation with Cert Manager). So the code signing process is not finished on AppVeyor.

I think I will still sign the files and test the driver in my local machine for now.


`key` should be `Binary`.


Testing the authorization before_filter and policy is something for the test suite. Testing that it is used should happen in the running code and not in the test suite IMHO.


Jorge Ugarte Macías


Hmm. Boxing the value seems to work;

    let mut offset_cache = HashMap::<u32, Box<[u32]>>::new();
    offset_cache.insert(MASK_OUT_X, Box::new([0, 512, 1024, 1536, 2048, 2560, 3072, 3584]));

I tried that, without the box, but that wouldn't compile because [u32] isn't sized.


@vilchik-elena We have no test for that.


Francisco Javier Santos Rodrigo


Populate metadata branches and script PR have now been merged


sounds right :-)


You don't need to heap allocate it.

https://play.rust-lang.org/?gist=3476253bb6262abd8c54&version=stable


I believe both "hidden" and "hid" are correct, the first is the perfect past tense and the latter is imperfect tense. "adding" and "removing" are regular verbs and so are the same in both tenses. (had to look this up) So it comes down to a choice of what tense to use in the logs. Maybe the imperfect tense is best, as "hidden" can also be an adjective -- which may be confusing.

Thinking about this more, maybe "unhide" isn't really a verb and we should replace it with something like "display". So my new proposal would be to use: "added", "removed", "hid", "displayed"


Start


I think I would combine the changes in this commit into the [earlier commit][1] i.e. re-write the original commit as if you knew about the changes which caused this merge conflict.

I realise that having this commit as a separate commit is a more accurate record of what happened. However, I don't think that will be of much interest to people looking back at these changes in the future. And I think it would be easier to follow with the two commits combined.

Does that make sense?

[1]: 4eaaad641465787b1890b2d0c636e0bce693dbfa


If the former - I lean towards this being more annoying than it's worth. Unused imports may be untidy but they're not an error.


now it's on server, you made a blind typo.


Im blue
I belive im an applepie
Am 25.02.2016 11:56 schrieb "NoRelect" <notifications@github.com>:

> I'm Blue dabedidabedei
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NoRelect/SneakySnake/commit/b4704e93228a9883e4ef76f2f419df782ee74143#commitcomment-16322733>
> .
>


I guess openssl being broken is pretty important though ?


indeed; PR? I'm in the middle of something else and can't get to this right now.


J


hh


As I always do... I am blind... Poor cookies LOL


oh it's from here.


I tried that early on, but unfortunately that only works if you ever insert arrays of the same length into the hashtable, as the inferred type is actually [_; 8] or possibly [u32; 8]


IMO the most beautiful way would be have a generalized fetchAllCombine() and have it return an array with the selected column as key and an array of all other columns as value.


@vilchik-elena We have no test for the first part of this condition.


Are these whitespace changes expected?

In any case, assuming you combine the merge conflict commit into the earlier commit, I'd also prefer to see this commit combined into the [earlier commit][1] which updates the regression test artefacts.

[1]: d16c454b8bf390f1ae835d1d71cbb66303868772


Thanks...


Also @thebigsmileXD I reported it to you on ClearSky.


yeah, it assumes that the length is the same. I think in this case it's better to just please the borrow checker and have the data outside where the hashtable is being used and do a borrowed ref, that way one doesn't need to resort to allocations.


Argh, didn't thought about that :/ @rsudev what do you prefer?


proprietary/system/lib/libnetutils.so   is missing


@philolo1 Sorry I didn't actually see the whole diff. I forgot about the media query. :)

I will merge this now.


@Syphurith I looked Decrypt9 code a bit... Looks like based on the original Roxas rx <2.6.
When this fork will be finished, new functions could be added as easy as importing a single function into a sources. So yes, I hope Decrypt9/EmuNand9 will be obsolete and absorbed by rxTools.


Not really. Almost all packages use the version that is not broken.


Yep, we'll leave it as is.

     // TODO: Revisit all code when I'm a certified Borrow Fu Master


I suspect rather than `value:Id` we may want something else, like `QuotedTerminal` or `StringLiteral`, but it depends how flexible we want to be. Rust is rather restrictive here and requires a string literal. I've been wanting to change that forever though =)


this echo command needs to be run as root. (not recommended to try it with sudo, the redirection is tricky then)


Why this engine doesn't use classic long option notation like --project-manager?


After looking into it I think we should use IOException instead of FileNotFoundException, as the file might exist but something else could have happenned to the reading of it


@vilchik-elena Why does this method name start with "check"?


trough -> through


Not sure if you can really change the location without breaking stuff like kernel extension.


Boop


Hey ya!

It looks like your code is older than that on the main repository because it does a typeof Cookies !== "undefined" check now. I've also just updated the package to use "use strict"; inside it's closure so I believe your changes are no longer required... could you pull from the main repo and check?

Thanks!


This looks very impressive, thanks!


We should add 'Test/**/*.js', 'Tests/**/*.js' for the case-sensitive os.


Hello!  Is it possible that this NPE would also occur in the getComponentsProvidersInfo() method?  I'm at master and still getting a NPE.  (It might be that this is a temporary problem until the new nightly build.)


/r-


Typo


Hmmm...


hey, over in IRC I agreed to submit a PR on behalf of Kolifanes to main kodi repo, these chnages included some older version of your code.

The PR I submitted is in https://github.com/xbmc/repo-plugins/pull/367

Im not a developer and was just doing this Kolifanes person a favor submitting the changes.


Sorry, it is not needed =)


A une dimension, c'est plus simple à coder au niveau du constructeur, mais
c'est sûr que c'est moins intuitif au moment de l'accès aux cases ^^ en
deux dimensions ça me convient aussi, par contre ça doit jouer au niveau du
temps de calcul ;)

Le 25 février 2016 à 11:48, Persalteas <notifications@github.com> a écrit :

> je pensais à deux dimensions, pour être à l'aise avec la syntaxe, pourquoi
> à une seule dimension ?
> (Bonne idée les obstacles sinon)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Acathignol/projet_3BIM/commit/86159721d1c972fd6bcdc0d1fa272de6b9752618#commitcomment-16322516>
> .
>


part of #2271


@vcunat i need to use nodejs-5_x which uses the broken one :smile: I don't actually understand how the version of openssl to be used is determined per package ? I am wondering if I should merge this fix to my branch or instead overwrite nodejs-5_x to somehow use the non-broken version


can't see anything wrong, same error, you seem to not update the code...


My comment was just an example, how to find an application on OS X and how to get it's version.

I'm not sure, if you can move Fusion without breaking something, but I'm sure you can move Wireshark.


nice. But I would opt for `applyMiddleware(thunk, routing)` which appear shorter to me.


This fixes the crash however it introduces another but.

* Open a term in ws 5
* Send to scratch
* Bring scratch back
* Switch to ws 1
* Bring scratch back

Notice how ws 5 is still shown in swaybar


`swf_lite`? why are both included here?


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


This statement can be removed without changing the validity of the test.


effectively, good


Thanks! It fixed @1419b049bd684b0cae22c23284404fe23bd7a026


this will not work server-side, so we need to pass history instance to `createStore`, like it was before (https://github.com/SIGSEV/minus/commit/9942c277ec6c85b252ca49019ab3c163358796e4#diff-01b6e016dc33dc6cf4466e7abd376200L8)


you have mako hotplug set here x2.


@vilchik-elena Maybe a ListMultimap would be easier to use here.


Travis has the same error...


no... they are test for this and no bug with size optimization


Will it be better to set the path as /dsn/idl/thrift_helper.h?
Since there will be some other idl helpers


We already have the means of controlling XML ingester input at content dispatching phase where we put contents into baskets based on `mime-type` (and in future by ObjectStore's `interpretation`, see #193).

Currently our agreement is that all contents marked with `xml` mime-type are JATS records. When it changes we will have to either supplement `mime-type` with more detailed information or additionally use ObjectStore `interpretation` field in order to be able to perform more fine grained dispatching.


Realised there's absolutely no point on clicking on null values - as the links have no href, no page will be loaded doh!


I think it went to about 10 seconds, but that was just on my laptop, so server perf may vary. Though, indexes matter a lot. I haven't gone through the tables one-by-one, finding out which indexes are really necessary, just looked up all userIds (it seemed that lookups by userId are often). I could revert the DB and try to go through these, and measure the times.


@ImagicalMine/developers any ideas?


@jtkech @sfmskywalker Sounds like a good compromise to me.


Done in 0cd5bb8691b08b75034533b2c05e0e8c62309ad3.


This is a workaround for https://github.com/NixOS/nixpkgs/pull/13445


raw heuristic calculation for min mem required


Yes, the translate is include into the template, and the xml are translated server side to have less issue and different behavior


Yes


Why not create a issue, so this todo does not get lost?


The STM32 support both arithmetic and logical shift
It should make sense that the shift type is determined by the type of data (signed/unsigned)
Anyway it is better to test this stuff, as there is no real standard as what happens to signed values on bit shifting.

Also, the operator >>= may result a bit of optimization here (if its supported!)


So, an `Exception` is a “serious error” while an `Error` is a “normal error”?


ありがとうございます。
直します。
SendCtrlCmd.h で実際に使用されているものは数えるほどしかないですね‥。


yuuyuytu


`ArtifactUtils.convertArtifactToPath(Artifact artifact)`, see [here](https://github.com/carlspring/maven-commons/blob/master/src/main/java/org/carlspring/maven/commons/util/ArtifactUtils.java#L196).


@varund7726  no need of updating this now, we're tracking CMs bionic


Should this also remove the import flag from function definitions.

Might also note that this section must be preceded by a Signatures section.


test purpose


EDIT working fine


oops. Thanks, I'll fix.


Right, but for the backward compatibility ? (if a custom module is not converted ?)


How about check user test allow with maximum user test per contest?


Yes it does..


Nevermind, it's just my thought.
Allow user test boolean is also great. :D


gonna check that env variable now...


Bad


Nooooo! It's empty!!


Could not analyse class: "In2code\Powermail\ViewHelpers\Condition\BackendContainerViewHelper" maybe not loaded or no autoloader?
Received this message while accessing powermail module in backend from TYPO3 7.6.2 onwards !
Is that some kinda config type error ??


lol


yes, we use some record ir.ui.view as template js, but may be we can use the controller "web/content/xmlid" instead of just the xmlid


updated


pcmd also contains command-specific arguments.


I took the liberty to update it.


Since the files `lib/checker/checker.jar` and `lib/checker/javac.jar` are no longer used, they should be removed from the repository.


Well
Am 25.02.2016 12:33 schrieb "NoRelect" <notifications@github.com>:

> Bad
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NoRelect/SneakySnake/commit/a936ad988b3365595c5db9e08130e25d28d11c27#commitcomment-16323540>
> .
>


I think this is more readable.
if (loadedPlaylistElement != null) {
    if (xLoadedPlaylist.HasNext ())) {
    } else {
   }
} else {
}


HAHAHAHAHAH


Sry I dont understand your question.
This is a systemd unit. It's used to start Spectrum 2.
There's nothing which needs to be implmented or added to Spectrum 2.

Maybe you want to send a pull requests to Spectrum 2 devs to include the unit file directly into their packages.


LGTM


Why?
User are custom with doubleclicking on stuff to make it run


I shared for this change, because it would be more compact effect but to work is easier with the header (which does not take size)


There's not re-rendering, there is a cache (that won't work because the invalidation is not handled correctly in this commit). You have to respect the ir_qweb api and behavior.


are you trying to put a file in each directory?  This may not do that unless you give the path?  Just put that file in the current directory.


In fact it can not. This is fixed in 545b07dfe73243dd7e62b715c8e3e1de3e4e669e


`collections` seems fine.


Hi, thanks for the tool! Just tried on Philips PHL224E5 and it works!


Don't blame me ;)


i just don't like the condition at line 1189, it's not clear enough to my taste


Oh, and I didn’t even notice it’s a new anonymous class! Sneaky! :D


Perhaps remove SOUTH_TESTS_MIGRATE while you're at it.


i found small error ; semicolon at the end of f.braces } . but it was really good . to download and work with


variableName should follow camel case.


I copied that from some other place in the code so it must be "official" ^^


What was the reason for removing these sandboxes from Hindsight source tree?
These were good examples of useful Hindsight lua sandboxes: are they available somewhere else?


Should probably be without `@`?


i wanted to know how to run mangodb . with the help of hdfs ?


The static constructor will be removed at all.


Meglesz.


\o/


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


v3 PDF


gem update rails 4.2.5.1
bundle update


forse volevi scrivere cazzo = true?


Thank you! Never thought someone will contribute to it ;-)


TODO: Still the problem of manual editing for directory. (line 6)

TODO: change if statement with "vegetarian" to  statement which allows multiple writings.


That was intentional … dunno…


Shouldn't it be `whoami` instead of `who am i` ?


I have no issue with the return type of `websafe_decode()`. It makes perfect sense *and* it hasn't changed.

I'm conflicted on the return type of `websafe_encode()`. Both old (binary) and new (text) make sense. I'm not keen that it (on Python 2.x) changes the API, but this is a major release so *if* it's to change now is the time.


I think @xidachen hasn't created a pull request yet. @xidachen do you know how to do that? That might be good to alert everyone of this addition, not just those that somehow ended up here.


`runtime.NumCPU()` returns `int`, why cast to `int64` ?


У нас для этого паршл есть отдельный, используй его.


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


I'm out of my reko build machine now. You could fix it when you will be ready.


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


still don't understand this prepatch


Sad to lose the nice use of partial, but if we must!


This never gets called. :recycle:


I'd be happy if we can contain the fragmentation under 0.5, we may have to bump later.


toString method should follow correct builder method that I share.


The pkgconfig file in question is part of upstream since almost 16 years ago:
https://git.gnome.org/browse/glib/commit/?id=0ae274954831e15b7513a07d8c1d6d05a0bbe0d7

What does ` rpm -q --provides glib2-devel` (or equivalent package) reply on that system?


added exceptions for method to raise exceptions when files are either not supported or not found.


The invite user helper method is never called so we don't need this.


AndroidManifest.xml

<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package=" mobilecoder.net.kga " >
    android:versionCode="1"
    android:versionName="1.0" >
<!—Burada uygulamamızın kurulabilmesi için gerekli olan minSDKVersion 8 targetSdkVersion 17 olarak tanımlıyoruz-->
    <uses-sdk
        android:minSdkVersion="8"
        android:targetSdkVersion="17" />
    <!-- Kamera donanımına erişim -->
   <uses-feature android:name="android.permission.camera" />
   <uses-feature android:name="android.hardware.camera" />
   <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
   <uses-permission android:name="android.permission.RECORD_AUDIO" />
    <application
        android:allowBackup="true"
        android:icon="@mipmap/icon"
        android:label="@string/app_name"
        android:theme="@style/AppTheme" >
<!-- Activity tanımlanması-->
        <activity
            android:name="androidcameraapi.androidhive.kga.MainActivity"
            android:label="@string/app_name"
<!—Ekran klavyesini sakla-->
            android:configChanges="orientation|keyboard|keyboardHidden"
<!—Ekran yatay durumda Default-->
            android:screenOrientation="landscape">
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />
                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
        </activity>
    </application>
</manifest>


Hi, I see diffs, but yes, it's latest version of that script, nothing more, they solved Chrome bug.
![image](https://cloud.githubusercontent.com/assets/5429548/13318953/6f96cdfa-dbbe-11e5-97ce-39fb90bfc87e.png)


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


Debian/Ubuntu: ``apt-get update; apt-get source owncloud-client; apt-get build-dep owncloud-client``


I suppose this has come about because originally the tool API stuff didn't know about modules and therefore didn't have to worry about loading tools from them. When @quintesse added support for that I guess the distinction between `ToolException` (a problem with the framework itself) and `ToolError` (an error within a tool within the framework) wasn't really clear. Really using `ToolError` is a lie, because the problem here lies in the framework itself, rather than the tool. But I guess I didn't account for the need for more control over how those framework errors got reported.