Can we guarantee that the `AuthorizationModules` will be returned [here](https://github.com/phenotips/phenotips/blob/issue-1822/components/security/authorization/src/main/java/org/phenotips/security/authorization/internal/DefaultAuthorizationService.java#L58) in the order that they are added in this test?
In other words will this code also set up that the modules will be returned in order of moduleOne, moduleTwo, moduleThree in every case:
```java
this.mocker.registerComponent(AuthorizationModule.class, "one", this.moduleOne);
this.mocker.registerComponent(AuthorizationModule.class, "two", this.moduleTwo);
this.mocker.registerComponent(AuthorizationModule.class, "two", this.moduleThree);
```


What type of parameter does setPrecision($precision) take?


Aaah, the bliss! :DD


What's the difference?


This is a great addition.


А! Я увидел warning в pdflatex.


Whats the formatting problem?


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1697 of 1700 tests passed)

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


Doesn't match the rest of the OpenBTS's format for tabs.


How does this work for you?
(using babel/webpack, it introduces a runtime dependency on `babel-runtime`)


This is a little disgusting.


Reinterprete cast is more explicit about pointer integer casting. C++ coding style.


@kltm  thanks Seth, I just removed the `rm -rf node_modules` after your suggestion. The reason I added rm -rf was in hope of a clean npm installation. I couldn't see the http://beta.monarchinitiative.org/page/phenogrid being updated this afternoon after I made changes to both phenogrid-master and monarch-app master. I also sent an email to Larry. So I was wondering if npm install didn't overwrite the old code with updated github repo. I did some local testing and it turned out I was wrong about the npm install. But after I added `rm -rf node_modules` in install.sh, which triggered a rebuild in jenkins, I could see the page updated. Really confused and donno why.


Note: this was removed, because Reset() some lines above already calls ClearGridIndex()


Thanks. Missed that.

Also reminded me why I don't like aliases: now I need to know two names for the function. Just knowing one that I use myself isn't enough, because if I read someone else's code, they can be using the alias.


Doesn't work on windows?!


Would you suggest just //?
What's the difference?


Comment on what we are doing here?


These are for change notification interrupts.


Stefy esta buenísimo este post! :) Me encantó!


just checking: Is the idea here to not expose the Fields property explicitly when you have a concrete message, because there's no real use for it in that case?

That probably makes sense, but would seeing the Fields property actually bother anyone?
For first 5-minutes experience, this is probably a bit counterproductive, because you are hiding one of the features from the early-adopter.

I am actually not 100% sure if we actually need the concept of IReflectedMessage. If "Fields" stays to be the only member of said interface, it might be just worth moving it to IMessage and getting rid of the
extra layer in the inheritance graph.


Yeah. I was looking to avoid adding any more dependencies. I'll just stick with this.


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


запятая перед будто


it will also spawn multiple extra processes - at least 3 forks, while using this `crude_grep` will cause just one - to run openssl itself

IOTW, using bash for that will be faster and less demanding on the cpu


There are inconsistencies in OpenBTS's tabbing standard. TransceiverRAD1 is all two spaces for some reason.. I did whatever the standard was for that particular file


Он пошёл за мной, когда я вышла на улицу


Note: This was removed, because Reset() some lines above already calls ClearGridIndex()


обернулся


Compare the two files across master and glpi084 ... many differences mainly on networkports!


@keithgee I'm waiting for you to accept invite.


Partly because there's little use for it, and partly to avoid creating a conflict with a property called Fields - such as exists in Type. Again, I only did this when I saw it causing problems :) I do like to reduce intellisense cognitive overload though.

+1 to sanitizing the interfaces when we're closer to completion. It depends on whether there might be other IMessage implementations that don't support reflection. (I suspect not, but that was the motivation for separating them for now.)


> А! Я увидел warning в pdflatex.

Всё правильно, только в warning-е и проблема, не люблю я их. Чисто визуально вроде бы ни на что не влияет.


@rcpeters I had to add the following to be able to add the attribute put-code to the XML -->
<xs:attribute name="put-code" type="common:put-code" use="optional" />


Thx!


косяк с грамматикой
она была прикована к нему уже какое-то время


