my bad. i didn't know about that. i'll update my report when i switch to linux. thanks.


**SUCCESS** - TeamCity Quickstarts :: Pull requests monitor [Build 125](http://ci.infinispan.org/viewLog.html?buildId=32294&buildTypeId=quickstarts_PullRequestsMonitor&guest=1) for 57ee7c5d71339149129b137ed6e60a28b918f440
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:07


Huh, how would (does) that work? i.e. how do you tell the difference between, given cases A, B and C "I only want to handle cases A and B" and "I forgot to handle case C"? Is there a special method or syntax that you have to use? I'm confused if you haven't guessed ;)


Yes, it crashed, trying to access an unavailable memory address. However, I did change the example to not require command line arguments and enter directly the else clause, so perhaps it only crashes in that case?


@nickandrew, can you do me a favour and roll up this and the previous two changes (de692f3, 53eee16) into a PR for `dev`.  Thanks and much appreciated.  Terry


Oups ! Je vais de suite modifier cela...voilà, c'est fait.

De: "Raphael-R-R" <notifications@github.com>
À: "Florent8/Projet-S1" <Projet-S1@noreply.github.com>
Cc: "Florent8" <florent.martin02@edu.univ-fcomte.fr>
Envoyé: Lundi 23 Novembre 2015 17:39:24
Objet: Re: [Projet-S1] Create createTicket.php (1708247)

Autant, y'en a qui sont doués qui sont doués pour le français, autant du côté de l'anglais c'est dégeulasse. "Qwery" ça n'existe pas, la fonction est définie dans ce fichier https://github.com/Florent8/Projet-S1/blob/master/functions.php

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


When you create a share and assign a quota at the create time, on mounting the share, you can see the quota tab on the windows client
However, at the create time, you do not specify a quota but later assign a quota value, on subsequent mounts, the quota tab still does not show up.
I see the issue in my setup even after importing the bug fix. Is there any other fix that I have to import along with this one?

![screen shot 2015-11-23 at 11 34 35 am](https://cloud.githubusercontent.com/assets/1840041/11347950/8fa24d7a-91d8-11e5-9e72-38a29440386a.png)


:+1: Sorry for the trouble, was just curious as I was borrowing parts for Terraform's Vagrant file :smile:


please collaborate upstream


Как я понял, здесь мы указываем, что настройки для этой части приложения находятся в классе ContentsGeneratorConfig, и они загружаются только для класса ContentsGenerator.


Typo introduced.


Why remove the `private` modifier?


$poretools
Traceback (most recent call last):
  File "/usr/local/bin/poretools", line 9, in <module>
    load_entry_point('poretools==0.5.1', 'console_scripts', 'poretools')()
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources.py", line 357, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources.py", line 2394, in load_entry_point
    return ep.load()
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources.py", line 2108, in load
    entry = __import__(self.module_name, globals(),globals(), ['__name__'])
  File "/Library/Python/2.7/site-packages/poretools-0.5.1-py2.7.egg/poretools/__init__.py", line 4, in <module>
    from Fast5File import *
  File "/Library/Python/2.7/site-packages/poretools-0.5.1-py2.7.egg/poretools/Fast5File.py", line 291
    Return the base called sequence in the FAST5
             ^
SyntaxError: invalid syntax


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


Same here and below regarding `private`.


shall we add TU DA in the header -- lots of LT groups exist :-)


The structure of the tests mirrors the structure of the code (`r2/tests/unit/config` tests things in `r2/config`, etc.).

> Tests the functions related to the diferent features of each user account. Example: test admin features, beta testers features, gold accounts, etc...)

