Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/tests/misc/StringConverterTest.py`, lines 115 to 115.*

```
                                                                               v
        uut = StringConverter("\\  : \\  , hello: \\ world, \\\\ A \\\\ : B\\ ")
```


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


@hartmannaf It would easily crash if what you've implemented is in such a way. There is reason why some software contains a updater executable along with its main part. And just to launch a updater for the job should surely be easier than keeping the program in memory while updating it.
Let me explain a little more. If you've updated it, and exefs/romfs changes, but your app isn't again loaded, thus its symbol tables break - while assumed the system itself would not check something for security reason.
So reasonable methods:
1.Start a updater, so your apps are seperated and you can update the updater after updated in main.
2.Prepare a constant bootstrap for your app, so you can download it, copy your inside updater to a constant memory region, load the new program, and start the constant bootstrap. Just as what FIRM does, this is implemented in some hardware products and you can regard it as a software reboot.
Indeed.. I don't see a real reason except ones of research that you want In-App-Update.


You should unwrap like this:
```Swift
if let retweetDic = dictionary["retweeted_status"] as? NSDictionary, userDic = dic["user"] as? NSDictionary {
  let retweetUser = User(dictionary: userDic)
  dic = retweetDic
  // Another code
}
```
It's unsafe when casting by using `as!`.


I think it's odd to treat `0` as `undefined`.


@uwolfer
Thanks. We are using this implementation for our Gerrit access and it works fine. Is it ok for you when I create a pull request without new test cases? At the moment I haven't the time to implement them.

I changed the indentation to 2 spaces for your project. But it seems that the file ProjectApiRestClient.java has another format then all the other files. If you ignore the whitespaces in a diff then the change is just a few lines.

Shall I set the Indentation back to 4 spaces for this file?


TeamCity Habitat :: Cruise [Build 308](http://build8ua1:8081/viewLog.html?buildId=772&buildTypeId=Habitat_Cruise) outcome was **UNKNOWN**
Summary: Canceled (Exit code 1 (new)) Build time: 00:00:06


Create implementation/deploy script plus rollback


de* nouvelles façons ? (pas bien important comme correction)


And it was so hard to tell me what the problem was, right?


Продублированные Url в таблицах сайта и страниц имеет смысла?
Полагаю, так в таблице сайта хранится домен сайта, например, www.lenta.ru, записать в поле Name. В таблице страниц указывает на сайта и хранит как полные ссылки, например, www.lenta.ru/new/100500.
Желательно сохранить ссылки без https:// и http://


Was this the problem? @ninapetrop


ambiente de desenvolvimento


Done.

At some point, we should consider using sub-namespaces.  I've found that fighting Visual Studio is usually the wrong thing to do.  You'll be having this conversation over and over.  :-)

In fact, where I work we have a verification script that makes sure the namespace of a type in our source matches the folder structure of where it's located.  Anyway, if you decide to use sub-namespaces, we can make that change quickly and safely.


The addition of all of these constants is curious; how does this help with the list view? Is it truly necessary, or something that ended up in here that is related, but separate?


Newline has been added in next commit.


Minor:

You could combine those 2 objects into one.


le "donc" est inutile.


See line above.


Sorry, I forgot that, just when I saw this commit I remembered. Really.


TeamCity Habitat :: Cruise [Build 309](http://build8ua1:8081/viewLog.html?buildId=773&buildTypeId=Habitat_Cruise) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 104 Build time: 00:01:19
### Failed tests
```
Test collection for Habitat.Search.Tests.PagedSearchResultsTests (2): Habitat.Search.Tests.PagedSearchResultsTests.LastPageEdgeCase(page: 121, totalResults: 230, pagesToShow: 2, resultsOnPage: 25): <no details avaliable>

```


utilisée*


@afred: This isn't good in medium:

![screen shot 2015-12-01 at 10 29 39 am](https://cloud.githubusercontent.com/assets/730388/11505081/9b3d3106-9816-11e5-91b0-345a07a79c62.png)

I think it's safest just to keep it right aligned. Centering does work at the widest width, but I'd really wish we could avoid making special cases for full-screen: These will be hard to maintain over time, and having radically different layout for different widths will only cause us headaches.


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/tests/misc/StringConverterTest.py`, lines 86 to 86.*

```
                                                                               v
        uut = StringConverter("abc\\\\ , def\\ \\ \\ ,   \\\\ unstrip \\\\\\  ",
```


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/tests/misc/StringConverterTest.py`, lines 115 to 115.*

```
                                                                               v
        uut = StringConverter("\\  : \\  , hello: \\ world, \\\\ A \\\\ : B\\ ")
