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


you missed another one further down in the same file


You have to remove branch !
Because on github url of openPli there is no branch in gstreamer git
And alsob better is to use another URL for gst-plugin-subsink -->  git://github.com/christophecvr/gstreamer1.0-plugin-subsink.git;protocol=git


This is bad. A failure in the first test will prevent remaining tests from being run.

Make several test methods with meaningful names instead. Figuring out what went wrong is difficult when you only have a call with files named test1, incl, test4, etc.


ок


Plasma5 install doesn't work properly for me, one of the sub-menus doesn't allow exit back.


OK


This adds support for the upsBasicOutputStatus variable (online, on battery power etc.)


Can you explain (not in code, but in the GH conversation) how these lines are formed?  I don't understand what the "(100)" means.


Hi,

Do you by any chance know what the original sepolicy was on the stock rom?


Done in 4dadc28


Blocked on #360.


using the production constants is tempting because it saves on duplication but is actually a common pitfall
the problem is that changing the constant will also change the test, therefor any change in the production code (on purpose or not) will not be detected by a failing test


Is this likely to fix the stabilize "bite" issue that I see?


Did you talk about this with @zidarsk8?


On line 242, you need to unify the condition with something.

You could add Boolean to the type environment by adjusting `buildTypeEnvironment` on line 573 and unifying against that.

As written, the typechecker presently does its best to infer the type of the conditional expression, but doesn't impose any constraint on it.


Fixes #113 , too.


You are constructing the whole matrix but only the diagonals are of interest. You could make this a lot more efficient by only taking into consideration the diagonals and computing their values. Because for every level (side length) you have only 4 corners, then you could only generate 4 numbers each time, correct? After generating those 4 numbers of interest, you only have to check which one of them (if any) are prime and then increment a counter or two. After that, you can forget about those 4 numbers and move on to the next 4 and so on and so forth.

You can find a direct relation between the numbers on each corner of your `Nth` degree matrix.
For example, if `n` =  3 -> side length = 3
```
5 4 3
6 1 2
7 8 9
```
Starting from 3, you know that the next corner will be `3 + sideLength -1` -> `5`.
Then, to 7 you get by adding `5 + sideLength -1`
To 9, `7 + sideLength-1`.
=> 3 steps

After 3 steps, you increase your side length and start off from another corner which you could also easily compute.


Well done otherwise


Hello


commit title is too long


On master, i get an Caught exception: SyntaxError: Unexpected token else.
Did i broke something, or is it because of the merges of the forks?
Do you want me to fix it?


You could use `0o666` for better readability.


It's just confusing because the comment does not really fit the code.


Luckily `repartitionAndSortWithinPartitions()` is available also in spark 1.2.0


This closes yhoogstrate/segmentation-fold#5


I probably didn't need to change that one.


ah i thought you wanted to get rid of all required leading whitespace


Now it's rather ErlangModuleTextBuilder.


> I'd rather see these also having UPPERCASE names as that would clearly mark them as global things in the code below using them.

Will do.


Oh, so you're referring to
```ruby
       # Have to redefine should= here so we can sort the array that is given to
```
Yeah, I agree that's weird. But like I said, insufficient Ruby-fu on my end to determine whether this has merit or not.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1244 of 1244 tests passed)

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


ack


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 580](http://build.prxm.it:8811/viewLog.html?buildId=3528&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 264 Build time: 00:11:14


@alexdunn220 what?

Note: I'm probably going to delete this file, since it duplicates what I add in the changelog.


this is already covered here:

```C
evhtp_connection_t *
evhtp_connection_new(evbase_t * evbase, const char * addr, uint16_t port) {
    return evhtp_connection_new_dns(evbase, NULL, addr, port);
}
```


Hello
I'm a bit of a time to fining these packages for our RedHat server?
The Perl mods do not seem to all be easy to find.  Do you have a source you might be able to share?
Thanks
~Ben


If you want to take another look - I think I did everything you mentioned.


Does Maven provide a way to treat these directories as _patterns_? I had expected that the input directories specified here were _roots_ below which the plugin would search for `proto` files _recursively_, but reading the code now I see why it wasn't finding my files sitting in directories nested beneath such a root directory.


i don't see kernel source :(


Wow, @opadron you are forcing me into describing two vague preferences :sweat: .  Other's might have different opinions, though I'm trying to keep with the general trends here.

I would prefer not to have names in TODOs (I know we have a couple in the codebase) as this info can come from git blame, and I'm not sure what benefit we get from including the name in the code comment.

I would also prefer to have a more factual and less conversational/rhetorical tone in the code comment.  An example would be

    TODO provide an explanation of why we break away from normal Python naming conventions.

And then in your PR comment you could include

    Is this a case of the prevailing style argument made in pep8?  If so, exactly who's style is prevailing?

These are a bit subjective I know.


@MatthewHiscock  +1!


I'm mean:
loggin -> logging
"." at the end sentence (`... when logging the traceback).`)


Do you think you should be doing work in the initializer.

Especially if this is an object instance to be passed by composition. You may end up doing work that will never be used.

I think these would be better in a `def setup` method that returns the instance so you can do `Thing.new.setup` or something like that


Instead of using `explode()` you can pass `$dateValues` in third parameter here, it will be modified by reference with your matches.


@katkad have you seen the comment in line 14?


I really like this


yep


I believe we can also toggle a 'vjs-hidden' class, but this will work fine as well.


why? this is covered by a hook,

```
_evhtp_connection_eventcb(evbev_t * bev, short events, void * arg) {
    evhtp_connection_t * c = arg;

    if (c->hooks && c->hooks->on_event) {
        (c->hooks->on_event)(c, events, c->hooks->on_event_arg);
    }
```


I really like your reorganization and sorting


Why are you calling `.then` on an ElementArrayFinder without invoking some method on it first? This is typically how ElementFinders are used. What's the difference here?


reack


why? this is already covered:

```C
evhtp_connection_t *
evhtp_connection_new(evbase_t * evbase, const char * addr, uint16_t port) {
    return evhtp_connection_new_dns(evbase, NULL, addr, port);
}
```


Yes, I am running conpot with this commit for 11 days, no error occured. And there was modbus traffic since.


Learn?


ack


Got it, now it works.....new int[m+n]....Thanks a lot, will upload it in a final state along with the second problem.

     On Monday, July 20, 2015 5:28 PM, Cosma Alex <notifications@github.com> wrote:

 1) format your code (ctrl+shift+f)
2) put e.printStackTrace(); to see the full stack trace of your error
3) notice that the problem is not reading from the file but in the actual multiplication 'algorithm' at line 93.Have you tried debugging it?—
Reply to this email directly or view it on GitHub.


closes issue #1


This was definitely the logical end I was looking for. That this method would be housed by the `PronounceablePassword` class


Oh yeah, it took pretty a lot of time to find it :game_die:

@lakshmi-kannan was right with idea about different `gpg` versions :+1:

If someone will need to debug Travis-CI builds, here is Vagrant environment for that:
https://github.com/armab/travis-python
With Python, RabbitMQ and Mongo for st2.


In other words: show me the error. According to my knowledge, conpot/core/databus.py takes care of the template.


I use as example com.intellij.openapi.externalSystem.test.ExternalSystemTestCase#compile.May be assumed that the `compile` method used in Runnable. I missed it when changing.


Then ignore my comment :+1:


I also considered this an opportunity to produce a recursive methods whose base case was the whole password with length was met.


Try to revert to previous version of FFmpeg from working UMS version. If AR will be OK, definitely problem is FFmpeg but maybe between UMS commits there was also another update which can influence AR on specific renderer (changing file extension, changing renderer.conf as using custor AR options and custom paramerers etc...)
Btw. your log shows you are using MEncoder only. There is no try with FFmpeg engine.


This is a good candidate to be `final`


Seems that could have been done by splitting the measure and hiding the
barline, and that would be sufficient to handle these corner cases?

Anyhow, if we do allow this, I'd rather it not be too easy to create -
maybe require an explicit drag and drop to a specific note, and have double
click continue to add to the beginning of the measure.

On Mon, Jul 20, 2015 at 6:23 AM Werner Schweer <notifications@github.com>
wrote:

> There was an request on the german forum. My first reaction was that this
> will not make sense but someone showed and example: Schubert Impromptu
> Ges-Dur op. 90/3 bzw. D899/3:
> http://burrito.whatbox.ca:15263/imglnks/usimg/4/4d/IMSLP39751-PMLP02062-Schubert-Impromptu-Op90No3.pdf
> (last page in the middle).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/musescore/MuseScore/commit/c51fa84817da537fba95cad863b348d0651be8a0#commitcomment-12251469>
> .
>


Would be good to also demonstrate that even when the spam filtering config params are set, it delivers a mail without a spam header successfully.


are you ok returning this as a writable instance that someone can modify behind you?


If something other than -j is passed do you want to do a 'usage' here?


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


Первое, что я сделал - Изменил функцию mso_widget_create_form (добавил отступ для hint_widget).
function mso_widget_create_form($name = '', $input = '', $hint = '')
{
if ($hint)
	$out = '
' . $name . '' . $input . '
' . $hint . '
';
else
	$out = '
' . $name . '' . $input . '
';
return $out;
}

Второе. Удалил стиль в style.css.
div.admin-edit-widgets div.admin_widgets input,div.admin-edit-widgets div.admin_widgets select,div.admin-edit-widgets div.admin_widgets textarea{width:auto}

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


@RandomAltThing You mean on MCPC? I got a weird NullPointerException when I connect. xD I thought it was my client's problem.


Fixes ceylon/ceylon-spec/issues/1377.


@DefinitlyEvil I mean MCPE it can't find the server.


Why is this check needed? Isn't it enough to verify if the prefix starts with "https" ?


Looks good.


Let me propose using a utf8 ellipsis instead?
http://www.fileformat.info/info/unicode/char/2026/index.htm


ping @ThomasBurleson


Maybe it's better to declare and initialize the variable on the same line


You don't seem to have imported com.google.i18n.phonenumbers.Phonemetadata.PhoneMetadata anywhere?


I think it's time we move the JS to its own file, it takes some space in map.php


@DanielRuf: I've re-written the text describing usage (currently in English); Will need to translate the new text. Cheers. :-)


You've left some merge conflict in


