Great work! Works perfectly. Functionally, I have nothing to add. The only thing I'll say is that there are some formatting things that you can do to make the code even cleaner and more organized. My favorite JavaScript "style guide" is the one made by AirBNB. Take a look at that to start thinking about the things that are important to standardize when writing JS: https://github.com/airbnb/javascript/tree/master/es5

Let me know if you have any other questions!

#FEWD Week #5 Project: More JavaScript

---

###Description

JavaScript is powerful: we're able to take user interaction and information and use it to update the page without requiring a refresh. This week, a new startup in your city, CitiPix, is looking to create a personalized picture app that allows users to store pictures they've taken around the world and quickly access them by typing in keywords or tags associated with each photo; they have asked for your help to start building the front-end prototype. You will begin by building a web-app that will take a user's input and update the background image of the page based on that input. Once again, you have been given the starter code that includes the HTML and the CSS. Use the images in the image folder and write JavaScript to make the background image reflect user input based on the Technical Requirements  below.

Student: Saline Jiang

|                                                                                |                                |                        |                          |
|-------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                                | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Call ```$.val()``` on inputs to get the string value of your user's input                                                     |                                |                        |           X               |
| Store user input in ```var city```                                                                                |                                |                        |             X             |
| Use $.on(submit) or ```$.click``` to figure out when the user clicks the "submit" button                                      |                                |                        |              X            |
| Create ```if / else if / else``` conditionals to control the flow of your application                                         |                                |                        |                X          |
| Write at least six different lines of pseudocode and display them inline as JavaScript comments                               |                                |                        |        X                  |
| Prevent a form submission using the ```event.preventDefault()``` function                                                     |                                |                        |                 X         |
| Use the ```$.ready()``` handler to delay your code from executing until all DOM assets have been loaded                       |                                |                        |              X            |
| Use the ```||``` operator in your conditionals to allow for multiple string values to execute ```if/else if``` statement code |                                |                        |             X             |
| If a user submits "New York" or "New York City" or "NYC" make the background of the page nyc.jpg                              |                                |                        |             X             |
| If a user submits "San Francisco" or "SF" or "Bay Area" make the background of the page sf.jpg                                |                                |                        |             X             |
| If a user submits "Los Angeles" or "LA" or "LAX" make the background of the page la.jpg                                       |                                |                        |             X             |
| If a user submits "Austin" or "ATX" make the background of the page austin.jpg                                                |                                |                        |             X             |
| If a user submits "Sydney" or "SYD" make the background of the page sydney.jpg                                                |                                |                        |             X             |
| TOTAL: *26*/26


Hi,

This block of code was intended to implement the function timer.delayed_call() when I did not know there is timer.delayed_call(). It is not useful anymore.


I think this change resulted in https://github.com/camptocamp/puppet-postfix/issues/101


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


I rebuild this addon to use tvheadend 4.0.7, you can download it here:
http://openelec.tv/forum/79-tvheadend/78918-build-tvheadend-addon-tvheadend-4-0-7-inside#151856


Probably you forgot to update the localStorage value to.


...and looks like we don't need to do anything :) https://atom.io/docs/v1.2.0/behind-atom-maintaining-your-packages#adding-a-collaborator

@steak3 you can publish as you want with `apm publish patch` or `apm publish minor` directly from your repo's local clone and the version bump and tag will work out of the box :sparkles:


the folders including all files get created as soon as the drc tool gets installed (drc package in ubuntu : https://launchpad.net/ubuntu/+source/drc).
The DRC RB plugin is just a frontend for this tool. That's also why I decided to remove my  erb44100.drc  config and rather take the default delivered with the tool and modify during runtime.


ah, right. of course there's an API for that :wink:


Hm, is this by accident?


nice.


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


You are correct. I'll fix that. I mistyped intending to tell developers that their apps should target iOS 8.0 and above.


According to [the docs](http://www.postgresql.org/docs/current/static/app-psql.html) the flag you're looking for is the `-f` or `--file`.


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 4](http://build.prxm.it:8811/viewLog.html?buildId=4712&buildTypeId=Jarvis_DocumentStore_GitVersion) is now running


Add spaces between tests.


Андрію, в тебе "some" - улюблене слово? Воно ж нічого не відображає.


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


I see, nice! Thanks! :v:


Ah, OK that makes sense - unfortunately clang also defines __GNUC__ so I wasn't sure if the idea was to test for platform or compiler (or feature).

In that case the existing message sounds reasonable, the BSD equivalent would be kqueue but the poll() fallback also works on Mac OS X too.


The older request was http to, but i will fix this soon with https.


et #2


I suppose it needs some more work, I will review the methods I added and make sure that they compile.


Okay, then I see no point in keeping it commented, just remove it. It will be still in the git history :)


Hlavne zmeny
*  novy Interface Reporter - to vyriesi problem v triede IssueTicket, nemusime tam mat atribut specialne na Customera a na Techniciana.
* presunul som niektore metody blizsie k triede s ktorou pracuju(napr. ked ma Customer referenciu na KnowledgeBase tak som mu odobral metodu listCommonIssues a dal ju do triedy KnowledgeBase )
* nova trieda IssueCatalog - povodne sme mali  navrhu ze kazdy Employee/Customer si drzi List<Issue> ktore vytvoril. To nie je dobre, takze kazdy employee ma odkaz na IssueCatalog ktory mu poskytne Issues. Customer nema, ten len reportuje.

TODO
* skontrolovat ci sa dostaneme po sipkach vsade tam, kam potrebujeme


This is a bit redundant (and ambiguous with respect to `GROUP BY groupingElement (',' groupingElement)*`)

I would just change it to a single expression.


well, you are 50% less rubbish now, but the 'i' in Implementing still needs to be an 'I'


Call the label `#rollup`


Call the label `#cube`


I see it as a general callback for the coordinator to do initialization when a group is loaded in the cache. At the moment it's just logging because there's no initialization necessary, but I'd prefer (slightly) to keep the name general at least to keep the symmetry with `unloadGroup`.


need you the variable i global? otherwise add a var. ;-)


Maybe a comment as to why the "python" is necessary?


We each had to edit the build.properties file locally to build the project, so we added the file to gitignore so as not to propagate our local changes. I guess it was deleted when we pushed our changes.


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 1.1.5](http://build.prxm.it:8811/viewLog.html?buildId=4712&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **SUCCESS**
Summary: Running Build time: 00:04:14


Yeah, that's fair, probably debug is more appropriate. This case would be hit if a partition gets reloaded unnecessarily.


Ahh, this was the line @VesnaT was talking about.

Thanks for this.


I also added the missing secrets.yml file with some dummy keys. This is ok for development?


Good idea.


Why set Win7 for this ? And other ?


Can you use `ModelAsController::controller_for()`?

We shouldn't program this internal logic into our code.


spelling mistake


Please remove automatic formatting changes.


You don't have to declare controllers any more so this will fail on edge cases


Nothing else at all, unless a build system adds things to that first line. :+1:


@InnaBoitsun there is a spelling mistake 'spartpage-view' should be 'startpage-view'


Just to ignore some build files


This `if` does not take into account that Firefox on Android also has `vjs.ANDROID_VERSION` set but does not support HLS at all. I have a video that only plays in Firefox/Android if I choose “Request Desktop Site” from the menu (because then the user agent string does not include the android version and `vjs.ANDROID_VERSION` becomes `null`).


Split this change into a separate commit.


Fixed gitignore to have secrets.yml and this other file. The other file being something I have no idea about yet :) #securityBestPracticse


A dummy test.


seterrno?


Great work! It all works perfectly as far as I can tell. Couple of small things that can improve your code or help you see it in a different way:

1.) For `var array = ["NYC" , "SF" , "LA" , "ATX" , "SYD"]; `: the variable name `array` is not very descriptive. Every Array is an array; we should come up with a name that describes what type of things are _inside_ of the array. In this case `cityArray` or `cities` would probably be a better name.

2.) Just as a demonstration, here is how you would do your `for` loop as a `forEach` loop:

```
var cities = ["NYC" , "SF" , "LA" , "ATX" , "SYD"];

cities.forEach(function(cityName) {
    $('#city-type').append('<option>' + cityName + '</option>');
});
```

Let me know if you have questions about these!

#FEWD Week #6 Project: Working with Arrays

---

###Description

The team from CitiPix has reached out for iteration on their prototype.  Instead of using a user's text input to change the background of the interface, the CitiPix user experience designers have asked to see how a drop-down menu would work in place of an input field.  The product managers are hopeful it will also lead to less error handling for their engineers and data team.  Use what you've learned this week about data-types and arrays and use the starter code - or your code from last week - and make the adjustments to your JavaScript file. Be sure to start out with Pseudocode.

Student: Savanna Rovira

