obvious cheaters!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

UPDATE:

ADDING GENERAL HEX


It just hit me too. Fortunately it's simple to update the code to make it work again.


About this message - I think ', altogether with the possibility of manual interaction' could be safely removed. It doesn't seem to fill a lot of purpose, or quench any pressing doubts a user might have ('OMG, can I interact manually? I *need* to know, otherwise I'm going insane!')


If we want to make this opensource, we should look for things like this constantly

I'm not even sure if the migration scripts should be in this repo, since our previous review solution is proprietary so we are the only possible users – let's just remove it once we're done, ok?


Ok.


#1639 has been closed


Note: captcha is not supported.


Frownies.


Yes, in fact, I already moved it but haven't pushed the latest changes yet. The reason why I originally added it: the test strings for the test files specified in InfolisBaseTest are not helpful for testing the Linker class. I did not want to add the new strings to the InfolisBaseTest class as they are not needed by tests in other packages. I thought of creating separate BaseTest classes inside of different packages to keep all shared specifications there. However, separate BaseTest classes are useful only if tests in one package share items and share different items than tests in other packages. I initially thought this would be the case but I checked and it's indeed not.

Btw, I think we should move the InfolisBaseTest class to another package, it does not really belong to luceneIndexing.


and you really should fix that, or we might get incorrect data after migrating


qPrintable(o->objectName())


okay.  i went in a fixed it.  i cloned it to a new directory just to make
sure it compiled.

let me know if it works on your machine.

btw: I'm not sure what happened on github.  Somehow I accidentally created
a separate submodule or something.  weird.

Jim

On Wed, Jul 1, 2015 at 7:04 AM, Paolo Oliveira <notifications@github.com>
wrote:

> Haha, relax, take some more beers. And never stop working late night with
> beers, world is better this way(and software quality increases).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jvburnes/node9/commit/0ca97e87e38515b6a2317d5a0548b8f28891cee4#commitcomment-11952766>
> .
>


Yes.

Whenever a context is marked as _dirty_, the context is closed, but it is not immediately recreated.

The context will only be recreated if a test is actually configured to use the context.

FYI: the context gets (re)created in the `DependencyInjectionTestExecutionListener` which is configured _after_ the `DirtiesContextTestExecutionListener`.


Hi, one question, QlikView Desktop is necesary to run this tool?


Shoot, sorry I missed that completely. Thanks for catching that one.


Looking through all views with calls to `base_url()` I found another one, in `application/views/manage/attribute_policy_detail_view.php`
```php
$link_policy = anchor(base_url() . "/manage/attributepolicy/globals/" . $idp_id, '' . lang('backtodefaultlist') . '');
```


ანანო, ამ კომიტზე გეუბნებოდი დღეს. მეგონა რაღაცაში დაგჭირდა და იმიტომ გადააკეთე, ბაგიანი კოდია ეს არადა, დღეს შევამჩნიე, stmt.executeQuery(); ვერ იმუშავებს, ადრე გადავაკეთე, უცნაურია საიდან მოხვდა აქ. მერჯების ამბავი იყო ალბათ ხო?


... Why?


ok


good job on posting your password(s) in an public repo.


Style: `QObject *`


However, since you've asked... your question makes me think that there in fact might be an ordering issue with regard to dependency injection and dirtying of contexts before test methods.

I'll definitely put some more thought into this. So thanks for asking the question! ;)


`++iteration;`


Test


you can unwatch the repo if the notifications bother you – and thanks for noticing this problem mate :)


Rest LGTM


thank peter, will fix it


lol


the CAVESIZE is not used as recursion depth https://github.com/slemonide/gen/blob/master/init.lua#L15


According to the documentation, FLUSH LOGS is not written to the binary log

http://dev.mysql.com/doc/refman/5.6/en/flush.html

I believe by now all FLUSH commands that do not cause tables to be locked and are replicated under async replication are also replicated under Galera. Commands which cause tables to be locked are not replicated, as Galera does not support the UNLOCK TABLE statement required to unlock them.


Even more surprising:
```
biskup@cathedral:~/go/src/github.com/GoogleCloudPlatform/kubernetes$ cat a.sh
#!/bin/bash

ab="a/b"
IFS='/' read a b  <<< ${ab}
echo "a=$a, b=$b ab=$ab"
biskup@cathedral:~/go/src/github.com/GoogleCloudPlatform/kubernetes$ ./a.sh
a=a, b=b ab=a/b
biskup@cathedral:~/go/src/github.com/GoogleCloudPlatform/kubernetes$ ab="a/b";IFS='/';read a b  <<< ${ab};echo "a=$a, b=$b ab=$ab"
a=a b, b= ab=a/b
```


READMEの見出しをかっこよくしました


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


Adding gitter link to readme


:sob:


Why is the "title" property being introduced?  none of the other assertions in this test use it


함수들은 보통(대개) main함수 이전에는 call만 해주고,
main함수 이후에 본 함수들을 나열해주는 것을 선호합니다.


@mkasztelnik Your editor alters its behavior depending on the contents of `rubocop.yml`? :confused:

You asked me to do something about Hound, so I did. If you want me to go through all of Atmosphere code and trim all lines to match the 80-character requirement then I'll see you next week. :grin:


Was this submitted to MS? I couldn't find it so I created my own report:
https://connect.microsoft.com/IE/feedback/details/1494442

We should be good Web citizens, though, and report such issues to browser vendors. If they don't know about it, they can't fix it!


I will be dedicating my day to have your coding and my coding updated. Int has been a bit of a hassle but the outcome will be well worth it...

Hey so I have downloaded the bootstrap.js as you mentioned, now JQuerry script has many downloads which one in specific?


84691154_AutomateBuildingOfDocs (lefticus) - Win64-Windows-7-VisualStudio-12: OK (1682 of 1682 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)


First of all, it's the encrypted password for artifactory, used for pushing artifacts

Second, it doesn't even matter since the repo is not public and is only local host


bootgitstrap or bootstrap ?


you can use `nme.EMPTY`.


Nice!


@shankardevy Even if we knew the correct auth type currently we would need to pass it to the `HTTPClient`, so the current implementation isn't ideal. I'd suggest changing the `get` function to `get(url, credentials, auth_type)` so guessing the auth_type stops being the responsibility of the `HTTPClient`. We would just need to pass that information from the `SnapshotController` - and there we can make checks for vendors (temporarily).

In other words, I'm suggesting to implement the `HTTPClient` module as if we had all the correct information, and move the hacky fallback outside so it will be easily replaced.


eeerk, not at all! thx for noticing it


I dont get the link between #15 and this commit? #15 is about email stuff


This looks good in principle.  I'm worried that master or Kilo might have removed a root_helper parameter in various places (e.g. IptablesManager constructor), and that you might need to add that back - but that should be discovered by testing.

You are planning to test as well, right?


<> is a SQL operator and will cause strange things to happen on NoSQL databases. In summary this is not database-agnostic. I'd vote for merging in https://github.com/apotonick/reform/pull/224 because it has a ORM base class that helps with implementing uniqueness and other validators for other ORMs. So agnosticism through different adapters.


This is confusing.

* `attr_pk` is not a helpful name. Isn't it really `attribute_code`?

* You are executing two queries per attribute which is two too much.


You're welcome, and thank you very much for your answer :)

I've asked this question because actually I am facing a little problem with this at the moment. I have a test class annotated as follows:
`@TestExecutionListeners({CustomDirtiesContextTestExecutionListener.class})`
`@SpringApplicationConfiguration(classes = Application.class, initializers = TestProfileInitializer.class)`
My `CustomDirtiesContextTestExecutionListener` extends `DirtiesContextTestExecutionListener` and implements `beforeTestClass` method in exactly the same way as `DirtiesContextTestExecutionListener`'s
`afterTestClass` does. However during the test execution the context is only being closed without a new context being created.

I am afraid that there might be no point in closing a context before the test without providing it a new one. What do you think?


fixed in https://github.com/mirage/irmin/pull/216


I don't think this went through correctly. I'm only showing one changed file, and the change I think breaks the method. The reason I had left do +Max-1 is because Java's mod operator doesn't do what you're expecting. If you Start with 0 and go to -1 and then modulus that by the maximum, you're going to get a negative number.


Isn't this a `ModelMultipleChoiceField` in disguise?


Falling to sleep and never awake 🙏


I realise I haven't implemented that yet, but I don't think it's so crazy to render item and choice labels together – why would it be? At the moment we're actually rendering choice labels before they're being used, that's not so smart.

1. decide which items will be on the page
2. collect all dynamic labels (choice or item) for those items
3. bulk opencpu
4. consumers collect

seems sane to me.


ho cambiato questo tipo di sincronizzazione dei file tra macchina fisica e macchina virtuale. Velocizza molto l'esecuzione della sincronizzazione e quindi velocizza molto l'utilizzo stesso della macchina virtuale


gaah.. Thanks @someara


Hi Szymon Nowak,
   Thanks your kind response. I sort out that issue.

Regards
Hassan

On Wed, Jul 1, 2015 at 1:30 PM, Philip <notifications@github.com> wrote:

> @Hassan829 <https://github.com/Hassan829> See
> http://php.net/manual/en/function.file-put-contents.php.
>
> file_put_contents($path_to_file, $data_uri);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/szimek/signature_pad/commit/b9082722e78e6fbdee7a35a7244bc30c341fa5cd#commitcomment-11948228>
> .
>


Keep on !~


Actually, the stop action was missing a while ago in the service file. I thought I had fixed that, though.

Do we know for sure that the check for ready is returning immediately?


If you write `model = self.model` then you will save yourself from the trouble of raising an exception here. Defensive programming is nice but this is too defensive and raises an incorrect exception type.


Sorry, I'm not really confident with git, I had to delete the whole opengapps folder and init again because I was not able to revert my changes.


Hello,
maximum value of len is 0xffff, so data[0x10000] will not be overwriten. So, possible stack overflow is fixed.

Checking against len < 2 will not help much against corrupted/random data (try to check case with tag == 0xffc0 and len == 2)


fellow라는 string에 대한 배열(char에 대한 더블 포인터)을 만든 후 각각의 배열에 나눠담고
그것을 이용하여 분대원의 정보가 들어있는 구조체에 다시 assign 해주는 시도가 상당히 좋았습니다.

그렇지만 만약 구조체를 선언함에 있어서 name을 char[]에 대한 포인터로 선언했더라면 더 좋지 않았을까요?
이중으로 메모리를 잡을 필요도, assign을 두 번 해줄 필요도 없어질테니까요.


@ilanaseid fyi


Em relação ao atleta contratado, preferir utilizar o cpf como melhor restrição.


Guess what I don't like about this commit message? :-)


@multigcs  I just saw this typo.


Thank you for this! :+1:


fixed in next commit


