Do we need this file?
We generate swagger spec for only the versioned API.
Internal API is not exposed to users.
For ex: currently pkg/api/types.go does not have description tags.


Visual Studio will catch that and report an error message. The built in remote transport does the same thing.


_They_ would need to call super, but this class doesn't (it's super class is fixed to be Ember.Component)


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.433.fe0bf86b750a64d50d5300eb3c54938260da195f](http://build.palaso.org/viewLog.html?buildId=61167&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:31:33


Easy win.  Massive improvement in performance for block de-serialization.


Dang. We should fix that.


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


Default values are useless (don't exists is sensors blob)..
You can use https://github.com/ZdrowyGosciu/device_g2/commit/068631d8c20a17726e8fd2e01e05665bc72dd142 instead :+1:


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.428.6080bb9210ddd6f8ad0081e961b3a59a836701c4](http://build.palaso.org/viewLog.html?buildId=61170&buildTypeId=bt322) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:04:59


I'm tracking improvements to CONTRIBUTING.md in #42. Can you go ahead and
add these there, if they're not already?

On Mon, Aug 10, 2015 at 2:47 AM, Jonathan Leung <notifications@github.com>
wrote:

> I notice that this guide and the things the guide does not speak
> specifically to
>
>    - what a good commit message is
>    - squashing commits
>    - using rebase vs merge
>
> I wrote those 3 bullet points because I've been critiqued on the fact that
> I should do the above, but the guide does not provide information on how
> specifically we may want to do these for hackEDU.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hackedu/hackedu/commit/9259bfefd0d1b39686c1fbdca768640f9b0a4b6a#commitcomment-12610802>
> .
>

--
Zach Latta
hackEDU | zach@hackedu.us


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.13.fe0bf86b750a64d50d5300eb3c54938260da195f](http://build.palaso.org/viewLog.html?buildId=61166&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:37:56


This is the same assertion as in line 9, I would just remove it


ScheduleApp will replace SchedulerApp. I just haven't gotten around to completing the task.

Date: Thu, 13 Aug 2015 11:43:15 -0700
From: notifications@github.com
To: AndroidSchedulerApp@noreply.github.com
CC: peter_lu18@hotmail.com
Subject: Re: [AndroidSchedulerApp] Forgot to update Xamarin packages (78a445c)

Hey Peter, what is the relation between ScheduleApp folder and SchedulerApp? does one supercede the other, if so maybe we should delete the older one?

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


Ah right.. sorry about that -- must have been late at night. :)


Note that this file was huge.

I've tried to clean it out of the git history with BFG (https://rtyley.github.io/bfg-repo-cleaner/)


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 317](https://teamcity.labkey.org/viewLog.html?buildId=266222&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0 Build time: 00:02:17
### Failed tests
```
org.labkey.test.Runner.BeforeFirstTest: <no details avaliable>

```


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

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


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


Is there a reason we are doing that now as well? There is a return value after-all.


The only thing is that you don't know the sender, because CTCP can't send the sender or else the CTCP becomes corrupted.

Add an AllowCTCPCommand (or similar) for security reasons.


Great work! Like that you modified the CSS to create a nice appealing look. Also enjoy how you pull the information inside your JS file


arr my clean up Tool Deleted it :/


#### メモ

この関数(push_refzmp_xxx)の引数からfnsをなくそうと思って，ここの

```cpp
for (size_t i = 0; i < fns.size(); i++) {
         swing_legs.push_back(fns.at(i).l_r);
}
```

を

```cpp
for (std::vector<step_node>::const_iterator it = _swing_leg_steps.begin(); it != _swing_leg_steps.end(); it++) {
       swing_legs.push_back(it->l_r)
}
```

にしたら， swing leg zmp offset が 支持脚の zmp offset で置き換えられてしまう現象が起きた．

``rosrun hrpsys testGaitGenetarot --test8``を見てて運良くたまたま気づいたけど，``test-samplerobot-abc.py``は突破できてしまったので，どういうテストがあったら防げたか確認


That sounds like a fun weekend! Yea sounds great. We can work on the scripting and we definitely need more models and artwork


This is a decent idea but I think we can do a little better. We could temporarily replace Rails.logger with a custom logger that stores it's logs in an array. For example:

```ruby
class ArrayLogger < Logger
  attr_reader :messages

  def info(message)
    @messages ||= []
    @messages << message
    super
  end
end

Rails.logger = ArrayLogger.new(STDOUT)
```

Then we can assert that `Rails.logger.messages` includes the message we are expecting using `expect(Rails.logger.messages).to include "<message>"`.

I don't know if this will work perfectly but just giving you an idea of it


101162692-IddBiquadriaticCurvesObjList-4856 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1791 of 1791 tests passed)

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


@deniseschannon actually my interpretation was incorrect, sorry for the confusion. I've mixed it up with networkFrom parameter defining net=container mode for the container.


Um what I don't think I make a falceso branch.


Defaults to Oculus head rotation; to switch to KSensor change activity_jni.h: instead of "typedef GVRActivityT<OculusHeadRotation> GVRActivity;" use "typedef GVRActivityT<KSensorHeadRotation> GVRActivity;".


TYPO: (each to read) => (easy to read)
Found out that beginners have difficulty to imagine the variable structure by its name


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


Still seeing it clip with the scale. I tinkered a bit and found that setting `clipToPadding` and `clipChildren` to `false` on the following Views allows the animation to play unhindered:

* reader_cardview_post
  * `card_view`
  * `layout_footer`
  * `count_likes`
* reader_include_post_detail_footer
  * `layout_post_detail_footer`
  * `layout_icons`
  * `count_likes`


