I see, you need the `.`. Yeah, I just wanted to understand what it was we were wanting to change here as Ulli and I went through a fair bit of tweaking with these lines so I just wanted know what was wrong. It seems every platform has a different name for libboost_python. Thanks for clarifying.


Nothing changed, just looking.


I made this edit in github by clicking the pencil icon. Seemed easier than round-tripping a trivial change. I should have also used a fixup-style commit message to facilitate squashin'. I'll do that next time.


How's the loading doing it this way?


I had already fixed this in https://github.com/grpc/grpc-go/pull/508 too.


Cool - good to know :+1:


Hey, how can I implement use this API with an android application?
Thanks :)


It wasn't a Domino,  just another stupid site that responded with 200 for everything. I put a string in db_404strings but it didn't fix the domino plugin... so looked in and saw no is_404 call.


Any compiler _could_, but I'm under the impression that in practice no compiler _will_ if you have optimizations turned off — which is likely to be the case in a debug build.

I'm not a C++ expert so I might be missing something, but I couldn't find any other way to make this work than to rely on an implicit argument cast, which leaves no room for doing anything at all before `operator Debuggable()` is called.  All other viable options require C++11 features, which we cannot currently use.


Yo ! Tu pourrais mettre à jour les screens du soft ?


Nice sinon !


be careful, fix repository class mess here


My memory is that abbr is missing a fast-path check for "exact same abbreviation already exists" which in principle ought to be very cheap. I should add that.


nit: formatting ` = ` and ` ? `


Thanks, @markelog. :) I didn't have time to properly investigate last time.


you can move this constant calculation to the training step


Code is really more cleaner!


@Marplex you can run the API on a server and then make requests to it via http(s), or if you are brave enough you can try to implement a NodeJS environment on Android and then embed it in you app. Any feedback is appreciated!


If/else in same line is hard to read, which performance taken only add one/two more lines?


```diff
- Alien::Base::ModuleBuild->neW(
+ Alien::Base::ModuleBuild->new(
```


Public User Switches visible guidelines


++ other than the above typo


Nice! Code is going really better! Sulakore will be a really good Framework, and i think this SWF Cracker maybe can be a Part of Sulakore, like a plugin...

But this SWF Cracker is really awesome! Features that no one have.

Good luck.


This is causing `df -k /system` return results like `518.8M` and failed the patching on genymotion. Why are we dropping 'busybox' here?


You should use the CppMicroServices specific platform macro


Yes, for sure. A timeStack of a single metric should generate multiple timeShifted versions of it. That's the behavior of the python version as well.


s/Gnu/GNU/


It's going to be OK!  Did you notice that I put one of them directly into the html, so that's one less line of JS.  :)


Jess needs to remove this line


s/arry/array/


Closes #36
TIT
INI
ICI
T= Transistor
I=Iron
N=Number Pad
C=T1 Microchip


This could be set to `<%= homepage %>`

which is set at https://github.com/puppetlabs/puppet-agent/blob/master/configs/projects/puppet-agent.rb#L67


s/for m/form/


What should this expand to? Will this be something like `windows-x64` or is it going to be more specific like `windows-2012r2-x64`?


I wasn't sure that comments were supported.


What does product code expand to? Will it be `puppet-agent`? Will it be a string of numbers?


OK, that sounds like a reasonable goal.

If it's possible to use the `bookworm serve` protocol instead of adding another script, that would be my preference; it makes it more likely any future code refactoring will pull this along with it. I suspect that the weight associated with running CGIHTTP server over manually setting the port isn't much compared to running a full Apache server. If it's desirable to turn off the two web sites also served that way, we could do it with a flag I guess.

On the security thing; I'm wrong. Actually, it looks like I fixed this a while ago, so it should be querying as the global user if the function `bookwormDB.configuration.Configfile("global")` returns a configuration file for one.


xml fixed


ОМГ какой комит попал на проверку... Более поздние были уже гораздо ровнее. Может отмена пулреквеста тогда и новый с двумя нормальными коммитами? В одном исправленный код, во втором картинки?


Any way to silence this once I am aware of the deprecation?  :) I'm happy with the default theme.


Hard to say compared to ExtendedInfo. But I think it's okay. It's nearly instantly visible. Just the actor images take a little bit longer if they aren't cached yet.


will do the needful over the weekend. This was a task on my plate ..

On Fri, Jan 29, 2016 at 5:53 AM, Sumedh Wale <notifications@github.com>
wrote:

> @ahshahid <https://github.com/ahshahid> we need to rationalize changes to
> Spark. Are you sure this is a Spark bug? If so then need to file a bug in
> Apache Spark and propose this fix via a PR. The snappy-spark repo is now
> public (and will make it a public fork soon).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SnappyDataInc/snappy-spark/commit/d10b123e49abfa7993f9a1a83b56944ec4458697#commitcomment-15762131>
> .
>


yes, as far as I could understand it is a bug .

On Fri, Jan 29, 2016 at 6:24 AM, Sumedh Wale <notifications@github.com>
wrote:

> @ahshahid <https://github.com/ahshahid> also in future, all spark layer
> additions need a SnappyData modification header comment in the file.
> Correcting that for the files I am fixing for scalastyle bugs we introduced.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SnappyDataInc/snappy-spark/commit/552ab0b5a3c83cf6aeafa044e072c82f03c1b149#commitcomment-15762735>
> .
>


I was notified by a server owner that they were getting a crash that happened about once a day with the unofficial builds. From his crash log, turned out that there was a NPE for the `isGC()` method that I added. It looks like some mods might actually have custom tile-entity classes ?! Safety check implemented which


FixExteriorShadingSurfacesSolar (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (927 of 927 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-25.87%25-green.svg)


Interesting thanks.


these speed values need to multiplied by `-1` correct?


@cjryan, can we use v1 instead of v1beta1?


In `Unmount`, you explicitly dispose of the package, here you don't. Would the package be disposed of automatically by `Clear()`, and why wouldn't it by `RemoveAll()`?


If you add a `@property` decorator to these methods, then you don't need to change `managers` to `managers()` in the rest of the code.


Ah dang, looks like _data_ is always nil here, so this throws an error on any jpeg extraction, I think.


`<img src="../images/photofour.jpg" alt="Mom"/>` should be `<img src="photofour.jpg" alt="Mom"/>` since all jpg files and this html file is in the same folder :)

