I finally had time to look at this, it looks good, I don't know hiredis that well, so I'll have to experiment over the weekend.


Now it's working in bootstrap3 (PurityIII)
But in bootstrap2 (Protostar) the closed group is still showing the "open" icon.


Sir,

I have connected the Grove multichannel sensor to the I2C port on my Grove pi + board which is attached to my Raspberry pi  2. I have saved the latest library in sketchbook folder. From my Grove Raspbian wheezy OS when I open Arduino 1.0.1 IDE and open the file from File --> Sketchbook --> Grove Multi channel sensor and verify it, I get a compilation error.
Is it because I'm using a Raspberry pi?
Is there another way to work it with Raspberry Pi. If I develop a script in Python would it work or would I be wasting my time as the device is incompatible. Thanks in advance.

Richard


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


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


Sir,

I have connected the Grove multichannel sensor to the I2C port on my Grove pi + board which is attached to my Raspberry pi 2. I have saved the latest library in sketchbook folder. From my Grove Raspbian wheezy OS when I open Arduino 1.0.1 IDE and open the file from File --> Sketchbook --> Grove Multi channel sensor and verify it, I get a compilation error.
 Is it because I'm using a Raspberry pi?
 Is there another way to work it with Raspberry Pi. If I develop a script in Python would it work or would I be wasting my time as the device is incompatible. Thanks in advance.

Richard


fuck that shit, won't be able to do anything in the evening again, so good luck if u still will try. :)


I don't understand line 7.


fast-forwarding end-to-end/timeseries to borshop-integration-237-bk-timeseries-empty-results = 6933d382


Same remark about optional SHA1 (+update doc if ok)


Please remove unnecessary comments(e.g #pip install flask...)


and it is same error as before @DevSwift


Useful to note this change, I was initially confused why we were getting notifications in Slack every time a new user joined, but that was because we automatically send them a direct message, which this previously was not filtering. :smile:


"Markus Alpers - Do, 8. Okt 2015, 11:53
Fast perfekt. ;) Die Übertragung ins Internet ist automatisch integriert, wenn Sie die Sprachen nutzen, die wir in dieser Veranstaltung behandeln. Bei den Spielen achten Sie bitte noch drauf, dass die Ähnlichkeit zu den genannten Titeln nicht zu groß wird; nicht dass Sie am Ende Probleme bekommen, weil da eine Klage wegen des Diebstahls geistigen Eigentums erfolgt. Viele Grüße, Markus Alpers" Also dem Projekt sollte nichts im Weg stehen.


why would you want to reap a `bash` with `tini`?


that isn't how format works...


@jamesthechamp Remove all .pyc files we don't need it


> It might be useful to think of the Mailstore interface as a data accessor for the underlying storage, not the actual data itself. In the same sense as an ORM.

Exactly. But if you modify the data in a mailstore, the same needs to be done for the underlying storage, right? And in my opinion, concurrency safe code should be the responsibility of the developer implementing the Mailstore, and not ours. Secondly, an immutable Mailstore does seem to work but we soon run into problems with code that retrieves modified messages.. like `MessageSetBySequenceNumber`. What if some other mailstore had a message with a sequence number of 2 removed? If it's intended to behave like an ORM, it should NOT return that message. However, with the design that you propose, it will do just that since it'll keep its own copy of messages (different from the underlying storage). If the Mailstore were truly immutable, it would keep a copy of each and every message that it had, irrespective of it was really deleted or not.

Another problem is with the Save function. What if the sequence number of a message is changed because some other message was expunged? With an immutable Message, it would have absolutely no idea of what needs to be done. It could go and overwrite other messages. But, if all the Messages were 'tied' to a Mailbox, the Mailbox could update what's needed (sequence number).

Moreover, when we go on to add functionality for adding, renaming and deleting mailboxes (which is the next step), managing different mutable Mailboxes and keeping track of what goes where is going to quickly become very complicated.

> If we were to switch to a mutable interface, we'd also need to go through and change all of the functions that modify a message struct (`AddFlags`, `RemoveFlags`, `SetBody` etc) and remove the Message return value, since it would otherwise be redundant and confusing.

You're right. This should be done. Also, I'm not actively working on imap-server anymore since I'm no longer with Monetas and don't have a good use-case for an IMAP server.


Added 'toy' solution in a single line


Could be, but this is following style from sceneGraphInvalidated().


wait, let me use %


Wrong!


beat me to it :package:


Do you think is it easily possible to move this DockerFile into a folder, to keep it compatible with old docker versions which don't support other DockerFile than DockerFile?


First, why not?

Second, the point of starting up with `bash` would be to do things interactively. There is no way of knowing what one might start and whether it has subprocesses in advance. So, the default/safe decision should be to always have `tini` up and running.


please no double-underscore methods. Python's name munging is nasty when you do this


Same remark about optional SHA1 handling


[Function.name](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function/name) is part of ES6 with poor browser support :(


Sorry, must've missed the original note.  The KINGSTON_1_35_VOLT flag was replaced with ALLOW_NON_COMPLIANT_DIMM to be a little less supplier specific.  It was a deliberate choice to default to not allowing non-compliant dimms on all systems.  The Centaur only supports running dimms at 1.5V so if you run a system with dimms that report at something other than 1.5V, you are running out of spec.  If you know that your dimms have been qualified to run at 1.5V then you need to do a custom build with this flag set to allow them to be used.

Note - you can also do it via attribute override of ATTR_MSS_VOLT_COMPLIANT_DIMMS=1 instead of a custom build.  The override text file would look like this.
    # Allow 1.5V DIMMs
    CLEAR
    target = k0:n0:s0
    ATTR_MSS_VOLT_COMPLIANT_DIMMS 0x1

See https://github.com/open-power/hostboot/tree/master/src/usr/targeting/attroverride for instructions on creating and applying attribute overrides.

It is unfortunate that so many of these unsupported dimms made it out to the wild...  The problem is that if we change the default to allow non-1.5V dimms, there is nothing stopping anyone from plugging a dimm in that really doesn't work at 1.5V.  That would lead to indeterminate system instability, maybe even damage to the dimm.


Ack.


published/RHCSA Series--Part 04--Editing Text Files with Nano and Vim or Analyzing text with grep and regexps.md

不是第 XX 行，就是“多少”行


please... take out the peerDependencies thing!!! it's totally unnecessary and we can use last stable mongoose!


looks like  "PocketMine 1.6 was released!" never happens :(


Well, it is form/selector anyway.


If you are using curly brackets with attributes please stick to the one attribute per line. It's more readable like this.


this will fail for anything without `__len__` (eg integers)


If i remember correctly you can remove `nzbget-$(PKG_VERS)-testing-r$(PKG_REV)-dist.$(PKG_EXT)` at the end because `PKG_DIST_FILE` is automatically added to the URL when downloading.

cc @Dr-Bean


You're right! Will do.


you can remove the parenthesis


@ggouaillardet I was thinking that we would use the 2.x APIs in the rest of the code, prefixed with "opal_hwloc_base". In the hwloc/base, I would then have those functions perform the actual call based on version. So if we are using v2, it's just a direct pass thru. If not, then we have a giant switch statement to convert HWLOC_OBJ_L2CACHE into separate level and cache values and call the v1 API.

This way the rest of the code base is cleanly v2. There is a slight penalty for using v1, but that will go away over time as the upstream packagers pickup v2. In a few years, we can simply remove the v1 code in the base.

Make sense?


Configurable. Very nice.

Yeah I'm sure somebody will figure out a way to cause problems with it, but then they always do, don't they? If we left every feature out that could cause problems we wouldn't have any features at all


- Are the access rights to this directory on cluster set properly so other users than `mhorst` can run the workflow?
- I guess that ideally this property should not be part of our code. Is it possible to make it a part of configuration of deployment environment (the configuration held in `~/.iis` directory by default)?


I haven't touched that code in years and don't even have VC installed to
confirm, but as far as I could remember you could just ignore it.

It happens because the EXTIO interface defined by HDSDR uses doubles, but
the rtl_srd library functions uses integer types. I guess if you want to
suppress the warnings you could use casts.

José Maria Araújo

2015-10-08 14:11 GMT+01:00 Gisle Vanem <notifications@github.com>:

> Hi. There probably have been a RTLSDR version with sample-rates taking a
> double type. But with the latest RTLSDR, there are many warnings from
> MSVC regarding this:
>
> ExtIO_RTL.cpp(195): warning C4244: 'argument': conversion from 'double' to 'uint32_t', possible loss of data
>
> Could this be fixed by a cast to (uint32_t) or just change
> samplerates::value to uint32_t?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/josemariaaraujo/ExtIO_RTL/commit/13883d72aab4413107d4521b26d982ea09602c6a#commitcomment-13662219>
> .
>


Could benefit from caching


Ah, it is not from ES6, it is added based on toString extraction. So we will have the name in all browsers: -)


Aha. :)


Again, can we move this configuration out of our code?


use a set instead for better complexity


I know. In the beginning I didn't realise it would be so long. I also thought I could get `yaml` to format those OpenClinica item tuples nicely (so that they looked like tuples), but no. And I was worried that I'd need a human to edit this file if they couldn't name all questions in CommCare in a way that could automatically be matched to OpenClinica items.

But it turned out this file is REALLY big because there are A LOT of forms. (We could actually cut out many of those forms, but I'd prefer to leave the app as it is.) And all questions can be matched (at least all the ones that need to be matched), so we don't need to edit it.

Do you think we should use a database instead of a file?


Make sure you write a readme


Any reason for `display: block`? Can’t `text-align` images with this applied, does it do anything that leaving them as `inline-block` doesn’t?


Nice work seh! Testcases are back at normal, and I found a NAL6 rule I forgot which I now implemented and which I will commit later. Only 5 failing NAL6 tests left, which is great! :)


Again, can we move this configuration out of our code and put in local configuration files?


:+1: Agreed.


@chirag04 what are you running to make it fail. I'm being able to run packager/packager.sh without seeing any error (asme deal for starting an example app on xcode).

@ide I noticed lots of failures when running `npm test`, all of them with the same `Object.assign` error. However reverting the change doesn't fix them. Was this what you were referring to?


One of the changes in this branch is removing the iis-3rdparty-pig-avrostorage project from the source. However, this project is currently still in the master branch. Is this because Pig in CDH5 supports Avro out-of-the-box, while CHD4 does not?


@trustin The `StreamBufferingEncoder` is only supposed to be used on the client-side.  It doesn't is not needed (and probably doesn't make sense) on the server-side.


Имам предвид всички функции които са закачени да се изпълнят на някакъв определен Event.
Callback го считам за alias на подадена функция на функция от по висок ред.

