TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.541.83eb80eec2409dbaa8e76cd0c9212a4569e73270](http://build.palaso.org/viewLog.html?buildId=64611&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3337, ignored: 130 Build time: 00:04:57


Hi kramester,

We've pushed a new commit to correct these bugs: https://github.com/zync/zync-nuke/commit/956d94ed4bc6f3d6fb7b0d72ba348c5bf9a9bdf0


> Granted, the loss of precision is quite small

Well, wait, I guess that's more subjective on Javascript with its 53 bit Integers.


PIMPL не работает с unique_ptr - ему нужна декларация класса для вызова деструктора.

> On Oct 20, 2015, at 09:03, ygorshenin <notifications@github.com> wrote:
>
> unique_ptr?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/deathbaba/omim/commit/73d6f851e8cd71c925a7bd87ef12f1d0487720e4#commitcomment-13879558>.
>


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2323 of 2365 tests passed)

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


Why do you reduce the number of uses? Does that mean I can 'heal' the machine just picking it up and dropping it a few times?
I'd rather see the "recalculate" code be refactored out of the (I guess) "machineUsed" code, and called explicitly.

(Also, could you please document the machineUsed function while you are at it? You must have looked at it to understand you can call it; you might as well type a few lines of text explaining what it does at that time, saving time for everybody. Thank you.)


I picked the last one so it would not change the output from what we currently have. When the next stable release happen, we can bump it to a stable release sha-1.


I'll :sunflower: around it and just describe the two modes of operation. Default doesn't matter.


This is causing a crash.


Это не ошибка, если файла нет. Просто пользователь еще не брался за редактор.
Но вообще, надо разделять эти случаи и скорее всего удалять "плохой" xml.


This reads nicely.


Nein. Wieso? OO


@brev I've created #424 with the changes to the  `mocha.opts` files.
Let me know if it works for you


This widget creates a fancy table view showing candidates for your meetups. It can operate in one of two modes:

* Live update - table refreshes every time new candidates are identified in the stream
* Click to update - table shows the number of new candidates but only shows them when the user clicks the number


Fixed.


Please don't mix different things in one commit. This is strictly not related to the announced change.


ДА


git blame pls


Okay. So the function is called at the end?

Does that mean I can remove 5-15 as they are redundant?

On Tue, Oct 20, 2015 at 9:29 AM, Victor Miclovich <notifications@github.com>
wrote:

> var app = angular.module('madLibs', ['ngAnimate']);
>     app.controller('Answers', ['$scope', function($scope) {
>
>         $scope.showTheForm = false;
>         $scope.showTheBody = true;
>         $scope.forfemaleName = '';
>         $scope.forjobTitle = '';
>         $scope.fortediousTask = '';
>         $scope.fordirtyTask = '';
>         $scope.forcelebrity = '';
>         $scope.foruselessSkill = '';
>         $scope.foradjective = '';
>         $scope.forobCelebrity = '';
>         $scope.forhugeNumber = '';
>
>         $scope.submit = function() {
>             $scope.showTheForm = true;
>             $scope.showTheBody = false;
>             $scope.femaleName = angular.copy($scope.forfemaleName);
>             $scope.jobTitle = angular.copy($scope.forjobTitle);
>             $scope.tediousTask = angular.copy($scope.fortediousTask);
>             $scope.dirtyTask = angular.copy($scope.fordirtyTask);
>             $scope.celebrity = angular.copy($scope.forcelebrity);
>             $scope.uselessSkill = angular.copy($scope.foruselessSkill);
>             $scope.adjective = angular.copy($scope.foradjective);
>             $scope.obCelebrity = angular.copy($scope.forobCelebrity);
>             $scope.hugeNumber = angular.copy($scope.forhugeNumber);
>         };
>
>         $scope.startOver = function() {
>             console.log("Function called");
>             $scope.showTheForm = false;
>             $scope.showTheBody = true;
>             $scope.forfemaleName = '';
>             $scope.forjobTitle = '';
>             $scope.fortediousTask = '';
>             $scope.fordirtyTask = '';
>             $scope.forcelebrity = '';
>             $scope.foruselessSkill = '';
>             $scope.foradjective = '';
>             $scope.forobCelebrity = '';
>             $scope.forhugeNumber = '';
>         };
>         $scope.startOver()
> }]);
>
> See how startOver() is just running on page load
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/isvictorious/thinkful-madlibs/commit/ef8980d47c2fe92fc735f690be9d988d4f27d84d#commitcomment-13874669>
> .
>

--
Victor(ious) Ramirez
@isvictorious
isvictorious.com
<http://isvictorious.com/?utm_source=gmail&utm_medium=email&utm_campaign=brandedurl>
P 347 586 9670


Also the amount of characters that shows on board is dependent on the size now.


Venues's search bar will disappear after this commit


Я бы сперва проверил isValid(), только после этого лез бы за саундами в JNI.


OK, let's try to avoid skipping code generation.
I'll take a look on this issue now and make it work just like running tests with sol-fbp-runner


r+


`s/klämma ihop/mosa/`!


Hi @rbgarga Have any of you had any experience with PFSense and Auto Scaling in AWS ?http://goo.gl/FuPi0u


why this change?

I believe the rule for ./relative includes is that they are checked relative to the path of the current file first, so as long as the referenced file exists in that location, there is no danger of including something random above another include dir.


The issue mentioned above is now fixed in e89a10dd4794c0a8ae09b56012affe50bd35417c. Please merge my pull request.

The breakage with sys.maxint was due to a local issue where Unauthenticated was raised during traversal because z3c.form sometimes updates widgets already during traversal.


duplicate if line 205-208


Looks good!  I'm going to suggest holding off on a PR for the moment, while we're still sorting out the Outreachy applications and there are lots of people looking at this issues, but this looks like a good fix to me.  I particularly appreciate that you made the effort to clean up the structure of the existing code rather than just fixing the immediate issue.  I like that kind of anti-entropic coding!  I don't have any criticisms to make: what you've done is good and clear.


You made a mistake on one of the Memory bars in Bridge.js:
You did divide the Max ram and the Ram usage values but you forgot to Multiply it by 100.. (* 100)
document.getElementById("RAMpercent").style.width=((status.ram/status.maxram * 100) + "%");

Also some server statistics don't work.


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This is fantastic. This issue has plagued us from the beginning!


I guess GitHub doesn't impose a limit on issue titles (what we call summary). Here's a goofy test: https://github.com/webcompat/webcompat-tests/issues/277

256 is probably OK here.


I guess restarting is fine. I just don't want people to have to be copying folders and consensus data and whatnot


It's still considered a race because line 32 is reading a variable that might be written by another goroutine at line 38.
E.g., https://play.golang.org/p/-QaqyrUlVP, if you run `go run -race` with this example, go would complain about data races.


Cool!

I was thinking that when it's time to work on this, we should write some cuke tests to prove that the redirects work nicely.


Thought about it, but I wrap only functions which don't pause anyway. Another option was to just inline context + loop, but decided to go the easy way.


where best_priority used?


It seems that `mirage describe --dot` is broken anyway... but ok, I'll implement a new `Funtoria_graph.pp` function.


Why don't you just call remap twice?


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/e36f316974ab56998a7743bf918790282b0be2f2


merging MonsieurNicolas/stellar-core/formatWarn = e36f3169 into auto


MonsieurNicolas/stellar-core/formatWarn = e36f3169 merged ok, testing candidate = 36d4e227


To wyglada dziwnie i pozatym "GTK2" czy "GTK3" jest uzywane wszedzie (rowniez w oryginale: https://hg.mozilla.org/mozilla-central/file/f7b746b4e913/dom/locales/en-US/chrome/layout/xbl.properties#l7). GTK+ 2 moze i jest poprawna pelna nazwa - jej skrocona forma jest jednak zazwyczaj uzywana.


Да. Так и было сделано раньше. Но сейчас логика изменилась. TurnSound выступает в роли генерации разного рода оповещений. (Не только звуковых.) Т.о. ему не требуется валидность TtsPlayer для их генерации.


No, just warning.


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


Зачем здесь pImpl? К чему эти усложнения.


This looks good!  I'm going to suggest not putting in a PR yet, while the Outreachy selection is still going on and there are other people working on this issue, but this fix looks good to me.


## что сделал


ASSERT() что фича была в мапе?


using и TFeaturesContainer.


Good catch! Fixed here: https://github.com/discourse/discourse/commit/f8ff6fc0b56e9a415d4f0de703a18b5d752b9409


I believe you forgot to reference the thread for cmh at line 1084...Fail Low CMH bonus...


Didn't catch that the additions to standalone.js where only to the whitelist, that's probably ignorant on non-existing modules? And then tracked that back to this commit.


considering description written on json (I didn't check implementation) you should send a "int" packet for any number, so it would be nice to validate if the number for key "float_val" is correct (it's integer representation).


Finit


I'm fairly certain that the `:jvm-opts` setting only applies when running services using leiningen. Were you intending for all devs to have these logging configuration files on their workstations when testing locally? It appears that these options are already in the Dockerfiles, which will ensure that the `logback.configurationFile` system property is available when services are running in docker containers.


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

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


Whoops!


Oh...I see, you are not testing against the latest (merged) lazy.


6/10 must do better


I thought this was deprecated?


s/found/it/
found больше подходит для булева флага а не для итератора.


@belaban shouldn't this be before prot.init(); like below?


Looks good :)


