95077770-HVACTemplateLoadManage-4493 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (633 of 633 tests passed)

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


This line fails to compile in OS X.


delSpan is cleary having problems being rendered in general, but that's my problem. Try to encode every deleting "revision" as one. The encoding needs to formalize what you think happened, not how things should look like.
I would also keep the unclears, and the del[rend='unmarked'] they are useful for more than rendering. My suggestion for these two lines:

```xml
<line>have <del rend="strikethrough">added</del>
  <mod>
    <del rend="strikethrough">adorned</del>
    <add place="superlinear">illustrated</add>
  </mod> Homer<del rend="unmarked">,</del>

  <mod spanTo="#e1-0036.01"/><delSpan spanTo="#e1-0036.01"/>
  <add place="superlinear">
    <mod>
      <del rend="strikethrough">the hidden</del>
      <add place="sublinear">
        <del rend="strikethrough"><unclear reason="illegible">of</unclear></del>
        <del rend="strikethrough"><unclear reason="illegible">from</unclear></del>
      </add>
    </mod> beauties</add>
  revealing his</line>
<line>inner beauties<anchor xml:id="e1-0036.01"/>: I deserve a golden crown</line>
```


Not likely, but will accommodate


Fixed, changed Response to SearchingResults and removed metadata


To make it work in IntelliJ for tests I need to put  :

    def propertiesFile = rootProject.file('dependencies.properties')

Is that normal ?


After further review, MongoDB as well as SQLite both treat the dividend and the divisor as a rounded down integer.  This being the case this function has been re-written and the NSExpression modulo call has been replaced by the `%` operator in 2b135fceef34feb66693846c89e4ea1e9b7b9790.


Any chance you could make this notification observer add similar style to the others in this file using the block NSNotificationCenter api and adding the observer to _observers? I know it shouldn't really matter since this notification should only be called once, but it will be nice to keep the style consistent. Those observers also have the behavior of removing themselves when the plugin is disabled.


