Seems like this change resulted in a bug: https://meta.discourse.org/t/image-upload-inside-topic-doesnt-work-if-language-is-not-english/33585


Isn't better to have a type "key-value/splitter" with output ports KEY, VALUE_STRING and VALUE_INT that is able to handle both types of key-value packets?


This whole file is unnecessary, but building a sandbox fails without it. Having an empty file triggers a test failures according to [TE-630](https://openedx.atlassian.net/browse/TE-630) so I've placed this here in the meantime. @andy-armstrong


Please move this below Entry and add a doc comment that starts with:

// A ListAttr ...


If you are using a callback then you don't need to do this client check it is done automatically, you just set the var and everything is taken care of, and you should remove the same code from where you copied it from so it doesn't run the same thing again.


@librehat Currently, it's still an experimental feature and would take months to be stable. So don't be hurry, let's keep refining this protocol and see what will happen.


@SubJunk I've reverted 773f72e and partially reverted f93f21b. I left the Linux and OS X profiles at 1.7 as they were as I don't know if that was intentional or not. I bumped the "windows-jar" profile to 1.8 since that must be an omission as long as the "windows" profile uses 1.8.


This is incorrect. `if (!document) return;` will fail if document is not defined.

Try `if (sdlfjsdklfj) {}` in Chrome devtools, for instance, and you get
```
Uncaught ReferenceError: sdlfjsdklfj is not defined(…)
```

You need `typeof document`, sadly.


I would rather turn it into a set of exports like the game import thing.

Also, the ui syscalls is totally useless as the UI is directly part of the engine.


that's what I'm saying.  FreeBSD does not compile on anything less than 9 these days.  Everything else is considered EOL.

Or in other words, one may assume the lowest platform used is FreeBSD 9, thus checks like >7 are ALWAYS true.


I belive that IsList is true iff ListAttr is not nil.  In this case we should eliminate IsList and use e.ListAttr != nil to indicate a list


What exactly is this fixing?


We're no longer sending focus to this container so we don't need this.


It's because start auto calls stop anyway before doing anything else
but for the sake of appearances I'll change it in the code


Ok, pirmā problēma:
`composer require jurchiks/numbers2words`
met eroru:
[InvalidArgumentException]
Could not find package jurchiks/numbers2words at any version for your minimum-stability (stable). Check the package spelling or your minimum-stability

Tas pats, ja es iemetu viņu composer.json failā un palaižu `composer install`.


This allows all of the containers to be closed at initial load, rather than waiting for the transition to complete (which ends with `display: none` as the result).


I guess the "todo" item here is now obsolete?


nice. We might have to take advice on the "5 days" bit, most GPs would easily meet this target but some would object to being constrained by it. It works for me as a 'placeholder'. In fact I'd like to see it say "48 hours"


`getSlicesForArticle` returned NULL oder ein ArticleSlice daher reicht das so aus.


@alee The file system structure works for both DeployR and Radiant containers (just need different mounting points for different containers).


Okay. But these hipster ciphers are only supported by libsodium. Even Botan releases their 2.0 by time, it won't get into old Linux distributions. Anyway, I may do a benchmarking for typical algorithms when this protocol stabilise. Let's focus on new protocol atm.


das is_array hier müsste nicht sein, könnte man auch weglassen


@Yannik this has been added with a918376a7f1e4c92ebe25fc09ff2e4ca11c0bd85


No, this can't be moved to the parser because incremental parsing can't easily maintain the correct states.


Would you mind refactoring these kind of (long) if statements into functions with semantic names (e.g. findAdapterForResource or similar)


outbound = unbound?


Orchestrator sends a model to SFA. SFA forwards it to Translator. The model contains all information about resources, but for links is more information is required


This method appears unused.


Thanks for catching that bug!

This is the only place that I plan to use insertAt() in Feathers right now, so that's why I checked for it directly in this class. I also suspect that Daniel might create a utility function in Starling that checks if these new Array/Vector functions exist, and if he does, I'll use it.


@dnehls Looks like a lot of boiler plate code for an always empty adapter...


I was under the impression that if the key -- 1 in the case of 'mapper_lr_final' -- is the same then the mapper will not divide the datasets among the nodes. So in this case all the data will be processed by the same node. I could well be wrong because until yesterday I didn't know what Hadoop was...


# This commit made me feel like a Angry Cat.
![giphy Angry Cat](http://media.giphy.com/media/KQwZNdsEtMecg/100.gif)


With the above suggested change, this becomes "a List has the ListAttr field set."


hour="0", that's right


I have posted a few comments that should be addressed.

Please send mail to borman@google.com once addressed (I have not figured out this whole git and github.com workflow)

Thanks


Spaces in branch names? :-1:

Feel free to create whatever branch needed, and let me know where to push this PR.


`FlagsToModifiers.CONST` Surrounding code uses unqualified references.


let's set this to PerformanceTimeInterval.WEEK, rather than the magic number 86400


you don't need the if...else since we provide a default


root能远程登陆


Nevaru izveidot savu package, jo:
The vendor is already taken by someone else. You may ask them to add your package and give you maintainership access. The packages already in that vendor namespace can be found at `jurchiks`.


Also, could you change to the commit message to:

`Allow user to specify time interval when importing performance data`


What is the advantage of using `combineReducers` instead of the manual way? Now we are mixing POJO with immutable, i.e. now we need helper methods such as `stateToJs`.


Izdzēsu savu package. Try again

Matīss Jānis Āboltiņš
---
Web: http://mja.lv/
Phone: +371 28369585
Twitter: @matissjanis <https://twitter.com/matissjanis>
LinkedIn: Matīss Jānis Āboltiņš
<https://www.linkedin.com/pub/mat%C4%ABss-j%C4%81nis-%C4%81bolti%C5%86%C5%A1/4b/706/633>
Github: MatissJanis <https://github.com/MatissJanis>

On Tue, Sep 22, 2015 at 5:36 PM, Juris <notifications@github.com> wrote:

> Nevaru izveidot savu package, jo:
> The vendor is already taken by someone else. You may ask them to add your
> package and give you maintainership access. The packages already in that
> vendor namespace can be found at jurchiks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jurchiks/numbers2words/commit/8b4aad79b9aa3e48d14f85e97a75b0c342c402c4#commitcomment-13368728>
> .
>


Shouldn't this be called `DeviceState` or something? It's not just the initial state that has this record format...


No problem. This was a huge headache for me, I hope you get it right, too. :)


As far as I understand, it is only used to translate multi-level security labels, and we have `SELINUXTYPE=default` in [`selinux/config`](https://github.com/hashbang/shell-etc/blob/master/selinux/config).


Deleted Williams, but result seems to be worse than before


e se vida for 0?


@Myoldmopar Did I break something or is something up with the CI machines?


'Puke'


Puked


Get exploder


There is negligible improvement in performance having squashfs in RAM, with either RPi or RPi2, but you will lose 110MB+ of RAM.

You can mount boot= from wherever you like, including /dev/sda1 ie. your USB stick, but by doing so you'll confuse the automatic update procedure and potentially hose[1] your system when updating from tar files dropped into the Update folder.

Personally I'd suggest investing in an 8GB NOOBS SD card - very good performance, cheap as chips.

1. On the RPi, the kernel, firmware and overlay files are always read from /dev/mmcblk0p1, however the update procedure will write these files to your boot= location eg. /dev/sda1. Next time you boot, you'll boot with the old kernel & firmware (from SD card) and only your new SYSTEM file being read from /dev/sda1, with unpredictable results.


Uztaisīju: https://packagist.org/packages/jurchiks/numbers2words
Bet vienalga `composer require jurchiks/numbers2words` met to pašu eroru.


This was actually fine before. Perhaps you got confused with C++# because it still uses many public fields. This is a bad practice and we're going to eliminate it in time,


*this* could probably be a separate module as well


Should this still be `synchronized`? Also, should I add back in the comments (and add comments for the rest of the methods)?


Sorry, i mean on the source code. I can't seem to understand which part i should do that. Or are you suggesting another solution ? Thanks


yes, ```mono16 == uint16```, my bad, it was more about ```int16```: how should they be handled ?

Right now, cv_bridge scales:
https://github.com/ros-perception/vision_opencv/blob/indigo/cv_bridge/include/cv_bridge/cv_bridge.h#L180

This Python piece of code is supposed to convert WITHOUT scaling. I actually believe there is a bug in the current code:

=> if ```encoding``` is ```uint16``` or ```mono16``` or ```16UC16```, truncate to ```uint8```
=> if ```encoding``` is ```int16```, truncate to ```int8``` (which it does not do right now as it converts to ```mono16``` and truncates to ```uint8```: it should convert to ```int8``` and then add 128

What's missing is also that kind of function (in Python): https://github.com/ros/common_msgs/blob/89788101b21aec550ef4d0b93c7f668e9e8d19c0/sensor_msgs/include/sensor_msgs/image_encodings.h#L182


Ignite exploder


Man gan iet. Iespējams, ka tev jāiztīra kešs.

Matīss Jānis Āboltiņš
---
Web: http://mja.lv/
Phone: +371 28369585
Twitter: @matissjanis <https://twitter.com/matissjanis>
LinkedIn: Matīss Jānis Āboltiņš
<https://www.linkedin.com/pub/mat%C4%ABss-j%C4%81nis-%C4%81bolti%C5%86%C5%A1/4b/706/633>
Github: MatissJanis <https://github.com/MatissJanis>


Any reason this was merged into 3.4 branch rather than 3.4.9?


Kāds kešs? Es šo komandu izpildu terminālī.


I would not put `page_size` in initializer. Client construction is not responsible for providing `page_size` me thinks. As a user this confuses me because you need private knowledge why you are doing this setup.


ya, i mean unbound


tested and working


Yea – also keeps pyflakes happy in different places, where you're doing aliasing exports :)


não é só quando atacado por anão que usa o escudo?


Also add that path as a constant in lib/pf/file_paths.pm


Initial code pushed on GitHub repo.


@thz this seems to be needed for some bitbake targets as otherwise bitbake fails
```
 * check_conflicts_for: The following packages conflict with openssh:
 * check_conflicts_for:         dropbear *
```

any idea?


"if (!ntokens) {
		if (sscanf(buf, "%u\n", &val) != 1)
			return -EINVAL;
		for_each_possible_cpu(i)
			per_cpu(sync_info, i).input_boost_freq = val;
		goto check_enable;
	}"
Should it be here?

By removing this part and inserting:

per_cpu(sync_info, 1).input_boost_freq = 1497000
per_cpu(sync_info, 2).input_boost_freq = 0
per_cpu(sync_info, 3).input_boost_freq = 0
per_cpu(sync_info, 4).input_boost_freq = 0

Thanks !


i forgot to delete it. Now done


You don't have to.  It's optional, set up with the same default that exists today, and it's an attr_accessor if someone wishes to change it after the client is created.


This fixes the unit test for you on Windows? If so, that would be a good thing to mention in the commit message in the follow-up paragraph.


good work , continue,  ill check updates when i can


[This](
https://stackoverflow.com/questions/21043551/invalidargumentexception-composer-install-symfony
)

Matīss Jānis Āboltiņš
---
Web: http://mja.lv/
Phone: +371 28369585
Twitter: @matissjanis <https://twitter.com/matissjanis>
LinkedIn: Matīss Jānis Āboltiņš
<https://www.linkedin.com/pub/mat%C4%ABss-j%C4%81nis-%C4%81bolti%C5%86%C5%A1/4b/706/633>
Github: MatissJanis <https://github.com/MatissJanis>

On Tue, Sep 22, 2015 at 5:43 PM, Juris <notifications@github.com> wrote:

> Kāds kešs? Es šo komandu izpildu terminālī.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jurchiks/numbers2words/commit/8b4aad79b9aa3e48d14f85e97a75b0c342c402c4#commitcomment-13368919>
> .
>


tested and working -- remove styles, I meant this commit to say


Watch out for these missing EOF newlines. Every file should have a newline at EOF.


merge


Why, it's a class name and thus camel case would've been right?


Sim, eu estava com esta dúvida, então resolvi deixar como você deixou na classe, mas um objeto com x quantidade, faz mais sentido, vou alterar.


Eq=vomit ()[]||\\*+[x]*$git/vomited x enumerator starts 'innertelligent' puke keep puking empty puke


I understand @gunturtif , in a first look it seems complicated. But let me try to clear some things.

All the training data and examples were created for me, I created a special network (client-server) to the experiment, and with a sniffer I could see all the data, I choosed the clean packets (normally HTTP requests or other things that are not a attack). Then I created the vector with 50 bits like the post above. With these 50bits vector I created the "training.txt" file, and insert the vectors including an '-1' in the first line for the clean examples.
I did the same thing to the bad data, I simulated attacks that I plan to teach the RNA, so after capture the bad packets and make the 50bits vector, I insert it in the "training.txt" and put an '1' in the first.
Now I've around 100 examples at all, all made MANUALLY. Each line in "training.txt" is a example.

After alter the "training.txt", you execute only with the arguments '--mode 2' or '-m 2', so the RNA engine will remake the value for the weights. After the conclusion you can use the system normally.


You want a javadoc at the start of every class, of the form:
```java
/**
 * A short description of this class.
 * <p>
 * Follow-up paragraphs, if needed, provide more detail on the purpose and function of the class.
 * </p>
 *
 * @author Leon Yang
 */
```


Hm, I find that odd, if not even a bug. The sha1sum manpage is silent on it. Anyway, that's something for a separate patch and pull request in my opinion, it's not related to this particular change. Actually it seems that diffutils is now a requirement of rpm-build anyway (at least on my F-22 box) so this comparison could be changed to use cmp(1).


This is a memory leak.  Every time you come into this method you will create a new dialog.  If you wish to do it this way, initialize `addScanDialog_` in your constructor with `addScanDialog_ = 0;`, then in this method you would wrap the creation of objects in an if-statement like:

```c++
if (addScanDialog_){
    // Do some stuff.
}
```

You will then need to take care of it in your destructor with code like this:

```c++
XPSDisplayWidget::~XPSDisplayWidget()
{
    if (addScanDialog_){
        addScanDialog_->deleteLater();
        addScanDialog_ = 0;
}
```


Use IDEA. It will auto group 5+ imports of the same package.


Finally.


Falta de atenção minha, faltou um <= no segundo if.


yes, I didn't noticed.


LGTM


Looks good.  I don't think you need to say 'book a non-urgent appointment'.


I did some time ago https://youtrack.jetbrains.com/issues/RSRP?q=by%3A+a553


Agreed, I think that's something that should be in a separate patch/PR.


Wow. :-)


Juro que estava identado quando eu enviei.


tested and working


I think resharper can take care of the renaming. At least, I noticed it's capable of finding derived names and will change those as well. So that part might be easier than you think.

By the way, an alternative would be to simply leave out the prefix "Render" for the general classes.

E.g. something like:

    Chain<T>
    AudioChain : Chain<IMediaSample>
    RenderChain : Chain<IFilter>

Or, if we combine the IFilter types:

    Chain<T> : where T : IFilter<IBaseType>
    AudioChain : Chain<IAudioFilter>
    RenderChain : Chain<IVideoFilter>

That aside, there are some difference between the AudioScript and RenderScript, which might make it difficult to treat them in a common way.

Firstly there doesn't seem to be an equivalent to IRenderScriptUi for AudioScripts, but that should be easy enough to remedy.

Secondly the IAudioScript is quite a bit different from the IRenderScript. The "Process()" and "Render()" are similar enought, but where IRenderScript has a method "Update()" which is called whenever something changes, IAudioScript has *several* callbacks depending on what changes. Might be better to generalize Update() or simplify the IAudioScript methods. Things would also be easier if they can be made more similar, but at the very least they'd need to have a common ancestor.

To make it possible to give them a common UI class I think you'd need a structure like:

    IScript { ... }
    IRenderScript : IScript { ... }
    IAudioScript : IScript { ... }

    IScriptUi<out IScript> : IExtensionUi
    {
         IScript CreateScript();
    }

    IRenderScriptUi : IScriptUi<IRenderScript> { }
    IAudioScriptUi : IScriptUi<IAudioScript> { }

The "IRenderScriptUi" and "IAudioScriptUi" are basically just aliases (which unfortunately don't exist in c#), so they aren't strictly necessary, but it's probably best to add them for clarity (although that means you can't have just one generic class which implements both IRenderScriptUi and IAudioScriptUi, which is a shame)


Пробовал собрать сегодня, но падает с ошибкой:
storage.c: In function ‘hex2string’:
storage.c:22: error: ‘for’ loop initial declarations are only allowed in C99 mode
storage.c:22: note: use option -std=c99 or -std=gnu99 to compile your code

Если в CFLAGS+ добавить -std=gnu99, то собирается. gcc version 4.4.7


Temporary types?  You can't typically return values from void functions.


For commits like this, it is very important to explain _why_ you deleted it. In this case, you should have another paragraph in the commit message that states something like:
```
These classes are now autogenerated; see:
src/main/templates/net/imagej/ops/math
```


ありがとうございます！
`executeOnFile()`の方が良いのはファイル形式のチェックくらいでしょうか？テキストバッファ渡した方がioない分早いかなと思ってるんですが、大差ないか。


Note, that we use new0() here, which means the structure is actually already initialized with all zeroes, these two lines are hence unnecessary...


It might be less confusing for beginners to add directly to the db with Tasks.insert()


Check for null


Use `this()`


e se o item ficar com quantidade negativa?


Right, but pyflakes disagrees with camel case...


Is `--id` really a command? I'd rather put it in a different section (maybe 'parameters' or similar) together with the `-h` and `--help` parameters.


This is where you'd need to delete the XPSScan that is returned from `takeItem()`.

```c++
XPSScan *removedScan = scanListWidget_->takeItem(numberOfScans_-1);
removedScan->deleteLater();
```


# This commit made me feel like a Confused Cat.
![giphy Confused Cat](http://media.giphy.com/media/bftrwFOK5haIE/100.gif)


Types probably not needed


Or conversely, you need to do that in your model.


Last argument should be wrapped


Maybe also capitalize 'log'?


Your commit headline is too long. See [how to write good commit messages](http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html).

This commit also changes too many things. For example, it seems to superfluously reorders many methods (it is hard to tell because the diff is too big for GitHub to show).

If you want to reorder things, that should be done as its own commit. Then add and/or remove things in their own commits, so that it is easier to see what has been added and/or removed.


I see. I have a couple of other problems with the constructor :

- You can pass non-meaningful things like Messy for the state
- The default comes after the flags, whereas for all our other constructors it doesn't

The conventional image merge actually checks for inputs without a connection and skips them, which might be an option for the DeepMerge. Can we leave this constructor out of the initial PR and then reconsider when we have an immediate use case for it? In general I'm reluctant to change APIs without having an immediate use case (or two) - in my experience with Gaffer so far I've often found I get it wrong if I try to do things in advance.


Any idea why 21 means "directory"? Is this portable? Can't we use some function from glib for this?


tested and working


What are these `push_back()` and `pop_back()` methods?


The hub doesn't by default use pronamic widgets so another option could be to comment out ...


Current workaround. Will be fixed in the next version.


```
$ lua -e 'function t(p) return require("lgi").Gio.File.new_for_path(p):query_file_type(require("lgi").Gio.FileQueryInfoFlags.NONE) end print(t("/etc"), t("/etc/passwd"), t("does-not-exist"))'
DIRECTORY	REGULAR	UNKNOWN
```
https://developer.gnome.org/gio/stable/GFile.html


fjdfgjdhgjksdfgjsdfjghsdfhsdfghdfh


I'm not seeing the uncaught exception/dump now that I had been seeing when I had submitted the pull request now that I tried testing with/without my check and your streamlined suggestion.

AutoPkg v0.5.1 does not have the problem (but v0.5.0 and earlier did), so the pull request isn't necessary.

Cheers.


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


Why do you use 50 layer used ?


I don't like the case logic around `$op.name` here. There must be a way to avoid it.

How about simplifying the class names. You already have `ConstantToArrayImage` on the outside... so why not things like `ConstantToArrayImage.ByteAdd`, `ConstantToArrayImage.DoubleMultiply`, etc.? Then you don't need the prepositions or the case logic, right?


The ci machines were having trouble, but they should be coming back healthy
again as we speak.


Line 8 shouldn't it be export V=Tesla-5.1.1 ?


Similar comments as above for all the templates, regarding naming. Keep it shorter and less complex if possible.


To __CA__:
  What's your strategy to find all the links of these sites and how can you filter them out?
It's intelligent.


Естественно, я изначально имел ввиду международные телефоны. Междугородние я написал впопыхах. Да еще и с ошибкой :)


It seems this change to the method "getRelativePath" has broken the correct generation of relative symlinks.
There is one "../" added to the path and therefore the symlink is not working.
Tested with Magento2 "magento2-module". Getting a broken link below app/code/Namespace


I've mad an issue (#170) where we can discus this further, since this is bit of a weird place to discuss this.


ro.build.version.release=5.1.1 in build.prop
On Sep 22, 2015 8:27 PM, "John Brewer" <notifications@github.com> wrote:

> Line 8 shouldn't it be export V=Tesla-5.1.1 ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/akhilnarang/vendor_tesla/commit/340933e599aad7ee73f1f545771e27f886cbaf49#commitcomment-13369325>
> .
>


Why do you need margin of 250px here


I guess that I'm missing the point. Why wasn't there any encoding problems while getting `lemmas` or `semantic_tags` alone?

As Mongo uses utf-8 everywhere, shouldn't we always decode saved strings even without mongodict?


Splitting a debug message over multiple lines is not allowed... Then it is no longer grepable.


I've adapted the comment now and added some more tests for the configuration setting


Immutable had to be shallowly (via toObject) converted to object anyway. So the code is actually refactored, not redesigned.


@zcbenz This should have been a major version bump as it doesn't build against the current version of Atom. We've locked it down to 5.1.11 in atom's `package.json`, but maybe could unpublish this when you get a chance? Thanks!


@mwr our default and recommended deployment strategy - copy.
Have you tried symlink on previous version of plugin?


Use `createPartitionName()` from `HiveUtil`


What????


this part is probably useless


Don't we have a Dictionary type here


そうですね。
`executeOnText`には今のところファイルフォーマットの指定がないので…
`executeOnText`でフォーマット指定できるようにすればいいのですが、任意のフォーマットとパーサをサポートできるようにしたいので、フォーマット or 拡張子 どっちで指定するのかで迷って実装してない感じでした。


@gunturtif This project was presented to obtain a degree, it was result of a big research and a 50 layer showed the best perform in my tests. I had to take care with the slowness in the RNA, besides a lot of information can cause overfit. 50 was good for my choice.


I'm using it as namespace, so for stuff of main.fbp I'm using MainButton, for goal-detector, DetectorLed, DetectorSensor, etc.


Are you sure xlim is necessary here? Also I would prefer 2*yi1


Nothing, I forget to delete that comment.


Also 2*yi1/xi here


Hi.
Is there a way to modify row data? We are using some macros in data tables, which we replace with actual value. Now replacing doesn't work because data.raw() return new instance each time.


Tole vrstico pobriši, saj v `slovenija1.pdf` nič ne narišeš, datoteke ne zapreš z `dev.off()` in je niti ne uporabiš v poročilu.


replace with 'src_images = self.filter_images()'


:bulb: While 7-zip does an excellent job unzipping tarballs on Windows, it would probably make more sense to have `.zip` archive available for Windows instead, to avoid third-party software 7-zip dependency.


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1347 of 1347 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)


