@Grawl yes, it's correct The block order is defined in the parent template. In the child template the block order inside `<extends>` doesn't matter. And `append/prepend` affects only the content order inside blocks. In the [Jade documentation](http://jade-lang.com/reference/inheritance/) you can find more examples. I took its approach for templates extending.


Should AC_MSG_WARN or AC_MSG_NOTICE be used instead of the deleted AC_MSG_ERROR, to let the user know why john will be built without rexgn support?


This should be empty and marked 'notr' so we don't spend time translating it.


The existing practices allowed spoofing to occur in the first place and now fixing that **could** conceptually break some deployments. Holding back because of that concern and us having this discussion shows there is the potential for a real technical debt existing. It's practically the same thing therefore.

I would absolutely love to see ""anonymous (no realm) or "{empty_string}@realm" enforced as the outer, mandating identity privacy use in all cases, we cannot do that as it breaks many systems that we do know about. That's certainly not theoretical.

I wholeheartedly agree that it is insecure to not enforce privacy, it’s also an entirely different issue and question to spoofing. We don't need to conflate the two.

Commercial SSO systems that integrate snooping RADIUS auth/accounting and wired/wireless management platforms **should** be integrating so that they do not use the outer identity for logging or access control purposes. They nearly all don't do that so it is incumbent on RADIUS servers to protect them as best they can from using spoofed identities.

Nobody is being told, fuck you, upgrade - anybody can change the configuration of FreeRADIUS as they desire if they have problems. This is therefore just about choosing the best default behaviour that balances compatibility with security. I’m trying to convince you that you’re taking the wrong tradeoff here, not battle :-) I really wouldn’t want to get your back up such that you close yourself off to changing this. I just want what’s best for the ecosystem. I think you do too.

The security argument against what FreeRADIUS has right now is that:

1)	It doesn’t protect against spoofing for any account name that happens to begin with anon so the fix is incomplete. To solve that, we have to deny identities we would consider being anonymized in the outer from being present in the inner.
2)	With that in mind, what FreeRADIUS currently considers as being anonymized is too broad. Anything that starts with anon is too wide a brush, “anonymous” and “anon” are in contrast tightly defined. Additionally, we lose privacy guarantees when we allow a practically unconstrained set of identities to be considered anonymized.

Non-theoretical systems from major players that are affected in the SSO space are:

Check Point
https://sc1.checkpoint.com/documents/R76/CP_R76_IdentityAwareness_AdminGuide/62050.htm#o100097

Fortinet
http://docs.fortinet.com/uploaded/files/1279/fortiauthenticator-radius-accounting-proxy-guide.pdf

Palo Alto
https://paloaltonetworks.com/content/dam/paloaltonetworks-com/en_US/assets/pdf/tech-briefs/techbrief-user-id.pdf

Websense
https://www.websense.com/content/support/library/web/v77/radius_agent/first.aspx


All the rest of the source avoids lines that wrap, it's worth formatting this consistently.
In this case, putting each property on its own line would probably be sufficient.


I'm curious, why the move from `views/*.html` to `templates/*.html`?


@dcs619 I don't remember the exact reason, but if it's going to be more intensive, let's keep it as OnLoad, and we can change it if need be down the road.


I don't know what would be better to improve definition, because
Unicode standard covers many characters, as well as decimal digits (Arabic–Indic), various alphabets and scripts. All of them are UnicodeLetter, by a definition.

A question is: what code points are covered by UnicodeLetter?


It's sadly a lot easier to fix the RADIUS server that it is to fix many other products from many vendors.


Its possible this is the better way to do it. I just tend to give special cases a second look, in case they're not as special as I first thought.


Actually I had implemented this patch a whole ago....around 3 weeks...this was the missing commit
And it works without issues :)


en fait ct pas compliqué, j'ai juste créé une branche temporaire pour l'occasion, on pourra la virer après.

j'ai checké que les opérations sur mod1/2.js étaient équivalentes en faisant un diff entre la branche better-diff et les versions faites dans les commits foireux.


I just reviewed your add-on and was wondering why you changed the API. I am using the NewTabURL API in one of my add-ons as well and could not find much information on the new one. What made you change?


The print-head hits the bed when using this code. Both the home button and the "Auto home" in the menu.


这段逻辑有点问题, 其实它的作用只是截取图片的strokeWidth*strokeWidth的部分而已, 那为什么我们不用photoshop直接改图片呢?


Treating Auto-Away as a `isAway() = false` condition makes it difficult to use this as a Characteristic-based trigger, or as a condition in other triggers. For example, I might want a trigger that sounds like *"Turn my lights off when my house transitions to Away,"* in which case the only time the trigger currently would run is if you explicitly tell Nest you're leaving. Auto-Away is described as being more of a subclass of Away than it's own state.

> Nest uses a variety of signals to detect human presence in a Home, and can set a sub-state of Away, called Auto-Away.

There also is not supposed to be a way to transition from Auto-Away to Away mode (should instead be Auto-Away to Home), but when in Auto-Away, that is the transition that would occur. I think in HomeKit's mind, the fact that Away could mean either explicitly or implied Away is not relevant.


tag was declared as a global variable in string_process.h


To get the evidence of impact that you rightly desire, FreeRADIUS 3.1 could have the stricter protection implemented and enabled by default, FreeRADIUS 3.0 could log where this would be tripped up by default but not enforce it.


@daviddgm vc sabe responder o @maracaipe ?


We have not released `final` version of `Phaser` project yet. I would delete this class.


global variable "char c" re-declaration conflicts..


Typo.


For example, such Unicode-friendly language as Python defined by following grammar:
```
identifier ::=
             (letter|"_") (letter | digit | "_")*

letter ::=
             lowercase | uppercase

lowercase ::=
             "a"..."z"

uppercase ::=
             "A"..."Z"

digit ::=
             "0"..."9"
```
Though it doesn't allows non-Latin characters, like CP does.

Possible solution is defining explicitly ranges of all scripts (Greek, Cyrillic, Hebrew, etc), but this doesn't sounds as a solution.


Yep. Thanks for finding this. I guess hardly anyone uses Mac OSX.


Very nice!


Look reasonable to me too.


Testing comment for slack


Oh I got it. Maybe we need few more test cases for that?


As suggested by the original author.


Nick's right.
I don't think we should see a half arsed fix for spoofing in FR.


The value of 'line' can be quite long. Is it really appropriate for a status that is used in tables? (Which should be short.)


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 112](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=605&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


sdasd


You are right. Sorry. I will create another branch and update files by using git tool. Thanks.


why?


Why don't you use just `synchronized` method modifier? It is equivalent and IMHO more transparent for this case.


Another comment


@chester1000 Do you know where can i change it? I don't have any clue where to search it.


agsdsdg


`fDimension` ain't required anymore.


Don't the brackets around 'options' cause the problem with startswith?


fix ticket #19 , add some syntax changes.


@twiro: Can you please add a commit to your pull request that fixes the release notes for 1.7.1? Either leave it empty, or maybe say: "Bugfix: entry_id must be unique"


https://github.com/nodejs/node/blob/5396baf7c12e118cf42e246d1d0773337edd23f1/lib/buffer.js#L175

I would use `b instanceof Buffer;`


有进步！


Can we replace this with StreamIO?


Verified this isn't needed in the original code, noticed it there as well, and should be safe to remove if you go the route of using the user_id in assigns.


You can't spawn wolf... you're run a different version of IM


Yes, the line starting with error is always short and at the beginning


Hello, I think this is related to django 1.9 no longer support python versions below 3.4


another one


@azbshiri Why did you commit code unrelated to commit message? Please revert this commit


If multiple requests comes in for the same url isn't there a possibility deleting this key will prevent waiters from being awaken?

Ex:
- Req A locks
- Req B waits
- Req C waits
- Req A finishes
- Req B awakens
- Req A deletes the url from the dict
- Req C never awakes ?


Wow, that sucks


Yes they do... hence the next commit that reverts this one (I dont know how to remove that one from the pull request).


Btw if you submit a new PR with your change for handling 3x, 2x, 1x I would be happy to review it and merge it :)


(or at least git checking part), I did not approve it


Maybe we can add a "cwd" argument to addOpenTypeFeatures.

Or better yet, use StringIO and set .name on it.


@magellan-13016
http://gnome-look.org/content/show.php/Windows+Se7en+Bootslpash?content=17445213016


> How about implementing this just in terms of findExecutable?

I see two downsides to that approach if I understand your comment correctly:

1. If the user installs multiple executables and he hasn't added the installation path to his PATH, he gets multiple warnings about his PATH not containing the installation path.

2. If the user's PATH doesn't contain the installation path and an executable he installs is already available on his PATH he mistakenly gets the warning about shadowing.

So I don't think we can avoid explicitly checking the PATH for the installation path.

What I didn't quite understand in the first place and what should possibly be changed is that `distDirIsInPath` (originally `isInPath`) searches `envSearchPath menv` instead of the original PATH.
It seems unlikely that the current definition would ever lead to a false positive (i.e. a user picks one of stack's internal bin-dirs as installation path but forgets to add it to his PATH) but changing the definition to use the original PATH might reduce the cognitive load for the next person to look at that bit of code.


Ah nevermind, reading http://www.tornadoweb.org/en/stable/_modules/tornado/locks.html#Condition.notify_all makes it clear the condition itself will awake all waiters and not behave like a traditional lock.


You mean StringIO?
Well currently feaLib only takes file paths as argument but it would be cool to have indeed.


Objects.hash() will simplify this code


`obj.view.window`. Keep in mind that `view` call will trigger view loading if it's not loaded yet. But I assume at the time of this call this VC is on screen which means its view is already loaded.


@kristofmoens Why is `sdkIncludes` a `Set` and not a `List`? As written, the order may differ from the old code. And the order of include paths might matter, right? Or it is not worth worrying about this...?


Python 3


The docs for Comparable.compareTo() have this: "It is strongly recommended, but not strictly required that (x.compareTo(y)==0) == (x.equals(y))"; the description of the interface elaborates a bit more.

Can we follow this recommendation here ?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1094](http://teamcity.monogame.net/viewLog.html?buildId=13141&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Yes, meant StringIO.

@brawer and I discussed this before.  He should fix it to take a file object and use its .name attribute.


Hi joachim,

Are you decompress the Windows 10 prefetch on the fly with the libagb ?

Thanks


This regexp can be reduced to
```javascript
/('angular2-now':\s*')([^']+)'/gi
```
by removing the escapes on ' and :


Adressed in https://github.com/commercialhaskell/stack/commit/c6151bba7ba20d351bc719bf2d44eb6e9283c4be.


Anyone reading the commit log is left in the dark as to why this was reverted.


Does StringIO have the name attribute? I see only FileIO having it.


I cloned the repository and included the libraries and it started working fine again.
Weird.


Thanks for your comments, don't worry tho, this commit is full of debugging units.


Needs one more space of indentation.


Я имел в виду автора. Вы ведь создавали этот файл не по служебному заданию ООО "КиберТех Лабс", так что оное ООО не может претендовать на интеллектуальную собственность. Пишите себя. Кстати, с новым годом :)


