Internships are in San Francisco, but if you can find a way to get here and for us to legally pay you, we only judge based on skill.


Yeah, I would expect that `_.concat(xs...)` works like `[].concat(xs...)`, not like `[].concat.apply(xs[0], xs.slice(1))`, which would make `_.concat(null) = [null]` and `_.concat(null, null) = [null, null]`


done.


Looking for a job? I see skills I might need ;)


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


Needs to be done for the leaderboard as well


The restore.php is reloaded on each request – barring any OPcache blunders, e.g. opcache set up to ignore file timestamps. As for the Javascript side of things, the JS code I had written in 2011 will still work with the new version of restore.php from 2016. I don't break b/c unless there is a _really damn good_ reason and I actually think forward and backward compatibility before writing any code. I know, I know, that's not the Joomla! Way™ :p


I had the same question and the answer is simply there is no need, all OpenPLi >=4 images use the same libc.


??? When I try to create a variable with `var`, I get an error. If I type `testVar =0` into a fresh console, it registers and shows up in the completion already


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


There is no need to have the Spinner class there. It is already inside dotastyles. I just put it there so you could see what it was doing.


Good catch! thanks


Most probably you are not using OpenPLi supported stbs.

All OpenPLi stbs use the same libc and the DTV_ENUM_DELSYS exists.

I guess you can use a simple patch to define the what your libc misses.


What about Ground wire do you use this one ??A0   & A1   IS ANALOG IN PINSso you not using gnd. Just power 5v and 2 A&B pins  ?
Date: Mon, 28 Mar 2016 10:54:00 -0700
From: notifications@github.com
To: barbell-tracker@noreply.github.com
CC: raimiss8@hotmail.co.uk
Subject: Re: [wierzba3/barbell-tracker] Update README.md (e43ec2a)

@raimiss8

I use arduino uno board. Only 3 wires required, I put pin A to pin 14 (also called A0), and pin B to pin 15 (also called A1)

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub


It's also hard to fix problems in the spec if no one lets us know about the problem. Looking at the patch I wasn't able to figure out what problem you are referring to.

Please file an issue on the spec if there's a problem with Blob.close()


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2165 of 2170 tests passed, 470 test warnings)

 * 475 tests had: AUD diffs.
 * 5 tests had: Table big diffs.

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


boringg


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


@mlp6 This module can be used to visualize timestep information for a patient in 3D Slicer. User selects a patient number and then all 8 timesteps are loaded with equal Window/Level across volumes. Scrolling at the name in the slice pane of the viewer scrolls through the volumes.


Yana hi,

I downloaded your files for module2-solution and tried your kitchen. It is very tasty, the only thing is that your html has unclosed div (<div class="row"> and it's recommended to use h1-h6 within section elements). Your css is simpler compare to my thing (but in mockup for the mobile divice, vertical spacing is smaller than  horizontal:)) ).

Sincerely,
Igor Fedorov


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


// , @Grokzen, well, regardless, thanks for the quick service, mate.


Watch your indentation and formatting!


Use: http://127.0.0.1:8080/json.htm?type=command&param=setactivetimerplan&ActiveTimerPlan=1


Please move it into static variable. You don't need to compile the pattern during each method call.


@sicking `Blob.close()` works fine, but how am I supposed to know when it's safe to call it? My app saves files of arbitrary sizes, and I need to revoke object URLs to make room for more (500MB limit in Chrome, similar or higher limits in other browsers) in case the user saves multiple files. There's no event for me to listen to, so I'm just arbitrarily waiting for 40 seconds.


Awesome, please update the changelog to mention this!


Some things in the FAQ (e.g. threads, the pointer to private data, etc.) were quite useful for a project with so few documentation. Is there a replacement? :-(  The old website is gone, which was quite limited, but now I feel we are worse.


I haven't included dotastyles because it breaks some of my classes/ids which I carelessly used. I rather keep my dirty css clean :laughing:


Looks like there are a few instances of `lfe_shell` still.


tt is nowadays `code`.


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


You just come off as ranting about that something was changed, having 0 understanding or interest in knowing why it was done. It was done out of simplicity and consistency, and frankly there was a lot things wrong with previous behavior as well. You can't just flip the '1' here.

>want me to show you some very obvious use cases, where regular joe has absolutely no chance to recover after deleting Addons*.db ?

>there is an update to random system addon in kodi official repo, but the update depends on other addon. after deleting addons*.db, is that "system" addon broken ?

>not counting that it's at least annoying for joe to manaly re-enable everything his favourite skin might depend on..

First of all an update, like that will never happen. Regardless, that is a completely different issue that has nothing to do with this or what the default is.

And this "joe" is the same one that goes deleting his database files? If so, we have very different definition of regular joe. If you delete your video db you loose your library metadata and settings. If you delete your addon db you loose addon metadata and settings. If you deleted the addon db before, you lost you enable/disable states, exactly the same as now. For any normal users these things are irrelevant though.

>here one, how is joe suppsed to re-enable this without un/re-installing? what if it was not (anymore) available in a repo.

Will look into it. But seriously, this is the first time I'm hearing about this and this is not how you report a bug.


Saaks paremini


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


some solution for echo?


How do you save the blob? Generally one a read has started it should be safe to call Blob.close(), it should not affect read operations, like that of `FileReader` or `<a href="blob:..." download>`, that are already in progress.

If it does then please file a bug on the browser.


e/100.0 for instance will return a floating point number. so the retunr value of eval() will be a floating point number in the '%', 'c', and 'center' cases. The old parseCoordinate returns a integer value so I did not want to change that. in the new version.
Note: not doing floating point calculations will result in bad results as str(40/100) will return the string '0'...


I believe this needs to go above your `sketch.js` file or you will be getting an error.


Thanks, the change makes a lot of sense. Please make sure to indent by 2 spaces to keep the code coherent.


My favourite kind of commit, look at all the lines of code this removes! :-)


This is sort of a temporary thing I implemented to get campaigns working for my purposes (I'm using your awesome API in another project that is private for now). I think it definitely needs more abstraction, like you've done [here](https://github.com/biow0lf/eve_online/commit/0b22caab836a66f3c0e18e3a4fe61d27adfbe7b0). I have a branch I haven't pushed yet where I'm setting up the abstraction, but I'm in TEST and we're currently engaged in the largest fight in two years, so it might some time before I can push it. I sent you an email with my ideas for the abstraction.


Thanks Ming!


@aronatkins Here is the change we discussed.


LLR: -1.58 (-2.94,2.94) [0.00,5.00]
Total: 11404 W: 2078 L: 2096 D: 7230


Why not just leave permissions out of it and instead allow someone to define the display of menu and tabs in settings? We'd prefer to have configurers not mucking with permissions to make tabs/menu items show/hide anyway?  I'd rather not create a hole in the authorization framework.  If we do then at least have an issue to fix it or create the proper feature later?


>  If you delete your addon db you loose addon metadata and settings

if I delete addon db I dont lose settings.

> If you deleted the addon db before, you lost you enable/disable states

the case I shown with screenshots above is very real use case. it's one more reason addons should be ENABLED per default.

addon*.db can vanish for many reasons (including fs corruption). at any time.

normal users delete their addons*.db when shit happens and they are told to do so.

the previous behaviour was fine.


Shouldn't the Week 13 folder be at the root level, and not inside the Week 12 folder?


Oh. Whoops. I guess my git commit bash shortcut doesn't handle --amend.


блин


а якщо туди передати якусь фігню нічого не впаде? ) @vasylhoban


Well ... we usually think that the Stop button should work for anyone, especially an adult who wanders into a shop where a CNC is going wild ... don't know about the kid issue (they certainly have a habit of flipping the toggle switch on desktops as Maker faires) ... Actually, one reason we have kept the look of the Sb3 interface consistent is so that anyone who is at all familiar with ShopBot will know how to stop one without having to go looking around on the screen ...


I also get an error with hscript 2.0.5. Works fine with latest from GitHub though.


v3.2.0 is out! https://supermarket.chef.io/cookbooks/apache2


The settings could just be a list of `title, link` items separated by newlines for now. The link would get appended after the # (hash). e.g.
```
Messages, messages
User Config, configuration/user
```


@raimiss8
My rotary encoder has only 3 pins, A, B, and C....C goes to ground


Oh, if only C++ had a system command, then one could do
`system("cls");`
or
`system("clear");`


TeamCity Glyssen :: Glyssen-master Continuous [Build 0.100.6.bc2d2e45394c4b764933059378000d1b2a32c893](http://build.palaso.org/viewLog.html?buildId=70767&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1062, ignored: 13 Build time: 00:08:44


Please avoid insertion of pylint-specific commands directly into source files in this manner. We have not standardized usage of pylint and I do not support such changes until we agree on a plan on if/how probcomp repos will use pylint.


Yes that would be another solution - have a different configuration for what stuff is shown for each role. I felt it was similar enough to permissions that it made sense to use the same framework. That would allow us to get rid of the `!` operator and work around this problem altogether.


Oh shoot, we'd still need permissions involved in the link rendering so it happens automatically based on the user's permissions... ack.


we can make this choice optional then. (like having a page in the settings were the admin can choose the default behaviors of the authentication module)


а gulpfile.js#L13 ? @vasylhoban


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1062 of 1062 tests passed, 0 test warnings)

![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-32.55%25-green.svg)


How does this change hurt non-pylint users? For those who wish to use pylint, what alternatives can you suggest?


Missing `a`. thx @Lordnoname


Yeah, when a script became shorter, this can be called IMPROVEMENT! :-D Thanks to your changes on kwargs.


Array is not initialized.


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2171 of 2176 tests passed, 0 test warnings)

 * 5 tests had: Table big diffs.

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


He should name it `coupons`.


What if FAIL or PASS called not from main()?
I guess it is better to call exit(1) here.


Maybe you should try to find a way to **actually** clear the screen


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


It would be less confusing if he named coupon as coupons.


Ok if the roles are hard coded (preset) then we could associate a menu/tabs configuration app_settings property to it...


Create a branch for each change you would like to make a pull request for.  The branch is based on develop. Then you can create a pull request for the changes in your branch.

Create a branch:
git checkout -b your_branch_name origin/develop

Push changes to github
git push origin your_branch_name

Open your github repository to create a pull request. Normally github shows your latest change ready for pull request.


AFAICT this is a backwards-compatibility breaking change.

| CSV | Before | After |
|------------|----------|---------|
| Has header | Import crashed with an exception | Works fine |
| Has NO header | Works fine | Quietly ignores the first line |

So CSV files that didn't work before the change start working, and this is great. But CSV files that worked before the change lose their first line, without giving potential users any notification that this is happenin, and this is not so great. Am I missing something?

(I read this commit before upgrading to HEAD, so no harm done here)


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2171 of 2176 tests passed, 0 test warnings)

 * 5 tests had: Table big diffs.

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