Specifically, you're looking at tests of [the feature-flagging system](https://github.com/reddit/reddit/tree/master/r2/r2/config/feature), which allows toggling sections of code according to a variety of attributes, include those you mentioned.  There's some discussion of this kind of system over at the [Flickr](https://code.flickr.net/2009/12/02/flipping-out/) and [Etsy](https://codeascraft.com/2011/02/04/how-does-etsy-manage-development-and-operations/) engineering blogs.

> This package tests reddit's AI, like the sorting of posts after being upvoted or comment sorting after a responded by the OP.
>
> It also tests the update of the database after user interactions such as comments and upvotes.

The model subset of tests are for testing the various models reddit uses - that is, the object representations of data from the database, and the methods contained within them (see [ORM](https://en.wikipedia.org/wiki/Object-relational_mapping) for a description of the more general system).  As such, this is one of the hardest places to test, as any interaction with the database must be mocked out, and therefore, one of the least tested parts of the codebase.


FYI: this has broken the docs in Chrome 46 / Opera 33.  Page is no longer scrollable.  Setting height:100% fixes the docs page scrolling.


Penis


I think you meant c**a**nonical.


typo: "daemosn"


Maybe we can just inject this logger through config? In this case we can skip these two methods (we still need to default it to `ioutil.Discard` though)


The name grantTablePrivilege is consistent with names used in the existing code though. If you look in ```HiveMetastore.java```, there are methods named as ```getTablePrivileges()``` and ```getDatabasePrivileges()```. So in lieu of consistency, I would prefer to go with grantTablePrivilege (or rather grantTablePrivileges).


Since you're doing the same thing regardless of `data`'s value, you can reduce this all to `self.cachedCursor = result.successValue`.


Without these subtracts that shift the triangle outside the lambda block, it doesn't work reliably.
I have no clue how to resolve this.


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


Hm not even the typical cannon vs canon issue. Thanks.


Technically, if there were no observations at all with length one, this site would be dropped because "reference allele is an insertion or complex variant" -- which would not be true.

This logic also allows for indel detection with ref.size == 1. Is that okay?

Both of those facts suggests to me that the intent was
`pointMutatior = ref.size ==  && !experimentalMutectIndelDetector`


@mikemcquaid No worries, take your time! Happy to hear that my response made it into your inbox.


It would be good to remove this commented out code since it'll be no longer used if this PR is merged.


forgot the "


This seems to make it impossible to remove resources at urls generated by the `jsonapi_resource` (singular) route helper, which don't have an `id` parameter. For instance if you wanted a client to be able to log out by sending a `DELETE` request to `/session`--does that violate the JSONAPI spec in some way that I don't understand?


For local consistency there should be a space between the `//` and the actual comments on these lines.


check again, I have updated my comment...


Is there a reason you rolled your own instead of creating a subclass from the MZmine parameters class?


While here:
`now` -> `now that`
`udpated` -> `updated`


No this here


No :) We usually say `description` when are talking about a long text. I think `label` is better, but `getAttributeLabel` is already taken.


> what's a cookie?

That would be [an HTTP cookie](https://en.wikipedia.org/wiki/HTTP_cookie).


```
# return the sequence to be consumed/parallelized
return [1,2,3,4]
```


Oh god, sorry. @arq5x we must build some tests for poretools. Do you have a preferred harness? Happy to write some tests for it.


I think i should have mentioned this on the ticket, i think it would be nice to add all these from the same place, see https://github.com/openmrs/openmrs-module-legacyui/blob/master/omod/src/main/java/org/openmrs/web/WebComponentRegistrar.java
What do you think?


What's the point of having a default `(val length: Int = 1)` in Observation if we're just going to call `.allele.size`?
That's dangerous. It's best to implement this as a method so we don't forget to override the default length.


Yeah, the message is correct...

So do you plan to update the message when the correct ruleset is released or do you just want to leave it as it is?


@paraboom доработал код, чтобы подсвечивался активный фильтр.


These are mutually exclusive. I believe the `||` should be a `^`


@github needs to be a good community steward and clarify it's opinion/direction on this issue.  This discussion has been going on for ~2 years which is far too long.


@zidarsk8 you keep forgetting putting `;` at the end of the line ;-)


OK - will ts allow you to do this?  (and will it achieve the same?)

function debug(message) {
    exports.debug(message);
}
exports.debug = function(message) {
   // do nothing
}


ResortedSurfaces (amirroth) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (897 of 897 tests passed)

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


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


Arghs, sry


:open_mouth:, whoops, I must of pasted that in the code when switching through apps


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


hi
I found this plug-in and use it at Logbook.
Is working right now and I can see the MouseXY  in the log.
But have one major problem.
The plug-in doesn't show MouseXY at the "Expedition".
That only show MouseXY at "sortie" and "Akashi's Improvement Arsenal".
Can you check this problem?
My logbook version is 2.0.3 logbook extend.
and Thank you for your read.


For my benefit and understanding: What's the difference between tumorsRaw and tumors?
Specifically,  why can tumorsRaw be used below in passStringentFilters and passMapq0Filter?


Did that remove the deprecation warning in your case? I can't seem to reach this `if` since `this.get('application.registry')` exists.


i cannot really understand :: operator, can you explain what this line does?


the thing is, when i looked at the tests, they had names like "upgrade cookie" and similar stuff. i know what a cookie is, i just didn't know if it was indeed an http cookie.


also important to know that switch unfortunately compare with `==` and not `===`.
who defines this silly behaviour...


I also get a "Segmentation fault: 11" with an OSX python problem report pop-up window stating: "Python quit unexpectedly while using the libR.dylib plug-in". Not sure if this is my system or something else.


I would print this to stderr if possible


As a nonbiologist reading this magic number: What does this mean? Why is this significant? Can we name this boolean?


also getting real lazy


@dclorena11 Looks great!


Жаль, что в хроме не работает `unsafeWindow`. Из-за этого простая проверка `if(unsafeWindow.boardRequiresCaptcha) { ... }` превращается в такой-то кусок асинхронного кода.


I looked up clip to remind myself what it did and found this: MDN says "This property is deprecated. Use clip-path instead....This feature has been removed from the Web standards. Though some browsers may still support it, it is in the process of being dropped. Do not use it in old or new projects. Pages or Web apps using it may break at any time."


There are big fat COMPILE file there :)


Otherwise, :+1:


TeamCity LabKey :: Trunk :: LabkeyTrunk :: Git sqlserver 2014 [Build 67](https://teamcity.labkey.org/viewLog.html?buildId=286911&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:54


Thanks for the patch. `successors` is undefined for undirected graphs, so I've tweaked this to use `successors` for directed graphs and `neighbors` for undirected graphs. This is a backwards incompatible change, so I'll bump the major version on the next release.


@fabricioflores podemos restringir el diff de este archivo a lo que agregamos / quitamos? podemos hacer otro commit para ordenarlos


si... al final cuando termine de moverlos lo dejo como estaba, y q se vea solo lo q agregué...


CAS needs rCharts for drawing interactive bar plots.


ということは、このあと書くんでしょうか？私には書ける自信がないので。


Что имеется ввиду? Это избранное для топиков, на избранное других объектов никак не влияет.


How to create TRP file?


fixes #13 and #14


would you mind adding TORRENT_OVERRIDE in new code and as you update old? and also TORRENT_FINAL if it applies


that format control, whee!


I wonder what would be faster on heroku.. The ruby version with the ignore of the vendor directory or the c_ext including the vendor directory.


@yumetodo ええ、書きますよ、もちろん。


do this instead:
``````
   $sqlBindArray = array();
   $where = "";
   foreach ($_POST['form_cb'] as $key => $value) {
       $where .= " OR f.id = ?";
       array_push($sqlBindArray, $key);
    }
    $where = substr($where, 4);
````


I don't understand the error. Is there an incorrect down-cast, where we assume an observer to be of a different type than it is?

If so, I imagine fundamentally the design should be reconsidered to improve type safety (would you mind adding an appropriate TODO comment where the crash happened? Would that make sense?)


I take it traversal_algorithm::start() does something we don't want to happen in this case?


do this instead:
``````
       "WHERE ( $where ) AND " .
       "p.pid = f.pid " .
       "ORDER BY p.lname, p.fname, f.pid, f.date, f.encounter", $sqlBindArray);
``````


Seems like this should be inside the Core class, since it's (presumably) only used from there. Just in case someone else decides to make a Message class in the global scope ...


Personally I think this is advantage, because otherwise you need to waste CPU for initialization of the entire array each time you call the function that uses it.


В него вносятся изменения, без которых вся конструкция не работает.


using C++ style casts instead of C style casts (should be faster at runtime)


@bostko , I am personally not very fond of long ```throws``` declarations. Wouldn't the same result be achieved by catch Exception, check if something like isAssignableFrom against the Exception types you care about and otherwise throw again? Without a list of exceptions that in my experience is a nightmare to maintain?


Minor recommendation


Why did even bother writing this, i should just use local-time:timestamp instead of universal-time in seconds.


ya cachaste como funciona, excelente. Ya descargué el escritorio.


This also isn't valid according to spec.  `data` has to contain entries with ID and t and x and y.  `@OMG` isn't one of them.  It has to be at the top level (outside `data`).


Ahh then sorry about that.  Go ahead that's fine knock yourself out.


Typo here


Can we add a Notes field here?


Can you change this to a TEXT field?


The simpletokenizer.java and USEnglishTokenizer.java are causing issue with character literals while building using gradle. Please help!


This commit breaks the -L option. It now creates only a directory called b' in the directory where the script is ran.

Tested on ArchLinux.


I believe there is an issue here. you have removed the function updateParameter, but the new line 125 trys to call it. When I attempted to use this module I got an error telling me updateParameter could not be found on line 125. This was when I was trying to go through the authentication routine by hitting debug.


Ta uma bosta essa parte


应对岛风Go代理 api返回头部没有Date信息, 修改记录方式,增加判断


Thanks. I don't think having 0.2% less range is much of an issue if it's 99% faster. ;)