needs space between `require` and it's argument.


s/found/it/


.... doing this makes it harder to merge ... need cherry pick


Could you rename `okbit32` to `has_bit32` for consistency?


I was not finished with this yet. I committed this morning so that I could continue my work from my office (which is not at home)


I agree, this is a bit hacky.

I'll comment machineUsed & extract some sort of recalculateSmoke method from it.
Leave this PR open & I'll come back to this when I have time.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.518.83eb80eec2409dbaa8e76cd0c9212a4569e73270](http://build.palaso.org/viewLog.html?buildId=64612&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3377, ignored: 124 Build time: 00:22:04


UDP, very cool. Working on a particular use-case?


ASSERT(!fPtr.get(), ("This functor must be called at most once."));


toArray seems implementation specific and not really a standard. Suggest removing this.


Weil wir dann überall den gleichen Stand haben. Die Datei aus dem fastd-Paket kann sich durchaus mal ändern.


defs.py is not only used by the cleaner. It's a general set of HTML tags and comments are not tags.


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1419 of 1419 tests passed)

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


Why cant it be in dllEntry and getmoduleapi?  That is the entry point of when it should happen, no?


please don't name this function `unpack`: it clashes with the standard `unpack` in the base library in lua 5.1
`sunpack` would be fine.


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


Removed because I added it in an earlier commit in this PR & didn't want to be touching a file for no reason other than a 1 line comment.
Can add it back easily enough if you like? :)


It is the same case as here https://forum.nette.org/en/24536-nette-framework-2-3-7-released#p164093. It is better to warn than leave users in a false illusion.


`string.unpack` takes a position as a 3rd argument; which means you can remove the `:sub` call :)


indentation.


This needs to be earlier because of RMG.setInt() above.


indentation


This should all be on the same line, imo. So that it reads
`var dbDir = path.resolve (`

Also I think i prefer the name databaseDir


После разговора договорились добавить комментарий.