means the features that you use only 5 ie protocol, flags, Destination Port, Packet Size, List of keywords


LGTM


I just removed the definition since it was only relevant for msys1.


Dangerous. :speech_balloon:


Yes :+1:


It's 4 inches diagonal.  It's 1136 x 640 pixels, but from what I gather it uses pixel doubling technology, so it is equivalent to  568 x 320.


Resolves


This breaks the sortable on `/dashboard/ranges/3/products/`.

Issue here: https://github.com/django-oscar/django-oscar/issues/1853


This should be named something closer to what it means.


This is not being used.


Leave a comment


uh oh Amanda's wildlife rescue will be opening soon


Actually there are a variety of psql commands and arrays that are being used in the file, but yes I can make a function that should encompass a majority of functions.


Somehow now I need to activate the mod ABEMTool in order to be able to open the mod in the game editor and push it to STEAM ,


@bdbcat why shouldn't we track .gitignore?


Without thinking in whys, the code is beautiful.


Pointless line.


Wouldn't this already be an empty array?


It was already fixed, forgot to add the file. Check in the PR


aqui ele tbm não tem que verificar se morreu?


I'll take it off for now, but I do think that it's important to be able to
initialise a plug as empty. I was just using DeepMerge as an example - I
can come up with more!

It could be a Boolean default, although I felt that defining the default
statePlug value would be better than a straight Boolean that didn't
correspond to an actual value on a resulting child plug.