missing closing parenthesis


was this line intended?


Add __ to ASSERT. This macro is intended for usage only inside your framework.


no method called "authorize"


implicit conversion of e to string


Port Source Please !!! And are orginal author of this commit .


Hey - this doesn't work now. I think because you are referencing a non-existent variable below (exit $cacheExists). This is causing the script to hang when it gets to this place.


@abesto I wouldn't expect this library to be final, without breaking changes for quite a while. It's still under very early development.


Nice!


This is a very similar situation to a tool that has an E-stop button on it. Anyone could walk by and press the button to stop the tool and in this case cause a lot more havoc than stopping it in a more controller fashion. If they don't have access to the app then the file should bee able to be resumed.


This violates single responsibility principle. IMO it's not a very good design.


File needs to have some tabbing corrected.


is this right? why is _customersDataService_ getting assigned to the _dataServiceFactory_?


How about moving the creation of the field into `__init__` so that it's clearer to all readers, not just to pylint?


missing closing brace for class


modif des comms pour ceux qui comprennent pas tout du premier coup et qui montre bien que tous les cas sont pris en compte.


It is non-standard and noise to the source -- if I use `flake8`, Martin uses `pychecker`, etc and we now start littering the source with linter directives claiming "it does not affect anyone else because it is a comment" the situation becomes messy.

My alternative is to ignore pylint when it is wrong, or figure out why it is making a mistake and notify the maintainers or submit a PR etc.


__ASSERT(__tasks_counter < __MAX_TASKS)


remove space before ]


(Note that 'creation of the field' does not mean 'training' -- `self.X_categorical = None` in `__init__` would suffice.)


what is _Factory.of_ - does it come from aurelia-framework or some other library?


I'm not clear on why `counter` shouldn't be accessible here, as this is within the scope of `counter`'s declaration. If you want to have a class nested within `A` but enforce the safety given by `private`, you should write this:

```swift
class A {
    private var counter = 0
    // ...
    class B {
        // ...
    }
}

extension A.B {
    func foo() {
        // can't see A.counter
    }
}
```


Why do you need the specific class, and not the generic interface?


byte[] inputBuffer = new byte[128];


__TEST_PROPAGATE_SIG - private macro


This is an unfortunate check. Been try to remove all prototype assumptions so that we can switch to simple functions style. I don't have any better suggest atm. :/


remove 2 empty lines here


That works too


The fact it's complaining about dead code here means this function isn't public, which will cause errors if external crates try and use the macros. Either this module needs to be public, or you need to re-export this function from the top level. If you go for the second I'd rename it something like 'empty_attrs' since it won't be attribute specific in any way.


@alanthai indent?


let's make some commit


for efficiency, switch statement is non-needed, only possible path through code is case 1, due to variable "i" only ever being modified to be equal to value "1"


Test


Very occasionally (1 in 100?), the codepoint loads, but the font texture fails to do so & instead returns a null reference.
This causes blank text on the loading screen and F10 debug display.

I can't find a cause for this (I suspect a threading race somewhere), but deliberately checking for the existance of the texture when we attempt to render a string would appear to be a logical fix.


Oh! I see... Totally missed it... Will fix it...


I'll fix those in later commits. The main one will be fixing lfe_guide/usr_guide so it works.


@yihui


Well that figures. Since they finally closed issue 10, you can get rid of the custom `get` and `set` functions I added in `ConsoleUtil.Interp`. They changed it to use `get/setProperty` by default, so it's not breaking unless the error handling makes a difference.


@fsaad The problem with ignoring a checker when it's wrong is that you have to keep examining the problem and making that decision each time, rather than once noting that you've decided it was wrong, and not needing to look at it again until you're ready to understand the source again. By contrast, ignoring a comment line is relatively little work. If we do at some later date standardize, it's also very easy to clean the source from this kind of litter, because it mentions the tool to which it is a hint.

@riastradh-probcomp, for some reason I thought the no-member here referred to something within scipy rather than something local, but I'll check. If you're right, then that's a much better fix.


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


Your patch is essentially the same as what you tested before...
http://tests.stockfishchess.org/tests/view/56f062b80ebc59301a353f2e


Get rid of this, please


and this


Seems `try` should be right after previous `set_pointer_mapping`. ) An exception might happen at any time.


Distance can be zero, and 500 is arbitrary. I am not sure of the best solution...


nice


What is that proxy that you use when you start the ember app?


They dont do anything.


Does this actually work? I though, when returning multiple elements/components from another component, they had to all be under a single parent.


Running sprt(0,5) locally to compare


System runs a command correct.


They blocked command console And I think that goes through command console.


wrong parenthesis? Eval::pieceTypeToValue **(** move.getPromotedPieceType()-Piece::PAWN_VALUE **)**;


> How do you save the blob?

For browsers that support `a[download]`, here is what I do, simplified:

    var link = document.createElementNS(HTML, "a");
    link.download = filename;
    link.href = object URL;
    link.click();
    setTimeout(revoke, some random number that I felt was enough);

> Generally one a read has started it should be safe to call Blob.close()

Not a single browser implements saving blobs like this (not Edge, not IE, not Chrome (anymore), not Firefox, and n/a in Safari as Safari doesn't support `a[download]`). If you close the Blob immediately after starting a 50MB download, you can be sure that the download will fail with an error.


Oops, thanks for cleaning up my mistake. I had experimented with other build options so I didn't catch this with my usual picky compiler options...


Correcting


weave will detect unmount and clean up


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


Attach files by dragging & dropping or  selecting them.

 Styling with Markdown is supported

Comment  Cancel

    +=======
    +I am new to GitHub and this is part of the Tutorial
    +


You're right, it's broken, good find @SirPython - I need to wrap this component into an outer div


Add myself to the list of contributors.


unused var i


Thanks for adding this.
I got it working with http://127.0.0.1:8080/json.htm?type=command&param=setactivetimerplan&ActiveTimerPlan=1
ActiveTimerPlan is case sensitive.


I think you can safely revert this now that tls-0.7.1/nocrypto-0.5.3/x509-0.5.1 is out in opam...


We may use a subprocess-like module to access powershell!


Add initial localisation file support.


Small punctuation fixes, etc.


Added -l to helpStr.


Also, the depth map may be problematic if the `near` and `far` are not tight. `near ` is actually ignored here. Perhaps we will just have to advise users to set a custom `LightShadow` in that case.


Can confirm that this now tries to connect to localhost with ssh.


"Updating to the latest Android SDK" seems like a lofty, maybe necessary goal.


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


final version


Hm.. I like the idea of keeping them for now just to make sure people with slightly outdated hscript versions don't have a different behavior.


Bummer
Score of sfb008 vs sf160327fbv: 738 - 794 - 868  [0.488] 2400
ELO difference: -8.11 +/- 11.09
SPRT: llr -1.65, lbound -2.94, ubound 2.94


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2203 of 2208 tests passed, 472 test warnings)

 * 477 tests had: AUD diffs.
 * 5 tests had: Table big diffs.

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


@davelab6 can you review this, please?


