1: D400 First line should end with a period (not 'e')
23:1: I100 Imports statements are in the wrong order. from pprint should be before from string
24:1: I100 Imports statements are in the wrong order. import os should be before from pprint
29:1: I101 Imported names are in the wrong order. Should be task_low_level_submission, task_update_progress, write_message
30:30: E126 continuation line over-indented for hanging indent
34:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibdocfile.api should be before from invenio.legacy.bibsched.bibtask
35:1: I101 Imported names are in the wrong order. Should be assure_quality, encode_video
38:41: E126 continuation line over-indented for hanging indent
42:1: I100 Imports statements are in the wrong order. from invenio.modules.encoder.metadata should be before from invenio.modules.encoder.utils
43:41: E126 continuation line over-indented for hanging indent
45:1: I101 Imported names are in the wrong order. Should be chose2, generate_timestamp, getval
45:1: F999 redefinition of unused 'chose2' from line 41
46:1: I101 Imported names are in the wrong order. Should be CFG_BIBENCODE_ASPECT_RATIO_MARC_FIELD, CFG_BIBENCODE_DAEMON_DIR_NEWJOBS, CFG_BIBENCODE_PBCORE_MARC_XSLT
46:1: I100 Imports statements are in the wrong order. from invenio.modules.encoder.config should be before from invenio.modules.encoder.utils
47:39: E126 continuation line over-indented for hanging indent
51:1: I100 Imports statements are in the wrong order. from invenio.ext.email should be before from invenio.modules.encoder.config
52:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.ext.email
55:1: I100 Imports statements are in the wrong order. import invenio.config should be before from invenio.utils.json
65:1: E302 expected 2 blank lines, found 1
65: D401 First line should be in imperative mood ('Send', not 'Sends')
65: D400 First line should end with a period (not 'g')
65: D205 1 blank line required between summary line and description (found 0)
76:13: E128 continuation line under-indented for visual indent
77:13: E128 continuation line under-indented for visual indent
78:13: E128 continuation line under-indented for visual indent
79:13: E128 continuation line under-indented for visual indent
80:13: E128 continuation line under-indented for visual indent
82:14: E128 continuation line under-indented for visual indent
90:1: E302 expected 2 blank lines, found 1
90: D401 First line should be in imperative mood ('Send', not 'Sends')
90: D205 1 blank line required between summary line and description (found 0)
108:13: E128 continuation line under-indented for visual indent
109:13: E128 continuation line under-indented for visual indent
118:13: E128 continuation line under-indented for visual indent
119:13: E128 continuation line under-indented for visual indent
121:14: E128 continuation line under-indented for visual indent
122:14: E128 continuation line under-indented for visual indent
123:14: E128 continuation line under-indented for visual indent
126:14: E128 continuation line under-indented for visual indent
127:14: E128 continuation line under-indented for visual indent
128:14: E128 continuation line under-indented for visual indent
136:1: E302 expected 2 blank lines, found 1
136: D401 First line should be in imperative mood ('Send', not 'Sends')
136: D205 1 blank line required between summary line and description (found 0)
152:13: E128 continuation line under-indented for visual indent
153:13: E126 continuation line over-indented for hanging indent
159:13: E128 continuation line under-indented for visual indent
160:13: E128 continuation line under-indented for visual indent
162:14: E128 continuation line under-indented for visual indent
163:14: E128 continuation line under-indented for visual indent
164:14: E128 continuation line under-indented for visual indent
167:14: E128 continuation line under-indented for visual indent
168:14: E128 continuation line under-indented for visual indent
169:14: E128 continuation line under-indented for visual indent
177:1: E302 expected 2 blank lines, found 1
177: D210 No whitespaces allowed surrounding docstring text
177: D401 First line should be in imperative mood ('Generate', not 'Generates')
177: D205 1 blank line required between summary line and description (found 0)
177: D208 Docstring is over-indented
188:1: E302 expected 2 blank lines, found 1
188: D210 No whitespaces allowed surrounding docstring text
188: D401 First line should be in imperative mood ('Store', not 'Stores')
188: D400 First line should end with a period (not 's')
188: D205 1 blank line required between summary line and description (found 0)
188: D208 Docstring is over-indented
197:1: E302 expected 2 blank lines, found 1
197: D401 First line should be in imperative mood ('Remove', not 'Removes')
197: D400 First line should end with a period (not 'r')
197: D205 1 blank line required between summary line and description (found 0)
221:25: E128 continuation line under-indented for visual indent
222:25: E128 continuation line under-indented for visual indent
223:25: E128 continuation line under-indented for visual indent
224:25: E128 continuation line under-indented for visual indent
238:1: E302 expected 2 blank lines, found 1
238: D210 No whitespaces allowed surrounding docstring text
238: D401 First line should be in imperative mood ('Create', not 'Creates')
238: D400 First line should end with a period (not 'n')
238: D205 1 blank line required between summary line and description (found 0)
239:29: E126 continuation line over-indented for hanging indent
255:1: E302 expected 2 blank lines, found 1
255: D210 No whitespaces allowed surrounding docstring text
255: D401 First line should be in imperative mood ('Create', not 'Creates')
255: D400 First line should end with a period (not 'a')
255: D205 1 blank line required between summary line and description (found 0)
273:1: E302 expected 2 blank lines, found 1
273: D210 No whitespaces allowed surrounding docstring text
273: D401 First line should be in imperative mood ('Create', not 'Creates')
273: D400 First line should end with a period (not 's')
273: D205 1 blank line required between summary line and description (found 0)
296:1: E302 expected 2 blank lines, found 1
296: D210 No whitespaces allowed surrounding docstring text
296: D401 First line should be in imperative mood ('Create', not 'Creates')
296: D400 First line should end with a period (not 'y')
296: D205 1 blank line required between summary line and description (found 0)
297:21: E126 continuation line over-indented for hanging indent
319:1: E302 expected 2 blank lines, found 1
319: D210 No whitespaces allowed surrounding docstring text
319: D401 First line should be in imperative mood ('Check', not 'Checks')
319: D400 First line should end with a period (not 'y')
319: D205 1 blank line required between summary line and description (found 0)
325: D210 No whitespaces allowed surrounding docstring text
325: D401 First line should be in imperative mood ('Sanitize', not 'Sanitizes')
325: D400 First line should end with a period (not 's')
325: D200 One-line docstring should fit on one line with quotes (found 2)
354:5: E129 visually indented line with same indent as next logical line
356:21: E128 continuation line under-indented for visual indent
372:1: E302 expected 2 blank lines, found 1
372: D210 No whitespaces allowed surrounding docstring text
372: D401 First line should be in imperative mood ('Processe', not 'Processes')
372: D400 First line should end with a period (not 'y')
382: D210 No whitespaces allowed surrounding docstring text
382: D401 First line should be in imperative mood ('Create', not 'Creates')
382: D400 First line should end with a period (not 'd')
382: D200 One-line docstring should fit on one line with quotes (found 2)
385:36: E225 missing whitespace around operator
399:5: E266 too many leading '#' for block comment
402:5: E266 too many leading '#' for block comment
405:5: E266 too many leading '#' for block comment
415:5: E266 too many leading '#' for block comment
430:21: E129 visually indented line with same indent as next logical line
433:21: E266 too many leading '#' for block comment
435:25: E266 too many leading '#' for block comment
436:47: F821 undefined name 'get_fieldvalues'
444:27: E128 continuation line under-indented for visual indent
447:13: E266 too many leading '#' for block comment
450:5: E266 too many leading '#' for block comment
457:5: E266 too many leading '#' for block comment
465:5: E266 too many leading '#' for block comment
468:5: E266 too many leading '#' for block comment
472:5: E266 too many leading '#' for block comment
480:13: E266 too many leading '#' for block comment
481:13: E266 too many leading '#' for block comment
482:13: E266 too many leading '#' for block comment
483:13: E266 too many leading '#' for block comment
485:27: E128 continuation line under-indented for visual indent
487:37: E126 continuation line over-indented for hanging indent
490:13: E266 too many leading '#' for block comment
491:13: E266 too many leading '#' for block comment
495:21: E126 continuation line over-indented for hanging indent
512:28: E128 continuation line under-indented for visual indent
514:9: E266 too many leading '#' for block comment
524:13: E266 too many leading '#' for block comment
532:13: E266 too many leading '#' for block comment
537:13: E266 too many leading '#' for block comment
540:13: E266 too many leading '#' for block comment
541:13: E266 too many leading '#' for block comment
543:50: E126 continuation line over-indented for hanging indent
549:13: E266 too many leading '#' for block comment
551:18: E126 continuation line over-indented for hanging indent
565:53: E261 at least two spaces before inline comment
571:13: E266 too many leading '#' for block comment
572:15: E271 multiple spaces after keyword
573:17: E266 too many leading '#' for block comment
588:47: E127 continuation line over-indented for visual indent
614:25: W503 line break before binary operator
615:25: E128 continuation line under-indented for visual indent
617:28: E128 continuation line under-indented for visual indent
618:28: E128 continuation line under-indented for visual indent
619:28: E126 continuation line over-indented for hanging indent
629:13: E266 too many leading '#' for block comment
631:17: E266 too many leading '#' for block comment
632:17: E266 too many leading '#' for block comment
635:21: E266 too many leading '#' for block comment
636:21: E266 too many leading '#' for block comment
639:21: E266 too many leading '#' for block comment
642:21: E266 too many leading '#' for block comment
646:21: E266 too many leading '#' for block comment
650:21: E266 too many leading '#' for block comment
651:21: E266 too many leading '#' for block comment
655:35: E128 continuation line under-indented for visual indent
658:37: E126 continuation line over-indented for hanging indent
663:13: E266 too many leading '#' for block comment
675:5: E266 too many leading '#' for block comment
679:5: E266 too many leading '#' for block comment
684:9: E266 too many leading '#' for block comment
697:44: E251 unexpected spaces around keyword / parameter equals
697:46: E251 unexpected spaces around keyword / parameter equals
698:50: E251 unexpected spaces around keyword / parameter equals
698:52: E251 unexpected spaces around keyword / parameter equals
699:49: E251 unexpected spaces around keyword / parameter equals
699:51: E251 unexpected spaces around keyword / parameter equals
733:5: E266 too many leading '#' for block comment
743:55: E721 do not compare types, use 'isinstance()'
743:69: E202 whitespace before ')'
753:32: E128 continuation line under-indented for visual indent
754:32: E128 continuation line under-indented for visual indent
755:32: E128 continuation line under-indented for visual indent
757:1: W391 blank line at end of file


Dahh, too quick on that one. Will revert the commit. I think we have to think some more about it. It's not as easy as expected. :/


