sample output - 20150402


doubled line


Just to note myself for possible later time – this was four wards in Braintree all starting "Witham"; I manually worked out which boundary was which from looking at some random points and manually used those names on import.


This file shouldn't be in the PR.


Much in favor of splitting out things like this into traits


Ready


Tree.js already has a definition for _setLabelAttr().   You should replace the old definition, not add another one.


La description est un peu non standard, mais je suis d'accord avec vous probablement plus claire. Elle sera intégrée dans le prochain build. Merci


送信先と同じと仮定するか、URL を返す API を用意するかかなー


Yeah, you're right. It's already on master, so next time.
And what is your opinion about my comment on the fact that maybe we should reflect this change to all methods for more consistency?


Looks like this is breaking the build:

```
libavcodec/vda_h264.c:347:9: error: use of undeclared identifier 'h'
    if (h->is_avc == 1)
        ^
1 error generated.


Maybe we can consider passing an options object ? { preventUpdate: true }
I would need an eventSouce to pass to the event callback creator


I think we should mention `match`, not `let+` here, since that's the primary use case.


ping @Warkot @rogatty


yes I think it would be great


Ok, so let's go with the options object then. Just warn me about to don't break my brand new code :sparkles:


Indentation is wrong on lines 20, 23, 27, 30, 33.  Actually, all the indentation is strange; it should look like:

```js
items: [
   {
       ...
   },
   {
     ...
   }
]
```


By now, Runner raises RuntimeError on non-zero exit. But we should change this, yes. For this PR, I have to deal with it, I guess.


Bad merge


So, see above ;)


ping @Warkot @dianafa


indentation on 10-11 wrong


r+


in the python-script this becomes part of a filename, so can not easily be fuzzy.


change comment


You shouldn't be loading something from ajax.googleapis.com.   Also you don't need a slash to close a link tag.   This is HTML not XML.


That's a typo


哦哦！


This script block seems useless.


LGTM :shipit:


Use static for internal functions.
Also use _U gpointer data if the argument is unused.


I'm getting
```
 PHP Fatal error:  require(): Failed opening required '' (include_path='.:/usr/share/php:/usr/share/pear') in /var/www/html/mautic-api/apitester/index.php on line 5
```
on this line. The `realpath` function returns `false` in my case. When I looked the documentation:
> The running script must have executable permissions on all directories in the hierarchy, otherwise realpath() will return FALSE.
> - http://php.net/manual/en/function.realpath.php

Is it my fault that I have wrong permissions?


Also, to be transparent here, I'm still disgruntled by statements like this one:

>He's just mad that I called him names and disregarded his argument that doing something broken is ok because ASS subtitles need it. (Though I didn't change how ASS colors are handled. Hell, I've even implemented the vsfilter bt.601 hack myself personally, so shitty weeaboos can have their shit.)

I don't think this is a fair assessment of the matter, and I also think we're more than capable of discussing these things objectively in the future, rather than resorting to calling each others' approaches “broken”. If anything, we have a drastically different viewpoints on these things, and it is this viewpoint that lets the other approach seem broken from our perspective.

As for me, it's nothing new that my viewpoint for these kinds of things is very solidly based on a foundation of color management, which is not something that has to be a given (in truth, you can always of course just buy a display that has the correct response to begin with), but it's a given for me in particular. So personally, I generally care a great deal about the use cases of people watching video on wide gamut monitors, for example, since I personally use one. Most of what I develop is motivated by personally needing it, after all.

Sometimes it doesn't always coincide with what the author wishes to do - and in those situations, I end up having to maintain my own fork that does color management in a way compatible with my environment (eg. https://github.com/haasn/sxiv).

I'm more than comfortable with being forced to keep things that work the way I expect them to in my own personal fork, and I don't wish to claim to be in the “right” about everything, either. The question, I think, is how much we are willing to settle on a possible common viewpoint when it comes to **mpv**, which as much a question of the differences in our viewpoints as it is a question about the needs of mpv's users.


For 2.6


I see that Tree_node_rename.html is a test file, but this needs an automated test.   Look to the other files for examples.


docstring, e.g.

"""Thrown if file is not under git-annex (but might be under Git) control"""


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


May be even wipe out Annex prefix?


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


r+


Les SPI c'est des branleurs.

2015-04-02 10:41 GMT+02:00 Antoine Foucault <notifications@github.com>:

> @Strauss72 <https://github.com/Strauss72> Je veux bien oui :)
> Ou fait un git reset ;)
> Ça roule en SPI ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/El-gitano/CProject/commit/48ce004d5966357aa1e4b52a177ac0407d439b71#commitcomment-10540999>
> .
>

--
Elyan Lanvin
Etudiant en 1ère année de Master Informatique

Mail : elyan.lanvin@gmail.com

Tel. : +33 (0)6.21.53.97.60


result = message.is_a?(String) ?  { error: message } : message
Here are two space.


saw approval from jdm
at https://github.com/yodalee/servo/commit/37b0c39bca0760cc2fa0c4e302ed6be391c779b7


ok, sorry for spaming, this is something with my host setup


merging yodalee/servo/issue5397-location-pathname = 37b0c39b into auto


yodalee/servo/issue5397-location-pathname = 37b0c39b merged ok, testing candidate = c12bd897


On second thought, you're just adjusting for the ship's position. Disregard lol


Oh, nice. I just sent you an email about some things I had questions about.
(edit: TL;DR how do you host it? heroku doesn't (or refuses) to work.) I'll
quote it here.

Hey, Eefreak.
> First of all thank you for noticing that I forked your repo, and also for
> looking at it. I am attempting to make some kind of Dungeons and Dragons
> steam group dungeon master helper thing whick,mh bot thing and I was having
> issues just trying to modify the node-steam-trash-bot. I really like what
> you have done with node-steam-chat-bot and I had some questions.
>
> Firstly, what do you use to host your bot when you test? I have an
> raspberry pi but node-steam and node-gyp mainly are giving me issues, like
> refusing to install all together. I also attempted to use Heroku, but that
> failed as well for various reasons.
>
> Do you have any tips that you could potentially give me to help trouble
> shoot my problems?
>
> Oh and some more back-story, if you are so interested in the reason I want
> to make what I am calling a D&D expansion pack for your node-steam-chat-bot
> is because I get accused of cheating, a lot of times in fact, so I wanted
> to make a chat bot so I could prove I got a certain roll. Also have you
> seen "Sir Govan Bot"? It is made by a friend, Sergiovan and he made a bot
> for the Nerd³ steam group chat and it is quite the thing. He built it from
> the ground up from the node-steam-trash-bot. Unfortunately it didn't get
> much attention most likely due to the fact he didn't fork the repo or make
> a new repo on git-hub but instead put it up on bit bucket and made it
> mostly open source with some restrictions because he is quite attached to
> the little guy. It has RPG elements so people in the chat can fight each
> other, and level up. It's actually pretty cool. You can check it out :
> here. <https://bitbucket.org/Sergiovan/govanbot> It's actually pretty
> freaking cool.
>
> Oh and thanks for pointing out that I accidentally commited my username
> and password. Whoops.
>
> Godspeed From,
> Titus
>

On Wed, Apr 1, 2015 at 4:34 PM, Efreak <notifications@github.com> wrote:

> [image: :+1:]
>
> There'a also a dice roller plugin, but it's vulnerable to attack by users
> rolling too many dice at once, given the way it works (I had to disable
> mine...). If you trust your users not to abuse it, it allows !roll #d#
> for a command.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/titusec/node-steam-chat-bot/commit/c8bed2ff35fedbf54ad131a6ad3c8b056955fdea#commitcomment-10533091>
> .
>


Usually we will just embed an anoymous function, rather than having a function named init.  i.e.:

```js
dojo.addOnLoad(function(){
   ...
});
```


My point is to raise any exception, that may be caused by OS or FS, since git-annex doesn't. So, all I can get from the Runner by now is an unspecific RuntimeError, reporting the exitcode (if returned).
The caller can't handle exceptions, that nobody raised. So, how to distinguish handling while not raising these?


It really does! Thankfully, I only wanted to rebuild one plugin so I exported it manually from Eclipse.


`result = message.is_a?(String) ?  { error: message } : message`
Here are two space.


Thanks.

Please fix the opening brackets to follow the consistent style.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (658 of 658 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-4.95%25-green.svg)


OK but to execute the blocks, [first we check if there is a block and then we execute it](https://github.com/cloudant/CDTDatastore/commit/902d9f99405722c3739047188bc38f5bddcf77e9#diff-bac15dfde454859cd8e3a8dc04fcf29fR102). Should we re-write that part? To make sure that the block does not disappear in between.


This has been cherry-picked as 4e4a574 into master.


Actually, I just realized there is a further problem. There is no guarantee that the attributes of the object that get hashed on will not get changed during the course of that object's life...which will lead to it hashing to different values at different periods in program execution.

pyanaconda/kickstart.py actually does set the mountpoint attribute of its BTRFSData object to a new value in some circumstances, and because it is the value that __eq__ uses for comparison, it is also the value that __hash__ uses for hashing. So that is definitely a bug.

I'm not familiar enough with pykickstart to know what other of these attributes that get compared for equality might also be changed during execution. It might be really painful to audit them, or it might not.
An alternative to verifying that none of the attributes get changed is just to have all __hash__() functions return, e.g., 0. This maintains the relationship required between hashing and equality; hashing itself is __very__ inexpensive, but storing things in dicts is only as efficient as a single bucket can make it.


@Mikey1993 agreed, but I don't have the write permissions (so I can add new strings, etc.) to the Crowdin project.


``!isNaN(target.substr(3)))`` won't stop genInfinity and gen-Infinity from passing through. What about ``Number.isInteger(target.substr(3))``?


Not clear what it means...  docstring?

Also note that any annex'ed file is also under Git.  I wonder if it could happen that Annex knows about the file but git doesn't ...  probably not:  http://pastebin.com/VwFzKgb8   but might be worth to keep in mind


This is included in ed9bf8e.


I'd rather not advertise a pay-only service explicitly. There's a number of free alternatives such as
http://truetimestamp.org/
http://www.freetsa.org/
or
https://www.pki.dfn.de/zeitstempeldienst/


Done.


initial fix for ZD 7862 - will need a full review for mongo 3


This is invalid syntax `QuestionPaper $qPaper = new QuestionPaper($questionpaper->title);`
Should be `$qPaper = new QuestionPaper($questionpaper->title);`


Hello Sir,

in c++ the completion doesn`t seem to find my includes, unless I `#include "/absolute/path"`
Is this the expected behavior? Is there any workaround?

Thanks for your help so far!


worked on my end i guess i didn't think that CI doesn't have a persistent database between tests....


Where is this code coming from?
If it's coming from ACE we should fix the ACE update script to include it.


Potential improvement: Keep ->children sorted and use bsearch to find the right place to insert.


:+1: curti


Ditto.


This one I think is the best one, sorry for writing so many comments

