@thebigsmileXD OMFG..... its 16...learn to read

 http://minecraftpocketedition.wikia.com/wiki/Bucket


node == null || node.next == null 로 바꾸고


return compareWithNextNode(first.next, node.next.next); 바꾸고


geography (macumber) - i386-Windows-7-VisualStudio-12: OK (2348 of 2385 tests passed, 0 test warnings)

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


Well I think we should follow the JSDoc here and not try to see if they got it right or wrong but I won't block this PR just for that.


See @internal note in the function documentation above it (which I will adjust for the new style expected for @internal).


@oeoeaio if you can bring this commit across that would be good (just two lines to add a failure msg to the reset function):
https://github.com/openfoodfoundation/openfoodnetwork/commit/b71454e7c74ec0a847044897c09b6881372aa20e

@mkllnk Thanks for the info. I'll just stick to uk/* branches so I don't step on your toes :)


Only when compared with a variable


rollInterval added


hmm, "unread_conter"?


Oops :)


and i didnt remove "$this->setNameTag($tag); m8, you should be apologising...


It did cause it to crash. Fixed in 7b0c2abf8a13ca7c9c67b6e3f3d19928cec0a70a.


I forgot comment "rollInterval added"


저러면 0, 1도 다 지워도 되겠다
````
  public boolean hasCycle(ListNode head) {
    if (head == null)
      return false;

    return compareWithNextNode(head, head.next);
  }

  private boolean compareWithNextNode(ListNode first, ListNode node) {
    if (node == null || node.next == null)
      return false;

    if (first == node.next) {
      return true;
    } else {
      return compareWithNextNode(first.next, node.next.next);
    }
  }
````
최종


Annoying. I think this might cause search filtering for egg groups to crash, too...


No worries, I'd use bitshifts I guess


ok


I see it now :) Ignore me.


Why are you adding documentation for Gtk.ToggleToolButton here?  This class ToggleToolButton is *not* a Gtk.ToggleToolButton, though it does itself depend on it.


AFAIR this useless code was needed in order for scripted skills to work in the modified areas.