Happy to leave it out for now, but we should definitely revisit.

Hugh Macdonald
Nvizible - Visual Effects

020 3167 3860
07773 764708
www.nvizible.com

On 22 Sep 2015, at 15:54, John Haddon <notifications@github.com> wrote:

I see. I have a couple of other problems with the constructor :

   - You can pass non-meaningful things like Messy for the state
   - The default comes after the flags, whereas for all our other
   constructors it doesn't

The conventional image merge actually checks for inputs without a
connection and skips them, which might be an option for the DeepMerge. Can
we leave this constructor out of the initial PR and then reconsider when we
have an immediate use case for it? In general I'm reluctant to change APIs
without having an immediate use case (or two) - in my experience with
Gaffer so far I've often found I get it wrong if I try to do things in
advance.

—
Reply to this email directly or view it on GitHub
<https://github.com/HughMacdonald/gaffer/commit/13f668427c2c669f14fb7ea09e2c77dff2ecdf12#commitcomment-13369245>
.


raw data that you use in the training data like what ?, I want to try using a dataset of KDD cup 99, what can


This should address #2 but needs more testing before I can merge it into develop.


ok :)


What if this errors?


Formatting in this file.


Are these underscores here on purpose?


Does it work with different device trees?


Yes, works fine now.
Thank you!
8-)


Resolves #8


could the ids be passed in or read from attributes on the element?


`elif`


Why set `user`?


For your loop on line 27: Python makes it really easy to do loops with a set number of iterations like so:
```python
for i in xrange(N):
    # do something N times
```
where i will iterate through the values 0 ~ N-1, inclusive.


There is a version property (with a different value!) already defined for this.


**Name** Tony Gilerovich
**Title** Tester
**Location** Milwaukee


será que não poderia ser só um item flecha e mudar só a quantidade?


I hope the next one isn't a bear.


rax retry tests


Second time seeing this reduce iterator -- could this be centralized?


Izskatās, ka packagistam kaut kas aizgājis grīstē, ne cache iztīrīšana, ne restarts, ne versijas palielināšana, ne versijas taga push, nekas nepalīdz...
Packagist lapā rāda pareizu pēdējā apdeita laiku, bet versija kā nemainās, tā nemainās, un, acīmredzot, composer defaultā pieprasa stable versijas, bet man packagist redz tikai dev-master, tāpēc met to eroru.


