Great work, Anna! Nice structure and you're doing a good job thinking through the process, obtaining the city value a user enters and testing to see what city it is. Excellent work updating the background image on the body using the `.attr` jQuery function.

--

###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: __Anna Jacoby________

|                                                                                |                                |                        |                          |
|-------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| 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                                                     |                                |                        |                          |
| Use the ```$.ready()``` handler to delay your code from executing until all DOM assets have been loaded                       |                                |                        |             X             |
| Get the first element from an attribute name using ```$.attr()```                                                             |                                |                        |            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: __27__/28                                                                                |                                |                        |                          |


This is getting all the chess pieces not just the current game's.


ein 'entspricht' zuviel ;)


Gehört hier ein Komma rein?


This change also fixed #1414.


Kombi-Wort ;)


I think I'm going to fix some more Linux bugs soon.


@nateps @dadisn @enjalot
`app.loadStyles(__dirname + '/node_modules/bootstrap/dist/css/bootstrap.min')`
doesn't work for node v5. `node_modules` now has plain structure and bootstrap package gets installed into project's `node_modules`, not `d-bootstrap/node_modules`


fuzzy wär schon okay hier - sollten wir uns nicht noch eine Übersetzung aussuchen?


Vielleicht lieber 'aus'?


Hast Du das ausprobiert? (ich nicht, daher die Frage) - weil das Englische doch irgendwie eine andere Zeit verwendet.


vielleicht mit Komma und Großschreibung?


Ew global variables.


Vielleicht mit Bindestrich?


javascript code


Nice event listener.


Yes, the repo will grow in size indefinitely, but I have not seen other problems as of yet.
(RepositoryCheckTestCase.test_crash_before_compact fails (only tested py34))

My wish is that I can transfer my backups to an untrusted remote location, and revoke my permissions for modifying the files. This way my backup is safe even from my own tampering, or that of someone who gets access to my local server. The cost of this, as I see it, is that it requires manual intervention when the remote server is out of disk space.

I have not found an easy way to do the same for the hints and index files.

Do you see any other way of achieving the same outcome?


Good work using an `event.preventDefault()` to prevent the form submission, so your javascript will run.


What if we added a "Badge" public page that could let us name "our tech support employee of the month" (or something), which would let us see our MVP (internal user), status (online or offline) and group data? I made a few changes to my install, (and ended up using bootstrap for that page), but I think this would knock out a few "paid" options. Just a thought :)


Great job using a variable to store the city value entered by a users.


Entschuldige bitte die vielen Kommentare, Eduard. Ich habe leider gerade keine Zeit selbst was hieran zu machen und wollte wenigstens ein Minimum beitragen :/


Sure.  When you can get rid of the FOUT, you can change it back :-)


Nice if/else statements and checking for the different city names.


Perfect use of the `.attr` jQuery function. This allows you to ensure presentation is handled with your CSS and javascript sticks to interactivity.


^^ Looks good, still needs to be tested before merge.


@smo-key right now the image height is being stored as a variable in the class which is shouldnt be I just dont know how to find it. Also this is a pretty basic class for the position analysis. I dont know how you want to have it be integrated with your extensions idea.


可以扩大一下你的参考资料搜集和阅读范围，来点学术论文、英文资料如何？


When to wait it in master? :)


## First to create a "matrix" that can cache its inverse.

makeCacheMatrix <- function(x = matrix()) {
        inv <- NULL
        set <- function(y) {
                x <<- y
                inv <<- NULL
        }
        get <- function() x
        setInverse <- function(inverse) inv <<- inverse
        getInverse <- function() inv
        list(set = set,
             get = get,
             setInverse = setInverse,
             getInverse = getInverse)
}

## then computes the inverse of the special "matrix" and retrieve the inverse from the cache.

cacheSolve <- function(x, ...) {
        inv <- x$getInverse()
        if (!is.null(inv)) {
                message("getting cached data")
                return(inv)
        }
        mat <- x$get()
        inv <- solve(mat, ...)
        x$setInverse(inv)
        inv
}


@forjeff ok


Put comment about removing this when combat scene is loaded additive.


Nice work, David! Good structure and job thinking through the process, obtaining the city value a user enters and testing to see what city it is. Excellent work updating the background image on the body using the .attr jQuery function.

--

###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: ___David Klaassen________

|                                                                                |                                |                        |                          |
|-------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| 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               |
| Get the first element from an attribute name using ```$.attr()```                                                             |                                |                        |            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__/28                                                                                |                                |                        |                          |


1、未经分析，不要轻易给我们可爱的校园网应用扣上“网页漏洞”的黑帽子药；
2、你们需要一个JS高手参与屏幕键盘代码的分析工作。


ah, good catch, fix incoming...


多来几k字行不行？


Mir fällt keine ein... "S" haben wir schon für "Sichtbarkeit" und doppeln sollten wir wohl kaum.
Ich dachte mir "L" bzw. "Lock" versteht man auch noch als Deutscher.

Meiner Meinung nach machen die einzelnen Buchstaben an dieser Stelle sowieso keinen Sinn: Entweder man weiß sowieso schon was sich hinter einer Spalte verbirgt, oder man muss sowieso aufs Tooltip vertrauen. Aus dem Buchstaben allein wird keiner Schlau. Ist also hin wie her viel vergebene Liebesmüh.


Ja, das erscheint als "Rückgängig: Auswahlgruppe bewegen"
Verangenheitsformen erscheinen mir hier unpassend (und haben wir in der deutschen Übersetzung in der Regel auch nicht wenn ich mich nicht täusche)


ごたじぇー


1、你们可以用这个工具https://dnsdumpster.com/ 试试好玩不；
2、除了截图之外，注意保留原始出处URL。


Ja, schon in Ordnung... solange du nicht auf jeden eine Antwort erwartest, sondern zufrieden bist, dass ich das dann bei Gelegeneit abarbeite ;-)


1、建议你们扩大一下搜索的范围，比如youtube.com  ；
2、建议你更换英文搜索关键词再用Google.com和youtube.com等等找找资料，比如: mitm https / mitm ssl 等等；
3、请描述一下你们的实验环境网络拓扑情况，有图有真相。


Why make them macros?

Macro definitions aren't any better than `static const` globals.
Unless your reason is the fancy purple highlighting Visual Studio IDE gives macro syntax.


@cgutman yet again: very much appreciated! Thanks for all the work contributed to this great project :gift_heart:


Standalone


Looks promising. I'll integrate it tonight or we can on Monday. If I get it done tonight, be sure to look at the changes for future reference.

ALL HANDS ON DECK MON, TUE, THU!


Thanks @pgrudzien12

I've renamed and added a specific section on migrations after a note from Rowan Miller about them coming for RTM.

I also agree about adding the other notes about postgres and mysql. I don't have as much experience with these, so if you want to add them?


Because there could be location problems that relative to pose transition at Blender different versions.
Actually, I cleanly do not know source of problem, Is there a latency problem? But this solution works perfectly for pose transitions. (I do not get error about that issue)
Maybe future, It will be comment-out or delete, but for now I recommend to retain for a while.


Great job using an array to store the city values.


Nice work! Using a for loop to iterate through your array and dynamically add options to your form's select list makes things dynamic and portable.


@jackle1127  can you move this script out of the html page ?


@willfishe I can integrate it with the regular scoring method for now and smart scoring soon.


Same thing here. Move it out and call it from wherever you move it too @jackle1127
Let's keep things separate and structured from the begin so we don't need to refactor so much later.


