seems like a bunch of stuff snuck into this pr


Classes/Eat Now/Detail View Controller/MealTableViewController.swift

@natashaarmbrust, do you just hate me?


Can't you underflow here?  What happens if you go negative?


Actually that is the point of the implementation, to ignore those hidden directories. It should work as you expect it to work. I'll try to reproduce the issue and if it doesn't, I'll make sure that it does :)


If you place the empty struct at the end of another struct it can happen that you increase the size of it.
You should place it at the first position, AFAIK.


you mean having a `executable_not_found_msg`? Sorry, I don't understand.


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


don't you love `ebable`ing tests!


Classes/Eat Now/Detail View Controller/MenuViewController.swift

Remove all your print statements.


this got a little tricky with the `deletion` state being additive


Do you mind if I ask why are you setting them to `hidden`? I don't have context whatsoever (I landed to this PR by chance) but my gut feeling tells me that this is wrong. Every time you add new CSS you are signing a contract for life :) And `overflow: hidden` on the `body` feels not natural to me. Do you handle the page scroll with JS or does any of the descendant has `overflow: auto`? Both solutions are not as good as default browser scrolling imho


These two lines can be removed as well, as 'retain' is now unused. And that means the 'alreadyTracked' param can go as well.


As far as I understand there are two different things: GitHub account and git configuration, You may have logged in as yourself on GitHub but pushed the committed the changes without modifying the git configuration of the local repo.


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9: OK (397 of 397 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)


maybe throw on the `for more details.` at the end. I donno I go back n' forth on if it's needed or not. It's what I use on many of my links though.


Remove the leading # from the color to give it the same format as the rest of the routes.


I just remembered, I never did the files in the medical folder, so I'll do those either today or tomorrow, probably tomorrow.


the whole function, why isn't it just a regular function as part of the class


Just when I thought it couldn't get any better, IT DID!!!  Amazing stuff here...working on incorporating it all into the BibleVerses script.


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


It looks good


Kinda feel like we shouldn't support this at all since it doesn't completely undo `retire`. Missing stuff:
* restore deleted case indices
* rebuild cases
* restore phone numbers
* restore reminders


@bartlettr changed this so vote count matches value on slider -- yes?


oh, I thought about that. I should also move `check_command()`.


will do.


not really part of this PR - could move to separate PR


Also /cc @izaakschroeder :D


Thanks,

Sent from my iPad

> On 4 Mar 2016, at 20:14, computourist <notifications@github.com> wrote:
>
> You should check to see whether the pinout is OK. The signal names are
> included in the diagram. To display data you could use the LCD end node.
>
> 2016-03-01 7:55 GMT+01:00 jezturner100 <notifications@github.com>:
>
> > Hi, I'm trying to set up a remote node with a Pro Micro connected to a
> > rfm69hw, will the pinout be the same do you know? Also to recieve data how
> > do you convince a second identical pro micro-rfm69hw to display data
> > through sub in real time?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/computourist/RFM69-MQTT-client/commit/8beb3577e96b3407aa6b78611c1e8258fa994146#commitcomment-16410183>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


I see only one place where it is deleted, while in the second one you assign a new value after deletion. The second case is even simpler with a smart pointer. The first one also is not a problem, just a reset() call will do what we need. What is more important, in my eyes, is that the type of the smart pointer bears owning strategy information.


Typo in the fix!  segDimValues.  :-)


Pulled in via CLI. Thanks!


why do you need this change


@bcastanos thanks that looks awesome


Is there a prettier URL to use?


Nevermind, I just fixed it.


I actually did request a ticket for adding validation on email some times ago. It's in the icebox. I forgot about it too, because it didn't seem important until the major function of the haiku was completed.


`& 0xffffffff` is unnecessary. The unchecked cast to `int` will take care of it.


Oops merged instead of rebase... my bad.


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - x86_64-MacOS-10.9-clang: OK (2119 of 2129 tests passed, 205 test warnings)

 * 214 tests had: AUD diffs.
 * 10 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.53%25-yellow.svg)


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2157 of 2167 tests passed, 205 test warnings)

 * 214 tests had: AUD diffs.
 * 10 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.54%25-yellow.svg)