|                                                                                |                                |                        |                          |
|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                                | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Create an array with the following values: "NYC", "SF", "LA", "ATX", "SYD"; use the array to add values to the `````` menu by using a ```for``` loop in JavaScript (don't update the HTML to do this!) |                                |                        |            X              |
| When the user changes the input of the drop-down, update the background image based on what they selected                                                                                |                                |                        |    X                      |
| Use $.append() in your iteration on the drop-down menu                                                                                |                                |                        |                X          |
| Use the ```$.attr()``` function to update html classes                                                                                |                                |                        |            X              |
| Get the value of user input using ```$.val()```                                                                                |                                |                        |            X              |
| Use the ```$.change``` event handler to capture user actions                                                                                |                                |                        |            X              |
| Use ```if/else if/else ``` conditionals to control the flow of your application                                                                                |                                |                        |           X               |
| Display your pseudocode as Javascript comments                                                                                |                                |                        |            X              |
| TOTAL: *16*/16                                                                                |                                |                        |                          |


Add it back.  If you have difficulties with that I can help you revert this commit.


This array needs to be wrapped in triple backtics so the formatting will look right once it's rendered. See: https://github.com/ResponsiveImagesCG/wp-tevko-responsive-images/blob/5caa98a69f0dd32a471ba2ff9a7384fcd8c623e4/readme.md#hook-wp_calculate_image_srcset


cool, good idea with %


Can this not just default to `true` and thus be compatible with other client libraries instead?


Whoops! That should have read "v0.0.2".


Hm, this works:
```
[root@nixos:/]# ln -s foo bar
[root@nixos:/]# ln bar bar2
[root@nixos:/]# ls -Al |grep foo
lrwxrwxrwx   2 root root        3 Nov 16 17:26 bar -> foo
lrwxrwxrwx   2 root root        3 Nov 16 17:26 bar2 -> foo
```

But this does not?
```
[root@nixos:/]# ln /nix/store/04980l8njsdziz58p30iwiq41wyr6shz-linux-pam-1.2.1/sbin baz
ln: failed to create hard link ‘baz’ => ‘/nix/store/04980l8njsdziz58p30iwiq41wyr6shz-linux-pam-1.2.1/sbin’: Operation not permitted
```

Hmm looks like a bug in smartos, not in nix:  https://gist.github.com/vizanto/da1d4f5628ae8b931e17


Also, see related update from Core: https://core.trac.wordpress.org/changeset/35592


107712554-FixRefrigerationCondenserWaterCooledNotRegistering (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1912 of 1963 tests passed)

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


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 5](http://build.prxm.it:8811/viewLog.html?buildId=4714&buildTypeId=Jarvis_DocumentStore_GitVersion) is now running


@nicolas-grekas I will do this evening (unless someone else does it first, for instance you)


Still i am getting the below issue.

[launcher] Process exited with error code 1
/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/node_modules/q/q.js:126
                    throw e;
                          ^
Error: No selenium server jar found at the specified location (/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/selenium/selenium-server-standalone-2.45.0.jar). Check that the version number is up to date.
    at LocalDriverProvider.addDefaultBinaryLocs_ (/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/lib/driverProviders/local.js:37:11)
    at LocalDriverProvider.setupEnv (/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/lib/driverProviders/local.js:71:8)
    at Runner.run (/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/lib/runner.js:254:31)
    at TaskRunner.run (/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/lib/taskRunner.js:123:19)
    at createNextTaskRunner (/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/lib/launcher.js:220:20)
    at /usr/local/lib/node_modules/protractor-perf/node_modules/protractor/lib/launcher.js:243:7
    at _fulfilled (/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/node_modules/q/q.js:797:54)
    at self.promiseDispatch.done (/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/node_modules/q/q.js:826:30)
    at Promise.promise.promiseDispatch (/usr/local/lib/node_modules/protractor-perf/node_modules/protractor/node_modules/q/q.js:759:13)
    at /usr/local/lib/node_modules/protractor-perf/node_modules/protractor/node_modules


I agree, log is a verb, whereas logger is a noun, so it's important in the context of `func log()` in that it does not log, but instead returns a logger


@chriddyp fixed with #214


TeamCity DNN JavaScript Libraries :: CI [Build 114](http://teamcity.engagesoftware.com/viewLog.html?buildId=392&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 114](http://teamcity.engagesoftware.com/viewLog.html?buildId=392&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:19


I understand, but since this is just a private method, it seems to me that we want to keep the name descriptive and refactor to `loadGroup` once the time comes for it to be more than just logging. It's a trivial rename that only affects the current file at that point. It will avoid confusing readers until then (who knows, we may never needs anything else besides logging).


thanks!!


Well all my texts that I've gotten from the tests are in a similar format to:
```
1 of 2
FRM: paxchecker@gmail
SUBJ: PAXChecker
MSG: blah blah blah
(Con't) 2 of 2
more blah blah(End)
```
I'm noticing in the last one I got the (End) is part of the URL, and thats probably why they're 404ing.


Thresholded or scaled? We know have frequencies between zero and one so would be pretty easy to scale them. I think this is going to be a major component of fine tuning. We don't know the exact distance measure they used to compare names, so even if we get the weights rights the actual value we're weighting could be way off.


wouldn't it be better to add the to gitignore?


I would say this is AT LEAST a `LogError` severity given a message has now been discarded.


If you're making `Assignable` semantically meaningful, you have to go all the way and require `Common<T, U>`.


As Brmbrmcar said, they are heavily modified. The originals are far different from those, so they are not the creation of the original authors but they are a creation of myself. It's called art proccessing. It's just my good will that I put there the names of original authors. Also note that many of the authors have no contact there, so noone is able to get any feedback from the original authors. And again, they are not just copied. Just compare them and you find that they are very different. If I just converted them, then you would be true, but I'm an artist! I agree that the red part isn't formulated well, but it's true. So there's no need for removing them. When is a level recreated using art processing, the holder of copyright rules is the restaurator, not the original author. The original author has still copyright rules to the original, but not to the restaured one. That's what is written in the red section. It's just my good will that I listed the original authors and I'm willing to apply any changes. Compare those levels and you'll see. After you see it, then say those things.


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 1.2.0-cloneHandle+1](http://build.prxm.it:8811/viewLog.html?buildId=4714&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:55


Also, @rjeczalik in what instance will this occur, and will discarding this message trigger a failure in the case of publishing?


I've changed this expression because
```javascript
if(callback)
```
just check for undefined or null, if callback isn't a function, will throw an error
With:
```javascript
if(typeof callback === 'function'
```
we strictly verify if callback isn't null, undefined and is a valid function!


Surely the rest of the settings in the default rule set are based on what you like? Why make the change here?


You should move that last ``if`` check up, just after ``real newTime = getCurrentTime() + timeElapsed;``.
Otherwise, if, newTime is strictly greater than the length, the frame shown is not the ending one.


Well it should either be `logLevel` and `logHandler`, see http://docs.ably.io/client-lib-development-guide/features/#TO3b, or it should (if a Logger interface is allowed), then be `Logger`


CDN will actually probably be faster. Since it's a file that's prolly on like 10% of all sites on the internet, it will most likely be cached in lots of browsers.


регистры шины порты объявляются в начале модуля.


I have very mixed feelings about this code.  It took me a while to verify all the magic numbers.  The comments are much appreciated.  I don't have any suggestions at the moment to make it less mysterious nor less error prone against remote future maintenance.  It would be better if details like this were right next to other places where these details exist (e.g. `Ledger::setRaw`).


Interesting, wonder if that's a carrier/phone determined thing, because the texts I'm receiving (TMobile/Android) aren't getting split at all.

Maybe I could move the link to the beginning of the message to try and limit the affect of this. I'd rather not remove the actual content of the tweet, as that helps us to identify false alarms a bit.


Cool, thanks!


it's not. so, if you remember back to the first pull request for speech, the purpose of `dontListen()` was to just stop the listener from listening by pressing the button again (otherwise it was unneccsary). speechListener stops listening for voice automatically, so if you call `sr.stopListening()` when it isn't listening you'll get an error.


1) Yes, my suggestion to return `Object` was not a well thought-out solution :( I guess what troubles me is that `V` is never checked at runtime, so it's only there to make the compiler happy. Everywhere you use a `Value` it's doing implicit casts. Hence my other comment about including a type token in the class and do a runtime check. I'll give it a try.
2) (closely related to the above) I think a dedicated error with an explicit message would be better. If the end user is getting CCE and their code is not doing any cast (as in my example), it looks like the bug is in the driver.
3) At first read it was not clear to me that the parent class handles setBytesUnsafe, but YMMV. Maybe a comment would help. I don't have a strong opinion about this.
4) I hadn't noticed that codecRegistry and protocolVersion came from the enclosing instance. Again, no strong opinion here.


Great work -- I have nothing to add functionally, it all works as expected. One thing I'll add just as a demonstration. Here is how you could have added the select `option`s using a `forEach` loop in place of a `for` loop:

```
$(document).ready (function(){
	var cities = [ "NYC", "SF", "LA", "ATX", "SYD" ];

	cities.forEach(function(cityName) {
		$('#city-type').append('<option value="' + cityName + '">' + cityName + '</option>');
	});
        ...
```

Let me know if you have any questions!

#FEWD Week #6 Project: Working with Arrays

---

###Description

The team from CitiPix has reached out for iteration on their prototype.  Instead of using a user's text input to change the background of the interface, the CitiPix user experience designers have asked to see how a drop-down menu would work in place of an input field.  The product managers are hopeful it will also lead to less error handling for their engineers and data team.  Use what you've learned this week about data-types and arrays and use the starter code - or your code from last week - and make the adjustments to your JavaScript file. Be sure to start out with Pseudocode.

Student: Selome Welday