Great use of the `.change


This is a pretty confusing switch to me (and probably my fault!).

I think it would be more obvious if it was {% if site.mode != 'development' %} and so on. Having those two empty blocks makes it pretty hard to grok that most of the time it falls through to the last else statement.

No a priority, just a note.


LGTM


This does not work with our SoC


Can you use something like this here ?
$query = "INSERT INTO Users (username, password, firstname, lastname) VALUES ( :user, :pass, :first, :last)";
    $query_params = array(
      ':user' => $_POST['username'],
      ':pass' => $_POST['password'],
      ':first' => $_POST['firstname'],
      ':last' => $_POST['lastname']
    );
@jackle1127


Nice job structuring your if/else statement and checking against your array values.


b295e3a

added


Looks good. I will look into how feasible this is. :)
@Katnis


Ok , i'll check that , i am 95% done with fullscreen stuff now .


@jef-n thank you! Glad I wasn't the only one hitting this..


Awesome work, Ellie! This was really well thought through and executed - it's a challenging homework assignment. You did a great job leveraging arrays to power dynamically populating your drop-down list and updating the background on change.

---

###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: __Ellie Yanagisawa_____

|                                                                                |                                |                        |                          |
|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| 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!) |                                |                        |                          |
| 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                                                                                |                                |                        |                          |


Working fine, thx


This commit is old.  I'm looking for the current version of this commit in a pull.  I think it was close for merging, but I can't remember where it was.


I think the reason it isn't currently working is that we valid_move doesn't prevent cannibalism :smile: It also doesn't prevent moving onto the same square.


6. In kfs-core/src/main/resources/edu/arizona/kfs/coa/businessobject/datadictionary/CrossOrganizationCode.xml, need attribute CrossOrganizationCode-organization.codeAndDescription to properly display organization code plus description. See screen shot in help text section of functional specs (page 20).
7. In kfs-core/src/main/resources/edu/arizona/kfs/coa/businessobject/datadictionary/CrossOrganizationCode.xml, please add:
 <property name="validationPattern">
        <bean parent="AlphaNumericValidationPattern" p:allowWhitespace="false"
  </property
to attribute definition for attribute CrossOrganizationCode-crossOrganizationCode.
8. In kfs-core/src/main/resources/edu/arizona/kfs/coa/businessobject/datadictionary/BudgetShellCode.xml, please add:
 <property name="validationPattern">
        <bean parent="AlphaNumericValidationPattern" p:allowWhitespace="false"
  </property
to attribute definition for attributes BudgetShellCode-budgetShellCode and BudgetShellCode-groupCode.


this is a test repository to python


We could remove the loop. Just increment once.


@BessHsu ok


@RebeccaKung OK


@s102213010 OK


@s102213013 ok


I'am sorry.


@102213015 ok


@s102213018 ok


externalify is a NPM module

https://www.npmjs.com/package/externalify


@shengshenggu ok


@chenfan617 ok


	"+- Edited post-deployment script for 'Debug' build to stop/start K2 blackpearl during deploymen"

It will fail if k2 run as console application (most of devs use console mode because of license)


no, incCnt is how many times we do inc test.
I don't think just incrementing once is ok.


@102213029 ok


What the fuck


@102213035 ok


@102213036 ok


Hi @FrancoGag is this a problem with the current hosts file?  Are any of the current hosts providers giving us this problem?


@102213066ruby ok


@mddon20100512 ok


@sam9494 ok


@102213043 ok


My only comment on this commit is that we will probably remove 2.25 in 1.23, as we need to only support the latest releases of binutils. It's ok to commit it, just fyi that 2.25 will be removed soon, since 2.25.1 is a bug fix release of the 2.25 branch.


@nanoha1fate ok


http://emberobserver.com/addons/ember-cli-d3

Initial thought is wow does D3 look good (the output at least, no idea
about use/under the hood) and the examples look great.
https://github.com/mbostock/d3/wiki/Gallery

http://blog.simplereach.com/why-use-d3-ember-for-data-visualization/

D3 seems very powerful/complex.  My desire is to pick a winning
visualization lib that can do the simple stuff initially and then also go
all the way if we need it.

Thoughts?

On Sat, Nov 21, 2015 at 8:06 PM, Dan Nelson <notifications@github.com>
wrote:

> Hahahaha, "He remembers me!" LMFAO.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cybertoothca/project-gustav/commit/746f20385b6057d66d22dcec20e18baa6a35af1e#commitcomment-14545699>
> .
>


@s102213039 ok


All I can say is...You the man Beans. Keep it up!


@tina96391 ok


@BBrandon320 ok


```
2015-11-23 02:23:18.204 CocoaPods[60932:4928129] {
    "target_definitions" =     (
                {
            dependencies =             (
                                {
                    Fragaria =                     (
                                                {
                            podspec = "Fragaria.podspec.json";
                        }
                    );
                },
                                {
                    ANSIEscapeHelper =                     (
                                                {
                            podspec = "ANSIEscapeHelper.podspec";
                        }
                    );
                },
                Sparkle
            );
            "inhibit_warnings" =             {
                "for_pods" =                 (
                    Fragaria,
                    ANSIEscapeHelper
                );
            };
            "link_with_first_target" = 1;
            name = Pods;
            platform =             {
                osx = "10.9";
            };
            "uses_frameworks" = 1;
        }
    );
}
```


fasd


发送到爱上f


I'm on latest babel (6.2.1) using es2015 and react presets, but it seems the `transform-object-rest-spread` plug-in is not in either of them, and has to be added exclusively to enable this feature.


ghhh


Not immediately obvious what the string is - I assume it's supposed to be the ETag. Should this return `ETagged<object>` instead? Also, should `ETagged<object>` be `ETagged<TState>`?


//hola


The `contentsScaleFactor` is available only in Flash >= 11.5 so this change will be causing errors with older players. This is important especially for Linux users who don't get updates to versions >11.4. Some conditional logic should be used for detection of this feature.


La anterior estaba bien ¬¬
return Redirect::route('cliente.reservas.create');


Is this code from #1036? You may need to rebase against master, if so.


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


The default example keybindings for switching desktops interfere with browser keyboard navigation (Alt+Left/Right go back/forward a page)
Its a bit annoying to switch desktops
I know it is just an example, but people often see the example as a "standard".
It would also be nice to show examples of some of each feature like:
 tooltips ... put a description
 icon name extensions ... a couple examples without extension  (only use extension with full path)
  ... there are so many, perhaps its time to have a ./contrib folder?


I like the generic approach. Should we use an interface instead of `ETagged<T>`, though? That would give flexibility for those who don't have a need for ETags. See https://github.com/dotnet/orleans/issues/1035#issuecomment-158825916


No waaaaayyy!
I really like this "and" , "or" etc instead of symbols.
Couldn't we just include <iso646.h> instead of so much changes?


Thanks for the example. This colour scheme actually works better than the original, I think, because the yellow is easier to distinguish from white.


這段註解保留的用意？


bool fullScreenViewMode; in windowMain.h is unused , please check and delete if needed .


註解?


It's your call, but as it stands the code can't be compiled with MSVC which is a pity. Also, subjective preferences aside (I strongly dislike the alphabetical versions of the operators), the fact is that 99.9% of all C++ code uses the symbols and not the keywords, so this makes the code base more awkward to work with for the vast majority of people.


註解保留的用意是照片有可能會在新增。


有版控其實不需要用註解保留程式碼。


As mentioned at https://gist.github.com/cobyism/4730490#gistcomment-1243110, I would also favor the solution presented at https://github.com/X1011/git-directory-deploy. That solution keeps the development repository clean. :umbrella:


This looks good to me!


I should add that `CurrentSuper` modifications made the following code to not compile:

```
CURRENT_STRUCT(Foo) { ... }
Foo foo;
::current::CurrentSuper* super;
*super = foo;
```

It was compiling before, as well as the real-life "usecase" of the above is assigning to `std::unique_ptr<CurrentSuper>`.

CC @mzhurovich


In case of all easy moves(both existing and the new SMP kind of this patch) only the main thread's move is verified to be easy.  Therefore in order for this patch to be correct we can't pick another thread if an easy move was made.   This is of course a bug in the existing easy move code as well so if this patch fails line 331 should still be submitted as a bug fix.


I learn that from hard way before.
I believe that I avoid this using some other code by resizing window but thanks. :+1:


I'll add description of the return type to the <returns> if this approach is acceptable.


Ooops


lehet hogy ez hozta elo azt a furcsa lehalast. viszont a clearCookie fontos lenne


I would remove the dependency from `lodash` what you have here is easily done without libs.

```javascript
var plugins = merged.use;
var i = plugins.indexOf('postcss-reporter');

if ([-1, plugins.length].indexOf(i) == -1) {
    plugins.splice(i, 1);
    plugins.push('postcss-reporter');
    merged.use = plugins;
}
```


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


Well this is work-a-round for MacOSX.
Program magically broke if I use it like that because MacOSX try to destroy mcd twice.


Yes, I'll change ETagged to the interfaces.


Doxygen commented Mine class.


Unfortunately, this patch will make OSX version crash.


This workaround is definitely unnecessary, there must be some other bug elsewhere if the program crashes just due to creating a dialog on the stack.


Sorry, but \iIf it crashes with this patch, it will crash without it too. It's plain impossible for this patch to break anything.


`NSString *encodedText = [rawURLString stringByAddingPercentEscapesUsingEncoding:NSUTF8StringEncoding];`


`length` returns NSInteger and hence can't return nil or false. It will return 0 in both cases since 0 is technically the same as nil.


@hu-weihua  Thanks your enhancement,
 Could you look into the "-l, --lazy" and "-f, --force" options for umount command, since we want to umount /dev successfully?


Indeed it can show the dialog, when you try to close the dialog, crashes...
If I remember correctly, I debugged and saw that OSX try to destroy the object that already destroyed.
Don't know why.
I ask help for that bu there are no wxWidgets developers interested with it, on Mac world.


Yes,althought S3 api does not define "usage", but I think it should be there if we want to support usage query.
I am not quite familiar with the rgw code.Or can you tell me where should I put it?
usage can not be a parameter of a bucket operate in deed.


Теперь `aib.krau` не нужен :grinning:
Для тестирование можешь в консольке выполнить `boardRequiresCaptcha = true;`. Это включит капчу. Так же капча теперь должна включаться при ошибке постинга, если там регекс детектирует что-то связанное с капчей, ну ты понял.


@g1itch FYI -> https://github.com/openwrt/packages/pull/2010


To be fair I use Win 10 for testing and have the issue - I did a test with these being 9's again and the vanishing hotbar items bug persists so not actually sure this was the problem?


This is also a solution to issue #79.


should be `/var/run`?


@itsjeyd

> Ideally, WISYSIG editor should just replace the "Vectors" edit control

Looks like I've edited that sentence a couple of times, so in the end it didn't quite meant what I was trying to say :) To be precise, "replace" is a wrong word there. The idea is to have a switch between text and WISYWIG editor in that field. Or place them side-by-side. Or somehow make it clear that it is two representations of the same thing.

One more thing - you might want to move common JS parts of WISYWIG and actual studen view into a helper file, like in [Problem Builder|https://github.com/open-craft/problem-builder/blob/master/problem_builder/public/js/util.js] (unfortunately RequireJS effort that was supposed to provide a better way of doing that is kind of stuck at the moment).


Maybe...


Line is too long. [101/100]


The first argument to describe should be the class or module being tested.


Any plan to release an hotfix for this?


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


No. :)

In practice `/var/run` is a symlink to `/run`. `/var/run being` legacy, `/run` being the real thing.


Oh noes! :smile:


Has/Get -> Get/if, please


right, so we should IMHO rename the method to something like "close_requests" or "revoke_or_decline_requests"


Ok. Just tried to help with possible typos.


will look into this after merge


@yumetodo いっそのことprintfとscanfをいない子ということにしてしまうとか。


It seems like this class is useless.


Appreciated. It's always nice to have someone looking :+1:


Good :+1:


RAM is not really a tool for continuous integration, though. It is for continuous Requirements engineering, and for early requirements selection/filtering/alignment with product strategies. But it won't help you with continuous integration.


Hi @ahuarte47 , Unfortunatly not. I have no compile environment ready and a lot of other priorities this week.
I can share a dataset + qlr file that currently triggers the crash every time if needed.


Should be called "isAccentColor" imo


Nullable argument to NotNull argument


> You don't remember that coverage has extra warnings enabled, right?

I remember everyday because, eg, microbit's build system forces lots of warning on you (although things have improved just recently).


いやあ、しかし似たようなコードを全部に貼るのは汚いですよ。JekyllはMarkdownパースを無効にすればそこら辺の自動化ツールになってくれるはず。


使用快排并不行，快排交换位置需要使用o(n)的时间，并不能在常数时间内完成


Unused field.


perfect!


> Follows the work from @jponge in the cradle-simplest project

The French in me laughed at `cradle` :smiley:


```event``` should be ```$event```


True! But the function does exclude one particular other color (black) to keep the grid rectangular. So the name should also cover this.


I thought I was wrong but I was trong.


Later today


Well, that's what we do in some other places.


Unused method


Just nitpick, but can you please insert a space between eid", and "thumb.


I guess it should be "garbage_rank..." instead of "grabage_rank..."

My 2 cents ...


user.incapacitated()


thanks :)
:+1:


Why assuming i'm using bower ?


Great news


Hmm, this may have been problematic - well spotted!


Briefly describe each of these challenges.


Unfortunately, there are a lot of string IDs with syntax errors like the one you just reported, but we must not modify them : we only modify text contents.


I like this pattern, makes it very easy to read. But also I think we should extend it for collection types. A common pattern is to have:

```
@Managed
interface Thing {
    List<String> getTargetPlatforms()
    void setTargetPlatforms(List<String> targetPlatforms)
}
```
and I would love that we automatically generate this:
```
model {
   thing {
      targetPlatform 'java7' // *adds* a target platform (note that we remove the trailing `s` in the method name)
      targetPlaforms 'java7', 'java8' // *adds* multiple target platforms
      targetPlaforms = ['java7', 'java8'] // *replaces* target platforms
   }
}
```


As discussed during class: Is this total number of requirements, or is it the inflow of new/changed requirements per a given time unit?


Works fine here, as far as I can tell (using Jekyll 3.0.1) but I'll keep an eye on this once we merge this


Source?


looks good.
Did you address all the comments or decided to skip some?
It is hard to find this out with github PR inteface.


Cool, thanks! ^_^


Changes are looking okay.

@Montellese Not sure whether https://github.com/xbmc/xbmc/blob/master/xbmc/interfaces/json-rpc/schema/version.txt needs to be bumped.


Which are these disadvantages?


debug print


What if you want to /need to plan ahead for more than two releases?


Maybe generalize this and add possibillity to define exclusion List with regex values? Current configuration could be the default. This method would then simply Check against the blacklist


@devil1234 Well, no. Because right now, the way it works is it uses hardcoded items from Classes.FreedomCore.php Extension, and in order to make Boost Level Dynamic, i will have to create database table with items => boost level relation. Since i'm on last week of the term for bachelor, i cant do it right now (lots of stuff to be done for lvl 80, 85, 90, 100). But later, this will be implemented


OK ... Too bad ... Just trying to give a help :-(


`<button class="travel-foto__close">×</button>`


Changing from ClientTypeSignature to ClientTypeParameterSignature breaks compatibility anyway.


Which paper?


No parens needed here.

`2.9.3` should also be updated to `3.0` above.


I can't see this in the README


Hi there, can you incorporate this fix into a Magento connect release to avoid confusion? Thanks!


ok, do not worry, it can be proposed


Writter? ;)


Ok summary; Now connect it to what you read in other articles and your own experience.


> The following properties exists and have get/set methods available:
>
> ...
>
> excludeMediaQueries |true|Exclude the media queries from the inlined styles.


@isbm that was not for leap, that was for SLE11SP4 failing to start services. The Leap issue is unrelated and handled on a different commit


hello，I have a opptunity for android leader


Is `only` still needed?


It should be explained, possibly with an example, that given characters is not considered a substring but a list of characters to be removed. This often causes bugs in Python too. For example, `'abba'.rstrip('ba') != 'ab'`.


Good catch @mlq thanks!!! :smile:


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


initial commit


How does the CMMI level affect release planning?


Please provide proper references.

How does it help to assign weights the customers? Which criteria are used?


Pls define "maximum value"


I think you are looking at the README of the master-branch instead of the 2.0-branch: https://github.com/tijsverkoyen/CssToInlineStyles/tree/2.0


Ezee, I have two little suggestions:
1 - Change **MainController::setFullScreenView** to **MainController::saveFullScreenState**. The name **setFullScreenView** suggests you are setting the "mainController" to full screen. But is a controller, not a view, controller can't be full screen. It's just a name stuff.

2. You can avoid **if( mainController->getSettings().windowsWasFullViewMode() )** creating a new convenience method/function in MainController to return the settings value, so you don't need these 2 leves of delegation:
```
class MainController{
      inline bool windowWasFullViewMode() const { return settings.windowsWasFullViewMode() ;  }
}
```
Now the if can be reduced to
```
if( mainController->windowWasFullViewMode() ){
    ....
}
```


Pls explain revenue projections further.


@aboe76 can you please show us the services code where it expected 'SUSE' instead of 'SLES'? I am trying to find it without any luck so far


You're right. So this is a dropped feature I guess? Media queries are now always removed?


Please describe the process involved in using QUPER.


```python
if query_string in ['pretty=1', 'pretty=true'] or  request.accept_mimetypes.best == "text/html":
    pass