Hint: [std::hypot](http://en.cppreference.com/w/cpp/numeric/math/hypot)


Yeah, I do think I should add more to the blockquote element part with a description, since I was writing the content like paragraph blocks for an essay and I picked up steam for writing it midway through. Since this would be made into a website, demonstrating the element in practice according to HTML 4 specification and then HTML 5 specs might be a good way to add onto to that section. There's also how CSS is often used to make them more visually obvious as quotes, which would be how most people "see" blockquotes on a web page..


BclXmlChoiceValues (macumber) - i386-Windows-7-VisualStudio-12: OK (2338 of 2365 tests passed)

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


Hint: [std::atan2](http://en.cppreference.com/w/cpp/numeric/math/atan2) ;)


Эта правка идет в разрез с содержимым комментария.


Very nice link! Great idea to have it open in a new tab.


Не хотел светить всякие XML и прочие потроха реализации. Это не нужно знать использующему коду.


Reference BaEvCo93 and GasBar95 are not in siminos.bib


handai-trema/cbench-yamatchan.git に権限をください(切実)


А ее может и не быть - код валиден. Это и "обнуление" отредактированной фичи, и добавление в список фич "удаленной" фичи одновременно.


> On Oct 20, 2015, at 9:01 AM, jzmaddock <notifications@github.com> wrote:
>
>
> > I read this as only using our utf8 codecvt facet only the standard one
> > std::codecvt_utf8<wchar_t> does not exist.
> > I believe that recent versions of MSVC do include this standard header
> > so this issue should never arise in any case.
> >
>
> It appears it's buggy in VC14, we have:
>
> // Bug specific to VC14,
> // See
> https://connect.microsoft.com/VisualStudio/feedback/details/1348277/link-error-when-using-std-codecvt-utf8-utf16-char16-t
> // and discussion here:
> http://blogs.msdn.com/b/vcblog/archive/2014/11/12/visual-studio-2015-preview-now-available.aspx?PageIndex=2
> #if defined(_CPPLIB_VER) && (_CPPLIB_VER == 650)
> # define BOOST_NO_CXX11_HDR_CODECVT
> #endif
>
> In Boost.Config.

LOL - there is not rest for us.

I looked in this some more and I’ve still got issues.

I can add utf_codecvt to the serialization.dll - OK - no problem.

BUT the magic export macro used in the source code header is BOOST_UTF8_DECL not BOOST_WARCHIVE_DECL.

OK that would be fine - BUT I’m using visibility-includes-hidden for GCC platforms so I need to update the source to
our utf8 facet to specify BOOST_UTF8_DECL for each function.  I can do this.  I’m just letting you know in case
I’m missing something else.

Robert Ramey
Robert Ramey Software Development
(805)569-3793
ramey@rrsd.com
www.rrsd.com


You're reading way too much into that comment. On the JVM we use up to 18 digits until the point at which it gets converted to a `Float` using `nearestFloat`, at which point the precision gets truncated.


You don't need to have both, reset and normalize in here. It doesn't really matter, but it's just additional effort you don't need.


Well, I disagree with this change.
`prepareStyles` is not part of a component. My philosophy with it is that it can be used anywhere we have a context regardless whether it's a Material UI component or not. In other words, it should be usable by other developers in any component. Forcing developers to add +20 lines to their components to support RTL is a bit too much in my opinion.
Moreover, clock shouldn't be able to stand on its own. It's an internal component of `<TimePicker>`. It shouldn't have a theme in its state.


Also about the state/context thing. Can't we just make it a mixin?


@bartzon Hi there, quick question for you: was the reasoning behind using the `with_lock` here just to have the state change occur within a transaction? Or was there actually a race condition you were trying to prevent?  Many thanks!


It should be... We need to validate jobs to ensure that they have CPU and memory set on job submission. I will do that at some point today


(And on JS we use up to 16 digits.)


Не идет, Юра. Комментарий был написан к раскомментированной строке ниже, и не был исправлен ранее после комментирования.


If you want to you can shorten this up by saying
`margin: 25px 0;`
That way it knows to apply it to both top and bottom.


Wait, no, 15 digits.


Если кратко - то все правильно в моем варианте сейчас.


Yeah I rather have one of these puppies not two, since your the dog person I'll let you axe one


@scoder thanks for your quick answer :)

@scoder on the scope of the cleaner, do you think my change, is valid  (in terms of feature, setting aside the way I implemented the change) ?

On the `defs.tag`, what would it breaks ?

On a philosophical/ontologic point of view you say "comments are not tags", any doc/pointer to detail on that (I just want more input, I'm far from being XML expert...)


Very nice job! It looks great overall.


Still needed? If so, can 8 minutes be reduced?


cherry-picked into qa/1.x as 0c11f9089f1f02bf80f2e32fc3a0ae95d56dcea4


Do you think the default values for poolSize and poolIdleTimeout will be sufficient? Otherwise, LGTM.


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


LGTM


`separete` -> `separate` and this should be good to go.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.497.7d833d163d0e49c97166eed46c119f0019513a5b](http://build.palaso.org/viewLog.html?buildId=64622&buildTypeId=bt323) outcome was **FAILURE**
Summary: Tests failed: 181 (181 new), passed: 681, ignored: 54 Build time: 00:12:38
### Failed tests
```
Chorus.Tests.dll: Chorus.Tests.ChorusSystemTests.CanMakeNotesBarWithOtherFiles: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.FileHandlers.image.ImageFileTypeHandlerTests.LargeFilesWithSupportedExtension_ShouldNotBeInRepo: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.FileHandlers.ldml.LdmlFileHandlerTests.Find2WayDifferencesShouldReportOneChangeEvenWhenNoneArePresent: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.FileHandlers.ldml.LdmlFileHandlerTests.Find2WayDifferencesShouldReportOneChangeNoMatterHowManyWereMade: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.sync.LargeFileFilterTests.LargeFileInExcludedPathIsNotFilteredOut: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.sync.LargeFileFilterTests.LargeFileInNonExcludedFolderFiltered: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.sync.LargeFileFilterTests.LargeMp3FileIsNotAllowed: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.sync.LargeFileFilterTests.LongAddedFileIsfilteredOut: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.sync.LargeFileFilterTests.LongModifiedFileIsfilteredOut: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.sync.LargeFileFilterTests.LongUnknownFileIsfilteredOut: <no details avaliable>

LibChorus.Tests.dll: LibChorus.Tests.sync.LargeFileFilterTests.LongWavFileIsFilteredOutAndNotInRepo: <no details avaliable>

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


If you do this, the user will expect to be able to specify it by just typing it on the next line as with some other bots, but that will not work here


@josephdburdick The mixin is declared via compass http://compass-style.org/reference/compass/css3/flexbox/. Using the newest spec of flexbox.

Good call on modernizr for the last 10%. I'm going to look at my google analytics and see how many users are coming from older browsers that don't support this implementation and then determine if it's worth adding the .no-flexbox. IE 10 or less is what I'll have to worry about http://caniuse.com/#feat=flexbox.


@mattfordham cool beans. That warning, I think, is one of the blockers that @thenickcox mentioned for upgrading React.


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


Awesome job! I like how you added this with CSS :) +1:


I like your idea to have a definition page. .I think it would be helpful for when we post our full content if we read through each others and marked words that weren't immediately clear so that we could know which need to be defined. Personally, I find that sometimes I get a little too close to my content and what makes sense to me may not make sense to someone else. It's hard to objectively look at how understandable something is when you're writing about a technical subject that you've been studying for a while.


@josephdburdick what are your thoughts on the nesting here? Some people like keeping nesting to a minimum. I really like grouping styles together like indentation in html. Sometimes I go pretty far in the nest though.


So I guess you could lose a tiny bit of precision right on the edge in JS because JS is capable of exactly representing a handful of 16-digit numbers but I ignore any digits after the 15th.

This problem doesn't occur in Java because `Integer` has a couple of extra digits of precision than `Float`.


Reading List App/Reading List App/LoginViewController.swift

stub is still here :P by stub I mean a func that does nothing
Its not a massive deal, but if the only thing in an override is a call to super you should take it out. in this instance less code is better because its less people need to think about when reading.

And if we need viewDidLoad later we can re-add it


That's necessarily going to require an installer executable, right? Isn't that how every other app works?


indentation


You need to explain how this path is used.
It may even be just the link you've sent on commit msg, if it's the best reference we have. But users need to know how to write this path.
It's valid for all options / ports that receive paths.


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

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


Or if you want a 3x3 result (i.e. an outer product) you would pass in a 3x1 array and a 1x3 array. Ex:
```python
In [8]: blackbody_nu([2, 4, 6] * u.mm, [[100], [200], [300]])
Out[8]:
<Quantity [[  6.65791517e-13,  1.69495885e-13,  7.57867358e-14],
           [  1.35596708e-12,  3.42067682e-13,  1.52487609e-13],
           [  2.04624187e-12,  5.14645681e-13,  2.29189708e-13]] erg / (cm2 Hz s sr)>
```


Why it has been reverted?


Vire l'option !


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


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


Nice feature.


CI


please fix all lines like this. there should be an space between the statements e.g.
var expect = require('chai').expect;


the same that above, spaces between the '=' equal symbol


@Sykopompos changed the structure in the parent commit of this, and this is probably why this repo produces errors.
Also, I should notice that my Sprint tester is reporting not working data with this commit, something more is needed for M.
Check the "forked from" repo: https://github.com/MinimalOS-AOSP/proprietary_vendor_lge which I created, it works without problems for N5.
2. No timeservice FCs
3. I found the list of the binaries with the extract-files script. Check https://github.com/MinimalOS-AOSP/android_device_lge_hammerhead/commit/a14bbafa695766aea15098e51b2631b23736426a


try to keep a correct tabulation space for line breaks


spaces between parameters function(err, res) {


the same that above spaces between operators:
console.log('Error Create Project' + JSON.stringify(res.body));


why commented?


the same, spaces...


spaces...


tabulation spaces
if (err) {
	console.log('Error Create Task'+JSON.stringify(res.body));
}


(Note, in the above example the first input's shape is actually just 1-D--`(3,)`--but when broadcasting with a 2-D array it gets promoted automatically to `(3, 1)`.


LiDAR -> Lidar (2x)


issue5221_memoryAllocation (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Вот я и прошу поставить ASSERT() как документацию к коду.


LGTM


Yes, I know: I checked R-devel version of WRE. It said something like "in the ideal world" all standard packages are listed in imports, but did not spell out any enforcement. (Last time I saw this "in the ideal world" was when I was cooking coq au vin, and the recipe told me to marinate the coq in two bottles of Chambertin -- I ignored that, too.)

"Belt and braces"? Shouldn't that be "belt and suspenders" over there across the Pond?


Just noticed: another approach here is `asks (bcStackYaml . getBuildConfig)`


try to keep the spaces in between statements, the majority observations are spaces between statements and parameters, tabulations.

review all files, fix that observations and update again your code


Ok I think there is a little problem in testing it. The order of the eigenvalues must be from the highest to the smallest, but it seems that dti eigenvalues are already ordered in that fashion (or I couldn't find an example where it is not the case).
If it always the case, then we can just delete those lines, otherwise the new reordering is the correct one.


pretty sure these were already fixed


Still slightly confused as to why you're using this instead of `Yaml.encodeFile`, though this is certainly an improvement.


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


Там написано "Don not remove depend_includepath in CONFIG.". А ты ее удаляешь. Идет.


twa ne e ok


yeah my b


Oh sorry, I must have accidentally reverted this commit. Will re-revert it in a sec, thanks for pointing it out!


best commit ever.


ASSERT сделает код невалидным - он будет падать там, где должен работать, в дебаге.


@CallMeAldy first #beerplz


All I'm saying is that if clock is supposed to get theme from `TimePicker`, store it inside clock's `state` and use `state` instead of `context` to access theme. Then we can get rid of the `context` check in `prepareStyles` and be consistent and relatively error-free for the future (many other components that I modified follow this convention). If you still don't like the idea, that's cool.

Umm... because of [this](https://medium.com/@dan_abramov/mixins-are-dead-long-live-higher-order-components-94a0d2f9e750#.1reletofc) maybe


Also we might want to rely on the CreateInstance() extension method since it caches and is hella fast.


Note: this is triggering react-eslint warnings due to this bug:
https://github.com/yannickcr/eslint-plugin-react/issues/259
The bug is fixed (18 hours ago) so once it's incorporated into a patch release of `eslint-plugin-react` we can make these invalid warnings go away.


I recognize that this is a lame workaround.  Anyone know why it doesn't shutdown on its own?


@mrietveld could this commit be the reason of failed tests in kie-remote:
```
Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.494 sec <<< FAILURE! - in org.kie.remote.jaxb.gen.GeneratedClassesCompatibilityTest
roundTripTest(org.kie.remote.jaxb.gen.GeneratedClassesCompatibilityTest)  Time elapsed: 0.226 sec  <<< FAILURE!
java.lang.AssertionError: FireAllRulesCommand fields expected:<4> but was:<3>
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.failNotEquals(Assert.java:743)
	at org.junit.Assert.assertEquals(Assert.java:118)
	at org.junit.Assert.assertEquals(Assert.java:555)
	at org.kie.remote.jaxb.gen.GeneratedClassesCompatibilityTest.roundTripTest(GeneratedClassesCompatibilityTest.java:190)

Running org.kie.remote.jaxb.gen.OriginalClassesSerializationTest
```


andy found them first


I'm not whether or not this is what you talked to Brian about during class, but when you open up your site in the browser and inspect the element, it helps you see where the gaps in your layout are coming from. So here in this case, you could apply a `margin-top: 0;` and that would take away one of the gaps.


10 instead of 1?


Add


issue5221_memoryAllocation (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1896 of 1896 tests passed)

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


Looks cool


I like that you have such a tangible, common practice that you are framing the canvas element around. Through using video game design to provide context to the element, you make it that much easier to understand the element's function. I'm not at all familiar with canvas, so I look forward to seeing where you go with this.

I like that you want to examine the philosophies that browser-based game designers use to design a "good" platform level in canvas and compare this with older practices. Bringing Super Mario Bros into the equation provides  specific instance of canvas contributing to the development of gaming over time and even gives you the opportunity to bring in some of Haas' writing on whether the newer versions with canvas are necessarily better than the older.


Is there a reason the minimum height is 30 dp on Android?


fast-forwarding master to auto = 36d4e227


Ок. Верхние две строчки валидные, третья - нет. Надо исправлять, спасибо.


Ja, du hast Recht, so sieht es im Quellcode aus. "Öffne" wäre hier also richtig.


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


This one isn't in use, is it?


ОК


Here we created an `elasticsearch instance`


Done.


bah pior! nem tinha visto isso, só fui no ctrl+c ctrl+v auhsuahsuh  valeu!


Did you have a question about these? If so, feel free to write me on Slack!


I am confused too.


sdasdsa


Here we pass in an `elasticsearch instance` and then` cfg["cdr_elastic_search"]["index"] == "index_some_string"`


Мне не принципиально, оставь если тебе нравится, но имхо тут перегруз ) Клиенты и так будут читать public API а не какие там private поля у этого класса.


I don't think its good to handle Coopmode on collisionresolver the way it is done now. This implementation requires two extra getters and an extra field. I'd suggest letting CollisionResolver check what kind of Game it is in, and handle accordingly.
Alle these getters and fields will flag inCode (analysis tool)


This is probably a lie :speak_no_evil:


Hey there,
Looking for a way to get in touch with you.  Is the classpathtemplateloader supposed to support loading files from the classpath when they are packaged in a jar file?  This doesn't appear to work in my environment.  I cloned the master branch of your repo and most of the test are failing due to missing files (comments.yml for one example) - are these files available in the repo or is there something other than 'mvn clean test' that needs to be run to successsfully run the tests?  If loading resources from jar files isn't supported now, I'd be happy to add that feature, but I'd like to know the tests all work before I do that.
Thanks,
-Scott


I really like how nicely your code is indented! It makes it so easy to read. Great job on that!


Should be `osx` and `ios` (if supported by the addon).


:heart:


Very nice use of inline-block!


Not sure, but this feels like we are leaking topology into global settings. Also, I am really not crazy about the name and bool. It all covers this one use case but something just feels weird about it.


@LukasReschke nope, just an assumption ;)


It's not really a hard problem. It's just that it's defined backwards given that starting with GHC 7.10 it's even redundant to define `return` at all. So bringing this into normal form just makes the code a bit more future-proof.


++ ah, thats right, totally forgot about those extensions


I think this switch is missing a case for `st_paul_municipal_2015`


Why did you always write "first translation" into GIT commit messages?


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


Confirmations:
- https://packetstormsecurity.com/files/111242/libzip-0.10-Heap-Overflow-Information-Leak.html
- http://www.securityfocus.com/archive/1/522083/30/7170/threaded
- http://www.openwall.com/lists/oss-security/2012/03/21/2


GHX_Vert_IDD (mmatts) - i386-Windows-7-VisualStudio-12: OK (1957 of 1957 tests passed)

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


Yep, noticed that too in the latest commit. My bad. :+1:


Synonyms! :stuck_out_tongue_winking_eye:


Why are you excluding `BatchAutoConfiguration`?


hmm... you don't need to set this per node instance.
You may have it set for the node type (you can declare "extra_methods" on json) setting these specific processes.


Gifs also exploded around the mid-2000's *after* the LZW patent expired in the US in 2003 and globally in 2004. I feel like our historicizing projects can be closely related.


Debug?


I just forked it to report the error, I take absolutely no credit for it. Original code is https://github.com/cgarvis/react-vendor-prefix


sim nao precisa


Doesn't this kind of defeat the purpose of the template approach because you need to manually define them here as `friend`?


That looks great! I love the change of the background color, great idea!


@traversaro I'm always getting false as return value here. Suggestions?


The sql trace changed because the simulation code path now theoretically admits referring to row ids that exist, which means it goes through checking the subsample table via `_crosscat_get_rows`.  Most of the extra queries are from that.  There is also a reordering effect because this path fetches some cached thing (the metadata?) earlier relative to some other queries.


Isn't it always json_object_path_process()?
Anyway, this function is a c & p of json_object_get_key_in_process (only a function call differs). It may be easily re-written as a base function that receives the pointer to the function you need (sol_flow_packet_get_json_array or sol_flow_packet_get_json_object), right?


Good point.  Will move it.


Can you dedent this so we close the file first before reopening it for writing?


I‘m going to ask some questions before further revision.


I'll keep it in mind for future reference.  In this case, the problem is obviated by reducing back to one simulate_joint method.


Thanks @rgmark. We have run the deployment script so the update is now shown on the live site: http://mimic.physionet.org/mimictables/callout/


Since this element is displayed as block it still takes up the entire line. Did you see how the nav is on the right, but still underneath this h1? If you float the h1 left, you'll be able to get them onto the same line.


Perfect, Thanks @ayjee.


TeamCity pull requests :: netty [Build 445](https://garage.netty.io/teamcity/viewLog.html?buildId=2535&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4785, ignored: 21 Build time: 00:30:40


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


I think this text may be a little long to be considered a header, which is what `<h2>` means. Maybe consider making anything longer than a line a `<p>` tag.


Nice job overall!


Archivo


That should be ___iswspace in lib/setup.c, of course.


Done in latest version.


Fixes CSS3 non-validation also; trivial though.


I think it would help if you defined what a lossless data compression algorithm is in simpler terms somewhere in your overview.


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1419 of 1419 tests passed)

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


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


`get_by_slug` will raise an `Election.DoesNotExist`, rather than return `None` if it's not a known election.

More generally, I'd be tempted to remove the `is_valid` method and its only use completely and replace the `self.get` in `get_by_slug` with `get_object_or_404`. (It's also nicer to return a 404 to the user rather than us getting an error email if someone's just misedited the URL, say.)


I think they should definitely work for now. This is a very, very lightly used site atm.


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


1) Maybe name the test testLogContextWithStructuralValues. Alternatively some add the issue number/title into the name, testIssue50PhpErrorsForNonStringContextValues. Looking at the Gelf repo they seem to have a convention of adding the ```@see``` with a link to the issue being fixed.
2) Feels like you're doing too much in one test. Perhaps split using the dataProvider for each type? Would simplify the method.


This should be at the same indentation level as the pipes above and below it.


Thanks :)


We're keeping these on the state. Why not just use that?
```javascript
if (this.state.readOffset == offset) { return; }
let next = this.state.update((next)=> {
  next.readOffset = offset;
  //... the rest of the stuff
});
this.observe(this.state = next);
```


Because that isn't used anywhere.  But I can still add it.


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


i like!


enable gnome shell version 3.18


This does not overwrite anything.  Also I will document that when I write the documentation for the multi-gpu in a later PR.


I tried to compile this *bleep* for a long time before taking it to the [Unix Stack Exchange](https://unix.stackexchange.com/questions/153417/how-to-compile-dcp2icc), and even there I have received zero response. If you have ever compiled this package, please let me know either on Stack Exchange or issue #1.


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


issue5221_memoryAllocation (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (822 of 822 tests passed)

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


Because the message is much clearer.  Otherwise you would get an error like `Context name None is not defined`


Ok.


留言


Once you're ready to commit this, can you hand it to me so I can rewrite this message?


Comment


Sure.  -Alan

On 10/20/2015 12:44 PM, Andrew Melo wrote:
>
> Once you're ready to commit this, can you hand it to me so I can
> rewrite this message?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/accre/lstore-lio/commit/01b0c46858f7cc066797b6abc9b14e6f864a2c26#commitcomment-13881917>.
>


I think you did the same thing here you did last week. Check out my comments from last week to see how to fix that this isn't showing up in the middle for a different size screen.


# Movie Project Version 2.0 19 October 2015

In order to run this program, please run module for entertainment_center.py

You can also run this program by double-clicking fresh_tomatoes.html

### General Notes

- This is a project for Udacity's Full Stack Web Developer Nanodegree program

- The general purpose of this project is to display movie information for
  six movies and play a trailer when the movie poster is clicked.

- Resources used to ensure proper formatting include:

	- [pep8online.com/] (http://pep8online.com/) for proper formatting of "*.py" files

	- [wikihow.com/] (http://www.wikihow.com/) for proper readme formatting

### Changelog

20 October 2015

- Changed readme file to .md from .txt :+1:

- Added notes to entertainment_center.py

19 October 2015

- Fixed formatting, white space, and line length issues.

- Added README file per requirements and to add project clarity

- Added CSS and HTML formatting

	- Changed number of columns from three to two

	- Floated the movie poster to the left to display movie information to the
	  right.

	- Moved movie title from bottom of movie tile to the top of movie
	  information

16 October 2015

- Original submission without any CSS or HTML updates to fresh_tomatoes.py

### Movie Project can be reached at:

Email: james@seniorcaredirectory.com
Website: http://www.seniorcaredirectory.com/


SingleSubtabInMemory (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2322 of 2365 tests passed)

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


very nice use of a pseudo class here! I really like how it turned out :)


I would say less verbose is better in this case:

'The cloud service name "%s" is already in use in another region please choose a different name."


$ Thanks.


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


awesome, thanks


Thank you!

The commit message wants to reflect our discussion, though, don't you think?

I would have expected at least a note that while the documentation at <URL> does not list `none` as valid value, it was actually a valid value, and that the release notes of <version> suggested the change to `lowest` (see <URL>). And actually mention that the change from `none` to `poweruser` was made under the erroneous assumption that specifying an invalid value would fall back to `admin` and that switching it to `poweruser` would actually *weaken* the requirement (not make it more demanding).

Also, please do not forget to [sign off](https://github.com/git/git/blob/v2.6.2/Documentation/SubmittingPatches#L234-L286).


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


Oh, and of course I completely missed that it is not written anywhere whether you actually tested this patch?


Nice job overall!


bb


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


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


OK for `Fl Fl`, but all the examples I found use a comma-separated list of equivalent options, not a pipe-separated one. So I kept the commas in 17d042f613da17a0461301ac11bce0e5ea73edb7.


hello fwiend


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


I said that it was tested in the issue but I can add that to the PR.
On Oct 20, 2015 1:48 PM, "Johannes Schindelin" <notifications@github.com>
wrote:

> Oh, and of course I completely missed that it is not written anywhere
> whether you actually tested this patch?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kjeremy/build-extra/commit/25ddadd761e529d1cdfe54b17705c0ab6dbb1fe9#commitcomment-13882012>
> .
>


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


Hey! I put it there for a purpose! Thats why there is a comment!


@mattfordham This commit breaks the build, because of #341

Specifically, see how Raven is part of many of the ajax routines: https://github.com/WikiEducationFoundation/WikiEduDashboard/blob/master/app/assets/javascripts/utils/api.coffee#L232


Thanks! I will keep this young project alive and kicking.


Agreed we should just let the machine the command is being run on do the resolution. In order to use azurectl the user is presumably on a system that has internet access. Thus name resolution works somehow. Lets use the same way the system resolves names. Companies may have their own name server and resolution rules. We should try avoiding pointing to a name server that we decree to be superior.


it does work, but mustache spec are in a diff repository you need to initialize the sub-modules :S

anyway, you can write me at espina dot edgar at gmail dot com


I guess one way around that would be to add that ENV variable to travis. Probably not a bad idea.


Is get_real_freezer_state() is needed in cgroup.h?
It is implemented in seize.c looks like some mixing of interfaces.


Do you know why we need the reloading for unit tests? It seems that we only have one jaas config file to load.


SingleSubtabInMemory (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yup, I just discovered and pushed an undo. Sorry 'bout that. Was trying to get the console cleaned up locally. Yes, would be good to add to Travis, and for me to get the `sentry_public_dsn` info from you via email ;)


The exit status of a daemon doesn't have much sense, and after checking several man pages of FreeBSD daemons, I haven't found a single one having such a section. Moreover, I don't really know exactly what status it returns, since I only use the symbols `EXIT_SUCCESS` and `EXIT_FAILURE`, without knowing their value.

I would add an _AUTHORS_ section, but I don't know how to proceed: my legal name doesn't fit in the ASCII charset that seems required, and I'm a bit afraid it would be confusing to use my pen name in the text while using my legal name in the header.


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


LGTM. Let's move Domain to backend as we don't use this object in our public facing apis.


This isn't required - you can remove this method and it'll delegate to the queryset's `all` method.


Better use `contains()` because `isset()` is evil and banned in strict mode.


Yup this request render makes a difference. Good add.


I don't think this is used anywhere, so it's probably best to remove it


lgtm


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


I have most of these except for changing the constants.


@frgm thanks a lot for the thorough review!

I noticed I received 9 notifications from github, but the page here only shows 7. I don't know what happened to the other two, but I can't make sense of them without context (one is about chaning "watchtab" to "file" and the other is about moving "desc to other options").

I also answered to 2 of your notes here, while the others should be accounted for in 17d042f613da17a0461301ac11bce0e5ea73edb7.


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


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


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


Ok?


TeamCity pull requests :: netty [Build 446](https://garage.netty.io/teamcity/viewLog.html?buildId=2536&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4785, ignored: 21 Build time: 00:56:16


`.then()` doesn't describe things that always happen- it accepts two arguments (a success and fail handler) https://api.jquery.com/deferred.then/

Thus this logic is equivalent to a success handler; it might make sense to move it into `.done()`, below.


config.php _cannot_ be read by anything aside from the server, while an improperly secured ini file could be read. Not all web servers abide by htaccess


Yeah I'd say that it's an over-complication. Let's discuss this in an issue however. The issue of having to copy 20+ lines to each file. I'd say higher-order components is a considerable solution.

Honestly, I disagree with the article as higher-order components do not always work.
Like consider the prepareStyles function we have. Should we wrap every DOM element with a higher-order component? Or should we subclass React.DOM?
Passing the context to every DOM element is not a beautiful thing to do.


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


Sure, something like what we have with `vm.args` or the `overlay_vars=` argument can work, and is a nicer way to save configuration. That said, I'm not sure it makes sense to accept a list of files like `overlay_vars=` does.


Capitalize it


Same here?


TeamCity pull requests :: netty [Build 447](https://garage.netty.io/teamcity/viewLog.html?buildId=2537&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4785, ignored: 21 Build time: 00:30:35


I made apache read the htaccess but if you suggest php we can go with it.


Do you mean we don't even need the lines from `ind_evals` down to here?


Maybe use a fallible version of `item_path_str` instead?


I was testing out the code with different versions of boot and batch-admin and to fix the issues raising at the server start, i have excluded some files but not needed now as my code runs fine without those exclusions.

Even after taking off, BatchAutoConfiguration from exclusions list, Batch_admin is still not showing up the jobs in the ui. Thanks


Isn't SupSupConflict dead now?


Fixed.


@acreilly Just curious, any reasoning behind this? Just best practices?


@shankardevy you do understand that this breaks the architecture you shown on slack?
@marcoherbst

![media_refactor](https://cloud.githubusercontent.com/assets/1857263/10616456/9877c8b8-7765-11e5-97b7-ecef3a44d514.png)


@abought Noted!


Forgot to revert those, sorry! Yeah, that should not affect the 7.8 build since it excludes those modules completely.


FYI, home page still says 12kb :smile:


Close issue #18


So much win in this change, @zzak ... I was just about to issue a PR, but noticed that it was fixed already!  :thumbsup:


See #1941.


It is easier to identify a failing test if you do not have too many asserts in 1 method.


What exactly is "old and new flexbox" and how do they play in with browser support? What issues did "old flexbox" address and how was it modeled? Does "new flexbox" simply build on the old or is it fully reimagined? I'm thinking back to the Haas about the nonlinearity of the progression of technologies.

Are there reasons that old flexbox might be used over new flexbox? Just browser support? Any practical value?


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


Do you mean `scheme`? ref https://github.com/iron-io/docs/commit/4d3d33108b525a588872705c70f77388885adc98


It'd be nice to rename this to something else (although I'm not sure quite what - `area_tuples_and_elections` perhaps) to avoid the potential confusion between:

* `election.area_types` (the many-to-many field)
* `Election.objects.area_types` (this method)

It feels like something that we should be able to remove later, but I'm sure it's best to do the straight translation as in this PR in the first case.


Congratulations. Nice work.


:black_nib: :dragon:


Hmm yeah, maybe you are right. I dont know, you decide :)


THAAAAAANKS


You're sneaking in schema changes from the training_tool branch, @mattfordham


when I looked into it I found the convention in Go is to use MixedCaps


Are these your own suggestions or are these floating around in the web design world? I ask because that changes whether this is an examination of existing conversation, or a jumping point for further discussion and implications.


Dang it... how did I do that??!?? Will fix.


Looks wrong to me


Now that's really convenient


hah! sorry.


I think I probably commented on this before, but I think this should be squashed into the commit that removed it


Ок, но можно я тогда не буду тестировать код в ejudge? Я пробовал отправлять результат g++ -E но это не работает - слишком большой размер получается


Там же где и мое свободное время(


@abought Done.
I'll wait for a confirmation that review is complete before pushing again


Beautiful! Love OSGi!


Yeah, it's not due to our code as far as I can see. The likely explanation is that another library is calling `Configuration.setConfiguration`, but it's not easy to verify (on my machine, I can only reproduce it by running the whole test suite via gradle with maxForks = 1).

Harsha's original code was calling `setConfiguration` here, so I thought that this was a good compromise where we force a reload if we can't find our entries and we don't use the Sun-specific provider.

I will file a JIRA to investigate this in more detail, but I think this approach is probably the safe option for the 0.9.0.0 release.


Hello David Rose, i am from the panda3d community and i had a few questions about disneys OTP server, i saw the lecture videos but i have a few more questions that weren't answered in the videos. Please contact me at my email proeffectsbro@gmail.com Thanks.


ok. I failing it needs to use minimum-stability .... ahhh .


In general I'd suggest a namedtuple for stuff like this rather than a dict, but no big deal.


Hmm, is this a typo? Not sure if BUILD.md would be useful for builds


I guess this `print` statement can be removed now


@arisch Yes, it should. For zip_borrower_repayment_hold_cleared and zip_borrower_repayment_hold, the item_refers_to column is 'repayment_collected, and for zip_m_borrower_disbursal_wire the ref_refers_to column is 'disbursal'. So, these should get picked up by the Vertex fact. I verified that by changing the test to looks for those credit change types and it pulls the same count and values from Verse and Vertex for the three credit change types.


Thanks @VelocityRa, that's exactly correct, and my friend still hasn't had time to write up or record instructions.


Как их сделать непубличными?


Great - Thanks for extending the test!


Why are you using two different ways to get the docker version?


Why you changed this from markdown to YAML?


thanks. I will add the error handling with a trace


These links aren't broken for me.  The first one is missing an "l" in ".html", but otherwise it is fine.  Besides a broken link, was there a reason you thought it should point to the .rst on GitHub?


por que hiciste ese cambio?


Looks great to me.


You're right, thanks.


hice el cambio por que vi que mas factible


Just to cross-reference, this is the usage that I commented could be removed on rewriting `get_by_slug` to use `get_object_or_404`


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.694](http://teamcity.monogame.net/viewLog.html?buildId=10733&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 543, ignored: 6 Build time: 00:19:11


Awesome, thanks!


Since the object returned by `genInstrumentationEntry` is a 1-liner, it would be better to write them out here. Yes, there is some duplication, but it makes it more clear what the contents of this map are.


what are those!??


Looks good!


Because I am not sure if old version of docker supported this new method or not.


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


[ed:ack!! missed one!!]
... so let's try ...


This is a general comment, not related to this PR. This code is not obvious, and I had to read this carefully to re-remember how this works. It would be nice to have a comment that explains why this test is adequate.


May want to explain what this version means, how it can be used. There is a name overload with the server notion of schema_version.


partman changes in preseed.cfg aren't being overwritten, but still getting a generic disk layout


Move the `// Load the hook` comment before this line.


Meetings/2015-10-14 DEP Committee Meeting.md

one of those two words seems redundant: reports syntax


You called the init method of che wholeBodySensors ?


<uses-permission android:name="android.permission.INTERNET" />


Need to use VS Professional 2013


 we need to wait for the completion of the requests. If there are still requests pending, the driver can not free the resources and exit, as it will crash the system.

If such a situation comes where the loop never exists then there would be another fault which we would have to handle at other location.


BclXmlChoiceValues (macumber) - Win64-Windows-7-VisualStudio-12: OK (2345 of 2365 tests passed)

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


`moduleHook` is a function that returns a map of files contained and appropriate functions to patch those files upon loading. Use a more descriptive name, or leave a comment describing what is going on here.


I think we only need the `ermrestLocation` key and not a `chaiseRecordBaseUrl` since they should always point to the same host.


falseにしてもoption menuが表示される理由考えてみて。
ブログとかjavadocを読むとfalseにすると表示されへんはずやけど、今実際に実行してアプリを見てみると表示されてる。なんでか考えてみよう。


This code is at wrong place. The variable remote_name can hold a pure file system path `/var/tmp/ccpp:jfilak@`. I shall move it to a library function and call the function from create_and_upload_archive().


nit: alphabetize imports


one nit in addition to Boris's comments


When will this be available in CDN version 5.0.1 or as a Release for self-hosting? Doing a manual build is beyond our knowledge.


Uważaj na wielkość liter ;)
Twój plik ma końcówkę JPG

No i nie bójmy się usuwać plików.
`speaker_old.jpg` jest niepotrzebny ;)


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1916 of 1916 tests passed)

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


