ouch, I hope I don't offend you ;)
The idea was just to warn you to not spend too much time of cleaning the master as the Leshan Client code is changing a lot . If you want to help on the branch you're welcome :)


First, this commit seems to fix more issues, not all related to the %20 being displayed in the history log. There should be one issue for each problem the commit addresses, and all these issues should be mentioned in the commit log.


nice catch,
will fix soon


Thanks for the fix Roman! There's still one issue though, that I mentioned [here](https://youtrack.jetbrains.com/issue/IDEA-132606#comment=27-1267389), when StubBuildingVisitor reads class references (supertypes, method return types etc).

The problem is in `SignatureParsing.parseParameterizedClassRefSignature()`, which also transforms `Lfoo/bar/Class$impl;` to `foo.bar.Class.impl` even if the correct class name is `Class$impl`.

Could you please have a look at this as well? Thanks.


@dinkoivanov  - we should send the error to the Activator.logError() method - so they're easy to navigate.


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


In all places where you call free(pfh) it is better to set pfh = NULL. But again, I suggest to avoid dynamic allocation for pfh at all.


Nice streamlining!


hey - you did a lot of research there, well done.

Again, could you send the errors to the Activator?


You'd normally use the `children` (the third argument) to the `view` function here, so that the usage reads better (and you can stuff other html and components inside —

```jsx
<f.alert>
  <div class='exclamation'>this is a cool alert</div>
</f.alert>
```


Ah, sorry.  Artifact of testing out always stripping the leading slash.  It worked, so, I'll remove the comment. :)


Are all these changes about adding extension points in PatientHeader? Or are you doing some other things in addition to just that?


You need to add the [sponge repo](https://github.com/SpongePowered/SpongeCommon/blob/master/build.gradle#L13-L16) to the classpath repositories first, or it won't be able to resolve the Gradle plugin. I can also change this if you want, don't want to break everything if you're doing it right now though ;)


I'll do it tonight.


Thanks :+1:


Hmm, can we infer that propertyOwner will *always* be some kind of collection?  Passing **Object** seems the RCP way, but we're in a more limited context here.


Don't forget to add trailing dot here too.

Sorry, for my boring comments..


For future will be better to remake it as factory or pool of browsers


如果人体各个


you might want to escape javascript as well in this value? Otherwise, if you have a " in the translation, the javascript will have syntax errors... Same for all these.


why OR feature bits (instead AND them) ?


No, not at all. I am glad you told me :-) But really, I just stumbled across the constructors and the weird way the client address is configured and had the urge to fix this ...
But you are right, the client needs a lot more changes than just a clean up of the constructors. If I find the time I'll take a look at what you are doing in the PR and see if I come up with something helpful.
Again, no harm taken :-) I think it is important that we can tell each other what we think ...


Do we really need to have world-writable file here?


i think you need to make it class_exists('APCIterator', false) otherwise you get autoloader error ( or at least I do ).
Sorry if i am annoying you :-)


We will reconsider that datetime type and we will talk to Siva and our BAs whether storing the end_datetime is needed or not because we havent not touched that part during inception


With `Predicate<T>` the following code:

```js
t.String.meta.predicate(1); // => false
```

would raise an error. However it's a legit code.


@alexsuter Heder zruggschribe?


why using a translation from the profile? (password change translation?) What if the translation from the profile changes to something more specific to that screen, something like "You are not allowed to change the password"?


same remark about the translation key.


image fix


forgot a $services.localization.render?


Yeah.
I didn't meant "graceful", I've meant "hard". :)


Nei er het eifach Kontrollfrage vo CDI ufeglade. Chasch die no kontrolliere? Bi de Frag 1 isch chli kommisch.


If we're fixing translations, let's fix this one here too


Notifying more than one person about an alert is spammy and an anti-pattern. This should default to false.


we should really use keymap...


I found no duplicate Object array ( an array of the same objects ( same = equals ) )


Sorry, I meant that this should be a `TypeGuardPredicate`. Shouldn't it be?


I'd suggest using __subject here, and have the format as text


@clausjensen I'm slightly worried about this change since there's been so many other little bugs regarding these date formats, moment and this date picker in the past.

* 10108d75b8746d4697df3feea9ec41eab81f56f7
* 24cb64579b8fe7220217c9fff2a8e98b42a633f4
* 86239cc7778135819c6697f24839bb3fb7d64c2e


same remark about the translation key, especially as you have a key for this one, it seems: xe.blog.manageCategories.js.add.error.403


I don't know where to get a list of factions, layers and techlevels from in the CreateBrainShared function that correspond to the General.FactionName, General.Icon and General.TechLevel fields of blueprints.

This code may look ugly but it will continue working if a modder creates a new faction or a new tech level.


The cast to `Stream<T>` requires them, but I'm open to options.


 -
 +
      Google
 -
 +


It's my belief that we should move away from the AsyncTask framework since it was just an attempt to easy the UI synchronization from threads.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10712](https://teamcity.mesosphere.io/viewLog.html?buildId=100255&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:08:43


When would language not be in user?


我看代码好像现在没有定期检查pool里面pooledobject是否有效的策略？对于数据库连接池来，我不想在borrow或者return的时候做检查，这样做太低效了。


You need to switch the message between the HTML templates and the text templates based on this so the escaping is correct. It should probably just be a plain string.


aaaassssssssssssssssssssss


@evilaliv3 Thanks for this!  Do you want to create a pull request w/ the correct URL's?


I think i know the reason for this change - essentially the $scope.model.value format must always be `YYYY-MM-DD HH:mm:ss` because that is how we persist it, whereas the `$scope.model.config.format` is used for display purposes only. Is that correct?

I guess the other change is on line 103, is that change required?


So here you don't use the style change that you do earlier ;)


The return dict(x=y) matches the web2py docs which is useful...the performance increasde is marginal, so I'd leave as-was


ahh sorry I forgot those tabs.. dunno why .editorconfig doesn't work in Atom.


As far as I understand it did, but the disposal pattern isn't something I grasp too well.


Create and throw System.IndexOutOfRangeException() without calling an internal runtime method.


get_auth = self.auth.get
id = get_auth(...)


This looks slightly less efficient than `TYPES_DEFAULT_VALUES.containsKey(modelType)`.


Could you do either of:

  * Keep the `/* Moments */` section and move the literals there instead of the deleted items.
  * Move `MomentConstant` & `DurationConstant` below `MomentLiteral` & `DurationLiteral` and move all the moment and duration stuff above `ArrayLiteral` (the idea being that compound literals should follow simple literals).


+1 nice one


`get` once and check for null here for efficiency?


Hey pananormalny,

Care to send in a PR with those 3 recent commits of yours? :smile:


You are right, let's merge a simpler version first and then we can work on adding multi-pass support in apitrace retrace.

This part won't be trivial though and could require glretrace to return not only the number of passes necessary, but also the actual configuration that needs to be executed. One way to simplify this could be to let glretrace trace as much as possible and let apitrace retrace figure out what is missing and call glretrace again.


Thanks Markus!


Just to imply immutability.
If I'm not mistaken this also forces evaluation of the query instead of deferring that to the caller.
@RoosterDragon has more insight on this, LINQ isn't my strong point unfortunately.


Maybe `rmSecretsCmd` would be a better reflect what this variable contains?


You got them to add it for us? :)
Nice!


This is cool!!


I would add more detail, at least what value exactly was invalid. Also I would see the expected format.


当A，B其中一个容量为0时特判。
求出两种飞机的性价比，使得性价比高的飞机尽量多。
如果除不尽，则少一架性价比高的飞机，多一架性价比低的飞机，看看能不能坐满。以此类推。

需要注意的是，出现除数的地方要注意有没有可能为0，避免掉坑。


how to download?


What the reason behind using `fmt.Sprintf()` here and `strings.Join()` on the next line? Why we are not using `strings.Join()` here, for instance?


Just set `disableGzip: true` to avoid this.
Actually, it works fine and makes no negative effect. I'll keep these dry codes until I find another elegant way. AFAIC, it crawls sites faster.


`s` a bit confusing for me. I would name it `sl` at least.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10711](https://teamcity.mesosphere.io/viewLog.html?buildId=100256&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:16:27


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10715](https://teamcity.mesosphere.io/viewLog.html?buildId=100260&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 178 Build time: 00:04:09


Hover over the td right next to a td containing the FontAwesome icon, and then the icon blends in to the background because of this code line
```less
td:hover > .fa {color: @body-bg;}
```


I don't think WITH_LTTNG should default to ON. configure.ac defaults to off, but it has a `$have_good_lttng_gen_tp` check that auto-enables it.


You could *probably* get away without defining this function on `this`


@temasek wanted to know. do you also have systemUI fc after while adding tiles after recent cm changes to qspanel?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10714](https://teamcity.mesosphere.io/viewLog.html?buildId=100259&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 178 Build time: 00:06:36


The reason I used important was partly because I couldn't guarantee the order the styles would be applied in and partly because the styles should be enforced.

I don't think using !important in this situation is an issue as print is the endpoint and should be consistent, nobody should need to override any of the styles defined in this stylesheet.


CNAME change to zyuu.xyz


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10713](https://teamcity.mesosphere.io/viewLog.html?buildId=100258&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:07:26


> This removes all tests for expressions like "last 3 days" since
> these are no longer supported by the moment-parser grammer.

Why? Are there any other grammar changes? If so, could you summarize them, along with reasoning behind them? (This is a language change, so I care, plus it has a documentation impact.)

Also, typo: s/grammer/grammar/.


sorry for asking again. why not store the current connection state in `CPVRClient` and only pass the current state into the callback method. This way `CPVRClient` knows the connection state and IMO we could use the connection state for other things + drop the member `m_bReadyToUse` which is currently used to indicate a connected client and rely on the state instead.


add Animation completion block


:+1: I second this


@ncannasse: Could you test DCE times on this branch? I don't really have any code where it takes very long to run, but this should speed it up a bit already.


Not really, no


can we name the library target `rocksdb`, and use `build_rocksdb` for the add_custom_target?


LD_PRELOAD i guess xD
This fixes the cam but unfortunately causes trouble to fb,messenger, whatsapp, snapchat, insta blah blah blah


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


None of the OAuthAccounts returns language.


The manual.sxw file from directly before the migration to ReST is available here:
https://github.com/TYPO3-Documentation/TYPO3CMS-Guide-FrontendLocalization/commit/5ba12a7c2f2b1f429d297b6ad58a79fb4877e21a

Does this enable you to update the table?


remove console.log


Yeah, too bad I missed that.


@ged-odoo this line cause edge browser fail since it don't have window.Notification
I think maybe need fix this


@zeazri - what do you think about this hack?


Would be a good idea to rename to install-meteor.sh


Oh yes! You are right.

Moreover now I can also remove all the `is: TypeGuardPredicate<T>;` overrides in the combinators... I don't get, this morning I've got a problem with `is: TypeGuardPredicate<T>` in `Type<T>` but now everything compiles fine. Well whatever, I'm still a noob with ts, thanks for pointing out :)


web2py is also moving to drop the dict constructor where it is not needed.

{x: y} is more pythonic, recommended, 8x faster (not marginal at all!), the original web2py style was rather poor (which the web2py devs have recognized). I would not change it back, our style is better.


Ultimately, I think `{Moment,Duration}Literal` and `{Moment,Duration}Constant` should be unified and the processing currently done in the semantic pass should be done in the parser already. (Just noting, this is out of scope of this PR.)


please add the include directory to the rocksdb library's interface, i.e. `target_include_directories(librocksdb INTERFACE "${CMAKE_SOURCE_DIR}/src/rocksdb/include")` instead of adding it to the os target below


@DjalmaPessoa @hannesmuehleisen  hey djalma, can you look at whether this removal of the fpc= parameter from censolite is acceptable for the standard error computation?  it will make the script run much faster, i believe.  thanks


Not yet ;) But working on that. It's open source after all.