Stephen


Let's keep the new line indentation at 4 spaces (1 tab) for CSS properties

Stephen


Yeah, when you activate the editor, a 2 part form appears at the bottom. You can summarize and explicate before clicking Commit.

It's stealth ui. My eyes skip over it all the time.


Is targetdir the option that users can set? So, for current installs, the default suggestion is `C:\Program Files\Puppet Labs\Puppet`? We'll probably want to set the id as a variable in puppet-agent so we can reference it elsewhere. We're going to want a whole host of wix specific settings like that...


Why do you revert this?


Both `<title>, <meta>` should be inside of `<head>` of any one HTML page. Think of `<head>` as the place where you gather resources and info for the HTML page that are not SHOWN to the users. What's going to be displayed is dealt inside of `<body>`

Stephen


just one `<!DOCTYPE html>`


Yeah, this totally works, and jQuery can make it much more succinct. I was just being a smart ass.


What I meant was that I wasn't sure that comments were supported in a `.dockerignore` file.


Remove this if you don't use this styling :+1:


Do we need factory with 2 methods?

The first method is the same with the second when the user is not signed in. I think we just need to call the first always on sign-in state change.


:) yeah, I just got that. Looking it up now. You may be right.

confirming


Seems right to me, I don't know if we are using the throttle of anything else than the api?


Noticed that properties were no longer being refreshed even though the watch was catching them and pinging the refresh endpoint. I believe this is caused by a change to spring-cloud-context but I haven't fully tracked down the root cause. This gets things going again although I can't say I'm a huge fan of it.


im checking


I didn't use hash as it's only called when the game is loaded


Regarding PEP8,  we have to add 2 blank line among global methods or class.


What if I want to trigger onbeforeunload? if i use window.onbeforeunload(); this will causes error. I am not talking about spy on this event.


@jonschlinkert btw, i'm curious what's that `update` that you running? What config and what's the flow?
Is it based only on changes around `verb` using `.verb*`-files?


@voddan Please take a look.


This could be because looksparam got some new entries. Maybe @Kyle873 should investigate.


@bocharsky-bw I realized by seeing the challenges, that it was very confusing when you needed to check if things were *not* true - my head was spinning :). So now, we have `assertTrue` and `assertFalse` - I think it's much more clear


Hey Alex,

I'd say this is a good start! Remember to pay attention to code indentation and spacing, use all lower case file names (because some operating systems do not understand the difference between `style.css` and `StYle.css` for eg).

See you on the next assignment :100:

Stephen


