I love that you guys are continuing to work on this!!!
--
Boisy G. Pitre / Lead Developer
337.781.3570/  <http://www.tee-boy.com/>boisy.pitre@tee-boy.com <mailto:boisy@tee-boy.com>
Tee-Boy
441 Saint Paul Avenue • Opelousas, LA 70570
http://www.tee-boy.com <http://www.tee-boy.com/>
This e-mail message may contain confidential or legally privileged information and is intended only for the use of the intended recipient(s). Any unauthorized disclosure, dissemination, distribution, copying or the taking of any action in reliance on the information herein is prohibited. E-mails are not secure and cannot be guaranteed to be error free as they can be intercepted, amended, or contain viruses. Anyone who communicates with us by e-mail is deemed to have accepted these risks. Company Name is not responsible for errors or omissions in this message and denies any responsibility for any damage arising from the use of e-mail. Any opinion and other statement contained in this message and any attachment are solely those of the author and do not necessarily represent those of the company. <http://htmlsig.com/www.linkedin.com/boisypitre>
> On Jan 28, 2016, at 9:44 AM, Forest Handford <notifications@github.com> wrote:
>
>
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Affectiva/OfficeThreatLevel/commit/33ea5300d637dad54642b05e1a264c5d19acd12c#commitcomment-15734469>.
>


a comment....


Here's a slightly easier way to do this.
https://github.com/cwisecarver/opentrials-demo/blob/develop/app/serializers/document.js#L6


Done in adafe9b. I commented the apply method from the abstract type, don't see much point in it; feel free to put it back


Hi - what configuration are you running on? Are you using a TFS 2015 on-premise server and building with the XAML Build system? If so, please install the TFS 2013 Object Model on the build agents - https://visualstudiogallery.msdn.microsoft.com/3278bfa7-64a7-4a75-b0da-ec4ccb8d21b6


WindowsアプリケーションをGit管理しない理由って、
処理を自分が書いた部分とデザイナで自動生成された部分混ざって
よく分からなくなってしまうからなのかなぁ...


Direct conversion of pointer to ssl, probably better something like:
b->flush = !!r->connection->ssl;