Apologies for joining this conversation. There are a few issues with this change which put Rubinius into a dangerous state, legally speaking. You have exposed yourself to potential liability. IANAL so this is based on some decades of copyright experience.

Relicensing to MPLv2 is a fine idea per-se, yet you need to go about it the right way. Removing the BSD text and replacing it with MPLv2 text is not the right way.

Let me break this down:

* You have no contributor agreement, so every patch is owned by its contributor. You can argue that trivial patches are excluded from this. Probably not worth it though.

* The original patches were made to a BSD project and we can assume they were licensed under BSD. (There is a legal gray area here since patches on a BSD/MIT/X11 project are not automatically licensed the same way. Still, it is a reasonable assumption to make.)

* It's worth noting that one does not "hand out copyrights." The author of a patch owns the copyright on that patch. By submitting it they (one assumes) grant the project a license to use and distribute the patch. One advantage of MPLv2 and other share-alike licenses is they remove this assumption.

* To fully re-license the project you MUST contact every single contributor, and get their approval. In effect they re-license their contributions, allowing you to re-license the whole project. We've done this in cases. It can be a lot of work (months) if you have many contributors. During that process you must either freeze new patches, or ensure they are licensed under both old and new license.

* You can absolutely NOT argue that any contributor who "disagrees" must notify you. This is not how it works. Copyright exists on all creative works by default, under international law, and expires only after the full period. If you distribute a work without valid license (as you are doing currently), you are violating copyright law.

* You can avoid that work by wrapping the BSD license in an MPLv2 license. What you are doing here is creating a derived product that has the MPLv2 license, and which includes BSD code.

* In that case, you MUST continue to respect the original BSD license, otherwise your derived work is illegal and you MAY NOT distribute it. (If you do, any original contributor can sue you, if they cared, and probably win.)

* The BSD license explicitly forbids removal of the copyright statements and the license text. You have removed these (from LICENSE text, as they were not in the source files). This is not legally safe.

* The simplest solution is to leave the original copyright statements, and the BSD text, in LICENSE, then add the MPLv2 statement and other copyrights ahead of that.

* Note that copyrights can only be claimed by individuals or legal entities. Projects cannot claim copyrights (or any other property for that matter) since they will have no standing in a court.


Oh, didn't realise this was an old commit. -_-


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


Hey sorry you had to do this, it was still on my list. Does this also has the new table?

Side question; What is nekoc?


Nice job


Already in the review commit https://github.com/smarterclayton/kubernetes/commit/c2767eb13b893b149037303c834ff3eca407880a#diff-71b26e1e133ec6d3c4da26366b6502acR136


Replacing tabs with spaces and calling it beautifying is disgusting xD


closes #109


If you use rild binary from TouchWiz you need libstlport regardless of the device variant, as you're not using it then you can directly drop it (e.g. do not build) :)


I bumbed the version file.


@hintjens thanks for weighing in, your extensive experience is much appreciated. It wasn't clear to me that the BSD text needed to be retained. I'll re-add it.


@RMistrick Please pull this to test whether this fixes issue #110


This has no issue with the actuall players, correct?


You probably want to remove the test code here:
https://github.com/Pro/dkim-exchange/commit/f396057fbea84c45ab4509a275f0f0ad00b854e4#diff-6263010e098ac376d6a3b7fdf4cf6763R35


Perhaps we should warn if an unknown `Quodlibet::` tag is found - this sounds like something quite bad has happened.


Closes #38


`string[]`


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 112](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=605&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:36:59


Thanks, already saw it and removed :)


People really use that notation?


Thanks!


> Are you decompress the Windows 10 prefetch on the fly with the libagb ?

No. Prefetch files are handled by libscca. Decompression for both libagdb and libscca are done in [libfwnt](https://github.com/libyal/libfwnt/blob/master/libfwnt/libfwnt_lzxpress.c). Note that there is still an [edge case](https://github.com/libyal/libfwnt/issues/3) in the decompression to fix.


The clone can just be 'git clone -b 1.0.8 "$LIBSODIUM_SRC" libsodium'. $LIBSODIUM_SRC should be double-quoted.


semantic.DeepEqual now ignores type meta comparisons.


Would it not be better if these were already in the correct (flipped) form so we would not need recreate them here in this cache?


Please do not sneak in changes under a different commit subject. You know and I know you are trying to remove C++11 build requirements, describe that properly in a separate commit (or rather, fix the original commit which introduces the bug (but I don't think you can at the moment because the commit history is too messed up)).

This is a big problem in your patch series. There are a lot of irrelevant things under a vague commit subject.


This addition is really a repetition of what's in the first paragraph of this Answer. I would also get rid of the link... noise.


why did you take axon subscribe out?


Hrm - I've been trying to reduce the complexity of our Errorf messages in general - there's no need to say much in most cases (i.e. instead of `t.Fatalf("expected error: %v")` use `t.Fatal(err)`).  I don't know that applies here - this is some of our very oldest scheme tests.  However, taking away the verbose messages doesn't seem like it helps much.  Will change it if you want it.


seeing this pizza image I realize we probably won't be able to support images taken at an angle like this. Only top down. The slices won't be evenly split if we calculate from this angle :) I know it's just for the detection algorithm but it made me think of it


Whilst I can agree about the first half of the comment, I think the link is useful as it gives a lot more info one why we don't use `--release` by default. It helped an absolute beginner like myself, so it feels like it would be useful for an FAQ.  :D


I'm not sure about this. In fact, I didn't want to commit it. But something like this is necessary to use the classes in languatool-tools.


Yeah, some very old code.


IzNoGoD pointed out, I just wrote the code ^^


*rogue

it's rogue

rouge is a shade of red


@brixen no problem. Good luck.


All or nothing please.  Having some buildings (e.g. the starport) go dark when disabled, but others not is horrible.


@paragonie-scott yeap. There is also `<string>array`, but it is really uncommon


So, this is a new version of my solitaire java project. I found something to help me move the cards, and also, a kind of new way to draw all the elements on the screen. Now I am using graphics and also a window adapter - window manager.


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


Tagging @mavasani


Already fixed


Do not need to check Linux. Do not need REQUIRED. Do not need RETURN shortcut. `SET(OpenNI2_INSTALL_DIR ${OpenNI2_LIBRARY})` this makes it not work.

Do not need to explicitly encode library suffixes.

But do put in path heuristics "/opt/include" etc.

I wrote this
```cmake
# - Find OpenNI2
#
# If the OPENNI2_INCLUDE and OPENNI2_REDIST environment variables
# are defined, they will be used as search path.
# The following standard variables get defined:
#  OpenNI2_FOUND:    true if found
#  OpenNI2_INCLUDE_DIRS: the directory that contains the include file
#  OpenNI2_LIBRARY_DIR: the directory that contains the library

IF(PKG_CONFIG_FOUND)
  PKG_CHECK_MODULES(OpenNI2 libopenni2)
ENDIF()

FIND_PATH(OpenNI2_INCLUDE_DIRS
  NAMES OpenNI.h
  PATHS
    ENV OPENNI2_INCLUDE
  HINTS ${OpenNI2_INCLUDE_DIRS}
  PATH_SUFFIXES
    openni2
)

FIND_LIBRARY(OpenNI2_LIBRARY
  NAMES OpenNI2 ${OpenNI2_LIBRARIES}
  PATHS
    ENV OPENNI2_REDIST
  HINTS ${OpenNI2_LIBRARY_DIRS}
)

GET_FILENAME_COMPONENT(OpenNI2_LIBRARY_DIR ${OpenNI2_LIBRARY} DIRECTORY)

INCLUDE(FindPackageHandleStandardArgs)
FIND_PACKAGE_HANDLE_STANDARD_ARGS(OpenNI2 DEFAULT_MSG OpenNI2_LIBRARY_DIR OpenNI2_INCLUDE_DIRS)
```


Yeah, it's different.


Minimally documented and written for my own use, with my own paths etc hard-coded into the scripts.


Glen,

All of these methods actually execute the very same G28 code.
Until one works, none of them will.

Please try the following:
Add the option
#define DEBUG_LEVELING_FEATURE
Load the system.

Issue the following commands
M111 S32
G28

As soon as it crashes, kill it and send me the logged output.
Perhaps that output will help me figure out what is going wrong on your system.

> On Jan 3, 2016, at 10:13 AM, a4jp-com <notifications@github.com> wrote:
>
> The print-head hits the bed when using this code. Both the home button and the "Auto home" in the menu—
>


One empty line is enough


It doesn't, but we can set it.


Привет


I think there is a file missing in this commit, the class `org.microg.tools.selfcheck.AbstractSelfCheckActivity` seems to be missing (and the build fails on my system because of this).


because the recovery had problems with bootdevice/by-name


If you're up for it the next thing is using skin shortcuts' templates to manage the widgets so users can have as many as they more as many home items as they want. ;)


Late link at least: #12100.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1094](http://teamcity.monogame.net/viewLog.html?buildId=13141&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:25:52


ok


.dylib should not be necessary here, CMake automatically adds the appropriate library extension. liblua.5.3 should be moved earlier in the path ahead of any lua52 / lua51 references.


The previous lesson plan had the students read docs with goal of identifying how to utilize an existing jQuery plugin, attempt a few different installation methods, follow a published tutorial to create one, then create one on their own.  The focus was on the students learning and doing.  Why the shift?


Nickel! Merci beaucoup !


This is a ton of content.


Similarly, move lua 5.3 ahead of 5.1


pour archive, le diff est donc visible ici https://github.com/regardscitoyens/the-law-factory/commit/155cdf51a9380f73da1064d9e97f681c5acc9ef2


We have stopped using sf.net for our files, they are now available at github.com/niftools
We also nolonger support NifSkope 1.1.3. The last official version is available here - https://github.com/niftools/nifskope/releases

Our NifSkope developer is working on version v2.0. I would recommend using that you can grab a copy here - https://github.com/jonwd7/nifskope/releases/tag/v2.0.dev4, we should be making a version available via the @niftools repo soon.(TM)


\\\(CA\\) , \\\(ISRG\\\) ist also possible withour quotation marks


et ce diff ci ici https://github.com/regardscitoyens/the-law-factory/commit/c876efaf7aa0c71d95e836b2dc8abbdf4f32f20e


Statements like this are a red flag for me.  Why all these files?  What is their purpose?  Recommend creating each one, after we identify its need/use.


Hmm, seems like a significant logic change for ratings - this is now writing ratings _all_ the time, default or otherwise. We should move carefully not to produce surprising results for users...

This could probably do with some unit tests (first) to prove there are no regressions around this


@matcracker But with spawn eggs?


On Jan 3, 2016, at 11:11 AM, Nick Lowe <notifications@github.com> wrote:
>
> It's sadly a lot easier to fix the RADIUS server that it is to fix many other products from many vendors.

  "fixing" the RADIUS server won't cause tens of millions of supplicants to suddenly start using the correct outer identity.


You can, but it is not necessary to detect valid CSS: @media followed by white-spaces followed by the shortest term not containing a "{" followed by a ; or {. It can actually be further simplified to "@media\s.*?[{;]"


Perhaps these could live in `Tags` or somewhere similar? It seems to me they're more to do with QL's tag definitions than then individual file formats and thus belong in that (single) place


what kind on error
??


What does this change from the default THREE.LinearMipMapLinearFilter ?


If you put
```
.. code-block:: python
```
before this block, it will be highlighted and more readable.


That's what it was added for.  unversioned.TypeMeta is skipped during conversion (there was a bug where it was still generated that was fixed in the same review comment as the other changes).


Those newlines are a bit strange...


Those that did break would likely be due to configuration error, not an intrinsic fault in the supplicant. I highly doubt it is anywhere near that number that will be affected.
Where there is configuration that is spoofing another identity, it should break.


@kyuupichan this commit breaks do_update_fee because dummy_adress is replaced by None


Welp.. I testing it out, the bots are scrambled, but my playercell... So confusing. It's doesn't listen to me :/


I believe the errors are ineherently connected to F-bounds and then the & does indeed always go through a TypeBound.


Help?


On Jan 3, 2016, at 11:08 AM, Nick Lowe <notifications@github.com> wrote:
>
> The existing practices allowed spoofing to occur in the first place and now fixing that could conceptually break some deployments. Holding back because of that concern and us having this discussion shows there is the potential for a real technical debt existing. It's practically the same thing therefore.

  You're going in circles and making the same arguments I've already addressed.

> I would absolutely love to see ""anonymous (no realm) or "{empty_string}@realm" enforced as the outer, mandating identity privacy use in all cases, we cannot do that as it breaks many systems that we do know about. That's certainly not theoretical.
>
> I wholeheartedly agree that it is insecure to not enforce privacy, it’s also an entirely different issue and question to spoofing. We don't need to conflate the two.

  I'm pointing out that if you want to do the "right thing", we can do the right thing everywhere.

  What you want is to do the "right thing" where it impacts my users and not yours.  But to not do the "right thing" where it impacts your users and not mine.

  That's not acceptable.

> Commercial SSO systems that integrate snooping RADIUS auth/accounting and wired/wireless management platforms should be integrating so that they do not use the outer identity for logging or access control purposes. They nearly all don't do that so it is incumbent on RADIUS servers to protect them as best they can from using spoofed identities.

  Show me that spoofing is a problem.

  Show me that the commercial RADIUS servers enforce the rules you propose.

> Nobody is being told, fuck you, upgrade - anybody can change the configuration of FreeRADIUS as they desire if they have problems.

  You're demanding that the *supplicants* be upgraded.  That is not acceptable for problems which have minimal impact.

> This is therefore just about choosing the best default behaviour that balances compatibility with security.

  That's what I'm doing.

> I’m trying to convince you that you’re taking the wrong tradeoff here, not battle :-) I really wouldn’t want to get your back up such that you close yourself off to changing this. I just want what’s best for the ecosystem. I think you do too.

  You want to enforce security where it doesn't impact you.  You don't want to enforce security where it impacts you.

  I'm trying to convince you that's the case.  You're not seeing it.

> The security argument against what FreeRADIUS has right now is that:
>
> 1) It doesn’t protect against spoofing for any account name that happens to begin with anon so the fix is incomplete. To solve that, we have to deny identities we would consider being anonymized in the outer from being present in the inner.

  You're not listening.

  We CANNOT enforce restrictions on the inner identity.  The inner identity is allowed to be ANYTHING.

  I gave you an example, and you ignored it.  Sticking your fingers in your ears and going <la la la la> is not a good argument.

