#5412-Mixed-air-setpoint-less-than-0-causes-max-OA-operation (mjwitte) - x86_64-MacOS-10.9-clang: OK (2228 of 2228 tests passed, 0 test warnings)

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


По поводу проверки,
ТФ всех проверять не будет, только проверяющих, а проверяющие остальных. Если хотите чтобы Вас проверил ТФ, станьте проверяющим, напишите ему письмо и укажите в теме что Вы хотите проверять, тогда к Вам прикрепят народ для проверки. Если проверять не хотите, напишите, что хотите чтобы Вас проверили, тогда Вас к кому-нибудь прикрепят. Вот как-то так))


fix formatting issues on spacing.


Oh yes, that's correct. Good point.


add_coils_to_unitary_system (mbadams5) - i386-Windows-7-VisualStudio-12: OK (2425 of 2447 tests passed, 0 test warnings)

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


OK, dzięki.

Teraz daję 4 spacje, na najbardziej standardowe wcięcie np.

Ale w .js to jest bardziej skomplikowane, więc muszę tam coś wymyślić
innego.

2016-07-15 14:56 GMT+02:00 Piotr Kowalski <notifications@github.com>:

> Muszą być wcięcia. Niech będą one 4 spacjami 👾
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/rmkwiatkowski/slider_basic/commit/87fbdabda227a8b4e2ddfbe48d22e04fec2bf48d#commitcomment-18260886>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ATFtg_TIvYSgNJ4a4izfCQ3q4nJpgMD0ks5qV4NigaJpZM4JNXya>
> .
>


Ok, but we are not closing the TCP connection directly as we are only calling `end` on the stream.
With this implementation the client will not call `stream.end()` if the server doesn't do the same first.
In other words, the client call to `stream.end()` depends on the server call to `stream.end()`.

I've asked because removing this check works as a potential fix for #784.
Anyway, thank you.


By reference this article, might the syntax is `<dfn>rgb()</dfn> = rgb( <<rgb-component>>#{3} [, <<alpha-value>> ]? )` ?
https://www.smashingmagazine.com/2016/05/understanding-the-css-property-value-syntax/#1-the-symbol


remove comments


Submit full report on what you're using and create an issue.
EB orb is done and gone. No support for it.

Sent from Outlook Mobile, please excuse any errors or shorthand.

On Fri, Jul 15, 2016 at 9:10 AM -0400, "deome" <notifications@github.com> wrote:

Why u disable eb orbwalk? just curious because it make my botting account stuck in base when use L# orbwalk :^)

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


There's a file tests/README that has a one-liner about the purpose of each test. May be good to add an entry for test88 there. Incidentally there are things in tests/ that are not tests at all, like hashscan.


Thanks Xu! Apologies for the lack of reply this morning. If still any use to you:
```
System libraries:
  /System/Library/Frameworks/AppKit.framework/Versions/C/AppKit
  /System/Library/Frameworks/Cocoa.framework/Versions/A/Cocoa
  /System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation
  /System/Library/Frameworks/CoreServices.framework/Versions/A/CoreServices
  /System/Library/Frameworks/Foundation.framework/Versions/C/Foundation
  /usr/lib/libSystem.B.dylib
  /usr/lib/libiconv.2.dylib
  /usr/lib/libncurses.5.4.dylib
  /usr/lib/libobjc.A.dylib
  /usr/lib/libutil.dylib
Homebrew libraries:
  /usr/local/opt/luajit/lib/libluajit-5.1.2.dylib (luajit)
  /usr/local/opt/perl/lib/perl5/5.24.0/darwin-thread-multi-2level/CORE/libperl.dylib (perl)
  /usr/local/opt/python/Frameworks/Python.framework/Versions/2.7/Python (python)
  /usr/local/opt/ruby/lib/libruby.2.3.0.dylib (ruby)
```


There is an opening bracket here, but no closing bracket.


Yes, it works otherwise I wouldn't have pushed a revert. This is starting
to get fucking patronising. I'm not liking that one bit.


どうやらコード規約的には`self`を付けなくてもいいなら付けないほうがよいらしい
[ここ](https://github.com/shuyuhey/MyTodoList/commit/6b7a4d5a2b8814c6468694258def9037051fd78d#diff-9401f913e916412ce1e3c04d469e444fR47)ではつけてないけれども!笑


@lidashuang 外包的项目要用到 PHP，没法不学😔


はあん。これはちょっと調べてみよう。


@meame 4 GB.


あれ？どっかからコピった時についたな。thx


As you read the callback address from the config file, could you add it in the example file from the root folder?


Should work now


/w\


Thanks! I wasn't sure if you'd left this here as a deliberate placeholder and it wasn't causing any harm, so thought I'd leave it to you to confirm.


今どきの書き方ではあるけど, ビルド時間が増加するのでプロダクトが大きくなってきた時は要注意
まぁappleさんが何とかしてくれるやろ...


Good idea, feel free to send a pull request!


I'm not patronising you, I didn't see the revert. I'm doing stuff elsewhere and just have this thread open. Checked now, saw you've revamped the button logic too. Nice work and thank you kindly.


You're welcome.


this widget only shows the device
not whether its got a link or not.


Extreme performance improvements noted


Just revert my change when adding yours.  Kale asked me to see if I could
make the fdc go, so, well, I need some kind of format working :-)

  OG.

