That sounds good. I'll add it to my todo list.


Thanks @AdeebNqo , is there any way to make this backwards compatible as well?  For it to work with and without the -m -o options?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17284](https://teamcity.mesosphere.io/viewLog.html?buildId=146099&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:10


Resurrection Remix ROM is coming?


nit: move up and defer


ahahah oookey, so we are really "okey" :D


Done. I removed the blueprint.

Odd that it got added. Maybe I added xcshareddata before I updated the .gitignore.


Woot! :tada: :balloon: :tada:


@phhusson Ah, I see, you're right, thank you for explaining.

Okay, this makes sense to me, I see why it's necessary. That's a very good catch.

I think this is a race in the syscall handler and I'm going to see if it can be fixed, but as things stand now I agree this is needed.


@fritsch: http://paste.ubuntu.com/15200975/


@jef-n r comment above should probably be fixed too


How about !strcmp(fullname, "ui") || !strncpy(fullname, "ui_", strlen("ui_"))?


You could have just looked at the documentation tab on the forums XD


bytes.Equal()


It's `get_meta("displayname")`, to serve `vdirsyncer`'s own needs (generic metadata access).


LOL


Thanks for making the updates. Its looking good now. Will see what happens with the darwin fix PR.


Looks like a race between thread 1 that wants to destruct it and Thread 12 that still uses the underlaying libs and is controlled by this thread ...


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1569 of 1569 tests passed, 0 test warnings)

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


Do we need recursion? I mean, I don't think it should be valid to nest vdirs or anything like that. It could lead to performance issues.


:P


@fishbeck can you take a look at this addition to esp.ecm


What's the problem?


Call this method `newInstance`


Let's make sure this doesn't mess up any of our tokenization or completion (I recall a lot of fragility around detecting block types and it's impact on code folding and completion).


you mean there should be a function `setS11Edit(Mag, Phase)` and similarly for `S21`? The sequence `block signals`, `change`, `unblock signal` is repeated 3 times for S11 and S21...

Not sure I got which S{1,2}{1,2} duplicates you mean, sure the entire stuff could have been done differently, maybe using also `complex`, but that will be for after `0.0.19`, likely


Just curious if these removed dependencies get added to runtime dependencies?


No :'(


Have them use TDD (test driven development) to confirm that the code works as they write it piece by piece. This is more efficient that getting to the end only to find out that there is an issues somewhere in the code that needs to be fixed.


Hi,

this is the result of the changes by the perltidy merge. Please see #566 on how to fix this pull request.

Best regards,
 Lieven.


some of these are still needed as build dependencies.


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


We need to work on the feature branch now, make it work for both and then merge to master, so the ones that clone the master branch would also be able to use it properly


This local variable is unnecessary. Return the value directly in each brach of the `if`


Ok, I will take a look. I'll also look into the shadow things you suggested. Screenshots of the current state of things :) (Im not finished)

Im glad that ground units generally dont stack well in a hex...

Btw, some things that are on my mind:

1) MM doesn't seem to remember when I change keybinds in-game after restarting MM. Is that normal when running it from Eclipse?

2) The keybinds offer the option of instant-repeat for keybinds. I'd say this should be baseline for the WASD keys and not an option for anything else. I'd replace that with the option to have the key activate the particular thing on key-down and deactivate on key-up (not for all but for some of the keybinds). That would give us quick access to some display options like firing arcs and whatever, maybe C3 lines or a quick unit health display (another project I started to get rid of right-clicking through 10 units just to check their CT armor...). What do you think?

[cid:8bed21d3-9c0b-4790-888f-e840f08ee5da] [cid:89add68d-3df3-4217-bccd-e3ed0c1a7ef3]
________________________________
Von: Nicholas Walczak <notifications@github.com>
Gesendet: Donnerstag, 25. Februar 2016 08:29
An: SJuliez/megamek
Cc: SJuliez
Betreff: Re: [megamek] : EntitySprite (46144b7)

I was poking around and looking at the forks of megamek and I saw this branch. It reminded me, when I was working on hidden units I noticed that, when the EntitySprite is made translucent because it's not visible to any enemies (in isometric mode), all of its status messages are also translucent. This can make the statuses hard to read. While you're working on this code, it might be nice to see if you could fix that. (This section of the code really did need to be refactored too, so I'm glad to see you're working on it.)

-
Reply to this email directly or view it on GitHub<https://github.com/SJuliez/megamek/commit/46144b7e0633b016ab2c067415b5a237c2abcef3#commitcomment-16331128>.


There is a specific `f.submit` method that you can use instead of `f.button :submit`.


Addressing tripit/slate#463


all tests pass:
success: http://buildbot.bos1/builders/test-bitcask/builds/583


@ypkkhatri :

Thanks for you hard work! This is starting to shape up!

I have a few things I'd like raise regarding the tests:

- `Assert.assertTrue(false);` makes absolutely no sense. If you want it to throw an exception, or fail, this isn't the right way.
- Don't catch the exceptions in the test cases, unless you have to. Make the test methods throw the exceptions themselves, if there are such.
- Please, annotate all the `test*` methods with `@Test` and remove the `testLogger()` method. JUnit is smart enough to run all the tests, if they have that annotation.
- Import all the `assert*` methods that you need as static imports, please.
- Please, make sure that all files have end with a new line, because some parsers can otherwise break.
- I see that you are doing: `Assert.assertTrue(true);` after each change/delete/update/etc... Please, don't do this! Instead, create a few more classes in different packages under `src/test/java` and make each of them simply log output messages... and test this output like i showed you [here](https://github.com/strongbox/strongbox/blob/master/strongbox-storage/strongbox-storage-api/src/test/java/org/carlspring/strongbox/storage/resolvers/GroupLocationResolverTest.java). What you basically need is:

`
    @Rule
    public final ExpectedLogs logs = new ExpectedLogs()
    {{
        captureFor(GroupLocationResolver.class, LogLevel.DEBUG);
    }};
`

Then in your test method do something that logs and assert like this:
`
 assertThat(logs.contains("my log message"), is(true));
`

Also, when you store the `logback.xml` file, make the test, actually read the file and check that the changes you made were indeed written. Otherwise, there is no actual validation of that and this is very important.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1569 of 1569 tests passed, 0 test warnings)

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


I wasn't aware we had MessageRef but I will happily use it. Watch this space.


I can do some investigating to see what I can find.  You can go ahead and turn it into an issue.

Interesting it already exists.  Are you guys using bootstrap already?


Fortunately, the default behavior is to assume that the document is in Markdown mode, so this should work transparently with `block` chunks.

Some light manual testing does confirm that things work as expected with these rules embedded.


Good luck with LTC!


should be like this

    for i in 11 12 21 21:
        do_whatever( i _ stuff)

will try to find my example. might have never been published `:/`


Have you seen the comment about the workaround? https://github.com/xbmc/xbmc/blob/master/xbmc/network/linux/ZeroconfBrowserAvahi.cpp#L77 The mentioned avahi bug is fixed meanwhile. Maybe one can do it without a workaround (and the crash ;-) nowadays?


