klksdjllssagdfggdf


if we want to avoid bloating the caller with too many pg, will page the pgs.


We appear to be calling `shaderNode->typePlug()->getValue()` twice here, rather than just using the `shaderType` variable.


Do you have to set the setAutoExpireDate?


Las otras clases se encuentran en el primer Commit, ya que solo se subieron a este segundo commit las clases nuevas / modificadas.


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


Patrick & Achraf, recevez-vous ce commentaire sur un commit ?


Patrick & Achraf, recevez-vous ce commentaire sur une ligne d'un commit ?


I think the above should be isDiscontinued(new Date())


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


Hmm interesting, the logic here is indeed flawed and confusing. I updated the logic for Always Show Tray and fixed the icon not showing up when Startup Minimised was specified in commit a8cd4fc095553916fb13b380cb2753478a67e50c


Ähm ich konvertiere schon in der Bearbeitung des RGBW Controllers den Hue Wert auf Basis der 199 ... Ist das jetzt doppelt ?


actually am trying to set discontinue date by this line and i thought autoexpire does that if i pass a current date to it, is this the right way to do this?


101162956-IncorrectIDDError-4647 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1977 of 1977 tests passed)

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


@soulfrost That's not the only change. iirc with CMSGs there are 4 empty bytes before the opcode. I haven't got a clue what they are.


you could use also a
ifvarclass => "${class_prefix}_reached";
so that the pass number would not be an issue


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.561.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=66887&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:22:44


1. I would install django on a virtualenv
2. Are these made for Mac OSX or ubuntu ? especially this
 : mkdir -p ~/Library/LaunchAgents
     ln -sfv /usr/local/opt/postgresql/*.plist ~/Library/LaunchAgents


Thanks!

While you're at it, maybe you can modify ``obj // { z: 42 }`` into ``let obj = { z: 42 }`` (can't do it myself until tonight)


If there's no prefix, how about we just use "light" as the name of the attribute? We allow unprefixed shaders with the intention that someone could choose a workflow where they write shaders with the same parameters for different renderers, so perhaps we should support the same thing here? I guess we'd also need to update the other spots where we're looking for a prefix?


Hi @EladBezalel,

Could you explain why you explicitly added a zero margin for paragraph elements within an md-card-content [at line 120](https://github.com/angular/material/commit/b8ffdfe02e9ffd37a184185db41497432f2f589c#diff-674508709be58ba24eb6867770747230R120)? It kind of breaks my expectations of at least some bottom margin for a paragraph (when there are multiple p's).

In all the examples you added, you use a single p within the md-card-content, so I guess you removed the margin to make the examples look like the design spec. But with multiple p's this doesn't look like expected (no whitespace between them). I realize there are no examples of multiple paragraphs in the spec, but it doesn't say there can't be more than one either (maybe I'm missing something however).

If your goal is to make content elements 'stick tightly' to the edges of md-card-content area, I (humbly) suggest doing it for the first (top margin) and last (bottom margin) elements only. Left and right margin can be 0 for all, I suppose. And maybe do it for all content elements (not only paragraphs, which seems arbitrary to me)?


The forked process (if pid == 0) that is replaced by execl().


:beer:


Patrick & Achraf, recevez-vous ce commentaire sur un commit maintenant que je vous ai ajoutés dans une team ?


This looks like a mock .. and for mock we have a nice library called mockito, so please use mockito in test instead of implementing a mock class


Patrick & Achraf, recevez-vous ce commentaire sur une ligne d'un commit maintenant que je vous ai ajoutés dans une team ?


no need to special case here. Your code already covers this case.


I think you would have to call OrderService.discontinueOrder()


向葱哥学习！


FWIW, I prefer to have a capital letter.


Seeing as how the server and canonical development environments both run Linux, I question the use of including Mac OS installation instructions as anything other than an aside.


Thanks for your response :)

Yes, the `ippool_hashdel()`is used only once, for now... it may be used after, in other ways ! So it's a basic API check.
For the check of `this` parameter, we can add it.

About the #106 issue, it was a wrong fix:
According to the code analyzer the `fd` can be zero, and this is a valid descriptor value.
So may be we must check `fd` validity if different from -1 :
```C
     if (fd != -1) {
       dhcp->relayfd = fd;
     } else {
       close(dhcp->rawif[0].fd);
       free(dhcp);
       return -1;
```


I think you would need to call OrderService.discontinueOrder() as in OrderServiceTest


da pohui


And some more missiles with euls interaction etc.


Yeah, but there are no couts, no writes in the DataHandler... I dont't seem to get this.


I'm not an expert but I wonder if from the accessibility point of view is confusing to have a `figure` with one `figcaption` and multiple images.


Same concern here.


Good catch @agcolom!


This commit is being reviewed as a part of review issue #140.


This issue has been resolved.

Best Regards,

*Surajudeen Akande*
Software Developer, *Andela*
+2348037690966 | Skype: sirolad <https://webapp.wisestamp.com/#>
<http://facebook.com/runthinz>  <http://twitter.com/babatoclean1>
<http://github.com/andela-sakande>

On Thu, Dec 3, 2015 at 9:54 AM, Prosper Otemuyiwa <notifications@github.com>
wrote:

> *Code Quality - Critical**
>
> Your token should be read from a .env file. It shouldn't be out there for
> everyone to see.
>
> Then in the various tests methods, the assertions should be grouped
> together and there should be proper spacing.
>
> Example: This is how your testIndex() look currently.
>
>  /**     * testIndex homepage route     * @return int     */    public function testIndex()    {        $test = $this->client->request('GET', $this->api_url);        $this->assertEquals('200', $test->getStatusCode());        $this->assertInternalType('object', $test->getBody());        $content = $test->getHeader('content-type')[0];        $this->assertEquals('text/html;charset=UTF-8', $content);    }
>
> It could a look a lot better like this:
>
>  /**     * testIndex homepage route     * @return int     */    public function testIndex()    {        $test = $this->client->request('GET', $this->api_url);        $content = $test->getHeader('content-type')[0];        $this->assertEquals('200', $test->getStatusCode());        $this->assertInternalType('object', $test->getBody());              $this->assertEquals('text/html;charset=UTF-8', $content);    }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andela-sakande/9jamoji/commit/5557e4e713d1bfa8a33fbffabd39bcd3458e21db#commitcomment-14756024>
> .
>


Any reason for this empty line or a simple mistake? I recall we don't use them in our XML files.


Ok! it's an appropriate solution, more simple and i just missed it. However, it seems like there can be a breaking link issue when make operation like compression or decompression (zip, rar, ...).


Patrick & Achraf, recevez-vous ce commentaire sur une ligne d'un commit maintenant que je vous ai ajoutés dans une team sur project1 ?


101162956-IncorrectIDDError-4647 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1440 of 1441 tests passed)

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


Same here


Patrick & Achraf, recevez-vous ce commentaire sur un commit maintenant que je vous ai ajoutés dans une team sur project1 ?


This line gives a compilation warning now:

~~~~
utils.c: In function ‘janus_pidfile_create’:
utils.c:334:9: warning: ignoring return value of ‘fscanf’, declared with attribute warn_unused_result [-Wunused-result]
   fscanf(pidf, "%d", &pid);
~~~~


@aadabi
1. We typically run one server per user, so I think installing libraries locally to the service user is sufficient.
2. These instructions are clearly for Mac OS. While I think Mac OS instructions can be a useful addition, I think they should be clearly marked as such and should should be secondary to Ubuntu installation instructions as Ubuntu is used in both our server server and canonical development environments.


m


Scratch that - I see it's corrected in a later commit.


please look directory name that has prefix `_` prefix. ex: `/_xxx` too.


Agreed, We should only have Ubuntu installation.


Complimenti!


Can you remove the carret ? We prefer to stick to the exact version.


`const obj = {z: 42} `  non ?


well done


keep up the good work!


when EVFILT_PS isn't defined, ki is left uninitialized here so we definitely shouldn't be calling kevent in that case.


many congrats


kudos


Shouldn't we update to version 4.0 instead?


Do we really need these out of the box?


葱哥，这句话是会initialize所有256个元素，还是只是第一个？


what does kudos mean?


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


Assume this needs removing?


assume this needs removing


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


This import is not used.


OK great I think it makes sense that its the same as for SAG. I'll remove the option if thats fine


Encore un test de notification


Yes, that is good. `3.3` is required, but `4.0` is of course better.


I've added your Signed-off-by and updated the commit message a bit to conform with Linux conventions and and merged it.


Whoa, no one told me this happened.

Are they that spammy during lockdowns?


haha


@schaloner Boom. The issue is still there. Like I thought - the nexted JPA transaction stuff has nothing to do with the blocking code.
I created a simple [test project](https://github.com/mkurz/deadbold-blocking), the last two commits are the relevant ones. You can simple run the project with `activator run` and you will see the exception.

I am very busy right now, could you please re-add the blocking code? Actually the same code changes like in https://github.com/schaloner/deadbolt-2-java/pull/14/files needs to be applied again.


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


Don't know, feels like standard to have them these days?


I don't understand the context of this change - was there ever the case that document was null or what?


reference https://discuss.erpnext.com/t/why-on-submit-called-twice/8811


Отсутствует зависимость пакета cities_light от progressbar2. Поставил в чистый virtualenv - не работает. Думаю надо добавить в setup.py зависимость и указать через >= версию.


That's a valid concern but I believe the only use of README.md (symlinked from README) is here on GitHub :-)


@erylee I just found this in a crash dump.
```
Slogan: Kernel pid terminated (application_controller) ({application_terminated,emqttd,{noproc,{gen_server,call,[kernel_safe_sup,{start_child,{timer_server,{timer,start_link,[]},permanent,1000,worker,[timer
```

And I have a very high suspicion that it is related to this change, because
```elixir
iex> :kernel_safe_sup |> Process.whereis |> Process.info |> Keyword.get :priority
:normal
```

That elixir code just looks up the process priority of `kernel_safe_sup`, which happens to be `normal`.

I don't think the pubsub is more important than kernel_safe_sup, since one of its responsibilities is to supervise timer_server which manages timers, which are a very crucial part of OTP.

I am submitting a PR reverting this.


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


@NoahCarnahan how's this?


je te conseille ce post sur `let` `const` et `var`
http://putaindecode.io/fr/articles/js/es2015/const-let-var/
c'est bien expliqué
:trollface:


True, but we still need the pass number for the failure/success class calls, so I don't really see the benefit of removing it here. Also, class:: is faster than ifvarclass.


It shouldn't be a problem since we're storing the output and not the inputs of a relation...


this obviously must be removed


bummer, tests fail. Will work in it later


Read about execl(), you don't need to do anything, since the whole forked process is *replaced* with a new process.
It just keeps the environment (including the redirected STDOUT, STDERR).
So the new process (be it a bash script, or some program) can print to STDOUT like it always did, and then the pipe catches and contains it for us to read on the other side.


> Yes, the ippool_hashdel()is used only once, for now... it may be used after, in other ways ! So it's a basic API check. For the check of this parameter, we can add it.

It's wrong. It's not public API so stop doing this. Just read about it, this defensive programming for internal API is simply wrong, t's adding more code, making it hard to follow, debug, slower to run and compile with no added value. And it's a sign, that you've some bigger problem hidden deep underneath.

> According to the code analyzer the fd can be zero, and this is a valid descriptor value.

What kind of code analyzer is that? It must be really of poor quality. Please read that code again and please read something about file descriptors 0,1,2. Hint: stdin, stdout, stderr.

I'm locking this conversation, if you think, that something is wrong, please file a new issue so others can benefit from this conversation or better fix it and submit a pull request. Thanks!


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


se um objeto dessa classe descreve um numero, o nome da classe eh NumerosPorExtenso? O que acha? Sugestao?
DescritorPorExtenso, algo do genero


gdg


Tu crois que cela va être joli le cyan !


se sao so constantes, isso pode ser uma interface. NumerosPorExtenso? NumerosEmExtenso? perguntar ao vasco :)