@Flameshot :(


Don't have to check `$scope.user` three times.


@yehudasa it looks to me that _Content-Type_ simply shouldn't be present in _rgw_to_http_attr_list_ anymore as it must be treated differently. I'm preparing a patch.


This is a lot of requests potentially.


Hm, any reason for this? Now my standard always overrides the db value stored in the meta field. I was assuming standard would be a default but once this is set, shouldn't it be similar to the text field?


Hmm, that's odd, it is only required by tests and should be created by the test and then deleted by it, [see here](https://github.com/alphagov/smart-answers/blob/a3d0f20dea7e411fc913f34487b0ab5a61b3e0ec/test/unit/smart_answer_files_test.rb#L131-L148). I know you mentioned it fails on your VM, I can investigate it later, but I'd remove it from this commit as Jenkins builds (and tests on our machines) seem to pass without it, is this correct?


If it's a larger task than some minutes, please create an issue and don't fix it on this branch.


This change appears to break scripted testing. The Input type="text" is required for scripted texting.


I'm curious :) Why do you need this?


Why is there a css file in this directory?


Pamēģini composer require jurchiks/numbers2words@dev-master


> Shouldn't the function be named is_older_than?

yes


Why was this added? Purely for debugging? Or is it used elsewhere?


will do


WAT


realmente precisa desta propriedade?


Would it not be less confusing to have a separate setter method or simply `attr_accessible`?


Why contenteditable?


This line throws a null pointer exception if you issue `stream list` when one of the modules is in an unknown (or still staging) state. For example, doing a list immediately after a successful deploy returns in the shell often fails with a NPE.


This may be not the right place to this. We have other components that need keyboard-based navigation as well. Previously, it was in a mixin which you can attach to anything. Examples of such components: vcl-token-input, vcl-autocomplete and may be others.


closes #5


Pilnīgi tas pats erors.


Note :  call once `use strict` into the main JS file is enough


Leave a comment on 0bf90d2


I wouldn't put more than very simple express in html


merge


typo?


screensaver.asteroids https://github.com/notspiff/screensaver.asteroids
typo, I guess ?


Because we are changing the weight (which is an int) between 'min' and 'max' gradually and we don't need to cast the enum back to int.


Ļoti dīvaini. Man šī pati komanda strādā flawlessly.


Tev visdrīzāk minimum-stability ir cits.


Would you be able to use one of the [smartdown fixtures](https://github.com/alphagov/smart-answers/tree/master/test/fixtures/smartdown_flows) here instead of a real flow? I think it could be worth updating those fixtures with content_id too.


> Shouldn't be the comparison done with the > operator if we want to check that it's older than a given date?

the old behavior was >=, but you are right, then the name is misleading


This is in `BucketUtils`, no?


# This commit made me feel like a Confused Unicorn.
![giphy Confused Unicorn](http://media.giphy.com/media/I1hC7mQZpW4jS/100.gif)


Note that Javadoc is HTML. If you mouse over the class name in Eclipse, it will show you the rendered Javadoc, and youll see that this block does not render the way you would want.


Remember (mc-1)/8 will only be an integer since mc is an integer.
If you are expecting a decimal...you would need to cast it..


Please keep the whitespace consistent when indenting the table-like hash structures.


Fortunately 2 is ok (only repairs when ships are critically damaged). If it was set to 3 on the other hand, that would've been bad :P


Date filter.


.gitignore is a user preference file, IMHO.  Makes git more convenient to use locally, so that one can ignore local build products in the same tree.
It is not really necessary for any build to succeed.

IMHO, opinions solicited.

Dave


use own sort()? Overall I must say I would prefere this as a prerequisite.


Arent they the methods used to add/delete from vectors?


Here the xlim declaration is only usefull when data contains no null coverage, to see regression lower limit. (for being more rigorous we can to add it to the first regression plot)


Formatting in this file.


Must properly bash-escape, I thought I did amend this branch already. Will do when addressing other comments


<img width="785" alt="screen shot 2015-09-22 at 8 24 32 am" src="https://cloud.githubusercontent.com/assets/1016365/10022782/67d6233a-6103-11e5-8e7c-06364d61b102.png">
the website has it listed as `Angular 2.0`


I've just been writing it that way as a temporary thing.  So I can think about what the functions are without thinking about a return type yet.


Elsewhere in ops, we use `private long seed` rather than `Random` instance. Which do you like better? My naive intuition is that the `long` is better because it is easier for other SciJava software to support—even non-Java software which might not like various Java objects. But I could be convinced that `Random` is better somehow. Regardless though, we should be consistent throughout Ops and use the same strategy for RNG everywhere.


I am not sure I agree with this change, it makes it possible to route stanzas inconsistently. If routing decisions need to be taken, shouldn't that be straight in "routeData"?


Why is this unsupported?  Are you intending to implement this later?


I guess this means we have 42 Smart Answers. It doesn't get any more symbolic [than this](https://en.wikipedia.org/wiki/Phrases_from_The_Hitchhiker%27s_Guide_to_the_Galaxy#Answer_to_the_Ultimate_Question_of_Life.2C_the_Universe.2C_and_Everything_.2842.29).

Minor: I've seen you're changing this later, but it might be worth mentioning this in the commit note.


What kind of scripted testing?  Testing of WPT itself or automating the UI for submitting tests?  Automated testing should use the API.


Yo te puedo comentar solo, lo veo y si me parece puta madre luego hago un pull.


Eliminates "File now" link because there is not direct link to upload to FEC site. Is done through software.


@ziogaschr to compile with bitcode using xcode 7.x all static libraries that is included should also support bitcode. That's why these lines were added.


Couldn't we combine these two cases (basically is overlapping)


Just a heads up, the "--pkg" option will soon be removed from makepkg.

https://lists.archlinux.org/pipermail/pacman-dev/2015-September/020355.html


This commit does more than refactoring. E.g. this commits adds a new method: _operator_onload. Please don't do it again.
Also this commit leaves Wiring.js failing to pass jshint and jscs. Check the [guide about how to contribute](https://github.com/Wirecloud/wirecloud/blob/develop/CONTRIBUTING.md).


This is a bit arbitrary but I can see why you added it there. I don't like it but I can't think of anything better. @floehopper @chrisroos?


Ahh in that case, try static instead of extern. I'll take a deeper look into this later.


existia alguma definição de "perder escudo" lá na atividade?


s/solution/ranges


I thought that NULL is converted to QVariant(), and if yes then QVariant() is returned when 'argument == QVariant()' what is the same as returning 'argument' itself. In your example, your line would return the same ( QVariant() ) as my simpler version. Maybe I do not see something.


didn't mean it
fixed anyway.:)


Shrink 'result' if to big.


I like your user picture by the way, but this bio is a load of :hankey:


ok.


No comment


Uhm, right. I meant to check for stuff in `/usr/lib/nx/X11/`, not for stuff in `$libdir`. Basically, I wanted the scripts to create the symlinks only once and not OVERWRITE them. I didn't notice that the checks are for the original X.Org files but for some reason assumed that's for the destination files, which was indeed stupid.

Instead of reverting, some other fixup is needed, I think. I'll look into it again.


Fixed


Did you check that this parameter is set when compiling normal Theano function without scan? If so, this could speed up other stuff then scan.


Fixed.


faltou verificar as flechas =)


I use a [global `.gitignore`](https://robots.thoughtbot.com/global-gitignore) for user preference (things like Thumbs.db on windows, editor temp files, etc.), and repository `.gitingore`s to decrease the chance anyone accidentally commits files likely to be in the repo that I don't want to me committed.


Where's the assertion/expectation in this test?
Any reason you did not include meta description?


in case we need to extend result


Typo in "App stared", should be "App started"


In retrospect now that I know more stuff about it we could have leveraged `bower` to build [tinyMCEs javascript at build](http://www.tinymce.com/wiki.php/Installation_using_package_managers) via a custom heroku buildpack, but commiting the whole package works too for as long as there is no tinyMCE CDN (if we keep up with tinyMCE). I believe the more popular WYSIWYG editor however is ckeditor, which I chose not to install back in the day because it seemed hard :P


We cannot use Dictionary type. It is unable to convert it in the method block. Hence sticking with this.


fine :exclamation: :boom:


Please consider changing "Large Shield" to "Greatshield" as that is what they were called in Dragon's Dogma, and in the current patch.

"Large Staff" should also be "Archistaff."

http://dragonsdogma.wikia.com/wiki/Archistaves


@tremainebuchanan why differentiate? Why not allow for an input to have any possible units? Inputs can be weight/mass, length or any other unit of measure.  The only thing that has restriction are demands/supplies because of the matching capability.


Maybe this should be final, too.


It's a long story, but basically I always start from Developer Console's Element Inspector.
If I can find the desired link/image, then the only thing I need to do is to figure out the element's selector.
This is the simplest case.

If it's not that simple then I'll check scripts on the page, especially the event handlers on the links.
Sometimes I just need to repeat what the scripts does.

Then I'll check the Network Inspector, some sites use AJAX to retrieve the real link.

Sometimes the solution is very tricky, but you can find all clues in these Developer Tools.


"recebeFlechada" ? mas não foi um ataque com espada?


>  returned NULL oder ein ArticleSlice-Array daher reicht das so aus.

oder ein Object, wenn nur ein Slice gefunden wurde


@rxaviers sir,

Hi! Sorry, but I don't know Arabic. Won't be able to help much here.


`PublishArtifact` doesn't carry the information, so there is no way to populate it here. See https://docs.gradle.org/2.7/javadoc/org/gradle/api/artifacts/PublishArtifact.html .

Moreover `extraInfo` is a property of the publication, not of the artifact. One might think that this shouldn't be in `PublishArtifactInfo` then, but JFrog has been consistently using this class in `BuildInfoPublicationsTask.java` for both the artifact properties and the publication properties.


Coalesces a vector of ranges


this is important for test accuracy: set to ~0.01 to ensure 1% compliance with expected. here i had it set to 5% tolerance that is +/-2.5% around the expected value


Deshalb muss hier auf array geprüft werden. Das else weiter unten wandelt dann das Objekt in ein array


I think that's a symptom of not calling init() first?

Sent from my iPad

> On Sep 22, 2015, at 12:15 AM, Kristoffer Snabb <notifications@github.com> wrote:
>
> What was the initalization bug, I get this error from this change:
> TypeError: Cannot use 'in' operator to search for '/here/is/the/path' in null
>
> —
> Reply to this email directly or view it on GitHub.
>


Heh, ignore me :)


poderia parar o `for` quando achasse a espada, correto?


@Flamefire

ah, darum funktionieren die nightlys nicht mehr! du hast die file (einfach so) geändert ohne das zu kommunizieren!
konsequenterweise muss die build_version.h nicht mehr über cmake benutzt werden.

siehe stellen in der die build_version.h benutzt wurde: b5fa18aa6330bacb0ca9490f8398fee7ed734eea


Does this mean that '.' is twice as frequent as any other char? Should we just let this be handled by the whitelist?


Its temporary until we figure out something to deal with unicode. For now windows user get an error if they try to print some strange character. We can handle the exception somehow and print without that characters in windows... Its something to discuss (or not...).

we can fix with something like this in the main function:

try:
  print(joke)
except:
  print(joke.encode('ascii', 'ignore'))

Not the best solution... but should work.

Arratsalde on!


I fixed the problems spotted by jscs (so this commit is not going to be merged directly), jshint problems requires a lot of work... added to your homework ;-).


Can you update the README to explain how to create these content_ids?


add preconditions


LGTM


Note that this can't lead to an NPE. I've just checked Gradle's source:

* `DefaultIvyPublication::getDescriptor()` won't return `null`, and
* `DefaultIvyModuleDescriptorSpec::getExtraInfo()` won't return `null`.


rough


@VoyagerOne: I understand this. In this case I need an integer because I'm tuning log function using 8 discrete values. mc_tune is an array, not a function. I'm sorry for not deleting the code from yesterday. Click on parent to see more.


add Check


In your model for improving I/O bandwidth, perhaps it would also be interesting to consider what your "large table" would correspond to in a real machine. Then it would be important to remember access times for shared memory, like DRAM or L3 cache. Are there any cases in which distributing data to the L1 and L2 caches for many (infinite) processors is faster than having a finite number of processors access shared memory?


Yeah, we should use a fixture flow instead of the real thing here.


> hum?
Somehow  i thought we should use print statements in scripts


In addition, "1-Handed Sword" is technically incorrect because Fighter uses a variety of 1-Handed weapons such as Maces.


Nitpick: Use &suse; here. (Inevitably, the company will change its name again...)


This wasn't necessary but I think it's a nice feature: doing so, the user-defined tokens set on the publication can also be used inside artifact paths.

Indeed I think usually the "directory" where to find an artifact is a subdirectory of the one where to find the `ivy.xml`. Which means that in this case, the artifact pattern's set of tokens is a *superset* of the publication pattern's set of tokens.

Thus I think this makes sense to inherit the publication tokens for the artifact resolution too as, as said, the artifact path is likely to contain the publication path.


Use &slea; here.


This should fix issue #11 about expire notices.


check out np.diag() for filling in diagonal matrices


For ExtraStore you should be importing the pfx into an X509Certificate2Collection, because the PFX contains 3 certs; the root we're maybe not supposed to trust being one of them.


por que tem `Boolean` no nome dos métodos? não seria melhor ajustar o nome para `existeItemComDescricao`?


You are right, this would make it possible to route a stanza to another resource than specified in the "to" attribute.
If I didn't misinterpret the code, routeData does not have a clue about the client connections' message priorities and thus cannot do the decision.

Would it be OK if I changed the code to only accept a resource when the "to" element is a bare JID and reject sending (return false) when  "to" is a fullJID and a resource has been specified?


range = rhs.add_range();


Editor doesn't like curly quotes? Also, who put curly quotes‽


In Gradle 1.8, `IvyModuleDescriptorSpec` didn't exist. It was called `IvyModuleDescriptor` and didn't have the `extraInfo` property. Gradle 2.1 is the lowest version with which this feature can be implemented.

I use an explicit `distributionUrl` here as otherwise it pulls `gradle-2.1-bin.zip` instead of `gradle-2.1-all.zip`. `gradle-wrapper.properties` looks like we prefer `gradle-*-all.zip`.


I have a bug in X509Certificate2(pfx) wherein I return the first non-private-key cert instead of the first-private-key cert.  So this is the intermediate cert on Unix, and the leaf/EE cert on Windows.

Should still work in the end.


Since we weren't changing any of the defaults anyway (http://doc.qt.io/qt-4.8/qinputdialog.html#getText) then I think this is fine.


This looks like a good option to have, but maybe it should be an explicit option, e.g.

```js
cxtmenu.destroy({ removeContainer: false }); // default true
```


Now that we build with Gradle 2, Groovy 2 is being used. I took here the same version as in the project `artifactory-maven-plugin`, which is the only other project having this dependency.


I don't like this. How about stubbing `SmartAnswer::FlowRegistry.new(FLOW_REGISTRY_OPTIONS).flows` and `SmartdownAdapter::Registry.instance.flows` to return known flows that you can assert against?


Humm.. looks like it's still problematic with http read,

file.zip / 208'702'657 bytes :
```rebol
  length? read/binary http://domain/file.zip

*** Runtime Error 1: access violation
*** in file: /red/allocator.reds
*** at line: 207
***
***   stack: red/allocate-virtual 268439552 false
***   stack: red/alloc-big 268435476
***   stack: red/alloc-series-buffer 268435456 1 0
***   stack: red/expand-series 1287F014h 268435456
***   stack: red/alloc-tail-unit 1287F014h 1360
***   stack: red/binary/rs-append 0201D154h 02800000h 1360
***   stack: red/simple-io/request-http 0 0201D144h 00000000h 00000000h true false
***   stack: red/url/read 0201D144h 0201D134h 0201D134h true false 0201D134h
***   stack: red/actions/read 0201D144h 0201D134h 0201D134h true false 0201D134h
***   stack: red/actions/read* -1 -1 1 -1 -1
***   stack: red/interpreter/eval-arguments 020A75BCh 02159494h 02159494h 02159474h 021594DCh
***   stack: red/interpreter/eval-code 020A75BCh 02159484h 02159494h true 02159474h 021594DCh 020A75BCh
***   stack: red/interpreter/eval-path 02159474h 02159484h 02159494h false false true false
***   stack: red/interpreter/eval-expression 02159484h 02159494h false true
***   stack: red/interpreter/eval-arguments 020A742Ch 02159474h 02159494h 00000000h 00000000h
***   stack: red/interpreter/eval-code 020A742Ch 02159474h 02159494h false 00000000h 00000000h 020A742Ch
***   stack: red/interpreter/eval-expression 02159474h 02159494h false false
***   stack: red/interpreter/eval 0201D134h true
***   stack: red/natives/try* 1
***   stack: red/interpreter/eval-arguments 020A6EDCh 0208EED0h 0208EF20h 0208EEB0h 0208EF68h
***   stack: red/interpreter/eval-code 020A6EDCh 0208EEC0h 0208EF20h true 0208EEB0h 0208EF68h 020A6EDCh
***   stack: red/interpreter/eval-path 0208EEB0h 0208EEC0h 0208EF20h false false true false
***   stack: red/interpreter/eval-expression 0208EEC0h 0208EF20h false true
***   stack: red/interpreter/eval-arguments 020A6DFCh 0208EEB0h 0208EF20h 0208EE90h 0208EF34h
***   stack: red/interpreter/eval-code 020A6DFCh 0208EEA0h 0208EF20h true 0208EE90h 0208EF34h 020A6DFCh
***   stack: red/interpreter/eval-path 0208EE90h 0208EEA0h 0208EF20h false false false false
***   stack: red/interpreter/eval-expression 0208EEA0h 0208EF20h false false
***   stack: red/interpreter/eval 0201D114h true
***   stack: red/natives/unless*
***   stack: red/interpreter/eval-arguments 020A767Ch 0208ED6Ch 0208ED8Ch 00000000h 00000000h
***   stack: red/interpreter/eval-code 020A767Ch 0208ED3Ch 0208ED8Ch false 00000000h 00000000h 020A767Ch
***   stack: red/interpreter/eval-expression 0208ED3Ch 0208ED8Ch false false
***   stack: red/interpreter/eval 0201D0F4h true
***   stack: red/natives/do*
***   stack: ctx181~run 001CC160h
***   stack: ctx181~launch 001CC160h
```

```
< HTTP/1.1 200 OK
< Server: h2o
< Date: Tue, 22 Sep 2015 15:18:24 GMT
< Content-Type: application/zip
< Last-Modified: Sat, 07 Feb 2015 15:06:26 GMT
< ETag: "54d629f2-c708cc1"
< Expires: Wed, 23 Sep 2015 15:18:24 GMT
< Cache-Control: max-age=86400
< Strict-Transport-Security: max-age=63072000; includeSubdomains; preload
< X-Frame-Options: DENY
< X-Content-Type-Options: nosniff
< Accept-Ranges: bytes
< Content-Length: 208702657
< Connection: Keep-Alive
< Age: 0
```


We also need this one in WHAT'S NEW for v 0.9.0 as well as a mention of `npm update`


Fixed in 5e7bd4617bf798f6b3e3c8bf13f442751b25898f.


This one is nasty. With `transitive = true`, I get:

```
            Caused by:
            groovy.lang.GroovyRuntimeException: Conflicting module versions. Module [groovy-all is loaded in version 2.3.6 and you are trying to load version 2.0.5
                at org.codehaus.groovy.runtime.metaclass.MetaClassRegistryImpl$DefaultModuleListener.onModule(MetaClassRegistryImpl.java:509)
                at org.codehaus.groovy.runtime.m12n.ExtensionModuleScanner.scanExtensionModuleFromProperties(ExtensionModuleScanner.java:77)
                at org.codehaus.groovy.runtime.m12n.ExtensionModuleScanner.scanExtensionModuleFromMetaInf(ExtensionModuleScanner.java:71)
                at org.codehaus.groovy.runtime.m12n.ExtensionModuleScanner.scanClasspathModules(ExtensionModuleScanner.java:53)
                at org.codehaus.groovy.runtime.metaclass.MetaClassRegistryImpl.<init>(MetaClassRegistryImpl.java:110)
                at org.codehaus.groovy.runtime.metaclass.MetaClassRegistryImpl.<init>(MetaClassRegistryImpl.java:71)
                at groovy.lang.GroovySystem.<clinit>(GroovySystem.java:33)
                ... 6 more
```

I don't know why this should be transitive anyway. In the last released version (i.e. `build-info-extractor-gradle-3.1.2`) it isn't. Any ideas?


yes it is true that in the case when "all" is checked, it still hard coded


youre nasty @mollings


how much of a performance impact is this change?


totalRanges = lhs->range_size()


I'll merge and see how it works out


:shipit:


ok, passt. dann muss es so sein.


@PEMapModder I can just edit the eclipse preferences :P


I can see that this replaces the prefix technique. The counter-arguments are

1. it is possible to list the module applications directly if a prefix is used (using filters on the REST i/f)
2. users can SEE which applications are modules in a list of all applications
3. if an application is pushed which matches the name of a module being deployed, the deploy fails, but it is not obvious why
4. although the application namespace is polluted with a prefix technique (but see 3) the environment namespace is not.


sdasdasdasd


tem essa regra no exercício?


Reading this scares me


Is it more preferable to revert this change in vmpooler or fix the qatests portion?


Would it make sense for the `registerable` to include a `content_id` too? Is that the new thing that should now be sent when registering with panopticon?


`customize the installation to include Visual C++, since it is not installed by default`?


@pimpmaneaton Thanks


mind blown.


I meant to delete this loop.


I also tried this. The warning disappear, but the thing is then comes this error:

drivers/built-in.o: In function `do_dbs_timer':
msm_iommu_dev.c:(.text+0x25a408): undefined reference to `get_input_time'
make: *** [.tmp_vmlinux1] Error 1


Ignore me. I see that you've addressed this in the next commit.


I don't like the implementation of this method. It is missing a lot of the features of the original log method (you only use it passing a log level, but the original method does support more options. E.g. the [details option](https://github.com/Wirecloud/wirecloud/blob/6858b926462df91e970d3fa6691bce9a6a095a11/src/wirecloud/platform/static/js/wirecloud/wiring/OperatorTargetEndpoint.js#L103))

Also, due to Wirecloud.constants.LOGGING.*_MSG being constants, it would be a better solution to replace the constant numbers by constant strings.


test


rax retry tests


Here I go with a couple suggestions.

1. Don't forget to also change [Skeleton/class.php.twig](https://github.com/stwe/DatatablesBundle/blob/master/Resources/views/Skeleton/class.php.twig), adding to it an `'extensions' => array()` sentence, pretty much like the one in Features.php.
Skeleton/class.php.twig is the class used to generate new Datatable php files when we use that `sg:datatable:generate` command within Symfony console.

2. In Features.php, method `public function setExtensions($extensions)`, I'd `return $this;` to make it a fluent setter, like all other setters in that very same class.


@alawvt would you change your mind if it came with a fantastic poem?


Thanks for the quick response as well as the tip!


In order to keep the dependencies down, this should use

chain.ChainPolicy.RevocationMode = X509RevocationMode.NoCheck

That way it won't be trying to download CRLs or anything.
And you'd probably want to set chain.ChainPolicy.VerificationTime to some value (picking a time for today is fine) if you want to leave this on long-term; otherwise it'll mysteriously fail one day when a cert expires :)


Do we really need this?


(Copied from MainLogger.php)


I copied this from [`Flow#name`](https://github.com/alphagov/smart-answers/blob/8ba34206b24186f82ccc07d89c9e3e581894a4fb/lib/smart_answer/flow.rb#L29).

I don't like the pattern either. The problem in this class is that it mixes defining a DSL syntax with the object it creates - this method is called from the Flow classes as a setter, but then used as a getter from other places. `attr_accessible` can't be used because of that.

Refactoring seems out of scope for his PR. I could rename it `set_content_id` and add a getter, but that's also ugly (and adds another pattern to the class). What do you think?


Not needed


This is not the best choice for versioning :) In order to make solution stable we should always use strict versions. Otherwise:
- New release of a lib may cause unexpected errors that didn't happen yesterday (and this *does* happen)
- You, your colleagues and your CI may download different version of a package. This may result in differences depending on the environment.


Also, this commit is modifying the log messages.


This test should be [OuterLoop] before merging into master; it'll fail on any machine where they end up trusting the CA in that PFX... and it's not very nice to have inner-loop tests fail due to machine state.


I think that what you have here (i.e. `content_id(cid = nil)`) is OK given that it follows the existing pattern.


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1846 of 1882 tests passed)

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