stepsForm.prototype._validate = function() {
		// current question´s input
		var input = this.questions[ this.current ].querySelector( 'input' );

		if (input.hasAttribute('data-validate')) {
			switch (input.getAttribute('data-validate')) {
				case 'none' :
					// there's no validation in this field, moving on
					break;
				case 'email' :
					if (!this._validateEmail(input.value)) {
						this._showError( 'INVALIDEMAIL' );
						return false;
					}
					break;
               case 'checkbox':
               if (!this._validateCheckbox(input)) {
                   this._showError('INVALIDCHECKBOX');
                   return false;
               }
               break;
			}
		} else {
			if( input.value === '' ) {
				this._showError( 'EMPTYSTR' );
				return false;
			}

		}

		return true;
	}

    // email validation
	stepsForm.prototype._validateEmail = function( email ) {
		var regex = /^([a-zA-Z0-9_.+-])+\@(([a-zA-Z0-9-])+\.)+([a-zA-Z0-9]{2,4})+$/;
		return regex.test(email);
	}
    stepsForm.prototype._validateCheckbox = function( checkbox ) {
    return $(checkbox).prop( "checked" )
    }


Hmm one trailing parenthesis too many, I'll fix that.


May be. Not sure yet. I wanted the catcher to know, what layer (AnnexRepo) an exception came from for proper handling. But it may as well be unnecessary.


Or you just remove the date entirely, that doesn't void your copyright anyway.


Couldn't this just be an else? I imagine schema won't change again in 0.9+, and having multiple elif's here for each version "won't scale."


@hyperNURb you need to be more careful with making changes like this. This caused the Search button in the top bar to throw a script error.


This change broke search in all modals :/


oh... could we avoid this?

See btw https://github.com/datalad/datalad/issues/96 which is revealed now to me by this test ;)   Assigned it to Jason as for the excercise so you could just proceed


r+


I get `Cannot GET /` at http://localhost:8080/


Now you can do stuff like:

    def get_avg_position(entities):
        return sum([ent.pose.position.x for ent in entities]) / len(entities)

    FuncDesignator(EdEntityCollectionDesignator(amigo), get_avg_position, resolve_type=float).resolve()
    0.5121250483751295


So, the cache content is a simple concatenation of records values. Tags and records lengths are not cached.
Is it really your intention?


Make "name" a member of the etcd object.  If it's empty, this block should be skipped (found && len(e.WatchSingleFieldName) > 0)


Yes, I think this commit gives the code the right semantics. Thanks!


nice


We can. And we will. ;)


testar


Here is the very first error message - the one that triggered repeated peeks into the code of this module while troubleshooting:

    /srv/MRI/WORKING/COPB/19/cmd_150331_151052_EPI2T1.sh: line 982: syntax error near unexpected token `('
    /srv/MRI/WORKING/COPB/19/cmd_150331_151052_EPI2T1.sh: line 982: `echo '    - Uses simpler blurring (iterative on input dataset).''

Hope this helps Joseph.


Add a test for the disabled mode discussed above


https://github.com/wkentaro/2014-semi/commit/27efa7f21cbfe4fd47a4f6a00b594bcab6a0dcf3
対応部分を更新しました。問題ありません。

2015-04-01 22:31 GMT+09:00 和田 健太郎 / Kentaro Wada <www.kentaro.wada@gmail.com
>:

> もうマージされていましたね。了解です。
>
>
> 2015年4月1日水曜日、Kei Okada<notifications@github.com>さんは書きました:
>
>> あ，すでにbranchは消してしまいました．ということで，origin/masterを見てみてください．
>>
>> --
>> ◉ Kei Okada
>>
>>
>> 2015-04-01 22:24 GMT+09:00 Kentaro Wada <notifications@github.com>:
>>
>> > これは、branch作らずにpushされているようですが、mergeできないので、copyすればいいでしょうか。
>> > また、このようにしたいときはどうすればいいのでしょうか。
>> >
>> > 2015-04-01 22:02 GMT+09:00 和田 健太郎 / Kentaro Wada <
>> > www.kentaro.wada@gmail.com
>> > >:
>> >
>> > > 了解です。
>> > > 明日になりますが、見てみます。
>> > >
>> > > 2015年4月1日水曜日、Kei Okada<notifications@github.com>さんは書きました:
>> > >
>> > >> @wkentaro <https://github.com/wkentaro> これで副作用無いか見てみてください．
>> > >>
>> > >> —
>> > >> Reply to this email directly or view it on GitHub
>> > >> <
>> >
>> https://github.com/k-okada/2014-semi/commit/2411d3bd582f6c8a42a2d832497e957c7699ae49#commitcomment-10521423>
>>
>> >
>> > >> .
>> > >>
>> > >
>> > >
>> > > --
>> > > 和田 健太郎 / Kentaro Wada
>> > > http://wkentaro.com <http://www.wkentaro.com>
>> > >
>> > >
>> > >
>> >
>> >
>> > --
>> > 和田 健太郎 / Kentaro Wada
>> > http://wkentaro.com <http://www.wkentaro.com>
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/k-okada/2014-semi/commit/2411d3bd582f6c8a42a2d832497e957c7699ae49#commitcomment-10522012>
>>
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/k-okada/2014-semi/commit/2411d3bd582f6c8a42a2d832497e957c7699ae49#commitcomment-10522082>
>> .
>>
>
>
> --
> 和田 健太郎 / Kentaro Wada
> http://wkentaro.com <http://www.wkentaro.com>
>
>
>

--
和田 健太郎 / Kentaro Wada
http://wkentaro.com <http://www.wkentaro.com>


I'd call it AddContainerPortConfigNames, since there might be other entities that also have ports...


aiui I changed that to appease a test, (not sure if that's because of an error, or just I couldn't easily elicit the utf8 character.  I'll try reverting and adding 'use utf8' and copy/pasting the character...)


What if you set MAX_SLAVES_PER_SPLITPOINT = 100, what is your fastest minSplitDepth? If it is 7 then you might be onto something.


Maybe?  In truth we shouldn't need realpath here.  But, sometimes it's beneficial for path comparisons or Windows differences.


Pushed?


Following our discussion, we should come up with some unified way we reference/name filenames/variables.  Is this path_to_file intended to be a full path, relative to top of the repo, or current?  For now in my code I have started to use "_path" suffix whenever I mean full path,   so may be

file_ -- some filename, could be full/relative/just_a_filename.  Function has some logic to deal with each, or doesn'
t care really
file_name - just a file name (e.g. ==os.path.basename(file_name))
file_path - full path to the file, mustn't be os.path.join'ed etc
file_rpath - relative path (which could also be smth like "./file_name")?


It would be good to also have a test for the `print` method.


Sorry, why this?
It causes this exception:

```
PDOException in PDOConnection.php line 43:
SQLSTATE[08006] [7] invalid connection option "client_encoding"
```


In "liblwm2m.h" line 310 is still an unused typedef for the "lwm2m_copy_callback_t".


L261 study missing a 't'


Ah index from 0 :-)


wondering:  should we add a parameter "raise_=False" to Runner.run  and raise only if True, otherwise return ?  Then we could code for both ways


Use spaces. Check editorconfig.


Maybe player in players. So we will have player.img, player.score...


io.write("Invalid password. Please insert a valid password\n");


why not following the name pattern and define that in a metatag for consistency?


Looks like this commit broke test_add, test_callable, test_negate, test_nested, test_reference, test_relaxed, and test_tuple on all compilers in in C++11 mode:
http://www.boost.org/development/tests/develop/developer/type_erasure.html


Beware, you can use `distinct=true` if there are no facet filters, but some tag filters or some numeric filters or a query.

So probably something like:
```
if (!query && facetFilters.length === 0 && numericFilters.length === 0 && tagFilters.length === 0) {
  distinct = false;
}
```


inline styles? :)


io.write("Password mismatch. Please try again\n");


The problem is the inconsistent behavior between ruby scripts +  tasks inspector which all use floor and the yaml configurations. How about a warning/error on the console?

    if field_type.integer? && val.to_i != val
        warn "Converting '12.deg' into integer 'XXX'  Use '12.deg.round' to suppress warning"
    end


io.write("Power must be a valid number\n")


Yes, the PR is updated.


LGTM


Returning a list makes sense to me


That bad?


Hi Aman, I'm using the module fine, not sure what is going on over there.


Такое надо делать на всех репозиториях с пользовательской документацией. А то будет все по разному выглядеть. Поменяю и у мобильного nextgis/docs_ngmobile


@iCandyLabs: This is an issue caused by resolution matching as the legislature pages changed their names; it's not due to code, and your local `billy-update` runs should be correct. I'll fix this in our database this morning.


