Agree @deads2k, I wanted to be sure that I wasn't missing anything.  I
think it should be a follow-up since it's not strictly necessary to finish
the rebase.

On Friday, August 7, 2015, David Eads <notifications@github.com> wrote:

> @pweil- <https://github.com/pweil-> @deads2k <https://github.com/deads2k>
> Is this really the only place we use the EtcdClient from the MasterConfig?
>
> That's all I saw right off and it can be factored out without too much
> trouble. I don't care whether you do it here or not. I just want to make
> sure we don't forget to clear the debt.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pweil-/origin/commit/31785ae4ba2d106919b9a3ce5acf4ec0ddc28706#commitcomment-12590835>
> .
>


Oops, checked in the spring PID file again... :smile:


Both the selection and highlighting work backwards. As you can see, you now have the state in two places and you need to keep them in sync manually. Instead, they should both be computed properties on the item view, which can be passed in when the item views are created.


this should probably be:

```
   "mbed-os": {
        "mbed-drivers": "*"
   }
```


Oh and structures work now and can build stuff. A lot of features went from not feasible to already working, or only 1 line of code to add, as a part of the action system. The game is a lot closer to actually being a game with this :D


Add `'args: ['size, -size']` so that this logic still works when eval is enabled.


Replacing LEFT SEMI JOIN with JOIN changed the logic of the query. LEFT SEMI JOIN  is equivalent to SQL EXISTS. JOIN could potentially return more rows than you expect. You can try to move the WHERE condition into the ON condition like ON (q07_cat_avg_price.i_category = i.i_category AND i.i_current_price > q07_cat_avg_price.avg_price)


The `User` model shouldn't be defined in `app/models/order.rb`, it should go in `app/models/user.rb` (where it already is :smile:). Just move the `has_many :orders` into that file and delete the `User` class here.


adb pull on system version (installed by our project (with arm libs removed)) instead of the data one installed from the playstore


Still working in the release of binaries. The fix is about the issue we were getting while searching. https://github.com/arvidn/libtorrent/pull/72


We should have a flag that disables this new mode - `--userspace-proxy` or similar - and default it to true.  That gives us more freedom in checking this in.

When do you dry up for real?


Typo ``wagtaiforms``. Should this also have .html on the end?


So these checks are independent of each other?
If we're after High Priority Earthquake Areas, then we will show the button if we have both Low Priority Earthquake Areas & High Priority Flood Areas?
However we would not show the button if we only had Low Priority Earthquake OR High Priority Flood?
This seems like faulty logic.
Why not simply apply these as a filter to the earlier query if we only wish to show this button when we have High Priority Earthquake areas?

I still don't understand this conflation of priority with predefined areas at all but if it really does exist then it should be consistent?


@laverdet This is required on Windows. Also, I don't think it is the solution, is more like a workaround that should hopefully work most of the time. @sdarnell understands this much better than I do.


What is this model for? I couldn't find it in the exercise.


If you want to make a copy of 6 items, use `this.fontMatrix.slice(0, 6);`.
If you don't want to hardcode the argument length, use `this.fontMatrix.slice();`


Shouldn't it read STATUS_TIME_OUT?


NB I see this is exactly what the actual method called does...so in the case I mention above, ther user would see the Assign button here but then see no suitable areas in the assign method currently...


need for team use oe-a 2.3, 3.0, 3.1


Say you have several servers, and one desktop/laptop. You run “jounal-eventc-daemon” on each server and you have only one eventd.
With a plugin, you need one eventd everywhere. :-)

Anyway, I do not want to disallow such plugins… for now, ;-)


@jbgo LOL - good stuff! (as good as you can get for Rspec at least) :smile:


@thockin The flag seems like a good plan, and pretty important for getting this in sooner, will add.

"Dry up for real"?
Assuming that means what I think it means, GSoC is supposed to soft end the 18th and hard end the 21st but I can continue beyond that.
Classes start the 17th and next week is a little busy, getting a filling tuesday morning :/, and moving my sister in to her dorm sometime Wednesday, then going up myself Friday. Once classes start I plan to continue helping out with this and vagrant but I'll become less active over time.
I think my coursework is going to keep my pretty busy once we get a few weeks into class.


Test


This "swizzling" mechanism, copied in so many other Cordova plugins, is completely unnecessary.

1.  configure your iOS plugin with [onload="true"](https://github.com/christocracy/push-cordova-plugin/blob/master/plugin.xml#L16)

2.  Set up `NSNotificationCenter` [listeners for key life-cycle events](https://github.com/christocracy/push-cordova-plugin/blob/master/src/ios/CloudFivePush.m#L10)


thanks @rafrombrc! this is great news. This and other filters with limits are causing us some throughput pain. Will check out the `output_limit` and report back here!


fixed the test a bit


@gamaanderson @pfhein I decided to just push these changes.
It's the same functionality as the large number of if statements, but cleaner using the set method. What do you think?

It's approaching the RSL "field like Z" and maybe some intermediate step between the ugly hack and pyart config.


Remove the log statement.


I am not sure how this change got in. This change is not intentional.


12/31 or ~40% of the checks in hostinfo still require this platform check. (on the refactor/hostinfo2 branch)
Imo should be pulled out into the HostInfo base class.
e.g.
```lua
function HostInfo:isRestrictedPlatform()
  local currentPlatform = los.type()
  local tableContains = misc.tableContains
  if not next(self:getPlatforms()) then return false end
  if not tableContains(self:getPlatforms(), currentPlatform) then
    self._error = 'unsupported platform'
    return true
  end
  return false
end
```
And triggered on every hostinfo check run. We can check if self:getPlatforms is defined and if it isnt quickly abort the rest of the check.


Hi,
Thanks for your notice.
I agree, we could give the  hives query optimizier more hints on what whe want to achive, by changeing both joins (date filter and average price in category) to be left semi joins. However if we are spending time to change this, we should move on to IN/NOT IN/EXISTS/NOT EXISTS syntax, since we are moving away from supporting old hive versions <=0.12.
Cheers,
Michael


@asdofindia  hey,
Thanks for looking into this and sharing that link. I didn't know about this : http://learnlearn.in/urls/#clean-urls

I have changed it now : http://sagarism.github.io/as-per-akshays-suggestion-:P/

[I usually post at http://sagarhani.wordpress.com. This repository is just to learn and keep experimenting with Jekyll and other stuffs.] :stuck_out_tongue:


`STATUS_TIMEOUT` -> `STATUS_TIME_OUT`? (I'm getting a compile error on OSX with clang)


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 283](https://teamcity.labkey.org/viewLog.html?buildId=265221&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:09:13
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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

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

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

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

```


Drop quotes.


Sounds good to me. End of code review.


arsg -> args


Just call it `time` so it's easier to follow.


saw approval from jvoegele
at https://github.com/basho/yokozuna/commit/f627fe534c70f22caabe5e723cd1b229f808d5ba


merging basho/yokozuna/bugfix/zl/assert-that-verify = f627fe53 into borshop-integration-535-bugfix/zl/assert-that-verify


basho/yokozuna/bugfix/zl/assert-that-verify = f627fe53 merged ok, testing candidate = 3a128523


@WillPresley @simonrjones, what's going on here?
I just tried to use this and for me, BitBucket was returning URLs which match this pattern:
```php
$repoPattern = '!https://bitbucket.org/\!api/2.0/repositories/' . $account . '/(.+)/hooks$!';
```
For example https://bitbucket.org/!api/2.0/repositories/exicon/nix/hooks

Here is some more context:
```
  [4]=>
  object(stdClass)#437 (16) {
    ["scm"]=>
    string(3) "git"
    ["has_wiki"]=>
    bool(false)
    ["name"]=>
    string(3) "nix"
    ["links"]=>
    object(stdClass)#438 (10) {
      ["watchers"]=>
      object(stdClass)#439 (1) {
        ["href"]=>
        string(63) "https://bitbucket.org/!api/2.0/repositories/exicon/nix/watchers"
      }
      ["hooks"]=>
      object(stdClass)#440 (1) {
        ["href"]=>
        string(60) "https://bitbucket.org/!api/2.0/repositories/exicon/nix/hooks"
      }
      ["clone"]=>
      array(2) {
        [0]=>
        object(stdClass)#441 (2) {
          ["href"]=>
          string(43) "https://onetom@bitbucket.org/exicon/nix.git"
          ["name"]=>
          string(5) "https"
        }
        [1]=>
        object(stdClass)#442 (2) {
          ["href"]=>
          string(38) "ssh://git@bitbucket.org/exicon/nix.git"
          ["name"]=>
          string(3) "ssh"
        }
      }
```


Improve CacheProvider inheritance


Since Palaso and Chorus aren't products with versions, then this seems like it will work.


You left merge markup in the committed code :). There are some good visual tools for resolving conflicts. We use IntelliJ/PyCharm as our Python/Scala IDE and it has some great tools.


Может быть лучше

> То, о чем я хотел бы рассказать в этой статье, является не альтернативой или ещё одним вариантом синтаксиса БЭМ, а его развитием:


Is there a reason we need to have this method be public? It looks like it's only called from within the MainVC right?


Do we need the Handler to be there? Wouldnt this cause a Handler is undefined err?


Стоит определится БЭМ или BEM и использовать только один вариант во всей статье, включая название.


It is possible to select e.g. IO pins (and then copy them somewhere or delete).

Also, after selection process is started, I'd rather *clip* selection to board, instead of ignoring mouse movement when it gets outside. It just seems to be annoying otherwise.

Changes I suggest: (`git apply`-able)
```
diff --git a/src/main/java/moe/nightfall/vic/integratedcircuits/client/gui/cad/SelectionHandler.java b/src/main/java/moe/nightfall/vic/integratedcircuits/client/gui/cad/SelectionHandler.java
index e7ca0a5..9370ff1 100644
--- a/src/main/java/moe/nightfall/vic/integratedcircuits/client/gui/cad/SelectionHandler.java
+++ b/src/main/java/moe/nightfall/vic/integratedcircuits/client/gui/cad/SelectionHandler.java
@@ -140,9 +140,17 @@ public class SelectionHandler extends CADHandler {
 			int gridX = (int) Math.floor(parent.boardAbs2RelX(mx));
 			int gridY = (int) Math.floor(parent.boardAbs2RelY(my));

-			if (gridX >= 0 && gridX < parent.getCircuitData().getSize() && gridY > 0 && gridY < parent.getCircuitData().getSize()) {
-				selectionEnd = new Vec2(gridX, gridY);
-			}
+			int m = parent.getCircuitData().getSize() - 2;
+			if (gridX < 1)
+				gridX = 1;
+			else if (gridX > m)
+				gridX = m;
+			if (gridY < 1)
+				gridY = 1;
+			else if (gridY > m)
+				gridY = m;
+
+			selectionEnd = new Vec2(gridX, gridY);
 		}

 		Vec2 slStart = selectionStart();
