Right, I'll use bs_direct everywhere. No need to compute it twice


Maybe it doesn't make sense to put the C-w thing here. What do you think @syl20bnr?


It need to be include into the tcc file no ?
Of course it maybe the same, juste simplify the reading.
Can stay like that, not a major issue.


It appears that this made editprojecthook push=False by default, however the api has push=True by default.


Ta got that one :)


these training dummies in the script above have about 100 hp, they are instantly dead, noone cares about the threatstuff.


Well, I'm dumb. I have no idea how I managed to not notice this when I had done it correctly moments earlier.


if buffer is zero-sized *nothing* will be output, i propose we wrap buffer contents in `[` and `]` so empty buffers are also visible


Thanks fixed too :) Still works :+1: (*client at least*)


`(unsigned int)` is faster/better here


It's a very tiring thing to do though. For example look at [this](https://github.com/roxas75/rxTools/blob/master/rxtools/source/lib/draw.c). It looks simple enough, you can trace everything back to roxas in git history. But then you see [this](https://github.com/archshift/Decrypt9/blob/master/source/draw.c). See the copyright at the top?


also don't use C-style casts


if not distance_to_us and distance_to_current_room then you should update the queue, as if other_score > our_score.

In this scenario the room they are trying to reach is unreachable but the other room is not.


Yep, in my mind hybrid is to use emacs state in place of insert state so we can let this kind of more advance hybrid tweaking to the user.


v2ex观光团！！


str("") and clear()? why


ok


instead, create ss in each block


ok, I'll remove it now


this should be a new file


Am incercat si imi merge, adica nu apare eroare sau failure, dar o sa modifica ca sa nu am de 2 ori


Can you re-add them? :)


Add tests for 0-sized buffers.


If someone want script this npcs differently, just do it, comments will change nothing.


Nice!


@ck86 Why? It's just extra bytes


It happens to everyone. I bet you wouldn't notice it, no matter how long you looked at it.


Hmm, we may open a new issue. It's not so easy to know the source of every source code xD


@ck86 Oh, I understand. I just will add a linbreaks.


https://www.youtube.com/watch?v=nuHfVn_cfHU :D


hmm yes fair point, that had not actually occurred to me


Returning nil nil won't break anything


Maybe this deserves an explanation of the term "orphan" to the user.


I was thinking about that, but with MPTCs it's not so easy to state in a short sentence - figured the error page could elaborate. If we can come up with something good for the message I'm all for that though.


@remh all set


Authorship :3


please open a pr for ompi-release v2.x for this commit


Yes. The two instantiations create extra database calls that are not needed and therefore should be removed.


You also have to take the "\tSet str = \"" out or it'll look for an object.
i.e. delete line 78 and replace lines 79 and 80 with:
for chunk in chunks[]:
     payload += "\tstr = str + \"" + str(chunk) + "\"\n"


A simple exception seems the way to go for me. The eval makes the behaviour harder to understand.
We should go either the eval way and remove the additional classes, or implement a simple exception, but not both.

By the way thank you @sebastianbergmann for being so effective in applying those patches.


That didn't seem very natural to me either, but the only other option that I first had was a `setPath` method, but this solution is quite painful to use as you have to create the object and set the path because this second step is mandatory. Unless you create a factory, I haven't given more thought to this solution but I guess we can suggest a factory example in the documentation (we might also need one in the bundle implementation, such as the twig extension).

Another solution that come to mind is the usage of a static method : `ImageInterface::createFromPath`. I'm not very found of these kind of methods but it can be useful in our case.

Which one sounds best to you ?


Lel, no authorship required here. Found these fixes earlier on the net as well.


Duh!! Was reading the wrong lines :smile:, will fix.


`seekg(0)` could be used instead of closing and reopening the file.


Ohhhk delete all these comments then :v
On Aug 13, 2015 7:23 PM, "Ateek Ujjawal" <notifications@github.com> wrote:

> Lel, no authorship required here. Found these fixes earlier on the net as
> well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team-Radium/hardware_qcom_display/commit/8724930ef740508b6d5d74c9f85ab605b84f43e4#commitcomment-12689836>
> .
>


Sign should test for `x==0`


It runs when the user logs in as admin. The script is called setup-when-root but runs when the user logs in as admin.

TODO: Change the name to setup-when-admin.sh
TODO: Remove from /var/scripts


Needs a comment to explain that this is a compatibility shim for old code.


Hello JM!,

Hope you are doing good man! :)

On Wed, Aug 12, 2015 at 7:27 PM, David VanHook <notifications@github.com>
wrote:

> Wait, do they let Australians have Github accounts??
>
> ;)
>
> Hope you are well, JM!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mshanken/Publications-Form/commit/9bbb83539ae59a0535e3d7abd54e5144d6efa2ee#commitcomment-12677886>
> .
>

--
*Joan Nonon - Software Engineer*


It works flawlessly. Don't know why that error popped out at that time, I rechecked everything and it works out just fine. Thanks Matthias, sorry again for the noise.


It's needed indeed.
I intend to remove some hard coded values before the PR, especially the image quality in this line : https://github.com/skwi/NovawayFileManagementBundle/commit/6009e4c28fe1f512ff177ee27304ff482c078af5#diff-84c9193288db56d8889e42fe9943ad8cR27


No need to keep extra setw and setfill here.


project_id filtering is still returning the wrong values.


Whops! I think I pushed the wrong button. :-1:  @dscho shall I revert his or rebase and force push to remove this?


```
bool first = true;
for (auto& c : s) {
  os << (first ? "" : " ") << ...c << ...
  first = false;
}
```
throw out the if and the rest


This is awesome. Thank you for that!


No.
It's actually documented from a video that I saw on youtube.


Welcome and enjoy working on this source to help better the LC community!


I've noticed with TravisCI failure that image filesize comparison is not a sure solution for testing image manipulation, but I'd like to test more than just the file existence. Do you have any solution in mind ? I'll keep looking in the meanwhile.


Looks good, the message should be consistent with ‘Mark as complete’ button. But I think we need to go through the language around ‘Make complete’ / ‘Mark as complete’ as a whole thing.

*In summary LGTM :+1: *


* TODO: rewrite this as it is doing way too much and is very messy now.


Please do full-upgrade. :)


I already fixed the variable name in a later commit, but didn't notice the reversed logic.


good idea - changed into container_of


I'm not sure exactly what else you want written that actually pertains to question #2. What you've explained here doesn't explain why `ok: 0` and `ok: 1` generate failed and succeeded events. It was simply if the server deems a command successful it gives `ok: 1` and if the command failed it gives `ok: 0` and we stay in line with server behaviour. Write errors are a separate thing. It seems that you're asking a potential question #3 "Why aren't write errors or write concern errors treated as failures?"


了解ですー。先ほどマージしました。
余談ですがプラグイン出力も $modx->event->cm->addOutput('ほげ'); で追加できます。
これも例によってプラグインのネスト実行でも出力が混ざらないです。


Tabs are evil :imp:


It was just a simple one-time function so I could get a list of all the messages received. Nothing special.


The arrows should show up when ncui is running, so it's probably stopping as soon as it starts. Could be because ncsvc isn't running. Is it executable? This app should spawn ncsvc on first log in, so once the arrows disappear, you can check if it's running with "ps aux | grep ncsvc". It should be running...


Ahh, pensei que fosse a data! kkkk :smile:


Tabs are evil! It looks fine on my screen but looks crap on here.. lol


MarkdownTable?


no break


need default


indeed, looks better


it's invalid to have `;` here


Right. Added to the TODO


fixed. See commit 50755bc9f3e26dc0c374a6ae44d5fcfe8a0cb79f


Shouldn't you use `SOL_BUFFER_FLAGS_MEMORY_NOT_OWNED`? That's exactly your comment there "...it can't be resized and free() should not be called at it at sol_buffer_fini()."


Since this is a new driver, there is no meaning to the order of the patches.
I will be compilable only after the last patch with makefile is applied.
I will try to put as many .h files at the begining.


`docker_args` seems an obvious name, but `--docker-args` sets `user_args`. I guess I should use `config_docker_args` and `cmdline_docker_args`?


Just spotted this typo as well.