```
The two conditions could be merged in one, and it's still readable


Why do you set the userid to 1000 in line 9, after you explicitly create the user with uid 5001 in line 6?


Thoughts on this? Will it work? Why? Why not? What is missing? Does it scale?


pls describe each step with one or two sentences as well.


The eta cut can be set to 2.4 ;)

I think it would be best not to add a bool to the Jet structure, but simply only storing in:
https://github.com/blinkseb/TTAnalysis/blob/83ec3750959b06cb73fe53d75cd7fd617552ba28/plugins/TTAnalyzer.cc#L284
https://github.com/blinkseb/TTAnalysis/blob/83ec3750959b06cb73fe53d75cd7fd617552ba28/plugins/TTAnalyzer.cc#L361
https://github.com/blinkseb/TTAnalysis/blob/83ec3750959b06cb73fe53d75cd7fd617552ba28/plugins/TTAnalyzer.cc#L471
https://github.com/blinkseb/TTAnalysis/blob/83ec3750959b06cb73fe53d75cd7fd617552ba28/plugins/TTAnalyzer.cc#L656
https://github.com/blinkseb/TTAnalysis/blob/83ec3750959b06cb73fe53d75cd7fd617552ba28/plugins/TTAnalyzer.cc#L672

Even if it is a bit cumbersome because of these many places, I think it will makes things easier afterwards...


y u hef 2 cast 2 void m8?

is cursz gr8 m8?


resolves #3, improved by 59601c6


C'est propre !
Par contre l'implémentation de la fonction cocobot_position_task est-elle terminée ?
C'est normal que la distance curviligne soit la somme et non pas la moyenne de la valeur des encodeurs ? Et aussi que l'angle calculé de dépende pas de la distance entre les roues des encodeurs ?


You are right, the channel size is not configurable yet. Your implementation is using the async publisher which is allowed to drop messages after max_retries (set 0 to always retro => no event loss, but potentially blocking queues). The default queue size is 1000. I think you can easily is PublishEvents. Why do you pass a map [string]interface into Publish? Maybe passing defined structs now and build events in last step.


Does this scale? Check up how AHP works.


uhm, so what is the advantage of this?


OK


Isn't it just easier to use only one error type in a crate? redis has `RedisError` iron `IronError` so we could just define `SpaceapiServerError`


Not working, as in mailbox-cassandra, maven doesn't export dependencies with a test scope.


How about moving the one enum to two variables and transform the struct to an class to be used in Lua?


nice job


tu jest literówka czy zamierzona nazwa funkcji?


there's a list of exceptions in LICENSE.md and contrib/add_license_to_files.jl for this kind of thing


```sh
$ git stash
```
### To make your working directory of local repo same as that of origin remote ([github.com])
[github.com]: <https://github.com/>


@e-kolpakov

> The idea is to have a switch between text and WISYWIG editor in that field. Or place them side-by-side. Or somehow make it clear that it is two representations of the same thing.

Gotcha :) The thing is, we still need to discuss how we are going to implement support for specifying the "Expected result" for an exercise (e.g., are we going to use the same board or a different board?). Depending on the answer to that question, it might make sense to have all of the WYSIWYG functionality in one place. So for now I'm going to with the minimal option: Adjusting the descriptions of the "Vectors" field and the WYSIWYG editor to establish a clear relationship between them.

> you might want to move common JS parts of WISYWIG and actual student view into a helper file

Agreed, it would definitely make sense to do that :) But to avoid having to iterate this process, it would be good to do it when both parts of the code have stabilized (i.e., after adding the remaining WYSIWYG functionality and addressing accessibility issues).


This commit is being reviewed as a part of review issue #130.


Xiaomi provides shitty translation files, we have to deal with them... :-/


oh sorry I forgot to remove


How about ```if self.reviews.none?```?;-)


Please give some explanation in the commit log which justifies this change.


当初それをしようとしたんですよ。わざわざHello Worldもputs使ったぐらいで。scanfも9章まで伸ばしに伸ばしましたし。多くの入門書では真っ先に``printf``と``scanf``使ってるんですが。

ただ、それするとループやら条件文の説明で全然sampleや練習問題が浮かばなくて

#46

みたいに7章でやりたかったのに9章に持っていくはめになったりしました。

というか、配列禁止、scanf禁止でループやらifのいいsampleが浮かばなかった。command line argumentはなぜかWindowsユーザーには馴染みがないらしく(なんでや、ffmpegとか使ったことないのか！)、反対意見が出たので(issueで出たと思ったけど確認したら口頭でやり取りしてた時にでたのか)これも禁止になって。


Thanks for the information. I just checked it with Firefox 43 beta and this workaround is not needed anymore. But still we need it for Firefox 38 - 42


```
self.commit_opts = {
  no_backend_write: 1,
  request: self.project.commit_opts[:request]
}
```


hotfix on master


Il manque juste de quoi faire un hook pour lancer un calcul d'asserv après avoir eu la position.

C'est normal oui. Si tu veux une moyenne il faut diviser par deux, c'est donc juste une histoire de constante. Il faut donc ajuster CONFIG_LIBCOCOBOT_POSITION_TICK_PER_METER en conséquence. L'aventage est double : une opération de moins, pas de perte de précision (division par deux sur un entier) ou pas de perte de temps (passage sur des flottants)

Même chose pour l'angle. C'est une constante


这里不用 less 吧。


Good point about the names. I wanted to rename `RC_ANY`/`RCMASK_ANY` into `RC_ALL`/`RCMASK_ALL`, but I didn't for backward compatibility reasons (just in case it's used in third party scripts, because official ones don't seem to use it at all). The names you suggest make more sense.


because of
https://github.com/discourse/discourse/commit/262f561a877e0296d7ca7f6ec9f27edd0d30ca8e
user_name_suggester.rb
in
def self.sanitize_username(name)
    name = ActiveSupport::Inflector.transliterate(name)
    name = name.gsub(/^[^[:alnum:]]+|\W+$/, "")
               .gsub(/\W+/, "_")
               .gsub(/^\_+/, '')
               .gsub(/[\-_\.]{2,}/, "_")
    name
  end
is wrong?


OK


fixed in https://github.com/tijsverkoyen/CssToInlineStyles/pull/111


Ok thanks for info.


fixed in c6359dc626fa3a3f4c752755273ebca711220865


But `UnknownSensor` is not really an error, it's a validation problem.


Please provide complete references.

Go through these papers in slightly further detail.


Fixed in https://github.com/kdeldycke/Tipue-Search/commit/5067f612ee692ca7885e01cef968fbdaa78c2940


Nice catch. It should be possible to write a unit test for it though right?


Where is the art, and where is the science of RP?


@yumetodo > command line argumentはなぜかWindowsユーザーには馴染みがないらしく(なんでや、ffmpegとか使ったことないのか！)、反対意見が出たので(issueで出たと思ったけど確認したら口頭でやり取りしてた時にでたのか)これも禁止になって。

なるほど、つまり端末上での引数の渡し方も説明しろと。あとパイプ関係の記述もあるから…


The second ID is wrong.
The IDs should be 0x44 and 0x4D
See http://ams.com/eng/content/download/319364/1117183/287875
Page 24


pls define "maximum value"


Never compare strings via !=


i think so. Will make a JNI test case.


By which criteria are weights assigned?


s/descriped/described/


Pointless


This commit includes a breaking change to the month picker, all projects using this will need to be updated to follow the setup listed in the attached document.
[month_picker.pdf](https://github.com/BookingBug/bookingbug-angular/files/41472/month_picker.pdf)


Regardless of whether it's used in official servers any longer, we'll still need to use RC_DemiPlayer it in pre-renewal. When this was implemented, I remember that the decision was to implement it in a flexible way, so that, regardless of what changes the official servers made, we'd still have the three DemiHuman + Player = DemiPlayer types, usable in the necessary combination


fixed in acff8170e4d9df11a724345dc71281cffca24119


Merci.


@jhass: For some reason here, it does not render the correct layout and I'm obliged to add `.mobile`, have I missed something?


I found that comment quite helpful to understand what is going on here, and be sure that it's really intended... Mind if we keep it?


fixed in e3b73228610e4e5782f432361efae8508fcf935f


Why don't you do `$objects[] = $object`? I see no reason for doing this call twice


Few more typos.


How does QUPER work?


Wrong math :( This is the right one:
var mainDmg =
                                        new[]
                                            {
                                                0.75, 0.76, 0.78, 0.79, 0.81, 0.82, 0.84, 0.86, 0.88, 0.9, 0.92, 0.95,
                                                0.97, 0.99, 1.02, 1.04, 1.07, 1.1
                                            }[hero.Level - 1];


and what is your answer?


how?


Přidej k výjimkám interface, ať se dají chytat libovolné z tohoto balíčku


target Hasbuff not source


Good work extracting this into a partial. It can be taken one tiny step further:

```diff
-    <% @topic.posts.each do |post| %>
-      <%= render 'posts/post', post: post %>
-    <% end %>
+    <%= render @topic.posts %>
```


Działało i tak, bo wszędzie była ta literówka uwzględniona, ale poprawiłem przed chwilą na getPagination() ;-)


前に調べたんですけどその頃はその辺の知識0だったので挫折したんですよね・・・。いまならわかるかな。


Please reflect more on what you write. These are ok summaries of the papers, but what is good in each paper? What do you disagree with? Why? How does it relate to other papers and your experience?


fix_times is too general name for this method. What does it really mean to 'fix' some time object? This method doesn't fix anything, it in fact truncates milisecond. Maybe better naming would be 'truncate_miliseconds'.

Another thing is that the method serves another purpose. That is iterating over a hash. So the correct name should be "truncate_miliseconds_on_hash_keys". You can mitigate this long method name by spliting this utility function into two: 1) function that truncates miliseconds 2) function that iterates over a hash and runs a function on specified keys.


Any chance of releasing 1.4.2 with this change?


It's a shame we can't register anymore to xiaomi.eu :-( ... It's frustrating to not being able to give input and feedback ...


Issues are already tracked and I found the corresponding issue: https://development.invoiceplane.com/browse/IP-337
It's already fixed but not jet released.


Everything should be perfect now. All the requirements that can fail should be showing fails when the ALL FAIL file is run, and should be showing pass when the ALL PASS file is run.

To switch between passing and failing the tests, you can simply change the value of the bool "runFail" in Main.cpp. runFail = true makes all the tests fail, and runFail = false makes them all pass, by parsing the appropriate GEDCOM file.

If you have any questions, please let us know!


Go slightly deeper: Don't just list things; expain them in one or two sentences as well.

Also; see earlier comments about reflections


Added database credits.


```opt.returnCallback(field.value)``` here


纠正错别字：
peremater --> parameter


Fixed in fa040ce760b7c981b51209af6e24f44817d39899


create ACM dir


I am not Ruby expert but if this was Java, I would try to include this "miliseconds removing" functionality into asserts (expect() method on line 50). Because it seems that any test method that is going to be asserting time based data will need to truncate miliseconds.


create this


Sorry spoco2, I could not get the event emission and catching for checkNewState, could you explain that in detail that how would I catch event stateAdded when the children states are added...
If you could update the code itself, that w


personally i'd expect client.duplicate() to keep the same database selection - if this isn't going to be the case then it should probably be made clear in the docs to prevent confusion


@matthewd out of curiosity – what led to this?


Windwosユーザーでcommand line argument理解してるのはプログラマか動画編集をフリーソフトでしてる人か、くらいしかいない気がしてきた。良くも悪くもGUI文化ってことだな・・・。

一応main関数の引数の意味を書くために5章に``main関数の引数って何？``という項目があり、講習会でも実際にコンパイルさせたんですが。

ていうかMacユーザーすらも、Xcodeで開発してるから端末使えない気がしてきた(恐怖)


Hmm.. Try it on a iOS/android device, we should look at the majority, as most of them are iOS/ android users :/


Je suppose que tu es français, donc je t'invite à aller sur ce forum : http://mi.france.free.fr/
C'est une extension de xiaomi.eu exclusivement dédiée au français :)


Btw, do slots 8 and 9 works in MCPE after the update?


Нет смысла. На самом деле мы никого не блокируем, лишь откладываем процесс возвращения соединения в пул — дедлайн контекста ничего не прерывает и не блокирует клиент. Это лишь подсказка серверу, что он может перестать работать над ответом.


Pros and cons with RAM? Please connect to your experiences.


@patriknw
What should we do regarding "regular" messages that are not part of the protocol (ActorIdentity, etc.)?
Should we just forward them, or use ask for them as well?


Rebase this in your work (2ff4e39804f6dca222563f60ec0eb9374e4a60b3)


Pls also describe the different levels (Project, Process, ..., Society), what is in focus on each view, and how are they connected?


Please describe how the QUPER model works.


It could be the moment we use several lines :)
```python
if os.environ.get('AVTRANSCODER_TEST_AUDIO_WAVE_FILE') is None or \
   os.environ.get('AVTRANSCODER_TEST_VIDEO_AVI_FILE') is None or \
   os.environ.get('AVTRANSCODER_TEST_VIDEO_MOV_FILE') is None:
```
Don't you think ?


@yumetodo まず今回は入門ですから、IDEなんていう高度なものは使わずにコマンドラインという簡単なものを使うようにしましょう。
まずCygwin (not MinGW)とGUIのエディタをインストールさせて、それからgccを叩けば…


Does this scale? Please check up on AHP.


更新了您的代码，我把 properties文件在如下几个位置各放一份。但是程序还是报告无法找到properties文件
----------------------------------------------------
C:\Debug>dir *.properties /s/b
C:\Debug\hanlp.properties
C:\Debug\Appserver\lib\hanlp.properties
C:\Debug\lib\hanlp.properties
C:\Debug\Webapp\WEB-INF\classes\hanlp.properties
C:\Debug\Webapp\WEB-INF\lib\hanlp.properties
----------------------------------------------------
C:\Debug>ConsoleTester
十一月 23, 2015 6:44:09 下午 com.hankcs.hanlp.HanLP$Config <clinit>
SEVERE: 没有找到HanLP.properties，可能会导致找不到data
========Tips========
请将HanLP.properties放在下列目录：
Web项目则请放到下列目录：
Webapp/WEB-INF/lib
Webapp/WEB-INF/classes
Appserver/lib
JRE/lib
并且编辑root=PARENT/path/to/your/data


```js
.get('/posts?filter[where][title]=deer+can+jump')
```


@Halama aby ten `yum -y install php MariaDB-server MariaDB-client` nepřepsaů ten `/etc/my.cnf` nějaký, defaultem!


Won't this trigger twice? It'll go into advance and then startSpectating and trigger again. Or potentially spam if advance doesn't advance for any reason.


Ok summaries, but I want you to work on reflecting a bit more when reading. What do you think works well, and what works less well. It is ok (and encouraged in scientific communities) to disagree with a paper, as long as you also motivate why you disagree.


Are you saying the function being called in mraa_i2c_read_byte was wrong here? @whbruce


Can't we use JdkChooserPanel instead?


この書き方で書いた場合、
例えばWeb DesignとApplicationだけタグを設定して、UIのところ空欄で保存した場合に、
@prototype.tag_list[2]って存在し得ないんじゃない？

それよりも、eachとかである分だけ回してあげたほうが正確だと思う。


@klaus1982 thanks for the PR. Looks good.


well, `resend` there is the configuration param so, if you want to use `resend` and do that, you're turning off the `resend` param. Maybe you need to remove the `#matching{}` entry from `timem` part instead to avoid to resend it again. To do that you'll need the namespace `NS` param and the `ID` from the message.


OK


Can we make this method return a boolean and assert on it `Neostore`? I don't see much benefit in checking for not null in production environments.


@klaus1982 Would you mind to create another PR against https://github.com/kubernetes/kubernetes/blob/master/contrib/mesos/docs/scheduler.md ? This copy of the scheduler docs in kubernetes-mesos should actually be deleted because they went into the upstream repo now.


そうしよう。そもそもあんな小規模なプログラムにいちいちプロジェクト作るとか大変だし。

でもCygwinではなくmsys2 mingwのほうがいいです。できた実行ファイルのdependencyがえげつないので。しかもstaticにできない。


npm3 需要


tag_listが複数ある場合って、これで保存できるんだっけ？
(※俺はtagだけ切り出してeachとかで保存してたから、これでいけるならこっちのほうが良い)


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


we should add trim_trailing_whitespace too, as our git hooks don't allow commiting stuff that has trailing whitepsace.


Please use a full reference here.