> 2) With that in mind, what FreeRADIUS currently considers as being anonymized is too broad. Anything that starts with anon is too wide a brush, “anonymous” and “anon” are in contrast tightly defined.

  As I've said repeatedly, those rules are designed for a particular reason.  You haven't addressed my reasons, other than to say other choices are better.

  So I'm going to stop this discussion.

> Additionally, we lose privacy guarantees when we allow a practically unconstrained set of identities to be considered anonymized.

  It's RADIUS.  We can't *guarantee* much of anything.

  Mandating perfection is a pipe dream.  And a waste of everyone's time.

> Non-theoretical systems from major players that are affected in the SSO space are:
>
> Check Point
> https://sc1.checkpoint.com/documents/R76/CP_R76_IdentityAwareness_AdminGuide/62050.htm#o100097
>
> Fortinet
> http://docs.fortinet.com/uploaded/files/1279/fortiauthenticator-radius-accounting-proxy-guide.pdf
>
> Palo Alto
> https://paloaltonetworks.com/content/dam/paloaltonetworks-com/en_US/assets/pdf/tech-briefs/techbrief-user-id.pdf
>
> Websense
> https://www.websense.com/content/support/library/web/v77/radius_agent/first.aspx
>
> —
> Reply to this email directly or view it on GitHub.
>


> On Jan 3, 2016, at 12:25 PM, Nick Lowe <notifications@github.com> wrote:
>
> Those that did break would likely be due to configuration error, not an intrinsic fault in the supplicant.

  You're entirely missing the point.

  The problem is the misconfiguration of the supplicant.  There are tens of millions of supplicants which are misconfigured and using the wrong outer identity.  You can't simply wave a magic wand and say "updating the RADIUS server will fix the supplicant".

> I highly doubt it is anywhere near that number that will be affected.
> Where there is configuration that is spoofing another identity, it should break.

  How do you tell if it's spoofing?  Magic?

  Please argue using facts and logic.

  Your insistence on theoretical perfection over practical realities is a HUGE problem.


I actually meant fixes #2 :/


OK so there is a + before and after "-"  Thanks, syntax error gone!


Thank you.


@matcracker with the Mobs-AI branch yes, and you can edit the code for spawning them.


thanks!


Okay; I've revised the OCSP logic to check every hour starting halfway through, then do an update if the NextUpdate field changes. I think this is resolved. Thanks for your help @jsha and @tbroyer !


This is going to take a while. I did push the filter for it so I don't know why it isn't working.


Actually the "if (inst1.dword[0] != inst2.dword[0] || inst1.dword[1] != inst2.dword[1]) return false;" ensure that inst1.word[0] and inst2.word[0] are equals but I should maybe add a comment or use a || if code changes later.


I did try that, but my Sphinx installation does not render it very well - I may upgrade and try again shortly...but first I need to go for a walk with the dog.


@IIvec Again, values are not moving. Maybe you hit a wall of possible number of parameters to be tuned at once, either a limit of fishtest or the SPSA implementation.


Are you in Google Hangouts?

On 2016/01/04 Mon 2:09 a.m., Richard Wackerbarth wrote:
> Glen,
>
> All of these methods actually execute the very same G28 code.
> Until one works, none of them will.
>
> Please try the following:
> Add the option
> #define DEBUG_LEVELING_FEATURE
> Load the system.
>
> Issue the following commands
> M111 S32
> G28
>
> As soon as it crashes, kill it and send me the logged output.
> Perhaps that output will help me figure out what is going wrong on
> your system.
>
> > On Jan 3, 2016, at 10:13 AM, a4jp-com <notifications@github.com> wrote:
> >
> > The print-head hits the bed when using this code. Both the home
> button and the "Auto home" in the menu—
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wackerbarth/Marlin/commit/2a9c35fecb5db8358c1038c9038a5b53e815c5c5#commitcomment-15234457>.
>

--
グレン チャールズ ロウ
Glen Charles Rowell

〒060-0806
札幌市北区北6条西6丁目2-1朝日プラザ偕楽園506号室

携帯電話　090-2812-8177
TEL/FAX　011-768-8116
スカイプ　glencharlesrowell
ラインID　glencharlesrowell
Eメール　glen@a4jp.com

デザイン HP　http://a4jp.com （PC・iPad・iPhone・携帯 対応）
英会話 HP　http://agreatdream.com （PC・iPad・iPhone・携帯 対応）

--


Comments addressed in https://github.com/smarterclayton/kubernetes/commit/c54b8a6f0fef46441e2aa766af66c6227c7b722c


See c54b8a6
<https://github.com/smarterclayton/kubernetes/commit/c54b8a6f0fef46441e2aa766af66c6227c7b722c>,
but it looks to be correct

On Sat, Jan 2, 2016 at 9:08 PM, Clayton Coleman <ccoleman@redhat.com> wrote:

> Hrm, I'll add some test cases as another commit and track it down.  I
> think whatever behavior we choose is probably breaking some assumption, so
> I'll make sure to test against 1.1 and see what we did there just in case.
>
> On Sat, Jan 2, 2016 at 7:29 PM, Chao Xu <notifications@github.com> wrote:
>
>> The value here is the value serialized to the wire when you map a
>> conversion
>>
>> FYI, I did some experiment on Status, the apiVersion on the wire is
>> determined by typeToGVK, not the unversionedTypes. If you grep the code,
>> we never use the groupVersion recorded in unversionedTypes.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/smarterclayton/kubernetes/commit/67bb9031ab16b2220dd5ba08e07abc7f8836d43a#commitcomment-15231783>
>> .
>>
>
>


このへん、スレッドで処理しないと待ってくれないっぽい、めんどくさいなあ
http://docs.wxwidgets.org/3.0/classwx_thread_helper.html


I can't tell from this commit where the "pool management" you are talking about it occurring. As a minor point, a format string like `'{protocol}://{host}:{port}'.format(**connection_config)` would be more readable.


@brixen For good measurement we should probably Email all contributors and ask them to reply to an issue whether they're OK with this or not. I can set this up if needed. For future references it may be wiser to discuss these things first before changing them (though I have nothing against the MPL 2.0).


https://hangouts.google.com/hangouts/_/jgx5ywnjhtjqdl4tf76f6o3jlqa <https://hangouts.google.com/hangouts/_/jgx5ywnjhtjqdl4tf76f6o3jlqa>