|                                                                                |                                |                        |                          |
|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                                | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Create an array with the following values: "NYC", "SF", "LA", "ATX", "SYD"; use the array to add values to the `````` menu by using a ```for``` loop in JavaScript (don't update the HTML to do this!) |                                |                        |               X           |
| When the user changes the input of the drop-down, update the background image based on what they selected                                                                                |                                |                        |    X                      |
| Use $.append() in your iteration on the drop-down menu                                                                                |                                |                        |             X             |
| Use the ```$.attr()``` function to update html classes                                                                                |                                |                        |            X              |
| Get the value of user input using ```$.val()```                                                                                |                                |                        |          X                |
| Use the ```$.change``` event handler to capture user actions                                                                                |                                |                        |                X          |
| Use ```if/else if/else ``` conditionals to control the flow of your application                                                                                |                                |                        |            X              |
| Display your pseudocode as Javascript comments                                                                                |                                |                        |           X               |
| TOTAL: *16*/16                                                                                |                                |                        |                          |


@Hume2 Try and get it back.Do you have write access to the addon repository?


Why is the client library adding a connection ID to an incoming message?


апт


we already, have these 3 details in user model, `first_name`, `last_name`, and `email`, i don't think this the right way to do it, and `email` can't be `null=True`


Hi!

I think you could write this as:
```rust
tag!("bool") => { |_| ScalarType::Bool } |
```

Also, there is the [`switch!` combinator](https://github.com/Geal/nom/blob/master/src/macros.rs#L2285-L2288) if you need.


:beers:


Сигнал ready выставляется когда обмен завершен, что не совсем верно. так как сигнал ready должен выставляться после завершения обмена(CS в 1) + тайм аута между транзакциями. То есть когда модуль готов снова к транзакции


[Ask and you shall receive.](https://github.com/lpil/dogma/pull/166)


Perhaps it would be clearer to name these functions onGroupUnload and onGroupLoad? That should make the usage clearer.


Thats a valid point, but there is also a possibility that this is all it takes on the client side and it would save people from having to upgrade clients along with brokers, so I suggest we keep it.


@magnumripper  I did move the CPU_detect_*() function call to a later location within john_init.  This was due to where the john.conf file was loaded.  I am not 100% sure of what side effects the movement of the detection logic could have.  If this has to be kept very early, then we may be at a catch-22 state, of where the config requires the fall-back to happen first, and to avoid fall-back, we have to have the config loaded.

I have checked this in, and things 'run' fine, but I am not 100% sure how to check if the fall-back is working (I never build that way).

The ignore of the CPUID is pretty easy (and doing this with a [Local:Options] section on those VM's allows them to work properly once again).   As long as there are no negative side effects, this looks like a viable 'hack' while waiting for virtualbox to fix their crap.


thx :)


much cleaner :+1:


446000000 is 2014-02-17 4:53:20pm PST
Is this inconsistent with the comment?  The inconsistency leaves me wondering if the magic number 446000000 is incorrect.


@nsh87 I was just working on DOB stuff and I am wondering if I need to tweak my integration of weights. I think that I need to get the values that make up the OX-link table. Then get the weights and apply those to the table values as an additional part of the overall calculation (e.g. for birth_surname, we might get a value of 6 for S with a common surname, then use 2S for an exact match, then weight it by some frequency proportion of .01).


Hotfix to a hotfix.


Should this be in configuration?


Even though it looks like the list is intended to be "Maps less than or equal to this ID", it was only checking "Maps less than this ID" and seemed to be causing issues until I added the jr z command after the jr c. With that in place, maps seemed to be loading as intended.

Also, added some new entries to the IMAP table while I was fixing it to use constants like main Pokered, to account for the new areas. Inside the Seagallop Ferry and areas that aren't actually part of Kanto will show up as "Area Unknown" with the cursor on Vermilion City.


Did you run it? I had the variable isExploring set to false and still was getting 0.001 lose rate.

Jed
________________________________
From: tranchieuminh [notifications@github.com]
Sent: Sunday, November 15, 2015 11:46 PM
To: kochkov92/Tictactoe
Cc: Chou, Jed H
Subject: Re: [Tictactoe] Once qValues =loseScore, no longer change it. (53b61b2)

Hi,

That is because of the exploration.

If we turn exploration off, losing rate is 0.

CM

On Sun, Nov 15, 2015 at 6:08 PM, j-chou <notifications@github.com> wrote:

> Excellent. I think there's still something slightly wrong though. I
> trained on 25 million games with winScore = 10, loseScore = -500, drawScore
> = 0, learningRate = 0.7, discountFactor = 0.9, setExploring(false), and
> then played 50*10k games. The win rate is about 0.98 and the lose rate is
> from 10^-3 to 10^-4, so sometimes perfectLearner is still losing 10 in 10k
> games.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kochkov92/Tictactoe/commit/53b61b2126e2636d99197b72b8354a821def970e#commitcomment-14405451>
> .
>

—
Reply to this email directly or view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_kochkov92_Tictactoe_commit_53b61b2126e2636d99197b72b8354a821def970e-23commitcomment-2D14406914&d=BQMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=1KfiozGfx5LnNfrTY5zLztV2UAUBX_95zLkKySJMpUw&m=WYY49cI_OC22SO8Hs9LgJWJA3wBBU5kPma7vuDWhv74&s=CS2rRA0NQtYMX4GXPokYkOhEcHqTqgOEgkWVjRz5_Vs&e=>.


Just FYI, RuntimeInformation works down to net451 and works on mono


`canInterrupt`


could just put in `updateListeningIndicator()` here instead of `dontListen`. then, in line 159, do the same. then, get rid of `if (isListening)`. because without the if statement, you're just calling `dontListen()` to do a single method.
also, from the viewpoint of someone who doesn't know what's going on, it'd seem like `isListening=false`  then `dontListen()` would do the same thing (based on their names).


OK, instead of KFSPropertyConstants.CASH_RECEIPT, use CashReceipt.class.


Is the pipeline document anywhere?  If it is, would be nice to reference the docs here.


It's just a mock, but might want to change the message to phone instead of password match


@brmbrmcar Yes, I have permission, but I shouldn't do that. Anyway, it's not totally removed, but I don't want to spoil it here.


@bohdan-malets , thanks! Check out my last commits =)


Looks good, got all fields


Seems useful, no doubt about it. This is accepted.


:+1: , I need to add a jobs field to user as well


About commit message. Please write down particular problem that led you to decision to add const qualifier. This is the main reason of doing this patching. And also write down that PMC concept implies that it won't change vmas that it uses.


I suppose that for ultimate compatibility (at the expense of tying to the system name) we could follow the EV3 LED convention where system name is `ev3:left:red:ev3dev` which follows the pattern `platform:group:color:e3dev` - resulting in `brickpi:blue1:blue:ev3dev` - of course that's a breaking change in LED names.

@dlech - any comments?


All looks fine


@ShadowMine64 At the moment the width must be 16 and the height may be any multiple of 16.


nm, see it at the top of the file.


Just for my own understanding.. this is also a small bug fix?


All of these fit expected inputs and outputs


@enoter I'm sorry, but I still can't tell what you're asking me. What do you mean by "taking" all the parameters?


Same here


Should this be more specific?  Are there other ways to get to this code path if this is not a third party linked account?


Is this the right language here anymore?  "tests" is a bit loaded.


Should "return" here, so that user can see the error and reboot the server?


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


this one not correct


Looks good, but what exactly is "jobs". A string of ids delimited by whitespace/commas?


Sounds good to me.


Good call, just committed change


What is `${listunmount}` accomplishing here?  I think in practice it's empty.


As said in the line comment where I realize how horribly I messed this up, Lua prepends a stacktrace, so you need to compensate for that or you'll miss the error.
Apparently my compensation method meant pcall was essentially useless.


It's harmless, just to enable GetFileInformationByHandleEx and other functions which are not available on WinXP and older systems (it seems mingw targets old Windows by default).


Better to use `angular ui-router`


Why playing around?


Why not?


Seems a bit weird to have empty functions by default. Could use `undefined`.


Genial! Saludos y éxitos :)


Login, falta cerrar sesion


Wrong way to start application using jquery.
You need just include js file in the end of html


True dat


Better to extract filter for other file


A couple comments explaining the reason for the `tail -r` would probably be a good idea.  It's wasn't immediately obvious why you doing two reversals.


Hey! @prokaos súbete una captura que tengamos una referencia visual rápida en el repo.


Две строчки можно заменить одной если написать это через объединение. data_bus_out_reg <= {data_bus_out_reg[14:0],  ADC_Dout};


Yes, essentially.  It's been around for quite a while.  As long as the pair of flats for any given exposure time are well-matched, i.e., same illumination and same exposure time, (the correct implementation of) this step would reduce the noise somewhat, but the implied signal level wouldn't really be changed.  This bug was uncovered in analyzing a recent TS3 run where one of the flats in a pair genuinely had zero exposure time (presumably the slit did not open), so there was a factor of two disparity with the expected incident flux estimate.


After trying edit  for a day, I added the try-again loop. Nothing worse then changing 50 tags for half and hour to see all your changes get thrown away on a technicality :) If anyone knows a better solution.... feel free.


That's exactly right, @TommyByrne. I would use `background-size: cover` and then on a smaller resolution just center it using `background-position` (probably with a value of `center`).