Keine Ahnung was dieser Commit macht, der wurde ausversehen durch einen falschen Buttonklick bei GitHub Desktop erstellt.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.907.99337e636e0a0536946287c58b86e0ccff04408c](http://build.palaso.org/viewLog.html?buildId=61119&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:03:44


No, it isn't.  It would probably be better to have console.log though.  Submit a pull request if you like.


@foresto can you please provide a source that confirms this?


It's actually just the names of the files being stored on disk... I was getting lots of python ascii encoding errors from all over the website when opening/saving/closing/moving files with unicode in the title - it seems to be a lot simpler to escape the file names when they are created.

The objects in the DB do have Unicode though. The movements named `erhör’`'s files will be named `erhor_[composer name]_file1.midi1`


> A simple exception seems the way to go for me.

I very much agree. Can we please change this @sebastianbergmann.


It seems ok, but those macros don't inspire much confidence in my judgement.


It translates to max_replace.val


Actually, this test should fail because the loop body has a call to `m()`, which throws an exception according to its header. Do you agree?


TeamCity Jarvis Framework :: CI-GitFlow [Build 35](http://build.prxm.it:8811/viewLog.html?buildId=3679&buildTypeId=CqrsFramework_CiGitFlow) is now running


Both numbers seem a bit out of thin air to me, but I have no objections to this.  It's just a default in the end.


Can we get something .env-like?


The problem is I need the goto variants a lot :( ICU should provide these helpers, that would be ideal.


why?


To be clear, you are **adding** a test case, not changing an existing one. Correct?


we have different tabsize here.
Please use 2spaces tabs ,-)


na->up.ifp is sometimes null


use constant and logs


no because if the user said "fixed_capacity" but not "no_free", then you want to free it.

This is the case if the user allocated the memory and wants sol-buffer to free it, but not resize. In this case `& MEMORY_NOT_OWNED` would be non-zero, since it would match the bit for `FIXED_CAPACITY`.


Do you have an example of how to test this by any chance? I tried this but my spec file run failed with NoMethodError: undefined method `addormodify_selinux_policy_fcontext'
```
  it 'creates selinux policies for the file contexts' do
    expect(chef_run).to addormodify_selinux_policy_fcontext('/apps/bweb/apps/.*/bin(/.*)?').with(
      secontext: 'tomcat_exec_t'
    )
  end
```


na->up.na_hostvp is sometimes null


Checkmarx scan results 13 Aug 2015 17:17 UTC +03:00

High
Medium
Low
Info

New
1
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

1e6db40957da8a680b26fa003325c70e7e6fd5a2	@dortal


On 13-08-2015 11:16, Gustavo Lima Chaves wrote:
> The problem is I need the goto variants a lot :( ICU should provide
> these helpers, that would be ideal.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/glima/soletta/commit/c7d7b3d2808e6690afce9841c01cd403e657b9ab#commitcomment-12690286>.
>
And lots of free errors (missing or in excess) appear =/


done


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.908.2b5686114d04a8beea965a966feb2e119ec13517](http://build.palaso.org/viewLog.html?buildId=61120&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:03:23


Can we have a message here?


Why, the code compile, if we throw new Exception() in any of those method then it should fail, isn't


This folder is a Web App " start up".daukantas/gdaukant..github.io


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-unstable.4+4](http://build.prxm.it:8811/viewLog.html?buildId=3679&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 122 Build time: 00:03:47
### Failed tests
```
Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.BasicRebuild.start_then_rebuild: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.ProjectionEngineBasicTestBase: <no details avaliable>

```


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.429.d8aab02ee9fcc96791c4081c03cf32040982c68b](http://build.palaso.org/viewLog.html?buildId=61118&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:20:01


This looks like it's missing a space.


~$cd username.github.io

~$echo "Hello World" > index.html


We will remove oe_button classname from every button, but this is not the scope of this task. You can revert all your changes in web/. Obviously, the btn-primary style won't apply on buttons having classname oe_button, but it is ok for now.


Yeah, makes sense. I misunderstood the comment, maybe because of bit '...resized and free()...', '...resized or free()...' maybe could've avoided that.


fixed thanks.


The modals of the DebugManager haven't been updated (ex: Edit Action)


Right again, adding a note on it to the TODO ...


AOKP has already been out of active development for more than a year......
I'm looking forward to seeing AOKP's back :) Cheers!


Why are you using static variable for RTC? each reset will reset RTC, is that desired? I noticed this for other ST as well.


use constants


The run time type should be an `IMethodBinding`, correct? Can you confirm this with the debugger? If so, please cast the binding to an `IMethodBinding` and then see, from there, if you can get the thrown exceptions.


Thanks swesty. I updated the module. All that was needed was to remove the "Set" portion of str instantiation.


Same as the code above. Can we check here the peripheral if it's running?


should it support the optional 3rd arg of `fwrite`?


`min_iops` is an integer, and i'd suggest to use the in-class initialization

```patch
diff --git a/src/common/obj_bencher.h b/src/common/obj_bencher.h
index 832461b..e349a60 100644
--- a/src/common/obj_bencher.h
+++ b/src/common/obj_bencher.h
@@ -20,10 +20,10 @@
 struct bench_interval_data {
-  double min_bandwidth;
-  double max_bandwidth;
-  int min_iops;
-  int max_iops;
+  double min_bandwidth = DBL_MAX;
+  double max_bandwidth = 0;
+  int min_iops = INT_MAX;
+  int max_iops = 0;
 };
```


Not that kind of don't understand… Ask @PEMapModder


I think cabal is free to choose time < 1.5 in all cases, by automatically toggling the old-locale flag on (and requiring old-locale). Could it be that you need to make old-locale available somehow ?


why not `$this->incrementor( -1);`?


You can't refer to `_self` anymore


:+1:  Yes


I answered in the wrong place.
What I meant is that the shared queue between user to kernel is something similar to our PRM and is specific to the RXE. Therefore it might be different than the verbs.
I will add a note on it in the TODO.


I like this idea a lot. Let's wait and see if fishtest also likes it.


Unfortunately, this bug fix has only made it in for the `-file` parameter. BTSX1 still causes Choco to crash if run with `chocolate-doom -merge ~/Downloads/btsx_e1_99i9/btsx_e1.wad`. If it doesn't crash immediately, try appending some `-mb X` parameter with `X` being small values such as 5 or 6.


Why this `do...while(1)`? Is it because this function is `inline`?


I've never seen a .stackdump file. One thing I've been doing in my own projects is making sure I have a local .gitignore file - so that  *.bak, *~ and the other more local-specific to yourself would go into there.


It doesn't - but kernel is using it.


Forgot to add a description to this change - just added kit to each rank


Well, try converting the for loop in `A.m()` to a lambda expression and see if it compiles.


> It would be nice if all Git on Windows users, including people who regularly do work in an msys2 shell beyond just building Git for Windows, but are not Git for Windows users, could benefit from (e.g.) the fscache patch. :)

Right. So if you call the Git Bash, or if you call the Git SDK's Bash, or (in the future) if you use MSys2 and call the MinGW bash (and have MinGW Git installed), you *always* get the version of Git that knows about fscache. Because you would always use the MinGW Git.

Which is good, because MinGW Git is much faster than MSys2 Git, according to several users' accounts.

Sometimes, however, most notably when developing MSys2 software, you will *want* to use the MSys2 Git because it knows about the POSIX emulation of symlinks, of `/dev/null`, etc.

Eventually, (i.e. when I am undistracted enough to actually get Git for Windows 2.x out of the door), I will submit all, or at least most, of Git for Windows' patches to upstream Git. If nobody else finds it important enough to submit the fscache patches upstream, then those will be part of my submission.

Therefore, we can simply drop this patch because everything works as it should already, right?


Ah, nevermind, found the case where it does not return.


Consiidated → Consolidated


oh, needs to be
```
$inc = $this->incrementor;
$inc(-1);
```

`call_user_func` has a perf penality for php<7


@alalek I did several changes but when I compile I get the following error at linking:

Linking CXX executable ../../bin/opencv_test_sfm
../../lib/libopencv_sfm.so.3.0.0: referencia a `typeinfo for cv::sfm::SFMLibmvUncalibratedReconstruction' sin definir
../../lib/libopencv_sfm.so.3.0.0: referencia a `typeinfo for cv::sfm::SFMLibmvEuclideanReconstruction' sin definir
../../lib/libopencv_sfm.so.3.0.0: referencia a `typeinfo for cv::sfm::SFMLibmvProjectiveReconstruction' sin definir
collect2: error: ld returned 1 exit status
make[2]: *** [bin/opencv_test_sfm] Error 1
make[1]: *** [modules/sfm/CMakeFiles/opencv_test_sfm.dir/all] Error 2
make: *** [all] Error 2

It looks like don't override the virtual functions


Core translations :) That is awesome!

Could you remove `dist/dialogs.html` from your repo so it won't get into the master branch ? It's not supposed to be there :)


The kcauldron builds from that time frame were extremely unstable.

Try this one instead:  http://tcpr.ca/files/cauldron/cauldron-1.7.10-1.1307.06.218.zip

I BELIEVE all the blightfall mods are compatible with that forge version.


I wouldn't have introduced an else{} branch, there's a return anyway; the patch would be smaller and does the same. But apart from that, this does the trick.


Would it be a problem if I try to integrate that branch into this?


Okay ;-) No worries, I force-pushed myself many times in the past for the same reason ;-)


This is intentional. PHP7 is the only environment that matters. If people need better performance they have to upgrade their php install. 5.5 and 5.6 are supported but we don't waste time optimizing for those environments.


Correct the title.


Why are .pl files in your lib directory