1640-AddHVACPart1 (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Thanks for contributing!


Hold on a few; this was from before K-T ask to convert to a class variable.


Thanks :-D.

--
Med venlig hilsen
  Ulrik Hardt Schønnemann
  61274152

> Den 26/05/2015 kl. 20.54 skrev Benedicte Emilie Brækken <notifications@github.com>:
>
> Removed addr field and always set to @ulrik.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/benedicteb/Houston-Bot/commit/33172bcf9bf917b8f1393dba1c4d5d844b566637


Ah okay cool. Ya I think i confused the comment with a new PR when it came in through email. Cheers!


This works for me :+1: :smile:

```
$ brew install libunwind.rb
==> Downloading http://download.savannah.gnu.org/releases/libunwind/libunwind-1.1.tar.gz
Already downloaded: /home/stevemoss/.cache/Homebrew/libunwind-1.1.tar.gz
==> ./configure --prefix=/home/stevemoss/.linuxbrew/Cellar/libunwind/1.1 --disable-silent-rules
==> make install
/home/stevemoss/.linuxbrew/Cellar/libunwind/1.1: 59 files, 764K, built in 22 seconds
```


Could you comment on why we need both self._root_url and self._meta['root_url'] set?  It seems odd that the original design would require both to be set, and I wonder if this is an under-developed part of Nap, or if we are using it in an new and unexpected way.


The `<zone type="left_margin">` elements should be outside of the main zone (that's why they're appearing twice.


Validation logic has been added to check for division by 0 in f727ec876d7d3b8ce0daeae9a28f00150d3d9470 and validation tests have been added in e7ac8ced187afdaba029c44f80a073504b1a306e.

Previously missing unindexed matcher tests have been added in 1c13e6bb7fe1c2ae10458eec636505d9245f208e and executor tests have also been added to test additional `$mod` conditions in 1524c0a7d9154c1db0c9203bebe608538989106b.


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: db_printview.php


I suggest the container variable to have the value of 1200px (max-width)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.179](http://teamcity.monogame.net/viewLog.html?buildId=7670&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Ah, OK. Now that you point this out I see that it's "correct", though when I read it (without cross-comparing the two doc strings) it looked like it hadn't changed. Maybe you want to add something in the "Notes" to this effect?

Also, this probably suggests that you ought to update your omnia/conda version since this resulted in a peculiar scenario for us where people installing via conda got code which wouldn't run (different API) and those installing directly from GitHub got code which WOULD work. While conda releases may not be a high priority it's probably bad news if there are different APIs?


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2135 of 2211 tests passed)

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


Oh well, it's harder to know why @cevich put that particular conditional there... Given the time this has been written, even him might not remember :)


Looks good to me


Could `pytest.raises` be used here?  It looks like the pytest module is already imported in this file and is available for further use.


Keep the anchors here. Do not create new lines, just paste them after the sublinear insertion. They do nothing now in the rendering, but they are useful to know where the left marginal additions fit.


flash-gord**o**n (=


private mode is not yet enabled for the version that is currently
published. i might add it in the following updates.
On May 23, 2015 8:04 PM, "tatufan" <notifications@github.com> wrote:

> i'm using firefox and it's not working in private mode
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nicu03/dark-youtube-theme/commit/ba0705cae32b280ee9a7738de92d5ca18a8b7a97#commitcomment-11340019>
> .
>


Will there be an if $financialTypes is an empty array?


Let's make opts.transform an array so that you can pass any number of transforms in.
Reactify is required though, so merge in the opts.transform array with `util.arrayMerge`.

```
util = require('./util.js')
var transforms = opts.transform || []
...
transform: util.arrayMerge(transforms, ['reactify'])


I just added the new workspace add modal that Cornelis designed. When I came to the new security settings I stumbled over the mix in wording which looks as if it should be possible to set values affecting the first two slider settings in the security settings sidebar. I am not sure what to set here. Can you take a look and tell me or amend directly @mattss @wengole?


Sure, sounds good to me :smiley:


@selimnairb Here is my understanding. In django_irods/urls.py, it defines download url as url(r'^download/$', 'django_irods.views.download'), a high-level relaxed url pattern definition for downloading a file from irods. Then in django_irods/storage.py, url API definition returns a URL that follows the url pattern mapping as specified in urls.py, but with file name to be downloaded being passed in defining the path in URL. If we need to change URL in the future, we can simply change url in this storage.py. As long as it followed the high level download url pattern (which I don't see any reason it could change in the future), it should be fine. So from this perspective, we only need to change url in one place in storage.py if there is ever a reason to change URL in the future. I think this should work just fine without the need to call reverse in this specific case.


Acknowledged.


Just wanted to ping you guys. I am not leaving you on the lurch, I promise. I just got really busy and will finish this on a Ubuntu VM I've got here *eventually*. Also, I haven't checked all the other translations so far, but do I need to translate everything? Some stuff seems simpler in plain English (like, 10% of the entries I can estimate).


Sounds good, we can add something to the notes.


In this instance, I've added the ".". In other places, where are just phrases, I don't feel it is appropriate to add the ".".


@jcansdale please open an issue for this, it sounds like it could be a bug: https://github.com/Microsoft/visualfsharp/issues/new

How exactly does this come up -- are you just building a project? Is there a pop-up crash dialog?


Here's a more complete list.
```
local burstHasteBuffs = {
	2825,		-- Bloodlust (Horde Shaman)
	32182,		-- Heroism (Alliance Shaman)
	90355,		-- Ancient Hysteria (Core Hound, BM Hunter Pet)
	80353,		-- Time Warp (Mage)
	120257,		-- Drums of Fury (Leatherworking)
	146555,		-- Drums of Rage (Leatherworking)
	160452,		-- Netherwinds (Nether Ray, Hunter Pet)
	178207,		-- Drums of Fury (Leatherworking)
}
````


I think it means that every time the value has been changed, the item is updated. I'm not sure, though.


Old variable?


Prepared Response?


Kept the same Exception pattern as the Java version of this library which uses the IOException.


See https://github.com/phpmyadmin/phpmyadmin/commit/830d39816b2bcf53233cd684f55b17b62e22fe4f


of New Response Rule?


Try and keep spacing consistent. Your colleagues will thank you.


Yeah, sorry I should have probably sent this subjective change as a PR for feedback instead.

Removing the `fs` dependency means the `brfs` browserify transform is not necessary for `friends-swarm`, which makes browser usage a little more straightforward.

Using `multiline` is just a small convenience that could be removed if others feel its confusing.

We can switch back to `fs` + `schema.proto` if folks feel like it offers more clarity.


Nie dodawaj głupot!


maybe better to read the quote directly from the file instead to building the list


Bo będzie dym!


Hello,

I have a question, for example I have run an application. it detected e.x 6 upnp devices.

I turn one one more - it has been detected. So in tottal 7 at the list - FINE.

I remove one device, and seems that UPNP Observer does not get that and do not remove it. Any idea ?


I see what you mean. I'll fix that tomorrow. Thanks


Ao invés de `[var, init]`, é `(List [var, init])`.


Please upload @drawable/*_ic
Error:(27, 30) No resource found that matches the given name (at 'src' with value '@drawable/play_ic').
Error:(68, 30) No resource found that matches the given name (at 'src' with value '@drawable/crown_ic').
Error:(108, 30) No resource found that matches the given name (at 'src' with value '@drawable/chart_ic').
Error:(147, 30) No resource found that matches the given name (at 'src' with value '@drawable/ppl_ic').


Certainly better, but of course that takes us into the realm of the blog post you linked to where it is clear that there are some (not-named) people in charge who will censor other people if they do things that they don't like.

How about 'We believe strongly that behavior that belittles or excludes other people is damaging to the community, and we are careful to avoid:"


It is extra source lines and also implies that there could be code after the else block which would not be reachable. I think it is cleaner without the else. I also prefer the style if "if (!something to do) return; do lots of stuff;" to "if (something to do) { do lots of stuff; }" since it reduces the number of indented lines and makes clear the early exit.


rax retry tests


`1 % 5` does not equal `0.2`.  It equals `1` but with that said, this point is irrelevant now on account of the desired behavior is the rounding down of dividend and divisor to the nearest integer value just prior to applying modulo arithmetic.  This behavior is confirmed in MongoDB as well as the SQLite engine.  Based on these findings I've simplified the matcher code to match the desired behavior and to use the `%` operator to perform the modulo arithmetic.  Logic done in 2b135fceef34feb66693846c89e4ea1e9b7b9790 and tests added in 1c13e6bb7fe1c2ae10458eec636505d9245f208e and 1524c0a7d9154c1db0c9203bebe608538989106b.


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

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


Will update documentation shortly.


I think we still ought to encode the modification, even if we make a small concession to facilitate the rendering:

```xml
<mod>
  <del rend="strikethrough">see, as they flas</del>
  <add place="sublinear"><del rend="strikethrough">as</del> the beams flash
    <mod>
      <del rend="strikethrough">deep</del>
      <add>on</add>
    </mod>
  </add>
</mod>
<add place="sublinear"><metamark function="count">250</metamark></add>
</line>
```


Ok, I've tried the following test, and unless I am totally missing something wrong in my setup, I am seeing behavior I don't expect.  In particular, here is what I am doing:
 * Create a slice (managed tag mode, the switch itself is handling timeouts)
 * Install a set of flows for matching my ICMP/ARP traffic in the data plane with a very large idle timeout.
 * Send traffic in the data plane within that slice
 * Delete the slice

If I don't stop sending traffic, then the flow never goes away.  I looked at the control plane packet capture, and I am seeing the flow stats reply return the flows I installed, but I never see anything trying to purge those flows.  I don't see anything in the logs about trying to remove the flows, although I am running at the warn level.

Do you want me to run the test again and gather some specific info?  Should I file a new issue?


valeu!!!


@hiskonog Tim O'Neil graciously accepted to read the texts and made some simple edits, i hope you like them. review the deployed website and feel free to make new changes and pull requests if you have some updates


Acknowledged.


ok


There is no need for an aria-label here.  An ```<ul>``` does not have an accessible name.  Actually, the accessible name is the contents or children of the element, so by overriding the accessible name with an aria-label here, you risk making the actual content (Review, Payment, Confirmation), inaccessible in User Agents who strictly follow the accessible name calculation algorithm.  Also, if these steps are to be completed in order, then this should be an ordered list ```<ol>``` , not an unordered list.


cannot call ranker.init() method and im gettin NPE when using ranker.rank()


no idea what you mean by it... @martijnbastiaan


Keep the superlinear add. Ok with ditching the "separate" metamark for an underline.


By the way, I have yet to see a creature that needs to be scaled.


sorry i am on kitkat


See #13


@severin-lemaignan @McNopper i fixed this by tweaking CMakeLists.txt
```
find_package(GLEW REQUIRED)
find_package(GLFW REQUIRED)
#find_package(PkgConfig)
#pkg_search_module(GLFW REQUIRED glfw3)
```
adding the file glfwConfig.cmake under dir GLUS

```
# - Config file for the glfw package
# It defines the following variables
#   GLFW_INCLUDE_DIR, the path where GLFW headers are located
#   GLFW_LIBRARY_DIR, folder in which the GLFW library is located
#   GLFW_LIBRARY, library to link against to use GLFW

set(GLFW_INCLUDE_DIR "/usr/local/include")
set(GLFW_LIBRARY_DIR "/usr/local/lib")

find_library(GLFW_LIBRARY "glfw3" HINTS ${GLFW_LIBRARY_DIR})
```
and variable GLFW_DIR in cmake gui pointing to the GLUS path. As i said, i am using homebrew, which by default installs stuff at /usr/local. I want to build examples with xcode, not with eclipse. For now, I am getting a linking error due to GLEW - GLEW is irrelevant on mac.


please remove aria-hidden="true" here.  This is already hidden by type="hidden"


`'ngSanitize'` is a dependency of showdown and is imported in the `ng-showdown.js` file so theres no need to import it yourself. I opened #4 to address this.


Valeu ; D .. agora tô procurando esse feliz pra sempre KKKKKKKKKK


Well let's pick one method. If we're doing underline in 0022, we should do the same here.


I'll try this out again with the debug-level logs and pcaps in both managed tag mode and normal mode, and then I'll file the new issue.  As you probably noticed, it took me a long time to get this last test done... lots of stuff going on today.  I'll try to get this one filed today if at all possible.


@hyi I would prefer reverse be used, as there is still some duplication, but I guess this is okay.


Yep, realized the typo after i pushed the commit but was not sure how to fix it ex post
([amend, rebase, push force?](https://help.github.com/articles/changing-a-commit-message/))


Acknowledged.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.179](http://teamcity.monogame.net/viewLog.html?buildId=7670&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:19:02


@clebergnu after talking with @adereis, we considered that involving `$PYTHONPATH` is too intrusive, so we'll keep things the way we were doing previously.


`mChangeAction` is a callback which will be called everytime the value of the element changes. We should consider changing it to a event tho.


This won't work when booting with Device Tree because [of_dma_request_slave_channel()](http://lxr.free-electrons.com/ident?i=of_dma_request_slave_channel) does not return NULL on error (missing dmas property). It returns an error pointer: ```ERR_PTR(-ENODEV)```

Tested with and without Device Tree:
```c
	if (IS_ERR_OR_NULL(host->dma_chan_tx) || IS_ERR_OR_NULL(host->dma_chan_rx)) {
		dma_cap_mask_t mask;

		dma_cap_zero(mask);
		/* we don't care about the channel, any would work */
		dma_cap_set(DMA_SLAVE, mask);
		host->dma_chan_tx = dma_request_channel(mask, NULL, NULL);
		host->dma_chan_rx = dma_request_channel(mask, NULL, NULL);
	}
```


Done.


Why is "ShowTrayNotifications" still negated? Shouldn't it be without the negation? - HideTrayNotification was negated and ShowTrayNotifications is the opposite.


Headings are used to indicate document structure, not font size/appearance. They hold special meaning and behaviors for users of assistive technology.  It appears as though this should all be one heading "Registration for [course name]"  Can this effect be achieved using child span's to achieve the physical appearance desired?


Certainly not with "L___", oh dear :) Anyway, if we want to keep the metamark, we need to update occurrences where it was switched for an underline. I think there is only one case in 22


@cesine Thanks, and thanks to Tim!


Mislabeled as route, should be "added contacts controller"


16px labāk. ;)


The idea is that we are not only careful to avoid, but we also say
something when we see someone behaving in these manners. Not anyone in
particular, but any member of the community. That's the "we" in "we do not
tolerate".

On Tue, May 26, 2015 at 12:08 PM, Matthew Brett <notifications@github.com>
wrote:

> Certainly better, but of course that takes us into the realm of the blog
> post you linked to where it is clear that there are some (not-named) people
> in charge who will censor other people if they do things that they don't
> like.
>
> How about 'We believe strongly that behavior that belittles or excludes
> other people is damaging to the community, and we are careful to avoid:"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arokem/nipy.github.com/commit/a80cc6b9a2977ab8f837164c3e65d138452a8083#commitcomment-11371326>
> .
>


How can it go wrong if it's set to you? Sorry about the notification ;)


Why is this needed? If we wrap route components by default, we are giving up some of the advantages we got from removing RouteHandler, like being able to ref child components, or using TransitionGroup without extra effort.

I think at the very least we should give it a name so it doesn't appear as `<Unknown>` in react tools.


Isn't the style guide to have a blank space between the '{' and '}'? Since I was in the neighborhood, I wanted to fix this also.


I just moved that performance-guard clause before the replacement of the math sections - otherwise the math sections get never replaced back (ends up with `__MATH0__` in your titles etc.)


Sorry, my bad, I'll fix it :)


```for``` attributes *MUST* be paired with ```ID``` attributes, not ```name``` attributes.  The value of ```for``` here should be


Class NotifyEmail does not have a Docblock comment.


@nahuelsotelo Hola Nahuel!! Muchas gracias problema resuelto!! buena semana!


Keep unmarked deletions, please


Yeah I guess so


Just for future reference, the `ERROR_MSG` and `SUCCESS_MSG` spans can be deleted. They were replaced by the Notification class.


Acknowledged.


opening*


This mixes different changes in one commit and contains a print...


@selimnairb OK. Thanks. @mjstealey I think this branch is ready for merge from coding perspective unless some other issues would be brought up.


Yeah this is strange. Good idea adding the superlinear addition and I would remove the milestone, it's not useful.


I don't think this is proper. Just because the objectClass value in the filter cannot be directly uses as a permission value does not mean the bundle should request permission to get all services. I think it just means bnd cannot set the ServicePermission. The user would need to do themselves.

So this code can set a ServicePermission only if the objectClass value in the filter is either (1) just "*", (2) endsWith(".*") and otherwise does not contain *, or (3) has no "*". If the osgi.service requirement was set by some other part of bnd (i.e. DS annotations processing), then the value will be of type 3 (no "*"). The other cases are likely only if the user has manually set an osgi.service requirement in which case, if it is not type 1 or 2, then they can also set the ServicePermission manually.


What? Do you know WS after `}`


This is a static method and hence has no object instance.


Yep, no remember :frowning: It must have been to avoid reading really short lines since they're less likely to be valid, and we hope and pray nobody uses 'up' as a key :grinning:  It's probably mostly safe to remove the '<3 len', assuming it doesn't break any smoke runs / unittests.


Need to fix the same reference on assert.


if dropping `$_POST` parameter is not an option, would like to know list of parameters it may modify, so could provide appropriate array for it


hi


These buttons do not have accessible text.  please add ```<span class="sr">Increase</span>```  and ```<span class="sr">Decrease</span>``` as children of these ```<button>``` elements


Please put the local ID into a separate `Server` field and use it here.


...and here.


s/being/begin/g


Please, attach a small code sample which breaks to the Trac ticket.


Please make this `NoDiscovery` so discovery is on by default.


Uhm, what? Why is this suddenly necessary? Is this s hack to make the previous patch work? If yes, this warrants a comment. If no, this still needs a comment.


Cool! :+1:


Good :)
But in GenerelFunctions is already a Bloodlust Function. Should get extended instead using another function.


:+1:, thanks :)


Agreed


Nice catch, this is actually a bug. It turns out that if the `openid.ns` value is absent, the response should be interpreted as an OpenID 1.1 response.


nit here and elsewhere, \n is superfluous


https://projects.artefactual.com/issues/8459#note-2 for details


The patch is not sufficient, though, so I won't pull. Will submit a fix shortly.


Is the aria-hidden state being toggled here?  It's really not necessary to include.  If the "hidden" class hides this message using ```display:none``` then aria-hidden is not necessary.  ARIA would only used to override native roles states and properties in situations like these.  Never necessary to be redundant.  I'm assuming that error messages are added to this span dynamically.  If so, you want to add ```aria-live="polite"``` to this span.  But please remove the aria-hidden first.


I know enough Git to use it, but not enough to fix mistakes ex post. Sorry. :-p


You can use '*' here instead of listing out all of the attributes


This seems a bit hackish... Can we share some code with the client xproperty and property stuff etc?


We should raise an error if we don't know how to handle this.  Can you also add a test?


No accessible text here.  please add ```<span class="sr">Remove</span>``` as a child of this ```<button>```


@popcornmix any quick idea how i could compile this viz for openelec wetek? ^^


'SET' comes before 'WHERE' here


проблема с подгрузкой (issue: <audio>


Yes additions at the beginning of lines are a viewer problem.


You need to say 'DELETE FROM' instead of just 'DELETE'


This input *MUST* have an accessible ```<label>``` associated with it.  placeholder is not enough.  It does not have to be onscreen, so feel free to add class="sr" to the label.


Agreed. I will change. What do you think of this solution overall?  I'm not in love with using exceptions to handle valid use cases.


Shouldn't this output the prebuilt library?  It'd certainly be more inline with what we do for `prebuilt_jar`, for instance.


aria-live="polite" is necessary here if error messages are inserted dynamically.


This breaks CanJS's build.


I use a script like this for building binary addons:
```
#!/bin/bash
mkdir -p build
cd build
cmake -DCMAKE_TOOLCHAIN_FILE=~/bin/cmake-pi.conf -DCMAKE_INSTALL_PREFIX:PATH=/opt/xbmc-bcm/xbmc-dbg/arm-linux-gnueabihf -DKODI_INCLUDE_DIR=/opt/bcm-rootfs/opt/vc/include/interface/vcos/pthreads ..
make && make install
cd ..
```
with
```
$ cat ~/bin/cmake-pi.conf
#
# CMake defines to cross-compile to ARM/Linux on BCM2708 using glibc.
#