When the RequestSizing function was created, the component sizing methods were moved to a central function. At that time, the HW coil was sizing using InitConvTemp and I couldn't change it then since it would have caused diff's (I do recall seeing this). Also, we used to call CPCW and CPHW, where Cp was passed back as 4180 regardless of temperature. Now we use GetDensityGlycol (I don't recall when this was changed) and temperature does matter. So yes this should be corrected using a separate issue.


:tada: :balloon: :birthday:


In the serializers you'll be passed the model that you're going to deserialize to. I *think* you can peel the name off there.


`layoutSubviews` potentially can get call multiple times. `snp_makeConstraints` is used more for setting up initial constraints then letting autolayout worry about the frame changes. If we wanted to repeatly set the constraints we'd want to use `snp_remakeConstraints`. Howevever, layoutSubviews shouldn't really be called with autolayout code if possible.

I think what we want to do here instead is the following:

1. Move the snp_makeConstraints to the init call
2. Store a reference to the constraints for the toolbar height `heightConstraint = make.height.equalto(0).constraint`
3. Whenever we toggle the toolbar visibility, update the offset of the heightConstraint to 0/ToolbarHeight and call setNeedsLayout to notify autolayout to update the view.


OK


ack, seems sane


Your right. I forgot to test the PDF bit which would have highlighted this.


- [ ] 「Spring Securityから提供されている`DelegatingAccessDeniedHandler`を使用して、`Exception`と`AccessDeniedHandler`のマッピングを行う。」といった感じの方がよい。


Shouldn't you use Locale.getDefault() so that it can fallback to JVM default (which can be set with startup properties)?


:+1:

Maybe we can extend the [test](https://github.com/killbill/killbill/blob/master/invoice/src/test/java/org/killbill/billing/invoice/api/user/TestDefaultInvoiceUserApi.java#L64-L74) to check it's correctly saved?


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


Ich hab das einfach so aus seinen Folien übernommen. (3. Foliensatz, Seite 22)


Should set a default path here, so I don't need to always specify it.  /etc/registeracl/ or something like that.


Bra, syns dere er flinke. Brancher og slikt. Stå på!


Actually instead of storing the reference you could probably use `snp_updateConstraints `.


This does not seem to work on Windows. It works if I add the bool: true to a config file. How is this hack supposed to work?


`serialize_read_cstring` doesn't set `buf` to null. So if it returns `false`, we may end up accessing `buf[0]`.


nice


pull test 2


Should we take into account user names and groups.

QE is allowed to push to production.redhat.io

DEV is allowed to push to qe.redhat.io


Yes, these directories are required, but just if you do not use the option "-u". We are working in the new version that allows to run the script without any additional files. I'll change the documentation to download the directories. Thanks!!


Preparing for behavior 27.


Thanks :D


Damn.... Not only a crash during #333, but I forgot to push the correct files too. [This should set this right](https://github.com/Tricky1975/Star-Story/commit/bbd13a98cdefa19b41ac7ba293691009c46e5026).

https://github.com/Tricky1975/Star-Story/commit/bbd13a98cdefa19b41ac7ba293691009c46e5026


Измени пожалуйста 2015 на 2016 год


Ah! Good catch, I'll fix that.


- [ ] 以降、5.0.1からのコピーだと思われますが、０ベースで（お勧めな）記載内容に見直してほしい。
- [ ] 加えて、インラインコードブロックを適切に使ってください。


I'm getting an error since this change:

Parse error: syntax error, unexpected '[' in /usr/share/icingaweb2/library/Icinga/Legacy/DashboardConfig.php on line 58


we can take this into account only when authn will be in, I can leave spece for this for sure though


Yes, please remove that commit from PR, thanks!


Ok


sure


You're missing tests for `Delay`.


Overriding the Reset function should be rarely needed. I haven't checked, but I don't think any extension does it. It would be great if you could do a search on factory in all the extension source code folders to see if you find Reset() in any of the vtkMRML*.h files.


TeamCity Civic Assembly :: Build backend + frontend [Build 34](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=534&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Issue #3 Fix


this Fixes #174.


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


Frankly I just did it that way because that is what @rick did for the website advanced search.  If you feel it is worth it to refactor these to be simpler I could change both of them.


I don't know what it is either, but it keeps wanted to be there. I'm guessing that some latest version of XCLogger has a watchOS and tvOS target which has now been added to our project and because we both recently ran `.checkout.sh` as we reset after testing the carthage branch it's decided to make those changes?

I don't want to keep fucking about with editing the xcodeproj file to remove them. Shall we just check it in or will it screw up everyone who hasn't updated their carthage recently?


The value for annotation attribute NotNull.message must be a constant expression.
не могу использовать там локализацию


Sure
Thanks


D: sorry, where do you find the network ids?


git add


ну русский язык типа) может когда то будет en


Alternatively, we could move this to the overdue module since it's only used there and have `DefaultInternalBlockingApi#getBlockingStateForService` simply return what's on disk. It looks like only `DefaultOverdueInternalApi` and `OverdueWrapper` would need to be updated to handle the `null` case.


Здесь написано "Добавьте…".


И тут повелительное наклонение


`| bool` is simply a [Jinja2 filter](http://docs.ansible.com/ansible/playbooks_filters.html); it should be built into Jinja2, and if you're using Ansible 1.4 or greater, it should work.

For some reason, it looks like the Jinja2 parser/filtering isn't working in your setup... since Windows isn't a supported platform (even with cygwin) for running Ansible, I'm not sure what you can do to get around this—maybe ask on the Ansible mailing list?


- [ ] 内容的には、CSRFの節ではなく、認可の節に追加すべき内容な気もします。ここで説明するのが適切か見直してもらえますか？


The code does not comply to PEP8.

```
<Result object(id=0xa9cd52f890f54083abea21450b8064e2, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpcfw2nynr/test.py', line=2, column=None) at 0x7f1c6e6821d0>, end=<SourcePosition object(file='/tmp/tmpcfw2nynr/test.py', line=2, column=None) at 0x7f1c6e680a58>) at 0x7f1c6e680a20>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f1c6e680eb8>
```


The code does not comply to PEP8.

```
<Result object(id=0xf7fd6e3692c64c40ad6815163b483d36, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpcfw2nynr/test.py', line=4, column=None) at 0x7f1c6e6866d8>, end=<SourcePosition object(file='/tmp/tmpcfw2nynr/test.py', line=4, column=None) at 0x7f1c6e686e80>) at 0x7f1c6e686470>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f1c6e686c88>
```


> So two calls two resolve calls on two threads can not share a single stack property like this. This lock is superfluous.

True, good call. I'm going to revert that commit.

> We need to seperate the visitor and FieldResolver (with cache) in to two so we can new a visitor each time.

Not sure I totally follow.  Why do we need to new a visitor for each Resolve()?


Чекбокс и ссылка на макете разнесены по краям, для этого их нужно будет флоатить, соответственно ты забыл для них обертку. у ссылки забыл title.


Oh I'm sorry! Will do!


The code does not comply to PEP8.

```
<Result object(id=0x63b03f584d8d40f7bd417e9443170e25, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpcfw2nynr/test.py', line=6, column=None) at 0x7f1c6e680080>, end=<SourcePosition object(file='/tmp/tmpcfw2nynr/test.py', line=6, column=None) at 0x7f1c6e680828>) at 0x7f1c6e6809b0>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f1c6e680e10>
```


Локаль, к слову, есть, и надо ее просто активировать.

> On 28.1.2016, at 13.52, therearesomewhocallmetim <notifications@github.com> wrote:
>
> Алекс, у нас много разных полей, которые по-разному называются в Британии и США, но т.к. у нас нет отдельно локали en_GB, то мы ориентируемся на американский английский.
>
> —
> Reply to this email directly or view it on GitHub.
>


Думаю, тут допустимо сказать "акаунт"


@rgerhards Incidentally came across this in commit history. Actually it still leak if processKafkaParam fails. I think you meant to free in finalize.


Random xD


Давайте для перевода на украинский отдавать русские строки, а не английские?


OK


Happy to drop the 'past and present'. I wonder if 'leverage' has too vague a meaning. Might I suggest simply:

> Document and share experience gathered throughout the processes to build expertise in creating accessible websites


In the base class these members are set to false. This way if a view doesn't need ruler and orientation marker (Chart, Table) simply ignore the OrientationMarkerEnabled and RulerEnabled members. Views that need ruler and orientation (View, Slice) enable them in their constructor.


Thanks! :)


mhm O.o


I will add changelog once we have required git commit tag (#1).


3. Removed useless check command == "ghost"


Whatever this was, it didn't make much sense.


Very cool.


`stop_last` was always true


Not sure if and how important it is, but npm docs specifies that [license field](https://docs.npmjs.com/files/package.json#license) should contain a valid [SPDX](http://spdx.org/) expression, which in this case is `SEE LICENSE IN <filename>`.


ага, вижу что title поисправлял потом.


I'm glad we can lose this.


This pattern matches hings like `udf:tree:123.Status` (where 123 is the primary key of the UDFD and Status is the "default_value" column to search on).  Normal stewardship does not have a default value column, nor has it ever been used for mobile search, so I think it is fine to keep this as `ALERT_IDENTIFIER_PATTERN`


Я думаю, стиль (дефис или без) должен быть одинаков для всех языков.


Yes, I kept it for testing while running from the command line.


have you checked how this affects the UI? When I tried this, CSS was messed up due to simple form overriding the label's class etc.


The code does not comply to PEP8.

```
<Result object(id=0x834ccdcf6a0746eba2922c53a33e40dc, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmph6u_w576/test.py', line=2, column=None) at 0x7f2e064cc898>, end=<SourcePosition object(file='/tmp/tmph6u_w576/test.py', line=2, column=None) at 0x7f2e064ccfd0>) at 0x7f2e064cca58>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f2e064ccb38>
```


The code does not comply to PEP8.

```
<Result object(id=0xd5f71f7d802a44779fae276858a4cc5e, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmph6u_w576/test.py', line=4, column=None) at 0x7f2e064d2208>, end=<SourcePosition object(file='/tmp/tmph6u_w576/test.py', line=4, column=None) at 0x7f2e064d2240>) at 0x7f2e064d2fd0>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f2e064d22e8>
```


Yes , we can do this  instead of hard coded one .


The code does not comply to PEP8.

```
<Result object(id=0x6430fae47e8e47228a5f724ea1dd1d5b, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmph6u_w576/test.py', line=6, column=None) at 0x7f2e064cc780>, end=<SourcePosition object(file='/tmp/tmph6u_w576/test.py', line=6, column=None) at 0x7f2e064cc5c0>) at 0x7f2e064cc940>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f2e064cc2e8>
```


_GNU_SOURCE doesn't do anything on OS X. I did set _DARWIN_C_SOURCE so we get O_CLOEXEC.


Само слово Индекс имеет связь с почтой только в нескольких языках.

> On 28.1.2016, at 13.53, therearesomewhocallmetim <notifications@github.com> wrote:
>
> Мне кажется, тут очевидно, что индекс почтовый, а не какой-либо еще.
>
> —
> Reply to this email directly or view it on GitHub.
>


Makes sense.


Trivial typo fix


Wie wärs mit (Fuhrmann-Folien Sprech):

Ein Schattierungsmodell bestimmt, wie Polygone nach der Transformation und Abbildung mit Farbwerten gefüllt werden. Das heißt, es bestimmt, wie das Beleuchtungsmodell an einer gewissen Fläche ausgewertet wird. _(Bin mir unsicher, ob diese Umformulierung richtig ist. (Letzter Satz: Foliensatz 3, Seite 44) )_

Beispiele für Schattierungsmodelle sind:
_Hier würd ich dann die bereits aufgelisteten einfügen._


How about moving this into a separate file cmake/MacOSXPaths.cmake and just use

    include(MacOSXPaths)


аккаунт? По-русски же не "имя пользователя"


I wanted to differentiate it from custom fields on line 11. I agree it's not great though :D


@jonpryor Just FYI I've made some changes (in addition to this commit at least 4279991 and 0f83276b as well), to mono-api-info that makes it 3x faster, but unfortunately the output is also sorted differently, so the resulting xml files changes.


Эту надпись нужно будет картинкой вставлять


The code does not comply to PEP8.

```
<Result object(id=0xdf484d3e05d4e6492d4caaf5ee1cea7, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpujuenw8r/test.py', line=2, column=None) at 0x7ff4cffb0438>, end=<SourcePosition object(file='/tmp/tmpujuenw8r/test.py', line=2, column=None) at 0x7ff4cffb0c88>) at 0x7ff4cffb0630>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7ff4cffacc18>
```


что за класс выравнивание?
это лишний элемент. Все можно задать для article. Кстати названия дублируются теперь на строках 147 и 152. в прошлом комите замечания как раз по попам были, посмотри.


The code does not comply to PEP8.

```
<Result object(id=0x2813967412c04971a7949e6b12c1eadc, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpujuenw8r/test.py', line=4, column=None) at 0x7ff4cffafeb8>, end=<SourcePosition object(file='/tmp/tmpujuenw8r/test.py', line=4, column=None) at 0x7ff4cffafe10>) at 0x7ff4cffaf4e0>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7ff4cffaf390>
```


The code does not comply to PEP8.

```
<Result object(id=0xe86d68633fcc483cb7f2bdc0327f330b, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpujuenw8r/test.py', line=6, column=None) at 0x7ff4cffac438>, end=<SourcePosition object(file='/tmp/tmpujuenw8r/test.py', line=6, column=None) at 0x7ff4cffac7b8>) at 0x7ff4cffac748>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7ff4cffacc18>
```


sup


a cosa serve tutto cio'???


Дима, строки с высокой вероятностью изменятся вместе с UI на момент их добавления.
Я согласен с тем, что не нужно их терять. Для этого оставим закрытый пулл реквест.

> On 28.1.2016, at 13.57, Dmitry Yunitsky <notifications@github.com> wrote:
>
> 1. Мы экономим место в гитхабе? Строк немного больше получается, да - но они все переводились не на пустом месте, Игорь отдавал их и подразумевается что где-то они используются. Не добавить их сейчас равносильно ещё одному кругу добавления строчек через некоторое время.
> Предлагаю не отвлекать сейчас столько сил на определение “нужны-не нужны строки”. Уверен, что в редакторе все новые строки так или иначе понадобятся.
> Плюс, если сейчас их не добавить - совершенно не факт, что у Наташи эти строки где-то сохранятся. Вполне может быть, что как раз если не добавить их сейчас - они могут быть утеряны к следующему разу, когда они действительно понадобятся.
> > On Jan 28, 2016, at 1:05 PM, Alex Zolotarev <notifications@github.com> wrote:
> >
> > Я против добавления не используемых строк. Они никем не ревьюятся и забивают место. А к следующему релизу Наташа заново их переводит, потому что все забыли, что они уже были вставлены в код.
> > Где конкретно запланирована эта строка в нашем UI? В каком кейзе в будущем мы ее будем вставлять?
> > —
> > Reply to this email directly or view it on GitHub <https://github.com/therearesomewhocallmetim/omim/commit/49e8ada922b6628e35c80a68e4a5862143879a6e#commitcomment-15728483>.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>


её


This should be in your global .gitignore instead.


Ein Glanzlicht ist doch eine spiegelnde Reflexion oder? Ansonsten reden wir natürlich aneinander vorbei :)


yeah that is kind of strange. i think originally i was thinking that the default would be a string constant like `"MAIN"`, but then somewhere along the line decided it was easier to do it like those. agreed that this should be `[]`. will update


Это тоже картинка


И это картинка


Less good at base infiltration scouting and easier to find with a sweep.


:+1:
Also the logic in `org.jetbrains.kotlin.cli.CliBaseTest#removePerfOutput` is no longer needed


Только на русском фраза заканчивается восклицательным знаком. На других языках точно не нужен?


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


Now it seems that page loads pretty quick. I need to refactor some things but this is some working solution. Additionally I want to extend errors to also return Reason(machine understandable string) this way we could map some learn more links to errors.

I didn't wanted to extend our current api very much, but also I wanted to avoid calling core again to get the data that is already there.

@bryk could you take a look?


@tskogberg Ah, good point. I'll leave it in since it's less work, but if the build ever breaks in 1.9.3 it's good to know we could probably just remove it.


Может, «Не зарегистрированы в OpenStreetMap?»?


I just initiated a search. I will report back shortly.


The code does not comply to PEP8.

```
<Result object(id=0xc715fc20d19b4cd08c6bd5acfd4b1c4b, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpmhy6iwig/test.py', line=2, column=None) at 0x7f06c2a9b240>, end=<SourcePosition object(file='/tmp/tmpmhy6iwig/test.py', line=2, column=None) at 0x7f06c2a9ba58>) at 0x7f06c2a9bd68>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f06c2a96c50>
```


The code does not comply to PEP8.

```
<Result object(id=0xc777956f48704e63a1b7e71d5513430d, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpmhy6iwig/test.py', line=4, column=None) at 0x7f06c2a9c908>, end=<SourcePosition object(file='/tmp/tmpmhy6iwig/test.py', line=4, column=None) at 0x7f06c2a9cac8>) at 0x7f06c2a9ce48>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f06c2a9c278>
```


The code does not comply to PEP8.

```
<Result object(id=0xd803981ddb1c4d34b39ede8680039294, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpmhy6iwig/test.py', line=6, column=None) at 0x7f06c2a96860>, end=<SourcePosition object(file='/tmp/tmpmhy6iwig/test.py', line=6, column=None) at 0x7f06c2a96390>) at 0x7f06c2a96400>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f06c2a96be0>
```


Who let all these Ricks in here? :-)

I don't feel strongly about this. Consistency is good.


ja, ist es, es ist eine Näherung von Phong und Blinn in diesem Fall.
Wenn es so bei ihm in den Folien steht, dann sollte es passen. :)


А эти переводы?


может, выгрузка?


Agreed. Thanks.


лучше action и method оставлять пустыми, программисты сами заполнят


Это заголовок или кнопка? Если второе, то «восстановить»


Yeah, this works well. The difference is using `f.label` as a separate tag vs using the `label` attribute on the `f.input`.

> On Jan 28, 2016, at 10:03 AM, Nicholas Olds <notifications@github.com> wrote:
>
> have you checked how this affects the UI? When I tried this, CSS was messed up due to simple form overriding the label's class etc.
>
> —
> Reply to this email directly or view it on GitHub.
>


- [ ] `@P` ではなく、 `@PreAuthorize`をインタフェースのメソッドに定義している場合では？＋`@P`を省略している時の注意点だと思います。状況を整理してください。


This line of code makes me tear.


Oh, interesting, completely missed that file. Maybe it's not in use? I'll investigate.


Кажется, исправил это?

> On 28.1.2016, at 14.35, Dmitry Yunitsky <notifications@github.com> wrote:
>
> Опечатка, удАлить
>
> —
> Reply to this email directly or view it on GitHub.
>


@ash2k you may apply this to other volatiles if you find 'same' pattern


Was totally about to do this, when I saw you'd already done it!


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


Может, Sign In / Sign Out?


The code does not comply to PEP8.

```
<Result object(id=0xbd6cbfe042464168b6cee7eff818a25b, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmp18wafn3m/test.py', line=2, column=None) at 0x7f3370693128>, end=<SourcePosition object(file='/tmp/tmp18wafn3m/test.py', line=2, column=None) at 0x7f3370693dd8>) at 0x7f337068e470>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f337068ef28>
```


The code does not comply to PEP8.

```
<Result object(id=0x415a0833bca84ba1ad6cf84950a66d06, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmp18wafn3m/test.py', line=4, column=None) at 0x7f3370697518>, end=<SourcePosition object(file='/tmp/tmp18wafn3m/test.py', line=4, column=None) at 0x7f33706972e8>) at 0x7f3370697668>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f3370697f60>
```


The code does not comply to PEP8.

```
<Result object(id=0xbb2f861c346840439b36a8a4853d682d, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmp18wafn3m/test.py', line=6, column=None) at 0x7f337068ee48>, end=<SourcePosition object(file='/tmp/tmp18wafn3m/test.py', line=6, column=None) at 0x7f337068e9b0>) at 0x7f337068e5f8>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f337068ea58>
```


Ok, I will fix it. Thanks for notifying me.


添加数据库自启动服务。修复服务器重启后./server.sh start 会提示错误——“Starting jumpsever
service:run_websocket.py not running”


I say lets go for it and check it it because its super annoying.


The code does not comply to PEP8.

```
<Result object(id=0x8179192394184ce9a8aa6d5a3b859078, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmplmpa7jy7/test.py', line=2, column=None) at 0x7f80d51f8438>, end=<SourcePosition object(file='/tmp/tmplmpa7jy7/test.py', line=2, column=None) at 0x7f80d51f8ac8>) at 0x7f80d51f86d8>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f80d51f8908>
```


The code does not comply to PEP8.

```
<Result object(id=0x96fa11fa720c45398c8d9bc19b5c4429, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmplmpa7jy7/test.py', line=4, column=None) at 0x7f80d51fa588>, end=<SourcePosition object(file='/tmp/tmplmpa7jy7/test.py', line=4, column=None) at 0x7f80d51fad30>) at 0x7f80d51fa438>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f80d51fac88>
```


The code does not comply to PEP8.

```
<Result object(id=0x3c27c3a7b9d142958a814e0a52e5f9c2, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmplmpa7jy7/test.py', line=6, column=None) at 0x7f80d51f3048>, end=<SourcePosition object(file='/tmp/tmplmpa7jy7/test.py', line=6, column=None) at 0x7f80d51f3860>) at 0x7f80d51f38d0>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f80d51f3630>
```


as I understood for usb-sound cards


The code does not comply to PEP8.

```
<Result object(id=0x92ab7bb8e631437f8c66905a8481e400, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmprj_ek17l/test.py', line=2, column=None) at 0x7f9972404828>, end=<SourcePosition object(file='/tmp/tmprj_ek17l/test.py', line=2, column=None) at 0x7f9972404240>) at 0x7f9972404748>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f9972404a20>
```


The code does not comply to PEP8.

```
<Result object(id=0xe9119afb4dd84b1b9fe81221022a96cf, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmprj_ek17l/test.py', line=4, column=None) at 0x7f9972409470>, end=<SourcePosition object(file='/tmp/tmprj_ek17l/test.py', line=4, column=None) at 0x7f9972409fd0>) at 0x7f9972409a20>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f9972409f98>
```


The code does not comply to PEP8.

```
<Result object(id=0x7511b3d4824c4a1e8b1c9447372c74a4, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmprj_ek17l/test.py', line=6, column=None) at 0x7f99723ff6a0>, end=<SourcePosition object(file='/tmp/tmprj_ek17l/test.py', line=6, column=None) at 0x7f99723ff828>) at 0x7f99723ff518>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f99723ff550>
```


A new addon version only for some withespace cleanups? What are you doing?


Тут неправильная формулировка. "Оскiльки" означает "поскольку", то есть неверно перевели оборот "as it is".


information of the лишнее. Также, я бы добавил запятую перед and.


- [ ] 「view名として"login"を返却するメソッド。」は「view名として"login"を返却する。」でよい。


Please add a full stop to the end of the description and change the NodeJS to Nodejs.


fixing @kehusa removal of the JS HelloWorld tutorial :D


Watch these little spaces!


The code does not comply to PEP8.

```
<Result object(id=0x1d0cefd10ef8438cb2a2402eb879e516, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpk7z_qrj8/test.py', line=2, column=None) at 0x7f65fb190278>, end=<SourcePosition object(file='/tmp/tmpk7z_qrj8/test.py', line=2, column=None) at 0x7f65fb190748>) at 0x7f65fb190080>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f65fb190940>
```


The code does not comply to PEP8.

```
<Result object(id=0xf849ba7ef6ed4c478408591c92585742, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpk7z_qrj8/test.py', line=4, column=None) at 0x7f65fb193cc0>, end=<SourcePosition object(file='/tmp/tmpk7z_qrj8/test.py', line=4, column=None) at 0x7f65fb193240>) at 0x7f65fb1938d0>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f65fb193cf8>
```


Может, «правок»? As in «правок: 4».


The code does not comply to PEP8.

```
<Result object(id=0xeb488821cb2342edbd89542461b9d5c2, origin='PEP8Bear', affected_code=(<SourceRange object(start=<SourcePosition object(file='/tmp/tmpk7z_qrj8/test.py', line=6, column=None) at 0x7f65fb18b860>, end=<SourcePosition object(file='/tmp/tmpk7z_qrj8/test.py', line=6, column=None) at 0x7f65fb18b080>) at 0x7f65fb18b588>,), severity=NORMAL, message='The code does not comply to PEP8.') at 0x7f65fb18b278>
```


Почему Languages с большой буквы? И что с остальными переводами?


![screen shot 2016-01-28 at 10 11 51 am](https://cloud.githubusercontent.com/assets/378969/12648355/9e16ebd8-c5a7-11e5-9148-ac5100b3f902.jpg)
… see yeah if those big long type-signature lines that contain `im::InterleavedImage` actually had `_Nullable`, `_Nonnul`, or `_Null_unspecified` in there somewhere – I don’t care where just somewhere – I’d be a bit less irritable circa now-ish


TeamCity Civic Assembly :: Build backend + frontend [Build 34](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=534&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 33 (13 new), passed: 1704, ignored: 7 Build time: 00:20:21
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1453993803199: <no details avaliable>

Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should not create new media type if cancelled: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

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


I snuck in a `bundle exec` too :)


I assume this extra `res => {}` step was just here for debugging purposes.


Введите?


choice-block и catalog должны быть в обертке с clearfix.
пагинатор не должен находиться на одном уровне с catalog-item, лучше их обернуть в элемент, который будет на одном уровне с пагинатором.


Line 10 - why is the first parameter `0`? And why is the second parameter `toplimit+1`? Also, remember Python is case sensitive, so `toplimit` is not the same thing as `topLimit`


We could also add now the test for the persistHandshakeReactResult.


I'll have to look at this one in context before I take any action, but you may well be right, Lorenz. Thanks. :)


Подробности? (см.  cs)


but that's useless without a caller.


Слова в надписях на кнопках не капитализируются


Указать улицу?


> Ich find, daß Beleuchtungsrichtung nicht korrekt ist, weil ein Punktlicht 360 Grad strahlt, da gibt es keine Richtung.
Naja, die Beleuchtungsrichtung hat ja nichts mit der Lichtquelle zu tun, sonst nur mit dem beleuchteten Objekt. Auch wenn eine Punktlichtquelle 360° strahlt, so wir ein gewisses Objekt trotzdem aus einer bestimmten Richtung bestrahlt. Ändert man die Position der Lichtquelle, ändert man indirekt auch die Richtung aus der das Objekt beleuchtet wird.


Why move the actual toggling out of this method?  Seems like it's now mis-named :)


Hmm, the purpose of the Fixer is setup and concurrency management (perhaps these two should be separate to make this clearer).  The application logic lies in toFix (found in fix.go).

I can't help but notice that the second example you have given there is *extremely* close to what occurs, just with functions wrapped around each section.  Maybe this is causing issues?
NewFixer() sets up all of the go routines doing almost exactly:
`var wg sync.WaitGroup
for i := 0; i < 10; i++ {
  wg.Add(1)
  go func() {
    defer wg.Done()
    for order := range inCh {
      res, err := f.Fix(in)
      …
      Log(res)
    }
  }()
}`
f.QueueChain() adds orders to the inCh
f.Wait() (closes inCh and) does the final `wg.Wait()` line.


For the below line 119, where `execute()` is invoked, we could add a comment to inform that the output on stderr of the child process will be discarded.
Otherwise, we could explicitly set `:combine` to `false`, to make the intent clearer.


@hudokkow are you sure always returning true is okay and will not have any bad side effects? Why not false? ;-) My question is not just for this addon, but holds for for all 4.2.0 API PRs you just opened.

@FernetMenta is this save to do???


"not yet supported" is also not accurate since it never needs to be supported. This size is used in Windows code only to do buffer allocations in Encrypt path. On Unix, the library allocates the buffer and returns as output


xD


' убери)


Какой тут контекст? Здание как постройка или как "дом" в адресе?


For client, only Init will be set - Accept is meant for server code path


Fixed


Two empty lines here, between functions, whereas elsewhere we're using one.


That said, I don't see any harm in setting both - it will also make the 2 methods consistent


Bad idea or not, that may be what happens.  Of course it would crash a real console as there are interrupt handlers right at the start of the kernel.


Аналогично с e-mail/email


Womp. My bad. Thanks for the fix!


- [ ] いったん「・・は4.0**以上**を使用する前提である。」にしましょう。


This can go.


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


Students/30422/Negrea V. Mihnea Vlad/Bricks/src/Components/Brick.java

Why do you have a static reference to a `Brick` object inside of the `Brick` class? Why not just create an instance of the object!?


Porque é que estás a imprimir o máximo e o mínimo? Estás a fazer debugging? Não faz sentido imprimir o que acabaste de ler.


Key revoked on Heroku's end.


Students/30422/Negrea V. Mihnea Vlad/Bricks/src/Components/ball.java

Class names should start with upper case letters


+ также не должно быть в id.
http://www.456bereastreet.com/archive/200902/creating_valid_names_with_the_id_attribute/


Students/30422/Negrea V. Mihnea Vlad/Bricks/src/bricks/game/Game.java

You don't need this


group:tree trunk *8 + 1 mese block is maybe better ?


`/3/ =~ i.to_s` でいいですね。 `/3/.match i.to_s` というのもあります。（ちょっと動き違うけど）


I don't understand why you keep a static reference of this class inside the class itself


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


TeamCity Civic Assembly :: Build all [Build 32](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=535&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Hi ,
How can  I change the defult route /Home/Index  instead of index.html ,
I tried to change that  but when I run the porject  (index.html) show again  .
Thank you capesean ..
:+1:


Students/30422/Negrea V. Mihnea Vlad/Bricks/src/bricks/game/Mario.java

This should not be static (nor public for that matter)


Neither should this


s/inwanted/unwanted


what do you think about moving main.go into app/pillar?

[10:01]
The thoughts being:
1)  that we may want to be able to build more than one flavor of pillar, and
2) the repo is decoupled from the idea of a single program, and that programs are always buildable from subdirs in app