сделал все, как в гайде в геме
https://github.com/aishek/activerecord-sortable/blob/master/README.ru.md
выдает ошибку
No route matches {:action=>"move", :controller=>"users/lessons", :course_id=>#<CourseLesson id: 2, title: "ee", position: 0, description: "eee", lecture_notes: "ee", picture: nil, home_task: "eee", course_id: 2, created_at: "2016-03-04 20:00:31", updated_at: "2016-03-04 20:00:31">, :id=>nil} missing required keys: [:id]
вроде бы проблема в урле, но немогу понять в чем дело, вроде все правильно указанно?


shouldn't this be two spaces to the right?


@JoseaugustoSReis Terminar a Descrição das Avaliações


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-5: OK (397 of 397 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)


Odd, do you have Issues disable on this repo? I can't seem to open one.


The following doc(s) have been deployed to the [1390codea](http://1390codea-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/backups/backup-creation](http://1390codea-static-docs.pantheon.io/docs/articles/sites/backups/backup-creation)
- [docs/articles/sites/backups/cloning-an-existing-site-from-a-dashboard-backup](http://1390codea-static-docs.pantheon.io/docs/articles/sites/backups/cloning-an-existing-site-from-a-dashboard-backup)
- [docs/articles/sites/backups/restoring-an-environment-from-a-backup](http://1390codea-static-docs.pantheon.io/docs/articles/sites/backups/restoring-an-environment-from-a-backup)
- [source/docs/assets/images/desk_images/169628.png](https://github.com/pantheon-systems/documentation/commit/e26c53ae4f74df8cf6455543524831d1df1387ea/source/docs/assets/images/desk_images/169628.png)
- [source/docs/assets/images/direct-download-archive.png](https://github.com/pantheon-systems/documentation/commit/e26c53ae4f74df8cf6455543524831d1df1387ea/source/docs/assets/images/direct-download-archive.png)
- [source/docs/assets/images/commit-id.png](https://github.com/pantheon-systems/documentation/commit/2202eb017d258cb329c299f90ad14cbdc0d80fff/source/docs/assets/images/commit-id.png)


On some versions of GDAL, the RasterIO function is marked as warn_unused_result, which leads to a compilation error with the change you've suggested.


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


I found out via Fishbench that !(b & pos.pieces(Them))) is more efficient than !(pos.pieces(Them) & b)). So I will include this in take 3.


done


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (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)


lgtm


@sils1297  All the names are the same names that would be stored in a ``.jshintrc`` file. All of them are valid options for jshint. Please take a look [here]( https://github.com/jshint/jshint/blob/master/examples/.jshintrc). So i just used the same name, its easier for users.


some parens here might make it easier to reason about the operator precedence.


you can make set one up on the lodash.com dns =D?


@obigriffith here're some updates.


it is enough to define one field in item in parse(), and leave all others to detail_page_parse().
but it is ok to do like this.


Ship it


f*ck le parametres exp_label


Paired-end mod only works with BED file, but it's more often to get failed
because of "too many iterations, bad library" than not use it.

On Fri, Mar 4, 2016 at 8:36 PM, Phil Ewels <notifications@github.com> wrote:

> Thanks @jun-wan <https://github.com/jun-wan> - does running preseq on BED
> files stop the paired-end mode from failing or something?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ewels/NGI-Nextflow/commit/d29e5d5b41a11751b814c8d9679c3e5b353f09d3#commitcomment-16508763>
> .
>


``shadow`` can take a value ``false`` or string values like "inner" , "outer". So I thought shadow is better


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


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


Yes, we can do that, maybe later when we take a look at the xfce profile with the new themes, etc.


so the goal of coala is to make stuff simple and unified, so if two linters have settings `use_spaces` and the other has `use_tabs` in coala they'd both be exposed as `use_spaces`. The user shouldn't even need to care about how JSHint works so we should just expose intuitive names and that's it.


I'm not quite following how the `instanceof` comes into play here?


Can this please be done via settings file? oc.nl does like it how it was before, 1 titled_cache on main site.


Use 24 hour time-- see downloaded CSV for reference


Do we want to clear `this.properties` here as well? Right now they cached.


This message needs to die a horrible death.  It is spamming Kibana by a factor of 2x everything else.


indentation here is a bit confusing


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 12](https://teamcity.labkey.org/viewLog.html?buildId=307906&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (1 new), passed: 39, muted: 4 Build time: 01:25:08
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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

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

```


FUNCIONA BIEN LA PARTE DE LOS RECOMENDADOS!!


Okayl :+1:


update_travis_toolchain (lefticus) - 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)


update_travis_toolchain (lefticus) - 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-warnings-yellow.svg)


Sorry I didn't do a great job with that one.


You keep overriding the travis configuration for the HL branch which is supposed to only test HL. When you come across a merge conflict just go for "use mine" on that one.


Is this WE 1.9?


Looks good.  I came to about the same end in commit d43d4d003c97c80333b9804f99ce842815c7ff0e. My only nit-pick is that perhaps `finish_ctx` could be named `err_finish_ctx`, to drive home the point that this is an error-exit label, and not a common exit-point.  (I THINK that's what Dan would say.)  I've tried to get into the habit of `out_*` for common exit points (success or failure), and `err_*` for error exit points.  Do you think that's a reasonable convention that is accepted by the upstream?


возможна ли ошибка из-за многоуровневости в роутах?


I don't think this is the right way to guard against it, I think we'll need to go to the docs and find the error code and do all the casting with `awserr.Error` etc.

Otherwise here we'd be just wiping the resource on _any_ error.


I didn't include it in the commit message :tired_face: but the reason I removed this is because the `production` flag no longer makes sense since the `sass` and `javascript` tasks now build two copies of their source files. A compiled / bundled version and a compiled / bundled minified version. :tada:


Losowość taka średnia, cpp11 daje fajne możliwości poprawienia tego aspektu:
http://www.cplusplus.com/reference/random/


std::vector > dynamiczna tablica []


Raczej `inner` niż `inside`


Thank you for this addition. Made integration of auth0 far simpler than the approach that I thought I would need.


Looks better, thanks!


Usuń-usuń-usuń-usuń tooo


##### Changed zoom, turned off cartodb logo


### Deleted a shitton of commented lines


Is this safe to use yet?


Można obyć się bez tej zmiennej na conajmniej 2 sposoby:

1. używanie std::swap
2. swapowanie zmiennych XORem


Dlaczego liczysz od 1 a nie klasycznie od 0?


Indentation here is messed up


Do you have a better name for this flag? I couldn't think in anything better than that


Ranged-based loop:
http://en.cppreference.com/w/cpp/language/range-for


Addresses #29


@Morfeusys Подскажите как обновить модуль KeyDetector.java у себя на локалке ?


newline


yeah let's just say this is indeed a terrible idea. :-1:


you're correct. The corresponding error would be `NotFoundException`. I'll update it for the other resources soon.


This should be just `return unchecked((int)m_value)` to avoid unnecessary conversion to long.

(Sorry about the nitpicking ... trying to make the sure that it is 100% right. I hope this is the last one.)


@UUSR Подскажите, зачем комментить исходный код, чтобы задать этот вопрос? )) А обновить просто - собрать из исходников.


`n` => `size` - to nie boli a poprawia czytelność


No this is WE 6.1~, with an adapter to handle copying entities and tile entities and changing biomes on bukkit-based 1.9 servers.


> On 4 Mar 2016, at 7:01 pm, Xl-0rd <notifications@github.com> wrote:
>
> Unfortunately "EnableCloseButton(false)" does not stick: If the window is zoomed to full screen, the close button is enbled again. Is this a OS X bug or is there something wxWidgets can do?
>
> —
> Reply to this email directly or view it on GitHub.
>

EnableCloseButton( false ) doesn’t seem to work when used before or after the window is shown fullscreen. The current method accesses the button to disable it.

Not specifying wxCLOSE_BOX works and removing NSClosableWindowMask from an already shown window using setStyleMask works but wxWidgets doesn’t provide a means to do this.

Regards, John


Is there a reason why appendix can't be used instead? As per MinecraftForge/MinecraftForge#2371
It might even be a good idea to just put this in the default build script so devs who don't want it can remove it.


hey would you mind changing code to java code.


1901_radiance_translator_reporting (macumber) - i386-Windows-7-VisualStudio-12: OK (2361 of 2395 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-98.58%25-yellow.svg)


we could replace realloc with free+clear+malloc, but this would do free and clear every time we need to increase the size of the buffer. I don't think this is a good approach too.


There is no need for this on the older Frsky Telemetry.
There is a Cell voltage calculator built into OpenTX that has been used for FLVSS.
See [OpenTX 2.1 & FrSky FLVSS Individual Cell Voltages](http://openrcforums.com/forum/viewtopic.php?t=7266).
> Add a new sensor.
>
> To display the lowest cell voltage set it up like this
> Type: Calculated
> Formula: Cell
> Cell Sensor: Cels _(pack total voltage)_
> Cell Index: Lowest

However, I have tested this method and it does not work for VFAS when using SPort Telematry, not sure why, but it looks like cleanflight is not sending the individual cell data. See: [Smartport telemetry for Cell voltage(VFAS / # of cells)?](https://github.com/cleanflight/cleanflight/issues/1268)


"Disable creation of desktop shortcut"
Start menu shortcut is still created


These should go in the UserConstants file.


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2125 of 2135 tests passed, 205 test warnings)

 * 214 tests had: AUD diffs.
 * 10 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.53%25-yellow.svg)


Niepotrzebna pusta linia


Since when has a `mod-` prefix ever been needed?


Addresses https://github.com/alexandertuna/MMFE8Readout/issues/2


I do like the single-end point. Less boilerplate for the end user, but one thing that would be great if we'd add a static function that takes an existing instance of  `Sabre\Xml\Service` and sets up all the mappings. That way you can have inheritance, but also composition if you need it.


Instead of:
`int err = 0;`
you now only need:
`int err;`
Fine other than that.


Nawias kończący ma o jeden poziom wcięcia za dużo


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1579 of 1579 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.85%25-green.svg)


I'm totally into this idea. It's perfect for a meeting, it applies to everyone, and lots of people can give input.


2097-EP-8-5 (kbenne) - 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-NA-red.svg)


typed_function_ordering (lefticus) - i386-Windows-10-VisualStudio-14: OK (390 of 390 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)


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1021 of 1021 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-30.56%25-green.svg)


2097-EP-8-5 (kbenne) - 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-NA-red.svg)


OMG! I swear I have already replied here. Where's my comment?
So...
>but I am still hesitant to kill it

Why? What's the problem?
`datasource.revision` is `undefined` all the time, so expression `$scope.$watch(datasource.revision, () => reload());` does nothing but calls `reload()` via `$evalAsync`


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (398 of 398 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)


Why not delete the guide itself? This will just get added back to the list of unpublished because the guide still exists.


On 04-03-2016 17:52, Otávio wrote:
> we could replace realloc with free+clear+malloc, but this would do free
> and clear every time we need to increase the size of the buffer. I don't
> think this is a good approach too.

But only if the flag is set, user shall know that it comes with a cost.

> —
> Reply to this email directly or view it on GitHub
> <https://github.com/otaviobp/soletta/commit/93a7d3e71d7f0045fc9c1bb4622c2ce44dfd24cd#commitcomment-16510360>.
>


Good idea !


This seems to have broken the regression suite. While I think we should eventually remove this, I don't think it can be just removed so easily.


komentář


Там просто именование такое же)
Да, ты правильно поняла что название показывает ее назначение и без использования транслита)


I do not think this is a good idea to have this as a default setting. AFAIK, you can always get the same result by passing those flags when configuring the CMake build on `cmake` command line.


wow


Sorry, I worded it wrong. I meant to ask if it's WE for 1.9 spigot/bukkit?


perfect :+1: :-)


Ops... OfflineError is not declared


awesome commit description


Мы там указываем ссылку на Layout и далее она присоединяется автоматически, если не указанно обратное.


I don't think this line of documentation is necessary,


`statues` typo? Or are they actually statues? :statue_of_liberty:


Sorting works, except for float values:

```python
 /home/jhunk/miniconda3/envs/pandokia/lib/python3.5/site-packages/pandokia/text_table.py in __lt__(self=<pandokia.text_table.text_table_row object>, other=<pandokia.text_table.text_table_row object>)
    120             r = 0
    121             for x in self.sort_order :
=>  122                 if self.lst[x].sort_key < other.lst[x].sort_key :
    123                     return r - 1
    124             return r
self = <pandokia.text_table.text_table_row object>, self.lst = ['', 'regtest', '', 'Y', 'user_jhunk_2016-03-04-15-31-32', 'stsci_python', 'nott', 'default', 'pyraf/cli/division/test', '', '', 'E'], x = 10, ].sort_key undefined, other = <pandokia.text_table.text_table_row object>, other.lst = ['', 'nose', '', 'N', 'user_jhunk_2016-03-04-15-31-32', 'stsci_python', 'nott', 'default', 'acstools/wfc_acsccd.test_fullframe_1', '', '15.166', 'P']
TypeError: unorderable types: str() < float()
      args = ('unorderable types: str() < float()',)
      with_traceback = <built-in method with_traceback of TypeError object>
```


Yeah, I noticed that bug too in commit d43d4d003c97c80333b9804f99ce842815c7ff0e.  I fixed it differently, but your way is MUCH better.   My only comment regarding this code is I believe Dan would suggest that `away` be renamed, perhaps to something like `out_respond` or `out_respond_and_notify`.


"Bukkit-based 1.9" -> 1.9 spigot

On Friday, March 4, 2016, Druxe0 <notifications@github.com> wrote:

> Sorry, I worded it wrong. I meant to ask if it's WE for 1.9 spigot/bukkit?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sk89q/WorldEdit/commit/de3578137b90a83e7b9322c6792fb0a7482244af#commitcomment-16510510>
> .
>

--
~Kenzie Togami


correct! nice catch.


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (390 of 390 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)


hahahahhahaha

é o id que eu separei para o professor. Ficou de forma bem porca, mas é
basicamente uma conta que não pode submeter avaliações. É a forma
provisória de mostrar o site para os professores, enquanto não fazemos a
página dos professores.

2016-03-04 16:13 GMT-03:00 Rodrigo Surita <notifications@github.com>:

> Quem é user_id 78?? hahah
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/faraj7/gda/commit/4fe583fb0868bc1e92b5b1665dc81095b9a50a70#commitcomment-16508212>
> .
>

--
*Raul Guidolini Cecato*
Graduando | Engenharia Elétrica Unicamp

Equipe Phoenix de Desafio de Robôs

*Coordenador de Patrocínio*
*raulcecato@gmail.com <raulcecato@gmail.com>*
Tel: (19) 9-8346-8011


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/c07f4136943629acfa0f7a4fcf19161be26699c1/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/c07f4136943629acfa0f7a4fcf19161be26699c1/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/c07f4136943629acfa0f7a4fcf19161be26699c1/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/c07f4136943629acfa0f7a4fcf19161be26699c1/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/c07f4136943629acfa0f7a4fcf19161be26699c1/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/c07f4136943629acfa0f7a4fcf19161be26699c1/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/c07f4136943629acfa0f7a4fcf19161be26699c1/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/c07f4136943629acfa0f7a4fcf19161be26699c1/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/c07f4136943629acfa0f7a4fcf19161be26699c1/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/c07f4136943629acfa0f7a4fcf19161be26699c1/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/c07f4136943629acfa0f7a4fcf19161be26699c1/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/c07f4136943629acfa0f7a4fcf19161be26699c1/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/c07f4136943629acfa0f7a4fcf19161be26699c1/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


5.5.33 is also released :smile:


Fixed


This is really great stuff, it adds a lot to your game.


testin difference


I could. What's the best way?


That looks a lot better! :+1:


It looks like you can remove `rc`:
`int rc = CONTROLVM_RESP_SUCCESS;`
Also, wouldn't
`return 0;`
be better written:
`return CONTROLVM_RESP_SUCCESS;`


The checksums for `4.3.2` and `5.7.1` are missing


NIT: something went wrong with sorting here


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1021 of 1021 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-30.56%25-green.svg)


@wilx well it can be disabled by default but enabled with

`option(MINGW_STATIC_RUNTIME "Link with static stdc++ library" OFF)`
`if (MINGW AND MINGW_STATIC_RUNTIME)`

Well, in general, it's not good idea to manipulate linker flags through CMake command line.


Yep, this is good to go.


so a chain of comments can be made on a deleted file


why, yes...


@kerezov, just a quick questions here:
Is there a reason for the "chess.js" and "passport-local" dependencies to have hardcoded versions instead of relaxed versions using ^.


Ось починаючи звідси


@adrigrassi como primeiroEnvio() joga uma Exception por sua assinatura, seria mais correto capturar uma Exception ao invés de um Throwable. Quanto mais específico, melhor :).

No caso, como o código do catch apenas joga a exceção novamente, ele é prejudicial, pois aumenta a stacktrace, dificultando a leitura e poluindo ainda mais os logs, o que atrapalha na hora de identificar um problema.

Nesse caso, o bloco try-catch pode ser eliminado, sem prejuízo à semântica do código e eliminando o fator prejudicial citado acima.

[]'s


Perfect, thanks!


Agreed. I'm not sure which works out better though at the moment (Conn or Client, both?), but I'll be needing that anyway for the challenge/ssl support on the next PR. I will fix the overloaded method up later this evening.

Are you in agreement on what was done in regards to the _type_, using the class name rather than strictly mapping the default type (and all the changes throughout that are associated)? The original test ExtendedUser.TestExtendedProperties() and the new test CustomNames.CustomIndexAndTypeName() demo that. I think I need to look at that further, but just wondering what your thoughts are there.


І кінчаючи сюдою, можна було просто копіювати


The change in line "4" is incorrect. Old translation used to say "The missing package manager...", now it says "The manager of missing packages..."


What if Manage slave User is False and Manage Client Jar is true


Why not just put the icons in the standard location:

ZenPacks.zenoss.LinuxMonitor/resources/icon/LinuxDevice.png ??


One option could be to move the `retire` operation completely into a celery task that get's delayed by 1 minute during which time the user has the option of 'undoing' (cancelling the task).

If un-cancelled after 1 min the task starts and the operation is permanent.


Hey, can you please make sure to update docstrings when copy-pasting functions? It's confusing when I notice a new function has been added, try to figure out why / what it does, and see the docstring from another function.


我真佩服你。我就在一个美国大学里研究p2p平台网路。你可以告诉我叫一个
项目文件的办法是什么？如果一个“description”的文件叫2123，“record”的文件就叫2123吗？如果不是，请你可以告诉我号码表征色呢么。

因为我要把records 比作description，所以我打算检查交易的数量、号码等等。

谢谢你。


I see, sorry! Want me to revert it until we can fix the rest?


Hi Randy, thanks for this. When I used the above SQL code, the test returned a syntax error:
```
Failure/Error: expect( Claim.claims_to_remind.count ).to eq(0)
     ActiveRecord::StatementInvalid:
       SQLite3::SQLException: near "FROM": syntax error: SELECT COUNT(*) FROM "claims" WHERE (EXTRACT(days FROM now() - created_at) = 7)
     # ./spec/models/claim_spec.rb:36:in `block (4 levels) in <top (required)>'
```
It seems like somehow when "Claim.claims_to_remind.count" gets parsed, "count" becomes part of the SQL query and that's what's causing the error. To be honest I haven't used a lot of SQL in my apps and though I've spent some time trying to modify the query and get it to work, I have not yet figured out how to address it so that the test passes. I'll keep looking into this and see what I can find :)


I see! I was poking around:)

Sent from my iPhone 6

> On Mar 4, 2016, at 12:58 PM, Luke Lee <notifications@github.com> wrote:
>
> Why not delete the guide itself? This will just get added back to the list of unpublished because the guide still exists.
>
> —
> Reply to this email directly or view it on GitHub.
>


Empire - The Last Battle/Assets/Unity/Scripts/MonoBehaviours/BattleUnitPositionManager.cs

Units shouldn't be removed (i.e this function shouldn't even be called) if there's no active units, so it's right to throw an exception.


`.sheet` is a read only property, so you need the `_set` version.


> well it can be disabled by default but enabled with (...)

OK. Submit a patch which implements this instead and I will accept it.


@cprice404 A TreeWalk needs to enter a subtree in the following cases:

* An entry in the work tree has TREE as its File Mode, and it exists in the index with File Mode TREE
* An entry in the work tree has TREE as its File Mode and does not exist in the index
* An entry in the work tree has GITLINK as its File Mode, but it exists in the index with File Mode TREE.

Basically, in the third case, we need to enter a `GITLINK` entry seen in the working directory. This `instanceof` ensures that we will treat a `GITLINK` as if we were given a file mode of `TREE`, but only if that `GITLINK` is in the working directory (and thus was found with a `FileTreeIterator`). If a file has mode `GITLINK` in the `DirCacheIterator`, it has mode `GITLINK` in the index, and thus should be treated as a `GITLINK` rather than a `TREE`.

This `instanceof` should be safe, as if a `GITLINK` is seen in the working tree and there's no entry for that directory in the index, this `enterSubtree()` method will not be called.


This is giving me an error this morning.

![2016-03-05 07_42_00-photos](https://cloud.githubusercontent.com/assets/8046697/13540348/d390bbca-e2a5-11e5-8386-6280948f6f3f.jpg)


Yeah, go for it!


Thanks Tim, I'll update this with a new patch.


This also makes sense to me, and the results show the fix.


Гарна робота, але тепер виникла проблема з відправкою даних на фрагменти


one can dynamically increase disk size where data is stored; does this change imply that exist-db must be restarted to continue?
Additionally.... is `getUsableSpace()` stable in use? we had a change here after 2.x right?


Well.. You have a point here.


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3048 of 3048 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-100.0%25-green.svg)


wystarczy samo
`return (a>0 && b>0 && c>0 && (a+b)>c && (a+c)>b && (b+c)>a);`
wtedy zwiększa się czytelność i funkcja skraca się do jednej linijki


i'd prefer to keep the isReadOnly pattern ; it makes the if-statements more readable (personal opinion)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18363](https://teamcity.mesosphere.io/viewLog.html?buildId=154559&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:04


`os.wait()` returns a 16-bit integer value. Here we are separating the high and low bytes instead of returning the _mask_.

See also: https://docs.python.org/3/library/os.html#os.wait


We'd love a pull request!


@wloczynutka Here is one more item that needs to be configurable on a per node basis.


UnitConvInIDDforIDFed (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2127 of 2127 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)


From the looks of things, I'd say `pdk export` has not been used for anything in a while.


typed_function_ordering (lefticus) - Win64-Windows-10-VisualStudio-14: OK (390 of 390 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)


However you decide to fix it, be careful not to remove an optimization here just to make the code look a little simpler.


Also fixed two other issues:
- last one in the set wasn't index, but actual label; this was wrong since it was meant to collect indices
- didn't know what `labels[order(indices)]` actually did, now I do. I ordered them weirdly instead just selecting them in traversed order


may want to switch 30 to 15.

Additional notes: Make the counter max a variables to change depending on how long the button is pressed.


Great suggestion, Scott.


`path` is already available, brought in [here](https://github.com/AtomLinter/linter-flake8/blob/a1e00223d3f633f9848421933d6973e240f8ab18/lib/main.coffee#L175).


Eae, Max! =p
Posso dar dicas? (Perguntei, mas já vou falar xD )

Essa parte aqui deixa a gente sem ar. Faltampontos finais. Muita vírgula.
rsrs
Dá uma revisada na estrutura textual. Só isso. O Resto está massa!


I agree with @adamretter , the proposed code is too much magic (not transparant); and can be done with other xquery functions


ListOfPets для класса не лучшее название. Лучше PetsFactory


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: Build Failed

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


Valid observation, this need to be converted into a before relationship from the user resource.  Do you want to take a stab at it?


thanks, it was a mistake :)


Ты была близка, но этого тоже не должно здесь быть)


Hey man!
I'm really sorry about this haha. It seems that you somehow got added to this repository's watchlist. Probably due to us having the same name, really sorry about that. But our admin can't seem to do anything about it, he says you can take yourself from the watchlist on your end. If you need any help feel free to email me personally, or our admin.
My email is: antoniocarrion11@gmail.com
Admin email: kgary@email.asu.edu

> On Mar 3, 2016, at 1:14 PM, acarrion <notifications@github.com> wrote:
>
> Hello? Can you guys see my email responses?
>
> I don't know who you are or what this project even is, but I keep getting
> your pull requests and comment notifications in my email. I don't think I
> even have an active Github account. Can someone please remove my email
> from your project?
>
> On Tue, Mar 1, 2016 at 7:30 PM, atta2 <notifications@github.com> wrote:
>
> > Arron Helped me finding the block of code.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/cst316/spring16project-Basic/commit/6e51933d49e114d13936441580638420f92ab101#commitcomment-16435959>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


add `event.preventDefault()` on the first line in this function (and also in the `lessSlide()` function); the reason why you don't see these functions working is because the browser is attempting to redirect the user to another page (because they clicked on the `a` link).  to prevent this from happening, we use `event.preventDefault()`


Стили лучше помещать в Content/css


contrib/codeanalysis/runcppcheck.sh: line 9: ../cppcheck-1.66/cppcheck: No such file or directory


typed_function_ordering (lefticus) - i386-Windows-10-VisualStudio-12: OK (389 of 390 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-99.74%25-yellow.svg)


@MichaelChirico What does this line do?


`connection` or `configuration`?


>  I fixed a dangerous behavior with memoization in case of reconfigure

Cool; I didn't really test that much

> Also, do you want to support multiple configure blocks updating the same config? because I removed that possibility but it looks safer this way to me.

i'm not sure what you mean?


If there's a calculated height that is based on the width ratio as a maximum, why would we then be using the media_height (which will be much higher in many cases - e.g. if the image is 1080p resolution) instead of the height, as it was before?

I'm just trying to understand the reason for some of this change and whether I missed something to see if I need to tweak my "crop at 4:3" code.


Warmup which requires us to display light sensor readings using the color sensor.


Can you document what the return value should be here? Does `check_init_y() == True` mean that y has been checked and is OK, or that it contains NaN?


Confirmed, it does break... wish I had a 10.6 machine to test with. See notes on #1972


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-NoThreads: OK (394 of 397 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-99.24%25-yellow.svg)


My commit message got autocorrected to "false" rather than "falsy" just FYI


This change seems to break the test on OS X: http://ci.ros2.org/job/ci_osx/871/testReport/


Yup!


Error returned on wrong line, should be returned on line 105.  Line 108 already correctly sets error to be returned here.


jjj


Probably want to prefix with a semicolon.


Trying to create another pull request


Valid point. I can keep the `isabs` as that just checks the path is not relative, I will drop the isdir (as that actually checks the dir exists)


oh, nvm.


@VerticalDynamics duuude


I did css stuff for the process svgs. Can someone who has access to ie11 test the localization branch for that ? Anyway the process images are now inline svg, so this will need to be selectively applied for the non inline images...


@VerticalDynamics try to write some sass instead of inline styles. You might like it!


the code does not work so far. any idea?


@dizzzz No dynamic disk space increase has never been possible; Feel free to implement ;-)
I am not sure what your comment about `getUsableSpace()` refers to?


You got these flip-flopped.  Should be checking for `SERVER_PROCESS` here.

It's Friday.  :stuck_out_tongue_winking_eye:


Empêche le fichier d'être définitivement supprimé lorsque -xs et -r sont employés ensemble... :rage:


chinese LOL


I agree, but I prefer to follow convention. None of our other booleans are prefixed `is` and this is more generally an unusual thing to do.


typed_function_ordering (lefticus) - Win64-Windows-10-VisualStudio-12: OK (389 of 390 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-99.74%25-yellow.svg)


Sure, will be my first commit to a public project :+1:


Yeah, leaning on the fact that we initialize variables to nil, so the first assignment of a variable to itself isn't a problem. Perhaps there's a better way to fix this? The alternatives I came up with were more complex.


@VerticalDynamics instead of "align='center'" use "class='text-center'"


Why ?


Shall I rebuild binaries ?


Если просто открыть твой сайт сейчас, без указания пути к контроллеру, то вылетает ошибка. Это происходит потому что у тебя неверно указан путь по умолчанию. В случе твоей задачи это может выглядеть так
 defaults: new { controller = "Home", action = "English", id = UrlParameter.Optional }

И тогда по умолчанию будет открыватся эта страница


@VerticalDynamics I thought the footer text was supposed to be blue?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#ccDuWGDzS6W_Okx4nygjPQ


It looks like this commit move the target of the "tab link" rendering into TabLinks. Am I right about that?

I have unfortunately be preoccupied with QA test issues today, so I haven't end a chance to go back to the usage and definition of TabLinks.

But, if I *am* right, then _"TabLinks"_ may not be the best name for the component. If we're encapsulating a "tab link bar with the _target_ pane" for display - we might want to consider another name. I think we definitely want a "TabPane" sort of components like this and have it as reusable because there several places we'd leverage it.


This can be fixed ?


152: matches the `class` of each column of `x` to its abbreviation given by `class_abb`. Needed `unname` lest the names be kept around in the `rbind` step at 157.

153: `class_abb` doesn't exhaust the list of possible `class`es that might be stored in a `data.table`, just what I deemed the most common ones. If a column is of a `class` not found in `class_abb`, we print the full class name (or the full first element of `class`, if `class` is length-2 or more).

This is similar to `dplyr` which (IIRC) truncates unknown class names to 6 characters but otherwise prints them as is.


Can we actually make this "Loading secure payment portal"? Thanks!


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


ok :+1:


Название должно отражать суть. В данном случае класс ContactsCreate звучит как то странно. ContactsFactory лучше


No because I am a nub. I welcome PRs though!

I'm trying to get basic support of logging these things, and rudimentary rollbacks at the very least.


Общее замечание: описывай формат ввода.

Задача 1: исправить
![image](https://cloud.githubusercontent.com/assets/17623743/13540635/baeccda8-e268-11e5-9a8a-aab480758557.png)
Задача 2: все ок
Задача 3: внимательнее читай условие. нужна проверка вводимых значений
![image](https://cloud.githubusercontent.com/assets/17623743/13540874/1bb6e7bc-e26a-11e5-9a08-95772a4ee32b.png)


`fs.writeFile` can take an options object with a mode key:
```
{
    mode: 0o600
}
```


This was already discussed.


Probably want to get rid of this?


Never, but I've always recommended it.


Code style, it should be :
> for (i = 0; i < PyTuple_Size(list); ++i) {

Also i prefer create `i` in the loop and use a cached size:
> for (unsigned int i = 0, size =PyTuple_Size(list); i < size; ++i) {
But is not very important about cached size...


Well we just handled it with explaining the devs how to use "chef exec bundler exec" to handle it. But just saying out of the box (fresh Chef DK setup) the Chef DK is broken with it.


Histogram of Daily Step and time series of average number of steps


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


What is appendix?


Eu cadastrei um novo usuário mas deu um erro de imagem tava pedindo a API Key do aws.
Ai eu criei o usuário sem foto e deu certo. Mas quando faço login da outro erro de imagem. Esse erro aqui: CloudinaryException in Events#index


#5518-Multispeed-Coils-used-with-AirloopHVACUnitarySystem-do-not-control-air-flow-properly-at-Speed-=-1 (rraustad) - x86_64-MacOS-10.9-clang: OK (2116 of 2124 tests passed, 332 test warnings)

 * 334 tests had: AUD diffs.
 * 334 tests had: BND diffs.
 * 7 tests had: EIO diffs.
 * 6 tests had: ESO small diffs.
 * 2 tests had: MTR small diffs.
 * 16 tests had: MTD diffs.
 * 39 tests had: RDD diffs.
 * 2 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 6 tests had: MTR big diffs.
 * 6 tests had: Table big diffs.
 * 2 tests had: Table small diffs.

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


A better option is to take a JobTypeData param. The assert in finishJob could call param.type()
Pass in 'data' from processTask, which we already have.


Add braces even for one line:
> if (type == CPROP_TYPE_INT) {
>      cprop->data = data;
>}
> else ....


Also, I'm desperate for this to be merged so I can tell people they can finally used LL 1.8.9. This PR has been here for **3 days**. Why the hell are you commenting **now**?


Code style:
> if (cprop->poin) {
> MEM_freeN(cprop->poin);
> }


Looks like the end tags have been missed. There's code for 3 feats in the Observant bootstrap. Will post fix ASAP.


jesus christ


How can it happen on connect?


Correct you need `]]></eval>` at the end of the 2nd script.  Plus don't need to check the disable tag twice...


I'm not sure how much I like mine better and I think this just needs to be rewritten.

I'm not sure now why we even need the notifiers when we have visorbus and visorchipset being the same driver.


It doesn't request a error message ? It is an error ?


This is why its best to "test" ALL changes inside of HL first!  Unit Testing is your friend. =)


Everything looks good.  No comments.


You're right. I must have wiped it accidentally. I'll add it back.


Extra lines


Will do from now on


Thanks :) I really like your solution @david-perez.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#NWHmvt3aS72aWZkqeb7TYQ


This debug message should be removed.


Good day Marcel,

I am not sure what is causing the crashes and it doesn't always happen - so I can't reproduce them everytime but it has become more frequent lately (maybe since I upgraded to Jarvel release?).

but the last log message always is:
13:30:43 T:6536   ERROR: CPythonInvoker(2, C:\Users\Mario\AppData\Roaming\Kodi\addons\script.skin.helper.service\service.py): script didn't stop in 5 seconds - let's kill it

and always seems to happens when starting up Kodi and an update of the skin and helper are available and in progress...

![2016-03-04_13-33-30](https://cloud.githubusercontent.com/assets/6510026/13540917/1fcd04a0-e20e-11e5-98d8-8a8bc247887c.png)

I think that 5 seconds is not alot of grace time to allow a script to end on a startup of Kodi for an attempted upgrade but maybe that is what is causing the crash?

thoughts?


Awesome, I was about to add this!


#### This is a readme file.


Horrible code style here....


It might be worth having getNextJob return its JobTypeData and use it here instead of re-creating it. The local 'type' could be removed.


hey


Fancy!

And since I don't know ruby, this could do anything and I would approve it in review! :)


Appendix appears right before the version. so `mod-modname-mc1.8.9-1.0.3` and mc1.8.9 would be the appendix.

And I'm commenting right now because I forgot about the PR until now and I had some free time. I figured that I should voice my opinion before it's too late.


I agree that a lot of the reasons for having notifiers in the first place has "boiled away" over the course of many simplifications since going to the upstream.  I think simplifying this area is a definite "to-do" once we get out of staging.


Thank you


Hi @sudhirkhanger shouldn't the padding be 24dp like you mentioned in the material design reference? :wink:


@gaearon is this OK? I am not sure about the environment names. The 'native-classes' one omits a few other transforms - templates, arrow functions etc.


Thank you! :grinning:


even a small one is a good one


This is needed for what ?


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (398 of 398 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)


@broneks haha, I figure you'd have something to say about that :) It's a work-in-progress but it achieves the right in-your-face effect while not taking away from the horizontal and vertical symmetry of the two blue column boxes.


It tells the option parser to stop scanning for options when it sees the first non-option parameter is found; aka POSIXLY_CORRECT. It's almost always the desired behavior and is particularly important in this situation since when typing something like this `fish script -h` you almost always want the -h to be passed to the script and not interpreted as the help option by the shell.


It's a bit vague name `import`


Fixed locally (self spank).


зачем статические переменные?

зачем рекурсия?


Add an explanation to the commit.


lgtm


Fixed #232


Code style .....


Thanks! :smile:


SQLite may not have extract. I generally recommend using the same system for both production and local development so you don't have to change SQL queries between environments but for this case you could do:
```if Rails.env.production?
<fast query>
else
<sqlite query>
end```


This is great, but I actually meant something slightly different here (Apologies for not being precise), but I don't think it's worth putting you through the code change. Something like:

```
{% for credentials in cfg['static_slave_credentials'] %}
   <com.cloudbees.jenkins.plugins.sshcredentials.impl.BasicSSHUserPrivateKey plugin="ssh-credentials@1.10">
      <scope> ....
       {% for key, value in credentials.iteritems() %}
       <{{ key }}>{{ value }}</{{ key }}>
       {% endfor %}
   .....
{% endfor %}
```

Just because then we could add new keys by just changing the yaml.

In reality though, these two repos so tightly coupled I don't think it saves that much time to guard for that now.


is_admin var demiştim zaten onu eklemiycektin.


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


saw approval from JeetKunDoug
at https://github.com/basho/riak_core/commit/36461661b3eefa49f2b7dd6b9fab81be0e460ee5


merging basho/riak_core/basho/bugfix/riak-2383-alt = 36461661 into borshop-integration-821-basho/bugfix/riak-2383-alt


basho/riak_core/basho/bugfix/riak-2383-alt = 36461661 merged ok, testing candidate = 24012f4b


verbose_name=_("Yazar") şeklinde olmalı. "_"'yi unutmuşşsun.


verbose_name=("Yazar") şeklinde olmalı. ""'yi unutmuşşsun.


in which case appendix is not what I want, because litemod convention has always been `mod-<name>-<version>-mc<mcver>.litemod`


hey @kaapstorm did this work for you? mine doesn't seem to add properly


Klüp Üyesi şeklinde yaz. Türkçe karakter kullanabilirsin.


I would like `npm test` to run both suites just for safety, and have `test:watch` running the more common one (Babel). As for the names, I don’t have a strong opinion as long as both of them are explicit about what they mean. Something like `babel-es2015` and `native-es2015` (or variations) could work too.


What is it ?


Can we just pass pod IP to this function? SyncPod() has enough information to get the pod IP.


Remove this


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


for consistency i will change the loading image, i used for this code. That is because i could not found the blue spinner we are using in many places in the project


I know the forge guys always just append it to the version i.e. `version = '1.0-mc1.8.9'`


oh I see ... the word event is still there on line 25


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18365](https://teamcity.mesosphere.io/viewLog.html?buildId=154572&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 250 Build time: 00:16:16
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


add `#define WITH_GAMEENGINE` here an later.


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


leave the periods on the titles. it looks cool


virgülden sonra bir boşluk koyman lazım.


```
$iterator = $collection->getIterator();
$iterator->uasort(function ($a, $b) {
    return ($a->getPropery() < $b->getProperty()) ? -1 : 1;
});
$collection = new ArrayCollection(iterator_to_array($iterator));

```


Instead of doing this, SyncPod() can get the information by
 1. If infra container was just (re-)created, inspect the container and get the container IP
 2. If infra container was created, get it from [kubecontainer.PodStatus](https://github.com/kubernetes/kubernetes/blob/402e44d26ec52729d28dff3604cbfb22085a7081/pkg/kubelet/dockertools/manager.go#L1752).


typed_function_ordering (lefticus) - 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)


typed_function_ordering (lefticus) - 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-warnings-yellow.svg)


Yes but this makes it easy for people to remove the suffix if they don't want it without mucking around with the version. The whole reason I used classifier is that people might want to set it to something else, or set it to blank afterwards.


This function used to return the strands used in the payment. I changed it so it now returns the result.


org.abstractj.kalium.crypto.SecretBox should be quoted.  Thanks so much, I reviewed it and didn't see the lack of quoting at all...


i prefer:
> for (ComponentList::iterator it = m_components.begin(), end = m_components.end(); it != end; ++it)


Great Scott!


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


Определение общей части страниц ты полностью выносишь в мастер страницы.
<!DOCTYPE html> этого тоже не нужно


These function are not duplicated ?


`arg_dict` is freed but not set to `NULL` ?


Mutable default args a bad idea: http://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments


Would it make sense to support expression language here? Then you could set the value to something like ${index.operation} and have the operation determined by the incoming flow files, essentially one PutElasticSearch processor that can do both updates and inserts.


Remove these line if it's not needed and not useful for devs.


update_travis_toolchain (lefticus) - i386-Windows-10-VisualStudio-14: OK (389 of 389 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 18372](https://teamcity.mesosphere.io/viewLog.html?buildId=154582&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:06:16


I think the last checkin broke something.

Can't locate object method "interactive" via package "CommandLine" at C:\Users\A
dministrator.MICRONSTORAGE\Desktop\StorScore-master\StorScore.cmd line 208.

if I add --interactive or --nointeractive, I get back unknown option text like the following:
StorScore.cmd --target=\\.\physicaldrive1 --recipe=recipes\turkey_test.rcp --interactive returns "Unknown option: interactive"


Why this change ?


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1579 of 1579 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.85%25-green.svg)


You want call not apply here.


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - 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)


@yehudasa: reworked that.


Typo here s/FLOW_OIC_AND_OIC_LIGHT_SAMPLE/FLOW_OIC_AND_HTTP_LIGHT_SAMPLE/


Olá Raphael,

Na verdade não alterei esta parte do código. Apareceu como alteração porque
precisei comentar esta linha para testar uma vez que o envio para o DJE não
funciona em desenvolvimento.

Obrigada pela análise! Iremos verificar assim que possível.

Abs,
Adriana

Em 4 de março de 2016 18:08, Raphael Paiva <notifications@github.com>
escreveu:

> @adrigrassi <https://github.com/adrigrassi> como primeiroEnvio() joga uma
> Exception por sua assinatura, seria mais correto capturar uma Exception ao
> invés de um Throwable. Quanto mais específico, melhor :).
>
> No caso, como o código do catch apenas joga a exceção novamente, ele é
> prejudicial, pois aumenta a stacktrace, dificultando a leitura e poluindo
> ainda mais os logs, o que atrapalha na hora de identificar um problema.
>
> Nesse caso, o bloco try-catch pode ser eliminado, sem prejuízo à semântica
> do código e eliminando o fator prejudicial citado acima.
>
> []'s
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/projeto-siga/siga/commit/e99a32d2c6f7e0503e499e7eccc52009b99205eb#commitcomment-16510668>
> .
>