SET(CMAKE_SYSTEM_NAME Linux)
SET(CMAKE_C_COMPILER arm-linux-gnueabihf-gcc)
SET(CMAKE_CXX_COMPILER arm-linux-gnueabihf-g++)
SET(CMAKE_ASM_COMPILER arm-linux-gnueabihf-gcc)
SET(CMAKE_SYSTEM_PROCESSOR arm)

#ADD_DEFINITIONS("-march=armv6")
add_definitions("-mcpu=cortex-a7 -mfpu=neon-vfpv4 -mfloat-abi=hard -std=c++11")

# rdynamic means the backtrace should work
IF (CMAKE_BUILD_TYPE MATCHES "Debug")
   add_definitions(-rdynamic)
ENDIF()

# avoids annoying and pointless warnings from gcc
SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -U_FORTIFY_SOURCE")
SET(CMAKE_ASM_FLAGS "${CMAKE_ASM_FLAGS} -c")

# where is the target environment
SET(CMAKE_FIND_ROOT_PATH  /opt/xbmc-bcm/xbmc-dbg/arm-linux-gnueabihf/)

SET(CMAKE_INCLUDE_PATH ${CMAKE_INCLUDE_PATH} /opt/bcm-rootfs/opt/vc/include /opt/bcm-rootfs/opt/vc/include/interface/vcos/pthreads)
SET(CMAKE_LIBRARY_PATH ${CMAKE_LIBRARY_PATH} /opt/bcm-rootfs/opt/vc/lib)

SET(CMAKE_INSTALL_PREFIX:PATH /opt/xbmc-bcm/xbmc-dbg/arm-linux-gnueabihf)
SET(KODI_BUILD_DIR /opt/xbmc-bcm/xbmc-dbg/arm-linux-gnueabihf)
```


This comment isn't terribly useful.  If you want to keep it, please at least use the proper `@returns`


Setters and getters are not necesary due to all variables are public in Play Framework.


It's helpful to put ```aria-describedby="code"``` here to add the error message to the accessible label for this field.


It's helpful to put ```aria-describedby="students-${item.id}"``` here to add the error message to the accessible label for this field.


We keep the list of rules alphabetized.  This should go higher up.


Indentation here is wacky.


teste


I think the encoding is clearer as it was before. Though the `<del rend="unmarked">i</del>` is probably a bit overkill...


I ran this against my code and I am getting:
`ERROR:  table name "referral" specified more than once`


Why do we need to do this?  We run tests in parallel, so modifying properties is not likely to work out well.


For some reason the state will not reset in my setup:
```
    {
      "class": "DummyPresenceSensor",
      "id": "esp01-pir",
      "name": "Wohnzimmer PIR",
      "autoReset": true,
      "resetTimer": 5000
    }
```
I use the latest pimatic version and restarted


nit: `results`


Can you put a unique ID on the ```<p>``` below and add an ```aria-describedby``` to this button element referencing the unique ID?  Otherwise, a screen reader user would not be exposed to the help text that appears *after* the button.


Yes, even better - fewer words !


teste 2


indeed, will use this approach for OSX! thanks!


This message should have the same ID as [Line 153](https://github.com/edx/edx-platform/commit/f5b166f120c950f0290a6c166dceb756b7dc964e#diff-dfe065bc6d506bc0c00cd6635357985bR162)


saw approval from cmeiklejohn
at https://github.com/basho/riak_kv/commit/9473567ca112b84e1ceb42d90304bea62e60a323


merging basho/riak_kv/bugfix/jrd/drop-cover = 9473567c into borshop-integration-1124-bugfix/jrd/drop-cover


basho/riak_kv/bugfix/jrd/drop-cover = 9473567c merged ok, testing candidate = addff2e5


Para Arreglar Issue https://github.com/CamilaAlvarez/IndicadoresDCC/issues/3


see above about aria-describedby on the associated button.


This should be `React.PropTypes.bool` - React complains about it currently! @KyleAMathews


I would keep the same name as the original error codes, for instance

`MethodError` -> `MethodNotAllowedError`
`UnsupportedError` -> `UnsupportedMediaTypeError`

PS: why not just iterate through `http.STATUS_CODES` and build error types out of it?


shouldn't we have subdirectories `pos` and `run` inside `tests/disabled` ?


Can we avoid this? This smells too much like a kind-of-cyclic-dependency. I'd rather now have awful depend on beautiful and the other way around, too.


This default value causes the following warning:
`Warning: Failed propType: Invalid prop 'overrideSpinnerClassName' of type 'boolean' supplied to 'SpinKit', expected 'string'.`


Can you check now ?


Can we disable rmparser entirely? Or can you not disable one parser without disabling all of them?


Why 15? Does this mean to check if all entries are present?


@ctrueden I did this to mitigate skew between the properties and what's attached to the `ImgPlus`. I'm happy with this PR if you are.


Is fixed;)
I also moved AEStream options to [AEStreamData.h](https://github.com/AchimTuran/kodi/blob/adsp_library.kodi.audioengine/xbmc/cores/AudioEngine/Utils/AEStreamData.h)

It was also possible to bypass the ADSP-System with a [few lines](https://github.com/AchimTuran/kodi/commit/f2d99e8ea612710c068beac0ca0249538d90d2a6). At the moment it's untested, but I will do some tests this week.


What's xsettingsd? I don't have it nor know it. Might warrant some comment in the source.


IMO this should return some default value.


@krakatoa I believe this code doesn't work. You should be using the `load_file` method and passing `yml_settings_path` as an argument.

If you need this gem to support empty files, and just deleting the file isn't an option, then please rewrite it like this:

```ruby
def to_hash
  hash = YAML.load_file(yml_settings_path)
  hash ? hash.deep_symbolize_keys! : {}
  # [...]