@@ -168,7 +176,8 @@ public class SelectionHandler extends CADHandler {
 		int gridY = (int) Math.floor(parent.boardAbs2RelY(my));

 		selectionStart = selectionEnd = Vec2.zero;
-		if (gridX > 0 && gridX < parent.getCircuitData().getSize() && gridY > 0 && gridY < parent.getCircuitData().getSize()) {
+		int m = parent.getCircuitData().getSize() - 1;
+		if (gridX > 0 && gridX < m && gridY > 0 && gridY < m) {
 			selectionStart = new Vec2(gridX, gridY);
 			mouseDown = true;
 		}
```


fix-writing-output-without-permissions-5063 (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (747 of 747 tests passed)

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


aaa


Like!


Good catch, thank you


@sjackman To avoid #42553 repeating again for Linuxbrew, I suggest you to cherry-pick this commit(https://github.com/Homebrew/homebrew/commit/ac5707c37470c25583d8fb712bf5dc04ab91db4b) and commit https://github.com/Homebrew/homebrew/commit/fbfc9c40d2fb284bda3cd107c3eb08931d877d27 immediately and then wait a period of time(maybe a week or so) before synchronizing Linuxbrew with Homebrew.


Why is a delta branch needed and why are the characters above the loser sometimes added first when the indexes have certain values?


I'm trying to get away from creating errors at runtime.  Could you put something like `var InvalidTimestamp = errors.New(...)` and set `er` to that?


http://ru.urbandictionary.com/define.php?term=kick+it+up+a+notch

> это небольшие дополнения, которые делают БЭМ ещё лучше.


Ops, missed dangerfarms/fuqu-app#22


Этот метод должен возвращать Set<Color> - мы теперь все содержим в Set, вместо List.


Other settings to also pay attention to include memory_limit and instruction_limit (which are [sandbox specific](http://hekad.readthedocs.org/en/v0.9.2/config/common_sandbox_parameter.html#config-common-sandbox-parameters)); and max_process_inject, max_process_duration and max_timer_inject (which are [global to heka](http://hekad.readthedocs.org/en/v0.9.2/config/index.html#global-configuration-options)).


Awesome thanks!

Now how do we force the rebuilding of python?


I don't know how I feel about this. I thought the spec required unix time as integer.  Going to talk to jsonapi.org folks about this.


Now during transaction I got:
```
...
Running transaction check
Transaction check succeeded.
Running transaction test
Transaction test succeeded.
Running transaction
  Installing  : ...
  Verifying   : ...

Installed:
...

Traceback (most recent call last):
  File "/home/jsilhan/Documents/dnf/bin/dnf-2", line 36, in <module>
    main.user_main(sys.argv[1:], exit_code=True)
  File "/home/jsilhan/Documents/dnf/dnf/cli/main.py", line 193, in user_main
    errcode = main(args)
  File "/home/jsilhan/Documents/dnf/dnf/cli/main.py", line 84, in main
    return _main(base, args)
  File "/home/jsilhan/Documents/dnf/dnf/cli/main.py", line 146, in _main
    ret = resolving(cli, base)
  File "/home/jsilhan/Documents/dnf/dnf/cli/main.py", line 178, in resolving
    base.plugins.run_transaction()
  File "/home/jsilhan/Documents/dnf/dnf/plugin.py", line 82, in fn
    dnf.util.mapall(operator.methodcaller(method), self.plugins)
  File "/home/jsilhan/Documents/dnf/dnf/util.py", line 157, in mapall
    return list(map(fn, *seq))
  File "/home/jsilhan/Documents/dnf-plugins-core/plugins/generate_completion_cache.py", line 85, in transaction
    inst_pkgs_insert)
sqlite3.ProgrammingError: Incorrect number of bindings supplied. The current statement uses 1, and there are 29 supplied.
```
@ignatenkobrain could this patch break it or is it somewhere else?


@nguy good work, it is much more simple now. I was thinking of removing this test from RadarDisplay, and just use as parse in the scritps. This will result in the same behavior as it is only parsing the initial reflectivity field in start up


Load `angular.js` before `script.js`. Libraries should be loaded first.
Also, as per the best practice, all scripts should be loaded at the bottom of the `body` tag.


why do we need it? keep the original


Я назвал расширение *БЭМОТ* (BEMIT), так как оно берет идеи и некоторые паттерны из  CSS-архитектуры обратного треугольника (ITCSS, Inverted Triangle CSS architecture), которая так и не была опубликована.

Снова вопрос, стоит ли переводить аббривеатуры?


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1213


@sergey-tihon to match lists ignoring an order of the elements


revert this


pentru ca suge cum arata momentan aia cu absenti. Cand vii repari.


Dropped use of search cursor in favor of describe object's fidset property. Clear def query didn't seem to work so I fixed and tested. Added some comments describing changed functionality.


this is really nice! got a song done in record time! (only 30 minutes).


Или скорее ППБЭМ — Перевернутая Пирамида Блок Элемент Модификатор?


Actually, @Marty-D does this apply to every ability that calls `-boost`? In other words, can I move this to `default` like how it's done for `Item`.


[**@onetom**](https://github.com/onetom) commented on [Aug 7, 2015, 1:14 PM EDT](https://github.com/studio24/bitbucket-bulk-add-hook/commit/4e04a328523bf95be4eea157919669f1734b3275#commitcomment-12591480 "2015-08-07T17:14:21Z - Replied by Github Comment Enhancer"):
> [@WillPresley](https://github.com/WillPresley) [@simonrjones](https://github.com/simonrjones), what's going on here?
>
> I just tried to use this and for me, BitBucket was returning URLs which match this pattern:
>
>

$repoPattern = '!https://bitbucket.org/\!api/2.0/repositories/' . $account . '/(.+)/hooks$!';

>
> For example [https://bitbucket.org/!api/2.0/repositories/exicon/nix/hooks](https://bitbucket.org/!api/2.0/repositories/exicon/nix/hooks)
>
> Here is some more context:
>
>       [4]=>
>       object(stdClass)#437 (16) {
>         ["scm"]=>
>         string(3) "git"
>         ["has_wiki"]=>
>         bool(false)
>         ["name"]=>
>         string(3) "nix"
>         ["links"]=>
>         object(stdClass)#438 (10) {
>           ["watchers"]=>
>           object(stdClass)#439 (1) {
>             ["href"]=>
>             string(63) "https://bitbucket.org/!api/2.0/repositories/exicon/nix/watchers"
>           }
>           ["hooks"]=>
>           object(stdClass)#440 (1) {
>             ["href"]=>
>             string(60) "https://bitbucket.org/!api/2.0/repositories/exicon/nix/hooks"
>           }
>           ["clone"]=>
>           array(2) {
>             [0]=>
>             object(stdClass)#441 (2) {
>               ["href"]=>
>               string(43) "https://onetom@bitbucket.org/exicon/nix.git"
>               ["name"]=>
>               string(5) "https"
>             }
>             [1]=>
>             object(stdClass)#442 (2) {
>               ["href"]=>
>               string(38) "ssh://git@bitbucket.org/exicon/nix.git"
>               ["name"]=>
>               string(3) "ssh"
>             }
>           }

You are absolutely right. Good catch. They added an exclamation point sometime in the last 3 days, because my first PR was working great, and now it isn't. Your adjusted regex works well! See [Pull Request](https://github.com/studio24/bitbucket-bulk-add-hook/pull/3)


thanks! :)


thanks @seisman


Thanks for the super quick response!


It was added automatically....


@zoek1 vas a enlazar en algún contenedor Redis?


Why not use a refactored readcast since thats what readcast was made to do and automatically checks and ignores blank and comment lines using the isblank and iscomment functions.


closed #2


This block would be prolly nice to have in a misc function, like readcast


ok. I do think a general 'file utils' bucket is pretty useful, but maybe we can call it something different than 'fs'. we could also document it on the readme.


not this again. (upstream problem lool)


這個 .update 總是會出現 Missing where attribute in the options parameter passed to update 無法使用...已[參考文件](http://sailsjs.org/documentation/reference/waterline-orm/models/update)


It could be the configurable bit for views which is optional. Rails g devise:views...however I ran it incorrect,y at first so maybe it added a file it wasn't supposed to. I can remove it.


@peti This update drops the `cpphs` dependency from `thyme`, which causes the `thyme` build to fail on my system:

```
building path(s) ‘/nix/store/ccpk5j9lipy4ddvs7m3wkq0qm728j1yk-haskell-thyme-0.3.5.5’
setupCompilerEnvironmentPhase
Build with /nix/store/413wnlmncixpsh0f95smksnwqdcnnc5l-ghc-7.8.4.
unpacking sources
unpacking source archive /nix/store/mlii6qs9hracsw9v0qninwxkjsfkqibi-thyme-0.3.5.5.tar.gz
source root is thyme-0.3.5.5
patching sources
compileBuildDriverPhase
setupCompileFlags: -package-db=/private/var/folders/4j/br7bdhjx4b384_snb2087gt00000gn/T/nix-build-haskell-thyme-0.3.5.5.drv-0/package.conf.d -j8
[1 of 1] Compiling Main             ( Setup.hs, /private/var/folders/4j/br7bdhjx4b384_snb2087gt00000gn/T/nix-build-haskell-thyme-0.3.5.5.drv-0/Main.o )
Linking Setup ...
configuring
configureFlags: --verbose --prefix=/nix/store/ccpk5j9lipy4ddvs7m3wkq0qm728j1yk-haskell-thyme-0.3.5.5 --libdir=$prefix/lib/$compiler --libsubdir=$pkgid --with-gcc=clang --package-db=/private/var/folders/4j/br7bdhjx4b384_snb2087gt00000gn/T/nix-build-haskell-thyme-0.3.5.5.drv-0/package.conf.d --ghc-option=-optl=-Wl,-headerpad_max_install_names --disable-split-objs --disable-library-profiling --enable-shared --enable-library-vanilla --enable-executable-dynamic --enable-tests
Configuring thyme-0.3.5.5...
Flags chosen: werror=False, show-internal=False, lens=False, hlint=False,
bug-for-bug=True
Dependency Cabal -any: using Cabal-1.18.1.5
Dependency QuickCheck >=2.4: using QuickCheck-2.8.1
Dependency aeson >=0.6: using aeson-0.9.0.1
Dependency attoparsec >=0.10: using attoparsec-0.13.0.1
Dependency base >=4.5 && <5: using base-4.7.0.2
Dependency bytestring >=0.9: using bytestring-0.10.4.0
Dependency containers -any: using containers-0.5.5.1
Dependency deepseq >=1.2: using deepseq-1.3.0.2
Dependency directory -any: using directory-1.2.1.0
Dependency filepath -any: using filepath-1.3.0.2
Dependency mtl >=1.1: using mtl-2.1.3.1
Dependency old-locale >=1.0: using old-locale-1.0.0.6
Dependency profunctors >=3.1.2: using profunctors-5.1.1
Dependency random -any: using random-1.1
Dependency system-posix-redirect >=1.1: using system-posix-redirect-1.1.0.1
Dependency text >=0.11: using text-1.2.1.3
Dependency thyme -any: using thyme-0.3.5.5
Dependency time >=1.4: using time-1.4.2
Dependency vector >=0.9: using vector-0.10.12.3
Dependency vector-space >=0.8: using vector-space-0.10.2
Dependency vector-th-unbox >=0.2.1.0: using vector-th-unbox-0.2.1.2
Setup: The program cpphs is required but it could not be found.
builder for ‘/nix/store/mfzjicjgnyw1qy5vszh6kcz74x9a42wi-haskell-thyme-0.3.5.5.drv’ failed with exit code 1
```


We should prolly change this guy to use the Info:_transform instead of passing a casterfunc in.
It's used in 4 files, and would prolly make sense to use it in pam.lua as well.


@pinda you might want to see, https://github.com/shwoodard/jsonapi/compare/json-extras...json-extras-unmarshal.  I'm working further on this links feature.


fast-forwarding 2.0 to borshop-integration-535-bugfix/zl/assert-that-verify = 3a128523


lgtm


Fix #19


```lua
HostInfoStdoutSubProc.initialize(self, 'sshd', {'-T'}, Handler)
```
Should be
```lua
HostInfoStdoutSubProc.initialize(self, 'sshd', {'-T'})
```
I think


Sounds good to me. I've made the change in https://github.com/personalrobotics/prpy/commit/e346ca61fc0cc37818fdb61dc8ba8cff45128a2b.


That works. I'll let you do that with your next update if that is okay. But hold off as I'm adding a ```plot_simple.py``` (simpler ```plot_radar.py```) for quick generating plots. Specifically for SelectRegion histogram.


terrible code


Remove commented out code


s/empy/empty/


Здесь это не имеет значения. У нас нет повторов, повторные Color не пройдут валидацию которую мы добавим позже. ИМХО


ok @lfcj please do the rails tutorial on how to do migrations, which are like mini timestamped commits, so changing the migration file manually like this is gonna cause a lot of chaos... since there will be confusion on the rails side and the actual database side...

next you wanna read up on rails model associations on how to do that properly...

finally, you wanna read up on rails validations, to check if a name is taken etc. instead of doing it manually.

Again if you want to continue with rails its important to do the tutorial first, otherwise your effort here is gonna be wasted.


This also addresses an earlier comment by @mkoval about using `numpy.add`. That's no longer needed as we're explicitly casting the `list` to a `numpy.array`


#skeptical


I think its more of a 'whatever looks and reads better' rule


I will as soon as I aquire the needed info. I'm also checking to make sure I'm not using old libraries or some similar mistake on my end. If I cannot solve this I will create an issue.


@cstrahan, I'm now building `jailbreak-cabal'  with an unreleased git snapshot of `Cabal`: https://github.com/peti/nixpkgs/commit/ffe269fbbee887578f866cb76a534c8dd8de964c. It's not ideal, but it seems to work.


Migrations shouldn't have any data inputs. These should be in fixtures.


You don't seem to do anything with the equal check so it always returns false, doesn't it?


Fixes #7


It should be "every programmer I [Jonathan] know uses."


String interpolation is the recommended style for all Scala projects I have seen. For Kafka, in particular, there hasn't been a discussion about it, but it seems pretty clear for single-line strings like this one (I found and fixed a number of bugs in `format` usage in Kafka as an example). For multi-line strings it's less clear.


you are evil! :imp:


fix this!


That are stock mt6592 5.1 libraries.


really? coiso? guys... ?


I think we should have an opt-in flag instead like `--iptables-proxy` and default to false unless specified since the pflag package seems to expect booleans to be toggled by presence only IE: `--userspace-proxy=false` seems to not be idomatic, if even possible.


Fixes #16


@borshop: retry


По негласным правилам вроде на английском всё решили писать.


fast-forwarding develop to borshop-integration-8-gg/timeseries = df014a5c


This file is exactly a helper to select what compiler will be used to build sml, so the properties CMAKE_C_COMPILER, CMAKE_CXX_COMPILER and CMAKE_FIND_ROOT_PATH changes for each toolchain used. This file was created using the default values from the official Edison sdk. And the same applies to galileo-toolchain.cmake.

If having a cmake file with default values is not a good approach, what we can do is to remove this file, and users will need to inform all information as parameters in cmake file.

But I don't see a problem to have a sample .cmake file with default SDK values.


ok, I don't know where is that plot_simple be fast then radar, but I will wait and see. I will take some time with my updates, so you can take the weekend off, you have done phenomenal job this week.


Well it was only a quick idea at that moment. Though the more I think
about it the more I like it :) I find the CamelCase writing a lot easier
to read to be honest...