> On Jan 3, 2016, at 11:37 AM, a4jp-com <notifications@github.com> wrote:
>
> Are you in Google Hangouts?


@IIvec OK, simply ignore my comment. ;-)


@joergoster : I guess something is still wrong because number of iterations is low compared to number of games played!


Ola... eh para imprimir a matriz 2D igual as que constam na etiqueta do
SIGEP java (Correios).

On Sun, Jan 3, 2016 at 2:15 PM, Stavarengo <notifications@github.com> wrote:

> @daviddgm <https://github.com/daviddgm> vc sabe responder o @maracaipe
> <https://github.com/maracaipe> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stavarengo/php-sigep/commit/ec3d23ba37525e1790cfd9f2203fc007dd08e306#commitcomment-15234256>
> .
>

--

Siga-nos e ganhe um brinde.

<http://instagram.com/dgmled>
<http://www.facebook.com/pages/DGM/196934773685708>

Atenciosamente,

DGM/David.


@hintjens the main thing I wanted to clarify is that going forward, new contributions are licensed under MPL-2.0. I didn't want there to be confusion about whether those were BSD or MPL-2.0. Unsure whether there's an effective way to do that.

Many thanks for all your extensive work on fixing the horribly broken copyright system.


Can you be a bit more specific? What exactly didn't work?


@hintjens er, *trying to fix*; it's hopeless. :)


I can do the following:

1) Move all the files generated included/modified by this commit to
rootfs-packages/puppyapps
2) Restore the old defaultapps that were there (it doesn't make sense,
but I can)
3) Continue working there, then zigbert can create pet files and
released them in the wild for more exhaustive testing.

But I need zigbert to approve my pull requests first, the script is
getting faster and faster


Why no spaces around `=` like the ones above?


commit message: `ignore _out_ Test`?


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


Спасибо большое) все учту!)
3 Янв 2016 г. 17:54 пользователь "Sergey Strizhak" <notifications@github.com>
написал:

> быстрый способ сделать ведущий ноль:
> а) hours = hours > 9 ? hours : "0" + hours;
> б) hours = ("0" + d.getHours()).slice(-2);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AlexBatohova/task_13/commit/092841599553e0c0d1f52fc409c04a2ffa7c1466#commitcomment-15234190>
> .
>


Eliminar esta pequeña prueba


It worked here: https://raw.githubusercontent.com/ddemidov/ev3dev-lang-python/master/README.rst


Tried to run `julia -i run.jl --max-long-edge 100 --save-epochs 10`
and it seemed to run out of memory on my consumer-grade graphics card.
dimensions are (66,100,3,1)
epoch 60, relative change 0.027967887
epoch 61, relative change 0.025238868
epoch 62, relative change 0.026262058
epoch 63, relative change 0.023287928
epoch 64, relative change 0.026921919
epoch 65, relative change 0.021259923
[12:27:55] ./dmlc-core/include/dmlc/././logging.h:208: [12:27:55] src/storage/./gpu_device_storage.h:39: Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: out of memory
[12:27:55] ./dmlc-core/include/dmlc/logging.h:208: [12:27:55] src/engine/./threaded_engine.h:295: [12:27:55] src/storage/./gpu_device_storage.h:39: Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: out of memory
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operatiogive you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to
terminate called after throwing an instance of 'dmlc::Error'
  what():  [12:27:55] src/engine/./threaded_engine.h:295: [12:27:55] src/storage/./gpu_device_stss || e == cudaErrorCudartUnloading CUDA: out of memory
An fatal error occurred in asynchronous engine operation. If you do not know what caused this ere MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operatiogive you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to

signal (6): Aborted
[12:27:55] ./dmlc-core/include/dmlc/././logging.h:208: [12:27:55] src/storage/./gpu_device_stora|| e == cudaErrorCudartUnloading CUDA: out of memory
[12:27:55] ./dmlc-core/include/dmlc/logging.h:208: [12:27:55] src/engine/./threaded_engine.h:295orage.h:39: Check failed: e == cudaSuccess || e == cudaErrorCudartUnloading CUDA: out of memory
An fatal error occurred in asynchronous engine operation. If you do not know what caused this ere MXNET_ENGINE_TYPEto NaiveEngine and run with debugger (i.e. gdb). This will force all operatiogive you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to
terminate called recursively
Aborted (core dumped)


@IIvec I think this depends on the machines participating. Afaik, higher core machines are playing more games per iteration.


D'oh! How was this working!? Nice catch.


var obj = new Object;

This assigns a new Object to a obj variable, but this value is never used in following code. Later you have this statement:

obj = JSON.parse(xmlHttp.responseText);

that overwrites original value.


time_frequency_compute_epochs_psd -> time_frequency.compute_epochs_psd

please check that the doc renders ok


Also don't know if this is intentional or not, but the array shapes are reverse the order of the Python version.


You have to rise an error to provide valuable information for library user that the call to OpenWeather API can't processed.


There were missing dependencies (morfologik) when trying to run org.languagetool.tools.DictionaryExporter in the command line.


That would be great. But if I understand the documentation correctly this won't work for such fake-multi widgets like your fTV skin uses. The hardcoded hidden lists are still required.

Or I'm too stupid to understand another manual again :P

To be honest:
I think 8 widget groups with 8 items are more than enough. I have a giant library, dozends of add-ons installed and I don't even use all of them.
Using the same view of the previously active widget in the library windows would be a bigger improvment :) Already gave it a try with properties and Container.SetViewMode(id) without success.


Close, but there are minor differences still (e.g. the special tag for half of those in id3). In the end I decided that they are not large enough to be put in a single place.


Ah cool. Yeah I'll remove the token usage later then.


It's much better to use some content variable instead of raw number.


не очень понял вашу проблему. Напишите поподробнее.


The problem with this function was that it was trying to be way too smart. Prioritizing competitors with fewer matches is doable without all this complexity.


Yes, I meant an options object, much easier to handle multiple optional
params.
On Sun, Jan 3, 2016 at 2:41 AM Robin Frischmann <notifications@github.com>
wrote:

> @ianobermiller <https://github.com/ianobermiller> What do you mean by
> object hash? An options object perhaps? Yeah true, I am fine with
> 'keepUnprefixed'.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rofrischmann/inline-style-prefixer/commit/d13bfe6e90643945ec32bbb9c86561646c930423#commitcomment-15233391>
> .
>


They are present in the UnifiedNlp subrepo. Run `git submodule update --init --recursive`


i could take a look for it too, if u want.. i just written a filter after the handler i think.. nothing more :)


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


fixed. for the doc, do you mean with plots or just the documentation for the new functions? If it's with plots then I need to figure out how to get them to build on a mac without throwing an error (apparently anaconda doesn't come bundled with a framework version of python, so it throws an error and says it doesn't have access to the screen)


@ianobermiller Jup, I think the same, only for 2 args I sometimes just use those as given before. Btw, it already got updated :P


Instead, we always start pairing with the competitor that has the fewest matches, matching them up with their closest rated neighbour. This should keep match counts in check in all cases except 3-competitor competitions (might be worth short-circuiting round robin for such cases).

We also bias towards the higher score as that's what matters in a competition. We want the upper order to be more precise than the bottom.


So much simpler, but accomplishes almost all of the same goals.


@copenhas playing with pry, just a demo branch. If you pull this down and `run iex -S mix phoenix server` when Player.join is called it will prompt you to allow it pry the code.

It should break at line 9, then when you are ready type respawn and it will continue the function and if verify passes then it should break again at 12.

I dig it.


Which backend are you using? That sounds like a Wx error. You should be able to use the PyQt backend or the OSX backend.


Great, I give it a try asap


Th


@YorickPeterse we're not changing the license for any existing contribution, that wasn't the intent. Prior contributions don't dictate the license for future contributions. Any prior contributions continue to be licensed exactly as they were.


Awesome to have all this, very nice :+1:


could you do
```
return execute_python_file(...)
```


Well done!


Hm in fact we don't need the whole function then...


@brixen In that case the license file should explicitly state that the BSD license applies to old code only, right now it looks like we're dual licensing both past and future changes.


It is the same problem on openATV. One colleague has copied a file from a NAS to the box using a Windows PC (copying from one network mount (the NAS) to another (the box)). Afterwards EMC displayed the file on the NAS with today's date, though mtime was older.
Unfortunately I don't have a NAS available myself.


this needs to be
```
int ret = RSA_verify(hashKind, (const unsigned char*) hash.c_str(), hash.length(), (unsigned char*) signature.c_str(), signature.length(), d_key);
```

to match the RSA_verify definition


but the console.log like the color red


but now no error code is returned when a test is skipped, is it?


@joergoster : Hmm, maybe my previous test was also OK, but 23 cores machine didn't report its iterations yet! We'll see. The ratio is high so far.


Oooops, sorry! :-/


ack


Solving spoofing is a higher priority issue than enforcing privacy. We absolutely should push towards ultimately enforcing identity privacy as a second phase after solving spoofing.

We should not just put our hands up, wave them around and say, lets do nothing at all or partially fix things so it's still crap 10 years from now with little to nothing solved. There's always pain when we raise the bar and improve things. This happens all over the IT industry, short term pain is sometimes necessary for the greater good, on the encryption from look at the deprecation of SSLv3, RC4, SHA-1, MD5 etc etc etc. Why should RADIUS be different from that?

If you think that this could be a painful change, lets find out through appropriate logging but not enforcement in FreeRADIUS 3.0

We could do the right thing everywhere by mandating privacy in all cases, but that has a much higher compatibility impact with existing systems that cannot be corrected with configuration changes alone. That's where it's manifestly different and not equatable therefore.

For any users that I have, I'll have configured FreeRADIUS appropriately as I want it. This is about me wanting what's best for the ecosystem, it's not about a personal want at all. A straw man argument of my users vs. your users isn't helpful ;-) It's not about that at all.

Your insistence on maintaining the status quo as much as possible and not being prepared to break anything to improve things is a HUGE problem. ;-)

You have the capability to improve and influence things because FreeRADIUS has been so successful and is used so widely, either directly or as part of commercial products. You should do so, responsibly.

Commercial RADIUS servers don't enforce these rules because they're inadequate and poorly designed. It's not an argument at all to compare to them. Every RADIUS server is broken here. That doesn't make the problem magically go away.

Spoofing is a real problem because in the aforementioned integrations, by impersonating another account, which can be higher privileged, this often allows a user to circumvent Web filtering restrictions, to falsely implicate another user for actions that they themselves have carried out or to get access to resources that their user account has no rights to. FreeRADIUS shouldn't be complicit in that at all so we should solve this properly.

It's a security fault where that is possible and the responsibility is shared. We can demonstrate all these

