@kehusa
Last week I've wrote a new TypeDefinition called **RemoteWSChan** that uses a 'classic' Web Socket broadcaster without any protocol overhead. With this new channel you can get rid of the **WSMsgBroker** component in your models as we already provide a public server at **ws://ws.kevoree.org**

> The counter part of using this channel instead of **WSChan** is that your components output ports won't receive any ACK for the messages they send.

The server at **ws://ws.kevoree.org** uses [this](https://github.com/maxleiko/ws-broadcast/blob/master/index.js) module, which is a straightforward server that broadcasts messages to all connected clients but the sender in a specific "room". Room can be dynamically created by appending something to the **URL**

**Example:**
`client0` connects to the server using ws://ws.kevoree.org/**my/room/0**
`client1` connects to the server using ws://ws.kevoree.org/**my/room/0**
`client2` connects to the server using ws://ws.kevoree.org/**my/room/0**
`client3` connects to the server using ws://ws.kevoree.org/**my/room/1**

If one of `client0`, `client1` or `client2` send a message, then, they will all receive that message but the sender. `client3` is alone in another 'room'
The room names can be anything, so ws://ws.kevoree.org/aaa also works.

This new TypeDefinition is available under a SNAPSHOT version for now:
  - RemoteWSChan/5.2.9-SNAPSHOT

It uses the 'room' concept to connect the ports by using the following convention:
ws://**hostAttr**:**portAttr**/**pathAttr**/**uuidAttr**/**portPath**
Be aware that, if two (or more) channels use the same UUID attribute, then the messages will be shared between the connected ports if they have the same paths

**Usage example:**
```kevs
// ... some instances creations ...
add chan: RemoteWSChan/5.2.9-SNAPSHOT
set chan.host = 'ws.kevoree.org'
set chan.uuid = '0123456789' // this is mandatory to prevent conflicts between channels
// ... some bindings ...
```


Where did I push something myself? I created a proper pull request and did not receive any email.

Regards
Andreas Jung
-----
Sorry for being brief - sent from a mobile device.

> Am 13.04.2015 um 01:39 schrieb Timo Stollenwerk <notifications@github.com>:
>
> @zopyx Please make sure a package is in auto-checkout in buildout.coredev before you push changes. You should also have received an email from mr.roboto telling you that. It seems your change (or @davisagli's) broke some tests: http://jenkins.plone.org/job/plone-5.0-python-2.7/785/console
>
> —
> Reply to this email directly or view it on GitHub.
>


v 2.4.1.
change: CargoInfo-skin [previous as additional-skin] moved to /tab-to-insert/, will be as one of tab;

Previous version in version-log.xml


Thanks!
This will be better.


$browser.defer.flush() must be used when testing methods that use $scope.$applyAsync(), hence having a definition for it helps.

Ref: https://github.com/angular/angular.js/issues/10788#issuecomment-70376834

IBrowserService.defer in Angular source: https://github.com/angular/angular.js/blob/master/src/ng/browser.js#L303


aaaaa


Maybe add alert here and set hidden to true to hide this button.


In general I'd say only to expose things that we know will be used by the user. "If in doubt use `internal`" :)

+1 for making it internal


Better contact oracle, if this is the first time they have this problem. This is not a desirable behavior.


@moonso: after loading the research sample the "variants" list took forever to load so I restricted the number of compounds to 15 per variant on this view. All show up when you go to a specific variant. This is OK right?


Det var godt det her blev slettet, det gav ikke noget functionality til projektet.


Hello,
I have this bug in the 1.13.1 version as well. Should this fix be included in that version or not? The problem is, that the invalidHandler get called multiple times during on submit.


Enig i det her, dårlig author.


I'll check it. Thanks @masterrr


/* */ fails when there are other comments between it so it's just easier this way.

visual studio adds and removes this in 1 button click anyway. Just highlight the whole code (Ctrl + A) then use with (Ctrl + K, Ctrl + U) or (Ctrl + K, Ctrl + C) to comment / uncomment them all.


Maybe to protect against unintended introduction of moves/pokemon/items/etc into past gens, add a few test cases that fails if the moves/pokemon/items/etc aren't exactly matching a test list?


제파일에서 session.js 수정을 안해서 그랬네요 db 생겼어요 감사감사


This does not fix the issue. pdev,1 is not available still.


Yes, it is only 10000 ids&flags, not the whole message.
With it, on my cozy, the RAM usage spikes to around 6% of the total 2Go of RAM.


Good.


ok thank you for additional information.


If you haven't already, I'd change those passwords ;)


I started to use special software to analyze completed hands by tracking hands count, evaluating my actions, showing total profit etc. This piece of software, called Poker Tracker 4, is working while I'm playing and is using my hand history and store it's data in postgresql, so I had to create database to store data. `pt4` script is starting postgresql database with given datadir and then doing some wine setup (I write that commands using torrent's given bat script) and the whole system is working fine.


I had wondered why the timeout was for 1 2/3 hours :)


Should be `gemfile` according to [Travis Doc about Ruby](http://docs.travis-ci.com/user/languages/ruby/)


That's not a very interesting message, I agree. I'm thinking of unexpected/new warnings that might appear.


@rodricios could you explain how this may be used and how the code works?


missing `&:focus`


I think this wine setup is not needed to run twice, but I don't care.


space missing


related to https://github.com/tohecz/ctuthesis/issues/3


It looks like method name doesn't match our naming convention.  Can you change method name to **validateMetadata** ?


если не ошибаюсь, этот стиль относится только к кнопкам, т.е. к 'btn'. в стиле bootstrap правильно назвать этот класс btn-pink. :-) В духе btn, btn-success, btn-lg.


Oh yeah this was because of reasons.


TeamCity ChameleonForms :: Continuous Integration [Build 238](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3559&buildTypeId=bt20) is now running


On Mon, Apr 13, 2015 at 12:56 AM, plieven <notifications@github.com> wrote:

> I am not sure if we should ignore this messages per default. As far as I
> could find out the target is not allowed to send Async Event Messages of
> Type 0 unless its explicitly activated.
>
> "If the target supports SCSI asynchronous event reporting (see SAM-2) as
> indicated in the standard INQUIRY data (see SPC-3), its use may be enabled
> by parameters in the SCSI Control mode page (see SPC-3)."
>

I think we have to, at least until there is an API that allows propagating
them back to the application.
We don't have control of this setting from within libiscsi since this
setting can both be saved, and persist across target resets,
or be shared across all IT nexuses, and thus a different initiator could
change this without libiscsi knowing about it.

Even then, if the application is using the iscsi_scsi_command_async()
function directly,
we would not even know from within libiscsi when the application itself
drives changes to this field.

So silently eating and ignoring these UAs are better than triggering a full
reconnect since we can not prevent the target from sending them.

—
> Reply to this email directly or view it on GitHub
> <https://github.com/sahlberg/libiscsi/commit/1a0897ae100a625976fb54379933a0ddabcfc145#commitcomment-10687614>
> .
>


OMG. Use the list of files from the updater-script


@cibo94 you should have created log dir at first. @regeciovad reported the problem.
```
$ python3 RPG.py
Traceback (most recent call last):
  File "RPG.py", line 28, in <module>
    main()
  File "RPG.py", line 12, in main
    base = Base()
  File "/home/dominika/rpg2/rpg/__init__.py", line 24, in __init__
    self._setup_logging()
  File "/home/dominika/rpg2/rpg/__init__.py", line 41, in _setup_logging
    handlers=[logging.FileHandler(log_dir + "rpg.log"),
  File "/usr/lib/python3.4/logging/__init__.py", line 999, in __init__
    StreamHandler.__init__(self, self._open())
  File "/usr/lib/python3.4/logging/__init__.py", line 1023, in _open
    return open(self.baseFilename, self.mode, encoding=self.encoding)
FileNotFoundError: [Errno 2] No such file or directory: '/var/tmp/rpg/rpg.log'
```


@sedovalx
Саш, мне это кажется не очень сложным. Может я сам попробую сперва? Я представляю, как сделать выпадающий список по аналогии с ролями пользователей, могу сегодня попробовать.
Мне кажется, что если он будет большой, то это не проблема. Часто на всяких сайтах так предлагается выбрать страну или язык. Это может не мегаудобно, но вряд ли спи ско классов будет сопоставим с количеством языков или стран. Ну 10 штук будет, может 20, а то и 5 всего.


Please revert this commit. The requirements.txt generated it wrong . It contains packages which are not being used in this project. Always validate your commits before you push them. Also venv folder should not be part of the commit. Please add to gitignore. @rahulrrixe can you please help @vivekanand1101 to fix this commit.?


fixed in 897b1ef


Why partly in German?


I really like this design, providing a `ValidationError` is exactly what I wanted to achieve, but you did it months ago!
Seems much cleaner, and makes express-validation simply throw a specific type of error (that can be catch quite easily!)

Would you propose a PR for this commit?


this is a leak


I forked the package properly and created this PR

https://github.com/plone/plone.schemaeditor/pull/22

I still do not see where I pushed myself to the plone master and not only to my fork?

Regards
Andreas Jung
-----
Sorry for being brief - sent from a mobile device.

> Am 13.04.2015 um 04:54 schrieb Zopyxfilter <zopyxfilter@gmail.com>:
>
> Where did I push something myself? I created a proper pull request and did not receive any email.
>
> Regards
> Andreas Jung
> -----
> Sorry for being brief - sent from a mobile device.
>
>
>> Am 13.04.2015 um 01:39 schrieb Timo Stollenwerk <notifications@github.com>:
>>
>> @zopyx Please make sure a package is in auto-checkout in buildout.coredev before you push changes. You should also have received an email from mr.roboto telling you that. It seems your change (or @davisagli's) broke some tests: http://jenkins.plone.org/job/plone-5.0-python-2.7/785/console
>>
>> —
>> Reply to this email directly or view it on GitHub.
>>


no need to wrap a single line code into a new method.


This splendid addition has been merged !


This is a complete npc design to allow flowing #IF #ACT commands under each other (as the official npc structure allows)

However doing so stopped the ARG system from working - so i had to revert back until i find a solution


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0-PullRequest120+013](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3559&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3005 Build time: 0:0:0


I missed this year ;_;


can you change the method name to **validateLicenseIdAndDeviceId** ?


this is inside the .csproj file - everything here is autogenerated by visual studio. In this case these lines are created when new files were added. They were since moved in to their own folder.


Can't you use `Optional.fromNullable` here?


хорошо) поправлю.


You need to verify that it still exists in the list here before you unref, or two concurrent revoke will crash


@maxandersen - I've implemented the changes and hopefully this should now be acceptable. Please let me know we need any further changes and thanks for your patience with our first article publishing attempt.


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


It is still cutting off some reports.  Some of the stuff used in the guide I found I have not used before, so it will take a little longer to find and fix this issue.


Hoeveel paginas thesis heb je al he beste?


Actually, that should cause a fail in the resource delete operation. That will only succeed once.


why delete Multiple routers?