1: M111 unrecognized component name: legacy


1: D100 Missing docstring in public module


1: D400 First line should end with a period (not 's')
1: D205 1 blank line required between summary line and description (found 0)
1: D208 Docstring is over-indented
34:1: I100 Imports statements are in the wrong order. from flask_login should be before import six
37:1: I101 Imported names are in the wrong order. Should be acc_get_role_id, acc_is_user_in_role
37:1: I100 Imports statements are in the wrong order. from .control should be before from .engine
38:1: I101 Imported names are in the wrong order. Should be acc_firerole_check_user, compile_role_definition
39:1: I101 Imported names are in the wrong order. Should be CFG_WEBACCESS_WARNING_MSGS, SUPERADMINROLE
42: D401 First line should be in imperative mood ('Create', not 'Creates')
42: D202 No blank lines allowed after function docstring (found 1)
51: D203 1 blank line required before class docstring (found 0)
51: D200 One-line docstring should fit on one line with quotes (found 3)
56: D400 First line should end with a period (not 'e')
56: D205 1 blank line required between summary line and description (found 0)
56: D207 Docstring is under-indented


Before posting the working solution to the exercise sandbox, consider using some lint tool like (http://jshint.com/) that would check if the code is valid or not. Use tidy tool to prevent bad formatting like `(opeartion_);


24:1: I101 Imported names are in the wrong order. Should be current_app, url_for
25:1: I201 Missing newline before sections or imports.


Since `type` is guaranteed to be either `Array` or `{primitive}Array`, maybe take builtins from it?


Please add missing coma after "admin_password"


single() and remove previous assertion ;)


Making this only usable during Playing mode will cause an editor Issue. Creating issue now...


My bad. I actually realised the stupid mistake I made just after the comment I made. I should delete this or edit it to not waste ur time. Thanks!


Not sure I understand this.... When ccache is installed (at least on my linux), it is automatically enabled:

sipadan-> which gcc
/usr/lib64/ccache/gcc

sipadan-> which g++
/usr/lib64/ccache/g++

Also please endif() not endif(CCACHE_FOUND)...


@robterodr It appears to work for python 2.6 and 2.7. The python 3.3+ detection appears to have an issue finding the correct library, but after looking into it I think this may have been an anaconda python issue.

We may want to pop the python 3.3 part until we can get a installation that works with python3+.

-Daniel Smith

> On Jul 17, 2015, at 3:26 AM, Roberto Di Remigio <notifications@github.com> wrote:
>
> Hi @dgasmith <https://github.com/dgasmith>, I think this could work as intended. CMake documentation is not particularly explicit on the effects of using Python_ADDITIONAL_VERSIONS. Have you tested it with the different versions of Python supported? @loriab <https://github.com/loriab> what is your opinion?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/psi4/psi4public/commit/02918632ccc1d0d94c92678552ea7929d59b3253#commitcomment-12219981>.
>


I changed this just because I could


This could potentially cause problems. i'd follow the previous lines and cache, set and restore rather than just force backface culling.


No worries!  [😊]

________________________________
From: riveryc <notifications@github.com>
Sent: Friday, July 17, 2015 9:51 AM
To: PowerShellOrg/cWebAdministration
Cc: Dave Wyatt
Subject: Re: [cWebAdministration] should be the last autocrlf issue to resolve (398c936)

My bad. I actually realised the stupid mistake I made just after the comment I made. I should delete this or edit it to not waste ur time. Thanks!

—
Reply to this email directly or view it on GitHub<https://github.com/PowerShellOrg/cWebAdministration/commit/398c936e7ec4559022d448f8e320f2fc81736c90#commitcomment-12225721>.


Can't see this as done:
> Rewrite countBs to make use of this new function.


It is ~just~ simply ... written and ~It~ is constructed ...