```


Ah, haha. Cheers. Sorry for the lacking investigation before pestering you with comments :)

Also, turns out I didn't run hue from master, but instead latest release. So that explains that too. Thanks!


Hmm, after the update I can't join the server anymore.
Can't paste the error log at the moment, but it is complaining about invalid packets.

Removing the playerdata seems to have fixed it, so it seems to be unable to load playerdata from before this version, maybe?


sql??????
please up sql


G000576 tested and working


changed references back to modern format and changed some of the /'s


tested and working


oops! Fixed and pushed a 1.1.1 release


which would be a `flatMap`. oops :)


Yeah, you need to add an alias to the referral relation. Sequelize will use the alias to alias the table in sql, so it cannot be the same as the parent table


They're not meant to be coupled to HTTP actually. The names are intentionally a bit generic. I don't want to cover 2xx-3xx responses for errors, and only a handful are applicable. Moreover, there may be error types that don't correspond to a HTTP status code.

I know this sounds hypothetical, but Fortune itself isn't built to be coupled to an application protocol, it supports HTTP through the serializers.


Sorry--I saw your comment come in, but it didn't show up in the PR so I had trouble finding it.

The `merged` dir will be part of the "exec" path that the actual container process will need full filesystem access to.  Only the daemon/filesystem driver will need access to the other dirs in the course of how CoW is implemented by overlay, and since the daemon, and therefore the context in which the filesystem driver runs, are both actual root on the host, no special work is required to make them accessible to another uid/gid.


Why do we want to change this at all? IMO all this DPI handling should be removed from here and instead fixed "in the right place".
You can call pango_cairo_font_map_set_resolution() on the font map. https://developer.gnome.org/pango/unstable/pango-Cairo-Rendering.html#pango-cairo-font-map-set-resolution
You can call pango_cairo_context_set_resolution() on any context (GTK uses this to implement per-screen DPI, I guess). https://developer.gnome.org/pango/unstable/pango-Cairo-Rendering.html#pango-cairo-context-set-resolution


These two lines are the only ones I'd like to take from the patch. If you don't mind, please make a new pull request with just this change.


Removal of `public` breaks custom BalanceStrategy implementations since the method is no longer visible.


yes, it got quite a bit longer than I had originally envisioned due to needing to handle migration of a pre-user namespace enabled daemon.  I think it would be cleaner to pull it out as a `setupRoot` subroutine.  I'll put it on the todo list.


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_kv/builds/2045


saw approval from cmeiklejohn
at https://github.com/basho/riak_kv/commit/4e232264dbdd846062d3728398c69290024c4702


merging basho/riak_kv/bugfix-2.0/jrd/drop-cover = 4e232264 into borshop-integration-1125-bugfix-2.0/jrd/drop-cover


basho/riak_kv/bugfix-2.0/jrd/drop-cover = 4e232264 merged ok, testing candidate = 93cf82fb


saw approval from cmeiklejohn
at https://github.com/basho/riak_kv/commit/405210dff8209dd254d81af47da9236f7881d76f


merging basho/riak_kv/bugfix-2.1/jrd/drop-cover = 405210df into borshop-integration-1126-bugfix-2.1/jrd/drop-cover


basho/riak_kv/bugfix-2.1/jrd/drop-cover = 405210df merged ok, testing candidate = 739bce1f


skuska commitu


95077770-HVACTemplateLoadManage-4493 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1656 of 1657 tests passed)

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


Could we assign "good" names to this instead of e.g. "12".? :-/


The previous version of this code is fascinating! I cannot comprehend what this was doing.


Ok


Well, you can use GLEW on Mac as well and so you have the same code base.

As I am doing cross platform development, I decided for one IDE - which is Eclipse in my case. Normally, when using Eclipse, just check out, build and the examples should run.

If I have time, I will look into your solution. Maybe I can integrate it somehow.

Regards Norbert


95077770-HVACTemplateLoadManage-4493 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1146 of 1146 tests passed)

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


Another good catch. This script is pretty rough...


Not entirely sure what's going on here. I would remove those metamarks, however, they're not useful or correctly transcribed in my opinion.


@iraj0711 you got me,fuck you


this class now needs to declare embeddedFieldToColumnName() method.


Oiniba joins the battle


Ooh nice and minimal with the raw HTTP server here.


Now you see how can fatigued mind play with you. (size - 68) is of course correct.
From the partition size of the disk i'm using with the installation it looks like i did fix it the right way, just wrote a buggy comment. Nice catch.


I only tested the sanity of the code by hand. Now I am fixing the format.
Will add test case soon.


Do not create new object on line 38. Let IoC to create this. Ranker implemets InitializingBean and ioc will fire afterPropertiesSet on bean ready.

To sum up, try delete lines 38 and 39 and use IRanker.


awesome!


Sorry, intention was to use
YAML.load_file.tap ....
Not, YAML#load


Put "2" in its own line with rend center, it's not really a superlinear addition, right?


Oiniba joins the battle !


Wow doing your own templating too.


On 05/26/2015 04:08 AM, mejiaa wrote:
>
> For the 'SUPPORTS_TRANSACTIONS' change, seems that is only used in
> Django v1.2.x anyway and only for the creation of test databases. It's
> not used in any future release of Django as far as I'm aware.
>

OK, I'll drop Django 1.2.x soon anyway, it's starting to be rather antique.
>
> Setting 'self.autocommit = True' is needed to satisfy Django's checks
> of atomicity for certain transactions (mainly model.save() and
> model.delete()). It doesn't actually tell the LDAP database to do
> anything differently, as you can see by the noop implementation of
> _set_autocommit(). The alternative would have meant that the end user
> would have needed to maintain atomicity themselves through the Django
> API. In Django v1.8, there are checks that will prevent atomicity
> failures.
>

OK, if it's required by Django's internals we'll let it be. I just
didn't want people to start doing "with transaction.atomic()" and expect
it to work.


:zap: Looking good.


Wouldn't it be more precise to say you can write in LaTeX (`.Rnw` files) or Markdown (`.Rmd` files)?


This is unsafe, createDirectoryIfMissing doesn't error if the directory already exists. If another user creates this directory beforehand which is entirely possible since it's a name you can know in advance that user will have the ability to remove and replace files in the directory.


keep the delspan


e quanto a do If? tá um pouco complicado pensar como fazer esses testes :/


These changes are intentional? If so, do you still need the URI-related code in generateCiteProcJSExportData?


That looks weird. I'll change it.


:+1: Buenisimo!


So is there a specific action item here?  David, would a conda update be
the correct solution here that would resolve everything?

On Tue, May 26, 2015 at 3:04 PM, Kyle Beauchamp <notifications@github.com>
wrote:

> Sounds good, we can add something to the notes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/choderalab/pymbar/commit/62ec3301d3f830fc0cc072d4910e049582fc4340#commitcomment-11371234>
> .
>


Hmm?  Maybe I'm misunderstanding what cl-flet, but json-encode-keyword is defined inside of the scope of ycmd--json-encode and then it's never called.  json-encode is called instead.  Seems like it's just dead unreachable code.

I just noticed this while checking for references to cl-flet in a larger codebase.  I don't think it affects the world either way too much if it stays. :P


Yes, looks good. Cross-page references won't be dealt with in the rendering, but it's good to have them encoded.


When I install monasca on Ubuntu 14.04 there was an error for the default falcon version.
It's newer than 0.1.9. so I test 0.1.8 and it works fine.
But I didn't find 0.1.9 since it's also an older version.


```
SSInterpreter.exe "(if (lt ((Number 5):(Number 8):[])) then \"Verdade\" else \"Mentira\" )"
```


Right - I would suggest to keep it in the main zone, but in its own line.


perfect :+1:


The changes relate back to https://github.com/aurimasv/zotero/commit/700e4a34240e4ce538a67e1521a2f4cb65e123fd but you're right that we don't need the URI code for now.


If we don't actually call `createElement` inside here, context won't go down, unless you know something I don't.

> What's even worse

Ah that's right, that's not acceptable. Care to make something else work?


rax retry tests


将博客放进来的话 那就太多了。。。不太好控制，还是分离开好点

Best Regards,
Frank Bian

> On May 26, 2015, at 9:53 AM, 迷渡 <notifications@github.com> wrote:
>
> 可以建另一个repo放优质博客。或者做一个小站，IT技术博客导航站。
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/justjavac/free-programming-books-zh_CN/commit/f5783af3b309bfc68e3f6eec163da489594c2d15#commitcomment-11355910>.
>


renamed as suggested in the code-review


Added extensive error handling. Please note that the FOR loop deliberately starts from index 1 so that it skips the first line of text (which is headers)


This is where it forwards the data on to the server.


Ah, makes sense.


Did one version where I only had one variable. Should have left it.  I got tangled up some where so changed it back.


interesting!


it would be nice to see the weapon names not hidden when "all slots" is selected

also, when "select a loadout each game" is selected it does not need to appear during insta or medieval matches


This is very clean and concise, good job Joseph!


For the 'SUPPORTS_TRANSACTIONS' change, seems that is only used in Django v1.2.x anyway and only for the creation of test databases. It's not used in any future release of Django as far as I'm aware.

Setting 'self.autocommit = True' is needed to satisfy Django's checks of atomicity for certain transactions (mainly model.save() and model.delete()). It doesn't actually tell the LDAP database to do anything differently, as you can see by the noop implementation of _set_autocommit(). The alternative would have meant that the end user would have needed to maintain atomicity themselves through the Django API. In Django v1.8, there are checks that will prevent atomicity failures.


freebaseidentifier


I think I was feeling like I should delineate the pairs.  Not good syntax tho.


where is the code project? not found!


where are you come from


This was when I set up the Repository.


How do I delete the file itself?

-----Original Message-----
From: sareg0 <notifications@github.com>
To: emblou2/skillcrush-104 <skillcrush-104@noreply.github.com>
Cc: Emily Blanchard <emblou2@aol.com>
Sent: Mon, May 25, 2015 2:21 pm
Subject: Re: [skillcrush-104] second pull request attempt (1dc12b6)

Hi @emblou2 I see that you have deleted the contents of the 'emily.rb' file, but not the file itself.

You committing the 'EmilyB.rb' file in the 'ruby' folder is the right thing to do, we just need to you to delete the 'emily.rb' file from the root folder.

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


nice work!


@vlj .Stencil_reflect.ppu.elf got broken here

https://github.com/vlj/rpcs3/blob/f85ee2d89525d7aecaebae01aac85f6fa749a8cc/rpcs3/Emu/RSX/D3D12/D3D12Buffer.cpp#L212


@bronson That commits was for focusing panes, this is for moving panes.


Not completely true - the `:value` key does not exist when trying to save a blank answer (I believe I test for this in the specs) - but `instance_of?` still works on null keys so it still works :)


thanks!


TeamCity NEventStore :: Raven :: Build [Build 32](http://build.prxm.it:8811/viewLog.html?buildId=2778&buildTypeId=NEventStore_Raven_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 64 Build time: 00:01:00


This is awesome.


Schweet


nice


exactly


You should check which property it's asking for and add a `case` statement.


Using the HTMLX editor for non-HTML files will be annoying, since it will try to make you use a different set of tags.  On the other hand, the XML editor does not support projection buffers at all (it hasn't been ported from the old pre-WPF editor APIs)

It might be possible for you to provide a custom schema that exposes the BimScript elements & attributes instead of HTML elements & attributes.  Look at Microsoft.Html.Editor.dll in your favorite decompiler or ask  @madskristensen for help.


I suppose I could have just pulled from the portfolio page if all I wanted to do is alerts, but if I want to also include purchase suggestions I'll need to pull from the main list too.


rax retry tests


one sets up the initial page, the other updates it after the user makes any selection


Comentario


Can you go on CircleCI and check if there is an error message?


I agree, but at the moment works for most cases and the rest I'll have to addess them later.

Thanks for the input anyway!


ナイスガッツ！


Awesome! Much appreciated, it's been on my TODO list for a while.

Could you please rename `lookAt` to `look_at`?

Will also need some tests added, if you could add some it would be awesome.
I think best way would be to take unit vectors of X,Y,Z and then rotate them by a look at and check that the vectors are facing the appropriate directions.
If it's too much trouble we can look at adding them later.


It was just one dev build from a few hours before the revert. Check changes on the CI if you want to know for sure.


Do you need the BI tooling? VS2015 with Roslyn will provide a much better experience.

You will need a separate parser / `ArtifactGenerator` (which would be exposed by a `ContentTypeHandler` for your new content type that inherits `Markdown`) that parses these different kinds of code blocks and passes the block kind to your new `ArtifactBasedBufferGenerator`.  It would then have an equivalent of a `CodeLanguageEmbedder` (you should not use the existing interface or implementation) that returns different wrappers for different kinds of code blocks.

Allowing custom references / `using` statements is mostly a matter of configuration.  You can parse `<import>` directives into a block kind that wraps in `using ` and `;` (this will give you namespace IntelliSense for free).


soso


keyima


Monkey patching is for chumps.


I think this mapping exists in every codebase I've worked on at EDH.


Pelo que eu to entendendo, o problema aqui é com strings, certo? Tu não pode usar `String` no programa em Scheme, `String` é coisa do interpretador, e de Haskell. Tu tem que colocar uma barra, `\`, antes das aspas pra o parser indentificar a string.


@rpastrana please rereview


Yay!

On Tuesday, May 26, 2015, Devyn Stott <notifications@github.com> wrote:

> feat(*): Add youtube extension src
>
> Add youtube extension source provided by
> https://github.com/showdownjs/showdown/pull/73 . Thanks to all the hard
> work done from others concerning this extension. @mehulkar
>
> Solves https://github.com/showdownjs/showdown/pull/73 No breaking changes.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/SyntaxRules/youtube/commit/0361b0fed58a8223a4603c24d4bcef156e63d5fe
>

--

------
Mehul Kar
@mehulkar


These unclear[reason=case] don't make much sense, but they don't hurt either so let's leave them alone for now...


Alright thanks! Do you have docs for that? Also, on a related note.

If I call `SomeModel.findAll()` and `Referral` is in `SomeModel`, than the `SomeModel` reference inside `Referral` is not expanded. For convenience, we added those to our tables and it seems like the eager loading should take care of that.


What do you mean "Referral is in SomeModel"?


PEP-8 requires a lot of refactoring of the format, I will do it after we integrate the alarm and notification


I edited my response above. Thanks for your help :)


Re your comment in the commit: I'll see if I can have them rendering without a strikethrough, but it's good to keep the del in the encoding


You're right -- the properties width and height properties don't change anything. We can remove them.


Ok, i'll check it. Looks better like this on 100%.


A few more details:

https://github.com/samsteam/algorithm-common/blob/master/src/common/Behavior.js#L7
if you use '@as': 'module' then '@static' is not required. Just define methods as usual. The module will export the mix as an object.

https://github.com/samsteam/algorithm-common/blob/master/src/common/Page.js#L19
https://github.com/samsteam/algorithm-common/blob/master/src/common/Requirement.js#L17
Configurable is intended to be a Trait. You don't need to add it with cocktail.use(Configurable); use() is intended to add annotations.

my $0,02!

Thanks for using cocktail!


Funciona el meter en la base de datos el favorito de las preguntas. no funciona el mostrar las preguntas fovoritas ni el desmarcar las preguntas


I think the namespace of this module is too high-level.

The name of the module suggests using it will give you thread magic, when the module in question just exports some utils that help with threading.

Like, I feel like it should be `Test::Stream::ThreadUtils` or `Test::Stream::Utils::Threads`.

Otherwise the name implies using it infers something about your code which it may not.


Хм, пропустил сообщение.

Можешь попробовать расписать временную сложность твоего алгоритма, считая, что у каждой строки длина `n`, а количество строк равно `m`. При необходимости можно использовать ещё и количество блоков в строке `k`.

Если испытываешь затруднения, скажи -- это вы можете не уметь делать.


Side note : this is rather useless if we don't have a test that enforces the respect of this format...


I've opened the bug here: https://svn.boost.org/trac/boost/ticket/11348


@Underground78: I've added two small commits. This one and https://github.com/kasper93/mpc-hc/commit/a2cc569c9a822ffd5b0c24771ebc10fbfbd871b9 Dunno if it helps for the #5436 ticket, but definitely make sense to remove extensive calls.


I've though about this PR for a while and think that you can avoid extra complexity with new function and flag by adding following LoC:
```javascript
if (this.isOpen) {
   this.props.onChange();
}
```
and
```javascript
if (!this.isOpen) {
   this.props.onChange();
}
```
(before `this.isOpen = true` and `this.isOpen = false`)
instead of `this.isChanged()` and extra flag `prevState`. What do you think?


The message presented to the user could be worded better.


Why is this even necessary?


place `"*"` please, don't freeze if not broken


develop (kbenne) - x86_64-MacOS-10.9-clang: OK (1143 of 1143 tests passed)

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


Evidently we never copy or move a `NetworkOPsImp`.  How about adding deleted copy members to `NetworkOPsImp` to document this fact.  This can already be deduced from the data member `mMode`, but that doesn't make for readable code.  Adding the deleted copy members would make it more obvious.


Please don't tag prereleases unless necessary. This means "0.3 prereleases or higher", not "0.3 or higher". Ditto for other entries.


This change causes passing the alternate data-directory and no-daemon args to fail when starting from within linux.


Other option is to not even have a rendering option. People have to `React.cloneElement` later if they want to do something differently.

Other ideas?


{break} should be here :smile:


I'm getting an API error after this (or next) commit, when posting with feed2red.pl:

```
PHP Parse error:  syntax error, unexpected ')' in /var/www/redmatrix/include/api.php on line 464
```


Good guess on the importing of a CSV file, but this does not actually import the csv's into your tables. try using: http://www.postgresql.org/docs/9.2/static/sql-copy.html


Hi! I'm glad I found this comment above, I was perplexed why the linter was still throwing errors for HTML5 elements. For what it's worth, symlinking tidy5 as described above (`ln -s $(which tidy5) /usr/local/bin/tidy`) did indeed make the linter use tidy5.

These instructions in the README.md:


On 05/26/2015 10:06 PM, Jiahao Chen wrote:
> Please don't tag prereleases unless necessary. This means "0.3 prereleases or higher", not "0.3 or higher". Ditto for other entries.

I see. Should I update and re-request immediately?


r u sure? ik was destijds voorzichtig met dit.


you need trailing semi-colons. killing me on thesre last few questions


Keep it, it's useful at the very least for the "reading" view.


@MaxXor , while you're working at this and if you have time, would you mind adding the option to remove entries from this form?  E.G. right click -> context menu -> remove entry


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2135 of 2211 tests passed)

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


[Deployment available online](https://s.brightspace.com/lib/vui/dev/7e3abe0b31512d0a108cc080b033601ef89e7b5b/valenceui.css)


lol I missed those semi-colons to piss you off

On Tue, May 26, 2015 at 4:09 PM, Max Blaushild <notifications@github.com>
wrote:

> you need trailing semi-colons. killing me on thesre last few questions
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LPayyapilli/wdi_5_sql_crud_quiz/commit/d53a6abbe79cb5f29f479b421ecabbd30264ba75#commitcomment-11372376>
> .
>


@o1da Set your git config to always rebase on pull - to prevent unwanted commits like this one. See: http://kernowsoul.com/blog/2012/06/20/4-ways-to-avoid-merge-commits-in-git/


I tried the following two command on FreeBSD 10.1 sh and Ubuntu 14.04.2 bash.

`if ! command -v > /dev/null 2>&1; then echo hello; fi`
This prints nothing.

`if ! command -v '' > /dev/null 2>&1; then echo hello; fi`
This prints hello.

Also I tried the following script `hello.sh`.

```
#!/bin/sh
run() {
  if ! command -v $1 > /dev/null 2>&1; then echo hello; fi
}
run "$@"
```

FreeBSD 10.1 sh prints `hello`.
Ubuntu 14.04.2 bash prints nothing.


This one ok but the other one I'm not so sure. Those ShowWindow calls just set the flags, they don't actually show the window so they make sense as they are.


I noticed it a second after pushing. It will be changed in the next commit.


95077770-HVACTemplateLoadManage-4493 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1656 of 1657 tests passed)

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


@JoelKatz I wrote that comment from what you told me


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2046


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_kv/builds/2047


Cheers


@Alx31 The ROM isn't of now, it was also there long back, but wasn't public !
Moreover we had came across to know that the ROM name has been leaked on XDA !

One doesn't easily decide any name, first they search and keep, the ROM's name wasn't there before, Before, This ROM was Project Tesla only which you're using Tesla Project !
And the older org was deleted, but after wards new org was made and here you can find it : https://github.com/Project-Tesla

@icoolguy1997


Yes, a conda update is what's most needed.

I've already fixed alchemical-analysis to resolve the immediate problem,
but it's still wonky to have the number of variables returned by
getFreeEnergyDifferences depend on the method of installation.
Specifically,

a, b = mbar.getFreeEnergyDifferences( )
works via conda install but fails if the install is from github, whereas
a, b, c = mbar.getFreeEnergyDifferences( )
fails via conda install but works if the install is from github

On Tue, May 26, 2015 at 12:57 PM, Michael Shirts <notifications@github.com>
wrote:

> So is there a specific action item here? David, would a conda update be
> the correct solution here that would resolve everything?
>
> On Tue, May 26, 2015 at 3:04 PM, Kyle Beauchamp <notifications@github.com>
> wrote:
>
> > Sounds good, we can add something to the notes.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/choderalab/pymbar/commit/62ec3301d3f830fc0cc072d4910e049582fc4340#commitcomment-11371234
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/choderalab/pymbar/commit/62ec3301d3f830fc0cc072d4910e049582fc4340#commitcomment-11372155>
> .
>

--
David Mobley
dmobley@gmail.com
949-385-2436


* TODO: Remove this and move to a view presenter


* TODO: Define a better validator for the repository, should ensure it is a valid URL and has .git at the end


* TODO: Move this to deployment controller


* TODO: Check for input, make sure it is a valid gitlab hook, check repo and branch are correct


* TODO: Check this is used


* TODO: Use a decorator pattern here


* TODO: Don't allow this to run if there is already a pending deploy or no servers


* TODO: Should this be an attribute?


* TODO: Check if this is used and if not then remove it


* TODO: Shouldn't this be a presenter?


* TODO: rewrite this as it is doing way too much and is very messy now


* TODO: refactor


:sob:


1640-AddHVACPart1 (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


oki


Actually I think something like throwing a DiagnosisMetadataNotConfiguredException makes sense for this case.


So ends Issue #59.


Same objection. Will delete the file and not replace it if `save` fails.


So version 3.0 was intended to be a major revision with some occasional API incompatibilities--the idea was that this release was the correct time for fixing any issues that might induce backwards incompatibility.


Very smart, I like it. I think less state the component has, the more predictable it will be.


@culmor30 For your info!


hi i found this thread to resolve my problem which is when i translate few things in prestashop and save, texts in modules disappear. I've copied those two files above and replaced them in my prestashop. Now i get errors. How to get back my shop?

errors in administration :
Fatal error: Class 'Translate' not found in /srv/www/stylovynakup.cz/public/www/classes/controller/AdminController.php on line 2366

error in frontend shop:
www.stylovynakup.cz

Fatal error: Class 'Translate' not found in /srv/www/stylovynakup.cz/public/www/classes/module/Module.php on line 1806

Any help guys?


Found it. The leading comma on line 463 should be deleted.


It's a light-weight replacement for gnome-settings-daemon etc.


Han,
    Making your program with certain coding style and write compliant code
are not that easy but they are a necessary thing to do. I would strong
encourage you to put these things at a very high priority.

Thanks.

Tong Li
OpenStack & Kafka Community Development
Building 501/B205
litong01@us.ibm.com

From:	Han Chen <notifications@github.com>
To:	TeamZenith/python-monasca <python-monasca@noreply.github.com>
Cc:	Tong Li/Raleigh/IBM@IBMUS
Date:	05/26/2015 04:02 PM
Subject:	Re: [python-monasca] Notification methods completed (f485c76)

PEP-8 requires a lot of refactoring of the format, I will do it after we
integrate the alarm and notification

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


For me we are inevitably coupling those errors with http errors. I mean when I get the error `405` and I look it up to the http error statuses I would expect to be named `MethodNotAllowedError` and not `MethodError`.

I'm just telling your this as an "external" user of your library trying to improve it, not criticise it :)


I know, but looking at this guy video it looks like it is moved after being activated for whatever reason.

I removed the other commit. I think this one should be fine even if it doesn't change anything.


The function is public, so that it could be used from other places - e.g. your custom theme.


What are the other DLS beamlines running?  The Model 2 PMAC driver?

A related question, what is the status of the  TPMAC support module?  Is the most recent version hosted at DLS, at GMCA at APS?  Could it be moved to epics-modules on github?  I have made changes to get it to run on Windows and sent them to GMCA, but I don't know if they have been put back into the distribution.
________________________________
From: Ulrik Kofoed Pedersen [notifications@github.com]
Sent: Tuesday, May 26, 2015 3:51 AM
To: EPICS-motor-wg/discussions
Cc: Mark Rivers
Subject: Re: [discussions] Update (4c5173a)

A "model 3" driver was also developed for the DeltaTau PMAC controller(s) by Matt Pearson @mp49<https://github.com/mp49> back in 2012. I think it was rolled out and is still running on a single beamline at DLS before Matt left.

Available in the tpmac module: http://controls.diamond.ac.uk/downloads/support/tpmac

�
Reply to this email directly or view it on GitHub<https://github.com/EPICS-motor-wg/discussions/commit/4c5173adfcdea33c75fe36e4e725903febc1d800#commitcomment-11359466>.


Fixed. Going for extra geek points. :-)


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

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


Makes sense.


There ain't nothing "pro" about this plugin. Just sayin...


Should not `no` without `use`


@JustArchi I have applied these changes https://github.com/ArchiDroid/android_frameworks_base/commit/3987eeacbb8568e4b195de2d795a54cde3baf8cc and these one in above commit and torch (from tile) isn't working. Any clue, do have I to apply anyother commit?


How much PHP do you know? Did you event look through the manual before doing it? I know this plugin won't work as of this commit.


i agree with d3agle it will be nice to add this


Awesome, thanks :)

I'll have to get that app updated some more at some point. Kind of holding out until the new Wear Maps API is out.


Well I wrote the code without warning on linux.


so why don't you just delete one of them?


Well yes, I don't mind this one.


I tried to install following the instruction but I am facing some errors.The errors something like

NAME ZONE MACHINE_TYPE PREEMPTIBLE INTERNAL_IP EXTERNAL_IP STATUS
ERROR: (gcloud.compute.instances.create) Some requests did not succeed:
– Invalid value ‘Test’. Values must match the following regular expression: ‘(?:(?:[-a-z0-9]{1,63}.)*(?:a-z?):)?(?:[0-9]{1,19
}|(?:a-z?))’

Could you help me to resolve this.

I am passing the following

python spark_gce.py ‘Test’ ‘3’ ‘n1-standard-4′ ‘n1-standard-4′ ‘/home/raghuram_mundru_meredith_com/.ss
h/google_compute_engine’ ‘us-central1-c’ ‘spark-test’


I know @yankejustin is working on a Registry editor but this would just be an ease thing


rax retry tests


thank you


This breaks the ability to start with the datadir and no-daemon arguments from within linux.


Fixed Issue #66


> Those ShowWindow calls just set the flags, they don't actually show the window so they make sense as they are.

In fact they do activate window.


Thanks for breaking Travis CI with this...?

Well, the output path should stay within project boundaries. The installation should be managed by a script or makefile called by the user. That's also why compilation and installation is split up for everything else.

But, well, that's just my opinion.


Great, so if you'll add this commit to PR, I'll merge it ;)


Thanks for the quick response!


Sorry I meant to 'Startup Manager'


unreachable code below here


Mobile GH sucks, can't edit.

With "everything else" I mean stuff as big as mono, but also stuff as small as qsstv


This was missing from the last commit. As explained, the CommandBindings of the Backstage are not resolved correctly when Window.Content is not a framework element. To circument this, the bindings are added directly to the AdornerLayer.


@thibaultCha ?


Is this why my entire library suddenly started re-downloading absurdly at dash 1080p using ffmpeg to merge when it was simply pulling highest native non-dash (720p)?  I'm desperately looking for this code change it was after 4/19 and I would like to see the exact code to determine and validate myself a reliable workaround.


Documentation updated with example and fixed the grammar section.  Done in 7bb59c0797452c693998454f004a2617de678968.


See the chat.


I find this passage a bit too unreadable. Suggestion:

```
or zip archives with the .sublime-package extension
located under XXX (short: XXX)
or any of its subdirectories.
```


Blame Visual Studio for putting this instead of an absolute path


nit: this could be written as `return pack.LockedVersion != "" && pack.LockedVersion != HEADString, recencyInfo, nil`


@marcotuts -- tried using these label/value styles which (I think) are defined, but not seeing anything different on the page.  Can you take a look?


Hmm, don't mean to harp on this, but you may want to allow a union type since I believe the property can be a string if passed in by the caller:

```js
overrideSpinnerClassName: React.PropTypes.oneOfType([
  React.PropTypes.string,
  React.PropTypes.bool
])
```


Thanks, Microsoft!


@CollinChaffin Read the last two paragraphs in the [FORMATS SELECTION section](https://github.com/rg3/youtube-dl/blob/23905927e19280d9217ecad377ef26ea9d5793fe/README.md#format-selection) for the instructions for restoring the old behaviour.


I think deleting code to raise test coverage might lose me my commit rights :D

more seriously, the transformations package seems to come from elsewhere,  I didn't want to edit it.  Also, having a readable python version is valuable as it is the "official" language of the project.


@marcotuts -- here's the inline layout mod I added to push the Signatories panel down a bit below the Details section.  Can you move into a Sass/CSS class?


wait, did we loose the client-sessions entry?


Merged, thanks! Please base further patches upon devel branch.


Can you please merge this commit to 2014_R2?


According to [this](http://eslint.org/docs/rules/global-strict.html) the `global-strict` rule is deprecated in favor of `strict`


This repo looks great! My favourite part is that you committed every step individually - good habit to get into!


@jaimeMF Thanks!  I can't believe that change wasn't made requiring cmdline changes, rather than without warning such a drastic change to default behavior!


@Bouckaert @isabarros

fiz este commit bem pequeno (só consertando uma spec que tava falhando) direto no master. Tranquilo, né?


Vs must be more anal about those things :p


fs


is will => will

But I'm thinking, if one of them is ignored, "taking precedence" doesn't make too much sense. An idea:

```
only the one in XXX will be used
(the other will be ignored).
```


I cannot force xcode to link w/ GLEW.. (that's a ridiculous linking error for a lib they don't use since they implement their GL drivers)
![screen shot 2015-05-26 at 22 29 34](https://cloud.githubusercontent.com/assets/658719/7822622/27a768d2-03f7-11e5-976c-699e68e1a565.png)


How about making 3.0 beta available via conda? Right now it's 2.1 beta.
Certainly at the very least the README.md needs updating to note some of
the backwards-incompatibility which is now present.

(The immediate problem in our case was caused, I think, by Kyle or someone
else making a 3.0-compatibility adjustment in alchemical-analysis; then
when this was split off, anyone installing alchemical-analysis with conda
pymbar had a version of alchemical-analysis which wouldn't run properly. It
took me a while to track down why it wasn't working for a user when it WAS
working for me...).

On Tue, May 26, 2015 at 1:17 PM, Kyle Beauchamp <notifications@github.com>
wrote:

> So version 3.0 was intended to be a major revision with some occasional
> API incompatibilities--the idea was that this release was the correct time
> for fixing any issues that might induce backwards incompatibility.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/choderalab/pymbar/commit/62ec3301d3f830fc0cc072d4910e049582fc4340#commitcomment-11372524>
> .
>

--
David Mobley
dmobley@gmail.com
949-385-2436


It breaks compatibility with Python 2.6
```python
parser_copy = self.__class__(self.argument_class, self.namespace_class)
TypeError: __init__() takes at most 2 arguments (3 given)
```


do ==> and do


@lmr right, if you guys agree on that I'm fine with it.


The doc is pretty unclear about this... They have two different defines for the same value but with different descriptions so I'm not sure what to think. Anyway it cannot explain the problem (if the window is maximized, the next call will just do nothing) and I'm not sure it makes sense to remove those from here because they mostly restore the window size.


I'm always dumbfounded by how context works in 0.13..

>Other option is to not even have a rendering option. People have to React.cloneElement later if they want to do something differently.

Maybe we can do this (if it helps?). And when 0.14 comes out, make it minimal dep, and then add `renderComponent` because with 0.14 context changes the context should just propagate no matter where you `createElement`.


Finally,


@danpovey: I am not actively working on it this very moment, but I am planning to rework it in the coming couple days. There aren't any short time constraints, are there?


Ça sent le fdp.


changed the shining of the planet from {0.0f, 0.75f} to {0.1f, 0.9f}


or => or for


@manleo ololo :)


rax retry tests


Test sur commentaire


@jaimeMF You're a friggin lifesaver!  Did not go back to re-read all the damn docs when I've been using it so long and when I finally did, I honestly missed it but judging from other folks comments I know I'm not alone I still think a new cmdline for this would been better idea I do see value to new behavior - just not what I personally want right now!


Reviewed!


sure, I did this in a hurry...


En el init pones self.lista_cartas = listado_inicial.
Le cambias el nombre acá?


Align equals, i.e;

```
$dateFrom    = new DateTime($datefrom);
$newdatefrom = $dateFrom->format('Y-m-d');
```


`$datefrom,$dateto` should be `$datefrom, $dateto` see https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md#44-method-arguments


[95394984] Implementar testes do cenário


No, it's fine, just wondered.
Dan

On Tue, May 26, 2015 at 4:35 PM, Kirill Katsnelson <notifications@github.com
> wrote:

> @danpovey <https://github.com/danpovey>: I am not actively working on it
> this very moment, but I am planning to rework it in the coming couple days.
> There aren't any short time constraints, are there?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/5d2cc33b614294c36aa6e7e0e5619d69e9ca7aaf#commitcomment-11372834>
> .
>


That is also an option, although it is hard to give this one a good name


wat.


Spacing could be better here.


Remove comment


s/Product/Products


set(self.lista_cartas) funciona bien cuando se trasta de listas, pero lista_cartas no es una lista, es un objeto. No estoy seguro de que funcione bien esta línea.


Align `->` with line above


No hace falta el "else"


It is also possible to give no response or add a request body (for validation), so it is very hard to specify. I think this one is the best


No tenes que pasarle self.nombre, porque la funcion despues hace jugador.getnombre(). Por ende le tenes que pasar una refencia al jugador, no un string!
Self hace referencia a sí mismo, o sea, un jugador
return self.pedidos.pedir_carta_a_mostrar(self, cartas_a_mostrar)


pas bien ;)


Yup. Why would you otherwise create a rule, since that is what critter is for. Also, if used wisely a rule is created for a specific scenario (usually a test) and after exection of that scenario it should be removed, as not to pollute the system. And it is pretty anoying if you add one manually that you have to enable it manually in another screen.


Not sure this comment is necessary.


Spacing and alignment.


Yeah I *think* it should work with `{ "stage": 0 }` in `.babelrc`.


agree, but it is not really fixed. It uses a template form (just like delay) is. After creating the rule, it can be modified,extended just like a generic rule. right?


The ratings will need to be updated when OGRS is adopted.


Adding a comment to this row.


I would do this on 1 line


When you say web, does that just mean desktop?


Indentation


Donde declaraste "movimiento"?
Se te va a complicar declararla si de entrada le pasas los resultados de las lanzadas directamente a "mostrar_dados".
Deberias guardar de alguna manera las lanzadas para usarla 2 veces, una con "mostrar_dados" y otra donde obtengas la suma (la suma es la cantidad de casilleros a moverse)


This is actually not preferred way. Testing `#save` method is testing ActiveRecord method which you have not written. Always focus on the code you write only.

