@staabm the ProcessHelper is here to give debugging output for the build process (getting composer dependencies).
However, for the output of the script itself, we want the output to be displayed all the time (not only in verbose mode) and without the extra debug formatting.


rollup


Not really, had a [bug with that kind of binding](https://github.com/CartoDB/cartodb/blob/master/lib/assets/javascripts/cartodb/new_common/views/pagination/view.js#L27-L29), have not had time or managed to encounter the culprit just yet.. meanwhile I've been using this kind of native binding, [works in by the browsers we support](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function/bind#Browser_compatibility).


It is additional fix as getLoaderType never can return empty string and if it is fixed, then "none" bootlaoder is not error, but just warning, that such option is only for experts. I found it during debugging, when I try to write tests.


Because some code don't need the be wrapped.
I mean, the output of `compose update` could be displayed
and the output of the user script did not.


Actually, thinking about it some more, for point 3) above, this might work also for appended skeletons, since all included and appended animations are updated at definition reload. So the Clonk skeleton would be directly updated after a Clonk.appendto.skeleton has been reloaded. We should try it out, but I think first we still need to do 1) and 2).


Why not `require __DIR__ . '/../vendor/autoload.php'`? Otherwise the `autoload` section in `composer.json` is useless.


Just saw the @stof answer ;) His one is better than mine ;)


If you like you can do 1) and 2), and I take care of updating the animation state correctly (and thus fixing the crash that we have at the moment in SkeletonInclude.ocs).


It could / should ...


You shouldn't need any CSS if you use bootstrap's css classes for radio buttons instead. Take a look at the radio button example on http://getbootstrap.com/css/#forms.


yep, here it is bug, as nil bootloader result in nil in other operations, but new getter will set it properly. so with new code it won't crash.


俊哥啊，我都看不下去了，不如你出两万，竹尖儿帮你做简历吧！:joy:


These vars are not in alphabetical order.


Yes, it can be removed


Thanks Fred!


That's not quite right indeed, will integrate.


As a style matter please leave brackets in.  Thanks!


http://blog.csdn.net/im_kk/article/details/42435471
参考这里可以全面兼容


`a` instead of `an` here


No. I agree that the `except` clause should not do the vacuous `from_deprecated_string` call, but that's not the only legal course key format and we're about to switch all new courses to split which will mean fewer and fewer courses looking like `org/course/run` and more looking like `course-v1:org+course+run`.


See updated code


Yes. Another thing: What is the best strategy if I have other commits that I want to bring to the master branch before I merge the skeleton stuff?

For example there is a docu bug that I want to fix. Also, when zooming I get a debug assertion sometimes, because a variable is used without being initialized. I wanted to fix that, too.

I would:
- Just fix those bugs, create separate commits (those appear after the skeleton stuff)
- rebase, so that they are on top of OpenClonk master and below skeleton stuff
- force push to my private repository
- reset my local master branch to include both commits, but no skeleton stuff
- push to OpenClonk master


Can you add the upstream sha for future reference?


Oh yeah, that's a better option.


My bad, your commit works fine because you are picking up the number from a different attribute.


This test is now failing for me due to some mismatch with utc tz types.


I would make an extra branch for the skeleton stuff, then resetting your master to origin master, make the commits unrelated to the skeleton stuff in the master branch, push the master branch, then rebase the skeleton branch on top of the new master.


0 after all :D


@cdaversin there is some trouble to compile that code, are you aware about it ?


You shouldn't need the dd modules, since you're not doing drag and drop on aui-radio-group-data-editor.


I agree


I use PHPStorm, but I think these line breaks is less about my IDE adding in breaks and more about me being a bit thick :) Sorry about that...


Awesome, clean up breaks!


Could you elaborate on why will it be difficult to use?


turns out it is --use-symlink after fix in https://github.com/moai/moai-dev/pull/1061


Not something you wrote, of course, just good to catch it while it's in a diff people are looking at before it's forgotten.


Haha. Jolly good - so long as you're aware :)


Yeah :D


Here you don't need an else, because it will return a value with above verifications and if no value doesn't be returned the function will return UnknownSize (this is in case if the function request a return value).


Fair point. All I did was put it back the way it was before. I think renaming the stream itself would work better in this case. I'll fix it in the next commit.


Ok thanks for the explanation, I would remove vertical-align: middle: it is no-op if the widget has its computed height (which seems the most common use-case) and if an explicit height is set, I think it's up to the user to set vertical-align.


`<output_dir>/src` remove the html


Double check that PhpStorm isn't set to add in "soft" line breaks.


1: M101 signature is missing


> This test is now failing for me due to some mismatch with utc tz types.

Nevermind, it was a red herring.  Fixed in 85d9a68


我是后加的拷贝，这个地方尽量能不拷贝就不拷贝比较好


@JanTvrdik exactly.


Just commited this change.


```getArrayCopy``` no ?


Lol yep.... Damn _I's!

Thank you,

Keegan Carr

(636) 578-0099 St. Louis

June's customer survey score: 4.97/5.00

2012 Best in Business “Best Consulting Firm” from STL Small Business Monthly

Sent from my iPhone

> On Jan 5, 2015, at 8:17 AM, TeoTwawki <notifications@github.com> wrote:
>
> that dastardly status list gotcha again eh?
>
> —
> Reply to this email directly or view it on GitHub.
>


This is not working as expected as the processing will never be deferred to background as long as processing is enabled on class level.

I think the expression should be something as follows:
```
def proceed_with_versioning?
  enable_processing && (
    !model.respond_to?(:"process_#{mounted_as}_upload") ||
    model.send(:"process_#{mounted_as}_upload"))
end
```


Did you get this label from some wireframe? I still can't find it anywhere, but I remember Jadson showing it to be and asking for this change. Can you confirm the label with him in case this isn't the one on the wireframe?


remove html from output path here too


Yes, am parsing it from a json string which is stored as part of some page element. This is to keep the display(number without any formatting) similar to FB likes count.


Missing indentation here and on line 79


Too bad PGNs won't be saved...


Hello Marian,
I updated some typos in 'additional/lecture-01/commands.txt'


Fixed :)


Instead of using 0x100 as a magic number in memset of cue, you should use sizeof(cue). That makes the code easier to maintain and avoids bugs when people change the number in one place only. Also, I still don't understand the TRASH stuff. Can you explain what the trash is? I didn't really understand the redump-thread you had linked to.


Very good point. I had not thought of that type issue, but you are very correct. It is likely something that we should complain.

One 'issue' here, is we have no way of failing validation within the prepare().  That is when we detect it. By the time we get to valid, do we have that information?   I really think in that case, it is something that the user should be forced to fix.   It may be that they have simply picked up a file of prepared (but raw) hashes, and simply stuck a ':' in front of all of them, or something like 'user:' in front of all of them.  BUT since in this format the user$ prepared part is supposed to match, that really is a red flag situation.


Grmbl... C'est vraiment nécessaire ? On ne peut pas juste stocker un pixel blanc ?


Looks like there is an extra space on line 21..


Whoops! Fixed f5ef645


Its "speech" not "spech" :)


Johann Kl=C3=A4hn <notifications@github.com> writes:

> Can you try replacing the condition with (version< org-version
> "8.3beta") (as I wrote above)? I do not use org-caldav but I use
> ox-icalendar in one of my own projects, so I had to introduce a
> similar work-around.

That works, too. Better than my =E2=80=9Edirty patch=E2=80=9C.
:-)
gp

--
https://ragp.de


fijate que los parametros de success y failure no los estás usando en la funcion, si no los usas volalos...


    def test_users_not_belonging_to_fun_backoffice_should_not_login(self):
        ...


What if instead of having the "addWhereMemberOfFilter" function invoke "getExpressionsForWhereMembersOf", it had a simpler signature of

```
protected function addWhereMemberOfFilter(Params $params, QueryBuilder $qb, array $expressions)
{
  if (!empty($expressions)) {
    $qb->andWhere(join(' OR ', $expressions));
  }
}
```

That way the addWhereMemberOfFilter function could be reused for multiple parameter filters and passed the appropriate array of expressions.


Why remove these?


>  renaming the stream itself would work better in this case. I'll fix it in the next commit.

:+1: Sounds good.


I think we should be using `!r` to grab the repr from those objects, actually.
E.g., `'{0!r}'.format('testing')` => `"'testing'"`
And it should be done on the value/rbv as well. What do you think?


This is a different test. E.g:

    def test_logged_in_non_staff_user(self):
        ...


No floating code please, put in inside a function with a semantically proper name, and call it from the main, before creating the http server.


Missed base method.


some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3014
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3019
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3013
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3005
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3005
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3008
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3001
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2996
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2672
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2670
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1509
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1500


This is yet another test:

    def test_logged_in_staff_user(self):
        ...


This could probably be split in two different tests; I don't see the connection with certificates, is this work in progress?


Yes, correct we construct routes between 2-4 nodes using viaroute and cache the hints returned from these calls along with the travel time is what we are caching and reusing in future viaroute calls using those nodes.

The reason for the extra nodes is because routes are dependent on direction of travel that you can not control with just two nodes. For example if you have say pickup nodes along a major street A and C and one on a side street, B, that is a two way street, then looking at A-B and then B-C is different the looking at A-B-C. Looking at A-B and B-C there is an implied U-Turn at B which we do not want so we look at A-B-C minus A-B to get B-C


Looks good to me.


Ca ne marche pas ça ?

    self.assertEqual('text/pdf', response['content-type'])


j'ai presque honte.


84202472-TurnOffInputEchoByDefault (JasonGlazer) - x86_64-MacOS-10.9-clang-Debug: OK (1576 of 1576 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)


竹尖儿看不上我们这小打小闹 :pensive:


A possible fix is in 3827cbd84c8d42817ae18fc960900c9764bc6ca2 (part of [a followup branch of mine](/b4n/geany/commits/SiegeLord/selection_document_mods+selection_stuff)) .  Another solution is to remember the columns of the anchor and caret and restore those, but it's not really simpler.

Though, I don't know how often we need to stabilize selection for things like that, but if somebody thinks of a generic solution it might be worth a shot.


If you are running in development and SSL redirection is enforced, you explicitly need your webserver to start with ssl support. Thin for example is not able to handle ssl when started through `bin/rails` by default, neither is WebRick.

So, for development purposes one have to turn off ssl enforcement  in the config now and needs to discard the changes of that file when committing.

I suggest to change the usage of the configuration like this:

    # alchemy/config.yml
    require_ssl_for: [:production, :whatever]

Then we redirect to https only if this array includes `Rails.env`'s value
What do you think about that?