TeamCity Jarvis Framework :: CI-GitFlow [Build 30](http://build.prxm.it:8811/viewLog.html?buildId=3512&buildTypeId=CqrsFramework_CiGitFlow) is now running


Fixes #523.

Before:
![hover-before-ii](https://cloud.githubusercontent.com/assets/836140/8748621/afd7d36c-2c93-11e5-91ea-07edf00a8138.gif)

After:
![hover-after-ii](https://cloud.githubusercontent.com/assets/836140/8748623/b4b2ffd8-2c93-11e5-8ddb-7650a2c6ad55.gif)


Yo había llegado a la misma conclusión sobre el type de los eventos. Quizás en algún futuro se le encuentre funcionalidad pero por ahora lo dudo.

Así que :+1:


Ha! (saving that gif for sure!)


Consider:
```php
$cache_roledefs = $uncached + $cached;
```


muy bueno.


Count DNA nucleotides


Actually, looking at this more, I think this would work as written - the important part is that the indexes of array used in the binary search for the deviation are always the same.  Because scheduled arrivalTime is used both when constructing `scheduleDeviations` and as the index when searching for the deviation, the indexes would match, and the search would be successful.  This is dangerous, though, because its assuming that schedule arrival time is used in both locations - if someone came along later and edited it to use schedule departure time in one of the two locations (and arrivals were different from departures), this would fall apart and binary searches for deviations would fail.  There should at least be some comments stating that assumption, and preferably a test.


¿Debería? ¿No sería mejor que en onLogout los borre de la lista de jugadores en el evento?


Yup, silly mistake from trying to move too fast ;-)

Port at will. Anything I can do to help? Either way, naturally I'll just delete my decorator and replace it with the one in Astropy whenever it gets integrated.


`brew install` must not be run with `sudo`!


s/a/A stack is maintained;


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.1.0-unstable.14+14](http://build.prxm.it:8811/viewLog.html?buildId=3512&buildTypeId=CqrsFramework_CiGitFlow) outcome was **SUCCESS**
Summary: Tests passed: 113 Build time: 00:03:00


s/the/The argument ...


ok, i modify my parser, and i correct this ! ty


Consider:
```php
$rdefs += array($rd->roleid => array());
$rdefs[$rd->roleid] += array($rd->path => array());
```
or remove block entirely because you are only doing assignment and not addition.


:scissors: whitespace please


sure, fair enough ;)


Well, it was for more coherence with the rest (plus the fact that at the hackfest, people on inria wifi could not acce itk.org git...) . The fact that it's delayed doesn't really matter to us, we're using the release tags anyway


Na odstraneni pouzij unset - uz ho tam jednou mas


This should be reverse to match the new order:
```
 * [rdef] => [roleid][/path][capability]=permission
```


I think this has been sufficiently been discussed to go in a roadmap.
I have a very clear idea about the general capabilities this tool should have, that's why I initially asked for it and started with the very first pull request. Nothing substantial has changed since then, and it is a quite common "idiom" which is used in the [Edirom](http://www.edirom.de/en/welcome-page/) tools.

There are significant aspects of implementation and various parts of the user interface/functionality that haven't been discussed very much, but that shouldn't stop adding the tool's future to the roadmap.


Would you like to take https://github.com/letsencrypt/letsencrypt/issues/413?


LOL


LGTM


You're right for version check, I'll change that. I don't see indentation problems for the rest (at least it looks similar to the rest of TTK public to me)


i lold


[Fixed #13]


So many things to worry about.


Oh well hi there


Commit message should read:   Allow Partialling of mock_data.yml into a folder called data at the root


Just experienced the new crypto plugin mismatch functionality which is solved it by re-running the wizard again. Maybe when a mismatch occurs, why not display a pop-up to notify the user and prompt to re-run the wizard again to update the crypto plugin, instead of simply disabling crypto and carrying on like nothing is wrong.


Thanks for the advertisement bro, I'll send the payment to you in a second!


Closes #4


kek


Hmmm, I'm getting "Shortcode attributes decode error: No decoder found" in frontend when rendering a pricing table with buttons.

http://prntscr.com/7trnsr
http://prntscr.com/7troei

I traced this button to line 54 in https://github.com/ThemeFuse/Unyson-Shortcodes-Extension/blob/master/shortcodes/table/views/pricing.php

    <?php echo $button->render($atts['table']['content'][ $row_key ][ $col_key ]['button']); ?>

The content in my page:

[table table="{"rows":[{"name":"default-row"},{"name":"pricing-row"},{"name":"button-row"}],"cols":[{"name":"default-col"},{"name":"default-col"}],"header_options":{"table_purpose":"pricing"},"content":[[{"textarea":"\\t\\tColumn 1\\t\\t"},{"textarea":"\\t\\tColumn 2\\t\\t"}],[{"amount":"100","description":""},{"amount":"200","description":""}],[{"button":{"label":"Contact","link":"#","target":"_self","color":""}},{"button":{"label":"Contact","link":"#","target":"_self","color":""}}]]}" _array_keys="{"table":"table"}" _made_with_builder="true"][/table]

... so, this is how the atts for the button are stored:

    {
        "button": {
            "label": "Contact",
            "link": "#",
            "target": "_self",
            "color": ""
        }
    }

When I do simply a button shortcode, it's stored like this:


The -rc series moves so fast that I normally don't add it.
If you really want this version in, please add a ```depends on EXPERIMENTAL```.


lol


This has way too much PHP logic. You know the input, so you also know the output (like it should be).

I recommend simplifying this by hardcoding it a bit more.


Is elobuddy dead btw


(y) is that on the roadmap or just a self initiative?


If this is going to be official, please let's put this in nixos-artwork rather than a personal repository: https://github.com/NixOS/nixos-artwork


Thanks @DomT4 .

RE: the `head` issue -- should this line have been removed? Just want to clarify.


Fix misprint.


I killed it off in https://github.com/Homebrew/homebrew-versions/commit/1ea60320b3b8a530fa9d0f5f6f116362b5f1b118 :)


I think unit tests are better suited to provide examples.


So is never called with more than one blacklisted edge?

I prefer my single line stream solution but perhaps is more readable this way. up to you.


@lethalman can I do a submodule in that repository?


Yo


Man


Still waiting for the ads monei... :clock1:


Since this is already merged (see http://docs.julialang.org/en/latest/devdocs/ast/), these would be better as a pull request. My request was for something I was hesitant to write myself.


Yeah, I didn't know how to cleanly do that with the suppressMessages, maybe
should have just include the library(methods) on the line above

On Fri, Jul 17, 2015 at 9:33 AM, Dirk Eddelbuettel <notifications@github.com
> wrote:

> Thanks for cleaning up on this.
>
> I do find the two-statements-on-one-line a little icky though. Not that it
> matters :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jjallaire/rcpp-gallery/commit/7f0ad87050c7ff34c2b663578e855000a745f651#commitcomment-12225397>
> .
>


Yep, anyway have nothing better to do


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


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.360](http://teamcity.monogame.net/viewLog.html?buildId=8821&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 522, ignored: 6 Build time: 00:27:22


I had remembered my adventures with ``Python_ADDITIONAL_VERSIONS`` in Feb 2015 as giving mismatched libraries, but the problem really was that you tell Psi4 to detect one verno of Python Interp/Libs then there was another program (swig) w/o the constraint that detected another verno. At runtime, problems ensue.

Now, I would say we should be forwarding our detected python *to* that program like we do math libs to chemps2. Let's give ``Python_ADDITIONAL_VERSIONS`` a try, adding 3.4.


Please accept PR-https://github.com/DZ191/LeagueSharp/pull/26 to get the payout!


Hi fin Kappa


forgive me, but I'm confused as to where these magical constants came from. I get where Russ picked 96 because I've seen a few sources say that was a historical value used for Windows. And I see a source that mentions 96 for WPF and 160 for Android here: https://en.wikipedia.org/wiki/Device_independent_pixel

I think we may need to provide another value to the layout engine to use for dp/dip calculations. For Androidn we can use a 16- constant, but for Windows, I would think the value must come from some of the values we used to calculate the PPI here: https://github.com/appcelerator/titanium_mobile_windows/blob/master/Source/UI/src/WindowsViewLayoutDelegate.cpp#L842

In particular, my guess is that the dp calculation would be: parsedValue / info->RawPixelsPerViewPixel


Oh I think

```r
suppressMessages({
   library(methods)
   library(Matrix)
})
```
ought to be fine.   I usually just line up several `suppressMessages(library(foo))` in a row ...


Yeah!


Such advertisement much qq


Btw we can make `getArrayElementType` static


Not sure I'm qualified to comment, but the error message refers to MS Studio 2012 or 2013 but the test is only for MSVC12. Reading a little I suspect that the 2013 version has the same tag... But there is also a MSVC14, which should be fine. Maybe we should check MSVC_VERSION instead, it will simplify the test even though the version numbers are not as well known as the year of the compiler...

http://www.cmake.org/cmake/help/v3.3/manual/cmake-variables.7.html?highlight=msvc
http://www.cmake.org/cmake/help/v3.3/variable/MSVC_VERSION.html

Similarly, the second patch seems to mandate Visual Studio 2012, it would be nice if it also worked with newer versions.... All this microsoft stuff seems so fragile !!!


tst


Ismerős a hiba.
A múltkori frissítésnél én az utolsóra snapshotra frissítettem, aminél még nem volt ez.
A könnyebb utat választottam.


Parentheses are not needed


Okay, fixed!

On Fri, Jul 17, 2015 at 10:10 AM, Dirk Eddelbuettel <
notifications@github.com> wrote:

> Oh I think
>
> suppressMessages({
>    library(methods)
>    library(Matrix)
> })
>
> ought to be fine. I usually just line up several
> suppressMessages(library(foo)) in a row ...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jjallaire/rcpp-gallery/commit/7f0ad87050c7ff34c2b663578e855000a745f651#commitcomment-12226100>
> .
>


Oops, I missed that it's merged already


- Did not implement the detail view, i.e. opening the image after tapping on it.
- Did not implement "About"


hi finndev u pay for me pls?


ditto, parentheses


I've just realised that Status Code 500 (not 400) is returned when no parameters are provided.


По идее, надо бы завязаться на размер иконок. Раз высота тулбара в винде и в линуксе разные, а иконки одинаковые. Но вообще можно и на пропорциях все сделать. Только выходит, что размер шрифта тоже придется подстраивать.


Besides that, this can safely be pushed into the dtk1.0 branch. I do not see the need to multiply the branches...


Fixes #523 for real this time.


Ok I think Bank might have made some changes and I don't know exactly what
they are. In any case, it highlights a pertinent difficulty for the
de-duplication stuff, namely when first and last names are switched, or the
division between them varies. Seems like one would have to do something
greedy to catch these cases (of which there are quite a few).

2015-07-17 15:43 GMT+02:00 Robert Forkel <notifications@github.com>:

> hm. it seems the merge algorithm isn't working as well(?) as before. It
> seems as if - in this case at least - it created a new duplicate:
> http://glottolog.org/resource/reference/id/472445 which was merged with
> http://glottolog.org/resource/reference/id/116817 before?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/clld/glottolog-data/commit/998e3a985672a8d00913b4bb273ebc18d9b39499#commitcomment-12225588>
> .
>


instead of duplicating all those cases, could you make a test_prop_sum_opt? that would test both of them at the same time?


Do not commit that ! TDE should fix the controller !


Doesn't*


Just a question - if the method has no params and it runs the second check, will the check fire a NPE? It might do to have a null check here to be safe, because if params is null/empty the params[0].equals method call will fire a NPE.


I think this should be pretty printed in order to understand it easier.


Excuse my ignorance of protobufs but what is the interpolation message for?


Typo bruh.


```python
import random
list = ['red', 'white', 'blue']
random.shuffle(list)
print list
```
Ref.: http://ubuntuforums.org/showthread.php?t=555188&p=3394356#post3394356


This change seems unintentional? `DLYV` is not defined anywhere.


r+


saw approval from graydon
at https://github.com/stellar/stellar-core/commit/fbc8b0970ca9cd267a3f5840755d4063114c1d52


merging stellar/stellar-core/custom-travis = fbc8b097 into auto


stellar/stellar-core/custom-travis = fbc8b097 merged ok, testing candidate = 7ce3dde7


r=jedmccaleb


세미콜론이 안붙었네요. 추가부탁드립니다


I'm getting the same error:
```
hydra-eval-jobs returned exit code 1:
error: access to path ‘/nix/store/vcrz2n65l29bi6bfajmsprk3y2h95nvm-nix-1.9pre4087_afa433e’ is forbidden in restricted mode
(use ‘--show-trace’ to show detailed location information)
```
I'll revert back to a commit before the strict mode.


@latobarita: retry


I've got a problem to compile the whole project in intellijj idea...
Wanted to switch - but its a pita to get swt work in intellij...

now i'm going back to eclipse...


This code is a terrible mistake, fixed it there https://github.com/playcanvas/engine/pull/358/files


lol


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


Há um erro na linha 27,  onde está "Oops! Esa página não pode ser encontrada.\n", falta um "S" em Essa.
Atenciosamente.
Francisco David


For posterity OC changed their version number as off 8.0.0

"Version numbering changed with ownCloud 8.0. Major releases are now indicated by the second number, so 8.0. 8.1, and 8.2 are major releases. The third number indicates an intermediate release, e.g. 8.0.5."


Specifies the inclusion of fields to return for all matching documents. *


This one stems from the order of the title words now being significant (95ef7f684a6356c82a2258a88aa7402ff9ace1d7).
I tested different possibilities and this created far more good merges/splits than bad ones (like this one).
If we want to assess/tune the merging/splitting, the (now more informative) merge/split-messages from `monster.py` (or the outputs of `_bibfiles_db.Database.show_*()`) should provide better metrics.


@jozefg Yes... "Formally" what I suggested can work, but you are right that for the proper meaning explanation, we must understand what it means for `A` to be a type beforehand in order to avoid impredicativity.

A set-theoretic version of CTT's semantics is given in Allen's dissertation, but as I mentioned, it is pretty complicated... If I recall correctly, each type is explained as a PER (of types); then, the reflexive elements in those PERs are then assigned PERs (their canonical elements).

This lets you explain equality of types on a type-by-type basis, rather than all at once as in ETT. For example, we first have a PER which contains `Pi(A;x.B) = Pi(A';x.B') (A = A' in U{i}; x : A >> B = B' in U{i})`; then for each member of that PER `Pi(A;x.B)`, we have a PER of elements that contains `λ(x. E) = λ(x. E') (x : A >> E = E' in B`. The definition of a PER of types as above is called a "multitype".


When using nested queries, for example with "select * from table1 where id not in (select id from table2)", DBFlow adds an extra set of parentheses around the select statement:
select * from table1 where id not in ((select id from table2))

With Sqlite, this extra set of parentheses is a problem as it is changed to a SCALAR SUBQUERY, causing the inner select to return one element instead of a list. The result is thus far from the intended one.

Please also see this link for further reference:
http://sqlite.1065341.n5.nabble.com/Unexpected-behaviour-when-use-quot-IN-quot-clause-and-double-parentheses-td81057.html


You should specify here the default values for this params.


what to do with it ?


<section> should be at the same indent level as <header>


Why is this removed?


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


we add the vix skins to the atv feed. thx rossi.


I've now moved the stuff in the TODO file (90e84de0)


I'm not sure tha this should be here since we're not returning the roles yet. I think I'll implement this in the next pull request.


I haven't had any debug warnings but I'll remove it!


For clients that want to query based on user-supplies values. Instead of providing a safe escaping function for a half-dozen languages, clients can just do placeholders and provide a key-value mapping.


Interesting... These are the first two events, and they are written in this order. Note, however, that the event at line 55 has a time of "3000" (it is the "real" starting time of the Parallel Haskell system). If you have changed the parsing behaviour wrt. ordering by time, the lines will probably be swapped.
Could that be the reason?

BTW this test file is quite big, I should probably provide a new, smaller, one.


test comment


页面文件，在本地浏览，排版ok，可是，git到github之后，IE浏览器浏览后，出现排版问题，


ack


I love this method of using case statement Scott. It is very easy to read and soft on eyes too. I think I'll go with the case statement instead of if-else.

Though not relevant in this scenario, I was wondering how would we deal with numbers that are not whole numbers (say some weird decimal like 4.2389) using such an arrangement ?


I guess the "proper" fix would be to write a real lexer/parser so that it can distinguish strings, I think at the moment we have [this](http://stackoverflow.com/a/1732454/874671) sort of problem :)

That might be a lot of work though. I'm actually doing the coursera compilers course at the moment, so when I'm finished I might want some practice... maybe..


count c'est pas un int ?


you should put this in configuration


ack, maybe a comment would be good so noone adds this later?


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/71424994


c.f. http://guides.rubyonrails.org/active_record_querying.html#conditions


Otherwise you get a double-free in rd_kafka_q_destroy as it purges the queue.


I think this should be UserNotFound*


exit a script is not a good practice.


typo in fdefault


Shouldn't this be `user_socket.ex?`


the whole sentence is superfluous, the one before mentions that its for linux only so I think its even clearer leaving this one awat


As can see, that views_delegate_ is defined in wrong place, needs moving to better place...


That's a laravel issue ;)


s/awat/away/


why the extra newline?


Why would you set dataRegistry in the LocationManager rather than when projectController is created in the KickStarter class?


>=5.4.4 as for now you didn't used any 5.5 specifics


grammar


why don't you put the example command in there to make it people easier just like below?


Well, `afa433e` predates my fix, so it's not surprising it doesn't work.


Is the expected error code in test1801 guaranteed to always be the same?

I get ```CURLE_RECV_ERROR``` (Recv failure: Connection reset by peer) on Fedora ARM builders whereas ```CURLE_HTTP2``` (Send failure: Broken pipe) everywhere else.  Tested with ```curl-7.43.0``` (which includes this fix) and ```nghttp2-1.1.1```.

I came to conclusion that curl works as expected but the test is based on an assumption that is not guaranteed to hold in general.  Is my conclusion correct?


r+


munkres is done automatically via setup.py, we agreed to not have those in here


Nice!


See issue #2.


aaaah you need to do it manyually because travis doesn't support this stuff on mac yet, right?


Heh, you beat me to it :)


dont like the j


There is an issue with this. If the job already exists, it won't get updated :(


-sa not+sa


My main point was of course if the userguide is the right forum for the roadmap. But it also seemed to me to be premature to state a roadmap when there is an ongoing (or rather postponed) discussion of the future development of the tool.


fast-forwarding master to auto = 7ce3dde7


awesome


don't you want to set `os: mac` or so explicitly?


Ага, все сложнее
- разница в единицы пикселей по высоте приводит к разным пропорциям в винде/линухе
- под ретиной вообще не смотрели

Если с константой 24 прокатит под ретиной, то думаю к ней вернуться. Потому что это самое простое решение из всех.


you need to state that this is known to work but not supported yet


cc @jgranick


Sorry, I don’t know which for loop you’re referring to.

> 16 jul 2015 kl. 16:07 skrev George Fletcher <notifications@github.com>:
>
> Hello,
> is it on purpose that the for loop is ended by this return (or the other return statements underneath) ?
> Is the data dict always only one entry?

- Roland

”Capitalism is the extraordinary belief that the nastiest of men for the nastiest of motives will somehow work for the benefit of all" - John Maynard Keynes.


Is this not used?


Discussion at https://groups.google.com/d/msg/cryptopp-users/__m0euOdbEQ/tvINItctzjAJ.


en effet ! $count === 0


Well, the actual "roadmap" of this tool is much more clear than that of the other items starting with line 75 of the [TODO file](https://github.com/wbsoft/frescobaldi/blob/master/TODO)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.361](http://teamcity.monogame.net/viewLog.html?buildId=8825&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 522, ignored: 6 Build time: 00:28:24


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


I don’t know which method you’re referring to.
There is nothing in the message that tells me that.

> 17 jul 2015 kl. 13:42 skrev spaceone <notifications@github.com>:
>
> This method seems unused. Can it be removed?
>
> —
> Reply to this email directly or view it on GitHub.
>

- Roland

”Capitalism is the extraordinary belief that the nastiest of men for the nastiest of motives will somehow work for the benefit of all" - John Maynard Keynes.


Will there be a concept of a default page? With the old admin if I don't specify the path it redirects to the index page:

```
$ curl -vs 'http://localhost:8077'
> GET / HTTP/1.1
> User-Agent: curl/7.37.1
> Host: localhost:8077
> Accept: */*
>
< HTTP/1.1 302 Found
< Location: http://localhost:8077/baseserver
< Content-Length: 0
< Server: Jetty(6.1.26)
<
```


My point in this test was to generate a custom wildcard called $distance. It's a simple example that calculates the distance between the argument (7) and the input value (4). I do understand that line 82 might be a bit overwhelming so I guess we could use a more simple approach.


It was duplicated below


has this ever worked for u?


Good point on struct vs class with respect to IEquatable, @roji. My comment on optimization relates to storing the (X,Y) pair in the PostgisPoint object itself instead of through a BBPoint. I think everything else follows the OGC spec closely -which is commendable-, and this is a departure which causes a loss of clarity, for those versed in the field.

As far as the value of structs here, I'll add a separate note below


Discussion at https://groups.google.com/d/msg/cryptopp-users/LtXUZj8UbV4/NTxspaT4QrYJ.


great


the `os: mac` is a beta method ... this is the current supported method. The "objective-c" language automatically uses OSX


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/31844c0276759c876ed7c4ecfc6143adee3fc17b


merging graydon/stellar-core/bug-615-flush-tombstones = 31844c02 into auto


graydon/stellar-core/bug-615-flush-tombstones = 31844c02 merged ok, testing candidate = ad891fe8


This is then similar to issue we had with installing plugins once the Github auth module was installed. We can either revert the merge then at this stage, fix the issue and remerge or create a new PR to fix the state of master. I'd prefer to not have master in a known bad state but open to options.


Yep - we can switch to the `os: mac` thing when it becomes supported


Magnus, if you accept the change removing the call to rd_kafka_op_destroy then you should be able to substitute TAILQ_FOREACH for the TAILQ_FOREACH_SAFE and get rid of the tmp variable. -karl


then a comment would be nice

2015-07-17 16:39 GMT+02:00 AbdealiJK <notifications@github.com>:

> the os: mac is a beta method ... this is the current supported method.
> The "objective-c" language automatically uses OSX
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/5aea1fd0e8e71d29a395ab83c9ac3857e2f334ff#commitcomment-12226759>
> .
>


Why you need Game instead of Board?


Hi @pasfreelance, thanks for your contribution!

This needs a couple of edges sanded off. Fist of all - the error you cited is RDBMS specific. I would prefer if the heading was called something neutral, and there was a code block (one level indented) that listed samples from several popular engines (MySQL, Pg, SQLite, maybe ask in the channel for couple others).

Also thr `DRY` principle doesn't really apply to documentation - you need to add a short paragraph on how to  use `->current_source_alias` (in addition to linking to the more substantial docs) so that the reader sees right away what to do.

Thanks in advance!


The same. Is Board is not enough to print?


This should be
```go
confirm = make(chan amqp.Confirmation, 1)
```


Since BBPoint is a struct, do the conversion to array result in the creation of copies and new memory allocations?


Thanks for the doSave/Close/Cancel() functions. Still only 1 minor bug. When integrating Ace or CodeMirror, if you hover the dialog buttons (Save, Cancel, etc.) and you hover the textarea again, the focus doesn't return. Hovering the buttons should invoke only the `:hover` style and not `:focus`, don't you think?


I've just changed the our hydra's channel to 14.12-small (I recall this was updated much faster) I'm hoping this channel has got it.


@carpodaster no worries, we will resize/compress it :computer:


I wonder how long it will take for this API key to hit its rate limit. Unless, of course, there is no rate limit. I guess we'll know when the tests start failing.


I think the notion that payers get to enter a fiat amount and use their own rate is flawed. That's not how bitcoin payments are executed at the moment, for some reasons mentioned in this BIP and others too.


Missing commit prefix: `oc-mail`


Game is a POJO that contains Board and Player, signature changed to reduce number of method inbound parameters.


LGTM!


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1241 of 1241 tests passed)

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


The same as I've explained beforehand.
Game is a POJO that contains Board and Player, signature changed to reduce number of method inbound parameters.


nice


Worst decision ever :/

At least you should assume your choice and remove the useless "ircmaxell/password-compat" dependency.


nit: rtype


How about `1.0.1`? `1.0.0` would prevent using exposify with non-valid identifiers, which is what prompted me to post this fix to begin with https://github.com/bendrucker/globo/pull/2


Зачем потёр Csrf?


> Worst decision ever :/

Huh? Why?

> At least you should assume your choice and remove the useless "ircmaxell/password-compat" dependency.

It was already dropped.


This is an LTS release so we want to support a php versions that will still be maintained in a year's time.


Yeah it was dropped in #8932.


Great! (the todo file is a good place, I think!)


Wow.  Great work!


@HelloWorld017 제보 감사합니다. 다음 커밋에서 같이 수정하겠습니다.


Well done bro.. We are so lucky to have you :-)


Hmm I see that it broke https://github.com/thlorenz/browserify-shim/issues/180. Perhaps we can find a way to make it work for both


I'm just looking at the diffs on github, the comment (and the code after) starting with "//  Now" has only two spaces before it , whereas the code before it has 3 spaces (which is strange). But this is probably an artifact of looking the diff on github.


It doesn't have to use `lambda` and the fact that it takes a single argument is not mentioned.


closing </head> tag should be at same level as opening tag


Hmm... I'm not sure about tightening it so much... but maybe it's ok.

If so, then the 2nd condition should set it to `Development/Behat` as the 1st condition would not be executed. It relied on the fact that during Behat tests you'd access it under something which contains behat and dev in host name (e.g. behat.dev.domain.com).


Also, I think, maybe we shouldn't create ```TimepointPredictionRecord``` if there is no exact ```arrival``` or ```departure``` time.
Here is HART gtfs-rt example:
```
entity {
  id: "trip_update_89415"
  trip_update {
    trip {
      trip_id: "89415"
    }
    stop_time_update {
      stop_sequence: 54
      arrival {
        delay: -120
        uncertainty: 30
      }
      departure {
        delay: -120
        uncertainty: 30
      }
      stop_id: "4458"
    }
    vehicle {
      id: "1007"
    }
  }
}
```

Here is PSTA gtfs-rt example:
```
entity {
  id: "1"
  trip_update {
    trip {
      trip_id: "1207248060"
      route_id: "18"
    }
    stop_time_update {
      arrival {
        time: 1437145633
      }
      stop_id: "3582"
    }
    stop_time_update {
      arrival {
        time: 1437145633
      }
      stop_id: "686"
    }
```

Since, there is no absolute arrival times for HART, I think we shouldn't create ```TimepointPredicitionRecord``` in ```GtfsRealtimeTripLibrary```. @barbeau  What do you think?


:heart:


Yes you are right, I will update the example. Thank you for the heads up


Why not just raise `SwaggerValidationError`?


remember to include an alt="" property in your <img> tags so that if the image doesn't load the text will.


inconsistent with parent pom, must be `findbugs-maven-plugin.version`


Yes, that sounds good!


This is not nested inside of anything more than the <article> tag so it should not be more indented.


Can this console.log be removed?


What is `v1.2`?


Discussion further down in thread at https://github.com/weidai11/cryptopp/issues/4.


sorry, I did this in a bit of a hurry :|


thanks!


This is an awful awful hack, abusing the way `lazy val` works in really unexpected ways:

* 'lazy' so that it doesn't get evaluated immediately (ie before the auth kicks in)
* also lazy' so that it doesn't get executed more than once.


This commit would need more proper formatting. The splitting is good imho but extra blank lines can be found in the code, also docstrings are missing and some identation are not according to the ones we are trying to follow in OWTF (even though are pep8-valid) regarding the 120chars longs.


Thanks for fixing.


It looks like we're going to have to add our own Tags and Groups buttons and provide the functionality for them ourselves. WP FEE offers no such functionality - it simply looks like unfinished code, and indeed the master branch of the Github repo seems to have removed the toolbar altogether.

I've hacked it such that we can actually do so with the included lib. FWIW, I use a modified version of 1.0.4 with the same class `FEE` as the one in our lib.


can you make this a separate condition?  we changed the config keys so that we run 'powershell MSSQL' only once per instancename for performance.


Should that : be there in this scenario? It is not present in ln. 38.


Nicely Done!


> I've hacked it such that we can actually do so with the included lib.

 See 6482497897eacd364bb3ec3842d3171e90846d5b for the action I added.


The splitting for the new request is really hard to read. Several splits on different values with different indexes. Would it be possible to make it easier to read?


@GrahamCampbell @willrowe thank you for information, nice catch.
Too bad to drop PHP 5.4 support as you are based on Symfony2.7 which stay the PHP 5.4 support, maybe can you make this requirement not mandatory ?

For now, Laravel LTS can be used with 5.4 right ? I think about a system like many projects does for PHP 7.0: let the tests run on 5.4 but allow broken tests on 5.4 as the core team don't want spend time on deprecated version, what do you think ?


Hello friends lets start our project library management system


This is what caused the NSEntity error!!


Это ошибочный коммит?


:tada: Thanks! -j


OK I am getting the same error on my mac. I think the problem is mac specific. I'm looking into it.


why are you adding '$instance' to the end of the instance name only to remove it later?  did something come up during your testing?


HelloWIN


"please see docs/cfsb.md"


``print`` statement is not python3.x compatible. Please use ``from __future__ import print_function`` and ``print()`` to keep the compatibility across versions.


Looks good.


We have no intention of supporting PHP 5.4.


reack


ack


+1 for this functionality


unsupported note?


- prevent .component-background being applied to blank component.
- .blank-component class updated for scaling.


Oh, i don't know why, i thought seconds ;-)


@johnpaulbalagolan Please be more careful about overwriting my changes. It's annoying having to fix things multiple times.


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


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.362](http://teamcity.monogame.net/viewLog.html?buildId=8829&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 522, ignored: 6 Build time: 00:22:35


+1 - I vote for this enhancement!


Well shit, I didn't rebase it.


Nice to start this off with adding a test.


yeah, I don't know :)


用 ~1.0.0


s/manager/management/
s/wirte/write/
"has three nodes" => "has at least three nodes"
"internally one is selected as a write master, which is transparent to outside clients (eg. CF CC)"
"The write master talks" => "The management cluster talks"


I think you must have missed: `npm install --build-from-source`. Docs at https://github.com/mapbox/node-pre-gyp#options


Changing again baseLayer stuff :D.


s/create/delete


This function is poorly named because it does not do what the name seems to indicate. When finding functions named ``is_something()`` we expect to have this function returning a boolean (``True`` or ``False``).

It would be better to rename that function according to what it does.


Cool, thanks - I was just working on the groups stuff this morning.

On 07/17/15 10:56, Christian Wach wrote:
>     I've hacked it such that we can actually do so with the included lib.
>
> See 6482497
> <https://github.com/cuny-academic-commons/social-paper/commit/6482497897eacd364bb3ec3842d3171e90846d5b>
> for the action I added.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cuny-academic-commons/social-paper/commit/789aa14c59d6f4b65e633c99cb99c1c4769bdd6f#commitcomment-12227075>.
>


Thanks.  Fixed in d5d23470a4b6277bcfdc54e6c0dc7c398fb26946.


Same as ``is_https()``.


OK. This is gonna be a lot more verbose (we have to allow all the parameters explicitly) but I agree it'll be safer and cleaner. :+1:


nice job including the target attribute on your <a> tag


That was resolved by bendrucker/globo#3

This test needs to pass before I can unpin globo and exposify can support non-identifiers: https://github.com/thlorenz/exposify/blob/61d8991641d0bda9f2413e4f8ceea534dd1aead6/test/lodash-noconflict.js

And that will happen via bendrucker/accessory#1 when I get around to it


can you call `setMinervaMetadata` instead?


@taylorotwell roger that, but why restrict the installation on PHP 5.4 environments ?

Or do you agree to document the installation with composer using ``--ignore-platform-reqs`` option , but indicates Laravel team won't support PHP 5.4 ?


It should be on the global .gitignore.
Try to validate your package on SensionLabsInsight ;)


ack, can you cache brew stuff?


I did a clean build and it failed the first time through unable to find the include path:
[ 48%] Building CXX object Base/QTCLI/CMakeFiles/qSlicerBaseQTCLI.dir/vtkSlicerCLIModuleLogic.cxx.o
/Users/nicole/Slicer-git/Slicer/Base/QTCLI/vtkSlicerCLIModuleLogic.cxx:21:10: fatal error: 'vtkMRMLMarkupsStorageNode.h' file not found
#include <vtkMRMLMarkupsStorageNode.h>
         ^

I added a print out in the QTCLI makefile to check and then redid the compile and it succeeded:
-- Configuring Slicer Qt base library: qSlicerBaseQTGUI
Markups Source = /Users/nicole/Slicer-git/Slicer/Modules/Loadable/Markups/MRML ; Markups bin = /Users/nicole/Slicer-git/Slicer-build/Slicer-build/Modules/Loadable/Markups/MRML

I would argue that since I only need the MRML library from the Markups module, it shouldn't be an issue to link from QTCLI to the Markups MRML.


@lindamartin It seems a bit strange to deprecate an overridden method: should we deprecate JavaScriptTree.getLine() instead?


"talk to the write master" => "talk to the management cluster"
"catalog, instance provision and binding , instance unbinding and deprovision." => "catalog, instance provision/de-provision/bind/unbind."
"from service" => "from a service"


add todo asking if applying in groups would make result MethodHandler faster


Even though the scenario I described in issue #650 is rare, I think a table would be of at most benefit.


понял


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/71428069


please don't copy-paste code in the future; just merge upstream/master


No - the cache directory for pyenv also doesnt work. I'm removing it in the
next push
Ive asked a SO question (Travis>Help takes me to StackOverflow)

http://stackoverflow.com/questions/31475294/travis-not-cacheing-pyenv-in-osx-environment

On Fri, Jul 17, 2015 at 8:33 PM, Lasse Schuirmann <notifications@github.com>
wrote:

> ack, can you cache brew stuff?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/0f5107eea27be5a5eac0ac736111e780889b3a6c#commitcomment-12227235>
> .
>


No, we want to have the original return, and DRY and stuff


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1244 of 1244 tests passed)

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


> ... or how we created cross-cloud

We didn't learn to do this, it's what we've been doing forever now


OK, I have created a smaller test file. Same order of events here; as mentioned, they are written in this order.
FWIW our EdenTV software ignores the startup event, so it should not matter for us if they come out of the new parser in a different order.


s/.It/. It/
s/avaible/available/
s/reconsile/reconcile/


@flori please deploy. thx.


I like how we go over this, but I'd love even more to see how we took this and made it repeatable in the charms


I'm confused about this, should it instead be?

    dataset_folder = findDatasetFolder

I probably don't understand your data model


scheduler.md => docs/scheduler.md


Watch out for the number of ids you are using for selectors. It is better to use descendant selectors than it is to put an id on everything.


Remove this line for now, we'll fill in later.


Should this have a different method name? Like "initializeSlider()" or something to set it apart from the Slider class's setSliderSize()


Something like "Leveraging Juju to model the infrastructure we were able to condense [the install instructions], [tuning guides], and [benchmarking instructions] into composable, repeatable, executable architectures


I think this commit changes resolve cancel promise behavior. It is returning true instead of clicked item index:

```javascript
$mdBottomSheet.show({
                    preserveScope: true
                }).then(function(clickedItemIndex) {
                    // clickedItemIndex returning true instead of index
                    listItemClick(clickedItemIndex);
                });
```


@pynicolas indeed but the thing is that we have no replacement from the JavaScriptTree point of view, have we ?


@lifehackett @seanperkins @Ortuna if one of you could verify that this is correct, I think we can close these PRs and push it all to production.  Thanks! :)