I needed Integer out params (Integer is immutable, can't use it), the java workaround for this is to use AtomicInteger


https://vimeo.com/112319901
Watch from 14:24 ( or watch the whole video, it's kinda good :palm_tree: )


develop (kbenne) - x86_64-MacOS-10.9-clang: OK (1237 of 1237 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)


@yuanzhou cheers! I get nightmare scenarios in my head since I sometimes link within ./node_modules/ to other projects to play around modified versions (and, honestly, since I've bricked machines before with "rm -rf" shenanigans).


Yes sorry. I cant fix it till Friday
Rob.

-------- Original message --------

From: segepe <notifications@github.com>

Date:16/07/2015  02:24  (GMT+12:00)

To: rob42/freeboard-server <freeboard-server@noreply.github.com>

Cc: rob42 <robert@42.co.nz>

Subject: Re: [freeboard-server] Added pcb announcement (6336dec)

Hi
Cant reach your site at www.42.co.nz, is it down ?

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


There is only one AGPL, and it's based on the GPLv3.


краем глаза же)
ну и поменять местами части предложения, чтоб по-русски было


Two things

1. I want to upgrade pylabrad however considering the problems already experienced from this I wish to be careful and test it first with a new virtual env. This breaks my code so I do not wish to push it to master yet, however a seperate branch can be made.

2. In the future can you separate pull requests with space management and pro-grammatical changes as it took a while to find the change to datavault and I also think separating this makes logical sense as far as git goes


Please fix indentations here.


Его видно из окна моей комнаты


+1 just got one of these myself. http://i.imgur.com/xOKBqc1.png


:shipit:


Submitted upstream
https://review.openstack.org/#/c/202276/


посмотрела*


I think the comment above the function should be adequate?

    // We use full path for the source and local for the destination to use relative
    // path for the symlink


We don't seem to be testing import with the wrong passphrase.

It would also be interesting to test import with the wrong passphrase on only one of the PEM-encrypted keys.


поначалу
самолетиков


Fixed in https://github.com/CUTR-at-USF/onebusaway-application-modules/commit/84f7a6eb92b92ac0e817349729ff192c5e820735.


Please move interface functions definitions below here. Users don't want to dig into lib source to find their definitions.
Add comments to them in jsdoc format. https://www.jetbrains.com/webstorm/help/creating-jsdoc-comments.html


само собой, ведь у него бла бла


https://github.com/JeromeHinds/Testproject/commit/e3d809d4478510335ff330da981bb3fc1324068f is where I added more info


Should work because the archive/zip docs say the separator is always "/" in names it exposes from the zip headers. I had a comment about this somewhere but I missed it here. I'll add a comment here.


Updated https://github.com/rackerlabs/api-site/tree/rs-heat-dollar-vars


Thanks @cagryInside, looks good!  Bumping `serialVersionUID` to `2L` in `TimepointPredictionBean` is the only TODO I see here.


Не нужно "чтобы"


maxOutput


This if could be moved inside the template definition around the source declaration.  No need to define another template block when the only difference is the source.


I'm not sure, I will check.


запятая перед это


Actually a modified version of LuaJIT 2.0


if required then optional? isn't that the opposite?


запятая после самолета


запятая перед заметил


fixed with https://github.com/mehanig/stepic_manual/commit/3ea58faf899c35527dc3b4895d64111224380a42


запятая перед будто


decompress


Given how frequently this happens, is regexp efficiency an issue? Should we just check if `id_or_str[0...2] == 'on'`? Not sure...


One thing that I just realized is that a normal keyfilestore can operate on root keys, when given a gun of "root_keys".


nit: I'd prefer
if (!IsMapEntryMessage(...)) {
  MessageGenerator messageGenerator(descriptor_->nested_type(i));
  messageGenerator.GenerateStaticVariables(printer);
}

instead of the continue.


запятая перед откуда
мб лишнее местоимение он


this doesn't follow our standard for hierarchy and we standardize on dot notation vs slash.


Welcome!


ok i will try it later


this would need to be a defined property.   best to `docker-registry.framework-name`


extra space


На это есть причины, но для тех, кто о них не знает, он выглядит весьма причудливо


You want just a TLS version intolerant in general or example for every type of test (that may be rather hard)

I'm slowly working towards a test server that we will be able to run locally for checking that


pre and post notes are always recommended


Why an anonymous function and not an actual function?


Back again! My Precision is doing its job, cause basically everything seems to work! But there´s one thing thats pinching my nerves: With some Apps (chrome/flash) and during boottime the screen is flickering, so animations are not fluid!

Any Idea what went wrong here?


I think you need `while (input < inputLimit)`


any properties required would need to be defined in a required property array


Have you done a pull request for the main branch? Would be nice to have this there.


запятая после холмов
казался -> был


Comment: "List all files but no symlinks" ?


@aransfor ,  I think the separate branch should be made for code that is not up-to-date with the latest version of pyvisa and pylabrad.


EvansHVACObjects (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


yeah! definitively the opposite.


I suppose a ML post is in order, yes.


возобновляемых источников энергии


No.


Literal arrays!

```factor
${
    "codesign" "--force" "--sign"
    "Developer ID Application"
    cert-path "Factor.app/
} short-running-process ;
```


Come on :)


@ibdknox  Where does this version originate from, what source was used to build it ?


i don't use the integrated wrapper script... i have another. but video is working now.
if you have video disabled in the settings you have to enable it and restart skype in order for the cams to be detected.


The idea of creating an N-dimensional general purpose computational geometry library scares me. I would want someone really dedicated who can follow through, as there are many complex aspects to consider.

Remember that every line of code is a maintenance liability. Much better would be to find and use an existing computational geometry library written in Java. Unfortunately I could not locate a good one after 10 minutes of searching.

If we do keep this code, let's please name it `Polytope` (English spelling). And `Polyhedron` rather than `Polyeder` if that type ever gets created.


you don't need the cast to `int` anymore


i'd be surprised if this worked?


You must be kidding :)


прийти не очень нужно. просто не смогу встретить тебя


When you create methods of DynStr instance this way, you (at least) create new reference to each DynStr method in each DynStr instance. You could use common DynStr.prototype to store method references in it and thus easily save memory because prototype is a single object with single set of references to DynStr methods.
Also I use code style for private and protected object fields. Such field names should start with _. Let's rename them. I'll tell about other ways for private/protected fields creation on skype meeting.


How come you dont gen the hashes automatically?


lol i don't know where y'all keep getting that photo of me but i love that it's a meme now :santa:


add comment that this is the end of input due to how the format works


Definitely the wrong way to do this. Should have used capitalization & punctuation. @stillyslalom - fix


ошибочно читают как "Мидори"
если я правильно поняла, иероглиф один и тот же, чтения разные


Before:

```
mnunberg@mbp15 /tmp/jsonsl/build-clang $ time ./bench-simple share/auction 100

real	0m4.213s
user	0m4.149s
sys	0m0.021s
```

After:
```
mnunberg@mbp15 ~/Source/jsonsl/build-clang $ time ./bench-simple share/auction 100
SPEED: 463 MB/sec

real	0m2.924s
user	0m2.871s
sys	0m0.019s
```


Аои


Comments addressed in bf05dc777c36e4e0e5c5cbdfa24ba1952278c467


need bounds check


It originally needed a local variable, but that didn't work with literal
arrays. I can clean it up

On Wed, Jul 15, 2015, 13:49 John Benediktsson <notifications@github.com>
wrote:

> Literal arrays!
>
> ${
>     "codesign" "--force" "--sign"
>     "Developer ID Application"
>     cert-path "Factor.app/} short-running-process ;
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/slavapestov/factor/commit/a488d47dfb0d07abae9a5f497a45fbe05ebaef08#commitcomment-12189183>
> .
>


This links to the algorithm for containment for polygons in 2D... but where is that actual code? It's not here in this method! The link should go in the javadoc of whichever method does contain it.


This is not new


I'm on the fence with reason no.1, but the conflict with System.Type makes it pretty clear we need a separate namespace for WKT. Thanks for the explanation.


Sure, happy to fix that up.


depends on what `Equals` actually means:

```php
<?php

class Foo {}

var_dump(new Foo() == new Foo()); // true
var_dump(new Foo() === new Foo()); // false
```


Они стали


As we discussed, I would force the reconf rule to always run it verbose.

```patch
diff --git a/tools/build/Makefile.rules b/tools/build/Makefile.rules
index adb2ecf..2037ee0 100644
--- a/tools/build/Makefile.rules
+++ b/tools/build/Makefile.rules
@@ -275,7 +275,7 @@ reconf: $(DEPENDENCY_SCRIPT)
 	$(Q)echo "[re]running dependency-resolver..."
 	$(Q)$(RM) -f $(DEPENDENCY_FILES)
 	$(Q)$(RM) $(DEPENDENCY_CACHE)
-	$(Q)$(PYTHON) $(DEPENDENCY_SCRIPT) --compiler=$(TARGETCC) --cflags="$(DEP_RESOLVER_CFLAGS)" \
+	$(Q) V=1 $(PYTHON) $(DEPENDENCY_SCRIPT) --compiler=$(TARGETCC) --cflags="$(DEP_RESOLVER_CFLAGS)" \
 		--prefix="$(PREFIX)" --definitions-header="$(DEFINITIONS_H)" \
 		--cache="$(DEPENDENCY_CACHE)" --kconfig-gen --makefile-gen

```


What's the purpose of cljsDeps.js ?


Please use === instead of == nearly everywhere in js code.


таким образом -> поэтому
запятая после город


Wait, how?


Yeah, I was also wondering if one of the reasons was that you ran into a conflict with a field name. Explicitly implementing Fields makes sense.


I don't think this is the right fix, especially since the certificates file is not guaranteed to exist at that path on all set ups. You should instead use `:cert_store => OpenSSL::SSL::SSLContext::DEFAULT_CERT_STORE`.


Name of this function doesn't imply any side effects inside it.


GCC:
Before
```
mnunberg@mbp15 /tmp/jsonsl/build-gcc $ time ./bench-simple share/auction 100

real	0m3.668s
user	0m3.643s
sys	0m0.018s
```

After:
```
SPEED: 309 MB/sec

real	0m2.889s
user	0m2.842s
sys	0m0.016s
```


перестройка заставляет ссср вспоминать, хорошо бы как-то иначе сказать
хотя если не получится, то не критично


You can remove the parenthesis here now


Works with RC3 now. volume could be mounted and the processes on other machines can apperently acces the data on the other volume :+1: Thanks a lot!


This is good. We do not want the commented line appearing on the homepage of our project.


Not clear what this function does here.


LGTM.


develop (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Indentations again.


just glad to share the bits of info I have picked up.

On Wed, Jul 15, 2015 at 2:06 PM, Lunatixz <notifications@github.com> wrote:

> Thanks, waiting for more updates so I can push them all at once...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/halcyonforever/XBMC_Addons/commit/e1d0d13aba45f7edf6f6ade51131390731ac75ad#commitcomment-12187111>
> .
>


Verified Stream.Builder.TryWith also works now. Thanks Vesa.


It's unnecessary indeed, but it's optimized out anyway. I should change it.


:smile: i'd like to have your avatar up there but apparently twitter can't keep the urls around!


What are we doing here?


мелькнуло


Could do this in a later commit, there are several other places which use 0x30


запятая перед будто
будто это лопасть отлетела от ветряка


запятая перед что


Makes the comparison line longer, but I agree.


check that output limit is within the bounds of the output


前面应该是在Load 各种输入文件和设置各种参数
现在这里开始计算


Did the elimination of the idl for generator already make it in and I just missed it?


We should bail if the key is not encrypted. This will break our internal assumptions of how root keys are being handled.


@eteq Halp!


терял свой импульс - ниочень
что случается с самолетиками?
замедлялся - вроде нет
я хз) но надо переделать


подпрыгнул


Also we probably want to test this behavior.


Blueprint doesn't actually need a go variable named BinDir to match a ninja variable named BinDir.  That pattern is copied from pctx.StaticRule, where you need to reference the result from go to create a BuildParams that uses that rule.  I've started using a func init() instead of a var for StaticVariables, which would let you have BinDir be a valid go variable and a ninja variable:

    var (
        BinDir = filepath.Join(bootstrapDir, "bin")
    )

    func init() {
        pctx.StaticVariable("BinDir", BinDir)
    }

or if you prefer:

    var (
        BinDir = filepath.Join(bootstrapDir, "bin")
        _ = pctx.StaticVariable("BinDir", BinDir)
    }


@Cl3kener really where's a linaro or aosp script that uses sysroot in the exact same way that sabermod does?


вторгся
в её личное пространство
запятая перед будто


Видимо, внезапное бла бла


Originally, this function used several variables from the scope of ExportAllKeys, so it was convenient to have a closure. In its current form, it only uses outputPassphrase, so I'm fine with making it an actual function.

addKeysToArchive uses three variables from ExportAllKeys: dedup, zipWriter, and tempBaseDir. I'm fine with moving it out of ExportAllKeys and making these explicit parameters if you think that's cleaner. It's your call.


Added locally.


It seems as though they are being returned in order based on their role hint strings


Nevermind, I see the problem, you add a ninja variable to bootstrapDir later, which makes the go BinDir variable useless.  I suggest not exporting it, since referencing it from go is always going to be a mistake.


This function doesn't belong to lib interface so you can choose both LOCALES key and value as return value. I'll rather choose key because it belongs to lib interface and should be more stable.


окаменело как-то жестковато
застыло мб


@kykdev sabermod.com

You can also follow updates from G+ and xda communities.


(summoning @robinsch for reply to joschiwald)


This should be checking for `this._pagination` directly instead of calling `this._getPagination()`, since we don't want to create a pagination instance if it hasn't been created yet just to destroy it. Same for the tab view.


Did you ask on the systemd mailing list or IRC channel about this?

Not a huge deal since it only happens when the system is being migrated, but if we end up needing this, it would be yet another point to use a more standard approach like the update target.


появившиеся
и выделить запятыми причастный оборот после определяемого слова


попал к тебе?


Extra " ; "


This fixes issue #1.


Please keep the WeakAuras name for the table name instead of Internal


In Nodejs it is common to write it this way:
function onDictDownloaded(err, localeDict)

Let's use nodejs agreement. err is instance of some object that represents error. It is null if no error has occurred.


I would say it makes it more obvious to the reader what is in scope and what isn't.


запятая перед как


Since the `pageHeader` element is passed as an attribute, then it's not destroyed together with FormBuilderPages' contents when it's destroyed. That's fine, but we need to make sure that we destroy the events that were attached to it in this case, since they won't be detached automatically like the ones you've attached to contentBox.


I guess I don't understand this comment.


100% fine with this commit


At the moment I wasn't completely sure that go calculates it the same way. It turns out that it does. Those lines are the part of the reason why the commit description is what it is.


This commit is deprecated by https://github.com/stepanv/hk2/commit/c60e0f846b782c4f66a5b8c113789a9345131543
Unfortunately, none of those commits is able to cleanup all the ThreadLocals when the servlet container is under a heavy load. Probably a concurrency issue.


oka. Let's avoid having empty loops in our code. thks


Did you measure a performance difference with that change?


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (650 of 650 tests passed)

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


Indentation


No, I do think it's likely a systemd bug that it doesn't properly reconfigure the endless.mount and local-fs.target units that are already active. Don't merge this and I'll poke around a little more tomorrow.


Please do it for those that are public.
And please do not waste too much time on this.
thks.


The non-root key store has the base directory `private` and root key store has the base directory of `private/root_keys`. This means that calling ListFiles on the non-root key store will also list the files in the root key store. I deduplicate here to avoid including the same file twice in the zip.

I guess it would be possible to only call `addKeysToArchive` on the temporary non-root store since we know it also contains the root keys. Maybe this is cleaner. What do you think?


`_createTabView` should come before `_createUntitledPageLabel` in alphabetical order.


Glad to hear it's all come together finally. Let me know how it progresses and if you run up against more features you'd like to see.


Where is this token used? Is it not sensitive data?


@drgroovestarr don't derail the topic.  These scripts have nothing to do with linaro.


Ya lodash v4 is removing most of the aliases.


This uses a map as a set data structure. struct{}{} is an instance of an empty struct. It's how we mark a file as present in the map.

I've seen this idiom quite a bit in go code, but I agree it's not the most intuitive thing. It would be nice if the language had a native set datastructure.


I actually think we should strongly consider creating a `tuf_keys` at the same level as `root_keys` so we never have a filestore of keys being able to operate on roots.


Sure.. I guess it was a problem on how github presented me the information.
We talked about it, please majke the suggested changes.


Autosave made right.


And by strongly consider I'm saying: we should do it :)

It makes our protection against keystore escape even more meaningful.


Thanks!


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


It's fine the way it is, just looking for opportunities to use literal arrays since we were discussing the change with the new parser.


вместо знака вопроса почему-то запятая)


謝謝 :sunglasses:


```C++
if (deviceAddress != NULL && checkForConversion && !parasite){
        unsigned long start = millis() + delms;
        while (!isConversionAvailable(deviceAddress) && (millis() - start < delms));
```
would be the right way todo it


It's not safe to have 2 widgets use the same content box, since they'll both be adding things to it. FormBuilder should add an element to its content box for FormBuilderPages and pass it here instead of the content box itself.


I say we follow my advice up there and remove all dedupping.


only thing I can think of that still uses it are fishing rods so we may just name it Fishing Lure?


ok


@jmartinm Good finding! It's now possible to drop some modules from the final file, like `text`, `hgn`, `jsx`, etc. that are only useful during the compilation. See: https://github.com/jrburke/r.js/blob/master/build/example.build.js#L316-L329


I was actually thinking of providing more info if the test failed in unexpected way, so the `True` and non `True` is the only way to say if it has passed or not that will continue to work


also would this not work with `dedup[fullKeyPath] = struct{}` ?


From what I saw this._pages is being set in the `initializer` function, so it should never be null. Can you check if the test coverage is showing that this if is never false, and if so remove it?


Use {this} instead of card name.


Got it, thanks!


`struct{}` is a type, and `struct{}{}` is a value.


This looks like a hack. Strings in our language can't consist only of control characters. So let's warn that there is a problem in dictionary file. And then we should make this bad template just static empty string ''.
Move this check out of loop body. Check i after the loop has ended.


Thanks for your quick reply

I can wait

BR

Gunnar

Från: rob42 [mailto:notifications@github.com]
Skickat: den 15 juli 2015 22:39
Till: rob42/freeboard-server <freeboard-server@noreply.github.com>
Kopia: segepe <perssongunnar@hotmail.com>
Ämne: Re: [freeboard-server] Added pcb announcement (6336dec)

Yes sorry. I cant fix it till Friday
Rob.

-------- Original message --------

From: segepe < <mailto:notifications@github.com> notifications@github.com>

Date:16/07/2015 02:24 (GMT+12:00)

To: rob42/freeboard-server < <mailto:freeboard-server@noreply.github.com> freeboard-server@noreply.github.com>

Cc: rob42 < <mailto:robert@42.co.nz> robert@42.co.nz>

Subject: Re: [freeboard-server] Added pcb announcement (6336dec)

Hi
Cant reach your site at  <http://www.42.co.nz> www.42.co.nz, is it down ?

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

—
Reply to this email directly or view it on GitHub <https://github.com/rob42/freeboard-server/commit/6336dec29fbc44d6a9d110f294a20f2f8f1241b1#commitcomment-12188972> .  <https://github.com/notifications/beacon/AMu5aeSm0ljFQjSOePyG_LL1666ot3asks5odrxegaJpZM4FZHKo.gif>


Caused issue #566


ya it actually works, i just tested it out ;)