Make include guard naming compatible with the one used in test/zdtm/lib/*.h
Ass empty line after include guard.


Before pushing any more changes would you join us on the Dayz Mod Discord server at https://discord.gg/0bcNYiZIEmHtQfIc for discussion.


I did refactoring and added other source for SVG like Blob in Android.


Maybe we could try using a subprocess-like way to open powershell.exe, then run `cls` from powershell, then change the title back to normal?


This is an interactive governor modified by thehacker911.
Authors: (mentioned in the files).
Mike Chan (mike@android.com) (Base)
thehacker911 (maikdiebenkorn@gmail.com) (Tweaked interactive, rename to interextrem)


**being** the zeroth


Yes, maybe the 0 -> 25 gradient for MG -> EG is too big, as the failed test http://tests.stockfishchess.org/tests/view/56f9154a0ebc59301a354102 seems to show (where I extended the gradient to 0 -> 30). Another option would be to try LooseEnemies = S(0, 20);

Good luck with your tests :-)


Alright. Will have a look or maybe pick this up. Thx!


**heterogeneous** datasets


I KNOW THEY BLOCKED CMD, ALL BECAUSE OF YOU!!!!!!!
(maybe we can say this is for Maryville Matters?)


Then we get BATCH BACK, HELL YEAH!


look into debain max path length


maybe change to user id


is this long enough? If so justify


What do you mean? This is all internal anyway.


password has to go


what is the added value of id over student id?


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


look into debian max path length


I tried to release and it failed due to tag conflict, or something. So I had to manually bump the version then rerun the patch release. That succeed, but I guess got the version/tag out of sync.


Validar los estados que corresponden, al parecer hay mas de los que se necesita


should probably have a default false defined


default false?


Sure, it just seems strange to just have the port with no IP and a single colon. I think it would have been better to have a list of IPs to listen on or something, but I guess that's not how they designed it.


TODO: improve these regexs, using the following stolen from PHPCI (sorry Dan!)


haha you do not get anything


pull test inserts into a separate file please


bad variable name - it's not a column


default value false please


106994938-FixFrontSideSolarReflectance (mjwitte) - x86_64-MacOS-10.9-clang: OK (2170 of 2170 tests passed, 0 test warnings)

![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)


Make it a bit less technical and show MDSynthesis again. It makes a great point that your solution is generic and can be craft solutions for individual fields or even just within a research group to allow them a more effective management of data.


You pollute local scope with your variables here.


I'd like to apologize for using the term "dick" in reference to @ScottPJones earlier in this thread (and for necromancing such a long-dead thread in the process). I regret losing my temper and using inappropriate language on @quinnj's repository.


(pid == __CHILD_DISABLED) || (pid == __CURRENT_TASK)


First of all, you didn't mention it in math1.cabal. And why is it needed to create a special module for just two functions? Can't you place it in `Types.hs`?


if the IColumnReference is a hierarchy node you need to check if the metadata is null before treating it as a column reference. see below


@Planeshifter I modified the implementation to assume a __finite__ double. The reason for doing so is considering how such a check has been used in various special function implementations. In these implementations, positive and negative `infinity` are often dealt with as special cases __prior__ to checking whether `x` is an `integer`. In which case, checking that `x` is finite is unnecessary.

We could use `%`, which does, effectively, account for `+-infinity`, as `pinf%1 = NaN`; however, modulo is a considerably more expensive operation than `floor(x)`. If we can assume a finite number, we get a slight perf boost.

Your thoughts?


@XhmikosR thanks a lot, waiting for the PR. The project needed some code style for a long time. Although I'd change JSHint to ESLint.


:(
sprt (0,5)
Local test finished
Score of sfb008 vs sf160327fbv: 1038 - 1146 - 1197  [0.484] 3381
ELO difference: -11.10 +/- 9.40
SPRT: llr -2.95, lbound -2.94, ubound 2.94 - H0 was accepted
Finished match


this check for null column metadata is necessary because not all hierarchy nodes are column references.


Instead of the "Incomplete" icon this should just have text that says "Below Bronze" using the same font as the sub-rating text (but smaller of course).


move __temp_result_cr_start and ASSERT at line 122 into TEST_PROPAGATE_SIG
This two are excessive here.


I only have Nexa switches and it works like a charm.
How is your switches set up in Telldus Live? Mine are actually setup as Nexa self-learning switches.
If you want to solve this yourself just add following in the function TelldusDevice:

`this.log(device)

run homebridge from your shell and see what it prints out and then add the model in configureControllerServices


You'll be able to remove this 2 defines once you fix
> move __temp_result_cr_start and ASSERT at line 122 into TEST_PROPAGATE_SIG
> This two are excessive here.

below.


TODO: improve these regexs, using the following stolen from PHPCI (sorry Dan!)


I can't walk through the code line by line, but it looks like you have two different spellings for "medal" so you might want to just do a quick scan to make sure that this isn't causing any breakages.


SetMaxRunSpeed() is never set from our side. As long it is not set by cw this function call will be nil. See #58.


rerunning with sprt all my betas :(


Alteração em lógica para permitir protesto ou não do título.


:cake:


Is there any reason for this synchronization here?


Hi Red Ox! What about tagging it with the Redox atom or Rust's 'R'?


closing else would've been nice here


`Array#reverse()`?


MakeDegreeDaysConsistWithASH901 (JasonGlazer) - Win64-Windows-7-VisualStudio-12: Tests Failed (1665 of 2177 tests passed, 5 test warnings)

 * 5 tests had: AUD diffs.

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


@alexdean @celly I noticed that if we are given an invalid URL that query params are stripped off when passed by to the client.  Is this what we want? Is there any chance that the client would want their params back with the url?


:+1: didn't have time to fix this, thanks


TeamCity Device Info :: Prod Tests [Build 82](http://82.35.147.206:8111/viewLog.html?buildId=313&buildTypeId=DI_Test) is now running


Looks good! I tried to deploy the [bookdown-demo](https://github.com/rstudio/bookdown-demo) repo with `bookdown::publish_book(render = 'server')` but failed, and I don't understand the error message:

```
> bookdown::publish_book(render = 'server')
Preparing to deploy site...DONE
Uploading bundle for site: 8...DONE
Deploying bundle: 52 for site: 8 ...
Deploying to R version '3.2.4' (Local version is '3.2.4')
2016/03/28 20:56:48.804238864 R version: 3.2.4
2016/03/28 20:56:48.804391389 packrat version: 0.4.6.10
2016/03/28 20:56:48.988479149 Audited package hashes with local packrat installation.
2016/03/28 20:56:50.978863620 Installing R6 (2.1.2) ...
2016/03/28 20:56:53.591290190 	OK (built source)
2016/03/28 20:56:53.591745045 Installing RJSONIO (1.3-0) ...
2016/03/28 20:57:09.890759680 	OK (built source)
2016/03/28 20:57:09.891220674 Installing Rcpp (0.12.4) ...
2016/03/28 20:58:02.768478236 	OK (built source)
2016/03/28 20:58:02.768917361 Installing base64enc (0.1-3) ...
2016/03/28 20:58:06.104348478 	OK (built source)
2016/03/28 20:58:06.104799194 Installing bitops (1.0-6) ...
2016/03/28 20:58:06.134405729 	OK (symlinked cache)
2016/03/28 20:58:06.134825837 Installing digest (0.6.9) ...
2016/03/28 20:58:06.168377413 	OK (symlinked cache)
2016/03/28 20:58:06.168704259 Installing formatR (1.3.2) ...
2016/03/28 20:58:08.027406796 	OK (built source)
2016/03/28 20:58:08.027854714 Installing highr (0.5.3) ...
2016/03/28 20:58:09.756235196 	OK (built source)
2016/03/28 20:58:09.764964514 Installing jsonlite (0.9.19) ...
2016/03/28 20:58:18.300805741 	OK (built source)
2016/03/28 20:58:18.301269898 Installing magrittr (1.5) ...
2016/03/28 20:58:18.331649986 	OK (symlinked cache)
2016/03/28 20:58:18.332029230 Installing mime (0.4) ...
2016/03/28 20:58:18.363146090 	OK (symlinked cache)
2016/03/28 20:58:18.363468984 Installing packrat (0.4.7) ...
2016/03/28 20:58:18.396697587 	OK (symlinked cache)
2016/03/28 20:58:18.396993209 Installing rstudioapi (0.5) ...
2016/03/28 20:58:21.053658421 	OK (built source)
2016/03/28 20:58:21.054071233 Installing stringi (1.0-1) ...
2016/03/28 20:58:21.088428740 	OK (symlinked cache)
2016/03/28 20:58:21.088859361 Installing xtable (1.8-2) ...
2016/03/28 20:58:24.322302217 	OK (built source)
2016/03/28 20:58:24.322579754 Installing yaml (2.1.13) ...
2016/03/28 20:58:24.343647531 	OK (symlinked cache)
2016/03/28 20:58:24.343919101 Installing httpuv (1.3.3) ...
2016/03/28 20:59:27.552135500 	OK (built source)
2016/03/28 20:59:27.552633959 Installing PKI (0.1-3) ...
2016/03/28 20:59:31.694547177 	OK (built source)
2016/03/28 20:59:31.695018942 Installing RCurl (1.95-4.8) ...
2016/03/28 20:59:51.595907530 	OK (built source)
2016/03/28 20:59:51.596360400 Installing caTools (1.17.1) ...
2016/03/28 20:59:51.631948317 	OK (symlinked cache)
2016/03/28 20:59:51.632342695 Installing htmltools (0.3.5) ...
2016/03/28 21:00:08.745963083 	OK (built source)
2016/03/28 21:00:08.746688539 Installing markdown (0.7.7) ...
2016/03/28 21:00:08.779876133 	OK (symlinked cache)
2016/03/28 21:00:08.780249248 Installing stringr (1.0.0.9000) ...
2016/03/28 21:00:17.277318427 	OK (built source)
2016/03/28 21:00:17.277769068 Installing rsconnect (0.4.2.2) ...
2016/03/28 21:00:28.946547295 	OK (built source)
2016/03/28 21:00:28.947004917 Installing shiny (0.13.2) ...
2016/03/28 21:00:35.574885605 	OK (built source)
2016/03/28 21:00:35.575345908 Installing evaluate (0.8.4) ...
2016/03/28 21:00:43.973785746 	OK (built source)
2016/03/28 21:00:43.974267126 Installing miniUI (0.1.1) ...
2016/03/28 21:00:47.298721531 	OK (built source)
2016/03/28 21:00:47.299204533 Installing knitr (1.12.22) ...
2016/03/28 21:00:47.359034470 	OK (symlinked cache)
2016/03/28 21:00:47.359500529 Installing rmarkdown (0.9.5.3) ...
2016/03/28 21:00:54.041029937 	OK (built source)
2016/03/28 21:00:54.054552822 Installing bookdown (0.0.57) ...
2016/03/28 21:00:56.431734935 Error: Command failed (1)
2016/03/28 21:00:56.431746606
2016/03/28 21:00:56.431757862 Failed to run system command:
2016/03/28 21:00:56.431760063
2016/03/28 21:00:56.431768646 	'/usr/lib/R/bin/R' --vanilla CMD INSTALL '/tmp/RtmpboYHqf/bookdown-2597f1804d3e40313e3848e95ae42c51e035ac8a' --library='/opt/rstudio-connect/mnt/app/packrat/lib/x86_64-pc-linux-gnu/3.2.4' --install-tests --no-docs --no-multiarch --no-demo
2016/03/28 21:00:56.431778013
2016/03/28 21:00:56.431783824 The command failed with output:
2016/03/28 21:00:56.431785756 Error: contains a blank line
2016/03/28 21:00:56.431866715
2016/03/28 21:00:56.431871317 Unable to fully restore the R packages associated with this deployment.
2016/03/28 21:00:56.431879634 Please review the preceeding messages to determine which package
2016/03/28 21:00:56.431881799 encountered installation difficulty and the cause of the failure.
2016/03/28 21:00:56.431887470
2016/03/28 21:00:56.431889310 Some typical reasons for package installation failures:
2016/03/28 21:00:56.431894909   * A system library needed by the R package is not installed.
2016/03/28 21:00:56.431896749     Some of the most common package dependencies are cataloged at:
2016/03/28 21:00:56.431906005     https://github.com/rstudio/shinyapps-package-dependencies
2016/03/28 21:00:56.431908062   * The R package is Windows-only or otherwise unavailable for this
2016/03/28 21:00:56.431913405     operating system.
2016/03/28 21:00:56.431915217   * The package is housed in a private repository that requires
2016/03/28 21:00:56.431920294     authentication to access. For more details on this, see:
2016/03/28 21:00:56.431922374     https://github.com/rstudio/packrat/issues/270
2016/03/28 21:00:56.431927610
2016/03/28 21:00:56.431929442 Please contact your RStudio Connect administrator for further help
2016/03/28 21:00:56.431934885 resolving this issue.
Site deployment failed with error: exit status 1
```


@feimosi: I agree with ESLint. It's just that you have JSHint already so I tried to tweak that first.

Do you want me to split this or you can accept it one PR?


Weird


This would be the case without my change to the proposal about immediate
scope. I think that the names that I picked in my example show why I want
the immediate scope rule. I also described it at length in my reply to
Jordan. In short, I want private to mean the most private it can be. It's
also a very simple rule that is very easy to understand. If you want
sharing with other classes or extensions, you can use fileprivate.
On Mon, Mar 28, 2016 at 4:40 PM Nate Cook <notifications@github.com> wrote:

> I'm not clear on why counter shouldn't be accessible here, as this is
> within the scope of counter's declaration. If you want to have a class
> nested within A but enforce the safety given by private, you should write
> this:
>
> class A {
>     private var counter = 0
>     // ...
>     class B {
>         // ...
>     }
> }
> extension A.B {
>     func foo() {
>         // can't see A.counter
>     }
> }
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/ilyathewhite/swift-evolution/commit/c49c8587d09bbb5fde0fb29f390eeeb41aad815f#commitcomment-16868297>
>


I have to get my head out of Lodash every so often and see what native stuff has to offer. I learned `Array#some()` and  `Array#every()` the other day too.
:+1: Will remove it. Thanks for taking the time to look at this :)


That's what commenting/uncommenting huge blocks of code does to your code. :+1:


Does this all work?

I don't know if it's a good idea to expect the outfits themselves to have their "max wear" and "wear per day" set as attributes.  Even if we did want to allow that to be defined separately for each outfit, there would **need** to a be a default value, since no existing outfits define these attributes.

Also having a double instead of a int as the key to the innerMap makes me vaguely uncomfortable because I'm afraid of floating-point exact-match errors.  Plus you're going to have to write out that double to the save file and it may come out as "0.007500000000001" or some such thing instead of just "3".


It would be helpful if the contributors somehow had a bunch of free time for all the various stuff they intend to do at some point (preferrably sooner than later).


TeamCity Device Info :: Prod Tests [Build 82](http://82.35.147.206:8111/viewLog.html?buildId=313&buildTypeId=DI_Test) outcome was **SUCCESS**
Summary: Tests passed: 80 Build time: 00:03:34


TeamCity Device Info :: Mock tests [Build 30](http://82.35.147.206:8111/viewLog.html?buildId=314&buildTypeId=DI_MockTests) is now running


Make __tasks array to be explicit list of child task pids - not this __tasks wtf.


Just did a quick google search and it seems to be a devtools issue https://github.com/hadley/devtools/issues/394 Do we have an old version of devtools on the server?


Коммит я видел, чат смотрел, JS теперь вроде как подходит, а CSS - нет :(


__tasks_counter => __pstree_num_tasks


go ni!


BTW, how about using curly braces in all statements?


IMO this is a bad design. Tax calculation class does not need to know information about the cart. It creates a tight coupling. I don't like the way he uses callback to calculate tax: https://github.com/hjr3/halo-sample-cart/commit/4505347a821be17e2a2981e1fc0552a25caee46d#diff-9929612f48b744d4445787c3eadad3b8R130.


It's also valid for a value to be a boolean `False`, which would fail this check too.


I'm gonna open another issue to investigate this more.


__MAX_TASKS => __PSTREE_MAX_TASKS


Use xrange for python < 3 version


render = "server" doesn't work yet :-\
On Mon, Mar 28, 2016 at 5:17 PM Yihui Xie <notifications@github.com> wrote:

> Just did a quick google search and it seems to be a devtools issue
> hadley/devtools#394 <https://github.com/hadley/devtools/issues/394> Do we
> have an old version of devtools on the server?
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/bookdown/commit/2597f1804d3e40313e3848e95ae42c51e035ac8a#commitcomment-16868887>
>


@flaiming Sorry about the deletion, but design has been externally peer-reviewed;)


У Вас было такое классное название пакета, зачем же Вы так :)


Overuse of interface.


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


Did you mean to commit this to master?  Master is supposed to always be safe to push to cloudmasters, upon which all new proxies would fetch this config.


extra whitespaces near with brackets


TeamCity Device Info :: Mock tests [Build 30](http://82.35.147.206:8111/viewLog.html?buildId=314&buildTypeId=DI_MockTests) outcome was **SUCCESS**
Summary: Tests passed: 73 Build time: 00:03:42


isInvalidMovie? :dancer:


Yea, but IMO we should stop using classes for internal instances and switch to pattern matching only where needed. That pattern matching would like include some nominal type information that could power this check.


Right, the gulp task also needs to be switched. It can stay as one PR as long as it touches only the styling.


Yes, I agree with curly braces.


@psychobunny I did a git fetch around 4:15PM Eastern and restarted nodebb and don't seem to see any difference.  Maybe I'm just misunderstanding, or this is resulting from a different plugin?


Shouldn't this class be deprecated?


Serigne bi ta supprimé tout ce que j'avais rajouté ?


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1062 of 1062 tests passed, 0 test warnings)

![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-32.55%25-green.svg)


Hey @aranhoide, sorry, I was a little aggressive here.  I did verify that this build is safe to push by deploying and running a client against it.


All right, I'll split it just to be safe. It's mostly style changes, the global variable issue and a couple of es3 fixes.


These block string tests shouldn't actually be run, or should they?


nice, thanks danny.


Np.


I think this line accidentally got in!


Yes, I agree. I almost started making a PR for it, but I think that is a bit much for only changing a 0 to 1 in a text.


Could you elaborate on why you removed the string scope name variations?


Do make sure that the download is actually started before revoking the URL, or calling .close(). In both Firefox and Chrome, if I revoke the URL after the download has actually started the download finishes just fine.

Firefox pops up a save-as dialog which the user has to interact with before the download starts. I wouldn't be surprised if there's a small delay between when .click() is called and chrome actually starts the download too.

I agree that this is a problem since there's no notification about when the download starts. But there wouldn't be much we could do in the Blob API to indicate when it's safe to revoke a URL or to call .close() if these other APIs haven't even started to read from the blob.

The fix here would either be to introduce a save-as API which takes an actual Blob object, or to fire some form of event from the <a download> when the download actually has started (i.e. after any needed user approval).


The test runner pre-processes the file


fixed hash


These look like code smell to me `&$under10, &$totalWeight`. He's basically changing the global variables inside a function. I don't think he needs to use closure here as it makes the code more complicated and harder to read.


fair point.


Уже неплохо, но есть некоторые замечания.
а) пути к ресурсам (стилям и скриптам), указанные в header, в HTML файле, не соответсвуют реальному положению и исходном варианте не работает ниченго.

б) если привести к рабочему состоянию, то есть возможность влиять на чужие сообщения


Les commentaires je les mis à l'intérieur des if juste pour voir la doc qd je pose la souris sur une fonction


Because there is no way to properly apply them while also allowing a linebreak after the initial quote. Thus we'd have opening quotes without the extra scope name. Plus, they aren't really all that useful since there are scopes on the actual contents.


Don't call "pid vars" "pid vars". They doesn't store pids. They store either __CURRENT_TASK value or __NOT_CURRENT_TASK value.


@eabatalov but how should I specify this if it works both under python2 and python3?


__CHILD_DISABLED => __NOT_CURRENT_TASK


This is where the event would be sent to other servers on the network?


So much for FishWithATie ;)


remove empty line here


Telegraphing @ingalls 's likely objections since he's offline: options.stacks isn't specific to countries; carmen has no notion of what it's being used for, and as far as carmen is concerned it could just as easily be used to filter by state or whatever else. I agree that the current error message is bad, but if we want "country" in the error message it should get there by way of configuration, not by way of being hard-coded.


@eabatalov oops...


Define a Maven project variable for common version numbers so you only have one to change

Something like that: http://www.mkyong.com/maven/how-to-create-user-defined-properties-in-maven/


Sure. We can change to a flag on the internal instance, at least.


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


Does not matter whether the user answers the prompt with yes or no to delete the template, it always deletes the template. Also, we may not want to automatically close the template dialog after deleting a template.


Use Javadoc. https://en.wikipedia.org/wiki/Javadoc

Simply add `/**` above your method, your IDE should complete it.


nice catch


WorkflowJSON_Model (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


Naming conventions in java are camelCase.

Should be `dirCreated`. This applies to every variable and method


If it works on both then that's ok.
Check that format string stuff won't break compatibility.


I would suggest using a logging framework, like logback. It is very simple to setup and use and you'll thank me later.

http://logback.qos.ch/manual/introduction.html

Looks like a very quick start: https://wiki.base22.com/display/btg/How+to+setup+SLF4J+and+LOGBack+in+a+web+app+-+fast


createImage*C*opy*F*older


Nice, thanks.  I was just checking because of the 'test' in the name.


@sarbbottam @kentcdodds @ta2edchimp Added a prompt asking the username and/or contribution types (available in v2.0.0-beta7 (I think I'll make a real v2 release soon-ish).
Let me know what you think, and I hope this will make the tool more useful to you guys.
If you have actionable feedback, please create an issue so we can talk about it ;)


Branch `comb_dec2015_patches` confirmed working up through this commit for:

* qc on Lisa
* pca on Broad and Lisa
* imputation on Broad
* gwas on Broad


I think the calculation is wrong based on

` When order is $100 or more, and each individual item is under 10 lb, then shipping is free`


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


Picasso doesn't work well with android-gif-drawable, hence use of ION.
TODO: Replace Picasso with ION everywhere


Here, your ImageCloner is aware that it is running in a UI application.

For example, if you wanted to use this in a web application in a server context, you wouldn't be able to re-use this class as-is because it would prompt a dialog. A dialog on a server makes little sense.

It's not a huge deal but it is also an easy fix.

I would recommend defining a custom exception (perhaps ImageClonerException?) and rethrow the IOException in your custom exception. The class that *uses* the ImageCloner should know what to do with here (e.g. in the case of a JavaFX app, it knows it can simply display the error to the user, in a server log an error, etc..)


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1620 of 1620 tests passed, 0 test warnings)

![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-70.91%25-green.svg)


Oh which? 165? I didn't notice that before!


LGTM


^^ Maybe a different error code (`EINVALIDSTACK`) and then we can reinterpret at the API level?


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


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


Guard naming convention + empty line after it


+#endif /* TEST_ZDTM_LIB_PSTREE_ZDTMTST_WRAPPERS_H_ */