there's a typo in the commit message :P "synce"


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.9.d8aab02ee9fcc96791c4081c03cf32040982c68b](http://build.palaso.org/viewLog.html?buildId=61117&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:34:21


No because the API must be consistent across all the handle implementations and the additional argument is completely superfluous and should never have been added in the `fwrite()` API. Simply pass the string you want to write and no more.


Checkmarx scan results 13 Aug 2015 17:34 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

1e6db40957da8a680b26fa003325c70e7e6fd5a2	@dortal


Ack.


Approved :+1:


Thanks so much for the regular updated. I really look forward to have the browser work because of a webrtc app am doing. But my present experience is such that, when i load a webpage that is not available online, the app loads and gives me an alert that the page isnt available. But when i load an online page, the app crashes. how can you help. thanks alot


cool!


They both provide the same output for the testCase
https://gist.github.com/mdarefin/e1364a347d0927e3c4c8
```
Out Put:
name of method: println
This is IMethodinding: public void println(java.lang.Object)
name of method: checkArray
This is IMethodinding: static void checkArray()
name of method: m
This is IMethodinding: public static void m()
name of method: throwException
This is IMethodinding: public static void throwException() throws java.io.IOException
```


Damn cmake :P


Please use repoman next time to prevent bumps with stable keywords.


You have added this to LivingCreature as well, so you don't need it here.


Guess this can go now ?


Mark this virtual functions pure:
```
virtual void run(...) = 0;
```
Same for "run" functions below.


Should be fixed in fad2e8a3ca5481fb163eb5b75a059039e7f3aea4.


they are mandatory, looks like.  testing now.


This revision is clean.


:smile:


It works as well as it's going to for now, yeah--we can forget about this.


Patch looks good.


@Dani723 could you post upload-file component to ember-vcl?


To add this, my OS version is 8.3, and the code is also fine on simulator


can remove the `<'a>`?


Guess this can go now ?

On another note, will you squash the commits together before you merge them into master ?


I prefer `{}` operator on strings when reading it char per char. makes it easier to disambiguate vs.array


can you add the explanation to the commit msg?
```Tests fail because the reactor is "unclean" when it is stopped. In this case, the cached http connections were still hanging around somewhere, and closing the sync_target removed the cached connections.```


That´s what I've been saying. A bit like cheating.

But JJ, you convince me that is the nature of the project.
What we could do about it IMHO:

1. Just report it like it is. When we compare, all algorithms must be
implemented the same.
 Justification: We don´t try to have control of everything because that
will have a cost.
In real experiments we care about finding a solution eventuality.

2. Try to restart all workers when there is a new experiment:
2.1 Just like EvoSpace, after every iteration, do another GET to the
server: are we finished?
If the experiment has finished restart the worker,
Problems: Is asynchronous, there can be slow workers that never do anything
because
every time they ask the experiment is over, or the experiment has finished
more than one time.
2.2 MVCC style, every worker is assigned the current ExperimentID, just
like 2.1 but
he can only work for his ExperimentID, if there is a new one, the PUT
chromosome request
fails with 404 or must first look with a HEADERS if the resource has
changed and the Worker restarts with the new ExperimentID.
2.3 Try to get all of the participants in sync. Totally against the
asynchronous idea.

On Thu, Aug 13, 2015 at 3:06 AM, Juan Julián Merelo Guervós <
notifications@github.com> wrote:

> Adds processing including number of puts
>
> Which reveals a curious fact. In many cases, experiment take a single PUT
> which means that it is probably a run over from workers running previous
> experiment. I don't know if that's a bug or a feature, but I think we would
> have to deal with it somehow, right? What do you think @mariosky
> @pacastillo @ccottap @fergunet?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/JJ/modeling-volunteer-computing/commit/2a21984067c911a7ce2c36f3f2f7524762e7f043
>


I think this is redundant, isn't it? The start of the line is saying the same thing as the first part of the added text. Is there a reason for the second part of the added text? It seems to me that if everyone is able to see real JIDs (non-anonymous) that unaffiliated users should be able to see them.


@bhklein I am not following the logic here. I thought if crossValidate was negative the first partition would be used for test, and the remaining partitions would be used to train a single instance of the tranform. However, this seems to pass all the data in as training data.

Empirically, when trying to use this for a bootstrap sample, my algorithm is training on all the data as well.


PS: I was missing a `.pop()` method for the set here, as in Python: remove and return an arbitrary element of the set.


To your question, no - it is a single port device.
Do you know what does this cap means? Or where can I find the description of the IB capability bits?


That's not the right way to use `dt` or `t`.

Don't concatenate message, set message to dt('Put all the...') then concatenate the warning.

```php
$message = dt('Put all the...')
return $message . ' ' . dt('Moving modules...');
```


Vecīt, šito nu gan nevajag publiski atstāt... Bet nu jau par vēlu, pat ja uzkommitosi labojumu, tad vēsturē tie konti paliks, var vienīgi dzēst repository.


I am not sure whether it is good idea to reuse  _MAX_PATH for long path name in PAL - the difference in the name is very subtle.

I think it would be better to introduce a new LONG_MAX_PATH define.


Just double checked, and I'm not even using the library - I recall I was going to use it for the example. It does raise a good point though - @stable will always use the latest tagged revision. Now, with the UUID library, that will soon (as of Aug 2015 or so) be v3.0.0 or more - which, if the code was actually being used, would be a BC break (for the namespace alone).

So, that is why "@stable" meta-constraint isn't recommended. In this instance, it would probably be the most recent "~2.8" (which could go up to v2.9999.999 etc) and if it followed the SEMVER standards, shouldn't allow any BC breaks at a v2.* version.


We've 5th codina revision(AFAIK), so there condition "system_rev > CODINA_TMO_R0_4" is met (and given line won't run)..


or it's tightly connected with rhea?


I guess in tests we could save us the root-namspacing (no perf requirements)


Thanks! I've removed the else branch and updated openSUSE/kiwi#479.


What happens if we queue up multiple syncs but we need to cancel them before they're finished? (ie, shutting down). I guess they pile up as deferred chains, but will they receive a cancel signal?
Not something for now, but good to know if there's pending work in here.


Have no idea where did it came from - but could be increased or be a tuneble in a later version


cc @shortdudey123  :-)


If the absolute value `crossValidate` is < 2, then you are asking the algorithm to crossValidate with a single partition, so  it will train on everything.


Hasthe golr-table change been fully tested? Safe to get this one into production?


I think this comment is no longer needed


I would comment that in this module the sync method is not protected against concurrency, and it's reponsibility of the caller to ensure it (as it happens in api).


FYI @casecode forgot to PR this one in, but diff is: the addition of https://github.com/clearhead/clearhead/blob/master/src/get-param.js


They shouldn't be.  They are set to omitempty, which means they won't be in the json if they don't exist in the struct.  afaiu anyway


Why change it?


This allows the extender to take options in to consideration.

Use case: extend data based on particular option.


Removed


Of course false, and you won't need to add anything to DataDelegatingCrudResource and MetadataDelegatingCrudResource


Here is my testCase I run into Lambda Expression, it runs
https://gist.github.com/mdarefin/6dbd96c08c8ec693333f


Hmm I don't think it's a good practice to mix imports and assignments. Can you move these after the imports?


yeh we converted it into dynamic debug - seems that we missed this one.
Done.


You're right. Thanks


But it's unused elsewhere besides for calculation pixclock(which is for showing only).


Ignore me. It obviously helps if you upload the new commits to chef server first :)Working fine now thanks!

Sent from my Samsung device

-------- Original message --------
From: BackSlasher <notifications@github.com>
Date: 13/08/2015  15:45  (GMT+00:00)
To: BackSlasher/chef-selinuxpolicy <chef-selinuxpolicy@noreply.github.com>
Cc: equick <edwardquick@hotmail.com>
Subject: Re: [chef-selinuxpolicy] Add ChefSpec matchers (5060057)

cc @shortdudey123  :-)

---
Reply to this email directly or view it on GitHub:
https://github.com/BackSlasher/chef-selinuxpolicy/commit/506005790b54ac0ee13374b0a82951531c42a025#commitcomment-12691027


Just noticed that I've renamed the file PersistentFSMActor.scala into PersistentFSM.scala.
@ktoso, have you meant to leave the file name unchanged for some reason?


Sounds good.

Sent from my iPhone

> On Aug 13, 2015, at 5:06 AM, Vivek Goyal <notifications@github.com> wrote:
>
> Good catch. I think I like underscore version better then hyphen one. So I will change commit message.
>
> —
> Reply to this email directly or view it on GitHub.
>


Hmm, as libvirt and qemu use different default, how about using:

```
# Force net type (bridge, user, none)
nettype =
```

and use different defaults for libvirt and qemu?


After using a single method called isRetireableOrVoidable, how do i tell whether to add description.addProperty("retired"); or description.addProperty("voided"); ?


thanks. Working!


TeamCity Jarvis Framework :: CI-GitFlow [Build 36](http://build.prxm.it:8811/viewLog.html?buildId=3680&buildTypeId=CqrsFramework_CiGitFlow) is now running


I am if it visit a throwStatement then it's throwing a exception, I think we could work on that, but I am guessing the problem with throwStatement is what if there is an if loop and like
if(i==0){
throw new Exception();
}
even though the i never bo "0" should it visit the throw exception than it breaks.


Mm, you enlarge the buffer but does not copy what was missing?


Nit: perhaps make that a level 3 heading.


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1804 of 1805 tests passed)

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


Ah, the damn loop. OK.


.... and here are the counter-examples to bite me in the butt. Sami probably could stay at stable since it is quite independent of the code - though, again, if it was being used more deeply (maybe with custom templates), it might break the build system to jump from v1.3 to v3.0. Checking it is OK at v3.* and then so setting the constraint to ~3.0 would probably be best here.

For PHPUnit, it's quite fast moving - for OSS code. Certainly setting it to something like "~4.7.5" for the entire 4.* series at "~4.7" (and it's already at 4.8.3 now) is probably better on the whole.

PHPDox is probably a lot more stable at a release version that when I wrote it - I recall the dev-master at the time had a serious bug, hence locking to a commit before that or so.

If you would like to put those ideas in, rebase to clean it up and let me know - I'll merge them!   And Thanks You for caring!


This works for me:

```
expect(chef_run).to addormodify_selinux_policy_fcontext('/apps/bweb/apps/.*/bin(/.*)?').with(secontext: 'tomcat_exec_t')
```


done


I have a different version -- I'm afraid this one (for the no builtin path) is faulty. Check my PR.


should not we require 1.8?


This should be fixed in v6r13
https://groups.google.com/forum/?hl=en#!topic/diracgrid-forum/_y9D3-XlMmI


Is it only for CQs?


@schleichardt What benefit do we have from activator in Sunrise? Are we going to provide templates and generate projects out of it?


