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


There is nothing that has EventTarget on the right-hand side of "implements".


Can you make it

```
log.Debug("my validating project - ", string(res.Body[:]))
```

so that we can see the error message ?


Different global


do we need to worry about field name conflicts (one overriding the other)?


Yo yo yo


Is the registration timing independent? Meaning can it run before or after the external storage is configure? We should include this in the doc


I could have sworn there was something. But okay, that would make that easier.


Just make an account/login with github - https://packagist.org/packages/submit and set up a package and put in your Repo.
Should work automatically when you push, assuming you're tagging with semver.


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


In the Network Entities table, what do you mean with 'w/Controller' is that VDSM controller like iproute2 or OVS controller?


Hi,

I'm sorry that I'm new to PR, could you explain what do you mean main repo?
I'll make to a PR to that as soon as I understand the meaning.

Thanks.

Chin-Yu Lin
fuyushimoya@gmail.com
Phone: 886-918-763022

2016-03-03 1:37 GMT+09:00 Nako Sung <notifications@github.com>:

> Could you make a PR for main repo?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fuyushimoya/Unreal.js/commit/fd1debe23d39a91dd364ab0cda3f746726c1a420#commitcomment-16450367>
> .
>


??? Why is status stealth needed when attacking a samurai?  Or kills?


Obviously I don't really understand staging commits. The commit message is only intended for the "models.py" changes.

Every other change (besides the requirements/local.txt change) was used to establish more shift archive view functionality.


i've found it already


The last 2 paragraphs only concern Manager Server, thus they should move ahead of the section that concerns Manager Proxy.


FYI, I am also going to move root `checkstyle` folder into 'src`


I thought these tests were a duplicate of the ones that are now in `tests/test-refs.sh` that is why I dropped them.


If this needs Qt5 to build the imaging tools someone will need to change:
```ruby
depends_on "qt" if build.with? "imaging"
```
To:
```ruby
depends_on "qt5" if build.with? "imaging"
```
It perhaps speaks to the popularity of the option if nobody has reported this in nearly three months though :confused:.


@nghiant2710 is this supposed to be hardcoded to `jessie` ? shouldnt it use the `$suite`


we cant merge this one, it will break preempt mode and sync group


We want this to load through the settings page. Setting it directly like this will be a problem since this wont work if the openmrs is hosted on a different port or IP


I noticed that input files needs to on the same folder where the script is, otherwise the weights aren't detected. So, I will try to fix this before doing the pull-request if you agree. Thank you


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1014 of 1015 tests passed, 0 test warnings)

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


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


shouldn't this pass expect_modified=True?


You can make a PR against https://github.com/ncsoft/Unreal.js (this is a main repo for Unreal.js) :)


Why does pcl need conda's `gcc` as a dependency?


For what it is worth, this changed behavior where previously a user could set the `:template` option with an absolute path to a custom template.

Now a relative path is required.


This also gets the "universal" part of the menu working.


@imurchie nope, it's 9221


而Marvel 可以让你交互式 ，“而Marvel ”中间是不是少了一个空格


DivByZero is just how I roll.


ah ah, fair enough! ;-)


OMG, bien sûr…


test


I blame Python.


Would be too much to fix this inside this PR, though this needs a fix sometime. Because this behaviour is some sort of "multi-expansion". If I provide a glob I don't want that^^


I know how to spell "unforunately." :|