FixGSHPweirdOutputOfAsterisksEtc (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Yeah any non Hindsight specific sandboxes are in the process of being relocated to lua_sandbox/sandboxes/heka.  They are sitting in my work directory at the moment because I was hoping to pull in some additional code lying around  but got pulled away.  I will commit at least these today.


It's minor, but you could also make `users#create` available via `post 'signup'` (to match `get 'signup'` above)


Does this make target depend on any other build configuration parameter?


lol


I think the use of the `self.` prefix makes the code more readable and maintainable, not to mention future-proof. Since it was there originally, it would seem that this project follows the guideline of using the `self.` prefix even when not required for disambiguation, possibly for the reasons mentioned.

This is not a coding error, but a matter of style, and it up to the project to decide on its preferred style.


Can this be null?


Does nothing, copied as you do not read out accepted_protocols


ezt miért?


If I understand properly what this function does, I think the name and comment are not very descriptive. I would expect to see the phrase `lub` appearing somewhere. Basically it's job is to find some type `T` that `prev_ty` and `new_ty` can be coerced to?

Presumably this may require new coercions on the expressions already processed? I haven't really grokked this yet.


Instead of just saying "invalid precision", provide an indication of what the allowed range is.


@ruralocity  that was something that i did before, doing a little clean up in that div, but, @rachelannejames found a really weird bug related with that change, so we rollback the change a leave it as it was


Same here


What is the purpose of this method? I might be missing something, but there's no output and doesn't seem to do anything?


We can't call printk here since we are not in a Linux kernel thread context and we could race with a running Linux kernel thread. We should use lkl_ops.print instead.

Also please use a space between if and ( - Linux kernel coding style again :)


Minor nit, you could do options.merge(data, :verb => 'PUT') to make it a one-liner.


````javascript
var tagList = sidebar.substring(sidebar.lastIndexOf("<h5>Tags</h5>\n		<ul>") + 20, sidebar.lastIndexOf("<strong>Statistics</strong>")).replace(/_\(artist\)">/g, "_(artist)\" style='color:#A00;'>").replace(/_\(character\)">/g, "_(character)\" style='color:#0A0;'>").replace(/_\(copyright\)">/g, "_(copyright)\" style='color:#A0A;'>").replace(/<\/a> /g, "</a> ").replace(/"/g, "'");
````

Setting to tagList variable: gets the tag list on the side (located from `<h5>Tags</h5>` to `<strong>Statistics</strong>`), then injects CSS into tags with `_(artist)`, `_(character)`, and `_(copyright)` so it is the correct color.

````javascript
document.getElementById("tag_list").innerHTML = document.getElementById("tag_list").innerHTML.replace(/<ul>.*<strong>/g, tagList + "<strong>");
````

Sets #tag_list's inner HTML to the modified tag list


Hehe, that's exactly what I thought and the above sprunge does.


How about using `You gain extra movement for the current turn` as the subtitle? I have generally used the text from the PHB for subtitles.


Call this SIZE_OF_LONG_DECIMAL for consistency with the other SIZE_OF constants.


Any help you can offer is appreciated, but my gut says more and more this is not a bug with ember-cli-dynamic-forms. We are already using the ember-bootstrap addon.


Done.


wirklich?


found one example, which is very close.

0038bc557258bd79252d142198fdff21e7ed2c62


Fixed it.


redis desc is rejected_connections: Number of connections rejected because of maxclients limit, so I think it is a counter.


It seems that you forget a few occurences in [/crypto/rsa/rsa_lib.c](url)  mostly.


typo


any chance that you provide my a test build with that patch (running 14.04 lts)


You may also wish to change rand_len * 2 to rand_len in the ntru_rand_generate().


I actually agree in most cases! The use of self was actually removed in the
update to Swift 2 that I pulled from ayaka. I considered changing it back
but felt I would rather leave that up to the project!
On Thu, Feb 25, 2016 at 12:26 PM Reid Ellis <notifications@github.com>
wrote:

> I think the use of the self. prefix makes the code more readable and
> maintainable, not to mention future-proof. Since it was there originally,
> it would seem that this project follows the guideline of using the self.
> prefix even when not required for disambiguation, possibly for the reasons
> mentioned.
>
> This is not a coding error, but a matter of style, and it up to the
> project to decide on its preferred style.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rchatham/Polyglot/commit/ec9b5ad6e4f32609fb509b81b88be0c7daf54d34#commitcomment-16336900>
> .
>


IT is in the docs http://redis.io/commands/INFO


Will it be safer to check only for files which [were configured to be installed](https://github.com/coccinelle/coccinelle/blob/e284b4277e9b1eb8a08726f5862fd91bb7ec17e1/configure.ac#L452 "Using of the native software version?")?


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


so maybe the bootstrap addon is laying down that file before this.  we could probably try to reproduce with a simple cli project that imports the bootstrap addon and the dynamic forms addon


I still don't quite understand why, but when the second div is moved to its own line, we see that JS bug that @marko07 was showing us the other day. I expected it would be something more like a messed up selector, but it was just the line break. It seems like somehow the newline messes up the firstChild selector (which seems insane).


Okay, but how do you know which tag is artist, character or copyright? On boorus that provide classification tags are already colored, on basic boorus there's no way to know without creating a pre-set classification manually.


Nope - sadly I cannot provide ubuntu .deb testbuilds.


We should derive this from the max allowed precision. This is somewhat brittle, as it's encoding the same information in two different, unconnected places.


fast-forwarding 1.7 to borshop-integration-234-nem-bump-bitcask-to-1.7.4 = baa1b69b


Name this POWERS_OF_TEN


You can use **react-autobind** for `this.selectTab.bind(this, tab)`. See:

[https://github.com/OSBI/meteorite-core-ui/blob/development/src/main/resources/src/js/components/saiku/Login.jsx#L42-L43](https://github.com/OSBI/meteorite-core-ui/blob/development/src/main/resources/src/js/components/saiku/Login.jsx#L42-L43)


Ah ya, misread docs.


nvm


Addresses in
https://github.com/AppFolioOnboarding/Li_Image_Share/commit/63890d03f37464cde49d33c0b0b277fae68e671f

>


Would the HipChat and Null adapters also specify `worker(Connector, [<AdapterMod>])` in their trees?


You’re right.  Oops.  I’ll open an internal MR to get that fixed; thanks!


I will revert my fix as I could not reproduce the segfault with this "fixed" code. I still don't understand the original issue.


Sorry if I wasn't clear.

The problem is `setup.py sdist` produces `commissaire-VERSION.tar.gz` but `rpmbuild` needs `VERSION.tar.gz`.  Might be easiest to just add some verbage.


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


With this commit yes. But I'm working on rolling all of that into an adapter process; see discussion at the bottom.


Ah, front-end web development :dancers:


No?


I got the string for #tag_list (only the tag list as #tag_list includes Statistics), then if an a element looks like this: `<a href="..._(artist/character/copyright)">` it is replaced with this: `<a href="..._(artist/character/copyright)" style='color:#HexColorCode'>`


So, I think that the Hair translation works hard to make this structure for us, and now we are working hard to turn it back -- we can probably just change how the hair is defined.


:+1: on the above conversation.

I think following Elixir's `GenServer` example is a good design to consider. Namely, provide base functionality and do-nothing implementations of the behavior callback functions and allow `use`ing modules to override them.


AKA, sorcery.


Wäre es nicht besser wenn das ein ``int`` wäre, dann könnte man alles sortieren damit.


I like the 'sorcery' explanation!


Makes sense, and already done!


Kannst du beschreiben was diese aussagen? für mich würde es nur nicht geckeckt, gecheckt und unsichtbar geben?


@mbj I've added escaping now. I will add real escaping later. I am still doing some research on how implement it correctly (or any proper library for doing this).


Handling the macro to enable and disable KVO. It seems like something that should be configurable based on a flag, but I don't think that CocoaPods has a way to configure it.


I think that is better you use `_.defer()` of underscore. See:

[http://underscorejs.org/#defer](http://underscorejs.org/#defer)


Spawning check move to goTo function


Java/Ch 03. Stacks and Queues/Stack_Via_Queue/StackUsingQueues.java

Version control makes these obsolete


Ahh, OK I follow.


Related to https://github.com/careercup/CtCI-6th-Edition/tree/master/Java/Ch%2003.%20Stacks%20and%20Queues/Q3_04_Queue_via_Stacks


Did you intend to attach a screenshot?  It doesn't appear to have worked, or at least I don't see the option to view/download one.

1) How are you exiting MegaMek?  If you're killing it from within Eclipse, it won't save (the keybinds are saved when you exit the UI).  It should be saving your keybinds if you are exiting the UI though.  I'm not exactly certain why it wouldn't... Maybe check the logfile?  One thought would be that it's not running the jar file from the right location, but if that were the case you'd know it (it wouldn't find unit files, or the art assets, or config, etc.)

2) The main reason for the repeat option is to overcome the machine dependent key repeat timer.  The default machine behavior is to... press a key, wait a time period, and then start repeating.  This time can generally be set in the bios (or maybe the OS, but somewhere outside of MM).  By having an internal timer mechanism, MM can overcome this (and allow the user to adjust the values within MM).

You are right, WASD (or rather, the map scroll commands) are really the only commands that this is required for (as I don't think there's any other command that you really want to have repeated in rapid succession), but I was going for maximum configurability.

I could add support for having a key-down activate and a key-up deactivate.  I think it's been discussed before.  Not sure exactly when I'd get to it though.


Replace with +=


Done in the upcoming commit.


this isn't the formatting used throughout the project


[R/pols_501.R:196:1:](https://github.com/UW-POLS501/r-uwpols501/blob/8f918be70ddede54e0a2a3c1db8817f5e6b7db6f/R/pols_501.R#L196) *style:* **lines should not be more than 80 characters.**
```r
  template <- system.file("rmarkdown/templates/tufte_handout/resources/tufte-handout.tex",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


Very cool.


Yea, that pattern doesn't apply to a lot of things where folks aren't building from source.  I don't think there's a good way to model it and providing two versions of the source/library doesn't work because then we get collisions. I'd say expose it without the extra support enabled, and folks will just edit the source locally to turn it on when needed.  Maybe we'll stumble on a better pattern in the future.


Nope, sorry I don't catch this segfault - so most likely it's a race. Could you try with the above diff?


Addresses in
https://github.com/AppFolioOnboarding/Li_Image_Share/commit/1e10ef2865f574a224364576338e4aa4fd0888dc


I think you should log the brush for polylines aswell  as it is actually being used.


посмотри как указываются дркгие опциональные параметры. в таком варианте если не указан HS оно просто упадет.. или параметру скажи что он required или обрабатывай как с параметрами портов для http/avr


Nice! Though I wonder if having a separate script for those settings is necessary?

I did toy with the idea of making a bunch of bash function scripts so that things were more readable instead of using comments. Any thoughts on that?


Why only checkNotNull() in some of the classes that extend SignalAwaitingService?


That makes sense to me!


FixGSHPweirdOutputOfAsterisksEtc (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2158 of 2158 tests passed, 0 test warnings)

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


This `at_start` capability is definitely handy.


Think it'd be better if `rspec-targetize-file-name` took a second argument (extension), and the `cl-loop` form in `rspec-target-file-for` contained the list of extensions. Otherwise, you'd have to change the docstring here, at least.

Also, please try to keep the line length below 100 (preferably even below 80).


No. Constant *values* are supposed to be concrete - it is constant *expressions* that are allowed to be generic. This case plays with things a bit because the value is a duck-typed `()`.


FixGSHPweirdOutputOfAsterisksEtc (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


please modularize the code (i.e. small methods doing a single thing, SRP), your methods are too long and there's a lot of repetition (i.e. DRY, this seems like a swap (where you override the newly created `LinkedList`)), e.g. lines `43` and `59`


formatting


This is use only in a test, so make it package-private and make sure the test is in the same package.


I think we should not merge "known to be bugged" code.

If you use this library right now, without escaping: We end up in having a major XSS vulnerability. The value needs to be escaped prior to merging this.

So maybe you should close this PR for now and first add a function that does proper string attribute value escaping, than reopen this one on top of the PR I'm proposing?


It should, fixed. Thanks for catching that.


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


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


Well, if it were me, I'd leave some comments in here about the steps in the computations.  That would be really handy for someone (like me) who's just trying to understand how the algorithm works.  But it might also be handy for you six months from now if you need to do some maintenance in here.  But I'll leave it up to you.


JSON doesn't allow trailing commas in objects.


Could you add a test for this case? Something based on https://github.com/rust-lang/rust/pull/31824#commitcomment-16263266


ok.


It seems like a debug line left there by accident, maybe even by myself.


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


nice charset!


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


Got rid of this big wad of nasty looking code. All of these helper functions can now be specified with haxe.macro.Compiler.includeFile.

I broke them apart into individual helper methods. It makes them easier to reason about, and many of them pertain to specific functionality (e.g. the bit methods), so I can attach them to the init methods of those externs.


We should probably make the notation here consistent with `switch_register`.


apimachinery.GroupMeta calls this RESTMapper, so I was divided between the two. I am making it Mapper to at least be consistent in this package.


Well, they have different use cases. For cases like this, where the capacity is an optional thing, default values would obviously make more sense. Overloading is useful at times, but can be argued to introduce additional mental overhead. With generics it should be even less necessary. One thing we should probably consider, that would make it even less neccesary, is if/how to do specialized implementations of generic functions


Sure! (Sorry for not noticing your comment until now)


Please revert this change.

If someone did something like this:
```python
from pagarme import *

try:
    # anything
except PagarmeTransactionError:
   # do something
```
will get a `NameError`.

This kind of change should be in a medium or even in a great release, never in a minor.


hi, can you provide the libraries you used on this one?


@jgcaaprom you are damn right! It was ugly before, it's uglier now.
Is there anyplace where I can get the raw drawables so I can put them another color?


Done. Calling it `validFullyQualifiedKind`


Is there another way of getting output from a lua script if it doesn't have "print"?


Ah, yes it did. I was having issues getting zest.releaser working with the old setuptools and was poking around at possible fixes. The versions.cfg update during the release must have grabbed that.

I'm torn on it. We've done our best to keep the old pin, but it's oooold at this point.

> On Feb 25, 2016, at 2:55 PM, Maurits van Rees <notifications@github.com> wrote:
>
> @esteele Was this big setuptools update intentional or did it accidentally slip in?
> Jenkins is still happy.
>
> It thought using setuptools 8 and higher requires zc.buildout 2.3 or higher. But it might just be that only zc.buildout 2.0-2.2 has problems with such a setuptools, and that our 1.7.1 works fine.
>
> —
> Reply to this email directly or view it on GitHub.
>


The opts.host option should be used here for consistency. It is a security provision. It is so you can set the host to 'localhost' (or some other ip if you have multiple network interfaces) so other people on the network can't access the server.


@pscollins @phhusson one way we can fix it is to change the API to create both the kernel thread and host thread:

 struct bootstrap_syscall_thread_data {
   struct lkl_sem *sem;
   void (*fn)(void*);
   void *arg;
 };

int bootstrap_syscall_thread(void *arg)
{
  struct boostrap_syscall_thread_data *data = arg;

  __lkl_create_syscall_thread();
  sem_up(data.sem);
  data->fn(data->arg);
  __lkl_stop_syscall_thread();
}

int lkl_create_syscall_thread(void (*fn)(void*), void *arg)
{
   struct bootstrap_syscall_thread_data {
     struct lkl_sem *sem;
     void (*fn)(void*);
     void *arg;
   } data;

   lkl_ops.create_thread(bootstrap_syscall_thread, &data);
   lkl_ops.sem_down(data.sem);
}


I suppose one TODO is to deprecate the respective externs for these methods on the lua versions we support.


Changed to supportedGV (since I am also including version now).


FixGSHPweirdOutputOfAsterisksEtc (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2120 of 2120 tests passed, 0 test warnings)

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


Open - Offener Task, hier wurde noch gar nichts gemacht
Progress - in Bearbeitung, Task ist in Arbeit
Ready - Abgeschlossen, Task ist vollständig
Archived - Archiviert, Task wurde "gelöscht", bleibt aber für die Historie bestehen

deine letzte Frage verstehe ich nicht

> für mich würde es nur nicht geckeckt, gecheckt und unsichtbar geben?


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


Did you test this on a collection?


I believe the opts.host parameter should be passed into server.listen above. You can see the 'host' option in ws is ignored if the server option is passed in: https://github.com/websockets/ws/blob/master/lib/WebSocketServer.js#L62-L75


> so the setter is invoked during the anarchical process

hehe the `anarchical` process... sounds perilous ;)


I see the distinction you are drawing, but I'm not sure how important it is. It's certainly true that monomorphic values are a useful subset of expressions -- but it seems to me that they are basically "fully reduced" expressions, and that we might meaningfully talk about "values" that are not fully reduced.

More concretely, how would you prefer to see this piece of code changed? Not re-using `Literal::Constant` but rather `Literal::ConstantExpr` or something like that?


Hope so :)


Change read.csv -> read.delim then you can drop the `sep` argument. Also, change `res_v <-` to `res_v =`


Kann ich das nicht auch mit der TaskList?


Ich habe gedacht das wir so wie eine todo liste ein kästchen hat welches man dann abhacken kann, das mein ich mit gechecked.


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


How difficult did you find this (out of 10)? 8
Was there anything that you struggled with? Yes, I couldn't get the accounts to not go into negative values. And I'm struggling a little with 'return' - when and where to use it.
Is there anything that you'd like some further information on? If you could help me with the above that would be excellent!
Roughly, how long did it take? ~8 hours


:laughing: :dancers:


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


here it is : https://groups.google.com/forum/#!searchin/pupil-discuss/gstreamer/pupil-discuss/HsZeOahjaOQ/wLehTEeKT4oJ


enableExpandAll doesn't work under 3.1.1 yet? see example here, the expand all icon still show there.

http://jsfiddle.net/wyx2000/ysmg1qkt/80/


Why not to give some description string unconditionally? It may be used somewhere else in the future.
E.g. SelfTy - "self type", Local - "binding", PrimTy - "type", TyParam - "type parameter", Upvar - "lambda capture", Label - "label", Err - "unresolved item".


Since I merged the two patch pull requests into master, I ran a rebase for
my working branch. This is the new commit
https://github.com/AppFolioOnboarding/Li_Image_Share/commit/c9404e607ac1eebf56e6ed4ceebb70ab23853260

Li Zhang

Software Engineer | AppFolio Inc.
P: (614) 648-2368
li.zhang@appfolio.comwww.appfolioinc.com

On Thu, Feb 25, 2016 at 12:25 PM, rmacklin <notifications@github.com> wrote:

> It's minor, but you could also make users#create available via post
> 'signup' (to match get 'signup' above)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AppFolioOnboarding/Li_Image_Share/commit/7502b6b084525f4f7c4bff445570f0a92e6e1f05#commitcomment-16336891>
> .
>


I would like to use https://github.com/expressjs/serve-static instead of the full express suite. I'd like to be conservative in how many/large our dependencies are. This won't stop me from merging the pull request though.


Change `function(x) as.numeric(x)` to just `as.numeric`


OK, I think I've managed to convince myself; thanks.


Did the encapsulation refactor.  Not entirely sure I like it better this way, TBH.

https://github.com/ddfisher/typed_ast/commit/6c9686338712e57a29858ec0042e00f4f3d45851


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1335.01c67125c50fe5645b125a0bebf341d19dea219d](http://build.palaso.org/viewLog.html?buildId=69678&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1020, ignored: 12 Build time: 00:06:29


I found a typo in the previous commit message, so I rebased this commit.
The new commit is
https://github.com/AppFolioOnboarding/Li_Image_Share/commit/0a24146a13a2e6e0397cdf14deeb1dfda074d829

Li Zhang

Software Engineer | AppFolio Inc.
P: (614) 648-2368
li.zhang@appfolio.comwww.appfolioinc.com

On Thu, Feb 25, 2016 at 12:23 PM, rmacklin <notifications@github.com> wrote:

> There is a specific f.submit method that you can use instead of f.button
> :submit.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AppFolioOnboarding/Li_Image_Share/commit/40f2891888acca7d862a44afe649ec5ae2b13ff6#commitcomment-16336848>
> .
>


I'm with it! Defer to you on this one, was just trying to get something going quickly.

And I went a little bad boys here and worked directly on a branch instead of pull request. :/


Would you change `<-` to `=` in this section


Nitpicking: newline before the if?


I normally associate `performance` as speed. But you use it in terms of bugs. Thoughts?


cool story bro


Maybe also include the error message here, like in open tracking


+1 @mbodock


Oh.. Yes I understand. I will try to modify and keep the format intact.


poprawione


Wait, why don't we want to zero fat pointers?


Nice


Any chance this will be released soon? Thanks!


Will address that with a separate Jira.


Is there a point in having a default value? The function only has one caller.


I disagree. Code should too. It's very annoying when code starts flying off the screen. Maybe we can exclude `raise` clauses and code that does not need to be read completely to be understood (like string literals), but stuff like if conditions and math expressions should be split.


Is it worth mentioning `data.matrix`. This function converts a data frame to a numeric matrix


The downside is that `stats::filter` has been masked.


FixGSHPweirdOutputOfAsterisksEtc (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1012 of 1012 tests passed, 0 test warnings)

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


OctreeTestEioMathDiff (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2157 of 2157 tests passed, 0 test warnings)

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


checkVexAccidental_ note = note.accidental /= Natural


if not self.players_settled:
...

Если это действительно bool


@ErwanAliasr1 Please wrap this in an `%if 0%{?fedora} || 0%{?rhel}` conditional to ensure it doesn't break SUSE builds.

@branto1 Thanks for bringing this to my attention.


I would take this:

"Clocked"

"16MHz (default)"
"8MHz"

You'll also need to uncomment all the lines for them to be effective.


Typo: "we have *a* large ....` However I would suggest deleting large. Since 4GB isn't that large.


This UUID value will change on every render . It looks like the plugin will still work since the value only gets written to metadata once when enabling tracking, but it might be better to move this just for clarity


What do you think about losing these conditionals?  They don't serve anything except to prevent a user from calling this manually when they probably shouldn't - it that reason enough to keep them?


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


For the same reason you do not zero-out references to things.


Just my impression: I do not think that it is a nice solution to store a reference to communicator in `UTETeachIn` class. It could be provided as an argument to `send_response()` instead.

But even this is not nice, IMO. I do not think that a `Packet` class should autonomously send radio packets. I think this should be handled by the `Communicator` class instead.


@lcw Dear AMD: Something to be proud of.


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


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


Alternative in point-free style:

    map (checkVexAccidental <<< fst) <<< _.note


OctreeTestEioMathDiff (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Well, it's slower to do `"my name is " + "John"` than it is to do `"my name is John"`, so readability should never interfere with performance.

I feel like it's better to leave code length up to individual coder style. If people want to make it span multiple lines, that's fine, but if not, that's fine too.


Another issue with the lance: MakeThing error: PsychicShockLance is madeFromStuff but stuff=null. Assigning default.

I don't think that lance should be made of stuff, btw:)


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


OctreeTestEioMathDiff (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2125 of 2125 tests passed, 0 test warnings)

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


OK one more try, this time not inline but as an attachment...

________________________________
Von: Nicholas Walczak <notifications@github.com>
Gesendet: Donnerstag, 25. Februar 2016 12:43
An: SJuliez/megamek
Cc: SJuliez
Betreff: Re: [megamek] : EntitySprite (46144b7)

Did you intend to attach a screenshot? It doesn't appear to have worked, or at least I don't see the option to view/download one.

1) How are you exiting MegaMek? If you're killing it from within Eclipse, it won't save (the keybinds are saved when you exit the UI). It should be saving your keybinds if you are exiting the UI though. I'm not exactly certain why it wouldn't... Maybe check the logfile? One thought would be that it's not running the jar file from the right location, but if that were the case you'd know it (it wouldn't find unit files, or the art assets, or config, etc.)

2) The main reason for the repeat option is to overcome the machine dependent key repeat timer. The default machine behavior is to... press a key, wait a time period, and then start repeating. This time can generally be set in the bios (or maybe the OS, but somewhere outside of MM). By having an internal timer mechanism, MM can overcome this (and allow the user to adjust the values within MM).

You are right, WASD (or rather, the map scroll commands) are really the only commands that this is required for (as I don't think there's any other command that you really want to have repeated in rapid succession), but I was going for maximum configurability.

I could add support for having a key-down activate and a key-up deactivate. I think it's been discussed before. Not sure exactly when I'd get to it though.

-
Reply to this email directly or view it on GitHub<https://github.com/SJuliez/megamek/commit/46144b7e0633b016ab2c067415b5a237c2abcef3#commitcomment-16337256>.


`$exceptionID` is not replaced in here.


This isn't returning a time stamp.  Its the elapsed time.


@LisaMc I noticed the giant fingerprint icon on the dashboard and was about to fix it then discovered you already beat me to it. Not sure how that happened...


Can you make pbcore an optional dependency?

I would perfer pbcommand to not have a hard dependency due to it's heavy h5py and pysam dependencies.


Done & rebased against master.


not is on content/data/murder?


UnitConvInIDDforIDFed (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2119 of 2119 tests passed, 0 test warnings)

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


Perfect! Config should be something like this in my `config.cson` right?

    markdownGrammarScopes: [
      "source.gfm"
      "text.md"
      "source.madoko"
      "text.plain"
    ]


Some syntactic hints to consider, especially as an alternative to the explicit lambda notation with `\... -> ...`:

    extractAccidentals = map _.accidental <<< _.note


Dedicated servers no have IsListenServerHost e.e


This must be wrong. The type is `unsigned int`


Should we use lkl_printf("dump_file lkl_sys_open %s: %s\n", path, lkl_strerror(fd)) so that the filename is visible as well?


Should we use lkl_printf here, instead of printf?


This seems redundant, we already have the file output and if we don't get any error message it means that the dump completed successfully.


Same here, we can use lkl_printf to show the dir name.


I still don't think this help much. If mount/mkdir crashes you still don't have enough information from these prints to figure out what went wrong. It is also probably an internal issue and not related to the program.

Sorry to be so picky on this, but I would prefer to not have the success prints. We can certainly review this later if we run into a use-case where this is helpful.


Technically correct, but it is what IEspContext currently calls a time stamp. I considered a name change, but retained the terminology so someone transitioning from the old interface to this one might more easily understand what to expect.


One other thing I noticed:

"swupd_download_file" is a function pointer, so printing its value introduces a compiler warning; did you want to print the value of variable "filename" instead?


This seems to be used specifically for turning the string representation to the object representation. Given that, it doesn't seem to make sense to expose the notion of "unscaled value". You could move this concept into DecimalType as an implementation detail and rename the `DecimalType.unscaledValueToObject` method to `parse(...)`


edit


@kiljacken Yeah I agree that they are pretty much redundant with generics. If I recall correctly I think @MovingtoMars didn't like the idea of default parameter values or maybe overloading methods (I forgot which one)... ?


Do you mean wrapping the import in try/except, or changing one of the requirements txt files?


I would be +1 on having newer setuptools and zc.buildout 2.5.0 (new zc.recipe.egg to match) plus the bootstrap.py of course.

Some info in the release notes would be good. Something like this:

We have updated zc.buildout and setuptools to more recent versions in versions.cfg. If that causes problems for you, easiest may be to override the version pins to the ones previously used. In your buildout.cfg file, add this:

[versions]
setuptools = 0.6c11
zc.buildout = 1.7.1
zc.recipe.egg = 1.3.2


Java/Ch 05. Bit Manipulation/Q5_07_Pairwise_Swap/Question.java

* We can reuse the mask, if we apply it before and after the shift
* We don't need unsigned shift (`>>>`), if we choose a mask in the way that we mask after right shift


So going forward, to replicate dialogs over the binary interface, you must throw the /b flag in modparam?


storing it as binary number, delimited in 8 bit chunks might make it simpler to read than hex or decimal


But `Box<[T]>`, for example, is also a fat-pointer...?


need to implement the Image Capture API and the point calculation on images


@mbj I think it's not bugged code. My plan is to wrap the initialization of HtmlNode in functions like this:

```
function p(attributes, ...children:HtmlNode[]) {
  var someRealEscapingStrategy = new SomeRealEscapingStrategy();
  return new DefaultNode('p', attributes, children, someRealEscapingStrategy);
}
```
Consumers of the api should not (but can) use DefaultNode directly.


(And (sadly) still built-in to the compiler, afaik.)


var red = [0, 100, 63];
var orange = [40, 100, 60];
var green = [75, 100, 40];
var blue = [196, 77, 55];
var purple = [280, 50, 60];

var myName = "MakoWebDev";
letterColors = [blue, blue, blue, blue, black, black, black, black, black, black, black]
if( 1 > 0) {
    bubbleShape = "circle"
}
else {
    bubbleShape = "square"
}

drawName(myName, letterColors);

bounceBubbles ()


Gets rid of extraneous empty line in REPL.


from CodeCademy, makes bubble name bounce


Using an enum here is awesome. Just learned about the `indirect` keyword. Neat stuff.


Is this logic copied from somewhere?


Typo: "guaranteed"


It's not necessary to add `+` to have multi line strings.

It can work like this too:
``` ruby
str = 'this is a multi-line string'\
  ' using implicit concatenation'\
  ' to prevent spare \n\'s'
```


ping @tarruda could you update the version at PyPI? (or if you prefer, I could take maintenance of the PyPI package)


But I don't want to document whole package here. There are too many other files. It's related just to this module. Package documentation would be nice though.


That won't preserve code uniformity. IMO all the code should at least _look_ uniform.


Clearly a derp.
Had the same on the op2 long ago and corrected it.

Seems I forgot that commit or did not adjust it when porting it to the opx.

Thanks for the hint, should fix that by tomorrow.

Andi


I don't think poly-lines would use the brush.  Lines should only use the pen.  Only shapes like rectangles, polygons etc would use a brush for filling purposes.


Good points, and I agree with you.
`Communicator` should indeed handle all sending and receiving, however I'm not sure if it's possible to implement without a reference at the moment. As the implementation is currently, the base `Communicator` doesn't implement any kind of loop. This makes it difficult to implement any kind of "packet interception" to the `Communicator` (I'd like all communicators to behave the same).

One option would be to call something like `Communicator.send_responses()` on each loop in the implementations. This would then check, if there's responses to be sent and sent them.

Another option would be to re-structure the communicators to implement a `SerialCommunicator.loop()` or something like that, which would take care of the implementation specific behavior, while `Communicator.run()` would implement the main loop. In this implementation, `SerialCommunicator.loop()` would return a list of new packages to be parsed, and `Communicator.run()` would take care of handling them (mainly putting to `receive` -queue), sending a response as needed.

Raising an issue about this for further discussion.


Will do


That is already done - see the last commit 21859f219 or the overall diff.


Adicionalmente con Martín le agregamos a ese script (para que funcione de
template para varios casos) la posibilidad de añadir código custom a
ejecutar sobre el objeto "DataCleaner()" antes y después de la aplicación
de las reglas. Son pasos de limpieza que puede haber codeado el cleaner en
forma específica para ese dataset.

El 25 de febrero de 2016, 13:36, Carolina Hadad <notifications@github.com>
escribió:

> Y una cosa que hizo beni y está buena es poner un command line al final,
> asi podemos correr todas las limpiezas de nuevo si hay algun cambio en la
> librería antes de publicar los datastes.
>
> Acá hay un ejemplo:
> https://github.com/gobabiertoAR/datasets-portal/blob/master/audiencias/cleaner-audiencias.py
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gobabiertoAR/datasets-portal/commit/a232a95df0cb3e1c01b5fbbef0db3d6d6ebee56a#commitcomment-16331297>
> .
>


It's a bad idea to interleave global variables with function, which will lead to unmanageable complexity quickly. Actually, global variables should be avoided as much as possible.


Nope, did not measure it. Have no equipment for it, but suspected something similar.

Personally I don't care so much. I use it rather to charge slower than normal than faster. This works fine.
I prefer battery life span over quick charging.
Maybe we can find and remove the limitation after more investigation...

Andi


Nitpicking: Fingolfin has left the project now, so this comment should be updated


**are you using walk as well for processing all the files in one directory ?
**
def get_files():
    files = []
    for root, dirnames, filenames in os.walk('/tmp'):
        print root, dirnames, filenames
        files.extend(glob.glob(root + "/*"))
    print files

# string to date
from datetime import datetime

def str_to_date_12():
**This is not a recipe. The recipe should deal with converting a str date to a date, nothing about lists. This is not a recipe
A recipe is

def str_to_date_12(str_date):
    new_date = datetime.strptime("Jun 1 2005  1:33PM", '%b %d %Y %I:%M%p')
    return new_date

BTW, don't use reserved python words as variables. You have just screwed the date module.
**
    dates = []
    dates.append('Jun 1 2005  1:33PM')
    dates.append('Aug 28 1999 12:00AM')
    for d in dates:
        date = datetime.strptime(d, '%b %d %Y %I:%M%p')
        print date

def str_to_date_24():
**Same**
    dates = []
    dates.append('Jun 1 2005  13:33:04')
    dates.append('Aug 28 1999 12:00:06')
    for d in dates:
        date = datetime.strptime(d, '%b %d %Y %H:%M:%S')
        print date

# string to timestamp

def str_to_timestamp():
    import datetime
    import calendar
    date_text = "13SEP2014"
   **Dont use reserved words as variables -date-**
   date = datetime.datetime.strptime(date_text, "%d%b%Y")
    print(date)
    print(calendar.timegm(date.utctimetuple()))

# datestamp to string
def datestamp_to_str():
    import datetime
    **Are you serious ? int("1386181800") ???? like str("this is a string") ???**
    temp = datetime.datetime.fromtimestamp(int("1386181800")).strftime('%Y-%m-%d %H:%M:%S')
    print temp

# make script executable
###       chmod 777
**Not really. You are giving all the possible permissions, not just executable. The executable flag is only the 3th bit of each number. And still wrong. try to type "chmod 777" on your terminal. The proper recipe is: chmod +x <file>
**

# python executable
# /usr/bin/python
**Wrong. it is #!/usr/bin/python**

# get size of directory
**Not a recipe. You already have a recipe for getting all the files in a directory.
You only need a recipe now to get the size for one file os.path.getsize(<fullpath>)**
def get_dir_size():
    import os
    total_size = 0
    for dirpath, dirnames, filenames in os.walk('/Users/boyanmihovski/Documents'):
        for f in filenames:
            fp = os.path.join(dirpath, f)
            total_size += os.path.getsize(fp)
    print total_size

# rename file
def rename_file():
    import os
    file_full_path = '/tmp/test'
    new_file_path = '/tmp/testa'
    os.rename(file_full_path, new_file_path)
    **Next line is not part of the recipe, is it ? It could be a different one: test_if_path_is_a_file, for example**
    print os.path.isfile(new_file_path)

# execute command
**What about os.system("/bin/ping -c2 www.google.bg") ? Too easy ?**
def execute_sys_com():
    import subprocess
    subprocess.call(["ping", "-c 2", "www.google.bg"])

**Rest is not bad**


Merged here.


я не уверен что это нужно


weAver_resources are the zip file contains the icons and other stuffs required to the application


sourceCompatibility = 1.6
https://github.com/Codearte/jfairy/blob/master/build.gradle#L27

but java.util.Optional is a java 8 class. :(


Still don't think see it.  I think it's because you're replying via email.  You could either use the GitHub web interface and attach it, or you could upload it somewhere like imgur.com and link to it.


Follow the instruction at:
https://azure.microsoft.com/en-us/documentation/articles/storage-ios-how-to-use-blob-storage/


They were there in the original code, so I left them in. Preventing users from calling was the only reason I could think that they were there, as well.

I'd say leave them in. They don't cause any problems and they sort of document the purpose of the code if you don't actually look to see where it's called from (even though you don't have to look far in this case).


Okay, great, thanks for taking an additional look!

On Thu, Feb 25, 2016 at 3:24 PM, Kevin Ushey <notifications@github.com>
wrote:

> Fortunately, the default behavior is to assume that the document is in
> Markdown mode, so this should work transparently with block chunks.
>
> Some light manual testing does confirm that things work as expected with
> these rules embedded.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rstudio/commit/fbe48e6fb8440344b703bb2050cfde3fd3e23189#commitcomment-16336869>
> .
>


FixGSHPweirdOutputOfAsterisksEtc (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1570 of 1570 tests passed, 0 test warnings)

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


OctreeTestEioMathDiff (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


J'ai ajouté le support pour des dictionnaires (qui permettent aussi de faire des tableaux du coup). Les différentes instructions pour cela ne sont pas encore documentées dans le guide mais cela ne va pas tarder. En attendant, regardez les deux exemples ératosthène.orv et ératosthène2.orv dans le répertoire exemples qui illustrent comment les utiliser (ce n'est pas très compliqué).


Currently you can use this API: And inject a value like `" other="attribute-value"` to cause XSS. So its by definition (if merged) adding a bug.


Is that one more nail in the OpenCL coffin?


As you can see it uses the brush for the fill option. When I implemented it I didn't set that at first, resulting in a black polygon so I guess it is necessary, at least for svg.


I can see the usefulness of these convenience methods but they make the abstract interface a bit less clear.
And they don't help 99% of our code which uses JLIB primitives rather than std.

Can you move them out of the interface and into another header?  having to do append<valueType>(txsummary, key, value) rather than txsummary->append<valuetype>(key, value) isn't that bad.


Not sure - probably -there is some other logic I want to refactor too… I'm just resisting temptation right now.


Fixed.


@eliotsykes - this should've gone in a separate branch, I guess, but here is an attempt to rewrite template_spec to eliminate the primary key dependencies. refactoring for guest specs on this branch also.


Inside the `Input` struct, there's a named variable called `Source` without a json tag.
Without an example of the json I'm not sure if you should be adding a json tag to the variable like so:
```go
type Input struct {
 	Source  Source `json:"source"`
```
or just embed the struct
```go
type Input struct {
 	Source
```


Well it doesn't say it only does one thing. And that'd be duplicated code then.


Thanks, @piotrpietrzak can you change this?


Shouldn't this be `TIMESTAMP`


same here


Yep. Any ckeditor buttons that use popovers (like the Accessibility Checker, doksofts google maps and a few others) will now allow access to text fields in the popover.


This allows the Text widget's "state" attribute to be in "DISABLED" mode.  If the "state" mode is "NORMAL" the chat window is editable (i.e. writing/deleting text from the screen).


Whoops .. this should be changed before merging - thats something left from testing and it should have not been pushed :P


I think "isinstance(nd.op.scalar_op, scalar.Add)" is better. It is about 3x faster:

```
In [7]: timeit a.owner.op.scalar_op == theano.scalar.add
The slowest run took 13.08 times longer than the fastest. This could mean that an intermediate result is being cached
1000000 loops, best of 3: 747 ns per loop

In [9]: timeit isinstance(a.owner.op.scalar_op, theano.scalar.Add)
The slowest run took 35.28 times longer than the fastest. This could mean that an intermediate result is being cached
1000000 loops, best of 3: 230 ns per loop
```

This don't block this PR.


Do we need to update `TestHiveFileFormats` also?


Hello Daniel,

Thank you for your reply. Now I understand the problem.
So why not doing the test like that:
list<Profile> prof = [Select id From Profile where Name='Standard User'];

        User u = new User(
            FirstName = 'John'
            , LastName = 'Doe'
            , Username = 'john.doe.test@testcase.com'
            , Email = 'john.doe.test@testcase.com'
            , IsActive = true
            , Alias = 'jdoe'
            , TimeZoneSidKey = 'GMT'
            , LocaleSidKey = 'en_US'
            , EmailEncodingKey = 'ISO-8859-1'
            , LanguageLocaleKey = 'en_US'
            , ProfileId = prof[0].id);

        insert U;
system.runAs(U){ ..... }

Best Regards,

David

2016-02-25 20:06 GMT+01:00 Daniel Stange <notifications@github.com>:

> Hi David,
> you're right - the time stamp expects that the time zone of the running
> user isn't GMT - 3 or anything that makes the date to turn into anything
> other than the condition. It should actually verify the time stamp versus
> its UTC value (or convert results before assertion).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dstdia/stripeforce/commit/a9ca799067ce1ae89b4215f1c34dab58474ed2d9#commitcomment-16334993>
> .
>


Shouldn't this be `DATE`


Honestly, no. CL is the right thing as an abstraction. It lives. I'm OK with short-term stopgaps to paper over temporary, um, viability deficits. But I refuse to back off the long-term right answer. CL is the only thing that has half a chance as a non-proprietary compute abstraction. So until all the proprietary shit dies off, we'll have to make do.


Also, did the main reader code already support date, or do we need to update that also?


well, it looks like i am too late to prevent you from merging, but this change should be reverted again :P this has been mistakenly pushed.


Right. Need a better solution for this.


No, but we could have:

```
somedir
├ blah
│   ├another_dir (not a vdir!)
│   │   └ dir that's actually vdir
│   └another_dir (not a vdir!)
│       └ dir that's actually vdir
└ yet another vdir
```


I don't know why this push decided to remove extra whitespace on blank lines, but that shouldn't be an issue.


Have both `@task.creator_id = session["user_id"]` and `@task.assignor_id = session["user_id"]` seems redundant. Can you describe why both would be needed?


I'm not a php7 master, but I am 70% sure that "listtag" is php5.6 only, and "EnumTag" is what it is supposed to be changed to for php7...


Then do a
```Lua
hook.Add("PlayerNoClip", "SuperAdminNoClip", function(ply)
	return ply:IsSuperAdmin()
end)
```
in a custom addon yourself, jeez.


唐鳳：教育部來信，174行的教育部發言，要改成：

1.建議調查的對象是有系統性跟代表性的抽樣，應該要從後台分析，分析哪些人曾特別關注過哪些議題，就其關注的議題去調查。
2.問卷調查的部分，有很多題目是回答了哪個就要跳到哪題，沒仔細看會做錯，建議能比照線上適性測驗方式，自動反應跳題。
3.問卷的題數過多，建議針對被調查者曾關心過的議題來區分詢問。
4.問卷題目有許多的答項序號跳號，以及有許多錯漏字現象，建議再全盤檢視。
5.有關詢問「參與過哪些議題」的題目，建議列表供勾選，不要讓填答者還要去回憶才能作答，同樣的第九跟十六題，也建議列表說明有哪些議題，俾供勾選。
6.關於詢問被調查者每天上網幾分鐘這１題，因網路的使用已是現代人生活的一部分，建議讓填答者用小時來回答會比較恰當。

--
對不起我不知道怎麼直接改QQ


check for error


@MasterWolf i know ^^


We might want to consider *not* using a one-time binding here. This won't update if we make changes to the date from the outside. The consumer can always bind to the dateTimeStatic component with a one-time binding, but if we do this we can never use it without a one-time binding


What I'm missing is exact line numbers in output.


Not sure that I like it.
It is valid to do this.
For example, I might be overwriting a value with the same value.


Ok! Lots of idea going around, but I think we are on basically the same page about everything.

I suspect that adding a blank header to a child section would make the child section's header simply blank. And it wouldn't inherit the parent's header. But we should confirm.

Here's my stab at summing it all up so we can move on the `header-footer-2.rst`:

1. Let's punt on candidate protocol until we figure out the word behavior.
2. I'll move all the HeaderPart stuff starting at line 20 to a separate file.
3. I'll move all the XML specimen stuff in here to a section with that name and make it less verbose.
4. I'll take a stab at a Word Behavior section when there is only a single section.
5. I'll take a stab at at a Schema Excerpt section.

And then once that's done, maybe you can flesh out the Word Behavior for multiple sections and inheritance? I would but I actually only have access to Word Online (does not allow adding / editing sections, only viewing) and Libreoffice (sections + headers seem to be a total cluster).

And then once we have all that, the Candidate Protocol should be obvious and then we should be close to done with the spec.


Good lord :smile: , will revert on develop.

Currently testing how fancy a Builders class will look like.


Sure, if you like to do so, please send a pull request. Otherwise I'll add
this later or on the weekend. Yet, since you came up with it, I'd prefer to
have you getting the credit for your idea.

Interaktive Grüße,

Daniel Stange
Technical Architect

die.interaktiven GmbH & Co. KG
Charlotte-Bamberg-Str. 2
D-35578 Wetzlar

T +49 (0) 6441. 38 457 - 0
M +49 (0) 177. 22 024 026
F +49 (0) 6441. 38 457 - 90

www.die-interaktiven.de
www.facebook.com/die.interaktiven
www.twitter.com/interaktives

Sitz der Gesellschaft ist Wetzlar. Eingetragen im Handelsregister beim
Amtsgericht Wetzlar: HRA 6824

Persönlich haftende Gesellschafterin: die.interaktiven Verwaltungs GmbH
mit Sitz in Wetzlar. Eingetragen im Handelsregister beim Amtsgericht
Wetzlar: HRB 5417. Geschäftsführer: Hendrik Adam, Manuel Busse

2016-02-25 22:35 GMT+01:00 davidboukhors <notifications@github.com>:

> Hello Daniel,
>
> Thank you for your reply. Now I understand the problem.
> So why not doing the test like that:
> list<Profile> prof = [Select id From Profile where Name='Standard User'];
>
> User u = new User(
> FirstName = 'John'
> , LastName = 'Doe'
> , Username = 'john.doe.test@testcase.com'
> , Email = 'john.doe.test@testcase.com'
> , IsActive = true
> , Alias = 'jdoe'
> , TimeZoneSidKey = 'GMT'
> , LocaleSidKey = 'en_US'
> , EmailEncodingKey = 'ISO-8859-1'
> , LanguageLocaleKey = 'en_US'
> , ProfileId = prof[0].id);
>
> insert U;
> system.runAs(U){ ..... }
>
>
>
> Best Regards,
>
> David
>
> 2016-02-25 20:06 GMT+01:00 Daniel Stange <notifications@github.com>:
>
> > Hi David,
> > you're right - the time stamp expects that the time zone of the running
> > user isn't GMT - 3 or anything that makes the date to turn into anything
> > other than the condition. It should actually verify the time stamp versus
> > its UTC value (or convert results before assertion).
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/dstdia/stripeforce/commit/a9ca799067ce1ae89b4215f1c34dab58474ed2d9#commitcomment-16334993
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dstdia/stripeforce/commit/a9ca799067ce1ae89b4215f1c34dab58474ed2d9#commitcomment-16338438>
> .
>


@liggitt Added, thanks for the review.


@bfredl let me see how I can register you as maintainer


Or just BIP112, or BIP68_112 if they're going to be deployed together. These are defined in the DeploymentPos enum after all.
We already have more descriptive names in the consensus validation flags (which I believe we should unify in script/interpreter.h).
Or CSV_112, or VER_TX_SEQ_BIP68_AND_CSV_BIP112. I don't care about long names (they should be only used in a couple of places anyway), but I agree that including all the bip numbers of the deployment would be very useful for traceability.
Plus, bike-shedding: starting all of them with ```DEPLOYMENT_``` is redundantly verbose (just think of how horrible chainparams.cpp will look when we've deployed 10 differet things using versionbits...).


I like to use `'='` for boolean bindings, because `'@'` only sets the property to a string. In this case it would still work as long as you don't set `include-time="false"`, which would set the value to `'false'` in the controller and not `false`


This still could lead to performance issues. I'm not sure if like this. How would you even *detect* whether a given directory is a vdir? "contains_only_file" seems like a flawed approach.

And the perf thing... What if you accidentally have a symlink to `/` somewhere?


а это что означает?


I assume you mean "will not report details"?


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 134](https://teamcity.labkey.org/viewLog.html?buildId=305611&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 21 (3 new), passed: 269 Build time: 01:41:09
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyLearnAboutStudyDetails: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.util.JUnitFooter.afterJUnit: <no details avaliable>

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

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

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

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

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

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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


develop (energy-plus) - x86_64-MacOS-10.9-clang: OK (1566 of 1566 tests passed, 0 test warnings)

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


@fritsch your patch seems to work just fine as it is, no SIGSEGV anymore :-)

Regarding your suggestion to call avahi_threaded_poll_quit after avahi_threaded_poll_stop: This leads to a SIGABRT. Looking at the implementation of these to functions it seems that calling only avahi_threaded_poll_stop is okay, as the "stop" function does everything that the "quit" function does plus some addition stuff (pthread_join after quit, namely). Bus as I said I'm not an expert on this...


в общем случае рекурсия работает медленнее, чем просто цикл. попробуйте развернуть рекурсию gcdex в цикл


Oh and I wanted to add you to the developers section in pom.xml:

`<developers>
        <developer>
            <id>casid</id>
            <name>Andreas Hager</name>
            <email>andy@mazebert.com</email>
            <roles>
                <role>Project-Administrator</role>
                <role>Developer</role>
            </roles>
        </developer>
    </developers>`

If you'd like to add a tag with your details in another pull request :+1:


Why is this necessary and what are the implications for compatibility?  See https://github.com/PacificBiosciences/PacBioFileFormats/pull/31 ...


These kinds of commits shouldn't usually be part of a PR


what is your pypi username?


This is still exporting `libcurve25519.so`, I posted a change in `exportDirectory()` in the discussion thread to exclude it.


Yeah you are right.  In this particular case it should set the fill property based on the pen, though.


OctreeTestEioMathDiff (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1011 of 1011 tests passed, 0 test warnings)

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


logo is better, but the yellow footer looks bad :P


the same, bfredl


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


Yeah I'm looking into what colour to use for the background.


> I can not agree

I understand that it's easier to implement this in a script which can be maintained in DB.
However in the current eventAI logic, this isn't very efficient from the performance point of view.
Ideally we should have an alternative logic that will allow us to move the code implemented in this commit in a table format similar to the way DBscripts are working.


I forgot to mention this in the commit message, my apologies. AWS was not handling the output of this properly: the cloud-init logs said the file was NOT multipart. However, using shell scripts should cover our use cases


Consider respecting the read `limit`?


I think recursion is nice to have, you can just set the path to `~/.local/share/calendars/` and have collections from different servers in there.


Even deeper: as far as I understand, this method does calculate how much of given space will this Layout take, and this solution is to take only square part of given one. At this angle, I am not doing anything bad.


Broke something


You're a whore.


Actually, no I take that back.  Shouldn't this still work?  The stroke property is set to the pen's color, so the line should be visible.  And polylines shouldn't fill anything, therefore the fill property should be absent.  I think.  All the samples seem to render, still, or which one is broken specifically?


It probably makes more sense in the context of khal than vdirsyncer's.

It's your decision, really. I don't feel that strongly about it.


This is kind of hard to follow what is actually happening. I think this could be simplified:

```
udc.Controller.WriteString(cloudConfigControllerTemplate)
udc.Worker.WriteString(cloudConfigControllerTemplate)
```


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


I can't see how this changes anything for this precise problem.
__lkl_create_syscall_thread is still a syscall which will lock, because of the same reason.

Now that I think of it again, one way to fix it would be to start right from the beginning with two threads. One for main() thread, one to receive requests from other threads.


http://avahi.sourcearchive.com/documentation/0.6.24/thread-watch_8h_ec90bd3bb24ce256955941d82952fb7f.html

=> avahi_threaded_poll_stop: "Request that the event loop quits and the associated thread stops."


that's what I meant, we might want to limit the recursion depth for
performance reasons.

Quoting Hugo Osvaldo Barrera (2016-02-25 22:38:03)
> No, but we could have:
>
> ```
> somedir
> ├ blah
> │   ├another_dir (not a vdir!)
> │   │   └ dir that's actually vdir
> │   └another_dir (not a vdir!)
> │       └ dir that's actually vdir
> └ yet another vdir
> ```
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/geier/khal/commit/f34dd061b78924a20acc9b6a541b9ff8cbb7f90e#commitcomment-16338484


UnitConvInIDDforIDFed (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Could make a function on output instead
```go
func (o *Output) AddMetadata(m Metadata){
o.Metadata = append(o.Metadata, m)
}
```
then just in here call it as
o.AddMetadata(Metadata{"error", err.Error()})


OctreeTestEioMathDiff (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 tests passed, 0 test warnings)

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


To futureproof the code, you might want to add something like `EncryptionAlgo` to `Source`. Then you'll know how to decrypt always.


Simpler than I thought.

Yes, there are constants, magic variables, duplication, delays. I just wanted to get it working. Refactoring later.


```go
json:"-"
```
to skip this field


cherry-picked from #11


Should we request a debug printout of all ace_setttings instead of this?


Can you create a PR please?


@cagryInside Thanks for this - everything looks pretty good!  Let's review the Javadoc for this method tomorrow in our meeting - I believe it needs to be changed.  Same for another method I'll comment on.


@cagryInside Let's review the Javadoc for this method tomorrow in our meeting - I believe it needs to be changed.


Yeah we should.


this would only be correct if the --to is not specified...
if you do "premake5 --to=build vs2015" the prj.location is going to build "build/" and so --scripts has to become "$(ProjectDir)/.."

if you wanted this to remain portable it will have to be:
```lua
{ "--scripts=%{prj.location}/%{premake.project.getrelative(prj, prj.basedir)} test"}
```


Risk for NPE.


@verticalgrain  I'm finding `$space-unit-breakpoint` a bit confusing.  I think I liked `mid` a bit better before because it aligns with the breakpoint that it is being changed at.


Probably provide instructions on how to do that then


howto recompile violet?


@cagryInside Everything looks pretty good.  I pushed this commit that had the cleanup we were going through in our last meeting, plus me finishing it up.  I commented on two Javadocs that need to be updated - we can discuss more tomorrow.  Would you also mind adding one final test for the loop in the middle of the route?  This test should fail, as we currently don't support this scenario, but then we can comment it out and add a TODO with some explanation of why it fails.


Might be worth expanding this comment to show why it is needed in the first place, as we discussed on IRC.


Why only 1 level deep?


Fixes #46


Updated field games_id on 17 records


Are you certain this doesn't do the same thing as a `+`? I'd bet it does.


Without the space between Henry and /m, so Henry/mn, but yes


If you set this._applyOpts(null, true, true, false); to this._applyOpts(null, true, false, false); (don't set fill based on brush) and test it with the big file with all the graphs it breaks starting with figure 7.


travis too, presumably - is the failure expected?


This can become a for loop now.


Fat pointers seem wrong. See this commit, which reverts this condition and adds a corresponding test: https://github.com/nikomatsakis/rust/commit/b7f53b8aa6bafe0642efda7e387507fdb6be56df


Just curious why are you using `i_` (which is the `<i>` HTML element)?


I love these strings. :smile:


@johanw666 Can you open a PR?


For me not works :/, i have installed drivers Vulkan Only 356.43, available here and i have this error
![error](https://cloud.githubusercontent.com/assets/5261759/13335804/29e92a5a-dc13-11e5-92b7-2a2cc8ae1e96.jpg)

I have Windows 10


"Clocked" is kind of subtle and still a bit ambiguous to me. It doesn't really make it obvious that it's a software change to adjust a fixed hardware clock. Perhaps if "Clocked" was just "Clock" and "8MHz" was instead "change to 8MHz" it would be a bit clearer.

Note that the IDE will put a colon after the menu item and quotes around the currently selected value, so after selecting, the user would see:
`Clock: "16MHz (default)"`
`Clock: "change to 8MHz"`

I wonder if "Clocked" or "Clock" will have any meaning to some beginners? I still think "Run ... at" better indicates a selectable speed.
Perhaps:
`Run Arduboy at: "full speed (16MHz default)"`
`Run Arduboy at: "half speed (8MHz)"`

>You'll also need to uncomment all the lines for them to be effective.

You had them commented out. I just followed suit. Anyway, I wasn't sure if you would even want to apply this PR. It was more of an easy way to indicate my suggested changes. I can still clean it up, based on exactly what you want, so it can be applied, if you wish.


> also need handle emotion:

@boxa the emote is already handled by creature_template_addon.
I don't think we need do do anything about that, since it's removed automatically on movement and added on evade.


I know. There are just three different copy options. So I feel like as a user I might be confused as to why there was "copy" and "copy to marked input". Where is generic copy copying to? So then I wanted to say "copy to clipboard", but that's not true because it's not actually copying to the clipboard...


NIT: identation


If you're test the updating, you'll need to use the check again button once to clear the transients, or wait 12 hours for them to clear automatically.

If you need help let me know. If I'm guessing wrong, sorry.


by the way, you should base this off of the truecaller branch if you want it to work. unless you've removed the detach/attach load balancer logic.


Didn't we decide that namespace could be customize per project?


Might want to update the comments


Either I was scarred forever from the time I thought I knew what concat was doing or I just didn't think of it. I can change it to concat if that's better?


Please do this for our master? :)

I think we might want to have at least 1 subbranch and that's it in the future so that we dont confuse each other...

But great job on this VCS implementation!


both 4 and 3 backticks are supported but 3 was the original spec so I'm happy either way.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1569 of 1569 tests passed, 0 test warnings)

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


Is there a reason to not have `<a href="mailto:%{contact_email}">%{contact_email}</a>` directly in description_html, as opposed to the extra key?


In your branch of `omnibus-software`, it would probably be best to make the default version `8u74` so that it's not required to override it everywhere when we update `omnibus-software`. This will ensure that every project gets the new version.


Co jeżeli gracz zmieni nick? :)


Oops. Fixed in 69aaddfe. Thanks!


Right now, you're assuming you can only have two files `boshCA.crt` and `secrets.yml` if that's the case, this is fine. but if you think you might have to change it up later on, you might want to have read an array of Sources or an array of Inputs.


These effects are already added from a long time ago:
https://github.com/cmangos/mangos-wotlk/blob/5ff7687d04dbb457509282c89a3fdb6d5fe0b13c/src/game/SpellEffects.cpp#L8095
https://github.com/cmangos/mangos-wotlk/blob/5ff7687d04dbb457509282c89a3fdb6d5fe0b13c/src/game/SpellEffects.cpp#L7841


Fix: Change Client Title (tooltip) according to status


@ItzLevvie i think i undid some of your format but idk. sry if i did


I already submitted an issue [here](https://github.com/mechanicalmind/murder/issues/38) referencing this topic.


This is getting common resources so the above comment can probably be ignored.


PPA has been using this metatype to label sts.xml files in all SubreadSets since CL 171158 (Jan 19th). These files are already identified using this metatype in pbcore and pbcommand. This would formalize the name of the MetaType and document it. That is about it.

The file and its accessibility are necessary for some instrument and secondary reports.


This it was already like this but it is wrong,
here we are not tracking how long it takes to send the events to the backend
but we are tracking the payload. So since we are touching it
better to rename to *ingestion.payload.backend-send*


OctreeTestEioMathDiff (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1569 of 1569 tests passed, 0 test warnings)

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


Good catch! Thanks!


To będzie problem :D
W w następnej wersji będę musiał przepisać na UUID... no i jakiś lepszy system zapisu, może SQLite...


@vito @xoebus any chance of reverting this? The default theme in 0.73.0 is just doing my eyes in, google was by far my favourite. I honestly can't see any harm in leaving this in the codebase, it was a semi-hidden feature.


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


The main thread will block until the new host thread and kernel threads are created, hence the main thread won't be able to issue a syscall. This means that the new host thread can issue a syscall without the risk of being blocked.

We can still have an issue if the app creates a new thread itself and runs syscalls from there. But there is where this patch helps :) - to warn users not to do that bu use lkl_create_syscall_thread if the want to issue syscalls from other host treads then main.


Doesn't this change change behaviour? Previously if there was only a single transaction with `loginuid`of `None`,  `_getLoginUIDs` would return `None`. Filtering during the set creation would make it return an empty list instead. Is it likely to break anything?


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


Yupp :( still now easy way to test OpenGL on travis... Maybe I should invest some time into this at some point...


pz


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


Why do you force an hash-map here? any particular reason?
with just two entries a array-map is more efficient, however better to use a map literal and let the compiler decide.
`{:status :error :message (map #(if % % "OK") errors)}`


when I measure these, I get numbers that are 20x or 25x faster than this.  These numbers are misleading in that they include the overhead of calling `scheme_eval_h` -- which, as you mention above, is limited to 13K/sec.  A comparable figure for python is 8K/sec .. or that's what it was, when I measured it a few years ago -- it was how fast one could call `PythonEval::eval("2+2");`  -- as a result, the "cost" of adding a single node, via python was about 5K/sec  -- at the time, I felt proud that scheme was twice as fast ... before I realized that most of the overhead was in entering and exiting the evaluators.


Good question. Changing now :+1:


same here, better a map literal


Looks good


@haslinger @ramonh I added a special `:deactivated` notice (bilingual) for only deactivating, not really cancelling accounts. Maybe you have a use for it too.


There is a problem:
```
[    3.299888] systemd[1]: Caught <ABRT>, core dump failed (child 102, code=killed, status=6/ABRT).
sh-4.3# dmesg
...
[    3.288258] systemd[1]: Assertion 'a' failed at src/basic/path-util.c:389, function path_compare(). Aborting.
[    3.297512] Core dump to |/bin/false pipe failed
[    3.299888] systemd[1]: Caught <ABRT>, core dump failed (child 102, code=killed, status=6/ABRT).
[    3.307388] systemd[1]: Executing crash shell in 10s...
[   13.309854] systemd[1]: Spawned crash shell as PID 104.
```
See https://github.com/systemd/systemd/issues/2730#issuecomment-188999798


I see this as the same feature if it's done at the same time. That's why I suggested that we don't have a commit that is using only the Enter key and another right on top that completes the feature but it's the same context and block.


This needs to happen in the test harness before initializing libprocess.


I would have expected this fn to be part of the ns you called **...core.processor**, why it is here?


Just run `golint` to get some comments in places.


The idea here is that we are causing io when it is not really needed. We need to find out about that too.


I addressed all comments from the first review, committed the changes and did a "git push". Do I need to anything else before you can review again?


LGTM


OK. I'll address and resubmit.


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


@arek is right. It is missing the check for the allocate. He has a commit to fix that


can you remove this lines


Yeah, that's been bugging me too. I'll look into seeing how much code we can merge. On the one hand, it's nice to keep toolbox and workspace code somewhat separated, but still.


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


Yeah, caught that this morning, fixed it here: https://github.com/AtomLinter/linter-jscs/commit/728ecafa0762d4e7f78a06297bcfe05d85be2578


@fraenkel - I'm frankly struggling with this a bit.  I think how the logic should work is pretty clear - start up a client with no server running (actually two clients), have one of them subscribe to a topic.  Start the server, have the other client publish to that topic.  Check that the message never gets received.  Straighforward on paper.  Where I'm falling down is with how to handle the blocking by the NATS.connect(:max_reconnect_attempts => -1) (This is how we're invoking it, so I want to use the same behavior where possible.)  If I put them into separate threads and start EM inside each one, then it passes even without the L639 change (removing the guard on the subs).  In fact, it's already called that method by the time the server comes up.  What I really need is for the server to be an entirely separate process, because what needs to happen to mimic what we see is for @pending to be nulled out, which isn't happening in my test case.  Since it hasn't been nulled out, we don't even need the L639 fix, since those SUB commands are simply queued up and sent once the server comes up and the connection is completed.  Any advice?  I really need to have a case where the server is down long enough, and the scheduler can run the clients enough, so that although @subs still holds the topic subscriptions, @pending no longer holds the queued commands.  It's difficult to prove the bug otherwise.

I'm probably missing something obvious, but any guidance would be greatly appreciated.


[BrainTraining.zip](https://github.com/ryanmuzzy/BrainTraining/files/147350/BrainTraining.zip)

version 1


Ok, I fixed this.  for polylines, they really should never fill anything, so I set fill to "none" and for the line itself it'll just use the stroke property from the selected pen.  Seems to work.  Thanks for the review!


For our code practices if statements need to have {}
nit picky but easy fix.


you just killed this metric? any reason? or is somewhere else?


@Arcanemagus can you publish master when you get a chance? I am AFK and wont be able to push this fix until late tonight


Hi, I'm wondering why this was reverted?


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


We also ignore transient fields, right? Maybe create a unified location for the Modifiers we ignore? Create a shared method for this kind of check.


Oh ok, understood.
With your change, you can make lkl_create_syscall_thread static, so users won't risk doing that.
But then users might need to create the thread by themsevles.
Also, we could add a warning if we see that a syscall is blocked because of another syscall in the same queue


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


I don't understand what you mean. None of the lines I added deal with the empty case. For example, what you simplify while still making an input `[1, 0, None]` return `[1]`? Speaking of that, is the presence of `None`s in the result accounted for and be kept, or is removing them fine (just being liberal with input)?


do we really need this check here? thi is going to be done on every request.
isn't better to put this in the system construction?


One thing I haven't figured out is why figure 10 has a red box that shouldn't be there...


Here a problem, what I talking about. Just now you need or 1) **don't use auto-summon** and you no recieve warnings **or** 2) **stay in correct channel** and bot connected without warning.


Shouldn't it be tgt=tmp ?


ok


What does it change?


Trying to create new object with attribute id. Class declared in 1521 line.


Actually, a user can't call it anyway, since its a protected function. So it's you're call.


Yes, it is safe to back port and worth adding to the NEWS. The lip sync animation is much better in Pajama Sam: Games to Play on Any Day, and maybe a few Backyard Sports games too.


Since this is an attempt to get metadata associated with the Class I think we may need to check both the prototype and the constructor itself (because of different polyfill implementations).


Does this work to use a pre-packaged CentOS 6.7 version of libhdf5? I had to build my own on with my custom gcc 5.3 on my VM.


just check that process-fn is defined here and avoid it check above


imaged or images?


Good question. Also in figure 18 and 19 the data points should be filled with the same color as the outline yet somehow aren't...


Does this change the behavior of debanding on GL_TEXTURE_RECTANGLE?


We are now reaching the next level of complexity. This is a custom solution and we need a generalized one.

What if the Type could return it's Properties? Maybe using something similar to that we could create a command structure with depth. The two different child properties should have child commands?


@casesandberg Sure, I'll publish a release in a few minutes.


UnitConvInIDDforIDFed (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2157 of 2157 tests passed, 0 test warnings)

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


@twoseat @cgfrost Here's a perfect example of an opportunity for parallelization.  The `serviceInstanceId` and `applicationId` can be resolved in parallel with one another but aren't here.  I'll fix it up.


That's not obvious .. and might even be wrong.  The whole point of an atomspace is to store atoms so that they can be found later, as otherwise, atoms can be used just fine outside of an atomspace.  The is more-or-less true to the temporary atomspaces as well.

I say "more or less", because, due to murky and confusing design, many atoms are stuffed into the temporary atomspace, that probably don't need to be.  Its partly tangled up with the fact that "eager evaluation" is what got implemented, when "lazy evaluation" would be ... better(?)   However, implementing lazy execution adds a bunch of new complexity, and is a fair amount of work.   And it all tangles back up with these temporary atomspaces.   Its quite a mess.

Again: its not at all obvious that the above throw is correct.  I guess you'll find out.


UnitConvInIDDforIDFed (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


(yey)!


this change is a bit of a mess , there are duplicate functions between here, core.processor and ingestion-api.events.
Can you please review it?

[ci skip] is cool but i think make sense to use it only when you make changes which won't affect the code or it's execution such as doc change etc.


This is so the separator line between characters and news doesnt get placed next to the first character


we are not following symlinks :)


I will revert this since in shouldn't be part of the test.


UnitConvInIDDforIDFed (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2125 of 2125 tests passed, 0 test warnings)

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


Commented just in case I ever want to use it again


Yes, this file can just be deleted in favor of the earlier file.


I'll hazard a guess, based on this line, that the old `Offer` class is gone.  Assuming that's correct, how much pain would it be to rename `TOffer` to `Offer`, since the old class is (now) gone?  I see how having different names was really useful during development...


Create gwent.db


12 and 24 core CPU's are now common.  More coming in the future.  Suggest at least 32 here. Lots of opencog threads are not yet common, but the robot is running the pattern matcher in at least two or three, maybe more.


I have tried changing 'poly' to 'polynomial' but that fails the install-check, some further edits might be needed.


Fixed in Merge https://github.com/premake/premake-core/pull/437


It's odd to me that you're returning `[T]?`.  I expect `take 1 []` to be `[]`, not `nil`.  Haskell!


Can you phrase this as a question?


v3.4.8 published!


UnitConvInIDDforIDFed (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1011 of 1011 tests passed, 0 test warnings)

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


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


Missing `@tailrec`


The terminology could be improved here. There is too many synonyms.

Order and Index.
Tech and Techy.

Also maybe we could use simpler method naming like `getIndexUser(int indexTech)` or something similiar.


Upon further inspection, nothing.


FixGSHPweirdOutputOfAsterisksEtc (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2126 of 2126 tests passed, 0 test warnings)

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


I wanted to have an Array but concourse doesn't like that :)


Nice!


@blakejohnson we don't use unreproducible builds like  "custom gcc 5.3 on my VM" on this project :grin:

Docker all the way:

```
FROM centos:6.7

RUN yum update -y
RUN yum install -y centos-release-scl-rh
RUN yum install -y devtoolset-3-gcc devtoolset-3-gcc-c++ devtoolset-3-gcc-gfortran
RUN yum install -y git cmake wget zlib-devel ed

#install hdf5
RUN mkdir hdf5 && cd /hdf5 && \
  wget http://www.hdfgroup.org/ftp/HDF5/current/bin/RPMS/hdf5-1.8.16-1.with.szip.encoder.el6.x86_64.rpm && \
  wget http://www.hdfgroup.org/ftp/HDF5/current/bin/RPMS/hdf5-devel-1.8.16-1.with.szip.encoder.el6.x86_64.rpm && \
  rpm -Uvh *.rpm

#default to the devtools toolchain
RUN echo "source /opt/rh/devtoolset-3/enable" >> /root/.bashrc
```


You are such a nitpicker


Yes, these are measuring the overhead of calls from Python. They're not intended to be measurements of the underlying calls. The eval overhead is a killer.

The above are certainly not measurements of the speed of scheme itself.


Obsolete code, forget to remove it.


I still fixed it in the interest of it not being forgotten.


Add bug number in comment, like Android.


No, this was demonstrating an issue with inclusion order where the let should override the external definition as it's setup after the include.