Seeing as the items in almost any shop are going to be priced in fiat then the payer is always going to be paying in a fiat amount, the amount of Bitcoin is of little matter.


I like this, we may even want to talk about networking here as well


Think i fixed new lines also :)


This is really weird, it doesn't happen on my systems..

I'm going to roll it back right now again.

Sorry for the inconvenience.

 @lanoxx again job for you ;-) try to get a development environment based on ubuntu (I'll do as well)


I'd almost want this to be the first results, to show a progression of the benchmark much like with the stress sized instances


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


We don't intend to support anything older than 5.5.9, and inf act we strong recommend nobody uses php 5.4. It will stop receiving security updates from the core team very shortly.


When you change multiple border properties for one element, try using the shorthand to reduce the length of your code.


I'd totally nix this post, we should have an /about/benchmarking page which covers this


Could we trim this list down to `Microsoft.AspNet.Mvc.Abstractions`? Doesn't everything Mvc have to reference that assembly indirectly?


saw approval from graydon
at https://github.com/stellar/stellar-core/commit/e419789c6df25445844a8d1fb8c3a76c53e01846


merging stellar/stellar-core/update-xdrpp = e419789c into auto


this is list of files to be ignored.
i just added backup files that are created by gedit.


stellar/stellar-core/update-xdrpp = e419789c merged ok, testing candidate = 85d4ed2b


Too bad, but thank you for this clear answer.


@cagryInside I believe its still ok to create `TimepointPredicitionRecord` for delay values (i.e., when no absolute time is provided).  There are three possible cases here for the type of data we **could** get from GTFS-rt real-time and schedule GTFS data:

1.  *Agency provides absolute times in GTFS-rt* - Whether or not we have a schedule arrival/departure time for the stop, we know exactly what predicted time we should show the user (predictedTime - now).  This is the case with PSTA.
2.  *Agency provides delay values in GTFS-rt, but provides schedule arrival/departure time in GTFS data for that stop* - Because we have a schedule time that comes directly from the agency, we know exactly what predicted time we should show the user (now + delay).  This is HART's case.
3.  *Agency provides delay values in GTFS-rt, but does NOT provide schedule arrival/departure time in GTFS data* - This isn't allowed in the GTFS-rt spec, because we don't have a schedule time to apply the delay to.  The best we could do is interpolate a schedule time for this stop from the other schedule times, but each GTFS consumer would likely do this differently.

So, for cases 1 and 2 above, I think we should still create `TimepointPredicitionRecord` records, because we know exactly what time we should be showing the user for that stop.


Thanks for testing. That confirm my hypothesis.

To summarize: the current observation is that QTCLI depends on Markups module. This is a problem because modules are built after the base libraries (SlicerQTBase, SlicerQTCLI, SlicerQTGUI, ... ).

In short, you should remove the dependency of QTCLI onto a specific module (Markups in that case).

I think what I suggested will allow you to do so with minimum change.

Ideally, we would like to generalized the CLI infrastructure so that new type of node can be registered. But that would be more work ...

Let me know if you have any questions,
Jc


Because the form factory does not set the method to 'PUT', $form->handleRequest silently fails.


And for case 3 we should be filtering out the data and throwing warnings when seeing this type of erroneous information, but we'll leave that for another patch :).


I'm aware of the issue and created a PR https://github.com/FriendsOfSymfony/FOSCommentBundle/pull/479/files


:sparkles: :joy_cat: :sparkles:


Is this for something like callpower? Don't you think we'd want a dedicated widget for that?


Makes sense.


reack


@lmadeira - I think it is a different commit that may be causing you some issues. Please file a new issue with a demo and version details. Thx.


This should start at 0.


utilities


True. But all the other languages suck in some other way; I chose JS because it has the least number of major handicaps when it comes to writing something like this.


Remove this print


the readme says php5.5+


You are not ensuring that the optimization has been applied. Also you are not comparing the output of that optimized function to the expected output to make sure that the optimization doesn't change the result


Ha! I actually don't know what that's for. I dropped it in because a spec for petition widget was expecting it.


I couldnt find a nicer link which had only master branch's builds. So, i had to put the "Latest build" link


With the <header> semantic tag, your <div class="wrapper"> tag becomes superfluous. You could simply apply any rules you have on the .wrapper to your header instead in css.


Hard to say, because I kept running into Haiku's netresolv/DHCP issues while doing this work. And I'm pretty sure (from looking at the code) that this is Python's fault in some way, as text (ASCII/UTF) files work, but binary ones (e.g. HPKG) don't. Based on the way the server is operating and the debug output I was getting, the indicators point to Python reading the file off of disk and trying to encode/decode it in some way I don't expect. And if I used HTTP POSTs, I'd have the same problem. TLS gives you data integrity, at least.


Seems that ` source-highlight ` is quite rare and is not in MSYS2 package list. Can we use Pygments instead?


I have changed "private" to "public" at one of my last commits. And I have forgotten about annotations.
Is it normal to change it here at the same time (when changes are minor)? Or I should make separate commit?


Ahhh that was the source of the crash !
I thought C# would catch a stackoverflow.

Btw what does the PlayerControl do with an exception ?


ack


ok


ok, I make it greater than one, I would make the change, start = 0 , would line 28 would be >=1. because, I think I should put the counter before if


:heart:


And of course I got the mention wrong. Meant to say:

CC: @ntnn


Cool!


reack


Please rename this method to `containsMultipleReturns`.


@albertopadilla justo @gajon me mostró eso, en el próximo commit va ese cambio.


Please rename to `throwsCheckedException`


hahah git blame says that was me! well I guess I just copied it out of seeds.rb. I suppose we'll change all these fields when we're actually making the widgets do things.


Cloudwatt line probably redundant?


How should I make sure that the optimization has been applied?


Nope. Should be a throws statement. Why `catch`?


Wow. Another day.


@albertopadilla reviso las líneas y lo simplifico.


Thank you.  Definitely continue to help with r-travis but have little to add on 'v2' right now ... as I don't use it.


I'm going to fix this spec in the next commit, but I wanted to show part of my original motivations in pushing everything in through `content`


You have a few things missing here:

1. You need to check if the exception is a checked exception.
2. Does the body call methods that throw checked exceptions?

Please make sure that you understand the difference between run time and checked exceptions in Java.


got it


Let's have a comment that explains what the two strings are (I couldn't guess which was ID and which was namespace without reading elsewhere)