Não faltou debitar a flecha?


Why color `theme-color` and `manifest` is not from MD color palette?

https://www.google.com/design/spec/style/color.html#color-color-palette


Which means we can use [`.forty_two`](http://apidock.com/rails/Array/forty_two) as an alias for `.last`...


Sry for my pool English.
Short corpseDeacayTimer for some summoning npcs. Those npcs' respawnDelay is only 25s. So their corpseDeacayTimer  is like 8.3 seconds.
For the old loot system, the corpseDecayTimer will begin at the time we loot all items from corpse(AllLootRemovedFromCorpse()).However, in the new loot system, it will begin at the time the npc is killed (PrepareBodyLootState()).
what do you think about this issue?


for test github


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


local TM form crashing...


users form seeing


lol well i did set up the script this morning before sleeping but it stopped about an hour later haha


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1347 of 1347 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)


display the contact form


by the way, about the name "failAfter"... why not "tryFor"? I think it would be more readable... tryFor(10, TimeUnit.Seconds);


rename to get service


Eric, you may want to invest a little time trying to figure out if there is a way to log in as a parent during your tests _without_ having to use the login form. I know in FreedomSoft this saves a ton of time in the tests (using the Warden `#login_as` method).


This is never used


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (rraustad) - 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)


See also https://github.com/blueyed/awesome/commit/ea0b70a7621a5bb81eeeb7d93828fafb41e0825f (from https://github.com/awesomeWM/awesome/pull/187#commitcomment-10569374).


This is a system app. That native lib gets put in /system/lib. You can't adb install it.


@tadast `stub_request(:put, x).with(y)` asserts that url `x` is requested with parameters `y`. I'll see if webmock has a better syntax for that.


Yep.  That looks like a leftover from the previous version.  I'll remove it.


Cleaned up lingering classes from "Rest Hello" example...


That function should be moved to `awful.util.is_dir` then probably.


Is there a guarantee that getApplicationInstances and getApplicationStats return instance info in the same order? Furthermore, these are asynchronous calls so the data may change between calls. We should protect ourselves against this at the very least.


Externalized the wrapPartials method


"The maximum number of registrations has been reached"


same here


Because result and options are both passed now, this method no longer breaks compatibility of adding other reporters, it can now get what it needs from options.


what about:
`from __future__ import unicode_literals`
?

On Tue, 22 Sep 2015 17:36 Oier <notifications@github.com> wrote:

> Its temporary until we figure out something to deal with unicode. For now
> windows user get an error if they try to print some strange character. We
> can handle the exception somehow and print without that characters in
> windows... Its something to discuss (or not...).
>
> we can fix with something like this in the main function:
>
> try:
> print(joke)
> except:
> print(joke.encode('ascii', 'ignore'))
>
> Not the best solution... but should work.
>
> Arratsalde on!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pyjokes/pyjokes/commit/55a3d5f2edadc27b3b8e3ce30b17e0a53f43c49d#commitcomment-13370448>
> .
>
--

Sent from my phone, sorry for brevity or typos.


Sounds strange. "You have been registered" sounds more like someone else registered you


for test github


You're right, so let's ask put it for the first plot.


I'm nervous about using URL here because I think URLs are _active_ (comparing two URLs uses a network connection to try to establish if they point to the same place). URI is probably safer.


result and options are now passed to all reporters, so they can use values placed on options.
wrapData is being passed as a property on options.


isn't "any" simpler here?


Woot
 Will try this out soon


Do you want to JavaComment these fields like the others in this class?


done


All I wanna do is
https://code.soundsoftware.ac.uk/projects/segmenter-vamp-plugin


wrapData is put on options for the reporter.
files are replaced so that it now reads files written to in the tmp directory that have been wrapped wit the partial wrapper so they can be validated as full html documents.


Still the issue with not finding the Line Ahead formation you mentioned in #11? Do the sorties always fail or does it sometimes go through properly?


This method below is now externalized. See comment above.


I see that this fix has not been included in weld 2.3.Final or am I missing something here?


**no-undef**: "it" is not defined.


**valid-jsdoc**: Missing JSDoc @returns for function.


**no-undef**: "ReactDOM" is not defined.


**no-undef**: "expect" is not defined.


**no-unused-vars**: component is defined but never used


**no-undef**: "describe" is not defined.


**react/jsx-sort-props**: Props should be sorted alphabetically