Yeah, I found it on the Facebook article I think... :(


Recommended to use angular rails resources


oh ok, I understand.


On Mon, Nov 16, 2015 at 12:06:20AM -0800, Piotr Banaszkiewicz wrote:
> I decided that it was wrong, didn't comply with the filter's name
> (`unpublished`, not `unpublished_in_future`) and was
> counter-intuitive (published + unpublished should be equal to all
> events, but wasn't).

I agree with all of these points, I'm just suggesting that “ignore
past events” is probably also a useful filter.  If folks are turning
filters on and off directly, you can make “ignore past events” a
separate filter from published/unpublished.


Nice to allow a Slice to be made from an `array`.

Just to be picky, should the `char` and `unsigned char` template parameters be `std::int8_t` and `std::uint8_t`?  The underlying type of a `Slice` is `std::uint8_t`.  Although a `char` is 8 bits on all of our current platforms, a `char` is guaranteed to be *at* *least* 8 bits.  I've worked on a platform where a `char` was 32 bits.


Good catch. Should be an array of objects. Just fixed in a commit


Before:
![screen00100](https://cloud.githubusercontent.com/assets/109391/11189947/667589f2-8c93-11e5-9e03-db6019472752.png)

After:
![screen00102](https://cloud.githubusercontent.com/assets/109391/11189953/71c60a48-8c93-11e5-8a21-e748e9267341.png)

Note that the unit's diffuse color is replaced by 'shadow' to aid visual differentiation.


Just remove instead of commenting out.


Should ignore bower component


Oops, missed this in my clean up :)


All looks good


I'll remove this too, it was a note for myself


~/.config/haskell-vim-now/local seems to be a bit too much for this. Why not ~/.config/haskell-vim-now/ ? (the same for $XDG..)


ack


Is there some reason why we can't have the grid work in uncompensated skewed mode?


Looking great, just a couple nits...


@coreyfloyd to be clear, we _want_ these to expand vertically now? i was under the impression we wanted search results to have the same height


Maybe move thos above (line 143)? To stay closer to summ.Date = ....


If I have it as undefined, then I'll have to check if it exists when I call it later on, right? I figured it was clearer to have empty functions here because it acts as API documentation for future use of the component.


Mark, I suppose I will have to learn github soon, but for lack of time, I'll leave a comment here.

I appreciate your motivation in this post. But if developers are abusing tests to meet an artificial metric, that says more about the developer than the goal of code coverage. It's not fair to blame a tool when you misuse it. Allow me to present two counter-arguments as to why I think a high number (if not 100%) in unit test code coverage is a worthy goal.

1) If your unit tests adequately verify correct business behavior and you have code that isn't covered by unit tests, how can you justify the existence of the code not covered? Isn't it one of the tenets of TDD to only code what you absolutely need to meet the requirements?

2) if the there is code that you "need" that is untested, how can you be sure it is correct? One could argue that writing tests to make sure this "needed" code is verified is falling into the trap of testing the implementation. Which could be true, although there are many times I find that the some of the plumbing code of classes is necessary, and I unit test them even though they have no bearing on the desired behavior other than verifying that I've wired things up correctly (i.e. checking for valid objects in constructor injected arguments, etc.).


Maybe I'm overseeing something, but doesn't `database.debug: true`. Also output the SQL statements that are generated? If no, maybe it would make sense to reuse the setting that is already in our config file?


need to ignore idea files


нет необходимости тактировать прием от clk, лучше сделать прием по negedge SCLK. таким образом будем чуть меньше аппаратные затраты и меньше энергопотребление(sclk<clk)


The path is probably ".config/haskell.vim.now/local/haskell.vim.now/vimrc.local.pre'?


@Ephemera does this fix a bug? Also, shouldn't the selector be nested like the original so that it's only in effect if the Octotree sidebar is shown?


Looks good. I like this patch.
This patch should be the last in the patch set because in_parent = true isn't set until dirty bits tracking is implemented for SHMEM.


Good point. This bock of  code has been removed.


Looks like for `paths` it just ignores thouse parts that are unresolved. That is,

```Rust
fn main() {
    let x = 92;
    let z = foo::x;
}
```

Here the `foo::x` will be resolved to `let x`. Is this intentional?


Unneeded tab


I'm not a huge fan of the magic import pattern (I know we also use it in custom reports but I think it'd be better to be more explicit).

What if we just have something in settings that maps this by domain name? e.g.

```python
CUSTOM_LANDING_PAGE_URLS =  {
    'ews-ghana': 'dashboard_page'
}
```

That at least makes the dependency a bit clearer.


IMO if we're denormalizing here we should reflect that in the method name. also, do we even need to be denormalizing in the context you're calling this, what was the regression?


Этот мультиплексор  можно не ставить. неважно что будет находиться на  data_bus_out в промежуточные моменты времени. важно что на ней будет по отрицательному фронту сигнала ready и когда он равен 1.


lgtm


Why did you develop CRUD application?
Task was to build catalogue with pagination


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


good point that's a much better idea I'll do that!


Why the redundant `blue`? I would probably just say `brickpi:led1:blue:ev3dev` and `brickpi:led2:blue:ev3dev`.

Also, this is probably not relevant here, but just throwing it out there so you can see the big picture... For non-platform LEDs, such as those attached to output ports and WeDo ports, the convention is `port-name::ev3dev`. The port name could just be `outA` in the case of EV3 or `usb:1-1:1.0:wedo1` in the case of WeDo. Since the color cannot be known, it is left blank, hence the `::`.


@dbelcher - I updated the layout with line spacing and headlines to look more like the primer documentation.  It is much tighter.  It is getting closer.


what do you think of calling observe w/in the observe method?  extra unobserve is harmless, and making observations less error-prone is always a good thing.  maybe worthy of a best practice.


Works on my end.


@Darya713 хорошо, завтра ещё во время сдачи посмотрю


It would be great if there were a comment about why these particular values for the `array` were chosen.  You don't need to go through byte-by-byte, but at least a sentence about what the data represents would be helpful.


Not only for this. This dir is used for all user config files.


this makes them invasives, and breaks BC when your error handler converts them to exceptions. What about using something like the same convention than the Symfony Debug ErrorHandler to display silenced deprecations, but avoid breaking things for unsupported error handlers ?


That's not OK, we need to revert it. The CPU/OMP fallback chain needs to be super quick.

Perhaps do the same but as a build-time option?


Yup, my bad. Forgot to edit this one after making local_config_dir point to /local. Thank you!


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


You probably need an `elif` here.


Could you please elobarate what the method does in detail?
Ex returns false if this AITask cannot be aborted right now because it is in a critical state like some kind of cooldown? or not interrupted at all?

Whats wrong with the previous/current name?


Ofc. Thanks! It was residue of experiments.


@elahikai just made a PR to the main repo at https://github.com/murphydanger/gulp-minify-css/pull/126
Hopefully @murphydanger will be able to have a look at it soon.


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


`Common` is not my favorite opt-in mechanism, as I'm sure you've noticed, but I can live with it being here just as it is in `Swappable` until something better comes along.


trying to merge with configurator branch ( if you like the long story , if not use the scrollbar ah ah ah ... )

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator)
$ git checkout master
Switched to branch 'master'
Your branch is up-to-date with 'origin/master'.

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (master)
$ git pull
remote: Counting objects: 46, done.
remote: Compressing objects: 100% (46/46), done.
remote: Total 46 (delta 4), reused 0 (delta 0), pack-reused 0
Unpacking objects: 100% (46/46), done.
From https://github.com/elieserdejesus/JamTaba
   24ee5a2..9556049  master     -> origin/master
Updating 24ee5a2..9556049
Fast-forward
 Client/Jamtaba-common.pri                          |  17 +-
 Client/src/audio/core/AudioNode.cpp                |  11 +
 Client/src/audio/core/AudioNode.h                  |   6 +-
 Client/src/audio/vst/VstPlugin.cpp                 |   7 +
 Client/src/audio/vst/VstPlugin.h                   |   1 +
 Client/src/gui/BpiUtils.cpp                        |  22 ++
 Client/src/gui/BpiUtils.h                          |  15 ++
 Client/src/gui/ChatPanel.cpp                       |  45 +++-
 Client/src/gui/ChatPanel.h                         |   5 +-
 Client/src/gui/FancyProgressDisplay.cpp            | 136 -----------
 Client/src/gui/FancyProgressDisplay.h              |  40 ----
 Client/src/gui/IntervalProgressDisplay.cpp         |  20 +-
 Client/src/gui/IntervalProgressDisplay.h           |   2 +-
 Client/src/gui/LocalTrackView.cpp                  |   8 +-
 Client/src/gui/NinjamPanel.cpp                     |  14 +-
 Client/src/gui/NinjamPanel.ui                      |   2 +-
 Client/src/gui/NinjamRoomWindow.cpp                | 132 +++++++----
 Client/src/gui/NinjamRoomWindow.h                  |  14 +-
 Client/src/gui/chords/ChordLabel.cpp               |  65 ++++++
 Client/src/gui/chords/ChordLabel.h                 |  25 ++
 Client/src/gui/chords/ChordProgression.cpp         | 258 +++++++++++++++++++++
 Client/src/gui/chords/ChordProgression.h           | 113 +++++++++
 Client/src/gui/chords/ChordsPanel.cpp              |  76 ++++++
 Client/src/gui/chords/ChordsPanel.h                |  37 +++
 Client/src/gui/chords/ChordsPanel.ui               | 115 +++++++++
 Client/src/gui/chords/ChordsWidget.cpp             |  50 ++++
 Client/src/gui/chords/ChordsWidget.h               |  31 +++
 Client/src/main.cpp                                |   1 +
 .../src/performance/WindowsPerformanceMonitor.cpp  |  44 ++--
 Client/src/resources/chat_small.png                | Bin 0 -> 563 bytes
 Client/src/resources/jamtaba.css                   |  39 ++++
 Client/src/resources/jamtaba.qrc                   |   1 +
 32 files changed, 1073 insertions(+), 279 deletions(-)
 create mode 100644 Client/src/gui/BpiUtils.cpp
 create mode 100644 Client/src/gui/BpiUtils.h
 delete mode 100644 Client/src/gui/FancyProgressDisplay.cpp
 delete mode 100644 Client/src/gui/FancyProgressDisplay.h
 create mode 100644 Client/src/gui/chords/ChordLabel.cpp
 create mode 100644 Client/src/gui/chords/ChordLabel.h
 create mode 100644 Client/src/gui/chords/ChordProgression.cpp
 create mode 100644 Client/src/gui/chords/ChordProgression.h
 create mode 100644 Client/src/gui/chords/ChordsPanel.cpp
 create mode 100644 Client/src/gui/chords/ChordsPanel.h
 create mode 100644 Client/src/gui/chords/ChordsPanel.ui
 create mode 100644 Client/src/gui/chords/ChordsWidget.cpp
 create mode 100644 Client/src/gui/chords/ChordsWidget.h
 create mode 100644 Client/src/resources/chat_small.png

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (master)
$ git checkout configurator
Switched to branch 'configurator'
Your branch is up-to-date with 'origin/configurator'.

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator)
$ git merge master
Auto-merging Client/src/main.cpp
Removing Client/src/gui/FancyProgressDisplay.h
Removing Client/src/gui/FancyProgressDisplay.cpp
Auto-merging Client/Jamtaba-common.pri
CONFLICT (content): Merge conflict in Client/Jamtaba-common.pri
Automatic merge failed; fix conflicts and then commit the result.

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator|MERGING)
$ git add .

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator|MERGING)
$ git status
On branch configurator
Your branch is up-to-date with 'origin/configurator'.
All conflicts fixed but you are still merging.
  (use "git commit" to conclude merge)