There is only 1 case where you would use `#save` and that is if you would have written a callback that handles lets say for example `after_save` stuff. To test the callback you have to trigger that `#save`, but again that is because you'd have written the code for the callback.


Noice!!!!


s/Product/Products


so it is more a convenience thing.  I also thought about first editing a rule bit by bit and then enable it once ready. It is a different workflow. We could have the initial template say it is enabled by default. If the editor (me) want to disable it upon save,it can be changed. right?


Call Recording Issue (X-s version)
Hi, I tested Call Recorders two different Rom's MIUI V6 version 23/5 Color OS (wuxianlin version) & CyanogenMod M11 Official (X-s version) But Official X-s Version has bit cracking voice opposite party voice call but wuxianlin version is Very clear call records both the Side without losing live call quality ... is it possible to change call record modification setting ????? anyone have an IDEA?>pls suggest me


Same feedback as for Video saving.


testing


True, but we had a discussion about it and came up with this one. If you
feel something else is better I have no problem with that. As long as it
does not have 'recording' in it :-)

On 26 May 2015 at 22:46, Ernest Micklei <notifications@github.com> wrote:

> agree, but it is not really fixed. It uses a template form (just like
> delay) is. After creating the rule, it can be modified,extended just like a
> generic rule. right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/salandur/critter/commit/faa9dd61de9150d37d47597a53c5efe2ec8eb8bb#commitcomment-11372994>
> .
>