:+1:

On Tue, Jan 5, 2016 at 2:08 PM, Silvio Traversaro
<notifications@github.com> wrote:
> [sensors] Add Sensor postfix to Accelerometer and Gyroscope class names
>
> This is necessary because the ACCELEROMETER enum value was mapped in matlab
> to the ACCELEROMETER.m file, while the Accelerometer class was mapped to the
> Accelerometer.m file. There was not problem in Linux, but in OS X and Windows
> the filesystem are case insensitive, and so this clash was causing problems even
> when just cloning the repo.
>
> cc @naveenoid
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/robotology/idyntree/commit/b1cd10511fa4deb2f6d3278d3d48cced35e5a550


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


This should not have been removed


I think we should pin the dependency to a version to force us to explicitly acknowledge updated versions.That way we can safely know which version of GCDWebServer we ship with each version of our app.


Following our conversation I think it would be a good idea to create a plugin that links the scripts to your folder of choice (based on json file), you can see less package as a reference. you should probably also make sure to add +x permissions


Maybe this would work as a given-when? Even with postfix `when`s, since we return here


Is this patch still necessary? I thought you were able to build on master without it.


En alle scrum/release masters overgeslagen ook nog


We need to add the MIT license


Wieso komisch? Also Inversion of Control isch eifach sparadigma uf dem basiert auch z.B. Strategie Pattern usw. und DI-Pattern. Vo mir us gse scho richtig oder nid?