No, crash is not completely solved.
When setting same page, feels like crash is getting few.
I'm monitoring crash frequency in production application now.
This is not a complete answer, sorry.


If it is deemed fit, I have it in b09b2bfdd109484bd48b6310cef0e7f4633fb8a3 (part of [a followup branch of mine](/b4n/geany/commits/SiegeLord/selection_document_mods+selection_stuff)).


@uhafner Could you file a Jira issue with the information you have?


@trustin - use `isEmpty()` instead of `.length() == 0`?


I will have to investigate.


Please avoid unnecessary whitespace changes like this, it makes the patch harder to review.


This should be marked a `[breaking-change]` since it changes the type of the function.


This happens in a few places in this commit.


We'll get a nice error if it has a '\n', though, right?


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


saw approval from aturon
at https://github.com/alexcrichton/rust/commit/0cb7a4062a3c69bb0c54f0c9136889a1006e4f62


merging alexcrichton/rust/serialize-associated-type = 0cb7a406 into auto


status: {"merge_sha": "03268bbf35d3ff2350d987fe7b60375839abdf2e"}


alexcrichton/rust/serialize-associated-type = 0cb7a406 merged ok, testing candidate = 03268bbf


@kecinzer this patch isn't related to your graphite crashes.


More whitespace changes.


Then fine for me, thanks.


désolé pour le retard, j'ai accidentellement verolé mon windows en voulant
partitionner mon DD principal pour un dual boot linux.
J'essaierai de réparer git quand je peux , à bientôt !

Le 31 décembre 2014 17:19, Daniel Gillet <notifications@github.com> a écrit
:

> J'avais donné un exemple de fonction avec self.get_description. Mais cette
> fonction est inutile. On a déjà l'attribut description qu'on expose sans
> problème en Python. Autrement dit, si l'utilisateur veut la description de
> son objet, il fera: print(item.description).
> Tu peux enlever cette fonction.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buffalo974/Pythoria/commit/d11f3fab512d549804d0f36dd069e0ef50f6586b#commitcomment-9127564>
> .
>


Jeszcze tylko taby -> spacje :pizza:


nope


Unnecessary indentation/whitespace change confuses the reviewer.


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


@Zikk420 Whadya think? :)


r+ p=222


There's a problem with this query. In a portal that has already link categories all of them are not available because they were not added in the itemproperties table at the beginning.

https://github.com/chamilo/chamilo-lms/commit/ea4b07fe8b6d35b2cd07929f44f2477e98bd0e52


@temasek does your frameworks/av fix no boot with gcc 4.9?


Extra empty line detected at body end.


Use the new Ruby 1.9 hash syntax.
Space inside { missing.
Space inside } missing.


Line is too long. [110/100]


Hey it's supposed to be "Hello World"!!


Ideally we should use Twitter API as the page design tend to change frequently.


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


OK i'll change it.


1: M101 signature is missing
1: M111 unrecognized component name: MiscUtil
1: M191 must not end with a dot '.'


Non c'est une paire (clef, valeur)

('Content-Type', 'text/pdf')


yo habia pensando en un primer momento usarlos como utilizamos el afnetworking pero ahora me di cuenta que es al pedo en este caso..para comprobar errores si el save se uso bien voy a utilizar el otro save que es con bloques..me parece


e nevoie de lucru cu fisiere


Well, "use with care" rather belongs to setting ``enableParallelBuilding``. We use non-default ``buildCores`` even on hydra.nixos.org AFAIK.


I would give a function as second param to `wirsing.module` and assign its return value to the last namespace (`switch` in this case)


Sorry for the delay... holidays! The regex was there to check if the value was a real integer. After checking my code, the regex function is irrelevant because the if statement has already determine it was an integer. Your approach is better and is also working in my app. Thanks for the test function. I hope you ahd a great holidays!


typo "attaches it"


it would be more flexible to assign the return value of a function instead of looping over properties


What's the advantage over this approach?


My concern is that it is not clear enough to set the logging level to "NONE" turns it off. I would be inclined to add "(no logging)" after NONE. Other than that trivial clarification, I am happy.


*+14 - 4th Jan


I ... believe you ;)


а я заполняю sample (объявление строкой ниде)
states = [0,1,..m-1] - это просто как перечисление всех состойний. т. е их вид


j


You're welcome.


More flexibility, now you can only assign plain objects, by using the return value of a function you can assign anything. For example a class, or an object that uses "private" state within the function body etc.


Sweet, thanks for the help.


Oh no....frameworks/av? I don't think I touch that.....i'll gather more info.


Can't it be :
std::copy(&shape[j], &shape[M], &new_shape[i]);


What problem was this fixing?  I don't recall seeing any issues with this dialog?


That's not possible, consider two files defining stuff on the same module:

```coffeescript
# file a.coffee
wirsing.module 'mymodule',
  a: -> # whatever

# file b.coffee
wirsing.module 'mymodule',
  b: -> # whatever
```

The file that's loaded last would overwrite the properties of the module defined in other files.


p.s.
If we're all cool with this I'll make starter files for all tables there is custom content in, and write batch scripts to make importing it all at once an easy thing.


Hi Barry

I hope you're well. I've just noticed you forked the repo when you submitted your Blagger solution. So the exercise is valid for other potential candidates, would you mind deleting the forked repo?

Many thanks
Neill


This file have more "}" than "{" ???


Actually https://github.com/zooniverse/Planet-Hunters-2/issues/260


Yes, very much!


r=nikomatsakis p=2222222


r+


Ok, so you would have to write

```coffeescript
# file a.coffee
wirsing.module 'mymodule.a', ->
  -> # whatever

# file b.coffee
wirsing.module 'mymodule.b', ->
  -> # whatever
```

right?


@shaggytwodope how does this look?


I have never split a module into multiple files in my life before, i guess that's a weak point :-)
What you suggest is much slower than using a return value, because you have to loop through everything.


Happy new year!


Any reason why scaling_max_freq is limited to 1.2ghz instead of 1.5ghz?


We *so* need to untangle this `XRegExp` mess and only activate that for as3 code, not our internal use ...


That all makes sense and sounds good to me!


Wow, how does anything ever work?


Is this line supposed to be commented out?


```
wirsing.module 'filters.switch', ->
  on: (items, key) ->
     switchObject = {}
```


It should be fairly simple to actually implement this. File a bug?


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


Right, main contribution here a test :)


Can you file a bug for investigating what we should do here and add a comment here referring to it?


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


merging Ms2ger/servo/MouseWindowEvent = be683a08 into auto


Ms2ger/servo/MouseWindowEvent = be683a08 merged ok, testing candidate = ca876edc


You haven't worked a lot with Ruby, where we love to dynamically patch classes at Runtime :stuck_out_tongue:


Closes #101.


This "int comp_id"/rtapi_app_main/surrounding empty lines whitespace change is another example.


good thks


See line 366


Makes sense, I'll change it.


Hi! Yeah, it's a minor commit, which will most likely result in insignificant to none improvements in term of performances as the compiler already cycles through the if..else if..else statements.

Thanks for the heads up. :+1:


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


I can see __DIR__ as well but it did not work. Actually i don't understand why we need the dir change at all, isn't the present dir the default?


@CrazyBits If you have interest, maybe you could help with building V8 under https://github.com/xicilion/fibjs

They have a very clear structure and should work.


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


did not looks much close, but 'clone!' in instanceSymbols creates some conflict during binding.


Why is that even remotey required ? The toplevel patch is there because we can't properly have sub-packages in a repository, but this one ?


This would be useful. At the moment I have to resort to https://github.com/kana/vim-repeat


I simply adapted the existing patches to HEAD?!?!


a bit wired...


On a related note we could fix hashes that come in prepared state but has an empty login field - we could fix it in prepare(). I'm not sure we're allowed to do that though, the current description in formats.h does not mention altering any of the fields.


@Whisno why this test? Is this not going to break things? See for instance this tax in l10n_be which has a refund base code but not refund tax code?

![image](https://cloud.githubusercontent.com/assets/692075/5614915/1413f58a-94f6-11e4-9007-691b26578da9.png)


Guess this is old code from me. Later I learned that you should always use `respond_to_missing?` and not redefine `respond_to?` directly. The original intent was to make it convenient to call model factory class methods to construct models for testing. If spec tests pass without this the respond_to? method is not really needed - it is however prudent to add a `respond_to_missing?` to make it "symmetrical" (say if you are debugging and wonders if you can call a method ...). I am fine with just deleting it since this is in an an rspec helper module.


Add the implementation here like this:
```cpp
ServerConfig(const ServerPaths paths, const bool flag, const int max)
    : server_paths(paths)
    , use_shared_memory(flag)
    , max_locations_distance_table(max)
{}
```


Remove this file completely.


This will allocate more Sets than there are rows -- one option would be doing something simple like this for now:
```julia
nameset = Set(arrayofhashes[1])
for i in 2:length(arrayofhashes)
    union!(nameset, keys(arrayofhashes[i]))
end
```
, which also has a better chance of being tightly typed.

Longterm it would be better to not generate `colnames` at all here -- and cut the hashing in half by calling `get!(df, name, ...)` in the loop below -- I'm adding that function to `DataFrames.jl` now, but it probably won't be tagged in a release for a while.

Good improvement in functionality overall.


Remove this line.


Давай не будем в try переименовывать. В ES3 проблемки, придется через `['try']` запускать.


Remember to rollback these changes. @rkoster README replaces the tokens for new users.


Missing license info header.


Why rename and split these files out? It should work fine as one Vagrantfile AFAIK..


Change this to the version that works; not `latest`, which might not work for a future version of cf-release.


What do you mean?   Not merged to which product or branch?

On Sat, Jan 3, 2015 at 11:11 AM, Pranav Vashi <notifications@github.com>
wrote:

> Why this is not merged?
> 250571c
> <https://github.com/MotorolaMobilityLLC/kernel-msm/commit/250571cfd605edcf6d69d5f518f847c4263bd83c>
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/MotorolaMobilityLLC/kernel-msm/commit/cbd1cc49516c6a0db05b108c149d5796eb2fbf08#commitcomment-9143088>
> .
>


First we are brought to the main page were you have two options either to start the process or go into the modification of Costs and EC, this page shows you all the different materials,costs,units and EC of each type of material used in the process. There is also 2 videos to show you about different CyclePaths in Ireland.
PathType page:
You get the option of selecting 'Greenway', 'Disused Railway'


Thanks!


Sure, here is a bug https://bugzilla.mozilla.org/show_bug.cgi?id=1117779


Missing $timeout


Fuck this file modes.


I think this will now break this template for the larger deployments?


@Allactaga Please review this.


Creating a new Django model hasn't been the standard way of configuring the UI in Studio.  I have seen this done so using Django configuration settings instead.  @andy-armstrong or @cahrens Is there a design pattern for configuring UI options in Studio?  In this particular case, we want to limit the types (i.e., profiles) and statuses of video encodings that are included in a CSV download UI feature.


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


Okay, thanks for clarifying. I will fix the problem with keypresses on an open drop-down highlighting every item.  (Good catch!)

I designed KeyPress to return the virtual key ID, not the character code.  This was so I could detect which physical key was pressed in PD.  (For example, 1 on the regular keyboard vs 1 on the number pad will raise different virtual key codes.)  You are right that you can change it to return characters by raising the event inside WM_CHAR instead of the hook proc.

If you want to modify the value from inside the event, change the event declaration to use ByRef instead of ByVal.  (Right now it is ByVal, so you can only change the local copy, not the one used by the hook proc.)


In combination running [stub matchesInvocation:anInvocation] ahead in the first for loop, then is the -evaluate method in OCMBlockConstraint called twice which leads into errors if you use for example a block evaluation of arguments which is using a XCTestExpectation and call fulfill.


You're welcome.  Sorry it took me so long to fix it.


derp, sorry, I forgot I'd been testing the old configuration & didn't uncomment it before committing : P


@hanke are you sure?  They both have .walk() and os.walk is not the one appropriate here afaik... with this change I believe test builds start to fail:
http://nipy.bic.berkeley.edu/builders/pymvpa-py2.x-sid-sparc


Why not create a delegate object which will execute a block after it receives start (`CDTRunBlockOnStartDelegate`)? Then you can put the delete logic in the test itself rather than creating delegates with difficult to decipher class names.


Remove logs? Or use lumberjack?


?

https://github.com/apiaryio/black-belt/pull/49/


A little note to this solution:

If chat will not contain ```message``` (hypothetically, this may be, example: only ```title```, ```face``` and ```options list```)
then no margin before ```options```

if it never will happen then no problem, it's just a digression:)


No. This check is to make sure the request is for all the connections and not a specific connection (with an id). Hence checking for nil value seems appropriate here.


Does this work with multiple certs?


Inspection? Also please keep support for eclipse :).