Sorry for commenting here. Is the only way to talk about specific file. As far as I tested, this is only intended to create one type of guy: that with machines names and IPs. There is no way to create the labels based on the parsed JSON, right?


Why were the useful social icons removed from social-icons.html?


@Argelein если считаешь, что несложно, то делай. Но учти, что в редакторе аренды будет аналогичный выбор машин и водителей. Там их кол-во будет немалым. Ко всему этому там нужно будет отображать в этих выпадающих списках не всех водителей и машины, а с некоторым условием. Например, только незанятые машины. Это тоже надо учесть при реализации такого комбика. Первая мысль - это указание серверного адреса (к примеру, /api/cars?isOccupied=false) при описании html комбика.


Lo único que cambiaría es que añadiría un comentario en `NSPersistentStoreCoordinator+MagicalRecord.m:95` rollo "in iOS7, CoreData fails to migrate the model even with simple upgrades on the model, and on top of that, this API returns store=nil and error=nil, thanks Jonny Ive"

Ah, y quizás demasiadas notificaciones, pero mejor tenerlas y no necesitarlas, que necesitarlas y no tenerlas :)

Incluso pinta bien para mergearlo en el proyecto principal!


We use 3 spaces indentation, please fix and interactive rebase your commits, otherwise this becomes unreadable pretty quick.


Understood.


On Win with VS10.0:
62>..\..\..\OpenMS\src\topp\OpenSwathAssayGenerator.cpp(239): error C3861: 'round': identifier not found


thank you for cleaning up the code --- and adding a well defined return value.


Fixed in 417c072fb7c49abefafdb6354c8fae490b79c5cb.


dropshop 不是锻工车间。这里指用来做飞机中转的意思。就是临时泊位。比方说，你的飞机飞出去了，不等于你可以用这个位置停新的飞机。如果从别的地方转架飞机过来，即使飞机还没到，这个位置也被占用了。
怎么翻译再考虑一下吧。


This change may need to be propagated to the next branches (I only checked this branch, not the rest).


`final` стоит добавлять параметрам методов тоже


Если поля класса объявлены публичными, то методы для их получения не нужны.

Тут есть два варианта:
1. сразу объявить поля x и y публичными и обойтись без геттеров
2. объявить их приватными и сделать для них публичные геттеры

Для такого простого класса вполне сойдет и первый вариант, тем более что они `final`.


Ist ein bisschen inkonsistent bei den Fallennamen. Einige haben noch das Falle dran, andere nicht. Evtl. sollte man das allgemein verändern in "Falle: <Fallentyp>"? (Oder "<Fallentyp> (Falle)"? Wobei ich das erste vorziehen würde).

Es würde die Sache auf jeden Fall einfacher machen zu erkennen, dass es sich um eine Falle handelt, wenn es weiterhin dran steht ;)


Then, we should mention the unit in the comment


Кажется, что `typeInfo` (и, следовательно, `typeNum`) не используются.


Метод `skipPart()` здесь вызывать не нужно, потому что буфер читается не последовательно, а по конкретному оффсету.


Should find a way to do this without the use of the `binary` and `bytes` packages.


Could you make sure that all the new long fields are declared in corresponding java classes as longs as well ?

Have a look at those files:
https://github.com/facebook/fresco/blob/master/imagepipeline/src/main/java/com/facebook/imagepipeline/gif/GifFrame.java
https://github.com/facebook/fresco/blob/master/imagepipeline/src/main/java/com/facebook/imagepipeline/gif/GifImage.java
https://github.com/facebook/fresco/blob/master/imagepipeline/src/main/java/com/facebook/imagepipeline/webp/WebPFrame.java
https://github.com/facebook/fresco/blob/master/imagepipeline/src/main/java/com/facebook/imagepipeline/webp/WebPImage.java


It is possible to implement the update of object 0x1F89/A conditionally if node-ID is in range 241..250


дебажный принт


This patch breaks build


manufactured or recovered equipment 从意思上说，指自己生产出来的东西，和装备补给品。翻译成 制造设备和维护设备 有点奇怪。


Кажется, это осталось от тестового примера и больше не нужно.


`@locked` stays true If executing the `select release_lock` raises an exception


Maybe it's worth mentioning in the readme, that the gem is not threadsafe


There is already an attempt to do same, around line 240:

    #if BOOST_WORKAROUND(BOOST_MSVC, BOOST_TESTED_AT(1800))
        // prevent warning C4512: assignment operator could not be generated
        options_description& operator=(const options_description&);
    #endif

I would prefer a single solution, along with a comment explaining why this warning is generated.


That's something you'll have to guide me through than... probably via whisper or something.