indentation


As soon as we figure out how the new release process works. Sooner rather than later.


Fixes #22


i will add info here. thanks.


No, it's at line 28 if you're talking about importing ```spawn_process``` it's there :)
But I can see the following error:
```
======================================================================
ERROR: Install from scratch test case
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/fabio/development/python/ubuntu-make/tests/large/test_ide.py", line 468, in test_default_install
    self.child = spawn_process(self.command('{} ide netbeans'.format(UMAKE)))
  File "/home/fabio/development/python/ubuntu-make/tests/tools/__init__.py", line 214, in spawn_process
    return pexpect.spawnu(command, dimensions=(24, 250))
  File "/usr/lib/python3/dist-packages/pexpect/__init__.py", line 1754, in __init__
    super(spawnu, self).__init__(*args, **kwargs)
TypeError: __init__() got an unexpected keyword argument 'dimensions'

----------------------------------------------------------------------
Ran 1 test in 0.002s

FAILED (errors=1)
```

Do you know what's the problem here?


yeah, i will update.


okay :)


когда комит больше строк убирает чем добавляет- это хорошо :-)


have


It seems that this line causes problems when the wp is headed to a json serialization.  Is there another way to get this function into the chameleon context so the template can use it without needing to put it into the json structure?


What is the case where `moduleRoot` would be empty? I don't think we ought to reach here for that case. Either remove the test or make it an assertion.