Looks good to me.  Congrats on eliminating a special case, particularly one that was only exercised on shutdown.


It would be better to use a a CListValue since it contains permanent python items. So CListWrapper callback can be removed.


![image](https://cloud.githubusercontent.com/assets/17623743/13541518/c5f0b610-e26d-11e5-9f52-686bd76a7f14.png)
???


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1881


Great. Looking forward to it.


Little boy, you accidentally left in the quotation marks "new messages"


I tested your branch, the generator is fine. However you should fix two problems in order to generate C code using sol-fbp-generator.


This variable is never used in the code and it is inside the closure so it does nothing. Worse yet if this script is ever included in an environment where `this` is `undefined` the whole thing blows up with:

```text
TypeError: Cannot read property 'MD5' of undefined
    at $:/plugins/sukima/md5/md5.js:16:36
    at $:/plugins/sukima/md5/md5.js:258:4
    at ContextifyScript.Script.runInNewContext (vm.js:18:15)
    at Object.exports.runInNewContext (vm.js:49:17)
    at $tw.utils.evalSandboxed (/Users/suki/source/personal/homepage-redux/node_modules/tiddlywiki/boot/boot.js:450:5)
    at Object.$tw.modules.execute (/Users/suki/source/personal/homepage-redux/node_modules/tiddlywiki/boot/boot.js:707:15)
    at sandbox.require (/Users/suki/source/personal/homepage-redux/node_modules/tiddlywiki/boot/boot.js:654:24)
    at $:/plugins/sukima/comments/comment.js:17:11
    at ContextifyScript.Script.runInNewContext (vm.js:18:15)
    at Object.exports.runInNewContext (vm.js:49:17)
```


very nice, ty


Typo here s/min:0|max:100:step:1/min:0|max:100|step:1/


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6: OK (397 of 397 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)


Is this supposed to be its own parent..? Shouldn't MainActivity be the parent?


http://lint.travis-ci.org/ :D


it should not be `#ifdef WITH_PYTHON` ?


Grate cheese.


Code style..


version 1.82R9 by cicciocb


@gaearon better?


I didn't realize that `M.update` is no-op for absent key (even though it's obvious from its type) :(


I blame HaxeDevelop for using tabs in a file that has nothing but spaces.


The use of the `this` keyword in the file is only to be renamed as `exports`

````js
var exports = this;
```

So the need to create a context is moot in this case. You could easily pass `this` in as a function argument.


Yes, that would be good for now. Let's talk early next week to figure out how to remove it definitively.


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


Also, I still think it's a good idea to open the MessageActivity directly if all the messages are from the same person.

eg.
Nick: Will
Nick: what
Nick: are
Nick: you
Nick: doing

should open up Nick's thread. You can just loop over the notifications you're showing and decide there (since only showing one is a subset of 'only talking to one person')


Nice!

Thanks, Robert :-)


This is not similar to py_base_init except for the SetGameobject ?


Why doesn't TextManager.getInstance(context).getThread(id) work? Does it throw an error?


It looks like one of your swift commits.


Might as well cat right into the file. Unless it really matters where it is in the file.

cat >> /etc/hue/conf.hue.ini << EOF


Why this MAKE_REF?


Rename to 'onDismissPendingIntent'


Please use `GameObject` not `Gameobject`


Rename to 'onClickPendingIntent'


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-LibC++: OK (389 of 389 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)


The logger is static, so presumably will close itself on program exit. If so, is the "TODO" comment necessary? Is is misleading?


Ok that sorted it.


Is this a convention in the app styles to use Title case class names?


Nit: no need to -r files


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


I'm not fully sure about the use of this function. It's this function called from C/C++ ?


Seems good to me


I still think this would be best in the script. Do what you feel is best.


`connection`, that's to trigger the memoization in connection.rb


I would leave the settings. No need to defend against people sourcing the script.


put here
> class KX_GameObject;


Also below in write_property/unset_property, does it miss zval_ptr_dtors on the old values?


Remove `class` here.


Can we do name-prefix or suffix instead?  With a name, it is very easy to have name collision since duplicate name with KMS key is not allowed.


12345 should STILL be a private static final constant


I would either stop hue right after installing it, or use "service hue restart" here.


I should have included this in the message:

http://www.django-rest-framework.org/topics/3.3-announcement/#deprecations

You can see that `'PAGINATE_BY', 'PAGINATE_BY_PARAM', 'MAX_PAGINATE_BY'` were being ignored and that is noted in the link (surprised it didn't cause a process startup failure).


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


PokemonModel, все таки это модель одного конкретного покемона


i am a bit confused with this layout. why is sender in the properties file and not in the yaml? Why does money transfer specify an email address via `notificationMailAddress`? What were your thoughts when you designed the config file?


If you need the `GetName` function why not inherit from CValue ? Also it will allow you to use CListValue.


Can we move this color to /res/values/colors.xml?


same here


Strange macro...


I like the pil and ms checks. It was too easy to mangle the command without this (these should be flag type arguments since they are indicator bits, but that's a different issue).

I don't think we should limit PI and secure erase settings, though. The spec defines values up to 7 for each of these, even though most are marked "Reserved". Basically, it's for forward compatibility to allow the max value for the size of the field.


It *can be* in the script, this is just a default which gets applied if the user doesn't do anything. This plugin is supposed to replace the old ant templates and so that's what it does.


I'm sure that the target is ok as IDA clearly shows it, but I can't assign a name. Using gm-editor shows that the value is common to several buildings in a site, so probably would be a leader or governor/major.
As I'm not sure, I only change the ref-target and keep the name anonynous


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 13](https://teamcity.labkey.org/viewLog.html?buildId=307925&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 9 (2 new), passed: 38, muted: 4 Build time: 01:20:03
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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

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

```


I feel the '???' :laughing:


Why the check is commented ?


IMHO we cannot do everything configurable. If you require everything to be configurable, we soon will be unalbe to handle the code. We must say stop some day.


It's the best place of it ?


Why not `${networking.domain}`? hostName doesn't seem right here?


Personally I think it's worth balancing the cost of a "run time penalty" against potential gains.  Here the run-time cost is initializing a small built-in type; that's considerably cheaper than a single virtual call.  The benefit would be that it may catch problems when the code is maintained in the future.  "Always initialize variables" is item 19 in C++ Coding Standards by Sutter and Alexandrecu.

But I'm not insisting.  What you have works.


Thank you for the update!


In my experience omitting the -v flag does not create an emplty 'data' directory - at least not one I could locate.


added the main archive


I mean that previously it was possible to do:
```ruby
SurveyGizmo.configure do |config|
  config.api_token = 'still_tippin_woodgraingrip'
  config.api_token_secret = 'it_takes_grindin_to_be_a_king'
end

SurveyGizmo.configure do |config|
  config.retry_attempts = 3
  config.retry_interval = 60
end
```
for example, because the `configure` method was only creating a new config object if there wasn't already one. And I changed it to reset the connection state and the configuration each time we call `configure`. It looks safer to me because when you call this block you don't expect previous config to leak into the new one. And it fixes a flappy spec that fell exactly into this issue.


:+1:

Can also tweak `.travis.yml` to just use `npm test` now


I figured something like that had happened.

How do I go about removing myself from my end?  Can I do it from email or
do I have to log in to Github to do it?

On Fri, Mar 4, 2016 at 1:22 PM, ACarrion316 <notifications@github.com>
wrote:

>
> Hey man!
> I'm really sorry about this haha. It seems that you somehow got added to
> this repository's watchlist. Probably due to us having the same name,
> really sorry about that. But our admin can't seem to do anything about it,
> he says you can take yourself from the watchlist on your end. If you need
> any help feel free to email me personally, or our admin.
> My email is: antoniocarrion11@gmail.com
> Admin email: kgary@email.asu.edu
>
> > On Mar 3, 2016, at 1:14 PM, acarrion <notifications@github.com> wrote:
> >
> > Hello? Can you guys see my email responses?
> >
> > I don't know who you are or what this project even is, but I keep getting
> > your pull requests and comment notifications in my email. I don't think I
> > even have an active Github account. Can someone please remove my email
> > from your project?
> >
> > On Tue, Mar 1, 2016 at 7:30 PM, atta2 <notifications@github.com> wrote:
> >
> > > Arron Helped me finding the block of code.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/cst316/spring16project-Basic/commit/6e51933d49e114d13936441580638420f92ab101#commitcomment-16435959
> >
>
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cst316/spring16project-Basic/commit/6e51933d49e114d13936441580638420f92ab101#commitcomment-16510884>
> .
>


@VerticalDynamics generally speaking, if you have a list with one list item in it, you're doing something wrong


Thanks! Just curious, do you usually use NA_Addr_lookup_wait? It had kept it for convenience but there's some time now that I'd like to delete it...


Uh, what? :)