Changes to be committed:

        deleted:    Jamtaba-common.pri
        modified:   src/audio/core/AudioNode.cpp
        modified:   src/audio/core/AudioNode.h
        modified:   src/audio/vst/VstPlugin.cpp
        modified:   src/audio/vst/VstPlugin.h
        new file:   src/gui/BpiUtils.cpp
        new file:   src/gui/BpiUtils.h
        modified:   src/gui/ChatPanel.cpp
        modified:   src/gui/ChatPanel.h
        deleted:    src/gui/FancyProgressDisplay.cpp
        deleted:    src/gui/FancyProgressDisplay.h
        modified:   src/gui/IntervalProgressDisplay.cpp
        modified:   src/gui/IntervalProgressDisplay.h
        modified:   src/gui/LocalTrackView.cpp
        modified:   src/gui/NinjamPanel.cpp
        modified:   src/gui/NinjamPanel.ui
        modified:   src/gui/NinjamRoomWindow.cpp
        modified:   src/gui/NinjamRoomWindow.h
        new file:   src/gui/chords/ChordLabel.cpp
        new file:   src/gui/chords/ChordLabel.h
        new file:   src/gui/chords/ChordProgression.cpp
        new file:   src/gui/chords/ChordProgression.h
        new file:   src/gui/chords/ChordsPanel.cpp
        new file:   src/gui/chords/ChordsPanel.h
        new file:   src/gui/chords/ChordsPanel.ui
        new file:   src/gui/chords/ChordsWidget.cpp
        new file:   src/gui/chords/ChordsWidget.h
        modified:   src/main.cpp
        modified:   src/performance/WindowsPerformanceMonitor.cpp
        new file:   src/resources/chat_small.png
        modified:   src/resources/jamtaba.css
        modified:   src/resources/jamtaba.qrc

Untracked files:
  (use "git add <file>..." to include in what will be committed)

        ../DOC/
        ../Icons/jpg/Thumbs.db
        ../Icons/png/Thumbs.db
        ../build-Jamtaba-Desktop_Qt_5_5_1_MinGW_32bit-Debug/
        ../build-Jamtaba-Desktop_Qt_5_5_1_MinGW_32bit-Release/
        ../logo.blend
        ../logo.blend1
        ../logo.blend2
        ../logoAni.blend
        ../logoDown.blend
        ../logoDown.blend1
        ../logoDown.blend2

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator|MERGING)
$ git commit -m" trying fix merge "
[configurator a59be19]  trying fix merge

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator)
$ git fetch master
fatal: 'master' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator)
$ git merge master
Already up-to-date.

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator)
$ git status
On branch configurator
Your branch is ahead of 'origin/configurator' by 3 commits.
  (use "git push" to publish your local commits)
Untracked files:
  (use "git add <file>..." to include in what will be committed)

        Jamtaba-common.pri
        ../DOC/
        ../Icons/jpg/Thumbs.db
        ../Icons/png/Thumbs.db
        ../build-Jamtaba-Desktop_Qt_5_5_1_MinGW_32bit-Debug/
        ../build-Jamtaba-Desktop_Qt_5_5_1_MinGW_32bit-Release/
        ../logo.blend
        ../logo.blend1
        ../logo.blend2
        ../logoAni.blend
        ../logoDown.blend
        ../logoDown.blend1
        ../logoDown.blend2

nothing added to commit but untracked files present (use "git add" to track)

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator)
$ git merge master
Already up-to-date.

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator)
$ git status
On branch configurator
Your branch is ahead of 'origin/configurator' by 3 commits.
  (use "git push" to publish your local commits)
Untracked files:
  (use "git add <file>..." to include in what will be committed)

        Jamtaba-common.pri
        ../DOC/
        ../Icons/jpg/Thumbs.db
        ../Icons/png/Thumbs.db
        ../build-Jamtaba-Desktop_Qt_5_5_1_MinGW_32bit-Debug/
        ../build-Jamtaba-Desktop_Qt_5_5_1_MinGW_32bit-Release/
        ../logo.blend
        ../logo.blend1
        ../logo.blend2
        ../logoAni.blend
        ../logoDown.blend
        ../logoDown.blend1
        ../logoDown.blend2

nothing added to commit but untracked files present (use "git add" to track)

Salon@salon-d59g9e2l8 MINGW32 /e/JamTaba/Client (configurator)
$
.


It would be probably good to force object field initialization on the class definition, since C++11 has that nice touch.
In other words:
```
uint32 healthBeforeDuel = 0;
```
in https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/server/game/Entities/Player/Player.h#L2551


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


Saretioooooooon


Done.


необходимо сделать параметризованный делитель частоты из clk в SCLK. причем параметр деления частоты должен быть задан в шапке модуля. это удобно. например clk равна 100 Мгц а SCLK = 10 Мгц. Поменяв только один параметр при объявлении модуля мы можем сразу его использовать без измения описания внутри.


Great idea! Could you please add app.yaml to .gitignore and add to this PR?


Ok, can you make your pull request again without that? Thanks!


cossa vuto bueo


It was a placeholder until I had real unit tests, now that there are consequential ones, I removed it.


I'd love to see this be kept as an option if possible. I currently cannot use the gecode resolver without it locking up. Keeping this to a "if you have 'dep_selector' present then we will use gecode" option would be better if that could be done.


You must keep the span tags. There should be a better way to do it in the ruby code, like first putting %{enterprise} into a variable, but I have no idea.


You would have to do a check later, but if you ever need to know that these props are actually undefined it becomes harder because they'll still have type `function` even though they're undefined in practice.

If you want to document the API, it's better to use `propTypes` to specify the schema. It will give you warnings in the console if the wrong types are passed in. You can also annotate your prop expectations there w. comments.

More info: [https://facebook.github.io/react/docs/reusable-components.html](https://facebook.github.io/react/docs/reusable-components.html)


"How is it possible to indicate which tasks they support?"

I guess I have this question too. I was looking at concepts and references as relevant, standalone topics. But it seems like there should be a way to link them to related tasks so the user even knows the topic exists. Is there a way to set up DITA so every time you use the word, say, GitHub it links to the GitHub concept topic?

 Why use these instead of just multiple ditamaps?

I believe the book describes using multiple ditamaps when a certain chunk of content is updated frequently (so those frequently-updated chunks are segregated). It also helps to keep the size of the ditamap human-readable. We probably won't get a file that big in our work in this class, but if we were maintaining 500+ page manuals, I can see that being a mess to have to edit. You wouldn't be able to find anything easily.


@ssatorres me lembre de remover as keys quando for passar pra master.


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


The `continue` above makes them equivalent -- this block will never be entered if the above condition was true.


no, i meant this:
```python
con = sqlite3.connect('mynote.db')
cur = con.cursor()
cur.execute("CREATE ...")
```


Hmmm? I had a look at `generate-html.py` today, and I wonder if this wouldn't be better handled by changing
`details.append(strings['uses_n_gpio_pins'].format(uses))`
to
`details.append('* ' + strings['uses_n_gpio_pins'].format(uses))`
?
(i.e. just like `details.append('* ' + strings['uses_i2c'])` a few lines further down)

Give me the nod @Gadgetoid and I'll knock up a PR :)


4665_hpwh_energybal (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1432 of 1432 tests passed)

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


Oh right, my brain was in fortran mode apparently.


Need to extend commit description.
Add commit title tag (something:) to this patch. Take a look at git log and you'll find examples.
Change commit title to 'tag: add shared memory deduplication to manual criu dedup command';
Tell the following in description:
Why you've merged pids and shmem ids into single variable.
Proof that we can't occasionaly use shared mem id instead of pid and vice versa. Show that we have enough information provided in flags to treat ids right.
Proof that this change won't introduce any backward incompatibilities.


02865d5687b315b387a6ab737bd2aad636d7a94f
919e9b2a5aad780a15233e344fbcf993230474e5


1. Yes. we can do `fail`.
2. We can reboot the server from here by the command.

Well, I have doubts that this will happen. Volume is not mounted on this phase, so we don't need reboot after a corrections. But for sure we can implement (1) or (2).  Thanks.


Thanks! Sorry this is getting merged so late. I missed the notification.


I'm actually a little surprised escaping new lines in a string passed lint.


@LMH0013 I'll reimplement it, but with an addition in the intro level:

It is legally a derived copy, so while the original author does legally still own the idea, the resulting level is so different that it is legally classed as separate work, making all of this perfectly legal.


``user.is_active = True``


Change the title


Is there a reason to change this test ?


But if a build time, that means that it is some non-standard ./configure call.   This change is a do it once (john-local.conf), and forget it.

I guess it 'could' be an option vs a config.  That would not be so bad. We could even change the warning message to list the option.

Thoughts ....... ?


Unfortunately, it's not quite symmetric, but see https://github.com/killbill/killbill-platform/commit/0aa8abf7d42362ff23582ad0325853e62c08a45d.


:) Please don't change the number of spaces in the files. We've been operating on 2 space indentation in our files.


You should extract application templates to other files.
One of the most important things – made your code easy to maintain


Please add a comment here, your code intent is not clear.