I've tried the wiki entry, first exactly as it is there but that tried to delete a bunch of stuff in the master so I cancelled that... Then I did it like this:
![image](https://cloud.githubusercontent.com/assets/848114/7113153/3b2a7f40-e1d1-11e4-8981-1c82d49690c5.png)

But I have no idea how to resolve the conflicts now :/


this is WIP, there are some pages missing


It is working as it should be, so don't think anything else needs to be added here.


the "zero temperature" error does occur when running inside c, too.


good, we don't use this anymore since https://bugzilla.mozilla.org/show_bug.cgi?id=858908...

However we might want to revert that bug behavior change at one point (sending notification for class0 messages) so we'll add it back if we ever need it :)


Аббас. Из view сюда приходят занчения фильтров. Найдешь их в хэше params.
Попробуй сделать выборку данных по этим значениям.
Основной затык в том, что нужно проверять каждое условие на nil.


Yep! Sorry added comment about it in "files" section to save your time [1], but it's not visible in "commit" section :(

[1] https://github.com/mozilla-b2g/gaia/pull/29382/files#r27910703


Чтобы не заводить много локальных переменных можно просто:
```java
item = new PlotItem(time, data.getInt(resourceParameterIndex.getOffset()));
```


@Argelein очевидно эта идея не ложится в функционал фильтрации через queryParams, так как у машины нет поля isOccupied. Поэтому лучшей идеей выглядит #38. В html комбика ты будешь указывать псевдоним серверного запроса и получать данные по запросу. Сервер будешь вызывать например так: /api/report/not-occupied-cars, где последняя часть -это псевдоним запроса.


Ошибочка: #37, а не #38.


AHa


sure


I saw your related problem about QTimer precision, and how it lose precision while running in background, this code fixed it? And it seems that there's no equivalent in QML Timer type.


sprite เคลื่อนที่ได้4ทิศ


<script
src="http://cdn.mathjax.org/mathjax/latest/MathJax.js?config=TeX-
AMS-MML_HTMLorMML">
</script>

\[yv_{xx} +\frac{9}{4}\alpha^2yv_{zz}+\frac{3}{4} \frac{\alpha}{\sqrt{2}}v_z=0,\]
\[\Rightarrow \varphi_x = \frac{-b\pm\sqrt{b^2-ac}}{a}\varphi^\pm_y\]


สร้าง class flag เพื่อสร้างธงให้ sprite วิ่งเก็บ


@vincent-michael is working on locales, he is also converting some locale tables to a new format (i.e https://github.com/TrinityCore/TrinityCore/commit/cb2bb1c1eeaed549327e862ef9568b06463adee2)


```php
if (
    (isset($_SESSION[ 'language' ]) && ($_SESSION[ 'language' ] == 'ac')) ||
    (isset($_COOKIE[ 'language' ]) && ($_COOKIE[ 'language' ] == 'ac'))
) {
```
Should be indented this way (easier to read and understand) - also we use `||` instead of `or`, don't we?


ส้รางให้ sprite เป็นรูปภาพ


aaaah I missed it :D si I did the archeology once again :p


`unittest.TestCase` is new-style object. You should be using `super()`.


@crazycat69 - Your Geniatech T220a patch I take from here:

https://tvheadend.org/boards/5/topics/10864?r=15567#message-15567

> Copy T220-fix.patch from here : https://tvheadend.org/attachments/download/1976/t220-fix.patch to the backports directory in media_build.

However it fails to apply some hints in the `usb/dvb-usb/dw2102.c`. When I inspect the file in Ordoid 3.10 kernel branch, there is not yet any function named `static int t220_frontend_attach(struct dvb_usb_adapter *d)`. So it seems like that patch I found was to complete the HD (T2) hidef channels support. However I seem to be needing a more full / complete T220a patches. For such earlier kernel.

@crazycat69 Will go to look in the media_build (media tree) backports project for a missing initial T220a code. Otherwise if it's not there please help if you can.


I was thinking about the namespace issue as well. What I think that install arguments are irrelevant for the Mac version (as it uses zipped archives), and if it someday becomes relevant, it would be hard to interpret it otherwise.
On the other hand, introducing a new namespace would (I feel) unnecessarily complicate a bit things for the end user.

So, please let's decide about the namespace and I will make it so.


There is also [`wp_cache_incr()`](https://developer.wordpress.org/reference/functions/wp_cache_incr/). However, I feel like having the microtime metadata might come handy in the future maybe.


You need a ```<span>``` inside the label for this method to work which is what breaks the possibility to use the checkbox function. As I don't want to modify the original helper method and add the html tag there, I've implemented another function. If you have ideas on how to do the same thing (adding a span with class) with the checkbox function let me know.


I'm new to node as well as to git. Correct me if I'm wrong but isn't node-rest-client your package dependency? I don't even require it anywhere in my code. How (or why) could I use other version than the one you specified in your package.json? To be exact npm install downloaded node-rest-client 1.4.8 in my project.

What I do in my code is basically follow example from README and pass created client to other part of code that manage searching.
I'm not sure if this has anything to do with differences in Solr versions but I'm 100% sure I get XML response in data (line 97) unless I use 'wt=json', which provide me with JSON.

Please let me know what I can do to help further. I don't have much of free time on my hands but I'd like to contribute :)


It'd be nice to put description to commit message of what "-r" does and why do you add it here.


Ugh this is a disaster... some files have the >>>>> and ===== stuff in it so I can edit it and place the right code... but some say there's a conflict but when I open the file there is no >>>>> and ===== so I can't edit it to fix it... I'll wait for you to come online so you can walk me through it, but it's probably faster to just do it manually :)


ostree and atomic are not one-and-the-same; really this should be in a separate plugin since it's possible to use the two independently.


Note to self, testnet is being mined heavily with almost 10,000 blocks in the last 24 hours - weird results (inverted pricing?) seen in metadex maps when client is syncing.


@k88hudson I am pretty sure that you don't need to assign the value null, since by default it is null anyways unless you assigned it a value.


Should we do something about the TLS dependencies?
http://labs.twistedmatrix.com/2015/04/twisted-1510-released.html


Methods like `close()`, `open()`, and `handleConnection()` should not be turned into procedures because they are marked `private`.

Also, did you mean to put this on the `[examples/expected-generated-src]` branch? I would have expected it to go on the `[examples/original-benchmarks]` branch. (I think that we could rename the branch if you want.)


geocoding*


Should we force a boolean instead of potentially returning iterator or null/undefined here?


Force boolean here too?


Looks like you forgot to update docs for `isIterator`


Can we force everyone to use the same version of Java? I'm guessing I'm using 7 if I didn't run into any issue, but I'm fine switching to 8.


@k88hudson This logic should probably be simplified to match the logic similarly in the main activity.


I think I took care of that in an easier way, I'm looking for the letter itself, so a regex wouldn't be the best option. Check it out and let me know if you have any further thoughts!


In the commit messages it's better to format it like this:

Present-tense summary under 50 characters

* More information about commit (under 72 characters).
* More information about commit (under 72 characters).

http://project.management-system.com/ticket/123

It's the best practise in Git committing.

[1] http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html


Keywhiz should only be using Java 8 so it's strange that you haven't run into this issue.  Travis CI ran into it too: https://travis-ci.org/square/keywhiz/builds/58204609

But yeah I agree, this is a strange bug we should share with the jooq mailing list.


This bug also bit me a few times when I edited a config.sys and then immediately rebooted.


This can be removed AFAICT


Akso this


And this


Why are you defining _WIN64 and _AMD64_ for Release|Win32


should be context.module.exports


cleanup needed


I'd recommend that you move this logic into a property (or method) in the `Room` class. Something like `MyRoom.IsValid`. You should talk to @kyranstar about the [logic](https://github.com/kyranstar/Hunt-The-Wumpus/blob/master/Final%20Game/Shared%20Code/Map/Map.cs#L171) that he has written already for validity testing -- that code should probably be moved into the cave or the room instead of the map.


For me, this commit fails to compile. I think it is because of two reasons:

1. There is a missing `import org.paninij.lang.Wired` in `WorkerTemplate`. This just needs to be manually added.
2. There are a few missing imports in the `java_net_Socket$Duck$$Thread` auto-generated duck artifact, `OutputStream`, for example. These need to be automatically added by the duck-generation logic. I think that we need to look into using the `TypeCollector` to do this.


Remove please


LGTM


We might want some way of ensuring that there is only one of these present -- when I try running the maven test suite I get an error in some tests:

`1) Error in custom provider, org.postgresql.util.PSQLException: FATAL: sorry, too many clients already
  at keywhiz.ServiceModule.jooqContext(ServiceModule.java:127)
  at keywhiz.ServiceModule.jooqContext(ServiceModule.java:127)
  while locating org.jooq.DSLContext`


Is this in the jenkins plugin repository? I'm still seeing this error when I try to use it.


@schneems this is good, but we should probably specify that it's all objects allocated since _when_..


@kyranstar These constants for room shape and other map info should be given a home. Any idea where that might be? We need somewhere that can be accessed by this renderer, the map, and the cave.


i think this one might just be able to be pass through of the input properties


Hi Nesan,

I just received 8 emails today from you, but none of them show the original message you appeared to be replying to, is there a reason the original post is not being sent but your replies are?

Just wondering.

Tony

From: Nesan Waran [mailto:notifications@github.com]
Sent: Monday, April 13, 2015 9:54 AM
To: Gilroy-Morgan-Hill-Devs/stats-tracker
Subject: Re: [stats-tracker] fixed homepage rendering bug, created skeleton games app (94d1f74)

I understand the allure of shortening names. But, IMO, it's a bad habit.

Think of it from the perspective of a second programmer (or yourself in 6 months). Grepping <http://en.wikipedia.org/wiki/Grep>  for the name will only produce the import, without finding any uses of it. The 2nd programmer then has to grok <http://en.wikipedia.org/wiki/Grok>  the fact that the module was imported under another name in this file, open the file, then use your notion of the module while reading about it's use here.

The as clause of the import statement is useful when you have namespace collisions.

from django.db import models  # django's stock models
from common import models  # our models, which add functionality to django's models

In this case the least imported module in the project is django.db.models. So I'd write the following.

from django.db import models as djang_models
from common import models

Again, just my 2 cents.

—
Reply to this email directly or view it on GitHub <https://github.com/Gilroy-Morgan-Hill-Devs/stats-tracker/commit/94d1f74fcbd1c45928d86affdac467962aeda37e#commitcomment-10696709> .  <https://github.com/notifications/beacon/ADt2-tGXXW7tjm-pjhNt2RXKD9rXjwNkks5n--w0gaJpZM4D_jmE.gif>


nice one, I should have noticed it


Gah whoops


@wjps, this looks very promising. Thx!


We could just create a constants class?


already reported at DNN-4352


Yeah good call


It seems that there is a health check somewhere (though maybe not here) -- after deleting this code, the dropwizard AdminEnvironment gives a warning about how this app lacks healthchecks.  It'd be nice if we can add one, though we can discuss if there should be one for jooq (and where it should go)

I get the following warning on some tests (such as keywhiz.service.resources.SessionLogoutResourceIntegrationTest):
`!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!    THIS APPLICATION HAS NO HEALTHCHECKS. THIS MEANS YOU WILL NEVER KNOW      !
!     IF IT DIES IN PRODUCTION, WHICH MEANS YOU WILL NEVER KNOW IF YOU'RE      !
!    LETTING YOUR USERS DOWN. YOU SHOULD ADD A HEALTHCHECK FOR EACH OF YOUR    !
!         APPLICATION'S DEPENDENCIES WHICH FULLY (BUT LIGHTLY) TESTS IT.       !
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!`


@b005t3r made the build scirpt much easier to use, see the readme how to


Thanks


I don't really like this switch statement: it separates the enum from its meaning, and is sort of fragile. I'd prefer to see something more like

  isNonNull(vec4Vector);
  isValidData(4, vec4Vector.length);


MarkdownDocs (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1639 of 1639 tests passed)

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


@gwendalaze, I think we should unite boolean variable and float one like you did in smoking


This is unhygenic as it might bind to some other value in scope named `Some`.

Prefer `Ident(definitions.SomeModule)` / `Ident(definitions.NoneModule)`.

When referring to members of `defintions` in the `Parser`, we have to dance around a little to support multi-threaded parsing by forcing the lazy vals eagering in `interactive.Global#forceSymbolsUsedByParser`.

That change should be test driven by expanding the test case added in 760df9843a910d6c3618e490c752eb03fb6924bd.


I think we should also check players for "poisoned" just not to poison them again if they're already poisoned


86046200_CondenserToZone_New (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1135 of 1135 tests passed)

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


how about tying this with Fingerbank?


I'm not sure that this test is even right. It looks OK for set_vertices(), set_normals(), and set_triangles() - but isn't setVec2Vector() expecting a glm:vec2 for each vertex (or, perhaps, a single vec 2 for each vertex)?


I don't think this is the right patch.


1570-EnergyPlus-8-3 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


I don't think that for comphrehension desugaring should be allowed to assume this much about the library. (special status for `Option` and the `flatten`)


1570-EnergyPlus-8-3 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


What do you think about the following?
```
# import reverse
from rest_framework.reverse import reverse

# add `self.taps_api_url` to setUp()
self.taps_api_url = reverse('v1:taps-list')

# Then use self.taps_api_url in self.client.<get, post, put, delete>
resp = self.client.get('{0}{1}'.format(self.taps_api_url, tap.id)
```


@simianhacker getting an error here when appState is not defined and I try to redirect.


To randomly generate a cave, there are two main ways that I think you can go about this:
 1. You can brute-force it. Start in a room, start randomly adding connections to rooms that have <3 connections until you have hit a dead-end. Then run some assertion logic to check the validity: if it's not valid, reset and try again.
 2. Solve it like a puzzle. Start with some known pattern, start changing connections until every room has met the required conditions. Be methodical, so that by the end you are 100% sure that every room is valid.

Either way, I think you will need two basic things:
- A system to randomly add hazards with some constant probability.
- A system to make sure that the connections are valid. This should probably be run in a unit test.


Why did you move it here?
It's ordered by skill Id not by level.


Got a firebug error scope.product.interestRecalculationData.recalculationRestFrequencyType as undefined.Hence made this change here


Optional.of(columns) => Optional.of(ImmutableSet.copyOf(columns))


See also @clhodapp's exploration of extensions for for-comphrensions to add support for `groupBy`: https://groups.google.com/forum/#!topic/scala-sips/TIvZRM8-jcE

That proposal never quite met the simplicity threshold.


do we need this flag? or is an empty LocalProperty list sufficient to indicate this?


thanks :)


Or should it be `//` so it aligns with the page protocol?


If you run gradle eclipse it will fix your project. Regardless, I use IntelliJ so


Changes to the syntax should also consider the impact on quasiquotes. Support for for-comprehensions was initially added to quasiquotes by @densh in scala/scala#3090.


Can you uncomment this line and set it to the default port for go servers? That way people don't need to change their host files?


A static class? That seems like it might be our best option. Do you think we should just add static members to the `Map` or `Cave` classes, or create new one?


Uncomment this section and leave it at 1024. Remove the vb.gui portion though


Maybe take the function call out of the assert, even if it is a release assert? :P


Same problem. In fact, I added traces around this call and they don't appear, so....  it seems to me that this cell is not executed (but I know nearly zero of ecto, so I don't know what's going on, sorry)


It doesn't really matter. If you think we're going to have a lot of
constants, maybe we should make a new class. If not, probably just put them
in an already existing class.

On Monday, April 13, 2015, Wasabi Fan <notifications@github.com> wrote:

> A static class? That seems like it might be our best option. Do you think
> we should just add static members to the Map or Cave classes, or create
> new one?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kyranstar/Hunt-The-Wumpus/commit/8490809a2e8f96009788590000b605235ebd8d46#commitcomment-10703724>
> .
>


do you really want to use the old constraints format?


Hey @jleandroperez  - could we adjust the strings for `codeText` and `smsText` so they aren't split up into two portions? This causes some translation problems and ideally we should have one string to be translated.


rax retry tests


@rap1ds  Here should be [{id: 2, value: :d}, {id: 3, value: :c}], :id)


There were 1226 walking, 1073 walking-upstairs, 986 walking-downstairs, 1286 sitting, 1374 standing, and 1407 laying movements were observed.  The binary decision tree was constructed using all the features and it was noticed in figure 1 that only eight variables were used to construct the classifier.
We also observed from figure 2 that body accelerated jerk bands energy feature assumed more variation among different activities than accelerated mean in linear axes.


that comes in a later patch


We would hate the lack of space later... x]


This branch works with Boulder.  I will reorganize Registration and write test cases for a majority of the auth_handler class tonight.  I will also fix the test cases I broke while editing other classes.

There are a couple of hacks thrown in due to Boulder noncompliance with the spec and for problems with the spec, I will avoid writing test cases for these functions in order to avoid unnecessary pain... but I will note it in the code.


MacOSX local_settings.py uses sqlite, so don't see a need to install postgre


Check this out: http://forum.xda-developers.com/showpost.php?p=60095065&postcount=112


it's a little awkward but it avoids duplicating a bunch of the _set_secret guts.. :/


looks good!


So inheritance doesn't work naturally with the jose.Fields  I imagine there is some way to make it magically work, but invested too much time even tracking down the source of the problem.  These 3 fields should eventually be inherited from messages2.Challenge.


This one is ready too...