yeah i mean, that was sort of intentional?  it should only create a new configuration object if there isn't one OR if you call `reset!`?

the one real reason it was like that was so that you could drop
```ruby
SurveyGizmo.configure do |config|
  config.api_debug = true
end
```
anywhere to force debug output.


@VerticalDynamics you want to use a <textarea></textarea> tag here


UnitConvInIDDforIDFed (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2133 of 2133 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)


Woo!


Okay, that makes some sense.  So the issue is basically that there is also another subclass of `AbstractTreeIterator` (at least `DirCacheIterator`) where you need the behavior to be different.

A couple of comments:
1. If you were going to stick with `instanceof`, it seems like you might want to consider `WorkingTreeIterator` rather than `FileTreeIterator`?
2. Using `instanceof` is a pretty fragile way to build just about anything.  It seems like in this case it might make more sense to add some method to the interface/abstract class that allows you to distinguish between the cases you care about.    So, e.g., you could create an Enum somewhere called something like `TreeIteratorType`, that had values like `WORKING_TREE` and `INDEX_TREE`.  Then you could add an abstract method to the `AbstractTreeIterator` class, called e.g. `getIteratorType`.  Then you could base your logic on that, rather than reflection.

WDYT?


Nope.  We were using it a while back until we (well, John) found the problem addressed in this pull request.  I just realized that this example program still followed that old model.  In Margo we have a wrapper for the NA_Addr_lookup() function that does the right thing rather than calling NA_Addr_lookup_wait().


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