Scratch that also.  opt_init() is called right before the conf_init()  :(


Since we are considering breaking changes, we could just create a map between system led name and led instance. Then, in python for example, we could access individual leds as `Leds.led['brickpi:led1:blue:ev3dev']`, or `Leds.led['usb:1-1:1.0:wedo1']`. The map/dict can be be filled at runtime, and we can still provide convenient grouping in `spec.json`.


Yeah, so that we can see failures on the server logs (which doesn't have the awesome debugging tools of the browser).


You should ignore bower component
Use gem `rails bower`


I think this is the most important things to note in your users needs. Your presentation framework may not be suitable if they're giving a presentation outdoors, or in a place with no Wi-fi, or no projector. Those will have to be thought out by the user before they start.


les prochaines en .jpg si possible... sinon jte victimise :3


Forget calling anything outside john.c


Try this:

```php
add_filter( 'reveal-js-slide-defaults', function( $retval ) {
	$retval['use-title'] = true;
	return $retval;
} );
```


`return` means we continue booting but ignore the resize phase. Do we need raise an exception?


yes, I did it mostly because @LizardM4 had fixed it that way in the newly submitted IT files, but alternative welcome.


Does "cordova build ios" still create the IPA even for cordova-ios versions prior to 3.9.0? I thought the answer was no.


What is being starved from staying inside of processInbound too long?


I see no point in having separate DOP and SOP definitions.
You can distinguish which is which because DOP are only used in _mca_dbin and SOP are only used in _mca_sbin.
Moreover this would allow factoring the switch-cases with a clean macro.


You've added a typo here by deleting a parenthesis


Shared credentials should still work, but the profile name would not be configurable. The default profile name should be `"default"`.


This sentence was correct before, I don't think this change makes it more readable


There is some problem with vertical movement. Logic is correct but somehow code is not working.
Will try again in some time.


Please call pmc_fini() on the newly created pmc_t


I sent you a patch for testing


Might be slightly cleaner to guard on this condition at the top of the function and return early, rather than create an `alert` that goes unused here.


Hmm.  That sux.   Is there a 'real' reason for higher arch usage on files where it will make NO difference, since they are simply setup/shutdown code ???


Why is this moved up?


The redundant `blue` is there because I don't always think before I type (or talk for that matter).

I vote for `brickpi:led1:blue:ev3dev` and `brickpi:led2:blue:ev3dev`.


Why not?
pmc_t pmc = PMC_INIT;


There's also the performance impact I mentioned. We are not opening and parsing files, just forget that.


return 2


return 3


return 4


Ezee, please don't merge the configurator branch yet.
I'm looking at the code right now.
The code not work with msvc compiler (see the image). If I change to mingw the code works ok.
![image](https://cloud.githubusercontent.com/assets/1012741/11190727/601de3f2-8c7e-11e5-9064-d22fedad4910.png)


I have not fully reviewed this code, but have left some comments. I need some more time to go over the IEEE details.


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


Saw it late. Fixed in the next commit.


Yep. I ran it. Strange.

CM

On Mon, Nov 16, 2015 at 9:44 AM, j-chou <notifications@github.com> wrote:

> Did you run it? I had the variable isExploring set to false and still was
> getting 0.001 lose rate.
>
> Jed
> ________________________________
> From: tranchieuminh [notifications@github.com]
> Sent: Sunday, November 15, 2015 11:46 PM
> To: kochkov92/Tictactoe
> Cc: Chou, Jed H
> Subject: Re: [Tictactoe] Once qValues =loseScore, no longer change it.
> (53b61b2)
>
> Hi,
>
> That is because of the exploration.
>
> If we turn exploration off, losing rate is 0.
>
> CM
>
> On Sun, Nov 15, 2015 at 6:08 PM, j-chou <notifications@github.com> wrote:
>
> > Excellent. I think there's still something slightly wrong though. I
> > trained on 25 million games with winScore = 10, loseScore = -500,
> drawScore
> > = 0, learningRate = 0.7, discountFactor = 0.9, setExploring(false), and
> > then played 50*10k games. The win rate is about 0.98 and the lose rate is
> > from 10^-3 to 10^-4, so sometimes perfectLearner is still losing 10 in
> 10k
> > games.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/kochkov92/Tictactoe/commit/53b61b2126e2636d99197b72b8354a821def970e#commitcomment-14405451
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub<
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_kochkov92_Tictactoe_commit_53b61b2126e2636d99197b72b8354a821def970e-23commitcomment-2D14406914&d=BQMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=1KfiozGfx5LnNfrTY5zLztV2UAUBX_95zLkKySJMpUw&m=WYY49cI_OC22SO8Hs9LgJWJA3wBBU5kPma7vuDWhv74&s=CS2rRA0NQtYMX4GXPokYkOhEcHqTqgOEgkWVjRz5_Vs&e=
> >.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kochkov92/Tictactoe/commit/53b61b2126e2636d99197b72b8354a821def970e#commitcomment-14421727>
> .
>


Nit: Having booleans about not doing something can be confusing (e.g., `false` means "do not not restore"). What do you think about `skipSessionRestore`, or something similar?


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


What I would suggest is to go over the set and truly make it your own. Right now it looks like you took other's levels and claimed ownership, which will cause issue. If the levels are completely rebuilt like you say, then introduce them as "inspired" by levels on the old fan site, but of your own design. Run them by people on IRC to make sure that is fine (they know licensing better than I do). If things are acceptable, we'll get it back up. Feel free to acknowledge the original authors, but please leave out any remarks that might drive them away. I was actually about to post a tweak to the intro when this came up, so I'd rather see more levels. Just check with others first. Usually addons are quality checked before posting, let the normal process happen first.

		 ---- On Mon, 16 Nov 2015 07:45:36 -1000  notifications@github.com  wrote ----@brmbrmcar Yes, I have permission, but I shouldn't do that. Anyway, it's not totally removed, but I don't want to spoil it here.
—Reply to this email directly or view it on GitHub.


We should raise exception except for code 0 and 1 and not allow chef-client to continue. Thank you.


@JorjMcKie, many thanks for the works towards the new release.

One simple question: do you really think that compressing already compressed files (even if the gain is noticeable, as in this case) makes sense?

Sorry, but the user will have to uncompress those files. In my personal opinion, this compression should be avoided.


Same suggestion here about naming.


Cool. This fix should be moved here (in the final patch set). We'll discuss general considerations about patch sets for open source projects on next skype meeting.


Do we use the direct map between the string name and the led instance, or do we convert the names to identifiers like `brickpi_led1_blue_ev3dev`?


`referencedLayers` was used to keep track of which layers need to be released when this function returned. With these changes, it's unused and the layers don't get released.

You should remove `referencedLayers`, and add `defer layer.ReleaseAndLog(p.config.LayerStore, l)` here and in the other place where `referencedLayers` was appended to.


"In what kinds of situations would I want to make an attribute of a ‘topicref’ be ‘toc= no’?"

I think if you had content you only wanted available alongside the task, this would be useful.

Lets say you had a section about a special setting on a chainsaw that puts the chainsaw into Juggle Mode. Juggle Mode is for experts only. We don't want the average tree-sawing Joe to stumble across Juggle Mode in the table of contents, think that sounds cool and hurt himself. The information about Juggle Mode is only available under the 'How to Be a Chainsaw Master' task, to prevent Joe from seeing it before he's ready. We want to make sure Joe knows Juggle Mode is only for Chainsaw Masters, and that's one way to enforce that.


Why?


Looks like you forgot to push latest master to your github repository!


Я забыл учесть смещение на 20 пикселей при расчете вертикальной координаты для прямоугольника.


Yes, as far as I know we're still referring to them as tests.


testing, still testing


Completed the exercise about capitalizing the first letter of each word


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5306](https://teamcity.mesosphere.io/viewLog.html?buildId=76196&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:11


Also CRIU people may ask you to init this cache with the largest process VMA size only if anonymous shared memory is present and use max anon shmem vma size as cache size. But lets skip this optimization for now. We have no prof that this optimization may improve perf in real scenarios. But it complicates code with 100% probability :)


I see in your git history a non resolved CONFLICT in Jamtaba-common.pri. Be carefull with this, if you ignore the conflict and just add your local changes you are replacing my changes.

To solve conflics I'm using [Meld](http://stackoverflow.com/questions/11133290/git-merging-using-meld), so I can mix my changes with your changes, and no work is loosed IF I merge correctly. This merge stuff is a manual process, so unfortunally is easy erase something and break the code.


I have a scenario where I'm sending 300 messages in one batch which causes the entire event loop to be busy for ~100ms which is way to long for my application.


I sent it to your @openwall address. Apply it with `git am`.


Because at least one of those images is created when daemon starts.


As discussed offline, seems like gitignore won't work for tracked files... Bummer :(


Let's start a conversation on that issue and clearly justify why we are doing this. I am more than happy to add PMD, but with good reason. You pointed out a few things that you wanted to check for using PMD -- tell us why they will be useful, why no other tool can support them for now ... that way @tariq1890 can also make his case for suggesting PMD in the first place.


Good job.

One little suggestion:

When we made a move, I think there should be some preview effect before we click on the submit button to tell the user about the move going to be applied.


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


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


@autrejacoupa , FYI.  Thanks.


@skeptical @valib
I need some clarification here. I've loaded this up on my Ubuntu VM, but I'm not sure what to test. What is Unix specific?

I agree that it's unnecessary and probably quite expensive to launch two threads for each process. I can see the need though as we have to consume both the "normal" and the "error" stream. Can't these be redirected/merged into one, and we can skip the whole thread creation?


I think having an identifier for the well known platform LEDs would be a nice convenience. If would be impossible to have identifiers for other LEDs though.

Also, all LEDs will end with `:ev3dev`, so that should not be part of the identifier name. Any led class device that does not end in `:ev3dev` should be ignored (you will not have permission to use it). This should be part of the spec.


This extent got split up from is physfacet -- it looks like this happens anytime there is an existing physdesc with both an extent and a physfacet


Of course , i had no idea to merge it myself .
If i really wanted , i would have send before a pull request .
The rule is : YOU ONLY HAS RIGHT TO MERGE BRANCHES TO TRUNK .
That way , trunk ( master ) is always under your control/reponsability .
Good ?

For the msvc compiler , try to change 'type' by 'tape' ?
If APPTYPE is a problem , try a int instead ( 0 is standalone , 1 plugin )


The version that is published to nuget is 1.7. I've bumped up this one to mirror Authorization version, but since you've changed that package version to 2.0 then 2.0.0 is good for this one as well.


нужно использовать instanceof Array, в javascript почти все имеет тип object


Not sure htmlescape is used in Mini otherwise it's Blackberry build will be broken...

But you will suggest to keep that change in its own repo I guess.


Done.


см. комментарий ниже


> Any led class device that does not end in :ev3dev should be ignored

Even 'usb:1-1:1.0:wedo1'?


Changes look OK, but looks like you have the wrong bug number in your commit (should be 1223098).


Yes, the full name of that device will be 'usb:1-1:1.0:wedo1::ev3dev`


I would prefer more descriptive name for weakly typed vma pagemap parameter of interface function add_shmem_area(). Something like vma_pagemap. But it is not ok to break u64 *map naming used in CRIU. So let's leave it as is.


Maybe `TmpFileManager` or something would be a better name than `Disk`? For all you know, it's an SSD, not a disk :P


And the numbers will be different depending on if you have a USB hub and things like that.


DITA markup and HTML come from the same family called XML. XML is an organization method for information, so that things can be labeled as a certain type of information and assigned characteristics.

HTML uses this organization and makes a webpage. DITA uses this organization and makes a document. So DITA isn't super HTML so much as HTML's sister.


Trabalho da Disciplina Redes Móveis


`import os.path`. importing pure `os` should be avoided if possible.


I don't think this is right. `rootFS` already has all the DiffIDs, from the previous loop. What you really need to do is store an index into `rootFS.DiffIDs` so you can slice it, or store the DiffIDs so far, like the old code did.

The change to a download manager will clean this up a lot.


Agree about the merge rule.

I change the code here to
```
APPTYPE appType= APPTYPE::standalone;
```
and the problem is solved. Two variables "type" in the same file, msvc complain about this but mingw not.

Now I have another error:
```
E:\Jamtaba2\jamtaba2\Client\src\persistence\Settings.cpp:546: error: C4716: 'Persistence::Settings::writeFile' : must return a value
```

writeFile is not return a value?


Should be merged into "Add structure of files" commit.


I am getting:

TypeError: $(...)[0] is undefined messenger.js:412:0

Is that supposed to be '#footbox' instead of 'footbox'? I.e.

$('#footbox')[0].datset.dirty = false;


4665_hpwh_energybal (nmerket) - i386-Windows-7-VisualStudio-12: OK (1957 of 1968 tests passed)

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


so, to summarize, is this what should be done?
```
'brickpi:led1:blue:ev3dev' -> brickpi_led1_blue # named instance
'ev3:left:red:ev3dev' -> ev3_left_red # named instance, breaks current api
...

'usb:1-1:1.0:wedo1::ev3dev' -> led('usb:1-1:1.0:wedo1::ev3dev') # Created by user request or obtained from enumeration


With this `const` data member, is it worth explicitly deleting the assignment operator?


Actually, I think this works.


lgtm


`from generator import generate, generator`. Multi-line import statements for two imports is a bit of an overkill.


Repalced request.data['reset_string'] with user.userinfo.reset_string.


Та просто я не знав, яе краще описати зміни - писати точно, що змінюєш? там
ж і так видно, що я змінював

2015-11-16 19:24 GMT+02:00 LynxUA <notifications@github.com>:

> Андрію, в тебе "some" - улюблене слово? Воно ж нічого не відображає.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Baratheons/LMS/commit/e46968c23e8bed60a3550249a558c23880d30b58#commitcomment-14421224>
> .
>

--
З найкращими побажаннями,
Андрій Дигдало.


It won't work on windows. They have backward slashes there, like this: `\`. That is why one should use `os.path.join` instead of providing the slashes manually.


@allenwb should this link over to https://github.com/wycats/javascript-private-state instead of the Gist?


Island 2 is on its way.


Thanks for working on this, John. Did you intend to leave the blue background?


"~@" is an exact match due to the fact that both "~" and "@" are also themselves tokens. So due to pcre being greedy by default, this will match "~@" as a single token instead of two separate tokens. The "(?:\\.|[^\\"])*" section is working around the greedy nature of regex. It basically says capture everything after a double-quote except for backslashes and double-quotes, but if you encounter a backslash following by another character (double-quote or otherwise) then capture those two characters too. It appears a bit redundant at first but consider if you have two separate string tokens together like "abc\"def" "ghi", you need some way to prevent greedy capture of both string together as a single token (abc\"def" "ghi), but you also don't want to just capture just up to the first double-quote either (abc\). So that was the best simple matcher I came up with that addressed both cases.


Hmm no, that doesn't seem to fix it either...


What does your message callback do? What other things do you want your application to be responding to? Can you process the messages in clustered mode? What specific things to you want to interweave into the message processing?


i wanted to write this part the short way using ?: , but i think it's better to optimize when everything works
and is well understoood .

**writeFile is not return a value?**
Ooops ... plase fix , check the equivalent readFile to see how should works .....i forgot the last return false ;

I thing i've seen the warning in the compiler , but that worked so i forgot to finish ..


I would also caution against SVG for everything. I use SVG a lot, but if I've got a plot with a large number of points, SVG is going to make some really big files that no-one wants to work with (like 100 MB files).


Between your Who Performs This Task answer and this one, I think your view of your audience is too narrow.

Many people do personal website design in their free time. A lot of people have blogs, portfolios or other websites. In these cases, the audience won't be filled with professional developers, or in an office setting. I would say your audience is closer to anyone with a website they can customize.


Its not.
On Mon, 16 Nov 2015 at 21:00, Aleksey Kladov <notifications@github.com>
wrote:

> Looks like for paths it just ignores thouse parts that are unresolved.
> That is,
>
> fn main() {
>     let x = 92;
>     let z = foo::x;
> }
>
> Here the foo::x will be resolved to let x. Is this intentional?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alexeykudinkin/intellij-rust/commit/0d030f4e1f41dd18a258d1a5dc01a112c6ea847a#commitcomment-14422080>
> .
>


Btw, @rhempel, what do we do with rhempel/ev3dev-lang-python#90?


Should probably be

    list->tid = ss.str();


Thanks for catching that.  Fixed in the next commit.

On 11/16/2015 12:36 PM, A. Breust wrote:
>
> You should move that last |if| check up, just after |real newTime =
> getCurrentTime() + timeElapsed;|.
> Otherwise, if, newTime is strictly greater than the length, the frame
> shown is not the ending one.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lucidspriter/SpriterPlusPlus/commit/c07844d2fda87f56eaa0bd0c80fbb6253dbc1c7e#commitcomment-14421537>.
>


@rogeralsing any shot that this can create weird / crazy recursive issues?


There are good practices to sorting imports in python. Namely imports should be grouped into stdlib, third party, framework and package groups and sorted within each group. See this: https://github.com/timothycrosley/isort


no problem, fixed here and working for standalone. A software called "Jomtobo" create some folders and files in my machine, ha ha
![image](https://cloud.githubusercontent.com/assets/1012741/11191247/85affbd4-8c81-11e5-9d21-1e643b93ffa5.png)

Testing the plugin ...


Trabalho de conclusão da matéria Laboratório de Programação Web


Please clean up commented-out code.


I was also wondering when I did it and I thought I will figure out later.
But I can change it immediatly.


I don't really agree this is a problem. It's a choice people make to
convert everything or not to exceptions. Like we don't convert deprecation
warnings. I don't want to add hacks just because someone might not have
made a conscious decision about this. It's low level enough that it doesn't
affect most people.


I think this change is breaking injectors - for stereo the frame size in usecs is the same, just more data


We could also do this: http://trusktr.io:7777/inomuhuseg


delete


This function returns the largest number of an array


d


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


Hi! I'm glad this is helping you out. Thats the main reason why I started uploading code that I write when learning.

I can't remember if the human_detector.pkl was trained with all the data or just a subset. I think I would just recommend that you convert all the pngs in the INRIA dataset to the new png version. I did this with my dataset. If you have a Mac, it's very easy to do:

Open Preview --> File -> export selected images -> choose a new folder and save (make sure you aren't replacing the files. This made my computer freeze up)

If you don't want to do that, you could download the .pkl.

> On Nov 10, 2015, at 8:33 AM, Gunshi Gupta <notifications@github.com> wrote:
>
> Hi! i'm a beginner in the field of computer vision and i found this repo very helpful since i wanted to get a working human detector using the webcam.
> Just one request though, the opencv version i have uses a libpng16 and for this trainer to work we need opencv compiled with libpng12, since the india dataset uses the older png version images.I'd do this, however i need opencv to keep working with the newer pngs alongside.
> i just wanted to confirm if i could download the cpickle(human_detector.pkl) file you'd uploaded and use it like one would, after training had completed, and if it has the correct training results?
> Thank you!
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/ironvietman/detect_human/commit/2aef066c7c794764a97339ce85b77f6ed52960a5#commitcomment-14305478>.
>


won't work on windows. Please `os.path.join`, not hardcoded forward slashes.


why was lentic added? :O is there a more in depth tutorial to use this package?


this line change is not required i think.


Thefre is 2 cases for the bit-1 position when it becomes subnormal power of 2. Either it is between bits 48 and 64 (hx) or between 64 and 128 (lx) if it is higher than 128, it will automatically becomes a zero. But I can also add a test for it if you think it will be more clear.


There's no point in leaving this here at all. Remove it.


And, @dlech, are system names for brickpi leds still `brickpi1:blue:ev3dev`? Because what we had above is not in the current spec.


" APPTYPE appType= APPTYPE::standalone; "
Standing ovation !! that is ART !!! lol
very nice trick !
ezee
--------------------------------------------
En date de : Lun 16.11.15, Elieser Ademir de Jesus <notifications@github.com> a écrit :

 Objet: Re: [JamTaba] chord progression display finished (9556049)
 À: "elieserdejesus/JamTaba" <JamTaba@noreply.github.com>
 Cc: "Ezee" <h2oreggae@yahoo.fr>
 Date: Lundi 16 novembre 2015, 19h37

 Agree about the merge rule.

 I change the code here to

 APPTYPE appType= APPTYPE::standalone;

 and the problem is solved. Two variables "type"
 in the same file, msvc complain about this but mingw
 not.

 Now I have another error:

 E:\Jamtaba2\jamtaba2\Client\src\persistence\Settings.cpp:546:
 error: C4716: 'Persistence::Settings::writeFile' :
 must return a value

 writeFile is not return a value?

 —
 Reply to this email directly or view
 it on GitHub.


this line should not be in the `with` block


@davidfowl for the solutions where this was a problem tracing showed this area of the code taking 90% of the time. With this change it takes 0.2% of the time.


is_negative = x < 0


comment on test


Does this commit just utilize system proxies or can one be defined from the config?


where are we getting searchMetaData from ?


I tested here and it seems fine. I got it working with windows either as a slave or as a master


That allowed my XOP system to work.  I am not sure why I was not 'seeing' the hypervisor bit.

NOTE, I have rolled back the changes, since calling cfg_*() functions early on is not readily available.

2872cf3


nop, it was for personnal testing, it happen to be commited with the rest of the code...


x = abs(x)

It's a good start to get familiar with python built-in function because you are gonna use it every single day if you code in Python: https://docs.python.org/2/library/functions.html


I would still like to have 'some' way to do this, just not sure how.

Removing it,


Go find out what's the difference between xrange and range.


search api.
check this: http://craftsvilla.unbxdapi.com/?s=shirt - the message is empty.


We can also consider to install miniconda (which also makes installing NumPy very easy).


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

```


@tenorz007 Thought we weren't supposed to put the '_id' in the transformers. You said that yourself


@baztian


I'm not sure this is appropriate.


These should be oxford commas. Except for the last one.


I'll merge this as soon as the checks complete.  Thanks!


Actually in Python if you want to reverse a list you just need to: array[::-1]


correct, this what is in the current kernel. But I like `brickpi:led1:blue:ev3dev` better, so I am planning on making the change in the next kernel release.


This change breaks my attempts to build on linux (debian & centos): ds3_init_get_objects() doesn't take any params..??

from ds3_c_sdk::ds3.h:
LIBRARY_API ds3_request* ds3_init_get_objects();


We're going to need to specify a common type for `tagged<Base, Tags...>` and `Base`, since `tagged` has converting assignment operators from `Base&&` and `const Base&` that require `Assignable`. In cmcstl2 the common type *is* `tagged` since I have conversion constructors from `Base&&` and `const Base&` for consistency with those assignment operators. Does that seem like a reasonable approach?

The constructors should probably do the *EXPLICIT* dance from the WP.


Maybe we should bind our `Diff` class also to a file, so you can turn this into a property^^?


do you have this rom because i don't know how to put it. i tried use custom camera app


Bullshit, there is no int overflow in Python. Try this:
import sys
print sys.maxint
print sys.maxint + 1


Password has since been changed... Man I suck.


Hey Flo,

last open item on pll_api is the naming of the typedef / enums. We already discussed this today, so it's up to you if you change it or not.
I think this is the last think to do and then we can merge pll_api into master.


Can we do this with a get_env() call ???

That would be just as good.  Set it in bash.rc and then those systems would ignore the checks.


Cambiado 'source' => 'content' por que petaba constantemente y no añadia el contenido del directorio files a los index.php


+1 I like it :)


<semaphore.h> breaks windows build


Use proper nouns for headings.


@AustinDidierTran you shouldn't commit your .idea files on git. Instead, create a .gitignore file and add `.idea/`


This should be a code block.


Yes, checking the shift position according to ieee standard would be usefull, this is really an error prone processus to determine it...
Actually None of the seems to produce subnormal of infinity. To fully test the lib we should think about it.
I can try to think about synthetic benchmark  converging to 0  passing by subnormals. (sum of 1/2^i from 0 to double Emin+52 ? and sum of 2^i to emax and then +1? )


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


Not by default.  There are no required params.  They are all optional.


jojo


Probably best to do that!


@ienliven What's the reasoning behind `pods.length >= 3`?


I think you have a point about the reader not necessarily knowing text will appear when they move their mouse. However, I also think it's something they'd only need to learn once, and then it would become natural to check the mouseover text.

At the end of the day, it's nice to have the option and not be trapped absolutely needing more space. But it might not be the right option in certain cases.


This line seems vestigial, why is it needed?


If I go to the teachers [page](https://codecombat.com/teachers) and then leave it (navigate to other page - for example about [page](https://codecombat.com/about)), when I click the teachers page button again, the redirection doesn't work.


Nice. Much better.


Why are double quotes used here?


 "Jomtobo" create some folders and files in my
 machine, ha ha

lol .. yeah it was to distinguish official and non-offcial stuff .
jomtobo is in the black market , like adidus or nikes ... ah ah ah
--------------------------------------------
En date de : Lun 16.11.15, Elieser Ademir de Jesus <notifications@github.com> a écrit :

 Objet: Re: [JamTaba] chord progression display finished (9556049)
 À: "elieserdejesus/JamTaba" <JamTaba@noreply.github.com>
 Cc: "Ezee" <h2oreggae@yahoo.fr>
 Date: Lundi 16 novembre 2015, 19h46

 no problem, fixed here and
 working for standalone. A software called
 "Jomtobo" create some folders and files in my
 machine, ha ha

 Testing the plugin ...

 —
 Reply to this email directly or view
 it on GitHub.


Why do this refactoring? The other one is the accepted format for no argument constructors. It's even on php.net article about [constructors] (http://php.net/manual/en/oop4.constructor.php)


I would expect the common type to be `Base`. `tagged` is required to inherit from `Base` and has no converting constructors from `Base`. If cmcstl2 followed the draft, I'm guessing this would Just Work. Can you give that a shot?


NullReferenceException is possible on line 122 when Assembly.GetEntryAssembly() returns null in case the managed assembly has been loaded from the unmanaged application.


This is possibly a dumb quibble, but let's try and avoid variable names like "a". That could be anything. No reason to not just type out "analysis" or something else relatively descriptive.


@yuslepukhin how about blocking it from running in Windows? This benchmark is not important.


I'm a little surprised this didn't cause any problems.  It's fine, understanding that it might be a little unexpected to future developers.  All this tinkering with `<hwhatevers>` implies that maybe you might just rather have a `<span>`. No need to change it if it looks okay to you, just some thoughts for the future.


I think you can just drop the `--user` argument entirely, as the code runs in a python virtualenv on travis. (It errors currently (https://travis-ci.org/Bioconductor/BBS/builds/91442154#L115))


Yeah, it does :(.

My thinking right now is to handle all the key bindings, check what mode you are in, and, if we don't care about it (eg. it's in insert mode), then pass it through.

This sucks cause:
* We'll need to define all the keybindings in package.json and extension.ts
* Need to figure out how to pass-through (not sure if VSCode will even allow).


Yes without a dedicated list item "Uses X pins" seemed a bit out of context. If the list item layout is preferred, I also think that the * should be inserted by code.


107712554-FixRefrigerationCondenserWaterCooledNotRegistering (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1968 of 1968 tests passed)

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


@siying I can either #ifdef it  or remove entirely from Windows build. Will check either way soon.


This is a file being uploaded.


I thought that this had been fixed in the most recent bp browser code.


@nuvipannu In this example the alert is reached... Would be interesting to run this on your machine and see what we get. :thought_balloon:


Not belonging to this commit, but the `__add__` method should create a new instance instead of manipulating the old one^^


I think this should be:

```
"map[bytes:type:VARBINARY value:\"\\001\\002\\003\"  float:type:FLOAT64 value:\"2.5\"  int:type:INT64 value:\"123\" ]"
```

Based on the error from Travis:

```
org.junit.ComparisonFailure: expected:<[<map[bytes:[type:VARBINARY value:""  float:type:FLOAT64 value:"2.5"  int:type:INT64 value:"123" ]]>]> but was:<[map[bytes:type:VARBINARY value:"\001\002\003"  float:type:FLOAT64 value:"2.5"  int:type:INT64 value:"123" ]]>
```


ok - cool :)


Not exactly what I had in mind.
No reason to warn about it. If it isn't handled on flow, the error packet will be warned, right?

Also, there is another usage of check_index() on argv_process().

The same should be done.


it looks like you reverted some rename improvements that you made?


Didn't you get rid of the import?


-- ídem --


Hmmm.  @robertgrimm you may want to take a look at this commit again.  It looks like Sam might have used an older version of the c sdk when he made this change.


The accepted way of naming interfaces is with an 'Interface' suffix. Also, this interface is not necessary


If you want to allow only modifying behaviour there's another special function (called nearly like `__add__`...)


Is there a reason this function definition is _inside_ the loop?


Yeah, that's what I do in Sublime Text. It's pretty ugly. There must be an `insert` command or something like that, I suppose.


@dmick yeah 2755? I  could see doing that.


That's exactly what I had in mind as well.


Also this one?


Nope.


Sure thing.


10/10 would check for updates. :P


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


@abellgithub I think it will fail, this change breaks the sort filter when building with debugging. I'll look into it.


Needs to be "dataset" instead of "datset" as well...


Sorry to disappoint, I did not perform any benchmarks. I was trying to fine tune the values to support backup to a cloud provider by setting the segment size to something closer to the cloud provider storage block size.


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


:+1: Yep, that's correct correct :) Good catch.


This statement is coming after a `return`. I don't think this would actually compile, so consider trying to build the binary locally. I am going to add a Travis CI hook to do this as well.


I think Sam may have messed up a merge - the commit 2 before this resolves this issue, and then commit undoes that work...