Once again this chaining should in the task model as a scope

```
def self.completed
  where(complete: true)
end
```

you can then just call:

```
@tasks.completed
```


Add-on Documentation/Performance & Monitoring/Blackfire.md

I think those would be great to reference here. Something like: After adding the add-on everything is set up for you. The next step is to set up the [browser companion](https://blackfire.io/doc/web-page) and learn how to [analyse your first profile](https://blackfire.io/doc/first-profile).


Why not `Radius :: Point -> Bool`?


Right, done.

-[Unknown]


We don't need this, as we are mocking the entity.


Good to know. Be careful using this if you've already pushed commits to a fairly active public repo, though. It effectively destroys the old commit.


I see.


once again you can just call

```
<%= render @patients %>
```

as long as you have a _patient.html.erb partial


Okay, I left a bunch of line notes on this, sorry if it was annoying, but I'm curious about some of your decisions, and also about a handful of Haskell Things.


@sagartalla, When click to play is enabled, then the IFrameSaver javascript code no longer works.  I didn't test it but I suspect the logging code `LoggerUtils.prototype._logInteractiveEvents` doesn't work either.

I reverted this commit, please make a pull request for this work.

You can get some sense about the IFrameSaver with this test interactive:
https://concord-consortium.github.io/lara-interactive-api/iframe.html
It will show a log of the messages being sent from LARA to the interactive.


We don't need to mock the BaseFieldDefinition


@drindt not sure why a divider should be clickable or enabled?


Ok, I'll make a new PR when have a little bit more time.


In order to not break the backward compatibility, I am going to add a new argument called flags to `dd_create_skeleton` which could be used to control used function for creating the directory and I am also going to introduce a new flag DD_CREATE_PARENTS which will force `dd_create_skeleton` to use `g_mkdir_with_parents` and use it from `dd_create`.


No problem for me


Snyggt! Men borde vara
```
if (...
```
inte
```
if(...
```

https://github.com/jcmandersson/Operation/commit/6d98570def2753cb92558f41323e594008bfa3ca#diff-fe34f0c8f0cec90a3b40eca51cbada0fR286


The Divider items are OOTB clickable here.


There is no need to mock the EntityStorageBase, we are not working with the storage of the entities, just the alias storage.


Suggestion: ".. ingestion task ends and you should be .."


This would be better as a separate PR since it's not related to adding the profile.


We only need to Mock the entityManager for the getDefinitions method. The getStorage and getBaseFieldDefinitions are not needed.


Alla variabler borde ligga under `this.data`
https://github.com/jcmandersson/Operation/commit/faedda6a710412c4d71fa9ee02109411f0b4a613#diff-fe34f0c8f0cec90a3b40eca51cbada0fR18


Good work :D

A simple question, is normal that led notification don't work on vibration mode ? :/


You mean ``nbunch=list()``?

Still it is an equivalently bad like ``nbunch=[]``.


Yes, I agree it would be helpful to have a brief comment for tests, especially for those accepting or returning params. I will add this to the new updateGroup() test and we can then also start updating others with the new submissions. For historical reasons all of them do not have any comments.


Thank you.


OOTB?


Because that means less restrictions for our users!
I like to keep my code as generic as reasonably possible.

But you're right, all this iterator stuff is ugly.

I tried to find an alternative:

```Python
def _split(value):
    """Split input/output value into two values."""
    if isinstance(value, str):
        # iterable, but not meant for splitting
        return value, value
    try:
        valuetuple = tuple(value)  # convert iterable to sequence
    except TypeError:
        return value, value
    if len(valuetuple) != 2:
        raise ValueError("Iterables must have exactly two items")
    else:
        return valuetuple
```

I guess this is easier to read.
It has 7 lines less and it might have the exact same behaviour (at least extremely similar) to my original code.

Would this be better?

Even with dropping support for non-sequence iterables I couldn't fit it into less lines:

```Python
    try:
        if len(value) == 2:
            return value
        else:
            raise ValueError("Sequences must have exactly two items")
    except TypeError:
        pass
    return value, value
```


Comment out for now until we have a feature page for it. Ideally, if we can link to a spec page for the spread operator or MDN page in the interim that would be great.


The username is only collected if the `require-login` addon is activate (it's disabled by default), and it stores the username in the `tix_username` field.


What about just toggling `self.glSnapshotView.hidden` instead of changing the view hierarchy? That's what I meant by safer, since this operates under the assumption that the snapshot view is above all of the accessory views, right?

Or else we can put in assertions that the view order is as expected, to codify them.


Add instructions on how to enable the feature. As it's behind flags in M44, we should mention what flag needs to be used and maybe link up to how to run Chrome with flags for those that are new to this. We should also call out the Chrome version this is in.


This should probably be implemented as a field addon (e.g., `CampTix_Addon_URL_Field`) rather than build directly into the shortcode.


wow dah jago split


It would be nice to mock the method of AliasStorageInterface that actually creates the alias, and to use the expect method on it to actually test that it's being executed.


This should use `esc_attr()` to prevent XSS.


This should use `esc_attr()` and `esc_html()`.


Thanks to Michael Barz for reporting


Yes, I tried your fork in order to understand the events that were triggered and why my "click" was not triggered, but I give up and i will use the "mouseup" event on my button. I don't understand why "mouseup" is fired and not "click", but it will do the job


1570-EnergyPlus-8-3 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


I just tried it with the sample app. nothing happens if i click on the divider. and i think this is the correct implementation.

So what do you want? What do you need? And on which version and phone do you test it with the wrong behavior?


it was pushed when phasemask wasn#t changed


Spread source LGTM.


looks like fail


u request every sql with this issue now?


We can just remove this if there is no dependencies. As far as I know, no one actually use it.


On 2015/04/15 16:25, Bryan Drewery wrote:
> This still seems more scary than it probably is. Do we even need to show this if it is recoverable?

I'm thinking that trying to /upgrade/ a repo schema at all is probably
pointless.

Given that routinely updating the repo data consists of dropping the
contents of the repo DB and reloading, you might as well just go one
step further and drop the entire repo DB then recreate and reinstantiate
the schema each time, which will pick up repo schema changes automatically.

	Matthew


Use logger, and probably fail the build


1570-EnergyPlus-8-3 (kbenne) - x86_64-MacOS-10.9-clang: Build Failed

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


:+1: to exposing dark little nuggets of :hankey: to the world.


would be a little bit more secure if you would use `fetch('name')` here. Also you could provide some meaningful label if you have nil here.


If you are developing using `devtools::load_all()` and have never installed the package you could call this function on a package that is not actually installed.

`devtools::load_all()` seems to have some conflicts with `S4` methods at times, so it doesn't work that great with Bioconductor packages, also it seems usually the test code assumes the package is installed.

Regardless of the above it is probably safe to assume the package is installed.


FileUtils.deleteQuietly is probably better here.


You can replace all the will($this->returnValue($value)) with willReturn($value)


With this change, I would expect that tests would be failing. Are you seeing that, @dan-f? This change appeared in my PR as well, and I just removed it.


You can replace mocking the language with $this->languageManager->getCurrentLanguage(), this is already mocked in RulesEntityIntegrationTestBase::setUp()


It's a general one, so it's nice to support different charset.
http://docs.oracle.com/javase/7/docs/api/java/lang/String.html#String(byte[],%20java.lang.String)


The comment is not really useful - should be updated to refer to the method.


duplicate


This comment is not related to the method.


幸运的是有一些网站可用来搜寻色系。以下是其中一些的链接及主要功能介绍。叙事美学博大浩瀚，它们可以帮你做出更明智的选择。本节末尾将评介两个桌面版的付费工具。

我觉得这段第二版改得有点脱离文本了……

Fortunately a number of sites are available (for us) to explore color themes. 这里应该指的是在网站上搜寻所需的色系。sites 和下一句的 links 对应，应该是不包含最后两个 desktop tools 的。

... that will help you more easily navigate the seas of storytelling via aesthetics 说实话这个 storytelling via aesthetics 说法挺奇怪的，直译是用美学来叙事，我凑合理解成叙事美学了…… 这个 sea 的比喻实在保留不能，只能用“浩瀚”来暗喻，这样 navigation 直译似乎也比较奇怪，替换成实指大概是“指明正确的方法(道路)”之意，差不多可认为 正确的方法=明智的选择吧


The same danger: default arg-values are bind during import-time, so invoking this twice with no `edge_del` and you will get double the nodes to delete.

In general i would judge that the use of default arguments throughout the code is more prevalent than necessary.
Default args is a facility to the programmer, but if abused, nasty bugs appear.


FIX 1. Correction of nodename `Debtor_FindByName`
FIX 2. Correction of nodename `DebtorHandle`
FIX 3. Error in search results with a single search result


ack


it should be `size = options | 0;`


Actually, do you mind if I do that after this is merged? There are a bunch of places throughout the code that use `DateTimeProperty().wrap` where they could now use `iso_string_to_datetime` and it'd be nice to do those all at once, without adding that to this already-big PR. Just added it to my personal cleanup backlog


commit 5c25eab makes abrt-hook-ccpp using `mkdir`


Can you check that the bundle created in 'target' contains the right files ? As well as the bundle located in target/wisdom/application after the integration test execution ?


readme


You are simply re-declaring the user which is already in $user, this just adds more overhead. If it failed the fist time, it'll just fail again re-declaring it unless it was a rare fluke.


1570-EnergyPlus-8-3 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


What can I do to help release this code? Looking to use this feature as soon as possible. Thanks!


Metti a posto le indentazioni Luca cosi, pur essendo una prova, diventa molto più semplice intuirne la struttura ;)


1570-EnergyPlus-8-3 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


in an ideal world we would check for changes, but you know


r+


Noes!


doesn't follow codestyle


Thanks for taking the time to review this change.  I have added braces on the ```else```.


Go ahead.


There's a syntax error in the class definition. Also, the 'summary' attribute isn't supported anymore in HTML5; I'd remove all references.


Seems it needs further investigation. The use of ClickListener or something else in conjunctions seems leading to this misbehavior.


``def define_random_network()``, correct?


The code for each field is too specific to be generalized in a template method. Also some of it modifies members of the current class.


this was commited before the phases change


1570-EnergyPlus-8-3 (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)


If you try in the sample app the form, it crash when show the UIAlertController


stop spam useless comments, see the timeline


This shouldn't be here!


Please fix the extra link.


Follow the style of the code (ie one per line).


The better solution is to omit this empty anchor, and add the `$name` as id attribute to the table itself. Anchors with name attributes are relics from HTML 3.backinthedays ;)


We had the same issue, solved different.


15- Mina Mouris <br/>


@nknize , thanks for the fix. Will this go directly into 2.0.0, or can we expect an intermediate bugfix 1.5.x release before that?


They are not placed in the source directory any more, they are placed in the build directory. I altered the include search for Cythonize to add the build directory. So those .pyx files are being picked up from the build directory.


ack


It does, but it's nice to avoid writing a new one when you don't need to