Probably a good idea to rename this, since is_a? is a commonly used built-in method


Quick suggestion


show back button in search page


Looks good!


Yes, it adds router nodes, which doesn't make sense for a put operation and causes a crash.


it's encrypted with a private key that's not checked in, but thanks for the heads up :)


Setup Sass Automatically :
ionic setup sass


さすがやで・・・。


EPTU.


Can you comment information on your variables inline?


Just in case I made a caching.
If the bbcode2email function () where it won't be used more, the caching can be removed.
https://github.com/MioVisman/FluxBB_by_Visman/commit/ff7d01bf6a2be8c96debb1044088c24ce719a49e#diff-2f5fd9e98a178d566a749eb385264f12L270 - this line there is a check for a cache.

$wtotes - yes, I pressed not that key.


How about ``services = nLocalServices & ~NODE_GETUTXO``?


Not sure I follow...explain more? :thought_balloon:


Custom movesets were lost here, fixed in https://github.com/Zarel/Pokemon-Showdown/commit/0d37b7946d38ae81c3e0c47020ad2ab0b18d90ac.


Good call.


You know, you could use a static code analysis tool to enforce this ;). Maybe Checkstyle? It can search for arbitrary regexen.


It's me or your test is seeing a much higher draw rate than other 7-thread tests at the same TC?


@yehudasa
I put a new commit,please check it, thanks


Pushed in 1d3a775302afdf33df9dea85457625aa9860de71 and 9daed5656be2ba06a00b036b1d021eabb36f8d58.


Yes, the problem is `put_data::invoke` assumes it will only be called with `o` being an instance of `put_data_observer`. But in the case that `traversal_algorithm::start` has added router nodes, `o` will actually be an instance of `null_observer` per `traversal_algorithm::new_observer`.

I agree this code needs a redesign. I'm really not happy with the state of things after 194ad410dccf477547e526ac918e9973fbee6831.


Added code to make div suppressible using checkboxes.  Note that hiding a div will also clear the entry.

