Idea for in game or tournament matches with tag team

During the start of you and your opponents turn begins after you have drawn, but before you have attacked, you gain 3 turn counters to activate these power ups depending on how many you have;
• 5 counters or more: before you draw you can remove 3 counters but cannot draw at the start of your turn, but you can add an ace of your choice from your deck to your hand and shuffle, but cannot use it until your next turn, or if you cannot add an ace, you can gain 5 points instead,

• 10 counters or more: if you have any hearts in your power zone you gain an extra 2 points per heart in the power zone per turn

• 12 counters or more: by removing 4 counters, if you have a spade in the field zone or power zone, your opponent’s field cards loose half of the attack and defence for 1 turn, this can only be done once per turn.
-Tag team, If this is a tag team match all your opponents cards loose half of their attack and defence.

• 15 counters or more: you can remove 3 counters to have one card of your choice attack twice during your turn only. The card you choose gains +3 in attack only.

• 20 counters or more: You can only during your turn remove 7 counters to discard all cards from your opponents hand to the used area.
Tag team - If you are in a tag team match both of your opponents lose all their cards in their hand.

• 24 counters or more: you can remove 10 counters to remove 10 cards from your opponent’s deck and your opponent loses 10 points of their points.


Combine this and the previous line and re-align the following code. Missing space after `()` and before the `{`


This probably shouldn't have a delay. The time should just fire right as the page is loaded to set the "isLoaded" var to true and the delay should be in the css.


We have "magic statics", no?


Let's kill "meet up" and leave the others.


Please fix the spelling of manger.


This has the benefit of reusing all the same code, so feel free to treat this is a nit. I might have done this as so:

```js
if (handler === undefined) {
  var noHandler = fromError(new Error('No onRequest handler for type "' + requestType + '"'));
  waiting.forEach(function (w) {
    me._sendMessage('res', requestType, { id: w.id, err: noHandler });
  });
  return;
}

waiting.forEach(//...
```


Surprise surprise... We have a merge message here.


10000 cannot be 5 seconds


you can't just look at pitch here, if y changes you also want to change the head angle


Kill this. Consider: "This Park Slope restaurant, on a quiet block midway between Prospect Park and the clamorous eateries of Fifth Avenue,…"


Hmm. I really like the login loaded fade-in effect. I think since we are using ng-animate we should use the ng-animate classes. It would make the changes look something like this:

client/app/account/login/login.html:
```
<section class="fill login-page">
  <div class="container banner">
    <img src="assets/images/CFALogoLarge.jpg" alt="Child First Authority Logo" class="logo" ng-if="isLoaded">
    <p class="lead" ng-if="isLoaded">Student absenteeism tracking and reporting</p>
    <div class="glogin" ng-click="loginOauth('google')" ng-if="isLoaded">
      <span class="glogin-icon"><i class="fa fa-google-plus"></i></span>
      <span class="glogin-text"> Sign in with Google</span>
    </div>
  </div>
</section>
```
client/app/account/login/login.controller.js:
```
'use strict';

angular.module('app')
  .controller('LoginCtrl', function($scope, $window, $timeout) {
    $scope.$on('$viewContentLoaded', function() {
      $timeout(function() {
        $scope.isLoaded = true;
      });
    });

    $scope.loginOauth = function(provider) {
      $window.location.href = '/auth/' + provider;
    };
  });
```

client/app/account/login/login.scss:
```
@import '../../common.scss';

// Colors
// --------------------------------------------------

$btnText: #FFF;
$btnTextAlt: #000;

$btnGooglePlusBackground: #DC4E41;
$btnGooglePlusBackgroundHighlight: #C53727;

.login-page {
  background-color: $cfa-background;
}

.banner {
  text-align: center;
  color: white;
  .logo {
    max-height: 300px;
    &.ng-enter {
      transition: opacity 1s ease 0.5s;
    }
  }
  .lead {
    &.ng-enter {
      transition: opacity 1.5s ease 0.5s;
    }
  }
  .glogin {
    text-align: center;
    display: inline-block;
    white-space: nowrap;
    color: $btnText;
    font-family: 'Roboto', sans-serif;
    background-color: $btnGooglePlusBackground;
    border-radius: 5px;
    &:hover:not(:active) {
      cursor: pointer;
      background-color: $btnGooglePlusBackgroundHighlight;
    }
    span.glogin-icon {
      display: inline-block;
      vertical-align: middle;
      width: 42px;
      height: 42px;
      line-height: 42px;
      font-size: 18px;
      border-right: white 1px solid;
    }
    span.glogin-text {
      display: inline-block;
      vertical-align: middle;
      padding: 0 18px;
      font-size: 14px;
      font-weight: bold;
      font-family: 'Roboto', sans-serif;
    }
    &.ng-enter {
      transition: opacity 2s ease 0.5s;
    }
  }
  .ng-enter {
    opacity: 0;
    &.ng-enter-active {
      opacity: 1;
    }
  }
}
```

ng-animate handles adding the animation classes.

Instead of having the delay on the timeout in the controller for setting the isLoaded variable to true, The same effect can be done by adding a transition delay in the css `transition`.

I think reducing the time of the animation by a bit might be good too. As the novelty of the animation wanes, waiting 3.25 seconds might be too long.

Nice work on this change list.


This is a decent start but it would be nice to get another tweet or two at some point (especially from a player that is not necessarily part of the team)


Kill this; it's allowed by The New Yorker.


Ok xD sry


Kill this; they have slightly different meanings.


Following what we did for gamma, we could simplify this by setting sigmaMax = p2/(1-p2). We just need sigmaMax to define a reasonable range for the solution algorithm.


And 0.15.0 is not existing this is fake for trolling


If originalDate is milliseconds and now.getTime() is millseconds, why do you need to "/ 1000" it?


Dont use member variables. Just local to where they are used


@kuangami


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


Display calendar with using the Calendar and GregorianCalendar classes


typo


会玩，不过国内访问Github的速度这么慢 你确定要用😂


Can we exclude "Bare Naked Ladies"?


Kill this, it's idiomatic.


call this in onResume


fixed in https://github.com/SeleniumHQ/selenium/commit/9eb5e30af65af13bb777aadff93e9b2ef9e7a273


call this in onPause


Kill "never before" ("He has played competitive Scrabble in Warsaw, Prague, Oslo, Las Vegas, and Cleveland, but never before encountered such fanfare.", and others from https://www.google.com/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=site:newyorker.com+%22never+before%22)


@zhangjingye03 谢谢学长！


Is it particularly onerous to expect that the user is aware of the details of the NDK?  It seems that this would be best done by forcing the user to be responsible for providing either:

a) a proper sysroot
b) providing the required linker search path to the linker

Driver code tends to already be hairy enough in my experience with clang and gcc.  I dont see why adding something like this is beneficial to maintaining the driver code.

The ANDROID_NDK_HOME variable is part of the NDK and I can see why this would be rather convenient for the android build itself.  However, this linker search path is something which would be needed for normal linking in the case of C/C++ code.  Why is swift special here in that it should honour this environment variable?


