> Documents does not work for workers, where this may or may not become a thing.
>
> Can't we just say that it "can be used to express ..." without addressing anyone?

That would be fine by me. But I leave it up to Ted if he thinks just keeping it as addressed to authors is more clear.


Thanks for suggestion, seems that'll be more feasible for control testcases.


Sounds good. I'll make those changes before I submit a PR for this.


There's UserIds.create(String) method that wraps this.


Yes. I'm not done with the changes.. I suspect `<T extends Criteria> T getCriteria()` will suffice


103122604-ClimaticDataSummaryBug (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1857 of 1857 tests passed)

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


A lot of this will need to be changed


No, the naming convention of the output wrapper should be the same as what all the other target languages use, that is xxx_wrap.cxx, not xxxMATLAB_wrap.cxx. If a users wants to change it, then they can use the -o option, as with all the other languages.


`getWritePath()` already returns a `String`


It works only because IDS_FOO has the same value as ID_FOO.
 IDS are meant to refer to strings, not commands.
Also there is one more afxres.h in dasher.rc


1853_fix_waterheater_stratified (mbadams5) - x86_64-MacOS-10.9-clang: Tests Failed

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


Why these packages in dependencies not in dev? 11 mb! Fix please


@aibaars That depends on what we expect from the interface of the package and if there is value in exporting a package specific type. If we aren't using it, then it doesn't make sense to export `Driver`. I didn't write the original S3 driver, so I am not sure why we exported the type.


102941444_CorrectBSDFs (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2291 of 2337 tests passed)

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


Great!