nn je to v pohode


@yumetodo 今回は入門ですから、環境の導入が簡単に出来たほうが良いと思うのです。何かとCygwinは (手元でUnixとして使っている限りは) 簡単ですから。


I don't think you need to fetch the `DependencyResolver` every time. I think a better approach is to take it in construction instead.


Super! Díky
On Nov 23, 2015 6:52 PM, "Martin Halamíček" <notifications@github.com>
wrote:

> nn je to v pohode
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/keboola/cookbook-keboola-tapi-sql/commit/bad01040aa8316bd2785398f5197b8851a3760fe#commitcomment-14556834>
> .
>


select2 should not be activated on mobile browsers. You need this statement inside the `if` block


Is this comment still useful?


Please describe how QUPER works.


これも親要素が消えたら連動して消える記述を書いてあげましょう。


Same remark here :
```python
raise SkipTest("Need to define environment variables "
               "AVTRANSCODER_TEST_VIDEO_AVI_FILE and "
               "AVTRANSCODER_TEST_AUDIO_WAVE_FILE and "
               "AVTRANSCODER_TEST_VIDEO_MOV_FILE")
```


Когда я это писал, (26 Дней назад) этого еще не было сделано


Let's have classes start with capital letters.


ok


そうだ、そもそも。なぜWindowsの上でUnixもどきを使う必要があるんですか。Ubuntu入ったUSB配布でいいじゃないですか。そのほうがインストールの手間もないし。


I want you to reflect more while reading. What works? Why? Why not? How does it relate to other papers? Your experience?


include the stack trace in the log


You're missing the last line break at EOF, please update your IDE settings. (PSR-2)

I wonder why this was implemented manually at all, if there is a lib call for that. I hope we don't break anything.