ack


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1244 of 1244 tests passed)

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


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/71433332


Sorry, I'm confused. Why is this in a `it` block, and why is the params hash inside of an array?


It's better to have it as a separate commit.


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


@chriwup It not safer but it fixes a bug that can happen with doctrine cache.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.364](http://teamcity.monogame.net/viewLog.html?buildId=8837&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:01


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


yes, it does. they don't have any exception statement, I use CatchClause ASTNode because it extend exception


saw approval from bkerley
at https://github.com/basho/riak_ql/commit/4b6fc7ff0ade16f8ca6ddeb1407e3d475fbe7344


merging basho/riak_ql/gg/timeseries = 4b6fc7ff into borshop-integration-7-gg/timeseries


basho/riak_ql/gg/timeseries = 4b6fc7ff merged ok, testing candidate = f939aa1e


The off by 1 error is fixed.


Still, the weird thing is that the "old" sorting algorithm was meant to sort the events in order of time as well, but for some reason it is ignoring this particular event.
I'm not entirely sure what would the best way of resolving this would be: I could just manually swap the lines in the .reference file but doing that to match the behaviour of the parser would kind of defeat the purpose of having the reference outputs in the first place.
However, in this instance the "old" output is objectively wrong, so perhaps an exception would be justified.. Since you know much more about the Eden side of things, what's your opinion on this?


I think you could accomplish that with the "triple dot" range operator, which is non-inclusive on the right-hand side. But if that doesn't work for non-integers, then I suspect you'd have to do `if/else`'s so that you could employ `<=` and `<`.


Working on the commit to move show_bears to the interaction


reack


the params hash is inside an array like the one above it because `widgets_attributes` expects a hash or list of attribute hashes, not just one. it's an `it` block cause it's a standalone example, but the semantics in this file are kinda wonky.


whitin blneder,
```python
sys.prefix == '/home/pkoch/work/blender-2.72b-linux-glibc211-x86_64/2.72/python'
```

so it's easier to get it here


Yes, that would be optimal, but quite complex. I had an idea to rewrite the parser when semantic was added to Emacs, but it would be quite some work and who has the time! :)