Nothing right now.

But you hid "socket" away privately on basesocketparser, so I was re-exposing it, as a slightly-earlier version of this had quit() poke socket to send the output directly rather than the queue clear/immediate/discard dance it does now (and rawSocket was useless for the task when connected to SSL)


Why is this required?


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3047 of 3047 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)


Oh, I didn't notice. Thanks. :)


I think that it should be a @ManyToOne relation.


I'm not sure this approach isn't going to work because OSGroup will not flow through to the different projects I would expect this to be empty in most projects. We probably need to use something like TestNugetRuntimeId or something similar that we can flow through.


Maybe it was OK ?


пппппппппппппппппппппппп


Typically SpigotMC, these TextComponent objects are so weird :-1:


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-14: OK (389 of 389 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)


There are now two macros: one thread safe and the other not. Should we have just one thread-safe macro? If DRILL_LOG is needed for backward compatibility, should it just call DRILL_MT_LOG?


> Why this MAKE_REF?

This test: [Map/pairs::testPairsAreNotReferences](https://github.com/php-ds/tests/blob/master/tests/Map/pairs.php#L25), was failing with `Creating default object from empty value` at `$pairs[0]->key->state = false;`. Seemed clear that the key wasn't being returned as reference in a write context.

> Also below in write_property/unset_property, does it miss zval_ptr_dtors on the old values?

Looks like it. Shouldn't this be causing a memory leak in the tests? Currently not checking memory during test / deploy.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18378](https://teamcity.mesosphere.io/viewLog.html?buildId=154595&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:39


Avoid the use of `import *;`.


Yuck. Just add a protected sendRawLine method or whatever you need to the base class instead?


For ListenTCP, clientAuth should not be required and there should be custom validation to check if SSL context is set then Client auth should be set.


FixCrashWithZoneAndSystemERV (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2135 of 2135 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)


$pname is defined lol xD


Here:
$pname = array_shift($args);


The reason std::vector is used everywhere is because these containers will potentially get passed to the crypto_plugin interface. I didn't want to expose an implementation detail like the max number of receive buffers to an external API.


Add a nullity validation.


Add ?w=1 to the end of the URL of this diff to see, that there are indeed no real Code changes.


Create a new method that determines the value of `presenteeism`.


m_pOutStream is initialized twice, meaning that the first is a no-op. However, m_pOutFileStream is NOT initialized and so (if I remember correctly) will contain random garbage.


Sorry to be a nag, condition checks are executed in an order. your `.firstObject` will throw an out of range index exception, so do a safety check first then && the rest.


What's the purpose of the counter?


Signed-off-by: Timothy Sell <timothy.sell@unisys.com>


@WolfganP can you test the #851


TeamCity LabKey :: Sprint :: Premium :: Git sqlserver 2014 [Build 24](https://teamcity.labkey.org/viewLog.html?buildId=307939&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitSqlserver2014) outcome was **FAILURE**
Summary: Tests failed: 11 (1 new), passed: 244 Build time: 00:51:43
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.JUnitTest$RemoteTest.org.labkey.sequenceanalysis.TestHelper$SequenceImportPipelineTestCase: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testPublicationDetail: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testStudySummaryWindow: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testStudyCards: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testStudyCardStudyLinks: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testFilterOnStatus: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testSSOwithCASFromHeaderLink: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testNewUserSSO: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testSSOWithCASFromLoginPage: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testCASUnreachable: <no details avaliable>

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


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (389 of 389 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)


Fixed in 467231074870f00ae520fb98a3bf419ffaceaef7


It should throws an `UnsupportedOperationException`.


Thank you for this!


:+1:

Thanks for your contribution.


This function is in the header and thus inline. However, is there any reason to do this? Since this is not a performance-critical function, seems it should instead be implemented in the cpp file.


ok


When submitting a review, the star rating was being converted to a string which affected a restaurant's average rating and amount of stars displayed. I also added useful, funny and cool properties to each review #7.


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2167 of 2167 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)


@gizmo98 thanks for including this in retropie!

I think for future a switch to SDL2/GLES may help improve performance, especially on pi1 (build options -DUSE_SDL2=1 -DUSE_OPENGL=2)   However it fails compiling the SDLVideo plugin.  It's missing the /opt/vc paths and will likely need bcm_host libraries.  Unfortunately i wasn't sure where to add these for cmake when i created the script.


Add the window's title as a constant.


This should probably not be here!


LGTM


No protection for bogus log levels: will simply read random garbage outside the array bounds.


Empty labels?


Avoid import *


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2119 of 2129 tests passed, 205 test warnings)

 * 214 tests had: AUD diffs.
 * 10 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.53%25-yellow.svg)


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