getSupportFragmentManagerを利用するのではなくて、activityに対して、一つのfragmentしか利用しないならxmlで<fragment>タグでの利用したほうが冗長じゃなくていいね。


thanks


okay.


i think you should first do a git pull to make sure your code is up to date, then do a git revert dc1dc5c84de251c1ed8dacb2db2b205cec3cb7f7 to undo this commit, then do a git push to push that fix to github, and then try the pedigree upload again. sorry for confusion


То есть массив у нас размера MAXLINE*n а в цикле мы считываем MAXLINE строк? Кажется что-то пошло не так...


okay


@pgilad, @doratias, we can remove this repo; the original developer (https://github.com/codetunnel/mongoose-auto-increment) fixed the issue, after my comment ;-)


@RobertPeters I'm not entirely sure. Do you have the latest version? What version of PHP are you running?

That said, I don't support this lib any more. Can I suggest that you look at Anthony Ferrara's [password_compat](https://github.com/ircmaxell/password_compat) lib instead. He wrote the PHP 5.5 implementation so his is better than mine :)


So what do you propose? I still didn't hear any alternative proposal from you. But we could gradually introduce type checking to prevent this.


We should do the wrapping here in the view, not in the template.


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


ForecastFragment.javaのクラスのコンストラクタで、
```
public ForecastFragment() {
}
```
ってものを追加してたけど、デフォルトで呼ばれるので、ここで例えnew ForecastFragment()と利用していても、ForecastFragment.javaクラスでの記述は必要ありません。


Look at https://github.com/acemod/ACE3/blob/gh-pages/wiki/framework/goggles-framework.md
Not sure if you want to do it via exactly same standard everywhere, which is currently exactly `(0-<thing>, 1-<thing>)`, just noting.


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


This change looks familiar... You could've saved 3 months: https://github.com/TheMuppets/proprietary_vendor_oneplus/pull/2


"GTK+ 2" to faktycznie oficjalna nazwa, ale jeśli chcecie GTK2, to nie ma sprawy. :) @splewako - ?


The `election_data` here would be more accurately `election_tuples` as elsewhere. (Since it's not used in the loop body I'd suggest replacing it with the conventional dummy variable `_` but that's also the conventional alias for `ugettext`, so I'm not sure what the next conventional dummy variable name is after that... :))