All looks good, just some minor coding standards changes required.


Applied this.


self.pos_inicial? No sera self.pos?


Oh right... grr... didn't think about this very much. Yeah, I'll return it to PropType string and make default "" and check if the override prop isn't an empty string.


Well there isn't really anything new being written to the player data between the old and new version, nor have I changed anything with the sending or receiving of packets.


Certainly. But that is already an option in the XML template. And it is not needed for the delay and the other new rule form, you fill them in correctly there.


Mike was here. ;-)


Remove empty line - it makes it clearer that class is intended to be empty.


yeah this function in custom functions will be droped


Ah true! Want to send a pull request? :)


si_syscall was added in OpenWrt and also upstream in uClibc by now. See https://github.com/cjdelisle/cjdns/pull/777 for details.


[Deployment available online](https://s.brightspace.com/lib/vui/dev/25c00faf4c626f30d66cf2602c5dfc04f3b194b8/valenceui.css)


tested and working


Published to http://ui.edev.d2l.com/62_94ae9d54123ee14a45f04ac98cf4cf9c960a3abb


Сковороду надо тоже от `list` унаследовать, кстати (или же это у меня мания)


i don't think it's a good idea -- using numbers from 0 to 15 is conventional for such colorscheme

but it's possible to define "aliases" before, like
```lua
local white = 7
local bright_black = 8
local bright_blue = 12
```


Why??


I think we need to fail if the duration is too long.

It implies a user mistake if the duration exceeds the periodic cycle rendering the reboot window moot, we should probably disallow it.

/cc @crawford @marineam thoughts?


I liked this block. It made sure the template was only parsed once while in production mode.


ladfk


Could our implementation throw an exception here instead? Our implementation of Future is not respecting the method's [specification](http://docs.oracle.com/javase/8/docs/api/java/util/concurrent/Future.html#get-long-java.util.concurrent.TimeUnit-) since we do not have a timeout mechanism. Therefore, it seems disingenuous to simply call and return `get()`, even if the timeout was grossly exceeded.


tenes que pasar un iterable a mostar_mano en lugar de un objeto.
Pïsta: podrias pedir el str de lista_cartas y separarlo por las ","


Released a 1.1.2. Hopefully a longer-lived release :)