Are all the asserts in these methods necessary? It seems like this test could be slimmed down to just checking what it needs to.


Looks good. Although i would wrap this (and line 77) in `\b` pair. And may be line 77 also needs re.IGNORECASE, m?


not "demo"?


> initDeferred... Same elsewhere.

Thanks!

Fixed at
https://github.com/firebug/firebug.next/commit/26164e4b478506ebd8f71278c2a0b1b38336a440

> we should probably be moving towards native promises

(I think I replied somewhere already, but yes agree)

Honza


Sean, vendor blobs aren't needed for Flounder. It has a separate partition for vendor. To update, you have to manually flash vendor.img from the factory image(s).


Adding commented-out lines?


Ok, let's go for a more permissive version


fixed codestyle according to code review


Yup, makes sense to look at ImageStreams. However, I still want the metadata for a single image. So once I find the ImageStream I want to pull the latest image that's associated with it. I don't believe we have a way to do that in the API today.
The Docker client doesn't talk directly to registries. We can make a simple REST call to find out if an image is in a registry, but to get the image metadata we'd still need to pull it locally since the registry API doesn't really give you much.


r+, thanks!


I guess it's ok if this gets it working for the categorical variables, although it would be nice to somehow keep this check for numerical variables.


Thanks! BitHub has sent payment of  $9.14USD for this commit.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/461
success: http://build.servo.org/builders/linux2/builds/460
success: http://build.servo.org/builders/android/builds/496
success: http://build.servo.org/builders/mac1/builds/677
success: http://build.servo.org/builders/mac2/builds/678


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1779](http://teamcity.monogame.net/viewLog.html?buildId=4175&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:00


This was the first bug.  femFun is not indexed by quantity_id, which is an index into the operator row, not the appropriate field function on the mesh.


This was the off-by-one error I found and fixed.


Automatic numbering of [env.foo], [env.bar] etc blocks now relies entirely on Asciidoctor's `caption` method, following a suggestion of @mojavelinux .  However, the automatic crossreferencing of equations is broken, since calling `parent.document.register :ids, [attrs['id'], caption]` in `EnvironmentBlock` does not permanently modify the `@references` map: it seems to get overwritten with default info.  @movajelinux, do you see how to proceed?


yeah, thanks mate


fast-forwarding master to auto = ca876edc


Can we also remove the `production:` block from this file? It's potentially confusing to keep it in, even though you've added the comment about the file being overwritten.


No need to write "HACK" here, this is perfectly reasonable.


Please note that the code involving `Asciidoctor::Parser.foo` was a wild (and unsuccessful) stab in the dark. It will be deleted, as will numerous warn statements that I use to see how Asciidoctor operates.


nit: begin error message with a lowercase letter


@dentmaged we are not using xml or yaml.
The api for hocon might not be used but iirc current plans still are using the format itself


saw approval from jdm
at https://github.com/laumann/servo/commit/9b1fd5a05f481ab87c4f8ad010168ec836cc3c5a


merging laumann/servo/master = 9b1fd5a0 into auto


laumann/servo/master = 9b1fd5a0 merged ok, testing candidate = 3639c4e9


I don't think this is required. Is that? It is still here I think it should be removed. cc: @seb-pereira @clmath


That's the second printf left in to as many PRs. D'oh!


As long as a format has a prefix for the canonical hash representation, prepare could add just the prefix and drop everything else from the ciphertext if the user name doesn't match.
Then we have an invalid hash that is very unlikely to be valid for any other format.
Prepare could print a warning in this case.


Looks good to me :-)


ist auch schon längst geändert.


This is the integration of accY into rate setpoint... note the "-="


Maybe add a note saying: *If a module doesn't work with 1.4 but worked with 1.3, feel free to open an issue at the GitHub Core Repository and we will try to fix any problems ASAP*


The reason it has to be done the way you implemented it now is that `clone` can, and almost always is` overridden in subclasses. The override is only called when `asCallPublicProperty` is used.


Ha, very clever! :)

This interacts strangely with the line number counting code, though. We only count `\n` as increasing the line number (which I think is correct), so treating a bare `\r` as a newline might confuse the (purely hypothetical) users running Mac OS 9.

I think it's probably safer to just specify that `\r` is a whitespace character and skip it. I'm fine with `\n` being the only line separator.


i don't like this syntax with the _


couldn't prepare just change ciphertext into an empty string in this case?


Nice.


I'll rewrite that later.


The CI builds don't contain a src folder.


This sounds like you have to create a MySQL username and password (instead of just using the ones provided by the hosting provider)


Missing a newline here :no_entry_sign:


*Juwelier Händler


In a lot of the other cases, we can just dump the stuff straight into the functions, instead of assigning it to a variable first.


Hi,

Yes, this is insignificant but I tell you to know,maybe in future maybe you will need to return a value from a function like that that's necessary to return a value and you don't need an else for that.


Is there no automatic redirect to the installer (isn't it enough to visit `http://example.com`?


"If you feel like your hack is useful / a bug fix / a nice feature, feel absolutely free to open an issue at the GitHub Core Repository. We're glad to merge any improvements of the core"


Re: log10 - probably any android device - both Galaxy Nexus (Android 4.4 - Cyanogenmod) and Nexus 7 (Android 5,0 - stock) failed with log10


trollop 2.1.1 has been released without the curses dependency, you may be able to revert this now.


It feels a bit like the superclass is just de-duping coincidental similarities rather than providing a meaningful abstraction. It doesn't match the broadness of the classname, which suggests this could be used as is for any Content Item. Since it doesn't provide much in the way of shared functionality, what about dropping it entirely?


Right, yes.


Yeah, unless we're changing the data of the result, we don't need to separate it.


Modules is uppercase, plugins and themes aren't.


Suggest adding an additional note here about how beta testers will not be able to preview discussion TOPICS, either.


I did wonder that myself too. I'll drop it and amend.


as-is: App crashes, need to move stream reader to background thread
to-be: Bit of refactoring, moving fragment to another-individual class


ich weiß nicht wie die posix funktionen realisiert wurde, kann mir aber vorstellen, dass diese nichts anderes als wrapper sind. willst du wrapper wirklich der os api selber vorziehen?


Will using the current time have the effect of publishing a new minor version every time the app is deployed, even if there haven't been any changes? If so, is that a problem?


..looked up that bug and cannot reproduce it now, so changed both these cases to use the "standard" way of binding the context.


Would you consider a slightly different version where JSON was added as a 'RuntimeRecommends' dep and the 'if ($self->json)' block tries to conditionally load the module and displays an error message if it's not installed?


Currently this isn't being used anywhere so I'm not sure it's an issue.


(Apologies for the 2 month delay, BTW.)


Obviously i knew that. The main reason why I chose to go with this syntax variation is code readability. Also, the function returns a value anyways. That's what the last else is for. It's called a forced else. If the function didn't return any value it wouldn't compile it.

Thanks for the observations. Cheers! :smile:


We shouldn't be changing the data. That's a bad design. Anyway, this should be massively be improved by the command stuff.


I am not sure what's going on, but I don't see this style being applied for blockquotes (locally or in prod).


Do we not want it to update when `order` changes? Seems risky. Probably `Ember.computed.equal('order', 'op_likes')` is better no?


BAM type should be added to the 2 ```label```s.


The most important style convention for a developer to accept for themselves is the willingness to **observe** the existing conventions in surrounding code and **preserve** the conventions in code they add. In addition to differences that exist between projects, it is common to find differences even within a single project. By conforming to existing styles, you gain the following advantages:

1. Differences in style start to "fade away", so your focus stays on the relevant semantics of the code you are working on.
2. Your code will be more easily accepted by project maintainers, regardless of the project you are working on.
3. You won't need to spend as much time deliberating styles/conventions for a project.

As a developer, you will frequently encounter situations that are problematic for the long-term viability of a project. But you don't want to be the developer that causes a situation to become problematic.

As a final note, discussions of style transcend individual projects. Be willing to provide feedback/rationale for your preferred style convention, but try to reserve them for cases where you are specifically asked to provide feedback or the style cannot be inferred from the project you are working on.


[slightly different version] Sure.

Oh, and apologies accepted, admiral Genehack. ;-)


BAM type should be added to the ```label```.


ups. :-D


existiert sowieso nicht mehr ;)


Oh, I see. It is wrapped in the `.questions {` instead of the `.answers {`


The entire admin order form was updated to have a cart & shipment page/step it shouldn't matter in 2.4


parameter und klassen-eigenschaften dürfen vom namen gleich sein oder möchtest du das im speziellem nicht?