Или "Подробнее"


This is a bug, I fixed the same for `Organization` here: https://github.com/stormpath/stormpath-sdk-node/commit/175ce35a536d0dbd43a3150968469e486dd6015c


I like your authentication example. Is there a way to set the user and password with the help of the [Jenkins Credentials Plugin](https://wiki.jenkins-ci.org/display/JENKINS/Credentials+Plugin) maybe in way like it is used in the [job dsl wiki](https://github.com/jenkinsci/job-dsl-plugin/wiki/Handling-Credentials#the-credentials-plugin)?


Because you used `a lot of` static, your code tends to be extremely coupled and in a very bad way. Please refrain yourself from using the static keyword for now, you shouldn't need it. Simply pass references wherever needed


Лучше вместо недоссылки просто #


Should strings like OK and Cancel that are used in many places be moved to somewhere like UIConstants (This is why I think we should move to ids for strings rather than using the string itself as an id)?


It depends on how this addon handles recordings. If recordings are handles by DVDInputStreamPVRManager, means the start with pvr://, I won't consider this safe.


Malbone testita – kaŭzas problemon kun la __str__ funkcio (uzata interalie por kunloĝantoj).


And it has fixed issue with [pypy](http://pypy.org/) :)


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


И тут недоссылка


`print "AHO "` が２回でますけど、この行を `if i % 3 == 0 || /3/ =~ i.to_s` で１行にまとめるというのもあります。


I believe we dropped our `d.ts` file in favor of pointing people to DefinitelyTyped. @christopherthielen does that sound right?


Typo.


Hi Barry
I'm still trying to figure out this GitHub thing - did I do this right?

Also, feel free to make any corrections/deletions/suggestions on the code - I was mostly just trying to get the hang of dealing with structures.

Thanks!


there should always be an alias for the master branch if you release from it.


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


Thank you!


Calling Windows API [PathCchAddExtension](https://msdn.microsoft.com/en-us/library/windows/desktop/hh707080(v=vs.85).aspx) will do the job of adding an extension if one is absent and would considerably simplify this code


I fixed this here: https://github.com/stormpath/stormpath-sdk-node/commit/175ce35a536d0dbd43a3150968469e486dd6015c


@Lactem Noticed this issue (language spacing) using the broadcasting lottery status feature when there is no minute in the broadcast.

![link](http://i.imgur.com/V6jlNj8.png)

Also, any possibility in allowing a setting to define when to broadcast? Such as on the hour mark every hour instead of whenever the cooldown begins so no minute or second time is listed?


This commit introduced a bug by changing the electromagnetic field component variable names on line 426. This causes any code using emf_macros.mortran to fail compilation. This bug is fixed in commit 2a36e18.


I think you could shorten this by going:

```$details.toggleClass("open", close);```

(Though might wanna rename the "close" variable in this case.)


Just to clarify, the reason for this is that windows users don't want an empty terminal window.  Mac and Linux can suck it up.


Thanks, I expected time to be installed by default.  Please send a pull request.


весь aside - одна форма. Производители и группы товаров лучше сделать fieldset + legend


You may want to do this to save some additional space after everything is installed:  `&& rm -rf /var/cache/apk/*`


nice!


Maybe I miss something but it seems to me that checks start from line 1538 to 1541 are unnecessary.
Why you need them?

MovieContextMenu is called from def doContext: https://github.com/OpenPLi/enigma2/blob/8ef48b7019595da598cbbf88f385b49036f6ff91/lib/python/Screens/MovieSelection.py#L1191
where already is:
current = self.getCurrent()
if current is not None:

Before you call csel.isBlurayFolderAndFile() already is chech:
if not (service.flags & eServiceReference.mustDescent): https://github.com/OpenPLi/enigma2/blob/8ef48b7019595da598cbbf88f385b49036f6ff91/lib/python/Screens/MovieSelection.py#L395

I think that it would be better call isBlurayFolderAndFile with service and get getPath from that without aditional checks:
elif csel.isBlurayFolderAndFile(service):
...
def isBlurayFolderAndFile(self, service):
          path = service.getPath()
          folder = path + 'STREAM/'
          ....


It's valid for `YNMP_DB_HOST` to be empty (I have that locally, for example) which would mean that the index name begins with `_` which isn't allowed - just swapping `YNMP_DB_HOST` and `YNMP_DB_NAME` should work fine.


This was not generated by the program, but it's better than having an empty file.


Don't need anymore with GUI


@ruhri please better file a bug report on hub.qgis.org rather than commenting to a commit (that is most likely unrelated)


В общем, всё норм


I guess you need that for the `/metrics` endpoint, right?


I would add it right above Guava.


Students/30421/Alb V. Corina Bianca/Gin Rummy/app/src/main/java/com/example/corina/ginrummy/Activities/Play.java

First off, from the get-go, this class is too big


これ、正規表現だけわかればできる、という感じの問題でした :clap: ( ﾟДﾟﾉﾉ☆ﾊﾟﾁﾊﾟﾁﾊﾟﾁﾊﾟﾁ

これを知ってると簡単にできるようになること、というのがものすごく多いので機会有ったら是非使いましょう :hammer:


We  don't expose that property in the other cache implementations. I don't think it's useful to do that as preventing null values is a corner case that does not warrant a dedicated property IMO.


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


@mr-salty I'm not sure if this change helped. I tried out to install this plugin on the latest two version of [td-agent (2.2.1, 2.3.0)](https://docs.treasuredata.com/articles/td-agent-changelog#20151224-version-v230) and on both versions the embedded gem command installed the wrong version of `google-api-client`.

```
# /opt/td-agent/embedded/bin/fluent-gem install fluent-plugin-google-cloud
Fetching: google-api-client-0.9.pre5.gem (100%)
Successfully installed google-api-client-0.9.pre5
Successfully installed fluent-plugin-google-cloud-0.4.14
Parsing documentation for fluent-plugin-google-cloud-0.4.14
Parsing documentation for google-api-client-0.9.pre5
Installing ri documentation for google-api-client-0.9.pre5
Done installing documentation for fluent-plugin-google-cloud, google-api-client after 79 seconds
2 gems installed
```

I wasn't able to see what is the actual problem. Do you think I should create a new issue here or on td-agent support?


Idem, right before Guava.


This code is duplicated below, I think you can optimize this


yay! I was going to suggest you did this earlier but thought I would wait to see what happened later :)


Will do tomorrow :)


The change from CPCW/CPHW to GetDensityGlycol should be revisited. If a variable temp is being used, seems it should be tied to the Sizing:Plant design supply temp or the coil object Rated Inlet Water Temperature for coils that have that input, or the average of the design input/output water temps?


Yes, that's a known issue. It's a general issue in fact and the problem is that we have a triple license. Maybe npm got smarter and accept it now.


It's actually a stack. It represents the nesting of the type we are processing recursively (in this case, recursion in the processor, not necessarily recursion in the type structure), and we pop stuff off as we finish processing the nested types and go back up to the parents.

I've tried to add comments to that effect in the code as you suggested, let me know if you think they should be more clear.


Haha nickel :D


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


Even if apk cache is not enabled?

> On 28 Jan 2016, at 16:30, Andreas Linz <notifications@github.com> wrote:
>
> You may want to do this to save some additional space after everything is installed: && rm -rf /var/cache/apk/*
>
> —
> Reply to this email directly or view it on GitHub.
>


I have no idea if its safe or not.
I just grepped add-ons for timeshift capabilities and added the comment you can read above. Don't have time to investigate further.
If its safer to return false, I'll change the commits accordingly.
@FernetMenta, your call.


To be clear, I mean I've tried to add them in the commits which I'm about to add to this pull... they should be there soon, but weren't *already* there.


@kamk cool, will probably be using some of the binascii tricks for the Python 3 porting.


There is an almost identical method below: `verifyComputerCards`. The main difference is that you are in one place checking `mPlayerCards` and in the other `mComputerCards`. Are you sure you can't somehow make just 1 function out of these and then pass some extra parameters just to not have so much duplicate code?


スッキリしていて読みやすいです :100:

ループ処理、条件分岐、正規表現辺りが分かるとプログラマ入り口に入った感じです :exclamation:


sounds right  #1990

The 1.0 branch is written in typescript


stone*8 + 1 mese block is maybe better ?


hey gregg. It works! thanks so much.
But it's still bit buggy.
It runs well when I click first on pause, then paste the new url, and then click play again.
it still crashes when I past the link first into the input field then click on pause and then click on play.

anyway, you're my connect()-hero for today!
best,
christian


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


price + btn обернуть. т.к. он используется на странице детализации. Чтобы не было проблем с потоком элементов и велосипедов с заданием размеров.


This won't work if there are multiple environments with different prefixes.  It will return all of the bmcs from all of the environments.


активная страница - не ссылка


Ok


The default behaviour is to print this. Are we changing what is printed?


Everything else looks quite nice! Great work


This isn't going to work once we have multiple networks attached to the baremetal instances, which will be needed for network isolation testing at the very least.


On 28.01.2016 16:34, Sebastian Marr wrote:
> Even if apk cache is not enabled?
>
> > On 28 Jan 2016, at 16:30, Andreas Linz <notifications@github.com> wrote:
> >
> > You may want to do this to save some additional space after
> everything is installed: && rm -rf /var/cache/apk/*
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sebastianmarr/docker-ruby/commit/c7a717bcaea4e84a1eadfc14b69d8fe3720de879#commitcomment-15736077>.
>
Maybe not, but I couldn't find the line where you've deactivated the cache.

--
mit freundlichen Grüßen/with best regards,
Andreas Linz

www.klingt.net
alinz@klingt.net
PGP Fingerprint: F1E8 B1CC E0B7 FBF0 8E7E  33A2 A57E A370 7387 1146
Jabber: alinz@swissjabber.org


Too many changes in one commit, please make changes separate to each other, do not combine them, so that other people can understand what are you doing!


with `td-agent-2.2.1` this is the rubygems environment settings
```
#  /opt/td-agent/embedded/bin/fluent-gem env
RubyGems Environment:
  - RUBYGEMS VERSION: 2.2.1
  - RUBY VERSION: 2.1.5 (2014-11-13 patchlevel 273) [x86_64-linux]
  - INSTALLATION DIRECTORY: /opt/td-agent/embedded/lib/ruby/gems/2.1.0
  - RUBY EXECUTABLE: /opt/td-agent/embedded/bin/ruby
  - EXECUTABLE DIRECTORY: /opt/td-agent/embedded/bin
  - SPEC CACHE DIRECTORY: /root/.gem/specs
  - RUBYGEMS PLATFORMS:
    - ruby
    - x86_64-linux
  - GEM PATHS:
     - /opt/td-agent/embedded/lib/ruby/gems/2.1.0
     - /root/.gem/ruby/2.1.0
  - GEM CONFIGURATION:
     - :update_sources => true
     - :verbose => true
     - :backtrace => false
     - :bulk_threshold => 1000
  - REMOTE SOURCES:
     - https://rubygems.org/
  - SHELL PATH:
     - /sbin
     - /bin
     - /usr/sbin
     - /usr/bin
```


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


Надо пересмотреть


section является лишней обёрткой для p. таких дублирующих контейнеров не должно быть


Надо сказать об этом Игорю и Наташе.


Я поправлю.


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


TeamCity Civic Assembly :: Build backend + frontend [Build 35](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=537&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Agreed.  Will do tomorrow.


Disabled by default afaik


+ fix strlower


I didn't add the whitespace hehe...the patch maker did.


OK, thank for the advice.
See:
https://hub.qgis.org/issues/14187


Thank you for your PR Simon!

This is indeed a better approach.

Regards.


I might have been overthinking this scenario. I removed this in a subsequent commit to keep things simple.


Yes, my bad.


going to be tracking these sorts of things here: https://docs.google.com/document/d/10y0Wqk_C6ry9JRHzanbhuhYFQmwf962iu8q2lky8VZ0/edit


A couple of specific comments inline, but overall I'm not sure why this is needed.  The ports should always be named now (it's how single bmc was intended to work), and the new build-nodes-json just isn't going to work with environments deployed before single bmc.  Are you hitting problems with environments deployed with the current templates and the current build-nodes-json?


description??
type-product - логичнее. потому что это название а не описание. Описание на странице детализации.
или же наоборот название что капсом - class='brand' и задать стили ему
лучше второе.


Yes, fixed in next update.


OK


Good point!


it confuses me


Copy paste error...


Nice.


здесь лучше использовать p вместо span. спан используют для стилизации отдельных фраз в тексте, а здесь вполне отдельный блок


Confirmed working 1-28-2016


My change introduced tabs, replacing with spaces for this file.


убери name - это ссылка, а не якорь


I say what I always do in those situations: if there is no active maintainer for this addon, let it die. Do not pretend that it is maintained if this is not the case. Announce in the forum that it will be dropped from our repos, if there is no maintainer.
If you fix it the wrong way, user don't have much fun with this addon and situation will get worse more and more.


не думаю, что это h1 или вообще любой h


Yea, that make sense. I should always disconnect even if it's not playing. Pushed a new version. Hope it works now


Tabs->spaces here as well.


It looks like it does so I think it's fine.

The only problem is it generates strange link in license field in package preview page: `http://opensource.org/licenses/For%20licensing,%20see%20LICENSE.md%20or%20http://ckeditor.com/license.` (but same happens with `http://opensource.org/licenses/SEE%20LICENSE%20IN%20LICENSE.md` so it doesn't matter in this case).


TeamCity Civic Assembly :: Build all [Build 32](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=535&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 103 (22 new), passed: 1634, ignored: 7 Build time: 00:19:17
### Failed tests
```
Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blog categories tests: Should delete blog category new blog category 1453995552557: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

Blogs tests: Should delete blog: <no details avaliable>

Document type tests: Should not create new document type if cancelled: <no details avaliable>

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


здесь элемент активный и поэтому ссылкой не является


That's strange, when I tried it myself it installed 0.8.6, although I used a different method (I installed google-fluentd then uninstalled the two gems and reinstalled fluent-plugin-google-cloud).  According to the dependency it shouldn't have installed 0.9, and rubygems has the right thing, so I didn't push a bad gem or anything like that: https://rubygems.org/gems/fluent-plugin-google-cloud

I'll take another look, although I'm busy all day today.  A workaround might be to install google-api-client 0.8.6 first and then install fluent-plugin-google-cloud using --conservative.

You could also install google-fluentd instead; it's really just a renamed td-agent with the cloud logging plugin bundled: https://cloud.google.com/logging/docs/agent/installation


Tab->spaces, fixed.


поправлю


Hi Matt,

Did you test these modifications? I don't have time right now to look deeper.

Regards.


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


и тут. и на индексовой странице наверняка потребуется прописать ещё стилей, но это можно позже. И ещё не хватает всплывающего окна, который скрыт в psd. с ним тогда будем разбираться, когда разберёмся с psd


http://elections.huffingtonpost.com/pollster/api

http://ajax.googleapis.com/ajax/libs/jquery/1/jquery.js

http://d3js.org


Done for all instances in this file.


В хэдере пожалуйста выровняй ul по отступам одинаково с a, в следующий коммит пойдёт, а это я уже приму тогда.


Tabs->spaces, fixed.


@gkalpak @lgalfaso - I can confirm this is occurring in my application - after updating to 1.5.0-rc.2, there are a few instances where my code is throwing `TypeError: 'undefined' is not an object (evaluating 'inputExpressions.length')`. rc.1 and 1.4.x are fine.

I haven't tracked it down fully but it seems to happen when we `$compile()` an element with `ng-class` on it. I can try to put together a test case to verify, but wanted to let you know sooner rather than later.


активный элемент не будет ссылкой


лого - ссылка


Why not add utility to nyans ? nyan_cat rainbow instead of mese blocks ? or diamond blocks ? Mese really is overused, IMO.


갓님 그저 경배밖에 드릴 것이 없습니다


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


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


лого всегда ссылка


Just diff -wrt eclipse/cauldron/src/main/java between git checkout prev and git checkout now


лишняя обёртка


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


Since 0.14 supports survival-like inventory, I think it will be suitable to save "survival-only" inventories inside player.dat files


логотип - ссылка на главную страницу (лучше с классом так как ссылок может быть много)


My point here is, if `_GNU_SOURCE` is not defined this test will always fail even on linux.

But i've check that `-D_GNU_SOURCE=1` is in the common cflags, so this is will not happen.
Sorry for the noise.


вот тут как-раз можно использовать .btn-light


I can understand your reasoning but if we go that route I predict several shit storms coming our way...


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


Does is still have any terrible terrible words in it?


How well did you test this?  Seems potentially very buggy.

I think we should do this with os.path.expandvars()  See if that solves most of the problems.


@lokimeyburg Please review.


Actually the sensor id might not be the most correct one for Angler. And why did you choose sensor id 18 for bullhead? According to the dts file on both 8992 and 8994 sensor_id is 7, then you have individual sensors for each core but I don't see anything regarding "HOTPLUG_SENSOR_ID 18". I'm either looking in the wrong place or just not plain following your reasoning.


все производители - ссылки


Fixed ;)


just checked, nope still only the first click-flow works. (deleted cache in safari to make sure to get the recent patch) but at least it works, so there's a light on the end of the tunnel ;).


The scan failed.
Included commits

Commit ID	Owner

84652d046be449561880fdd008307d81cc08a6d7	@BearSpace


+1 for closing this "generic 4.2.0 API" PRs.

@hudokkow I formerly also did once a series of "generic" API PRs, but for those I really did know that the "generic" part will actually work (and not cause bad side effects).


If `allExcluded` is a boolean, no need for the `assign_by_ref`.


положи пока картинку в блок с картой


Thanks Charles,

I'm assuming you've seen the demo but if not, please take a look here :
http://dc25.github.io/dragonCurveElm/ or embedded in this blog post :
http://dc25.github.io/myBlog/2016/01/28/an-elm-dragon-curve-program-for-rosettacode.org.html

- Dave

On Thu, Jan 28, 2016 at 7:30 AM, Charles Cooper <notifications@github.com>
wrote:

> nice!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dc25/dragonCurveElm/commit/ad88b02f482d86b158cf547f5b898e325bf22ce9#commitcomment-15735952>
> .
>


This is about three months late :-( -- automatic updates weren't enabled when this was installed, whoops. Subsequently, we'll use the EveryPoliticianBot's access credentials, so these webhook-triggered bot-build automatic updates don't get logged as being done by me, heh.


лого - ссылка


Глаз-алмаз! Спасибо )


Would it work in Windows?


логотип - ссылка на главную страницу. (лучше добавить отдельный класс, чтобы обращаться без каскада)


Let me update the status: all `job.properties` were removed and added to `.gitignore` after committing #93.


<img width="641" alt="screen shot 2016-01-28 at 9 51 41 am" src="https://cloud.githubusercontent.com/assets/16668795/12649578/ca74f5ce-c5a4-11e5-9661-64e5a9e17004.png">


@JustAkan 흥미로운 쓰레드를 발견했습니다
와이파이문제가 모두어게 나타나고있고 6.0.1 업데이트 이후로 그런다고 다들 말씀하시네요.

https://productforums.google.com/forum/m/#!topic/nexus/gkPdghmnj_o

이로써 저희 커널문제는 아니고 6.0.0 으로 되돌리며 테스트 해봐야 할 것 같습니다


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


shit storm? I will happily point people to the names of the no-longer-active maintainers and ask for their active work on improving the situation. ;-)


task 4 did work for me. Not sure what happened with the other review.


dakujem, potom si to pozmenim :)

On Wed, Jan 27, 2016 at 10:51 PM, Tomas Jasek <notifications@github.com>
wrote:

> Keby si to chcel skraslit monospace fontom, v github flavored markdown sa
> da command odlisit:
>
> `nano /var/www/openbiblio/database_constants.php`
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lamario/EAL_OpenBiblio_Setup/commit/40da08394ff84b3cb33b5c65f2c7e6471b46650d#commitcomment-15718831>
> .
>

--
riso


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


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


Hmmm,

Can you list the steps? I tried both this

1.  Pick visualization with music
2.  Wait for music to start
3.  Past in new music url

and Also

1.  Pick visualization with music
2.  Pause
3.  Past in new music url


targetSdkVersion should be 23 for Android M


Ah, good call.


Oh I see you're clicking play. I'll try that


Agree what hardware PWM is better. But why you have removed analogRead???


@seanupton - with current zope.interface, this test fails. see: https://github.com/zopefoundation/zope.interface/issues/31
I've created a test fix: https://github.com/plone/plone.autoform/commit/ff903900c3506969f735bc30cda5566c4904637f
If the interface comes from a different module, the comparison works again.
That doesn't solve your usecase, though - AFAIK. Dynamically generated classes will have the same module, I guess.
Can you explain a bit deeper, when this usecase is happening?


product-name?
это артикул(vendor-code), а не название.


@shkodm Please update with the upstream - as there are changes to cumulant.

I deleted the c000 because of this: https://travis-ci.org/CFD-GO/TCLB/jobs/105429942


This is how gofmt treats comments on case statements.  And this is the correct position - it's indicating this is an empty switch block.


Done.


Added strip-units, otherwise: https://dl.dropboxusercontent.com/u/7422112/screenshots/Screen%20Shot%202016-01-28%20at%2016.53.40.png


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


Funciona, muchas gracias.


yep, it's very pretty :)


s/is/if/


super @ssibin26 !


Is this per-chance meant to be super.resetItemState?
I'm getting a StackOverflowException on this line when trying to create a new Workspace in DlangIDE.


If you ship this in a Red Hat product and call it Docker you will be violating the trademark guidelines.


TeamCity Civic Assembly :: Build backend + frontend [Build 35](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=537&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 37 (5 new), passed: 1700, ignored: 7 Build time: 00:20:41
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1453996478668: <no details avaliable>

User profile tests: Should change user profile info: <no details avaliable>

User profile tests: Should cancel upload new user avatar: <no details avaliable>

User profile tests: Should upload new user avatar: <no details avaliable>

User profile tests: Should not change user password: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

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


After some time we lose the ability to click the welcome page flag. This error is reported in the javascript console.

![image](https://cloud.githubusercontent.com/assets/12127/12649733/b3a29dd4-c594-11e5-8956-64819483b822.png)

This is a sure sign that we are not keeping the lineup model in proper sync with the contents of the dom.


Yes, we spoke internally about carousels in general and decided we didn't want to include OMP with one. Adding one is still definitely possible, and we hope to ship OJS 3.0 (and perhaps even OMP 1.2) with some examples of using theming to extend the base template set.


@skevy We have another recently added transform that needs to run *before* this one, because it modifies code within the template expansions within `Relay.QL` queries:

```javascript
foo: () => Relay.QL`
  fragment on Foo {
    ${SometimesWeTransformThis}
  }
`
```

We'll sort out that issue and reapply this, the code itself is :+1:


May be worth considering a random port here, but issue may be accept X failures. I am concerned this may affect other tests or in fact fail if other process have used or use these sockets. The socket will / should stay for LINGER_TIME. Can you think of a way to do this? My best thought is random port try up to 10 times.


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


Unknown name attribute


Hey Theo. I found something in your quiz that needs to be modified, and it's just something I missed the first time around. Not only do you want to accept information from the button click, you also want to be able to accept it when a user presses the enter key from either input field. Focus on your group project now, and make a PR when you get a chance to write the new code.


Thanks @pra85


sowas nennt sich guard clause. ganz wat feines

http://www.thechrisoshow.com/2009/02/16/using-guard-clauses-in-your-ruby-code/

für die interessierten :smile:


This seems to be Unix specific (line endings)


Good riddance!


WHY DOES GITHUB HATE MY FORMATTING


Should this be doing what the comment says? or maybe we decided in previous convos that we don't need this method anymore?


Good idea Obani ! and it limit the amount of monster eggs you can craft ;P


I don't like pulling in Boost.MultiIndex just for the purpose of defining `uintptr_type`.


ага, price, то о чем я говорил в прошлом коммите.


Yes


1- please format
2- why an activity with transparent layout to check permissions ? Re-use the RI activity to do such.


@bgerstle reviewing the code on XCode now, I see the retain and releases are necessary because the previous code is making use of [CFBridgingRetain](https://developer.apple.com/library/mac/documentation/CoreFoundation/Conceptual/CFDesignConcepts/Articles/tollFreeBridgedTypes.html)
 which transfers ownership to the caller.
But I think it's not necessary here to transfer ownership so I will remove CFBridgingRetain and replace with __bridge, to perform the cast but without the increment in the reference count.

![screen shot 2016-01-28 at 4 59 03 pm](https://cloud.githubusercontent.com/assets/815372/12649832/82179e80-c5e0-11e5-8917-18d35f20422c.png)

So:
1. I will only use __bridge and let me know if it's ok. OR:
2. If taking ownership is required (via CFBridgingRetain), for some reason I can not see right now, can you explain me why? And I will take back my latest change.


Is there a reason you chose not to add og: decription?


This should be addressed between management of the two companies.  Not something we control.


Whoops... I committed directly instead of a PR. GitHub changed the default option I guess


Oh, didn't realize it that the stream starts automatically after pasting in a new url.

what causes the error is:
1. Pick visualization with music
2. Wait for music to start
3. Past in new music url
4. Press Pause
5. Press Play (before it starts automatically)
6. Error Message


Line 109 below should have the url be https://search.mapzen.com!


Keep the google API (we use google map in RI)


Hey Jeremy. I found something in your quiz that needs to be modified, and it's just something I missed the first time around. When you add the event listener for the enter key in your code it is actually added to the window. The better practice would be to attach it to the inputs instead. Just make another PR when you get a chance to write the new code, and I'll look over it.


Yes, that's much better -- many thanks.


Hi Feng

After run on machine is work fine but after few hours of work machine stop
transit traffic

after down and up imq traffic start again.
not see any error

m.

On Thu, Jan 28, 2016 at 9:58 AM, Martin Zaharinov <micron10@gmail.com>
wrote:

> Hi Feng
> Thanks now build and run fine.
>
> m.
>
>
> On Thu, Jan 28, 2016 at 2:08 AM, Gao Feng <notifications@github.com>
> wrote:
>
>> Hi m,
>> you could try the patch
>> https://github.com/gfreewind/linuximq-HighPerf/blob/master/kernel/v3.x/linux-3.2-imq.diff
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/gfreewind/linuximq-HighPerf/commit/7ff578c40cebd1d640472a38c2e31133e785e830#commitcomment-15721762>
>> .
>>
>
>


:star:


Nice work on the README and cleaning up outdated cruft! :+1:


awesome, thanks


@flavour I didn't understand why we would need two rows limitby = (0, 2)?
If only 1 found, then that'll be the default (non-location specific) so use that -> this means when location_id is not None?
If 2 found then reduce to the 1 with a location_id (which is a location-specific override) & then use that -> this being location_id None? If location_id is None, how we can do location-specific override?


To be removed. No need to have a dedicated activity to control permissions.


Fixed #275


I'm not sure on that because Qt transfers file in text mode from native to unix back.


See previous comment.


Fixed


It was because I didn't think there was a page description attribute—but now that you've made me think to check, I see that there is! I should add that.


@barbaramartina `__bridge` is since the key is a static variable which doesn't need to be retained


Report this into RI activity.


Это заголовок диалога. Добавлю комментарий.


sure, we could.

@ceolin can you change to LONG_MAX and commit please?


Hey Dylan. I found something in your quiz that needs to be modified, and it's just something I missed the first time around. Not only do you want to accept information from the button click, you also want to be able to accept it when a user presses the enter key from either input field. Focus on your group project now, and make a PR when you get a chance to write the new code.


sure


Right.

```
Checking for socket (optional)... found.
Checking for pipe2 (optional)... found.
Checking for accept4 (optional)... found.
```


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


base64mem is not changed. Only the return type was is modified to return the length of the generated string.


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


Cool. Same goes for og: image.


For concision; I feel the two nested invocations read better this way. If you feel it's a net negative, I'm happy to expand it in both places.


formatting


Could you please share it to develop it further? I will use it in a project and develop it.


p нужно сделать не заглавной


Hmm, this doesn't really replace the previous functionality where `suppressRendering` was supposed to ignore a header. Now you cannot ignore it anymore but must remove it before sending it to the renderer in the first place if I understand this correctly.


здесь br лишний


giving a little background info about me


I believe "pathReq"' should be "path"


Good catch.


Ok, i can extend the test.


Useless method since all is managed within RI activity.


Looks fine, thanks :+1:


コミットメッセージみすりましたが消しました。


#40 #9


Useless class. All can be managed by RI activity.


yes


The open question here is around the prospect of expanding the set of content types -- do we want to allow the `:content-type` option passed in the request to reference more than "application/json"?


форма должна быть одна. внутри неё могут быть div, которые и группируют товары. а можно использовать тег fieldset - http://htmlbook.ru/html/fieldset и legend - http://htmlbook.ru/html/legend


Yes, in some downstream deployment, the port with the name is detached and the one attached to the instance has no name. In this case the nodes.json file is wrong.


Perform permission checking here. No need to start an activity.


А зачем отдельный метод?


@thatoddmailbox Can you poke?


This class does nothing. You may safely remove it.


This is cool!


А, вижу рекурсию)


кнопку нужно положить внутрь формы, чтобы она отвечала за её отработку


I just updated the fix by removing hard coded bmc and baremetal values.


Может, переименовать параметр в root или children?


картинка должна в коде идти раньше подписи


@dstufft I believe this line is still necessary, without it I get the following error from `warehouse_static_1` because `node_modules` is empty:
```
static_1        | [19:24:54] Local gulp not found in /app
static_1        | [19:24:54] Try running: npm install gulp
warehouse_static_1 exited with code 1
```
See http://stackoverflow.com/a/32785014 for more explanation.


Саша, не везде это принято. Я перепроверю и обновлю как скажет гугл.


targetSdkVersion should be aligned with build.gradle


блок пагинации лучше оформлять списком


:shipit:


I wonder why the etag is sometimes not returned? strange!


I was modeling it on something like https://github.com/peti/autoconf-archive/blob/master/m4/ax_normalize_path.m4#L100 but that is probably fairly meaningless as well...


_mem is not passed by reference, so this line would only be good style, but it won't be given back to the caller. Is that what you wanted to do?


Sounds good. Thanks Rob!


It's good finish.


This results in the play store badge being gigantic


Trivial, but good practice to wrap conditionals in braces { }


Yep, it is really strange, maybe some connection failures.


Tested this on my local instance and all fine. Will merge.


Is this usefull ? It used to work without this in previous release.


kRootItemIndex?


Looks great! Looking forward to using it.


Ага, у нас уже есть старые проверенные строчки email, эти тогда удалю.


Why did you remove it here?


описание с в 2 колонки и заголовок - это отдельный блок.
смотри если ты разделяешь блоки float'ами - оборачивай эти элементы в блок отдельный, иначе могут быть проблемы с последующими элементами в потоке.


This seems good @bbcrddave.  So when you go to commit to git it will run this first and kick it back?  Should we also add jshint to this?  I think we should get this in so if you are OK can you submit PR?


##### Sorry in japanese

ここで Force unwrapしていることも気になりますが、
Spotlightを継承するViewControllerが全画面ではない(厳密には左上から開始のViewを持たない)場合、このtoCoordinateSpaceの指定ではサポートできないです。
対応するならSpotlight初期化時にSpotlightを継承するViewControllerのViewインスタンスを渡す必要があります。


is it useful ? purpose ?


Hmm, wonder why it works for me. Okay! we can add it back, want to submit a PR? :]


I was thinking along those lines as well. I'll put that in.


I suppose that's where he stores the elements, I'm just refactoring at this point, and I've noticed a lot of things that need fixing, this could be one of them, but I first want to make sure all the fragments are working as before with the abstract parent.

Then there's this adapter, which I know will probably bring even more methods that do counting in a weird way.

I'll ask for a revision as soon as I'm finished, but I've been super limited with computer time here.


I missed the second invocation in the exception handler. It's OK to leave the partial.


ah good point. suggestion withdrawn :)

On Thu, Jan 28, 2016 at 10:51 AM, John-David Dalton <
notifications@github.com> wrote:

> Would it work in Windows?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lodash/lodash/commit/f87af6c642cae9e63123d0c4d5c119f04f7e882b#commitcomment-15736757>
> .
>


Хм. Из клиента сейчас всегда приходят опции MapOnly.


These pass by referenecs should be added to some internal style-guide :D. I began using it, in order to hopefully save some stack operations for the passing of the arguments, but that is only beneficial for often called performance critical functions of especially long arguments. Well string is longer than integer.
Anyway, now thinking about threading I actually tried to stop using these, to avoid situations where I pass some value by reference to a forked function and then change that value in parallel. I don't know how that case would behave.


I just noticed that :)

I also saw you accomplished it as well. You were faster with the html than me.


test


lol maybe we should ignore this file


I find the two asterisks at the bottom more symmetrical :S but I guess even they only use 1 https://www.stack.nl/~dimitri/doxygen/manual/docblocks.html
Doesn't matter really


this too lol


Ah, that clears things up. Cheers.


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


To be removed


A) This would already require users had done something wrong (add duplicates).
B) No known providers are broken, we have more important things to work on.


Good! Use your feelings, boy. Let the power flow through you!


Agreed, thanks: 32da46d


case when придумали специально для этого, если не вдаваться в подробности реализации в целом.


Couldn't this
    ((!filter.fileOnly && di->isDir()) || (!filter.dirOnly && !di->isDir())
be replaced with
    ((!filter.fileOnly))


I see (for the first time actually :-). Sorry for the fuss.


проверь все в валидаторе http://validator.w3.org/nu/#textarea


она используется где то ?


:+1:  this is a fair assumption


Why having added this style ? Why no using Theme.UserDialog which has transparent background also ?


As long as it fixes something, should be fine :-)


Enforce relationship between data and meta with foreign key.


Does this line need to be executed if foundMatchNew == true?


ugh I forgot to implement it :( Will do that now


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.663.79af072d440d57331ebba4d490ae933333c1fe98](http://build.palaso.org/viewLog.html?buildId=68606&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3426, ignored: 125 Build time: 00:08:02


где то используется? Java 8 не хватает?


The more `bash`-y `[[` can be used here.


naming is not consistant.


можно сделать короче (без обертки) Img.index-logo


This needs clarification as I don't understand it either. I'll work with @michaelstingl to clarify.


Seriously???


TeamCity Civic Assembly :: Build backend + frontend [Build 36](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=540&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Very good work, very good test coverage. Well done :+1:


Wrong package


Уверен что эта зависимость нужна в главном pom-е?


Added sleep function to make more realistic


@hjc1710 @garethgreenaway @truckingsim

this line was causing my local server to not work.
```
vagrant@mc3lite-dev:~$ sudo service nginx restart
Restarting nginx: nginx: [emerg] "worker_processes" directive invalid number in /etc/nginx/nginx.conf:2
nginx: configuration file /etc/nginx/nginx.conf test failed
```

changing it to 4 makes things work.


Bookmark for Font-Family: "initial"


`"${COMPREPLY[@]}"` or just use `+=`: `COMPREPLY+=($(compgen ....) $OPTIONS)`.
You might want to set `IFS` to `$'\n'` temporarily so it won't split on other things from compgen…


все заголовки - одного стиля оформления - для удобства можно сделать один класс title, который будет задавать оформление текста, главное без маргинов и паддингов. Только текст.


Thank you for cleaning up the project files / .gitignore!


Логика во вьюхах не нужна, выноси в хелпер.


Кстати, да, спасибо. Забыл исправить


We need some explanations here. It was working well before. Why do we need a stupid timeout now?


a good way to enforce this w/ the compiler is to make the base "abstract" class _not_ declare conformance to the protocol, but to declare in the header the methods in the protocol it implements.  that way, when "concrete" subclasses declare their conformance, they get some helpful compiler errors that tell them which methods they have to implement—excluding those already declared in the base class header


formatting


название новости - ссылка на детализацию


This comment is no longer correct.


Would there be any issue here if this processor was schedule with more than one concurrent task? Meaning, if onTrigger got called by two threads at the same time, and one of them overwrote the bindings of the previous one? If so we could use the @TriggerSerially annotation on the processor to prevent this.


@hjc1710 @garethgreenaway we chose 4 because that is what it is on a cached box that works.  Its only new boxes (or in this case the minion id got messed up on pak's box), and it had to grab a fresh copy that this problem is happening.


Эти три зависимости использует sdk yandex`а , через okhttp я делаю запрос на oAuth2 авторизацию, Gson использую для парсинга ответа от яндекса


is `-init` supported? if not, can you mark as `NS_UNAVAILABLE`?  by the same token, is this the designated initializer? if so, it should be marked w/ `NS_DESIGNATED_INITIALIZER`


I am not sure about this one, but should SecAccess_Read be SecAccess_Access instead?


Lol
I like things in pull requests - in this case it would have allowed the two fixes to become one merge commit... But that's history now ;-)


first


Actually, after reading about the __restrict__ keyword these pass by references could actually slow down things, because less optimizations could appear :). Adding __restrict__ to the pass by reference would make further optimizations possible again.


Thanks, will fix


if the default number of placeholders is 3, do these also have defaults?


Not sure what is going on here, but please make this a separate patch. Not related to this PR.


I think you might want to quote the whole `"${TODO...}"` thing.


Тут как раз нормально. @yunikkk @igrechugin это скорее всего строчка в меню настроек, которая показывает диалог логина или профайл.


Does this method need to be wrapped in try/catch?


Can you explain what you are trying to do here?


This fixes all reqire import issues, and lintting is working on component and app builds.


Hello,
Are you sure for this comment ?


The icon for the keyboard button is not very suggestive of what the button does.


If one of the above exceptions is thrown, won't this flag them as as ECLWatch internal error?


Unnecessary quotes for `[[`. Quoting everything is an easier-to-follow guideline than quote-as-needed though.


private method. Suggest different naming like "notifyPermissionsResultThenQuit"


Lol Kevin c un pd


Are you a windows user?


А какая разница? Пускай так останется.


why????


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.673.79af072d440d57331ebba4d490ae933333c1fe98](http://build.palaso.org/viewLog.html?buildId=68613&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3406, ignored: 106 Build time: 00:08:35


@wangkx Review complete


OK. We still thing it might be useful to have it for diagnostics, but let's leave it out for now.


Not clear to me what happens when I return `NO` here, does it show placeholders instead?


Fix this newline issue


There was one fix in _parse_server_info_payload that was included by accident in this commit. I'll split it. All other fixes were necessary because of the changes needed to call the cb on errors in _server_info_reply_cb and _platform_info_reply_cb.

I didn't get your question about calling the user cb. Where should I call it?


Dies ist ein Kommentar.


Точно )


2010-2016 Orange.


There are lots and lots of unrelated changes in this patch. Can you please separate them out?


cvnv


@dragotin please review.


is there a specific reason why we need to mutate the array instead of initializing it with the given items? if not, IMO this would be cleaner if `init` called `initWithItems:` w/ an empty array


This one here seems to be caused by an error, so EINVAL looks right to me.


easy way to do this is: `[self doesNotRecognizeSelector:_cmd]`


I see it as a behavior/function instead of an object. Anyone else have thoughts?


In terms of the user experience, why do we need the button at all? Why isn't it always enabled?


Do you want me to change this one to ENOENT?


You have some unnecessary complication going on here. You don't need to add an event listener inside the function triggered by an event listener. Once the "keyup" listener is attached, you can call `getInfo` directly.


название класса меня смущает.


Ну, я бы не был так категоричен, учитывая что перевод всё же был с английской фразы, которая не заканчивается восклицательным знаком)


but, that might not be necessary if you [enforce protocol conformance with the compiler](https://github.com/wikimedia/wikipedia-ios/commit/648efc6f8028af32592188abce3edee398153169#commitcomment-15737982)


TeamCity Waterfall :: Build [Build 44](https://tc.demonwav.com/viewLog.html?buildId=233&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 46 Build time: 00:01:12


Just checked and there is no way to split the commits. Actually all changes in parse methods were made because free is not being called on errors anymore. And It makes more sense to me too free variables on erros in the function that produces it.


Just FYI, in PeachPy you can use virtual registers for temporaries. PeachPy would automatically map them on hardware registers during code-generation (or generate an error, if you use too many). So, the macro could look like:

```python
def RotW7(x):
    tmp = XMMRegister()
    MOVDQA(tmp, x)
    PSLLD(tmp, 7)
    PSRLD(x, 25)
    PXOR(x, tmp)
```


hello @netali
i can see you are done with the commit.
Now do
**git push origin your-branch-name**
provide your username and password
come back to this website and compare and pull request


EnderPearl is comfirmed by tomcc that it won't be added in 0.14


этот div можно убрать


Will try.


Add a comment here to explain the purpose of this private constructor.

"A private constructor is defined here to prevent external instance initializing".


про класс для заголовком писал уже. как раз этот случай


@addyosmani: feel free to cut a new 0.5.0 release.


Названия неинформативные - Tdeck, Pdeck


15.04 support ends on Feb 4th. https://wiki.ubuntu.com/Releases

Thanks Kay.


Meaning of this comment ? Add a javadoc comment on the class to explain the purpose.


.shop-content


awesome! thanks for adding @hrecht


Complete javadoc


52 - magic number - надо вынести в константу
13 тоже хорошо бы, типа VALUES_IN_SUIT


лишний div


Please use `git revert <commit>` to undo a commit, instead of create a new commit with some useless messages. Thanks.


Лишние пробелы


Removed correct code and replaced with name which will not acceptable under naming conventions we discussed before

Please read carefully, This is the most commonly used open standards
https://isobar-idev.github.io/code-standards/

Better to go with above standards, because I also follow


Thanks for the fix!


See previous comment.


This is one of the very very rare things where I am actually more in tune with the guide line than you :)
https://github.com/ComputationalRadiationPhysics/contributing/blob/master/codingGuideLines/cpp.md
> & (reference) and * (pointer) must be surrounded by one space


Элемент aside.
Внутри него все - одна form.


canola oil isn't spelled that way!


:tada:


You are calling `tree()` whether or not `check()` passes or not. Consider moving the `tree` call to inside of `check`, so it won't be called unless the form has been filled properly


same as line 120


Это deck, да


почему у тебя лого лежит вне шапки? и логотип всегда ссылкой должен быть


Wrong package


What's the implication of this change? I don't really understand that part of the simulator.


А это на самом деле возможные номиналы, скорее SuitValues - что то такое


If we have have some locations with location-specific & some without.
The ones without will use the default (location_id == None)
To avoid 2 queries it is more efficient to look for both in 1 query:
... & ((location_id == None) | (location_id == location_id))
If only one is found, we have to assume this is the default...and can use direct
If 2 are found, then we wish to ignore the default (i.e. reduce to remove the one with location_id == None) & just use the one with a location_id

Here is an example of the pattern:
https://github.com/flavour/eden/blob/master/modules/s3/s3gis.py#L1561


Они еще в работе.


Последняя отправка


логотип всегда ссылка


Needs to be changed back!


ок


Цикл странный) Но молодец, так то сработает