I think you can use the iterators more here (not for now, but later) and use .iter().filter(|chunk| { if remote { decrypt etc. } there is also a `.as_slice().contains() ` is nice as well. Just FYI no worry right now as I say.


@cv @jeankirchner O que acham de componentes neste estilo? Atenção especial para os parâmetros passados para o fragmento e a url sendo cria dinamicamente pela expressão ```@{${url}(__${parametros}__)}```


Nice one


data3 -> winner ?


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


@iCandyLabs Please stick to support channels. I've got your email, and eating up my team's time by commenting on every avenue you can find isn't helping us get work done.

Thanks.


Now that you mention it, I forgot I get 10x more bandwidth than I did on
my previous web host. I could put it back for now... but I'd rather just
say "NOPE" until someone comes up with a solution

On 02/04/15 14:49, Wouter Wijsman wrote:
>
> That bad?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/steamos-community/stephensons-rocket/commit/791d7ed8a57d52375250a41792d16f3c8103543a#commitcomment-10546297>.
>


Yes I see your point. But Universign is certified RGS (a french stuff) and compliant with ETSI TS 102 023. The free services you cite are more of a proof of concept thing that no lab should use for their intellectual property TS. They can be offline at any moment, or totally disappear.

We're dealing with labs that receive A LOT of money from different sources and the price of timestamping is so low compared to let's say the price of an antibody, that the issue here is more to have something you can rely on, rather than something that is free. Universign is used by le Conseil d'État, which is the highest juridiction in France, so I'm pretty sure that Universign is the best there is out there.

Also, I want elab to be easily usable. Do you really think that a lot of people will go through :
- looking for a TSA
- looking for its certificate (in PEM ascii format)
- put that on the server
- configure everything

Whereas here they basically need to input login/pass as before.

I hope I made my point clear :)


Hello,
may I ask what this pkcs11-proxy.diff is ?


If you had waited for us to do the 1.8.3 update you would not have had to remove this because we resolved the issue upstream.


@riccardonar can you provide more details of your usage?

See the pull request / merge:

https://github.com/doctrine/dbal/pull/823

This fix was applied to fix support for pgbouncer (which doesn't like the 'options' param).


should be `_document`


Since git-annex is called with that path, it can be an absolute path as well as relative to current working dir.
But this is not entirely clear or unified within this class, that's why I started the discussion you mentioned above.

I'm fine with your naming, but then the word "relative" has to be clarified. git-annex calls could always run in current process' cwd, so this would mean file_rpath is relative to os.getcwd(). Or AnnexRepo could call git-annex always from within repo's base dir, so file_rpath would mean relative to this one. That's the decision to make.


General things to consider:

- ```--cache``` probably needs to use the new infrastructure too by not using the full path as key for ```ext_cksums``` but by storing only an id in it (either pointer to GNode or rowid integer of sqlite)
- treemerge.c is using libart currently for similar reasons.
  Drawback: It stores the full key for each path node, so it will be a memory problem:
  https://github.com/armon/libart/issues/10

  So future work might be to move the tree code partly in an own library which is specialized on storing
  paths compressed and might additionally store a value for each node. But that should be low priority for now.

Very solid work btw.

Sidenote:
You might want to check your ```git config user.email```/```user.name```. Your avatar is not showing right.


I would document or remove undocumented @param and @return


these are not `isMobile`


Why are there two?


This has been added in 4e4a574.


Why is_dst=True? Surely there's no DST in UTC?


Why not merge these classes into single class?


rock-gazebo creates the port using the source_link name.
I will modify rock-gazebo to name the ports as you said "source_frame2target_frame".
So, there is nothing to change in this package (bundles-flat_fish) ?


Talking about midnight is a bit ambiguous. Maybe talk about the start of the day/00:00:00 instead?


@paultag I apologize.  Where shall I communicate then?  I don't want to create a distraction for your team. I thought I was helping.


@El-gitano voilivoilou, en plus c'est juste apres la mise en ligne de la derniere version que y a eu ce merdier.
et oui Elyan les SPI sont des branleurs ^^


Surely it would be great to have customizable logging. But until it is implemented I would prefer to have these printlns.


I would call it only name and not port_name, it's already under container_port_config so it's clearly related.


turns out it's supposed to be negative not positive, gotta change this elsewhere where we allocate memory too


some tests failed:
failure: http://build.servo.org/builders/linux2/builds/1257


@iCandyLabs this is a totally unrelated changeset - and duplicated legislators happens when we update a session, it's something we check for and process -- I also got your email 40 minutes ago, on the train into work.


Well ... you'll have to update the script ;-). But no, the comment was indeed in the wrong place.


Why not just use `context_timestamp` to compute until here, and only do the resetting and conversion back to UTC in this method?


Forgot story number: this is for UIT-37


AddContainerPortConfigNames sounds more clear, port config is too general


Sums up to a lot of parameters. May be we don't to raise at all, as long as
there is an exit code to be returned. Besides, we also might have to
rethink how to handle stdout/stderr. Since information about what went
wrong is in there, we don't want to just log it, but actually get it from
the instance, that is calling the runner. So, I wonder, whether it would be
a good solution to have the loghandler storing recent log-reports or sth
along these lines.

2015-04-02 15:46 GMT+02:00 Yaroslav Halchenko <notifications@github.com>:

> wondering: should we add a parameter "raise_=False" to Runner.run and
> raise only if True, otherwise return ? Then we could code for both ways
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/datalad/datalad/commit/f06a505a8593ec7dae4f441c9f4521e9e69e6642#commitcomment-10546216>
> .
>


If you don't need this part remove it :)


I like that ... Will do


last line 168 seems not aligned with the rest (the => part)


@rocksfrow did you actually test the `client_encoding` connection option with a setup that does not use pgbouncer? Looks like the `client_encoding` option is not supported with those setups... If so we might need to revert this change


assert(c.bucket_count() != 0)


Hmmm, with the instr() code, it's looking for the PPRID value of that row within the list of all PPRIDs (results) being shown on that page. Though with this LIKE version, it's not doing that, which I believe would cause an issue when viewing a page with multiple results (multiple PPRIDs)?


what if in the future we want to play a marathon that contains like 100 rounds :) ?


@hoholee12 - try this color control :) , its based on what you used which is the sh equivalent (ansi stuff) of tput & setaf in bash, its just easier to color text with this method since you use the variables instead of complex messy stuff...


Use a one-line ?


saw approval from Manishearth
at https://github.com/jgraham/servo/commit/d6a608806e84ea756eac171312adc2d08a0e30e8


merging jgraham/servo/css_tests_1 = d6a60880 into auto


to much indentation


merging jgraham/servo/css_tests_1 = d6a60880 into auto failed


@deeky666 yes of course - but it must have been introduced in a newer version.

Give me a few minutes to figure this out and I'll create another pull request.

Reverting the change means we're removing support for pgbouncer. Let me get some more details and see what's going on.

client_encoding is definitely supported -
http://www.postgresql.org/docs/9.4/static/libpq-connect.html


Well, have to add docstring of course. It is meant to indicate, that annex doesn't know about a file, but git does.


looks like a constant, can be also PLAYER_DEFAULT_NAME


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/a65c80231a5aa059fd782739ab77f52a60f59d8a


merging Ms2ger/servo/int = a65c8023 into auto


Ms2ger/servo/int = a65c8023 merged ok, testing candidate = d999fb9d


Why was this removed?


Looks good.


+1

Will need double checking for testing. But this refers to this issue.

https://groups.google.com/forum/#!topic/sensu-users/n6cUZ5GaP1Q

Albit annoying us Debian users just need to replace the init.d file


I think you mean "treat"?


I changed the distance calculation to not adjust for ship position so the values display correctly in accordance with distanceTraveled. If this was the wrong way to fix this, let me know a better way


@deeky666 damn -- yeah just confirmed what I was saying. The client_encoding option is def supported -- but if you're using a non-supported version of postgres -- it doesn't.

So obviously we need to support connecting to older versions of PG -- so that's my bad, I didn't realize it was introduced in 9.x.

So this fix is valid for 9.1+ but not previous versions.


One could shorten this: make the resource name an argument of uploadData() and doGet().


@pcqpcq joker哥，欢迎啊(^_^)


well -- if not to throw exceptions at all, then we would need to come up with some additional error codes for any possible exception Popen could throw... imho not worth it.

processing outputs -- we have return_outputs so that should be sufficient if outside caller needs to process them.  There should be no need to throw them up all the way in the stack. If there is a need -- we should stop and think... e.g. one of the ways is to throw the exception instead of just returning and let "where needed" to analyze it.


defined twice?


Thanks, I opened the issue #1. Please wait for the fix (or submit your patch).


Just to keep in mind: there is a length restriction for string fields.


If the python script uses that as a file name, then the python script could format it adequately. This would make the command line interface more robust, and separate responsibilities better.

By the way, what will happen in sprint 100?


`#include <stdint.h>` missing.. :)


It created an infinite loop :-)


@deeky666 does the doctrine have any policies on this?


tere hommigjust


tested and working


The RadRibbonBackstageItem control exposes


first time


In Css you have:
``` css
.circle {
cursor: pointer;
border-radius: 50px;
width: 100px;
height: 100px;
margin-right: 24px;
margin-left: 8px;
}
```
try to avoid magic numbers. You can make e.g.: `border-radius: 50%;` to always have perfect circle. Also you can use variables e.g.: `circle-size: 100px` -> `border-radius: circle-size / 2; `


Done.


Btw if you want to test it:
fails: `
CREATE TABLE my_test_table
(
	from VARCHAR(50)
)
DISTSTYLE EVEN
; `

succeeds: `
CREATE TABLE my_test_table
(
	"from" VARCHAR(50)
)
DISTSTYLE EVEN
; `


First point: That's not what I meant. If popen throws an exception, then
there is no exitcode, so we would throw it upwards. But if popen retunrs
properly don't throw, but return exit code.

Second point: You're right. Forgot how returning the outputs already
changed. Should do it indeed.

2015-04-02 16:09 GMT+02:00 Yaroslav Halchenko <notifications@github.com>:

> well -- if not to throw exceptions at all, then we would need to come up
> with some additional error codes for any possible exception Popen could
> throw... imho not worth it.
>
> processing outputs -- we have return_outputs so that should be sufficient
> if outside caller needs to process them. There should be no need to throw
> them up all the way in the stack. If there is a need -- we should stop and
> think... e.g. one of the ways is to throw the exception instead of just
> returning and let "where needed" to analyze it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/datalad/datalad/commit/f06a505a8593ec7dae4f441c9f4521e9e69e6642#commitcomment-10546716>
> .
>


"how to distinguish" -- to analyze the situation: is the file present?  is it a symlink? is it a broken symlink? etc  and then as a result throw more meaningful exceptions which we reserve this function to throw (ideally should be present in `Raises` section of the docstring.


clear all old photos before we choose another photos to post


Trailing white space here :)


LGTM


well, "levels" could be explicit if necessary by importing this submodule "as annex" and then "except annex.CommandNotAvailableError" etc


I just remembered that org mode is one of the main reasons I like emacs. There's a von version Google it it's great


i really meant nettables


I think you want `arch` here - that defines the current kernel personality which may be `i686` even on an `x86_64` kernel.


if you want to upload a image to post , you can use a nested form for it but here I want to upload multiphotos to post so I need some jquery process here


Yes you did :) But is it really necessary to put in Universign's config as default? Because this will surely lead to people just rely on the provided defaults and complainig afterwards that it's not working/have to pay for/etc.
Imho it's enough to provide the certificate and to explain in the README how to enable Timestamping and using Universign there as an example config.

Cheers :)


@mhoeber this adds the docs email address and makes this version parallel with the edx-analytics-dashboard version


We're not actually using any of this in this test right?  It feels like the current `libtest.sh` should probably be `libtest-client.sh` and we create a new `libtest-server.sh` (with some common inheritance for assertion functions or so).

Anyways, don't need to do the latter part now, but if we can drop this line, might as well.


It looks like the only totally safe way to set the client encoding is via the SET sql command -- not via connection param.

If we add it to options like you had before - you break support for pgbouncer (this change was introduced since a previous release and actually broke my environment after upgrading the class). BUT, leaving the fix as-is removes support for versions before 9.1.


Ok, so I'll build proper exceptions here (all of them would be derived from
IOError as you requested in #91). Basically the same as is, but ensure
everything is IOError and not this somewhat unclear and implicit way, right?

2015-04-02 16:15 GMT+02:00 Yaroslav Halchenko <notifications@github.com>:

> "how to distinguish" -- to analyze the situation: is the file present? is
> it a symlink? is it a broken symlink? etc and then as a result throw more
> meaningful exceptions which we reserve this function to throw (ideally
> should be present in Raises section of the docstring.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/datalad/datalad/commit/f06a505a8593ec7dae4f441c9f4521e9e69e6642#commitcomment-10546817>
> .
>


suite


Not sure but it should be

*This serie ... works*

or

*These series ... work*

No?


RadRibbonView provides a __RibbonCommands__ class that contains several __RoutedUICommand__ properties


umm.. that's for group, but no group opcodes actually working. How you tested it? :)


Are you sure? Oder etwa This way?


mto bom


LGTM


@cbeer we are setting both. Setting `SECRET_KEY_BASE` by itself does not prevent a Blacklight error from bubbling up when we run rake tasks in production, e.g. `rake assets:precompile`. Are the two supposed to accomplish the same thing?


`uri = URI query` is even more ruby-like


Actually, this is the third option `This series ... works`, my bad !


@riccardonar can you confirm you're connecting to an unsupported version of PG? (unsupported by postgres, not this class heh)


OK nice gan


test


test 2


This is what you need to explain. What is concatened, how, how many times, is there configuration options?

I don't want to bragg, but I think that the [bench_markdown readme](https://github.com/privat/nit/tree/master/benchmarks/markdown) is a good example of what we expect from the README


It won't rebuild the VBOs, we don't have a good mechanism for that right now


Will be fixed by 4207c491a17f9c721868ef9e125a365921e6b704, once that gets merged.


Need to expand/describe what a TCE is


fixed


Line is too long. [107/100]


Missing space after #.


Line is too long. [113/100]


Avoid using {...} for multi-line blocks.<br>Use the `lambda` method for multi-line lambdas.


Line is too long. [119/100]


Line is too long. [141/100]


Yes, I guessed so. Just wanted to make sure if that was the intent.


They are a probleme with this line: the check if node['php-fpm']['pools'] exist is made after this block....


Don't! There must not be a space between these elements!
@AliaksandrVahura-ScienceSoft please restore the comments


コメントテスト


Ohhhh shit, News Ketchup is back!


should be {SITEURL}


I doubt that cxl.txt will refer to the Flash AFU. Perhaps we just describe it inline here as the Flash AFU being the equivalent of firmware on a PCIe adapter, with the CAPI exception that it can address host memory.


So these comments were for that purpose? Ok, i will revert them.


thanks


@jamesward what was the reason for excluding the `vendor` directory? The entire app (HAL Browser) is broken because all dependencies are missing. Thanks.


All tests pass on master. Manually ran `_.each` and it works fine now.


Or

    try:
        return shutil.get_terminal_size()[0]
    except:
        return 80


asdasdd


asddffdsfd


@bors: retry


+1

We need to document this somewhere (eventually).


Were you in Gateway mode? If yes, you have the reason. It's the same problem as with gamecard update bricks.


What's the typo? Hard to tell here...


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

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


Yeah, I bumped into the same issue. To get things going, I simply tweaked MakeVerboseFormatProcess and hooked into that: https://github.com/jdduke/assimp/blob/opengex_support/code/OpenGEXImporter.cpp#L571. Probably not optimal, but it satisfied the import pipeline :).


I have some issues with the new checksumming code.

The various versions differ in the endianness of in- and output. AFAICT, `cksum_generic()` expects `initial` in host byte order and returns the checksum in network byte order, while the other variants do the opposite.

`ipsum()` returns a finished checksum. It appears to me that this makes it impossible to calculate the sum over a set of non-contiguous blocks as with the old `update_csum()` and `finish_csum()` API (or is there a trick to do this anyway?). That would make the `initial` argument a misnomer, too, because it would only serve for checksum verification (which is probably the reason why it's expected to be in nework byte order by the SIMD versions).


These two lines really should be commented out, as they're leftover debugging. The pass is correct.


You can also use a `\d` for consistency with the other routes


haha, not gonna leave a job half done!

let me know if you want to work on it too!

On Thu, Apr 2, 2015 at 9:28 AM, Shomari <notifications@github.com> wrote:

> Ohhhh shit, News Ketchup is back!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/elyotli/Project-Clue/commit/010d7d3222cd677cba2aed43efa77b0c41f69075#commitcomment-10547075>
> .
>


Table `format` was deleted here, which I think is causing [this error](https://gist.github.com/jacquerie/1f9453605570fe762b73#file-invenio-demosite-log-L2724-L2741).

I'm not sure if this is incidental (legacy code was looking for a table called `format`, which was created here for unrelated purposes, now that it was deleted it's throwing that error) or an actual error, but I seem to get fewer records in the UI than what I'm supposed to see: 108 instead of 142.


I'm pretty sure this isn't working right - all my jenkins user comments in bitbucket now report as "? SUCCESS" - i.e. the unicode character is being stripped.

Also, even *if* this worked, the lack of SVG is a real usability problem if you've got lots of commits and lots of builds - without color, we've overlooked several failures when attempting a merge.


go prod


不要かと。


Yeah, either way.


Move that function to the top level of the file. This will make the make_all code more readable (and allow to reuse it for the older linker if we wish later)


@deeky666 I am going to create an alternate fix -- which IMO is the only way to do it AND support both pgbouncer and old version of postgres. Obviously the fix you already merged is much cleaner -- but this will just call SET client_encoding after connection instead. The class has similar behavior with setting a couple attributes after connection (like disabling prepares).

I'll include the revert of my previous change in that pull request so you don't have to worry about reverting the previous one.


This change actually breaks more then it helps ...

testCopyOverwrite()
````

Expected /target.txt to be a copy of /home/deepdiver/Development/ownCloud/master/tests/data/lorem.txt
Failed asserting that two strings are equal. <Click to see difference>

 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:194
 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:234

Expected /target with space.txt to be a copy of /home/deepdiver/Development/ownCloud/master/tests/data/lorem.txt
Failed asserting that two strings are equal. <Click to see difference>

 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:194
 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:234

Expected /target.txt to be a copy of /home/deepdiver/Development/ownCloud/master/tests/data/lorem.txt
Failed asserting that two strings are equal. <Click to see difference>

 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:194
 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:234

Expected /target with space.txt to be a copy of /home/deepdiver/Development/ownCloud/master/tests/data/lorem.txt
Failed asserting that two strings are equal. <Click to see difference>

 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:194
 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:234

Expected /tärgét.txt to be a copy of /home/deepdiver/Development/ownCloud/master/tests/data/lorem.txt
Failed asserting that two strings are equal. <Click to see difference>

 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:194
 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:234

Expected /target.txt to be a copy of /home/deepdiver/Development/ownCloud/master/tests/data/lorem.txt
Failed asserting that two strings are equal. <Click to see difference>

 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:194
 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:234

Expected /tärgét.txt to be a copy of /home/deepdiver/Development/ownCloud/master/tests/data/lorem.txt
Failed asserting that two strings are equal. <Click to see difference>

 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:194
 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:234

Expected /tar'get.txt to be a copy of /home/deepdiver/Development/ownCloud/master/tests/data/lorem.txt
Failed asserting that two strings are equal. <Click to see difference>

 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:194
 /home/deepdiver/Development/ownCloud/master/tests/lib/files/storage/storage.php:234


instead of g_strdup, just assign split_path[i] and free the split_path vector with g_free later.


編集画面に初期値をセットしています。


ここは`sh "convert \`(半角スペース)じゃなくて大丈夫ですか？（Windowsのビルド環境がないため確認できずすいません…


It's not clear how the owner in `assert_owner_dirty` is related to the cases created and/or updated by `self.factory.create_or_update_case`. Does `CaseStructure` create a new case or does it magically reference an existing case in the factory? What information from the factory is influencing `create_or_update_case`?

I noticed in https://github.com/dimagi/commcare-hq/pull/6147 that `CaseRelationship` defaults to a "parent" relationship. Are we expecting to have many other types of relationships other than "parent"? Would it make sense to create concrete relationship classes like `Parent` to make this more clear? Example:
```py
CaseStructure(
    relationships=[
        Parent(CaseStructure(attrs=...))
    ]
)
```
Would also be nice if `CaseStructure` could be simply `Case` here, but I suppose that would introduce confusion at other places?


新規登録時にデータを保存する処理になります。


Before I do that -- I am going to try one other thing. Sorry about all of the comments -- just look out for my pull request soon.


更新時にデータを保存する処理になります。


>    Ok, so I'll build proper exceptions here (all of them would be derived
>    from IOError as you requested in #91). Basically the same as is, but ensure
>    everything is IOError and not this somewhat unclear and implicit way,
>    right?

I think so


The purpose of this block is to remove `www` pool which comes as default on most distributions, unless there is `www` pull defined on the cookbook. In other words it should remove `www` with zero pools as well.

So, it missing part of condition, something like

    unless node['php-fpm']['pools'] and node['php-fpm']['pools'].key?('www')
      #...
    end

Anyway that is nice catch :+1:

Thank you @pojer-s


Confirm it's fixed. Also the commit seems to help with a lot of samples that were previously broken.


Could these tests have different pass/fail outcome if they were run in a different order that the order they are defined on the class?


ALCommonUtils.compressStringは文字を丸める処理なので、長いURLだと省略されてしまうかもしれません。。。


Ist das mit dem 1000ms warten wirklich gut? Reichen nicht vllt ein paar hundert Millisekunden? Ich würde noch eine Prüfung in doRefresh einbauen, dass er nichts macht, wenn er schon gerade ein refresh ausführt. Sonst kann man es mehrmals drücken.
Sieht sonst gut aus. Werde ich heute abend vermutlich rein mergen


jadsljds


of course gateway mode. How else can you run a .cia file?


Actually, we do not always know what is the original source of the document. Since I have bulletproof the xwikimodel to provide the best it can, it will be support anywhere the wikimodel parser will be used. To me it looks better to support them in a unsupporting syntax than ignore them purely, which cause data to be merged.


nevermind. `setUp` is called before each test. I tend to forget that.


ALCommonUtils.replaceToAutoCRは改行コードをhtmlのbrタグに変換する処理になります。
テキストエリアなどの複数行入力のデータで使う感じですね。


改行がおかしくなってしまったかな？


Can this specific commit stand on it's own?


well, its needed when groups are done, I have been already working on groups, so it will come
there are no big changes which needs to be tested, the logic is still the same, just updated structure


The Learning Scala link gives this result:
You do not have access to this book.

You are currently logged out. Try logging in!


this eliminated an inspection warning, but this really doesn't matter since the current library has the minify method defined as private which should make the function not-callable.


getUrlNameでResultDataのUrlが取得できます。
ここでは呼ばなくても良い処理かなと思います。


Äääääääh, ich hab "That's way" gelesen. Ja, this way ist besser.


Hi Marc

... and rock on!

Thanks for your advise and interest regarding Emacs & Vim. I'm pretty sure
that I know the pros and cons of the two editors, since I have spend half a
year with Vim and the last half year with Emacs.

I must say that I'm in love with the light feeling I get when coding in Vim
and Vundle package management now makes it a lot easier. But I mainly do
scientific stuff in python and nothing beats Elpy in Emacs (if you do
python stuff, man your in for a treat). I tried Evil mode and while it is
okay, I feel that it is not obtimal.

I looked at yadr and spacemacs, but since I like doing the config my self
it is not really an option. By the way spacemacs do not use Elpy.

I've looked into org mode and while it is nice I prefer writing my todo's
on a piece of paper.

Best,
Willi

On 2 April 2015 at 16:17, RocknrollMarc <notifications@github.com> wrote:

> I just remembered that org mode is one of the main reasons I like emacs.
> There's a von version Google it it's great
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wcarlsen/Homebrew/commit/f33227cf7f0eb41a0b598d640e93f5d84d941f30#commitcomment-10546855>
> .
>


ALEipUtils.getMessageList
が自動リンク追加処理・wbr追加処理などを行っています。


2. Reading "Learning JavaScript Design Patterns" :http://addyosmani.com/resources/essentialjsdesignpatterns/book/#revealingmodulepatternjavascript

3. Angular style guide: https://github.com/johnpapa/angular-styleguide#modules


@prudhomm Are you sure? I have an error on curie : c++: error: unrecognized command line option '-std=c++14'
I use gcc4.8.1


data-lazy-srcですが、Aipoではこのプラグイン入れてないので上手く動作しないかもしれません。


done


TeamCity xBIM :: XbimExchange :: Build [Build 183](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1279&buildTypeId=XBIM_XbimExchange_Build) is now running


Have to start somewhere!


why not but I would really like to know if setting errno to 0 and testing it not nul value is needed. It would be better if avoided.


Hey, do I see it right that `cactus()` is now `cir_query()` in webchem? Just wanted to add a pullrequest for `cactus()`.
If so, what is the reason for removing the possibility to have `length(query) > 1`?


Yeah but that wouldn't work with combination or minification.


Added Shipyard initialization


getUrl()と同じ値を返しているのでどちらかのメソッドは消してしまってもよさそうです。


Merge local copy with README + license


DebZ advice


And what about fixing the server so it doesn't send malformed packets?


@chrisaljoudi use this http://www.oneskyapp.com/ its free :+1:


Yes, perfect!


This concludes #48


thi is


@jianghejie 菠萝哥？


Habit I picked up at google, where their static analysis package (which I think is v. similar to `pylint`) requires it.


TeamCity xBIM :: XbimExchange :: Build [Build 183](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1279&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:02:58


I tracked down #2125 to this line which is causing the problem. Seems to be inside $mdTheming link provider. I hope this helps fixing the problem.


可以积累几个COMMIT在一起 MERGE, 不要每次咦 PUSH 就 MERGE

@moonsin


I discussed with @Zeno- about some solutions for this problem of reliability, the packet ignoring is the solution we choose. A patch will be provided and pushed today or tomorrow.


LGTM


This should probably be part of the XAR plugin format mojo.


@MostafaSuper This is not the appropriate place for requesting assistance. Please open a new [issue](https://github.com/tmcgee/cmv-widgets/issues) or ask a question at [gis.stackexchange.com](https://gis.stackexchange.com/questions/tagged/configurable-map-viewer).


why you used multiline comment for 1 line?


Are you delegating most of the logic now to the Node.js service instead of database? It can be faster but remember that Node.js is single-threaded so when many requests hit the server it will be the bottleneck


NICE :+1:


Well, with private stuff and there it works fine. I can't do anything really about it. It's Brickways fault. I already explicitly update these partitions: https://github.com/profi200/sysUpdater/blob/master/source/main.c#L103


`GetOwner()` can and will be null


ちょっと待っててね。
今遅延ロード機能付きのプラグインマネージャの仕様を調査しています。
`exists(':CtrlP')` を使うことで問題なさそうなら、その仕様にします。
恐らく3日以内に結論が出るでしょう。
それまでお待ち下さい。


Yes. When proper documentation is written, I'll remove the default Universign (and I'll add it back if they pay me to do so :p)


How can I utilize the plot generation?

Here are my instructions:
* make qemuauto
* ./extract_context_switch_time.py
* gnuplot plotfrequency.gp

But I didn't get any visible result.


Oh, right. I originally wasn't doing anything else, so the pass was needed, but then I added the print statements afterwards.


root@ocean:/home/rpusbdisp/drivers/linux-driver# make
make -C /lib/modules/`uname -r`/build KCPPFLAGS="-I/home/rpusbdisp/drivers/linux-driver/src -I/home/rpusbdisp/drivers/linux-driver/../common" M=/home/rpusbdisp/drivers/linux-driver modules
make[1]: *** /lib/modules/3.10.17-poky-edison+/build: No such file or directory.  Stop.
Makefile:30: recipe for target 'modules' failed
make: *** [modules] Error 2
root@ocean:/home/rpusbdisp/drivers/linux-driver#

遇到这样的情况应该怎么办呢


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1260
success: http://build.servo.org/builders/linux2/builds/1258
success: http://build.servo.org/builders/android/builds/1294
success: http://build.servo.org/builders/mac1/builds/1473
success: http://build.servo.org/builders/mac2/builds/1476
success: http://build.servo.org/builders/gonk/builds/439


4.Using The $http Service In AngularJS To Make AJAX Requests
http://www.bennadel.com/blog/2612-using-the-http-service-in-angularjs-to-make-ajax-requests.htm


Isn't the point of making a public release, so that the public can use it? What I get from this is that you have your own way of running a .cia, and this *private* way is the only way sysUpdater will work.


@normanmaurer - So are we expecting a compile error with openssl < 1.0.2?


the `.sample` seems more standard


Good idea. We would need to do is generically, i.e. by introducing some new configuration options to list pages (using regexes) that should have some specific syntaxes.


see https://ci.owncloud.org/job/server-master-linux-encryption-2.0/21/database=sqlite,label=SLAVE/#showFailuresLink


On 02/04/2015 14:45, jidur wrote:
>
> L261 study missing a 't'
>

Fixed. Sorry... Marina
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mgcam/npg_seq_melt/commit/0e93fe6a6cd9287559c8bc34ce5f495d91600247#commitcomment-10546200>.
>

--
 The Wellcome Trust Sanger Institute is operated by Genome Research
 Limited, a charity registered in England with number 1021457 and a
 company registered in England with number 2742969, whose registered
 office is 215 Euston Road, London, NW1 2BE.


fast-forwarding master to auto = d999fb9d


Should this bugfix have a History.md update?


Should this new feature have a History.md update and/or a doc update?


Should use GetCharmerOrOwnerPlayerOrPlayerItself() instead


Should this new feature have a History.md update?


I also think we should not install the whole TLS stuff by default


nope... it will just compile and works when 1.0.2 is a shared lib, if not it will not work but the rest.


For some reason I though that instr() returns 0 when it finds the substring on the beginning of string. But in fact, 0 is returned when substring is NOT found. So yeah, this is wrong. -.-


@mitio да, в началото ще изглеждаме като неандерталци :)


Done, thank you.


Yes, if it's assets have conditions we shouldn't merge them (but we can minify them). I can't see any reason why we can't make assets any asset kind...


LGTM


saw approval from Manishearth
at https://github.com/jgraham/servo/commit/d6a608806e84ea756eac171312adc2d08a0e30e8


merging jgraham/servo/css_tests_1 = d6a60880 into auto


When I change something simple like the link port name (in simulation/orogen/rock-gazebo),
    RBSOutPort* link_out_port = new RBSOutPort( (*it).source_frame + "2" + (*it).target_frame );
Do I need to open a PR just for this ? Can I push to the master ?


@JeffreyWay Why can't we have `.sass()` with no parameters? Now getting the error `[TypeError: Object resources/assets/sass/**/*.+(sass|scss) has no method 'join']` Previously it worked fine... Even if for whatever reason a parameter is now required, why can't we do it like with CoffeeScript compilation? `mix.coffee('resources/assets/coffee/**/*.coffee')` Doing `mix.sass('resources/assets/sass/**/*.scss')` throws an error.

What's more, putting in parameters like .sass(['vendor/','app.scss']) doesn't give the expected result. The app.scss gets compiled to public/css/app.css OK but then vendor/foo.scss gets compiled to public/css/foo.css as well!?! What happened to the folder structure?!

And the cherry on the cake is that chaining this "new" .sass() with .copy() straight after to get the misplaced vendor files into the /css/vendor folder DOESN'T WORK. The public/css/foo.css file doesn't move.


Why are we doing all unreg should we just change the nodes with the pid equal to admin?
UPDATE `node` SET pid = 'default' WHERE status = 'unreg' and pid = 'admin';


merging jgraham/servo/css_tests_1 = d6a60880 into auto failed


Hmm, we could set up a redirect to strip urls that end with `.md`


Не съм съгласен :) Справяш се страхотно :clap: Много ти се радваме тук всички :)

Пращай на имейл каквито и въпроси да имаш по GitHub.


The release is about this tool, not private stuff.
It's not the only way. Gateway just does it wrong. I suggest asking them to do it right.


No need to specify the default, as the --help output will mention that


Mettre le lien vers la documentation *française* de la version *1.7* sera mieux ! ;) https://docs.djangoproject.com/fr/1.7/ref/templates/api/#subclassing-context-requestcontext


Pareil ici, en *français* et pour la *1.7* : https://docs.djangoproject.com/fr/1.7/ref/validators/


Thanks @sp4ke! We'll take a look.


The listener composite is wired as component
so you don't need to create it in any case

https://github.com/ethereum/ethereumj/blob/0e2720894b6d5126e41c3dbdd8f37c8ec5ab29e1/ethereumj-core/src/main/java/org/ethereum/listener/CompositeEthereumListener.java#L19


"nécessaires" et pas "nécéssaires"

> Dans celui-ci, principalement de nouvelles vues (``api/views.py``), URLs (``api/urls.py``) et tests (``api/tests.py``).

Il n'y a pas de verbe conjugué donc c'est un peu bizarre, non ? Je verrai un truc du genre "Dans celui-ci, il y a principalement".


@vladichimescu
De ce nu mergea fara sa specifici si care e diferenta dintre ExternalCookie si ApplicationCookie?


@Minecrell  @simon816 This was true in 1.7 but is no longer the case in 1.8. The Integrated server is created on the client within the Minecraft constructor and is always available.


Woot!


It actually should be the other way around. The following tests are all testing the `puts` method. But there should be a separate test for testing the `print` method. So better not rename this section, but add the above test as test for `print` (and change it to actually test `print`, not `puts`).


I assume that all references to those files were removed from the code, so it's a cleaning phase? If yes, then LGTM. :shipit:


Yeah, the action bar didn't immediately do what I wanted until I switched to the NavDrawer, I'm sure its an easy fix control the action bar without navdrawer.

I agree that an activity that's reached via settings isn't the best place to have a navdrawer.

When I got this notice I thought it was via the google groups, and hoped this meant you read the xml (nav drawer is at the bottom of the xml i posted).

I think it would be really good if we had a shared strategy for layouts, padding,  dimens. etc.  _Not_ another layer of prerequisites for merging, that would be annoying and unnecessarily discourage contribution,   but an end goal which we can use to polish things when appropriate.    For example, I'm using lots of numerical padding values in the xml instead of dimens.  That's not optimal, but nor is having people like me create dimen values willy nilly just to use them on principle.   Ideally we would use a common set of padding dimens  that  make sense for the majority of layouts.  That's just one example, I'm sure there's other ways we could benefit from having a shared understanding while writing the xml layout files.


What's wrong with `(char)('0' + i)`?


Are you sure about this change? ;-)


It shouldn't have made it through sorry
I will revert tonight
feel free to do it before


There's an assumption here that the line is quoted with double quotes. It can actually be quoted with anything sh accepts, i.e., also with single quotes or not at all.


I'm a bit confused as to the use of inline assembly here...what is this buying us over just `exit 0`?


Does anyone know if it is possible to export any data except what is in the ticket_cdata table?


Is this even defined behavior?


The ``OpportunityId`` should be passed as parameter, `$scope` shouldn't actually go to a service.


:O "40" instead of "30"


Please convert tabs to spaces


The `controller` is modular now.


The `service` is modular now.


Using the module `active_support/inflector` this can be done simpler:

```ruby
"Upgraded #{migrations_done} #{"system description".pluralize(migrations_done)}"
```


I like the simplicity of this. But whatever.

On Thu, Apr 2, 2015 at 8:00 AM, Michele Locati <notifications@github.com>
wrote:

> Yes, if it's assets have conditions we shouldn't merge them (but we can
> minify them). I can't see any reason why we can't make assets any asset
> kind...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/concrete5/concrete5-5.7.0/commit/0971447ce11de35affd5c544cf6c2034f86f9715#commitcomment-10547719>
> .
>


There we go! Be sure to commit each change.


Simply call/include the `controller` and `service`, but the `app.js` should always come first.


Make sure you continue to fight for Windows-idiomatic stuff :-)


@YuryBY
 The implementation of .quoteDateLiteral() in the parent class does some extra validation and massaging of the value passed in- i.e. it checks whether it's a valid Date or Timestamp value and assures yyyy-mm-dd.  Is there a reason not to maintain consistency with that logic?  I believe Sybase doesn't support the DATE keyword, and your change correctly omits it, but it would be good to still keep consistency with how we construct the date value across other dialects.


Thx, works fine on openembedded/yocto


Please change it to something more C++ friendly. @josephwilk and I both mentioned that this was pretty nasty and we decided to pull it out to a sep fn so someone could easily slip in a more idiomatic solution...


Actually no, we're keeping it this way.  We can only register one kind of
type/handle combination. So if I register respond.js as having ie
conditional stuff and it's 'javascript' 'respond' – no one else will be
able to use respond without the conditional. In this case, we have a
javascript conditional respond and a traditional respond.

On Thu, Apr 2, 2015 at 8:14 AM, Andrew Embler <andrew@concrete5.org> wrote:

> I like the simplicity of this. But whatever.
>
> On Thu, Apr 2, 2015 at 8:00 AM, Michele Locati <notifications@github.com>
> wrote:
>
>> Yes, if it's assets have conditions we shouldn't merge them (but we can
>> minify them). I can't see any reason why we can't make assets any asset
>> kind...
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/concrete5/concrete5-5.7.0/commit/0971447ce11de35affd5c544cf6c2034f86f9715#commitcomment-10547719>
>> .
>>
>
>


Add comment explaining why this is a hack.


There is. I'll make some other change and push it up again...


It's not meant as a replacement (for us), just a little joke with a person that likes peppers. I will switch back tomorrow.


make the mapped here true, regardless add_keybind_mapping returns true or false. The reason we want this is to have consistent behavior. If keybinding exists in configuration, we will never map defaults for it, even if it never gets mapped due to conflict. Only map defaults when it does not exist in configuration.


the only reason is that there is no c library installed so it is compiled with ```-nostdlib```.
If I try to use ```exit``` (or even ```main```) the linker fails.


You don't have to specify the changed filename since GIT always tracks.


Good find. Surprised that rubocop didn't find that.


Instead, please mention the reason why you would like to change the priority.


Forgot to say in commit message: This commit also adds a `Fresh` statement to create variables.


może boolean?


you can use !mapped && syntax && syntax[i] to avoid identation.


Does it make sense?


Na moje bez sensu :D


You should mention why you changed the data type.


This string should be marked for translation (keeping in mind you're doing multiple substitutions into it, so you'll have to use a dict) and then pyanaconda/ui/lib/disks.py needs to get added to po/POTFILES.in.


LOL


thanks for pointing it


Powinno być

```java
public static String setHex
```

```java
return Util.setHEX(text);
```

bo na razie to nic nie robi :D


`boolean` & return


Let's say that the first time through this autorun, the render function invalidates the computation, so we don't call destroyMembers here.  And let's say that the autorun then gets **stopped** before it gets a chance to rerun.  It seems to me that `destroyMembers` will never get called on the DOMRange.

Now I am not an expert on what a DOMRange is or how important `destroyMembers` is.  Is this a potentially problematic resource leak?


in that specific case, the definition of that 'NullDryRunArguments' is precisely to have all its fields set to null (which indicates that this is a dryRun with targetDate and not dryRun with subscription info


Sprawdzaj czy wysłano z konsoli


Tu także

> Sprawdzaj czy wysłano z konsoli


Także i tu

> Sprawdzaj czy wysłano z konsoli


Yes, it's done in the setSpatialObjectsDisplayNode already.
And setSpatialObjectsDisplayNode called in this function.
So it updates it one extra time.


should all the comments be intended 1 more space?


SHADOWFILE is None for DarwinUser so it wont work.


it's done!


Good point :)


Hmm ... the old library can still be enforced by setting constant AUTOPTIMIZE_LEGACY_MINIFIERS to true in wp-config.php (this is done in autoptimize.php), so we can't just remove that call. I'm not even sure the old library expects that 2nd parameter?


Ok will fix.


Heh, ogólnie API robiłem na superszybko, żeby je ogarnąć już początkowo. w 0.5 dodam dużo więcej opcji związanych z samym pluginem.


This fix an issue which exists in 2.54, and makes request.post to throw an error.
It would be great to have a 2.54.n to fix this.
Actually, I need to go back to 2.53


@maikany


LGTM :shipit:


If you wrap this in a translation call - `<%= _("Submit Search") %>`, it'll appear in the language the site is being rendered in (once translators have translated it).


it should not be calling this function anyways, it should be overridden by the darwin class


This should stay here! Clearly that's not related to the code you are refactoring.


It's a matter of definition, i can change it if you want.


Funny, i think this is the right thing to do, and yet i am very preoccupied of starting to have code that needs to do special things for various versions..


Excellent. This allows you to pass req in the callback. A useful feature and supported by Passport.

A TODO for me is to update the sample app for this and tell the developer to set `passReqToCallback: true` in their code for it to work, however it's well documented in Passport's docs so a developer should be able to find it.

Those interested in what this allows can visit [here](http://passportjs.org/guide/authorize/)


Looking on my phone. Will give this a more thorough look over the weekend. Obvious question: you added an ISignal interface. Should we hav an IPromise?


:+1: Pomiędzy 1.7.10, a 1.8 nie ma zmian w API. Dodali tylko API tytułów, bordera, itd., itp.


@rehia the new release with this fix is coming soon.


Ok hmm...I see.  So we're avoiding any use of other RPMs.  I see how that's convenient but OTOH if we're not processing real world RPMs we're not testing as much as we could.

So, for now I guess just keep the inline assembly and down the line we can consider a test framework which uses real-world actual distribution content.


Missing semi-colons here.
It's fine not to have them in these examples, but be consistent throughout to avoid potential confusion.


I assume that `WikiaPageHeader` class is left for some compatibility purposes, is it? If yes, then maybe add a `TODO` (in such `<? //TODO ?>` fashion, to not render it to user) to remove it *someday*.


we're going to move to lower case (like IBM had us do) for phrases that you follow with an initialization


Wow, you've done a lot on this, you're gonna have to teach me when we both have time haha.


LGTM


Minor comment and LGTM. :hamster:


There are currently no instances of `_` as a variable in the code base. (though there are a couple in the tests)


interesting


change "may" to "can" on both lines above


I'm working on it


@adrianbk should now be fixed. However the demo still has some issues resolving the ${property} in the swagger-ui


change colon at end of phrase to period --> colons).


oh yes, I have done that already. Not committed yet.

Will be `@desc("Checks if the array is instantiated using [], not with new")`


change may to can, change colon to period


may --> can


My last comment here referred to the problems we ran into yesterday. This was just for your information. It turns out naailk had sent the text from the errors in his terminal in an email. So I thought I'll let you know as you'd asked in the beginning of this thread for the errors returned.


OK thanks, is the issue resolved?


We need one Parser for each HTML version: one for HTML 4.x, one for HTML 5.0 and one for XHTML 1.0. For example if there's ```<aside>```in XHTML 1.0 it should be ignored since it's not valid in XHTML 1.0. But if the parser is HTML 5 then it should not be ignored since it's valid in HTML 5.


:+1: on that note!
if you have time for some of the little nits, it will mean that much less when we get to the globalization tasks...


Is there a typo here? Same condition's repeated.


shouldn't order be after '@include' ? also l. 203- 206?


Is there a reason for creating a new counter variable, when `j` already works for that and was already being used before? If there isn't, then it's best to use `j`, since we won't need code to define it or update it ourselves.


这句原文：Enable data to contain multi value tags, numbers, and full text.
感觉这样翻译有些让人无法理解，但我还不知道该怎么翻，等了解后面的内容，在看看...


i think this url is no longer being used. the [github repo](https://github.com/apache/cordova-plugin-device) is mirroring here: git://git.apache.org/cordova-plugin-device.git

getting a time out error when trying to install


It may be of interest to look into the [JSON-LD Compacted Document Form](http://www.w3.org/TR/json-ld/#compacted-document-form) and perhaps use the [ruby-rdf/json-ld feature](https://github.com/ruby-rdf/json-ld/#compact-a-document) to perform compaction. On the other hand, what you have here might be better with regards to performance.


Thanks very much for sharing this information. This is the document for the
bee I am trying to make work with the Sodaq Mbili:
http://wiki.iteadstudio.com/Wee_Serial_WIFI_Module. I am trying to connect
via Serial1, but no success as yet.

I can access and configure the bee via UARTsBee with a few lines of Lua code

On Thu, Apr 2, 2015 at 3:02 PM, Alexey Andreyev <notifications@github.com>
wrote:

> You're welcome :)
> I guess you should chenk bee pinout. I don't have the Mbili board so
> didn't really understood how you connecting the esp8266.
> My esp8266 module look like this:
> [image: ScreenShot]
> <https://camo.githubusercontent.com/f915f5b2b3a3a419f588d967cb2260306faa629d/687474703a2f2f63646e2e626f787465632e63682f696d616765732f33313038332e6a7067>
> so I connecting it to my arduino without the bee socket according to this
> pinout:
> [image: ScreenShot]
> <https://camo.githubusercontent.com/428f5f864fbd8382464baddc5aa991e386cbabff/687474703a2f2f706c617967726f756e642e626f787465632e63682f6c69622f6578652f66657463682e7068702f776972656c6573732f657370383236362d70696e6f75745f657463685f636f707065725f746f702e706e67>
>
>  esp8266 RX --> to Arduino Hardware Serial TX (but not for the Serial1,
> that used for the flashing the device)
> esp8266 TX --> to the same Arduino Hardware Serial RX
> esp8266 GND --> to the Arduino ground
> esp8266 VCC --> to the Arduino 3.3V Power
> esp8266 CH_PD --> to the Arduino 3.3V Power too
> Other ports are not using.
>
>  In my code I'm using something like that to restart and setup the device:
> directly in the my .ino:
>
> ESP8266 esp8266(Serial3,BAUDRATE); // creating new ESP8266 object and connecting in to the 3rd Hardware Serial in my Arduino MEGA
>
> later in the setup method:
>
> Serial.begin(9600); // Serial debug
> Serial.println("Serial debug working");
> Serial3.begin(9600); // Initializing the 3rd Hardware Serial which is used to talking with the ESP8266 module
>
> while(1) // trying to restart and "kicking" the esp8266 module if restart fails
>     {
>         Serial.print("restaring esp8266...");
>         if (esp8266.restart())
>         {
>             Serial.print("ok\r\n");
>             break;
>         }
>         else
>         {
>             Serial.print("not ok...\r\n");
>             Serial.print("Trying to kick...");
>             while(1) {
>                 if (esp8266.kick())
>                 {
>                     Serial.print("ok\r\n");
>                     break;
>                 }
>                 else
>                 {
>                     Serial.print("not ok... Wait 5 sec and retry...\r\n");
>                     delay(5000);
>                 }
>             }
>         }
>     }
>
>     Serial.print("setup begin\r\n");
>
>     Serial.print("FW Version:");
>     Serial.println(esp8266.getVersion().c_str());
>
> At my linux-based machine I prefer to use minicom
> <http://en.wikipedia.org/wiki/Minicom> software to watch the Arduino
> Serail output.
> But I heard about the CoolTerm <http://freeware.the-meiers.org/> as for
> popular software that is avaliable for Mac OS/Linux/Windows.
>
> You can also try to connect the ESP8266 through some usb-ttl converters
> without the Mbili board to send AT commands directly from your computer via
> terminal to make sure that esp8266 is alive according to the ESP8266
> Datasheet AT Instruction Set.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itead/ITEADLIB_Arduino_WeeESP8266/commit/712e1e3a95f0980d4c8765d3ecf04768a89e7d57#commitcomment-10545286>
> .
>

--

This email and any files sent with it are confidential and intended only
for the person it is addressed to. If you have received this message in
error please let us know immediately and delete it. The content of this
message belongs to the sender and the views expressed do not necessarily
reflect the views of United Bible Societies.

United Bible Societies Association, a company limited by guarantee.
Registered in England and Wales No 2264875. Registered Charity No 800058
Registered Office: Stonehill Green, Westlea, Swindon, Wiltshire, SN5 7PJ,
England.


The NFS test succeed for me with your cleanups.


This is incorrect test: this is not left recursion, because `!''` not match anything. Current online parser not freeze on this.


This still has wrong indentation, it should be on the same level as the variable declarations above it.


detail- also about the order. can we sort also mixin declarations?


Ah, my bad. Was reading the dca file just before and mixed up the namespace. Sorry.


You should write 10-20 e2e tests for peer review. (Havannah: 8)
You should also report bugs or explain the failed test either in the commit description or in the codes. (Havannah)


This test is superfluous: this is same, as `start = "a" start`


Let's keep it simple. Always throw as is, but then use (Annex)CommandError instead of RuntimeError, attach exitcode and never return status at all, since it will always be zero. Agree?


Krzysztof Tadeusz Pieton live in 139 Church Hill Tullamore Offaly Ireland


Да, это задача все еще за мной.


We don't static import generic method names like `valueOf` since the name isn't descriptive when reading the code.


No,it's not done in the widget: https://github.com/commontk/CTK/blob/master/Libs/Visualization/VTK/Widgets/ctkVTKDataSetArrayComboBox.cpp#L178
I took that piece of code from Slicer: https://github.com/Slicer/Slicer/blob/master/Modules/Loadable/Models/Widgets/qMRMLModelDisplayNodeWidget.cxx#L540-549
I imagine we do this comparison to avoid firing unnecessary events.


I discovered last night with 5.1 how touch works and how it can emulate the mouse.  So now I see Boop when I touch and it follows my finger around if I start to drag.  This is good because it means you can make something that uses the mouse and can use it a desktop PC and iPhone.

```C++
    //Prepare to receive mouse events
al_install_mouse();
        al_install_touch_input();
        //al_set_mouse_emulation_mode(ALLEGRO_MOUSE_EMULATION_5_0_x);
        al_set_mouse_emulation_mode(ALLEGRO_MOUSE_EMULATION_TRANSPARENT);
        al_register_event_source(eq, al_get_touch_input_mouse_emulation_event_source());
```

I am not sure of difference in the various MOUSE_EMULATION modes yet, but transparent seemed nice.  Somehow ALLEGREO_MOUSE_EMULATION_5_0_x is supposed to be compatible with 5.0 version, which is what you are running.

This was a useful forum article that pointed me in the correct direction:
https://www.allegro.cc/forums/thread/608343/931092


@alexcdahl Looks like you fixed it! What was the issue?


Nine Men's Morris:

Good job :)


Forgot to add the patch file to the commit. I will push the file when I'm near my computer again.


Banqi:

Good job :)


The setCurrentIndex would do the comparison, but fair enough.


Barricade:

You should write 10-20 e2e tests.
The tests do not pass 100%. (3 tests, 856 assertions, 1 failure)


DesignDaysGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


/CC @Jellyfrog: I went with this approach and updated Apache to not redirect `/rss.xml` to `https` in order to avoid mixed content. If you agree, I'll merge this. The server has the files from this branch at the moment.


What about "confirm changes to this tiddler".  Is impossible to misunderstand that


You beat me I was coding it on eclipse sorry I got busy all of a sudden, my code looked different then yours tho haha


Méthode*
Les titres commencent souvent par des majuscules en français.


I reverted this change locally; it was a leftover.


Hai Daniel,

I wasn't aware that somebody is actually using `esmisc` (otherwise I would have implemented an informative `Deprecated` statement) - I'm pleased :)