**no-undef**: "TestUtils" is not defined.


**no-unused-vars**: domNode is defined but never used


**no-unused-vars**: ReactDOM is defined but never used


**no-undef**: "before" is not defined.


**semi**: Missing semicolon.


**no-undef**: "sinon" is not defined.


**no-unused-vars**: onChangeSpy is defined but never used


Cleanup the tmp directory that the wrapped partials were written to. Only if they were written.


I think changing time zone on a field which is almost certainly a _duration_ is a mistake.


maybe change the name to "maybe_extract_keys_from_opengpg_header" ?
cause it attempt to extract keys from the header.


An interesting use of the interpolate method, but in the end, I think this is not what one expect when calling a log method. This way, developers doesn't have control for adding other interpolation patterns or renaming some of them for adding more info into the message. Finally, I'm going to drop this commit except the messages changes that are going to be merged into the correct commit.


Where is that it needed to be converted anyway?


Validates that the html content and partial were merged correctly.


Looks like we need to pass `app_port` in here. I've added it and performing tests right now.


Well this made after some considerations:
1. Commons has own tests to be sure, that it hasn't regression errors.
2. I test commons with mocha reporter and [this pack of tests](https://github.com/allure-examples/mocha-allure-example) before publish
3. I made a lot of different fixes regarding runtime code. All of them is backward-compatible and I don't want to release mocha reporter after every fix
4. Now compatible versions declared as `1.1.0-1.1.9`. If here will be incompatible change I would release `1.2.0` and users will remain unaffected

@ctapobep Maybe I should add my tests with mocha reporter and example as a public pre-publish hook? This is a restriction from travis, that I can't run tests on SauceLabs from pull-requests


Validates that the data modified by the partials.js is done correctly for the reporter.


Also, summon @baev @vania-pooh as a release specialist


The problem & solution:
http://blog.csdn.net/wdjhzw/article/details/39807609


あいうえお消す必要ありましたか？？


@krishicks


**Challenge faced:** `margin: auto` didn't work to put the logo in the center of the page.
**Solution:** Use `display: block`
**Explanation:** The default setting for an image is `display:inline` which basically has no margins. Hence, using `margin: auto` did not work. Adding `display: block` made the image take up the whole page width and hence `margin: auto` able to divide the margins equally to put image in the center.


s/return/continue/


I checked my version of the scrip and it only adds a single space in the  /etc/pihole/gravity.list file


これ使ってないですよね？
必要ないのは削除しましょう :+1:


@colinPL @joshcooper @sschneid It is better option to fix the client utility which is already tracked in the QA-2050 ticket. Also, the "vmpooler.py" script has moved from "qatests" to "qatools" and been renamed to "vmpooler_client.py".


I think you never tested this ;)

Postgres has a maximum identifier length of 63 chars, so the name of the FK is being mangled at the end.
The solution for this is to provide a name manually (both in here and in the model) instead of using an autogenerated one

`fk_regform_items_current_data_id_regform_field_data` would be an option. It's still pretty clear but short enough to work without problems.


了解しました。まーじします。


Yes. Tried it previously and it worked just find for me.
When I'm using Version 0.1.4 of the plugin the symlinks are generated correctly.


Good use of e`x`tended syntax :stars:


Misleading text as payment settings are global for the event.


Is this needed? What if `HiveWritableTableHandle` implements  `ConnectorInsertTableHandle` directly?


thanks!
does IE11 needs any prefixes?


Only remove the plot.restore if you don't want the restoring. Otherwise your scoreboard doesn't go away and your inventory isn't reset.


if a different response needs to be sent for any of these particular exceptions, then it's impossible to override the default exception mapper behavior, nor can they be removed from the jersey server configuration. specific use case was the AeroFS API needing to send back a "type" field in the response for an IllegalArgumentException


I think this is in regard to the IJ workflow. If so, this is superfluour, as the measurement table interactions from IJ are mentioned below.


This looks odd


We consulted a Material Design designer regarding this. The web context for `theme-color` is different to what is currently found in the spec relating to the native equivalent and required a different color.


:disappointed:

Why not `user+{whatever}@example.com`?


why bother with the default? this is nullable.


Ah, that syntax is so confusing. After reading [the docs](https://github.com/bblimke/webmock#stubbing) I see how it works. I though it stubs the request and returns the hash in `#with` as json :)

:+1:


Please, help to write it correct


Does this actually work? `=True` is kind of dangerous with hybrid properties since the generated SQL will be `foo = True` with `foo` being the expression of the property. Since it's `foo` and not `(foo)` it depends on the statement whether it will work or not. Generally it's better to avoid using filter_by-style kwargs with hybrids and use `find_all(RegistrationForm.is_visible, RegistrationForm.event_id=...)` instead.

See https://bitbucket.org/zzzeek/sqlalchemy/issues/3308/cannot-use-hybrid_property-in-filter_by


need trailing line


@IIvec
No need to be sorry...I just misread the code.
I understand now.


@AmeenAli ah now I see :-)


Please, split class rewrites and modifications


Forgot to update the package version, 3.1 -> 3.2.


I might like to see these (if they're all in code you added in this PR) rebased into their respective originating commits, but I'm not *too* fussed.


Add the improved UI experience when Moving annotated data between groups in Web. See https://github.com/openmicroscopy/openmicroscopy/pull/4165 (but I would stick to ``Move data between groups`` formulation ``chrgrp is too developery


This commit is supposed to include changes of the following commit.


test 2


This will break `setLayoutProperty`.


I've looked at https://github.com/alphagov/smart-answers/blame/send-content-id/test/unit/smartdown_adapter/presenter_test.rb#L8 for this.

This seems to be the way:

```ruby
should "use the flow content_id" do
  SmartdownAdapter::Registry.reset_instance
  flow_registry_options = {
    show_drafts: true,
    preload_flows: true,
    smartdown_load_path: Rails.root.join('test', 'fixtures', 'smartdown_flows')
  }
  flow = SmartdownAdapter::Registry.instance(flow_registry_options).find("animal-example-simple")

  presenter = SmartdownAdapter::FlowRegistrationPresenter.new(flow)

  assert_equal "5bb6964b-3147-4423-aab3-b87e1cb8b838", presenter.content_id

  SmartdownAdapter::Registry.reset_instance
end
```

All the setup makes me sad. Am I missing something?


Heh. I had that locally. Pushed it up and forgot to merge it.
On Sep 22, 2015 9:08 AM, "Eron Wright" <notifications@github.com> wrote:

> Forgot to update the package version, 3.1 -> 3.2.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/deeplearning4j/dl4j-spark-ml/commit/e0f6c2447f078488d4cd07c1e2f467e47f5bba33#commitcomment-13371445>
> .
>


... dam forgot the setters. :/


I do not agree on this at all. Either do not hardcode the uri or add a way for giving the uris. I do not like commandline but if you want to remove these boxes provide the functinality in command line. So it is -1 from me.


Maybe, we should set default value to `0`, otherwise we may get TypeError when doing `port=int(port)`.


No need to wrap


Static import


Yea gotcha. But someday, Angular 2.1 will be out, do you want to remove this resource then?


`requireNonNull`


commit message not aligned with the patch.


This check is duplicated below


Nit: full stop


This is also a surprise: why not say unknown here?


i think using the nullability of this field would be cleaner here


The `FLAPPING` state is a surprise.  Where does it come from? We've never seen this from the REST i/f. I see it is hardcoded in the enum in the java client.


s/%d/index %d/


Doesn't work
samsung galaxy s5 android 5
logcat https://gist.github.com/Rubenoo/f80b0db029014c439f21


Does this need to be public?  And is it really a Builder?


it's following what is listed in TodoMVC so we should probably open an issue in their repo about it


well done


I didn't know, probably in cartodb we are using the torque reference for both mapnik and torque cc @fdansv


Let's call this `node`, so we don't have the redundant `action->action`.


or just timeout?


Wouldn't recompiling the library with bitcode be the right fix, then?

That said, bitcode is a black box that may introduce unwanted optimizations.


Not sure I like the name here. It might be an Undoer. :-)  Incidentally, "rollback" implies some sort of database transaction, which this is clearly not.


Looks like the sections about myself and my session are mixed up...


Hardcoded values.....


well sometimes it does go sometimes it doesnt


Thanks.


I feel that stubbing those would couple the tests too much to the implementation. Ideally we wouldn't have to change the test when the smartdown thing is deprecated.

Since this had no tests before, maybe just `assert registerables.any?`


Attached is a *ZIP* containing new binary of config tool, and a sample of new format of config(with comment on new item).
Can someone replace the old files in packges with these new ones?