Double hash


Let us advance with pride beyond the days of "Press O"


Thanks for fixing! this one is good as well~


参考这个改一下吧，要考虑到url里传入的参数：
```
{% macro Pagination(pagination, endpoint=None, kwargs=None) %}
  {% if not endpoint %}
    {% set endpoint = request.endpoint %}
  {% endif %}
  {% if kwargs == None %}
    {% set kwargs = {} %}
  {% endif %}
  <div class=text-center>
    <ul class="pagination">
      {% for page in pagination.iter_pages() %}
        {% if page and page == pagination.page %}
          <li class="active"><a href="{{ url_for(endpoint, page=page, **kwargs) }}">{{ page }}</a></li>
        {% elif page %}
          <li><a href="{{ url_for(endpoint, page=page, **kwargs) }}">{{ page }}</a></li>
        {% else %}
          <li><span>…</span></li>
        {% endif %}
      {% endfor %}
    </ul>
  </div>
{% endmacro %}
```


Allright.


Yeah, I only came across this because I realised the user context stopped working after a Composer update and dug through the commits to find this. Perhaps some sort of exception should be thrown if user data exists in Raven and is going to be cleared as this is going to be completely invisible to Raven developers otherwise (and they will just assume they are doing something wrong).


This is outdated. Please use 'openshift admin registry --create --credentials="${OPENSHIFTCONFIG}"' instead.


That's how I got here too.


why sudo?


The purpose of this isn't to remove the data from shared user context (where you set the context on raven client and then use monolog to send records), but to keep each log entry separate from previous one. All of this code would be avoided if Raven's client would inherently understand `user_context`, but it does not, it has to be set explicitly and is always global to the client, which is far from ideal.

Sorry it broke your use cases.


GHX_Performance_Enhancements (mmatts) - i386-Windows-7-VisualStudio-12: OK (1639 of 1639 tests passed)

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


If it's our own assets, // would be great. Since this is an external asset, it's OK to use https all the time.


I can do it if you like


Would a solution be to save the data before you tamper with it, make any changes (whether a merge or a replacement -- I'd favour a merge), and then after the request has been sent reset it to how it was before?


If the saturation level never changes, it would make sense to run this code on a lot of data to get decent statistics and then save the value in the camera configuration rather than running this code every time.


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1128 of 1129 tests passed)

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


萨迪飞


撒旦


any better suggestions for how to deal w/ this?


Why is the code duplicated from lib/pf/cmd/pf/configreload.pm.

They should use the same code


Should [this](https://github.com/Flink/dokku-psql-single-container/blob/22ce918f0735fb3cb02130f83f167f329721921b/commands#L3) line be deleted?


note that these `-----------` is a markdown syntax for headings, so your last list item will be rendered as a heading.


see this :P http://review.cyanogenmod.org/#/c/77880/


my change in the code


`openshift admin registry` can check for existence as well - no need to run `osc get pods`


I was going to use that but I thought it needed the `request` to reverse it. If that works then yeah, let's use `reverse`.


GitHub poll? is this a feature request? :trollface:


Also adds the time remaining time a chest is locked for on the /cenlist screen and fixes lock time-out not showing properly on the message sent to players who just died and made a deathchest.


中介拓扑结构


Typo: "og" should be "of"


What's this?


mediator统一翻译成中介


Yes.. this is a bit of a hack since the current P4 behavioral model does
not preserve/provide device_id (or context) to the transmit callback
function. Based on ingress and egress port #, it is required to find
associated world and rocker device for further processing.

if we can provide system wide (across multiple rockers) port numbers, then
we can map system_port <--> {device, port}.
But I was not sure about the whether/how we want to support multiple rocker
devices. i.e.
1. Do we allow traffic from one rocker to another rocker ? If no, above
solution will  work.
2. How do we create internal (rocker to rocker) ports and provide
programming and packet path on those ports.
3. Do we plan to support chassis architecture (LineCard - FabricCard).

I would like to discuss/decide on the scope based on #1-3 so that we can
pick right global mappings.
#1 with traffic contained within a device is probably same as adding more
ports and worlds to a single rocker ??
#2 with no packet processing on FC will be same as #1

Let me know which models we want to include in the scope of rocker?  I need
to think about #2 and #3 a little more.

On Sat, Apr 11, 2015 at 8:07 PM, Scott Feldman <notifications@github.com>
wrote:

> This ins't going to work because ROCKER_FP_PORTS_MAX is max ports per
> rocker device instance, not global for all devices. Any global is going to
> be for all device instances, which means we'll almost never have anything
> in the global space, but rather hang stuff off the rocker-> instance.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pbhide/qemu-rocker/commit/9bdf6f6c9c4cfaa2e8127cbbf22b8864538979fc#commitcomment-10680528>
> .
>


I see at least a dozen npc with a 0 y cord in my npc list, all popping just fine. So whatever went wrong on you before, this wasn't it.


It's still complaining. Try this instead:

    local err=""

Or remove the `-u` option from `set -eu` ...


Does cref work with C# keyword names like this, or should this be Object with a capital 'O'?


Same question here; I'll stop asking about these.


Tip:
Make the commit message a bit more descriptive (but not too much). All commits are usually "more changes" right?

in this case something like `layout adjustments on wizard page` would be more appropiate.

Sometimes a commit requires a bit more of a description to explain what happens, when you have that situation, put a descriptive title, leave one line blank, and then on the third line start writing what happened and why.

This way when we need to go back in time it's easier to find changes.


Is this correct? The one liner stays nightly but the 2 line command isn't? Did rustup change here?


rax retry tests


Commit message is wrong. Meant Quick 0.3.1, Nimble 0.4.2.


这段有问题撒，是部署不是调度。你得理解一下部署这个概念


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1636 of 1639 tests passed)

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


spinner was pointless


@alaska Yep, that makes perfect sense! It's best to use raw strings for strings that do not contain \`.


see #541


Oh jeez. Sorry! I will go fix that.


2015-04-14

kmcfly

发件人：Arthur Nogueira Neves <notifications@github.com>
发送时间：2015-04-14 09:11
主题：Re: [linux] Linux 4.0-rc1 (c517d83)
收件人："torvalds/linux"<linux@noreply.github.com>
抄送：

GitHub poll? is this a feature request?
—
Reply to this email directly or view it on GitHub.


> Avoid using fully-qualified names in 'cref' attributes, since the IDE does not understand them

Is that not something we should fix in the IDE instead?


wonderful


Correct me if I am wrong, but `Propagate` seems to be used here to convert the native type to the foreign type for Glib. Would it be possible to do this conversion after the closure has returned?


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


コードに対するコメントです。
あいうえお。


Thanks for going the length and implementing a proper fix, I was just too lazy.


Thanks :)


good


If this is moved to here, then binding with *empty strings* for admin credentials doesn't work, whereas it used to bind correctly in this case (assuming of course that anonymous binding was permitted). Perhaps that was wrong behaviour though! Still, it breaks backwards compatibility.


Hi there,
DoorkeeperFor also supports the :unless and :if options. Is there a reason for not checking these ones too?


The current NUT README file is formatted in Asciidoc, so it should be possible to just rename it to README.asciidoc (or README.adoc) to have the formatting render on the Github project page. The advantage of renaming in Git is that we keep the history. I didn't want to do that right before the release, though.


A previous commit implied there was a problem with using the "T:..." syntax...?


No problem, I love working on this driver.


I personally think that when you get to the last element, the focus should stay there and only go back if one  presses SHIFT+TAB and this is why i was suggesting that we just switch the buttons so that cancel is the last


I'd say remove the "Press <Enter> to continue strings, and have two types of debuggers- stream and scalar. Stream will print and yield each element and scalar would just print and return.


btw you can follow me, because I plan to give new breath to this project. :) its seems its dead, atm and dont have even one person who can give rights to few more devs to keep the project alive.


`record.id` is addressing the `id` JSON field that's in the generic record envelope. It's not part of the tabs payload; we only know which client it is because it's part of the envelope. This is a protocol ugliness/elegance, depending on which way you look at it.


:+1: Really nice to have it explicitly stated


Yeah, it's really not bad. The complex ones will be basically a composition of this with a local resolver.


You want to call the `will_paginate` method in your view, not your controller.


In some places in previous commits, occurrences of text like this reference to TKey were changed away from using "<c>" to instead use typeparamref.


Once this split into two functions, it became much less painful to do an ordinary conditional, so this little trick is gone.


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


You don't need the `public` keyword here as it's the default scope already.


I'm going to venture a guess that this was an unintended substitution? Not everyone who installs will be named Todd :octopus:


I think we should have a little bit of documentation on available commands up on Github so we don't have to look around in the file to see what we can do.


Same question as in a previous commit about using the full "T:..." syntax here when in other commits it was explicitly being removed.  Maybe I misunderstood?


Does your `Topic` class inherit the methods from the `Paginate` module anywhere? If not, this will break.


92380320_RadianceGTest (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2174 of 2197 tests passed)

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


adding /media/ in the img src broke the image paths?
i'm putting it back.


No Translations?


@dazhangts


Yup you're completely right, apologies for this, will be rolled back to previous implementation! I appreciate you looking through the commits! The more feedback the better.


You can also send a pull if you'd like, I'm finished sending commits until tomorrow morning. I'm hoping to have an alpha ready and tested by late tomorrow.


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1639 of 1639 tests passed)

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


Oh good, I'm glad I wasn't completely off the mark. :) I'll see how i go, but I suspect that once I get the demo thing working I'll be off on other things for the rest of the day (I think we're at opposite ends of the day).


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.197](http://teamcity.monogame.net/viewLog.html?buildId=6136&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 481, ignored: 6 Build time: 00:14:36


Hello,
 The solution won't solve the OOM error.  A new CommandLine is already created every time that a FirefoxBinary is created, CommandLine creates a UnixProcess which creates a new ByteArrayOutputStream.

I see no reason to keep the output into a ByteArrayOutputStream, specially if the output is directed into another stream (drainTo).

Seems to me that the only purpose of the inputOut stream is perhaps for some debugging of UnixProcess.
Could we eliminate the method getStdOut() from UnixProcess ?

The variable inputOut could be set to System.out (so, by default printing the output of the plugins calling console.print) or something equivalent to org.apache.commons.io.output.NullOutputStream (in this case, by default the output would go into a black hole )


"We should be able to reset a signal with the value of another one" no we should not.


Propagate is just a 'newtype' that lets me give a name to a boolean so the reader knows what the meaning of true is in this signal handler.


Try to run and understand my example code.
You should see the output
```
AabbAaAaamlk
```
which basically does the whole job.
In connection with the for loop you can simplify and shorten your program significantly. I guess the size of the file will be reduced by 50% and also memory consumption and execution time will improve.
Look at http://php.net/manual/en/language.operators.bitwise.php Example 3.
Note that for example lowercase a can be represented using char code 97, uppercase A 65, and space 32.
This explains why
```
a xor space -> A  and
A xor space -> to a
```


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Except that if `c` is not in `self.clients` then we need to handle `ValueError: list.remove(x): x not in list`


this is if dont delete the make.cache file correct ?


Hey @zenazn, just wondering why you removed the web.HanderFunc and instead of an unexported handlerFuncWrap? it requires any libraries that build on goji to implement the web.HandlerFunc in their library ...


why?


I'm extremely new to iOS and Swift though, but I can try to help


C'est une erreur interne, les fatal_errors c'est juste des aborts plus jolis :)