103122604-ClimaticDataSummaryBug (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


One comment on this commit. The fitting base class `BaseFitter` is the base class for *any* fitter. So putting options, like `replace_cov`, here doesn't make much sense since on could have other kinds of fitters that do different things. Since this is specific to the ngmix bootstrappers, these options belong in `NGMixBootFitter`. A fitter may not even have models, so I think all of these options should be moved to `NGMixBootFitter`. I am personally exempting this from our no bike-shedding policy, since these seems like a fundamental design issue.


hi , this project working fine for world of tank? because i am playing this game and afraid if with hacks they find and ban me ^^ , it works like wow?^^


This pattern is fundamentally weird, and I don't really like documenting it. WSGI environ vars are per-request, it's strange and inefficient to re-set process-global OS environ vars based on them on every single request. I also wonder (but am not sure; Graham would know) if this technique could cause strange race conditions if you fail to use daemon mode with a separate process group per site, because it would actually be changing the environ for _all_ your sites on each request to any of the sites.

IMO the right way to set environment variables in Apache is the `envvars` file. That doesn't work if you're serving multiple Django sites from a single Apache instance. In that case I think it's better to just direct each site to a different WSGI script file, which sets the env vars it needs once at startup to fixed values.


That's clear.  `xxx_wrap.cxx` is the name of the source file. But what about the shared library compiled `xxx_wrap.cxx` is compiled into? For Python it appears to be `_xxx.so`. Can it e.g. be `xxx_wrap.mexmaci64`? Should it match the name of the c++ file?


Please do the following:
1.  Define a method Commands.createContext(UserId, ZoneOffset) that would create the context with the current time and passed parameters. Create new context each time you pass a command.
2. For ZoneOffset create a ZoneOffsets class and define UTC timezone. Use it as a parameter in this test.
3. Create a task for adding more zone offsets in Projects. +mention me.


This file has whitespace issues, make sure to respect the original code: tabs everywhere.

Also, add an EOF line after the `};`.


Still testing


4967-SZRH-spm (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1843 of 1861 tests passed)

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


Could you also remove the instances where we check for isValueSet() then setValueToDefault() in ShiftValueRecycleViewAdapter for each type (line 142, 158, 184, 218)? I made a mistake on adding those instead of what you have here.


This file has whitespace issues, see [my other comment](https://github.com/astorije/shout/commit/aad7a66f40f180310cc2d4d22cce2a51a3650c9d#commitcomment-13192887).


Sorry about that! I realised this initially but I must have carelessly changed the extensions at some point anyway.


rename to accept_id_in_constructor()

We may have other types for aggregate roots. See JavaDoc for the latest version of AggregateRoot. Have tests that cover all supported parameter types.


102941444_CorrectBSDFs (rpg777) - x86_64-MacOS-10.9-clang: Tests Failed

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


Rename to rename to accept_String_id_in_constructor()


rename to accept_Integer_id_in_constructor()


rename to accept_Long_id_in_constructor()


Remove this, not used.


Why autoload function need to be disabled in vim_starting? My vim configuration breaks on this commit.


Oh boy, inheritance in JS... I wouldn't bet an arm on this, but pretty sure this way of doing it is [a bad practice](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Inheritance_and_the_prototype_chain#Bad_practice_Extension_of_native_prototypes). Could you double-check with that page? Maybe use of [Object.create()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/create) instead, if applicable?


why are you doing this check?


I got it to work with a loop, although it still doesn't work with negative numbers:

`function factorial(num) {`
    `var factNum = 1`
   ` for (var i = 2; i <= num; i++)`
  `      factNum = factNum * i;`
 `   return factNum;`
`}`


Or [this](https://nodejs.org/api/events.html#events_inheriting_from_eventemitter) maybe?


Not sure why you require `Client` to be an `EventEmitter`, could you explain me please?


You do not need to do this. The difference of num1 and num2 is always num1 - num2.


Sorry my bad. :) you're right sir, moving them now


No need to do that. Keep the order the numbers were given in. The answer is much simpler!


OK, just thought I'd check, never know when help my come in handy. BlackDragon and I work together on many things so if you ever need any help just let us know, we'd be more than willing


Gotta be honest, every time I click a button in Git I'm convinced I'm about to break something.


This is great :) It doesn't need to work with negative numbers :)


fixes #24


Please run gofmt on your code.  This will (amongst other things) order these headers correctly.


make this URL a const


such as [0,quotes.length] ??


check and return the error


What lol?


Why exclude negative numbers? is (-1 - 5 = -6) not a valid difference?


У меня верстка поплыла.


Eigentlich sollte mit dem 2. Parameter dotrigger aufgerufen werden.
Funktioniert das denn jetzt?

Wo kommt denn der Wert Taupunkt her?


develop (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


102941444_CorrectBSDFs (rpg777) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


I think this fixes it:

https://github.com/walshb/pyavroc/commit/d4a9df0033e6a64eaef846c49ba33570092b030d


The bug is with WindowsPhone  If you navigate to a view, then PopModalAsync, an exception is thrown.  I spoke with Xamarin and this is supposed to be fixed in an up coming hotfix.


Great job!


Do you have curl bindings for dos?


What happens if I run "longestWord("yellow, are the candies")" ?


Its the new and improved naming scheme for gitignore files. With 100% more ognore


res.Body.Close()


посмотрю завтра с утра


Exercise : "returns the same phrase with every word capitalized"


So - if I'm moving the manifest processing off the main thread - what should I respond with?

I don't wan't to create a manifest object in the DB before checking if the URL actually leads somewhere nice.
I can't return a pk for the front-end to eventually redirect too (to check if the import was successful) without creating a manifest in the DB.
The importer needs some kind of useful response in the case that the import fails.

How can I get feedback on the process to the front end?


so how do i help edit or something?


И не забывай про `git pull` перед тем как начинать работать с репозиторием.


И шрифт страшный. :)


And let's throw in a screenshot of the current situation :)
![image](https://cloud.githubusercontent.com/assets/11202073/9827721/74eeb666-58e5-11e5-9874-475ce6aab79f.png)


I am using method 2 in the rflink loader program which functions under Windows 10..
Note that for serial ports > 9 you need to have the slashes in \\\\.\\COM%d


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2303 of 2342 tests passed)

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


OK, feel free to move things around


Cool.


If I see it correctly, the Ivy parser does not create constants (apar from nil) anymore. How are input constants created now? e.g. in the term f(a,v0) I'd expect a to be a constant.


return from the function here and remove the else


вот Pull я и не сделал перед правками, потом мержить пришлось, а шрифт такой мы использовали на каком-то сайте, можно и нужно поменять потом


И настрой редактор на использование двух пробелов для отступов.


@staticinteger Should rename this commit to "There and Back Again: A Node Tale." Thanks for updating the README too — awesome to have.


Hi guys,

I updated my html version. Slapped a "clear" button that simply clears the current counts and entry point in one step as Jason and I discussed the other night. Also add a basic info firm and a menu button.

I also modified the way I display the traveler counts to get rid of the absolute positioning I was doing.

As always, I look forward to your comments and suggestions.

Regards,
    Bill Leddy

> On Sep 11, 2015, at 8:40 AM, Jason Ody <notifications@github.com> wrote:
>
> @jesserosato Bill has sent me some HTML and jQuery code that he's currently working on so I'm thinking of giving the jQuery only route a go (thus the other jQuery branch). I expect that it will only be a bunch of click bindings, managing some state and posting the data. It should be sufficient for what SABA will need for their count in November and will keep things as simple a possible...initially. I expect that once/if we want to include offline storage and the ability to customize what you counting (plus anything else), we'll want to move to using a front end framework.
>
> —
> Reply to this email directly or view it on GitHub.
>


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 595](https://teamcity.labkey.org/viewLog.html?buildId=271628&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:21:57
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Надо будет стайл гайд создать видимо.


Is it really that hard to write this as a switch?


Forgot to mention. The html version is at bikeandwalk.org/v2/

Regards,
    Bill Leddy

> On Sep 11, 2015, at 8:40 AM, Jason Ody <notifications@github.com> wrote:
>
> @jesserosato Bill has sent me some HTML and jQuery code that he's currently working on so I'm thinking of giving the jQuery only route a go (thus the other jQuery branch). I expect that it will only be a bunch of click bindings, managing some state and posting the data. It should be sufficient for what SABA will need for their count in November and will keep things as simple a possible...initially. I expect that once/if we want to include offline storage and the ability to customize what you counting (plus anything else), we'll want to move to using a front end framework.
>
> —
> Reply to this email directly or view it on GitHub.
>


Typo here.


These solutions are meant to be given out the students. Just that we want to wait until after the Monday session so that people have an incentive to show up.


Can you please add a verb?


+1 para remover

Em sex, 11 de set de 2015 17:58, Héctor Velarde <notifications@github.com>
escreveu:

> eu não quero fazer as coisas de meu jeito, simplesmente estou te falando o
> que eu acho; se todo mundo concorda, removemos e problema resolvido.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plonegovbr/brasil.gov.vcge/commit/c45345f4b91a1e2fc77b1a3e95aff3baf0f4055c#commitcomment-13191813>
> .
>


Is `setFields` really a method that will get called on this registered object? If not, feels like you should move this method out of this object or just pull this logic down into the `execute` function.


FWIW @tarruda mentioned this is a bug and shouldn't be exposed in the api-info metadata.


Agh I don't know how I missed your comment! And I just pushed a beta to npm less than a minute ago, haha.


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


This mapping makes me sad :(


@abarth, any reason why the top end of the version constraint is so narrow? `">=0.12.4+5 <0.13.0"` would be a bit wider, and very narrow constraints can cause problems for consumers of the package -


Is there a specific error returned for `stat`ing a bucket?  I'm wondering if there is an alternate to calling `ListObjects`...


Is this method (`onFieldChanged`) really called when fields are changed on the form? Something feels weird about some of the methods on this object (everything but "execute" really). I wouldn't generally think of the "handler" for a menu action also being notified about changes on the work item form. You might consider an new contribution type for "work item form handler". This would allow a developer to hook into events on the work item form w/o having to contribute a UI element (like a tab or toolbar button).


че-то мне не удобно 2-а пробела, я всегда использовал 4-е
а чего такое стайл гайд?


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@ClaytonJY : This should close issue #30.
@sbpcs59 : Would you or @ellisvalentiner please check weather the develop branch works on Mac


I note that an Object has a `deleted` attribute which suggests to me that stat may return non-error for a deleted object.  Can you confirm that and check if necessary?


got it: https://github.com/garbear/xbmc/commit/4f37e98

running `sudo make -C tools/depends/target/ejdb PREFIX=/usr/local` installs some files:
```
Install the project...
-- Install configuration: "RelWithDebInfo"
-- Up-to-date: /usr/lib/lib/x86_64-linux-gnu/pkgconfig/libejdb.pc
-- Up-to-date: /usr/lib/lib/x86_64-linux-gnu/libejdb-1.a
-- Up-to-date: /usr/lib/include/ejdb/tcutil.h
...
```

but `./configure` isn't finding EJDB
```
checking for EJDB... no
configure: error: ejdb not found
```
The line from `configure.ac` is:
```
PKG_CHECK_MODULES([EJDB], [libejdb],
  [INCLUDES="$INCLUDES $EJDB_CFLAGS"; LIBS="$LIBS $EJDB_LIBS"],
  [AC_MSG_ERROR([ejdb not found])])
```
is there some way to point pkg-config at EJDB's `.pc` file?


You really need this then?


This doesn't seem to be used anywhere? What's the purpose of this?


Readme File


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


oh ja, das sieht gut aus!


101884090--NewVenting-Availability-Schedule (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1324 of 1324 tests passed)

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


103122604-ClimaticDataSummaryBug (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (789 of 789 tests passed)

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


Can you reuse `ListAll` here?


Для этого проекта будем использовать два пробела. Style guide - документ в котором как раз и указывается сколько пробелов ставить, где ставить пустые строки, как называть переменные и т.п.


:/ I need to find a way to prevent that


Faudrait trouuvé un facon de merger les themes. C'est un peu plate de devoir les ajouté a chaque fois qu'on en a un nouveau.


build error:
```
EGLNativeTypeRaspberryPI.cpp: In member function ‘virtual bool CEGLNativeTypeRaspberryPI::ProbeResolutions(std::vector<RESOLUTION_INFO>&)’:
EGLNativeTypeRaspberryPI.cpp:572:32: error: ‘VC_LCD_ATTACHED_DEFAULT’ was not declared in this scope
     else if ((tv_state.state & VC_LCD_ATTACHED_DEFAULT) != 0) // lcd
                                ^
```


Ты вообще похоже табы используешь судя по `base.html`.


Needs updated firmware (/opt/vc/include)


Посмотри на эти гигантские отступы.


> I suppose that does make me wonder why we're PR'ing it directly into dgrid but not directly into dijit...

I agree, I think dgrid theme should be part of this PR too, although the counterargument is that dgrid is on a separate release cycle (and more philosophically, it's in SitePen/dgrid rather than dojo/dgrid, so it's totally outside the dojo github account).


*Added Leverington's DataType.cpp


Nice. Not sure it's necessary though, so for now it won't be a setting. Let me know if there's a reason it should be.


How would this handle cases like:

```php
'commands' => [
    App\Bot\Commands\StartCommand::class,
    new App\Bot\Commands\TestCommand(),
   'App\Bot\Commands\SettingsCommand'
]
```

And i have command folders array filled with `App\Bot\Commands` as the namespace and the folder as `app_path().'/Bot/Commands'`

How will it prevent from duplicates?


to start the test from completely clean slate, use "drop schema if exists <table name> cascade"


The `|| event.srcElement` is unecessary, jQuery normalizes the event for us.


Fair enough, and good point about deferred updates – which would seem to be the more elegant forward-looking solution, then.


Is this recursively scanning the folders and building the list (With proper namespace)?


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


Instead of `data(...)`, I'm using the ParsleyForm directly and storing this in an instance variables `_$submitSource`.


exceptions should be more specific


ну да, тк. чтобы поставить таб нужно нажать одну кнопку, и кстати при табе = 4 пробела лучше видно код в js


I believe you meant `data('parsleyButton')` here...


This appears to have gone through...


Is it in "namespace prefix => commands directory" format and it'll take care of building namespaces for all the commands under sub directories?


Not 5? :-)


@MattSurabian Does this change mean that your viper repo supports cascading access to multiple config files?


This is limited to Laravel, What about users that are not using Laravel?


The problem now is that when user send a multipart message it seems right, however currently it is not allowed and user will not know this. Few solutions:

* drop this test
* have an option to allow multipart - will be used for the test and for users who still wants multipart. However socket will not be safe anymore.


Opposites of what we want.


done


I see.
I build userland from git https://github.com/raspberrypi/userland
that repo wan not updated.


I think it's best to use `if ($button.prop('formnovalidate'))` here. I think it's equivalent to what you wrote though.


Should be reverted to avoid that people get master (w/o assets) if they by mistake have minimum stability set to dev.


Finally :+1:


could you add a link to the issue? Github doesn't automatically generate the link for docs.


No problem at all. Thanks for the very thorough review!


Just tried to take this out. It blows up [here](https://github.com/kubernetes/kubernetes/blob/master/pkg%2Fruntime%2Fcodec.go#L64) with:

```
panic: type unversioned.Status is not registered for "v1" and it will be impossible to Decode it, therefore Encode will refuse to encode it. [recovered]
        panic: type unversioned.Status is not registered for "v1" and it will be impossible to Decode it, therefore Encode will refuse to encode it.

goroutine 72 [running]:
testing.tRunner.func1(0xc8204ca3f0)
        /usr/local/google/home/krousey/golangsrc/src/testing/testing.go:450 +0x171
k8s.io/kubernetes/pkg/runtime.EncodeOrDie(0x2b58d325e290, 0xc82013d8a0, 0x2b58d349f6c8, 0xc8204b8e00, 0x0, 0x0)
        /usr/local/google/home/krousey/goproj/src/k8s.io/kubernetes/_output/local/go/src/k8s.io/kubernetes/pkg/runtime/codec.go:64 +0xb7
...
```


Ah sorry, I misunderstood that. Usually we create a shared object after the name of the module specified in %module. For the examples, that is ```%module example```. This should translate into a language specific module called 'example'. Many languages use the name of the output .so file as the name of the example, but this does vary quite a bit amongst the different languages. I don't know if MATLAB has any relation between the output shared object file name and the name of the module used within MATLAB, so not sure I can offer much more advice other than suggesting to keep the module name as specified by %module. Adding in _wrap or _wrapMATLAB is just unwanted noise and the user should just get what they asked for in %module.


Done

On Fri, Sep 11, 2015 at 3:55 PM, Chao Xu <notifications@github.com> wrote:

> could you add a link to the issue? Github doesn't automatically generate
> the link for docs.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lavalamp/kubernetes/commit/a83c9ec069b3fe3a3c7c41caca1951f71bcfdbaa#commitcomment-13193453>
> .
>


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1328 of 1328 tests passed)

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


Did you try adding ticks as it's done in equalizer?


1853_fix_waterheater_stratified (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.39](https://pixelated.design/build/viewLog.html?buildId=121&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 282 Build time: 00:00:19


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


The `clean-up` function should be rather declared as:
```
clean-up: does [
	hScreen:		null
	hInstance:		null
	default-font:	null
	current-msg: 	null
]
```

`version-info` is a static structure, so it shouldn't be redeclared.

But this is not the right way to handle it anyway. The `init-view` function should be called only to initialize the View engine the first time.


But I want it to be...


This looks great. I think this makes symbols that begin with `-` not parse anymore, but that's probably fine.


I think for the .loadout-button class, make right: 10px; it's all misaligned for me


4967-SZRH-spm (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1847 of 1861 tests passed)

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


*275

That didn't that long to get wrong.


Hey. I didn't even look when I removed the #if false


Just curious here what you're doing where you might have a symlink. Are you using this yourself?


We don't enable Update Status on Sites, so no erus. No can haz.


:smile:


Good catch, but strictly speaking not related to this PR as it does not deal with fixing magic numbers for channel uids. Don't want to pull other things into this PR, as I wrote in the respective Kodi PR.


Declaration in for loops was added in C99 (16 years ago), thanks for pointing it out though because a lot of distros still default gcc to C89 I think (mine defaults to C99 so I didn't run into this problem). I think that it's safe to enforce C11, I'll add this to the build flags.

 As for the icon, can you please open up `gtk3-icon-browser` and check if go-down-symbolic or something similar is there? The default gnome icon theme on my machine has it. I would prefer to keep it as down because it points in the same direction as the player transition, so it's more inuitive that way.

On a side note, I noticed that the volume is styled differently to the other buttons thanks to your scrot :+1: . I'll fix this as well.


Try now...


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Can you explain why we expect `.range` to have same indentation level as `IntStream`?


mit powinnien być... taki jest w orginale


Fula de tal, está faltando o Funcionário


author jest Leon Revill

ale authors może być arrayem o ile mnie pamięć nie myli


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 596](https://teamcity.labkey.org/viewLog.html?buildId=271649&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:25:07
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Aún con la v2 vi que era posible realizar el cambio. Hago revert entonces?


dodaj też noda osx linux-a windows intelij :wink:


The directory structure is easy enough to change, but we had several people contributing and it was becoming difficult for them to manage working off of each other's branches.

The idea though was that themes would contain stuff for dojo/dijit/dojox, whereas things like dgrid and gridx would have a similar PR in their packages. So it seemed easier to organize them by package, e.g. here are themes for dijit, and then add similar for dojox within this repo.

That said, we could flip that around, and have themeName/packageName if that seems more sensical. In terms of why separating them into dijit and dojox, the idea was to make it easier to track what modules each theme corresponds to. I think the current structure was easier to map into the existing way that themes are used, which is why it was chosen.


Outro comentário


De acuerdo, yo también lo uso. Lo prox es irlo preparando para probar con el gcc5 a ver que tal


Using sparse matrices can help with some memory issues but this version is considerably slower!


Thanks!


In MATLAB, the user will never call the generated shared library directly. It's just an internal name referenced from the generated proxy classes and functions. In MATLAB you don't really need to load the module, just having it in your path is enough. You can do `import example.*`, but that's only to be able to write `foo` instead of example.foo`. By default, a module `example` will be placed in a package `example`.

I wanted to avoid the name `xxx.mexmaci64`, since I think there is a risk for ambiguity with the package or with functions of the same name. In the current iteration, the name is `xxx_wrap.mexmaci64` (different suffixes in Windows and Linux), where the `_wrap` infix can be changed to something else by setting the SWIG option `-mexinfix`.


Can you add a comment to make this more apparent/explicit?

```coffeescript
  return undefined    # otherwise coffescript returns Counts.publish handle when Meteor
                      # expects a Mongo.Cursor object
```


Why can't this be `= default`?


Clean build.
Thank you.


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


Aparte de esos 3 hay más. También fuera los otros?


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.33](https://pixelated.design/build/viewLog.html?buildId=122&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


Tanto cambio hacen las flags?


Parece que es una actualización de nuestro thermal


Si


:+1: Oh right. Gotcha.


No vayas tan rápido, primero coge experiencia con el código, sigues teniendo muchos errores, tienes que fijarte bien como aplicarlos en orden, fíjate mis últimos commits


Claro, son para compilar.


bhbjhbiiiuh


Se compila de acuerdo con el toolchain que uses.


你也在用cmatrix啊，哈哈 我也要参考配置来玩一下


Estoy probando los valores. Tanto Hells como Quanta los tienen así. Hells tiene como cpufreq_unplug_limit 125000 y Quanta, 1026000. Voy a ver por tanto que tal con 1134000mhz, manteniendo el theshold y el max_load_counter igual que los otros dos kernels


why include the .swp? :)


Pero yo no uso el STOCK, uso el franco.


Sin embargo no ha dado ni aviso ni error


De acuerdo, eso lo voy dejando por ahora entonces


Eso no lo había pensado, cierto entonces.


Además ese usa otro tipo de worqueque con otro delayed work.


I had to format this example differently than the others to get `pending` to work. This indicates to me that we may want to reformat all of our examples so that they aren't all defined as their own example group.


One of the links there is for Fixes for Facter 3.1.1, which is empty. Intentional? I was expecting Fixes for Facter 3.1.0.


Hi @sunnyking, do you plan to increase the stake modifier interval as once suggested ?


@sbromberger There's the link to the related pull request below the commit message on the github web interface. You can find the discussion there (and in particular Tony's plan).


Recuerda que así será más lento, yo ya lo probé con el del N6 y no me convenció, ya me dirás.


OK, I rechecked.  You are correct.  I did my calculation of the wrong tex coordinates.


develop (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Okey, te iré comentando. Como puedo cambiarlo desde synapse por ahora prefiero dejarlo tal que así. Ahora me queda ver si inicia ya el kernel. El error que ví con lo que me dijiste es que Hells puso 125000, cuando debería ser 1242000mhz


what is original?


Don't you need to remove this?


I'm not sure honestly about return values and exceptions.  I'd read up on [SHFileOperation](https://msdn.microsoft.com/en-us/library/windows/desktop/bb762164(v=vs.85).aspx) (return codes would be here), and [SHFileOperation Mappings](https://msdn.microsoft.com/en-us/library/windows/desktop/bb759795(v=vs.85).aspx) (scroll down to almost the bottom and there's a name mapping section).  That's the C code of course, so the Python equivalent won't be exactly the same, but it should give you an idea where to start looking.


Muchas veces no te dan error pero si warnings o no te da, y luego no arranca. Yo que tu dejaría las flags de los makefiles como están, ya las optimice al máximo y probando hace tiempo.


it was the only dependency of `eventsource`.


OK, let's worry about this later, then.

On Fri, Sep 11, 2015 at 3:57 PM, krousey <notifications@github.com> wrote:

> Just tried to take this out. It blows up here
> <https://github.com/kubernetes/kubernetes/blob/master/pkg%2Fruntime%2Fcodec.go#L64>
> with:
>
> panic: type unversioned.Status is not registered for "v1" and it will be impossible to Decode it, therefore Encode will refuse to encode it. [recovered]
>         panic: type unversioned.Status is not registered for "v1" and it will be impossible to Decode it, therefore Encode will refuse to encode it.
>
> goroutine 72 [running]:
> testing.tRunner.func1(0xc8204ca3f0)
>         /usr/local/google/home/krousey/golangsrc/src/testing/testing.go:450 +0x171k8s.io/kubernetes/pkg/runtime.EncodeOrDie(0x2b58d325e290, 0xc82013d8a0, 0x2b58d349f6c8, 0xc8204b8e00, 0x0, 0x0)
>         /usr/local/google/home/krousey/goproj/src/k8s.io/kubernetes/_output/local/go/src/k8s.io/kubernetes/pkg/runtime/codec.go:64 +0xb7
> ...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/krousey/kubernetes/commit/8573b124b4859dc01e3dbbcb7ea51bb8ee09132b#commitcomment-13193473>
> .
>


oops, good catch


s/have the enough/have enough

This also needs to be corrected in the respective error for 'max count'


Creó que esa es su mínima frecuencia y la nuestra es 192000.


It's BSD 2-Clause. http://opensource.org/licenses/BSD-2-Clause

I think you can just throw "BSD" in the field — not sure how you'd specify which variant to NPM...


`valueChanged` is defined shortly after but the observer doesn't reference this. Is there a reason it's commented out?


Why are these here?


Nit: All of your comments should end in a period.


This looks almost needlessly complex...


Isn't useLibrary 'org.apache.http.legacy' the right solution?


Me refiero a DEFAULT_CPUFREQ_UNPLUG_LIMIT. Nosotros lo tenemos 702mhz


Not to pick nits (I appreciate your fixing this!), but it should say, `the last job's PID`, since it's possessive.  :)  Thanks!


Ah, apparently they use SPDX. https://spdx.org/licenses/


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 597](https://teamcity.labkey.org/viewLog.html?buildId=271652&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:21:39
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Resetting the minAllowableSelectionCount shouldn't be necessary since it's reset in the 'before' function.


Package comment -- explain that this has to be separate as long as it imports testapi.


The way it was done was actually on purpose: If legacy resolvers do not report they are supporting a certain type of collection, they are supporting browsing artists - because that's all that was supported up until now. Why are you changing this?


s/registered throught/enabled in/


:clap: Thanks


For reference, the list I was scraping was [openssl-announce](https://mta.openssl.org/pipermail/openssl-announce/).


FWIW, the practical reason to put everything together is for them all to share one https://github.com/dojo/themes/blob/master/dijit/themes/flat/variables.styl file (or at least, to share the generic variables like `$theme-base-color`).


gj


@joseihf Why did you copy these fonts here? Isn't it better to load them from the static website?

In production it should work as GitHub pages send the correct CORS header for that.

Or am I missing something?


You're right, redoing that.


Si, porque tenemos esa frecuencia, 125mhz no existe en el nuestro.


Does this remove the country's flag from the homepage?


lgtm once `package.json` is updated.


Actually it's not very visualized. BTW, do you think if we can simplify the code of `texparameter-test.html` as this patch did?


Promise support is great! :+1: Why don't you submit a pull request?


yeah, I updated it in https://github.com/launchdarkly/node-client/commit/e92ec18f28c0d56f932812bd82f55eb29f146b4b


You can avoid using GroupOrDie here-- combine checking the error with your fatalf call. Then you don't need to check the registered package at all.


Instead of calling registered, I think it's nicer to see if latest.Group("experimental") gives an error.


test


OMG I'm never gonna live this down lol...now I'm just going to have to make a raptorch item/object ;)


With the new CSS, the class "icon-flag" adds a flag icon next to the name. So I have changed to "country-name" I have added the css into custom.css


I am guessing the coma will be added to the word?


也就偶尔装下逼，并不好玩儿。印象中这货太耗资源，我很早就在 tmux 配置中注掉了，而且在系统中卸载了。


What are you getting?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 598](https://teamcity.labkey.org/viewLog.html?buildId=271661&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:22:02
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


That makes sense. I will change according to your comments.

My consideration was that checking the registered package right after enableExp is assigned a value in line 332 makes the code more readable.


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (793 of 793 tests passed)

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


Ya he hecho los revert y desgraciadamente, sigue en el logo de google. Mañana si tengo tiempo iré pasando poco a poco mis commits para ver cual produce que no arranque. En mi código local he puesto DEFAULT_CPUFREQ_UNPLUG_LIMIT 702000 y nada.


@sivarv is it not possible to have test cases for things like this? Or was that in a diff commit?


To me the author is going to choose a package name that isn't the same as an existing package, so in that case, surely there is nothing to worry about? Having some pre-chosen suffix seems a bit surprising given the input module name. I don't really follow what the potential ambiguity is. I think you are saying that you can't you have a C function called xxx and wrapped as a function called xxx in a mex package called xxx? Surely functions within a package can have the same name as the package?

I'd go with xxx.mexmaci64 by default and provide a command line option to change to change xxx to something else.


Looks good


I have a small problem with the _AR Drone Controller with PS3 Support.vi_. The values do not "flow" out of the _PS3 Controller.vi_ (everything works fine in the VI separately).
Help! @kovacsgellert


Demonstrar o uso do checkout no NetBeans


Well then....


How rude. I demand you remove this at once!


What happens when you do characterPosition("aloha", -1) ?


1853_fix_waterheater_stratified (mbadams5) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2307 of 2342 tests passed)

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


1853_fix_waterheater_stratified (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


I saw that the urls contained in font-icons.css were not working, so I copied this file (and the font folder) and modified the urls.

If it will work in production I can undo the changes.


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


Thanks for comment


nit: weird spacing, but it's just an example config


@dietzc:
> first problem: your interval is bigger than the input (as you define min/max and not dimensions).

And that solves my problem! I was not aware that the maximum extent of the interval of an image is `dimension(d) -1`. Thank you :)


I agree simplifying is a good idea.  The matrix projection doesn't add value to the test coverage.


Many thanks.


103122604-ClimaticDataSummaryBug (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1324 of 1324 tests passed)

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


Cool~ an easier way around this would be phrase.toUpperCase();


Please add a space after the colon.


Please add after the colon.


adf


YOU'RE NOT THE BOSS OF ME


Looks like you have misspelled "canola"!


searchpcc 小生态


wow nice


`my_price = (@price * @number * calculate_tax/100.0).round(3) + @price * @number`
Just `(@price * @number * calculate_tax/100.0).round(3) + @price * @number` is OK.


请问是什么没有引用吗？未能找到类型或命名空间名称“ECommon”(是否缺少 using 指令或程序集引用?)


`count_price` => `calculate_price`


Nice change!


Pay attention at `nameing convention`


Fix is out and passing on tests on v0.10.x. Note that I bumped the version to 1.0.0 as we've been using it in production for some time now and I'd like to have the unambiguity of full semver going forward.


Please DO NOT set this to threadrecorder.fxml .
ThreadRecorder plugin sets column width automatically because we want to expand timeline column as possible.

This change will cause of unexpected behavior of view.


Hello @Will-So
cc:  @jonhanke  @julialintern

Q1-Q3 - done, OK

Q4 LC, Map, Filter - not done
Q5 datetime - not done
Q6 - not done
Q7 - not done
Q8 - not done

Answers for Q5 (datetime):

a)  937
b)  513
c)  7850