This is something like 'get dynamic string type' indeed.


... you could also make it a `char` array - while it's unlikely to make much of a difference in speed it should reduce the D$ foorprint which for larger documents may be beneficial.


@pbeeler excuse me,  you're on my Github genius. I can put whatever I want
on these files because it's my Github.  Whether or not you agree is
irrelevant. I have tried to be respectful towards you and you refuse to
extend this courtesy in return. I can see why the amount of developers
using SaberMod is dwindling,  you are acting like a child.
On Jul 15, 2015 2:06 PM, "Saber-Dev" <notifications@github.com> wrote:

> @drgroovestarr <https://github.com/drgroovestarr> don't derail the topic.
> These scripts have nothing to do with linaro.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/drgroovestarr/linaro-scripts/commit/0200814c2afa0b0b94bab51e93032b478f100eb2#commitcomment-12189522>
> .
>


Addessed most comments in 25ef4e5. Will add the `tuf_keys` subdirectory in a separate commit.


This should be `NetClock::time_point`


great comment


remove `.myapplication` from your package name


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

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

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

```


Do no to this


Как ни странно настройка android:checkableBehavior="single", которая должна обеспечивать выбор только одного пункта не работает.
https://code.google.com/p/android/issues/detail?id=57990


yeah I tested it, it hashes fine


@ricardotk , @dorileo has done some tests yesterday.
Our conclusion is that we can go with submodules, but it would be better to use sparsecheckout.
It doesn't seem to be something configurable on .gitmodules, so we would provide a script that could do
git submodule init && git submodule update using sparsecheckout

On duktape-releases there are tags with code instead of tarballs, and there is a directory src/ with 2 files, duktape.c and duktape.h, that is exactly what we want. So they can be placed on src/thirdparty/duktape and everything else should just work.

@dorileo , am I missing something here?


Ah, thanks, that makes a lot of sense. I mixed up with npmignore and gitignore.


This line seems to be a bug. Phong shader should not output normal as color.

![captura de tela de 2015-07-15 18 16 36](https://cloud.githubusercontent.com/assets/3521158/8710111/bf24739c-2b1d-11e5-8c64-a3fe598666cc.png)


The second parameter lets the user choose whether this parameter will be a 32bit signed integer or a 32bit float - with the original SDK and rSDK this was easy to accomplish, but in EDIF I'm going to have to get creative.


Is it going to work with external modules?
Why would they use sol_flow_node_type prefix on their options struct?


looking good .. slight improvement would be to do the `ArJdbc::AR42` check at the method level ... so that it only happens once while building the methods (more code but less runtime overhead) e.g.

```ruby
 if ArJdbc::AR42
   def limit_for(limit_or_node)
      limit_or_node.respond_to?(:expr) ? limit_or_node.expr.value.to_i : limit_or_node
   end
  else
    def limit_for(limit_or_node) ...
  end
```


Oops


Thanks, this fixed the issue


r+


Yeah, will do that.


Is this just a problem with btrfs? I ran Aaron's repro (https://github.com/aaron42net/dockertest) on a test cluster I have running docker 1.6.2 and didn't get the fatal errors.


OK, I misread the PackageContext.Import documentation -- I'll drop the go export here.


develop (kbenne) - Win64-Windows-7-VisualStudio-12: OK (1240 of 1240 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)


saw approval from graydon
at https://github.com/stellar/stellar-core/commit/a8a56ec77acdef93b17b358583bda6c8eaf4b806


merging stellar/stellar-core/pgenv = a8a56ec7 into auto


stellar/stellar-core/pgenv = a8a56ec7 merged ok, testing candidate = bc6c9052


Are you going to make a pull request to the docker repository with this fix as well?


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1237 of 1237 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)


@afterthought Nope, we were seeing this on overlay. It seems to be partially a timing issue and what DockerHub CDN you're getting. We're hoping the fixes that landed in Docker 1.7.1 will address this - we're looking forward to testing it once it lands in the stable channel.


What was modified in this blueprint? It's hard to see when the commit that changes it also introduces it.


Filtering: I mean (for a start)... A floor which could be selected in the right panel, and then could be hidden by clicking a button. That way, for example, viewing "level 1" (only) could be permitted.

In a second time, I'd like to be able to hide ifc types: this could lead to show only systems (pumps, pipes, etc...). Extended windows version could do it, with controls on the lower part of the right panel.

Basic checking: I mean... be able to check a model quality (collisions detecting, counting and viewing...). I suppose those repo are about to try to do it:
* [IfcValidator](https://github.com/opensourceBIM/IfcValidator)
* [mvdXMLChecker](https://github.com/opensourceBIM/mvdXMLChecker)
* perhaps [modelchecking-snippets](https://github.com/opensourceBIM/modelchecking-snippets)

I think filtering should be easier to start, don't you think so ?


Why is this code commented out?


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

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

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

```


Not isFork?


Thanks! This should be good like this.


@FTG-003 I added a link to the dashboard, so that appears on http://peeragogy.github.io/ now.  I figure it's "about time" but if you prefer it to be incognito for a while longer we can comment that out.  Thanks again for developing this resource.


defaults filling and validation added


woops!


@ChronoMonochrome No, I was wrong
check this: https://github.com/Meticulus/android_device_samsung_codinalte/commit/271f1d7803afb889793035e78467e9ff1fd26d1f
apparently don't is necessary add the include headers :)


Great! A simple verification would be to launch the tool with default settings, make sure it comes up on the default port. Then launch it again with -D... to override the port number. If it launched on the specified port, you know the config parsing through Tool is working.


Weird... `go build` would still compile without this?


I've only just noticed this. The reason I removed this bit of code was that it caused breakage if you were using elementary OS but not using Pantheon or XFCE. Unless I've missed something, isn't this just adding the regression back in again?


try
    --geometry 50%
to keep window width static (Idea for mpv would be to implement --areasize (ploščina))


and it also -> and is also


defines


"ubuntu, while the version-specific" (combine sentences and add -)


Committed b660f36 to add this check and add testing around it.


across versions


Found this today not applied as of 2015 July 17th, and spent time on look for this PR is already made or not. Hope other people not doing same thing.


Thanks for making these changes mate. It was an annoying issue that I'd kind of put into the too much effort basket.


Yes, I plan to merge later this week.


indentation


I actually accounted for this by making an option to disable the swt file
chooser in the settings window. Did that not work?
On Jul 15, 2015 12:13 PM, "Danijel Koržinek" <notifications@github.com>
wrote:

> I'm really sorry. But I had to temporarily revert all the changes
> regarding SWT. The problem with this is that SWT is system dependent and we
> would need to either deploy multiple versions of the program on the website
> or pack all the various system libraries in the program and load them at
> runtime. In any case it's a huge hassle and not sure if it's worth it at
> the moment.
>
> If you still want, we can make a branch for SWT and move everything to
> that, including the various system-dependent versions. Otherwise, I'd
> recommend checking out Apache Pivot or JavaFX instead. What do you think?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jmc2obj/j-mc-2-obj/commit/d0d10929afabd3426d330003b12e25233ec1e1dc#commitcomment-12187254>
> .
>


remove spaces between imports


This version does not work for `Microsoft.Compute`. I have used `2015-06-15`. Should we build a lookup table for all operations (and providers) to get required api versions? This way the caller does not need to remember what api version to pass in.


Ah i forgot that :P We need a better detect_de function, i will open a new issue for that!