ここはActivityを書きます。


I think we can drop both of these globals and replace them with an Angular "service". You might not even need this file `config.js`. Instead you could put this code into the `app.js` in the Services section. You could create a `configService` like this:

```
chaiseRecordApp.service('configService', function(){
  this.baseURL = <set the default value here>;
  if (chaiseConfig...
     this.baseURL = <set the value based on the chaiseConfig>;
  }
};
```


@xaniasd why do we skip push builds here? This is the cause of https://github.com/DABSquared/gitlab-plugin/issues/86
1. Push creates commit abc123 and triggers build
2. I create an MR of that commit, no build is triggered
3. Gitlab searches for build status at that commit to display in the MR, but finds nothing because it skips the Push build
4. Thus, build status in the MR page is stuck in 'pending' forever


если есть git, он за тебя уже сохраняет код, нет смысла мусорить :) стирай к чертям! уии..


yes they are not related to OPLK version.

The NDIS versions represents the major and minor number supported by this driver i.e. NDIS 6.0


After thinking about this more, I think we do reach here for `http`. I don't think my previous comment is applicable.


primiteve?


syntax1 ? :)


Hey @rutgerfick - how did you figure out this was erroneous? Did you have this show up as a bug doing some operation, or did you just look at this and it made no sense?


Ah


Becomes changes, has thought again to fast ;) and not tried. On check here at home was it OK during build.


можно просто return current == '"' || isCharOfEndValue();


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


That... is a terrible way to solve the prob...


OK, very criptic! Do you think: a) hours b) days c) weeks ???
Rough estimate would do, thanks.


Just zoom in and you'll see what i mean


So what is he point anyway? _recv is blocking operation anyway. Unles you do polling you're going to wait forever anyway.


Why these functions now is declared with static?


если почитать спеку, можно придумать название получше =)


It's a bit unfortunate that these uses of `Election.objects.get_by_slug` in `standing.py` and `metadescription.py` cause an (admittedly very cheap) extra query each time one of these template tags is used - they could be changed to take the election object rather than slug, but that's probably something to create a new issue for as a reminder rather than bothering with in this PR.


would it be work expounding "..._meta_USER_type_..." to specify "user type"? I caught myself wondering what "meta type" would describe.


What is the actual historical application here? What would Huffman Coding have been used for, and who would have used it? Basically, what was the audience/demographic that the technology was aimed at, and why did those people use it?

It might also be helpful to look at the development of compression technology in general. What was its original purpose and application? Was it developed for military use? Government? Business?

Thinking back the Table vs Grid reading we did- there were certain shortcomings of tables that need to be addressed because of the implementation of tables as a formatting and layout tool. There was this development of tables in direct response to the desire of developers to have print-like control over layout, and it worked okay for its intended application. But the landscape has changed, and CSS grids are addressing the problems with table. Defining compression's original use/application context and specific *purpose* for development might help in clarifying the specific reasons for and contributions of LZW, and why the shortcomings of Huffman needed to be addressed.


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


support libraryのバージョンが古いです。あげてください。


First, read my note below about the `configService`. Using that we can then do "Dependency Injection" with Angular and we can remove this `initConfig()` and we can also remove the `CR_BASE_URL` global variable from this code.

So anywhere you find some usage of `CR_BASE_URL` I think you could replace it in 2 steps.

Step 1. inject the config service, for example:
```
chaiseRecordApp.service('ermrestService', ['$http', '$rootScope', 'schemaService', 'spinnerService', 'notFoundService', 'configService', function($http, $rootScope, schemaService, spinnerService, notFoundService, configService){
  ...
```

Step 2. replace the CR_BASE_URL global, for example:

```
var path = configService.baseURL + schemaService.schema.cid + '/entity/' + fixedEncodeURIComponent(schemaService.schema.schema_name) + ':' + fixedEncodeURIComponent(tableName) + '/' + self.buildPredicate(keys);
```

I _think_ that will work...


Yes, you're right. It should be coupled to the [`StandardTopology`](https://github.com/Particular/NServiceBus.AzureServiceBus/blob/BatchSend/src/Transport/Topology/Topologies/StandardTopology.cs), not the general ASB connectivity settings.
@yvesgoeleven what do you think?


Кстати, насчет этого.. ты немного отходишь от RFC:
```javacript
2.1. A JSON value MUST be an object, array, number, or string, or one of
   the following three literal names:
      false null true
The literal names MUST be lowercase.  No other literal names are allowed.
```


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


close #28