On Fri, Jul 15, 2016 at 2:35 PM, Nigel Barnes <notifications@github.com>
wrote:

> Fine by me, am also using it for electron anyway. Here's the 1600K
> geometry:
> { // 1600K 3 1/2 inch 80 track double sided quad density - gaps unverified
> floppy_image::FF_35, floppy_image::DSQD, floppy_image::MFM,
> 2000, 10, 80, 2, 1024, {}, -1, { 0,1,2,3,4,5,6,7,8,9 }, 60, 22, 43
> },
> I'll probably also improve find_size as it was originally only intended to
> detect DD but now we're adding QD.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/mamedev/mame/commit/9eb57470c93ab76feba0a7955a9d181c24c4e2ce#commitcomment-18260638>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AI0i8YJQ3DQgn5ym4IFG5ZjsuV0jhNwMks5qV35-gaJpZM4JNS86>
> .
>


We have warning values for things that are probably not going to work. We have error values for things that are impossible. As such, we give a warning for when you try to print faster than 150mm/s. We give an error when you try to print faster than light speed. It's not that 299792458000mm/s is a realistic speed to expect from your printer, but the user is allowed to try.

A correct function could be a good solution.


今のプロダクトでは, `class`には基本的に`internal final`(継承禁止)のアクセス修飾子をつけてるぞい


How useful is it to log the CouchDB document IDs, rather than their identifiers? It adds a level of indirection, when debugging, only saving the extraction time of the identifiers from the batch...


aha so this is the problem #2 the function was removed in this commit, maybe by mistake?


ほう、ちゃんと明示するみたいな意味合いかな


That's good.


見た記事のブコメにも書いてあったな。泣く泣く統合することもあるとかなんとか。


There's ~no cost of logging: log both the document ID and the identifier?


It would break too many views that use the Auth facade.


ok


Why are you using login / logout with API requests?


@herrbischoff 😄


__`DONE`__ See 53f2c29


I'm not so sure now, but I thought that is what the protocol means, and there is the timeout for this case.


here we go https://github.com/mnapoli/externals/pull/8


Looks good, please consider creating a pull request.


Why are you using ctrl instead of vm?
https://github.com/johnpapa/angular-styleguide/tree/master/a1#controlleras-with-vm


Reverting this commit disables a usecase of converting yaml to json statically.

In 0.2.0, configuration and usage like this convert data/foo.yaml to assets/foo.json.

```
// webpack.config.js
module: {
  loaders: [
    {
      test: /\.yaml$/,
      include: path.resolve('data'),
      loader: 'yaml',
    },
  ],
}
```

```
// applicatin.js
const actualFilename = require(`file?name=[name].json!./../data/${file}.yaml`);
window.fetch(actualFilename).then(res => {
  // ...
});
```

In 0.3.0, yaml-loader will produce json file with 'module.exports = '.
It is not a valid json.


Typo in here.


Anyone know why GitHub wanted to remove and then re-add this exact same file?


wwwwww


Not ideal to use a mix of procedural calls & object oriented calls to t() function.


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


Yes timeout works, but it should close the connection only when something wrong. Under normal circumstances it shouldn't be used.


Note that I agree that a comment would be nice in this case. I only commented on the quoted line. I will send a mail to decide on the best practice.


Are we sure this doesn't break anything in the module? Can you please refer to calls to this function in the module.


@sensescape I'm changing the names of the abstract styles to keep them consisten and easy to then transform into `blocks`.  Every custom style that adds something to the `color` block of a shader have the word `lines-` or `polygons-` behind it according to the `base` they have. This allow us to have shorter names


Where are we defining this route? Don't see a routing yml file.


I said there will be no 0.16 PM anywhere
I'll do nothing for php5 anymore


I'm thinking about stripes angles... we should play with something that is not the angle... rotating the pattern is in the line of the expensive things to do


#5412-Mixed-air-setpoint-less-than-0-causes-max-OA-operation (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1118 of 1118 tests passed, 0 test warnings)

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


I am committing my first commit..


Pardon, 4.2 und 4.1!


If we hard-coding the offset value to 0 here, its as good as being hardcoded in the searchAll service. Can we make this a configuration as well?


trying to keep names consisten here


__`DONE`__ Resolved


Lets remove commented code. If you need this for later, please add a todo comment.


`_` is for separate two words like "mayor_roads", while `-` is for more logical structures