I think this class could `extends AnyVal`, causing the implicit to not require allocation, see http://docs.scala-lang.org/overviews/core/value-classes.html


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 938](https://teamcity.labkey.org/viewLog.html?buildId=296521&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 244 Build time: 00:33:47
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.DataFinderTest: <no details avaliable>

```


const makeHook = () => cb => { } ? ;)


This description is not correct. It is the number of columns that code will be indented. Depending on the `useTabs` setting, it will be filled with either tabs and/or spaces.


imo should be `cb(error)` in that case and we should process.exit once, in async.waterfall callback.


Have you tested this?  None of the code that uses this class does so without giving an icon_name.


Good, the code is restored.


was prolly frunk again


Much more simple now :smile_cat:


By `gcc_version` I was referring to the

```
         if gccVersion:
              environ['CC'] = 'g++-' + gccVersion
```

block, not the environment variable. But see Michael's comments first.


Yeah, I think you're right, `process.exit` is a wrong choice in this case.


Above vendor entry points of `react` and `react-dom` need to be removed.


lol


`ServiceWorkerState`?


well, as you wish )


К сожалению, на `master.apis.dev.osm.org` не подключены facebook/google.


Can we have an early return here?
```js
if (!dependency.config.hooks || !dependency.config.hooks[name]) {
  return cb();
}
```


This is already revised in the original repo :)


I have deleted this branch since it's obsolete


TYPO ! REGENERATE THE POT FILES !
![a1zes](https://cloud.githubusercontent.com/assets/564822/11565077/a7732ddc-99dc-11e5-895a-75db2112ec16.gif)


This is also fixed in the original repo :)


https://github.com/open62541/open62541/blob/master/src/server/ua_server_binary.c#L24

We set the localconf->recvbuffer and sendbuffer to the maximum that can be received on the other end.
For maxchunk, there is no "inverse". So we need to look into remoteconf to see what the other end can receive.


Это брейкпоинт! :)


src/gitsync/src/core/Классы/МенеджерСинхронизации.os

Ну раз пришлось правки внести, неплохо бы изменения сразу пулреквестом прислать :)


То, что некоторые числа изменились на полпикселя — это баг, или ты специально делал?


ok


@olivier3400 please open an issue instead. Commit comments are not a good way to report issues as such comments get forgotten as soon as we mark the notification we received as read.


原來如此，我懂了... 先前整個沒看懂 code >_<|||||b
謝謝 @mrbigmouth 老師 >o<


Huh- I've not come across this usage of `js.native` being assigned to a default parameter before, what is the intention here?


`img_tts` пропал, потому что больше не используется?


Leave a comment


Нет перевода строки в конце.


ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 답변도 가능하지
막 추궁할거임 ㅋㅋㅋㅋㅋㅋ


Can we move `link` to separate function, so it's `async.waterfall([prelink, link, postlink])` ?


I've chosen the prefix to prevent any possible name clashes with other tools, just as a precaution. I would choose a name that contains upnp in its name, because it is much more common than tr64.

Ideally, I would like to have one command-line tool that can be used like `upnp discover` or `upnp device-info hostname`. But that should be done in a latter stage.


Any two interfaces would do, but I only have the one Leaf v2 that I got when I left the bunker, so testing has been a challenge... ;) And of course, CAN FD is completely untested.


@aumayr is there a way to have this done automatically through some pre-request handler?


but on the other hand, for the `postlink` you can't catch an error


animaged-adventure


Haha :)


Hello...can you provide the file of 'handwrite'? When I use this algorithm, the value of dictionary is NaN....Thank you!


Cool! Now to add the exact same thing to Storm Drain


Luckily you are keen-eyed. This happens when changing names at the last minute. Thanks for fixing it.

Maybe we can add `jshint` task to `grunt test`, to prevent this in future. Currently I have only eslint enabeld in my IDE for instant linting.


Very good IMHO. Thanks for the continued effort on this!


ok


> maybe windows port?

indeed, mingw nor msvc are C99 compliant when it come to '''z'''


Why?


More a style tip, but I quite like:
```
async.waterfall([
   prelink,
   done => {

   },
   postlink
], next);
```
way of formatting.

Also, I think we should remove parens from arrows when there's only single argument to keep it consistent with the rest of the codebase.


Извиняюсь за такое огромное опоздание. Готовился к сессии.


@matcracker could you possibly port the old mob AI system over? XD


d


I'll kill you both if you keep arguing...


@luobuwa  很有道理啊，我表示醍醐灌顶啊


`def update(): Promise[Unit] = js.native`


Shouldn't we set send/recv buffer to the minimum of both (send/recv)? Same goes for the maxChunkCount.


It's not a 'possible' test, the main.pm makes it an 'actual' test run in all of openSUSE's console test runs, which is the vast majority of the scenarios..

Why are stagings so different from the product which they are staging?


Здесь не нужен `@end` в конце?


imo not required. I can imagine that there are some backends out there not providing an API for retrieving tuner properties (=> iptv.simple comes to mind). I would document here that implemenation is optional and PVR_ERROR_NOT_IMPLENTED musz be returned in case


Could you remove ->getActive() and upgrade Property::__construtor accept object instave of string, because in constructor is created object from string.


hey, thank you for reporting. The bug is corrected by this commit. Being a javascript fix, you must be sure that your javascript files are reloaded. If debug mode is off, should be done automatically by the asset manager. If debug mode is on, please, refresh your browser cache.


There aren't mobs AI D:
Il 13/gen/2016 10:00, "Keith The Koder" <notifications@github.com> ha
scritto:

> @matcracker <https://github.com/matcracker> could you possibly port the
> old mob AI system over? XD
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ImagicalCorp/ImagicalMine/commit/c5bf4b555aca1262597adc3b980ce6f01592099f#commitcomment-15418112>
> .
>


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 65](https://teamcity.labkey.org/viewLog.html?buildId=296514&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 34 Build time: 01:04:44
### Failed tests
```
org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <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>

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


if we're heading for cherry-picking ths for https://github.com/xbmc/xbmc/pull/8736 there's no need for the bump in this commit


Can technically also return `undefined` here.


Stable version, every week. Development version, every month.


`getRegistrations`?


There's no need for the `str()` conversions so this could be written as:
```python
logger.debug("%s != %s", self._cache_load_timestamp.get(this_id, 0), os.stat(fn).st_ctime)
```


Is there no DVBS2, DVBT2, ...?


写个轮子啥的....


I have 1024! Just kidding... 32 should be fine.


// Save body styles. (Note 1)
 // Note: Don't turn this on unless you know what your doing.
 // Sprites are not released officially.
 save_body_style: yes <-- change to yes

in client.conf


My fix works exactly the same way for Lightning Rod and Storm Drain. Having the same test in two places looks like you're just slowing down the tests for no reason :|


It was a mistake, cause you don't have the images. Its better to work with the  Van gogh image, I'm adding images in a development branch, don't worry


Imo you need to define and document values for "n/a"/"unknown" etc. I can imagine that there are backends providing only part of the data needed to fill this struct.


Iteration over a dictionary defaults to iterating over the keys so this could be just `for this_id in objs:`


I feel like this sentence missed the word *only*. "Currently we are *only* supporting..."


It seems that testsuite/tests/InternetDevices.rb tests are testing the behaviour of NetworkInterfaces.Fastest, so i was thinking on test it in https://github.com/yast/yast-yast2/blob/master/library/network/test/network_interfaces_helpers_test.rb but if this method is not needed anymore don't have sense.

What do you think?


-1 from my side, we are still comparing strings.
```
>>> "010" > "1"
False
```


When is this ever triggered? Why is there an option to enable self signed SSL certificates?


Ok, never mind. I like plural for collections, but ok.


Yes, from the old ImagicalMine to the NEW ImagicalMine. Please?


Already asked and answered / see https://github.com/TGMPA/TGM-Plugin-Activation/pull/519#discussion_r49274081


Voici a peu près ce que l'on peut utiliser dans updfam.txt.

recherche de | [" dans geneweb/src/updateFam.ml
116: | ["child" :: sl] ->
128: | ["cnt"] -> eval_int_env "cnt" env
129: | ["comment"] -> str_val (quote_escaped fam.comment)
130: | ["digest"] -> eval_string_env "digest" env
131: | ["divorce"] ->
139: | ["divorce"; s] ->
146: | ["father" :: sl] -> eval_key (father cpl) sl
147: | ["fsources"] -> str_val (quote_escaped fam.fsources)
148: | ["is_first"] ->
152: | ["is_last"] ->
156: | ["marriage"; s] -> eval_date_var (Adef.od_of_codate fam.marriage) s
157: | ["marriage_place"] -> str_val (quote_escaped fam.marriage_place)
158: | ["marriage_note"] -> str_val (quote_escaped fam.marriage_note)
159: | ["marriage_src"] -> str_val (quote_escaped fam.marriage_src)
160: | ["mrel"] -> str_val (eval_relation_kind fam.relation)
161: | ["nb_fevents"] -> str_val (string_of_int (List.length fam.fevents))
162: | ["origin_file"] -> str_val (quote_escaped fam.origin_file)
163: | ["parent" :: sl] ->
176: | ["wcnt"] -> eval_int_env "wcnt" env
177: | ["witness" :: sl] ->
190: | ["has_fevents"] -> bool_val (fam.fevents <> [])
191: | ["event" :: sl] ->
199: | ["event_date"; s] ->
211: | ["has_fwitness"] ->
221: | ["fwitness" :: sl] ->
245: | ["fwitness_kind"] ->
384: | ["e_place"] ->
388: | ["e_note"] ->
392: | ["e_src"] ->
412: | ["create"; s] -> str_val (eval_create create s)
413: | ["first_name"] -> str_val (quote_escaped fn)
414: | ["occ"] -> str_val (if oc = 0 then "" else string_of_int oc)
415: | ["surname"] -> str_val (quote_escaped sn)
560: | ["fevent"] -> print_foreach_fevent env fcd al fam.fevents s
561: | ["fwitness"] -> print_foreach_fwitness env fcd al fam.fevents s
562: | ["witness"] -> print_foreach_witness env fcd al fam.witnesses s
563: | ["parent"] -> print_foreach_parent env fcd al (parent_array cpl) s

Donc, on ne peut pas faire de %foreach;event; dans updfam.txt


This is just reverted due to lava and water cannot be stacked. -.-"


Values should be inbetween ``. This goes for the rest of this page too. :)


Actually not so...

![capture](https://cloud.githubusercontent.com/assets/3521959/12289873/1386b25a-b9d6-11e5-9825-602f4f01c834.PNG)


don't do this, never ever :)


Fio is use for Bank and token. Account is much universal.


adb 커맨드 실행을 기다립니다.
다른 방식으로 개선하였습니다.


@colder spl_object_hash is still much less efficient than the SplObjectStorage implementation, I don't think we should use it there. There are two obvious improvements we can do: a) Use the handle instead of the object pointer, as it is half as large and b) support integer hashes in the implementation, so we can use the handle directly.


수정하였습니다.


This is better.


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


@tamarazuk The variable assigned to the method is already filtered - see the use of woocommerce_variation_prices at the bottom of the method.


Re-design of interflora.com.ua using Bootstrap 3.3.6 framework. It is supposed to create a web application for mobile devices so there is no need for adaptive markup for -xs and -sm resolutions. But I used it anyway :)


> The Elements enum was used for two different purposes. For node masks, and to OR an icon ID that is used in pre-existing icon files. As for these icon IDs, changing an enumeration value caused the editor to break, which was not obvious from the enum itself. As for node masks, we had to shift the bits by 1 to prevent a collision with the internal 0x1 node mask used within the nif loader. That is unnecessarily error prone. By using a separate enum we can define the enumeration values more freely, so now the bit shifting is no longer needed.

Sounds like overkill to me. The bit shifting saved us from having a separate enum, which is redundant in my opinion. I see your point about the 0x1 shift being error prone. But we could just as well had the shift as a base value in SceneToolToggle and then change the elements enum accordingly. We are free to do that since SceneToolToggle is the only constraint on the values of the enum we have. Using two enums adds flexibility that we don't need.


I must thinking about this but look like very well.


Okay, I see what you mean. However the implementation I am seeing here does not cut it. When we talk about particle systems we mean instance related particle systems, right? That should be explicitly stated in the enum, in case we ever add other kind of particle systems.
Also, the way Mask_ParticleSystem is or'ed into the viability mask looks problematic. The visibility should be bound to the visibility of instances instead. I'll comment about that on the other commit in a moment.


This property is on by default in Qt 5, yes.


싱글턴 패턴으로 구현을 바꿨습니다.


@ZombiePower OMG 1 for lava and water, LEARN TO THINK


Okay, forget everything I wrote above. Considering the other commit in question this flexibility is actually useful. I don't think we have a suitable location for the new enum. Maybe a new file in apps/opencs/view/render ?


I can rename to AccountsCollection, but Accounts didn't represented class purpose.


This is the comment I mentioned in the other commit:

I suggest to or Mask_Reference with Mask_ParticleSystem here.


TeamCity pull requests :: netty [Build 1020](https://garage.netty.io/teamcity/viewLog.html?buildId=4708&buildTypeId=pulls_netty&guest=1) is now running


The first time the display group is send to the wall is on a change (real user interaction, open contents, etc). Until then, the display group on the wall is not initialized(!), for instance it has a size of (-1,-1). Especially that size is necessary to be correct for computing the fullscreen size of the background content.

I'll comment it, and we can discuss this once you're back.


You mean that Property constructor should accept Account instance?


I don't know if we talked about this, but it was on my list of things to improve so :+1:


Wow I must have been tired when I missed that eariler


@mikejolley I did notice that filter, but I am having issues filtering the prices in Measurement Price Calculator because it bails early on this line. The issue is most likely that the *filtered* array is not stored in cache, though `var_dump`ing within the function hooked to the `woocommerce_variation_prices` filter seems to have no effect until I wrap this return in the same filter. Am i missing something?


TeamCity testProject :: build [Build 46](https://teamcity.midnighthub.com/viewLog.html?buildId=73&buildTypeId=TestProject_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:52


Where did `Client` come from here? I can only see `ServiceWorker | MessagePort` in https://slightlyoff.github.io/ServiceWorker/spec/service_worker_1/#serviceworkermessage-event-interface


yep - compiles. fixed


This class needs a constructor- `(DOMString type, optional ServiceWorkerMessageEventInit eventInitDict)`


No its Ok. I accept it.


改行！


inclue


Nothing is in stone yet, let me know how you like it, but I believe that renaming to `Account` and `AccountColletion` makes sense. I'd like to rename `Bank` too and move it to Utils, but I don't know how to name it yet.


this message


Before this change it was called always whenever a new content was created. Also for the background content, but this one has no window, so the action(s) won't be visible and usable at all. Furthermore, as usually contents are created from the event thread (also the thread where the QmlEngine lives which is vital for the actions), the creation of a movie background crashes as it originates from the wrong thread (main thread in that case, see restoreBackground()).


What are these things with asserting (in)equality with empty strings?


Yes :)


ありがとうございます！
問題ないことを確認しました。


This causes an error when the classes generated are not on the global namespace.
To prevent an error we should change that to <code>$this->class->addImplementation('\ArrayAccess');<code>

The above is also true for Iterator and countable


Kiem tra


I changed this now.


Ok this work

Greetings,
Rui Ferreira

On Wed, Jan 13, 2016 at 11:24 AM, Itumeleng Tease <notifications@github.com>
wrote:

> this message
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itumelengtease/First_AngularJS_App/commit/4c45b27a170fb0c60eef7a2dc7680ea3eb97e9f4#commitcomment-15418497>
> .
>


This happens once per translation unit when the client of the visitor provides the new source manager and preprocessor for the next translation unit.


Вынеси получение инструмента в отдельный метод.
Методы не должны быть сложными. Один запрос к БД - один метод .


Thanks @icxbb-xx , i know this.
Could you provide a script command to change bodystyle? Or i just using the "changelook" command to do this?


You're right, just to make it easier to understand & change / add new things.


:boom:


Top!
Goed getest? Heb ook zo veel mogelijk uitzonderingen proberen te testen idd.

Ben alleen nog niet tevreden dat we de complete listview opnieuw opbouwen bij een wijziging dus daar zal een van mijn volgende commits over gaan


тут зпт не нужно, нужно в след. предложении перед "соискатель"


this.context.router.push('/chat');


TeamCity testProject :: build [Build 47](https://teamcity.midnighthub.com/viewLog.html?buildId=75&buildTypeId=TestProject_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:48


download from http://pan.baidu.com/s/1eRev1dw


Just to make it easier to check which versions are installed - usually you care about more recent versions, that's why we decided to use desc. order.


ResultSet, PreparedStatement, Connection нужно закрывать в каждом методе где ты их открываешь.


@josepjaume NOW, I'm happy :smile:


if it throws, you leak the stream, you need a catch block


TeamCity testProject :: build [Build 48](https://teamcity.midnighthub.com/viewLog.html?buildId=77&buildTypeId=TestProject_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:44


Sorry @thebigsmileXD , I had a bad day, plus this :/ Buckets have a different ID to lavabuckets and water buckets. I tried adding them, but I couldn't make a .php file. Check the history, can you please copy that and make lava and water buckets?

Thanks and sorry for my behaviour


http://qd.haoduoshipin.com/p/read-data.html

Meteor.call 的说明


I don't get it, why don't we import assertj-guava ?


Thanks and done #136


@josepjaume Wow, cool idea!


Is there a reason this needs to be a list?


Lol on changing animation


I'm betraying my ignorance here, but what's the purpose of a pushd without a popd? I've always used them in pairs before now.


Почему Ability, может Factory ?


Bucket has an ID of 326, Lava_Bucket has an ID of 327 and Water_Bucket has an ID of 328


FYI, I have a ticket to expose SSL Labs simulation data via JSON. In the data you'll have each client's raw ClientHello, allowing you for a more reliable simulation. https://github.com/ssllabs/ssllabs-scan/issues/295


@swatijadhav I think it's better to set this in a controllers/concerns rather than in the Package model. Perhaps also store the value in the GoodCity module. So GoodCity.request_from_stockit = true|false


Indentation of this line should not be changed (2 blank spaces are required). Ditto for the other changed line below: please review your commits and cancel the changes that are only indentation-related.


I am not really sure what happened then. Apparently a pull request for a
revision sometime last year was sent, but the 0.3.0 tag was correctly set
to point to a recent revision. I'll try again.

On Mon, Jan 11, 2016 at 10:46 PM, Miles Lubin <notifications@github.com>
wrote:

> @ulfworsoe <https://github.com/ulfworsoe>, I don't see any open pull
> request to METADATA.jl
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaOpt/Mosek.jl/commit/6d4daa8514bbeb6eeb96fc340200c1904ec8d32e#commitcomment-15382056>
> .
>

--
Ulf Worsøe
Mosek ApS

ulf.worsoe@mosek.com
www.mosek.com


нужно вынести в отдельный файл .properties


why do you need the guard here? Is there a bug in the DSL?


There's also a useless blank space before ``><span``.


Incorrect indentation change here as well.


@CairoLee
yes, you can use 2 script command

*setlook <look type>,<look value>{,<char_id>};
*changelook <look type>,<look value>{,<char_id>};

 LOOK_BASE - Base sprite
 LOOK_HAIR - Hairstyle
 LOOK_WEAPON - Weapon
 LOOK_HEAD_BOTTOM - Head bottom
 LOOK_HEAD_TOP - Head top
 LOOK_HEAD_MID - Head mid
 LOOK_HAIR_COLOR - Hair color
 LOOK_CLOTHES_COLOR - Clothes color
 LOOK_SHIELD - Shield
 LOOK_SHOES - Shoes
 LOOK_BODY2 - bodystyle


Hey, @irenemaja Ich habe dieses alte Commit gefunden. Ich denke, es war nur ein Test, gell? Oder wolltest du es wirklich commiten?


so would number 8 for border-radius be bad? Anyways I changed it to 10px. the lines r indented too. tx for the comment :+1:


Typo in commit message "layot" => "layout"


Sure @steveyken


Prompt user if the files are being overwritten on Windows


Doesn't this change mess up the whole linker ordering thing you said we needed to have? @MovingtoMars


add file


They are not that different: it's all upgrades that fail: just see the current status of TW Snapshot 20160111 - never been that bloody red


nope, it's didn't work on my machine, i also tried turning off the debug mode and also after clear the cache and clear cache when the debug mode is on and it still didn't work properly

https://www.dropbox.com/s/s5m6ikcxopo2m3r/note-2.png?dl=0
https://www.dropbox.com/s/qfqdcpm43jrptkn/note-1.png?dl=0


@piedshag I changed this so that the two cores are required to use different instances of the extension object. If they are both referencing the same extension object, then they can "cheat" the unit test by sending messages using that extension object. In real life, the two cores would probably be running on different machines.


Works nicely!!
Thanks you very much.


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


I've noticed you like using `( arg )` as opposed to `(arg)`, which is not my preference, but if consistent then we should stick with that.  You've not used it here though


I think the level of indentation inside the function got lost.


When adding a new term, add it in .pot file please


@peteward we've a little discussion with @michalmarcinkowski in the morning about ``0`` quantity... and we agreed that we don't see any benefits with this feature. Isn't setting ``0`` quantity equivalent of removing item? There is validation on ``OrderItem``/``CartItem`` form that does not allow to set quantity below ``0`` and even if it would, I think it can be quite confusing for user to **checkout** cart with 0-quantity item... Do you have any thoughts about that? Any benefits from ``0`` quantity? Glad to see your point ;)


Isn't it good to have `Articles.author_id` ? So users follow the convention ?


TeamCity testProject :: build [Build 49](https://teamcity.midnighthub.com/viewLog.html?buildId=78&buildTypeId=TestProject_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:02:49


No we didn't, but good to know :dancer:


Useless extra space after ``${self.pagetitle()} ({{nbItems}})``.
By the way using an angular expression is not needed here, you can directly inject the mako variable:
```
<h3>${self.pagetitle()} (${total})</h3>
```
In the future we might use an expression if the page content is dynamically updated (using search criterias etc.) but we'll do it when needed.


Because otherwise I get a class not found exception comming from assertj-guava when unit testing with java 6. Hence the choice to not rely on assertj-gava for these two operations


Any reason for committed commented code?


@piedshag I don't think my previous test made this clear, but I was hoping that the core would return an instance of the extension here. That way, the user can just send messages into the instance of the extension and they will go directly to the other core.


The `values.equal` function works only on valid Juttle values but both `row.stored[fld]` and `pt[fld]` can easily be `undefined` if given field doesn’t exist. Additional guard (e.g. using `_.has`) is needed before applying `values.equal`.


Чё за х)


Here we used an expression because it is required by angular-gettext when using content variables in the translated strings.


oh: and if the system was updated using zdup, the ay_profilecheck can't work neither... as there was no yast profile created.


Useless extra blank line here


Pretty sure the code works in the same way, just moved around.


Hi, thanks a lot! I'll give it a shot again. Appreciate the help :D


best commit 2k16, i r8 8/8


I changed it back to ?=


Is the recursive call really necessary here? Why not simply

```
resp <- request_fetch(req$output, req$url, handle)
```

? In particular, the recursive call effectively calls request_prepare(req) twice.


(7e364f1) The title of the commit doesn't respect guideline ("type(scope): title (refs #<number>[, ...])")


This would become an issue with "nieuwe" || nieuwe heeft gereageerd op.... etc. instead of nieuwe reactie op.


hmm...I didn't check the staging DVD yet, so staging DVD also supported yast profile?


euml is correct in all cases


You are missing `connectionStateTtl`, see http://docs.ably.io/client-lib-development-guide/features/#CD2f, however that is an unused field so not important.  The `serverId` is a `String` and should be added to the spec, I will do that now.


I’d consider naming this `fields`, which is the term we conventionally use for “point properties”.

(The problem is present in `this.columns` already, which should really be `this.fields` if we want to be consistent with our terminology. But this is out of scope of this PR and it affects more procs.)


Indentation is incorrect in the whole ``<section>`` block:
* embedded elements must be indented using 2 blank spaces.
* ditto for the embedded Mako variables (eg. ``${helpers.add_pagination_links('waypoints', waypoints, total, filter_params)}``)
* when on different lines, opening and closing tags should be vertically aligned (same identation)
* avoid useless blank lines unless they *really* help making the code readable


t is needed since gebruiker is in front of word ( stem + t )


Unneeded use of another slash, sentence says nothing extra, yet is longer


Are you sure this will always return `Client`? the spec says `Any`.  I wonder if it's `js.UndefOr` again.


Always use double quotes for HTML attributes.
Please remove useless extra spaces in the button class attribute and before opening the ``<span>`` element.


commentaar or opmerking has to be used throughout the entire translation


Want a space after the : here.


Hi @CandyFace
I noticed that you didn't block `mBrushSpin`'s signals before set value.

To avoid never ending notification loops, it's better to block signals if we only want to update UI value, and It could be done by simply adding a line `QSignalBlocker b2( mBrushSpin );` just like line 165.

I'll do it after merge this PR, just let you know. :)


Extra space at the beginning of the ``<a>`` element.


This fixes #4


Sorry, but I don't understand... Where is the old and where is the new?


Growl title would overlap with this length, extra info is stated in the actual growl


Cong hoa xa hoi chu nghia viet nam


Very well.


Both correct, choose shortes option imho


This fixes #3


невозможно слитно (в редких случаях можно раздельно и то не приветствуется)


и тут "невозможно"


Personally use the Email version on websites, using E-mail for everything printing related


@paulkernfeld ahhh i see what you mean. To do that each extension would have to have a `message` event and a `.push` method. I will see what i can do tomorrow. If you want to try anything you should try doing it [here](https://github.com/mafintosh/hypercore/blob/master/lib/protocol.js#L210) and [here](https://github.com/mafintosh/hypercore/blob/master/lib/protocol.js#L303)


This is the kind of cleanup I prefer to be on a separate commit when I am reviewing some commit that has actual changes.


Should we change `java_outer_classname` to `EventStorageProto`?


Cache is visible in the api, but cannot be instantiated.  I believe this wants to be `class Cache private () extends js.Object`


I am pretty committed to it!  I think the way we incorporated vlan support was a serious error.  I think we will be able to use hardware vlan tagging for our use case, so performance is not a short/medium-term concern.  In parallel we have other performance options like improving the packet data structure.  Finally, with the packet header helpers introduced in the last commits, the only function we would need to modify for a hypothetical return of the vlans would be get_ethernet_payload().


And yet the grammar is better, though the first "hem/haar" could be removed indeed, I would not remove the word "eens", as it makes the sentence flow better.


I would say so: https://openqa.opensuse.org/tests/113414/modules/installation_overview/steps/22
And for an upgrade, I would agree that autoyast doesn't really make sense

Except: even in TW where the needle exists now, none of the tests passes over the installer: https://openqa.opensuse.org/tests/113430/modules/start_install/steps/2


it is ok


Not afraid of toil around the core, card card, can you please re optimization?


OK, I'll try to take care in the future


Out of interest, if you are providing a `MessagePackMember` index here, what is this used for?  For MessagePack messages received, we cannot always guarantee all keys will exist or be in that order as far as I know, so I am hoping that these indexes are indicating that expectation?


`js.Array[Response]`, rather than `Array[Response]`


http://www.mijnwoordenboek.nl/vertaal/NL/EN/valide   <>  valide is better used since it converts better to english paypal terminology


Are you aware that binary search is available in the C++ standard library as [`std::lower_bound`](http://en.cppreference.com/w/cpp/algorithm/lower_bound) and [`std::upper_bound`](http://en.cppreference.com/w/cpp/algorithm/upper_bound)?


ok, if there's a clear separate interface for removing then fine :+1:


good catch !


Can agree with this


s velkým rest je to jen v novém is core


stejne


`request: RequestInfo`


BTW. It would be good to update the README soon with instructions on how to run the test suite now with these changes along with any helpful hints for developers who want to debug their work.


tbh though, I'm quite suffocating in work right now. I'll make an issue as a reminder.


the sum of 23 patters,including creational patters,structural patters and behavioral patters.
Creational insists of five patters.Followed Abstract Factory,Builder,Factory Method,Prototype and Singleton.
Structural patters insists of seven.Followed Adapter,Bridge,Composite,Decorator,Facade,Flyweight and Proxy.
Behavioral patters insists of eleven.Followed Chain of Responsibility,Command,Interpreter,Iterator,Mediator,Memento,Observer,State,Strategy,Template Method and Visitor.


Games just use DLC terminology here in Holland too, no need to translate literally. ex: http://www.gamekings.nl/tag/downloadable-content/


I assume you mean `2016` instead of `2015` here? ;)


fixed


renamed


Please be more specific :)

I do not think that the debugging information that is removed in this commit ("got a pkt") is useful in any production situation.  For that reason it doesn not belong in the lwaftr.  For development usually you need to add much more precise and ad-hoc printouts.  When I do this, I always make sure to remove them before I submit a PR :)


We launch products, we don't start them: https://www.ikgastarten.nl/marketing-en-verkoop/marketing-en-reclame/10-tips-om-jouw-product-succesvol-te-lanceren


remove the tailing space?


germanium should be only inside tests on as library requirement


Hey! Are you planning to merge this PR? It fixes problems which arise when using nodejs 4


We'll use one view for showing results. Let's name it `series/search_result`.


It returns a pointer, so I think it is correctly named.  WDYT?


Capital used as in resemblance with original file


Need to rename `series_list` -> `series`


Ok. Could you please use the bug tracker, so you can provide me the lss file your using ?


Александр, прикольно, что ты занялся переводом keepachangelog.com. Надо продвигать правильные идеи в массы.

Кстати, раз ты интересуешься данной темой, то может быть тебе будет полезен сервис про ченьджлоги, которые я пилю: https://allmychanges.com


Need to rename: `seriesList` -> `series`.


ok, I've checked staging DVD, yes it should be the same, so this is the right one https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/932 , make sure yast profile enabled only when nomral minimalx dvd test.


Then I would NEVER trust "mijn woordenboek" again, as the word "valide" and "invalide" is in Dutch really used only in in the context of physical handicaps only. I'm even quite sure people who have no knowledge of the English language at all would not understand the word "valide" in this context at all. To say it in C: Dutch != English, and the guys at mijnwoordenboek should know that more than anybody else. Shame on them!


A bit hacky but input change event is called if the user clicks a suggestion as well.


It is unused because of changes to the way the binding table works.


Ahh, right. I see now


P.S. Even better: `series_list` -> `searchResults`.


delegated to class DEREncodingConverter. Could also be turned into a Supplier (lambda)


you use responses only inside example application, this shoud not be here


about https://openqa.opensuse.org/tests/113430/modules/start_install/steps/2 , it just because installer need some time re-evaluating packages selection, press alt-i while re-evaluating was ignored.


Thanks for pointing out the obvious eacute error! Many of the things you found don't translate as intended with the use of Gamejolt UI. Per line comments added


It might be better to check if the ctime field is there or not. If ctime field is not there (e.g old metadata), fallback to use zookeeper znode's creation time (stat has the information)


h


This is not your fault, but I think this error message is quite wrong, it should simply be "No API key exists to authenticate with Ably"


@jvasileff Well unless `take()` works differently than I expected I would assume that take = 1, step = 2 would only return a single item.


remove tailing space?


See other comment.  I would agree with the comment if the return value were an integer, but since it's a pointer I disagree.


This one is shadowing `helm-resume` (https://github.com/syl20bnr/spacemacs/blob/develop/layers/+completion/spacemacs-helm/packages.el#L179)


Why did you remove `Timeout`?


extracted


Oh, sorry, I misunderstood your original comment. :bow:

> So it's only a matter of making the reason clearer.

OK, change that and go forward. :wink:


static import please


# AngularJS
Es un framework JavaScript frontend dedicado a construir single-page apps usando la arquitectura MVC.
La idea de este framework es extender la funcionalidad de HTML usando atributos especiales que bindean lógica de negocio JavaScript con elementos HTML. Permite mejorar la estructura del código de nuestra aplicación y testeabilidad usando MVC e **inyección de dependencias**.

### El objeto global *angular*
Este objeto contiene un conjunto de métodos que vamos a usar para crear y darle funcionamiento a nuestra aplicación. Es importante saber que este mismo contiene un elemento sencillo de jQuery llamado *jqLite*, el cual permite que angular manipule el DOM. También contiene métodos estáticos que vamos a usar para crear, manipular y editar las entidades de nuestra aplicación, incluyendo la creación y llamado de módulos.

### Módulos
Con este framework todo está encapsulado en módulos.
Toda aplicación AJS necesita de un módulo, y nos vamos a referir a este módulo como el *application module*. Estos son creados y llamados usando el método:

`angular.module(name, [requires], [configFn])`

el cual acepta tres argumentos:
- **name**: El nombre del módulo
- **requires**: Array de Strings para definir otros módulos y dependencias.
- **configFN**: Esta es una función que va a correr cuando el módulo esté siendo registrado.

Cuando usamos este método con un solo argumento, va a tratar de llamar un módulo existente con el nombre pasado como argumento, si no existe arroja un error. Si pasamos más de un argumento creará un módulo configurándolo con los argumentos pasados

### Two-way data binding


It looks like the issue is that your string of input notes has spaces in, whereas the string is only being split at ",".
This means that after the first note, the next ones are " C#3", " D3" with a space in front.
Hence why None is returned, as it can't find a value for " C#3" in the dictionary, which has a key for "C#3" with no space in front.


Opened an issue for this: https://bugzilla.mozilla.org/show_bug.cgi?id=1239506


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


The logic of sparse summa should be correct for sparse*dense as long as shape computation is worked out. Not sure how much code needs to be adjusted in summa itself.


OK, great. You got me scared there for a moment, that I somehow build 16 bytes larger binaries.
Problem is the buildscripts, they just move on hiding the problems in the logs. My makefiles returns pass/fail status so Jenkins can see it and flag it. The Travis scripts just moves on, don't know how Travis handles that. And I don't know anything about how your build scripts work....


ack


Whoops just noticed that these aren't the input notes... How are you inputting your notes when you enter them in response to the notesRequest? Are you including spaces?


what's the failure?


This is a test.


Change the name of the method to match the field it is returning?


Yes, glib 2.6.6 is the last version to support Windows 95/98/ME, but completely lacks support for static build on Windows. It looks like official support for static build was not added, until glib 2.17.1.

I would rather avoid bundling even more DLLs with ScummVM.


神，我要请你吃饭:)


if only this was an actual, working way to fix it with the latest build :( :-1:


develop (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: OK (2333 of 2384 tests passed, 0 test warnings)

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


My script also doesnt run those checks. It is pretty basic for now. I need to stop developing FC stuff and work a bit more on things around it :dancer:

16bytes is easy to remove. I could simply remove IBUS for CC3D target only perhaps or clean something else up. There are few variables that can be thrown away. I know where


The failure is that `ccall(:shm_open)` and `ccall(:shm_unlink)` is not able to find the symble since `libjulia.so` doesn't need `librt.so`.


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


missed an unlock()


Периодически может выскакивать
"DetachedInstanceError: Parent instance <Message at 0x10578b650> is not bound to a Session, and no contextual session is established; lazy load operation of attribute 'picture' cannot proceed"

Это завтра пофикшу, заодним подаутирую код, до эстетического вида


Thanks for this. I have now applied your changes in pbatard/rufus@2430c66a68e24fd00941b52638fb25c1578ef50e.


Sounds good. I'll make an update.

> On Jan 13, 2016, at 5:02 PM, Orcun Simsek <notifications@github.com> wrote:
>
> I think it would be better if we call 'closeClients' only in base ZookeeperRule's after method, and have an abstract 'closeServer' method for the classes that extends ZooKeeperRule.
>
> That will ensure that if we want another implementation of the ZooKeeperRule, we will not need to worry about the order of Client/Server shutdown.
>
> —
> Reply to this email directly or view it on GitHub.
>


I'm locking between Begin() and End()


No please. keep up the great FC work you do. Leave the surrounding support systems to ... well... someone else.


right


This is more what I was thinking, although I think each title shouldn't have the ' -' at the end, but rather the separator should go here. You could either put 'Home' as the title for the homepage, or have the '-' only show up if there's content from :title.


Let's stick with FluidSynth 1.0.9 on Windows for the time being. Someone on FluidSynth's development mailing list wanted to look into getting rid of the glib dependency. If that ever makes it into an release, we might just switch to that version in the future.


Дуб, я посмотрел. Писать тут все неудобно, я тебе завтра подойду и расскажу бока.

Самая важная ошибка: это затягивание очень частных зависимостей в общие файлы/компоненты. Это делает код не универсальным и сильно затрудняет переиспользование.

Подробности завтра, а то я сотру пальцы о телефон пока все напишу :(


I don't think this loop is necessary. You should be able to build a list directly from `PrivilegeType.values()`. I think that returns an array and you almost certainly make a List from an array.


need rebase


I fixed it in the last commit. removed some old variables used for old pid controllers

really need to do a cleanup. I bet there is some more left


Comment out.


@olefredrik what was the reason for the change here? The import paths are set in the gulpfile so `@import util/util` works fine.


Thanks for letting me know btw! Really appreciate your help!


sorry to do this to you, but to keep the previous behaviours intact, however wonky, looking at this we will need 2 methods.

1) def globHasSuccessFile(globPath: String, conf: Configuration): Boolean = globHasFiles(globPath,conf, successFilter=true, hiddenFilter=False)

def globHasFiles(globPath: String, conf: Configuration, successFilter: Boolen, hiddenFilter: Boolean)

and then call the globHasFiles(...,true,true) from the one below.

The only downside here is just with the addition of the hidden like this we have changed the behavior of globHasSuccess to be the same as our SuccessFileSource trait...which as logical as it sounds they should be the same, arent....


Humn... Right. @guludo we need to fix this in waf as well


Are you sure DATA is a file-handle?  I couldn't determine what it was from looking at this file.


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


It's writeable by the administrative user that has sudo privileges. Docker has root access so the container can write to it as well.


Jenkins has started chewing on the new stuff.


Oh yeah, did that because we're keeping the hostname in the session data, which is inside the client data, which gets memset'd to 0 on each new ClientConnect() and at this moment we need at least 2 of those calls to get the hostname so had to work around that.


Please comment on file changes, not commit :) .


Disable this


The algorithm is determined based on the output shape. So dense*sparse -> dense will give incorrect behavior. Taking into consideration the sparsity of the arguments can save a lot of time.

I am not saying the algorithm cannot be modified to do the right thing, only that it does not do it now.


Is there a way to get rid of wrapping div to avoid extra elements in the layout?


In this case, the server we were connecting to was in a different vCenter not managed by CloudBolt.

> On Jan 13, 2016, at 17:22, Bernard <notifications@github.com> wrote:
>
> One could also get all of the connection info for VMware from server.resource_handler, since it is already stored there.
>
> —
> Reply to this email directly or view it on GitHub.
>


Old strawman, use http://wiki.ecmascript.org/doku.php?id=strawman:weak_references

(The `register` method is necessary)


Can you give me that link from you where you keep the track of my repo?


Wrong treatment


Sure


"Or replace the ips to random ones." I can just remove them, move this to __Issues__


YAY! Now I can actually run my server properly


Since highcharts doesn't reflow upon print media query. Wrap your chart in `RedrawOnPrint` component.


@sgress454 here's that link over to you if you end up making the relevant conceptual page (and I figure we put `.load`, `.getBaseUrl` et al in this folder).  I'ma add the new method for reverse routing, but only in the v0.12 branch.  Merging changes from master up to 0.12 momentarily.


Hello!


Try this.


This is an extremely important commit.


Please get rid of a new line


Please get add a new line


Actually, we need the data from the stream even if the responseCode signifies an error. For example, some sites return an error description in the content, and sometimes we must process the description data also. Of course, the situation is very rare, but this is what the other clients do, so Android client should be similar


Hi Git!


Ok


Looks great.


ok commit;


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


Hello again!


Looks like a search and replace fail in multiple places.


Hehe


We could get name from both of them potentially, and they can in theory be passed to the resource printer.  We don't handle them in most places in the code yet because we haven't had the need.


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


The is_atomic can be skipped for 3.0 to 3.1 upgrades, but should be kept for 3.1 minor upgrades.


For bbc/microbit-extras#2665


What's up with these commented html lines?  Are you actually going to use them?


This is where I run a strait 1:1 clone of betaflight, using the makefiles: http://andwho.sytes.net:8080/job/AndWho%20Betaflight/
Here I run my buildsystem dev branch : http://andwho.sytes.net:8080/job/AndWho%20Betaflight%20buildsystem/
And here are your stuff, using travis scripts: http://andwho.sytes.net:8080/job/BorisB%20BetaFlight/
I am thining of re-organziing this a bit, I have to do too much manual sync between your repo and mine.
Not sure what it will look like.

There are also a Hydra Cleanflight job tracking cleanflight master, and an iNav job.

NB. This is on a private Ubuntu server at home, can not guarantee uptime nor accessability in any way.


All green balls now. Go 2.3.0 go.


I agree.


This should be squashed into the other commit which changes to `@definition`.


Not my work! Just porting it over, but its good to finally get some much needed systems into the core :)

Hey! Close those issues on your bug tracker :)


// FIXME: Move this to another class as there is a lot more we should do here


// FIXME: There has to be a way to do this without converting to an array and then back to a collection


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


Doh! Sorry! No, I didn't see that - last I looked it was up for grabs, so I thought I'd start chipping away at it. #failme Feel free to clobber completely, yours is probably better anyway!


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 200](https://teamcity.labkey.org/viewLog.html?buildId=296633&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:09


@priyatam I think this was a good idea but it's left the README looking *very* bare. Could you please bring a few of the examples that were on the README back and place it under the "Examples" section. This library has been around for awhile and I think it looks sort of embarrassing that it says "Coming soon" there when the there's always been something there.

In the future a change like this still needs to go through the pull request process. I know you're a maintainer and I don't mind if you patch small things here or there but for large changes like this please open a pull request. Thanks!


* TODO: rewrite this as it is doing way too much and is very messy now.


In PowerShell we need to pass-in x-ms-client-request-id globally to all calls for telemetry. Having the value generated in the method or only modifiable via method parameter makes it impossible to globally inject this behavior. Let us think of another way to make sure the old behavior is still there.


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


Makes sense. My bad for not submitting the PR, first. I'll work on porting the examples back to README soon.


Ah OK. So the only thing that would be an issue would be the `mkdir /mnt/ephemeral/hdfs`. If omitted that and changed the volume mount flag to `-v /mnt/ephemeral/:/hdfs/:rw`, we would be fine.

I think @hannes-ucsc wanted to avoid cluttering `/mnt/ephemeral` though. I think we could work around this by making a directory _under_ `/hdfs` from inside of the docker container. Thoughts?


What if the caller is interested in the `Closure`s return value? Can that flow-through?


@priyatam All good, man!


Class UpdateGitMirror is missing a docblock.


// FIXME: Delete tar file if it exists


exit 0 # FIXME: Not sure why I need this, otherwise it fails


@priyagupta, changed the code according to your suggestions, and created a new repository to host it.


Is this actually all you need to do to support the swift package manager? Create Package.swift?


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


what do you mean, appear to move towards and away from us? isn't the shift part of the appearance, not the cause?


climes --> waters?


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


I think we need to escape some invalid json characters, for example double quote, backslash, control chars(\n, \t).
https://jsonformatter.curiousconcept.com/ is a online json format validator, hope it's helpful to you.


can you unpack this sentence, it's a little long.
you = recording station, whale = ambulance, to go along with explanation from the previous graf...?
weather-induced --> seasonal?


Now it is dropped automatically, what does this mean for users like I?

Normally, I have <'window> that operates the events and renders. It has a drop function. So what should I do with the drop function?

impl<'window> Phi<'window> {
    fn new(events: Events, renderer: Renderer<'window>) -> Phi<'window> {
        ::sdl2_image::init(::sdl2_image::INIT_PNG);

        Phi {
            events: events,
            renderer: renderer,
        }
    }

    pub fn output_size(&self) -> (f64, f64) {
        let (w, h) = self.renderer.output_size().unwrap();
        (w as f64, h as f64)
    }
}

impl<'window> Drop for Phi<'window> {
    fn drop(&mut self) {
        ::sdl2_image::quit();
    }
}


Yes, that's correct. If @hannes-ucsc agrees, I think making the directory from within docker is a good idea.


Non-functional changes
in https://github.com/glinscott/fishtest/wiki/Creating-my-first-test

This should not be tested on fishbench


Are you giving your self credit there anywhere?

On 1/8/2016 12:33 PM, josephsl wrote:
> Hi, Take time (the code looks fine to me; in effect, you are extending
> my own work (#4584)).
>
>
>
> From: derekriemer [mailto:notifications@github.com]
> Sent: Friday, January 8, 2016 11:25 AM
> To: derekriemer/nvda <nvda@noreply.github.com>
> Cc: josephsl <joseph.lee22590@gmail.com>
> Subject: Re: [nvda] The case sensitivity of the last find is
> remembered when searching using NVDA's find dialog. For Example, if
> Food is searched for with case sensitivity on Food will be found when
> pressing NVDA+f3, but food will not. (33189af)
>
>
>
> OK do you mind waiting until tomorrow night until I do this or doing
> it yourself? I am going to be away from my computer for the night.
> Because I'll be in the mountains.
>
> Sent from my iPhone
> Please disregard errors as this is a smaller device.
>
> > On Jan 8, 2016, at 11:53, josephsl <notifications@github.com> wrote:
> >
> > Hi, They would go on the copyright line.
> >
> >
> >
> > From: derekriemer [mailto:notifications@github.com]
> > Sent: Friday, January 8, 2016 10:39 AM
> > To: derekriemer/nvda <nvda@noreply.github.com>
> > Cc: josephsl <joseph.lee22590@gmail.com>
> > Subject: Re: [nvda] The case sensitivity of the last find is
> remembered when searching using NVDA's find dialog. For Example, if
> Food is searched for with case sensitivity on Food will be found when
> pressing NVDA+f3, but food will not. (33189af)
> >
> >
> >
> > Where would the names go?
> >
> > On 1/8/2016 10:09 AM, josephsl wrote:
> > >
> > > If you want, you could add two names: mine and yours, as both of us
> > > have worked on this and this is a good time to get recognized for
> your
> > > work (just my thoughts).
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15344152>.
> > >
> >
> > --
> > ------------------------------------------------------------------------
> >
> >
> > Derek Riemer
> >
> > * Department of computer science, third year undergraduate student.
> > * Proud user of the NVDA screen reader.
> > * Open source enthusiast.
> > * Member of Bridge Cu
> > * Avid skiier.
> >
> > Websites:
> > Honors portfolio <http://derekriemer.drupalgardens.com>
> > Non-proffessional website.
> <http://derekriemer.pythonanywhere.com/personal>
> > Awesome little hand built weather app that rocks!
> > <http://derekriemer.pythonanywhere.com/weather>
> >
> > email me at derek.riemer@colorado.edu <mailto:derek.riemer@colorado.edu>
> > Phone: (303) 906-2194
> >
> >
> >
> >
> > —
> > Reply to this email directly or view it on GitHub
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15346027>
> .
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15346891>
> .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15347088>.
>

--
------------------------------------------------------------------------

    Derek Riemer

  * Department of computer science, third year undergraduate student.
  * Proud user of the NVDA screen reader.
  * Open source enthusiast.
  * Member of Bridge Cu
  * Avid skiier.

Websites:
Honors portfolio <http://derekriemer.com>
Awesome little hand built weather app!
<http://django.derekriemer.com/weather/>

email me at derek.riemer@colorado.edu <mailto:derek.riemer@colorado.edu>
Phone: (303) 906-2194


Nice change about browserHistory. Want to create a PR? Thanks.


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


missed changing 'native' to 'none' here. Look good otherwise.


are effect and shift synonymous throughout?
can you just say "the variation in whale song pitch was in line with the Doppler shift prediction"?
how is it both correlated but then also greater?


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


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


how does blubber affect pitch?


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


so it's all blubber, not Doppler at all?
remove "too"


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


Does the Doppler effect explain pitch variation in whale song?


Rebased
and @CalmCodeC rename to "D3D adapter"


I think if there is a 'parentPath' or 'parentId' data within 'Component' will be helpful to navigation the tree up, what do you think ?


Disable InsertionSort too.


You never come back to this; do they touch on or explain why the pitch changes over years? Is that to do with blubber too?


For me, this change caused the globe to disappear when the window is resized.
- DD0x0


Avoid unused local variables such as '$tokens'.


00-HTML-CSS-basics/01-HTML/01-The basics of html/expenses.html

I'd use a section tag here but anyway is OK


* TODO: rewrite this as it is doing way too much and is very messy now.


// FIXME: Move the archive to a seperate command


We must write all in english. :P


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


estimating accurately --> accurately estimating


00-HTML-CSS-basics/01-HTML/01-The basics of html/multimedia.html

iframes are not supported, please read the html5 documentation attached to the topic and try to change this, also try to use multiple sources for audio and video to apply browser compatibility, just the files format. You dont really need to upload the parsed files.


In the event stderr is written to, `service icinga2 status > /dev/null 2>&1` may make more sense. (I haven't played with logrotate output redirection much.)


I clearly don't know what I'm doing!!


calculate the extent... --> determine how well the whale-song-pitch variations agreed with what would be predicted by the Doppler shift


00-HTML-CSS-basics/01-HTML/01-The basics of html/signup.html

label doesn't seems write for me, maybe you must check into this website. You may need to set another attribute to the tags http://www.w3schools.com/tags/tag_option.asp


added another sentence, just looking at making changes and how that works.


scrubbed to just include meaningful changes via 248d0f1193938e1e017cf64811b3cbda4282645b


nit: We usually use "%q" to print string so as to avoid empty string. :)


Uncomment this!


I am quite confused.  Unless there have been recent changes, a $config hash passed in to the elasticsearch class appears to be ignored other than to be merged here into the elasticsearch::instance class's $config hash.  What is the point of this.  Why not simply require the user to pass the $config has into the elasticsearch::instance class, if that's the only place it is used?


greater than predicted, that is, the ambulance (whale) seemed to zooming by even faster, with a higher pitch, than you would expect from a simple Doppler shift explanation. The scientists then considered whether the pitch could be affected by environmental factors...


Compilation failed in Visual studio
>1> ------ Start of generation: Project: rpcs3 tests, Configuration: Release x64 ------
2> ------ Start of generation: Project: rpcs3, Configuration: Release - LLVM x64 ------
2> LINK: fatal error LNK1104: can not open the file 'asmjit.lib'
1> LINK: warning LNK4075: '/ INCREMENTAL' ignored because of the specification '/ LTCG'
1> Creation of the library G: \ GIT \ rpcs3 \ bin \ rpcs3-tests.lib and object G: \ GIT \ rpcs3 \ bin \ rpcs3-tests.exp
1> MSVCRT.lib (initializers.obj): warning LNK4098: conflict between the default library 'libcmt.lib' and other libraries; use / NODEFAULTLIB: library
1> emucore.lib (sys_net.obj): error LNK2001: unresolved external symbol inet_ntop
1> G: \ GIT \ rpcs3 \ bin \ rpcs3-tests.dll: fatal error LNK1120: 1 unresolved externals
========== Generation: 0 succeeded, 2 failed, 32 upgraded, 0 skipped ==========


@datakurre It was the [U2F security key](https://www.yubico.com/products/yubikey-hardware/fido-u2f-security-key/).


Had to look twice for the difference :mag_right:


scrubbed as cfe8a97953d79764b42272c76b680696e07c9329


would result --> results


via Wikimedia Commons


If we introduce an internal interface (NonArgument) this can be condensed down to a single instanceof check


hahaha just added the tip word to the logo


why italicized?


updated PR now


Gotta say it's a pretty cool trick...


been curious for a while but never bothered to ask, why the `show_exceptions` config default to `false`? doesn't it lead to confusing errors on assertions or even hide bugs if you don't do any assertion on the template rendered? I'm running tests on rspec / capybara, wonder if this is any different on minitest?


Thanks for submitting this enhancement for Open API!


Hi,

I should have (in 2014 when I worked on #4584)… That’s fine (I believe users should deserve greater credit than us, as they are the stars of the show). I do know that we may need to update file headers (that’s a separate ticket), thereby giving people credit for their contributions.

From: derekriemer [mailto:notifications@github.com]
Sent: Wednesday, January 13, 2016 4:49 PM
To: derekriemer/nvda <nvda@noreply.github.com>
Cc: josephsl <joseph.lee22590@gmail.com>
Subject: Re: [nvda] The case sensitivity of the last find is remembered when searching using NVDA's find dialog. For Example, if Food is searched for with case sensitivity on Food will be found when pressing NVDA+f3, but food will not. (33189af)

Are you giving your self credit there anywhere?

On 1/8/2016 12:33 PM, josephsl wrote:
> Hi, Take time (the code looks fine to me; in effect, you are extending
> my own work (#4584)).
>
>
>
> From: derekriemer [mailto:notifications@github.com]
> Sent: Friday, January 8, 2016 11:25 AM
> To: derekriemer/nvda <nvda@noreply.github.com>
> Cc: josephsl <joseph.lee22590@gmail.com>
> Subject: Re: [nvda] The case sensitivity of the last find is
> remembered when searching using NVDA's find dialog. For Example, if
> Food is searched for with case sensitivity on Food will be found when
> pressing NVDA+f3, but food will not. (33189af)
>
>
>
> OK do you mind waiting until tomorrow night until I do this or doing
> it yourself? I am going to be away from my computer for the night.
> Because I'll be in the mountains.
>
> Sent from my iPhone
> Please disregard errors as this is a smaller device.
>
> > On Jan 8, 2016, at 11:53, josephsl <notifications@github.com> wrote:
> >
> > Hi, They would go on the copyright line.
> >
> >
> >
> > From: derekriemer [mailto:notifications@github.com]
> > Sent: Friday, January 8, 2016 10:39 AM
> > To: derekriemer/nvda <nvda@noreply.github.com>
> > Cc: josephsl <joseph.lee22590@gmail.com>
> > Subject: Re: [nvda] The case sensitivity of the last find is
> remembered when searching using NVDA's find dialog. For Example, if
> Food is searched for with case sensitivity on Food will be found when
> pressing NVDA+f3, but food will not. (33189af)
> >
> >
> >
> > Where would the names go?
> >
> > On 1/8/2016 10:09 AM, josephsl wrote:
> > >
> > > If you want, you could add two names: mine and yours, as both of us
> > > have worked on this and this is a good time to get recognized for
> your
> > > work (just my thoughts).
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15344152>.
> > >
> >
> > --
> > ------------------------------------------------------------------------
> >
> >
> > Derek Riemer
> >
> > * Department of computer science, third year undergraduate student.
> > * Proud user of the NVDA screen reader.
> > * Open source enthusiast.
> > * Member of Bridge Cu
> > * Avid skiier.
> >
> > Websites:
> > Honors portfolio <http://derekriemer.drupalgardens.com>
> > Non-proffessional website.
> <http://derekriemer.pythonanywhere.com/personal>
> > Awesome little hand built weather app that rocks!
> > <http://derekriemer.pythonanywhere.com/weather>
> >
> > email me at derek.riemer@colorado.edu <mailto:derek.riemer@colorado.edu>
> > Phone: (303) 906-2194
> >
> >
> >
> >
> > —
> > Reply to this email directly or view it on GitHub
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15346027>
> .
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15346891>
> .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15347088>.
>

--
------------------------------------------------------------------------

Derek Riemer

* Department of computer science, third year undergraduate student.
* Proud user of the NVDA screen reader.
* Open source enthusiast.
* Member of Bridge Cu
* Avid skiier.

Websites:
Honors portfolio <http://derekriemer.com>
Awesome little hand built weather app!
<http://django.derekriemer.com/weather/>

email me at derek.riemer@colorado.edu <mailto:derek.riemer@colorado.edu>
Phone: (303) 906-2194

—
Reply to this email directly or view it on GitHub <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15438878> .


testing github web ui


Did you make this SVG!?


spell out "meter"


Do you care about :abc:


To avoid the `!!` and be more explicit:

```elixir
def coerce(nil), do: false
def coerce(false), do: false
def coerce(value), do: true
```


@CasAndreu  We'll use Bootstrap panels for challenges. Use `r challenge_start()` to generate the code for the start of a challenge, and `r_challenge_end()` to generate the code for the end of the challenge


no camel case in citation


@CasAndreu This generates the code for the Bootstrap panels (e.g. see Panels in https://bootswatch.com/cerulean/)


@cbalster why would anyone want to do that though?


This indent is unnecessary and is just a waste of space. The philosophy of an indent is to improve readibility so you can understand logic more easily but here it serves no purpose.


i mean disabling that feature


s/property/properly/


Not really my place, but would mention that the initial semicolon is defensive; a normal JS file as written by a human probably wouldn't have such a semicolon, so I don't think it makes much sense for browserify to output an initial semicolon. A trailing semicolon though, certainly makes sense; it's proper etiquette for a JS file to properly terminate itself, and not clobber/mangle a following JS file when concatenated.

That said, your dist now seems to begin with `-e`, which I'm not sure I understand; so I leave the rest up to you.


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


use double quotes to avoid lint failures in your eslint pr


Closes #142


Are you using 4 spaces or a tab character for indents? Please always use 4 spaces. You can set your text editor to expand tabs to spaces. If you've been using tab characters your editor will likely also have a feature to convert all existing tabs to spaces.


Can we please put a linebreak here?


as in

```js
console.error(
  'stuff' +
  'other stuff'
);
```


Why would we shut it down but not delete it? It seems like either we shouldn't interact with it at all, or we should finish the task?


add note to PR or commit about why you moved from map to obj


update


Not sure if you read my comments on your last commit, so repeating the comment. Why is this exit() here? Wouldn't the header() call essentially jump out of this script? What happens if you remove exit()?


why aren't we deleting the transport? It appears we now leak it?


This is Hector: It looks good.


Working on it!

On Wed, Jan 13, 2016 at 7:00 PM, Ning Leng <notifications@github.com> wrote:

> Uncomment this!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tanzhang0504/try/commit/b9325f2f75e7ee1756b0694bcce296057844abb2#commitcomment-15439098>
> .
>


Update the first %v but keep the later %v, as it's easier to copy the manifest from log without double quotes


Looks like mFootstepSoundId was erroneously replaced with mImpactSoundId here.


[You got it.](7489e579)


In aurora.dme,
#include "code\modules\spells\targeted\equip\remove_horsemask.dm"
Will need to be added in manually. I suggest line 1644, I couldn't upload the .dme for reasons, and haven't tested the spells ability if isn't in there.

Either way, aurora.dme needs to be modified in this pull request.


this commit also adds the ability to designate an "owner" for the DiscordPlatform, who is considered elevated regardless of role or server ownership, but i fucked up and i guess it's two-in-one now


This will never be true with any impact ids (MaxSoundOffsets < ImpactStart). MaxSounds would work instead. It's not obvious what the difference is between MaxSoundOffsets and MaxSounds though.


Not sure what this comment is about. Could you explain more?


Minor opinion:  Perhaps tag blocks like this <div> which only have one line in them can be collapsed to a single line. It'll save you vertical space:

<div> <input ....> </div>

Your decision.


TeamCity ReOpenLDAP :: master [Build 41](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1668&buildTypeId=ReOpenLDAP_Master&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:34:56


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


Unnecessary blank line.


Sorry, that my repo was up-to-date before pushing commit a4a9e4d.


I believe this may be interfereing with the website, that is when I try to set it up, it just wont load the new homepage. any ideas why???


Merge this block with the one above. Also, I'm questioning if the indent is adding any value inside these php blocks. Just seems like code pushed unnaturally to the right. Your decision to make.


sos un capo


后面可以写上负责人名字


I've been finding `dimap` more and more useful these days. Perhaps a mini Profunctor lib would be worthwhile.


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


Can you fix this to use the JARs which are the result of
> git clone https://github.com/freenet/fred.git
> git clone https://github.com/ArneBab/freenet-plugin-bare-guide.git

I.e. assume the user cloned the repos of fred and this into the same workspace directory, and did *NOT* rename fred to "freenet" as you currently assume?
Also, I think the fred JARs are not in the main directory of the repository, but somewhere in lib/ and build/ or whatever


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1155](http://teamcity.monogame.net/viewLog.html?buildId=13545&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:29:26


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


@Notch8864 When you say you try to set it up, has the installation finished, or will the installer not load?


extra indent


Space before brace {


https://hub.spigotmc.org/jira/browse/SPIGOT-596

They say it's fixed here, but we are running Spigot 1.8.8 and I confirmed that it's not calling the listener method. Maybe the fix will be included in the 1.9 version.


MaxSounds is the total number of sounds in the entire vector. Which is what makes going off of that enum value entirely inappropriate.


Merge with block above


Don't mix exception handling and promises. You can restructure the try/catch as:

```js
return self.execute(...)
.catch(function(e) {
  ...
)
.then(function() {
  // "finally" logic here
});
```

Also, you can avoid `try { may throw } catch { return Promise.reject(); }` by doing:

```js
return new Promise(function(resolve) {
  // anything thrown here turns into a rejection
});
```
Same thing applies within any `then()` handler as well.


add ">" to header


This is good :+1: you may push the answer too when it is ready. Need not wait for the session.


Hey, just saw this and other changes. Would be nice to get these merged into `commonmark-java`, could you contribute them as pull requests?

As for the Android support, I haven't looked through all of it, but I imagine we could merge it into `commonmark-java` as a separate artifact (like the extensions). It can't be in core because we can't depend on `android.` packages for obvious reasons.


@MCDad working on it now


Does #pragma clang work for clang-cl as well?


![holy fucking shit](https://cloud.githubusercontent.com/assets/12599850/12313365/1c66c5ea-ba67-11e5-89aa-1a7d25102822.jpg)


ACK, good reason, hahaha


Needs an extra indent


Unfortunately I have no idea, nor any place to test it.  It looks like `clang-cl` is just a CLI facade over the actual `clang` engine, so I am hopeful that it will work just fine?


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


... wu... what?


This was present before, but I think this `&` is redundant and will just get deref-coerced out of existence.


I found the issue. It was in the extension library. The multitouch was disable for its subview. Sorry for the wrong PR.


I make this commit to save some work from home. Not match,  i just fix some minor issues from latest pull request.


@wilkinsona Amazing work! :+1: Thank you so much for all you do.


***NEEDS FIXING***
This is bad. The original verify_credentials() function was perfect where you give it the username and password, and it just returns True or False to say if the credentials are correct. Remember functions are meant to form logical self-contained blocks to perform a specific task.

- What is the point of returning the password hash and then comparing it later when you can do it within verify_credentials()?
- In case you didn't notice, your function name is also incorrect now. Its returning the password hash and is not actually verifying the credentials.

Please fix.


I pushed a fix. I don't have a testing setup yet. So I can't test it myself.


s/named/of named fields/


Asked one question, otherwise LGTM.


now it is serviCe


I think we should manage token boundaries as a separate configuration option, since we share these among both regexes. Something like:

```scala
tokenBoundaries = "(?:but)|(?:yet)|(?:nevertheless)|(?:nonetheless)|(?:however)|(?:notwithstanding)|(?:although)|(?:though)|(?:whereas)"
boundaryTokenRegex = "(?i)\\.|[!?]+|"${nlp.tokenBoundaries}
tokenPatternsToDiscard = "(?i)"${nlp.tokenBoundaries}
```

Or build the regex in the code:

*nlp.conf*

```scala
tokenBoundaries = [but, yet, nevertheless, nonetheless, however, notwithstanding, although, though, whereas]
```

*NLPAnalysisService.scala*

```scala
val tokenBoundariesRegex =
  config.getList("nlp.tokenBoundaries").stream().collect(Collectors.joining(",")).mkString("(?:", ")|(?:", ")")
val boundaryTokenRegex = "(?i)\\.|[!?]+|" + tokenBoundariesRegex
val tokenPatternsToDiscard = "(?i) + tokenBoundariesRegex
```


changed $rootScope to $scope (read the article)


Em chỉ cần để {{form.username}}  là tự động nó sẽ tự động tạo control cho mình giống như này <input name="{{ form.username.html_name }}" type="text" class="form-control" id="{{ form.username.id_for_label }}" placeholder="Username" value="{{ form.username.value|default:'' }}">


I am not sure whether disabling the prefix is going to work. `ld` kind of expects the `lib` prefix to be there. Have you tried whether the static library without the `lib` prefix works?

If we really want to disable the prefix, we should do it one level up - same way as it is done for shared libraries.


@nicolai86 it looks like you committed this directly to `master`, without making a pull request first. Please don't do this. There are a few problems with this change that I'll correct in a pull request.


Agh, apparently I'm commenting on an outdated version of this commit. I'll move them over to the latest so other reviewers see.


develop (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: OK (2351 of 2384 tests passed, 0 test warnings)

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


Either works for me


this is not the right logic. a nullable is represented as a union with a null and some other type.  You can use the Schema.isNullable() and Schema.getNonNullable() methods to help.  Please also add a test where the value is a nullable type.


I'm aware that this policy wasn't communicated anywhere, so I just filed #358 to do that.


@bngabonziza The patch commit log and the Documentation of optee have such information:
```
4.2. Boot setting in uboot:
 run loadfdt;
 run loadimage;
 fatload mmc 1:1 0x9c100000 optee.bin;
 run mmcargs;
 bootz ${loadaddr} - ${fdt_addr};
```
Here with CONFIG_ARMV7_TEE enabled in the uboot as I listed before, bootz will first runs into optee os and then back to uboot, then boot linux kernel. The linux kernel source code, you can download from `http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/` branch `imx_3.14.38_6ul_ga`, it shoud work.


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


OK


是


My original sql functions didn't have documentation either, but we need to fix this now. As your codebase increases, it'll be hard to remember what a specific function does, the arguments it takes and the value it returns. So each function needs a documentation header explaining all this.

Apparently phpDocoumentor is the most widely used PHP documentation standard/tool so lets use that:
http://www.phpdoc.org/

You can run the tool on your codebase and it'll generate a documentation reference site for your code.


looks great!


Yup, you're using tab characters. These lines have a tab character and then 4 spaces.


Indentation is weirded


What happens for clients that already set the client request ID from a parameter? Won't this override it by default?


ㅋㅋ 시작!!


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


很好~


should not use instanceof here, but should base it on the field type from the schema.


similar comment here


Perhaps `(navigator.language).indexOf("fr") === 0` ? That way we match French and not other languages that happens to contain `fr`


Extra line


Indentation messy


See #359


How will this be represented in the model (on `org.gradle.plugins.ide.idea.model.IdeaModule`?  We'll probably need a new property (`IdeaModule.languageLevel`). This property will be used to configure the tooling model as well as the `.iml` file.

Making this property read-only will avoid the confusion of allowing a user to override the value that is exactly mapped from `project.sourceCompatibility`.

If we make it user-editable, we'll need to decide whether we take the user-configured value (for module) into account when calculating the default language level for a project. This calculation could get a bit confusing, particularly if we want to use a 'null' value for module language level to indicate "inherit project language level".

I'm tempted to make `IdeaModule.languageLevel` a read-only property that exactly matches `project.sourceCompatibility`.


Save the value it returns so you only have to call it once


Not your bug, but seeing as I'm looking over this...

Does this fail to capture the names referred to in the self type? Perhaps the name of the self type itself is also relevant.


Put this in a for loop


引数なしの`count`がほしい


Co AccountInfo?


iframe层


Use this include in the header where ssize_t is defined (behind ifdef)


Funny, those sort of casts are OK on msvc but not on gcc or clang

THEYRE MESSING WITH MY MIND!!


***Code Quality: Awesome***

Using shorthand properties like padding and margin allows for code efficiency and better understanding


fix result type of the Write(use ssize_t) in Pipe header, in pipe impl declaration and definition.


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 250](http://ci.infinispan.org/viewLog.html?buildId=34027&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for 3f30df33b14952e088f946f3268a40c1260597ea
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:02

Process exited with code 1


:D :D :D


```
// matches contain index pairs for each match
t.Fatalf("Expected only 1 match, got %d", len(matches/2))
```


Hi @fredblain!

I am not sure whether QuEst++ works on Java7 or not. Have you ever tested it?
In the new build.xml I removed javadoc generation from the default running (mainly because it is not working yet). Therefore, when we run

`$ ant "-Dplatforms.JDK_1.<<version>>.home=/usr/lib/jvm/java-<<version>>"`

the javadoc is not longer generated.


… sorry ;)


There are "command" tags in many places, where there are command line commands.
See: grep -nr '<command>' en/*


WARNING: Useless assignment to variable - `foo`.


functions.php に JavaScript のコードを書くのはやめてください。古本市日付入力ボタンが含まれているページ、またはそのページで読み込んでいる header.php などのファイルに書くようにしてください。functions.php にはサーバサイドの処理を書くようにお願いします。


IMO, the collection of `ARTRealtimeChannel` should be responsible for `pendingMessages` and maybe for `queuedMessages`. Currently the collection of `ARTRealtimeChannel` is a `NSDictionary`.


nice algorithm.


TeamCity pull requests :: netty [Build 1027](https://garage.netty.io/teamcity/viewLog.html?buildId=4736&buildTypeId=pulls_netty&guest=1) is now running


Thanks. I have adjusted the wording to
> The `AGGREGATE` keyword can only be used after the `COLLECT` keyword. If used, it must directly follow the declaration of the grouping keys. If no grouping keys are used, it must follow the `COLLECT` keyword directly:


Wofür ist  `src/Scripts/HelloWorld` von Nöten?


In the interest of not hard coding hostnames should this be a class param with either an inline default or a hiera lookup of something indirect like 'monitoring_host' to make future moving around easier?


ほかの関数では古本市のことを「bookfair」と呼んでいるので、統一しましょう。


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


@andne How come Appveyor passed? I hadn't added this to the VS project files yet.


@pb00068 : These are also my thoughts, but I'll give myself time to get familiar with SF code. Invasive concepts are not always bad, but a lot of knowledge and patience is needed to make them work well, especially if we want code as simple as possible, which is reasonable for open source.


change to r2


this is the old commit


Confirmed, I fixed it. Thanks!


the imgur adapter sets the url, perhaps we should force all adapter to do the same? @matpompili


Do we decided to use hierarchy logging?
Have you investigated nesting inheritance of logging lovel in log4cxx?
Did it implemented for Qt and Win-logging?


:smile:


Ok, forget it. Looked again and it failed as expected. Strange.


Were you having a problem with MultiJson that prompted this change?


IMO, the collection of `ARTRealtimeChannel` should be responsible for `pendingMessages` and maybe for `queuedMessages`. Currently the collection of `ARTRealtimeChannel` is a `NSArray`.


mmh, you should use the full range for plotting the attribute (-1 to 1) of as I did is scale is between 0 and 1 and talk about normalize SS attributes


This function and s_loggingFilters could be declared static


Considering this will form the basis of our recommended structure way to handle route auth, what about publishing an event here instead that the RouteController subscribes to? I understand that this may seem like overkill, but to me *not* having the controller make this decision undermines the reason for a controller. Thoughts?


Cool


Sorry, but I don't know Java. This code is fairly easy so try to understand what each line does and implement it in Java.

-----Original Message-----
From: "roselin-yohan" <notifications@github.com>
Sent: 14.01.2016 9:25
To: "kirillmorozov/CodeEval" <CodeEval@noreply.github.com>
Cc: "Kirill Morozov" <mail2kirill@gmail.com>
Subject: Re: [CodeEval] 100% right solution. (c583370)

Please provide the solution in Java program.
Please; please
—
Reply to this email directly or view it on GitHub.


Hallo Felix,
du brauchst deine Commits nicht mit Versionsnummern ausstatten. Füge lieber eine sinnvolle commit Nachricht hinzu, z.b "Klasse ThreadsMain hinzugefügt" Zudem solltest du nicht alle deine Datein auf einmal commiten, sondern das, was inhaltlich zusammenpasst z.b. alle Testfälle die du hinzugefügt hast.<br>
Zudem fehlt bei dem Code die Kommentierung und der Codestyle muss noch eingehalten werden.
Bitte commit deine Code regelmäßig, das heißt spätestens alle drei Tage.


static?


Please remove this new end of line

Thanks


why not do this `cp dist` in the  `npm run demo` step?


せっかく便利な関数なので、HTTPリクエスト以外の方法でもコールできたほうがみんなにとって嬉しいです。
finish_trade と finish_trade_by_ajax の関係を参考にしてみてください。


prepare を使う必要無いのでは？


Yeah, am trying to do that in java.

If i give a input as:
XX.YY,XXX.Y,X..YY,XX..Y
XXX.YYYY,X...Y..Y,XX..YYYY,X.....YY,XX....YY
XX...YY,X....YY,XX..YYY,X..YYYY
XXYY,X..Y,XX.Y

Getting Output as:
0
1
2
0

Is this the right answer!!


Why not my regex solution? "/(\w)=(\d)(([^\s]))/"
String parsed should be "uid=1231(username)"
"1231" always is a number and username could contain many different characters except for space (I think It depends on the linux distro)
I think this solution "/(\w*)=(\w*)\(([\w-]*)\)/" will only works with dash character in username.


99 release => 99 releases


Yeah, that's always the problem between KISS and "good conventions". In this case we could/should keep it stupid simple as there is no need for more complex auth. I also think that's one problem we have with our Space architecture / conventions. You cannot see the (there is no) benefit of controllers before they are necessary!

But im not sure how to handle this situation. I think Space badly needs straight forward examples that do not scare people away with abstractions.


ここをリロードにした理由は何かありますか？


Such a big commit :O Try to split changes next time ;-)


Ok, will test again. The `--skip_venv` was also misleading in my case.


Co mi to Info říká? mě nic.

AccountFio se nelíbí? AccountToken? AccountFrom? MerchantAccount? AccountMerchant? AccountOwner?


understood. I changed it back to an explicit range


Looks good to me


Could we expose a `public DataBuffer write(DataBuffer... others)` method in the `DataBuffer` interface and `@Override` it here instead?


@Rub21 Why did you remove this check?


And this @Rub21?


fixed later, thx ;-)


@Rub21 these checks ensure we don't write empty geojson to stdout. Any reason why you took them out?


43行目 document.querySelector を使わなくても jQuery('#insert_bookfair_info')でいいと思うのですが、何か理由はありますか？


@alexharv074 the config hash part indeed gets merged into the instance config.
This is done that if you have multiple instances on a single machine you can set some options globally instead of duplicating that at all the instance configs.


I think the initial PR was *much* better. Putting only one check per line is a bit inconvenient. Casting integer literals around is annoying as hell... The output also seems to be step back compared to the original code. Also your `equal_to` for floats is just weird. What benefit do you see over epsilon comparison?


Looks like duplicated commit :(


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


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


Remove a non-existing item?


Denke nicht dass das hilft, da der / sollte eigentlich auf windows und linux gehen.


Where were these strings before?


Error is optional


not parameters for getMe?
missing updatetMe


Looks like you misspelled "canols"!!


Document the behaviour for removing item not in set?


develop (kbenne) - i386-Windows-7-VisualStudio-12: OK (2350 of 2384 tests passed, 0 test warnings)

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


But we do have a [controller](https://github.com/meteor-space/donations/blob/develop/packages/app/source/client/controllers/route-controller.js), so I'd be asking, why there are two places where the router logic is managed. If we didn't have the existing controller it would be a no-brainer, but there's a reason we use the controller now (event-driven UI), so why not stick with that pattern?


echo が複数行に渡ると可読性が低くなるので、heredoc を使ってみてください。


$ grep "std::binde" */include/CGAL -r
Bounding_volumes/include/CGAL/Min_annulus_d.h:  typedef  std::binder2nd< std::divides<int> >
Mesh_3/include/CGAL/Gray_image_mesh_domain_3.h:         typename Transform = std::binder2nd<std::less<Image_word_type> >,

afabri@twombly /cygdrive/c/cgal/git


Prevent commands to be sent to disconnected devices


Thanks for the changes. Most of the comments inline appear a couple times, please fix them across the code.


there the indentation somehow looks wrong


If the controllers is making a decision than we are note making use of flow router route groups and triggers. Which are really good features of flow router.
We could publish an event and than the controller could decide which route to go if the user is not logged in but that is always login route.


This is actually really cool.


This commit doesn't really do what it says on the tin.  Some squashing needed?


This comes from AccessToken.fromString, and the parameter should be an UUID.
Antoine


OK


@jsuereth you remember with what this should be replaced? it just says "_Use higher level abstraction_" (added in 34cbfca1b3d0b347d0ef3033a06e269ff55088fd)


Thanks, it helped me lot.


some of the indentation seems to have gotten wrong. did you replace the tabs with the wrong amount of spaces? I think we usually had 3 spaces for HDL code as in optimsoc, but I don't overly care.


I'm not suggesting removing groups or triggers, just delegate the logic and resulting state change to the controller via an event


Thanks! Could you create an issue for such things?