Hi mate! Sorry for the delay. From memory I _think_ I took the `pngquant` from within ImageOptim.app but I'm not 100% sure, but I didn't build it myself - it was taken either from there or the pngquant website.


I can also see loading TomLib in lines 842 and 2716 above - perhaps the same should be done there? However, I think that it could be also useful to print about missing TomLib in this case, to advice the user that they have incomplete installation, rather then failing silently.


Originally I commented out the whole file.   I thought that was much worse.   I did make an issue to remind us to fix this.   What is your suggestion?


 Дмитрий. Ответил на ваши замечания в GitHub, но не знаю, дошли ли они до вас. Просто не знаю, приходит ли вам об этом оповещение.

>Среда, 23 декабря 2015, 4:32 -08:00 от Dmitry Kozliuk <notifications@github.com>:
>
>Но что произойдет, если попытаться перевести температуру из градусов Цельсия в них же (в общем случае, при from == to)?
>—
>Reply to this email directly or  view it on GitHub .


It should also do a nested `EvalRes` here, similar to `Compile`.


Might as well make them overloadable?


That's my bad :disappointed:


Style thing: can you put the } and the else if onto the same line? Not the only place :)


I don't know,  but every configure I use fails,  and I have found numerous real bugs with regards to variable initialization and memory freeing.

These test consistently fail for me,  and in the for that I fixed,   They are real bugs.

Are you building with stock clang?   I'm not doing anything special.   Came;  make; ctest;


Сбрасывать `last_error` необходимо, но делать это нужно не в `main()`, а в `convert()`: внешний код (то есть всё, что не генерирует ошибки) не должен изменять `last_error` (по определению), а иметь к ней доступ только на чтение --- собственно, это смысл существования `get_last_error()`.


@mcostalba @zamar @glinscott


BME does not have localisation for German language, ELTE has them localised.
"Un"fortunately, I haven't failed or missed any of my exams, so I can't get how the failed grades are called in each and every localisation.


@Malabarba Can this go into the v0.10.1 release, or would you rather I backported the fix in 003adaa11c7b417486f58a7265e85c5e4c6d188b to the old version?


Can this message be more adjusted?


Why wait so long with this recursive call? Can this not be placed at the top of this function?


test


You seem to mix tabs and spaces here. this is discouraged, indentation is with tabs only.


Please only indent with a single tab.


`docker stop` by default tries to `kill -1` the primary process, waits 10 seconds, then does a `kill -9` (`docker kill` just does `kill -9` outright).

This has been causing problems for me in that containers do not renew their leases.

On the other hand, if it sticks around and the container has been removed, then we will have zombie dhclient processes, which is not very good either.


Оповещения приходят. Ошибки нужно исправить. Когда я комментирую то, что исправлять не нужно, я прописываю это явно ("не ошибка" --- что-нибудь в этом роде).


nice! :heart:


Yes! Good catch. I should see about writing unit tests for this lib at some point.


@aravindvs ack. Will take a look over the next few days while OOO.


make some changes please


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


Awesome! I like it more with shadow, I'd also say we get some of the [wc3 design](http://puu.sh/m5Wsm/a0dbdb8d81.jpg) and align-right with some box radius, getting rid of the green/yellow/red color on the Food to add another No/Low/High Upkeep now that there is space for it.

The gold icon can be found here https://raw.githubusercontent.com/MNoya/PMP/master/content/dota_addons/pmp/panorama/images/custom_game/gold_icon.png


Эту короткую регулярку можно было засунуть сразу на след. сроку...


@m4dz I think you have to do a `cd client && npm install && cd ..` before that to install brunch & co.


Tested and full working :+1:


I can't figure out what for do you need this `include`


ou ça je vois pas ??


`std::swap`


Fixed, thanks. This was left over from when I was setting exceptions in this routine. (I had thought CMake was setting reasonable warning flags somewhere, but I guess not.)


@twoseat Does this actually work?  Specifically, when you run this in the non-security-group-rules mode, does `null` make this behave as expected?


This seems to duplicate https://github.com/gap-system/gap/commit/cff1d5109e65a18d6ae3e70fd2e500ab929ef813


Fixed, thanks.


@hulpke Sorry, please never mind - I've understood that it now follows my suggestion and uses IsOne instead of Order


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 869](https://teamcity.labkey.org/viewLog.html?buildId=293151&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 14 (1 new), passed: 248 Build time: 00:33:17
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.premium.FooterTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.JUnitTest$RemoteTest.org.labkey.study.model.StudyManager$DatasetImportTestCase: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDataTypes: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testCounts: <no details avaliable>

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


sounds good. thx!


hibernate 產生的DAO不會用 所以刪掉了


I'd probably call this `AbstractSpace`, since `SpaceDetail extends AbstractSpaceSummary` is a bit weird.


+Copyright (c) 2015 by JOHNSON S, SURAJ P, SUVITH KUMAR CV, VARSHAN BP <br>


拼接json格式字符串

需要包含`id` + `name/title`


Misspelled canols for canola


You may remove this copyright thingy ;)


Could you capture this as a TODO:? Then when that "rainy day" finally arrives we can find all these tasks and clear them up!


`.next().get()` is a blocking call.  Almost certainly wrong.


@bdilly Do you mean about my other comment in this series? Sure, I'll make it appear there.


True :-) I will do it


You can check its content using:


yes, sure


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 44](https://teamcity.labkey.org/viewLog.html?buildId=293144&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 15 (2 new), passed: 33 Build time: 01:06:06
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

org.labkey.test.tests.CDSSecurityTest.verifyFirstTimeUse: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <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.verifyGutterPlotBrushing: <no details avaliable>

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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


I want to learn more


@oney any reason why this needs to be done ?
"Before you run `react-native run-android`, you must run `react-native bundle --platform android --dev false --entry-file index.android.js --bundle-output android/app/src/main/assets/index.android.bundle`. And every time you modify code, you should run this command again."


Heck, if this is all you're sharing, is it even worth having a common class?


@DakotaLMartinez: thanks for submitting a pull request to fix this!


Agreed, although if you have e.g. `// Don't 'use strict'` at the top you probably want to keep it there (which will be the effect with the current code).


I've no problem with making multiple site hosting a paid-for feature, but as mentioned above, there are legitimate reasons to keep this bit of functionality and UI in place.

I think @burningice2866 's suggested solution of restricting hostnames to one instance is probably the best way forward. It lets Orkestra make some money, but keeps important single website functionality intact. We all hope you'll reconsider this change.


did you forget to set the .flags here?


@twoseat It generally looks good, but this isn't a final review.  I think that the rebase is going to hurt quite a bit, but should result in something more understandable.


Explain why – mention that the method acknowledges `first_day_override`. !fix


Use spaces, not tabs pls


nice example!


Shouln't we have a param to pass here? the default is not to run it as LOCALSYSTEM


nope, in C itl will automatically set to 0, but in c++ it's necessay explicitly set all the elements in order.


ok.


try to avoid this loop


Cool, thanks!

On Wed, Dec 23, 2015 at 3:44 PM Niklas Baudy <notifications@github.com>
wrote:

> Yup we could do that. I'm anyway already looking at every PR here.
>
> On 23 December 2015 13:20:11 CET, Artem Zinnatullin <
> notifications@github.com> wrote:
> >BTW, Niklas, do you want to review each PR on this repo so I could wait
> >for
> >your feedback before merge? Any answer is okay, don't worry :)
> >
> >On Wed, Dec 23, 2015, 13:23 Niklas Baudy <notifications@github.com>
> >wrote:
> >
> >> Isn't this going to breake incremental builds with every new git
> >commit?
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >>
> ><
> https://github.com/artem-zinnatullin/qualitymatters/commit/fd133553ef84dfe4ce3d88edda476213d2fbb080#commitcomment-15136560
> >
> >> .
> >>
> >--
> >
> >@artem_zin
> >
> >
> >---
> >Reply to this email directly or view it on GitHub:
> >
> https://github.com/artem-zinnatullin/qualitymatters/commit/fd133553ef84dfe4ce3d88edda476213d2fbb080#commitcomment-15137952
>
> --
> Sent from my Android device with K-9 Mail. Please excuse my brevity.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/artem-zinnatullin/qualitymatters/commit/fd133553ef84dfe4ce3d88edda476213d2fbb080#commitcomment-15138243>
> .
>
--

@artem_zin


Will investigate when I return from seasonal vacation on Jan 8th (modulo email backlog)

Sent from my iPhone

> On 23 Dec 2015, at 00:12, prem-jason <notifications@github.com> wrote:
>
> Hi please give me permission to contribute for organization memberships. Pls add my Id prem-jason
>
> —
> Reply to this email directly or view it on GitHub.
>


I found this image somehow redundant.
You can not really recognize anything on it. If the goal is to save bandwidth- drop it, if you want to show an overview of the interface, just show it frontal, without the random background and the tilt.

I`d drop it completely, since the important informations are not visible on page load (at least on normal size monitor/mobile) and you have to scroll down because of the tilt image. The frontal shot of the 2.0 interface is already on the front page, works for me.


Doesn't uncrustify bitch about the spacing diff?


це можна оптимізувати з допомогою циклу


Should be a ```debug()```


i don't see why, we can just try the next row


TeamCity Civic Assembly :: Build backend + frontend [Build 129](http://54.229.176.143/tc/viewLog.html?buildId=333&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


What if another thread is waiting on the same ```z_hold_locks``` entry?  Doesn't this need to be delayed until after the new ```zh_lock``` is taken below in order to prevent the same type of deadlock this is trying to prevent?


有bug，获取到的都是0，原因不详


actually uncrustify complains if there is no spacing diff, i meant it was wrong the code style


Note that Stephane recommends methods rather than properties for this.  Take a look at the [`master`](https://github.com/cloudfoundry/cf-java-client/blob/master/cloudfoundry-operations/src/main/java/org/cloudfoundry/operations/DefaultSpaceQuotas.java) version of this after his handiwork.


не виколристовуй echo для відображення html.
старайся розділювати файли: спершу бізнес логіка (php) потім вивід інформації (html)


Hooray!

> Am 23.12.2015 um 14:04 schrieb matcracker <notifications@github.com>:
>
> Tested and full working
>
> —
> Reply to this email directly or view it on GitHub.
>


not sure, but I notice it's been that way since 8f126bc14c1954d460f1c68a862bf7d7f23ad792 in June


この / なんですか？


подібний функціонал я вже в інших файлах - це можна винести в окремий файл\функцію і підключати\визивати коли потрібно і не дублювати кілька раз


So I haven't read the new code and don't know the details but.. If you don't derive from r_walls wouldn't that mean it would be super easy to dismantle the wall then? Besides the high fire resistance, or whatever, the walls would be pretty useless as a "security barrier" into restricted areas.

If you're going to replace all r_walls with this new type of walls I would call it an oversight. If you're simply adding it in for the joy of the players, without replacing existing r_walls, I think it would be confusing to call it a reinforced wall as it won't behave like the regular r_walls.


не явна перевірка...


いったんこれで
あとから必要なものは追加していきましょ


Seems `/EHsc` required for both win native and qt port


ここで修正していたのか


:tada:


congratz !


my idea was to say it a bit more imperative, since the user explicitly configures the import-sort-bear (like any other bear), and "can" is for me a bit like: "do as you please, it's your code" :P


Is this the expected behavior?  When I ask for a space-quota that does not exist from the CLI, I get:

```
$: cf space-quota alpha
Getting space quota alpha info as admin...
FAILED
Space Quota alpha not found
$: echo $?
1
```

That feels like this should actually be producing an error instead.  That being said, I believe that this can't be implemented properly for the same reason that [this can't](https://github.com/cloudfoundry/cf-java-client/blob/86cfe3e91b4ec370b7722a78bd1371a5c8629519/cloudfoundry-operations/src/main/java/org/cloudfoundry/operations/CloudFoundryOperationsBuilder.java#L126-L128).  I believe that we're waiting for a new feature in Reactor until this can be done.

Can you please `@Ignore` this test with the [same message](https://github.com/cloudfoundry/cf-java-client/blob/86cfe3e91b4ec370b7722a78bd1371a5c8629519/cloudfoundry-operations/src/test/java/org/cloudfoundry/operations/CloudFoundryOperationsBuilderTest.java#L50) as the tests waiting on the same functionality in `CloudFoundryOperationsBuilder`.


навіщо шифрувати $repeatpassword?


@muhamadazmy did so so that it doesn't spawn a child python process to run jspython, but rather to be in the same one.


тут варто використовувати тринарні оператори. і <?= ?>


`if(${QT_PORT})` -> `if(QT_PORT)`


Hello :)

Any News ?
Thanks for your job

Erwan


файно.


good point, please explain more,
tell me on telegram

On Wed, Dec 23, 2015 at 3:33 PM, Reem Khamis <notifications@github.com>
wrote:

> @muhamadazmy <https://github.com/muhamadazmy> did so so that it doesn't
> spawn a child python process to run jspython, but rather to be in the same
> one.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Jumpscale/jumpscale_core8/commit/f451a2f1984664d89e5ae5313cdff1d2d770a7f7#commitcomment-15138964>
> .
>

--
------------------------
Kristof De Spiegeleer
+971525609014
+201206927877
+32 475405474
skype: despiegk


`Qt5Core REQUIRED` -> Qt5 REQUIRED Core


the `mkdir` will return false in case directory already exists. Maybe it worth to skeep useless
check `!DirectoryExists`?


@Zteve Generally quite a good implementation.  The major functionality was exactly right, and Stephane was happy enough with it that he used it as an example for refactoring to an idiomatic implementation.  The "not found" case isn't handled properly at the moment (should be an error, not empty), but we don't actually have the Reactor functionality to implement it yet.


не лінуйся називати змінні зі смислом того що в них буде зберігатись...


typo


TeamCity Civic Assembly :: Build backend + frontend [Build 129](http://54.229.176.143/tc/viewLog.html?buildId=333&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 62 (60 new), passed: 73, ignored: 4 Build time: 00:08:29
### Failed tests
```
Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

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

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

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

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

Blogs tests: Should not create new blogs if cancelled: <no details avaliable>

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

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

Blogs tests: Should not delete blog if cancel pressed: <no details avaliable>

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

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

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


period?


It looks like

```js
db.query('byId', {
    key: id,
    include_docs: true
  })
```

tries to create a design doc, although `_design/byId` exists, 100% sure.

run `node tests/integration/api/accounts-test.js` to get the weird  `[forbidden: doc.type must be user]` error ...


оптимізувати з допомогою циклів


how about one line return statement?
means: `checkFile.exists() && checkFile.isDir()`


these lines (34 and 35) before commit 3c4960c were just really stupid :tongue:


можна простіше зробити


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 79](https://teamcity.labkey.org/viewLog.html?buildId=293139&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 16 (2 new), passed: 276, muted: 1 Build time: 01:41:00
### Failed tests
```
org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

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

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

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

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

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

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

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


also could be substituted with one line return


Т.к. вы решили не использовать getParam() из library.js, то в этой функции не хватает проверки
https://github.com/dukei/any-balance-providers/wiki/AnyBalanceAPI#isavailable


Move `if(ENABLE_LOG)` block to the left. Move `shared_library_qt.cc` and `shared_library_win.cc` out of the `ENABLE_LOG`. This is not related to the logging


Supers should stop the server. If they got reload ops would be able to do /ban for a period of time.


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 Total|||||


fixed: f8284e3


зрозуміло) на попередній комент до цього рядка не звертай уваги)


it looks like `std::copy`, isn't it?


looks like an issue with `pouchdb-users`

```
      forbidden: doc.type must be user
          at throwError (/Users/gregor/JavaScripts/hood.ie/server-account/node_modules/pouchdb-users/lib/validate-doc-update.js:5:17)
          at module.exports (/Users/gregor/JavaScripts/hood.ie/server-account/node_modules/pouchdb-users/lib/validate-doc-update.js:15:5)
          at modifyDoc (/Users/gregor/JavaScripts/hood.ie/server-account/node_modules/pouchdb-users/lib/modify-doc.js:9:5)
          at writeWrappers.put (/Users/gregor/JavaScripts/hood.ie/server-account/node_modules/pouchdb-users/index.js:18:14)
          at callHandlers (/Users/gregor/JavaScripts/hood.ie/server-account/node_modules/pouchdb-wrappers/index.js:447:17)
          at PouchAlt.put (/Users/gregor/JavaScripts/hood.ie/server-account/node_modules/pouchdb-wrappers/index.js:134:12)
          at tryAndPut (/Users/gregor/JavaScripts/hood.ie/server-account/node_modules/pouchdb/lib/deps/upsert.js:39:13)
          at /Users/gregor/JavaScripts/hood.ie/server-account/node_modules/pouchdb/lib/deps/upsert.js:33:15
          at /Users/gregor/JavaScripts/hood.ie/server-account/node_modules/pouchdb/lib/deps/toPromise.js:23:11
          at doNTCallback0 (node.js:419:9)
```


else if (num === 0) {


辛苦菊苣，效果很赞，能否把边框设为圆角，位置再稍微移下一行？
另外，鼠标 hover 上去后希望颜色能加深~


Before merging, I'd like to know is this really thread safe. Can we have multiple threads to use the same client, if so here we can do the use, and have another thread to also call use before this here calls the put command. There is no documentation, so I'm not sure is it meant to use this way... Otherwise giving one client per thread would solve the issue, or then I need to do this differently...


뭐 이건 코드를 설치 해보고 사용해보는 방법 뿐이겠네요 :) 좋은 코드 감사!... 근데..깃 패치로 우찌 받지.. 오피티컬이 아니네 ㅠㅠ..


Thanks, fixed!


Oh yes. I have just had several projects ahead in the queue. I will
probably put the quizlet plugins up there. I saw your theme went up there.
Good stuff.

On Tue, Dec 22, 2015 at 2:05 AM, Gareth J Barnard <notifications@github.com>
wrote:

> No worries Justin :) - will you sell some of your plugins on
> www.lmsmarketplace.com ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/justinhunt/moodle-filter_poodll/commit/7b593acc68a18d062d7050e8298c6f8dcd25eeb3#commitcomment-15101008>
> .
>


thank you for taking this step as didnt test for linux and mac.
Waiting for Pull request :+1: once complete.


no no.. thats my machine specific :P i wrote it for testing in my local machine ..


значительное ускорение


does your text editor force wrapping of Markdown files or something?

line breaks in paragraphs do get rendered: https://aframe.io/docs/core/component.html#Building_a_Component

![image](https://cloud.githubusercontent.com/assets/203725/11977466/1f75b9fc-a951-11e5-8d18-7e16fca4dd0b.png)

I tried looking into how to get `marked` to ignore line breaks in paragraph. filed #117 for this.

anyway, fixed: fa61dbe


After all it *is* his code...

2015-12-23 14:33 GMT+01:00 Mischa Krüger <notifications@github.com>:

> my idea was to say it a bit more imperative, since the user explicitly
> configures the import-sort-bear (like any other bear), and "can" is for me
> a bit like: "do as you please, it's your code" :P
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/8067942f51f3dd20d5117a12fd0e3ac3c2901a4d#commitcomment-15138951>
> .
>


typo: "a handful built-in"

fixed: 0ad5868


> These primitives are to help people get started with using A-Frame. To uncover the full composability and extensibility of A-Frame, dive down into the underlying [entity-component system](../core/index.html).

is there an agenda here to dissuade folks from using Primitives?


数据连接层


:+1:
although this function is getting quite big, we need to refactor it. Remove the shared state.


Happy to take a pull request with better logging. When the server isn't able to load a remote key set, we don't want it to crash, though.


Should be:
$this->jsEscape

As it generates the following error:
ErrorException: BBM Phrases to string error:Undefined variable: jsEscape - library/BBM/Helper/PhraseWrapper.php:41


This fixes issue #23 for newly uploaded projects; a9c408cb5ee3e16b421fce26846a12545ff673f3 does it for existing ones.


Yeah, we need something generic here or use the os module so that it runs on any machine.


it's good to add a disclaimer, but we haven't actually agreed on what we're doing here


Do we need to update all blocks with new API of redstone?


This fixes issue #23 existing projects; 86d87d46ed06db953565c98f07fb1b786cdea8e3 does it for  for newly uploaded ones.


...typo amendment


Maybe move the check for clang up to avoid gcc to be used on mac.


Great! :+1:


What needs fixing in DefaultFormulaVisitor?


Thanks for this pull request @ika18! Let's just update the comment to say:
    // It's best to also add width and height attributes for img elements.


'carthage update --no-use-binaries' worked as well


Should be:
 $this->jsEscape

As it generates the following error:
 ErrorException: BBM Phrases to string error:Undefined variable: jsEscape - library/BBM/Helper/PhraseWrapper.php:41


l


For a PF, I think you can do just
```
def errMatching = {
  case ... =>
}
```?


I thought about it, but generally g++ still produces better code in my experience.


Right, sorry.
Not that we had any quantifiers now though.


will do that. i will change it to os module. Thanks :dart:


아..제가건의할문제는 그냥터치한방으로 그냥 코어가전부켜저버리는게문제입니다 터치조금햇다고 코어가조금만돌면저도고민이없겠지만 그냥터치만했다고 그냥코어가줄줄이켜져버리니 졸라맨님이짜놓으신트릭이있으신듯하지만 무엇보다 코어를너무오바해버리는걸수정해야 실사배터리가잡힐듯합니다


Pour avoir la couche en édition sur la légende, décommenter la ligne 588 dans formDirective.js
Mais il y a une issue associée, c'est à dire que la couche en édition se dupliquait dès qu'on ajoute un nouvel élément puisque la légende qui est liée à la carte n'est pas recréée à chaque chargement de page.
Si cette fonctionnalité est remise, voir comment supprimer la couche en édition dans la légende une fois le nouvel élément enregistré.


Can you point me to the StackOverflow errors? I'm not seeing them in the vcloud diff you have linked to before.


Begin eines Events


This is an example script which will not be in the PR. Moreover, AFAIK if the switch is speciffied, the param value is TRUE. If the switch is not, the actual value is the one set as the default(false)


will do ;)


A bit of sztuka dla sztuki, but maybe you can fold the PFs sequence (advices) into a single PF, and then lift it to get a total function `String => Option[String]`? ;)


This is failing for valid class names. A simple project that uses JMS Serializer, for example, will complaining that 'Metadata\\MetadataFactory' is an invalid class name.


Hi,

Ooo, what version of CMake are you running, by chance? Mine definitely did
not like any other attempt at escaping.
On Dec 23, 2015 3:13 AM, "Edoardo Prezioso" <notifications@github.com>
wrote:

> This causes a CMake warning:
>
> CMake Warning (dev) at src/CMakeLists.txt:21 (ADD_DEFINITIONS):
>   Policy CMP0005 is not set: Preprocessor definition values are now escaped
>   automatically.  Run "cmake --help-policy CMP0005" for policy details.  Use
>   the cmake_policy command to set the policy and suppress this warning.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sirjuddington/SLADE/commit/8d9427452fb24c08f4c8dc2d65d78ccc20c47f38#commitcomment-15135528>
> .
>


`loadFromProjectClasspath`? As opposed to the compiler's classpath? :)


this kind of assumes that anything ending with `.jar` had the `jar:file://` prefix


код с запашком. большим. очень.


unclosed stream? Why not just `XML.load(URL)`?


Стоит еще раз детально изучить код maple. 70 строчка Motor.pde - управление esc происходит на частоте 500 гц.


Really brave getting into the LINQ provider! Thanks!


Why do you need this test? I guess you'd another error in getMetricsFromNode anyway, right?


Congratulations! Seems like a nice christmas present you gave to Stockfish.


Fixes #544, #545


Hello brandon, im currently writing an OTP server for Disneys Toontown Online. I have some very promising progress done. But i need someone to manage the launcher and website side of the project, would you be interested in doing just that? Thanks, Skipps


Also: my skype is potco108


PRs ought to be less so trivial and more necessary...


Of course, I'll tune this


Why don't you use ```GrabFromKubelet``` method?


Not needed


**Code Quality: Nitpick**

You could implement this template using the `base.html` template. Just extend and modify.


JavaDoc


en conflit avec quoi?


I'm not - I put it here. I can move it there though.


效果炒鸡棒！

不过可能要稍微右移一些，让对下和对右的空白区域大致相同，注意滚动条~


I don't want to list all Nodes every single time, and I need to get port from each.


**Code Quality: Nitpick**

Don't leave style syntax within the templates. Refactor wherever visible.


Андрей, можешь рассказать, что происходит при таком подключении?
Выглядит как: при подключении  git-sync-test, вызывается некое фиктивное подключение `package-loader.os`, в котором переопределяется стандартное поведение загрузчика библиотек...
Что за шаманство? :) для чего это?