@snarfed have you seen this before? WIthout adding this new metaclass, I get the mysterious error

```
the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases
```


Also added fix for Container JSON load.


you set this on the settings page at slack. there's no need to include it in the script.


html5lib chokes on self-closing <a> tags (because that is apparently not allowed in html5)


Nitpick: `< Object >` should be `<Object>`


输出单精度浮点数的存储方式。


GAME DEV TYCOON


Extra spaces in `< Object >`


you should try to be consistent in how you set up your code. For example, while it is okay to put the text that goes between a <h3> tag on a separate line like you have here, you don't do this throughout. Either put them all on separate lines, or all on the same line.


Sure sure.. Thanks Scott.

 Right now I was just thinking of ways to make this more object oriented. Do you think I should spend a little more thinking about that or is that an over-kill?


There's a dependency in QTCLI Logic on the vtkMRMLFiducialListNode which was kept in Libs/MRML/Core, so there is an argument to be made that the Markups version of the fiducials node is a core class (because it can be passed to CLIs) and should be moved. This exposes a larger issue with nodes that modules manage and I'm leaning toward the solution of generalising the CLI infrastructure (and more of Slicer in general), but yes, lots more work.
The attribute solution always seems fragile to me as it depends on hard coded text strings. The coordinate system setting seems like a core element, what about moving just that that into vtkMRMLStorageNode?


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/9087508ad8e1da22ffcb7b732c76baf7fba61d8c


merging graydon/stellar-core/bug-507-race-in-bucket-tests = 9087508a into auto


graydon/stellar-core/bug-507-race-in-bucket-tests = 9087508a merged ok, testing candidate = 06baf9ee


reack


I'll supplement this by mentioning that it is expected to be the structure's first member.


This check should go in the "validate_values()" method, not the "validate_types()" method.


thanks for the link, what if there is try-catch block which is not throwing exception


I guess, we need to implement both for this one


Allows the Title of the MetroWindow to be somewhere else than centered.


keep everywhere or drop everywhere "extract..." naming convention


@SeanNguyen  new db https://www.dropbox.com/s/htb7lez0vqhgrsr/forumdb.sql?dl=0


I guess `ThreadGroup#add` is not thread safe itself? Should I use a mutex to protect it while adding another thread to the group manually?

Thanks!

Reference: https://travis-ci.org/godfat/pork/jobs/71427711


Sag mal hast Du da die Tests laufen lassen? Mit Deinen Changes kriege ich dieses Test-Failure unter Java 8 (und da ist Javascript nicht involviert, der `WithHelper` ist in Java geschrieben):
```
Failed tests:
  HelpersTest>SpecTest.run:95->SpecTest.run:176 expected:<...<h1>First Post</h1>
[  <div class="intro"></div>
  <div class="body"></div>
]</div>
> but was:<...<h1>First Post</h1>
[]</div>
>
```


Atomic asynchronous cache file writes, cool!

I wonder if we can put that `deepHash` function somewhere where it's reusable…


@imsimond + @philipjohn

I've had a think on the naming, as they all are browser notifications, we could rename the title of the settings "Browser notification settings", so it defines the type of notifications for all options below. Then perhaps we can name "Author alerts" to "Alerts from the author" to be more clear what type of browser alert it is?

I'll have a look into options for a curated list of tags, we could combine both ideas into an auto-complete text box, so it gives users guidance on what tags they can use, with the benefit of targeting their own.


You use a large number of different html and css files which makes it very hard for your project to update and stay up to date as I'm sure you've noticed while working on your project. For projects of this size, try to keep your css more condensed so you don't repeat code across multiple css files/have to remember which css files each rule is in.

Also, you do a great job with line indents and spacing in your html, but some of your css is off on the spacing.


pues realmente no existe una diferencia a nivel de programación ya q la verificación siempre ocurre dentro del evento (_currentEvent.isPlayerInEvent).
Solo lo hice asi porque queria q la clase EventEngineManager solo maneje si existe o no el evento y el try y todos los demás checks se realicen dentro de la clase abstracta.
Pero insisto q esto es solo algo estetico


!= 1?


Be careful about using so many classes and ids. It is much better practice for cleaner and simpler code to use more descendant selectors instead.


why do you need to create a new CancellationException?


Frank Underwood quote. House of Cards.


think about your ordering of css selectors. Remember that rules "cascade" down the page so you want to have more general selectors towards the top like this <article> tag and more specific as you go down the page.


Great! But what about `#{json_string() => json_value()}`?


should probably be in a finally block


Hey man, graag zou ik met je in contact willen komen. heb je ook hangouts?

Mvg
BlackDragon


Discovery seems better, what about updating the other usages?