change "OSTRO" to "Ostro OS"


I'm think that "UMS mode" will be less oblivious than "USB-storage mode" for russian people (not all of them knows english acronyms very well). Maybe even "USB-disk mode", but devices looks like brick much more than disk.

Думаю, "Включить UMS" будет менее понятно, нежели USB-накопитель (не все пользователи хорошо знают англоязычные акронимы, да). Можно даже "Режим USB-диска", но по сути девайсы обычно напоминают больше кирпич, нежели диск.
Про внесение в патч - вносите. Я тут ещё кое-что добавлю - подождите чуть-чуть?


Use `this` keyword for class attributes.


Add the javadoc to the constructors.


:sunglasses:


It would be good to leave an inline comment explaining what the conditional is checking for as it's not immediately obvious


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


This function should needs documentation


LaTeXDummy (Myoldmopar) - x86_64-MacOS-10.9-clang: Build Failed

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


What does this function do? Please document :+1:


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


int or Constant or Scalar Tensor of int* dtype


Declare this through its interface (List).


You indentation here is inconsistent with the rest of the file


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


FixCrashWithZoneAndSystemERV (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)


Same comment here.


Awesome. Thanks for fixing this!


Add a TODO comment.


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.7: OK (397 of 397 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)


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2125 of 2135 tests passed, 205 test warnings)

 * 214 tests had: AUD diffs.
 * 10 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.53%25-yellow.svg)


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