Hi,

Without effect  for me, cache:clear command creates fileName too long


This is causing me a problem: I have code where I'm removing a line class followed by removing a line widget. Therefore by the time the editor update runs, parentNode becomes null. Adding `if (line.widgets[i].node.parentNode)` helps.


Remember that I will file KEYWORDREQ for both mpv and cmocka once this is merged. I've discussed this with @idella and he suggested to do it this way. I personally don't mind to do it either way.


**Code Quality: Nitpick**

Indentation here is a bit off and you can make use of the `{% empty %}` within the `{% for %}` to take care of cases where there are no items within the list e.g.

```jinja
{% for i in l %}
<!-- Do something -->
{% empty %}
<!-- Do something when there are no items in the list -->
{% endfor %}
```

This is much more elegant and more readable.


cc @francesco-romano @jeljaik


hi @ibauersachs,
you haven't updated jitsi-universe
https://github.com/jitsi/jitsi-universe/blob/cb1bfb1c9ed4de820496bfae2aa95a06114f2379/pom.xml#L190


cc @francesco-romano


Should be covered by the file in the impl


Without effect.


**Code Quality: Nitpick**

You could replace this bit with an `{% else %}`.


Этот метод можно написать одной строчкой "return column != 0 && column != 1"


Remove


No. Tabs should align comments perfectly. Open file in a text editor and you'll see for yourself.


The now missing "NUnit.Runners" is breaking NuGet. The ID cannot be empty or null!


what..


Comment


**Code Readability: Critical**

Indentation for this file could be much better.


I wonder if it's possible to redefine undefined in any version of Node.  Still, I think it's good practice to make your code work in as wide a range of execution contexts as possible if only minor changes are required to do so.  On the other hand, Object.keys is an ECMAScript 5 feature so the code probably wouldn't work in older browsers anyway without adding a polyfill for it.


I wonder if it's possible to redefine undefined in any version of Node. Still, I think it's good practice to make your code work in as wide a range of execution contexts as possible if only minor changes are required to do so. On the other hand, Object.keys is an ECMAScript 5 feature so the code probably wouldn't work in older browsers anyway without adding a polyfill for it.


Do we need this ?


**Code Quality: Nitpick**

Remove this bits of commented out code.


**Code Quality: Nitpick**

Javascript functions should be written using Camel Case.


koluni kana be???


are you sure you want to keep this?


but only in the new major version. It may be worth doing a patch release first, no ?


and it may be good to make a minor or patch release triggering a deprecation warning when using Glob without escaping btw, as 4.0 will remove it. Easier upgrade path for users.


This line is to long! You don't need to check against boolean {% if sortable == true %}, just use {% if sortable %}
For false check just use {% if !border %}. If the line is still to long, than make each html attribute to one line.


**Code Quality: Nitpick**

There's a popular debate about this in the programming community.

```js
function() {

}

// Versus

function()
{

}
```

I really don't care which you do but whichever you're doing, you need to stick to it. I personally prefer.

```js
function() {
    // Function Body
}
```


so nice


Simply checking for null would prevent the unnecessary String.valueOf(allowedFailedLoginCount) :)


i think the demo won't cut it, you need to apply the bds8vcl FLIRT signature


What is sbd? Can you make nicer names, so everybody can understand what you mean.


New addition to my init file: `(setq text-quoting-style 'grave)`.


Actually this link does not have the CIEL dictionary. It just shows how to get it. :)


In this particular case it seems better to explicitly set flags to zero (since it works for C), avoiding having to sync those two almost identical macro definitions.


:heart:


Why use append here vs. just adding them in the list as above?


变量的作用域改成：带$符号的是用户变量，绑定到一个动作序列（剧本文件）上；带&的是系统变量，系统自动存储。这样的话，就不需要考虑同一个剧本文件里不同语句块中的变量作用域问题了……


Thanks for the suggestion. I'm going to rewrite a bit more of this paragraph and incorporate your change.


err is a string so can't be nil and, will be initialized as an empty string. So I believe you can't make that evaluation.
Then the error is because each verification function, for local and for domain, know why it failed and have different messages. Should we just provide the error in the function client?


@damianszczepanik i just encountered this issue in the jenkins cucumber reports plugin.
was this added to version 1.0.0? thanks in advance


How about just have an `apply` and a `detach` method (and also make this a class so there are no name clashes)? Then have the MaterialScrollFire attach an AttachEvent.Handler (addAttachHandler) to the target widget that checks `!event.isAttach()` so when the widget is detached physically it calls `scrollFire.detach(element)`. Rather than firing an event timer every 100ms for the applications entire lifespan, which also adds a new timer every time you call `apply` causing major performance problems.


Oh, I see. This is essentially superceded by a later patch. Perhaps it may be useful to squash them.


Idea for these `sol_buffer` entries: have a definition called `SOL_BUFFER_C_CAST`, that is empty for C++ but expands to `(struct sol_buffer)` for C. And use that in all those defines, avoiding the repetition of each of them.


A nice feature would be to allow the user to enter the budget and see how it affects the selection of players.


Init and its catch here smells fishy to me. PromptPanel.init() accesses the paramDfn member directly, always assuming that there is one[1]. This will then throw a exception that is swallowed as "meh, I have been initialized already. Bugger off" even though that may have not been the case.

You cannot call init() until you have called "render", as init still requires requires a not-null parameter definition.
And we won't have a valid parameter XML until you called render,