the bundle contains all the right files in the 'target' and in bundle located in target/wisdom/application as well.


This break is unnecessary. If there's a padding / margin issue, please let me know so I can take a look at it and solve it with CSS :)


r+


The argument I added to CYTHON_ADD_MODULE_PYX is:

 -I ${PROJECT_BINARY_DIR}

to tell it to look in the build directory.


@jdm This is a left over of a refactoring I did after the review comments. It's weird because I didn't see any compilation warnings. It's gone now!


I like this ;)


I'll try it later.


Hmm @Jake0oo0 I think you broke what this was intended to do:
> remove trailing and double slashes + other junk

Will add this back in #116.


Change the `ph` flag to `H`


Make sure that you see my answer on trello: https://trello.com/c/wr0ndACB/69-todo-find-out-why-do-the-contributions-from-the-phone-appear-with-toolbar-search-as-a-source-in-the-word-list


please head over to https://gitter.im/mikepenz/MaterialDrawer and we can discuss this there.
If you need some special behavior you can just add it as your own custom  DrawerItem to your project. in fact it's just the DividerItem so i'm still not sure about the onClick stuff you are talking about.


Either lowercase account or uppercase Information :)


I figured it out. It was my mistake, but I also made a small patch: edenhill/librdkafka#243


The `statement if condition;` syntax is more concise and readable--use it when you can!


Was this +1 just for the name or for the whole patch?


ack


will correct and update by tomorrow.


NESTOOOOOOOOOOOOOO


Cm11 I think
On 15 Apr 2015 9:08 pm, "jgcaap" <notifications@github.com> wrote:
>
> Question, in which place you found this fix? Thanks !
>
> Btw i am managing to make GCC 4.9 work, only bluetooth missing, soon will
share :)
>
> —
> Reply to this email directly or view it on GitHub.


Moved this check up to where the options are evaluated immediately after parsing. Future checks for incorrect command usage should be in the same region of the code.


Correct. Add the HTML with instructions on how to try Pyramid. I also included the two assets, prism.css and prism.js, which should add syntax highlighting. Someone who understands the machinery of webpack, npm, and whatever other voodoo is going on within the architecture will have to figure out how to include those assets. My stupid way of doing it would be to edit the template, but that is the wrong way. @blaflamme probably knows how to do it the right way.


for the build is boost higher than version 1,49 needed.