i agree the use of the add method makes sense. i had npt seen the method when i made the original patch.

Dewayne
   o-*

> On Sep 10, 2015, at 11:17 AM, Kelley Reynolds <notifications@github.com> wrote:
>
> #45 Has the goods
>
> —
> Reply to this email directly or view it on GitHub.
>


okay,  in my opinion, the '$AGE_LIMIT' can be moved to my Human class, but '$arr_human' can't


I will fix it.


tried with SR bs4 and BS4parser:

```
from sickbeard.bs4_parser import BS4Parser
....
with BS4Parser(data, features=["html5lib", "permissive"]) as html:
```

still get: [HDTorrents] :: Could not find table of torrents mainblockcontenttt

can we add bs3 in lib?!


I think this should be
```
if (on&&!completed(index))
  completedCounter += 1
else if (completed(index)&&!on)
  completedCounter -= 1
else //NOOP
```


Where is the `get_gender` definition?


101884090--NewVenting-Availability-Schedule (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (776 of 776 tests passed)

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


since you have submit a patch tocodec parser upstream, can we use this one?


Consider this: http://ruby-doc.org/core-2.2.0/Array.html#method-i-index


This might be the cause of some issues for you https://github.com/ltearno/hexa.tools/pull/10 I forgot to committed this change last week since I've been busy with work. Try it with those changes.


```
if @name.downcase.include?"import"
			return true
		end
		false
```
===>
Just `@name.downcase.include?"import"`is OK.


Why dis you need the `Global attribute` (`$` sign)?


@eugenehp sorry for the delayed reply. I just need to keep tabs on my github notifications :-)