[1] @see the line with
    if (!this.isRefresh && this.paramDefn.showParameterUI()) { // First time init"


Nice!


Hi,

This has been addressed [here](https://github.com/sirjuddington/SLADE/pull/410). My apologies for that-- no idea how I missed it. The new behavior is so much better (and better looking).


The code is correct.

If the dbPropsCache object returns *null* the db properties are not cached, thus they must be loaded (via getLoading() method invocation).

If the dbPropsCache returns an Optional with null value, it means that the properties are actually cached and that they are null (already tried to load them but not actually existing).


Move to a config file?


I wanted to check whether file can be loaded in order to ignore errors on missing file but I just wrote better error handling without openingStream.


this looks weird to me, as the (exported) class name matches the regex and so will be valid. Please open an issue with a complete description to reproduce it.

A comment on an old commit is one of the worse way to report an issue, as anyone not reading all their github email notifications (or having disabled them) will never see it, and other people will forget about the issue report as soon as they close the page where the comment was posted


maxlength 애트리뷰트를 안 쓰는 이유 : [엔터!!!!!](http://stackoverflow.com/questions/10030921/chrome-counts-characters-wrong-in-textarea-with-maxlength-attribute)


It assumes that it had `file://` prefix. Is it possible to have different kind of urls in the classpath?


'k


I think calling a string return err seems misleading. I suspect the idiomatic thing to do is to just use an error instance instead, yeah.


This is contributed by @haniokaisai . ;)


I don't know for sure since it's not my machine, but I think it's the stock clang.
clang++ --version
Apple LLVM version 6.1.0 (clang-602.0.53) (based on LLVM 3.6.0svn)
Target: x86_64-apple-darwin14.5.0
Thread model: posix

Can we just disable/make not default if (NOT VXL_BUILD_DASHBOARDS) ?


Are we going to handle the configuration to the backend via context decorator as in mds or use an attribute for the class? My vote is on AmalysisClient.connection = 'my_url'


No crashes, agreed :-)   though I am not sure why it would crash less from an IllegalArgumentException than from the previous JsonParseException. The method still throws an exception either way.


I didn't see the issue. That is good enough, I suspect.


Move the values of url, salaryURL, and salaryPostBody to a config file?


@nairdo This was pushed to 1.4 branch after we tagged and released to beta, so I reset the 1.4 branch back to the previous commit. So this commit is now in the hotfix-1.4.1 branch (and develop).


Missing parameters argument for renderArchive
see https://github.com/sonata-project/SonataNewsBundle/issues/284


Missing parameters argument for renderArchive
see #284


Why not kick an amount of bots ?


Global comment: The API feels better with the ID being supplied when you grab a reference to the API instead of giving it to the render function. I like that change.


@AByzhynar , @malirod  created new Task: APPLINK-20235 SH: 189ea41


Nifty! :+1:


OK. I figured it out. The 'dateModified' column is the version column which you already have logic for built into for updating LocalDateTime types. Consequently any Update transaction will cause incrementing of that property. I'll add a different example to the test that covers the more general idea of a preInsert/preUpdate behavior.


> The previous way it was looks fine to me: if both identifiers are `nil` then it'll skip to the next test.

That wasn’t happening. It was returning `NO` here and never reaching the other tests.

I think if we were doing `nil != nil`, it would work as you expect. But we’re sending a message to `nil` and testing the return value. I suspect when you send a message to `nil`, the return value is always `nil` too, and `!nil` means true which is why this returned.

> If one is nill and the other not, then surely they aren't the same right? If I'm wrong, why only check to see `[self identifier]` and not `[anObject identifier]`?

I wasn’t 100% sure that I could skip testing `[anObject identifier]`. I’ve thought it through and it needs to be here. Fix coming right up.


``` sql
--
DELETE FROM `game_event_creature` WHERE (`guid` BETWEEN 52265 AND 52269) AND `eventEntry`=2;
INSERT INTO `game_event_creature` (`eventEntry`, `guid`) VALUES
(2, 52265),
(2, 52266),
(2, 52267),
(2, 52268),
(2, 52269);


These blank lines should be removed. Also, this commit should be rebased into previous commits.


Working on a reproducible test case for you guys to open an issue, left the comment more for myself to recall the problem area :)


Fixes #60


We should check first, whether `random.sample` from the Python standard library is not good enough for our purpose, I guess.


So results are inserted as embedded docs in the body of headers?


Used in matchManagedConnections in MCF


Please add space before />.


Credit give to Nathan Benis, I cant cherry-pick his commit then I must do it manual


feels like this should be covered by a test


what's this stuff?


Oops, this commit removes `--no-merges`


Awesome, thanks. The rest looks great, thanks very much for this!


What I'm suggesting is to remove it completely. I don't think it gives us much and it takes space. But it's up to you :)


Well, shizzle my dizzle, I didn't know about the possibility of lifting partial functions! :astonished:
Will change that :)


I would guess XML.load also throws an exc if the file is missing ;)


no idea :)


Good addition then.


The angle is directly set only if we are not currently in cruise control. If in cruise control, then we should just update the angle value (_angle) and wait for the control loop to set it accordingly, when it is time.


This module supports notification handling from JS even the app is closed.It will launch a new `ReactInstanceManager` on the background in an [Android Service](http://developer.android.com/intl/zh-tw/guide/components/services.html), the ReactInstanceManager will load `android/app/src/main/assets/index.android.bundle` to running the app.
It can be more specific.


142 additions and 115 deletions.  :-P


Will this code also work for the ServiceResponse stuff? E.g. you're replying async to an earlier invoke call.


Netbeans project files, should i remove them?


I don't like a blanket except. This should probably be "except OSError:". Is OSError the correct exception in this case? On my system I have ftd2xx64.dll


what is a 'summary node'?


Changing this project id hinders me from compiling the latest version of the library. If I revert the value, it works.


I note that unsafeUpdate doesn't currently publish UpdateEvent. Is this intentional or should we add it?


That's really cool :+1:


@steve-manywho the `sendResponse` method? No it won't but I'll add it in. Should it happen on any other calls too?


:+1: x 1 mil for proper use of singledispatch!


git


bah, sorry about that, will undo


I looked at the only test that wasn't using it :) Mind fixing that test to use that method too, for consistency?
There's no need for another review round, just fix that and squash the commits into a single one and I'll merge the PR. Thanks!


move back


add README.md


给头像点赞，不过unity是什么鬼?


it is also necessary `=` in c.
I thought do this, but i prefered make it explicity since it's very trick. But if you guys prefer the macro i can easily change


input 에서는 maxlength 괜찮. 어차피 엔터 안 되니까.
한글도 서버/프론트 다 괜찮은거 확인함.


感谢作者


you don't need to preload boring-ssl compat since it's already preloaded (rom wouldn't boot without it) and  why did you add boringssl-compat to our sources since it's already into cm source? :)


tks bruno, it works fine:
http://vdlmap.com/parco/pai/indexfullscreen.htm
valerio


Yup - that's the one. I think it's just execute and response stuff - nothing else. Anywhere where you're expecting the engine to do some crunching against the state. Join doesn't need it for example as you're not doing processing.


It didn't close the pull request when I pushed the updated branch, so please review that when you have a chance.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 78](https://teamcity.labkey.org/viewLog.html?buildId=293142&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:14:10


Cool - I'll extract it out and add it to that method too then :+1:


Open a new issue for this.


What do you mean. Please explain.
I known same variants:
1. current for (..)  it's simple variant
2. file_stream->write(reinterpret_cast<const uint8_t*>(data), std::streamsize(data_size*sizeof(double)));
for ofstream, because std::copy worked with vectors.(We have uint8_t *.
or
3. your variant....


@jtkech Thanks for the clarification, that's helpful. That also means I effectively undid what the original change "fixed". I hadn't realized this was about an external Save button. In that case, the code in the changed file as part of this commit should be removed, as it doesn't belong here. Do you know which Save button this change was concerned with?

The right fix would be for the media library picker script to trigger an event when media items have been added/removed, so that the external Save button can hide/show itself based on that.


ack


Put images here - https://github.com/Microsoft/PowerBI-visuals/tree/resources


[minor] or you could reduceLeft starting with PF.empty ;)


Yes.  It is an OSError.

The other possibility would be to use the find_library function.   Then there would be no try/execpt at all.

I used the most recent windows installer to have the drivers/libraries installed on a Windows 7 x64 machine.  Somehow, the installer changes the name of the dll when installing.  The library is called 'ft2xx.dll', but it is identical to the fdt2xx64.dll in the driver cabinet (and different from the 32 bit library in the cabinet.)  Go figure.

Anyway... thanks for the work... especially getting it to work with python 3.


I was thinking configuration per instance.


test


hahaha

I'll check on the normals... it was working for me with the sphere though...

happy holidays :snowman:


@dstoup

I am very happy to disable "if (NOT VXL_BUILD_DASHBOARDS)"

NOTE: I sent a mail to the vxl-users list in an attempt to motivate a conversation that may help determine who wants to maintain the BRL code.  Since I don't know who uses it, or what it is suppposed to do, I have found it really difficult to fix errors.

Thanks for your help.  Hans


3rd version. Exiting on finish() makes changing UI settings impossible (they call finish()) ;(


test commit


![disc](https://cloud.githubusercontent.com/assets/1950251/11978447/fbba5384-a98b-11e5-9bee-3eea9ec3f4b3.png)

@carlosjoserg , @memnone:
By looking at the pictures the tangent disc are overall acceptable and the normal (blue arrow) is reasonable with respect to the GP (green points).
However for some points the normal is completely weird, i think maybe it is because the the normal gets computed in a close-to-the-border region where the variance is higher with respect to center of the object (blue points).

I think we should investigate if there's some weird math error or it is simply this way, cause you know the GP is still an approximation.


Oh, meant that to just be `notifObj`... fixing.


Well, with current loading scheme they should be skipped (`IOException` on `XML.load()`), so I guess that's not a concert atm


no, they go in their own collection, I was sleepy when I wrote this so the language is not perfect.  Should be more like "Add results associated with header" or "Add results for a given execution" or something like that.


How can you have literals as subjects?


+|Bill Presentment API                              | `50400`-`50599`            |


Pressurestat (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1980 of 1980 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)


I guess if we pass an URL to XML.load it would work with any URL, but true, not a concern atm :)


Methods whose singular parameter is `void` cannot be handled correctly in JRuby 9.0.4.0, which differs from MRI.


这里可以改为：

“``record`` 函数将为传递给它的日志信息创建一个只包含单个元素的列表。”


also, in C++ is necessary initialize all elements in the order they are declared


SadGit


Wtf is that pr :O You've remove all the security about gamemode? :O


reack


TeamCity GitLab Projects :: Build [Build 15](http://teamcity.ppolivka.com/viewLog.html?buildId=39&buildTypeId=GitLabProjects_Build&guest=1) is now running


:+1: thanks.


We'll readd only the forcesurvival feature


TeamCity GitLab Projects :: Build [Build 15](http://teamcity.ppolivka.com/viewLog.html?buildId=39&buildTypeId=GitLabProjects_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:33


TeamCity GitLab Projects :: Build [Build 16](http://teamcity.ppolivka.com/viewLog.html?buildId=40&buildTypeId=GitLabProjects_Build&guest=1) is now running


Add the delayfromnow constants here


Why do we need to remove and restore gamemode?


Includes styling to Cart and additional elements introduced by own plugin.


TeamCity GitLab Projects :: Build [Build 16](http://teamcity.ppolivka.com/viewLog.html?buildId=40&buildTypeId=GitLabProjects_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:31


Curious: After this change; Archiving from /users/?/messages DOES seem to move it to Archived, but archiving from t/test-mail-for-support2/7768 (Handle button, then Archive Mesage) doesn't move it, but does lock it.


@dotansimha
What was the problem before and why did this commit fix it?


Presently, the load operation fails with:

```
NameError: uninitialized constant Fiddle::CParser::TYPE_PTRDIFF_T
    from org/jruby/RubyModule.java:3181:in `const_missing'
    from /home/developer/jruby-9.0.4.0/lib/ruby/stdlib/fiddle/cparser.rb:160:in `parse_ctype'
    from /home/developer/jruby-9.0.4.0/lib/ruby/stdlib/fiddle/cparser.rb:169:in `parse_ctype'
    from /home/developer/jruby-9.0.4.0/lib/ruby/stdlib/fiddle/cparser.rb:88:in `block in parse_signature'
    from org/jruby/RubyArray.java:2295:in `collect'
    from /home/developer/jruby-9.0.4.0/lib/ruby/stdlib/fiddle/cparser.rb:88:in `parse_signature'
    from lib/opengl/gl.rb:47:in `extern'
    from lib/opengl/gl.rb:212:in `import_symbols'
    from lib/opengl/gl.rb:42:in `load_lib'
    from (irb):13:in `<eval>'
    from org/jruby/RubyKernel.java:978:in `eval'
    from org/jruby/RubyKernel.java:1291:in `loop'
    from org/jruby/RubyKernel.java:1098:in `catch'
    from org/jruby/RubyKernel.java:1098:in `catch'
    from /home/developer/jruby-9.0.4.0/bin/jirb:13:in `<top>'
```


Change list of fonts to Tahoma,Verdana,Geneva,sans-serif


`local_part_of_iri` is basically `(REPLACE(STR(?iri), "^.*[/#](.+)$", "$1") AS ?lname)`?


nice


:+1: this was bugging me too


ack


std::copy works with continous array not with vector, so my suggestion looks like this

  std::ostream_iterator<int> out_it(*file_stream);
  std::copy(data, data + data_size, out_it);

actualy `std::copy` reauires some random access iterator. The raw pointer is such kind of iterator


Remove brackets


Just saying.. this would crash my pocketmine on windows/not run it.


для чего uri ?


Other changes look good.


“然后我看开始写 ``Monad`` 实例的 ``return`` 方法”

这句可以改为：

“让我们以 ``return`` 为开始，构建 ``Monad`` 实例”。


napredujes


Why did you add nodejs package here? It is already being installed from source.


Can the problem be reproduced in the online demo?


Haha, thanks! Makes it alot shorter


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


Why


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1054](http://teamcity.monogame.net/viewLog.html?buildId=12891&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Please revert the change of ordering. It's not needed and it interferes with this diff.


return false;


`static_cast<int(...)`. I think Clang will warn on old-style cast.


After much thought; agreed. As the number of operations increases, the number of such private functions required will increase, and it doesn't make sense to construct them all on every instantiation—yes. However, in the case of static properties (function bodies that do not depend upon instance properties of the containing class) these function objects would be created only once.  Even then, this might be once too much; so yes, we can see that this policy is probably the best one to adopt uniformly, and consistently.  Can we document this in guidelines, somewhere, please?


“``(>>=)`` 把一个普通的值和一个monadic的函数结合起来然后得到新的运算结果和一个新的日志信息。”

这里可以用逗号分割一下：

“``(>>=)`` 把一个普通……结合起来，得到新的……信息。”


rdfh


здесь не хватает курсора, который покажет, в каком виде ты хочешь получить результат
->as_array() или ->current()


Pressurestat (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1986 of 1986 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)


@matcracker No. You could add it to every block, but because every block extends this block.php class they would already have the code in them


“ 之前的输出的日志后面。”

改为：

“之前输出的日志之后。”


Some people don't allow a permanent creative mode, and they maybe want it to be remove at the end of the session :)


зачем внутри модели вызывать эту же модель?


что с табуляцией? проверь все отстпупы


I have done this some places in the RM application, but it seems fragile if we have to do this in every view because of a bug in the framework


Я экспериментирую с возможными способами организации программ и библиотек. Заодно и тестирую алгоритмы загрузчика внутри движка - сработает или нет. Работает, как задумано.

Теперь по шаманству: библиотека может иметь собственный загрузчик (отличный от штатного). При подключении #Использовать gitsync берется переопределенный загрузчик и штатным загрузчиком подключает **core** по пути.


зачем эта функция?


ping @Guite


откуда цифра 13


Thanks for making this fix!


If this is not a public header, then it can be removed from the new `exports` file. Ideally, that file would list only a few headers.


Steps 6 seems not work for both unfactor.py  and unfactor-ecdsa.py


@netanelgilad
the $watch callback called with `newValue` and `oldValue` with the same value (the new one), not sure why, checking it now...
Do you have any idea?


Pressurestat (Myoldmopar) - 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)