This is weird - it looks like github ate a comment. Reposting (apologies if you've seen this already):
This still worries me a bit as I'm not convinced it'll be noticed (or remembered for 1.4). Could it be protected with something like:
<pre>
if [ -e "${SPARK_INSTALL_DIR}/bin/utils.sh" ]; then
  sed ...
fi
</pre>


Thanks :+1:


Would be better to use os.WEXITSTATUS here I think, no?


Was the code without the `linux_distribution()` function not working? The tests I did suggested that `environ.get('DESKTOP_SESSION')` would be enough


thanks!


I will test and merge.


@Foggalong It was not working in my case. The result of `environ.get('DESKTOP_SESSION')` was nothing


instead of status >> 8?

On Wed, Jul 15, 2015 at 2:41 PM, Florent Viard <notifications@github.com>
wrote:

> Would be better to use os.WEXITSTATUS here I think, no?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Tim-Anderson/s3cmd/commit/47fdbcc21444801fd9c9f6c7afd2e7ea8e06ef6d#commitcomment-12190144>
> .
>


Another question to bring up wrt consistency and the API for the future.  In other places we are very consistent about the cancellationToken being the last parameter.  Here we pass it through the context object.  Feels weird to me.  But i def get keeping these consistent to start.


This is detail, prefer camelcase name on v8: cf https://github.com/OCA/maintainer-tools/blob/master/CONTRIBUTING.md#odoo-python-class


Does that mean when you op all, the bcast msg "name opping everyone on the server" will have the [ImmaFreedomMod] prefix?
@Adaptified


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


u can fix it, don't you?


It wasn't possible to compile it and make a working build for all the platforms. Again, this can be checked during runtime and switched accordingly, but it should be tested first.


Also closes #354.


So developers can use ./sys? are u sure?


As long as possible, can you create one file per model.

cf https://github.com/OCA/maintainer-tools/blob/master/CONTRIBUTING.md#differences-with-odoo-guidelines


I wouldn't say it's the reference counting aspect that we're using, more just the enumeration of the attributes (for proxy_to_tp_traverse).  Now that I think about it, I think `proxy_to_tp_clear` is a bad idea since there's no guarantee that it does everything that tp_dealloc does; I think we should just mark these tp_deallocs as safe.


i would rather keep the more accurate logging.


I don't like using `...` It is hard to read.  Can you use `from binstar_client import errors`?


This wasn't quite a good refactor. If `$this->parentSlug` is NULL (for example if line 121 sets it to NULL) this used to go to the else before this refactor, and now it doesn't. I propose changing this condition to:

    if (!empty($this->parentSlug)) {
       ...
    } ...


We need it for arrays such that `undefined[]` is considered a duplicate of any other array. For example, we want `[[1, 2, 3], []]` to get the inferred type `number[]`, not `number[] | undefined[]`.


How odd! What elementary are you running?


FINALLY


I'm not sure where this kind of configuration is supposed to go. From the docs I'm assuming this is the right place.

https://github.com/syl20bnr/spacemacs/blob/master/doc/DOCUMENTATION.org#structure

> Emacs built-in configuration or mandatory configuration

This isn't really built-in configuration, but it uses built-in functions/macros so maybe it qualifies?


Completely useless commit.
I want to remove it.


Allow application to override the default encoding for cases where, for instance, the MWL records contains a known encoding (non-ASCII) but do not have the Specific Character Set tag populated.


Lines 607:660 in server.rb should already be taking care of this. However, because of structural changes in  internal_wrangler things may have broken down slightly.


Yeah, we talked about that in the diagnostics design meeting a long while ago and decided to go with the CancellationToken on the context object. Having the CancellationToken on the context object means that the user will always have it if they create helper methods that take the context object and there won't be any confusion about which CancellationToken is which. I'll make a note of that so we can bring it up when we start looking at the API design. I'm just working on getting the basic shape right now.


Freya latest stable version with daily & staging ppa


@adamjonas should I start using `objective_c` for my code snippets from now on?


It's looking like I was really running afoul of a quotes string issue. Testing as we speak.

On July 15, 2015 5:47:59 PM EDT, Amiram Rahav <notifications@github.com> wrote:
>Lines 607:660 in server.rb should already be taking care of this.
>However, because of structural changes in  internal_wrangler things may
>have broken down slightly.
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/cloudamatic/mu/commit/efdfba5881a2ae78c42060bbdbdef78170752893#commitcomment-12190249

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


This is a stunning commit, if I do say so myself.


At this stage, I'm not sure this is necessary, I think reverse one2many from product_bundle_id is fine. Probably I've done a mistake from my original request!... I'll go deeper on this item.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/71155181


Please submit this as a pull request to the main Clamp.js repository. It is a good fix that fixes a bug I was seeing with the main repository version.


We do a bit more than that. Specifically, we use `isTypeIdenticalTo` to check for identical types. That function in turn may drill into the structure of the type which would be a problem for a type that recursively references itself.


seems like dah settle, haha, so kena masukkan id utk teacher mcm tu, no wonder lah, good share, aku baru tahu kena buat camtu.


No. we were testing a problem. Not positive it was this lab.


I am just on the phone atm,  what pr  is this part of?  :-)


With two functions you made the storage engine class very simple and, I think, that translates into very usable. I can see third parties easily being able to integrate with their storage system of choice.


Don't know if missing jquery is causing problems, but if so I'm pretty sure that this link syntax:
`<script src="//code.jquery.com/jquery-1.11.3.min.js"></script>` doesn't work on a project that is not on a server -- e.g. just viewing files locally.
It's necessary to use the whole url including http:
so `<script src="http://code.jquery.com/jquery-1.11.3.min.js"></script>` will work from your local file structure


This wasn't quite a good refactor. If `$this->parentSlug` is NULL (for example if line 121 sets it to NULL) this used to go to the else before this refactor, and now it doesn't. I propose changing this condition to:

    if (!empty($this->parentSlug)) {
       ...
    } ...


correct me if I'm wrong. but if we import this module from outside we are going to have a hard time using absolute imports.


If you don't plan to share the parser between the tests or beforeEach function, then moving this statement into the  tests will prevent accidental leakage.


Hey @atelierZuppinger -- thanks for bringing this up! @samccone can probably help with this. We just need to make sure the watichify process is ended after a compile, I think.


@jklontz @JordanCheney FYI. QPoints and QRects were being serialized to disk as null because string conversion fails (which is baffling). They were also being deserialized as QStrings.


said that, I agree with you. relative imports are hard to read


fast-forwarding master to auto = bc6c9052


Hi Ankur,

You are right. My points could be fixed relatively easily. I was,
however, bummed by the sluggishness of the configure script due to the
boost dependencies. I tried to hack up a simple cmake solution to see
how it is. So far I found it in terms of maintainability and speed more
convincing. How big of a fan are you of cmake?

Enjoy CNS! I will be there only for one day during the workshops.

Ciao,

Friedemann