Otherwise this cannot be used in our work project.


This has 2 spaces in it, and the anchor below has 1. I wonder if this will break the link.


@mpc-hc: Do you mind if I push this while at it?


acá estas obteniendo las no vistas y no haces nada más. Deberias mostrarselas al jugador
*self.pedidos.mostrar_listado()*


We might as well call this "rev" to match its URL. The other controllers all match.


Dear X-s ,, I like Maxx audio too please tell me how to add in Xs_Offical 23/5 build  and Solve the problem call rec also,, see you bye


Docstring goes at the very top of the module.


Go ahead.


Sure go ahead.


Mike was here. ;-)


Hey Sam, can you give ideas on how we can randomize the shape of the field but also how the game can detect if an entity is within the field, on the boundary of the field, or outside of the field? I was thinking that we could draw a straight line out from the entity and see how many times it intersects the field boundaries to test if it was inside or outside the field, but I am afraid that that would require every point on the boundary to be known. Is there an efficient way to do this?


this should be indented once more


The chai expect has a different api than the jasmine expect.  Pulling this in makes all of those existing tests fail.


func setFileCreateLabel(mountLabel string) error {
+	runtime.LockOSThread()
+	defer runtime.UnlockOSThread()
+
+	return label.SetFileCreateLabel(mountLabel)
+}

Doesn;t this unlock the thread as soo as the SetFileCreateLabel returns?  If yes, then this is wrong.  We only want the thread unlocked after the

 			v, err := createVolume(bind.Name, bind.Driver)

And the followon SetFileCreateLabel("") is called.


flattery will get you nowhere


shavot


[Deployment available online](https://s.brightspace.com/lib/vui/0.10.5/valenceui.css)


Boa @Victor-R! Eu ainda escrevi seu nome errado =)


Looks good thanks!


Why is this bit needed?


Te devuelve *None* cuando el jugador no tiene ninguna carta de las propuestas, deberias mostrar la carta en caso de que sea distinto de *None* y agregar un *Return* para terminar el turno (;


I set a breakpoint before `ShowWindow(SW_MAXIMIZE);` and it did maximize and activate window. One would think after reading doc that it wouldn't activate window.

Anyway, could you push translations? Because I changed one id name and it need manual update.


@sagikazarmark according to PSR-2, the visibility needs to be declared.


Not sure if this the cause but now have problems again with pushing a YouTube video.
It doesn't clear what has previously been played.
So even if there's nothing playing any more, the last played gets restarted again and the pushed youtube video gets put into the queue


I thought I did? I can check again when I'm home in an hour.

Vidur

On Wed, May 27, 2015 at 2:22 AM, Richard B. Kaufman-López
<notifications@github.com> wrote:

> Ah true! Want to send a pull request? :)
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/vyder/imma-let-you-finish/commit/de7e9dde451acb7ac43a18d04d2abefeab891874#commitcomment-11373099


The id would probably auto-fix because Transifex is clever. :) However I have things that need manual update anyway so I'm doing it now.


Good catch


Nice work.


> I set a breakpoint before ShowWindow(SW_MAXIMIZE); and it did maximize and activate window.

No wonders here. `SW_MAXIMIZE` and `SW_SHOWMAXIMIZED` is both `3` :)


It isn't an important point, I would personally prefer if this class was made immutable. By this I mean make each of the fields final and make the initialization of `parameters` and `AnnotationKind` happen during object construction (rather than lazily).


Looks good!


number *of* characters?


@samabed Remove this?


So sorry to hear about that, Maddy. Hugs to your family.


Yes, it's what I meant but here it doesn't really show the window. Rather it makes it visible in the taskbar but that all. After I click on it, it shows maximized.


This might need some fleshing out. Are we still going with the term "Releases"?


Nice, I think the tables were getting too cumbersome to style


Mike was here. ;-)


In Biochemistry/reactions.master.tsv there are 12,146 reactions out of 34,701 reactions that have a status that is not "OK".  Are there any conditions where the "not OK" reactions are used?  Should SOLR only include "OK" reactions?


We can just use `with closing(open(repo))`, especially since this is only needed in one place. (It also takes care of wrapping the close() call in try/finally.)


Aha, more than, in some cases in script uses model id's instead entity id's... I think model id's config was blended with entity id's config... I will fix code in resource.cpp to model id's and and next will try to fix script...


Hey @frodsan, note that @seppo0010 found that later versions of the msgpack gem break Ohm due to their use of str8, which in turn exposes a bug in Redis lua-cmsgpack: https://github.com/antirez/lua-cmsgpack/pull/44.


We'll have to keep an eye on this to make sure adding a linebreak doesn't throw off some naive piece of the deployment script. It does look like there's a strip() in there.


Shoving Ajax'd HTML directly into the DOM isn't generally something we do - we normally use JS templates (hence the Mustache) via JSON. Feels like since this is breaking that convention we should take it to wider discussion, particularly with regards to accessibility. cc @edds @dsingleton


Please, add a test case involving non-ascii characters.


Also an empty name.


Package version comparison will be based on CPACK package version variables.
Applied change to RPM based as well.  Tried on OS X and Windows but it doesn't make any difference since the saved version information does not include build/sequence/release information.. Actually on Windows when download the package from browser "~" is changed to "-". So we will keep original "-" on OS X and Windows.


As @JohnnyCrazy said, our MenuItems have different callbacks depending on how you activate them (select with num5 or change with num4 & num6). Someone forgot to invoke the callback for when you change the value of the scroller. The callbacks could be used to implement custom behavior that uses the values returned etc.


Nice...


[Deployment available online](https://s.brightspace.com/lib/vui/dev/29a98c1109e12c92c7b82e19ce2f44f8e621a0da/valenceui.css)


Try to add more explanation in commit messages. What was added? Why was it added? Does it fix things and if so how do we know it fixed them? Does it add new functionality? What is that functionality and why do we need it?

Trying to do this will make it easier for you to go back and debug things that went wrong and understand why.


This is gonna have to be in single quotes or our annoying linter will get all :angry:


As discussed before, whilst we do not want to use "not OK" reactions in
metabolic modeling, they are kept in the biochemistry for purposes of
record-keeping, i.e. we want to be able to merge future generations of the
public databases, and we'd want users to see what we did with the publicly
available reactions.

If a user has a KEGG reaction that is "not OK" and they are looking for it
in the website, we should probably make it available for them to find, but
with the emphasis that the reaction is "not OK" for modeling purposes, for
the sake of transparency. An empty hit can mean any number of things.

On Tue, May 26, 2015 at 4:08 PM, Mike Mundy <notifications@github.com>
wrote:

> In Biochemistry/reactions.master.tsv there are 12,146 reactions out of
> 34,701 reactions that have a status that is not "OK". Are there any
> conditions where the "not OK" reactions are used? Should SOLR only include
> "OK" reactions?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ModelSEED/ModelSEEDDatabase/commit/f395d6845ad87955236143996fc85d5c2f703230#commitcomment-11373384>
> .
>

--
Asst. Computational Scientist
Mathematics and Computer Science Division
Biosciences Division
Argonne National Laboratory
9700 S. Cass Avenue
Argonne, IL 60439

http://www.linkedin.com/pub/sam-seaver/0/412/168
samseaver@gmail.com
(773) 796-7144

"We shall not cease from exploration
And the end of all our exploring
Will be to arrive where we started
And know the place for the first time."
   --T. S. Eliot


What's wrong with the `renderComponent` that was used before? I don't get any errors/warnings with that (dynamic-segments).


Added.


95077770-HVACTemplateLoadManage-4493 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1651 of 1651 tests passed)

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


Thank you very much. It has been a pleasure to be of assistance! - The new version is very welcome.


derp. thanks


`filter_from_fields` is the same as what's in `ItaOfficeLocationQuery`.
They're both overriding a `filter_from_fields` from `Query`.
The one in `Query` isn't being called anywhere--maybe these two should be the default?


Dummy scheduler is usable only for debugging, it's very unlikely that somebody will use it for real life workload.
So it's better to add configuration option for default scheduler.


Somehow this doesn't always work. Even though I've increased the sleep-time to 4 seconds. I guess you can't really predict it and sometimes it takes the server longer to stop. Is there a way (script-wise) to wait for it?
@gsand