Hmm .. I wonder if this will screw me up.  Am am currently blanking out the units in `forecast.py` for Humidity and Precip intensity so that I can use custom icons. ;(

![image](https://cloud.githubusercontent.com/assets/2160436/10617342/fd312cbe-7737-11e5-80f8-1032a2aa7588.png)

https://github.com/balloob/home-assistant/commit/293ed275ab4a659248dbf0e70d5ee5c674a45757#diff-f39240498111d4ab02be098713ba9d96R70


Je me suis permis de te piquer ton patch :)


@celsowhite nesting is great for you now because it encapsulates what you're working on. Nesting more than a couple levels quickly becomes unwieldy when you try to scale to larger "applications" of your code. For example, if you want to define a list item great– but what happens when you want to extend said list-item to be used in a completely different location? Well, you can't because you've nested too far.

I nest but I've also learned my lesson on nesting too far. I also have my own BEM namespace approach to help with keeping track of what derivative of a CSS object I'm using instead of nesting. Use whatever feels good, until you can locally scope CSS you just gotta feel it out.


I'm wondering whether the correct fix is here or at L177 (return "" not null).  According to the Oracle docs,

  http://docs.oracle.com/javase/7/docs/api/java/util/regex/Matcher.html#group(int),

group(g) is equivalent to  s.substring(m.start(g), m.end(g)), which argues in favor of "", but then it contradicts itself by saying "If the match was successful but the group specified failed to match any part of the input sequence, then null is returned".

Could you verify the actual behavior of the Oracle implementation in this case, and make the appropriate change?

thanks
alan


@dapplebeforedawn It's a little less straightforward, `child_key` is a string and can represent a nested hash like:
```ruby
[a][b][c]
```
or just be a regular key name like `a`. The current implementation just assumes it's *not* going to be nested, so the key lookup fails and tries to add it to the existing hash in the array. This change attempts to grab the first param from a value like `[a][b][c]` (a) and check and see if it has been set, allowing it to correctly hit the else condition and create a new hash.


See the next commit in the PR. This commit added the argument to mca_base_select.


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


And the build farm weighs in :+1:


Corrected


"Unable to *get* column value" (missing word?)


Err, fixed.  Thakns.


missing word


スーパークラスのonCrateOptionsMenu(menu)を返してください。


Anthony, thanks so much for replying. I once lived in England so it's always exciting to hear from someone over there. I'm running PHP 5.44. I Assume I've got the latest version of your script, which Looks like it's two years old. Thanks for pointing me to an updated script. Unless yours is a security risk, at this point I think I'm going to use it, cause I've already done quite a bit of work adapting into my code. Can you tell me who whether you feel like it's care at this point? Thanks so much.

Sent from my iPhone

> On Oct 20, 2015, at 11:28 AM, Anthony Chambers <notifications@github.com> wrote:
>
> @RobertPeters I'm not entirely sure. Do you have the latest version? What version of PHP are you running?
>
> That said, I don't support this lib any more. Can I suggest that you look at Anthony Ferrara's password_compat lib instead. He wrote the PHP 5.5 implementation so his is better than mine :)
>
> —
> Reply to this email directly or view it on GitHub.
>


Те кто ленятся выдумывать хорошие названия функциям обречены писать документацию :)
например `boolean enclosedInQuotes(String value)` или что-нибудь в этом роде...

Ну и да, зачем лишние переменные? возвращай сразу s.startsWith ...


the protocol name represents the device node id for this driver.

I think PLK makes sense and will be consistent with the PLK dev name used in Linux drivers. I will update this. Thanks


last one


上のタグの最後に </>をつける


thanks


Activityを書く。DetailActivity?


lxsession-logout itself just kills it: https://github.com/lxde/lxsession/blob/master/lxsession-logout/lxsession-logout.c#L190
So it's tempting to just use `pkill lxsession`.


will change thanks


@xfra35 added like you suggested, at https://github.com/bcosca/fatfree-core/commit/048feb04dc3e64441571d0e8b7e71af70e276653


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


wrong author


yeah, I will update


xml指定


Makes sense, but I wondered, and I found out there's official Racket style guide (which agrees, hence fixed in 2cd33a527fd22114212e625f973d52cfbf726927):

http://docs.racket-lang.org/style/Textual_Matters.html#%28part._.Line_.Breaks%29