Whom is this example aimed at? This looks like this would of more use to the user of this RP then to someone maintaining the code. If so, this should be part of the description (formatted as  a reStructuredText literal block), as the user will never see this comment.


Instructs travis to test this against the current stable io.js as well as Nodes 0.10 and 0.12.


Thanks. Would there be a use case for this in a real application? My only concern is that JDG already exposes REST endpoint where data is available under localhost:8080/rest/cache/{key} . But that is only in client-server mode. I can see that your example basically adds REST endpoint to JDG deployed as a library. Which might be useful afterall. WDYT?


This is a template of api doc. There will be one certain doc for each role of the system.


This looks incomplete.


В функции описаны аргументы, а ты вызываешь без них. что-то нужно поправить


Je suis pas fan du `must` car peut etre la bonne soltion serait de changer la signature du constructeur ou de faire autre chose.


Если у тебя есть EventOnOff, почему бы не передавать ему boolean флаг, включить или выключить?


ack


cr2? For the sake of a generic example, I would use a template that only uses Python standard library (or even just a single cell with a print "hello, world"). This will avoid the potential confusion that cr2 is required for a template.


workshopped.


the following settings?


OK pour le `is invalit Nit` mais Je suis obligé de garder le `there is` parceque dans un commentaire il peut y avoir plein de blocs de code mais un seul invalide.
Malheureusement, `markdown` ( @Morriar  ) perd les lignes originales du source donc je peux pas dire où est le block.


В собираетесь делать что-либо дальше?


je vais mettre `there is a block of invalid Nit code`. c'est bon?


s/array/list/. also, why? would pass context.result  object, which will contain everything; this will make this ResultProcessor a lot more generally-applicable.


ça me va


When maintenance window is large enough, no need to do it online, furthermore a parallel rebuild with option online can increase the fragmentation (http://blogs.msdn.com/b/psssql/archive/2012/09/05/how-it-works-online-index-rebuild-can-cause-increased-fragmentation.aspx). Therefore the additional parameters.


Что мы говорили про модификаторы доступа?


И про то, что следует следовать POJO?


Консольный клиент можно было не переписывать. Хотя, + за то, что его тоже подвязали к модели message


Yes, in fact that change to imageMarshal did break OMERO.figure, but should be fine with the fix above - Thanks.


too many spaces in front of `suggestion` ;-)


я бы настоял на том, чтобы хотя бы один из конструкторов был {{public}}.


Why not provide a basic template (e.g. that prints all collected results as a table)? Mandatory parameters should be avoided if possible.


hum it is still not good...


I guess we need to use "analogstick-left" "analogstick-right" etc. here.
That way we can also map keyboard arrow keys to analog movement?
So you can play n64 with only a keyboard - at least retroarch seems to do it that way.
What do you think?


для этого стоило переопределить equals в классе Message, тогда вся эта логика заменяется одним messageList.indexOf и contains


о том, что такая реализация equals не особо хороша, я уже писал и рассказывал.
более того, отсутствует реализация метода hashCode который как праивло определяется вместе с equals


то же самое - в List есть удаление объекта, а не по индексу. если переопределить equals, это заменится 1 строкой


I added some new patches that are needed.  I already squashed the commits (I think it is easier to review than some "!fixup !fixup" :) )


Nice work, Matt!

Instead of the console.log statements can you use the visual log we use in some other examples? The Permissions and ES6 samples should be a good indicator of how we use those.


Right.


When using the app namespace, `showAsAction` does nothing. All menu items are now hidden. (At least for me on 5.0)

This is because only with appcompat v7 `showAsAction` must be in app namespace, otherwise it must remain in the `android` one. Since I don't use any v7 Fragment or Activity, this must be changed back to `android` in order to correctly process `showAsAction`.

Is there a particular reason why you changed the namespace @friederbluemle ? Did the menu items behave incorrectly in some constellation? Maybe having both namespaces would be an option is that case.


but it's in next-css/js-setup now??


Here condition `error_code == make_error_code(writer_errc::insufficient_storage` is always true, because `has_internal_error` is set to true otherwise (at the beginning of the method):
```(cpp)
if (error_code != make_error_code(writer_errc::insufficient_storage)) {
		has_internal_error = true;
}
```


Wait let me fix this, I have some ongoing changes that I would like to propose.


не самая лучшая реализация генерации ID
сойдёт для учебной лабы, но на серьёзном проекте - не проканает.
П.С. как правило пишут какой-то утилитный класс типа IdGenerator который инкапсулирует в себе логику (хорошую или не очень) по генерации значений ID. в таком случае и ваш класс Message выглядел бы красиво, и логику генерации можно было бы более гибко переопределять


эмм, что за именование полей такое "ID" ?


Fixed issue #7


Marcel

Thanks for that.  I had not thought to use C++11 as I had not seen that the MSVC tests are doing so.  I cannot see that declared.  I am now seeing some Clang failures with Clang 3.6 and C++11 with BOOST_NO_SFINAE_EXPR

I am also building Clang 3.7 from SVN to keep uptodate.

I like to use the distribution versions and so will also build the 3.6 release.

Thanks

John

________________________________
From: Marcel Raad [notifications@github.com]
Sent: 14 April 2015 10:59
To: boostorg/phoenix
Cc: Fletcher, John P
Subject: Re: [phoenix] lambda_tests1b1r and 1b2r updated for MSVC 12 and 14 (be6cf5c)

I tested with clang-cl from SVN trunk r234404, but I guess any version in C++11 mode should do and only the combination of BOOST_NO_SFINAE_EXPR and BOOST_RESULT_OF_USE_DECLTYPE is important.

�
Reply to this email directly or view it on GitHub<https://github.com/boostorg/phoenix/commit/be6cf5c307d76bec6156115aafb67377ecc49ef8#commitcomment-10710117>.


tested on desktop, works great now.


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


Wow :-)


нужна синхронизация, мы о ней рассказывали


я бы рекомендавал унифицировать проверку на то что сообщение не было удалено ранее, т.к. при удалении у вас проверка происходит в коде класса Server, а в при редактировании - в коде класса Message. Желательно делать похожие вещи похожими способами.


@metasansana no since dokku functions are used (like `verify_app_name` or `dokku_log_info1` for example)


HashMap не хранит значения в том порядке, ка кты их туда забросил, и даже не с порядке возрастания ключа. В лексикографическом порядке хранит TreeMap, но это костыль. Ты же теряешь порядок сообщений в тот момент, когда отдаешь history.values()


Zip file of My Project


this is to create label and make later references


Rzeczywiście, da się prościej, ale nie zmieniałem tego, bo po co zmieniać coś, co działa? Ale spokojnie, poprawię.


:+1: за использование JS модулей в каком-то виде и отсутствие глобальных переменных, функций


Let me see if I've understood this correctly…

- This new `std::find` test is detecting cyclic dependencies between task batches, correct? We're about to make `batch` a requirement of `parent`, so if `parent` is already a requirement of `batch`, we're pooched - we'd be making a cyclic dependency, and dispatch would go into an infinite loop.
- In your test case, there doesn't _appear_ to be a cyclic dependency, but one is created by the implicit merging of `s["e1"]` and `s["e2"]`, since they have the same hashes.

I think we should throw in the case of detecting a cyclic dependency, rather than silently fail. Cyclic dependencies imply that either the user has built a crazy graph, or there's a bug in an `ExecutableNode::requirements()` implementation - I'd rather we were alerted to that with a failed dispatch than we just swept it all under the rug and carried on.


@vijaydev https://github.com/rails/rails/pull/18305#issuecomment-73847595


While @tobytailor pushed a follow-up that added a union type here and in other cases, I commented that out because IntelliJ became entirely unusable for me. Large parts of `lazy.ts` not being parsed is bad enough, but `nat.ts` just has to be. We can go through and add lots more union types once our tools are up to speed.


fast-forwarding 2.0 to borshop-integration-1123-feature/filter-detailed-storage-calc = 163f4927


Because it's a .js file and not .jsx


message of success not needed i think


This was a forced commit, in a first attempt git push pushed from the wrong branch.  You live and learn.


This will remove all entities, which isn't the intended effect.


There could be ... although a narrow one with keys as strings, same as in the client-server model. As we all know, the goal of Camel is to to achieve a lot with minimal coding and this example just proves/does that regardless of whether its run in Fuse or otherwise.

I chose this specific route as it provides the user an means to interact with the cache, all other routes will require additional steps to verify the cache entries. Essentially saying, I didn't give much thought to whether it can be used in a real application :). But I don't a reason why it can't be.


This one looks like the longest line added by this patch, and is about as long a line as will fit within our margins. We already have some lines this long (see `swap` at the end of [stack.syn]). I'm OK with us adding more -- if we ever move to a larger code font, we'll need to go through and fix all the long lines anyway -- but in this case I think the code would be clearer with a line break after the `&&`, because then we can make it clearer with indentation that the third `noexcept` is part of the operand of the first one:

    void swap(priority_queue& q) noexcept(noexcept(swap(c, q.c)) &&
                                          noexcept(swap(comp, q.comp)))

I'm happy with the rest of the changes in this patch.


This setting should presumably be in the other commit (or just squash that in with this).


:+1: :tada: This is the kind of contribution that makes open source great!


The next few commits are required to make this work.  I didn't realize that my ROOT installation was incomplete until I had it far enough to make this commit, and then I was working between two computers.  Next thing to learn: how to remove intermediate revisions :-)


directly throw the exception here. it makes the test below more complex to write. And error prone for the future developer that will modify it.


You should :)! Despite the minor fixes that are required, its fun to watch everything unfold as the containers start up.


I found this logic a little hard to follow. I think maybe if we renamed "requireMyself" to "preExecution" then things would be more self explanatory. We should also prefix it with "selfExecutingNode:", so this is a good example for people to follow.


error 201 is not explicit, need a comment to explain it's a functional error and that retrying 10 times to request this to Marketo won't work better.


Yes, Buckley has been adopted.

Sorry. I thought I had done it right this time. :|

-Joel Stuemke

Sent from my iPhone

> On Apr 13, 2015, at 4:42 PM, Zach Whaley <notifications@github.com> wrote:
>
> @JoelStuemke Did you plan on making this a pull request? Has Buckley been adopted?
>
> —
> Reply to this email directly or view it on GitHub.
>


模块原型开发/第一期原型(模块尚未整合完毕版)/1_2.WebSocket通信服务模块/websocket

#ifndef _BASE64_H_
#define _BASE64_H_
#include <stdio.h>
#include <stdlib.h>
#include<string.h>
const char base[]="ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/=";

char* base64_encode(const char* data, int data_len);