Not according to [Oracles examples](http://www.oracle.com/technetwork/java/javase/documentation/index-137868.html#examples).


Based on [Egret engine](https://github.com/egret-labs/egret-core).


Don't you think it's better to send an error packet in this case and in the one above?
So the node type user could try to do something with this information? Eventually trying to tick again.


Maybe is better use "newComponentHash.fileName " like parameter to avoid createing the variable "relativeFileName"


sorry sorry sorry i have absolutely no clue what im doing please forgive me I'm learning

     On Monday, July 20, 2015 7:40 AM, Jose Diaz-Gonzalez <notifications@github.com> wrote:

 @alexdunn220 what?Note: I'm probably going to delete this file, since it duplicates what I add in the changelog.—
Reply to this email directly or view it on GitHub.


@RandomAltThing You may need to add a server manually.


@RandomAltThing What do you think I've done


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

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


Please try to fix it and commit Your changes.

We still miss tuxterm.


1.0 / in.w();


Why did you write this? 由于数据库 users 表中没有 **AUTO_INCRESEMENT** 字段，`mysql_insert_id()` 函数永远都会返回 0。这一条语句相当于 `if(true)`。验证用户名是否存在的功能在第 70-74 行已经有了。


No, it shouldn't. If the file had window's line endings, old_windows should be true, so the function would convert the new buffer to window's line endings (if needed). The only problem here is that if the file has mixed lines endings, it would convert them all to window's line endings.

The original problem I was trying to solve was that omnisharp always used its configured line endings, not the end lines used in the file, whenever it had to insert a new line. So it you were editing a unix file on a windows machine, or vise versa, the inserted line endings would be wrong.


Typo: it should be written as "nautical".


Not shrink but extend


@DefinitlyEvil I'm during the demo server I can't find it on PE.


Using*


I will import com.google.i18n.phonenumbers.Phonemetadata.PhoneMetadata and com.google.i18n.phonenumbers.Phonemetadata.PhoneMetadataCollection in stead of using Phonemetadata.PhoneMetadata or Phonemetadata.PhoneMetadataCollection.


hmm, we will need to add tests to ensure this doesn't happen again.


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


Even if there is a nice explanation of parameters, "script_name" and "string" variables look pretty bad. I would go for script_type instead of script_name for example. "string" variable can be replaced with script_name. Does it make sense?


I haven't done this because it's quite difficult to make it correct for all distributions; most users today seem to be using git hooks rather than Makefile integration to automate unit testing today anyway.

The problem is that /your/ distro calls these `nosetests3`; mine calls them `nostetests-3.3`. Which should "win" for the git repo?

I'd be happy to take a patch to try to detect the correct nosetests binary to use but I'm not going to merge something that hard codes one distro preference over another.

(and to be honest, one day soon, it'd be nice to just drop the whole Makefile mess entirely).


文字化けしてる・・・


Looks like something that should be merged upstream?


I'd rather not as people might not be utf-8 so it's bad enough in that case if we treat their strings as utf8, but if we inject utf-8 in that's even worse :)


Its good way of coding


Lol, I got it from elsewhere, didn't know it was kanged. Will remove it as I have separate trees anyways. Cool to have scary Alex Cruz on my case. Regards to DU team.


Yup, makefile.boot is now the only checked in file in the make directory...and makefile is in the .gitignore.

 `make all` could work just as `make top` does to build r3.  First make the makefile, then use that makefile's make all.  But then we ask where does it stop (make clean?  make install?  How big is makefile.boot anyway?)

I took the small step of thinking it might be nice to just say `make -f makefile.boot` and have it do as much automatic as it could...so we can get some experience with it.  So far I like that.  I'll let other people hammer on it a bit, because for now the things I hated most are taken care of and the scripts are less impenetrable...


Ok


Credit: http://filsinger.me/workflow/system-clipboard-with-tmux/


@lindamartin I added a comment in 2ceec78, I hope it's a bit clearer.


Hi, Thanks for the fix!

Just a few comments:
TypeTranslator.cpp:283 has an extra semicolon at the end.
QpidMarshal.h:73 has a warning that can be suppresssed with ( (int)native.length() ).

-Chug


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


Turning on HW Video Decoding does very little without the implementation of StageVideo to actually support GPU usage.

The HW Video Decoding flag works wonders on Chromeless (which has StageVideo enabled if the HW video flag is set to true), but does not appreciably affect the CPU usage on Video-js-swf which does not currently support StageVideo.


There is no point of hiding this already people are already leaked  the sandbox break.


Use the find_or_create_by instead


@Matoxxi
Kernel source  is declared here:
https://github.com/Omni-Cancro/android_device_xiaomi_cancro/blob/android-5.1/omni.dependencies#L28


nit: the package which provides `nuget` might not always be installed with Mono and since the repo already contains nuget.exe I think it's better to just use that one like in .cmd: `mono .nuget/nuget.exe ...`


you need to sign it with the same email as the one you are [using to commit](https://help.github.com/articles/setting-your-email-in-git/) (`mohamedhagag1981...` in your case)


I mean do not change the serialization of the basic existing member.
I have design the view concept to be extented by using the inheritance scheme not by adding members.

Using a map in the main View class allow user to store data in a unstructured way wit no strict convention and from my viewpoint does not help to provide a strict API to guide the user.

Remember that the sensor_width field is only an optional value that is required for a specific purpose and not for all the case.


Removing this was actually his idea.


space between > and s


Sorry, my bad.
I was always confused by this src/newsrc naming. Better to be called ` file ` and ` uncomp_block `.


@samdutton Was this tested?  We don't need a seconds title in the article when we have the one at the top and h1's are massive.

cc @petele


Hi @igor0511, please don't forget to use Translate() calls to translate all visible texts.


reack


These aren't technically working. They need to be named ```new?``` and ```create?```.  I think it's working because the application policy is taking care of the default implementation for you. You may want to double check those and make sure that's what you want. I'm about 99% sure you are good to go here though.


需要给出使用方法，例子，及文档。。。


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.771.b8ca26ef0ce3529fc9349e10f36171195dccaba1](http://build.palaso.org/viewLog.html?buildId=59831&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 380, ignored: 3 Build time: 00:03:52


When I try this rules some time ago I got many errors for our project. I create only one issue https://github.com/jscs-dev/jscs-jsdoc/issues/94
We need try again.

In common :-1: for `yandex` preset because codestyle doesn't specify how people should write jsdoc (google closure style, jsdoc3 / jsdoc2, jsduck, yui jsdoc or something else). I think this is project and tools specific rules.


An interesting idea that we can try is to add an extra condition to our Reduction array.
Right now it's Reductions[pv][imp][d][mc].
What if we try Reductions[gp][pv][imp][d][mc].
Where GP=Game Phase.  A bool if MG/EG.

The EG reductions array will be less aggressive compared to MG.


haha, this one is pretty cool though. =p


I'd like to use the definition of a prime number being defined over a principal ideal domain (Z) where any positive integer can be written as the products of primes in an essentially unique way.

I agree that they would return false, also. I can change it.


Because your question is on a commit rather than an issue (most likely won't be found by others) I'm just going to write a blog post about how to do this and I'll link it here.


Pan works properly now, although it's no longer bounded with the data.


Отступ, точнее перенос делается в css-стилях span.hint_widget {display: block;} Шириной здесь нельзя работать.

Я всё-таки хочу увидеть ошибку/недочет в админке своими глазами. Где (адрес страницы, виджет и т.п.)?


why not merge the two conditions ?


No worries :)


using a do while permit to remove this ugly break :(


@banjaxbanjo
No, it depend on RAT.

>  	only for UMTS; must be in the range of 0 to 511


This indentation is very bad!!!!!!!!!!!!!!! :-1:


There is a duplicate here. Is this intended?


Duplicate fields.


Empty line!


Duplicate fields GmailSaves


This is just a first test on Custom Post Types, might be useful to include a text field to customize the "textdomain" as asked in issue https://github.com/WebDevStudios/custom-post-type-ui/issues/167


It's a great idea for default module to create a default `CompilationRunner`


Duplicate field


Alright I supose


Because this was the first time that I pushed to NPM, and for some reason, the release number on NPM was 0.0.0, and I could not change it to 1.0.0 when publishing. If you have any tip, I can revert to 1.0.0 here and on NPM


Duplicate fields


I think `kernel_modules` or `linux_modules`, but I also don't remember it exactly...


What if jsdoc validations is turned on by setting a special option in jscs config and turned off by default?


OK


Given that you have now a dedicated class, I would move the code of the `FrontUtils` method to this one, and make the `FrontUtils` method use that `FrontNormalizer`. I would also make the `FrontUtils` method deprecated by adding this in its Javadoc:
```
@deprecated Use a {@link org.uma.jmetal.util.front.util.FrontNormalizer} instead.
```
You can then copy (and adapt) any test related to this method to the test class of the `FrontNormalizer`.

This is usually what I do when I identify a dedicated feature (that I expect to keep) and create its dedicated class. This way, using it in the old method is a small check to ensure that it actually does the same (the tests of the old method should still pass) and the deprecation add warnings which help to see where the old method should be replaced, so you can do all the changes and re-run all your tests to check that it still works well. Then, keeping the deprecated method is a way to maintain the retro-compatibility until the next major version, and the description gives to the users the explanation of how to replace it by a non-deprecated code.


LGTM


Sisi ça serait bien, j'y ai pensé en allant voir le site sur mobile. Le plus est petit c'est pas pratique.
Une ligne de JS et c'est réglé ;)


for the indenting take care about tabs - that's what screws indenting most of all times. And the forums complain about long check times, so I think they will kiss ya if you propose the patch :)


I'm really not keen on polluting the API like this. The main reason I
like kv as opposed to other key-value store packages is the cleanliness
of the API and this *2 business really turns me off.

I see that there are quite a few existing users, so I agree that breaking
the API is something to be avoided, but how about moving to a versioned
API package path (e.g. gopkg.in/kv.v1) to make it a) clear that you
intend to maintain the API as stable and b) easier to change major
API versions in the future without breaking clients.


97902934_ZoneMixing (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Ah I see now forgot about that 2G/GSM doesn't use psc


This looks great! I think one of my top favourite feature request with its roots in the Friendica era. Do I just use 'lang=fr' to include or exclude French (e.g.)? One concern though; is detecting the language of every item not resource hungry?


Specifically this one

![test](https://cloud.githubusercontent.com/assets/8170914/8779420/6100c1fa-2ed1-11e5-9e96-f8d3058ee92f.png)


Thanks Tim!


This implies that there can only be one sameAs relation. To allow multiple sameAs relations, this should read

    "sameAs": {
        "type": "array",
        "items": {
            "type": "string",
            "format": "uri"
        }
    }


PLEASE insert question description in source code and put filename understanding question.

[mojigyaku.cs]
Use 'string' class not 'String' class.
Use Upper-camel description in class 'mojigyaku'.
Delete unusual comment : line 21-34.

[patatokukashi.cs]
Modify same description by [mojigyaku.cs].
Let's challenge using 'StringBuilder' unusing 'char[] str'.

[mojirenketsu.cs]
Modify same description by [mojigyaku.cs].
Let's challenge unusing value '8' in line 11,15. It means using string.length from variable patoka and takushi.


Hi Thomas, just one question: did you process SingleMu or SingleMuon?
for this thread it seems the latter is the one to use moving forward

https://hypernews.cern.ch/HyperNews/CMS/get/muon-hlt/512/1/1/1.html


Has this been tested with SiK radios in bootloader mode and/or updating the bootloader? I recall there's some timing issue there


Well this fixes the cursor in form filed after closing the editor. So it makes the behaviour more consistent in this case. But wouldn't it be better to focus the form field after the ocntent of the edito is put into it, would it?


[![LGTM](http://lgtm.in/p/wsWvVe3vn)](http://lgtm.in/i/wsWvVe3vn)

[:+1:](http://lgtm.in/u/wsWvVe3vn) [:-1:](http://lgtm.in/r/wsWvVe3vn)


@pynicolas yes perfect! Thanks!


@mickeyren Let's remove empty files, especially those that reference the frontend.


hahaha "scary Alex Cruz"

Just funny that it has the same white space removed and all but OK man, whatever you say.

As long as we're not kanging petty shit, we're good dude.....


I'll just remove this note and submit the response code correction upstream.  Thanks


Sure. `org.eclipse.jdt.internal.codeassist.ThrownExceptionFinder.getThrownUncaughtExceptions()` is probably the way to go (or at least to look into further). By the way, does that NetBeans method return both checked and unchecked exceptions or just checked exceptions?

Another way to go about it is to have a look at the JDT compiler code for lambda expressions.


Hi Marino,

I have processed the sample /SingleMuon/Run2015B-PromptReco-v1/MINIAOD, SingleMu is just the label I have chosen for the xml file.


I do not believe your contribution broke something if everything is working for you. I have had this issue since upgrading > Pelican 3.4.0 so maybe something else is broken for me. It seems that `instance._summary = text_type(summary)` does not work for `clean_summary` and `summary = instance._summary` does not for `read_more_link`. I have found a work around by leveraging `summary = truncate_html_words(instance.content, SUMMARY_MAX_LENGTH)`.


no ack needed for generated commits


ack


ok, great, thanks!


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1244 of 1244 tests passed)

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


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


My only regret is all of this


Some immediate-type OptState might be also required for tracing (like it's done in ```Optim```). Basically, it's just about separating the state fields of OptController from the rest, but the assembly of such OptState could be delegated to the callback code as well.
I think passing the whole optcontroller under an agreement that the callback does not modify its state would already cover lots of possibilities. The user can still modify some the internals from within the callback, but then he would be on his own. The only "official" way to modify the state would be though the return value. At the moment only stopping (via true/false) comes to my mind, but, of course, something more than just boolean return type could be supported.


super !


According to [this](http://blog.travis-ci.com/2015-03-31-docker-default-on-the-way/) you shouldn't need to specify sudo any longer. (Though I don't see _This job is running on container-based infrastructure_ in the logs, so I guess the build is still on the old infrastructure)


Thanks, changed in d13fafa.


Yep, totally agree. I blindly assumed that nosetests3 was a standard.


@samjewell please have a look at the new links / dollars page / whatever. X


в константы


In some requests the `Host` field might already contain the correct string and the `prefix` can too have https, then adding it again will be redundant. Don't you think ?


first commit


можно сразу вызывать findViewById


Yeah, that totally makes sense ;). I'll do the update.


Wops used mecoder last and forgot to change it but if was messing up on some movies also so maybe update did it not sure roling back on official builds now most recent failed 5.2.1


зачем эта строка?


Ah, I see it. ```@bookmark``` should be ```bookmark```.

Let's review partials, and how local variables get passed to them on Tuesday. That and routes!

Levi


I'm not all that familiar with the `@XmlElement()` macros so I could be missing something implicit, but I don't see anywhere that `setAutoStart()` is called, nor how, if it is set in the .printer file, it gets read and assigned.  It looks like maybe we're missing calls to [un]marshall this in `HostProperties.java` as well?

How would I specify this property in the .printer file?  As a simple element, whose existence makes it `true`, like this:
```
    <AutoStart/>
```
or with a value of `true`, like this:
```
    <AutoStart>true</AutoStart>
```

I'm guessing the first one will be the correct answer, but, again, as I've not played with JavaBeans in a while, I'm a bit foggy


Is there an avalible iso of ye olde steamose brewmaster on here? :)


Can probably drop this entirely, or at least the JVOEGELE part.


typo?


Please tell me you didn't delete your branch because now my update went away.


Hahahahah I commented thid


D:
That explains a lot ...

-____-||


Ask a question link isn't working because issues aren't turned on :wink:


Nope, I didn't delete my branch. Go ahead and make your updates and then I'll add mine back in. My bad, I didn't realize you were working on it right now.


97902934_ZoneMixing (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed

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


This class is private to the PlanPrinter, so making a copy of the map is not really necessary


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


・機能項目リストのように書けたのであれば、各項目名称を章立てのタイトルとして各機能に対する詳細を書いてみてください。それが機能仕様書となります。
・ざっくり画面は各種四角の箱が何を指しているか(画面のパーツなのか画面の説明なのかそれ以外なのか)不明です。凡例を書くか、実際の画面イメージを書いてみましょう。
　→画面プロトタイプにはpancil(http://pencil.evolus.vn/)等を使うと便利
・上記の対応ができるのがいつになる見込みか、もしくはいつ頃を目標にするか考えてみてください。


Yeah, I'll remove that one anyway.


it is still displayed in launcher. Kernel Adiutor too


This should be return $rc, right?


I should remove this again. It doesn't prevent use of the base class's `load()` unless it's called on a `MCStack` instance.  If you cast an `MCStack` to an `MCObject`, then this won't prevent use of 2-argument `load()`.


This needs to stay in -- and the `using` declaration needs to go.


all tests pass:
success: http://buildbot.bos1/builders/python-client-test/builds/343


@esigler awesome thanks for the heads up!


I think eventually this one needs to be reactivated to Neo4J as we will simply piggy back on the JTA tx lifecycles to call the neo4j tx begin / commit or rollback cycles.


This commit is causing compile errors for me with "undefined reference to `seL4_DebugNameThread'".


我们大家都在评论一个commit😀，有什么具体的问题要问吗？

https://help.github.com/articles/github-glossary/


I am sure now it was the Vizio update rolled back 5 official same thing happened and this started when they updated


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


Yeah. Fixed was just merged, thanks.


same here.


You need to make sure you have the latest Layers manager, then go into the app itself and under settings, use the switch to Remove Launcher Icon.


97902934_ZoneMixing (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Does this work as expected if the inbox folder label is localized?


Hello, I've just installed and checked with taxes working fine


Commit

A commit, or "revision", is an individual change to a file (or set of files). It's like when you save a file, except with Git, every time you save it creates a unique ID (a.k.a. the "SHA" or "hash") that allows you to keep record of what changes were made when and by who. Commits usually contain a commit message which is a brief description of what changes were made.

我们这一页就是一个commit，描述的是文件的变化而不是文件本身。


I don't believe so. I lost interest, sorry.


@Salaboy Shouldn't these methods also contain the user id, in order to preserve task rights? Or does that not apply to comments?


We're good, I love what you do on DU, too bad you already have maintainer. Also I'm new in this game so things like this might happen but not on purpose. I am all about giving credit where credit is due.


hi


oh, I thought it should be disabled by default. nevermind :P


TeamCity DNN JavaScript Libraries :: CI [Build 43](http://teamcity.engagesoftware.com/viewLog.html?buildId=311&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 43](http://teamcity.engagesoftware.com/viewLog.html?buildId=311&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:8


TeamCity DNN JavaScript Libraries :: CI [Build 44](http://teamcity.engagesoftware.com/viewLog.html?buildId=312&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 44](http://teamcity.engagesoftware.com/viewLog.html?buildId=312&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:8


TeamCity DNN JavaScript Libraries :: CI [Build 45](http://teamcity.engagesoftware.com/viewLog.html?buildId=313&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 45](http://teamcity.engagesoftware.com/viewLog.html?buildId=313&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:8


TeamCity DNN JavaScript Libraries :: CI [Build 46](http://teamcity.engagesoftware.com/viewLog.html?buildId=314&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 46](http://teamcity.engagesoftware.com/viewLog.html?buildId=314&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:9


Approved


I would move this outside of the each. Otherwise, since there is no return or break in this branch of the if/elsif, it will do this for every entry in the address book.


Not yet, but you can build it already. It won't support as much hardware yet, though.


Cool! Thank you @beckettsean! Looking forward to collaborate with you!


smh...


@jon-hall Thanks for the PR! The `promise` module exports a larger API than ES6 Promises by default, though, which I'm trying to avoid. Can you update this to:

```javascript
var Promise = require('promise/lib/es6-extensions');
```

I think that should be npm@2/3 friendly, and ensure that it's a spec-compliant polyfill being exposed from `suspend`.


You are totally right David. I started pushing stuff to try Travis and I should have definitely used branches to do it. Thanks for the advice tocayo =)


reack


to my friends


Teste 2


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 171](https://teamcity.labkey.org/viewLog.html?buildId=261675&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 0 Build time: 00:07:06
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

```


Only way to remove apps from launcher by default is to remove
<category android:name="android.intent.category.LAUNCHER" />
from AndroidManifest.xml and since this is a precompiled apk, we would need to decompile it then remove that line and recompile it. It can sometimes be a pain depending on how frequently each app is updated.


One tab too much! :P


Yolo nice cowboy commit here :cow: :boy:


I think this should be `unless`.


This results in `board=generic`being sent as a TXT record regardless of which service is being announced. While this might be desired for the arduino OTA service, it is annoying for other services, such as HTTP. It would be nice if TXT records could be set from the sketch.


I was hoping if I ignored it long enough make based builds would just quietly disappear :)


Please remove this comment, the `pr` title & description should address the bug it addresses.


fixed


When building a new Biochemistry object, adding a compound uses the addCompoundFromHash() method (https://github.com/ModelSEED/ProbModelSEED/blob/master/lib/Bio/KBase/ObjectAPI/KBaseBiochem/Biochemistry.pm#L708) and adding a reaction uses the addReactionFromHash() method (https://github.com/ModelSEED/ProbModelSEED/blob/master/lib/Bio/KBase/ObjectAPI/KBaseBiochem/Biochemistry.pm#L922).  For the values that are deleted, there are hard-coded defaults that are set when the input arguments are processed.  If possible, I would prefer that default values be set in the Biochemistry object instead of getting propagated through the code.

Good point on compartments.  While reactions are specified in a compartment-free notation, compartments still need to defined somewhere.  @cshenry, do you want to keep in compartments in the master biochemistry?


Some plugins don't use numeric version, so it might look kinda odd, to cover all cases, I would remove `v`.


Can you explain this? Why is it K:K? Shouldn't it be K:1? Or do you need to add something to the documentation here?


Please don't leave commented code in the `pr`. The use of `git log` and `git diff` will allow you to view changes from commit to commit.


Same here, remove the commented code.


hmm - I think it might still be good to use the keywords `OCR Scanning` and also `blind` and `iPhone` - to turn up in more specific Google search results, and target users more exactly? What do you think?


These lines should not be deleted.


Да, добавлю.


You do not need exception information so no need for ```as e```


This pin.getPin function looks pretty good to me. For some reason, I thought it was already implemented. This might still need to be run through js-beautify.


@r12a,

Sorry, I failed to see why we shouldn’t use base character here. Could you explain the definition of phonetic characters more specifically?


next time let's make a PR ;)


Do you know why this was there? Why are you removing it?


This typo is also in README.md.


very nice - lovely changes here


I assume you'll remove these commented out lines in the final commit.


This does seem to help the rook endgames.
R & P, 1m/100+1m/100+(0m+1s)  0

Komodo 9.1 64-bit 8T   - Stockfish 190715 64 POPCNTLJT     278.5 - 284.5    +35/=487/-41    49.47%

Komodo was winning about 2 to 1 (decisive games).  Now we are close to even, maybe a bit better.


Handled in fb61cbd


commit title just a bit too long (you could use `CE` instead of `Compute Engine`)


should only issue a reflow for each parent CSS class change that contains ready-to-fire child animations


This has been fixed now.


I think if we change it to `const INJECTOR_KEY = Key.get(Injector);`, it will become final in Dart and const in JS.


@boghison Don't push directly. Use PRs with self merge instead. It makes it easier for people to follow the changes. See https://github.com/PistonDevelopers/piston/issues/674


Hey man - `click here` is not recommended for accessibility. http://webaim.org/techniques/hypertext/ (see the end of the article)
Maybe a good solution here is to just remove the `click here` text entirely, and just say:
```
 <p>Pay in British Pounds below, or <a href="index-pay-dollars.html">pay in U.S. Dollars.</a></p>
```


are you sure calling the setters when the value is null is actually useful?
maybe that implicitly sets the type in Measure?


yes ^^
sry


@DoumanAsh I think it's no need to catch AttributeError. Can I remove?
El 20/7/2015 17:49, "Douman" <notifications@github.com> escribió:

> You do not need exception information so no need for as e
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ngosang/qBittorrent/commit/ca3589c766e192e0f712313a330002d79e480aab#commitcomment-12255876>
> .
>


Must check for success/failure in creating the directory, and return failure if needed.


??

On Monday, July 20, 2015, Shajed Ahmed <notifications@github.com> wrote:

> smh...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mkarroqe/Project-Euler-Problems/commit/d58cb00e079766ae4f4b7e72b2a5c5841fcc4c3b#commitcomment-12255722>
> .
>

--
Mary Karroqe
Brooklyn Technical High School
Software Engineering Major '16
Girls Who Code '14


Move it to selectPin event.


why remove this test on property description? (in addition, does not seem to be related to the current commit)


very nice. At some future date we'll rename this - `features` and `description` for the link and the id are a bit ambiguous. Definitely scope creep for now though - I'll add it to the backlog instead :-)


Running js-beautify will reformat some of your code style, will it be ok?


Sure.


These two lines are common on both sides of the 'if'.  It looks like they can be hoisted above the 'if' and combined,  and so eliminating the need for `enc` and `dec`.


Much safer/simpler to use sscanf instead of strtok here.


main reason is to not make it too complex for something that is not. I am not a fan of making interfaces for DTOs as they bring not enough value for the additional layer - with interfaces for DTOs we need to provide factories, then we need to have link between interface - factory and impl. In most of the cases that is causing issues in different runtime environments - e.g. OSGi so thought it will be easier to stick to simple POJOs in api


Maybe there should be a variable for the background color. It would make it clearer that what this is doing is erasing the button.


This is fine for now- we should talk about "free delivery worldwide" somewhere on the site though - I'll add this to the backlog


I had a feeling it was something like this. :+1:


Let's add a comment for both of these.


Similar to the comment above: Repeated lines on both sides of the 'if' that can be hoisted.  Looks like it might end up with two identical lines in `doIntroV1` and `doIntroV2`.


What about CPU?


Update source with missing shaders


Wie sagt man so schön, "nicht mein problem". In dem Exception log wird nicht mit einem mal das Coremod erwähnt. Die help bringt dir eh nichts, ist ne dummy command klasse. Die einzige Nachricht die da kommt ist der Hinweis, "reload" als parameter zu übergeben.


@bvssvni I am new to this project, therefore I am going to need a lot of instruction on how to do things like the rest of the contributors. I pinged you on IRC, could we talk a bit?


@Salaboy What do you think about moving this method to the `InternalTaskService` and adding
`addComment(long taskid, String userId, String commentText)` here instead? The `addedAt` date would then just be the current date? (`new Date()`).

I'm already working with this code so I could probably do it fairly quickly..


nice edit btw
![capture](https://cloud.githubusercontent.com/assets/10624942/8780420/d01fbc08-2ed6-11e5-90d9-10633a227aaf.JPG)
don't worry I'm just playing


Not sure whether it's still needed. Spells might have changed.


Do you really need the over-padding and can you not use the values already in the context? Would this be cleaner if there was a button.erase() function?


User should not need to know that gadget is represented as a directory so Entity or something similar would be more suitable here


I think integer IDs will be fine, as @mdoering says, clashes can be resolved when committing to github. But I do think we could have a couple of extra columns in the list of names (namely scientificNameid and namePublishedInID). I'll open an issue with more details.


скобки лишние


Perfect!! good call on the default!!


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1244 of 1244 tests passed)

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


This is a string, so should be "if (s) {".


??? а зачем


Smh @ you stalking me

(((But the headers were too big they bothered me so)))

On Monday, July 20, 2015, Shajed Ahmed <notifications@github.com> wrote:

> nice edit btw
> [image: capture]
> <https://cloud.githubusercontent.com/assets/10624942/8780420/d01fbc08-2ed6-11e5-90d9-10633a227aaf.JPG>
> don't worry I'm just playing
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mkarroqe/Project-Euler-Problems/commit/d58cb00e079766ae4f4b7e72b2a5c5841fcc4c3b#commitcomment-12256092>
> .
>

--
Mary Karroqe
Brooklyn Technical High School
Software Engineering Major '16
Girls Who Code '14


I think it has a default of `.jinja2` just by including pyramid_jinja2. Can we just use that?


Закоментированный код не надо в git


TeamCity DNN JavaScript Libraries :: CI [Build 47](http://teamcity.engagesoftware.com/viewLog.html?buildId=315&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 47](http://teamcity.engagesoftware.com/viewLog.html?buildId=315&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


STR_SERIAL_NUMBER would be more readable


Then the keyword argument to the next two calls can just be dropped.


TeamCity DNN JavaScript Libraries :: CI [Build 48](http://teamcity.engagesoftware.com/viewLog.html?buildId=316&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 48](http://teamcity.engagesoftware.com/viewLog.html?buildId=316&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


I think the copy here is for the sake of allowing calls from multiple goroutines without locking?  Might be nice to have a comment to that effect.


TeamCity DNN JavaScript Libraries :: CI [Build 49](http://teamcity.engagesoftware.com/viewLog.html?buildId=317&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


I agree! -100 on interfaces for DTOs or entities. :/

However, does this class need to be in kie-api, or can it be placed in another module that can be shared in the places it needs to be?


TeamCity DNN JavaScript Libraries :: CI [Build 49](http://teamcity.engagesoftware.com/viewLog.html?buildId=317&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


Looks good


Hi @lepture,

I think that you should still check `WTF_CSRF_ENABLED` and `WTF_CSRF_CHECK_DEFAULT` inside `_csrf_protect`. The reason is that flask-testing seems to encourage using the same `app` as main application, only configuring it for testing before actually running the test (see [here](https://pythonhosted.org/Flask-Testing/#writing-tests)), which means that if flask-wtf is init'd with `WTF_CSRF_ENABLED=True`, then imported inside flask-testing `create_app` and set `WTF_CSRF_ENABLED=False`, then the test would fail because flask-wtf still considers csrf as enabled.


TeamCity DNN JavaScript Libraries :: CI [Build 50](http://teamcity.engagesoftware.com/viewLog.html?buildId=318&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


Depending on what you put in the XML, the setAutoStart() will actually get called by the marshaller, but that's not what I intended and is a bug!  :) The @XmlElement() can be applied to fields or accessors. Since I put the annotation on the field, the marshaling will operate at the field level. I like doing this because it allows freedom in how I design my accessor methods for the internal application. The problem in the above code is that all accessor methods(unless annotated with XMLTransient) are going to be marshalled to/from XML. That means that either one of the two will probably work:
<AutoStart>true</AutoStart>
<autoStart>true</autoStart>

Now, had I named the XmlElement(name="autoStart")  with a lowercase 'A', then the marshaller would completely break and throw an exception saying I have a duplicate property problem, that's because it's already setup the implicit property on my accessors.


TeamCity DNN JavaScript Libraries :: CI [Build 50](http://teamcity.engagesoftware.com/viewLog.html?buildId=318&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


Закоментированный код за борт


Not only tuxterm.
Also tux com is missed.

Changes are inside and pull request done.


Maybe make this a separate commit.


Add ffs_hb.H to document pseudo partition indicator


This error will be raised if multiprocessing does not have ```cpu_count()``` so just to be on safe side you would need to have it to avoid exceptions on older py versions. ```NotImplementedError``` is raised internally by ```cpu_count()```


Depending on what you put in the XML, the setAutoStart() could actually get called by the marshaller, but that's not what I intended and is a bug!  :) The @XmlElement() can be applied to fields or accessors. Since I put the annotation on the field, the marshaling will operate at the field level. I like doing this because it allows freedom in how I design my accessor methods for the internal application. The problem in the above code is that all accessor methods(unless annotated with @XMLTransient) are going to be marshalled to/from XML. That means that either one of the two will probably work:
<AutoStart>true</AutoStart>
<autoStart>true</autoStart>

Now, had I named the XmlElement(name="autoStart")  with a lowercase 'a', then the marshaller would completely break and throw an exception saying I have a duplicate property problem, which is because the marshaller already setup the implicit property on my accessors.


I wasn't stalking I was just procrastinating, and I was checking out those crypts :octocat:


@boghison OK!


hpwh_residential (nmerket) - x86_64-MacOS-10.9-clang: OK (1614 of 1755 tests passed)

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


Issue #1


OK, will do that.


The ruby interpreter is getting confused by this line.

It thinks that you are saying this:

```
upper = entries.length(-1)
```

when in reality you want this:

```
upper = entries.length - 1
```

The space is very important here, otherwise the interpreter will think you are passing an argument to the method you are calling, as opposed to calling a method on the result of that method.

I'm going to double check the source in the assignment, because if you copied and pasted this, then it needs to work. I did a similar thing in my iOS homework a few weeks ago, so I understand the frustration!


It looks like ``qrc`` no longer does anything with the argument supplied. Are those still necessary?


@simeydotme is it gonna be released to the bower package soon?


Was there a reason we took out the complete part for the lightbox?
Would it be possible to simply deactivate it via commenting out the JS Snippet?


this BPMSPL jira was all about moving things up to kie-api otherwise it is considered internal and shall not be used directly by users


The comment preceding doIntroV2 describing the protocol needs updating accordingly.


@lutzbickhardt I will create a new `CONTRIBUTING` file to address requirements for any new `pr` anyone wishes to contribute. It will end up mirroring the one corresponding to the `[node.js](https://github.com/joyent/node/blob/master/CONTRIBUTING.md)` project as it is very well thought out.


Das war ja nicht gemeint das das dein Problem werden soll :-) Ich wollte halt nur fragen da ich sonst mit deiner Reload funktion nichts machen kann und die arbeit wäre für die Katz.


Ah good change. Wander mode needs the same conditional check that dock has though, currently if wander was requested while e-stop was pressed, it will resume as soon as the motors are re-enabled


Bist du heute im Boni Modus ;-)


You are missing "OpenCV\build\x86\vc12"


Hmm, thanks for pointing that out.  I'll investigate.  It's possible that the test runner forcibly clears the cache at the end of test run.  That shouldn't be necessary.

One thing I'll point out about your decorator is that it might not always restore the old `XDG_CACHE_HOME` environment variable.  If an error occurs in your test function (even just an AssertionError) it might not restore the old value.  Really you should have a

```python
try:
    result = func(*args, **kwargs)
finally:
    os.environ['XDG_CACHE_HOME'] = xch
```

in order for your wrapper to be safer.  I don't think my version has that issue but I'll definitely look into it.


Rather use:

MODULE_LICENSE("GPL\0NVIDIA");

... instead of just GPL.


Unused import


Ah, that's a good reason. ;D Hah.. sorry, and thanks for taking the time to explain!


Have you considered using (Vanilla.js)[http://vanilla-js.com/] for your project?


Thanks Yohan, I will use localconfig and cartocc for this


This button.getCurrentBtn() function looks helpful. Are there other places it could be used to add more clarity to the code?


@samjewell I changed the click here's to something more accessible i guess, added a euro page unless that was already in there and added some meta tag description shite, NOW STICK IT UP YER BUM :palm_tree: :pager:


Hab die übrigens Coal, Compressed Coal sowie Double und Triple Blöcke gemacht.

https://www.dropbox.com/s/49ymgqdfyqi4j13/Screenshot%202015-07-20%2018.12.24.png?dl=0


@chafey Putting the `src/misc/footer.js` file after `src/*.js` will expose all the library functions, whereas putting it before `src/*.js` only exposes the utility functions and `readEncapsulatedPixelData.js`. Don't know if you needed to do that to fix AMD, or if you decided to expose all the functions, but wanted to make sure you know.


Separate multiple simulation from single simulation. Only do multiple times if number of queries are sufficiently large.


und die comprimierungs Kosten kriegst du locker raus wenn du die Burn values sieht.
``````````````````````````
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<CustomFuels>
    <FuelItem ItemName="minecraft:diamond" BurnTime="102400"/>
	<FuelItem ItemName="dreamcraft:tile.Charcoal" BurnTime="16000"/>
	<FuelItem ItemName="dreamcraft:tile.CompressedCharcoal" BurnTime="160000"/>
	<FuelItem ItemName="dreamcraft:tile.CompressedCoal" BurnTime="160000"/>
	<FuelItem ItemName="dreamcraft:tile.CompressedCoalCoke" BurnTime="320000"/>
	<FuelItem ItemName="dreamcraft:tile.DoubleCompressedCharcoal" BurnTime="1600000"/>
	<FuelItem ItemName="dreamcraft:tile.DoubleCompressedCoal" BurnTime="1600000"/>
	<FuelItem ItemName="dreamcraft:tile.DoubleCompressedCoalCoke" BurnTime="3200000"/>
	<FuelItem ItemName="dreamcraft:tile.TripleCompressedCharcoal" BurnTime="16000000"/>
	<FuelItem ItemName="dreamcraft:tile.TripleCompressedCoal" BurnTime="16000000"/>
	<FuelItem ItemName="dreamcraft:tile.TripleCompressedCoalCoke" BurnTime="32000000"/>
</CustomFuels>
``````````````````````````


We should not use case insensitive comparison for paths.


@KevinW1998 when I get home I will provide this ;)


http://www.wowhead.com/spell=50526 - nope
http://www.wowhead.com/spell=15290/vampiric-embrace - yes


These are cpu implementation registers and so these functions should only be defined in the relevant cpu header file, protected by an `#ifndef __ASSEMBLY__` conditional directive.

In this case, these functions are not used in the C code and can just be dropped from the patchset.


Same as above


This is not an architectural register, but an implementation defined one (even if it has the same encoding in multiple ARM cpus, it is still not an architectural register). These should be defined in the relevant cpu header file (or files), i.e. in `include/lib/cpus/aarch64/cortex_a53.h`

As an example see the definition of L2CTLR_EL1 in the a53 and a57 headers.


Perfect.


`callableQualifier` may be inlined


@michael-jerred Thanks for the correction!
BTW, could you also synchronize the corresponding JSDoc `(@param {<updated type>} target)`?


> I will use […] cartocc

I'm curious: are you using Tilemill?


Oups, my bad.


Noted. Will edit and keep in mind for future reference. Thx @matjames007


Please review the existing implementation of errata workarounds in the `cortex_a57.S` source code, and use that as a template for the changes here.

In particular, in order to support the use of the same firmware binary in multiple versions of a platform (with different cpu revisions) - the code to apply errata workarounds should check the CPU revision and only apply the workaround if appropriate.

It probably makes sense to pull in the code from `cortex_a57.S` to construct a variant/revision value in `X0` and use this to check for appropriate revisions (<=r0p2 for 826319 and <=r0p3 for 836870 according to the A53 errata notice - the latter is present in r0p4 but the workaround is enabled by default).

This is in addition to the build option configuration that you have already included in this patch.


Please indicate the revisions known to be affected (compare with the existing workaround support for A57 below).


Seems reasonable to add this function. It might be slightly clearer as 'drawSelectPinMessage', but I'll defer to your judgement.


Please indicate revision affected


Ah, you are right. (Though a slightly better way to put it is that the map is not stored -- it is only used while the function is executing)


Actually it can't (`qualifier` is modified later)


my pleasure, and thanks for additional pair of eyes looking at this!


I don't like keeping an enum in memory with a reference to ```BatchReport.ActiveRule.Builder```. Moreover for sure this private class is instantiated once.


The previous commit avoid you to do that xD


Can we limit the scope of the lock only to the mutable state access and not lock while waiting for IO?


Maybe better include in class instead of external


Oops, yeah I guess so.  Oh well I can revisit that when I get more time (or
maybe just leave it this way, IDK)

On Mon, Jul 20, 2015 at 11:13 AM, Troy Astorino <notifications@github.com>
wrote:

> @chafey <https://github.com/chafey> Putting the src/misc/footer.js file
> after src/*.js will expose all the library functions, whereas putting it
> before src/*.js only exposes the utility functions and
> readEncapsulatedPixelData.js. Don't know if you needed to do that to fix
> AMD, or if you decided to expose all the functions, but wanted to make sure
> you know.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chafey/dicomParser/commit/097c9a6ccd36f036bda19a7ace4bd74d70514767#commitcomment-12256405>
> .
>


Ich wollte halt auch mal das tun was 50%+ der mod Authoren machen :P
Denke ich weiß warum der command nich geht. Commit folgt...


I've just checked and sorting on Workflow/Mapped object does not seem to work, can you check what's going on?

<img width="1199" alt="screen shot 2015-07-20 at 17 58 00" src="https://cloud.githubusercontent.com/assets/513444/8780842/fb115b9e-2f0b-11e5-9e05-0e584dadb39e.png">

You can get this list in my tasks widget on the dashboard or tasks widget on object pages.


Thanks for your translations. They are very much appreciated. I'm deploying them now.


add destruction logo


Your debug statement here is just going to print each button, not the type of button for which the message will need to be displayed.


everything else is OK for me


LGTM


That totally fixes #4096 for me


Thanks for remind me. This was required in previous version of code. I will delete it in the next push.


is there a reason the renderer must be specified as a parameter for settings? i believe it could just as well be initialised to default in line 93, and set explicitly by the user, if need be...


Obsoleted by the more general Features support in the V2 protocols version.


Revert it, I'll do another pull request.


I suppose you can then return this part :)


the comparator will need comparing the renderers, too =)


Interesting that you are using clearRect here and overwriting buttons when 'popped' (ie., cancelled). Can you explain why one approach vs. the other or remain consistent? Is this directly what Jenn was doing?


:P


I didn't know Python 2 supported that too. I will change this. Thank you.


So we can only load an assembly if it was specified upfront as a dependency (including transitive dependencies)? Could you add a comment to the loader class that describes this?


why?


I rewrote this to be slightly different, not sure what you think. It's the same but only uses the %i once:

```
[Unit]
Description=Download Kubernetes
Documentation=https://github.com/GoogleCloudPlatform/kubernetes
Requires=network.target
After=network.target

[Service]
Type=oneshot
RemainAfterExit=yes
ExecStart=/usr/bin/mkdir -p /opt/share /opt/bin /opt/share/kubernetes /opt/share/kubernetes-server
ExecStart=/usr/bin/curl --fail --location --remote-time --output /opt/share/kubernetes.tar.gz https://github.com/GoogleCloudPlatform/kubernetes/releases/download/v%i/kubernetes.tar.gz
ExecStart=/usr/bin/tar -xzf /opt/share/kubernetes.tar.gz -C /opt/share/kubernetes
ExecStart=/usr/bin/tar -xzf /opt/share/kubernetes/kubernetes/server/kubernetes-server-linux-amd64.tar.gz -C /opt/share/kubernetes-server
ExecStart=/usr/bin/rsync -a /opt/share/kubernetes-server/kubernetes/server/bin/ /opt/bin/

[X-Fleet]
Global=true
```


Log from the script:

+  CHINESE (gb18030)
 - Number of files changed: 45
 - Line breaks toll: 25
 - Unnecessary spaces toll: 305

+  CZECH (cp1250)
 - Number of files changed: 286
 - Line breaks toll: 4324
 - Unnecessary spaces toll: 230

+  ENGLISH_FIXT (None)
 - Number of files changed: 10
 - Line breaks toll: 1
 - Unnecessary spaces toll: 71

+  FRENCH (cp1252)
 - Number of files changed: 16
 - Line breaks toll: 4
 - Unnecessary spaces toll: 223

+  GERMAN (cp1252)
 - Number of files changed: 471
 - Line breaks toll: 2807
 - Unnecessary spaces toll: 136

+  HUNGARIAN (latin2)
 - Number of files changed: 27
 - Line breaks toll: 165
 - Unnecessary spaces toll: 78

+  ITALIAN (cp1252)
 - Number of files changed: 23
 - Line breaks toll: 75
 - Unnecessary spaces toll: 54

+  POLISH (cp1250)
 - Number of files changed: 24
 - Line breaks toll: 290
 - Unnecessary spaces toll: 17

+  RUSSIAN_1C (cp1251)
 - Number of files changed: 2
 - Line breaks toll: 0
 - Unnecessary spaces toll: 593

+  RUSSIAN_FARGUS (cp866)
 - Number of files changed: 605
 - Line breaks toll: 8572
 - Unnecessary spaces toll: 131

+  SPANISH (cp1252)
 - Number of files changed: 20
 - Line breaks toll: 0
 - Unnecessary spaces toll: 56


why this


I'm using renderd for a mod_tile server
(I was with you last Wednesday in Vincennes)


Not sure what you asking for, but if it is about where did I used it, it's in the pinSelected event.


Note that the same issue exists in many other places


I see, I wasn't 100% sure exactly what their use cases were. I was thinking that it would be nice if any time someone could choose options that they could use any option. Indeed it doesn't quite make sense that anyone would need to change this after initialization.


I did not realised this.
By the way, I just found that ```shape``` from ```theano.scan_module.scan_utils.infer_shape()``` are computed form outer_inputs.
I add a break point before return the ```shape```, and find the input of ```shape```, just to find that it's ```node.inputs```

```python
```


Looks good to me from a high level.


It seems odd that we mix the context with analyzer loader. I think it would be better to separate these two - a custom load context that has clearly defined way of loading dependencies and then a loader that uses it.


I belive 2.6+ only, if that's fine


Ah, yeah, sorry


space


That was just a temporary debugging, it should be removed.


crazy indent


> Я всё-таки хочу увидеть ошибку/недочет в админке своими глазами. Где (адрес страницы, виджет и т.п.)?
Виджет "случайные статьи". Там добавь описание к чекбоксу и можно увидеть все во всей красе. чтобы все протестировать.


single quotes


assure -> ensure?


negative check here necessary?


np :)


each declaration on its own line


Why is this needed? I was specifically trying to prevent other objects from interacting with the canvas and forcing them to go through the 'ui' object.


I agree. In fact, I don't like the `whateverUtil` classes. They constitute an easy way to group many functions in one class, but they are non-SRP. In the end, they become a bag containing many unrelated features.


as in: `totalCounter + c < 0`


i think we might find a better name for this - also worth noticing that this is only available/meaningful when multisample is active.


You reverted some of your changes.


Hm, this is in the cross platform code but the progress bar is too small on OS X.
I also tried setting it in QEvent::Show handler with

```
ui->quotaProgressBar->setMinimumHeight(qMax(_quotaLabel->height(), ui->quotaProgressBar->minimumHeight()));
```
because the label height is only 6 and it still does not work.

Maybe the QProgressBar somehow clips children


LTC test: please check  (is using STC characteristics)


Just added the namespace decl


Don't put UI elements within Events!


clearRect will coz a white bg, while removing a btn will need to have a gray bg.


You are correct.

On Mon, Jul 20, 2015 at 11:07 AM, tjphilpot <notifications@github.com>
wrote:

> It looks like qrc no longer does anything with the argument supplied. Are
> those still necessary?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/seyfarth/ebe/commit/d3b3a89dc5f4a426853c2c8efbd08da35d345f4d#commitcomment-12256281>
> .
>

--
Ray Seyfarth               http://rayseyfarth.com
http://rayseyfarth.blogspot.com
Personal email:           Ray.Seyfarth@gmail.com
Google Voice:              601-336-0128  (Calls Cell & Skype)
Cell:  601-270-7995     Skype: 601-255-4985


第一次上傳測試~


Nous sommes dans le cas d'une entreprise présentant plusieurs branches qui sont à leurs tours organisées en différents départements et sous-départements.
Chacune de ces branches peut spécifier ses besoins en matériels et fournitures ainsi qu'évaluer et choisir les différents marques et fournisseurs d'une façon indépendante. Chaque branche possède sa propre "gestion des stock" et ce service est responsable de la gestion adéquate des approvisionnements et de leur rotation efficace intra-branche ainsi que des transferts entre-branches en cas de besoin. L'objectif de ce projet est d'organiser cette gestion des stocks.
Nous proposons une organisation selon les étapes suivantes:
•	L'entreprise est organisée en différentes branches
•	Chaque Branche a une seule location et est divisée en différents départements.
•	Chaque département présente différents sous-départements ou sections, chacun présentant plusieurs employés.
•	A chaque employé est attribué un seul poste principal.
•	Chaque branche possède un seul Stock de matériels, et ces différents produits sont organisés en catégories.
•	Ce stock fournit les différents départements de la même branche.
•	Ce stock peut fournir aussi les stocks des autres branches. Ce transfert est effectué par le service de transport (service postal de l'entreprise).

•	A chaque acquisition d'un nouveau produit, un code lui sera attribué  selon sa catégorie.
Ainsi, pour chaque produit, et à n'importe quel moment et grâce à son code, un repérage et distribution rapide intra-branche ou entre-branches sera possible, de même qu'un inventaire minutieux et continuel.


LTC test: please check (55ad11f60ebc590abbe1bbb8 is using STC characteristics)


Include `target="_blank"`


This is bad. The Events should only deal with events and should call UI functions when updating UI elements. Look at how I restructured some of the other code to try to isolate these a bit. The idea is to simplify both debugging and readability. Events should handle all of the high-level state transitions, reacting to all events in the system, and UI should maintain the basic state of the UI and provide helper functions to translate events like clicking at various locations into knowing that a particular item was clicked.


Don't do this!


Done


@ksooo I've gone for a comporomise: If SupportsFirstDay(), always use it. If not and FirstDay is non-zero use it. Otherwise use now(). (If the client doesn't set first day it stays 0 - at least with gcc on linux!). Does this work for you?


what does the param name mean?


This line is too long.


!== true?


Great idea, but I can not judge if thats feasible. I think what you should do is: grep out a list of all definitions of *shortname* and *application_executable* and see if they differ in some customer branding. If they do, this change becomes a bit more difficult.

Also, I would only plus-one that if it goes along with a patch that removes *shortname* from the genbranding.pl script, so that we see if it is still needed it somewhere.


:swimmer: Swim it. I mean, ship it


do you need this var? Maybe function is better?


Let's try to be very consistent about these 'listen' calls. They should be in the same location within the definition of each handler function.

Also, Jenn seemed to have done a reasonable job at debugging the various states/handlers, so let's try to reproduce the handler state transitions that she had previously designed, before we consider modifying anything.


Oh well. I should probably name my branches something more generic like [tkrokli/patch-x] to avoid confusion


I worked off the assumption that these are the important things that we need to be connected if we are to consider the beamline to be connected. If there are any more things that need to go in, or any of these that need to be removed, let me know.


so this if can be improved too


taille image


can't we have function here


Add link for downloading image files


Closes issue #5


also funny dup


I think this only works if there are no spaces in the attribute name... Maybe use attribute_label somehow instead?


add EOF line


i think this could be written even more generically, i.e. `bindForBlitting(const ofFbo & fboFrom, const ofFbo & fboTo, int attachmentPoint)`

that way, there the coupling (and the assumptions about ofFbo internals) are less tight. this method could then also be interesting for future applications when blitting from one fbo to another.


Oops. Just discovered it. Thanks


?Hi,

you were right assuming that I am unfamiliar with the whole github-process; this being the first time, I was happy with what I had done. I will read the referenced article and then renew tomorrow.

Cordialement

--------------------------
Lutz BICKHARDT
lutz@bickhardt.fr

NEW COORDINATES !!!
(privat) Graumannsweg 42, D-22087 Hamburg, Germany
(Büro)  Borselstr. 3 - 7 (im Hof), D-22765 Hamburg, Germany

SKYPE: bickhardt

+49 172 3846086 (deutsches Handy / German cell phone)
+49 40 22758122 (privat Hamburg)
+49 40 39190921 (Büro Hamburg)
+33 6 18386906 (portable francais/ French cell phone)
________________________________
De : Jason Gerfen <notifications@github.com>
Envoyé : lundi 20 juillet 2015 18:09
À : lutzbickhardt/node-libnmap
Cc : Bickhardt, Lutz
Objet : Re: [node-libnmap] doFix (1ca63c2)

@lutzbickhardt<https://github.com/lutzbickhardt> I will create a new CONTRIBUTING file to address requirements for any new pr anyone wishes to contribute. It will end up mirroring the one corresponding to the [node.js](https://github.com/joyent/node/blob/master/CONTRIBUTING.md) project as it is very well thought out.

-
Reply to this email directly or view it on GitHub<https://github.com/lutzbickhardt/node-libnmap/commit/1ca63c2d78ec30bb7b2440966a4f7061697675aa#commitcomment-12256302>.


I ran through the tutorial. Most of it makes sense to me. Some of the later parts dive into a number of stats concepts that will require oral explanation as one is stepping through it. It might help to describe the expected plot outputs and what we're looking at after each plotting snippet.

The arrows plot is cool and stepping through 10, 100, to the whole point cloud, to the all parameter point cloud is very clear. I wonder if one step prior, the students should plot a two panel plot just of the series of alpha0 and Jmax, and then the arrows plot to see how the evolution of these two are linked.

A few of the steps made a bunch of plots at once (for example the MCMC diagnostics) - how do you step back through them in Rstudio?

Equations didn't render properly in view on github of Rmd file in my browser (Chrome, Mac) - maybe I'm missing an extension? They were filled with $ signs.

I wonder if before running fita, we should have the students plot some portions of “dat” first, so we know what we’re looking at. Or at least look at it in the variable viewer - plot A-PAR data before the MCMC?

When I ran plot.photo I got:
“[1] "No AQ data available"
[1] "No ACi data available””
I did get a plot of A-PAR
I assume that is expected?

I think we could easily get through this in ~45 minutes. I cut and pasted each step from the Rmd file viewed on one window in my Browser into Rstudio running on another Window, using a fresh install of Pecan 32 bit 1.4.3RC1 and ran into no errors.

Now I'm wondering, though, if we do the single parameter sensitivity in Sipnet first, then this, then the pre-compiled Sipnet MCMC? To discuss later.


what?


That what I was afraid of, putting code in the wrong place. Will fix it.


The Function that we have named `memerize` was named that for a reason, it is a pun of sorts, a play on the word meme if you will and it goes along with the theme of the script we call **Memer**


Sorry @adriantoine .. @kenwheeler is the only one who knows how that build script magic works, haha. Otherwise I would have patched it right away. xx


for phonetic annotations in bopomofo there are three elements that interact: (1) the han character or characters which are always referred to as 'base characters' _for the annotation_, (2) the tone marks, and (3) the bopomofo characters used to represent phonetic articulatory sounds. If i understood it correctly, the latter, are called 'phonetic characters' in other parts of this section, which is the only reason i picked that term (i'm not particularly keen to use it).  We could call them 'zhuyin characters', as long as we clarify somewhere that this excludes the tone marks for the purpose of this discussion. We'd also need to be careful that we don't confuse the reader when saying something like "in vertical zhuyin", where 'zhuyin' really means the whole annotation.  Let's call them 'phonetic character' for the rest of this comment, just to be clear.

you could argue that one  'phonetic character' is the base character for a tone mark, although that's not always the case.  In particular, it's not clear to me that the  'phonetic characters' are the base characters for the neutral tone. Until you edited it today, i actually thought that the 10% referred to 10% of the size of the han character.

so, basically, i found the text a little confusing or unclear in parts. Each time i read 'base character' i think of the hanzi character, because in the long time i've been working on phonetic annotations in japanese and chinese 'base character' this is the first time i have come across it being used to refer to something else. I'm guessing that others will also fall into this trap.

that's why i'm suggesting to use the term 'base character' only for the hanzi, and not for the zhuyin/phonetic/bopopmofo characters (i don't have any strong preference for what we call them as long as it's clear).  Does that make it clearer?


background: doesn't currently work in FreeBSD


do you need that?


...I like this strategy for the registry.


Good use of constants.


regex could be a constant rather than hard coded.


@luistinygod Did you already provide custom code for a customer? I'd like to make a few tweaks.


From a practical perspective, I don't think that it is bad to have some, because you always find some useful stuff that you would like somewhere but you don't really know where (separating the responsibilities is not trivial, and you often fall on cases for which it is tricky to decide who should manage it unless you have a really rigorous design). Utilitary classes helps to keep things relatively clean and easy to use until you find a better design.

But I agree that some effort should be made on reducing them when it can be done. This situation here is an example of such opportunity. You can also often think about factories and builders to avoid making too much utilitary methods, but it does not applies for everything.


Is it really the responsibility of the ````unique?```` method to determine whether an error should be raised?


This is a comment.


I think, that do not help .. but .. I SEND a type !!!!

it mus by change as I say ..
if(!empty($type))
   @touch($cachedir .'/index.php');

That works perfect for all situations !!!!


You probably need the ask_lang selector, not the ask_intmode_lang. Also this is a Ruby page only so you should set filter_choices


This method could be shortened to a one liner with interpolation. This would be cleaner if your lambdas took a parameter for the number of characters you want to generate, rather than you hard coding them into this method.

You may be able to use #tap to shorten as well.


Ah ok no problem @simeydotme!


Duplicate `;`


Will be this ever included in the master? is there any other way to achieve this with current version?


Please sort includes alphabetically.


Could you also add to the test, something that tests the path in ExtractFactory.java:41-44 ?


Fixed via a58cb049b296e05f929cb6a999187b03a7589d41

Maybe @ogoffart can explain why this was only a problem on Mac. Stylesheets?


Done a6f493ff9498d630c7ec112036b2c4650e765b06


@thequixotic There is no loading here.


It would probably be a good idea to have something that keeps track of the state change (member variable or some such) because otherwise you'll bombard the event loop with a whole bunch of falses (one for each control) before the single true.  Also, if things go offline, then you'll only get the single true/false, which is preferable.


No need for curly braces.


Are there cases where we should print a warning message when this happens? What does a 500 mean, or a 404? (Or do those even occur in this context?)


a


Typo


I bet you didn't mean to add these changes to the pull request?


In that case you should probably clean `/opt/share/kubernetes` before downloading? Otherwise you might accidentally mix things from different releases.


comments?


How about we call these methods `getFboId()` and `getFboTexturesId()`?


przemyslmy to...


Found the bug! That was an interesting one. 031bc51bac2796b109b6b0bca6e8d0cbf2af5a91


Makes sense.

On Mon, Jul 20, 2015 at 12:45 PM, Dennis Schridde <notifications@github.com>
wrote:

> In that case you should probably clean /opt/share/kubernetes before
> downloading? Otherwise you might accidentally mix things from different
> releases.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devurandom/kubernetes-coreos-units/commit/a76815422f36458e859741e73738ab3008a5f3ad#commitcomment-12257089>
> .
>


fast-forwarding master to borshop-integration-407-feature/bch/cleanup-pep8 = 723bd59b


Move this file completely into src/static_operator/fullc_op-inl.h


for GPU bind, add macro guard MXNET_USE_CUDA


@clshortfuse - Thanks for this catch; can you post a formal issue for this ?
@rschmukler - got it.


Is a visibility condition mandatory?


toumy taille image


TypeString is something you can use as a key, instead of description


No need for curly braces.


新しいプロジェクト提案


> Then again, it's not that big of an issue

Right, that's why. In pre-Lenexa review comments from committee members, it became clear to me that trivial default constructors are less important than the extra safety that comes from value-initialization. It's not terribly important, though.

> copy construction and destruction are the important two to be trivial so the x64 ABI can pass objects in registers

I didn't know that!


This doesn't seem to be legit. You take reference to the pixel buffer's bytes, and then unlock it, which allows the OS to unmap it from memory.

I'd do it like Apple's SquareCam sample does:
```
static void ReleaseCVPixelBuffer(void *pixel, const void *data, size_t size)
{
	CVPixelBufferRef pixelBuffer = (CVPixelBufferRef)pixel;
	CVPixelBufferUnlockBaseAddress( pixelBuffer, 0 );
	CVPixelBufferRelease( pixelBuffer );
}

...

	CVPixelBufferLockBaseAddress( pixelBuffer, 0 );
	sourceBaseAddr = CVPixelBufferGetBaseAddress( pixelBuffer );

	colorspace = CGColorSpaceCreateDeviceRGB();

	CVPixelBufferRetain( pixelBuffer );
	provider = CGDataProviderCreateWithData( (void *)pixelBuffer, sourceBaseAddr, sourceRowBytes * height, ReleaseCVPixelBuffer);
```


may be (v1[0] * v2[0] + v1[1] * v2[1] < 0.0)


Is this release official, or is there more polishing before that?  I'm not quite understanding after reading https://github.com/ligi/FAST/pull/81 .…  If the former, please release to Google Play; if the latter, please make 6.3 the preferred release in F-droid.

Thanks!


Oh, I figured out how to step through plots with the little arrows. Worth mentioning, maybe on the first tutorial that uses Rstudio.


Computer code

Keyboard input

<tt>Teletype text</tt>

<samp>Sample text</samp>

<var>Computer variable</var>

注释: 这些标签常用于显示计算机/编程代码


del


whoops, had to recreate these, so these are trashed now :/


Yes you are correct, search is performed on "body" field, which is hardcoded [1]. At first I thought search is done on different document representations (fields) with different boosts, however I was wrong. I figured this out and didn't create pull request for that reason. Sorry for the confusion.

[1] https://github.com/lintool/IR-Reproducibility/blob/master/systems/lucene/ingester/src/main/java/luceneingester/TrecDriver.java#L80


At the end I removed all this checks, this is only for internal use.


Yeah forgot to exclude it in gitignore, good catch.

On Tuesday, July 21, 2015, Alexander Zeillinger <notifications@github.com>
wrote:

> You are missing "OpenCV\build\x86\vc12"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ractiv/touch_plus_source_code/commit/815471e312e55ca81ca5a84b1f528fe862ef018b#commitcomment-12256326>
> .
>


sure


cleanup_unit_test (mbadams5) - x86_64-MacOS-10.9-clang: OK (1752 of 1753 tests passed)

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


cleanup_unit_test (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1759 of 1759 tests passed)

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


third ?


Well, `counted_iterator`s and their sentinels always model `SizedIteratorRange`, even if the base iterator is not random access. Right now that's achieved with extra overloads of `operator-`, but ideally `basic_iterator` would be smart enough to accommodate that such that the extra overloads were unnecessary. It's something that needs to be addressed before we propose `basic_iterator`, but for now it can wait.


Did you notice the base command, `"cmd": ["makensis", "nsis_build", "$file"]`?


Can this ever happen or is it just a safety check? It it can, shouldn't we skip match offsets bigger than MAX_DISTANCE in the matching code?


I'm lukewarm on breaking up the lines like this. I don't really think it's more readable.


An nonexpected response code is always fatal, but better error handling for this situation is on my todo list (this affects every model, not just ps).


@adamhooper I notice you got rid of sort here but didn't add it back at Line #76. Is that intentional (e.g. if token bin sorts automatically) or an oversight?


Here's where the problem is...


:) No hard feelings!

I guess the padding zero part should be configurable, since in real-life some zeros occur frequently and most software handles them quite well - no matter what the RFC says.


I did it just to make the 200 char limit happier. Do you have a better way to break them up? Or i can just leave them as is


That and easier debugging was the reason for me downloading into `/opt/share/kubernetes-%i`, btw.


Ooops...


whoops, no.  also i think these are the changes that added to bootstrap time, we only used to have ``check-length`` in copy, but now I added it here before iteration which speeds it up because its not doing generic ``<``.


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

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


nah


Found Typo Should be
```
contentInset={{top:0}}
automaticallyAdjustContentInsets={false}
```

instead of

```
contentInset={{top:0}}
automaticallyAdjustContentInsets={false}c
```


I'm curious to know a little more about this. From https://travis-ci.org/HarvardEconCS/turkserver-meteor/builds/71602891 you can see that it broke this test:

https://github.com/HarvardEconCS/turkserver-meteor/blob/master/tests/experiment_tests.coffee#L250

Is the behavior of this causing problems?


It's already covered. This test creates 100 Extracts, as long as two Extracts are created in the same ms, they will have the same Extract ID and line 41-44 will be executed.


This revision is clean.


# Before
![before](https://cloud.githubusercontent.com/assets/10423645/8781811/a497c540-2f11-11e5-8bdc-cad535a1969b.png)
# After
![after](https://cloud.githubusercontent.com/assets/10423645/8781816/accba24a-2f11-11e5-83ae-c29743804552.png)


Similarly lukewarm on this.


Clean/Clear/Reset AudioDSP database would all be fine to me.


develop (rraustad) - x86_64-MacOS-10.9-clang: OK (1243 of 1243 tests passed)

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


You have added two files that are auto-generated by Eclipse: `.cache-main` and `.cache-test`.  These should not be in the repository.

Please [add them to the `.gitignore` file](http://git-scm.com/book/ch2-2.html#Ignoring-Files) so that they don't get added accidentally/automatically.


If you want them broken up, I'd do it something like

```
"create or replace function x_tan ( input double ) called on null input"
"returns double language java as 'if (input==null) return null; return Double.valueOf(Math.tan(input.doubleValue()));'"
```

but I think the long lines are probably ok here? I'm not vehement about any particular solution right now.


Please explain a bit more what you're trying to solve here, also when can this situation actually occur?


Yeah, you're right, that makes sense. But on the other hand, in situations when ACL instance is queried by using strings for resources or roles, generic resource/role will be generated which won't be `ProprietaryInterface` instance. So for example, if I set permissions this way: `$acl->allow('author', 'blogPost', 'edit', new OwnershipAssertion());`, and then later on I do the check this way: `$acl->isAllowed($userObject, 'blogPost', 'edit')`, this one still should be allowed, because Assertion can not determine ownership.


For the comments, you can understand it better this way.


I also find assertions to be somewhat pointless if the code is correct.


0x ?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.772.b7ebd6a33484c9dcc87c585732dcea5a6b9e165d](http://build.palaso.org/viewLog.html?buildId=59834&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 380, ignored: 3 Build time: 00:02:51


excellent here


That's odd, because I just verified that the correct days were printed in the past, but then it started diverging a few weeks ago. Anyway, thanks for the fix!


@simeydotme soon gentlemen. been swamped at my FT


Hm this makes things a little tricky. In the Yeoman generator we make WCT optional (https://github.com/yeoman/generator-polymer/issues/131). When a user says 'No' to WCT, we remove it from package.json (https://github.com/yeoman/generator-polymer/blob/master/app/index.js#L69-L72), bower.json, and do a few other things.

If we want to use a git submodule of PSK or download it directly (https://github.com/yeoman/generator-polymer/issues/205) then we might need the `try` to stick around.

@addyosmani @arthurvr wdyt?


Should this verify that it's a non-negative integer? What about when two extensions have the same priority?


Should you be consistent in your use of @ vs. \ to prepend anchor & brief (I believe it's the former that should be used)?


Thanks. I will incorporate your change.


I created an example implementation in #24, I'd be happy to hear your comments about it!


This is equivalent to `val up_npart = npart.flatMap(_.parents)`.

I personally find `map`, `flatMap`, `sum`, `max`, etc. much more readable than the versions with folds, but coding style varies greatly in the gapt code base.


Language in commit message?
"This ported from" -> "This is a port from" OR "This was ported from"
And you often link to the original HSC commit (I'm not requiring this, but it's often handy!)


usual:
``` $ pip install -e git+http://github.com/jeanjar/sqlbuilder.git#egg=sqlbuilder ```


Good catch, thanks! Fixed by https://github.com/nylas/sync-engine/commit/50dc64c08146530a94f80690726ad73b961ef36d


This replaces the contents of `~/.bash_profile` with just the echo'd command.  I just wiped out my highly customised `~/.bash_profile` with this command.  Not cool.  In my opinion this should have been an append `>>` at the very least:
`echo "PATH=$PATH:~/.composer/vendor/bin" >> ~/.bash_profile`

backups to the rescue!


Same here, this is equivalent to `val up_npartFold = up_npart.map(_.formula)`.


Fixed in  d1930a52aa0cb0a3e77ec216a57a5bafc45eb8e1 .
I informed maintainer nvidia-mainline package about this fix.

Thanks.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 172](https://teamcity.labkey.org/viewLog.html?buildId=261687&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 0 Build time: 00:07:38
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

```


The generator should just remove this line. If the user didn't want WCT, this line ends up just being bloat in their project, no? Also I feel like the actual starter kit shouldn't have code which is specifically there for the generator.

> If we want to use a git submodule of PSK or download it directly (yeoman/generator-polymer#205) then we might need the try to stick around.

Why? Removing this should be as easy as a `.replace()` call.


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: templates/structure/body_for_table_summary.phtml, templates/structure/insert_empty_drop_action_links.phtml


Понятно. Нужно просто добавить display: block. Исправлю.


Ah, that's clever, Much cleaner.


wtf? b632070da0dc3e9ce1099ecd8095f00bb33292e5


Can we leave the test files in `modules/__tests__` instead of `test`?


You can write `p :: q :: Nil` as `List(p, q)`, or even `Seq(p, q)`.

In this case there is even a dedicated constructor to create axioms of the form A :- A, you can just write `Axiom(p)` here.


haha, nice title :D


test


``/usr/bin`` won't be accessible in Mac OS X 10.11 but [/usr/local/bin will](http://www.macworld.com/article/2948140/os-x/private-i-el-capitans-system-integrity-protection-will-shift-utilities-functions.html)


no need for \Entity


http://www.wowhead.com/spell=74100

SpellEffect 11 = Bind(4721 = Shipwreck Shore)
SpellEffect 5 = TeleportUnits


no empty line when you start a function unless the function params are broken.


This should be the new `test` script. Let's get rid of jsdom.


I assume you're setting the audio path to always be null because there's no audio associated with this warning? What was that demo.warning.filepath string originally intended to be?


On the raspberry pi I used for testing, I have created a root account, When I have this script run on boot, it gives me a root shell.  I'm not sure how this script will work if root has not yet been enabled on the raspberry pi


</nav> is out of alignment.


Yes I don't know why I did this, will remove in next commit.


No Problem, @kenwheeler  ! :)
Sorry dude; I was getting a few emails/messages about it so redirecting them back your way :P -- although it's totally my fault.

https://github.com/simeydotme/slick/blob/gulp-on-this-%F0%9F%8D%BA/gulpfile.js
Look over this once more when u get a second... I just merged master trunk back in to the branch and commented the gulpfile :)


They have not fixed these functions in the Windows 10 SDK. They are still not annotated with _Success_.

While testing this I noticed that VS 2015 adds two new false positives when building ETWProviders - it complains about possibly non-null-terminated buffers after calling vsprintf_s. Thanks Microsoft - I'm calling your guaranteed-safe functions and that is causing warnings.


just return <nav>...


Right. I'll see if that solves it. Sorry for the sort of unhelpfully cryptic warning; I thought that the time constraints of your PR made this worth mentioning even before I figured out the solution.


commit 3


Can you please extend the fix to the other languages files ?

Thank you.


I think the return type is :x:nav


@adiherzog: This change affects our docker image that relies on this path in the default config (replacing it) ... see https://github.com/scenarioo/scenarioo/blob/develop/docker/Dockerfile where this same directory is replaced in the default config ... can you please also change the Dockerfile accordingly. Thank you.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.773.e311f0c221471c584221e8f1d904bca14ae35aba](http://build.palaso.org/viewLog.html?buildId=59835&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 380, ignored: 3 Build time: 00:03:29


Just tested this and it returns a 302.  Will discard these changes


:+1: thanks a lot!


This change doesn't match with your description.


@aeslaughter @permcody I'll look into it.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.774.d7e0f73158def6243c3f994b999f846423cbc797](http://build.palaso.org/viewLog.html?buildId=59836&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 380, ignored: 3 Build time: 00:04:44


should this be pushed to 2.x?


Same here


cache $context->m_request->m_params to a variable. also no empty line after { right above.


should this be pushed back to v2.x?


Targets (1, 17) (TARGET_UNIT_CASTER, TARGET_DEST_DB)

but then you need to add the teleportlocation to the db (just saying)


why return frag when you can just return table (or div around table)?


#101 this seems to be the first appearance of unicast routing attribute, implemented as BridgeDomain.route


why is this necessary?


Adding SpellTargetPosition Data in database for spell 74100 and adding spelllinked spell 74100 to 91847. should fix it correctly without an ugly script like this ^^


break after <tr>


static import


I know that animations are cool, but I think this one is really unnecessary and also first FAB in OverlayDetail has no purpose what so ever.


Hey @swannodette, it looks like the `string` alias is not set up.


I don't plan on making a release until Mesa works


out of alignment...


I've changed save function. So if we update now e.g. `Test Event` , which already has `test-event` slug, it won't generate `test-event-1` slug instead of previous one.

`def save(self, *args, **kwargs):
        if self.slug != 'default':
            return super(Event, self).save(*args, **kwargs)
        else:
            self.slug = orig = slugify(self.title)
            for x in itertools.count(1):
                if not Event.objects.filter(slug=self.slug).exclude(pk=self.pk).exists():
                    break
                self.slug = '%s-%d' % (orig, x)
            return super(Event, self).save(*args, **kwargs)`


got it!


remove extra line.


#101 ability to change unicast routing attribute was implemented here as BridgeDomain.unicast_route


you can just do
return
  <x:frag>
    {$table}
  </x:frag>


This looks weird. Don't wrap the first argument.


other posibility would be use of sai and just call the spells :) u would need to check sniffs whats the correct way.