Regarding your questions:
1) Jupp, `esmisc:::cactus()` is now `webchem:::cir_query()`. I moved to webchem, because I collected some functions over time and though I may be helpful for others and fits to the @ropensci project.

Development for chemistry API will be only in `webchem` and in future I will also move the remainding tox-parts (after some overhaul) to webchem.
Moreover, I plan to add more tox-APIs (Toxnet, echem, ...) and some other chemical APIs (e.g. physprop). - But only if time permits (which is currently NOT the case :(( ). [1]

2)
> Just wanted to add a pull request for cactus().

Would be happy if you contribute to webchem (will be acknowledged!). Feel free to contribute to `webchem` - code is similar.  What where you improving?

3)
> If so, what is the reason for removing the possibility to have length(query) > 1?

Convenience - No ;)

I didn't want to hassle with multiple return values (and the return values in general) and the user should decide.

How to deal with multiple values? There are many possibilities:
1) return all
2) return only first
3) return by user input

We have covered all these in [taxize](https://github.com/ropensci/taxize), but it blows up code for little gain.

Moreover, this avoids an additional dependency (most likely on `plyr`) and the user can decide which framework to use (base, plyr, dplyr, data.table, etc...).

Running `webchem` functions in a loop is fairly easy and all functions should have examples for that [2] :
```r
require(webchem)
compounds <- c('Triclosan', 'Formaldehyde')

# query Triclosan, return all results
cir_query(identifier = compounds[1], representation = 'cas')
# query Triclosan, return first result
cir_query(identifier = compounds[1], representation = 'cas', first = TRUE)

# query all compounds, return first hit (-> named vector)
sapply(compounds,
       function(x) cir_query(identifier = x , representation = 'cas', first = TRUE))
# query all compounds, return all hits (-> named list)
sapply(compounds,
       function(x) cir_query(identifier = x , representation = 'cas'))

# format as data.frame
ll <- sapply(compounds,
             function(x) cir_query(identifier = x , representation = 'cas'))
data.frame(compound = rep(names(ll), sapply(ll, length)),
           cas = unlist(ll))
```

### TODO
- [ ] [1] Add aTODO-list / Issues to webchem
- [ ] [2] check `if (is.true())` and add!


I am sorry, messed this up on copy from mac to Linux system. Our cross-compile system is not ideal for development on libs.


I'm keeping it until I find a better hat SVG.


@khoude24 Pourquoi `itemprop` n'est pas sur le `<a>` ?


update #1


Rock. We already had a unit test for the issue, it also turns out we had a `getLength` helper too.


Connect 6:

Good Job :)


