Your tests seem to alternate between defining an expected output in a variable and then testing for it, or testing with it passed directly to the test function. You should pick one and be consistent.


к чему тут else?


@ktoso I assume we had to solve this for the test that validated that an onError was issued on overflow?


Капслок по-умолчанию включать не надо.


Is this pull based on previous one? Can you fold this 2 pull into one and fix the issue I pointed out in #2, then I will merge it.


You should validate the array argument.


This may not be the best place to put this as this is called multiple times and you will get warnings many times.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2074 of 2190 tests passed)

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


алсо лучше сделать через if и ifels


I'm not totally sure why we're singling out ! here. Wouldn't we want this space to come before any unary prefix operator, like +, -, ~, etc.?


the help button should be a proportional distance from the side, not (width-10,100). more like (width-50, 50). that'll make it look better and placed intentionally.


Ах, я забыл про сосач.


Можно было все эти поля добавить одной миграцией :)


the error you get is due to the kodiplatform library not being available. you find this at notspiff/kodi-platform. note that you have to have installed kodi at least one time to be able to build it. build and install that and all should be good.

this is with mainline, it's all done automagically in my system.


Resolve #1367


Add some tests for invalid request, e.g. missing required fields?


Eigentlich war `pacat` gemeint...


구글 플레이 연동하는거만 하였습니다.


with this approach, can we go back to the approach of not returning the addSerializersToPipe function and instead take a function from `(TupleSetter[(K, V)], Fields) => Pipe` and call on that or not? I think that is a bit harder to get wrong than returning the function.


Did you mean to include this?


This resolves issue #4


@JuliaBackports @tkelman Done!


arr.slice() is more appropriate here.


if we go this route, we might as well just put a counter down to make the key a bit smaller.


if user likes no games, please return an empty list


What if I pass in -Infinity as my array elements?


Thanks @tmontgomery - I've moved to using that approach. pthreads are needed by gtest afaict, well I get compile errors otherwise.


How about we not copy this stuff and instead just stick symlinks in stage1/rootfs/net-plugins/bin/$plugin -> ../../../../bin/$plugin?


Ah ok right, my bad. Thanks for fixing it. :)


fuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuu


please avoid dropping `else if` and `else` statements onto their own lines when coding in Javascript. it should look more like:

    ...
    } else {
    ...

or

    ...
    } else if (...) {


sgdt storing all 32 bits has been documented in PCem, as apparently, Win32s doesn't work without this. LGTM.


Why have two different vars? Also if you don't preface assignments with `var` they become globals, which is bad practice.


Из-за trim'а в геттере текста не работает детектор `whitespace` для постов вида

```

САЖА
```

Можно ли безболезненно этот трим убрать?

```js
	get text() {
		var val = this.post.msg.innerHTML
			.replace(/<\/?(?:br|p|li)[^>]*?>/gi,'\n')
			.replace(/<[^>]+?>/g,'')
			.replace(/>/g, '>')
			.replace(/</g, '<')
			.replace(/ /g, '\u00A0')
			.trim();
		Object.defineProperty(this, 'text', { value: val });
		return val;
	},
```

```js
		// (1 << 5): whitespace
		if(val & 64) {
			if(/(?:\n\s*){5}/i.test(txt)) {
				this._wipeMsg = 'whitespace';
				return true;
			}
		}
```


It's better to avoid such #if and implement mono specific UnsafeNclNativeMethods.GetCurrentThreadId


`os` isn't imported, so this file doesn't compile.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

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


Good that you got rid of this test routine.


need to return an empty list if user has no primary connections


+1 Lgtm


You should comment your code, explaining your logic. There are no comments anywhere.


The idea here is for the vector and markers layers to be used only for current data (current positions and "follow" device). Then an archiveLayer for all related to archive plotting.
Later on, when multiple devices track print gets added, we'll need to add/remove layers dynamically for each device being plotted.


Hello!


Yes it does! Thank you :+1: :)


А некоторые вайпы вообще из оджних пробелов состоят. Соответственно, они обрезаются до пустой строки.


about damn time you edit my post.


Explain what you're doing here.


Ah... Had that changed back locally... Forgot to save before commit...


Нахрена, спрашивается?)
С учётом вот [этого](https://github.com/jFoxSingularity/ReflectHero/blob/140833547adbf5b92358bafe8ff612751b3b19af/Gemfile#L55-L56) и того что при бандле сразу всплывает сообщение об ошибке.


initial commit
indeed


Not needed anymore?


I like kvm name definition...


Убирай.


I get now an error when running mvn clean install in my checkout.

```
Running net.uaznia.lukanus.hudson.plugins.gitparameter.GitParameterDefinitionTest
Running InjectedTest
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 11.005 sec - in InjectedTest
Tests run: 25, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 16.777 sec <<< FAILURE! - in net.uaznia.lukanus.hudson.plugins.gitparameter.GitParameterDefinitionTest
testDoFillValueItems_listTags(net.uaznia.lukanus.hudson.plugins.gitparameter.GitParameterDefinitionTest)  Time elapsed: 0.116 sec  <<< ERROR!
java.lang.IllegalStateException: second instance
        at jenkins.model.Jenkins.<init>(Jenkins.java:742)
        at hudson.model.Hudson.<init>(Hudson.java:82)
        at org.jvnet.hudson.test.HudsonTestCase.newHudson(HudsonTestCase.java:474)
        at org.jvnet.hudson.test.HudsonTestCase.setUp(HudsonTestCase.java:315)
        at net.uaznia.lukanus.hudson.plugins.gitparameter.GitParameterDefinitionTest.testDoFillValueItems_listTags(GitParameterDefinitionTest.java:357)

Results :

Tests in error:
  GitParameterDefinitionTest>HudsonTestCase.runBare:293->HudsonTestCase.runTest:433->testDoFillValueItems_listTags:357->HudsonTestCase.setUp:315->HudsonTestCase.newHudson:474 » IllegalState
```

Fixed by adding `super.tearDown();` in line 420. Will push a commit for this change


Looks good.


Yes Travis alarms just went off everywhere heh. Now fixed


`QMutex` isn't a POD. However, `QBasicMutex` is.


Пока полет нормальный. Может, какие косяки и вылезут. А может, и нет.


+1 to this. I think it's terrible that you can't just put the service in place and not start it. I have a chef/capistrano deployment in a jenkins job and the chef converge fails because the runit service fails to start. Capistrano needs to run on it before it gets started.


I was hesitant to actually create the sprite image file for this reason. Since only a few of the files are in use, there is little point in putting them all together.

A way to possibly save space is to use the SVG files rather than PNG. The zips from http://game-icons.net/ are 11MB for PNG, and 1MB for SVG. [caniuse.com](http://caniuse.com/#feat=svg) approximates 92% of all internet users are able to use SVG.


DONE, better test in 573074bf27c3a15c636ade71a78e518b5e200912


What is the logic here?


Second version LGTM.


Can you do this without two for loops? That's O(mn) complexity, where m and n are the lengths of the two arrays.


I am not sure that we should remove build from `.gitignore` file.


What are you doing here?


Nice! This will slightly simplify #21351 - hopefully your PR merges soon.


r+


I made the `data_spells.js` have a possible template, so that players could add their own spells, rather than list all of them. To do that would take up  a lot more space, and players probably aren't interested in spells apart from the ones their character uses.


This has made it clear, thx.


Hard to remember.  My guess is that it was buggy and I removed it rather
than fix it?  It looks like it wasn't properly connected in
BackingExtension maybe.

On Thursday, January 22, 2015, Joe Wilm <notifications@github.com> wrote:

> Was removing centerAnchor intentional?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gameclosure/timestep/commit/455e15d05dadc46b90eb58f2e1f50c44544a1a11#commitcomment-9403690>
> .
>


has to check if g_BaseSystem already exists, and either delete it or reuse it


How is this different than your `intersection()` code?


I completely agree in being puzzled by the warnings/complaints of MSC compiler for completely safe and working code (some of the "*_s" safe functions are over-pedantic).

I also agree that having these two precompiler branches to maintain would be an ugly pain in the neck. I Don't like it, too.

So... let's go for project-wise settings?

As I sidenote, Lua uses pre-compilation directives to configure the target binary but does not need compatibility warnings to be disabled. It uses a "luaconf.h" file to detect and handle the platform dependent differences. Moreover, in some parts Lua uses Windows specific functions without being platform agnostic (see file "loadlib.c", although porting is certainly possible).

I'm still


This has a bit of an impact on performance, but is a nice / clean way to make this simple and repeatable. :+1:


Use linux :)


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=bb607565b1ca4a1b9a78334ad82b3e1a
 plone-5.0-python-2.7 [FAILURE]