Regarding the benchmarks, the one that I compared are the graphs in New Relic. I would be interested to know if you have any other better way of benchmarking.


Thanks for your comment.


You can override the `to_s` method


wtf brooo lol


`while gender.downcase != 'male' && gender.downcase != 'female'`
===>
```
while gender_is_invalid
def gener_is_invalid
  #code
end
```


@Inndy that is used to generate a valid sequence without duplicated digits ... copied from the generator :)


could we have some comment here


Hi


do not need this


xaxa


This makes sense for the scan operation. Most other operations are waiting for responses from the BlueGiga module so they should spin in order to keep maximum speed.


Sorry, this is my mistake. I wrong in the get and set method before and I delete it but I'm not change the command in method


thanks, I get it


ここがいいね


Why do this?


Same number `7`, two different places in the code - why isn't it a constant then?


Would n+=2 work also?


got it


great


@dujiepeng
你好，这个是为了自动管理吗？

Jack


求大神添加https://www.appannie.com/
appannie登陆不进去


He sure has quite a lot of personality eh XD
Also, it changes each day....


You may have to go into rstan/StanHeaders/inst/include and make sure that
the git subrepo inside the upstream folder and the mathlib folder are both
pointing to master or develop or something that has the inv_Phi branch
merged.

On Thu, Sep 10, 2015 at 10:44 PM, Jonah Gabry <notifications@github.com>
wrote:

> Yeah that's exactly what I've tried. StanHeaders and rstan seem to install
> fine but I still get an error about inv_Phi not being found when trying to
> build rstanarm.
>
> On Thu, Sep 10, 2015 at 7:59 PM, bgoodri <notifications@github.com> wrote:
>
> > Probably just do a
> >
> > git checkout develop
> > git pull -ff
> >
> > in the rstan clone and then R CMD INSTALL StanHeaders/
> >
> > On Thu, Sep 10, 2015 at 7:56 PM, Jonah Gabry <notifications@github.com>
> > wrote:
> >
> > > @bgoodri <https://github.com/bgoodri> What do I have to do to install
> > the
> > > versions of rstan/StanHeaders with inv_Phi working?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/stan-dev/rstanarm/commit/133c7e355b9b58d2ba4843b746799a37e5f6090d#commitcomment-13172770
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstanarm/commit/133c7e355b9b58d2ba4843b746799a37e5f6090d#commitcomment-13172795
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/133c7e355b9b58d2ba4843b746799a37e5f6090d#commitcomment-13174269>
> .
>


@nim-odoo @ahu-odoo @qdp-odoo Can you update the cancel method of cash statement to allow single cash register line unreconcile, to have the same behaviour as bank statement lines...it's a pitty of not having the same methods...it's just of adding the button in the view...


there is extra code here from when i was practicing earlier


didn't quite get the jokeception there :v


Done


actually once it's completed it can not be reversed. I'd rather change method signature to complete(index: Int) that will set index as completed. WDYT?


acho que seria

``` php
else{
    	header('Location: ../views/login.php');
}

```


please remove this line


somehow seeing "troll" as a language reminds me with the "pirate" and "mirrored" language in FB XD


Let's discuss when you get back.  In order to do this, I think you'd have to remove all the Jackson annotations from the JsonPatchOperations, and write a Factory for each one.


isso aqui é temporário certo?


This could, and should, have been combined with your other request.


estou  perdido, gostaria de uma ajuda


when using user create command with the same uid and dispaly-name as an existing user, it actually modifies its information


Thanks catching, I delete this change and write this information at commit message.


Changed to generate Diagnostic if source contains ```#load``` and ```SourceReferenceResolver``` is not specified in ```ParseOptions```


101884340-Crash-with-missing-AirflowNetworkMultiZoneZone-object-#5029 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1829 of 1839 tests passed)

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


All of the conditional #ifndef / #endif  should be removed. (See language_de.h as an example). They are in the language_en.h file only because that file serves as a default in case there is another language file present which, for some reason, fails to define all of the necessary messages.


oops


I remember you add the moveCallback in your previous PR, but why remove it here?


Why Change the position of InnerContainer should dispatch scrollEvent here?
Thanks.


Elaz


You can just do `weak?.didDisconnect = false`.

This is only creating a strong reference until the block is dispatched, so there's no need at all to make `self` weak.


@pedro-guimaraes @NyKaren pessoal, subi aqui pra vcs n se distanciarem do modelo predefinido nos wireframes.

Seria bom tbm na pasta db termos um `seed`


Same here, this isn't a retain cycle, you don't need to make `self ` weak.


Ditto


Not quite sure but cool 😀


101884340-Crash-with-missing-AirflowNetworkMultiZoneZone-object-#5029 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I wrote this and the following code six years ago, I know that it won't compile. Anyway :shipit:


I'm sure there are other places for change tracking that are required, I haven't looked into that yet but was planning on it once I got to writing the data back.  This code solved the exceptions I was getting in the following scenario and allowed me to bring back the data I was working with  ( I wrote this code for scratch in this editor to match what I was working on the other day, so it may not compile if you just paste it into studio right off the bat ):

{
   'EmployeeId': '1',
   'StoreId': '1',
   'Permissions': [
      {  'Entity' : 'Contacts', 'Read' : 'True', 'Write' : 'False', 'Create' :  'False' },
      {  'Entity' : 'Notes', 'Read' : 'True', 'Write' : 'True' , 'Create' : 'True' }
   ],
   'RoleIDs' : [ '1', '2', '3' ]
}

public class EmployeeStorePermissions
{
   public string EmployeeId { get; set; }
   public string StoreId { get; set; }
   public int[] RoleIDs { get; set; }
   public List<EntityPermissions> EntityPermissions { get; set; }
}

public class EntityPermissions
{
  public  string Entity { get; set; }
   public bool Read { get; set; }
   public bool Write { get; set; }
   public bool Create { get; set; }
}


101884090--NewVenting-Availability-Schedule (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1309 of 1309 tests passed)

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


GroundDomainUpdate (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1846 of 1851 tests passed)

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


Looks like you misspelled "canola"