char *base64_decode(const char* data, int data_len);

static char find_pos(char ch);

/* */
char *base64_encode(const char* data, int data_len)
{
//int data_len = strlen(data);
	int prepare = 0;
	int ret_len;
	int temp = 0;
	char *ret = NULL;
	char *f = NULL;
	int tmp = 0;
	char changed[4];
	int i = 0;

	ret_len = data_len / 3;
	temp = data_len % 3;

	if (temp > 0)
		ret_len += 1;

	ret_len = ret_len*4 + 1;
 	ret = (char *)malloc(ret_len);

	if ( ret == NULL)
	{
 		printf("No enough memory.\n");
 		exit(0);
	}

	memset(ret, 0, ret_len);
	f = ret;

	while (tmp < data_len)
	{
 		temp = 0;
 		prepare = 0;
 		memset(changed, '\0', 4);
 		while (temp < 3)
		{
			//printf("tmp = %d\n", tmp);
 			if (tmp >= data_len)
 				break;
 			prepare = ((prepare << 8) | (data[tmp] & 0xFF));
			tmp++;
 			temp++;
		}
 		prepare = (prepare<<((3-temp)*8));
 		//printf("before for : temp = %d, prepare = %d\n", temp, prepare);
		for (i = 0; i < 4; i++)
		{
			if (temp < i)
 				changed[i] = 0x40;
 			else
 			changed[i] = (prepare>>((3-i)*6)) & 0x3F;
 			*f = base[changed[i]];
 			//printf("%.2X", changed[i]);
			f++;
		}
	}
	*f = '\0';

	return ret;
}

/* */
static char find_pos(char ch)
{
	char *ptr = (char*)strrchr(base, ch);
	//the last position (the only) in base[]
	return (ptr - base);
}

/* */
char *base64_decode(const char *data, int data_len)
{
	int ret_len = (data_len / 4) * 3;
	int equal_count = 0;
	char *ret = NULL;
	char *f = NULL;
	int tmp = 0;
	int temp = 0;
	char need[3];
	int prepare = 0;
	int i = 0;
	if (*(data + data_len - 1) == '=')
		equal_count += 1;

	if (*(data + data_len - 2) == '=')
		equal_count += 1;

	if (*(data + data_len - 3) == '=')
	//seems impossible
		equal_count += 1;
	switch (equal_count)
	{
 		case 0:
			ret_len += 4;	//3 + 1 [1 for NULL]
			break;
 		case 1:
			ret_len += 4;	//Ceil((6*3)/8)+1
			break;
		case 2:
			ret_len += 3;	//Ceil((6*2)/8)+1
			break;
		case 3:
			ret_len += 2;	//Ceil((6*1)/8)+1
			break;
 	}
	ret = (char *)malloc(ret_len);
	if (ret == NULL)
	{
		printf("No enough memory.\n");
		exit(0);
	}
	memset(ret, 0, ret_len);
	f = ret;
	while (tmp < (data_len - equal_count))
	{
		temp = 0;
		prepare = 0;
		memset(need, 0, 4);
		while (temp < 4)
		{
 			if (tmp >= (data_len - equal_count))
				break;

			prepare = (prepare << 6) | (find_pos(data[tmp]));
 			temp++;
 			tmp++;
		}
		prepare = prepare << ((4-temp) * 6);
 		for (i=0; i<3; i++)
		{
 			if (i == temp)
				break;

			*f = (char)((prepare>>((2-i)*8)) & 0xFF);
			f++;
		}
	}
 	*f = '\0';
 	return ret;
}

#endif


It produces notices, so probably - yes, but I doubt that maintainer is interested in his repo. The last commit was made more then a year ago..


When `RT_USING_COMPONENTS_INIT` is defined, `LINKFLAGS` will be over-written and ` --keep __rtmsym_*` will lost. I have a fix for this.


From email to @psiorx:

> most of the zero radius spheres in the v3 model are from the vrc era and are unused.

I think we don't need all these extra collision geometries.


add a "opacity" mode for space that made those backgrounds wall lucid


Can confirm @belal-almassri-spongeuk has tested this +1


This closes #13.
2 tests are still failing, I've left them there to remind us of things to think about...


Personally I prefer https://github.com/waterlink/rspec-json_expectations for making expectations on API responses.


The logic here confused me a little at first, but I think it's actually spot on. What I found odd was that the preExecute task needed to be the one that called the base class requirements, and I was surprised that we had to delete the context entry. But it's obviously correct - without doing that, the upstream nodes can be executed in parallel with the preExecute tasks. I think a little comment describing the necessity for this logic would be good, just so this can serve as an example.


Why not use tempdir() as the default instead of mandating that this is set?


fixed the display problem for long length  axis label or coordinate values


what?


I copied it word by word from the sqlite result_processor.  What would be a better wording?


I havenʼt heard about it. Thankʼs for it :ok_hand:


for both of these checks maybe automatically set the dependant setting  if it is None and only compalain if it has been explicitly set to False by the user?


Do we need here new lines as well as we had with f659f6abcda44597b6a6dd4818e0b17b592647ce


I would add this into the patched CDC file.


As far as I can see, the description is shown by "wa show", and that doesn't understand reStructuredText literal blocks.  What am I missing?


fix the Automatic zoom for nagitave X and nagitave Y.
i.e: if X∈[-10,-20],Y∈[-100,-200]
former vasion will produce a [-20,0]X[-200,0] grid
now it changes to a [-20,-10]X[-200,-100]


OK


From or from user?


Ok, I'll create a more generic example


I meant ceylon/ceylon-compiler#566


No, use connect and let hooks sort it out


Then all the call sites don't compile with:

`could not find implicit value for parameter ee: org.specs2.concurrent.ExecutionEnv`

Where should the implicit then be coming from?


OK. I'll try it out:)


sorry, i included local changes that were not supposed to be comitted


Bug fixed  https://github.com/ongakuer/CircleIndicator/issues/12


ok


Just remove that entire phrase -- generated documentation will automatically list the parameters.


Where should I put it? in wlauto/results/ as well?


"wa show" doesn't (yet); but generated html documentation will do.


meslekler  yanlis yazilmis


This doesn't do what we need: if a peer has lost contact then Peers will garbage-collect it and hence be unable to recognize the name.


:ship:


Bi dahakine direk master'a yollama mahir kendi branch'ini ac sonra bitirincede pull & request olustur biz onaylarsak merge ederiz kucuk hatalarida pull & request'te duzeltiriz hem.


Why does this proposal cover only cover for comprehensions, and not patterns in general? Left hand side of the for comprehension is just a pattern. Wouldn't it be more uniform to make `implicit` an acceptable modifier on binding pattern? E.g.

```scala
  Foo(1, 2) match {
    case Foo(implicit a, b) => ...
  }
```
Quasiquotes love consistency and scream and cry where there is none.


On Mon, 2015-04-13 at 22:41 -0700, Nicolas Hafner wrote:
> I'm not particularly happy with this commit, to be honest. I would like to retain the ability to store an entire CSV file in memory, rather than requiring it to be mapped manually in some fashion. The calling convention you came up with for the header and body separation also seems quite awkward to me.
> I'll think of a solution of my own.

Fair enough.

The ``(<event> <data>)`` signature was loosely inspired by event-based
parser interfaces, albeit admittedly being a poor person's version, but
see below.

Please consider the following suggestions for your new solution:

      * If possible, do not read and regex-split the whole file in
        memory. I don't know how big these files can get, so constant
        memory complexity may be necessary at some point.
      * Consider a builder protocol for decoupling parsing from result
        construction. Maybe the architecture.builder-protocol system [1]
        is to your liking.

If I can advise in general: do not put too much effort into this system.
The use-case of converting Vicon textfiles may arise again, it will
probably not be a very common one, though.

[1] https://github.com/scymtym/architecture.builder-protocol


Because this is where the user is running "ipython notebook".  Maybe I should reword this to clarify it.


If these are deprecated, they might as well be removed. We got a history of them on git anyway, if we ever require them again for whatever reason.


IPAM is, so far, independent of the details of Router. If we need some further access then it should be behind an interface.


put it into the processor's directory wlauto/result_processors/ipynb_export/notebook.template (and rename this file to __init__.py in the same directory).

You can then load the template as

    context.resolver.get(File(self, 'notebook.template'))

(side benefit is that it would then be also possible to override the default plate with ~/.workloa_automation/dependencies/ipynb_exporter/notebook.template).


std::list<const Entity*>::iterator itr;
for (itr = m_entitiesToDraw.begin(); itr != m_entitiesToDraw.end(); ++itr)

||

for (auto itr = begin(m_entitiesToDraw); itr != end(m_entitiesToDraw); ++itr)

...poza (za) pętlą itr nie jest już potrzebny.


asd


qwe


as


sa


\o/


take this file


Warto upraszczać i zwiększać czytelność gdzie się da. Wyobraź sobie, że musisz wrócić do tego kodu po roku przerwy, albo jakiś inny programista musi zmodyfikować ten kod. Im prościej i czytelniej jest on napisany tym łatwiej będzie to zrobić.
http://en.wikipedia.org/wiki/Code_refactoring


Ah right, ok. In that case, yeah, this needs to be mandatory as we can't assume a reasonble default for the server dir.

Yeah, I think description needs to be clarified to indicate that this is the directory served by the notebook server.


It listens if a method _has been called_ on it. In other words, you can use rspec `have_received` helpers on it. Be as it may, it caused problems so I changed to ordinary `expect sh to receive` syntax.


##何の事は無いCの**共用体**のニホイがするではないか。


It's a quote from Johannes' posting to Google Groups, linked below. I've moved the source above the quote, so it's clearer where it comes from: https://github.com/git/git.github.io/commit/4a48c04372bb35a8e70db6ca9da2b276e9033438


What happens if this blocks, and somewhere on the actor side is a call that sends to resultChan?


To prawda, po prostu umysł miałem zajęty rozwiązaniem problemu deformacji twarzy i nie zastanawiałem się nad dopieszczaniem kodu.


This message doesn't seem appropriate in all cases.


Ok, spoko. Ja nie mówię, że to błąd, albo że masz to poprawiać. Po prostu rzucam luźne, ogólne uwagi.

Again: nie musisz wprowadzać do kodu wszystkiego co Ci tu zrzucam. Twój projekt - Twoja decyzja. Simple as that. :)


Оу, что это за прямые коммиты в v-ветку???


So, every time you want to consume a Petals service with Camel, you have to describe it in the jbi.xml?


Eklemek istediginiz yada eksik olarak dusunduklerinizi soyleyin eklerim onlarida. Yada siz de ekleyebilirsiniz :)


What worries me is why we're getting empty package names....


@Synchro

Change in phpdoc does not match `\POP3::popBeforeSmtp` mehod signature - `$tval` is still there


Nope. Check is not needed.


Yep, that's exactly the idea: we decided that to follow the JBI philosophy, a SU that uses a service on the bus must declare it in the jbi.xml.
We could imagine to refer directly to petals services from camel using some kind of URI like "petals:interface[:service:[endpoint]]" but for now, we don't support it.
What do you think?