Can't you simply add the System.out _inside_ the try block and drop the use of `dircreated` altogether?


@Diethart это ещё что?)


O.k. I'm going to delete that one @acabunoc if that's what you're referencing?


Ah, but it is *not* equivalent to ```AfterInitial { IEUNG }```. Maybe it is easier to envision the algorithm as a stack machine of sorts. Once an input that cannot belong to the current block is found, the valid parts of the current block is popped, and the "invalid" parts are used as the foundation for the next block.
The input I found the bug with was "rora" which returned the output for "roxa" IIRC. The bug *should* affect about half of all words or so.


I think overweight shipping is calculated on per item basis. So this calculation is wrong.


WorkflowJSON_Model (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


@jcf Thanks, I ran into the same issue to.

Maybe this should go into its own github issue.


Работа зачтена, чат у Вас очень хороший.
Есть только пара вопросов
а) после смены пользователя, можно редактировать чужие сообщения
б) после удаления сообщения и перезагрузки страницы, сообщение восстанавливается (видимо не удаляется из local storage).
В остальном вроде как все здорово


This is a somewhat weak method name. It is always a great exercise to put yourself in the shows of someone who sees your class for the first time. This person knows nothing of what ImageCloner does.

That person sees `ImagerCloner.getNameonly()`. What does he think this method does?