I've decided empty_list, empty_assoc, and undef should all be considered resolved since values were provided. I'm going to close this since that only leaves 3 lines changed, and is not very valuable.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 318](https://teamcity.labkey.org/viewLog.html?buildId=266248&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0 Build time: 00:02:16
### Failed tests
```
org.labkey.test.Runner.BeforeFirstTest: <no details avaliable>

```


rxinstaller has our own changes, so we shouldn't remove this notice. But we also should add credits for all contributors.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 319](https://teamcity.labkey.org/viewLog.html?buildId=266235&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0 Build time: 00:02:14
### Failed tests
```
org.labkey.test.Runner.BeforeFirstTest: <no details avaliable>

```


Don't forget to [memoize](https://en.wikipedia.org/wiki/Memoization) this so you can use `current_user` in multiple places without needing to worry about multiple database queries.

```ruby
@current_user ||= User.find_by(id: session[:id])
```


fast-forwarding master to auto = b4272b78


Hmm, yes, I think so.


This can be condensed, since `session[:id].nil?` itself will return true or false. Any time you're returning `true` or `false` from an `if`, consider if there's a better way to do it just using a boolean expression:

```ruby
def logged_in?
  !session[:id].nil?
end
```


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


Here's a guide *to* the most... #nitpick #preference #notrequired


PlantOperationSchemes (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2201 of 2235 tests passed)

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


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.14.fe0bf86b750a64d50d5300eb3c54938260da195f](http://build.palaso.org/viewLog.html?buildId=61176&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:34:33


@jnweiger Not working.

![image](https://cloud.githubusercontent.com/assets/4511254/9263533/4f48a8dc-421e-11e5-91a6-503323caf158.png)

We need to install the correct packages in build-ubuntu.sh as the Ubuntu package is 4.0.2.

**DEBUG=true**


Security reasons? Which of the CTCP queries impose a security risk to the server, especially considering it doesn't even care about the responses and only handles the VERSION query anyways?


Yeah that should work


Wrap to 80columns per style guide


no need for "iOS" twice


spaces on the insides of {} per style guide, common practice.


Seems reasonable to me.


remove "iOS"


I set the Cubes health to 30, the Gumbas health to 20, and the fireballs damage to 10 so the Cube takes 3 fireballs to destroy and the Gumba 2 fireballs. The values can be changed in the Inspector on the EnemyDestructableScript and the fireballs FireballController script.


Can you delete MODULE_HEADER up at line 22


It seems to be hard coded. Here is not right place to set the device radius.


THX for Fix.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 320](https://teamcity.labkey.org/viewLog.html?buildId=266233&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0 Build time: 00:02:23
### Failed tests
```
org.labkey.test.Runner.BeforeFirstTest: <no details avaliable>

```


Please add more details why you added FIXME.


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

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


>80 wrap per style guide


@barwinco , thanks for the reply. I think I did this correctly lol, I don't really use github much. Here is the repo [termite] (https://github.com/blacksky86/termite)


Right, they can send evil messages without a hint of who sent it. In that case I'll make the CTCP command OP-only, then it'll be fine.


Put these in an anonymous namespace.


No feedback on the PR, and then after this lands on master, I now get some. What's the expression I'm thinking of? Oh, right:

> `git push` it and they will come"

:wink:

There are plenty of places where they refer to it as "Ubuntu Fan", but I don't mind renaming it to "fan":

* https://lists.ubuntu.com/archives/ubuntu-server-bugs/2015-April/127917.html
* https://bugs.launchpad.net/ubuntu/+source/ubuntu-fan/+bug/1470091

They're going to submit pull requests to the kernel upstream, which, if accepted, will certainly make this worthy of being named just "fan".

**If you two will confirm that "fan" still sounds good (or some other name, for that matter), I'll rename it.**


Nice job!


`remap = false`. It's a forge class


I'd remove "the" at the end of this line so it reads "for use with Particle devices."


@borshop: retry


the same as I commented.


_s * i is the offset in bytes of the node we want to get, therefore we need a byte pointer to do the pointer arithmetic. I think you assumed that char does the trick. I know of no platform where char is not a byte, but uint8_t is maybe a bit more clear.


Well, that was because I got a `too many branches` error when running `tox`.


For issue #4


In Build, the heading in the libraries drawer is "Official Libraries", so let's use that rather than "Internal".


What's this commit? :fearful:


Keep the text, but link `spark-protocol` to https://github.com/spark/spark-protocol


Should the condition be negated:
```java
if (!ContentType.TEXT_RECTANGLE.equals(ct)) {
//  ^
```
I'm pretty sure this broke copy-pasting of block selections.


Erased from history <sniff>


@jasonody good catch! Must have forgot the --save flag! 😳


Do we want people to contribute to spark-protocol as well?


Change "more" to "mostly" since there's no "more A than B" clause.

I realize the shield are linked above, but I'd recommend adding them to this list as well.


No EOL. What editor are you using?


And actually listing the repo with the eagle libraries here as well would be helpful and prevent some questions.


lgtm


Note the lower-case ENA**b**LE_RELEASE.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 321](https://teamcity.labkey.org/viewLog.html?buildId=266341&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0 Build time: 00:02:17
### Failed tests
```
org.labkey.test.Runner.BeforeFirstTest: <no details avaliable>

```


Also TODO: make methods used the excluded variable from shared.py


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.489](http://teamcity.monogame.net/viewLog.html?buildId=9623&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:23:41


Ch


Love this section! We should also add a link to [Understanding the GitHub Flow](https://guides.github.com/introduction/flow/index.html).


great job, ancient issue


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 322](https://teamcity.labkey.org/viewLog.html?buildId=266348&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0 Build time: 00:02:20
### Failed tests
```
org.labkey.test.Runner.BeforeFirstTest: <no details avaliable>

```


saw approval from cmeiklejohn
at https://github.com/basho/riak_core/commit/63ff909175d702e069f6c51ffad8e3295c811d33


merging basho/riak_core/bugfix/jdm/vnode-proxy-stuck-in-overload-v2 = 63ff9091 into borshop-integration-767-bugfix/jdm/vnode-proxy-stuck-in-overload-v2


basho/riak_core/bugfix/jdm/vnode-proxy-stuck-in-overload-v2 = 63ff9091 merged ok, testing candidate = a80486b2


98318570_RemoveRadianceCheckbox (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Visual Studio 2013 for everything


TeamCity pull requests :: netty [Build 29](https://garage.netty.io/teamcity/viewLog.html?buildId=322&buildTypeId=pulls_netty&guest=1) is now running


Remove the `##`, Change this to InterlockedCompareExchange64


Instead of running a regex match on choice, turning choice into an integer, and evaluating it's amount this line is taking the return of the int() function on choice and assigning the value to goldnum.


Maybe change it by 0.12.0 for french?


Remove the casts to _int64*, _int64 is what InterlockedCompareExchange64


@keithbostic I compiled the code on Windows, and added comments for the minor corrections. It looks good to me otherwise.


LOL. Throw away it and use something better. Gedit is far lighter than VS. You may also like atom, a modern editor developed by GitHub. atom is heavier than Gedit, but less heavier than VS.


Hah, the whole thing sounds cool; thanks for doing it! :smile: was just staggered at how many times the word Ubuntu appears on this diff page!


changing the else if statement to:
If goldnum (which is an integer value derived from choice) is any number not less than 50:
    call dead("You greedy bastard!")


A *Shiny* example for 10x10 problem spaces:

![image](https://cloud.githubusercontent.com/assets/3079749/9263987/8742fc4a-4211-11e5-9dfe-747e84bbd336.png)


I like how VS automatically reformat the code... Once I put this-> } it reorders the code and the spaces to make the code look nicer :)


r+


@AlbertoSONIC Try Visual Studio Code
https://code.visualstudio.com/


if statement no longer needs to run the int() function on choice when evaluating. Code is much cleaner and only evaluates if goldnum is less than 50 instead of working again to turn choice into an integer.


Oh yeah lol. Got to change my mindset on it not being relational. lol


commit com uma linha em branco. faz isso não gente :) limpaê


Arf.. It's because MongoDB use a custom class for the ID instead of a simple string/integer...
Nothing to do with Python, just a crappy DB connector :)


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


merging graydon/stellar-core/asset-code-fixes = 7c26d1c9 into auto


graydon/stellar-core/asset-code-fixes = 7c26d1c9 merged ok, testing candidate = 3a47f1ff


sua ajuda está aqui:
https://paste.kde.org/pcta4kk4m

Tomaz


Do you use `search` method defined at `model/product.rb`?


It's possible even on atom. Try other editors. You won't be back to VS unless you are using a project file.


@jesserosato @jasonody Hey guys, thanks so much for getting this started! I'm wondering if we could use this app as an opportunity to document how we built something from beginning to end. One way to MVP that documentation is with more descriptive commits so that we can reuse the content from each commit description to create a step-by-step tutorial that anyone could replicate. I could take the lead on documenting user research and usability testing.

I don't want this to be too big of a load on anyone, but it seems like it could be really useful (esp. for ppl like me who might be new to Backbone). Thoughts?


Lol. Yeah, dropping so many references to Ubuntu did make me feel a bit dirty, to be honest :blush:. I'm starting to feel persuaded to rename it to just "fan", instead of "ubuntu-fan" (that is the end goal, after all)...


Yes, In that case, the conversion to json just freezes the program, I don't know why.


TeamCity pull requests :: netty [Build 30](https://garage.netty.io/teamcity/viewLog.html?buildId=323&buildTypeId=pulls_netty&guest=1) is now running


Nah evil messages no, but they can spam...


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1274 of 1274 tests passed)

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


@nastys I have also it. But doesn't reorder the code when i tap }!

@173210 I'll try it once I have a better connection.. I'm in roaming with a mobile hotspot :(


TeamCity pull requests :: netty [Build 29](https://garage.netty.io/teamcity/viewLog.html?buildId=322&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 4639, ignored: 16 Build time: 00:09:47
### Failed tests
```
io.netty.handler.codec.http2.DefaultHttp2FrameIOTest.binaryHeadersWithoutPriorityShouldRoundtrip: <no details avaliable>

```


Missing a trailing "."


@vene did I make it too verbose? or does this seem okay?


I had put three dots instead of 2!


saw approval from cmeiklejohn
at https://github.com/basho/riak_core/commit/63ff909175d702e069f6c51ffad8e3295c811d33


merging basho/riak_core/bugfix/jdm/vnode-proxy-stuck-in-overload-v2 = 63ff9091 into borshop-integration-767-bugfix/jdm/vnode-proxy-stuck-in-overload-v2


basho/riak_core/bugfix/jdm/vnode-proxy-stuck-in-overload-v2 = 63ff9091 merged ok, testing candidate = 806e8575


saw approval from jonmeredith
at https://github.com/basho/riak_kv/commit/33178e05048449bf5bb4d18a23ba7d9ad7b800c5


merging basho/riak_kv/bugfix/lww-dvv-validation = 33178e05 into borshop-integration-1162-bugfix/lww-dvv-validation


basho/riak_kv/bugfix/lww-dvv-validation = 33178e05 merged ok, testing candidate = 81278a2f


Yes, it should raise a TypeError exception. It's because the default JSON encoder follows the specification of JSON. So, it only convert the default type: https://docs.python.org/2/library/json.html#encoders-and-decoders


If I rename the option, that still leaves the question: do I rename the other bits? I would say "no", at least for now: the Fan Networking thing is still new enough that I figure the extra context helps. Perhaps when it's a little more mainstream, we could rename the patch files and stuff. But I'm cool with whatever.

> Hah, the whole thing sounds cool; thanks for doing it!

I think so! I hope someone will be able to put it to good use. I have tentative plans for using it for some large-ish Mesos deployments, but I haven't got around to it yet.


Use `git pull --rebase`.


This is not correct.

Your line 182 in pawns.cpp is part of the pawn evaluate function.
which is called only when there is a new pawn move.

The result is stored in a hash function, and will be reused for the next few moves,

So it is not logical to call pos.pieces(Us) at this place.
This is why we have this is evaluate.cpp instead.


FWIW: in the new container infrastructure you can install custom packages, but they must be included in [the whitelist](https://github.com/travis-ci/apt-package-whitelist/blob/master/ubuntu-precise) (and, at the moment, `optipng` seems to be the only one whitelisted among the four).

This is the new syntax:

```
addons:
  apt:
    packages:
    - optipng
    - and
    - so
    - on
```

Maybe you can ask to add the other three packages to the whitelist. The README says: "The approval process is mostly about ensuring the .deb installation hooks don't do anything malicious or goofy that would open up a container to potential attack from a neighbor".


Yeah, totally!


airtartan test


if you are receiving the value in the attr_accessor you can directly use that variable, like in this case you are not using the instanced variable in any place into your class.


Done. http://pastebin.com/FDeCb4mc
Now what? "git push origin master"?


Fix'd!


```
auto const ledger = getValidatedLedger()
if (ledger)
```
...


Thanks! After looking a bit more closely at the native interface of `FBSDKLoginManagerLoginResult`, I think the resulting object here should be of the type `FBSDKAccessToken`. That would mirror the iOS and Android APIs. So I think the solution here is to send down a token dictionary from native and before we actually call the callback on the JS side, we create an `FBSDKAccessToken` object from it:

```objective-c
@"token": result.isCancelled ? [NSNull null] : @{
  @"tokenString": result.token.tokenString,
  @"permissions": result.token.permissions.allObjects,
  // etc., see RCTFBSDKAccessToken.m
}@
```

and then in `FBSDKLoginManager.ios.js`:

```javascript
static logInWithReadPermissions(permissions: Array<string>, callback: (error: ?Object, result: ?Object) => void): void {
  FBSDKLoginManagerInterface.logInWithReadPermissions(permissions, (error, result => {
    var token = new FBSDKAccessToken(
      result.token.tokenString,
      result.token.permissions,
      // etc., see FBSDKAccessToken.ios.js
    );
    callback(error, {...result, token});
  });
}
```

Bonus points if you actually make the `FBSDKAccessToken` constructor eat a dictionary/object instead of individual parameters, because then it becomes a lot easier, and `FBSDKAccessToken.getCurrentAccessToken` becomes simpler, too.

Sorry for not spotting this earlier, but I think we should try to be as close to the iOS/Android APIs as we can here and not cut corners.


This should be `||= false` for the default of `false` ... but a file called `celluloid/unlocker.rb` ought to be added with this:

```
$CELLULOID_UNLOCKER = true
require 'celluloid/managed`
```

And also a file `celluloid/managed.rb` which I will make:

```ruby
$CELLULOID_MANAGED = true
require 'celluloid/current'
```


The call to `(remove-usernames-from-template)` can go away here as well, and I think this is the last call to it.


Yes, but, unfortunately we have already commits after this commit. Please be careful from now on. ;)


I will, thank you :wink:


I think that a comment which explains that which is trivially apparent by examination of the code in question is superfluous. I'm not sure that a comment is even necessary here but if we are going to have one, it should be one that explains why its okay to return a default constructed `Rules` object.


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


```
bool const allowMultiSign =
    m_ledgerMaster.getValidatedRules()->enabled(
        featureMultiSign, getConfig().features);
```


It says:
> Everything up-to-date

I'm sorry, I'm quite new to git...
Do I need to add a commit?


This comments reads a bit weird. Are punctuation/some words missing?


Actually, I already have `celluloid/managed.rb` on the `0.18.0-release` branch and earlier.


Si je comprends bien, stripBOM cherche le caractère '\ufeff' dans le text et l'élimine? Je remplacerais ta fonction avec:

    def stripBOM(fileName):
        with open(fileName, encoding='utf-8', mode='r') as f:
            text = f.read()
        text = text.replace('\ufeff','')
        with open(fileName, encoding='utf-8', mode='w') as f:
            f.write(text)

ce qui devrais être équivalent, mais plus facile à comprendre? Mais bon, c'est plus une question de stylistique.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1706


Could use `JLOG(m_journal.warning) << "Submitted"`...


We prefer to keep closing braces on the subsequent line, indented to the level of the line containing the matching opening brace. For reference: https://en.wikipedia.org/wiki/Indent_style#Variant:_1TBS


If you want to add changes, of course you should.


Although not incorrect, it's better to say `return undef` here or even just `return` (the undef is implicit).


101235614-Fix-AFN-Error-Output (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


lol can't you do this in your fork?


101235614-Fix-AFN-Error-Output (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


File names in both Updates entries should change"i15bl ..." to "i015bl..."


> nastys@ULNS:~/GITmain/rxTools$ git status
> Sul branch master
> Your branch is up-to-date with 'origin/master'.
> nothing to commit, working directory clean

Maybe it's easier to just revert the changes manually...


test web hook


Although `foreach` and `for` are functionally interchangeable in Perl, it is strongly preferred to use `foreach` for the syntax here, and reserve `for` for the syntax that uses an explicit iterator (e.g. `for ($i=0; $i<0; $i++)`


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


So you want to remove this commit? It's not recommended because it's already uploaded.


A organização das classes do layout está começando nesse commit... é importante notar que a largura está maior do que os 740px estipulados...estando assim em 760px.. rever classes e suas configurações de margin.


I'll work on re-wording that.  It's one of those strange areas where I want to keep the goal clear -- that `Task` and `Task<T>` are mechanisms for coordinating your asynchronous work -- but I do agree that some more elaboration on what a `Task` object could potentially do is warranted here.


PluginZoneInitialization (macumber) - i386-Windows-7-VisualStudio-12: OK (2213 of 2233 tests passed)

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


TeamCity pull requests :: netty [Build 30](https://garage.netty.io/teamcity/viewLog.html?buildId=323&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 4639, ignored: 16 Build time: 00:19:39
### Failed tests
```
io.netty.handler.codec.http2.DefaultHttp2FrameIOTest.binaryHeadersWithoutPriorityShouldRoundtrip: <no details avaliable>

```


Calling setStickyExtensionView would always be YES. This fixes it to behave how it should.


I am doing the travis test with fork, which is why I disable it for JuliaLang travis.

I somehow couldn't get my personal AppVeyor working though.


Note that adjusting the translate and clipExtent here assumes we are using the default projection. It may be best to adjust how we accept projections (I'm thinking a function).


I shouldn't have to read the code to figure out what a commit does, "Fix typo" or "Rename passwordPrompt -> eateryPrompt" would probably have been better. Very high-level yet unambiguous


I thought it couldn't here, but I might be misunderstanding -- it looked like it was removing them from the template itself in order to pass it along in query parameters instead -- the MetadataTemplateUpdate schema doesn't allow created_by/modified_by attributes. But maybe replace-template-user-ids is what put them there in the first place so it's not necessary?


I'm confused because this commit added things that were already added in previous commits...
It doesn't make any sense!!! :confused:
But now I think this commit actually reverted nothing, so there's no need to revert it.


Pour ce qui est de la bonification des erreurs, une solution simple, c'est d'ajouter le str de l'exception. Ça donne plus d'infomation sur ce qui s'est passé. Donc faire:

    except Exception as e:
        print("Erreur dans stripBOM. Ne peut pas ouvrier le fichier:" + fileName + " erreure: " + str(e))

Personellement, j'aime bien les stack traces (toi aussi, d'ailleurs, si j'interprète les lignes 231 à 235. Je créerais ma propre class d'exception comme ceci en haut du fichier:

    class ExceptionDeContract(Exception):
        pass

et ensuite, dans cet example, je ferais:

    except Exception as e:
        raise ExceptionDeContract("Erreur dans stripBOM. Ne peut pas ouvrier le fichier:" + fileName + " erreur: " + str(e))

De cette façon, on 'bonifie' l'exception d'information supplémentaire comme le nom du fichier, mais on garde la trace du stack trace.


Yeah makes sense. I think the JuliaLang AppVeyor is on faster machines with 2 cores per worker, the open-source plan can probably work but will take longer to run tests in serial.


fast-forwarding master to auto = 3a47f1ff


er, that doesn't make any sense either, since replace-template-user-ids was running after this (`->` confuses me at times, heh, I like my backwards-thinking paren-nesting :P)


@markbenvenuto, I think I did what you wanted, would you please check?


And I'm pushing this branch mainly for AppVeyor.

Since @staticfloat still haven't setup the buildbot he promissed, I figured that I might as well use the CI for it on a different branch/repo....


Hmm, maybe I'm missing something. I added this function when I migrated the
service, and I thought it was only to ensure that usernames weren't being
passed in. I'll take a closer look at the metadata service and let you know.

On Thu, Aug 13, 2015 at 4:43 PM Ian McEwen <notifications@github.com> wrote:

> I thought it couldn't here, but I might be misunderstanding -- it looked
> like it was removing them from the template itself in order to pass it
> along in query parameters instead -- the MetadataTemplateUpdate schema
> doesn't allow created_by/modified_by attributes. But maybe
> replace-template-user-ids is what put them there in the first place so it's
> not necessary?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ianmcorvidae/DiscoveryEnvironmentBackend/commit/921df86504ab1a7db33eb8b49eb214a2ef9816f3#commitcomment-12701939>
> .
>


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2287


@nastys The merging commit shows when you merged commits. So Git shows all changes on those commits.


PlantOperationSchemes (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2206 of 2235 tests passed)

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


What's wrong with our build of it?


Thank you for the informations!

I wanted to compensate for the fact that no more passed pawns were evaluated. So I need to put my modification in the evaluate.cpp, evaluate_pieces function, is this correct?

Also, is the removal of the else if in evaluate.cpp is considered a simplification?


having trouble, getting this error message when I  try to mine.

"miner  23:40:15|ethminer  Getting work package...
JSON-RPC problem. Probably couldn't connect. Retrying in 1..."


He'd probably entertain a PR or offer guidance for what to change at https://github.com/staticfloat/julia-buildbot

Do you have a login to the buildbots? Either me or Elliot can email you one. You can manually trigger "rc" builds from a branch, though we still don't run the windows tests there due to #11727 so appveyor is more useful for that. AppVeyor's not too busy today so not a big deal.


Tu peux combiner les deux lignes en `if left(ligne2,2) != "20" and len(ligne2) > 0:` si tu veux moins d'indentation.


```
lib/kmer_hash.hh:142: warning: Found unknown command `\Class
```


Same as the other; lowercase 'c'?


Les regex, c'est fort, mais souvent c'est dure à suivre. C'est très bien quand il y a beaucoup de conditionels, genre je cherge les caractères `a`, `b` ou `c` suivi d'un `)` et de un ou plus d'espaces. Pour ces contracts ci, le formattage semble assez simple que juste des `finds()` font l'affaire.


See .Travis.yml in frau-jwt?


@tremainebuchanan actually that `DEFAULT_PASS` shouldn't have been changed enuh.  It is necessary for the setup.  After the setup the administrator would have to delete the environmental variable.


thx.

Just wondering what the purpose of  drivers/adodb-mysqlpo.inc.php is.

is 'po' standing for the name 'James Poon'? Is the file obsolete?
*scratching head*


Not sure, but should it be `max(...)`?  I thought `startAtZeroEnabled` means on the graph we should see zero or positive values only. No negative values at all.


Ah right sorry I was looking at this too quickly, thought it was in some section other than the Cygwin build for whatever reason. Was probably fine the way it was minus the trailing whitespace, but this is good too.

Minor note that in the future it's preferable to avoid the `@` mentions in commit messages, since that will trigger a new notification every time someone does a rebase that includes this commit. Really that's a github bug, but an avoidable one.


Containing #28, #30, #33, #34 and a custom (obsolete) menu entry for led control


@mo4islona I was looking for a npm-ified version of blockly that worked with browserify, and was able to get yours working with very minimal changes. Maybe this can be turned into an option or flag somehow? The only differences are: 1) don't need to bring in `DOMParser` and `XMLSerializer` from node, and 2) set `goog.global = global`.


Faudrait qu'on se parte un peti projets rails ici. Genre on fait un site simple qui va parse de l'info d'un autre site. Genre on fait un mini boxe rec.


Missing period at the end.


> in fact, this commit is from janice.. and the trick with changing the two video modes to make it more "smooth" doesn't work for janice.. but u can be "sure" that here on codina the trick with the two Values on WS LCD works as well on S6D LCD..... tested and works by me ;-)

I didn't mean changing those values itself, I'm aware it works as well on s6d.
This commit also required following change(or am I wrong?) https://github.com/ChronoMonochrome/Chrono_Kernel-1/commit/eed81ef35d37aaba59185f421d83b7b59f060fde , which might not work on s6d.


Why do we need merge deep for an object like this that's not that deep?


Missing period at end.


For some reason, when I was experimenting with index_col, it shifted the headers to the right. The only way I got it to stop doing that was to set the index_col equal to -1.


This is not part of Boilerplate.


Awesome! Ok, I just pushed v1.25.0 with the new changes, and a few other additions :grin:

Thanks again!

Should I add you as a collaborator? :smiling_imp:


You need a CHANGELOG is not part of Boilerplate.


is there an example of using this in meteor project?


i removed it, also from package.json


```java


Cuz I guess Nunjucks can have something deep.


start small. :- )


Not part of Boilerplate... Propose that fix in a PR over there.


TeamCity pull requests :: netty [Build 22](https://garage.netty.io/teamcity/viewLog.html?buildId=280&buildTypeId=pulls_netty&guest=1) is now running


@flavour But parameter url is constructed from resource_id, so should be inside loop, no?


`html`.... Still no... Adding all templates to `"html"` ends up clogging up processors for https://github.com/jstransformers/inputformat-to-jstransformer/blob/master/dictionary.json#L161 . There is no reason to Nunjucks on pure HTML. Where are you using this?


TeamCity pull requests :: netty [Build 22](https://garage.netty.io/teamcity/viewLog.html?buildId=280&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 3600, ignored: 12; exit code 1 (new) Build time: 00:03:33


We need to get a 'known' proper hash with some 4 byte data.  Is this format in pass_gen.pl ?   MSsql05, yet it is.

First off, are we 100% SURE we know how this software actually WRITES data that would be > 65k in the character set???   I thought mssql05 used UCS2 vs unicode or UTF16.  USC2 is deprecated and was replaced by UTF16, but it was not 100% the same.  I do not know what the differences were, btw.


`{watch: false}` is a one-level object. It is not deep at all. `merge-deep` is not needed here.


Should we just strive to always use camelCase for all new code? My natural tendency is to use the casing that pervades in the current context (not sure if snake_case is exactly appropriate here but 'ya get mah gist' ) ?

You could possibly argue that making the move to camel case while fresh with the code is less risky?


```
(defn fib [a b]
  (cons a (lazy-seq (fib b (+ a b)))))

(apply + (filter even? (take-while #(< % 4e6) (fib 1 2))))
```

Clojure power !!


My mistake. Edited coors of dead_man


Hmm right now i see u've also to change .clock_div @ "1" if u want to use SHARP_FREQ ;-)


For me the question is why gray-matter, lodash and etc accepts "html", LOL! They can accept mustache or nunjucks in same way that they accept html :laughing:

Same because
https://github.com/jstransformers/jstransformer-qejs/blob/master/index.js#L7
https://github.com/jstransformers/jstransformer-marked/blob/master/index.js#L7
https://github.com/jstransformers/jstransformer-commonmark/blob/master/index.js#L8

and more


***important***  1048576 is not the best file size to use.  You will lose some a/v data at the end of each file this way.  If you wish the size to be near 1MB, then please instead set it to 1048476.

1024*1024 div 188 (bytes per TS packet) * 188 = 1048476

side note:  in the future, we should have this size limit a loose guideline, while dvbtee's parser will actually just cut the file at the closest keyframe.


> The intent of inputFormats is to indicate what file extension the engine should process the format.

exactly. can accept html, modern editors have syntax highlighting for popular template languages.


Should this be a more generic name? The actual behavior is to search for any text in the concept?


For 2.2 are we planning on adding a separate option for enabling/disabling token metadata separately from schema metadata?  Based on your comments in [JAVA-828](https://datastax-oss.atlassian.net/browse/JAVA-828) you can do TokenAwareRouting without schema metadata, but you still need the token map.


Yea, know that. But didn't know that Nunjucks have only one-level options.
http://mozilla.github.io/nunjucks/api.html#configure

:+1:


I guess either it could be a more generic name or it could be a more specific query.


to ensure that `output_file_size_limit` is a multiple of 188, lets do something like:

```output_file_size_limit = ((long)(strtoul(optarg, NULL, 0) / 188)) * 188


Update your editor to accept the needed file extension.


Can you explain why this change has been made and also how you have come to the values for both GetStakeCombineThreshold and GetStakeSplitThreshold please.


I wonder if this should be more restrictive. But maybe the Dialer will throw if `port==0` or is 21 or somesuch?


いやっほう！


97204612_MultiSpeedFan (Nigusse) - i386-Windows-7-VisualStudio-12: Build Failed

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


通过


urgh, thanks @robhudson :(


This file is definitely not from pasta. Can you guys please mark files from outside sources as such, and not license them under GPL without the authors' permission. And please don't misrepresent(by omission) where you got the patches for n3ds from. At least make a note in the commit or the file that you got it from Rei (and cowritten telepathically with Cakes).

https://github.com/roxas75/rxTools/commit/5efcf233acd2b6341c8f740c3d95143430c097a3
https://github.com/Reisyukaku/rnInstaller/commit/29f7e65fb8edeecba9e250449a8babbcab098217

https://github.com/roxas75/rxTools/commit/6d36ec6fd23bcf24c080e98bc6dca3104fdc8caa
https://github.com/Reisyukaku/rnInstaller/commit/7c51504d4621c1648dd61f9359a83f50ca193ba3


@jych is on an internship right now but I'm going to guess that the implementation was broken in some way.


Java开发者哭晕在厕所..... (Java规范: 方法名用小驼峰命名)


It would be more intuitive to have `spTrainingSize` and `tmTrainingSize` define the start of SP and TM training respectively, don't you think? Then we would also want a `classifierTrainingSize`. This may seem overkill in this model, but we shouldn't assume all htm network models will follow sp to tm to classifier.

And update the docstring please.


972124-2NewDetailedIceStorage (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

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


any ideas on what about this commit broke NPC and projectile rendering?


Even if it was more than one level deep, it doesn't matter, since `watch` is only one level. And passed options should always just override it.

I suggest you revert this change and propose it in a PR.


这是git commit --author 还是真的@torvalds来了，吓死我了！


My first read through of the Golang source made me think the HTTP/TLS
handlers would fail out if we tried to connect to some port that wasn't in
use/presented the wrong protocol etc. I should probably check again though.

On Wed, Aug 12, 2015 at 8:23 PM jcjones <notifications@github.com> wrote:

> I wonder if this should be more restrictive. But maybe the Dialer will
> throw if port==0 or is 21 or somesuch?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/letsencrypt/boulder/commit/e629c61d5df0b7cc56e768ecda43761c04a88ebd#commitcomment-12680370>
> .
>


Need one case to detach_disk from a VM without any datadisks. Expect an error is thrown.


环境如下：
ubuntu15.04
virtualenv
mysql-server, mysql-client
virtualenv --python=python3 testenv
使用source激活testenv后，安装django，得到最新版1.8.3（新建一个标准django项目数据库使用mysql，运行runserver正常）
在mysql里创建mis数据，使用source命令导入SQL/mis.sql，没有报错
进入django-erp文件夹，执行python manage.py runserver，无法正常启动服务器。
请指点下是怎么回事，谢谢！


It won't make difference. Every required changes are done in init function.


Need a case if the disk does not exist.


ちなinline要素でborderは改行すると思った動きをしない可能性もあるので注意です。
なんか良い感じのスクショ付き説明があったので貼っときます。
http://mozilla.gr.jp/standards/webtips0015.html


I would avoid anything that is just a variation of a vanilla road.

The 2 lane highways were a nice addition.

I'm not sure about the planning roads. I would prefer some kind of planning overlay that can be toggled instead.


empd (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


> This commit also required following change(or am I wrong?) eed81ef , which might not work on s6d.

right now i'm not sure, but i can suggest to compare s6d & WS driver .. there are some differences with ESD, lcd_power_on and maybe one other option .. backlight i.e.

> I'm not sure - decreaing of others seem to reduce tearings a bit. Did you test every possible config(or at least change of a few parameters at the same time?). I think no.

i've tested every single parameter.. one time org. Value x2 and one time org. Value /2 :-)

I didn't test smaller changes in values.. bacause it was a bit painful to build and flash on the Phone so many times... xD ... but the 2 working Values, those 2 i've test it all (not every single number..) ^^ .. even with every LCD freq .... And the best result is my default.

btw: even if it really helps a bit, u should keep in mind that reducing that values is like overclocking the lcd freq ... probably like more aggressive timings .. and yes it drains also more battery .. so at the end it's much easier to just change lcd freq instead to mess it up as it changes also the speed at all ^^ Especially for WS, stock values are already great... only the freq is crappy. But here on S6D is bit different since stock Values are sooo high ... (63/63) and on WS already nice low ---> 8/8.

btw2: for me the common reason to change those 2 values here on s6d is this: With stock video modes the best result was on 39Mhz .. but after some time of use the screen had some issues (i don't remember exactly what it was...freezes or so) now, am down to 36Mhz + tuned video modes and the result is ALMOST the same.. AND the LCD is completely STABLE. :-)

In that case it's helpful ... but on my old WS LCD .. i was able to use highest freq ---> 80Mhz without problems .. BUT the smoothest freq was STILL 57Mhz (with stock video_modes) !!!!!!


now master segfault while trying to make test:
````
(gdb) bt
#0  0x00002afe64a3a016 in ?? () from /lib/x86_64-linux-gnu/libc.so.6
#1  0x000000000047604c in read_location (tzf=0x7ffe71170120, tz=0x3213640)
    at /home/huixinchen/opensource/trunk/ext/date/lib/parse_tz.c:251
#2  0x00000000004768b9 in timelib_parse_tzfile (timezone=0x2afe69a027a8 "UTC", tzdb=0x12b9400 <timezonedb_builtin>)
    at /home/huixinchen/opensource/trunk/ext/date/lib/parse_tz.c:394
#3  0x000000000042e77b in php_date_parse_tzfile (formal_tzname=0x2afe69a027a8 "UTC",
    tzdb=0x12b9400 <timezonedb_builtin>) at /home/huixinchen/opensource/trunk/ext/date/php_date.c:930
#4  0x000000000042eab1 in get_timezone_info () at /home/huixinchen/opensource/trunk/ext/date/php_date.c:1005
#5  0x00000000004303eb in php_format_date (format=0x2afe69a96858 "Ymd_Hi", format_len=6, ts=1439437119, localtime=1)
    at /home/huixinchen/opensource/trunk/ext/date/php_date.c:1256
#6  0x0000000000430389 in php_date (execute_data=0x2afe69a16110, return_value=0x2afe69a14dd0, localtime=1)
    at /home/huixinchen/opensource/trunk/ext/date/php_date.c:1243
#7  0x0000000000430a8e in zif_date (execute_data=0x2afe69a16110, return_value=0x2afe69a14dd0)
    at /home/huixinchen/opensource/trunk/ext/date/php_date.c:1372
#8  0x0000000000a5615b in ZEND_DO_ICALL_SPEC_HANDLER () at /home/huixinchen/opensource/trunk/Zend/zend_vm_execute.h:577
#9  0x0000000000a55ba4 in execute_ex (ex=0x2afe69a14030)
    at /home/huixinchen/opensource/trunk/Zend/zend_vm_execute.h:406
#10 0x0000000000a55cb5 in zend_execute (op_array=0x2afe69b1e300, return_value=0x0)
    at /home/huixinchen/opensource/trunk/Zend/zend_vm_execute.h:450
#11 0x00000000009fad26 in zend_execute_scripts (type=8, retval=0x0, file_count=3)
    at /home/huixinchen/opensource/trunk/Zend/zend.c:1404
#12 0x0000000000965998 in php_execute_script (primary_file=0x7ffe71172980)
    at /home/huixinchen/opensource/trunk/main/main.c:2475
#13 0x0000000000ab9d1d in do_cli (argc=18, argv=0x30c5f80) at /home/huixinchen/opensource/trunk/sapi/cli/php_cli.c:971
#14 0x0000000000abaee4 in main (argc=18, argv=0x30c5f80) at /home/huixinchen/opensource/trunk/sapi/cli/php_cli.c:1338
(gdb)

````


Is there a summary somewhere of what build-ng changes/improves?


ChallengeResponseAuthentication is no by default in debian


que pedo


Removing this = fix


Not sure I understand


Wouldn't it be better for debugging if this is after `network.run(1)`? And the training sizes can just increment from 0.


done, thanks for the lesson, much better now.


and thanks for caring!


Cmake internally defaults to trying to use the parent jobserver with make, specifying a build command invalidates that


Not working.


Is submitNodeListRefresh needed since submitSchemaRefresh will call submitNodeListRefresh if table is null?


I thought that this might be a symptom of a long-computation in the render() method, but behavior shown in 7835831bff0c955f969a2e8775dce33d139e56bd seems to show this isn't the case. Why was this not a problem when dealing with the y-axis scrolling?


@bigdaz https://github.com/gradle/gradle/blob/master/subprojects/model-core/src/main/java/org/gradle/model/internal/registry/ModelNodeInternal.java#L184-184


> right now i'm not sure, but i can suggest to compare s6d & WS driver .. there are some differences with ESD, lcd_power_on and maybe one other option .. backlight i.e.

It probably anyway makes no sense without a real test.
@ videomode params: ok, thanks for your advices, I'll play around tuning video mode a bit, and if I won't be satisfated with the result, I'll just left defaults.


Will figure this out later


okay but did u ever tried to use your LCD with wrong clock divider ? xD
That looks terrible ;-)

I now that it'll make no changes at all.. but imagine PolishVodka are right .. then u've it messed up even more xD

Okay fun by side :-)

It's probably really ALL nonsense since i did following test with ADB:

set an too high unstable LCD freq ----> immediately black screen or sometimes blinking....
Set a org. freq with ADB after it was set too high = everything okay again.

That means even if the LCD start to work with an unstable Freq, it's just not working (BS) for some milliseconds :-)


I forgot to mention that I also rename the fragment to ChrootManagerFragment...


UpdateCIsettings (Myoldmopar) - i386-Windows-7-VisualStudio-12: Tests Failed

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


RoomAirflowNetwork (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1792 of 1792 tests passed)

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


Nice catch, @rohgarg!  Rather than generate a formal pull request, let's say that I reviewed it, and LGTM.  Why don't you go ahead an push in this fix as a new commit.


IS this temporary? Can it get a comment?


> Update your editor to accept the needed file extension.

It's not why i'm using it. As I said, everything is possible with normal editor.
It's about principles. Marked and CommonMark also aren't exactly proper "markdown". CommonMark accepts commonmark not, markdown, not gfm or whatever. But yea, with the markdown is little bit tricky and point of view.

QEJS transformer accepts html? And it's not only him as i said.


It's not finished. So, I kept comment until fix it in next commit.


shouldn't you call `textValue` here instead of `text()`


https://www.codeaurora.org/cgit/quic/la/kernel/lk/commit/?h=master&id=40b227df7c5ab3ecea9e34a9bd8603ee33118ec6
Below are the steps/commands to convert the splash image into a character array:
1. $convert -depth 8 <splash image in PNG format> rgb:splash.raw
2. $host/linux-x86/bin/rgb2565 < ./splash.raw > final.img
3. $hexdump -C -v final.img | tr -s " " | cut -d ' ' -f2-17 | awk 'BEGIN {print "{"} {OFS=""; ORS=""; print


Maybe add some comments to the code?


Unharsh the logo. That's what I meant. Why can't I make up words.


fwiw here's groovebasin's code for this: https://github.com/andrewrk/groovebasin/blob/master/src/client/uuid.js


Is this github ident issue or this one really went all the way to the right? :)


Thx!

Am Donnerstag, 13. August 2015 schrieb Simon Cropp :

> @danielmarbach <https://github.com/danielmarbach> FYI i updated the
> callbacks doco
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/docs.particular.net/commit/d60d3ffa7fe56694fd618d2ad6d1a3e4b265623b#commitcomment-12678745>
> .
>


`outlives` should be `components`


The root of this issue is use of the CHARACTER zIndex. It seems like anything that renders after the grid gets ignored.


good job michelle


Okay.
On 2015 අගෝ 13, බ්‍රහස් at පෙ.ව. 9.45 Ind <notifications@github.com> wrote:

> It's not finished. So, I kept comment until fix it in next commit.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kadirahq/meteor-debug/commit/1d182525a47ad1b1b7d3f9e6dcf25e8803fc365a#commitcomment-12680658>
> .
>


You should not have changed this.


בחלק האחרון של השיעור הגדרנו מחלקה משלנו שיורשת מ-
ArrayAdapter
ומשכללת את המתודה getView שלו
ושם טיפלנו בתיבת טקסט ובתמונה.
יכול להיות שהגירסה האחרונה לא מעודכנת ב- github


Well, that's my fault.


btw: i've found one weird thing:

https://github.com/ace2nutzer/Samsung_STE_Kernel/blob/0600ecccfb12ef26592c6957ab0315eb17dad5af/arch/arm/mach-ux500/board-codina-mcde.c#L211

isn't weird ? xD it means WS2401 LCD .. but that sounds like S6D (problems with wakeup after suspend..) .. maybe someone did it messed up with S6D ?? :D ... i'm nearly sure this is the reason for Blackscreens on S6D :-)


TeamCity Subsonic :: Debug [Build 43](http://109.103.226.38:8512/viewLog.html?buildId=35005&buildTypeId=Subsonic_Debug) is now running


TeamCity Subsonic :: Debug [Build 43](http://109.103.226.38:8512/viewLog.html?buildId=35005&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:51


please see create figure


It was guaranteed to be equal in any case.


And with debuggex i found the real mistake..


Yes.


8格缩进！不要用tab，我强迫症啊


So, we won't support a string like `99999.9999`?


This excludes *.srcsrv files from ../src/Client/Client/bin/Releasex64


This excludes *.srcsrv files from ./src/Client/ClientExt/bin/Releasex64/


プラグインからMODXの変数に対して変更を行う方法で、 $modx->event->vars を使うとプラグイン実行がネストした際(プラグインの中で別のプラグインが呼び出された場合)に、データを破壊しあう問題があります。
ConfigModication API経由でデータの変更を行うとデータの破壊も行われないので、今後はこちらを使っていきたいと考えています。
例として次ように修正してみました↓
https://github.com/modxcms-jp/evolution-jp/commit/82fb0dee53a98b2922eddf5eac1543a2b7a52f38

そしてプラグイン内での値の変更方法はこんな感じです(tv_varsもやり方は一緒です)。

```php
$tmp= $modx->event->cm->getParam('doc_vars');
$tmp['introtext']= '新しいイントロテキスト';
$modx->event->cm->setParam('doc_vars',$tmp);
```

少し書き方が長くなってしまいますが…。
余談ですが別の利点として

```php
$tmp= $modx->event->cm->getParams();
```

と書くと$modx->invokeEvent()メソッドで渡しているパラメータ一覧を一気にとる事ができます。
プラグインにわたってくるパラメーターはイベントによってバラバラなので、こちらで確認できるようになっています。
あとsetParams()で一気に中身を書き換えて返すこともできます。


nueva linea


Should be all fixed now. Sorry for the nuisance!


Want a design comment which you wrote at https://gist.github.com/mururu/ae4496c19af0f24c275c#file-description-md


File name should be fluent/nano_time.rb following the ruby convention.


Does this not break this on older kernels?  I think you need to throw a detect on the version of the kernel source being used.


:) no, definitely my fault.


self-> is not proper Objective-C. http://stackoverflow.com/questions/9072688/dot-operator-and-arrow-operator-use-in-c-vs-objective-c


Committed this. I did change all the text() calls to attr() calls, since these strings are used in the html attributes.


Is this correct? Or maybe it's a better idea to have `Product`s even for `0` and `1` arity constructors?


Does this take care of ensuring that the cache isn't hit when making requests with different auth tokens?


なるほど、こっちのほうがいいですね


@cmosguy almost :) I created couple issues in your repo!


instead of adding new file,  can you update this  cloudstack/test/integration/smoke/test_routers.py in one test case N/W is getting restart , you need to add some verification steps.


(a little self-plug) If you have a chance, take a look at my little repo Graf.  I'd be interested in your thoughts.


 [Server thread/WARNING]: [PurePerms] Key mysql-settings not found in config.yml.

Im guessing you forgot something in config.yml


It looks like the alphaCodeType is called here:
https://github.com/robertogagliotta/openemr/blob/befef32ab15aab6ca1d5a589b710c6e8bf3ee00a/interface/forms/fee_sheet/new.php#L883

And the return value is already escaped here:
https://github.com/robertogagliotta/openemr/blob/befef32ab15aab6ca1d5a589b710c6e8bf3ee00a/interface/forms/fee_sheet/new.php#L885

So, agree, this item is already escaped in the proper place and can mark this as completed. So, no need for above code.


Unhandled Fundamental Failure


> Want a design comment which you wrote at https://gist.github.com/mururu/ae4496c19af0f24c275c#file-description-md

Okey. I will do it.


What ESLint really wants here is for the colons to be aligned.  By putting a newline in between you've faked it into thinking there are two groups of variables.  Just need two spaces after `boot2docker` and it will be happy.


Hola mundo


> File name should be fluent/nano_time.rb following the ruby convention.

repeatedly said that time.rb is better because we may want put some utilities for time on it in future. I think both ideas are fair enough.


The issue with your first line is that functions can not be within the output string or else it won't be processed(the text() call is in the string). Here is correct example for line 111:
``````
$sortlink[$i] = "<a href=\"patient_reminders.php?patient_id=" . attr($patient_id) . "&mode=" . attr($mode) . "&sortby=" . attr($sort[$i]) . "&sortorder=asc\" onclick=\"top.restoreSession()\">" .


This is very minor, but wouldn't the syntax make more sense from a readability perspective if it was

```
with gpkit.SignomialsEnabled():
          # do stuff
```

instead of
```
with gpkit.EnableSignomials():
```


Well, now the method does not change `peerNodeStatus` anymore but it still messes with `previousRoutingBackoffReasonRT`, `lastRoutingBackoffReasonRT`, `lastRoutingBackoffReasonBulk` and `previousRoutingBackoffReasonBulk`, and it changes the contents of `peers`. Is that still intended?


No tiene sentido este rename. El LoginController es para una view de Login. Tenías que renombrar LoginService!!!


In order to fix the problem, change this line to:
```javascript
}).then(function () {
    if (image) ...
}).catch(function (resp) {...
```


It is not intended. If do so, we may want format NanoTime as string like float too. umm...


2 comit


Hey Mike, I just updated my installment of the application and I am running into an error on the index page. Here's the error, let me know if you have questions/solutions

![image](https://cloud.githubusercontent.com/assets/11304929/9242857/22037884-413d-11e5-905f-971546d5624b.png)


Interesting. I already forgot about that one. Different projects, different fixes. Both seem to work fine, but I'll have to check that.

All this import stuff is overly complicated and buggy in my eyes – I never fully understand where and how to fix things. Sorry!


`Game` and `HidingSpot` was actually not exported from `database.js` hence their values are undefined.


IPython is installed with anaconda. IPython is only missing if you have Miniconda installed instead.


We should separate files in such future. Putting all stuff into one file is a bad habit of Fluentd. It is not rubyish cc: @repeatedly


lodash is not being used and hence, should be removed.


btw: check this shocking Value on STOCK GB Kernel:

https://github.com/ace2nutzer/Samsung_STE_Kernel/blob/2.6.35.7/arch/arm/mach-ux500/board-codina-mcde.c#L131

O_o


fixed (the 2 input version of add new task was from a different version)


Changed this note to ***(Note that this code currently shows duplicate data. No worries, we'll fix it later!)***


It's [Celine Dion](https://en.wikipedia.org/wiki/Celine_Dion), Mr. President.


Can you add some tests for this? Something like:

```python
@pytest.mark.windows
@pytest.mark.parametrize('directory, is_root', [
    ('C:\\foo', False),
    ('C:\\', True)
])
def test_is_root_windows(directory, is_root):
    assert filescheme.is_root(directory) == is_root

@pytest.mark.posix
@pytest.mark.parametrize('directory, is_root', [
    ('/foo', False),
    ('/', True)
])
def test_is_root_windows(directory, is_root):
    assert filescheme.is_root(directory) == is_root
```


This is the only way I found to change the grabbing cursor, by the way: ungrabbing and grabbing again with a different cursor. Feel free to suggest anything better that I might have missed.


`urlstring` seems to be a path now, not a full `file://`-URL, right? Then it'd be better to call it `path`.


That would indeed make more sense. I still think it's less straightforward and a typical use-case than what I described, so that would confuse most students of mine. Got no feedback yet, wait & see :monkey:


`PermissionError` is a subclass of `OSError`, so no need in catching it separately.


@flavour Does this looks good to merge?


I mean i added / to nbproject because I forgot it.


@clark800

The last commit also removes the ripple-wallet-generator module and uses generateWallet from ripple-keypairs for the api.generateWallet.

The flow type check seems to be crying about these changes:

```

/home/ubuntu/ripple-lib/build/flow/api/common/utils.js:8:16,41: index.js
Required module not found

/home/ubuntu/ripple-lib/build/flow/api/common/utils.js:31:25,31: parameter options
Missing annotation

/home/ubuntu/ripple-lib/build/flow/api/common/utils.js:31:33,33: return
Missing annotation

Found 3 errors
```


Unhandled Fundamental Failure


It seems many of these changes are unrelated to your PR.


Ignoring the fact that the environment is already *badly* polluted by pulling in Bullet and OpenAL, what else am I polluting? I'm not seeing any error in having the same flags project-wide.


97204612_MultiSpeedFan (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1805 of 1805 tests passed)

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


i thought this test and the following test seemed dup ones. but they are not quite the same, the first one will likely to form the quorum with only one monitor, as it waits long enough to run "`ceph -s`", and **then** the quorum is joined by the mon.b, and mon.a will .

but the second one mimic the scenario where two monitors starts in a row. and both of them are likely to send out the probe message, and to start the election after finding the other. and mon.a will always be acked as mon.b's rank is not lower than that of mon.a.


use return $.getJson(..)
new var not required..


Perhaps we should not have this plugin in the first release since its considered experimental.


Base has no symbol show_error...?


this will cause multi-dimensional array..
this.items = data.tableData;


compiles just fone for me on VS2013
try adding this line to replace the other one

ObjectGuid(ObjectGuid&) = default;

instead of ObjectGuid(ObjectGuid&&) = default;


not required to set everytime you call get BAD !
getItems should return whatever items are already there..

get should only get
get should only set


consider not making whitespace changes as it makes it easier to diff.


It would be nice if sjcl had npm packages similar to lodash. I noticed recently some lib depended on lodash.x, lodash.y. Very modular


After seeing you JSON. object wrapping not required.
let it be just [
{...},
{...},
{...}
]


コミットメッセージにはコミットの要約を書いてください


We actually don't need to check for nptr->server->all_registered here.
If this is local client that wasn't registered yet - we will fail here and discover that we are not finished in lines 228-247.


Reactor craft will not have a proper uranium source (I havent seen a single pitcheblende ore in the world in nab, 100chunks of quarrying with no result), Reactorcraft requires uranium for everything...


Could very easily create an sjcl-codec module though?


left class should only position / align left
use namespace right..


For stuff like this you should be using a StringBuilder, much cleaner.
When you need the output, it's stringbuildervar.toString()


好吧


Can we do this? Everything is _DEPRECATED now but maybe `internals` would better describe all the inner turmoil :)


I also forgot to mention: you should not make the List a global variable. Horribly inefficient


Same as GET.  Something like this:
```
/ssh:lxc32: #$ curl  -X DELETE -D /dev/stdout http://127.0.0.1:9001/api/12.03/instances/i-jakoadd/password
HTTP/1.1 404 Not Found
Date: Wed, 12 Aug 2015 10:40:16 GMT
Status: 404 Not Found
Connection: close
Content-Type: application/json;charset=utf-8
X-Cascade: pass
Content-Length: 92
X-Content-Type-Options: nosniff
Vary: Origin
X-VDC-Request-ID: 03d2f780a630ddcf17eeba1d7618c95b749f3552

{"error":"Dcmgr::Endpoints::Errors::UnknownUUIDResource","message":"i-jakoadd","code":"100"}
```


This is the only way I found to change the grabbing cursor, by the way: ungrabbing and grabbing again with a different cursor. Feel free to suggest anything better that I might have missed.


That is true. I found this problem with fence buffer. RM-server may point us to some middle-buffer space.
In this case free(data) won't be a correct operation. As a quick solution we indeed should copy it to the new buffer.
As longer term solution we may think about providing pmix thread ability (a callback) to tell the server that it is OK to free the buffer. In this case we'll reduce copy'ing overhead.


I have no idea, lol. Sorry.

On Wed, Aug 12, 2015 at 9:42 PM, Samuel E. Giddins
<notifications@github.com> wrote:

> Does this take care of ensuring that the cache isn't hit when making requests with different auth tokens?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/indirect/patronus/commit/84b6135cf9d38426b1bda5f2acd88676f5212284#commitcomment-12680838


Thanks :+1:


Unhandled Fundamental Failure


I don't understand the effect of this option. From what I understand SO_REUSEPORT allows only processes of the same UID to share a port. How does it work with the kernel? Any kernel user will be allowed to share? Isn't that too broad?


ここのインデントはそのままで良かったのでは？


Thanks @geofffranks :)


`-INT_MIN` is not an issue here; `255w : word8` is -1, not -128.


How does dcmgr write a message to dcmgr.log?


Perhaps, you should revert this change.


Gotcha! Register what needs to be done and when with the timer and then the timer can batch them up appropriately.


its good add it


Ahh, indeed this is nicer and more robust way. Will use it in future.


您好，我现在有点忙，有时间我帮您看一下。

在 2015-08-13 11:33:59，"dreamgogo" <notifications@github.com> 写道：

环境如下：
ubuntu15.04
virtualenv
mysql-server, mysql-client
virtualenv --python=python3 testenv
使用source激活testenv后，安装django，得到最新版1.8.3（新建一个标准django项目数据库使用mysql，运行runserver正常）
在mysql里创建mis数据，使用source命令导入SQL/mis.sql，没有报错
进入django-erp文件夹，执行python manage.py runserver，无法正常启动服务器。
请指点下是怎么回事，谢谢！

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


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1788 of 1796 tests passed)

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


This looks like something that belongs in ib_user_verbs.h


Did you forget `-Wall` or are there warnings when compiling?


This is identical to ib_uverbs_global_route, except for the dgid and the reserved byte (which would still be here as padding). I think it would be better to modify the original.


ほんとですね！　直しました。


Наверное, все же `width`, не?


Anjali, could you please give me the references of your Question 1 and 4?


This is identical to ib_uverbs_send_wr.


必須を外すことはあまり無いと思うので、presence: true　を外す修正に疑問を持ちました。

コミット内になぜ必須を外すのかが分かる箇所が欲しいかもです。
（例えばmigrationでnot null制約を外したとか）


I think you should merge as much as possible of this file with ib_user_verbs.h


RoomAirflowNetwork (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1798 of 1798 tests passed)

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


You're right, that will skew the first few times through, but it will skew all of the connections equally if they all join at the same time.  After about 10 seconds (with one round per second), I'd expect the startup effect to be swamped by the calculated values.

Are you still seeing slow growth or caches being forced to use too-small sizes?


Thanks for having fixed this.


Looks good, including the new theme. I see you haven't created a release yet, but since that's imminent, I'll go ahead and make the changes to the wiki.

You're welcome to add me as a collaborator if you wish but I can't promise to contribute much more. Being fairly new to Github, I'm not 100% sure what that means, either, but I'm sure it can't hurt. :smile:


You try already with update 5?


iiuc, there might be explicit bounds too, which this function does not return, so "full set" is a little misleading, no?


Does WQEs use this struct or the send_wr struct above? If one of them is obsolete please remove it.


The tool does work now. Thanks!


感谢你的回复，请加个qq1501816519吧，方便向您学习，有项目合作的话也方便沟通，谢谢！


He's using the generic `find_by_uuid` helper method. A lot of validation for UUIDs is done there. https://github.com/axsh/wakame-vdc/blob/86e037a9fe82df26b15b5c5fd673be16eca910a3/dcmgr/lib/dcmgr/endpoints/12.03/core_api.rb#L58-L74

It raises errors that are defined in `errors.rb` and inherit from `APIError`. That means they will be caught, logged to dcmgr.log and then outputted as the response @triggers showed in his last comment. https://github.com/axsh/wakame-vdc/blob/86e037a9fe82df26b15b5c5fd673be16eca910a3/dcmgr/lib/sinatra/dcmgr_api_setup.rb#L81-L85


move these two lines to specific styling


Could you explain why the userspace driver needs to know about all these details (first_psn, last_psn, ssn, dma.cur_sge...)?


Agreed!


d209f4f should keep the flags local.


Line 233:
```
$dsl_deprecation_wrapper = $compiled if $symbol eq 'dsl';
```
should be removed too; the same logic is contained within the `if` block in this commit.


The way i was able to fix this was remove one of the &

ObjectGuid(ObjectGuid&) = default;

ObjectGuid& operator=(ObjectGuid const&) = default;
ObjectGuid& operator=(ObjectGuid&&) = default;

and i compiled it without errors and removed the error


This misses double backticks after  ``method='lm'``, which is why the refguide check is failing.


How often do you actually see the cast listed? Got any specific shows I can test this against?


these don't need the blank line above them and need to be indented


I am quite sure that Maven Javadoc plugin will complain when releasing that the method does not have a description for its Java doc comment.


指摘ありがとうございます。コミットメッセージを追加しました。


What is button "Remove Storage"? No this button in test steps.


@Optimusprimeums tags are up :)


@markjcrane I think it valid for group intercept.
At least it should be optional.


should i update to 5 to remove this error ?


No button "Remove Storage" in steps, is it "Delete Cookie" button?


Please make compatibility for WatchOS 2 and WatchOS deployment_target in Pod file

eg. http://blog.cocoapods.org/CocoaPods-0.38/


This case is about "reopen", not "rebooting"?


I don't think this is needed since I don't see a case where this will return 404.


I have ever seen some users ask how to parse a time field like `99999.99999`. It seems some users have such log files. It is better to support such format.


It seems that tempnam is not recommended to use.
Can I use tmpfile? It is part of C99.

Could you tell me more about "MinGW has mkstemp(), but it deletes a tempfile automatically when the fd
is closed. This behavior is not expected for us."?
Removing a temporary file when it is closed is not so abnourmal.
Our expectation may be wrong.

Utilizing AppVoyer has high priority. However, this is a related to security issue. So let's take time more for this issue.


Explain ICS or link to wiki


Agreed. Fixed in https://github.com/computationalevolutionarydynamics/repeatedmistakes/commit/8adee59c0b228ea7f16091aa237f8aa1ee7015f0


This comment is on an old commit:
Here's the current code / clearer docstring:
https://github.com/airbnb/airflow/blob/master/airflow/operators/sensors.py#L378


这里使用单引号会造成内容无法解析，还是需要使用双引号。


I did create a new release, I just forgot to publish it to npm :P


Well that makes sense now. I did wonder if it was meant to close when removing a serie :P


this is needed for the service loader, I don't understand the question


Input key which needs removed in "Delete Record" area, click button "Delete"

Do we need this in last step? Or i think the data should be lost.


string = 'Developed parallel implementation for existing Ray Tracer engine'
string_list = string.split()
print(string_list)
string_list.sort(key = lambda s: len(s))
print(string_list)
print(string_list[-2])


A few things come into mind:

* How can we ensure that *all* implementations would respect the naming?
* How about renaming the method to `asMap()` or `toMap()`?
* Java doc should contain that it is the responsibility of the implementation class. It is possible that one implementation chooses to implement the method in such a way that it allows modifications. But the user should be warned that they might also get exceptions.


Just prints the second last string in the sorted list. Can be modified to return the second largest string accordingly.


Errr and I forgot to push the tag... I just updated to Windows 10 but I had to reinstall everything, so now I need to add my git aliases back.


Input key which needs removed in "Delete Record" area, click button "Delete"

Do we need this in last step? Or i think the data should be lost.

Please fix such issues in all other test cases.


I'm feeling this line lacks formatting.


Please remove this.


pagination is not working properly. Can you suggest something to make it work?

If I use `object_list` instead of `upcoming_meetups` in for loop than it will work properly.


I think it's probably simpler to just implement --delay -- the min/max thing is something I thought was useful, but turns out that it's more often annoying than useful.


I took it from draper's README. I seems it's not up to date.


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1793 of 1798 tests passed)

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


972124-2NewDetailedIceStorage (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1275 of 1275 tests passed)

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


Do we assign null to objects?


thanks =)


Use `e.strerror` here?


nice


Please put a meaningful message when throwing DeviceManagementException.


It should be loaded via `Bundler.require(*Rails.groups)`, but It wasn't working until I added this line.


Ok, I'll move it.


The profile tool works on 2.10 but not on Master (at least in the Debian 8 VM):

Traceback (most recent call last):
  File "/home/web/.qgis2/python/plugins/qgepplugin/ui/qgepplotsvgwidget.py", line 98, in initJs
    self.frame.addToJavaScriptWindowObject("i18n", self.jsTranslator)
RuntimeError: super-class __init__() of type QgepJsTranslator was never called


読まれるかわからなけど。。。
app/views/battles/new.html.erbに関して
form内にformを入れてしまうとsubmitした時に違うformが送信されてしまうので要注意


Avoid log and throw unless specifically needed


For Mangos

2739In file included from /home/travis/build/CoronaCore/OneServer/src/game/LuaEngine/LuaFunctions.cpp:23:0:

2740/home/travis/build/CoronaCore/OneServer/src/game/LuaEngine/UnitMethods.h: In function ‘int LuaUnit::IsCasting(Eluna*, lua_State*, Unit*)’:

2741/home/travis/build/CoronaCore/OneServer/src/game/LuaEngine/UnitMethods.h:551:43: error: ‘UNIT_STATE_CASTING’ was not declared in this scope

2742         Eluna::Push(L, unit->hasUnitState(UNIT_STATE_CASTING));

2743                                           ^

2744In file included from /home/travis/build/CoronaCore/OneServer/src/game/LuaEngine/LuaFunctions.cpp:28:0:

2745/home/travis/build/CoronaCore/OneServer/src/game/LuaEngine/GameObjectMethods.h: In function ‘int LuaGameObject::RemoveFromWorld(Eluna*, lua_State*, GameObject*)’:

2746/home/travis/build/CoronaCore/OneServer/src/game/LuaEngine/GameObjectMethods.h:250:36: error: ‘class GameObject’ has no member named ‘GetOwnerGUID’

2747         ObjectGuid ownerGuid = go->GetOwnerGUID();


Okay, there is already such a structure defined :) Yeah then take it instead of argument pointer approach.


Incorrect spacing after extras.


Incorrect spacing after resources.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 5](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=58&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


Remove code if don't use


My bad. I did not get the point.


If you are not implementing RD, I don't know if this is needed. Other than that +1.


客户端与服务器端都要遵循的规范，【交通规则】
1xx:请求没有送到服务器 “在路上”
2XX：成功
3XX缓存状态
4XX客户端错
5XX服务器端错


Wrong indentation for the if block.


Need 2 bank line (Pep8)


done.


Add the pullstop to comments.


test 1


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 5](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=58&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:02:04


@Optimusprimeums tags are up for when you're ready mate :)


It works just the same, here is the info object with the Class type
```AS3
override    public function info():Object
    {
        if (!_info)
        {
            _info = {
            compiledLocales: [ "en_US" ],
            currentDomain: ApplicationDomain.currentDomain,
            preloader: spark.preloaders.SplashScreen,
            rootClassName: "FeathersTest",
            themeClassName: "feathers.themes.MetalWorksMobileTheme"
            }
        }
        return _info;
    }
```
http://forum.starling-framework.org/topic/feathgerssdk-themes#post-83310


Then I would prefer the other way around: Removing the "32" prefixes in the macros?

````C
#define TARGET_SYSTEM _WINDOWS_
#define DEV_SYSTEM    _DEV_WINDOWS_
````

Hope this doesn't mess up the commit due to lots of changes!


equivalent to
```
session[:return_to} = request.referer
```


Can't we use column names instead of column indexes to fetch data?


Thank you for imformation. I will do it.


仔细读了一下发现重复使用两个”一个“！！


Also your include files.


Wrong indentation


Yes, exactly. The assembler threw errors and I couldn't figure out the correct constraints for the register allocator. So I used fixed regs instead.


I think a more specific website would be appropriate. How about the [wiki](https://github.com/SoftRoCE/rxe-dev/wiki/rxe-dev:-Home)?


I should have put this in a separate commit ..


Looks good. :+1:


Uh wtf?


ah


Yes, i forgot to delete it after i was done porting it.


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 3](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=59&buildTypeId=Xbim_XbimScripting_XbimScripting) is now running


When modifying files, don't use indentations (unless the original file does) or else the spacing does not line up.


wrong indentation


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 3](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=59&buildTypeId=Xbim_XbimScripting_XbimScripting) outcome was **SUCCESS**
Summary: Running Build time: 00:00:24