Is a simple **||*.filenuke.com/*/*$script** not possible ? Same for sharesix.com ?


I see. Thx.


Did you mean `onDeQueue`?


Got rid in e3a372fa7d1e1bd4f848ae7d76e7ffe6928b6224


I think this method should be static and better move it to GitCommandHelper. Can you write simple unit test (https://github.com/gitextensions/gitextensions/blob/master/GitExtensionsTest/GitCommands/Git/GitCommandHelpersTest.cs) for it?


something's weird with the end of the sentence


use backticks instead of weird double quotes


testing testing


no need for this comma now.


you can remove the example tests :wink:


you can remove the example test :wink:


No, by _structure_ I mean RDD. Yes, as expected, ``map``ping is lazy,


Good data structure, but variable names could be more clear. Try to use longer, more descriptive names - it'll make your life easier and make the code more readable for others. You can also check to see if ' is connected to ' and ' likes to play ' are in the sentence without assigning additional variable names like conns_str and games_str. Use lowercase? comments are ok


CLI tests pass on Windows.
Publishing.


```java
if ((deployment.getExport() != null) && Boolean.TRUE.equals(deployment.getExport().isEnabled()))
```


How about this one:
+### windowing-with-ddl ###
+  This example is a modified form of the "windowing" example. It
+  defines a table constraint to age out old tuples automatically
+  without Java stored procedures needed.


Got it all fixed up EXCEPT one thing, didn't know if you could point me in the right direction. I got the white text everywhere now BUT google. In all google apps, hangouts, play store etc the dark background is still with dark text and I can't for the life of me fidn where this text is defined. I had my themer looking and he fixed the rest but neither of us can find this and I know people will complain as they should, I am too LOL. Thanks in advance man, You have been of MASSIVE help just with the small issue I had


looks good.


Instead of the try/except/pass dance here, could you use `ResponsePI.objects.get_or_create()`?


yeah :)


Cool, thanks a lot!


Not sure if we plan to use them, but this won't work for json numbers or bool types, and will throw an error. Is False the expected result for [] or {}?


Should be `item.id` :)


why don't you like composer autoload functionality?


wrong. that'd kill the entire purpose of the system (which is so that long cooldowns (for example comet) aren't reset on logout, which'd be a exploit)


nope. will re-add later. removal was requested but turned out it has nothing to do with the issues we had, and I was too fast pushing it ;)


Oh yes, I see your point. On it.


asdf


These are micro-optimizations, which I wouldn't bother with in a lot of stuff unless it is something that happens a lot or makes it more readable.

regex is almost always going to be the slowest way to do something

If the $v is always going to start out a string then using if ( ctype_digit( $v )) is going to be the fastest and cleanest way.

If it could be a string or an int and you want to test for if it is a valid int then the absolute fastest method I could come up with is:
```
function test_cast3( $v ) {
    if ( is_numeric( $v ) ) {
         $new = (int) $v;
         if ( $new == $v ) {
              return true;
         }
    }
    return false;
}
```
In my tests (500k itterations on 8 different strings/ints, total of 4mil per) this was 29% faster than the regex version.

If this is something that happens a lot in the code it'd be useful to have a helper function to do this so it doesn't have to get reproduced all of the place.  Maybe $v = pods_v( $v, 'int_or_string' );


`JSON.parse` accepts Buffers. Who knew!


I think this last commit is a good PR to start.

I have avoided to take out the paths and properties used in the cassandra.yaml and cassandra-env.sh, I think is not a good solution to put them in properties file.

I think formatting now is good.

Please let me know if you have other plans for this project.

Thanks a lot.
Bye,
Andrea


I tried building your branch but it looks like I'm hitting dependency problems, even after removing most add-ons: [`Could NOT find GIF (missing: GIF_LIBRARY GIF_INCLUDE_DIR)`](http://pastebin.com/NXnp75KY)

does what you're saying mean I have to do `sudo make install` on mainline before the add-on will compile?


> only two methods are supported : `CRAM-MD5`, `PLAIN` and `LOGIN`. Be aware, the `PLAIN`

I think that 'two' is no longer true, eh?


no, i rebased you tree on current mainline. libgif is a new dependency - sudo apt-get install libgif-dev


Add worspace.xml, .idea to gitignore


Campaign x Demo Impact Analysis Script


(comments): should have a package comment, unless it's in another file for this package


(comments): exported type Test should have comment or be unexported


Yep, no accent. I'm sorry I was in a frenzy to add some.
Who's doing the fix and PR?
Cheers :)


It might be slightly nicer code for this function to be idempotent; in other words above do:

    if (sack->cmdline_repo_created)
        return;


Then here just call it unconditionally.  But it's a minor style thing, up to you.


sorry, misread that.

you need to supply the kodi-platform library. problem is, this library has a dependency on kodi's cmake config.
so you have to a make install on kodi first. then build the kodi-platform library and make install that. then you are good.

the system in mainline is very WIP - it won't be like this when it's done. but for the time being, this is how it is.


r+


maravilloso


Yup, it should be 1. But otherwise, :+1:


This now causes a lisp error if the file can't be parsed, e.g. if the first line is

```
;; evil-visual-mark-mode.el --- Display evil marks on buffer
```

we get

```
Error from syntax checker emacs-lisp-package: Wrong type argument: arrayp, nil
```


Please remove "file" from commit message.


@bors: retry


spacing between symbols should be improved for readability. e.g.

    for(var i = 0; j < 3l j++){


I'm with @LukeMarlin 's option.
I'd suggest "par tout autre groupe de moines" or "par n'importe lequel des autres groupes de moines" if you want to go for a non-obvious way.
Cheers!


saw approval from larsbergstrom
at https://github.com/glennw/servo/commit/0f525d908da229218c88a30cdaa837d492f802be


merging glennw/servo/split-headless = 0f525d90 into auto


glennw/servo/split-headless = 0f525d90 merged ok, testing candidate = d39cef0b


Please fix typos in this line.


@bsaiken, @danish2014 : Can you please explain what you don't understand so I can help you?


Why this?


Test for Slack!


Ahah merci !


Whoops, did not realize that PR was a breaking API change. Does PkgEvaluator test IJulia? I thought it would usually catch this kind of thing faster.


odd indentation


If I recall the offending bit was init-debug on Nexus 5.
It kept rebooting over and over because it failed to see the usb interface (prolly line 102).
An additional sleep before might help.
Then I rolled my own /sbin/preinit which does nothing more than: exec /sbin/init --unit=default.target
It used to save some diagnostic logs and mount the android file systems before I got d-h-i to behave.


Die Daten, die Netmon da lieferte, waren eh outdated.


(comments): should have a package comment, unless it's in another file for this package


(comments): exported type Test should have comment or be unexported


Cool! I really need to start working on this new menu!


tanx for replay

i wana under pop work on  firefox -chrome-explorer

i put your code on myself.js  on dir www.mysite.com/myself.js

and i put this on my template:
<sc ript type="text/javascript" src="www.mysite.com/myself.js"></sc ript>
<sc ript type="text/javascript">
SmartPopunder.make('http://mysite.com', {width: 100, height: 100, newTab: false,blur : true});
</sc ript>
is it correct
if not how can i use for show new window back of browser
sorry for poor english


Why it's taking so long for this or its complement to get into omnirom is beyond me.


Idem


Yeah!


woo!


Since the provision for allowing overflow is qualified with `MAY` I don’t see what we could verify. Could you elaborate? (i.e. “it is OK to request more” is not true in general)


This looks wrong. You inherit MapGenerator_D1 from MapGenerator class,  so "new MapGenerator_D1()" creates map generator that only can generate GEN_ALGORITHM_D1 maps. And when using resulting object to generate map you still pass algorithm type as agrument. but algorithm type is already defined by previous "if" checks when creating object. This is not how Inheritance should be used. Generate method should not accept algorithm agrument


Mmm... So the `.get_or_create()` thing will give me an object, but then I still have to test to see if it's a new one or not before I call `.save()` so as to reduce saves. I'm not sure it helps much here. Having said that, I should be doing bulk creates.


I can't help but wonder if it's actually the most efficient way to do it.


Ack


Actually, `.get_or_create()` returns a tuple of a boolean and the object: `(was_it_created, the_object)`.

Bulk creates would be better anyways though.


Nice!  I'd split separate the tests and implementation into separate files.  By convention, if you had `roman.rb` then the spec file would be named `roman_spec.rb` and the first few lines would look something like...

```ruby
require "rspec"
require "./roman"

# specs go here
```

You'd then run it with the command

```shell-session
$ rspec roman_spec.rb
```

Other remarks coming inline (shortly)


It worked!


Possibly my favorite commit ever where you just toss all the work and begin again. :smile:


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2178 of 2190 tests passed)

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


Got rid of this in 1bf9302b45c9b3871dca1136c82692aef27f7b4d, as there was a duplication of code (``getValues`` == ``\`` on an XML)


comentando


Si @Riichard007 , hasta hiciste demás jejeje, muy bien :see_no_evil:


@rkuhn the MAY clause is just that it is OK to stop tracking demand past that point. We can still test for not getting an onError?


I like how exhaustive the test cases are!  Make sure to include both valid and invalid inputs in your test cases.  How should your method behave if you give it a negative number?  A decimal?  A string?  It's your call as to how you want it to behave, but you always want to test both valid and plausible-but-invalid inputs.

There are three ways to handle invalid input:

1. Coerce — try to convert the invalid input into something valid
2. Assert — detect the invalid input and raise an error
3. Ignore — do nothing and trust that something farther down the line will handle the invalid input


Add spaces between each of your test cases to improve readability.


(comments): should have a package comment, unless it's in another file for this package


(comments): exported type Test should have comment or be unexported


Yes, this is kind of awful, but it's the only way I could find to allow the calibration screen to correctly close on the particular Windows 7 machine I am working on. [Using the standalone PsychoPy distribution of python, 1.81.02.]


Oops, I realize the specifics of my comment would've made for sense for the complementary `number_to_roman` function.  In this case, invalid input wouldn't be a negative number, but maybe a malformed Roman numeral, e.g., `"XVTNBBBU"`.


Well it isn't a problem on my server, however if the loading order is different on another server and the events are registered before the mercenaries are loaded into 'mercs' then it basically registers nothing. It is not Eluna related, but part of my script indeed. ".ext" will fix it, but you'd have to give the mercenary script files a .ext extension which is not what I want. Seeing as I already released the script, I sort of 'fixed' it really fast so things should run fine. However I am planning on changing this because giving it a one second delay is just a bit messy .. xd


:cake:


note that you can install kodi to any prefix, that isn't relevant (such as a directory in your home). you just have to point the kodi-platform system to the dir you installed to using cmake -DCMAKE_PREFIX_PATH=<install path>. the paths arent relevant, it just needs the version info and some small helper cmake scripts which are supplied by kodi.


@jzoldak I [looked over the docs and it mentioned needing to decorate pages](http://bok-choy.readthedocs.org/en/latest/guidelines.html#wait-for-javascript-to-load) again, and noticed that it talks about decorating the class with `js_defined`. Is this not required here?


See #183.


False is the default, right?  Is there a reason we're being explicit here?


Marked to be done as https://issues.ypg.com/browse/EDM-136 OK?


Maybe use `make -C ./docs html` ?

In case something fails with the make command, the last cd command will not run and things might get confusing.


Nice work, but I doubt it's needed, I never heard of somebody using PPM on the internal module


Victor, yes, certainly not a "BIG" issue... But I want to note things like this, because to me these are "quality issues", because we altered (and some people may say we broke) functionality which worked (differently) before.

And before, the color coding I once introduced said: "green=latest version, light-blue=newer-version available". And that was regardless if you had your map the active or in the backup folder.

So I consider it my duty to note stuff like this and mention it, (before our users do :-) ), because that makes the difference between a "quality-controlled" and a "we don't care so much" type of product,.

Hope you don't mind ...  :-)

Best, Hardy


Published to http://vui-docs-internal.s3-website-us-east-1.amazonaws.com/24_8c757e085e677933b7df6c106fcffde36b03ccfe


It is only the default if the property doesn't already exist. In the code below I override and existing prop that is enumerable.


And what are they?


I'd rather keep parity with msmbuilder.


do these need to stay mutable? Can we make them immutable?

```java
import com.google.common.collect.ImmutableSortedSet;
// ...
ImmutableSortedSet.Builder<Table> exportTables = ImmutableSortedSet.naturalOrder();
// ...
return exportTables.build();
```


good :), but now its this error
<E {PPU[1] Thread (main_thread)[0x0001f898]} Exception: Access violation reading location 0x0 (is_alive=1, m_last_syscall=0x0 ())


Does this add or remove BOM? If this adds it, please remove this change


same here


dafuq is dis


CDataStore::Read1Bit(&v7) != 0;

means

data.WriteBit(isPublic);

== 0 would mean !bool

if it's really !isPublic  (pls test ingame) then it's a rare exception


@therebelrobot just a note that this builds to main.css instead of styles.css


@joetam This will prevent the disk cache from being cleaned up automatically, yes. An extension writer should probably call cleanup themselves. Alternatively we could add something in the sharedImageCache initialization to clean the disk cache?


81605912-Multispeed-Heating-Defrost-Bug (lgu1234) - i386-Windows-7-VisualStudio-12: Tests Failed

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


update_sqlite_coverage (mbadams5) - x86_64-MacOS-10.9-clang: OK (1555 of 1597 tests passed)

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


Hello Aaron,
i am trying to install Redmine RubyCAS plugin. But when i click the configure it shows the following error.

404
The page you were trying to access doesn't exist or has been removed.

I m using the following settings:

Environment:
  Redmine version                2.3.4.stable
  Ruby version                   1.9.3-p448 (2013-06-27) [i386-mingw32]
  Rails version                  3.2.13
  Environment                    production
  Database adapter               SQLServer
Redmine plugins:
  redmine_rubycas                0.0.6

Please help.


(comments): should have a package comment, unless it's in another file for this package


(comments): exported type Test should have comment or be unexported


Regarding the names, please follow the discussion here: https://github.com/aspnet/Home/issues/261


@Purdom here's a scaffold for using the `login_with_oauth` method. I can show you the keys (https://console.developers.google.com) for `CLIENT_ID` and `CLIENT_SECRET` which are a PITA.


Can't reproduce. Can you make a proper bug report with clear repro instructions?


Hot damn--I had no idea it returned the boolean. I shall change my ways!

But in this instance, I'll switch to bulk creates.


Do you mean `#if (LIBTORRENT_VERSION_NUM < 10000)`?


wouldn't

```java
else if (export.getConfiguration().size() > 1) {
```

be clearer?


Yeah I'm open to abstracting it. As long as it casts int on strings that contain only digits, no other characters (floats, hex, etc).

`$post_terms = pods_string_or_int( $post_terms );`

Could handle arrays of values mixed with string / int values.


Ack


bede pamietal


Initially I had done the way you described. I moved it out to improve the readability of the Logger class interface. Then I forgot about it.
Now I am going to move it back in.


Thanks! Looks good. Have applied this patch to https://git.gnome.org/browse/gnome-commander/


Another item for the checklist; good catch, thanks.


Mentioned footprint in file name is a duplicate and will be removed once there is one real footprint added to the repo.


only corrected indentation here


Sorry previous message should have read (freakn markdown):

Was there an issue with the <jar href="startup.jar">  not executing the main correctly?

But, it's coming back to me. Because of the recent security changes, you must use: main="true" and download="eager".

<jar href="startup.jar" main="true" download="eager"/%gt;

This is a better solution than explicitly using main-class, since it could change at anytime.

Can you try this?


@ai the check works fine:

![auto6](https://cloud.githubusercontent.com/assets/1139472/5864987/2835ac6e-a254-11e4-81f5-50655c4025ee.jpg)

I also forked the new master, and tested on local. I got few things that I think make sense reporting to you:

(1) Run Bundle update

I got an error warning about 'too many gems' , the following is a screenshot:

![auto1](https://cloud.githubusercontent.com/assets/1139472/5865024/884c8bae-a254-11e4-9b5e-4aada1db3411.jpg)

Not really a drama because bundler resolves the dependencies fine.

(2) Run Test with Node

autoprefixer_spec pass with no failures

processor_spec pass with no failure

compass_spec doesn't pass and got the following failure (I don't use compass maybe it's this):
![auto2](https://cloud.githubusercontent.com/assets/1139472/5865103/1882217a-a255-11e4-9b9d-cce34a3f6e7d.jpg)

rails_spec doesn't pass and got the following failure:
![auto3](https://cloud.githubusercontent.com/assets/1139472/5865127/3d368862-a255-11e4-90da-8745cd6ef88b.jpg)

(3) Run Test with JScript

autoprefixer_spec doesn't pass (obviously)

processor_spec pass with no failure

compass_spec doesn't pass and got the following failure (same as with node)
![auto2](https://cloud.githubusercontent.com/assets/1139472/5865103/1882217a-a255-11e4-9b9d-cce34a3f6e7d.jpg)

rails_spec doesn't pass and got the following failure (same as with node):
![auto3](https://cloud.githubusercontent.com/assets/1139472/5865127/3d368862-a255-11e4-90da-8745cd6ef88b.jpg)

Hope this helps!

Why you don't create a gemspec with conditionals (if rails 3 if rails 4)? This should avoid the warning issue of bundler.

Best
Dinuz


Keine Ahnung wie ich das geschafft habe, auf jeden Fall kann man die tags ohne diese Datei nicht ändern.


is_numeric can pick up values like '1.0' etc and still evaluate as == true for the values.


@bors: retry


(comments): should have a package comment, unless it's in another file for this package


(comments): exported type Test should have comment or be unexported


Nice!


Generally yes. I tend to ommit them when only one logical operator is involved and the expressions aren't too long.
However, since you feel that this needs more readability I have added parentheses (which look odd to me in this case).


I think we should log the exception here, we don't need the stack so just log the exception message with the path.


Hi Alexei, aren't we risking fragmenting the documentation by creating this page?


saw approval from larsbergstrom
at https://github.com/glennw/servo/commit/0f525d908da229218c88a30cdaa837d492f802be


merging glennw/servo/split-headless = 0f525d90 into auto


glennw/servo/split-headless = 0f525d90 merged ok, testing candidate = 1b496d80


Feel free to correct it then.


Actually it wasn't a `to` I removed and reworded, but rather the wonky `is in your`.


What do you mean by "a link to the correlation"?


Shouldn't have both "virtual" and "override" (or final).


Teşekkür ederim, harika olmuş, şimdi çalışıyor. :)


virtual + override again


**NOTE**: The `type` element is deprecated and has been replaced with `perm_type`, as the word "type" is a reserved keyword in Puppet 4.


```data.WriteBit(isPublic);``` is correct. Think about it another way, ```x != 0``` translates to "x is not equal to false", or in other words when x is true. Use that mnemonic to help you understand this for the future.


Such #if's should be avoided wherever possible, but it is required here.

As a general rule, modifying a method's signature should be avoided unless there is a very strong reason for doing otherwise.  Since `UnsafeNclNativeMethods.GetCurrentThreadId` is a `[DllImport (KERNEL32)]` native method, we would have to either turn it into a managed one or provide a "fake" native dll.


I think this will need to be public for other people to use it/read it?


but the constructor might not need to be public


lowercase field names!


I created a new Exception called RulesPluginException. This might be a start for a discussion. Maybe i can get a hint about how to structure Exceptions in rules.
Is there a better way than to throw an Exception, if $value is not numeric? Currently Datatype for "value" is set to "Any". What i need is "Numeric" or sth. like that.


(comments): should have a package comment, unless it's in another file for this package


(comments): exported type Test should have comment or be unexported


I guess you could say that I redid FaceCardViewController.


Le redirect action ne fonctionnait pas du tout car il etait surchargé par un redirect apres que toutes les actions soit iterees il me semble. Du coup obligé de sortir du script pour le faire fonctionner... J avais vu avec marc avant de push pour valider justement parce que c est violent mais en tout cas sur que ca ne fonctionnait pas ou alors j avais omis qqchose. Voit avec marc ou camille il se rapellera. On l utilise sur MG  apres je pense pas sur d autre proje car ca ne fonctionnait pas. Tu peux m enviyer le lien du doc pour snoopump stp ? Desole j ai pas eu le temps encore. A+

Envoyé de mon iPhone

> Le 22 janv. 2015 à 13:42, David Négrier <notifications@github.com> a écrit :
>
> Heu, David, t'es sûr qu'on a besoin d'un "exit" ici?
> C'est pas un peu violent? On pourrait tout à fait vouloir faire d'autres actions après le redirect!
> On peut l'annuler? Ca a des impacts sur quel projet?
>
> —
> Reply to this email directly or view it on GitHub.
>


Hi @walterxie, you can avoid these little merges if you like by doing a `git pull --rebase` before pushing your changes.


ok what is sort -V - there is no -V on the bsd/osx version of sort ... without the -V it also works @koying fyi ping ...


Well, the output is a number, but we are not counting it. So this must be incorrect and should probably be ``$this->error( wfMessage( 'passwordtooshort' )->params( $wgMinimalPasswordLength )->plain() );``


Published on Sonatype.


Hey @drewfradette thanks for submitting this! We'll take a look at it!

Would be curious to hear how you're using log-shuttle, and this patch specifically?


I would change this variable name to something like 'errorResponse'


dear god it's real


Would someone mind explaining why these methods are blank and the `-[UIViewController presentModalViewController:animated:]` method before has had its body commented out?

cc @BigZaphod


I don't see why, firstly this is some logging class which mono does not need and will cause more problems on mobile. Using empty dummy implementation should be enough. If you really want this useless code in mono you can still not include UnsafeNclNativeMethods (which we don't) and implement your own UnsafeNclNativeMethods:: GetCurrentThreadId without any change to reference source code


That's fine. We'll just need to make the needed adjustments in the layout template


Rather than parsing the input string one character at a time and "backtracking" when we see we've run into something like `"IV"`, consider occasionally looking ahead one character.  Deleting data from the middle of an `Array` is a (relatively) expensive operation.

There's also a nice recursive approach if you want to take a stab.  Consider the roman number `"MCIV"`.  You can take advantage of the following equalities to implement a recursive solution:

```ruby
convert("MCIV") == convert("M") + convert("CIV")
                == convert("M") + (convert("C") + convert("IV"))
                == convert("M") + (convert("C") + (convert("IV") + convert("")))
                == 1000         +  100          +  4             + 0
                == 1104
```


you are probably right. ;)


However, since a number could be the result of a count than this may probably be correct.


I *thought* there was something strange.


Shouldn't this be checked... I thought Jana is leaving it.


Mark
yeah I know, but I have seen it written that async/await is a DI. So I would like an authoritive person to debunk that notion.

But async/await  is just a small part.
Your other musings are more important eg Pure DI as a real alternative to a IoC container

Please have a good look at this.

Regards

Greg Frazer
0414 352 865

Date: Thu, 22 Jan 2015 02:44:21 -0800
From: notifications@github.com
To: ploeh.github.com@noreply.github.com
CC: gregfrazer@hotmail.com
Subject: Re: [ploeh.github.com] Answerer reader questions. (aefb632)

What is there to say about async/await? Those keywords are only syntactic sugar for methods that return Task or Task<T>.

You can define interface methods that return Task or Task<T>. At that point, such an interface is like any other interface, and can be injected and consumed using the techniques described in my book.

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

 		 	   		  =


I assume it can find the platform library because cmake succeeds, but build fails with

```
In file included from /home/garrett/Documents/kodi-cmake/build/build/bootstrap/build/peripheral.joystick/src/peripheral.joystick/src/addon.cpp:24:0:
/home/garrett/Documents/kodi-cmake/build/build/bootstrap/build/peripheral.joystick/src/peripheral.joystick/src/api/JoystickManager.h:22:40: fatal error: platform/threads/mutex.h: No such file or directory
 #include "platform/threads/mutex.h"
                                        ^
```

Do I need to include by a different name? `#include "lib/platform/threads/mutex.h"` doesn't help either. how can I build and install the kodi-platform library?


Hmm, but calling onError after a request is not forbidden … so you are saying that for the purposes of the TCK we should assume that only expected (i.e. explicitly stimulated) onError is permitted? Asserting the presence of a notification is a lot more reliable than trying to assert its absence (which strictly speaking is impossible). I’ll have to think more about this and confer with @ktoso tomorrow.


this error message is still incorrect. it suggests that at least one of these four is required for a successful request, which isn't the case. your logic checking is still off too -- contentlang is completely inconsequential to the validity of a request.
you should have two separate checks - first, check for taxonomy only, and if it's null or empty provide an appropriate message. then, see if title AND desc are both null-or-empty, and if so, provide a different message about that.


We need a big couch for sure ! I am happy with getting a used one, like the one Remus found.


The Perl side of me misses stuff like this but the PHP side of me had to do a double take to make sure this worked out.


Thanks, @ajdavis!


Not a fan of carpets, but maybe we could get something smaller between the kitchen table and the couch ?


:+1:

:)


it's all built for you and available, you just need the correct header paths. see my first post - that applies to *all* headers, from kodiplatform and the libXBMC_xxx.h and the xbmc_peripheral_xxx.h - all have kodi prefix. i.e., make it kodi/threads/mutex.h


done


Needs a space between "navigation" and "to"


You could do another invalid argument check here for -90 <= lat <= 90 and -180 <= lon <= 180.


I think, because it's deprecated in last version of iOS SDK
2015-01-23 0:55 GMT+03:00 Michael Melanson <notifications@github.com>:

> Would someone mind explaining why these methods are blank and the -[UIViewController
> presentModalViewController:animated:] method before has had its body
> commented out?
>
>


null *or* empty


I need to find our old discussion of why we could not just have a normal salt/binary for this format (with lots of things going to the "binary"). Right now I can't see why we couldn't.


-- DB/Misc: Fix startup error
-- Quest hast ItemDropQuantity1 and not 2
UPDATE `quest_template` SET `ItemDropQuantity1`=1, `ItemDropQuantity2`=0 WHERE  (`ID`=28777);


Never thought of taking mpal into consideration! Cheers for that


ah i see. `everything has a kodi/ prefix` ... thought you were talking about an install directory or something :) I'll change to kodi/threads/mutex.h and see what happens


At run time, there will be only 5 keyboard objects - English, Symbol, Shifted Symbol, Emoji and Current keyboard. Before there are 14 keyboard objects are declared in MyIME class. I think it will be the better way for future memory management and more independent supporting new Languages.


Ooh, This is awesome. Thanks for doing this!


nit: attemts in commit message


2.2.2 is out... :-)


btw, you can see the layout in <builddir>/build/include - that's the -I / include path used..


there is a todo, please remove... oh wait


File name should be `person.rb`


TeamCity Pester :: Test Pull Requests [Build 1](http://teamcity.codebetter.com/viewLog.html?buildId=179217&buildTypeId=Pester_TestPullRequests) outcome was **UNKNOWN**
Summary: Canceled (Exit code 1 (new)) Build time: 00:00:30


If it is causing problems, then we can modify `Logging.InitializeLogging()` by making it conditional and/or a dummy call and we could also change the default value of `s_LoggingEnabled` to `false`.  I could also live with stubbing this class out - but it would have to be a full stub of the entire "internal" and "protected" API - and then we quickly reach the point where stubbing out will become a lot more work than just bringing the class.


Looks like this was a bug: https://github.com/rails/rails/issues/18646

But `.model_name` will be needed anyway to work with current and old versions


:+1:  for this :-)


Remove this


I'd suggest adding a separate emit_load_general () or something function to avoid changing all call sites. Same for emit_store ().


Why don't you catch the exception anymore?


@roman-shuhov see #12


fiar


https://www.microsoft.com/resources/documentation/windows/xp/all/proddocs/en-us/sort.mspx?mfr=true

2015-01-22 23:03 GMT+01:00 davilla <notifications@github.com>:

>
> https://developer.apple.com/library/mac/documentation/Darwin/Reference/ManPages/man1/sort.1.html
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/xbmc/commit/396417883fe1c34180a6a86fa90629394b20109b#commitcomment-9406022>
> .
>

--
                   Key-ID:     0x1A995A9B
                   keyserver: pgp.mit.edu
==============================================================
Fingerprint: 4606 DA19 EC2E 9A0B 0157  C81B DA07 CF63 1A99 5A9B


@XhmikosR: You might wanna check the submodule. Hopefully the 64-bit pdb package will get uploaded this time.


That refactor/rename would be so nice if you would have also renamed those defines to CHANNEL_ instead of HCS :-(


Deprecated doesn't mean it should be disabled...


This commit attempts to fix #2


Change README file


This file was synchronized in develop branch right after Container was modified. We only need to merge the changes into master.


I don't understand how you chose to address this. Can you enlighten me about why you're taking a strong reference to the messagesViewController (that should still be in the view hierarchy) in the FingerprintViewController class?


@bjo81 Sollen wir Oldenburg dan raus nehmen?


TeamCity Pester :: Test Pull Requests [Build 2](http://teamcity.codebetter.com/viewLog.html?buildId=179218&buildTypeId=Pester_TestPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 311 Build time: 00:01:39


I think this was one of my first ones, that I 'reworked', prior to knowing what I was really doing. I really think this SHOULD be a normal binary.

In fact, I think it 'is' a normal binary, it is just we are not using crypt_out as the name.  That mic[] is the crypt_out (I think).


Done.


Do not ask me why this ended up in master. :(


sdfsdfsdfsdf


Yep but I picked an acronym (Hercules Channel System) in order to keep the constants/enums to a reasonable length, 'channel' is way too long, and 'chan' doesn't sound too well.


Perhaps this could be changed for one of the garray functions that @philix et al invented for freeing a garray?


It's good practice to include file extension for local file includes.


http://www.freebsd.org/cgi/man.cgi?format=html&query=sort(1)

:)


All I did, was to blank out the binary element from the salt.   There is a single function that loads a structure, parsing the input line.  Both binary() and salt() use this function.  binary() memcpy this keymic data into what it uses.  salt simply was using the entire structure.  Now, I memset this keymic (binary) data within the salt() function prior to saving it as the salt.

So I 'think' we have a real binary. It is just we use a common parser function.


1540218-SlinkyGroundHX (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


Can you explain why this is better than settings vars based on the environment?


Please add some test to it https://github.com/gitextensions/gitextensions/blob/master/GitExtensionsTest/GitCommands/Helpers/PathUtilTest.cs


Brief comments explaining the logic should be here.


Why split this out by env? Feels like it should be all in one place, unless we explicitly remove permissions from one environment.


@rkuhn A Publisher could call onError as a response to `subscribe` too, but that doesn't prevent us from trying to test more :)


Reconsider using ```a``` tags these links rather than heading tags.


Can we pass a tag rather than hash here (if not the comment makes it fine with me) ?


I don't seen an opening ul tag. And I change all the l1's to li's (I think we talked about this already.)


Nothing wrong with this in my opinion.  I'm fine with test function names being fairly long and descriptive.  The most important thing though is to just link to the issue this is testing, which you did :)


Yeah, looks good now. :) Thanks.


Hey Tanner,

Is this what was meant by making the output sensible? Now the key/value pairs get printed out rather than returned. Is that an issue?


Treated as False yes. Updated to look at iter, but probably don't need to worry about this too much assuming use of protos sometime in the future.


The submodules seem fine. As for the x64 PDB, it doesn't make any sense so I don't know why it failed last time. We'll see tomorrow.


s/ object//


"into k-mers"


consider using [bower](http://bower.io/) to manage these dependencies


Not exactly
in object scala
type Exception = java.lang.Exception

Meaning I'm dealing with checked exceptions. I agree that this is not the best pattern anyway. I will try to improve that part.


1540218-SlinkyGroundHX (mmatts) - 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)


Mach mal, ich mach dann für nordwest einen neuen Pullrequest.


"... from file, break into k-mers, and add each..."


are these really necessary? regular variables seems to work fine in the last foreach


That argument name is solid gibberish.


updated commit message,

merged two if statements and moved two for-loops into seperate function. Happy now?


Looks good overall. I would add a note that shows how to import the YowIqProtocolLayer as well for the auto ping thread to be used automatically.

from yowsup.layers.protocol_iq.protocolentities          import *
from yowsup.layers.protocol_iq import *

..
layers = (
        MyLayerName,
        (YowAuthenticationProtocolLayer,
         YowMessagesProtocolLayer,
         YowReceiptProtocolLayer,
         YowAckProtocolLayer,
         YowMediaProtocolLayer,
         YowIqProtocolLayer) # Needed for auto ping
    ) + YOWSUP_CORE_LAYERS
..


Philosophically, it's nice to be explicit, or at least have a way to be explicit if you want to. Practically, I wanted to use the same vars file for a different inventory, and it seemed wrong to have to make a copy of the vars file to do that


Why the cast given the new signature for the method?


sry is for 6_x


Naww, that's ok :smile:


Yeah, those were left in from troubleshooting an unrelated issue. I can flatten them tonight if you like, or if you want you can as well.


It forces placing a variable in your call to `ansible-playbook` which helps to ensure that you are thinking about it when you execute the playbook.  Same for `version`


Nothing to see here folks, move along please...


Hmm... I'm not crazy about using regular expressions for this.  I'd rather the logic simply did not include the final ampersand in the first place.  On the other hand this isn't performance critical, so I'm not vehemently against it either.  However, you could change this to work with a single `re.sub` call instead of using a superfluous `re.match` first.  See the docs on `re.sub` for how you could do this, if you want to pursue the regular expression route.


The libtool scripts are just there to install.  Once it's installed it doesn't matter.  There's other ways to call these libraries like -I$(location) -lisl -losl
And LD_LIBRARY_PATH/LIBRARY_PATH


I think a newline between 72 and 73 would increase readability, you agree?


same here (newline)


As a matter of fact I'll remove libtool scripts from the toolchains completely in the future.


Yeah, I may change the way this is done again. But it's nice for example to be able to give a third party access to one of the instances's machines without giving them access to all our instances's machines. Etc. In the future we could, in our private config repo where we store the vars for the various environments, find some way of having a shared file that's imported to the others or something, but I didn't look into how to do that for now.


New PR is created https://github.com/jbosstools/jbosstools-jst/pull/439


@aldenml This solves the Out of Memory error coming from scanning songs to send to iTunes we've been getting.

Unbelievable, we were creating a single process to execute an OS script to send files to iTunes, files were being sent one by one when you selected them on the library and used the right click menu action..

After a few seconds of execution the whole operating system would collapse completely with a few hundred files.
I had no choice but to turn off the computer.

Now it imports the files no problem.

Have to test on Windows next.


would be nice to also mention that one can append an arbitrary number of such settings


Just curious: why `rem` (as opposed to `1px`)?


I'll add some setup info and screenshots later - came in useful for us

On Thu, 22 Jan 2015 22:00 Agustin Colchado <notifications@github.com> wrote:

> [image: :+1:]
>
> :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hollandben/team-dashboard/commit/d61462096a8c52b106422bdce39daf2702281743#commitcomment-9405965>
> .
>


Yeah but you used chan for a lot of variables too. And it is only 3 characters longer. I get your point though.
Anyway nice refactor. :+1:


Looks like these URLs aren’t quite right — `http://localhost:3000/27/500/lit0fgki1Z1qfh1tao1/jpg` on my local just now.

`'%s/%s%s/%s/%s'` should be `'%s/%s%s/%s.%s'` perhaps?


made it so that it creates up to 4 script calls at the time.


1540218-SlinkyGroundHX (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

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


@DefinitlyEvil As in use a seperate class to handle the commands, and use this one just to call the method? (Or something similar?)


Eh, yeah I could have gone further in figuring out whether there was a tag for this. I just went to one of our production servers, went to the repo in /opt/lessc, noted the revision hash, did a git log and noticed the last commit was about how this was now 1.7.3, and called it a day.

I'm going to include the word todo-item in this comment so I can search for smaller things like this that I agree in principal are correct, but am not going to chnage until other things are working


Cool new method for array splitting. Useful for task batch processing.


Forward declare


Cleaner as vector of strings


Typo on this line?


Well it just stops executing from there. So no bugs just an error message. So I don't see the issue.


oh, and before i forget. there's one minor annoyance you need to be aware of.

if you change the addon bindings, you have to explicitly rerun cmake in the build folder to get the new files into the build tree. another todo. this one is probably gonna be a bit irritating for you, i will see if i can fix that one up tomorrow. it's soon midnight over here so i'm crashing for the night.


Yes, should use org.apache.hadoop.hive.ql.io.parquet.read.DataWritableReadSupport#PARQUET_COLUMN_INDEX_ACCESS
This is added in:
https://github.com/Parquet/parquet-mr/commit/f284238631cb1026b4977f6f0b7ef342260d35c5
which is available in 1.6.0rc releases:
http://mvnrepository.com/artifact/com.twitter/parquet-column
shall we consider updating parquet version to 1.6.0rc in presto-hive-apache(it is still rc)?
Or, I add todo here


Wow, great idea @barneycarroll! This has really blown me down!

I keep finding myself short of time lately. I hope I'll have a while on Saturday to set up browser tests. That should make development way easier than it is now.

I'll also try to clean up the ES5 generated code.


Even if that flag should be cleared (which maybe it is). I'm not sure it's FEProblem's job to clear the flag. It seems like adaptivity or the mesh should do that instead - hmm...


all tests pass:
success: http://build.servo.org/builders/linux1/builds/593
success: http://build.servo.org/builders/linux2/builds/593
success: http://build.servo.org/builders/android/builds/629
success: http://build.servo.org/builders/mac1/builds/810
success: http://build.servo.org/builders/mac2/builds/810


Noob.


Meiner Meinung nach wäre es noch super, wenn man beim Upload gleich eine Kategorie auswählen könnte. Muss jetzt mal schauen, hab gerade erst angefangen das Media module zu integrieren, hoffe es hat sich nicht allzu viel verändert. :D

http://static.dastii.de/files/2015-01/2015-01-22_23-30-33.mp4


Wär schön, wenn Ihr euren alten Oldenburg-Eintrag dann zusammen mit dem neuen raus schmeißt. Dann ist hinterher klarer, was passiert ist. thx!


Hi

I have opened a request for this some time ago and it was closed with Won't
Fix ...

Just to let you know that I have already reported this.

Regards,
Areg
On 22 Jan 2015 21:35, "Hardy" <notifications@github.com> wrote:

> Victor, yes, certainly not a "BIG" issue... But I want to note things like
> this, because to me these are "quality issues", because we altered (and
> some people may say we broke) functionality which worked (differently)
> before.
>
> And before, the color coding I once introduced said: "green=latest
> version, light-blue=newer-version available". And that was regardless if
> you had your map the active or in the backup folder.
>
> So I consider it my duty to note stuff like this and mention it, (before
> our users do :-) ), because that makes the difference between a
> "quality-controlled" and a "we don't care so much" type of product,.
>
> Hope you don't mind ... :-)
>
> Best, Hardy
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/osmandapp/Osmand/commit/261b12a651ac297c0e129ccf7ed3e96a7c2584ba#commitcomment-9405477>
> .
>


fast-forwarding master to auto = 1b496d80


I'm sure you can get Maven working very well in case you are ready to invest loads and loads of time learning all the odds and ends of the system.

But why would I want to spend that time? Maven has terrible usability forcing the developer to become an expert in writing special xml files. Eclipse jar export works very well and gets the job done. And Eclipse jar export just works. You mark what you want to export by selecting it in a GUI. Maven is just not worth it because of the time you need to invest :smile_cat:.

Some of the issues I experienced:
* Building Maven projects takes longer than when using Eclipse jar export.
* Maven projects randomly fail at building. Some times I try building three times in a row before it works.
* Bad cooperation in Eclipse between Maven projects and non maven projects. For example non maven projects don't see shaded libraries (rendering the only useful feature for me useless).
* Ugly folder structure. Maven enforces a bloated folder structure I have no use of (I don't code tests).
* Maven is an extremist cultist. You may only use Maven. You must join the cult 100% and leave your old life behind. Because Maven only works with Maven. It has compatibility with nothing else.


Woot! I have a repo!


Maybe a list instead?


@Optimusprimeums hey mate, I'll be tagging 5.0.0 for OS X soon, would be great if you can build them :) thanks a lot


// Start enforcing the new rules, for block.nVersion=3 blocks, when 75% of the network has upgraded:


You need a ```ul``` or ```ol``` parent for lists.


Well it says settings in plural form and the help text generated by coala -h says:

-S SETTING [SETTING ...], --settings SETTING [SETTING ...]
                        Arbitrary settings in the form of section.key=value

as opposed to i.e.

  -s [FILE], --save [FILE]
                        Filename of file to be saved to, defaults to config
                        file


@magnusulf - The problem is: It does not get the job done. We don't want an error message. We want the job done.


I believe that Breath of Fury *does* have to check the return value of Permanent.sacrifice. [Commander and Rest in Peace style replacement effects don't effect whether the sacrifice was successful or not.](https://github.com/magefree/mage/blob/dbffbad3cbbcc5a16d79401a2c82875b3ace6edf/Mage/src/mage/game/permanent/PermanentImpl.java#L856) Proposed change at [my github](https://github.com/duncant/mage/commit/f9443f90f09485a55287a47f9c442b485ed8db3a)


oh... mutability?


@jverburg shouldn't be `element` instead `value` here?


ScalaDoc and CLI assemblies published on the Website.


We aren't using the variable instance, correct?  Assigning to an unused variable seems like more of a lint issue than discarding a return value does.


:star:


Ja könnte man auch noch machen, aber ich bin noch lange nicht fertig. Ich möchte noch einige sachen verändern für den user und entwickler.


Apparently not enough?  Travis fails with:
```
Gem::InstallError: i18n requires Ruby version >= 1.9.3.
Installing rspec-mocks 3.1.3
An error occurred while installing i18n (0.7.0), and Bundler cannot continue.
Make sure that `gem install i18n -v '0.7.0'` succeeds before bundling.
```


i'm sure we'll end up doing the same thing with nolimit.


Mmm, I'll have a look.


I would create as many variables as needed to make this readable/understandable by a human.


Yeah, I wouldn't consider this an article. You can probably do without an article tag.


Wat is deze haat Jens, en waarom krijg ik deze mail?
On 22 Jan 2015 23:37, Jens de Waard <notifications@github.com> wrote:Noob.

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


Well it says "settings" in plural and the help text generated by "coala -h" says:

>  -S SETTING [SETTING ...], --settings SETTING [SETTING ...]
>                        Arbitrary settings in the form of section.key=value

as opposed to i.e.:
>  -s [FILE], --save [FILE]
>                        Filename of file to be saved to, defaults to config
>                        file


Ah, right, the flag on displacedmesh needs to be cleared too.


Minimal headers, cache disabled, compression disabled…
```
< HTTP/1.1 200 OK
< X-Powered-By: PHP
< Server: Mind your own business !
< Content-type: text/html; charset=utf-8
< Content-Length: 1361
< Date: Thu, 22 Jan 2015 22:44:53 GMT
```


This isn't really a nav.....?


You can also probably do something super simple like

```
var nextIndex = blah blah;
if (!$slides[nextIndex]) {
  nextIndex = 0;
}
```


If you're going to do this, comment them out.


    public ulong Target { get; set; }
    public byte Symbol  { get; set; }


well its okay then, care to add the newlines?

2015-01-22 23:41 GMT+01:00 Fabian Neuschmidt <notifications@github.com>:

> Well it says settings in plural form and the help text generated by coala
> -h says:
>
> -S SETTING [SETTING ...], --settings SETTING [SETTING ...]
> Arbitrary settings in the form of section.key=value
>
> as opposed to i.e.
>
> -s [FILE], --save [FILE]
> Filename of file to be saved to, defaults to config
> file
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/da265fa07eaf7440efd9b5d92da4554054c7b17b#commitcomment-9406651>
> .
>


Subj. It totally not supported, so don't let people install it by mistake.


Please note that it's complaining about i18n, not simplecov :)


Good idea, I opened issue #17 as  this is certainly missing!
Currently, I am working on some other things (like showing errors in (original) BBCode, to make to problem(s) more clear), so it might take a while before I can get to it...


Not that I care about what you guys end up doing, but personally when I install an npm package I never copy-paste the `--save` part because I want to choose which version to put in my package.json. I always pin it to a minor or major release, like `1.2.x` for example


Hang on, I'm not sure clearing that flag is correct. The documentation says it's for reporting the uniform refinement level so clearing it could be misleading. We may need a function on the mesh that reports the difference between uniform_refinement and the refinement steps performed (i.e. The refinement level yet to be performed). We could the use that number to perform refinement instead which keeps all of the accessors correct over the course of the simulation.

Alternatively we could simply pass that value in from somewhere else so you could control it. I'm not on board with the clearing that number - that creates state and abuses that variable's meaning.


I see, so if sychronizing the flag was too expensive and the appendable flag was only stored thread-local then the worst thing that could happen is a reallocation because a thread might not know about the flag when it got the array from another thread, i.e. the capacity might differ.


heyasdasd


Looks excellent Andrew.


cue the shitstorm


@oloflarsson I didn't think I was in a cult. Besides the clearly invalid points, I'd just like to point out that not everyone in the world has Eclipse. I use IntelliJ IDEA. Eclipse just wouldn't work. I'd never download that piece of junk to compile a project.

Maven is a dependency manager, as well. It allows for people without the dependencies to download them (the correct versions) with no hassle, and everything is built automatically using these dependencies. You never have to upload binaries to a repository, which is a cardinal sin.

While I'm going, I should mention that three folders isn't bloated. It's just not. Ugly is an opinion, but once you write good code ("I don't code tests"), you'll find it's quite nice to have everything separated.

Seeing the logs of the second bullet point would be interesting. Maven works all the time for me, unless I break it.

You can get rid of all of your Eclipse problems by using better software.

P.S. Please don't call me an extremist and a cultist. I am neither of those things. It's not very nice internet etiquette, and it just makes people feel bad.


Ok, I'm just a lawyer so forgive me for having now idea of whether I'm "editing" or doing what here! But, I did just share some initial views on this issue with the CyberLaw type mailing list of the American Bar Ass'n so I will reshare them here:

While I do find some level of appeal to the concept of defining "exceeds authorized access" as is being thrown out there, the task of actually defining it can be quite another thing -- but I do think this is a valuable area of inquiry.

What makes my head hurt, however, is all of this talk of changing the penalties for the violations at the lower end of the spectrum: have you ever tried to get the authorities to investigate -- much less prosecute -- a case at the lower end of the spectrum where these changes are focused? It is next to impossible -- not because they do not want to, but because they do not have the manpower. They are overwhelmed. In my conversations with the local SS / FBI / DOJ, there seems to be a significant threshold that must be met that involves either a substantial amount of money or some significant threat to the public before they can focus on the case. The ranges that the President's CFAA proposals focus on do not even sniff that threshold.

Assuming this is generally the case throughout the country, what this means is that whether these proposals were to become law or not, enforcement of cases at those lower end levels will still be random and based on the attention the cases generate, which gets us right back to the biggest gripe that most people seem to have with the existing CFAA.


ack, thanks. Will merge if travis agrees too.


Simplified OutputWriter interface as those additional methods now belong to BatchedOutputWriter.


Batching done by batched output writer, at this point, we just write results individually.


I'm not sure this will be better...

Try adding a `logError(t, msg, args)` which has this if block inside, so we don't have to repeat the message.


Yes, yes it should!


sweet.


No need for this Eric.  You're not using hash rockets so Ruby 1.8.7 isn't supported already


That's actually a deletion of the unused class and creation of a new one. Git did too much magic here.


82420068_GridViewMultiEdit (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2174 of 2188 tests passed)

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


Again, too much Git magic, actually a deletion and an addition


I would also add an example explaining the quirk with inferential typing.


@kecinzer have you tried virtualbox?

http://download.virtualbox.org/virtualbox/4.3.20/VirtualBox-4.3.20-96996-OSX.dmg


SocketOutputStream and SocketOutputWriter were actually not used anywhere. Removed...


Weirdly, this seems to say Firefox is the only browser which supports it, as of two days ago:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/from#Browser_compatibility

…although I thought all the ES6 stuff was getting auto-converted back to 5 as part of our make step, no?


@bufke Significant code review requested here - there's only so much I can test on my local machine, though everything seems to be working ok - my unfamiliarity with supervisor & celery leave me feeling that there's got to be something not quite right with the above implementation!


As someone who uses Maven, ANT, and Eclipse build scripts, I can confirm that I am in fact a member of the Maven death cult dedicated to eliminating all infidel build tools from the civilised world.

Oh. Wait.


@Jalle19 why you have removed this? Without the set of index the addon becomes always a 0.
Was needed to identify related timer on backend.


Good question. I didn't find any other usage in getnikola/nikola and getnikola/plugins. @ralsina, @Kwpolska: do you know other uses?

Anyway, even with stages, getting rid of for example the GZip plugin is not trivial at all. After all, it compresses all ```.html``` (or something else, depending on its config) files in the output folder generated by any other task. It has to be able to inspect every generated task for its targets, as it has no way of knowing which plugins/tasks will create ```.html``` files. So I don't think we can get rid of ```TaskMultiplier``` plugins so easily, even if this would be its only use.


Committing commented code? Foei!


I don't want the worker here. It belongs in it's own container. If the worker decides to request 90GB of ram it shouldn't bring down the whole thing.

Isn't the -B embedded beat? But then also another celery-beat running?


oooooops


Why not just make a method that takes the info and does the formatting, so you get consistent error messages also


Add one more call where you pass something of type `{ name: string; description: any; }`. It should pick the second overload because it's not a subtype of Item.


Why did you remove the comment? It appears to be quite valid and useful.


1540218-SlinkyGroundHX (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Build Failed

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


Ah yes, we should probably remove that `-B` read: http://docs.celeryproject.org/en/latest/userguide/periodic-tasks.html#starting-the-scheduler,

However, I am totally unsure how to have the celery-worker living inside it's own container and talking to the Django container


@bugventure , have you got any idea why this would happen? do you modify any native objects?
the moment I put:
```
var RequestValidator = require("request-validator");
```
the test suite breaks.

before:
```
$ node benchmark.js
uniqueItems is a no-op
z-schema-3#basicObject x 41,329 ops/sec ±1.22% (99 runs sampled)
z-schema-2#basicObject x 13,844 ops/sec ±1.59% (98 runs sampled)
is-my-json-valid#basicObject x 3,705,582 ops/sec ±1.15% (94 runs sampled)
jassi#basicObject x 34,803 ops/sec ±1.45% (99 runs sampled)
jayschema#basicObject x 853 ops/sec ±2.33% (92 runs sampled)
jjv#basicObject x 10,137 ops/sec ±0.64% (98 runs sampled)
json-model#basicObject x 75,547 ops/sec ±0.87% (97 runs sampled)
jsck#basicObject x 170,960 ops/sec ±1.08% (98 runs sampled)
jsonschema#basicObject x 1,352 ops/sec ±1.44% (93 runs sampled)
tv4#basicObject x 34,564 ops/sec ±1.40% (97 runs sampled)
Fastest is is-my-json-valid#basicObject
```
after:
```
$ node benchmark.js
jsck failed the test basicObject
tv4 failed the test basicObject
z-schema-3#basicObject x 39,727 ops/sec ±1.60% (94 runs sampled)
z-schema-2#basicObject x 13,696 ops/sec ±10.05% (181 runs sampled)
is-my-json-valid#basicObject x 20,694 ops/sec ±13.11% (274 runs sampled)
jassi#basicObject x 23,292 ops/sec ±11.14% (370 runs sampled)
jayschema#basicObject x 4,003 ops/sec ±16.14% (454 runs sampled)
jjv#basicObject x 4,274 ops/sec ±14.43% (544 runs sampled)
json-model#basicObject x 4,918 ops/sec ±14.59% (632 runs sampled)
jsonschema#basicObject x 2,979 ops/sec ±11.06% (718 runs sampled)
Fastest is z-schema-3#basicObject,z-schema-2#basicObject,is-my-json-valid#basicObject,jassi#basicObject,jayschema#basicObject,jjv#basicObject,json-model#basicObject,jsonschema#basicObject
```
notice also the performace difference for `is-my-json-valid`
cc @geraintluff @automatthew


pff :D


Missing the resolve flag on this line.


zlib1g-dev is a debian package name, which does not apply to Fedora, for example. Will you change it to `zlib-devel` please? Otherwise it looks good to me :)


i will probe it ...


Wait a second..  You said during class that we didn't actually need to write in all the text of every element, and that the goal was to get the semantic structure laid out.   It is exactly for that reason that I added that in there.   I'm extremely frustrated and confused.


This is working in every demo in which it's used – a few of them are using branches which haven't been merged yet, and they're still nonfunctional in Firefox. For reference, those are:
- http://tangrams.github.io/osm-webgl-demos/demo1/
- http://tangrams.github.io/osm-webgl-demos/demo2/
- http://tangrams.github.io/osm-webgl-demos/demo4/


Ik zit met Hugo whisky te drinken en code te bekijken.
Dat zijn teksten van merge conflicts, een conflict krijgen en deze dingen vervolgens committen ipv afhandelen is een beginnersfout, dus noob.

De mails krijg je omdat dat ingesteld is. https://github.com/settings/notifications


I'll see what I can do.


So in other words, I can leave out the "server" information in the config.json file now?


Sorry, I moved stuff around to make more sense in my head!


Suggestion: No need to disable both logfilter and minlevel as you're reloading the page right after. It will be set by your windows.load function.


"no client certificate is provided", even


Whoops, these changes here are actually not necessary. I will delete them before pushing.


I am imagining some difficulty handling the ternary relation between node, dataset and region of sky with the current ER diagram; but haven't got any concrete cases yet.

Do you assert that a node always host the same portion of sky, regard less of the data set?


Is this for us or 3rd party apps?

on either case I don't think we should make it fully controllable, I think we should provide 3/4 different sizes and cache them locally so they are not  generated the app developer is not very smart.


I meant to ask, can I add a prerequisite section above for Ubuntu and Fedora? You need `elfutils`, `libdw-dev` and `zlib1g-dev` along with `build-essentials` on Ubuntu. I'll add `zlib-devel` under Fedora. Sorry, I should've checked what the Fedora package name was instead of being lazy :)


the actual re-sizing should be done in core not in the api, with integration tests to make sure it works in mono.


We shouldn't allow device drivers to write invalid values; this would contradict memory safety. We can, however, allow them to write incorrect, valid values. That being said, `set_mode` isn't exposed to device drivers anymore, so it doesn't need to be unsafe. Before, 'parity' and `stop_bits` were integers being used in the calculation of the mode register value, which leaked the unsafety to device drivers.


(y)


Suggestion: You could remove everything related to logsearch here and put under logsearch.keyup with a "if(event.keyCode == 13)" so it will reload the page when the user press enter.


:muscle:


With this empty boot method parent method is not called and listens and suscribes are no more checked.


Can you add the case with the indexer as well?


Well... it does in the tests :S It was really just a convenient way of mocking what happens in the actual DataServiceController


Job pr-scala failed for a315a26e Took 180 min. (ping @lrytz) [(results)](https://scala-webapps.epfl.ch/jenkins/job/pr-scala/9568/):

To retry exactly this commit, comment `PLS REBUILD/pr-scala@a315a26e509988fb87bba4a14dcbd2ac88a0a9de` on PR 4262.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.


CBECC (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2041 of 2157 tests passed)

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


Thanks man
On 23 Jan 2015 00:16, Jens de Waard <notifications@github.com> wrote:

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


I found some weirdness when benchmarking validators if I required z-schema as well:

https://github.com/pandastrike/jsck/blob/master/benchmarks/draft4/validators.coffee#L52-L53


Need to update this to reflect the 4.1.2 version.   Need to see if I can get a test version running on Ironhorse before I move it into production on Phoenix.


That’s true, I did say that. But I’m just saying comment it out.

—
Sent from Mailbox

On Thu, Jan 22, 2015 at 3:07 PM, KegMack <notifications@github.com> wrote:

> Wait a second..  You said during class that we didn't actually need to write in all the text of every element, and that the goal was to get the semantic structure laid out.   It is exactly for that reason that I added that in there.   I'm extremely frustrated and confused.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/KegMack/rebuild_website/commit/afc60f5167f0557efc1f46c97bc7a07b31135337#commitcomment-9407061


81605912-Multispeed-Heating-Defrost-Bug (lgu1234) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Please review the database timestamp format..

http://dev.mysql.com/doc/refman/5.5/en/date-and-time-functions.html


I just can't stop lol-ing on this!


ok sounds good.  I appreciate the feedback


FYI: What CountingHash does can break if a sub-class is GC aware.


Mod of line 1857 isn't necessary.


Er kan natuurlijk niet alleen maar gezeken worden ;)
Ik kan niet genoeg fouten vinden in je code :(


Dan kijk je niet goed genoeg. Te diep in je whisky glas gekeken?
On 23 Jan 2015 00:20, Jens de Waard <notifications@github.com> wrote:Er kan natuurlijk niet alleen maar gezeken worden ;)
Ik kan niet genoeg fouten vinden in je code :(

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


CBECC (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Exactly, I had discussion with Areg and I don't feel good idea to update deactivated maps. Concerns about traffic and reports how many deactivated maps users can have. May be we could disable color, so people will be less confused?


I don't agree, DiskProvider is a utility, the method is not 'VerifyFolderWritable' or 'EnsureFolderWritable'.
It should only report the results, not log exceptions.


Why is this duplicate code still here? The entire function should just return the ```Ext.util.Format.fileSize()``` value. No other code should be here...


@Sirupsen Yes, please :)


We don't currently have tests for IJulia.  Probably we could do something using the IPython console?


So it doesn't need logging?
we log other errors in `DiskProvider`. Don't log exception just why did it fail the test.


Yes, it should be ``Message``


Yeah, I needed the replace encoding (for switching to a non-ascii lang hence running `SITELANG=ja`) stuff and didn't want to start using `SMWInfolink::encodeParameters` because this class needs to be dismantled in a proper manner (I just don't have to energy to do this right now) but for the `UrlEncoder` I can least claim that it is tested to a certain degree. The test uses a round-trip with the same dataprovider as for decode (which comprises several edge cases found in earlier bug reports).


This test will only pass if:
```
npm install phantomjs -g
```
Not on a fresh system.


This needs to be rolled back after Alexey discussed with Sunil. Final decision is one word.


![screen shot 2015-01-22 at 18 25 31](https://cloud.githubusercontent.com/assets/1155163/5866926/291c1e1e-a264-11e4-9022-bee2714c299c.png)


Ah this is exactly the error I'm getting attempting to get Idris compiling!

I'm a noob to the haskell ecosystem and cabal etc, how can I get this new version to get Idris installed?


Ik heb de code niet gepulld, dus ik kan niets zeggen over de technische correctheid ervan, maar ik heb slordigere code gezien. Ik zou niet op zulk niveau werktuigbouwdingen kunnen doen.


I do need more documentation; that is still on my plate. Trillian is currently modeled to have a central server (represented by the "server" table, which will be filled out with properties I'm sure over time).

Node: A second server that is connected to the central server. The node may provide storage, compute capability, or both. Note that the central server is also a node in this sense. (For a while, it will be the only node.)

Dataset: A catalog or specific release of data. Examples would be SDSS, WISE, 2MASS, etc. A dataset may cover the full sky or some portion of it; this is wholly independent of Trillian.

Trixel: I divide the sky into regions (mapped to HEALPix pixels, though "region" might be a better word than "pixel" here). A "trixel" (Trillian pixel) is a container that stores all of the data within a HEALPix region - there is a one to one mapping between HEALPix regions and trixels. A simple way to think of a trixel is as a folder with all of the data relating to that part of the sky. (In reality, it will also contain a database.)

> Do you assert that a node always host the same portion of sky, regard less of the data set?

Trixels are stored on nodes. When Trillian is "initialized", twelve trixels will be created (the most course resolution of HEALPix) and placed on the first node (i.e. the central server). As data is added, it is placed into the trixel that represents that part of the sky. When the disk becomes full, a trixel is subdivided into four (the next higher resolution, or n_side++), and, for example, three of the new trixels are moved to a new node (another server). (There is a lot of room for optimization of how trixels are subdivided and stored.)

So to answer the question, yes, a node will host a particular part of the sky at any given time, but the part of the sky a node contains may change over time as new data is added.

I hope that's clear! Diagrams will be helpful at some point.


Hah, right, yeah - https://github.com/JuliaLang/IJulia.jl/tree/master/test is a 404 isn't it.

Something with xvfb maybe? I'm not sure what the best way to self-test would be here, it could get complicated.


Is this a good way to split on things besides spaces? Thanks for your previous feedback on this one.


Yeah I guess it is pretty verbose. I just wanted to make sure that for any falsy value it returned false.
That double negation looks pretty handy though.


@aaronjudd can you define "order" vs "sale" as mode values?


Yeah you're right that looks way nicer :)


We don't even throw an exception.

We can add a Logger.Trace(...) to report that particular dir isn't writable, but that's Trace at the most because it is not an error.


can we use the circumference of the wheels to measure distance?
Mr. Calvin said that the encoder can code for 1:47 ratio accuracy.
so if the circumference is 12 inches, then the distance traveled can be measured by a 12/47 inch increment.


I did it!


Great! Now waiting for the ST3 version update.


Initialize() is called only once when the Command starts - in this case of a default Command with no other Commands that would interrupt it it is effectively once per code run (until next "Restart Code"). Note this in case if you encounter and wonder why the P, I and D values of the PID controller do not update when you change them on the dashboard.


Relates to #254


I still don't get how to do this...
What is an enum? Also how do you state the states? I don't know how to set up the classes and the different methods for this. also objects. I am just confused.

Is there imports for this?


we don't but one get thrown, this is what I mean,

```
catch (Exception e)
             {
                 _logger.Trace("Directory {0} isn't writable. {1}",dir, e.Message);
                 return false;
             }

```


Not actually seeing the busy system... And how come the file was completely re-created?


@flowlo I notice you aren't a contributor to [this project](https://github.com/jcreigno/nodejs-mail-notifier/graphs/contributors). Perhaps you should submit a pull request?


In this first test we are modifying the public field string (Value): https://msdn.microsoft.com/en-US/library/ms173118(v=vs.80).aspx by using an accessor Get to read the value and store it as data


Does this only accept positive exponents?


@viktorklang:

> @ktoso I assume we had to solve this for the test that validated that an onError was issued on overflow?

It was solved there by using the callsCounter here: https://github.com/reactive-streams/reactive-streams/commit/f89509fd08c43a779a5eab2ce088d7a6642169e7#diff-4e925492f7c6dd36a00c2ff18714fa46L950

We talked today about what needs changing here but that was "from memory", and looking at it now I think we could provide a test which can still be useful, though needs some more thought (aborting the execution in case of synchronous publisher trying to really fulfil that demand...)

So thinking about the new wording... what we can do is:
* on each `onNext` signal `request(Long.MAX_VALUE - 1)`
* limit the number of onNext calls, similar to the now removed test
* assert that during a while of this "spinning" we don't get onError

I'll prep a draft in the morning, gotta run now :)


This also made the following test pass:
 [TestMethod]
        public void NodeEqualityWithNull()
        {
            SinglyLinkedListNode node1 = new SinglyLinkedListNode("foo");
            Assert.AreNotEqual(node1, null);
        }
 [TestMethod]
        public void NodeEqualityWithString()
        {
            SinglyLinkedListNode node1 = new SinglyLinkedListNode("foo");
            Assert.AreNotEqual(node1, "foo");
        }
 [TestMethod]
        public void NodeInequality()
        {
            SinglyLinkedListNode node1 = new SinglyLinkedListNode("foo");
            SinglyLinkedListNode node2 = new SinglyLinkedListNode("bar");
            Assert.AreNotEqual(node1, node2);
        }


1: M111 unrecognized component name: websubmit


Why would you even remove maven? I need one goddamn reason.


Simplified a bit in 0eb8e2a6782d398ba2fd709160e1f39b261ced0a, also got rid of the unnecessary barrier parameters on `mono_llvm_build_aligned_load/store`.


How did you generate your performance data.  I have been looking for a good way to benchmark performance, but haven't spent enough time to get one working.


I think you actually want to re-raise the exception and not trap it at all. See: https://github.com/cancan101/nolearn/commit/2f66de05083084567e04f2f5d2ab69fa517e50ff#diff-2cc3045bedfbedf10a7a6e3df96782a5R170.

If you trap it like you have, then hitting the stop button in IPython might not work.


84202576-DesSpecOA-PerPersonUsedWhenFlowArea (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1589 of 1591 tests passed)

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


Hi Kai,

No, I was agreeing that what I did was bad. I only did it because I saw
that's how it was done in CipherSuite.

In fact at first I did what you suggest, but then I undid it and
implemented what you now see.

I suggest that an exception may be better as I believe the callers
should be made to deal with it. I'd do it but unfortunately I'm running
out of time for the test fest.

Thanks,
Bill

On 23/01/15 02:01, Kai Hudalla wrote:
>
> Hi Bill,
>
> I think that we have a misunderstanding regarding the proper handling
> of unknown codes :-)
> The way that you have implemented it now resembles the way it is
> (still) implemented in CipherSuite. In both cases I think it should
> actually be done as it was originally done in CompressionMethod, i.e.
> if an unknown code is fed into the getByCode() method it should return
> null.
> In this case, however, returning the NULL compression method leads to
> the correct behavior during the handshake, i.e. the server will
> indicate the NULL compression method in its ServerHello message.
> However, I think that the selection of the NULL compression method in
> this case (code provided by client is unknown) should better be made
> explicitly in the ServerHandshaker.receivedClientHello() method
> because it is reasonable in that context but may be wrong in another
> context.
> I would also decrease the log level in this context since it is a
> totally expected (and easily handled) case that is not worth pointing
> out explicitly to operations personnel, is it? I'd rather log at
> Level.FINE or Level.FINER, i.e. a developer might be interested in
> this information when debugging the code ...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BillBennett/californium.scandium/commit/1af2c5c20ac77d17ee78f254e7f106ea26cdf8fc#commitcomment-9398287>.
>


but it already loads when user press enter. at least on chrome.


@dmyersturnbull  then this would suggest we need to set this up as a configurable behavior...


The idea behind the original `pdb.set_trace()` line was that it would give you a crude way to interrupt learning but to not lose all learned parameters.  The `pass` works similarly; code that called `fit()` could still store away the network's parameters if you wanted to inspect them.  I guess the workaround for you would be to hit ^C twice (unless this causes a problem with iPython again).


dis is so much code


Just added a file in the examples folder an then this stuff was also commited ?! don't know where it comes from. I explicitly added only the one file in GitBash...Does anybody know how to delete/revert this commit?


Your code has no comments.


@marksamman can you explain what is "cylinders" and the new possibilities with this changes? Thanks.


I'm sorry, I interpreted your first message incorrectly, Thought you said we should log the exception here instead of letting it be thrown up the stack. Totally missed the part about only the message.
In the end we were apparently saying the same thing.


axet betrifft doch eigentlich nur UI oder? Hab ich beim PR nicht drauf geachtet, könnteste bei Gelegenheit noch verschieben


You commented the commas.


This is robust against failure, but unpredictable to the caller.


Well, reference imported bears are not checked for whether they're imported, and since I will soon remove that functionality anyway, I will not add it here anymore.


Kind of override the previous commit, but eh. It got the job done. Everything has been tested.


Yeah I messed up here in the first commit ;)


It would be better to refactor this so the type checking and argument existence checking don't overlap.


The `^C` twice does work; that's probably good enough.


[TestMethod]
        public void NodeDefaultNext()
        {
            SinglyLinkedListNode node = new SinglyLinkedListNode("foo");
            Assert.IsNull(node.Next);
        }


@automatthew z-schema puts its flags to the schema objects when compiled, something like this should fix it:
https://github.com/zaggino/z-schema/blob/master/benchmark/tester.js#L44-L46


Cylinders are objects that can hold items. The changes allow you to move items to players and containers.


If we really want a 0.41 patch for this let me know and do not merge this, I'll base it against 0.40 instead and then merge that into master. My feeling though is that there isn't a strong demand, mostly developers would run into it.


Added hello


this does not strictly match the behavioral description.


I recommend using the lua hook for lua open.
It executes after all scripts have been executed, so then naturally all files have been loaded at that point.
It fires on startup and reload of engine. (basically just after all scripts have been loaded)


84202576-DesSpecOA-PerPersonUsedWhenFlowArea (JasonGlazer) - 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)


The goal of this test was to assert that the next node had the value of null.

I achieved this by modifying the accessor Get to read the value of the method on the node invoked (the node was foo).

Then using Set we wrote the value for the next node of the invoked node to be null.


This should be done through an addon. This is to keep the listeners as clean as possible. See the famous warning person addon and just adapt it for this :)


Sounds awesome :) Will this support 2d normal maps as well?


Other than the addon thing I commented on, it looks good :)


"a:" label could go away here?


6 is a magic number here. Might as well pass in random numeric that's not too huge.

Also, "numeric" includes floats. What then?


omg, finally


@Wild1145 I didn't re-create it, and the system works, I tested it. This did the same thing for my other commit.


and why shitstorm @nattofriends


@Hockeyfan360 I was referring to the diff. Its like you selected all, then hit paste. Its not a biggy but it screws with the git history.


If you have an encoder you can do that, but you might not have an encoder on all wheels.  An encoder will always be optimal, but you might be able to get away with just using time for now.


Nice utility, but maybe think of a less generic name ;)


in line comment


it evolves!


@Wild1145 I have tagged you like 10 times within 10m. Would you like me to make another commit?


fix this


I'll add that when I started with biojava, I was porting an algorithm from
biopython to biojava (we needed it in our web application). It did an
alignment which was marked as a global alignment. When I made it a global
alignment in biojava, then I would get markedly different results
(biopython looked like it was giving a result out which looked like it was
ignoring gaps in the ends, but biojava was penalizing end gaps). When I
converted it to a local alignment in biojava, I got the expected result
(however I had to change how I was processing the alignment output since
local and global disagree in that area). I tried to change the python
version to match, code-wise, but local alignment is much slower in
biopython for some reason (although it did eventually give the correct
result).
On Jan 22, 2015 6:42 PM, "Andreas Prlic" <notifications@github.com> wrote:

> @dmyersturnbull <https://github.com/dmyersturnbull> then this would
> suggest we need to set this up as a configurable behavior...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/biojava/biojava/commit/45235c07b7f2d056274565a021b86d3ee2c5f76c#commitcomment-9407488>
> .
>


@Wild1145 I didn't paste, I used NetBeans to commit. I only edited what I needed to.


Interesting, never heard of that - where can I find documentation about that? Is it public API? Also, that means Oliviers article linked to above is incorrect and needs to be corrected. Can you confirm that? Also note how the documentation also has an example with a global QMutex: http://doc.qt.io/qt-5/qmutex.html


The TraitInfos are distinct from the traits, so this probably wants to be renamed to `TraitInfo<T>`.


 [TestMethod]
        public void NodeSetNext()
        {
            SinglyLinkedListNode node1 = new SinglyLinkedListNode("foo");
            SinglyLinkedListNode node2 = new SinglyLinkedListNode("bar");
            node1.Next = node2;
            Assert.AreEqual(node2, node1.Next);
        }


This removes support for baseurl.

Baseurl was originally in #42, to fix #41

Is there any reason why this feature was dropped?


There actually is a default shutdown/kick message set by Vanilla. "Server closed."


@Hockeyfan360 If you have stuff to add then do commit. I will need to make some changes myself soon to do some minor fixing up where I screwed some local branch management up earlier.


@Hockeyfan360 I know, its a thing with Netbeans, its not a biggy, but it has a habbit of making the history all screwy.


Why is the commit log just `-m`?


ship it


The goal of this test was to read the next value.

We have established at this point for a list consisting of a single node, THE NEXT NODE'S VALUE IS ALWAYS NULL.  This is proven by the previous test still passing.

The Next method will return the Value field for the node immediately following the invoked node.


The class name probably should reflect that this is specifically for paths.


Question:

Is it possible for a single node to have multiple Next Nodes? If so, what is the expected return value for calling Next on such a Node?


[![Super Bowl](http://api.artsmia.org/images/120771/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120771)


[![Piano Babies](http://api.artsmia.org/images/120772/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120772)


[![Whale's Tooth](http://api.artsmia.org/images/120774/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120774)


[![Exhibition Furniture](http://api.artsmia.org/images/120776/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120776)


[![Exhibition prop items](http://api.artsmia.org/images/120777/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120777)


[![Indefinite Coda](http://api.artsmia.org/images/120779/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120779)


[![Cave #2](http://api.artsmia.org/images/120780/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120780)


[![Cave #3](http://api.artsmia.org/images/120781/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120781)


[!["Brussels" armchair](http://api.artsmia.org/images/96509/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=96509)


"To address the inevitable response: I don't think anybody actually holds
an expectation that GET parameters are private just because they're
using HTTPS. If they are, they should stop."


This doesn't really belong in the game assembly either, since this is pathfinding specific code.


Hi @KangOl,

forcecreate='0' is missing on the last currency rate XOF/CFA


This would be better as a property IMO.


snot a generator, I believe


この位置にこの処理が現れることが不思議なんですが、何故必要なのでしょうか。
（単にreply_list.txtの中身はUTF-8で記載という仕様にしてしまえば終わるような。
あまり文字コードの推定というのはやらない方がいいです。アテにならないので。
どうしてもやりたいなら、表示の直前ではなく読み込み部分へ移動した方が良いと思います。
『システム内部は常にUTF-8』という前提をおいた方がバグが出づらいので。）


Broke the >> and the (counter) in the title for me. Looks to be the same on meguca.


Your English is fine.  I see what you meant about the scrollbar, makes sense.

Regarding breaking other pages, try this (Chrome):

* Turn on "Show on non-code pages"
* Go to diff view page
* Click to another other page (e.g. Issues) => layout is broken


Problem is I don't know how to map over an array with promises executing
synchronously (execute 1st promise, then 2nd when first is done, ...)

Neither Promise.all _.map or Promise.map work as desired.

Any ideas?

On Wed, Jan 21, 2015, 6:19 PM Zoli Kahan <notifications@github.com> wrote:

> Nth?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/claydotio/clay-mobile/commit/50231c4270d5b7d5fd1a2c37cba778fcf13e0bc7#commitcomment-9389608>
> .
>


Fast filters are used to filter databases as well as items inside database such as tables and views. So this message is not correct for the latter. However we can distinct the former from the others with 'db_fast_filter' class.


WIP


Removed, thanks.


I am using this to learn Github


ok we are real


@jbaltazar this is good! I was just wondering, have you tried to use one of the mysql python ORM wrapper instead of writing SQL directly?


@iwillig what do you think of this?


Hmm?


If you have code that works to your satisfaction, go ahead and rebase to a single commit on browser-develop and do a PR. I don't mind rebasing again if the ground shifts, as long as the PR is has just one commit that encompasses an issue. Thanks!


kicking it back old skool. :baby:


I think we can just flip these around and remove the null check?

`"TRANSIT".equals(params().get("mode"))`


Hey Tanner,

I completed the new method (string to array) but don't really understand what you mean by feeding that method into the first method. Also, what do you mean make it look pretty?


Seems okay, but is it needed?


为什么这里不直接改掉  '(t=(' + affix + part0 + ')) != undefined ? ',


@michaelwehrley what is the purpose of this function?