**At Startup with both CC and Schedule suppressed.**
![Startup](https://c4265878.ssl.cf2.rackcdn.com/gsbcfamily.1511222045.Email_Form_CCHidden.png)

**Toggle CC checkbox to make CC div appear**
![CC Shown](https://c4265878.ssl.cf2.rackcdn.com/gsbcfamily.1511222045.Email_Form_CC_Shown.png)

**Toggle Schedule checkbox to make Schedule div appear**
![Schedule Shown](https://c4265878.ssl.cf2.rackcdn.com/gsbcfamily.1511222046.Email_Sched_Shown.png)

**Both CC and Schedule shown**
![Both Shown](https://c4265878.ssl.cf2.rackcdn.com/gsbcfamily.1511222045.Email_Form_CC_Sched_Shown.png)


Somehow, make command ask for clang compile after merging this.
Doesn't it need to use clang only if I issue ````make clang```` command?


"umount -l ..." can umount it in most situations, if "umount" command is not successful, the code checks the return status,  I am fine for the fix;
Thanks;


OK I see, 8-move book, sorry...


reloadErrCh <- do.reload()


MU-2014-AMC: 170.314(g)(1)/(2)–10:
(note record_vitals_1_stage1_amc(AMC_302f_1_STG1) and record_vitals_stage2_amc(AMC_302f_STG2) are not even being used)


got it.


I'm trying to spur more interest in accessibility within my group, and this is an awesome resource for learning more so thanks! I'm interested in helping with the repo if you're open to it.

You're probably already aware of it, but this change broke the Travis build because the `guides-style` gem can't be found in this location on the build server. Would you mind if I submitted a pull request to fix that? If not, are there other areas I can help with?

Thanks again!


Testing Integration


Also return 1 if incorrect input (aka an error)


Yes, we can. It will be in a future refactoring


damn!!!!!! actually I had this change locally and forgot to commit it :-)


L195 below: re: `// is this a known site?`
Known for what, exactly?


Looks good, could you add one more line break between raise Exception and details, so that the if statement is more prominent?


This should probably say `cmake -DPYTHON_DESIRED=3 .`


brace should be on the next line by itself


Very nice!


can you provide links to these files in some external repository?


Ah okay. But in this case if the key exists and has value `None` it gets skipped -- I thought we wanted it to be included, since it should be null?

In any case I think your first snippet is clearer and more explicit.


I use this feature.


I also think it's `application.resolveRegistration` (not `container.resolveRegistration`).


James,
Thank you for your contributions to this repository!
Sincerely,
Alex


Nice catch!


Are these attributes really stored on the connection and not the connected nodes?


not sure what do you mean. if the date is present, just use it. if not use the publication data of the WikiLeaks collection


Are you referring to [total](http://hackage.haskell.org/package/total)? Or something else? Something like this wouldn't work for `EnvironmentError` though as it isn't parametric, right?


@pglombardo github DGAF. They used to, but now that they have the users they can do what they want. We might still be discussing it, but it's pretty clear its a done deal for them and they have shown no interest in taking up the issue any further. It sucks, but thats life.


hmm that should go in plugins. Stuff in lib/ (and not in lib/plugins) shouldn't contain network stuff (packets)


@hlinnaka I found CR/JIRA's that are pertinent to the need for these functions:

https://jira.eng.pivotal.io/browse/MPP-23435
https://fisheye.eng.pivotal.io/cru/CR-643
https://fisheye.eng.pivotal.io/cru/CR-630
https://fisheye.eng.pivotal.io/cru/CR-603


@akalicki the sast printer is now working. you might want to change how Sast.exprs are output, feel free.

Right now, they are being output in the form: `type id`


this file needs a proper comment block at the top.  something like:

~~~
/* Provide a snprintf on Windows for older Visual Studio builds.
 * VS2013 and older do not correctly NUL-terminate buffers in some cases.
 */
~~~

you can then delete the comment above the snprintf func below


thanks alot


the commit message has a typo "snpritnf"


I think you might want to dispatch this callback on the main thread. You're using `self.layout.thumbnailCount`, which uses `thumbnailRows`, which uses `self.collectionView?.frame.height` (not thread-safe).

Might also be a good idea to `assert` in some of these dynamic getters (`thumbnailRows`/`thumbnailCols`/`thumbnailCount`/etc.) since that's an easy mistake to make.


Also this is really unrelated to this PR (inventory), put that in a different branch ;)


the .js result i got from my ts code is:
function debugHolder(message) {
}
exports.debug = debugHolder;

not sure whether i should code in js directly.


This is backwards, should be "if you use a non-standard port, add the port after the IP address, separated by a colon.  Ex. 192.168.2.1:5400".  Don't have this computer set up to edit though...


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


Nit: remove, or use `XCLogger` instead.


Add a test in which an error throwing is tested:
```lua
assert.has_error(function()
  -- something that must throw Lua error
end)
```
A test like this fails if the tested code throws no error.


still spurious whitespace changes


I had no idea there is something like this and I doubt someone will see it as it doesn't appear in diffs. Maybe move this message to the beginning so we won't have to change it every time? Maybe we can even just remove it?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.827](http://teamcity.monogame.net/viewLog.html?buildId=11673&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 552, ignored: 6 Build time: 00:20:34


Proszę nie robić chałtury - dodawać zalezności do mavena!


Awesome! Thanks for the creds :+1:


So, you want me to use `this`? I haven't actually tested what it looks like at runtime, but I can't imagine it is going to be useful considering the object naming is sometimes "interesting" to say the least.


'url' or 'uri'? I know you made the name compatible with function arg, but the phrase 'base uri' makes things confusing.


Did this work? I reckon this might lead to errors when test takes longer than 2 minutes


Werd eens tijd. En please doe niet dat @cunothe1. Ik shockte gem omdat het leek alsof het iemand anders was genaamd b9d9aae


Some stuff is behind master branch, I can merge it later


LOL


Najpierw mavena, trzeba ogarnąć, jak ogarnę, to dodam.


Here are some discrepancies I can currently observe between running Maple directly (typing `read("./NewSLO.mpl");` on its command line) and running Maple through `System.MapleSSH` and the updated archive. The former gives correct behavior:
```
> NewSLO:-TestHakaru(Gaussian(0,1));
Test passed
> NewSLO:-app(lam(x,lam(x,x)),3);
lam(x, x)
```
The latter gives incorrect behavior:
```
*System.MapleSSH> maple "NewSLO:-TestHakaru(Gaussian(0,1));"
"Test failed\n Expected result : Gaussian(0,1)\n Evaluated result: Gaussian(0,1)\nError, (in CodeTools:-Test) TEST FAILED\n"
*System.MapleSSH> maple "NewSLO:-app(lam(x,lam(x,x)),3);"
"lam(3,3)\n"
```

Granted these discrepancies may be buried by whoever calls `System.MapleSSH`. Still, I hope y'all'd agree with me that they are problems that should be fixed rather than buried. And Jacques seems to have described how to fix them.


Thanks, @abbyraskin!


Daruj, ja zrobię!


The fact that an account isn't limited to one mobile phone per account isn't documented. Just try it.

The Facebook mobile app auth is called "Code Generator" there and is documented at:
https://www.facebook.com/help/270942386330392

OATH setup using a different app, e.g. Google Authenticator, is documented here:
https://www.facebook.com/help/358336074294704

Unfortunately these are not all documented in the same place.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 41](https://teamcity.labkey.org/viewLog.html?buildId=287036&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 13 (5 new), passed: 271, muted: 4 Build time: 01:33:16
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

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

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

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.util.JUnitFooter.afterJUnit: <no details avaliable>

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

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

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

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

##### there are 3 more failed tests, see build details
```


but wich type should be used


Będzie w kilka minut.


@jstjohn Does the lack of a SNP table really imply that this is a novel somatic mutation?


Very nice.  Thanks for making this change.  I'm fine with you merging this when you're ready.


This was changed recently (21e25faa05e2ed7483038cf339d386b4ffe00aa7). Filenames in TOCs are no longer normalized.


@schellap sounds good to me. Please share out the PR for CLI change as well - please coordinate both committs.


@eviltrout It would fail massively for inline replies like with https://discourse.chef.io/t/the-migration-from-lists-opscode-com-to-discourse-chef-io/7309/7. Imagine that with the quotes removed and trying to reconstruct the context. Seems better to deal with the reply headers than lose all that. For bonus points you could try to parse the reply chunks an make quotes out of them.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 42](https://teamcity.labkey.org/viewLog.html?buildId=287078&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:24


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 42](https://teamcity.labkey.org/viewLog.html?buildId=287078&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:25


@jstjohn Is this appropriate behavior? Act as if it's not noisy if there's no data to support or deny that? That strikes me as dangerous...


I was going to see if I can write a unit test for this later so exposing it into the internal space would let me use @testable and test it directly.


Same note as above.


I updated to 0.10.0 today, everything worked fine :-)
Thanks a lot for merging and releasing this so fast, that was really awesome :-)


Val-ify the tesla tree? O_o lol Yayo u're high again :p


NetBeans (nightly build). Po wpisaniu "/**" nad metodą i wciśnięciu
Enter sam generuje @param i @return w komentarzach.

W dniu 23.11.2015 o 20:03, nicraMarcin pisze:
>
> :)
> tak na marginesie, jakiego IDE używasz, że tak ładnie robi Ci
> komentarze do funkcji i metod?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lmsgit/lms/commit/190511aabaca1743fe184eebcce4ddc641f56d78#commitcomment-14569478>.
>

--
lion.net.pl - wdrożenia i rozwój Lan Management System


Good catch - I'll definitely add those asserts just in case.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 108](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=513&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


Thank you for believing in the usefulness of this info (in an otherwise empty column) & committing this. :+1:

I see the 3 lines from commit 659428f are added (for unknown reasons) at the end of this SQL:
```sql
UPDATE `creature_template` SET `skinloot` = 0 WHERE `entry` IN (721,883,890,1933,2098,2442,2620,4166,5951,10780,12296,12297,12298,12299,17467);
-- Remove unreferenced skinning_loot_template entries because of this update:
DELETE FROM `skinning_loot_template` WHERE `Entry` IN (1933,12299,100000);
```
Is it possible to edit and remove them, or maybe at least add a commend line ( -- ) before the last 3 lines?


Just for testing, will remove later


Seems legit.


That was me incorrectly resolving a merge, not you, @staabm. :P


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 108](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=513&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:01:06


@alanjds I can not remind what was exactly the issue. As far as I'm concerned I wanted to get rid of prefixes in lookup names. The bottom line probably was that then if you are not careful system could generate two same looking lookups. In my case lookups was way to long(at worst case view was nested 5 times) so I wanted to shorten lookups. I probably did not make ticket because it's more feature than bugfix and highly depends on ones needs. Although I'm not quite sure if with original code anyway I was able to make nested lookup more than once.

Sorry but right now I have quite much work to do. It could take a while before I get my hands on that problem.


Makes sense. It brings up something I didn't know before how to ask: how much of this do we want to expose from Java?

I think my understanding was that from Java we want user to only access `List<Thing> getThings()` and `setThings(List<Thing>)`, and we only want to give them the DSL magic (like `thing(Thing)` and `things(Closure)`), well, from the DSL.

Do we want these to be somehow accessible from Java as well (with `things(Action)` instead of `things(Closure)` of course)?


Why not having the same design as Users, Printers, languages plugins? With a headphone icon for output and mic for input..


Yes I agree. Let's not bury errors. How would we catch this kind of error
automatically next time?
On Nov 23, 2015 3:11 PM, "Chung-chieh Shan" <notifications@github.com>
wrote:

> Here are some discrepancies I can currently observe between running Maple
> directly (typing read("./NewSLO.mpl"); on its command line) and running
> Maple through System.MapleSSH and the updated archive. The former gives
> correct behavior:
>
> > NewSLO:-TestHakaru(Gaussian(0,1));
> Test passed
> > NewSLO:-app(lam(x,lam(x,x)),3);
> lam(x, x)
>
> The latter gives incorrect behavior:
>
> *System.MapleSSH> maple "NewSLO:-TestHakaru(Gaussian(0,1));"
> "Test failed\n Expected result : Gaussian(0,1)\n Evaluated result: Gaussian(0,1)\nError, (in CodeTools:-Test) TEST FAILED\n"
> *System.MapleSSH> maple "NewSLO:-app(lam(x,lam(x,x)),3);"
> "lam(3,3)\n"
>
> Granted these discrepancies may be buried by whoever calls System.MapleSSH.
> Still, I hope y'all'd agree with me that they are problems that should be
> fixed rather than buried. And Jacques seems to have described how to fix
> them.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hakaru-dev/hakaru/commit/cc9623d89aaf751db8b0ab1410bb8c31a067d813#commitcomment-14571067>
> .
>


Ku przestrodze - używajcie UTF-8!!!


I see that you've replaced these files with the package.json file. What is the best method for installation? `npm install` at root installs the needed files, but it also installs 209 files of content I'm not sure we need. Is this the expected outcome?


Yes, all the network-related sandbox profiles should be removed. This is vestigial code I forgot about.


Well, spit. So what is the distinction between an EXTENSION and a BINARY, then?


Not sure I understand what you mean by that.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 49](https://teamcity.labkey.org/viewLog.html?buildId=287056&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 19 (2 new), passed: 8 Build time: 00:51:31
### Failed tests
```
org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 9 more failed tests, see build details
```


remove?


this won't work for arbitrary arithmetic types T, because quiet_NaN is only defined for float, double, and long double =( Hmmm


Yeah, we should be using the `{{ site.description }}` object here instead of hard coding.


P {margin-top:0;margin-bottom:0;}

Tests, tests, tests!

We need a test suite for all our pieces, and it needs to be part of development process that it be run before any new code is checked in.  If tests fail, then the things are not ready to be pushed.

[The main exception to this is when explicitly making "breaking changes", that ought to be known to everyone in advance]

From: zaxtax [notifications@github.com]
Sent: November 23, 2015 15:18
To: hakaru-dev/hakaru
Cc: Carette, Jacques
Subject: Re: [hakaru] Adding RoundTripLO to wrap the new steps in NewSLO (cc9623d)

Yes I agree. Let's not bury errors. How would we catch this kind of error
automatically next time?

On Nov 23, 2015 3:11 PM, "Chung-chieh Shan" <notifications@github.com>
wrote:

> Here are some discrepancies I can currently observe between running Maple
> directly (typing read("./NewSLO.mpl"); on its command line) and running
> Maple through System.MapleSSH and the updated archive. The former gives
> correct behavior:
>
> > NewSLO:-TestHakaru(Gaussian(0,1));
> Test passed
> > NewSLO:-app(lam(x,lam(x,x)),3);
> lam(x, x)
>
> The latter gives incorrect behavior:
>
> *System.MapleSSH> maple "NewSLO:-TestHakaru(Gaussian(0,1));"
> "Test failed\n Expected result : Gaussian(0,1)\n Evaluated result: Gaussian(0,1)\nError, (in CodeTools:-Test) TEST FAILED\n"
> *System.MapleSSH> maple "NewSLO:-app(lam(x,lam(x,x)),3);"
> "lam(3,3)\n"
>
> Granted these discrepancies may be buried by whoever calls System.MapleSSH.
> Still, I hope y'all'd agree with me that they are problems that should be
> fixed rather than buried. And Jacques seems to have described how to fix
> them.
>
> �
> Reply to this email directly or view it on GitHub
> <https://github.com/hakaru-dev/hakaru/commit/cc9623d89aaf751db8b0ab1410bb8c31a067d813#commitcomment-14571067>
> .
>
�
Reply to this email directly or
view it on GitHub.


Thanks, I was just working on measure test change so didn't think of launching the GUI to generate xml change. I'm working on various fixes for measure on a branch, so I'll overwrite these checksums and the version_id when I merge it back.


I'm a little puzzled that this needs `: Equatable`.


Let's make sure that the SQLite error constant name at least appears in the comment.


But I can totally use `SQLITE_CORRUPT` in SwiftData.swift. Xcode even autocompletes it for me…


Dobra, sytuacja uratowana.


TeamCity TerraFX :: Debug [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=9&buildTypeId=TerraFX_Debug&guest=1) is now running


Nie rozbijałbym na 2 wiadomości. Może wpleść się coś pomiędzy.


<img width="429" alt="screenshot 2015-11-23 12 23 04" src="https://cloud.githubusercontent.com/assets/91722/11348709/fd3d5204-91dc-11e5-813a-b3f2b0f1079e.png">


TeamCity TerraFX :: Release [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=10&buildTypeId=TerraFX_Release&guest=1) is now running


s/Filenamed/filename


TeamCity TerraFX :: Debug [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=9&buildTypeId=TerraFX_Debug&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:46


TeamCity TerraFX :: Release [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=10&buildTypeId=TerraFX_Release&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:48


Oops, I meant #218


TeamCity pull requests :: netty [Build 725](https://garage.netty.io/teamcity/viewLog.html?buildId=3531&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5707, ignored: 22 Build time: 00:41:21


@hugho-ad
Remove print, please


Plagiarism is wrong.


TODO z powiadamianiem użytkownika


Thanks Chris for reviewing the changes. Jenkins jobs are underway; once they are finished and blue, will merge in the changes.


Good point. Changes next commit.


What is this being used for?


For attaching an action, yes, in some form. Exactly what form that takes is another question. Possibly something more like `someModel.getThings().configure(new Action<>() { })` than `someModel.things(new Action<>() { })`.

That is, have all managed things extend some interface which provides an API for working with that managed thing, such as attaching rules.


@bil-elmoussaoui the sidebar in those plugs is used a place where you can add things –further users, printers, languages, etc.– so a list view is more appropriate in those cases.


To wywołanie dałbym do start zamiast tutaj.


You're welcome. Thank you for your contribution!


@hugho-ad
Avoid the `for`and reuse all ids in a sentecen


So, the problem is that in this context = would refer to the CLASS object I believe, I think this is correct. I'll test it.


 toshi/ 092b8d0
Add 'Deploy to Heroku' button
@adrianmacneiladrianmacneil committed about 1 year ago
2 changed files
45 additions and 21 deletions
README.md
app.json
 README.md
@@ -34,7 +34,7 @@ Some examples of queries which Toshi can easily answer, which are not possible w

 ## Usage

-#### Using the hosted version
+### Hosted Toshi

 Coinbase maintains a hosted version of Toshi that you can use at:

@@ -42,25 +42,7 @@ Coinbase maintains a hosted version of Toshi that you can use at:

 This is the easiest way to get up and running. You can also run your own version of Toshi as described below.

-#### Running your own copy in production
-
-Toshi can be installed on Heroku in just a few minutes:
-
-    $ git clone https://github.com/coinbase/toshi.git
-    $ cd toshi
-    $ cp config/toshi.yml.example config/toshi.yml
-    $ heroku create [APP NAME]
-    $ heroku addons:add heroku-postgresql:dev
-    $ heroku addons:add redistogo
-    $ git push heroku master
-    $ heroku run rake db:migrate
-    $ heroku scale block_worker=1 peer_manager=1 transaction_worker=2 web=1
-    $ heroku open
-    $ heroku logs -t
-
-Note that the free PostgreSQL database on Heroku will need to be upgraded if you want to sync mainnet of bitcoin.
-
-#### Running your own copy in development
+### Running Toshi locally

 Toshi uses [Vagrant](http://www.vagrantup.com/) to install and run all prerequisites (postgresql, redis).

@@ -78,8 +60,27 @@ Alternatively, you can use Docker:
     $ docker build -t=coinbase/node .
     $ docker run -e REDIS_URL=redis://... -e DATABASE_URL=postgres://... -e TOSHI_ENV=production coinbase/node foreman start

+### Deployment
+
+Toshi can be deployed directly to Heroku:
+
+[![Deploy](https://www.herokucdn.com/deploy/button.png)](https://heroku.com/deploy?template=https://github.com/coinbase/toshi)
+
+Toshi can also be installed on your own server. You will need:
+
+* PostgreSQL (300gb+ disk space required to sync mainnet)
+* Redis (50mb+ RAM recommended)
+* Ruby 2.0.0+
+
+To run Toshi on your server, simply:
+
+    $ git clone https://github.com/coinbase/toshi.git
+    $ cd toshi
+    $ cp config/toshi.yml.example config/toshi.yml
+    $ vi config/toshi.yml
+    $ REDIS_URL=redis://... DATABASE_URL=postgres://... TOSHI_ENV=production bundle exec foreman start

-## HTTP API
+## API

 > Note: The Toshi API provides raw blockchain data only. If you are looking for APIs to store bitcoin securely, buy/sell bitcoin, send/request bitcoin, accept merchant payments, etc) please check out the proprietary [Coinbase API](https://coinbase.com/docs/api/overview).

 app.json
@@ -0,0 +1,23 @@
+{
+  "name": "Toshi",
+  "description": "Bitcoin node using Ruby + PostgreSQL",
+  "website": "http://toshi.io/",
+  "repository": "https://github.com/coinbase/toshi",
+  "logo": "https://github.com/coinbase/toshi/blob/master/lib/toshi/web/static/images/toshi_logo.png",
+  "keywords": [
+    "bitcoin",
+    "node",
+    "ruby"
+  ],
+  "env": {
+    "TOSHI_NETWORK": {
+      "description": "Specifies which network to connect to (one of `bitcoin` or `testnet3`)",
+      "value": "bitcoin"
+    },
+    "TOSHI_ENV": "production"
+  },
+  "addons": [
+    "heroku-postgresql:standard-4",
+    "openredis"
+  ]
+}
0 comments on commit 092b8d0

Comment on 092b8d0

Comment
 Desktop version


Technically, EXTENSION is a Python module while BINARY is any kind of dynamic library. None of the build targets treat them differently from each other, so there's no practical distinction except when debugging the output of the Analysis target.


I'd extract common logic


can you describe what sort of validations?


should we skip `this` daemon?


what's the purpose of this attribute?


The if statement checks it is a floating point type std::is_floating_point<T>::value


Thanks to @CheerfulYeti for the bug report.


Uwaga, uwaga! Niech Eclipse i Source -> Format stanie się Twoim przyjacielem! Proszę używać przed każdym commitem.


wrong name its `SPEC_OPTS`


I see, thanks for your answer! :+1:


@rahulbot and @kanarinka you'll probably want to fix the language here (and I'm also not sure how to interpret cosine similarity, so these are just my best guesses)


Maybe you already considered this, but it looks to me as if you can clean up a single branch via the REST endpoints.  e.g. system("curl -i -X DELETE localhost:8474/proxies/test_redis") works just fine (as it does via the httr package as well)

```
library(httr)
DELETE("localhost:8474/proxies/test_redis")
```


Consider generating this rather than hard-coding the length.


O, to dodawanie zależności do bibliotek w mavenie, tylko tyle potrzebuje.  Dzięki, następnym razem będę ogarniał. I sorry za złe kodowanie znaków, nie zauważyłem nawet, że kod się wykrzaczył na polskich znakach.


I think Xcode wasn't indexing or something. I got it too but had to cast it to an Int since I can't comp Int <-> Int 32


Prawdopodobnie będzie to wyglądać tak, że jeżeli experiment ma flagę że jest algorytmowany, a algorytmu nie ma w bazie, to wyświetl error.


Uznałem, że to bez różnicy, ale ok.


Продолжаю чинить мелкоборды.
Так как есть несколько клонов нульчана с общими особенностями, вынес их в отдельный класс, который наследуется от кусабы. Столкнулся с проблемой - `ibEngines` перебираются прямо. Т.е., класс `_0chan` не мог быть применен, а выбиралась Kusaba, так как `_0chan` стоит позже (а раньше чем родителя его объявить нельзя). Та же херня теоретически могла быть у vichan-досок без указания домена, т.к. vichan наследуется от tinyboard.


:: operator is like `cons` in functional languages and can be used to contruct lists e.g 1::2::3::Nil will evaluate to List(1,2,3).  In this case, pattern matching is applied to the next document line (a Seq) that is represented as <content>, <date>, <metadata> in the file.


:panda_face: :+1:


Racja, będzie lepiej testować nawet.


Visual Studio indentation için default olarak tab kulanıyor (ayarlanabiliyor). Burada sanırım space kullanılmış. Bunu standartlaştırmamız gerekir mi?


I think you might be meaning `dispatch_once` here. But that's not quite what I meant; instead, something like:

```
static var corruptionLogsWritten = Set<String>()
…
func writeCorruption…(dbFilename: String, …) {
    dispatch_sync(…) {
        if corruptionLogsWritten.contains(dbFilename) {
             return;
        }
        …
        corruptionLogsWritten.add(dbFilename)
```


http://www.0-chan.ru/b
http://0chan.cc/b

Может, и другие есть, еще не добрался.


Ezee, I think a fork is radical decision, but if is your final decision I respect. We all appreciate your work and your enthusiasm, we just need some balance.
I think the easy solution is
1- Keep your branches as locals.
2- Avoid too much notifications.

You are welcome here, you don't need work in a separated stuff. We just need adjust our internal clocks :)


Looks interesting, I'll get it right away!


Gah!  Having `parentid` be a GUID and having `id` be a primary key integer... sadness.


`graceful` is `true` in all usages


Zrobione.


Careful! Even if `spectator` were to send something, it would send (player, 1, 2), not (player, 2, 1). You can use `any()` as an argument to match all invocations of the method.


На некоторых мелкобордах есть ловушки с полями, скрытыми через `display: none`.


Yes. The `cables.csv` is already present on frink (src/data/..) and the other file can be found in my home directory. But you can also use the provided documents in the resource folder for testing.


like it, will test soon & we will need to add that to documentation as well, but I can do that


@hjmjohnson @dgellis90
There is a conflict here shown by "<<<<<<" and ">>>>>>".


I think I would find this line would be easier to understand (and compare to line 1582) if this line were written:
```
if ((closeTime - 120) > val->getFieldU32(sfSigningTime))
```
Just a suggestion.

With Howard's upcoming change the two `120` values will become `120s` or `2min` which will also help.


Can you do the same on your devDependencies with "css-loader": "^0.21.0"? Maybe it solve the entire problem with peer because i still cant install the loader, take a look:

└── UNMET PEER DEPENDENCY css-loader@0.23.0

npm WARN EPEERINVALID vue-loader@7.1.1 requires a peer of css-loader@^0.21.0 but none was installed.
npm ERR! code 1

Thanks in advance


Иногда ловушки могут стоять прямо рядом с полями ввода.


Chiyo logic:

>doesn't know what something does

<!-- -->

Anyway, I'm guess that's for "http\://example.com/**wiki**/Foobar".

>"let's comment it out!"


Strongly consider `s/id/guid`.


TeamCity TerraFX :: Debug [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=11&buildTypeId=TerraFX_Debug&guest=1) is now running


TeamCity TerraFX :: Release [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=12&buildTypeId=TerraFX_Release&guest=1) is now running


The if-statement is a runtime check, not a compile-time check. The whole block must be reducible before code-folding can occur, but it's not. I've got a fix, give me another moment


Only the password really needs to be private here.


TeamCity TerraFX :: Debug [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=11&buildTypeId=TerraFX_Debug&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:40


Ah, sorry – I meant to link to the library, but then got distracted: http://hackage.haskell.org/package/total-1.0.4/docs/Lens-Family-Total.html – but it seems like it’s too weak/hacky to work for us.

So … (once I finish 😭 about sum types again) I feel like the best option is

1. case classes in the usual place (not in `object Case`), with private constructors, so we can pattern match exhaustively,
2. `val` “constructors” as you’ve added here, but I guess they’ll need to be named lowercase, and
3. and then the prisms, which … need another naming convention?

At least it’s no more code than is already here, but still: 😭

Also, I don’t think it has to be fixed in this PR, but we do have to come to _some_ consensus about sum types.


TeamCity TerraFX :: Release [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=12&buildTypeId=TerraFX_Release&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:44


I think `lazy` is unnecessary here


I kept these here to mark that is is known and possible, but not used and therefore commented. I know not necessarily the best reason but I wanted to avoid PRs which "added support" for sha512 even though nothing is generating such digests. While allowing sha512 is technically possible we want to make sure everything is fine with sha256 before expanding the list of support ID formats.


For my own curiosity: can you do these two `SELECT` statements in one shot?


This test can be deleted.


also added a bunch of comments


Is there a good reason for this either/or condition? I changed this to only use the `readKey` in my latest PR (now in master): https://github.com/keen/keen-js/blob/master/src/core/utils/sendSavedQuery.js#L8


Lo sai che io non giudico
Il 23/nov/2015 08:17 PM, "Manuel Uberti" <notifications@github.com> ha
scritto:

> Non so se mi sento all'altezza.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/enricodangelo/hello-world/commit/af1f465c8a559ecf52f54d984a286b3714fadc11#commitcomment-14569860>
> .
>


Note that this was tested and approved by @blandis


What does `is_overridden` mean?


Can we do a comment explaining that instead of having commented out code?


Getting

```
2015-11-23T20:35:10.442221+00:00 app[web.1]: [23-Nov-2015 20:35:10 UTC] PHP Warning:  Illegal string offset 'GITHUB_API_KEY' in /app/index.php on line 19
```

on this line, and am still getting NAN.

![](https://reposs.herokuapp.com/?path=CocoaPods/Specs&style=flat)


why not extract this into a class?


Cool, should be all set! Released as 0.6.0.


красава


why not use `String.join(",\n", elems);` instead?


Basically, the following code:

```
var domain = require('domain');
var assert = require('assert');

var d = domain.create();
d.run(function() {
	process.nextTick(function shouldNotRun() {
		assert(false);
	});

	d.dispose();
})
```
asserts on node v4.x/v5.x and master, and exits successfully with node v0.10.


It' a MUST include a set here, but a MAY on line 429?


Very first version of EZOP Ontology Server Class Diagram. This one is pretty far from the final version.


Yep, which is why it's moved back in the most recent version.


Great, thank you for the PR @nithinnara!


Good job, peasant!


fixed in ToT. thanks bzhu5!


Please check https://github.com/lotus/assets/blob/master/lib/lotus/assets/config/asset_types.rb#L12-L24


In kernel, we seem to add a new line for another year sometimes instead of modifying exisitng one. But I am not expert here.


Why is it bad if it's a runtime check?


Fixed, but imho it's superfluous and makes code look inconsistent. In very this code block, few lines above, we use simple #if and suddenly that &&.


it should be inlined


Just in case you might request a merge of this at some stage, I would not merge this as is. That's because I do have occasional foreign dll added, and temporary testing *.exe generated while experimenting, and I do want to not ignore those.

It is also for this reason (I have occasional foreign dll's, and occasional testing *.exe) that one of the difference between the 189/194 build I sent back to Microsoft a few months ago, and the 194/194 current I have, is that "make clean" no longer wipe *.dll and *.exe, but only delete those that were made.

If you want to merge this, I suggest you make this more specific.


Is there a reason to not fold this into the update below?  I'm surprised you're tracking favicons in the mirror, tbh, but I don't see another natural place for them.


humm, is this absolutely safe? need to think about this


i've checked, the user receive an const pointer to connection, call ``sol_http_client_connection_cancel`` will generate a warning and sould not be done. It's the same problem of calling this function from ``response_cb``


https://github.com/NARKOZ/hacker-scripts/pull/6#discussion_r45599244

```python
if today.strftime('%A') in ('Saturday', 'Sunday'):
```


Fixed by adding a helper method using the provided source for coreclr.


This is because this commit is not published yet...


yeah, the change was reverted in a later commit.


TeamCity TerraFX :: Debug [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=13&buildTypeId=TerraFX_Debug&guest=1) is now running


TeamCity TerraFX :: Release [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=14&buildTypeId=TerraFX_Release&guest=1) is now running


oh i'm sorry and thanks you


TeamCity TerraFX :: Debug [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=13&buildTypeId=TerraFX_Debug&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:29


TeamCity TerraFX :: Release [Build 0.0.1](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=14&buildTypeId=TerraFX_Release&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:31


You mean, "We could implement this with a view", right?  Not something (nonsensical) about the output of the query.


No, but there are occasions you want to run it without a snp table and
output standard snps. This is what you do when you build the panel of
normals.
On Mon, Nov 23, 2015 at 12:12 PM Stewart Stewart <notifications@github.com>
wrote:

> @jstjohn <https://github.com/jstjohn> Does the lack of a SNP table really
> imply that this is a novel somatic mutation?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/drivergroup/avocado/commit/7218475f35ebb24b253543de6934e13b866ce05b#commitcomment-14571084>
> .
>


The TiberianSunRefinery trait needs a WithVoxelBody vehicle to dock. Not sure if the regular Refinery trait is sufficient here


yes, this is simpler.


Why include the `id` at all?


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


Operations with this lock are (relatively) slow.
May require a lock per name instead of a lock for all operations. (see pkg/locker).


issue #1
fixed in 95b8512fa9b7ee5aafff77c05c47966e139f6b8e


fixed in 95b8512fa9b7ee5aafff77c05c47966e139f6b8e


There are a couple unsafe methods in the code.


issue #2
fixed in 98c59e80747ae60dfbeec2e02c33128dec9f7c8a


Ok, so I just tested this on master, here's the results, "sequenced" refers to the current way we're running tests with coverage, by explicitly sequencing the test order, "all" refers to the way it is done in this PR, "total coverage" is obtained via `sbt coverageAggregate` in both cases:

run|`core` coverage|total coverage
----|---------------------|-------------------
sequenced|80.05%|80.23%
all|80.05%|80.22%

this is consistent with what we [see on travis](https://travis-ci.org/quasar-analytics/quasar/builds/92474252) (though travis is slightly higher as I didn't run the "delete your db" tests locally). However all three are a couple of percent lower than what codecov reports, 82.11%, which I can't explain.

Also, I think we do need to keep the `coverageReport` line as sometimes `coverageAggregate` fails if you don't run it first.


I haven't been using VS, I just wanted to make it easier to change the picture size.


Why by reflection instead of direct call?


На клонах нульчана есть каталоги. Заодно избавляемся от лишних вызовов имен.


I was under the impression that was the reason for the CircleCI failure, but it seems you came along before I did and include'd limits, fixing the problem. Seems numeric_limits<T>::quiet_NaN is defined to be undefined for non-floating-point types. @_@


Say a word on `excludingGUID`, please.


This is ok. Basically it is just a flag for the case where you have a list
of known false positive mutations. There are lots of other kinds of noise,
this just covers that one kind. We could change the variable name to make
that more clear.
On Mon, Nov 23, 2015 at 12:13 PM Stewart Stewart <notifications@github.com>
wrote:

> @jstjohn <https://github.com/jstjohn> Is this appropriate behavior? Act
> as if it's not noisy if there's no data to support or deny that? That
> strikes me as dangerous...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/drivergroup/avocado/commit/b50159553e57edd8e6407294ea73892c14fea26b#commitcomment-14571115>
> .
>


I know the README says to use this syntax but... it seems wrong. I would move it to line 21 like so:

`  _exclude: /N7K/`


Have you tested this diffset on an N7K ?

If you don't have an N7K testbed then as a test you can try changing that regexp above to N3K or N9K just to see if the vn_segment config_get_token gets called.


removing these in a coming PR since they are no longer arguments to setup()


I suggest you can use unordered_set rather than unordered_map here (you are just interested in whether the int exists in the set, right?)


This is pretty brutal to follow.  Is there a way to name arguments in SQL?


I had to `yo faithlife-app`.


This looks good!


Please do not use loopv. If you want to loop over all elements of a container you can usee a range base loop.
http://www.cprogramming.com/c++11/c++11-ranged-for-loop.html
(the container needs to provide begin() and end())


TeamCity TerraFX :: Debug [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=15&buildTypeId=TerraFX_Debug&guest=1) is now running


TeamCity TerraFX :: Release [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=16&buildTypeId=TerraFX_Release&guest=1) is now running


Oh.  Hahaha.  I should have known... I have written this base case avoidance myself so many times :)


TeamCity TerraFX :: Debug [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=15&buildTypeId=TerraFX_Debug&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:29


Yeah, it directly uses the gem configuration and local config can be used to override. I think that's simpler to manage in our volume of tested gems.


TeamCity TerraFX :: Release [Build 0.0.2](http://terrafx-ci.westus.cloudapp.azure.com/viewLog.html?buildId=16&buildTypeId=TerraFX_Release&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:31


Made changes according to the comments above. Ready to review


Done


I am not sure we need to exit on devices having more then one partition. One can create 1 partition on a device and then add it to DEV and that's an error condition too. We just don't have a way to differentiate between two.

So may be we can just detect that if device has partition or not and issue an INFO message device $dev has paritions. Skipping partition creation. And we simply skip that device. Does not matter whether partition was created by us or not.