![simulator screen shot 2016 3 3 01 13 53](https://cloud.githubusercontent.com/assets/16911423/13468462/959ed76c-e0dd-11e5-80fe-037eab342b1c.png)
![simulator screen shot 2016 3 3 01 13 56](https://cloud.githubusercontent.com/assets/16911423/13468461/959ea918-e0dd-11e5-9dae-a2b686e52190.png)
![simulator screen shot 2016 3 3 01 13 58](https://cloud.githubusercontent.com/assets/16911423/13468463/959fe4cc-e0dd-11e5-9f2f-3eacc15ce170.png)
![simulator screen shot 2016 3 3 01 13 59](https://cloud.githubusercontent.com/assets/16911423/13468465/95a04822-e0dd-11e5-8ed2-99e2eabc204e.png)
![simulator screen shot 2016 3 3 01 14 00](https://cloud.githubusercontent.com/assets/16911423/13468466/95a1ac9e-e0dd-11e5-8b01-a73d3586c6bd.png)


Unfortunately it doesn't fix my chunk loading problem :\


oh i think i see - is it because last modified is only updated when a form is submitted?


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-MacOS-10.9-clang: OK (2121 of 2123 tests passed, 0 test warnings)

 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

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


j'aime !


it should be just `withVariadicBound`.
Renamed


I guess there are two ways to merge:

1. I send you pull requests

2. You merge into Cleanflight and I send Hydra pull requests

(For completeness there is 3: you send me pull requests, but I think your code is closer to being ready to ship - I haven't got the SPI stuff working yet, and I'm not really a hardware guy, so this probably is not the way to go.)

I don't really have a preference for either. (1) is more work for you, whereas (2) is more work for Hydra - I don't know who is busiest. Do you have a preference for either?

Also, it will probably take me a couple of days to get my head around your code so that I can send useful full requests. The things I'd like to get in are my framework for supporting multiple protocols, and also my implementation of the SymaX protocol.

By the way, have you done any work on getting SPI working on the CJMCU? That's next on my todo list and help would be appreciated.

As an aside, one other thing I've been working on is a sub-32K build of Cleanflight (indeed the NRF24 work is really a subproject of that). My investigations show that sub-32K is certainly feasible and it will open the way to putting Cleanflight on toy quads - something I'd really like to see.


We aim to please. :)


For implementing C behavior? :P


LGTM


Ok. It demonstrably is not, but oh well.


suspended: false?
o.O


I guess 130 is equivalent to incrementing 1289 by 1 because you're also rounding to three significant digits?

Wouldn't it be useful to instead actually use the build number?


Readme


I decided to keep this function here for now, rather than converting it to an `_initializeTo` overload in `CollectionType`.
I'm worried about having to get `count` twice, which would be a visible change for custom collection types, even if they do not provide (or know about) `_initializeTo`.


m3/m is 1000L/m or i am wrong.


:metal:


Was this added for testing purposes or really needed here?


should we add `sample-config.json` to the gruntfile so it gets published to haxelib?


:'D


@srisankethu  We should rather mention the circumstances in which a rebase is required. That'll help people understand rather than just typing some commands. The current explanation seems to be more simplified. :)


@ianwjhalliday @phistuck - good suggestion. I'll file an issue on this one.


Is this an OK name for this? I'm not loving it, but I think it's slightly better than the old.


use nopoll_log() for printf()'s


Will do.


Please do NOT commit reformats like that. It makes it impossible to figure out changes over time. Also note @dulanism ..


Since `doctrine/orm` requires `doctrine/dbal`, I just change this requirement rather than having both.


why not `DT[, "region", with=FALSE]`? no space on missing `i` arg, my proposal was to not put empty space in `i`. Of course unless Arun decided otherwise :)


@andreytech please, be sure, that array always exists, even it's empty ( to prevent errors ).

```
$d = $this->get( 'property_types', false );
// Always must be:
// is_array( $d ) === true
``


Why that / .. thats wrong!


@srisankethu  Mention about current branch. You do `git fetch upstream` to update your master. Now to do rebase you need to first checkout ( `git checkout <existing_branch> or git checkout -b <new_branch>` ) to some other branch and then do rebase on your current branch. :)


Adding the slash is pointless for this link


Is your proposal to pass `argumentType` and `resultType` as parameters to method implementing cast and compute scale factor each time method is called?

That would be more simple. We can try that. That will take some work though, as I do not thing we have decent performance tests for scalars right now.


The last remaining use of `evilBoundsError` is in line 1626 below.


no, that's not the way to do it, that would mean that reboots are ALWAYS suggested even when not necessary ;-)


저에게 왜이러세요 ;ㅅ;ㅠㅠ 이거 e31998d7da4e7ec91cf891e3725242006cad09c0 랑 같잖아요 ;ㅅ;ㅠㅠㅠ


Initialisation du projet fil rouge


test test test


this estava pegando o contexto do evento...


It would be nice if this cop was smart enough to emit `b <= 1` rather than `!(b > 1)`.


test 1


There should be NO empty line between anchor and title.. why did you add it?


Ok, give me one or day first to make some cleanup and also make it mergeable with trunk first.


renamed


For adding the remote I would refer to github or to our other guides, since this step is only performed once, not for every rebase^^


git checkout <existing branch> isn't needed.I do a rebase directly


@mosabua I am not sure why it did this.  I changed "Nexus Repositor Manager" to "Nexus Repository Manager" via the web interface.  The rest, as far as I can tell, github did itself.


Yes I was thinking about that


@andreytech Please, try to avoid committing 'dummy' changes. Thanks.


@tushar-rishav sorry I don't understand that, the commands seem right for me^^


Why did you make HtmlLauncher abstract?


@lclc98 When authors run the mod in their dev environment from source (like if they want to do a PR) the version of the mod will show as 0, because the build script doesn't change the source. I have decided to periodically update the source version every now and then, because it doesn't interfere with anything.


@srisankethu IMO a brief description about `push` and `push --force` would be good here.  As said before, our aim should be to help them understand rather than just mentioning some series of commands without actually explaining it. Not very extensive but a brief explanation would be perfect :)


Please don't merge, there are lots of mistakes


That should be very easy to make it work on the CJMCU. It's on my todo list as well ;)
The only "problem" I have is that I have a V1 CJMCU, with SPI connector rotated 90° compared to the V2 :/


Unfortunately, I just haven't the time to work on this right now (uni
just started up again). And to be brutally honest, this project is a
massive mess. SBPrime's idea of dependency management was to just throw
all of the modules into the repo with zero organization. It also doesn't
help that the included project files (ew, netbeans) are broken outside
of his filesystem. As it is, I would have to spend a whole day just to
get the current repo to build.

tl;dr, the project is a mess and it would probably be faster for me to
re-write it from scratch, which is just something I dont have the time
for right now. Sorry :(

On 2016-03-02 09:44, Max Lee wrote:
> Is there any chance this project will be continued? Because AWE
> doesn't seem to be compatible with latest WorldEdit versions.
>
> --
> Reply to this email directly or view it on GitHub [1].
>
> 	*
>
> Links:
> ------
> [1]
> https://github.com/thecalcium/AsyncWorldEdit/commit/0bd4b356c848ed06280af0dd6c93b81b61b6be0a#commitcomment-16447116


Added path to render file


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


Nevermind you my reasons!

*It's good to be Emir.*


Where do you use this method?


Will do in the next release.


My understanding is that [dylang/shortid](https://github.com/dylang/shortid/issues/60#issuecomment-190651009) doesn't just randomly generate different length strings. The strings start at the length of 7, and then grow, dynamically increasing their uniqueness (theoretically) as you generate increasing amounts of ID's. To most accurately accomplish this would mean making it stateful, and keeping track of the total number of ID's generated (across clusters?), but since that is not pragmatic for most use cases, it looks like this is accomplished by using the seconds calculation from the hard-coded base REDUCE_TIME.

If you were in the business of generating unique id's with a preference to keep them as short as possible (like if you were building a URL shortening service), then the idea of having a state, and increasing the length of the ID as you generate more of them to increase the uniqueness is a clever feature.

For the majority of us that just want a unique string-based ID for collections of records, then simply picking a standard size of id that offers reasonable uniqueness (without having to resort to UUID), then I think that just being able to set the size (from a parameter/environment variable) is actually the better solution.

What do you think?


such a nice app, would you love contributions?


If they are suspended that will be set to true, They will not get permissions and also will not be deleted from this YAML file


This is a prototype we have discussed with Vladimir, related to recent cordova-common changes regarding logging and events handling (https://issues.apache.org/jira/browse/CB-10052, https://issues.apache.org/jira/browse/CB-10176, https://issues.apache.org/jira/browse/CB-10430).


@mosabua when the lines were together I was still getting the .Listing: <etc>.


Is there a way to use 'elif' to make this statement cleaner?


what is this change for?


On Wed, 2016-03-02 at 09:11 -0800, Alexandre Cassen wrote:

> we cant merge this one, it will break preempt mode and sync group
>

OK, I certainly see that for sync groups now. Could you explain for me
what the issue is with preempt mode; is it to do with nopreempt being
specified?

I'll need to rework some of the main SNMP patch code to do with the
reason for transitioning to master now, so is it simplest if you reject
the pull request, and I'll submit a further one once I have it ready.


Oh yeah especially that it overwrites history!!


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2159 of 2161 tests passed, 0 test warnings)

 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

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


Can you use the BUFFER_PROCESSED_OK constant here?


Also, this mistake was admittedly *unforunate*.


Also the anchors were inconsistent (some on top, some on the bottom)


Error de sintaxis.


This should probably read "inBuffer length".


Actually I prefer the first one... personal taste, I guess...


Updated grunt in dev.


needs to be .modal-body > .list-group > .list-group-item


and #168 .


see gl_str https://github.com/lunixbochs/glshim/blob/unstable/src/util/gl_str.c which can print basically any GL constant


I don't remember :-) But most probably because of C+11 features present in PCL.


Ah, it was a stab in the dark before I had valgrind output.  I'll drop it.


Why did we add the else.  With it, user without bid will not see if a claim has been made.


If we fail here, we've messed up the whole frame. We should brake out of the loop, possibly clear 'data', and return failure.


This could be `static`.


@Makman2 Which part? Umm shouldn't we work on a separate branch? :)


This could also be `static`.


__API Description file syntax error on line 14__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


Then something else is wrong.. check the asciidoc cheatsheet or doc or other usage in the docs but dont introduce this inconsistent usage just in one chapter.


Yes, thanks for removing this. :+1:


@Makman2  Exactly!


done


That might be up for an update now...


That sucks then... well.. too late now. Better not to use the web interface I guess.


@nimaje from [#117](https://github.com/phalt/pokeapi/pull/117):
> @nimaje broken link in the table of content (was referring to e instead of é).


:+1:  :+1: :+1:


The following doc(s) have been deployed to the [1119wpsca](http://1119wpsca-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/horizontal-scalability](http://1119wpsca-static-docs.pantheon.io/docs/articles/sites/horizontal-scalability)


Nicola, Thanks for these links. We'll be sure to review them and take this feedback under consideration.


Right, once a version has been released, I think any subsequent changes should be counted in the next version. And if changes are important, then that next version can be released immediately, like you said, as a 1.1.1. But in this case, many changes have accrued, so it should probably be 1.2.0.


Oops.  Sure thing.


Hmm; I see your style, but I think it reads better my way. The reason is that the following `.map` gets indented further to the left (exactly where `return` is in the original), so that the stage it modifies is very far away. It looks (and reads) like `.map` is modifying the `function(()->...)`.  This is exacerbated by the closing brackets all going on the `.map(...)` line.
There is also creeping cramp relief.

Of course I'll accede, but under (mild) protest.


I'm referring to the `console.log` statement - do we need it?


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


This works for us, not sure how you guys would want to modify it


This should use `ImmutableSet` so that the set of permissions can't be (accidentally or intentionally by a "bad" plugin) modified when calling `allPermissions()`. Same for `allPermissionsMap()`.


if this functions would be used only here just create local function

someName = ->


Yeah, that colon looks like a typo


Is there a cleaner way to write this using elif?


>  I was not sure where to put it.

Today we suggested to move such code to a bot/lib or similar folder. We should discuss this with upstream.
In addition we've heard about an alternative xmpp-bot.
@rolandgeider can you dig into this, please?


As long as the indentation isn't too far off and creates some valid YAML meaning, it doesn't seem to matter. Feel free to fix it.


@deepakunni3  @cmdcolin   This is related to #902 (I am momentarily fixing in a pull request).   I added it a long time ago.  Can't figure out what / why we need this here.  I'll remove it unless you have a good idea.


log.Debug("my validating project - ", string(res.Body[:]))
```
(node-1) DBG | my validating project -  {"errorCode":"MISSING_HEADER","httpCode":"400 Bad Request","message":"Missing X-Ovh-Consumer header"}
```


This may have consequnces for resource type - puppet turns somthing like File["foo"] into "File[foo]" - I am not sure it can deal with a quoted title - and that is used *everywhere* in the 3.x code unfortunately. May need to differentiate the method when producing resource and class references (the value must be a valid name anyway).


Broken yaml file


:fireworks:


@waderyan Not even.  This was just me flailing around after I got vscode-website compiling again.  Sounds like we want the Subscribe link on the connect widget on every page so Brad and I can fix this today.


Interesting stuff! And seems quite helpful. Learning to code via youtube seems pretty handy to raise and address issues. Kind of makes me wish I was a programmer, building powerful yet elegant apps with maximum efficacy and efficiency. Glad I can at least have a front row seat!


I am not sure that this body works. because we tried to send ajax request before and it didn't work.


Add a method to generate the id.
`this.id = this.generate(period, id);`


You forgot the `.java` extention.


LGTM


I am not sure this is the fastest way in Ruby - see the perf snippets repo. Did you measure?


Done.


Add to use Passive actions.


OK. So there will just be one feed for VS Code docs.  It isn't just for the blogs.  We can sent updates to the feed for new blogs, releases and to call out new doc topics.


why do you need direction here. Have you write something in Arabic?


You are right that it is not super clear what to expect from `overflows(long, 30)`.
I do not see better solution than to add checkArgument on `precision` argument and verify if is not greater than max precision for short decimal.

I will change the functions code to use `checkOverflow` methods instead of calls to `overflows`


yup. will resubmit.

On Wed, Mar 2, 2016 at 9:31 AM, Nick Becker <notifications@github.com>
wrote:

> Is there a way to use 'elif' to make this statement cleaner?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DZwell/math-series/commit/ae0e7c00542296be0fb12c0ef09200b78f1150b0#commitcomment-16451925>
> .
>

--
Daniel Zwelling
Drummer | Bandleader | Educator
www.dzwellingmusic.com


@lesander zou een functie voor moeten zijn om een commit niet op newsfeed te laten komen eigenlijk


no - this is bad. Cannot do it that way.


Yup, @czue has a better version of this which doesn't clobber. Fine to
leave as is for this PR though.


could you rename .countdown-show1 to countdown-show98.
Name should bring some logic


I think the issue is that #ifCond isn't supported on plain HTML.  The page is called "blog".


esta linea no me gusta, pon mayusculas


I believe it needs to be R/W if the user chooses to make a new folder


@michaelmalonenz I had some issues with setting up the test like you suggested. the promise out of showDialog depends on a transition end event that isn't firing in the test runner..

not sure how far I want to dig into that before hearing what some of the core team has to even say about this feature.


Wait. Is this 1.8.9->1.7.10 migration or are you making 1.8.9 blocks metadata-compatible with 1.7.10 and *then* adding the hacky workaround for existing 1.8.9 blocks?


OK. :-)

For the record, PutForwards means that the assigned value is forwarded to the specified IDL attribute on the `DOMTokenList` object, so `elm.classList = 'foo'` is the same as `elm.classList.value = 'foo'`


I think you need to cat dockerfile.js here as well.

```
RUN cat *.cmis.js $(opam config var lib)/iocamljs-kernel/kernel.js dockerfile.js > ...
```


I guess I didn't really add any files...


hi,

i got an error, the foreign key is not found...

Regards
karl

 ERROR liquibase (line ?): Change Set 2016-02-24-SCHEME-v341.groovy::1456401696309-2::birger (generat                               ed) failed.  Error: Error executing SQL ALTER TABLE `web_performance_waterfall_waterfall_entry` DROP FOREIGN KEY `FK297D11DE5F1B69E4`: Error on rename of './osm                               /web_performance_waterfall_waterfall_entry' to './osm/#sql2-475-41' (errno: 152)
liquibase.exception.DatabaseException: Error executing SQL ALTER TABLE `web_performance_waterfall_waterfall_entry` DROP FOREIGN KEY `FK297D11DE5F1B69E4`: Error                                on rename of './osm/web_performance_waterfall_waterfall_entry' to './osm/#sql2-475-41' (errno: 152)


why you didn't use  `@test_setting.tests` ?


Just a comment


Just another branch


no, bad - case produces a value


and put above query here without temporary variable


Do you mind reviewing the whole doc for links that aren't hyperlinked correctly? ie see above


Sorry, my question was more about why "variadic bound". I made a similar comment in another commit.


no, not meaningful, not a definition


no


112259371_UnitHeaterHWCoilUASizing (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2127 of 2129 tests passed, 0 test warnings)

 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

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


Can we make sure that all the headings in this doc are title case since they are levels H1, H2, or H3. Bold is fine to leave as sentence case but headings should be title case.


no, produces a value


Measure will fail/exit if the model has more than 3,000 simulation points, as the results files are simply too large. Multiple window group results files are iteratively added now, rather than doing them all in one go. Minimal performance impact on small files, and this is really the only way to make this work on models with lots of points and/or window groups. Data from the old merge and new merge methods match up perfectly:
![merge_test](https://cloud.githubusercontent.com/assets/3161332/13469263/cea90052-e063-11e5-829d-4ef9e85d551d.png)


CONSUMES is missing


not meaningful - remove


This is so that existing 1.8.9 blocks will convert metadata to the 1.7.10 version. I'm also going to just hardcode the meta values for everything into the existing 1.7.10 way. Essentially this is for 7.2.0-pre -> 7.2.0 proper. 1.7.10 -> 1.8.9 won't do this hacky workaround and will use the existing meta values.


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


112796613_DesicDehum (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Great! I love it!

2016-03-02 18:48 GMT+01:00 AlexIIL <notifications@github.com>:

> This is so that existing 1.8.9 blocks will convert metadata to the 1.7.10
> version. I'm also going to just hardcode the meta values for everything
> into the existing 1.7.10 way. Essentially this is for 7.2.0-pre -> 7.2.0
> proper. 1.7.10 -> 1.8.9 won't do this hacky workaround and will use the
> existing meta values.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BuildCraft/BuildCraft/commit/ba0c3a026fc594a38604b620a3feb81b4fed9cde#commitcomment-16452410>
> .
>


I might be wrong, but this looks like a regression. `self.view.window()` is None when you use CMD+P switcher (at least in ST3 build 3103) and so project specific syntax is not loaded and not activated when you select the file. It works when you select the file in the sidebar, but not with CMD-P.

The previous behaviour worked correctly, you just need to put `project_syntaxes` under `settings` in the project file:

```JSON
{
"settings":
	{
        "project_syntaxes": [
            {
                "syntax": "Javascript ES6/Javascript ES6",
                "extensions": ["js"],
                "rules": [
                    {"first_line": "^#!/.*\\b(node|js)$\\n?"}
                ]
            }
        ],
	},
}
```


^Hi,
Actually I forgot to stop the loop, I will fix, thanks


I didn't mean to commit the changes to PCB_GUIDE at this point. Many of these changes will be removed in an upcoming commit.


Are you sure? Look in the [routes file](https://github.com/Microsoft/vscode-website/blob/master/src/routes/blogs.ts). Its clearly called "blogs".

#ifCond is handlebars syntax.


rs=me, but I don't have time to look in great detail


Additional FFMPEG API changes:

2014-08-07 - bb78903 - lsws 2.1.3 - swscale.h
  sws_getContext is not going to be removed in the future.

2014-10-15 - 17085a0 / 7ea1b34 - lavc 56.7.100 / 56.5.0 - avcodec.h
  Replace AVCodecContext.time_base used for decoding
  with AVCodecContext.framerate.

2015-10-29 - lavc 57.12.100 / 57.8.0 - avcodec.h
  xxxxxx - Deprecate av_free_packet(). Use av_packet_unref() as replacement,
           it resets the packet in a more consistent way.
  xxxxxx - Deprecate av_dup_packet(), it is a no-op for most cases.
           Use av_packet_ref() to make a non-refcounted AVPacket refcounted.
  xxxxxx - Add av_packet_alloc(), av_packet_clone(), av_packet_free().
           They match the AVFrame functions with the same name.

2014-07-29 - 80a3a66 / 3a19405 - lavf 56.01.100 / 56.01.0 - avformat.h
  Add mime_type field to AVProbeData, which now MUST be initialized in
  order to avoid uninitialized reads of the mime_type pointer, likely
  leading to crashes.
  Typically, this means you will do 'AVProbeData pd = { 0 };' instead of
  'AVProbeData pd;'.


how can attr and private cause conflicts - they are not even used in the grammar are they? (if they are, do they lack precedence? Don't want UNDEF, or DEFAULT as special - they are values.


Yes, or just make it possible to pass the values of the type arguments to the method (which could be done by tagging the relevant arguments with annotations).

But, it depends on whether there's no performance impact. The "extra parameters" abstraction adds a lot of complexity, and it makes it harder to figure out how all the pieces fit together due to the loose-typed nature of passing lists of objects around, etc. So, anything we can do to simplify that would be good.


Yeah. That model makes sense in my mind, but I'm open to different opinions.


Hi,
thanks for reporting!
Die you try a `grails dbm-clear-checksums`?

This error occurs because a migration was been changed after it was executed in your db.

Regards, Birger


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#aKAEEMmbS6maXP1wyuzuTw


why 'MyModule'. Please put notes 'Name must be logical' and read it more than 10 times.
or 'Your code should be self-documented'. That means I or other developers read it and understand what is going on.


hahahahahahahHAHHAHAHAHAHAHAHAHAHHAHAHAHAA


assignments_or_keywords is kind of odd term - makes you think it is all keywords. Since this is only used for array/hash entries name it 'collection_entry' or somesuch


Ok cool. We have a discussion on Slack with @Tyriar so the jury is still a bit out.


2055_CommasInNames (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


@phistuck - Yeah I realized the comment is not clear over there. This PR is nothing to do with any `Symbol::ToString `behavior change. This PR is fixing an AV caused by returning an unexpected `null  `from `ToString` code. We were wrongly checking the `DisableImplicitCall `over there - instead we need to check `ImplicitException`.
@ianwjhalliday  - the test is correct, the flow is
    1) Accessing index of `ab `should throw an error (reference error), as `ab `is undefined.
    2) While populating information about that error we will do `ToString `call on the index object (which is `Symbol`) and that also throws an error.
However we were wrongly checking the `DisableImplicitCall `flag in the `ToString `code and returning `null `instead of throwing. So the test is exercising and validating that code path.
Does it make sense?


Ik vind het gek dat een commits van niet-main branches op de newsfeed verschijnen.


Ah, yes, that's a good point, and something we need to solve eventually.

However, we don't need plugins to depend on the classes that implement those types, only on the knowledge of whether it uses the short or long encoding. So the snippet you posted would work.


```
    output_regex = re.compile(r'(?P<file>(.+)):(?P<line>\d+):'
                              r'(?P<column>\d+):.'
                              r'(?P<severity>error|warning|info):.'
                              r'(?P<message>(\w+.*))')
```
could be
```
    output_regex = re.compile(
        r'(?P<file>(.+)):(?P<line>\d+):(?P<column>\d+):.'
        r'(?P<severity>error|warning|info):.(?P<message>(\w+.*))')
```

I find it easier to read as crunching up things into a small width in the corner gets confusing


Done.


No, it was just for testing, I wiil remove it.


Also not that APPLICATION, PRODUCES, CONSUMES are delivered in alternative form when application management feature is off. The alternative form tokens must also be added.


@nathandunn I suppose thats fine.


Added this to upstream


I suggest that the Subscribe button to be in the upper right for the blog pages similar to the Edit in GitHub button for docs.  This is where the Subscribe button is on MSDN and I think it is the most important action on the blog topics.


Could you update the `ContentType` fetching while you're at it? (lines 265-270)


done


yea yea i understand the issue!!!!!


Makes sense. We will do some microbenchmark to verify how this behaves.


I would use `ndio.remote.neurodata.hostname` for this, and let them specify it (and pass to the constructor, as i think you already do) in the constructor


@koobs I had the same intuition, but a lot of time there's a bunch of detritus in the folders that would be included (like `.pyc` files and whatnot) with a wildcard include, and currently there is nothing by `.py` files in the tests.

I'm not super familiar with the MANIFEST syntax, but I assume there's an `exclude` directive we could use to exclude `.pyc` files, then we can switch to a wildcard formulation.


2055_CommasInNames (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


```
return token_name in nd.get_public_tokens()
```


It was needed due to the condition in validate if statement that I was able to remove. For live (playbackController.getPlayedRanges().length > 0) was returning validate so we called again here.  Not needed anymore.


where is it getting label from?


Hmm.. that is a very good point. The location in the connect widget is not optimal for conversions... I'm open to that. Maybe @bgashler1 could hammer out the details (specifically in a responsive way). I put it on the connect widget because it used to be on the connect page.


@fenollp what to update ?


I'm looking at the top of blog-template.html:
{{#regObject page="blog" section="{$ Link $}" title="{$ Title $}"}}{{/regObject}}
I don't think you can use handlebars syntax in plain HTML.
Anyway, we've landed on removing the conditional and just show Subscribe on every page, correct?


Good point.


No improvement in efficiency...
- Does not take 209 days to run due to smaller data sets
     - (Diff_Exp_Genes found under Nucleotide BLAST results vs. BirnaVirus and Xvirus genomes found online)


:dancers:


OK, I should add better error messages, especially in case of broken authentication. Did you follow https://github.com/yadutaf/docker-machine-driver-ovh#1-get-your-ovh-credentials configurations steps ? init file should be in either the current working directory, ``/etc/ovh.conf`` or ``$HOME/.ovh.conf``. If all fail, please try again with explicit options from https://github.com/yadutaf/docker-machine-driver-ovh#options


fixed


Is this necessary? Java 8 cleaned up their date time functionality based on Joda time.


delegates to the contained type ?


> I don't understand - "+?" is exactly equal to "*"

No it isn't and neither did I imply otherwise. In PCREs a question-mark turns a greedy, match as many chars as possible, into a non-greedy, match the minimum necessary, pattern. Try the following at a shell prompt:

```
string replace -r '^([^ ]+)=(.*)' '$1 $2' -- a=b=c
string replace -r '^([^ ]+?)=(.*)' '$1 $2' -- a=b=c
```

Adding the equal-sign to the excluded char class works just as well so it's a coin-toss in this situation which to use.


doesn't variadic bound mean that we want the type parameter "X" to be any "DECIMAL".
We can change the code here to use `decimal(fp,fs)` as sql type for argument and `decimal(tp, ts)` as type of return type. And not use type arguments mechanism at all.
Like we do in operators definition.

Would that be better?


This is a cuda-c program used to get the device specifications of the system containing graphical programming units, ie, GPU's


Maybe also remove the "Selon huffPo" + link? I've made the TelQuel one before just for liability.


True. I thought about suggesting this, but I decided that anything's preferable to the current situation.


standard_report_area_check_tol (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Attribute lists were being loaded from the graph without checking the ordering of the keys in dict to list conversion


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 1](https://teamcity.labkey.org/viewLog.html?buildId=307174&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 34 (34 new), passed: 9, muted: 5) Build time: 00:39:26
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


That makes sense, thanks for the pull req!


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 128](https://teamcity.labkey.org/viewLog.html?buildId=307162&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 16 (3 new), passed: 276, muted: 5 Build time: 01:45:26
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.util.JUnitFooter.afterJUnit: <no details avaliable>

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

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

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

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

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

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

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

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

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

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


remove the (_ok) what would you otherwise expect an assert to do ?
I would also expect it to return what was asserted


using `window = sublime.active_window()` instead of  `self.view.window()` works fine, but I'm not an expert in Sublime API.


@cmfcmf apparently not - it only calls the DB when rebuilding the routes.


Another issue with the connect widget is that it gets pushed down the page when you have many right navigation h2 headers.  In some cases, it is completely lost.  I don't like limiting the number of h2 headers on a page just to leave room for the connect widget.  h2 is important for quick navigation on longer topics and really important for website accessibility.


Make sense - but see my comment https://github.com/Teradata/presto/commit/5f2f22003ed4ebdca3675cc6a2ef8deec41f6bb7#commitcomment-16452207


I think this won't work as intended - changing from PROGRESS_INSTALL to ERROR triggers this piece of code:

```
elif line.startswith("ERROR:"):
    log.error(line[6:])
    install_errors.append(line[6:])
```
A then on the end of installation phase:

```
if install_errors:
    exn = PayloadInstallError("\n".join(install_errors))
    if errorHandler.cb(exn) == ERROR_RAISE:
        progressQ.send_quit(1)
        iutil.ipmi_report(IPMI_ABORTED)
        sys.exit(1)
```

I'm sure we want to avoid that, so I have this suggestion:

* change the "PROGRESS_INSTALL" to something like "PROGRESS_RETRY"
* then in pyanaconda/packaging/yumpayload.py add the new string to the progress_map dict with a appropriate prefix or even just ""

That should fix the installation status reporting message correctly.


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1572 of 1573 tests passed, 0 test warnings)

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


This is the same way this is handled in our other invite email code


This function is only used when there is a terminal, no ? I will check with an unpatched version later...


standard_report_area_check_tol (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


`{{#regObject page="blog" section="{$ Link $}" title="{$ Title $}"}}{{/regObject}}` is handlebar syntax.

I actually am being persuaded to look into making it more prominent on the blog like you suggested.


@mhinz You force pushed over my commit!


about `self`. If you want to save state it would be easiest to create Class inside Module and call it directly.
But It would be behave like module.

I mean instead of include this module somewhere would be easiest to call something like Calculations::AnswersCalculation.count_and_paste_test_result_into_test_table(test_record)

```ruby
module SomeModule
  class SomeClass
    attr_reader :test

    def initialize(test)
      @test = test
    end

    def foo
        p test
    end

  end
end

SomeModule::SomeClass.new('some_var').foo #=> 'some_var'
```


Think it needs to assert that the given default value matches the type


Yes agreed. We might need to do something there.


Запити в циклах недопустимі, треба переробити


Sorryyyyyyyyy

On Wednesday, March 2, 2016, Paul Tagliamonte <notifications@github.com>
wrote:

> @jfrazelle changed API on me :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/paultag/go-dictd/commit/5b7a28d26bf5dffaa8c3d54296e7345402c58df8
>

--

Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu <http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3>


Doesn't Short array syntax need 5.4.0


I've written a `<chart>`'ing C3/D3 wrapper directive that dynamically instantiates chart-type implementations based on a chart's options. Thanks to this fix I can do `var ChartImpl = this[type.charAt(0).toUpperCase() + type.substr(1) + "ChartImpl"]; var chart = new ChartImpl(data, options);`.

My implementation is not backwards compatible to prior releases of Angular because of the screwed up `this` context in those releases.

Please ensure that `this` here and elsewhere remains the global context so we can utilize the full power of non-strict JavaScript! Thanks!


There are spaces in this blank line - your linter should grump at you if you have a blank line containing whitespace.


شاید پیدا کردن علل هم یکی از دلایل این بحث‌های اینجا باشه. لیست کردنشون جالب نیست.


Again, minor blank-line-whitespace linting error.


112796613_DesicDehum (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2162 of 2162 tests passed, 1 test warnings)

 * 1 test had: AUD diffs.
 * 1 test had: Table small diffs.

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


assuming this is not cached, maybe name it `to_feature_hash` to indicate that a copy is returned (expensive).


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


Theoretically, `schedule_tick/1` *could* return a modified state.


Yeah, I'll sort out some more kinks.  Getting travis to handle all these jit/version variations in one pass would be ideal, but a configuration mess.


مرجع و رفرنس هم معنی اند و رفرنس زائد است


В пхп є спеціальний клас Ітераторів, треба використати його


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.5 [Build 118](https://teamcity.labkey.org/viewLog.html?buildId=307166&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 9, passed: 38, muted: 4 Build time: 01:14:46
### Failed tests
```
org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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

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

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

```


Indeed. LGTM.


put one more note 'Try to avoid using temporary variables' and read it 10 times.

what do you think how we can simplify this 2 lines:

```ruby
 array1 = test_answers.map {|k,v| [k, v.length]}
array_with_question_id_and_answers_number = array1.flatten
```


reverted


Was that literally it? Or was there stuff for factions not in our `00_factions.txt`?


112796613_DesicDehum (Nigusse) - x86_64-MacOS-10.9-clang: OK (2124 of 2124 tests passed, 1 test warnings)

 * 1 test had: AUD diffs.
 * 1 test had: Table small diffs.

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


We are working on the migration of this module, but now that is deprecated we have to migrate project_issue_management?


در کل به نظرم متن خوبی نیست. شاید بهتر بود به عنوان یکی از بحث‌های داخل ایشوها پست می‌کردیش تا متن صفحه اول


Код не універсальний, бо якщо добавиться чи видалеться елемент, то треба переписувати кучу кода.


من جهت خالی نبودن این متن رو نوشتم.
ادبیات من ضعیف هست. دست خودت رو می بوسه :دی


The idea of the payment_options table is it will replace the gateways table as well as settings.payment_options and settings.gateways


jQuery є селектори попередніх і наступних елементів


Couldn't we shorten this to `return attrs.type + attrs.id`?


@akroshg: the thing is, there shouldn't be any `ToString()` call at all in this particular situation, that's the main thing that is confusing about this.


@spencerpbartz This is an example but if possible we should to keep it constant


... got missing


'locales' => ['en'], // TODO: Change this to include all locales!


that's literally it. the new faction you disabled had some ai likelihood changes but i left those out of course


Yep, definitely :)


@Makman2 Ok!


ok, will remove this


"....the non existent..."


112796613_DesicDehum (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1016 of 1016 tests passed, 0 test warnings)

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


Perhaps just use @ to suppress echoing of the command?


112796613_DesicDehum (Nigusse) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


there is typo - plam instead of palm


I wasn't sure which way to do this and am very glad that you have an opinion.


did you actually test this commit? rosters are not working anymore, filters are obviously not working anymore because you are not replacing the %u anymore, thanks for this really helpful commit -.-


minor: space between identifier and paren


@avichalp Interesting. Is it no longer maintained by @nrk? I contributed to it some years ago and I could contribute again if needed.


This needs to include the key, e.g. "order by entry for '#{key}' has direction #{'direction'}, must be either...."


first think about names. `@result_from_2_method` really?

second think is we lost some part of my comments. Because I already write about calling methods.
Instead of using method variables we can call method directly.

```ruby
class SomeClass

  def foo
    p bar
  end

  private
  def bar
    'some my result'
  end

end

SomeClass.new.foo #=> 'some my result'
```


@permcody I think this line is not quite correct. How is `defaultCoupledValue` assigning `default_reals[j][i]` to the
right `coupledValue`?


Huh, when did that happen? :X


> I might be wrong, but this looks like a regression. self.view.window() is None when you use CMD+P switcher (at least in ST3 build 3103) and so project specific syntax is not loaded and not activated when you select the file. It works when you select the file in the sidebar, but not with CMD-P.

Hmm.  That is interesting.  I didn't test that case.  I honestly don't use the project specific option much, so that is why I may have made the change too hastily.  I wasn't aware you could just stick them under `settings` and have it work (or maybe I have since forgotten).  I will have to reopen this issue and look into it more.  ref #93.


Thanks!


OK, can do.

On Wed, Mar 2, 2016 at 4:08 PM, Paul Cercueil <notifications@github.com>
wrote:

> Ok, it makes sense. Could you add that info to the commit message maybe?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gnieboer/gnuradio/commit/2faa6cfab436ebc7bd0d2a9a2cc7fe904a85096f#commitcomment-16444646>
> .
>


remove include and call something like Calculations::AnswersCalculation.count_and_paste_test_result_into_test_table(test_record) within method e.g. `cout result` for Test model.


Ciulu nie tak, zmien to!!!!!!!!!!!!


Maybe we can ditch this `kAll`


Writing program


on windows you just say "make html" as well (it runs make.bat)


isn't that infer of low quality? infer_set?


This seems to be the source of issue #50


Closed.

I added the checkboxes back for the old themes and hid them for the Bootstrap theme.


favorite (we're being consistent with American English spellings)


call to iterable? takes one param I think


move it to a helper method.

"No calculations within views".


this is not `@Nonnull` - if you want `Nonnull` use `getActiveInstance()`


@tin315
Sorry if it wasn't clear, but @@CONVERSION_API_VERSION will be swapped out with the current API version, so instead of updating the date every single time like you did here, we can just write this once:

```
The API version is "@@CONVERSION_API_VERSION" and the unique Conversion Reporting ID is...
```


Nice, I like PRIsz. Looks like it's already defined on Linux, so I added a Windows definition. Just pushed a version with that fix.
FYI, I've also tried reverting one of Damian's uses of PRIu64 for something that was explicitly defined as "unsigned long long" (so "%llu" was supposedly correct), but that apparently requires building with -std=c99, which doesn't work in our build (some compilation errors about SSE intrinsics).


You're right...

I guess the code is here that *should* work but we don't have that capability in `InputParameters` right now. We might want to document that, or possibly open a ticket to expand that capability.


None of these should be passing by reference. This breaks PHP 5.6


isn't it better to return an iterator over members?


same here


What happens when you use sum_series() to return, say, the sixth Lucas number (where n = 5, x = 2, and y = 1)? It *should* return 11, but instead this function would return 5.

What are x and y at the start of each recursively-called sum_series()?


and here


So this would keep on assigning the values over and over and the last one sticks.


Yep

On Wed, Mar 2, 2016 at 11:20 AM Sebastian Schunert <notifications@github.com>
wrote:

> So this would keep on assigning the values over and over and the last one
> sticks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/snschune/moose/commit/7d33ce2674f2b55728fc294d60c0747c0c4e525d#commitcomment-16453279>
> .
>


This should be an elif statement


Καλησπέρα! Τι κάνεις;Θα ήθελα να σε ρωτήσω κάποιες λεπτομέρεις σχετικά με
το datasheet.
Που μπορώ να το βρω λεπτομερώς για να υπλογίσω αν μπορέσει να χωρέσει σε
μια fpga;
Επισης εχεις κάποιος skype ή email για μια πιο άμεση επικοινωνία; :P :P

2016-02-04 20:15 GMT+02:00 Kominos Charalampos <notifications@github.com>:

> Egw tha souo elega na vreis kai na kaneis thesis pano se kati to opoio
> exei
> ergasiako antikrisma. FPGA den asxoleitai kaneis. Kane tpt se Spark ama se
> psinei.
> Kalo thesis borei na shmainei kali douleia!
> Apo kei kai pera genika o genetikos thelei douleia gia na ginei antropinos
> kirios to kommati tis dimiourgias indicidual,detect ama exei lathos kai
> kirios to repair! to repair paizei na kostizei O(n!) opos to xw kanei .
> Ama telika apofasiseis na to ksanagrapseis steile mou gia perissoteres
> leptomeries
>
> 2016-02-04 14:42 GMT+01:00 SotirisKordas <notifications@github.com>:
>
> > Το repo το βρήκα ψάχνωντας στο search "Diploma Thesis" και πηγαίνωντας
> > πολλές πολλές σελίδες πιο μετά βρήκα ελληνικό όνομα και λέω να τσεκάρω.
> Ε
> > και μου άρεσε η ιδέα οπότε είπα να ρωτήσω χαχαχα
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/hkominos/Diploma_thesis/commit/7333705facef27da05c4eb3551f4f402e8499a72#commitcomment-15886027>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hkominos/Diploma_thesis/commit/7333705facef27da05c4eb3551f4f402e8499a72#commitcomment-15893753>
> .
>


Elif again


Ah, right, I forgot to put that in the comments; it's rounded due to some old platform limitations (possibly just the AFDKO?) which only allowed 3 digits in the minor version.

Build vs. version number, eh I'm not really sure it's important. I think the version number has been referenced previously, and anyways I never liked the idea of bumping up the build number every time binaries are generated.


+1. But we shoudl remove the [superfluous div](https://github.com/DavidOliver/default_event_values/commit/7092b7d5bdbe1e10e3a48a6435f2376ad2fe159c#diff-fc6235a4a7c268fd53eb36ad01a5b772R274) element as well, which would require a change in adding children (around 35 lines below that line).


@akroshg - I am not sure the comment is unclear, I think it is incorrect. The comment (`we will try to do implicit toString call on Symbol which will also throw`) says that the this implicit ToString on the Symbol should throw, but it apparently should not (unless Chrome is not following the specification in this exact example). Or, the comment may be wrong because (@caitp suggests), this is not really an implicit ToString (and also therefore should not throw).


is this not the fully woking app ???


@shlok007 ,please go through the issue Setting up Of Project #4 before moving ahead.


@kutzi Would it be possible to do a release of the irc plugin with these changes? I've encountered several issues where the `shutdown` will block forever when using the `/safeRestart` url.


This can be an else because if the other catches weren't activated, the only other possible outcome is that the number is greater than 3.


fix-window-frame-calcs-5095 (nealkruis) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


hmm - I'd let @boingoing to comment about whether should this even do implicit `ToString`() call.


yes, but there are still utility methods like `unscaledValueToSlice` in `LongDecimalType`, which are used in connectors. They are specific to long decimals. Putting those in `DecimalType` class wouldn't hide the fact that there is decimal dualism. I think leaving them in `LongDecimalType` class is actually cleaner.


"with from" ?


Good job figuring this out! The curriculum shouldn't give you all the answers :sunglasses:


Implementing feature #12


When I change this line manually and try to use Google auth, I get Error:

> AttributeError at /accounts/social/complete/google-oauth2/
> 'str' object has no attribute 'update'
> Request Method: GET
> Request URL: http://myapp.com:8000/accounts/social/complete/google-oauth2/?state=2M0kJYG5Y6gY2168PvXZYoSASqvkaGhk&code=4/oSlipMXmeksLQO8ekcGnWVtm9ZPFv1oBpJptr4Cm0wk&authuser=0&session_state=150dc7ee96507db45d657099378028204c576a4a..dfee&prompt=none
> Django Version: 1.9.2
> Exception Type: AttributeError
> Exception Value:
>
> 'str' object has no attribute 'update'
> Exception Location: /home/albert/HomeFinder/env/lib/python3.4/site-packages/social/storage/base.py in set_extra_data, line 85
> Python Executable: /home/albert/HomeFinder/env/bin/python
> Python Version: 3.4.3

Without this change everything works fine.
What am I doing wrong?


rename commit name to some logical one. "Commit names should be logical!"

```bash
$ git rebase -i HEAD~3
```

```
pick e499d89 Delete CNAME
pick 0c39034 Some name1 README
pick f7fde4a Some name2

change to
pick e499d89 Delete CNAME
reword 0c39034 Better name README
reword f7fde4a Better name 2
```

```bash
git push --force
```


I'd expect for internal values, you'd be able to just grab the Symbol's internal description slot.


No, this is a Rails Engine, essentially a plugin to provide functionality to an existing Rails app. The documentation explains how to install it into your own Rails app. You can see an example of this gem, Talking Stick, in a working app: https://github.com/mojolingo/talking-stick-demo


@arunsrinivasan up to you. I'm partial to the space for blank `i` (for no particular reason), but you're obviously the final arbiter.


@llysz looks like that. if i try to use a separate decoder instance it will fail :/


This is a ruby "formatting best practice" suggestion, not something you need to change. Generally when you have blocks like this with one line that are very long, we'd break it up into a `do...end` block instead for readability.

```ruby
authenticate_or_request_with_http_basic do |email, password|
  User.find_by_email(email).try(:valid_password?, password)
end
```


Good catch. This should be a conditional in case the base `url` already has a trailing slash. Feel to make it look nicer than this..

```js
url += (url.charAt(url.length-1) == '/' ? '/' : '') + (this.dataModelAliases[model] || model);
```


Do you think we should just not remove this temporarily until the backend APIs are in place?  I'm not sure how much we're blocking @tamird on this.

I suppose there could just be a separate PR to strip out all of the SQLQuery stuff, though.


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


I'm excited for Ruby 2.3's safe navigation operator... some day when Ruby 2.3 is more common. :smile:

```ruby
  User.find_by_email(email)&.valid_password?(password)
```


Edited code.


http://www.cs.nott.ac.uk/~pszrq/data.htm exei ta panta.
hkominos@gmail.com steile oti thes ama exeis tpt apories. Episis ama
apofasiseis na to kaneise sigoura tha prepei na diavaseis kai kati papers.
nmz ta exw sto keimeno sta references

2016-03-02 19:20 GMT+01:00 SotirisKordas <notifications@github.com>:

> Καλησπέρα! Τι κάνεις;Θα ήθελα να σε ρωτήσω κάποιες λεπτομέρεις σχετικά με
> το datasheet.
> Που μπορώ να το βρω λεπτομερώς για να υπλογίσω αν μπορέσει να χωρέσει σε
> μια fpga;
> Επισης εχεις κάποιος skype ή email για μια πιο άμεση επικοινωνία; :P :P
>
> 2016-02-04 20:15 GMT+02:00 Kominos Charalampos <notifications@github.com>:
>
>
> > Egw tha souo elega na vreis kai na kaneis thesis pano se kati to opoio
> > exei
> > ergasiako antikrisma. FPGA den asxoleitai kaneis. Kane tpt se Spark ama
> se
> > psinei.
> > Kalo thesis borei na shmainei kali douleia!
> > Apo kei kai pera genika o genetikos thelei douleia gia na ginei
> antropinos
> > kirios to kommati tis dimiourgias indicidual,detect ama exei lathos kai
> > kirios to repair! to repair paizei na kostizei O(n!) opos to xw kanei .
> > Ama telika apofasiseis na to ksanagrapseis steile mou gia perissoteres
> > leptomeries
> >
> > 2016-02-04 14:42 GMT+01:00 SotirisKordas <notifications@github.com>:
> >
> > > Το repo το βρήκα ψάχνωντας στο search "Diploma Thesis" και πηγαίνωντας
> > > πολλές πολλές σελίδες πιο μετά βρήκα ελληνικό όνομα και λέω να
> τσεκάρω.
> > Ε
> > > και μου άρεσε η ιδέα οπότε είπα να ρωτήσω χαχαχα
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/hkominos/Diploma_thesis/commit/7333705facef27da05c4eb3551f4f402e8499a72#commitcomment-15886027>
>
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/hkominos/Diploma_thesis/commit/7333705facef27da05c4eb3551f4f402e8499a72#commitcomment-15893753>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hkominos/Diploma_thesis/commit/7333705facef27da05c4eb3551f4f402e8499a72#commitcomment-16453293>
> .
>


1) Что именно тут неправильно?
2) Надо не выпиливать, а реализовать правильно.


Because we are testing subscriptions, I think its a good idea to verify that unsubscribe works correctly. Thanks.


What you said makes a lot of sense.

I think you are right, one should just define the length with regards to the number of records they want to store. Also one could start with a small length and increase it later if the number of ids is start go get bigger than expected.


memcmp.S was left out of the previous git push,  with this routine the kernel should build with no issues


i din't understand the concept of participants....... which is not in  https://github.com/mojolingo/talking-stick-demo ...... and also i can't get rooms_controller  please help


You could use a few more test cases here for testing the sum_series(). Out of these four tests, only one of them causes sum_series() to enter its recursion, and that's only when x and y are configured for the Fibonacci sequence. Testing an n > 1 for a non-Fibonacci sequence would have allowed you to catch the problem with sum_series().


'locales' => ['en'], // TODO: Change this to include all locales!


fix-window-frame-calcs-5095 (nealkruis) - x86_64-MacOS-10.9-clang: OK (2120 of 2121 tests passed, 485 test warnings)

 * 485 tests had: AUD diffs.
 * 479 tests had: RDD diffs.

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


you have a lot of changes here. It was bumped by only this gems
gem 'sprockets-rails', '~> 2'
gem 'rmega'
gem "carrierwave-mega", :git => "git://github.com/zdubzdab/carrierwave-mega.git"

Is it happens because you executed `bundle update` ?


You're right about this being problematic, but let's look at what [that code](https://github.com/Stackdriver/collectd/blob/windows-fixes/src/daemon/common.c#L301) is doing in the first place. It first calls `poll()`, checks for a non-zero value (indicating that a socket state changed), then attempts to `recv()` data and uses the return value of 0 to mean that the socket is in a disconnected state.
According to the docs, poll() will set bits in the data structure passed in, which are completely ignored in the code. Unless I'm missing something, checking `pfd.revents` for the `POLLHUP` bit should achieve the same result as the `recv()` call, in a more portable way.
Does it make sense to simply replace the `recv()` call by a check of `pfd.revents & POLLHUP`?


please help me out bklang


Actually, try to avoid mentioning people in the commit message since it will notify the person whenever this commit is rebased....


@pamval These changes have been made in https://github.com/pentaho/pentaho-kettle/pull/2273.


sweet!


Please follow the installation instructions in the Talking Stick README. That should get you what you need: https://github.com/mojolingo/talking_stick#installation


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


It throws a twisted.internet.error.DNSLookupError exception, which seems appropriate.


Weird, i am not sure where that came from- re-pushing that file.


Joe sólo por curiosidad, no funcionaba aqui un ```return &UNIQUE, nil```? Es que me resultó confuso entender el  ```.Operator()```


Without kills and level, samurai won't be attacked at all.
Samurai also strips stealth, so I wanted to exercise the stealth removal
code, requiring that stealth is present at the outset.

On Wed, Mar 2, 2016 at 12:08 PM, Roy R. <notifications@github.com> wrote:

> ??? Why is status stealth needed when attacking a samurai? Or kills?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BitLucid/ninjawars/commit/4d608f0d973f3a792dd5bdc963936e0c9aee0fb2#commitcomment-16451335>
> .
>


getNumArrayValues() + Preconditions.checkState(type == INT_ARRAY)?


There is on code shared here, so please split it up into getNextIdValue and getNextCustomIdValue functions.


this is an Eclipse thing, right? Regardless, maybe it's cleaner just to fold this into the `if`:
```
if (rDotTxtEntry.idType == INT_ARRAY) {
  idValue = getNextIdValue(rDotTxtEntry.type, rDotTxtEntry.numValues());
} else if (RType.STYLEABLE.equals(rDotTxtEntry.type)) {
  idValue = rDotTxtEntry.idValue;
} else {
   idValue = getNextIdValue(rDotTxtEntry.type, rDotTxtEntry.custom);
}
```
?


done


no, tronó. tuve que implementar Operator()


awesome! Tests look good, code as well. Just some naming/organization nits.


File.expand_path("../", __FILE__) #=> /home/zdubzdab/rails_projects/schoolTest


Not sure I understand the exact usage here.  A `TableData` has two ReadOnlyProperties, and in all of these cases the `columns` property is completely static, unless i've missed something.  You should just define a table data on the controller:

```typescript
module DatabaseListPage {
  // Statically define columns at module level.
  let columns = Prop([
   /// column definition
  ])

  class DBListController {
    // tableData defined in controller.
    let tableData: TableData<string> = {
       columns: columns,
       rows: Utils.Computed(this.data, (data) => data.Databases)
    }
  }
}
```


Don't push int this repo please.


like @tushar-rishav said you should describe the circumstances, maybe like
`Maybe you come to the point where we ask you to rebase your PR, as other PRs might get already merged. To update yours follow these steps (assuming you have already [setup a remote](link-to-our-guide)):`


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


You analyzed nodeMCU firmware! You mean, you downloaded the firmware?

What is it? Is it the boot loading code, and then programmer upload their Lua code which have .init extension?


> while values from swf are hardcodes that are the same for each video.

the same apply for hardcoded width and height, then they should be removed also from _formats.
like the video in #8734, most of the reported resolutions are wrong.


Seeing that phenomenon now, I'll look into it.


Might be cleaner to go this route: https://github.com/domenic/chai-as-promised#multiple-promise-assertions
```javascript
promiseC.should.be.rejectedWith(TypeError, "only joyful types are allowed")
```
(requires chai-as-promised)


I dont like this mapgen code, its very very slow, I think I will need to rewrite the whole mod...


This still doesn't handle other characters needing escaping, like `&`. Try using this: https://github.com/felixge/node-bash


I don't think the bundle is populated until after the `super` call.  This should be moved afterwards.


Closed.


Another thing I found out: 1.7.10 won't need an update to be able to migrate pipes! :D


I think he can just strip it out. This'll break but that's ok.


heheheehehehehehe


Een commits


pull_true_answers_from_questions_with_one_answer.to_i  - not needed .to_i because you already returned

@true_answers.count - array_with_number_common_elements.size integer here


@Makman2 Ok! I need to sleep right now! I will do it tomorrow.


Makes sense.


argh! epic fail... I'm going to script this up next to avoid ever having to type code manually again :P thanks!


Commit message accidentally changed to some standard language from within vim.


I would amend `Updates your master on \`<remote name>\``


Should remove `protractory.By = protractor.By`


Ah yeah, i started with a case thinking the enum had more values, def better as an if!


'locales' => ['en'], // TODO: Change this to include all locales!


Hardcoded width and height are **accurate**. All inaccurate width and height are removed upon examples with different values are found (see itag 36).


Nice solution Joe!!


This sounds complicated :O
What about "reapplies your commits onto the newest changes from `<remote name>/master`"?


>
The second  also has a downside, in that any typing which occurs prior to invoking Console.ReadKey(true) will be visible even though it wasn't supposed to be.

Looking at the code, I believe this is consistent with windows as well. ReadConsoleInput() which only reads the stream w/o echoing gets called only for ReadKey() and so anything that is read before the ReadKey() call, should be intercepted by the console there as well.


Closed.


"log history" -> "commit history"
"It force pushes" -> "Force-pushes"


This doesn't need to be a list. At this point, you have control over your code. You can just change a and b to be whatever you want. Putting them in a list, then trying to edit them in the list just makes it messy, and uses up memory. Try and come up with a more simple solution.


usually we wouldn't want to mock/spy the class we're testing - is `mSpy` really necessary, or could we test `mHdfsFileInputStream` instead?


this only tests the spy object.


instead of relying on randomness to eventually test all of these cases, we should test each one directly.


Shell script to run system tests


Technically you don't need to have this 0 and 1 in there. The way you wrote it, they default as 0 and 1 already.


@EisenbergEffect after seeing Babel's generated code, definetely is better to stay of `for..of`, so this is for the best


check that this returns 5


@karthikarunach it works for me with bcprov-jdk15on-1.51.jar, bcpkix-jdk15on-1.51.jar and Netty 4.1.0.CR3 on some Android 4 ... 6 devices.

Hmm, please answer:

* Do you have a problem with this apk too? https://ganskef.github.io/MoCuishle/android-install/
* Is your client working on an emulator with Android 5.0?
* Is your client working with a JDK, no Android?


can you explain what the bug was?


@jeherve typo in `inifinity.php`


i saw the lines,, what to want to fix ?


This does not affect `double-tap-drag`. I just noticed that we (at least in code) seem to want to track `double-tap` (and no drag) specifically with `[self trackGestureEvent:MGLEventGestureDoubleTap forRecognizer:doubleTap];` and that event was never actually logged until this change. And, is possible to just `double-tap` and get a less fancy, hard coded "quick zoom" behavior. So, I guess it is actually worth logging as a gesture in its own right?


@DavidS :+1: thanks man


Натомість ніякої помилки не буде, якщо `let` оголошення створить нову змінну в області видимості із однаковим ім'ям як і в змінної, яка знаходиться...
Після "якщо" і перед "як" кома не потрібна.


Seems that two additional files have been created


It's good to be in individual project only.


Seguro??


This is in the curriculum because it's easier to understand, but I want to remind you that there's a "better way".

When you're at scale, you want to cut down to the bare minimum number of sql queries possible. When you do `find` and then `destroy` you're doing two separate database queries. Let's look at it:

```ruby
[12] pry(main)> user = User.first;
  User Load (0.8ms)  SELECT  "users".* FROM "users"   ORDER BY "users"."id" ASC LIMIT 1
D, [2016-03-02T12:44:21.270535 #26726] DEBUG -- :   User Load (0.8ms)  SELECT  "users".* FROM "users"   ORDER BY "users"."id" ASC LIMIT 1
[13] pry(main)> user.delete
  SQL (1.2ms)  DELETE FROM "users" WHERE "users"."id" = 3
```

vs just using `User.delete` class method:

```ruby
[21] pry(main)> User.delete(2028)
  SQL (1.5ms)  DELETE FROM "users" WHERE "users"."id" = 2028
```

It's a really small thing, but if you want to be a Rails developer long term, thinking about "how does this line of code create performance side effects?" will help a lot. :sunglasses: When in doubt, try things out in the console!


I guess it should be "x + 1 = 4".
Do put spaces in between.


Source: http://9to5google.com/2015/06/14/how-to-make-a-chrome-extensions/


hier nur zum teil eingefügt, damit klar wird es wird sich nicht nur auf das bezogen, später kommt dazu eh mehr


initialize the map above when setting up APIGroupInfo{...}, and just add the particular subresource here, to avoid accidentally stomping other registered subresources


I don't think this is fatal... I think we would just not add the /scale subresource


```user.is_superuser = claims.get('is_superuser', False)```


hier nochmal die beispiele dafür was wir meinen, und am anfang wurde auch das mit dem notwendig umformuliert, hoffe dass es so klarer ist.


LGTM


Hmm, copy from HOME directory in /etc/ovh.conf and all worked!  VPS created, docker installed! Cool!
Try again $HOME/.ovh.conf - not working.
Great thanks!


außerdem ein absatz der nochmal erklärt wie das jetzt ist mit gebrauchswert und anerkennung :)


The problem are only lawyers which argue about everything. The convention is to list the years of copyrightable works explicitly in the headers. Only the first year makes not much sense as the last year counts.


You need to add the files Holmes, no git repo.


besser erklärt wie das mit maßlos bei anerkennung ist. wenn das nicht ganz ausreicht könnte man auch nochmal als fazit einfach dazuschreiben dass das dann auch maßlos ist, aber eben nur aus dem anerkennungsinteresse.


The `buildingCookieValue` was being reset every time a new header was parsed. But the cookies aren't created until the end of the headers being parsed. So unless the cookie header was the very last header parsed, which is the case in Chrome, they were always empty.


hier nur das konkretisiert wie das bei der notenkonkurrenz ist, ist keine große änderung aber da haben wir vorher einfach was falsches behauptet.


Nit: Should we change the name to AppendUnprocessedBuffer()?


much ( ͡°           ͜ʖ           ͡°) space


hier klargemacht dass konsum nicht das einzige feld ist, aber eben eines wo diese naerkennung gesucht wird. fußnote bringt andere beispiele die außerhalb des konsums liegen.


hier werden andere beispiele gebracht die über den konsum funktionieren.


Neat!

Does it need a `name` (as opposed to `wof:name`)?


Since Headers are string pairs now, was saying that the keys of the props call should be strings


die fußnote, die einfach andere sachen aufzählt wie leute sich anerkennung holen. auf jeden fall besser beispiele bzw. mehr gesucht, also füg ein wenn dir auffällt!


Your linter should probably be pointing out to you when a comment tag isn't immediately followed by a space - "# "


I think it should be "x + 1 = 4".
Do leave spaces in between to match the output.


Some game defined a function pointer for an EXT that has the same name as the function (like glGenFramebuffers ) and it conflict with the one from glshim. So I mangled everything to avoid that (I could have limited to common EXT functions, but once I started, I did everything).


Laura Muñoz Parejo
Maria Peñate Garrido


Also, still needs a hierarchy.

@nvkelso - `wof:name` is the standard/default name key. `name:STUFF` is for language/usage specific versions.


@g105b this new value means that the library will also work with php8 and 9 etc (i.e. php doesn't follow semantic versioning and won't be backward breaking).  Is that correct?


>No it isn't and neither did I imply otherwise. In PCREs a question-mark turns a greedy, match as many chars as possible, into a non-greedy, match the minimum necessary, pattern.

Neat, I did not know that. Thanks!

>Adding the equal-sign to the excluded char class works just as well so it's a coin-toss in this situation which to use.

I've implemented that so I guess the coin came up class.


Sorry put you under pressure meyskens. ;)
yeah I hope fix that. If Cast become 100% compatible with Shoutcast V2 as announced, will be working fine. Get better soon. Tks for while.


Looks good!!


@01Cyp Tu penseras à ajouter cbouisso@polytech-lille.net à tes addresses mail sur GitHub parce qu'il te reconnait pas ^^


Closed.

Old prototype code from 43d2eff reverted.


i wouldn't count on that. needs one of you magic `sed` commands

On Wed, Mar 2, 2016 at 6:55 PM, Pierre Fenoll <notifications@github.com>
wrote:

> Since Headers are string pairs now, was saying that the keys of the props
> call should be strings
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/2600hz/kazoo/commit/98d3081c0c4dc87b2970455e5286b9b6b3f8f94d#commitcomment-16454166>
> .
>


It's surely more complete than my simple macro. I'll switch eventually to your version


may be it can be more permessive.
```python
        error_element = find_xpath_attr(meta, _x('.//smil:ref'), 'src', 'http://link.theplatform.com/s/errorFiles/Unavailable.mp4')
        if error_element:
            raise ExtractorError(error_element.attrib['abstract'], expected=True)
```
it work when i remove this:
```python
         assert re.match(r'^[a-zA-Z_-]+$', key)
         if val:
             assert re.match(r'^[a-zA-Z0-9@\s:._-]*$', val)
```


fix-window-frame-calcs-5095 (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1012 of 1013 tests passed, 0 test warnings)

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


Looks good


Will an interface be implemented that could possibly be extended


Is this adding one or multiple pallets?


Do I have to change the code in parse-callback.js to catch the surprise callback error?


Minor - "to check for overflow"
(word "signed" can be dropped now).


I'm still not sure I understand the purpose of this.


Also, do you know what field types are created when using a `boolean` column type?


How do i make the config?


112796613_DesicDehum (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1574 of 1574 tests passed, 0 test warnings)

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


What's example URL?


Will address it in the next checkin.


Name has to be changed because it was a conflict with filename.


fix-window-frame-calcs-5095 (nealkruis) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


I mean user **password**!


I thought about this as well, but it could be dangerous.  There is no guarantee that `b` responds to `<=`.  `b`  could be some atrocity like `class X; def <(x); x + 1; end; end`

Does Rubocop currently do any comparison operator modification?  I couldn't find any, but I am very new to the code base.


No, we didn't test it. Full discussion is at https://support.process-one.net/browse/EJAB-1480


I thought about this as well, but it could be dangerous. There is no guarantee that b responds to <=. b could be some atrocity like class X; def <(x); x + 1; end; end

Does Rubocop currently do any comparison operator modification? I couldn't find any, but I am very new to the code base.


@LuisPB7 what's so funny? xD


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


fix-window-frame-calcs-5095 (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2158 of 2159 tests passed, 488 test warnings)

 * 488 tests had: AUD diffs.
 * 482 tests had: RDD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.

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


`updateTimestamp` would be a better name


merge


Why did you change the delay?


for example: http://www.fox.com/watch/633183811552/7684109568
related to #8584
there two other errors that doesn't catched before:
`The signature is invalid.`
`This content requires a valid, unexpired auth token.`


I think we should change the TPCH connector to only return DECIMAL of the correct size.  If a test or user wants different types, they can either use a cast or write the data to a new store.  Once we make this change, this commit should be a lot smaller.


Use ComponentAdapter?


This is identical to:
```cpp
#if defined(_WIN32) && (ZSTD_DLL_EXPORT==1)
```
Because undefined symbols expand to `0`


Looks good.


зачем проверять на NSCocoaErrorDomain + NSUserCancelledError? это может быть вообще любая операция отмены
этот метод можно правильно реализовать если выпилить условие про `[self eln_matchesDomain:NSCocoaErrorDomain code:NSUserCancelledError];`, но тогда смысла особого нет в этом методе — можно и руками это все написать


This is really interesting. Did you find a resource suggesting this approach? I'm curious if it would be cleaner to hijack an existing resource for the test.


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


This deserves some comment explaining why this code exists


sure thing feel free

On Wed, Mar 2, 2016 at 8:30 PM, Nyaundi Brian <notifications@github.com>
wrote:

> such a nice app, would you love contributions?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Onikah/ShareACoke/commit/1b6ba92829e773048bc2cd6c6df381e09b16395c#commitcomment-16451894>
> .
>

--
Regards,
M.K


ah gotcha


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 169](https://teamcity.labkey.org/viewLog.html?buildId=307200&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:46


:+1: Should we also allow images with an https suffix?


parameter name is a bit confusing, imho. Some proposals: logFileFn, fileNameFn, logFileNameMap, logFileTransform, etc.


- так як -  _**оскільки**_

- визначена _**та**_ ініціалізована

- викличе помилку - _**призведе до помилки**_

- _безпосередньо_

- це ж - _**те саме**_


Miss spaces between operators~


Ah yes. We might be able to do it with a lambda that generates the error message:

```java
checkOverflow(decimal, precision, () -> format(...))
```

We'd need to benchmark to see whether the VM is smart enough to avoid creating the lambda object (not likely, unfortunately).

I like the descriptive error messages, btw.


Ah, good point.


This seems good.


This seems good too.


ARCK, T_T don't break the line before brackets


I agree that we should certainly include all aggregates supported directly by SqlAlchemy, and even some common ones which are not defined in the `func` namespace (e.g., `STDDEV`): "Note that any name not known to `func` generates the function name as is." (http://docs.sqlalchemy.org/en/latest/core/functions.html). I think that comprehensively addressing this issue is already out of scope of this PR, though, and we should open a separate issue (I can do that).


Nope, but I will add it.


Your linter should be throwing you some sort of style error about not having a blank line at the end of a file.


It has to be >=7.0.0 for now until CircleCI correct their PHP versioning. They're a bit behind with PHP 7 releases, and ^7 doesn't match the pre-release version they are using.


Again, there should be a space after the "#", and your linter should probably be catching that for you.


Unfortunately, I can't add this. My users don't want to display the filename by default. I realize now there are other filename references in there I need to amend.


good stuff! i have some time today, give me some feature requests :+1: i'll work on them.


Is ModBlocks.pool right for the open crate and the spreader?


Put the Syntax directory inside %APPDATA%\Sublime Text 3\Packages\Markdown Extended


@barbieri, I know, but unfortunately we can't at present: https://github.com/solettaproject/soletta/pull/1507#issuecomment-190962769


I changed the method name, and added a Precondition check at the beginning of the method


uai


I've never used an URLConnection based client on Android. I'm using a Mozilla browser which provides the possibility to set the proxy settings and to import a pem file by an add-on.
You could try to set `withAllowLocalOnly(false)` in the Bootstrap and use the proxy with an external client.
I use logback-android-classic-1.1.1-2.jar, logback-android-core-1.1.1-2.jar to get log files.


Sort of. "Variadic bound" was a hack to be able to represent row types, which were modeled as parametric types with an unknown number of arguments. This is something we'll remove in the future, so we shouldn't use it for anything else.

Your suggestion is exactly how I'd do it.


I disagree with this. Key navigation would more intuitively be on a slideshow than on page to page navigation. 'Plus most people wont know about them' applies equally to slideshow or page to page navigation. This is an issue 'we might have several slideshow on the same page'. But they are unlikely to be viewable at the same time. Still an issue however.


What changes did you make to this file?


https://github.com/sarabatash/ProgrammingAssignment2-1.git
This is my project, please review it.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 129](https://teamcity.labkey.org/viewLog.html?buildId=307207&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:51


please rename commit name


Ok, primeiro quanto ao nome da funçao. É pra começar com urf mesmo, tipo, pra ficar uniforme com o trabalho q a gente fez antes sacas? E outra, tu já testou?
No mais é só isso mesmo.
Pra mudar é só tu adicionar os commits a branch que tu tá usando pra comparar com a minha, blz, nem precisa dar um pull request de novo se eu não me engano.


Simple updates.


@elchicovzl  si haces esto la aplicacion es menos configurable. Creo que sabes el porq


It's good that you're writing your tests, but recall in class the usage of the `@pytest.mark.parametrize()` decorator. How could you refactor your tests so that you don't need to write a completely new test function for every single set of a, b, and c for which you want to test?


This should be in the exported below that lock the mutex functions, not in the function to get a new address. The reason for their separation is that wallet often makes multiple calls to the getnewaddress function when creating a transaction, and then only writes to the db if the transaction succeeds. It should NOT write to the db if the transaction fails, which this commit causes it to do.


Did you intentionally remove the next assertion?


As long as we have tests for `DecimalType.parse()`, we should be good.


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


Ceci va marcher mais le mieux c'est d'attacher un évènement PrePersist à l'entity Compte pour la date de création.


i added the comment on the wrong commit, this is related to `key` regex and the error actually happen because of the `val` regex.


Add a newline at the end of the file :)


Yes, I had actually done that before, but I think I must have reverted it, so I wanted to make sure it was in this PR before it went in. It closes the full requirement as I understood it in the buganizer bug, but if it needs to also go into another tar anyway, I'll send you another one soon.


@elchicovzl Este pano marker es una libreria. Hay que buscar la forma de sacarlo del repositorio nuestro


112259371_UnitHeaterHWCoilUASizing (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2127 of 2129 tests passed, 0 test warnings)

 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

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


Having both `kube_config_config` and `kubeconfig_config` seems like a recipe for confusion.

Could we rename the first to `kubernetes_config` or something clearer?


You probably want to update the example to include `remark-html`.

And, don’t forget to change `'md'` to `'html'` on the last line.


I think you should exit before doing "--help". By the way, direct comparison between arrays at C and C++ have undefined behavior, you must consider use strcmp or other alternative to do that. C++ have string objects and using == with overloading operators [works fine](http://stackoverflow.com/questions/6222583/how-to-compare-strings), but argv[] is a primitive vector of char arrays and is better [use strcmp](http://stackoverflow.com/questions/1330550/c-compare-char-array-with-string). That way you don't receive warning from compiler~


* Removed `/g_tauntWhileMoving`

* Removed the `1` (random gloat/flourish/taunt anim) option for `g_moreTaunts`; what was formerly the `2` option is now the only option if it is enabled.

* Fixed bug with taunting in vehicles breaking weapons.


This seems like the parameter specifically for uploading.
Is there any parameter than is more generic (not just upload)?


У більшості С-подібних мовах ВОНИ створюються там, де їх оголошують (тавтологія.)

Після проте коми не потрібно

Місце створення ваших змін залежить від того, як ви їх оголосили


:sob: that brackets


I also cannot add this. I don't use Lightbox at all. I think it provides a bad user experience. It also does not make sense to me to launch Lightbox from a slideshow.

Is there a reason to move the class to the li element rather than the a element? Users already using the plugin will have to write new CSS potentially based on this change.


you should move defining @true_answers to private method like

```ruby

private
def true_answers
  @true_answers ||= Answer.where(some cases)
end
```

and call it as `true_answers`


:mask:


I suppose val assert may be relaxed according to https://www.w3.org/TR/xml/#NT-AttValue.


^ The double space is triggering my OCD.  :grimacing:


:-1: ~BLAME BUTTON ACTIVATED


:D

Mvh.
Michael

On Wed, Mar 2, 2016 at 9:19 AM, Thomas Vriens <notifications@github.com>
wrote:

> looks good, I'm gonna check the source in VS a bit later. It's getting a
> bit hard to keep a good overview from the website.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LordMike/TMDbLib/commit/3b7273330104fdd1fc3b62ba2fb986e44ed3abb0#commitcomment-16438914>
> .
>


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


You can probably use [`vfile-location`](https://github.com/wooorm/vfile-location/) here!


use coffee string interpolation like "#{some_variable}/finish_test"


Quotes needed, or doesn't matter?


It's actually properties and fields that are our biggest offenders.


The following doc(s) have been deployed to the [aeligature](http://aeligature-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/wordpress/cron-for-wordpress](http://aeligature-static-docs.pantheon.io/docs/articles/wordpress/cron-for-wordpress)


I know the feeling of Realm sadness :disappointed: Sounds like a good work around. I'll get to it this evening.


#FixMe


name needs to be changed to Liron Lavi Turkenich


you can create a class called Cache which deal with operations for Cache and use key as parameter.


utACK


Please add a line instead of writing over someone else's line


just keep in mind the bits_* functions depend on tack.c and return a newly allocated string which needs to be freed.


Ok, I'll remove putting the packet back to the queue.
The full merge of communicator branch coming soon, changing _a lot_ of things... I'll try and get the test coverage up before that though.


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


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


Thanks. It seems, though, like Windows only echos while reading... so if you type before reading, it won't get echo'd if you use ReadKey(true).  In contrast, on Unix it'll echo even without reading.  At least, that's what I've observed.


No need to specify `'htmlContent': ''`.

Also: why not create an [`image`](https://github.com/wooorm/mdast#image) node? Then, you don’t need to specify all attributes.


looks good


Sure, I'll rename it.


please create a class Database and in this class, never print anything. Just return value and throw error.


Wondering what is better, the unicode emoji as alt (think screen-readers), or the gemoji shortcode? Maybe the first?


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


Pas d'objectifs particulier, si ce n'est qu'à cet instant t, c'est la seule manière que je connais pour l'autoloading. Je me conformerai dès que j'aurai un peu étudier comment tu fais.


:+1:  so it works with any ordering, now? (:


please throw error instead of printing it.


Any clue what the problem is?


@delroth: Any update? Are you getting notified?


Rafael Olmo Cejudo
Miguel Merelo Hernández


Note: I think the `title` should be the gemoji shortcode, but the `alt` should be unicode emoji


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


I like this idea of adding a check like this, but I think it should actually check if it's a stream or not.
```js
if (!sinks[key] || typeof sinks[key].observe !== `function`) {
  ...
}
```

What do you think?


create a class called Command, and put start into that class.


`filter_` is unused?