Buena! Manda un pull request y lo ponemos en upstream.


C'est demandé par le client !


I don't understant why you need the clone. It's not going to clone values inside the context and VelocityContext is already designed to not modify the inner VelocityContext.


pacote :)
br.com.caelum.alura........;


I think we should also get a test on permission here ? (not all users should see this tab)


arquivos binarios foram comitados (.class). colocar no .gitignore e remove-los


See https://github.com/ubergeek42/weechat-android/issues/252 for an explanation.


It is also important to create puppet deployment scripts for deploying the website either to a vagrant development environment or to a stand-alone server.


se o setup é só dar um new repara que fica mais complexo ter isso do que nao ter isso. setup so pra economizar uma palavra adiciona complexidade desnecessaria


And what if our Iterator is not std::vector<Candle>::iterator? It can be, for example, std::list<Candle>::iterator or even std::istream_iterator<Cande>.
Please use generic type here.
I'd rather having the following code:
    for (auto it = begin; it != end; ++it)


se tirar o setup e sempre dar new ja resolve e isso some, mas se nao sumisse, essa variavel devia ser privada.


So we're looking for


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


Why not use the fixed "/proc/self/stat" as the file to read?


A more specific exception catch wasn't possible since the code directly throw `RuntimeExceptions`. I now changed these into `IOException`.


em caso de equals, eu sou fã de assertEquals mesmo :)


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


'lectoriy.views.index' у меня даёт варнинг; убери кавычки


Useful for delphi samples (http://rghost.net/7XMXnJpDq - "test"). Check out af + afl on EP.


20a99... 10a19... fica mais facil de ler, o que acha?


a co využít callbacku $value, který v kombinaci s callback postrádá smysl víc než options (které teoreticky můžou v callback condition být)?

Vyhovuje i ten název, callback je value callback condition.

Chtělo by to jen zohlednit [tady](https://github.com/wodCZ/Nette-NoGrid/blob/master/src/NoGrid.php#L185-L187)


kaoskaoskaos


When displaying the place of an event, it is better to use:

`poe = _pd.display_event(database, event)`


Nevermind, I just saw https://github.com/emqtt/emqttd/commit/0b1540f1db5d207f3c3ac4da98249a32228a891c.


NOTE: Since December 2015 Redback is officially in MOOSE's [repos](https://moosebuild.org). This will cause a bit of rearrangement on the way Redback is being developed, maintained and updated. For any issues you encounter do not hesitate to contact us.


Here is a typo: hostname -> $hostname


tested with numpy 1.9.2 and 1.10.1


Indeed the inner context construct should be enough to protect it. Fixing


repara que seu teste está igual a implementacao? se o teste é uma copia da implementacao, não estamos testando... pq se erramos a implementacao, erramos o teste tambem. o teste deve conhecer o resultado, de preferencia não calcular o resultado da mesma maneira que a implementacao (pq ai cai nesse caso). entao seriam:
assertEquals("vinte", extenso.descreve(20));
assertEquals("vinte e um", extenso.descreve(21));
...


Temple added


@dmtch o/ Awesome fixes those day, however, without test there is a very very high chance that this will not work anymore after some week. So please if you can add tests for all those awesome new eval variable you have done that would ensure that they will not break over the time.


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


saw approval from hmmr
at https://github.com/basho/riak_kv/commit/f48c73805179d11b6fbbbedbeb0d2a720f5229db


merging basho/riak_kv/at/configure_concurrent_queries = f48c7380 into borshop-integration-1289-at/configure_concurrent_queries


basho/riak_kv/at/configure_concurrent_queries = f48c7380 merged ok, testing candidate = 2964182b


cc @davidkudera (nevím jestli ti oznámí každou odpověď, nebo jen mention)


idem aqui :)
repara que o tette vai ficar muito mais simples quando escrito da forma que citei (mas bem mais manual claro).
repara tambem que o teste atual esta dependendo da implementacao (nome das variaveis estaticas, que deveriam provavelmente ser privadas)


Just a note on tests, there's a somewhat general facility in the `commands` file, but I'm not sure if it supports different major modes (it should).  You can give it a series of buffer states and it walks through that automatically.

I keep saying it but I really should write something about all the testing stuff and maybe we could throw away the useless bits and pimp the useful bits so we make it even simpler.  I'll look at it this week, I promise.  Please write me an angry message if I won't!


Actually no, it's not enough. If you have an object in the inner context (for example a list) and it's modified by a mail template, it's still going to be modified when the template evaluation returns


Awesome!


Where is the -min suffix added? If it's done by gulp-minify, please add a note to the code.


ahhhhhhhhhh. This refactoring does not work as `test_backend_with_ws` is no in the `adhocracy_test` group of `supervisord.conf`


Also, is it possible to change the suffix to .min.js?


Looks so: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/freeze


I know that VelocityContext clone() is shallow but that's the best we have ATM


When the `PtyProcess` instance (`self.ptyproc` on a Pexpect spawn instance) is garbage collected, it will indeed close the file descriptor. But I don't see how that could have happened when you're still calling `E.sendline()` - clearly there is still a reference to the object, so it shouldn't be garbage collected.

Could something else be closing an fd explicitly? I have run into odd effects before because the numbers representing fds are readily reused. If one fd gets closed while its corresponding 'handle' object (e.g. a `PtyProcess` instance) still exists, when the handle object is garbage collected, it may close an unrelated file which has been opened in the meantime:

```
A =  open('foo')
# handle A with fd=5
os.close(5)
B = open('bar')
# handle A with fd=5
# handle B with fd=5
del A
# handle A is garbage collected, closing fd 5
B.read()
# Bad fd!
```


I agree, I will change this when implementing token renewal.


Note that the comment has been improved after this commit. The new one reads:

"
        // Note: We create an inner Velocity Context to make it read only and try to prevent the script to modify
        // its bindings. We also clone its values to try to protect them. It's not guaranteed though since the
        // clone() method of VelocityContext only performs shallow cloning.
        // However, this whole code is executed in a thread and we recreate the context for each email so it should be
        // pretty safe!
"


Holi


Also if VelocityContext.clone() impl is improve in the future, this code will work to the best extent.


This commit should fix that: https://github.com/meetecho/janus-gateway/commit/c07b32bd5d5643f11f10af10569427a3fabb3447


Updated for formatting issues and version number.


Comments taken from `dnsmasq.conf` default file.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.153.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=66886&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:44:22


I have 2 problems with this. Firstly I don't think it does what you intend: this greps the *whole patch*, not just the title.

Secondly, we've already grepped this stuff above. I don't think we should do the patch filtering twice, independently, because the implementations might be subtly (or even wildly) different. If possible, I think you should confine this to a single filter. So either make grep do what you want here and don't produce the log message above which might be inconsistent, or pass the results of the previous filtering to format-patch somehow.


Adding a strtoupper may increase compatibility.


... && Config.ConfigData.Location < MiningLocation.Length)


Or, actually -- could it be left as third_party.js? The 'dev'/'prod' in the filepath already distinguish the files.


ok


It's done, I just haven't committed it yet.  It'll go in this evening.

On Thu, 3 Dec 2015 at 11:48 Matthias Kurz <notifications@github.com> wrote:

> @schaloner <https://github.com/schaloner> Boom. The issue is still there.
> Like I thought - the nexted JPA transaction stuff has nothing to do with
> the blocking code.
> I created a simple test project
> <https://github.com/mkurz/deadbold-blocking>, the last two commits are
> the relevant ones. You can simple run the project with activator run and
> you will see the exception.
>
> I am very busy right now, could you please re-add the blocking code?
> Actually the same code changes like in
> https://github.com/schaloner/deadbolt-2-java/pull/14/files needs to be
> applied again.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/schaloner/deadbolt-2-java/commit/9b9f11e172084f62c9b1589b98032b7587898a15#commitcomment-14758559>
> .
>


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


I pretty much installed the website in my area but the DB is not set up rightbecause of the user name error: File "/home/aadabi/website/myenv/lib/python3.4/site-packages/django/db/backends/postgresql/base.py", line 175, in get_new_connection
    connection = Database.connect(**conn_params)
  File "/home/aadabi/website/myenv/lib/python3.4/site-packages/psycopg2/__init__.py", line 164, in connect
    conn = _connect(dsn, connection_factory=connection_factory, async=async)
django.db.utils.OperationalError: FATAL:  role "myusername" does not exist


This is not a good idea. If you want to wait for a future to complete, invoke get/checkedGet on it.


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


For testing
On Dec 3, 2015 12:24 PM, "Eran Harel" <notifications@github.com> wrote:

> MockMetricFactory?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/outbrain/ob1k/commit/6b5112335611b70911e59e45e88d46cfc12ba927#commitcomment-14758011>
> .
>


@alexandrudima There was another bug that introduced this. It's all fixed now. I started to comment before seeing the real bug. This change has been reverted


Oh you mean change its name?
On Dec 3, 2015 1:08 PM, "Guy Marom" <marom.guy@gmail.com> wrote:

> For testing
> On Dec 3, 2015 12:24 PM, "Eran Harel" <notifications@github.com> wrote:
>
>> MockMetricFactory?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/outbrain/ob1k/commit/6b5112335611b70911e59e45e88d46cfc12ba927#commitcomment-14758011>
>> .
>>
>


No assertion in this test


To avoid warning, can you replace

if ($conf->global->FICHINTER_EMPTY_LINE_DESC!=1))

with

if (empty($conf->global->FICHINTER_EMPTY_LINE_DESC))


@ashishchoure

Can you please make more changes


Thanks!


Is the async execution in a dedicated thread necessary here ? I would say no, because you are not testing if the SettableFuture works properly and in the test you are just waiting for the future to finish. So regular synchronous execution from a mock(see comment above) would be more suitable here


Good idea, will push it in with my next commit


@mhgoldman  Couple changes:
- Realized the join I put to regions wasn't needed if we relied on region_ids alone
- `ENDD` typo -> `END`


could be replaced by a mock


OK


> Also if VelocityContext.clone() impl is improved in the future, this code will work to the best extent.

I doubt this will ever happen (among other thing that would be an API breakage) and in the meantime we make a useless clone.


Nice one!


Set without a generic type ? why ?


no assertions in the tests


Why restrict to `Img`?  Can't we work on `RAI`s.


I think it's better if it returns an empty `RAI`.


hi ,bạn có thể thêm chiasenhac của vietmedia vào để nghe lossless được không


@mhgoldman Thanks for notes you've sent on sortable - will go through on Fri.

One thing that might happen here (line 3) is this will run once at startup. Any Regions added after app startup won't be included (until restart).


Hoping to increase performance


what does this mean?


what if instead of a  sysout, the size of tha cache would be in the message for assertEquals ?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2843


I did a code walk through and look good.
LGTM !!


We should implement `Conforms` to make sure that we work with 2d objects.


printStackTrace is a bad idea even in tests


Import data


why `LocalzingCursor`?


add `Conforms`.


can be defaulted to true and `required=false`


Moving the discussion back to PR #2446


jo to by asi taky dávalo smysl, sice se mi moc nelíbí, že člověk bude muset psát prázdný pole `[]` ve většině případů, ale to není tak hrozný.. Nejlepší by bylo to zmínit někde v dokumentaci asi.


Formatting change.


@borshop merge


En fait, j'ai volontairement utilisé le mot-clé "let" dans les cas où l'objet serait muté, mais c'est dans l'autre article sur Object.assign, là il n'y a aucune raison logique d'utiliser autre chose que "const", agreed ;)

Sinon dans Object.assign vous verrez l'utilisation de "let", c'est volontaire même si inutile, pour marquer le côté mutable justement.


Speed  up


don't return null, rather an empty image?!


@bradenmacdonald I'm not sure. When I investigated this issue I noticed that Selenium was doing some scrolling, but the problem seemed to be that it was scrolling down *too far*. So before it started to scroll, the button to click was hidden *below* the visible portion of the page, and after it scrolled down the button was hidden *above* the visible portion of the page. When I set a timeout before the `click` and then scrolled the button into view manually, the test(s) would pass.


Why using reflection when a specific class is being instantiated ? You can get the class probably from InstanceIdentifier, but is it really necessary to return different objects ? Using a mock, you can specify different return values and their order, so you can craft the mock to return what's expected in the test (would be faster and simpler)


Inbox


Oh! thanks :D :dancers:


- Coucou c'est mwa.


After the `var` changes, `origFile` and `checkRemote` will probably not have to be put in this exception rule.


Thanks, I will make some test for this.


Bundler now includes version in Gemfile.lock. Consider updating bundler to latest (1.10.6) with `gem update bundler`.


This could break existing manager themes having their own `header.tpl` Smarty template.
While chances are there aren't so many custom manager themes, probably worth a "bold" notice/warning in the release notes


-Valentin, Coucou c'est mwaaa.


why are the fields package private ?


remove


ru?


Based on what your sending its impossible to tell. We would need to know
your exact setup. What rom are you running? And where is the error from fb
or settings?

On December 3, 2015 4:01:25 AM Idrees Dargahwala <notifications@github.com>
wrote:

> Hello .. i picked recent panel commits frm base and settings .. without any
> conflicts
>
> i am getting this error
> The method dismissTask(long) in the type TaskView is not applicable for the
> arguments ()
>
> please help
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Team-OctOS/platform_frameworks_base/commit/6878b1c6fbd674d2113661865ff2e9341fea2ad1#commitcomment-14756128


Done, thanks.


Je ne comprends pas comment tu veux structurer les données... Pourquoi y a-t-il `$scope.tableau` et le filtre `valeursIndices2`? J'avais cru comprendre que les données dans le tableau et dans le graphe devaient être les mêmes. Dans ce cas, je pense qu'il ne faut pas les dupliquer...


why deleting these ? doesnt it break the tests ?


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


Thanks for notifying it.Corrected


this seems like an odd change, replacing distributed-tx-features with sal-binding-api...


A better name might be `getMatching` - a bit confusing while reading the code at the usage sites.


This is the commit we need to be working from in the reduce-debug branch.


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


All data


Dit is een test omschrijving


Can use FluentIterable here and do the filtering just for the catalog items.


Yes the comments will go. Dont worry.

I might do multiply by 2 at the end. Not sure yet. I just found multiply by 3 is definately not the same like CF values and gives too narrow D range to play with


Je pense que l'on devrait trouver de meilleurs noms aux filtres que `valeursIndices` et `valeursIndices2`.


ancora kudos!


Please check if logic is true before approving.


I have my own custom rom based on AOSP .. I wanted recents clear option .. So i picked it from fwb and settings frm octos .. I picked it up widout any conflicts in fwb .. The error above is in fwb ..


Nice, I like this refactoring :+1:


gbhnjkm


fygihuijmk


???? I never wrote this patch


huji


请问一下, 是出于什么目的把`this.__backdrop.css('zIndex', index);` 这句给注释了的.

我看之前老版的用`__zIndex`来设定层级的时候还保留了这句.

现在我这边有一个业务逻辑, 需要能够在设定`popup`的`zIndex`的时候也一同设定`backdrop`的`zIndex`, 不然会导致`backdrop`的层级高于`popup`.

场景如下:

有弹窗A 设定层级200 ; 有弹窗B 设定层级1000;

弹窗A是根据`hover`事件临时创建, 又在`mouseleave`事件中销毁

弹窗B是只创建一次, **后面只是close但是不remove**.

在弹窗A展现的时候 `Popup.zIndex 为201`

在弹窗A中有按钮, 点击之后, 假设是第一次点击  初始化弹窗B同时展现 `Popup.zIndex 为 1001`

当用户的鼠标移到弹窗B上的时候, 弹窗A的`mouseleave`事件触发, 销毁了弹窗A `Popup.zIndex 为 1001`

用于关闭了弹窗B, 同时再次通过`hover`事件创建和初始化了弹窗A `Popup.zIndex 为 201`

在新建的弹窗A中点击按钮, 此时是第二次点击 **只是show** 而不需要create `Popup.zIndex 为 201`

此时 弹窗B的`popup`的`zIndex`变成`201` 但是弹窗B的`backdrop`的`zIndex`还是`1000`

导致`backdrop`遮住了`popup`.


gbhn


hello hi  korean telcom LG uplus APN update.
additional please Thank you
<apn carrier="LG uplus IMS" mcc="450" mnc="06" apn="IMS" type="ims" mmsc="http://omammsc.uplus.co.kr:9084" protocol="IPV4V6" roaming_protocol="IPV4V6" />
  <apn carrier="LG uplus" mcc="450" mnc="06" apn="internet.lguplus.co.kr" type="default,mms,supl,fota,cbs" mmsc="http://omammsc.uplus.co.kr:9084" protocol="IPV4V6" roaming_protocol="IPV4V6" />
  <apn carrier="LG uplus IA" mcc="450" mnc="06" apn="" type="ia" protocol="IPV4V6" roaming_protocol="IPV4V6" />


Sorry that was a typo on my part, I'll fix it - My fingers are used to typing sysctl, not scrypt.    Basically the configure passing, but make was failing because the former was not checking for aes.h.


You don't need to duplicate names in both the lists, they serve different purposes. Use `global` for things that came from other files and are not defined here. Use `exported` for things that are defined here, not used here, but will be used in other files.

After this, we can merge.


As discussed, I'm not a huge fan of this kind of debug logging in production. Personally, I'd move the `__VERSION__` into a header at the top of the file.