Interestingly, I think this entire phrase is more unnecessary than redundant. When I search The New Yorker archives (https://www.google.com/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=site:newyorker.com+%22right+now%22), all the sentences are improved by striking the whole phrase. Can you split this off into a separate function?


Typo fixed


A comment about why this particular C++ library is being used would be nice.  IIRC, android also has a copy of the STLport C++ library as well.  This path also seems to be tying the swift driver into the tree layout for the android source code which is ... a bit odd IMO.


I'll work on this, I don't know how easy it will be to do, but I think it might be possible with D-Bus.


Kill this; it's idiomatic (https://www.google.com/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=site:newyorker.com+%22every+now+and+then%22)


Remove "onto" and "onward"; I don't want us to get in the habit of banning words unless there is good reason to.


why the filename "fullchain.pem" when all the other certs have a .cer file ending? IMO it should be named "$DOMAIN-bundle.cer" or similar to follow the same scheme as the other files.


Great! Thanks @wwmayer! :smile:


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


Kill this, it's okay and useful in places.


Cleaner code.


I cannot find the way to reproduce this bug, even using your own code from https://github.com/xpenatan/dragome-backend


@gribozavr This change means this documentation comment for `_failEarlyRangeCheck` is no longer accurate, right?
```
/// The default implementation for forward and bidirectional indices is a
/// no-op.  The default implementation for random access indices performs a
/// range check.
```
Does this still need to be a customization point if all indices are comparable?


```
def btce_order_book():
  # Order book URL
  url = "https://btc-e.com/api/2/btc_usd/depth"

  # Response object
  urlResponse = urllib.request.urlopen(url)

  # Read response
  bitRead = urlResponse.read()

  # Convert bitstream response to string
  stringResponse = bitRead.decode("utf-8")

  # Load the associative array from the string
  IOobj = io.StringIO(stringResponse)
  assocArray = json.load(IOobj)

  def canonicalize(book):
    return book

  return canonicalize(assocArray)
```

comments should explain the "why" question of the code, and code should be self-explanatory with "how"

there's no need to comment things that are obvious -- semantic code does that by virtue of being well written code.

for example, you should not write:
```
def one_plus_one():
    # adds 1 to 1
    x = 1 + 1
    # returns x
    return x
```

whereas you should write comments like:

```
# due to a (example) bug in python, the native 1 + 1
#returns not 2, but a float of about 2.00000001
# according to [bug report], a workaround is to
# wrap the addition in a variable in a function
def one_plus_one():
    x = 1 + 1
    return x
```

also, python style guides insist on using snake_case, not camelCase


Let's make this "my personal opinion" to "my opinion", because an opinion can be shared, so the phrase is not redundant by itself.


It's just because the official LetEncrypt generate it as `fullchain.pem`.


Let's include an entry for the singulars, too.


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


@lvyb
This is new branch.


Kill this (Garner again).


Kill this, although the the "alternatively" is almost always set off in commas, so I don't think this will catch anything anyways.


1. Stack machines can have a `dup` operator and could also load from elements other than just the stack top. It is not clear that either would translate cleanly into the current wasm ast. The functionality of `dup` would seem to be handled via local variables in wasm - that is a value could be stored in a local and reused, and similarly for loading values other than the top. Can you see a principle that would rule out such stack operations or others such as a `swap` operation? What would you say to someone submitting a PR to add such operators?

2. From what I have been able to make of the plans, it would seem that multiple value expressions would be represented by just one effective stack element?  So that conceptually each wasm stack element is a tuple of values. Some stack machines represent multiple values as multiple stack elements and might even push the number of values on the stack. Has this point been discussed and are there any plans?

3. From what I understand, a post-order interpreter might need to box values on the stack and they would need to be typed, or can it depend on the consumer always reading the exact expected typed value?  In the case of `block` effectively popping values, it would need to know the size (type) unless the stack elements are big enough to hold any sized value, including a tuple of values up to the wasm limit on the number of values? Would you concur that boxing and typing is required?

4. Re: 'the stack machine would be constrained such that one can always statically know the types, definitions, and uses of all operands on the stack'. A 'stack machine' describes a virtual machine to me, something that can be interpreted, but 'statically know' suggests a 'stack' based AST decoder. If talking about a pre-order versus post-order encoding it does not appear to me to be useful to discuss 'stack machines'. How are stack machines relevant?

5. Personally I don't think wasm to date is a 'stack machine' rather uses local variables to connect operators. In the limit expressions could be almost entirely avoided and it would appear that expressions could be removed from wasm and it should still compile to the same quality of code.  Perhaps from the compilation target use case it is fundamentally a 'register' based code. What do you think?


only launch at trip creation time, e.g. addWatchersFragment

and StartupReceiver (if the user has a trip)


I agree. Wasn't aware of asprim, looks like a better way to do it, at the cost of another dependency.


好吧！感谢解答


Hm, this is weird. "He was output out of the cannon" is not fixed by "He was output the cannon".


well the official client also names the other filenames differently, doesn't it?


If only the :url from params is being used by this method it would be better to pass it directly. Otherwise you are passing a params to the method that could contain anything. It's a good practice not to do that.

```ruby
@slyp = Slyp.fetch(params[:url])
```


Why did all the prefixer mixins get commented out here?


I think this is too common to ban (see https://www.google.com/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=site:newyorker.com+%22outside+of%22).


how about  `fullchain.cer`  ?


Done so btw. Build version is calculated from 1st January 2015. But better
find a more special date. Date of first commit of the mod? Change to it
before merging.


Why is this being defined as a global variable?


Yes, the documentation is not correct.

It still needs to be a customization point since (1) there can be a faster way to check than compare indices, (2) this method can check that indices are also valid, not just in range.


Well it does in a way, it supports discarding zero or one value in all but the last expression of a block, without an explicit and matching `discard` consumer. The ABI use case does seem important and would it 'introduce some complexity for some implementations' if the MVP only supported multiple values for function call results, rather than for expressions in general?


Kill this, way too many false alarms.


I know I recommended this pattern but i think it will be easier to follow this code if it was laid out like this:

```ruby
if @slyp.valid?
...
else
...
end
```


much better - will avoid confusion about pem vs cer


testing the github comment functionality


Eh, this seems fine to me.


I'd make this one line, no need to define current_user_slyp


this is a line note


could you just pass current_user.id as the second argument and encapsulate the UserSlyp query in the SlypPresenter? i.e so SlypPresenter expects a slyp and a user_id as arguments


renamed to `fullchain.cer`

Thanks.


Hm, this isn't really redundant if you think about it as specifying which part of the hand. "He put it in the palm of his hand" as opposed to "He grasped it with his fingers". Maybe "palm of his hand" should become "his palm", but I don't feel strongly about that. This seems like we can remove it.


Thanks, good point.


Looks great. But a few possible improvements.
 - Save to local storage only once after reading all the user preferences instead of saving for every preference in the loop.


`${PWD}` gives us the whole path of the current working directory.

> `/Users/BEAN/dn-m/Frameworks`


But then here, were are try to go to the grandparent directory of this whole path:

>`../../Users/BEAN/dn-m/Frameworks`


@sils1297 How to write the custom conversion function.
What i have written fails the tests. Can you help me in correcting it.
Also if I do simple str then tests pass.


what? 你怎么和riicy那群人扯上关系的 :smile:


Oh wow, right obviously.

OK. Give me twenty minutes to clean this shit up. Going to clean up my branches and fix this. You can close the extant pull requests.

On March 12, 2016 at 23:38:29, James Bean (notifications@github.com) wrote:

But then here, were are try to go to the grandparent directory of this whole path:

../../Users/BEAN/dn-m/Frameworks

—
Reply to this email directly or view it on GitHub.


Testing for authentication shouldn't be done inside the tests for a single action if the authentication is a before_filter that is applied to every action that happens on this controller. Instead you can test in a seperate describe block like this - requires you to setup a dummy action on the controller:

```ruby
controller do
  def foobar
  end
end

describe "before_filter#authenticate_user!" do
  context "blocks unauthenticated requests" do
    get :foobar
    expect(response.status).to eq(401)
  end
end
```

And


给我自己写个comment


Зачем раскомментировали эти строки?


写第二个comment


The sign_in method can be moved to the top of this describe block and put into a before_each block so that it happens for every test. Again, this is because authentication doesn't need to be tested in this action since it is globally applied across this controller.

```ruby
before_each do
  sign_in FactoryGirl.create(:user)
end


For the record I forgot the first commit message was a :seedling: . I just thought :evergreen_tree: was an appropriate message, even without a historical context.


@zhangjingye03
执信OA呗，那次准备报选修二时好奇去看了源代码。发现挺有用:joy: 所以就拿来用了。


@zhangjingye03
而且还找到了他们的试用OA，在那里测试hhh
最后还找到了他们的项目开发经理，勾搭成功√


嗯况且你知道riicy是啥？惊悚！


This second argument could become user_id as mentioned on comments to previous commit. This presenter only really needs a slyp and user_id after all and it can figure out the rest.. Will make your life easier if this gets reused in other controllers as you won't have to duplicate the UserSlyp.where.... query in all those places.


Indentation ?


In the else case, instead of returning `type` we could do something like

`strictNulLChecks && !assumeTrue && isMatchingReference(expr, reference) ? nullsOnly(type) : type`

I.e. if type is `T | null` we can strip `T` as long as T is truthy, this would work in the case of `Object | null`

There may be no value in knowing that in the following:

```js
if (foo) {

} else {
  /* type system knows foo is always null */
}
```

In which case such a narrowing would provide little to no value.


Deleted title? I want this to go away now.


Just cherrypicked to see if the changes in mdss_dsi.c and mdss_dsi_panel.c helps with SOD. Didn't turn out successful, though :(


Hi @plaes!

I think it's not a problem. As soon as PySide2 becomes stable Qt4 support will be dropped permanently. So the ifs will be dropped as well.

Moreover we can do something like this:
```
#if QT_VERSION >= 0x060000
  //Qt6 code here
#elif QT_VERSION >= 0x050000
  //Qt5 code here
#else
  //Qt4 code here
#endif
```

And this style is my personal preference (latest version on top).

Cheers,
Mateusz

P.S. AFAIK Qt4 has reached it's end of life on 19.12.2015.


ASSSSS


Didn't experience one so far, but will test more and see if this makes troubles. Thanks.


FWIW my NetBSD uniq has no -i option but is very nearly the same as the OS X one:
> usage: uniq [-cdu] [-f fields] [-s chars] [input_file [output_file]]


Have you tried using just a polygon, like how you can do with the triangle list in rviz's markers: http://wiki.ros.org/rviz/DisplayTypes/Marker#Triangle_List_.28TRIANGLE_LIST.3D11.29_.5B1.1.2B-.5D

I think it might be more robust for you if you just create a mesh out of points (either a two-sided face or a very thin triangular prism) rather than relying on a mesh file.


looks good!


Not sure if this is WIP, because you didn't add another comment to the PR. This is not in the right place; "-args" is new in Go 1.6, and so can't be used unconditionally.

Instead, internal/testflag.Split should be updated to support -args. The user can then pass it on the command line if they need the disambiguation, but it'll only be expected to work correctly for Go 1.6.


estamos modificando


thank you!


unused?


Why remove these?


@akshayaurora clipboard is broken after this commit


Isn't this redundant?


I think we eventually got there with 5.1

On Sunday, 13 March 2016, kdon80 <notifications@github.com> wrote:

> This never made it to Laravel 5, did it?
> I'm still trying to figure how to OrderBy a Union.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/65e45d6f03de3c8403d606bbbefe49652909444c#commitcomment-16665515>
> .
>


Is the NMS necessary here?


補足、勉強になります。

現状のコードでは、

- クライアント側（フォーム）
    - 空白かどうかを判断する
- サーバ側（データベース）
    - unique属性で同一nameを弾く

というバリデーションを行っているという認識でいいですか？
（バリデーションとしては貧弱すぎると感じていますが）


Is the NMS necessary?


@AbdealiJK ^


This is causing my builds to fail:
cp: ‘/home/ragora/Desktop/Leonard/Sources/Ecc/Parser.cpp’ and ‘/home/ragora/Desktop/Leonard/Sources/Ecc/Parser.cpp’ are the same file

Probably because I'm doing in-source builds.


What I meant is that the ``Converter`` should only be able to operate on layouts that are of the form ``[("data", nbits)]``. It could create its endpoints itself, and take only the bit sizes (integers) as parameter.


This way you do not have to call ``raw_bits`` anywhere in the converter, and the addition of the ``chunk`` fields is easier.


added css file


These are (supposed to be) bought in by:

- js-babel
- js-babel-ui


i found a few free images from freeimages.com. Here are the urls, you guys can select
http://www.freeimages.com/photo/sushi-bar-1461764

http://www.freeimages.com/photo/sushi-conveyor-1549001

http://www.freeimages.com/photo/restaurant-1522573


From this commit, the minimum SDK has changed from 16 to 17.


That's perhaps now working with Meteor (??)


``chunks`` and ``insert_chunks`` are confusing names. What about ``valid_tokens`` and ``report_valid_tokens``?


sweet.


``valid_token_count`` even


https://github.com/philcockfield/babel/tree/master/js-babel

Let's remove the `js-babel` reference if they are all accounted for explicitly now.


A quick solution is to check if it zero and terminate. Is it supposed to be 0? I guess it can in some cases.


updated css file


What is the purpose of this change?


contrib/codeanalysis/runcppcheck.sh: line 9: ../cppcheck-1.66/cppcheck: No such file or directory


Here you have a space before the opening paren but no space after the closing paren. You need a space in both places.


Now it's the opposite. No space before the open paren but a space after the closing one. See if you can get an automatic formatter to enforce the common JavaScript conventions.


this line is not really needed. _data always contains correct keys and timestamps. The only problem with _data is ordering.


You should use `arg_.real_` and `arg.imaginary_` here


Space after `,`


updated css


I think your OctoberCMS is on 5.1 right now, isn't it? I have the latest code there, but I can't seem to orderBy in union for the life of me.


@sellnat77 @jtjr @Ronald-N
Needs review.

I think it probably working as my ` Log.d("Location", "Latitude: "+userLoc.getLatitude() +"Longitude: "+ userLoc.getLongitude());` is returning a good value for my device.

**Few things I was unsure about**:
1. onLocationChanged() - Did I do this part right?  It has been shaved down a bit from your original implemintation in LocationActivity.
2. If the GPS times out (line 246) do I need to kill the async task? (line 389)
3. Did I update AmazonDynamoDBClient correctly (line 422)


I think `stopRunner(true)` should be called everywhere except on restart, no?


Oh, good points! Thanks for clarifying that.


I encounter some problem here.

#define VA_ARGS(...) , ##__VA_ARGS__
#define INVOKE_VIRTUAL_FUNC(obj, member, ...) (obj).vt->member(&(obj)VA_ARGS(__VA_ARGS__))

INVOKE_VIRTUAL_FUNC((*list)->element, release);
is expanded correctly, I checked it with `gcc -E ...`
But compiler keep complaining...

Other invocation with non-zero arguments is compiled correctly


develop (lefticus) - i386-Windows-10-VisualStudio-14: OK (391 of 391 tests passed, 0 test warnings)

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


Kill this; it's not redundant. One can be part of something without being integral to it.


Kill this, somewhere between idiom and technical language ("We get catalogues of the ship's component parts.")


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


Kill this, not redundant. A pass need not be free.


Kill "penetrate into". If something has an outer barrier, then it's possible to penetrate into something without penetrating it.


Kill this, it's idiomatic.


Kill this, there are enough uses of period that "time period" isn't so bad.


리뷰확인했씁니다~1


Kill this, it's an idiom.


Kill this, it's not redundant. One can have a slice of pizza, a whole pizza pie, &c.


Add the singular.


Kill this, one can plunge into something.


Split this into two lines because the usages are so different.


Kill this; we already have a rule that strikes out "very".


Kill this, you can't always replace "present time" with "present".


I am able to write a marco to deal with this. But function with zero argument has some problem.

This works well but it a gcc extension
 #define INVOKE_VIRTUAL_FUNC2(obj, member, ...) (obj).vt->member(&(obj), ##__VA_ARGS__)
is there any other solution?


This is technically redundant but "under false pretenses" is so idiomatic that I wouldn't even understand it if written any other way.


https://youtu.be/My02F-4yY1Y

My homework #2....I got adding events to work.


you were supposed to write your name at line number 10 pushing down other names.
without deleting this PR, fix it


@aleph7 unfortunately it can't in it's present state because of the biasDiff on line 47. But we could split this up into two functions if you wanted?


Kill this, it false alarms on sentences such as "I never saw a protest against that.".


it's missing m_accel and m_steer value...
In my log it keep saying [warn   ] ControllerEventProtocol: The data seems corrupted. Remains 2

Actually i think we should not need to send m_accel and m_steer value anyway, as they are computed by serialized_1


you were supposed to write your name at line number 10 pushing down other names not removing them (`This will effectively push all other names downward.`)
without deleting this PR, fix it


This is good, but we should split this off into a different issue, because there are several usages with different ways to fix them. Compare "His hips protrude out.", where the above correction is good, vs. "The pencils protrude out of their holder", where that fix won't work.


develop (lefticus) - Win64-Windows-10-VisualStudio-14: OK (391 of 391 tests passed, 0 test warnings)

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


Hm, this substitution won't work perfectly. "They stood in close proximity" vs. "He stood in close proximity to the wall". I think I'd kill it.


That was all you man!


fb不用急着修
见光死系列


We need to think about extensibility of this stuff.
What do u think about converting this enum to class and reusing our skills with some additional info (periodical casts (5 casts per minute, e.g.), etc.) for this purpose?


这个地方的number 拼写没有错误哦！！


These changes were included in https://github.com/cfinke/Typo.js/commit/569dbe70d59283b3ffe974e25caa47ad60234845


Final presentation for our team assignment 2 - Technical Presentation


Pretty sure this was broken before (assuming other Makefile.nt's have a good reason to avoid -MM).


Name is a misnomer.  It's about forcing people to use cast items.  Thought I'd done more.  I think I thought to do more and didn't.


do you want to have all resource routes defined here? if not you should specify which ones you want


is it working? if it is I can add it to the buildbot


is it working? if so I can add it to the buildbot


1. We should be able to define map configuration on the server-side (size, texture pack, cells with concrete texture from the pack).
2. Also cell may affect on the hero (cell may be cursed by other heroes or terrain type may be LAVA, so we can pass through cell with dot effect). => TerrainType should be converted to object also


111247952_Remove_QUUID (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2327 of 2398 tests passed, 0 test warnings)

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


We already check at the parser level that the number of identifiers and expressions is the same (see `Parser.Decl.varSpec`). If we keep this code, we need a special case for when there are no expressions, which is allowed (e.g. `var x, y int;`)


Normally the parser shouldn't generate invalid combinations.


develop (lefticus) - Win64-Windows-10-VisualStudio-14-Debug: OK (391 of 391 tests passed, 0 test warnings)

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


What do u think about:
```java
abstract class Buff {}
abstract class NegativeBuff extends Buff {}
abstract class PositiveBuff extends Buff {}
```
We can use this in order to dispel negative buffs or other logic in future.


-*.js  diff=astextplain
-*.html  diff=astextplain


*.jpg binary


Actually that is not quite correct. You should have hit the Merge Pull Request button at #1.


I've worked around this in another way. Please take a look at my updated PR.


111247952_Remove_QUUID (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Ah, looks like this was fixed in #598.


I'd added a macro specifically for these sketchy usages... VAL_DECIMAL_BITS()

https://github.com/metaeducation/ren-c/blob/3b8a4820f709dd78ca81dd1a0b25774100971678/src/include/sys-value.h#L938


* "Every branch returns a value": it's a bit more than that; a function that returns a value must end in a [terminating statement](https://golang.org/ref/spec#Terminating_statements). This check is currently implemented in the weeder (`Weeder.weedFunDecl`).
* "No statement follows a return statement": we could check for that if we'd like, but [golang doesn't](https://play.golang.org/p/tGaJfgAdUW).


Heh, but it doesn't make much of a difference does it?


This will cause divide by zero errors. I feel it is reasonable to have intensity = 1 at radius 1, the closest any other node will get to the TNT. So i recommend 'dist' being limited to a minimum of 1.


Skill with AoE effect should be casted at point/cell (we need think about this feature too (for example skill may affect only part of the cell)). So there should be target and then area of effect.
```java
class AoeSkill extends TargetSkill {}
```


Should have been 'Add package file'... Oh well.


'form' 'centrer'
'block' -> 'nodes'.


Hi
kuck Dir doch mal bitte generic.make und echoLuaRev.sh in den Branchen master und webui an und Du wirst sehen, die sind alle unterschiedlich und passen nicht zusammen. Bitte suche die Variante aus, die Dir am besten gefällt, korrigiere die anderen und check dann alle ein.


'prefix'


I will have a look at it tomorrow morning.


If anyone would ever look at this commit, I apologize, meant 'model' instead of 'module'.


囧先更改


'center' or 'centre'


Hi! Please Try this const:
int rHist = (hValue + cmhValue + fmhValue) / **16384**;


'from'
'center' or 'centre'
'... within which players and mobs can be damaged'


This is a more refined version of prior commits, hence why it's pushed to master. @xhaggi I have not yet touched classic home, but that's just a matter of adding a couple of new vars for the artwork and matching the above changes.

Issues included with this commit:

- Focus problems on some areas (easily fixed)
- Switching between library content, plugin://foo requires changes to home and most likely startup in order for the images and labels to change
- The actual content of the widgets needs further consideration
- Obvious cleanup required

This is far lighter on low powered devices. Testing on a RPI I've noticed much faster widget propagation.


That's a good idea, but it never actually worked that way. It just wrapped the value it had in a promise - it never actually waited for anything. So I figured why bother.


maybe now you can add a test to check that an url with an anchor is actually not redirected?


Что-то у Вас происходит непонятное с кодировками файлов.


Why do you need that?


添加样式


Amazing. Included your changes in https://github.com/cfinke/Typo.js/commit/2b86aa07378530c0da794afdeaf49f9c9899a48d


5 lines above, spaces around '=' and '..'


У меня в итоге всё выглядит как-то так http://take.ms/86QIy


2 lines above, spaces around '=' and '*'


1 less indent


spaces around '..'


리뷰했습니다~


手慢无系列


4 lines above spaces around '..'


Oh, I think I ported bad fix :D


spaces around '='


:D nah i choose to blame you, never happens.


#winning :100:


browserHistory is importing in top of the file. so it will be undefined in test that need it.


Yes. Added it locally. Will push it soon


好的 正在提交。你看一下。一直忘记了。


[INTERNAL] initial commit


Ok


Spaces around '='


line too long


limit dist to a minimum of 1 to avoid divide by zero


How common are repeat calls to suggest() for the same word? My gut says not common, so I wonder if it's worth the memory to keep a record of all suggestions.

This also introduces a bug where if suggest() is called first with limit=1, all subsequent calls to suggest() for the same word will only return 1 suggestion even if the limit is increased. That's probably not common either though.


should be after 'write to map'


Included in https://github.com/cfinke/Typo.js/commit/c205a1f0c13dad6ee36f196d953900c2c1bba52c, although I'm not seeing big performance improvements from it myself.


What's the problem here ?

```elisp
(macroexpand
 (lambda (elm)
  (cl-typecase elm
    (stringp elm)
    (numberp (number-to-string elm))
    (symbolp (symbol-name elm)))))
(closure (test/tick t) (elm) (cond ((stringp elm) elm) ((numberp elm) (number-to-string elm)) ((symbolp elm) (symbol-name elm))))

(macroexpand
 (lambda (elm)
  (cl-typecase elm
    (string elm)
    (number (number-to-string elm))
    (symbol (symbol-name elm)))))
(closure (test/tick t) (elm) (cond ((stringp elm) elm) ((numberp elm) (number-to-string elm)) ((symbolp elm) (symbol-name elm))))
```


Looks great!


Yes you can write `string` instead of `stringp` but AFAIK both are allowed or did I miss something ?


111247952_Remove_QUUID (lefticus) - i386-Windows-7-VisualStudio-12: OK (2359 of 2398 tests passed, 0 test warnings)

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


trying to keep all db access in controller. suggestion from ian.


This one is convenient and I think belongs to be in the README though I see you haven't had much time to keep documentation up to date. The amount of undocumented features is incredible...


Nice. But you can use `be_success` matcher of rspec in here. :)


:P


Yup, that line should no longer be necessary!


um? line 97 in MainClass?


ae


Adjusted all the above comments in this file.


Probably we can remove the ais.pl emails, and need to include support@ravendb.net


Continue it inline so validation is no more necessary.


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


Huh, works for me. :-/


Załatwione, już powinno być ok. Błąd w bibliotece jest, autor pisze, żeby na razie używać innej strategii niż zalecana :)


Aw dont know about it, thank you. I have removed this code (it is auto-generated). Will give "be_success" a try @nvh0412


If you added `rspec-rails` to this gemfile, you don't need to add `rspec` gem then. :).


Developing for less than a full cycle so didn't see nextmoondata until now (even though it's in the USNO docs :)


OK @nvh0412 I will remove it


I've thrown all these fixes into a separate commit to make sure I've caught all the mistakes after each patch.


I'm working on a ghost theme, so I was looking at yours because I remembered that you used ghost.
This is sad / hilarious. To hell with these turds.


Really nice!! :100: . But i think you shouldn't create responed_to example for each attribute in here, since you had created `validate_presence_of ` example for each attribute. :+1:


啦啦啦


Aw got it, will remove them soon :+1:


You shouldn't commit this file, if you don't want your colleagues wonder about it.


Leave a comment


Does disabling the banner brings any value? Haven't seen someone doing it, that's why I'm curious and asking.


Using name attribute in theses jpa annotations is not necessary due you're providing the default names.


I don't like that funny strings during application startup...


Use long instead of int


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

```


Same as in container class


I have not finished it yet T.T, will spend time to write all missing time today


done


What's the reason for using JAX-RS and not typical Spring annotations as everyone is using these in spring boot?


moved


in the style fix patch.


Why are you using XML annotation here?


And please don't use anykind of DTOs. I'd suggest we implement a sufficient domain model for this where DTOs are not necessary.


If you'd make fetching of features eagerly in Container this DTO is not necessary anymore. Is there a reason for this?


Why is it when loading minecraft LXDE quits and i get this
`*** Error in `/usr/bin/Xorg': double free or corruption (fasttop): 0x00007f8b10f688d0 ***
> /usr/bin/xinit: connection to X server lost
> Hangup
> Hangup
> Running exit commands...
> Unmounting /mnt/stateful_partition/crouton/chroots/trusty...
> Sending SIGTERM to processes under /mnt/stateful_partition/crouton/chroots/trusty...`


I will create 100 containers per day every day for 100 years (I think it's quite long time, probably application won't live for that time). And I will create 100 * 100 * 365 =
```
    3 650 000 Containers.
MAX_INT is
2 147 483 647
```


Reasons to use long?


:grin:


staticの有無で、その文字列が置かれるメモリ領域が異なるはず。
教科書でも同じような書き方していた記憶が。。。


Nice!


Great that you pushed a first draft of the new API. I added some line notes for specific points I want to discuss with you on this.

To sum up:
- Production critical: Database migrations are missing.
- Unusual development style: JAX-RS in Spring web app. Possible but totally uncommon imho.
- DTO where we don't need them.
- Anykind of tests are missing.
- Anykind of documentation is missing. Even rudimentary javadoc comments.

So, after all I'm not ok with merging this pull request.


Capitalization is now taken into account as of https://github.com/cfinke/Typo.js/commit/ff87aecf1ace3b5ee2dee3064090207deead1f9e (only really works for all-caps or words with the first letter capitalized).


This will **break** things. Non-arrow function was there for a reason. In this child_process callback, `this` refers to a child process, not the Manager class. Hence, turning it into an arrow function will break workers


Not sure how critical it is, but some time ago I measured performance of [Object vs Set/Map](https://github.com/anvaka/set-vs-object). Turns out Set/Map are 2x times faster.


Won't agree. There are several rules that are better to use with naming tables and entities:
- explicit is better than implicit.
- all that is used somewhere must be defined somewhere.


Ok, I'm fine with this. We should do database migrations anyway and there we define table and columns by our own (by explicitly naming them).


What happens when the pan gesture is on the canvas?


Well, it brings the advantage that everyone knows she's running a boot app when starting it. Which furthermore helps in finding health check resources and other usefull resources (convention over configuration is used everywhere in spring boot universe)


JAX-RS 2.0 comply with JSR 339 and actually was discussed by many people and is a **part of java specs**.
Spring annotations - is a part of spring.


If we're running this on Postgresql primary keys are typically mapped to BIGINT which is 8 byte big. Therefore Long is more safe in java.


:c


Did you know about `make maintainer-clean`?


Ещё это взять в тэг центр


commented on LICENSE in 23e2839


아래 내용은 다음번 pr시 수정하시면 될것 같네요
* log의 문구가 mx, my가 없다는 문구가 되어야 할듯하네요. already lastest data는 위의 567 line의 상태죠.


@mgechev Curious the intent of this change here?

It appears to cause problems when extending config with other settings like this:
```
this.SYSTEM_CONFIG['map'] = {
   '@ngrx/store': `${this.APP_BASE}node_modules/@ngrx/store/dist/index`
};
```

Build fails with this `packageConfigPaths` change in there. If I remove that line, everything works again.
Likely because `@ngrx/store` install does not include `package.json` in the root of `@ngrx`?... rather it's inside the `@ngrx/store` subdirectory.

Should the line be:
```
packageConfigPaths: [`${this.APP_BASE}node_modules/**/package.json`]
```

to solve?


Well, but we are using spring here and I don't think we'll change the underlying web container implementation in near future. By using spring annotations we would probably make it easier for other maintainer in future to understand the code faster. Which is great value for open source development.


comment


Welcome, and thank you!


그 부분이 already일 경우만 해당해서 그대로 적어놓았습니다. 결과적으론 그래도 코드상으론 혼란이 있을 수 있으니 다음 pr시 수정하도록 하겠습니다~ 감사합니다~


so that the newly appended contents from two fields in above steps can be accessed as a single field, and can provide a way to change field parameters like name, datasize, currentvalue, etc in further steps.


With this commit zk (1.4.3.3) units selection frame and bars often renders black.
I see this spec when i spec a 3v3 CircuitAI test game on Nuclear_Winter_v1.
Perhaps this change exposes a zk bug though.


Посмотри тут


If ch is LF then the new code will not trigger as it's entirely in the "if (ch == CR)" scope. If it's just CR it will print CRLF in 210 and the following character in 214. What am I missing?


The resultant current value would be 'None'


Here is what I think:
1. Database migration is missing - there is no existing database now. And event if it were - migration script is a separate part of application.
2. Described reasons several pages before
3. There is an approach to development - separate model (business model) and view. I thought that everyone do it. If you think that it will be better - I'll remove 'Dto' ending and add 'View' ending. But actually it's dto (I remove 'id' field, after all)
4. Tests on what? There is everything missing.
5. I've created documentation in swagger file. Don't see any reasons for retyping it in many places.


test


args defaults to sys.argv[1:]. it's the command-line input, given as a list of strings (split by white-spaces).
It's used is for directing the method to work on something other than CLI input (for example, in testing).


Assignment 3


plugins/.../Makefile shouldn't be included: autoconf will generate it from the configure.ac and with correctly expanded variables.

You have to edit plugins/Makefile.am for the module to be able to compile.

Good start though!


now you simply need to make it impossible to draw the same card twice. presumably a function of some sort... or an angular 'listener', or a combo, or blahblahblah


Ruby 1.8 is not a thing anymore. See https://www.ruby-lang.org/en/news/2013/06/30/we-retire-1-8-7/.


Wow, it's comprehensive support!!
It showcases the power of composed annotation.
Great, and love the SpEL support as well!!


In the `clearFlash` method you're calling `.empty()` on the `flashContainer` object. As I read this line, that could be the forst object in the html body, which might not be the flash container? This is perhaps a little to flexible?


we need this


@aofall 不算修复，只是昨天比较懒，几个域名直接用一个 IP 了。今早测试 FB， 发现视频放不。
于是把上一次能用的全部搬回来了。


Hi, glad you found it.


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


ah, sneaky


probably this must be causing it


ну в принципе мы примерно это и сделаем. Можешь еще допилить класс Route. Видишь, у тебя там в routeMap все элементы однотипные, а так будет new Route('/bla-bla', 'index', 'index')


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 286](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=840&buildTypeId=XbimEssentials_XbimEssentials) is now running


probably this must be causing it and here its been used


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 286](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=840&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Compilation error: Xbim.IO\Xbim.IO.csproj (new) Build time: 00:01:35


@andimarek: Perhaps I should have made the comment more clear, but this code is actually important. [java.math.BigInteger#intValue()](https://docs.oracle.com/javase/6/docs/api/java/math/BigInteger.html#intValue()) will not throw if the value is out of bounds and will silently return the lowest 32 bits. So although the return value was unused, this PR relied on [java.lang.Integer#parseInt(java.lang.String)](https://docs.oracle.com/javase/6/docs/api/java/lang/Integer.html#parseInt(java.lang.String)) throwing when the value overflows.

It is cleaner to use [java.math.BigInteger#intValueExact()](https://docs.oracle.com/javase/8/docs/api/java/math/BigInteger.html#intValueExact--), but this was only added in Java 8.


Sure, but when we start using AeroGear code, we didn't insist on feature branches, therefore it is not a simple task.
I'll try do describe some of the changes we had, please replay which are relevant and ill give it a try.

Technical issues,
1) Maven changes to support Wildfly 8.2.1
2) Maven eclipse integration, include js resources into output war.
3) unifiedpush-service - use arquillian/wildfly instead of openejb
4) unifiedpush-jaxrs - additional tests using arquillian

Feature changes:
1) Documents API - Store & Forward documents.
2) Register installation in disabled mode
3) Enable devices based on verification process (SMS/Email)
4) Link aliases to application.
5) Move installation to new variant/application based on alias relation.

We can also schedule a GTM/Skype session for further clarifications.

Regards


The comment was fine.  just decided to to do it in a extra commit: https://github.com/andimarek/graphql-java/commit/aabed3ae8f08550740f58b1f83c727ed24630437


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 287](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=841&buildTypeId=XbimEssentials_XbimEssentials) is now running


Ironically, looks like it was noted on @ngrx/store just an hour ago to fix the npm distribution:
https://github.com/ngrx/store/issues/65

So maybe not change a thing and once the next release of store is out, all should be good.


Yes, I see I have spoken too soon. Thanks! :+1:


@Nikita-Guskov
ok I try your value but i interested how you get this value.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 287](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=841&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:59


Change


ой-ой =)


Я убрал правило валидации, переменная e должна приниматься


так на самом деле лучше


What do you mean?
On 13 Mar 2016 05:01, "Aaron Peromsik" <notifications@github.com> wrote:

> Search-replace much? Intentional?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openintents/safe/commit/73fc5d69f1c779516c228cc3066c3e4157faa8a6#commitcomment-16665715>
> .
>


Thinking abotu this, does this mean you didn't alter the syntax of `\DeclareNAmeFormat` for BibTeX? That will break almost all styles outside the core, as threre's no way for them to 'know' which backend is in use (nor really should they need to).


Looks like good stuff here :)


fix #16


this fixes issue #22


Kill it, also an idiom.


@jystic Sorry to ask you such a horrible question, you know what might be causing this build failure?

https://travis-ci.org/charleso/webship/builds/115637140

```
Building ambiata-mafia-0.0.1...
Installed ambiata-mafia-0.0.1
Writing a default package environment file to
/home/travis/build/charleso/webship/webship-decision/cabal.sandbox.config
Creating a new sandbox at
/home/travis/build/charleso/webship/webship-decision/.cabal-sandbox/7.8.4
Failed to run 'cabal sdist' for source package: ../ghc-7.8.4/compiler
```


When I imagine of merging two fields, I think there should be something in common for combinations, domains could be one thing, but may be it's not that essential, I don't know. After combination, the field parameters should change like size of the field, etc. for example - if the 1st field length is (0,16)bytes and 2nd field length (0,8)bytes -> then new combined length is something like (0,24)bytes. Or if both the fields are constant fields then the new field after merging is also a constant field (say if first field is a constant field with value '\xff' and 2nd with '\x01', then new resultant field has value '\xff\x01'). If any one of the fields or both the fields are variable fields then the resultant field is not a constant field (say '/xff' and None -> None).
How do you mean when you say creating a new raw domain for merged field ? Say we have a data domain and an Agg domain, then how the final result should look like after merging ?


@tomjaguarpaw it's quite simple, if you try to compile with GHC 7.6 you get this:

```
[ 3 of 26] Compiling Data.Time.Clock.Scale ( lib/Data/Time/Clock/Scale.hs, dist/build/Data/Time/Clock/Scale.o )

lib/Data/Time/Clock/Scale.hs:108:39-45:
    Not in scope: data constructor `MkFixed'

lib/Data/Time/Clock/Scale.hs:112:36-42:
    Not in scope: data constructor `MkFixed'
```

and if you look at http://hackage.haskell.org/package/base/docs/Data-Fixed.html you'll notice that `MkFixed` was exposed only since base-4.7 (i.e. GHC 7.8)


111247952_Remove_QUUID (lefticus) - Win64-Windows-7-VisualStudio-12: OK (2360 of 2398 tests passed, 0 test warnings)

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


Great! Nice learning tutorial by git-it


test


In keygen.c file, computation time doesn't print in terminal. so how to print ?


@diliger you should not commit the `.vs` folder and change the `UserAdmin.csproj` file. Could you please fix those and rise another pull request?


Hello, I am using development branch of videojs (rewritten in ES6) and I would like to know why this was reverted

I uncommented this line and pass the tests successfully on my local clone. So why not removing this comment and removing `videojs` from global scope.

Thanks for any explanation.


congress


heh, this is what I had first, but it's not actually the right solution. the summary is here:
https://fedoraproject.org/wiki/Packaging:Systemd#DBus_activation

TLDR is: we have an Alias= in the systemd unit file that can be started with systemctl enable/disable ratbagd.service. that puts that dbus-org... file in place. the dbus bits activate that dbus-org... service. As a result, we have a dbus-activated service that can be enabled and disabled by the administrator.

The dbus activation should work once you run systemctl enable ratbagd.service


Kill this, it false alarms on sentences like "Barclays made an incredibly useful chart showing all the possible policies different UK governments could pursue after the election.", which are common.


@eska014
Ok, I buy that. I'll make a PR against the instancing documentation.


+ faster travis builds


Why'd you have to change this? I hated it like that lol.


Kill this, it's an idiom.


改行


This is good if it means lift, but false alarms in a sentence like "John hosts benefits that raise up to a million dollars for cancer research." Let's split this off into its own issue.


Can you use a CListWrapper ?


This is a technical term, not a false alarm.


Thanks for (partial) correction but tier and color are still wrong.


Hi Petter,

does it make sens to move the patch into the linux-raspberrypi.inc so each kernel version can make use of it without declaring it for each version? In addition if the patch is not used in a kernel recipe (such as the newest version 4.4) the build breaks because the system tries to deploy pitft22-overlay.dtb which is not present in case of not applying the patch.


Ano, tohle přesahovalo 120 znaků;)


Kill this. It's slightly redundant, but all over The New Yorker.


what's this?


Eh, kill this, also slightly redundant, but sounds nice and is used in reputable and highly edited sources.


unless there's a good reason to, I would leave imports at the top of the file


hi,

if you need help, just create an issue. I just stumbled over this fork.

In the preferences of atom you can set the options in the setting 'default options'. But i prefer creating a file named .htmltidy in the working directory. The package prefers these options over the setting.

have a nice day,
andreas


The changes above are, of course, not the cause of this failed build.

It looks as if this is due to new default index behavior with pandas==0.18.

It seems to only be affecting `urbansim/models/transition.py` (based on Travis and on tests locally). For example, with `test_remove_rows_all()` in `urbansim/models/tests/test_transition.py`:

```
AssertionError: Index are different

Index classes are different
[left]:  Index([], dtype='object')
[right]: Int64Index([], dtype='int64')
```

All tests pass with pandas==0.17. (I've only looked at the master branch.)

I'm looking for a fix.


it's probably there to make sure everything that isn't AA doesn't fire on air units.


Kill this — the first is not redundant (one can set a record for the month or year or school or some other subset), and the latter will false alarm when "record" refers to an album.


Okay. I'll try to do a release today.


doesnt work!!!!! you crazy


I think we can combine these two conditions in a single line - &&


@schulz3000 Thanks a lot for this contribution. Please also update the upgrade.sql file (add locales)


Wouldn't it have been better to download MkDocs from PyPI like the others are below?


this should be `datetime.utcnow()`.

`timedelta` has a `minutes` attr, but even if seconds was minutes this math wouldn't be correct


测试更新


@jystic Hmm, apologies. It seems to have corrected itself. Weird.

https://travis-ci.org/charleso/webship/builds/115637815


Норм


The issue was about the relative path. `resources/redirect.py?location=image.gif` redirects to `resources/image.gif`, so I have to remove the `resources/` before `image.gif`.


Hi,

Thanks for your interest!. I have to be honest that it wasn't really intended for public consumption, -- I mostly just added it to Github as a personal backup, so I can't promise that the code is all that great, or that it works anywhere other than my laptop, so be warned!

As to the licence, I'm not entirely sure what it should be covered by. It's mostly a C++ translation of the Java code from the tutorials, so I suppose from a legal point of view it's probably under whatever licence that uses. (Of course, I'm not a legal expert in any way shape or form so don't take my word on that, I'm pretty much just guessing.)

For the parts that are purely mine and don't have any corresponding Java code (like gl_handles.hpp) then you can consider them to be under the MIT licence. Use them for whatever you like.


When I saw that commit title I thought you'd disabled damaging players with spells :O


I use an esp8266 rather than an arduino seems to go a lot faster. The only other way is to make the code more efficient but not sure how.

Sent from my iPhone

> On 12 Mar 2016, at 23:37, ptaudio <notifications@github.com> wrote:
>
> Hi,i love this library. I only experience one problem. When i use 8 displays. The scrolling gets really sluggish. What can i do to solve this ? Thx
>
> —
> Reply to this email directly or view it on GitHub.
>


lol no :P terrain dmg


why do we need a separate list of column names? why don't just use columns?


Kill this, for the same reason that it false alarms when "record" refers to an album.


Good Job


check if index < columns.size and throw proper error message to avoid ArrayIndexOutOfBound exception


Nice work


Qie


I'm currently testing #3143 and this makes my build fail with `done is not defined` -> @deepu105 can you have a look?


Kill this, it's so common in the given form that I don't recognize it without the word "recruit".


A more detailed resume is available via the link to my Linkedin account, which is provided in the designated input field within the application, as requested.


Could the two functions that use `MkFixed` not just be disabled by CPP?


Import = আমদানি

এই ছাড়া তো কিছু লেখার মত নাই :confused:


This is good, but isn't really a redundancy. Can we put this in `garner.preferred_forms`?


could you also use authWithCustomToken?


You wouldn't want that, as that would lead to a conditional API...

you'd rather have to find a (less performant) fallback implementation for base<4.7 to switch to via CPP. It's certainly possible and not much work, but somebody needs to come up with a patch... :-)


i have add FIREBASE_BASE_URL as variable. please use the variable


Not sure how I feel about this. It's not redundant, per se, because something can be "only somewhat relevant". But there are certainly many times when it's used unnecessarily as an intensifier. Let's split this off into a new issue.


That makes sense; thanks!


- please use double quotes
- please reorder some things:
 - move export statement to the end
 - move _makeRequest above _getUserMail


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


even though the markup is meant to redirect the user to bsanchezportfolio.com, it doesn't seem to be working.


hahaa


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


@mauroartizzu Non sono molto convinto su `parole chiave` perchè dovrei metterlo singolare?


When I try now, I can still compile on i686 even with the following patch. Any reason not to push that?

```diff
diff --git a/pkgs/applications/networking/browsers/firefox/default.nix b/pkgs/applications/networking/browsers/firefox/default.nix
index 0ef3bc7..e384196 100644
--- a/pkgs/applications/networking/browsers/firefox/default.nix
+++ b/pkgs/applications/networking/browsers/firefox/default.nix
@@ -73,7 +73,7 @@ common = { pname, version, sha256 }: stdenv.mkDerivation rec {
     ++ lib.optional enableGTK3 "--enable-default-toolkit=cairo-gtk3"
     ++ (if debugBuild then [ "--enable-debug" "--enable-profiling" ]
                       else [ "--disable-debug" "--enable-release"
-                             "--enable-optimize${lib.optionalString (stdenv.system == "i686-linux") "=-O1"}"
+                             "--enable-optimize"
                              "--enable-strip" ])
     ++ lib.optional enableOfficialBranding "--enable-official-branding";
```


yeah, was late. I would like to get smooth scrolling like on main page.


This commit is missing in develop.


why don't you do it like:

```python
def path_or_url(setting):
    if str(setting).startswith("http"):
        return str(setting)
    return path(setting)
```


@timmartin19 Just something I've been playing with ;)


@sils1297 We've discussed this on gitter.
All these ideas like `startswith('http')`, etc are just solutions to immediate problems. Not a good method ...

Better to have a `url()` in coalib.settings.Setting which all other bears can use too.


Kill this, it's not redundant. You do something. You repeat it. And then you can repeat it, again.


Well, then we have a larger problem with many other places in the code (that were already there before my change).

So far I have heard nobody complaining about that issue with the rest of OSPOS. If a problem we do an overhaul.


Kill this, it's not redundant. A reprieve can last for a long time, or forever.


Idem ;)


sure, then use try url(...) except path(...)

Sincerely,

Lasse Schuirmann

contact@viperdev.io
http://viperdev.io/

2016-03-13 10:10 GMT+01:00 AbdealiJK <notifications@github.com>:

> @sils1297 <https://github.com/sils1297> We've discussed this on gitter.
> All these ideas like startswith('http'), etc are just solutions to
> immediate problems. Not a good method ...
>
> Better to have a url() in coalib.settings.Setting which all other bears
> can use too.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vivek425ster/coala-bears/commit/85470eaaa48ee80092dcbb556e7b53d8349d27e6#commitcomment-16666628>
> .
>


Ooh, this is tricky. This is only redundant if one is not contrasting with requirements that were impose needlessly. "John argued that they were necessary requirements for new students; without the requirements, students may never learn the skills they would need in their future line of work."


Kill this, it's idiomatic.


I was under the impression that use of CPP in Haskell often led to conditional APIs, but perhaps I was wrong.  Anyway, in the absence of an alternative implementation a conditional API sounds much better than not being available at all on GHC 7.6.


Not bad for locally tuned values. I guess you used pretty short TC and pretty low number of games.


@gre sorry, yes the docs aren't very clear. It's a resize (measured in display points, relative to the scale factor, not pixels).


Kill this, also idiomatic.


Hi, what difference makes this?


Kill this, Garner says it's preferred.


why have you restored this? Nobody writes anymore E-Mail nowadays, email is is widely accepted: http://www.oxforddictionaries.com/definition/english/email


Kill this; there are times when the two have different meanings. "In the morning, the townspeople will rise" vs. "In the morning, the townspeople with rise up against the King".


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

```


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


Kill this, idiomatic.


we are duplicating this because it's already in config. Maybe we should put all of this in common as they are standard sentences that can be reused in other context.


Hi, you're twice right, and I was a bit tired.
3 more things:
- this 'init' change can also concerns stable branches .
- class_index could be declared as an enum to enforce it a bit; ( not a bulletproof idea, i know)
- as it's not an external/outside data (from user inputs), just use an assertion to check the range.


ok i'll fix it for the point unit


OK< maybe this was actually not change in the language csv file. I'll fix it.


Is giving up on the Lint checks really what you intended ;-) ? I know that there's millions of those errors, but I am more than happy to spend some time ...


Remove "seek out", which is idiomatic.


Kill this. They have different meanings.


@joergoster: I think this is an opportunity for SPSA2:
https://github.com/locutus2/Stockfish/compare/a273b6e...664b7b0
if you want to try.


s/CRYPTO_F_GET_AND_LOCK/CRYPTO_F_CRYPTO_GET_EX_NEW_INDEX
No ?


Hi Roberto,

Looks good and I just committed this to the codebase.

Here are several more files to convert(if you want):
interface/patient_file/encounter/delete_form.php
interface/patient_file/deleter.php
interface/patient_file/problem_encounter.php
interface/reports/referrals_report.php
interface/usergroup/usergroup_admin.php

-brady


I thought about curse and bless as a substitute of negative and positive, but your version sounds much clear. Agreed


This function is not required and sometime returns some invalid results.


Hm, this is redundant, but "slow" isn't always the right fix. "He walked at a slow speed." should become "He walked slowly." Let's create a new issue for now.


Excellent, I was having issues with my gooey installation. I had been copying out the functions I was using now I can go back to just calling plot_fits


I think this also needs its own issue: "Its small size bothered her."


Kill this, it's descriptive. Snow can be nonwhite if it's dirty or peed on.


Oh hmm


Those two function are interesting and quite flexible.


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


Kill this, it's not really redundant: "…they were gay, so therefore justified in their anger."


I agree. It's like target of fireball gets 1x damage and everuthing in area gets only half of it.


are these images loaded? I'm not sure the url there is pointing to the right place. I saw an error.


This is weird, kill it. It's used in stories to be vivid.


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


Kill this. This isn't redundant — it's what maybe half the U.S. calls soda.


Is there any possibility for `hr == S_OK && bcBuffer != nullptr` hence it leaks ?


probably we need to re-style since we don't use these fonts.


Kill this, space can be filled.


please run a bower update


It's definitely true that CPP easily leads to conditional APIs if used incautiously.

The problems with a conditional API arise once a package on Hackage starts using an API point that's not always available. Then if install-plans start bit-rotting, Hackage trustees are forced to edit in destructive upper/lower bounds which would cut of otherwise legitimate install-plans to avoid the provably incorrect ones (which the solver would otherwise run into). That's why I strongly advise against providing conditional APIs whose conditions  can't be expressed in `.cabal` files properly.

It's just too easy to inadvertently depend on features of a library and forget to properly declare that dependency in the `.cabal` file. It's already hard enough to properly specify direct dependencies, I have little hope that this gets better if you'd also require to specify indirect dependencies to account for conditional aspects of an exposed API. We don't even have a way to annotate the requirements for conditional APIs via `@since` yet in Haddock markup (which would give us a way to mechanically infer the required `.cabal` specifications).

anyway, in the case of `time` it's certainly possible to provide a fallback implementation for the affected functions without incurring the pain of a conditional API. The question is only whether @AshleyYakeley would be willing to merge such a patch, or if he wants to officially drop support for GHC < 7.8 (after all, soon the latest 3 major GHC releases will be GHC 8.0, 7.10 and 7.8).


The link gets tested in the money transfer test every time you execute submit gift codes


Kill this, it's idiomatic.


Kill this, it's an idiom.


Kill this, too many false alarms.


nice


I indeed changed it to *renforcez* in the commit.


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


I don't really like this approch. I agree it will make it works anyway but it also means we will never fix this issue as it never show up.


Kill this, it's not redundant — one could compare the lengths of summaries and say that one is a long summary, the other a brief summary.


Kill this, it's not redundant. Compare "talking to himself in his head"


像twitter facebook 这类社交网站我都不管做的....
一开始我连G+都不想加，后来发现有些东西还是得用到G+，就加了上去
主要还是被G+上的轮子恶心到了（虽然屏蔽热门动态就没了）


Looks OK to me, can't test though.


You store node? Not ID?


Kill this, it's not redundant. If you tear your shirt, it has a tear in it. If you tear it apart, it's in pieces.


Kill this, it's not redundant. You can cook teriyaki on a stove top.


It was altered on for biber - it takes one parameter instead of 6. A lot of styles conditionalise on the backend.


//v4.x search slow due to join and order_by #863


Looks good, and works for me. But the second region at the top (where the text disappears) shows the artifacts too. And, of, course, still after this commit :)


![netflix-full-house-fuller-xT9DPxPuTYumXxmfGo](https://camo.githubusercontent.com/1924e5bf19cb965502323de335bc8c0a12319fe8/68747470733a2f2f6d65646961332e67697068792e636f6d2f6d656469612f78543944507850755459756d58786d66476f2f67697068792e676966)

works like a charm :+1:


Save for later (see "he himself").


@DennisLfromGA

If you read closely you see that I answered your question in my first response, but thank you. I was able to figure it out.


I opened an issue for this. https://github.com/ros-perception/image_pipeline/issues/185


Kill this, it's an idiom.


Kill this, too close to banning a common use


Kill this, these aren't redundant.


The useQueryString requires special attention when dealing with client side routing, therefore it has been disabled by default. The filtersWizard default has been removed, since currently is not used.


Kill this, there are some uses where it works, "You learn how to do the tough stuff and bring people together at the same time." Maybe we could create a new issue for this.


As it happens, string concatenation can be generally faster than Array.join.


Kill this, close to a technical term.


Kill this — what if we are talking about past trends and current trends?


Kill this. A tube can be filled.


Kill this, it's a common term for the canned food product.


How about a constant?


I think we want this to be replaced with "truth"? "It's the undeniable truth."


As you expand full_args each time you use it. Can't you do it here?


I think the replacement should be "mistake". As in "It was an unintentional mistake."


Do you really mean to say `Put the code under the desired folder inside tests or tests`? I think you can remove `or tests`. Same issue at line 18.


Kill this, it's already covered in `wallace.uncomparables`.


Kill this, you could have many units, or a single unit.


Kill this, Garner says it's okay.


Kill this, it's an idiom.


Good


(Same as "he himself".)


Kill this, idiomatic.


There is also a "stringdot" option for learners classif.lssvm and regr.rvm. I assume they are also affected?


You don't handle user defined function def aliasing?


Ops, that already be fixed a few minutes ago. Therefore occur merge conflict.


1. And we will :-) texture will be just aliases like Forest/Water/Bridge etc. And client will choose texture according to this names.
2. Sure


Oops my bad correcting it now


更新OK？


@IIvec Search parameters shouldn't be tuned at very short tc, imho. So this seems to better be done in the framework.

This time I used 2.5"+0.025" which is 2 - 3 times slower than what I usually use to tune evaluation parameters. And I had quite some success in the past. ;-)
We must not forget that all these values have already been tuned to death. So I consider this to be a confirmation for SPSA2 to be working. Maybe I will try to fine-tune some more parameters.