The `DependencyInjectionTestExecutionListener` is responsible for loading the context and then injecting dependencies from the context into the test instance.

So you have to make sure that the `TestExecutionListeners` are ordered properly.

In your example, it appears that the standard ordering mechanism should apply. See [Ordering TestExecutionListeners](http://docs.spring.io/spring/docs/current/spring-framework-reference/html/testing.html#testcontext-tel-config-ordering) in the reference manual for details.


Woohoo!  I like that it's a giant ass RSS icon.


@TSC21 defining a frame which has Y north as conventional is absurd. The whole NED / ENU notation is specifying accurately which axis is pointing where, which is relevant for the zero yaw rotation.


Is this something I can use on my OPO which is an SD801 and 3.4 based kernel? I know sometimes there are dependencies and there may be some for some upstream Linux changes. It looks fine to me but I wanted to check. Thanks brother.


I'm not sure this class is useful. You should use a `ModelMultipleChoiceField` that requires a queryset which in turn requires a model. You should then only override the queryset in `__init__()` which leaves `delete()` as the only remotely reusable method.


Was there ever a PR for this? Looks like this didn't get pulled into Kilo and it's causing merge conflicts for at least one of my backports.


By the way, any time a `TestExecutionListener` invokes the `getApplicationContext()` method on the `TestContext`, a new context will be loaded if it's not already in the `ContextCache`.


Please make each `for` a separate view. There is way too much logic here for a single function.


Yes, I got this to work. Thanks a lot! :)


Evitar de subir o development.log para o repositório.


oh....... please share the solution!


@rldhont desperate to troubleshoot I have tested on qgis server on Windows (yikes) and as a matter of fact it works (but the request has to be done twice because the first time it crashes, only happens when map=none). Now I just need to understand why on Ubuntu server does not. Ideas?


Because of this test https://github.com/JuliaLang/julia/blob/fe7203e8b747a7caf472b59e5fa41f6cc86b1489/appveyor.yml#L53

Could've gotten rid of that line, but it only takes a couple of seconds and does check that things keep working without a precompiled sysimg. The quoting was oddly picky though.


fflush함수가 scanf함수의 약점인 '남아있는 stream 버퍼'를 비워주는 역할을 잘 해주었습니다.


fixed: 58909dacd9cfe2ed69b872cd6ef9710191d7c342


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.427.94ce61c40440c1a08e42d56dc316d48ca0bc174f](http://build.palaso.org/viewLog.html?buildId=59133&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3306, ignored: 124 Build time: 00:10:03


you're right

there was an issue in my version with #15 .. because there it's possible to also add get parameters to the redirect, and if a date field is added as a get parameter it will break


logs subiram nos commits anteriores, mas sem problema.


This is still experimental, google committed on shamu m-preview branch and later reverted.
Also, only framework won't help unless you have users for that framework.
I would suggest to wait for Google to release this for hammerhead.


If you like, you can keep the benefits of the old `options = options || {}` line by changing the constructor to use a default parameter value:
```js
constructor({outputPath, previousBuilder} = {}) {
```


I simply added the `DependencyInjectionTestExecutionListener` to `@TestExecutionListeners({ListingServiceITDirtiesContextTestExecutionListener.class, DependencyInjectionTestExecutionListener.class})` :)


The docs have actually been changed to encourage a more explicit way of using rpi config:

    kalite start settings=kalite.project.settings.raspberry_pi

Or change your .kalite/settings.py

It will explain here, once docs are rebuilt...
http://field.learningequality.org/develop/docs/usermanual/userman_admin.html#configuration-settings

@MCGallaspy could you update the docs build?


You need to sanitize URLs that come from the browser.


This makes the SparkIntervalTimer work on the Photon, right?

It would be great if you made a pull request with your changes so that the library would work for the Photon from the web IDE.


> @TSC21 defining a frame which has Y north as conventional is absurd.

Well I found out this to be present in many docs, including REP 145. Doesn't make sense to me also, so I could put brackets on conventional. Anyhow, the current implementation (and consequent patch fix) has the conversion I stated in https://github.com/mavlink/mavros/commit/6628ce554e30049d00e2716a1a3029fbe9b3cf5f#commitcomment-11950471.


Procurar fazer os steps de teste que o cucumber cria, mas imagino que dessa forma, se ela tiver puxando os dados das próprias features do cucumber.


Ah, I see you already know about that trick :)


Is this more readable?


Sorry-misunderstood. Please move to run and build

On Wed, Jul 1, 2015, 12:42 AM ludamad <notifications@github.com> wrote:

> I suggested 'hashkat', but this was if we were going to have run.sh as
> well only. I strongly prefer run.sh and build.sh otherwise.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hashkat/hashkat/commit/5b21673c75d143c5c7962d61cc65038a40faecfb#commitcomment-11946035>
> .
>


This could be solve using `lists:key_find/3`, something like:

```erlang
{M,F,A} = lists:keyfind(Mod,1,List),
```


Como comentei antes, procurar usar o cpf, para restringir melhor o atleta.


I'm not sure this entire commit is a clear improvement.


topkek


Move lines 16-20 to just below line 11. Then set the Item's user to the user we create on line 5.

You're creating orphaned items, they won't show up on any users to do list.


the question was more around should there be a cleanup in that __init__
file.

On Wed, Jul 1, 2015 at 3:57 PM benjaoming <notifications@github.com> wrote:

> The docs have actually been changed to encourage a more explicit way of
> using rpi config:
>
> kalite start settings=kalite.project.settings.raspberry_pi
>
> Or change your .kalite/settings.py
>
> It will explain here, once docs are rebuilt...
>
> http://field.learningequality.org/develop/docs/usermanual/userman_admin.html#configuration-settings
>
> @MCGallaspy <https://github.com/MCGallaspy> could you update the docs
> build?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/learningequality/ka-lite/commit/c82d9ee099019d7eace625f3d66e926c3552f4e6#commitcomment-11955382>
> .
>


For consistency and compatibility with the AWS SDK, the encoded characters should be in uppercase: `c.charCodeAt(0).toString(16).toUpperCase()`
Also, I'm testing this against my Java backend and the AWS Java SDK uses uppercase encoding like `%2A`, but `%2a` produces a different signature and the request fails. It's quite annoying.

Source - the RFC spec:
> The uppercase hexadecimal digits 'A' through 'F' are equivalent to
>  the lowercase digits 'a' through 'f', respectively.  If two URIs
>  differ only in the case of hexadecimal digits used in percent-encoded
>  octets, they are equivalent.  For consistency, URI producers and
>  normalizers **should use uppercase hexadecimal digits for all percent-encodings**.


No caso, não é "updated", não? porque ele já está salvo.


It would work in basic cases, but managed objects can be registered after this call.

