Ah. That makes sense. I should remove it from my init.el :-)

Anders

On Fri, Nov 13, 2015 at 1:47 AM, Christian Schwarzgruber <
notifications@github.com> wrote:

> Hey Anders,
>
> I use the same settings, but found out that I didn't set, indent-tabs-mode
> to nil. Tabs are now replaced by spaces, on indent.
> Side note cmake-mode.el adds the file names already to auto-mode-alist,
> at least for me
>
> ;;;###autoload
> (progn
>   (add-to-list 'auto-mode-alist '("CMakeLists\\.txt\\'" . cmake-mode))
>   (add-to-list 'auto-mode-alist '("\\.cmake\\'" . cmake-mode)))
>
> Christian
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Andersbakken/rtags/commit/fc7fb958529c41060ff02b2f1da6d9b24ddb43f5#commitcomment-14377646>
> .
>


A copy of the players array for when we need it in the future too


have to fix whole html file...


It is probably not essential. If you present limits of presheafs in terms of natural transformations than you can rely on the terminal object in Set. The relationship between Kan extensions, limits, and coends can be understood in one sense in terms of weighted limits which generalize the usual conical limits. See [here](http://ncatlab.org/nlab/show/Kan+extension#PointwiseByConicalLimits) and then the section earlier on weighted limits.


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


Wait, what's this doing? `is_same_name` and `is_diff_id` are booleans, and you're just passing them to `filter`. Does this work? I thought `filter` took kwargs, not conditionals.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1430 of 1430 tests passed)

![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)


Incorporated pre-build path due to locked version bug.


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


Minor point, but it's a `403` and not a `404`. `404` is for a resource not found, `403` is for access denied.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1427 of 1427 tests passed)

![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)


I don't see how this is any different than the previous implementation in terms of lifetime/gc.  In fact, previously, the byte arrays were held by the MusicService object.  Now, the references are cleared when the thread exits so I would argue that the previous implementation would have byte arrays that outlive this one.


Reading it again, I stand corrected. It's `404` if the `id` is not found. It's `403` if the listing does not match the vendor.

Honestly, this is the lazier way of doing it (I'd have thought to get the listing with the given `id`—return `404` if unfound—and then check the vendor_id of the found listing, aborting with `403` if it doesn't match), but for MVP I suppose it's fine as is. As long as @jondubin agrees.


"No one will need more than 637 kB of memory for a personal computer."
Why limit it so strictly if there's no technological reason to do so?


I don't quite understand this type; in the paper version, I was taking the limit of something like `J : 𝔍^𝒞`. Is that wrong, or is this somehow equivalent?


Oh, just a note.. please leave the 'DISTRO_VERSION' string alone as it's replaced with a sed call with the currently buildin $DISTRO_VERSION in the main pinstall.sh


unless you're anticipating those maps being keyed by gvk instead (which, looking at it, I sort of would), I would still want to do gvk.GroupVersion().String() once instead of 3 times every time through the method. A comment in the function godoc about the anticipated signature, and a comment here about the anticipated keying would help orient me.


Why do you create another `url` variable? Just use the `URL`.


This will not work since you also have the URL as valid string.


`filter_by` takes kwargs and`filter` takes these things that aren't exactly booleans, I think they're called "criteria", and they get evaluated later on


Should mention that it uses file.getPath().endsWith(...)


Perhaps this should be static, so you don't need a new random cache for every object.


It is equivalent if you uncurry `𝒥` (so that it is a presheaf over a product rather than an exponential).


:+1: thx


Ah, ok. thanks!


Still not very familiar with `branch`.


注意矩阵乘法中使用`anstmp`的必要性
矩阵乘法A*A 之后的`A[i][j]`代表`i`走两步后能到达`j`的方法数
矩阵乘法A*A*A 之后的`A[i][j]`代表`i`走三步后能到达`j`的方法数
以此类推


The "!" is stronger than the "in" operator, so the condition was actually equivalent to
((!serialPort) in portToDevice) instead of the intended (!(serialPort in portToDevice).
Once this is fixed, the next check is no longer needed.


oh.. i thought the issues is voice on second sim ..


OK; I guess another way to know that we don't really need a terminal object is every category of presheaves is a topos, and in topoi, all finite limits exist.


ปรับให้เป้นภาษาไทย


On second thought I agree. Will remove it from the todo list


Oh hey, awesome!


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


First of all, my goodness, your code looks so much better.

But I'm not sure I want to accept this.

My issue is that if this code were to ever execute, that means you will have 1 buffer used by the audio thread, 3 filled buffers.  This is one additional buffer of latency compared with the previous version.

The goal of the previous implementation was to reduce the latency as much as possible.  This is why I tried to limit the number of buffers to 3.  And that's also the reason I implemented the really dumb "drop everything from the network" logic.  I did not want the latency to build up.  If the network was faster or the audio was slower, I wanted to be able to get to the most recent audio data as soon as possible.

I was afraid that if I don't purge the buffers, then the underlying networking layer would keep buffering audio data until the audio thread finally caught up.  I did not want that.  If I detected that the audio wasn't consuming as fast as the network, then I purposely dumped any and all queued audio data, in hopes that when the audio was ready, then it would always have access to the most recent network data.

With this, it is entirely possible the lower level networking code is buffering data and thus introducing latency that this code may not be aware of.

And note that this can happen because the two clocks on the server and client are not synchronized; they will drift and eventually somewhere latency will build up.  This code will not account for that.


@neokamikai as in 27858bd4, I don't get the error anymore. Please update your repo and confirm if the issue is still exist or not.

The data came from the list @Badarosk0 provided, I have rechecked them one by one against Divine Pride and translation project.


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


May be a bit messy in terms of with our architecture pattern... lol


It's hard to tell on here, due to the change I made in https://github.com/docker/docker/pull/17671. Since create and mount used to be separate, I was able to move the mount/dismount into create_unix. As the code currently stands, mount is only called on UNIX, so this change will require undoing the change I made in the other PR and move the mount back into the common code path. That's one of the fixes I'm looking at to make these changes compatible with Windows.


I've been thinking about this and I really think I'd be happier if we wrote this as a series of independent folds combined with the "arrow" combinators, similar to what we had before. I realize this "map-fusion" stuff is cool, but I'm 100% sure I couldn't add a new phase to it or change a dependency. I don't think we need to be worried about performance at this point; is there some other reason we should be working so hard here?


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1430 of 1430 tests passed)

![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)


爷爷，别黑我了，我都累成狗了，我比赛情况你网上都能看得到，除此之外还要照顾7个学弟学妹中期发表，自己12月还有一篇论文deadline，等忙过这一阵再帮你好么

2015-11-14 9:47 GMT+09:00 ding jibang <notifications@github.com>:

> 不错不错！
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dingjibang/GDX-RPG/commit/f276894692eda7a64ccc9d503541a58aa3a3ed55#commitcomment-14394699>
> .
>


@C-Aniruddh where did this go?


yes


@VladRassokhin, @sikachu sorry about this issue, and thank you for the fix.


爷爷我没说你，我说这个提交，我们新来个码农@Lchaos 我让他做个东西，这个是他的result，你忙你的吧爷爷


Hello @Kennwood . We decided to move towards using Phoenix and Elixir for our MVP. I'd like to have a golang branch as well though. I'm going to put some more vision related stuff in the wiki and I would love to have a conversation about these directions. Please feel free to continue as you are, knowing I'm going to put the golang in it's own branch for future pull requests. Enjoy!


TeamCity LabkeyRelease15.3 :: Git postgres 9.4 [Build 66](https://teamcity.labkey.org/viewLog.html?buildId=284930&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Execution timeout (new); tests failed: 186 (182 new), passed: 3 Build time: 04:01:10
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.DataFinderTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Он доходил до switch до того как человек жмакнет на кнопку. пофиксил это нормально в следующем комите


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (892 of 892 tests passed)

![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-25.12%25-green.svg)


Added favorite color info


You're overwriting whatever packet_id's have already been received this way. This should be an append() operation, plus you need to sort whatever list is already in there.


It is an error, but it is not related to threads.


@gbiellem look ok?  also i note here https://github.com/Particular/PlatformInstaller/commit/228051de6c07751813504967f354169bfe376cc9  we are swallowing navigates.  would the clipboard approach above be better in these cases as well?


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


"callable type" is defined in func.def; func.require is INVOKE.


I do not know how Dima will name it but "Libre" is the important part here and not if is coming from "cleaned" Debian or Fedora packages.


You shouldn't be updating host_packet_sent. This should only be updated when the the host of interest is sending a packet, not receiving one. I'm not sure what @sharonyang intended with this statistic, so you may have to update it in schedule_new_events for sending an ACK, but that at most.


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


nit: prefer instanceof()


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


@rnhmjoj The version is missing in the package name. This will break upgrades with `nix-env`.


Comments to previous changes on previous diff. I clearly don't know how to use github >.>


I'll put link comments on the rest of this synopsis as well.


I wouldn't just say "not at all" but what I can say is that Prism isn't actively developed for Bukkit/Spigot (there is some fork that is I think) and we _already_ have one setup that's sufficient for the needs of this server.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1430 of 1430 tests passed)

![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-70.48%25-green.svg)


In C++11, enums whose values fit in the range `[INT_MIN, INT_MAX]` will always have type underlying `int`, with the underlying type expanded only as needed to accommodate the enum values. Unfortunately, the underlying enum type is implementation defined in C and prior C++ versions, so I'm not sure what the correct C type should be to correctly correspond to enum. It really depends on how the external library was compiled.


nit: prefer instanceof


try ticket_compticket_path(params:params)


Agreed! However... just an FYI for whenever people file bugs on the 16.01 release because their tools break unexpectedly ;)


Hello you could help? I made this change and I'm having problems with my Windows ADB G3 Beat ... any idea about it?


I'm not convinced this is the best way to do this. Prefer instanceof(packet, RouterPacket).


listing model needs vendor relationship


You fixed this in the other diff but see comments on that one; host_packet_sent should not be called here.


It is my gdb info, and I also uploaded my build log.

(gdb) file nixnote2
Reading symbols from nixnote2...(no debugging symbols found)...done.
(gdb) r
Starting program: /var/tmp/portage/app-misc/nixnote2-9999/image/usr/bin/nixnote2
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".