using [ttezel's twit] (https://github.com/ttezel/twit) and [thinkroth's Sentimental] (https://github.com/thinkroth/Sentimental)


I just used a 1:1 mapping to the executable names for now; maybe `spearofdestiny-demo-data` would be better ?


Doh.


Fixes #4


no, this was originally conceived with one cert per app.


arbeite generell immer mit this wenn es sich um attribute handelt. stört dich das oder kannst du dich damit arrangieren?


@harrison should we take this out again, do you think?


It would be easier if you could pass the direction you're starting in, as suggested here: #605.


wow this is starting to get complicated for just 1 item lol, ill add a cap on dmg too while im at it


I've incorporated more links in the latest version, thanks :)


Gute Sache. Gefällt mir :+1:


pretty cool!


I tested the module and confirmed that this worketh. :)


вытащил


Incorrect how?


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


@GrahamCampbell this "fix" shouldn't be in there anyway. It's only because of #287.


oc到底是弱类型还是会强类型啊。。


r- some more fence markers are needed: https://github.com/alexcrichton/rust/commit/acef2920d46350cbd9dc4f495072e3cc61bf7545


Yeah, I mistakenly thought `isEmpty()` is since Java 7 but it's actually since 6. Thanks for the heads up!


Here is an obvious typo, you're trying to decrement a function return value


**FAILURE** - TeamCity Infinispan :: Pull requests monitor [Build 8194](http://ci.infinispan.org/viewLog.html?buildId=22556&buildTypeId=bt9&guest=1) for 4f7e5f8a4a17f560485558f03895a030a2da5648
Tests: 11215, 4 failed (0 new), 23 ignored. Build time: 01:52:36

Failed tests detected
* (since 30-Dec-2014) TestSuite: org.infinispan.client.hotrod.query.MultiHotRodServerIspnDirReplQueryTest.testAttributeQuery

* (since 30-Dec-2014) TestSuite: org.infinispan.client.hotrod.query.MultiHotRodServerIspnDirReplQueryTest.testEmbeddedAttributeQuery

* (since 30-Dec-2014) TestSuite: org.infinispan.client.hotrod.query.MultiHotRodServerIspnDirReplQueryTest.testProjections

* (since 30-Dec-2014) org.infinispan.server.test.query.ManualIndexingIT.testManualIndexing


at 10 mins per game and four 12-core machines available 24x7, 10K games would take around 2.5 weeks... :)


all tests pass:
success: http://build.servo.org/builders/linux1/builds/462
success: http://build.servo.org/builders/linux2/builds/461
success: http://build.servo.org/builders/android/builds/497
success: http://build.servo.org/builders/mac1/builds/678
success: http://build.servo.org/builders/mac2/builds/679


Need to get this change into a release :)

Will that cause unexpected keyword argument errors with older versions of virtualenv/pip ?