ok


TODO:
Create VRT._add_mask() to be used in mapper. Mappers add a VRT with mask (so that it is not visible as a band but is used in __get_item__ if exists.


Thanks for the comments! Take a look at the updated version.


I can force-push that change now, or I can put it into the next PR that I've lined up already. Preferences?


If it's easier to split that into a separate change, I'm fine with that.


:angry:


I've removed the analogous line from  sqlite description as well. Thanks for identifying that.


Я обновил ДЗ по loops и logic.

2015-04-14 12:28 GMT+03:00 NickLukianenko <notifications@github.com>:

> on line 47 - to make this more readable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pavliukStanislav/csharp-lectures/commit/442a377b0a8d3e1236aa10a9ee5762dc6e32e999#commitcomment-10709512>
> .
>

--
Sincerely, Pavliuk Stanislav.


It was the way the EIP component used to work.
I think this is redundant and time-consuming to declare consumed services in both the jbi.xml file and in the Camel routes. One of the interest of Camel should be to reduce the jbi.xml file to its minimum. And with respect to the EIP component, it would be a huge progress. And the JBI philosophy worked so well almost no one use it anymore. People expect things to be as simple as possible. So, rather than considering the JBI philosphy, user experience should be given priority. And duplicating information is not the best option IMO. :)

I agree that the URL option would be better.
From memories, it is what was done for Service Mix.

Also, I do not understand why the Camel route is not declared under the provides section.


There is a bug with `this.remove` of estraverse which throws an error if you try to remove an expression without removing it's parent (it should automatically remove the parent). I've filled an issue about it: https://github.com/estools/estraverse/issues/47


Now Its perfact :+1: . I think i get idea about Url routing :)


test


Исходя из того что виджет могут использовать разные разработчики, и вероятность использования этих стилей в проекте высока, хотя-бы фонтасом часто используется, то думаю лучше использовать бауер пакеты, вместо CDN так как это позволит избежать конфликтов между скриптами, и позволит использовать одну и ту же версию для всего проекта.


Тоже самое что и с стилями. лучше `bower`-ом загружать и использовать оттуда все.


c'est fait

2015-04-14 12:47 GMT+02:00 Maxime Teneur <notifications@github.com>:
> Merge pull request #90 from mtisserant/inscription-en
>
> Ajoute la page d'inscription en anglais.
> @omansour tu fais la MEP cette aprem ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/afup/web/commit/42dbbdfd4a7697a57c37bac6fdec1459f2a7d34c

--
Olivier Mansour
http://oliviermansour.com/
http://fr.linkedin.com/in/omansour


`throw` пишутся отдельно от `return`


Le 14/04/2015 13:04, Vincent Zurczak a écrit :
>
> It was the way the EIP component used to work.
>
If you look at the SUs generated for the EIP component, the consumes are
declared in the jbi.xml, even though they are generated by Eclipse so
that the user does not have to.

> I think this is redundant and time-consuming to declare consumed
> services in both the jbi.xml file and in the Camel routes. One of the
> interest of Camel should be to reduce the jbi.xml file to its minimum.
> And with respect to the EIP component, it would be a huge progress.
> And the JBI philosophy worked so well almost no one use it anymore.
> People expect things to be as simple as possible. So, rather than
> considering the JBI philosphy, user experience should be given
> priority. And duplicating information is not the best option IMO. :)
>
You don't duplicate the information: you assign an id to a consumes
section, and you refer to this id in the route: if you change the
consumes, you don't have to change the id, and thus the route.

> I agree that the URL option would be better.
>
I agree to that to, but for now we decided not to, but in the future we
may decide to change it. Maybe we could open a jira to discuss it
instead of here?
>
> From memories, it is what was done for Service Mix.
>
Well, they completely abandonned JBI so it makes sense ;)
>
> Also, I do not understand why the Camel route is not declared under
> the provides section.
>
Because a route is associated to an operation, not an interface. So the
association is done in the wsdl as a (proprietary, not well documented)
binding extension (in the same way as soap).
This is documented here:
https://doc.petalslink.com/display/petalscomponents/Petals-SE-Camel+1.0.0-SNAPSHOT
And the use of the wsdl to declare things related to operations is
discussed here in more general terms:
https://jira.petalslink.com/browse/PETALSDISTRIB-118

> —
> Reply to this email directly or view it on GitHub
> <https://github.com/petalslink/petals-camel/commit/d1ab2052cb5cab674a4f504e9edfa4be431cf452#commitcomment-10711092>.
>


Если можно то было бы хорошо добавить в двух словах описание параметров


The arguments are wrong for the remove method here.


yep, I noticed, we're in c++11 after committing


This is [unord.req] not [associative.reqmts], I'll fix the commit message before merging.

I'm a little unsure about the changes like "[i, j) denotes a valid range" because I think what we're trying to say is that i and j denote iterators, _such that_ [i, j) is a valid range, rather than talking about what [i, j) denotes (we already know what it denotes, it's the half-open range from i to j). But the current wording doesn't really achieve that either, so this isn't


フォーク元への取り込みに伴い、1f764cf でrevert


I think this should be on the document object itself intead.


* Förstår att `wkhtmltopdf `är namnet på biblioteket, men det är jäkligt jobbit att läsa :) Skulle vara trevligt att köra camelCase eller något, men också trevligt att använda biblioteketets namn. Trade-off antar jag.

* Varför är det så mycket newlines här: https://github.com/jcmandersson/Operation/commit/e36b78d8e4f46b3fdca1ee3fac5cb8e994dbda5b#diff-198da736874dc99cf4d98bbfea07062aR12 ?