I am this person right now and .. I don't know the answer! :wink:

Perhaps it shold be named `getFileName`. But then again, it doesn't really make sense in a `ImageCloner` class.. right?

One option would be to replace this by https://commons.apache.org/proper/commons-io/javadocs/api-1.4/org/apache/commons/io/FilenameUtils.html#getBaseName(java.lang.String)


Test_ - uppercase should be used


Все будет исправлено
29 марта 2016 г. 12:40 AM пользователь "nalivajr" <notifications@github.com>
написал:

> Работа зачтена, чат у Вас очень хороший.
> Есть только пара вопросов
> а) после смены пользователя, можно редактировать чужие сообщения
> б) после удаления сообщения и перезагрузки страницы, сообщение
> восстанавливается (видимо не удаляется из local storage).
> В остальном вроде как все здорово
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/EugeneSayko/webChat/commit/e16f19193b5d573a250dcc7963b5f84c92898d68#commitcomment-16869245>
>


If it is a simple utilitary method that is only used in the ImageCloner class, it is appropriate to have it here. It should however be private and static

`private static String getFileNameWithoutExtension(File file)`


I think `font: 0/0 serif;` overrides `font-size: inherit;` and `line-height: normal;` so they become useless, no?

Also setting the size to zero made also affected the margins making the boxes a little tight. Is n't `color: transparent;` and `text-shadow: none;` enough to hide the actual text?


WTF!!!!!.


I like camel case better


Consider using `Path` to create paths instead of manipulating strings. Usually when you're doing a lot of work with strings, you're doing something wrong.

`copyFile.toPath().resolve(getNameonly(currentimagefile)+ "_Overlay");`


plural seemed to fit better, but i could make a case either way


WorkflowJSON_Model (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


SE -> se


same comment as earlier


Can you add a permission node to use command selectors?


Perhaps create a Github issue for this? I think you can refer to them in the code as well.


hi this is a test


I think the Behat scenario means weight of each individual item, not `$totalWeight`.


Круто!
Но сегодня уже можно идти отдыхать :)


You should be fine with `Members.FirstOrDefault(x => x.Key == key).Value;` since it should return null when no match is found.


@alefisico When I run I get the error
Can't read input file 'Fall15_25nsV2_MC_SF_AK4PFPuppi.txt'
I am not sure if the above file is it, and we just need to rename. Can you check and commit the .txt file if this is not the correct one? Thanks.