We can-and-should impose restrictions on the inner identity by default to stop spoofing. The RFCs were really poorly through through in this regard, actual implementations just as bad, it's disappointing. We can-and-should do better.


What I pushed is totally broken (it won't run). Hold on a few days - I've got more updates.


maybe you want just remove the complete paragraph? Since you only refer to the python docs^^


No problem. :-)


Is the default package 48 kHz? I had to include `AudioPackage13_48.mk` to make our audio HAL/driver happy, so assumed 44.1 kHz is default (in both AOSP and CM).


Hopefully version updates work.


No, no and no. Why didn't this go through a pull request? The prices seem completely wrong (based on the f2p server rather than the normal prices.)


It is a wx error, but I'm not sure why it's popping up. My default backend
in the .matplotlibrc file is "Qt4Agg", and it's not explicitly defining
'wx' anywhere in the script...

On Sun, Jan 3, 2016 at 12:02 PM, Eric Larson <notifications@github.com>
wrote:

> Which backend are you using? That sounds like a Wx error. You should be
> able to use the PyQt backend or the OSX backend.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mne-tools/mne-python/commit/4b866e009bbbaa3192906e4655280407f147deb9#commitcomment-15234648>
> .
>


Now it calculates: base ip, broadcast ip and first and last host.
You can introduce the mask in decimal format or at CIDR


wot da phuk


`and not skip`? why don't let the test shutdown everything itself, also yielding a message that the test was actually skipped than just returning nothing.


LGTM.


yes its now theme dependant, you and he know now how to patch other themes , make it visible to other users tommorow here.


@vlj ici


Maybe sphinx is calling a different backend of its own?


@gkuppelmeyer You want to memoize what you're returning (output) by storing it with some sort of key that uniquely identifies the corresponding input that produced it.  The point is that after the function has computed the answer and is ready to return the result, you want to store it first so that if you ever need the solution to that problem again, you can simply look it up in the table with your stored results instead of computing it again.

The natural data structure to use to do this in JavaScript is an ordinary object because an object associates string property names with values of any type.  So a possible strategy would be to create a new memo object outside of the function (or passed as an additional (optional) parameter to vorp_algorithm, supplying it with {} when it's undefined in the function's body).  Then instead of simply returning the result, set memo[string_version_of_input] to output first.  At the beginning of your function, you would check if string_version_of_input is already in your memo table and if it is, simply return it.  If it's not, simply have it run the same instructions that it's running now (with the additional aforementioned assignment statement before the return).


"All rights reserved" effectively means "you can't use this" - leave it to the license file


I think this has to go into a new method CPVRTimerInfoTag::ResetChildState() which gets called by CPVRTimers once just before starting to update the child states.


Made change to `self.new_by_filename(filename)`, as title is an attribute of the instance, not of the class itself.

The formatting above is a bit wonky though - some sort of strange auto-indentation going on in the WYSIWYG editor. Might be worth taking a look at.


@felixhammerl  thank you!


Thank you. You may want to check whether the AsymmetricKeyParameter object is a private key:
```
if (key == null) {
  var pair = (AsymmetricCipherKeyPair) keyObject;
  key = pair.Private;
}
else if (!key.IsPrivate ())
{
  key = null;
}
```


Vectors are objects in R that have the same mode of data.


"sys.executable ensures that the subprocess uses exactly the same Python interpreter as the test environment" - from Russell's comment on the last commit


Maybe there is something wrong with your Qt. Does this work?
```
python -c "import matplotlib; matplotlib.use('Qt4Agg'); print(matplotlib.get_backend()); import matplotlib.pyplot as plt; plt.plot([0, 1]); plt.show()"
```
If it does, then it suggests that your RC file is failing you somehow. You can use variants of the above commands to figure out why your config is failing.


Use GetTimerRuleId() instead of accessingv the member directly, please.


Oh it's in Sphinx... I guess it could be, but I'm not sure why it would want/need to do that.


The reason I ask about sphinx is because if I add the line `print('Backend:
' + plt.get_backend())` to the script then run it via sphinx, it prints out
the line:

`Backend: agg`

However if I run the same script with just `python
plot_compute_mne_inverse.py` then it outputs:

`Backend: PyQt4`

On Sun, Jan 3, 2016 at 12:17 PM, Eric Larson <notifications@github.com>
wrote:

> Oh it's in Sphinx... I guess it could be, but I'm not sure why it would
> want/need to do that.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mne-tools/mne-python/commit/4b866e009bbbaa3192906e4655280407f147deb9#commitcomment-15234714>
> .
>


yes, not much we can do about this. PyTest will recognize skipped tests as soon as we switch


Wouldn't DXGI_FORMAT_B8G8R8A8_UNORM make more sense?


thx bukkit xD


Yes, I know about that.  Seems like a poor man's solution though.


FWIW I use the dev version of sphinx. I had some error too ... and the only way to fix it was to use sphinx master. Give it a try to see if it fixes your problem.


ok


@patrys Thanks for your brief reply!
..I just setup a fresh install of saleor and the problem with this change is as follows:

in saleor/order/views.py#payment line 42 -> order.is_fully_paid returs „true“ (so no payment form will be displayed..)

in saleor.checkout.core.Checkout#create_order line 102 self.get_total() returns 0 (should be 247,00 $)

result: total_net and total_tax are not set properly when creating the order in saleor.checkout.core.Checkout#create_order

-> so I guess the error is in saleor.checkout.core.Checkout#get_total ... but I couldn't figure it out yet.

-> quick fix: I just changed the behaviour of  saleor/order/models.py -> get_total() back


Hm yes but I don't have that issue also not on network shares. Seems to be depending on kernel/image (filesystem bug in smbfs/nfs?!?). Also EMC doesn't even know it is reading from NAS :-)
Also users said its only happening after upgrading to VTi 9 which is also strange.


Kill me


I don't really agree with this change. Is it up for debate?


I think I'm just using the anaconda version of sphinx...

On Sun, Jan 3, 2016 at 12:24 PM, Mainak Jas <notifications@github.com>
wrote:

> FWIW I use the dev version of sphinx. I had some error too ... and the
> only way to fix it was to use sphinx master. Give it a try to see if it
> fixes your problem.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mne-tools/mne-python/commit/4b866e009bbbaa3192906e4655280407f147deb9#commitcomment-15234732>
> .
>


@magellan-13016
I hope things will sort out. I was interested in this before but now that I've learned about your job, I'm even more interested.


You can look. Just remember to go with the most compact solution.


This causes an elemental weapon to lose its elemental attack when using regular bullets - is that intended?

Can someone check in Aegis?


Look good?


@YorickPeterse that's what [this](https://github.com/rubinius/rubinius/blob/master/LICENSE#L1-L3) and [this](https://github.com/rubinius/rubinius/blob/master/LICENSE#L366-L369) are intended to say. The README files are also explicit.

Feel free to update with clarifying language if you can.


@hnryjms Unless you have proof otherwise, from my understanding it would still say "auto-away" if it was automatically set to home by auto away.  So making it "auto-away"=true would have the same issue. The only true way to do what you are asking is to figure out what does change in auto-away mode (as it says in the TODO).


approve changes


@bluedxca93
Please do so.


@dstoup David.  I don't know the answer to your question.  I also don't know how to test.  Given that vil1 is listed as a deprecated library, I opted to maintain current behavior rather than try to change the code.

I'd rather remove vil1 all together, but brl seems to depend on this library yet.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1095](http://teamcity.monogame.net/viewLog.html?buildId=13145&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Bonjour,
Cette modification provoque un bug, il faut également modifier la ligne 278 de:
		wp_enqueue_script('etendard_combined');
vers
		wp_enqueue_script('etendard_js');


In RHEL 7.2, when the initial setup is running during first boot, the header bar is seen with title named as __main__.py. This patch makes the intiail setup to run in full screen so that the title and the header bar are hidden.


What do you think about this way to avoid forgetting to set the RootPanel for the modal's parent?


The only issue I see is having multiple player needs help messages show up, which I am about to fix.


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


All good then !


@dstoup Yes!   There are many #if 0's that can be removed.  This is a big change already, and eventually all the #if 0's should be removed.

There is already too many changes.  Addressing everything at once seems to much complexity.

Removing #if 0's will be easy later.


PO, FOA a FOB dáva "0" do <ucet>


Shouldn't this be conditional on actually building with `stemmer`?


Comments with "TODO:" seems to be used in many parts of VXL.  I've been using that as welll.


is this variable being used? `QDate datum`


Yes, strange indeed. Did you get any feedback whether your patch here is solving the issue?


This is redundant, isn't it? That's what `Set` is for...


Yes, doh.


Fixing.


Ah alright, was just about to :smile:


Yes Matti,
One way to do this would be to put the array_flip command in the static class definition earlier. Do you have a specific suggestion on the most efficient way to define these static arrays?


Fix for #432


:+1:, Cheers!


It's correct. `NSMutableURLRequest` conforms to `URLRequestConvertible`. It's perfectly valid to pass in either the mutable or non-mutable form in this case.


Under what circumstances do we have duplicate filenames in `git-ls-files`?


@dstoup   I don't understand this comment.  VCL_SGI_CC can never be defined any longer.  Can you please provide more guidance about what you would like done?


@lucasart : I think we have more bad iterations without nodestime=600. Look at the ratio games/iterations in our tests (especially mine at VSTC). Why do you think that nodestime=600 is bad option?


if you know its DVD navigation window maybe you can enable it ? it really helps in OSD as the swipe up/down jumps 30 min :)


Ok thanks


See gitter :D


`IArborRender` is the base class. The `redraw` method declared in the `IArborViewer` class. Why do you cast `this` to `IArborRender`?


Oh, thank you, didn't know that one needs to run that on every update. Sorry for the noise then, now it compiles just fine!


@tambry actually with both value, this game show black screen, juste wait vlj fix it in d3d12 and after see this value is correct


Bootdevice not recognized when flashing. Where this one works.


Excellent :+1:


@dstoup Probably, but I don't know how to write a test to determine that.  Again, this is something that could reviewed and removed later.   Removing this function is not required for removing code for SGI compilers.


I'm not sure what the final semantics of `caml_stat_*` may turn out to be in case your changes were to be integrated into the runtime.  I agree that uses of `caml_stat_alloc` should be associated with `caml_stat_free` for consistency.  But what about the runtime lock?  Currently it is safe to use `caml_stat_free` without holding the OCaml runtime lock whereas `caml_stat_alloc` requires it, because it can raise `Out_of_memory` exceptions.  If `caml_stat_free` were to adopt a different semantics, many of its occurrences would need to be moved outside of "blocking sections".


Here and in several other places, you should return -1 in case of error instead of continuing