@millerdev i'm using the defaults argument on `CaseFactory` to assume the same owner throughout the tests.

`Relationship` is always directional and represents what we commonly call parents. The `relationship_type` field is generally quite unimportant, so I don't think we need to do anything special to call attention to specific ones.

Agree the name of `CaseStructure` could be better, though not sure what I would choose instead.

Would be helpful in the future if you make feedback on the relevant PR - this one is really just for the ownership stuff so probably not the best place to discuss changes to `CaseFactory` etc.


Chess:

Good Job :)


You just need consider two pair. That is enough.


@khoude24 Enlève "- Ellipse Synergie" de la description par défaut svp. Peux-tu aussi ajouter le `<meta name="author">` ?


Filenames should not contain colons (battery-3:4.svg), it causes problems in Windows.


Tu jest jakaś mega zmiana. To są nasze customizacje?


AMDG

On 04/02/2015 07:46 AM, Marcel Raad wrote:
> Looks like this commit broke test_add, test_callable, test_negate, test_nested, test_reference, test_relaxed, and test_tuple on all compilers in in C++11 mode:
> http://www.boost.org/development/tests/develop/developer/type_erasure.html
>

Fixed.  Thanks for the report.

In Christ,
Steven Watanabe


@DanBadea
Avea un comportament awkward in sensul in care daca verificai cu IsAuthenticated dadea true si totusi autentificarea nu era facuta deci nu puteai accessa chestii