@e-kolpakov Addressed the main points in this commit: [c3a098d](https://github.com/open-craft/xblock-vectordraw/commit/c3a098db2fc942ba5003d02bcb40806d9947c46f). Could you please have a look?


prototypeとのアソシエーション使って作りましょう。
そうすると引数でわざわざprototype_id　指定しなくても済みます


Very nice improvements and code, @rasuttmus!


@atd, if I understand well the problem, it is something @pablojan and I discussed already. It seams reasonable that in SwellRT a set of handlers can be registered to the same event. However, now only one handler can be registered per event.

We could fix it doing this in Pear2Pear, but I believe it is a SwellRT issue. If @pablojan agrees, we can add the issue in SwellRT


ok 有時間來改


I had the same thought about the `overseas-passports-` prefix but decided to leave it for another time.


put a comment as to why we have commented code here


This is true. There could likely be more things to add. I just implemented it roughly and add this until I release v.2.4.


改行


OK


> As an aside, I wonder whether the _button partial should use Govspeak/Markdown to create the link tag.

Assuming we can use Govspeak to generate the same markup then I think you're probably right that that's what we should use.


msys2もDLして解凍して一度msys2_shell.bat走らせて

```
update-core
pacman -S mingw-w64-i686-clang mingw-w64-x86_64-clang
```

するだけですし・・・。

>Ubuntu入ったUSB配布でいいじゃないですか。そのほうがインストールの手間もないし。

えっ


s/the //


Thanks


s/the //y


LGTM


@nemccarthy, thoughts on this?


This time, It was difficult to find a minimal example, but here is another bug:

```r
algoMBO = function(testfun, learner) {
  BUDGET = function(d) d * 40 #N_INIT_FUN(d)+20
  N_INIT_FUN = function(d) d * 10

  ps = smoof::getParamSet(testfun)
  d = sum(getParamLengths(ps))
  n.init = N_INIT_FUN(d)
  initdes = generateDesign(n.init, par.set = ps)

  # ### GENERAL (init, iters, y.trafo)
  ctrl = makeMBOControl(
    init.design.points = n.init,
    iters = BUDGET(d) - n.init
  )

  ### INFILL (crit, opt, interleave)
  ctrl = setMBOControlInfill(ctrl,
    crit = "ei",
    opt = "focussearch",
    opt.focussearch.points = 5000L,
    opt.focussearch.maxit = 2L,
    opt.restarts = 2L,
    interleave.random.points = 0
  )

  fun = makeMBOFunction(testfun)
  initdes = initdes
  assertDataFrame(initdes, nrow = N_INIT_FUN(d)) # if initdes is NULL we have a problem, autgen in next call then...
  mbo.res = mbo(fun = fun, design = initdes, par.set = ps, learner = learner, control = ctrl)
  list(mbo.res = mbo.res, d = d, ps = ps, opt = smoof::getGlobalOptimum(testfun))
}

lrn = makeLearner(id = "rf-noisy-bootstrap", "regr.randomForest", predict.type = "se",
    ntree = 1000, ntree.for.se = 100, keep.inbag = T, nodesize = 1L, se.method = "bootstrap",
    se.boot = 25)

set.seed(1)

d = 2
testfun = makeAckleyFunction(d)
a = algoMBO(testfun, lrn)
```

```
Computing y column(s) for design. Not provided.
[mbo] 0: x=-0.587,-10.1 : y = 17 : 0.0 secs
[mbo] 0: x=5.4,23.9 : y = 21.1 : 0.0 secs
[mbo] 0: x=-17.1,-21.4 : y = 21.3 : 0.0 secs
[mbo] 0: x=1.81,10.9 : y = 16.7 : 0.0 secs
[mbo] 0: x=-31,-27.4 : y = 21.5 : 0.0 secs
[mbo] 0: x=-26.9,-18.8 : y = 20.7 : 0.0 secs
[mbo] 0: x=26.3,-1.71 : y = 21.5 : 0.0 secs
[mbo] 0: x=11.4,-7.32 : y = 19.2 : 0.0 secs
[mbo] 0: x=18.8,-16.1 : y = 20.5 : 0.0 secs
[mbo] 0: x=-23.9,32.4 : y = 21.5 : 0.0 secs
[mbo] 0: x=-4.99,1.11 : y = 10.6 : 0.0 secs
[mbo] 0: x=-7.01,9.3 : y = 17.5 : 0.0 secs
[mbo] 0: x=7.99,4.41 : y = 16.1 : 0.0 secs
[mbo] 0: x=20.5,-25.1 : y = 21.6 : 0.0 secs
[mbo] 0: x=13.3,17.9 : y = 20.6 : 0.0 secs
[mbo] 0: x=29.8,29.1 : y = 21 : 0.0 secs
[mbo] 0: x=-15.3,22.5 : y = 21.8 : 0.0 secs
[mbo] 0: x=-21.2,14.4 : y = 21.5 : 0.0 secs
[mbo] 0: x=-10.9,-4.01 : y = 16.3 : 0.0 secs
[mbo] 0: x=24.3,-29.6 : y = 22 : 0.0 secs
[mbo] 1: x=-5.89,0.672 : y = 12.9 : 0.0 secs
[mbo] 2: x=-4.22,1.06 : y = 10.1 : 0.0 secs
[mbo] 3: x=-3.58,0.983 : y = 9.83 : 0.0 secs
[mbo] 4: x=-2.34,0.884 : y = 7.57 : 0.0 secs
[mbo] 5: x=-2.18,0.8 : y = 6.86 : 0.0 secs
[mbo] 6: x=-1.51,0.88 : y = 6.22 : 0.0 secs
[mbo] 7: x=-1.46,0.838 : y = 6.16 : 0.0 secs
[mbo] 8: x=-1.61,0.85 : y = 6.34 : 0.0 secs
[mbo] 9: x=-1.48,0.832 : y = 6.2 : 0.0 secs
[mbo] 10: x=-0.427,0.826 : y = 4.38 : 0.0 secs
[mbo] 11: x=0.374,0.821 : y = 4.24 : 0.0 secs
[mbo] 12: x=1.01,0.766 : y = 4.26 : 0.0 secs
[mbo] 13: x=0.608,0.725 : y = 4.6 : 0.0 secs
[mbo] 14: x=1.33,0.714 : y = 5.85 : 0.0 secs
[mbo] 15: x=-0.255,0.736 : y = 3.86 : 0.0 secs
[mbo] 16: x=0.821,-0.168 : y = 3.37 : 0.0 secs
[mbo] 17: x=0.812,-0.56 : y = 4.56 : 0.0 secs
[mbo] 18: x=0.854,-0.667 : y = 4.5 : 0.0 secs
[mbo] 19: x=0.316,-1.12 : y = 4.6 : 0.0 secs
[mbo] 20: x=-0.332,-1.5 : y = 6.16 : 0.0 secs
 Show Traceback

 Rerun with Debug
 Error in if (local.y < global.y) { :
  missing value where TRUE/FALSE needed In addition: Warning message:
In sqrt(apply(pred.boot, 1, var) - bias) : NaNs produced
```

I was not able to reproduce the same error with a smaller number of iterations/budget/focussearch.points. With smaller numbers I only get the warning, don't know if the warning is causing the bug.

```r
algoMBO = function(testfun, learner) {
  BUDGET = function(d) d * 5 #N_INIT_FUN(d)+20
  N_INIT_FUN = function(d) d * 1

  ps = smoof::getParamSet(testfun)
  d = sum(getParamLengths(ps))
  n.init = N_INIT_FUN(d)
  initdes = generateDesign(n.init, par.set = ps)

  # ### GENERAL (init, iters, y.trafo)
  ctrl = makeMBOControl(
    init.design.points = n.init,
    iters = BUDGET(d) - n.init
  )

  ### INFILL (crit, opt, interleave)
  ctrl = setMBOControlInfill(ctrl,
    crit = "ei",
    opt = "focussearch",
    opt.focussearch.points = 500L,
    opt.focussearch.maxit = 1L,
    opt.restarts = 1L,
    interleave.random.points = 0
  )

  fun = makeMBOFunction(testfun)
  initdes = initdes
  assertDataFrame(initdes, nrow = N_INIT_FUN(d)) # if initdes is NULL we have a problem, autgen in next call then...
  mbo.res = mbo(fun = fun, design = initdes, par.set = ps, learner = learner, control = ctrl)
  list(mbo.res = mbo.res, d = d, ps = ps, opt = smoof::getGlobalOptimum(testfun))
}

lrn = makeLearner(id = "rf-noisy-bootstrap", "regr.randomForest", predict.type = "se",
    ntree = 200, ntree.for.se = 100, keep.inbag = T, nodesize = 1L, se.method = "bootstrap",
    se.boot = 25)

set.seed(1)

d = 2
testfun = makeAckleyFunction(d)
a = algoMBO(testfun, lrn)
```

```
In randomForest.default(x = data[["data"]], y = data[["target"]],  ... :
  The response has five or fewer unique values.  Are you sure you want to do regression?
```


Worded the commit message wrong, meant to say that I want to only have a single instance of the Apocalypse class.


Too much logic for sending an email. Use Laravel's Mail facade


Почему?


これだと()が入らなくない？


@gut Is the problem here just that the output goes to a GPR? In x64, vasm cvtsi2sd converts to a DP-FP, not a SP-FP. When we use fcfids instruction we are converting to a SP-FP. In the same sense, it seems wrong to me that asm fcdids uses RegXMM in its signature: fcfids(RegXMM&, RegXMM& frb, ...) My understating is that RegXMM represents 128-bit registers, not FPR (64-bit).


謝摟！不知道之前為何把它安裝在裡面最新版拿掉了


Isn't this operation dangerous ? Can't it be a source of irregular PTS ?


杜宇类?


@yumetodo 開発環境と本番環境を合わせるのは基本中の基本だと…(そうじゃない)


Or in fact the same app, now, unless either item is a gone or redirect: https://github.com/alphagov/publishing-api/pull/141


Content IDs


感觉，这个老古董是不是可以考虑替换成https://github.com/Alamofire/AlamofireImage 或者 https://github.com/onevcat/Kingfisher


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


笔误，我改下。谢谢反馈


Indent these lines please.


Now that I think about it, it's probably better to make all of this GUI-based instead of command based. It'll make it much easier on server owners.


**Code Quality - Critical**

```php
$request->has('_token') && $request->session()->has('_token')  && $request->_token === $request->session()->pull('_token')
```
This is very very messy, why is it written this way?


Hello 這是我們這組的協同倉儲，未來creo畫的東西可以上傳到這邊


this change is just to sync up with the latest on master to avoid merge conflict.


Obiously, this commit also adds read/write support for POPM frames (for the ratings). I forget this to mention in the title.


خوب sdk V, hMNDJ ;K ND'I


these classes needn't be header-mostly, move to. cpp for object code reuse across module boundaries.


この記述簡略化しようか。(コントローラーネストさせてるからちょっと難しいだろうけど)

render @prototype  とかすると、
@prototypeにどこのモデルに属したインスタンスが入っているかをrailsが読み取ってくれて、
その名前と合致したviewファイル配下の同名テンプレートを引っ張ってきてくれる(ごめん理論は若干違うかも)ので

profyのquestion,answerのところとかを参考にしながら、
render: @~~~~~~~~
の記述だけで済ませるようにしましょう。


ぼくも始めeachを考えたのですが、placeholderのところが微妙かなと……
配列にして回すのも考えたのですが、あまりよろしくないかなということでベタで書いてしまいました。
結果論になってしまいますが、
tag_list[2]が存在しない=空欄
と処理されるようですので、編集段階でもタグを3つまでつけられるようになりますしこういう形に落ち着きました。


I think this adds unnecessary complication to the code. It is nice and abstract, but are we going to log with any other prefixes? I would keep the print_impl function, as it is handy, but I don't think we need a Logger class.

Let's discuss it offline.


@ikaruce @woongbinni  Sorry. for that


:sparkles: :camel: :boom:


Does adding this single declaration really do something? How?


"Address" has two Ds, and my username is `elliotcm`.


اینا کلا لازم نیست
این مال maven هست
تو الان از گریدل استفاده میکنی و اینارو نبایداضافه کنی


@popcornmix does this fix the issue you mentioned?


Hi Paul,

I think it makes sense to add one more parameter **executionId** to a method signature **countSubTaskStatesByUrlSourceType**. Because the collectionId is not unique, but the executionId is. In a current version of the method when I get the resulting map<<URLSourceType, UrlSourceTypeWithProcessingJobSubTaskStateCounts>>, it will be difficult to filter out only the entries that correspond to some specific executionId.
It was not mentioned explicitly in the document but we generate the reports not per collection but per executionId. It derives from the UIM page table, there might be multiple lines for one collection but with the unique executionIds.

Looking forward for your feedback,
Alena


changes between line 195 and 198 are just to sync up with the latest on master to avoid merge conflict. it has nothing to do with my fix.


به ریسایکل ویو آداپتیر رو ندادی
باید این رو اضافه کنی

        newsRC.setAdapter(mAdapter);


![image](https://cloud.githubusercontent.com/assets/15060223/11335171/073bdc88-918f-11e5-9a61-870055c99c4b.png)Christmas Wishes
Happy Valentine's Day 2016
Valentine Week 2016

Merry Christmas Wishes
merry christmas messages
merry christmas messages
merry christmas quotes


چرا کوچکتر مساوی؟
اکسپشن میده
کوچکتر خالی


やってみて難しそうなら相談して。
コントローラーネストさせてる関係で、俺もやってみんとどういう挙動になるかわからんから。


این جاش اینجا نیست


Agreed!


Which is actually not that important, what's more important is that the commit message should say what it does unrelated to the PR, as mentioned in the PR guidelines: https://github.com/alphagov/styleguides/blob/master/pull-requests.md#addressing-comments


این باید بعد از ساکسس فراخانی شه

mAdapter.notifyDataSetChanged();


I'd replace `none` with `isEmpty`


さていくつUSB買う必要があるんだ？(錯乱)

なんでLinux Mintではだめなのか(さらに混乱)


如何在java中声明数组的。


Thanks a lot for the clean-up!


To be clear what this change really means:

It protects private data from Google grabbing backups without user consent - really a threat. But this threat hits currently 0,0% of all users.

At the same time it takes away the one and only backup possibility for more than text from all non rooted devices - from the vast majority of all users. Read: if you value your chats including all media (and more chats than not in my case, and I think in most other's, contain images), what means you want to backup it (if you don't backup data, you don't value them), then Signal is not an option any more.

Unrooted (most) phones cannot backup any more ...

Unrooted (most) phones cannot migrate to another phone any more ...

And it is really impossible, not only hard - the only way to to keep your chats with more than text is to root your device, then e.g. TitaniumBackup will help you. But if rooting is not an option you're lost.

I see only two possibilities to remedy this situation: either provide (again) a complete encrypted backup (it once did work flawlessly in most cases), or limit this change to API 23.


They seem to work client side

I see a related conversation https://github.com/PocketMine/PocketMine-MP/pull/3707#issuecomment-158071155 - its seems that 9 slots match the nbt data format and clients that use 7 just don't use the extra 2. So the server allocating 9 slots in all cases should work fine? I'm going to test later on with both an Android device and win10 and see if that's true


paramsの中はこのような形でした……
今出先なので、帰ったらまた確認します！


Is this sysout on purpose?


Здесь вылетает исключение на случайном вопросе:
*Converting the Json result to a GoogleResultSet failed*


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


Same here, sysout on purpose?


@e-kolpakov

> WISYWIG editor adds a lot of noise to vector config - it might make sense to skip adding default values

A quick note about this: Functionality for modifying vectors on the board that will be introduced as part of the follow-up task for this will include controls for specifying a name and label for individual vectors, but it might not include controls for editing all properties of a vector. Based on this it makes sense to add default values to the JSON representation for vectors right away, as it allows authors to customize, e.g., the style of individual vectors by simply changing the default values of individual properties (instead of having to write the JSON from scratch themselves.


请告诉我你的IKVM版本


Previously we've added comment above this logic, so I think this comment is a marker that logic is not clear by itself. The way for logic clarification is introduction new method. It's Martin Fowler's recommendation. Please look at this: http://www.refactoring.com/catalog/decomposeConditional.html


わざわざhtmlから変えにいかなくても、今回の場合はtextの変更で事足りるから、
$('#comments').text(("<%= @prototype.comments_count %>")); とかでいいかもね。


but here we can provide better/separate message


This should be one line of code like so:

```php
User::find($user_id)->updateProfile($input);
```


LGTM. Thanks!


This is fine, nice progress!


What about cellNeighborsAmmount?


Do you have a reference for this information? I only found https://bugzilla.mozilla.org/show_bug.cgi?id=1058311


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


Use `redirect()` helper function instead of `Redirect` facade


@itsjeyd :+1: for this pair of commits.


OK


それはメンバー関数ではなくフリー関数として作って第一引数に構造体を渡すということだろうか


Where is this property used?


This commit introduced a bug where the text in the input box in the IM-module is invisible... Good luck @hffahle


No was just for debugging.


processedNodes muss ein Array sein mit Länge Anzahl Threads oder meinetwegen ein komma-separierter string


Was just for debugging.


Job done.


In the verticle sending the metric.


Thanks for the update @cescoffier


Done, in JmapServer itself, as we want to test a single feature in those integration tests.


Pls, try to convert these examples into proper tests ASAP and fix ASAP.

We must rerun the exps soon to have a chance


@yumetodo 局長が「お金ならいくらでも…」と言っていたのでゴネれば (以下自粛) まあUSBが無駄になるのは良くないので各自用意させるべきですかね。イメージを１つ作ればどうせこのコマンドで一発でしょ。
```
dd if=ubuntu_for_lecture.img of=/dev/the/path/of/usb/storage
resize2fs /dev/the/the/path/of/usb/storage
```
単純な話、私はUbuntuのDevelopmental Branchを使っているので、Linux Mintになるといろいろとバージョンが古くなってしまうはずです。せめてUbuntu。~~願わくは受講者をUbuntuユーザーに。~~


I was thinking of that, but if something breaks cassandra-guice, then the jmap integration tests will be broken.


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


Ferocity.Opressor not .Sorcery


@kof Maybe include rule name only in development mode? (`process.env.NODE_ENV !== 'production'`)?


No it's a bug, should be `insert-timestamp`


Thanks. That's why I like unit tests that can be run under Tcl for comparison.


Maximum number  of neurons in ranking


The original code was naive about `elem` being a valid jQuery object. To address this we can either take the hit of trying to wrap `elem` as a jQ object regardless or check if it is one before attempting to run jQuery methods against it.


My thoughts are "I hate our data model" :-)

But I also agree with Pavel's analysis and if anything am inclined toward the `cgroups` option.


Applied. Thx!

btw, if you want this to go to the 3.3 release, we suggest to move it to community.


Ok, thanks.


Change description to "WPF"


@lrusak, thank you for answering.
Who took the place instead of @stefansaraev ? Or should I open the Issue?
About libXi, I'm asking because 'xinput' use it


Please replace all instances of `Redirect` facade wit `redirect()` helper function


Mismatched indent.


Missing indent.


ok,  coerce all inputs if the dim(X) or dim(Y) == 1 at the beginning of checkingXY


恩 这个是肯定要换的，如果你有兴趣参与，欢迎PR~


of course, it's what "integration" means, don't you think ? We should refrain from having a lot of application wiring for tests as they will diverge from actual server usage with time.


Extra `})`?


Why don't you test the duration of the other streams ?
In this cas, we could use a list of input files and properties, instead of variable suffixes! Something like:
```python
inputFileNames = [
    os.environ['AVTRANSCODER_TEST_VIDEO_AVI_FILE'],
    os.environ['AVTRANSCODER_TEST_AUDIO_WAVE_FILE'],
    os.environ['AVTRANSCODER_TEST_AUDIO_MOV_FILE']
]
outputFileName = "testEProcessMethodBasedOnStream.mov"

ouputFile = av.OutputFile( outputFileName )
transcoder = av.Transcoder( ouputFile )
transcoder.setProcessMethod( av.eProcessMethodBasedOnStream, 1 )

for inputFileName in inputFileNames:
    transcoder.add( inputFileName, 0 )

progress = av.ConsoleProgress()
transcoder.process( progress )

# get src file
src_inputFiles = list()
for inputFileName in inputFileNames:
    src_inputFiles.append(av.InputFile( inputFileName ))

# get src file properties
src_inputFiles_properties = list()
for src_inputFile in src_inputFiles:
    src_inputFiles_properties.append(src_inputFile.getProperties())

# get dst file
dst_inputFile = av.InputFile( outputFileName )
dst_properties = dst_inputFile.getProperties()

for streamIndex, src_inutFile_properties in enumerate(src_inputFiles_properties):
    assert_equals( dst_properties.getStreamPropertiesWithIndex(streamIndex).getDuration(), src_inutFile_properties.getDuration() )

```


There are 3 conditions right there.
I could choose to cascade the if statements and put 1 condition in each.
What do you think?

On Mon, Nov 23, 2015 at 12:03 PM, Prosper Otemuyiwa <
notifications@github.com> wrote:

> *Code Quality - Critical*
>
> $request->has('_token') && $request->session()->has('_token')  && $request->_token === $request->session()->pull('_token')
>
> This is very very messy, why is it written this way?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andela/pibbble/commit/f84ceb0fc4d7272355abf890fe8f933d0b150547#commitcomment-14557077>
> .
>

--
*Opeyemi Ogunjimi | Software Developer | Andela*
(c) +234-703-936-2475 | Andela.com <http://andela.com/>


Looks good from my side but please do next:
1. Each time when you creates a pull request or sending just a commit - scroll down and review your code on your own. Git helps you to find all files that ends not with empty line. That is no difficult.

2. Change this line:
+var searchApp = angular.module('searchApp', [uiRouter, uibs, 'searchApp.Search', 'searchApp.Cart', 'searchApp.Tags']);
with this:
+var searchApp = angular.module('searchApp', [
    uiRouter,
    uibs,
    'searchApp.Search',
    'searchApp.Cart',
    'searchApp.Tags'
]);
It looks much better.

3. If we decided to use 'configCb' please use it every time. That is not applicable when 1 module uses .(config(function(){}); but second one uses .(config(callbackName));

4. Please, set-up your code style. I see that file 'angular-spa-ui/package.json' has only one difference. But the file has been overriden at all.


The expected file contains '\r'. I added replaceAll for any additional occurrences with this issue maybe on actual files also.


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


I don't think it is a problem of registering several callbacks (we can do it already in the session service)

The problem is that different SwellRT syncs may require user feedback or not, depending on the nature of the change. Example: it is not the same saving the pad after an edition, than tracking the last view to the pad. In the first one, the user must know her edition is correctly saved and will not be lost. The last one should be transparent to the user.

Or maybe I am missing something..


The "remove the trailing 's'" part is very hard to get right (what about `setCopies(List<Copy>)`? do we break with non-English names?). It sounds like too much magic. If we want to allow this, we could do it something like this:

```groovy
@Managed
interface Thing {
    List<String> getTargetPlatforms()
    void setTargetPlatforms(List<String> targetPlatforms)
    @AppendTo("targetPlatforms") void targetPlatform(String targetPlatform)
}
```

The method would still be auto-generated, but you'd have control over its name.

Not sure if we want to support this in the general sense, though.


I'd make it `inline`


 or would  $request->_token === $request->session()->pull('_token') alone
just do it?

On Mon, Nov 23, 2015 at 12:20 PM, Opeyemi Ogunjimi <
opeyemi.ogunjimi@andela.com> wrote:

> There are 3 conditions right there.
> I could choose to cascade the if statements and put 1 condition in each.
> What do you think?
>
> On Mon, Nov 23, 2015 at 12:03 PM, Prosper Otemuyiwa <
> notifications@github.com> wrote:
>
>> *Code Quality - Critical*
>>
>> $request->has('_token') && $request->session()->has('_token')  && $request->_token === $request->session()->pull('_token')
>>
>> This is very very messy, why is it written this way?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/andela/pibbble/commit/f84ceb0fc4d7272355abf890fe8f933d0b150547#commitcomment-14557077>
>> .
>>
>
>
>
> --
> *Opeyemi Ogunjimi | Software Developer | Andela*
> (c) +234-703-936-2475 | Andela.com <http://andela.com/>
>

--
*Opeyemi Ogunjimi | Software Developer | Andela*
(c) +234-703-936-2475 | Andela.com <http://andela.com/>


Please, use multiple lines! :)
```c++
_wrappedPacketsDuration += av_rescale(
    1,
    audioPacketDuration.num * (int64_t)_outputAVStream.time_base.den * _outputAVStream.codec->ticks_per_frame,
    audioPacketDuration.den * (int64_t)_outputAVStream.time_base.num
);
```


@mdeniz Looks like this reverts changes made in e5016964461c65e678a . Could you squash both commits. So we have a clean commit history?


`doAppend` has been introduced for mixing CCTAG and SIFT with `SIFT_CCTAGdescriber`. If set to true, instead of allocating the space for `Regions` it just append the detected CCTAG to the `Regions` passed as parameter (in `describe()` method of the class). Actually when using only CCTAG it should be set to false.


Is this something to fix in the future ?


That commit looks good to me (just to give you preliminary feedback and keep you motivated despite my limited time)


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=2)


まあ話がそれてるので戻すとして、一応さらっとcommand line argumentにふれて、それからHello　Worldに行きましょう。


I'm last? :D


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


そうですね。まずはそれを書かないと。


Pretty cool :package:


If we have large elseif constructs it is ok to use switch case constructs for better reading of the code. But I think we should not generally change to switch/case. Switch/Case will be compiled as if/else constructs internal. Switch/case is 63% slower than using if/else constructs.


What's the reason for the timestamp?


this line confuses me more than it helps. what do you mean by it? will the client always overwrite those options? or only under specific conditions? or are you thinking about adding a feature in the future to make it overwrite those?


どっちが書きます？私に書かせると間違いなくffmpegがでてきますが。


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=4)


You dont need to check if it has..

Checking if the _token equals the one from the session is okay, but why are you storing _tokens up and down in the sessions?


зачем тогда это вообще?


I guess it is something like we override the `exception` option in case of guzzle to make sure no unwanted exceptions are thrown.


@yumetodo `ls`にして_お願いできますか_。みんな持ってるでしょうし（素のWindowsには黙ってもらう）


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=5)