```


scopé*


Eclipse doesn't ignore the target folder when this line is missing. But anyway I can delete this line, it's works fine for git shell.


This works for #6


Hi doadin
Currently im not working on cm13 for ancora :(
But as i said here i loved new approach..
İn my opinion we can overload/shim our special "frameworks/native legacy overlay patch for camera" with using this method instead of patching/bloating frameworks/native
CM doesnt approve our legacy hacks..
So we can get official cm support with this method :+1:


Just use `!loadingMoreResult ` and remember adding a white space before `{`.


No, but that's a good idea –– I'll try making the changes this afternoon.


Fixed in https://pagure.io/mdapi/pull-request/26


Le début est mal dit.


Con python 3 también me da error, con 3.4.3 mas específicamente:

    Traceback (most recent call last):
       File "<stdin>", line 1, in <module>
       File "<stdin>", line 10, in get_human_readable
    ValueError: Invalid format specifier

De todas formas la sintaxis de esa función es compatible con Python 3 (con 3.4.3 al menos), lo que no es compatible son los print (y no se si algún otro detalle).

De momento me interesa mantener compatibilidad con Python 2 (2.7 al menos), es posible mantener la compatibilidad con 2 y 3, que es lo que quiero hacer, pero no lo tengo como prioridad ahora, si quieres ayudar con eso sería genial.

Saludos.


Yeah, I personally like sub-namespaces but didn't want it to be something that caused confusion for new users of the library.  Maybe it's not such a big deal so long as I have good documentation (which I still need to produce).


Menno heeft nu officieel meer gedaan dan Jurryt en Nout.


:thumbsup:


En vrai, c'est "JavaScript" mais bon.


Alright I'm in favor of that, more or less copied the signature from `SourceRange` as close as possible.


missing lua part I guess.


This fix is already included in release 2.4.2


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


Sure. Not sure how this got in here.


Remove eclipse files from repository


"Vu que la déclaration de votre variable n'est pas remontée au scope de la fonction, il existe un moment où votre variable n'existe pas. Ce moment, c'est la TDZ."


This is lua not C/C++ so is this correct code - {I'm not so hot on Lua as C :grinning: }


That's strange... But finally it works, happy to hear that.

For the two questions:
1. This is currently not supported by the AWS API. Really looking forward to that.
2. The code is uploaded as ZipFile. What do you mean exactly? Do you want to have it visible there as inline code? As far as I know this is not supported by the API.

Considering your remark on changing the timeout and memory size.. What exactly is not working and do you get any error messages?

This is my site.yml. It will first create the function and will update the memory size and timeout afterwards.

```yaml
  - name: ensure lambda hello world function is present
    tags: present
    local_action:
      module: lambda
      name: helloworld
      state: present
      src: /tmp/code.zip
      runtime: python2.7
      role_arn: arn:aws:iam::XXXXXXXXXXXX:role/lambda_basic_execution
      handler: bla.bla

  - name: change memory
    tags: memory
    local_action:
      module: lambda
      name: helloworld
      state: present
      src: /tmp/code.zip
      runtime: python2.7
      role_arn: arn:aws:iam::XXXXXXXXXXXX:role/lambda_basic_execution
      handler: bla.bla
      memory_size: 256
      timeout: 200

  - name: change timeout
    tags: timeout
    local_action:
      module: lambda
      name: helloworld
      state: present
      src: /tmp/code.zip
      runtime: python2.7
      role_arn: arn:aws:iam::XXXXXXXXXXXX:role/lambda_basic_execution
      handler: bla.bla
      memory_size: 256
      timeout: 123
```

This is my result:
```
TASK [ensure lambda hello world function is present] ***************************
changed: [localhost -> localhost]

TASK [change memory] ***********************************************************
changed: [localhost -> localhost]

TASK [change timeout] **********************************************************
changed: [localhost -> localhost]
```


enca 1.16.1-1 est dans lxqt-extra depuis le 30/09
openal 1.160-1 est dans kde5-extra depuis le 6/10

correction libxvmc  ---> xorg-libxvmc commit ce jour


The main problem was that eslint reported many violations on the current code base, so I though that there’s something missing. It’s not very usable for other developers when you run eslint and get lost in dozens of violations in the current codebase.


You should explain within GIT commit messages.


"Si au fil de votre code vous changez sa valeur, modifiez pour un `let` (1%)" pas besoin de la virgule.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 146](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=551&buildTypeId=XbimEssentials_XbimEssentials) is now running


Très bon post. Merci.


But could we do that in another commit? Like said I try to match the old signature as close as possible, and this would go into #1069. Are you okay with that? @sils1297


see
https://github.com/shogun-toolbox/shogun/pull/2932


Missed one in eZ/Bundle/EzPublishCoreBundle/Composer/ScriptHandler.php line 67 in dumpassethelp


Nope, a layer plan (gcodePlanner) contains a vector of ExtruderPlan's, which might need to reallocate.
L250 below

Perhaps I should do a extruder_plans.reserve(extruder_count) , but I suppose a standard vector reserves more than 3 element, in which case a reserve does nothing. (A reserve never makes the size of a vector smaller)
What do you think?


- [ ] Is this configuration depends on environments ?


atom example

https://gist.github.com/tankywoo/594c24097acddf24acd3


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 146](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=551&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:20


@lpshane Thanks men! Had the same issue and it's fixed now! The issue only happend on iOS 8.1 > and < iOS 8.4


Remember removing unnecessary `self` in your code. You use `self` everywhere :persevere:


@tlvince, please review


At this place this means it is to be defined for ALL platforms - is LUA_DEFAULT_DIR valid here for all of them?


Тогда в архитектуре, в примере, где описана таблица Sites, надо жестко указать, что вводиться URL, в поле Name?


python will raise UnboundLocalError here, in case if you pass empty string as argument in function.


correction libxlst  ---> libxslt commit ce jour


lgtm


@ninapetrop is this the part that you said in the comment?


Cool - all protocols it is then :)

Could really do with a backwards compatibility check in ci_build.sh (just copy the simplest one and leave off the http:// )?


Sprawdziłem i wygląda na porządnie zrobione.
Jeśli naprawdę nie masz co robić to pomyśl nad tym czy by Krystiana nie wyręczyć z tego jego wyszukiwania zaawansowanego. Albo sam spróbuj coś nowego zrobić. Ja dzisiaj Javy nie ruszam bo mam inną robotę.


commit ce jour


The new method

https://github.com/UnORoms/android_device_htc_msm8660-common/commit/c714655b89ff092671ba036c40fd07de77f2fb3e

On Tue, Dec 1, 2015 at 3:37 PM, Oğuzhan Yiğit <notifications@github.com>
wrote:

> Hi doadin
> Currently im not working on cm13 for ancora :(
> But as i said here i loved new approach..
> İn my opinion we can overload/shim our special "frameworks/native legacy
> overlay patch for camera" with using this method instead of
> patching/bloating frameworks/native
> CM doesnt approve our legacy hacks..
> So we can get official cm support with this method [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/UnORoms/android_device_htc_msm8660-common/commit/d26859c031a0f5a8b1a88c025fcc49e58caf1ae7#commitcomment-14711690>
> .
>


c'était pas du tout le bon mot en plus, merci :smile:


reack


There's already a `ValidEmail` function in the `user` package.


Ok. Podziałam z tym wyszukiwaniem jak będę miał chwilę. Powiedz mi jeszcze jakiego dodatku do projektowania okien używacie ?


correction openldad ---> openldap commit ce jour


Interesting. I think this could be implemented quite well using [StreamMapFilter](https://github.com/haxetink/tink_streams/blob/master/src/tink/streams/Stream.hx#L308) (for which I am open to better naming suggestions).

```haxe
var last = null,
      first = true;
new StreamMapFilter(source, function (v) {
  var ret = if (first) {
     first = false;
     None;
  }
  else Some(accumulator(last, v));
  last = v;
  return ret;
});
```

Also, concatenation is basically implemented here: https://github.com/haxetink/tink_streams/blob/master/src/tink/streams/Stream.hx#L43

The implementations are somewhat convoluted, but in many cases that does contribute a lot towards performance.


It's better if you set default value for these properties, such as `""` for String and `0` for Int. By this way, you needn't to unwrap them before using it later.


TeamCity ReOpenLDAP :: ps-stable [Build 96](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1320&buildTypeId=ReOpenLDAP_PsStable&guest=1) outcome was **FAILURE**
Summary: Tests passed Build time: 00:29:32
### Failed tests
```
all LDAP tests for mdb/KFA: test050-syncrepl-multimaster for mdb/KFA: <no details avaliable>

```


Inline styles bad practice


Do we have this convention written down anywhere? I've added notifications and didn't know we had a convention.


how else should we control the background image?


You probably should also check for the domain. Otherwise, this code would also run if a cookie with the name `trac_auth` is set on another website (e.g. a different Trac instance). I'm not sure though whether this might not happen as we don't have a permission for that site in the `manifest.json`. But either way, I'd rather not rely on that fact, if it even limits the cookie API.


Doh, looks like I put the wrong bug number in the commit summary, it should be 2033 like in the bugzilla link.


Wow, jsonnet is pretty neat. Surprised I hadn't heard of it until now.


Shouldn't it be alphabetically sorted?


got it


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


I think the Helvetica Neue Light is TOO light 'across the board.'

And what about moving the whole 'Smoke Signals' sentence to the very bottom?


TeamCity Habitat :: Cruise [Build 310](http://build8ua1:8081/viewLog.html?buildId=776&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 97 Build time: 00:01:18


Nit: Speaking on conventions, would it make sense to name this NotificationDatabaseDidRecreate?


@shantur new method(LD_SHIM) doesnt require "hex editing libs" ? Right?


TeamCity Habitat :: Cruise [Build 311](http://build8ua1:8081/viewLog.html?buildId=777&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 105 Build time: 00:01:30


@alcohol Works for me, thanks!


No hex editing

On Tue, Dec 1, 2015 at 3:48 PM, Oğuzhan Yiğit <notifications@github.com>
wrote:

> @shantur <https://github.com/shantur> new method(LD_SHIM) doesnt require
> "hex editing libs" ? Right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/UnORoms/android_device_htc_msm8660-common/commit/d26859c031a0f5a8b1a88c025fcc49e58caf1ae7#commitcomment-14712006>
> .
>


@spocke


reack


It is held in the instance variable to call the cancel within the reset method.


Update 12-1


You can still install the latest version of this package via [bower shorthand urls](http://bower.io/docs/api/#install):

```sh
bower install blueimp/JavaScript-Templates#2.5.6 --save
```

Or directly in your `bower.json`:

```json
{
  "dependencies": {
    "JavaScript-Templates": "blueimp/JavaScript-Templates#2.5.6"
  }
}
```

That's one of bower's advantages, it doesn't require any work from package authors.
As long as they publish their package somewhere, it's quite likely installable via bower.


This scripthas been fixed into a new one


Can the vector in GCodePlanner be a std::list? As then you do not have copying. (When this object is copied it's only a performance hit, so not a huge issue)

Reserving is always a good idea, even if the implementation doesn't do much with it. Don't depend on implementation specific behavior.


Hexo's version : https://github.com/hexojs/hexo-generator-feed/blob/master/atom.xml

hope this will be helpful

*by the way: sleep early, take care of yourself* :smile:


我他妈快猝死了！我想要享受我剩余不多的人生！


Jak ściągasz eclipsa to patrz dokładnie czy ma wbudowany WindowBuilder.
Albo masz link:
http://www.eclipse.org/downloads/packages/eclipse-ide-java-developers/mars1
I po prawej masz downloady


Lol


You're totally right; KenLM supports oov-feature.  I didn't have the latest
commit of moses the first time I tested it.  It may be helpful to add a
note about enabling this feature to the "Building a Language Model" section
on statmt.org so people searching for "Moses LM OOV" find it.  I see a few
old posts on the mailing list about people asking whether it's supported.
Thanks again!

Best,
Michael

On Mon, Nov 30, 2015 at 7:08 PM, Ulrich Germann <notifications@github.com>
wrote:

> Hi Michael,
>
> but it should, since it's derived from LanguageModel (check (virtual) void
> (?) LanguageModel::SetParameter() in moses/LM/Base.cpp) and doesn't
> override EvaluateInIsolation. Is the "looks like" based on code inspection
> or looking at feature vectors in the output? If the latter, it's probably a
> bug, not a feature. On the other hand, the commented-out special member
> variable is likely to be residue of just fiddling with the code. If you
> want to investigate, I'd suggest that you look at the output with feature
> values (e.g. from n-best lists) first. Somewhere in the logging output (I
> can't remember where, there's a record in which oder feature values are
> reported ("moses --show-weights -f moses.ini" should produce interpretable
> information as well).
>
> Cheers - Uli
>
>
> On Mon, Nov 30, 2015 at 11:28 PM, Michael Denkowski <
>
> notifications@github.com> wrote:
>
> > Thanks for the explanation! It looks like KenLM doesn't support
> > oov-feature=1, though it does track the number of OOVs in part of the
> > code. I may look at implementing this when I have some time.
> >
> > Best,
> > Michael
> >
> > On Mon, Nov 30, 2015 at 6:11 PM, Ulrich Germann <
> notifications@github.com>
> > wrote:
> >
> >
> > > Hi Michael,
> > >
> > > David Madl implemented most of this change, so he may {want|have} to
> > chime
> > > in on the details, especially if I'm misrepresenting things here. We
> are
> > > currently working towards reducing the dependency of Moses components
> on
> > > static data, in hopes that Moses components can eventually be
> > instantiated
> > > even outside of the Moses Monolith.
> > >
> > > In essence, this change only moves the specification "OOV feature
> > enabled"
> > > from the global space (via moses.ini) into the local space (via the
> > > configuration line (oov-feature=1) within moses.ini, which can be
> > supplied
> > > to the constructor even when StaticData is not available globally). So
> if
> > > you have any legacy moses.ini files that do have
> > >
> > > [lmodel-enable-oov]
> > > 1
> > >
> > > you'll have to rewrite them into
> > >
> > > WhateverLM ... oov-feature=1 ...
> > >
> > > The abstract LM class LanguageModel sets a boolean member
> > > m_enableOOVFeature during construction and sets the feature count to 2
> > > (instead of 1) if m_enableOOVFeature is true. The (virtual) default
> > > implementation of LanguageModel::EvaluateInIsolation(...) provides the
> > OOV
> > > feature and counts OOVs.
> > >
> > > LM sub-class implementations that override this virtual function must
> > take
> > > responsibility for this themselves. This is unchanged from the way
> things
> > > were before. According to 'find moses/LM -name '*.h' | xargs grep
> > > EvaluateInIsolation', none of the current implementations in the core
> > Moses
> > > code repository actually do override EvaluateInIsolation().
> > >
> > > LDHT (whatever that is, it seems to be related to RANDLM, which in my
> > > opinion is primarily of historical interest) will need some fixing up
> (it
> > > resets m_enableOOVFeature to false but doesn't update the
> > > m_numScoreComponents), but I'm inclined not to cross that bridge
> before I
> > > get there.
> > >
> > > Feature functions are now registered in the respective FF factory after
> > > they have been constructed (instead of
> > FeatureFunction::FeatureFunction()),
> > > because the number of feature values a FF provides is now unknown until
> > the
> > > FF has been constructed. That's a purely technical change (with some
> > > benefits for future modularization).
> > >
> > > So bottom line, except for the need to enable oov-feature=1
> individually
> > on
> > > each LM feature in moses.ini (instead of the [lmodel-enable-oov]
> > > specification), nothing should have changed from the user perspective.
> > >
> > > I hope I got it right and I hope this helps
> > >
> > > - Uli
> > >
> > >
> > > On Mon, Nov 30, 2015 at 9:25 PM, Michael Denkowski <
> > > notifications@github.com
> > > > wrote:
> > >
> > > > Does this mean there's a generally usable LM OOV feature or do
> > individual
> > > > LMs have to implement it? Does the KenLM interface support it?
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14693766
> > > >
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > Ulrich Germann
> > > Senior Researcher
> > > School of Informatics
> > > University of Edinburgh
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14695969
> > >
> > > .
> >
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14696321
> >
>
> > .
> >
>
>
>
> --
> Ulrich Germann
> Senior Researcher
> School of Informatics
> University of Edinburgh
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14696894>
> .
>


现在时间 2015-12-01 23:51:50秒


@dkayiwa  This is a massive commit.  I will add some comments on the ticket re: this, but at minimun, can we do a release of HFE directly before this commit, and then increment to HFE 3.0>?


This one is version 0.5.1, am I right? Didn't find the proper git tag.


Great !! :yum:


Do I assume correctly that we're using a PPA here because the keepalived version in precise is too old to be useful?


should we also update the matching model properties for `locked`, `available` and `refund_pending`?


Nit: Indent code inside block.


I think it should be `%r{[^\/]+}`


@berenm, I hope this makes some kind of sense? I wanted to replicate the checks for the warning compiler flags that I had added to configure.ac. So that it dynamically detects which are supported, and enabled/disables the ones we care or don't care about. Yeah, it's a bit hacky, I'm aware. :)

If I'm fundamentally misunderstanding something here, and it only works on my system by chance, please tell me.


Looks good.


I see, so this is used for the main image within the hero unit of the home page. It's a weird one. But by declaring this is here, it means that it'll be impossible to write any css to override it due to specificity. But if we don't ever need to do that it won't matter.


@misterfifths fwiw, Learn's new markdown parser does read the empty lines following a code-block, unlike Github-flavored markdown.


foodcritic change: node['os_version'] <= "6.1"


Mind also replacing underscores with camel case? Can be addressed in a seperate commit though. Frankly, these are already quite some changes for a single commit IMO.


TeamCity Habitat :: Cruise [Build 312](http://build8ua1:8081/viewLog.html?buildId=778&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 56 Build time: 00:01:11


Okay, Is it like this?

```java

private ObjectAnimator animator;

@Override public void start(@Nullable final IEndListener listener) {
    ...
    ObjectAnimator animator; // remove this line
    ...
}
```


@VisualKei, чому ты не пользуешь раскрытие по центру? Просто интересны мотивы.


learning curve.....lol


How about a for-of loop?


Hi, Thanks for the tip but it doesn't work, as blueimp-tml is a dependency of something else. I'll revisit next time something specifically needs a new version of this. Sticking with 2.5.5 for now.


@Xuanwo 多谢链接和关心 :smiley:  :+1:  这个版本只是临时推上来，写的比较粗糙。后续我可能会--force覆盖掉

atom.xml有些地方可能需要额外写jinja插件支持，比如日期转换那块，还在考虑在


@misterfifths fwiw, Apple's Swift docs seem to use the word "variable" only in reference to instances created with `var`, instances created with `let` are referred to as "constants", and the word "instance" is used to refer to those created with either `var` or `let`


I do not get this one. `Time.at` requires at least one argument


@sirmordred ok i was just wondering cause we could use this on older android versions as well right? That way we dont have to mess with CM code for any version.


sure

2015-12-01 16:39 GMT+01:00 Makman2 <notifications@github.com>:

> But could we do that in another commit? Like said I try to match the old
> signature as close as possible, and this would go into #1069
> <https://github.com/coala-analyzer/coala/issues/1069>. Are you okay with
> that? @sils1297 <https://github.com/sils1297>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/96f27622d8734a1140319ea8f9d963f82f4f24a3#commitcomment-14711772>
> .
>


It might be a good idea to stop the spinner after notifying the user that there are no interpretations to display ?

But that requires you to have a reference to the InterpretationsFragment.java, which you don't have.
So maybe instead modify the InterpretationFragment.updateList(list) to handle the case where the list is empty ? And stop the spinner from there locally.


Всё работает.


лучше if всегда в скобочки помещать, особенно такой большой


foodcritic change: code '#{cmd_text}'


It looks like you are removing ``funfactory`` imports and you are manually copy/pasting the definitions from ``funfactory`` settings to ``mozillians`` settings.


do we need the period? isnt the watermark enough?


- Compile errors https://github.com/sirjuddington/SLADE/pull/353
- Timidity process stop function doesn't work anymore
- Every entry type is OKT module http://www.zimagez.com/zimage/2015-12-0118-00-02.php
Slade is broken now!!


Лучше наверное file.getAbsolutePath()


了解，明天有一门专业考试，滚去睡啦，晚安~


Sorry, but no. There should not be any need to add new styles.

What changed in between master and your branch that changed the styles in the first place?


Я им ещё как пользуюсь, только избирательно по Ctrl-клику для больших картинок, чтобы зумить можно было. А так не вижу смысла раскрывать на весь экран маленькие пикчи или когда по быстрому в превью пикчу увеличить.


@Syphurith I implemented it this way, because I tested it with FBI and BigBlueMenu, I installed different versions of them and started them from within their old version. It worked without a problem, but after I couldn't implement it this way, I simply close my application after a successful update. Maybe later I will create a simple Updater application, which will use a txt or json file for the target cia url, so that other applications could use it too.


I think we should change the above from yes to no otherwise people may contact him, which doesn't seem to be the interest.


Is the `3,0` part a typo?


Looks good to me. I was thinking that this could be automated during the parsing of configure.ac, but if you're fine about duplicating the checks, I guess it's OK :)

`set_check_compiler_flag_c_cxx` could be defined as a `macro` instead of a `function`, so that it doesn't create a new variable scope and you can simply call the two other functions without having to call `set(... PARENT_SCOPE)` again.


I don't think the first line where it cats itself applies. I think you only
need the one line. The second one.
On Nov 30, 2015 08:42, "Peter Robinson" <notifications@github.com> wrote:

> It was a direct copy from a git hub help file on removing library files
> from being counted as part of the repository. It works which I'm happy
> about. What do you think it should be?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GarageGames/Torque2D/commit/5f2850497411cfb57791afee17c5eb01b96d0979#commitcomment-14681784>
> .
>


Да, и конечно для просмотра нескольких пикч подряд тоже часто юзаю.


Should it? I didn't think so. These are in the order that Qt wants them, IIRC; and this is the order the BUILD_REQUIRES has them below.


I'm sorry, I forgot the definition of the constructor : (

```java

  public Loop(ISurfaceAnimation... animations) {
    super(animations);
  }

```


По канвасу вопрос.
/**
   * Вспомогательный тип, описывающий квадрат.
   * @constructor
   * @param {number} x
   * @param {number} y
   * @param {number} side
   * @private
   */
  var Square = function(x, y, side) {
    this.x = x;
    this.y = y;
    this.side = side;
  };

это отрисовка квадрата. Получается мне подобное только прямоугольник?
Главный вопрос где в коде контур желтый прописан.

2015-12-01 16:53 GMT+02:00 Evgeniy Podgaetskiy <epodgaetskiy@gmail.com>:

> Отправил изменения.
>
> 2015-12-01 14:03 GMT+02:00 Kirill Brusenskiy <notifications@github.com>:
>
>> Будет не лишним, если ты переменную i объявишь сначала как объявлял
>> другие. Но в данном случае можно сделать и for (var i = 0; .... Только
>> не забудь, что цикла у тебя два.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/epodgaetskiy/55260-keksogram/commit/57e68cc81a4267e980723b76a56675c333515788#commitcomment-14706581>
>> .
>>
>
>


While on it, any reason you filter for `xmlhttprequest` here? Sure, this is the only context in which this API is called by Trac itself. But unless there is a good reason, I think it's a bad idea to introduce inconsistent behavior here. It also might get quite confusing if somebody opens the endpoint in a new tab to inspect/debug it.


здесь можно добавить свойство
             margin: 0 auto;
чтобы выровнять wrapper по центру страницы


I think this needs static.


I don't know how this changed to @tests in a process, must have been my IDE


You can freely choose the dependency name:

```json
{
  "dependencies": {
    "blueimp-tmpl": "blueimp/JavaScript-Templates#2.5.6"
  }
}
```


I can verify that sandbox efischer19.sandbox.edx.org does NOT exhibit the bad behavior we're trying to fix as of this commit.


@kayladnls Should we tag a new release, so that omnipay/common 2.4.0 can be installed?


I believe is covers java and cs tests too, because tests for them are transpiled further from C transpilation (see end of this file where it happens).


The declaration should be unnecessary, since the definition follows on the next line.


The fact that `input` had `height: auto` messed the searching fields in `Manage Account` tabs, that's why I removed it from `input` styles.
If you prefer I can go the other way around, put `height: auto` back in `input` and make a special class for the searching fields in `Manage Account` tabs.


Недошел до второй функции, а уже вопрос задаю... Отмена последнего😊

2015-12-01 18:03 GMT+02:00 Evgeniy Podgaetskiy <epodgaetskiy@gmail.com>:

> По канвасу вопрос.
> /**
>    * Вспомогательный тип, описывающий квадрат.
>    * @constructor
>    * @param {number} x
>    * @param {number} y
>    * @param {number} side
>    * @private
>    */
>   var Square = function(x, y, side) {
>     this.x = x;
>     this.y = y;
>     this.side = side;
>   };
>
> это отрисовка квадрата. Получается мне подобное только прямоугольник?
> Главный вопрос где в коде контур желтый прописан.
>
> 2015-12-01 16:53 GMT+02:00 Evgeniy Podgaetskiy <epodgaetskiy@gmail.com>:
>
>> Отправил изменения.
>>
>> 2015-12-01 14:03 GMT+02:00 Kirill Brusenskiy <notifications@github.com>:
>>
>>> Будет не лишним, если ты переменную i объявишь сначала как объявлял
>>> другие. Но в данном случае можно сделать и for (var i = 0; .... Только
>>> не забудь, что цикла у тебя два.
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/epodgaetskiy/55260-keksogram/commit/57e68cc81a4267e980723b76a56675c333515788#commitcomment-14706581>
>>> .
>>>
>>
>>
>


Кстати ещё момент - по центру зумит во вьюпорт целиком, а по месту - только по ширине. Иногда удобно так, иногда эдак. Потому для меня оба эти режима вполне востребованы.


When I develop the extension I almost always test against the production API. When I'm changing the API locally then I just change the settings manually and reset the settings in the extension (which are persisted).

Please remove this.


fixed the post.dt_create to let time localization like other feeds


I don't recall having that issue. I'll check now, just to be sure.


@VisualKei  Так у тебя работает с последним коммитом или нет?


What happens if `file_name` isn't a valid key?


Hello to variable arguments and dynamic templates


This requires `CLOCK_POWEROFF_ALARM` to be defined in `include/uapi/linux/time.h` kernel header.

You may download [this patch](https://www.androidfilehost.com/?fid=24269982087008643) and apply it in the kernel directory (e.g., `cd kernel/samsung/trlte`) as follows:

 Check the patch:
`git apply --check 0001-Define-CLOCK_POWEROFF_ALARM.patch`

If it works, apply it:
`
git am --signoff 0001-Define-CLOCK_POWEROFF_ALARM.patch
`

Otherwise, edit `include/uapi/linux/time.h` and manually insert the following line:

`
#define CLOCK_POWEROFF_ALARM            12
`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7464](https://teamcity.mesosphere.io/viewLog.html?buildId=83476&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout Build time: 01:00:09


Я работаю над фиксом, спокойно.


nice! Looking good Lily! @akalicki and I still need to add the anonymous_function flag to the sast


@bvw

Is this wise someone could potentially craft a request and obtain information we would not want to disclose:

```
proc /url_test.html {next_url} {
    set password secret
    return [url $next_url foo bar]
}
```

![image](https://cloud.githubusercontent.com/assets/3089689/11506144/c8baffee-9845-11e5-9310-ee669c346d83.png)


Oh! you are write. This does not work as I expected!

bool('False') == True of course !

It should be:

return True if val == 'True' else False

2015-12-01 15:36 GMT+01:00 kernc <notifications@github.com>:

> >>> bool('False')True
>
> Is this intended?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jamartinh/Orange3-Spark/commit/94324ed27cbb7c6d1bad76c79cf6b3f7bce547e2#commitcomment-14710042>
> .
>

--

--
/ .- .-.. .-.. / -.-- --- ..- / -. . . -.. / .. ... / .-.. --- ...- .
José Antonio Martín H. (PhD)           E-Mail: jamartinh@fdi.ucm.es
Computer Science Faculty               Phone: (+34) 91 3947650
Complutense University of Madrid       Fax: (+34) 91 3947527
C/ Prof. José García Santesmases,s/n   28040 Madrid, Spain
web: http://www.dacya.ucm.es/jam/
LinkedIn: http://www.linkedin.com/in/jamartinh (Let's connect)
.-.. --- ...- . / .. ... / .- .-.. .-.. / .-- . / -. . . -..


Readthe [RFC](http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html)


Нет, не работает же.


I use it because I want to make it clear with local variables. They have a different highlight color though :blush: I will avoid this.


Unless parts of this file are copied, you can drop the copyrights from other contributors.


The truth of a list is whether it has any elements, so `(len(args) == 0) == (not args)`.


closes #27


Please revert the height: auto on all inputs. The search fields in Manage Account should reuse the same styles of the extension -> meaning that the search icon should be removed.


Not sure that this fix is enough
see for example when we try to build libprelude with Python3.5 under Mageia
we are using swig 3.0.7 with this patch http://svnweb.mageia.org/packages/cauldron/swig/current/SOURCES/swig-3.0.7-python35.patch?revision=902918&view=markup


While adding a comment for that, you probably should also mention why we do that in the first place, i.e. that email addresses are truncated unless logged in with `TRAC_EMAIL` permssion.


This can be reduced to

    return '\n'.join(self.get_config_files())


merging my changes


Definitely, rolled back


No need for `else`


I think I'd switch those conditions for performance, python will evaluate them left to right in an `and` so you can rely in the second condition on the first one (useful if the first one is a type check)


[Using Data Access class.docx](https://github.com/AndyNewham/Taxis/files/48735/Using.Data.Access.class.docx)


Leading underscores are reserved for the system in the C language spec.  These variables should be renamed to something like shmem_internal_op_max...


ack otherwise, performance fix if you want


good idea, will change order


A bit late: You could this by AND'ing together the bitmaps, and then use the intersection of the index with the bitmap ```(1 << index) & bitmapAgg``` to determine if the attribute is ```nullable```.


>Revpunch range sux, set it to 110 or I break your elevators.
110 is op, 80 is useless

(110+80)/2 = 95 - the best value!


This macro seems unnecessary.  Can you just call the function directly?

Also, this name makes me a little nervous about a potential naming conflict.


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


My most recent local compose apparently had an invalid timestamp.  I don't know why; still investigating.

The daemon side looks correct to me.  The code is basically:
```
  timestamp = ostree_commit_get_timestamp (commit);
  if (timestamp > 0)
    g_variant_dict_insert (dict, "timestamp", "t", timestamp);
```

On the client side, I did notice the `gint64 t` variable should be unsigned.  But I just tested and that doesn't change anything.


@jtormey oops... should "SettingsAddressBookCtrl"


这里有一个问题，就是我们实际上无法这么简单的从文本中提取真实的单词（因为有标点符号，换行，和其他东西），有专门的分词 lib 来处理这种大量计算，匹配和自然语言处理相关的东西


The fact that you can't take it in a Firebase Support Cadre is completely
unrelated.

On Tue, Dec 1, 2015 at 7:24 AM, Collin Watts <notifications@github.com>
wrote:

> That line reads further saying "restrictions of choice". The Y'vahra is
> not a Riptide. I can't take one in a Firebase Support Cadre, so both the
> Y'vahra and the R'varna should have the cheaper one, yeah?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BSData/wh40k/commit/aa806b42d1b03fc5093860cf03459375e9f165f4#commitcomment-14711323>
> .
>

--
Eric Falsken
eric@everylittlething.net

"Expect everything, and anything seems nothing.
Expect nothing, and anything seems everything."
– Samuel Hazo


I'm using a data access class here to group all the SQL database statements in one place. They are then not in the presentation layer (no code in the HTML) and none in the main code behind page (.cs) supporting the webpage.  The data access calls to classes then can be accessed anywhere in the project.


Are you sure this is the right fix?

I've had that code which worked fine in Processing 2.x:

```Java
PShape shape = ...;
if (shape.getChildren() != null) {
    for (PShape child : shape.getChildren()) {
        // Do something with the child
    }
}
```

Now with Processing 3.0.1 I get:

> java.lang.NullPointerException
>   at processing.core.PShape.crop(PShape.java:1902)
>   at processing.core.PShape.getChildren(PShape.java:1909)
>   ...

I fixed it to:

```Java
PShape shape = ...;
for (int i = 0; i < shape.getChildCount(); i++) {
    // Do something with shape.getChild(i)
}
```

It works now, but this has an extra crop() for each iteration which I though could be avoided by calling getChildren() just once.


Not sure if you're going for sphinx docs one day, but this is a poorly formatted docstring regardless :)


mmmmm.. changing to g_strlcpy


Yeah, that's what #1356 has


Not a fix: the bug is still present when I test the website in staging and locally. Running locally, it's throwing an AttributeError and failing to perform the appropriate check (i.e. it's passing through the exception below). Examining this bit of Django's code, it's unclear to me that this code would ever NOT be an AttributeError. I may be misreading this, but the middleware seems to be processed before resolver_match is added to the request object: https://github.com/django/django/blob/master/django/core/handlers/base.py#L134


Perhaps.  I'm following the instructions found [here](https://github.com/github/linguist#troubleshooting).  Even if I don't need the line I won't change it unless it's causing trouble.  We have bigger fish to fry.


pazzesco


why change var? Just call allies everywhere :palm_tree:


@shantur  yeah...ok thanx man..

@doadin  absouletly right bro :+1:  we just need to hex edit strings in proprietary libs(such as libcamera or sensors.GT-I8150.so) to load libbinder2(lib that we will create in our device tree) instead of libbinder..

with that way followings patches wont be required...
-https://github.com/AriesVE-DevCon-TEAM/android_frameworks_native/commit/c3cda2753863c8ad04bf194cfc1c4dfea39475ff
-https://github.com/AriesVE-DevCon-TEAM/android_frameworks_native/commit/60795e5056d3c353729e02741bbf7d059c20d246


Same here. Since we are still using ``funfactory`` why are we moving this inside our codebase?


ja a010239


Same here.


Poorly formatted docstring.


Done. I changed the .gitattributes file directly (as it's unlikely to cause debate) and send a pull request for the .gitignores (which is likely to cause debate ;))


reack


@ericdill, @tacaswell , this is ready for review,


If a user passes a `step` of `0` they should `step` zero.


> I was thinking that this could be automated during the parsing of configure.ac

Well, it's wrapped in the if check, and then I didn't know how to best do that without making a mess out of everything. :P

True, it would be cleaner if it were parsed. If you want to implement that, please, that would be great. But I'm fine with that amount of duplication as a compromise.

> `set_check_compiler_flag_c_cxx` could be defined as a `macro` instead of a `function`

Oh, good idea :)


Hey @jreback is this description of behavior before and after OK?


@christhekeele would you like to create a PR for this?


I will do a 1.0.9 release sometime this week, thanks!


s/will spawned/will be spawned/


OK


Added here:
http://www.statmt.org/moses/?n=FactoredTraining.BuildingLanguageModel. Let
me know if you can thing of a better place to put it. I'm still having a
hard time finding my way around the Moses documentation.

- Uli

On Tue, Dec 1, 2015 at 3:52 PM, Michael Denkowski <notifications@github.com>
wrote:

> You're totally right; KenLM supports oov-feature. I didn't have the latest
> commit of moses the first time I tested it. It may be helpful to add a
> note about enabling this feature to the "Building a Language Model" section
> on statmt.org so people searching for "Moses LM OOV" find it. I see a few
> old posts on the mailing list about people asking whether it's supported.
> Thanks again!
>
> Best,
> Michael
>
> On Mon, Nov 30, 2015 at 7:08 PM, Ulrich Germann <notifications@github.com>
>
> wrote:
>
> > Hi Michael,
> >
> > but it should, since it's derived from LanguageModel (check (virtual)
> void
> > (?) LanguageModel::SetParameter() in moses/LM/Base.cpp) and doesn't
> > override EvaluateInIsolation. Is the "looks like" based on code
> inspection
> > or looking at feature vectors in the output? If the latter, it's
> probably a
> > bug, not a feature. On the other hand, the commented-out special member
> > variable is likely to be residue of just fiddling with the code. If you
> > want to investigate, I'd suggest that you look at the output with feature
> > values (e.g. from n-best lists) first. Somewhere in the logging output (I
> > can't remember where, there's a record in which oder feature values are
> > reported ("moses --show-weights -f moses.ini" should produce
> interpretable
> > information as well).
> >
> > Cheers - Uli
> >
> >
> > On Mon, Nov 30, 2015 at 11:28 PM, Michael Denkowski <
> >
> > notifications@github.com> wrote:
> >
> > > Thanks for the explanation! It looks like KenLM doesn't support
> > > oov-feature=1, though it does track the number of OOVs in part of the
> > > code. I may look at implementing this when I have some time.
> > >
> > > Best,
> > > Michael
> > >
> > > On Mon, Nov 30, 2015 at 6:11 PM, Ulrich Germann <
> > notifications@github.com>
> > > wrote:
> > >
> > >
> > > > Hi Michael,
> > > >
> > > > David Madl implemented most of this change, so he may {want|have} to
> > > chime
> > > > in on the details, especially if I'm misrepresenting things here. We
> > are
> > > > currently working towards reducing the dependency of Moses components
> > on
> > > > static data, in hopes that Moses components can eventually be
> > > instantiated
> > > > even outside of the Moses Monolith.
> > > >
> > > > In essence, this change only moves the specification "OOV feature
> > > enabled"
> > > > from the global space (via moses.ini) into the local space (via the
> > > > configuration line (oov-feature=1) within moses.ini, which can be
> > > supplied
> > > > to the constructor even when StaticData is not available globally).
> So
> > if
> > > > you have any legacy moses.ini files that do have
> > > >
> > > > [lmodel-enable-oov]
> > > > 1
> > > >
> > > > you'll have to rewrite them into
> > > >
> > > > WhateverLM ... oov-feature=1 ...
> > > >
> > > > The abstract LM class LanguageModel sets a boolean member
> > > > m_enableOOVFeature during construction and sets the feature count to
> 2
> > > > (instead of 1) if m_enableOOVFeature is true. The (virtual) default
> > > > implementation of LanguageModel::EvaluateInIsolation(...) provides
> the
> > > OOV
> > > > feature and counts OOVs.
> > > >
> > > > LM sub-class implementations that override this virtual function must
> > > take
> > > > responsibility for this themselves. This is unchanged from the way
> > things
> > > > were before. According to 'find moses/LM -name '*.h' | xargs grep
> > > > EvaluateInIsolation', none of the current implementations in the core
> > > Moses
> > > > code repository actually do override EvaluateInIsolation().
> > > >
> > > > LDHT (whatever that is, it seems to be related to RANDLM, which in my
> > > > opinion is primarily of historical interest) will need some fixing up
> > (it
> > > > resets m_enableOOVFeature to false but doesn't update the
> > > > m_numScoreComponents), but I'm inclined not to cross that bridge
> > before I
> > > > get there.
> > > >
> > > > Feature functions are now registered in the respective FF factory
> after
> > > > they have been constructed (instead of
> > > FeatureFunction::FeatureFunction()),
> > > > because the number of feature values a FF provides is now unknown
> until
> > > the
> > > > FF has been constructed. That's a purely technical change (with some
> > > > benefits for future modularization).
> > > >
> > > > So bottom line, except for the need to enable oov-feature=1
> > individually
> > > on
> > > > each LM feature in moses.ini (instead of the [lmodel-enable-oov]
> > > > specification), nothing should have changed from the user
> perspective.
> > > >
> > > > I hope I got it right and I hope this helps
> > > >
> > > > - Uli
> > > >
> > > >
> > > > On Mon, Nov 30, 2015 at 9:25 PM, Michael Denkowski <
> > > > notifications@github.com
> > > > > wrote:
> > > >
> > > > > Does this mean there's a generally usable LM OOV feature or do
> > > individual
> > > > > LMs have to implement it? Does the KenLM interface support it?
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14693766
> > > > >
> > > > > .
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Ulrich Germann
> > > > Senior Researcher
> > > > School of Informatics
> > > > University of Edinburgh
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14695969
> > > >
> > > > .
> > >
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14696321
> > >
> >
> > > .
> > >
> >
> >
> >
> > --
> > Ulrich Germann
> > Senior Researcher
> > School of Informatics
> > University of Edinburgh
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14696894
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14712112>
> .
>

--
Ulrich Germann
Senior Researcher
School of Informatics
University of Edinburgh


The docstring is used by errbot to provide help to users.


I can't really say from looking at a glance, but I think random access is quite a nice thing to have.

I will reserve then.


Wow-thought I did! Good catch.


Added some tests although I have converted everything to add the protocol and added a test to test without as I think it would be better to have the protocol in general


C Programming Language


Here the default should be 1, not 0, for consistency.


你的代码旁边那个+号是咋来的？


should probably be `ceil`?

This needs to be documented as an API change.


oh good find.  reverse dfa searches always use longest (just fyi, should be very easy to handle)


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


Sure, I will add it


Tested by https://github.com/w3c/csswg-test/pull/965


Может загрузить DH сразу методом SSL_dh_get_param_from_file()?


So right now, I can't seem to get it to work. However, it is not even starting the daemon. It fails way earlier in
`company-sourcekit` `    (interactive (company-begin-backend 'company-sourcekit))`

There, company-mode returns `or: Cannot complete at point`. I'm trying it out in a very simple file
```
let s = ""

s.
```
And it still doesn't work. It seems what happens is that it is calling `company-manual-begin` in company.el and there it fails somewhere. Interestingly, it never even calling `company-sourcekit--daemon-for`. I'm still looking into why this is happening on my system.


Could you please add a comment why we can just do this?


I'm also using a connection manager to handle open/close of SQL connections in one place. Saves on code. Also uses a central connection string which can be modified with a text editor in one config file. This allows ease of moving between computers with different SQL Server instances and it allows moving code from test to live or live to test.

[Using a Connection manager Class to open a connection to handle data.docx](https://github.com/AndyNewham/Taxis/files/48756/Using.a.Connection.manager.Class.to.open.a.connection.to.handle.data.docx)


you're right; this needs to be a `process_view` and not a `process_request`; docs here:

https://docs.djangoproject.com/en/1.8/ref/request-response/#django.http.HttpRequest.resolver_match


Is this not merged into the master branch yet? Seems very useful


I already wondered before, any reason you add a space at beginning/end of an inline object? It's probably matter of taste, but isn't that inconsistent with the way you use inline arrays?


See 2cf0d40775492390a9aa43ccb77f50d973626c09


@imuchnik this commit snuck into the PR I had previously opened


I set it actually!


Ah I see, we're converting on the daemon side.  LGTM then.


I think I would prefer to see this conditional moved down a layer to inside of shmem_internal_op_to_all.  Logically, it's an attribute of the transport layer, which we would like to keep below the language bindings.  That should also have the benefit of collapsing the several instances of this conditional into one.


much nicer. thanks.


Cool stuff


Duplicate of #1527


Is "Passworte" really correct? Duden says "Passwörter": http://www.duden.de/rechtschreibung/Passwort.
IMHO: "Passwörter standardmäßig im Klartext anzeigen"


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


Нужна консультация по заданию. Сегодня буду 3 вебинар смотреть и делать.

2015-12-01 18:06 GMT+02:00 Evgeniy Podgaetskiy <epodgaetskiy@gmail.com>:

> Недошел до второй функции, а уже вопрос задаю... Отмена последнего😊
>
> 2015-12-01 18:03 GMT+02:00 Evgeniy Podgaetskiy <epodgaetskiy@gmail.com>:
>
>> По канвасу вопрос.
>> /**
>>    * Вспомогательный тип, описывающий квадрат.
>>    * @constructor
>>    * @param {number} x
>>    * @param {number} y
>>    * @param {number} side
>>    * @private
>>    */
>>   var Square = function(x, y, side) {
>>     this.x = x;
>>     this.y = y;
>>     this.side = side;
>>   };
>>
>> это отрисовка квадрата. Получается мне подобное только прямоугольник?
>> Главный вопрос где в коде контур желтый прописан.
>>
>> 2015-12-01 16:53 GMT+02:00 Evgeniy Podgaetskiy <epodgaetskiy@gmail.com>:
>>
>>> Отправил изменения.
>>>
>>> 2015-12-01 14:03 GMT+02:00 Kirill Brusenskiy <notifications@github.com>:
>>>
>>>> Будет не лишним, если ты переменную i объявишь сначала как объявлял
>>>> другие. Но в данном случае можно сделать и for (var i = 0; .... Только
>>>> не забудь, что цикла у тебя два.
>>>>
>>>> —
>>>> Reply to this email directly or view it on GitHub
>>>> <https://github.com/epodgaetskiy/55260-keksogram/commit/57e68cc81a4267e980723b76a56675c333515788#commitcomment-14706581>
>>>> .
>>>>
>>>
>>>
>>
>


re2 also has prefix information, so I should be able to get rid of the analysis at the beginning that does it.


@justacan Sounds like your README is quite empty, is the service testable in some way ?


Hi,
Do you plan to make a release with this commit? Or there is something else that needs to be done before that?


It would be better to memset to SHMEM_SYNC_VALUE, which is also 0.  I don't foresee changing this value, but that's more defensive code.


Extra space.


Shouldn't this `/**< ` thing be used when on same line?


@olavphibm, well, I don't have time to show that serving the kickstart through the http server works, and @clebergnu is on vacation. I'm going to merge this.


`StreamMapFilter` about sums it up. You do invert your folds from the usual implementations. You use  `T->R->R` rather than `R->A->R`. I think there is a reason, but I'm not sure what it is off-hand, or if it applies here.
It looks very nice. I'm looking forward to throwing away some code.


Coding convention. Please don't use upper case for constant.


should this be in a finally { } ? Or is it taken care of by onError() if any of the above throws an exception? Not clear to me, because onError() gets called from the catch clause of the previous block.


similar here, can the responder throw an exception?


Or would your rather release 2.4.0 with the `~2.4.0` constraint?


@VisualKei теперь точно исправлено. Заодно посмотри #833 может тоже заодно исправил.

Теперь сдвиг справа от края поста будет такой же как и слева. Выглядит симметрично и красиво.


turns out, we don't need the marathon module (which James wrote and is using) ... Apollo added the dcos_cli role ... logic is in tasks/apps.yml


Reset to SHMEM_SYNC_VALUE instead of 0?


now there is some issue with distance calculation in renderworker, so only background is drawn.
CalculateDistance returns high value for distance ~> 200?


maybe a line or two of documentation here?


Ditto.


@tasomaniac - AppState was [deprecated in November 2014](http://android-developers.blogspot.com/2014/11/keeping-your-saved-games-in-cloud.html), replaced with [Saved Games](https://developers.google.com/games/services/common/concepts/savedgames). As stated in the blog post, AppState was scheduled to go read only in Q2 2015, well before the [Google Play services 8.1 release](http://android-developers.blogspot.com/2015/09/google-play-services-81-get-ready-for.html) where it was removed.


thanks for the comment, very helpful here :)


Oh... brilliant. A conflation of helps.


add documentation


Shouldn't here have the `/**<` thing?


add some doc


@DavidGoldwasser do we really need these files committed in the repo?


Would this ever be false?


Does this appear correctly on generated docs?


:ok:


Why not just leave out documentation on deprecated approaches? This leads to another good question. Do we want to introduce any sort of deprecation message for such things to hint at users that they will eventually be broken why a change?


Maybe talking about what this returns...


(deleting this thread because I think the comments are easier to follow if they are on the PR diff, not the commit diff)


Glad to hear that :)


Correct


Dont commit migration file without content.
Migration file is no longer edit after you have commited.


(Actually, I had left them out on purpose to distinguish the Maybes from the Firm Yes folks :)


Now that this is a queue for each thread: the queue can have more than one contextPair, because the same thread my process another request, while the previous context is still used by a body consumer from a previous request. In that case we create another context pair for this thread. That means, we potentially have many more contexts than threads, correct? Is that intentional? Or should we close a context in the cancel() above, if the queue is not empty? In other word, should we attempt to minimize the size of the queue for each thread?


We might consider omitting documentation of deprecated approaches as well. Is that legit?


Operator bool is not good enough to use it directly. Until we can use `explicit operator bool()` from c++11, we should use the "safe bool idiom": https://en.wikibooks.org/wiki/More_C%2B%2B_Idioms/Safe_bool . But on my opinion this is an overhead and we can use simply some kind of `is_valid()`.


Fantastic!


is this change relevant to the clearing of the stack? it doesn't look like it


I never really quite noticed, since all my tests are with `Int->Int->Int` anyway :D

And yes, I should probably change it. This is a left fold, which in Scala at least is `R->T->R` while right fold is `T->R->R`. The return value is passed at the "side of the fold" it would seem and it kind of makes sense.


_Waay_ too groovy for me, I actually had to look it up because I didn't know it existed! Anyway yes I changed it while on my rampage to make everything the same styles, I think it's clearer to have a solid colour border and was also scared by the description: "Defines a 3D grooved border" :scream:


I thought about that

```
fatchSetClearFunction([
  ['setInterval', 'clearInterval', true],
  ...
])
```
But I thought it was just adding lots of bytes to the payload without adding much in value.

Do you feel strongly about it?


is it possible that the context pair queue is already garbage collected at this time? I would hope that because in this case, the loading cache uses weak keys (not values), it does not have the same problem as in https://issues.cask.co/browse/CDAP-4338


are there any advantages to this over the original patch?

I'm concerned that this implementation is:
-  a bit more fragile since we are patching the internals rather than the public api
- won't work for mocha


fix indent


// Only check computed styles of element nodes.


It never did get merged or commented on by the project...


indent


@robertbaker Yes I am, but I need to rethink this or discuss. The issue with the changes I've made, other than them being breaking changes, is that the ResolvePageModel call is now async. This means that if you have that call as a response to a button press or such, you may be able to hit it twice. That is, it's called again before the first has resolved.

Yes, you can account for this in your code. The question though is whether it makes sense for ResolvePageModel to be async in the first place. The alternative being instead of cascading the async change to Init, the call to Init is wrapped in a Task.Run. That's not great either really, as then the page might not be ready by the time it's loaded. I'm not too opposed to this alternative as it just puts us back where we started with async void Init, with the addition of being able to test and catch exceptions in Init.


We were aware of that but were not expecting complete removal of the API. The migration process is kinda painful :( And during that period, we cannot update our gms libraries to 8.3. Even though gms has granular dependencies, it is mandatory to use the same version for all.

The only solution seems to be that we will stick around at GMS 7.8 for a long time for most of our users to update the app and migrate to new Saved Games API.


can you put the comment right before this line so it's closer to the logic it's describing. (I think JS allows comments anywhere, but if not that's fine.)


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.601.80e14a1ec208f6634548f7649f21405fb242b039](http://build.palaso.org/viewLog.html?buildId=66694&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:10:07


Looks ok (but I know nothing about this code).


For the time period that we don't have special announcement, can we default the message to be something like this? Please enter all your personal and administrative leave hours as Out-of-Office.


If it works (I haven't tested it yet), it's really useful. Could you please merge into master asap? I'm trying to capture the report as html so that its better received by the non-techies as to the impact of the security checks.


Также созданы тесты для удаления групп и контактов.


а зачем тут buf - len?


I LOVE the cheesecake. A coworker just brought it into the office. Have to try this on my own. Thanks @ffffux for providing the recipe. :heart:


Isolated cases


The line immediately above this refers to running "maven test". Can you reword this to refer to running "gradle check"?


OK, great. I'll check it out after the merge. Thanks


> Do you feel strongly about it?

I don't like those lines, feels a little hacky to me - but I could live with them.


Did you just copy in the src? I'm thinking about updating to 1.72


вместо айдишника bord можно использовать псевдокласс :last-child
http://htmlbook.ru/css/last-child
т.е.
.topic:last-child {
    border: none;
}


@thatoddmailbox Why did you delete my new image. (We actually own the rights to it and can use it without attributions)


Hi lordhoto,

indeed, it's weird that you received emails, because this fork is just a work in progress, not something which supposed to be merged yet :)

Anyway, thank you very much for the comments, I’ll do my best to update the project!

Here are some response to your questions:
* I’ll move the README.md file,
* I’m aware of the problem with the current Xcode project inside of the list directory. Like I said in the README.MD file, the « create_project » is not finished yet. I made all these changes only in order to be able to work on the ScumVM project itself, first. Now, I’m having some progress on the Xcode backend of the create_project tool, but this is still not finished. Once done, I’ll remove the Xcode project, and all the generated files.
* Regarding the Info.plist file, I’ll modify the « Info.plist.in » file instead, once the create_project tool has been updates.
* The rotation has not been removed; it has been fixes in order to be able to work with the latest iOS version. The rotation handling has changed with iOS 7.

In general, about the commits, I’m not sure if I can easily rewrite the history of this repository in order to match your requirements. I’ll follow your commit rules for the next commits, but for the existing ones, I assume that it is possible to make pull requests which are only parts of these commits, isn’t it?


Убери выше `return;`, а `console.log` перенеси в блок `else`.


Why .classpath file added?


Ah you're right! And that's exactly what I was seeing, I had tried to use mp4 video and aac audio, however I'd mistakenly left in the audio codec for mp4. It is working now.

Do you know if it's possible to use h264 encoded video in a SourceBuffer without boxing it into mp4 first? I haven't been able to find a working MIME type yet.


Have we been putting `GET`s in `code`?
https://github.com/cfpb/grasshopper/blob/master/docs/geocoder_public_api_spec.md


No need to define interface methods as public, by default it is public. Isn't it?


Well... If you agree to pull some `SimpleMessageGroup` actions to its interface, we can go ahead with the fix for this your concern.
I mean that in many cases we are restricted with the `SimpleMessageGroup` just because its operations aren't the part of the `MessageGroup` contract.


add some doc


Keepalived is poorly documented.  I had trouble making vrrp_script work, and reports around the web suggested upgrading it past 1.2.7 (precise is at 1.2.2).  That alone didn't solve the particular issue I was having, but I left the PPA in after everything worked, not only because rolling back might bring the issue back, but in the interest of preventing other bugs.


Pass `&:empty?` as an argument to `reject` instead of a block.


This is an old commit -- it ended up as 20m in master: e177f102a8fbbffa1753b2206c1c0363bfc2438e

Periodic GC is run regardless of the GCPeriod option being present, from what I understand: https://github.com/rht/go-ipfs/blob/48a33ffb673fb3eaf6d20cd7ca6bed04426a2abb/cmd/ipfs/daemon.go#L492-L507

We need periodic GC because for now only `ipfs cat` maybe-runs GC.


final, AFSamplersTest -> AFSamplersUnitTest


Some not obvious comparison for me...


ADSGLKADHSG


You probably should not use bare yum commands. See https://github.com/distributed-system-analysis/pbench/issues/112 for details: the plan is to rename the check_installed_rpm function in the base file and eventually add support for other package managers.


INTEGRATION TESTING_2015-12-01 18:53:53


INTEGRATION TESTING_2015-12-01 18:53:56


nice!


It's a minor issue, but I don't think this should be the definition of dual numbers. Here's a use of (complex) dual numbers that is unrelated to differentiation: http://www.euclideanspace.com/maths/algebra/realNormedAlgebra/other/dualComplex/index.htm


This particular sidenav was the first ng-material widget I implemented and I therefore used the same markup as they do in the documentation demo here http://material.angularjs.org/latest/demo/sidenav. That is to say, my sidenav is a sibling of the md-content that scrolls with the tooltips in it. There is a codepen on that demo page and you can drop a md-tooltip into the "Toggle Right" button and you have my basic setup.

________________________________
From: Ed Pelc <notifications@github.com>
Sent: Monday, November 30, 2015 4:57 PM
To: angular/material
Cc: Kate Rachwal
Subject: Re: [material] fix(tooltip): corrected md-tooltip positioning when scrolled (f62f693)

@kasiarachwal<https://github.com/kasiarachwal> can you provide a plunkr for your setup?

The .toUpperCase() fix didn't work. I found out why and I need to make a pr to fix it for both our usecases but I'd like to be able t confirm it works with your setup.

As for the why it doesn't work. With/without the toUpperCase() to && current.nodeName !== 'MD-CONTENT' essentially always skips md-content and uses a higher(body or html tag?) tag for the root to attatch the tooltip too. This is what should happen with your specific case(I'm gussesing whenever a sidenav is open) but when you do not have a sidenav open it should use the md-content as the root like it did before.

-
Reply to this email directly or view it on GitHub<https://github.com/angular/material/commit/f62f6934c02f4c994f3e9a9157cc0de351dd8c79#commitcomment-14694446>.


So, the first official release is "1.1"?

Going forward, do you want to maintain different versions of the documentation? For example, have a 1.x version of the doc that reflects the 1.x version of RackHD. Probably overkill at this point, but thought I'd ask anyway.


Okay. I'd suggest for that PPA to be dropped, though, once Open edX moves to 14.04, which is on 1.2.7 (trusty) and 1.2.13 (trusty-backports).


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


That would be a matter of a browser's MSE implementation supporting raw h264.  To the best of my knowledge, no browser supports this.  Most MSE browsers support ISO BMFF only, some also support WebM, and rarely you will find some that only have WebM support.


Ok, so here's a pin-pointed reason, even though the changelog description is too jargony to understand the symptoms.

2012-08-29 Release 1.2.7

* vrrp: fix issue in while using vrrp_script.
  Previous patch introduced by Ryan O'Hara about removing
  shadow declaration was kind of too much hunting.
  Removing element e in this block simply create inconsitency
  in upper list walk. So resurected element declaration with e2.


Isn't the position a single integer value? Why are we using list here?


Well, original myUPB was created when most free servers haven't SQL database, that changed over time, I thought about making "database connector" which should be able to select one of the available database connectors (I thought about every connector in single file: sqlite, mysql etc. so it would be easy to select it on installation, not sure how about changing it later, but this can be added later). So as I see it I wan to support sqlite AND mysql from begging (I find mysql with phpmyadmin easier to debug without additional tools)


Also fine by me


I.e., I think we should leave the PPA in.


I'd like to move the "Smoke Signals" sentence to the bottom, because then it's a natural lead in to the other new sentence we're about to add, which is about how this site's code is open source & we welcome participation etc.  Meanwhile, the current closing paragraph obviously should be closely associated with the form fields, so it makes sense for it to come right after them.

Regarding Helvetica Neue Light: yeah.  We'll do some screenshots of other typefaces and run them by you.


need to fix up the modifiers for the fields in this class.  check order (private static final) as well


Const ref


Why giving null inside constructor?


If response code is bigger or equal then 400 - it means that there was an error


please add a `const` version of `GetCurrentStats`, don't remove const qualifiers


Well it's a frustrating feature because it *sometimes* works, leading you to look elsewhere for troubleshooting when it doesn't, after being led to believe it behaves as advertised, or even in a generally useful manner.

The problem, IIRC, is that it doesn't share transitive dependencies, but with Maven, the "depth" of a dependency isn't all that important, so this aspect of a pom is subject to change from version to version of a Maven module. And, of course, most complex Maven modules don't re-state all transitive dependencies.

There's no nice way to put it. This is an anti-feature.


The API internally in the system is version 1.1, so I made the docs match. We're talking about an API 2.0 that would be backwards incompatible (although it could run alongside 1.1 for a transition period if we do this right), so that's where I was thinking about updating these docs to 2.0. Most of the content in the "users guide" space would change significantly with a 2.0 API set


ack


If we're going to use the convenience syntax for `.Some(foo)`, then we might as well use `nil` over `.None`.


Yeah, agreed, for as long as edX is on precise (see earlier comment).


OwnCloud actually have a requirement for mysql/postgresql


JavaVersionImpl ?


We need to handle `case (.Insert, .Some, _)` because of a bug on iOS 8.0; it's invalid state it happens. See [here](https://github.com/bignerdranch/CoreDataStack/pull/55#discussion_r46189388).


Update for new ```rbd nbd``` command


this closes #1


should be an int.


The default value for GCPeriod is set [here](https://github.com/rht/go-ipfs/blob/48a33ffb673fb3eaf6d20cd7ca6bed04426a2abb/core/corerepo/gc.go#L132).

> We need periodic GC because for now only ipfs cat maybe-runs GC.
The latter because surveying the entire repo for each `core.Resolve` is too expensive.


As a good practice, we are keeping all public methods up and private methods down in a class. Shall we follow that here as well?


@tessi ohai! I only realised through you that I still got this repo. And thanks very much for the fantastic feedback! :smiley:


Keep the filter on volt tables commented.
Other changes look good.


For what it's worth, I also solved this problem on my local copy with a slightly different approach. Sharing here in case it's useful:

```js
function initElements() {
    var self = this;

    this.elements.forEach(function (element, index) {
        if (!this.options.disableEditing && !element.getAttribute('data-disable-editing')) {
            element.setAttribute('contentEditable', true);
            element.setAttribute('spellcheck', this.options.spellcheck);
        }
        element.setAttribute('data-medium-editor-element', true);
        element.setAttribute('role', 'textbox');
        element.setAttribute('aria-multiline', true);
        element.setAttribute('medium-editor-index', index);

        if (element.hasAttribute('medium-editor-textarea-id')) {
            this.subscribe('editableInput', function () {
              var textarea = element.parentNode.querySelector('textarea[medium-editor-textarea-id="' + element.getAttribute('medium-editor-textarea-id') + '"]');
                if (textarea) {
                    textarea.value = self.serialize()[element.id].value;
                }
            });
        }
    }, this);
}
```

Tested on IE 9, 10, Edge


@ChrEggenberger why did you push to `master` instead of `develop` ??


:+1:
I'll add few. The "Undefined width + Undefined Height" will be a bit tricky, since `measureWithRatio2` as it is now should return `{Infinity, Infinity}`. (And actually would return `{NaN, NaN}` now.)


It's superfluous having two versions for every Getter (mutable and immutable).

Reducing code when unnecessary seems like a good reason.


upstream already has the concept of string sets. we should reuse. i think I last saw it in `pkg/util`


Почему в less? Он же `normalize.css`


Done.


> If it works (I haven't tested it yet), it's really useful

It works well! I'm not personally using it anymore, but the project I set it up on still has it running on each build, and the output captured by jenkins as an artifact.

> Could you please merge into master asap?

Not up to me - I'm not on the project, I'm just a guy that sent a PR :stuck_out_tongue:

> I'm trying to capture the report as html so that its better received by the non-techies as to the impact of the security checks.

That's exactly what I used it for too :)


ここまで入らないので削除しました。次のcommitで


поясни почему логотип решил сделать картинкой


I've created a new issue on the main repo. Please follow up there. https://github.com/rid00z/FreshMvvm/issues/24


`redux-pouch/middleware/POUCHDB`


сюда бы добавил текст или символ хотя бы...


ack


I was tempted to change that when I added @baseigneurie  as a contributor, but I didn't have the heart to change it.


oh that's nice, thanks for the pointer!


the logic here doesn't seem complicated enough to necessitate this static initialization block (the use of which is idiosyncratic here, i think).


Good idea.


можем сразу завалидировать знак &: заменить на `&`


@nywillb SEE SLACK PLOX


unack, you mean `end is no SourcePosition or None` not start,right?


Aha, knew there was something missing. Easy fix.


What about "copy-and-swap" in all assignment ops?


it actually is correctly "benefitted"


You mean until you port Tiny Storage? ;)


Ohh, so many imports from org.wso2.developerstudio.eclipse.gmf.esb package, can't we use .* here?


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


A note on why building a pull request with secure parameters can be dangerous (possible leak, isn't it?) can be useful.
Maybe add it to securityParametersReport.jsp?


in the file "select_vxls.R" in the function "pixelize" line 54 it seems that we need to eliminate "#" in order to have " tcs=D@data$tcs".


Why was this moved?


Что-то опять нихрена, блджад. Видимо снова сраные некропроблемы, посмотри пожалуйста в 28, если это тебя ещё не совсем заебало.

Обнаружил, что в консоли после клика появляется `Warning: Error in parsing value for 'height'.  Declaration dropped. Source Code: undefinedpx` и `Warning: Error in parsing value for 'width'.  Declaration dropped. Source Code: NaNpx`. Оно и раньше было, но всё масштабировалось. Возможно и проблема #833 оттуда же, потому что в упоминаемой там старой версии, на которой проблемы не было, это не выводится. Раньше я варнинги не смотрел, потому не сообщал.


If ```device_name.empty()```, you should return a friendly error message to avoid:

```
# rbd nbd unmap
rbd-nbd: failed to open device:
```


Promoting use of `const` is very good coding practice, and really not open to debate.

Seeing `const` instantly tells you that an object's state is not going to be changed, which is more valuable than reducing code size by one line.


ack


I wonder if we should call this something else, e.g. `AbstractShellHelper` or `BaseShellHelper` in case someone's application code has a view helped called `ShellHelper` already?


again, why is this so large?


I know you’d prefer to be version agnostic, however the Segment SDK has [no more subspecs since v3](https://github.com/segmentio/analytics-ios/blob/3.0.0/Analytics.podspec), and so depending on `Analytics/Segmentio` fails.


Does `this.mapView` exist in this context?


We don't; I just noticed that most of the notifications in the tree — including the ones right above and below this — were prefixed, not suffixed.


Не совсем понятно использование универсального селектора, если у нас подключен нормалайз. [Можно почитать это](http://www.xiper.net/collect/html-and-css-tricks/css-tricks/reset-margin-padding.html)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.599.80e14a1ec208f6634548f7649f21405fb242b039](http://build.palaso.org/viewLog.html?buildId=66695&buildTypeId=bt322) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 2159, ignored: 79) Build time: 00:38:38


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 295](http://ci.infinispan.org/viewLog.html?buildId=32801&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for 38a15091e2f4b0b3be774b478e2e92e628573d84
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:02:26

Process exited with code 2


"To recreate" means to play :smile: :rugby_football:


I think LD_PRELOAD, unusually, takes a space separate list


This file is actually not used (I was trying to extract things from vis but could not get things to work properly :(), it should not have been committed, I'll :fire:


I was thinking exactly the same thing, also on the templates. I was already looking around, but no straightforward EJS library for java. Using JSP would be rather easy though.


final


А графику не выдали разве ещё?


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*The issue occurred in the lines 2 to 2.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,6 @@
 def main():
     pass
+
+
 def other():
         pass
```


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*The issue occurred in the lines 4 to 4.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,4 @@
 def main():
     pass
 def other():
-        pass
+    pass
```


This needs commenting. Need to mention what it does in general and comments on the known limitations


I was leaving it to make sure we don't break people's code. But they are going to have to go in and replace ncdf with ncdf4 anyway. I could go either way.


It looks like this comment is exactly the same as [this other one](https://github.com/CartoDB/cartodb.js/commit/ecad090ac64f956feb0bddea299edade43087688#diff-5247b5f6682b4c89a4cf20d7b5f4670aR5).


Why? HHVM is working.


Thanks!


I am wondering how the while loop get executed here. I mean how it is running as a loop.


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


should be ```vm["device"]``` instead of ```vm["count"]```


Agreed!


The bucket we create here does not necessarily have to correspond to a RiakCS "bucket." We'll be writing a `BlobDB` implementation that will adapt the bucket and attachment names to the RiakCS backend.

Some quick digging turned up a few interesting pieces of information about RiakCS:

- Bucket names must be 3 to 63 characters ([source](http://docs.basho.com/riakcs/latest/references/apis/storage/s3/RiakCS-PUT-Bucket/#Bucket-Naming)).
- All objects in a bucket must be deleted before you can delete the bucket ([source](http://docs.basho.com/riakcs/latest/references/apis/storage/s3/RiakCS-DELETE-Bucket/)).
- "DELETE Multiple Objects" is "Planned for future release" ([source](http://docs.basho.com/riakcs/latest/references/apis/storage/#API-Feature-Comparison)).

I think a RiakCS bucket should either be the db_name or something more generic. Another possibility, if RiakCS buckets are deemed to be scalable to the level of having one per document (seems doubtful), is to pass the HQ bucket name through a hash function like MD5 to get the RiakCS bucket name.

Looks like we'll need to implement our own recursive delete if we want to delete entire buckets at a time.

If we make the RiakCS bucket be the database name or something more generic then we'll need to store an "index" document either in RiakCS or in some other place. This document can be looked up with only the database name and doc id, and it will contain a list of all blobs attached to the document.

Yet another option would be to skip RiakCS entirely and use the Amazon S3 API directly, which may map to the characteristics of the `BlobDB` interface better than RiakCS. We would need to consider how this will scale to many nodes (although that may not be very hard); presumably this is the primary benefit of using RiakCS.


## This is WIP at the moment, refactoring on the way.


You forget to mention the theoretical incompatibilities we discussed. We decided these are not realistic enough that we want to prevent them, but they still should be mentioned.


It is for the moment, but if at some point the build were to fail, that failure should be allowed...

Hope this helps!


@tasomaniac - that's correct. That's why there was over 10 months of time between the deprecation and release of 8.1 offered to smoothly transition - you're now more than 2 months past that point with the only option now to use an older version of Google Play services.


Wasn't iconv also made optional (again)?


Checking out the latest Suricata and libhtp tags in anticipation of Suricata's 3.0 release. Drone's builds are available for testing.


This is related to #379


In which cases faultSeqName can be null?


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


Yes, the other file is now removed (in 0d5c3eb)


We should probably first check an error wasn't returned by real_ioctl, where a context may not have be successfully created


You can just use `getChildAt`, check if it returns `null` and then use the FAB.
Reduces complexity compared to your implementation.


Hi

Sorry for taking so long to reply.

I've just done some major refactoring and updating of the code so that the tests now pass.

I've also incorporated Parse.CoreManager.getRESTController to prevent needing to modify the parse source as suggested - thanks.

I've tidied up the code but am not familiar with the ins and outs of AirBNB styling.

I had to make more modifications to the request logic and since the request format is different in the new sdk, I don't think it makes sense to retain 1.5 support within the same codebase, I could be wrong though!

Thanks
David


Good point.


It's an internal tool, unfortunately.


Good find and, again, great contribution.
Would like to merge that into Topeka after my comment has been addressed.


General class level comments,
Please keep all the public/protected methods above and private methods below.


можно импортить именно css


Oh, I should have mentioned, can you bump the package version to `2.1.0.1`? (four digits for a dev version; usually that would be in place already but last commit was a CRAN release).  Also drop a quick entry into NEWS for the fix.  Thanks much!


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.602.967577e5be45acf7939490aaf9e49485065eb151](http://build.palaso.org/viewLog.html?buildId=66707&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:09:38


DbMemoryCacheDecorator


Just a small error here.


Ok thank you for the quick replies. :)
We will figure it out. Since it is a community work, this was forgotten.

But at the end, I think Google usually takes more than 10 months to completely remove an API. I think this is unusual.


If it only were one line, I wouldn't argue. But it effectively **doubles** your getters:
https://github.com/spring/spring/blob/develop/rts/Rendering/Models/WorldObjectModelRenderer.h#L77

I agree const is a wonderful thing, but there's a limit where it's more a burden than an asset.


Thanks, I wasn't aware of that. It will be pretty easy to move these constants somewhere else, although MetaData is a good fit in my opinion.


@kasiarachwal I tried to replicate your setup but I don't think I have it right as they are all broken in 1.0.0-rc1+ but working in 0.11.4(your change was added just after this) just as before. I couldn't find your case in 0.11.4 where it was broken.

Demo with right sidenav locked open and tooltips on the input and the toggle right sidenav button.
http://codepen.io/anon/pen/YwKXKp?editors=101

Here is the same demo using `0.11.4`(version just before this change was added). It seems to function perfectly. Both tooltips work correctly when scrolled. Also I don't see the bug you mentioned with the tooltip being moved
http://codepen.io/anon/pen/zrOGYO?editors=101

Here is another copy of the last demo with just a sidenav on the left(to make sure the two sidenavs weren't canceling each other out by moving the tooltip back and forth).
http://codepen.io/anon/pen/rxBVNE?editors=101

Is your setup different then this? Can you update one of them if it is?


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 296](http://ci.infinispan.org/viewLog.html?buildId=32802&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for f8e336b281588a18b8c8f02baffca124b5660785
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:08:02


just make sure, this word is written correctly, in every place, or then, one will have a File Not Found Error:
DbMemoryCacheDecorator
dbMemoryCacheDecorator
Ernie


all these tests verify exact element order on the merged stream, but isn't part of the `flatMapMerge` semantics that order can't be guaranteed since [breath] streams are consumed in parallel? Or do you additionally want to guarantee that earlier stream items have priority to contribute downstream elements when available?


Ah ok, went back and re-read that code. You are correct in what it does.


I can tag a point release...  I was hoping to get a few other things finished first, but...


missed one, strdup("guest")); -> strdup("guest");


Normally we are not printing e.getMessage() with our custom message since it may not give some meaningful/ necessary information related to Exception. In that case error message may be misleading. So I suggest like follow,
log.error("Error while increasing debug point position value",  e);

What do you think?


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


Does this test indeed verify that breadth is respected? i.e. (why) will it start failing if breadth is changed to `4`?


Can't we use multi catch here since both error messages are same?


The podspec for the Segment subspec was broken, as it would _only_ support tvOS.


Can we just print errors and continue without any issues?


I'm getting a couple of build errors:
/Users/nicole/Slicer-git/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerOpenGLRayCastImageDisplayHelper.cxx:32:2: error: #else without #if
#else
/Users/nicole/Slicer-git/Slicer/Modules/Loadable/VolumeRendering/VolumeRenderingReplacements/vtkSlicerOpenGLRayCastImageDisplayHelper.cxx:37:2: error: #endif without #if
#endif


Hi @kitsonk,
Thank you for the fast reply.

No worries. I'm just trying to schedule when I will update the TS compiler. I would like to have it done by the end of the week but it's not critical.

If you could give an estimate it would be awesome :)

Thank you for all your work. :+1:


Same comments as above method


Nit: You added two spaces too much for the indentation here.


<?php

/*
author:munyakaberaa jean claude (ironmann350@gmail.com)
license:open source
agreement:this program can be edited modified or distributed free of fees
specification: a simple modest calculator able to calculate (+,-,*,,/) operations with two variables
warranty:there is no warranty attached to this product any outcomes will not be addressed to the author
testing:it has be tested on linux cylon 12.31 ,gnome 3.41,linux ubuntu 12.04,windows 7 and windows 8 under internet explore,google chrome and mozilla firefox browsers
technical data: programmed in php 3.2 server used apache 2.2.14 on port 8080 but can adapt most of other systems
*/

$ans="";
$equation=""; //initialization
if (isset($_POST['equation'])) $equation = sanitizeString($_POST['equation']); // get equation from inputbox
$length=strlen($equation); // equation's length
for ($count = 0 ; $count <= $length ; ++$count)// loop through the equation

{
$value=substr($equation,$count,1);//split the equation in individual characters

switch ($value)//various options
{
case "*":

$first= substr($equation,0,$count);//number(s) before the operand
$second=substr($equation,$count+1,$length-$count+1);//number(s) after operand
echo"\n";
$ans= $first * $second ;//operation
echo"\n";

$operation=true;
break;
case "/":
$first= substr($equation,0,$count);
$second=substr($equation,$count+1,$length-$count+1);
if ($second=="0")// error trapping for 0/0
{
echo"\n";
 $ans= "impossible operation";
echo"\n";
}
else
{
echo"\n";
$ans=  $first / $second;
echo"\n";
}
$operation=true;
break;
case "+":
$first= substr($equation,0,$count);
$second=substr($equation,$count+1,$length-$count+1);
echo"\n";
$ans=  $first + $second;
echo"\n";
$operation=true;
break;
case "-":
$first= substr($equation,0,$count);
$second=substr($equation,$count+1,$length-$count+1);
echo"\n";
$ans=  $first - $second;
echo"\n";
$operation=true;
break;
default://if no option avalaible
if ($value=0 || $value=1 || $value=2 ||$value=3 ||$value=4 ||$value=5 ||$value=6 ||$value=7 ||$value=8 ||$value=9) //skip numbers
{
// do nothing
$ans=$ans;
}
else
{
$ans="unkown operand";//display this if operand is not detected
}
break;
}
}
/*
the following lines are html codes that define the form
and the input box
it also contain the user manual and other information
it uses a post to send the equation to itself and reload itself
enabling the above codes to make the calculations
*/
echo <<<_END

<title>a modest calculator</title>

usage:enter the whole equation eg: 23+23 and hit enter or click "calculate"

 warning:do not put more than one operand eg:23+23+23 it may display inaccurate values it is not yet implemented

-author:munyakabera jean claude

-contact: ironmann350@gmail.com

-phone:0788612817/0725088820

-language:html/php

-release date : 1 december 2015

<form method='post' action='calc.php' >
equation <input type="text" name="equation">
<input type='submit' value='calculate'>

$equation=$ans

</form>

_END;
/* this following function makes neat the variables we are using
it also offers security because it actually do get variables from the post function
but use the post to create a html entity which is unaltered
this may be useful when some hackers use the post function to gain access to your page
when it's an html entity it does not have any action
*/
function sanitizestring($var)
{
$var = stripslashes($var);
$var = htmlentities($var);
$var = strip_tags($var);
return $var;
}
?>


Napis powiniem być dość mały aby zmieścić się obok górnej częsci ekwipunku.


I guess we could use `w.layer.isVisible()` here


Yes but still you need to solve this issue because we cannot have everyone commit to a failing master branch, right?


読み返していて今さら気付いたところですが一応。
condemn の訳ですが、ここで確定的に接収してはまずいです。ここでは、GCはまだメモリブロックを裁きにかけて、死刑を宣告するかどうかを決定する段階であって、実際の死刑宣告はここで発令してはいけません。これから有罪か無罪かを判断して、有罪のものに罰を与える不確定な段階である、という意図が訳語に反映されるべきです。
（condemn = 糾弾する、に近いニュアンスで説明していますが、要するにここの大意は「マーキングする世代を決定する」という意味です。）

このcondemn、日本語の辞書では「これから裁判にかける」に近い適切な翻訳がなかなか見つからないので、英英辞書を見ます。
http://dictionary.reference.com/browse/condemn

"to judge or pronounce to be unfit for use or service" - この judge のほう、が、割と近い雰囲気です。

以上を前提に、condemnを「それなりに気取った言葉として」訳すなら「裁定/裁判にかける」「ふるいにかける」とか、あるいは意味を確実に伝わるためには単に「マーキングする」と訳すのが適切であると思います。


Może okazać się konieczne wrócenie do framefunctions, ponieważ funkcja do waitpointów działa zbyt wolno.


What about `_onFilterChanged`?


:+1: I can finally deprecate https://github.com/chirag04/react-native-dashed-border.

@davidaurelio I'm confused, is this is the full implementation?


Same comments as above method regarding exception handling


Thanks! @kevinansfield


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#fImwCIavRgKGkt7mQFdjdg


@bnorton did you mean `$div.appendTo(document.body);` here? Otherwise the sanitized page body appears to be appended to the page without the wrapper div.


According to the [doc](http://www.alsa-project.org/alsa-doc/alsa-lib/group___control.html#ga11729138fcecfb37f9b7c744c4743fba) snd_ctl_elem_value_get_bytes returns a `const void *`.


Lol!


1.6.3 is done and up on NPM.  No problem and thanks.


Who wants to append `finally` to the thousands of promises used everywhere. ^_^

If you call `next` with `setImmediate`, you can get around the issue, but who has time to do that everywhere as well. Screw it. Better have some tea and a biscuit. That's what I'm going to do!


Done


I don't think we have any styles for `cartodb-popup` anymore.


I don’t think this has changed in any way since 0.13.0. The last change was 3e8ea9421204b51a819d983ee1d924621259043c that eliminated the use of iconv from fontselect, but it’s already in 0.13.0.


You inverted source and destination. In all c++ algoritms, sources are left from destination.


nope, almost all tests use Sets for comparison