![new config binary and config file with comment](https://cloud.githubusercontent.com/assets/6811759/10024080/6f4df326-6190-11e5-9bd3-db0a36f12f1a.JPG)


I did not understand this as a duration, but as the (UTC) time of start.


This is a clever class helper. The call sequence could be made a little easier to read, however, by making the action and the undo action more obvious and more succinct. Is there a way you could refactor this?


Absolutely, will do


I think only @KevinKelley can make such a branch, and then someone else can send a PR to it?


yea let's do that.


Let's go with `type` instead of `title or app_id`, and `action` instead of `arguments`.


@fritsch @FernetMenta was this done by intention?


In the page add the correct corresponding class and ID. EXAMPLE: Learn More


oh, I will undo that


Fix pushed, but untested.


The blocking is a bug in the darwin tty driver.  The kernel driver's close() blocks while there is data pending for read.  The solution is to flush that data just prior to the close.  Probably close() should also perform the same flushing action -- which is what happens on all other platforms.

It might be possible to attempt to flush *after* the close, which would avoid the race of character arriving prior to the close but after the flush.


POXA ;(


Looks like there is a typo here, it's missing a closing single quote. Should it be `command => "bash -c 'export PATH=${path}; ${mcollective::site_libdir}/refresh-mcollective-metadata >/dev/null 2>&1'"`?

This currently causes cron to output:
```
/bin/sh: -c: line 0: unexpected EOF while looking for matching `''
/bin/sh: -c: line 1: syntax error: unexpected end of file
```


I'd go with `*cannot*` here. Reads slightly better.


`if the issue...`


and delete this pls :P


Aqui não tem espaço para tentativas.
ehuaehuaehaeaef


Please indent by 8ch. See CODING_STYLE.


Thought about that. We're not comparing URL here, merely creating them (through Converters). The Client API will need a URL anyway


Yeah, true... I'll see what I can do to fix it while keeping the card-castle of making automatic pull requests via Google Calendar events work :)


Otherwise looks great.


tableView头部拉伸效果 + auto Layout


You should see "Queuing schedule for every_1m" every minute even if none of your agents are run at the time.
That is how my log is looking:
```
18:14:56 jobs.1 | Queuing schedule for every_1m
18:14:56 jobs.1 | ## Set TWITTER_OAUTH_KEY and TWITTER_OAUTH_SECRET in your environment to use Twitter Agents.
18:14:56 jobs.1 | Running event cleanup
18:14:56 jobs.1 | Queuing event propagation
18:15:56 jobs.1 | Queuing schedule for every_1m
18:15:56 jobs.1 | ## Set TWITTER_OAUTH_KEY and TWITTER_OAUTH_SECRET in your environment to use Twitter Agents.
18:15:56 jobs.1 | Running event cleanup
18:15:56 jobs.1 | Queuing event propagation
18:16:53 jobs.1 | Queuing schedule for every_2m
```


Sure, go ahead, I thought it was set to your master2 branch!


Checked.


Addressed with https://github.com/alphagov/smart-answers/commit/d0b3840bb217b79229b7c77af27c8eda73f23cbc


ok I did


Imagine AE would get another internal float planar format which we don't specifically handle here - then we would return AV_SAMPLE_FMT_FLT - that would sound quite evil :-)

It's not nice - but it might help someone that implements the next AE planar type we don't know of yet.


Do we need to get the translation for this filter.heading?


Não respeitou a lei de Demeter.


If you use `ngAnimateMock` then it exposes `$animate.queue` as well. This isn't essential here, but it's just a FYI.


In general I would like to see `this.` prefixes on all internal references to instance properties and methods. It makes it easier to read.

I think there are question marks over the stability and reliability of the cf-java-client code that would mean we would be involved in "maintenance" of this code. We use a lot of the classes from there; and they are subject to change in the near (if not, forseeable) future.

I would like to see some unit tests. The stubbing of the cf-java-client in tests would be easier if the encapsulation of module data vs cf application data was a little more clear, and if the module deployer used a relatively few calls to another layer which dealt purely with could-foundry applications. At the moment, the cf-java-client is too low-level and too broad an interface to make testing simple.


Ah, damn.


Ah, but this only works with the tty in raw mode.


another commit went after :p
https://github.com/Tesla-Redux/platform_manifest/commit/f0bc3d8bc232e010f2ea7e7d4a4e338b0eb86803


Is this line for testing?


This means that the window never set it's `app_id`. I think the same thing happens for `title`.

How about this: for each case, we just do `identifier = window->swc->whatever;`, and for the `default` case, we do `identifier = NULL;`. Then, outside of the case, we can just say

```c
if (!identifier)
	continue;
```


You've probably got these X's covered?...


LGTM


True because it isn't defined. I guess I would have the error as well. The warning appears because of the type of get_input_time. But I don't believe you should not have any error while compiling right?

The function should be re-formatted for use with GCC 5 because there shouldn't be any issues when using GCC 4.x.


Verdade, esqueci :cactus:


:cry:

No problem - i'll change this


Complete disagreement at this one (unacceptable)


Prefix this with the connector ID rather than "hive"


Nit: This line might be better worded as:
```patch
! This factory *does not* accept certificates issued by many common certificate authorities, such as StartCom
```


Since we don't refer to the `enum rule_type` type anywhere, we can just inline that.

```c
enum {
	RULE_TYPE_WINDOW_TITLE,
	RULE_TYPE_APP_ID,
} type;
```


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (808 of 808 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-21.59%25-green.svg)


I think this will mean that we can get rid of the [fields mappings from the `gds_zendesk` gem](https://github.com/alphagov/gds_zendesk/blob/master/lib/gds_zendesk/field_mappings.rb).


Tested — speed same as with `Object.create()`, which is good.

Btw, `_.clone()` produces same speed. I guess, it makes something similar to https://github.com/vitkarpov/grunt-nunjucks-2-html/commit/49fcaf76fed59d0dfb829d2b59e70372ec2b5f72#diff-ecf6f36c6e253178e1f6195756409653R41 under hood?


moved


AFAICT, zh_TW and zh_HK have the same handling, why not merge those two cases? E.g.,:
zh_TW*|zh_HK*) ...


Let's order these fields `type`, `identifier`, `action`, because it matches the order given in the config file, and `identifier` doesn't have any meaning until you know what `type` is.


Thanks for the comments. I already fixed most of the issues so I am not closing the request.


hmm, copy-paste error? :)


After running some tests, I noticed that after re-rooting, TreeBest was inserting some zero branch lengths.
That on its own is not a problem, but it causes Ktreedist to crash when computing the BLs.
I'll try with a different format_mode and see if TreeBest keeps including the zero BLs.


You shouldn't hardcode wine here, but rather use the $WINE variable to match what the user used to run winetricks.


fixes #29


**Eslint**: 'return' outside of function


**require-jsdoc/require-jsdoc**: Missing @jsdoc for autocompleteFor


**require-jsdoc/require-jsdoc**: Missing @jsdoc for componentWillReceiveProps


Yes, I believe you're right.


Great info to print out!


Well done Antonino!


Is a non-svg icon needed here? Could be a nice future enhancement rather than a blocker.


@Flow86 Oh sorry. Dachte das hatte ich. Das "erzeugen" der build_version.h über cmake ist mehr dafür, dass die im build-ordner landet und über die includes gefunden wird. Kann man sicher auch anders lösen, aber fand das ganz sauber so (ein copy und rename würde in dem fall reichen, aber was solls)


Might be quiet tricky to do as they were added over quiet a few commits.


When viewing thetas plots with shiny graphing, webpage showed the following error:

Expected sep ('	') but new line, EOF (or other non printing character) ends field 1 on line 31 when detecting types: -------------


@rogersb11 This part is needed with the Settings commit:
https://github.com/MrApocalypse/packages_apps_Settings/commit/6463343f2f70232dfabef33844f9384959ee531b

@pimpmaneaton fixed it


shouldn't that be `cp ./build_version_defines.h.cmake ./build/build_version_defines.h` ?


it was using the max_samples attribute from the object if this answers the question


yep, same thing here


sorry yes,  will clean up (actually file save as)


For 1 GB devices this is too much, lmk thrashing causes visible UI pauses. Please reduce to 128mb for 1gb devices.


If you say it works, then it is fine with me. I'll do an XFCE build later today.


Can someone please review https://github.com/puppet-community/puppet-mcollective/pull/237? Thanks


Done in https://github.com/alphagov/smart-answers/commit/a13d89635dc4fcea54c09d5becb383def5c24e26


takes


Another minor fixup.
Instead of having another implementation of the of the comparison for != operator, it can be implemented using the == operator.
```
{
return !(*this == c2);
}
```


That was, and it was eliminated in my last PR, fixing a bug.


This last statement does not actually have any effect. It will set `rule` to `NULL`, only for the remainder of this function.


:+1: Nice work, good job! :smiley:


Normally it should been done in all or our profiles. This one I missed. It was a bug regarding loading other desktops as the pre-installed one.


I have the errors in both cases, but the warning only shows up when using external.

The thing is that there is some file related to this that cause the error. I'm compiling hellscore for nexus 4 (same touchboost.c and touchboost.h as yours) and have the error, but using the same toolchain and the sources of taurusPrime (in my github, which have same touchboost.c and touchboost.h too) there isn't any problem.

I'm not sure what can causes the error then, but maybe you will be also able to compile without any error.


- [ ] Please add a link to your github account inside readme.txt


I hesitated to remove this. I just feel uneasy about the fact all the other tests just silently pass if zips is not there and the loading of the datasets in fairly new code even though it was a few tests of it's own. I can do wtv you think is best.


Merged, thanks!


I downloaded WinRABCDasm, can u explain more about how to add trace to iqiyi code ? Where is Iqiyi's encode function? How to update the code in WinRARCDasm and reassemble that code.


+1, we might want to change the format in the future.


Yeah, it was more of a “in future, maybe see if you can `fixup` them” type thing than a “this is a blocker for merging”.


You can just use `free(rule->identifier)` here. I think we won't need a `rule_destroy` function, unless we decide to implement config reloading in the future.

I think we probably don't need separate `rule.c` and `rule.h` files until we have a larger amount of stuff to put in them.

For now, just moving the definition of `struct rule` to `velox.h` should be fine.


Great stuff.


This signal should be emitted. Just don't connect the app on it if you don't need to.


i jusst added a Detail Activity with the Coolapsing tool bar


You are wrong, I tested it and I didn't get any errors.


Средство от мусора в консоли:
```html
<md-list-item ... aria-label="{{channel.title}}" ... >
```
В днях таким же образом можно поступить


why do we allow string to be null ?


Uh, no.  I've never used these methods in my life lol.


maybe use text, instead of string as variable name.


@heejaechang you probably need to sync closed as well? I made a corresponding change to closed. PR 172 in internal repo


Так нужно было


Hrm, weird. Can you check in pgAdmin what name the FK has?


This block isn't quite correct. SWS uses CPU auto-sensing just like most of everything else in FFmpeg, so this whole function shouldn't be needed for anything anymore.


TeamCity Colore :: Main Build [Build 100](http://tc.sharpblade.net/viewLog.html?buildId=635&buildTypeId=colore_mainbuild) is now running


The `exec()` call should be outside of the if statement.  You still want it to exec when you come in here, but you only want to build and connect it to everything once.


:+1: :top:


@aShevc we should attempt to  display the result.errors even if its not an Array, right?


const


TeamCity Colore :: Main Build [Build 100](http://tc.sharpblade.net/viewLog.html?buildId=635&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:5


Actually, more experimentation reveals an even better solution.  We need to change the value of VMIN and VTIME.  VMIN should be 0, and VTIME should be some small value -- 5 works well as this means 1/2 second.  Then the read loop will return 0 bytes (and EOF) every 1/2 second, but it will return immediately if a character is available.  This means that the "hanging close" will stall for 1/2 second, but I think that's unavoidable on darwin.  I still believe the problem here is a bug in the tty driver's close which should just flush all the data, and wake up the reader.


:clap:


... lol...  @helfrij, how bad is your refactor OCD starting to kick in with this fileloader?


Hi KreAch3R,
when I press the update button the following happen in logcat when I have your commit i
```
W/nfire.opendelta( 4594): type=1400 audit(0.0:415): avc: denied { search } for name="recovery" dev="mmcblk0p16" ino=24577 scontext=u:r:platform_app:s0 tcontext=u:object_r:recovery_cache_file:s0 tclass=dir permissive=0
W/nfire.opendelta( 4594): type=1300 audit(0.0:415): arch=40000028 syscall=322 per=800000 success=no exit=-13 a0=ffffff9c a1=b4bfb088 a2=20241 a3=180 items=0 ppid=271 auid=4294967295 uid=10013 gid=10013 euid=10013 suid=10013 fsuid=10013 egid=10013 sgid=10013 fsgid=10013 tty=(none) ses=4294967295 exe="/system/bin/app_process32" subj=u:r:platform_app:s0 key=(null)
W/auditd  (  312): type=1320 audit(0.0:415):
```
Argh, hate this selinux thing :( Hope you can help me.


There is no reason to drop "and"


This commit does not make sense, specially taking into account  b6dafc3ec9dfea49a4143985ea3e6feeaebadb11.


It seems that envvars does not quite fit into java/maven ecosystem. Can we move to system properties and deprecate envvars to simplify this?


Ok I see. Nice catch. :+1:  I came into this line when I inspected the current online documents and then traced down to your changes. Did not notice that it was fixed. Thanks for your quick response!


Do you know where `err.code` comes from?


@Rubenoo thanks for such rapid testing, I've committed before testing because I'm on a separate branch and I'm still working on it.
thanks for the report :wink:


Coolieo


@snimavat you have a class level variable named page too. can you rename this. Thanks


Is there something wrong with this? It's in the documentation, I swear I didn't make them up.


Hier kannst du wohl die Funktion `mix()` verwenden:
https://github.com/raoulbq/libwaveblocks/blob/master/waveblocks/hawp_paramset.hpp#L83


Is the CWL YAML file missing from this commit?


Players cannott be found that match these Conditions


Se podría cambiar a ` testCompile "org.robolectric:robolectric:3.1-SNAPSHOT"`


should this really be done before the if() below, and not in updateSizes()?


@mvinan revísame


Do this mean that it is now possible to draw curves in automations, not just break-point functions?


Yeah np,  csploit keeps telling me that it failed to connect/start to msf rpcd.

And why won't you stop by the irc, that makes it easier to contact you. I an on the rest of the night if you need me


tested and working


**Eslint**: 'return' outside of function


**require-jsdoc/require-jsdoc**: Missing @jsdoc for componentWillReceiveProps


**require-jsdoc/require-jsdoc**: Missing @jsdoc for autocompleteFor


Yes, you are right. Fixing now.


Can you give me the IRC channel ?
I didn't known we had one :P


I just tested it and you are right. I will change it accordingly.


pimpl for public class?


Se debería poder mockear la respuesta pasandole el path a un archivo, cosa de no hardcodear toda la respuesta. Chequear como se hace en el FileSynchronizationTest


Thanks :)


The issue is that they are provided more for STL compatibility.  You should use the more Qt-standard methods for accessing, adding, and removing from lists.  For instance, push and pop are unnecessarily stack-focused.  You should use add(), insert(), append(), prepend(), removeOne(), removeAll(), takeAt() methods.  They are the methods that are used more commonly and also have very explicit contextual meaning.


freenode.net, #csploit
Le 22 sept. 2015 18:49, "tux-mind" <notifications@github.com> a écrit :

> Can you give me the IRC channel ?
> I didn't known we had one :P
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cSploit/android/commit/81ce257387533032b9e68663db0473746e2cdabc#commitcomment-13372390>
> .
>


good thinking @pescobar :+1:


Vitor, eu tenho feito assim, só não fiz o merge desta última tarefa
propositalmente, pois o problema está resolvido mas a solução não está
"componentizada".

Em 22 de setembro de 2015 10:42, Victor Azevedo <notifications@github.com>
escreveu:

> Raul,
>
> Você tem feito o push dos seus branchs para o servidor e arece que não tem
> feito merge. A maneira correta é:
> 1) Criar o branch da tarefa no repositório local com o nome da tarefa no
> Jira
> 2) Quando terminar a tarefa, fazer o merge com o master no repositório
> local
> 3) Fazer o push do master e não fazer o push do branch da tarefa.
>
> Por favor, acerte isso no repositório.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Plataformas-Cidadania/mapaosc/commit/48aa86a9eb70ec2bc128ee6f8167bdacfb1879cb#commitcomment-13367395>
> .
>

--
Mestrando em Engenharia de Sistemas e Computação UFRJ
Bacharel em Ciência da Computação UFRRJ
Linkedin <http://www.linkedin.com/in/raulsenaferreira>


Me, I'm not yet real clear on everything about branches, but need to learn.  I've just created an 'unstable' branch on the project, feel free to send PR, and let me know if there's anything I can or need to do


@deads2k mind having another look? Working on my own custom conversion though I'm still not sure this is the right way


Oops. Pushed now.


Разбираюсь как правильно реализовать вывод данных из БД на main.jsp странницу при помощи jsp:useBean.

На данный момент:
1. Реализована страница index.jsp и main.jsp.
2. Осуществляется вывод данных из БД в консоль IDE, при запуске Main.java из package com.busdmv.backend.db;

Не реализовано:
1. Вывод данных БД на jsp странницу, выдает ошибку.(код вывода из проекта удалил, иначе не отображается main.jsp)


hehe maaaan...nice commit though


The problem is now it always display the same icon.


tested and working


I don't think you're missing anything, unfortunately.


http://webchat.freenode.net/?channels=csploit , alot of other developers are on it


This newline strip should have been in the `spawn_action` function. I just fixed that in `master` and `variant`.


TeamCity Colore :: Main Build [Build 101](http://tc.sharpblade.net/viewLog.html?buildId=636&buildTypeId=colore_mainbuild) is now running


Change this from `Name of Project` to `Recent Posts`


The commit must have worked, because the first line depicts eu.chainfire.opendelta as a platform_app, if I read that correctly. But I don't understand why that was denied. I have also never seen the second line. When I was troubleshooting, I was live patching the sepolicy with SuperSU's supolicy --live command. Try to pass the correct selinux arguments there, and maybe you find out where it stops.
I could also think that you are missing other commits in sepolicy that enables OpenDelta to work in the first place (before this commit). But this is just speculating, I don't know what you might need.

I'm sorry I can't help you any further with this, but given that this is not even on our source, I couldn't be any much help from the start. I hope you the best.


But there was indeed a problem


TeamCity Colore :: Main Build [Build 101](http://tc.sharpblade.net/viewLog.html?buildId=636&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:17


Ok


Listo, subido


Our custom errors like `ValidationError`


test onsite


I didn't know any, it's great :)


Mas não precisa subir o branch para o repositório remoto. Tem alguns
branchs no remote que não deveriam estar lá.

*Victor Azevedo*
vhmeirelles@gmail.com
+55 21 8101-1441

2015-09-22 13:50 GMT-03:00 Raul Sena Ferreira <notifications@github.com>:

> Vitor, eu tenho feito assim, só não fiz o merge desta última tarefa
> propositalmente, pois o problema está resolvido mas a solução não está
> "componentizada".
>
> Em 22 de setembro de 2015 10:42, Victor Azevedo <notifications@github.com>
> escreveu:
>
> > Raul,
> >
> > Você tem feito o push dos seus branchs para o servidor e arece que não
> tem
> > feito merge. A maneira correta é:
> > 1) Criar o branch da tarefa no repositório local com o nome da tarefa no
> > Jira
> > 2) Quando terminar a tarefa, fazer o merge com o master no repositório
> > local
> > 3) Fazer o push do master e não fazer o push do branch da tarefa.
> >
> > Por favor, acerte isso no repositório.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/Plataformas-Cidadania/mapaosc/commit/48aa86a9eb70ec2bc128ee6f8167bdacfb1879cb#commitcomment-13367395
> >
> > .
> >
>
>
>
> --
> Mestrando em Engenharia de Sistemas e Computação UFRJ
> Bacharel em Ciência da Computação UFRRJ
> Linkedin <http://www.linkedin.com/in/raulsenaferreira>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Plataformas-Cidadania/mapaosc/commit/48aa86a9eb70ec2bc128ee6f8167bdacfb1879cb#commitcomment-13372417>
> .
>


Thank you very much, this was more help than I expected to get.


Sorry, but this change still cause BbileTime to crush, i ll move assert under BT_MOBILE macro.


thanks @KenthMobiTV . silly mistake.


Para acceder a la documentación por favor consultar el wiki.


tested and working


101682488_EvapCoolPrimaryAirSizing (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1857 of 1887 tests passed)

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


Cheerio@0.4.0 was the last time the $(...).html() behaved the same as the main documentation (ref. the third example in the Selectors section).

The following examples are based from the main documentation:

```Javascript
var cheerio = require("cheerio");

var markup = `
<!DOCTYPE html>
<html>
  <body>
    <ul id="fruits">
      <li class="apple">Apple</li>
      <li class="orange">Orange</li>
      <li class="pear">Pear</li>
      <li class="orange">Valencia</li>
    </ul>
  </body>
</html>
`;

var $ = cheerio.load(markup);

var output1 = $('.apple', '#fruits').text();
console.log("OUTPUT 1:\n", output1);
//=> Apple

var output2 = $('ul .pear').attr('class');
console.log("OUTPUT 2:\n", output2);
//=> pear

var output3 = $('li[class=orange]').html();
console.log("OUTPUT 3:\n", output3);
//=> <li class="orange">Orange</li>
// WARNING: The actual output is just => Orange

var output4 = $('li[class=orange]');
console.log("OUTPUT 4:\n", $.html(output4));
//=> <li class="orange">Orange</li><li class="orange">Valencia</li>

var output5 = $('li[class=orange]');
console.log("OUTPUT 5:\n", output5.toString());
//=> <li class="orange">Orange</li><li class="orange">Valencia</li>
```

Tested with Node v4.1.0 x64, the program output looks like this:

```
OUTPUT 1:
 Apple
OUTPUT 2:
 pear
OUTPUT 3:
 Orange
OUTPUT 4:
 <li class="orange">Orange</li><li class="orange">Valencia</li>
OUTPUT 5:
 <li class="orange">Orange</li><li class="orange">Valencia</li>
```

The problem is exports.html(). It isn't clear why parsing and returning only the first matched item's children is desirable (refer to the diff). $("...").children().first().html() should behave the same, according to documentation, if this behavior is desired.

The patch attempts to revert the behavior semantically back to Cheerio@0.4.0. With the patched changes the program output looks as expected from this:

```
OUTPUT 1:
 Apple
OUTPUT 2:
 pear
OUTPUT 3:
 <li class="orange">Orange</li><li class="orange">Valencia</li>
OUTPUT 4:
 <li class="orange">Orange</li><li class="orange">Valencia</li>
OUTPUT 5:
 <li class="orange">Orange</li><li class="orange">Valencia</li>
```


Argh, I was under the assumption that we renamed them some time ago. Sorry :-/.


:-1:
maintainers: should match openstack-zaqar


My bad, I didn't realise it would throw a Reference Error.


The check on line 16 doesn't make sense anymore with this. As it it here, it check that `this` is an object. Which will always be the case.

 We might as well not pass the context in and keep the check on line 16, which will then be testing for the global object window, and not the local variable named `window`.


**Eslint**: 'return' outside of function


**require-jsdoc/require-jsdoc**: Missing @jsdoc for componentWillReceiveProps


**require-jsdoc/require-jsdoc**: Missing @jsdoc for autocompleteFor


There seem to be spaces missing - do you have the formatter enabled?


These lines would not be needed anymore after applying my other comment.


Wow, this is a good one! Neat @gaearon!


I'm using ifdev __IPHONE_8_0 in some other places.   Will those fail also, then?  Are we actually using Xcode 5 on the build servers for all builds?

(I don't have a working Xcode 5 on my local machine anymore.... when I try to open up the old versions it fails and keeps asking to install extra things that will most likely break my new Xcodes).


This will make it so that all the internal posts appear in the email. It should only include the posts that have been created since the last email. Also, it's okay to include all posts, not just internal posts.


@diox, any suggestions? my best idea right now is to go through the dict and replace any lists with their first element...


Please launch a new update in manjaro with this update of mhwd-db.


Actually, we coud still pass `this` in and check both that the global variable `window` exists, and that it is `===` to the context passed in. But that might be overkill.


Sometimes not working.
Need more test.


How about testing that the number of `registerables` matches the number of `SmartdownAdapter::Registry.instance.flows + SmartAnswer::FlowRegistry.new(FLOW_REGISTRY_OPTIONS).flows` ? That would at least give us some idea as to why we're expecting to get a certain number of `registerables` back.


yeaaaah see, this is the kind of stuff that makes me say we should remove beautifulsoup entirely :)


tested and working


A bit hacky.... this CSS class should be included in the PreferencesWindowMenu class code.

Also component-update-form is better written as wc-component-preferences-dialog to follow the CSS naming rules.


LOL cuz u need coffeh ?


Sure. I have no problem with that. Let me know what you prefer and I will make the changes for that.