do `headers` and `payload` need to be functions?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10716](https://teamcity.mesosphere.io/viewLog.html?buildId=100261&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:16:35


you should probably add support for username and password


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


No problem, just glad I can help!


Can we specify to which exact version we'd like to up/down ?


# self test
#learning how to use github


Hi Stefan-
You should try adding this logic to LMR.  Similar to cmh.
So history<0 && cmh<=0 && gp<=0..


Hi @TomBZombie,

i am only curious, is this really your expected behavior?
when i adding new rule, it will take rule from * and merge it, so if i have add rules after defining default rule, the default rule will be copied to all new rules

e.g.
```

        $dice->addRule('SingleInterface', ['shared' => true]);
        $dice->addRule('*', [
            'substitutions' => [
                'DummyInterface' => ['instance' => 'Dummy'],
                'SingleInterface' => ['instance' => 'Single'
                ]
            ]
        ]);
        $dice->addRule('SingleInterface2', ['shared' => true]);

        var_dump($dice);
```

the result of var_dump is
```
object(Dice\Dice)#16 (3) {
  ["rules":"Dice\Dice":private]=>
  array(3) {
    ["singleinterface"]=>
    array(1) {
      ["shared"]=>
      bool(true)
    }
    ["*"]=>
    array(1) {
      ["substitutions"]=>
      array(2) {
        ["DummyInterface"]=>
        array(1) {
          ["instance"]=>
          string(5) "Dummy"
        }
        ["SingleInterface"]=>
        array(1) {
          ["instance"]=>
          string(6) "Single"
        }
      }
    }
    ["singleinterface2"]=>
    array(2) {
      ["substitutions"]=>
      array(2) {
        ["DummyInterface"]=>
        array(1) {
          ["instance"]=>
          string(5) "Dummy"
        }
        ["SingleInterface"]=>
        array(1) {
          ["instance"]=>
          string(6) "Single"
        }
      }
      ["shared"]=>
      bool(true)
    }
  }
  ["cache":"Dice\Dice":private]=>
  array(0) {
  }
  ["instances":"Dice\Dice":private]=>
  array(0) {
  }
}
```
in rule singleinterface2, i will also have substitutions from default rule, which is duplication..


rewgregrg


We need to alert the user in case he tries to stop the execution of a migration using ctrl+c
You achieve that using "trap"


Good Job!


Solving this issue https://github.com/symfony-cmf/standard-edition/issues/45#issuecomment-168789066


If I think about it it might be good idea to move the ci scripts to a different folder (maybe scripts/ci) to avoid confusion in the future


Updated version:
https://github.com/popcornmix/xbmc/commit/985e2824ffaa7eb4d359f9bff47b1e37f6f81246


"Parity", то есть "паритет"? Допустимо. Но, на всякий случай, "ничья" - "draw" :smiling_imp:.


Just a heads-up.  This shouldn't really have any additional functionality, besides copy the symbolic links's target directory, without any of its contained files, because Walk (https://golang.org/pkg/path/filepath/#WalkFunc) doesn't follow symbolic links.  This would be a good starting point if someone needs to actually recursively copy all files within the symbolic link's target directory.


Can you please push that to extensions.gnome.org?


Joe, this makes sense. Thank you for the fix!


This needs to be adapted to v9 as accounting has changed


You're using `Routes` but not requiring it via `js-routes`


remove


Fixed in https://github.com/guardian/subscriptions-frontend/pull/306 & https://github.com/guardian/membership-frontend/pull/915


Yes. I checked this https://maven.apache.org/guides/plugin/guide-java-plugin-development.html


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

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


already done at 65962672857f21acb99b47e3ff9b63f9da0591c0 :+1:


Offending line.


Can we use a serial number instead ?


How is this accessed? The web page (index) doesn't seem to include links to \crs, though a decent amount of processing happens when loading the page.


That's correct. Without this behaviour, as we had in dice 1.0, the default rule was completely overridden by other rules. If you wanted some default behaviour you had to add it to every single rule.


It seems more heavy-handed to me to make people put files in a certain location.


Perhaps set `errorMessage` to blank?


Very nice.


oops, fixed!


Can we use `([Template strings](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/template_strings)) notation instead ?


You don't need this


No quotation needed here and below


I will move the necessary term id's to variables so we avoid this.


Add new line here


Let's remove default max size and require to specify it explicitly.


Typo


Sort lib: and devel: alphabetically


Separate request building


I think you forgot this


timestamp lo guarda una sóla vez al instanciar el modulo y después queda fijo, la idea es que lo calcule cada vez...


This must also be ordered incorrectly...


`if thisRef?.length and thisRef isnt '#'` generates more javascript than `thisRef and thisRef.length and thisRef isnt '#'`


Would be good to add the review reference in commit comment via `+review GRADLE-xxx`. Makes it much easier for the later review in Crucible.


So as default unit?


@IAHM-COL, absolutely beautiful. Coasting along taxiway Foxtrot in the Bluebird -- all signs are just right. With Gilberto's new textures... wonderful!


You might want to be careful, `withdraw` and `deposit` aren't guaranteed use the default balance for the specified currency if there is no balance for it. Just in case, I'd recommend checking if it has a balance and then setting that balance to the default if it doesn't.


You need to escape the string with "\" at the end, otherwise you'll have a line break.


but is it really okay?, from my example, *singleinterface2* rule will have duplication, because by default it will **array_merge** from default rule, if i have a lot of substitutions and a lot of additional rules, the array size will become so big..


`.build()` on a second line.


Please squash your commits.


Thx for the pointer


@tde-banana-odoo the code has changed in v9 so could you please adapt it in a different commit?


Oops, will update :)


A triumph of computer science


Seems like a bad situation to have our CI tooling dictate what we do and don't log, but it's probably outside the scope of #4735 to fix.


Nice try.


I would prefer to change the default units to bytes. What do you think?


When I use python 3 for the installation and source the setups.zsh file I get this error
```
setup: name 'basestring' is not defined
```
I think this is fixed for python 2 and 3 replacing basestring by str using the follow-up commit 2881d9bae3d987004c10053ae61a33fb395cc65f on this PR.


Not sure if I made this change, but it should be reverted (mostly).  We want to use the `Resources` type to extract `id` and `entity`s.


Let's check for the file size _before_ checking it's type.


Change the order (see above).


message should more closely match the name of the property.  So `name must be specified`.


@pikajude You needed this, right? Do we need to (partially) revert 8a7f0dfd68a785d254f7156c4b57c8809eb4bbbe?


Class should be `public` so that other people can use it.


This is my first commit!


La fecha está fija


We're in the "Troubleshooting" section which doesn't seem quite right. Maybe a new section something like "Tips for writing tests" would be appropriate. Also the previous highlight is `.. highlight:: console` so it needs to be changed back to Python for this section.


Thanks. I will add memcached as a dependency for nDeploy-nginx as well.

As a future note . You should be modifying the  rpm_buildtree/nginx-pkg-64-common/etc/nginx/conf.d/pagespeed.conf .The build script then rsyncs these files at build time to the centos7 and centos6 specific rpm's . I will merge this now and make the change.


How about adding
```
import Data.ByteString.Char8   (intercalate, ByteString)
```
so that you don't have to use ```BS.intercalate``` everywhere but use ```intercalate``` directly (and also use ByteString instead of BS.ByteString)


`basestring` is the correct base class to use in python 2.


`Resources.getId()`


这是什么环境是？


It will only be as big as the default rule + the rule for that class. The real answer is to only apply rules to classes that actually need them.

The problem that the merge was designed to solve is this:

```php
$dice->addRule('*', [
    'substitutions' => [ 'PDO' => 'MyPDO' ]
]);

$dice->addRule('MyClass', ['shared' => true]);
```

Without the merge, if MyClass needed an instance of `PDO` it would be passed an instance of `PDO` and not `MyPDO` unless the rule was altered:

```
$dice->addRule('*', [
    'substitutions' => [ 'PDO' => 'MyPDO' ]
]);

$dice->addRule('MyClass', ['shared' => true,     'substitutions' => [ 'PDO' => 'MyPDO' ]]);

```

Leading to repetition. Really the default rule should only be used for rules which really should be applied to everything. Otherwise, add rules to specific classes/interfaces.


`Resources.getEntity()`


28800000 = timestamp + 8*hours;


indeed! will fix and force push a new version!


Ha - great. Please add your name (and the plugin name) to the README in the Plugin directory.


ah, i see you already merged the patch set. Will queue for a later PR then.


Adding to my previous comment, in case we go with the plugin way, we should allow the user to change the default migration dir


In your task #129 you wanna set upper limit to 500Kb.
If we remove default value - we must to remove default unit too.
It is hard for understanding if default value is 512000 bytes.


Why require username instead of defaulting to requesting blocks for self?  To my eyes, this makes the endpoint more complex (by making the parameters non-orthogonal) without resolving any edge cases that I see.


gddfh


Yeah, figured there is no need to hold up stuff for such trivialities.

On Tue, Jan 5, 2016 at 3:56 PM, Lennart Poettering <notifications@github.com
> wrote:

> ah, i see you already merged the patch set. Will queue for a later PR then.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/poettering/systemd/commit/519d39deeeec7121649f28e7287b7790e50d2979#commitcomment-15266588>
> .
>


right, good catch :-)


I see, so thats the thing i need to do, thanks Tom


Nit: Other format strings use `{requested_username}` and `{requesting_username}`.  Consider using the same placeholder here for consistency.


```
Validators
    .stream()
    .zipWith(this.organizationId)
   ...
```


I did this to explicitly prevent it doing string concatenation rather than addition.
I had issues when testing giving me `[11]` instead of `[2]` and figured this is the easiest way to make sure it's always doing addition


If you take path as an argument why do you need to add migrations to the path ? You can allow the script to set that anyway


Good catch! 2 things need to be addressed - if you would like to do them that'd be great - otherwise I'll take care of them.

1) [snowflake.js line 52](https://github.com/bartonhammond/snowflake/blob/master/src/snowflake.js#L52) should not set Version since it ```dispatch```s later
```
device: (new deviceInitialState).set('isMobile',true).set('version',VERSION),
should now be:
device: (new deviceInitialState).set('isMobile',true)
```

2) deviceReducer needs to have a Jest - my bad - if you are up to it, that would be very appreciated!


`Resources.getEntity()`.


As far as I can tell, this would add unnecessary overhead. I'll have a look into whether it's worth using Hexagon to add these events or not.


Why?
Each check generates its own error message, doesn't it?
If i send text file 501 Kb long - i need first to know, that file format is wrong, right?


We need to find a solution for logging here, we don't want to spam the console


Should be `Monoid`


Indetifiers -> Identifiers


i know lel ...typo xD and for debugging purpose im globally loading it.


1) I suggest to quote file (`[ ! -f "%s" ]`) name to protect us from file names with spaces
2) I would add more spaces after `;` to make it more readable
3) Maybe `rc`, `code` or `exitCode` would be better name for `r`?
4) Is it possible that this file never get created? I mean could this loop turns into infinity loop? Maybe we should limit attempts or time here?
5) Because we never validate `s.Destination` and just appended them to the end of the `truncate` command I see here possibility of injecting any code. Hm. Maybe it's impossible. Should we care about security here?


Besides, I don't think we need to copy style quirks from web2py in order to match their docs - we're telling people to copy from our examples rather than to follow web2py docs, anyway (which is also recommendable).

Looking up the dict constructor from locals/globals and invoking it function-style is simply not necessary to construct a dict with constant keys, and the dict literal is more readable anyway. Behaves better, looks better, and it generally considered the more pythonic style, and is more consistent with the rest of Sahana...lots of good reasons to use the dict literal wherever possible. Web2py docs is not quite a heavyweight argument ;)


you should always rebase with master, not merge. Otherwise, you'll get superfluous merge commits


UI is a particular pain point with abbreviations (trying to avoid altogether), but is an abbreviation nonetheless. if we grant it exception status ("Ui"), it becomes an extra thing to remember.


Create a tar for each of the files looks as overhead. Could we create one tar and stream all the files in one shot?


Probably overkill for private methods. Bear in mind forcing the array means you create an array (obviously). If a caller does `FirstOfDefault` or simply evaluates it right away, creating a whole array was wasted effort (unless you're caching that array for later re-use for efficiency).


IMHO `t` is too short for `stiTar`. I would name it `st` or `sti`.


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


I felt so!! ;) lel


> now they fail

But they fail silently, no? As I've been skimming the latest commits, it does strike me that I've got some fogginess around `absolute_url`, `absolute_api_v2_url`, `get_absolute_url` ... this deepens my fog. :)


Should we have the end_datetime different than the start_datetime?


Nice catch. I will change that


I'm not certain I understand. I added the spaces, didn't take them away.


Like, why don't *all* objects have an API representation? Because we haven't converted them all over to v2 yet?


To support this proposal https://github.com/jeffmo/es-trailing-function-commas, babel-preset-stage-0 is required.


Not possible because each tar unpacks the files in different location... We will have to add directories into tar and unpack it to '/' which is even uglier than this ;-)


Did you intentionally use `SimpleTestCase` here but `unittest.TestCase` in the others? Might be simplest to use `SimpleTestCase` everywhere as I expect that will be more common.


Derp, I always end up doing this because on my machine it works because it's in my local maven repo


Following our conversation,
we need to add a script that initializes a baseline. here we should alert the user and suggest him to execute "init" first


@bmuschko I haven't used it myself. It basically adds the commit to review in Crucible which you can also do manually, right?


Why?


I should say the last commits. (plural)


Thanks for the fix.


`v` as short version of `secrets` is counterintuitive for me. Do we have a better alternatives?


Please explain the rationale behind this.


@davidyaha what up ?


@php-coder there is a timeout for the build itself so when it gets to infinite loop, it will get killed after some time.
I don't think that case can happen, if the upload fails then we return error from the Execute() function immediately and it don't even gets into launching this command.


Looks good, but could you rephrase a little bit:

> A Ruby implementation of the Open Assets Protocol. This library is able to perform Open Assets operations using APIs like Colorcore.


Поздравляю с первенством сдачи лабораторной на Rust. Вопросы:
* Поясните, почему доступ к _send_queue_ считается потокобезопасным?
* Генерированный код всё же лучше не коммитить (message.rs)


@php-coder yeah we should validate Destination


The nemea-supervisor.service was missing during RPM build on systems without systemd...
This solution appends the file name into the spec file when systemd is detected.
Commit message for this bugfix is not clear enough.


ok cool


I don't understand where the "Matrix::at" error was introduced.


Yes. Now, it is called when adding the creature to the room. I think this comment needs some update.


unintentionally, but surprisingly it still works :-) I will add that back


One question that could be answered is how to choose `'app_label'`. Does it need to be the same or different from the app in which the tests are located?


> why not store the current connection state in CPVRClient

Ooops. This one splipped just through. Yep, storing the state at CPVRClient is a good idea. Will change that.

I once tried to change m_bReadyToUse at runtime according to connection state. Currently it is only set once after the addon was successfully initialized after addon creation, never again during CPVRClient instance lifetime. My try ended a nightmare... ;-) Can't remember the details, but to change this is definitely a larger task - to be done in another PR - for sure.


yeah, otherwise you won't be able to truncate it... it has to be writeable by the user we run assemble with which can be whatever user.


Excelente mi joven padawan


also secrets are drop-in-drop-out files, so I don't think having more strict perms on them makes sense


I think the sentence
> Indetifiers preceded by `!` mean that the automaton does not have the corresponding properties."

can be misread depending on if you think of the "automaton" as a mathematical object for which the property is either true or false, or as some data structure representing the HOA file in which the property can be true, false, or missing.  In the latter case, I'm tempted to read "does not have the property" as "the property is missing".

I'd suggest to replace "do not have the property" by "have the opposite property" to avoid any confusion.


If that's the case, then it should be added to 'db/import-tmpl/item_db.txt' and commented out. The regular item_db should have the script entry empty.


Probably we could provide more validation? For example, we could filter out duplicated pairs of src:dst or print a warning when multiple src bind to the same dst.


`port_name` does not belong in this PR, it is very useful


"as an argument on the test method  when used..."?


Please see section 2.Users of the [edX API conventions document](https://openedx.atlassian.net/wiki/display/AC/edX+REST+API+Conventions).  In our most recent API working group meeting, we decided to require endpoints that have user-specific payloads to include a username in the URL (either as the resource or as a parameter).


@php-coder yeah we can, I think in that case (multiple src binds to same dst, the last one wins...) however I don't want to make it more complex ;-) I think docker does not validate that as well. Duplicated pairs should be deduplicated.


I suppose you mean these whitespaces?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10717](https://teamcity.mesosphere.io/viewLog.html?buildId=100281&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:04:10


Thanks! We can add this, but please ensure the code addition uses the same spacing and formatting as the existing code, and please document it in the `README`


May be if `src` doesn't exist we could print a warning and continue?


Me neither; it's your branch ;-)


WDYT about showing this information _before_ trying to upload it to container? In case of error we would see the name of the secret file and it could simplify debugging.


no, if one of the secrets does not exists, then it is probably not what you want :-) the build will fail or produce unexpected/broken image


```block()``` only returns Turfs, so you can write this as a faster, istypeless, loop.

```
for(var/t in block(locate(1,1,zlevel), locate(world.maxx, world.maxy, zlevel)))
    var/turf/T = t
    ...
```


And what is the pattern for computing their URLs? Don't answer that! I want to answer it for myself. :-)


Thanks. Sounds like I can remove the explicit array creation then.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10718](https://teamcity.mesosphere.io/viewLog.html?buildId=100282&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:04:00


It should work just as `unit = Unit.Kb` I believe.


markdown is slick


+ lambdawerk :)


I suggest to try to move this sizes to the `Unit` enum.


[Hanno Zulla]
> Please explain the rationale behind this.

The error reported by lintian without this change provide specifics, and
<URL: https://wiki.debian.org/binNMU > explain some background
information.  the >= + << dependency is the best idea we could come up
with on #debian-multimedia for the in general unsolvable problem of a
dependency from a arch:any to an arch:all package.

Lintian in unstable reported other issues too that needed to be
addressed before uploading.

--
Happy hacking
Petter Reinholdtsen


This overrides a method on django fields objects that is an instancemethod (See, for example: https://github.com/django/django/blob/master/django/db/models/fields/__init__.py#L1047).  We probably shouldn't change it to a classmethod here.


@K-Guan 感谢！感谢提供的相关内容。
你的学习方法很赞。

另外提一下，我只是不赞同他们的课本学习方法。

对于考试，它看的是结果，不用考虑是通过什么方法得到的结果。
所以，这个过程产生了什么意义，或者有什么副产物，那就不是我们要讨论的了。
只对事不对人，不能说高考考好了，这个人就怎么怎么样，只能说明课本学习的方法有效。其他的，什么也说明不了。

> 至少在写代码方面，我推荐首先将回答附带的解释全部看完，然后尽量想办法将代码看明白。实在有看不懂的地方也至少记录下来有时间再慢慢研究。
不断学习，不断改进。


chill chill ...i forgot :3


IDE에서는 username을 자동 제공했으나 principal 없이 실행시 오류남.


Ah, ok, so here is the place to answer. I'm not familiar yet with this inline stuff at Github ;-|

Chrissitopher, in the meantime I had answered in a PM at Slack to Michiel:
I don't know where to answer to the "weird" thing at Github:
https://github.com/TYPO3-Documentation/TYPO3CMS-Guide-FrontendLocalization/

I looks like that weird stuff was created by my OpenOffice to ReST converter of that time and nowbody ever since 2012 edited that manual. This could be a good moment to update my converter and run it again on the OpenOffice document. But I'm not sure whether to give that priority. To find out what is meant - pick the old OpenOffice document. Should be in the repo.

I would prefere that we just fix the text if that's possible.


[Hanno Zulla]
> Why?

Because it isn't needed.  The binary work fine without this specific
plugin package installed.  Why did you add it in the first place?

--
Happy hacking
Petter Reinholdtsen


looks ok to me


We definitely should improve the message. At least we should show the max file size because without this info user can't fix the error.


@legoboyvdlp, as far as I can tell, at this point in time, models generated by Aeromatic are **vastly inferior** to anything YASim ever did.

From not very realistic moment of inertia over limited range of valid lift values and lack of asymmetric stall through lack of ground effect to outright absurd values for engine thrust and I could probably go on as quite a few more interesting aerodynamic effects are simply ignored by the default aeromatic-generated models.

Note that:
 - Add moment of inertia, principal axes of inertia should be tilted forward (because vertical stabilizer extends upwards), so at least `<ixz>` should be non-zero.
 - Your model actually has lift from about -12° to about 34°, which is better then most JSBSim models we currently have, but the 13° stall AoA, from body axis (there is no angle of incidence defined to be added) seems too low as does CLmax 1.2. The sharp peak at stall is not very realistic either.
 - Lack of support for asymmetric stall means the aircraft does not tend to drop wing when stalled and of course it means the aircraft won't spin, which the real thing definitely will.
 - The thrust table is actually taken from real data, but they are data for a turbojet, Rolls Royce Olympus, and using them for a turbofan, even low bypass like the P&W JT8D, results in way too much thrust being available at cruise speeds and consequently absurdly high flight levels being possible.

All those things _can_ be done in a JSBSim model, but you'd either need real aerodynamic data, or you'd need to take some educated guesses and do a lot of math. That is what YASim does.

That said, I have to admit I am not really satisfied with YASim models either. Its flap model is only appropriate for simple flaps, not Fowler and double-slotted ones jet airliners have, as far as I can tell the slats simulation does not work at all (not that your model even tried) and there is absolutely no support for compression effects. The last point is where your model is clearly better, because it adds at least supersonic drag, though not supersonic loss of lift.

But so far, YASim is the best option we have. So please, don't replace the YASim model with this, at least not in this state.


CONTENT


what performance gains are we talking here?

Is it related to analysis of big applications only or to all (even small) generally?
I see that the referenced link says ids.block-size should be set to approximately to a number of vertices. Isn't that value 25000 a little small for big applications?


細かいことですが、ここで言う「開発担当者」はしょぼちむさんの会社の開発担当者ということでしょうか?


Please fix what is necessary for this change in `HelloWorldTest.cpp` from `OOVisualization`


Offensive


this should be a separeted PR stating the fix of #49 (IMO)


```zsh
emacs () {
	t=()
	if [ ${#@} -ne 0 ]
	then
		while IFS= read -r file
		do
			[ ! -f "$file" ] && t+=("$file")  && /usr/bin/touch "$file"
			file=$(echo $(cd $(dirname "$file") && pwd -P)/$(basename "$file"))
			$(/usr/bin/osascript <<-END
        if application "Emacs.app" is running then
          tell application id (id of application "Emacs.app") to open POSIX file "$file"
        else
          tell application ((path to applications folder as text) & "Emacs.app")
            activate
            open POSIX file "$file"
          end tell
        end if
END
      ) &
		done <<< "$(printf '%s\n' "$@")"
	else
		open -a "Emacs"
	fi
	if [ ! -z "$t" ]
	then
		$(/bin/sleep 5; for file in "${t[@]}"; do
      [ ! -s "$file" ] && /bin/rm "$file";
    done) &
	fi
}
```


This check is useless, as encoding an `str` is a no-op in Python 2.


List of gluten-containing ingredients


Файл слишком велик для загрузки (размер больше {value}{unit}) - так сработает?


This is also a great performance and pageload speed increase in all modern browsers, because they can use HTTP/2.0 or any other fast protocol behind SSL and behind the scenes load all fonts at once.


ERR("!read");


Pushed to upstream. Thanks!


```
Python 2.7.9 (default, Mar  1 2015, 12:57:24)
[GCC 4.9.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> 'é'.encode()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 0: ordinal not in range(128)
```


How is `https2http` supposed to work after this commit?


Same logic is duplicated above, extract a method?


@minrk Nice. I like codecov quite a bit for its flexibility on reporting on PRs and the detail it can provide.


Hopefully this will be solved tonight. More details on #57.

But again PR addressing multiple issues are rarely merged (IMO).
So keep it simple to increase the chances of your PR to be merge.


The 2 trailing spaces mean a new line in markdown and shouldn't be removed.


Do you feel strongly that having the `modelView` inlined here is important?


Ah! Very nice I didn't notice this one!
I had been testing them inside of the already .NET wrapped `<form>`. Didn't think to try double: `<ng-form> <ng-form>`

Mind putting in a PR so I can merge this?

Thanks Chris


http://docs.mitmproxy.org/en/stable/features/reverseproxy.html

> Here, http[s] signifies if the proxy should use TLS to connect to the server. mitmproxy always accepts both encrypted and unencrypted requests and transforms them to what the server expects.


You're welcome! Don't forget to ask me if you need someone to complain about your code organization. ;)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10719](https://teamcity.mesosphere.io/viewLog.html?buildId=100284&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:17


Ah, but we don't know how much, right? I've sent them a reminder.


the docstring now shows an example with a pattern with one capture group: `expand=True` returns `DataFrame` and `expand=False` returns `Series`.


Version pas encore finie


I think I missed your link when you sent it originally. Great writeup Milan! Thanks!


Thanks for the link, the docs I was reading were out of date.


Consider adding `vim Dockerfile`, just as a reminder that the Dockerfile needs to be updated.


remove it.


Hi and Happy New Year to everyone.

It’s sad that you are so disappointed with the work. Well, at least it’s moving somewhere (or nowhere at last) now.

I understand what you want to see changed and what you have already proposed. But I see some problems with invoking glretrace for each pass independently. Specifically, it could be problematic with some backends -- for example, with Nvidia PerfKit. It has a number of aggregated counters (called simplified experiments, most are dedicated to finding bottlenecks) which have to be collected  in several passes. PerfKit uses its own objects to keep the process state and has its own versions of BeginPass/EndPass functions. So the way to use these counters in PerfKit with Apitrace as it seems to me would be to have only one copy of PerfKit backend in the memory (some kind of shared memory).
Actually, I realized recently that PerfKit is even more problematic than that, since in initialization it binds API to the specified GL context (which of course changes every pass), so it would be even more difficult to implement these counters in PerfKit. Anyway, there could potentially be more backends with multi-pass complex counters that would be difficult to implement for the reasons described above.

Also, I’m ready to do the changes (I have a feeling you already assume I’m not interested or just out).
If you decide that multi-pass counters are a no-go for Apitrace, I can remove everything related to multi-pass in glretrace and make a simplified version. Afterwards I can work on retrace (series of glretrace calls; output merge).


@eregon I had to pull some code in from `DefineOrGetModuleNode` here. We probably should work out what's common and what's not. For the time being, I've only handled the "no inherit" case.

I also was looking at adding specs, but it's not clear if they should be in the `const_get` or the `autoload` specs. It's sort of an integration test. Thoughts?


use bootstrap variable instead of 768


@emanuil-tolev you found the same solution to env-ing the widget as I did - i.e. using the file suffixes and rendering different files. Unfortunately this doesn't work for the one generated by facetview on the search page - that is still fixed to load ```fixed_query.js``` rather than the other versions, no matter where it's generated from.

So, shall I replicate the env switcher in js?


Default methods should also be here.


Where did you get this source? I've been looking for it everywhere but couldn't find anything reliable.

Thanks!!!


I would concur with that, I don't know why the markup died but I had actually put a double space before each entry as well.

    directories:
      tvshow_dir: /path/to/tvshows
      incomplete_dir: /path/to/incomplete
      download_dir: /path/to/downloads
      completed_dir: /path/to/completed

is what it should have looked like... apparently code is indented by four spaces in markdown =)


I have made the compiler settings explicit, so one can use the either C++ '11' for xlc, or the old one.


I don't think this help makes it obvious that the name and the searchString are separate sections. Would it read better if it said " [--name=...]" or something like that? I think i'd like it if the detailed help also explicitly said that the name has to be an exact match


Does this Git use the proper config files? IIRC they changed how they do that in 2.4 or so, which is why we're still on 2.2.2.


>But so far, YASim is the best option we have. So please, don't replace the YASim model with this, at least not in this state.

Yes, I already corrected that.


@hnguyen94 I agree. I can add that to my PR, but I'm waiting to even hear back from the repo owner.


No, but changed a bit in 386871029eb512d46e88e676d2482efa0c5376aa .


ver.1.0.6 was skipped?


Didn't we decide to use release tags instead of SHAs?  The idea is expressed below, if this is meant more for people testing pre-release RWD code, perhaps it should be a little more explicit that you can use a commit SHA for that purpose.  Otherwise, this particular section might be superfluous.


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


I think we need a test of the help for find and findAndDelete in this class


Please keep consistent indentation with both code and HTML. It is difficult to visually inspect and debug when lines are oddly split or indented inconsistently.


Lol i literally created a whole new module because I wanted prompts :sweat_smile:

[`git-labelmaker`](https://github.com/himynameisdave/git-labelmaker)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10721](https://teamcity.mesosphere.io/viewLog.html?buildId=100286&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:04:15


Durch die Änderung pos+8 auf pos+9 fehlt bei einigen Beschreibungen das erste Zeichen.
Bsp. The Homesman
----
Western
ary Bee Cuddy .... statt Mary Bee Cuddy ...


Für mich unverständlich. Ist eigentlich Sinn  einer topliste, das man daran misst und deswegen auch mal mehr spielt.


Because of the normalization step in [UTS46](http://unicode.org/reports/tr46/#ProcessingStepNormalize).

```js
> 'hello\u0000'.length
< 6
> 'hello\u0000'.normalize('NFC').length
< 5
```

I ... hope that's what normalize(NFC) does, at least. I'm no Unicode expert.


Can you get rid of this?


This is from https://github.com/conda/conda-recipes/blob/master/statsmodels/meta.yaml

But this has a problem on 3.5 (mac build: https://travis-ci.org/JanSchulz/package-builder/builds/100366387#L5374):

```
The command "export CONDA_PY=35" exited with 0.
24.00s$ obvci_conda_build_dir ./recipes janschulz --channel main --build-condition "python >=3.5,<3.6"
Fetching package metadata: ......
Resolving distributions from 5 recipes...
Computed that there are 4 distributions from the 5 recipes:
Resolved dependencies, will be built in the following order:
	matplotlib-1.5.0+532.g3d942f7.dirty-py35_0 (will be built: False)
	pypandoc-1.1.2-py35_0 (will be built: False)
	statsmodels-0.6.1-py35_0 (will be built: True)
	ipyext-0.1.0-py35_0 (will be built: False)
Nothing to be done for matplotlib - it is already on main.
Nothing to be done for pypandoc - it is already on main.
Building  statsmodels-0.6.1-py35_0
Removing old build environment
Removing old work directory
BUILD START: statsmodels-0.6.1-py35_0
Fetching package metadata: ........
Solving package specifications: ...............
Error: Unsatisfiable package specifications.
Generating hint:
[      COMPLETE      ]|| 100%
Hint: the following packages conflict with each other:
  - python 3.5*
  - distribute
Use 'conda info python' etc. to see the dependencies for each package.
The command "obvci_conda_build_dir ./recipes janschulz --channel main --build-condition "python >=3.5,<3.6"" exited with 1.
0.00s$ export CONDA_PY=27
```

@ josef-pkt: is that dependency still needed on py3.5?


Put a TODO: Use `{}` instead of `=`


It should be `sudo vagrant up --provider=libvirt`


Hello there. I wrote a new plugin which is inspired by the same need that this plugin covers. Although it is more user-friedly and very easy to use in combination with template languages. You might want to have a look:
https://github.com/gillesgoetsch/acf-smart-button


>Поясните, почему доступ к `send_queue` считается потокобезопасным?

Проще всего объяснить не конструктивно: программа написана на Rust, в ней нет блоков `unsafe`, она компилируется, следовательно в ней нет гонок данных.

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

Проблем с данными, лежащими в очереди (`ByteBuf`), тоже быть не может, так как в каждый момент времени ссылка на эти данные есть ровно у одного потока. Сначала она есть только у потока, инициирующего броадкаст ([`proto.clone()`](https://github.com/eugenyk/hpcourse/commit/273519baaaaae338d4c49fcb64a87597c3e873e7#diff-256235857c353bc7b2cbd53aa39ae172R120)), а затем только у принимающего потока
([`proto`](https://github.com/eugenyk/hpcourse/commit/273519baaaaae338d4c49fcb64a87597c3e873e7#diff-256235857c353bc7b2cbd53aa39ae172R206)).  Система типов Rust гарантирует, что передача владения осуществленна честно.

Так как каждый поток владеет своими сообщениями, то их приходится копировать, что может быть не очень эффективно. Это можно вылечить, обернув `ByteBuf` в [атомарный счётчик ссылок](https://doc.rust-lang.org/std/sync/struct.Arc.html).

> Генерированный код всё же лучше не коммитить (message.rs)

Да, что-то я забыл, что к `Cargo.toml` можно приложить `build.rs` для кодогенерации :(


@josef-pkt: is that dependency on distribute still needed on py3.5?


Глупо использовать сцеление мелких строк, если можно все куски сразу запихнуть в одну.
Напоминалка из "Java. Библиотека профессионала" Хорстманна и Корнелла: "Ведь при каждом сцеплении символьных строк конструируется новый объект типа String, на что расходуется время и память".


Why do you use such a complicated form of that algo? The simplest form is (from 3dbrew)
NormalKey = (((KeyX ROL 2) XOR KeyY) + C) ROR 41

The C is 1FF9E9AAC5FE0408024591DC5D52768A.


I think not all of these are being used yet. Could you remove the un-used ones, and just keep the relevant ones for the current changes?


110568302-FixCrashInZoneComponentLoadSummary (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2033 of 2034 tests passed)

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


Done


Update: This decreased the pageload time from approx. 3 seconds to **1 second**!


Where are the tests?
I don't see them in the repo. :smiley:


In any case it is my opinion that replacing the previous work is not such great idea anyways. I think there are two highways (both ok and valid).
1) add a new (emphasis in add) FDM specification (aircraft-jsbsim.xml) and a new set file that uses that (aircraft-J-set.xml). This way both the previous Yasim and Jsbsim coexist peacefully, and can be toggle on launching time by the user.
2) fork the plane away, and create a new plane with a new name -- like Aircraft-Jsbsim or something similar, so that the new plane can be specifically the JSBsim version (look, per example @wlbragg  J3Cub)

But just, deleting the old Yasim version is not that polite, or good idea (as in work that destroys, not that constructs).

Just my two penkies.


It's a general purpose one, the TWL key generator could be plugged into it also.


I don't. But don't want it to be expressed in different ways between `UnmanagedModelProjection` and `ManagedModelProjection` even if it is not exactly the same.
I simplified the code, please comment on 2983e6baf6030255b216e73b24bcb9c4c4e44f0d


> Specifically, it could be problematic with some backends -- for example, with Nvidia PerfKit. It has a number of aggregated counters (called simplified experiments, most are dedicated to finding bottlenecks) which have to be collected in several passes. PerfKit uses its own objects to keep the process state and has its own versions of BeginPass/EndPass functions. So the way to use these counters in PerfKit with Apitrace as it seems to me would be to have only one copy of PerfKit backend in the memory (some kind of shared memory).

hmm... you mean the backend decides on its own which sub-set of enabled counters are used on each pass, and application is not at all involved in the process?

That would indeed make multiple process unpracticle...

Could you please point me to the relevant PerfKit docs?


Du hast die Variable 'isLink' nicht mit an den 'def getValues_startWorker(...' übergeben. Dadurch keine Ordneranzeige wenn Verzeichnis-Info auf 'Beschreibung' eingestellt ist.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10720](https://teamcity.mesosphere.io/viewLog.html?buildId=100285&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:10:26


not that it matters, but is this `fn slice` just dead code?


not that it matters, but is this `fn slicify` just dead code?


>Why gamenanj?!
TIL @gamemanj was naan bread.


Also, I find my rearrangement of the algorithm to be more readable.


this is wrong with the current folder structure
should be "../content_scripts/beastify.js"


Глупо использовать сцеление мелких строк, если можно всех их сразу запихнуть в одну. Тем более, они не являются постоянными полями каких-то классов / интерфейсов.


Heh, I've never seen error-pattern used this way. Cute.


I adjusted the patches to 2.6.4 recently, but didn't test my changes yet.


probably we don't want to be dumping graphviz here. We should really make this command line switches.


Why not add that check to the `| x -> x` case at the bottom?


110568302-FixCrashInZoneComponentLoadSummary (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2001 of 2002 tests passed)

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


Please change 276 like:
```
-     # robot.send {room: "#general"}, "社畜の通過... おっと休日だったか"
+     null
```


what is this test .. trying to test? oh, it's using -Z no-landing-pads...


Please don't define gratuitous copy constructors when the default would work just fine. See also rule-of-3 (or rule-of-5, rule-of-0).


Nit: please add a blank line after this closing brace


thanks


110568302-FixCrashInZoneComponentLoadSummary (JasonGlazer) - 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)


83811630_ReheatCoilSizingImprove (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1910 of 2034 tests passed)

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


Otherwise it's at a bizarre halfway point - the search configured widget is rendered from assets on the live site but using data from the correct env.


I think most of that is mitigated by making the ets owner process handle the writes. Two player processes spawn but one errors out because the other one won.


You are absolutely right. This was just my test house, and since I found some good aerodynamic data, it seemed a fairly good idea. I'll be trying the new Aeromatic.


Two things:
1) you have fields named idMatch (WelcomeMessages) and idRegexp (ReturnMessages). They should probably have the same name, and idRegexp is probably the better one.

2) we can take advantage of default field values to shorten the file, i.e., if I try to retrieve timeRegexp and get null, that means there is none, and we can declare a match -- same for the idRegexp


Makes sense :)


this definitely seems like a win


Hi Armen,
thanks for your observation.
Will look right now at it and take actions.
Regards
Mario


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10722](https://teamcity.mesosphere.io/viewLog.html?buildId=100287&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:03:50


White space before `null` is  important.
It needs 6 white spaces.


probably worth a comment here indicating what is going on


In the experiments I did with continuations, this was a macro actually but yeah, one of the two :)


Nit: comment on the commit itself is no longer accurate


Great catch!


> Could you please point me to the relevant PerfKit docs?
Can be found here:
http://developer.download.nvidia.com/tools/files/PerfKit_4_5_User_Guide.pdf
You might be interested in the "Experiment Mode" sections, as "Realtime Mode" is not problematic in the sense of multiple processes.


Is this test working?


Not sure I follow, sorry :(


:+1: Nice!


83811630_ReheatCoilSizingImprove (Myoldmopar) - 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)


Work is crazy busy but it isn't much work; I'll try to get it done before the EOW.


@thatoddmailbox Plox allow gitKracken access for this repo


@FGDATA, to be honest I think we should work toward _merging_ aircraft, not forking them. There is a lot of code duplication among the various variants of aircraft as they are and there are many bugs in them and fixing them one by one is just huge amount of work. We could achieve much better quality if we managed to reduce all that duplication. Obviously we shouldn't replace things unless we can generally agree it is an improvement and in this case we don't.

---

@legoboyvdlp, which aeromatic are you trying? I would like to do something towards generating better models (for A320; those FDMs are disastrous) and would like to look at it too.


this would be simpler to achieve with another call to executeScript


I'm talking about capture syntax: `&{:ok, %Redix.Error{message: &1}, &2}`. :smile:


@expenect Первое замечание - в коде не должно быть созданий внешних классов или вызовов внешних функций(аля dbQuery()). Нужно передавать это связи через конструктор или сеттер(лучше первое и присваивать в переменную)

2е В запросах ты юзаешь тестовые строки аля 'materialfield' такого тоже в коде быть не должно так как они там останутся на век и если что-то поменяется то нужно и код менять поэтому для этого есть MaterialField::ENTITY, Material::F_PRIMARY посмотри другие классы в проекте, например https://github.com/samsoncms/api/blob/master/src/query/Generic.php

Также для получения связи через конструктор используй интерфейсы - так как это не привязывает к конкретной реализации и делает твой класс универсальный и SOLID


Oh, right :) makes sense!


Mostly format changes, some of them worse than previous version.


There are just some more places.


Got it, thanks :)


Awesome! :+1:


@AdrieanKhisbe you are right, today I have only tests in we-core (80% coverage) and generator (simple tests)
I need more time to make some tests in we cli ( this repo ) and i will remove this npm script from here


:balloon:


The iRODS attribute-value representation does not have to be equal to the representation in the web interface. I.e. in the web interface I would like this to be a link (which means a URL starting with http://), whereas internally it can be a technical number such as collection ID (either persistent or not).

Would it not be (technically) possible to always be able to translate between the different representations? I.e. irodsID <==> handleID <==> URL and thereby get the representation of the link that is appropriate at the given moment where that representation is needed?


I don't remember.


Not a native speaker, but s/timerange/time range/? And similarly, s/TIMERANGE/TIME-RANGE/ in the key?

Thinking about the message, I think a better ending would actually be “…define a query time range”. It is more explicit about what is meant.


Does this additional implementation of adding really make a measurable difference on construction throughput?  If not, I'd prefer not to have the additional code.


@mitar The blaze-component update from 0.15.1 to 0.16.2 broke the following call: https://github.com/wekan/wekan/blob/701262a4394ea3fad1d64275930804db4f4ba182/client/components/lists/listBody.js#L28 as reported in #471.

Is that expected?


@vincentchabannes a short line to explain the expected behavior will be awesome ;-)


You might consider implementing this using "Poor Man's RCU" instead of a pure rw-lock.
https://github.com/pramalhe/ConcurrencyFreaks/blob/master/papers/poormanurcu-2015.pdf
with that approach at least rcu_read_lock() will be lock-free instead of blocking ;)


lucize -

The error message you are receiving indicates to me that you are running
EIGRP as non-root.  Are you?

donald

On Sun, Dec 27, 2015 at 9:53 PM, lucize <notifications@github.com> wrote:

> I tried a port to openwrt, my simple test was working so I tried in
> production and I got the Neighbor adjacency became full
> I saw your comment on the original branch.
> I've build it using the integrated spec file (adapted to centos 6), is
> running as quagga
> quagga 1283 0.0 0.0 99700 2032 ? Ss 04:09 0:00 eigrpd -d -A 127.0.0.1 -f
> /etc/quagga/eigrpd.conf
>
> eigrpd -f /etc/quagga/eigrpd.conf -u root -i /tmp/eigrpd.pid
> 2015/12/28 04:51:08 EIGRP: vty_serv_un: could chown socket, Operation not
> permitted
> 2015/12/28 04:51:08 EIGRP: EIGRPd 0.99.25-dev starting: vty@2609
>
> root 6598 0.0 0.0 99696 4636 pts/0 S+ 04:51 0:00 eigrpd -f
> /etc/quagga/eigrpd.conf -u root -i /tmp/eigrpd.pid
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/donaldsharp/quagga/commit/f186783a11421b47d81cd94c76cdb854159eb22b#commitcomment-15169439>
> .
>


This URL is taken. Should we use `equiphp` instead? Or perhaps `equipframework`?


> is that dependency on distribute still needed on py3.5?

I guess not, but @bashtage was handling all the conda setup.
I thought we have a PR for adding py 3.5 to travis testing, but don't find any right now.


what kind of brain seizure did I have when I was writing this code :) if defaultplayer is in the list, push it again, hehe


100% agreed, @jan-hudec


`equipframework`


WTF???:-\ :-\ :-\ :-\


+1 to `equipframework`


:+1: Good catch.  I would have forgotten this and it would have affected our friends over at O.C. :smile:


Should never reach here


this prevents the addon from working on file URLs

the easiest approach is to use `"<all_urls>"`
but it is probably more educational to add `"file://*/*" along side


Thanks!


Is the "*" here correct? I think that a find with no arguments should return all the assets in the repository (this is sticking to the same pattern as installUtility), and it looks like the code for the find command is doing that.


Rather than writing out all of these duplicates, which makes a reader like me have to strain to see that they are really all duplicate lines, how about a bit more code to make it clear what's going on:
```C#
int[] line = new int[] { 4, -23, -23, -2, 4, 6, 0, 4, 123, -4, 123, 4, -23, -23, -2, 4, ... };
int[] items = new int[line.Length * 10];
for (int i = 0; i < items.Length; i += line.Length)
    Array.Copy(line, 0, items, i, line.Length);
```
or something like that?


I fail to see how this is related to `ComponentContext::replaceHttpRequest()`!?  Can you enlighten me and/or adjust the commit message to explain why this change is required?


You mean just use `++_lastIndex`?


This probably needs a `debounce`, so we don't get 1000 setStates while the window is being resized.


I know, but I wasn't very worried about performance for this proc in specific (it only gets ran once for every Z level).


Changes for #5, Check Bypass string in associated WI.


No, I meant this whole else block, over just sizing appropriately and then calling Add or something similar.


Indentation.


Is this expect assertion required? Doesn't seem to be if we're counting the number of times the each loop runs.


Ok for me.

Hopefully Tristan will show up in a few days :)

Thanks a lot!


I mně to přijde neuvěřitelné ... ale je fakt, že lidi museli do tohoto stavu "dozrát" :smile:


That leads to a compile error in line 192 if defined(NDEBUG) :)


please remove this line!


#### Lack of background info?
* I don't understand why Fanboy is worried about third-party subscription. This subscription is NOT a EasyList family. In addition to that, Fanboy should NOT be a user, based on the [Support Policy](https://github.com/k2jp/abp-japanese-filters/wiki/Support_Policy).
* Reducing repeated and same Issue Reports is my priority. The change works fine so far. If users were unhappy with the change, they would start sending false positive reports repeatedly.
* Fixing all false positives is not my priority. That's why the subscription is limited for experienced users as the title of subscription says "ABP Japanese filters (ONLY FOR **Japanese** and **experienced** users: READ **Support Policy**)". The expected subscription users should manipulate white-list to fix a few false positives on their own.
* I simply share this subscription. New users seem to misunderstand the subscription as service. That's not my fault.


I'm thinking we should have a `guessedColumns` prop, given that `themes-list` is in use elsewhere, e.g. NUX.


What is the significance of the extra indentation on these three lines? Is that intentional?


Am I right in assuming that if I have this fixed (as in, with out pulling from git hub, this bug was already fixed), I have the latest version of the code, with reference to the email sent out to MLP-students on Tuesday, January 5th, and I do not have to worry about pulling again and running diff?


but this will send a mention, when the post is saved, not when it should be published in the future?!?


Condensing this whole method to just:
```
int root = (int)Math.Sqrt(value);
return value == root * root);
```
isn't sufficient?  This is "just" test code... does the perf difference really make a noticeable difference on test running time?  If we need to keep this complicated implementation, a comment explaining the reasoning behind the implementation and why it's necessary for performance would be good.


This method name did not make sense as it implied:

 n = this->ram[PC];
 n2 = this->ram[PC+1];
PC+=2;
 return (n2 << 8) | (n & 0xff);

When what is really happening is:
 n = this->ram[++PC];
 n2 = this->ram[++PC];
 return (n2 << 8) | (n & 0xff);

The method was renamed to "getWordWithPC()"


@xlab

I've just [stolen](https://github.com/cznic/c/commit/1850a6f2de90758c3daec6f7df8196fbe3eb084b) this bug fix, thanks ;-)

You now have write access to cznic/c, please add yourself to the AUTHORS and CONTRIBUTORS file. Thanks in advance.


@chenrozenes @asafdav The API will not change so you can use it right now. I'll push today but there are some things to do still. I will get around doing them this weekend.


No, you are completely fine!. That email (who was just accepted) was really
old and was waiting a confiromation. That's why Steve re-sent it with small
updates.

2016-01-05 16:02 GMT+00:00 Tony Knapp <notifications@github.com>:

> Am I right in assuming that if I have this fixed (as in, with out pulling
> from git hub, this bug was already fixed), I have the latest version of the
> code, with reference to the email sent out to MLP-students on Tuesday,
> January 5th, and I do not have to worry about pulling again and running
> diff?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CSTR-Edinburgh/mlpractical/commit/d8c7ae96a0e2c88b151d1bc51b94a1133d443b6b#commitcomment-15268194>
> .
>


Will this create a compiler warning or error? In general, we don't really want to clutter up the warnings for things we aren't changing right away since it makes it easy to lose warnings that we should actually fix.


Why not just:
```C#
var source = new HashSet<int>(Enumerable.Range(0, 1000).Where(i => IsSquare(i)));
```
?


simplecal


내용이 깨지네요 뭐가 문제일까요?


according to https://codex.wordpress.org/Plugin_API/Action_Reference/publish_future_post:
"publish_future_post is an action triggered when a post is transitioned from 'future' to 'publish' status"

so this is the hook that runs when a scheduled post go publish. In theory.


"Aims on" is not typical English grammar. Perhaps consider using "Targets" instead e.g. "Targets Zikula 1.4.2 or later."


Okey dokey. :)
Thanks @albertosouza for the fast answer.

(have a look to coveralls to have coverage badge ;)
eg: https://raw.githubusercontent.com/senecajs/seneca-pubsub/master/README.md)


Ah, good luck.  Yeah, after looking through the two PRs (yours included) for sourcemaps, among other open PRs and issues, I don't have high hopes of much being merged upstream at all.  We'll see, though.


Don't give away our secrets!!


The same set of changes needs to be done in `compileSync`. (Yes, `compile` and `compileSync` should be DRYed up. It’s on my TODO list.).


Baja Mali Knidza on GitHub :D


...you sure? This very `Monad` instance is what gets removed in 2.1 again... isn't it?


Rajko Kuzmanovic


Hi,
as you can see, the first run is made by services and is running as quagga, the second one is manually running as user root, I added more options in compile flags and I'm not getting the vty_serv_un anymore, but the service will not listen to any interface.
regards


Date filter been removed from the front end in 405b08, but the tests are still here because the widgets still exist in the application. Maybe we keep this for now as we might want to re-enable the data filter.


Lead filter been removed from the front end in 405b08, but the tests are still here because the widgets still exist in the application. Maybe we keep this for now as we might want to re-enable the lead filter.


Zde bych měl jen dotaz. Nebylo by lepší tento odstavec spíš formulovat jako odklon od Photoshopu? Adobe Comet ještě není hotový. Přijde mně, že je ještě předčasné o něm takto mluvit.

Třeba moje zkušenost je taková, že v roce 2014 jsem používal Photoshop na 100%. V roce 2015 už jen tak na 10% a zbytek času Sketch. Osobně bych zde raději viděl nějaký článek o vektorových nástrojích pro web, jako je Sketch. Dále by se mohlo zmínit, že boom zažívání nástroje jako je inVision apod. Opravdu užitečné aplikace, které je možné už nyní používat.


<?xml version="1.0" encoding="UTF-8" standalone="yes"?>

<items>
<items_info> <!-- this is opitonal -->
      <title>LISTAS DE JORGE </title>
      <genre>Tv Live</genre>
      <description>: Lista de Canales para el Livestreamspro :</description>
      <thumbnail>https://www.dropbox.com/s/ugs6zan0b7nx2o3/icon.png?dl=0</thumbnail>
      <fanart>https://www.dropbox.com/s/uc90b4dgfwogbux/fanart.jpg?dl=0</fanart>
      <date>04.01.2016</date>
      <cwhiteits>JORGE</cwhiteits>
    </items_info>

	<!--http://i.imgur.com/ oxTNPnp .png-->

<item>
<title>          [COLOR gray]Donaciones por [COLOR silver]PayPal:[/COLOR] [COLOR gray]adrian.bu@gmail.com[/COLOR][/COLOR]</title>
<link>http://adrian.kodistream.info/adryan/in.xml</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
</item>

<item>
<title>                           [COLOR skyblue][B]A D R Y A N [/COLOR][COLOR deepskyblue]L I S T[/B][/COLOR]</title>
<link>http://adrian.kodistream.info/adryan/liist.xml</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
</item>

<item>
<title>              [COLOR gray]   update: [COLOR white]02[/COLOR] ՠEnero ՠ2016[/COLOR]      </title>
<link>http://adrian.kodistream.info/adryan/liist.xml</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Deportes[/COLOR] [COLOR skyblue][Sports] [/COLOR]</title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/deportes_adryan_list.xml</externallink>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR skyblue]  [B]Peliculas[/B] [/COLOR]</title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/pyt_vod.xml</externallink>
</item>
<item>
<title>[COLOR lime]֛/COLOR][COLOR skyblue]  [B]Series de TV[/B] [/COLOR]</title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/tv.xml</externallink>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Espa񡠛/COLOR]</title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://i.imgur.com/O4Kz3Al.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/esp.xml</externallink>
</item>
<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Mexico[/COLOR] </title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/mx.xml</externallink>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Stalker   [/COLOR][COLOR skyblue][Nescafe][/COLOR] </title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/nescafe.xml</externallink>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  TotalPlay    [/COLOR] [COLOR darkwhite] [/COLOR]  </title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/totalp.xml</externallink>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  IPTV[B]+[/B]   [/COLOR] [COLOR skyblue][/COLOR]  </title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/iptv.m3u</externallink>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR coral]  [B]AloFlix [/B]  [/COLOR] [COLOR skyblue][Peliculas Audio Latino en HD y SD][/COLOR]  </title>
<link>http://ignora.me</link>
<externallink>http://dl.dropboxusercontent.com/s/ukw944z5k82jdk0/AloFlix.xml</externallink>
<fanart>http://fotonin.com/data_images/out/5/776482-cool-movie-wallpaper.jpg</fanart>
<thumbnail>http://i.imgur.com/iYCbuTY.jpg</thumbnail>
<info></info>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Entretenimiento[/COLOR]  [COLOR skyblue][/COLOR]  </title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/ent.xml</externallink>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Peliculas & [B]Series de TV [COLOR skyblue]24/7[/COLOR][/B][/COLOR] [COLOR green] [/COLOR]</title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg/fanart>
<externallink>http://adrian.kodistream.info/adryan/p247.xml</externallink>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Musica[/COLOR]</title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/musi.xml</externallink>
</item>
<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Paises[/COLOR]</title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/paises.xml</externallink>
</item>
<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Internacionales[/COLOR] </title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/inter.xml</externallink>
</item>