@eeckstein  can you please review this change ?


Yea, unfortunately the FactoryGirl generator has messes up the spacing :(


@dstoup   ??? I forgot to push the change that fixes this.  Sorry.


You should make sure at least one address worked


Please split this line


Why `void IArborRenderer.redraw()` and not `void redraw()`?


@dstoup Done.


Made change to `self.new_by_filename(filename)`, as `title` is an attribute of the instance, not of the class itself.

The formatting above is a bit wonky though - some sort of strange auto-indentation going on in the WYSIWYG editor. Might be worth taking a look at.


Often we don't auto-generate controllers so we don't end up with all these objects that often aren't ever used.


@FSMaxB I'm working on implementing support for objects in the expression parser (branch `feature/objectsupport`). I've introduced a new Node for this: ObjectNode.

I'm not sure how we should format an (JSON like) object in LaTeX, do you have an idea?


@dstoup Done!


Actually I see something that may be causing that. An admin testing it won't get more than one add, but a client testing it would due to a slight error with how the code is sent to clients. Even your fix wouldn't work because of how I set up the server variable. I was probably working with multiple languages and for whatever reason I confused global variable with public variable.


@mabdurrahman Great!! It's incredibly pleasurably to hear, that it helped someone. jFYI - i've uploaded the library into jCentral, so from now it's possible to add RtlViewPager into the project as a dependency, instead of copying the code.


Next-level debugging.


adding image


hi sebastian,

when developing on one of the nightly versions the console stated that the NewTabURL interface is deprecated. since there was not much information on the new interface, i was simply digging through the sourcecode of firefox itself: https://dxr.mozilla.org/mozilla-central/source/browser/components/nsIAboutNewTabService.idl

i did not intend to switch to it already though, but a user reported an issue that was eventually related to the removal of the NewTabURL interface: https://addons.mozilla.org/en-US/firefox/addon/bookrect/reviews/770002/

having done a little bit more research right now reveals the following bug report as the origin of the change and it also shows that a few developers have valid concerns about the interface change: https://bugzilla.mozilla.org/show_bug.cgi?id=1204983

thanks - simon


A pillar of dust! :D


This is interesting.  Another (maybe more idiomatic) way of accomplishing this would be to memoize resource.
```
def resource
  @resource ||= CommunityResource.find(params['id'])
end
```

Then can just use the "bareword" `resource` in this object.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1095](http://teamcity.monogame.net/viewLog.html?buildId=13145&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:24:02


No what I mean is that we should not need to call that array_flip at all... Why are those arrays even defined in that flipped form to begin with when we really do not care about the ordering of those unique terms in those arrays? Why not just do this:

```private static $globalforbid    = array('%th', 'adapth_depth', 'alias', 'aliases',...```
=>
```private static $globalforbid    = array('%th' => false, 'adapth_depth' => false, 'alias' => false, 'aliases' => false,...```

Why give the arrays in one form just to flip them to use them in another. Note that conversion to that is simple string replace "," with " => false". And you can even start using that value for something i.e. list something in the forbidden
"map" but actually allow it by setting it to true so that people can see from the code that this has been thought about and allowed anyway or vice versa with the allowed maps...

If the arrays were actually directly usable as maps/sets you could just drop that new cache system and use isset() directly you wouldn't even need that array_merge() cache (but it would still be useful) the only thing you would need is the unit-list caching.


@felixhammerl
I've just tested the new version but I've got some problem.
It seems the forge object is not defined in the right way (the same error before this commit).

It only works if I set a fake process var `process.versions.electron = 'fake-app' `


@bigdinotech @calvinatintel what do you think about this?


rly


@Baughn
I am not a GIT expert, but shouldn't that file be rather in the folder .git/hooks and without extension in order to get executed before commiting?

Anyway, as we have now a CI system for pushes and PRs, do we need this hook anymore?


A switch-case statement has some advantage here: Compiler will warn us, when status was added.


Line is obsolete.


This modification causes problems with the system. The client helpQueue is not updated. This can only been seen when using a non-admin client as they are the ones without the variable updates.


Is this still needed since it is only pulling the wanted refresh rate from VideoInterface now?


the argument shouldnt be an Id object, but a (Long id) right?


* comment should read "%d scheduled"
* insert a blank between "," and "m_iActiveChildTimers", please (hey, meanwhile you should know me.)
* could you please add comments for all the string resources you added? (like you did for 19255 already)


These are not required by our style guide, but I don't care personally… :)


Set in this case is a bit different. It's not caring about the associated value, so doing something like https://github.com/socketio/socket.io-client-swift/blob/master/Source/SocketIOClient.swift#L96 would overwrite their option if it were just a straight insert. It's actually acting a different type of collection, something like a dictionary with a predefined set of keys.


@dstoup This change will be used in a future patch set.  It was accidently included here.  Next patch set removes this.


Should be good compat wise


Honestly, I don't remember why it was...


Sorry if I screwed this up! With Bukkit ceasing to release new versions and removing old ones I had a heck of a time tracking some of that stuff down. Hope the rest of the plugin is functioning as intended :)


zoom fit isn't working anymore


Looks like this method was there in 1.19 already


My worker had problems as explained on the forum. I'm stopping the test!


No, please not another iteration over all timers! We have to find another way of doing this...


It's actually not your fault, it's because I had set it wrongly before and it worked because we both had cached copies.

I had to fix it for continuous integration, which I'm going to get set up really soon :)


postfix ofcourse ;(


@dstoup  Removed configure and configure.in.  Only CMAKE is supported.


Please report a platform, the compiler and conditions under which doesn't work, in big or smaller the party?


I'm no sure about this. Adding it as a model attribute will being it will get included with the attributes returned when the resource is being sent back to the server, unless `toJSON` is overridden to remove it from being included.


Are these stored as a number or as the name?


i didnt wanted to fix anything
i just wanted to let the message look a bit better.. nothing more :)

and thats the reason too, why i let the helpQueue as it is.. for every new request it will new calculate the message....

about the multiple messages im not shure atm.. i hoped, that always 1 message is shown for all current requests... ^^


I like


Actually it's LivesController, not LivesManager


Hi ualex73. In my setup I use OZW in Domoticz but I cannot include device (or exclude) It is correctly loaded in Domoticz and I can see the OZW control panle etc.

I use: OpenZWave USB
Version: 1.3-614-gadf3467

So can it be that issue 398 is not 100% ok?


@kghbln all fluff here? CI seems to be happy...


Do it here: https://github.com/metaron-uk/xbmc/commit/73edcb0f4231d9d35dc4bce9578ccbf27dfa097d#diff-d2cbe06d7cec0a05912311f32f371931R128


Use ResetChildState() here to replace above four lines.


uhm... today i havent time left for coding... maybe u will faster find a fix for the multiple name topic than me, but i will check it definitely tomorrow dont matter how far your r ^^


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


Yes.  See comments: https://github.com/vxl/vxl/issues/32


Nice :) Can't wait for that. Will be nice for testing updates. I forgot I wanted to ask you, do you think you should change the name of the plugin? It both attracts a certain type of people and repels others. Lately when we release updates on Bukkit or Spigot, EVERYONE comments on the name, some good, some bad. Maybe something more PG like WildMate or something? I dunno lol. Up to you. The other thing I was thinking about it tamed animals. Is there a check in the code to see if the animal is tame first? One problem I had on my Towny server was players would have a pen with 3 sheep, and even though they don't log on for 2 weeks, when they come back they got 80 sheep and their gfx lag is terrible(This happens because the chunks are loaded when players with houses in the same town log on). We could add a config option : "TamedAnimalsBreed: true/false" and a method to handle it in case other players/server owners experience the same issue. We had to manually slay all of the sheep because we didn't want to butcher everyone's sheep in the town.


Hmm, what configuration needs this? I wrote it this way intentionally so we'd get a compile-time error if any case of the enum were not handled. It compiles for me on g++ 5.2 and g++ 4.9 on Ubuntu 15.10, with and without the debug flags -- can you please paste the error you're getting?


Yay........ Go Java...

Hey if you get a chance will you start writing up a ChiefDelphi release post for all of our code? Maybe explaining some of the cool features. Don't post it yet, but we need to do so before the season starts. Include the practice week stuff as well, as that will be made public.


I read that using asynchronous requests is the most trustworthy way of requesting data from a website. The downside with this is that a callback function is required. I didn't find any solution to retrieving a value from a callback function so i don't know how to proceed with testing isJSON assertion, Connection Error.


I've already found the fix for that. I explained above that it was another contributor who modified my working code and messed it up a bit unknowingly. His modification was to increase the efficiency of the code, but unwittingly broke part of it.

BTW, a fast and easy way to add the player name to the message:
```java
"EAT_PVEH_contactAdminClient" addPublicVariableEventHandler {
	helpQueue = (_this select 1);
	if (EAT_isAdmin) then {
                _cnt = (count helpQueue);
                if(_cnt > 0) then {
                     _cnt = _cnt -1;
    		     systemChat format["**** A player needs help ****",helpQueue select _cnt];
                };
	};
};
```


spurious 'git' above causes `CompilerException java.lang.RuntimeException: Unable to resolve symbol: %3git in this context, compiling:(/private/var/folders/7d/rrx9jwvj3s37wzwdp0mb87hc0000gn/T/form-init9014234768548886538.clj:1:1)` on `(load!)`


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


Windows 7, MSVC2013 64bits. It's weird it's only happening when using a smart folder


Tested with the following main:
#include "mainwindow.h"
#include <QApplication>
#include <QLabel>
#include "imagefilter.h"
#include "grayscale.h"

int main(int argc, char *argv[])
{
    QApplication a(argc, argv);

    QImage myImage;
    myImage.load("cat.jpg");
    ImageFilter* filter = new GrayScale();
    filter->apply(myImage);
    QLabel myLabel;
    myLabel.setPixmap(QPixmap::fromImage(myImage));
    myLabel.show();

    return a.exec();
}

The image for testing should be in the directory with the executables.


Yes. but doesn't work


It's better to use something like:
function IsJsonString(str) {
    try {
        JSON.parse(str);
    } catch (e) {
        return false;
    }
    return true;
}

Your code just checking provided value is is not an Object.


console.warn is rouge


Also when zooming over the zoom fit value, you have to press fit twice to to zoom to fit.


Yeah good commit! <-- Example comment on a commit that i pushed.


you are both terrible


Thanks! :D


Yes Matti, I thought you might suggest something like this!
Ok, I'm too tired to do more today, but I'll do this tomorrow.  Good idea.  Thanks.


Just noticed a bunch more are `CONTACT` too


@bluedxca93 & @magellan-13016
I'm starting to realize that Git is way more useful to mankind than all social networks put together.