From a semantical viewpoint, I think this information in this table would be better off in a definition list. You can use the default [Bootstrap definition list](http://getbootstrap.com/css/#horizontal-description) for that (`<dl class="dl-horizontal">` etc.).


Very minor thing: we prefer the ``{`` on the previous line. (For anything except functions.)


I'm guessing this header comment may need to be updated with the new behavior


Removed appending hash keys to arrays. By keeping three sets of the data, you're tripling the overall memory usage. Not a big deal for small lists, but unnecessary, especially since we will only be printing the arrays anyway, and not modifying or reevaluating them in any way.


Does your gemspec need to be updated to reflect this?


No need for the if check, btw. ``talloc_free(NULL)`` is ok (just like ``free(NULL)``).


why bulk? do we need old build logic anymore?

I think it should be names "scopeFilter"


Yes, as soon as I obtain my project supervisor's approval, that there's no need for alteration in the source code to avoid any possible issues with intellectual property law.


one line docstrings must be in one line as per [PEP 257](https://www.python.org/dev/peps/pep-0257/#one-line-docstrings); also, for consistency, always use """triple double quotes""" around docstrings.


Ну как бы нет, это два разных символа, один из которых разрешает рвать строку, а второй — нет.


No reason. Switched to lite


`or die $!` is a common way to escape code if your open command doesn't work as expected.


Vet inte vad som är merge och vad som är nytt. Ser att en del fixades för att följa kodstandarden, men några ställen glömdes :)
* https://github.com/jcmandersson/Operation/commit/213d0f85419bde0e45d5b5da2179b0ae82a03340#diff-a494f640e7497ea4b9efad3470ea7ba3L83
* https://github.com/jcmandersson/Operation/commit/213d0f85419bde0e45d5b5da2179b0ae82a03340#diff-b650441988d58d47097ed9b88a30eee3R290
* https://github.com/jcmandersson/Operation/commit/213d0f85419bde0e45d5b5da2179b0ae82a03340#diff-230835f14bd3d4e984a32e9be341c0deR76


Se ha añadido la primera plantilla de "proyecto" donde se debería poder imputar horas a ese proyecto desde un usuario.


But is this really needed?
Networkx provides a similar one.
Document what is the difference with it (if any).


I understand this is the function signature you have in PAMBOX, but for SRMR it makes no sense (and leads to calling `predict` on SRMR being weird, since you're only interested in the second parameter). I think a better choice would be to have the `mixture` signal as the first parameter and have `clean` and `noise` as optional, named parameters. That way, one could call `srmr_obj.predict(noisy)` but call other methods as `other_algo.predict(noisy, clean=mycleansignal, noise=mynoise)`. What do you think?


Use of minutes convertd to milliseconds looks good -- thanks!


Here we're manually formatting the text returned by the help command to make it a bit prettier.


0\_255 在选取不同深浅的特定颜色时非常好用，所以很适合制作连续色系。任一颜色都有30种深浅可选，并可直接拷贝为十六进制值。用户可先从一个很大的颜色网格中选取初始色，0\_255便基于该色生成一系列深浅色。如果你在网格中找不到想要的颜色，也可以输入十六进制色值来选取自己的颜色。

这里提供一个直译。我觉得改版讲的更清楚，不考虑拘泥原文的话更优。


use upgrade step to profile 11 here; there is no need to create a new one.


Looks like this does the job well, but I'm curious why the StrictVersion class wasn't used.

Cheers!

Dan


The name. Is this ready for another round of review? I can take a look today.


見にくいだけだし、SPAM大作意味ないから不要だと思う


Seems fine, so long as it doesn't break any other tests


先改到这，全文其它部分其后看……


Well the user login would work perfectly without passwords! But you
implement it for security reasons ;-)

Best regards
Am 15.04.2015 15:45 schrieb "Vincent Swarte" <notifications@github.com>:

> @Kovah <https://github.com/Kovah> Could you maybe clarify why this
> RewriteBase is here? It's causing some issues when the installation is not
> in the root directory, but everything works perfect without it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/InvoicePlane/InvoicePlane/commit/358ec6cde8b892b4fd2ec797de4b00d4ce2c64ac#commitcomment-10736129>
> .
>


Doumentation for each class


スクリーンショットなど

HipChat使い方ページに誘導

（Redmineにあるやつをポータルに移した方が良いかも？）


I fixed the JS part to match the PHP logic and the actual translated string ;)


Failing how? Related at all to https://github.com/KyleAMathews/superagent-bluebird-promise/issues/18


OK. Will add replacing a member to the test.  FYI: I did test adding and removing members before but outside the demo test. It works well


Will check that.


Builder extends BuildFactory -> names is missleading


リンク貼る


Hey! :<


On Wed, Apr 15, 2015 at 08:24:19AM -0700, Park Ju Hyung wrote:
> Erm, I'm actually stuck again.
>
> I've rebuilt the entire toolchain with latest HJ's binutils with LTO patches and enabled --enable-plugins from the GCC, but I still don't have 'ld-plugin' binary.

ld-plugin is just (HJ's) ld. I use a different name usually for it because i
don't want to override my system ld. You can create a symlink or copy the binary.
Just make sure to not sure gold.

-Andi


Does this system object need to be attached to the link object?  I think it's always accessed through promise


This breaks running `blanket` for me.
The blanket docs suggest:
`mocha -r blanket -R html-cov > coverage.html`

But I believe there is an issue with the args as there are no results in `coverage.html`. Should I be doing something different now?


These should just be made static. No need to add an initializer.


Line too long; and needlessly convoluted.
We are not building an obfuscated javascript library.
Simplify all such code and
please make sure everything fits in 80 chars!
If not, split it.


if I'd taken the time to read your pull request, I might have. but I ended up ditching this too, and just going to straight tuple comparisons. it's just cleaner that way, IMO.


I think we had some earlier discussion about using the TEST-UNEXPECTED-FAIL syntax in a console statement. In this case it looks like it would be possible to just throw with a descriptive message (which will probably cause any relevant tests to fail).  It's going to throw on the next line anyway


Debug write alert


I'm just glad to have the issue fixed.  Thanks.

-------------------------------------
Dan Gentry
dan@dashdrum.com

On Wed, Apr 15, 2015 at 11:12 AM, John Comeau <notifications@github.com>
wrote:

> if I'd taken the time to read your pull request, I might have. but I ended
> up ditching this too, and just going to straight tuple comparisons. it's
> just cleaner that way, IMO.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jcomeauictx/django-crispy-forms/commit/a0e1aecf2790d1cee9bab4c7f64b7edfa6d4a6c2#commitcomment-10739548>
> .
>


Oh I see you use it in releaseSystem()


This comment should include `system: System` in the Links object


weird. well I'm deleted temp.js now anyways. Thanks for the hershey headsup!


lolol


Cards were not used in the original demo. Happy to put the styles in if we use them


*Disabled* | 0 = Global
*Disabled* | 1 = Side
*Disabled* | 2 = Command
3 = Group
4 = Vehicle
5 = Direct
*Disabled* 6 = System


There will be an intermediate bugfix in 1.5.x


@stevepiercy yes I'll take care of that, but it seems prism is not included in npmjs so I'll probably just use another one I can install from npmjs and make it works as expected


Then I guess I'm back to square one.

Do you have any idea why LTO fails on latest HJ binutils whereas
https://www.kernel.org/pub/linux/devel/binutils version works fine?


On my build system, there is vchi_initialise in bcm_host library, but unfortunately cmake does not find the library in /opt/vc/lib. If I copy this lib to /usr/lib, it works.
So, what is missing on my system?


Thanks for the update! I've been looking for some of these changes. :+1:


Please use `CommandUtils.destroyRequisition` here if possible


What's wrong with this??

```
results = node_attr['function'](*args)
if n_output > 1:
    results = [results]
```


mdr


This overall seems fine.  It's a little scary that we need to remember to call CommandUtils.destroyRequisition instead of req.destroy() but it seems like this isn't going to be added in many other places.  And I'm guessing the test harness will catch a leak automatically if this is forgotten?


计划先使用tqeditor，URL：http://www.e512.net/tqeditor/demo.htm


r+


:sparkles"


I've read the [docs for signIn()](https://developers.google.com/identity/sign-in/web/reference#gapiauth2googleauth). The only parameter supported is 'scope'. I commented the rest out to ask you whether to rip it out for good.


Maybe I'm being pedantic here, but I think it should be the other way.  Like they go in order of parsing:

1. First haml is parsed, creating a blade file
2. Then blade syntax is parsed, resulting in a php file
3. Then the php is rendered

Like in the normal haml.php, haml is first, then php.

Thoughts @trupedia ?


Could be done easier:
class CompileScope(val name: String)

object MacroSope extends CompileScope(MacroScope)

we can use also case objects here.

I think that "Compile" in names is not needed


hay un error que se produce en la consola del servidor es este:

[15/Apr/2015 16:22:57] "GET /guardar-pregunta/?id=888 HTTP/1.1" 500 11209
Traceback (most recent call last):
  File "/usr/lib/python2.7/wsgiref/handlers.py", line 86, in run
    self.finish_response()
  File "/usr/lib/python2.7/wsgiref/handlers.py", line 128, in finish_response
    self.write(data)
  File "/usr/lib/python2.7/wsgiref/handlers.py", line 212, in write
    self.send_headers()
  File "/usr/lib/python2.7/wsgiref/handlers.py", line 270, in send_headers
    self.send_preamble()
  File "/usr/lib/python2.7/wsgiref/handlers.py", line 194, in send_preamble
    'Date: %s\r\n' % format_date_time(time.time())
  File "/usr/lib/python2.7/socket.py", line 324, in write
    self.flush()
  File "/usr/lib/python2.7/socket.py", line 303, in flush
    self._sock.sendall(view[write_offset:write_offset+buffer_size])
error: [Errno 32] Broken pipe
[15/Apr/2015 16:22:57] "GET /guardar-pregunta/?id=888 HTTP/1.1" 500 59
----------------------------------------
Exception happened during processing of request from ('127.0.0.1', 53454)
Traceback (most recent call last):
  File "/usr/lib/python2.7/SocketServer.py", line 595, in process_request_thread
    self.finish_request(request, client_address)
  File "/usr/lib/python2.7/SocketServer.py", line 334, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/usr/local/lib/python2.7/dist-packages/django/core/servers/basehttp.py", line 129, in __init__
    super(WSGIRequestHandler, self).__init__(*args, **kwargs)
  File "/usr/lib/python2.7/SocketServer.py", line 653, in __init__
    self.finish()
  File "/usr/lib/python2.7/SocketServer.py", line 712, in finish
    self.wfile.close()
  File "/usr/lib/python2.7/socket.py", line 279, in close
    self.flush()
  File "/usr/lib/python2.7/socket.py", line 303, in flush
    self._sock.sendall(view[write_offset:write_offset+buffer_size])
error: [Errno 32] Broken pipe
----------------------------------------


you'll need to mod the bluetooth and camara app gcc with these: http://forum.xda-developers.com/showpost.php?p=60129913&postcount=540 =))
btw you forgot to answer my question in case you can help out =p


creo que es por que el en el view no retorno nada...


Geef deze ff met een pull request, geen zin om het over te typen XD


I believe the main purpose of this test is to check whether the ScreenNameValidatorFactory registers the specified screen name validator class correctly. It doesn't really test the validator itself. That's why all the test does is to check that the methods are being called from the right validator class.


@sam4215 no..

@exstar check to see if certain classes exist. :/


no he podido mandar datos desde el servidor porque el objeto seraliazer al parecer esta descontinuado para la nueva version de django asi que no puedo escribir los json's


@jamierocks I was thinking about that.


El boton para guardar no acepta el css de dojo, si se lo pongo me bloquea el actionperformed, debo trabajar en el combo y en los otros campos para guardarlos


Typo: s/ahs/has


How dare those browsers break on broken HTML.


typo automatically


Ticket #59


Make sure it's gone from the Cargo.toml files too, and add any changes to Cargo.lock as a result of building after making that change.


Сделал пример, действительно работает, когда писал v.2 не работало.
Тогда может еще так упростить: /\s*\u00a0{1,}\s*/ (не проверял).


I said crappy because I haven't seen any of the major ones break on this :P


The full explanation:
http://mymonalisasmile.com/upload/comments-votes/comments-voting.html


Tony,

Note, this is not a "conversation", there was no "other half". Please view
the original *comment on code*, in context here:
https://github.com/Gilroy-Morgan-Hill-Devs/stats-tracker/commit/94d1f74fcbd1c45928d86affdac467962aeda37e

The emails you received is how github communicates that there was a new
comment to those interested in the repository. If you'd rather disable
these notifications, you may have to "unwatch" the repo by going here:
https://github.com/Gilroy-Morgan-Hill-Devs/stats-tracker

clicking on the "Unwatch" dropdown, and selecting "Not Watching"
(screenshot attached).

Also note that your replies to these emails are being logged in github as
further comments on code and notifications are sent to the rest of the
group.

-Nesan

On Tue, Apr 14, 2015 at 1:58 PM, tonybabb <notifications@github.com> wrote:

> Naasen,
>
>
>
> Understood and I agree that receiving email notifications of half a
> conversation are pretty useless, which is why I raised the question. What I
> don’t understand is why I’m not receiving email notifications of the other
> half, is it something I can/ should fix or is there something in the group
> settings that is causing me to only see half a conversation?
>
>
>
> I think that adjusting my email filters will not fix this problem as all
> they can do is react to the email I receive when surely the problem lies in
> the intent of the sender and my notification settings. Thanks for the links
> to notification types. I took a look at these and it appears that from
> others in the team I am only receiving “Participating notifications” but
> from you I’m receiving “Watching notifications”. Am I interpreting it
> correctly? Certainly I didn’t knowingly set it up this way. Surely I, or
> any individual in the team, should be receiving entire conversations or
> none? Where should I look in github to fix this, assuming that this is
> something under my control.
>
>
>
> Thanks for your advice.
>
>
>
> Tony
>
>
>
> From: Nesan Waran [mailto:notifications@github.com]
> Sent: Tuesday, April 14, 2015 11:05 AM
> To: Gilroy-Morgan-Hill-Devs/stats-tracker
> Cc: tonybabb
> Subject: Re: [stats-tracker] fixed homepage rendering bug, created
> skeleton games app (94d1f74)
>
>
>
> These emails are exposing my comments on a Pull Request <
> https://help.github.com/articles/using-pull-requests/> . Github will send
> comments to your email address based on your notifications <
> https://help.github.com/articles/about-notifications/> .
>
> You should see a link "view it on GitHub", which will take you to the
> comment, in context. In my opinion, these emails are generally useless
> without clicking on the link to view in context. Adjust your notifications,
> or email filters accordingly.
>
> —
> Reply to this email directly or view it on GitHub <
> https://github.com/Gilroy-Morgan-Hill-Devs/stats-tracker/commit/94d1f74fcbd1c45928d86affdac467962aeda37e#commitcomment-10720022>
> . <
> https://github.com/notifications/beacon/ADt2-tLOyEynjT0SAJzh9ngPrd71uATRks5n_U48gaJpZM4D_jmE.gif>
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Gilroy-Morgan-Hill-Devs/stats-tracker/commit/94d1f74fcbd1c45928d86affdac467962aeda37e#commitcomment-10724005>
> .
>


Granskad!


Insread of GamePiece! in all addGamePiece methods, do

if let  piece = self.currentPiece {
   self.addGamePiece(...)
}

This way is safe. The other was could crash.


@weotch What parts should be switched exactly?

The register method? That it's like before?


init


FYI, this redirects with a status code 302 Found - but in this case it looks like 303 See Other would probably be more appropriate.


#### Achtung!
- Metti sempre un `<!DOCTYPE html>` prima di `<html>` *per essere conforme agli standard*;
- Il motore di layout non è tenuto e **non** deve visualizzare a schermo il contenuto di un tag `<head>`, tantomeno questo deve contenere solo metadati, CSS, JS e titolo della pagina e/o simili;
- Stando al W3C, `<head>` **deve** contenere un `<title>` e un `<meta charset="...">`;
- Tutto ciò che non è nel tag `<head>` va nel tag `<body>`;
- Il tag `<center>` *è considerato obsoleto* in HTML5, la cosa rammarica anche me perchè lo consideravo comodo... comunque per mantenere la compatibilità e consigliabile usare CSS equivalente;
- Gli stili inline con `style=""` sono spesso considerati come *cattiva pratica*, dato che rendono HTML meno leggibile e non separano bene il contenuto dalla vista, questi vanno piuttosto messi in CSS.

**Se vi va**, quando ho tempo modifico questo esempio in modo da fargli passare il test <http://validator.w3.org>, vedrete che risulterà anche più leggibile, **se mi permettete** provo anche a metterci un po' di JavaScript, cosi sapete come fare nelle prossime pagine :smiley:.


It would be good to keep this docstring up to date and describe the new ``slug`` param


Lambda nested in lambda


It would be cleaner to separate the logging output from the tex output (this saves the user from hacking around later). Perhaps use VerbatimLogger for the "Sorting in order of age ..." output?


Teste


we need to add the code


errornousScopes


Suggest [highlight.js](https://highlightjs.org/static/demo/). If you agree, I'll take a stab at finishing off this task and PR.


Just tested it! This is way better! :)


@stevepiercy yep already selected it, doing the work on a branch, I'll push to let you see


Why does it need to be as broad as `*`?


I get errors in depended scopes (compile). It should said from which projects this scoped came from.


@stevepiercy you should have done a branch off the main repo, we would have been able to work on the same branch


this is fully documented in the cfunction documentation below (the abstract type `Ref` has special behavior in the context of cfunction and ccall argument lists). perhaps it would be clearer if we wrote the mycompare function to only accept arguments of type `Cdouble`?


yeah. it would be preferable actually.


def instead of val that is function


@srdee looks pretty good - just a few fixes needed I think:

1) the fix-misc.pl script doesn't do the numbering right when the line numbering continues across the xml hierarchy (or you have to specify a flag not to restart numbering but I'm not sure if that works either). There are 11 lines at  //l[@n="1"] that need to be fixed

2) can you change the xml:lang value on text to "grc"?

3) There are 21 line numbers which need the alphas changed to .1, .2 etc (to work around the limitations in the CTS 3 implementation) //l[matches(@n,'[a-z]')]

4) I'm not sure if we will have problems with the different nesting levels of the lines or not (some are at /text/body/div/div/div and some at /text/body/div/div. I think we can leave it for now and use the // in the xpath of the citationmapping but eventually we might need to fix this further.

Thanks!


The extra opening bracket is causing a build error:
```
OMXAudio.cpp: In member function 'bool COMXAudio::PortSettingsChanged()':
OMXAudio.cpp:252:5: error: expected ')' before '{' token
     {
     ^
OMXAudio.cpp:263:14: error: 'audioDest' was not declared in this scope
     OMX_INIT_STRUCTURE(audioDest);
              ^
```


It is override everywhere so why we implement it there?


Manque Mayotte, La Polynésie française, la Nouvelle Calédonie, Wallis et Futuna.


Generally I think that we BuildXXUnits and BuildScopes should be refactor. We should discusse it tommorow


Yeah, I'm proposing we revert this commit, that the file extensions **should** be `haml.blade.php` rarther than `blade.haml.php`.


Aaaa


This is gross, but so was FilterActivePods honestly, and this is the only place we use this function, and I plan to replace with a dict lookup when the informer is stable.


realizar commit


Try now.


New Readme


@balmas

1) doing that now! (but see 3)

2) done