Oh, and by default, mocha will look for a directory named `test`, so `"test": "mocha",` is actually sufficient :)


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-newSlotOnrebuildAreNotRebuilded+1](http://build.prxm.it:8811/viewLog.html?buildId=3680&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 114 Build time: 00:03:21
### Failed tests
```
Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.BasicRebuild.start_then_rebuild: <no details avaliable>

```


TeamCity Jarvis Framework :: CI-GitFlow [Build 37](http://build.prxm.it:8811/viewLog.html?buildId=3681&buildTypeId=CqrsFramework_CiGitFlow) is now running


Looks safe to me.


nice job, but:
1. please make your commit smaller, then it will be easier  to discus about  each part.
2.  why you edit the ".gitignore" file?! (i guess it was an accident)
3. before commit please first check codes in your own sys! (I'm 100% sure if u run this code you will get many many errors)
4.  "admin_delete_patient.php" is an admin page, and you use "$_SESSION['patient_id']" in it?!!!
5. and so many other bugs! first run it in your sys and fix them, then recommit them.


> It would probably be better to have console.log though.

Exactly. That's why I #1 to add it and some ther things.


知错能改，善莫大焉


Looks like canola is misspelled!


These errors are logged in the service level in OAuth2Service. e.getMessage() is replaced by a custom message in a later commit. https://github.com/nuwandi-is/carbon-identity/commit/5100fa4#diff-5c968bc1e0baf0f6d874457bf98e3a7aR53


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Hey guys - should this have been merged? (See commit message)


It is a good idea to check for self-assignment -- to make sure (&rhs != this) -- before modifying anything.
It might be fine here, depending on how memcpy works on various systems when passed the same pointer for first and second arguments.
See:
https://isocpp.org/wiki/faq/assignment-operators


I think this might be better as:

> A MUC archive MUST check that the user requesting the archive has the right to enter it at the time of the query and only allow access if so. In a members-only chat room, only owners, admins or members can query a room archive. In the case of open MUC rooms, the MUC archives can generally be accessed by any users (including those who have never entered the room) who do not have an affiliation of 'outcast', but a MUC archive MAY further limit access based on other criteria as part of the deployment policy. A MUC archive MAY, if it stores historical data about previous configuration states, limit the results returned to only those that the querying user would have been authorised to see at the time (e.g. it MAY limit the results to not include results while a user was an outcast).

or similar?


:trophy: any chance of some screenshots? :smile_cat:


Then i guess just check the resource to see if it DataDelegatingCrudResource vs MetaDataDelegatingCrudResource  and get rid of this method? Is this what Lluis' code was doing?


Seems sensible.


This warning will be printed also for the larger mmap case


```
It displays correctly, bad formatting on github
```


I think Duplicity should also depend on the gpg agent package you created, because it won't work without it.


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-unstable.4+4](http://build.prxm.it:8811/viewLog.html?buildId=3681&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 114 Build time: 00:02:16
### Failed tests
```
Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.BasicRebuild.start_then_rebuild: <no details avaliable>

```


The (index >= 1000 && index <= 1010) range check and the return LuaProxy::Player(i + 1001) don't quite make sense together. Something has an off-by-one error. Note that that range check allows 11 indexes as valid for the playerTemplates array which I'd tend to suspect is supposed to have 10 valid entries (5 characters for two players?)

Did you mean (index > 1000 && index <= 1010) perhaps? Or did you want (index >= 1000 && index < 1010)


Perhaps this could be moved to the "Contributing" section?


This method has been removed by the commit https://github.com/nuwandi-is/carbon-identity/commit/4e39d15915988c12eb36885a16a1f950ed107651 and strict validation is checked in AbstractClientAuthHandler class.
Added debug logs in AbstractClientAuthHandler. https://github.com/wso2/carbon-identity/compare/master...nuwandi-is:refac#diff-29840da4853cd05a291755a466df6d4bL21


Hah, you're right. Thanks again.


Added debug logs https://github.com/wso2/carbon-identity/compare/master...nuwandi-is:refac#diff-29840da4853cd05a291755a466df6d4bL21


I think this is slightly misled. The "Occupant JID" in XEP-0045 is room@service/nick, so the nickname is already accessible.


_day_cell_big.html template doesn't exist


Is this basically reverting the previous commit?


Is there code to release pending mmaps if the process dies before it mapped them?


For loading the demo record, check how it is done here:
https://github.com/inveniosoftware/invenio-records/blob/master/tests/test_record.py#L40-L42


Ok Roos. Just to be sure: now you need to create an array of DD-topics, and use the iteration ( ` ... .all do | l | ` ) to pick one topic out of the array.
Not saying that is a better way to do this, but it is the array exercise :-)


The changelog can be improved. Write what kind of resources are mapped and a high level description of the flow (creation -> mapping -> unmapping -> destruction)?


And what do we do for resources that do not sub class DataDelegatingCrudResource or MetaDataDelegatingCrudResource?


Added this fix to my badge, LEDs not working more reliably. Is there any way to inspect this issue further?


I guess i see what you saying, you can ignore my comments, sorry!


Hi Gu,
I like the use of binary number notation here to identify the available gains... However, as a rule-of-thumb **every** switch statement must have a 'default' case. Otherwise you risk passing through the switch without any effect and the code might then carry on as if everything is OK when it's not.

In this case your 'default' case could either raise an exception (invalid gain value) - or even throw an 'assert' (as the HW does not support more than 2 bits of gain, we should never ever get to the 'default' case).

Cheers,
Ulrik


Ok. I got the inspiration from https://github.com/inveniosoftware/invenio-workflows/blob/e044307d22abaea4dfbb956b78976d1246a60b70/tests/test_workflows.py#L75-L108


You accidentally copied the plus.


for loadmore adapter


1043, 1044, 1045 and 1046 should have `RollerCoaster Tycoon 2` replaced with `OpenRCT2`.


It will actually be faster *not* to create `delta` first. What `numpy` does in this operation is creates a new array during the arithmetic on the right, then memcopies the result into `delta[:, :, :]`, then deletes the original result array. If you don't need the values in `delta` to persist (which you don`t), then you should just do:
```
delta = pos[:, np.newaxis, :] - pos
```
and don't pass in `delta` at all


I have updated my own `tmpfile` implementation. I think this is now safe.
https://github.com/k-takata/lv-mod/blob/9d780440c28382f545387c0f7313db0ab800e5a0/src/stream.c#L70-L120
However this cannot be used in u-ctags directly. This requires `windows.h` to be included, but it conflicts with our `general.h`.


`pun_level = pun.count(':')` would save on creating a list


Yep


I didn't know about this rubocop comment trick. Nice!


:smile:


you should move this `newaxis` out *after* the `.sum`, so it would probably be `)).sum(axis=1)[:, np.newaxis]`


@Juan33650 la clase `line` no se usa por nuestro JS, vos la pensás usar para el icono?


I don't remember if that's actually feasible in the code.


oh wait, never mind, you need it inside for compatibilty / broadcasting with `delta`. Never mind!


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.430.549367b8218015beeed4509f304cf8f40636d6ef](http://build.palaso.org/viewLog.html?buildId=61125&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:21:57


`Math.min((model.getAtomKineticEnergy(i) - minKE) / (maxKE - minKE), 1);`
is equal to
`Math.min(5 * model.getAtomKineticEnergy(i), 1);`
if maxKE = 0.2 and minKE = 0. That's why these values are default.


yes, and i = 0 for the first iteration and size == MAX_INT if the user doesn't specify a size. So it always happen.


vreau sa folosesti transparency jquery nu sa faci append. Uitate la ei pe site cum ti-am zis si foloseste ala. Ai si o gramada de exemple la errors page!


TeamCity Jarvis Framework :: CI-GitFlow [Build 38](http://build.prxm.it:8811/viewLog.html?buildId=3682&buildTypeId=CqrsFramework_CiGitFlow) is now running


redirectul la websites trebuie sa se intample doar daca face signup nu si daca faci sign in.


Fixed


@OrKoN
Currently it is connected to rhea. It makes use of 2 rhea-specific services.

One service is for uploading data. I already planed to outsource it.
The other service could simply be stripped from the component.

It is not connected to the VCL in any way, so I would suggest to release it within the n-fuse namespace.
The component is more a service than a web-component.


Ok @kaedroho, here are the gist: https://gist.github.com/SalahAdDin/4ad0d532f684e39e7ffc
Thank you :+1:


To rehash what @bhklein said in a different way, the `crossValidate` parameter controls n in our n-fold crossValidation. So, set to 3, you have 3 total partitions: two for training and one for testing.  If `crossValidate` is negative (as in -3), then you'll only train on the first two partitions and test on the last (versus training 3 separate "models" with the various combinations of subjects).  Similarly, if `crossValidate` was -5, then you'd have 5 partitions and you'd only train on the first four and test on the last.  The `randomSeed` parameter controls which subjects get added to a given partition.  So, if you want to see 3 different examples of 5 fold cross-validation where you only trained on the first four partitions and tested on the last, then you'd said `crossValidate` to -5 and make a script that looped 3 times, incrementing the `randomSeed` parameter in your algorithm string each time.


I see... Well, I think using just "inner runner" (everywhere) would be more consistent, but it's not a big deal.


Adds cmd-shift-w to close-other-tabs


hi


@LukasReschke @ogoffart @danimo security review?;)


Yeah, I think that would make more sense. I opened PR #14  that changes that.


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-unstable.5+5](http://build.prxm.it:8811/viewLog.html?buildId=3682&buildTypeId=CqrsFramework_CiGitFlow) outcome was **SUCCESS**
Summary: Tests passed: 115 Build time: 00:03:00


done


Did the change inadvertently regress the ability to do `-m` ?


I'd still rather we threw a runtime exception or soemthing.


\zsdf


Hello!


Again, for new driver we split to files for convenience only. The patches are not logically split of the driver.


add the issue number here?


cc @fernandoneto


Well I think we should change `accept-range` to `none` instead of drop it since client may still try to make range request. Ref. Section 2.3 from RFC 7233.


This is cool... one deficiency in the test approach is that line numbers will ultimately change thus causing problems for stack traces. Perhaps the full stack is not even needed here (they don't really help the end-user for `load-file`, in which case try/catch and then call print-error specifying the 2nd arg which surpasses the stack trace.)


I don't know what $_SERVER['SP_TZ'] is for. On our server the server var is not available.


Wait, can you elaborate why you made the change?

I thought the advantage of using `vinyl-transform` was the ability to use browserify on globs such as `gulp.src([‘./src/**/*.js’])`, now you have to define each files on the entries property or use the `glob` plugin.


This won't work this way. If you want to override it would be something like `Jörg Lenhard <joerg.lenhard@uni-bamberg.de> <joerg@lenhard.info>` or the like. Please consult the documentation for more details.


@Elrhino an else clause


Adding line 85 and 87 breaks all GET requests.


I'm pretty sure we had an older leanback that was committed be for we had the leanback designation.  I'll delete that version and squash it into this one


@mlschroe @adrianschroeter Looks like this broke our test suite: https://travis-ci.org/openSUSE/open-build-service/builds/75447557


No worries! It's what Github is for!


I could be wrong about the stacktrace being helpful or not... I wonder what happens if there is an error in the code in file itself.


Wasn't DX11 GL ES 3 and not 5?


@RandomAltThing It doesn't matter. Both of them returns the same Server instance.
PluginBase->getServer()
```
/**
	 * @return Server
	 */
	public final function getServer(){
		return $this->server;
	}
```
Server::getInstance()
```
/**
	 * @return Server
	 */
	public static function getInstance(){
		return self::$instance;
	}
```


以至于独立完成了这次观光


51 as in testapp seems more reliable


For loading a whole file, don't want `:context :expr` (this is really just needed when evaluating a single form at the REPL). So, I think it is safe to delete this line.


Very nice! You'll want to replace the filename references with commands like \autoref{sec:lenstimedelays}, to enable easy navigation around the PDF document (and for the PDF to make sense :-) ). I'll merge this now, and issue the cross-refs.

Before next Friday, you might want to add *brief notes* on your initial thoughts on the AGN metric analysis. You have good notes on the things you want to detect and measure about AGN, and what you expect these parameters to depend on, roughly, in terms of which observing schedule properties - but which are the metrics you think actually need to be coded up, in order to test your hypotheses for a given observing schedule?

Thanks!


Can you please relax, @GrahamCampbell? I don't get why you keep saying "we" in discussions on projects you are not involved in but that's okay because often enough your contributions are useful.

When I have time I will revisit the issue of the exception. The code is "good enough" as it is.


I think this is Oscar's new way of referencing the mailing list.


@iksaku Could you please help me with this? <3


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 313](https://teamcity.labkey.org/viewLog.html?buildId=266137&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0 Build time: 00:02:23
### Failed tests
```
org.labkey.test.Runner.BeforeFirstTest: <no details avaliable>

```


So the range check is actually right, because the internal player template has space for 11 Elements, however only element index 1 (Mario Template) to 5 (Link Template) is used by SMBX.


I say we to refer to everyone using this project, which, is a lot of people. :)


Thanks dude.... did not work..

Getting

20150813171657.250795 ncui[p3969.t3969] ncapp.info pid of ncsvc = 3976
(ncapp.cpp:122)
20150813171700.146760 ncui[p3969.t3969] ncui.info received onDisconnect
with reason = 9 (ncui.cpp:445)
20150813171700.146907 ncui[p3969.t3969] ncapp.info waiting for NC service
to stop! (ncapp.cpp:150)
20150813171700.146947 ncui[p3969.t3969] ncapp.info done... (ncapp.cpp:160)

Did not realize you are at Disco... will chat on Monday will be great to
package this as many guys are using it.... was driving me mad as I hat
having to use windows to connect!

On Thu, Aug 13, 2015 at 4:07 PM, Nicholas Prozesky <notifications@github.com
> wrote:

> The arrows should show up when ncui is running, so it's probably stopping
> as soon as it starts. Could be because ncsvc isn't running. Is it
> executable? This app should spawn ncsvc on first log in, so once the arrows
> disappear, you can check if it's running with "ps aux | grep ncsvc". It
> should be running...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nicholas-prozesky/juniper-connecter/commit/f8f29a632921d91f461cbd0e9bcb7855232bdde2#commitcomment-12690123>
> .
>


Correct. Done.


cabon/bran?


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.10.549367b8218015beeed4509f304cf8f40636d6ef](http://build.palaso.org/viewLog.html?buildId=61123&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 871, ignored: 50 Build time: 00:35:36


It would feel strange to say i'd rather you threw an exception, because you're not executing the code, the users are.


Fair enough.


Do you need to bounds check MaxUnavailabilityPercentage and error if it is > 100?


How about calling it "RootMountPropagation"


Bounds check MaxSurgePercentage and error?


thanks!


I know the config calls them this, but moving to routes vs cluster. WDYT?


Loopt 'ie nu niet meer vast als je wil cancellen? :)

Trouwens, als het bestand naar `.js` hernoemd toont github ook syntax highlighting.


I did "docker run --help" and it shows following.

--rootmount=                    Rootfs mount propogation mode

I thought it would show --root-mount instead of --rootmount.


NAVER - http://www.naver.com/
--------------------------------------------
양혜경(hk.yang@navercorp.com) 님은 현재 부재중입니다.

보내신 메일 <Re: [spring] style; nlc (cefdcad)> 은 저장되어 있으므로 다시 보내실 필요는 없습니다.

양혜경(hk.yang@navercorp.com) 님이 남기신 메시지 입니다.
--------------------------------------------
[Absence period : 2015-08-10 00:00 ~ 2015-08-14 23:59 GMT +09:00]
휴가입니다. 급한 문의는 핸드폰으로 연락 주세요. 감사합니다.
--------------------------------------------


@ghenry22 hey nice addition. Rather than having basePath, basePath2...can you make a function called checkPath() that returns fileExists.


+1 much cleaner


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.11.9ecd8b70ee446680ffb8f92bd8cda61e30fde686](http://build.palaso.org/viewLog.html?buildId=61124&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:37:03


I doubt it does any harm in the non-systemd case, as it just sets a path, doesn't enable or disable anything. It should probably go into the packageconfig when we push it upstream, but I don't want to duplicate the entire systemd packageconfig in the bbappend or mangle it with anonymous python for something that doesn't have much impact at this point. Agreed re: the merge.


Thank you, I tried but the same error. Also mysterious is the fact that when you start the world with Cauldron/KCauldron forge tells you about missing blocks from Undergroundbiomes which makes no sense to me because it works with the same version of stock forge. Maybe I need to look into the item ID's for UBC.


everything is possible. My approach would be:

```python
# FILE: virt_test
def _process_qemu_specific_options():
    nettype = options.vt_nettype if options.vt_nettype else "user"
# ...
def _process_libvirt_specific_options():
    nettype = options.vt_nettype if options.vt_nettype else "bridge"
# ...
```


Fine!


May be we should call it --root-mount-propagation  to make it clear what we are talking amount. -root-mount is too generic.


@cmac2992 @disorn Can you guys finish styling scenario.html during today's mentoring session? It would be great if you, Disorn, can finish styling this before the session. If not, it's okay.


It does indeed.

Fixed already in 2c384ddc1219eb77b52d313a68accd2e76d18d1f


Nope, it was an omission on my PR. Good that you fixed it here, thanks! :)


Hm, this breaks the logic for null values and empty strings. maybe we should keep these conditions as this would brake the formatter.


Yeah it would be nice considering I am currently facing that a lot... except many of the developers I encounter these days write something like `(someString == null || someString.Trim() == string.Empty) == false` -.-

I monitor people I follow more these days so hopefully I don't cause too much ruckus and if so just tell me to go pound sand.


LGTM


`$length` is just a non-sense parameter.


Should we not just return fieldText in case there is no DOI?


1: M111 unrecognized component name: legacy


form_for [@summary.post.topc, @summary.post, @summary]


26:1: F401 'CFG_DEVEL_SITE' imported but unused
80: D102 Missing docstring in public method
109: D209 Multi-line docstring closing quotes should be on a separate line
109: D203 1 blank line required before class docstring (found 0)
109: D400 First line should end with a period (not 'f')
109: D205 1 blank line required between summary line and description (found 0)
115: D102 Missing docstring in public method
163: D102 Missing docstring in public method
167:5: E303 too many blank lines (2)
170: D210 No whitespaces allowed surrounding docstring text
170: D209 Multi-line docstring closing quotes should be on a separate line
170: D203 1 blank line required before class docstring (found 0)
170: D400 First line should end with a period (not 'l')
170: D205 1 blank line required between summary line and description (found 0)
196: D102 Missing docstring in public method
218: D102 Missing docstring in public method


s/in it/associated with it/ ?
Also isn't it before 5.18 ?


After the discussion I won't block this. Still I take it as a temporary solution before we implement `--loader`, which would be more generic and would actually solve other problems we still have (like specifying loader plugins order on the cmdline)


@dianlujitao radio is not stable without these files, at least on my phone.


Current default propagation mode seems to be  "rprivate". But you seem to be falling back to "rslave"?

Also  mountPropagationMapping["rslave"] will return null as new keys are "container_private", "container_slave" and "container_shared".


@cmac2992 @disorn This page has to be styled too please! Look at the wire frame for reference.


This isn't bless ( {}, '0' ) safe. I am **not** saying it has to be - just pointing it out.


As long as you don't use `{}` for arrays. ;-)


Looks like it comes from here. I think calling it --root-mount-propagation is better. Little long but it is more clear.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.431.9ecd8b70ee446680ffb8f92bd8cda61e30fde686](http://build.palaso.org/viewLog.html?buildId=61126&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:19:08


`return nil` -> `return`


/me happy I wasn't the only one =P


:dancer:


Yeah. Does it seem plausible that since this subset of the config will end up in the new deployment API, we could reuse validation later?


Ура :)