<apn carrier="LG uplus IMS" mcc="450" mnc="06" apn="IMS" type="ims" mmsc="http://omammsc.uplus.co.kr:9084" protocol="IPV4V6" roaming_protocol="IPV4V6" />
  <apn carrier="LG uplus" mcc="450" mnc="06" apn="internet.lguplus.co.kr" type="default,mms,supl,fota,cbs" mmsc="http://omammsc.uplus.co.kr:9084" protocol="IPV4V6" roaming_protocol="IPV4V6" />
  <apn carrier="LG uplus IA" mcc="450" mnc="06" apn="" type="ia" protocol="IPV4V6" roaming_protocol="IPV4V6" />


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.609.bc95a36ad12c09573876fe534ab425875be0d631](http://build.palaso.org/viewLog.html?buildId=66898&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:05:27


Was just purely cosmetic, no functionality addition.   Just thought it read better.  If you like I can remove that in another commit.


Major challenge in this code was that the nodejs works asynchronously. So the msg publishing was delegated to the lower rung and the other db related code were executed. As the control returned to callback function, the array variable storing the macids had been destroyed already and hence the msg publishing failed. Therefore in order to send the msg  soon as possible I removed the callback in mqtt msg publish script.


I think I removed this... configd looping with firmware actions :(


There's probably a better way to do this, but no need for this PR. We'll fix it later. :)


and `ALL`, although I'm not sure what it will be used for


`bin/rake db:seed` is giving an error when `Settings.set ...` is called:

```
$ bin/rake db:seed
rake aborted!
ActiveRecord::UnknownAttributeError: unknown attribute 'user_password_email_text' for Settings.
...
NameError: undefined local variable or method `user_password_email_text=' for #<Settings:0x007fc2c2e4ac00>
```


What about this implementation?


according to [this](https://www.polymer-project.org/1.0/docs/devguide/styling.html) `style` tag should be inside the  `template` .


Should check the catalog for existing item as well?


No problem - I appreciate the response and your work here - I've used the repo a few times in research etc, and just wanted to tweak so the test functions built automatically.  Thanks for the replies!


This breaks any content scrolling on desktop (Chrome/Linux)


Sorry - did mean I left a typo in somewhere?


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


重新设计了构造函数


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.617.bc95a36ad12c09573876fe534ab425875be0d631](http://build.palaso.org/viewLog.html?buildId=66899&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:07:17


Please see 4f43181d25843f53c6


Perhaps it should be user_password_**reset**_email_text ?


No worries :)


Should we escape `type` here?


i ithink now it worked...

thx @philmmanjaro

vlw fwi, Holmes :)


I haven't experienced issues when adding this one, the rc.restart_webgui should  do only one thing... but if it doesn't we might have to fix that...


Okey.


Aha, thanks for the tip. Do let me know if I can help.

I'm still a little confused about the relationship between `sp-local-pair` and `sp-pair`. `sp-pair` seems to be virtually unused (smartparens-config.el seems to have the only call to it).


Here should 'continue'  instead of `return`.


我觉得，可以把这两个测试分开，x.0 < price < x.5 和 x.5 < price < x + 1
再加一个测试整数保持不变。


起步价内也应该收等待费。


Je ne comprends pas vraiment tes noms de variables...
Je pense que `$scope.tableau` devrait s'appeler `$scope.valeursIndices` par exemple.
Au lieu de `ligne in tableau` il faudrait `valeur in valeursIndices`.


hello hi korean telcom LG uplus APN update.
additional please Thank you

!.<apn carrier="LG uplus IMS" mcc="450" mnc="06" apn="IMS" type="ims" mmsc="http://omammsc.uplus.co.kr:9084" protocol="IPV4V6" roaming_protocol="IPV4V6"
<apn carrier="LG uplus" mcc="450" mnc="06" apn="internet.lguplus.co.kr" type="default,mms,supl,fota,cbs" mmsc="http://omammsc.uplus.co.kr:9084" protocol="IPV4V6" roaming_protocol="IPV4V6"
<apn carrier="LG uplus IA" mcc="450" mnc="06" apn="" type="ia" protocol="IPV4V6" roaming_protocol="IPV4V6"


图挂了


I don't want to risk destabilising firmware upgrades with this, it might look ok now, but this is required code on repo migration , e.g. 15.7 -> 16.1

The problem really is that going back into PHP should be deprecated for configd, we don't know what is going to be called (now and later).


which line? 36? no, it is as it should be. pair constrcutors are intended only for VS2013.


@davidkudera tak to prosím rozsekni, já se přikláním k $value, ale ty jsi šéf a máš poslední slovo :)

---
Teoreticky bych mohl ten typehint na array u $options oddělat a udělat ho "volitelný".

Pokud typ bude callable:
```php
$grid->addFilter('person', Condition::CALLBACK, function($data, $value){})
```
by se uložilo jako
`$options = []; $value = function..`

---
```php
$grid->addFilter('person', Condition::CALLBACK, [$this, 'filterPerson'])
```
by se uložilo jako
`$options = []; $value = [$this, 'filterPerson']`

---
```php
$grid->addFilter('person', Condition::CALLBACK, [whatever], function($data, $value){})
```
by se uložilo jako
`$options = [whatever]; $value = function..`

---
```php
$grid->addFilter('person', Condition::CALLBACK, ['a' => 'b'])
```
by vyhodilo výjimku `InvalidStateException('Condition of type CALLBACK needs valid callback');`

---
```php
$grid->addFilter('person', Condition::CALLBACK, function(){}, function(){})
```
by vyhodilo výjimku `InvalidArgumentException('Either provide options array as third parameter and callback as fourth parameter or callback as fourth parameter if you do not need options.');`

---
Ikdyž ta poslední message je hodně divoká :D


Hugo config crashes now, suspect this commit ...:
```
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x50 pc=0x63119]

goroutine 1 [running]:
github.com/spf13/hugo/commands.InitializeConfig(0xc8200bda58, 0x1, 0x1, 0x0, 0x0)
	/Users/bep/go/src/github.com/spf13/hugo/commands/hugo.go:313 +0xd59
github.com/spf13/hugo/commands.config(0xb18ee0, 0xb45ec8, 0x0, 0x0, 0x0, 0x0)
	/Users/bep/go/src/github.com/spf13/hugo/commands/list_config.go:36 +0xb3
github.com/spf13/cobra.(*Command).execute(0xb18ee0, 0xb45ec8, 0x0, 0x0, 0x0, 0x0)
	/Users/bep/go/src/github.com/spf13/cobra/command.go:568 +0x63b
github.com/spf13/cobra.(*Command).ExecuteC(0xb180e0, 0xb18ee0, 0x0, 0x0)
	/Users/bep/go/src/github.com/spf13/cobra/command.go:662 +0x54e
github.com/spf13/hugo/commands.Execute()
	/Users/bep/go/src/github.com/spf13/hugo/commands/hugo.go:125 +0x71
main.main()
	/Users/bep/go/src/github.com/spf13/hugo/main.go:24 +0x28