3) will do--but keep in mind that some of these predate my intervention, more will be required and they all require some guesswork that may be wrong--a lasting issue and more editorial intervention will certainly be necessary

4) that's how it is in the inventory for now--will probably have to stay this way since the divs are used for so much qualitative editorial content, unless we switch them to milestones if they're not part of the canonical citation scheme

...will commit another version shortly!


Might be better to not call the parent method and move all the response preparation logic into this child method.


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


Thank you for seeing that, I corrected it.


saw approval from jaredmorrow
at https://github.com/helium/libhelium/commit/5ed48869aae769e4d9e667c6dc2017cbc504052f


merging helium/libhelium/bugfix/newer-openssl-cmake = 5ed48869 into he1000-integration-64-bugfix/newer-openssl-cmake


helium/libhelium/bugfix/newer-openssl-cmake = 5ed48869 merged ok, testing candidate = 8f987d14


TeamCity xBIM :: XbimExchange :: Build [Build 221](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1400&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:04


Yay, milk.


Check https://github.com/JuliaStats/RCall.jl/pull/50


Feel free to send me a patch


Doesn't this code allow only an array of instances to be passed?
https://github.com/appcelerator/arrow-orm/blob/master/lib/collection.js#L28


Also created a layer that, when selected onto an object, will make them clickable without making them collide with the player. We can create a menu and hud and buttons using this layer.


I agree - I pointed out that very same thing in another commit (probably earlier in the sequence).


some tests failed:
failure: http://buildbot.helium.io:8010/builders/test-libhelium-build-centos-7-64/builds/33
failure: http://buildbot.helium.io:8010/builders/test-libhelium-build-debian-7-32/builds/25
failure: http://buildbot.helium.io:8010/builders/test-libhelium-build-debian-7-64/builds/33


Can you uncomment this part. Officially it is the only option, but we still need to pass these through for when they are supported as options.


Ok, then your comment here:

https://github.com/BKWLD/laravel-haml/issues/5

told me the wrong extensions. I was just checking, if they are working as expected and made the commit for it.

Actually I find the first one cleaner, because with the blade.haml syntax, you've still syntax highlighting for haml out of the box in your editor, which is pretty nice with your new adjustments to the package.


Great, added it, and thanks for the heads up. Feel free to submit a PR when you create the NCG package.


Isso aqui não devia ter acontecido...


extra line break intended?


re the as and bs in the line numbers, yes, we'll have to go back and revisit all of these at some point -- the a and b stuff is some sort of convention and I'm breaking it by switching it to decimals for the citation...


Bah, sorry for the misdirect on #5.  Ok cool, I'll push up a revert.


There's a 0.12.3 one now, please use that


@lpugin is this what you were looking for?


PEP8 spaces around = ;)


@bobbytung Can we use this directly? Is it a copyright-reserved picture?


See https://bugs.launchpad.net/openobject-server/+bug/882036


Actually I find the one in https://github.com/BKWLD/laravel-haml/issues/5 great, because of the syntax highlighting feature in editors as said above.

But it's up to you.


Ah I see, push.node is already the mechanism we use for validating schema and such. So I guess we can leave that part as is for now. We still have the same SPoF for these operations, but we are not introducing a new one...


Oh, I misunderstood.  Well, yeah, lets go with haml.blade.php, thanks for being flexible.


It's git's fault


rax retry tests


擷取中間有行內注音的區塊，以合理使用授權，我可以處理。


Eliminar este comentario.


teste


@jordonias I don't know anything about `blanket`, but I suspect you're barking up the wrong tree -- there shouldn't be any difference in the execution of the line you show above with and without this patch.

But if you think otherwise, you should fork, try running as-is, then revert this patch and try running again and post your results here.


I understand the benefit of it, but maybe using TeX directly instead of the Lua bindings will make it a little more optimized... something like `\numexpr 161+#1` (or with `\the\numexpr`)?


Published, thnaks


test repository


No elimines el espacio.


Esta variable no debe de estar.


Identa correctamente y utiliza espacios en lugar de tabuladores.


@the-kenny: This breaks nixpkgs eval, because older emacs expressions still refer to that patch.

```
$ git grep darwin-new-sections.patch
pkgs/applications/editors/emacs-24/macport-24.3.nix:  patches = [ ./darwin-new-sections.patch ];
pkgs/applications/editors/emacs-24/macport-24.4.nix:  patches = [ ./darwin-new-sections.patch ];
pkgs/applications/editors/emacs-24/macport-24.5.nix:  # patches = [ ./darwin-new-sections.patch ];
```


Este espacio esta vacío, eliminalo.


Yeah, I agreed.


Este es un pequeño bug corregido del commit anterior, el tipo estaba errado


This is due to me not really being as facile with TeX as I should be.  I will implement that.


the new H3 doesn't make sense as a heading


Esta función quedaría mejor en CleanTask


You need the snippet:

	if sys.version_info[:2] == (2, 6):
	    import unittest2 as unittest
	else:
	    import unittest

Inserted here.


No problem :)

2015-04-15 18:54 GMT+02:00 Robert Reinhard <notifications@github.com>:

> Oh, I misunderstood. Well, yeah, lets go with haml.blade.php, thanks for
> being flexible.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BKWLD/laravel-haml/commit/08b70ffaa27833ab59a1449a28d684d9176129d3#commitcomment-10740373>
> .
>


Elimina el espacio en blanco que añadiste.


@blaflamme OK, I'll fix this right now.


Esta función debe eliminarse


This TODO was removed in subsequent check in.  It works with the encoded key.


Aplica el formatter sobre esta clase.


@atavism After a more detailed review I realized the blocking was actually in another place still within the `autoupdate` package, and that was causing the main config goroutine to block itself. At the end I used @myleshorton's suggestion. Please see https://github.com/getlantern/flashlight-build/pull/179


Are you sure with the print 'eee' in function.lib.php ?


Si no modificaste la clase,no añadas el 2015.


successfully en lugar de successfull.


Correct - Group ID should not be updatable. This comment was removed in subsequent check ins.


ok


Many thanks.


We're locking actual version in the parent project, and we're moving from 0.8 to our 2.x to probably official 1.x


Corrected.


Actually, i was going to suggest zooming in on the picture anyway, since it's not immediately obvious to non-Chinese people what to look for. We should make sure we catch at least the instance with the tone mark, and perhaps one without, btw.


this is interesting, what's the use case?


Done.


No, I was trying to help.


Apparently, this mod will be maybe added on homedecor, so we need to stay vigilant when we update homedecor_modpack


@stevepiercy I've updated the repo with first cut at highlightjs


Oh thanks!


In function get_prefix, 17 is added to character for mapping 0-9 to A-J and A-F to R-W


typo? chromat -> chroma?


awesome!


That works too, but I like the fact that we’re using a turnkey API for this sort of thing rather than a `UIImageView` that we dump the snapshot into. I don’t feel strongly either way; the performance is equivalent, even if we’re now recreating a view each time. Asserting about the view order sounds good though. (We’re using `-insertSubview:aboveSubview:` to ensure the order already, but an assertion would keep the host app from messing with the order for whatever reason.)


Wait, where would we assert? Right after `-insertSubview:aboveSubview:`? That seems a bit pedantic.

In `-glkView:drawInRect:` all we do is to check whether there’s a snapshot view in the first place.


Yes, but the first argument is the Model itself and the 2nd is the array of instances.

Example: https://github.com/appcelerator/appc.arrowdb/blob/master/lib/index.js#L433


I did not change the spacing. I moved properties to the top of the file, to be consistent with all other elements. That really confused the diff.


@blaflamme OK.

I found I do not have access rights to push to the Pylons/tpc repo. I must go through my fork at stevepiercy/tpc until I get access.


I can make a new one of the Zhuyin instance part tomorrow. Probably using .svg format too.


**Nice to see you are working on extending HALE!**

Just browsed through your code and thought I leave a comment on this. You should not store state in a transformation function in that way. It will do no harm as actually currently the functions objects are not reused, but it is planned to do that and this will most likely be done per transformation thread and not per cell. But this also means that your caching does have no effect currently and each time a new `MathTransform` is created.

You should cache the *MathTransform* per Cell, as the reference system parameter is part of the mapping cell configuration. You can do that through the `ExecutionContext` (retrieve it with `getExecutionContext()`). It allows you to share information between different transformation calls - please take a look at the *ExecutionContext* documentation for more info. Keep in mind that when using the execution context to share information, there is an overhead due to the need of thread synchronisation.

Also keep in mind, that the *MathTransform* also depends on the source CRS of the property being passed in. Although it may be a safe assumption for most data sets that the CRS will be the same for the geometries from the same source property, it is not true in general.


all console.logs are coming out in final


corrected


@stevepiercy oups sorry... it appears I forgot to add the group... doing now


We are in the process of finishing up our blockers. After that I am all in favour of a release myself.


你不喜欢我升级！:(


アイテム一覧みたいなの、どこに書くべきか悩んで結局ここに書いちゃったんですが、どこが良さげだったんですかね？


This `temporary_response` context manager, as written, will not restore the original ``response`` attribute if an exception is raised in the body of the ``with`` statement.  (Any exception raised by the body of the ``with`` will be re-raised in the ``contextmanager`` generator at the point where ``yield`` was called.)

Is this the intended behavior?

In order to have the ``request.response`` restored, even upon exception, the ``yield`` should be wrapped in a try/finally, with the code to restore the response in the finally block.


store these in ENV vars.


トップページへ戻るためのリンクがなかったのでPCの時は左上にロゴ（今はただのテキスト）を出すようにしました


gemspec should me updated to >= 4.x I suggest


OK. Thank you. I did a 'git pull' on your first cut, and it worked for me and looks good.

We should minimize the number of languages instead of using the default.
http://highlightjs.readthedocs.org/en/latest/building-testing.html#building

I'm heading to lunch now, and I'll check in in about an hour, maybe open a new issue and close this one.


Can't believe I missed (or rather included) those! Sorry.


Hardcoded IDs?


Haskell uses `>>=`.  But I like this.  Can I get a test showing the desired formatting?  I guess it'll look like:
```
onward >>= { successValue in
...
} >>= { nextSuccessValue in
...
}
``` which is pretty nice.


I worry about operator precedence and would like to see it in a few examples.


I highly recommend you to put meaningful component names in place where possible :)


If it works as expected and there is a real performance gain I guess it cannot hurt.


@akariv comments?


also we tend to ask to delete commented out code during the code reviews


I prefer `deferResult`, because we have legit uses of `Deferred<T>`.


Technically, we don't use `VER_NODE_PUBLIC` in `Manifest.cpp`, though we do mention its name and its value in logging.

The `VersionEncoding` constants are baked into the Ripple protocol, whereas the `RippleAddress` type is not, so there's a case for separating them.  What would be a good home for `VersionEncoding`?

