Да. А где я могу её посмотреть? :)

> 26 янв. 2016 г., в 12:43, roman-melnishin <notifications@github.com> написал(а):
>
> У меня вопрос: правильно ли я сделал (в новом исправлении увидите), что всю левую колонку объеденил в одну
>
> и туда засунул заголовки также h2 от checkbox'ов и radiobutton'ов и кнопку сабмита.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/roman-melnishin/153965-barbershop/commit/4fb3f4086cf275dc7185ac847c8142ddff6b60d0#commitcomment-15672230>.
>


I´d set it to `false` by default since most type of layers won't need an update (eg: tileLayers)


also, this is the same order it was registered in before


форму обернуть в div


W obecnej chwili kluczem głównym tabeli PartsList jest złożenie kolum Parts_id i Orders_id. Przy usuwaniu pojedynczej części konieczne będzie podanie obydwu tych wartości


Thank you!


feature_initializer_list_conversion (vrennert) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (385 of 385 tests passed, 0 test warnings)

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


Wow, in the past it didn't even depend on libstdc++ it seems… Maybe I should fall back to good old C malloc stuffs.


You messed up the vehicle lines BTW.

There are a bunch of empty lines there for a reason. It's off by like 8 lines now.


could you use uint8_t instead of char here? We generally use char only for actual characters, and when we mean a generic byte we use "uint8_t" instead.


yeah, this is weird... I'd expect to use our codec


Bummer! Thanks for noticing, I wasn't aware of that difference. I guess I need to change it back to `[~U] SourceCharacter but not one of \ or ] or -`, which will make `/[\c%]/` a syntax error again. And implementations will just define their own semantics per [ch. 16 Language Extensions](http://tc39.github.io/ecma262/#sec-error-handling-and-language-extensions).


TeamCity ReOpenLDAP :: ps-stable [Build 103](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1725&buildTypeId=ReOpenLDAP_PsStable&guest=1) is now running


`ParamDecl` created for `acb` in

```swift
class Clazz {
  var y: Int {
    get { return 2 }
    set (abc) { }
  }
}
```


`\7` is recognised as a legacy octal escape sequence:
ClassEscape → CharacterEscape → LegacyOctalEscapeSequence (appears before IdentityEscape and therefore has a higher precedence).


Ok, I'll move the definition to the end of the annex section.


Also, please enclose all uses of i and vt and brackets, to avoid unexpected operator binding effects.


`end` variable name is very confusing. Something like `len` or `effectiveLength` or something might better


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


For `torque-layer` I think i'd make sense to have (AFAIK they don't have `sql` and `cartocss` attributes, right?):

```
needsUpdate: function () {
  return this.hasChanged('visible');
}
```

For `cartodb-layer` I think we'd need something like:

```
needsUpdate: function () {
  return this.hasChanged('visible') || this.hasChanged('sql') || this.hasChanged('cartocss');
}
```


I like these two macros, they make things a lot more readable!


Почему лучше метод GET?
Да, кнопка отправки формы нужна и одной будет достаточно.

Попробуй посмотреть ещё это.

https://developer.mozilla.org/ru/docs/Web/HTML/Element/form <https://developer.mozilla.org/ru/docs/Web/HTML/Element/form>
https://developer.mozilla.org/en-US/docs/Web/Guide/HTML/Forms/Sending_and_retrieving_form_data <https://developer.mozilla.org/en-US/docs/Web/Guide/HTML/Forms/Sending_and_retrieving_form_data>
> 26 янв. 2016 г., в 12:31, roman-melnishin <notifications@github.com> написал(а):
>
> ну вообще я уже понял, что тут лучше метод GET, но какой-бы метод не был, все равно нужна же кнопка отправки формы, так вот я тут немного не понял, нужно всегда после формы делать Показать
>
>


Why do you need to cast here? I cannot see `getSpecificValidationInfo` returning a `field.Error`


Would it be possible to push this up to a parent abstract impl to not duplicate this sort of code?


Oh shit I'm replaying in a commit look how close all these messages are this is craaaaaaaaaaaazy


"поиск гостиницы ..." ссылкой и отдельно от формы


Newest results looking very good :)
http://hastebin.com/exerujezeg.avrasm


карта после формы


Could do, but additional markup always has risks, like `.something > input` not matching anymore.


Hello Sarah,

I am interested in submitting two javascript files, but when I push the new files to Git I get an error msg saying that permission is denied. Can you or someone else help me in submitting my work? This is great practice for me btw.

Thanks!


> Can we please keep this?
I am afraid not, for both practical and theoretical reasons.
Practical: that's how Z3 formats it's output, and that's that. The previous function we have used is deprecated and associated to SMT-LIB v.1.

Theoretical: the contract of the method is "output should be valid SMT-LIB2". I do not want to add restrictions due to CPAchecker.

> Otherwise it can get really difficult in CPAchecker to separate the assert from the other lines.

Would it? I can suggest a working solution straight away: search for "^\(assert", and grab the line from that point onward.
If we want to be more proper, we can use one of the existing SMT-LIB2 parsers, either the one shipped with Princess which we already have or jsmt-lib.


That one is nice! :-) reviewed by @fritsch


Another way of spelling the `min` is:
```
std::min<milliseconds>(mPreviousMSeconds, AV_MIN_CONSENSUS_TIME);
```
There's no functional or performance difference.  The only difference is readability, and I don't claim that my alternative is more readable.  Just exposing the option.


@matiasgarciaisaia with this change the csv export seems to be affected by the pagination. Am I right? cc: @nekron


Nice, thanks!


Thanks


I would reverse the logic: add devices here (RHT) and export all RHT into TEMPERATURE and HUMIDITIY then you don't need to compare if they're in both


feature_initializer_list_conversion (vrennert) - Win64-Windows-7-VisualStudio-12: OK (384 of 385 tests passed, 0 test warnings)

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


@lisamariew7 What am I doing wrong? I can't get this IMG to share space with the text.


0x42 is QTranslatorPrivate::Hashes


+1 cool usage of mocks


Ok, thank very much for this complete example (I forgot about the `.cost`).

I'll implement it as soon as possible, update this branch and let you know!


What do you think? My only worry is overlap with our JCDL submission, and it's perhaps a bit out of scope (as it speaks more to collection development rather than social media archiving itself?).


Is this log needed or was it just for debugging?


That one is av_malloced below - your memleak?


This was left commented for a reason?


feature_initializer_list_conversion (vrennert) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (385 of 385 tests passed, 0 test warnings)

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


обернуть в див с классом лого внутри ссылка и только внутри ссылки img


дать класс


это не нав, нав это главная навигация, хлебные крошки для удобства пользователя и для seo, удалить див а класс перенести в ul


@gribozavr, was there still a reason to prefer computed vars over let constants?


каждую группу инпутов с заголовком обернуть в див с классом filter-group, каждому h2 дать один и тот же класс


все слова внутри тегов пиши с заглавной буквы


дать класс btn , баттон будет идти после 2х дивов filter group


класс pagination дать ul див удалить


убрать обертку


дать класс product-price спану


дать класс btn


this function doesn't exist. You could just add an getter to the locations field of the LogicalFile class.


Yeah, that make sense. I can yank it out, unless @dltj thinks otherwise. ...or if that is even in scope for him :smile:


This change is not necessary


Napačn branch editaš, dela se na branchu multiplayer.


no need for allocation. I know Java has a garbage collector, but it's not C either.


I thin it would be good to add additional check before action.
Therefore you'll be sure that action was successful.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1244](http://teamcity.monogame.net/viewLog.html?buildId=14041&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 588, ignored: 6 Build time: 00:26:30
### Failed tests
```
MonoGameTests.exe: MonoGame.Tests.Visual.SpriteBatchTest.Draw_without_blend: <no details avaliable>

```


We could put #FFFFFF but is the default value for any browser.


до этого места по аналогии с предыдущей страницей


див в данном случае это просто обертка, которая делит страницу пополам и класс дать column-left


`return` statement is redundant after `raise`.


дать класс


див класс  класс дать column-left , это обертка которая делит страницу пополам


спан


удалить


класс product-price


класс btn


дать класс который мы давали h2 в filter-group


Right, I 'over-simplified' my original fix in this pull request.

I believe the flag should come from `argParen` before modification, though.

Pushed the fix.


pulling or comparing master with edit version #1