goroutine 17 [syscall, locked to thread]:
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:1696 +0x1
```


It has a point after debug


Make it an else to the above if to highlight that it's the alternative code path. (took me a bit to realize it, I'm a slow one :) ).


haxe.Utf8.decode should be deprecated. It “decodes” to Latin-1, which can not represents Unicode strings.


Nope, RAI doesn't have a Cursor. Maybe we could restrict it to IterableInterval. Problem is with Regions.iterable because we get a IterableInterval<Void> which only iterates over the foreground pixels.


hello hi korean telcom LG uplus APN update.
additional please Thank you

!.<apn carrier="LG uplus IMS" mcc="450" mnc="06" apn="IMS" type="ims" mmsc="http://omammsc.uplus.co.kr:9084" protocol="IPV4V6" roaming_protocol="IPV4V6"
<apn carrier="LG uplus" mcc="450" mnc="06" apn="internet.lguplus.co.kr" type="default,mms,supl,fota,cbs" mmsc="http://omammsc.uplus.co.kr:9084" protocol="IPV4V6" roaming_protocol="IPV4V6"
<apn carrier="LG uplus IA" mcc="450" mnc="06" apn="" type="ia" protocol="IPV4V6" roaming_protocol="IPV4V6"
APNs-conf.xml
LG Uplus file dropbox textfile
https://www.dropbox.com/s/bjovorfqrzoaf6g/new%203?dl=0


Apparently it didn't pull in the last commit, so there really won't be a change here - I'll make sure it doesn't show up on the next commit.


Very nice! Two things:

 - this should be in master
 - in master, it was moved to https://github.com/transloadit/uppy/blob/master/website/src/api/index.md, so it's actually part of our visible documentation


oh, I see _MSC_VER > 1900 should be >=


:heart: :D


I'm not sure I understand this, could you explain a little?


`Views.iterable(rai)`.


If the actions are singularities, then it's not problem.. but if all glues together.... your right.
I will revert this change for now, as soon as we're replacing the http server part, we can re-evaluate.

p.s. I don't think system_firmware_configure() should be in the webgui restart part... it kind of looks like the right event on the wrong place I guess..


I would really like to return an Optional, but that isn't possible. I could return an empty Img/RAI, but that way you wouldn't know if holes were found or not.


tak udělejme to teda přes to value a dal bych to asi bez toho kouzlení s nepovinným parametrem.. Ať to moc nekomplikujeme a neděláme z toho jquery :wink:. Co myslíš?

A šlo by to třeba někam připsat, jak se to používá?


停车费用为0.25


very good choice to use a placeholder instead of a class :+1:


错别字


".kibana_$3' is not flexible enough. It should be just $3 so that any index name can be specified.
In our use case we will create an index based on department name so we can segregate who has access to which kibana instance. e.g: ".sales_dash"


Right, then I'm restricting it to IterableInterval


Do you think we could merge this into master and I fix it there?


dobře, provedu.

Dokumentaci dopíšu, testy zkusím taky.


Why treat the catalog as a fallback and not the same as other registered types? In the `DEFAULT_VERSION` case I would expect that `getBestVersion` picks amongst all available registered + catalog types. A catalog item might have a higher version with the same symbolicName.


Sure :+1:


Shouldn't we also change the class name? Feels a little jumbled. What do you think?


You are right. This was needed for backwards compatibility to make seamless upgrades from 15.7.17 -> 15.7.18 happen as the repo needed to be upgraded. Let me think of a better way. :)


paráda, díky ti moc


没做过出租车没经验啊!


tookfint kjuw adawdaw
dwae
htffhyt
fth
hft
jgygjy


why do we need AUTO_INCREMENT=3 ?


You can check before with the other `Op` right? We will soon have Java8, maybe until then you can add a fixme.


Can you pastebin your build. Your missing more of the error. I need to see more

On December 3, 2015 6:26:48 AM Idrees Dargahwala <notifications@github.com>
wrote:

> I have my own custom rom based on AOSP .. I wanted recents clear option ..
> So i picked it from fwb and settings frm octos .. I picked it up widout any
> conflicts in fwb .. The error above is in fwb ..
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Team-OctOS/platform_frameworks_base/commit/6878b1c6fbd674d2113661865ff2e9341fea2ad1#commitcomment-14759316


I think its key to retain the .kibana_ root for clarity, but its just a personal view of course and based on your own example in the issue (.kibana_env1)


Fixed it


Fair enough - I'd missed the /bin/sh - on all the systems I test /bin/sh can use the ${array} construct, but I suppose there are edge cases.  I'll remove it.


Whooaa!


e.getMessage() is missing in the Web UI warning


Are you sure?


Okay, sounds fair. Maybe we can just use one or the either? I vote for `npm run dev` and `npm run prod`.


Again personal preference to fail early approach. Good probability to pick the wrong item here. Perhaps should fail even earlier, at insertion time, making aliases unique per symbolicName? But what's the point of aliases then... Seems scary not to know which one to pick.


No rush, the webgui restart is back as it was...
There are more challenges when detaching functionality.


I guess that's better :)


Here you are not tied to a single level log?


use Path.Combine instead of concat string


Isn't pylint going to complain about this?


seems to be previous PR fixed that..? sorry just merged.


You didnt set the framework to v140 ? its making bug in 2013


:santa:


if possible, using a `Cursor` is always faster. However, keep in mind that you can't assume `FlatIterationOrder`, so I think you have to use a `RandomAccessibleInterval` as the input.


The problem is webpack's define plugin won't replace the __VERSION__ with the number in a comment, so it needs to be in the code :disappointed: . I tried it out. I'm happy to change this, we can probably use the [banner plugin](http://webpack.github.io/docs/list-of-plugins.html#bannerplugin) on the entry points (since this file is an entry point.

I'll explore that.


I think it does, though it is not streamed.  There is a `ByteArrayHttpMessageConverter` in the `RestTemplate` converters list that can read the body directly (any MediaType), and recognises `byte[].class` as a response type.


This breaks the unit-tests because you are moving the file and in the unit tests it gets opened twice. Once when running over all HL7 files and once in the specialized tests. I will rework the tests to make it pass again.


Oh, I now see it's addressed in the next commit.


Hi. Really happy to see you patch. However I have problem downloading the zip file with reporting :"file broken" when I uncompressing it. So I just patch my own kernel.(BTW,I use 4.0.0-kali1-686-pae) When I run

make xconfig

in tty7, I found this:
![qq 20151203193909](https://cloud.githubusercontent.com/assets/16133680/11559717/071fbeda-99f6-11e5-9853-64bbe9352770.png)

So here are my problems:
1.Do I have to add   "CONFIG_88EU_MON_MOD" or something like that in the .config file?
2.Could you please upload the final r8818.ko file? It will be very helpful.


Sorry for making you change so many things, but `getDiff` will never have to be in the exported list anyway, will it? Since it's inside another function's scope, we can't and won't export it.


this only works for `FlatIterationOrder` right? Use a `RandomAccessibleInterval` here.


Hmm, probably because some trouble I had with npm.
Is this a problem for you?


you can't cast any adress to a multi byte variable like int. the only reliable way it to write bytes. i used memcopy to be consistent with the rest of this block.


changed to valuable name valid_user?


fixes #14


The commit message sounds crazy, but I understand the purpose of this patch


Yes, please do


`versionMatches` - should start with a capital letter.


fixes #13


Ok, if we are to merge this, then it would be good to add an issue to remind us to come back and fix this.  However, as discussed, I would really like to try, if possible, only issue PRs for spec we can satisfy.  And we should try and resolve any issues you find when adding tests too.


getDuration is in ms


Actually, I forgot the most important part ("Improve performance of simplifyTypedef by using Token::simpleMatch by ~15%").


@doadin  https://github.com/RR-msm7x30/android_device_samsung_ancora/commit/50ad00ed0fd92b7b7bd080e797f0733bd6aa6fb1


I do not see much point in defining/including `nil()` instead of `[]`, because the type language is perfectly capable of defining and dealing with singleton types (this is explained in the text before the point where `nil()` is mentioned), but I can live with the above change (i.e., defining it that way).

The rest of the changes above are fine, of course.

I guess if somebody wants to be strict in this document, then the occurrence of `[]` in the definition of `iolist()` in the table below should also become `nil()`.  Also, `[any()]` should become `list(any())` in that table - and similar changes should occur for `[char()]` and `[char(),...]`.


I'd use `? super Set<String>`.


Im no sure for this for 5.1


Same here.


Hi @aidanhs I wonder if you could explain why `-t 100000000` was added here?


It's for defining global/base pairs, usable in all modes.  There is some sort of inheritance mechanism involved so in theory you can derive local pairs from global ones by overriding certain declarations (like `:when` etc).

It is my long-term goal to come up with some better interface for configuration, including buffer-local settings


Hey what was causing the lag?


Very good change!


Well I can if I add ```in().iterationOrder().getClass().equals(FlatIterationOrder.class)``` to the conforms Method. The jump from Views.iterable to Views.flatIterable should be acceptable, don't you think?


yes

On Thu, Dec 3, 2015 at 1:09 PM, Guy Marom <notifications@github.com> wrote:

> Oh you mean change its name?
>
> On Dec 3, 2015 1:08 PM, "Guy Marom" <marom.guy@gmail.com> wrote:
>
> > For testing
> > On Dec 3, 2015 12:24 PM, "Eran Harel" <notifications@github.com> wrote:
> >
> >> MockMetricFactory?
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/outbrain/ob1k/commit/6b5112335611b70911e59e45e88d46cfc12ba927#commitcomment-14758011
> >
> >> .
> >>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/outbrain/ob1k/commit/6b5112335611b70911e59e45e88d46cfc12ba927#commitcomment-14758997>
> .
>


As for the usage, most of the default pairs are hard-coded in `sp-pairs` variable directly so the default config doesn't call it.


> per your comment, I want to make sure I am not forcing your hands to make it GPL. If you wish /need to release it with another license due to restrictive material use, I will just switch the repo to the non-GPL section.

That was not my point, this was a GPL project since the beginning. My point is that you forked and redistributed this project _before_ we stated its license. If a license isn't stated in a project, you can of course ask the team behind it about it, but while you don't know anything about it you must assume it's "all rights reserved". See:

> You're under no obligation to choose a license. It's your right not to include one with your code or project, but please be aware of the implications. Generally speaking, the absence of a license means that the default copyright laws apply. This means that you retain all rights to your source code and that nobody else may __reproduce, distribute, or create derivative works__ from your work.

[my emphasis]. Source: https://help.github.com/articles/open-source-licensing/#what-happens-if-i-dont-choose-a-license


@Anonymas I couldn't find it anywhere. Maybe there is a share feature in the app that produces a link?


2015-12-03 14:51:49 SmartAIMgr: Entry 2061000 SourceType 9 Event 1 Action 27 uses non-existent Spell entry 0, skipped.
2015-12-03 14:51:49 SmartAIMgr: Entry 2077700 SourceType 9 Event 1 Action 27 uses non-existent Spell entry 0, skipped.


I don't think so. This is Mongoid and not vulnerable to SQL-injection-like problems.  As far as I can see, the string entered by the user can be anything and won't cause unexpected things to be executed.


I have a slight preference for numbered patches, if it's only a matter of size, I'd rather keep them. If it makes `git am` work better, then remove them.


Fan vad lol med hej hej!


The reason is that these symbols use the bold font attribute while the numbers use normal font weight. The additional `set_color` commands are needed to reset the font weight. It's a cosmetic issue that probably strongly depends on individual font settings. It might be better to use bold everywhere or nowhere in this context. With my terminal font, I would prefer the current behaviour. But it's not that important to me. At least, I would like to have the same font weight for each number within the prompt.

By the way, is there a simple command to reset only the bold attribute without changing the font color?


I've checked that in mobile view zoom element is not present ever, so pan is always disabled.


Yes, I can check with the other Op but that way I would need two iterations over the Object. But if Java 8 is coming soon I'll just add a FIXME and for now return an empty Object.


Note that this code isn't new, I just moved it around to make the `useChroot` variable available when determining `TMPDIR`. But the comment is outdated because fixed-output derivations do run in a sandbox nowadays.


By


or `return bool(self.get('assuranceApproach')` (not important, just informational)


Was this ever be merged? Because I currently struggle with this part:

```html
<div v-for="childNumber in customerChildrenCount">
  <label for="input-age-child-{{childNumber}}">Child #{{childNumber+1}}</label>
  <input type="number" step="1" min="0" max="26" id="input-age-child-{{childNumber}}" name="input-age-child-{{childNumber}}" v-model=" 'ageChild' + childNumber "/>
</div>
```

The loop works, but the dynamic v-model throws an error:

```
Invalid setter expression: 'ageChild' + childNumber
```

Is there any alternative way?


you can't convert an `IterableInterval` to another `IterationOrder`. Input type must be `RandomAccessibleInterval`. Then we directly do a `Views.flatIterable(...)` in the compute method which is never worse than using the `RandomAccess` itself. `Views.flatIterable(...)` already checks if the incoming `RAI` is flatIterable or not.


MongoDB is most certainly vulnerable to injection attacks: https://www.owasp.org/index.php/Testing_for_NoSQL_injection#How_to_Test

Please ensure user input is escaped.


don't we need EVENT_OUTPUT_START / EVENT_OUTPUT_END for tasks?


in *this particular commit* you can see that I'm removing an actual kill of the process. it must be said that I added back that line of code in my branch purely for testing, none but me ever ran that code. otherwise it might look like I'm lying in #252 :s


I am going to make small steps for logs view. I ma going to create this view first:
https://raw.githubusercontent.com/kubernetes/dashboard/master/designs/11-11-2015/16_card%20variations.png

@bryk Could you tell me if my approach for this task is correct. I used directive for the log menu. I retrieve ReplicaSetDetails on logs link click. What do you think about it. Should I continue this way ?


aaa


Duplicate with line 99


Thanks for continuously looking at this, @contra. It’s highly appreciated :)

I can see/agree with your preference of loading plugins individually rather than using `gulp-load-plugins`.

What advantage do you see in having a file of *each task*, effectively having a lot of files with mostly just one function? Admittedly, files are cheap, but any practical advantages?


@ricardoruas88 Ricardo, estou desenvolvendo aqui para a carteira CSB e assim que homologar vou encaminhar o pull request.


ALL is not used in graph traversals. We decided to reserve it for future use, most likely for combination with operators such as `==`, `>=` etc.


Raise a ticket on CSS to suggest adding the 'hide readback' functionality. Would be trivial to implement.


The "alternative" I thought about does also not work:

```
v-model="{{ 'ageChild' + childNumber }}"
```

Error: ```attribute interpolation is not allowed in Vue.js directives and special attributes.```


oh, okay I don't get it ... :(


oh, in the comment it says I can change that, so I did


@gaearon why there is such code? It causes app react reconciliation every second (if there is no shouldUpdateComponent optimizations)
Any setInterval looks really hacky here..


Ahh ok! I didn't know that. I'll check dependency injection. ;)


test


thank you for the help.
hope we may find a possible solution in the future.


todo:remove this redundant ifdef


this too


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


Typo: where -> which


We do not. Maybe we should? @liggitt what do you think, worth it?


Was this meant to say something like "...of the `expose` option"?


`validate` on null item returns `Present(null)`? Hmm dubious :).


我好像记得这个字体是一个免费字体，建议使用其官方网址，或者干脆不贴网址


It's the same as donate.mozilla.org: you can change it as long as it's available on the other end (the form only takes you to the other website). Probably worth updating the note for clarity.


Thank you, good pull request! Also affects me, I certainly don't want to commit my credentials into repository - which would happen otherwise since the config file is committed. Tried it and it works as expected.


when the label is 1.0 return -score, and when is 0.0 return score. It works the opposite of LibLinearAlg


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


I see, you're round-tripping the patches - ignore previous comment.


이거 리소스에 들어있음 이거 없으면 날짜 입력 못함


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


wyglada na to ze to jest dobrze zrobione, jesli wpiszesz za krotka wiadomosc i nastepnie klikniesz gdzies poza elementem textarea, czy cos Ci sie wtedy wyswietla?


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


Thanks Eliot, I fixed the seed.

After doing a little reading, I _think_ the default scope is being run
every time. As far as I can tell, the old syntax

default_scope where(foo: 123)

Would have that problem. But
http://apidock.com/rails/ActiveRecord/Base/default_scope/class says: "You
can also use default_scope with a block, in order to have it lazily
evaluated."

I also tried putting a "puts Time.now.to_s" in the default_scope, and every
time I reload the page, it's displayed with a different time.

On Thu, Dec 3, 2015 at 6:30 AM, Eliot Sykes <notifications@github.com>
wrote:

> Perhaps it should be user_password_*reset*_email_text ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mhgoldman/course_manager/commit/b142a26b19db41f546f07f4f0029d6763c1d7cff#commitcomment-14759419>
> .
>


Done


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


parent is changed in the while loop: https://github.com/wout/svg.js/blob/2.2.3/src/element.js#L174
it loops through parent nodes and eventually it hits null.

you might be right about this not having to do with image loads.. but it happens only in such scenario in my application.


Use `Ordering.from().max()` instead, to avoid creating the list?


fajnie ze poznalas pattern i umiesz go wykorzystac :D tutaj mozesz jednak tez to zastapic `min="2"`, ale nie musisz tego poprawiac, to tez jest dobrze ;)


gg


SVMWithSGD performs bad from my experience. Better to use LBFGS http://spark.apache.org/docs/latest/mllib-optimization.html#limited-memory-bfgs-l-bfgs . For SVM you need to use HingeGradient instead, and off course SVMModel.


That's nice!


ogolenie jezeli imie lub email jest wprowadzony niepoprawnie to wtedy nie mozna wyslac wiadomosci, ale niestety przechodzi mi formularz nawet jezeli wpisze tylko jedna litere w textarea. Jak wpisze w textarea jedna litere i kilkne gdzies indziej to nie wyswietla sie komunikat jak w pozostalych.


Sorry for the confusion.  The simple specs, without function noise, where perfect here.,


dont cherry-pick
just by my mistake


Looks good! :+1:


should restore this


Ok


i wonder, should check aliases here as well?


Thx. That makes sense, yes!


I worked with one project that required to specify comment in following format: "#ISSUE_NUMBER comment"


@kostis

In the (unchanged) docs, nil() is not even mentioned. All that is stated is:

       | []                    %% nil

How would anyone know that nil() is a pre-defined type?

In the description of the abstract format,
https://github.com/uabboli/otp/commit/639f11e0ba018935e1cb225847ee1b57291193f2,
I've removed all shorthands. If [] is the defined type for the empty list, then we need to
add one more clause just for [], which seems a bit unnecessary.

I don't understand the relevance of your comment about strictness. I think one can use shorthands and still be strict (since the it's stated precisely what the shorthands mean).


The way i see it: it tries to find image parent element of SVG.Pattern type, but it doesn't find any and it raises an exception. I'm not familiar with svg.js internals.. The weird thing is that it still renders the image properly even though the error is thrown, so my initial guess is that it is due to redraw cycles.. but i might be wrong.


Thanks @jonrandahl - good spot! Feel free to send over a pull-request.


i ll test


I can confirm there is no such feature.
On Dec 3, 2015 5:23 PM, "Daniel Hickman" <notifications@github.com> wrote:

> @Anonymas <https://github.com/Anonymas> I couldn't find it anywhere.
> Maybe there is a share feature in the app that produces a link?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/materialos/CM12-Theme/commit/a0b96a17bb5fee03c091db0192afbb0271fb52bd#commitcomment-14759869>
> .
>


How could we have "failing tests"?  There are no assertions.  I'm ok with this messaging, it's just not what we are thinking about at that point.  This "run" can only verify that we don't have syntax issues.  This would be a great time to introduce `--verbose`, then the verification step would be "does the output describe what your system should do?"


Done


Boa


now really fixed - sorry, I thought I fixed it but didn't check


with 2 packages : controleurs and modeles


Jediny co, tak bych misto `onEdit` to pojmenoval neco jako `onTagClick`. Ale neres to.


Path to x-space/xcache-v2 currently hard coded. Will correct.


This commit is unrelated to the rest of the branch


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


"For specs to verify that our system is doing what we expect, we need to write a callback for each spec that asserts proper behavior."


No enunciado ele pede uma função que retorne o ASCII, e não o ASCII%10.
[A função de espalhamento "transformará"
a string passada por parâmetro em um
número inteiro. Este número será o
somatório dos códigos ASCII de cada caracter
da string;] SIC


HelloWorld::create?
initでしてるよね。


Yes. I had that at one point, but when the initial state is "collapsed", the top of the page scrolls up by the height of the panel. I have not been able to determine why yet. I'll spend some more time on it soon.


Agree with removal - irrelevant (and we do not want to create "premature optimization")


You can use {this} once more again at "If a card named Stomping Slabs was revealed this way". (I'm not exactly sure, not a dev on this project. :S)


jump.pngなんて画像コミットされてないけど？


bomb.pngも同じく。


奥对了，这个版本加入了回调函数。回掉函数主要与日志有关，下一步要在回调函数里加入日志编号这个参数（现在只有一个char*参数），具体的参数类型和含义在"LogReferenceCode.txt"里能找到。

还有就是Epic_Debugger.Create()里面的有关回调的东西还没写完，，，，还有就是Epic_Debugger.cpp文件里面的cprintf();彩色化控制台输出也不要了。。。。


そこまで大きな事をしなくてはいけないのかな？もう一度やる事を考えてみよう。


Thanks, this reinforces my view of where the likely issue is. I hope to be able to find time to work on it over the weekend.


Why Put nil? I thought deleted marker should call `b.Delete`.


Surely this needs to be a PX value. 1rem will default to 16px all the while otherwise.


@maxpeterson This is backwards incompatible. This release should have been `v3.0.0`.


Why not just use "while(true)"
What will be a real cpu-heater. ;)


It's MVCC, we can't delete it.


Once finished I guess here bashim will take relevance

https://ci.appveyor.com/project/albfan/ag-vim/build/1.0.1


We should at least log the exception.


@tu6619 I think (I assume this is what's required looking at the other solutions) the code needs to be in a file with a .js extension. The .md file (again I think) should contain the definition of the problem and any other notes like a readme. Really sorry, the point of the exercise is to learn to use github so this is a minor point.


@Ian-Foote is right.


Now that you have a failing test, make that one test pass.   This gives the students an example of a passing test before their "You do".  Recommend that they proceed likewise.  Add assertions to one test, make it pass.  Working with multiple failing tests is a "bad place".  It takes too long to get to green.  We need to share this with the students too.


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


Can you make a new tag, please ?


Done.


As I wrote, I can live with the change.  It was intentional to not mention `nil()` in this document (and define only `[]`) but perhaps it was not a good idea from a global perspective (i.e. if used by the abstract syntax, as you say).

The point about strictness is related to the above.  Now that `nil()` is defined in the document, and is the main way of referring to this type, the table entry `maybe_improper_list(byte() | binary() | iolist(), binary() | [])` seems weird.  For example, why don't we use the shorthand `<<_:_*8>>` for `binary()` also?  (well, not so much of a shorthand in this case, but you see the point...)  Or `1..255` for `byte()`?  I suggest that at least this entry is changed to `maybe_improper_list(byte() | binary() | iolist(), binary() | nil())`


If you want to provide hints for the student's You do portion.  I recommend you just share the expectation lines.  And place these hints in the lesson after you ask them to start the exercise.


Kan du ikke lige købe noget mere rolade.


asdf


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


As discussed,  this will be removed once the FVP VE memory map is obsolete.


Nivelacion


Please explain?


Why the change in style for defining components here?


Ouch! Looks good to me!


Classic GitHub
- one line
+ another


利驭  li yu


Why did you add SlidingPagination concrete class dependency? What if I want other thant SlidingPagination to render?


I figure this would have been avoided if we had used "newa(struct iovec, n*5+3)" here... Which is what we really should do, to avoid this confusion...


Tout y est ! 2 choses:
* enlève le commentaire avec : Created by ... (ca ne sert à rien et tu es l'auteur du commit)
* enlève les lignes en trop entre ta dernière méthode et le dernier crochet (une seule suffit)
* ajoute un retour à la ligne à la fin
* et avec ton IDE fait un coup de formatage


Removed with commit tbaudier/vv_src@9fd1972


Agree wot @edgework, this is really wrong


Good catch. May have to copy len+1 bytes to include the \0 at the end? Alternatively set dup[len] to \0.


Done


It's like an early christmas present... :christmas_tree: - Finally 0.1.0 beta! YAY :+1:


Done.


Perhaps rephrase like "Content-Type charset: must be empty or UTF-8, got: %s" ?


If we can ditch the artificial inclusion on request.el in core/lib I'm all for it, I had an egg and chicken issue so I included it but it is far from ideal, relying on pure url.el would be ideal.

Actually I wonder why I went straight to request.el for this :-)


Could be written as the following, to allow any dynamically added links via AJAX etc. to be tracked too. This also doesn't hold up main thread with loop execution.
```js
$(document).on('click', 'a[data-clicktrack="link"]', function() {
	$this = $(this);
	ga('send', 'event', $this.data('action'), $this.data('value'), $this.data('label'));
});
```


Sorry, it's a bad habit I got into in the early days.  There didn't used to be a reasonable way to run a single test, so I got in the habit of adding the 'ignore_' prefix to all the other tests in a file when I need to do that and it can be handy when you need to run a small subset of tests.  Fortunately, there's a better way now for single tests:

rake test <file-path> <test>

I just have to remember to use it.

I still haven't found a great way to run a handful of tests in a particular order other than copying them to a temporary file (which is probably better than using the 'ignore_' prefix).  However, I'm hoping the need for that will diminish over time as we get rid of interdependencies between tests.

-Nathan

> On Dec 2, 2015, at 9:37 PM, Joseph D. Cohen <notifications@github.com> wrote:
>
> Instead of changing test names to ignore them, you make the first line of test
>
> skip(msg = nil, bt = caller)
> msg can be an explanation why the test is being skipped.
> See http://ruby-doc.org/stdlib-2.0.0/libdoc/minitest/rdoc/MiniTest/Assertions.html#method-i-skip <http://ruby-doc.org/stdlib-2.0.0/libdoc/minitest/rdoc/MiniTest/Assertions.html#method-i-skip>
> —
> Reply to this email directly or view it on GitHub <https://github.com/MushroomObserver/mushroom-observer/commit/b1c529a7a1e4e8d0b38f1349a7fa4460c129d7bf#commitcomment-14752089>.
>

_________________________
Nathan Wilson
Director of Software Development | MassChallenge, Inc.
1.888.782.7820 x 712 | nathan <mailto:nathan@masschallenge.org>@masschallenge.org <mailto:nathan@masschallenge.org>
Cell: 818.919.5947 | Skype: nathanjwilson
21 Drydock Ave., 6th Floor | Boston, MA 02210
@MassChallenge <http://twitter.com/masschallenge> | MassChallenge on Facebook <https://www.facebook.com/masschallenge>

We help entrepreneurs win!
www.masschallenge.org <http://www.masschallenge.org/>

Congrats to the 2015 MassChallenge Semi-Finalists! <http://masschallenge.org/startups/2015/semifinalists>
Global Finalists Announced May 20


你太出色了 佩服你！！！！


Hmm, I think it would be nicer to solve this with passing a "struct sockaddr" i.e. do an:

```
int sendto_one_fd(int transport_fd, int fd, int flags, const struct sockaddr *sa, socklen_t);
```

That would nicely map to the relationship of send() and sendto()


I didn't know :) feel free to update it


"É claro que deve nos mostrar suas habilidades" para "É claro que deve mostrar suas habilidades"
"Derrotar #pending Desert Chupacabras ou Zinutes serão suficientes para se demonstrar" para "Derrotar #pending Desert Chupacabras ou Zinutes será suficiente."


I recant.  I read through the whole lesson - outside of the diff - again.  I think I am being to pedantic about a process that will support them as they move forward.  The "process" in this lesson isn't correct, but I can agree that it is probably a good way for them to learn how to write assertions.  What you have is acceptable.  We should just be clear, that they should be moving toward Red/Green/Refactor (write a test, see it fail, write the code to make it pass, then refactor).  I'm still al little worried about creating bad habits - they live with us for a long time.


If you say so. Ok, it's not a big deal.
Be gentle please.


Il faudrait rajouter un filtre pour n'afficher que les valeurs des indices sélectionnés. Je le ferai, si tu veux.


Globalement, c'est cool! Je suis sontent de voir que ça avance. Merci Yann.


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


OK - your exact code, browser and operating system please. I can't
reproduce the error

Milovan Zogovic <notifications@github.com> schrieb am Do., 03.12.2015,
13:17:

> The way i see it: it tries to find image parent element of SVG.Pattern
> type, but it doesn't find any and it raises an exception. I'm not familiar
> with svg.js internals.. The weird thing is that it still renders the image
> properly even though the error is thrown, so my initial guess is that it is
> due to redraw cycles.. but i might be wrong.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wout/svg.js/commit/74bb0144ccda4bf3cc4f34b497e696a0a7eb99ec#commitcomment-14760275>
> .
>


Hopefully the number of toasts would never get that high that this could be optimised…


When either entering an incorrect password or leaving either field blank (basically, when you get an error message) the panel jumps back to the user list. Should stay on the password edit page really.


@satya164 the one in `ReactContextBaseJavaModule` is `protected` which means that it can be accessed by subclasses.


PEP8 recommends to have the django imports before the local app imports.


You are right; I overlooked that… In that case, the fix is absolutely valid.

> By the way, is there a simple command to reset only the bold attribute without changing the font color?

Not that I am aware of.


@foghina Hmm... Might be some error on my part. Will check.


Concratulations! Looking forward to use the open beta ;)


Banking application code sir??


thanks for adding python 3 support


Ugh, really? Any idea why? Is it related to specific GTK versions? It's really just a regular menu with menuitems.


I guess rule 201.4b does not count here, because it's not a reference to the card/object itself but all cards with that name. So to use the name itself in the rule text is correct here.
Also to use the name itself in the code for the check is the correct way to compare.


More info on this:
- while I was able to ping MELPA and GELPA, I was not able to ping orgmode.org
- initially I wanted to use "HEAD" verb to not download the response body but in the end request.el does not support it


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


neither do i, we don't even have its pool in the osdmap!


Nice. Thanks a lot.  I was working on something like this today too but you beat me to it.


Czy to zadziała dla starszych wersji perla?


No, actually, what I mean is, when a person writes in a custom .Server through the UI, we set SkipSRVLookup. It's only when it is set through direct manipulation of the config file, or imported from earlier, where we need the non-strict behavior.


In the future, we should be strict about what should invoke `store#dispatch`. I'd prefer there to be higher-level, more human-readable APIs that do this – maybe statics on React components would be a good first attempt, e.g. `getStore().dispatch(showToast('Ready to work offline'))` might become `Toasts.show('Ready to work offline')`.


The above file should not be re-added to the code base.  Instead, a symlink should be created from this tool's directory to the single file in the directory above (i.e., ln -s ../icqsol_macros.xml .).


Ok


The problem with Google is that in this case, the server setting is correct and valid, but we can't really make that distinction most of the other times.


If you want you can review this one:
https://github.com/qgis/QGIS-Website/pull/297


done


Merged to master.

On Thu, Dec 3, 2015 at 7:09 AM, Martin Goldman <martin@mgoldman.com> wrote:

> Thanks Eliot, I fixed the seed.
>
> After doing a little reading, I _think_ the default scope is being run
> every time. As far as I can tell, the old syntax
>
> default_scope where(foo: 123)
>
> Would have that problem. But
> http://apidock.com/rails/ActiveRecord/Base/default_scope/class says: "You
> can also use default_scope with a block, in order to have it lazily
> evaluated."
>
> I also tried putting a "puts Time.now.to_s" in the default_scope, and
> every time I reload the page, it's displayed with a different time.
>
>
> On Thu, Dec 3, 2015 at 6:30 AM, Eliot Sykes <notifications@github.com>
> wrote:
>
>> Perhaps it should be user_password_*reset*_email_text ?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/mhgoldman/course_manager/commit/b142a26b19db41f546f07f4f0029d6763c1d7cff#commitcomment-14759419>
>> .
>>
>
>


We need to make sure that we are feeding our human need for progress and give ample room for steering as forward progress informs us about our system.


How about "All API requests will be made via the network first, falling back to the client database when the client is offline"?


FWIW as discussed on the list I'm rather against this. But in any case:

1. "currently unreleased" will soon be an outdated comment.
2. I don't think mentioning logs here brings anything.
3. This signature is incompatible with Logs.


Ok let's see


Each of these code lines should be a single line.  It will not pass flake8, but will be more readable.  The only exceptions we allow for flake8 is "Line too long" exceptions.  Flake8 still assumes terminal windows are 80 characters, which is no longer true.


Hmm one idea, why don't we insert `autotest


Make the above a single line for better readability.


Lucky issue #1!


Single line.


I've addressed this in 742f1a75d1556796170e6c1dc6b9aaeb39573283.


Done


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


python $__tool_directory__/icqsol_refine_shape.py?  Should this be python $__tool_directory__/icqsol_rotate_shape.py?


Hopefully!


This matches URLs from others sites (breaking `test_no_duplicates`).
Removing the last '|':
```
|iplayer(?:/[^/]+)?/(?:episode/|playlist/))|music/clips[/#])|)
                                                            ^
```
seems to fix it. Could you check if it breaks something?


Resolves issue #27


Addressed in c616a900206908efb7701176a9dc72a4a4c2746a.


input_dataset_type is not in your command line - see above...


Because I'm dumb, this should be an stateless component. I'll fix it


Fix proposed in https://github.com/modxcms/revolution/pull/12798


The above file should not be re-added to the code base. Instead, a symlink should be created from this tool's directory to the single file in the directory above (i.e., ln -s ../icqsol_utils.py .).


One should also add that having such an interface as it stands will encourage people to pass the same functor argument to different subcomponents which will prevent source based control over log reporting.


谢谢提醒 : )