Hm. "whether or not" is fine, but "as to whether" doesn't work. "They offer an early clue as to whether the Court will divide along familiar left-right grounds."


Ready


This CSS rule was causing the problem described here [https://github.com/RobertoPrevato/jQuery-KingTable/issues/1](https://github.com/RobertoPrevato/jQuery-KingTable/issues/1). Tested using an Apple iPad Mini 16Gb (MD531FD/A).


@mintsoft I almost *always* use `git commit --patch` (or a shortened version :wink:) so I can see *exactly* what goes into each commit (and approve/deny it); you come across it before? It is *soooo* useful!


I think the correct form should be "in which we live". Consider "The town in which we live in has many trees."


Kill this. "At first, Ulukaya dismissed the idea—why come all the way from Turkey to do what he could have done back home?"


@Rocky640 Yes, I used the latest version.


Kill this, it's idiomatic.


Ditto.


ack


Add `Permission Denied` exception to this? When you don't have the permission to run the file.


ack, though btw. an alternative would be to allow `==...dev0` in coala-bears requirementst.txt explicitly, that'd be cleaner on the long run wouldn't it?


@aofall 赞同。FB 基本用不上，TW 倒是能关注几个喜欢的歌手，还有一些游戏的开发进度。
想想封了这些网站也好，赚我们的钱还来坑我们、、、
不讨论这些了，还是看书做题实在。


Interesting. Let me try that out


TOTALLY READ EVERYTHING LOL YUP LOOKS GOOD TO ME


addon loading is still broken, at least for inputstream:
10:40:10 T:140453239039744   DEBUG: ADDON: Dll Initializing - Input Stream
10:40:10 T:140453239039744  NOTICE: ADDON: Loaded virtual child addon special://temp/inputstream.mpd-006.0
10:40:10 T:140453239039744   DEBUG: SECTION:LoadDLL(special://temp/inputstream.mpd-006.0)
10:40:10 T:140453239039744   DEBUG: Loading: /home/wuff/.kodi/temp/inputstream.mpd-006.0

1.) Addon Loader expects the addon in /usr/share/kodi/addons/inputstream.mpd/ instead on /usr/lib/kodi/inputstream.mpd. For the log snippet above, I've created a symlink, so it at least finds it.
2.) Kodi creates a _new copy_ of the addon in special://temp/ _each time_ its instantiated. Why?