i checked timem table after couple of rounds of requesting and the table is empty , so there is no resource leaking:
---> Active tables <---
timem          : with 0        records occupying 299      words of mem
monitor        : with 0        records occupying 299      words of mem
schema         : with 3        records occupying 663      words of mem

perhaps changing state to resend =‘false’ results with necessary request clean-up

Regards,
Igor

> On 23 Nov 2015, at 12:49, Manuel Rubio <notifications@github.com> wrote:
>
> well, resend there is the configuration param so, if you want to use resend and do that, you're turning off the resend param. Maybe you need to remove the #matching{} entry from timem part instead to avoid to resend it again. To do that you'll need the namespace NS param and the ID from the message.
>
> —
> Reply to this email directly or view it on GitHub.
>


@petermelias
Just a quick note regarding [your fork](https://github.com/petermelias/django-configurations): it would be better to cherry-pick commits, instead of just applying them like you've done in https://github.com/petermelias/django-configurations/commit/cebbae2813044f0b2a821b7f1d44ed0b423ed7bf.


这里加个字符串的原因，是避免变成数字的加法么？


一般来讲，你可以通过-cp将配置文件所在的路径加入classpath：

D:\app>dir /a
 D:\app 的目录

2015/11/23  19:28    <DIR>          .
2015/11/23  19:28    <DIR>          ..
2015/11/11  19:45           742,815 hanlp-1.2.7.jar
2015/11/10  22:31             1,513 hanlp.properties
2015/03/01  12:49                95 SetPathAndRun.cmd
               3 个文件        744,423 字节
               2 个目录 121,755,140,096 可用字节

D:\app>D:\Tools\ikvm-7.2.4630.5\bin\ikvm  -cp .;hanlp-1.2.7.jar com.hankcs.hanlp
.summary.TextRankSentence
[无限算法的产生是由于未能确定的定义终止条件, 这类算法在有限的时间内终止, 有限的
非确定算法]

D:\app>


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=7)


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


Thanks.


Use `errors.New(...)` if you just need an error string


How about

"TextSearch returns information about a set of places based on a string."


This breaks at least one of the existing tests. I'm not sure how it made it through. I'll submit a PR for it.

```
ERROR: test_get_packages (tests.test_download.DownloadCommandTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/atodorov/dnf-plugins-core/tests/test_download.py", line 280, in test_get_packages
    pkgs = self.cmd._get_packages(['notfound'])
  File "/home/atodorov/dnf-plugins-core/plugins/download.py", line 136, in _get_packages
    raise dnf.exceptions.Error(e)
dnf.exceptions.Error: No package notfound available.
-------------------- >> begin captured logging << --------------------
dnf.plugin: ERROR: No package notfound available.
dnf.plugin: ERROR: Exiting due to strict setting.
--------------------- >> end captured logging << ---------------------
```


you can not open issues here. but you can do what you need yourself, and then open a PR.