<item>
<title>[COLOR lime]֛/COLOR][COLOR deepskyblue]  Mas Listas[/COLOR]</title>
<link>http://ignorame</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
<externallink>http://adrian.kodistream.info/adryan/lists.xml</externallink>
</item>

<item>
<title>[COLOR gray]Si te Gusta la Lista Considera hacer una DONACION, para apoyarme y seguirla actualizando.[/COLOR]</title>
<link>http://adrian.kodistream.info/adryan/in.xml</link><thumbnail>http://i.imgur.com/6pvNVON.jpg</thumbnail><fanart>http://adrian.kodistream.info/fanart.jpg</fanart>
</item>
-->

</items>


We could document it as such, but the purpose is to have a string that can be displayed to users which would be what they type in SQL.


This was a serious issue with rvv initialization (```i``` and ```j``` switched, where i is the particle index and j is the sample number). This caused the "max 100-ish particles before segfault") symptom.


>Each check generates its own error message, doesn't it?

No :-) It should skip other checks if one of them fails. You could try it yourself.

>If i send text file 501 Kb long - i need first to know, that file format is wrong, right?

I'm thinking about how to fail fast. In case of the size we don't do any extra checks, but for determining image type we need to read part of the file from the disk. IMHO it's easier to just say that the file too big and let the user upload another one.