Yeah, but it doesn't really matter. At least it's consistent that way.


Will you take care of it?


Should this validation be happening in the calling code for now?


I would tend to guess that index 1 to 5 are for player 1, and index 6 to 10 being a copy of the same except for player 2, right?


I think you also need to add uuid


Remove the URL at the end of the description, and then I will accept. Thanks for dong this.
@changetip 1000 bits


Well, it would require much more testing then :disappointed: Though, I understand the trouble of dealing with two different versions of the same project. The problem with CMake changes lies in its platform specific code. It will require testing on Linux, Windows and Mac and that is more difficult than testing just the loader. Even if there are no bugs in the loader (and there are) it won't help to merge it until everyone will agree on CMake changes.

I'd better fix bugs in the loader ASAP with a bare CMake support and then free my hands to deal with CMake exclusively.


Hi @lowks, @super3 sent you a Bitcoin tip worth 1000 bits ($0.27), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/883888)**.

**[Learn more about ChangeTip](https://www.changetip.com/tip-online/github)**


Instead of having it be a "global" setting it should really be test specific. Some tests want to have real delays and some tests don't care. And those delays can probably still be small fractions of a second, just so it's properly async and the rest of the code has time to settle. I agree disabling it seems fine for now.


I don't think it matters. I just try to always use the root slash for built-in functions in namespaced code to make it unambiguous (it's just habit now).