Is the rest of this file bash-using [[ ?  If so, having some consistency would be nice.

I'll take this request once esproul is happy.


> On Jan 5, 2015, at 8:18 AM, Brad Davis <notifications@github.com> wrote:
>
> Why rename and split these files out? It should work fine as one Vagrantfile AFAIK..
>
Unfortunately the same file won’t work for both.  If the extra private_network is in the vmware file, vmware doesn’t come up with a conflicting address error (sigh).  If it’s not there on virtualbox, the shared folder doesn’t mount.  I couldn’t figure out a way to make one file work for both.=


nah, I meant the "type: full" field ;)


Any worry about exceptions causing leaks here?


> On Jan 5, 2015, at 7:38 AM, Brad Davis <notifications@github.com> wrote:
>
> Why remove these?
>
Not necessary.  Trying to minimize the Vagrantfile.


this is not correct you have to "Multiply every other digit by 2, STARTING with the number’s second-to-last digit"


fast-forwarding master to auto = 3639c4e9


r=aturon


@williamespindola and @rogeriopradoj, I think this will fix the problem in #11.


No. This is a nagios plugin, so it's lifetime is very short.


No Luca no Work :japanese_ogre:


Ok thanks to spot this


nit: space after if


Can use utils.clamp to do that right before the line declaring message as 0.
```lua
dmg = utils.clamp(chance, minNumberHere, maxNumberHere);
```
And that means adding
```lua
require("scripts/globals/utils");
```
to the requires abiove as well.

lol...aren't add effect scripts a pain in the rear? Hold off on creating more of them, because ima nail every single them soon and save you the trouble.


I thought the django workers needed to generate the manifest on preindex,
but looking at the fabfile I think I made that up. Anyhow, surprised that
copying that directory would be time consuming, especially if it's only
done once.

On Sun, Jan 4, 2015 at 12:23 PM, Cory Zue <notifications@github.com> wrote:

> i think preindex directories are only necessary on the db machine and the
> proxy.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcarehq-ansible/commit/3e1bca4c5d650572e26448a148c0eead2bae5f22#commitcomment-9146574>
> .
>


支持你哦！！


cvbc


Cycle Columbus is a bicycle demand surveying app used by the planning department. It is based off of the Cycle Atlanta program.


Does the audio device get created if these are removed? It is worthwhile to remove the extra devices to trim down overhead.


Per the [Gecko user agent string reference](https://developer.mozilla.org/en-US/docs/Web/HTTP/Gecko_user_agent_string_reference), I believe this is more robust (aka, it works on desktop):
```js
navigator.userAgent.match(/rv:([\d\.]+)/)[1]
```


So good work :smile:


Should probably be >= since track_samples is a size and new_loop_out is an index.


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


That's nice. Is an image in single view mode also reloaded after key handler has run? That would be necessary as well when an image is rotate or otherwise modified.


you too :smile:


Our repos https://github.com/motog2014devteam


No we are not. I'll remove that classes thanks!


@sharwell strongly agree.


@alkemist Would a user ever provide their own `ByteBufAllocator` instance? If so, then above we would need to check the `userRegistry` for an entry otherwise this will always be the default getting passed to `HttpClient`.


So Here's what I do.

    var root = location.pathname.replace(/\/(index\.html)?$/, '');

on android this gives us /android_asset/www
on iPhone it's something crazy long I don't have in front of me.

Once you have the root, I just run

    plugins.LowLatencyAudio.preloadFX(name, file, suc, fail);

where file is like `root + ...path/of/sound.mp3`

This can be anywhere on the device.


hmm i might not be able to use clamp cos i need to control 3 dif var amounts on the fly.
but if im wrong let me know


After running the key handler, sxiv checks the modification time of the file. If it has changed, the files is reloaded. This should work regardless of which mode sxiv is in.


ij


Is there anyway to verify that this newsletter 1 was sent?


YEAH!! Good Work!


This saves the switches twice, the create method already calls save.


if its needs to be diff amounts for each of the 3 subeffects, use it 3 sep times inside each conditional branch instead. I just assumed all 3 were doing the same amount.

I also assumed the hp drain was not an additional damage on top of the melee swing btw.


We moved it to vendor overlays and controlling power menu items from there https://github.com/SimpleAOSP-Lollipop/platform_vendor/blob/L/simpleaosp/overlays/common/frameworks/base/core/res/res/values/config.xml

But i added "screen record" instead of "screenrecord in config file" .No wonder why it didn't work lol.
Thank you so much :)


I've fixed it now :)


I can match content in the email that is in newsletter 1. Do you want me to add that to the test?


Sent from Yahoo Mail on Android


>  I would remove vertical-align: middle: it is no-op if the widget has its computed height (which seems the most common use-case)

Well, an HTML such as

```
Toto: <d-combobox ...>...</d-combobox>
```

renders in Chrome 39/Win7 wth the current theme as follows:

![image](https://cloud.githubusercontent.com/assets/1583981/5615466/d78b33ca-94fb-11e4-94b8-28a92d2fe0b3.png)

while the vertical alignment is lost if vertical-align: middle is removed (we get the default baseline alignement instead):

![image](https://cloud.githubusercontent.com/assets/1583981/5615564/9090a4fe-94fc-11e4-9321-d84738d5c17e.png)

To better see the effect, here I increased the font-size on `d-combobox`, but it also holds with the default font-size (and with no hardcoded height set for the combobox element).

Sure, as you say, the user can set vertical-align: middle by himself, but I still wonder if bootstrap didn't had a good reason to set the middle alignment.

Anyway, if we do think it's better to not set it, I would remind that, besides Combobox and Select, this setting is done on the root element by a bunch of other widgets: Button, ProgressBar, ProgressIndicator, Slider, ToggleButton (and subclasses). I guess we should treat them all consistently with this respect, isn't it?


Though I do understand why branches would be your first choice I believe it would make a bit more sense to rather create separate repositories here on github called say `gate-controller-sw`, `gate-controller-hw`, `gate-reader-sw` and `gate-reader-hw` and then put them together in this repository, where they would be submodules. That way each subproject could have its own issue tracking, pull request discussion, git log and so on.

Does that make any sense?


This broke us - why did we remove setDefaults? I need the ability to configure defaults to the provider.


Thank you both!
(version< org-version "8.3beta") seems to work but then I have another problem. It says "No UID could be found for current event" after syncing 4 events.

Do you think there may be something that I did wrong when I was playing with it. One thing I remember doing is to delete everything in the calendar and try to sync it again:

C-u M-x org-caldav-delete-everything

followed by M-x org-caldav-sync

thanks in advance,

Steve


haarspalterei!


I mention this because the test says that you are testing that "newsetter 1" was sent.  If you do want to verify this, then I would put in some content checks to verify.


You need to add a comma at the end of the "true" part.


fyi the "accepted style" is to dump the space on the start of the additional logic-wrapped splices- makes things easier if you want to throw in various punctuation later on down the line.


There are a couple of typos in the commit message, and the subject needs neatening up.


My ideas on this commit:
1. I did not write this code.
2. I think that the `City` class should handle all of the gameplay-related components of a tile and the `Tile` class should handle all of the graphics-related components.
3. I do not believe the `Utils` class belongs in the `io` package. Maybe just `com.lqtz.globaldomination.Utils`?


nit: use underscore for variables: $error_type


Shouldn't this be exposed in the local.yml instead of hardcoded in the vagrantfile?


Ok lets revert that until Steve Kondik or someone else knows a solution.


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


Same as above + ")," should be added between lines 1034 and 1035 to close the 'vip array'.


Excellent.

And agreed on all points: spanners and indicators should implement exactly the same formatting interface and the old _lilypond_format_bundle property should have been reimplemented as a method a long time ago.

To add to the formatting design discussion:

The formatting system is currently component-centric. But consider the recent development of the more complex spanners (BowSpanner, TempoSpanner, extensions to TextSpanner). I think the formatting logic of the more complex spanner classes would be more cleanly implemented if, for example, each spanner could implement spanner leaves in order and produce format contributions along the way (and somehow depositing the format contributions on the appropriate leaves as the things progress).

Do the new spanner classes suggest a completely independent spanner formatting pass to be implemented in the formatting regime?


rax retry tests


I'm hesitant to do so because the content of the newsletter could easily change. I think this is covered in the #email_newsletter method on the Membership class.


(I'm not suggesting holding the PR up until a new DataFrames is tagged, in case my last comment is confusing.)


I should have probably asked you about it. thought it would be safe to select "resolve using theirs"


Rename `update_question_metadata` to be clear it's not user metadata.


弱类型……，奶奶的看得真仔细啊


On Mon, Jan 05 2015 07:58:57 -0800, Dan McDonald wrote:
> Is the rest of this file bash-using [[ ?  If so, having some
> consistency would be nice.

Mostly ([ is used in 7 places on omniti-labs:master). I'll change this
to [[.

--
Lauri Tirkkonen | +358 50 5341376 | lotheac @ IRCnet


Neat!


Added nav-wrapper-black


'strand2' macro CANNOT be used here: unfortunately ```-S``` option is different for mergeBed and must be followed by '+' or '-' .


Prueba


I dont recall exactly, but if there is no diff then i assume that means I simply selected to discard the changes from master. probably something i had independently fixed, or something that was removed in the rewrite.


More specifically, from the Messages buffer:

Contacting host: myserver.com:443
ID properties created in file "/home/myaccount/org/myorgfile.org"
Putting event 1 of 133
Putting event 2 of 133
Putting event 3 of 133
Putting event 4 of 133
if: No UID could be found for current event.

Everything used to work before I started to have the above mentioned sync problem. What would be your suggestions?

thanks again,

Steve


@fabiangreffrath this `Depends:` should do the trick

I can't find any proper copyright notice in my Steam tree; only the ones from dosbox


I guess you agree that none of your screenshot is realistic so I continue to think about the simpler solution : removing vertical-align. It's not a strong opinion, for sure the main point here is the consistence across widgets.


All but required on asset pipeline Rails 3 and 4 projects!

On Mon, Jan 5, 2015 at 11:23 AM, Brian Sierakowski <notifications@github.com
> wrote:

> Neat!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bmoreonrails/bmoreonrails.org/commit/25befd4f1700887cbdac9f15c7f129c366d8738b#commitcomment-9155708>
> .
>


The old TERM_LINK_RECORD_LENGTH=10 means a taskLink should remember the
past 10 termLinks used with it to prevent too many repeated reasoning. To
make it larger will in general improve the efficiency. However, it is not a
good idea to make it too large. Beside the time/space cost, a certain level
of repetition is needed for control purpose. One case: NARS does not keep a
record of supertask-subtask relationship. Instead, "repeated derivation" is
used to keep this relation in a dynamic manner. For example, task T1 and
belief B2 generate task T3, then if after a while, it happens again, the
two T3's will be merged to become a task with more resource budget. Similar
things happen in the human mind --- the ideas that are are repeatedly
generated will get more attention. In this way, in each task derivation,
the new task does not need to get all of the budget it will be given, since
at that time the system may not know how much resource is proper.

A lot of testing will be needed to decide a proper value for this
parameter. 10 is a pure guess.

Pei

On Mon, Jan 5, 2015 at 6:34 AM, automenta <notifications@github.com> wrote:

> please review this
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennars/opennars/commit/531e7d9330f8e5c4170e8c2434f0d2cb93779b3d#commitcomment-9151991>
> .
>


really? I dont think so. We had both fixed this leak problem differently and I liked your fix better :) notice the loop change, dup doesn't need to be freed anymore because `j` begins at `0` now. Anyway I selected to resolve using master here so unless you didnt completely plug the leak then it should be fine.


@floehopper did you decide whether to open an upstream pull request? The Ahoy maintainer has merged our other changes into master, so it'd be good to remove our local dependency when a new version of the gem is released.


Add "(-d)" to help.


Holy hell. This patch is **awesome**! Bravo @lydell.


good catch. ;-)


r=aturon


Intentional?


@borshop: retry


It doesn't look like there is a space separating these capabilities in the list "name": "capabilities/bounding-set",  "val": "CAP_NET_BIND_SERVICECAP_SYS_ADMIN" in spec/examples/image.json


I can't find any mistake... :/


@httpete
This is now a provider, and you can use $get() for such a job. See [here](https://github.com/nohros/nsPopover/blob/master/src/nsPopover.js#L27).

Info on how to use this: https://teamgaslight.com/blog/how-to-create-a-configurable-angularjs-directive


:us:


I measured one allocation before this change and then zero after.
On Jan 5, 2015 3:11 AM, "jakubkrolewski" <notifications@github.com> wrote:

> I was just looking through the Timber code and found this optimized
> iteration and I really don't get it...
> What is the point of using CopyOnWriteArrayList if the iterators are not
> used at all?
> Additionally, you probably remember was Knuth said about optimization :)
> Was it actually measured how much resources was saved with this
> optimization for some common logging tasks?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JakeWharton/timber/commit/b0f03bee10f39588b3b582b3d1f326049e417dab#commitcomment-9151777>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1781](http://teamcity.monogame.net/viewLog.html?buildId=4179&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 293, ignored: 6 Build time: 00:16:23


Nice desu ne !


dup is initialised with strdup, so it has a copy. In master, there is no dup at all anymore:
0 $ ggn -w dup master
1 $ ggn -w dup fr-me
fr-me:gemrb/core/Scriptable/Actor.cpp:2168:                     char* dup = strdup((char*)classname);
fr-me:gemrb/core/Scriptable/Actor.cpp:2169:                     classnames[0] = (char*)strtok(dup, "_");


r+


I agree with malte. There should be an additional plugin for ground truth pose. And the IMU should be properly handled internally.


saw approval from jdm
at https://github.com/guillaumebort/servo/commit/16103f9752493f34a550ec31dc9f92f10d9c02b1


merging guillaumebort/servo/fix/3936 = 16103f97 into auto


guillaumebort/servo/fix/3936 = 16103f97 merged ok, testing candidate = 1f62a6ad


On Mon, Jan 5, 2015 at 6:26 AM, Richard Höchenberger <
notifications@github.com> wrote:

> But then again, the "principal"
> device we are dealing with is iohub.devices.serial.Serial, so maybe it's a
> good idea to leave the settings for this "master device" in
> supported_config_settings.yaml, just to emphasize that this is our base
> class/device here. Right?
>

I think this makes sense. It may not really be of any purpose right now,
but the config settings yaml files for a device could also be made to
support device inheritance (with some coding of course).

So maybe it would be useful for the base iohub.devices.Device class to have
the 2 yaml files. Params in these files would be 'inherited' automatically
by all other devices instead of each device having to duplicate all the
settings that are common to devices. Then a device, say MyDevice, that
extends a Device or a subclass of it, could have its own yaml config file
that only adds new settings that MyDevice needs. If the MyDevice yaml has a
setting that is already defined in a parent class, then the default value
for that setting would be updated to use what MyDevice specified.

What do you think?


These removed line should have been replaced with ```<expand macro="strand2" />``` .


At this point, nothing is actually loaded. The message should be something like:

```
"Duplicate Component with name "' + componentName
```


ah yes loki just showed me what you suggested, ty


Oops. @alexcrichton do I need to do anything here?


Done: https://issues.jenkins-ci.org/browse/JENKINS-26294


Add "(-bed12)" to help.


This let's Sonic Advance boot!


If we can get the test suite to pass on Oracle, and get the patch reviewed before alpha freeze then I definitely do want to get this committed for 1.8.

The first thing to do is to run tests on Oracle and fix found failures.


use org.kuali.coeus.sys.framework.persistence.OjbScaleTwoDecimalFieldConversion


org.kuali.coeus.sys.framework.persistence.OjbScaleTwoDecimalFieldConversion


This commit => https://github.com/MotorolaMobilityLLC/kernel-msm/commit/250571cfd605edcf6d69d5f518f847c4263bd83c
Is not merged in this branch => https://github.com/MotorolaMobilityLLC/kernel-msm/tree/kitkat-4.4.4-release-condor-repw


commit message should be 1.1.0, sorry for the confusion


What is the purpose of the +/- 20 on the limits and should they only be on the int specialization?


tabbing?


I think he has a point, how is he supposed to set the defaults without that method? There is only `getDefaults` and the `defaults` variable itself is local.


No idea, isn't it your code ? :)


Just add a few `{rust,ignore}` to the initial function examples


On Mon, Jan 5, 2015 at 6:44 AM, Richard Höchenberger <
notifications@github.com> wrote:

> PstboxButtonEventNT

This is expected actually. ;) Events are sent from iohub to psychopy as a
list of lists, where each list element is the list of attribute values for
that event. Then on the psychopy side, getEvents() turns each event list
into one of 3 types:

   1. a list (so unchanged)
   2. a dict, keys are event field names, values are their value. ;)
   3. a collections.namedtuple. This is what supports the '.' attribute
   access to events, but also supports accessing event attributes by index.

#3 is the default. You can change the event type by using
getEvents(astype='dict') for example (I think the kwarg is called 'astype',
better check the code if you get an error. ;)

Note that none of the options are to turn the event into the actual iohub
device's event class, like PstboxButtonEvent, since for high volume events
this conversion would take too long.

Is the 'NT' part a big issue for you?

Thanks again.


r+


rollup


Test `asciidoc-latex` with this file using
```
asciidoctor -r asciidoctor-latex -a stem=latexmath -b html try-out/math_article.adoc
```


Cool, I'll do that right now. Since it was in your branch, wasn't sure if you'd fixed it up or something.


This is not meant for urls. It's meant for engines that look like : <cdv:engine id='android' version='file://c:/path/to/android/platform'> . If instead a url is provided, it won't handle it and that's fine.


Ah yep, that seems to be a bug caused by copy-pasta late night. Will PR a fix, possibly also cleaning something up


Works great with newest version of Greasemonkey now.


@ianwalter
Well, I just showed you how :) Just follow the link.
In JavaScript, arrays and objects are passed by reference, so `getDefaults()` will return a reference to the object, which you may adjust according to your needs.


the drastic plot changes i think we should put into a post-release-branch.


Esse padrão é melhor? pois a própria documentação do Angular recomenda escrever a função separada e atribuir a referencia no link.


Can we please not put fprintf() in common code.


r=cmr rollup


> I'm not sure we're allowed to do that though, the current description in formats.h

This was the exact reason why prepare was made, to alter the format string, using other fields that are NOT available either before or after prepare.


this fixes #18


get the generic_types by index is fragile. Can be broken just adding a new type or changing the order. Os better use get_generic_type()


Para que serve esse "find" vazio?


Bumping too 2014 in 2015? xD


And yes, if we detect a dependency in field[0] and the prepared name of the hash, we could return a blank field, and I think it would keep this from being valid for md5ns. The original field [1] is STILL used for all other formats (prepare and valid), BUT it would keep it from being valid within this format.

That is an option.


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


What are the application/* myme types? Is safe add them to audio _and_video types?


This 2014 should probably not be changed.


thanks for the explanation.  TERM_LINK_RECORD_LENGTH works the same.  the length of the record list and the time it compares for novelty are probably 2 separate parameters but prior to this change, the record length was used in both.
it was originally:
                if (currentTime < recordingTime[next] + recordLength) {
and i changed it to:
+                if (currentTime < r.getTime() + Parameters.NOVELTY_HORIZON) {

both are currently set to 10


Fully agree. We do need to have a stable 1.6.1 finished in this week, with
examples and documentations. There will surely be future changes and
extensions, but let's leave them in separate places for now.

On Mon, Jan 5, 2015 at 11:51 AM, patham9 <notifications@github.com> wrote:

> the drastic plot changes i think we should put into a post-release-branch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennars/opennars/commit/415f9997ac77c4b28a51b9090aac5c31161e88db#commitcomment-9156072>
> .
>


I made edits in 2014, but I never bumped the year. When this commit was made I had not made any edits in 2015, so it was not bumped to 2015. But it has been bumped to 2015 too now.


```$tag``` is a text parameter, so it should be quoted.


This method is similar to what we do in Android: http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/home/BrowserSearch.java#430 I think we could steal some (all?) of that code here.


r+


r+ thanks!


lgtm


Ah, yes, I saw it had been, but still slightly funny.


I think this line should be
```
        Vagrant::Util::Platform.bsd?
```


there is a leading space before `?`


Ah, right.  "why" is because of the timing of product stabilization branches etc., it wasn't picked up even though it was fixed on our mainline in April.  It'll show up in upgrades.

-Chris


Oh, sorry, Copy and paste


ahh of course, amnesia ;)


Hmm, I see you are using this method in get_all_generic_types() then, you can't use get_generic_type() because will recurse. You can add a private method to get a element in _generic_types by id


Thanks.
Can you please fix the remaining typo ?


This probably wasn't intended to go in the live docs.


> I noticed that the red pipes and yellow pipes do not have author/license information for the ver.settings and up.settings files

That’s bad. We’ll have to add that.

>  I'd assume files like right.settings and left.settings don't need this information because they do not have the exact same name as a png file in the directory.

The settings files itself are not really copyrightable stuff, but the asset graphics are. Hence, the "virtual" settings files do not need author and license information, it’s all about the copyright information on the asset graphics (the PNG files).

> If I know what the goal is for the settings files

The settings files specify the dimensions, rotation and other stuff for sprites. Additionally, we "abuse" them to store author/license information for the sprites they refer to so we have that information as near to the graphic as possible.

Does this clear things up for you?

Vale,
Quintus


Thanks for your work on this bug! While I appreciate the attempt to fix these posts, I would not like to see this upgrade routine be forced. I'm looking at a database that has over 17000 posts that meet the criteria in the query. Updating the plugin and kicking in the upgrade routine, would most likely cause PHP time errors, database slowness, and/or fatal errors.

Perhaps this is better as a simple one-off WP CLI command? I would be happy to execute the command on the database, but not in an upgrade routine kicked off from a web browser.

Thanks again for your work on this bug. It's a nasty one.


Yes, the change looks reasonable.

On Mon, Jan 5, 2015 at 11:56 AM, automenta <notifications@github.com> wrote:

> thanks for the explanation. TERM_LINK_RECORD_LENGTH works the same. the
> length of the record list and the time it compares for novelty are probably
> 2 separate parameters but prior to this change, the record length was used
> in both.
> it was originally:
> if (currentTime < recordingTime[next] + recordLength) {
> and i changed it to:
>
>    - if (currentTime < r.getTime() + Parameters.NOVELTY_HORIZON) {
>
> both are currently set to 10
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennars/opennars/commit/531e7d9330f8e5c4170e8c2434f0d2cb93779b3d#commitcomment-9156163>
> .
>


Hahaha. Oh man.


I see. It should be possible to extend the `ValidationError` to support two flavours:

```
new ValidationError(user) // current variant
new ValidationError(User, errors) // new variant
```

However, this is not strictly necessary for the purpose of this pull request, let's leave it for another day.


Can we rise the logger level here to warn? I mean this is not usual situation so perhaps it's better to have it in the logs as a warning.


Great I will change the field to be related to the field in ir.module.module.
Any one knows where I can find the header for other licences than AGPL-3?
So I will update to have the header.template to be dynamically adapted to the licence of the module.


Whoops I totally read the diff wrong. :)


its called "Juwelier". "Juwelier Händler" is bullshit my friend ;)
http://de.wikipedia.org/wiki/Juwelier


I'll try and test these items out this week on windows and linux at least.
Will post back what I find.

Thanks again.

On Mon, Jan 5, 2015 at 8:15 AM, Jon Peirce <notifications@github.com> wrote:

> Cool.
> For the first issue I think I'd be seeing a strangely-colored movie
> during test, so I think that must be fine (not sure why - I agree it
> seems like an issue).
>
> For the second, I'm traveling right now but I'll try and test on my
> linux box when I get home.
>
> For the subImage2D thing I have no idea, but it sounds plausible.
>
> best wishes
> Jon
>
> On 04/01/2015 20:32, Sol Simpson wrote:
> >
> > Seems like a good direction, even if it ends up just for some
> > significant code simplification.
> >
> > Two things to check:
> >
> > 1) The color format of the numpy images returned by opencv is suppose
> > to be BGR, but the glTexImage2D is saying it is RGB. So this may need
> > to be updated.
> > 2) the reason for the numpy.copy that is now removed was because on
> > Linux a tester of the code reported that without this line the image
> > was not being drawn. On Windows it was OK without it though. Need to
> > retest to see if this /oddity/ still exits, or if the new code fixes
> > it somehow.
> >
> > Also, another speed up maybe achieved by using glTexImage2D for the
> > first frame draw, but then use the glTexSubImage2D call in all other
> > frame draws. At least on Windows I think this is faster because the
> > video card reuses the video memory allocated by the glTexImage2D call,
> > instead of freeing and recreating it every time a frame is transferred.
> >
> > Thanks for helping on this Jon.
> >
> > Sol
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/peircej/psychopy/commit/8af4931fe58d1618af652d9648bd0a89f60c7f3a#commitcomment-9148282
> >.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/peircej/psychopy/commit/8af4931fe58d1618af652d9648bd0a89f60c7f3a#commitcomment-9153128>
> .
>


Yes. It felt a bit weired to bump another year.


TODO: try with other Xorg versions, and with Java Runtime Environment and the mainstream OS's before Release, I think this is really a Xorg related minor bug and not Oracles fault, switching from version 8_11 to 8_25 didnt bring a change.


@kslong @smangham @Higginbottom I've removed version.h from the working tree, so it's jsut created locally by the makefile and not tracked. Don't worry, it's still involved in installation process.

I did this so I could add a commit hash to the file for reproducing past runs exactly.

Happy New Year all, btw.


What's going on in these touch methods? I assume we're just ignoring touches if there's a completion? That feels a bit annoying. Again, Android actually watches for selection/cursor changes and does different things depending on if you could the autocompletion text or the regular text:

http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/toolbar/ToolbarEditText.java#441

Maybe good for a follow up (but if this isn't the behavior we want, I don't think we should check in the touchesX() overrides.


jak chcesz robic protected to daj lepiej jako property


Thanks


to jest slabe ;)


refs #900


refer https://github.com/bitsuperlab/v8-fibjs


这个设置了也不管用阿。。。。难道附加写静态规则么？


:thumbsup:


LGTM


Yes, this could be a possible solution also. I added a note #605 regarding the fact that you might not be able to turn around or want to turn around. There are two issues here: 1) performance related to hints, and 2) direction indication to avoid implied u-turns.


If the local host is present among shuffled replicas, it would be nice to return it as the *first* one here.


Thanks very much for the patch!

Did you check that rotation, unicode font sizes, etc. does work ok (all seems to use float, but just to be sure)?
Also, self.zoom_mode / 100 is not enclosed by an int conversion nor a float placeholder, is ok to pass a float?

If all of this is ok, I think this could be merged.
Do you want to get access rights in the project site to close the related issues?


+1 for keeping the error code constant, -1 for `'MODEL_NOT_FOUND:' + modelId`.

Seems I commented on a wrong line, sorry for that. What I mean:

```js
404: {
  message: ('could not find ' + modelName + ' with id ' + modelId),
  // ...
}
```

IIRC you can get the model name via `var modelName = Model.modelName`

A possibly better solution is to include model name and model id in the error details:

```js
404: {
  message: ('could not find ' + modelName + ' with id ' + modelId),
  code: 'MODEL_NOT_FOUND',
  details: {
    modelName: modelName,
    id: modelId
  }
}
```


to bym zrobila jako konwerter. w xamlu binding normalnie do KinectCameraImage i converter=HasValueToVisibility i napisac tak jak jest napisany bool to visibility


Thanks. Nice stuff. :+1:


I would probably name these methods without locationTextField. i.e. didBeginEditing, didCommit, and didEnterText. That matches WKWebView (but these method names could wind up too generic.... I also think making a generic AutocompleteTextField would actually be nice, and these are sorta url-specific. Maybe that's someone else's project.


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3020
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3014
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3015
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3006
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3006
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3009
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3002
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2997
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2673
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2671
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1510
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1501


fast-forwarding master to auto = 03268bbf


I think a change in ordering of material could be beneficial. We can do our intro to Rails together and touch on Controllers. The following lecture could be more Controllers & and an intro to Views. Then the following day we focus on Models & Active Record. Thoughts?


Let's leave out the second ("better") variant proposed above, for the sake of consistency with other places returning MODEL_NOT_FOUND. Just rework the message please.


Overall looks good :)


What are these numbers? A comment about what's going on here would be helpful.


What's the "-" sign for? It messes up the alignment of the menu items :(


```$tag``` is a text parameter, so it should be quoted.


okay :) I will begin putting this state into a post-release branch and reverting the news which broke the plots.


This could be writed better. For example:
$this -> panthera -> template -> push(array(
    'file_path' => $filePath,
    'contents' => $fileLines,
    ...
);


Hrm. I suppose it technically does. I almost wish "run the following steps" linked to the caveat.

Note that there's no direct anchor for that part of #conformance. Could you at least add an anchor, and possibly consider linking to it when using run-the-following-steps?


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


Merging.


Thanks for the suggestion. We have discussed this with Tomi, and he did not
fancy the idea of spamming with 5 (or 6) repositories. Although I would
slightly prefer separate repos, it's actually not such a big deal and it is
true that having 6 repos for a single project would be quite a lot.

On Mon Jan 05 2015 at 5:17:30 PM Marek Šuppa <notifications@github.com>
wrote:

> Though I do understand why branches would be your first choice I believe
> it would make a bit more sense to rather create separate repositories here
> on github called say gate-controller-sw, gate-controller-hw,
> gate-reader-sw and gate-reader-hw and then put them together in this
> repository, where they would be submodules. That way each subproject could
> have its own issue tracking, pull request discussion, git log and so on.
>
> Does that make any sense?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fmfi-svt/gate/commit/3541d9e317332ea159999fe7f4738b770fa6fe0f#commitcomment-9155618>
> .
>


Fixed per @assertchris


saw approval from nikomatsakis
at https://github.com/nikomatsakis/rust/commit/a839b4e06cce73f4f24a241d8ef058ae674463f2


merging nikomatsakis/rust/nick29581-arrays-3 = a839b4e0 into auto


merging nikomatsakis/rust/nick29581-arrays-3 = a839b4e0 into auto failed


No. Basicaly i removed the question in french that had been left there.


Yeah for keybindings it'd work just fine but as @techee said for cases where we try and catch `<Primary>` with a mouse click we need something else.  So we might as well go ahead and use the same `<Primary>` everywhere and not maintain the keybinding map in addition to it, especially as using `<Primary>` in keybindings seems really the easy part.

BTW, this supposes that we should use `<Primary>` for button clicks and not `<Control>`, and while it seems to make sense I have no idea if it's the expected behavior on OSX :)


Update


Wouldn't our weak ref have only been made `nil` if there were no strong references left, of which, presumable `n.object` is one?


This should be written as:

        words (list of strings): list of words to choose from


Split this up into separate rules:

    : ( SIMPLE_COMMENT | BRACKETED_COMMENT) -> channel(HIDDEN)


Line length. Use the clang-format stuff.


What if someone explicitly specifies 5985? Then we should honor it.


Interesting, what exactly is not python2.6 compatible, the str.format method?


@dontcallmedom: that's incomplete (see the next update to this pr)


Probably worth giving the reason as makes sense in a UI, "Replicator stopped as local database deleted during replication".


Hint path seems odd. If this is a .NET framework dll, you don't need a hint path. If it is not a .NET framework DLL, you need to get it via NuGet - in which case path would be different.


Return early


Apologies.  When I realized my mistake I immediately added a second commit removing the offending line.

From: andrew-elder [mailto:notifications@github.com]
Sent: Monday, January 05, 2015 8:52 AM
To: christopher-s-hall/Open-AVB
Cc: Hall, Christopher S
Subject: Re: [Open-AVB] Added interface for standard Linux system cross timestamp code (5668e63)

Can we please not put fprintf() in common code.

—
Reply to this email directly or view it on GitHub<https://github.com/christopher-s-hall/Open-AVB/commit/5668e632c2063510b7f35e8a326c1622eab86325#commitcomment-9156077>.


I thought about the last one `(in a device usage context) ?`
so maybe remove the question mark.


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


Nice to say when should caller pass true, and when false.


saw approval from steveklabnik
at https://github.com/th0114nd/rust/commit/4ee73a124c828831e97201649a6e71f6d5b4f83c


merging th0114nd/rust/unicode-bottom = 4ee73a12 into auto


status: {"merge_sha": "8f6141cf1ecc591d7507cd092f7b198d0ba4718d"}


th0114nd/rust/unicode-bottom = 4ee73a12 merged ok, testing candidate = 8f6141cf


Done


Introduce method and return early


What is different in this code, vs in the ``` static int read_vm_code(unpack_data_t *unpack_data, const unsigned char **fd) ``` function?   Or does this other function also need the +1?

Note, I would really like to drop most of the allocation required by a lot of this file. Much 'can' easily be removed.  However, it would cause the source to diverge (somewhat) from original.  Is that something that could be done?  NOTE, this would be done by providing a fixed size stack buffer, and ONLY allocate/free a buffer if the amount of data we need is larger.  For the function you are doing, I would make a 256 + 2 +1 sized buffer, and only allocate if we enter the ```if (length==9)``` section, where we have a buffer from 256 to 64k bytes long.

The code would be something as simple as:

```
int allocated=0;
char local_buf[256+2+1], *vmcode=local_buf;

...
if (length==8) {
   len = git_bits(16);  // whatever the method is
   vmcode = rar_malloc(len);
   allocated=1;
}
...
if (allocated) MEM_FREE(vmcode);

```

There are numerous other calls to allocation that could be eliminated. It is not done that way in clam, to try to keep the stack size down. That really is a poor excuse for us, since this format is so heavy, that no one with a crap system is going to be working on these hashes anyway.  They are going to have decent boxes, where a couple extra K of stack space, is not a big deal.


I'm not sure what this method does from its name.


> I guess you agree that none of your screenshot is realistic

As I said, I just increased the font-size to make more evident that vertical-align: middle is not no-op (differently than what you said, if I got it well). With the same font size in both the text element and the combo the difference between midddle and baseline alignments is tiny, but the point here is that it's not no-op.

> It's not a strong opinion, for sure the main point here is the consistence across widgets.

Because of this consistency aspect, if you insist (even as your slight preference, not as a strong opinion) about the removal from Combobox/Select, I can't go for it without knowing if Button, ProgressBar, ProgressIndicator, Slider, ToggleButton will go the same way.


I'm not sure what this method does from its name. If its a getter, can we use a getter?


oops this was for postrelease1 branch, but doesnt matter, I will just revert it also :D


Please send the exact command line, and its help content via email to Rochak - and request him to signoff. The help content should go into the Help xml file.


verbietet const in diesem fall nicht, dass der komplette stream modifiziert werden kann?
wenn ja, wäre das ziemlich suboptimal, da .get() usw nicht funktioniert :D


How about the following?

"""Initializes the game state.

Selects the secret word for the current game by a random choice
from a list of words.


@sarfmomin please give me more information.  Try this branch too https://github.com/jchuong/ios-webkit-debug-proxy/tree/applicationUpdated


rax retry tests


@domcleal thanks!

@tokengeek I don't have a lot of the context, let me know if you think we should revert this now or not. Thanks!


We should also change the command line description to say that the default for plaintext is 5985, for ssl it is 5986.


We don't take cloud service name anywhere. Please remove this. Your code should iterate all cloud services for the given vault name.


Can't we use just `System.setProperty("local.server.port", this.port)` instead of this strange code?
And inject `port` as `String` as well.


Name or VaultName. Make sure you have signoff on cmdlet signature from Rochak.


I disagree with cvmfs-config-egi requiring cvmfs-config-cern.   I think the cern.ch configuration should be in its CVMFS_CONFIG_REPOSITORY=config-egi.egi.eu or perhaps copied into the cvmfs-config-egi rpm, depending on which of the two that EGI prefers.   I would instead put a Obsoletes: cvmfs-config-cern in the cvmfs-config-egi package.  Likewise the opensciencegrid.org configuration will be in either the cvmfs-config-egi rpm or /cvmfs/config-egi.egi.eu depending on what EGI prefers.

We could go ahead and now put in CVMFS_CONFIG_REPOSITORY=config-egi.egi.eu into 60-egi.conf


Why is there a separate if statement here? is completedAndMarkedString ever set outside of the above block?


Looks much better now. I have noted a few minor fixes.


On python 2.6 you have to specify the index in the `.format` method, like : `"http://dl-ondemand.radiobremen.de/mediabase/{0:}/{1:}_{2:}_{3:}.mp4"`, it's quite annoying. (I haven't looked if there's something else).


if the requirements are 3.8+,
why not add these to the unit testing too
just to be sure everything actually works fine then


For the next time, these two changes are unrelated and would have been fine as two commits :wink:


Nice catch!


Not having the caret when the completion is active seems sad. Follow up to fix?


Cool! I'll try it out as soon as I can.
On Jan 4, 2015 4:12 PM, "Tim Cooper" <notifications@github.com> wrote:

> @matthieugrieger <https://github.com/matthieugrieger> Would it be
> possible for you try using this in mumbledj? It may fix audio cutting in
> and out.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/layeh/gumble/commit/8215d611d67473776438d091278bf21f53df183b#commitcomment-9147788>
> .
>


And what are openHAB maintainers using? ;-)


unnötiges shadowing mmn. muss nicht sein. ich gebe parametern immer einen unterstrich am ende, dann spart man sich auch das `this` in bestimmten fällen.


```java
check(window != null, "OVER clause not valid for 'if' function", ctx.getStart());


did you mean olive oil?


stört mich nicht extrem, aber ich lasse es bewusst weg. wenn dir `this` lieber ist dann nur zu ^^


There's a convention to add a newline at the end of the file :wink:


path-separator wäre unter windows btw `;` und unter *nix `:` kannst die ja direkt mit einbauen ;-)


This is pretty nuts. I guess you're basing this on how Snap's implementation of ``<|>`` works? I can't help but feel that we might be able to clean up a bunch of this stuff if we use e.g. ``StateT`` rather than hand-defining our own monad.


That works for me and yes also on the value/rbv!


Is there a specific reason for removing ```-ubam``` from this wrapper, but leaving it in bedToBam.xml and windowBed.xml ?


Why? We have like 10 different emails for the same people now...


You should have another parameter set where the user can pass the Vault object itself (with option to take it from pipleine as well)

 I.e. flows like this:
> Get-AzureSiteRecoveryVault -Name "EusVault" | Get-AzureSiteRecoveryCredentialsFile
> Get-AzureSiteRecoveryCredentialsFile -Vault $vault

In this case, Vault name, Geo, etc. everything is available. I believe this is the mainline flow.

Please add a TODO for this - and make sure we put this.


using arp to get ip on which search is applied


FYI.  All I am doing is curiosity on this branch:
```
~/pism/buildnewerror[new-error-handling*]$ make
Scanning dependencies of target touch_pism_revision
Scanning dependencies of target pism_config
Scanning dependencies of target pismcalcalcs
Scanning dependencies of target install_site_packages_in_the_build_directory
[  0%] Built target touch_pism_revision
Scanning dependencies of target pismrevision
[  1%] Generating pism_config.nc
[  1%] Building C object src/calcalcs/CMakeFiles/pismcalcalcs.dir/calcalcs.c.o
[  1%] Built target install_site_packages_in_the_build_directory
[  2%] Building C object src/calcalcs/CMakeFiles/pismcalcalcs.dir/utCalendar2_cal.c.o
Linking C shared library libpismcalcalcs.so
[  2%] Built target pismcalcalcs
[  2%] Building CXX object src/CMakeFiles/pismrevision.dir/base/pism_revision.cc.o
[  2%] Built target pism_config
Scanning dependencies of target pismutil
[  2%] [  3%] [  3%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PetscInitializer.cc.o
Building CXX object src/CMakeFiles/pismutil.dir/base/util/error_handling.cc.o
Building CXX object src/CMakeFiles/pismutil.dir/base/util/options.cc.o
[  4%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/Context.cc.o
/home/ed/pism/src/base/util/error_handling.cc: In static member function ‘static pism::RuntimeError pism::RuntimeError::formatted(const char*, ...)’:
/home/ed/pism/src/base/util/error_handling.cc:37:49: warning: function might be possible candidate for ‘gnu_printf’ format attribute [-Wsuggest-attribute=format]
   vsnprintf(buffer, sizeof(buffer), format, argp);
                                                 ^
/home/ed/pism/src/base/util/error_handling.cc:37:49: warning: function might be possible candidate for ‘gnu_printf’ format attribute [-Wsuggest-attribute=format]
/home/ed/pism/src/base/util/error_handling.cc: In member function ‘void pism::RuntimeError::add_context(const char*, ...)’:
/home/ed/pism/src/base/util/error_handling.cc:56:49: warning: function might be possible candidate for ‘gnu_printf’ format attribute [-Wsuggest-attribute=format]
   vsnprintf(buffer, sizeof(buffer), format, argp);
                                                 ^
/home/ed/pism/src/base/util/error_handling.cc:56:49: warning: function might be possible candidate for ‘gnu_printf’ format attribute [-Wsuggest-attribute=format]
Linking CXX static library libpismrevision.a
[  4%] Building CXX object src/CMakeFiles/pismutil.dir/base/enthalpyConverter.cc.o
[  4%] Built target pismrevision
[  5%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/IceGrid.cc.o
[  5%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/ColumnInterpolation.cc.o
[  6%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/Viewer.cc.o
[  6%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/Mask.cc.o
[  6%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/NCVariable.cc.o
[  7%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMConfig.cc.o
[  7%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMComponent.cc.o
[  8%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMTime.cc.o
[  8%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMTime_Calendar.cc.o
[  9%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMVars.cc.o
[  9%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/Timeseries.cc.o
[ 10%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/iceModelVec.cc.o
[ 10%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/iceModelVec2.cc.o
[ 11%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/iceModelVec2T.cc.o
[ 11%] [ 12%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/iceModelVec2V.cc.o
Building CXX object src/CMakeFiles/pismutil.dir/base/util/iceModelVec3.cc.o
[ 12%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/iceModelVec3Custom.cc.o
[ 13%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/io/LocalInterpCtx.cc.o
[ 13%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/io/PIO.cc.o
[ 13%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/io/PISMNC3File.cc.o
/home/ed/pism/src/base/util/io/PIO.cc:136:3: warning: identifier ‘nullptr’ is a keyword in C++11 [-Wc++0x-compat]
   assert(m_nc == nullptr);
   ^
[ 14%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/io/PISMNC4File.cc.o
[ 14%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/io/PISMNC4_Quilt.cc.o
In file included from /usr/include/c++/4.8/cassert:43:0,
                 from /home/ed/pism/src/base/util/io/PIO.cc:30:
/home/ed/pism/src/base/util/io/PIO.cc: In member function ‘void pism::PIO::detect_mode(const string&)’:
/home/ed/pism/src/base/util/io/PIO.cc:136:18: error: ‘nullptr’ was not declared in this scope
   assert(m_nc == nullptr);
                  ^
/home/ed/pism/src/base/util/io/PIO.cc: In member function ‘void pism::PIO::open(const string&, pism::IO_Mode)’:
/home/ed/pism/src/base/util/io/PIO.cc:266:22: error: ‘nullptr’ was not declared in this scope
       assert(m_nc != nullptr);
                      ^
/home/ed/pism/src/base/util/io/PIO.cc:272:22: error: ‘nullptr’ was not declared in this scope
       assert(m_nc != nullptr);
                      ^
/home/ed/pism/src/base/util/io/PIO.cc:285:22: error: ‘nullptr’ was not declared in this scope
       assert(m_nc != nullptr);
                      ^
/home/ed/pism/src/base/util/io/PIO.cc: In member function ‘void pism::PIO::regrid_vec(const pism::IceGrid*, const string&, const std::vector<double>&, unsigned int, Vec) const’:
/home/ed/pism/src/base/util/io/PIO.cc:1126:19: error: ‘nullptr’ was not declared in this scope
     assert(lic != nullptr);
                   ^
/home/ed/pism/src/base/util/io/PIO.cc: In member function ‘void pism::PIO::regrid_vec_fill_missing(const pism::IceGrid*, const string&, const std::vector<double>&, unsigned int, double, Vec) const’:
/home/ed/pism/src/base/util/io/PIO.cc:1172:19: error: ‘nullptr’ was not declared in this scope
     assert(lic != nullptr);
                   ^
make[2]: *** [src/CMakeFiles/pismutil.dir/base/util/io/PIO.cc.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [src/CMakeFiles/pismutil.dir/all] Error 2
make: *** [all] Error 2
```


Is this correct or should it have been?
```
if (!isset($params['lastModifiedDate'])) {
...
```


Hrm, that sort of helped, I suppose. I don't think it's good enough.

The add code has this style:

If <var>track</var> is already in <var>stream's</var> track set, then abort these steps.

Could you use the same style? note that the verb is `in` there's no `present`, and it uses `abort these steps` instead of a complicated if.


On Mon, 5 Jan 2015, Danny Al-Gaaf wrote:
> Then we may should move this code block into the error handling of dlopen and not checking it before.

Yeah that works!


Nice change. It makes it more clear what is happening.

If you want to go a step further you could change the method name to `mfa_devices` and remove the need for the local variable assignment. If you go that route, you would probably want to cache the result in an instance variable to avoid additional API hits each time the method is called:

```
def mfa_devices
  @mfa_devices ||= JSON.parse(`aws --output json iam list-mfa-devices`).fetch('MFADevices')
end
```


We should use standard names for parameters. MSDN says "...Your cmdlet should use standard parameter names so that the user can quickly determine what a particular parameter means...."

For example, this field in Azure Powershell is called "Location". Suggest we use that.

Do a "Get-Command -ParameterName Location" to see where all it is used.


Fair enough; I realized after adding the comment that timeouts could be
vitiated (it's really silly that DataSource.getConnection() doesn't have
a timed version, given that a DataSource has a loginTimeout, which is
really connection-timeout - ahh JDBC), although I'm sure you're aware
that #borrow() doesn't offer *hard* timeout guarantees (nor should it).

I guess I just care more about max throughput/latency than timeout soft
guarantees, but like so much else, it depends (and there are definitely
flaky drivers that don't respect DataSource#loginTimeout - so an unlucky
thread might wait up to a second or more)

Of course there is no upper bound for the latency of releaseShared ->
unpark-last, it depends on innumerable factors, but it's definitely
O(waiters). I have benchmarks of AQS where the last thread to
acquire/unpark is well above 20ms (with waiters > 100), but that's a
digression.

As far as disabling maxIdle - I totally agree; and in that case, it
makes this discussion essentially moot - it probably can't be emphasized
enough

On 2015-01-05 05:35, Brett Wooldridge wrote:
> Here we have a fundamental disagreement. addBagItem() cannot be
> executed on the caller thread. getConnection(), and therefore
> ConcurrentBag.borrow(), provides strict time guarantees. Either you
> will get a connection within connectionTimeout or you will get can
> exception. addConnection() does not (and cannot) provide strict time
> guarantees and therefore cannot be executed on the caller thread.
>
> Moreover, the user does not care where the Connection returned by
> getConnection() comes from (nor should they). If it happens to be a
> new Connection generated by addBagItem() that is fine, but it is
> _equally_ valid (and more likely in an active pool) for the Connection
> to be one returned to the pool by another thread. If a caller to
> getConnection() were ultimately blocked in addConnection(), even for a
> few hundred milliseconds, it wouldn't matter if 10 Connections became
> free in the pool because that thread would remain blocked waiting for
> the underlying DataSource. The overhead of an asynchronous handoff is
> 20ms at the _outside_, but cost of blocking a thread on connection
> establishment (and missing opportunities for returned connections) is
> almost always higher.
>
> The current design was not accidental -- asynchronicity is there for a
> reason, and it is quite deliberate.
>
> If you have "spikey" loads, as our documentation suggests, do not set
> minimumIdle and allow HikariCP to operate at its default -- a fixed
> size pool. In 90% of the codebases I've encountered in 25 years of
> coding, there was never a solid justification for dynamic pool
> allocation. But that is getting into a much deeper debate.
>
> --
> Reply to this email directly or view it on GitHub [1].
>
> 	*
>
> Links:
> ------
> [1]
> https://github.com/brettwooldridge/HikariCP/commit/8aa2034709ff321e8a910a2c1c450edc0c6ea7b0#commitcomment-9153352


Can we keep the PEP8 import order here (alphabetical, "import blahs" before "from blah import blahs").


Please verify that this code can run without elevation.


merge


I copied that description from the `nix.conf` man page. :-)


@bueler: Thanks! (Input is always welcome. I still need to get GCC-4.9-plus-MPI working on my laptop, but that's a low priority for now...)


```optional="true"``` is not needed for a text parameter (not having a validator).


This fix is not on the latest PyPI released version which makes this app unusable :(
please release a new version ;)


```$fraction``` is a text parameter, so it should be quoted.


This will make it into the next 1.6 release. I tested it with both the latest pip (6.0.6) and the one I've been using (1.5.4) without any keyword argument issues. I'll see if I can test with other versions to confirm. Worst case scenario is I'll just have to not use pip to parse requirements here.


Kai, duplicate for my name in the Authors file


Same reason :) We can add them now


```$basename``` is a text parameter, so it should be quoted.


Hehe, ninja'd.
I did "Landvehicle", "Air", "Ship" =)


Good to name this method as GenerateOrGetChannelIntegrityKey( ). Get has a connotation that the method won't have side effects - which isn't the case here.


Hmm, I don't know. Working for me now.