Ah thanks, I see now.
The first line determines the "comment token" (I'd declare it as "syntax test line token", probably) and not the line being scoped as `comment`.

That invalidates what I wrote here: https://github.com/sublimehq/Packages/blob/1def87e88c3c21910e3cf33e06d4687418aee5fe/YAML/tests/syntax_test_flow.yaml#L55-L58


Roger that. Thanks for the quick response!

On Mon, Mar 28, 2016, 10:38 PM Scott Robertson <notifications@github.com>
wrote:

> @abesto <https://github.com/abesto> I wouldn't expect this library to be
> final, without breaking changes for quite a while. It's still under very
> early development.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
> <https://github.com/rienafairefr/nYNABapi/commit/8c29c70eddb333d33bdc12df9a3b71a8b39801c0#commitcomment-16868250>
>
--
Zoltán Nagy
https://abesto.net


np. hey, 1 away from 1000 forks.


It is [a gcc extension](https://gcc.gnu.org/onlinedocs/gcc-5.3.0/gcc/Conditionals.html#Conditionals).


исправил удаление

2016-03-29 0:47 GMT+03:00 nalivajr <notifications@github.com>:

> Круто!
> Но сегодня уже можно идти отдыхать :)
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/EugeneSayko/webChat/commit/e16f19193b5d573a250dcc7963b5f84c92898d68#commitcomment-16869450>
>


Fixed now.


@ebatalov here is fixed version


yeah I forgot to add these files


> Do make sure that the download is actually started before revoking the URL

If I start a 500MB blob download in Chrome, and cancel it 500ms later it also fails, even though the download "started" in my UI before being cancelled. I think that a finished event might be easier for browser vendors to implement than a started event + the necessary changes to Blobs.


@eabatalov here is fixed version


Best practice is to set all members to `private` and expose thoses that need to be exposed via `getters` and `setters` (if applicable)


type 8 was already used, so now you're conflicting with Lux Anima Runestone:
```c
	//Removes bonus_script
	if (bl->type == BL_PC) {
		i = 0;
		if (type&1) i |= BSF_REM_BUFF;
		if (type&2) i |= BSF_REM_DEBUFF;
		if (type&4) i |= BSF_REM_ON_REFRESH;
		if (type&8) i |= BSF_REM_ON_LUXANIMA;
		pc_bonus_script_clear(BL_CAST(BL_PC,bl),i);
	}
```


Is that regexp going to capture all emails ?
It's better to use a standard js library to validate email signature.


/cc @nikomatsakis I think this is the same Rust bug we talked a few weeks back.


`Iterator<>` is a somewhat dated way to iterate on a list.

I would recommend

```
for (Row row : projectFile.getCurrentSheet().getRows()) {
  ...
}
```


Unless you're playing with binary, you should use two ampersands (`&&`)


Can be simply replaced by `return duplicatadiag.yesPressed()`


`return labeleddia.yesPressed()`


@eabatalov oh, found one more bug. Sorry. I'll fix it now


Not your most useful comment :stuck_out_tongue:


Added B-Cycle Stations where the gbps feed is accessible.


You have already 10 `if` blocks, i think they look very repititive, do it in a loop and make it concise.


Right you see my encoder has 5 pins i'm using A  B   ground and 5vI did tried all settings nothing working for me also i'm using mega board so all that difference makes trouble I'm guessing.Could you send some screen shot of terminal data (would like to see what you see on screen from your arduino)
Up to now I'm just loading data in real time to excel and there one click does all trick for me.It's tricky to find real min and real max it took about 3 month for me to put all logic and formulas to excel.Now started making arduino code for finding all min max and so on but not sure if arduino will be powerful enough to do all alone
Date: Mon, 28 Mar 2016 13:29:23 -0700
From: notifications@github.com
To: barbell-tracker@noreply.github.com
CC: raimiss8@hotmail.co.uk
Subject: Re: [wierzba3/barbell-tracker] Update README.md (e43ec2a)

@raimiss8

My rotary encoder has only 3 pins, A, B, and C....C goes to ground

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub


The string types are determined by the opening quotation marks and I fail to see how the ending (or non-ending due to a bad line break) are relevant for those?


grantRuntimePermissionsLPw(smsPackage, PHONE_PERMISSIONS, userId);
twice ?

Line 629/631


All right. One branch per pull request. Push distinct and connected changes to this branch. Make pull request on branch, and I suppose - I can delete each branch after the pull request is finished.


Not necessary. The app will close once the `main` method exits (and all non-daemon threads have closed but don't worry about that)


I would suggest to split you commits into smaller chunk changes. Try to keep one file per commit, one commit per new component should be a good start.


@cybersiddhu Sure, I will use a standard library to validate emails.


I'm no longer sure what you are referring to. I removed the `sql`, `raw` and `raw-regex` variations because they can no be determined when consuming the initial `'''` or `"""`.

If you are referring to something else, please comment on the line.


Check out https://en.wikipedia.org/wiki/Law_of_Demeter

There is probably a better way to do this. In general, if you see code using the member of a member of a member, it's a bad smell.

AKA `something.somethingElse.someOtherThing`


Yeah that's a good point. I don't know for sure and if you want to leave it alone for now that's fine by me since the Table components are convoluted enough. I think you're right though.


@Viper2k4 fixt du das bei allen Sites?


I suspect you might be able to refactor this code so you could reuse it every time you create a NUMERIC cell (which is quite a lot)


So are we not bothering with proto checks or have no changes been made to teams throughout 1.0.0 to 1.4.0?


This commit looks good @mbrookes.


<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.


Perhaps simply create a `List<String>` with all the headers and then do

```
for (int i=0;i < list.size(); i++) {
  Cell cell = activeRow.createCell(i);
  cell.setCellType(Cell.CELL_TYPE_STRING);
  cell.setCellValue(list.get(i));
}
```

That would save you a lot of time and ensure that the next time you want to add a cell, it'll be very simple! Also, changing the order would be easier.


[minor] Typo: SLides -> Slides


Looks good! Congrats on your first repository, first commit and first substantial Java Project!!!


I meant `biblio-style: jss` here


Might be better as Github issues.


Incorporating order fees into this comparison led to some regression tests failing, and when I think about it, it makes sense that we should include fees in the comparion with the target order value, we should just compare the order value (no fees). Including fees (which is assumed to be done via margin required) is for more of a 'can I afford this' question instead of 'how much do I need' question. Remove the `+ orderFees` makes the regression tests pass again.

@jaredbroad  @StefanoRaggi thoughts?


@eabatalov looks like fixed all issues now


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2171 of 2176 tests passed, 472 test warnings)

 * 477 tests had: AUD diffs.
 * 5 tests had: Table big diffs.

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


keep up with mainline


Не совсем понятно, что за класс Console. Вы его ещё только планирует написать? В любом случае, имена методов принято всегда писать со строчной буквы.


@rakuco The chromium-crosswalk next is still pointing to cbb3901 so not sure if this change was in error or you forgot to push the two commits you mention in the commit message from master to next?


あ、そうか！
なるほどです。ありがとうございます〜


@JoshRosen This PR results in 20% perf improvement for at least one benchmark (K-Means), maybe more. No perf losses. Nicely done!


Почему только для вин32? Надо чтобы и под линуском было, и имя бинарнику дать например psimedia-demo и для всех систем инсталлы прописать, а то нехорошо без них как-то


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __style__: [include/chaiscript/dispatchkit/boxed_value.hpp:248](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/include/chaiscript/dispatchkit/boxed_value.hpp#L248) 'operator=' should return reference to 'this' instance. (operatorEqRetRefThis)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/c07c2a9cc2e6e05290c33810cb601912748f96b6/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Здесь вы немного не так, как я предполагал разбили  код на три строки.
В данном случае отделять объявление переменной от её инициализации нет смысла - им лучше быть на одной строке.


Yup. I screwed up. Fixing now.


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


The Travis [builds](https://travis-ci.org/agda/agda/builds/119085010) are failing in maint-2.4 due to the cpphs version. I'll fix it.


Looks good.


Looks like I was mistaken, the commit was actually in nodebb-plugin-composer-default

version 3.0.17 and up contains this change


Just return false. Also, you should use ply:ConCommand instead of SendLua.


Архиважно добавить инсталл-секцию по умолчанию для пси (/usr/lib/psi/plugins в линуксе), кому надо для пси+ сам префикс сменит. И там в корне чейнджлог и ридми лежат их бы в доки (/usr/share/doc/psimedia-ver в линуксе)


you are right, sorry :S


**Test**


DZ's code has a few major flows.

It appears that he wants to make the tax calculation logic resuable. The idea was good but the design was poorly executed. In his tax calculation class he referred attribute of an external class, which created a tight coupling. And then he decided to use closures to calculate tax and shipping rates, which was unnecessary and made the code more complicated.

With a mixture of global states, closures, and dynamic function calls (call_user_func), I had hard time to follow his code:
https://github.com/hjr3/halo-sample-cart/commit/4505347a821be17e2a2981e1fc0552a25caee46d#diff-9929612f48b744d4445787c3eadad3b8R105
https://github.com/hjr3/halo-sample-cart/commit/4505347a821be17e2a2981e1fc0552a25caee46d#diff-9929612f48b744d4445787c3eadad3b8R113

The shipping calculation seems to have a few bugs. Since DZ is under such time pressure, I can tolerant these few bugs. But judging from the design, I would say :-1: .


Ee, to sa nema ignorovat. To sa ma riesit:

https://developer.apple.com/library/ios/documentation/AppleApplications/Reference/SafariWebContent/ConfiguringWebApplications/ConfiguringWebApplications.html


**Test**

**Test**


I think I fixed coverage stats for develop but it hasn't be merged to master yet.. hopefully this will fix itself at the next merge/release.


Ach, ja som bol v tom, ze tomu bude stacit favicon... Takze #169


Thanks!


@klq. Finished linear algebra.  Ready to submit.

Thanks.


Nice test


CC @CavidSalahov :ship: :package:


WorkflowJSON_Model (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


and now color isn't working in CS:S as PrintToChat don't support colors ! :) `s/PrintToChat/CPrintToChat/`


Старайся избавляться от лишних комментариев. В данном случае это можно оставить, но во всех остальных случаях: меньше кода (включая комментарии) - меньше проблем :)


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2340 of 2406 tests passed, 0 test warnings)

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


Is there a gcloud 1.1.0? The latest version I can see @ https://pypi.python.org/pypi/gcloud is 0.11.0


Ugh that's an ugly typo... will fix!


Старайся придерживаться одного стиля программирования: если ставишь пробел перед фигурной скобкой, ставь его везде.


Снова по стилю: ставь пробел после запятой в данном случае.


I'd suggest to add a link to [www.peachpy.io](http://www.peachpy.io) - an online IDE for PeachPy


@psychobunny Thanks.  Was able to confirm as much by hunting down the original issue #4453.  How does the schedule work as far as adding new changes like that to NPM?


The following doc(s) have been deployed to the [dependencie](http://dependencie-static-docs.pantheonsite.io/docs) Multidev environment:
- [app/config/sculpin_site.yml](https://github.com/pantheon-systems/documentation/commit/c75e54e178c2902d1c084d066e723df15c1940be/app/config/sculpin_site.yml)
- [docs/wordpress-custom-code](http://dependencie-static-docs.pantheonsite.io/docs/wordpress-custom-code)


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


@nishkamravi2 glad to hear it!


Initial project mess around code


Aside from working around some LightXML.jl memory leak issues in response to a user reported issue, I haven't touched XMLDict.jl for a while. It is a dependancy of https://github.com/samoconnor/AWSCore.jl, so I guess it will stick around for a while. I'll check out XmlToDict.jl in more detail at some point if I revisit XMLDict.jl to see if there is any scope for combining functionality.


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


Инициализируй локальные переменные. Это поможет избежать ненужных ошибок в будущем.


Попытайся назвать экземпляр класса более осмысленно.


Please can we use curly braces '{'


@qbittorrent/webdev I was almost ready to release v3.3.4 with binaries etc. But I realized I need to bump the API version too.
I think `API_VERSION_MIN` needs bumping also with this change, correct?

Btw, I think notes of this new fields should be put in the comments above, `btjson::getTorrents()` and `btjson::getSyncMainData()`.

NOTE: This is the commit that exists in v3_3_x. There's an equivalent one in master too.


Сделай для данной стринги константу - будет лучше читаться, как по мне.

К примеру:

`private static final String ENTER_EXPRESSION_TEXT = "Enter the calculation expression: ";`


Haha you should make a PR :heart:


Пробелы перед и после присваивания.


И снова-таки - осмысленные названия переменных.


Thanks. Fixed, 749b75e


TeamCity Lobster :: Common [Build 0.0.1.9](https://pixelated.design/build/viewLog.html?buildId=358&buildTypeId=Lobster_Common&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 120 Build time: 00:00:28


За правильное сравнение +, красава :+1:


Можно убрать лишнюю пустую строку.


Nice!


man kann übrigens auch nur prüfen ob sGui true ist dann brauch man keine zusätzliche Variable
Und in seach schließ man dann die liste wie gehabt


Nope, trying to focus more on acute care surgery and being a surgical hospitalist. That and doing more medical administrative stuff. Should make my life much less complicated. Tough getting up at 2am too many times during the week and still trying to have outside interests, like family. :100:


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1620 of 1620 tests passed, 0 test warnings)

![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-70.91%25-green.svg)


I would accept this but it's just the recorder module I'm afraid will use too much CPU :/


Does not compile. Message from javac: "Fat.java:54: error: MondrianOlap4jLevel is not public in mondrian.olap4j; cannot be accessed from outside package"


Oops, fixed, thanks.


SubcategoryTagFixAndDocumentForLEEDreport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


106994938-FixFrontSideSolarReflectance (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2176 of 2176 tests passed, 0 test warnings)

![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)


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


Closes #11


Hello world


The benchmark wasn't changed, so it's running the same tests as before. As such, the results are in calls/sec, so higher is "better".


SETm starts will all spells at 1%, even Blading Hands at turn 0 :)


люкс!


#127 to restore


Ping @ivang (I hope this works)

I had to revert this because it broke inserting quotes. What was the error you were trying to fix with this?


Wow why you give it such a crappy name like BlahMine and even added a dash MP suffix. xD


What does CPrintToChat do that I'm not already doing? As far as I can see only AddPrefixAndDefaultColor().


This is a comment about a specific line of code.


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


Do make reward like that More kills more reward


Oh and also removed share buttons on pages.


Well I don't know about the specific commit, but it does compile because we rolled a release on it.


It seems both IP and Port properties are not used.


@sraedler Naja betrifft nur meinen (und meine Vorlage für diesen Teil. Filmpalast)
@Lynx187 in meinen Animie.Loads Plugin hatte ich das genau so :/ (ich habe es später wohl miss verstanden und wieder geändert)


bless you @yennycheung


Console.WriteLine looks like UI code, so probably it should not be used in a controller class.


Can't we use the PHP client here? @infojunkie @melsawy


I attempted to fit the else-if chain with an equation to enhance testability. Its a lot more scalable this way. Starting point 1ms quantum for borrowing should be palletable to most hardware config and was close to original. (I'd happily adress comments when they appear on a pull request.)


Alterações apenas para organização das pastas


Consider change the namespace's name. DesktopApp is not so descriptive ;)


Far Out!  Congratulations!


Do you need all the namespaces here?


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


@sarinhaxp esse location.reload aqui pode ser executado antes que o JavaScript remova o código.

Eu iria indicar que você colocasse dentro da função excluirPrograma(), mas parece que você até já fez isso.

Então seria melhor remover daqui, em produção pode ser que dê problema, já que as coisas não acontecem tão rápido quando numa conexão local.


А если я введу некорректные данные на входе? Допустим, вместо "**5 * 2**" введу "**олол примат окно**"? Как ты обрабатываешь данные ситуации?


thats not the background problem but nvm


Alright. Made some updates based on your feedback @kain88-de . It's here: e765364f9ad5a4e7c6b6d0c0c11efac1adc3087f


Sempre que puder evitar abreviação: selecionarDest -> selecionarDestinatario.


Hello there, I think that change will make maven expect that every submodule of a project must have the same pom file name that was specified with the --file parameter.
I think this is not the correct behaivor because the submodules pom file names can be specified in the parent pom.
I have already submited a [pull request](https://github.com/mojohaus/versions-maven-plugin/pull/58) that add the posibility to specify only the pom file name of the parent pom(or the actually pom from where you are making the call) with the --file parameter.


Why the -1 here? Shouldn't it be just `len`?


Does the README.md get committed automatically? If not, could that be done?


[![Landscape with a House behind Large Oak Trees near Bordeaux](http://api.artsmia.org/images/103181/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=103181)


[![Landscape with Pond](http://api.artsmia.org/images/111126/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=111126)


[![Morning in Late May](http://api.artsmia.org/images/114619/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=114619)


[![Stunted Tree](http://api.artsmia.org/images/115992/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115992)


> What does CPrintToChat do that I'm not already doing? As far as I can see only AddPrefixAndDefaultColor().

Dunno, it just don't work in CS:S with `PrintToChat` if I add `{teamcolor}` and therefor "force" `CSayText2 `it works again.
![240_screenshots_20160329005400_1](https://cloud.githubusercontent.com/assets/1080056/14092681/ffaa0c9c-f548-11e5-8e83-a8e45476e863.jpg)


So what was the primary objective in your 0day? To get the exploit fixed as quickly as possible, or to prevent as much damage as possible? They're not necessarily the same thing.


Eternal a toujours la meme erreur depuis la derniere PR :/


It's a bit inconsistent: you catch exceptions in Connect(), but don't do it in SendCommand(). I think it could be better to either don't catch exceptions at all (or catch only some kinds), or catch exceptions in both methods.


This .nuspec isn't getting used at the moment.  Unless we can figure out how to get it consumed, it should probably be removed in the next update.


How do you get focalID? when merging two entities, just take the first in the list (of duplicates) and update it. Also all relations, documentEntity table should be updated


Can this fix make it into the 1.11.1 release?


Ok, I'll look into it.


Is it necessary? You already specify this handlers in MainWindow.xaml, so this code looks redundant.


If we drop this dependency declaration existing projects that use it may fail. Perhaps that's not such a big deal but probably worth considering.


Also blacklist any relationship this entity participating


Can we do similar validation for IP textbox?


It doesn't. I guess it could, at least with a flag in the rc file. I'll take a look at it tomorrow/soonish.


Is this really needed? Why does GemFire need spring-shell for general use?


If you merge two entities, then the target is the focalId and the duplicate list contains just one element. But you could also merge more than one entity into the target e.g "Angii", "Merkel" to "Angela Merkel".

But you are right, I missed to update the relation and documentEntity table.


i need like, actual, real, sleep


I don't think this should be a `@Bean`, the `Properties` type is too generic to add to the application context.


Why not just:
   car.Connect(ipTextBox.Text, Convert.ToInt32(portTextBox.Text));
?


Weird, ugly, it's `default` but u put hardcoded skill id, only Meteor Storm unit that use `group->val2`.
just move them to new `case`


Does GemFire support regular log level configuration?


Right, missed that. Thanks.


What if Go() throws an exception? :)


Nice fix! Feel free to submit a pull request to [orsee/develop](https://github.com/orsee/orsee/tree/develop), there is a [manual ](https://github.com/orsee/orsee/wiki/Code-contributing-manual) how to do that.


> To get the exploit fixed as quickly as possible

Bingo! It got fixed within an hour.

If you're going to follow me around on GitHub, you may as well save yourself the effort and hop on `irc.spi.gt` and message `Jarcode`.


"g" and "s" seem to be a magic values, we should avoid it.


If the java config replaces this file I think we should just drop it.


IO operations shouldn't be performed in the UI thread, btw.


The `this.` prefixes should remain when accessing private fields.


One of my dev boards has a mistake in USB sense pin routing and I've tried to do a software detection of USB disconnect. Didn't found that sources, but remember what it was "bDeviceState" variable and I modified Resume() function in usb_pwr.c to count RESUME_WAIT events and made decision what USB has been disconnected when count goes over some value (500 if I remember correct).

As for MSC and CDC problem... I suffer such thing what after connecting MSC (worked fine), I've flashed CDC (VCP) on same board and driver did not worked (yellow exclamation sign in devmanager). The solutions: remove MSC device from windows registry prior connection of CDC device or modify CDC code so what it will return a different USB serial code.


OK, now I see that it has been fixed in the next commits


OK, Go doesn't throw exception in the newest code.


Plase


In fact, in IR, there should be no distinction between ADD and ADDi. There should be just ADD.
It happens because of optimization so this case is valid


On Synology DSM 6 using "df -B K" appends a "K" to the result. I think it's better to use "df -k"


btw what test case is this?


Damn I should stop using tabs and use spaces


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


Now I see that exception handling has been added to the SendCommand method. But I'm not really convinced that catching very general "Exception" is a good idea.


Also wasn't meant to make it pre-r5.


Ты просто сказал раньше как правильно это делать. Спасибо.


This class shouldn't be dependent on the UI. What if someone will want to use this class in the console-based application?


Oh, mea culpa! Will be more attentive the next time :)


Imagine you're user of this application. Do you really want to see a stacktrace in error message? :)


Excellent work! My function did the same- here's the rough code I wrote:

----begin code-----
import numpy as np
def OptionValue(input_stream, strike):
    x = input_stream
    K = strike
    y = x-K
    indic = y>0
    return np.average(indic*y)
---end code------

Since we have a basic set up, I'll send a more involved assignment in the next couple of days. We should meet next weekend over skype just to chat and catch up.

Have a great week guys!


This change is unrelated to the React on Rails upgrade.


I've updated to 1.655 on both server and slave.  When I start the slave, I get this:

Mar 28, 2016 3:07:05 PM hudson.remoting.jnlp.Main$CuiListener status
INFO: Server didn't accept the handshake: Unknown protocol:Protocol:JNLP3-connect

Other than updating to 1.655, is there anything else I need to do to get JNLP3 working?


Sorry, this is just the AI code that I'm working on. I'm not ready to make it a PR yet.


#24 Closes


youtube app , camera video recording failed  with Decoder init failed: OMX.SEC.AVC.Decoder, MediaFormat(1, video/avc, -1, -1, 640, 360, 0, 1.0, -1, -1, null, 856605744, false, 1280, 720, -1, -1)  so reverted back


@dasa, yes.


Good Job.  Here is written instruction to Update with the FireStarter app that is installed on version 2 FTV (you can see verion at top of instructions given with FTV) if you can double click to get to Kodi you have FireStarter; do the following.
FTV HOME SCREEN< SETTINGS< APPS< MANAGE APPS < LAUNCH FIRE STARTER< UPDATES< UPDATE KODI TO RC (RELEASE CANDIDATE)

) If you do not see the ability to update, then you need to go into settings and change the update policy settings and then update.

https://kodi.tv/kodi-16-1-jarvis-release-candidate-2/

Also the final key to the kingdom. https://github.com/sphinx02/FireStarter/wiki/Install-FireStarter-and-Kodi-(only-FireTV-needed) Welcome aboard;


@DenisCarriere Thank you so much!!  I really appreciate that.  It is a pleasure to work with your code. -Michael


Скобка.


The problem with this is that it needs to check the ymin/ymax of all the fonts in the family; so it needs to be a in a 'family check' section of the program that actually performs the check after the central `for font in font_list` loop; inside that method all you can do is append the current font's vertical metrics to a `dict(vmetrics)`


master (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


This is about an order of magnitude faster (with only three directories under posts) if you see the previous benchmarking commit. File IO is expensive.


Haha alright :+1:


I should mention its not previs but pre-vis as in pre visual. sorry for the typo.


Where is the search button for the tv shows on PopcornTime? Could you please add that as soon as possible. Thanks.


if `node_modules` is removed from `.gitignore`, shouldn't the `node_modules` directory be checked into this project? that's what you've done with [`webvr-boilerplate`](https://github.com/borismus/webvr-boilerplate/tree/master/node_modules)


Yes @Ruskiz you're right. This is left over from before i stored meta data in a separate object.
I'll add an issue and fix this, thanks!


The new mechanic made me do some really weird things, for example, I started training Spellcasting exclusively after reaching XL 3 (I had 6 max MP, needed 9 to cast Spider Form). And then I ignored Poison/Ice/Transmutations, training UC/Fighting/Dodging only. Seems too easy.


Todo: check for update query in edit_orders.php!!!
Todo: display all prices in order stats page


I see...

Then this will impose that the tool can only be invoked with a set of font files that belong to the same family. Can we imply that?


I think you're right @fduraibi
I'll do a bit more tests and see which looks better.. thank you again


master (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


WOAH


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 20096](https://teamcity.mesosphere.io/viewLog.html?buildId=181639&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:11


>>at least with a flag in the rc file

Or may be a option passed to `add`


are these comments deliberately removed?


**_So far, I have tested this on:_**
Nexus 6P - Android N (2nd build)
Nexus 6 - Android N (1st build)
Moto G (3rd gen) - Android 6.0
Lenovo Tab2 A10-70F - Android 5.0.1

I don't have anything lower so would love to see a test done on Android 4.0.3, which is API 15 (our minSdkVersion).
@wightwulf1944?


I'm okay that. All that really means is that instead of casting blade hands after getting to XL 8 or so, they do it at 1, but when they get to XL 8 their blade hands will be worse than other players at XL 8.

What worries me is any spell that has minimum damage that is independent of spell power. I should probably remove all minimum damages in spells. For example, you could probably cast shatter by Lair, if you could find it. I think shatter has minimum damage it causes, even if your spellpower is 0, which I don't like in this case. If I remove these minimums, the level 9 destruction spells will also be almost worthless for a SE, leaving them with TM and TL, which is still quite powerful, but interesting. And I think you could play a very competent hybrid too, since you could probably pull off controlled teleport in heavy armour with an SE. A stealth build is probably also very doable.


dsfa


What's happening? Will we have any non-joke index entries left?


This is idiotic right? Why are you requiring Django 1.9?


inin'm


this is all the files to work


Seems reasonable to me. I expect it to be too easy at the beginning, but that ought to change pretty quicky by about mid game I think. Unless you get lucky enough to find a spellbook with dragon form in it by then, but that's unlikely. The fact that it is motivating you to do some unconventional things I don't think is a bad thing. I think it opens up some creativite potential. Now if it's still way too easy by the end of lair we will need to do something about that I think. I would think death yaks and dire elephants ought to pose a bit of a challenge even for a player that can easily cast blade hands and ice form.


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


I wouldn't say "never", just that the feature needs more thought; we need some API to signify opting into error handling.


Please do not comment on code blocks or commit messages.
You are more than welcome to submit a pull request.
TV shows is still being worked on. Be patient.


@AdySan cool, don't get too excited, this core is still in a relatively early stage of development.


Well, we now need the loaders to run the tests, so they are somewhat redundant /confusing.


easter weekend run

[BEST_CANDIDATES - Run 0329.txt](https://github.com/unworry/Biota/files/192633/BEST_CANDIDATES.-.Run.0329.txt)


106994938-FixFrontSideSolarReflectance (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Objects will be replaced by identifiers shortly


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


you are right xD, fixing now


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


```
	STLX	Pts	  HOX		Daily	WLR	GPF	SLE         RTH
       =====   =====   =============   ======  ======  ======  =========   =====
*	S--R	955	5 T-C/SAR 3m	$385	0.37	1.93	89-0-61     -1015
       -----   -----   -------------   ------  ------  ------  ---------   -----
1	S--	899	0 3m       	$362	0.38	1.85	78-0-59     -1046
2	-T-	862	1 T-C     4m	$348	0.5	2.49	44-0-36     -168
3	ST-	781	1 T-A     3m	$315	0.36	1.59	106-0-57    -1100
-
5	S-L	439	3 T-A/LMT 4m	$177	0.46	1.32	122-93-37   298
6	-TL	626	3 T-A/LMT 4m	$252	0.48	2.17	67-25-1     627
7	STL	448	3 T-A/LMT 4m	$181	0.45	1.3	146-104-34  441
8	...R	867	4 SAR     3m	$350	0.36	1.77	93-0-58     -1108
9	S--R	955	5 T-C/SAR 3m	$385	0.37	1.93	89-0-61     -1015
10	-T-R	809	5 T-A/SAR 4m	$326	0.3	1.67	134-0-58    -1058
11	ST-R	847	5 T-A/SAR 3m	$342	0.31	1.7	153-0-61    -885
```


L'ajout de je sais plus quoi je sais plus où est donc la suppression du commentaire correspondant à l'ancienne version du typage de matrix_monster dans map


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


TL;DR the `marginRequired > marginRemaining` check covers insufficient funds, where as the `orderValue > targetOrderValue` check covers correct order quantity... we're searching for the min quantity that satisfies both conditions


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


@hrmohr I made it `df -B 1K`


106994938-FixFrontSideSolarReflectance (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2208 of 2208 tests passed, 0 test warnings)

![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)


@lox would you be able to look over this and update/remove/clarify anything I've said that is misleading… or perhaps add any missing nuance that a fellow Docker build traveller might find interesting?


Fixt for next patch


Fixt


Sigh. VERSION.


develop (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2392 of 2406 tests passed, 0 test warnings)

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


To create the binaries "e2fsck" and "mke2fs" yourself, you need my [patch](https://github.com/PeterPawn/freetz/commit/614f1cfa2ab8701050526b784e863efd078d1be3) for the Freetz toolchain.
The Busybox binary is also modified ... it contains this [patch](https://github.com/PeterPawn/freetz/commit/1f6666d18b77c97f7a99af92fc933d5359f37dcc) for the cksum applet.


Good call, I forgot about the migration db state changes that can occur outside of the rows themselves.


OK!


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


I merged it!  Woo!


frameout.cpp is SCI32 only so no need to `#ifdef` code in here


Good catch - I assume this was a problem when x was of dtype int? Do you mind changing it to be consistent with the current bleeding edge of keras? It's:
```
def mean(x, axis=None, keepdims=False):
    dtype = None
    if 'int' in x.dtype:
        dtype = _FLOATX
    return T.mean(x, axis=axis, keepdims=keepdims, dtype=dtype)
```


Another Test


You should rebase this on top of pipeline before merging.


we should revert this merge... it makes things confusing!


If my understanding of [strndup](http://linux.die.net/man/3/strndup) is right, `len` is the total number of bytes, including the null terminator. `neededLen` here is the size without the null terminator.


done :)


What does "standardizing line endings" mean, and is it something I can do for you in future?


OK I have reverted this commit, and put the x64 builds back in.


:warning: dxli is unstoppable

值得称赞的工作


Valeu mesmo pelas correções =) . Já estou trabalhando nelas;


I dunno. The other way is to use a string, but I don't know what string it expects for code page 1252, and I didn't feel like figuring it out