あとストリーミング接続時、切断時にToastを表示するようにした


It seems there is no much sense for these static variables - basically you just compile strings from 2 or 3 static strings below.


Haciendo mi aporte


Too comples IF statement, the initial version was more convenient.


I am currently re-implementing this because executing `router.navigate` to call router methods to switch views is problematic; I think I am instead going to listen to `Events.UserDidNavigate` and switch views in a callback (in AppLayoutView or NavigationCollectionView), then within the same callback trigger `router.navigate` to update the URL.


Remove this?


looks like you have a typo!


We'll need to update the links to the API documentation and overview page.


+1, will do this tonight.


I think it would be a good idea to do InterlockedIncrement on the content of the page - to make sure that it is actually dirty and the mprotect below has to do work.


TeamCity xBIM :: Xbim Geometry :: Release [Build 71](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1745&buildTypeId=XBIM_XbimGeometry_Release) outcome was **SUCCESS**
Summary: Tests passed: 70 Build time: 03:56:55


TeamCity xBIM :: XbimExchange :: Build [Build 345](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1746&buildTypeId=XBIM_XbimExchange_Build) is now running


Thank you!!!! :100:


Can you add a space before each comment (like it's done for the other comments)?


`required=True` can be removed because providing a `default` implies that. Add `allows_blank=True` and I have a hunch that the problem will be solved. fingers crossed


I'd put the `pragma` here and not later


TeamCity xBIM :: XbimExchange :: Build [Build 345](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1746&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:03:01


TeamCity xBIM :: XbimWebUI :: Build [Build 54](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1747&buildTypeId=XBIM_XbimWebUI_Build) is now running


Thanks @MaxXor , You Are The BEST :+1:


I think it's better with the pragma in the outer loop


TeamCity xBIM :: XbimWebUI :: Build [Build 54](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1747&buildTypeId=XBIM_XbimWebUI_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:34


TeamCity xBIM :: Xbim Geometry :: Release [Build 72](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1748&buildTypeId=XBIM_XbimGeometry_Release) is now running


This looks like a crutch and there is no reason to process default server group in special way. Intentionally there should be default rule (also created implicitly) which works for default server group. This should work as basic routing: if a packet doesn't match to any specified rule, then it's routed by default rule specified at the end of the rules chain.


Does the code between the line 520-550 should be updated? like in my commit 350bbb7f856b31f6ce3692a4c0517d2aab0e2766 ?


I did the same thing - my bad, I didn't account for the job queue. Thanks for catching this.


Good catch


is this easier than document.createElement('tr') and appendChild()?


Gotcha, thanks!


The else() case is actually not for models that aren't adiabatic, but for times when the adiabatic conditions aren't initialized yet, so maybe you could change the comment to something like "so we need to convert the depth to pressure if we already have the adiabatic conditions, or else use the depth itself"


Update POI fixed


That's a very clever approach. Seems more reliable.


@Trikolon try https://github.com/gsand/mark2/blob/27d9851ee597fd2a2592e6f7cebaebd9f854fffc/samples/start.sh


Good man! Also, thanks for fixing my poor test!


does this make the route in the parent app `/callback`? we probably want to namespace this to avoid a conflict with say OAuth callback routes.


This can become `<xsl:apply-templates select="node()"/>` then.


More coding on patch https://github.com/natsys/tempesta/commit/c1be712a73adca11c49939a2d95b3687fb964fba .
The initial patch is wrong.


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2127 of 2213 tests passed)

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


Also add a test for strings containing NUL in the middle. Perhaps change the logic to find first non-NUL character from the end. Although unlikely I think we should allow NUL in the middle to match the PE/COFF spec literally.


s/IMG2_OPTS/IMG2_SIG ?


$ git clone https://github.com/plusjade/jekyll-bootstrap.git yiluke.github.io$ cd yiluke.github.io$ git remote set-url origin git@github.com:yiluke/yiluke.github.io.git$ git push origin master


ugh ok


ditto


We need to validate that "not OK" reactions really are excluded from modeling and we need to make it clear in the search interface that a reaction is going to be excluded.  The users I've worked with are confused by a status of "CI" or "MI".


yeah, you are right! I only fixed the route because it wasn't showing up.


go ahead, I only have to push the


Resolved issue #13


mooie test, x


Yes, would be good to reorganize along these lines. Not for this PR though.

On Tue, May 26, 2015 at 9:39 PM, Guillaume Martres <notifications@github.com
> wrote:

> shouldn't we have subdirectories pos and run inside tests/disabled ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dotty-staging/dotty/commit/0c4ca21dafd067af3d2953356b57ffb2f30093c3#commitcomment-11371844>
> .
>

--
Martin Odersky
EPFL


Hi Mike,

1.) Regarding MFAToolkit, I plan to maintain the MFAToolkit repo indefinitely, and that is where I will do all dev (the primary code base for MFAToolkit). I�m happy to add you as a dev in that repo. I will copy updates from the MFAToolkit repo into the ProbModelSEED repo as needed. I know git submodules could achieve the same purpose, but I feel that those complicate deploys needlessly. This mechanism just seems cleaner and more explicit to me.

2.) Regarding GLPK, you should not compile GLPKapiEMPTY.cpp, because that gives you a nonfunctional MFAToolkit. For now, just compile against version 4.48 of the GLPK. Ultimately, we need to resolve this and update our use of the GLPK API in the MFAToolkit.

Chris

On May 26, 2015, at 10:05 AM, Mike Mundy <notifications@github.com> wrote:

> Chris, can you clarify what is going on with the MFAToolkit. Will the version in this repo be the primary version? Will the MFAToolkit repo still be updated?
>
> Also, note MFAToolkit has a dependency on the version of GLPK. In version 4.49 from April 2013, GLPK removed support for the deprecated lpx_ APIs. So version 4.48 is the last version that has support for the old lpx_ APIs. The latest version is 4.55 and many systems have a current version of GLPK installed.
>
> I worked around the dependency by building with GLPKapiEMPTY.cpp.
>
> �
> Reply to this email directly or view it on GitHub.
>


SO NICE
![omg](http://i.giphy.com/MziKDo6gO7x8A.gif)


Dude, you broke this

``Error: validate_re(): "573BFD6B3D8FBC641079A6ABABF5BD827BD9BF62" does not match ["\\A(0x)?[0-9a-fA-F]{8}\\Z", "\\A(0x)?[0-9a-fA-F]{16}\\Z"] on node``


This could be reduced to:
```
return locationActionsForURLBar(urlBar).map { $0.accessibilityCustomAction }
```


This resolves issue #12, not #9


Fine with me :)
On May 26, 2015 10:55 PM, "Loy" <notifications@github.com> wrote:

> Made looking and reseting head goals optional because if messed up the
> RoboNurse challenge. That behaviour was introduced by @reinzor in
> https://github.com/tue-robotics/robot_smach_states/commit/197f4ff4a227c1fe6788c4036a495495c4625f75#diff-98ba1c9b09e928f465f5c9158d067575R78
> but is not composable as you see :sad:
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/tue-robotics/robot_smach_states/commit/eacb642feb3cb09114cfb3543241deec156b3a0c
>


Try going to a site like usesky.co


Thanks! Seems to work flawlessly.


Right click on an image or link


You duplicate some code here (what you do inside of the loop is the same for both cases), so you could reduce the lines of code by defining one variable (like number_of_phase_transitions) which you set to either transition_depths.size() or transition_pressures.size(), depending on what is used, and then you can use that for every loop in this function.


Shouldn't this be depth?


Multiflag?


rax retry tests


https://github.com/Efreak/node-steam-chat-bot/commit/f0f3aca1c61422c0e7360b7d5bdb4a9aa2e4abec fixes the breakage. You left out a bracket.

I'm guessing you put the variables there for debugging. Defining variables outside of functions is useful for that, and I doubt Codacy would care.

As far as splitting and requesting every word, what I meant was that there's no check to make sure it's a valid URL first. On the other hand, I'm fairly sure that request does such checks on it's own, so (like I said) I'm not changing it. It's not terribly efficient, but I'm not here for efficiency right now.

On a side note:
* Apparently I need to look at my editor settings, it keeps removing the trailing lines (instead of leaving them alone), which makes looking at diffs annoying.
* I really wish github's diff function ignored indentation and other spacing changes when there's no changes in the actual code.


in the commit message, "also checks fro the" ?


Thanks, but you need to reference *which* GLSL spec you're referencing. Different versions can move things around. (even though they don't seem to in this case)


thanks a lot


This name is interesting, but can we have something more instructive? Optflag? I am bad at naming too, any suggestion @jonboulle


well let's refactor it:

```py
if not os.path.isabs(filename):
    filename = os.path.realpath(os.path.join(data_dir.get_root_dir(), filename))
return ", ".join([_.strip() for _ in open(filename) if not _.lstrip().startswith('#')])
```
It should do the same... (alternatively we might check for `os.path.exists()` if we want to avoid OSError when the file does not exists.


I don�t think we can absolutely confirm that �not OK� reactions are excluded. If they are needed to complete a model and make it work, we must use them. The status is just there as a reference to problems with the reactions. Ultimately, it is the �template� that decides what does and does not go into a model� not hard and fast rules...

If a reaction really is garbage and should be excluded from ever being used in a model or template, then it should be flagged as �non-core� in the SOLR table...

On May 26, 2015, at 4:25 PM, Mike Mundy <notifications@github.com> wrote:

> We need to validate that "not OK" reactions really are excluded from modeling and we need to make it clear in the search interface that a reaction is going to be excluded. The users I've worked with are confused by a status of "CI" or "MI".
>
> �
> Reply to this email directly or view it on GitHub.
>


optional nit: change `60` to `60.`, if that's what you meant


Remove this?


Also, why am I the author of this commit??


Alternatively,the objects could have a law of motion set up so that it only
stays inside with out there being an explicit calculation for detecting
in/out. Idk how exactly that might work though.
On May 26, 2015 16:59, "shy401" <notifications@github.com> wrote:

> Hey Sam, can you give ideas on how we can randomize the shape of the field
> but also how the game can detect if an entity is within the field, on the
> boundary of the field, or outside of the field? I was thinking that we
> could draw a straight line out from the entity and see how many times it
> intersects the field boundaries to test if it was inside or outside the
> field, but I am afraid that that would require every point on the boundary
> to be known. Is there an efficient way to do this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wl-acsl/toasty-penguin-party/commit/094d326031aa5e2aaee2e959fa4281bc688e75a1#commitcomment-11373217>
> .
>


What is the intention here? Are non-existing scripts supposed to be a fatal error, or not?
The code looks like a mishmash of the two. If the script doesn't exist, find(..) will throw an exception. The if statement then goes on to check for the script being NULL, which will never happen.
Either the if statement should be removed, or find(..) changed to search(..).


While I suppose the point of this issue is literal spec adherence, NUL in the middle of a section name is pretty pathological.  If you think it's worth doing, it's no trouble to actually implement.


Nit: `var` -> `let`