On 07/14/2015 10:01 AM, Ankur Sinha wrote:
> Heya,
>
> The --enable-mpi part can be removed easily enough. That's just from
> configure.in. Initially I thought that auryn could also be run
> without mpi, which is why I'd added it.
>
> I'll have to look at the flags that autotools passes. They can be
> overridden by defining environment variables like CFLAGS and CXXFLAGS
> as far as I know, since this is how we do it in Fedora packages. (I'm
> quite sure options can be added to the configure script, if they
> aren't already provided by default)
>
> I can look into it sometime, but I'm flying to CNS this week and not
> carrying a laptop so it'll probably be when I get back. You could
> either merge the branch now and I could open a pull request for this
> later, or you could wait until I figure it out and merge it all at
> once?
>
> Cheers, Ankur
>
> --- Reply to this email directly or view it on GitHub:
> https://github.com/sanjayankur31/auryn/commit/8724ab739bf9e9ed41b2e14ed52b8e474907f47e#commitcomment-12160456
>


I agree, the only thing that is a bit "non-optimal" is that there is no "abort" support. That's why I'm waiting for the `_handleFile` to complete before calling `_removeFile`. I think having a third `_abortFile` for aborting mid-stream would be too complicated for storage engine implementors.

We could possibly add this as an optional feature later on...


separate this into interface and implementation


add javadoc and @since 3.8.0


no friend I'm not using psiphon, I'm using my ssh as endpoint, runs and connects normally when there is no restriction, but the problem is that the place where live need of toht proxy traffic to pass through an intermediary proxy squid but not is how to implement I hope understand me an example
DIRECT NOT PROXY SQUID

POS/list HTTP/1.1
Host: ipmyserver----------------------------------------------------------------> toht server response with 200 OK

*-----------------------------------------------------------------------------------
WITH PROXY SQUID

POST /listen HTTP/1.1
Host:ipserver --------------PROXYSQUID----------------------- servertoht response with 400 bad request or 501 not implemented

This is an example


Whut. That's exactly what I'm running too


hola


this class/interface (and it's impementation) should have package visibility


This commit deserve an update to language files, so that translators can work on changes.


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


Maybe you can add simply decompose.ssa.auto proxy method? Which would determine neig and branch to appropriate decompose? Do we have any other use of svd.method?


since this is now a factory, the attempt prefix on the method name seems weird. I would call it createImplicitConfigurationInstance and specify in the javadoc that it might return null or throw some kind of checked exception. I leave that choice up to you


I'd suggest using `prefix, localpart = ...` to be less confusing. The following line would then be:
`uri = element.nsmap[prefix]`


Same comment as in https://github.com/qterminal/qterminal/commit/49671234305f533b58494e44ad5e9da2fd111d5c


also note that the only public method in a factory class being void is suspicious


Fixed in my latest commit :+1:


Sometimes checking for null with two == makes sense, because that also matches `undefined`
Don't know if that's the case here, though


I just tested your code again and this is working lol, i will never understand how those things work.. i added an other test to be sure that everything will work in the future


Need comment explaining all of this. And are you sure that this actually solves the problem?


it doesn't seem right for the factory to also register the instance. I would just return the instance and let the invoker deal with that.


updated


The above five lines forbid namespace definition in an element, which I think is a bit restrictive.
If we know the prefix and the namespace's URI, I would create the namespace and add it to the bundle.
There is no harm in adding more namespace at this point (unless PROV-XML disallow it, I haven't checked!)


Much better :+1:


97648014-DocIssues4906 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1749 of 1749 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)


Looks like a pretty even mix, actually, but I'll change it to an underscore since that seems more common across the rest of Olympia.


Yeah, I just moved this from one of the old tests and didn't think to change it.


Oops :smile:, thanks @barisusakli


Thanks for adding this comment. Should we add a Preconditions.checkState to ensure that the volume is the same, so we catch any bugs wrt comparing across volumes?


Also assert volumes are the same?


EvansHVACObjects (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Improved here: https://github.com/meteor/react-packages/commit/ac69e973835b4e6ca43f78e1b8644af769a9f69d


bug spotted! my fault not yours.... this shouldn't be registered here... The ConfigurationObjectBuilder will register this automatically


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1176 of 1176 tests passed)

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


The above can simply be:
```python
rec_id = ns[localpart]
```
Creating a qualified name from the namespace allows it to cached the result QualifiedName instance.


Probably. I copied what I saw other tests that worked with file uploads doing, but creating them directly does seem like a better idea.


Maybe set default polling interval to 3 instead of 0 ?


> but if we import this module from outside we are going to have a hard time using absolute imports

how so?


because the comment on line 60 and 71, this is not needed anymore


**Why use struct here instead of union**
You can't use members with complex ctor in `union`.
Iterators in MSVC have complex constructor, that's why it fails to compile. But you may want to use here struct even for Linux in case you want to specialize `baisc_json` with your custom array and object types. You may face with the same issue, because your custom types may have complext ctor. So I believe it's better to use `struct` instead of `union` here.


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 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)


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This looks good.  I like the Math.Max(), reads allot clearer.


don't use the extensionManager directly. Receive a ConfigurationInstanceRegistrationCallback argument


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

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


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


ListVolumes --> ListBuckets


@drgroovestarr How did I treat you poorly?  I asked how this was associated with linaro.org.  You don't represent linaro right?  And neither do these build scripts.  Also Cl3kener doesn't represent linaro either.


EvansHVACObjects (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


@Foggalong I forgot to add you as a contributor, can you add your real name and your email too ?


```
In file included from opencl_lm.h:13:0,
                 from opencl_lm_b_plug.c:15:
opencl_lm_b_plug.c: In function 'find_smem_lws_limit':
opencl_lm_b_plug.c:189:3: error: 'CL_DEVICE_WAVEFRONT_WIDTH_AMD' undeclared (first use in this function)
   CL_DEVICE_WAVEFRONT_WIDTH_AMD,
   ^
common-opencl.h:225:59: note: in definition of macro 'HANDLE_CLERROR'
 #define HANDLE_CLERROR(cl_error, message) (handle_clerror(cl_error,message,__FILE__,__LINE__))
                                                           ^
opencl_lm_b_plug.c:189:3: note: each undeclared identifier is reported only once for each function it appears in
   CL_DEVICE_WAVEFRONT_WIDTH_AMD,
   ^
common-opencl.h:225:59: note: in definition of macro 'HANDLE_CLERROR'
 #define HANDLE_CLERROR(cl_error, message) (handle_clerror(cl_error,message,__FILE__,__LINE__))
                                                           ^
opencl_lm_b_plug.c: In function 'auto_tune_all':
opencl_lm_b_plug.c:350:5: error: 'CL_DEVICE_WAVEFRONT_WIDTH_AMD' undeclared (first use in this function)
     CL_DEVICE_WAVEFRONT_WIDTH_AMD,
     ^
common-opencl.h:225:59: note: in definition of macro 'HANDLE_CLERROR'
 #define HANDLE_CLERROR(cl_error, message) (handle_clerror(cl_error,message,__FILE__,__LINE__))
                                                           ^
opencl_lm_b_plug.c: At top level:
opencl_lm_b_plug.c:283:13: warning: 'auto_tune_all' defined but not used [-Wunused-function]
 static void auto_tune_all(unsigned int num_loaded_hashes, long double kernel_run_ms)
             ^
make[1]: *** [opencl_lm_b_plug.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [default] Error 2
```

To fix this, add definitions in common-opencl.h line 93 and on. We already do for some others.


Nitpick: type bucketNaem


`3.0.0`? That escalated quickly :grin:


final


I do not completely follow the code above, but it looks like it does not allow new namespace definitions at the `subel`. My same comment about unnecessary restriction above applies, I think.


зачем, если поставить nasm всё норм соберётся.


Looks good.  Allot of those measurement was looking at aspect s of our timing algorithm.  So they are a bit redundant with the boundary checks.


Proposed fix in #3022


If this a pure getter i.e. the caller is not expected to modify the list of buckets then we should probably wrap {{buckets}} in Collections.unmodifiableList.


When we see an array literal like the one in your example above, who knows what it really means? There is simply not enough data to say for sure. We currently remove all subtypes, but as we've discovered on many occasions, that's too aggressive. The new algorithm makes the very conservative assumption that if two types are identical except one has null or undefined in places where the other one doesn't, we can safely discard the less specific type. Beyond that, it's not clear there's anything meaningful we can do that doesn't loose some information.

Now, it is true that once we widen a type we end up with `any` in places where the user wrote `null` or `undefined`, but this only happens when the containing type has some other unmatched information. I think that's quite defensible and very much in line with what we do for just plain object types with `null` or `undefined` properties.

Returning to your example above, let's say neither `Foo` nor `Bar` was a subtype of the other. At that point we get the full type even today. I think it is really hard to explain why that is correct, but not when they happen to be subtypes. After all, the user might not even think of them as subtypes.


#EXTM3U
2
3
4 #EXTINF:0, RedTV HD
5 http://movilstream.digitalproserver.com:443/redtv/redtvvivo/livestream1/playlist.m3u8

8 #EXTINF:0, UCV HD
9 http://movilstream.digitalproserver.com/ucvtv/ucvtv.smil/chunklist-b1248000.m3u8

12 #EXTINF:0, TVN HD
13 http://wow3.tvn.cl:1935/liveedge/tvnelemental.smil/chunklist-b1431000.m3u8

24 #EXTINF:0, Mega HD
25 http://edge-12-cl.edge.mdstrm.com/live/smil:db6a38a0a2bffd96d01965c4d975b9fb/playlist_b1500.m3u8

28 #EXTINF:0, CHV HD
29 http://mi.chv.cl/playlist/stream.m3u8?s=hd

34 #EXTINF:0, C13 HD
35 http://13.live.cdn.cl/13hddesktop/13hd-1250.m3u8

40 #EXTINF:0, C13 Cable
41 http://cluster.13cmovil.ztreaming.com/13cm/grpz13c/playlist.m3u8
42 #EXTINF:0, RecTV
43 http://13rec.movil.ztreaming.com/13rec/13rec/playlist.m3u8
44 #EXTINF:0, ETC TV
45 http://movilstream.digitalproserver.com/etctv/etctvvivo/livestream1/playlist.m3u8

48 #EXTINF:-1, CDF Premium
49 http://65.202.39.110:5000/live/cdf_chile_deporte/playlist.m3u8
50 #EXTINF:-1, CDO Premium
51 http://wow2.cl.digitalproserver.com/cdo/cdovivo/cdo.smil/chunklist_w913906597_b1064000.m3u8

149 #EXTINF:0, C5N
150 http://c5n.stweb.tv/c5n/live/playlist.m3u8

189 #EXTINF:0, Antena 3 HD
190 http://a3live-lh.akamaihd.net/i/antena3_1@35248/index_4_av-b.m3u8

197 #EXTINF:0, La Sexta HD
198 http://a3live-lh.akamaihd.net/i/lasexta_1@35272/index_4_av-b.m3u8

216 #EXTINF:0, Telesistema
217 http://mek4.mekstream.com/telesistema/smil:telesistema.smil/chunklist_w1839003835_b764000.m3u8

289 #EXTINF:0, NASATV HD
290 http://nasatv-lh.akamaihd.net/i/NASA_101@319270/index_1000_av-p.m3u8

345 #EXTINF:0,--Musica--
346 http://separador.m3u
347 #EXTINF:0, CapitalTV
348 http://ooyalahd2-f.akamaihd.net/i/globalradio01_delivery@156521/index_656_av-p.m3u8
349 #EXTINF:0, First Music HD 1080p
350 http://109.239.142.62:1935/live/livestream1/playlist.m3u8
351 #EXTINF:0, First Music HD 720p
352 http://109.239.142.62:1935/live/hlsstream/playlist2.m3u8
353 #EXTINF:0, First Music HD 480p
354 http://109.239.142.62:1935/live/livestream3/playlist.m3u8
355 #EXTINF:0, VEVO 1
356 http://vevoplaylist-live.hls.adaptive.level3.net/vevo/ch1/06/prog_index.m3u8
357 #EXTINF:0, VEVO 2
358 http://vevoplaylist-live.hls.adaptive.level3.net/vevo/ch2/06/prog_index.m3u8
359 #EXTINF:0, VEVO 3
360 http://vevoplaylist-live.hls.adaptive.level3.net/vevo/ch3/06/prog_index.m3u8
361 #EXTINF:0,--Radios--
362 http://separador.m3u
363 #EXTINF:-1,Duna
364 http://wow1.dallas.digitalproserver.com/dunatv/dunatvvivo/dunatv.smil/playlist.m3u8
365 #EXTINF:-1,Tiempo
366 http://movilstream.digitalproserver.com/tiempotv/tiempotvvivo/tiempotv.smil/playlist.m3u8
367 #EXTINF:-1,Zero
368 http://movilstream.digitalproserver.com/zerotv/zerotvvivo/zerotv.smil/playlist.m3u8
369 #EXTINF:-1,Radio la clave
370 http://movilstream.digitalproserver.com/laclavetv/laclavetvvivo/laclavetv.smil/playlist.m3u8
371 #EXTINF:0,--Religion--
372 http://separador.m3u
373 #EXTINF:0, BYU Latino
374 http://byuhd2-lh.akamaihd.net/i/byutvintspanhd2_live@103133/index_1240_av-b.m3u8
375 #EXTINF:0, BYU Latino-2
376 http://byuhd2-lh.akamaihd.net/i/byutvintspanhd2_live@103133/index_840_av-b.m3u8
377 #EXTINF:0, BYU USA HD
378 http://byuhd2-lh.akamaihd.net/i/byutvglobalhd2_live@103134/index_2540_av-b.m3u8
379 #EXTINF:0, BYU USA
380 http://byuhd2-lh.akamaihd.net/i/byutvglobalhd2_live@103134/index_840_av-b.m3u8
381 #EXTINF:0, TBN Enlace
382 http://acaooyalahd2-lh.akamaihd.net/i/TBN07_delivery@186245/index_1728_av-b.m3u8
383 #EXTINF:0, TBN Enlace-2
384 http://acaooyalahd2-lh.akamaihd.net/i/TBN07_delivery@186245/index_1164_av-b.m3u8
385 #EXTINF:0, TBN Enlace-3
386 http://livestream-f.akamaihd.net/2675843_1839332_3798b35c_1_678@77359
387 #EXTINF:0, Enlace Juvenil
388 http://acaooyalahd2-lh.akamaihd.net/i/TBN08_delivery@186246/index_1728_av-b.m3u8
389 #EXTINF:0, Enlace Juvenil-2
390 http://acaooyalahd2-lh.akamaihd.net/i/TBN08_delivery@186246/index_1164_av-b.m3u8
391 #EXTINF:0, TBN USA HD
392 http://acaooyalahd2-lh.akamaihd.net/i/TBN01_delivery@186239/index_2528_av-b.m3u8
393 #EXTINF:0, TBN USA
394 http://acaooyalahd2-lh.akamaihd.net/i/TBN01_delivery@186239/index_1164_av-b.m3u8
395 #EXTINF:0,--Adultos--
396 http://separador.m3u
397 #EXTINF:0, Playboy TV
398 http://46.61.226.21/hls/CH_PLAYBOY/variant.m3u8?version=2
399 #EXTINF:0, Private
400 http://178.33.126.213:1935/leved/privateCom/.m3u8
401 #EXTINF:0, Oh La La
402 http://hlsstr04.svc.iptv.rt.ru/hls/CH_OLALA/variant.m3u8?version=2
403 #EXTINF:0, Blue Hustler
404 http://hlsstr04.svc.iptv.rt.ru/hls/CH_BLUEHUSTLER/variant.m3u8?version=2


I can just put .json at the end of any file I want. It doesn't mean it's JSON. You should check if you can read it as JSON.


:flushed:


Looks like you may have left some debug code in your commit. Please remove.


They were from a few OSX CI tests, perhaps they can be kept in a dummy `osx` node that a script uses to rewrite and force push to the `osx-ci` branch


Please remove any commented out code.


Will add the daemon BSD license


ok


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1240 of 1240 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)


These are very general. You should be testing that your data was parsed correctly. Check that the values you have in your test json file are actually parsed-- at least check that the size of the object is correct, doesn't have to be too crazy.


I get an error on this line, saying "Illegal invocation"


Everything other than MSVC12 fails. Appveyor only runs one at a time though, running the full matrix for one commit takes ~40 minutes


```diff
diff --git a/src/common-opencl.h b/src/common-opencl.h
index 27620b4..28e01dc 100644
--- a/src/common-opencl.h
+++ b/src/common-opencl.h
@@ -91,6 +91,10 @@ typedef union {
 #define CL_DEVICE_BOARD_NAME_AMD                    0x4038
 #endif

+#ifndef CL_DEVICE_WAVEFRONT_WIDTH_AMD
+#define CL_DEVICE_WAVEFRONT_WIDTH_AMD               0x4043
+#endif
+
 #ifdef DEBUG_CL_ALLOC
 static inline cl_mem
 john_clCreateBuffer(int l, char *f, cl_context context, cl_mem_flags flags,
```
```
$ make -sj8
opencl_lm_b_plug.c:283:13: warning: 'auto_tune_all' defined but not used [-Wunused-function]
 static void auto_tune_all(unsigned int num_loaded_hashes, long double kernel_run_ms)
             ^
ar: creating archive aes.a

Make process completed.
```


Yup, after testing, it turns out that the first time you press 'g', nothing happens.
We should either initialize loopStart and loopEnd to null, or revert the checking.


The toolbar is being used by extending 'page'


philosophical question: this behavior is already tested on the DefaultExtensionManagerTestCase and should continue to be so. However, shouldn't this class have tests of its own? how would they be different from what's tested in DEfaultExtensionManagerTestCase?

One idea is to add a package level setter for this factory in DefaultExtensionManager and change that test case to only verify that a mock is used. Then you move the real test logic here.

This is just a thought, I'm open to a different view on this


This disables m_slipEnabled on any slotControlSlipCancel update. Is it desired?


Yes!


Good night, Rowan Miller.
How could trade names (example: AspNetUsers) of EF7 tables. I'm not finding a solution to add an attribute (eg description) AspNetRoleClaims the table. I can add attributes the AspNetUsers and AspNetRoles tables.
-------------------------------------------------- --------------
  public class ApplicationUser: {IdentityUser
         public string Name {get; set; }
  }
-------------------------------------------------- -------------
public class ApplicationRole: IdentityRole
     {
         public int AppId {I get; set; }
         public string Description {get; set; }
         App App virtual public {get; set; }
     }
-------------------------------------------------- -----------
public class ApplicationDbContext: IdentityDbContext <ApplicationUser, ApplicationRole, string>
     ............... {}


Code Climate: :four:.:zero:


I see, good point though this view is needed too because it will be used for 404 errors.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yep!


Initial readme file.


nitpick: instead of "will determine" use more active language. Your code, including tests, should be concise.

it "calculates sample size"
it "counts members of group"
it "counts conversions"


97647908-DocIssues4615 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1749 of 1749 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)