If we make it an enum class, than we'd have to write `VersionEncoding::VER_NODE_PUBLIC`.  We could rename that to `VersionEncoding::Public`, but if we rename at all then we may as well clean up the terminology at the same time — this isn't a "version" in any sense.  I've tried to come up with a word that captures the sense of "large numeric value that will be encoded in Base58 for human consumption", and so far I have "credential".  So, `CredentialType::NodePublicKey` is an option.


You are correct, we should fix this. Do you have any interest in doing this and writing a test?


Lazy results are indeed cached.  The expression is only evaluated once on the first call to `Force()`.  It doesn't really matter, though, since in this case it's only being evaluated once anyways.

That being said, you are right that the guts of the Lazy wrapper looks more heavyweight than plain F# func (adds a bit of thread-safety stuff to ensure expression is truly only executed once).  It's not going to make a noticeable difference in a small case like this, but you're right better to stick with plain func.  Thanks!


nit: newlines.


@SatoTakumi
場所は自分も悩んでる　後で直すのでそのままでOK

あとメンションは必ず付けて欲しい


So should I change that t.equal with t.notEqual?


Thanks @xiam!


you don't need type cast here


@nchammas My guess it that we might want to skip the re-downloading if it's already present so that lint-python can be run without an internet connection.


The issue with old version of cython has been fixed (see #387).

Strictly speaking the minimum version of cython should be 0.14.
It is correctly reported in the tables/req_versions.py, in the RELEASE_NOTES.rst (3.2.0) and in doc/source/usersguide/installation.rst, but not in the requirements.txt file that still reports 0.13).

IMO we should oly revert this change and update requirements.txt file.


That data wasn't even getting loaded


shredded


I don't think this is enough of a win to use the weird symbol.  How about `@`, if that's legal?  Or can we use `*`?


I understand, TeX is no easy thing... If you take too much time (like more than 10mn) tell me, I'll help you


better to use getString(String) rather than getString(int) - first option has less chances of breaking if the columns will be reordered (also given your query does not specify order explicitly)


I don't think I support composing with `Void`.  If you're sequencing, you should use a block and really sequence.


I'll remove "convenience".  This is the one most users would likely use anyway. But there are just other methods for exactly the same operations which use the list of API params exactly as documented in API documentation. This method provided extra convenience :)


@SatoTakumi

とりあえずはいいかな。

あと

```
div(class="navbar-logo" flex) #{meta.title}
```

より

```
div(class="navbar-logo" flex)= meta.title
```

のがいいかな

シャープを使う場合は文字連結の時にしてほしい。


why do you handle the two cases differently? (SQLException & ClassNotFoundException) :) both seem to dump stuff into the system.out, right?


Empty `Result<()>` values are an anti-pattern.


Shouldn't this be called `isOnPushChangeDetection`?


MIT?


Random q: will the cleanup execute here?:

````
try {
  System.exit
} finally {
  // do cleanup
}
````


Extra `(ResultValue <= 0 AndAlso s) OrElse`


Did you forget to actually use `updateMeta`?


In the case of WK, we use window instead of global, but window.gc doesn't exist. When it runs, window.GCController exists too so no need to run gcRec.


Should be improved in 41c5fe2


re: the first part of your comment, since I'm already dealing with the parts I see no reason to prefer tag.startswith, though no reason against either.  Your suggestion about parts[3] would cause an exception if a malformed tag "AI_TAG" came through, and I prefer not to cause an exception in that case.  The suggestion about endswith again assumes the tag is properly structured, though at least it would not cause an exception.


ah yes, ok, I put that change in.


Oh, my bad.  Fixed in 4056a31 (develop) and cherry-picked in release-3.2.0 branch.


Is `>>=` associative?  I haven't thought of what `a >>= (b >>= c)` does with the deferred queues.


Thing to google for: SQL injection(as user inputs can be anything). Ideally you should use PreparedStatement to prevent that from happening


Please, use ``m_`` when creating new classes.


Maybe it was Steve TODO... that display-name tag is for what? I thought it was for what appears in the platform menus, but instead it's the platform tag. For example: platform: "gens", display-name: "Mega Drive"


This is much better, but I do prefer `deferResult`.


:clap:


I see these 5 lines duplicated all over - you can move them to a same class and reuse from every place


Perfect, have some scattered files here and there, will add some structure and descriptions then submit a PR.

One that already has extensive documentation is a mindfulness bell (using it every day), made with inspiration from living in a Buddhist monastery (one of the nuns had a more intrusive variant that aborted all activity for a few seconds...): http://artai.co/julia/mindfulnessBell.html


Good question.  I can start by replacing the set with a map and storing the comment.  The rest of it is a design question:  Which clients call `unl_list`, and why?  Should ephemeral keys be in a separate array, or mixed in?  Do we need a `key_type`?  Will all keys eventually be ephemeral anyway?


He sent me a pull request for S2W to be updated but did not mention if it was working or not. It is however working on other devices. I'm going to look at his pull request when I have more time to update that as well.


@jacekkopecky we recently added a `coffeelint` step to our Travis build. Among other things, it fails if you have an empty pair of parentheses in a function. If you have `./node_modules/.bin` on your `PATH`, you can run it like this:

```
coffeelint lib/*
```


    $ sudo apt-get install golang gccgo-go ruby -y
    Reading package lists... Done
    Building dependency tree
    Reading state information... Done
    E: Unable to locate package gccgo-go

