This guy is used in the routes under topics and posts. The idea behind this guy was to identify which comments were from topic and which came from post.
 So basically when creating a comment on the Topic#show page the route will go to this controller first set set_commentable to ```@commentable = Topic.find(params[topic_id])``` than go to the generic comments controller for the create action.


das ist doch eine methode??
das einzige was groß geschrieben wird ist eine Konstante (alles in großbuchstaben und underscore anstelle des camelcase) und Klassen sowie Konstruktoren
alles andere beginnt klein


TeamCity SmokSmog :: Pull requests [Build 217](http://home.antyzero.com:8111/viewLog.html?buildId=786&buildTypeId=SmokSmog_PullRequests&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2 (1 new), passed: 18 Build time: 00:07:12
### Failed tests
```
pl.malopolska.smoksmog.debug.test: com.antyzero.smoksmog.screen.StartActivityTest.addStation: <no details avaliable>

pl.malopolska.smoksmog.debug.test: com.antyzero.smoksmog.screen.HistoryActivityTest.checkCreation: <no details avaliable>

```


Makes sense; but with the release scheduled for tomorrow morning, I'd skip this suggestion due to the real cost/benefit of the whole update.


Naja, dann weiß ich mal kurz, warum das so und so ist, aber kurze Zeit später hab ich das wieder vergessen und stolpere über die selbe Zeile Code…

Extra-Funktionen für solche Bedingungen zu schreiben, mag zwar einen Kommentar sparen, allerdings musst du dann dafür eine neue Funktion schreiben :/

Abgesehen davon verursachen Funktionen ständig neue Unterprogrammaufrufe, was das Programm extrem verlangsamen kann.

Ich bin daher kein Fan von dieser Vorgehensweise.

Von: Stefan Walesch [mailto:notifications@github.com]
Gesendet: Donnerstag, 31. März 2016 19:51
An: swalesch/FiniteFields <FiniteFields@noreply.github.com>
Cc: Bluesky787 <bluesky787@nurfuerspam.de>
Betreff: Re: [swalesch/FiniteFields] Grundlegende Behandlung von Spinner-Ereignissen eingefuegt. (e1fcc87)

…

ich behaupte nicht das ich das überall richitg gemacht habe.... dafür ist so ein CR ja da, das du mir sagen kannst, pass auf methode so und so verstehe ich nicht, was machst du da? ..... dann kann ich das verbessern und umschreiben, solange bis es lesbar ist, und das ohne eine zeile kommentar

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub <https://github.com/swalesch/FiniteFields/commit/e1fcc87d560a8e898dad7fcc5d9fcb81754d402c#commitcomment-16919257>


TeamCity SmokSmog :: Pull requests [Build 218](http://home.antyzero.com:8111/viewLog.html?buildId=787&buildTypeId=SmokSmog_PullRequests&guest=1) is now running


This commit was incorrectly named. It is actually an update to version 1.0.3.


congrats


no it isn't


Same intention as the other one. The idea was just to set ```@commentable``` to ```Post.find(params[post_id])```
so that the generic controller page knows what commentable is.


ja, die sind an einer zeile angehängt
du musst auf commits klicken und da auf deinen commit, dann sollten die kommentare an der richtigen zeile sein


Updated. Thanks


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


POI-исключения: которые не принадлежат к общим классам (amenity, shop). Раньше компонента была в cpp, но после анализа кода, выяснилось что такая же штука нужна в другом месте. Вынесена в публичный интерфейс. Допишу коммент, что тут по чем.


oh shucks. I didn't notice that. I can explain in the meeting what I wanted to do here. I have to show you on the website.


ooooh du beantwortest das alles als emails ..... das ergibt natürlich sinn :D
du solltest auf die webseite gehen, hier zu deinem commit: https://github.com/swalesch/FiniteFields/commit/e1fcc87d560a8e898dad7fcc5d9fcb81754d402c


Indeed, you are correct, perhaps I should as it certainly would be much easier.


มั่ว


Cambio para arreglar búsqueda de servicios


หหห


wir können das sicher mal besser ausgiebig diskutieren wenn wir uns zusammen setzen, über 1 Million verteilte komments wird das so nichts


`which printf` suggests that this is the case. Fixing this and another minor mistake in a second


![Relavent xkcd](https://imgs.xkcd.com/comics/hypotheticals.png)


Habs gefunden und geht klar ;)

Von: Stefan Walesch [mailto:notifications@github.com]
Gesendet: Donnerstag, 31. März 2016 20:00
An: swalesch/FiniteFields <FiniteFields@noreply.github.com>
Cc: Bluesky787 <bluesky787@nurfuerspam.de>
Betreff: Re: [swalesch/FiniteFields] Grundlegende Behandlung von Spinner-Ereignissen eingefuegt. (e1fcc87)

wir können das sicher mal besser ausgiebig diskutieren wenn wir uns zusammen setzen, über 1 Million verteilte komments wird das so nichts

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub <https://github.com/swalesch/FiniteFields/commit/e1fcc87d560a8e898dad7fcc5d9fcb81754d402c#commitcomment-16919382>


But that Zones is used for 3 AI only and all of them got skills. Den of Evil made tons of put/remove, think that create / delete is more bad. Also that maps will cost almost nothing.


@JonathanHDavis you are attempting to find the user with `params[:id]` but you are in the movies _controller, which means the the `:id` will be for the specific movie not the user.

Since you are using devise, you have access to `current_user` in both your controllers and views, and this will represent the current user that is logged in. so you don't need this code delete this `find_user` method


delete this line


yesyesyesyesyesyesyesyesyesyes THANK YOU SO MUCH RYAN YOU ARE THE BEST
(and I'm pretty stupid)


Ik heb het nog eens langs een andere weg gecontroleerd, en ddx5 zou echt wel in orde moeten zijn. Er zou iets moeten schelen bij ddx5_check alleen...


TeamCity SmokSmog :: Pull requests [Build 218](http://home.antyzero.com:8111/viewLog.html?buildId=787&buildTypeId=SmokSmog_PullRequests&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2, passed: 18 Build time: 00:07:09
### Failed tests
```
pl.malopolska.smoksmog.debug.test: com.antyzero.smoksmog.screen.HistoryActivityTest.checkCreation: <no details avaliable>

pl.malopolska.smoksmog.debug.test: com.antyzero.smoksmog.screen.StartActivityTest.addStation: <no details avaliable>

```


```@commentable``` depends on what page you delete or create a comment. If you perform any of those actions in the Topic#show  ```@commentable``` will be set to ```Topic.find(params[:topic_id])```. Oh shucks. I thought you had to make the comments work on the topics as well.

1. "Associate comments with topics as well as posts. This will allow users to comment on a topic as well as a post."
2. "Display comments on the topic show view."
3. "Add create and destroy comment routes under topics."


TeamCity SmokSmog :: Pull requests [Build 219](http://home.antyzero.com:8111/viewLog.html?buildId=788&buildTypeId=SmokSmog_PullRequests&guest=1) is now running


file_desc is not closed


Yes, it is a big commit, and it is also difficult for me to verify that all the changes are intentional. They seem ok, though. We have done some smoke tests with the content team over the changes, and they results were correct.
When it comes to your approach with a preparatory commit, I would love to know more. If you have the time to point me into a PR or commit with an example it would be much appreciated.


file_desc is not closed too


I would try to avoid setting a height at all


The code looks good. However, in the first line of code the print function can be avoided since the input function will print the message anyway.


Because I didn't want to break backward compatibility, don't know if id of a message can be changed after it was instantiated but I will check that and make the change if it is possible.
Thank you!


this doesn't need to be set twice


Yes, this one is easier to follow. I do also feel more confident about it.


Thanks @andrew :tada:

I have published v0.9.8 manually. I will figure out the auto deployment issue later.


This is samples from Bluehawk2 compared to DB. That is why bluehawk2 got zero matches because it doesn't exit in the DB. But we can see that it mostly matches Bluehawk, Great Tit, Bofink. With the right config we get it to match Bluehawk.


Is LTC test required for this patch?


can we create a global font component and store these in extends


@smcquaid If you want to improve Kubernetes on ARM, feel free to create a pull request :smile:


MMB29V was release 17 I believe and MHC19I is r22... And yea we've been on
22 for a bit now.
On Mar 31, 2016 1:52 PM, "Matthias" <notifications@github.com> wrote:

> Scott, with r22 aren't we already on MHC19J?
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/scotthartbti/android_device_huawei_angler/commit/cf6d87267db07239696916f4be6eea4b13b18183#commitcomment-16919283>
>


Redux has one global store that is accessible with the `connect` higher order component (which is part of the [react-redux](https://github.com/reactjs/react-redux) library). You should use the global store and dispatch actions to it, as that is where your reducers are setup. Let's hop on a call tonight and talk about this a little bit. I am free most of the night.


Вижу запрос от браузера. А где запрос от консольного клиента ?


You can use the `connect` HoC to map dispatch to your props, so you can use it within your component.


thx for clearing up.


Np
On Mar 31, 2016 2:09 PM, "Matthias" <notifications@github.com> wrote:

> thx for clearing up.
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/scotthartbti/android_device_huawei_angler/commit/cf6d87267db07239696916f4be6eea4b13b18183#commitcomment-16919525>
>


This is where your error occurs - there is no such thing as `reviews` rails doesn't know what that is...this should be `@reviews`


Исправил опечатку


add a comma after `item` here:   If you want access to both the outer include's `item` and the current task's `item`,


TeamCity SmokSmog :: Pull requests [Build 219](http://home.antyzero.com:8111/viewLog.html?buildId=788&buildTypeId=SmokSmog_PullRequests&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2, passed: 18 Build time: 00:07:20
### Failed tests
```
pl.malopolska.smoksmog.debug.test: com.antyzero.smoksmog.screen.HistoryActivityTest.checkCreation: <no details avaliable>

pl.malopolska.smoksmog.debug.test: com.antyzero.smoksmog.screen.StartActivityTest.addStation: <no details avaliable>

```


you can add another line here such as `<p><%= review.user.email %></p>` to display the user who created the comment


Done. Sorry for not being more thorough


@calasada So, you call this method 6 times in this method. Each time you do, you're requesting data from the network, which is kinda a time intensive task. Could you do what you did in getContours and get the data once (well, twice, for x, and y), store it to a variable, and do your checks on that?


is there a reason that we have to block here for processing instead of do go wc.process(e)?


TeamCity SmokSmog :: Pull requests [Build 220](http://home.antyzero.com:8111/viewLog.html?buildId=789&buildTypeId=SmokSmog_PullRequests&guest=1) is now running


求测试求 review


@nateware please release a new version on RubyGems with this bug fix included.


Do you still want the "_det" in this line? Just checking.


it looks like ghostery has added some more trackers today. 47 i counted


why do we want to do this async? it seems like startWatching has to wait for the signal anyway?


add a function called eventFromKV to construct the event?


System-Fan-Model (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2164 of 2176 tests passed, 544 test warnings)

 * 556 tests had: AUD diffs.
 * 411 tests had: RDD diffs.
 * 14 tests had: BND diffs.
 * 14 tests had: EIO diffs.
 * 6 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 13 tests had: MTD diffs.
 * 6 tests had: MTR big diffs.
 * 12 tests had: Table big diffs.
 * 4 tests had: ESO small diffs.
 * 2 tests had: MDD diffs.
 * 3 tests had: MTR small diffs.

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


Minor feedback: inline comment to add a comma and fix a possessive
Otherwise, docs look sane and sound clear.


new


asd afasd fsdggsdg


@Calasada Should you use implement compareTo in the contour class, or should you anonymously define a comparator when you call this method instead?


I merged accidentally. I thought I was merging with my own master but seem like I merged with yours.


Maybe STC was 40/40 and 40/10 is LTC :D

Regards


@Calasada You can store numerical values with each enum entry - that way you don't have to have this if statement. Try reading through this: https://docs.oracle.com/javase/tutorial/java/javaOO/enum.html


why do we need this struct? i feel we can just declare these vars inside the run loop?


@owenatkins29 you were supposed to add a constructor that took all of the parameters, remember? @Calasada did it now, though.


test


Need to use MAP_FAILED constant


thanks @llopv, it seems to be well merged here ;)


@klq Lin. alg. problems submission


The question is whether we need linearizability on resultChan? The old implementation is and I'm uncertain about the assumption made upon.


teste


@jjones646 is there a reason why these were setup as DigitalInOut?  Shouldn't they just be DigitalOut?


I made it based off of tutorials on the A* method, but I'm not sure what to call my interpretation of those tutorials.


I will write more docs for it.
Basically we will return it.


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


You should /2 each size only once MAP_FAILED is returned for current size.
If kernel allows to allocate current size then you should continue to allocate it.


The event will be sent to for-select loop.
We are uncertain about the number of current events here.


```return array_map([$this, 'buildValue'], $values);``` :)


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


we can do that by putting the item into another dedicated routine to process. it will still be serialized but does not block the watch queue.


probably at least leave a todo here.


_.once is implemented with _.before, which is a bit over-engineered for our needs.  This change knocks off 150 B from the minified/gzipped size.


Oh, right. Sorry for the noise. I didn't realize `brew info caprine` depends on `brew update` (that is some horrible UX btw).


Why do we kill entry on ERR_FSAL_DELAY?


my general feeling is that there are too many fields in this struct. and we probably do not really need most of them. i may be wrong.


Why did you do that...


Hmm, should we change that copyright to Red Hat?


$values is array if argument is Collection
```
if ($values instanceof Collection) {
    $values = $values->toArray();
} elseif(!is_array($values)) {
    $values = func_get_args();
}


20 matches (100 games each)

Your test, the yellow one: +10-7=3
+7 * pop:+11-8=1
+15*pop:+3-17=0
+9*pop:+6-13=1
+10*pop:+8-11=1

Doing some other numbers. Let's see

Regards


Have you considered doing a PR back to the original repository with these changes? Would love to see r11 support over there.


was caused by voidwater actually (alpha=0.5 didn't meet the threshold)


yes, voidwater is usually the cause for invisible ground


Needs unit test.


hmm, maybe should change the names of these functions from cih_ to mdch_ or something...


The deeper answer to non-blocking is queueing.

I will leave a TODO here.


Hope to see something soon here.

BTW, why did you added me to AnimatedImage?


to bronze IV :fappa:


Yes, I'll add some benchmark results.
Thank you.


what was reasoning to get rid of cache partitions?


For any questions Please Commit It


You are right. For some reason it didn't crash, but I'll correct that one. Thank you for pointing out.


Hi @hurtsky, congratulations for all the work you put in this kernel,I use a lot stuff from this for the kernel I build,Im trying to implement lcd notify to adreno_idler, Can you give me some help ?

[https://github.com/attack11/Attacked-Kernel/commit/b9d8c34b9b41445c6f98629e2ddafe6dbfc98c62](url)


Oh, just moved it...


hi @WillNetsky,

Thanks for your python submission!

Before I go in the details, let me just say it's such a *pleasure* :cake: to go through your work and I definitely don't say that very often. I really loved the way you explained things. They are clear, concise, to the point, and with great examples to illustrate the differences. I loved your explanations on lists vs. sets vs. tuples, and map functions and list comprehensions. They are very well thought out and so nicely written too. Bravo!! :clap: :tada: :tada:

Re: python code: You should remove all of the "Raise NotImplmentedError" statements once you've implemented the code. I know you did it on some exercises, but not all. You should remove them all.

Re: Q7 linear merge: Your solution is very neat. However, it's not strictly linear time solution because list.pop(0) in python is not constant time (whereas list.pop(-1) will be). So you are very close, but to achieve true linear time, you need to reverse the lists, and implement with list.pop(-1) to the reversed lists.  Try to fix that to make it run in real linear time. The code will be uglier I know.

Re: advanced_python: did you skip the first section on regular expression? Let me know if you still plan to finish them.

Re: Markov.  Great job for trying out the optional exercises!!! :+1:

Overall, you did a *GREAT* job and it is one of the best I've seen. Thank you for making grading into a much more enjoyable process :smile:

-klq


I don't see any reason to map file. Kernel demand paging won't allocate physical memory on mmap anyway.


If $value instanceof Model buider not need.
```
return $value instanceof Model ?
            $value : $this->getBuilder()->build($this->getEntity(), $this->getAttribute(), $value);
```


`extends JenkinsConfig` because `MavenInstallation` is expecting a `JenkinsConfig`. Just trying to get something working ;)


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


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


Thanks.
We want to see the trend, should we go higher, or lower.
What is your time control.


Typo!

Also what was the reason for adding this comment? If we want it, there are 2 other places to add it in this repo, and then 4 more repos to propagate the change to.


Right on track learning how to do this....


Hi @klq,
This was my last commit. I just fix emails.csv.
If you want I can organizer my commits in part1, part2 etc.
Thank you for reminding me.


The another way of implementation without these fields requires duplicate and long function arguments, spreading to not only run() but all other things. Not good for future changes.

These fields are static across all.


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


Typo fixed in b73f845.


System-Fan-Model (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (2164 of 2176 tests passed, 544 test warnings)

 * 556 tests had: AUD diffs.
 * 411 tests had: RDD diffs.
 * 14 tests had: BND diffs.
 * 14 tests had: EIO diffs.
 * 6 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 13 tests had: MTD diffs.
 * 6 tests had: MTR big diffs.
 * 12 tests had: Table big diffs.
 * 3 tests had: ESO small diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTR small diffs.
 * 1 test had: Table small diffs.

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


I ran it under rspec and it showed no errors. I will got back and modify the code


TeamCity Units.NET :: Master [Build 18](http://teamcity.anj.no/viewLog.html?buildId=507&buildTypeId=UnitsNet_Master) is now running


TeamCity Units.NET :: Master [Build 18](http://teamcity.anj.no/viewLog.html?buildId=507&buildTypeId=UnitsNet_Master) outcome was **SUCCESS**
Summary: Tests passed: 689 Build time: 0:0:23


TeamCity Units.NET :: Master [Build 19](http://teamcity.anj.no/viewLog.html?buildId=508&buildTypeId=UnitsNet_Master) is now running


That's ok. You've found the upper bound of continuous address space region.


I think I might like to see this broken out into more method calls instead of a for loop for the length of the method.


m


LOL


I'd also like to keep this hackish syntax in the FAQ. While there are more elegant solutions, it definitely has appeal to novice users. And PHP was built on ease of embedding innthe first place, why deny such possibilities to easily inject PHP to a template.

However, we should keep a wording to indicate that this is the most unelegant way to deal with custom code. :)

> On 31.03.2016, at 19:46, donchambers <notifications@github.com> wrote:
>
> Then perhaps a strongly worded recommendation against using it.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly or view it on GitHub
>


TeamCity Units.NET :: Master [Build 19](http://teamcity.anj.no/viewLog.html?buildId=508&buildTypeId=UnitsNet_Master) outcome was **SUCCESS**
Summary: Tests passed: 689 Build time: 0:0:21


you can use the try-with-resources semantics for all the pool accesses which ensures we don't leak the jedis if any of the calls fails:
```
try (Jedis resource = jedisPool.getResource()) {
}
```


Yes.  `deepCoadd_det` are the detections.  It's `deepCoadd_calexp_det` that's gone.


BTW -- this line is almost certainly the culprit of https://github.com/brownplt/pyret-lang/issues/680 -- even though it's following the CM documentation, it seems not to be doing enough...


These changes should be propagated to all repos if you think they're worth making.

Also noteworthy that `prince_options` are sometimes applied outside of PDFs.


Do we need any tests for multiple config files behaviors?


Oh god, these non-native speakers... :) Sorry about that...


switch `keys` with `scan`


TaskCluster: @nurav does not have permission to trigger tasks.


Tab indent.


This should be info


You have 2 mistakes:
1. you unmap. how you can occupy the whole virt address space if you unmap what you've allocated.
2. Your implementation doesn't allocate the maximal amount of memory because you allocate each available size only once. Suppose we have 10 non-continuous pages available in address space. You'll allocate only one.


Could use `bullets.len()`


I mean, I made a normal PR, without reading through the instructions for how to update all the repos in lockstep. I did not realize the example code is generated, if it is. I thought it was a reasonable comment since it is presented other options in use have comments.


question: why use `===` over `#`?


`RedisDao - redisDao`


is there a need for this?


`RedisDao -> redisDao`


closes #1


I think it's more readable as plain text (for those that clone a repo, and
open or `cat` the README).

On Thu, Mar 31, 2016 at 2:43 PM, Phil Leggetter <notifications@github.com>
wrote:

> question: why use === over #?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/Nexmo/client-library-specification/commit/a63226e61116e28f2244ec59f740d0eab5ae4337#commitcomment-16920071>
>


I left the extra function. for what I have currently its arguably overkill, but my app may become more complicated as it grows and this approach is better postured for scaling


Could probably check that with a `static_assert` and put the comment in there :).


seems wierd to need this - should we just mark the Action1 as json-ignored / transient instead?


При установке "Ошибка импорта базы". База cms_users создается, а данная запись не добавляется. Все таблицы после cms_users соответсвенно тоже не добавляются.


![Happy April Fools Day Frank](https://cloud.githubusercontent.com/assets/11591518/14187110/723c542e-f74f-11e5-8d71-d6acc012989b.png)


are we using snappy in redis?


Да, да, уже сообщили, скоро коммит сделаю.


is this needed?


Done.


Double-checked. The problem is not that X_categorical is unrecognized, but that
```
E:144, 8: Instance of 'RandomForestClassifier' has no 'fit_transform' member (no-member)
E:145, 8: Instance of 'RandomForestClassifier' has no 'fit_transform' member (no-member)
```

That's because of a choice inside sklearn. It's similar to cases where you do
```
if isinstance(foo, list):
    foo.append(bar)
```
and it complains that foo doesn't have an append method, because it can't reason about the isinstance ensuring that foo must be a list for the second line.


I mess all the other results. Just the yellow and +7*pop are correct. I will try again.

The fastest i can: 1s10ms. this way i can play thousand of games on my laptop


rachel is 30 years old and lives in chicago


That's so awesome!


Question: Should we have a set of examples in this folder or should we create/encourage a separate getting started set of apps/quick starts that have a dependency on the library?


How does this work when originally forked from my data?


Regessão
Materiais de Regressão


Yeah, I started looking into that when I saw it, but I wasn't able to find a fix. I'll look into it in the next few days.


Secret table lookups! Could you replace this with constant-time versions without table lookups? Also, did you consider MurMur3? Could you also provide CRC-8 and CRC-16 versions for Internet of Things?


No worries ;)


it should be called with an array type (http://dlang.org/phobos/std_array.html#.uninitializedArray.2)


Opinion: Configuration is also part of usage. How about we move each Example up to be a `h2` heading?


The following doc(s) have been deployed to the [1440migrat](http://1440migrat-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/manual-import](http://1440migrat-static-docs.pantheonsite.io/docs/manual-import)


the bad tests were with
+                     + value * (popcount<Max15>(b) + !!ei.pinnedPieces[Us])

forgot to separate


Double `API`


I don't think PiecewiseTools was meant to be extended by casual users (which is why InertFunctions is not documented, and not exported).  Which means that you have no choice but to use opaquemodules=false to be able to see it exists at all.  I'm impressed that adding binding forms to piecewise was that easy!


![screen shot 2016-03-31 at 1 51 33 pm](https://cloud.githubusercontent.com/assets/16783283/14187332/1a0d737a-f748-11e5-8866-d44b9c596659.png)


nice new file you added crebo


wait does the live version have this?


For consideration: Using a list of checkboxes is clear in markdown. But a table does look nicer :) See https://github.com/pusher/pusher-http-python#feature-support


master (evanweaver) - x86_64-MacOS-10.10-clang: Build Failed

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


System-Fan-Model (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1620 of 1620 tests passed, 0 test warnings)

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


System-Fan-Model (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


The `DS` is fine on line 42, because it's creating a file, and the directory separator is the correct choice here. But on line 46, the separator should always be `/` which is not what `DS` nor `DIRECTORY_SEPARATOR` is defined as on Windows.

Line 42 can be reverted, as it worked fine, and line 46 should have `DIRECTORY_SEPARATOR` replaced with `'/'`.


Hi @zcarver , for Task 3 you only need to put dates both in mm-dd-yyyy  format, no need of "start_date" or "end_date". Like:
"python Task3.py 03-30-2016 04-07-2016"

Try this !


Two state_hdl for directories?


Before this commit, I think `Pism_BUILD_PDFS` did affect whether `make browser` would work.


how did this ever get to the end of the array then if we didn't consume a token? seems like this would be an infinite recursion?


jesli uzywasz namespace std, to nie potrzebujesz pisac std::vector /std::string


Looks good :), on my local.


One set is checking if the port is open to TCP requests, the other for UDP.... as far as port 80... the common TCP list only had 98 instead of 100, so I just need to change the last two ports to something different... forgot to correct that.

Ideally, Main is an input/output operation only.  So the possible inputs are a range of ports or a list of ports.  The class should be able to take care of either input; if we rewrote Main then it would be doing the processing instead of the CustomClass.  Additionally Main was posted before anyone was expected to start coding their class, so in theory everyone should have used Main as a guide.

But diplomatically, either Main can be adjusted or CustomScan could be.  It ultimately doesn't matter which...   I'll debug it all at the end if no-one wants to adapt their code to anyone else's.


Hey @jgondin,

Thanks for the submission!  Great job! :+1:

I really like how you explained the similarities and differences among lists vs. tuples vs. sets. It's really good that you formatted it nicely and separate the similarities vs. the differences.

re: python 5a Q2: You didn't answer the question about lists vs. sets in terms of lookup performance and the reason. Could you complete that part?

re: python 5a Q5: Your second datetime answer is wrong. The two dates were in year 2013 and 2015, respectively. They are dates in 'string' format not  'timestamps'. Try to fix that one. The other datetime exercises are all good.

re: python 5a Q7: linear_merge: Your solution is very neat. However, it's not strictly linear time solution because list.pop(0) in python is not constant time (whereas list.pop(-1) will be). So you are very close, but to achieve true linear time, you need to reverse the lists, and implement with list.pop(-1) to the reversed lists. Try to fix that to make it run in real linear time. The code will be uglier I know.

re: python 5b: Q4: you need to print out the **unique** email domains, not a list of all of them.

Overall, your work is very well done! :+1:  I especially like how you explicitly write very nice test cases for all your python coding questions. Writing unit-tests is an excellent practice and will be really essential down the line.

-klq


Request: a bit more of an explanation on this one. I don't get it.
Suggestion: remove `tedious`


Your implementation doesn't allocate the maximal amount of memory because you allocate each available size only once. Suppose we have 10 non-continuous pages available in address space. You'll allocate only one.


GAME?!


That should go BEFORE the pf_version INSERT..
Actually, everything should go before the pf_version


This commit breaks the PKGBUILD.
You must test your package locally before push it to github.


Yep. Looks like that was the other unintended consequence of wrapping everything in `doc/CMakeLists.txt` in `if (Pism_BUILD_PDFS)`.


To wyglada troche nieintuicyjnie. Wg mojej koncepcji q mialo konczyc cale wpisywanie macierzy.
Czyli na konsoli wygladaloby to tak:
```
Podaj kolejne wartosci macierzy, linia po linii:
1,2,3 (enter wciska)
4,5,6 (enter wciska)
7,6q (enter)
Zakonczono wpisywanie macierzy. Podana macierz wyglada tak:
1 2 3
4 5 6
7 6 0 (program domysla sie, ze ma byc zero, skoro ostatni wiersz mial 2 kolumny)
```


@rcdailey Is the upstream repository still maintained? It seems like it hasn't had any activity for almost a year. I don't mind making a PR though if you would like me to do so.


Every 2000 games = 21minutes using 3 threads. 4 threads and my laptop start to fly. :D

Regards


This commit should resolve issue #2.


jezeli uzylas using namespace std; w headerze to nie trzeba tutaj juz


I'd like to understand:

1. if this approach adds complexity to a library
2. if there is significant overhead in having to use additional request objects
3. does IDE support and the need to define parameters as object properties provide significant value over looking at a reference when determining the parameters to be used?

i.e. I'd like to see a comparison of the two approaches.


one more not worked in autoupdate case sql 2016_03_14_creature_update.sql


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 90](https://teamcity.labkey.org/viewLog.html?buildId=314304&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 49 Build time: 01:41:06
### Failed tests
```
org.labkey.test.tests.cds.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

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

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

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

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

```


oh yes, good catch :)


I think it can be useful to use the big trait (it's what I do in https://github.com/dunglas/symfony-demo/blob/6985cec1793e957615718b50339c0ec85e8da002/src/AppBundle/Controller/BlogController.php).

Even if you don't use it, I think it has no downside to permit it.


Request: a bit more information on this please. Do we mean something that says "I'm using a key/secret"? If so, isn't the act of using a key/secret enough to say "I want to use key/secret auth"? I'm probably being dumb.


caly ten blok, gdzie informujesz uzytkownika jak kod zadziala powinien byc osobna funkcja, ktora sobie wolasz w addMatrces(). Zalozmy, ze ta funkcja co pomaga uzytkownikowi nazywalaby sie printMatrixInputMethod(), moglabys ja wolac z metody addMatrices(), multiplyMatrices, countWyznacznik().


I don't understand how this works.
You make the second fetch(), ignore it's result, and continue the first fetch.

In the previous version you were returning the return value of the second fetch()

of may be it is Java 8 way?


Question: How is this different from key/secret?


Suggestion: s/official/runtime supplied


I'm not seeing a PR.

Example code is not generated, but it is kept consistent across all clients because they all need essentially the same documentation.


co jak user poda:
1,2,3q,4,5?

W petli, ktora dodaje 2 macierze program sie wywali, bo nie bedzie istnial matrix_one[1][2]


You should leave 1 thread for the OS.
Also, before starting a new batch,
check in your Task manager if you have stockfish processes still running.

Multiple 100 matches are interesting, but the overall win - loss- draw  is more significant.


Would this break compatibility with older versions of RPM? Specifically, RPM version 4.11.3
I've noticed that a daemon-reload gets executed upon initial installation of the package, however uninstalling and reinstalling the package will not issue a daemon-reload.

Is this a bug, or to be expected? Can we resolve this?


Out of curiosity, does this collapse the array if entries are empty or does it do something else? Too pythonic for me.


http://hlslive.lcdn.une.net.co:80/v1/AUTH_HLSLIVE/0003/yd1_1.m3u8

hola ese enlase lo puedo reproducir en simple tv por ejemplo sin problemas ,, porque en rosadin no puedo ?  se podra compatibilizar ? si fuese posible se lo agradeceria mucho


Hi, I am relatively new on Spring. I have just two years of experience. I am reading the code and I have some gap that I will like someone to help me with it. `Configuration<?> configuration` declaration has been used. I can't see anywhere on the page which package this Configuration<?> element belongs to. Maybe I am not using Github well. Someone help.


@nniclausse , about a month or so ago, I was also looking at this pack/1 function... (`for fun?`) :smile:

Do we need so many lines of code?

What do you think of the following implementation... ?

http://lambdafoo.com/blog/2008/02/26/99-erlang-problems-1-15/

https://github.com/processone/tsung/compare/master...bltb:master#diff-4157491939dd423de3edcb63a6492d11

I am not sure if it is

1. efficient
2. functionally equivalent (however it does pass the eunit tests)


You should either create a function that does it, or use redux-act like I mentioned before. A class is overkill for that.


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


Nice, but a bit slower...


Suggestion: A simple code example may help clarify.


yeah it just removes empty / "None" entries.


Hey @klq, this was my first commit about the installation.


DepricatedFields (mitchute) - x86_64-MacOS-10.9-clang: OK (2150 of 2170 tests passed, 533 test warnings)

 * 533 tests had: AUD diffs.
 * 2 tests had: BND diffs.

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


I prefer `0x414c!`. Original credits for the theme goes to [Chris Kempson](https://github.com/chriskempson/tomorrow-theme), I just ported one of his themes for SpeedCrunch.


This one is for the linear algebra part. @klq


test


What approach is better for scaling? I think it is just a misunderstanding of what combineReducers does.
```
combineReducers({
  data: dataReducer,
  router
})
```
would end up generating a state tree that looks like:
```
{
  data: {
    // your uploaded data
  },
  router: {
    // react router state
  }
}
```
We can talk about this more tonight.


Why? As I know `raise` is preferred to be used only when re-throwing exception.


bold


@arramac Why was the client-side Partitioning project removed from the solution?  Will it be restored?  Will it still be supported going forward given the new server-side partitioning that has been introduced?


@WeeJeWel you have had a nice Easter Monday, you fixed (made a workaround) for the problem I expected to have ...
I was looking why my Insight missed some data. (before this commit ) ;-)
Since this commit it works great!
Thanks!


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


@snoyberg This comment is no longer true with this change. Do you plan to revert this back in a major version bump?


What filesystem is Windows using nowaways? Will a forward slash always work? And will a backwards slash always fail? What works in which situations? I was always told Windows uses the backslash where Unix uses the forward slash..


This is reference to "Assignments 5a-5b (Python and Advanced Python)".
@klq


Because the current report has been modified - if we clear it then we show "no report selected" then we re-render the doc which is what caused the double loading in the first place. So we just replace it inline.


Because it's already inited.


Is this redundant? The call to performConsumerUpdates above should check for guest migration for each of the guests that still exist for the given consumer.


This branch starts here


Interesting I've used similar syntax to combine lists without dups, never thought to use like this. Quite neat.


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


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


fixed QL to SPARQL bug


There can only be one B and one A thread, anything stronger enforced by the constraints is a mistake :p


Not sure if you care, but… text for the link here should end with `redux`, not `elm` again ; )


Oh, checking is very easy. My laptop is very noisy :D

I tried the overall thing but you need so many games and I don't have the time.
If you see on the framework, there are times where a vast number of games favor a side. This happens on my tests too. You need many games like the framework to have a reliable result. And some times backfires.
If I cut the games on 100, i want to believe i also cut the treads and the result is more fair. :D

I remember some test i did all night. I stopped when it was  +300 for my beta (like 15000 or something games). I sent it to the framework and was a disaster.
With this matches thing, the same beta failed bad. I tested with kingsafety081 and some others good betas or framework tests and passed. Of course, can be a coincidence, but is more fun :D

Regards


So was this something I did?


@cmnstmntmn ping!


++ looks good


Yep it was like this but just checked foreach run faster than array_map, that's why I changed ;D


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


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


Congrats, nice idea.

As time management  patch this should be tested at other types of time controls to avoid problems there.
I would do a non-regression test for our classical STC (10+0.1) and for a sudden death tc like 15 seconds for the whole game.


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


addad hibernate.properties. Not hibernate.config!!


Will not this break multiple database schemas in PostgreSQL? This was never a thing is core but it is a well know way to support that in a the community. I'm sure that a lot of applications and gems use this support.

https://github.com/influitive/apartment/blob/3f8af6ae5e4188bea5c819f1b9ba31ad3c15eea2/lib/apartment/adapters/postgresql_adapter.rb#L54


@hipsterbrown


Hey there...In line 40, we just need the '}'. It is the closing one to line 9: ` if (Meteor.isServer) {`
Best regards, Irene


Wasn't clear on this, my thought it you'll use one or the other. Compiled languages probably configuration (see the C# library as an example), interpreted would be usage.


we order by it


These magic values are not static for this encryption method. You might be able to look for 0x4A52CADA instead which appears to be static.


@shasum FYI


I'd rather group examples under a header, so it's easy to navigate to that section (see the nav links at the top).


Pull requests accepted. :)


This looks like something that will be easier to understand by actually *running* it.  What's the hairy part to you?


Not sure why there's two issues for it, but hey. Since I have FO4 as well I've been checking to make sure Bash still loads. Until Bethesda gets done messing with the mod loading system though, it isn't of much use beyond using BAIN (which is working fine).


veins*


yes this was copy pasted =P


adf


The idea is right but you need to use mmap because it doesn't add any physical memory overheads in userspace process.


@bcamper right, this should be good in https://github.com/tangrams/tangram/commit/1049136c6cf7705701ce48f4678b6d83400e82c9


We should not be able to access it here, as this should be part of the activities-service. This number should come through a call to the activities service, so we can't migrate it until we have that.


yey


The commit message is wrong!  This commit reflects update to beta.13


This was one of the attempts, there were several.


Yeah, I did look at that. I like this for readability as text.


this is not a good idea :D


This diff doesn't show the whole file:

https://github.com/meteor/simple-todos/blob/9a08b96bae018a4ecb3d23dada624accdb817cb0/imports/api/tasks.tests.js


There's missing something here


malloc? for what purpose?
you even don't account memory allocated by malloc.


Это, кстати, спорный момент хранить или не хранить `composer.lock` в репе. На мой взгляд лучше хранить, т.к. это жестко фиксирует версии пакетов, используемые в продакшине, и нет риска, что при выкладке на продакшине подсосётся более свежая версия пакета, с которой что-то не заработает.


Prototype kode:

1. Define Controller and Action variables.
1. If routing enabled
  * Try to load url with routing rules. And decide Controller and Action
1. If Controller and Action are set, then load them
1. If Controller and Action are not set, try to find them out the old way, and then load them.
1. If Controller and Action is still not set, try to find default ones and load them.
1. If Controller and Action is still not set, try to load App_Controller and dispaly 404.
1. If controller and Action is still not set, display error message.


Pierwszy kod


What is going on here?
If except not rise then nothing has returned.
Maybe return cap in 155 line is with icorect tab?


Pkgver is getting out of hand. Get rid of -Jarvis part please All16.x are called jarvis


I would like to have "**B**arley" like in "**B**elhaven **B**est". ;D


nit pick: 1 space back


Fine to merge this, by the way, it isn't going out on the network right away.


@chumer  added a new subclass of TruffleInstrument to the REPLServer, but it wasn't getting registered for DSL processing until I made this fix.


@robertlight Why? the "party" object is a function param... it does not a property of the class...


Probably not going to like this but, you should probably split this section out too.  We should have a separate function that runs at the beginning to fill in version_search and then we can be sure it exists later.


blame


That is the idea since we do not have one service for that yet , we need it there!


These are examples of *implementing* the spec (for library contributors to use as they implement it), not using the library.(for developers using the libraries).


That's what the `examples/` directory is for. Will make links from here to there as I build that out more.


Already spam filtered.


This is (not documented) signature for the request itself. You can confirm a webhook comes from Nexmo that way.


added some folders


jakis komantarz


Thats not what the description implies.

`--` means only regular files

if you want it to apply to all files:

`/var/run/tgtd.*			gen_context(system_u:object_r:tgtd_var_run_t,s0)`


Just that to create a client, you need to provide authentication (either key/secret or OAuth).


the additional dirs for this feature are missing for the x64 tag AdditionalIncludeDirectories -> x64 build is broken now, see PR #1698 for the fix


All commits in a pull request end up in the history when a pull request is merged. If you use a more descriptive commit message, tools like `git blame` are more useful.

To change the commit message of the last commit, you can use `git commit --amend`. To change the commit message of an earlier commit you can use `git rebase -i master` and set the action for the commit to “reword”. (Assuming all commits are on top of master, otherwise you can use e.g. `git rebase -i @~20` to change the commit message of commits up to 20 commits back.) You’ll have to force-push after doing that.

With `git rebase -i` you can also squash a commit like this one onto its parent commit. Then it will not appear in the history at all and it will look like you got it right the first time. Set the action to “squash” to do that.


Do we really _need_ it there though? As in: are you actually using SocialActivities inside one of the `listAllMembers*` methods? Otherwise it makes no difference in the portlet whether the service call goes to Liferay or to the MembersClient.


I keep running into this error when running the playground a few minutes in:

TypeError: troi.emotion is undefined
 troi.js:115:1


- the hairy part is the code, yeah. As far as seeing it work, I showed it
off this morning(click on "Tiananmen Square", add an exclamation point,
press Enter to save), the only obvious issue is focus jumping to the
beginning of the field.
- tbh mostly I didn't want to deal with merging tree/master ~into~ this.
I'm sure it will be fine, but in the meantime I wanted to work off of what
I had checked into arvo; I think when I ran npm watch there was some manner
of snag.

On Thursday, 31 March 2016, Galen Wolfe-Pauly <notifications@github.com>
wrote:

> Fine to merge this, by the way, it isn't going out on the network right
> away.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/urbit/tree/commit/08d37b6084e5592e4c2595cd510c51282788b009#commitcomment-16920932>
>


@eduardocalil
1. Os testes não podem ser dependentes de ambiente. O arquivo `.env` não é versionado e esses dados variam dependendo da instalação do usuário.
2. Os testes devem testar o que pode quebrar. O que esse teste está fazendo, em essência, é testar se o python-decouple funciona. Nós assumimos que as libs que usamos funcionam. Não precisa testá-las.


Footer works better now....


@larsgrefer, please do not include any diagram PNG's in our code. Never ever. Please send me a link, I will clean the image from metadata using [MAT](https://github.com/jvoisin/MAT) and will upload it to our [Research Directory](https://spideroak.com/browse/share/AIMSICD/GitHub) so that you get a link.


Still, I think we are currently only using one of the attempts (i.e. the utility class with static methods). `@Autowired` does not work with static fields. If we're not using it, it should not be there.


Зачем так сложно?
Используйте pygame.draw.rect


- The focus jumping bug is something I have gotten around before.  Just need to set state between keystrokes.
- Where was the snag running watch?  Was it a bad merge?

> On Mar 31, 2016, at 12:49 PM, Anton Dyudin <notifications@github.com> wrote:
>
> - the hairy part is the code, yeah. As far as seeing it work, I showed it
> off this morning(click on "Tiananmen Square", add an exclamation point,
> press Enter to save), the only obvious issue is focus jumping to the
> beginning of the field.
> - tbh mostly I didn't want to deal with merging tree/master ~into~ this.
> I'm sure it will be fine, but in the meantime I wanted to work off of what
> I had checked into arvo; I think when I ran npm watch there was some manner
> of snag.
>
> On Thursday, 31 March 2016, Galen Wolfe-Pauly <notifications@github.com>
> wrote:
>
> > Fine to merge this, by the way, it isn't going out on the network right
> > away.
> >
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly or view it on GitHub
> > <https://github.com/urbit/tree/commit/08d37b6084e5592e4c2595cd510c51282788b009#commitcomment-16920932>
> >
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub <https://github.com/urbit/tree/commit/08d37b6084e5592e4c2595cd510c51282788b009#commitcomment-16921036>


I think I have a better solution for that. The problem occurs in 5.0 at several places in the report. it seems it's related to the method get_name_group_mapping. it return either bytes or a string. why ?
Is it in my database ?


I was actually wondering whether to include the guard clauses at all (the zero check as well) since the callers explicitly check that (1) tokens != null and (2) tokens.length > 1. So if this method really only exists for those Tokens (Cho and Seq) we could remove them.


xx


Oh yes - I see...thanks!


очень много аргументов, подумай как можно с помощью ООП уменьшить число аргументов


drink java, see eclipse


https://github.com/neowutran/ShinraMeter/commit/ea5a3af810712ef94e27ab07ec65c050ff9b6e08#diff-0ecf9ce83652f6b4e5f4e248e2f20ec1R40 dosen't affect rage uptime?


Yeah. I hope to get some time to work on it but between work and juggling multiple projects I haven't had the chance. I recently locked write access to contributors and since you had made some commits I felt I may as well add you. If that's OK with you of course.
I may make an OSX Cocoa based UI at some point.
On Mar 31, 2016 11:31 AM, Charles Milette <notifications@github.com> wrote:Hope to see something soon here.

BTW, why did you added me to AnimatedImage?

—You are receiving this because you authored the thread.Reply to this email directly or view it on GitHub


I think this would be more efficient if you built a new dictionary instead of looping the entire metadata and popping off things not in key_list
```
new_metadata = {}
for key in self.key_list:
...
```

P.S. This is probably not true with the implementation of dictionaries but it is true with lists (likely) that pop at a particular index or shift/unshift is kind of an expensive operation.


и потом, Level это у вас что? Если он представляет собой карту, то карта должна быть его внутренней переменной. Передаваться в конструктор например. Или, если Level будет чемто другим, то сделайте отдельный класс карты Map, который умеет себя рисовать  и включает этот массив строк.


That is the thing. I ported all calls in members portlet but the ones to configuration.
This one for SocialActivity may have a new service of its own. As well as the points count too. But they may make sense to be relayed by members since they are related to it? In an API perspective you have the URL /members/ID/activityCount and so on? It is a matter of organizing the information. But I agree that they belong in their own services once we do them.


Done first


Would it also be a good idea to build x64 release (I'm it's possible to build both)? I know the x64 builds aren't "official" but since it is included in notepadPlus.vcxproj it might be good to make sure they both build (Maybe even debug and release of each?)


Done and squashed. Sorry.


If the goal is simply to remove the package, you don't need to check for its installed state—simply set `state: absent` on the `apt` module task.


this doesn't seem right. `maxgaplen < mingaplen` Or am I missing something?


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


Acked-by: Sebastian Lackner <sebastian@fds-team.de>


new_branch_for_ci_test (kbenne) - x86_64-MacOS-10.10-clang: OK (2368 of 2406 tests passed, 0 test warnings)

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


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


sure I`ll delete it


I think this SHOULD go in the constructor because this is a one time use object.   Especially since some of the data is built based on the search_results.


Looks like you should add `*.un~` to your gitignore file.


просто self.space = Space()


Hmm, this defeats the purpose of having a strict version for the Miniconda2 package (it will upgrade its components). Maybe if you ask for a specific version of `anaconda` as well (I'm not sure if they track the Miniconda versioning or not)?


Я пользуюсь автоматическим форматированием в Visual Studio и мне хлопотно искать эти некоторые места с табами. Может быть ты объяснишь почему, это так важно?
В остальном. Ок.


The PHP manual says
> If format does not contain the character ! then portions of the generated time which are not specified in format will be set to the current system time.

http://php.net/manual/en/datetime.createfromformat.php

***
In the end that's really the better fix for that.


Probably is different in your db. We should make sure it is the right type when we get it, and probably check and repair that in the tools.

I think with the new schema code, we could check/repair the types of all values.


Поправили https://github.com/instantsoft/icms2/commit/09fa9fb8c9151eabf1fe5f2e4d326066cdfda338


:warning: procure escrever código HTML todo em minúsculo.


:warning: além disso, crie nomes com algum significado para classes e ids.


now please add  a filter so this event can happen only to pawns from capsules - i hate when it happens to prisoners whom i've got from raids


class to get response from httpservlet


:information_source: substitua por:
```css
strong#NumPasso {
  ...
}
```


++ looks good. Works as expected.


😄 本来想继续改用枚举，你先一步合并了。


:information_source: para dar nomes a `class`es e `id`s, procure usar a notação mais comum da indústria na web, que é de colocar tudo minúsculo e separado por hífen. Por exemplo: `num-passo`, `nome-passo`.


No worries. I don't mind it.


@nightink 顺手就写下😄


Don't forget to change the number of games!


:boom: não é assim que devemos fazer pra importar um arquivo externo, como vimos na aula.


Yes, but otherwise the (hyperopia) match tabs are broken.


:information_source: procure dar nomes mais significativos para `id`s e `class`es


@Finb 下次可以 CR 的时候提出来的。不用不好意思


:warning: indentação


The future matches are under hyperopia. The match list is under players, like the player overview.


You could (and I think even should) use [startsWith](https://developer.android.com/reference/java/lang/String.html#startsWith(java.lang.String)) here.


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


Prized Amalgam should recur, because the creature entered the battlefield from your graveyard, even though it is under the opponent's control.


:tada: :clap: :clap:


Needs error catching in the case that con is not instantiated to anything.


:information_source: procure usar a versão "atalho" - top, right, bottom, left.


Seemingly pointless.


:warning: nunca coloque espaços entre um atributo. Deve ser sempre `nome="valor"`.


the issue is in fact with the save line. it means that if we want to avoid the effect on animated GIF. We have to fix it with one of the proposed hack on Pillow or to avoid Pillow for animated GIF. grmph.

(thinking mode)


:information_source: aqui poderia virar apenas: `margin: 20px auto;`.


@nightink  弱弱地问一句，CR 是什么的缩写?...


:question: Por que abriu uma citação `inline` aqui?


:boom: indentação!!


:warning: procure dar nomes mais significativos


You're referring to the text area, right? The above fix was a Firefox 45 compatibility fix, so that wouldn't be reverted.


Infinite loop detected.


:boom: indentação


Ok it crashes. But it is expected because it needs new boot images (again).


buf[0] need not be declared as arrary.


#5544-Fully-autosized-version-of-5ZoneWLHPPlantLoopTower-hangs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1062 of 1062 tests passed, 0 test warnings)

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


:information_source: o globo era pra aparecer o tempo todo, mas só para os links que começam com `http`


Fixed in 9431b94bff939457cb915da3728cda6067674558


Enhancement for issue #96


Unnecessary usage of variable and break.


This commit corresponds to #48


Ótimo trabalho, @gustavojordao


There are some more things these tests could do better. The setup for the test is using command line args instead of the API


wifi信息实时获取


@abraxxa, :+1:
I hope my perl6 is better than my english! :smile:


:sunglasses: Ótimo trabalho, @laisrmlc!


Oh! useful `is_animated` discovered in
https://github.com/python-pillow/Pillow/blame/master/PIL/GifImagePlugin.py#L108

```python
>>> from PIL import Image
>>> im = Image.open('/Users/karl/Desktop/animated.gif')
>>> im.info
{'duration': 90, 'version': 'GIF89a', 'extension': ('NETSCAPE2.0', 885), 'background': 0, 'loop': 65535}
>>> im.is_animated
True
>>>
```


Txs. This test is seriously swinging from -0.25 to 2.50 back and forth.


是否可以把需要push的文本放到另一个文件中,这里读取后循环push一下


I might have needed scripts/CMakeLists.txt to build the submodule properly.


How about an additional option...
![fanart 2016](https://cloud.githubusercontent.com/assets/718838/14163760/80dfcc38-f6c6-11e5-9e4c-7534715db05c.jpg)


<script src="http://libs.baidu.com/jquery/2.1.1/jquery.min.js"></script>


update


thx


:warning: há mais de 1 link externo. Sendo assim, um `id` não é apropriado. O melhor seletor, neste caso, seria este:
```css
a[href^="http"] {
  ...
}
```


Hey guys.  Here's the PHP version of the Web Scraper that I'm working on.  It currently retrieves the page source however my regular expression isn't working properly.  I've checked the type of $subject and it's a 'string'.  I've also validated the my regular expression at http://www.phpliveregex.com/.  I currently have this running at c9.io


:information_source: Para dar espaço para o ícone, basta colocar um `padding-left: 1em`.


:warning: `id` não é apropriado neste caso


:warning: `id` precisa ser unico em uma pagina - ele não pode aparecer várias vezes.


Fixes #344


Pretty crazy, but a lot of fun. I hope for the best. Your change deserve it!


nomal


Professor, acho

2016-03-30 23:17 GMT-03:00 Flávio <notifications@github.com>:

> [image: :boom:] indentação
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/marchezinixd/cefet-web-ninjas/commit/6304cd28cae6a40b842fa4a8792c0b3425defe89#commitcomment-16908306>
>


++ nice. Works as described.


Professor, acho que você vai me xingar de identação em todos os trabalhos
que já mandei...

2016-03-30 23:35 GMT-03:00 Guilherme Marchezini <guilhermefmar@gmail.com>:

> Professor, acho
>
> 2016-03-30 23:17 GMT-03:00 Flávio <notifications@github.com>:
>
>> [image: :boom:] indentação
>>
>> —
>> You are receiving this because you are subscribed to this thread.
>> Reply to this email directly or view it on GitHub
>> <https://github.com/marchezinixd/cefet-web-ninjas/commit/6304cd28cae6a40b842fa4a8792c0b3425defe89#commitcomment-16908306>
>>
>
>


:clap: :tada: muito bem! Agrupando seletores para escrever menos código - gostei de ver.


If there are multiple change notifications emitted for a file, causing this if statement to be hit, `newTask` will never be completed because nothing ever sets it to complete. This means that it's possible to cause the tool to never ever run on that file again because tasks will be blocked on the TCS that is never marked as complete


Function should be used.


You should be very careful to always prepend your variable declarations with var. Setting the value of an undeclared variable creates a global variable.


Also, I got rid of `TopTextFieldDelegate` and `BottomTextFieldDelegate` and put textField lines in `MemeViewController` since the two files pretty much had the same lines of code.


:information_source: faltou colocar o globinho aqui.


pero, a adress le falta una d :P


So why not players/hyperopia with the list?


Don't add this in automatically. Selecting a value from the dropdown doesn't submit the form so this is unnecessary.


Great :)
I was trying to do the same thing. I checked through each forks, however none of them updated the README.

Let's take this further.


Should be a method. Redundantly used multiple times.


Great stuff, you totally crushed it. Try changing your logic to a switch statement if you'd like an extra challenge.


Looks good!

Was the use of explicit `'T` required? I think it may work leaving the type argument anonymous as `_`.


Redundant instantiation of variable tokens.


class to create a command interface


Mostly F# code omits `new` unless the type is `IDisposable`. This is fine, too, just a style note.


:clap: :clap: :tada:


:boom: indentação


There is an API [`Stopwatch.Restart()`](https://msdn.microsoft.com/en-us/library/system.diagnostics.stopwatch.restart(v=vs.110).aspx) which does this.


:information_source: na verdade, estas são `pseudo-classes`.


with is not a keyword


:clap: :clap: este é o melhor seletor para a situação em questão.


:sunglasses: Ótimo trabalho, @pedrofelipefroes!


All right, then it seems we did some optimization. High five :)


cc @jonleighton

Hi Jon, I know it's been quite a while since you've written this code, but I'm curious if the order of operations in this method is what you intended. Specifically, if a coder threw an exception here, the state becomes corrupt, as the next attempt to get `serialized_value` or `unserialized_value` will attempt to operate on `value` as if this `unserialize` or `serialize`  operation had completed successfully, when in fact it had not.

I know I'm performing some serious code necromancy asking you questions about this, I was just curious if my statement above rang true for you, or if you had a specific reason for ordering the operations in this way. Thanks for any help!


:clap: :clap: :clap:


You could use [`List.replicate`](https://msdn.microsoft.com/en-us/library/ee353665.aspx) here.


Removed unused time import as well


:+1: que legal que já está usando os elementos semânticos do html5


:shipit:


Uploaded new boot images, but I'm not sure it works. :-P


Instead of `printf "...\n"` you can simply use `printfn "..."`, which adds the newline for you.


:+1: continue dividindo seus arquivos em pastas, isso é uma ótima prática.


Class is never used in project.


*******************...................THIS IS COMMENT ON COMMIT ..........*****************


The addon itself can only have one fanart, but I could add it as a theme fanart. Maybe for Iconic?


:sunglasses: Ótimo trabalho, @Pehesi97! Com _easter egg_ e tudo o mais!! :wink:


There's no way for this class to connect to the database in this way as it is.


All of the prepared String Queries except for repository are never used.


:warning: acho que não pode haver caracteres loucões como `=` no nome de uma `class` ou `id`.


This is a pattern that a lot of people don't realize is a perf problem.

`for i in [a .. b]` will actually allocate and populate the list `[a .. b]`, then enumerate over it. Creating a list is not needed if all you really want is to loops over the values.

`for i in a .. b` (without the `[` `]`) is an optimized `for` loop that has much lower overhead, and does no allocation.


Unused Import statement


:boom: indentação!


Bom trabalho, @rafaelbaarbosa


Awesome! Can you write a regression test for this as well?

You can find an [example here](https://github.com/aw/picolisp-json/blob/master/test/test_regressions.l) and you can test it locally with: `make check`


:warning: repare a indentação


@skyline75489 pull request 的 code review


Oh


:information_source: melhor do que usar uma `class` e usar um **seletor de início de valor de atributo**:
```css
a[href^="http"] {
  ...
}
```


:warning: nunca coloque espaço em um atributo... deve sempre seguir o formato `nome="valor"`.


:clap: :clap: :tada:


Кажется, EWOULDBLOCK и WSAEINVAL можно не убирать из этого кортежа. Например, EWOULDBLOCK здесь может говорить о том, что ephemeral ports закончились, и, наверное, стоит попробовать сделать connect() еще раз.
Дальше по коду будет еще одна попытка позвать connect() при self.state == ConnectionStates.CONNECTING.


:sunglasses: Ótimo trabalho, @rcdutra! Gostei do "tom sobre tom" :wink:


:boom: indentação


:information_source: procure usar o atalho `margin`: a ordem é top, right, bottom, left.


Hi Ling,

Thanks for the feedback!  This is very helpful and the comments are
appreciated.

Re: Q8, 5b - Are you asking for sorting by alphabetical order?  Or sorting
by the default that Python provides?

Also, as an update... I bit the bullet and bought a MacBook Air.  I
experimented with SSH through Amazon EC2 but unless I had a super-strong
internet connection, the GUI was slow and non-responsive.  I didn't want to
take any chances!

-Anthony

On Tue, Mar 29, 2016 at 6:44 PM, Lingqiang Kong <notifications@github.com>
wrote:

> Hi @ak2912 <https://github.com/ak2912>,
>
> I don't have a lot of comments for the python section because mostly you
> are doing a good job and your answers for most of these questions are
> correct. So good job on that!
>
> Several problems that I spotted:
> 1. set vs. list: yes, set look up is faster than list, but the reason is
> not that set has non-duplicate elements. These differences are trivial and
> do not cause any difference in order of magnitude. Look up in Set is
> constant time, aka O(1) whereas lookup in lists is linear time, aka O(N).
> That is a huge difference! The reason for this is that sets are implemented
> with hashes whereas you need to traverse the list for lookup. If you are
> not familiar with these concepts, make sure to read up on it.
>
>    1.
>
>    In Q7 lists, your answer to Q7 list, linear_merge is problematic. Your
>    solution is not a linear time solution but runs in O(nlogn) time. Try to
>    fix it so that it runs in linear time.
>    2.
>
>    In Q8 in the 5b section, your answers doesn't seem to print out the 3
>    dictionary keys by order of last name. And I didn't see how you address
>    this problem in your code either. Are there any confusions with this
>    question? Try to fix this part as well.
>
> Overall, great job! Keep up the good work!
>
> -klq
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
> <https://github.com/ak2912/dsp/commit/93ee22c2eaa068c298c71f5d1d5c303fe5b60f8b#commitcomment-16887677>
>


neste caso em particular, poderia ter feito assim: `margin: 5px 75px;`


Bom trabalho, @romulolemes


:+1: I had the same issue


@djspiewak: This was the leak.  Bytes read into bv beyond remFront don't make it into the tail on line 170.  Those leaked bytes may include the line terminator, so the boundary would be prepended to goo from the next part when we get back to `beginPart`.


:bomb: isto aqui está bem fora de lugar!!


This looks nice! My comments are mostly focused on details.

Unfortunately there is no particular place to commit a performance test like this. When you submit your PR you will only be expected to show sample code that others can try, to verify the benefits of the change. I would recommend you put your perf test code in a GitHub [gist](https://gist.github.com/) so it's easy to share.

A few suggestions:

  - Include more than 1 data type. You have `int`s here, what about `float` or `int64`?
  - You can also test within FSI using the `#time` directive, which will automatically print timing and memory statistics for every submitted command. You timing function is fine, but it's also beneficial to see how the memory usage has changed.


:bomb: `<div>`s só podem existir dentro do `<body></body>`


这次更新解决了我的问题。在这之前我是很沮丧的——为什么作为一个例子 npm install 后它会报错？


:information_source: para refletir: o que quer dizer esse sufixo "config" nos nomes das classes? Será que ele é mesmo necessário?


Bom trabalho, @Karovd


106994938-FixFrontSideSolarReflectance (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2176 of 2208 tests passed, 1 test warnings)

 * 33 tests had: EIO diffs.
 * 8 tests had: ERR diffs.
 * 30 tests had: ESO big diffs.
 * 17 tests had: MTR big diffs.
 * 27 tests had: SSZ big diffs.
 * 32 tests had: ZSZ big diffs.
 * 26 tests had: Table big diffs.
 * 5 tests had: Table small diffs.
 * 3 tests had: SSZ small diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MTR small diffs.

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


I thought about it today and I believe the issue is here. The previous issue I fixed was :
- One XML with all the ML was sent and it was to big and cause a bug. So I fixed it by breaking it into multiple xmls.

But we still try here to do a single multicall, which is very very big. So DedicatedServerApi will automaticall execute the multicallback when needed.
So even throught we are calling sendDisplayManialinkPage with multicall true it actually might do the executions imediately. So need to catch that exception as well.


> please do not include any diagram PNG's in our code.

Why?


If you think the issue is with size here is the code, maybe divide by 8?
The 4 is a magical number I found with many tests XD I coldn't find a logic to it.


And where is this overscan1280x720.png and overscan1920x1080.png files?


The following doc(s) have been deployed to the [1367mobile](http://1367mobile-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/mobile-redirection](http://1367mobile-static-docs.pantheonsite.io/docs/mobile-redirection)
- [source/_docs/mobile-redirection](https://github.com/pantheon-systems/documentation/commit/6c386a82f6324218555782a1a7a26c5564e39caa/source/_docs/mobile-redirection)


I agree with that.
I'll try to export then reimport my database. I think the problem occurs because this is an old test database I used it with 3.3, 3.4, ... and now 5.0.
So this can exist with user's databases.


Do you even Nerderlands?


ASIHPdev3 (mbadams5) - x86_64-MacOS-10.9-clang: OK (2176 of 2178 tests passed, 552 test warnings)

 * 553 tests had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

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


Hmm, a test?


тут ты чекнул на налл а ниже в цикле нет. Непонятно


That's an interesting point about the API and we should discuss that in our next meeting. Relaying calls like this could make for a nicer API, although it might be difficult to find a cut off point when we use /points?userId=xx and when we use /members/id/points. Relaying would also have a small performance impact (2 servers involved in 1 service call), so when in doubt I'd lean towards the former method.

When possible, I would try to touch everything only once rather than moving them around until the correct service is created. There is really nothing wrong with leaving service calls inside a portlet. As I mentioned on Tuesday, we will have to do this in more complex portlets unless we create all services at once.


@martijnboland why would this support webpack 2? It's still installing `1.12.14` even if the consumer has `"webpack": "^2.1.0-beta.4"`

Should this be under `peerDependencies`?


106994938-FixFrontSideSolarReflectance (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


no, it is not released yet. will be soon


Why does the version number contradict [http://blog.openvbx.org/](the blog) - which is still indexed by Google. Should probably delete the blog or do something to avoid confusion since it looks like 1.2.5 was released in Feb 2012 and 1.2.20 is released Feb 2016.

Sorry, that's all I have to add. Hopefully this cache change will fix some of the issues our install was having.


I'm going to fix that one as well and do some regression testing. Thx!


This will break on a new (re)install, as import depends on `use_hosts` but one can't enable block hosts since [there is no hosts file yet](https://github.com/M66B/NetGuard/blob/dd7ce17323b97c840f925cce48d195b2ee888b0d/app/src/main/java/eu/faircode/netguard/ActivitySettings.java#L271), you can't import one.


hi Roman,

thanks a lot  for your feedback. I'll fix all those issues next week.


Hi @thepeppi!

What went wrong with the install? Were you able to contact fail2rest, does fail2web load?

Also will cut the philmoore link.

thanks


I added a new utility function for it. This should make it clear.


The contents of the tables should have been converted to str as part of the python2 to python3 database upgrade.  However, we probably forgot about the name group table.


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


просмотрено


I am confused on this. So use it the way I was using it but put the formatting in the constructor instead of the property?


didn't initially spot that, but the _ns on the left sides in this block are obviously wrong.


Could you not just one file?


This path is really weird: `/Users/Seth/Sandbox/greatgiving/node_modules/material-ui/node_modules/react-event-listener`.
`react-event-listener` is not a dependency of Material-UI. It's not supposed to be here.


This actually has to be turned off when calibrating and we may want to auto calibrate one day so maybe state "Should be on when measuring multiple channels"


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


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


* *use_hosts* depends on *filter*
* *hosts_import* depends on *use_hosts*
* *hosts_url* depends on *use_hosts*
* *hosts_download* depends on *hosts_url*

I don't see what is wrong, but maybe I am missing something?


Логичнее тогда PrintInfo чтоб возвращало строку которую тоже тут же и выводишь


@robertramey That broke building of Serialization as a dynamic library with MSVC. It's complaining that the __declspec(dllimport/dllexport) must only appear on either class or function level, not both. You can remove the BOOST_UTF8_DECL from the function declarations.


I'm guessing it's installing 1.12.14 because it's the latest stable version?


@thewisenerd, happy?


LoL


Git pull John ;-)


maybe this would be a good time to return 'v[THz]' and change the client to use units... I can do this one day as well if not


Dat indentation


Again here with "v[V]'"


Yay, no more indexerror


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


Consider making test_provider_crud.py a py.test.


If you want to do this, please use "set_pid_p".


import java.io.*;

public class Hw1_Nuke2 {
	public String readWord() throws Exception {
		BufferedReader keyboard;
		String inputLine;

		keyboard = new BufferedReader(new InputStreamReader(System.in));
		System.out.print("Please enter the word (without spaces) : ");
		System.out.flush();
		inputLine = keyboard.readLine();
		return inputLine;
	}

	public String transWord(String word) throws Exception{
		int le = word.length();
		char[] result = new char[le-1];
		result[0] = word.charAt(0);
		for (int i = 1; i < le-1; i++) {
			result[i] = word.charAt(i + 1);
		}
		String reString = new String(result);
		return reString;
	}

	public void print(String word) {
		System.out.println(word);
	}

	public static void main(String[] args) throws Exception {
		Hw1_Nuke2 test = new Hw1_Nuke2();
		String input = test.readWord();
		String output = test.transWord(input);
		test.print(output);
	}
}


the problem here is to serialize response to user model :P because right now u will not have any spacial methods like `delete` or `save`


cut spaces.


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

Оффтопик: смешивание табов и пробелов не всегда плохо лишь с эстетической точки зрения. Программа на Python не отработает, если в ней выравнивание сделано и табами, и пробелами.


spaces.


can you do a PR against 1.0-maint with the fix?


basically the `advance` false broke the JArray parser b/c the JArray parser was broken to begin with


This is an extremely bad change, killing the project further.

After this, how can I find the memory check on the same thing (the same client subroutine/hack type) for different client builds? Note it cannot be in general the binary string (`result` field), because different builds may have a bit distinct binary code in addition to different offset. This will be the case if the fragment checked refers to memory, i.e. contains an instruction with an offset in the data segment. By comment? Then, who is ready to enforce such policy?


What if the files don't exist, shouldn't we put some of this in a header file or something?


перетаскивание объектов не влияет на детей. Необходимо на основании `event.point` и `coord` сделать пересчет координат coordDraw для детей.
В настоящий момент при перетаскивании `coord` не изменяется. В будущем может понадобиться для изменения формы стены.


List of issues

1. Reachability class extends from [ConnectivityManager.NetworkCallback](http://developer.android.com/intl/es/reference/android/net/ConnectivityManager.NetworkCallback.html) which was added in API level 21. So if you try to build the APK for API < 21, it's gonna throw an error when this class is compiled (When this happen, the Reachability.java generated try to extend from ConnectivityManager)

2. As default, 'resources' folder is selected as resources_dirs, but android.rb is declaring explicitly any value. So when you build the project R.java doesn't contains any constants referencing to your resources.

3.  In the android.rb file are instructions to delete ios and osx files if they exist. The problem is that `path` method is missing. After debugging i realized that `path` is an array.

4. Android theme -  android.rb file is declaring theme for application so if you want to use your own theme there will be a conflict.

5. AppCompat and PlayServices jars -> It could be a bad practice to use an appcompat.jar mainly for two reasons. The first one, the constant updates of it and the second one, to avoid duplications if you are also importing appcompat from your app configuration. Same case with play services.

6. We had to remove the ui component because seems that there's a problem with `Init_CSSNode`. I don't know what is exactly happening.


Exercicio GIT bittti


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


+15 * pop = +10-9=1
+10 * pop = +6-14=0
+12 * pop =+12-6=2
+20 * pop = ?
+5   * pop = ?


no host.txt -> block off
block off -> import host off
import host off -> no host.txt


The Description field should not start with the package name,
  'This package' or similar.
* checking package namespace information ... OK


Hi,

Congrats all for the great work!

I don't have a HoloLens so I will use my PC. I am unable to run Galaxy Explorer in Unity. Do I need to download the app as zip file to my computer and run it from there or can I run it from GitHub?


That's okay, and thank you :)

On Fri, Apr 1, 2016 at 2:34 AM, Ravi van Rooijen <notifications@github.com>
wrote:

> (it's hard to reply to commits on a phone...)
>
> I can translate tomorrow or at least the day after
>
> (also the 9 hour time difference doesn't really help...)
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/Techern/Igneous-Extras/commit/463b2847f926d5ecb22463d13228a4274cb30082#commitcomment-16918142>
>


Shouldn't be a problem with the new AbnormalityDuration (https://github.com/neowutran/ShinraMeter/blob/master/DamageMeter.Core/AbnormalityDuration.cs). I tested it on few run (~4 run), that look fine


```javascript
var s = new THREE.SpotLight(); // or DirectionalLight
s.clone();

three.js:17636 Uncaught TypeError: source.shadow.clone is not a function(…)
```


You have my name linked with your github acct ;-)


12 looks interesting. I will try 13 and so


[herd-service] Secrets should handle values in config with =


I've been using it with 3.5 and haven't encountered any issues for a week or two; is it not officially not supported?


Yes, you are right, I forgot that this is done in ActivitySettings


This breaks `test_no_duplicates`:
```
..F.............
======================================================================
FAIL: test_no_duplicates (__main__.TestAllURLsMatching)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test/test_all_urls.py", line 97, in test_no_duplicates
    '%s should not match URL %r . That URL belongs to %s.' % (type(ie).__name__, url, tc['name']))
AssertionError: VoxMediaIE should not match URL u'http://www.sbnation.com/college-football-recruiting/2015/2/3/7970291/national-signing-day-rationalizations-itl
l-be-ok-itll-be-ok' . That URL belongs to Generic.

----------------------------------------------------------------------
Ran 16 tests in 8.469s

FAILED (failures=1)
```


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


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


106994938-FixFrontSideSolarReflectance (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1062 of 1062 tests passed, 0 test warnings)

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


Hi @klq, I've finished the python sections.  All of them, I believe.  Let me know if you need me to comment on each one.


106994938-FixFrontSideSolarReflectance (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2144 of 2176 tests passed, 1 test warnings)

 * 33 tests had: EIO diffs.
 * 8 tests had: ERR diffs.
 * 30 tests had: ESO big diffs.
 * 17 tests had: MTR big diffs.
 * 27 tests had: SSZ big diffs.
 * 32 tests had: ZSZ big diffs.
 * 26 tests had: Table big diffs.
 * 5 tests had: Table small diffs.
 * 3 tests had: SSZ small diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MTR small diffs.

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


Yeah, the snippets are meant to show just the relevant lines, it's awkward here that there was exactly one line missing from the bottom.


Ack.


CC @Zverik

> On Mar 31, 2016, at 20:57, Viktor Govako <notifications@github.com> wrote:
>
> POI-исключения: которые не принадлежат к общим классам (amenity, shop). Раньше компонента была в cpp, но после анализа кода, выяснилось что такая же штука нужна в другом месте. Вынесена в публичный интерфейс. Допишу коммент, что тут по чем.
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub <https://github.com/vng/omim/commit/df751b653961f931c8ea3a4b12c08f64d2b1a490#commitcomment-16919341>


Доволен результатом)) теперь можно добавить определение типа файла по расширению


Ack.

(Actually, backporting the tests in 3fefd3aa2df5ebb91afd6cba81b94a0c85fef68e should also be trivial, but I'll leave it up to you of course.)


I'm working it out.

just take me out of botPlayground when you work on your code.

________________________________
From: Insectoid <notifications@github.com>
Sent: Thursday, March 31, 2016 12:48 PM
To: jyoshimi/phil_190_2016
Subject: Re: [jyoshimi/phil_190_2016] minor bug (e0bd15c)

I keep running into this error when running the playground a few minutes in:

TypeError: troi.emotion is undefined
troi.js:115:1

-
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub<https://github.com/jyoshimi/phil_190_2016/commit/e0bd15ce21a40f8e2c0e9c520359a7df5d5a7a97#commitcomment-16921034>


_modules returns an array not an array ref


add change1 and 2 test


5/7 logging would read again


This is only for initializing skills map. When adding a skill for the first time, it creates the map so it can put the skill, but when removing the last skill, it doesnt delete the map.
Actually its my mistake I didnt look at the surrounding code at all... I thought the NPE was because _skills map was being set to null somewhere, but thats not the case. The case is that when you do _skills.get(skillId) and at that moment, the entry got removed, so you get null returned. Because its a map with object Integer and not primitive int, return _skills.get(...) throws npe instead of returning the default for the primitive int - 0. The fix is similar... just make a cache of _skills.get(...) and check if its null...

Integer skillLevel = _skills.get(skillId);
if (skillLevel == null)
   return 0;
return skillLevel;

or more Java 8ish fix will be

return _skills.getOrDefault(skillId, 0);

And sorry again, I didnt see the whole code so I thought the NPE of slightly different kind.


No, it just happens. The csproj file will have inconsistencies between our branches as we add more files. The first few merges did not properly merge the csproj, I noticed this, and manually fixed it. Chances are, it's going to be happening a lot. Don't worry about it.


BA does not actually use reverse build reclaim so this particular typo would have went unnoticed regardless of how much they tested it.


I told you http://youareabadlet.com/


Showing positive intent with all checks passed image :smile:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 20312](https://teamcity.mesosphere.io/viewLog.html?buildId=186554&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 267 Build time: 00:17:50
### Failed tests
```
null: Kubernetes e2e suite.[k8s.io] Kubectl client [k8s.io] Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


@devanshk I'm not sure if you should make installing mysql instructions part of the main README.md

It would seem better as a page in the wiki because not everyone needs to know how to install mysql either because they know it or because they will be using other adapters.


Shh. :<


添加接口


Cool 😊


Ack.

You might want to backport 32f72989b6bec7f79f434bffc2d4b10358b18a8c as well (after this patch), otherwise those changes are not really necessary.


Ack.


@klq: Completed assignments 5 and 6. Still have lots to learn about regular expressions!


@remzr7 That's a good idea, I'll shift these into the FAQ


Good change, this makes the code more reliable.


This patch causes issues when crosscompiling Wine for OSX. I have not investigated if this is a problem of the toolchain we use for the WineHQ builds, or if its really a problem with this patch. Its easy to workaround, so I wouldn't mind if its applied anyway, but just to make you aware of the issue... ;)


Is the link needed? Browsers will automatically request /favicon.ico to get the favicon, so that doesn't need to be specified in the HTML document.


Be sure you download all of the HoloLens Tools from the Microsoft Developer website (http://lens.ms/Sa37sr). The project was built with Unity HoloLens Technical Preview and is not supported on other versions.
Once you have these tools installed, I would either download the project as a zip file or clone the Galaxy Explorer repo onto your machine. You should be able to run the app through Unity or with the HoloLens Emulator.
Enjoy! We look forward to seeing what you create!


(facepalm)


I haven't had a chance to test it out yet, but I also think the filename would be `favicon.ico`, not `.png`.


I see the discontinuities everywhere, not just at EVs.  On the teapot sides, for example.  Gregory basis didn't seem to fix it.


But yeah the `<link>` code might not be needed as @sb8244 said.


os_2_0_develop (macumber) - x86_64-MacOS-10.10-clang: OK (2061 of 2204 tests passed, 0 test warnings)

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


fix bug #3


is a 404 more appropriate here? since the item_id is part of the query string, anything could come in. also, the error message seems a little malformed.


is it necessary to set itemIndex? i don't see it used later.


what happens if your get(“quantity”) is not the string reprensentation of an int?


i think you meant {{ current_quantity }} here.


Ok, I'll think about it tomorrow ;)


the amount is never added to total_discount.


Ack.

If you are interested, there were two patches with test cases for this change, which apply fine on your current git tree:
1bebee9c1b119d727991aee50451e36e0865f927
90819e9c1b2c0575993128d962df1b348e93adc9


Buenas tardes Profesor, con respecto a las preguntas:

1. nodejs-legacy es el paquete que se encuentra en los repositorios estándar de Ubuntu/Debian y lo que hace es crear un link simbólico de node en /usr/bin y toma los paquetes nodejs y npm por separado como dependencias. La opción que esta planteada es el repositorio de NodeSource que sube las versiones nuevas de nodejs mas rápido que el repositorio estándar y con el paquete nodejs instala todas las dependencias directamente (node y npm) y crea el link simbólico como debe ser.

2. Si es el usuario que lance la app web.

3. Los módulos que necesita la aplicación se instalan posicionándose en la carpeta node-server del proyecto y ejecutando npm install. En el archivo package.json están listadas las dependencias del proyecto, es decir los módulos que necesita.

Cualquier otra cosa nos avisa. El fin de semana integraremos lo que hizo @SaraitHernandez.


did you mean to close this connection when done?


Makes sense to me!


## Task 1

When working with sequence data, you often need to iterate over two files at the same time (think of iterating over Read 1 and Read 2 fastq files simultaneously). Using the `argparse`, `itertools`, `glob`, and `os` package, write a program that iterates over the lines of the two files in the task-1 directory, simultaneously.  Your program should contain a main loop and an ifmain statement, and it should be formatted correctly.  You should run a linter against your code to make sure you have formatted it according to PEP8.

- [ ] Program is correctly formatted (2.0 pt)
- [ ] Program `argparse`, `itertools`, `glob`, and `os` (1.0 pt)
- [ ] Program functions as requested (2.0 pt)

## Task 2

When you're working with field data, you often need to manipulate dates. Using the `datetime` module, write a program that determines, from now until the end of 2016, how many weekend days there are.  You don't need to take any input from the user, and you can output the answer to the screen (it would be helpful if you could format this reasonably to explain what is being output). Your program should contain a main loop and an ifmain statement, and it should be formatted correctly.  You should run a linter against your code to make sure you have formatted it according to PEP8.

- [ ] Program is correctly formatted (2.0 pt)
- [ ] Program uses `datetime` (1.0 pt)
- [ ] Program functions as requested (2.0 pt)

## Task 3

Generalize the program you wrote above to take an arbitrary start and stop date and compute the number of weekend days between those two times (inclusive).  Use `argparse` to take the start and end times on the command line, and make sure that you make it clear for the user exactly how they are supposed to enter these start and stop dates. Your program should contain a main loop and an ifmain statement, and it should be formatted correctly.  You should run a linter against your code to make sure you have formatted it according to PEP8.

- [ ] Program is correctly formatted (2.0 pt)
- [ ] Program uses `argparse` and `datetime` (1.0 pt)
- [ ] Program functions as requested (2.0 pt)


No matter, only took me hours to figure out ;) now I know a lot more about
yml.


i don't think it's necessary to select the entire row here. you could select the count and check that against 0, rather than checking for a "None" result.


i appreciate the method comments!


This patch was wrong. You might also want to backport the follow-up fix:
97c70c86be26e02a398411877d6e6a26367d3ab8


Extra > here


hehe, Whut? is a dropper bug? i guess? si hablas español dime, para ayudarte :D


Dam your fast. I was just about to do this. ;)