Would be good to have the Edge gestures Tweaks enabled, but i can take care of that later.


I’d slightly prefer to pass the `now` itself and access `.moment` in the parser. The less components know about internal structure of `JuttleMoment`, the better.


@xlab

Note: Another bug in the same [place](https://github.com/cznic/c/commit/66663b5a52fbd7226f194d8666a2d040dcdd1e45).


So, how about to replace it with `\<Plug>`? You still don't agree?


I thought carefully about that. And wrongly, as it turns out!


Extra copypasta `+` slipped into this line.


Use a for loop rather than memcpy.  Might clear up issues?


ah ok, nice!

but what about the actions on "new", "draft", ...?


True, changing it to load from machine name instead.


Is it worth locking down our Node version?

[`package.json` example](https://github.com/alphagov/notifications-admin/blob/templating-flow/package.json#L6)

[`travis.yml` example](https://github.com/alphagov/notifications-admin/blob/templating-flow/.travis.yml#L10)


Done.


You think about server overload, not about user's happiness )))


PR-URL: https://github.com/nodejs/node/pull/2676


Python allows us to do so.  Regardless, I can revert this change now since it's no longer needed (based on other review comments).


If the assertions are synchronous in this test, is asserting the assertionCounter at the end useful/ or telling us anything more? Doesn't seem so


Или нужно:
 {value}{unit.name}
Никак не могу найти, где об этом написано (


Main.cpp was reorganized. A custom compiler flag was added that determines which version of main should run. Just as there is a “-D STANDALONE” there needs to be a “-D YOUR_NAME” for the code to now compiler, otherwise an architecture error occurs with _main not being recognized.

Package would not build because of old inclusion of RCPP_Trace modules that no longer existed.

Current state of things working:

Package builds? 		Yes
ROC runs in C++? 		Unknown
ROC runs in R? 		No
RFP runs in C++?		No
RFP runs in R?		Yes
FONSE runs in C++?	Unknown
FONSE runs in R?		Unknown


Ok, thanks.
@AdrieanKhisbe in we-core i have Travis and Codeclimade services how helps a lot but i want to improve it before add the badge
Ex: https://codeclimate.com/github/wejs/we-core


Thanks Werner for pointing this out. It all makes sense to me with the exception of the last one. I'm a bit lost ...
But isn't it important what we call from FreeCAD wen creating our mesh ?!? Means what kind of volume we will create ?!?

There are only two possibilities to create volumes from python and by importing unv files AFAIK. Per Python for sure it is not possible to create a 6 face prism.

BTW: a 6 face prism would be a hexahedron or am I wrong?


this should be in a diff PR, IMO


leave


so far, in my experience, publish_post only kicks in when you're directly publishing a new post, but I might be mistaken here. The other hooks are for situations to _really_ ensure the hook kicks in, but then again, I might be over doing this :)


@expenect Переименуй в  hasImages()


fixed in e7ac6b46beff16e5529c63f11577f28344a71617 :smile:


Perfect, within the next few minutes it should work fine.


hello


Also, we should consider a transaction as isRBF if any parent transactions that are unconfirmed and RBF exist.


@expenect А разве у нас нету в таблице колонки Active? ее не нужно учитывать?


>You think about server overload, not about user's happiness )))

You're right. Could you create a ticket for improving validation process and describe current situation and a way to improve it?


Yes! merged :)


@expenect Какой-то слабый комментарий - тут же вся соль в этом, это дополнительное поле которое имеет тип галлерея


:)

but I don't think it is a good idea to send pings for drafts, pendings or private posts...


This should default to false, hipchat has no notion of resolved.


SCRATCH THAT THERES A BUG I NEED TO FIX


Portrait


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 10723](https://teamcity.mesosphere.io/viewLog.html?buildId=100288&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:04:00


@expenect тут тоже - что комментарии значит? нужно это описать так что бы было всем понятно как это работает


Present even when not used? :/


Why you ask me? I need to run and try it to answer. You also could do it. Also try `{unit.()}`


Why removed? D:


@expenect Нету описание класса, кто автор, зачем и что этот класс делает - тут можно описать как в общем работают галлереи в АПИ


OK, I think I'm starting to understand why you need a mutable `TypeDescriptor` and `TypeParameter` classes. That's because you don't have the model declaration of the type function in the model loader, right?

Because otherwise I think you could simply represent them like we do for classes by name or `.class` constant, along with a list of bound/captured type arguments.

The problem is that I see that you also need to parse type functions in type signatures (in `TypeParser`), and that pretty much implies not having a model declaration, unless we can give them (generated) names even if they're anonymous. Contrary to any other type that occurs in a signature, these type functions have no name and are not subtypes of any common type like `Callable` with function types.


@expenect Убери пустую строку


> This probably needs a debounce, so we don't get 1000 setStates while the window is being resized.

The `numColumns !== this.state.numColumns` check should limit `setStates` to discrete column num changes without needing any further throttling.


works too ;)


Fixing the following bugs in the poly analysis:
- Branching was done exactly with the opposite values as correct (example: ‘if x then … else …’ had the abstract value ‘x = 0’ in the then branch)
- After an assert or printf  (example assert(i == 5)) the abstract value was top instead of the correct value i = 5
- Implementation of the assert_inv function did return for ‘i + j == 11’ an abstract representation of ‘i - 2j + 1 = 0’
- Integer division was rounded incorrect (47/6 was 8 instead of the C-like Integer division 7)


Тут избыточная строка - isset  итак вернет bool зачем еще инлайн иф? просто верни иссет


> If we remove default value - we must to remove default unit too.

Why we must remove default value for the `unit`?


All those logs are written only when you activated the Meteor call (from the script).
It won't log all the time.. I think its fine just to log it like that. What do you think?


а тебе рома не говорил, чтобы ты не выкладывал ключ в паблик?


@k2jp I did not realize you had a support policy page in here, so I [now link directly to it](https://github.com/gorhill/uBlock/commit/b7106ab82da5fb1f9c866745e78e4c3dabdc80c7).


use interface_exists here instead of kernel version.


Is it related to #1934?


Well, this is certainly easier to test, which is great. Let's change those `/` routes to `root_path` and LGTM.


@expenect Сдесь ты возвещаешь массив - подразумеваешь что всегда его нужно вернуть - хоть пустой, так сократи код - объяви вверху ```$images = array();``` и в конце ```return $images;```


Hey @JasperBRiedel, this doesn't check for errors and actually breaks the build system for OSX.

I've looked into it but would rather have one simple, defined behaviour for all platforms (location of the executable used as the base directory, all OSs then branch into a resources subdirectory) rather than create a heap of special conditions on a per-platform basis. Thoughts?


Is it possible to write a test without the --noprompt? If possible, I think it would be good to have at least one fat test which runs through that logic.


Только сейчас обратил внимание. А почему бы не хранить в SessionID string cookie? Тогда код мог бы упроститься до session.m_cookie = request.combined_cookies();  request2.set_cookie(session.m_cookie);


И тут нет m_ префиксов )


The NPM package that I use for running the migrations expects this format.
It also sorts all the migration files and runs them chronologically by this date.


we should check if the member is different from the passed in parameter before doing the refetch.


I moved the summary of what's new with `extract` to `enhancements` (it is no longer `api_breaking`).


trying


I updated text.rst and added a warning about the new ``expand`` argument.


Ok, thanks ... good to know !


Should we change these versions as well?


I can't.. the NPM migration package uses 'migrations' dir hard coded


Not sure if related to this commit but in between the version that has this and the version that does not Paket now adds:

```diff
 </PropertyGroup>
   <ItemGroup>
+    <Paket.BuildAction Include="razormachine.readme.txt">
+      <Paket>True</Paket>
+    </Paket.BuildAction>
     <Content Include="razormachine.readme.txt">
       <Paket>True</Paket>
     </Content>
```

To one of my projects which causes my build to fail with:

> CodeGeneration.LowLevelClient.csproj(37,5): error MSB5016: The name "Paket.BuildAction" contains an invalid character ".".


sure. thx.


The idea is to brainwash you into doing it so much it becomes second nature, then you never need to worry!


(I should dig a bit more into coveralls :D)


I think `haskey(d, :markersize)` and `haskey(d, :markershape)` will always be true... did you mean to check the type here?


Will this fix the worldgen lag I have with MO?


I don't think you're supporting custom shapes yet, right?  This change is premature.


Its just when you activate the Meteor call. I would want a good log. and its suppose to be in development only (creating migration will happen in development)


Sounds good, I mean I think if there is going to be an issue we can make it rear it's head in testing so not sure it matters right now just something to keep in mind I think.

I see the advantgages of the ETS table though I think, because each player process is like your gateway into the game so the player channel could resolve the user_id to process quickly and concurrently and relay the command.

Also very cool about the restarting with init args I could not remember if that was persisted, kind of curious how it does that, but yeah since your state for now is just those args then yeah no issues.


No. Either the targetNamespace will become versionless, or it will forever state "6.0.0" (which is ugly yes) - or until 7.0 at least.

@mariofusco is testing if we can do the former.


During my testing, I discovered that this implementation returns an incorrect result when more than one string is provided in `cloakedPaths`: even if `$/foo/bar.baz` is considered cloaked by one of (`$/foo`, `$/bar`), it currently returns `false`.

I'm adding some unit tests to make sure we get this right.


yep that's a bug


I want to edit the Help widget.  However, everytime I edit the HelpDialog.html file no changes are reflected.  How can I update this widget?


Does this make it impossible to have lines without markers?


I don't know. World lag is normal for complex building.


should this method be "inline"


no not at all but 1.0.6 is finished and ready for release but were currently waiting for the thaumonomicon texts


Nice catch. Those extra conditionals look like a check to ensure a report will only run if there are any `resultConfig.testPairs` data to report on.  I think you're right to move those conditionals out -- it's unnecessarily complex and any data dependency issues should really be handled in the reporting flow anyway.  Thanks.


I guess it's to show more clearly which plugins will end up in the sourcemaps


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


For this second, I think your logging here is fine. Logging is configurable per class, and we can always silence too much logging. There's also an overload `log.warn(msg, throwable)` we might use, and we should also figure out what's possibly keeping the exception from appearing properly in the logs without your line. As you said, we should figure out what works best for helping the user decode what's wrong. It's possible we may put extra `scalaz.validation` in to make sure environment variables / paths / etc. are setup _before_ the google exception is thrown, or just leave it to the existing google code.

Beyond logging, instead of returning a generic credential on the error, throw the original exception. Speaking of `return`, the statement isn't generally used in scala. Lastly, I still don't know why we use our own http transport and json factory, but I'd prefer to keep them in sync with our other credential providers that aren't using the Google defaults… for some reason:

```scala
try {
  GoogleCredential.getApplicationDefault(httpTransport, jsonFactory).createScoped(GoogleScopes.Scopes.asJava)
} catch {
  case e: IOException =>
    log.warn("Failed to get application default credentials", e)
    throw e
}
```


arquillian-bom should be imported in hawkular-bus-parent rather than hawkular-bus-common


@streetrat pdump doesn't even work in 3.3.5


I think I'll revert this change, cause in my device, there is a delay ( few seconds ) until bootanimation started - and I have not this delay with "qcom,dsi-pref-prim-pan = <&dsi_innolux_720p_video>" .

What do you think ?


is OverWrite is deprecated - why not remove it?


instead of squashing all `strncpy_from_user()` errors to `-EFAULT`, I think we can instead do something like:
```c
err = strncpy_from_user(...);
if (err < 0) {
        DRM_ERROR("...");
        return err;
}
```

I think we should also be strict about validating the uuid format from userspace, with `sscanf` to ensure it conforms to this format: `"%08x-%04x-%04x-%04x-%012x"`.


Also note that there is `version.org.jboss.arquillian` property in https://github.com/jsanda/hawkular-commons/blob/ce116353c644c6f54db6e0e45becafde2dfb5662/pom.xml#L94


Thanks, fixed. (I think comments on specific commits disappear from the PR when I force-push a rewritten history. Commenting from the "Files changed" tab works better. GitHub :-1: )


thanks


Parser had trouble with the following SMARTS: "[N&X4&+,N&X3&+0]"


This should probably be a singleton object, instead multiple empty class instances:

```scala
final case object ApplicationDefaultMode extends GoogleCromwellAuthMode
```


This works for me in reference to  https://players.brightcove.net/videojs-overlay/lib/videojs-overlay.js

1.  On line 6 add this code:  var annoTime = 0;

2.  Replace these following codes:
       ==================================================
             // check if we've seeked backwards and rewind the earliest
            // showing overlay as a result
            if (lastTime > time) {
              earliest = 0;
              overlay = overlays[earliest];
            }
       ===================================================

     with this codes:
       ==================================================
           if(overlay){
            	annoTime=overlay['start'];
            }
            // check if we've seeked backwards and rewind the earliest
            // showing overlay as a result
            if (lastTime > time) {
            	if(annoTime > time){   // remove existing annotation when rewind
            		$(".vjs-overlay").hide();
            	}
            	earliest = 0;
            	overlay = overlays[earliest];
            }
       ==================================================

Thanks.....