New-Fan-Object (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (2053 of 2056 tests passed, 0 test warnings)

 * 549 tests had: AUD diffs.
 * 3 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 3 tests had: Table big diffs.
 * 2 tests had: MTR big diffs.
 * 1 test had: ERR diffs.

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


VIPSimulator.gateInputFileNames is a Vector, you can iterate on it. look at line 187 of VIPServer.java for an example. this part is too C for that code.


Fixed in 3.0.1


should'nt this be named something like "handle SIGUSR1" ? Alternatively, we can keep the name
and mention (in a docstring) that it is only used to interpret this particular signal


`return originTypes[gvk]` ought to work since its a map to bool


Add a TODO noting that this works by coincidence.  Once builds, images, deployments, have their own API groups, this will fail.


Why not? Couldn't we simply ignore the `varargs`?


ops i forgot that hhvm coding conventions states that we have to use lower cases. I'll probably use your suggestion to ```using namespace```.


`IsKindInAnyOriginGroup`


Works now with 701c175


When I run -h on the standalone server, it says:
```
> java -jar selenium-server-standalone-2.49.1.jar -role hub -h
...
 -browserTimeout:
    The timeout in seconds a browser can hang
```
```
> java -jar selenium-server-standalone-2.49.1.jar -role node -h
...
 -browserTimeout:
    The timeout in seconds a browser can hang
```

However, this line looks like it is setting browser timeout in ms instead, is this intended?


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


Actually, sorry, my "practical reason" was wrong, I now took a better look at what the code was doing.
I still think that logic should be in CPAchecker though.


див, h3 не может быть если до этого на странице нет h2 ,  h2 не может быть если нет h1 тут строгая иерархия


Edit formatting please.


This is useless. free(NULL) is a noop. I'll fix it after integrating your PR, but don't do this next time if possible.


O_o пробела справа от чёрточек достаточно чтобы всё сломалось o_O


@deads2k you added authz types in Kube in a separate group, right? Do we plan to do something similar here?


I looked into the problem and it seems to be caused by the Ajax requests (the connection limit is a bottleneck here). A good solution might be to get data for each segment and process it just after a table row (with segment) is visible to the user. I will work on it.


There should be no reason to get the `GVK` as all.  You can perform the restmapping lookup based `a.GetKind()`, right?


> Do we plan to do something similar here?

Obviously not with the rebase


This call to Timer.delay was EXACTLY what you were supposed to not do. This call is a blocking call and will screw up the scheduler and possibly communication with the FMS and the DS at competition. Please fix this to not use any of the methods on the Timer class, and instead use the methods on Command.


Non `Makefile` is bugging me...


Thanks for the note @rjharmon. I'll do a version bump and address the react dependency as soon as I get a chance.


> @deads2k you added authz types in Kube in a separate group, right? Do we plan to do something similar here?

Yes.  After we get clayton's changes that allow group defaulting, we'll move everything in separate packages to their own groups.


A little test case to this new feature would be apprecied. It'd be perfect if you could contribute one in another PR, please.


This are just content drafts to help people get started - I know it has to be English first - like the "templates" or examples how to make a title, a list and a link.


This version string should not be hardcoded, but it should be something 3.5.5 or greater


Please do not commit commented out code without a specified reason (in the code). We are using git so we have history of the source. Commenting out code just clutters the source files and opens more possibilities for confusion.


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


Окей

> 26 янв. 2016 г., в 12:18, roman-melnishin <notifications@github.com> написал(а):
>
> Разобрался, нужно закрыть тег
>
> уже после радиобаттонов, так как там есть кнопка отправки формы "Показать". Да и метод get нужен 100%, так как запрос делается на сервер с целью получения информации и нам важен результат в адресной строке, чтобы можно было или в закладку добавить или с кем-то поделиться. И тут будет одна форма на всех, а то я аж две сделал, отдельно на checkbox и на radio.


this is my comment


Look strange. 2 strings are absolutely the same.


This package should definitely not rely on react-native-gcm-android, but react-native-gcm-android must rely on this one.

I don't think it's necessary to merge the two. Do one thing and do it well.


You're right. Looks like clicking on the link for `CharacterEscape` under `ClassEscape` in Annex B brings the default grammar in Section 21.2.1 into the view instead of the overridden one in Annex B and I looked at the wrong one.


можешь сразу прописать здесь путь на index.html


shiny happy people


а здесь на каталог


Could we add a test that verifies that the numbers in `messageType` and the indices in `MESSAGE_NAME` match up?


sdas


Yeah well PHP's fundamentally flawed design more, where it is an error handler's responsibility to check whether it was rightfully invoked :') no other language or framework has such funny surprises!


Two nits to pick, but they should be quick to fix.


I think a clearer/more appropriate description would be "recorded id of user who unlocked the volume if available".


112331505_SurfaceHeightRange (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (957 of 957 tests passed, 0 test warnings)

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


I know UUID is not currently present in the users table, but logging that (as well, for now) at least would help us tell if the users data would still be valid, even if we can't cross-reference/do a join with that info yet.


зачем здесь тэг form?


I barely use github but yes i'll have a better formatted commit message next time.

as for @AaronVanGeffen, probably, not sure. I really didn't look that far. All I wanted to add was a password protection function for headless servers.


нужна кнопка


Hey kyle this is a great commit, but lets try to put great changes like this into its own branch so in case it breaks we dont have to merge it into master.


Rename getReplicaLocations to getLocations (it's a simple getter)


This is your first commit...okay


Please undo all the modifications about random generator
check indentation


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


Это очень странная проверка. Выглядит примерно так: если строка с индексом(не существующим, кстати), это строка.. Но нам же нужно не проверять на то, что она строка, а на то, что она вообще существует. Тогда почему бы не if(string[string.length]), тогда и === не надо. То есть так мы проверяем на трушность и нас все устраивает. Пока строки есть, делаем то-то, как только строк нет - ничего не делаем.


please revert this commit. We have to discuss


I'm not sure that should have a location at all. Alternately, if isTypeLocImplicit() returns true, maybe we don't need to check the validity of endLoc (we can just assert it).


Russell's idea!! :)


слайдер – это не часть хедера


Thanks. All the issues should be fixed now.
On Tue, 2016-01-26 at 10:03 -0800, Vidar Tonaas Fauske wrote:
> Why not? Couldn't we simply ignore the varargs?
> —
> Reply to this email directly or view it on GitHub.


Will change to wchar16. (I used ```WCHAR``` because signatures of PAL (e.g. ```_wcsicmp```) use ```WCHAR```. We'll have to align with wchar16 anyway, so maybe using wchar16 consistently is more clear.)


Well, it depends on definition of the term `explicit`, in this case an empty tuple is created to represent original `nil` which was explicit. But I wouldn't argue on that.

The problem I am trying to solve is that without this change after type checker has done its work there is no suitable node representing the original `nil` and if I look through all (implicit or explicit) empty tuples I don't see any criteria to use for detecting if the original `nil` this tuple was crated for was implicit or explicit.

Suggestions?


Fast STC ! Good luck for LTC.


it's better to do something `.filter(employee__user = self.request.user)`


I dropped this because it doesn't do anything.


Забыл дефис


Trick from http://stackoverflow.com/a/18035699/1172541


Indent...


112331505_SurfaceHeightRange (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2057 of 2057 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

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


Thanks!


feature_initializer_list_conversion (vrennert) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3043 of 3043 tests passed, 0 test warnings)

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


Тут в целом не нужна проерка на то, что это именно строка. Ты это и так знаешь на стадии применения slice


ок, добавь ещё мнемонику &npsp перед /кг


Maybe use MakeShared?


зачем разное название классов?


Maybe you could change the InitAndChecked field to store an enum: "Unchecked", "Checked", or "CheckedAndRemoved". That way we can keep the reference to the existing Expr (which costs us nothing because it's ASTContext-allocated) but note that it's not really part of the AST any more—it's only available for syntactic use. Alternately, we could make it a PointerUnion that refers to the new Pattern that owns the checked expression.

Do either of those sound like good ideas, or do you think this is still the best way to go? If so, I would *still* suggest a PointerUnion, just to an ASTContext-allocated SourceRange. That way it'll only be used in rare circumstances.


You should use the `{% reverse %}` tag in your template


Это условие будет работать только если у тебя будет (в данном примере) 27 строк и ширина по символам строки тоже будет 27. Что мало вероятно. Может его убрать вообще?


lgtm


Look very nice. Good job!


The control flow here is bad, you always need to null held knife.
You should do something like:
```
if(putinhands)
sucess
else
fumble
held_knife = null
```


должна быть ссылка в диве


+1 makes sense to me!


You should get this data in the template, with the `|date` filter


Would it be easier to use PSModuleMessage here instead of object?


ну и тогда мы начинаем с нуля, чтобы первая буква не отрезалась


TeamCity ReOpenLDAP :: ps-stable [Build 103](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1725&buildTypeId=ReOpenLDAP_PsStable&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 00:29:40


If `validating == false` and `proposing == 0`, do we still have a consensus?


Because it can ;)

> Am 26.01.2016 um 16:42 schrieb SalmonDE <notifications@github.com>:
>
> Why is a "DISPENDER" in line 32?
>
> —
> Reply to this email directly or view it on GitHub.
>


These meta files should be added to `.gitignore` and should not be in the repo


TeamCity ReOpenLDAP :: devel [Build 229](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1726&buildTypeId=ReOpenLDAP_Devel&guest=1) is now running


Yes, this xplat-todo refers to #172. I'll merge.


nope, still used for verifying tags


> actually, I'd probably just leave this as is... these were the ones needed to let test code stop importing v1 and v1beta3 packages directly

Ok, that's fine for now.


Значёк акции, фотография мороженного и цена должны лежать в одном блоке


aktbo feen lamo25za ?! :v 😂😂

On Tue, Jan 26, 2016 at 6:47 PM, Ahmed Ashraf <notifications@github.com>
wrote:

> 😂😂
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/michaelfahmy/GravityOS/commit/637e2415a41a330c48d4458845df66b08a8e68b6#commitcomment-15682583>
> .
>

--
Best Regards,
Michael Fahmy


shit
why didn't you point this out BEFORE i made that commit
one-line commit city :cry:


Same problems here as I mention in the previous commit. Please fix in both.


это заголовок


Should be ASSERT (EXPECTED, ACTUAL)


Hi,
Daniel, Firstly many thanks for making this Library OpenSource, You guys are really awesome.
I just wanted to say If you can pls add a more line to the #### Can't compile?
> Adding import UIKit to files if getting error while compiling.(If building for iOS 7.0)
I know this bug has been closed months ago, but still at some places this is missing.

Again many thanks for the Library.


Didn't notice until I tested locally and knife was not nulled.


transitive?  squash into kube bump


Имхо лучше через switch(getItemViewType) сделать.


заголовок


Hm, that does seem unfortunate. Yeah, okay, I guess we don't need to solve that problem now.


это не article


Remember to put `;` at the end of every line


хорошо разобрался, молодец


Trick from http://stackoverflow.com/a/18035699/1172541


The issue is that DKNavigationBarTitleView also has a label. So we need to provide 'hooks' to customize the font, color, etc.. Before this was done via the plist colors.
The client can already put whatever view they want here via the navigationItem.titleView. I think this is another example of not really adding value by allowing customization via the plist. What do you think?


Why this?


карту будем вставлять бэкграундом и не нужно её делать кликабельной


The following doc(s) have been deployed to the [test-commen](http://test-commen-static-docs.pantheon.io/docs) Multidev environment:
- [source/_partials/footer_content.html](https://github.com/pantheon-systems/documentation/commit/222bc615ff7193b0a4b9de5500bb9d0960c396f8/source/_partials/footer_content.html)
- [docs/articles/drupal/8/create-services-yml-file](http://test-commen-static-docs.pantheon.io/docs/articles/drupal/8/create-services-yml-file)
- [docs/articles/drupal/configuring-jetbrains-phpstorm-ide-with-pantheon](http://test-commen-static-docs.pantheon.io/docs/articles/drupal/configuring-jetbrains-phpstorm-ide-with-pantheon)
- [docs/articles/local/cli/machine-tokens](http://test-commen-static-docs.pantheon.io/docs/articles/local/cli/machine-tokens)
- [docs/articles/local/continuous-integration-solutions](http://test-commen-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/wordpress/cloudFront-setup-for-wordpress](http://test-commen-static-docs.pantheon.io/docs/articles/wordpress/cloudFront-setup-for-wordpress)
- [docs/guides/rerouting-outbound-email](http://test-commen-static-docs.pantheon.io/docs/guides/rerouting-outbound-email)
- [docs/guides/two-factor-authentication](http://test-commen-static-docs.pantheon.io/docs/guides/two-factor-authentication)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/0a8faaeb405acb2cd8824c0f59d0f582ec706711/scripts/deploy-multidev.sh)


Try to keep good form in your if-else statements
```
else{
   alert("You do not have a fever");
}


Yes sir, for above function, it's fix the issue.

Another missing function is on `AbstractSoapClient.php`. On line 151

```php
    public function getAllGroupIds() {
        $clientIds  = $this->getAllClientIds();
        $groupIds   = array();
        $groupIds[] = 0;
        foreach ($clientIds as $clientId) {
            $groupIds[] = $this->getClientGroupId($clientId);
        }
        return $groupIds;
    }
```

I can't see any function `getAllClientIds();`.

And on line 170 on same file

```php
    public function domainExists($domainsToCheck) {
        $domainsToCheck  = is_array($domainsToCheck) ? $domainsToCheck : array($domainsToCheck);
        $reservedDomains = array("test.gdmedia.tv");
        $domains         = array_merge($reservedDomains, array_map(function($s) {
                    return $s['domain'];
                }, $this->getSites()
                ), $this->getAllClients()
        );
        $result = array();
        foreach ($domainsToCheck as $domain) {
            $domain   = strtolower(trim($domain));
            $result[] = DomainStatus::create($domain, in_array($domain, $domains) ? 1 : 0);
        }
        return $result;
    }
```

function `getSites()` also missing.

For now, I'm not using this function. Just my editor say that function is missing, so I search to all file in `GDM\ISPConfig`, and I don't see that function, or maybe I just don't see it. :(


looks transitive.  squash to kube bump.


*Actually, test creating new file locally to cloned repo, committing to local branch, and pushing to remote.


I'd rather use ((frames_count - 1) == i) . More safe


@Dbarnes1 you crushing this training, or what?


Do you mean move the cocktails instances to their own JS module in a seperate file, and expose the cocktails array from that module?


Doesn't calling this here mean we'll be calling the endpoint every time the page is loaded? Shouldn't we make sure we only ping it when the subscription actually changes?


This is suspicious.  Where are the missing files?  If they're transitive in another bump, squash them in.


Use `{% reverse %}` template tag


не нужно делать два отдельных параграфа p>span, если будет критично можно и br для отступа


тоже самое


Hmm the Java driver's Values.value should handle null, if not, please file as a bug :)


The thing is we would be adding a 'background-color: #', and it would be incomplete. I would add a default value for it, be it FFFFFF or 3A6A65 as we had before


For example when someone types
`.mix demo`
it will automatically say
`...D E M O P H O B I A: /connect 144.76.176.131 28785...`


значёк копирайта: http://www.w3schools.com/html/html_entities.asp


no sé si el ```&& labels["slave_name"] == ""``` sea necesario, al pareecer siempre viene vacío


Hi Programming team,

Good to see that we are doing commits and using the opportunity to do code reviews!

Let's get in the habit of checking the git comments and emails, and address issues that are identified.

If you have a question about the comments, then you can either talk to/call or email to get clarification

Regards,

Mrs. Koritzinsky

regards,

Ianne

Sent from my iPhone

On Jan 25, 2016, at 3:18 PM, Jeremy Koritzinsky <notifications@github.com<mailto:notifications@github.com>> wrote:

Additionally, this timer will never actually send the brake command since the get method is non blocking and the drive method call will not take exactly five seconds. If get were blocking, it would make the robot nonresponsive for five seconds and break the scheduler for commands and subsystems.

�
Reply to this email directly or view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_NicoletFEAR_StrongHold-2D2016_commit_ced1e59b0bd00c11dc41ad7f088726b3d7bfd176-23commitcomment-2D15661600&d=CwMCaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=nxvSZnTdUAvRJpW4681OHIp_QLk6bRgSdMlDpQXjftc&m=Y9JoaD4uVwZSMiOHhCelTdc_Z3RWUqu8ptF3ifOIeEo&s=NK2nlkbusIxa6jjAd5lxqtlKfEYUIrc7J0macO3BTZs&e=>.


@rodfersou I think we have to fix this by using https://pypi.python.org/pypi/buildout.variables


GNS3 creates exception after your changes

```
Traceback (most recent call last):
  File "/usr/local/bin/gns3", line 9, in <module>
    load_entry_point('gns3-gui==1.4.1.dev2', 'gui_scripts', 'gns3')()
  File "/usr/local/lib/python3.5/site-packages/gns3_gui-1.4.1.dev2-py3.5.egg/gns3/main.py", line 251, in main
    mainwindow = MainWindow()
  File "/usr/local/lib/python3.5/site-packages/gns3_gui-1.4.1.dev2-py3.5.egg/gns3/main_window.py", line 102, in __init__
    self._connections()
  File "/usr/local/lib/python3.5/site-packages/gns3_gui-1.4.1.dev2-py3.5.egg/gns3/main_window.py", line 250, in _connections
    self.uiExportDebugInformationAction.triggered.connect(self._exportDebugInformationSlot)
AttributeError: 'MainWindow' object has no attribute 'uiExportDebugInformationAction'
```


Edit your commit message. I mean grammar. And make it more simple.


I still don't see why we'd expect the *TypeExpr* to have a valid location. The CallExpr is implicit and has an implicit callee; shouldn't it get its end loc from its arguments?

(I'm guessing this code predates us parsing the `-` as part of the literal, and so we never had to worry about a literal being more than one token.)

````none
(assign_expr
  (declref_expr type='@lvalue CGFloat' accessKind=write location=<stdin>:7:3 range=[<stdin>:7:3 - line:7:3] decl=main.(file).zPosition@<stdin>:4:5 direct_to_storage specialized=no)
  (call_expr implicit type='CGFloat' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] nothrow
    (constructor_ref_call_expr implicit type='(floatLiteral: NativeType) -> CGFloat' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] nothrow
      (declref_expr implicit type='CGFloat.Type -> (floatLiteral: NativeType) -> CGFloat' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] decl=CoreGraphics.(file).CGFloat.init(floatLiteral:) specialized=no)
      (type_expr implicit type='CGFloat.Type' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] typerepr='CGFloat'))
    (tuple_expr implicit type='(floatLiteral: NativeType)' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] names=floatLiteral
      (call_expr implicit type='NativeType' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] nothrow
        (constructor_ref_call_expr implicit type='(_builtinFloatLiteral: FPIEEE80) -> NativeType' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] nothrow
          (declref_expr implicit type='Double.Type -> (_builtinFloatLiteral: FPIEEE80) -> Double' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] decl=Swift.(file).Double.init(_builtinFloatLiteral:) specialized=no)
          (type_expr implicit type='NativeType.Type' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] typerepr='NativeType'))
        (tuple_expr implicit type='(_builtinFloatLiteral: FPIEEE80)' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:15] names=_builtinFloatLiteral
          (float_literal_expr type='FPIEEE80' location=<stdin>:7:15 range=[<stdin>:7:15 - line:7:16] value=1.0)))))))))
````


`value.asObject()` should do what you need here


можно обойти без label


ok


Yes, MIME type is language identifier + one language may have multiple identifiers.


Did you consider using an enum or some other, more type safe representation for types instead of using strings? Java driver has some code to that effect but that is probably not directly reuseable here.


Hey Ted,

1. `<li>` should only exist inside of `<ul>` or `<ol>` since they are a part of `unordered LIST` or `ordered LIST`

2. Structure in general is very good. However, do pay more attention to giving 4 spaces for children elements at the start of line.

BEFORE:

```css
<ul>
<li>General Assembly: FEWD: Graduated 2016</li>
<li>Ithaca College: B.S. Accounting: Graduated 2009</li>
<li>School: Graduated 2005 </li>
</ul>
```

AFTER:

```css
<ul>
<li>General Assembly: FEWD: Graduated 2016</li>
<li>Ithaca College: B.S. Accounting: Graduated 2009</li>
<li>School: Graduated 2005 </li>
</ul>
```


разметка должна быть такой же, как и на главной странице


the modifications about random generator was done last week when i tried to compare the transfer times. I totally forgot that~ I will revert this commit....


зачем здесь `form`?


feature_initializer_list_conversion (vrennert) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (385 of 385 tests passed, 0 test warnings)

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


Contributors :
1-7azal2oom
2-zo7le2a
..etc
as simple as that :joy:


оберни форму в блок


TeamCity ReOpenLDAP :: devel [Build 229](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1726&buildTypeId=ReOpenLDAP_Devel&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 62; test(s) failed (new) Build time: 00:11:00
### Failed tests
```
all LDAP tests for mdb/DQD-KFA: test050-syncrepl-multimaster for mdb/DQD-KFA: <no details avaliable>

```


TeamCity ReOpenLDAP :: master [Build 43](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1727&buildTypeId=ReOpenLDAP_Master&guest=1) is now running


thiusi siis s


Is there a case for a) only doing some of these steps (or out of order?) and b) having any module besides `this` trigger these steps?


The following doc(s) have been deployed to the [test-commen](http://test-commen-static-docs.pantheon.io/docs) Multidev environment:
- [source/_partials/footer_content.html](https://github.com/pantheon-systems/documentation/commit/222bc615ff7193b0a4b9de5500bb9d0960c396f8/source/_partials/footer_content.html)
- [docs/articles/drupal/8/create-services-yml-file](http://test-commen-static-docs.pantheon.io/docs/articles/drupal/8/create-services-yml-file)
- [docs/articles/drupal/configuring-jetbrains-phpstorm-ide-with-pantheon](http://test-commen-static-docs.pantheon.io/docs/articles/drupal/configuring-jetbrains-phpstorm-ide-with-pantheon)
- [docs/articles/local/cli/machine-tokens](http://test-commen-static-docs.pantheon.io/docs/articles/local/cli/machine-tokens)
- [docs/articles/local/continuous-integration-solutions](http://test-commen-static-docs.pantheon.io/docs/articles/local/continuous-integration-solutions)
- [docs/articles/wordpress/cloudFront-setup-for-wordpress](http://test-commen-static-docs.pantheon.io/docs/articles/wordpress/cloudFront-setup-for-wordpress)
- [docs/guides/rerouting-outbound-email](http://test-commen-static-docs.pantheon.io/docs/guides/rerouting-outbound-email)
- [docs/guides/two-factor-authentication](http://test-commen-static-docs.pantheon.io/docs/guides/two-factor-authentication)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/0a8faaeb405acb2cd8824c0f59d0f582ec706711/scripts/deploy-multidev.sh)


Added target selection mode toggler into key combo menu (modes: ClosestToMouse, FastestKillable)
Added target lock mode toggler into key combo menu (modes: None, WhenTheyGoToFog, WhenKeyIsPressed)
Added more checks to not use nuke abilities when target will die from already casted nuke abilities
Adjusted size of text in AbilityOverlay
Fixed Illusion checking
Removed Zeus Thundergods Wrath from Specials


Thanks for this, I would like to understand better few things:
1) Why this change is required now?
2) How did you discover we needed this?
3) Is this change going to have an impact if we build with previous versions of Xen?


I dig this. using cover is :+1:


what is this?


@rodsouza9 typo: necesarily -> neces**s**arily ?


indent and clean your code properly here ;)


Ignore all typos.


Correct, looks like some old code made it's way back in the ui file :/


https://github.com/Shopify/liquid/pull/540 merged 429, and is part of a potential v4.0 release.


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


Could you use `value.type` here instead?


Массив срок, надо бы дописать


@hjmjohnson
Hans,

The order of arguments seems wrong:
InterpMode should come before the PostACPCAlignToAtlas, correct?

Regina


feature_initializer_list_conversion (vrennert) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (385 of 385 tests passed, 0 test warnings)

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


I would like ask for help with the bold letters in our Slovak translation.
In our language, we have letters like č/ľ/ň and these letters are always in bold in our and also in Czech translation, is it possible to change?
THX


Only set CC if not already defined (?=). This way, CC from the environment can be used instead.


feature_initializer_list_conversion (vrennert) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: Build Failed

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


112331505_SurfaceHeightRange (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1508 of 1508 tests passed, 0 test warnings)

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


se lo agregó con https://github.com/latam-airlines/mesos-framework-factory/commit/d4e233105bf0a07ec158c84939ee61cb62cff5bb


Oh thanks I am also trying in parallel a commit that points to the "cutting-edge" version of lightningmdb on Github (`https://raw.githubusercontent.com/shmul/lightningmdb/master/lightningmdb-scm-1.rockspec`), like we do for other additional packages (hdf5, lua-pb, nccl).

If anything `docs/BuildTorchLMDB.md` must be updated too to reflect the choice we've made.


make sure to keep the styling consistent and move the css property to a new line after opening bracket. readable code is the best code! :)


lets begin


From the perspective of this module, generators/collectors/processors are all the same just sequenced, so if the answer to "who else would call these events" is "no one", these may as well be ` return Promise.mapSeries(this.collectors.concat(this.generators, this.processors))` and remove the rest. But maybe there's a case for at least `staticSite.after('process')` etc?


I got the following error:
```
gpg --verify libbsd-0.8.1.tar.xz.asc
gpg: assuming signed data in 'libbsd-0.8.1.tar.xz'
gpg: Signature made Mon Dec 14 03:57:28 2015 CET using RSA key ID A4AE57A3
gpg: Can't check signature: No public key
Makefile:78: recipe for target 'libbsd-0.8.1.tar.xz' failed
make: *** [libbsd-0.8.1.tar.xz] Error 2
```
gpg sounds like a "heavy" dependency, especially on very minimal systems. Could be interesting to use it, only if found in the PATH


Don't know why this committed twice. I think my wifi blipped as i committed.


This file is misnamed? Do you have an uncommitted file `src/base/component.js` that's letting this run locally?


done


This is technically wrong to have `<nav>` in `<head>`. If you saw this page in Chrome, yes it would work. And that's because modern browsers are smart enough to any non-compliant elements down to `<body>` in your page. If you inspect element on this, you will see that now this `<nav>` is inside of your `<body>` element. Consider moving this down to `<body>`

Stephen


Why stop setting the handle to zero?


calling it out because it's shared with etcd and @smarterclayton found it to cause issues when there was a difference in version


I wrote some tests. When I enter "rake", I get failures for the tests that are testing my string_segmentation method and passes for the tests that are testing my dictionary method (I have actually written a method for that). That is what I expected to happen. So the tests appear to be....working?


closes #49


It its possible that some Bluetooth device has only a temperature sensor.

It does not happen with Sensirion sensors...


As classes que eu fiz anotação conseguir rodar menos a do SemestreProfessor
, vou ver aqui sua modificação

Att.
   Luana Almeida,

Em 26 de janeiro de 2016 02:29, Leonardo <notifications@github.com>
escreveu:

> Acho que algumas tabelas foram criadas um poucos diferentes, mas foram
> criadas. Se consiguirem rodar aí, só cada um revisar a classe que é
> responsável.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IFBAEunapolisTOP004/SADE/commit/ca521c537afb7e6b324eb86cd675e27bd464d517#commitcomment-15668197>
> .
>


This is tested now?


Overall this refactor is a lot cleaner, and no red flags. Still wondering about having all these sub-components look like modules, directly instantiated here (same question as for the administration sub-modules), or if there's a better way (by config?) we want to register sub-modules.


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


Are this usefull?


I moved any #defines of swap-like functionality into the corresponding .cpp files in both Adafruit_GFX and Adafruit_NeoMatrix to avoid collisions with existing defines. If any subclasses or examples were relying on those defines, they'll need to be updated...a quick skim through some of the more popular device-specific libraries only turned up one so far (in the SHARP Memory Display library)...I'll fix that in a moment. Just a heads-up that the swap() fix may create a temporary ripple of other needed bug fixes in other libs as a result.


sory

On 26. 01. 2016 18:59, grizeldi wrote:
>
> Napačn branch editaš, dela se na branchu multiplayer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mucamaca/Splatoon2D/commit/926cb12a9b8a4f70957510fa177658afce9cd4cd#commitcomment-15684358>.
>


Usually you would pass at registration time in which error you are interessted in (which is also faster):
http://php.net/set_error_handler 2nd arg


While it may seem silly, the variable OFFSETTHERESHOLD has a typo in it if you mean threshold - notice how the variable has an additional 'e'. This may make things difficult for people searching the code later.


Well I meant to say 0.4.0 instead of 0.3.1 (not 0.3.2).
You know what I meant, thanks.


well, i figured you probably expected me to do it :-)


This code is needed to copy the object to local file. Why do you think this is duplicate?


Actually, all the sub-component registering as collectors/processors/generators could be done as `after()` handlers, no need for the main module to even know anything about them or manually iterate them, it would just call its three pipeline steps.


NIT: we could omit the `0` argument by using [Arrays.copyOf](http://docs.oracle.com/javase/6/docs/api/java/util/Arrays.html#copyOf(byte[], int))


I'm not able to make work. I have started the review yesterday and your view was exactly as the CPU Usage view. Now, the view show nothing.
I will investigate a little bit.


godeps hadn't been run cleanly... those files were already up to date, the godep.json was wrong... will squash


Hi @taylorotwell

The ```$this->rules = array_merge((array) $this->rules, $rules);``` broke my validation form.

I'm using custom integer keys and need them on the validation messages. I guess this merge is overriding my keys.


I will fix this tomorrow.


Looks good.


Could we get also rid of the -dev package? That's bad practice, and usually a sign of a dllmap being missing or not set correctly. I think it could map to libgtkmm-2.4.so.1 since both dependencies are 2.4-1


What about using eigenclasses?


this isn't respecting solr defaults. eg: namespaces


I'd prefer an explicit export list over module exports - easier to not mess up the API by accident, and with StringLike(..) its not even very long. I appreciate it wasn't like that before, so happy to merge and fix it up myself.


There are certainly a variety of useful and reasonable thresholds, but there is also a sense of "false positive". At what point is a query against meta data really slow / unbearable? Hard to say.

5 seconds is a random value that certainly qualifies as too slow in most cases, and I'd like to see if we get any feedback / bug reports from this.

I will think again about thresholds, though. Perhaps, the stack trace is a bit excessive for "lower" thresholds, in case of which a simple INFO message might suffice


duck typing v Douglas Adams style:

If it looks like a duck <https://en.wikiquote.org/wiki/Duck>, and quacks
like a duck, we have at least to consider the possibility that we have a
small aquatic bird of the family Anatidae on our hands.

On 26. 01. 2016 18:59, grizeldi wrote:
>
> Napačn branch editaš, dela se na branchu multiplayer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mucamaca/Splatoon2D/commit/926cb12a9b8a4f70957510fa177658afce9cd4cd#commitcomment-15684358>.
>


I don't think we have the ability to easily support bold characters, but we can consider that for next year's report?


Perfect thank you for adding this.  Was on my list of stuff to do.  We also need to add this to other loader but if you can wait we can do all of this when we refactor the loaders in to a base class etc....


yes


Андрей, привет!

Можешь подсказать мне, как нужно назвать папку приложения "Обновление"
правильно в репозитории? У меня она называется Updater.apk. Вроде как
правильно, но почему-то в прошивках проекта в приложении папки с файлами
украинского перевода нет.

А еще в обновлении 6.1.21 почему-то не перевелась погода. Папки с файлами
из моего репо в приложении тоже нет

Спасибо за ответ заранее!

2015-12-07 8:49 GMT+02:00 Andrei Zhukouski <notifications@github.com>:

> А зачем все-таки эти картинки заменять?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bentezhniy/miui-v7-ukrainian/commit/97f90583d66c8c34811b1010a0ceeefa982dc2a6#commitcomment-14813765>
> .
>


feature_initializer_list_conversion (vrennert) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


added to follow-up checklist


Thanks for the suggestion. Do others agree?


Test Dyze sensor 500°C
HEATER_0_MINTEMP 21
MAX_CONSECUTIVE_LOW_TEMPERATURE_ERROR_ALLOWED = 5;
MILLISECONDS_PREHEAT_TIME = 20000;

TEST 7 - Hotend's thermistor disconnected before startup
No error is shown. Temperature shown is 20°C, 1°C below the mintemp error.

TEST 9 - Hotend's thermistor disconnected after heating begins AND before 20 seconds elapsed
Heater is still ON up to the end of the preheating time.

TEST 11 - Hotend's thermistor disconnected after heating begins AND after 20 seconds elapsed
Min_temp error, heater is disabled.

TEST 13 - Hotend's thermistor disconnected after target temperature reached.
Min_temp error, heater is disabled.

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Test 100K NTC
HEATER_0_MINTEMP 15
MAX_CONSECUTIVE_LOW_TEMPERATURE_ERROR_ALLOWED = 0;
MILLISECONDS_PREHEAT_TIME = 0;

TEST 7 - Hotend's thermistor disconnected before startup
No error is shown. Temperature shown is 0°C.

TEST 9 - Hotend's thermistor disconnected after heating begins AND before 0 seconds elapsed
Min_temp error, heater is disabled.

TEST 11 - Hotend's thermistor disconnected after heating begins AND after 0 seconds elapsed
Min_temp error, heater is disabled.

TEST 13 - Hotend's thermistor disconnected after target temperature reached.
Min_temp error, heater is disabled.

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-

The only difference with the original firmware is that when no temperature is set higher than 0°C, the min_temp error won't show. Since no heater should be enabled when the target is equal to 0, the is no real danger not triggering a min_temp error.


I'm about to issue a PR with the above changes incorporated just so you can have a look and see what you think. Initial testing shows that speakers are now behaving properly both initially after `request_stream` as well as when the listener leaves and returns to a speaker range, but I haven't done tons of testing yet.

Let me know what you think and I'll do more testing.


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


ok


Why add this?


Не, ты меня просто не поняла. Этот кусок кода проходит по созданному ранее массиву и у каждого элемента проверяет не начинается ли он с пробела, или не закнчивается он пробелом.

Алгоритм тут такой примерно
```
строка = элемент массива
если строка начинается с пробела то
  если строка заканчивается пробелом то
    новая строка равна старой строке с отрезанным первым и последним символом
  если нет
    новая строка равна строй строке с отрезанным первым символом
элемент массива = новой строке
```


Also in this change: adding a warning for containers killed by OOM Killer during `local_run`.


Told Serg about this fix for Android, Serg told Garvan.


Looks like this file is not Windows specific (e.g., it is mentioned in Xcode project), and CoreFoundation should not be disabled on non-Windows targets. I think the correct code would be

    #if USE(CF)
    #include <CoreFoundation/CoreFoundation.h>
    #endif

Then #if OS(WINDOWS) block follows as is.


The following doc(s) have been deployed to the [deployopti](http://deployopti-static-docs.pantheon.io/docs) Multidev environment:
- [scripts/deploy-live.sh](https://github.com/pantheon-systems/documentation/commit/2716c5bbc47fe388d20ae36852df959ee898eee1/scripts/deploy-live.sh)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/2716c5bbc47fe388d20ae36852df959ee898eee1/scripts/deploy-multidev.sh)


я просто когда резал строки столкнулся с тем, что у меня пробелы в начале могу быть и в конце, поэтому я закпустил дополнительный цикл, который их чистит. А не работало из за того что я с length накосячил. И сейчас проверка проходит везде


We are doubling the resource requirements. Is that ok for e2e clusters? Should we lower the limits?


yeah, got tangled with field validation changes on the next line, fixed in squash


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


My installation only had a `/lib` folder it in, so I dropped the `64` from the end. I was guessing this to be a version difference which caused the `uint` issue, but that turned out to be an `gcc` issue. So poke me if you want this change reverted.


Denis convinced me in person that this is the best way to handle this for now; the places where we split tokens are places where we'd really like to be consistent about it later, but it also requires both lookahead and either lookbehind or a (very small) amount of context.

The downside of this solution is that really *anyone* calling `tokenize` probably wants to tokenize correctly, even if we're not parsing, but the alternatives would be trying to jam this into the Lexer somehow or duplicating work in `tokenize`. So to make forward progress this is probably the best thing we can do.

(It could still use unit tests.)


Remove this comment.


Alex merged something similar to trunk recently: https://bugs.webkit.org/show_bug.cgi?id=153434


"dependancy" vs "dependency"


Generally very well done Arielle! :+1:

- in your `style.css` you have `body#background` selector. There's really no need to be this specific. Either just use the id or just body would be enough!

- No need to repeat `font-family` in every css block either. you can set this in `body {}` block.
```css
body#background {
  background: url('../images/ArielleAndre_forweb118.jpg') no-repeat 50% 50% fixed;
  background-size: cover;
  -moz-background-size: cover;
  -o-background-size: cover;
  -webkit-background-size: cover;
  font-family:'Lato', sans-serif; <-- here
}
```
- you can also group together css properties in one css block. for eg,
```css
div, a, h2, h3, p {
    color: #ffffff;
}
```

- start using classes for styling and not ids.

- lower case letters file names/css class names and ids and no spaces between words. If there's more than one word in class names, use dashes or underscores.

- nice one with `target=_"blank"`. love it!

Stephen


But not usable here as the error handler is exactly there to catch everything, unless temporarily relaxed through error_reporting().


oops. Got it


@AudreyAltman @no-reply @anarchivist I need some help with this one.  I just need to stub an `#index` method here but the `index` above appears to generate a 500 Server Error, and then Rspec complains about an `AbstractController::DoubleRenderError`.


Please do use the plain `super()` call when writing new code for the master branch.


New-Fan-Object (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2053 of 2056 tests passed, 0 test warnings)

 * 549 tests had: AUD diffs.
 * 3 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 3 tests had: Table big diffs.
 * 2 tests had: MTR big diffs.
 * 1 test had: ERR diffs.

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


It may be a better idea to put the MUELU_PROCESS_ETI_TEMPLATE function in one of the CMakeList.txt in src/Utils or src/Utils/ExplicitInstantiation


I am sad that this is the best thing we can do at this point, but I understand.


Here are the tests that I think would be necessary to test this problem.  Am I going in the right direction?


I think that this adds padding above tiles if you have slider disabled


This needs to be `true` for production. You can set up a self-signed cert for it to work on localhost.


Change confirmed


This is still wrong because it does not catch false positives.
I think this indicates a problem with you implementation. In this Builder, there are zero exported package. There is no `Export-Package` or `-exportcontents` instructions. Therefore, this builder must have zero exported packages.


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


The following doc(s) have been deployed to the [deployopti](http://deployopti-static-docs.pantheon.io/docs) Multidev environment:
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/36a926534767192b79eed4912995dacb263d2a46/scripts/deploy-multidev.sh)


Squash this commit with previous.


@gribozavr What is the reason behind this change? Can you link to XCTest on Linux docs that talk about this?


squash with first


Can also use: https://mochajs.org/#inclusive-tests


Ah, never mind. Found the protocol and Foundation's implementation as resources.


Ok, I thought our production redirect to https would cover it. But thanks, better security :)


yep, I'll fix up that pr.


Shouldn't `dataFactory` be `dateFactory` - note the "e"?


woops??


Benchmarked?

Sent from my iPhone

> On 26 jan. 2016, at 19:24, Kevin Vandenborne <notifications@github.com> wrote:
>
> 🙌
>
> —
> Reply to this email directly or view it on GitHub.
>


Nekonvenční pojmenování! (Dočasné řešení problému se složenými atributy, vazby ManyToOne.)


Just the abstract.

2016-01-26 19:55 GMT+01:00 Pablo García Sánchez <notifications@github.com>:

> Minor comments and changes for @JJ although I guess it is already sent :P
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/JJ/modeling-volunteer-computing/commit/59a5d6ec5c7b29bd049c92d617797789c8b819a6
>

--
JJ


Do we not want -e?

From <http://www.davidpashley.com/articles/writing-robust-shell-scripts/>:

> Every script you write should include set -e at the top. This tells bash that it should exit the script if any statement returns a non-true return value. The benefit of using -e is that it prevents errors snowballing into serious issues when they could have been caught earlier.


Could it be just /WebKitLibraries/win/?


Missed a period here. 0 / 10


It's usually bad to have features hiding like that, but is there any way to optionally move them somewhere else?


Yep, woops :) I'll fix it.


Yes. The keys are close to each other on the keyboard. My bad. :-)


# Замечания и пожелания:
1. Предложения и одиночные слова должны начинаться с заглавной буквы
2. Не нужна нумерация item01, item02 и т.д. Достаточно просто item


Can't you replace the magic numeric constant by tmp.getTime() ?


Right, improved the test a little by being explicit about which packages are expected. Still the underlying issue that the "version" property is used to decide if something is export remains, and I don't know how to improve that.


null- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/36a926534767192b79eed4912995dacb263d2a46/scripts/deploy-multidev.sh)


This should be logger.warn().
In any case there's a discussion in #382 about removing this commit. If we keep, it should be done with logging.


This should be logger.warn()


Maybe call it `Concat`?


C'est ici qu'on donne des permissions spécifiques à l'objet


Oh ok shoot I thought I read somewhere that you could just send it in a message.. Did they change that?


There are two files that  are used by other ports:
`include/WebKitSystemInterface.h` and `lib32/WebKitSystemInterface.lib`. That's why I did the same as above for WebKitSupportLibrary.


returned error, added test for invalid `--api-version`


*jpg


Big difference.

![](https://spiritofthething.files.wordpress.com/2013/06/better-off-dead-monique-and-lane.jpg)


See #763 please, that reassignment of AWK in Makefile.system should happen only inside the "ifeq ($(OSNAME), SunOS)" block that precedes it by a few lines.


Now I understand why the test is not failing. The cast function is using Date.parse (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/parse) and not the constructor. This returns the number of milliseconds, not a new Date object as I initially thought...
This leads to, in some cases having a Date object in `value` and in others having a number...
Sorry for the problem, but I think this should be changed to always contain a Date...
new Date( string ).


upstream they feed it discovered versions... not sure that bothers me


I just received a connection request and think this a regression. The old method was user friendlier. Now it's like you pressed 'approve later' as before. What was wrong with the 'old' method? It was quick and straightforward.

I also saw that a new connection is not automatically added to the default privacy group/collection. But maybe that's unrelated.


I'll let you take a look at what they're doing and open an issue upstream if you want


можешь сразу написать путь _index.html_


Updates from Batch 05 (14 files) are ready to merge with the master branch...


I meant check what k8s is doing next time we rebase, to make sure we're still building the right magic container


All issues fixed.


Ici on teste pour un « user », mais à ce que je vois plus haut on pourrait aussi tester pour un groupe et se servir des groupes comme de rôles, c'est bien ça ?


Um did you miss that I supplied code to make it 16 bytes again with in my first comment?


missed the commit message and can't edit it with git -- it should've been:

    Summary:
    This is part of a series of diffs to eliminate circular dependencies between
    directories (e.g., db/* files depending on util/* files and vice-versa).

    I submitted individual diffs for the more complex changes (D53361 and
    D53343). Now there were only simple changes left, so I bundled them
    together in this diff.

    Motivation for changes:

    - util/auto_roll_logger{.cc,.h,_test.cc} moved to db/ because their logic is specific to database logs
    - util/db_info_dumper.{h,cc} moved to db/ by similar reasoning
    - unused db includes removed from util/testutil.h, util/options.cc, util/file_util.cc
    - util/skiplistrep.cc and util/vectorrep.cc moved to memtable/ since they're memtable implementations
    - moved the tests for running a DB on mock/memory environments into DBTest since they require a full DB to test.

    Test Plan: waiting on "make commit-prereq -j32", so far everything's compiled and looks good

    Reviewers: yhchiang, IslamAbdelRahman, sdong

    Reviewed By: sdong

    Subscribers: dhruba

    Differential Revision: https://reviews.facebook.net/D53373


Can be simplified to
```
if !struct_def.is_struct() {
    let def = Def::Struct(self.ast_map.local_def_id(struct_def.id()));
```
and `ctor_id` can be removed


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1246](http://teamcity.monogame.net/viewLog.html?buildId=14049&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:00


Marathon launches the container via container.docker.image, so that all happens before the container is run.  How would marathon set that value from an env var?


Yes it's not yet ready. I'll finish it later. Thanks for your help


The method should return an array, with _all_ of the words represented. This only returns one of the words.


Can you rename "WithConversion", e.g. ConvertingIndexSeeker? Currently it sounds like a boolean parameter.


Are there any plans to fix this in jethro as-well?


OK c'est ici qu'on crée des Rules pour un objet spécifique


you're setting $uid above and $user here, that may cause some inconsistencies


Тут пробелов не хватает, лучше распихать всё по переменным и явно передать в `backgroundImage.set`


По семантике я бы не стала делать это отдельной секцией, нет заголовка и его описания. Это простой блок div.


Надо продумать класс, который будет делать пункт меню активным.


I did, I figured you were just quoting the original code :P I'm going to update it to your code now


Si tu veux, je peux regarder et te donner un coup de main. Tu peux passer
au bureau quand tu veux aussi.
C'est tres bien que tu essais de te baser sur le code de CPU usage. Ca va
donner une super belle vue a la fin. C'est deja super que tu es reussi a la
faire fonctionner de ton coté. Il te faut juste changer le provider pour
calculer la memoire allouée ou desallouee. Je pourrais t'aider la desssus.
N'hesite pas a me contacter.
On Jan 26, 2016 2:12 PM, "Najib Arbaoui" <notifications@github.com> wrote:

> Yes it's not yet ready. I'll finish it later. Thanks for your help
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/inf4990-equipe01/tracecompass/commit/36ab634db4e04ef35e5a8b91025e38ddb1987806#commitcomment-15686164>
> .
>


Please, no puns in the documentation.


Seriously guys!


Commit description should read - "Changing name of return variable" -m "Changing the name of the variable used to determine the return value of the checkIgnoreFiles() function from ans to allowFile, so that the purpose of this variable is clearer"


@sectioneight someone, maybe you, on IRC is requesting someone else be added as co-maintainer on this plugin, can you confirm that?


Very nice. Note that `solve_for_decoders` isn't wrapped by the decoder cache (although if you do wrap it, it may be advisable to fix the seeds or have an option to disable it, so that it doesn't blow up the file system). Also sorting is a bit overkill (min also accepts a `key`).

Also this may be too much for the present time, but it may also be a good idea to support returning the parameters that minimize testing error on unknown data. Otherwise this current scheme is biased to return overly redundant representations (as you pointed out during my talk).


You've set the port or not set the port - or tried both?


@pbauer @thet can we work to include some of this into plone.app.contenttypes?


@batmat yes, that is me on IRC.

Can we please add @kageiit and @jjx to have publish permissions?

Thanks


Both won't run though. It's a little convoluted and not ideal, but was working with getting the `$uid` passed as param, but ideally having the full `$user` object t pass to the `dosomething_helpers_get_current_campaign_run_for_user()` function.


Not sure if useful. Made for self educational purposes :smile:


@gregory38 done, check the commit after this.
Also, the ideal way to run irx's is by using a ps2link.elf. This is what people use on the real hardware, when fetching .expected results.
Ps2link.elf runs irx's from host0:


@dadoonet Thanks!


right, but in the query below it checks for `$uid`


Why are we adding `verbose_name` to some of these fields and not others?


Thrift 0.9.3 is now the required version ?
We need to specify this in the documentation.


Не хватает обязательного атрибута alt.


Yeah, you have to set the "secure" flag on cookies or the browser will send them over HTTP if a man-in-the-middle attacker redirects your site back to HTTP.


I suggest we drop the EqualsConstraint entirely. At some point nUnit is going to have a pluggable formatter, then all constraints will format the values properly.


_minor:_ For each of the three "UserInitiatedFlag" tests consider using the newer verify-after-running approach of OCMock using argument constraints in the verify statement which would allow deleting the `OCMExpect` statements.

`OCMVerify([photosVCMock dismissViewControllerAnimated:YES userInitiated:<YES or NO as appropriate> completion:[OCMArg isNil]]);`

With this change the expectation statement would the reinforce the name of the test making the test intent a bit more clear.


TeamCity pull requests :: netty [Build 1108](https://garage.netty.io/teamcity/viewLog.html?buildId=5080&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4911, ignored: 23 Build time: 00:36:36


Назови как-нибудь slider-points / paginator.


Done


ojo con esto.. si lo vas a dejar static vas a tener que sincronizarlo para que multiples hilos no generen un issue de concurrencia.
Dejalo así.. next time ponete una clase a parte que te devuelva un id válido y encapsulamos la generacion del id fuera de la entidad.


Oh well, I'm trying to have both exists a `$user` object and an `$uid` integer, since `$uid` is used in the query. If the `$uid` is passed in then I use it to find the `$user`, otherwise we just grab both using the `global $user`.

Maybe I'm misunderstanding what you mean though...


I'll throw in a couple notes here:

* Our 177 node should be called "Businesses and Services" now.
* For 185 (Refrigeration and Ice), I'd use NAICS 312113 (Ice Manufacturing), since I think that's closer to what our node is supposed to mean.
* 308 (Social) is a tough one. NAICS 71 (Arts, Entertainment, and Recreation) doesn't include restaurants and bars, so I don't think it's quite right. My first instinct would be to use NAICS 722 (Food Services and Drinking Places), but that leaves out the arts and entertainment stuff in 71.
* Everything else looks good to me!


Рекомендация, можно не исправлять. Списков на странице может быть много, поэтому стоит делать более индивидуальные классы.


Wow! So SDL was manually loading libGL with dlopen?


Done.


64 bit port issues are separate.  It would be a good idea to re-evaluate whether fully unrolled recompiled unpacks dependent on every parameter of the unpack are actually worth it vs more generic partially unrolled unpacks.


@Plotzy Does this mean week 1 is done? Ready for me to review?


null- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/59a4aff329bad217c80811ee37ffa5319ecf3564/scripts/deploy-multidev.sh)


Lo subio rosa


"When You feel blue, just do something meaningful and then everything will be fine!"


this looks good by me


... OK. I put in the u's (in an atempt) to make it not ambiguous.


It may already be that we're spending too long recompiling for vif on x86.  (Or it may not.)


Should the original_location be set to None if no original_location is found or should I let the ```AttributeError``` be left to the user to handle?


XML tidy by Notepad++, mostly :)


Thanks for this, Behnam. Not sure why i didn't see the notification. I made the change to the editorial-guidelines in a separate commit, since i had it open.  The other stuff is actually due to be removed.  I'll do that now.  It's new home is https://github.com/w3c/i18n-drafts.


This is no laughing matter! ;-)


А вот здесь кстати, это врядли будет ссылкой. Просто история о том, кто мы такие. Но не принципиально.


Ah, I see. Thanks for your help!


...and I really shouldn't say recompiled, these are dynamically generated routines, that's all.


+1 to @fergunet 's comments


Сравни, одинаковые ли они по стилям с прошлым списком. Если да, сделать одинаковые классы. Если нет, то придумать более конкретный, он не всегда будет номером -two. Бери по содержимому.


my mistake, youuuriteee


Note: This only works on my local machine. I am trying to get it to work on the server.


I argue that we shouldn't be shipping the compiled thrift sources because they may be version dependent. I created https://issues.apache.org/jira/browse/PARQUET-468 to track


Should be after call to stopMonitoring, surely?


These package are not exported. They just have versions. So calling the macro: `${packages;exported}` is rather misleading. At best it should be called: `${packages;versioned}`.


I don't see any reason to prevent that. As you know my default policy is: public unless there's clear evidence of harm.


Does this show up in Pull Request?


If `Def::Struct` is removed from the list, then `if !is_struct_ctor` can be removed as well.


tbh it doesn't take that long and games usually use 2, maybe 3 different unpack methods, so there isn't many to compile and they do get reused.


Оставь класс social. Не обязательно, что эти социальные кнопки всегда будут в футере. Это просто отдельный блок. И logo, чьи лого, социальных сетей? Тоже не совсем в тему)


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


It isn't ... it's a new name bound by the pattern match. It's there to consume and discard a prefix which is only present in 2.10.x.


Doing so would result in the `_networkReachability` object being destroyed. The existing code does not destroy the object when `stopMonitoring` is called.


<img width="561" alt="sk state of commons problems" src="https://cloud.githubusercontent.com/assets/16230043/12592192/a5a68fe4-c46b-11e5-863d-ada5511667a0.png">

 I am not sure if I described my problem with the letters correctly, just to make sure :). Here is a picture.


Actually I replaced with an invisible unicode character.  Not sure if it's ok.


Hey,

I've noticed that this refactoring changed the initial functionality. Previous implementation was breaking do-while loop if assertion was successful (see line 255-256).

Current implementation waits till count is reached even if $found is set to true earlier.

Hopefully my comments make sense.

Cheers,
Mario


проба коммента


As I said, not sent, only the abstract - JJ


Die Quelle ist gelogen :P


done


I'm trying my best and it is really demotivating read comments like yours, i would like to see your implementation :-1:


this is kind of confusing.  aren't we saying here that devise can do both authentication AND authorization. cancan can do authorization, but we can also use devise and its role system to do both authorization AND authentication?


I think "back_option" is a typo and should be "parent_menu", right?


oops


The following doc(s) have been deployed to the [deployopti]() Multidev environment:
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/fdd37b943d46b7211570f5eab52d56f9b112c677/scripts/deploy-multidev.sh)


frequency == 0 means to disable the timer, which is what an interval of 0 does.


usar metodos de List tales como contains. Por favor, fixealo antes de pushear.


Seriously, quite independently of working around the macro-compat deficiency, I think this is a better implementation (note the two calls on `typecheck` which are removed) and is also net negative in LoC.

FWIW, I spent some time seeing if we could fix the bug in macro-compat, but duplicating the forwarders for the supertype in the companion of the subtype. I think it probably is possible, but it's quite involved and it's not something I felt like doing just at the moment.


please remove `|` too - I think that's the style we are going with


Не надо оборачивать input в параграфы, мы потом с помощью css поставим их каждый на свою строку. И все инпуты надо обернуть в общую форму (form).


only 9%?? lgtm


Please put a dot (.) after the first sentence -- Javadoc is using the first sentence to describe a class in an overview (e.g., see http://www.unitime.org/api/cpsolver-1.3/org/cpsolver/coursett/criteria/additional/package-summary.html)


Этим элементам нам надо будет тоже задавать стили, соот. нужен класс.


`stopMonitoring` uses `_networkReachability` when calling `SCNetworkReachabilityUnscheduleFromRunLoop`. I'm not sure what the intent is here, but calling `SCNetworkReachabilityUnscheduleFromRunLoop` on an object that may have been already deallocated is surely wrong.


TeamCity Waterfall :: Build [Build 39](https://tc.demonwav.com/viewLog.html?buildId=228&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 43 Build time: 00:00:48


Please fix the Javadoc warning - @return tag has no arguments.


The `Weaponised` is to all vehicles, however not all give damage, ´shot´ for the vehicles only those who give.

Yeah, ih forget do realign the lines for the vehicles, sorry


This is a good point--that it is really aimed at people who already have some programming knowledge because otherwise it would be really hard for an artist to use. It isn't a very user friendly language for someone with no experience programming. However, it seems that someone with no programming background could still find some example code on the internet which they could modify to learn how different commands effect the picture, which might ease the learning curve a bit.


fast-forwarding master to borshop-integration-317-gh304 = a88fcd54


esta logica no es normal en un pojo (entidad). Yo haria el objeto inmutable poniendo los fields como final, la clase como final y le saco los setters. No lo fixees, solo quedate con el concepto que una entidad (pojo o dto) no tiene logica mas allá de los getters y setters.


while i was at it, i added the config values to the option.
wasn't sure if ii should make the stream option available as well, so for now it is just set it to true.


No need for the not null checks before any delete. It just adds overhead as delete also performs this check.

C++ standard section 5.3.5 point 2:

"...the value of the operand of delete may be a null pointer value"


I didn't set anything, I just started noticing the errors and tracked it back to this commit. Running on OS X 10.11


esto es del topic 4.


Смотри. Есть ссылки, а есть кнопки. Ссылки (тег a) - при нажатии на них происходит какой-нибудь переход или открытие модального окна. При нажатии на кнопку (тег button) - отправляются данные на сервер и там уже идет какая-то обработка.

В этом случае, при нажатии на кнопку "Показать" - должны отправиться данные фильтра и появиться выборка. Поэтому это должен быть button, который должен лежать внутри тега form, где все чекбоксы.


Great! But will this cause more problems in the future due to these version differences ?
Why did the updated parquet.thrift require Thrift 0.9.3 ?


line of interest, rest was autoindentation by vs2013...


Не все изображения. Нет робота из слайдера и красной с компютером картинки


Thanks


@weblancaster what if we use border-radius: 50% ?


done


Fixed in c795d33aa1906df4af80336e3f6df39344d7632f -- thanks!


Do you even HTML5?


I just merged the PR :)


krzesło


I like this. I didn't really think about it when I worked on it, but ContextFree really does make recursion easier for people who don't have a strong background in CS and understand all the base cases, or edge cases very well.


Давай сделаем sorting-item-arrow. Чтобы понимать, что это стрелочки.
И внутри будет не картинка. Делаем иконку через css.


Like you want, you just need an AirSpy in order to test all work fine
Antti has different AirSpy boards & version for that


Это все таки не кнопки, а пагинация. Давай сделаем класс pagination/pagers.


Продумать активный класс выбранной страницы.


@mhvk Addressed comments and fixed complex conjugation problems leading to the strange frequency structure. Moving the conjugation in the folding code makes no difference to when folding raw ARO data, but is necessary for the real PFB inverted data.


hides
```org
#+TITLE: Org layer
#+HTML_HEAD_EXTRA: <link rel="stylesheet" type="text/css" href="../../css/readtheorg.css" />
```
It might work. Might even work properly.


You're still going to need to accept user input on the character type


[Commit 0f8c21f](https://github.com/xbianonpi/xbian-update/commit/0f8c21fe6487d7af097f221257b3a9a12bf4aa9f) fixed [the issue we discussed](https://github.com/xbianonpi/xbian-update/commit/ad89fc037253504ba3ee11f67ff44631c4a67be2#commitcomment-15629437) in this commit, correct? Thanks!


第三种方法是说，位数每增加一的规则是基于上一次的结果，比如 00, 01,11,10; 到n=3时为{0-00, 0-01, 0-11, 0-10}, 对称逆序{1-10，1-11，1-01，1-00}


Well what I'm concerned about is:
a) Filling the recompiler cache enough with essentially identical unpacks of different lengths (num field) that it actually flushes and start recompiling all that again (easy to test if this is happening).
b) Spending enough time doing the above that it would've been faster not to unroll in the first place.


I have my assert_equal set to pass a string of the words separated. Is this correct or should it set to pass an array of strings of separate, valid words?


Is this demo functional? It doesn't look like the test touches the "Restart" button yet.


The compiled thrift sources are tied to a particular version of Thrift, so they should be automatically generated rather than shipped. The update was not meant to require a particular version of Thrift


nit: I'd prefer to use the ```()``` syntax for multiline imports which is what is being used elsewhere for multiline strings:

```python
from commissaire.handlers.clusters import (
    ClustersResource, ClusterResource,
    ClusterRestartResource, ClusterUpgradeResource)
```


Your image was one of my favorites! I love the "Why".


Right indeed. Would you send a PR?


Primera version


git Ifs install


Looks fine go ahead


Since it is already inside USE(CF) block, this check is redundant.
(In previous comment I've missed the point that all WINDOWS code is for CF)


Nice pyramid in the console. You still need to make sure you have a a tree function that accepts an object as an argument and that you follow all parts of the directions. Let us know if you need help!


Best way to do this would be to remove the adodb library embedded in gacl and instead have gacl use the new upgraded library at library/adodb/


Resolved issue #17


Please add:  This function is solely used in the galleryAlbums endpoint


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


Please add:  This is used via the assets endpoint


When I have time, sure. Just wanted to mention it for future code.


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


so I needed an extra set of parentheses.... some sort of Linter would find that


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


On Tue, Jan 26 2016 11:08:42 -0800, Martin Kroeker wrote:
> See #763 please, that reassignment of AWK in Makefile.system should happen only inside the "ifeq ($(OSNAME), SunOS)" block that precedes it by a few lines.

Ok, sorry, I kind of assumed nawk was available as a link just about
everywhere, but looks that's not the case.

--
Lauri Tirkkonen | lotheac @ IRCnet


Ah, I missed that part.


a) I can't say I've ever seen a VIF Recompiler reset happen, i've seen it say it has about 10 buckets but nothing about resetting, so I don't think it's filling up. Making a universal compiled block which works with all num sizes, while possible, might cause slow recompiler blocks as it'll have to check if there's any more data to process constantly, unless we can compile it in such a way we can create a loop in the unpack and modify the starting number every time, but even then you still have the comparison, I'd rather use up the extra memory.

b) sorry I pretty much talked about this in a) lol I don't believe it would be faster enough to benefit in all honesty.


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


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


Are we intentionally not changing the sln file? e.g. `EntityFramework.sln` to `EntityFrameworkCore.sln`


expected len(act) == 1?


if len(act) != 1?


Late night fixes... I've updated it to make it more robust. Will issue another pull request.


赞！


is this different from the cachedir setting?


this will be done globally after


Hmmm, I don't think it was talked about really. Could rename for consistency.


LGTM :+1:


Should the file be renamed to "compile.sh"?


So, when I'm coming up with tests I'm coming up with all of the ways a legitimate word can be entered? I know you said that we don't need to distinguish compound words but should I add tests for contracted and possessive words?  In short... should I write a test for every combination I can think of?


does it make sense to call this without a cachedir?


SeleniumServer is neither 'hub' nor 'node', it's legacy RC server and it should have no this option. I've added it by mistake, it should be removed.


Aren't these `amp_post_template_head` and `amp_post_template_footer`?


Opened #197.


.address нужно обернуть еще в container


Sadly doesnt fix the error.


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


is there a plan to merge this feature into 2.8 and 3.0 or will it come with 3.1?


So ```$scope.hide()``` already checks for ```$scope.visible``` and does ```$scope.$apply()``` as well. Maybe the ```$scope.$apply()``` should be removed from ```$scope.onMouseUp()``` instead.

I am currently away and don't really have a way to test this properly. If you wanted to test this theory and get back to me about it, that would speed up the time the pull request could be merged in.


@joedolson: this is wrong; woff should be before tiff. 1.4.5 loads tiff even if the browser supports woff.

You also seem to have missed adding woff2 here in the font-face.


Missing 'adds' after the WantedBy?


@adam187 really sorry, didn't notice that. somehow npm showed that last version is @1.0.2


This will be stored for each instance, please move to top-level/companion


komentarz ... 123


I know :( but _WINDLL_EXPORT - was completely wrong


we should file a ticket with dartfmt to support stdin/stdout


Понял ошибку, блоки под мороженки же будут одинаковые, все четыре, поэтому одинаковый класс нужен, исправил)


Definitely a nit picky change. I wouldn't worry about it. The repo name is still `EntityFramework` anyways


Take a look at  [line 5 of the ApplicationController](https://github.com/dpla/frontend/blob/develop/app/controllers/application_controller.rb#L5) - all `Exception`s (and their subclasses) are being caught and `#render_500` is being called.

Also, in [line 15 of this file](https://github.com/dpla/frontend/commit/86c76de04cccecb47997af187c10f97f33cd9f6e#diff-f803dcdbbef9948899992eb475f5af1eR15) you're expecting the response to render the `index` template, but in [line 6](https://github.com/dpla/frontend/commit/86c76de04cccecb47997af187c10f97f33cd9f6e#diff-f803dcdbbef9948899992eb475f5af1eR6) you're telling it to render nothing.


indent


Oh, I see..

What operating system and browser are you using? I don't see those myself.


@coreyfloyd Understood. I have changed it to:
```
self.navigationController?.view.accessibilityElementsHidden = hide
self.navigationController?.navigationBar.accessibilityElementsHidden = hide
self.navigationController?.tabBarController?.view.accessibilityElementsHidden = hide
```
Would this be OK?


why do a string switch when there is an enum?


Oops, I thought it was already in Companion, but missed closing }


Related to code here, opened #198 as suggested by @curtisman.


The following doc(s) have been deployed to the [deployopti](http://deployopti-static-docs.pantheon.io/docs) Multidev environment:
- [circle.yml](https://github.com/pantheon-systems/documentation/commit/d42f60129f8a8b00801cea2e8cb8fb59955f8796/circle.yml)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/d42f60129f8a8b00801cea2e8cb8fb59955f8796/scripts/deploy-multidev.sh)


This commit should have been in the myriax repo, right?


Can we remove this un-used code? Version control has our back if we need it again.


Hi @tarunc – also wondering when this can be published?


In order to retain current Buildship functionality, we are going to need to support setting `gradleUserHome` per participant.


Merged


En ymmärrä miks tää ei päivitä dashboardille tota lukemaa samalla tavalla kun vastaava toteutus Silossa. Silo toimii, tää ei.


Возможно, стоит, по аналогии с "обходись", сделать "смотри"


I think I like `GradleBuild addParticipant(...` : it gives hints as to both the type, and the role being played here.


Хедер можно внутрь контейнера положить, так как фон белый и нам все равно, тянется или нет хедер на всю ширину


I'll reformat this part a bit. I think if one does it the shift & mask way, the #ifdef is unnecessary. Also *words is unnecessary, when one can just take value into uint32_t in one go.


А здесь, соответственно, "избегай"


I see. Thanks


Oh, re, line 15, thanks.  This got mixed up between various iterations experimenting with different solutions.

Thanks for the tip about line 5 of ApplicationController! I will look at it in pry and see what happens when I comment it out.


You already have a reference for `self.stars_group`. You can then replace each star using the array index. Something like `self.stars_group[2] = new_star`.


если я правильно помню ТЗ, то на главной лого не ссылка, то есть вполне можно заменить на span


Could `opts[:layout] = false if ext != '.html'` read `opts[:layout] = false if ext != '.html .xhtml'` and still function as intended?
I can set `page "/*.xhtml", layout: 'layout'` but that seems to overide `blog.layout = "layout"`. Not a significant issue but it would be nice. There might also still be some people restricted to `.htm`.


It works on my machine.


Hi There - touch assignment, huh?  Gave me a headache on Sunday.
Lines 24-35, those plus marks, indicate some syntax issue going on that is not letting the function complete.
The discussions found on the course site through coursera provide some nice assistance.


removed required so that users may choose not to upload new photo upon update


I started to try to use space to give the column width for the columns but then forgot about it.
I also was going to try to get the width of the columns, add them up, subtract the number of characters in
the table heading, divide the difference by two, then  pad the heading by that number - but, then I
found the .center command and just counted the column widths.


you forgot to mention $urlencode:$ variable :p


I only wanted you to know why is this here.
Maybe ```modulePath.IndexOf(assemblyName, StringComparison.InvariantCultureIgnoreCase) >= 0``` ?


Se instala imagemagick : ` http://www.imagemagick.org/`, después de la instalación se comprueba ejecutando el comando en la consola `convert -version`.
Se agrega la `gem paperclip` y se ejecuta el comando `bundle install`


I don't know what a library guard is.


In that case should `pasteboardByFilteringContents:` be commented as 'not thread safe' ?


looks good, except the age-old worry that if cfURL is `null` we'll explode here.
I always put CFReleases in an `if (obj != null) { CFRelease(obj); }` check


WTF


Ah, f#, when I look again, that will match ```\mscorlib\```


That is OK, I named it for you so that it would be something you can find.  Basically, it guards the code so that it doesn't run if it is loaded by a program or a testing framework.  In short, if the file is loaded to only use the methods (or classes or modules, etc) as part of a larger collection of code... we would not want to run the program as an interactive thing, and this would guard against that.


Improves drexelforchrist/drexelforchrist.org#2


Remove the code that isn't being used.  The major parts to this change is the style considerations though.


Since my comment vanished i'm guessing you rebased this commit. This means this also works for repos in subfolders?


new line to be added


commit testing


Gosh I'm dumb. I guess I did do a pull request (and it got accepted). I thought I was looking at my fork.

Fix I guess is to just add a conditional to check which exists...


looks good to me~!


The resulting URL will be signed with Content-Type ="text/plain; charset=utf-8", therefore, this header must be included with the GET on the URL.


Oh, I cannot read. `audioBitsPerSecond` and `videoBitsPerSecond` are already dictionary members in the spec, and this commit is adding them as attributes on `MediaRecorder`, in analogy with `mimeType`. That makes sense to me, but why does the definition say that it should return zero if no specific bitrate was requested? Why not return the actual bitrate chosen, like for `mimeType`? (That does happen for `mimeType`, right?)


Yeah the variables come from my starting template I usually use for my
projects. Normally I define the variables there, but since we have a
separate variable file I just forgot to transfer them over.

And I was using the container to define the base width of the content,
while their parent elements are 100% width, so that if I need to use a
background color for that section (like the footer), it can be full bleed
instead of confined within the body content width.

Is there a better way to do that?

On Mon, Jan 25, 2016 at 11:00 AM, Rob Riggs <notifications@github.com>
wrote:

> What does this div class = container give me that the parent element
> doesn't? Would be nice to reduce here...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/YancyShmancy/dublin-apps/commit/5f24b028f628d3f87537ebd10c432d2316d699d5#commitcomment-15653279>
> .
>


This will make people have to re-setup their data who already use the browser right?  Not a big deal right now today, but maybe it is when we add another 1k tonight/tomorrow?


Caught by https everywhere already but better to not even need to do that. Thanks.


Never mind, I understand it's for the non webview urls :+1:


I made those variables because I'm going to have to import the fonts into it, and since I haven't done that yet, it's easier just to use the variable for now so that when I do import it I can just go back and change the value for the variable rather than doing it for every time I use that font.


Nice catch, copy paste error.


```?? modulePath``` is confusing, at least it confused me. Does it makes sense?
How about
```
if (string.Equals((Path.GetFileNameWithoutExtension(modulePath) ?? ""), assemblyName, StringComparison.InvariantCultureIgnoreCase))
```


Not sure what you mean by re-setup data. I'm still logged in to all the sites that I was before I changed the partition.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12725](https://teamcity.mesosphere.io/viewLog.html?buildId=112611&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:40:06


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12728](https://teamcity.mesosphere.io/viewLog.html?buildId=112626&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:18


It was my misunderstanding at first, was confusing browser's partition (which changes) with webview partition (which has user session data, but doesn't change).


Yeah as soon as I saw that failed Travis task I thought to myself "damn that's annoying it only prints the first line that's wrong". You just fixed it quicker than me :-)


@diogosantana Can you remove all dependencies listed on openEJB from core pom and then in the parent pom under openEjb, add the following?

```java
				<exclusions>
					<exclusion>
						<groupId>*</groupId>
						<artifactId>*</artifactId>
					</exclusion>
				</exclusions>
```


Lost my comment, but you know what's what


Well, this is the "old coinduction" which still works (but the type constructor \infty can be used to prove absurdity).  There is also the "new coinduction" using copatterns and coinductive records.


Sure.


Should that state 'should NOT be on the classpath'?


ah cool


ImageGallery definitely seems like the place most people would start upon
opening CPA. So even if we need to re-do instructional materials, it's
worth it!


Heh... thank you!


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


I'm too quick ;-)


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


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


Painful that we need to include this for all browsers :(


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


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


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


nnnnnnnoooooooooooooooooooo

On Mon, Jan 25, 2016 at 6:58 PM etosch <notifications@github.com> wrote:

> you cut out xena's watermark :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CSWomenUMass/cswomenumass.github.io/commit/18ea94b87eb5d5214b522a8001615f62cc5711f3#commitcomment-15664997>
> .
>


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


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


Yeah :)

I should really write something up like this for the prelude too actually…


AUTO-MESSAGE - JSHint detected erros in the following files: lala.js


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