For the method naming I was following the example of cpython. I generally find their code easier to read than jep code so I tried to copy their style. I dug around a little more tonight and there is a [pep for styling c code](https://www.python.org/dev/peps/pep-0007/) that includes the [camel case naming convention.](https://www.python.org/dev/peps/pep-0007/#naming-conventions) I don't think jep has any formal style guides so I think it would be good if jep moved toward following the pep more closely. I would like to just update the style whenever I modify an area of code, but if the inconsistent style bugs you then I can switch to lowercase and if we want to follow the pep I could redo the entire project at once so it stays consistent.


Na verdade a pasta db tem q estar fora segundo #14


*remove company name


This Good Anti-Piracy


Agree with your changes. Some background:

* This is intended to be guide for my team at 3DR. It's not just what I think the Swift world should do - it is/will be a concrete guide for my team and new hires.
* I'll put the spaces/tabs thing back but less confrontationally. It'll be important for my new hires.
* I'd like to end all style arguments with "Use apple's style". I'll want to emphasis that somehow. The way I did it before by putting in a mini-styleguide (despite pretending not to be a style guide) didn't work. I'll figure out something better.

Thanks for you PR. Appreciate it.


Any performance impacts? What advantages does this have over the original?


Seems that now the patch is gutted.


Correção de conflitos em Partida e Contrato


This is still under WIP but very promising, both camera sensors loaded, both video nodes registered to /dev, but v4l buffer creation failed. I also have tried reconstruct current camera driver to load old camera blobs with a same result. Back to old source, I hate a complex ifdef hell. Lot of problems came from incompatible header, still make sure this old driver and all media subsystem not use newer header.


OK. Thank you.


why we need put createPicture here? If current picture is rasl, decodeCurrentPicutre may decode a rasl picture


[trekkabout's solution](https://github.com/trekkabout/google-homepage) | [View in browser](http://htmlpreview.github.io/?https://github.com/trekkabout/google-homepage/blob/master/index.html)


Cherry-pick with `-x` please.


Hey, @mitchellh when this commit will be in release? That bug really made my night, when I was trying to figure out why again my vagrant fault to start


why we should check m_context  here since we have ensureContext


It's not a big deal, but what do you think of a helper method for this since it appears a few times?

    ClusterInfo clusterInfo(Map<String, DiskUsage> usages, Map<String, Long> shardSizes)


test


could we treat m_endOfSequence as m_newStream


Hello @reshama :

I apologize for not including the results of Q5. For the indenting and markdown.

I thought I was careful to tag all code as python and make sure all the indents are lining up correctly. Please let me know what I am doing wrong, so I can make it more readable for next time.

Thank you,
Jaon


That may be good, but I think that is outside the scope of this change. This is simply moving things around so the test-jar is actually completely usable.


I think these steps should be outside the setLocalManagerConfig()
do the same for HA config if it does that same too


merge latest code


Deciduous pine（落叶飞松←_←）


LGTM


Fixes #115


The data types macros are not defined at this point, so it is not possible to use them.

As this is a target define, I think it should be okay to use the standard data types here right?


Do you really think this is a good idea? This is basically like saying "bring on the bugs!", for which you will have no means of fixing them. Releases through B321 were all beta and each release included upgraded firmware. You see those files under `/firmware`? ROMs don't (or at least shouldn't) touch those firmware files. Namely, the modem and adsp firmware have seen a number of updates during the beta phase. The firmware included in the last release B322 ironed out a number of radio issues from previous releases. It's a shame you don't want users to have the best experience with your ROM.


:( my bad


@Alx31 I was gonna add it later on


Does this benefit FC4?


typo: a stream processing engine


Actually it is rather nice to view database changes like source code changes. We could zip whole database releases to keep memory usage in bounds, but create small sql update commits during development cycles. And when we think it is time for a new database release, we merge all update files into a new database release zip file and delete the old one.


#parse("/portlets/html/scheduleMacro.vm") している箇所を除去する

がされていないので、

```2015-09-11 13:14:27,847 <ERROR> (http-nio-8080-exec-4) [SimpleLog4JLogSystem.java:181]- ResourceManager : unable to find resource '/portlets/html/scheduleMacro.vm' in any resource loader.
2015-09-11 13:14:27,848 <ERROR> (http-nio-8080-exec-4) [SimpleLog4JLogSystem.java:181]- #parse(): cannot find template '/portlets/html/scheduleMacro.vm', called from template /portlets/html/ajax-schedule.vm at (30, 1)
2015-09-11 13:14:27,850 <ERROR> (http-nio-8080-exec-4) [SimpleLog4JLogSystem.java:181]- Exception rendering #parse( /portlets/html/ajax-schedule.vm )  : org.apache.velocity.exception.ResourceNotFoundException: Unable to find resource '/portlets/html/scheduleMacro.vm'```

のエラーが出てます。


`cherry-pick -x`


已更新规则，安徽联通，亲测可用。

Eric <notifications@github.com>于2015年9月11日周五 上午11:29写道：

> 求大神添加https://www.appannie.com/
> appannie登陆不进去
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/racaljk/hosts/commit/6d8bd8cb8d955b966de73be752f3ab856cbc8af2#commitcomment-13174613>
> .
>
--
Andy


Thank you!


Maybe explain or have a link what a spout and a bolt is.


Yeah the constant `DUALPROC_INSTANCE_COUNT` is defined in dualprocshm-noos.c.
Thanks, I will move `DUALPROC_INSTANCE_COUNT` to the target files.


Typo: by executing a whole


すみません、私が直せたら直そうと見てみたのですが、よく分からなかったのです。
ありがとうございます。
よろしくお願いします。


so much goto, could we use  SharedPtr<H265SliceHdr>?


oh hey that was fast :)


Typo: Code is packaged.


better return success when we get broken link.


Typo: class is to be changed


Typo: actual


Esta bien


Agreed.


I already considered this for the rework.
In memory its much more easier to identify the unique token then the counts so I preferred to keep the existing implementation.

Please let me know if we should revise this.


101884340-Crash-with-missing-AirflowNetworkMultiZoneZone-object-#5029 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (775 of 775 tests passed)

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


rax retry tests


okay.


Seems to be a problem when published when using some contest pages. Font does not change and also buttons do not line up even though text in each column is the same length and font.


Typo: Spouts and Bolts


为什么中国这么好的开源框架要太监？
Why should such a good open source framework China eunuch?
我英语不好一路找过来你让我看这个？
My English is not good all the way to get over you let me see this?


I only saw a `pa` group in the `mt7620` sources (`rt305x` has a separate `lna` group as well). Based on [this discussion](https://dev.openwrt.org/ticket/20369#comment:9), both functions may be present as a combo. Either way, only the `pa` binding/keyword would be valid on the Y1 as well, so try the above change, and see what the `pinctrl` driver finds. Should be harmless…


Closes #274


Fixes #273


102817062_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@accetone Бирюн И. А.


Any benefits?


solved/ICPCLivearchive/4682 - XOR Sum/solution.cc

I didn't understood why this line is here. Can you please explainit to me.!!


Why the `0.5 + (bottomOfBreak + topOfBottomColumn)`? Wouldn't it make sense to just use the `topOfBottomColumn` value?


Nice! :+1:


Logic makes sense, but I'm still confused as to way we take 1/2 of the max page value + the lowest "block".  Wouldn't it make sense to just take `Math.max.apply(null, leftBottomBlocks.map(b => b.ySpan()[1]))`?


Hey, move this to the master branch and delete this one, por favor. Also, no file extension is on this file.


Do the DNX ones map to something real on the tooling end?


Why do this?


Sort of. You need to tell the tooling folks what each code is so they can map it to fwlinks for help


Actually -W won't run networkb (yet). It's on my long-term things to do... which I should probably move into a markdown doc...

So today, you need to run networkb in a batch file that still loops.  Also I don't think beginday is wired into bbs -W  yet either.


Summary of changes During Cherry-pick:
src/Mod/PartDesign/Gui/Command.cpp: changes canceled
renamed file: ViewProviderDatumShapeBinder.{h,cpp} ->ViewProviderShapeBinder.{h,cpp}
renamed file: DatumShape.{h,cpp} ->ShapeBinder.{h,cpp}
TaskFeaturePick.cpp: minor changes
Workbench.cpp: separated into different commit
added several TODO's


just `0.4` until the release is out


I'm not entirely sure how to test this.


Managed to cobble together a rough example. Sorry about the function names :-)

```
{-# LANGUAGE DeriveGeneric #-}
{-# LANGUAGE FlexibleInstances #-}
{-# LANGUAGE KindSignatures #-}
{-# LANGUAGE MultiParamTypeClasses #-}
{-# LANGUAGE PolyKinds #-}
{-# LANGUAGE ScopedTypeVariables #-}
{-# LANGUAGE TypeFamilies #-}
{-# LANGUAGE TypeOperators #-}
{-# LANGUAGE TypeSynonymInstances #-}

module Main where

import Data.Proxy
import Data.Foldable
import GHC.Generics

data A = X | Y | Z | R | S | T deriving (Enum, Generic, Show)

class E (f :: * -> *) where
  e :: proxy f -> [(String, f p)]

instance E f => E (D1 d f) where
  e _ = fmap M1 <$> e (Proxy :: Proxy f)

instance (E x, E y) => E (x :+: y) where
  e _ =
    let l = fmap L1 <$> e (Proxy :: Proxy x)
        r = fmap R1 <$> e (Proxy :: Proxy y)
    in l ++ r

instance Constructor c => E (C1 c U1) where
  e _ = [(conName (undefined :: t c f a), M1 U1)]

main :: IO ()
main =
  for_ (e (Proxy :: Proxy (Rep A))) $ \ (n, p) ->
    putStrLn $ n ++ " " ++ show (fromEnum (to p :: A))
```


101884340-Crash-with-missing-AirflowNetworkMultiZoneZone-object-#5029 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1308 of 1308 tests passed)

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


My hope that, in time, we can do away with the need for this case, as it kind of re-bakes the inflexibility that we're looking to move away from. However, that will require further amendments to the config system that shouldn't block the inclusion of this feature.

So, I'm okay with this being merged on the assumption that it may not be long-lived.


where you able to find this binary?


I like "but but"s and "and and"s in the commit messages :3


Output:
```
*Main GHC.Generics> :main
X 0
Y 1
Z 2
R 3
S 4
T 5
```


How do I do? Edit now and send pull request again?


IMO this should be part of the `SilverStripe\Filesystem` namespace.


+1! 👍 Awesome work by the way!


please, bump version


I *think* this isn't necessary. As I understand it, bash always sources `.bashrc`, even in non-interactive sessions, and then subsequently *also* sources `.bash_profile` and `.profile`.


102966118-HeatRecoveryBypass-4568 (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


also if you are just using the values of LEAGUE_IDS to check for league in supported league, then i would say just to this:
supported_leagues = LEAGUE_IDS.values()


this provides daylight savings time offset - is this consistent with discussions in #602, #603? The function `GNtimezone` also provides `gmtOffset` and `rawOffset`. And should we go ahead and populate the entire sites table table?


Sugerencia: en vez de usar un while tal vez te sea más comodo usar el método each (o each_with_index) de array
http://ruby-doc.org/core-2.2.0/Array.html#method-i-each


Does not mlt_filter_get_length2(filter, frame) work here? That is what all the other keyframable filters are using, and what I would prefer to be used consistently.


let式は `let "bindings" in "expression"` という形になる

let で定義した変数は let 式全体から見える。


Unrelated to this commit. Can be same PR, but should be separate commit.


Unrelated to this commit. Can be same PR, but should be separate commit. Looks like it should be part of the CompositeDBField PR?


heh


* Use symbol table to keep track of locations
* Use dp to reduce time from O(n ^ 3) to O(n ^ 2)
* Use larger grid size first


Separate commit and/or PR. Think this is already included in a separate PR.


This is Travis-only so you don't really need to update your METADATA pull request or tag


If the unit case 50 by 50 works out, we could probably have a practical implementation.


* Link for the paper

* Mention MTP function, radius protecting contour

* Maximum constraint for grid size


Just do .entries()


THat #include should be dropped


Can we give a bit more detail about what this interface is supposed to be used for and represent. As best I can see it's a read-write interface to a single asset, that can then be linked to via URL or extracted as a string/stream.

I note that rendering in an HTML file (via getTag()) isn't part of this interface. That functionality could potentially be a trait?


There should imo be a dependency for libssh2 here (already in the repo)


See here: http://review.cyanogenmod.org/#/c/99707/. Not worth disabling as stock G2 uses C2D composition as well I believe.


Are we going to get into trouble with large files on this that we could otherwise avoid?


@asofold That is correct. It still does not compile with netbean due to requiring a bukkit 1.8 dependency from bukkit's outdated repo.


Tuple of what?


OK


This is not necessary, as it's already handled by cmake automatically.


Oh great, tested on one system. Committed on another 😓


This is a hack which is already discussed in #286.


looks like you have a test failure in `test_svmlight at /home/travis/.julia/v0.4/SoftConfidenceWeighted/test/linearly_separable.jl:87` ?


If you reach this point, you will call the g_key_file_free twice (see end of if-clause). I think that is wrong.


This file was changed


'tmp' needs no initialization as you always use strcpy. The line can be dropped.


My hope that, in time, we can do away with the need for this case, as it kind of re-bakes the inflexibility that we're looking to move away from. However, that will require further amendments to the config system that shouldn't block the inclusion of this feature.

So, I'm okay with this being merged on the assumption that it may not be long-lived.


@yorchek checa


Can we give a bit more detail about what this interface is supposed to be used for and represent. As best I can see it's a read-write interface to a single asset, that can then be linked to via URL or extracted as a string/stream.

I note that rendering in an HTML file (via getTag()) isn't part of this interface. That functionality could potentially be a trait of DBFile?


for...


Yeah, once we can use silverstripe-specific constants in yaml we can dump this.


Returns a filename, not a tuple.


Able to block new tasks from being added if:
1. Their starting time is between an existing task's starting and ending time
2. Their ending time is between an existing task's starting and ending time
3. Their starting time and ending time is the same as that of existing tasks
The existing tasks here ONLY refer to those which are NOT yet done, i.e. pending and blocked
The existing tasks here ALSO refer to tasks on the same date

Undo is still not working.


Fixed now.


This is because its dependency SVMLightLoader has issue.
The updated version SVMLightLoader is already pushed to master, but the tags are making troubles.
https://github.com/JuliaLang/julia/issues/13063

Tests on the latest version of SVMLightLoader works well.


TODO
Change these NULLS to 0


I think we should say "Tuple (path, hash, variant)" in all our `@return` clauses to help devs


This should be:
$rootScope.$on("$stateChangeError", function(event, toState, toParams, fromState, fromParams, error) {

The error is the 6th argument.


TODO
Change these NULLs to 0


What's a fileID? is it a path?


IMO this should be part of the SilverStripe\Filesystem namespace.


I've moved this to the composite db field PR.


TODO
These need to be changed in the original sql entries


``` python
def genall():
    return [ n for n in ("%.4d" % i for i in range(10000)) if len(set(n)) == len(n) ]
```

Sorry, I didn't notice that. This will be more elegant


文字化けしちゃってるね。なんだろう


yep, would be good.

For now, let it here.


TODO
This is incorrect and is the list_options entry and guessing it was flip-flopped with list_options below.


これが必要か不要か、どこかのタイミングで試してください。
不要なら消しましょう。


Do we want to separate `AssetReader` and `AssetWriter`, and have `DBFile` implement both? That would allow for an object that representing a read-only asset.


I added it because I (and other people) was not hearing the pickup sound when picking up the eggs, unless that has changed recently.


Do we need this?


I was trying to be conservative. If you use topOfBottomColumn I think in theory an ascender is over the baseline.


I'm going to add `This is used as a use-agnostic interface to a single asset backed by an AssetStore`

getTag is more DBField / ViewableData specific behaviour.


Yep I'll do this now.


Nit: IMHO `is used in documents` would be better here than `is used by authors`; and `quality to use when` would be better than `quality used when`; and it would be good to avoid repeating the word “use”, so something like:
```
+  <p>The <code>ImageSmoothingQuality</code> enumeration is specified in documents to express a preference
+  for the interpolation quality to use when smoothing images.</p>
```


LGTM after comments


ajax-userlist-detail.vm

```
#if(!$!{result.Detail.isOwner()})
-       <a href="javascript:void(0)" class="auiButton" onclick="aipo.message.hideProfile();aipo.message.openDirect($!{result.Detail.UserId})"><i class="icon-comment-alt fcGray mr5"></i>$l10n.MESSAGE_DIRECT_MESSAGE</a>
        #end
```

ajax-account-person-detail.vm

```
#set($dlLink = $!utils.escapeXML($!jslink.getPortletById($!portlet.ID).addQueryData('template','AccountPersonFormScreen')) )
		<a class="auiButton" title="$l10n.ACCOUNT_DO_EDIT_ACCOUNT" href="javascript:void(0);" onclick="aipo.common.showDialog('$dlLink','$!portlet.ID','aipo.account_person.onLoadPersonInfoDialog');">$l10n.ACCOUNT_DO_EDIT_ACCOUNT</a>
```

それぞれのテンプレートでリンク先とリンクを表示する条件が違うため、両方に対応しておく必要があります。


Oh İ meant errno should be the exit code when you call sys.exit.  İs the errno attribute for file not found the same as the one in the errno modüle? Please consult `pydoc errno` and `man 3 errno`.


(You can take it out of the error message too, it doesn't help humans)


102966118-HeatRecoveryBypass-4568 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1827 of 1840 tests passed)

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


Do you want to put this in your php7 compat PR and do this all at once?


102966118-HeatRecoveryBypass-4568 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (776 of 776 tests passed)

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


If we can build in support for silverstripe specific constants into yaml then yeah we can drop this. :)


Yeah ok!


Nit: In general I think it’s best to avoid stating things in terms of what “the author prefers” and such—because Web documents exist (and change through scripting, etc.) in isolation from their authors. So I suggest considering to make this `indicates a preference for a low level of` rather than `indicates that the author prefers a low level of`. (And yeah I’m aware of the language rule that says passive voice should be avoided in many forms of discourse, but I don’t think this is one of discourse forms where it should be.)


LGTM


See above about “author”.


“image-smoothing algorithms” (with a hyphen)


@lukemelia Nice, thanks! :smile:


Thanks - https://github.com/mxe/mxe/commit/3fb46e66434cf44aacedc2fbe56ac4e5839f985d


오늘은 비가왔다


You can remove this log4j stuff, the MethodExecutionTrace rule which is activated by ITUtils.createRulChain (which you are calling in the test) should look after it.


maybe “image interpolation” (with a hyphen), though in this specific case, it may be unambiguous enough without it.


Thanks for getting the ball rolling on the new structure @sandstrom! I hope it was worth the wait.


Если не сложно, хотелось бы, чтобы моя фамилия была написана через "е".


@K-Guan 好，谢谢


“Low-quality image” (with a hyphen)


initially the code didn't work (bubbles disappeared, but not with popping animation).  I saw that in the following code: that.getSprite().css("background-position","-50px -" + top + "px");  I had an extra space after the dash after the pixel spacing code "-50px -EXTRASPACE" like "-50px - " instead of -"  That was the difference though I haven't stopped to exactly understand it, but I think it must break the css that the js code is creating.


Fixed. Total typo


非常感谢！已经可以正常登录了！


Timezone preferred as: "±0000 (DST)?" for direct sense with user's local time.


You were adventurous here :)


`from sickbeard.bs4_parser import BS4Parser`

Thats just a wrapper around bs4.


suggest either:
- “Returns the current preference for image-smoothing quality.” (one hyphen)
- “Returns the current image-smoothing-quality preference.” (two hyphens)


I think the comma here can be dropped.


new a test


the name "CurrPrev" looks wired at the first glance...


이부분 textarea로 modal 전부 추가됨. 버튼 형식으로 다 바꿔야 한다!!!


Ah that makes sense!  We could also pad by some arbitrary value, but I like this better.


Should refer to `./hello_lwt.native`.


Is there a specific reason why we can't use cssnano? Minify CSS does some awkward changes to units and converts them to points and pica which is not wanted.


Awesome! I'd been meaning to port this over but am not a heavy user of lwt. Hopefully now that there are two implementations of the setup code, it'll be clearer how it should be factored out for both.


I suggest to move the logic into a function, I saw the adjustment twice in this PR.


Works just fine, +2!


Not sure, Jenkings seems not to like cssnano? Could not get tests through because of that.


The moveCallback had been added to handle event dispatching when it is out of boundary. But I found that this doesn't cover every case. In that way, events like EventType::BOUNCE_TOP are not dispatched during bouncing out. They were dispatched only when bouncing back. So I moved the event dispatching logic to `setInnerContainerPosition()` in order to handle every case.


Why does `OfType` not work?


why change the parameter from “second” to "time"?
I think second is more meaningful. It clarify the time unit. Thoughts?


I think the `start-httpd` parameter is not needed, if you want to preview your site, of course you need to start http server and preview it in your browser.


So it means you want to change PathMap to PathList initially?


great to have these as working examples, thanks @pgkirsch !


Long line >100 chars. The `alt` attribute can have line breaks in it without it having any ill effect on AT/screenreaders, asfaik.


> I think the word "creating" should be removed, otherwise it's not an english sentence.

With my foreign English I think it is correct; but I guess native speaker should chime in.

> "enabled" just needs at least one symlink in /etc/systemd/system that points to the unit file. What is in [install] is irrelevant.

If there is no `[Install]` section, `systemctl is-enabled` prints `linked`. If there is `[Install]` section it prints `enabled`. I do not know what happens when you mix `[Install]` and manual links but this change is specifically about links from `[Install]` section.

> Also, "enabled-runtime" is the state that is reported if the same is done via /run/systemd/system.

I did not touch _this_ part at all.


Issue4530UpdateIDDandIORefDOAToZonalUnits (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1839 of 1839 tests passed)

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


Second item in the returned tuple (args) is itself required to be a tuple, as per [object.__reduce__](https://docs.python.org/3.4/library/pickle.html?highlight=__reduce__#object.__reduce__) interface.


Could this be an issue if we have a custom (white label) URL specified?


s/in in/in/


Oops, was trying to push to Sirius branch but had wrong bash focused.

However this does fix an issue with the colour and restores the FAB drop shadow now it works on Samsung after https://github.com/mapbox/mapbox-gl-native/pull/2293


Summary on cherry-pick:
 - Split datum plane icon changes into a separate commit
 - Some additional renaming stuff to ShapeBinder to unify code
 - Removed try/catch around the command


Pedro, aquí borraste todos los términos que había yo traducido bien y volviste a poner los que había al principio, mal traducidos. Voy a volver a hacerlo pero, por favor, no volváis a descambiarlo.


style check : Move the function "{" to new line


Seems you use different code format from mine, so when I review the change, there are many lines changed without effective code change but only more spaces, could you please re-format the code and make the PR only contains effective change?


Sort of. How about `calculateCurrAndPrevTouchPoints()`?


:+1: !!!


@zamar, I see that one of the synchronization barriers added here was part of original SF-6 code, but it was removed in this patch:
https://github.com/official-stockfish/Stockfish/commit/81c7975dcdf37ec96d308f800c2a40f6cb08907e

i.e the one before decrementing splitPointsSize.
I guess you decided to remove it because searching = true which means no other thread will try to book this thread.

But I was wondering if the objective is to make volatile variable 'searching' visible to other threads, isn't it more appropriate to add a write memory barrier on line 208?(or is this automatically taken care for volatile variables?). How does adding a spinlock.acquire() here help ?? I'm confused... Can you help me understand why you removed it and added it back again?

PS: I'm not an expert in multithreaded programming :-).


Cf. jrl-umi3218/jrl-cmakemodules#59


Remove comments.


Remove all print function and comments.


Do we still need this?


why the `database` has to be `accountId`?


According to the coding style, default arguments are forbidden here.  https://github.com/cocos2d/cocos2d-x/blob/v3/docs/CODING_STYLE.md#default-arguments


OK, I got it. It happen during this merge
https://github.com/rism-ch/verovio/commit/16cfc3f75dbf2f1840bb05cde08374c2166fd997#diff-d195cff1d379dce58d848d901f42c993
The minMeasureWidth parameter of the functor was added again. All good now.


Second is good. The parameter had various names in various APIs. I just wanted to make them have one name. I'll change.


Ha ha, I'll take that as you're in!


That is currently what is done with Classic DB and probably UDB since they moved from svn to github. I assume the same philosophy would be used once the databases are integrated into the core repository. At least, it would make sense.


@aliprq the approach of deleting EVERYTHING unconditionally, and then building it up later is much harder than adding stuff you need and then clean up afterward. A lot of the things you've deleted here are actually required.


/home/zettata/Desktop/Untitled Folder/pro


I've just tried to apply your solution and it doesn't work. All items remember their owner, albeit they were already removed. Therefore, either I have to prepare items for deleting or change the remove method, so that it checks whether or not the removed item is still presented in the list.


the `.sass-cache`directory should not be added - add it to the `.gitignore`file


Generate a Vandermonde matrix.