That typo O.o


Same as previous commit:
`Fix merge conflict` e3c4822105e045a4b5c5818df0c38c3db83258a3


JCenter is a superset of maven central, adding both simply slows down resolving.


I'm generally against negative variable names, mainly because they always lead to double negatives which are confusing. E.g. if noaria is false, it's "no noaria"...


You are right - I should've done a separate PR; will remember that next time.

project.files() is a FileCollection.  'matching' is a method on FileTrees only.  So the old code fails with a missing property exception.


oops :(


On 7/15/15 2:58 PM, Ola Bini wrote:
> Well, I disagree. Returning an error instance might be right in some
> cases, but causing a panic is /never/ the right choice for a library to
> do, just as calling System.exit() is always the wrong thing to do in a
> Java library.

Not sure if the comparison is fair: does Java provide the same defer()
and recover() in the System.exit() context?

> I'm not saying the client has to know the details, but you will notice
> that a typical TCP will will give you levers to tweak congestion
> parameters, and incidentally linux 2.4 support explicit congestion
> notifications as well.
>
> Fundamentally this is part of feature parity with libotr that does offer
> error handling customization.

Fair enough, I'll postpone every error handling until we have something
implemented for this.


Ahh, ok

On Wed, Jul 15, 2015, 3:20 PM Advay Mengle <notifications@github.com> wrote:

> You are right - I should've done a separate PR; will remember that next
> time.
>
> project.files() is a FileCollection. 'matching' is a method on FileTrees
> only. So the old code fails with a missing property exception.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/j2objc-contrib/j2objc-gradle/commit/2f2e46fbeb293e18a187e79eac7151020f128d80#commitcomment-12190771>
> .
>


Was it really necessary to merge this into mainline?


this commit message can be shortened by removing "provide a means"

I'm going to nitpick on this because >35 character commit message subjects get cut off.

Here's more information: http://chris.beams.io/posts/git-commit/


Good reason to add a complex multi-project test case. You build that, then
I'll help you hooknup the tests.

On Wed, Jul 15, 2015, 3:21 PM Bruno <brunob@gmail.com> wrote:

> Ahh, ok
>
> On Wed, Jul 15, 2015, 3:20 PM Advay Mengle <notifications@github.com>
> wrote:
>
>> You are right - I should've done a separate PR; will remember that next
>> time.
>>
>> project.files() is a FileCollection. 'matching' is a method on FileTrees
>> only. So the old code fails with a missing property exception.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/j2objc-contrib/j2objc-gradle/commit/2f2e46fbeb293e18a187e79eac7151020f128d80#commitcomment-12190771>
>> .
>>
>


Commented code has been removed.


Debug code has been removed.


The `lhs` arg? No. Its a value type, and should always be available to the operator method. Otherwise, you'd probably want the compiler to complain.


The issue is that when `web` is imported, `web` itself tries to import `math`. When the program is run, the `nupic` directory is automatically added to the PythonPath (and it becomes the first path in the list), therefore the only way for web (or anything else for that matter) to use the real `math`, we need to either remove this file's directory from the pythonPath, or possibly move the path to the builtin `math` in front of the local directory. I chose the simpler of the two options and I it's worked in my tests.


You are right, it does not make sense to do so. So this use case can probably be disregarded.


Lazy enumerators are performant, especially with map. 100% fine to use `map` here, but check out `lazy.map` for your production quality code.


This cond should be an if.


Nice commit.


These parameters:

RealisticOrdinance = true,#added
        UseGravity = true,#added


This can be (map first (:dependencies sub-module))


From hitmail.IT to hotmail.COM? whooo bad idea! :·P


duplicate


asdasdasd


These messages have not been seen in prod for over a week.

I dont know if they have been in prod before; i cant remember seeing them.

That being said; might as well scan the heap every 10 seconds and do a sanity check. Especially since we have a new heap and its useful for rolling out in prod for extra verification.


EvansHVACObjects (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


WFM. @seanturner ?


+1 for breaking my app when I installed on a new server with a fresh go get and +1 for making if obvious when an update breaks things


Do you mean canola oil


You should correct the code style around your commit - this should be `if not f then`


I only changed the turn rate (360) instead of 240. The rest is GPG work.


What's this magic number? ^^


Yes, cuda programes can be run from linux or window. Only thing to Woerth
about is your graphic card. It should be from Nvidia.

To overcome This decrepancy of graphic card dependance, you can look at
OpenACC derivatives.
On Jul 15, 2015 12:07 PM, "JackeryShh" <notifications@github.com> wrote:

> thanks,recently i just learn the cuda,it's very useful for me. i have a
> question to answer that's it can run on both Windows and Linux,isn't it ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/3cHeLoN/cupoisson/commit/78e36e8b49df0ad0222bfe584767ea4517fc8f03#commitcomment-12175553>
> .
>


Extract regexp match to some func


Смотри как фильтр ещё умеет :)

https://github.com/hexlet-exercises/words-count/commit/002c55b2005f126b36fa0171d6d0aa467079b5bb#diff-8997713ffb9d87b4eb836b3ea076875fR18


if (m && m.group() in folders.keys())


if(year && year.group() > "2013")


if not dwnld_exist


TODO: if modelSize became less then selectedIndex selectedIndex must become modelSize - 1


A princípio, esta função é chamada dentro da addInvoicePayment, porém por algum motivo desconhecido, em alguns casos isto não acontecia.


does it werk?


m = getMatcher(op[0])

def getMatcher(str):
    return re.search("(.*?(?=\s\d{2,3}))|(.*?(?=,\s20\d{2}-\d{2}-\d{2})|(.*?(?=\s#\d\d?\d?)))", str)


TODO: migrate to set of checked index


The icon isn't displayed when the checkbox is checked. The icon background does get the active color though. I am not sure if this is a bug in the widget or in the markup.


could someone please regenerate the docs?
in particular, http://doc.rust-lang.org/book/the-stack-and-the-heap.html ?

the online documentation still has this typo and it can be quite confusing and make you waste some time back and forth to be sure you read the page correctly and verify you are understanding the example.
fortunately I thought of trying to make a PR to fix what I thought could be a typo (and be corrected in case it was a mistake) and found this commit.

the advantage of getting stuck is that it means you are actually grasping the concept. :)