Add the "-" String as a constant.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18383](https://teamcity.mesosphere.io/viewLog.html?buildId=154605&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:30


Pull request #523


Thanks for explaining that link! It wasn't clear who the source was. Unfortunately, I can't seem to find any official checksum to verify with. I'm probably just being paranoid anyway. :smile:

I will not use this to ship mission critical code, but it is great for experimenting!

I found this via Ruby Weekly, by the way.


I'm not sure I understand the issue, but I doubt that creating a reference is the correct solution. As long as you're returning internal storage rather than using rv indirected writes go through indirected vars, references aren't needed for that. That error message does not sound related to references.


I think the stype for a magic is always going to be dense. There's no reason to make a yale or list magic matrix because it'd end up taking more memory than dense. These other stypes are only for sparse matrices.

(I don't remember stype being in here before; why did you add it?)


Nice work!  Just a minor comment about the Tenant diagram:

I think of Accounts and Groups within a directory as peers - not in a hierarchy.  I'd probably represent the membership as a horizontal line between an Account box and a Group box, not a vertical one (as that usually implies hierarchy or communication flow).  Dunno if you'd find that useful. HTH!


You have an unnecessary `return;` at the end of the function.  If you remove that:
Signed-off-by: Timothy Sell timothy.sell@unisys.com


Signed-off-by: Timothy Sell timothy.sell@unisys.com


That's a great idea. That is currently the case with the 3 pictures currently included but we will continue following this naming convention for future images.


And another unrelated node, should the !value case maybe throw an exception? It seems weird that `$pair->foobar` will just be a silent null.


Good catch :+1:


Oops... I see there's also a typo in the headline of your log comment.


This line shouldn't be updated. It is not necessary.


Hi there, wildcard import @Fireflies


Avoid import *


Probably minor detail, but this method loops through all handlers every time, when it could just exit as soon as one handler says restrict:

    def _get_handler_result(self, item, weapon, attr):
        weapon_name = weapon_manager[weapo].basename
        for handler in self:
            if getattr(handler, attr)(item, weapon_name):
                return True
        return False  # Or None?


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.7-LibC++: OK (389 of 389 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)


Is it even worth keeping this as a function? I think it might be easier to inline in the two(?) places it is called?


@mrjbq7 I tried to leverage that. However, the process that I came up with had to copy a bunch of the factor compiler locally (because it wanted to create a directory in the compiler installation folder) and it ran for something like 1 hour on my fairly fast system and then crashed.

Can you provide some guidance on the correct way to build a self-contained executable? I would appreciate it.


Это немного путанная запись, потому что сюда поступает пустой GET запрос, он создает переменную и инициализиурет поля данными, которые переданы с запросом, но так как все данные генерируются внутри класса, то они там уже есть. Это работает, но немного странно. Лучше либо создать статический метод, либо создать экземпляр класса и получать данные из него. Потому что делая так, ты запутаешь себя в будущем.


Rename it to `attendanceMap`


nice! Neat JS trick! :100:


Если ссылаешься на представление из этого же контроллера и с таким же именем, то явно его указывать не нужно. То есть можно записать так return View(hobbies);


top/index.html.erbを新規作成。ブログ機能をやり直した。


Is the second argument of `combine` needed? It does not seem to change, so the call to `f` in L43 could just be replaced by `backjumpInfo` (or inlined version thereof, see the other comment), and `combine` itself could be simplified?


Если используешь мастер страницу, то на нее не обязательно явно ссылатся.


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


Since we're hardcoding the slash `/`, is this going to fly on Windows?


first release for sauerbraten justice edition


There are descendent nodes that do their own scrolling.


@remitamine How so? As far as I am aware, all of the porn sites we support are perfectly legal in both the countries they're hosted, as well as the countries where youtube-dl is hosted. Also, why personal morals may vary (up to decreeing all pornography as improper), I believe all of the sites we do support depict consenting adults.


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


Oh whoops. Didn't see it lol.


Sorry about the comment I made here before. I did not read the call to `foldr` correctly.


The method `isEnabled()` returns a boolean value so it's not necessary if the value if equals to `true`.


switch(String) worked great with jdk 1.7 , but openshift projects works only with jdk 1.6 , so i went back to jdk 1.6 and unfortunately switch(String) wasn't supported yet by the jdk 1.6
===> so i used if(){} structure


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


My tests weren't symlinked in /vendor, so I had a local false-positive. Making a reference doesn't fix it (like it would an indirect modification notice). The error does imply that the value is null, even though a `var_dump` before access shows a valid value.

Investigating.


can't wait to finish dis!


This is going to force us to eagerly emit type metadata for anything with an archetype in it, regardless of whether it's just an archetype or not; e.g. [T].  That seems unnecessary and expensive.


PS: Not sure how or when I'd use `rv`.


s/exits/exist/


If you could make this part of docnet.json it would be great and I'd love to merge a PR :) (with the other changes you've made!)


Also, I thought the problem here was that you specifically needed these things emitted in the prologue.


This is just older/crufty comments, so feel free to merge it (not sure what george is up to but there should be some updates soon hopefully).


release-5.x (lefticus) - i386-Windows-10-VisualStudio-14: Build Failed

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


1901_radiance_translator_reporting (macumber) - Win64-Windows-7-VisualStudio-12: OK (2361 of 2395 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-98.58%25-yellow.svg)


lol


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


RSE does inherit from SE but when the package is not loaded (in console testing with .GlobalEnv), inherits(object, "RangedSummarizedExperiment") == FALSE


creo que rompimos algo porque ahora no puedo crear items desde el html aunque desde postman le pega bien


doing this actually breaks ansible stack because tasks in deploy_db depend on the cchq user. going to mvoe to be above deploy_db