I think how `CompactFactorGraph` and `InferenceResult` (or `LearningState` or maybe just `Weights`) are resumed should be fully under this function's control, not hidden in `dw`.  Resuming assignments for each sampler may be better handled inside `dw` with a given dir path.


Similarly, how `CompactFactorGraph` and `InferenceResult` are checkpointed should be this function's responsibility.  dw may still checkpoint the state (assignments) of each of its samplers.  Moreover, these boolean flags are easy to write but usually very bad for understanding.


Another very minor thing: commit messages should generally have max 50 characters on the first line, which you should consider to be like the subject line. Then a blank line, then you can have more detail organised however you like, with each like max 72 characters.

If you just use git commit rather than -m it'll prompt you in a text editor for your message.

Might be too picky, so ignore if you want to


Hello @jamielinux

Additionally, the GPG key you you ship with in your github repo cannot verify your Redis package. What would be the perks of using jamilielinux-redis vs Redis package already available in the EPEL repos?

Regards


Thanks for the info. I need to think it out.


From/to must be always existent, so there is no need to create.
But I tried to be consistent because have a look to function load_from() or load_to().

What I tried to fixed there,
Is that for DoS and PBF rules there is another child under from/to in the XML file.

Is it better to use this:
$tmp = DH::findFirstElement('from', $this->xmlroot);

I followed this->load_to() / this->load_from()
And copied this here, was only a try to be consistent.

/**
 * For developper use only
 */
protected function load_from()
{
    $tmp = DH::findFirstElementOrCreate('from', $this->xmlroot);
    $this->from->load_from_domxml($tmp);
}

From: Christophe <notifications@github.com<mailto:notifications@github.com>>
Reply-To: cpainchaud/pan-configurator <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Monday, June 20, 2016 at 1:47 PM
To: cpainchaud/pan-configurator <pan-configurator@noreply.github.com<mailto:pan-configurator@noreply.github.com>>
Cc: Sven Waschkut <swaschkut@paloaltonetworks.com<mailto:swaschkut@paloaltonetworks.com>>, Mention <mention@noreply.github.com<mailto:mention@noreply.github.com>>
Subject: Re: [cpainchaud/pan-configurator] fix for <from>/<to> extraction for DoS and PBF Rules (b56b2cb)

why create 'from' and 'to' if they don't exist @swaschkut<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_swaschkut&d=CwMCaQ&c=V9IgWpI5PvzTw83UyHGVSoW3Uc1MFWe5J8PTfkrzVSo&r=Jb4dHoFDiFQrZ-z3V4d67X3hMAoYOJl734q7LTcgj6k&m=i-kxUF9rkZUzjc7JTV0NoZNERT1tzHvJU3r2RwxjAfY&s=SsUeCob7oj0P_vQe6aQzboU4q-hidDy81trudrXuBqs&e=> ?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_cpainchaud_pan-2Dconfigurator_commit_b56b2cbf390ac57a7cdde17833396ad24267ff39-23commitcomment-2D17932475&d=CwMCaQ&c=V9IgWpI5PvzTw83UyHGVSoW3Uc1MFWe5J8PTfkrzVSo&r=Jb4dHoFDiFQrZ-z3V4d67X3hMAoYOJl734q7LTcgj6k&m=i-kxUF9rkZUzjc7JTV0NoZNERT1tzHvJU3r2RwxjAfY&s=RMfTr7SLy6ax1HdQ2uOGtV1i0WmB7j96n3jLwQipCsg&e=>, or mute the thread<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe_AP5FvfRJ7IhlmgEuSThCIu8eG12GSWWnks5qNn3GgaJpZM4I5nqB&d=CwMCaQ&c=V9IgWpI5PvzTw83UyHGVSoW3Uc1MFWe5J8PTfkrzVSo&r=Jb4dHoFDiFQrZ-z3V4d67X3hMAoYOJl734q7LTcgj6k&m=i-kxUF9rkZUzjc7JTV0NoZNERT1tzHvJU3r2RwxjAfY&s=gidhAkG5J5A57lYNOgJJEt4NyqikYVa1XHcyWNUBQnw&e=>.


Use RAII: you can just say `infrs.load_assignments(std::ifstream(opts.snapshot_path + ...))` if it accepts a `std::istream&`.


pic


jepp. good catch. fikser


import System.IO.
So the code will be
File.Delete(attachmentFilePathAndName);


always put js at the bottom


Ja so hatte ich das gedacht, ohne Eintrag ist das Schild selbst die Teleport Position, mit einem Interact oder jedem anderen Trigger kann man sich so zu dem Schild teleportieren. Sonst kann man noch Koordinaten angeben, aber das braucht man eigentlich nicht.
Vielleicht müsste das etwas deutlicher sein


No worries.  Might have been the zip download from GitHub or unzip on my mac that did that....


Let's remove this.