커밋과는관련은없습니다만.
불머리작업시작하시는거같은데요
옵쥐쪽은손때신건가요?


Code is fine. I wonder if we should put it in ``Condition`` and alias it down to the root package?


I think if we're going to re-write this we should not add the unnecessary layer of option types. It could be written by matching against the exception from the string module as the termination condition for the recursive call. Something like this:

```ocaml
let split c s =
  let rec inner acc i j =
    match String.get s j with
    | ch when ch = c -> inner (String.sub s i (j-i)::acc) (j+1) (j+1)
    | exception Invalid_argument _ -> String.(sub s i (length s - i))::acc
    | _ -> inner acc i (j+1) in
  inner [] 0 0 |> List.rev
```

However I'd be in favour of pulling in https://github.com/rgrinberg/stringext for this since it's already in the build system and we shouldn't really be writing such general purpose functions.


いや書くとすればUnix/Windows併記にします。マルチプラットフォームな参考書にならないので。``dir``と``ls``、``grep``と``findstr``らへんを組み合わせればそれっぽくなるかな。


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=8)


Ah, whoops... @nickbristow forgot to pull out the `:path` argument after working locally to produce 18F/guides-style#17. You should totally feel free to send us a fix!


from `CCTAG_Image_describer::Describe()`
```
    if ( !_doAppend )
      Allocate(regions);
    // else regions are added to the current vector of features/descriptors

    // Build alias to cached data
    CCTAG_Regions * regionsCasted = dynamic_cast<CCTAG_Regions*>(regions.get());
    // reserve some memory for faster keypoint saving

    if ( !_doAppend )
    {
      regionsCasted->Features().reserve(50);
      regionsCasted->Descriptors().reserve(50);
    }
```
I think skipping what's inside the ifs may be dangerous when just using CCTAG (instead of SIFT_CCTAG)


Error strings should not be capitalized

https://github.com/golang/go/wiki/CodeReviewComments#error-strings


then we should say that the client ignores those two options, forces them to false|true|whatever and point to the respective plugins to use.


And whoops again! Repeating for @ericcf:

Ah, whoops... @nickbristow forgot to pull out the :path argument after working locally to produce 18F/guides-style#17. You should totally feel free to send us a fix!


@yumetodo Windowsは知らないのでお願いします。


I'd like to have valid annotations on file, so I would have to do the change in 2 commits and as kotlin here is not of much help I would skip the second part, sorry


Upstream: https://github.com/WrinklyNinja/libloadorder/issues/25


Ok, then I missunderstood the problem.

You can cross the information from SwellRT with a flag saying if the user is editing, e.g. the flag will be true when an user is editing a pad or sending a chat, but not when navigating.


mmmm... I'll check it just in case :-) I guess the tests should be working on this but it's possible the specific case is not in the current tests so, it should be improved at that part first. Let me check and I'll let you know.


Could we call it "rewrap" instead of "add" ?


わかりました。


Upstream: https://github.com/WrinklyNinja/libloadorder/issues/24


what is this? :)


1)10 строка -> в классе принимали const char *s
2)80-88 строки. в твоей программе не предусмотрено. но если бы str не была создана, ты ее сразу удаляешь. (см. как в деструкторе)(т.е. если бы assign была первой вызываемой функцией)
3) "*" и пробелы
4) деструктор логичнее было бы расположить в конце класса?


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


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


Fixes #8.


Great work guys! This will help me and a whole bunch of other raspi noobs out.


Thought it'd be wise to tell you before you make a PR. You do need some sort of max_length. There has to be a limit but it can be higher than 200. No limit means someone can put a very, very large string in your database.


OK I found https://bugzilla.mozilla.org/show_bug.cgi?id=1130684 which references v41.

@karolklp Why are you referencing v42? Is there a bug I'm not aware of?


Some of my packages provide variant functionality depending on what other packages are installed. The easiest way to test if a package is installed is to try and load it, then check if an object is created. The above change leads to large numbers of logged warning messages on any code that uses such a test for a package being available.


It just needs to be declared before using the variable, not before including the toolkit (e.g. `$toolkit-font-stack: $Helvetica-Regular;` should be valid).


you forgot to update the ignore alias


What is the test here ? Just to check if it processes without errors ?


It seems a good idea, a directive such as `save-feeback`


O nome da tabela é "Cliente"


Mapear como String


I'd leave it there since there is only likely to be a single one of these and it's much more closely related to `f:if` and `v:if` than the specialised conditions and should be similar to those in location - imho :)

That being said, it would be pretty damn nice if the condition viewhelpers themselves would be a bit smarter and would check if `condition` argument is used; if not, take the value of `condition` from `renderChildren` and return a simple TRUE/FALSE based on the evaluation, allowing us to do:

```
<v:unless condition="{suspectArray -> v:condition.type.isArray()}">
   Nope, wasn't an array.
</v:unless>
```

But honestly, then it's not that much harder then to write the `<v:condition.type.isArray condition="{suspectArray}" else="..." /> ` or tag form.


Nome da tabela é Item


Don't worry, be happy! :D


Utilizar BigDecimal


Shouldn't this be controlled by raise_oom as well?


comment removed in the next commit


Hm.
1) Initialization of non-const pointer `optional_object_` from `const T*`?
2) What is the use-case of creating non-owning `Optional`s? At first glance, it looks dangerous, since I can delete an object and `Optional` would still think that it exists.


Entrust now conflicts with Laravels soft delete trait because of the restore method in this class.
It now throws the following error - Fatal error: Method 'restore' declared in multiple traits


@yumetodo そういうことです。クラスみたいにドット演算子で関数呼べないですけど。


What is the reason for this? I mean it should not be necessary in Nette 2.3 as far as I know.


Adicionar @Temporal


Okay, because I didn't really know about Win10 until now lol


typo in `ckashes`


I'm creating a separate issue for this so I can investigate how to do it properly.


Using dot as the thousands separator will be confusing to non-European readers. Google's docs use a space. How about this phrasing:

"radius is too large, a maximum of 50 000 meters is allowed"


i only changed long elseif. I read somewhere that switch/case is faster than if/else if there are more than 3 cases. Mybe this was a javascript comparison.
For **me** switch case is better readable if there are many similar cases


IllegalStateException


schedule和scheduleAtFixedRate的区别在于，如果指定开始执行的时间在当前系统运行时间之前，scheduleAtFixedRate会把已经过去的时间也作为周期执行，而schedule不会把过去的时间算上。

比如

SimpleDateFormat fTime = new SimpleDateFormat("yyyy/MM/dd HH:mm:ss");
   Date d1 = fTime.parse("2005/12/30 14:10:00");

   t.scheduleAtFixedRate(new TimerTask(){
    public void run()
    {
        System.out.println("this is task you do6");
    }
   },d1,3*60*1000);

间隔时间是3分钟，指定开始时间是2005/12/30 14:10:00，如果我在14:17:00分执行这个程序，那么会立刻打印3次

this is task you do6      //14:10
this is task you do6      //14:13
this is task you do6      //14:16

并且注意，下一次执行是在14:19 而不是 14:20。就是说是从指定的开始时间开始计时，而不是从执行时间开始计时。

但是上面如果用schedule方法，间隔时间是3分钟，指定开始时间是2005/12/30 14:10:00，那么在14:17:00分执行这个程序，则立即执行程序一次。并且下一次的执行时间是 14:20，而不是从14:10开始算的周期(14:19)。


### Additional changes I forgot to mention:
- game is ended if the number of players falls below 2.


Noticed by accident: there's a typo in all occurrences of 'delimeter'. Should be 'delimiter'.


OK I figured out that the commit to blame is https://github.com/ua-parser/uap-core/commit/98d29ff4dbab665e380ef0c7eaeeecb6afce425c

I created https://github.com/selwin/python-user-agents/commit/ed386c818ff7a10849348711d4bdd00506088702 to fix this.


updatet!


This would be better as a complete sentence:

"Do performs the TextSearchCall request."


Maybe

"TextSearchCall represents a call to the Text Search API"


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


I see that Travis is unhappy but I assume that this is an ongoing aspiration to achieve pylint nirvana. As it is, I believe that this line is not actually pylint compliant so maybe we shouldn't add to the mess. I believe (happy to be corrected) that this line should be indented to line up with the open bracket or all arguments on new lines indented by a tab width.

However, I don't want to be "that guy" and it's your team's codebase. I can vouch for the semantic correctness of this log line.


"... in streaming data sources"
so it's consistent with the one in livre/core/data/VolumeDataSourcePlugin.h


I'd make `get` abstract and implement it in each class


When you declare a `Shiboken::GilState` instance, it ensures the GIL and when it gets destructed at the end of the function it releases it again

https://github.com/PySide/shiboken2/blob/master/libshiboken/gilstate.cpp

Took me a sec to figure out what it meant too, the PySide source is full of `Shiboken::GilState state` declarations


It should depend not only on the blockNumber, but mainly on the type of the transaction. Only ContractCrationTransaction cost 53000 gas, normal transaction still cost 21000)


we could do that, the question is if the rule name in class somehow bad ... I didn't want to have it before and used debug plugin just because of stylistic preferences ...


@iancrowther  - it looks like you are changing files that are managed by the build process - these changes will be overwritten by a build, which uses the [templates/index.html](https://github.com/iancrowther/lnug.github.io/blob/master/templates/index.html) file. Same for `sponsors.html`


sdfsfsd


> The expected file contains '\r'. I added replaceAll for any additional occurrences with this issue maybe on actual files also.

If the test case passes without `found = found.replaceAll("\r", "")`, then I am ready to accept this part of this PR after you remove `found = found.replaceAll("\r", "")`.

If the test case is not passing without `found = found.replaceAll("\r", "")`, then we need to investigate how it happens that `found` contains `\r` characters and fix the location where those `\r` characters are being added.


Missing bracket on the end here I think :)


`java.util.logger` ->  `java.util.Logger`


Sure, that sounds fine.


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


In fact, This commit is for notifications in workbench plugin related to #668. My mistake.


This is also done at the end of this branch...


Reducing function complexity is generally a good thing, but I think this function is too granular to ever be reused. It would probably be clearer to include inline.

Maybe it would help to use a different abstraction? I could see the FeatureType stuff being its own function:

```
func typeNameString(types []FeatureType) string {
	var typeNames []string
	for _, t := range types {
		typeNames = append(typeNames, string(t))
	}
	return strings.Join(typeNames, "|")
}
```

That way you don't have to construct a url.Values to use or test it.