TeamCity Jarvis Framework :: CI-GitFlow [Build 39](http://build.prxm.it:8811/viewLog.html?buildId=3683&buildTypeId=CqrsFramework_CiGitFlow) is now running


It is my understanding that IB_DEVICE_CHANGE_PHY_PORT represents the HCA’s ability to change the physical port on a SQD to RTS QP state transition. Representing the HCA's optional support of o10-19.a1/19.a2 in the IB specification v1r1.2.1 (page 462).


We use uuid to fetch the existing object.


Nobody calls this function, I guess there is a missing pro pull request.


Right!


Looks like a cert issue the command exported gibberish looking at it now... does it need the root CA ?


canvis pàgina pública


No, it seems like that version produces the same result as `devtools::install_github("timelyportfolio/svgPanZoom@dev/0.3")`


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-newSlotOnrebuildAreNotRebuilded+1](http://build.prxm.it:8811/viewLog.html?buildId=3683&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 114 Build time: 00:02:53
### Failed tests
```
Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.BasicRebuild.start_then_rebuild: <no details avaliable>

```


`rootmount` was first proposed [here](https://github.com/docker/libcontainer/pull/632#issuecomment-112177281), since then it gets [quoted](https://github.com/docker/docker/pull/14097#issuecomment-128089894) like a familiar name


Does this still actually make the request? What we really want is to just create a session programmatically for tests that require a logged in user. The tests should ideally never be making a real network request.


LGTM!


So I don't have a unit test for this specific return by reference but, as a general rule, the return by references are intended and removing the ampersand will actually break things.

#772 demonstrates this.

So I think for all return by references...  either come up with a unit test showing the before and after behavior for each return by reference you're removing or don't remove them at  all.


Лучше
```ruby
@vendor.products.first(3).each do |product|
...
end


This code which calls a method to get trans-spliced information of a transcript was initially added to eg-web-common. It was added to a method which overrides its equivalent method in ensembl-webcode.
Added the same code now in ensembl-webcode to make it work on ensembl website too.


Здесь можно передавать просто product


Please remove "else" so to not add an extra
indentation.
Note: you don't need to open new pull request
after every modification. Use "reset", "rebase"
and other amendment commands at will.
You can as well open a request per patch
if you put them into a separate branches.


`rslave` was proposed as new default [here](https://github.com/opencontainers/runc/pull/77#discussion_r34908344)

will fix the string


How does that break GET requests?


I think it's relevant to see whether `kmergenie` with ABySS outperforms the fixed values of *k*.


shuadefeiqi


I thought the uuid is already in the url and not the submitted json


`10 == lz4` - please don't do this :).

`zlib,N` syntax is a bit clunky.  But it shouldn't be a big problem if the default is reasonable.  `zlib`=`zlib,6` sounds good to me, especially if we can use lz4 instead of `zlib,1`.


better not to use the short aliases and use the full class names


Alright. I'll this PR with the suggestions:

* remove "level" from "inner level runner" mentions
* replicate docs from man page to "main" docs

Just bear with me that I have no option besides being offline for the next two hours.


Ask Sasi. The designer was not working until we commented these out.

On Thu, Aug 13, 2015 at 8:37 AM Alec Embke <notifications@github.com> wrote:

> How does that break GET requests?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/azuqua/restjs/commit/2422df86fa7838abf64f9f03b076c86e3b9efa8a#commitcomment-12692353>
> .
>


or as you're within a console command, you can resolve this from the application instance available at `$this->app`.


Did I forget to check the data portion and turn it into a query string for
GET requests ?

On Thursday, August 13, 2015, Alec Embke <notifications@github.com> wrote:

> How does that break GET requests?
>
> —
> Reply to this email directly or views  it on GitHub
> <https://github.com/azuqua/restjs/commit/2422df86fa7838abf64f9f03b076c86e3b9efa8a#commitcomment-12692353>
> .
>

--

*Quy Le*

Software Developer | @Azuqua

Unify the Cloud

*We have moved. Please make note of our new address: 619 Western Ave Suite
650, Seattle WA 98104*


I'll take a look once I get in.

On Thursday, August 13, 2015, Quy Le <quy@azuqua.com> wrote:

> Did I forget to check the data portion and turn it into a query string
> for GET requests ?
>
> On Thursday, August 13, 2015, Alec Embke <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
>> How does that break GET requests?
>>
>> —
>> Reply to this email directly or views  it on GitHub
>> <https://github.com/azuqua/restjs/commit/2422df86fa7838abf64f9f03b076c86e3b9efa8a#commitcomment-12692353>
>> .
>>
>
>
> --
>
> *Quy Le*
>
> Software Developer | @Azuqua
>
> Unify the Cloud
>
>
> *We have moved. Please make note of our new address: 619 Western Ave Suite
> 650, Seattle WA 98104*
>
>

--

*Quy Le*

Software Developer | @Azuqua

Unify the Cloud

*We have moved. Please make note of our new address: 619 Western Ave Suite
650, Seattle WA 98104*


@aleffert: There doesn't seem to be any "async" concept in Robolectric :frowning:. See my comment on the Robolectric issue here: https://github.com/robolectric/robolectric/issues/1758#issuecomment-129575924

I'll look into making it a per-test setting though.


The API endpoint is changed in our unit tests from the real URL to a localhost URL on which our mock webserver runs. No network calls are made to the API endpoints in the tests.


Also need to add this to line 8901.


Personally I think that change of default should either be done in a separate PR or atleast in a separate patch in the same PR.

I prefer that to reduce the complexity and to limit the scope of discussion, we don't change the defaults. Instead, we just focus on introducing new modes.  Once this patch gets merged, we can generate a separate PR for changing default mode and convince community that why that might be a better default.


Yeah... This one is a hard one...


@rhatdan, @mrunalp what do you guys think.


mh, I didn't thought of that, it is also strange that nobody report issues about it. At the moment I cannot push code though.


Those lines are fine. There might be an issue like Quy said with data serialization. But those `write` lines are correct assuming the user provided an `opts.method` value.


Uploading leanback is OK, since we will make a leanback package at some point.
Good squashing :-)


I'm using the anisotropy > 1 condition straightly at the first if statement, could be better in terms of ? readability ?


:+1: :sunglasses:


You really wanted to comment out this line?


Should be restored :)


You should also catch kCGColorSpaceModelIndexed since that's not supported by CGBitmapContextCreate, either. Leads to `<Error>: CGContextSaveGState: invalid context 0x0` messages in the log.


Adding startup files to working branch


So I don't have a unit test for this specific return by reference but, as a general rule, the return by references are intended and removing the ampersand will actually break things.

#772 demonstrates this.

So I think for all return by references... either come up with a unit test showing the before and after behavior for each return by reference you're removing or don't remove them at all.


What's the purpose of the variable "codecCounter"?


fixed with the highlight as item


HEHEHE...HEH...#winning.


Pretty sure this will break pkgcheck, you'll have to convert the usage there too (grep for local_use).


Finally. :walking:


The `xrange` is imported for pudb's `py3compat` module, so yes.


97204612_MultiSpeedFan (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (753 of 753 tests passed)

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


I'd probably use textwrap.dedent here to keep the indentation level.


Should not be a space there between `PVPGN_SOFTWARE` and quote?


This is breaking the plugin...


@futuretap i had one other user with the same issue and adding it fixed it. i will try and find a better way for this though the upcoming days if i have some free time


Previously this library did not require jQuery - but with this change my app is throwing `Uncaught ReferenceError: jQuery is not defined`.  Is there a way to add back the conditional assignment w/o breaking what this change fixed?


Ist ein Poll nicht: `[poll=41]` damit ein Poll mit der Id 41 aus der Datenbank entnommen wird?


It appears to be used on lines 291 and 389.  I'm not really sure what those lines are doing.

We can definitely remove it from lines 397 and 401 though because it isn't used there or in the rest of the method.


The doc explicitly states: "Note that indexed color spaces are not supported for bitmap graphics contexts"


Please use more helpful commit messages. It comes in helpful if you or anyone else needs to figure out the differences between workflow versions or needs to determine when a particular change occurred.

You can bunch up more changes into one commit to save a bit of time.


Параметр лучше сделать типа ScTypeElement


Auch hier: Keine gute Idee, `ugettext_lazy` in den Settings zu importieren, Django tuts auch nicht.


I hope it's a mistake, because it breaks LUA variables with these names.


This is going to work only for Chrome .. not sure what will happen for FireFox

Where did we get the quantization parameter value from ?
VP8 recommends something in 50-60 range .. Lower == higher quality

http://www.webmproject.org/docs/encoder-parameters/


Unhandled Fundamental Failure


Und das? https://docs.djangoproject.com/en/1.7/topics/i18n/translation/#how-django-discovers-language-preference


Couldn't the typed geometry conductors create (Buffer)Geometry and also
save their creation parameters along with a flag indicating if the geometry
was altered? Then serialization by parameters could still work unless the
geometry has been modified, in which case it would fall back to full
serialization.

My dream is to keep the creation parameters even if the flag is toggled,
and serialize a subset of operations, too, but that's another
conversation....
On Aug 13, 2015 11:40 AM, "Mr.doob" <notifications@github.com> wrote:


@magul zostawiłeś debugger


Nee zo loopt ie goed :) dat was inderdaad wat ik aan het oplossen was, die infinite loop.En met de nieuwe versie krijg je nu gewoon echte bedragen eruit. 21.54. ipv 21.5429999999ik liep in de laatste versie de output van mijn prompt te checken met true of false, ipv null. Ik dacht dat ik null eerder ook geprobeerd had. ik denk alleen dat ik de mist in ging met dat als false == false klopt het true geeft of zoiets.Nou, de 6% maken is simpel zat, nou denk ik maar aan judith vragen het uit te proberen en kijken wat ze misschien nog meer nodig heeft/beter kan.
Ik wist helemaal niet dat jij dit allemaal mee kon kijken :D
ah dat is handig. Ik gebruikte nog codeacademy als editor en github alleen voor opslaan, want ik wil constante feedback. Maar dat is goed om te weten :) Had je trello toevallig gezien ? de helikopter was in afghanistan, had het er even snel in gezet toen antoon het zei maar in de zelfde kamer bleef als ik :P

Date: Thu, 13 Aug 2015 08:20:11 -0700
From: notifications@github.com
To: Workspace@noreply.github.com
CC: balderclaassen@live.nl
Subject: Re: [Workspace] Update BTW tool (0481f8d)

Loopt 'ie nu niet meer vast als je wil cancellen? :)

Trouwens, als het bestand naar .js hernoemd toont github ook syntax highlighting.

—
Reply to this email directly or view it on GitHub.


Nobody uses `fix` about non-bug issue :cry:


A post can have some objects whose uuids are not in the url.


this set of tests might be more succinct as one test with a list of types that are acceptable, or even one list with a list of `[type, exception-or-null`]


ditto list


ditto ditto (also, "ann" -> "an")


avatar程度であれば、同じテーブルで管理してしまうのがいいと思います。プロトタイプの写真と混同してしまっても逆に管理しづらいので！


So defer gets run when panicking, which defeats the purpose of the consistency guard entirely.


Nope


`images` のような汎用性の非常に高い名前よりも `thumbnails` など、明確に用途のわかるテーブル名・モデル名にしましょう。


Looks good. I have done a similar thing for config.go. Will commit my changes soon.


Why has this line been changed?


i would prefer short name rdd_id


アバターはusersテーブルで一緒に管理してあげたほうが良い気がします


tutaj jest zbędna kropka w nazwie klasy


this is the activator launcher script which is a generic script and I won't change it.


'achiever_url' по умолчанию не надо. Я понимаю что ты сделал его для удобства но потом забудется и не поменяем


Hm. Thoughts on better wording for this then?


That is the plan.


Because I moved the exception handling into the method that collects locations and changed the guard to check for an empty array - https://github.com/guidance-guarantee-programme/pension_guidance/commit/03485a74b74fd690449136fa0d06e97bcca5f6c1#diff-54b4e4f2b562d74dad3af78e66e74e4fR41


So in my testing this logs the preview correctly. It only logs the preview if we are in popup mode - it should not fire in full mode.


`:hogehoge => piyopiyo` という記法は、現在のrubyのバージョンでは `hogehoge: piyopiyo`と書くことができます。全て新しい記法に統一しましょう！古い記法は撲滅します。


Danke!


Ebenfalls Danke! ;)


I think we should fall back to the systemClassLoader if the parent of the systemClassLoader is null. While normally there is an internal hierarchy, it's not obligatory.


`logPageView` does check the mode


Under which conditions will `index` not be set? This adds extra conditionals to `views/locations/show.html.erb`.


will remove - inadvertently left in after testing. thanks for finding


why two returns statements ?

keep only one and mention if the guess no. matched with the random no. then returns true otherwise returns false


These `tearDown` methods aren't needed - you get a fresh db each time.


If no nearest locations have been supplied.


Tag関連は自動でテーブルは生成されますが、例えばTagクラスのインスタンスにメソッドを定義したいという場合は、じぶんでモデルを作らなければいけません。しかしその際に通常のモデルの作成方法だと、Tagモデルはうまく機能しないのです。

一旦、Tagモデルに関する設定はなさそうなのでこのまま作らなくてもいいと思いますが、
Tagモデルの生成に関しては以前のコミットにコメントしてあるので、作る際はそちらを参照してください。


Actually, I was even wondering why you changed it from `null` in the first place?
Do you agree with my comment explanation that states that a module running _via_ the launcher should behave as if ran by `java -jar`, as much as possible?


Should be no reason for the `_` prefix to `nearest_locations`? One was used for `booking_location` as that is also an attribute on `Location`.


`for_main`と`for_sub`という２つのカラムは `role`というカラムにまとめてしまいましょう。
そこで、roleが0ならばmain、roleが1ならばsubという風に管理するのがスマートですし、管理のしやすさからも理にかなっています。

数字を用いて各レコードの属性を管理する際には`enum`というrailsの提供する機能を使ってあげましょう。


Forcing service authors to explicitly tag models in `#/definitions` with redundant information seems like a step backwards. Even if a `#/definitions` model is defined outside of `swagger.json`, references to it should be by the canonical `#/definitions/<model name>` given the wording in the [Swagger 2.0 specification](https://github.com/swaggerapi/swagger-spec/blob/master/versions/2.0.md#swagger-object):

An object to hold data types produced and consumed by operations.


上でもコメントしましたが、avatarはuserにまとめてしまいましょう。


:ok:


This change broke a lot of tests and introduced $ into the code base.  If you look at the DOM interaction, the module is using `angular.element`


It may be that this needs to load even if NOT admin...


No, smbx ignores that. The player templates are shared, that's the strange part.


Congratulations on completing the course!

However I can't merge your commit because the file name needs to be "restaurants.md" and you used "chipotle.md".

Try renaming the file and doing a new pull request.

Cheers!

-- Jorge (@fromzeroedu)


אלעד אתה יכול לעלות בבקשה את מה שלמדנו עם ההקשבה לשיחות? חייב לעבור על זה


I'm not 100% on conflating an empty list of `Location`s with an invalid postcode.


Could use `self.assertContains(response, expected_text)` here, though it doesn't gain you all that much!


seems legit to me


There's an index.md already here: https://github.com/Molnfront/molnfront.github.io/blob/gh-pages/pages/pages-root-folder/index.md that's the root of the website. It was moved to that dir so it's not mixed with other files.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.432.269cac6cf91b760d6dbc268e53138b68a63a87fc](http://build.palaso.org/viewLog.html?buildId=61134&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:27:48


This commit reverted some Gemfile changes that I've reapplied in #131 . without therubyracer, sparc-request doesn't work for us.


If the regex here is just for whitespace, I'd use

`self.assertContains(response, some_html, html=True)`


Perhaps `position` and `distance` could be members on some sort of value object related to the search results?


@xamgore
![image](https://cloud.githubusercontent.com/assets/1321838/9254949/eee09916-41ee-11e5-9051-1d4471e87993.png)


Так тоже должно работать но лучше сделать малость иначе:
1. в каталоге lib создается файл который будет иклюдить в себя все остальные патчи. как правило его называют именем плагина
`lib/redmine_achiever.rb`
2. в папке lib создается структура вида `lib/redmine_achiever/patches/тут патчи` `lib/redmine_achiever/hooks/тут хуки` и т.д.
3. в файле lib/redmine_achiever.rb указываются какие патчи/хуки/т.д. необходимо подключить при загрузке
`require_dependency 'redmine_achiver/patches/time_entry_patch.rb`
4. В `init.rb` подключается только главный файл `require 'redmine_achiever'` а он уже по зависимостям собирает тех кто нужен.
5. Патчить модель лучше сразу в файле патча (см. ниже комментарий).
К чему это приводит:
a. Четкая структура
b. Легко отключать/подключать плагины меняя одну строку в файле
c. Если файл с патчем ненужен удаляем и забываем как страшный сон, не надо лазить по коду и искать где мы его подключали.


Can these attributes ever not be present?


Good idea - we are the only ones doing this now - so there isn't standard way to report. I think getting the language in is pretty good idea


Instead of manually mapping `pkg.name` to `component` I'd just pass the entire pkg into the template and directly access the values for now.

`messageBody(pkg)`

`<%= name %> `

Should we want to add some computed stuff like links later on, we could change that to something like:

```js
messageBody({
  pkg: pkg,
  links: {
    // …
  }
})
```

`[<%= pkg.name %>](<%= links.name %>)`


Nevermind, they are for different cases. Disregard.


Тут лучше сразу и пропатчить файл:
`unless TimeEntry.included_modules.include?(RedmineAchiever::Patches::TimeEntryPatch)`
`  TimeEntry.send(:include, RedmineAchiever::Patches::TimeEntryPatch)`
`end`


The equip button seems significantly harder to read now.  It'll be interesting to see if the pass rates are affected by this change.


Все что написано выше чистая педагогика. В целом круто, я на это когда-то убил пару дней наверное)


Exactly.


Fix #62


I  followed the code from here but a new problem was found:

Error:Execution failed for task ':jamsMusicPlayer:processDebugResources'.
> com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command '/Users/williamzhang/Library/Android/sdk/build-tools/22.0.1/aapt'' finished with non-zero exit value 2


How about dealying caling fclose?

The current implementation:
1. fclose
2. sortTagFile()

My proposal:
1. fflush
2. sortTagFile
3. fclose


Yep Scott described it perfectly.  L75 sets the stopping condition of the loop used to remove testing data to 1 if `crossValidate` is less than zero.  This means training is only performed once, and the first partition's data is reserved for testing in the for loop on L85.


TeamCity Subsonic :: Debug [Build 44](http://109.103.226.38:8512/viewLog.html?buildId=35013&buildTypeId=Subsonic_Debug) is now running


bouhh :sob:


Right - we're using Roslyn 1.1.x, not 2.x.


TeamCity Subsonic :: Debug [Build 44](http://109.103.226.38:8512/viewLog.html?buildId=35013&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:59


@aleffert: I considered that, but that problem is that there is no way to ensure the correct generic type of the `ActivityController` without unsafe casting, or defining a generic type separately from the `getActivityClass()` method (which is also unsafe). Since we would be calling this method all over anyway in an unsafe manner, I ultimately decided that this slightly verbose method call was the better option.


This should be ctcpNotice("VERSION " + <stuff>)


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.12.269cac6cf91b760d6dbc268e53138b68a63a87fc](http://build.palaso.org/viewLog.html?buildId=61133&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:37:59


Done!, we have added the new annotation, it only works when the thresholds are not present in the header


TeamCity Subsonic :: Debug [Build 45](http://109.103.226.38:8512/viewLog.html?buildId=35014&buildTypeId=Subsonic_Debug) is now running


TeamCity Subsonic :: Debug [Build 45](http://109.103.226.38:8512/viewLog.html?buildId=35014&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:48


Called at 2218 in RealVoltDB.java


TeamCity pull requests :: netty [Build 24](https://garage.netty.io/teamcity/viewLog.html?buildId=295&buildTypeId=pulls_netty&guest=1) is now running


This script is obsolete now, because there's a `clld-create-downloads` console script installed with `clld`.


Aye, was wondering how do that without inline `.splitlines()` magic :).


RoomAirflowNetwork (lgu1234) - x86_64-MacOS-10.9-clang: OK (1792 of 1792 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)


TeamCity Subsonic :: Debug [Build 46](http://109.103.226.38:8512/viewLog.html?buildId=35016&buildTypeId=Subsonic_Debug) is now running


ERROR:

There are errors in the configuration of this machine. Please fix
the following errors and try again:

vm:
* Forwarded port '2222' (host port) is declared multiple times
with the protocol 'tcp'


ossn notification 'mark as read' missing translation #484


DEBUG=false


TeamCity Subsonic :: Debug [Build 46](http://109.103.226.38:8512/viewLog.html?buildId=35016&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:46


Of course it's only unsafe if you define one base method to be used from everywhere. We _could_ safely define a utility method for it in each class I guess, although I decided that was not worth the effort.


Yes, you're right. The token_credentials was wrong. Also, AblyException was outdated. Fixed it in the new commit.


no need for that check here, it should stupidly copy the values from PVR_RECORDING to CPVRRecording.


thx, will check.


I wanted to distinguish between id in returned object and id for which we are trying to get RDD. But can change it, no problem


We probably won't be able to detect that sensitive a change in the gross analytics; we'd likely have to measure time spent on the inventory screen as an A/B test. We could just revert; they don't end up looking that great. (Then again, the old buttons didn't look like buttons...)


you need to reset the values to -1 in `CPVRRecording::Reset()`


Probably needs to be `abs($0) > delta`. Negative frequencies can be a thing pretty easily with subtract.


Should this be `!message.getHeaders().containsKey(NOT_SAMPLED_NAME)`?  Seems odd to be if tracing or not sampled.


I am really sorry guys...totally missed the abs()


Unhandled Fundamental Failure


TeamCity pull requests :: netty [Build 25](https://garage.netty.io/teamcity/viewLog.html?buildId=296&buildTypeId=pulls_netty&guest=1) is now running


@etpinard @jackparmer updated plotly.js url, it was pretty outdated. also, @jackparmer here's a nice way to include variables in HTML to prevent ourselves from repeating ourselves


@sutaakar Added a test to make sure that I don't forget to add any commands in the future.. :)


wrong placed spaces, should be

```
std::string strYear(m_iYear > 0 ? StringUtils::Format(" (%i)", m_iYear) : "");
```


I think you need to add `#include <lustre/lustre_user.h>` to this file as well, since this is the one that should define this constant. With this addition (didn't try without), things work for me now.


TeamCity Subsonic :: Debug [Build 47](http://109.103.226.38:8512/viewLog.html?buildId=35022&buildTypeId=Subsonic_Debug) is now running


spaces between parameters


For some reason test is failing here in tipbot

`./src/test/test-buffer.c:231: test_append_printf: Assertion string_equal("[003]", "[001]") failed.`


space between keywords `if (...`


TeamCity Subsonic :: Debug [Build 47](http://109.103.226.38:8512/viewLog.html?buildId=35022&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Exit code -65536 (new) Build time: 00:00:43


@schlomo See the discussion in #3

Also, it may be better to raise a new issue for this instead of commenting on this commit.

FWIW, I think it would be better for the daemon to run as root, but my personal preference would be do to that via `libtcnative`. See https://github.com/docker-library/tomcat/issues/7 for some discussion of adding `libtcnative` to the image.


spaces between operators `season >= 0`


The bullet is actually extending the Character z-index right now. The interesting thing is that if you change it to

    public class Bullet extends Entity {

    private final GraphicsComponent graphicsComponent;
    private BulletComponent bc;

    public Bullet(Vector2 shootingPos, Vector2 velocity, float maxDist){
        super(ZIndex.PROJECTILE);
        graphicsComponent = new GraphicsComponent();
        graphicsComponent.register("idle", "bullet.png");
        graphicsComponent.play("idle");
        addComponent(this.graphicsComponent);

        bc = new BulletComponent(shootingPos, velocity, maxDist);
        addComponent(bc);
    }

    }

Bullets render fine, but the NPC doesn't render at all anymore. I'm wondering if this has something to do with the transparency of character sheets or something.

Below is how the z-indexes are ordered during run-time. As you can see, they are basically in the right order.

`[INFO]	18:16:39.569	scenes.Test.handleQueue	-	[com.emergentorganization.cellrpg.entities.backgrounds.TheEdge@1a5e3dda, com.emergentorganization.cellrpg.entities.buildings.BuildingLarge1@4081642b, com.emergentorganization.cellrpg.entities.buildings.BuildingRound1@3c3e8c53, com.emergentorganization.cellrpg.entities.FollowingCamera@af70a2a, com.emergentorganization.cellrpg.entities.characters.Player@61ac5643, com.emergentorganization.cellrpg.entities.characters.npcs.CharCiv1Blinker@2ac7d661, com.emergentorganization.cellrpg.entities.CAGrid@62d39b7e]`

There is definitely something fishy going on here, but I don't think z-indexes are the root cause of this issue


I have added another commit to remove the codecCounter lines and set the quantization parameter back to 56

2d9c43195bcfb6ca4a7cdd7d17921257edc55787


Awesome! + @NicoleGrondin


returns an empty string


Docs need to be updated.


I haven't published it yet, it's not done


I'm getting this when I use the test cases : "NameError: global name 'nested_sum_comp' is not defined"


should this be here? You're defining t as a global variable here right? so it seems like it should only be doing the function for the value of that you've been here, but actually it's working when you pass in other arguments, so now I'm confused.


@tremainebuchanan remove the credentials from this file... no credentials... just put placeholders in the instructions


Actually, it looks like #14 could be a reasonable place to discuss this.


I think event's current form is in the process of being axed anyway? We were going to rewrite model as well. (and hadn't decided what the precise scope of the Model module should be, I.e. How much game does Pathway provide in opinionated fashion?


Any particular reason you used float as opposed to int?


TeamCity Subsonic :: Debug [Build 48](http://109.103.226.38:8512/viewLog.html?buildId=35024&buildTypeId=Subsonic_Debug) is now running


This sounds more like a statement and not a question? I'm not sure how to help with this comment?


of course, done.. thx again


I haven't been doing this at all but I'm getting the sense that the preferred method for explaining what a function does is that string notation here rather than in-line commenting.


This seems really fragile. Can we get these by ids or something?


TeamCity Subsonic :: Debug [Build 48](http://109.103.226.38:8512/viewLog.html?buildId=35024&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:57


while `EPERM` for `sol_buffer_resize()` makes sense, it does not for `sol_buffer_ensure()`. In that scope, `ENOMEM` makes more sense, then the translation.


develop (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Calls a function that isn't defined. Should this be "nested_sum()"?


I think it should be a tuple of (Try[Result], Self) maybe? Because you get a result out of the script, which may have blown up, and the resultant environment which is the monad part.


Why? :stuck_out_tongue:


Hi,

2015-08-13 16:41 GMT+02:00 Mario Garcia Valdez <notifications@github.com>:

> That´s what I've been saying. A bit like cheating.
>
> But JJ, you convince me that is the nature of the project.
>

It really is However, we need to know when that happens.

> What we could do about it IMHO:
>
> 1. Just report it like it is. When we compare, all algorithms must be
> implemented the same.
> Justification: We don´t try to have control of everything because that
> will have a cost.
> In real experiments we care about finding a solution eventuality.
>

OK.

>
> 2. Try to restart all workers when there is a new experiment:
> 2.1 Just like EvoSpace, after every iteration, do another GET to the
> server: are we finished?
> If the experiment has finished restart the worker,
> Problems: Is asynchronous, there can be slow workers that never do anything
> because
> every time they ask the experiment is over, or the experiment has finished
> more than one time.
> 2.2 MVCC style, every worker is assigned the current ExperimentID, just
> like 2.1 but
> he can only work for his ExperimentID, if there is a new one, the PUT
> chromosome request
> fails with 404 or must first look with a HEADERS if the resource has
> changed and the Worker restarts with the new ExperimentID.
> 2.3 Try to get all of the participants in sync. Totally against the
> asynchronous idea.
>
>
>
We can return some signal, or the solution, when it's been found. However,
we don't know how many clients are there going to be and how long to return
the solution. I'll propose a solution in another issue.


we try to fit in the available space (should work most of the time), if it did not work due lack of space, then we grow via `sol_buffer_ensure()` and if that succeed we try again. The second try should always work if `vsnprintf()` is stable if called with the same `fmt` and `args`.


Oh I see, the goal of the test is to ensure that each row is what we expect. Never mind. That said, I'm not sure how useful a thing to test that is? Maybe better to just check that there exists an item with the right icon+id pair. That way we can reorder the list without breaking the test, but we still check that the names/titles/icons match up.


Doesn't return correct values for test cases:
	list_3 = []
	list_4 = [1]
(returns 32 & 32 instead of 0 & 1)


If its on github we're already compromised. We should change them as well.


good


This works great! :+1:


Hmm, the selected hover over styling is not so great either.  Let's leave them for now; we've got better things to do.

Any reason you didn't apply the changes to the big equip/unequip button in bottom right?  That might tie them together better and make it more discoverable.


weird, could you check the details? Maybe some of the `va_start()` or `args` handling is wrong. Would help to run with valgrind or coverity to see what it says.