This change seems break the assumption in `bin/dotc` [here](https://github.com/lampepfl/dotty/blob/master/bin/dotc#L24), which makes it impossible to use `bin/dotc` or `bin/dotr`.

The assumption in `bin/dotc` is very brittle, which is better to be avoided.


Yes, ok. The theme 18.03 now has patched-back version of `bookmark.cgi`.

Jamie, did you do aby changes in regard with wheel users? With the latest Webmin wheel users can't login anymore but on 1.7xx could, right?


@DrianHillman good good. 👍


time +dt


intent(in)


TeamCity Glyssen :: Glyssen pull requests [Build *.*.112.3451862d42de7f79dc9493d546d131493bd4aac6](http://build.palaso.org/viewLog.html?buildId=73860&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1256, ignored: 17 Build time: 00:22:54


Resolves gradle/gradle-script-kotlin#78.


I like this change!


Not sure what happens next @vexed87 ?


this static is very problematic because each editor (when you have more than one side by side) gets his own editor action associated. but with this static, the state is shared across all editors. this is causing https://github.com/Microsoft/vscode/issues/7882


@svenmuennich Sorry, I am not able to reproduce this. Please note that we use a customized version of the `AbstractProxyFactory` (https://github.com/shopware/shopware/blob/5.2/engine/Library/Doctrine/Common/Proxy/AbstractProxyFactory.php).


https://pypi.python.org/pypi/mycli/1.6.0 says mycli is py3.5 compatible. Maybe it's just the tests that don't work?


Yes after 2.7.1 new filter defaults I had a lot of messages / complaints
from folks with noisier setups. Mostly the ones with DAL props I noticed.
So I just made the defaults safer and dterm to 50 seemed to help those
folks.

2016-06-20 13:29 GMT+02:00 dexX7 <notifications@github.com>:

> Given that the 2.7.1 release notes promote valus of 100 for noisy, and 110
> for non-noisy setups, this seems like a drastical change. I've read mixed
> opinions about it on rcgroups.com, but I assume this change was done to
> support noisier setups, and continuing to use 110 should be fine, if
> there are no high frequency oscillations?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/betaflight/betaflight/commit/d214f8602d99595ba62fba17c4a6dcc9e9d5e24e#commitcomment-17932240>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AKQlhP_WH3ejlp4V4mFWQlph7TFbD7Qeks5qNnmZgaJpZM4I5m32>
> .
>


@egamma is the `!stripPrefix === false` syntax intentional?


(yelling ?) :-(

> Misko Hevery: "The filter filter is filter, but the filter of the pipe is actually a formatter. So it’s really a formatter filter. You follow?"


helps with https://github.com/owncloud/client/issues/4987


True. I'll remove that call to `finish()`


Why are all the compiled hex files included in this commit despite being added to the gitignore file?


LOL


Shouldn't this be CombGuidIdGeneration.NewGuid();


comment


Let's use a constant from Constants.hs instead.


Thanks. I think the main issue is the behavior of basicSampling has changed so that input objects are not modified in place.

> On Jun 20, 2016, at 1:08 AM, Troy Butler <notifications@github.com> wrote:
>
> I'll check in the morning. I used the beta as is without issues at uwyo
> last week.
> On Jun 19, 2016 10:43 PM, "Lindley Graham" <notifications@github.com> wrote:
>
> > Fix it and submit a pull-request. If you're unsure then create a new
> > issue. See #167 <https://github.com/UT-CHG/BET/issues/167>
> >
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/UT-CHG/BET/commit/5d0be414f4f2ddff0492576e86f9b8146c0c779d#commitcomment-17928694>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe/AEwKZfFLsS0O1p6NNRyavYzuQm6d9hMFks5qNhqPgaJpZM4I5VJf>
> > .
> >
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


@Sn0wStorm Ich hab die Schilder wieder rausgenommen, ich denke, ein Schild, das man auf zwei Wege benutzen kann ist auf jeden Fall übersichtlicher als drei, die was ähnliches machen. Solange ich es zur Dokumentation hinzufüge :)

(Der Rest der Konversation ist wahrscheinlich durch das doppelte Rebasen verschwunden)


I didn't think that added any value in this particular case


Wow, you're just scaring me around. Our base is more reliable than yours, we made our work on making it as much flawless as possible, and I think this hard work stays in our hands and won't be shared to anyone for free.

I'm not a disabled idiot who can't make a clue what the shit you've made to the code. Although I'm not quite sure that any useful changes will be made in your repo in foreseeable future.


jaja :)


If i'm right : it allow now to use limereplacement to add {INSERTANS:123X123X123}, and replace with html editor to {trim(INSERTANS:123X123X123)}.
Great improvement :+1:


ist gemacht


soweit ich das sehen konnte, war es sogar mehr als fehler zu interpretieren, dass diese markierung nicht gesetzt wurde. ich sehe keine folgeprobleme..


Including the compiled.hex files is now the standard - this only lets some of them through.


Is this tested? I see only write commands; what's with reading?


Oh nice, did this make it into a pull request?


If you've merged from the main CMS repo it seems like you might have had a merge conflict and not resolved it fully.  That manifest shouldn't exist anymore.  https://github.com/joomla/joomla-cms/tree/staging/administrator/manifests/libraries

Specifically https://github.com/joomla/joomla-cms/pull/8735 was the PR where it moved from a Joomla library extension to a Composer dependency with a couple follow up PRs linked from https://github.com/joomla/joomla-cms/issues/10280


accident, removed


However stable your base is, is irrelevant. Im talking about how easy it is to merge shit, not how good your or my base is. I don't think you understand my argument. I'm talking about merge conflicts, yet you reply arguing that your code is better.

Although the statement you've made about being sure youre not going to see any useful changes in the future seems to imply you are very much aware of merging commits becoming more and more manual labour as time goes on. Besides being a jab at me, it seems to reveal how far your code us already away from mine. That and the comment about having a different base.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 138](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5087&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


No, you are editing /tftpboot/cmdline.txt not /boot/cmdline.txt so it won't effect the SERVER


See #1321


it is the patch for settings->buttons->enable on-screen nav bar.


Is it intentional to add SASS sources alongside the LESS ones, duplicating all the code?


@dkk0000 good work here. 👍


Modified Scope structure and added Block Scope


see https://github.com/owncloud/client/issues/4987


TeamCity OTA+ :: rvi_sota_server :: ci [Build 138](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5087&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Compilation error: Scala compiler (new) Build time: 00:03:04


@dkk0000 this line of code is setting string1 equal to the value of string2 - yes? won't that always make the two equal if subsequently tested? did all the tests pass?


ok then why copy then if they already existed (inherited from class Rule)
 :)  I know I am asking lots of questions :)

On Mon, Jun 20, 2016 at 1:58 PM, Sven Waschkut <notifications@github.com>
wrote:

> From/to must be always existent, so there is no need to create.
> But I tried to be consistent because have a look to function load_from()
> or load_to().
>
> What I tried to fixed there,
> Is that for DoS and PBF rules there is another child under from/to in the
> XML file.
>
> Is it better to use this:
> $tmp = DH::findFirstElement('from', $this->xmlroot);
>
>
> I followed this->load_to() / this->load_from()
> And copied this here, was only a try to be consistent.
>
>
> /**
> * For developper use only
> */
> protected function load_from()
> {
> $tmp = DH::findFirstElementOrCreate('from', $this->xmlroot);
> $this->from->load_from_domxml($tmp);
> }
>
> From: Christophe <notifications@github.com<mailto:notifications@github.com>>
>
> Reply-To: cpainchaud/pan-configurator <reply@reply.github.com<mailto:
> reply@reply.github.com>>
> Date: Monday, June 20, 2016 at 1:47 PM
> To: cpainchaud/pan-configurator <pan-configurator@noreply.github.com
> <mailto:pan-configurator@noreply.github.com>>
> Cc: Sven Waschkut <swaschkut@paloaltonetworks.com<mailto:
> swaschkut@paloaltonetworks.com>>, Mention <mention@noreply.github.com
> <mailto:mention@noreply.github.com>>
> Subject: Re: [cpainchaud/pan-configurator] fix for <from>/<to> extraction
> for DoS and PBF Rules (b56b2cb)
>
>
> why create 'from' and 'to' if they don't exist @swaschkut<
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_swaschkut&d=CwMCaQ&c=V9IgWpI5PvzTw83UyHGVSoW3Uc1MFWe5J8PTfkrzVSo&r=Jb4dHoFDiFQrZ-z3V4d67X3hMAoYOJl734q7LTcgj6k&m=i-kxUF9rkZUzjc7JTV0NoZNERT1tzHvJU3r2RwxjAfY&s=SsUeCob7oj0P_vQe6aQzboU4q-hidDy81trudrXuBqs&e=>
> ?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub<
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_cpainchaud_pan-2Dconfigurator_commit_b56b2cbf390ac57a7cdde17833396ad24267ff39-23commitcomment-2D17932475&d=CwMCaQ&c=V9IgWpI5PvzTw83UyHGVSoW3Uc1MFWe5J8PTfkrzVSo&r=Jb4dHoFDiFQrZ-z3V4d67X3hMAoYOJl734q7LTcgj6k&m=i-kxUF9rkZUzjc7JTV0NoZNERT1tzHvJU3r2RwxjAfY&s=RMfTr7SLy6ax1HdQ2uOGtV1i0WmB7j96n3jLwQipCsg&e=>,
> or mute the thread<
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe_AP5FvfRJ7IhlmgEuSThCIu8eG12GSWWnks5qNn3GgaJpZM4I5nqB&d=CwMCaQ&c=V9IgWpI5PvzTw83UyHGVSoW3Uc1MFWe5J8PTfkrzVSo&r=Jb4dHoFDiFQrZ-z3V4d67X3hMAoYOJl734q7LTcgj6k&m=i-kxUF9rkZUzjc7JTV0NoZNERT1tzHvJU3r2RwxjAfY&s=gidhAkG5J5A57lYNOgJJEt4NyqikYVa1XHcyWNUBQnw&e=>.
>
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/cpainchaud/pan-configurator/commit/b56b2cbf390ac57a7cdde17833396ad24267ff39#commitcomment-17932594>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AGYuvhBOlpLOwcT0zo4-Q3LEdmAd4SwJks5qNoCDgaJpZM4I5nqB>
> .
>


first commit


test comment


Good job on HTML, but what happened with CSS?


test comment #2


testing 123


@Sakiyuki Good work here Bryant. Keep leaning into it! 👍


@Zooip Il me simplifiait la vie pour faire des tests. Je vais le supprimer après, il n'est pas dans la spec de toutes façon.


Hi Felix — I guess I probably don't. I put this in so as to have the convenience functions of querying the maps (that are in FeatureVector), and didn't realize why you guys had stuck with just a HashMap. I'll revert it.

In general, this relates to JOSHUA-273: I want to remove all the output code from KBestExtractor and place it in StructuredTranslation (and then, in turn, to collapse Translation and StructuredTranslation). There are at least two ways to do this; do you have a preference?

- StructuredTranslationFactory could be called to create the necessary entries in StructuredTranslation, which would then just be stored as strings. This approach makes StructuredTranslation basically just a container that is operated on.

- StructuredTranslation can do its own computation, formatting the "outputFormat" line from the config file. This view makes StructuredTranslation itself a bit smarter.

matt


@Sakiyuki excellent work. 🚀


Or alternatively, you could do:
```
            elif cmd in [OPTIONS['prefix'] + 'fs',
                         OPTIONS['prefix'] + 'fullscreen',
                         OPTIONS['prefix'] + 'noads'
                         ]:
                self.do_fs()
```


Would it have worked without `color: inherit`?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 139](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5090&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Instead of writing `margin: 0` in so many places, how could you have removed the duplication?


I think we cannot avoid the exponential growing of the the symbols.
Or do you suggest to use bitwise OR with symbols?


I think this doesn't go anywhere - you're not getting me and I still see the same scaring.


You are only using the `cta` class once, so why is it a class?


Same comment as above.


I was referring to editing /etc/fstab ...


perfect! Thanks @mgruner


Overall, this is a really good job. See my comments above. 👍


english ftw :hankey:


@Ocramius i know what the problem is. Make PR?


deltee


The /boot doesn't matter anyway...  The / is only there to make sure it
gets mounted read-write and noatime

Gordon
On 20 Jun 2016 1:38 p.m., "Andrew Scheller" <notifications@github.com>
wrote:

> I was referring to editing /etc/fstab ...
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/raspberrypi/documentation/commit/11b662063496857dc16f056782f31d74a345465f#commitcomment-17933136>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AB9CHEQn07042V9n_8kSDedaIi6aex2Bks5qNonSgaJpZM4I5mOh>
> .
>


A secound approach from mysite was to reuse Load_to(),
But I failed to use this function for all rules


probably. maybe just updating it to latest (1.7.1) would get this to work.
i want to first merge you work on xonsh.


I agree that the Ceylon file icon you did was reasonable. But everything else was a mess (for example, the same awful icon for modules and module descriptors, which was very confusing). And after staring at the slightly blurry and baby-shit-brownity of the file icon for two weeks I grew out of love with it. But most importantly it didn't give us any foundation for a visual language for the other icons: they CEY lettering wasn't going to scale down any further and I couldn't just go sticking that overlay on top of all the other icons for modules, descriptors, etc. (I tried.)

The colors aren't final: they're just a non-shit-colored approximation of the website colors. My main goal here is to develop an overall concept for how all the icons fit together that I can take to James' team to give them a starting point and understanding of the kind of thing we need.


Shouldn't this be
```
Before=network.target
Requires=network.target
```


... and
```
WantedBy=network.target
```


TeamCity OTA+ :: rvi_sota_server :: ci [Build 139](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5090&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 192, ignored: 3 Build time: 00:06:35


Nice job replacing with a relative path! 👍


Hmm, does setting `auto` on the top and bottom margin do anything?


How can it run before network.target and require network.target at the same time?


First you're setting top and bottom margin to auto, and then to 0 a few lines later. Make up your mind 😕  😛


`$lockMode` as a default equally `null`
In this case check `$lockMode == LockMode::NONE` same as `null == 0` (is true).
Before it was `null === 0` (is false)


Fixed it: https://github.com/npct/pct-load/commit/afb2e9d3ed63bdbbebe29bb3ab47a6a8a487f189


I don't remember exactly what it's being used for but I can tell you that I changed it to handle an error. This is what the code looked like before my change "return "" if path == subpath"


`text-align: center` ought to be either in a class selector or in a list of selectors, such that you may remove the repetition.


Ahhh. I guess in that case the only downside is that you won't be able to update the kernel or firmware used on the server. (unless you temporarily do `mount /dev/mmcblk0p1 /boot` before running `sudo apt-get upgrade`)


Is this an appropriate change for the stable branch? AFAIK, ffmpeg releases are not always perfectly backwards compatible.


id selector vs class selector usage... what's the purpose of css class selectors vs id selectors?


This file was misplaced. Fixed in fb60f30c0b2094a4035592aabb5e7d49090169af.


This is a good start overall, but needs work.


This repo is is based on the 3.6.x branch. Might be that there is an issue with that.


This breaks things pyrocksdb, why did you remove it ?


Is that intmask valid or was it just there before?


Thanks!


ffmpeg-full is not the lib, but the tool to be used more like with commandline.


This is done, see https://github.com/snaekobbi/pipeline-mod-sbs/commit/2c2b2434f9f75ea138b3f0e0322ffd86440049bd


yes


How backwards compatible is the command line tool?


This solution does not make sense.


This is a good start overall. There was definitely stuff that needed improvement though. See my comments.


are you using the "refresh"-timers to automatically refresh the retrospection?


There are some issues with indentation. Did you try using sublime's reindent?


It's valid, turns out that bridgeSpan also needed one: https://github.com/HPI-SWA-Lab/RSqueak/commit/51c27d08c9138f981ec358a596a7929fce857d21


@peter-gribanov see https://github.com/doctrine/doctrine2/pull/5885 - already fixed


I think it's ok project. I don't have any remarks.

Mark was there.


Class selectors vs id selectors... I see you have used class selectors throughout, not sure that's the right fit. Give it some thought.


Holy shit I'm about to unsubscribe from this shit.


works now all the time - tested the lib, with Cordova plugin

👍


Visualization works fine, Smaller dataset added.


I think that with your CSS you could have utilized the cascade a bit more.


Hi @mgruner
Thanks for the merge! I uploaded my latest language file to Transifex, which contains the changes, that you made with the last sync. Can you sync it again? Now Hungarian translation will be 100%!


So your fonts were working at some point, but then stoppeD?


재미있네 이거


I use it since pre-1.0 and, for the options I use, never found a problem until now.


이렇게 하는거구나


Err, and what on 32bit, where intmaks masks to 32bit?


Hm... are you sure you did `make install` before the invocation leading to the error message?


I think this should be `false`, otherwise the check at line 168 will never pass.
It also brakes the test cases for issue #26.


Thanks you :)


Instead of "FCM" string, let's use `FirebaseMessaging.INSTANCE_ID_SCOPE`


Yes, we are maintaining some apps with SASS, some with LESS.... it's this or we rollback and choose just 1 technology


Hi @fbocosta,
created an issue #12 for this.


Maybe not.....


@Particular/servicecontrol-maintainers @ramonsmits can one of you either submit a PR and work on finishing this or delete the branch


I'd say that about the only missed mark was overuse of class selectors vs id selectors for css properties that don't repeat. Use id selectors when things are unique to the page, and class selectors when they aren't.


does this work with the comma after the 7.days


What's happening with the indentation here? Sibling elements should be indented at the same level.


Pulled, rebuild is running. Thanks!

Martin Gruner
Team Lead R&D

OTRS AG
Bahnhofplatz 1a
94315 Straubing

T: +49 (0)6172 681988 0
F: +49 (0)9421 56818 18
I:  www.otrs.com/

Geschäftssitz: Bad Homburg, Amtsgericht: Bad Homburg, HRB 10751, USt-Nr.: DE256610065
Aufsichtsratsvorsitzender: Burchard Steinbild, Vorstand: André Mindermann (Vorsitzender), Christopher Kuhn, Sabine Riedel

Mobile Kommunikation und transparente Prozesse - Mit der OTRS Business Solution™ Managed 5 schnell und ohne eigene IT-Ressourcen starten  - Jetzt neue Features entdecken und bestellen
https://www.otrs.com/neu-in-otrs-business-solution-5-mobile-kommunikation-transparente-prozesse/?lang=de

> Am 20.06.2016 um 14:56 schrieb Balázs Úr <notifications@github.com>:
>
> Hi @mgruner
> Thanks for the merge! I uploaded my latest language file to Transifex, which contains the changes, that you made with the last sync. Can you sync it again? Now Hungarian translation will be 100%!
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


Okay, thanks!


If verovio allows <accid> sub-elements on notes, I'll move the information which repeating the data is encoding into the sub-element.

The duplication in this case means a cautionary accidental, as the red notes in this example are:

![screen shot 2016-06-20 at 5 49 24 am](https://cloud.githubusercontent.com/assets/3487289/16194801/5549dcec-36ab-11e6-82c0-8975c4c15f34.png)

```xml
<note oct="4' pname="c">
    <accid accid="n" func="caution">
</note>
```

Is that correct (no @accid or @accid.ges in note element as they are now in accid element)?

Also, does verovio do editorial accidentals (which I would want rendered as small accidentals above the notes)?

```xml
<note oct="4' pname="c">
    <accid accid="n" func="edit">
</note>
```


Although you are overusing class selectors, you are using very descriptive names, which is great.


I suppose the solution is to either use `cconvert` or store it in a Dict

http://docs.julialang.org/en/release-0.4/stdlib/c/#Base.cconvert


It's conceivable that there might be a load balancer/proxy upstream that is non-SSL, but forwards to an HTTPs backend. Rather than force-set x-forwarded-proto to https, haproxy should set the header only if the header does not exist. However, it looks like this should set it to https, rather than http. Will fix this shortly.


Looks good to me


extract drupal_get_path into a variable


@K-Guan 嗯 mdzz


Ah so you gave up on `<blockquote>`?


LOL!


Double LOL.


Is it possible to use some constant for numbers?


Any reason why you're using hard-coded image width and height vs applying those properties via css?


When the margin is `0` you may just say `margin: 0` - no need for unit specification, it's basically redundant.


yay, another emacs user at sg.


pddddddd !!!


Any time you have repetition like in the case of both the `h1` and the `h2` having the same `font-size`, that's a possibility to either use a css class selector to "decorate" those elements with the set of shared properties, or use a selector list. I know you've asked about css preprocessors... they have even better ways of dealing with this.


Thank you for the comment, however I looked through the rest of the code and do not see the "_" notation being used elsewhere. For that reason, I am vouching to leave it as-is for consistency sake.


Thanks, hope this is fixed now, finally ;)


How about an overall css class selector called `yellow` that encapsulates this color and then reusing it everywhere that's needed?


since this commit, the strongest belief of <ballpos --> [above]> and <ballpos --> [below]> when project-eternalized to current time really reflects the current ball position (the one with higher truth expectation).


This update is repair W?


Indentation.


You might want to have your release script pull the actual version number from this file since the [release page](https://github.com/COMBINE-lab/RapMap/releases/tag/v0.3.0) still shows v0.2.3.


Also, just while this comes to me, another feature idea is adding support for authentication (401 Authentication Required). I think that's a pretty cool one, and not as hard as you might thing, but it's probably best to build it on a system for returning different response codes (perhaps with exceptions, as I explained above).


I installed the cartridge (i.e. I executed the 'rhc cartridge add ...' command. Does this mean that we have autoskips now? :)


If so I will try


please check new commit override load_to()/load_from()


**Code Style - Nitpick**

Have a new line after every block statement


TeamCity OTA+ :: rvi_sota_server :: ci [Build 140](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5101&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


don't call it test


TeamCity OTA+ :: rvi_sota_server :: ci [Build 140](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5101&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:16


Call this test


It doesn't but I'm going to do the same thing I did with YasuoPro with YasuoSharp, I'm going to use Valvrave's Evade to replace the non-working one.


Really clean html work, very well done. However, there's one improvement I'd recommend here and that is to use id selectors for unique elements, and class selectors for non-unique elements, which have shared css properties.


@blueclock I have rephrased `increase your conversions` to `increase your conversion rate` in the commit above.

Could you confirm that this is grammatically correct, please? I'm not 100% sure regarding the plural form. Thx.


> I'm not sure what is the advantage of using global ctor here ? and how could we set the priority against object ctors. There is no such a thing.

Write a class where it's ctor is your process initialization code.
Add a global instance of the class so that the ctor will be call when process initialize.
Use the `INIT_PRIORITY` macro (which is `__attribute__((init_priority(x)))`) to set the initialization order of when the ctor will be call related to other global initialization code.


This really gives some room for troubles. Due to autocrlf I constantly have changes on these files. The global option doesn't event change anything here due to the local gitattributes.


like


> Good news! We have on thread exit trick too. Current implementation will fix the issue for both static and shared on xplat. Taking it back. So, there is a magic and no need for extra API.

That's great.


@m-kuhn had a look and proposed this #3224
thanks a lot and all the best for the moving in....


Ouch, `cconvert` is not easy to digest. I'm starting to feel that this PR is too high above my head!


Doing anything that might solve the problem


Hi Matt,

Thanks, StructuredTranslation is an object to be used in a downstream codebase and I think it would be good to have minimal dependencies on Joshua objects as possible there :)

Regarding your question:
I’d go for a container/value object for the reason stated above if possible.

Cheers,
Felix


Falco, you forgot about /yell !


@frioux: the [fix for this embarrassing error](https://github.com/dbsrgits/dbix-class/commit/0844bb3#diff-5c086f1a6d1ac54926c39c849635b355) is now on CPAN as 0.082840


here what is Listing[] you are importing Todo


This is way over the line length guidelines and there are easy ways to avoid that by creating a local variable for the ResolvableType.


use always _ with private variables


OK coolio


add .js files also in gitignore


delete this file .gitignore~


Ahh --- the release script still requires me to name things manually.  I'll re-compile with the new version info.  Thanks for catching this (and reminding me that the script should be more thoroughly automated ;P).


here u are doing same importing Todo as a type but using listing


Shouldn't be.  Spot checking things it looks like the referenced PRs are in the 3.6.x branch.  Also it seems like there were definitely some unresolved merge conflicts merging here, compare https://github.com/joomla/joomla-cms/blob/3.6.x/composer.json and https://github.com/joomla-projects/custom-fields/blob/custom-fields/composer.json (SimplePie is missing in the Composer definitions but exists in the filesystem).

The reason it's looking for SimplePie's extension manifest is because the record exists in your database schema still (compare https://github.com/joomla/joomla-cms/blob/3.6.x/installation/sql/mysql/joomla.sql#L516 and https://github.com/joomla-projects/custom-fields/blob/custom-fields/installation/sql/mysql/joomla.sql#L516).


TeamCity OTA+ :: rvi_sota_server :: ci [Build 141](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5103&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


You can get websocket server from here https://github.com/fitraditya/collider-heroku
It's written using go.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 141](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5103&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:12


It should be fine on 32bit because bridge span will at max be as wide as the addressable space - 1 large object header (the size of the penultimate bridge). So you shouldn't be able to find images where this breaks.


This seems to be broken


Storing a reference to `BigInt` in `MPZ` (at the end) should be good enough. You can also use `Ref{MPZ}` and remove the `&` here.

> Julia should maybe provide a function similar to `scm_remember_upto_here_1` in guile (cf. here), to instruct the compiler to keep a reference to `x` untill after the `ccall`.

This is exactly the `cconvert`/`unsafe_convert` contract and should be enough for `ccall` here. It is not for other cases and I am thinking of adding something similar.


__API Description file syntax error on line 2443__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


TeamCity OTA+ :: rvi_sota_server :: ci [Build 142](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5105&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Implements #136
Disasm class (disasm.cpp) is not going to be ported anymore. Modifications will be made in offline-x86-code.h .


TeamCity OTA+ :: rvi_sota_server :: ci [Build 142](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5105&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:13


![screenshot_2016-06-20-21-27-14_com android browser](https://cloud.githubusercontent.com/assets/18411104/16195794/da28face-372d-11e6-8917-1afd8ce67d4b.png)


We should. You can set the last_action field of some story to two days ago and wait for an hour, and see if it works.


![screenshot_2016-06-20-21-27-14_com android browser](https://cloud.githubusercontent.com/assets/18411104/16195851/0621a3e2-372e-11e6-959c-b22080ff502a.png)


@mef Yes, it is correct


oh but then we wouldn't know how many results to pull, the last 10 results
could all be for meal2, and not equally 1-2-3-1-2-3

On Sat, Jun 18, 2016 at 9:54 PM, Samuel Cousin <notifications@github.com>
wrote:

> I think that it would make more sense to do a single $.get request, then
> check where to place our results within our always callback. Something like
>
> results.forEach(function(element, index) {
>  if(element.currentChapter === 1) {
>    $(".meal1").html(element.message)
>  }
> }
>
> Am I missing something?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/dailyTLJ/foodsessions-app/commit/be21a090a7e2f62f5d2dc96d733b686dff32dad4#commitcomment-17924869>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAOjL6i3XdSljfgdUodNIcuidBSuuH2cks5qNKFIgaJpZM4I5Eqm>
> .
>


this wont increase dest for every sample!!!
the while loop is
```
	while (--numSamples >= 0)
	  *dest = state.process (*dest, *this);
```

to include it either use
```
	while (--numSamples >= 0){
	  *dest = state.process (*dest, *this);
	   dest++;
	}
```

or
```
	while (--numSamples >= 0)
	  *(dest++) = state.process (*dest, *this);
```
though im not sure about the second one but... as far as my C knowledge goes that should work
@michail-nikolaev @novichiv


é bom tirar os comentários né @kteixeira  hauahuahua :smile:


nice!


I thought policy was that 'obvious' buttons should not be named, or this should be done with external files, not in the source.


I personally think the incrementing of dest should stay in the same line.... that prevents some errors like the above.


device/qcom/common/power/power-8992.c:216:9: error: unknown type name 'launch_boost_info_t'
         launch_boost_info_t *info = (launch_boost_info_t *)data;
         ^
device/qcom/common/power/power-8992.c:216:38: error: 'launch_boost_info_t' undeclared (first use in this function)
         launch_boost_info_t *info = (launch_boost_info_t *)data;

(gives error please it :+1:  )


Same BUG!
@michail-nikolaev @novichiv


Sorry for the delay on this; I know it's mildly urgent. I've got a few
hours boxed out this evening to take a look. :D
On Jun 17, 2016 3:10 PM, "Jeff Atwood" <notifications@github.com> wrote:

> [image: image]
> <https://cloud.githubusercontent.com/assets/1522517/16161797/5cf016c4-3484-11e6-98a3-13e5211c4ff3.png>
>
> This style gets completely mangled as a result of the double call. It's
> pretty bad.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/discourse/discourse/commit/feffe23cc5fd7dffa68137b156910ee831ad6aa2#commitcomment-17917983>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAtzjYOw08dAMndLzykkFFrLprE3652dks5qMvEegaJpZM4I4Ofa>
> .
>


Yes - the name of the data has changed from `co2_ems` to `ghg_ems`.


Methods and fields of interfaces are implied public.


Please git reset --hard~1 to remove permanently, the last commit 6140e4c3d2aeb118fc9a9b79b9783dff73bc25fa

This is not open-source and you should not have been kanged out of my ramdisk without permission.

No-one has permission to use any part or derivative of that from within my private ramdisks.

Furthermore, I have made a private fork of this repo & contended commit for "safe-keeping".

Regards


why is commenting out stuff and leaving it "removing trash"? unused stuff should just be removed if its useless anyway


- released v6.0.3
- https://github.com/Shuttle/Shuttle.Esb.RabbitMQ/releases/tag/v6.0.3


Pourquoi le port est inclut dans la publicIP ?


Ok I didn't know storing a reference in `MPZ` at the end would be valid. I used globals `_V, _W` here to avoid heap-allocating a new `MPZ` for each call (admitedly the small overhead would be acceptable), but I believe to have read that `Ref` would heap-allocate an object, right?


So, we should replace `@name co2_ems` with `@name ghg_ems` and delete the aliases?


@alfonsogarciacaro Did you forget fable-fsharp/project.json ?


@5ource, is it normal that you are using little-endian byte order ?


Day has only one capture group therefore couldn't you write day.group(0) instead of day.groups()[0]?


It wont in this case, since mpz is already heap allocated.


New line after class statement, after the last method statement, and after the class `}`


#5737-Multistage-heating-coil-only-operates-at-full-flow (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


how about #1029


OK so I understand this is in anticipation of the deprecation of `&`.


Can we have more than one XDATA series?
Swimming files have "Stroke Type" I liked to add but the "cost" to add a new series was too high for me, to add Stroke Type as XDATA may be more viable


__API Description file syntax error on line 1__: Invalid member type of object in MSON definition (Autogenerated message from apiary.io)


When I try to write text in a webview — I can't type 'r' character


Why is it `20150418`?


Seems unnecessary as `args` doesn't change.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 143](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5106&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Would you mind explaining what does `xpos != xpos` do too? Thanks!


![urchinblue](https://cloud.githubusercontent.com/assets/17986980/16196349/57271b8c-3706-11e6-943f-c629c388121d.png)
![evil_shark](https://cloud.githubusercontent.com/assets/17986980/16196350/57450f7a-3706-11e6-8a26-2f156e69ffab.png)
![frog](https://cloud.githubusercontent.com/assets/17986980/16196353/575202a2-3706-11e6-8cf2-79d1aa4ca211.png)
![genesis 32x scd - ecco the dolphin - giant seahorse](https://cloud.githubusercontent.com/assets/17986980/16196355/576189d4-3706-11e6-9140-4ea6ee16897b.png)
![los-octo](https://cloud.githubusercontent.com/assets/17986980/16196352/57515992-3706-11e6-8c88-a367d11ab6ce.png)
![mediumjellyfish](https://cloud.githubusercontent.com/assets/17986980/16196351/5750b104-3706-11e6-9a33-151ce4ddab6c.png)
![octopus](https://cloud.githubusercontent.com/assets/17986980/16196354/5752c6ce-3706-11e6-829a-e1a908690389.png)
![octopus2](https://cloud.githubusercontent.com/assets/17986980/16196356/5769211c-3706-11e6-9b0a-0c02737cd698.png)
![pc computer - maplestory - bone fish](https://cloud.githubusercontent.com/assets/17986980/16196357/576dcd16-3706-11e6-9766-b1132fe4ec73.png)
![pc computer - maplestory - risell squid](https://cloud.githubusercontent.com/assets/17986980/16196358/5771112e-3706-11e6-882c-71cd706d3842.png)
![pc computer - spelunky - piranhas](https://cloud.githubusercontent.com/assets/17986980/16196359/5771f17a-3706-11e6-88fe-55fda7d1cdb5.png)
![playerspritesheetnew](https://cloud.githubusercontent.com/assets/17986980/16196360/577a162a-3706-11e6-8fe8-0a5ca609e866.png)
![playstation - breath of fire 3 - giant crab](https://cloud.githubusercontent.com/assets/17986980/16196361/5785797a-3706-11e6-9f2e-2bbfe28d4c77.png)
![red_eruma_nobg](https://cloud.githubusercontent.com/assets/17986980/16196362/578604bc-3706-11e6-930f-c7f6c384b61a.png)
![robot_fish](https://cloud.githubusercontent.com/assets/17986980/16196363/578adf00-3706-11e6-9b0d-e8d904051300.png)
![seacreature](https://cloud.githubusercontent.com/assets/17986980/16196366/579a99b8-3706-11e6-95dc-f5907c48efdc.png)
![seahorse](https://cloud.githubusercontent.com/assets/17986980/16196364/578d88f4-3706-11e6-9864-16106878bbc6.png)
![spritedivera](https://cloud.githubusercontent.com/assets/17986980/16196365/57926d38-3706-11e6-8ac3-c0408c1126cf.png)
![squid](https://cloud.githubusercontent.com/assets/17986980/16196367/57a240c8-3706-11e6-9aeb-c6c727240ab6.png)


TeamCity OTA+ :: rvi_sota_server :: ci [Build 144](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5109&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


TeamCity OTA+ :: rvi_sota_server :: ci [Build 145](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5110&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:14


GWAS no longer working on circularised track - perhaps some erroneous merges especially in hic-vis.js?


normaly it teleports you to homebind, i tested it as well but not on the very last patch, for this i think this fix is needed. maybe you have a GM char teleport to shatt but you have expansion set to 0 in this case it results to starting position.

my test was homebind on OG and tauren test char.


По погрешка съм добавил и водолаза, но има доста животни с прилично качество, които са готови за умиране :)


This is totally fixed now.


この目的関数はどうしてこのような式になっているのでしょうか。
// （「生の評価値の誤差」と「予想勝率の誤差」の双方を、バランスよく最小化するように配慮している）
とのことから、
f(score_y, score_t) = (y - t)**2 + a*(score_y - score_t)**2 // a は適当な係数
                    = y**2 - 2*y*t + t**2 + a*(score_y**2 - 2*score_y*score_t + score_t**2)
f(score_y, score_t) を score_y について微分したものが delta_y として、
delta_y = 2*y*d_sigmoid(score_y) + 2*t*d_sigmoid(score_y) + a*(2*score_y - 2*score_t)
        = 2*d_sigmoid(score_y)*(y + t) + 2*a*(score_y - score_t)
となるのではないかと思いましたが、ソースコードの式とは異なります。
また、生の評価値と勝率を一つの式で扱うには絶妙なバランスが必要になりそうなので、
勝率の誤差だけを最小化した方が簡単に思えるのですが、生の評価値の誤差も最小化した方が強くなるのでしょうか。


Ok; fixed the releases. I'll think about how to best automate the release script.  Thanks again.


@amitvtaneja Use the issue that I started here: https://github.com/metrumresearchgroup/mrgsolve/issues/66


Why not use "this(mergeType, stopType, new FlatLatticeNoTopDomain<>(), transfer);" here?
Then some sub-class-constructors would not have been changed.


wrong word


TeamCity OTA+ :: rvi_sota_server :: ci [Build 143](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5106&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 178, ignored: 3 Build time: 00:06:52
### Failed tests
```
org.genivi.sota.core.transfer.VehicleUpdatesSpec.after blocking installation queue, no packages are returned even if some are pending for installation: <no details avaliable>

```


Looks like a typo :)


Sounds Good.


should be ``^1.0.0``


Right that's possible, but I think that wouldn't be too different from the current version.
At the end of the day I think it's better if the CPA specifies what abstract domain it uses.


Removing the SimplePie row from the installer script fixed the issue as well. Could revert this change with commit fd9486539f51ae90e822e4d5808cf05d2bad1cf5.


The reason why I was not using this function is because it doesn't catch certain corner cases. So to be sure we could maybe change this function to use quoting using `'` and `'\''` here.


"… what your credentials are."


yup correct, just forgot to push the working version :/ sorry :snail:


This part needs review, from the existing code it's not very clear how joining should work for partitioned states.
Should they never be joined? Should join return some TOP element?


PyCharm does PEP8'ing very nicely in case you don't know :)


Nais :v


Use theme_item_list function, to generate html of the item list, and pass it here as variable.


Vllt sogar als teil des addons in einem docs folder ausliefern und in der installation via addon-help feature zugänglich machen?

Hätte den charme dass die docs auch immer passend zum source sind


@tivac @isiahmeadows @Zolmeister @aembke @barneycarroll @trueadm @Bobris @joelrich @paldepind @localvoid @leeoniya @jordwalke @ArthurClemens @StephanHoyer

Please let me know if I made a typo in anyone's names or if there's anyone else I forgot to put in that list. And thank you all.


I am not so sure, do you have a reference?


redis don't download


typo: space **before** `file`, not after


Podemos conversar sobre se o Pluto é um planeta ou não!


In addition to this, each platform should clamp the coordinates to sane limits. Will try to get that into 3.3.


Are you sure Pluto is a planet...? Do you have a reference?


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


Du har nok rett


While I agree that this is the case in A3, I am not sure if that is strictly needed.


So is it correct/reasonable to do this check *before* enqueuing the work?


good question. I only removed the `-`s. https://github.com/metabrainz/acousticbrainz-server/commit/1beab14c5728f2367f8c2c2634bd098f973c22da#diff-43145dfb39083678a4c99660199cbb3cR1 :angel:


Typo!


#5737-Multistage-heating-coil-only-operates-at-full-flow (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1101 of 1101 tests passed, 0 test warnings)

![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-34.28%25-green.svg)


HI,

I seen your code on spdif.
I do not understand why the "SNDRV_PCM_FORMAT_S32_LE" format is forced to 24bits.
in function sunxi_spdif_hw_params() in file sunxi_spdif.c


No because in https://github.com/openshift/origin/pull/6233 we will need to count all pods, available pods, etc. and that needs to happen in the controller loop.


BookKart Code for Angular JS version 1.0


So this is working again.

Would it be possible to put the jtracker.js files into a sub directory? e.g.
`/www/media/js/jtracker.js`
`/www/media/js/jtracker/jtracker.js`

The only reason is that the script that creates the language templates searches recursively and spits out those strange warnings when analyzing files like JQuery UI etc.

This is why those ugly lines have been added: https://github.com/joomla/jissues/blob/master/cli/Application/Command/Make/Langtemplates.php#L232-239 which are **not future proof** at all...


@didrix Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.namedfile/29/violations
```
plone/namedfile/file.py:291:1: C901 'getImageInfo' is too complex (15)
plone/namedfile/scaling.py:192:1: C901 'DefaultImageScalingFactory.__call__' is too complex (13)
plone/namedfile/scaling.py:380:1: C901 'ImageScaling.scale' is too complex (14)
plone/namedfile/scaling.py:421:9: B901 blind except: statement
plone/namedfile/scaling.py:425:80: E501 line too long (80 > 79 characters)
plone/namedfile/scaling.py:426:5: E125 continuation line with same indent as next logical line
plone/namedfile/scaling.py:465:15: E127 continuation line over-indented for visual indent
plone/namedfile/scaling.py:466:15: E127 continuation line over-indented for visual indent
plone/namedfile/scaling.py:467:15: E127 continuation line over-indented for visual indent
plone/namedfile/scaling.py:469:41: Q000 Remove bad quotes.
plone/namedfile/utils.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/utils.py:89:65: Q000 Remove bad quotes.
plone/namedfile/utils.py:89:80: E501 line too long (85 > 79 characters)
plone/namedfile/utils.py:98:65: Q000 Remove bad quotes.
plone/namedfile/utils.py:98:80: E501 line too long (85 > 79 characters)
plone/namedfile/utils.py:107:65: Q000 Remove bad quotes.
plone/namedfile/utils.py:107:80: E501 line too long (85 > 79 characters)
plone/namedfile/utils.py:115:13: E122 continuation line missing indentation or outdented
plone/namedfile/utils.py:119:13: E122 continuation line missing indentation or outdented
plone/namedfile/tests/test_scaling.py:144:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:146:80: E501 line too long (83 > 79 characters)
plone/namedfile/tests/test_scaling.py:269:13: E128 continuation line under-indented for visual indent
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Don't reformat code there is NO WAY we can review what you have done :-(


#5737-Multistage-heating-coil-only-operates-at-full-flow (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2217 of 2217 tests passed, 0 test warnings)

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


The equivalent would be:

    (ivy-set-prompt 'counsel-ag #'counsel-prompt-function-dir)

I could also add:

    (ivy-set-prompt t #'counsel-prompt-function-dir)

But I don't know if a catch-all clause would be useful. Maybe it's simpler to add `ivy-set-prompt` for each file completion function.


supi!


- misses section five
- perhaps you could add the chapter number to every topic. Now it is still inconsistent


Can you please update the field documentation for `serverHeader`? (+ the appendix).


We should somehow mention, that this might break existing implementations of user conditions. This method has must be always implemented now, even if the result is empty.


I thought about keying off of `enf`, but that would just make accessing the working enforester more cumbersome.


haii-brada


TeamCity OTA+ :: rvi_sota_server :: ci [Build 146](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5118&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


What's the reason for this specific commit? The regulations make a difference between "supervisor" (exactly one) and "co-supervisor" (optional)
https://admin.kuleuven.be/rd/doctoraatsreglement/en/phdregulation-set


If you want to update all the build and template references go for it.  Can't say I'm a huge fan of moving files to fix an unrelated script but if that's the most efficient way then so be it.


Why is there a HashSet created with the size of one element?

From my point of view it should be enough to return null and consider this on the caller.
It is not necessary to consume any memory when calling this method, is it?


- missing introduction abbreviation TEC
- also added abbreviation for the virtual human, but misses introduction: `GOAL Services Virtual Human (also known as the GSVH)`. Also I prefer 'agent' as abbreviation, as GSVH is quite an off one.
- `to Tygron Engine ` should be something like: `to the


@LouisGac : if it don't fix the issue : maybe revert it. I think we really need specific css for specific media (in fact print ....).


I believe the answer is, "yes". @jlcsmith did you have something other than this that you were thinking you'd see in this field?


https://issues.jboss.org/browse/JBEAP-3839 is completely unrelated.


We certainly can't avoid the exponential growth of ids when combining flags. I meant that we shouldn't create individual names on a per id basis like `KeystrokeWithModifiersWithLongPressAction`. Yes, I indeed meant bitwise ORing symbols.


Why is `Collection.prototype.findOne` being deprecated?


That attribute should not be there. Did you want to submit a pull request to remove it?


Thanks again.. for your spontaneous help.


`bind` won't work with lambas, so `this` will be the global context.


Why "kotlin.arrayOf"?


`on a pre-existing connector` should be somtehing like `on the pre-existing connector`, as the connector has been stated in the previous paragraph


2 comments here:
- I see the new v2p0 directory referenced here, but no new files were committed.
- I think the 1.6 case should be left as-is.


@RafaFP ?


Probably need logic that allows the user to specify the spark.version and then programmatically determine the scala version.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 146](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5118&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:08:00


It might be split into two parts so the game knows when to start drawing the ball.


Oops, thanks. Done: 2c8eaac


Hi,
1. A20 supports only 24bits resolution
2. I found information that in SND_PCM FORMAT S32_LE data is stored in 24bits of 4 bytes
I'm not expert in linux and I may be wrong


libcrypto is still producing a segmentation fault


Este tag div abre, pero no cierra.


- don't use '5', but use 'five'


Este div no tiene tag de cierre


How does that differ from owner-producer? Am I misunderstanding that field's intent?


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 68](http://build.prxm.it:8811/viewLog.html?buildId=7525&buildTypeId=Jarvis_DocumentStore_GitVersion) is now running


Thanks for pointing this out! This is now merged via e8f4ee8a390b3f52d71d487f767215e0e37a13a2 and 364db6762e51d2f46b065843f692afc2ed14196c


Revisar tag de cierre


Düzgün bir isimlendirme yapalım. Örneğin FibonacciTests


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 68](http://build.prxm.it:8811/viewLog.html?buildId=7525&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **FAILURE**
Summary: Compilation error: src\Jarvis.DocumentStore.Tests\Jarvis.DocumentStore.Tests.csproj (new) Build time: 00:01:39


In the CMS_get0_type or in some other function?


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


TeamCity Glyssen :: Glyssen pull requests [Build *.*.113.d98e1cded29980437657480bc9c5ed504a3e98a9](http://build.palaso.org/viewLog.html?buildId=73871&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1267, ignored: 17 Build time: 00:05:45


Los tags p pueden ir en minúsculas también, de hecho es recomendable que si se usan minúsculas para el html todo sea en minúsculas.


aa


zzz


Explain the format of the file.

Maybe say it is the same format as [EnvironmentFile=](https://www.freedesktop.org/software/systemd/man/systemd.exec.html#EnvironmentFile=)?


Thanks again!


NATO is applied _only_ to?


I'm not sure this is wanted.
At least, the scope changes must go in a `dependencyManagement` section in order to be retrievable by a `scope import`.
Then, actually, they should probably be changed at root. That must be discussed: it can make sense to overwrite those scopes in the distribution rather than setting `provided` by default in `nuxeo-ecm` but then it could be better to set this in the parent `org.nuxeo.ecm.distribution:nuxeo-distribution`.
Note that `nuxeo-distribution-server` cannot be inherited because of its `zip` packaging type, however its dependencyManagement can be imported.

Summarizing:

1. move to `dependencyManagement` (mandatory)
2. move that `dependencyManagement` to the parent `nuxeo-distribution` (recommended)
3. set those libraries as `provided` by default in `nuxeo-ecm` (alternative to 2.)


For me the SPDIF only supports 24-bit on my amplifier when mpv sends s32 I hear that noise.
pass through is NOK with mpv 0.16 due to no support of AES0 parameter. ( mpv team add fall back to iec958 in these case)
have you datasheet of SPDIF module A20?
thanks


MUPX use Docker, so there is some problem whit it, or simply someone do not want use Docker.
So i think that it is important continue to develop MUP.
May we start to mantain MUP here ?
Is it possible fork MUP included the PR not accepted ? So we can accept all PR here and become the new official MUP repo whitout Docker in the plan.


Revisar en general los tags divs.


Her bir metodda yalnızca bir adet assertion olmalı. Bu metodda 5 tane var.
Metod adları, yapılan işlemler ve beklenen değerler üzerine olmalı. Yani;
Test_Fibonacci_5_Verildiginde_8_Donmeli
gibi. Ayrıca mümkün olduğunca İngilizce lütfen.
Bazı uygun isimlendirme kalıpları için https://dzone.com/articles/7-popular-unit-test-naming


develop (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1664 of 1664 tests passed, 0 test warnings)

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


this comment does not add anything to the code


The two methods should be refactored to avoid duplication


A20 datasheet with SPDIF (OWA)
[http://dl.linux-sunxi.org/A20/A20%20user%20manual%20v1.3%2020141010.pdf](url)


## This is a markdown file


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 69](http://build.prxm.it:8811/viewLog.html?buildId=7527&buildTypeId=Jarvis_DocumentStore_GitVersion) is now running


Notice:

xenapi driver is not built on libvirt package version, so libvirt must be built:
* Download xenapi, make, make install
* On a Debian: apt-get install libxen-dev
* Download latest libvirt sources:
** Edit src/xenapi/xenapi_driver.c, function "callfunc" (bottom), force ssl_verifier's = 0
** ./configure --with-esx --with-xen --with-xenapi --sysconfdir=/etc --localstatedir=/var
** make, make install
* Test: virsh -c xenapi://user@ip -d4
* Add "/etc/libvirt/auth.conf" so username/pwd is not asked for (see libvirt documentation)


I really don't like this runWhenSmart() call. It can potentially hold the thread for multiple seconds or even minutes while IDEA is indexing data, and I don't think the caller would appreciate that.


This was a request from the faculty. The difference exists in the supervisory committee during your phd but not in the layout of the dissertation front page. See (Sec. "Cover and Title"): https://set.kuleuven.be/phd/researchers/template_kaft/guidelines_PhDthesis.htm

(Given the current set of rules we could also reinstate the \cosupervisor command and treat it as the same when building the title page.)


Thanks!


genau so ist es geplant - sofern ich dich richtig verstanden habe :) .. das yform addon bekommen einen reiter und dort ist diese doku dann verfügbar


Muy buena la estructura del html y también la semántica, esta perfecto :)


#5737-Multistage-heating-coil-only-operates-at-full-flow (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2251 of 2251 tests passed, 0 test warnings)

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


Did you intend to remove the dependency on spark-mllib?


#5737-Multistage-heating-coil-only-operates-at-full-flow (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


@annejan aren't git submodules way better for one big repo of stuff?


Object HelloWorld


Possibly @Techwolf12, but this is a repository of build files for Slackware .  . so I think it makes sense.
Also not my repository . .


Adding wget together with ca-certificates guarantees that wget will work properly when using https


Yes you can have an unlimited number of XDATA series, and each one has up to 8 values.
I hear you on the 'cost' of adding series, this is to try and address that, since we are going to see lots more data from different sports and devices and need an extensible framework. I think the datapoints will become a 'legacy' feature eventually.

For an example, Damien just added Weather data from FIT files that has windspeed, direction, temp and humidity. This will then be used by the windspeed tool he is writing.

The idea is that one XDATA series contains recording from a device with the same sample rate, where multiple XDATA series will have different recording rates. You can then group stuff together.

I haven't worked out how to make it work with R, USER DATA and USER METRICS but reckon if we are clever with that we can then EASILY plot them on the ride plot and metric plots relatively quickly.

We also need to think about how merge would work (would be nice to be able to merge ANY data with ride data, in some generic extensible way. Also would be neat to be able to add things like the workout "qwkcode" as an xdata series for a completed workout (?) and even plot it against the actuals.....

I'm sure we will find areas to improve it, like being able to specify sample rates and how resampling should work. The 'names' we use need to be catalogued and controlled (so they have a common meaning across sports and devices) and how we manage translation of them etc

But its the first step.


did you forget to `git add tests/env_file_test.conf`?


TeamCity Glyssen :: Glyssen pull requests [Build *.*.114.3a558acf3e0dc1b4e5890b41ca5f63e06724f200](http://build.palaso.org/viewLog.html?buildId=73872&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1267, ignored: 17 Build time: 00:05:20


En el ejercicio 1 revisar la indentación y usar tags de html5, es decir sustituir los tags table tr, td por los que correspondan usarse :)


Always put spaces between the comma and the next var.


Ditto. Also, merge the 'gint' lines.


Ah okay, I may not have been confronted with those cases. I left a note because I was unsure if the copy had a good reason and wanted it bring it up. In general I see a lot of places in code where things are copied or new properties are added to objects (such as in line 89 above), and these kinds of runtime mutations can have significant performance penalties.


22
О себе

Контакты
munimaev@yandex.ru
git
г. Сакнт-Петербург

Навыки
Умения

Опыт работы/образование
Период радоты
Наименование организации
Дейтельность
----
Сопроводительное письмо
----
Портфолио
О портфолио

Наименовение проетка
Описание проекта
стек технологий
ссылки
https://www.iconfinder.com/iconsets/free-98-icons
https://www.iconfinder.com/icons/809119/architecture_building_office_residential_icon#size=128

Иконки
24 - 16
36 - 24
48 - 32

=====================
О себе
Я web-frontend разработчик. И смысл своей работы вижу в том, что бы облегчать жизнь пользователям в решении их задач. Интерфейс должен быть удобным, понятным, предсказуемым, работать быстро и, в идеале, приносить радость.
---------------------
Контакты
munimaev@yandex.ru
г. Санкт-Петербург
Портфолио
Сопроводительное  письмо
Резюме
github
---------------------
Навыки
JavaScript, jQuery
HTML, CSS
PHP,
SQL,
Photoshop, GIMP
---------------------
Умения
Appache, SSL
Git
Angular.js
React.js
Java, C/C++
---------------------
Опыт работы

Middle/Senior web developer
2013 - по настоящее время / ЗАО "Банк Советский" (1200 сторудников)
Банк занимает 106 место в рейтинге банков России на 20.06.2016
Разработка и поддержка внутренних web-приложений:
Интерфейс для подачи заявок на кредит.
Интерфейс для IP-телефонии.
Веб приложения для работы с API биржей кредитов.
Корпоративный портал.
Сайты дочерних компаний.
Шлюзы для приемки платежей.
Настройка веб-серверов.

Специалист по рекламе
2011 - 2012 / Агентство недвижимости "Абсолют"
Автоматизация процессов связанных с рекламой, создание нескольких сайтов-визиток.
--------------
Образование

Курс по Angular.JS
2016 http://learn.javascript.ru/

Курс по Node.JS
2015 http://learn.javascript.ru/

Санкт-Петербургский Государственный Университет
2013 - 2015
Математико-механический факультет, кафедра Математического обеспечения и администрирования информационных систем

Санкт-Петербургский государственный лесотехнический университет им. С.М. Кирова
2006 - 2011
Факультет лесного хозяйства, кафедра лесной таксации и лесоустройства. Тема дипломной работы - Применение беспилотных летательных аппаратов для лесной таксации

----------

Качесвта
Креативность: визуальный дизайн всех создаваемых приложений был разработан или адаптирован мной.
Инициативность: создал свою информационную базу (аналог wiki) и авто документацию по основному проекту.
Усердие: один подробно заполнил всю информационную базу и написал комментарии для автодокументации для проекта. Размер папки с исходниками проекта более 25 МБ.

=====================

Портфолио
Некоторые мои проекты, написанные для себя. Писались либо с целью изучения новых технологий, либо вдохновлённые новым увлечением, либо по обеим причинам сразу.

1-------

Игра на canvas. Машина в лабиринте.
Посмотреть в живую
github
Небольшая игрушка, написанная с целью изучить коллизию геометрических фигур.
[] canvas (pixi.js)
[] математика

2--------

Коллекционная  карточная игра Naruto
Посмотреть видео
github
Реализация веб-варианта настольной карточной игры Naruto TCG. Естественно, с блэк-джеком. Это как Hearthstone, только по одноименному аниме-сериалу про ниндзя.
[] node.js - сервер
[] express - веб-фреймворк
[] socket.io - веб-сокеты
[] jQuery.js
[] CSS3 transform-origin
[] анимация

4--------

Конструктор армии для Warhammer 40000
Посмотреть в живую
github
Конструктор армии для настольного варгейма Warhammer 40000. В текущей реализации, позволяет полностью собрать и просчитать стоимость в игровых очках армию Темных Ангелов.
[] jQuery.js

5--------

Конструктор юнитов для Warhammer 40000
Посмотреть в живую
github
Конструктор отрядов для конструктора армий для настольного варгейма Warhammer 40000.
[] Angular 1.5

7--------

Конструктор боевых сцен для Dungeons and Dragons
Посмотреть в живую
github
Конструктор позволяет создавать боевые сцены и управлять ими.
[] jQuery
[] CSS3

6--------

Карточки заклинаний для Dungeons and Dragons
Посмотреть в живую
github
Наборы карт подготовленные для печати.
[] React.js
[] Верстка под печать

3--------

Настольная игра "Колонизаторы" на CSS3
Посмотреть в живую
github
Проект задумывался, как очередной перенос настольной игры "Колонизаторы" в веб. В репозитории лежит большая часть всего, что должно быть на стороне клиента.
[] jQuery.js
[] CSS3 transform-origin


@curtisman
> Write a class where it's ctor is your process initialization code.
Add a global instance of the class so that the ctor will be call when process initialize.

@obastemur
> and how could we set the priority against object ctors. There is no such a thing.

*object ctors without `any priority.`

What we `had` needed there is to put the initialization of process to the end.

> Use the INIT_PRIORITY macro (which is __attribute__((init_priority(x)))) to set the initialization order of when the ctor will be call related to other global initialization code.

When you put init_priority, that would be initialized before everything else no matter what. Repeating: We had need to init last. So we had to set priority to others. Not to this one.


gcal_event_widget_new() already returns a GtkWidget*, no need to cast it.


As mentioned in #721 this code is actually worse than before. I'm not sure there is any better way for testing this than looking at LZMA_VERSION.


Build https://app.shippable.com/runs/5767fbb91e09560c00f104a9 failed and I believe the issue is here (indentation) https://github.com/ansible/ansible/commit/8218591fecf124f3bb8db22030959e235b450f02#diff-b893b8b8319cd4518208dd4fe53f3d7bR368


Really, if it has to do with ptr argument, it should be fixed with this commit. May be it has to do with missing prototype (i.e. missing argttypes and restype settings) for CMS_get0_type function. See commit
0b71a49 may be it helps.


This code adds 7 columns every time an event is added. This is wrong. This should be done only once inside init().


The following doc(s) have been deployed to the [allenfearp](https://allenfearp-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/scope-of-support](https://allenfearp-static-docs.pantheonsite.io/docs/scope-of-support)


This is an internal implementation detail, you should not export the grid.

Instead, add a 'gcal_week_header_remove_event (header, event);' function.


Thanks. I use only `counsel-ag` (not `counsel-git-grep` or `counsel-git`). So your first suggestion works for me.


I think this part is incorrect. You need at least 2.7 because you use string.format calls without positional arguments in your code, which did not exist yet in Python 2.6.


Use 'gcal_week_header_remove_event ()' here instead of iterating the children.


Ditto.


aynı şekilde. örneğin; GetFibonacciNumberByIndex


Yes.  Is that giving you a compiler warning, or are you compiling it in your head? :-)

I am not getting compiler warnings when a data type is converted implicitly to a type with lower resolution, even with -Wall, so I am not noticing these cases (I just fixed a bunch that @lpugin sent from his XCode compiling).

I just added an explicit conversion to bool: https://github.com/rism-ch/verovio/commit/1c0f510aa39654fb40bcca5cd5d46328a50b34f1


This is how completion works in master branch. :(


ingilizce lütfen


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 1.4.0-nes6-0.1+55](http://build.prxm.it:8811/viewLog.html?buildId=7527&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **FAILURE**
Summary: Tests failed: 10, passed: 268 Build time: 00:13:27
### Failed tests
```
GlobalSetupFixture: Jarvis.DocumentStore.Tests.JobTests.Queue.QueueHandlerTests.verify_id_is_opaque_and_not_contains_blob_id: <no details avaliable>

GlobalSetupFixture: Jarvis.DocumentStore.Tests.JobTests.Queue.QueueHandlerTests.verify_job_filter_by_tenant_id: <no details avaliable>

GlobalSetupFixture: Jarvis.DocumentStore.Tests.ProjectionTests.DocumentDescriptorProjectionTests.adding_twice_same_format_overwrite_format: <no details avaliable>

GlobalSetupFixture: Jarvis.DocumentStore.Tests.ProjectionTests.DocumentDescriptorProjectionTests.should_add_format_to_document: <no details avaliable>

GlobalSetupFixture: Jarvis.DocumentStore.Tests.ProjectionTests.DocumentDescriptorProjectionTests.should_create_a_documentDescriptor: <no details avaliable>

GlobalSetupFixture: Jarvis.DocumentStore.Tests.ProjectionTests.DocumentDescriptorProjectionTests.should_deduplicate: <no details avaliable>

GlobalSetupFixture: Jarvis.DocumentStore.Tests.ProjectionTests.DocumentDescriptorProjectionTests.should_deduplicate_a_document_with_same_content_and_handle: <no details avaliable>

GlobalSetupFixture: Jarvis.DocumentStore.Tests.ProjectionTests.DocumentDescriptorProjectionTests.should_deduplicate_twice: <no details avaliable>

GlobalSetupFixture: Jarvis.DocumentStore.Tests.ProjectionTests.DocumentDescriptorProjectionTests.should_remove_handle_previous_document: <no details avaliable>

GlobalSetupFixture: Jarvis.DocumentStore.Tests.ProjectionTests.DocumentDescriptorProjectionTests.should_remove_orphaned_document: <no details avaliable>

```


done, thanks


Add space between 'start', '==' and '-1'.


But now you have a strange treatment of co-supervisors. They are all promoted to supervisors, while it was only necessary for the first page. The difference between supervisors and co-supervisors still exist on the level of the defense since you can see it on the announcements of the defenses...


Yes that's what I've been doing...


1. renamed to "assert_for_status_code_and_content_type"
2. now it will try to validate for the default content-type. I thought about declaring a default HTTP response code (200) and validating against it, but I'm not sure about this approach
3. moved to separate function "validate_content_type", reusing this in "assert_for_status_code_and_content_type"


nope, go ahead


yeah, my fault, I didn't realize that I'm already able to access obtained cookies


done


thanks!


Let's remove this block, screen controls are cool but for the tutorial I want to keep to joysticks as it teaches control editor operations.


Please don't. This will cause UI hangs at entirely unpredictable moments (because the service is accessed from various very low-level call sites).


Already fixed.


@bcremer Thanks for looking into this. I don't know how exactly you've tried to reproduce this, but I am still able to do so.

Here are the exact steps I used to reproduce the issue:

1. `git clone https://github.com/shopware/shopware.git -b 5.2 --single-branch`
2. `git checkout 9138d7ef7d00ec7ee6b61e80f904de2cc653c30d` (this commit)
3. Install shopware as [described in the `README.md`](https://github.com/shopware/shopware/tree/29cbe53bec6c3306ea56949362360db632e2f8fc#installation-via-git) (starting from step 2)
4. Open the shopware backend and log in
5. Click on *Items* > *Overview*
6. Edit the first item in the list, which should be *Spachtelmasse (SW10239)*
7. Change e.g. the name of the item and click on *Save Item*

The result is an error notification: `An error occurred while attempting to save the item:Missing value for primary key id on Shopware\Models\Customer\Group`. The exception is also logged:
```
exception 'Doctrine\Common\Proxy\Exception\OutOfBoundsException' with message 'Missing value for primary key id on Shopware\Models\Customer\Group' in /var/www/Shopware-test/vendor/doctrine/common/lib/Doctrine/Common/Proxy/Exception/OutOfBoundsException.php:40 Stack trace:
#0 /var/www/Shopware-test/vendor/doctrine/common/lib/Doctrine/Common/Proxy/AbstractProxyFactory.php(125): Doctrine\Common\Proxy\Exception\OutOfBoundsException::missingPrimaryKeyValue('Shopware\\Models...', 'id')
#1 /var/www/Shopware-test/vendor/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php(2707): Doctrine\Common\Proxy\AbstractProxyFactory->getProxy('Shopware\\Models...', Array)
#2 /var/www/Shopware-test/vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php(273): Doctrine\ORM\UnitOfWork->createEntity('Shopware\\Models...', Array, Array)
#3 /var/www/Shopware-test/vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php(491): Doctrine\ORM\Internal\Hydration\ObjectHydrator->getEntity(Array, 'r')
#4 /var/www/Shopware-test/vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php(165): Doctrine\ORM\Internal\Hydration\ObjectHydrator->hydrateRowData(Array, Array)
#5 /var/www/Shopware-test/vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/AbstractHydrator.php(147): Doctrine\ORM\Internal\Hydration\ObjectHydrator->hydrateAllData()
#6 /var/www/Shopware-test/engine/Library/Doctrine/ORM/Persisters/Entity/BasicEntityPersister.php(976): Doctrine\ORM\Internal\Hydration\AbstractHydrator->hydrateAll(Object(Doctrine\DBAL\Driver\PDOStatement), Object(Doctrine\ORM\Query\ResultSetMapping), Array)
#7 /var/www/Shopware-test/engine/Library/Doctrine/ORM/Persisters/Entity/BasicEntityPersister.php(1779): Doctrine\ORM\Persisters\Entity\BasicEntityPersister->loadCollectionFromStatement(Array, Object(Doctrine\DBAL\Driver\PDOStatement), Object(Doctrine\ORM\PersistentCollection))
#8 /var/www/Shopware-test/vendor/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php(2835): Doctrine\ORM\Persisters\Entity\BasicEntityPersister->loadOneToManyCollection(Array, Object(Shopware\Proxies\__CG__\Shopware\Models\Article\Detail), Object(Doctrine\ORM\PersistentCollection))
#9 /var/www/Shopware-test/vendor/doctrine/orm/lib/Doctrine/ORM/PersistentCollection.php(697): Doctrine\ORM\UnitOfWork->loadCollection(Object(Doctrine\ORM\PersistentCollection))
#10 /var/www/Shopware-test/vendor/doctrine/orm/lib/Doctrine/ORM/PersistentCollection.php(214): Doctrine\ORM\PersistentCollection->doInitialize()
#11 /var/www/Shopware-test/vendor/doctrine/collections/lib/Doctrine/Common/Collections/AbstractLazyCollection.php(49): Doctrine\ORM\PersistentCollection->initialize()
#12 /var/www/Shopware-test/vendor/doctrine/orm/lib/Doctrine/ORM/PersistentCollection.php(462): Doctrine\Common\Collections\AbstractLazyCollection->count()
#13 /var/www/Shopware-test/engine/Shopware/Components/Model/ModelEntity.php(325): Doctrine\ORM\PersistentCollection->count()
#14 /var/www/Shopware-test/engine/Shopware/Components/Model/ModelEntity.php(204): Shopware\Components\Model\ModelEntity->getArrayCollectionElementById(Object(Doctrine\ORM\PersistentCollection), 1028)
#15 /var/www/Shopware-test/var/cache/production____REVISION___/doctrine/proxies/__CG__ShopwareModelsArticleDetail.php(894): Shopware\Components\Model\ModelEntity->setOneToMany(Array, '\\Shopware\\Model...', 'prices', 'detail')
#16 /var/www/Shopware-test/engine/Shopware/Models/Article/Detail.php(567): Shopware\Proxies\__CG__\Shopware\Models\Article\Detail->setOneToMany(Array, '\\Shopware\\Model...', 'prices', 'detail')
#17 /var/www/Shopware-test/var/cache/production____REVISION___/doctrine/proxies/__CG__ShopwareModelsArticleDetail.php(463): Shopware\Models\Article\Detail->setPrices(Array)
#18 /var/www/Shopware-test/engine/Shopware/Components/Model/ModelEntity.php(55): Shopware\Proxies\__CG__\Shopware\Models\Article\Detail->setPrices(Array)
#19 /var/www/Shopware-test/var/cache/production____REVISION___/doctrine/proxies/__CG__ShopwareModelsArticleDetail.php(872): Shopware\Components\Model\ModelEntity->fromArray(Array, Array)
#20 /var/www/Shopware-test/engine/Shopware/Components/Model/ModelEntity.php(133): Shopware\Proxies\__CG__\Shopware\Models\Article\Detail->fromArray(Array)
#21 /var/www/Shopware-test/engine/Shopware/Models/Article/Article.php(1106): Shopware\Components\Model\ModelEntity->setOneToOne(Array, '\\Shopware\\Model...', 'mainDetail', 'article')
#22 /var/www/Shopware-test/engine/Shopware/Components/Model/ModelEntity.php(55): Shopware\Models\Article\Article->setMainDetail(Array)
#23 /var/www/Shopware-test/engine/Shopware/Controllers/Backend/Article.php(1392): Shopware\Components\Model\ModelEntity->fromArray(Array)
#24 /var/www/Shopware-test/engine/Shopware/Controllers/Backend/Article.php(326): Shopware_Controllers_Backend_Article->saveArticle(Array, Object(Shopware\Models\Article\Article))
#25 /var/www/Shopware-test/engine/Library/Enlight/Controller/Action.php(159): Shopware_Controllers_Backend_Article->saveAction()
#26 /var/www/Shopware-test/engine/Library/Enlight/Controller/Dispatcher/Default.php(523): Enlight_Controller_Action->dispatch('saveAction')
#27 /var/www/Shopware-test/engine/Library/Enlight/Controller/Front.php(226): Enlight_Controller_Dispatcher_Default->dispatch(Object(Enlight_Controller_Request_RequestHttp), Object(Enlight_Controller_Response_ResponseHttp))
#28 /var/www/Shopware-test/engine/Shopware/Kernel.php(157): Enlight_Controller_Front->dispatch()
#29 /var/www/Shopware-test/vendor/symfony/http-kernel/HttpCache/HttpCache.php(487): Shopware\Kernel->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#30 /var/www/Shopware-test/engine/Shopware/Components/HttpCache/AppCache.php(255): Symfony\Component\HttpKernel\HttpCache\HttpCache->forward(Object(Symfony\Component\HttpFoundation\Request), true, NULL)
#31 /var/www/Shopware-test/vendor/symfony/http-kernel/HttpCache/HttpCache.php(258): Shopware\Components\HttpCache\AppCache->forward(Object(Symfony\Component\HttpFoundation\Request), true)
#32 /var/www/Shopware-test/engine/Shopware/Components/HttpCache/AppCache.php(103): Symfony\Component\HttpKernel\HttpCache\HttpCache->pass(Object(Symfony\Component\HttpFoundation\Request), true)
#33 /var/www/Shopware-test/shopware.php(101): Shopware\Components\HttpCache\AppCache->handle(Object(Symfony\Component\HttpFoundation\Request))
#34 {main}
```

If I repeat the same steps, but do a `git checkout 133e31109a9debea55500499af139802bcf82293` (the parent of this commit) in step 2, there is no error notification and the item is saved as expected.

I don't think this bug is related to my PHP or MySQL version, but just for reference: I run `PHP 5.6.20` and `MySQL 5.7.10 Homebrew` on `OS X 10.11.3`.


I propose another way to calculate the width of the event. Instead of checking the start and end, you can do it like this:

```
start = g_date_time_get_day_of_week (event_start_dt);
width = g_date_time_difference (event_start_dt, event_end_dt) / G_TIME_SPAN_DAY;

if (start + width > 7)
  width = 7 - start;
```


well, I found and implemented xsd validation. it works for validating user response against an xsd file, but I couldn't create a proper xsd for "issues" response as any xsd generator I tried seems to generate a schema that fails validation


Either way is fine with me. But I agree that the entire background should be white.


Why not `!m_comment.empty()`? That returns a bool without the need to cast it...


Je pensais que `express-session` était déjà une dépendance de `castor-core`.
Du coup, je ne pense pas que ce soit utile d'en faire une dépendance d'`ezmaster`.

En parlant d'utile, `async` est censé être accessible via `utile`, donc `async` n'a pas besoin de figurer dans `package.json` non plus (moyennant l'ajout de `utile.` avant `async` partout où il est utilisé.

De même, `express` est une dépendance de `castor-core`, et il n'est jamais chargé dans le code d'`ezmaster`.

Pareil pour `ejs`.

Et d'ailleurs, [biHound](https://www.bithound.io/github/Inist-CNRS/ezmaster/master/dependencies/npm#filter-unused-dep) signale 9 dépendances non utilisées (il n'est pas toujours dans le vrai: `castor-core` est signalé comme inutilisé, ce qui est faux). Mais ça vaudrait le coup de vérifier pour:
- mongodb-querystring
- jbj-parse
- http (étrange)
- fs (étrange aussi)
- ent


These methods get called during testing


I'm just compiling it in my head.


Yep, makes sense. In general, the polygon and line styles are likely more efficient in that regard, because they process features as a "stream": they evaluate each feature as it comes in, generate any GL geometry, add it to the VBOs, and then don't need any state maintained about the feature after that, which means they can reuse one-time allocated objects for some of the intermediate steps (example: this `feature_style` property is [reused for each feature](https://github.com/tangrams/tangram/blob/master/src/styles/style.js#L197)).

By contrast, the points and text styles need to collect all the *potential* features before they construct the geometry, because they need to run the collision process against everything at once. This means they are more likely to hold on to references or make copies of objects. These styles were modified from the original polygons/lines pipeline to have this behavior, and at this point they've diverged in behavior enough that I'm sure there is a better way (and the way they are subclassed is awkward, it may make sense to have a new
base class/interface, or two separate pipelines).


Fixed in 31be008


good


You can simplify this to

```
d1 d2 SubsetQ
d2 d1 SubsetQ
and
```


Yeah I see, ah well. If it functions right? :)


@sampathweb Sorry, I forgot to tag you.


Para el formulario y los radios que se marcaban todos, lo que pasaba es que el atributo "name" tiene que ser el mismo para todos, en este caso por ejemplo pondrías a todos el name "size" (porque se están eligiendo talles).

Acá dejo el código con el ejemplo para que lo mires y lo puedas corregir:

`  <form>
 <fieldset>
 +        <legend>talle de remera</legend>
 +

 +          small
 +          <input type="radio" name="size" id="s" value="small">
 +

 +

 +          medium
  +         <input type="radio" name="size" id="m" value="medium">
 +

 +

 +          medium
  +         <input type="radio" name="size" id="l" value="large">
 +

 +        <select>
 +          <option>1</option>
 +          <option>2</option>
 +        </select>
 +     </fieldset>
 +     </form>`


Revert to prev. version!


Pardon, je n'avais pas vu ce commit.

À part pour `socket.io`, je ne vois pas l'intérêt d'ajouter ces dépendances.

`http` et `fs` font *a priori* partie de nodejs.

Pour le reste, voir [mon commentaire du commit suivant](https://github.com/Inist-CNRS/ezmaster/commit/094d1aa96807df17c67241a7883a5567e12dde9d#commitcomment-17935254).


Yeah, we're addressing this in #418.

On Mon, Jun 20, 2016 at 9:17 AM, Johannes Wienke <notifications@github.com>
wrote:

> This really gives some room for troubles. Due to autocrlf I constantly
> have changes on these files. The global option doesn't event change
> anything here due to the local gitattributes.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/jackhumbert/qmk_firmware/commit/d9e4dad0a828a8a904f44dda090a4d6d08fe2948#commitcomment-17933614>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAIod_iqx3Y4q68a6gOEUmVh1wa_aUFIks5qNpL_gaJpZM4I2-kL>
> .
>


What's the license of this?


Does not retroactively apply to characters that entered broken state.


any update on the php5?


thanks for your great help :)


I know the name was my suggestion, but maybe we should go for `EqualQ` for systematics sake?


What's with php5?


This fails to compile on Xcode 7.3.1 due to not finding std::round in this context.
Changing it to use JUCE's roundToInt makes it work.


@eecsu, @mpilosov has updated the linearMap and nonlinearMap examples. If you could push the updates for the FEniCS examples that would be great. Thanks.


very good, Rado ! Thanks for the fix !


En el HTML del form, revise el porque no funcionaban los radios y quedaban seleccionados todos.
Radio tiene el atributo "name" y estos tienen que tener el mismo nombre todos, en este caso por ejemplo se puede llamar name="size".

Dejo el ejemplo acá para que puedas mirarlo y corregirlo.

`  <form>
      <fieldset>
        <legend>talle de remera</legend>

        small
        <input type="radio" name="size" id="s" value="small">

        medium
        <input type="radio" name="size" id="m" value="medium">

        medium
        <input type="radio" name="size" id="l" value="large">
    </fieldset>
</form>`


Of course :+1:
I saw it so I mentioned it right away.


antonellaclavero commented on c8ad31b 6 minutes ago
En el HTML del form, revise el porque no funcionaban los radios y quedaban seleccionados todos.
Radio tiene el atributo "name" y estos tienen que tener el mismo nombre todos, en este caso por ejemplo se puede llamar name="size".

Dejo el ejemplo acá para que puedas mirarlo y corregirlo.

`    <fieldset>
        <legend>talle de remera</legend>

        small
        <input type="radio" name="size" id="s" value="small">

        medium
        <input type="radio" name="size" id="m" value="medium">

        medium
       <input type="radio" name="size" id="l" value="large">
</fieldset>`


Hey, I found a bug that's unrelated to this commit, would you be able to enable Issues for this report so it's easier to report stuff? Either way, the issue is that you changed the tutorial NPC in Cherrygrove from an old man to the trainer sprite, but if you enter his house, he's an old man again, and his dialog inside the house will also need to be changed because he mentions that he was once a "wee lad", which wouldn't make sense if his sprite was changed.


Needed to update .gitmodules to point to your repository. Should be fixed with a82f555.


develop (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1667 of 1667 tests passed, 0 test warnings)

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


No es necesario el uso de p en el form :)


You should also check if old_year != new_year (it can be setting the same week for different years.)


develop (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1666 of 1667 tests passed, 0 test warnings)

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


http://jenkins.clearskyteam.org/job/ClearSky/ Did you guys update it on jenkins? I now see you also use a new site but there are multiple branches from 0.15 php5 but i dont seem to find a download button or link?


This should be the first day of the week, not of the month.


http://bracp.carloshenrq.com/
http://forum.brathena.org/index.php/topic/23794-bracp-v011-beta/?p=122904


This should be the last day of the week, i.e. date_start + 7 (adding 2 months makes no sense.)


A E S T H E T I C S


#5737-Multistage-heating-coil-only-operates-at-full-flow (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1660 of 1660 tests passed, 0 test warnings)

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


What is the usecase for that? Is that a relevant aspect of the concept or a byproduct of the implementation?


documentation-fixes (michael-okeefe) - x86_64-MacOS-10.9-clang: OK (2211 of 2211 tests passed, 1 test warnings)

 * 1 test had: EIO diffs.

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


Can you give an example? I don't see how a participation process that is not announced or that does not document the results is useful.


Wish you a speedy recovery


Yes. Do you have a proposel for a better term? "spatial location" sounds very technical.


Damn, that should be [JBEAP-3898](https://issues.jboss.org/browse/JBEAP-3898).


Actually, Simon merged your pull request, so I've updated to point to that. See 05affb1.


By the way, have you considered adding a simple rockspec to your torch-util project to make it a Torch module and avoid modifying the path?

+ https://github.com/bamos/spg.torch/blob/master/spg-scm-1.rockspec
+ https://github.com/bamos/spg.torch#installation


Wow Steven! Nice Code, I am really astonished!


This breaks[at least one test](https://github.com/coduno/dpl/blob/master/spec/provider/gae_spec.rb#L12).


There are no code changes in TCPDF v6.2.12 and [6.2.13](https://github.com/tecnickcom/TCPDF/commit/009f2304c3a9c8152b89a5321a8825f250911dc3) and some changelog has been removed and consolidated in the README.md file in the latter. TCPDF v6.2.13 is stated to be the last release in the v6 series.

Many fonts are omitted to conserve size and can be taken from the [parent distribution](https://github.com/tecnickcom/TCPDF/tree/master/fonts). The ````[examples/lang](https://github.com/tecnickcom/TCPDF/tree/master/examples/lang)```` folder in the TCPDF distribution is now in it's own folder here in  the ````webroot````.


Why are you not using `private Integer age;` ?
But really nice bois


And it makes README.md inconsistent.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 147](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5125&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


You could add
```
return fmt.Errorf("environment variable must be specified as name=value (file %q)", file)
```

So it is easier to debug.


What about `xterm-color`? It would fail here.


ditto, `"(file %q)", file`


Sure, changing it to 20 is indeed a better option. Thanks for the comment.

Also, calm down... it's just a process viewer, it's not the end of the world. Keeping things in perspective leads to a healthier life. :)


ty


@blueclock would it make better sense to fix the letter spacing hard-coded in skeleton [code](https://github.com/Mango-information-systems/skeleton-framework/blob/mango-is/src/core/_typography.css), so that it's applied everywhere?


This project was builder for a client and is nos possible to replicate,
because,  the architecture is very particular.

Sorry,

Regards,
Emerson Perdomo.

On Sun, Jun 19, 2016 at 10:25 PM, jaheem <notifications@github.com> wrote:

> Dear Sir can past read me How to deply this project
>
> THank you
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/emerballen/Streaming_Twitter_BigData/commit/b0da8635cba23fbdd5c852d1ff3a7fdf9c374f49#commitcomment-17928458>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AGYygoPN_fHBjDScxrm6a0rRFpbXH8Tcks5qNggQgaJpZM4I5VQa>
> .
>


Because I only store the minimally sufficient number of functions in my head :-)

Fixed in https://github.com/rism-ch/verovio/commit/a133d49fe408559ffc141619a68f8d7a612c876d


Woops. Not sure why I didn't included this. This is from android. I will add the header ASAP.


I wasn't aware of that being a valid `TERM` value. So back to something like `return term != null && term.startsWith("xterm");`?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 147](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5125&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:05:47


test


- [ ] yes


MergeEnvs is only called in this file, so it does not have to be exported (MergeEnvs -> mergeEnvs).

I see it was already like that before...


να ξέρεις ότι σε μερικές περιπτώσεις είναι καλό να περιμένεις να γίνει manually resolve ένα http request πριν γίνει render το component (δεν ξέρω βέβαια αν θα πέσετε σε τέτοια περίπτωση) αλλά για να μην ψάχνετε ψίλους στα άχυρα μπορείς να κάνεις το εξής

```
var defer = $q.defer();
ProfileService.getProfile().then(
                     function (d) {

                         model.myprofile = d;
                         defer.resolve(model.myprofile);
                         $log.info('Crop name: ' + model.myprofile.crop.identifier);
                     },
                     function (errResponse) {
                         console.error('Error while fetching Profile');
                     }
                 );
return defer.promise;
```


When you specify init_priority, it will be call before other global initialize code that *doesn't' have init_priority specified, but it will be in order the priority number specified.  You can use that to make sure that your dependency is initialized before you.

Looking at what is in `AttachProcess`, a lot of the code in there doesn't really have any dependencies, and make more sense as global ctor anyways (e.g. `ValueType::Initialize`, `ThreadContext::GlobalInitialize`).

Stepping back: looking at the assembly of the function static code you show on for clang, it is as I described (check flag first before lock), and seems efficient enough.  I have verified that VC compiler generated the same code pattern as well.   So let's hope all self-respecting compiler will generate the same code :)  So we can just take that solution for now.

But for the long term architecturally, IMO it would be cleaner if we can move to global ctor to do most of these initialization instead of guarding all Jsrt API with validates.


                if (target->GetMap()->IsDungeon())
                {
                    uint32 spellId;

                    Difficulty diff = target->GetMap()->GetDifficulty();
                    if (diff == RAID_DIFFICULTY_10MAN_NORMAL || diff == RAID_DIFFICULTY_10MAN_HEROIC)
                        spellId = 70311;
                    else
                        spellId = 71503;

                    target->CastSpell(target, spellId, true, nullptr, this);
                }
                return;


Have you thought about setting up some basic tests and running on @travisci? You can easily do deployment/releases on tagged commits in master branch etc...


            if (target->GetMap()->IsDungeon())
            {
                uint32 spellId;

                Difficulty diff = target->GetMap()->GetDifficulty();
                if (diff == RAID_DIFFICULTY_10MAN_NORMAL || diff == RAID_DIFFICULTY_10MAN_HEROIC)
                    spellId = 70311;
                else
                    spellId = 71503;

                target->CastSpell(target, spellId, true, nullptr, this);
            }
            return;


            case 70308:                                     // Mutated Transformation (Putricide)
            {
                if (target->GetMap()->IsDungeon())
                {
                    uint32 spellId;

                    Difficulty diff = target->GetMap()->GetDifficulty();
                    if (diff == RAID_DIFFICULTY_10MAN_NORMAL || diff == RAID_DIFFICULTY_10MAN_HEROIC)
                        spellId = 70311;
                    else
                        spellId = 71503;

                    target->CastSpell(target, spellId, true, NULL, this);
                }
                return;
            }


ここ2行 controller_module(conn) は外で action_name(conn) が中にあるの気持ち悪いので揃えたほうがいい気がします（なお前回のレビューではスルーした模様）


Yea, I've used @travisci for sailfish / salmon to run automated tests on commits etc. I haven't dug much yet into how to do deployment or releases yet.  Specifically, the releases are built under Holy Build Box to ensure maximum compatibility, so I'd need some sort of travisci => docker setup etc.


Did you look at https://www.google.com/recaptcha/intro/index.html?


Isn't this removing the performance optimizations done in https://github.com/Huddle/Resemble.js/commit/a1a41fc5260a1ba7567173ed3137ce850e32840a ? :)


Sample PDF Outputs are in the [docs](https://github.com/apmuthu/phpLeaveManager/tree/master/docs) folder.