it's a rename from dat`a`formatter to dat`e`formatter


Incorrect spacing after getResources()


Put a meaningful variable name for temp


Iusually  do this to have correct result when start is after end. I didn't know about the undefined behaviour. In this function it doesn't make sense.


You appear to have deleted Follower.scala on accident


ok i see - this script is a mess, this would be better:

while [ $timeout -gt 0 ]; do
  grep ^"fe80.* $iface"$ /proc/net/if_inet6 && return
  timeout=$(( timeout - 1 ))
done

i will merge anyway. for hotplug make a file
/etc/hotplug.d/iface/50-alfred

with something like this

case "$ACTION" in
  ifup)
     alfred.init "$IFACE"
  ;;
esac

and in the alfred-init file you only allow execution if iface if up on first check (no 30 sec timeout)


start . sign should be moved up to insist continuation


would be more on the front end gui to label this:
"Administration Details" to avoid confusion with the Notes field.


start . sign should be moved up to insist method continuation


Why are you merging?


next_of_kin seems odd here. Is this to store the responsible_party ?


We can do better than this of course


Why are you adding another cvx code entry?


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1797 of 1797 tests passed)

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


The probably_infront part is a hack, yes. It is there because I am not entirely sure thate the stm32 hardware guarantees that the DCMI interrupt always happens after the DMA interrupt. I guess if the DMA is delayed because some other bus activity the interrupt is delayed as well. What do you think?