Yeaaah! :+1: (Looking forward to https://www.haskell.org/documentation/style/ within the next decade :wink: .)


Add a comment with the intent here. Based on my reading, you want to check if all of the files listed in the `patches` array are active. Use `Array.prototype.every` instead of `reduce`, if that's the case.

What's the scenario, where only some of the files have been patched before the same module is loaded again. Can you add a unit test that exercises that path to make sure the caller of this method deals with it properly.


shnur


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


onCreateViewにきたときにgetActivity()がnullじゃないとは限らない。
確実にnullじゃないときにこのメソッドを書くには、
onActivityCreated()にかこう


Aaaaahhhh! :boom:


It is legal?! Crimea is Ukraine, and the entire civilized world recognizes it.


This patch is a subset of the following Illumos commit:

https://github.com/illumos/illumos-gate/commit/078266a5aafa880521ea55488ef3d676f35e908e


typo nit: " keep track" -> "keeping track"


yeah, i see... the "naaa" branchs is test only xD in "naaa-7" the author is good! xD,


Add new comment to this. Sorry just learn Git more.


Unrelatedly, if Racket still had `empty = #f` (as IIUC other Lisps do), this refactoring wouldn't be needed: one could keep `(and dir ...)` and use a plain `map` on the result, without checking for pairness.


The parameter is maxRevTreeDepth. 20 is default value of  maxRevTreeDepth

in Database.java
```
    // Default value for maxRevTreeDepth, the max rev depth to preserve in a prune operation
    public static int DEFAULT_MAX_REVS = 20;
```

in ForestDBStore.java
```java
    private static final int kDefaultMaxRevTreeDepth = 20;
```

in SQLiteStore.java
```java
    // Default value for maxRevTreeDepth, the max rev depth to preserve in a prune operation
    private static final int DEFAULT_MAX_REVS = Integer.MAX_VALUE;
```


pretty sure this causes no-boot


I would suggest to apply this commit on master, release a new tag and notify the other community about it.


Link?


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


correct. thanks


What is the "wo" thing?


Not a huge deal, but I might quibble on this function name - you're validating the signature, not signing something yourself.


@lingeringcode That makes sense. I definitely would like to start with the framework of Haas' study of the coexistence of old and new technologies. This way my discussion of the varying landscape of devices and access methods (mediational means if you will) is grounded in theory.

I'd also like to use Haas' study of the influence of human design decisions to frame the perspective I'm taking with media queries and how the address the move *away* from the linearity of print > digital page design.


This looks good.


Just a note as a reminder to both of us that this undoes the just-in-time importing of static data, which should be squashed out


not sure how that happened. he removed it, though


I agree with you, this commit can easily be integrated before the review of the other commit.


'siad' is able to upgrade in-place, which means users never have to touch or copy the files.


Since the value can change immediately after this function returns and the lock is unlocked, it could as well be made atomic so reading does not require a lock. Though probably only worth it if could be a bottleneck - I guess this is not called that often?


looks like this is unused?


スーパークラス


@jonpas did you feel it ha? did you feel it
![Image of tasty bananas](https://farm3.staticflickr.com/2876/9681688504_b5e8689026_z.jpg)


just it was copied the style from C API test.

In C/C++, variables declared in a scope will be release at end of block. I believe C API test utilized its behavior.


looks good


Yeah that was removed in the subsequent commit.


Anthony's should be a drop in replacement if you're just using the functions. If you're using the class directly then it's a different story. Oh, and mine has non-standard algo support, such as SHA256. Don't use those.

My code uses `rand()` when it ought to be using something that is cryptographically strong, so yeah, it sort of has a security risk.


want to indent this or just declare a second variable. I have in the recent year or so got religion on always doing a var for every variable in most cases.


Yeah this was removed in the subsequent commit.


Just copy that file to your custom component folder.


I agree with you @shaurya947. Regarding using `prepareStyles` outside a Material-UI component I disagree. If you need to do so, simply use  `const Styles = require('../utils/styles');`.


@CCOSTAN There should be a better way to do icons with forecastio.  With the forecastio API you can request a weather icon and it will return a plaintext icon name:

icon: A machine-readable text summary of this data point, suitable for selecting an icon for display. If defined, this property will have one of the following values: clear-day, clear-night, rain, snow, sleet, wind, fog, cloudy, partly-cloudy-day, or partly-cloudy-night.

Once we have the icon name, we should be able to replace the badge with a suitable icon.  Unfortunately Google does not have weather icons in their material pack AFAIK.

Of course, this is a bit different than the icons you have where you want the state to show up where the units usually are.  If I were you I would just maintain my own custom_component where you set the unit_of_measurement to a blank string.


Yes, this is an issue. Can you please report an issue in the issue tracker? I will fix this asap and hopefully @areski can review it and merge my changes.


`void testAllDocsIncludeDeleted()` of c4DatabaseTest.cc is not yet ported. `setupAllDocs ()` is used from it.

I will ported `void testAllDocsIncludeDeleted()` C API test.


yes


issue5221_memoryAllocation (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1360 of 1360 tests passed)

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


done.


Silvia Ehrmann


Good catch. Fixed.


Ideally this should be optional


thanks, i will update the description


The BOSS class interaction fails due to the getRandomClass() method in the MassTest class. I think I fixed it now.


Hours through days. Not weeks.


Yes, it is better to call free() method explicatory.  Otherwise, it will be called when GC is executed.


hey


Not sure if this is what changed it or not but in beta7 you could put the following in your project.json
```
  "resources": [ "**/*.cshtml" ],
```
and it would add all of the .cshtml files as Resources for your project but this no longer works in beta8.    How can you embed files in a project as resources now?


This should go on the list of 'things we could move over now', correct?


Nevermind. The publicly facing part IS optional on line 121 below.


名前これでいいんかな？
http://www.techdoctranslator.com/android/guide/manifest/meta-data-element


closes #79


This seems to be registered always, and perhaps can be dropped altogether


Hi morevnaproject,

Are you cross-compiling Synfig on Fedora for Windows using ming-w64?
I tried the script earlier (http://www.synfig.org/forums/viewtopic.php?f=13&t=9330) but it failed after downloading some packages.

Secondly, have you tried using the Cygwin mingw package?

eausaig@NG00042732 /cygdrive/c/cygwin/home/synfig
$ bash autobuild/synfigstudio-cygwin-mingw-build.sh mkprep

chmod: cannot access ‘/usr/i686-w64-mingw32/sys-root/mingw/bin/*.dll’: No such file or directory


yo


Why not create a separate function to register those branches which needs trackVars=True to be created.
Also should avoid hardcoding parameters. Makes debugging difficult.


@rsyarif
You might just have well left the original lepton pTrel and other variables for the commissioning Ntuples.


superクラス


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


これは勉強なりました


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


Thanks. I'll eagerly await the release to fix our problem.


This is pretty cool. I didn't know we still use the same 72 "dpi" (or whatever the unit was at the time). Why is that still the standard? You'd think after hundreds of years of printing (and now web distribution) there'd be a better (better? or just new? paging Haas...) system for sizing and presentation of type faces. What's the importance of this aspect? What else has stayed the same since typeface has moved from exclusively print-based mediums to, now, the web and every form of delivery imaginable?


中身なし？


I'm getting "No ViewManager defined for class AndroidViewPager" when trying to use this


@hdachev - We have some simple flags that we set in Redis, so they are shared across a cluster of servers


The change is definitely a good one (since the test shouldn't rely on a particular numeric ID), but I don't quite follow the commit message:

> Becuase the database is no longer in memory it's not recreated every
time

The PostgreSQL database is recreated on every test run; what's different about the behaviour if you're using SQLite in memory? (Out of interest, and hoping to understand this better.)


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


LGTM.
It's always good to remember: I am not a rpm expert ;)


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


do you think that changing to caf ril could fix no sim error on 5.1.1 roms too?
there were some roms (mostly based on aosp/slim I think) where no sim card error occur.

I tried to fix it by this line but it didn't help:
USE_RIL_VERSION_10 := true

I'll try with changing to caf ril when my laptop will be fixed


Can you not just use the default sort, which should be alphabetical? Didn't test this, but feel like it may work to replace most of this:

```javascript
Object.keys(object).sort().forEach(function(key) {
  sortedObject[key] = object[key];
});
```


Suggestion: Why not use [sync.Once()](https://golang.org/pkg/sync/#Once) here?


Moved wrapping to view.


yeah that's cleaner
ack


I feel like some of this code is duplicated


ahh this is meant with debug stream...
ack


I think porting Synfig to WxWidget (wxGTK, wxMSV, wxMac) or Qt will make the building process and development easier. GTK+/GTKmm development and setup on Windows seems to be complex.


yo


This were intentional :)

The main reason it was shared is `generate_lexers` configuration, which used `gradle` task to build both lexers with correct jflex version and skeleton.


`closeIndex()` is just close view.
`deleteIndex()` deletes view database file.


We should add a memory test around this right?


hey


unack
maybe put that somewhere else and not into the tutorial?
the beginner doesn't want to bother with debug messages and technical details, I would go for a new markdown document `Debugging with coala` or similar^^


sd


I'm not sure it works
for instance
( [ ) ]


/target
/classes
/checkouts
pom.xml
pom.xml.asc
*.jar
*.class
/.lein-*
/.nrepl-port
.idea/*
clojure_humanize.iml


Yup. I'm handling that in another PR though. Once this is merged I'm going to do one PR that adds a few memory tests for as many components with ripples I can find. To make sure we are downgrading them as we should (since currently I don't believe we are in cases like buttons.)


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


I will update/fix `updateIndex(views)` and `updateIndex()` method.


Is there a need for separate configuration? It can be easily triggered with context-action right from `build.gradle`.


this makes no sense. post* methods cannot add SQL as the SQL has already run. If they want to do things here, they need to use the connection


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (842 of 842 tests passed)

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


the edrv module require a maximum frame length of 1536 bytes.

I don't remember why i added the extra 10 bytes here :/. I will update it to 1536.


thanks will update.


gg!


Explain this in the commit message.  (That's the reason the traces are hard-coded -- so if they change, we notice and ensure we understand and document why they changed.)


thanks will check other locations


this is unreachable


This is weird


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


dsd


I take it you have signed the FSF papers, Sebastian?


I agree with all the changes, makes the code look better! :)


Will do asap, thank.

For now I'm working on the integration of your work in nethserver, smeserver is already done -> http://wiki.contribs.org/Shellinabox. I took my source from epel.


asd


thanks


ads


sad


oh whoops, forgot to break out of for loop when errs channel is closed. will fix


nothing can be reading from this yet, so it will block forever


Oh yeah


This part is just me venting. I'll post the actual fucks I'm doing in the latest commit comment section, so go there.

So, I play Kancolle in chrome but all this while I was using Firefox to test stuff. Everything running smoothly, I managed to make equipment work and quests. I'm not used to using git, so I just kept coding. I bit more than I could chew, and before I noticed I couldn't get past the login screen. Instead of stopping and put things back together, I decided that I could fix it later when I finished doing the modifications I wanted.
Boy that was a mistake.
After 10 or so hours of manually reverting code, cloning my fork and even yours from scratch, I still couldn't get past the port. At this point had I already tried to copy the official port response from 2 different accounts and all kinds of json manipulation, but still water.
So I just giving up when I noticed that the response from my official account on Chrome was 2.7kb. From my server, it was 1.6kb. A clue!
I opened my local server in chrome...and it worked. Just like that. I have no idea for how long it was working, at what point of the 10 hours the code was working again, but Firefox was simply refusing to accept this.
So yeah, Firefox refuses to play Kancolle anymore even after I did all this progress while using Firefox. I have no fucking clue why. Go figure.

Vent over, I'll post progress in the latest commit.


There is no pressing need, it's just convenient, especially for future contributors :)


Is changing that commit message worth rewriting the history of this branch (into a fresh branch)?  Do we have any other durable way to record it?  (I could add a contentless commit whose message references the id of this one?)


Should I change them now? :))
Anyway,for the next assigment I'll make sure that they are matching the Java standards ^^.


Great :+1: That's a big step towards easy manipulation of monetary data. Thanks!


Windows uses `'\'` so this split won't work.

You can try something like `.split(/\/|\\/)`


First, this is the post-almost-apocalypse commit. There are a lot of trash/unused code,  redundancy and nasty stuff. I'll start fixing it later today.

What we have:
Quests - you can create new quests, give out rewards and item/ships as reward. Activate/Deactivate/Complete at will. You can create Quests requirements - when you complete one, it unlocks another. 50/80% progress notifications are working too.
I still can't show the reward bonus animation (when you get flames for instance it shows after the resources, you know?)

Equipment - Admiral can have items. Ships can equip/remove/change equipment. Equipment stats are NOT accounted yet though.

I'm running late so that's it for now. Also I forgot what else there is.


good catch, let me think about this error case some more


Add new modules into install script constants


yeah.


Yeah, that's probably cleaner. I'm considering to change for a next version.

Thanks for pointing that out.


Remove if not required


Looks like `MessageMixin.reply_with_photo` should have its `caption` and `extra` parameters default to `None` for anyone working with raw API objects.


(defproject bilus/clojure-humanize "0.1.0"
  :description "produce human readable strings in clojure"
  :url "https://github.com/trhura/clojure-humanize"
  :license {:name "Eclipse Public License"
            :url "http://www.eclipse.org/legal/epl-v10.html"}
  :dependencies [[org.clojure/clojure "1.5.1"]
                 [org.clojure/math.numeric-tower "0.0.4"]
                 [clj-time "0.6.0"]
                 ]){}


o.o I know that, that's why i named the method dequeueLogins :p


ds


sda


das


You forgot to rerun autogen with the updated templates.


Is it possible to make the QsQlQuery a protected member of the class? maybe try that


YAML can be easily worked with in ruby as objects, so changelogs for builds can be automated: https://github.com/artsy/energy/blob/master/fastlane/Fastfile#L19-L40


I think these should be defined in platform-specific parts of spec.json and then generated here.


what is `continue`?


Why do you need this?
Also this wont do shit if its not preloaded in the ramdisk ;)


"entries" :smiley:


I think these names/values should be defined in spec.json and then generated here.


I refer to NDIS intermediate driver solution with application in user layer and driver in kernel layer.
This was the old naming convention :).

I will change it to "for __Windows kernel interface__ which uses..."


@annagaiel @davidslee101 what do you mean I keep repeating?  This is your branch.


@nick-erhardt I couldn't remember what the scope name that we came up with was, but this sounds close enough. Feel free to update it to what's on the graffle


Sieht gut aus, hatte mich eben mit den selben zeilen beschäftigt :+1:


limit and skip features are not implemented. I will fix it.


I refer to circular buffer library. Will update.

Yes the queues are completely implemented in Windows kernel space


yeah. thanks


Another option would be to annotate the sql traces with comments explaining where in the code those queries come from; changes in that comment stream would constitute documentation of sorts of what happened.


@mattmuroya These were actual suggestions made by the CSS working group. You can see who wrote these suggestions and exactly what time and date they wrote them at because it is in the form of an email. http://lists.w3.org/Archives/Public/www-style/2012Apr/0352.html this is a link to one of the suggestions and the conversation surrounding it.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.695](http://teamcity.monogame.net/viewLog.html?buildId=10737&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 541, ignored: 6 Build time: 00:19:21


Great and I hope you manage to integrate shellinabox in your projects :+1:


This function declaration better to move to common.h and remove static keyword.