@msalvadores Could you please review this change to instance.rb?  The unit tests I wrote in the ontologies_api project uncovered an issue with page counts always being set to 0 in response objects, even when there are multiple pages of data.  I'm wondering if there are performance implications for running the SPARQL query to get all instances without a limit?


TODO: use a `Pair<String, Runnable>` instead?


@pbeeler Thank you for being constructive.  I did not ask that you leave Github,  I requested that you go back to Gitlab,  as in get back to your own projects.  You stated off guns blazing, using profanity. If you have a problem with the way that I have worded things,  I can see where you are coming from.  I have worded the readme in an unclear fashion,  in a way which is nonspecific and could be misinterpreted.  As you can see,  I lack your experience - - I was trying to give proper credit,  and could have done a better,  more concise job. I don't agree that Chet used your scripts but I have updated the readme based on your recommendation.


The `undoChanges()` routine can be dropped as well.


This OpenDMARC integration requires MySQL -- why not use Postgres?


So, I have offloaded the checking of two types of weak passwords( all-numeric like 111111, and all-alphabetic like asadgsdfg) from the bloom filter to the client side. I also removed frequent passwords in the bloom filter which were longer than 12 characters(these are not outright weak, and *may* actually be strong) and this helped me eliminate almost 99% of the passwords stored in the bloom filter(from 64+million to just over 545K). So now we can reject passwords that are outright weak, and only check with the bloom filter when we think that a password has a chance of being strong. If the bloom filter finds that the password isnt in it, we *know* that it is a very good password. If it finds it, then it might or might not be a weak password. We should reject these, just to be safe.
The file size is worrisome, weighing in at a little over 600k, but gzipped it is only 120+k, and can be loaded in async.
@shane-tomlinson @rfk @ckarlof


Most of the numbers you'll be dealing with are money. Never cast those values as floats. Always use `BigDecimal`. `Float` can give rounding errors. Just get in the habit of never using floats. You'll always want to specify the precision and scale, anyway.


better use some temporary variable for accessing op[0], because it's difficult to understand what does op[0] means


@jvendetti It is probably fine because normally there are not that many instances in ontology for this to be an issue. One way to be safe is to do first a count and then a query but that is probably an overkill here.

From what I can see is good to go.


After offline discussion, I see that common_packages is the "less-specific" version of packages, and the sum is used for the total list of packages to install.  I guess it'd be nice if we could augment 'packages' at each appropriate level, and maybe there's a way to do that with jinja2 templates (something like packages: {{packages|list}} + {{local-package-additions|list}} might work; needs testing).  But until we get a chance to revamp, probably just adding something like "see common_packages" above in the packages[] description, and "the set of packages to install is packages + common_packages here" will clear up the confusion well enough.


Radical commit.  Really game changing.  I'll be watching the project closely.  Solid.


It looks like there's a merge issue here -- I moved the initial `threads_waiting_on_gil` check to threading.h, and the check that happens later in this function is the second check (despite what the comment now says).  Though, I would have thought that that would mean we wouldn't call any destructors so I'm not sure how to explain the passing tests...

One optimization idea is to combine some of the checks into a single indicator variable.  For example, rather than putting this logic in front of the gil_check_count check (which would mean checking three counters every time), the destructor logic could set gil_check_count=GIL_CHECK_INTERVAL and then we put this code behind the gil_check_count check.


Thanks @keithbostic lgtm


This is great! Thank you for putting in the changes we requested so quickly. We should probably add a couple javascript integration tests for this feature as well, but I'll take care of that on our end, since it's pretty unintuitive. Will test and merge this in soon.

Kyle


just for the record (these look unused), but I am -1 on adding time-based timeouts to these processes for now.  let's just keep it simple and deterministic :)


Added mpv "issue" <https://github.com/mpv-player/mpv/issues/2142>.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (723 of 723 tests passed)

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


Thanks, I will try it.


Missing `#include <stdint.h>`


I wish I could like comments on here lolol


Excellent points. That makes sense. Reading up on BigDecimal as we speak ...


AFAICT this should be:
pkg_name = splitFilename(spec)[1]


And accusing jenkins ... *facepalm*


Excellent point and agreed. One of the RSpec tutorials I looked at was adamant about using "will" as an active statement of what *will* happen. Not too long after reading this, I found another tutorial that said this was wrong and was a reaction to the previous *it 'should...'* construct that had fallen out of favor.