Can I get more details on what the following is supposed to mean:
"Disease with presumed immunity, when no vaccine administered (SNOMED-CT Code)"


Commit message should read: “Don’t trigger editor.setText if fixString doesn’t change the text”


Maybe it would be better to have a union of pointers to the specific header types.


Could you remove the inline comment and move this memory handling code to a protected method with an explanation of what it does placed inside a docblock?


Isn't "Dose" and "Amount Administered" the same thing?


Many of the above fields are already in the header. Why are they needed again here?


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Done. https://github.com/mururu/fluentd/commit/cee5af466a2cfd5e6e55daf4026581df81e7ee25


Too many changes for one commit


Shouldn't this be completion_status to be more clear. It's also odd that need a "Completed" vs "Not Completed" status. Can't that be inferred from the entry itself? Or is there something I am missing?


You can pass anything to PKT_TO_SKB, not just rxe_pkt_info structs. It would be safer to use container_of.


Oops, I'm sorry. Shouldn't Travis have caught that?


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 3](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=60&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


Change git:// to https://


Unfortunately, no - the errstr for 2 is "file/folder not found" - which is very cryptic for some botmins who don't realise its actually the pem file that is missing/broken and not the module


You did not add the needed target restriction. Now you can counter every spell.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 3](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=60&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **SUCCESS**
Summary: Running Build time: 00:01:04


This is probably a task to copy to the TODO file.


The CLR is absolutely fine and has proper tail calls (and the jit is
fixed).

The issue is the C# compiler. IIRC unfortunately only the x64 version emits
tail calls, and only in some cases. So yes it's probably a good change.
On Aug 13, 2015 1:34 AM, "Phil Haack" <notifications@github.com> wrote:

> No, just a static analysis recommendation. As far as I know, the CLR
> doesn't do proper tail recursion optimization on its own, does it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/octokit/octokit.net/commit/0811d4cfd3b41077712cd8c8561a7d4243d7b218#commitcomment-12677992>
> .
>


for basic analyzer type define, no need to add macro if/esel check
too many #if #else make code hard to read.


Desyncing happens when the DMA interrupt deadline is missed which is about 300 to 400us. Mainly it is missed because of the I2C transfers which are done in the interrupt context. However I included the MAX LATENCY flag to test the maximum latency induced by i2c. It never reaches the deadline. I conclude that sometimes it hangs.


The new files aren't the problem (yet, much).  Crawling thought the maze of #ifdefs and the git diff is difficult since the gkt3 branch doesn't have the latest  world.h macros or the changes to Shoes that I've made recently.  Also I'm not the fastest runner in the race. That said, it's going pretty mostly OK but very slowly and I expect there will be errors - mostly in the alert/ask/confirm but they could be anywhere.

I  kind of wish I had created gtk3.c w/o the #ifdefs although that would have trouble too. But, I didn't so we soldier on.


The ID is not taken from the database, so you  can reference `study.id` all you want. Also, if `study` is already instantiated (since you can reference the id), why is it re-instantiated below instead of just being passed?


I said it on gbatemp, but I didn't update the credits yet. For the license, there was the same license before commited the n3ds mset. I just readded it. What do you say @17210 ?


OK JDK 6 doesn't behave that way. It only throws on the second attempt, as I had guessed.

By experimentation, it seems that JDK 8, when inserting the first element in a TreeSet, compares it *with itself* to "force" the potential NPE (or, I guess, any other exception) that would be thrown by the Comparator (either natural or provided) on whatever data it is given.

So then, if you want to maintain the NPE (well, in fact, a JavaScript `TypeError` in most cases), I would suggest doing the same, to sort of align with JDK 8's behavior:
```scala
  override def add(e: E): Boolean = {
    val box = Box(e)
    /* When the set is empty, we force one comparison so that
     * possible exceptions thrown by the comparator due to invalid
     * input data are thrown early (such as NPE/TypeError).
     */
    if (isEmpty)
      BoxOrdering.compare(box, box)
    inner.add(Box(e))
  }
```
(unlike your solution, this also supports *provided* comparators that do not support null values)

> so basically all the tests on collections that expects NullPointerException are not useful?

In theory, no, they are not useful. Or they can be useful but they should expect *any* exception rather than specifically an NPE. Typically you get a JavaScript `TypeError` instead, for null dereferencing, for example. That said, when it is simple to do so, we should try to "crash early", so that bugs in user code can be detected more easily. So in practice they are useful to assert early errors are thrown.


this is mine done;


rxe_opcode isn't seem to be defined in this patch. I think it would be better to either put it in this patch, or reorder the patches so it exists before this one.


Why always beginning with this


where is the definition of `emscripten_print_double`?


Wait


Invalid -> Invalidate


I'm not sure if we have to import this external package only for calling StringUtil.isBlank(getDescription()) method?


This loop is missing timeout decrement, I'll fix it locally.


![screen shot 2015-08-13 at 11 51 54 am](https://cloud.githubusercontent.com/assets/8743597/9244032/f4106522-41b4-11e5-9881-edd095209f42.png)


Hm. Doesn't this kind-of-almost result in a cyclic dependency between `wibox` and `awful`? I'd like to keep awful out of wibox. (Yes, this isn't really a cycle dependency, but it's relatively close to one)


if you want an more up to date SM toolchain just tell me


these codes are just in case


Isn't be better to reuse RuleStatusInfo objects instead of create a new one every time when we change the status?


fine by me


UTF8 ;)


Thank you very much for this. I have fixed the problem: https://github.com/tonytomov/jqGrid/commit/9ff1c77afecfff025ebf211f08337e36f97d3bc0

Kind Regards,
Tony


@kergoth, "pulseaudio-module-switch-on-port-available" is not added to  meta-mel/recipes-multimedia/pulseaudio/pulseaudio_%.bbappend. Is this module intentionally dropped or is it coming from elsewhere?


Thank you @PareshMayani
Which ones did you review? If you can review some of the small PRs also, it would be really nice. Thank you.


Had discussion with Josef. Let's leave it as it is now! If we need to modify it, we do it later!


if _stats has the same type and same life-cycle with DynamicAnalyzer's stats.  could you add a protected function from base class?
 e.g
SmartPtr <X3aStats> get_cur_stats() const { return _cur_stats; }


did you restart app?


In the stack **h** is currently used as the prefix for the _HANDLE_ type. I will use the same.


@tasomaniac I did but not able to complete due to the morning meeting, would do it EOD today!


echo "1234";


This needs a `cr.operator = cairo.Operator.OVER` before the paint, just to be sure. I don't think anything would produce sane results with some other operator, but might still be good to be sure.


Those are all pretty good rules, but they are also straight forward. I wonder if we really need to write them down.


Thanks. Fixed and pushed: https://github.com/PerlDancer/Dancer2/pull/976


Just tested with adding only Update 5 to VS2013. No progress on commit 57624e0d53, unfortunately.
6.x source compiles OK with `ObjectGuid(ObjectGuid&)` but not with `ObjectGuid(ObjectGuid&&)`.


where?


This approach mandates that a {{CapabilityMapping}} is instantiated after the {{ResourceTypes}} that references it. Is that always the case or would it be beneficial to be able to connect the dots once all capability mappings and resource types have been instantiated?


This is necessary because the hook actions themselves don't have any inherent context, so for example if in our theme we do a context check and then call remove_action() on one of the default pieces of content being output, that remove_action will also affect all other contexts. By manually resetting the hooks after each run through the shortcode and forcing all action registration to happen on a single mm_posts_register_hooks action, we can now safely modify actions for specific contexts.


Is it guaranteed that {{o.resource}} is always capitalized here?


Maybe ```The template is based on **Pundit policies** and you will need to create them``` ?


*use


This was done based on the input of @Lorenzmeier. We have been using this for the last 2 months and havent noticed any problems so far.


Summary also addresses #77


you are right, I will remove it.


I means `strerror` for the "else" case.


I think this could benefit from some comments explaining these bits. The fact that they define what goes in the pkt's mask field, and what each of them means.


I was mostly referring to the 180 MHz as per data sheet. Wether 192 is manageable long-term we have to qualify somehow.


The method signature is imho much nicer to have just the one parameter. Since rules do not change their status that often, I doubt that reusing brings any noticeable performance or memory benefit.


Side note, this commit marks the first Minimum Viable product for this project.


I will try to use `__get` for lazy init of the tokens' properties in the next step, so I can get rid of all the duplicated invocations.


Ah noted - that would be useful, although these two are the most common errors I've seen so far. Will add that in :) Thanks


Why are you asking him? I'm one of the author. And I'm telling you no.


Structuring the app part -1


Yes, the thread synchronize the access to the buffer through `instance_l.semUserData` semaphore and ta mutex locking.


@amanpwl92 revert it back to R.* if package name is giving correctly in gradle file eg:         applicationId "com.mobicomkit.sample"
then it will work with just R.*


This is not a valid operator< for a map. A map requires (a<b) && (b<c) => (a<c). Right now, let's assume (a!=b) && (a=c), so both (a<b) and (b<c) will be true, but not (a<c).


@AlbertoSONIC You should have had contact with the authors of ReiNAND and asked them to license under something compatible with GPL.

Anyway, The source code of ReiNAND looks nice...


I guess this can be reduced again, since the overclocking is unnecessary anyway.


ok fine


Looks fine - @rpastrana you might want to open a JIra for that so it doesn't get forgotten about.


@Ravnsara this should solve the sidebar issue. let me know if not.


> Ignoring the fact that the environment is already badly polluted by pulling in Bullet and OpenAL, what else am I polluting? I'm not seeing any error in having the same flags project-wide.

What are you talking about? These targets don't have a project-wide environment in a first place. And if they would have such a thing it should be considered as a bug.


Don't forget to create a file named author.txt in the "..\mage\Utils\" directory in which you only write your username. This will be used for @author in the generated file.


Concernant la pagination: Tu parlais de FormParam, mais il faut aussi envoyer l'ItemSearch dans la requete. Donc on a soit les pathParam (/search/{offset}/{limit}/{sort}), soit les query param (/serach?offset=0&limit=10), soit les matric param (/search;offset=0;limit=10:).

J'avais pris le dernier parce que j'imaginaais qu'on puisse plus tard faire /item/search;limit=10/picture ou quelque chose comme ça, mais en y repensant ça n'a pas beaucoup de sens.

Ici je perse juste les query parameters à la place des path paramaters et je check pour null.


ItemRef->ItemId : jersey se plaignait de ne pas pouvoir Unnmarchmaller Itemref, peut-être qu'il y avait un filtre que tu voulais  mettre en place?


don't use AssertionError exception for this, they can be globally disabled by a interpreter option.


ทดสอบๆ


Now most of the functions are working fine. `ReportEvent` is now reporting to `Unlocker` `report!`
Can you give me some explanation about `@watchers` & once a report comes to `report!` what should I do with `@watchers`??


J'utilise la référence ItemPicture->Item ici, d'où les @NonNull rajoutés


ไรว้ะะะ


I'm not sure about that. Yes the signature is nicer, but the Rule will change its status twice on each rule execution (IDLE->RUNNING and RUNNING->IDLE). If we have multiple rules and rules which are executed very often the new status object will increase memory usage.


Adding two empty arrays to all types might be a bit expensive. An alternative would be to not expose the arrays publicly and instead add methods add_produces/add_consumes to add entries. They could create the arrays on demand.

Methods to access the arrays could return a frozen copy of the created array or a constant EMPTY_ARRAY (also be frozen). Would require an extra step to freeze things once all produces/consumes had been added (perhaps the same step that would "connect the dots" in my comment above?)


Haa, AAAA again!


Interesting :+1:


It looks to me this view can't be used on its own. If so, why is it not a mixin instead?


Output buffer as hex byte sequence.


these 4 lines are a bit strange.
How about not giving error as default value and just catching the exception / reraising whatever you want?

In any case, you do not want value equality (==) in line 34, but object identity (is).


After changing the upgraded files, I keep getting the undefined index notice. First on line 379, after applying your changes on line 325:
Notice: Undefined index: page in /wp-content/plugins/fv-wordpress-flowplayer/controller/backend.php on line 325


This can just be `create`.


La función MessageManager::send_message envia un mensaje al inbox de chamilo y tambien un email.
No es necesario agregar api_mail_html


comment?


commit comment


This still has the same spelling mistake.


please remove this


and this


should be `#EDecEqui#`


How dis we replaced the deleted close brace here? This bracks the build


trouble_offset and trouble_location are redundant, please only use one of them


@b1l1s Because he's the one who did put the license in every rxtools file.


OK. I did not know, that the DWT was an optional feature of the cortex-m cores.


hey @pamval,
I have updated the xml file as per XSD file and also modified the lib folder for the plugin with only required libs. And also made a new zip which give only plugin folder when you extract it.

Please review these changes and let me know if there are any other changes to be made.


The `day_values` is still here.


@jimdowling can you please use the same code formatting for javascript and html as you use for java (indent 2, tab 2)? indentation width is very wide here and we keep overwriting each others formatted code all the time.


This one has also not been changed yet.


@jprichardson ACK?  Using standard now.


it is impossible to come on servers with build 8


Why not just `code << "\\\""` ?


@m-g-sonar EOF new line


@sirmordred For bionic we will use the one from CM and then we will push the commits for Linker..
The repo from ADC is outdated


I suspect this should be "Only useful for Google Maps for Work clients." instead.


If you'd use a CollectionView, you wouldn't need to create child views yourself.


Thanks Allan!


I agree with that change but we should implement it in a way the user has control over it because there can be two kinds of scenarious:

* I *want to* override *all* images because of a change in the quality settings for example
* I **dont** want to override all images but process new images that where somehow added in a batch, maybe via FTP upload

You agree?


+1
I haven't checked every opcode, but the idea looks okay.


And you contacted all contributors to rxTools and they permitted that? ROFL.

Just do this for me, remove the copyright notice and the GPL license from this one file. I don't care about the rest. Add a note that the file was originally written by zoogie with me and Reisyukaku as additional authors.