In normal case, when Jetty is already started, it call start() on registration of those objects. In dropwizard, it can be performed at any time by calling [lifecycle.manage(...)](https://github.com/dropwizard/dropwizard/blob/master/dropwizard-lifecycle/src/main/java/io/dropwizard/lifecycle/setup/LifecycleEnvironment.java#L40-L51). But with jetty bridge, it think it won't work because Jetty server is started.

In fact, I still think we should call start()/stop() on Jetty Server because it extends [ContainerLifeCycle](https://github.com/eclipse/jetty.project/blob/master/jetty-util/src/main/java/org/eclipse/jetty/util/component/ContainerLifeCycle.java) that does many things about those managed objects.

Implementation of [ContainerLifeCycle](https://github.com/eclipse/jetty.project/blob/master/jetty-util/src/main/java/org/eclipse/jetty/util/component/ContainerLifeCycle.java) is quite complex, so I doubt we could achieve a good result by only mocking it with a dumb start/stop implementation ...


"You can *also* work with encrypted *yaml structures*" maybe?


When there is no `Mercury.wiki.dbName` we redirect to community.wikia.com.


Considering that there's already a dracut/90zfs/zfs-lib.sh.in which content is similar to both etc/init.d/zfs-functions.in and initramfs/scripts/zfs, I don't see any reason why not to remove zfs-lib.sh and replace it with zfs-functions.

But some more restructuring of the source tree might be in order (in stage two), to indicate that zfs-functions is a _global_ script lib, and not just for the **init** scripts.

We can make this a three stage process:

1. Do minimal fixes to initramfs to get it accepted.
2. After tag, move common functionality from the initramfs zfs script to zfs-functions
3. Merge **dracut** with zfs-functions.


I'd rather have a minimal base stdlib with a set of default-installed packages/modules (#5155), of which JSON would certainly be one. Having packages installed under `site` is one simple idea that might already work, just needs people to experiment with it and identify issues.

This certainly would have been caught if we had the webhook plumbing and hardware in place to run PackageEvaluator on a PR prior to merging it.

On marking functions as private/internal to a module, most dynamic languages have managed fine without that capability, but if someone proposes an implementation of the feature that works well it would probably get enough buy-in to be merged. Stop telling us how we need features you want to see in every single unrelated issue (all you accomplish by doing that is making people mute you), start proposing implementations of them.


Currently, I still use a fork of wizard-in-a-box because of this, i'm not really confident about dropping the LifeCycle support from Jetty. But great work by the way !


Interesting. Yea I had done a little research on this and did not find much. How do you think it works for you?


The other one was here... if you want to put this back in the future, you'll need to pretend to be that platform in your spec

ChefSpec::SoloRunner.new(platform: 'windows' ... )


@Qix- Use `git pull --rebase` instead of `git pull` so we don't end up with lots of spammy merge commits like this ;)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.399.94ce61c40440c1a08e42d56dc316d48ca0bc174f](http://build.palaso.org/viewLog.html?buildId=59136&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3319, ignored: 130 Build time: 00:18:55


Gaaaaah!!! Why do I keep forgetting this?
I'm gonna have to write a git hook or something


~AUTISM AWARENESS DAY~


Das kommt mir bekannt vor. :)


TeamCity NEventStore :: Core :: Build [Build 5.3.0](http://build.prxm.it:8811/viewLog.html?buildId=3300&buildTypeId=NEventStore_CI&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 279 Build time: 00:02:33


Works only on hard assumption of globally installed `babel`


Imported all changes from sistlind/circuitikz master, all further development will be at lte-fau, a direct fork from the original repository


Why are ghostify all objects in the ZODB in-memory cache at request startup?  Doesn't that defeat the purpose of having them in RAM in the first place?  None of them should have `._p_changed` as a true value inside a `ContextFound` handler.


Hmm, we already had the `repoze.zodbconn.transferlog` WSGI filter [available](https://github.com/karlproject/dev-buildout/blob/master/etc/karl.ini#L56-L58).  Seems odd to add a tween for the same purpose.


What do you mean by "support models sub-directories"?


rax retry tests


I compiled with Linux (gentoo). `_MAX_PATH` is defined in file par2cmdline.h in lines 152 and 192 in the `#else` of `#ifdef WIN32`, so it should work with Linux, but - as I see now - maybe not in Windows?


Si de una, no se que onda.. ahora no se borro mas, ahi termine la parte dos con todos los metodos. probalo, no se eliminan ni modifican ni se crean nuevos pero te diria que no le erre en nada, el lunes preguntamos sino

Date: Mon, 29 Jun 2015 17:20:37 -0700
From: notifications@github.com
To: ".NetTP2"@noreply.github.com
Subject: Re: [.NetTP2] agregue comision.cs (f5aabce)

Ya es como la 5ta vez que agregamos esto

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


> This certainly would have been caught if we had the webhook plumbing and hardware in place to run PackageEvaluator on a PR prior to merging it.

This is true in most cases, except for @IainNZ's comment [here](https://github.com/JuliaLang/JSON.jl/issues/110#issuecomment-117708789). To me, that's a *big* reason JSON should be in Base.


[heavywater] New heavywater-ruby created (version: 2.1.5-25)

- https://fission.d2o.io/pipeline/job/d891b874-9562-4d76-81a4-00000fca0000


We have zsh completion too!


ok. cool!

Glad it worked so easily for you.


Yes


Out of curiosity, which version of `spring-test` are you using?


no regression found


Um caso de refatoração que você pode fazer, é criar uma função "assertValues" com os paramentros "contract e symbol", assim, essa função retornaria um bool com a mostragem do erro.


sorry I will fix that in my next PR

Christoph Fischer
Software Developer

Wikimedia Deutschland e.V. | Tempelhofer Ufer 23-24 | 10963 Berlin
Phone: +49 (0)30 219 158 26-0
http://wikimedia.de

Imagine a world, in which every single human being can freely share in the
sum of all knowledge. That‘s our commitment.

Wikimedia Deutschland - Gesellschaft zur Förderung Freien Wissens e. V.
Eingetragen im Vereinsregister des Amtsgerichts Berlin-Charlottenburg unter
der Nummer 23855 B. Als gemeinnützig anerkannt durch das Finanzamt für
Körperschaften I Berlin, Steuernummer 27/681/51985.

2015-07-01 13:14 GMT+02:00 j. kroll <notifications@github.com>:

> Why remove the indenting?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wmde/DeepCat-Gadget/commit/34313502dca00a9574d34af3a0826ed970c6fd4c#commitcomment-11950904>
> .
>


Awesome!


:hand: @ansis


os.path.splitext will only return an array of length 2. So at the end of line 56, fileNameWithoutExt would equal 'drawable_name.9' which isn't what we want..


may be more concrete to mention that all methods are implemented with just an empty method body.


At lease in Python 2.7.6 (:


Sorry for the breakage (but glad it was only in tests).  Note that on Py3k, your tests would have broken differently, because `None` is not comparable to integers.

I'm still sure that it was never the Right Thing (TM) to have an actual integer code associated with the base class, rather than the concrete one whose name corresponds directly to the code.


boot2docker is not an exceptional case - it may even be the more common one - so we should perhaps rephrase this to at least put them on equal footing.


Hadn't noticed it was in the wrong package, I moved it in my branch.

Multiple `BaseTest` classes are confusing. You could factor the `createTestFiles(int nrFiles)` method out of InfolisBaseTest, better to explicitly set the strings in every test class.


if (i == j)
				continue;
와 같은 방식의 checking if문보다는

if(i != j)
	if (temp[i].sum_score >= temp[j].sum_score)
		k--;

와 같이 사용하는 것이 가독성도 더 좋아지고 continue 함수를 최대한 피하는 방법입니다.


Why did you add `distance_sensor` on blacklist? It is working and I already sent you feeback on it.


Yes, irritatingly that part was in there for months :sweat_smile:


The commit message should say, "default --system flag".


Surry, will do.


von ? :P

Randy Schütt <notifications@github.com> schrieb am Mi., 1. Juli 2015 um
17:06 Uhr:

> Das kommt mir bekannt vor. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/snowiow/SIFT/commit/f1edfd08706ea79d061805c767ff543dc731a185#commitcomment-11955597>
> .
>


Потеряна кнопка на фоме с встроенными компиляторами.


кнопка вернулась. ведет хер знает куда.

в студии сломан дизайнер у формы с выбором .map-карты. не одно, так другое.


Do you think it is possible to do without it?
Would you like Do it?


@tkelman I *have* proposed ways of doing that already.  Also, julia is not just a "dynamic" language, it is a language that could be used for very large software projects, if some of these issues could be dealt with.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.710.a627fd497319a11f1fcbbd6ca8c7ba3b332db04a](http://build.palaso.org/viewLog.html?buildId=59143&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:04:24


Thank you for finding that. I've now removed the tmpfile file and added in `RuntimeDirectory` and `RuntimeDirectoryMode`.


2 percent milk


"30" litres, pequeno erro de inglês


Sorry for being a bit thick :) I didn't have that issue, because my sproutcore was too old to have those. I think that  commit df546aedf060117c3ac2bb2ddf6a11f748c253c9 should fix this issue. Please check again.


Yes, it would have prevented PackageEvaluator from working at all in this case, which would have been a pretty obvious sign that thre's a problem to fix.

I don't think JSON crosses the long-term threshold of a mandatory dependency for all code written in the language. Neither do blas, lapack, sparse matrices, fft's, bigints, bigfloats, etc, but for now it is what it is. The only benefit of moving code into base in the near term is load time (soon to be fixed for packages too) and immediate testing in CI? If there are current packages that rise to the "must function at all times" level of urgency, what would moving the code buy us as opposed to adding `Pkg.add("JSON"); Pkg.test("JSON")` to base's Travis?


It really is. Enums are kinda hobbled.


I'm out


The decission today was made that the whole script is going to be removed from the repo. I will have that deleted from the repo today


And I think it is impossible


See this example which demonstrates the problem: https://github.com/raszi/node-tmp/issues/48#issuecomment-117712262


Haha, removes satanism also..


Nope.


k = num;
매 i loop마다 k=num으로 초기화를 해주고싶다면, for문의 바디에 써주는 것도 좋지만
for(i = 0 ; i<num; i++, k=num)과 같이 for문안의 조건식에 적어주면 가독성이 좋아집니다.


This will AFAIK break tests in nette/application. At least based on what I remember from playing with this.


@szimek php tag is not working in index.html but if i rename index.html to index.php then it prints the app.js code on index file. Can you help me how to sort out this.


> I have proposed ways of doing that already.

Then please stop bringing it up at every opportunity in every other unrelated issue, and move forward with an implementation proposal - as an independent issue or PR.


고생했습니다!


JSON has become the lingua franca for a lot of things these days, and can be used in lots of places instead of having special parsers for different things like history files, metadata, requirements, etc.


TeamCity NEventStore :: Core :: Build [Build 5.3.0](http://build.prxm.it:8811/viewLog.html?buildId=3302&buildTypeId=NEventStore_CI&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 279 Build time: 00:01:47


Marcin Wielgus found that the issue can be solved by quoting the input string: <<< "${ab}". I still don't understand, but quotes are always good...


That makes complete sense! thanks!


Você já tinha adicionado um residueGenerator com esse endereço, em "nukaCola"


This got dropped at some point, don't know why


You're absolutely right – will fix this now – thanks for spotting


No space between ( and null.


no big deal :)


`else if(` should be `else if (`


Kkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkkk


curl -H "Content-Type: application/json" -X POST -d '{"firstName":"Sachin","lastName":"Tendulkar","country":"India"}' http://localhost:8080/user/create

Error: {"code":400,"message":"Unable to process JSON","details":null}

I am hitting this error. Have tried almost everything.
Any idea on how to resolve this issue ? Quick response will be appreciated
Thanks in advance


problem of password updation solved
[issue] problem with changing password #8


Hi, I noticed this bug today. Thanks for fixing it! Just one note - you don't need the culture at all: just build the date object using provided values and parsing the value you are creating with a specific culture format in mind:

    return new DateTime(year, month, day);

And then you can get rid of this method as well.

Anyway, good job

Karel


see https://ci.suse.de/job/openstack-mkcloud/4701/console

from what I can see the reboot handler is ultra quick, and my guess is that it shuts down the network before crowbar_join --stop can do anything.

in practice the code should check that the power state is being set after reboot, filed a bugreport for that.


Yeah, there are three phases here: producing deltas (which are unordered), comparing deltas (which is done through a kind of array-based Perlish Schwartzian transform, ordered), then applying a merged delta stream to a record (which again is unordered).

For convenience we keep all of these steps in a consistent order, but I'll leave a note explaining the underlying constraints.


Fixed in v1.0.4 – have now tested this against s3 with all the characters from the RFC and the string is signed correctly.


@teolisitza Thank you for pointing out this oversight... I am about to push 1.2.32, which will include the fix for this problem.


[heavywater] New heavywater-ruby created (version: 2.1.5-26)

- https://fission.d2o.io/pipeline/job/d891b874-9562-4d76-81a4-0000103e0000


Use this version
http://gerrit.aicp-rom.com/13288

the first one wasn't working correctly


je crois que t'avais raison au début, c'est bien `août` avec l'accent.
Cette erreur a été loupée dans les review précédentes.


rename your variable


isCompleteImport


Using a popcount instead of existing code is not a simplifcation IMO.


thanks for finding this copy&paste bug


Гоша, ты видишь этот коммент?)


Oops. This makes the buttons full width.


Thanks for the comment. I believe `fileNameWithoutExt` would equal drawable_name at the end of line 56, if `fileName` is either drawable_name.png or drawable_name.9.png.

Notice the nested if's which would lead to stripping out the extension twice for a .9.png file. I guess that's what we want.


remove comment


Const names should be capital


I decided to play it safe, but I think at this point `storage` is guaranteed to be `undefined`


Need to replace with an extension manager pattern and allow nulls (no domain changes in core for this feature).


ótimas mudanças, bela tela de inicio


No, see the lastest commits i made ;)


I think this should be in `develop` first.
Never mind, just be careful next time.


You are right, sorry for the mistake.


we probably don't rally care about `-mms-bitfields` as the code in the main executable is virtually non-existent, but it makes sense.

But do we need to keep `-mwindows` on the lib?


I know.


explanation: without quotes bash splits the string using IFS (now it is /) and prints it using OFS (space). So the input for read already has / replaced by spaces.
Quotes are required.
But in the current shape it also works (with a space included in IFS).


config your IDE


did you mean to delete this line? there are now 2 definitions of the exact same method


Why did you remove the test?


dead code ?


#HTML
Repositorio Alicia Arcos Méndez


indent with spaces


Could we instead use a slightly more public API?, something like:
```
def deserialize(self, data):
    if 'radio' in data and 'radioType' in data:
        del data['radio']
    return super(CellTowerInternalMapping, self).deserialize(data)
```


spaces


maybe it doesn't matter as we already had some here, but I would rather have added those first thing in the file, so whatever the included files include themselves we'd be safe


remove comment


why $window ?


LGTM if it actually works


@AbdullahAlger Usually preferable to just create a new migration to add a column, especially once you've got a production database with real data as you won't want to rollback (rolling back would delete the table and its data).


Why is this not `master`?


Is this the case for all actionURLs?  Or just in this particular case?


Yes, the ports are the container internal ports which are mapped dynamically by the Docker host. This port is then put into `${catalog.port}` which then can be handed over to intergration tests.

BTW, the ports in the `<build>` section are more or less for documentation purposes, what matters are the ports in the `<run>` section.


Hmm, it's not supposed to be giant-ass anymore. What browser are you on?


This could use a comment.  Perhaps:
`// Keep ledger alive for life of MetaView`
or something like that.


Apparently it breaks one test http://jenkins.plone.org/job/plone-5.0-python-2.7/1572/


@paulolieuthier SuperBuild should work with this


rule #0 Avoid pushing to master :)


@eliotsykes

I did this some time ago and added it when I thought I'd need it and before data was seeded. However, good point!


TeamCity NEventStore :: Core :: Build [Build 5.4.0-unstable.1+1](http://build.prxm.it:8811/viewLog.html?buildId=3303&buildTypeId=NEventStore_CI&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 279 Build time: 00:02:08


No, you are right. Can be omitted if put top-level in a `<pluginManagement>` section.


Never mind, I see it commented below.


@AbdullahAlger I see you since fixed this and put validations in the model. Let me know if you'd like to discuss.


relevant: https://github.com/mozilla/amo-validator/pull/271/files#r33687458


if you realy need to write HTML, create an HTML fragment


(and yes, it was a mistake)


why 2 ???
to method for the same logic ???


Grrrr !!!!


@eliotsykes

This was when I started. I was getting confused by some things and have since deleted it.


OK. Thanks for all of your input!


In ~/.nic/unikitty.properties set one of these two properties:
spring.profiles.active=hsqldb
spring.profiles.active=mysql

Mark was sighing about MySQL in Hipchat so probably hsqldb for now :)


Missed _common_ here?


SimpleCov.start is actually called a few lines earlier


Small thing, but consider making the var name here `input` instead of `mod`, which I read as a reference to modifications.


Sur les releases c'est dans le flow. Tu tag, et configure la branche pour la version d'après.


Is this already available on maven central?


@amite Check this I have created two new components and modified some exisiting components for this new landing page project..


oops! :wink:


This mapping from internal port to `${catalog.port}` is automatic?


Reloading the page "fixed" the issue (though I have to say, the huge ass icon was much better :smile: ).


Got a new git clone and everything was ok.


Hi @dcarty01 - this is missing a `/` at the beginning of the file, the first line should be:

```
/*
```
See the first line of the gist here: https://gist.githubusercontent.com/eliotsykes/44a9ca02fe5a71c981f9/raw/a8b119c57778db55b082ed1eb5abb93101a794fb/application.scss


ddsadasdas


Awesome, thank you!


Eh, strike that, tried it out myself locally and couldn't get that to work. Problem is I remove this method and all tests still pass, so we don't actually test this case :(


ack


- replace !important with extra class selector.


Can't we just delete these tests? I doubt anyone would want to port them to lua since they won't be run.


Is this real life?


Does this new loop need `uv_loop_close()`?


Actually. Now that I think about it...

order is:
 1. TImeout is set
 2. kill process

If the process exited between timeout is set and kill, then the os.kill should throw error saying that the process doesnt exist. So, Im not sure which is correct now.


hello


Hmm. I'm not sure what I was thinking there. I'll re-instate them in a fixup commit.


@gr2m Done. To do this you need to run `npm install` to make sure the latest modules are installed and then `npm shrinkwrap`. https://github.com/hoodiehq/my-first-hoodie/commit/2bd59656080437faf8600ea22d9c9afe5920b27a

Release as you like.


This won't work, because `$field_args['show_names'] = $this->prop( 'show_names' );` are no longer being passed to `get_field`. Should probably move 184 inside the hidden condition.


是啊，似乎这样才能一直保证保持服务。。。


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


Ah, whoops. Thank you!


reack


hello tuesday


If i remember correctly ... if an exception occurs inside the context manager ... it cannot be caught - am I right ? If that is so - then I *think* it makes sense.

May be safer to just throw it as a failed test.


see https://github.com/mozilla/amo-validator/pull/292 for a fix


why should an exception in a context manager be not catchable?


I'd checkout the anton branch i just pushed, try pulling up the autocorrect bar, you need to track the frame, since KeyBoardWillShow does not get called when you toggle it if i'm correct


pushing up an update now...


Also, when using `ThisShouldNotHappenError` the message should state the reason why it should not happen, i.e. why can those exceptions not be thrown here.


I thought this might explain a rendering problem, illogical though it seemed. But, in fact, GitHub Pages just isn't publishing the updated CSS.


Hm - if it is catchable means that something else is wrong and that the process didnt really exit on it's own.

But it still gave exit as 0.


I think most of user don't use that, and who use easily may reenable.


Please beware of the indentation issues, we should try to avoid tabs, use 4 spaces instead


I didn't remove it, the test is the same it just uses the beaker recommended syntax


Good to know, thanks. I was planning on working on this module next...


@jtsternberg Or perhaps we could move the extras up above 184?

```php
$field_args['context'] = $this->prop( 'context' );
$field_args['show_names'] = $this->prop( 'show_names' );
$field = $this->get_field( $field_args );
```
Would that work?


这个基本上完全重翻译了一遍。-_-||


I'll rename this to `item` since `input` is Python built-in function.


ahm the contextmanager isn't the problem I believe, that would just throw
out the exception. But we have that stuff in another thread. Let me try a
commit...

2015-07-01 17:51 GMT+02:00 AbdealiJK <notifications@github.com>:

> Hm - if it is catchable means that something else is wrong and that the
> process didnt really exit on it's own.
>
> But it still gave exit as 0.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/734150baa1a691a6a84c40b358b9087fefaa3163#commitcomment-11956706>
> .
>


Django 1.7 adds `JsonResponse` if you want to simplify this call: https://docs.djangoproject.com/en/1.8/ref/request-response/#jsonresponse-objects


Why not use <= here?


The easiest would probably be to define a function `onReceivedError` outside of the loop below that counts how many times it has been invoked. If it is ever invoked `adElements.length` times, then call `onError`, because it means there are no ads recursively below. You then pass `onReceivedError` as the `onError` argument to `queryVAST` below. `onReceivedError` should also be called `if (ad.isEmpty())`.


Here is the mapping to the variable.

Note that you have to refer to the var without ${ ... }, otherwise maven would interpolate the (empty) value here leading to a syntax error.


Eventually we want to get rid of the `ref` array, all pointer values should be written in the `i32` array instead.


Looks good to me.


Ok, my fault. I thought that removing the accept_empty handling would now also allow empty submit requests through. But in submit requests we actually require a top-level 'items' key and that triggers the rejection.

And the test before used `None` which I thought rendered to `''`, but instead actually rendered to `null`. Changing the test to use `[]` just means we now get `[]`, which is still not actually empty.

So all is good. Except that when really sending an empty body via ``res = self.app.post(self.url, '', status=400)`` we get a completely different actual error in our code. But that was there before, so unrelated to these changes.


Yes, the snapshot is pushed. The final 0.12.1 is expected until the end of this week.


In order to alter a commit status, the account that is using the api (the *account* not the login token) must have write access to the repo.


The weird thing about making this a CP is that the `window.location` can change. If they viewed it as HTTPS then came back as HTTP it would be cached.

I think you need to make this `volatile()`


Well for sure more will use this one than mocap and vision_speed...


Are you going to use this for other OS as well? If not, it's RIOT specific and not really common.


Sorry, I pasted the wrong thing it should be the rosdep sources not the apt sources.

more like
```
RUN echo "yaml https://raw.githubusercontent.com/ros/rosdistro/master/rosdep/base.yaml" > /etc/ros/rosdep/sources.list.d/25-custom.list
```


These should be `const` right?


There could be unintended consequences of initiating the field object that early, particularly w/ the group-field flow, and this way, we initiate the field object as late as possible. This commit should do the trick: 387d1c20b641056b4ccba1dc09519239a7f4b3b2 (ignore the doc-block change, i've put that back in place).


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 99](https://teamcity.labkey.org/viewLog.html?buildId=258784&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:04:48
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

```


TeamCity NEventStore :: Core :: Build [Build 92](http://build.prxm.it:8811/viewLog.html?buildId=3305&buildTypeId=NEventStore_CI&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:00:33


Testing


This is specific to APKs, but we have manifests on `android_library` too.  Either we need to reword this to be more generic (preferred, I think), or change the name to be specific about APKs.


Looks good to me.  I've tested it with Grace and it works properly.  This solution basically mirrors the code that the built-in Android camera uses when it takes a picture -- i.e., it immediately scans it so that it will appear in the Gallery.


@targeter21 - first of all, great work! thank you for resurrecting the package
It looks like there is a bug from this line - it removes the status bar altogether from current window. do you want me to raise a new bug?


```
* Addresses an issue with empty result when ...
```


Is all this raw formatting necessary? It's a lot more difficult to update than a handlebars template. This file has a lot of string rendering.


ditto here - the deps description here is specific to APKs we generate, so we should be sure to name it accordingly.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.4.453.906d1df991304ea46024abe54f48da36e54b7097](http://build.palaso.org/viewLog.html?buildId=59147&buildTypeId=bt2) outcome was **FAILURE**
Summary: Execution timeout (new); tests passed: 871, ignored: 50 Build time: 00:37:25


Let's use columnHandlers as |handler|


> ... in order for legacy oairepository admin to work.

Can you rephrase it?


Why?


I don't see an opening `<p>` for this


I don't see a closing `</p>` for this


TeamCity NEventStore :: Core :: Build [Build 93](http://build.prxm.it:8811/viewLog.html?buildId=3307&buildTypeId=NEventStore_CI&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:00:23


Can you remind me what issue did you have with those empty layers? Looking at the code, I can see that FontForge now has [special handling for empty Fore and Back layers](https://github.com/fontforge/fontforge/commit/f0a48a100a6cfd37a3035475f60fc561d8647a32) (why they are generating them is beyond me), but when I tried loading this file in Sorts Mill nothing ill happened. I can certainly build the fonts before this commit.


Sí, en este sentido iba el comentario.
Mucho mejor ahora  ;)


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1674 of 1676 tests passed)

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


Please instead of commenting out the original and adding a new line and comments, modify the line and add the comments and justification in the commit message.
The commit message can have a title line with ~50-70 characters, then a blank line and the " story of your life" if you want.
Something like:
```
Run welcome1stboot in the background

even if is not finish, to enable the recognition of audio devices, etc.
```
BTW, what exactly is the problem with devices when is not backgrounded?


who did this in the first place? :)


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.0](http://build.prxm.it:8811/viewLog.html?buildId=3309&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Tests passed: 75; exit code 1 (new) Build time: 00:02:49


Don't forget, you've got a focused test here.


Bover!


We could do that. For the moment I fixed the above to properly log


ack


@milos I would like your suggestion here.


Yes, exactly. We're spacing the deltas into an ordered sparse array so we can do a linear walk, merging them, then re-collapsing. Schwartzian transform.

We could also return a fixed-size array for all deltas, but that forces us to combine commutative changes here rather than just replaying them later.


s/with/want|need/


I am afraid this will break something. On Windows Dynamips is not in the current working directory but in a dynamips folder.


Same things for vpcs.


heh, I guess I am the first person to click on it since 2013. https://github.com/bitprophet/alabaster/commit/6278acf2dd3ce635383f0fb8ad6dbc332d81381a


Yes, the only reason to ever use this is for compat with external services that require such (e.g. at an edge to encode json).


Replace run.exec("sudo /home/turner/z3-str/Z3-str.py -f ./Z3_input");
with: run.exece("sudo /home/YOUR PATH TO  z3-str/Z3-str.py -f ./Z3_input");


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 100](https://teamcity.labkey.org/viewLog.html?buildId=258787&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:04:23
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

```


The `.all()` is a no-op here.


I think it would be better to have the directives in separate files, but the others were already in this file, so I put it here for consistency.


OF doesn't throw exceptions and in any case any error would make the locale default to the current one which is the expected behaviour


@lindamartin this is a change of behaviour.


Because `Login.username` is an optional string field. I'd like to revisit that later.


Done.


a bit strange what has been applied but currently it indeed merges file together.


@nsiefken We need to figure out a way to erase this.

@alycit Is there any way to erase past commit history so we don't expose nick's info here?


missing argument `flex($values)` should be `flex(1)`


Chachi :D :bear:


Re-instated in 11254240ead17a2618a14bfa4a3420d9c8aed6bb.


There is a tab character here - please stick to spaces.


Fifteen years of writing Common Lisp have made me the man I am today.


I don't think there is any way that unknownAtom can be null (it's a global set up at initialisation I think).


You're absolutely right. Can I make a nit to make it more readable?

replace
  if fileName.endswith(".9.png"):
with
   if fileNameWithoutExt.endswith(".9"):


Of course this only applies to packaged scenarios, which is fine since we are trying to avoid special handling of paths in the source tree.

But, I suggest that this actually points to `/usr/share/avocado-server/dashboard/static`, or simply `/usr/share/avocado-server/dashboard`.

The reason is that `/usr/share/avocado-server` is highly standardized path, which can be referred to in the RPM spec file as `%{_datadir}/%{name}`.


SourceKit hates me.


OK.


- icons were getting a hover state from .matching-select-icon. Additional selectors added to class to define correct/incorrect styling.


The same problem occurs with tabs. We could replace also tabs (\t) with whitespaces.


in .ssh there are not a lot of file
but in $topdir I've got 12918 files and directory

```
>ll
total 156
drwxr-xr-x 21 root     root      4096 Jun 23 15:20 3rdparty
-rw-r--r--  1 root     root       477 Jun  4 15:43 AUTHORS
-rw-r--r--  1 root     root     34520 Jun  4 15:43 COPYING-AGPL
drwxr-xr-x 31 www-data www-data  4096 Jun 26 10:46 apps
drwxrwx---  2 www-data www-data  4096 May  6 20:39 assets
drwxr-xr-x  2 www-data www-data  4096 Jun 23 17:29 config
-rw-r--r--  1 root     root      1941 Jun  4 15:43 console.php
drwxr-xr-x 19 root     root      4096 Jun 23 15:20 core
-rw-r--r--  1 root     root      5014 Jun  4 15:43 cron.php
drwxrwx---  2 www-data www-data  4096 May 28 10:32 data
-rw-r--r--  1 root     root     23751 Jun  4 15:43 db_structure.xml
-rw-r--r--  1 root     root       179 Jun  4 15:43 index.html
-rw-r--r--  1 root     root      1792 Jun  4 15:43 index.php
-rw-r--r--  1 root     root      2595 Jun  4 15:43 indie.json
drwxr-xr-x  2 root     root      4096 Jun 23 15:20 l10n
drwxr-xr-x  6 root     root      4096 Jun 23 15:20 lib
-rw-r--r--  1 root     root       283 Jun  4 15:43 occ
drwxr-xr-x  2 root     root      4096 Jun 23 15:20 ocs
-rw-r--r--  1 root     root      1838 Jun  4 15:43 public.php
-rw-r--r--  1 root     root      1820 Jun  4 15:43 remote.php
-rw-r--r--  1 root     root        26 Jun  4 15:43 robots.txt
drwxr-xr-x 13 root     root      4096 Jun 23 15:20 settings
-rw-r--r--  1 root     root      1525 Jun  4 15:43 status.php
drwxr-xr-x  2 root     root      4096 Jun 23 15:20 themes
-rw-r--r--  1 root     root       172 Jun  4 15:46 version.php
18:07:03 root@owncloud /var/www/owncloud ---------------------------------------------------------
>find . | wc -l
12918
>find core apps 3rdparty | wc -l
11847

```


when 8.3 is off, `name` equals to  `".git"`


Yeah, we don't use local_modified for much, so I could go either way.


no it's wrong

check ?page=V3rti7y   !


Now that I you mention it, I realize I should have look at the existing Constant Flow Analysis. I will rethink about it..


it didn't find `ppx_deriving` otherwise. Also, I had to revert the `libexec` part, as it didn't work with opam 1.2 which is the default on a lot of distributions.


Cool.  I didn't know that option existed!  I'll be using it all the time from now on. :-)


remove the commas


@pavel-kirienko  Thank you!


I'm just prototyping things right now - yes, that's the next logical step.


questo è un commento!


Those things are equivalent, with one difference that makes me do it this way: when we do this work *now*, it makes deletions cheap. If we don't flip all of these flags now, we'll be tracking deletions for no reason until you sign in again.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.711.aff1f00c9655e3289fa99a77cb5ec4b276d16f50](http://build.palaso.org/viewLog.html?buildId=59156&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:06:53


*jazz hands*


can you do release? I'm a bit overwhelmed with stuff today


whaaaat? Juan!


OK.

Note: I believe we'll later to have to deal with the fact that jquery is available on Fedora (`js-jquery`). I have no idea if this kind of bundling we're doing now is strictly forbidden.


Depending on how fast we can get https://github.com/hoodiehq/my-first-hoodie/pull/76 merged, I'd a bit more. But I'll do it then.


`mapViewDidStopLocatingUser:` is a good counter-argument — weird. Let's leave as-is.


Sure, thanks!


Looks like the build hit the timeout.  I've extend it to 55 minutes.


Instead of one entry per config option, I'd say something like: The experimental_big_query session property and the following config options are no longer supported: experimental.big-query-initial-hash-partitions, ....


I opted not to precisely for safety reasons.

If we have local rows in the DB that are marked as synced and deleted (which should never happen), this will find those, upload a deleted record (either good or harmless), then the upload deletion handler will remove them from the DB.

If there are no such rows… well, we just saved an additional WHERE clause.


E quindi war Ma w y Qatar Dewwas West e r2rq e ywweer wdiQatae ww we e poi non wet we ewe r we ww


Kyr'Ozch


All Sync records have a sortindex in the envelope. GET uses it by default, IIRC. In this case it means deletions come first. The value is arbitrary.


```
WARNING in ./~/fibers/fibers.js
Module parse failed: ......../node_modules/fibers/fibers.js Line 2: Illegal return statement
You may need an appropriate loader to handle this file type.
| if (process.fiberLib) {
| 	return module.exports = process.fiberLib;
| }
| var fs = require('fs'), path = require('path');
```


The following line is not required in the two yaml files, since it is the default value:
`imagePullPolicy: IfNotPresent`


Oh, it was my commit you're reverting. Okay then


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


On 07/01/2015 06:01 PM, Khaled Hosny wrote:
>
> Can you remind me what issue did you have with those empty layers?
>
I think Sorts Mill did not load some foreground layers with that
combination of

`
Back
Fore
`

In the glyph file i.e. when there is an empty Back layer. Also an empty
foreground was problematic. Some glyphs where "empty" I think.

When Kourosh saves the font with Fontforge and pushes it to git, I would
run
https://github.com/Tarobish/Jomhuria/blob/master/tools/removeEmptyLayers.py
on the font. Then commit it with a message like the one in the subject line.

I actually don't bother anymore to see if it would work in that specific
case without running the script, I just do it.

Fontforge will always add these empty layers to the files.

I could try to find a case where the sfdir is not loaded entirely, if
you are interested to investigate this.

> Looking at the code, I can see that FontForge now has special handling
> for empty Fore and Back layers
> <https://github.com/fontforge/fontforge/commit/f0a48a100a6cfd37a3035475f60fc561d8647a32>
> (why they are generating them is beyond me), but when I tried loading
> this file in Sorts Mill nothing ill happened. I can certainly build
> the fonts before this commit.
>


Agreed. It's a minor thing and doesn't deserve to be in such a short description of what Avocado is.

What if we always try to stick with an official definition of what Avocado is:

http://avocado-framework.readthedocs.org/en/latest/Introduction.html

This should be canonical version, and if we think it's not good enough, let's change it!


Actually, no, it didn't find `ppx_deriving` because of the change here: https://github.com/whitequark/ppx_deriving/commit/8f871996b5cc7bfba676f18e3adea5eff05182b5


Yeah, I like that better. I deleted my above comments about functions and do the same as previous by putting them together.


Where do I get the UUID? Is that checkin_code for Receipts?

Unsure.
@BenDrews


Our chaining operators abort on failure, so yes, this is entirely about making sure a failure in password sync doesn't stop us syncing history.

We *could* use `both` or some other deferred operator… but I don't want to sync in parallel just yet.


I tend to only add relations when they are needed. Otherwise they add extra methods that are not necessary, and that, though not by much, make the application grow unnecessarily.


I would say do it... looks like what you want!


I suspect this can be tricky when it comes to packaging. I see no harm in renaming `dashboard.html` to `index.html` since it's already under `dashboard/static`.


shouldn't we skip this whenever we run tests?


Any text


Woops, not intentional, nice catch. Do you want to try choosing an issue through a commit? When you're doing your tests make a commit to fix it, something like:

```
Fix data frame not being used in arg validation test
  - Fixes #21
```

If you add that bullet it will automatically close the ticket I opened once we merge into master.


@monkbroc , the problem is that the web IDE is not using the firmware library required to compile the timer code at this time.  It should be released in a couple of weeks or so.


@hoholee12 @DiamondBond ready :) works like a  b8


@Fevax thanks :) i have added you to the group , if possible kindly come online on Hangouts has its difficult to actually debug without contact.
Thanks .


Why? There are very valid reasons to 'stomp' lower precedence values with a nil. For example in this case (https://gist.github.com/Sauraus/512df5b9ff626716e6f6) I do not want anyone ever to able to add a mirrorlist to the yum repo files.


@hastalafiesta

According to a dev, the issue is in ACodec.cpp without doubts:

http://forum.xda-developers.com/showpost.php?p=61228126&postcount=21
http://forum.xda-developers.com/showpost.php?p=61228942&postcount=23


I realized that you removed the code that did `set timezone 'UTC` and you just added checks instead. There must be a reason for that. I'm interested to know what the reason is. Anyway, we can probably just update the docs at this point.


It might be simpler to just directly set Solver._use_inline=False here, but that does mean moving the weave imports to a place where the Solver class has already been defined. Maybe the weave imports should just be inside the try block in Solver._test_inline()? It's already catching ImportError (see c2588f1d26b19f755209c0d9c213c634ee966685) so a broken/missing weave should just nicely fall back to the pure-python RHS path.


I have a question about ```addStoredRelation()```.  Before, when we would delete a row from the ```relations``` table in order to update the metadata, a row from the ```stored_relations``` dataset would also get deleted (through ```on delete cascade```).

Now, since we do a ```REPLACE``` on relations, the corresponding row in ```stored_relations``` does not get deleted. So when we call ```addStoredRelation()``` on line 1485, will it be ending up adding metadata for that relation again?

In other words, I'm wondering if we're going to end up with duplicates in ```stored_relations``` or ```shards```, which also references ```relations```


Do you use mocks somewhere, or some other package for it? I'm asking, because PHPUnit 4.7 includes Prophecy support. Nice package for mocking.


I'm on the train so I can't look, but I remember we have some macro to retrieve cross versioned artifact names. Grep for "cross" in the build?

We should document what swing's osgi version is for (not used to resolve, should be derived from the maven version).

Otherwise, LGTM! Whew!


TeamCity NEventStore :: Core :: Build [Build 5.4.0-unstable.2+2](http://build.prxm.it:8811/viewLog.html?buildId=3310&buildTypeId=NEventStore_CI&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 279 Build time: 00:02:27


looks legit!


Can you please remove this dependency.


Question. What images are being requested?


Oh, I didn't know the apply_manifest was a test, I though it was just a command.


Why not call it `mavros_test` instead of `test_mavros` so to keep integrity?


OK.


What are the numbers for? The hardcoded 20 * 3000. milliseconds?


there is call to do_flags in defines
`$(call do_flags,-D,$(defines))`
I thinks this line is OK except possible miss of args with spaces or quotes.
May be this should replaced with next?
```
$(call do_flags,-D,$(additional_cflags))
```
Will it work?


Thank you for your introducing.


So people installing by deb packages can do `apt-get install ros-indigo-mavros-test` for example...


test


@jboeuf - refcount is here


@segiddins :heart:


missing @augment tag


97648166-FixPIUInducedNode-4926 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1680 of 1682 tests passed)

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


missing @augments tag
https://github.com/softlayer/ember-style-guide#jsdoc-tags-utilizing


Hi,

Replying to a random commit from two years ago isn't the way to get help with Chef. This only emails one person, who might be at a new job, or be on vacation, or might be filtering github emails and triaging github work via some other means. If you can join the Chef mailing list and ask a question like "how do I do X with attributes," you'll reach a much larger audience besides one individual contributor.

Regarding your use case, some features have been added in Chef 12.0 which may be applicable to what you want to do.


This commit isn't strictly necessary for the functionality.


Alex you are the Man!!!!!!!!


Ok, körde på några av förslagen, tack så mycket för hjälpen!

/Daniel

On 30.06.2015 22:04, Anton Osika wrote:
> Ser bra ut.
>
> Bildtemperatur borde skrivas så här - man skriver ihop typ allt på svenska
> ;).
>
> Kan vara bra med Döp om := Byt namn, mittimellan := mellan, det brukar
> vara
> enkelt i digitala samhangan på svenska, men man förstår ju.
>
>
> 2015-06-30 16:55 GMT+02:00 Daniel <notifications@github.com>:
>
> > Swedish translation
> >
> > Well, just quickly translated this to swedish. I've lived in Sweden
> for 12
> > years, now nearly three years ago, and I still can't believe how
> fast it's
> > possible to "forget" a language (it comes back after some time when you
> > speed it again).
> > So, I would like to ask my cousin @antonosika, who still lives in Sweden
> > to quickly check if I have done any too large misstakes.
> >
> > I hope I haven't ;)
> >
> > /Daniel
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/danieltroger/400plus/commit/a3249f4e2c2418ef2b2b75f362efe2503783c717
> >
>
>
>
> --
> Anton Osika
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danieltroger/400plus/commit/a3249f4e2c2418ef2b2b75f362efe2503783c717#commitcomment-11939988>.
>


I think the proper fix here is to prevent FontForge from generating those empty layers, but I can’t get it to generate empty layers here from me in the first place (even if I load the font with the empty layers, they are not saved back). Any idea how Kourosh is generating his SFD file? From UFO?


Thank you!


#jgaj


- Theme updated to apply contentObject '_classes' to pages.
- Class added to 'theme-extras.less' to remove assessment graphics (class needs to be applied to JSON).


:smile:


@alexism but it will be over 100 char


Because [REP 144](http://www.ros.org/reps/rep-0144.html).


It says "Friendly name" in the comment which I associate with the "Partition Name" you enter in ST3. But the Usage print out says [host_hash]. This is a bit confusing.


Code style: miniHTMLParser(HTMLParser):


Hi @mgruner,

Well I think that keep the examples files is still a good idea as a guide line, but let me know if you really want to delete it.


Note that in the new bytecode loop in `Py_FrameObject` in `master`, you need to set the flag `f.returnToThreadLoop` to signal that loop to return to the Thread's loop. This flag should be set whenever:

- The bytecode changes the Thread's state, e.g. as in the case of an asynchronous function call that puts the thread into the ASYNC_WAITING state.
- The bytecode returns from the current function, either through an exception or a return value.

You may wonder why I changed it in the first place. I changed it from a function that runs only one opcode back to a loop with that boolean as a termination condition for performance reasons; it's the same design I adopted in DoppioJVM.


@thanhdut DWMR. Good work here.


#it is a formal function to start the program.


tak to wygląda w review gdy dołożysz tylko spację na końcu linii, staraj się tak nie robić


This update mentions WebGL being disabled. That is a feature disabled through Ludei's dashboard and not in this code.


It is not clear to me whether we should keep the Lua interface in Lean or not.
Nobody really uses it. I think most users are not willing to learn yet another language just to be able to automate tasks in Lean. Moreover, we have to maintain a substantial amount of code, and we have this extra dependency: Lua.

I have been thinking about using Lean itself as our scripting language.
This will be feasible as soon as we have an efficient evaluator (https://github.com/leanprover/lean/issues/462).

Any thoughts?


oops, meant
```
$(call do_flags,,$(additional_cflags))
```
Note empt first arg


It looks sane. We'll need to iterate on this and generalize the threading support as well. Otherwise it's going to be a confusion on what comes from common and what not, since we're sharing just part of it.


Indeed, not necessary here.
as long as the bitboards are at the right of the expression.

See
https://github.com/official-stockfish/Stockfish/commit/a7592e69d728ac839f098f2ba17285c998a95839


break jest tylko w tym elsie, gdy wejdziesz do if'a to nie wykonasz polecenia break!


$ пропустил


shouldn't this be request.delete?


zbędny komentarz


I believe you should be able to do:

```
from django.http import HttpResponse

class HttpResponseConflict(HttpResponse):
    status_code = 409
```

And then raise `HttpResponseConflict`. Haven't tried though.


Example:
=======

```python
from datetime import datetime
from django.db import models

class AbstractPostStateMachine(models.Model):
    created = models.DateTimeField(auto_now_add=True)
    updated = models.DateTimeField(auto_now=True)

    class Meta:
        abstract = True

    class States:
        NEW = "New"
        PUBLISHED = "Published"

    state = FSMField(choices=state_choices(States),
                     default=States.NEW)

    @transition(field=state,
                source=States.NEW,
                target=States.PUBLISHED)
    def publish(self):
        pass

class BlogPostStateMachine(AbstractPostStateMachine):
    # Class that doesn't need anything else
    pass

class ForumPostStateMachine(AbstractPostStateMachine):
    publish_date = models.DateTimeField()

    class States(AbstractPostStateMachine.States):
        # Add one more State
        LOCKED = "Locked"

    # Add a new transition
    @transition(field=AbstractPostStateMachine.state,  # Not possible
before change
                source=States.PUBLISHED,
                target=States.LOCKED)
    def lock(self):
        pass

    # Override existing transition
    @transition(field=AbstractPostStateMachine.state,
                source=States.NEW,
                target=States.PUBLISHED)
    def publish(self):
        super(AbstractPostStateMachine, self).publish()
        self.publish_date = datetime.utcnow()


nazwa metody powinna mieć w sobie czasownik i mówić co robi


This is a daily update, it pulls in all clicks from Google Analytics and saves the new ones.


It's not clear to me what you mean by: "I'm relying purely on feedback by the department that is responsible for the calculator" in the commit note. At first glance it reads as if you have relied on them to tell you what the intention of the historic problematic commit was, but I assume you mean that you are relying on them having told you the current behaviour is incorrect.


wystarczy if(isServerRunning)


I don't really understand that but if that's how it works. It's ok.


It's kinda both, the apply_manifest is a helper method to run the Puppet apply on the given manifest, but it's also used to check that the actual compliation is working :smile:


Hi @mgruner,

What is the use case of this (try several times in the same second?) it could make sense if we also add a sleep time or so, but still same command with same data tried more than one time, it might no be any difference in the result (but just on very very special cases when the DB is not available to so)

We might specify how it should work and when

((enjoy))
cr


Based on REP 144:
> if a package is meant for test only, prepend test

It's kinda ambiguos. What "meant for test only means"? Testing the package? Testing something else (or package, a dataset,..)? Demo testing?


In the commit message, when you say: "minimum_hourly_rate (which is the rate at the given date)", does "the given date" mean today's date? Or is it a value supplied by the user?


WhY?


These are the offending changes


lgtm


dodajesz usera nawet wtedy gdy nie ma już miejsca? To prosi się o seq fault!


Actually I'm not sure of that.


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


Thanks so much. Good catch.


Where has the date, `2011-10-01`, come from?


BTW, these failures were happening when testing against upstream `bitcoind` (`0.10` branch)

4ae2a6b127312fe925774bc7f6b3c15b16ebeff6 seems to have made the problem go away (at least for now)


sort keys %{$Response} ? :-D


OK I found the answer... Seems good then


please escape code, I'm getting notices of being mentioned ;)


You should have first pulled the changes made by me and rajat to the footer (bhai dekh to leta) ...all changes by me and rajat are now lost. Next time before commiting any changes please make sure your code is up to date with master branch (our code). For now, I will try to reset this commit.


:tada:


oh... is that how you just keep the compression policy but delete the dynamicContent?


Addressed here: https://github.com/google/google-auth-library-ruby/pull/36


Yeah, it treats the lock file as invalid and requires that you re-restore. As I expected. That behavior is not ideal but I'm OK with it for beta5 because we don't expect you to mix and match project.json between then two right now. We'll update the version in beta6.


It doesn't look as if time is stubbed/frozen in this test. So doesn't using `Date.today` make this test brittle?


Yes, we still have monitors and objects on the ref array, but we're going to remove them (otherwise we can't even enable garbage collection).


I can see how commenting on a commit from 2 years ago is not the best approach, but on the IRC channel nobody could really say either why this was the case.

And unfortunately the comment in the code is rather slim on any reasoning for why this was choice was implemented and that IMHO leaves very little options to finding out the why?

PS. I did find the node.rm() function in Chef 12.x which provides a work around for this logic.


Good catch thanks. Sorry about that. I think I'm not getting CI notification from nuget-ci-3...


That requires <iostream>.


I've updated the comment. Good catch.


This can be moved down into the `prClient.commits` block


No, we're not deleting the scope. We're just nulling out that policy.


Yeah, we don't have CRUD APIs down to the individual policy level. Think of this as setting a field to null in a document store.


Thank you @rolfschmidt is missed this ones

((enjoy))
cr


We are duplicating here methods for AwsConfiguration. Why not stay with single interface, and put all AWS specific config there? These are just a few parameters.


It looks like this could now be:
if (is_array($pkg_info)) {
and the remaining other 3 references to $pkg_keys in this code can be removed.


I just tried, it seems like my Fontforge behaves as well.

Kourosh uses a Fontforge version for the mac. Probably that is not up to date and the error got fixed in the meantime.

The latest release version is "FontForge 2015-04-30" at http://fontforge.github.io/en-US/downloads/mac/

@Tarobish when you open your Fontforge and go to `Help -> About` Could you post a screenshot of the window that opens then? So we can see the version information.


This could use a more descriptive commit message :neckbeard:


Hmm I can add the path of the two directory on Windows. I will make a test
session tomorrow on Windows.

On Wed, Jul 1, 2015 at 6:06 PM, Jeremy Grossmann <notifications@github.com>
wrote:

> Same things for vpcs.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GNS3/gns3-gui/commit/563c762756b3f4140b565506c1be917999dde15f#commitcomment-11957013>
> .
>

--

*JULIEN DUPONCHELLE*
Software Engineer

E: julien@gns3.com
LinkedIn <http://linkedin.com/in/julienduponchelle/>

<http://www.linkedin.com/in/stephenguppy>288, 200 – Rivercrest Drive SE
Calgary, AB, T2T 2X5
Twitter <http://www.twitter.com/gns3_official> | Facebook
<https://www.facebook.com/gns3official> | LinkedIn
<http://www.linkedin.com/company/gns3-net> | Google+
<https://plus.google.com/+Gns3Net/posts> | CrunchBase
<http://www.crunchbase.com/organization/gns3>


@mpaladin indeed I had not realised it right away, that is why I created this ticket: [SONARWEB-35](http://jira.sonarsource.com/browse/SONARWEB-35). maybe the ticket name should be modified, shouldn't it ?


Yeah. Git is also a pain in the ass when working with branches and submodules together. I suggest that in the future, in the experimental branch, to add a .gitignore to completely ignore what goes on in 'sources' and never to call ./download-sources.sh in that branch.


TeamCity NEventStore :: MongoDB :: Build [Build 5.4.0-unstable.3+3](http://build.prxm.it:8811/viewLog.html?buildId=3313&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 75 Build time: 00:02:21


ack


commit message?


Do you know if there is a compiler flag for the HAL? It would be possible
to have 2 variants of the code and conditionally compile the correct one.

I know it's supposed to be ready in a couple weeks, but the timing might
slip. Plus more and more people have photons now.

So the interim solution would be useful for a while then the old code could
be thrown away.
 On Jul 1, 2015 12:26 PM, "Paul Kourany" <notifications@github.com> wrote:

> @monkbroc <https://github.com/monkbroc> , the problem is that the web IDE
> is not using the firmware library required to compile the timer code at
> this time. It should be released in a couple of weeks or so.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pkourany/SparkIntervalTimer/commit/9f25c7ee1099b5b8b9d1f9a3005488bd28e161c4#commitcomment-11957452>
> .
>


I would keep most of extensions out of testkit, as this is something external users would use. If somebody wants to run embedded cluster with aws or kafka, they may add it explicitly. But we do not provide necessary configuration to initialize it properly.


Thanks:) Didn't expect to get on this list that soon tho


See Issue #39


I had the transforms/py2llvm split to make py2llvm a fairly generic Python to LLVM compiler, and have most of the ARTIQ-specific stuff in transforms. Can't it stay that way?


y


update many scripts because many ambigous name and several unused scripts moved to deprecated folder.


Okay I can reproduce the error in 1.11.8. Problem was that before 1.11.9 the libstdc++ was used, which does not have C++11 features on OS X. Since 1.11.9 (b4407e3d2191a166d) we use libc++, which has `<cstdint>`.


probably it was the behavior before and we didn't think it was worth breaking


##This is a Markdown file


this one's got a hardcoded equivalent now


I prefer to keep this declaration because it makes the operation more explicit.
I will add a comment explaining it significance.


s


codestyes for all functions are different ?


So the issue is that you might have overlapping periods?


Doesn't this alter the code clone detection behaviour for tests since it needs to check also the .h file?


divyesh added


This commenting out of nearestAddress check was to support the cases of beacon heavy environments. Since ranging is somewhat noisy, the beacon you are trying to configure might not be considered nearest and as such you will not be able to configure it. But it is definitely true that in the future, we need a better overall solution than "if (true)"  :)


Is the type alignment like this on purpose?


I assume that the new literal-counting needs a new difference default for your tests.
I would mention that in commit message why you raise it^^


%obj..followObject(%obj.target, %this.repathTolerance); would be the source-side equivalent to allow navmesh derived pathing, where repathtolerance = the distance to maintain between bot and target


FIXME: processor_->


Hmm, I just got another question regarding vnswap.

Since it's just a dummy ramdisk driver, is there anything major different from creating a swapfile from tmpfs?

Maybe efficiency?


no, this loop is used for synchronous operations only and is always cleaned with the `uv_fs_req_cleanup` calls


xasxasxsa


looks good, just some notes if you want to incorporate
-put the auth info in an auth.json or config.json for now (then send it to us, don't push to github)
-put randomString function inside YelpService
-only inject needed dependencies
-you can do Date.now() instead of newDate().getTime()


saxasxasxasx


97648166-FixPIUInducedNode-4926 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1680 of 1682 tests passed)

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


Keep the double-check idiom to protect methods using lazy instantiation.
The double-check idiom improves performance by using synchronization only if multiple threads call instance() before the Singleton is constructed. Once the object has been instantiated, sInstance is no longer ==null, allowing the method to avoid blocking concurrent callers.
Avoid useless / extra synchronization.
Applicable to other singleton creation.


@AbdealiJK yes they are copied from the interned. Different codestyles shouldn't hinder a good clone detection from detecting as clone however bad clone detection may fall into that trap :)


let's move this into theme options


h files aren't checked but includes are resolved. The test altering is below through the adition of kernel_utilities.c


Why do you change that?


It'll pretty much stay that way, I just think that the name `py2llvm` is a bad one, since the usage of LLVM is more of an implementation detail of the codegen. I can rename back if you want.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.4.454.906d1df991304ea46024abe54f48da36e54b7097](http://build.palaso.org/viewLog.html?buildId=59165&buildTypeId=bt2) outcome was **FAILURE**
Summary: Execution timeout; tests passed: 732, ignored: 45 Build time: 00:55:50


ack


@flavour Please review this https://github.com/biplovbhandari/eden/commit/0e3b36ed22c2c74842d40c946aed13cf2b14029c
There are indent problems again as I see here...I setup eclipse to insert spaces for tabs as suggested http://eden.sahanafoundation.org/wiki/DeveloperGuidelines/Eclipse, then also I see indent problem


code's copied from some code where I used tabs and I'm to lazy to convert... that a bad thing? :P I don't care how my test code looks because the functionality of this code is completely irrelevant.


Yeah, we were blazing through and wanted to get it working. In a refactor, we would change the daily query to only pull that days clicks.


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


Looks like a duplicate of the line above - merge error?


Typo - "below"


any change in the algorithm results in the threshold value to change. I was lucky I hadn't do reweight the CCs as well.


Agregue esta linea como prueba


because it's easier to test and this file is for testing only. Zero is used later too and this makes this function having only one constant instead of two.


So there will be "compiler" and "transforms"?


mfw i just realized I could have just edited that by hand


@Kami it's better always to \n, if not piping gives us not nice results.
Like `echo 'additionall_package==version' >> requirements.txt`.


The `MenuController#nuke_entries` method should call another method on `@address_book` (which contains an `AddressBook` instance), like `@address_book.delete_all` for instance. The *business code* (`@entries = []`) only makes sense in the context of an `AddressBook` and belongs to that class.

Could you try to fix that?


Can you add tests?


Do:
```{javascript}
                        removedAdmins.forEach(function(name) {
                            allPromises.push(function(){
                                return GroupsService.removeGroupUser($scope.selectedAdminGroup.id, name);
                            });
                        });
                        removedAdmins.forEach(function(name) {
                            allPromises.push(function(){
                                return GroupsService.removeGroupUser($scope.selectedAdminGroup.id, name);
                            });
                        });
```
instead


Why did you do that ? This should be the description for the code action - not the analyzer - it was correct before.


You should check that there is only one RankSpecifier that has a Sizes expressions.


I believe you can make the checks mutually exclusive (using `else if`) if you put the union one first. In other words, you'd check if `C & D` is related to `A` or `C & D` is related to `B`. In doing so, you will check all pairs, which is similar to checking `C` related to `A | B` and `D` related to `A | B`. You'd also end up checking the aggregation `C & D` against each target constituent, because of the structural checks lower down. I think this is more natural than checking all pairs twice.


Have you ever tried setting `T <- FALSE;F <- TRUE` first? That's a good one. :)


no idea why the function call with a missing argument wasn't failing


@m-mcgowan We should discuss different ways of doing these all encompassing > expressions as it will be problematic for systems like the Electron.  Perhaps if some code/test is platform based we can include it more explicitly with `#if PLATFORM==photon` (which is anything based off of the PØ, P1 modules).  Currently we have 3 platforms: "core", "photon" and "electron".


this was successful


TeamCity NEventStore :: Core :: Build [Build 5.3.0](http://build.prxm.it:8811/viewLog.html?buildId=3326&buildTypeId=NEventStore_CI&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 279 Build time: 00:02:14


@carlosfrodriguez no problem mate :-) i wish you a nice day!!!


Fine with me, thanks
Gérard


I think `compiler` and `interleaver`, or something like that.


Ok.


Nope.  I didn't merge the users.  Uhhhhh


We do  not need to implement `fit_predict(X)` if there is no optimization w.r.t. `fit` and `predict` called sequentially.


I'm pretty sure most of the time you can just use T and F if the argument is expecting boolean data. Also, it's just super easy these days to set it TRUE and FALSE when tab complete is doing it for you.


Good catch! Thank you!


Not possible, nvm


Is there any reason why this variable not set to private?
If this is intended to be private could we prefix this variable with "m" (e.g., mDeliveredPI)


private PendingIntent mSentPI?


@Gerard-Huet I have added you as a collaborator in this repository.
So you have all the editing rights now.

As you use mac, you may like to visit https://mac.github.com/ and setup github in your local system.

Once you add the repository on your local system, you can also send corrections to the same file. I can sync the file.
When I edit the file, you can sync and get the updated version.

github will take care of documenting the changes to the file over time.


Okay


I believe this should be `diff`, not `dif`.


Thanks for bringing this to my attention. Your pull request wouldn't actually fix #4545, so I've gone ahead and just added a check for the case where primaryEmail is undefined.

I think this approach will give us the best default behavior and still allows people to override the requested permission if they need to do so.


yup, That is likely why. One sec


pep8: W292 no newline at end of file


@flavour This is not to display location component of area unless there is record in area


private SmsManager mSms?
mSmsManager would be a better name perhaps.


I think removing the duplication is good, but my 'we're missing something subtle sense' is tingling (but that could be a false positive.)

In what context was the removed function used? Did it have an intended difference?


Can this string message be transferred to strings.xml?


FWIW, you can commit an empty commit: `git commit --allow-empty`

I've done this occasionally when testing CI services.


private BroadcastReceiver mSendBroadcastReceiver


But other workers use the same constants, with the same value, each one need to declare it's contants?
We could set the contants in the config/initializer/ or in the WorkerManager.


:ok:


set this to private please and prefix with "m"


How do you find this shit?

interactive governor (why none of the others?) has a random ass newline following a standard CSC equation of variable + variable minus a specified value (I guess before they were using ++ to add and -- to subtract)

bbut wtf does this have to do with GCC 4.8/4.9/5.1? I am le confuse, is this the ONLY thing that was preventing the kernel?


these are the terminal output from running [this version of get-dat](https://github.com/maxogden/get-dat/tree/5b6e1245a4b1cc74091910ea18e9a48c6cc1de39/markdown), specifically the importing json and importing files sections

I checked them in so I could get line based diff commenting. We need to do some changes to the UX of these commands, as they are kind of confusing right now.


Tag this "0.5.2"?


Nice stuff. What do you think about limiting the output to txt records? It seems strange to ask for an a record and get back something that isn't an ip address right?


The `58 B` here isn't super helpful IMO, we should show the # of keys imported in addition to the size


@lmr and @clebergnu , thanks, I'm going to use the definition pointed.


method names should start with a small letter


Most of the existing DNS tests launch dns *without launching weave*. So you can't just drop the dns launch; you must launch weave instead.


The merge was incomplete.


What happened here was the file import overwrote the `bob` key, but we didn't warn the user or even return a message. Should this be allowed without a `y/n` or a e.g. `--force`?


The same.


I made git init and then ./download_sources.sh. Now can I safely commit without altering the sources?


hahaha


Suggest breaking out of the while loop if the stats request is not for all ports.


The output here could be improved as well, to show some sort of progress


@clebergnu OK, let's do it.


please make this as a constant variable and reuse it in this activity


Awesome, thanks.


Absolutely, by default T equals TRUE and F equals FALSE. And as long as no one reassigns them they're fine. The fact that you can re-assign them by mistake easily is a reason why some people prefer TRUE and FALSE. I don't really care. Was just making a joke/comment.


Also very confusing to workshop participants was the fact that `bob` no longer showed up in `dat export`. It didn't make sense to them that `export` didn't apply to files (and I agree that it's confusing).


"android.provider.Telephony.SMS_RECEIVED"
make this as a constant variable


This produces a code block with line numbers. I think this will be better for copying and pasting for users. However, It doesn't show the results as well so just wanted to see what you think looks better and will be more useful for users.


:+1:

Looks great @Epicurean306 !!! we do have a styling issue on our home comp....


newer versions of GCC find more errors and potential errors.


method names must be in small letters


- templates/list.jinja2~
- templates/base.jinja2~

What are these for? Try to explain in the commit message, or don't include them. They are cluttering up the codebase. [Also try making your commits smaller and atomic](http://www.freshconsulting.com/atomic-commits/).


Is it about test machine having access or it is about build service account having access?


Xaxaxaxaxax


So here we import the same file twice, but without specifying key, so we get 'duplicates'. This is part of the workshop. I think this behavior is OK -- people will have to grasp the concept of a key. Though one confusing this is that we use `--name` when working with files and `--key` when working with rows, we should just pick one.

How about getting rid of `key` and just calling everything `name`? Would that be more intuitive?


Should this be `UIButtonTypeCustom`?


In May 2012 I strongly advocated that if you are specifying a WCS section of a Sky frame that the RA should always come before the Dec as there is no way for the person writing the section to know what the position angle of the map is (and neither should they care).


The same reason why there are separate but identical definitions of RgbPacket and DepthPacket. And the templates around {Rgb,Depth}PacketProcessor explicitly instantiating {Rgb,Depth}PacketProcessor with {Rgb,Depth}Packet instead of having a single polymorphic Packet at the top level of PacketProcessor makes it look like there is some design objective here to separate Rgb and Depth. I followed that separation design though I don't quite understand it.

If your question is why I have PacketBuffer and Packet. I think it is good to abstract PacketBuffer from Packet. There is nothing in {Rgb,Depth}Packet that makes them polymorphic, only PacketBuffer needs subclasses.


Merge de varias CRs
IDs: 8, 10 e 12


This import was a no-op, but the output doesn't show that at all. Maybe the output for `import` could show the number of rows added and edited so that people can immediately see a summary of what their import did?


Path can have spaces. Should this be escaped with quotes?


привет)


Шалом


It's also confusing IMO that we duplicate the key in the output (e.g. `alice` and `bob` appear twice in the rows above).


Oh, the coding funnies. :) I'm apparently still too much of a newb to pick up what your laying down sometimes.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 101](https://teamcity.labkey.org/viewLog.html?buildId=258805&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:04:41
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

```


HELLO


Xaxaxaxa


test


ппц кто так пишет


I think that not all result error toast messages are helpful to the user (e.g., Null PDU, Radio OFF --> these are alien to me).

Maybe you could define a generic error message for this such as "There seems to be a problem sending this message. Please try again."


@Bernardstanislas Need a store default value for a node or a way to populate them with default without triggering events


I just missed that and should have fixed it in the following commit.


The export progress?


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


No retry?


I like `--key`. name doesnt seem unique to me. I agree it should be consistent though.


Might be nicer to break this out as a separate change.  It's not related to updating to 7.1.3 directly, right?  Just ensuring there's a release package?


Add verbose at required places.


We should also bump the version in the shell script right?

Otherwise looks right to me.


Yeah, I think it's fine as long as we have a way to get just rows or just files


How about `id`?


Looks good to me


rv = std::move(queue_.front()) might be more efficient


Agreed that this is confusing


Yeah, agreed. The only thing is building applications that do update operations on top of the cli is difficult if you don't know the key from a table export


getParentActivityIntent() may throw a NPE.. please add necessary checking


Hard coded powershell port number.


Yuan, your `Build Status` icon is pointing to my personal fork, please update it to point to PacificBiosciences.


Yes, I'm afraid Dilly's commit already does that:

```+EXTRA_HEADERS += $(COMMON_BUILDOPTS_H) $(FLOW_BUILDOPTS_H)```


```powershell
[Environment]::GetFolderPath('System')
```


```powershell
[Environment]::GetFolderPath('System')
````


Nice. Editing for me is now about as quick as in the Nuprl structure editor. This is great.


These are ripping out. Lost the notes - boo


why did I do that?


`jimhester/covr` installs the coveralls package from github, and `library(covr)` loads it so we can run the coveralls function.  Undo your changes to the .travis.yml file, and the build should work just fine.


add space after chr


Yup, the build triggered: https://travis-ci.org/robjhyndman/forecast/builds

Coveralls won't run correctly though.


okay!


This might result in more cases than we're interested in. There are two cases: if the guard set does not contain an `otherwise`, then we have to assume none of the cases have been covered, otherwise all of the cases matched by the binders are covered.


are you intentionally mixing `jqLite` and `$$jqLite` in the code above?


The patch was the one received from David Coffin (author of dcraw) off channel before making the vulnerability public if I remember correctly.

I think the `len < 2` check is related to the line

    len = (data[2] << 8 | data[3]) - 2;

where len can be USHORT-MAX-1, which may result in a possible read error in the following `fread()` call - but you can easily trigger a read error even without underflowing. I don't see any security related bugs triggered by omitting the check, len is an unsigned integer, and the data array is of size USHORT_MAX+1. Neither overflow nor underflow can occur.

The fix in netpbm and Fedora is somewhat cleaner than the upstream fix in dcraw, but all are good from a security perspective.

For the interested: Rawstudio plans to remove dcraw completely from a future version and rely solely on RawSpeed - and *maybe* calling an external dcraw binary as a fall-back option to support ancient file formats.


Olá Diego,

Este código acima foi obtido do fork da Shirlei, certo?
https://github.com/shirlei/helios-server/blob/master/helios_auth/auth_systems/ldapauth.py

A Shirlei já finalizou a autenticação com Shibboleth (para autenticar na federação CAFe), se desejar, talvez fosse interessante monitorar (ou mesmo fazer um fork) do projeto dela.

abraços


Sounds good to me