Shortened*


Nice @Zizaco . I like the idea to implement mongolid on top of `"mongodb/mongodb"` so we don't have to worry about driver low level implementation.


I will try using it.


Nice catch ;)


@FranDias Is there a way to make the text display vertically? like this?
![20160313200115](https://cloud.githubusercontent.com/assets/16702441/13732324/7d622c9a-e956-11e5-975d-0aa4e9546ca5.png)


Boom!

```npm install express@3.0.3```

Thanks @theelectricwiz !

...moving on! :)


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


Hi,

> Adding a .gitignore

good point, will look into it.


John I revised the face image with better pictures and formats please use that one was emailed to you after sent this one late last night.thx


After I made the change I noticed that the app no longer recovers from errors. The HMR  throws the `cannot find module` error on one of my components in the project. I am not sure if this is a symptom me un-mounting the original app or not. I had to add the NoErrorsPlugin back to make it recover again. I just started exploring the javascript world a few months back so i am not experienced enough to tell if it is caused by the un-mounting. Just wanted to share my experience in case someone encounters the same error.
here is the stack trace `./grid-rnd/GridRnd.js` is in my path
```
[HMR] Cannot check for update (Full reload needed)
process-update.js?e13e:96 [HMR] Error: Cannot find module "./grid-rnd/GridRnd"
    at webpackMissingModule (eval at <anonymous> (http://localhost:3000/static/0.9f2fdf9b8b636cb82ef8.hot-update.js:6:2), <anonymous>:19:80)
    at Object.eval (eval at <anonymous> (http://localhost:3000/static/0.9f2fdf9b8b636cb82ef8.hot-update.js:6:2), <anonymous>:19:175)
    at eval (eval at <anonymous> (http://localhost:3000/static/0.9f2fdf9b8b636cb82ef8.hot-update.js:6:2), <anonymous>:120:30)
    at Object.webpackHotUpdate.195 (http://localhost:3000/static/0.9f2fdf9b8b636cb82ef8.hot-update.js:6:2)
    at __webpack_require__ (http://localhost:3000/static/bundle.js:535:30)
    at fn (http://localhost:3000/static/bundle.js:76:20)
    at render (eval at <anonymous> (http://localhost:3000/static/bundle.js:4433:2), <anonymous>:51:25)
    at refresh (eval at <anonymous> (http://localhost:3000/static/bundle.js:4433:2), <anonymous>:78:9)
    at eval (eval at <anonymous> (http://localhost:3000/static/bundle.js:4433:2), <anonymous>:85:9)
    at Object.hotApply [as apply] (http://localhost:3000/static/bundle.js:475:16)
```


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