TeamCity pull requests :: netty-tcnative [Build 50](https://garage.netty.io/teamcity/viewLog.html?buildId=5213&buildTypeId=pulls_netty_tcnative&guest=1) is now running


TeamCity pull requests :: netty-tcnative [Build 50](https://garage.netty.io/teamcity/viewLog.html?buildId=5213&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:44


Is this equivalent to petal conversion dye exchanges?


error is an optional double pointer parameter on the method, so it'd have to be the caller passing in a double pointer (if they're interested in what the error was)


`wordWrap` is not an unused params and shouldn't be removed; see the following discussion https://github.com/tangrams/tangram/pull/205. `text_wrap` defaults to `true`, wrapping does not occur if set to false. It is misleading because not yet on the documentation (https://github.com/tangrams/tangram-docs/blob/1de3818aeb8d5e50ebd418d65f83d9227601ea47/pages/draw.md#align) and does not seem to be parsed either from `TextStyle`.


vm_list talbe would lack following 5 columns. https://github.com/ntt-sic/masakari/blob/master/masakari-controller/db/create_vmha_database.sql#L35-L39


The following doc(s) have been deployed to the [1030config](http://1030config-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/plan-configurations](http://1030config-static-docs.pantheon.io/docs/articles/sites/plan-configurations)
- [source/docs/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/e2380ddeb29d57084ef494867938868ddaf8cb1c/source/docs/assets/css/main.css)


reserve_list table would lack following 5 tables. https://github.com/ntt-sic/masakari/blob/master/masakari-controller/db/create_vmha_database.sql#L54-L58


Oh, OK.


TeamCity pull requests :: netty-tcnative [Build 51](https://garage.netty.io/teamcity/viewLog.html?buildId=5214&buildTypeId=pulls_netty_tcnative&guest=1) is now running


this link tag should go inside of `<head>` tag

Stephen


@chundongwang because not all recoveries are busybox-based anymore.
What is the exact failure that is caused?


`score` is a variable local to the `match` method. It's assigned either 0 or 1 depending on the cards in `otherCards`. It's used to return a value and then thrown away afterward.

Since the method returns either 0 or 1, we can simply return 1 when some `card.contents` matches `self.contents` and return 0 otherwise. The `score` variable isn't important.


Sorry, it was added, but not correctly.

Packaging.props != packaging.props


Tellement un copieur qu'il en oublie de changer les strings ! :D


TeamCity pull requests :: netty-tcnative [Build 51](https://garage.netty.io/teamcity/viewLog.html?buildId=5214&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:50


no need to use 2 `<style>` blocks in an HTML page. Well, no need to use a `<style>` block at all anymore since you already have an external `style.css` for styling :smile:

Stephen


I agree that that is indeed better English, however this naming matches Apple's notification/event naming conventions, all of their delegate methods look like:

-somethingWillHappen
-somethingDidHappen

In general I try to keep the naming and style within the ADAL ObjC code as close to Apple platform conventions as possible.

i.e. UIApplicationDelegateProtocol: https://developer.apple.com/library/prerelease/ios/documentation/UIKit/Reference/UIApplicationDelegate_Protocol/index.html


I think we can delete create_vmha_database.sql since we have create_tables.py.


Because, as stated in the commit description, it was missing leading to SSL context leak.

If you see a double free you must be freeing it yourself.

Dnia 29.01.2016 o godz. 19:25 Jacob Marble <notifications@github.com> napisał(a):

> @rraptorr please on why this was needed. I am using SSL by setting evhtp->ssl_ctx manually, and this snippet induces a double-free.
>
> More in #207
>
> —
> Reply to this email directly or view it on GitHub.
>


Why do you add g++ in a matrix (why do you even need g++) ?
matrix are used to for testing many configurations


You didn't defined TRAVIS_BUILD_NUMBER


If  currentInversion is -1 then it will be


Can we use const for NetworkTag and NetworkAccessTag?


Yes


if you're not use these tags, you can remove them.

Stephen


Why are the bower.json files not alphabetized?  when I do bower --save mine are always alphabetized and causes changes in the file that I don't want and stupid merge conflicts in my PR's.  Can we make sure these are alphabetized in the future?


I sincerely think the more readable version is to use the getter Obj-C implicitly makes for a property. Developers expect to see `card.chosen` instead of `card.isChosen`.


no need to use `<br>` anymore and instead, you can use either `padding` or `margin` to space things now.

Stephen


Why did you change from the standard semantic versioning system to using the date?


Nice! I'm once more drowning in work and other stuff, leaving way too little time for side fun projects :(


It's worth noting that if you pass an integer as the value in the conversion options, Guzzle will throw an exception: `"Invalid resource type: integer"`. I was able to get around this by ensuring all the values I give are strings, but this should probably either be documented or handled by casting all values to strings.

Otherwise this is now working. Thanks :smile:


FYI: it's sometimes useful to hold the convention that the first parent of each merge is the previous version of the branch (and not the other way around like in this merge). Of course, there's very little consequence in this case, as the merge base is very close.


>KVKYES< is not a proper form of comment in CSS. You can remove this otherwise, it will mess with your styling.

Stephen


Why do you need such an extra statement in a catch handler?


I got it. LGTM


merge


Nice! Definitely, it's more readable )


`find` might work too, I simply used `findstr` because I generally prefer it due to its capabilities.

Any simplifications are welcome.


désolé...


technically also when you reload (a) CEG(s); while the perf gain is minor it doesn't hurt either


Can you point me to the build failure?


I solved my problem. I used jsoup to retrieve the Classeviva dashboard. Anyway, thanks for the reply. Ps : (Anche se penso che tu sia italiano ti scrivo in inglese comunque :)


Why? design is assigned in both if and else branch.


Since this is basically some error, this should probably not only show up in ```verbose``` logging.


of the design


doesn't match parameter name.


Typo --> is basically


Eventhough this is a wrapper for a C++ function, parameter names should probably follow our python naming convention, i.e. ```under_score``` variable names and not ```camelCase```.


This is a bunch of code I copied and pasted in here in the order the code would have been executed before, which is why it's not terrible pretty. I'll move _complete up in this case. This particular bit of code is some of the stuff that was duct taped on as part of the broker work, I've cleaned up most but not all of it so far.

The one case where _complete is not set on all these if checks is when we get hit with a pkeyauth challenge. In that case we're still going to have farther to go before the request is complete, similar to if none of the if checks are hit. I'll add a comment to that effect in the code.


I thought it would be enough just to iterate line by line in which case `for` alone would be sufficient.


Sorry, bad decision :8ball:


The deployment should be in a after_success section


Fixed in 6fec28e04313161b0d330fc3a4d3dfa9f85e53f9.


The original data features and its description included in this projectCodebook was provided by:
==================================================================
Human Activity Recognition Using Smartphones Dataset
Version 1.0
==================================================================
Jorge L. Reyes-Ortiz, Davide Anguita, Alessandro Ghio, Luca Oneto.
Smartlab - Non Linear Complex Systems Laboratory
DITEN - Università degli Studi di Genova.
Via Opera Pia 11A, I-16145, Genoa, Italy.
activityrecognition@smartlab.ws
www.smartlab.ws
==================================================================


Ouai pas de soucis. J'avais pas la dernière version du programme au moment
où j'ai fait la présentation
Le 29 janv. 2016 7:37 PM, "Théo HYVON" <notifications@github.com> a écrit :

> Nice sinon !
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Nakio195/GBF_Project/commit/b314c6e87f590630c0b6ae680fc8aec295dd7462#commitcomment-15769078>
> .
>


hahahahaha Daniel :heart_eyes:


ah, clever! this way when `NSCache` purges it, it's also gone from the map ✅


nice!


There are several problems I can already identify. Should fix ASAP:

- The scheduler is _slow_ as ass. Need to improve it.
- The user programs can manipulate the scheduler too much.
- The scheduler algorithm is very primitive and locks up a ton.
- Need some tests.
- The timer used to preempt processes is not calibrated and is interrupting way too often.


@TheMightyWarPig oh sorry, I missed it above, nevermind!


Cherry-picked in 2fd9d7bf9243e2dd71f9624bf6665e194a935984.


It's just more code, and I'll assume you didn't update all the classes with CHECK_MEMBER_INFO, since it's a pain in the arse :)


Argh, I'm missing links here…


They weren't already, can you make a PR that does this so #1163 doesn't have that and makes it simpler?


Hey Kelley,

Good job, I like it! A few things

- Do all of your styling in your `css` file only and don't use a `<style>` block anymore. You would want to use `<style>` block in your html file for very specific reasons nowadays. One main application would be when you are writing html/css for emails. Writing emails in html/css for a wide audience has very tight restrictions and to get over those restrictions, developers would use a style block in their html page so that users don't have to load another file (css).

- Don't leave unused code in your code since every character counts as data.

- use lower case characters for file names, class names and id names. and don't leave spaces if the name is more than one word. Dashes or underscores should be used for naming.

:+1:

Stephen


I did? Maybe I branched from wrong revision? I am on 2.1. the only changes I need in CmakeList.txt files are:

+####################
 8   +subdirs( cura )
 9   +subdirs( icons )
 10   +subdirs( plugins )
 11   +subdirs( resources)
 12   +####################


@matthew-dean Honestly I'm not sure I properly follow your recent comment.
But yes:
>unless you are saying there are other types parsed as functions but converted to another node type, like Color.

w/o my commit, this:
```less
.clearfix {
    length(1 2 3);
    extract(1 + 3, 1);
    %("quoted");
    %(~"unquoted");
    extract(keyword, 1);
    extract(url(http://foo.bar), 1);
    extract({foo: bar}, 1);
    @l: 1 2, 3 4;
    extract(@l,    1); // space list, tree.Expression
    extract(@l @l, 1); // comma list, tree.Value
    e("whatever");
}
```
will compile to
```css
.clearfix {
  3
  4
  "quoted"
  unquoted
  keyword
  url(http://foo.bar)

  1 2
  1 2, 3 4
  whatever
}
```
>That's why I added the check for tree.Color, because it wasn't catching it as a function call.

`rgb()` (and other CSS color funcs)  *is* catching as a function call, and it's a plain Less function call returning a value of type `tree.Color` and that's why it can appear there and we need to blacklist it.
Same way `length()` is a function call returning `tree.Dimension`, same for all those types that can be returned by functions listed in the example (9 or so types), and thus they all should be blacklisted explicitly.


If you mean the `delete`, because I don't use std::unique_ptr at the moment. A hidden reason being that a side goal of mine is to almost never use pointers, by using policy-based design instead. Unfortunately this is not always possible when relying on external libs. So I keep track of heap memory without smart pointer, and can get criticize for it :)


should be `var properties = flockProperties[i]` ?


it looks like these are the same source files used for jerasure_neon. let's build them as an object library so we don't have to compile twice. it'll also avoid the ${CMAKE_SOURCE_DIR} stuff for pointing at files outside this subdirectory


Excuse me, this was the first time someone added `SSL_CTX_free()` to the `evhtp_free` function, not the second. I'll ping the person who added the second one.


@karimnaaji ok, reverting this change. This comparison of bool < int looked like a bug and I didn't found other places where wordWrap was used.

so it should be if (wordWrap && _params.maxLineWidth < _params.text.length()) - right?


Haha, if I wasnt using jquery already (for preserving scroll location on refresh) I think I would have tried it regularly.

Granted, I'm impressed with how easy it was in hindsight.  Thanks again for the tips yesterday.


LGTM. mentioned one slightly wording change on slack


Unix case sensitivity apparently.  None of the CIs are running MSBuild on unix today, they build the IL on windows then run the tests on unix. Simple fix here is to change the casing of the import.


@coreyfloyd this makes me a little uneasy.  keeping this isolated from the other feed constants makes it easy to forget to increase it after raising other limits (or if other limits are added).  this number should never be less than the maximum capacity of the feed, otherwise we could get into a situation where we're constantly purging section controllers when the feed is still below capacity.

the intent of this limit is to _eventually_ purge controllers which haven't been used recently.  for example, in the case where users start spamming the "save" button and refreshing explore so that there's a large influx (and outflux) of sections, at which point we'd want new section controllers to stay and old ones to be "pushed" out.  speaking of which, `NSCache` doesn't guarantee anything:

> This is not a strict limit—if the cache goes over the limit, an object in the cache could be evicted instantly, later, or possibly never, depending on the implementation details of the cache.

It's better than nothing, though, and we can roll our own, stricter LRU cache if this becomes a known performance problem.


@adamel please comment on why this was needed. I am using SSL by setting `evhtp->ssl_ctx` manually, and this snippet induces a double-free, since a second call to `SSL_CTX_free(evhtp->ssl_ctx)` is made about 20 lines later.

More in #207


Thanks for the kind words; I'm happy to help.

Hm, I changed from ~> to the current form here: https://github.com/GoogleCloudPlatform/fluent-plugin-google-cloud/commit/4aefa97fb13d2af703cf21cad59ce9050275ae21

So, this was ostensibly to quash a rubygems warning, but I tried today with a few versions of rubygems (2.2, 2.4, 2.5) with the ~> format and did not see any warnings, so it may have just been a mistake on my part - as you said, it should be equivalent.


related to #706


Отлично. Принимаю)


sounds dirty


you need to fetch `"rotation"` as well because you use it in the edit loop to mix with the target direction.


I'll merge this then since it seems to need the v1beta1 tag.


Oh. Weird. Ohhh so all Less functions that return.... GOT IT. And of course those checks need to happen because of functions that produce any disallowed values. Ok, not I understand.


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


Promises rulen!


I am unsure about the status for [the usage of smart pointers](https://en.wikipedia.org/wiki/Smart_pointer#unique_ptr "a smart pointer") also for this software. Are there any developments in the waiting queue?


It is actually parsed from https://github.com/tangrams/tangram-es/blob/master/core/src/scene/styleParam.cpp#L128, and the test from https://github.com/tangrams/tangram-es/commit/28a6ffb0f9d3d49c354844d82beaa8807aedd2f3#diff-aa423a445ea70ee6600a4bb30d503beaL70 has the same behavior :+1:


We should add a `Vec3::distance2` which returns the distance squared.  You could then compare it against a `MIN_POSITION_CHANGE_FOR_UPDATE_SQUARED` to do this calculation without invoking a costly square root.


include connection class file


Finally someone recognizes my hilarious writing.


```sql
DELETE FROM `creature_involvedrelation` WHERE `quest` = 10842;
INSERT INTO `creature_involvedrelation` (`id`, `quest`) VALUES ('22278', '10842');
```


I'm not clear on the distinction between mix and slerp, but you may want the latter here.


Sorry, it's hard to say without knowing what the app does. You do seem to be having low frame rates, so I'd suspect something afoul. Question is, is it something on my side or yours?


You need to check whether n<2 firstly. 1,0,-1,-2... are not prime numbers.


I don't understand you question. You have two nicely separate commits, so you can make two pull branches and two pull requests. I'm not going to explain how to use Git or GitHub, if that's where the problem is.


You can remove this line since you added `font-family` line with a new font. :+1:

Stephen


We should add a `Vec3::distance2` which returns the distance squared.  You could then compare it against a `MAX_SIGHT_DISTANCE_SQUARED` to do this calculation without invoking a costly square root.

Adding it in this PR is strictly optional.


env=os.environ


no joke that I was going to make this exact change earlier today. :+1: I am recently strongly against prefixing functions with _. Underscore prefixes serve as discards and I have to do a triple-take every time I encounter functions starting with  prefix.


use maxmin[0] and maxmin[1] directly. do not use max and min. You could try input {0,9,-9,1,-1}


Closes #2.


Same thing as separation...   using a Vec3::distance2 would eliminate a square root operation.  Less critical here since this is in the O(n) part, not the O(n^2) part.


FYI, if you wanted to center this body automatically.

```css
.body {
    width: 90%;
    margin: 0 auto;
}
```
Something like this would've centered this `div.body` perfectly. Also in general, naming a `<div>` with the name 'body' is not recommended as it can get confusing with the default HTML tag `<body>`

Stephen


How does the parser know to create an Execute object?


Actually since `text_wrap` is either a boolean or an integer value, everything is directly inside _params.maxLineWidth, which is maxIntValue when `text_wrap=false` and the default of `15` when set to true. It seems to be all good here!


@omarsotillo Gay


It looks like MongoEngine has an [`EmailField()`](http://docs.mongoengine.org/en/latest/apireference.html#mongoengine.fields.EmailField) that automatically checks that the input is a valid email address. Should we use that here?


really digging this `<nav>`


let me get the source for you when I get home.


Maybe also add a negative test where you try to execute a prepared statement that doesn't exist?


LGTM.


>I checked and found the needed changes in the SDK docs. When I checked for BoundingBix there were so many that I was unsure exactly how it was being used. I did try to add "ref" without success, so I'll be interested to see how it was added.

See https://msdn.microsoft.com/en-us/library/14akc2c7.aspx for an explanation of passing by reference

>You didn't get the error for ThrustProfiler?

I screwed up the commit.

>Is Sandbox.ModAPI.Interfaces.TerminalPropertyExtensions
just for modifying existing?

I don't understand the question.


it's OK. but for such changes it's better to ask Laurynas because it may cause a problem on merging from upstream


Hola loquinho33/IC-FICH-2015,

Tienes un mensaje de Renato Manzo en IQ Elite.

Haz clic aquí a.iqelite.com/j/roFQnM/4ut/?hg=xaV2CN9bAl para leer el mensaje

El equipo de IQ Elite

---------------------------------------------------------------------------
Este mensaje estaba destinado a reply@reply.github.com. Si no deseas seguir recibiendo mensajes de IQ Elite, haz clic en este enlace: a.iqelite.com/common/unsubscribe/?code=d79e9400&email=reply@reply.github.com.&hg=xaV2CN9bAl IQ Elite es un servicio con oficina principal en: C/ Gran de Gràcia 15, 2º 1ª, 08012 Barcelona, España.
---------------------------------------------------------------------------


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-MacOS-10.9-clang: OK (2042 of 2050 tests passed, 3 test warnings)

 * 5 tests had: AUD diffs.
 * 10 tests had: EIO diffs.
 * 6 tests had: ERR diffs.
 * 8 tests had: ESO big diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTD diffs.
 * 2 tests had: RDD diffs.
 * 7 tests had: Table big diffs.
 * 3 tests had: ESO small diffs.
 * 3 tests had: MTR small diffs.
 * 1 test had: Table small diffs.
 * 4 tests had: MTR big diffs.

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


What are your thoughts on using CSS property "will-change" on a Samsara surface?
https://developers.google.com/web/fundamentals/performance/rendering/simplify-paint-complexity-and-reduce-paint-areas?hl=en


not in this way. what if input is 2999? your algorithm just have one loop. if the sumNumber is still more than one digits, your code will not continue to add them. so actually you need nest loop


Remove console logs if you're done with them.


Better spacing would be,

```css
<div class="body">
    <h1>My Portfolio</h1>
    <div class="work">
        <p>INSERT WORK HERE</p>
    </div>
</div>
```
Notice no `one line space` between html tags and 4 spaces to represent an html element is a part of its parent element.

The reason you don't want random empty spaces and following a clean indentation guidelines in your code is for 1. readability 2. in HTML every whitespace and new line matters, some older browsers may understand your new empty line as `<br>` for example.

Stephen


Man... In English please. Siva will check your code sometimes.


Ah, I think I just forgot to mention that I'd probably suggest `2.6.0` for this release (it's not that there were some really major changes, but amount of those `reference` (and other) fixes done by @SomMeri is quite noticeable to deserve higher number :) If this won't bother you too much of course.


Wait, if we're not actively removing something from the `QueryStateMachine`, then why are we doing a round trip to the server? Why not just update the session on the client?


Hi @oyahiroki -- thanks for the feedback. The topic has been updated to note that depending on your version of Eclipse, the menu options might differ: https://hub.jazz.net/docs/jazz_scm_client/#connect_to_your_devops_services_projects_from_eclipse.


Hey Phil,

I see you put a lot of thought into this, esp with your CSS only navbar. Sweet stuff. Pay more attention to cleaning up your code, I'm pretty happy about what you have here. :+1:

Stephen


the module is composed by a rom memory to storage the coefficients and a logic part to make the operation


Ignore.


@lencioni, @janpaul123 et al - since renaming a prop won't cause uses of the old prop to fail, what do you think about adding some code that explicitly throws an error in case someone is using the old prop name? Or do you think bumping the minor release number and mentioning it in the release notes is enough?


This test documented behavior not tested elsewhere. It was odd, yes, but it was not *unnecessary* per se.

A less obscure way to test the exact same thing:
```python
def test_undo_does_not_raise(self):
    self.controller.undo_move()
```


@Rynchodon

Also removed all gui.* files from mods folder after build
Compiles and runs

2016-01-29 14:32:22.053 - Thread:   1 ->  MyScriptManager.LoadData() - START
2016-01-29 14:32:23.789 - Thread:   1 ->     Registered modules from: _StatLogic, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null
2016-01-29 14:32:23.789 - Thread:   1 ->     Script loaded: _StatLogic, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null
2016-01-29 14:32:23.789 - Thread:   1 ->     Registered modules from: ARMS Dev_Autopilot, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null
2016-01-29 14:32:23.794 - Thread:   1 ->     Script loaded: ARMS Dev_Autopilot, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null
2016-01-29 14:32:23.794 - Thread:   1 ->  MyScriptManager.LoadData() - END


@mattbenjamin are these magic numbers needed? if so, maybe #define them?


Sorry for not responding for a while,
I would rename ConnectionError before overwriting it.

It would be good to catch both ConnectionError and OSError and rethrow it with our own exception. Consider the following code:

```
c = mpd.MPDClient()
c.connect("not.existing.host", 6600)
```

This will throw:

```
socket.gaierror: [Errno -2] Name or service not known
```

which inherits from OSError not ConnectionError.


@ericstj is correct.

```
git mv Packaging.props packaging.props
```

Is all that is needed.  Also, this happens just doing ./build.sh on ubuntu.  Why are none of the CIs doing a full end to end build?  Thats a bit nuts.


mistake!


nice.

Stephen


Two other minuscule side notes:
- The inside function is a bit over-indented (code formatting)
- `logger.exception` automatically captures the traceback + message; `logger.exception(e.message)` would therefore be a bit redundant. You can add a separate message in its place.


Hi Tobi,

Thank you for reaching out. You bring up some valid concerns and I'd like to clarify where these updates came from and some of the thought that went into it.

Even though it perhaps isn't clear from the documentation, the `description` and `additional` fields have always supported html. Even before Lever supported fully rich text postings, we supported including links and other simplified markup. With the introduction of full rich text support, now even more styling options have become available.

Once we did introduce full rich text support, there were some companies integrating with the API that wanted to be able to access a plain text version of additional and description. In order to support this need, we introduced two additional API fields for postings `descriptionPlain` and `additionalPlain`. Now companies have a choice in whether they want to use the html version or a plain text version.

We realize these changes in succession have been confusing. We learned the hard way that some companies believed that the `description` and `additional` fields were serving plain text because they never used the previously existing rich text functionality in Lever. Now that rich text is much more discoverable to recruiters, this is causing some conflict. Updating the `description` and `additional` fields to always return plain text, unfortunately was not an option, because many of our customers had built job sites that relied on the html in `description` and `additional` (even before full rich text support was launched). We're working on having clearer documentation to avoid confusions like this in the future and really appreciate that you've taken the time to share with us how frustrating this process has been for you.

Please let me know how I might be able to help you further or if I can offer further clarification on the recent updates Lever has made to our app and API. And thank you for your patience as we learn how to best make all our updates as seamless as possible :)

Best wishes,
Rachael


The CIs run msbuild on windows, they only run the native portion of the build on ubuntu.  Native portion isn't broken by this.  The reason they don't run msbuild portion on ubuntu was due to reliability issues with mono. /cc @mmitche @joperezr


> Why are none of the CIs doing a full end to end build?

The build wasn't stable enough in CI.  We should have this up-and-running soon, though, based on MSBuild on CoreCLR now that it's running on Linux and OS X as of very recently.


FixExteriorShadingSurfacesSolar (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1465 of 1465 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.52%25-green.svg)


Oh lol I just asked you this very question. Yes, I'll do that. There's a syntactical change: "or" keyword.


Yeah, I think throwing an error or warning is a good idea.


So I don't know how to change the `setup` for testing (a). But I wrote a test and clear `Picture` to be empty like this:
```ruby
  test "can access index with no images saved" do
    Picture.delete_all

    get :index
    assert_response :success
    assert_equal 'Please upload some images first.', flash[:notice]
    assert_select '#picture_list .image', 0
  end
```
Is it acceptable? Or for changing the setup, can you give me any hint? Actually I don't know how to both set `@pictures` to be both empty and not empty.


Thinking an improvement of this function with the existing one on the branch:
```go
// NodeMatchesNodeSelectorTerms checks if a node's labels satisfy a list of node selector terms,
// terms are ORed, and an emtpy a list of terms will match nothing.
func NodeMatchesNodeSelectorTerms(node *api.Node, nodeSelectorTerms []api.NodeSelectorTerm) bool {
	for _, req := range nodeSelectorTerms {
		nodeSelector, err := api.NodeSelectorRequirementsAsSelector(req.MatchExpressions)
		if err != nil {
			glog.V(10).Infof("Failed to parse MatchExpressions: %+v, regarding as not match.", req.MatchExpressions)
			return false
		}
		if nodeSelector.Matches(labels.Set(node.Labels)) {
			return true
		}
	}
	return false
}
```
The pasted here one may be faster than the one on the branch, e.g. in the case matching result of the terms are [true, false, false, false].
The only difference "might exist" between the two version is the err handling, for example the matching result of the terms are [true, false, false, err]: the on branch version will return false, while the pasted here one will return true. But since all the value of terms are validated in validation, this case actually does not exist.


After this https://github.com/Particular/NServiceBus/pull/3310 gets merged in accessing `IMessageSerializer` would be close to impossible (cc @danielmarbach ). How about copying the bits you need to ASQ repo if the purpose is only backwards compat?


Hey Chantel,

I like it very much! The really only thing I see is you can try grouping shared styles together in your `style.css`

so for example,
```css
ol {
	padding: 0;
}

ul {
	padding: 0;
}
```

can be simply,

```css
ul, ol {
    padding: 0;
}
```

Other than that, sweet looking page, and sweet gif :wink:

Stephen


Would be better to the `MYSQL_INCLUDE_DIR`to be named after the Hunter package, correct? Either `MYSQL-CLIENT_INCLUDE_DIR` or whichever name is chosen to this package.


thanks for this, i am the one that emailed asking about it.


Trivial, this line should be removed.


Same as above: name `MYSQL_LIBRARY` according to Hunter package?


For some reason, Infuser was refusing to work because it did not receive a list. This forces the data frame to be a list.


1. Why don't you just redirect to right page ("/topics/{topicId}")? You do the same work with the opening "/topics/{topicId}".

2. If you decide to solve the problem in this way.
All code in this method (except  Topic topic = pluginTopicService.get(topicId, topicType);) is duplicate of
@RequestMapping(value = "/topics/{topicId}", method = RequestMethod.GET)
public ModelAndView showTopicPage........

You need to create a new method and put duplicate code there.


You may use name.setf().


Always linux?


Hey very awesome work! I could also use some of what you did. Would you be okay with merging this into the project?


Están pendientes los cambios del CUS Registrar Cierre de Caja


Changes should be made to this file, sorry, I tried many package names and the Find script didn't get fully refactored to reflect that


sbl.set("s")...


Same here


Since the other selector is named SELappDidBecomeActiveNotification, it might make more sense to move the detect clipboard logic into that method instead of adding it as another selector. An alternative is to name the SELappDidBecomeActiveNotification selector to something more specific to private tab state since I believe that is all that it does right now. Either or is fine.


Nota bene: you can just do `[Ariel Viña Rodríguez](http://qpcr4vir.github.io/)`


Ah, wait, no this was commented out on my computer as a test at some point. Shouldn't have committed that.


This reversion is due to a string of happenings:

1. [httr updated](https://github.com/hadley/httr/releases/tag/v1.1.0)
2. googlesheets passed text data to content and then transformed it into a data frame, but got caught on an error when `na.strings` was not a valid argument in httr 1.1.0
3. This was fixed by piping directly to `readr::read_csv()` as described in this pull request: https://github.com/jennybc/googlesheets/pull/191


Looks good, thanks!


This isn't a rowMap =)


Hey, thanks! I have been thinking about submitting a pull request, but there are still a couple of small things I'd like to do. I'll revert the copyright on the license back to you and I also want to check the JSON file for accuracy and to maybe add some more agents. I'll give you an update by the end of this weekend, if not sooner.


incorrect indent in this block


Should this be Math.max? Same for the `py_top` section


I think to be safe we want to move this to viewDidAppear instead of viewDidLoad in case we want to show the banner within this method since we want to show the banner after the BVC has loaded AND is on the screen.


Well, I'm not sure if I get you right.

Catching `ConnectionError` will not catch any other `OSError`. But catching `OSError` should catch every `ConnectionError` as `ConnectionError` inherits from `OSError`.

https://docs.python.org/3.5/library/exceptions.html#os-exceptions

Or did I miss something?


extra lines


How would you feel about `check_out` and `check_in` instead? As I understand it, these are new APIs so it might be relatively low-cost to change, if we would want to.

Since these functions just return `:ok`, I suspect they're intended to be verbs, and then that would be the correct form. A name like `checkout` suggests to me that it returns some noun rather than perform an action. More: http://thepugautomatic.com/2014/08/login-setup/


Starting the project.


@lukeolson fyi--comments?


Thanks! There are further not-well-formed errors, unfortunately:

https://github.com/IDPF/scriptable-components/blob/PubCoder/Samples/3-messaging_component/embedded/PubCoder/Quizzes/OEBPS/content.opf#L129
`<link href="components/PubCoder/Quiz1/styles.css">`

https://github.com/IDPF/scriptable-components/blob/PubCoder/Samples/3-messaging_component/embedded/PubCoder/Quizzes/OEBPS/content.opf#L247
`<link href="components/PubCoder/Quiz2/styles.css">`

https://github.com/IDPF/scriptable-components/blob/PubCoder/Samples/3-messaging_component/embedded/PubCoder/Quizzes/OEBPS/content.opf#L306
`<link href="components/PubCoder/QuizResult/styles.css">`


Awesome, I'll switch the helo script to reference this marker's position.


Please indent this +2.


LGTM.


This is great. Probably better to add a PR to the upstream date picker.


Change the variable name here too.


Nice, looking forward to this content!


for now you can trick it and add `let g:loaded_airline_theme = 1` to your vimrc.  i'll put on a better solution this weekend.


Sorry, not `py_top` but `py_bottom`


Another reminder, update once you get a reply from David.


You shouldn't need this anymore


使ってないです。他のものと統一します。


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


I think this line fixes a bug in the old code...


Checked using MinGW-w64 from MSYS2 project (https://msys2.github.io/) and all dependencies on Windows 7 x64:

- gcc version 5.3.0 (Rev1, Built by MSYS2 project), Target: x86_64-w64-mingw32, Thread model: posix
- ncurses 6.0.20150627-2
- winpthread 5.0.0.4573.628fdbf-1 (libpthread.a and libwinpthread.a compare binary same)
- AMD APP SDK 3.0
- AMD ADL SDK 9
- curl 7.46.0-2

Configured with:

export AMDAPPSDKROOT=/c/AMDAPPSDK/3.0
NOCONFIGURE=true ./autogen.sh
./configure --enable-opencl

<pre>------------------------------------------------------------------------
cgminer 3.7.2
------------------------------------------------------------------------

Configuration Options Summary:

  libcurl(GBT+getwork).: Enabled: -LV:/msys64/mingw64/lib -lcurl
  curses.TUI...........: FOUND: -lncurses
  OpenCL...............: FOUND. GPU mining support enabled
  scrypt...............: Disabled
  ADL..................: SDK found, GPU monitoring support enabled

  Avalon.ASICs.........: Disabled
  BFL.ASICs............: Disabled
  KnC.ASICs............: Disabled
  BitForce.FPGAs.......: Disabled
  BitFury.ASICs........: Disabled
  Hashfast.ASICs.......: Disabled
  BlackArrow.ASICs.....: Disabled
  Icarus.ASICs/FPGAs...: Disabled
  Klondike.ASICs.......: Disabled
  ModMiner.FPGAs.......: Disabled

Compilation............: make (or gmake)
  CPPFLAGS.............:
  CFLAGS...............: -g -O2
  LDFLAGS..............:
  LDADD................:  -LV:/msys64/mingw64/lib -lcurl compat/jansson-2.5/src/.libs/libjansson.a -lpthread -L/c/AMDAPPSDK/3.0/lib/x86_64 -lOpenCL   -lws2_32 -lm

Installation...........: make install (as root if needed, with 'su' or 'sudo')
  prefix...............: /usr/local
</pre>


Made more seamless with a zsh function. Auto-populates the folders map and sites map.


@dumganhar


新しくここ一つコメント入れよ


Made more seamless with zsh function.


This hoop-jumping, it is voluntary?


@Jesus-Allen
你更6 lol


I have found the answer.


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


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


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


@c0s I think, there seemed to be some problems while rebasing. So PR #76 has been indicated another tree. We can see it here; https://github.com/apache/bigtop/pull/76


ok


"idempotent"は「更新によって内容がまったく変わらない」(notが付いているので「まったく変わらないわけではない」）、"more or less equivalent"も「更新で大して内容が変わらない」ということで、これらを並列してproblematicに掛けているのですかね。ただ、そう解釈するには、2つ目の"if"の前に"and"が欲しいですね。
いずれにせよ、どうしてそれがproblematicになるのか見当がつかないのが悩みの種です。


很好奇这里为什么要用 函数？ 与直接写10 有什么区别吗？


Yeah it should be _name_. Can you change it?

On Thursday, January 28, 2016, Andy Wu <notifications@github.com> wrote:

> hey @david-truong <https://github.com/david-truong> @gamerson
> <https://github.com/gamerson> why did you change this to *package* ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/david-truong/liferay-blade-tools/commit/72bcfc407e288d38f423d0326b6aa6033f99d0d3#commitcomment-15750888>
> .
>

--
Greg Amerson
Liferay Developer Tools
Liferay, Inc. www.liferay.com


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


Of course not. It's just more expedient than the alternative and a by-product of using the WOF S3 bucket for hosting both the data and the WOF website itself.

The `data` directory is where the raw GeoJSON files are stored and I would prefer not to fill it up with extra ancillary bits like HTML files. But we do want to be able to point people to `wof.mz.com/data/` so we set up a rewrite rule to load the `all-the-data.html` file from that URL.

So, short of absolute paths and a local web server, it means that the paths for the CSS don't square up which is annoying but hardly the end of the world and since none of these things get updated with any degree of frequency it was just determined to be a poison we could live with and got on with other things.

Also this: https://whosonfirst.mapzen.com/data#knownknown-s3


Yes, that is OK. I was just making sure and giving a heads up in the
repositories I could check in between classes today.

Regards,
Esteban
On Jan 28, 2016 8:22 PM, "jjc21" <notifications@github.com> wrote:

> no i was currently still working until the due date tonight at 12. Is that
> ok?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jjc21/OracleJava/commit/23d07d7b50ebd3b8d0c616d5eced76f53b1cabde#commitcomment-15750481>
> .
>


:mag_right: Commit reference #283


グループ見るときも、$loadedかまさなきゃダメだよ。広島より


Correct.

On Thu, Jan 28, 2016, 2:47 PM Jonathan <notifications@github.com> wrote:

> Does this mean only sprinter is linked in the bin directory?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/toumorokoshi/sprinter/commit/c26a8c59d895f5efccb6f3019831dfaecbce2f61#commitcomment-15747927>
> .
>


:+1: :star: :smile:


sure


it seems we don't need to expose render frame ID to java world. this is the implementation details that should be kept in C++ world, we'd keep the java code as simple as possible if we could.


ditto.


For future runs. The old links from previous runs are still there.
On Thu, Jan 28, 2016, 6:14 PM Yusuke Tsutsumi <tsutsumi.yusuke@gmail.com>
wrote:

> Correct.
>
> On Thu, Jan 28, 2016, 2:47 PM Jonathan <notifications@github.com> wrote:
>
>> Does this mean only sprinter is linked in the bin directory?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/toumorokoshi/sprinter/commit/c26a8c59d895f5efccb6f3019831dfaecbce2f61#commitcomment-15747927>
>> .
>>
>


not quite sure why we implement it like this, would you please elaborate?


Ah, rewrites. I was wonder why I couldn’t get to `all-the-data.html` on the live site.


use a display ID or screen ID managed by C++ codes instead of the raw render process ID and frame ID could be a better idea.


use None here (immutable)


This looks ok.


v == 0


e == 0


https://whosonfirst.mapzen.com/all-the-data.html <-- works for me


Looks ok.


maybe it's not a good name, as this class represent the host screen of the presentation which initiate the remote presentation screen, could we call it XWalkPresentationHost? or any other suitable one?


All others are lowercase, let's make that consistent and be able to match the server error?


You have the same method inside of `ProductsHelper`, is that right?


I saw that you have some troubles with your editor, because there is no newline at the end of the files.
Try to add `.editorconfig`.


This change is really geared towards new players and simplifying the UX.
Coupling the map making tools with game releases I think is a mistake.
I'd rather drive people to the tripleA website that could then have links to these tools with documentation on how to use the tools.


Yes, just go ahead and grade that.


Don't do this, you also messed up the package again. OI is to be left to @Megapead and I to edit, as we'll have special functions for each intake area. Just leave them with placeholder values for now please @EthChil


Regarding this error: "Not a valid path. This is an unexpected error. If it happens again please report it."  Please change to:

"Not a valid path.  If this problem persists, please contact Ultimaker support."


并改成“当”吧。


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 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.69%25-green.svg)


test

gclub


good job


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


IMHO you can move this method to another class, maybe a `VideoService` and put all logic related to Videos inside it.