explicit path here i don't believe works in windows.


On 12/03/2015 02:00 AM, SVAKSHA wrote:
> @brombo : galgebra
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/svaksha/pythonidae/commit/caeb004e01953d1ba1c94b1a20d22ee0948d5c3a
>
Downloaded you repository.  You need a batch file (or python script) to
automatically convert all of the .md files to .html files and an
index.md file.  Perhaps setup.py?  Other than that it looks very useful.


@lmr might I ask you what the import should do? It's an empty `__init__` and it's not used. Can't the import be simply removed?


I think I am now confused heavily. As we write single bytes - and as layout might be a problem depending on the type / architecture wouldn't then a simple memset also work? Or is the representation as unsigned char that memset wants again problematic?


```javascript
1 || 2 = 1
1 | 2 = 3
_hasContentType use bitwise AND.


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


no i jest problem, bo dla 5.14.2 w debianie 7 "use encoding "utf-8"" krzyczy błędem :/


Change the above to use the standard call used in all other CSG tools:
output_format, output_file_type = icqsol_utils.get_format_and_type(args.output_vtk_type)


Single line.


Oh, that how it should be... thank you!


This may be caused by unusable capacity calculation. Namely, the driver constantly monitors the current drawn from battery and stores the maximum current. The value is stored until you reboot or charge the battery.

The driver uses maximum current value to calculate the value of unusable capacity. If your phone was heavily loaded even for a minute, then the value of unusable capacity (capacity_info.uuc) was probably quite high. You can check the actual value of unusable capacity in sys/kernel/debug/bcmpmu59xxx/fuelgauge. The variable name is uuc.

You can observe similar behavior when you reboot. Before reboot the uuc is usually small, around 5 to 6 per cent on my phone. When I reboot, it suddenly jumps to 10 per cent, probably because of the jump in max_discharge_current. Maybe we should use average current from last few measurements rather than maximum current from discharge.


You'll need these:
    --input_file_format_and_type $input.ext
    --input_dataset_type $input.metadata.dataset_type


Главное правило это не пытаться делать картинку больше, чем она есть. Нужно брать большую и подгонять под размер. Тогда никакой фотошоп не нужен. А ты берешь картинку шириной 555 и растягиваешь до 627.


better to use `snapshot_name = snapshot_num.to_s` which do exactly same thing


Enforcing & requiring a single dataset in a query doesn't make sense in two ways:

* objects may not be members of any ```Dataset``` (e.g. a number of data repositories don't have this concept
    - so better ```union (null, string) datasetId = null```
* one may want to query any number of datasets
    - so better ```array<string> datasetIds = []```

These are 2 separate issues, both being solved by keeping them as array (with OR usage). Where does this change stem from?


input_file_format_and_type is not in your command line - see above...


Now to build up this code slightly


you can also use case statement here with
```
case status
when /created/
  content << Left(Label(_("New file was created.")))