Но може би наистина "Асинхронно изпълнение на функции" е по точно като понятие :)


I'm ok with leaving it as a file.


The block is in there so they can be floated and margined, why text-align instead of float?
Don't see any harm in going inline-block on the aligned images.


Use 2 spaces for indentation in a hash, relative to the first position after the preceding left parenthesis.


Dependencies go to arch/common.spec, but it was a good catch. I'll check the arch packages for these new dependencies, tks.


@vvzar Probably agree with you


This module should be in a file called `src/Futhark/CodeGen/Backends/GenericPython.hs`.


Pas biennnnn :-p


I used to have square brackets in my screenshot URLs, had to change long ago because of compatibility with applications and styling. But yeah sometimes images have square brackets.

@swordling That is not a direct link to an image. What @JTBrinkmann posted is a direct link.


Hi Lucas:  Is there a chance  Less futility + more SE (10442788)   will now perform better in the lazy smp branch?  Thanks.


FYI there is a syntax error on this line. Could explain why the upstart script failed.


Looks good!


You are right I will change it :)


Not sure we actually need to work further on the renaming of a file when in ```Single Cell``` view since we are going to change that view quite a bit (e.g. #590).


hah I flashed rom from dmitry and I got more logcat:D


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


Ah! Actually, I believe there's a variable I should be using here - ajaxurl, as mentioned in https://codex.wordpress.org/AJAX_in_Plugins . I'll try making and testing an update now!


I flashed also boot.img from mako lol, but it hangs on audioflinger because of kernel I think


via fastboot?


hello


you should keep the original input format (lists) and not convert to sets


yep
I will upload extracted ramdisk


active for changing password, not email address.


You'd also need a `;` at the beginning in case the file concatenated before doesn't end with a semi-colons, since it was so simple I fixed it in my own commit, hope that's okay with you.


or just boot ?


Email address needs to be in the correct format, this will be rejected by the system.


try with my zImage so we can know if it kernel problem.


first I flashed only system.img and I got same error, then I flashed also boot.img and now it stuck on audioflinger


read me again


#me


I just ran npm start.

Seems like others also had the same issue. Here is the tracking issue: https://github.com/facebook/react-native/issues/3262


Explained [here](https://github.com/kslazarev/numbers_and_words/issues/115), but short answer; It's a rails convention.


I guess you forgot to remove "wiki=true".


mako ramdisk http://www50.zippyshare.com/v/6rxoTRqK/file.html


You should really learn to avoid exec. Why are you using it here? What's wrong with just building the result using "normal" code?


how we can build boot.img??


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


Morning Lucas:  I was wondering if tuning your previous work on aspiration under the lazy branch could find some elo?


The params here result in garbled text in the login form
![screen shot 2015-10-08 at 14 34 18](https://cloud.githubusercontent.com/assets/2912/10369387/e2634094-6dd1-11e5-9a07-070747519ea1.png). I think we should use a sentence fragment here as in https://github.com/mysociety/alaveteli/blob/develop/app/controllers/comment_controller.rb#L44 and revisit the approach in #2676


This can't be the right way to do this. Also doesn't it break for multiple evaluators calling the functions at the same time? (separate threads)


You forgot to update the tournaments bit (since you removed `Tools.getEffect()`) so it now only displays the format ID instead of name


SwitchyOmega会有bug，提示`System.Reflection.TargetInvocationException: 操作过程中出现异常，结果无效。有关异常的详细信息，请查看 InnerException。 ---> System.Net.WebException: 无法连接到远程服务器 ---> System.Net.Sockets.SocketException: 由于目标计算机积极拒绝，无法连接。 127.0.0.1:1080无法连接，`
换成switchysharp就可以了


数据结构


mka bootimage


No, I explain why just one line above :)


I honestly don't get the exec here. What's wrong with M['k'][j][i] = Data['k{}{}'.format(j, i)]?


これ前回の branch で消したやつ？いらないかも


I agree.


I'd prefer to still do this by adding a class that specifies the opacity and then removing the class when it's no longer needed. It would allow us to keep the visual considerations in one place.


This part is corresponding to the code at the moment,

https://github.com/parroty/extwitter/blob/master/lib/extwitter/config.ex

I wasn't thinking on this much on this naming, but if it's better, aliasing as `extwitter` could be one option (since just changing the name may cause breaking of previous code).


otherwise it won't boot


even with mako ramdisk


DEFAULT_SKIN is leftover from 12.x or 13.x. Now it reads default skin id from settings.xml :)


1. Создать двух людей: Klaus и Maria, создать одного HR (HR)
2. Нанять Клауса в HR
3. Поженить Марию и Клауса
4. Мария меняет телефон
HR выводит: "Старый телефон: xxx, новый телефон: yyy"


Nope, it is good.


In my life as a programmer I've seen a lot. From mesmerizing beautiful to horrid and repulsive code. But this commit is on another, transcendental level. Good work @pencil.


`not-for-print` class here doesn't seem to be having any effect - elements are still printed.


but definitely problem is in ramdisk or kernel


Line spacing is uneven.

![screen shot 2015-10-08 at 15 36 28](https://cloud.githubusercontent.com/assets/2912/10369573/e8bc7720-6dd2-11e5-814b-6ba6dd14340e.png)


これ「,」ついてないし，INT NOT NULL 省いちゃうと読み込めない気がする
あと，データベースのテーブルの内容変更したいんだったら，ここを修正するだけじゃ反映されなくて，このファイルを MySQL に読み込ませなくちゃだね

* 参考
>[ファイル読込｜データのインポート・エクスポート｜MySQL｜PHP & JavaScript Room](http://phpjavascriptroom.com/?t=mysql&p=mysqlimport)


Example usage within TravisCI: https://github.com/peterjc/galaxy_blast/commit/a1303d7c936c87b81217962fa69f501fa2367ddd and https://travis-ci.org/peterjc/galaxy_blast/builds/84294442

In this case I am using ``planemo dependency_script`` to run two ``tool_dependencies.xml`` files to install BLAST+ 2.2.31 and Blast2GO, which avoids duplicating relatively complex install recipes in ``.travis.yml``.

Note that the dependency chain here is being determined manually - for example at this point I am only wanting ``packages/package_blast_plus_2_2_31/tool_dependencies.xml`` for the current wrapper tests, and not the older versions of this package.


Careful with `$services.model.createDocumentReference('', 'XWiki', 'XWikiPreferences')` since `''` for wiki gets resolved according to the configured default model wiki reference (`xwiki`) instead of the current wiki reference. We want this to work on subwikis too.

Also, you could have used `xwiki.getURL(EntityReference reference, String action, String querystring)` for simplicity. I guess one could even argue that using 'XWiki.XWikiPreferences' instead of the reference API would be acceptable as well, to avoid verbosity.


I already removed it.


Fixed


faceUp is a better name


>I honestly don't get the exec here. What's wrong with M['k'][j][i] =
Data['k{}{}'.format(j, i)]?
I did not think of that to be perfectly honest, for some reason I never
thought that only needed to change the one string.

I'll use that, thank you.

On 8 October 2015 at 16:37, Carl Sandrock <notifications@github.com> wrote:

> I honestly don't get the exec here. What's wrong with M['k'][j][i] =
> Data['k{}{}'.format(j, i)]?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Stefan-Endres/DWMP-Mixutre-Model/commit/da9dadcf71baf1d343c969350554f703f8eb0874#commitcomment-13664356>
> .
>

--
Stefan Endres
Postgraduate Student: Institute of Applied Materials
Department of Chemical Engineering, University of Pretoria
Cell: +27 (0) 82 972 42 89
E-mail: Stefan.C.Endres@gmail.com
St. Number: 11004968


データベースどうなってるかよくわからないし CUI いじるのつらいようだったら，前にも言ったけど Sequel Pro とかいれるといいと思います

[Sequel Pro](http://www.sequelpro.com/)


This breaks upgrading players to V2. html5Url should be for 1.9.8


Don't forget the shape of the card
Heart, Diamond ...


google verification


OK.


merger


> First, why not?

srsly, this is not an argument. especially not for adding overhead.

i haven't yet encountered problems like with the Jupyter notebook that justify the usage of tini. i'd say it's usually safe to not use it and use it explicitly when needed.


better make it final


Yep, agreed.


I think it must be
'data-expand-icon' => 'icon-arrow-down', 'data-collapse-icon' => 'icon-arrow-right'
here and
'data-expand-icon' => 'fa fa-arrow-down', 'data-collapse-icon' => 'fa fa-arrow-right'
in bootstrap3-jlayouts


We've switched to using `pyrsistent.PClass` mostly; usage of `attributes` is legacy usage pre-dating that.


Why is this needed?


We do not need the return false, because function file_exists is returning it, if required file not found.
In summary the enhancement is working fine now, and the autoload stack switches to the next registered autoloader.


Don't need to document stuff already documented by the interface.


It is better to make it final


This is a lot of goodies! Was it all pylint/inpektor based?


Aaaah, good catch.  Thanks.


Makes sense


:+1: my bad for jumping the gun too early.


It is better to have 2 methods here setFaceUp() and setFaceDown()


The issue is with your test, not with this code. You're running up against the [limits of JavaScript's floating point precision][1]. For example, in the Chrome dev tools:

    > (-1618194002072370894).toString()
    "-1618194002072371000"

[1]: http://stackoverflow.com/questions/3428136/javascript-integer-math-incorrect-results


I think 'two factor' rather than '2factor' would be more comprehensible to the intelligent layperson throughout the user interface and documentation. It gives them a sporting chance to figure out that we mean that there are two factors that will be used :)


Ahh, that's what I was missing. Because each jupyter is an env install as well, I didn't think this could work. Thanks so much!


Thanks for info. I will try to apply attribute overrides on the palmettos and the habanero pass1 we have to keep them up to date. It seems to be a better option than to rebuild a new firmware each time.


Technically the path could be almost anything. Taking some parts from https://github.com/kevva/url-regex/blob/master/index.js might be a good idea if you want to catch _all_ valid URLs.


What are you using this for? Only ObjectId properties? This should use a custom field class.


After those changes, the test suite still passes after google closure advanced compilation (http://www.closure-compiler.appspot.com/home#code)

GC-A: 734 bytes gzipped (1.2KB uncompressed)
Uglifyjs: 1063 bytes gzipped (2.7KB uncompressed)


The hashCode should not depend on a mutable field because these will mean that when you put the card in a HashSet or HashMap as a key then change the open field then search for the Card you will not find it.


Is it working this way ? Keep in mind it's mandatory to be fully norm-compliant :)


Both objects and hashcode look to be unused imports


> Line spacing is uneven.

This was intentional, in that the print action is contained within the `one-time-password` div – I was thinking of them as conceptually different things, but maybe they could all go in the same list of buttons.

`not-for-print` working okay here – caching?

![screen shot 2015-10-08 at 15 47 47](https://cloud.githubusercontent.com/assets/282788/10369787/f344348e-6dd3-11e5-9828-b1453634f673.png)


> Careful with $services.model.createDocumentReference('', 'XWiki', 'XWikiPreferences') since '' for wiki gets resolved according to the configured default model wiki reference (xwiki) instead of the current wiki reference.

Actually the default hint used by the whole script service is "current".


what craziness did this fix?


Instead of checking for null and checking for the class you can do it in one step using instanceof operator because it is null safe.


`isDeleteKey` isn't being used here


Good shortcut :)


I don't agree with the first point.
On a subwiki, `$services.model.createDocumentReference('', 'A', 'B')` returns `subwiki:A.B`. I used it a lot in other scripts too without noticing problems. Maybe I could use `$services.model.resolveDocument('XWiki.XWikiPreferences', 'local')`, but it's not very user-friendly.

I agree for the 2nd point, I've copy/pasted this code from the original, and it could be improved more.


@thebluecat5445 ^ do you have anything to say to Joe?


Not anymore.... I'll fix it


It is also good to implement Comparable you will need it a lot


Good stuff :)


ごめん，古いコミット見てた...なんでもないです！！！


thx :space_invader:


Line length is longer than Drill style here.


I wonder why you capture the accidental result of emit and test it when you've cleverly captured the intentional side-effect of appending to $item within the mockNode? Better to `expect($item.output()).to.match`?

Perhaps you could also moved the expect.to.match inside the mockNode:

    $item.match /<i> a dummy video <\/i>/

(I am not sure mocha always reports errors well when employing this trick. You would have to check that.)


Looks good to me. r=me


Long line


+1, no check-ins sin pasar los tests 100%. Corriste los tests???


No worries, happy to help! :+1:

U SHIPZ IT NOW!


Monkey patching, because the ObjectIdField return ObjectId but it takes str as input.


This looks like it's non intended (see the index number 0):

```
./scripts/download_manager.py
Available download assets:

1 - CentOS 6.6 amd64 (http://mirror.isoc.org.il/pub/centos/6.6/isos/x86_64/CentOS-6.6-x86_64-minimal.iso)
2 - CentOS 6.6 i386 (http://mirror.isoc.org.il/pub/centos/6.6/isos/i386/CentOS-6.6-i386-minimal.iso)
3 - CentOS 7.0 amd64 (http://mirror.isoc.org.il/pub/centos/7/isos/x86_64/CentOS-7-x86_64-Minimal-1503-01.iso)
4 - Debian 6.0.7 x86_64 DVD (http://cdimage.debian.org/debian-cd/current/amd64/iso-cd/debian-6.0.7-amd64-CD-1.iso)
5 - Fedora 17 x86_64 DVD (http://download.fedoraproject.org/pub/fedora/linux/releases/17/Fedora/x86_64/iso/Fedora-17-x86_64-DVD.iso)
6 - Fedora 18 x86_64 DVD (http://download.fedoraproject.org/pub/fedora/linux/releases/18/Fedora/x86_64/iso/Fedora-18-x86_64-DVD.iso)
7 - Fedora 19 x86_64 DVD (http://download.fedoraproject.org/pub/fedora/linux/releases/19/Fedora/x86_64/iso/Fedora-19-x86_64-DVD.iso)
8 - Fedora 20 x86_64 DVD (http://download.fedoraproject.org/pub/fedora/linux/releases/20/Fedora/x86_64/iso/Fedora-20-x86_64-DVD.iso)
9 - JeOS 19 x86_64 (http://lmr.fedorapeople.org/jeos/jeos-19-64.qcow2.7z)
10 - JeOS 20 x86_64 (http://lmr.fedorapeople.org/jeos/jeos-20-64.qcow2.7z)
11 - JeOS 21 x86_64 (http://lmr.fedorapeople.org/jeos/jeos-21-64.qcow2.7z)
12 - Ubuntu 12.04.5 Server amd64 (http://releases.ubuntu.com/12.04/ubuntu-12.04.5-server-amd64.iso)
13 - Ubuntu 12.04.5 Server i386 (http://releases.ubuntu.com/12.04/ubuntu-12.04.5-server-i386.iso)
14 - Ubuntu 14.04 Server amd64 (http://releases.ubuntu.com/14.04/ubuntu-14.04-server-amd64.iso)
15 - Ubuntu 14.04 Server i386 (http://releases.ubuntu.com/14.04/ubuntu-14.04-server-i386.iso)
16 - Ubuntu 14.04.1 Server amd64 (http://releases.ubuntu.com/14.04.1/ubuntu-14.04.1-server-amd64.iso)
17 - Ubuntu 14.04.1 Server i386 (http://releases.ubuntu.com/14.04.1/ubuntu-14.04.1-server-i386.iso)
18 - Video sample for Spice tests (http://lmr.fedorapeople.org/video_sample/video_sample_test.ogv)
19 - Windows Utils ISO (http://lmr.fedorapeople.org/winutils/winutils.iso)
20 - Windows Virtio Drivers ISO (http://alt.fedoraproject.org/pub/alt/virtio-win/latest/images/bin/virtio-win-0.1-74.iso)
Type the index for the assets you want to download (comma separated, leave empty to abort): 0
HTTP Error 404: URL http://alt.fedoraproject.org/pub/alt/virtio-win/latest/images/bin/virtio-win-0.1-74.iso
```

Also, this locations is now deprecated:

http://alt.fedoraproject.org/pub/alt/virtio-win/latest/images/bin/deprecated-README

But I guess we can not just replace for the new URL, since they're DNF/YUM repos, and obviously with RPM files only:

https://fedorapeople.org/groups/virt/virtio-win/repo/stable/


Is it working? I am getting the error:

```
.rvm/gems/ruby-2.2.2/bundler/gems/rspec_api_documentation-538816497173/lib/rspec_api_documentation/writers/postman_writer.rb:10:in `block in write': uninitialized constant RspecApiDocumentation::Writers::PostmanWriter::Formatter (NameError)
.rvm/gems/ruby-2.2.2/bundler/gems/rspec_api_documentation-538816497173/lib/rspec_api_documentation/writers/postman_writer.rb:9:in `open'
.rvm/gems/ruby-2.2.2/bundler/gems/rspec_api_documentation-538816497173/lib/rspec_api_documentation/writers/postman_writer.rb:9:in `write'
```


@andrewkolas if you're ok with adding a customink core recipe, I can remove the unused gems and let the basic ones


**Code Quality - Critical**

Hello @andela-nmartin, why is inconsistent spacing and indentation still a problem?


Oops. the commit message has a few extra *Remove old spice video sample* lines.


@vineelsonnathi did this have a spec? somewhere?


nit, style: separa esta linea en varias


On RHEL based distros it should be `sudo yum install gstreamer-python*`. Note, I also had to install python-mock.


I think this can be a lazy variable?


you've specified integer not float, so is this needed?


good programmers always handle error ;)


As I mentioned in person I don't think we want to change the id bits as those affect the css used to render those cells in the table. I'd suggest installing the Firefox extension Firebug and modifying the html with it to see what ends up making the two lines align or look the way you want.


remove this since you are not handling memory warning


maybe a better key than this?


do you really need this, since you've already said an integer >= 0 ? it can't be a float. it might be good to specify a maximum (checked with sophie, she says 5000 more than adequate)
(this comment applies to all references here to numbers)


we only need to do this if there is a node playing?


remove unnecessary override


you need to call super viewWillAppear


need to find a way to have this a git tag rather than a version in code - maybe


Hmm...usually its a good idea to comment on every elements of the struct/enums. :) But I'm sure..


I have to move `<>` at the top of `[`...`]`. I wonder why...
Sometimes I surprised that I don't know well about the notation of regex.


Oops...comment was copy pasted. Will remove it.


this should be a struct, read: https://www.objc.io/issues/16-swift/swift-classes-vs-structs/


Getting this error due to this change, while installing the app:

Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/aditya/frappe-bench/apps/frappe/frappe/utils/bench_helper.py", line 79, in <module>
    main()
  File "/home/aditya/frappe-bench/apps/frappe/frappe/utils/bench_helper.py", line 16, in main
    click.Group(commands=commands)(prog_name='bench')
  File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 664, in __call__
    return self.main(*args, **kwargs)
  File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 644, in main
    rv = self.invoke(ctx)
  File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 991, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 991, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 837, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 464, in invoke
    return callback(*args, **kwargs)
  File "/home/aditya/frappe-bench/apps/frappe/frappe/commands.py", line 29, in _func
    ret = f(frappe._dict(ctx.obj), *args, **kwargs)
  File "/home/aditya/frappe-bench/apps/frappe/frappe/commands.py", line 164, in install_app
    _install_app(app, verbose=context.verbose)
  File "/home/aditya/frappe-bench/apps/frappe/frappe/installer.py", line 92, in install_app
    frappe.clear_cache()
  File "/home/aditya/frappe-bench/apps/frappe/frappe/__init__.py", line 410, in clear_cache
    get_attr(fn)()
  File "/home/aditya/frappe-bench/apps/frappe/frappe/__init__.py", line 743, in get_attr
    return getattr(get_module(modulename), methodname)
  File "/home/aditya/frappe-bench/apps/frappe/frappe/__init__.py", line 573, in get_module
    return importlib.import_module(modulename)
  File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "/home/aditya/frappe-bench/apps/mandrill_integration/mandrill_integration/webhook_events.py", line 12, in <module>
    @frappe.whitelist(allow_guest=True, xss_safe=True)
TypeError: whitelist() got an unexpected keyword argument 'xss_safe'


one should never call viewWillAppear or its siblings


gitignoreの観点やねんけど、
1. いらないファイルを"明示的"にする。
2. 共同開発の際に環境依存を消す。
の2つがある。
以下一般的なgitignore

local.properties
#for gradle, IDEA
.gradle
gradle.properties
!.idea/runConfigurations/*
.idea/copyright/
.idea/inspectionProfiles/
.idea/libraries/
.idea/*.xml
.idea/.name
build
*.iml

captures
.DS_Store

こういう形にして方がいいね。
ideaの中でも必要なものはあるから(runConfigurationsは必要。), xmlは全部いらないね。


> Actually the default hint used by the whole script service is "current". Using the default hint would have been useless 99% of the time.
Right, I forgot about that.


Agreed. :+1:


:)

We really need to do something about the output of those displayers.


missing newline


When is this returning nil?


**Prototype Functions - Critical**

The `.wordCount()` function should make use of the `.words()` function, as stated in the Checkpoint specs.

Here is a snippet from the Checkpoint's document below:

<img width="492" alt="screen shot 2015-10-08 at 4 00 54 pm" src="https://cloud.githubusercontent.com/assets/2857896/10370226/d0b925da-6dd5-11e5-9bed-7af084833060.png">


the image has been added twice – it wasn't meant to, right?


xml最後の改行は消す


This commit was a mistake. I did not intend to create this folder or contents.


"Add a `reset` Boolean parameter is added to control the behaviour of the function:" (in the commit message) "is added" is redundant.


This was for 02-Resources.pptx


性別のところ、DBさわることになっちゃうと思って後回しにしてました…


changing UI state like this is very fragile and could cause race conditions. a much better way is to update them in a single spot and derive UI state from our state. For example:

```
func recordAudio(sender: UIButton) {
  self.recordingState = .Recording // enum
}

var recordingState = {
  didSet : {
    updateRecordingButtons()
  }
}

func updateRecordingButtons() {
  // derive button states from self.recordingState
}


We've just finished merging the other CoA : https://github.com/odoo-dev/odoo/commit/1073e74f68382f4d096d2164be42482d6fe154e0
It hasn't been merged yet so any comment is welcome and appreciated :)


Capitalisation here doesn't match the rest of the form labels.


Need a comment to say how important this is.


annotate these methods with public/private, this is a homework for you to understand the difference and when to use them


How about the `bool HID_::setup(USBSetup& setup)` function? Doesnt it also need this?


@cagryInside Could you please take a look at this and confirm that I got the analytics code correct for this?

I'll ping you on two new activities as well for trip and stop problem reporting.


Great start, major feedback around how to update UI.


This should never return nil because it assumes we already checked we are inside a call. However since we are generalising its use, I should introduce a check. Then it'll return nil if not inside a call.

Does this also need to work when point is on a call name?


You're right! My original intention was to test against `$item.output()`. I'll fix that


You could rewrite `zero_extend` using `extend`.


Should be 0xD00B and 0xC00B


What is the advantage of this change?


nit, style: para mayor claridad, separa la asignación del if.


@keshavbhatt cool, I'll check it out!


I made ramdisk diff https://github.com/ProjectG/ramdisk/commit/a2a881199e8d822074dc604f311dcaae3cce423a


**Code Quality - Nitpick**

Use RegExp alone for this check.
Employ the `$` RegExp character to assert that the string ends with a question mark.


The for loop structure you proposed loops through an iterator. so we could do `for (auto c : str2)` if we wanted to iterate through the characters of a string.

If we do `for (auto i = 0; i < str2.length(); ++i)` the `i` variable will default to `i` and will have to be typecast to be compared to a `size_t`


After discussing, we decided we're going to leave as-is in the card. When the card returns from close(), essentially it's saying whether the list is allowed to open another card or not. Also added functionality for setting alwaysOpen on the list to trigger all cards to be always open. If this is the case, this becomes a moot point, since the cards will never call open() on the list.


AFAICT this test is effectively a noop when run (just based on what I'm seeing from Travis); I'm currently unable to run the `rpc-tests` locally to remedy, I think due to the make issue (https://github.com/bitcoin/bitcoin/issues/6778).

I'll continue working on these tests, but if we want to merge before then I'm okay with that also.


need end


You'll need to generate an App Password if you use 2-Step authentication on your google account.


I wonder whether, rather than requiring a specific non-nil check in the calling code, we require the calling code to explicitly say that the otp_code should be checked, by passing a 'require_otp' flag to the model - then the param itself can be nil and should fail validation. I don't think there's much in it, but it might be less easy to slip up in the calling code that way.


Related
http://stackoverflow.com/questions/54867/what-is-the-difference-between-old-style-and-new-style-classes-in-python


You could wrap this in a subshell instead.
```bash
(
  cd $COPY_DEST
  sha256sum * > sha256sum.txt
  cat sha256sum.txt
)
```


**Comments - Nitpick**

You should place your comments before the block/line of code they explain, as opposed to placing in-line.


It fails at least on incomplete calls:

```
    foo(sdsfds |

## or

  attr(x, "foo")( |
```


I don't think there's much of a conceptual difference to the user at that point - they're all just things you can do.


Clarification: When the card returns from `close()` it is not telling the list anything. It is returning success or fail. Did the close action succeed or fail? If the card is always open, close never fails.


yay...that was my suggestion in #2763 :stuck_out_tongue:


todo.goの修正は改行を入れただけです。


@peterjc Neat!


I removed this option, since I do not find documentation anymore in Liberty about this option.


I wonder spell checker can be run on travis against commts, comment and string literal.

Function and variable names will be reported as spell missings by the spell checker.
But it is not an issue for this project. You may know well why.


👍 looks good!


OK let me know what I can improve.


We will switch to FMDB/standard in the next bug release (v3.1.1)


Thank you. Fix it.


I think this should be 5.18.1 ?
![screenshot from 2015-10-08 17 10 20](https://cloud.githubusercontent.com/assets/2264697/10370530/a4a9627a-6ddf-11e5-8027-be4b8b058a7e.png)


Yeah – I was thinking that the latter two are more "danger zone" items. I think a slight rewording of the confirmations to include the action ("Are you sure you want to disable two factor authentication?") is probably the best way around it though.


Really nice, just two suggestions:
- move the disk_size to the variables with the default 1000, so very small or large VMs can modify this in a var-file
- remove the packages provisioner, as all VMs have different requirements. The packages can be installed in the setup_script (as you did it for firefox). The switch for the tce directory to the hard disk can be done at the end of the hd-install.


The range needs to be updated.


Hie .. I cherrypicked this commit and fwb commit d 2nd part of this commit .. then i forked Screenrecorder repo from manifest and compiled ... When i click on screenrecorder, there is no response .. Nothing comes up ... So can someone please tell me how to get working Screenrecorder ?

Hope someone replies

@Cl3Kener @0xD34D


@barbeau Yes, this is correct. Also, for fragments you can call `ObaAnalytics.reportFragmentStart(this);`


Rename the path, and get rid of all the PHP stuff and I can merge this in as a placeholder.


What kind of caching problem are you thinking of?

![screen shot 2015-10-08 at 16 11 05](https://cloud.githubusercontent.com/assets/2912/10370525/3ca329c0-6dd7-11e5-88ce-243dff24ee7f.png)


this is a test repository.It is my first project.


Do you need to have this reset boolean?   Can't you just fill in the pool level flags if the current flags are empty, and otherwise upgrade them if necessary?   Boolean parameters like these which completely change the meaning of functions seem ugly.


Got a code example of the interface you were thinking of?

Maybe `@user.valid?(:check_otp => true)`?


I haven't read the code so I have no idea what you are trying to do and therefore what the right answer is, but having done a lot of work in the past on the mapping of Joomla icons and Bootstrap Icons and Font Awesome icons (and certainly with BS2, there was a difference between BS2 and FA), I know that you have to be quite careful about what icon names you use.

At least Font Awesome has switched from icon-xxxx to fa-xxxx which avoids clashes of names between Joomla and FA.

If you want any help with this, drop me a line.


@soitanen

Hi Soitanen.
I remember you said this was still Work in Progress.
Are you planning to make the needed fixes, and polish this FDM?

If so, let us know
@IAHM-COL


> especially not for adding overhead.

As reaping is proper behavior in Linux and will soon be in `docker`, I am quite happy to keep `tini`. Reaping is planned behavior for `docker` in general ( https://github.com/docker/docker/issues/11529 ). So, this will be the default overhead. That being said, I have seen no notable overhead from using `tini`.


revert pls you break all my stuff


"A", not A


Let me know if this is alright. I am quite ignorant of elisp/spacemacs idioms.


jbrowse is expecting 'bundle_jbrowse' version 1.0 so I guess this would do the job:
```
s/package_perl_bundle_jbrowse/bundle_jbrowse/
```


Ok, thanks.  I haven't added any new fragments so haven't had to use that yet.


I implemented a method `getPoints()` since there wasn't one.


Maybe having switched from master which might not have the `not-for-print` style? Might need a `bundle exec rake assets:clean`? Will check in other browsers.


Certificates by definition don't contain sensitive data. Is this is being marked sensitive to keep error output from being too unwieldy?


this comment is misplaced


It also doesn't produce a useful span on nested calls:

```
df[a][, |]
```

> Does this also need to work when point is on a call name?

I think it should produce results irespectively of the location. We need to get full context around point.  Once we have that we decide at elisp level if it's worth asking R for completion. It's not meaningfull to ask cimpletions for `{` call or `(` if not followed by a call or something that doesn't evaluate to a call.


I commented out the original `calculate()` method.


hahaha... That sounds interesting, until you ponder the thought of how to do it.


Maybe extend the KeystoneHelper?


const useSSL = 'https:' === window.location.protocol;
        gads.src = (useSSL ? 'https:' : 'http:') + '//www.googletagservices.com/tag/js/gpt.js';
Doesn't http/https get automatically set if you have an URL like '//www.googletagservices.com/tag/js/gpt.js'?


![holdup](https://cloud.githubusercontent.com/assets/1294399/10370706/17704efe-6dae-11e5-9fb4-849d32552806.gif)
i didn't actually cancel!!


Arrays which hold the number of found faces for each source and target face.


Test


@yhoogstrate we cannot change it anyways, these decisions we have to live with


Did this fix a bug?  If so, what bug?

The new try/except should be split into two parts so we don't inadvertently mask legitimate errors:

    try:
        value_map_json = json.loads(...)
    except ValueError:
        continue
    try:
        value_map = dict(value_map_json)
    except TypeError:
        continue
    ...insert rows...

(I prefer asking isinstance over relying on Python's automagic type-conversion-with-occasional-exceptions here, in spite of the Python world's generally frowning on isinstance, but it's not a big issue.)


Thank you!
Also please look here is some examples need to be changed.
https://github.com/boomstick8x/Mycodo/commit/8e60b6e73e97c31718d99ec92e01325019487b65
p.s. sorry for ma bad english


Arrays which hold the intersecting faces for each source and target faces.


yes it sure would! There are numerous problems with these JBrowse packages, thanks for catching this one @yhoogstrate. I was planning to nail down all of the installation bugs with the next release of the JBrowse tool (v0.4, includes lots of new features)


No need to declare any variable in ruby. At least, not with var.

Plus, if you want your instance attributes to be visible, you need to declare a getter in attr_reader (don't forget the colon ":" before when declaring them.)


Transfer the addresses back to the CPU.


Style question: Would this be easier to read if the here-doc was replaced by double-quoting line 228?


Well, it's just a warning :-) Would you prefer that it checks for empty entries and skips them by doing nothing, rather that spitting out a warning?


If this fixes a bug, we ought to have a test case for it.


Highest level abstraction. I don't think our first instinct should be to go to Macros when a method has too many parameters - we should instead design better APIs.

If you want to reduce typing, we can use a more appropriate object oriented pattern. Just pass an object that conforms to a protocol which can provide the data store, recent pages, and saved pages.

Additionally, I know there was just code refactored to only pass the data store (instead of recent pages, saved pages, et al) to avoid this type of issue, why not just do that here?


More like `@user.require_otp = true` - then call `save` or `valid?` as required.


@peterhurford ^ , also appended this?


this commit is for Cm-13.0


Nice.  I was going to add that, but forgot!


:)  stefan@borgia.com


ROFL


I forgot to run the mvn xar:format


Bumped Retrofit version to beta2


normally, the /usr/bin/env/R is (according to internet) a matter of shell profile or bashrc or something like that. The idea is that it takes the default R of the shell of the person, no matter where it is actually. e.g. in my case it is not in /bin/R. I think this would be worth looking into it  for BIWS if you have time...


Hi Aditya,

You need to update your frappe app.

-Anand.

> On 08-Oct-2015, at 8:30 PM, Aditya Duggal <notifications@github.com> wrote:
>
> Getting this error due to this change, while installing the app:
>
> Traceback (most recent call last):
> File "/usr/lib/python2.7/runpy.py", line 162, in run_module_as_main
> "__main_", fname, loader, pkg_name)
> File "/usr/lib/python2.7/runpy.py", line 72, in run_code
> exec code in run_globals
> File "/home/aditya/frappe-bench/apps/frappe/frappe/utils/bench_helper.py", line 79, in
> main()
> File "/home/aditya/frappe-bench/apps/frappe/frappe/utils/bench_helper.py", line 16, in main
> click.Group(commands=commands)(prog_name='bench')
> File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 664, in __call_
> return self.main(args, *kwargs)
> File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 644, in main
> rv = self.invoke(ctx)
> File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 991, in invoke
> return process_result(sub_ctx.command.invoke(sub_ctx))
> File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 991, in invoke
> return process_result(sub_ctx.command.invoke(sub_ctx))
> File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 837, in invoke
> return ctx.invoke(self.callback, *ctx.params)
> File "/home/aditya/frappe-bench/env/local/lib/python2.7/site-packages/click/core.py", line 464, in invoke
> return callback(args, *kwargs)
> File "/home/aditya/frappe-bench/apps/frappe/frappe/commands.py", line 29, in _func
> ret = f(frappe._dict(ctx.obj), *args, *kwargs)
> File "/home/aditya/frappe-bench/apps/frappe/frappe/commands.py", line 164, in install_app
> _install_app(app, verbose=context.verbose)
> File "/home/aditya/frappe-bench/apps/frappe/frappe/installer.py", line 92, in install_app
> frappe.clear_cache()
> File "/home/aditya/frappe-bench/apps/frappe/frappe/__init.py", line 410, in clear_cache
> get_attr(fn)()
> File "/home/aditya/frappe-bench/apps/frappe/frappe/init.py", line 743, in get_attr
> return getattr(get_module(modulename), methodname)
> File "/home/aditya/frappe-bench/apps/frappe/frappe/init.py", line 573, in get_module
> return importlib.import_module(modulename)
> File "/usr/lib/python2.7/importlib/init.py", line 37, in import_module
> import(name)
> File "/home/aditya/frappe-bench/apps/mandrill_integration/mandrill_integration/webhook_events.py", line 12, in
> @frappe.whitelist(allow_guest=True, xss_safe=True)
> TypeError: whitelist() got an unexpected keyword argument 'xss_safe'
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/frappe/mandrill_integration/commit/22acb44e0a20b2fc67b1be40e387e42435ac19cd#commitcomment-13665142>.
>


this returns nil with the last commits


We should actually implement them, not remove this...


A pull request? I haven't received any: https://github.com/kizniche/Mycodo/pulls


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


Probably all that is needed is another rule in the grammar, another semantic action in the parser, and a test case.


Hi @martenson whoops I think I didn't make myself clear. Jbrowse can't find perl '5.18' (says 'Never Installed') and gives an empty green box everytime I install it. When I browsed to the perl package I saw that the version was named '5.18.1' which explains why jbrowse can't find it.

![screenshot from 2015-10-08 17 19 33](https://cloud.githubusercontent.com/assets/2264697/10370809/e121d24a-6de0-11e5-87f6-269aacc498ea.png)


I'm on the PR branch, have the style and have run `bundle exec rake assets:clean` - running in FF.


Given we only catch and error if we have a known input, and otherwise raise an error, I see very little risk of masking -- some error will percolate. The major advantages of this code are brevity and clarity. Separating the two tries would impede that clarity, masking the intent we're trying to achieve.

Python world frowns on isinstance because it promotes code on which type inference is hard (for both machines and humans), and where one is left scratching one's head as to why this thing might be a float.


This adds a smell since it's catching a more general exception now. The `ValueError` is an implementation detail of the way the `get_target_dm` is written.


@kizniche No, not a pull request, just changes in my branch of your `Mycodo project`, just testing how notifications works.


are you sure you're not looking for `ess-continuations-bounds`?


false?


Why is this definition of __init__ needed?


Fixed in e9b076d8c2d83ba6e712b67a2742f159ef76a861


There is no Canary for Linux AFAIK, this is the Dev Channel; see https://www.chromium.org/getting-involved/dev-channel. cc @vvo


ahh, sorry :)


There they are!
<bug?


From where did you cherry-pick


Why this change?  f is actually a local variable here, and I don't see what the word 'source' adds.


Thanks!


This was included intentionally so that you could copy & paste that fragment of the setup.py into another one without worrying about what goes on top.

It's a poor system -- but so is the rest of setuptools.


Many of the changes here should be reflected in Venture and possibly other repositories too.  (Maybe you've already done that -- I haven't checked.)


>If we left every feature out that could cause problems we wouldn't have any features at all

Yeah but we could at least *try* to avoid potential hurdles. For example what I said above about locking certain elements in the title bar so they're always present. I believe it's quite help to always know which renderer an end-user is using and what their resource usage is.

But since all of this is .ini-only at the moment and isn't being shown to the end-users in the GUI, I think we're fine for now. The point @willkuer brought up does concern me though. There's always going to be those people who distribute their own "super awesome builds" and shit like that. -_-


Resume test


Do you have the formulas for the other skills (howling moon and searing light)? Is thunder II formula similar to the other avatar skills (fire II, etc). If we have all the data we need lets get this done.


And if it doesn't, then we should eliminate the code. I'd be glad to review an additional test, but I suspect that neither of us has the stomach to extensively test legacy models.


I don't understand the purpose of this change. Please document it somewhere then submit.


:metal:


That'd be great. Until they're implemented, having this dead code is a mistake.


I started doing the other scripts last night. I'll finish them tonight


@do3cc  I don't think anybody mentionned so far that Fulvio's work flow is an/the exception. ;-) I've been struggling myself the last year with 'containment' and when to let which tool/system X perform functionality Y. They all seem to blend :-S.

We (Zest) used to keep everything 'inside' the buildout directory for all our projects, but with the advent of configuration management systems and overlapping functionalitty and the compass/ruby/node dependencies there it's very difficult to set a straight boundary there, also with buildout 2.X explicitly no longer saying it does isolation, but in practice it still does because of it's sys.path resetting.  Nodeenv has been on my list to grasp for quite a while now, I did some experiments with gp.recipe.nodeenv but it sucks at isolation. and npm has a slightly different packaging model to grasp.

Concerning training and virtualenv/virtualenvwrapper: I did some one one one training last month at a customer to help one of their developers get up to speed, (used Mastering Plone there as well). He had recently discovered virtualenvwrapper, insisted on using it and I watched him go astray as he installed some packages like mr.bob and 20-30 minutes later on couldn't find back any installed scripts or where they went.  "Psssst they're probably in your .venv in a subfolder", you were using venvwrapper,  right? ;-)"

For pro users like us the n:m virtualenv/project setup is great if you know what you're doing, but for newbies and training I tend push them to the "one directory to contain them all" model.


Then our builder should not use it on the server side and `StreamBufferingEncoder` should have a sanity check, which don't seem to be the case. Shall I send a PR?


I had a bug and used these two Syso's to check whether the method and if-statement were actually reached. I fixed the bug and therefore removed these two syso's.

<bug


Let's keep any simulator development in the simulator branch. In fact I think I'm going to remove the simulator package from master .


i.e.
```lisp
(ess-climb-outside-call)
(ess-continuations-bounds)
```

Though climbing outside calls does not handle chained brackets yet.


In the beginning of the round of storage update, we copy the master version of storage, download updates, parse new metadata, process existing jobs, etc., check the master version number, and simply substitute the storage if it didn't change. If it did, then some (or all) objects in master storage are more recent, so we do ```merge()``` in order to have all things latest. If all items in master storage are up-to-date, ```have_newer``` is never set to true.

So it is not *used* through the entire workflow, but it is calculated during ```merge()``` -- once you encounter an item, be it group, couple, or node backend, more recent than correspondent one in other Storage, you set this flag. And it's not global, it is a local variable passed to callees by reference. BTW, it reduces amount of code -- you just do nothing when nothing newer found, no need for trivial checks of ALL returned values.

So I think I have to write detailed comment on this function. And probably rename this variable -- for instance, will ```have_more_recent_items``` be more descriptive?


@trustin no need... I will incorporate the change into https://github.com/netty/netty/pull/4320


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


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


I don't fully understand the difference, but I think it's possible that old style classes don't call super by default, so Command may not have been properly initialized.  At any rate, lint suggested I fix it, and  I complied.


Modificado por Felipe


After splitting Flex.java, and making changes to Task.java, FlexTest1.java, FlexTest2.java, FlexTest3.java and FlexTest.txt, for the sake of abstraction into more classes, the new classes are as follows:

CRUD.java
SortAndShow.java
ShowDays.java
Checker.java
LastAction.java


> are you sure you're not looking for ess-continuations-bounds?

Probably yes. Then why did you add that that new function exactly?

I have added etc/completion_ideas.R for what we want to implement. Please add stuff to it. We will make a test file from it later.


Вроде серьёзный человек, а табы с пробелами мешаешь.


BTW not related to your commit, I don't like the hint name of this resolver,nor the class name (CurrentGetDocumentReferenceDocumentReferenceResolver). I'd have preferred to name it something like "currenthome" and CurrentHomeDocumentReferenceResolver.


@cagryInside New analytics for ReportStopProblemActivity.


@cagryInside New analytics for ReportTripProblemActivity.


I will add one for session, guild etc.


But isn't this an option from the keystone middleware (not swift)?


I would just put defaultCountry, flagsImagePath, and onchange on different lines so that it's a bit more legible. Great Stuff!


You're right, it doesn't add much.


@cagryInside I added this code so that the stop/trip problem activities will close after the user successfully submits a report.  I believe it shouldn't affect your use of fragments in the improved issue reporting WIP https://github.com/OneBusAway/onebusaway-android/pull/328, but please take a look to confirm.


Agreed that much of this should make it to other repos. No, I haven't gotten to it yet. I have a backlog. :-)


thanks, great!


> Probably yes

Actually not. I want the full meaningful context including the, potentially nested or chained, call name.


pycharm has an import optimizer that I wanted to use in our repos for a while. Now I have to modify inspektor to start screaming when unused imports come up. About the enforcement of module locations, I'm afraid I won't be able to do it.


OOOH BABY


@janecakemaster: let us know by 5pm whether you're going to speak, so that we can find a replacement if you're not.


Not related to this commit, but about the rotation of logs, take a look at: https://www.npmjs.com/package/log-rotate

(Just saw it, I don't know if it's good)


TeamCity pull requests :: netty [Build 384](https://garage.netty.io/teamcity/viewLog.html?buildId=2148&buildTypeId=pulls_netty&guest=1) is now running


@barbeau Yes, I am only using the fragments.


lulzk


Note that we should commit any changes related to ```merge()``` and ```have_newer``` in another changeset, this pull request is only about jobs/mongo.


This should be OK.


このファイルはローカルだけに保持しておくべきものなので、.gitignoreに入れる。


Hey @elvingm -- You can totally do this.

You can do this with the concept in Go called `interfaces`, which also exists in many other languages.  In many OOP languages, you'd be able to accomplish what you want to do with either generics, or using `Object` but in Go it's handled a little bit differently.

We're going to want to use `interface{}` (an empty interface) here instead of `[]Group`.  Why is that?  Because using interfaces, an object [IS-A](http://www.w3resource.com/java-tutorial/inheritance-composition-relationship.php) object that matches the interface if it implements all of it's functions.   Because we're passing an empty interface, EVERYTHING matches it, thus allowing you to pass in anything.


why not using CopSTASHPVN_set to save the flags rather than restoring the flags at init time ?


You're totally welcome to do it, but it's not mandatory (even stylistically) in Go.  I've seen developers do both.


@rzarzynski I'm concerned about the retry here. Is the objv_tracker going to be populated correctly if we ended up racing? Also, if we ended up racing with another writer, we need to restart everything, otherwise we might introduce obsolete data.


Add session


Thanks, can you make a PR please?


Now unique. A create will now fail, and we don't catch the exception, but the only place a user is created is with `get_or_create`, which will do a case-insensitive get first anyway.


xmlでやる方はそれでいいね。
動的にjavaのファイルでやる方法もやってみようか。
setOnClickListenerを利用したやり方でやってみよう。


already did it


Exactly.


have this be a prefix variable. The variable is set by checking if the known location exists, next whether some OS Env is set (i.e. `os.Getenv("DOCKER_HOOKS_DIR")`?), else look for a `./hooks.d/` relative to the where the docker binary was invoked from


@tmilewski might be able to help you with this one


True, the alternative is to pass in `target_module` purely for the sake of
using it in the exception. Though that doesn't seem as bad as it did when I
first thought about it.

On 8 October 2015 at 17:23, Daniel Miller <notifications@github.com> wrote:

> This adds a smell since it's catching a more general exception now. The
> ValueError is an implementation detail of the way the get_target_dm is
> written.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcare-hq/commit/78594a29496aa2468306f35a5074e45b6ee455f6#commitcomment-13665864>
> .
>

--
Simon Kelly
Senior Engineer | Dimagi South Africa


@rzarzynski make it a string reference to avoid copy


k


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


oops!  Thanks for fixing :)


eeek, so many problems with the v0.3 release. :crying_cat_face:

Thanks for getting these issues worked out @yhoogstrate. I'm doing development on the `jbrowse04` branch, however that tool is currently non-functional because I haven't finished a huge rewrite of it. I'd recommend opening a branch/PR for `jbrowse-v0.3` package fixes since you're motivated, and that you upload the testing packages to the TTS :)


Totally. I just didn't change things when the code was refactored from a 2 line script to a single line.


How does this work exactly? Are all the relations automated?


meinst du sowas wie last.fm?


> Then why did you add that that new function exactly?

It's used for call filling. It was moved to a better place in the commit.

> Actually not. I want the full meaningful context including the, potentially nested or chained, call name.

That should do it. Did you step out of the call before calling it, as I suggested above?


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


Really?? Now that's powerful. I think I'm starting to like closures...


:ok_hand::eyes::ok_hand::eyes::ok_hand::eyes::ok_hand::eyes::ok_hand::eyes: good shit go౦ԁ sHit:ok_hand: thats ✔ some good:ok_hand::ok_hand:shit right:ok_hand::ok_hand:there:ok_hand::ok_hand::ok_hand: right✔there ✔✔if i do ƽaү so my self :100: i say so :100: thats what im talking about right there right there (chorus: ʳᶦᵍʰᵗ ᵗʰᵉʳᵉ) mMMMMᎷМ:100: :ok_hand::ok_hand: :ok_hand:НO0ОଠOOOOOОଠଠOoooᵒᵒᵒᵒᵒᵒᵒᵒᵒ:ok_hand: :ok_hand::ok_hand: :ok_hand: :100: :ok_hand: :eyes: :eyes: :eyes: :ok_hand::ok_hand:Good shit


ここは基本的なコードの書き方やねんけど、publicは他のクラスで使うならpublicやねんけど、このクラスないでしか使わないメソッドならprivateにしよう。
よく言われる書き方としては、全部privateに書く癖をつけて、そのあとで外部で呼び出すメソッドとして必要ならば、publicに変えるというもの。
意識付けで始めはいい方法やね。


I'm not aware of such an automated spell checker.
But Vim allows for spell checking errors in comments and strings.


(I'm working on chained calls)


Why you supply `end` here? why not just `**kwargs` and delegate?


Yes I tried to break the required changes in logical non breaking changes.


SupportLibrary v7かな？AppcompatActivityっていうものがあるからそれを継承するようにしよう。
でもまずはActionBarActivityでやってみようって話やったやんね。


right, but nothing actually validates the sha1 sum, like is done for `INITSHA1`


Божественный щит - карта не получает урон в первый раз, когда ее атакуют.
Маскировка - карту с этим эффектом нельзя атаковать до тех пор, пока она сама не атакует.
Неистовство ветра - карта может атаковать два раза за ход.


sorry, i meant another example _instead_ of Registermachine ;-)


Lines form 57 to 70 look very similar, maybe one could refactor it to some function which would tak one argument "host", "port" etc


Already fixed in the develop and release-3.3.0 branch, CGNS-26. Will be merged to master when 3.3.0 is released.


I like this! Nice to warn them of unintended consequences.


Ah okay repeated arguments are not allowed, but then I would just delete/replace the `file` key, I think this is cleaner^^


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


Isn't there a CIGAR operation for a Match/Mismatch? This is why I had OP_MM.


@miketaylr ooops, I removed `data-labels` I didn't see it on another, view. Why there is not this attribute on the other Issue view?


Yes, I just realized. Reverting this.


As far as I can see, `socket` can never be null here.


基本的にやねんけど、文字列を使用する際には、
1. そのクラスのフィールドにconstrants(定数)として定義
2. (リソースファイルの利用)[http://so-zou.jp/mobile-app/tech/android/resource/]
の二つやね。二つとも覚えよう。


I tested with replacing `builtins.__dict__["print"]` with just `builtins.print`, and tests do pass (both, Windows 10 and Fedora 22)


大小写请与实际文件名一致,Model.h


Why these options have been removed ?


これもリソースファイルのdimenを利用しよう。
dimendで取得する際の名前(key)とそのdip(pixelみたいなもの)が設定できる。
それを自由に使える。


これも同じくリソースのstringを利用しよう。


file 有个hasData方法,判断如果有数据就直接拿图,没数据就异步获取图片.
图片不大,不需要获取缩略图,直接拿原图然后裁剪为方形比较合适


Why the node_submitted_page option is removed in multisite_drupal_core.install but not here ?


This change is not actually related to the issue, but I found the `SocketServer` code a bit messy and hard to understand, so I cleaned it up while debugging. If you'd prefer these changes to be done separately, let me know.


else是不是需要展示默认文字


The only thing I would add is that the https://github.com/plone/training repo
is not really where trainees are expected to run any commands.  Sure, the
more ambitious ones might want to build the docs themselves, but they will
also know what they are doing.

On Thu, Oct 8, 2015 at 5:29 PM Fred van Dijk <notifications@github.com>
wrote:

> @do3cc <https://github.com/do3cc> I don't think anybody mentionned so far
> that Fulvio's work flow is an/the exception. ;-) I've been struggling
> myself the last year with 'containment' and when to let which tool/system X
> perform functionality Y. They all seem to blend :-S.
>
> We (Zest) used to keep everything 'inside' the buildout directory for all
> our projects, but with the advent of configuration management systems and
> overlapping functionalitty and the compass/ruby/node dependencies there
> it's very difficult to set a straight boundary there, also with buildout
> 2.X explicitly no longer saying it does isolation, but in practice it still
> does because of it's sys.path resetting. Nodeenv has been on my list to
> grasp for quite a while now, I did some experiments with gp.recipe.nodeenv
> but it sucks at isolation. and npm has a slightly different packaging model
> to grasp.
>
> Concerning training and virtualenv/virtualenvwrapper: I did some one one
> one training last month at a customer to help one of their developers get
> up to speed, (used Mastering Plone there as well). He had recently
> discovered virtualenvwrapper, insisted on using it and I watched him go
> astray as he installed some packages like mr.bob and 20-30 minutes later on
> couldn't find back any installed scripts or where they went. "Psssst
> they're probably in your .venv in a subfolder", you were using venvwrapper,
> right? ;-)"
>
> For pro users like us the n:m virtualenv/project setup is great if you
> know what you're doing, but for newbies and training I tend push them to
> the "one directory to contain them all" model.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plone/training/commit/08fc5d341bb3f7e1ae87bcdad7a0f7f85c6c4b90#commitcomment-13666014>
> .
>


Now works with chained calls.


Lẽ ra m nên code luôn đi rồi push một lần :v:


good job


@smartAgent47  You need to include libscreenrecorder as well as the app in your PRODUCT_PACKAGES
https://github.com/DARKPOP/vendor/blob/dark-5.1/config/common.mk#L108


This is not great, so let me know if there's a better way to handle it (see `selector.disconnected` below to see where it's used).


> Actually not. I want the full meaningful context including the, potentially nested or chained, call name.

So you need to step outside all nested calls? I can add a plural version.


ko thich


biet may dinh nghia socketclient sao


Presumably: Remove store permissions from PCC.


This is only kind of safe if PlayMusicCL would run as it's own user and set the permissions of the config file to 600. Would be nice if it could remember the password, specially for 2-Step users. In the meantime, it's bigish security problem for your google account.


Can pyrsistent do PVectors of Interface providers? I think that's why I switched to this.


Oh my goodness.


Hiểu rồi, ok, t sẽ thêm phần đó cho tụi mình
:+1:


TeamCity pull requests :: netty [Build 384](https://garage.netty.io/teamcity/viewLog.html?buildId=2148&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5219, ignored: 21 Build time: 00:15:24


@ronchifabricio @JoeVogel Quem é Fernando B.?


print has no other kwargs does it?


this `i32BinIndex` result is always -1 on my builds (and then vector crashes because of negative indexing :)


but that we'd need to change consistently everywhere right?


But we dont return the return of file_exists. Therefore our function has an explizit return false.
I dont know if we must return something but in some examples they do http://php.net/manual/en/function.spl-autoload-register.php#113362


Will do.


Wow Why did you change that


I was debugging some problem at that time and trace back had clue. Sometimes it is useful, but I am fine, we can change it


oh yes ... oops lol.. i forgot that .. .

thanx alot brother ..


@jdegoes

> Anything encoded via return values will be opaque to interpreters

I think I see what you mean, though in this case, my intent was to allow interpreters to respond with an error back to the user of the api, i.e. the constructor would look like (in GADT form)

```scala
trait ReadFile[A]
final case class Open(file: RelFile[Sandboxed]) extends ReadFile[PathError \/ ReadHandle]
```

Adding a `PathError` constructor to the algebra would allow communicating the error to the interpreter, but not the user, no? Is there a better way to achieve this, or should this sort of thing be avoided (if so, why)?

>  "module fashion". Any reason to prefer that (e.g. better type inference) than "function style"

In my experience, type inference is much improved, implicit search works (period) and the the API as a whole integrates better with other libraries as we're able to use type aliases like `type F[A] = Free[S, A]` to partially apply `Free` in the module, enabling all the methods then return types in terms of `F`.

I have tried the function style before and it does work, but it is much noisier, especially when `Inject` is involved as you're forced to annotate types more often.


ytytjt


don't know, but if someone hacks in his own implementation we don't interfere. I would do like this:
```python
def newprint(*args, **kwargs):
    kwargs["file"] = sio
    oldprint(*args, **kwargs)
```


I do like the launcher. But more for managing envs and packages. For launching it's a bit too slow for me and not native enough to the OS. But that's personal. Someone else might quite like the experience of having all conda apps together.


Huh why? I mean it's definitely better readable. We can open an issue for that to change it everywhere^^


I'll do.


Klingt vordammt gut und trotzdem umsetzbar
.


Doesn't this create an infinate loop?

<logic


Dosen't the RT-AC66U have 128MB of flash?


Avoid variables with short names like $id. Configured minimum length is 3.


PHPCS: The closing brace for the class must go on the next line after the body


PHPCS: Line exceeds 120 characters; contains 125 characters


Method GroupController::init does not have a Docblock comment.


Method GroupController::index does not have a Docblock comment.


Method GroupController::edit does not have a Docblock comment.


Method GroupController::delete does not have a Docblock comment.


Method HomeController::getGroupInfo does not have a Docblock comment.


Method ProjectGroupStoreBase::getByPrimaryKey does not have a Docblock comment.


Method ProjectGroupStoreBase::getById does not have a Docblock comment.


Method ProjectStoreBase::getByPrimaryKey does not have a Docblock comment.


Method ProjectStoreBase::getById does not have a Docblock comment.


nice catch... thanks!


Method ProjectStoreBase::getByTitle does not have a Docblock comment.


Method ProjectStoreBase::getByGroupId does not have a Docblock comment.


Klingt nach einer guten Idee mit viel Input. Viel Glück dabei!


Hey, how's the Python3 support going?


这块接下来也记得修改,需要把UserDefault改成对接Model层


fair enough I can get behind this! It means we can make an automated something that runs the graphviz target...


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


1 space after commas, please.


Hi Chris,

Here is the documentation for TeamCity interactions.
https://confluence.jetbrains.com/display/TCD8/Build+Script+Interaction+with+TeamCity

IMO, you should define new functions for “IReporter” for skipped tests as
##teamcity[testIgnored name='testName' message='ignore comment']


This is just an example of how CSV exports could work, if we decide for variant 3 (see #398). Not ready yet, of course. Having this, it should be relatively (!) easy to create CSV exporters for variant 1 and 2. Comments appreciated!


I thought we will go with flat - 2d design. not removing box-shadow results as still 3D. Or am I missing something?


```js
Object.keys(keywords).forEach(keyword => hexes[keywords[keyword]] = keyword);
```


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


Stive! Alan!


1. The script already has GNU compiler check, right below your code. Why not do the version check in
 if("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU") section?

2. What's wrong with g++ 4.7? Just curious. Is it possible that check is necessary only for  Pi, while on Linux older g++ can be used?


Alan!


Reverse relationship --> ``sliderSettings.put("MASTER_VOLUME", AudioEngine.getInstance().getMasterVolumeProperty());``

Otherwise you are making Audio Engine more into a god class, and settings more into a data class.

<organisation


Der List Cast in der accept Methode ist sehr komisch und redundant, aber ohne ihn geht es nicht


![group 2x](https://cloud.githubusercontent.com/assets/185555/10372056/ffddc612-6d9a-11e5-9ace-7275b08d76df.png)


YO SOY UN MONSTRUO.


Uh.  Ooops.  Yes.  Copy and paste fail.


Cool, code looks much cleaner, thanks :smiley:


As far as the implementation, I think it may have been @Ben-G that made it


```js
return 'transparent';
```


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


@wvangeit I'm working on these files today... Can you work in a fork/branch and do a PR when you've changes to make?


I see you already working on AOSP-6.0. Here are some patches for CM-13.0(since for now between CM-13.0 and AOSP-6.0 almost is no difference now, they should also work on AOSP 6.0) https://yadi.sk/d/_JNM8ea2jaVwL .
P.s. I haven't yet tested these patches. All credits for them goes to user SergeyL from 4pda.ru, who is creator of them. Hope that helps.


Thanks for the input!


```js
return shorter(`hsla(${hsla.join()})`, `rgba(${rgba.join()})`);
```


![artboard 2 2x](https://cloud.githubusercontent.com/assets/185555/10372121/4667a0b2-6d9b-11e5-952f-7bc119b1b040.png)


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


@fulv I fully agree. Tthis was just a nice instance of dependency / installation defaults and preferences that you can pull more general with project setups as to make it as easy as possible to start. You can replace trainee with 'contributor trying to get up to speed with a new project'. Something for future discussion after the conf.


So do you mean my commit is not good? If so sorry let me know


shouldn't this have been fixed nicely with commits d69ed51b && 47a93da1 ???


It's just an off-by-one error ;-)


You can do a PR, just use "refs" instead of "fixes" in the commit message.

I usually don't extract general methods until we have a second use case. Probably good to at least add more tests to the rule, though.


klingt praktisch, ich hoffe es kommt zu ner Realisierung:


Could you remove the commented lines?


Will update these comments


*conversion*


Cool. I like the sound of that


I don't feel strongly about this, but a less verbose way of doing this is to remove the try-catch and decorate `FatalError` with `noexcept`.  Then if anything throws (like line 47), `std::terminate()` is called.  I'm not coming up with a super strong argument to prefer one way over the other, so I think it comes down to whichever you consider more readable.


This release needs to get pushed to the spec.

`pod trunk push`, please!


Need these lines?


Guys, let's not make it a bigger problem that it is (not). At any time anyone could recompile PCSX2 and put whatever at the titlebar, or add whatever spyware and call it the ultimate version etc, as with every other open source project. We only made it configurable to users who want to tweak the INI. Reckless people who distribute stupid packages would still be reckless even without our help. And you're only concerned with people who distribute pcsx2 but can't compile it themselves, well.. let's wait for this day to come before we start worrying.


Please use more descriptive names for both of these variables, e.g. constraint_colno and constraint_value, so we're not even tempted to have this bug again.


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


The new default write CL needs to be updated in a couple other places:
* User docs (5_saving.md)
* In WriteableToCassandra, saveToCassandra comments


I would prefer to find a solution to surpress this fault positive if possible.
Anyways, not a big thing :)


if( m_bFullscreen) {

m_bFullscreen is bool.


@alandion Could you explain a bit why this change was necessary? Jin and I are on the cusp of needing to reverse it.


If there is no test file, it is useless to test it. Could you add one?


There is now `(ess-climb-outside-calls)` To climb nested calls, handles chained calls as well.

Then you call `(ess-continuation-bounds)` to get bounds for the current continuation.


I was wondering if we should still inspect the runtime type of the elements to try to find a more precise codec.
But this will work for the default cases, and for more exotic mappings I guess it's not too much to ask users to prefer the variants that take an explicit type.


Klingt toll ich freu mich auf die Entwicklung des Spiels


unsigned - please don't use unsigned type. Use u32 instead.
g_fb_width; - I use prefix g_ for global variables. fbWidth.


Remove the "param feature" line


that makes me think that a bunch of keys are probably missing that one.


This file and others .min/libs shouldn't change


We can rewrite:

```js
if (hex.length !== 4) {
  return hex;
}

let h = hex.substring(1);
let r = h[0];
let g = h[1];
let b = h[2];
return '#' + r + r + g + g + b + b;
```


Small typo here: de**d**ided


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


Is the logic here that if you only have a private ip return that, but if you have a public ip return that?


`conda install` fails for me if I don't supply "menu_name" and "menu_items" explicitly.

See https://github.com/conda/conda/blob/dc90679af0a218715aabd12663468ba2269c3992/conda/install.py#L323


Version 1.0


Please move this platform specific code into separate function:
static
void getDisplaySize()
{
#ifdef VC
...
#endif // VC
}


@vvo On OS X & Windows you can have 2 independent Chrome versions installed, they don't share user profiles. One of them is stable, beta or dev & the other one is Canary. On Linux there's no Canary & in most distributions you can have only one version installed, either stable, beta or dev. Therefore, if you install `google-chrome-unstable` on Ubuntu you can't install `google-chrome-stable` at the same time.

How does it work for you locally? In what distribution is there a `google-chrome-unstable` binary? (In Ubuntu the `google-chrome-unstable` has a binary under a different name IIRC). AFAIK the only way you can have those two versions installed at the same time is to install both Chromium (available in many distros) & Chrome from a different channel. But it still won't be Canary and it won't work anyway if you install Chrome stable & Chromium from the dev channel.

This seems a little confusing to me.


ah ok, all good


Pak mohon maaf kami dari kelompok 12. Saya kesulitan pak untuk mengupdate code dan menggabungkan code teman2 saya. Jadi saya langsung copy code yang sudah dibuat oleh teman saya. Terima Kasih pak.
Berikut Nama anggota kelompok 12 (IF-38-06):
-David Winalda (Ketua kelompok)
-Dian Kusumasari
-Iqbal Bahari
-Rangga


**no-undef**: "it" is not defined.


**no-undef**: "expect" is not defined.


**no-undef**: "ReactDOM" is not defined.


**no-undef**: "before" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "TestUtils" is not defined.


**no-trailing-spaces**: Trailing spaces not allowed.


This is obviously a waste of memory. Why not allocating only what's needed?


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


This overwrites a contribution in #547 which excludes `lib/jx/_jx_marker.js` from jslint.


I should apply for work att Volkswagen...


The only change here was to eliminate the `sdc-listmachines` dependency in favor of plain `docker`.

It does have the effect, generically, that you describe, but in the context of the other configuration details, it will always return an internet-facing IP address.


Overwrites contribution in #547 to excludes `lib/jx/_jx_marker.js` from `make jslintfix`


Fixed: [54d8087](https://github.com/yandex/mastermind/commit/54d8087ed292553f0be6df24af1ba1b4659f741e).


Pretty sure I've mentioned this before, at the end of the day this is optional.

I use this with an app specific password on an encrypted home directory and don't worry too much about it.


Yes, but the rootfs partition which holds the firmware is only 32 MB, and I was hitting 30 MB with the latest test build.  Resizing it to 64 MB like Asus did for the RT-AC68U would require a bootloader upgrade.


@liewegas the only concern is https://github.com/ceph/ceph/blob/master/src/osdc/Objecter.cc#L2899 where we set the reqid using the one from op. and the op's reqid could be designated by the client.


I'm note sure `round.index()` is more clear than `round.roundNumber()`
At least `round.number()` but I feel index is not accurate here.
Round number like in Boxing. Round \#1 , Round \#2 etc


How do task ids get re-used? I guess we don't care about this, but this seems a bit odd.


Go team!


Wow, klingt sehr interessant, viel Glück dabei :)


@magsout that's only used for clicking on a label to do a "label search" on the issue-list.js page. We can include it for other views and just leave it blank (to be consistent). What do you think? Or I could think of some other way to implement the mapping of namespaced to non-namespaced labels.

I'm sort of tempted to just show the namespaces (status-, browser-) everywhere. I'm not really sure what we gain by removing it.


byteRangeSupported is always false here. (not used after initialisation)


I would guess this should somehow interact with the cmsis_task_counter variable, or use a different task id allocation.


Yeah, Audiosurf lässt grüßen, klasse Sache, viel Spaß und viel Erfolg :)


I think you should change this into List<List<Change<Position>>>, seeing as it doesn't really matter which list is being used, as long as it is a list. Same goes for the argument of setList.


if
else
else if
else if
else
end
Not sure this is what you wanted to do.


I'm making changes by adding a new file


Da bin ich auch echt auf die Umsetzung gespannt! Viel Glück :)


adding jgtyler


Hört sich nach einer guten Idee an. Viel Erfolg!


I guess I could use one of the other buffers like idata and cast it but
it makes it clearer
to have a float buffer. It's only done at construction time so I don't
think its a big deal.

On 2015-10-08 7:15 PM, Jakob Leben wrote:
>
> This is obviously a waste of memory. Why not allocating only what's
> needed?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marsyas/marsyas/commit/42aaf3cbc1a6d7a132d8aeb7a4a4683a472b6d50#commitcomment-13667140>.
>


How can `byteRangeSupported == false && rangeOk` happen?


Avoid variables with short names like $id. Configured minimum length is 3.


The method executePlugins() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10.


PHPCS: The closing brace for the class must go on the next line after the body


PHPCS: Line exceeds 120 characters; contains 125 characters


Method BuildStoreBase::getByPrimaryKey does not have a Docblock comment.


Method BuildStoreBase::getById does not have a Docblock comment.


Method GroupController::init does not have a Docblock comment.


Method BuildStoreBase::getByStatus does not have a Docblock comment.


Method BuildStoreBase::getByProjectId does not have a Docblock comment.


Method GroupController::index does not have a Docblock comment.


Method UserStoreBase::getByPrimaryKey does not have a Docblock comment.


Method GroupController::edit does not have a Docblock comment.


Method UserStoreBase::getById does not have a Docblock comment.


Method GroupController::delete does not have a Docblock comment.


Method UserStoreBase::getByEmail does not have a Docblock comment.


Method HomeController::getGroupInfo does not have a Docblock comment.


Method UserStoreBase::getByName does not have a Docblock comment.


Method ProjectGroupStoreBase::getByPrimaryKey does not have a Docblock comment.


Method BuildMetaStoreBase::getByPrimaryKey does not have a Docblock comment.


Method Executor::doExecutePlugins does not have a Docblock comment.


Method BuildMetaStoreBase::getById does not have a Docblock comment.


Method ProjectGroupStoreBase::getById does not have a Docblock comment.


Method BuildMetaStoreBase::getByProjectId does not have a Docblock comment.


Method BuildStoreBase::getByPrimaryKey does not have a Docblock comment.


Method BuildStoreBase::getById does not have a Docblock comment.


Method BuildMetaStoreBase::getByBuildId does not have a Docblock comment.


Method BuildStoreBase::getByProjectId does not have a Docblock comment.


Which resource is updated? the image?


Method BuildStoreBase::getByStatus does not have a Docblock comment.


Method ProjectStoreBase::getByPrimaryKey does not have a Docblock comment.


Method ProjectStoreBase::getById does not have a Docblock comment.


Method ProjectStoreBase::getByTitle does not have a Docblock comment.


Method ProjectStoreBase::getByGroupId does not have a Docblock comment.


Method UserStoreBase::getByPrimaryKey does not have a Docblock comment.


Method UserStoreBase::getById does not have a Docblock comment.


Method UserStoreBase::getByEmail does not have a Docblock comment.


Na dann Man Tau, viel Glück :)


This name sort of confuses me. This is static, but what does it do functionally?


I'm going to change this later today, right now this test relies on the servers explain format which is a moving target and is expected to change. I'm going to fix this in the python driver first then port it into ruby.


For the lolz


you can do : @best_value = @brother_best_value < @heuristic_value ? @brother_best_value : @heuristic value


Ditto


Yay!


Please @Mazda-- report this is pure CAF kang !!! You're da real dev ! https://www.codeaurora.org/cgit/quic/la/kernel/msm-3.10/commit/?h=6ae8bb817998ff95d02eaad35a061bdd9ebc5e4e


Do we need to `with`s?


uuuuh, da habt ihr euch ja was vorgenommen :) Viel Glück dabei, bin gespannt auf das Ergebnis!


You have an else that does exactly the same thing than your if


Hello mtzuit,
Firstly, thanks a lot for project. I found a problem with created column names.
When call columnname with ToUpper () then table created columns i to İ like "EMAİL" correct you know like "EMAIL". I think may be use culture or not use Upper function.
Thanks again, Kind regards.
modelBuilder.Properties().Configure(c => c.HasColumnName(c.ClrPropertyInfo.Name.ToUpper()));


**FAILURE**: Build failed with an exception.
34 errors
9 warnings

Total time: 35.63 secs
Build step 'Invoke Gradle script' changed build result to FAILURE

Please review your changes and ensure "gradle build" runs before pushing a fix.

See the [Jenkins console log](https://jenkins.jakimfett.com/job/NotEnoughResources/112/console) for details.


@sparanoid Very cool. Thank you for the acknowledgements.

In the README.md, do you want all questions to go to you, or should @[jjloomis] appear in line 18?


@lauraluiz I like the "now for real" part of the commit message


Auf die Umsetzung bin ich auch sehr gespannt :) Viel Glück ;)


kann mich nur dem bereits gesagten anschließen. Viel Erfolg bei der Umsetzung.


**no-undef**: "ReactDOM" is not defined.


**no-undef**: "it" is not defined.


**no-undef**: "expect" is not defined.


**no-undef**: "TestUtils" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "before" is not defined.


**no-trailing-spaces**: Trailing spaces not allowed.


Sehr interessante und kreative Idee, bin gespannt auf die Umsetzung :)


MongoDB guys are developing a new C++ driver API, it will use modern C++ features and will be completely different. Have a look at a simple example: https://github.com/mongodb/mongo-cxx-driver/wiki/Quickstart-Guide-%28New-Driver%29
I'm not going to pass auto pointer to cursor anywhere, let's leave it as is until a stable version of the new API is released.


Can we default to true? From experience with iOS, we ask for notification permission and people appreciate being reminded once a day. Or I suppose on the initial start screens (when they first open the app) we could let them tap a button to enable notifications.


Hört sich nach einer Menge arbeit an, aber wird bestimmt super, wenn es fertig ist. Viel Erfolg!


Any program and script that runs as your user would have access to a google password that bypasses 2-step, i think that's reason to be a little bit worried. Wouldn't be too much of a hassle to make it run as "PlayCLUser" or something.


Should the unimplemented things return osOK, or perhaps something more like osNotImplemented?


We can improve this by returning a cached version of the shorthand hex instead of converting it to longhand and back.


I guess it's fine to just include the `data-labels-map` for all view instances, they won't be used in some places but it doesn't cost us anything.


add properties sorted by name, to allow easy reading


I'm not sure that you can't spec them - all the functions return a certain type, just like the attributes, and they take parameters that we can specify, and we can add a default value that can be supported by certain languages. The implementation of the helper is usually a sequence of operations on the existing attributes. It should be specifiable - in theory.
In practice, we may use a language specific module to implement the helpers - I'm not saying we can't put helper functions into the `ev3dev.py` file, just that we should think about alternative implementations as well.


`unlocked=false`


Thanks.. I changed it. I'll push it in the next big update.


Executing the build.sh will happen outside of the docker script, so just

```
- ~/SlicerExecutionModel/test/Docker/build.sh
```