* [Magic constant!](https://github.com/jcmandersson/Operation/commit/e36b78d8e4f46b3fdca1ee3fac5cb8e994dbda5b#diff-8fa43110d61bb8b945a61fb293418667R75). Skulle vara bra med förklaring av vad den kommer ifrån. T.ex.
```
var seconds = 60;
var minutes = 60;
var toMilliseconds = 1000;
var oneHour = seconds*minutes*toMilliseconds;
```
och sen
```
... setInterval(..., oneHour); ...
```

* Stilguiden skriver inte någon om det explicit, men om man följer deras exempel så är det ett mellanrum mellan ``//`` och själva kommentaren. Alltså
```
// Kommentar
```
inte
```
//Kommentar
```
Detta sker mycket i pdfCreate, t.ex. [här](https://github.com/jcmandersson/Operation/commit/e36b78d8e4f46b3fdca1ee3fac5cb8e994dbda5b#diff-8fa43110d61bb8b945a61fb293418667R29).

* Annars bra!


Thanks for the update. Still got errors after the domain was changed. Tested the new domain through the browser and it does not seem to work directly either.


Misplaced `:` (see also other places).


Ser att något jag kommenterade fixades här :) Ignorera!


J2me/MANIFEST. in


The version here and in all sub-projects should be changed to 6.5.0-redhat-SNAPSHOT


You can define a client only mixin block in the json config, see [MixinConfig:130](https://github.com/SpongePowered/Mixin/blob/master/src/main/java/org/spongepowered/asm/mixin/transformer/MixinConfig.java#L130)
`"mixins": []` is both
`"client": []` is for client-only
`"server": []` is for server-only


`everyHour` kanske är bättre namn än `oneHour `


commenting


== 0


Нет проверок на наличие ключей массива. Хотя думаю лучше разделить все на параметры метода, и ничего подобного делать не надо будет.


Note, ensuring all xcb data is freed is a bit tedious.
Would consider using `goto finally` instead of returns in middle of some functions. Then free all allocated data there. Else it makes leaks easy to introduce, especially when moving blocks of code around.


please do
pip install pep257
pep257 <file>


Could we also rename it to camel-jbossdatagrid-fuse please? The component is called camel-jbossdatagrid so it would be good to have it consistent.


This change is designed to allow users to query the `isLogoutReceived` flag from within a `toAdmin` callback. The callback is called from within the `generateLogout` method (circa line 1304), so we are ensuring the flag is set before that code path is entered.


Good catch — it shouldnʼt be there at all.


好像之前改的漏了一个地方，应该是
有可能路径1...
少了个“可”字


Лучше использовать: http://www.yiiframework.com/doc-2.0/yii-helpers-basefilehelper.html#getMimeType()-detail и удалить ненужные строки вверху.


Read for Download Path


vital を使った表記に修正


*upgrade* saja bagaimana?


GaoXiaoL


based on my understanding, the crash happened in libavutil module of ffmpeg. however the fatal signal was reported by libc (bionic). so, it should have nothing to do with this commit.
i assume you were playing video or audio when the lib crashed and ffmpeg must be used to decode a particular codec. based on the backtrace, the last place that invokes ffmpeg is the libstagefright through sniffer function.
i wonder whether the last two commits in frameworks/av causes the crash that you experienced on hammerhead.

other than that, afaik, there is a configuration file media_profiles.xml (or something, can't remember the name) in the device/device common tree that should define which codec should be decoded with ffmpeg. my assumption is that particular codec cannot be decoded with ffmpeg. i might be wrong.

for tracking the culprit, i'd start with reverting some commits in frameworks/av.


すいません、ここlocalで使ってたときのままのコードになってました。修正します


Замечательно.


gbgbnb


it would be great to define a base currency for field and let the orm to do automatically conversion rates.
Great improvement!


By the way, libtimezonemap and hwinfo are no longer needed. But wait some days before removing them :grin:


@jeffposnick In the past I've noticed us using IDs over classes for most samples. This isn't a deal breaker here at all, but just calling out in case you had any opinions.


Same console.log comments as earlier. I'd recommend opting for https://github.com/GoogleChrome/samples/blob/gh-pages/collections-iterators-es6/index.html#L68


Selain dari itu silakan buka PR dan langsung merge ke master saja. :+1:


OK


@rattoma I am not sure your fix is correct. You simply append the saving of ```stock_logpo``` and ```stoch_ys``` to ```irun(5)```. But the condition for switching to a different matrix after saving the previous result is conditioned on ```MAX_nruns```. This variable is computed in ``` prior_posterior_statistics ``` based on the number of parameter + 2 (size of ```stock_param```) in order to achieve a file size of ```MaxNumberOfBytes```. Thus, at a minimum, you will be creating a really huge file as you are appending an additional variable with size ```M_.endo_nbr``` to a file of already prespecified size (particularly  as ```M_.endo_nbr``` is often larger than ```npar```).


Thanks for identifying these, Gloria has committed the changes.


Ну тут мелочь, докинул недостающий файлик.
А так больше нельзя делать?


redundant


and how about Windows 8.1 ?


Oh i just figured it out !
Thanks for the review :rabbit:


is not all for skill ID, missing only 3 races for me to complete the full commit, I will give my repository to merge after


what's this about?


Yeah, saw this blog post: http://www.phpied.com/smush-it-is-dead-long-live-smushing/. I'll see if I can setup our own hosted version instead that we can use.


Why not put `(` after `e`, rather than before `J`? It's confusing for me and less chainable.


Sorry. L884 is redundant.


@prudhomm That can not work (at this point) with [Travis](https://travis-ci.org/feelpp/quickstart/jobs/58413883)


I'll take a look--did it in a rush.  Thanks.
________________________________
From: Kerry Jiang<mailto:notifications@github.com>
Sent: 4/13/2015 11:56 PM
To: kjtags/WebSocket4Net<mailto:WebSocket4Net@noreply.github.com>
Subject: Re: [WebSocket4Net] Added close code and reason properties. (0c604b4)

Hey sir, it seems you didn't handle the case the close reason is sent from server side?

—
Reply to this email directly or view it on GitHub<https://github.com/kjtags/WebSocket4Net/commit/0c604b426033096390f166e978fa16914902e59e#commitcomment-10706053>.


Feel free to change it. The CALLPROC/CALLFUNC functions are only for interim until the game is fully reversed and since they haven't caused issues so far there isn't much need to change it.


Is there any value in us demonstrating feature detection for copy/paste support here?


Probably it is correct, but I have made my own change. You can take a look.


@n9iels How did you produce this commit? It looks awful in Github.


Realization possibility of deleting models and showing list of models.


This is working for me when I come in and out of the browser, though I'll keep a close eye on it!


Add function to them here.


hm, what happens in case a developer compiles in release and makes a mistake :)?


I personally don't like this prepend double colon style, if we use just `sio::socket::ptr`, did it resolve your problem?


what caused the crash, btw?


workaround the reload bug


As I understand this one, it's to avoid a little code duplication, so assigning 'user_id' within the brackets to use on the following line (so saving a repeat of 'session[:user_id]'). I've used this from a user auth method I learned a while back, though if it's bad form (and it is a little unclear), I'm happy to take your advice and alter!


dlopen support having NULL as filename (from man page).
>
>If filename is NULL, then the returned handle is for  the  main  program.
>

Add a null check to avoid the problem:
<code>
 if (filename != NULL && strchr(filename, '/') != NULL) {
</code>


@SDKiller I just pushed a fix for that.


@ldoktor the point is that we have two deadlines: the test's own and the one introduced by the job.


I've got a couple of the methods within the helper being used in controllers as well as views. Is this OK or should I be looking at storing them elsewhere?


I already tried but got in a fight with gcc about unsatisfiable
constraints. Here, the compiled code looks okay anyway. It's a nice dirty
stopgap. :)
Op 14 apr. 2015 13:30 schreef "Duncan" <notifications@github.com>:

> Feel free to change it. The CALLPROC/CALLFUNC functions are only for
> interim until the game is fully reversed and since they haven't caused
> issues so far there isn't much need to change it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IntelOrca/OpenRCT2/commit/fc5ff967410257e25b056e78a8bc709b84f3ab1c#commitcomment-10711518>
> .
>


Trying the quickstart, I see "Installation successfull". Then I put data by the echo command and later go to the browser and ask for http://127.0.0.1:8282/cache/get/1 . But I don't get any response - the webpage is not available.


OK - I see. if user_id cannot be assigned to session[:user_id] then this will return nil and the if statement will fail. That's fine


Sorry - thought I'd already done that. Updated now! I've noticed I formatted the italics wrong, so I'll correct that for the next commit.


ahhh now I understand it .. every song holds zero as default and if you listen every song minimum once you'll get 1 for the album, and if you listen every song minimum three times you get 3 for the album ;)


what the photh


lgtm 961d6d1d49b38a34f53e5971c31fb10318086378


yep. we decided that trying to enforce the non commercial clause was impossible, and that we cared most about having code changes released etc.


Please define awful? I dont see anything obvious


@varyform Can you provide the `fog` gem you are using, your `carrierwave` config and the error you are having so we can know it better?
Please, open a issue with that feedback. I will happily fix it. =)


Check this indentation


here is the error, should be :

```CSS
color: rgb(40 ,40 ,40 );
```


@yagudaev whats the reason for this line?


Será que não era bom o Arquivo saber os Registros que ele deu?
Até pra evitar o usuário de abrir uma quantidade arbitrária de Registros


The difference is between methods that are available in both the controller and the view. I tend to go with putting methods directly into the application_controller and if they are needed as view helper method you can add the following for each of them.

```
helper_method :my_method_to_call_from_view
```

You can always extract methods out of application controller later by creating a module that you then include in the class e.g.

```
include Authentication
```


use link_to rails helper here as in product the localhost:3000 might not be available

```
<%= link_to 'REKKI', root_url, class: 'navbar-brand' %>
```


method has side effects. renamed to mergeConstraintsAndRelations


lgtm 5250b3def38926208b35d5ce0453d72d47e9c69c


indentation


:question: see also `npm run build-dev` right below, we need to minify on a standard build.


This is a rails javascript adaptor - it's very important if you are going to do anything with AJAX


Strictly, the user will only see this for outstanding requests when the allocator is shutdown.  I'll change it anyway.


my file "20150414112202_add_recipe_id_to_sequences.rb" and yours "20141214171606_add_wall_id_to_posts.rb" are different, you have an index and a table.

what I am missing?


I haven't tested it, but I'd assume this should work:

```
FROM prom/alertmanager
USER root
RUN useradd -d /home/alertmanager -m -u 109008 alertmanager
USER alertmanager
```
Beside that, I agree with @stapelberg that we should run all other prometheus services as non-privileged user at least if it's not a blocker for specific user cases.


timeout在“ s.connect(hostname, 22, pkey=key, timeout=timeout)”这里已经定义了。


of course. Using ```npm run start```.


s/receieved/received/ (you'll often see this syntax in comments/StackOverflow/chats/… `s/old/new/`, which is [coming from vim](http://vim.wikia.com/wiki/Search_and_replace), to suggest an edit).


then I'd vote for job_deadline


Can these 5 implicits be deduplicated?


In principle, you could reify an array of characters as a string. The footprint will be much slimmer.


Seems perfect! You may have delayed committing this piece of code, though, to create a dedicated commit for the downgrade *feature* as a whole (routing + controller action + view/button [+ specs]).


This removal makes the project crash on line 64.


Alot neater !


Let's use `Vector[Token]` to be consistent. I wish back then I introduced a dedicated type `Tokens` just for this purpose.


a lot of planet   我觉得这里是原文笔误。在这个游戏里是飞不到别的星球去的。
可能指的是地球上的很多地方。

不过既然原文如此，先这样了。


I wonder whether you can do:

``` ruby
resources :subscriptions, only: [:new, :create] do
  get "downgrade"
end
```

It may work, needs to be tested.


A good procedure (which I need to document) for verifying that a configuration change does no harm is this:

<old configuration>
compare_codecs --score openh264
take_snapshot
<new configuration>
verify_scores --score openh264
verify_scores openh264

(the last one just verifies the current scores against the ones saved with take_snapshot, so that you get
the list of improvements/non-improvements without all the encoder traces.)

When I run this on your patch, I get:

Change evaluations:  {'worsened': 1, 'improved': 40, 'no change': 1}

The one that got worse:
openh264 rate 2000 file BasketballDrill_832x480_50 id d41d8cd98f00: worsened score from -98.306 to -98.847

(this probably means that it's slightly bigger than before; I regard overshooting the target bitrate as something to be detracting score for, and most of these do).

I'm accepting the patch.


It's a bit of a stretch to use `Parse[T]` here. I'd suggest to simply write down the code for quasiquote boilerplate manually instead of using `@quasiquote`.


in reply to [above](https://github.com/holodex/app/commit/021a40088aee612385ea011fd10f068f138118ba#commitcomment-10711868), this is the workaround that probably fixed the infinite reload issue. in my opinion it's messy to add these hacks (including commenting out debugs) to a commit called "move getDefaultLinkTypes", i usually just leave them in my working tree uncommitted until i find a more proper solution.


It would be nice if we could inject `toks` into the scala.meta package object, but for now we'll have to live like this.


:D


抱歉圈错人了，提醒的是@KrasnayaPloshchad
Sorry to mistake user id. I want to @KrasnayaPloshchad


Nice. Be aware here that's a strong UX choice you're making, about the default value for a wiki's state: is it public or private?

I mean that you could also reverse the UI to display a non-checked checkbox with the label "Make this wiki private", assuming a wiki is by default public, which is consistent with your `Wiki` model (where the `public` boolean attribute it `true` by default).


Thought these had been fixed with my new Sublime config! Weirdly, this one doesn't show on my comp.


My bad - I dropped it in temporarily when testing the sessions / cookies. I'll change now, thanks for flagging.


I think that this would cause problems if `interactables.get(curEventTarget)` return an interactable but there's no action for it.

This might work better:

```javascript
// if there's a target but no action is prepared yet
if (target && !this.prepared.name) {
    action = action || validateAction...
```


Patch 850 disable frontend sleep on close, 851 disable RC pooling. This important for OpenWrt.


    wvars['entries'].sort(key=lambda entry: (entry.getStartDate(),
                                             entry.getFullTitle() if hasattr(entry, 'getFullTitle') else None))


I've realised why I did it - I removed it instead of UI by mistake. Well, it's back now. Good spot. (I don't have the emojis, but if I did it would be the 'facepalm' one.)


Bye bye MagicMarker - always made me smile.


@khipkin I was considering it, but `calendar` seems to get caught in the generic URL pattern, and I'm not sure if we went ahead and handled it differently for a reason. I will test it and let you know. Thanks! :palm_tree:


You mean having only something like
```scala
implicit def liftBool2T[T]: Liftable[Boolean => T] = Liftable[Boolean => T] { f =>
  val tpe = weakTypeOf[T]
  q"(x: $tpe) => if (x) ${f(true} else ${f(false)}"
}
```
?
Wouldn't I need an implicit `Liftable[T]` to do this ?


I am a iphone app developer,when i use RSA to encrypt myapp,need to dump the PEM from the keychain.i found your project (Print private key and certificate in PEM form in 2015) in github,when i compile the code get some errors, maybe the Development Environment make it.can you send you binary(main.m) to me or post it to you github. THANK YOU that will help me lot.


I *think* what we really want is `printf "%s.%s" "$(git rev-list --count HEAD -- "$startdir")" "$(git rev-parse --short HEAD)"`, right?


We need to use spread operator to send everything to the client so 3rd party libraries can hookup to tracking calls.


my email is imjohnsmith001@gmail.com.thank you.


Damn, yes I did it too quickly :/


I don't understand. `trait Input` must have `val content: Array[Char]`, doesn't it ?


Please move all this to a separate file (treasure.coffee) -> tracker should remain generic.


It's the same code


Heh.


Then you'll say `implicit def liftBool2T[T: Liftable] = ...`


Actually, scratch that. That would give us the number of commits of the `mingw-w64-asciidoctor-extensions/` directory as well as the short commit name. But what we really want is the number of commits of the `asciidoctor-extensions` repository... So:

```sh
printf "%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)"
```


Sure. You could say `val content: ... = ${new String(input.content)}.toArray`.


Myślę, że można wywalić te pierwsze dwa zadania w takim razie :).