remember your shorthand when you write your css code. When you want to change multiple margins or paddings, you can use 1 property instead of multiple.


Ah yes.


Yeah, I already have, it's just tied to a bunch of other changes. This bit snuck in a bit too early.


saw approval from gordonguthrie
at https://github.com/basho/riak_pb/commit/437483fb407584a84624fdc6f3337ca6c650f3b3


merging basho/riak_pb/bk-rts-212-ts-query = 437483fb into borshop-integration-126-bk-rts-212-ts-query


basho/riak_pb/bk-rts-212-ts-query = 437483fb merged ok, testing candidate = 4b113abc


all tests pass:
success: http://buildbot.bos1/builders/test-riak_ql/builds/25


ack


its just a chache it can be changed by the user so yes it will work on all
OPs just the user can edit the var in gui/command and set it as somhting
but yes lets do that also

On Fri, Jul 17, 2015 at 1:16 PM, Jamie Mansfield <notifications@github.com>
wrote:

> Maybe have a switch for different operating systems, and their usual boost
> locations?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EthanRiley/SKSEBot/commit/3956284ca7804164dd6ee3e4bcfd72cab0de8022#commitcomment-12224345>
> .
>


Honestly, from here on out, I bet this lends itself more to a functional
approach than object-oriented. So I wouldn't try to hammer it into OO
unless it really feels like a good fit.

And while it would indeed be fun to take it further, I think that you've
already extracted all of the benefit of TDD, which was the primary goal.

So I would mark this project as complete (nice work!), and move on to the
HTML/CSS. Feel free to pick it up again on the side for fun if it grabs
your fancy. But it's now served its purpose as part of the "curriculum".

Cheers!

On Fri, Jul 17, 2015 at 11:29 AM, sparsh <notifications@github.com> wrote:

> Sure sure.. Thanks Scott.
>
> Right now I was just thinking of ways to make this more object oriented.
> Do you think I should spend a little more thinking about that or is that an
> over-kill?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simplysparsh/number_converter/commit/4e54f889a014c1136d495af9357f126cbd10c8d0#commitcomment-12227749>
> .
>

--
Scott Woods
West Arete
http://westarete.com/
scott@westarete.com
814-753-4951


### Running Sqlite Admin
       python sqlite.py <database path> or you can just change the database name at line 12.
#### Note
      For now it only execute a SELECT command there's no other than that for now.
### Benefits
      Well maybe it can help you check your sqlite database display number of columns and rows inside the table and execute query in your selected table and iterate data.


extract shouldSkipArgument method and handle session and literal parameters there


If we're happy to be explicit around this issue (i.e. when updating it requires manual intervention) then this cool to stand as is.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1244 of 1244 tests passed)

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


This will be very usefull in 1.13 branch too (as this syntax is working in 1.13.3), I don't see any reason it can't be backported. @rwjblue could it be cherry-picked to 1.13.5 if it will be released?


After thinking about it more, I decided to just go with nullable for a few reasons:

1. Usually I dislike nullable because I feel it's a "poor man's tri-state" and doesn't do a good job of representing the semantic it's being used for. In this case, I think it's OK because it's mostly transparent to the consumer. From the consumer's perspective, they just override a method and return a value that changes behavior or call the base. To them, it looks and feels like normal inheritance and overriding with just a single extra ? in the return type (which they might not even notice).
2. Nullable only works for value types, so it'll be inconsistently used if ever have a method that returns a reference type. However, that's pretty unlikely.

So, in the latest commit ```Result<T>``` is gone.


Nevermind. I opted to just go with ```Nullable<T>```. This type is gone now.


Ah that’s a good point. I am now wondering if the JDT has a way to find out if code can throw a checked exception.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (727 of 727 tests passed)

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


LOL, I knew I shouldn't have done this sleepy. Thank you.


I know I'm being really picky here, but can you add a space after `if`, please? :)


ㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴ


ㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴㄴ


@tenbits did this result in what you expected?


w+ truncates the file before writing, r+ does not.


precedence array


This should be moved up to line 193 because the print statement by itself (without actually pasting) is misleading.


ack


Is this needed?  I would think the object implements this interface or not. So the existence of the interface is the indicator that the other methods & properties are indeed available.


Your styles should all be in your css files. No inline styles within html.


You have a lot of css files which makes it very hard to manage and update your content. For projects of this size, it's best to consolidate those css rules into 1 or maybe 2 files instead of having multiple files for different pages, each having to be updated separately. In general, you don't want to repeat code whenever possible.


@ivanarrizabalaga I imagine this branch is what you used to determine where was the time being spent, isn't it? can I delete this branch now?


mutable default value


I'm not done with this yet. My goal is to split all of the packages into logical parts. For example, html scanner will identify tags, but not compile them with spacebars.


Is a rAF needed for READ?


Hi, I don't suppose to force the user to use a certificated encryption. I'd like to have a callback for user to make decision for whether trust.


You can't just change the project to GPL v3. It's licensed under GPL v2. It's a bit hard to tell since all of the headers got removed (wtf), but it's in the git history. The two aren't compatible so the only way you can legally upgrade to v3 is to go and ask every person that's worked on the project for permission.


You use a lot of classes and ids as selectors in your css. It is better for cleaner/simpler code to use more descendent selectors and fewer classes and ids if possible.


From looking at the other interfaces, the exiting properties are done in PascalCase, where the first letter is upper and each subsequent word is in upper, eg. 'IsOk'.


@irnc - No, see reasoning in the discussion on https://github.com/emberjs/ember.js/pull/11708.


Oh you're right, silly me! Changed.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_pb/builds/153


fast-forwarding develop to borshop-integration-7-gg/timeseries = f939aa1e


Yes!
El 17/7/2015 17:46, "Ignacio Foche" <notifications@github.com> escribió:

> @ivanarrizabalaga <https://github.com/ivanarrizabalaga> I imagine this
> branch is what you used to determine where was the time being spent, isn't
> it? can I delete this branch now?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EyeSeeTea/malariapp/commit/3bbd3329b2607c8a3e64e876649aa3c72f9bafd1#commitcomment-12228024>
> .
>


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/71436913


Watch out for consistency in spacing in your css files. They should all be 2 spaces. Some are, but some are 4 spaces and that should be consistent.


Oh, I'm sorry, I misread your code. you do have code to check that the optimization is applied(your assert that the last node in your fgraph is a True Div). What's missing is just checking the output value.


How can we be explicit about this issue?

Maybe we can add some lines in the README. Other issue is that we have to manually disable authentication to be able to add new plugins. Currently the jenkins-cli will fail due the integration with oauth :(


This is so that we can set a heroku config variable and have the heroku app use the correct mongo db on heroku, but still have ours locally.


Can't you just ifdef out the static definition?


\o/ thanks!

can we get a new release with this in it?


The way the PR is currently, `$DOCKER_OPTS` might fail the start of docker if it has global flags such as `--tls` or `-H ...`, because those are defined as global flags and thus have to be before the `daemon` command.

/cc @tianon

Do you think we should just make them daemon flags instead of global flags and have it global only for the docker client (when the daemon command is not present) ? What about `-D` ?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.365](http://teamcity.monogame.net/viewLog.html?buildId=8841&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 522, ignored: 6 Build time: 00:28:42


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


@rachelholladay: Can we use `numpy.allclose` for this?

@mkoval: Can we just add `numpy.allclose` to `compatibility.py` to support this?

@siddhss5: We don't need EPSILON to be adjustable right? It's only for numerical precision reasons?


@godfat hm, this is an interesting case. Could you please open an issue so we can discuss it. While ThreadGroup is simply using an Array underneath (hence the error you saw) so by itself, it's not thread-safe. However, given the facility of ThreadGroup, I'm inclined to make the operations synchronized on Rubinius.


fast-forwarding master to auto = 06baf9ee


I'm definitely +100 for all daemon-modifying flags being flags to `docker daemon`, and anything in front of that being client-only (and possibly even throwing a warning for at least confusing ones like `docker -H ... daemon`).


This function is for setting up the background blank tiles. Suggest name change to viewport_blank_tiles_paint_setup or something better


I thought it was simpler this way, but if you prefer #ifdefs, no problem.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/18059/testReport/) for this commit**


Thanks, caused by breaking API changes. Forgot to keep the doc up to date


Is there a better way to do it ? Not very clean imo..


Yes, because on a descent compiler what you've done will produce a warning.  You can't mark things as unused and then use them...


the PR does print a warning already for backwards compat.

What about `-D` ? that is the only one that *could* really be global, but then it would break the symmetry and init scripts.


Nice work! I will get these up on the weekend.


sorry, realised this should have been in a separate branch. nooning around.


@patrig Thanks for testing, typo has been fixed. Check latest build.


Generally, very good job on making your html code look good with proper spacing. Remember that the head and body elements are one level in from the html element.


theire is a bug with video addon when u access it via the movie selection


Understood, I'll push a fix.


Also consolidate some indentation and minor naming changes that are strictly internal to the engine.
Refactor a little bit of code for keybord events (minor)


nm, I misread


> This exposes a larger issue with nodes that modules manage and I'm leaning toward the solution of generalising the CLI infrastructure (and more of Slicer in general), but yes, lots more work.
Indeed ... could be a hackfest topic :)

> The attribute solution always seems fragile to me as it depends on hard coded text strings.
Agreed.

On the other hand, it is a rather convenient way of abstracting things. It also allow to easily introspect, serialize, ... Ideally all MRML node properties could be stored in an efficient key-value map. While keeping convenience access  function the way they are. That would also allow us to have an event like `AttributeChanged` that would have the name of the attribute in the callback data ...

> The coordinate system setting seems like a core element, what about moving just that that into vtkMRMLStorageNode?

That makes sense.

@lassoan @pieper What do you think ?

In case the metadata do not allow to determine the coordinate system of data loaded ... what should be the default ?


You use a lot of classes and ids for selectors. For cleaner and simpler code, it is better to use more descendent selectors and avoid giving everything a class or id to select it in your css.


I'm getting DOM Exception 8 when trying to render a Blaze component after first rendering a React component now.

I updated the repo:
https://github.com/maartenbusstra/ironrouter-react-issue


Was trashed afterwards anyway if you look at other comments it has it.


Thanks


Исправил


space comment bruh


ack


lack of semicolon


@michaelcahill, would you please review this change?