Program received signal SIGSEGV, Segmentation fault.
0x00007ffff07f1d9c in ?? () from /usr/lib64/qt4/libQtTest.so.4

[build.log.txt](https://github.com/baumgarr/nixnote2/files/34542/build.log.txt)
[debug.log.txt](https://github.com/baumgarr/nixnote2/files/34543/debug.log.txt)


You need to specify dest_dev, which is just get_other_side(self.link, self) or something like that (sorry ran out of time)


Good!


These look amazing Sean, nice work!


Yeah, I originally put "Libre Pup" but then decided to follow the specs.


Sketchup2016 (macumber) - i386-Windows-7-VisualStudio-12: OK (2335 of 2370 tests passed)

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


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


What is this for?


I mean the import you added


This is useless code,Please removed it.Thanks.


Sorry, this commit was a mistake, I submit a PR: https://github.com/nodejs/new.nodejs.org/pull/358


Thanks!


Let's change the specs then :smiley_cat:


'Fix' in the commit title should be all lowercase.


I think he did it to avoid conflict with his totally deblobbed fork which he builds from. The thing is though, if someone builds it with a non free firmware it ceases to be libre, so I see his point.


Hi Mirko, I implemented your fix, but using vectors rather than array allocation, which is the preferred way of doing this sort of thing in C++.


Ok, the StatMonitor.java formatting is no longer an issue.


Hi Itamar,

Your updated code works great. Thanks a lot for your effort.

With kind regards,
Hartmut


HOLY CRA ZHUOWEI COMMITING IN AN ADDON THIS IS AN UNIQUE EVENT IN HISTORY


good


that's it... early next week I'm writing myself a commit-lint tool for myself :-/


I thought including the real libs was illegal?


It should be `moved here from https://github.com/shenwei356/bio_scripts/`


I’m not sure it would look much better. If we’re lucky, it looks like
```scala
def AllPhases(expr: Expr) =
  expr.cata(projectSortKeysƒ)  >>>
    (exp => Corecursive.anaM(
      (TableScope(Map()), exp))(
      attributeCoalgebra((scopeTablesƒ).tupled(_)).disjunction)) >>>
    cofCataM(_)((a, node: ExprF[Cofree[ExprF, Annotations]]) =>
      attributeCoelgotM(
        zipCoelgotM(synthCoEƒ, inferProvenanceƒ)).apply(a._1, node).disjunction)
```
and note, that is with the new `synthCoEƒ` val still hanging around as well.

**Disclaimer**: I just typed this into GitHub directly, so it’s probably broken.


と言うよりも、
@prototype.comments.create
ってすることで自動的にprototype_idが入るので、
わざわざ
prototype_id: create_params[:prototype_id]
と明記する必要ないかと思って。


なんか無駄に行数稼いでる気がするので、前みたいに
%span.num (#{@prototype.comments_count})
でいいんじゃないですか？


gimana ry


There has to be a better way of doing this


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1430 of 1430 tests passed)

![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)


I'm curious why you chose to make the newline a separate string rather than including it at the end of the string on the prior line. This seems odd.


Just to keep lines under 80 cols.


i am proud of this one


When programming the atmega324PA where do I add this xml file?


Should've added I did a bit of renaming here to better align with the ReadStat repo, which seems to have already gone beyond the unofficial file specification.


What about
```scala
  def projectSortKeys(expr: Expr) = expr.cata(projectSortKeysƒ)
  def scopeTables(a: (TableScope, Expr)) = (scopeTablesƒ).tupled(a).disjunction
  def addAnnotations(a: (TableScope, Expr), node: ExprF[Cofree[ExprF, Annotations]]) =
    attributeCoelgotM(
      zipCoelgotM(synthCoEƒ, inferProvenanceƒ)).apply(a._1, node).disjunction

  // NB: projectSortKeys >>> (identifySynthetics &&& (scopeTables >>> inferProvenance))
  def AllPhases(expr: Expr) =
    coelgotM[NonEmptyList[SemanticError] \/ ?](
      (TableScope(Map()), projectSortKeys(expr)))(
      addAnnotations, scopeTables)
```


Add Melissa Data Download (including the fields in additional_csv_data) (2)


@amjith What do you think of this?


That convention is for internal `*_connect` functions. I'm not sure what exactly "bind a to b" should mean, it sounds to me like b's updates should go into a? This function is part of the API so we should chose whatever is intuitively better...


wow much bytez, many performance, such optimize


> Just to keep lines under 80 cols.

In this case I would place the initial string on a new line and merge the \n into it. But that' s a pretty minor, and perhaps idiosyncratic, recommendation. So feel free to leave it as is or modify it to the way I would have written it :-)


Great :) It works and I smoothed it up good now. I'd like to add a few more things to it though, like travel to other towns by town name by doing /dt tspawn <TownName>. I'm still not familiar enough with the plugin that I feel comfortable enough to Permanently fix the worldguard problem. We need a better way to detect if dragons are DragonTravel dragons that is currently in place. I think perhaps we should extend the EnderDragon Class and check for instanceof <our new EnderDragon Class>, or add more API with a method that has a concrete way of detecting DragonTravel dragons and returning true/false.


I think this works now but it's a little messy…


jkg


60092441 정해건


Of course! :blush:


we don't need to generate any move from here. according to the design in getTopMove


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


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


Why is there no `break` statement here?


the comment is a little confused. I didn't mean this is different than previous implementation , I mean every byte array
passed through the ArrayBlockingQueue is static allocated, instead of  new->use->throw .


Not really, but it makes for easier modding. Zhuowei will be changing addongen to do this by default


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


Perfect, Thanks!!! :D
Where do I donate?

On Fri, Nov 13, 2015 at 7:54 AM, Falkirks <notifications@github.com> wrote:

> That works. You can also pass an IP, port and optional message to transfer
> with ServerTransfer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Falkirks/SimpleWarp/commit/b134f7c7a4600f70a53a95a2cc9e5a6603ea1de7#commitcomment-14384813>
> .
>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5154](https://teamcity.mesosphere.io/viewLog.html?buildId=75752&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:31


Great job. But why can't I compile it? It says that there is a Exception in thread "main" java.lang.StackOverflowError
	at assign.pkg4.Quick.quickSort(Quick.java:22)
Do you know why?


tjhjhjhj


Part 1: Supports options to include comments and/or strings for renaming

Part 2: Updated Rename Popup Dialogue
![capture](https://cloud.githubusercontent.com/assets/13545633/11162400/afbdcca4-8b04-11e5-8b4e-88952219ffa9.PNG)

Part 3: Preview Changes Live
![animated](https://cloud.githubusercontent.com/assets/13545633/11162384/24548e28-8b04-11e5-9810-6b90f12db506.gif)


antipattern:
foo rific

justification:
hey goo

created on antipattern.io


autonJumper is a vex jumper which enables autonomous if it is inserted...


those are two different timers with two different values


@MGaetan89 not sure if this will cause any issues but it doesn't seem to so far.


Automatic upgrade to https check to address #98.


Sketchup2016 (macumber) - Win64-Windows-7-VisualStudio-12: OK (2336 of 2370 tests passed)

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


fixes #13


not working


Looks like this is causing flake8 errors in CI.


DIRTY


This shall fix issue #1


@colemanw looks good to me.


On Sat, Nov 14, 2015 at 10:28 AM, zhtengw <notifications@github.com> wrote:

> 0x00007ffff07f1d9c in ?? () from /usr/lib64/qt4/libQtTest.so.4

Well I guess there's your answer.  Did you `make clean` before switching to
Qt5?


In actually, this ami vagrant image is not working correctly.
I had to modify /etc/redhat-release file to have `Amazon` string in order to make test success,

it has ...
```
cat /etc/redhat-release
centos
```

so I did,
```
cat "Amazon" > /etc/redhat-release
```


Hi Steve, I feel that to put this paragraph into this section is a bit off-topic, what do you think?
this discussion is focusing on discussing why not use `position: sticky;` instead of `fixed` position.

Maybe we can put this paragraph to the very bottom as a Q&A section?


"LG/L" : {
			  "name": "Louisville GNU/Linux",
			  "frequency": "Meets every week",
			  "web": "http://www.meetup.com/LG-L-Meetup/",
			  "freenode": "#gamedevlou",
			  "mailing-list": "LG-L-Meetup-list@meetup.com",
			  "calendar": "http://www.meetup.com/LG-L-Meetup/events/ical/",
			}


New GNU/Linux and Free Software Meetyp


See https://github.com/danmar/cppcheck/pull/705


fdf


They are just command line arguments at this point, see invocations in [the forum](https://forum-ps.informatik.uni-tuebingen.de/t/erroneous-grades-validation-of-grade-rktd-now-possible/1165/8). I could of course still stick them into a file...


Oh, feel free to change it.
#2 I was asking why this module is better than just `position: sticky;`. Not why `position: fixed` in the implementation.


こちらのcompanyとgroupでお願いします。


spec/controllers/groups_controller_spec.rb
spec/models/company
でお願いします


How could I install this using homebrew at the current date?


@jnbt i like the approach :+1:

This gives us the opportunity to inject user specific commands that are not part of our codebase.


@kirbyfan64 IDE WORKS FINALLY! I AM SO AMAZING!!!!
(ok other people helped out too... thanks @Mego :heart:)


why comment out~~??


I see,

I believe the refactoring was incomplete then and the fixture should return a function which gets keyword arguments for get_version and the expected root as first positional argument


<img src="https://assets-cdn.github.com/images/icons/emoji/unicode/2764.png" alt="<3">


I don't like that generalized wbTNAM name, it could have a different meaning depending on record. Just remove it and use wbUnknown(TNAM) in place as before.


Ok, It seems that you might not fully understand what I am trying to achieve here. As I've answered, without the logic of the component, it's is not possible to apply `position: sticky` only to achieve this effect.


I understand this. We needed to add it in the readme. feel free to fix this.


`vgs2play` 等がコマンド終了時に abort する（Yosemiteでは発生していなかったが）
プロセス終了時の解放処理なので、解放はOS任せにしておく。


ハッシュいらない
get :index, company_id: @company.id, user_id: user.id


let使う

let(:user) { create :user }
いでっち自分で調べてなかったけ？？


Thanks


なぜならば、インスタンス変数はダサいから


いいと思われ!


请问，分享base64的图片时， 怎么设置参数


Where is defined `DESTDIR`?


let this function return error


let使う

```
let(group){ create :group }

it "" do
  get :show, id: group.id
end

```

で行ける。

NG

```
it "" do
  let(hoge){}
  hoge
end
```

NG

before do
  let(hoge)
end

it "" do
  hoge
end
```

あくまで実行環境の外に置かないと意味ない


create,group
-> create, group

),:company_id
-> ), :company_id


company_id: company.id


てか、ここカッコ無いじゃんｗ


and here print error reported by applyNamingConv
And it will be good to know which files didn't finish the renaming.


改行


@mdo: so what should I do with this?

Before:
![1_old](https://cloud.githubusercontent.com/assets/349621/11162571/08192eb0-8ab1-11e5-998b-eb3157a2609f.png)

After:
![2_new](https://cloud.githubusercontent.com/assets/349621/11162574/0c5f69f8-8ab1-11e5-8c2a-bb69794616e5.png)


AciD part:
http://paste2.org/ZkpdtpHs


remove


return the error


基本
:key => value
じゃなくて
key: value

こっちのほうがすっきりしてるから


```
subject{ delete: destroy, id: 1, user: user }
it 'hoge' do
  expect( subject.call ).to hogehoge
end

```


コメントずれちゃったけど、ここのexpectのカッコの中多いじゃん。
ダサいから外側にsubject出だす。
調べてー


こういう、いらないコードはちゃんと消す。


カンマの後　space


これなにやってるの？
このコード意味なくない？


The same old song jon :
https://www.youtube.com/watch?v=6NXnxTNIWkc

Before that marvin was singing :  https://www.youtube.com/watch?v=2p5qeGWKlFY

I really think a lot people don't pay attention to the lyrics ....


describeじゃないな。
contextだな。


http://qiita.com/geshi/items/a930461f36bae880657d#describecontextit%E3%81%AE%E4%BD%BF%E3%81%84%E5%88%86%E3%81%91

これは、userをテストしてるわけじゃないでしょ？
userがサインインしていない状況でしょ？


Unless we know the meaning, use wbUnknown everywhere for everything new. Would be much easier to spot and replace later.


describe "GET#index" do
  context "user not sign in ~~" do
end
end

みたいな


これなんで、 :index　シンボルなの？
#indexじゃなくて？
うろ覚えだからどっちが正しいか調べておいて


ブロックの間には行を挟む。
つまり改行


why did you delete?


If you've run `brew tap homebrew/science`, all you need is `brew update` to get the latest changes from this tap.


これら全部重複しているから、

```
subject{get: show , hogehoge}

it "" do
  subject.call
  expect~~
end

it "" do
  subject.call
  expect~~
end

it "" do
  subject.call
  expect~~
end
```


//sosi 4len


This markup is a bit odd. Shout if you have a better idea.


sorry for the delay. They serve two different cases. In fact, the one above may need to be a `break` as well.
The issue is around the golang `archive/tar` returning `io.EOF` on `tar.Reader` after all entries, and the standard 1024 null bytes. Which varying by implementation, there is often additional null bytes due to block rounding. Failing to get these will cause the checksum to not validate.


```Pascal
  EDID - Editor ID: ActionTrick
  CNAM - Color
    Red: 255
    Green: 255
    Blue: 255
    Unknown: 00
  DNAM - Name: Dog Trick
  TNAM - Unknown: 00 00 00 00
```

DNAM appears to be a string.


こういうのも何度も出てきてるから、let 使ってdryに


Good, but doesn't really matter for now. Just need to outline the overall structure for every main record first before looking at particular fields.


Okay.  I'll change it and comment it with what I think it is for future reverence.


зелеными которые выделенные) надо добавить или сменить в уже существующих файлов эти строчки?


### Comment op deze commit.

Zoals dit dus.


cleaned up file


Если используешь поле Thread t - пиши в заголовке implements Runnable. Если пишешь extends Thread - то используй this вместо Thread t. Короче, почитай как создавать потоки.


I know what happened, after looking at the commits using `gitk`. It looks like you didn't pull from master before creating your AbandonShip branch

----
Ideally the workflow is:
0. (on other branch) `git checkout master`
1. (on master) `git pull`
    * should only ever result in a fast-forward to the latest code
2. (on master) `git checkout -b new-feature`
3. (on new-feature, or whatever your new branch is called) do work, followed by `git add` and `git commit`, repeat as needed
4. (on new-feature) when ready to open a pull request, or when lots of work is going on with master, do a `git merge master` which will either say `Already up-to-date` or bring in the latest changes
    * make sure to test your code after the merge (ideally everyone would have tests for their features and they should all pass), also test if you have to manually resolve merge conflicts
5. open pull request and continue working on the new-feature branch to fix things if needed


@@ -0,0 +1 @@


I don't like this change.   Backends that call symbols in the swank package are obviously breaking abstraction barriers anda that I don't want to encourage that. The old approach to make those kind of calls really ugly is preferable. Please revert this.  Ideally the needed context should be passed as argument or callback function to the backend.


Идея функции getSocket в том, что она БЛОКИРУЕТСЯ на вызове accept. А у тебя она просто возвращает значение поля. В итоге этот цикл (строка 12) - просто "зависание" программы со 100% загрузкой ЦПУ.


....What the hell!


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


@aofall 只是把我本地的自签CA上传上来备用。。


красные строки старый код, зеленые строки новый код


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5157](https://teamcity.mesosphere.io/viewLog.html?buildId=75755&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:58


update


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


:wave:, you should use [`hub browse`](https://github.com/github/hub#git-browse)!

:ghost:


лучше назвать переменную `result`.
также тут нужно учитывать значение таймзоны.


YearDescription_Results (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


если бы в `this.date` хранился в типе `Date`, то можно было бы забирать дни из `date.getHours()`, `date.getMinutes()`


Can you try `killall PTPCamera` after connecting the camera?


тут точно нужен `JSON.parse`?
просто `const data = require('./data.json')` не нуждается в парсинге.


これなんで、 :index　シンボルなの？
#indexじゃなくて？
うろ覚えだからどっちが正しいか調べておいて

これだとインスタンスメソッドだと勘違いしてしまうからだと思います！

>>
例えば、Ruby文書の規約ではクラスメソッドの名前には.(もしくは::)をインスタンスメソッドの名前には#を使っています。


и получаешь таймзону без знака плюс или минус :)


Ah I see, thanks!


мамочки... жесть какая


Thanks. Better for now.


I think it would be useful to describe the data-format that gets in here.


@mattwarren, Check out my revert commit: https://github.com/PerfDotNet/BenchmarkDotNet/commit/ad12e14cd4f91bace12be341c3afc4c839fcfd72


A bit more description would be useful perhaps, like "Each section has a number of strings. Actual text is stored in #data." or so


Lately I've been jamin' a lot, man. When it sound good it's like w33d :D. I think I may have an addiction. I can't get this jam out of my head.....made a mix.....

https://soundcloud.com/the-brick-2/silent-space-between


A good start!
And great that you understand  so much of the machinery. Took quite a while for
me to get there.


Well, the comment explains it. For some reason i have the old example chapter with all the bad bad code. Please make sure the updated example from version 2 of the template is there. This might be a very silly error on my side, but i really don't bother with those examples anymore.


@tomsoftware you can use the w key instead.
In ```MapContent.getActionForKeyboard()``` you can find all possible keys.


So this is the only use of THStringList?

I really wonder why we bother making a class for it, surely lua is perfectly capable of storing strings itself?
In addition, this 'for each section: for each string: copy string' looks remarkably alike the THStringList constructor code. Don't know if now or a new patch would be best if you want to change it.


That would lead to race conditions and false security.


第二次


Why do we have to put this line inside the namespace?


Fixes #2 (not #1)


Why are we doing `React.findDOMNode(this.refs.textInput).value` instead of just `this.refs.textInput.value` here?


fix indentation (eclipse's xml editor provides a shortcut for auto-formatting (CTRL+F))


waow .. this is the kind of music that talks to me , i feel the malian roots on it :
https://youtu.be/Q2O93A8m1Dw?t=30s

an other brick in the wall ?
https://www.youtube.com/watch?v=2NQV2w3Z9QQ

That music is my addiction ...


cooh bro


mv repository cfg to top


mv properties to top


YearDescription_Results (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Based on who's idioms? ;) - http://stackoverflow.com/questions/20186809/endless-loop-in-c-c


cc @jtopjian


add test / example coding


Maybe 60 would be perfect, as, for example, on my device it would be 120 pixels which is enough
I think 200 pixels is too much.


If I'm recalling correctly, taking this out is just asking for trouble.  But that's an "if" :)


хрень какаято!!
1. локальная переменная elements перекрывает data-member с таким же именем! в результате тут просто утечка памяти (после выхода из тела конструктора полученный указатель на выделенную память тупо теряется)
2. имена dta-memberам лучше начинать с префикса `m_` чтобы в теле методов было видно, что это не локальная переменная и не параметр
3. почему у data-member'a `elements` другой тип??? что вообще должен делать класс `Stack` ??? -- лучше это писать в комментах к классу (перед объявлением)


определись уже с правилами именования (я писал тебе уже об этом)... код с разными стилями выглядит неопрятно!


вот это С++11


Well Nelson broke it,  haven't figured a new way to do it yet.


нету перевода строки!


лаба очевидно не полная?
в следующих лабах добавляй пожайлуста текстовый файл с полным заданием!


аккуратнее и логиченее сделать так:

    std::size_t stack_capacity = get_int_value();

то что у тебя это мягко говоря не красиво и неудобно (ну и нелогично конечно же)


暂时不支持base64图片。


нет никакой проблемы с возвращением тривиального типа как результат функции... не нужно городить такие "сложности" (как output parameter) на ровном месте... нет никакой реальной необходимости тут в этом


Boyer-Moore would certainly be a good candidate for cases where needle is vastly smaller than haystack, and haystack is large. However, I'd imagine the setup cost would dominate the search for many combinations which exist in LiveCode usage (lots of small needles and small haystacks). That being said different searching algorithms is something we can experiment with in the future (the glibc strstr uses the two-finger algorithm).

I'll try the reverse needle search though - I'll just add a ReverseEqual operation - and see how it compares.


`m_firtst` и `m_last`


I'm happy with that - I just like to use a clear prefix for static functions to make it obvious that they aren't used outside the file.


это лучше объявить в private части класса очереди т.к. это его "внутренняя" структура, и пользователям класса очереди она не должна быть видна -- это "внутренее дело" очереди: как она там хранит чего никого "снаружи" не касается...


I forgot to add it, I will push changes now


это еще что??? зачем это???


не нужно наверное копировать `task` (хоть он и не оч большой и по факту уместится в регистры проца при передаче)... тут лучше таки объявить это как `const task&` и избежать одного копирования


Should they? Skip is quite a complicated function to inline - so whether it would be depends depends on the compiler's heuristics for inlining and whether it is conservative and optimistic in doing so.


die 4 zeilen: 138, 139, 140, 141


They are supported - yes... There aren't any tests for unicode yet either...


That makes sense. I'll make the relevant changes.


Personally I think ``` `(( "cmd" . ,(concat plugin ":" command)))``` is more readable, but you're more proficient with elisp, so the decision is up to you.


That's what I thought initially, but I got a lot of new-line separation for some reason, hcat just worked. I'll review it again after changing things from `String`s to `Box`es.


great videos. I'm full for the rythm.....raw...like rockers! THe second video sounds very much like traditional cuban music,except for the singer....

https://www.youtube.com/watch?v=PRDd7pp7Wjo


можно было нарисовать конструктор классу `queue_node` и избежать "ручного" заполнения...
одна из фич ООП в том, чтобы инкапсулировать ("сростить") код с даными -- т.е. вот ты конструируешь ноду, и после конструирования она уже готова к употреблению -- вызывающему коду не нужно делать никаких дополнительных действий (как у тебя) -- это делает твой код проще в понимании и сопровождении


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


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


Bike shedding a little more here. :)

I'm a Sublime user, and never had to do anything extra for this repository before this commit. Now that it has a non-zero `.gitignore` file, I have to familiarize myself with it, and it ever-so-slightly increases the chance of there being some files in the repo without me realizing it because `git status` won't tell me. I prefer having as little content in repo's `.gitignore` files as possible because I find it lowers the mental overhead of tracking invisible/ignored files.

I don't feel strongly about this and nothing needs to be done, I just wanted to share my (slightly perfectionist) POV and motivation for avoiding adding more content to `.gitignore`. :)

I am curious what other people think, whether they prefer empty `.gitignores` or all-encompassing ones like [this](https://github.com/github/gitignore/blob/master/VisualStudio.gitignore) or something in between, but perhaps this isn't the best place to hold this _discussion_. ;)


чаще используй `assert` в своем коде!
к примеру когда очередь только сконструирована `m_first` и `m_last` должны быть оба равны `nullptr`, но чтобы понять что очередь не инициализированна, достаточно проверить на `nullptr` любой из мемберов ибо мы априори знаем (как авторы этого кода), что втыкание первого же элемента **должно** установить **оба** эти указателя -- т.е. ситуация в которой хоть один из них равен `nullptr` **невозможна** -- вот это и нужно ассертить!


используй `nullptr` раз ты используешь С++11.
**НЕИСПОЛЬЗУЙ `NULL` В С++ КОДЕ!!**


现在通过canvas生成了一张base64图片，想分享出去，请问怎么转化下比较好呢？


hi Michael, great you added MathJax support!
It seems though that MathJax.js takes some time to load, affecting thus the page load time. In my own fork, I tackle this by adding an yaml boolean to control which pages need the Mathjax library. Unless a fully scientific site, Mathjax is not needed in all pages; particularly the archive pages. Consider this in any future update. Thanks for your great theme, and cheers...


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


печатать чего-то из класса очереди в целом не оч хорошая идея -- сама очередь должна просто уметь сообщать вызывающему коду о результате операции, а что с этим делать это не область ее отвественности! (гуглим про принцип KISS %). она (очередь) не "знает", как нужно реагировать на ошибку, и ошибка ли это вообще -- держим код очереди как можно более простым (тогда он будет проще в отладке, тестировании и сопровождении)


`;` после тела метода не нужно!


TOO COMPLICATED for me lol
p/s lucky no need to present this time


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


if items has zero length, then if we try to access items[0] will throw an exception.


when i do my part need to add any function from here>?


@brendo you've got an extra parenthesis at the end there - causes a 500 when trying to use the updater.


@yehudasa Currently I see no necessity to do the check as this method is only used when we are removing a subuser. However, if you believe it might be called by other functions in the future, yes it is better to check here whether a subuser is specified in op_state to avoid potential bugs.
Also I got another question: do we need the parameter defer_user_update in this function?


This is just part of the problem. set_user() calls setuid(), which does according to manual:
setuid() sets the effective user ID of the calling process. If the effective UID of the caller is root, the real UID and saved set-user-ID are also set.

In other words if we didn't call setreuid above(), this call wont set the real UID and saved set-user-ID.


you're gonna like this......This is part of the best music from this part of the world....enjoy:

https://www.youtube.com/watch?v=1CfmJb63Kx0


Looks good. Will give it a 2nd look-through later today.


Decoded LMDB structure and identified how to extract data and labels


ah ah ah ... you catch me : sagrada familia !
One song i really like too about colors and painters :
https://www.youtube.com/watch?v=uQIS_kXeEWc


` Treefeller no longer lowers exp for big trees`
Why has this been changed? I liked that behavior ^^


@fpletz Thanks, I forgot about it.


TeamCity Common Provider :: CommonProvider => CI [Build 2.0.49](http://localhost:9090/viewLog.html?buildId=384&buildTypeId=CommonProvider_CoreCiRelease&guest=1) is now running


fffff


TeamCity Common Provider :: CommonProvider => CI [Build 2.0.49](http://localhost:9090/viewLog.html?buildId=384&buildTypeId=CommonProvider_CoreCiRelease&guest=1) outcome was **SUCCESS**
Summary: StyleCop violations: 229, Tests passed: 68 Build time: 00:01:28


Does it work? It didn't work for me when I added this to the syncthing-silk client some time ago:/


so cool


I had replied to each of your points individually, but this isn't really the place to discuss each one at the length it deserves.

For each individual change (as a client!) you want made, open an issue on my bug tracker for it, explaining what you want changed and why that change is necessary/a good thing, giving at least one concrete example. Then we can discuss each issue separately on its own merit, and hopefully reach some agreement one way or another on each one.

Please refrain from commenting on internal design / implementation details, because although I'm a lot happier with the `refactor` branch than `master`, I think it's just going to be a "spike" and I'll reimplement it using TDD, as that was very helpful when writing libespm2. There is no point arguing over code that is going to disappear.


@jython234 0.13 beta is out. I'm assuming you don't have it? If not, then I can show you all the features.


Thanks, fixed :)


This one is for the boss :
https://youtu.be/ETnnHRnpXxA?t=5m55s
lol


Uf, qué lío...

Bueno, voy a intentar poner al día los pull requests para poder preparar la nueva versión.

Si rompo algo disculpadme, es pura torpeza :(


gettext really needed for folder.name here?


space missing before type


How about using `rsync -a ext_scripts/ airootfs/usr/local/bin/` which will just work even if the dir is empty?


should check for this being numeric and non-empty after the read (lest you add broken iptables rules)


Yes, I think we already tried something similar in the past, but it did not work out as expected. We could give it another try though since the current way is not really efficient.


should check this is a real interface too, for the same reason (or at least not empty)


Merged, thank you!


Wow, that's a really short version of the method. :+1: Are there test cases existing?


Not really. All the complex geometry operations (like intersections, relation tests, buffers, etc) are handled by an external library ("geometry engine" in QGIS). At the moment in QGIS this is all handled by GEOS (this is how it was with the old geometry too). In the new geometry classes, calling `createGeometryEngine` will return a QgsGEOS object which allows you to more precisely control how these GEOS operations are handled. In this case, we create the engine to allow us to call `prepareGeometry`. This allows GEOS to use prepared geometry relation operations, which are dramatically faster for times when one prepared geometry is tested against multiple other geometries (See https://trac.osgeo.org/geos/wiki/PreparedGeometry)


should also quote $interface and $sshport to make sure nothing goes bad in a surprising way of one of those arguments ends up having a space


Hesch öppis ibaut, wo überprüeft eb sfeld valid isch bevor de code innerhalb vo calculateAmount() ufgrüefe wird?


thx or the vids. bed time, cya.


安装插件后，使用cordova-plugin-wechat-example-master，提示Wechat plugin is not installed. 怎么解决


Is it really necessary to have this commit in the git tree ? Please squash it.


Is that a required preconditions?


parece que estaba ne el branch principal y los cambios no los veia ahi hasta que me fui a mi branch


to prompt in a more fully functional way, see https://github.com/a-wortman/projectm/blob/master/init.sh#L308

mainly `Y/N` is confusing because it's not clear what the default is (and the code doesn't make it very clear either)
the prompt linked above capitalizes the default and then accepts `yes`, `y`, or any capitalized form for yes. it takes any non-yes as a no, though, so might still be more open to input than your tastes.


The LeveledSpell still exists in the engine. There is just none in the current game.


quote "$onvps" and put spaces on both sides of `==` - as is this comparison is always true. and if `$onvps` is empty string, this fails again in strange ways (`[` is weird)


same comment as the other if


thanks,for manager my project


hello sir

i am getting this error after i merged r26

target thumb C++: init <= system/core/init/builtins.cpp
system/core/init/builtins.cpp:862:12: error: use of undeclared identifier 'e4crypt_set_user_crypto_policies'
    return e4crypt_set_user_crypto_policies(args[1]);
           ^
1 error generated.
build/core/binary.mk:706: recipe for target 'out/target/product/hammerhead/obj/EXECUTABLES/init_intermediates/builtins.o' failed
make: *** [out/target/product/hammerhead/obj/EXECUTABLES/init_intermediates/builtins.o] Error 1

can u please help help me out .. please help me sir ..
thanks


might want to make the `else` case more specific to a negative input, than anything not-`Y`.. this seems like a drastic step for a typo


it's probably safe to default to the first non-`lo` here, and get the address from ifconfig


@xboxfanj
do you have fixed all the errors manually?


See this:
https://sinujohn.wordpress.com/2013/01/14/training-the-thulika-keyboard/


Yes, it passes the provided tests. Most of the removed code relates to the existing String.trim() method which solves 70% of the previous code (remove leading and trailing white spaces). I think there's quite a lot of these library functions which weren't originally available which now can be used.


Hm nai. Aber channi no mache.


please explain the need for this (for future reference)


afair we removed truncate for the introduction on the portal page, so remove it here, too?


Good spot


you've used `$raw = true` in line 431, here you specify the argument as `'raw'`, preferably to be a boolean unless we have multiple types other than `'raw'`


Surprising to make the inner margin bigger than the outer margin.

The standard classes (and most typographical style guides) specify larger outer margins. (Extra space which might be needed for binding may be specified elsewhere if using geometry package, and depends on the binding method used, not on the typographic design)


missing method docs


whitespace before type?


preferred syntax (similar to line 642):

```php
$this->matches[] = [
    'type' => 'Relation',
    'property' => 'id',
    ...
]
```


preferred usage of expressive variables (when applicable) in this case I'd use `$cypher`


let's get rid of commented out code unless it's there for a reason.


@david already commented on the margins, nothing for me to add.
----
*Fixed ^^J in fancyhdr error message.*
Do you know what the `^^J` is doing? And why i used multiple lines?


I am an idiot yet again.  Yes, I missed a file, and no, master should never ever be broken.  Sorry.  That was me rushing because I feel bad that I haven't explained to you properly what I am aiming for here.


@aleph7 It's super ugly I know. But this was the only way I could think of doing it. I'll take a look and see if I can do some refactoring/improving. Just committed a couple more for you to look at.


If this goes on not making sense we should do text or voice chat so I can try and explain what I'm trying to achieve.


#121 fixes everything related to this.


wat u doin m8


why even take a ref?


You got it :+1:


I will explain to u so dun worry


Then I guess we should copy recursively! :smile:


btw, y u no override?


@johannesbottcher Are you happy with this now?


@Nadahar this is used for the killing the Unix process. Can you test it for it?


pk pa for range?


aussi, c'est `++it`, pas `it++`


are you sure you want to copy?


should probably be const auto&


pk :[[[


putain c'est sale...


u can use the common function when u create deposit function
when u wan add a customer bal, use currCust->bal += amount;


Ah cool I got it to work. Still - is it worth using? You can't easily undo it or uncomment a specific line. I'm just not seeing the benefit.


tu fais 2 copies d'expressions là.


you actually want to clone?


Okay cool, got it to work. Thanks.


Here, you are cleaning what the underlying SM implementation has done wrong. You should actually fix `dnie_transmit_apdu`.


@easyaspi314 maybe you can build fresh new build? Since I have only Android Studio on my laptop, and it will take some time when I will go home on my PC to build it on Eclipse.


LOL

Funny story: remember when Stef pulled the best prank ever on you? He actually messaged me beforehand to basically tell me “hey, you’ll probably find something weird in commits I’m about to push. Don’t ask about it on the channel please.” (Apparently he also thinks it’s impossible to sneak stuff past me.) I don’t think I actually would’ve found anything, but with that motivation, of course I dug in. Had a good laugh when I understood what he had done :D


Why is this not a traverser: it only visits the current node but never recurses inside the tree.


Why don't classes receive new syms: this is done by the typer. Invalidate simply removes symbols from blocks so the namer can re-create them.


Why the special case for lazy vals: Dunno.


oki


`local dummy` is still better than `global dummy`, no? :stuck_out_tongue_winking_eye:


That would be a good addition.


Just need to upload the .exe files now and the dll files :)


(not part of this commit, but valid question nonetheless): Where's `text2Find` `delete[]`d?


HBIRE_vector_take2 (amirroth) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


see https://groups.google.com/forum/#!topic/open-nars/r5UJo64Qhrk


This `c` looks hanging. :-)


the repository should contain environment-agonistic stuff, i wouldn't put this inside the repo


@UniversalMediaServer/developers it seems to me that it is the bug because  the ``` !line.startsWith("100")``` doesn't make a sense. If it is a try to be a restriction for the logging of the more than 100 lines it definitely doesn't work.


great stuff, i've refactored my code to something similar


Wait so it used to load Sam's config before yours?


asfafsdfsd


asdas


Foi adicionado os forms responsivos de listagem de membros e instituições.
Consertado bug dos migrations.
Modificado as rotas dos controllers

Obs.: Quando forem dar subir algo, apaguem a pasta ".idea" do seu projeto.


:clap:


Why isn't it a “method name”?


this is an inline comment.


Or simply method?


do we really need dynamic css? [vertical align is pretty easy](http://zerosixthree.se/vertical-align-anything-with-just-3-lines-of-css/).


Oh wow, I made this change long ago and never committed it until now.  Bad me.


Me neither, but the play store was getting pretty upset at even slightly questionable names


you should use a single action type, with a simple bool payload (`busy`)


Yes, before [hosts were associated with a username](https://github.com/kotct/kotct.emacs/commit/5b09b829fbff23060be2824cb4eb3e9171212699).  It looks like I wrote that code, which is really weird to me.  I don't feel like I did.


Per your comment below, URLs are no longer possible as parameter values.


Seems reasonable; I updated it so values must be alphanumerical characters or spaces.


HBIRE_vector_take2 (amirroth) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


It would be good to use the new formatting "syntax": `{}.format(...)`


Done.


ici ça m'a permis de faire `interpretAll(begin + 2, end)` par exemple


Virus G-Del - Delete all file in your computer.
Using Net Framework 2.0


astron install does not run


It is said in the documentation.


ABOOOSE


it didn't cross my mind at any moment that a loop over each element of a C++ collection would *actually* clone each one :-1:


Hmm, I could not find it then


just an infinite loading circle
http://prntscr.com/92uyxd


Fecha de la creacion del proyecto 14/11/2015, primer commit realizado


You could format the `response` directly.


o god, totally forgot after this much time doing java :laughing:


just wait and see the macros m8


I believe I did on your machine, if I remember correctly.


The things that are incomplete:
1. The code may look "exactly" like the DS18B20 code, but it isn't.  Besides the family code, the bigger difference is with `isConversionAvailable()`.  It doesn't exist in the original driver.
2. I'd prefer to avoid copy/pasting the controller/driver code.  I prefer to abstract it and re-use it.
3. There are still `console.log` messages
4. (perhaps most importantly), this code will only work with a single device.  We need to make sure that this code works with more than one MAX31850K.

Knowing this, is this something you'd like to tackle?  If not, I will get to it... I've just been rather busy lately.  If I do it, you'll probably still need to test it for me with multiple devices... since I only have the one ;)


what do you mean?


https://dev.twitter.com/streaming/overview/messages-types
"User stream messages"


Start over.


:trumpet: :trumpet: :trumpet:


Done (it actually led to 2 new issues). I also removed the TODO.


On a Wednesday in the middle of April?  Hmm.  Part of me does remember writing it, but @samontea wrote the original design.  I'm really not sure if you or I wrote it.


If this spec is passing, then `BigDecimal` isn't duplicable, right?  Weird.  I wonder if @hipbyte is aware (that BigDecimal doesn't implement `#dup`).


This commit did not actually contain the protype but only some preparations for it. The TODO comments are still valid for the prototype downloadable from branch "enhancements", however.


Nice!

Btw, I know that conventions on `"` vs `'` often lean towards using `'`, I would ask that in the future try to either:

1. don't change it
2. put the change in a separate commit, so the *more important* changes can be seen separately.

:smile: Thanks for doing all this work!


Yeah I'm curious as to *why* these aren't working, that's my only question on this PR.


I suspect that there might be a way to do all of these in one line per exported var


One con of removing the bump package …


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


I'm pretty sure I wrote both? I don't know. It would seem not.


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


wait jk


Wouldn't it be cleaner if no `state_file` created the pet in memory? I may have had some reason for not doing it so, but I can't figure it out now. @oitkjf @JamesWyst


Does it leap of the ground or something? Can't you adjust Throttle Mid?


astrond.exe just hangs at the moment


Does it leap off the ground or something? Can't you adjust Throttle Mid?


both `import PIL` and `from PIL import Image`?

If you use Image enough that you want a shorthand, I believe you can actually just do

```
import PIL
import os
# ...

Image = PIL.Image
```

`from X import Y` is just shorthand for `Y = __import__(X).Y` IIRC, so if you need both I feel like you should do it manually. Not sure what PEP says about it though.


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


Empty line needed here to satisfy StyleCI.


`CreateCollage` is a weird name for this object. It sounds like an action, while objects should be nouns.

Maybe `CollageCreator`?


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


You're just a magical :shipit: that controls @samontea and I when we want to write Lisp. :)


You're setting this up here and then not using it until way down below. Technically you're now including the interpret time of the classes and functions in your runtime calculations, which I doubt is what you want.

Also in general you should try to keep like things close together. The start time is defined waaaaay up here, and the end time (and every other place you use it) is waaaaay down below.


Attempted to delete readme-edit files and start over. Not very successful.


This is great! Thank you. What version of WildFly does the plugin use?


@valib Which bug? I can't be meant for restricting logging to 100 lines, there must be some other use - that's not documented. I guess we have to look at how it's being used and git blame to try to figure it out.


did you experiment with different segment sizes and their implications on speed? do you still have the results?
did you also test how it impacts compaction?


dieses githup macht mich fertig, die texturen sind nicht drauf kotz


from testing result, it is concluded that using "positive:relative" is more adaptive to different screen sizes/devices because adaptive allow flexible layout while preserving the order of display.


I was hoping for `export * as enc from './enc';`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5194](https://teamcity.mesosphere.io/viewLog.html?buildId=75894&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:03:19


Using the debug menu functions of musicCat here should depend on a config file having "debugmenu=true" so I can merge this safely.


HBIRE_vector_take2 (amirroth) - i386-Windows-7-VisualStudio-12: Tests Failed

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


You SHOULD NOT commit this to master … this version doesn’t exist yet!


Please make a branch :)


I thought we agreed that master always relies on master of other repos. If not, I can push things into a branch.


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


I just reverted since this commit. We can redo once we released a new vl version.


`#{value} ` should be `%{value}`


But it will be breaking the things for `console` apps ?


```
rand(1..5).times { post.votes.create!(value: [-1, 1].sample, user: user.sample) }
```

should be:

```
rand(1..5).times { post.votes.create!(value: [-1, 1].sample, user: users.sample) }
```

`users` is plural.


Lets better use non private `WIN32` instead ?


Interesting change. Did you think about how this might impact the repo long term? It will accumulate old deleted data in (older) segment files and corresponding DELETE tags (in newer seg files).


@josuf107 I've used vim/vi/ed/sed for a long time (15-20yrs) and there's not really any standard other than "use slashes by default unless your pattern/replacement has a slash, in which case your first fall-back is some other non-meta-character that you don't use in your pattern/replacement."  Most folks pick a default that's easy for them to remember/type (in my case it's usually "#", falling back to "@").


Très bonne idée. Nous invitons tous les mourides de France de cloner ce projet et d'y apporter leur soutien.


Awesome <3


Besides lets not bet on `uv_guess_handle` return value. `UV_UNKNOWN_HANDLE` or `!UV_TTY` check might be more suitable? Apart from these 2 rest is fine. Thanks


@Nadahar it makes not sense.  If the line 100 is so special that it could be abandoned why it could be restricted for all InputStreams doesn't matter which is the origin?


This `ng-controller` is a duplicate (which causes your controller code to run twice).  You can just remove it, because it's already specified in the route.


![Magic Numbers](http://vignette4.wikia.nocookie.net/muppet/images/6/62/ElmosMagicNumbers.jpg/revision/latest?cb=20120508214609)

(Or I guess not a number persay, but where did this value come from? I see that an earlier step writes to this file, but your API is implicitly assuming that this file will exist.

Instead, this class should have some knowledge of the `CreateSortedStockPhotosFile` and ask it for its output filename.


n


You forgot to add the path here.  For example: `$http.get(url + ‘/api/v1/memories’`).  So what was happening was that you were requesting the home page (the swagger docs).  In your API, you included the CORS middleware _below_ the static middleware, so your `public/index.html` was not being served with CORS headers (https://github.com/LisaCarlson/API-app/blob/master/myApp/app.js#L26).  As a result, it just looked like it was a CORS issue, when really it was an issue of not hitting the correct path.

That was a fun one to track down :)


You're saving three different pieces of information here about the file, and I'm not convinced you need all three. At the very least, it looks like `self.origImage` is unused outside of this line, and `self.origImageFileName` looks like its only used to calculate the output filename.

Instead, you should take the output filename from above (aka let the caller decide where this object puts its output, so they know where to find it later).


Folder? What folder? A more descriptive name here would be helpful.


I think should update the description of the repo too 😏


Topo minitests:
# Running:

Node under test:
  - name  - n7k9
  - type  - N7K-C7010
  - image - bootflash:///n7000-s2-dk9.7.3.0.OAC.0.8.gbin

TestFabricpathTopo#test_member_vlans_invalid = 20.37 s = .
TestFabricpathTopo#test_member_vlans = 20.69 s = .
TestFabricpathTopo#test_topo_create = 20.05 s = .
TestFabricpathTopo#test_invalid_topo_id = 20.92 s = .
TestFabricpathTopo#test_topo_destroy = 20.63 s = .

Finished in 102.666102s, 0.0487 runs/s, 0.1558 assertions/s.

5 runs, 16 assertions, 0 failures, 0 errors, 0 skips

Global minitests:
# Running:

Node under test:
  - name  - n7k9
  - type  - N7K-C7010
  - image - bootflash:///n7000-s2-dk9.7.3.0.OAC.0.8.gbin

TestFabricpathGlobal#test_global_destroy = 22.77 s = .
TestFabricpathGlobal#test_allocate_delay = 21.91 s = .
TestFabricpathGlobal#test_linkup_delay_all = 22.95 s = .
TestFabricpathGlobal#test_loadbalance_algorithm = 22.02 s = .
TestFabricpathGlobal#test_ttl_multicast = 21.15 s = .
TestFabricpathGlobal#test_mode = 21.66 s = .
TestFabricpathGlobal#test_loadbalance_multicast = 21.21 s = .
TestFabricpathGlobal#test_global_collection_empty = 1.37 s = .
TestFabricpathGlobal#test_ttl_unicast = 22.58 s = .
TestFabricpathGlobal#test_transition_delay = 21.97 s = .
TestFabricpathGlobal#test_aggregate_multicast_routes = 20.49 s = .
TestFabricpathGlobal#test_graceful_merge = 20.99 s = .
TestFabricpathGlobal#test_global_create = 19.03 s = .
TestFabricpathGlobal#test_switch_id = 20.91 s = .
TestFabricpathGlobal#test_loadbalance_unicast = 21.48 s = .

Finished in 302.496399s, 0.0496 runs/s, 0.1355 assertions/s.

15 runs, 41 assertions, 0 failures, 0 errors, 0 skips
Coverage report generated for MiniTest to /ws/dcherian-sjc/git_projects/develop/cisco-network-node-utils/tests/coverage. 0.0 / 0.0 LOC (100.0%) covered.


**这句话是个测试**


@LinusU  I'm not sure about depending on 1.22 repo. Looks like depot tools uses its own copy, which is different.

I diffed the two versions here: https://gist.github.com/anonymous/d9eacf6b5c1ef44e81e9

I would recommend killing this dependency and adding repo to the PATH bits below.


fgfdgdfg


This commit breaks database creation

sqlalchemy.exc.IntegrityError: (_mysql_exceptions.IntegrityError) (1452, 'Cannot add or update a child row: a foreign key constraint fails (`amivapi`.`users`, CONSTRAINT `users_ibfk_1` FOREIGN KEY (`_author`) REFERENCES `users` (`id`))') [SQL: u'INSERT INTO users (id, _created, _updated, _etag, password, firstname, lastname, legi, rfid, nethz, department, phone, gender, email, membership, send_newsletter, _author) VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)'] [parameters: ((0, datetime.datetime(2015, 11, 15, 1, 26, 23, 8356), datetime.datetime(2015, 11, 15, 1, 26, 23, 8373), 'd34db33f', '$pbkdf2-sha256$100000$l5LyXmvtvTfGuPf.vzcGYA$lvGTmvbQ/HNMIZHwwZcP3J.pH7iKQr3/N/w5Of4hFZo', 'Lord', 'Root', None, None, None, None, None, 'male', 'api@amiv.ethz.ch', 'none', 1, None), (-1, datetime.datetime(2015, 11, 15, 1, 26, 23, 8377), datetime.datetime(2015, 11, 15, 1, 26, 23, 8380), '4l3x15F4G', '$pbkdf2-sha256$100000$7D0HICQEIGSMEYLw3hujVA$SVAifeEodiSlYTGZLG9hstDE.FVA1fPOyquerTOljds', 'Anon', 'X', None, None, None, None, None, 'male', u'nobody@example.com', 'none', 1, 0))]


Hey Nick, can you specify the "weirdness"? I've been using 1.3.1 for a while without any (apparent) issue.

Thank you.


``network.className`` returns ``email enabled``


I might not be understanding, but isn't a state file necessary for the pet to have a memory and history?


I think the commiting of the session between adding the root user and the anonymous user was necessary working with mysql. It passes only on sqlite.


TeamCity LabkeyModules15.3 :: Git postgres 9.4 [Build 13](https://teamcity.labkey.org/viewLog.html?buildId=285127&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Execution timeout (new); tests failed: 193 (187 new), passed: 0 Build time: 04:01:10
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.DataFinderTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


@IAHM-COL
We have permission to change the liscence to GNU GPL.


OK!


Ok this is the second time I've seen this `pic.size[0]` line to get the width.

Instead, you should write a helper function so that its obvious what its doing. I know from above that `size[0]` is the width and `size[1]` is the height, but later down the line if you did `width = pic.size[1]` it would be very easy to let that slip by.

One goal you should always have in mind when architecting your application is that **wrong code should look wrong**. Thats part of why magic numbers are so bad, cause the line

    pic.newSize = pic.originalSize * 5

looks fine but

    pic.newSize = pic.originalSize * kPaddingSize

is very obviously wrong. Oops, you wanted `kScalingRatio`, not `kPaddingSize`.

Anyway, I'm getting off topic here. The width, instead of continually doing `pic.size[0]` you should make a helper function like

```
def get_image_width(img):
  return img.size[0]

def get_image_height(img):
  return img.size[1]
```

Then lines like `width = get_image_width(pic)` are very obviously correct.


@hppritcha at first glance,  #1114 does a better job than this commit.
should I revert this one instead ?


`$filtered` is the best name I could think of. Maybe you would find better?


Just had a quick read-up on the YAML spec, and the other way to have fixed this (just for future reference) would have been to change it to:
`- more_information: 'plus d''information'`


"green"? Why "green"? A quick google suggests this is the default color if you dont override a pixel. Does the green matter?

Also, why is `newImage` set as an instance variable? Shouldn't a new image be created every time you run `main`? It seems like this object is single use only, and in general I consider that a smell.


These three lines are identical to the ones right below it. What are they doing, could you make it a helper function?

Also why -1? I'm trusting you that the logic is correct, but would you add a comment explaining why you have to subtract a pixel? (or maybe just explain it to me :P)


@robotboy655 Even though GMod does not use it, does it still function?


compare was introduced in 1.7 and uses that formulation (in fact compareTo calls compare) and it returns either -1 0 or 1. I'm just using the library implementation.


This should be two method calls.

```
tempFilename = self.saveNewImageAsTemp()
self.showImage(tempFilename)
```

Or hell, even

    self.showImage(self.saveNewImageAsTemp())

Methods should have one job and one job only. The fact that you added an `And` to the method name shows that its probably doing too much


Why is this returning something? What does `main` return? If this method has an API then it should be documented in a comment somewhere.


Could you explain more about what this is doing? It says `getAvePixelofEachStockPhoto` but I dont see any kind of loop. What is this doing that `recurseInImage` is not.

You could make an argument that this is to clarify the naming, but I would counter that `recurseInImage` should have a different name (will mention that again later :P)


You say "As Temp", but you are deterministicly choosing a filename. You could `saveNewImageAsTemp` twice and it would overwrite the previous entry.

When I see a method that says "As Temp" I always think that its going to make a new temp file for me at some random (unimportant) path. In fact python even has a nice library to do that for you - `import tempfile`


@mstembera is right. you need to test this with hash pressure. all things equal, this version uses 1.5x more hash, so without hash limitation (128 is largly oversized at STC), it's normal that there is no elo loss. but with Hash=4 at STC, I would expect a significant elo loss.

The MAX_PLY change alone is fine though (but completely unrelated). You could probably open a pull request for that change alone.


Why does this take an average pixel value? When I see a method named `getXXX` I think its going to be a dumb getter. This looks like its actually doing some important work, you should rename it to something else. Maybe `chooseThumbnailCloseToColor`?


leave a comment


Hello


Yeah, I noticed that too when I was chatting locally on IRC with Edwin. Very odd :)


yep, exactly.


please update your master branch, then i will validate your patch


I have no idea.


Also, I was using encodeURI. I changed it to encodeURIComponent for proper usage.


`3.0 * 255`

![Magic Numbers](http://vignette4.wikia.nocookie.net/muppet/images/6/62/ElmosMagicNumbers.jpg/revision/latest?cb=20120508214609)

This looks like some important logic - it seems to be the code to determine which image is closest to the pixel you want. This is a pretty naive strategy, and seems like something you might want to experiment with later.

One thing you could do is make a new object (call it like a `ThumbnailSelectionStrategy` or something) that encapsulates this logic - you give it a list of all of your thumbnails along with the color you want to find and it will return the right image. You could then do something like make two strategies (an `AveragePixelValueStrategy` and a `PixelDistanceStrategy`) and do two different runs to compare the results. Doing so would be a matter of saying

    CollageCreator(PixelDistanceStrategy(), ...)

vs

    CollageCreator(AveragePixelValueStrategy(), ...

Hell, you could even have that decided by a command line flag, so that you could make it easy to generate a lot of different images.


I'm pretty sure I wrote v1.0 of the hosts thing with 053d834152be1f30dbb737e5ce7c6cb13d54299d and then chris later finished it up on your machine, but i'm not sure. I'm 95% certain 053d834152be1f30dbb737e5ce7c6cb13d54299d was my own work though.


This doesn't look right to me.  Is it possible that c->queued will be true after this?  If so, we'll dispatch_suspend() the released timer a few lines later.


I feel like all of this logic should be in another object - a `ThumbnailImageCache` or something. Just tell it you want a given image and let it figure out whether to load it from disk or grab it in memory.


Merge pull request and finish first project.


@berrange I added the above into this commit: https://github.com/jaypipes/os_vif/commit/cef2b287da5b14c24a237830148e39735351b013

Note the commit message in the above, though. I changed things slightly due to needing to have a single version of object to track in the PluginInfo.describe() contract...


It doesnt actually look like you use `newPic` anywhere, and I missing something? Why store it in the cache?


imagenes no se cambiaron


@TedHartMS I had to roll back caffe to Oct version. There was static linking issue (similar to [this](http://stackoverflow.com/questions/599035/force-visual-studio-to-link-all-symbols-in-a-lib-file)) with the latest caffe that I couldn't figure out how to fix. It's not a priority to upgrade yet.

TL;DR: run `git pull` :)


Thx @byteandahalf for making this API, you made this mod so much easier :)


`format_firstline` and `format1` do not appear to have the same regex pattern, no?

Shouldn't `format_firstline` look like:
```
format_firstline /^[^ ]*[ ]*[^ ]*[ ]*[^ ]* [^ ]* [a-zA-Z0-9_\/\.\-]* ...
```

And if this is a multiline match, are you deliberately ignoring the remaining lines?

I had thought /var/log/messages is a single line output from rsyslog, is that not the case in OpenShift?  If it is a single line, do we need the multiline here?


I still don't know why version `1.0` was skipped - this might be confusing for end users.
Legacy version can be (if necessary) maintained at version `0.x`


we shouldn't add such a very long line.


This is a typo.


Proper recognition for MultiModeEngine.
Ensured Multi-mode engine > Single-mode engine > Gimbal sequence (later modules handled by former ones internally)


"ever"? State can change at any timer later, for example after user soling the conflict in the backend UI.


Cool. Good to know!


I don't understand this file, actually?


we usually don't use after all to delete tables as each spec file will take care of its tables


Been waiting for this, thank you!


delete unused files


controller spec should test controller functions not view


As long as this is one commit, you can merge these changes


This is a good example of testing the controller not the view here.


We can't use controller spec for view test


Same as above


you can add a test for invalid id no.


why are you using reload here ? Book.first reads a fresh copy from DB.


You can use context for admin and context for other users of the site


Why do we use reload here?


you can also test if the flash message is as what you expect.


I don't get the usage the point of this line


u can also check on the body of the flash msg


OK


We have multiple flashes here, can we test against specific one??


lol :D


Why are you removing/commenting out transition effects?
Have you tested this on other non-aero Windows operating systems as well?


specs don't run in order so if line (19 or 26) ran before 15 you will get a failure.


Added toList


What do you mean as "incorrect"?
1) I followed exists codestyle. https://github.com/OpenXRay/xray-16/blob/be47ce0876fb800f2219b0f5ae85cfbaa95a7e61/src/xrCore/Threading/Lock.cpp#L63
2) Why do you ever think that it would be better to mark Mutex unlocked *before* actual unlock? It's absolutely possible to have context/thread switch between `isLocked = false` and `mutex.unlock()`.


I think the hash table needs something to update the data :)


2 main notes:
1- Indentation is not correct.
2- we can't use controller specs for testing the UI.


@glinscott Thanks for raising number of games, but I parked it at prio -2. This patch is probably a slight ELO loss because of the interaction with hash collisions. Let's wait with more tests till the key is raised to either 21, 32, or 64 bits.


It is not working when the class is being exported.

**Sample**
```javascript
/**
 * My awesome App
 **/
export class App extends Component {
 ...
}
```

I changed the require-jsdoc.js to look like this:
```javascript
function checkJsDoc(node) {
        var jsdocComment = source.getJSDocComment(node);
        console.log(node.type, jsdocComment);//tracing

        if (!jsdocComment) {
            report(node);
        }
    }
```

Results in ```ClassDeclaration null```


:clap:


Your code didn't set `isLocked` ever, that's incorrect. For (2) - absolutely agree, my mistake.


@thSoft I would call atomRootPath or projectCWD  and also you don't use it anywhere. Otherwise great path I was so confused why this linter isn't working


add a test of update with non-signed in or non-admin user that tries to updates be he fails.


следует использовать ===, == может давать неверны сравнения, например 1 == "1"


лучше соблюдать такие отступы: if (a === true) {
это сильно улучшает читаемость кода


переменную следует назвать с маленькой буквы


тут, как и дальше тоже надо исправить отступы и сделать ===


@xdemolish
`--print("Warning: Regime ID not found! Returning reward as 10."),` lolwut

Btw, the commas at end of table don't cause error and all semicolon in all places get ignored in lua anyway. So your only real errors from your regex were the missing comma's in middle of your tables.

I keep semicolon-ing things (mostly vars and block ends) out of habit from other languages.


здесь нет необходимости в проверки typeof, instanceof Array будет достаточно, уйдут лишние скобки:
a instanceof Array && b instanceof Array


пробелы, правильные пробелы залог хорошо читаемого кода:
var square = 0;


@varp With this PR, change these links back to https://github.com/tanepiper.  I'll be giving you collaborator access to this repo and it makes it easier for package management.


No problem.


for (var i = 0; i < a.length; i++) {


Another thing: previously lock flag was provided by Windows `CRITICAL_SECTION` and was set inside `EnterCriticalSection`/`LeaveCriticalSection`. With `std::mutex` one can't provide exact equivalent. Context switch may occur just after lock/unlock. In that case, `isLocked` won't be set either. However, that would be sufficient to detect deadlock, for example.


Yeah see my comment somewhere else. 2 things:
1. Delimiters should probably be added by the code writing the list, not the elements of the list.
2. (Separate but related) I prefer serialisation and parsing code to be outside the classes being serialised.  Reasons: separation of responsibilities, most compellingly when you serialise to multiple formats.


Better than before. As i said before, a user will change it anyway. And a user has to read up on the bindingoffset, because that is individual to every document (paper thickness, amount of pages, different kinds of binding).


onox, can you explain to me the difference in this alteration, please?


To działa tylko dla EAN13!


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


Sorry, but maintaining software takes time, and I don't see the point to maintain it when there is more robust solution.

Maybe you can go to ratpack slack https://ratpack.slack.com to get help about how to plug pac4j jwt in a ratpack app.


This is great, is there a PR on jquery-ui for this?


three...


replaced all ; with ,
cause fk finding them one at a time


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


du hesch da es X gna als icon. es X am rechte rand vomene igabefeld staht eher für s'feld zrugsetze oder? chasch ned es warning icon nä für das?


Sorry I chose the wrong remote, I wanted to make this a PR. I hope everyone is ok with it.


good job @Naios this feature is really appreciated


s'validierigs icon setti nur azeigt werde wenn es feld $touched isch. sust öffnisch das formular und es wird bereits en fehler azeigt.


I do not like this change.


``$touched`` verwende zum de fehler am afang usblende


Also added support for semicolons in precedence checking


комменты подобного рода не должны присутствовать в финальном коде


Are you sure about this?

`print (6 % 2)` leads to: `Compile Error: Expecting ')' but encountered integer literal` in vanilla blitzmax.


два объявления с замещением? Что это?


Опция "Зачеркивать ссылки" не применлась без релоада.
Ссылки в аякснутых постах не зачеркивались.


1000th-commit within the last year, whooooooooooo.


What does this string constant stand for ?


du hesch jetzt be jedem einzelne textfeld es neus ``<span>`` hinzuegfügt oder? es gat au ohni de overhead: ``.inline-validtion.error:active { iconstyles }``


That's a completely reasonable point, but I would honestly expect most modern compilers to Do The Right Thing with this particular example.  I've also always lived by the rule that you shouldn't write the same code more than once unless it's completely unavoidable.  What's the difference in performance between the generic version and the hand-optimised verson?

Maybe it would be worth adding a comment to the effect that this operation is intended to be exactly the same as `__NativeOp_ForwardScan()` with a needle of length 1.


@SthephanShinkufag у меня из-за этого правила редактирование хоткеев немножко сломано. Можешь посмотреть?
![screenshot_20151115135220](https://cloud.githubusercontent.com/assets/1163958/11168441/2afb2c7c-8ba0-11e5-8ee5-8ee0f3e6cedf.png)


zeigsch denn da date vode letschte 12 mönet ah oder vom letschte jahr? will wenns di letschte 12 mönet sind, wasi vermuete, würi eher es datum ufd achse tue als erstes quartal und so..


Thanks you so much Kriztan :)

I'm loving the extra work you are doing on your fork :)


@mstembera: you were testing with 1 thread. what's the point of that? it's obviously a non functional change with 1 thread...


Sure thing @perlun


This covers *mean* fitness without being explicit about it, would it maybe be better to be explicit about that and leave space for *median*, *mode* and whatever other functions of average population fitness?

Or is _mean_ the default standard measure of population fitness in genetic programming?


@wfenchel: no hurry. i can wait. btw, throughput is now rescaled with threads. so using 5+0.05 throughput=2000 will give the same testing bandwidth to this test as to normal tests (10+0.1 throughput=1000).


ok,learning


kernel32.lib and other windows libraries are linked by default and not required here.


Ох. Щас.


You need to be careful with `tte->save()`. Have a look here:
https://github.com/lucasart/Stockfish/compare/cccf3c8...70f5661


I am curious - why is this necessary?


А чому у тебя шревты такие огромные? Специально увеличил?


Let's do it!

On Sun, Nov 15, 2015, 3:11 AM XhmikosR <notifications@github.com> wrote:

> @addyosmani <https://github.com/addyosmani>: should we drop 0.10 support?
> maxmin dropped support for it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/addyosmani/grunt-uncss/commit/dc01b72b2277a93fecf5ac5f6fd754a578698627#commitcomment-14400217>
> .
>


This addition looks very interesting. Do you want to open a pull request against my repo?


У меня просто экран повышенной плотности пикселей. Потому я и начал переходить на SVG, ибо растр смотрелся мыльно.


siedler3_99.f8007e01f.dat      Text / Strings (setups3.exe)


Что за девайс, если не секрет?


当我使用BootstrapV3.3.5时，我导航栏使用的是
``` html
<nav class="navbar navbar-inverse navbar-fixed-top">
```

Simditor配置的代码模仿的是官网的demo实现

``` javascript
(function() {
	$(function() {
		var $preview, editor, toolbar;
		Simditor.locale = 'zh-CN';
		toolbar = [ 'title', 'bold', 'italic', 'underline', 'strikethrough',
				'color', '|', 'ol', 'ul', 'blockquote', 'code', 'table', '|',
				'link', 'hr', '|', 'indent', 'outdent', 'alignment' ];

		editor = new Simditor({
			textarea : $('#txt-content'),
			placeholder : '在这里输入文章内容...',
			toolbar : toolbar,
			toolbarFloat : true,
			toobarFloatOffset : 200,
			toolbarHidden: false,
			// pasteImage: true,
			// defaultImage : 'assets/images/image.png',
			upload : location.search === '?upload' ? {
				url : '/upload'
			} : false
		});

		$preview = $('#preview');
		if ($preview.length > 0) {
			return editor.on('valuechanged', function(e) {
				return $preview.html(editor.getValue());
			});
		}
	});

}).call(this);
```
**尝试了一下把固定在页面顶部的导航栏去掉之后，Simditor就可以固定在页面顶部了。当固定导航栏存在的时候，toolbarFloatOffest的属性并没有起到相应的作用.**

这个Bug是不是没有很好的解决，毕竟Bootstrap是很常用的框架。


I'll check the other files later


Просто экран для пеки на 27''.


Ну вот именно об этом я как раз и говорил, лол.


Done :)


Это правило было ОЧЕНЬ ДАВНО. Недавний коммит тут не при чем.


x <- NULL


Change prefix Nitor -> N


Instead of usign world.getBlockMetadata() I'm using block.getMetadata() for compatiblity with GregTech ores, but in Vanilla code, for lapis that returns 4 -.-.
Trust me I don't like it either.


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


```
test
```


to save horizontal space. Reintroduced.


Just wondering - wouldn't just a

```
if (results[idx].isZero()) continue;
```

do the trick ? the code would be much more simple and readable IMHO.


Good edits


You should add post address also


chömer ja morn na aluege, und de bug na fixe ;)


are signals "plain" methods? Shouldn't they be set appart form others?


Minimum phase is the correct therm https://en.wikipedia.org/wiki/Minimum_phase.
Doe you have an idea to rephrase this sentence keeping the word minimum?


oh well
by mistake, i've just modified both this and main
total chaos now
shit


How about `Increasing diversity seeks to actively  onboard people`?


@liliexuan


Could you bump to 2.0.0 please? Major non-backward compatible changes.


This isn't actually necessary thinking about it - the FirstIndexOf operation will fail as skipping in this case will exhaust the haystack.


It shouldn't be confusing. We will (maybe) release 1.0.0 version of the legacy system. 2.0.0 is spring cloud based. What's confusing in here?


Good thing nobody does code reviews :)


stops log spew native crashes but still 80% wakelock


I have removed Phone 2FA for now.  Although Authy supports it in theory, it is not clear how well this is working in practise.


+1 to marcin


Discussed with Tim, references to a device will be removed.


jljljll


Hurr durr durr parallel streams make fast is not the right attitude to take here, this would likely be much faster using a non-parallel stream due to thread scheduling times and such.


Why are they unnecessary? To me they have meaning, e.g. if I see a chain of `else if`'s I know for sure two separate `if` blocks can never be entered one after another.


Change into this?

`[euler_primelist_generating.cpp](euler_primelist_generating.cpp)`


del L9-11?


目前，cntv有多个子站的video请求方式都不一样，所以最终要把它用``EmbedExtractor``改写。


Hey
I see. For a moment i was really astonished by configuration parameters but after a while i am not so enthusiastic. Maybe it is a problem of NPM v2 i am using but i see a couple of shortenings of config params:
* can not reference one from other
* can not reference environment variables (this is a pain for me)
* pretty weird names to reference them $npm_package_config_*
* not pretty handy way to overwrite
* it seems like npm-run-all does not propagate command line overwrite (https://github.com/mysticatea/npm-run-all/issues/13)


@smurfix I don't like this, the more I look at it the more I feel this should be handled as a special case from your own code. The library tries to connect and raises an exception if the server is not ready; that is a perfectly fine way for a client library to act. Adding this retry logic just for an admittedly absolute edge case is in my opinion absolutely not needed, sorry.


ok thanks


Not bad!


injured的参数是否可以简化一下？


After a few minor changes it works just perfect.


Why travis build failed? I don't fully understand the packages installation phase.

Tests passed on my PC.


Yes there is one case I can think of, and I have fixed it in dc83132, see commit message.


还需要==false吗？


纠正一下函数名：
createWeapon，isAlive


这个东西不应该放在Model里面，包括前面的Lost里面的ShowResult.Win(p);


不能使用静态方法，使用静态方法相当于没有View


Thanks for the credit :smile:


Yes, hoisting the label-button class to the enclosing `button` did the trick. I can see an instance of a long label being cut with `...`.


这是什么鬼？


this item are wrong


函数对应着动作和行为，函数名应该含有一定动词谓语属性，Game？


These names are used by hxGrid library. Need to check if it accepts file names without extensions (I suppose it doesn't).


怎么是get_weaponname呢？weapon是属性之一，你这是在获取属性内部的属性了，层次有点深了


the dllexport might have been added by us - not sure if we need to ccall these


You're right. But I think we can easily deduce the fact from the duty of `semanticTraits` function. `__traits(A, ...)` and `__traits(B,...)` must have individual code blocks, so an `if` block never be fall through to the next `if`.

That's why I think the `else if` chain is unnecessary.


test again


类名的单词首字母要大写


把if换成is


尽量不要使用单词缩写


This is odd, as it works on my machine with both MySQL, using `python manage.py create_database` with both a pre-populated as well as with an empty database.


Fixed. Thanks!


不能出现直接使用的数字，应该先定义为一个变量或者常量，比如ERROR_UNKNOWN = 3，然后loginStatus=ERROR_UNKNOWN，这样更易于阅读代码


# 2


@teryk @julienlancelot I don't agree. The field can be technically null, so getter must be annotated. An alternative is to implement a pattern like Builder that prevents from this field being null. Adding Objects.requireNonNull on the setter does not fully fix the problem .


Committed referencing the wrong ticket, and with a misspelled word.  Apologies to all who read it.


These secrets have been deleted, and will no longer work


Maybe it would make sense to allow the user to modify query? I would prefer to enable specific features over stages for instance.

If it was possible to pass custom Babel config to nwb, that would solve it I think.


Removed all the comments


It's injected by Flutie on our body, right? The suggestion is good. I think Hound will request it on different lines.


class name here will be ".lists-index"


and here ". authors-index"


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5206](https://teamcity.mesosphere.io/viewLog.html?buildId=75913&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 199 Build time: 00:20:43
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Who can tell me about LasmGratel language? It's really hard to understand, although it's of latin letters.
XD


@emgarten What were the results? Old vs New?


I guess I was thinking more for other servers, since there may be a plugin installed that conflicts with any of these Essentials signs.


That's the plan eventually, I was thinking `nwb` config in package.json could be used to configure a custom `query` for each webpack loader by loader name.


@tanepiper ok


чому?


Ok. *package.json* would work.

Experience has shown that going straight to JS (or doing both) would be a good alternative. The biggest benefit of having something like *nwb.js* or *nwb.config.js* (webpack style) is that you can actually program and comment (no JSON comments :( ).

I would probably start with *nwb.config.js* given it's flexible and merge other alternatives later if someone PRs them.


If the issue was for 'ProfilerCommunication.cpp' then why modify 'InstrumentationPoint.cs'?

I'll have to review this for performance issues as lock (and critical sections) can severely affect performance.


Завтыкал убрать


Why was the getter method removed as well? Isnt it useful to have a helper like that? (I would need it in this piece of code im porting over from jquery-ui 1.4)


I have encountered with a file with command `2` where the next 4 bits were `7`, the next byte was `0x15`. It meant to copy the next `0x2755` bytes, so I guess for `0`, `1`, `2` it is just an extension of `Copy64`. I am not sure about `3`.


ミスった。これはIssue #4 だった


我的程序用别的办法变通了。不过我觉得这个类应该能够被开发人员自由实例化使用是有价值的。
感谢鸟哥


hmmm, unsigned 8 bits can only hold 0-255 !
Are you sure this is neccessary ?


I added this back; we need it for the benchmark to work and it doesn't change normal run behavior


the format of the md_extensions setting has changed