It needs to be in .git/hooks, yes, but there's no way to get it there except each dev manually linking it in. Unfortunately. Anyway, I wouldn't want to make it obligatory.

I still think it has some value, since you get faster feedback with this than with Travis.


why would you even need forge? it should require net and tls and go down the node shim, without forge...


@paksman yep same. and someone else is also encountering the same issue:
http://forum.cyanogenmod.org/topic/120258-compile-from-source-code-failed-field-has-incomplete-type-struct-sysinfo/


HI Akiff:  sorry for the late review of your homework.  It's not a great excuse, but my sickness for the past 6 weeks derailed everything. Thank you for submitting this homework.   I ran it and didn't see a problem.  Can you be more specific on the question regarding the "last line" that you needed help to figure out.  Good job on this assignment.


If you make nodes_per_bucket a std::array this line could be shortened to `nodes_per_bucket.fill(8);`.


I've now merged all these changes with my own.  On the development branch.  I'll run the system tests tomorrow after I work out a bug in some other code I'm using in the tests.


I need to read up on them again but I was thinking of moving the hidden lists into the same group as each widget (they're hidden so it doesn't really matter where they are in the actual code) and then seeing if I could build them into to template.

As for the widget/media views I could unlock them so the user simply changed the media view to be the same as the widget and Kodi would remember it.


@bholagabbar do you mind creating a unit test for this controller?


No need for it. Already found a way with properties :) Too many options, too many confused users.
Don't think I'll get it done today but it seems to work great.


You're right, the name was meant as a pun but really people are either too immature or too politically correct to handle it, I never saw this becoming so big an issue, lol... We should change it. I think we can do better than WildMate, though! I'll think about it.

About the tamed animals, you're right - it gets really intensive in that it kills the server and the clients, so there needs to be a better algorithm - otherwise the number of animals increases either too fast or doesn't at all. We can perhaps try to check for animals that are confined to small spaces, and also implement some sort of population growth limit in that it doesn't exceed a certain number (i.e. less sheep will go on mate mode if there are too many sheep around)

I will definitely have a go at these soon, and will post updates here.


Very cool and will help a lot with triaging.


@stavarengo @daviddgm a documentação somente especifica que é um campo reservado ao usuário. vou criar um pull request depois para deixar a edição livre, para número de pedido, ou qql outro texto desejado.


Thanks! BitHub has sent payment of  $29.74USD for this commit.


you could chain the constructors (i.e. DRY)


Forgot to say, changed !queue response.


commit message misleading: this commit was about revision based on exp interval scale as discussed in the agreement with Pei in https://groups.google.com/forum/#!topic/open-nars/DiCktvz3uqE


ups. vaya error :dancers: sobre lo del django volia evitar fer-lo servir
Et fa res dir-me el teu User ID i aixi miro les pelis que fallen?

Gracies per les propostes. Em semblen bé, pero deixem fer algún test més!


Why do you need the parent link index to serialize the sensor measurements vector?
Nothing is assumed in BERDY papers about the order of sensor measurements vector y.


You have several files, just spread around. Consider using the Groovy template project as a backbone for these files (i.e. so that you have a completely buildable system, with tests separated and run)


TMI


Since I'm using the new thing, a couple of existing issues:
- /advmodelshading + gl.{Unit/Feature}Shape doesn't work with shaders, should it?
- Unit/feature shapes are rendered differently depending on the camera (and not just object) position, so there happens to be an extra "fog" of sorts rendered on the object if the camera is zoomed out or at bottom right map boundary. http://i.imgur.com/1ZRX2yZ.jpg vs http://i.imgur.com/g973Ahd.jpg . Lighting also affects gl.{Feature/Unit}Shape, is that wanted?


Cool, I was reluctant to open up all the views when I spent so long getting to work originally.


why are you multiplying by hand?
```return purchases.size() * 100```


And GitHub didn't even do it


now going fully .net 4.5.2?


Why is this the argument of constructor? I think it's better to initialise `final` field in place (i.e. right in declaration).


I'm certain you don't need this :)


So if they Google me now, your apps will show up :p


consider turning this into a Spock test instead of a JUnit one


useless comment


Remember fsharp/FAKE#1044.


Also in `bnt_time_varying` the sensor order is arbitrary: https://github.com/iron76/bnt_time_varying/blob/7b5b4b969eef5b5b494754b701c81b8f5e419411/experiments/icubFixedBase/DNEA/2015-02-iros/main.m#L28 .


This should have a package-level visibility. I don't see any reason to derive this class.


same, you're stating here what the code does, not what the intention is. These comments are just noise.


Даровки


excellent job @joschiwald :+1:


same general comments apply as in the original Java file


I believe it's better to name it `ChromeCastSpontaneousEventListener`


There should be a comment here explaining why this throws an exception rather than being `= delete`ed. In fact this should probably be an assert because move operations aren't supposed to throw.


WTF???


public is the default visibility in Groovy


Perfect!


you probably don't need getters and setters here (in Groovy in general)


instead of constructor chaining, as in Java, you could have a default parameter value in Groovy, with a single constructor:
```public CreditCard(String name, int amount = 0) ```


Why not to merge this feature into our master? It works properly as long as I use it.


There should be an `ChromeCastSpontaneousEvent` class, which will hold the entity and some additional data. Maybe it's worth to have an enumeration with type of spontaneous event. There should be a single method in listener like `spontaneousEventReceived`. This way it will be a most scalable and backward compatible in case new entities will be added in newer releases. I mean you will not have to add a method in implementation when you upgrade to the next version of library.


consider using the `+=` on the other lines also!!!