Aqui hay que mejorar el texto
![cvmlnp1i](https://cloud.githubusercontent.com/assets/5691112/8711681/cf1ff734-2b19-11e5-90b6-1f0e8a7fddc5.jpg)
@artownz


this is a case in which a comment is helpful, especially given that most engineers aren't too familiar with stats. go you!


This can be `if [[ $REALM_SWIFT_VERSION ]]`.


Great tip! Thanks. I will check into that.


@icalderond no mamaaaaar


quotes


In order to get rid of deep "if else" code enclosing, if you have no else block ever, and in case if this statement will return false than your code will exit, you can make the following:
```python
if not (m && m.group() in folders.keys()):
      return

year = re.search("\d{4}", op[0])

if not (year && year.group() > "2013"):
      return

dst_exists = os.path......

if not dwnld_exist:
      return
```


I don't think we call this function for Pyston builtin types; I don't think we have any with ordered finalizers though.

I think the only place we create Python objects that don't go through PyObject_Init is through DEFAULT_CLASS_SIMPLE (and the VAR variant); can we add an assert there that there aren't any ordered finalizers?  (and I suppose that it's not a class object, for good measure)


Round this number. 6 decimal precision should be more than enough.


I'm not removing any type information, I'm adding another type option for the parameter, which is perfectly supported in the 3rd party library that we are providing a *definition* for, and also by Typescript itself, using the above notation.

I'm not using Object[] because I want to be able to insert anything, its a fair use case; I receive a JSON array from the server, and want to create a Backbone.Collection from it without an additional parse step, and again, it's a part of the definition of Backbone.

In the end, I'm still able provide a generic parameter for my collection:

    var collection: Backbone.Collection<MyModel> = new Backbone.Collection(array);

As I see it, I'm not losing any type information along the way.


@msalvadores OK, thanks.  I'm only aware of two ontologies in BioPortal with a large number of instances, which are RADLEX at 46K and NCIT at 42K.  Do these numbers seem OK to you in regards to the query?  If not, maybe we could talk about it tomorrow in person.


Did you check in valgrind if this fixes the yaml leaks?


Got ya
On Jul 15, 2015 2:44 PM, "Danijel Koržinek" <notifications@github.com>
wrote:

> It wasn't possible to compile it and make a working build for all the
> platforms. Again, this can be checked during runtime and switched
> accordingly, but it should be tested first.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jmc2obj/j-mc-2-obj/commit/d0d10929afabd3426d330003b12e25233ec1e1dc#commitcomment-12190192>
> .
>


Oh yeah, you're right.


I'm not overly pleased with `static_cast` to a completely different class...
Would something like this work?
```cc
class CMessageProxy {
public:
  CMessageProxy(CMessage& msg) : m_msg(msg) {}

  CString GetTag(const CString& sKey) const { return m_msg.GetTag(sKey); }
  void SetChan(CChan* pChan) { m_msg.SetChan(pChan); }
  ...
private:
  CMessage& m_msg;
};

class CChanCTCP : public CMessageProxy {
public:
  using CMessageProxy::CMessageProxy;
  CString GetText() const { return GetParam(1).TrimLeft_n("\001").TrimRight_n("\001"); }
  void SetText(const CString& sText) { SetParam(1, "\001" + sText + "\001"); }
};
```

I can't say I like this very much either though... But it becames possible to add new fields to messages if needed.


Oh I didn't notice this, will change it to protocolVersion


Input time offset. I'm guessing it is used to nudge audio and video into sync. The number was chosen for heatseeker.


This should read
(Your Intermediate certificate: <YOUR-VENDOR/CA>.crt)


Standard error should be a `BigDecimal` since it's a square root.

Be careful about your types. Ruby is dynamic, but you should still think about your data first.


@jvendetti It might be an issue with those number of instances. We would have to test it.


merging the two


I'm not sure that `trial_size` is preferable considering you use `group` elsewhere. Was there a reason you didn't use `group_size`?


`using System.Collections.Generic` should be in alphabetical ordering w.r.t. the rest of the `using` statements.  So, add it after `using System.Collections;`


Avoid hardcoding values for a variable input like confidence interval. What if I want to be more or less confident in my analysis? I should be able to pass in a percentage confidence and you figure out the multiplier for standard error.


[![Mei Ping Vase with Cover](http://api.artsmia.org/images/3711/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3711)


[![Contents Sheet for Seven Day Diary (Not Knowing)](http://api.artsmia.org/images/56988/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=56988)


[![Day I](http://api.artsmia.org/images/56990/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=56990)


[![Day II](http://api.artsmia.org/images/56993/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=56993)


[![Day III](http://api.artsmia.org/images/56994/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=56994)


[![Day IV](http://api.artsmia.org/images/56995/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=56995)


[![Day V](http://api.artsmia.org/images/56997/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=56997)


[![Day VI](http://api.artsmia.org/images/56998/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=56998)


[![Day VII](http://api.artsmia.org/images/56999/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=56999)


another value to round


The documentation also follows the train model. The change is visible in the nightly docs: https://doc.rust-lang.org/nightly/book/the-stack-and-the-heap.html

But will not show up in the stable docs until the version this change is associated with is officially released.


Oh you're right. It's for the case where you only have null or undefined in your array, like `[]` or `[null]`, but we are trying to eliminate it at the next level up.

I think it is necessary for tuple types too.


Should I do PRs against this repo or Pauls? I'm also wanting to open an issue discussing modularizing es6-shim but noticed issues disabled here.


this is inconsistent with the `123` from another test. Types!


Good question. I guess the main reason was that in reading up on the domain terminology, I saw that term used quite a bit. "Trial size" that is. A group had a number of trials. The size of the group was, in this case, how many members were part of the group (A, B). Not saying I'm right on that, mind you. I'm more just saying that I went with what I thought seemed to be how the business domain was used in some sources.


Arquivo Publicado por Welton Thiago


ls-l.c


This function looks like it sends the message as is, without stripping all tags except time.
And I think, there should be such method.
Maybe rename this one to PutClientFilterTags or something?


Add braces around the `break;` statement.


yay! but it would still be cool if I could pass it in.


Thanks. I have applied that fix.


not sure about `variant` either.

naming is hard. you'll get better with time, but even then, you can always debate about it.


In order to decrease complexity and increase readability, you can make the following:

```python
if not (m && m.group() in folders.keys()):
      return

year = re.search("\d{4}", op[0])

if not (year && year.group() > "2013"):
      return

dst_exists = os.path......

if dst_exists:
      return

dwnld_exist = os.path.isfile(os.path.join("D:\\new week", op[0]))

if dwnld_exist:
      return
```

this will help you to get rid of deep `if else` code enclosing


Why the separate commit? Why not name it `fixup! fb: Change operations for get/set var info` after all?


There are dozens of section types. The ones I have mentioned here, happen to be very important ones that are used in many different places. The same does not apply to `stash`. So this is no improvement what-so-ever.


NP


probably doesn't matter, but `++iter` is good practice.


Yeah, in a later commit I did fix that. At least in terms of making it a constant. Is that what you mean?


This actually breaks support for dialogs and forms. {ENTER} will always submit the form and {ESCAPE} will close the dialog boxes. I worked around it by adding back the event.preventDefault(), but still keeping the event.stopPropagation().


Perhaps, but if it's so important to get rid of null and undefined in the case where there is no unmatched information, why does unmatched information suddenly make it less important to clean them out?


Okay, gotcha. So make that that an attribute that could be passed into an instance of TestData. Makes sense.


I like how you named this. Very clear.


This revision is clean.


yay https://github.com/twizmwazin/CardinalPGM/commit/65f1922eef3908d4ac3eb4076518db7a5653e0d6


But regardless of what modules did with line, the old original line is shown anyway.


c


I hear you. That particular term came from reading some papers on the statistics being used here. That was the term used in some contexts. I probably also conflated a bit, though. I worked in clinical trials before where we use similar statistics and trial sizes were broken up by particular variants (medication 1 vs medication 2).


I think it might be slightly more optimal to create an array of initial write pointers outside of the outer for loop because it's more optimal to increment through contiguous memory (rather than getting the write pointer for ever sample in every channel). I'll do when I get the chance before next time! Same with the other sine processor.


nooooooooooooooooo

when "no" suffices

I made a joke about decimals.

Nerd.


Please pardon my question if it appears too intrusive.

What would you think is the most appropriate way to call the projection matrix in [kinect2_bridge](https://github.com/code-iai/iai_kinect2/blob/d829df5cdbf7a3ed6d198089486dbcfe7c786515/kinect2_bridge/src/kinect2_bridge.cpp#L684-688) with this call back function?


product name will be part of the filename used to save files. specification filenames not yet drafted.


That wasn't bad. :) I'm totally stealing that.


Nice catch.


Why return? This doesn't apply to async lambdas?


will it be first every time? what if you add to the code such that it changes the ordering? should reference a key value pair, not just a position in an array if you want specific data.

I ran into this with address formatting on Portfolio. I was calling `#attributes` and assuming they would be ordered, but they were ordered by the value that was last updated. Stuff like that happens all the time, and your tests shouldn't be vulnerable.


@paul1956 This won't work with the latest sources. The APIs changed a lot since I wrote this extension.  I need to update it to work with the latest bits.


Legacy reasons. The buffer was originally bigger and I had to check for out of memory. Even so, starting from a certain size you should prefer allocating and actually checking if everything went okay over the simple way of unsigned char buffer[BUFFER_SIZE];


We have only one using statement: `using System.Collections.Generic`. There is no need in `using System.Collections;`


This condition is odd. Should be ```(oldAsyncIndex >= 0 && newAsyncIndex < 0)```


So correct me if I'm wrong, but this is the big change here with `MailWorker.perform_async` as an alternative to prevent the duplicates?


Just didn't realize it was there, oops


To use, swap in the application key here, and other credentials in tables_api.rb.


So the reason these changes are needed, is that there is logic in our CSSBuilder class (https://github.com/liferay/liferay-portal/blob/master/modules/util/css-builder/src/com/liferay/css/builder/CSSBuilder.java#L310) that relies on css being in a folder called css.

I worked around it by copying the build-css macrodef from build-common.xml, and adding the src/META-INF/resources directory to the sass.portal.common.dir property for when css in this module is built.  Without doing this part, the sass compilation will fail.

The other option would be to change the logic in CSSBuilder.


To use, swap in the full credentials here and the application key in NetSimShard.rb.


Fixed


97647908-DocIssues4615 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1749 of 1749 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)


minor, but these should probably be std::deque's.


Why is this only in C#? Doesn't VB have the same problem? I think this can be implemented in the common analyzer.


I'm not sure about this. Mounting and unmounting seemed to work fine right now. We can just omit this for now.


err sorry not this one (need to remove from it), but the two above it which I think are just used as stacks.


I like where you moved the constants in the `PhishingFrenzyMailer` thats nifty


changes in dojo_ruless.md by AA


>mr rabbit


This is awesome, btw, the interactive preview.  But, it seems like it only updates the camera's position.  If I move an object in the scene, nothing seems to change until I switch out of interactive rendering mode, and then back into it.  I'm thinking the interactive export isn't updating objects' transformation matrix, but that's just a guess.  Anyway, this is a very cool feature.  THx!


Oh, updating the objects' shaders does seem to work.


Yes, we had a MailWorker defined already, but I dont think it was ever called.

MailWorker is literally just a Sidekiq Worker which calls `PhishingFrenzyMailer.phish` so the logic all stays in one place. I tried extending PhishingFrenzyMailer similar to how Sidekiq extends ActionMailer to be able to call `PhishingFrenzyMailer.delay.phish()` but it was too much hassle in the end for my poor mind :) It is pretty much just show anyway.


would instance_cls return true for the last case (has a custom, non-safe tp_dealloc)?


Yep.  That's expected.  See the changelog.
- Only supports material edits, light moves/edits and camera moves for now.


BTW that's a PRMan limitation...


could we change this API now? wouldn't it make more sense to call `SearchActions.getQuery()` or something


Having mismatched constants defined in two places is one of the worst debugging headaches ever so best to avoid it :)


Won't this fail on *nix if the directory already exists?


The cljfmt test forced me to do that. I filed a bug report and they have fixed it but I don't know if travis knows yet.


Should revert this!


Why, before there been normal code...


I am having the opposite problem I am using Visual Studio 2015 RC (older bits). I don't know how to get the latest bits even if you fixed it.

Sent from my iPhone
I apologize for any typos Siri might have made.
(503) 803-6077

> On Jul 15, 2015, at 4:11 PM, Jared Parsons <notifications@github.com> wrote:
>
> @paul1956 This won't work with the latest sources. The APIs changed a lot since I wrote this extension. I need to update it to work with the latest bits.
>
> —
> Reply to this email directly or view it on GitHub.
>


Ah yes. Sometimes I lose track of which commit message is going where. :p


Indeed a good video. Thanks! It seems like I managed to implement the worst kind of shuffles σ_σ The last maze algorithm was really fascinating. I've been wanting to dive into visualization for some time, maybe this will give me the last push that I need :) Tack bror.


I was actually going to ask about this.  I don't think the user will want to see the details of `e.message`


Pagina HTML Wesk Informática "Teste"


:+1:  it's good to test specific errors.


Don't think that this parameter values are needed somewhere else than in init.


Oh crap, you’re absolutely right, that was the reason why I’ve used `-p` here.


97648014-DocIssues4906 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1755 of 1755 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)


Glömmer bort att man måste reply:a för att du ska få en notice. Skrev ett svar på committen nedan


I just gave up again trying to set up Github..   In local testing, I find S(0, 2) works well for both sides using 2014 Stockfish code and old BYO software.  We want to give the rooks a nudge and no more.


Should be fixed now in latest commit. It maybe useful to start putting in logger.error e.message for these cases to help more with debugging and troubleshooting.


Update to the new Akamai endpoints


:tada:


Not just making it a constant, but allowing me to pass in a value for the confidence.

It's a nice feature to add, but not explicitly required.


Look at `angles` package. I used it in gps (global position).
`angles::radians(angle)` are little readable.


Yeah, lots of options on improving the error handling here... damn, reading this old code is such a blast from the past...


Perhaps you can do this in build:static. I think browserify can create target directly but I'm not sure.


I don’t know how to solve it… I really don’t want to replace few damn simple commands with some complex build system (Gulp, Grunt etc.) just because some developers uses f*cked OS. :crying_cat_face:


Got it. What you say definitely makes sense.


Oops.  Sorry I didn't notice the changelog.  I did get a blender crash when I switched back into interactive rendering mode.  Not sure why.  Anyway, this is a great feature for lighting tweaks.  Well done!


I think this may be causing this issue https://github.com/azure-storage/xplat/issues/219


Note: Commit message has wrong status code, but the actual change is correct.


Nope, browserify doesn’t create target directory. :(


Re-added preventDefault in master

On Wed, Jul 15, 2015 at 4:05 PM, clshortfuse <notifications@github.com>
wrote:

> This actually breaks support for dialogs and forms. {ENTER} will always
> submit the form and {ESCAPE} will close the dialog boxes. I worked around
> it by adding back the event.preventDefault(), but still keeping the
> event.stopPropagation().
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/2781eac52c804655175f55e1a1e3b643ae3c3208#commitcomment-12191403>
> .
>


Researcher and admin APIs aren't currently in use outside of us. (And possibly the pitt-teambti guy, but he's still just testing stuff out.) We could presumably just delete the V1 versions of researcher and admin APIs and not have to do this massive refactoring.

Unless you really really really like having the consistency of having V1 versions of all our APIs.


What is this default value? It's not an API endpoint :/


Perhaps the IRCv3 capability specs could state something like that "CAP LS 30X" implies that the client is capable of receiving arbitrary sets of message tags.


Just remove mkdir here and do it in the script. Try if browserify creates dist dir for you.  In the worst case use [mkdirp] (https://www.npmjs.com/package/mkdirp).


I actually added this block on purpose. It makes sense for a depose cb to end when the leader to be deposed dies, right?


Nah, looks ugly. Define inline helper like
```
inline void pos_to_msg(double sx, double sy, souble sz, geometry_msgs::Point &out)
{
 auto pos = pos_setpoint();
 out.x = pos.x() * sx;
 out.y = pos.y() * sy;
 out.z = pos.z() * sz;
}
```


Good job yay!