I already wrote a patch for this, just waiting for ocfweb to build.

I also found out you can't comment on commits over email.


Because, when the document is created for an iframe, the content type is unknown until the "content-type" header is received but the document still have to be created on dom insertion.


Fixed issue #3 bug


Use the TotalOrderPartitioner class to globally sort the output of a MapReduce job.


works for me!


Use 5.2.0 versioning instead, you may run into problems.


Oh, nice!


TeamCity Device Info :: Mock tests [Build 4](http://82.35.147.206:8111/viewLog.html?buildId=253&buildTypeId=DI_MockTests) is now running


Don't use that method for the commands, it created problems for me when I tried doing stuff with $args and code format was like that


import java.util.InputMismatchException;
import java.util.Scanner;

public class DivideByZeroWithExceptionHandling
{

    public static int quotient( int numerator, int denominator ) throws ArithmeticException
    {
        return numerator / denominator;
    }

     public static void main(String[] args )
     {
		 Scanner scanner = new Scanner(System.in);
		 boolean continueLoop = true;

		 do
		 {
			 try
			 {
				 System.out.print("Please enter an integer numerator");
				 int numerator = scanner.nextInt();
				 System.out.print("Please enter an integer denomerator: ");
				 int denomerator = scanner.nextInt();

				 int result = quotient(numerator, denomerator );
				 System.out.printf( "\nResult: %d / %d = %d\n", numerator, denomerator, result);
				 continueLoop = false;
			}
			catch (InputMismatchException inputMismatchException )
			{
			   System.err.printf("\nException %s\n", inputMismatchException);
			   scanner.nextLine();
			   System.out.println("you must enter integers. Please try again. \n" );

			}

		catch (ArithmeticException arithmeticException )
		{
		   System.out.printf( "\nException: %s\n", arithmeticException );
		   System.out.println("Zero is an invalid denomirator. Please try again. \n");
		}
	} while (continueLoop);
  }
 }


If you want I will edit it and upload my Gamemode shortcuts code chunks and rename gms to s etc


Hi Zenon,

I've just pushed in 5056446 that adds the swappable property to the migrations model and that works as expected to disable the migrations if the accounts.User model is swapped out.

As for the swapper project: I see it replaces the direct links into the (undocumented) Django swappable feature with calls to this unofficial API.  While not seeing the value initially, I agree, it makes the code more resilient to change - as the swapping framework used by auth.User is undocumented, it might be changed (or go away) without any warning.  But switching to the swapper project might be a bit too much given we are only using the swapping to replace auth.User.

If we were relying on it anywhere else in the code base, it would very much make sense to get the better maintainability.

But, as we are using it just to replace the user model, and, we are really concerned only about the case where our new user model is directly referenced from settings.AUTH_USER_MODEL, we can leave it as is.

We are only doing as instructed in the official documentation at https://docs.djangoproject.com/en/1.8/topics/auth/customizing/#substituting-a-custom-user-model

If the underlying mechanism in how you swap out the inactive model changes, it would change in auth.User.  To cover the option to again swap out our model, we would have to replace the use of the undocumented swappable framework with whatever the replacement happens to be.

But the core use case of just replacing auth.User with accounts.User would still work.  So I think we have done all due diligence on this and  can leave this code as is...

Cheers,
Vlad


Ok I will switch it out tomorrow.

Sent from my iPhone

> On Mar 13, 2016, at 8:06 PM, Pathjump <notifications@github.com> wrote:
>
> John I revised the face image with better pictures and formats please use that one was emailed to you after sent this one late last night.thx
>
> ―
> Reply to this email directly or view it on GitHub.
>


test


Yes, I could, but `enable()` and `handleRequest()` are two quite semantically different things, aren't they? I think that using "magic" whatever should only be done sparsely. What does the script know about things that happen *before* `enable()` is called? I maintain that configurability with sane defaults is always better than hard-wiring stuff.


Ok, pushed with some comments to hopefully avoid confusion in future.


As in not that method with the strolower, the current gamemode command you have works but is not the best for more customised features


I'm :-1: on this. Why remove indentation completely?


It's fun that by reading this page, I got to know about `haxelib path/run <libname>:<version>` not working properly. :-) I guess we just need a bit more tests...


Big :+1:


Oh I didn't even notice that. Thanks.

Also I just looked into it. socket.io is at 1.4.5 right now. I'll fix that up.


TeamCity Device Info :: Mock tests [Build 4](http://82.35.147.206:8111/viewLog.html?buildId=253&buildTypeId=DI_MockTests) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:07:34


TeamCity Device Info :: Mock tests [Build 5](http://82.35.147.206:8111/viewLog.html?buildId=255&buildTypeId=DI_MockTests) is now running


How about just doing

```js
var ARTOOLKIT5_ROOT = process.env.ARTOOLKIT5_ROOT || "../emscripten/artoolkit5";
```

You don't need to set it, so I don't think there should be a warning if it's not set


# this is a test of the repository from the local client. More files need to be added for the repository and the prototype to work


@andyli This now gives an error for some reason: https://travis-ci.org/HaxeFoundation/haxelib/jobs/115754826


Because if you write `Tracy = ...` you're basically writing into an uninitialised *variable* - there's no `var` before `Tracy` - and that will fail in strict mode. It's also bad practice, because if it *doesn't* fail, the variable is defined in the global scope - this doesn't matter here because we *want* the Tracy object available globally, but I've noticed usage of uninitialised global variables elsewhere in Tracy JS code - not doing it properly in one place means you get used to doing it this way because it "somehow" still works; but that doesn't make it okay.


Also wow, it even installed haxe 2.7 here: https://travis-ci.org/HaxeFoundation/haxelib/jobs/115754825. Something wrong with the PPA?


TeamCity Device Info :: Mock tests [Build 5](http://82.35.147.206:8111/viewLog.html?buildId=255&buildTypeId=DI_MockTests) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:03:43


Hi Zenon, no worries, this all makes sense.  Just pushed in 3d537bc.

Cheers,
Vlad


I am not aware of any reason to avoid them.  And in the cases where I've used them, the type is obvious to the reader based on the code.  It just makes it easier to read since we're dealing with special pointers.


TeamCity Device Info :: Mock tests [Build 6](http://82.35.147.206:8111/viewLog.html?buildId=257&buildTypeId=DI_MockTests) is now running


As a matter of fact, I do. `e` is generally used for caught errors in JS; `evt` makes it clearer what you're dealing with. Also, using meaningless single-letter variables except where their meaning is absolutely obvious (like `i` in a `for` loop) makes code less readable overall.


Oh, maybe it's just PPA servers are in trouble...


TeamCity Device Info :: Mock tests [Build 6](http://82.35.147.206:8111/viewLog.html?buildId=257&buildTypeId=DI_MockTests) outcome was **SUCCESS**
Summary: Running Build time: 00:01:34


Yeah, well, according to MDN, you're right.. I've somehow still lived in a time when there *was* no default value..


`"test": "jest"` should suffice


Because it's polite :-)


if we're going to make this a permanent feature, then we need some way to ensure that we still get notified when it goes down


@fbenkstein:
this is a reworked version of what we used in house for some time. would you mind having a look? especially the documentation might need some improvements before bringing it upstream.


Because they're not the same. Notice that I'm calling the `getElementById()` method of the `doc` variable, which is actually the document object of the window that was just opened. The node I want to affect is the cloned panel in the opened window, not the original one.


but the delegate name is **ASMediasFocusDelegate** in your framework...I think it is you means, sorry.. forgive my english.


again, it's just polite :-)


Superb. Thanks!


TeamCity Device Info :: Mock tests [Build 7](http://82.35.147.206:8111/viewLog.html?buildId=259&buildTypeId=DI_MockTests) is now running


ok, this probably won't work on Windows :/


What I had in mind was getting the latest version of the "Django 1.4" branch.  Here, I assumed we would tag it as ````djnro-1.0.0````.  We might possibly also refer to it as the ```old_stable````, but, this would have to be a branch that includes the InstServerM2M changes. So, the current ````next```` branch.

Before merging this PR, can you please make a tag pointing to the current ````next```` tip so that we can refer to it later?

And, are we on the same page about what this line refers to?  (Just wasn't sure what you meant by ````new_stable```` - there might have been some confusion there).


![blank page](https://cloud.githubusercontent.com/assets/17078886/13732525/fc43a6e8-e941-11e5-90b0-b81da2748d03.jpeg)


moved


Yes, that I do. But the fact is that the individual panels & tabs in the bar should be opaque to the script - it doesn't know anything about their inner structure or how they should be updated. This is the cleanest way to be *sure* (well, *almost* sure).


TeamCity Device Info :: Mock tests [Build 7](http://82.35.147.206:8111/viewLog.html?buildId=259&buildTypeId=DI_MockTests) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:03:06


The Zulip style doesn't line-wrap code like this -- just put the .find() on the same line as the first part.

(Same with the lines after this too)


See note about about line-wrapping


@myfluxi I keep getting this error when trying to build this driver:

```
drivers/devfreq/governor_cpufreq.c: In function ‘read_tbl’:
drivers/devfreq/governor_cpufreq.c:593:3: error: implicit declaration of function ‘of_property_read_u32_index’ [-Werror=implicit-function-declaration]
   of_property_read_u32_index(of_node, prop_name, j, &data);
   ^
cc1: some warnings being treated as errors
```

There must be something missing in my kernel code.


Multiple requests in parallel will result in only the last respective bar being rendered. I'm not really sure how to solve that exactly.. I don't mean the backend, I could obviously store the data in an array, but how should it look on the front end? I don't know. I think that on a regular site multiple AJAX calls don't usually happen simultaneously, or at least they usually aren't all "page loads". This code is supposed to make an *AJAX website* behave as similarly to a non-AJAX one as possible when a link to a subpage is loaded via AJAX; it doesn't aim to provide a full-blown debugging environment for advanced JS/AJAX client-side apps which run multiple AJAX requests in parallel on a regular basis.


Fixed issue #8


Hey @iamnbutler, instead of using a `png` for the logo, use [the svg partial I made](https://github.com/ysdn-2016/ysdn-2016.github.io/blob/develop/layouts/partials/logo.html). Using an inline svg is faster to load (b/c there's no second request), and can be styled with CSS.

```liquid
{% include 'partials/logo.html' %}
```

It has the `.logo` class applied to it, so if you want it to be black, just do this in your CSS:

```scss
.home-introduction-grid .logo {
  fill: $black;
}
```


> matrixA<-matrix(1:4,2,2)  # Generate 2x2 matrix
> matrixA
     [,1] [,2]
[1,]    1    3
[2,]    2    4
> mA<-makeCacheMatrix(matrixA)
> mA$get()
     [,1] [,2]
[1,]    1    3
[2,]    2    4
> mA$getinv()
NULL
> mB<-cacheSolve(mA)
> mB     # inverse matrix for matrixA
     [,1] [,2]
[1,]   -2  1.5
[2,]    1 -0.5
> matrixA %*% mB  # to confirm mB is inverse matrix for matrix A. AxB=I
     [,1] [,2]
[1,]    1    0
[2,]    0    1


mal formaté :(


*.html


Sorry, I saw the message of your commit late.

Permission checks at routing rely on model-level permissions already been validated; In this case, at shortcode generation.

See https://github.com/open-sausages/silverstripe-framework/pull/13/files for an alternative solution to this that retains the checks, but still skips the getErrorRecordFor part.


Something I have been thinking about is, for functions like Point::rotate, should they return int's to designate success or failure? As in some positions the mechanism will be unable to assemble.

This becomes a bit more complicated, as sometimes we will simply want to know if it can assemble, and sometimes we will want it to adjust the assembly.


great!


Created a datascience file


I don't think you meant to change this file.


Instead of calling the heavy-weight `yeti::foundation::fs::info()` we should rather use `::GetFileAttributes`.


Oh my, this is now broken with haxe-markdown... :)


"any build"


- Github上的Wiki页面或README.MD文档中，说明如何部署运行你的项目/程序

- 发表博客，截图说明代码运行结果和功能点


8 was chosen with 4K images in mind.  It directly affects the bitrate the Geforce can encode at.  For 1080p, 4 slices should be fine.


fsadoi;


This fixes:

ActionChain.uiConsume() and similar methods only accepts lambdas of type Object -> Object. (They should accept lambdas of type T1 -> T2)


code for testing input methods...
Line 44 :  st.nextLine();
As the next line is reading input as a String only.


Hi Zenon,

Unfortunately, this would not really be doable with the migrations.

The migrations framework uses the data stored in ````contenttypes```` as part of the meta-information about the models.

This goes as far as even touching the migration framework (running the migrate command) creates new ContentType entries for ````accounts.User```` that would conflict with renaming the existing ````auth.User```` entries.  (Thinking about it, I've just added an extra clarification to this section in 15a3c1d.)

The migrations framework own idea of the change is to add new entries and remove entries for models that are swapped out (auth.User).   However, that removes any information directly associated with users, including any audit trail data in the django log.

To preserve the data, the trick is to rename the existing data instead - but this has to be unfortunately done outside migrations.  Sorry, cannot be made any simpler :-(


A note on codec/bitrate:
During testing, I found that h264@20Mbps worked; however, hevc@20Mbps had problems.  hevc@25Mpbs worked fine.  This is the reason for not making an implied correlation that codecs were interchangeable for a given bitrate.


if 는 빼세요


correcion este es el product backlog version1, no el sprint backlog


else 만 쓰세요


Hey @GriffinSloves, it may be better to wait the random time when you enter this function for the first time. The two calls here are for after the page has been reached.


Doesn't look right to me - shouldn't this be sum / (tickCounter + 1) ?


in the deal click function, we get an error with POST for sending a bad request (400).

I tried changing the data we send with data: JSON.stringify(game), but nothings really changing


Maybe these two functions can be combined - that way everytime getRandomTime is entered there'll be a new seed.


lots of nice cleaning, I like it!


@caomengmeng
 For comment 1 "/etc/resolv.conf" has been modified in xCATreg so no need modified it here again.
 And please have a check about the syncfile after the cn installed .


Tcp.Close will " first flush pending writes and then close the socket."

Do you think its trying to close down more nicely at this point?


sure.
> On Mar 13, 2016, at 4:31 PM, Mark Waite <notifications@github.com> wrote:
>
> Could I persuade you to use 4 space indentation instead of 3 space indentation? The formatting in the plugin source code is already messy enough without changing from 3 space indentation to 4 space indentation inside the same file.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/mlgiroux/git-plugin/commit/cc58ba0f9805e237b34d0edad8fb0512cf0e4c8c#commitcomment-16670425>.
>


is there a benefit of Option.empty over None?


Memo:
* SupervisorTree: https://gist.github.com/yosukehara/86b3607e0cffc6e1692d
* UnitTest: Passed


It appears that log level INFO is showing in more places than I was expecting (like in the default Jenkins log file, and in the unit test output).  Would you be willing to remove the info log statements rather than risking cluttering an already busy log file with messages that are only really information for you and me?


pushed reformatted source.  Only reformatted the deriveLocalBranchName method, but the eclipse preferences are updated to use 4 spaces for any further changes.

Michael
> On Mar 13, 2016, at 4:31 PM, Mark Waite <notifications@github.com> wrote:
>
> Could I persuade you to use 4 space indentation instead of 3 space indentation? The formatting in the plugin source code is already messy enough without changing from 3 space indentation to 4 space indentation inside the same file.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/mlgiroux/git-plugin/commit/cc58ba0f9805e237b34d0edad8fb0512cf0e4c8c#commitcomment-16670425>.
>


Same here.  It appears that log level INFO is showing in more places than I was expecting (like in the default Jenkins log file, and in the unit test output). Would you be willing to remove the info log statements rather than risking cluttering an already busy log file with messages that are only really information for you and me?


This is the comment for CodeLineTest


TeamCity Units.NET :: CI [Build 7](http://teamcity.anj.no/viewLog.html?buildId=460&buildTypeId=UnitsNet_CI) is now running


TeamCity Units.NET :: CI [Build 7](http://teamcity.anj.no/viewLog.html?buildId=460&buildTypeId=UnitsNet_CI) outcome was **SUCCESS**
Summary: Tests passed: 688 Build time: 0:0:20


no way chunkResponse could be None at this point right?


You already define the store URL and the model URL inside of the class. No need to create a new constructor


They are only there for debugging, so yes, they can be removed.  I’m happy to remove them, on my end.

> On Mar 13, 2016, at 6:10 PM, Mark Waite <notifications@github.com> wrote:
>
> It appears that log level INFO is showing in more places than I was expecting (like in the default Jenkins log file, and in the unit test output). Would you be willing to remove the info log statements rather than risking cluttering an already busy log file with messages that are only really information for you and me?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/mlgiroux/git-plugin/commit/cc58ba0f9805e237b34d0edad8fb0512cf0e4c8c#commitcomment-16670868>.
>


You don't need this controller


Here's the facts as I see them:

1. Sorkin Type Co is listed in the authors file, and
2. the STC copyright starts in 2010.

Do you dispute either one? :)

Its irrelevant what it looks like. It matters that you started from his work and derived from it.... otherwise I can take any of Rosetta's fonts and modify them so they doesn't look similar and OFL them ;)