well, other test skips are also not recognized, I don't see any point in issuing a message here. FWIW this test will be refactored soon with the switch to pytest


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1096](http://teamcity.monogame.net/viewLog.html?buildId=13149&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


What's the purpose of using `CopyOnWriteArraySet` here? Why not a simple `HashSet` or `Collections.synchronizedSet(new HashSet<>))`?


@KraigM Wait really? I guess I never tested the API for that transition. If that is the case, I'm a (sadpanda).


reack


Actually these were not the cause for kernel panics, but the KEYS commits, which were producing page faults during GC.

The touch issue with MultiROM is still there, but intermittent. I think it has something to do with Synaptics touch driver. May be Sultanxda can help.


The function of this app is to turn an android smart phone into a cloud based PC Tower.


Yeah, that actually seems better


:wave:


Before that, i'll just quickly introduce a concept of a multi-threaded efficiency: it's loss in efficiency  by adding new threads. Efficiency can be measured as number of nodes (or just plain time) to reach certain depth observing locally on a particular position, or just ELO loss if you look in full games (which is what I will focus on).

I claim that this efficiency, in theory, can be measured even in overloaded scenario (where number of search threads is higher than number of h/w

available threads). In practice, there could be several problems where overloaded scenario is not stable (for example time management can be CPU starved).

Luckily, SF does appear to stable in overload scenario.

I'm  not saying that findings in overload scenario will be exactly those found in environment with real h/w, but there should be enough correlation between the two, to be able to preliminary test various ideas in overloaded scenario.

Some things that will make the results different is:
- h/w threads must be fully loaded (100% cpu utilization) to be able to project the behavior in low h/w thread scenario (but same number of spawned threads). In other words, if there is any blocking/idling in high h/w thread scenario, it may disappear in low h/w thread scenarios, artificially inflating efficiency
- similarly one can not replicate amount and intesity of data races, atomic operation collisions, spin-locking on small number of h/w threads
- data propagation (cache coherency) between remote CPUs (especially in case of NUMA machines)

The best I know atomics, spin-locking, idleness are no or minor issue in lazy-SMP SF, but data races/propagation (in shared TT) cost is likely significant....

Your patches which try to eliminate duplicate search/eval work (or how you refer to it as reducing split-stream effect) aim to increase this efficiency, and should show effect on gameplay in overload scenario.

To demonstrate the point a did a "short" (well, took me two full days) tournament with two SF builds, one just before your original patch (Dec 14) and one with just your patch extra (Dec 18) using various search thread counts in otherwise identical environment (including number of h/w threads available).

My machine is 4 core Haswell, but I run it with HT  enabled, simply because I generally use this machine for many other things than just chess analysis.

In recent past, I measured that the gameplay in such (HT) conditions peak around 5-6 search threads with latest SF. I picked 6 threads as a base for multi-threaded comparison, and tested with overload scenarios of 24 and 48 threads. Intentionally picked 24 because it's (2^2)x6 and more importantly the highest number of threads that rely on the lookup table (unlike >24 threads) for the skip policy. Chose 48 as a double of 24.

Here is the outcome:

    Program                                                          Elo    +   -   Games   Score   Av.Op.  Draws

  1 Stockfish_15121822_x64_bmi2 threads6  :   47   22  21   250    58.0 %     -9   73.6 %
  2 Stockfish_15121419_x64_bmi2 threads6  :   38   21  20   250    56.4 %     -7   76.0 %
  3 Stockfish_15121822_x64_bmi2 threads24 :   37   22  22   250    56.2 %     -7   73.2 %
  4 Stockfish_15121822_x64_bmi2 threads48 :   25   21  20   250    54.2 %     -4   76.4 %
  5 Stockfish_15121419_x64_bmi2 threads24 :  -50   23  24   250    41.4 %     11   69.2 %
  6 Stockfish_15121419_x64_bmi2 threads48 :  -97   22  25   250    33.8 %     20   66.0 %

(Btw, these are ommon engine configs: 512MB cache, 6ply opening, 5man syzygy, TC 1min+0.5Sec, ran with Arena3.5)

Error margin (+/- 22 ELO average) is farily high, but even taking the worst case it's evident that your original patch had major impact on multi-thread efficiency with high search thread count. Again, we could not expect this gain amplitude to be replicated on real 24 or 48 core machine (if I remember Fishtest measured +8ELO for 20 threads), but my point here is an induction in testing methodology: If your original patch affected multi-threaded efficiency in an overloaded scenario, further improvements in skip  policies (for example, beyond 24 search threads, which, my understanding is, considered suboptimal in your opinion), should also be visible in the same environment.

If these results are to be believed (neglecting error margin), your original patch made a hugh leap forward even for 48 threads (in fact, this is where the gain is the largest). There does not seem that much room from improvement (but we can always hope for more):

Before your patch, loss in effciency was about 45ELo when doubling number of search threads (-88ELO from 6 to 24 threads, and another -47 from 24 to 48), or total -135ELO from 6 to 48 (btw, i'm neglecting the fact that 24&48 search theads were actually running on 8 HT threads, which, as previously measured, gives about 7% throughput (nps) imporvement relative to 6 H/T threads; thus the real loss vs 6 threads might be a bit larger)

With your patch, the average loss in efficiency is only like (47-25)/3 = ~7ELO, when doubling search threads. Huge improvment - one more time, congratulations on this finding and eventually the fix. Not only it will be hard to improve upon (I can imagine that even just extra context switching cost is a measurable ELO loss), but it will also  be very hard to verify the gain on real hardware.

In general, I believe, this metric (ELO loss when doubling search thread count, but on same number of h/w threads) is something that should be calculated frequently. It's fairly cheap to do it - even running it on a  signle core (and run with 1,2,4,8,...,64, 128 search threads) should be meaningful. I bet that there are occasionally patches that provide gain in say 3 core tests, but would actually be loss in 23 core test, but for whatever reason not tested. This metric could potentially isolate/eliminate such patches. On the other side, many patches are probably orthogonal to multi-thread efficiency, but again due to relatively low overhead of metric calculation, it might be worthwhile to do it anyway.

Regards,
Aleks


For clarity, playerName might be a better variable to use than i.


this is my first comment


Here is not everything innerHTML


When I  copy pasted my previous post from an editor, seems like I somehow missed my introductory paragraph. Not much, but here it is anyway...
-----

Hi Günther,

I'm fairly new here, but I'll like to comment on how potentially new improvements on large SMP machines could/should be tested in your local environment before moving on fishtest available machines.


Why do you put `customData` as the parameter? It should be a part of `Media` entity. I think it's better to have a builder for `Media` object, which will take mandatory field values in constructor and other you can fill with chain methods.


Creating a shallow copy of an object is a very common operation that is not specific to the VORP problem.  For better code reuse and improved readability (don't want reader to think this loop is directly related to the max VORP calculation), it should be factored out in a separate function.


If you are changing it from `@JsonIgnore` to `@JsonProperty` then you must configure the field to be ignored when there are `null` value set. This is for backwards compatibility.


which other test skips? At least they display a message, now it would immediately return^^


Maybe this is a bit nitpicky, but shouldn't this target be simply named after the file it generates: `ocamlbuild.install`


You should configure this field to be ignored during serialization when there is `null` set. This is for backwards compatibility.


I'll add an extra note to the LICENSE, I feel the current setup is too ambiguous.


OK, for all of the comments, I'll do some experiments and try craftr with
Sadie and give you more feed back.

On Sun, Jan 3, 2016, 4:36 AM Niklas Rosenstein <notifications@github.com>
wrote:

> As I said, the linker would link it *twice*. The file is already listed
> in the linker script. You would get
>
> [4/6] gcc Link /home/niklas/Desktop/baremetal-hi/build/test.elf
> FAILED: arm-eabi-gcc /home/niklas/Desktop/baremetal-hi/build/baremetal_hi/obj/test.o /home/niklas/Desktop/baremetal-hi/build/baremetal_hi/obj/startup.o -Wl,-T,baremetal_hi/link.ld -mcpu=arm926ej-s -nodefaultlibs -nostdlib -nostartfiles -ffreestanding -o /home/niklas/Desktop/baremetal-hi/build/test.elf
> baremetal_hi/obj/startup.o: In function `_Reset':
> (.text+0x0): multiple definition of `_Reset'
> /home/niklas/Desktop/baremetal-hi/build/baremetal_hi/obj/startup.o:(.text+0x0): first defined here
> collect2: error: ld returned 1 exit status
> ninja: build stopped: subcommand failed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/winksaville/baremetal-hi/commit/fb02d83b533036fc274b884d6a83e198a185b116#commitcomment-15233592>
> .
>


not in our test system anymore. Skips are not recognized, they'll just not be executed. PyTest will deal with that at some point.


@hnryjms Turns out you were right https://developer.nest.com/documentation/cloud/how-to-structures-object.


Ok. Thanks! Did not know that everyone needs to link it in manually...


Rather than discarding this work altogether I'd recommending moving it to a new repo and making it compatible with both [CurieBle](https://github.com/01org/corelibs-arduino101/tree/master/libraries/CurieBle) and [arduino-BLEPeripheral](https://github.com/sandeepmistry/arduino-BLEPeripheral) if possible. I could start this repo if no one else has interest in doing so.


Оу, дякую. Так, цей переклад значно кращий.


Hrm, I'll add some test cases as another commit and track it down.  I think
whatever behavior we choose is probably breaking some assumption, so I'll
make sure to test against 1.1 and see what we did there just in case.

On Sat, Jan 2, 2016 at 7:29 PM, Chao Xu <notifications@github.com> wrote:

> The value here is the value serialized to the wire when you map a
> conversion
>
> FYI, I did some experiment on Status, the apiVersion on the wire is
> determined by typeToGVK, not the unversionedTypes. If you grep the code,
> we never use the groupVersion recorded in unversionedTypes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/67bb9031ab16b2220dd5ba08e07abc7f8836d43a#commitcomment-15231783>
> .
>


Since breeding is in, there is no reason to have Hitmonchan and Hitmonlee in the wild anymore. You can just breed the one you get from the Fighting Dojo to get Tyrogue, and then evolve him to get the other two Hitmons.

Also, since it is a cave, there should theoretically be no need for separate Water data for Seafoam Islands.


If you have the dns-sd tool, the following command should work. You probably want to start librespot first.
If the advertisment is up but librespot isn't running, the other clients will blacklist the name, until you restart them.
```
dns-sd -R NAME _spotify-connect._tcp . 8000 VERSION=1.0 CPath=/
```
librespot currently lives in a magical world where everyone's a pony and they all eat rainbows and poop butterflies, and nothing wrong ever happens, otherwise it will just panic. This is just me being lazy and not handling errors properly :smile:. My priority at the moment is to make it work in this magical world, and then port it to the real world.

In this case, it your browser requested a favicon. The request does not have an `action` parameter, so librespot gets confused and panics :smile:


@magellan-13016
That's good news. I would like very much the problem to be solved me too. Yeah, I know all about Configurable. Stark can do the job nicely and without unnecessary overkill. That's just my arguable opinion.


This is something that should be handled by the AVI dumping class, not the renderer.


Well, if you really regret what you have done, then do this:

	git pull
	git reset --hard 6c8aae2ab75d2f5170c4bc4f1c0781c6bfd11777
	git push -f upstream testing

or something like that, then each to their own and happy new year..


`checked: profile.send("#{gender}?")`の部分はf.radio_buttonがよしなにやってくれる筈だから、わざわざ書かなくてOKのはず！


PEP8 에러
```./model/reply.py:21:80: E501 line too long (81 > 79 characters)```
줄 내려


@strahlex - there's an easier implementation in next [commit](https://github.com/araisrobo/machinekit/commit/9954afe543f01c1d312dfe5b6ce58350b5dbe0fa) to prevent pooling earlier than poll-Interval.
Is it an acceptable approach? (adding a ZMQ pipe is a very complicate task for me :-)


Maybe we should a `Integer` here?

* - Performance
* + Elegance


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


1.0.16 at least by default configuration still has the same effect. Firefox network panel shows a red square for these connections and Request tab has "Connection: Keep-Alive" field listed, so it's likely it's related.

It messes up some AJAX controls, because the browser makes the app think it's an active but endless connection going on, and UI gets stuck until you reload the whole page.


Tous ces changements ligne 32 39 43 et 50 doivent etre revertés, ils cassent l'affichage en noir du mod en cours dans la barre (pages interventions et amendements)


Although this is not wrong, why not check if `pn` is zero instead of letting an exception happen?
```
if (pn != 0):
    p_n.append(w* math.log(pn))
else:
    p_n.append(0)
```

Exceptions are supposed to be used for, well, exceptional conditions, and it seems having `pn == 0` is a likely occurence.


Thinking perhaps the time check should come before the check for `maxShrinks`. In order, we should continue shrinking if:

- not past the time limit
- not past the max shrink count
- not past the max shrink depth


vc escreveu supermercado erradondepois de corrigir


Fixes https://github.com/eblot/pyspiflash/issues/2


I was reading instruction about OmniAuth at https://github.com/intridea/omniauth. It didnt say anything like "install OmniAuth gem to gemfile with this: gem 'omniauth'. Why did they not mention it?


Bon comme je le sentais, il y a des modifs dans mod1.js que je n'arrive pas a retracer mais qui devraient l'etre parce qu'elles sont cassantes : la gestion de la colonne dédiée à l'étape complémentaire pour les textes en cours ne marche plus après ce commit (exemple /articles.html?loi=pjl14-psi )
Il y a peut-etre d'aurtes choses du coup...

J'insiste : ne pas pouvoir lire un diff est vraiment vraiment chiant :p


It appears the 202 is already in the standard gEDA symbol library.  How do I get access to the other symbols? Components: 74HC4051.sym, TS3A24157.sym, TEENSY31.sym, GE_EHHD.sym


There is no hard and fast rule that sysfs attributes will included a trailing newline. By convention though, pretty much all do include a trailing newline.


@magellan-13016
Can you put up a screenshot for one to have a representation of the gap between the left border and the menu?


The wiki page on the Circuit_Boards repo explains it.

On Sat, Jan 2, 2016 at 8:23 PM, Nick Novak <notifications@github.com> wrote:

> It appears the 202 is already in the standard gEDA symbol library. How do
> I get access to the other symbols? Components: 74HC4051.sym, TS3A24157.sym,
> TEENSY31.sym, GE_EHHD.sym
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chicagoedt/RMC_Electrical/commit/4307f042143b208ee5cdef2636773c071aaeccd6#commitcomment-15231989>
> .
>


ff


I'm sorry, I meant the gEDA_Symbols repo.

On Sat, Jan 2, 2016 at 8:27 PM, John Sabino <johntsabino@gmail.com> wrote:

> The wiki page on the Circuit_Boards repo explains it.
>
> On Sat, Jan 2, 2016 at 8:23 PM, Nick Novak <notifications@github.com>
> wrote:
>
>> It appears the 202 is already in the standard gEDA symbol library. How do
>> I get access to the other symbols? Components: 74HC4051.sym, TS3A24157.sym,
>> TEENSY31.sym, GE_EHHD.sym
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/chicagoedt/RMC_Electrical/commit/4307f042143b208ee5cdef2636773c071aaeccd6#commitcomment-15231989>
>> .
>>
>
>


Not quite.. This appears to work only with a new file being created.. If I open up an existing file, the black becomes white problem returns.


Added link


This dropped the size of the airports.json file to a little north of 1,000 (from 6,700)


@darlingtld 查看“上一期”时需要该期完整的赔率信息，目前lotteryOdds里只有一期的赔率，如需测试则需要生成一些之前期数的赔率。


This should've worked?


This is wrong


I will modify it.


good work! testing now