Is it possible to build an application without "-lpcap" option?


@copumpkin Yeah, I completely agree with you on that general principle. However, in this particular case, letting `openssh` use a directory within the nix store as its privilege separation directory (whatever that exactly means...) at runtime seems wrong to me. Using its default directory (`/var/empty`) seems less wrong, and is not something that is NixOS-specific.

The obvious solution would be if this was configurable at runtime. I don't know why `openssh` hard-codes this directory. I can try looking into why. We could maybe make a patch that would allow us to configure it through an environment variable or by other means, but I'm hesitant patching packages as security-critical as `openssh`.


We are synchronized!  I just submitted a bug on this topic..


sure, try it. or feel free to do a 2.2.0-rc.2


If you have a printer of 10x10x10 kilometer you're going to get integer overflows everywhere. Cura simply doesn't support that.

An infill percentage of 0.000000000001% will also give problems. Do you want to edit the json files so that they catch these errors?

I'm not going to put any work in this for this issue.
Perhaps you can make a new issue for supporting rediculous user values without the engine crashing
Maybe we could edit the TestSuite so that if no max_value is chosen, it tries a value beyond the maximum of int64_t.


Do something about this


ok, but it brakes HTML validation (no empty attribute allowed)


Yea, I understand.  One idea is to create another helper method in common_utils for the search and info endpoints.  Or, create a private helper `GET` requests method in design_document module.


Doesn't look like the right approach to fetch the query parameters since we the possible keys are knows. Can we use

https://api.drupal.org/api/drupal/vendor%21symfony%21http-foundation%21Request.php/function/Request%3A%3Aget/8.2.x

rather than fetching the entire query string & manipulating it?


I use if block and leave line 1670 out of it because the data value need to be updated by data.slice no matter whether the first part of data is a valid url or not.


> However, in this particular case, letting openssh use a directory within the nix store as its privilege separation directory (whatever that exactly means...) at runtime seems wrong to me.

Not really following; what's wrong about it? The only assumptions it seems to make about `/var/empty` is that it be empty and owned by root, and the version in the store seems to satisfy that requirement just as well as the global one. If anything, putting it in the store helps guarantee that the contents will never not be empty, unlike `/var/empty` which is only empty because by convention we don't put anything in it. This is what I could find on the topic: http://opensource.apple.com/source/OpenSSH/OpenSSH-74/openssh/README.privsep


Why is the response cache dependent on User accounts? Will the response vary based on the user logged in?


@piyuesh23 : I will work on the changes mentioned by you.
All the custom solr search file changes are temporary for REST API because Prateek needs to integrate the API on mobile so he need Endpoint.
So I kindly request you to merge my code.


This looks good to me.