Is it possible to put this file into `./setup/functions` and to have it automatically have the constants exported in the loop?


@oney is there way a way to just use the js bundle thats used for the app, since it can take quite a while for the bundle to be completely written.


Is there we would fix the issue where alarm doesn't boost itself to it's max volume when device is connected to Bluetooth audio?

Example: On any stock ROM, with Bluetooth audio connected, if Bluetooth volume is set low, and an alarm kicks in, Bluetooth volume with then be overridden by whatever alarm volume is set to. This is the proper way. However, it is not the case on cm.


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


I moved these duplicated methods up to a more general location, but forgot to delete them in one place.


Just moved beforeEach before afterEach.


Wrong indentation.


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


So mononcqc and jlouis666 [helped me understand](https://twitter.com/vidbina/status/679659666817859584) why `Sublist++` is a bad idea. The `++` operator [copies](http://www.erlang.org/doc/efficiency_guide/myths.html#id61192) the left-hand operand to append the right-hand operand. If `SubList` contains N items, that would mean copying N items on every pass.... oops


The helper method was also doing ".element", which means that it was effectively searching the whole DOM (since staffArea.element.element is undefined).


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


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


OK, yes, I see that now that I reviewed the rest of your code (and have a chance to respond).  Still, in the interest of making the function a bit more general and reducing coupling a slight bit, it might be good to make it work in the case that one of the positions doesn't have a player or at least use an assertion to let the user know there's a problem instead of producing an incorrect result.  You may also want to use JSDoc to let users know the expected form of the input positions.


your proposal does work, actually it is similar to "find_last", but will return on any match, instead of remembering the last found.


I think this is nothing AuthMe should deal with.


All commits including this one are aiming to make this module compatible with https://github.com/codemix/yii2-localeurls


Separate assertions are better because if one fails, you know which one.

Anyway, this commit really should be separate. Instead of opening a new pull-request, I'd squash the warning-fixes into a single commit (I can show you how via chat if you want to see the tricks) and keep this as its own commit.


Nice job


Thanks!


Congrats Panchjanya :+1:


Right, it's Rails 4x only :+1:


TeamCity GitLab Projects :: Build [Build 17](http://teamcity.ppolivka.com/viewLog.html?buildId=43&buildTypeId=GitLabProjects_Build&guest=1) is now running


I fixed start.cmd


lol, just download zip!


Verified working now!  Thank you.


TeamCity GitLab Projects :: Build [Build 17](http://teamcity.ppolivka.com/viewLog.html?buildId=43&buildTypeId=GitLabProjects_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:52


サーバーで実行できなかったので廃止


Good catch, @azeno. I've [corrected that](https://ci.appveyor.com/project/AArnott/immutableobjectgraph/build/artifacts).


`if tmp_date<self.time_interval[0]` should have taken care of that. did you find duplicate viruses?


另外，
![0o988 jomq2ab5oge_ioo0](https://cloud.githubusercontent.com/assets/1846356/11979034/0a8a190a-a9cb-11e5-91c2-6c7127554cc4.jpg) 让点击这里跳到首页吧

![f4 l5 fw r po 06ad a](https://cloud.githubusercontent.com/assets/1846356/11979044/0f6c8ce6-a9cb-11e5-8995-e16a96e21411.png) 下载按钮这里不是很居中的感觉


If `i<h`, then `i-h` will underflow, so `i-h+n` will underflow and then probably overflow. I *think* that's fine, but `n+i-h` is better.

Also, this commit is another that should not be squashed. (It's a good idea to squash most others before merging.) I guess that's obvious.


Presumably you don't want this commit in the pr.


辛苦菊苣，感觉那个 tip 的箭头好像不是很必要，不如去掉吧~ 另外 tip 外面能加一圈白框么，然后再在白框外面加一圈 tip 的颜色


fixed #4


Но разве это не то же самое, что и  `#Использовать "../src/core"` в старом варианте?


done


Add exception if remoteID is null.


I like the recoloring. in fact, this is what I also settled for in the refactored auspice I sent a couple of days ago. there I defined `updateVisibility()` that does that


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


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


Looks like you did not consider the configuration of ethernet device itself. My suggestion is even you don't want to do it now, you need to keep the interface for it.


Thank you!
May I ask, what auto-completer are you using for this REPL config?


You can call subroutine directly instead of to fork a process.


If it only happens on points close to the border, then I'd think it might be that f(x) = 0 is unbounded in the mug example (the surface is not closed). One expect the normal to continue along the surface, which might not be the case if the surface does not close.

Are you plotting the green dots as before, right? A grid, evaluate, and keep those below a given threshold. If so, how big is the grid? Is it like the bounding box of the object or bigger? Could you let it be twice the object size to confirm whether f(x) = 0 tends to be closed or open in this example?

In the sphere example, I'm pretty sure it is bounded, and in the google doc I pasted the normal vector plot.


I don't understand the point of this change. The copy ctor always exists, and the default ctor exists unless some *other* ctor is declared. Is the compiler-provided behavior for these wrong?


the name parse_nics is not reasonable for nic configuration.


They are encrypted (via `travis encrypt`) and just have access to write to one S3 bucket so I felt okay with that. But, I removed them again (https://github.com/cmake-maven-project/cmake-maven-project/commit/dd11cf7cbb5e1963fd60e75a987efe2e1de9870f#diff-354f30a63fb0907d4ad57269548329e3L56) because they are encrypted specifically for my fork. I was thinking they'd fail in a less catastrophic way in the main project repo but they ended up failing the Travis builds.  I'll have to keep them in a separate branch in my fork that's not intended for merging I guess.


Curious: why are we changing this? If anything, I'd personally prefer `.to eq true`.

Additionally, should we address the similar changes in https://github.com/tdouce/remote_database_cleaner_home_rails/blob/07d20d4f93fccb21b9db854bd40c2ead071749a8/spec/models/remote_database_cleaner_home_rails_spec.rb


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1054](http://teamcity.monogame.net/viewLog.html?buildId=12891&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:24:11


Like many things in here.


linan fyi.
download zip attach n pull req. pl.


Incorrect commit message. This update completed RecycleView animations.


That's because CMake offers the Linux 32 and 64 bit downloads, but only one (the 32 bit) Windows option: https://cmake.org/download/

If / when CMake offers the 64 bit Windows download we can support that too.


@danielinux,

I've fixed a couple of issues in previous commit. Can you confirm some, or maybe none, are fixed?
To get rid of the memmoves I implemented the moving myself, since I have to take overlapping chunks into account. Am I allowed to do this?

With geomess everything seems to run fine. I wanted to test this with the dev_radiotest-device, but I get segmentation faults though, which are kind of weird. When the radiotest-device is created it seems to assign only the 2 least significant bytes of the pointer to the radio-variable in pico-app. Which cause the application to crash in pico_sixlowpan_create().

Breakpoint 2, pico_radiotest_create (addr=1 '\001', area0=1 '\001', area1=0 '\000') at modules/pico_dev_radiotest.c:202
202	    return (struct ieee_radio *)dev;
(gdb) p dev
$28 = (struct radiotest_radio *) 0x600c0000bfc0
(gdb) finish
Run till exit from #0  pico_radiotest_create (addr=1 '\001', area0=1 '\001', area1=0 '\000') at modules/pico_dev_radiotest.c:202
0x000000000044b6c9 in main (argc=5, argv=0x7fffffffe478) at test/picoapp.c:487
487	rea1);
Value returned is $29 = (struct ieee_radio *) 0x600c0000bfc0
(gdb) s
488	rea1);
(gdb) p radio
$30 = (struct ieee_radio *) 0xbfc0
(gdb) p *(struct ieee_radio *)0xbfc0
Cannot access memory at address 0xbfc0

I've looked at it for some time now, but can't figure what's the cause here. Do you perhaps know what's going on here?

Jelle


Thanks Hans, I appreciated the response to those Ian and Tim ... I was
writing something similar as well, but never got it sent. It's great to
have them chiming in.

Regarding the BRL default, I do think it's probably better to keep it
turned off unless we've enabled the dashboard flag, rather than just
shutting it down on Mac. I would really like to prompt those guys to fix
issues ( which they are starting to do ) and my fear is that if we just
turn it off for Mac, then it will stagnate and die.

Thanks for your patience with my nitpicking .... !!

-Dave

On Wed, Dec 23, 2015 at 9:48 AM, Hans Johnson <notifications@github.com>
wrote:

> @dstoup <https://github.com/dstoup>
>
> I am very happy to disable "if (NOT VXL_BUILD_DASHBOARDS)"
>
> NOTE: I sent a mail to the vxl-users list in an attempt to motivate a
> conversation that may help determine who wants to maintain the BRL code.
> Since I don't know who uses it, or what it is suppposed to do, I have found
> it really difficult to fix errors.
>
> Thanks for your help. Hans
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/InsightSoftwareConsortium/vxl/commit/6e722c63363ee5fd6988da750c2a45ab200fcca8#commitcomment-15140082>
> .
>

--
David Stoup
R&D Engineer

Kitware, Inc.
28 Corporate Drive
Clifton Park, NY. 12065
518-881-4949 (W)
518-312-3946 (M)
518-371-4573 (F)


you really need to clarify the input parameters


Can you fix the `Usage` string?


Add more comments are recommended. You have not any comments in the whole subroutine.


Hi. How did you get this to work without the `sleep 300`? I am continually running to a race condition where all restart commands after the first fail because monit doesn't process commands instantaneously but every minute or so. Any commands that you try to issue when there are already pending commands on a process do not get queued, but fail miserably with this error:

```
monit: action failed -- Other action already in progress -- please try again later
```

This comes in to play during setup because the worker processes are started during the setup phase, then almost immediately restarted during the deploy step. That sleep was to give monit time to process the restart command, so the other restart commands can be issued later without failing. Is there anything else you did to get around this?


Not worked on msvc 2010x32.


I rather liked this micro-optimization. Is there a reason to not use it?


Why output the info and debug messages to &2 (Error)?


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


bench: 8529898


[I] should be [D]


It's not worth it for the sharing, but it's the only way (I know of) to have a Lombok builder annotation on both classes - if Detail inherits from Summary direct then Lombok barfs.


Unless I'm greatly mistaken, you can't capture the American command centre and get a paradrop. You have to actually be America. None of the other faction-specific stuff could be built by capturing.


@mathslinux could you use the notation of
```
Fixes: #14090, #14091
```
instead? you can put it right before your "Signed-off-by" line.


not sure we want that change in


This sentence looks like call itself again, why?


Ok. I'll just leave an official "ewww" here.


<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/> should be removed, <classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.8"/> should be kept


I think I may know where the fault is, there's no include in picoapp.c of pico_dev_radiotest, so pico_radiotest_create is an implicit function declaration.

Doesn't it trigger a segmentation fault on your host then, @danielinux?


Good job! But what if outputName, contains folder(for example: saveResource ("aaa.txt", "temp/aaa.txt",true)?
You need to check existence target file's parent folder too.


are you sure this is all ?


I have this one:

$mingw32-make -v

GNU Make 4.1
Built for x86_64-w64-mingw32
Copyright (C) 1988-2014 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html
>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

On Wed, Dec 23, 2015 at 1:11 PM, mstembera <notifications@github.com> wrote:

> See comment here #532
> <https://github.com/official-stockfish/Stockfish/pull/532>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/official-stockfish/Stockfish/commit/90f5937373974adf4ae7216feedf5abc4d62debd#commitcomment-15137835>
> .
>


As per last email this is no longer required. Please remove this file from repository.


Although cute, the popup is also (like all such cookie banners) very annoying. Is it possible to just put a (non hovering) notice at the bottom of the site instead?


Pressurestat (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (912 of 912 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-25.87%25-green.svg)


Is this just copied and pasted?  How many of these helpers are actually used by this module?


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


@Thunderbottom omnibox should now work correctly


For future reference, better way of getting a random record from DB without loading all records into memory:

`Person.limit(1).offset(rand(self.count)).first`


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


...seriously


Cool, will check it out


You don't need to sort here.. just use unshift above when similarity > maxSimilarity


travis picked `transformers-0.5` even on 7.0, and is green. Even with ghc-head!

Changelog doesn't mention anything `mtl` have to worry about  AFAICS http://hackage.haskell.org/package/transformers-0.5.0.0/changelog
`transfomers` diff: http://hdiff.luite.com/cgit/transformers/diff?id=0.5.0.0&id2=0.4.3.0


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


sdsdsssssss


@otakuchiyan You updated the version, but you didn't make a new release.


Да, то же самое, но уже не зависит от внутренней структуры. По сути, это тест поведения, когда внешний скрипт загружает просто gitsync не зная о структуре внутри.

-----Исходное сообщение-----
От: "Nikita Gryzlov" <notifications@github.com>
Отправлено: 23.12.2015 18:17
Кому: "EvilBeaver/oscript-library" <oscript-library@noreply.github.com>
Копия: "Andrei Ovsiankin" <ovsiankin.aa@gmail.com>
Тема: Re: [oscript-library] fixed #42 добавил возможность подключения gitsync как библиотеки (6d15699)

Но разве это не то же самое, что и #Использовать "../src/core" в старом варианте?
—
Reply to this email directly or view it on GitHub.


getAssessment is also doing ".element", resulting in staffArea.element.element, which is undefined (meaning that the selector acts on the whole DOM). So there is cleanup here and below to remove the extra ".element".


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


Why not `return_path=$(cygpath -u "${1}" | tr -d '\r')`?


Other than the aggressive asserts in `_unsetOp` and `_setOp` and the loss of the micro-optimization it looks good to me.
I understand why you want to get rid of the micro opt: it is this class taking matters of thread identity into its own hands. It occurs to me that we could restore the symmetry with the caching of the `_writeTask` and `_readTask` instances and make cached `_stopReadTask` and `_stopWriteTask` instances to avoid a pile of identical `Runnable`'s being allocated.


Removed extra duplicated methods (left behind in a refactoring), and moved beforeEach before afterEach.


Yes, it works well.  I and others have been using it on our machines.


What distinction do you make between `t_uint` and limb? Givent that `mpi.p` is an array of `t_uint`s, I think it's the same. However, I agree about the comment, it should probably not mention specific bit sizes.

Regarding the data-dependant branches, I don't know any single architecture that has constant-time integer divide, so this function is highly non-constant-time anyway. If you want to use this function on secret data, you should use blinding (that's what we do [here](https://github.com/ARMmbed/mbedtls/blob/caa8b4977c4af9b733b7ebce5c8ad8b5a5c48112/library/ecdsa.c#L142) for example).


Forgot to translate "commands" in "comandi"


=======================================
 FIND TA PARTITION
=======================================
Partition not found by name.
Do you want to perform an extensive search for the TA

=======================================
 INSPECTING PARTITIONS
=======================================
--- mmcblk0p1 ---
Searching for Operator Identifier...
+
Searching for Operator Name...
+
Searching for Rooting Status...
+
Searching for S1 Boot...
+
Searching for S1 Loader...
+
Searching for S1 Hardware Configuration...
+
=======================================
scripts/backup.sh: line 87: [: : unary operator expected
=======================================


These need to be semicolons. Typescript is blowing up now expecting ; instead of , I am assuming this is going to cause issue for a lot of people and anyone who used Yeoman to generate their gulp scripts.


Is this file replacing something or is it new functionality?


Yes, I think it would. Thanks for noticing.


shouldn't we add a comment in the XML to remember why we don't want to load system fonts?


Apologies for the long response on this, the only issues / changes I see is that since it's encoded now, I need to decode the request query param for it to display nicely in my input fields. For example:

Before change:
![capture](https://cloud.githubusercontent.com/assets/6421846/11979572/60c753d2-a962-11e5-9aa5-e8a021ce6164.PNG)

After change:
![capture2](https://cloud.githubusercontent.com/assets/6421846/11979573/685ea2a8-a962-11e5-8544-f6788e50b796.PNG)

Easy fix though, I just decode the input before passing it to the form builder:

```php
 {!! $formbuilder->text('q', rawurldecode($request->input('q', '')), ['placeholder' => 'Search', 'class' => 'form-control', 'role' => 'keyword']) !!}
```


I don't think it could, given the definition of `biL`. Do you have something specific in mind?


I think these API controllers all have a ``self.app`` - can you try to use that instead of ``trans.app``. It will ease the transition to managers and it makes it more clear that say ``repository_registry`` has nothing to do with the current user or web transaction.


Forgot to traslate "command"
It should be "Per accedere alle funzioni personalizzate di VK devi autorizzarmi tramite il comando /start\n"
Sorry for the trouble


See f70bd2d79fa49d06792f41a9a6a66cffc1ffcaf4.


Tests are failing as identities are separated and server is made using Express. We have to fix them accordingly


Vielleicht ist es sinnvoll den spieß umzudrehen und in der Website und den Parsern die "Core" Komponente als submodule zu integrieren?


I belive you should also mention that client.js and the bungle-[name].js should also be included in the host page.  Hot reload wasn't working for me until I added the client one.

For Example:
<script src="/static/bundle-client.js"></script>
<script src="/static/bundle-app.js"></script>

Thanks!

Eric


This is great . Thanks


Thanks @axilleas!


@tkelman According to me this is sufficient.

I have removed all other scale and scale! tests because for Quaternion all scale methods ultimately call the same method as called by the scale method in above test.
Any Inputs?


Nevermind, I get it now.


No probs


see eb76be29f48


I wasn't familiar enough with this code to know that a limb is always t_uint. Since it is always t_uint-sized, then that previous mask calculation shouldn't overflow, and code is fine.

It's just a minor thing that the comment says "32-bit" when it means "number of bits in a limb", and "64-bit" instead of "twice the number of bits in a limb".

Do we see any performance difference between using a data-dependent branch here and a mask? I'm also not familiar enough with this code to know how performance critical it is. Between either implementation, I wouldn't anticipate there existing a large difference in performance. However, for maintainability's sake, I would find a mask easier to read.


see eb76be2


There's an extra `-o` in the argument list


At the time, I was imagining an architecture where `biL` != number of bits in a `t_uint`. But, I now see that based on how `biL` is defined, such a situation is not possible.


@justusc comments welcome ... prototyping mechanism for injecting user code into mixed-type expressions. Existing nonintrusive interfaces are not enough (e.g. it's not even possible to create non-intrusive conversions/casts). The solution seems to be to introduce classes containing ops for unary and binary expressions for user type combos (e.g. if to perform A = B + A user will have to provide BinaryBraits<B,A,A> that contains add() and add_to()).

Currently I only have an example working for copying sparse tile-based array to dense array, see tests/expressions_mixed.cpp and tests/sparse_tile.h .

P.S. Braits is a temp name for these "trait"-like classes. They are not policies or traits.


We can't call `Pkg.add` here since that requires network access, tests that require network access need to be segregated and specific to Pkg.


Merge remote-tracking branch 'refs/remotes/cdnjs/master'


Well, I don't like the remaining `Exception` try/except blocks, but i feel like this is nitpicking at this point. I'll merge this.


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


There is no "hack" folder in release artifact so kube-up.sh will not work.


1.8.10 seems to handle it properly, at least checked `@c` before. I'll try to provide those correctly, but emacs line wrap will re-wrap everything and is easy to forget to check those.


Because my bash skills are very limited. Do you want me to correct it? It'll take some time though.


:tada: :smile:


All those changes should be in the previous commit?


It appears this was updated, but main.py was not changed. Main.py is still calling for ChartMogul and others but they no longer exist.


Pressurestat (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1985 of 1986 tests passed)

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


Понял, спасибо!


why is this `return` required? its the last non-catch-block line in this method?


It's actually more that we don't really need to for standard PDF export. That's why I did not removed it if anyone really need to load them for some reason.


@QQfoolsorellina
https://github.com/dastgir/Scripts/commit/dd0d86cb113e23c1508f09f1be51483f452f8237
I made it all configurable


nice nice ^^ i guess i can rebase my branch then


I'm sorry, I've done the string splitting text in another from scratch bash file with a hardcoded `string` :worried:


No worries. Anyone on Ninja would be glad to help.

On Tue, Dec 22, 2015 at 3:41 PM, Leah White <notifications@github.com>
wrote:

> Thanks for the resources- quite honestly I don't entirely know what I am
> doing - so sorry if that was a little sloppy. I'll check out these
> resources and ask any questions I have for the future.
>
> *Thanks,*
>
> *Leah White **|** Product Marketing*
>
>
>
> On Tue, Dec 22, 2015 at 3:25 PM, Lance <notifications@github.com> wrote:
>
> > Milk?
> >
> > It's a little weird embedding the html markup in the markdown. You can
> > create a list by beginning list items with a -.
> >
> > - A markdown guide is here
> > <https://help.github.com/articles/github-flavored-markdown/>
> > - A nice Mac OS X markdown editor is here
> > <http://macdown.uranusjr.com/>
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/leahwhite/spotx-demo-tvos/commit/08ce82c468fddd0e935c75cc5cee1db31640360c#commitcomment-15129387
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/leahwhite/spotx-demo-tvos/commit/08ce82c468fddd0e935c75cc5cee1db31640360c#commitcomment-15129753>
> .
>


Hah. Seems like npm wanted to sort the dependecies in alphabetical order. What an ugly commit :( Sry guys


thx. I guess a more robost way to checking for ipv6 would be to search for a containing `:`, wdyt?


It's robust, see `$ip = @\inet_pton($line[1]);`.


Yet another case:
```go
type a b
type b c
type c uint32
```
I suppose we need to use a base type here.


@steveklabnik Do you remember why you made these changes? At this point, Given the [history of AMS naming with respect to Rails](https://github.com/rails-api/active_model_serializers/pull/1388), it would seem `ActiveModelSerializers` to be a safer namespace than `ActiveModel::Serializer`, especially since in 0.10 the adapter and serializer work together to render a model serialization.

```diff
-require 'active_model_serializers/version'
 +require 'active_model/serializer/version'

  Gem::Specification.new do |spec|
    spec.name          = "active_model_serializers"
 -  spec.version       = ActiveModelSerializers::VERSION
 +  spec.version       = ActiveModel::Serializer::VERSION
```

and in https://github.com/rails-api/active_model_serializers/commit/4a2d9853ba7486acc1747752982aa5650e7fd6e9

```diff
-# ActiveModelSerializers
+# ActiveModel::Serializers
```


There's not a **huge** rush to get this into production as the previous rule almost guaranteed the latest security version.


OK. I just figure out how to solve this problem. In 0.1.5, you won't `react-native bundle` and it will work. I will remove these in README.


Missing protocol extension for proto "vpnc" /usr/lib/lua/luci/model/cbi/admin_network/proto_vpnc.lua:60: ')' expected (to close '(' at line 59) near 'natt_mode'


did some testing with the above samples, and as you said it works. thx for clarifying


I wonder if we shouldn't change this to `python setup.py build sdist`--it's not too uncommon that files will be excluded from the source distribution if they are files that are *generated* during the build command.  This goes back to the need for astropy/astropy-helpers#53


Что бы поддерживать концепцию «ни дня без коммита», предлагаю каждый день обновлять лиценс


/usr/lib/lua/luci/dispatcher.lua:452: Failed to execute arcombine dispatcher target for entry '/admin/network/network/vpn'.
The called action terminated with an exception:
/usr/lib/lua/luci/cbi.lua:864: Cannot assign option to not existing tab "table: 0xb9b460"
stack traceback:
	[C]: in function 'assert'
	/usr/lib/lua/luci/dispatcher.lua:452: in function 'dispatch'
	/usr/lib/lua/luci/dispatcher.lua:141: in function </usr/lib/lua/luci/dispatcher.lua:140>


+2 :koala:


Ajuste no texto de descrição.


Could you provide a test for that specific issue ? I suppose it is related to src:cid in images ?


@trentxintong looks good to me!


“它代表了一种可能失败的计算。”

的原文为：

“It represents a computation that might not produce a result.”

应该翻译为：

“它代表了一种可能不会产生计算结果的计算过程。”


Most of these references are unnecessary and can be removed.


“就不会对余下的任何计算进行求值。”

此处省略了主语，但补上更有利于读者理解：

“``(>>=)`` 和 ``(>>)`` 就不会对余下的任何计算进行求值”。


minor typo... and no field*s*


Hmmm....I thought I had fixed that.  Will check as I get time over the
holidays.

Regards,

Daniel

On 23/12/15 11:04 AM, qiuzi wrote:
> Missing protocol extension for proto "vpnc"
> /usr/lib/lua/luci/model/cbi/admin_network/proto_vpnc.lua:60: ')'
> expected (to close '(' at line 59) near 'natt_mode'
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openwrt/luci/commit/3f95160cbd3b82b8ba32a38347e0d3dcc368e2e9#commitcomment-15141599>.
>


Incluída informação da equipe


what if the pending_step is a purchase step?


@oney that would be great!
one more thing if you could help me with,  you have said "The app is killed/closed
GcmAndroid.launchNotification is your GCM data. You can create notification ... "
What I assumed from this was that this would allow me to find out which notification was tapped if the app is closed, but that does not seem to be the case.


@mabuaisha I know this is not a python file, but let's stick to having empty line at the end of each file.


J'aime pas trop le nom, vu que le concept des autres Short* était de ne pas inclure toutes les informations, seulement un subset significatif ; après c'est pas forcément significatif (ici, ça serait retirer par exemple created et last_modified)


Could you cut a new release when you find the time? Trying to get rid of all the warnings when compiling against 1.2.


Stef, I first corrected the logo link address ;) - but after inserting the logo the map download doesn't work. Maybe a workaround would be to use the logo as SVG instead(?)


“之前计算的即过” 这里的 “即过” 似乎是 “结果” 的笔误？


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


l forgot to tell you that minetest.get_node_or_nil may also be executed by the effector or conductor function, l fixed it.


If this is working then why don't you submit a pull request? :+1:


Ici les lignes 12 et 13 peuvent être supprimées, quoi que je serait probablement plus en faveur de supprimer 10 et 11 et mettre un TransactionSerializer par défaut.

Par ailleurs, tu as vérifié que ça se passait bien avec destroy ?


FYI: If your implementation will be user friendly and smart, then I'll be happy to create PR and review your code. Your current changes are hardcoded - it's good for POC, but not for production. Keep going :-)


Credits go to scratch developers.


What happens if
```
$id==10
// and
count($this->inc) == 5 ?
```


I'm feeling the opposite effect intended with this change :-(

Looks like the reloading as you scroll not only happens the first time you load TGD, but also after the auto refresh happens, making the whole thing feel slower since there are multiple loading delays during your interaction with TGD.


Set LC_ALL=C or LANG="" instead when building (on Arch Linux based systems) e.g.
LANG="" brunch u8833


第一句的原文是“ Its first parameter is the value to return if the result is Nothing.”，这里译文的意思有误。


Hey @p3drosola, I've already done this. Checkout branch feature/async-http-gradle-dependency.


I'm adding support for this in lodash at the moment and went hunting to see if there's an established way to detect. I thought about using the `DataView` too but I think using a single view may create false positives because of how it "views" the data in the buffer. Can you think of a way to create two array buffers with different contents that look the same when viewed as uint8.


Is this really a warning or something left from debugging? :)


Anything is possible, but I don't think this would be a good idea.  A vardecl without {}'s is a stored property.  The problem there is that you can't have a stored property in a protocol.  A subscript always has to have {}'s since there is no such thing as a stored subscript.  As such, lack of {}s really is a parser issue.

We could introduce the notion of a stored subscript for the purposes of this diagnostic, but it would probably produce crashers throughout the compiler by everything surprised to see a subscript with no accessors.


“先放下API设置的问题” 这里的 “设置” 似乎是 “设计” 的笔误。


I'll do that.

+++ Nick Wellnhofer [Dec 23 15 03:06 ]:
>   We should also add a version number to the namespace. Something like
>   http://commonmark.org/xml/1.0.
>
>   —
>   Reply to this email directly or [1]view it on GitHub.
>
>References
>
>   1. https://github.com/jgm/cmark/commit/4296462bfe31c877f4826f5734813c88c7240770#commitcomment-15137077


“我们来处理一下我们之前用 ``case`` 写出来的相当难看的代码。”

这里可以改为：

“让我们先来处理一下之前用 ``case`` 编写的丑陋代码”。


@mathslinux we should use encode_json() to do the formatter encoding wherever it's possible. See examples at rgw_json_enc.cc for examples.


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


use ldout, if missing CephContext, pass it on as a param


**Code Quality: Nitpick**

Always ensure to use camel case for method names in Javascript i.e `showSublimalAlert`


I'll ignore it, but we aren't waiting on the _rector_; he's busy at the moment.

We were going to challenge you on the TODOs w/o a story reference. :-)

I would also like to see `.zip(streamOfA, streamOfB, (a,b)-> new Tuple2<A,B>(a,b))` (and generalizations) builtin to Stream ops (e.g. `.zipTuple(streamOfA, streamOfB)`? It would make our code much simpler to read/write.


**Code Quality: Nitpick**

Leave a space between the parenthesis and the opening curly brace of the function.


@scottgonzalez - Line comments should be preceded by a line of whitespace correct? Even at the beginning of a method?


don't use g_ceph_context, use store->ctx()


Done.

+++ Nick Wellnhofer [Dec 23 15 03:03 ]:
>   Please add parens around the expression 1 << 1 and the ones below.
>
>   —
>   Reply to this email directly or [1]view it on GitHub.
>
>References
>
>   1. https://github.com/jgm/cmark/commit/56cd911ac2339a8ef80543780325a31e783f8c03#commitcomment-15137049


Yay, cool! Looks like this issue was in the code for a long time...


I'm pretty sure (double check) that when you're reading from the flash, `flash.now` and `flash` are equivalent. I would bed that the if statement on line 27 never runs if the flash alert is set.


There is a typo: test_praam should probably be test_param


@mathslinux now I see where this came from. Any reason why this was moved? We need it here so that ceph_dencoder can link correctly.


> Are you plotting the green dots as before, right? A grid, evaluate, and keep those below a given threshold. If so, how big is the grid? Is it like the bounding box of the object or bigger? Could you let it be twice the object size to confirm whether f(x) = 0 tends to be closed or open in this example?

Sort of, the grid is fixed size but its not bounded on the object dimensions. However it should be big enough to enclose the whole object and more. Anyway, just to be sure, I'll reshape it to 2x object bounding box.
so yea, the idea of those points is to confirm if f(x) ~=0 is closed surface.


@mathslinux please keep this one too for backward compatibility


... and also for json decoding


It's Ruby convention to only use `{ }` as block delimiters for single line blocks, and use `do end` for multi-line blocks.

```ruby
('a'..'z').map { |n| n.upcase }

('a'..'z').map do |n|
  n.upcase
end
```


@achurch I've gone ahead and refactored the platform stuff such that the backends will now implement a virtual interface and can be switched by config rather than at compile time. Compiling them is optional using defines DECAF_GLFW and DECAF_SDL, I tried making this a CMake option and forwarding the defines in but I'm not sure how successful I was, could you verify building with cmake and SDL still works for you with my changes? I seem to be having problems with my Linux VM at the moment.


per client instance right?


when submitting the settings form without changes i get the following error: `Enter a list of values.`


this can break cross versions sync operations, not sure we want to have this cosmetic change


Sorry man, I thought a < 2kB library (which could potentially be useful elsewhere) won't be such a problem ;-).
But all right, I'll add it to geokey-sapelli itself and we can get rid of it in the next geokey release then.


x"D thats just a CM vendor for you


Das kam von uns?


TeamCity Civic Assembly :: Build backend + frontend [Build 130](http://54.229.176.143/tc/viewLog.html?buildId=335&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Could you cut a new release when you find the time? Trying to get rid of all the warnings when compiling against 1.2.


You can make a pull request if you want. otherwise i will do it later today.


TeamCity YaccConstructor :: YC.Core General Build [Build 521](http://teamcity.codebetter.com/viewLog.html?buildId=204932&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Ah, ok I'll revert it. But in that case, that branch should be specified in the .gitmodules file of the android repo no?


Hm?


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


How does `InvalidCapability` sound?


why do you need @ ref here?


実はもう1カ所あるのだ！！！


I think maybe comparing it against 2 views or a Float64 may do it.


done!


TeamCity YaccConstructor :: YC.Core General Build [Build 521](http://teamcity.codebetter.com/viewLog.html?buildId=204932&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: Error during Nuget update. ..\tools\Build.Tools\NuGet\NuGet.exe install "Z:\TeamCityAgent\work\4138ce7c6332a78c\facio\.nuget\packages.config" -OutputDirectory "..\facio\packages" -ConfigFile "..\src\NuGet.config"   at FSI_0002.Packa... Build time: 00:01:15


Access to `rolePrefix` (in super class `DefaultLdapAuthoritiesPopulator.java`) now exceeds its access rights, for example here:

https://github.com/grails-plugins/grails-spring-security-ldap/blob/2.x/src/groovy/grails/plugin/springsecurity/ldap/userdetails/GrailsLdapAuthoritiesPopulator.groovy#L83

I guess it's not so important as Groovy doesn't respect the `private` modifier, but it still seems wrong to access a private field.


It's gone: 68eb0e8


One minor typo. Otherwise looks good.


This looks like we are refreshing on every request. Should probably refresh only on token expiration?


I will add a results handler to the server then. Are you going to push this to nsls-ii org or should we continue work in synchbot?


TeamCity YaccConstructor :: YC.Core General Build [Build 522](http://teamcity.codebetter.com/viewLog.html?buildId=204935&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Missing the test for this.


TeamCity YaccConstructor :: YC.Core General Build [Build 522](http://teamcity.codebetter.com/viewLog.html?buildId=204935&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: Error during Nuget update. ..\tools\Build.Tools\NuGet\NuGet.exe install "Z:\TeamCityAgent\work\4138ce7c6332a78c\facio\.nuget\packages.config" -OutputDirectory "..\facio\packages" -ConfigFile "..\src\NuGet.config"   at FSI_0002.Packa... Build time: 00:01:10


Ah nvm I think I read this wrong .... Laravel appears to be automatically sending a refreshed token


why is this test removed?


done: 0.5.6 is now on hex


Indeed these values deserve a better explanation.

I can't see any value in having something like that SOL_INITIALIZING /**< initializing */

You aren't adding any information, so you can leave it without documentation (it wouldn't be a problem for our doxy-coverage).

Anyway, for this enum specifically documentation is required and better explanation of each possible value.


it works fine tks ara:
http://vdlmap.com/parco/pai/indexstampa2.html
valerio


TeamCity Civic Assembly :: Build backend + frontend [Build 130](http://54.229.176.143/tc/viewLog.html?buildId=335&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 67 (66 new), passed: 68, ignored: 4 Build time: 00:08:25
### Failed tests
```
Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

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

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

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

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

Blogs tests: Should not create new blogs if cancelled: <no details avaliable>

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

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

Blogs tests: Should not delete blog if cancel pressed: <no details avaliable>

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

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

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


Pressurestat (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1986 of 1986 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)


Have you confirmed that a refreshed token is generated, by setting the timeout low? Also what will the UI do if the refresh token fails laravel side


Why a `long` instead of an `int`?


Thanks!


jajaja, muy bueno el "massive refactor"


test review comment


More information can be added for these types.
For example, you can explain why it's valuable (because it's composed not only by the value itself, but also for range, so it can be easily converted, etc). When should it be used instead of a double? How is it related to sol_drange_spec?

Try to think like if you don't know about it, you're a regular C coder with no knowledge about Soletta.

It worth to mention macros that can be used to initialize these types, functions used to convert double + range to a sol_drange. All these things can be useful for a user of our API.