when /removed/
  ....
```


|Så ser jeg at utgivelsen er tilgjengelig i katalogen
Hva vil katalogen si her? søk, patron-client, koha?


@eldy I use chrome on windows.
@eldy @rdoursenaud I think we should do a quick 3.8 fix with toFixed and find find a volunteer to cleanup javascript. Think this file needs some cleanup in develop branch, there are two $().ready's, in one file. We can create a lib with a price2numjs function build upon the MDN suggestion.


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


Sad to know this, but I guess there can still be nightlies even without a maintainer? I guess the build process is automated.


I understand the x52cli using libUSB, and not the kernel driver.

I like x52_test, as this tests all the buttons and lights.  Currently I seemingly have one LED button ("D") on the throttle controller malfunctioning.  The x52_test cycles through the colors, including the missing yellow LED and only displays green.  During movement of the throttle controller, I can likely see one of the wires leading to this LED likely incurring tension at some point, loosening a connection to the LED yellow contact.  (Or the LED is just, overall, going bad.)  I'm not a fan of tossing out useful scripts and code, unlike some people.  I would suggest renaming the utility to x52test.  You spent some time coding the utility, with the fact that if it were never coded, it might not be missed.

Also, you have something upstream that is proven to work, besides a simple user (maybe unreliably) scripting something on their own via x52cli.  However, I can also provide good arguments at times for keeping something that should be removed!  So, it's your project, and it's your choice.  (I can understand either of your choices, if you so choose to remove the tool.)


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


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


why not `s["post_num"] != 0`


`pre != - && index != -1`


trailing white spaces added


perchè non un "Fork me on Github"?

https://github.com/blog/273-github-ribbons


Thanks, will do.


> But should't recent objects include *all* objects from a combined object? Not just the last one?

It will include them all, since hitting `,` adds them to Recent Objects one at a time. But they will be listed individually.

> On the other hand... I guess it depends on the use case and the description of the combined object (usually not very good), but I'd have thought that if I did something with "those three images on the Desktop" that it might be useful in the future to keep that combined object around. But am I now confusing history and recent objects - is this what history is for?

Of course, this is debatable. The direction I’m going is: Recent Objects lists any object you’ve recently done something with individually. History lists the literal selections you’ve previously done something with.

I don’t feel strongly about it, but I do think that’s the right approach. One argument is that Recent Objects, being a catalog source, is meant to be searchable. Would you ever search for something like `2 [MPEG-4 movie]  in "Downloads"`? I wouldn’t. But I might think “I just had that (individual) thing selected, so let me get it again”.


The name `ATTR_MEMORY_*` is a bit confusing here because all these #defines operate on memory attributes. I think `ATTR_NORMAL_*` would be a better name, so that we have *normal* memory attributes as opposed to *device* memory ones. So I think:
*  `ATTR_MEMORY_NC_INDEX` should be renamed into `ATTR_NORMAL_NC_INDEX`
*  `ATTR_MEMORY_NC` should be renamed into `ATTR_NORMAL_NC`


this fuelgauge driver is more complex than I though it would be, in first place I though it only calculate battery percentage based on voltage value, and I'm not really understand how those thing working, keep your work and tell me when it's ready to test again


Proposed fix in https://github.com/modxcms/revolution/pull/12799


I wonder whether it's worth mentioning that all "state variable" are passed to outcome templates by default i.e. that's why the references to `calculator` work in the example show.


Sorry, the title makes no sense - those changes happened in parent commit a19936c34a941690374541b4e0ce6c1d69315aeb


This enum is missing the counterpart value of MT_MEMORY_NC. Just like we have MT_DEVICE/MT_MEMORY, MT_RO/MT_RW and MT_SECURE/MT_NS, we should have both "non-cacheable normal memory"/"cacheable normal memory".

Also, the comment above this enum says:
```
These are organised so that a clear bit gives a more restrictive mapping than a set bit
```
so to stay aligned with this convention, the "non-cacheable normal memory" enum value should be 0 (rather than 1), as non-cacheable is more restrictive than cacheable.


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


I wonder whether it's worth mentioning that all "state variable" are passed to landing page templates by default.


I wonder whether it's worth mentioning that all "state variable" are passed to question templates by default (and maybe include an example).


HAHAHAHAHAHAHAHHAHAHAH


There are better ways than restarting everything just because of a small update, GTWgui also uses the shitty storage solution based on a "settings" account. What we should do is to implement a dxGUI system where all GUI's are in the same resource, calls to events would still be managed by their original resources. This way we could compress all images used by the game mode and reduce the download size further.


May be it makes sense to move status code values to constants?


Great. I guess this was generated by an automated tool and you with some pylint disable addition, right?


Why ? This does not make any sense to have a different behavior, even if I click on myself.


No reason, except that my text editor automatically adds it on save. Probably no harm either. There's this: http://stackoverflow.com/a/729795. But I'm not sure how relevant it is to our situation here.  Is there something in the style guide about not ending with a new line? Is it something that deserves our attention? (honest questions; not being sarcastic.)


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


thanks, @agcolom !


It might also be worth mentioning here that all "state variables" are passed to node templates as local "assigns".


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


Joiner could be a constant


this set could be a constant


this property and the one below could be a constant


Ideally I think there would also be at least a brief summary of the motivation for the change in the PR itself or one of its commits.


Merge sha 6a04c491 is stale.


merging basho/riak_pb/feature/az/future-proof-tslistkeysreq = 7344d743 into borshop-integration-168-feature/az/future-proof-tslistkeysreq


basho/riak_pb/feature/az/future-proof-tslistkeysreq = 7344d743 merged ok, testing candidate = 11a0715c


well that'd be also hackish... Hmm... this is probably the best we can do with the given structure...


Yess,finally!


It might be worth saying that such screenshots should highlight the changes rather than just being screenshots of the whole page before and after i.e. some kind of visual "diff".


I tried without the --sudo, but it turns out that removing it causes some problems, because some modules such as Sysctl do not work, as sbin is not in the path.


@Jorch95 Esto en realidad iria adentro del metodo `#run()`, dentro del else, despues de que se cancela el timer :+1:


Is this a fix for React native looking for NPM modules in /users/node_modules ?


shouldn't it rather be named `debtProperties` or `debtConfiguration`
the fact that it's a rule does not matter in test methods, seeing `debtRule.setDevelopmentCost(2)` feels weird since we are effectively configuring the SQ server, not the rule itself


@Jorch95 Yo no quiero perder :-1: jajaj


@nroakton  I don't like these Ac* prefixes as it implies AutoCAD even though they are BricsCAD refs also.  We can either scrap the Ac prefix (Rx, Trx, etc) or perhaps Mc (for micad ;-)


@yisak apparently you have removed some files that were needed for the existing test suite: https://travis-ci.org/statkraft/shyft/builds/94626221.  Could you please restore them?


> I do not see much point in defining/including nil() instead of [],

I think that perhaps it's better to keep your original text, that is, defining [].

If I understand you correctly, you see no point in mentioning
nil()? Among the pre-defined types recognized by
erl_internal:is_type(), nil() is the only type that is not
mentioned in the refman.


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


whoops. Forgot to use filename from toml.config.


nice idea to add a rule :)


And as such it [does not include the ActiveRecord Railtie in application.rb](https://github.com/alphagov/smart-answers/blob/4eb1b80a698e6835e745c4ad1954a3892e929b64/config/application.rb#L3)


Better if you `git squash` this commit and `force push` :wink:


Existem casos que não existe nem elocation nem páginas.

Neste caso não deve mostrar nada.


Hmm, the comment says `it should not have authHeader`, but then the test expects the header to equal to `accessTokenId`. A typo?


This comment seems wrong.


Please add an empty line to visually delimit individual test cases.


It would be nice to make these links consistent e.g. "Developing with the GDS development VM" & "Developing without the GDS development VM".


log.debug?  I thought log.debugf was (sadly) being removed.


The testUrlBlockingRule test failed without this.


TODO for myself: still needs fixing -->  `/^\w{3,20}$/`


This revision is clean.


Overridden theme function "theme_media_gallery_media_item_lightbox" of contrib module media_gallery in template.php


It's not very pretty to raise an AssertionError when the settings file can't be found.
A FileNotFoundError is more appropriate.


obs - pg.a bøg i salt så kan vi ikke ha æøå i overview/init.sls :(


@dzafman if we will use (encoded) `hobject_t` as the cookie, we will need to **iterate** in the kvdb space. and we are generating the *key* using "DBObjectMap::user_prefix()" so it's basically a good enough auto-incrementing key gen.

and when we get the next n inconsistent objects, after we get the seq of the first inserted inconsistent object using the mapping from `HOBJECT_TO_SEQ` (see `DBObjectMap::_lookup_map_header()`), it's very fast to get the next n objects.


ok


yep


yop


It only subtracks hunger if the player is in Creative mode or spectator?


This timespan can be reduced to speed the test, but could start to cause transient failures / indeterminacy, below about 0.1.


Yes, I was sure you'd like it !


j'ai essayé sans succès


c'est vrai que ce n'est pas super clair


Je n'ai pas encore réfléchis à ça je t'avou que la dessus j'ai fait un truc "à l'arrache" pour que le tableau s'affiche, mais je voulais quand meme utiliser angular pour apprendre a l'utiliser. Mais oui les données devront être les mêmes dans le tableau et dans le graph


So anyway, the schema should be as follows
job -> [ optional container] -> [ docker iff = container/type docker  ]
                                         \->  [ optional volumes ]


oui


Should have a comma, not semicolon


Merci :) Ca ma permis de mieux comprendre comment fonctionne angular même si je suis pas encore super au point.


Hi Greg,

Do you want me to make the changes? Or are you working on this?

--Alex

On Thu, Dec 3, 2015 at 5:46 AM, Greg Von Kuster <notifications@github.com>
wrote:

> The above file should not be re-added to the code base. Instead, a symlink
> should be created from this tool's directory to the single file in the
> directory above (i.e., ln -s ../icqsol_macros.xml .).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14760856>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


this is fine although I'm also fine with using the isDebugEnabled wrapper only in frequently-executed code blocks.  Catch blocks are hopefully executed only in rare cases so it may not be worth the extra verbosity here or in other infrequently called code.


You're right => debtConfiguration is great. And the rule could also be rename DebtConfigurationRule.


Fixed


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


Where are the number of columns, or column names, set?

I added `ActiveMaster.ReferencedBy[i].ElementByName['Cell'].EditValue` above Signature, but now Cell appears in the Signature column, Signature appears in the File column, and the File names are missing.