You can use `simplescreenrecorder`  here the [instructions](https://gist.github.com/samely/a14933c4c29ee4d7c7c445526f5f2639) .


Weird... yes, I did a series of tests on 1.11.x *and* 1.10.x with the videoconference.


The library should still work with 0.10 but I don't test that.


From w3c : https://www.w3.org/TR/WCAG20-TECHS/H67.html and : https://validator.w3.org/nu/?showsource=yes&doc=http%3A%2F%2Flimesurvey.sondages.pro%2F

No issue .


Fixed, thanks!


Weird, I'm pretty sure I had this added.


Maybe the hard-coded json part can be externalized?


You're right, It does give details that are outside of the scope for design documents.  I'll remove this.


ahhh, that's probably the one that caused all sorts of errors after `jhalfs` finished running in my build


should stop these as well


I doubt that this revision is specific enough. To "remove the evidence that you were circumcised" might be interpreted as destroying a legal document, for example.


ok to use store for now. Nothing stops us from using ``Store`` later if we decide to move namespace metadata management back into the ``Store``, right?


I use LICEcap http://www.cockos.com/licecap/ (It is for Windows and OSX)


I just tried downloading ti and it came to 33MB (albeit in EPUB format). As for what you can do you can always reduce the max size further from 15 to 8


@vilchik-elena it's what we can test with with the new `SensorContextTester.coveredConditions`


@vilchik-elena thanks, leftover from refactory.


Good.


@mpaladin See how it was done for JS: https://github.com/SonarSource/sonar-javascript/blob/master/sonar-javascript-plugin/src/test/java/org/sonar/plugins/javascript/lcov/CoverageSensorTest.java#L98


(2) Add a link to our plugins in the WP plugins page, like Barry did for Never5.


Not necessarily.  I think any reference to Fluo, updater strategies, and storage strategies should ultimately be removed from ConfigUtils given that RyaDetails will handle this.  I created a sub-task for this in RYA-81.  Since the string will have to be deleted from ConfigUtils, it makes sense to keep it around wihin your PCJIndexerConfig.  Of course, we can always add it back in later as needed.  But since we know the changes will need to be made, it will be one less thing to do later.


See above.


@vilchik-elena this test is changed by #24. In any case, they don't have different base dir, this fs overwrite the context fs.


負荷軽減


Because I forgot to write the description. Now there isn't just one set of blue/white bones - there are two, and the blue bone catches up to the white bone!


Builds folder: https://www.dropbox.com/sh/vowd9qzzy5hosen/AAAydEjM_cf0x2bbYZq7oZr4a/a2dfe9caa31f7491ad3ddd31b41f77286be56134?dl=0


113837787-High-Temp-Radiant-Energy-to-Surface-Imbalance (mjwitte) - x86_64-MacOS-10.9-clang: OK (2228 of 2228 tests passed, 0 test warnings)

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


I also added a unit test for this so it won't happen again.


Ugh.  Fair point.  Yeah, I guess I'll fix that.


Yeah.  Think you added most of these in.  You might have moved some since around since then.  But this class should have access to ConfigUtils.  I can delete them.


软件设计文档


I don't know how to avoid the recursion here. We would need something like FormulaManager.transformRecursively() but it with a generic return type.


Doesn't needed


symbols[ i ].charCodeAt() can be executed just once in a cycle and saved to some var.


charCodeAt() - better use 0 as an argument


Hi, I have a feeling, the above changes are causing onMessage event not to fire, whenever websocket factory is used as a websocket resource.

https://groups.google.com/d/msg/autobahnws/_tsjB2-JRuY/cmNMOUH1AgAJ


a cho ne v console?


Thanks, I'll have a look through when I get a moment or two.

Just from looking, SP numbers for currently training skill are produced with `<!--[$skill.skillpoints]-->` and `<!--[$skill.skilllevel5]-->` for level 5, hopefully works for other levels too. So that should solve the issue with queues.

Was a bit of an oversight on my part, since the first skill in the queue should be the skill being trained I assumed it would be fine, but the code using the output from the training api if there is no queue.

I did actually check this against a character with no training, but only on my dev environment, which at the moment is on the same server as live, so the warnings didn't show, d'oh.


hmm it should be `(price*(1+vat_value)/100)`


Could we use approach without cycle?


wrong indentation


why you add `&` before `.`


#5412-Mixed-air-setpoint-less-than-0-causes-max-OA-operation (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2268 of 2268 tests passed, 0 test warnings)

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


Hey @gmkurtzer, It is my code but this was just a draft. I never submitted a pull request for this so I'm not sure what's going on. But I finished and debugged this and will be submitting a pull request for the finished code a bit later.


#5412-Mixed-air-setpoint-less-than-0-causes-max-OA-operation (mjwitte) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


call is a name of method in Function.prototype. Will be better to use different name


Oh, okay. Thanks for explaining.


why exit 1 here?


@mpaladin it's quite significant change. Are you thinking about minimizing it?


test


You are right, sorry about that. While I agree with not wanting to force the latest version of JSON.NET it seems that I'm unable to use v9.0.0.1 of that lib as InfluxData.Net explicitly specifies v8. I'll look into that.


Thanks, I had missed that.  Wrapped it in an if to take care of it.

On Fri, Jul 15, 2016 at 1:48 AM, Ralf Becker <notifications@github.com>
wrote:

> @nathangray <https://github.com/nathangray>: have you read the
> Scrutinizer comment:
> The variable last_format does not seem to be initialized in case last on
> line 3088 is false. Are you sure the function formatDate handles undefined
> variables?
> Please check and fix if necessary.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/EGroupware/egroupware/commit/5fa3d455a5d4c79453656d3e876a827ed41e76e6#commitcomment-18257690>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AJ7PHphyfJa1AH7ojBOWjiWFwBwe_XR9ks5qVztOgaJpZM4JNKDu>
> .
>


DeleteFromBasket, but delete what? name can be better


What sense have this last two tests?


Live statistics


TeamCity OTA+ :: rvi_sota_server :: ci [Build 517](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=8319&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


TeamCity OTA+ :: rvi_sota_server :: ci [Build 518](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=8320&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Thanks for pointing it out, if i recall well ( i haven't looked at the project in a while ) it was the intended behavior for the project i was using Velox on.


I agree that we should allow to execute code with warnings, but it seems that this proposal will execute allways, not only in the presence of warnings bat also in presence of errors.

Errored code will most probably result in an execution of an inconsistent program, which can produce very obscure errors. So I would avoid it.

Maybe we could add parameter in the request, to say "stop on any warning" (or even) "stop on issue level", being the default to stop only on errors. But I do not know if it is worth.

If you want to deploy it right now, I think the best is to PR the branch and I can integrate it and deploy it. Later travis should do this automatically... we will have to decide from which branch to fire automatic deployments.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 517](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=8319&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:02:35


Тут, по идеи, массив не нужен.


hm, tomcat expects .war files to be placed in `$out/webapps/`

https://github.com/NixOS/nixpkgs/blob/a04a727/nixos/modules/services/web-servers/tomcat.nix#L248

just a consistency note


Thanks! :)


the commit message was meant to mention https://github.com/rilwis/meta-box/issues/921 instead of #921


You didn't check the return value of function `New`, which allocates memory via `malloc`.


@vilchik-elena nope, we have to bundle guava and apache commons now, see commit comment.


Of course -> Done in 1d0cbcf


Including the formats in here creates another issue when creating a new image from within MAME, it will always use the first geometry in the list so will be 160K, Archimedes should create 800K by default.
I'll keep all formats in a new acorn_dsk but will separate bbc_adfs_format into new acorn_adfs_dd_format and acorn_adfs_qd_format.


Can you minimize the cost of memory walking by prefetching?


@mpaladin We did this way in JS https://github.com/SonarSource/sonar-javascript/blob/master/sonar-javascript-plugin/pom.xml#L87


TeamCity OTA+ :: rvi_sota_server :: ci [Build 518](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=8320&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 210, ignored: 3 Build time: 00:08:21


I should have mentioned this in my PR comments, but I did not run `grunt` in my commit so the `dist` files do not have the changes from my commit.  I thought the appropriate time to generate the dist files would be when the version was bumped.  Sorry!


My understanding of the runtime APIs (although I'll admit is limited and possibly woefully outdated) is that they cannot be executed from the security context of an extension's injected content script, and require a dedicated background page and message passing.  I believe that desktop notifications can be generated from content scripts but...

Either of these changes would require a change in the security context of Enhanced Steam.  By default, this means users which installed Enhanced Steam via the Chrome Web Store (the majority - nearly 93% of all users and over 99% of all Chrome users) would have the extension disabled and would require the user to re-enable the extension and accept the new permissions.  I believe desktop notifications have an additional permission the user has to agree to before they're shown as well, after the first notification attempts to fire.

I think this method is an acceptable compromise to work within those limitations.


unphar dude.

and the auto updater is not needed, but dont just delete it, other files use it so also remove the lines that call the autoupdater


Can we use a full URL here just so its clear where its coming from.


Why do you ignore `$SIG{__DIE__}`? With this snippet of code:
```perl
use Log::Log4perl;

sub DIE_handler() {
  print 'DIE Handler - ' . "@_";

  exit(1);
}

sub main() {
  my $conf = q(
    log4perl.rootLogger=TRACE, syslog

    log4perl.appender.syslog = Log::Dispatch::Syslog
    log4perl.appender.syslog.facility = mail
    log4perl.appender.syslog.socket.type = tcp
    log4perl.appender.syslog.socket.port = 514
    log4perl.appender.syslog.layout = Log::Log4perl::Layout::SimpleLayout
  );

  Log::Log4perl::init(\$conf);

  my $logger = Log::Log4perl::get_logger();

  $SIG{__DIE__} = \&DIE_handler;
  $logger->trace('TRACE');

  print "Log message has been written!\n";
}

main();
```
with v2.54 I have:
```
DIE Handler - no connection to syslog available
        - tcp connect: Connection refused at /opt/perl/lib/site_perl/5.16.3/Log/Dispatch/Syslog.pm line 109.
```
with v2.56 I have:
```
Log message has been written!
```


Should make this go all the way to 20 I think?


@Alexander-- :

Now that I have had the time to study up on thread interruption and Future cancellation, to try implementing a range of strategies, and to think through the ramifications of various design decisions, here is what I have learned.

> You don't need to know the number. Don't even try guess it. You are expected to derive the desirable degree of parallelism from amount of work you have to do. Then keep working normally, whether or not that amount of parallelism is available.

This is only correct if there are no data dependencies between any of your work units. Your assumption is incorrect if there are data dependencies between your work units (as in my case, with stage 2 depending on stage 1, and stage 3 depending on stage 2), because:

1. As pointed out previously, your ExecutorService is not obligated to service requests in a FIFO order.
2. You can enforce a FIFO order by wiring your tasks together with your own FIFO queues.
3. However, using your own FIFO queues can lead to deadlock if "downstream" tasks are scheduled before "upstream" tasks, and you have more downstream tasks than available threads to run in.
4. Additionally, even assuming that your ExecutorService schedules tasks in FIFO order, if the last task is the one that generates the Future that you return to the user in an async method (which is highly likely, because the final result of an async method will be at the end of the dataflow DAG), then if you call Future#cancel(true) on the returned Future but this last task *hasn't been scheduled to run yet* because there are more uncompleted upstream tasks than there are available threads, the cancellation request will be silently ignored. (Setting the interrupt status on a thread that hasn't started yet has no effect. I tested this.) This can cause the inability to cancel a task until the final task gets scheduled to run.
5. The poison pill staging technique doesn't work if you don't know the number of worker threads, because you don't know how many poison pills to add to the queue to signify the end of one stage and the beginning of the next. Restated, to use the poison pill technique with more than one pair of communicating threads (e.g. an M-to-N relationship between M threads in stage `i` and N threads in stage `i+1`), you need to know either M or N. However, you can't pick a safe M or N without knowing the number of available threads in the ExecutorService, for all the reasons listed above.

In the end, the solution I figured out for FastClasspathScanner was to put stage 1, one worker for stage 2, and stage 3 all in the same thread, running the stages in order (call it task 0). Then N-1 other threads are spawned that just run stage 2, where N is supplied by the user. Task 0 is the one that returns the Future<ScanResult>, so that is always available (it actually doesn't matter how many worker threads can actually be spawned by the ExecutorService, as long as we can allocate at least one.) Therefore, N now becomes the level of desired parallelism, regardless of how many threads are available. The deadlock issues etc. described above don't happen.

> > if one worker thread is interrupted, I want to interrupt all the other threads too
>
> Why would you do that? It makes no sense! [...] make the interrupt and end-of-input do the same: simply stop processing within this particular Callable. When your main thread (if you have any) gets interrupted, cancel worker Threads. If your worker threads get interrupted via `Future#cancel(true)`, the code that does so is expected to know, that cancellation have taken place (and one would interrupt all Threads at once anyway).

This only works if your caller thread actually has references to a `Future` object for each worker, so that the caller can call `Future#cancel(true)`. However, given what you have repeatedly proposed (internally having FastClasspathScanner submit fine-grained work units to the ExecutorService, and returning a single `Future` representing the result of the scan), the caller does not even have a reference to all the internally-used `Future`s.

In fact, it's even worse than this: the caller may not even have access to the references to the individual Threads (depending on whether or not it constructed the ExecutorService itself) --and it certainly doesn't know which Threads  are currently being used by the ThreadService to schedule work that must be canceled (unless all the threads in the ExecutorService are dedicated to the workers of this particular task -- but then it still needs references to the Threads in the ExecutorService to set their interrupt status).

The general situation is that a library API that exposes a single async, parallelized method will itself spawn off a bunch of jobs using the provided ExecutorService, and it will return a single `Future<Result>` object. *If the `cancel(true)` method of that object is called, the only reasonable thing to do is for the library to cancel its **own** tasks that it has spawned.* After all, it has references to all of them. It is completely unreasonable to expect that in the general case, the caller of a library method will be able to figure out exactly which Threads may or may not have work from the library running in them, and set the interrupt status of those threads.

Therefore, the solution I put in place (of having a shared AtomicBoolean between all the worker threads, indicating that the threads should all die) is the right thing to do. This is triggered by the interrupt status being set on the thread that returns Future<Result>, or (for completeness and safety) by the interrupt status being set on any of the other threads.


resolved merge conflict as well


Where is it? 😢


You are right, it doesn't. However, up until at least the end of 2013, it seems that the macro mistakenly didn't include `-pthread` in the `PTHREAD_LIBS` variable. The workaround [seems to be](http://stackoverflow.com/a/20253318/912144) including `PTHREAD_CFLAGS` alongside `PTHREAD_LIBS`.

It [seems like](http://lists.gnu.org/archive/html/autoconf-archive-maintainers/2013-06/msg00021.html) this behavior is intentional. The description of the macro is likely what needs to be updated.


added cordova camera plugin


Why removing `reject_non_fqdn_helo_hostname` from `smtpd_recipient_restrictions`? Is this checked (and scored) by something else outside postfix now?


For Testing Purpose. To be removed!


Here and later: converting to bound references will be implemented as https://youtrack.jetbrains.com/issue/KT-13111 for Kotlin 1.1.


Should we update this line because of the unit test we saw that one time? More along the lines of
`Write a unit test for a function you wrote`?


Great work!


`VALUE_IS_COMMA_SEPARATED` -> `VALUE_IS_INLINED_ARRAY` ?


Need change 'aprensentar' to 'apresentar'.


os-release.bb defines BUILD_ID, which triggers this warning multiple times when parsing all recipes.

Now that I think about this some more, should BUILD_ID be used at all as input for OS_VERSION when it is not in the special format from Jenkins?

I think I'd prefer to check for the special Jenkins BUILD_ID, use that if found, and otherwise just use the normal default.


buen ojo, acabo de enviar una correción, indicarme si funciona bien.

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


Не оставляй в коде такие места. Перед тем как выкладывать чисти.


@ReDetection спасибо, что заметил!


This is the wrong hook. There's no guarantee that all the content types are available when this hook is executed. Get rid of the loop and put it in hook_field_attach_create_bundle.


By the way, the upshot of this is that: interactive_mode is now automatically detected, so you don't have to pass `multi = True` when using the python console anymore; it will detect it automatically.


😆 This is an old project we had hosted elsewhere that I recently moved to GitHub. `shanecav` is the username I use elsewhere. Disregard!


It still happens evey now and again, that searching with scope on current service gives result, but the same search with scope on current bouquet doesn't. I can't find any logic for this to happen, but when it does, it is consistent (i.e. the same search on the same service always returns this result, also after a reboot).


works fine here but i will wait for a fix for eel-test before merging.


Use the following syntax for specifying changes / disabling activities and scheduled tasks:

`
$CFG->local_uniulm_initsettings_plugins = array(
		'uni_ulm_plugin1' => array('name1.1'=>'value1.1', 'name1.2'=>'value1.2'),
		'uni_ulm_plugin2' => array('name2.1'=>'value2.1', 'name2.2'=>'value2.2'),
		'uni_ulm_plugin3' => array('name3.1'=>'value3.1', 'name3.2'=>'value3.2')
);

$CFG->local_uniulm_initsettings_tasks = array(
		'disable' => array('\editor_atto\task\autosave_cleanup_task', '\core\task\calendar_cron_task')
);

$CFG->local_uniulm_initsettings_activities = array(
		'hide'=>array('assign', 'glossary')
);
`

Add at the end the following code:

`
if($PAGE->course->id == 1){
   if(file_exists($CFG->dirroot.'/local/uniulm_initsettings/lib.php')){
        require_once($CFG->dirroot.'/local/uniulm_initsettings/lib.php');
        local_uniulm_initsettings();
    }

}
`


Olá equipe da Valemobi!

Neste resultado aqui apresentado, sobre o projeto proposto, infelizmente não consegui atingir todos os objetivos propostos, o que já era esperado, pela falta de experiência com linguagens de programação WEB.

Porém, vocês não tem ideia do quanto tudo isso contribuiu diretamente à uma somatória de novos conhecimentos, novas tecnologias, novas tendências, ainda até então, não descobertas por minha pessoa, pois são algumas novas práticas não apresentadas na universidade. Venho estudando linguagens procedurais, orientadas à objetos para desktop, porém esta soma de acontecimentos, me despertou muito o interesse nas linguagens WEB e principalmente linguagens Open Source.

Tenho amigos desenvolvedores Front-End, Back-End e poderia tê-los usado para cumprir todo o proposto, apresentando um código impecável e com todo proposto finalizado. Porém, prefiro usar da coerência e mostrar que realmente preciso aprender e muito, e vontade para isso não falta, por isso o enorme interesse nessa vaga de estágio, principalmente em uma startup que valoriza muitos conceitos de que prezo.

As falhas em meu código permitiram identificar como é possível programar de uma maneira mais objetiva e rápida, com reúso de código e com uso de ferramentas incríveis. Meu projeto foi separado em 04 arquivos, sendo um index que mistura tags HTML com "chamado" individual de bibliotecas CSS e JavaScript, uma página de estilo em CSS para personalização, uma página de processo para conexão com o banco de dados MySQL e outra página de validação, em JavaScript, que não ficou completa. Conversando com um amigo, o mesmo me mostrou a facilidade do uso de API's e economia de tempo realizando-as em linguagem Back-End (PHP), trazendo-as para Front-End, o que facilita muito a obtenção dos resultados propostos, principalmente com aplicação dos Json's, que são fantásticos. Também pude ver a facilidade na conexão com o banco de dados local (host) utilizando o MySQLi, deixando a aplicação muito mais leve por conta da troca de informações e alimentação do Database, utilizando as API's criadas, o que também facilita as manutenções de código. O que mais me chamou a atenção foi a facilidade de aplicação do AngularJS, que mesmo utilizando-se do JavaScrip, é muito poderoso, flexível, cria "guardiões" independentes, fazendo com que toda aplicação JS seja aplicado diretamente ao script do HTML. Agora quando aos Framework do bootstrap, são de uma ajuda incrível, pois ao invés de realizar parâmetros individuais para um mesmo layout, os Framework aplicados deixam a página com um visual incrível, e com muita rapidez.

Por fim, quanto ao projeto realizado, não consegui hospedá-lo em um servidor WEB, na qual pretendia utilizar o OpenShift. Agora quanto ao código, é fantástico comitar códigos ao Github, e a oportunidade que o mesmo oferece de enriquecermos nosso conhecimento, e ao mesmo tempo colaborar para os projetos de outros desenvolvedores. Junto ao Github, foi upado os quatro códigos fonte citados acima.

Quero muita a oportunidade de fazer parte da equipe Valemobi, pois apesar de nunca ter trabalhado em uma empresa de desenvolvimento, fiquei com muita vontade de aprender e tenho muito à contribuir para a Empresa, que terá a oportunidade de moldar um profissional voltada às suas necessidades.

Um grande abraço e desde já agradeço à oportunidade de aprender ainda mais!

Guilherme Monteiro


Need change 'inquerito' to 'inquérito'.


Yes, you're right! I'll update this!


We should also include some musicXML and .mei files. The directory ```midifiles``` thus can just be named ```scores``` ... Also make sure that in the docstring you demonstrate usage of a particular function; if you don't use unit-tests, writing docstring tests automatically documents usage of a particular function.


这里需要和[spring-beans.xml](https://github.com/zhangkaitao/shiro-example/commit/54b9481e9bb54f960504e03b9dcd0bc2f474a889#diff-18016b1df94632f2a0146d1e3b5e7fd7R33)中配置的创建密码签名算法同步维护


Looks good to me. Liking the appearance of files here https://github.com/csgillespie/efficient/tree/master/vignettes - will review, build + submit now.


:+1: for using jQuery!


Yes, this seems to have broken all ability to do random number generation in julia using Distributions.jl. Is there some workaround?


add_coils_to_unitary_system (mbadams5) - Win64-Windows-7-VisualStudio-12: Tests Failed (2188 of 2447 tests passed, 0 test warnings)

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


This is a step back. This wildcard was really helpful, when developing plugins. Here you could just override everything except SQL files and scripts in alpha versions.

This is now no longer possible, and thereby the development of significantly more complex.

Perhaps we should simply admit this wildcard in Dev mode?


Yeah, I think so. Ideally it'd look for a number of well-known suffixes and send a response with the appropriate content type.


То же замечание что и выше. Перед коммитом весь мусор нужно удалять. Будем считать ветку master ветку на которой хранится только тот код который идет клиенту. Т. е. только рабочий. Такая практика нужна потому что программисты ставят всякие консольлоги типа console.log('x*й'), console.log('hehehe'), console.log('wazaaaaap'), и тому подобное а потом эти вещи идут на продукции где клиент потом смотри а там ему в консоль или еще хуже прямо на экран такие сообщения сыпятся. В общем ты понял.
Для следующих изменений будем создавать тематические ветки. Ветки будем сливать только после полного ревью и когда она будет полностью работоспособной.


WHAT?! @ImagicalGamer check this... is it true?


@ReDetection а как оно ломалось? Я нашёл только тикеты, где жаловались, что emoji просто не показываются


There may be servers which have maps outside of the -3000.0 - 3000.0 range.

This would be better:
```pawn
if( hittype == BULLET_HIT_TYPE_NONE )
{
	if( floatabs( fX ) > 20000.0 ) || floatabs( fY ) > 20000.0 ) || floatabs( fZ ) > 20000.0 ) // normal boundaries are in the range -20000 - 20000
	{
		#if defined BustAim_OnPlayerWeaponShot
			return BustAim_OnPlayerWeaponShot( playerid, weaponid, hittype, hitid, fX, fY, fZ );
		#else
			return 1;
		#endif
	}
}
else
{
	if( floatabs( fX ) > 1500.0 || floatabs( fY ) > 1500.0 || floatabs( fZ ) > 1500.0 )
	{
		#if defined BustAim_OnPlayerWeaponShot
			return BustAim_OnPlayerWeaponShot( playerid, weaponid, hittype, hitid, fX, fY, fZ );
		#else
			return 1;
		#endif
	}
}
```


@diegosteiner very nice!


Can you use a more generic she-bang here?


префикс можно не передавать а писать напрямую в шаблоне статьи. Он всегда одинаковый и к тому же не имеет отношения к полям объекта article. id - да, title - да, content - тоже, а prefix - это можно оставить шаблону.


It could be! These were the errors fixed by this commit:
/home/lfs/GoboALFS/LocalFunctions: line 177: sudo: command not found
/home/lfs/GoboALFS/LocalFunctions: line 178: sudo: command not found
/home/lfs/GoboALFS/LocalFunctions: line 179: sudo: command not found
/home/lfs/GoboALFS/LocalFunctions: line 183: sudo: command not found
/home/lfs/GoboALFS/LocalFunctions: line 184: sudo: command not found


You can remove this now - checking


commit


Typo in method name - should be `IfBypassUrl`


Without doubt, it has some benefits, but it does also introduce a whole bunch of other issues. It needs to be taken into account when running the automatic update and this was always a great source for issues, both in terms of miscalculated update paths and unintended updates.

This is a bit like catching `Exception` when you really just wanted to catch a specific exception: It does the  job, but can turn into a disaster. Especially in production systems.

Long story short, we've eventually came to the conclusion that this wildcard causes too much trouble just to allow developers to be a bit more lazy. We can understand your argument, but we simply felt way too uneasy supporting this any longer.


#5412-Mixed-air-setpoint-less-than-0-causes-max-OA-operation (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1677 of 1677 tests passed, 0 test warnings)

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


I had no clue. Sorry. Kinda thrown off by the fact that it says "firm0_offset". Any reason for two of them btw?


How about changing for future reference: (?P<value>[^&#]+) to (?P<value>[^&#]*) if user gives empty parameter/s?


Тут может быть так:
`<article class="article" id="art-<%= id %>">`