With WebJars the dependencies should be specified as build dependencies (in the pom.xml).  Loading them can be done manually or with something like RequireJS.  Feel free to send a PR to add the required dependencies and I can do another release.


Hm, I've heard it can have a large effect, but in our case not as much, because we use average data.


I'm not sure this is the right place to ask but i'll try it anyways since I see a mention of it in this commit.

I'm looking for heads up on how to get this compiled (and thus, running) under OSX. Any hint ?


发现并没有帮上什么忙![oi9opr5 916 1 w my5f5n](https://cloud.githubusercontent.com/assets/10368307/6967604/f61d5120-d992-11e4-927c-2d33fc2c8beb.png)


Actually it's `k` here, last attribute in mixin should be the one without the vendor prefix. The only thing that could be changed is the additional spaces for the prefixless attributes. :ram:


Hmm, tu też dziwnie, niby ta sama wersja, ale zawartości tych plików wskazywałyby na duże zmiany i to raczej nie naszy. Dlaczego tak?


>As summed up here, I discovered that window functions can indeed have a noticeable affect on image quality, in particular when it comes to ringing; and I feel it's worth keeping them around for this reason. If possible, I'd prefer to clean up the code instead of removing the feature altogether.

I do think the deeper nesting and the need to set a default window function makes things more complicated (and there's some code duplication), but ok then. In general, such specialized things and not-really-commonly-used things are what cause bugs and maintenance burden, though. So I'd be careful.

If I tried this, I'd probably add a window callback to filter_kernel for the window function, and hardcode some combinations which I thought would make sense as entries in mp_filter_kernels. Not sure if that would make sense.

>For what it's worth, I also apologize for my wording and behavior - and I don't even wish to pretend that I was not doing the same things myself that I was accusing you of.

And I apologize again for being a grumpy asshole.

>My feelings on this whole topic are somewhat mixed. Part of the reason I may have seemed excessively “glad” to “fork” the project is mostly a thing of personal freedom - I felt like having to constantly “explain” my decisions, motivations and reasoning, which often devolved into arguments - in order to “justify” them somehow. On the one hand, it does provoke a healthy bit of skepticism and thinking about whether something is really worth it or not. But on the other hand, I felt like it was a chore at times - like “oh, I have this great (for me) feature, but I have to convince wm4 that it's worth it.”

They say a maintainer's job is to say no. I also think that explaining changes or justifying features is a good thing, because it requires you to reflect whether a change is really going into the right direction, or whether a feature is really needed. Personally I'm trying to do this by doing the explaining/justifying in commit messages (and it often happens that I just go back to improve the change, or even discard it).

Strict patch review isn't unusual in other projects either. If you send a patch to Libav or FFmpeg, you'll of course have to explain why you want something that doesn't seem immediately useful or obvious to others. It's nothing unusual that a patchset is reposted over and over, with small changes each.

On the other hand, it's bad because can slow you down. Having to make multiple amends can be annoying to the point you just won't bother. It requires a balance, but it's quite possible that I'm not very good at this. I know that you've "bothered" me many times about features until I "gave in", which must have been very annoying to both of us. Well, what can I say - from my perspective it'd be best if there was a third party, which can overrule me if I make the wrong decision. Instead of waiting and "working" me until I'm convinced. Sorry for being such a shitty maintainer.

>So this has honestly been somewhat of a liberation for me, for better or for worse, and I didn't want to give up that perceived freedom. And honestly, I still don't - I enjoy being able to take things into the direction I want to take them to without having to agree or disagree about specifics. It's part of the reason why I love free software so much - if I want to do something my own way, I can.

Everyone wants this, including myself. It's true that I might be a bit selfish about this.

Though I don't really think this is the main issue. For now we've always agreed about the approximate direction. We agreed that vo_opengl had to be rewritten, we agreed on how to do it, we did it pretty much together, and in the end we were both happy with the result. Can't be too hard for us. Also, compromises are always needed when multiple people work on the same thing.

>I'm not sure what path is the best for me and for you both, though. There's a big benefit to merging changes back into master and keeping code synchronized, and for that to become a reality would basically mean going back to the old status quo. Is it egoistic of me to want some amount of independence? On the other hand, I don't know if independence can be taken far - subtle differences in code base could rapidly cause an exponential divergence that only causes unnecessary work for everybody involved.

I don't think maintaining two codebases is a good idea. Especially if there's no communication between them, like it was shaping up in the last days. We should stop this.

Now, didn't everything you made go into mpv git master anyway? I think the main thing you want is getting experimental changes quicker to users who are interested in testing such things, and who will give feedback. There's nothing wrong with that, though it wouldn't be a "real" fork. (Maybe it could even be just a branch on the main git repo. We could make it possible to select the branch with mpv-build and have lachs0r make builds etc.)

I was also fine with you having direct push access to master. I removed it because I in my opinion you pushed something that was premature, full of problems (not necessarily your fault, but because the rest of mpv doesn't work (yet) as you needed it), and not approved. On the other hand, this was probably my fault too, because I never communicated any rules about it to you.

(The rules are: bug fixes and other small changes can be pushed freely, but intrusive changes or things which affect other areas of the player should be reviewed first. Maybe user-visible changes too. If someone wants to require me to put up pull requests for review and reviews them quickly enough, I'd be fine with this too, btw.)

Honestly, I'd prefer not to review every little thing, and I'd be fine with giving you back push access, as long as we don't clash and explode immediately again.

>So on this basis alone, I'm more than comfortable with being forced to keep things that work the way I expect them to in my own personal fork, and I don't wish to claim to be in the “right” about everything, either. The question, I think, is how much we are willing to settle on a possible common viewpoint when it comes to mpv, which as much a question of the differences in our viewpoints as it is a question about the needs of mpv's users. I'll withhold any judgment on that matter, mostly since I (unfortunately?) don't care about end users.

While I very much like to have my own opinion about what's broken (such as ASS subtitles trying to match the video colors, which now has to be "emulated" by declaring ASS colors not being sRGB, but some bastardized video-dependent almost-RGB thing, even though they definitely are sRGB when using the colorpicker in Aegisub), and while I'm defending my opinion, I don't always absolutely insist on having my way. Especially if it has a destructive influence. Yes, I can make compromises, even if I might be grumpy about it. (Again, sorry for being an asshole, I guess I'm just one.)

So, I don't know in what direction you want to go from now on. At least I hope I'm not obstructing the direction into not-fork unnecessarily and that I'm acting property in the interested of this project, the community, and global peace without sugarcoating my views too much.


We can take this a step further:
```java
return ...
.map(new FileStatus(...))
.map(this::createLocatedFileStatus)
.collect(toList())
.iterator();


> Would be helpful in the future if you make feedback on the relevant PR

Yeah, I tried but couldn't find it. Which PR has this commit?


Oh, I didn't know about this! Thanks @literarymachine.

@infovore, @frankieroberto: it's probably sensible to use an existing standard for this rather than the ad-hoc thing I bodged together. I'll try to knock up a script that does it.


看到不满意的地方提出来或者直接改掉都可以的w


Nit: f and p are unused?

Nit: This seems copy/pasted? You're talking about images in this test, not scripts.


@RadoslavMilanov Не съм сигурен, че "source code" има мн. ч. По-скоро е небройно. Еквивалентно е на "Кодът на проектите, платени с публични средства, се публикува тук." На български също казваме "код", а не "кодове". "Кодове" е неправилно в контекст на изходен (source) код.

Референции:

1. Моят British English речник, който твърди, че "source code" в контекста на computing е "mass noun".
2. http://english.stackexchange.com/a/20456


DesignDaysGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


`f()` if no error is thrown?


@tinevez Completely wrong. If the modifications are made to the source graph, the listener won't be notified of course. Arrgh. Replace the constructor.


r+


Nit: Might be worth logging something useful in the message so that the `assert_equals` will give a useful error message.


I would also the way you suggest, @dianafa, but `scss-lint -c ../guidelines/SCSS/.scss-lint.yml skins/oasis/css/core/PageHeader.scss` doesn't bother actually.


r=jdm


Halatafl:

Your tests do not pass 100%. (12 tests, 57 assertions, 14 failures)
You should write 10-20 e2e tests for peer review.
You should also report bugs or explain the failed test either in the commit description or in the codes. (Risk)

One advice: You should not use the same description for all of your commits. It makes me difficult to find the exact commit for a specific homework. For good practice, you should give a more specific description, at least for some major commits.


Actually thinking a little bit more about this you're probably right. If someone needs to undo conditionality for an asset they can do that by updating the record. We will fold the functionality into the base asset class

Sent from my iPhone

> On Apr 2, 2015, at 8:16 AM, Andrew Embler <andrew@concrete5.org> wrote:
>
> Actually no, we're keeping it this way.  We can only register one kind of type/handle combination. So if I register respond.js as having ie conditional stuff and it's 'javascript' 'respond' – no one else will be able to use respond without the conditional. In this case, we have a javascript conditional respond and a traditional respond.
>
>> On Thu, Apr 2, 2015 at 8:14 AM, Andrew Embler <andrew@concrete5.org> wrote:
>> I like the simplicity of this. But whatever.
>>
>>> On Thu, Apr 2, 2015 at 8:00 AM, Michele Locati <notifications@github.com> wrote:
>>> Yes, if it's assets have conditions we shouldn't merge them (but we can minify them). I can't see any reason why we can't make assets any asset kind...
>>>
>>> —
>>> Reply to this email directly or view it on GitHub.
>>>
>


@tomstuart aha, that looks ideal; as ever, the trick is knowing the name of the thing you mean to do.


These tests LGTM % a few nits.


Rummikub:

You should report bugs or explain the failed tests either in the commit description or in the codes. (Halatafl & BackGammon)


(Do they pass anywhere? :) )


Yeah, otherwise I think it's difficult to see how/where the line match up.


I did a test (added line 1250) but now payment option is empty :(


saw approval from jdm
at https://github.com/pcwalton/servo/commit/1fe55a27b1e4106470bd77fdfbd28ebfd95d27b4


merging pcwalton/servo/mouse-over-incremental-reflow = 1fe55a27 into auto


pcwalton/servo/mouse-over-incremental-reflow = 1fe55a27 merged ok, testing candidate = 5457ec60


@danielo515 that's pretty good. Let's continue over at the original discussion: #1274


Alternatively, how about just adding `FUNC_ATTR_NONNUL_ARG(1)`? (Or `_ALL` if `hash` can't be `NULL`.)


Havannah:

your tests do not pass 100%. (9 tests, 347 assertions, 8 failures)
You should write 10-20 e2e tests, both for your game and peer review.
You should report bugs or explain the failed test either in the commit description or in the codes. (Blokus)


That already happens because the script would execute.


Never check mySubs.length to do this.  Since it's a callback, the list can (and often is) empty when modules load.  Just use isMod.  Also, there should be a self.settings('enabled') check here.


@psychob Ideowo proponowałem właśnie słownik, ale już nie wiedziałem jak to zrobić w cpp ;)

http://www.wykop.pl/wpis/12017143/#comment-36983659


Backgammon:

The tests do not pass 100%. (1 test, 1 assertion, 1 failure)
You should write 10-20 e2e tests.
You should report bugs or explain the failed test either in the commit description or in the codes. (Rummikub)


A lot of them pass in IE11, but not all, since they are also doing some silly things.


rax retry tests


Yeah oops. There is probably some more general code that could be written, but it might make it all a lot less understandable.


autopsychorhythmia ...
Wrong spelling...


@Underground78 @kasper93: I think things are stable now. The issue was caused by a bug in mod_spdy. I have disabled SPDY because the module was getting old plus has this bug. There's a workaround, but I have SPDY disabled for the time being.


А разве он не будет null и без этого присвоения?


I do not run into the case described. But this can't stay in production code, should at least specify `module=...`


@strangerstudios the *has* sorta makes it sound like a boolean return variable, though. What about just "pmpro_membership_access_post" ?


Nice catch, please fix @whatasunnyday