Hopefully this fixes the recent Windows build failure:
```
src\block\block_ext.c(382) : warning C4047: 'initializing' : 'int' differs in levels of indirection from 'char *'
src\block\block_ext.c(382) : error C2198: '__wt_err' : too few arguments for call
src\block\block_ext.c(1045) : warning C4244: 'initializing' : conversion from 'wt_off_t' to 'int', possible loss of data
src\block\block_ext.c(1045) : error C2198: '__wt_err' : too few arguments for call
src\block\block_ext.c(1058) : warning C4244: 'initializing' : conversion from 'wt_off_t' to 'int', possible loss of data
src\block\block_ext.c(1058) : error C2198: '__wt_err' : too few arguments for call
src\block\block_ext.c(1222) : warning C4244: 'initializing' : conversion from 'wt_off_t' to 'int', possible loss of data
src\block\block_ext.c(1222) : error C2198: '__wt_err' : too few arguments for call
```


Check max_health again here.


Please fix indenting here. Try this:

```
events = [
	Death(FRIENDLY + MINION).on(Find(FRIENDLY_MINIONS) & [
		Buff(RandomSelector(FRIENDLY_MINIONS), "FP1_020e"), Reveal(SELF)
	], zone=Zone.SECRET)
]
```


r+


For the record, I didn't change the code style of package.json. npm did when I did `npm install --save glob`.


only take effect on devices with API 21 or above -> only takes effect on devices with API 21 or above


https://dropmefiles.com/rojU7
## WARNING: There are 254 active turfs at roundstart, this is a mapping error caused by a difference of the air between the adjacent turfs.
Initializations complete


Thank you, I still have a problem with mysql. Started about 10 gits of back. Really does not connect to the database, no plug-in ...


Yes, this behavior a CodeIgniter 3 developer would expect by intuition. I think, this convention change FALSE -> NULL is necessary.

http://www.codeigniter.com/user_guide/installation/upgrade_300.html#step-10-many-functions-now-return-null-instead-of-false-on-missing-items


lgtm


Good catch :+1: , I totally missed this one when searching for the problem in #70.
@khaf This is a nice tool to catch missing error checks: [github.com/kisielk/errcheck](https://github.com/kisielk/errcheck)


(n_samples) -> (n_samples,)

it's a tuple of size 1


@jtormey yes


I'd like you to add a test (separate, below) that checks nothing gets buffed when multiple minions are killed at the same time. You can use two wisps and an arcane explosion.


Don't consider demo-style by mistake posted that  now doing the page 23


Error like:
```
Warning: slotLogLineEditChanged() error: value from "logedit_DE_factor" is
not greater zero</i>
```
can happen when you edit some parameter manually (not by slider) and for a while it's not a valid number. When that errors are generated? When you start application, when you edit some parameters, or when you close application?
I have pulled latest revision and I can't see that errors. So it's not because of your latest edits. You did everything very well.

About this problem:
<i>The only current problem is that
the tooltip for color button adds a new parameter description each time you
use it.</i>
I need to check ColorPaletteWidget class. There must be some bug.


cies: you were right: it is broken as of now in the readme


ok!


Yeeap, I know :(. I should change code style.


The [mysqlx tree](https://github.com/alliedmodders/amxmodx/tree/master/modules/mysqlx) hasn't changed in over four months, besides some tiny MSVS project file changes. Everything should still work. It would be very helpful if you could nail down the build where the issue started by bisecting.


merge master and gh-pages


mrhooray/gulp-mocha-phantomjs#34 is now merged - upgrade to 0.7.0 and you're good. Please no more comments here - any further comments should be in issues in `gulp-mocha-phantomjs` or another repro as appropriate.


This is the first time i use of github.


Yes EPSILON is only for floating point.


Hi,

I've tried, serial numbers for "Adobe Master Collection CS5", as you have said it, but unfortunately it is not working.

My system:
Windows 7 Home Premium, SP1, 32-bit
Processor: AMD Athlon Dual-Core Processor 7850
2.00 GB of RAM

Can you send me a keygen or serial numbers?

My email: Aaronkildom@gmail.com

I wish you a nice day and greetings from Austria
Aaron


Consider moving this config to the test environment config in `config/environments/` so its consistent with where the same config is defined for the dev & prod environments.


Glad this got fixed.


fast-forwarding end-to-end/timeseries to borshop-integration-126-bk-rts-212-ts-query = 4b113abc


This needs to be `igraph_adjlist_has_edge`.


`igraph_adjlist_replace_edge` please


I only remember asking why it was there in the first place ;-)
https://github.com/flowplayer/flowplayer/issues/110
which is referenced in an unrelated commit:
https://github.com/flowplayer/flowplayer/commit/7402eefbab5d5fc259d931d4976080dcfd39a54c
which in turn leads to:
https://github.com/flowplayer/flowplayer/commit/201ea3f3ecb532d1b320d48126d6d32421b95857#diff-2180f82b9c15e3ca82dd84058817dc71R155
ermh ...
Anyway, it's about pre 6.0 Safari ...


This is not part of the public API I guess, so it should not be in this file.


r+


Yes, definitely, think this should be done as a separate PR


that's a good idea. I'm not too up on vim plugin standard procedures. Thanks!


For this class = Fix no played sound


Right. I thought it would be helpful if `mkdtemp()` failed, so that the line `if self.tmpdir is not None` in `tearDown` would not crash, but `tearDown` is not called if an exception is raised during `setUp`.


Ajam, es sólo estético, no existe diferencia (y si la hay, es minúscula). Fue, dejalo así.

:+1:


Fix sign null pointer exeption !


Fix firework crash


The method body is only emitted if moduleBeingBuilt != null, that is, if the module is being emitted. This GetEntryPoint method is called when compiling method bodies only. By contrast, compilation.GetEntryPoint() simply looks up the entry point method, by name, from the members of the Script class, which does not invoke this method.


gg git, gg


I believe using `NetClock::duration` is a benign bug here.  `offset` has units of `decltype(mCloseTime)`.  Both of these happen to be `seconds`.  However we want to maintain the freedom to change the units of either of these two things independently without introducing errors.  Until we chrono-ize `mCloseTime`, I recommend here: `.adjustCloseTime(std::chrono::seconds(offset))`.  And when `mCloseTime` goes chrono, that should be simplified to `.adjustCloseTime(offset)`.  Then the units can be independently changed, and if there is any conversion error, it will happen at compile time, not run time.


I have tried, but it doesn't appear to see any branches to build:

![screenshot 2015-07-17 17 17 38](https://cloud.githubusercontent.com/assets/86222/8751861/e3fbf1fa-2ca7-11e5-81e1-917a95212688.png)

I assumed this was because there is something required to trigger it to see them.


saw approval from MonsieurNicolas
at https://github.com/FredericHeem/stellar-core/commit/c15d8bd4cac34f8bea40a42ff8fa50bd57601150


merging FredericHeem/stellar-core/master = c15d8bd4 into auto


FredericHeem/stellar-core/master = c15d8bd4 merged ok, testing candidate = c13748c6


This use of the ternary operator is confusing, I can't really tell what's the difference.
How about we use a temp and an if here?


Nothing. I made a mistake, it worked for me just accidentally. Fixed with the next pull request.(https://github.com/Microsoft/ApplicationInsights-JS/pull/34)
Thanks a lot!


Otherwise it looks great!


Interesting idea to make all node member variables attributes.  We should think about that for the future.

Slicer is inherently RAS right now, so the convention is to perform any coordinate system conversion during input and output.  Similarly I think any data with no metadata should be considered RAS unless it is part of a standard (like DICOM) where the coordinate system is defined.


The only issue is when I do that, I get the test failing again.


This does not print port numbers and also assumes that the addresses are always AF_INET.

Adding port numbers is a quick simple fix.  Making this more generic so it supports any address type may be a bit more work and probably not worth doing prior to 1.1.0.  Maybe what we need (post 1.1.0) is a simple function in the core to return a string given a sockaddr, which may be any of the supported FI_SOCKADDR_* types, that any provider may use in their debug output.  I would say "just use getsockname()" but that won't work for AF_IB.

@shefty Thoughts?


That wasn't a custom ordered dict, by the way.  It was an OrderedDict backport to Python 2.6 written by Raymond Hettinger (major python core contributor).  It is probably a more reliable drop-in replacement than SortedDict.


How do they determine this in NetBeans?


Что это?


The second field changed from IntPtr to a reference type. We need to bump the corlib version.


they have some special class, check through boolean


Hello! I´m trying to get confidence but it seems to be always 1. Does it happen to you too?


Почему игровой счёт хранится в переменной внутри функции-обработчика кнопки?


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


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


> "Added by James in the *next* commit."

You can see the future!


Filed. Thanks for looking into it! https://github.com/rubinius/rubinius/issues/3467


@supengyu I think we do need your help in finding the problem. Since it seems to be dependent on your system, we have others compiling cxxnet successfully and running on mnist without problems including windows platform.

cxxnet is a multithreaded platform, so the model initialization and data initialization can happen at the same time. Since the problem is segfault, we should be able to locate the place where the segfault happens by either look at the trace at fault, or simply doing some print and check where things halts.
Please try use the original config file provided by the repo and see if anything goes wrong on that one.

Since your problem does not relate to this commit, please open a new issue for future questions. The developers are willing to help but we do need your help as well, since non of us works actively on windows platform.

Thanks


Hello, how to work with elastic transcoder?


Yes both LM and DES are bitsliced as in Solar's implentation.


I suggest "Since version 1.6.1.0 of PrestaShop, we are using the [PSR-2 Coding Style Guide](http://www.php-fig.org/psr/psr-2/) as our coding standards. [Learn more about this here](http://build.prestashop.com/news/prestashop-moves-to-psr-2/)."


This is awesome and so much cleaner than the hideously verbose rpy2 solution. Are the axis labels and titles preserved in the new output? Can't see where they are now.


@jpatchell Is it OK for you to be mentioned as an author?


@GeertJohan Yes I have to add errcheck to our build process.


Python might have been OK, I'm not sure you gave it enough of a chance (though it is certainly not my first language choice for this by any means.) Go would be fine, though if you don't already know it, it probably doesn't make sense either.

I think the people recommending C++ were crazy though.


I can't even get the authorized form of my own name right! :) Thanks for the correction.


Stub function so I can call RequestBin for debugging rather than the orchestration service.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/71443842


What could cause this?

This is not in the original C code. We should add tests for this new failure scenario.


Thanks you . Gonna test this change :)


I see. Sounds good.


Alternative:
```js
demo.music[demo.music.paused ? "play" : "pause"]();
```

Other alternative (readability ftw):
```js
if (demo.music.paused) {
    demo.music.play();
} else {
    demo.music.pause();
}
```


I'm dying to know: 10 what?!


Great job on this commit! :+1:


can be static?


I had to add `array-seq` to this stack of madness because I was getting an "object filelist is not iseqable" error.


This terrifies me. Surely there's a better way?


@latobarita: retry


Answer is Yes, per the code example in #1.