On Debian Wheezy the package name is gccgogo (https://packages.debian.org/wheezy/gccgo)

Are you on Ubuntu and does it differ or is this just a typo?


rpc_deploy ==> openstack_deploy


Good catch.  Fixed.


Use `StringWrapper.equals`


 - contain **pages**, each containing content

We'd like users to be able to share a link to a project or a single page within a project, to lower the bar for participation.
We'd eventually like users to be able to remix projects or single pages, to increase the flexibility.


Did you notice the docstring "If no matching tags, returns empty string (which for most types should be considered equivalent to AVERAGE)"?  If you have a reason to make a contrary suggestion, then please explain the reason.   I see no good reason for this function to make the determination that "" should always be considered equivalent to "AVERAGE"


@k2wanko 直します！


How does it fail?


jhjhj


@jdm I removed the reference to the compositor from Cargo.toml, compiled and committed the updated Cargo.lock. Anything else?


I'm on Ubuntu 14.04 LTS

On Wed, Apr 15, 2015 at 7:24 PM, Richard van den Brand <
notifications@github.com> wrote:

> $ sudo apt-get install golang gccgo-go ruby -y
> Reading package lists... Done
> Building dependency tree
> Reading state information... Done
> E: Unable to locate package gccgo-go
>
> On Debian Wheezy the package name is gccgogo (
> https://packages.debian.org/wheezy/gccgo)
>
> Are you on Ubuntu and does it differ or is this just a typo?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Yuav/rabbitmq-cli-consumer/commit/165474d09437c5354ebd8c9162d0091f9f50ba6b#commitcomment-10741019>
> .
>


:tada: :+1:


No, I don't think we should declare types in `mycompare`.   Maybe just a note that the `Ref{Cdouble}` type is converted automatically to `Cdouble` types when passed to `myfunction`, and that this is documented further in Section XXX.


I think uInt should still be uint16_t and uLong should still be uint32_t on 32-bit MacOS for backwards compatibility. I think you need to guard the changes so that only 64-bit MacOS compiler uses uint32_t for uInt and uint64_t for uLong.


Agreed.


@SatoTakumi
あとclassの書き方はドットでお願い

```
div.navbar-logo(flex)= meta.title
```

複数の時は.foo.barみたいにつなげて

まぁクエリーを書くときは今の書き方でいいけど

ちなみにidは\#で


dalia


Here is a fundamental question:  should we not here be re-trying on error?

We have two alternatives:

1. we accept that at some point the system will be overloaded and will return
a "db locked" error.  In that case callers need to retry.  We could perhaps
do the retry in the lxc client itself?

2. we do NOT accept (1), rather we pull out Thor's hammer and bump the db
timeout to something like 60 minutes.

Now, I do think we still have real bugs (implied at least by the 'attempted
to rollback a tx already rolled back' error messages).  If we decide on
either (1) or (2) it should help us filter out noise and get to the bottom
of these.


@ShNURoK42, ну опять - 25 =))
[ ![](http://i.piccy.info/i9/7f71041e930dfa9ba8d6553ebd1b37b1/1429118890/4820/865755/y1_240.jpg)](http://i.piccy.info/i9/90ca9ee3348b50d886e455779d46ebc4/1429118890/22670/865755/y1.png)


:+1:
On 15 Apr 2015 17:46, "Daniel Roberts" <notifications@github.com> wrote:

> Actually, do you mind if I do that after this is merged? There are a bunch
> of places throughout the code that use DateTimeProperty().wrap where they
> could now use iso_string_to_datetime and it'd be nice to do those all at
> once, without adding that to this already-big PR. Just added it to my
> personal cleanup backlog
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcare-hq/commit/f70dafe39e463b84d3517e83d3e319d8926a58ed#commitcomment-10738912>
> .
>


Not needed, will be reset below anyhow


No `;`, this ain't Java ;)


cgwalters, I think even for --storage-opt dm.datadev and --storage-opt dm.metadatadev one, we probably need to give little stronger warning. Docker does not have capability to deal with pool when it is full or when pool is in bad state and needs-check flag is set.

So things will work fine as long as nothing goes wrong, after that one does not have a clue what to do. lvm thin pool gives commands to deal with these scenarios.

So to me it makes sense to recommend lvm thin pool for production use, even when user specified dm.datadev and dm.metadatadev.


@mpc-hc: I would like to push madVR change soon. Although madshi workarounded subclassing crash on x64 it is still better to avoid it. https://github.com/kasper93/mpc-hc/compare/develop


LGTM


Acho que os @1@, @ActionCode@(linhas 407, 408 e outras) poderiam vier pelas variáveis do template. O segundo login poderia ir tb par /login inves de centralrw.cgi
Acho que o javascript do catch na linha 451 poderia abrir tb.


Hi,
no real reason. The code I originally needed to support didn't use prepared statements inside the transaction blocks, were simple queries without arguments.

Agree with you, allowing args would be more appropriate,  and caller can just pass [] itself if not needed.


Is this maybe better suited to a separate repository? Obviously now it's been committed, so deleting it won't shrink repo size unless we actually destroy this commit, but it might logically fit better in a vrpn/vrpn-miscellany repo if you're more or less looking for public hosting.


This line is just the current directory, not $(pwd)/rpc_deploy (which should be openstack_deploy for Kilo/11.x) as mentioned in the comment.


In the images maintained in docker-library, we have moved the keyserver we use to `pool.sks-keyservers.net` and they seem to be pretty stable.  It uses a round robin of key servers.  Reading again the [overview of the pools](https://sks-keyservers.net/overview-of-pools.php) makes me think that `ha.pool.sks-keyservers.net` might be more reliable.  It's up to you what keyserver you use, I am just sharing some information :smile:.


My problem in this case is that I was unaware that numexpr existed.


Alternately, of course, we could put it in the vrpn.github.io repo (which I plan to eventually have as the vrpn.org domain target), though there it gets mingled with web content.


lgtm


`for hook in self._serial_hooks.values()` combined with a `serialT = hook(...)` below


@vbatts

Thoughts?


Looks good, some of potential improvements are listed below:

I don't see if there is version support (not sure if this is necessary, but I think we have this logic as part of prepare and I don't see we can override this here).

If we call parameter buildConfig I think we should support all build options including chip architecture and debug/release configuration.

Nor important:
extraConfig name may lead to confusion since we use separate variables for most of params and new params are just wrapped to this object. We want to move buildArgs processing logic to separate class, then

var buildArchs = BuildArchs.parse(argv); // all argv processing logic is hidden here
result
{
  type: 'Release'
  archs: ['anycpu']
  packaging: {
      displayName:
      appId:
     // no buildConfig here since we already read and applied all params from it
  }
}

Not important:
Another idea is to pass argv to prepare logic so it can decide what to do w/ passed parameters. Technically build should not be is on response for parsing prepare params.


Simplification des types :
Tout sauf Matrice mat est représenté par "sint" <=> unsigned short int


Do we still need to remove these attributes when activate an item with no `::getPath`?


Yes, it is intended.


Well, actually I learned its existence quite recently, it's not a very old primitive (only 17 years old!)


Derp, yeah, thanks


Oh nevermind. Just saw that you moved these up.


After reading some of the comments in the trello card, it sounds like there may be a desire for this to work in Juno, too - in that case, the openstack_deploy cases should be added to the current list of directories.


Or wait, they will never be added when no path, since they are cleared before on 70/71, or am i missing something?


The problem is that there are already two notifications, in ``reply`` and ``err``, not sure adding a new one in between is going to make sense. The error or the reply usually shows up in <5 seconds.


Cool, thanks for the :eyes:


Я с ней еще не закончил и почему-то на сервере не работает, на локалке нормально.


I don't see why changing a comment can make the script different :P


I'm going to incorporate this exactly but cannot do so based on this specific pull because of conflicts. Thanks  though.


thanks:D


@tfrommen no, I think you've covered it. I just haven't had a chance to test this yet.


```haskell
Radius       :: Point -> Bool
Neighborhood :: Radius -> [Boid]
```
or something like that? Radius might need to take a point and a float (currying?), I dunno.


@rultor version


Good point :)  Done.


Fixed locally.


Merging


Should there be a per-user directory inside C:\ProgramData as well?


hjhjhj


mklm


You're right, `size_t` here is an error.  I'd go with `std::uint32_t`.  The sequence number needs to account for the number of times we revoke and replace an ephemeral key.  If we did this every day for 100 years, we could still get by with only 16 bits, so 32 is more than enough.


This throws a warning:
1>..\src\m_misc.c(419): warning C4013: 'ABS' undefined; assuming extern returning int


@Sophist-UK A user [reported this error](https://wordpress.org/support/topic/fatal-error-help-39?replies=1) in the WordPress forums:

`Fatal error: Call to undefined function mb_strlen() in /home/cthou0/public_html/wp-content/plugins/list-category-posts/include/lcp-catlistdisplayer.php on line 403`

Do you think it would be fine to add a flag to check if `mb_strlen` is enabled? Would there be another workaround in case it's not enabled?


class myStack extends ArrayList{


Ah, now I see.  Can you merge this new version for the script please?  I would like to use it in PyTables too. Thanks.


dalia emad


Sorry I didn't delete the fork yet, it wasn't related with the version. I am using this with a mixed project that has to run in node/titanium and ti-superagent was the one giving problems.


@rultor release, tag=`1.6`


Looks like it didn't work. I should check all molIon$cameraobj values


Hmm, the complete "infrastructure" was already there. So I added both API contexts call tests after submitting all three transactions to the memory pool and before mining them.


@elikem We do not have a use for the primary option. Can you remove that? Then I will merge


Nearly, but you now have a memory leak line 51 because (*res) will not be deleted. Should I fix it?


Added mandatory description and removed "backdate". Yes, the date field is mandatory.


Where did you move the above log.error message?


@robertmesserle you guys are doing an amazing job btw, the tabs control is spectacular!


There is the same problem here.


If this class only needs the function registry and type registry, we should get rid of this method and the dependency on Metadata entirely


Top
Un peu plus gros, histoire d'optimiser l'espace et je pense que ca ira. Qu'en penses tu ?


Was this throwing a notice or a warning? Shouldn't throw a warning, but if you have notices enabled it may show "undefined variable".

Be careful with this fix because you could also get a notice if `$this->original_info['exif']` isn't set. You'd need something like this instead:

```php
if( isset($this->original_info['exif']) && isset($this->original_info['exif']['Orientation']) ) ...
```

Also, variable convention in this project is with underscores :)


The .io is the perfect place for it.  It should end up linked to the web content.  I'll put future such files there, and move this one over.  Not so much worried about the size of the repo as the proper location for things to live.


Isso funciona em qualquer servidor linux?
ex: Ubuntu server 14.04.02


Nit: Attached -> Attach


<5 seconds is fine, but I'm not sure if that's always the case or if we may run into some timeout or something. Well, lets leave it as it is for now.


If you want name tests then we should just test for the correct name. Before I can merge I'd need to come up with a way to mark reference implementation tests as known failures though :-/.


let's not add this...


why?


let's not add this


Great. I'd say make a directory under `source` called something like `assets` and put it there


When do you pretend release a new version with this fix?


I believe we only specified mandatory if it is mandatory. This note param is optional.


also let's not commit this.


These also aren't ES6-accurate. Methods in ES6 have no `prototype` property, and functions of any sort in strict code in ES6 don't have `arguments` or `caller` properties.

This is largely a test of the engine's ES6 conformance, so I am not sure they are worth including.


Yes please!


Throughout: s/2014/2015/


Yes! That's a good idea, that solves some of our "What if >1 user installs to ProgramData?" "Oh, they're fucked" that @CharlieHess and I were talking about


Such a descriptive commit message :P


why do we dump this?


Look into Qt to see if there is something like a WhileKeyDown or something. Myke already tried while loops and they uh, didn't agree with Qt very well.


Wrap this line for readability:

```java
... = node.getSources().stream()
    .map(...)
    .collect(...);
```


SHREDDED

Sincerely,

Jacob Martin
Rensselaer Hackathon Organization
(510)-898-8195
jmart.alb@gmail.com

Want to securely email me? Encrypt your message with PGP here
<https://www.igolder.com/pgp/encryption/> using my public key here
<http://pastebin.com/6dKLX77S> for a secure email unreadable by third
parties.

On Wed, Apr 15, 2015 at 1:17 PM, Noah Goldman <notifications@github.com>
wrote:

> shredded
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/noahgoldman/torwiz/commit/137968e32f6e87d09c20722882b28c075846f7f0#commitcomment-10740876>
> .
>


Why o why??


Also, I'd name the variable `sources` for conciseness (without loss of clarity)


itsy bitsy tiny typo - `depreicated` :mag:


On Wed, Apr 15, 2015 at 5:23 PM, Marcus Appelros
<notifications@github.com> wrote:
>
> Perfect, have some scattered files here and there, will add some structure and descriptions then submit a PR.
>

+1.

> One that already has extensive documentation is a mindfulness bell (using it every day), made with inspiration from living in a Buddhist monastery (one of the nuns had a more intrusive variant that aborted all activity for a few seconds...): http://artai.co/julia/mindfulnessBell.html
>

That is such a cool app! Did you write this? If yes, do upload it to
github (or any other repo) so that I can add it to `Julia.jl`.
Otherwise, could you ask the author to push it to a repo and add a
license to it too? I could not find an email on the site.

Thanks, SVAKSHA ॥  http://about.me/svaksha  ॥


Oh I see, gcRec is a fallback meant to try to artificially provoke GC in other VMs, got it.

I still want to know how to get access to window.GCController though so I can leave that in the docs/comments. I presume if I just boot up Safari out of the box as a normal user it's not present.


Really??


Thanks a lot!


Oops, thank you for noticing-- I saw the comment and I verified that the field selector was in place, but I guess that was insufficient? I will look for your PR and we can discuss there.

We clearly need an integration test that checks that behavior!


Generally speaking, when a user adds a suggestion, it will be shown to him as action=="new".
The client should use this to prevent the user from voting (but allowing her to delete).


You can get a new version (0.12) of the script, which just fixes the typo in the script (missing ``-squash`` in alternate command) by doing:

```
wget https://raw.githubusercontent.com/Blosc/subtree-merge-blosc/v0.12/subtree-merge-blosc.sh
```


indentation!!


I would rather keep it returning null for null.  It's safer here. Date can be null. But both options have pros and cons. I'll leave it to you guys to update.


> How would moving these repositories to some other account/organization contribute to their discoverability and standardization?

Just getting back to Swift dev after travel and noticed you [implemented this](https://antitypical.github.io) :metal:


Feel free to dump what you consider :)


I always forget that AI can't trust game data :)

re.match will be more short in that case if you common with regexpr.

ai_TAG_GOOD_WEAPONS is not valid tag, is it ok?


You shouldn't specify dependency version here because it already specified in `<dependencyManagement>` section of parent project.


:+1: Awesome - thanks for the fix. Working on an upstream fix as well - but no reason to make all those expensive calls if ``/tools`` gives you everything you need. https://github.com/galaxyproject/galaxy/compare/galaxyproject:dev...jmchilton:toolbox_auth?expand=1


But you may still have to comment in/out the code. Also, the script will not allow for a dirty working directory as a precautionary measure., so you need to commit the commenting in/out first.


This revision is clean.


did you mean to make these changes? And if so, why?


Alternately, I could make a new version of the script (v0.13) which contains the currently working version of the merge command. Would that be better perhaps?


@thejamesdempsey This is a specific feature request from @mattgasior and @nathan-brown-cru  for the article long form component.

>**Article Long Form**
>The author should be able to select which button style to use for Prev/Next buttons [Primary | Secondary | Subtle]

https://github.com/CruGlobal/cruorgaem6/pull/135

You guys hash it out and let me know (this prolly isn't being used now; as it is now `prod` is broken w/o this).


What is this adding?


I think technically we need a `+1` here because `HOST_NAME_MAX` does not include space for a terminating null byte and this is a C buffer. `gethostname` will not terminate the returned string if the buffer is too small.


`times(1)` is redundant here and below. `verify()` checks if method was called once by default.


Yes, I can do that.  It might take a day or two.

One other possible nit: if ``request.response`` is initially set to ``None`` (as opposed to unset), ``temporary_response`` currently, when it's done, will delete the attribute, rather than restoring it to its prior value.   (This probably doesn't matter much, since ``request.response=None`` is not particularly useful, but it's an easy fix.)  Should I fix that too?


^.^ Created the whole artai site, will host crowdtrained AI making art there in the future!


@elrayle Are we missing a couple of steps here?  Namely:

```
rails generate roles
rake db:migrate
```
:question:


Beserker/Cave-Man Maybe


simplify if statement to return attr['weight'] < 0


:+1: :clap:


That is embarrassing.....


I could only find Keypressevent, and keyreleaseevent. If I can get the game loop correct from the game loop branch it should fix this, and read for movement every 16.67 ms


I also did not realise that C++ compilers will allocate a buffer like that from a run time variable. Previously it was a compile-time buffer size. C compilers don't like it.


This looks simple enough, OK


OK


I cannot speak for Safari because I develop on WebKitGTK+, but it seems that it is only present in my development environment, but not in my Epiphany compiled with latest WebKitGTK+ release. So it seems that it is not included for releases.


This should do it. At each return, a copy constructor will copy everything returned to the local variable. This is a bit less efficient but reduces possible memory leak risks from the caller side. To be tested.


This doesn't work as you intended. has("i") only ever gets called once.


Resuse weighted_edges function at this point.


Reverted with commit 59ba983760f19445fdc6f85d2d3ec406521e38d6


Do we not check return values from system calls? I guess that we assume `ENAMETOOLONG` can never happen (because we are allocating a buffer that is the right size) and `EFAULT` is impossible for stack memory.


I know I put in there Primary, but I think Jimmy is correct – we shouldn't give authors that option. The gold would be too much.

Sorry for the confusion.


rax retry tests


Reuse weighted_edges function at this point.


This looked like a potential error to me but perhaps I don't fully understand Lwt's semantics.


This seems to break NFC on the find7.

I/NfcService( 1650): Starting NFC service
I/NFCJNI  ( 1650): NFC Service: loading nxp JNI
I/NfceeAccess( 1650): read 3 signature(s) for NFCEE access
D/NfcService( 1650): checking on firmware download
D/NfcService( 1650): NFC is on. Doing normal stuff
I/NfcService( 1650): Enabling NFC
D/NFCJNI  ( 1650): Start Initialization
D/NFCJNI  ( 1650): NFC capabilities: HAL = 8150100, FW = b10126, HW = 620003, Model = 11, HCI = 1, Full_FW = 1, Rev = 38, FW Update Info = 255
D/NFCJNI  ( 1650): Download new Firmware
W/NFCJNI  ( 1650): Firmware update FAILED
D/NFCJNI  ( 1650): Download new Firmware
W/NFCJNI  ( 1650): Firmware update FAILED
D/NFCJNI  ( 1650): Download new Firmware
W/NFCJNI  ( 1650): Firmware update FAILED
E/NFCJNI  ( 1650): Unable to update firmware, giving up
D/NFCJNI  ( 1650): Terminating client thread...
W/NfcService( 1650): Error enabling NFC

Using find7/proprietary/vendor/firmware/libpn544_fw.so from the previous commit works fine.


Hello There,

When I try https://github.com/boostsup/phpAmazonMWS this project, I got an issue, the error message is:

There was a problem with the Amazon library. Error: Secret Key is missing!

Actually I already put secret key in the amazon-config.php file.

I am wondering which part has some problem, could you help me figure it out?

Thanks.


Was this change related to the fix for `gethostname` or fixing some other compiler warning?


I feel like we should fail here in case of an invalid time string rather than setting the timeout to 0.


the 'get_related_to_search' tags returns tags related to search, annotated with the usage count within the search results.

This is an ambitious requirement and won't work for very large number of threads. This should be either dropped or fast enough approximation should be devised.


I believe this is correct and provides a nicer interface to users than simply terminating the response pipeline via a confusing error about the first line of a response not being a valid HTTP response first line (that behavior is also a potential security vulnerability if users can control server *resources* but not *headers*).


Add function to __all__ list


Simpify too.


Я не про перенос строк (это правильно), а про разделение одного string через точку (при большом запросе легко будет забыть поставить точку и конкатенация - доп.операция и нагрузка, причем довольно ресурсоемкая операция,а SQL запросы бывает, что достигают нескольких сот строк. )


1570-EnergyPlus-8-3 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


I think nothing breaks in that case because when we delete the value from the underlying dict the reify will kick in and create a new response upon next access. It would be really weird and unexpected if someone actually had done `request.response = None` but if they had then the restore + next access would trigger new response creation instead of returning `None`.

It's a small issue but I'm okay with fixing it if you want.


I'm not sure about the slight code duplication going on there. How about moving the whole default connection into its own method and just attaching this as last element to the hook list?

Something like this (completely untested):

``` python
def _detectPort(self, close):
	programmer = stk500v2.Stk500v2()
	self._log("Serial port list: %s" % (str(serialList())))
	for p in serialList():
		serial_obj = None

		try:
			self._log("Connecting to: %s" % (p))
			programmer.connect(p)
			serial_obj = programmer.leaveISP()
			break
		except ispBase.IspError as (e):
			self._log("Error while connecting to %s: %s" % (p, str(e)))
		except:
			self._log("Unexpected error while connecting to serial port: %s %s" % (p, get_exception_string()))

		if serial_obj is not None:
			if (close):
				serial_obj.close()
			return serial_obj

		programmer.close()

	return None

def _openSerial(self):
	def default(_, port, baudrate, connection_timeout):
		if self._port is None or self._port == 'AUTO':
			# no known port, try auto detection
			self._changeState(self.STATE_DETECT_SERIAL)
			serial_obj = self._detectPort(False)
			if serial_obj is None:
				self._log("Failed to autodetect serial port")
				self._errorValue = 'Failed to autodetect serial port.'
				self._changeState(self.STATE_ERROR)
				eventManager().fire(Events.ERROR, {"error": self.getErrorString()})
				return None

		elif port == "VIRTUAL":
			# virtual printer, use that
			serial_obj = VirtualPrinter()

		else:
			# connect to regular serial port
			self._log("Connecting to: %s" % self._port)
			if baudrate == 0:
				serial_obj = serial.Serial(str(port), 115200, timeout=connection_timeout, writeTimeout=10000, parity=serial.PARITY_ODD)
			else:
				serial_obj = serial.Serial(str(port), baudrate, timeout=connection_timeout, writeTimeout=10000, parity=serial.PARITY_ODD)
			serial_obj.close()
			serial_obj.parity = serial.PARITY_NONE
			serial_obj.open()

		return serial_obj

	serial_factories = self._serial_hooks.items() + [("default", default)]
	for name, factory in serial_factories:
		try:
			serial_obj = factory(self, self._port, self._baudrate, settings().getFloat(["serial", "timeout", "connection"]))
		except Exception:
			self._log("Unexpected error while connecting to serial port: %s %s (hook %s)" % (self._port, get_exception_string(), name))
			self._errorValue = "Failed to open serial port, permissions correct?"
			self._changeState(self.STATE_ERROR)
			eventManager().fire(Events.ERROR, {"error": self.getErrorString()})
			return False

		if serial_obj is not None:
			# first hook to succeed wins, but any can pass on to the next
			self._changeState(self.STATE_OPEN_SERIAL)
			self._serial = serial_obj
			return True

	# TODO error state etc...
	return False
```


1570-EnergyPlus-8-3 (kbenne) - x86_64-MacOS-10.9-clang: Build Failed

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


I have just checked that 0.12 works well for PyTables.  I don't like the 'precautionary measure' of the 'dirty' working directory and I think it would be better to get rid of it (unless you have good reasons for keeping it indeed).


managed to push the local repository up to github repository yey :-)


I don't think the C++ standard allows this either; I know gcc does, and it might be that clang does as well, but we probably shouldn't be using it.


Josh asked...
> What would be a good home for `VersionEncoding`?

My take lines up with Nik's.  If the only place the constants show up is in `RippleAddress.cpp`, move the enum in there.  If it's a classic (non `class`) enum then all of the `VER_*` names leak out into the `ripple` namespace.  So I'd prefer that it be an `enum class`, even inside the *.cpp file (since we have unity builds).  Also setting the underlying type to unsigned char would be good.  The default underlying type of an `enum class` is `int`.  A `CBase58Data` doesn't give us that much storage.

Since the names are only in `RippleAddress.cpp`, a rename is not too expensive.  I agree with Josh, these are not versions.  They are key types.  But the term "key type" is already used to distinguish `ed25519` from `secp256k1`.  So something like `Base58Encoded::nodePublic` might work.


1570-EnergyPlus-8-3 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


I'll add an extension point here via the hook `octoprint.filemanager.extension_tree`, will add that to the `devel` branch shortly.


Looks Good Keith


Maybe add a new line here?  I tend to get confused by that many blurb together ;)


1570-EnergyPlus-8-3 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Another new line here?


I think `yield` is generally okay here. The restrictions come into play for one of two reasons:

1. Strict mode code ([where Identifier cannot take on that value](https://people.mozilla.org/~jorendorff/es6-draft.html#sec-identifiers-static-semantics-early-errors))
2. The `[Yield]` production parameter ([where the definition of BindingIdentifier is *not* extended to include that keyword](https://people.mozilla.org/~jorendorff/es6-draft.html#sec-identifiers))

As written, the production is applied *without* `[Yield]`, so the code seems valid to me.


Other than that, this looks very good to me.


It can be eliminated, I probably tried adding some defaults like class names there at first. I don't see why we need it as things are just passed through to the li block.


проверка на стороне клиента - защита от дураков. Любой, кто хоть чуточку знает, что такое dev консоль и основы JS удалит твою проверку прямо в браузере и зарегистрирует тебе кого угодно. Валидация на сервер сайде в первую очередь, затем на клиент-сайде.


Yes, it could be if this definition is not used in other methods, and it's not.  But I think it's more clear when it's right before the call and would be consistent with some other similar cases. I suggest we leave it as is for now.


so we can tell which user (device) created it, and thus can remove it. from your comment below I understand we already know that?


LGTM


Yeah, use CSS to only display it on :focus.

On Wednesday, April 15, 2015, Adam Becker <notifications@github.com> wrote:

> Add 'skip to content' link
>
> cf http://a11yproject.com/posts/skip-nav-links/
>
> @jrubenoff: you good with this pattern?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/dobtco/dvl-core/commit/c311a2173c17eb36629b94d62831525de1949803
>


... and of course the implementation below will wait until all requests have been dispatched until beginning to read replies. If the incoming request stream is long (or infinite) then the reply stream will never be created and responses never read. I'm not really sure why the requests are in a stream, though, as `Lwt_stream.iter_s` will attempt to consume all elements. The part that looked like a bug was the ignorance of the return value without any binding into the monad.


This commit mentions that you removed all these extra classes, but it doesn't look like that happened?


Well, you don't things to sneak in by accident, so having a clean working directory is safer.


Can you rely on HTML5 validation or AngularJS validation here?


I tried this fix and MP now displays the current value.  However, it just displays the value entered in the Battery Monitor settings section.  It doesn't display the real-time Amp value.  e.g. If you disarm the quad or the throttle is down to 0, PM continuous displaying very high Amps.  I expect to see something under 1amp.


There is an API coming to retrieve Index interest rates. So we  can update this test when it's available,   Otherwise, It could just move the problem to hardcoding the product that is using Index Interest rates


Do you think this might be better off in the Room service?


I just committed some stuff in bwctl_server.py. Look for the limits_db = LimitsDB() line toward the bottom, and fill that in with parsing (assuming a limits file has been specified, otherwise just use the default LimitsDB()).


Ah, here it is.


Code refactoring.
I use tab and not white space because of "google format to java" give me that format. Too laborius to change. (Minor warning)
Also I use System.out.print or System.out.err and "SonarQube" give me a major warning to use logging.log instead of that.
I tried but I couldn't use logging.log,
![sonarqubeevaluation](https://cloud.githubusercontent.com/assets/11273988/7166097/0a175c8e-e3ac-11e4-9a7b-8030059a10ee.png)
 and I think it isn't too important.


yes, you're right.  Added the +1.


Snif.


AFAICT this won't work once Postgres reaches version 10...


ok ok


I check this now, and thrown an exception if the host name can't be retrieved.  Also checked the library call to gethostbyname below.


Warum?


Here's a quick patch I made to show my point

	diff --git a/avocado/plugins/runner.py b/avocado/plugins/runner.py
	index 0ecae4e..19f68bb 100644
	--- a/avocado/plugins/runner.py
	+++ b/avocado/plugins/runner.py
	@@ -35,6 +35,7 @@ class TestRunner(plugin.Plugin):
	     name = 'test_runner'
	     enabled = True
	     priority = 0
	+    view = None

	     def configure(self, parser):
		 """
	@@ -141,7 +142,9 @@ class TestRunner(plugin.Plugin):
		         else:
		             timeout = int(raw_timeout)
		     except (ValueError, TypeError):
	-                timeout = 0
	+                self.view.notify(event='error',
	+                                 msg="Invalid job timeout specification '%s'" % self.view.app_args.job_timeout)
	+                return sys.exit(exit_codes.AVOCADO_FAIL)
		 else:
		     timeout = 0
		 return timeout
	@@ -152,14 +155,14 @@ class TestRunner(plugin.Plugin):

		 :param args: Command line args received from the run subparser.
		 """
	-        view = output.View(app_args=args)
	+        self.view = output.View(app_args=args)
		 if args.unique_job_id is not None:
		     try:
		         int(args.unique_job_id, 16)
		         if len(args.unique_job_id) != 40:
		             raise ValueError
		     except ValueError:
	-                view.notify(event='error', msg='Unique Job ID needs to be a 40 digit hex number')
	+                self.view.notify(event='error', msg='Unique Job ID needs to be a 40 digit hex number')
		         return sys.exit(exit_codes.AVOCADO_FAIL)
		 args.job_timeout = self._validate_job_timeout(args.job_timeout)
		 job_instance = job.Job(args)


You can use ``git clean -xdf`` to restore a pristine checkout. Use ``git clean -xdn`` to check what would be removed.


Now providing an invalid value will turn into:

	$ scripts/avocado run --job-timeout=sbrubles synctest sleeptest
	Invalid job timeout specification 'sbrubles'


Add space between comments


And of course, a test for this could be added to the roaster of functional tests.


Indeed it is possible that there is a kingring, and a zero attackerscount,

But.if the attackercount is zero, no contact check is possible....
It leaves only the safe checks to compute, which are possibly harmless.. (because no follow-up is possible) or maybe not.. (this safe check can be mate or force the King to move into a mating net).

Very Interesting test.


remove these extra lines


Ok.


1570-EnergyPlus-8-3 (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)


I think I do this in a couple places too and I'm wondering if maybe we shouldn't go for a more generic  "just parse out the final class name from package path" here instead of assuming "com.etsy.scalding.jobs" is the base. Totally optional, especially if this makes the interpolation harder.


Unused?


Nit: Move the opening bracket to previous line... So sorry :(


This needs a copyright statement on top.


It works on my pc (win 7) ~ can you run the command like i have in the following image?
![screen shot 2015-04-16 at 4 18 41 am](https://cloud.githubusercontent.com/assets/2503364/7166235/cd3b46ec-e3ef-11e4-8cdd-81a19a10fae0.png)


That's a good future extension.  My use case also needed the remaining periods replaced with dashes, which not everything may need, so that needs to be made more extensible too.


Networkx has exactly the same algorithm, but it search SCC(strongly_connected_components) for all the nodes. So on graph that is made by disconnected graphs (let's call them sub-graphs) it search all the loops in all the sub-graphs.
Since we are interested to search and remove loops that are accessible from the sources (i.e., input data nodes), we can skip the searches on unreachable sub-graphs.
Therefore, this function (strongly_connected_components_function) is needed to limitate the search on those sub-graphs that are accessible from the sources.


Replace with `_lock = ...` to avoid triggering KVO on initializer.


Safe checks by knights and bishops are probably harmless.
But safe checks by Queens and Rooks may even result in a backrank mate!


lgtm!


d


Nit: `registrationWithSource:delegate:`


Double-check your config setup. Did you make sure to put the store settings in `amazon-config.php` and not `amazon-config.default.php`? Are your store's settings correctly indexed under the same store name in the `$store` array? If you have more than one store defined in your config, are you passing the correct store name to the Amazon object when you initialize it?

If you still have trouble, try `var_dump`ing the object to see what the property `storeName` is set to. It should  be an array key in the `$store` array.


Is this for the background color?


Oh, no it's for text color. I'll fix it ;)


Fixed - 96e7c94


## `\o/`
Glag to see you back!


Nit: `if (self.disposed)`


> 8:00 to 8:15AM - Getting up to date. (Q&A about web dev, own products, blog)
Talvez solo dejar claro que es un Hangout. _Hangout to get up to date_


Nit: `NSAssert(!self.disposed...`


ok it was added automatically by pycharm, but I haven't checked the style ;). I'm fixing it!


![server monitor](https://cloud.githubusercontent.com/assets/8058556/7166340/decfcc70-e372-11e4-8330-e7634a4983c5.png)

Screen shot of the working count down clock


Yes, I think so. I'll work out a fix soon.


Removes all registrations associated with the token, including disposing of all delegates.


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


Nit: indentation.


TeamCity xBIM :: XbimExchange :: Build [Build 222](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1401&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:02


Actually the <b>seventh</b> time...


Is this a leftover? :)


Hi Daniel,

If I understand correctly, for cases where a dialog timeout occurs and send_bye=1, you rely on the DLGCB_TERMINATED_CONFIRMED (cdr_on_end_confirmed) to write the CDR? Therefore, the DLGCB_EXPIRED (cdr_on_expired) callback is not executed in such case?

In any case, this change produces a warning in the ACC module (see acc_cdr.c line 297). I think the level should be DEBUG?

Also, I tested and the CDR is now written in this case.

Thanks,
Mickael


Actually the <b>eighth</b> time...


This commit broke compatibility with Forestry but already trying to communicate with @mezz so he implements IGrowable in Forestry's saplings.


Actually the <b>ninth</b> time...


This should go into the response object's documentation.


@kevinkreiser I kinda like that image actually… it explains tiles pretty well :) Is there a different map style or visual style that you had in mind? Send it along and I can make something cool


OK, it works with Play Music, but it doesn't work with Eleven from CM
don't know about AOSP's Music

thanks


Maybe add a corresponding positive test for `flags: [noStrict]`?


Yes, and an attack must start somewhere, and detecting any such possibilities early cannot harm.

Congratulations for this test, and also your remarkable work on the evaluate_threat routine in the last few months.


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 11](http://teamcity.otenv.com/viewLog.html?buildId=1257365&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


Sorry for the late response. I'm using Eclipse.


Je vais tenter de rajouter une flute à la musique. Je vais aussi faire
une fin moins brusque. Ce sera un peu plus long normalement mais un peu
plus "fini". Après, si j'ai d'autres idées pendant la composition, je
verrais. Mais pour l'instant, voici mes pistes. Merci de vos commentaires !


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 11](http://teamcity.otenv.com/viewLog.html?buildId=1257365&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


ekstrand, you copy/pasted the comments and it shows TX twice :)
Ideal for me to ask another question/feature request:)

1#What happens when I fill up the buf , will it send before it's full?
2# What if somehow the message isnt send? Is the message gone and will it not retry before clearing the bugger?
I want to send keys sometimes alot faster than the esp probably can because it's not that fast when sending and it will need to catch up somehow. Any suggetions how to solve this?
========================================
 if (stringComplete) {
inputString.toCharArray(buf, sizeof buf);
wifi.send(SERVER, buf);
inputString = "";
stringComplete = false;
}
======================================


It will if we maintain lexical ordering by using something like REL_10_x, which is what I'll be advocating when we get to that stage.


It should probably be called ```_setupMountOptions``` since there are also format options.


This is crashy, right?  Or does `anything as! Anything?` return nil as appropriate?   (I doubt it.)


居然加 `/noconsole`……


I was here. 2015


@rhvgoyal Suggested text?  `logrus.Warnf("NOTICE: --storage-opt dm.datadev or dm.metadatadev are deprecated in favor of --storage-opt dm.thinpooldev ")


Always use 登录.


Oops, didn't realize our repo is public.


@souperneon yeah i guess the things i dont like about it are the color choices i made for the tiles against this mapstyle. i tried other map styles (base, transport and humanitarian) however those ironically make it hard to tell what types of roads you are looking at. Maybe it would be easier we would just change the colors used for the tiles?


nit: no space before :, so `Section: Setting`.


nit throughout: full sentences, please!


Does this even need to be a list? We don't care about ordering, so we wouldn't lose anything by making it an unordered set instead. Would we gain anything from this? Does Haskell have an unordered set type?

@yarbroughw might know about these things


Is there a reason to or to not make this an abstract class and not a protocol?


hypothetically we could write a generic `Update` function by changing this signature to `Action -> World -> World` so that you can pass different `Action`s (i.e. strategies) and it'll just map whatever strategy/behaviour you want across each `Boid`...


Pull request https://github.com/dnnsoftware/Dnn.Platform/pull/473


is `Action` the best name for this? Maybe it's more of a `Strategy` or a `Behaviour`?


yoyo honet singh


These aren't descriptive at all.  What do the different types mean?  This distinction just feels wrong.  I think you should just include the indicator type in the `Setting` model directly.  You're using the class hierarchy for different setting "types" anyway.


Dynamic adding to the flowlayout trips when scrolling.


> Removes all registrations associated with the token, including disposing of all delegates.

Yes that's correct.


Oops :D


weil ich davon __überhaupt__ keine Ahnung habe


Whatever is going on here, separate it from the settings commit.


You can't just remove the -lrt, as the realtime extension are needed for the threads to work.
(Are you really cross-compiling or just trying to compile locally to see if it compiles but not actually run on the platform?)


(Note the /pmapi/context URL takes a timeout in seconds, not milliseconds)


Why wouldn't this be encapsulated in the profile?


Sorry, a little confuse here. Since the return value of value_type() is boost::typeindex::type_info&, how does boost::typeindex::type_id<T>() without .type_info() work?


Why do you need to change stdint.h to inttypes.h ?