I think I'll keep using std. Ok?


At first I was gonna complain about sticking pertinent information in the keys
i.e. ```lua obj[v] = who[i][v] ``` on line 32
But then i ran the check and it doesnt appear to return any information at all. At least on a ubuntu cloud box:
![screen shot 2015-08-07 at 10 49 21 am](https://cloud.githubusercontent.com/assets/3248143/9142071/35d9b980-3cf2-11e5-9063-160d199dd956.png)


@kellyredding are you just going to :trollface: me about rspec while I build this?


It looks like your commit breaks `dnf download existing nonexisting` and also it would be nice, if download plugin would produce the same output as `dnf install`.

```
No package sadsa available.
No package rerw available.
Error: Unable to find a match.
```


Shouldn't these be AUTOTOOLS_AFTER?


This is the repository for the Udacity "Data Analyst Nanodegree" Projects


shouldn't we follow this XXX and protect the callback with the deferredlock too?


Bump!  Calendar has flipped 4 times through.


Sorry, I don't know what happened here :-( This was the first (well, second, after trying `$1_type` initially) thing I tried and I got errors and so used `basetype`. But retrying it now I can't reproduce any errors any longer, so apparently I just made some stupid mistake, Thanks for noticing it!


add sts_ prefix to both structures


Get rid of (n, w, c) leftover


Your code here is unnecessarily complicated, just do:

```php
return $carry . str_pad(decbin(ord($char)), 8, '0', STR_PAD_LEFT);
```


I'm commenting here to try something out. Nothing to worry about.


this function doesn't exist anymore. That error message can be tweaked here:

https://github.com/virgo-agent-toolkit/rackspace-monitoring-agent/blob/refactor/hostinfo2/hostinfo/base.lua#L95


I'm commenting here too to try something out. Nothing to worry about lol.


Probably going to piss a few people off removing the copyright notices from other peoples code...
https://github.com/cjwl/cocotron/blob/master/Foundation/NSPropertyList/NSPropertyListReader_binary1.m


почему код не компилится?


But I don't think we need it..


saw approval from javajolt
at https://github.com/basho/riak_core/commit/430d329d010f01116960a711b794a2804a844489


merging basho/riak_core/feature/new-utility-functions = 430d329d into borshop-integration-762-feature/new-utility-functions


basho/riak_core/feature/new-utility-functions = 430d329d merged ok, testing candidate = 3bef2d6c


fixed


Oh! Thank you very much for the report. 7271b95d693e62e6f86d4b3f400dd96574815054 fixes this.


nope fixed in 465e46c


@LeadSongDog If you look at the current file, you will see that not only is this code gone, but that there's no longer any remnant of the functionality it was supporting: https://github.com/jquery/jquery/blob/9adfad19865837f5dffedb1eb41e407f196ca515/src/traversing.js#L171-L195


Everything inside this `_sync()` method is protected with the `DeferredLock`, isn't it? And, if i understand it correctly, we could remove the `XXX` here, right?


yeah, you can refactor it out to it's on transform object, but you can't attach it, because you are doing multiple async calls in parallel.


Scratch that, i see that I was looking at a nearly identical package. The docs for the one we use show that the former is possible.
https://godoc.org/github.com/spf13/pflag
> Boolean flags (in their long form) accept 1, 0, t, f, true, false, TRUE, FALSE, True, False


PlantOperationSchemes (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


it's more compact. nice


hmm isn't this leaving lingering temporal folders around?


as a general comment... do you think there would be some way of mocking up the remote replicas somehow, so that (in the future) we could have real-quick unittest, and move all these suites to integration tests?


yeah, looks like it should be
````
if (clearEntry.getPlot().getId().equals(id)){
   return true;
}
````


fixed!


#pichonStyle


I think the extra overhead of having to call get(i) instead of being able to access the native array is probably not worth it...


Yup.


what was happening if one didn't close the syncer?


WHO and needs to be root.


Actually, I tried with using atexit and it worked as well. May be, we should go with atexit if it is more robust.


Hahaha Did their use get removed somehow? Implementing it was my only contribution to GPUImage project.


hi jason... do you want to do a pull request?


@rexissimus:
> ModuleSuspended used to work without a handle, but the whole workflow needs to be re-executed to updating its status. If we are removing this we should also remove the code that re-executes the workflow from the job-checking code.


What is proxyPort for at this point?  Is it just NodePort?  I see you allocate it, but I don't see where you actually listen on that port anywhere


It is, and it will be added again, but not in your repositories list. People normally have a lot of starred projects and that kind of makes the repositories tab useless.


@nguy I like that commit.  It is not the "treat field like ___" option that is in rsl_in_idl but that should maybe be a pyart addition.  That array made me  think of user config files where the warning could direct the person to add it to their user config file instead of contacting us.  That may be something you do not want to deal with now but I think it is something to consider for the future.


timeouot seems to be totally unused now - can it just be removed?


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_core/builds/1669


yes, you're right, I was reading this too fragmented.


Just curious, was this causing a bug? or just best practices?


we have to keep it until we have a replacement


I don't know if this code works, if we believe what Colin said. Apparently jobs are just assumed to be finished the next time around.


r+


this was removed in a later commit IIRC


saw approval from graydon
at https://github.com/jedmccaleb/stellar-core/commit/b2fa4cd10c010b19e8f83700ded9469f54fb1158


merging jedmccaleb/stellar-core/master = b2fa4cd1 into auto


jedmccaleb/stellar-core/master = b2fa4cd1 merged ok, testing candidate = bb6a28db


Yes it's used in the nodePort rules generated in `syncProxyRules()`.


nifty, thank you @SirVer ! :rocket:


I want it to be obvious when users are doing something other than the norm.

--legacy-userspace-proxy (default false) is nice because when we delete
that code, the flag goes with it.  --iptables-proxy would have to stick
around without any meaning.

Alternative would be be --proxy-mode=iptables vs --proxy-mode=userspace

On Fri, Aug 7, 2015 at 10:49 AM, Benjamin Elder <notifications@github.com>
wrote:

> I think we should have an opt-in flag instead like --iptables-proxy and
> default to false unless specified since the pflag package seems to expect
> booleans to be toggled by presence only IE: --userspace-proxy=false seems
> to not be idomatic, if even possible.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/1aa20666a4fe33f8246dd6c83712c3fd52d77a6d#commitcomment-12592275>
> .
>


Fancy newline! :banana:


How is this different from `table.string`? It looks like `string` defaults to length 255.
http://knexjs.org/#Schema-string.


Nevermind ... it was a later commit that I did which broke the tests.


another general remark... Some time ago I think we had the assumption that the sqlcipher driver was free from twisted-related boilerplate, remaining all the twisted-related code contained in the wrapper and in api.
I realize now that this assumption is broken since we're calling the reactor callWhenReady in the constructor for the  class, but all the same I thought I'd raise the observation.
I'm more than happy having all soledad code embracing twisted, it's only that maybe there is some way of separating class concerns, keeping all code that touches reactor or threadpool isolated.


why size 8, while on CUDA, it is sizeof(int)? I think it should be the same size on both back-end.


Do you intend to store JSON in this column?  Would the `json` type be useful here? http://knexjs.org/#Schema-json


'posts' is what the database will be named as its created.


This check is acting weird when tested,
![screen shot 2015-08-07 at 10 59 25 am](https://cloud.githubusercontent.com/assets/3248143/9142687/4619e794-3cf6-11e5-8379-4ea3fd39d38b.png)
Vs the pam.lua in https://github.com/virgo-agent-toolkit/rackspace-monitoring-agent/pull/829
![screen shot 2015-08-07 at 11 19 33 am](https://cloud.githubusercontent.com/assets/3248143/9142696/523e5c1c-3cf6-11e5-8379-4af081ddf0d0.png)


I have noticed that gh-pages are not loading because index.html is not in
root directory. Could you fix it pls? I dont know what exactly did you mean
when you was writing about standalone docs-gh-pages.
On Aug 7, 2015 7:44 PM, "Philipp Alferov" <notifications@github.com> wrote:

> [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/luckylooke/dragular/commit/47e9b0ec447ad5f799dd0e6f61812bd32611843d#commitcomment-12592103>
> .
>


PlantOperationSchemes (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


it'd be good to comment how this is going to be used (with path as key)


Besides the dead code looks good.


comment the intent of this (although it's quite self-evident), but what's problem is fixing?


This name imo shouldn't be attached to `CommitAuthor`. I'd argue that this should be a field on `Commit` instead. Since technically, a user could commit with different names but use the same email. The email is the canonical match, but the name is meta data of the commit itself.


I went with the first suggestion defaulting to true, but renaming it to `--legacy-userspace-proxy` seems possibly clearer. Though we are defaulting that to true not false? IE are we adding this flag in everywhere or having the value default to disabling the new proxy for now? I was thinking the lattter so `--legacy-userspace-proxy=true` (default _**true**_) would be default and manually setting to false would enable the new proxy.


@warmerdam it appears that this function is not returning a CDF spread throughout `[0, 4095]`. Taking the upper bound of `cdf_out`'s index, we have `i = 254`, `j = 7`, resulting in a maximum index of `i * 8 + j = 2039`. The result is a compressed CDF.

![cdf-comparison](https://cloud.githubusercontent.com/assets/581337/9142653/0762ba80-3cf6-11e5-9a50-1d80cd702ab7.png)

Another approach to this would be something like:

```python
# Create the 8 bit domain
x = np.arange(0, 255)

# Create the 12 bit domain, spread with 255 values
xp = np.linspace(0, 4095, 255)

# Interpolate the points in between
cdf_interp = np.interp(xp, x, cdf)
```


PlantOperationSchemes (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


I think this should be simplified. All you really need is something like [this](https://gist.github.com/khatchad/f676b4a64533a8acca65).


We dont care if you want to have 10 names, we care that you're a single person.

Keep in mind this isn't intended to map up to Git or anything else. It's intended to handle basic high level metadata of what a commit log would be.


![screen shot 2015-08-07 at 11 22 54 am](https://cloud.githubusercontent.com/assets/3248143/9142769/b8365bb4-3cf6-11e5-8e46-c0c6b113f2c3.png)
Also remember #808 check names are case insensitive now


You can create a seed file to insert data: http://knexjs.org/#Seeds-CLI.


Why order and not a `DateTime` field? I guess, how are you going to enforce that the order is always incrementing here? Seems more sane imo to do a `date`.


Please use single quotes for strings.


We generally don't use this style for arguments - instead put them on one line and only wrap if you add extra.


Commits in most VCS aren't date ordered. Its enforced by the scripts which write to the data.


Move this to the end of the file


underscores!


godoc


Like so: `fs.BoolVar(&s.UserspaceProxy, "legacy-userspace-proxy", true, "Use the legacy userspace proxy (instead of the pure iptables proxy).")`

Aside: is there anywhere else we need to doc this besides docs/admin/kube-proxy.md ?


awesome - I totally forgot i wrote that :)


should 15 be in config.resources.moirai.time_to_shutdown or something?


Remove the second paragraph - it doesn't really make sense.


There is derp in that line - fstab has wrong name (fstab.qcom) and zRAM isn't enabled.


Whoah, what's going on here?  This is not how random port allocation works.  As soon as you Close() the socket someone else can grab the port.  BUt I don't understand why we need to do this at all.


Yeah, I understand. That's why it's not a part of the index or primary key and would never be used for a lookup. Just arguing that the author's name is meta data on the commit itself, rather than the author since it could change over time.

The the only suggestion I have is to make sure that `name` is the latest name being set. Right now, since you're doing a `get_or_create`, it'll never update a name if it's changed. It'll always be stuck on the first name seen.


This is the correct way - absolute positioning of the footer, not relative.


This is wrong!


I think doing create_or_update on name is good, will change to that.


Ah, so is this on the idea too, of say, a roll back, where they'd be going back in commit history, but moving forward in a release?

I'd call this `version` then, but order works too. `version` of the release makes more sense to a user.


I proposed in ARM-DOE/pyart#336 lists of fields that are the same physical quantity. That was however refused.


Start with the default value set to true, so nobody breaks this week.  That
way we can check it in with some impunity.  Then we can test it more and
more broadly.  Then we can flip it to default false.  Then we can delete it.

On Fri, Aug 7, 2015 at 11:23 AM, Benjamin Elder <notifications@github.com>
wrote:

> I went with the first suggestion defaulting to true, but renaming it to
> --legacy-userspace-proxy seems possibly clearer. Though we are defaulting
> that to true not false? IE are we adding this flag in everywhere or having
> the value default to disabling the new proxy for now? I was thinking the
> lattter so --legacy-userspace-proxy=true (default *true*) would be
> default and manually setting to false would enable the new proxy.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/1aa20666a4fe33f8246dd6c83712c3fd52d77a6d#commitcomment-12592999>
> .
>


Yeah, then that's a reasonable compromise instead of duplicating name on every single commit.


For now we don't want to document it further because we don't want mere
mortals using it

On Fri, Aug 7, 2015 at 11:26 AM, Benjamin Elder <notifications@github.com>
wrote:

> Like so: fs.BoolVar(&s.UserspaceProxy, "legacy-userspace-proxy", true,
> "Use the legacy userspace proxy (instead of the pure iptables proxy).")
>
> Aside: is there anywhere else we need to doc this besides docs/admin/
> kube-proxy.md ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/1aa20666a4fe33f8246dd6c83712c3fd52d77a6d#commitcomment-12593062>
> .
>


PlantOperationSchemes (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


:+1:  got it


I was having occasional issues with the port allocator giving me a 0 which
we of course can't use so I put in this hack to pick one myself. It
shouldn't be needed anymore.

Aside: how would you safely get a random port?

On Fri, Aug 7, 2015 at 2:27 PM, Tim Hockin <notifications@github.com> wrote:

> Whoah, what's going on here? This is not how random port allocation works.
> As soon as you Close() the socket someone else can grab the port. BUt I
> don't understand why we need to do this at all.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593099>
> .
>


Yes, That's a TODO


Ok.

On Fri, Aug 7, 2015 at 2:30 PM, Tim Hockin <notifications@github.com> wrote:

> For now we don't want to document it further because we don't want mere
> mortals using it
>
> On Fri, Aug 7, 2015 at 11:26 AM, Benjamin Elder <notifications@github.com>
> wrote:
>
> > Like so: fs.BoolVar(&s.UserspaceProxy, "legacy-userspace-proxy", true,
> > "Use the legacy userspace proxy (instead of the pure iptables proxy).")
> >
> > Aside: is there anywhere else we need to doc this besides docs/admin/
> > kube-proxy.md ?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/BenTheElder/kubernetes/commit/1aa20666a4fe33f8246dd6c83712c3fd52d77a6d#commitcomment-12593062
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/1aa20666a4fe33f8246dd6c83712c3fd52d77a6d#commitcomment-12593141>
> .
>


I don't think this belongs to MLStats. IMO, a pointer to MySQL documentation would be better.  In addition, granting privileges to everything without an explanation makes Mlstats to spread bad practices.


Starting tomorrow I'll stop copying your lead.


r+


trying!


I don't grok this whole section - you're behaving like we have a usermode proxy, but we don't.  Who is supposed to be receiving this REDIRECT traffic?


Singleton?


with this commit now storagecharts2 have a limit of 1000 users due the array parsing result of the list of users..

i wish to know how to made this configuration a option, i mean does not used the array build list, due in my company theres a load of 3000 users and in firts tests php hangs and nerver restusns a response if any admin user are logged and try to view the chart (i know that due the array, it happened to me in the selectra software)


Just a heads up, the new sul_styles gem includes the SU web identity sass variables:

https://github.com/sul-dlss/sul_styles/


I think listenIP is moot (unless we keep a hybrid mode here and do userspace proxy for nodePort and/or dnat-to-self, and we should do that as a whole separate commit or PR


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


Please use MB and not MiB. People don't know what's MiB. MB is commonly used. The values must not be changed.


This is redundant as @ConfigurationProperties is present RedisSinkProperites. Also I'm not sure why we need @ConditionalOnMissingBean here.  Simpler to  @EnableConfigurationProperties({RedisSinkProperties.class}) and @Autowired RedisSinkProperties.


the shim fn will attach SIMD directly to the global object, which only ever needs to be done once.  We can eliminate this first part of the predicate, and in fact, hoist the entire check (and import of glMatrix, and other imports) out of the buildMat4Tests fn.


bummer


Turns out that `2to3` put in the checks against the `Callable` abc, and wrongly detected an import of the `collections` module.


callback's not used anywhere


just best practices, cleaning up the code and updating to to current ruby standards.


Not sure why this is needed here. Don't these get created already with spring-boot-redis-starter?


doesn't this miss the distinction between cancelled and timed-out for a response?


Style and more exact assertion (I think this is the same)

```haskell
       deps  = forM_ patterns $ \x -> modHeadRule x id
   sort (runDep deps) `shouldBe` sort patterns
```


Yeah I don't think this ever got used anywhere. More old mimic'ing the old proxy, I literally started with a clone and trimmed it down and swapped out the userspace parts as I didn't want to break any functionality, was hoping to achieve a drop in replacement. I didn't know of anything we were allowed to change externally besides the preserving source ip and obviously not holding connections open.
Writing a commit now to trim off this and anything else I can find that still exists unnecessarily.


Also the `callable` built-in was removed in 3.0 but then put back in in 3.4, hence why `2to3` did this in the first place.


If we now have a flag-gate, we can check something in and add on to that.  I don't see how this is working at all - nothing is actually listening.

How's this for strategy - let's strip out anything that is not critical to getting the "normal" operating mode running.  No node-ports special-case, no dnat-to-self, no route_localnet, no hairpin.  Just sync from apiserver -> iptables.

Then we merge that.

Then we fix up special cases behind the protection of a flag.


Thanks!


I think this is supposed to be ```HttpResponse(f.read(), content_type='application/javascript')d


new line


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/74611513


should be 0?


@thockin Flag gate is in, already stripping out non-critical, will also remove node-ports, dnat to self.


Don't need any of this


you pass 0 to the kernel for bind(), which does an atomic allocate-and-consume.  Then you don't close that socket.


See https://github.com/Bumblebee-Project/Bumblebee/issues/644#issuecomment-128649134


q.q


understood, and it was the right attack.  Now that you have it mostly working, strip EVERYTHING that isn't required and figure out what we need to keep :)


Thanks.
I've removed the offending code and will double check that PortAllocator
won't misbehave again.

On Fri, Aug 7, 2015 at 2:41 PM, Tim Hockin <notifications@github.com> wrote:

> you pass 0 to the kernel for bind(), which does an atomic
> allocate-and-consume. Then you don't close that socket.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593325>
> .
>


https://staging.steamlug.org/favicon.ico this file needs replacing with your new one (and we need to keep it in the root)


We should not need the port allocator at all either.  We're not allocating
or using ports at all

On Fri, Aug 7, 2015 at 11:42 AM, Benjamin Elder <notifications@github.com>
wrote:

> Thanks.
> I've removed the offending code and will double check that PortAllocator
> won't misbehave again.
>
> On Fri, Aug 7, 2015 at 2:41 PM, Tim Hockin <notifications@github.com>
> wrote:
>
> > you pass 0 to the kernel for bind(), which does an atomic
> > allocate-and-consume. Then you don't close that socket.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593325
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593346>
> .
>


also not required, I think, unless we do hybrid mode, which we can add back


also not required


PlantOperationSchemes (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Use golang conventions - this should be "podRunningFieldSelector" and "nsActiveFieldSelector"


Right, but If I hit that, I either had something horribly wrong or there
was a bug potentially affecting the userspace proxy.

On Fri, Aug 7, 2015 at 2:43 PM, Tim Hockin <notifications@github.com> wrote:

> We should not need the port allocator at all either. We're not allocating
> or using ports at all
>
>
> On Fri, Aug 7, 2015 at 11:42 AM, Benjamin Elder <notifications@github.com>
> wrote:
>
> > Thanks.
> > I've removed the offending code and will double check that PortAllocator
> > won't misbehave again.
> >
> > On Fri, Aug 7, 2015 at 2:41 PM, Tim Hockin <notifications@github.com>
> > wrote:
> >
> > > you pass 0 to the kernel for bind(), which does an atomic
> > > allocate-and-consume. Then you don't close that socket.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593325
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593346
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593365>
> .
>


fast-forwarding master to auto = bb6a28db


Also - maybe IRC or hangouts would be faster to communicate?

On Fri, Aug 7, 2015 at 11:43 AM, Tim Hockin <thockin@google.com> wrote:

> We should not need the port allocator at all either.  We're not allocating
> or using ports at all
>
>
> On Fri, Aug 7, 2015 at 11:42 AM, Benjamin Elder <notifications@github.com>
> wrote:
>
>> Thanks.
>> I've removed the offending code and will double check that PortAllocator
>> won't misbehave again.
>>
>> On Fri, Aug 7, 2015 at 2:41 PM, Tim Hockin <notifications@github.com>
>> wrote:
>>
>> > you pass 0 to the kernel for bind(), which does an atomic
>> > allocate-and-consume. Then you don't close that socket.
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593325
>> >
>>
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593346>
>> .
>>
>
>


#MixtedRawfish


I do not wish my comments to seem negative :) I am very thankful you did the favicon work, but a bit miffed at the quantity of touch icon wankery this pulls in for the comparatively small userbase of Apple devices. I know this is due to Apple being shit, but we do not have to support the older iOS devices in full, at all.

If we can trim some of these resources down a little, I would be very happy. All the multi-size favicon.png are possibly redundant, just pick a decently big size image, and cull the rest.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 39](http://teamcity.otenv.com/viewLog.html?buildId=1444076&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 39](http://teamcity.otenv.com/viewLog.html?buildId=1444076&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 0:0:0


> Actually, I tried with using atexit and it worked as well. May be, we should go with atexit if it is more robust.

My concern about reaching up into the destructed `StaticMeta` are only hypothetical at the moment; neither clang nor gcc appear to do evil to the underlying storage, so we can still access the `tls_` member even though `~A()` fires later. You could imagine somebody doing some well-intentioned cleanup of `~StaticMeta()` in the future and breaking this, tho.

> My preference would be using RAI, possibly wrapping tls_ in unique_ptr. For that, we'd need to migrate from using __thread to thread_local for tls_. This should work for both MacOS and Linux, right?

Regrettably, we have `std::thread_local` everywhere _but_ OS X at present.

Since `StaticMeta` is a singleton object and is destructed on process exit only, the simplest alternative is probably just to `OnThreadExit(...)` in `~StaticMeta()`. Do note that if you have other classes that use your `thread_local` that _also_ have static scope, this can introduce terrible ordering issues; I dealt with this in a different TLS implementation recently: https://github.com/flandr/ccmetrics/blob/master/src/detail/thread_local_detail.h#L258


> Why did you add the `<head>` code and why didn't you leave it in `{% include head.html %}`

Otherwise I wouldn't have been able to reuse `head.html` in `xml-style.xsl`. Notice that the `DOCTYPE` needs to be escaped and `<html>` was also started in there. I'm not big fan of opening a tag in file and closing it in another, I had to make a double-take every time I was looking for the start of `</html>` and I had to go to `head.html` to see if it has the `<head></head>` tags.

I think it's much cleaner if you have both the opening and closing tag in the same file. It's also nice to see the full structure of the page in `default.html` and the full structure of the transformation in `xml-style.xsl`.

Another benefit is that the `head.html` can be unindented fully, leaving more characters for interesting stuff on smaller screens.


по змінам:
1. всі дані які стосуются виключно "діячів", напр відповіді на запити, токени, абощо, будуть перенеснені у відпповіних "діячів", це робится з метою запобігання плутанини - які дані кому належать, враховуйте це при написанні ключових слів під ваші майданчики, перенос ще не завершений, тому ще можливі зміни в існуючих тесткейсах.
2. в ${TENDER} будуть писатись тількі ті дані які стосуються ВСІХ учасників, напр. номер тендера
3. деякі тести провалюються, наразі 4, 2 потребують правок/дописування, що і буде зроблено, після обговорень питань з нимим повязаних, 2 потребують більш грунтовного вияснення причин поломки
4. можливі "дивна поведінка" деяких тестів, напр. незрозуміло чи вони щось перевіряють, мають помилки, абощо, такі тести виявляються і правляться, якщо ви найшли такі, прохання повідомляти.


Hmm, I just checked that. Everything works for me: http://alferov.github.io/dragular
Just run `gulp deploy:docs`


typo in scenarii


style - ``free?(port)``


works for building and launching -- I don't get the "Insecure Update" warning on launch, but if I open the Preferences window,  I do get it.  The Menu item "Check for Updates" (if the Dock icon is visible) is greyed out, though.

I'm assuming that's what this is about in the console log:
~~~
8/7/15 1:45:55.948 PM Hammerspoon[7454]: Unknown class 'SUUpdater', using 'NSObject' instead. Encountered in Interface Builder file at path /Volumes/DeepChaos/src/hammerspoon/hammerspoon-local/build/Hammerspoon.app/Contents/Resources/MainMenu.nib.
~~~

but whatever you're doing there isn't carrying over to PreferencesWindow.xib.

FWIW I've checked and the Sparkle framework is not in the Hammerspoon.app/Contents/Frameworks/ folder, so I'm guessing PreferencesWindow is either linking directly to the library at compile time, or it's getting it from somewhere else.


Jsjsjs


Also you need to remove this star.


I have quick-fixed it via moving docs/ files into root folder few minutes ago.. but I havent run gulp deploy:docs..


I can do either, reconnecting to irc now.

On Fri, Aug 7, 2015 at 2:45 PM, Tim Hockin <notifications@github.com> wrote:

> Also - maybe IRC or hangouts would be faster to communicate?
>
>
>
> On Fri, Aug 7, 2015 at 11:43 AM, Tim Hockin <thockin@google.com> wrote:
>
> > We should not need the port allocator at all either. We're not allocating
> > or using ports at all
> >
> >
> > On Fri, Aug 7, 2015 at 11:42 AM, Benjamin Elder <
> notifications@github.com>
> > wrote:
> >
> >> Thanks.
> >> I've removed the offending code and will double check that PortAllocator
> >> won't misbehave again.
> >>
> >> On Fri, Aug 7, 2015 at 2:41 PM, Tim Hockin <notifications@github.com>
> >> wrote:
> >>
> >> > you pass 0 to the kernel for bind(), which does an atomic
> >> > allocate-and-consume. Then you don't close that socket.
> >> >
> >> > —
> >> > Reply to this email directly or view it on GitHub
> >> > <
> >>
> https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593325
> >> >
> >>
> >> > .
> >> >
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593346
> >
> >> .
> >>
> >
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12593392>
> .
>


Looks good, I'll just update the mi_lttng.xsd version to 2.7.


Ah, I see. It is crystal clear now. One last outof topic question, does ffplay support hwaccel ?


had to return laptop and rushed this huge commit - sorry!


gulp deploy:docs rocks! Evrything is OK now ;)


Hi Alex:

Did you guys gave a demo of Somerville project in Dev Bootcamp 2 month
ago?  I was very impressed!

The reason I used smarter_csv with parallel csv was because I need to do
some data geocoding and other calculation for every record in csv.  If you
need just plain data import without calculation,  and your sql's storage is
under IO limit, you can still fire up multiple sql request processes by
moving row connection and sql copy to Parallel workers, without smart_csv.

Thanks for mentioning my name, you guys are awesome and have a nice weekend!

Best regards,
xjlin0/Jack

On Fri, Aug 7, 2015 at 11:47 AM, Alex Soble <notifications@github.com>
wrote:

> Implement parallelism for CSV parsing :+1:
>
> + Hat tip to this excellent blog post:
> https://xjlin0.github.io/tech/2015/05/25/faster-parsing-csv-with-parallel-processing/
> + Thank you @xjlin0!
> + Pairing buddy: @grantrobertsmith
> + Brings idle CPU down to zero while we're running the data import process
> + NEXT UP: separate Ruby from database parts in our importers so we can
> make big chunky database insertions instead of a swarm of tiny ones
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/codeforamerica/somerville-teacher-tool/commit/4f586d7fd53c95cc50a88696c471142392711776
>


Is there a specific reason for this change?


this is nice


please document that you actually (effectively) did `-cal sss_cal_3053.dat` because people at e.g. MGH will need to manually specify, even though you didn't because you are at UW


Is't it necessary to add '.clans' to "campaign-view.sass"?


No, because as the new leftmost button, it doesn't need a specific background image offset (its image is at 0, 0).


Switching to MonoGame to enable build server support.


This test should really be bringing up a zookeeper server and tearing it down by itself (use the Subprocess class that we have).

Otherwise it's far too manual a process to test this.


Tests fail because the reactor is "unclean" when it is stopped. In this case, the cached http connections were still hanging around somewhere, and closing the `sync_target` removed the cached connections.


Ok, didn't know. Thanks for the clarification


If there's a way to get MLStats running w/o this step, please document it.

In regards to redirecting, assuming familiarity with required toolsets is a way to unnecessarily limit those who can use your code. I shared how I did it because it worked and filled the gap - feel free to improve.


Thread safety?


I realize this is optional, but should probably specify where in the URI it goes in the Structure line above.


Looks good, although I think we should move this to a separate Examples project located at `/Examples/UnitsNet.Examples.csproj` or similar. You can still use nunit to run these proof of concept tests, but since they are not part of the core library, they don't belong with the core lib tests.


This was extraneous when I was using a different approach. Should not be needed in a final implementation.


thanks @guybedford


OH GOD something is fucked up.


@ChrisMissal noticed this doesn't have the right user on it.


fucking instr_build every time


I don't know why OpenRadar doesn't just use sub-templates here, and why it generates HTML strings that look like `
No matching results found.
' in the python code rather than leaving this to templates? Probably historical?


PlantOperationSchemes (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 284](https://teamcity.labkey.org/viewLog.html?buildId=265236&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:09:45
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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

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

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

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

```


I'd avoid the word "header" so as not confuse with HTTP headers. Probably something like "Response includes JSON object ... containing an array with invoicing data ..."


PlantOperationSchemes (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


There seem to be lots of places in OpenRadar's sources where there's a test like this that is an effective no-op.


This is an array of strings, mind the square brackets in the end.


I don't think it's hacky because it allows you to use them if you need them version independent


PlantOperationSchemes (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Number?


Updated: https://github.com/naucoin/CTK/commit/f3294326cf91ae92eeb1e028994f9bb69e00ef11


Oops, sorry. It looks like I totally forget to mention about `gulp deploy:docs` and `gulp-gh-pages`
It's a must have tool for github projects for sure.


Yeah, I considered it but its problematic in the following scenario:
Example:
- My type 'A' gets an logger injected `ILogger<A>` in its constructor
- In method `GetItems`()
```c#
IList<string> GetItems()
{
	using(_logger.BeginScope("Scope1"))
	{
		....some logic here
	}

       ...some logic here

	using(_logger.BeginScope("Scope2"))
	{
		....some logic here
	}
}
```


Example covers it, but consider adding a Line Items Entity Definition to define what each element of this array contains/looks like.


Updating the project's information.


why this is needed? I don't like this, but not sure why. So if you need it, this will need at least a comment.


how use your library???


added line about cost-benefit


This only add noise. Are we going to add how to create databases in postgres? how to install python? It does not make sense.

Anyhow, I will fix it.


lol did you just not finish your sentence


That's a little more than a typo, that looks like a straight up bug!


grammEr m8


@srpearce I'll take a look to make sure the info you included in SFD credit courses is not outdated. This chunk will probably end up being overwritten by the main student facing file. Thanks!


That's OK, I am forgetting all the time.. recently I forget to lint... so now it will be 2.0.1 patched after I fix all lints errors. I am wondering if it is not better to be wrapped in watch task, so lint errors are popping out immediately via development, what do you think?


Can you test the current case but also the reverse order:

        e = T.dot(x1, T.opt.assert_op(x2, (x2 > x3).all())) + T.dot(T.opt.assert_op(x1, (x1 > x3).all()), x2)

just to be sure to tests different orders. This could trigger different code path/order of the code and could in theory show problems. (but I think it will be ok)


"what are somethings" :+1:


PlantOperationSchemes (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Need to update this to v4.


Original License.


Yep, every Ability that raises or lowers stat stages (of the Pokemon or an attacker) just activates and shows the standard boost/unboost message(s) as appropriate.

Also, good call swapping "rose" and `amountString`'s places; just keep in mind `amountString` already has a leading space.


> Unlike migrations, every seed file will be executed when you run the command.

I  put this into migrations cause the values should be inserted only once when creating notification_statuses table. Besides this, it simplifies the deployment process, cause you don't need to run two separate commands. Does this sound convincing?


Would you mind pointing out what part of the model association is not right here? I already have generated another migration with in db/migrate and run it with `rake db:migrate VERSION=myTimeStamp`, but according to what I understood from the [official docu](http://guides.rubyonrails.org/association_basics.html) and the article I posted on the issue1, I should just need to link region with instances with a `belong_to`.

For better feedback, it is usually better to comment in the code that should be modified, and not a broad general critique.


Mapbox Light is not a gridLayer so can't be subbed here. Will sub in new gridLayers when created.


shouldn't this be != "X"?


Need to update to v4.


I think this whole function becomes newServiceInfo() or something


Thanks, will fix


I provided a reply for all this in the pull request itself. Please see it.


you're copying the 1x version and then upscaling to 2x. that's basically what it was doing already.


Could this be in a different patch, along with some of the whitespace fixes below?  Since it's not related to this particular patch.


This map has no markers, so there would be no featureLayer here. Should sub out `mapbox.dc-markers` or similar.


!= "X"


Mapbox Outdoors is not a gridLayer so can't be subbed here. Will sub in new gridLayers when created.


Probably not intuitive, but all except the Net35 project uses wildcards to include all the .cs files, so you should just revert 3 of the 4 projects. Just reload the projects or reload the solution for it to load any new files. It's a bit cumbersome, but works well.

One could argue that Net35 could also use wildcards, but it seemed convenient to have one project to "work in" without having to revert and reload changes all the time.


so dirty x]


remove


Was this meant to be merged in master?


It shouldn't have to be set.  If it's not set we can just assume "no".  That way, the patch is also backwards compatible to older magpie scripts.


I was fiddling around here and even though what I was coding was made too much work for the payoff, this had an unnecessary check.


fixed, thank you!


Yeah...

I think we need a big parameter reset anyway. I'd rather do it sooner than later.  But that's out of my control and I certainly don't want heli users to have to do it twice.

Fredrick:

I noticed you skipped the number 6?  If we go forward with this, will have to fix it.


I am not a lawyer, but by all means feel free to adopt our license formulation in your project. It has worked for us. Our formulation is derived from the copyright agreement of IPython: http://ipython.org/ipython-doc/rel-0.9.1/html/license_and_copyright.html


@jbgo Hey, somebody's gotta fight this spec-testing madness.  I don't always agree with DHH, but in this case I do: http://www.rubyinside.com/dhh-offended-by-rspec-debate-4610.html

"... but the proliferation of rSpec and Cucumber makes me sad."
"RSpec offends me aesthetically with no discernible benefit for its added complexity over test/unit."
"I think a cargo cult got started a while back.  People thought this was the way to do it without thinking why they were doing it."

I agree.

And, no, I will refrain from trolling you and enjoy the wondrous value you are going to add here.  Seriously, I'm really excited to see you work on this.

@tpett - have you seen this (chasqui) yet??


I think I'll pass this one. Even if the code would be better, the readability would be poor...


this whitespace change probably isn't necessary


 I would recommend to try the following instead of your RRQ before going to LTC,
 since it is more general and may save additional testing (but not sure if it is good or not).

 The side with more pieces should have more stopping and/or more pushing power,
 may have more ways to defend against checks, and have the additional trump to sacrifice.

  // favour the attacker with more non-pawns.
  bool moreNonPawns =    pos.count<ALL_PIECES>(Us)     - pos.count<PAWN>(Us)
                                        > pos.count<ALL_PIECES>(Them) - pos.count<PAWN>(Them);


This whole commit should be reverted.


si @poguez, están separada la aplicación de Redis. Si no agrego esta condición, la aplicación no tiene forma de saber a donde conectar o marcaría un error al decir que la url no tiene un formato valido.


this is not the periodic sync


I made it 'addService' but that might be better. First attempt at tearing everything out is in https://github.com/BenTheElder/kubernetes/commit/28e009d1dde66d31c5520b035ca091cae4b41874


But it was doing so with browser upscaling, which probably does some form of bad magic and makes the image blurry. In this version the upscaling is done with a point filter and that version is "forced" to the browser via the srcset attribute. So all devices with a pixel aspect of 2 should use the (point-) resized version but not resize it themselves.


@wesleytodd :+1:


@wesleytodd In line 6, will it ever return? It looks like `ended` will always be `false` until line 8.


Whoops. Yeah it's not, what should that say? "Syncing rules" or something?


that might not be so smart.


Nice port!

I would be very grateful if you could help me with one thing in addon.
I did have its effect, but I would like to add a feature like nausea, you can help with this? :)


So that is actually to prevent end from being called twice and causing issues.  This is necessary, and comes from some other module that I modeled this after (I cant seem to remember which one that is atm, sorry)


TeamCity LabkeyRelease15.2 :: CDS postgres 9.4 [Build 17](https://teamcity.labkey.org/viewLog.html?buildId=265238&buildTypeId=LabkeyRelease152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4 (1 new), passed: 19 Build time: 00:28:56
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

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

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

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

```


I don't think I left this without a reason, I will review it.


Generated java file does not belong in Git.  Please add the package dir to .gitignore.


picture a diagonal like this: "\"
a 2x2 pixel would look like:

    x.
    .x

if you upscale that, even with a point filter it'll be:

    xx..
    xx..
    ..xx
    ..xx

but rendering at 2x would get you:

    x...
    .x..
    ..x.
    ...x


Remove from git


I was looking at some examples of embedded IP with FreeRTOS. Mainly for a
UDP receiver. Got in a hurry and didn't really mean to add the entire
folder..
On Aug 6, 2015 9:36 PM, "Justin Buchanan" <notifications@github.com> wrote:

> What's all the FreeRTOS stuff for? Is this in place of the builtin mbed
> RTOS?
>
> Also, it's best to include large chunks of external source code with git
> submodules so we don't track their content in our repo. I can show you how
> to set it up if you want.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jjones646/robocup-software/commit/8acf298ed9f5609b99350622e93520286307beb1#commitcomment-12579101>
> .
>


works like a charm now.


what if curr is <domain>/foo/bar ? (in fact, don't pass root, just strip root from the link)


@Grrandi Kävi ilmi että latexmk:lla saa toteutettua make watchin :)


Yes, but that was intended. Bigger font sizes would be a better way, but trouble with bitmap fonts. See the issue


A pleasure ;)


Wow, you really put in a ton of work here. A couple of thoughts though:

* You really shouldn't copy & paste the constants in here, I feel it's too prone to have bugs. Instead use the construction methods that already deal with these calculations, like `Volume v = Volume.FromCubicMeters(val);`

* I want to automate this, building on top of the powershell scripts we already use to generate unit classes and tests. This way you always have all the units covered, no matter how many we add in the future. The scripts are located in [/UnitsNet/Scripts](https://github.com/anjdreas/UnitsNet/tree/master/UnitsNet/Scripts). I understand you may not be familiar with powershell, but the existing scripts should give you a good starting point and I'm happy to help out if you have any questions.

* I propose we make all these extension classes `partial` and name them `NumericExtensions`, since we are really extending the behavior of numbers, not unit types. So we have `NumericExtensions.Area.cs`, `NumericExtensions.Length.cs` etc.


oops this reverts a good change from before


Fix for issue #5 and #4


Very cool, thanks, I did not know about the sul_styles gem.


Not a formal ticket--did this in the process of examining homepage performance


Ah, THAT's the thing that broke. You just saved me a frustrating 10 minutes.


this "ktype" is the same as "uprintse"

Fortus 400 is fox2.


ррррррррр:(


other than on `AbstractSearchPerformer`'s `.stop()`, the `onCompleted()` method isn't invoked. (unless it's supposed to be invoked by the framework upon some other magic condition)


Or up, rather.


I applied the proposed changes to both:

``` */Sublime Text 2/Packages/SublimeREPL/config/Ruby/pry_repl.rb ```
``` */Sublime Text 2/Packages/SublimeREPL/config/Rails/pry_repl.rb ```

and now both REPLs work perfectly for me in ST 2. This really needs to be merged.


How far are you into FC compatibility?


Add a space between "(int) pid", same on line 1222


Clang static code analyzer complains in this line with message:

    Value stored to 'enoughBytesToParse' is never read


I was going to say that the "-f" argument could be dropped, but I see now that it cannot (or else there will be different attrpath depending on what channel you're on). But you can drop the trailing wildcard.


@rafaelfranca Could you elaborate a bit more? I can imagine `travis` guys happy because of the resource saving but I don't see how this benefits anyone else in the `Rails` community.

Even if it's failing this is useful for anyone considering working on JRuby/rbx support, cause it shows a quick overview of the amount and type of work that needs to be done.


Fuck you iron man,you fucking idiot!!!


I'd like to not encode details like this here.   It would be much nicer if the whole iptables API worked in the same basic mode of a list of arguments which are processed by the iptables module.  I'll comment more on this as I read on to the
use-sites


try using net/url instead of this: https://golang.org/pkg/net/url/

Pass a URL object to htmlParser (instead of a string) and call Parse() on it with the current link to get absolute URL in the context of this root


I don't think this will ever return


@jbgo so I'm too lazy to do a PR or whatever, and feel free tell me to "stop annoying me I'm still prototyping", but should this be `'transcoder.video.error'?

I like the examples - seems like a good way to wrap your head around things.


I don't think this will ever return (see above)


merged from 'reformatting' branch which :-
Reformatted in braces in 'Ratliff' or 'banner' style.
Fixed some odd dangling else's in #if defines.
Changed "for (;;)"'s to "while (TRUE)"'s and added TRUE and FALSE to defs.h


Does this overwrite `Org` from line 1?


@xjlin0 — Yes, we did!  So nice of you to say so Jack.

In this context, smarter_csv's chunking feature seems useful. Makes it easy to pass chunks of the CSV to workers! Couldn't find anything like that in the standard Ruby CSV library.


It also benefits the Rails team, because we have more free workers to give us a faster feedback about pull requests.

> Even if it's failing this is useful for anyone considering working on JRuby/rbx support, cause it shows a quick overview of the amount and type of work that needs to be done.

Yeah, I can see that, but the test suite is still available so people considering working on it can clone and run tests locally to work on it.


This has been removed. IIRC yes, all of this behaved exactly like the usermode proxy just with iptables replacing the usermode bit. Most of this was trimmed out in https://github.com/GoogleCloudPlatform/kubernetes/commit/28e009d1dde66d31c5520b035ca091cae4b41874


Better!


Whoops, yes that is correct.


I'll create the new patch.


:+1: added example to reame


@wsnipex: Could you check if this is ok to get `libuuid` into depends for RBPi? A test build worked fine (even though initially the download of the `util-linux` tarball failed because it wasn't synced to the mirrors yet) at http://jenkins.kodi.tv/job/LINUX-RBPI/4689/.
I checked ubuntu wheezy and it's using uuid-dev from util-linux-2.20.1 so that's what I went with.


mydata version: https://gist.github.com/raprasad/8c325cc664aecd4b5699


Main thing on log is info level. Normally a library should be careful
logging at this level, especially something recoverable.

The log statement about how many are present after sleeping.. Wouldn't that
only be important if it was still full? If so, you would hit the first log
message again and be able to tell, right?


But before you know it, somebody will have messed up the case in some subtle way, and fixing it will cause all kinds of grief with version control or operating systems that are case-insensitive and hence can't handle a rename properly.

Also, there are file names, and you'll want to deal with them from the console at some point. And consoles don't generally do smart camel-case aware completion. Instead, it'll be extra annoying to open files because now you have to get the capitals right.


@asmagill This should be addressed by 84f2727. It's some pretty ugly objc to bind to SUUpdater at runtime.

(The reason this is different to the menu items is that they are just doing simple bindings to Sparkle, where the preferences window does a KVO binding, which raises an exception if it can't find anything, causing us to die)


Not sure if it's the case here but since this never worked well with lollipop on every Moto G, replace the binary here too


Something like:
 `iptables.WriteRule(buff bytes.Buffer, pos iptables.RulePosition, args ...string)`
 `iptables.WriteChain(buff bytes.Buffer, chain iptables.Chain, bytes, packets int)`
?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.448.562f001300e918857b4a0826f311d58cb32abf0f](http://build.palaso.org/viewLog.html?buildId=60798&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3306, ignored: 124 Build time: 00:10:39


Not (a OR b)


Any specific v8 versions or Windows versions I should look at? I've haven't had any problems with fibers in Windows 7.


@R-Lefebvre Oops, my bad - added a commit that fixes it, let's squash it before merging.


but the defaultTo is a string


Does it @MatthewPhinney ?


Pager code
homepage version: https://gist.github.com/raprasad/3d6c6bc3b7deee51304b
mydata version: https://gist.github.com/raprasad/8c325cc664aecd4b5699


it's postgresql's syntax, you pass arrays as strings


cases?


Raise Not implemented.. On user and pass not null


I find the include from platform specific files like winsock2.h makes more sense in the top class NetworkWrapper which like the name says should include all specific >network< files.


Actually, checking for `!versionresult` here doesn't make any sense. This is fixed in commit 22f5187.

Fortunately, the code in this commit—albeit illogical—doesn't do any harm as it may only cause an error in the `ALTER TABLE` command below, which is wrapped in a `try` clause. Since we always execute the `CREATE TABLE IF NOT EXISTS` command afterwards, we're on the safe side. (Still good to have it fixed in commit 22f5187.)


fix-writing-output-without-permissions-5063 (nealkruis) - Win64-Windows-7-VisualStudio-12: OK (1795 of 1795 tests passed)

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


This test is not quite as thorough as it could be, instead use a function who's return value is dynamic:
```
var expected;
var narrowedCollection = Books.where(function () {
  return {genre: expected}
});
test.equal(narrowedCollection.find().count(),  0);
expected = 'science';
test.equal(narrowedCollection.find().count(), 2);
```


Ok, молодец, разобрался ;)


is there a `test.isString`?


Figured it would be annoying.

On Fri, Aug 7, 2015 at 3:35 PM, divmazie <notifications@github.com> wrote:

> Ah, THAT's the thing that broke. You just saved me a frustrating 10
> minutes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/divmazie/jonathancoulton-theme/commit/664b59f198ce827dd017c9d53eecf10be1c96590#commitcomment-12594176>
> .
>


The -n option for nodata value doesn't work any more with this change. How am I supposed to get my -n option back, without defining the mask?
Thanks,


On Ubuntu (or at least Kubuntu...), chromium puts that lib:

/usr/lib/chromium-browser/libffmpegsumo.so


I agree about having those log statements as info. I changed them to debug since that was a mistake on my part.

The log statement after the sleep will give you an idea as to how far behind you're falling. In other words your sleep time might not be enough to empty the queue. In our case by the time we got to the first log statement the queue would always be full because we were consuming from the queue way too fast. So we could gauge how much speed we were gaining with that second log statement after the sleep.


Looks pretty similar to what my code in extras is doing... I'll test it out with the new code later today and let you know and if it's all good, I'll add it to MJLua.m.

> On Aug 7, 2015, at 2:49 PM, Chris Jones <notifications@github.com> wrote:
>
> @asmagill <https://github.com/asmagill> This should be addressed by 84f2727. It's some pretty ugly objc to bind to SUUpdater at runtime.
>
> (The reason this is different to the menu items is that they are just doing simple bindings to Sparkle, where the preferences window does a KVO binding, which raises an exception if it can't find anything, causing us to die)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Hammerspoon/hammerspoon/commit/f00cdec2f35d5829c52f7ecf77ebc7fa73feb1b7#commitcomment-12594422>.
>


You've used this same code on line 37 - this could be a good case for writing a method in your model that returns this string for you! Then you would only have to do something like:

`puts contact.display` (or whatever you'd like to call your method).


This link is broken


А вот с кастомерами - схитрил. Номер кредитной карты немного длиннее, чем 5 цифр ))


hm


> Stack overflow

should also be

> Stack Overflow


From a user experience perspective, perhaps you could remind them what the available commands are?


OK, looks good.


fix-writing-output-without-permissions-5063 (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1272 of 1272 tests passed)

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


Unhandled Fundamental Failure


test-s3-fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


test-s3-fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

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


test-s3-fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


This was done midway to do the fetch


This was done midway for the fetch. Final HW in last commit


#8 #2


97903100_SolarCollectors (macumber) - Win64-Windows-7-VisualStudio-12: OK (2219 of 2241 tests passed)

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


Thanks.


Instead of failing, I suggest doing something along the lines of what I did in the `windows` branch, [here](https://github.com/ncreep/ide-backend/blob/f6b1f64b5cafdb5be54e6d866e9f7306b1474d43/ide-backend-server/RequestRunning.hs#L69). I.e., encoding the failure to support the feature in a value and send it back via RPC, then let the client [handle it](https://github.com/ncreep/ide-backend/blob/f6b1f64b5cafdb5be54e6d866e9f7306b1474d43/ide-backend/IdeSession/GHC/Client.hs#L247). Since there's no need to crash the whole server if something is not supported, and handling the failure on the client side is much easier.

Also, I suggest moving the whole request running related code into a separate file, since having the conditionals with large pieces of code in the middle of a file that deals with other things as well is somewhat confgusing.


Yours didn't suck...
because there wasn't anything to suck :P


I added the macro `build_addon_library`: https://github.com/garbear/xbmc/compare/92075ef...retroplayer-addon-libs

I copied the `build_addon` macro (first commit) and modified it (second commit, to be squashed, so you can see the changes). Because it uses the same algo as `build_addon`, the library should always end up in the same place as `game.netplay.so`

netplay cmake now looks like this: 4719db4

obviously, common code could be factored out of `build_addon` and `build_addon_library`, but it looks like it solves the problem

regarding the platform/arch tests: I'm trying to match the logic in [libXBMC_addon.h](https://github.com/garbear/xbmc/blob/retroplayer-15beta2/addons/library.xbmc.addon/libXBMC_addon.h) so that the netplay helper libraries can be loaded in place of the kodi libraries


r+


Good point!

I figure we can address such things when the windows branch gets merged in.  (sorry for the conflicts this change will cause..)


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/b6b286d314a98a798e2f38acbc220ad1a3f7d23e


merging graydon/stellar-core/bug-432-incremental-apply = b6b286d3 into auto


graydon/stellar-core/bug-432-incremental-apply = b6b286d3 merged ok, testing candidate = e688ed99


more words?


@dangra no worries about the back-and-forth, I like it. :)

So, similar thoughts passed through my mind, I had thought Parsel would expose a function:

```
def build_parser(parser_cls, text):
        body = text.strip().encode('utf8') or b'<html/>'
        parser = parser_cls(recover=True, encoding='utf8')
        return etree.fromstring(body, parser=parser)
```

I kinda like the `extended-html`, but my concern is this: https://github.com/eliasdorneles/scrapy/blob/migrate-parsel/scrapy/selector/unified.py#L17
That bit there is validating the type for the Scrapy's Selector subclass against `('xml', 'html')` so it would fail -- unless we change it to use parsel's `_cgroup` and then we need to promote `_cgroup` and come up with a good name for it. :)

What do you think?


Shouldn't it respect the variable name its provided though? It works initially and then gets reverted to default


people lookup through the whitepages has been broken since the initial implementation. "cant connect" looks like an api issue. tried reporting this on jira but got shot down. so figured id post it here. maybe someone can acknowledge it or are we not worrying about it because cyngn os is moving to "truecaller" for lookup


I've managed to deal with the conflicts, so no problem.


oops! This is not something that is intended to be committed, just a local change to avoid out of memory error. And read from config file is the bright idea. I'll look into that.


都很大方回电话


retorno


> That bit there is validating the type for the Scrapy's Selector subclass against ('xml', 'html') so it would fail in the case of 'extended-html

are you sure we can't just return `st`  without checking and let Parsel raise ValueError exception because value is not in ctgroups?


The message should probably be updated to reflect this change


So in a unit test you want to isolate the object you are testing. Isolation tests you are testing how two or more entities (objects, DB, other servers) interact together.


@cyanogen @romanbb @Danesh @invisiblek @maniac103 ^^


After the fact I realised the original implementation was actually returning an incorrect result. I changed the test to reflect what we we actually wanted to get back.


oh, you're right, that's much better! :+1:


Dear professor，
    My group has received a program ahout simulating the numa in a simulator.I have read your program published on the github and known that you have implemented it .I have download the Gem5 simulator and changes the python file according to your project.But there comes to a error said : "fatal:Cannot have two CPUs with the same id (0) @ cycle 0  [registerThreadContext : build/X86/sim/system.cc,line 225]".
    Would you please give the simulator to me ,or give some good suggestions to me ,It is so important to me . Hope to receive your response.Thanks very much.


This is the main key in terms of loading the has many through associations (eager loading) before doing the loop. This prevents what is known as an N+1 Query.


aczsd


fix-run-without-permissions-5062 (nealkruis) - Win64-Windows-7-VisualStudio-12: OK (1706 of 1795 tests passed)

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


Muy buenas tardes amigo me puedes ayudar con la base de datos por favor
SAludos desde Ecuador mi nombre es Anthony


`each_with_object` is a good one to know. It lets you pass in an object and modify it in an loop then at the end of the cycle it returns the modified object as a result. In this case we are passing in an empty hash and adding things to it.


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1760 of 1795 tests passed)

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


This is a violation of the [The Law of Demeter](https://en.wikipedia.org/wiki/Law_of_Demeter). I fixed this later.


Oops, I forgot. Thanks for submitting this :-)

- Charlie (mobile)
On Aug 6, 2015 15:47, "R. Tyler Croy" <notifications@github.com> wrote:

> Remove the requirement for varargs when generating Class objects for Java
>
> This is effectively a patch written by @headius and tested locally and
> applied
> by me. In my use-case this allows Java code to properly reflect back into a
> Ruby-generated Class and invoke a method which takes no arguments.
>
> Fixes #3206
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/rtyler/jruby/commit/89179e9aa55088e513bbacf8292b924a3311e3e3
>


@Ngrd Forgetting a comment here aren't we?


These aren't "apps", they're system firmware... I thought the distinction was enough to not put them in knownApps, not that this is necessarily the best place, haha.


这个仓库里都是算法题目的解答


这段检测代码一般都是放在lua文件的最后。并且这种方法有很多局限性，现在已经不推荐了


Also, oops I was passing in the name then finding it again here. 😊


这个工具非常重要，建议多写写，给个例子。


很多css3的知识，可以谷歌下f12 看下各个css规则的作用。


Don't forget libcore is LGPL so this section needs updating.


OR条件だと、isLastWordAddressの方の条件（続きを読むとURLがかぶってない）にマッチしてしまうとそのまま
subnoteを返してしまっています。
AND条件にして続きを読むとURLおよびキーワードいずれもかぶってない条件の時のみsubnoteを返してOKという風にしないとダメそうです。


The original format had a key name that was the name of the ingredient as well as a key called `name` that was in the sub hash with the same value (the ingredient name). So rather than the duplication I think an array of hashes would have worked as well (depending on how you plan on using the result).


Instead of building a hash we build a custom object. That way can make sure it has values, add behaviour if it ever needs it, and use "dot" notation instead of the square brackets notation in our views for example. (e.g. `detail.name` instead of `detail["name"]`)


I scrubbed these files from the history of all our repos since they were getting quite large.


文字の長さを取得するには
```
keyword.getValue().length();
```
を使います。


good


This is a change in behaviour of this method and so will break anything that was using it before. You can change it back if you want but I thought it might be an example you could reference another time.


oh


RoomAirflowNetwork (lgu1234) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Helpful `delegate` is a helpful method from Rails. (There is a non-Rails Ruby equivalent if you include a module called `Forwardable` if you ever what this in a non-Rails object.)


okay, fixed harcoded check, I'll work on the rest tomorrow cause I'm already making silly mistakes by now. =)


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/74509107


I have been using this pattern a bit lately. It just helps in building an instance in a specific way or even just making nicer method APIs.


Isn't that feature available only for the private repos?


For this test I am acting like the class is a different object than the instance (which they are) even though syntactically they are in the same file. Because of this and given this is a unit test I am stubbing out the call to another object (the instance) and just making sure it is passed the expected arguments.


Others may not agree with this but it is an idea I have been playing around with lately and have half of a blog post about it so far. 😀


Nope: http://blog.travis-ci.com/2014-12-17-faster-builds-with-container-based-infrastructure/


Trying to make my tests read like human documentation as much as possible. You your tests with `rspec --format=documentation` to see how your tests would read as documentation.


Finally, using the new object instead of the hash.


Le 06/08/2015 13:24, Michael Burman a écrit :
> I still don't understand why not collect() ? This subscriber just
> repeats that? You can have another sequence inside flatMap

As dicussed on IRC, I've replaced the Observable#create with #collect

See commit b4e806c2bbe6056d84f938a6415d3d9cf9987ba2


fast-forwarding master to auto = e688ed99


What he said^^ :-)


Thanks :)


LGTM


现在不乱码了 ！！ ！


可以


@AkimitsuHamaguchi 個々での検証はしてます。