Note that due to the different order of commits due to the cherry picks, this commit is now https://github.com/JanSchulz/conda-smithy/commit/9559bd869f29453e81ba3d21cf20b8d50211e66e and the above typs are anyway gone...


not a blocker, but we generally prefer to raise Exceptions rather than use asserts


You're missing the point: conformity is nice and all, but `AlarmActivity` doesn't have a `NavigationView`... so why are you adding unnecessary content to this class?


I may have tonned it down too mutch, but before it was way to quick.


Hi, I have bumped to this right after I fixed the bug with different approach. See: https://github.com/PetrDlouhy/django-denorm/commit/0c273f2bb2eb41ff1cd6febb64cb912e456858cd
I have just corrected the SQL query, that it doesn't contain subselect to nonexistent data (which is what doesn't work).

I am not sure if you approach will work in case when deletion fails - it might create dirty instance when the DB is not modified.

What do you think?


I don't particularly like using defaults for the purpose of unifying the two overloads, but it's not that bad. Just saying.


please add braces {...} to here


Again, why tack on `NavigationView` when you're not using it...


If found it useful to make a visual aid at understand where the blocs ends.
Updated & repushed.


Swap is much better naming than flip.


Dear god why.


I'm going to stab a puppy.


We're getting rid of activities, so why~?!?~!?@I*#(@$*(#@!#?!@?#?!@?#@!?#?????


Issue is filed. https://dev.etlegacy.com/issues/921


Is it possible that this fixes the issue I'm having: https://github.com/helpshift/ekaf/issues/32?

Are you planning on merging this into ekaf master?


Awesome! I tried finding this `.values` call, but for whatever reason I couldn't find it. It must have been Monday morning...


Thx for the comments, code changed as you suggested.


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2361 of 2397 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-98.5%25-yellow.svg)


hey you


+1 thanks :-)


I had a little trouble with the second sentence of the comment.  See if this reads better for you:

"If flow doesn't use all the liquidity of a strand, that strand is added to `next_`.  The stands in `next_` are searched after the current best liquidity is used."


Very good idea to move it to common.utils


> Nothing interesting. And actually I prefer to use her…

Did you notice an error referring to the not-found cross root, while he was not meant to use cross?


Should probably deal with null here - otherwise the whole changeset would be a breaking change. We can't expect others to implement IIOMultiPin...


@amoreira route é desnecessário agora


OK.


Nejspíš mi to failuje tady, protože to neví, kam šáhnout, ale nevím a ni nemůžu vygooglit, jak tam dostat to cestu do resources


s/or/os/


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1021 of 1021 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-30.56%25-green.svg)


Your last comment is reflexive. That annoys me. XD


weatherFile (evanweaver) - i386-Windows-7-VisualStudio-12: Build Failed

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


Line is longer than allowed. (86 > 80)

*LineLengthBear, severity NORMAL, section `linelength`.*


ABBS was confused by the existing `CROSS`. I have updated ABBS.


This here looks problematic. Since Metadata is going to be used to create a JSON object, it will take the literal name of the field and transform that to the JSON fieldname. With the change, it will actually change the JSON name, which should not happen.


@amoreira
```js
 <ActionBar title={this.props.title} onSave={this.handleSave}/>
```


Whoops. I can't imagine I would remember to update this - would it be sufficient to just state ```MIT``` in the readme?


Not at pc but I'm sure 2 would be good


I don't like this change. Please revert.

You moved the TODO around without fixing the TODO. If that is addressed, code is both cleaner and faster, once ownership is defined. This side should not even be allocating.


split long line after 'and'


woc I didn't.


Ah ok thanks


It'd be nice to see some checking to keep clients from putting bad values into that metadata that break their datasets. What happens if I try to set a garbage display type via this method?


whoa I can't believe the space after doctype is finally gone!


now in commit https://github.com/JanSchulz/conda-smithy/commit/4f0a833162e78bc4150aaa9e1733ab7a774b056b


lgtm


Nice and compact!


weatherFile (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

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


I don't think up need the `cur_.reserve()`.  You're only adding strands to `next_`.  Or I might have missed something...


@amoreira
```js
@editable({
  name: 'Banner@mycompany.my-first-app',
  title: 'Banner'
})
```


I know what S2325 says on this matter, but I'm a bit on the fence about this one. I don't much like having all kinds of static methods in my class, which even though do not operate on instance variables, still belong to the class operation. Need to sleep on this and chew on it a bit.


pokusaj prvi


@Duane11003 good work, please look over the styles in the footer.


zadatak


`// She's got scuuuurvy!`

ಠ_ಠ


task final


@amoreira
```js
import { editable } from 'editor';
```


task attempt


Pretty cool!


l


Why are you storing a Pouchy reference here?


@SilverSapo question: what is TS?


@amoreira @editable


>[KNOWLEDGE] or [STRENGTH] or [PATIENCE] or [PEACE]

What is comma?


does what it says on the tin!

Gateway#disconnect is a no-op though, so really as long as we're not throwing an exception (like we were) we're good


Absolutely. All operations under Paragraphs in the object model come with it. So add_paragraph, Paragraph.add_run(), etc. Not sure about add_image(), because I believe that's implemented on Document, but that's the type of detail we'll need to sort out as we get down to the next level.


Yes, I agree, but addressing the TODO requires an understanding of this code that exceeds mine at present. I don't see the downside to this change in the short term, but I've opened https://github.com/grpc/grpc-go/pull/588 which excludes this commit.


The solution here would be to change the public String values to final, after which the naming convention is alright once more.


So that's why...


@runrevmark doesn't context_query_legacy only have two parameters?


The following doc(s) have been deployed to the [ttrowellre](http://ttrowellre-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/articles/sites/code/redirect-incoming-requests](http://ttrowellre-static-docs.pantheonsite.io/docs/articles/sites/code/redirect-incoming-requests)


Yes, no problem in SimulationX. But I do not understand why the previous extent is now considered wrong. Can you enlighten me?


AFAIK IPython calls it on`%load_ext`.


It has to do with issues with bzip and python not building on OSX, AFAIK pip was removed from omnibus-software.

Also, it maybe worth waiting for https://github.com/chef/omnibus-software/pull/615 this to get merged so that bzip2 and python gets fixed upstream first.


Esto no debería estar aquí


same comment as https://github.com/lwyszomi/commcare-hq/commit/9081000dd0b0e6d6b39a5c75f206a6b4ab2ad6b7 about date__range


Good start!


muy bien :)


constructor and super are superfluous here, isn't it ?


Including the start date isn't necessary.


You dont need this entries array.


Im going to return the code to 100%


Not needed dependency


Almost there, you need to pass the entries array into the view
res.render('til/index', { title: 'Today I Learned', entries: entries});


Yes, exactly; a convenience method. A lot of the casual and beginner use cases just want to get or put some text without getting down to paragraphs and runs. But underlying it is basically a small handful of lines using paragraphs and runs. I use it here just because it keeps the focus on the headers. We might decide not to implement it at first, that's kind of a separate design and scope question :)


thanks! :+1: that was an artifact from when I was debugging the non-exiting process


Now with 8 working tests!

  Results
    √ renders entries with vote counts or zero (55ms)

  Voting
    √ renders a pair of buttons
    √ invokes callback when a button is clicked
    √ disables buttons when user has voted
    √ adds label to the voted entry
    √ renders just the winner when there is one
    √ renders as a pure component
    √ does update DOM when prop changes

  8 passing (204ms)


This will need to be entries[i].body


Although we only ever add to `next_`, `activateNext` swaps `cur_` and `next_`.


What is this for option?


Most lawyers think so, but advise is mostly to have a separate LICENSE file (and, if you're going to drop one, drop the section in README instead of the separate file).

But, do note that once you have started a project, you don't need to update the year, keeping this at 2016 forever is OK)


Disregard my very first comment, it is covered here


Ooops, thanks. I opened the [doc](http://en.cppreference.com/w/cpp/types/make_unsigned) for exactly this but forgot to actually add it in the end...............


'struct timespec' can not be resolved by inclusion of <time.h>.

There are several ways to fix it :
1. Add #include <sys/time.h> before <time.h>.
   This approach can fix un-recognized data type 'struct timespec', but there is still an ugly warning message that "warning: implicit declaration of function ‘clock_gettime’" due to function prototype in <time.h> is surrounded by macro #ifdef  __USE_POSIX199309 ... #endif.

2. Only #include <time.h> and add -D__USE_POSIX199309 in CFLAGS of Makefile to turn on POSIX 1993.09 support.

3. Only #include <time.h> and change -std=c99 to -std=gnu99 in Makefile

Which one is your preference ?


Probably, but it's commentary, whereas this snippet does not work (for me) due to a syntax error at `{{ else }}`, which I cannot explain yet.


Sounds good!

> On Mar 7, 2016, at 10:45 AM, Ben Zhang <notifications@github.com> wrote:
>
> @damellis <https://github.com/damellis> I changed pause command from e to p.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/damellis/sensors/commit/967c6b93869c1840e78e994232e23763d4cfe5ea#commitcomment-16543086>.
>


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1579 of 1579 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.92%25-green.svg)


Right ok. Thanks for clarifying


17 working tests:

  application logic
    setEntries
      √ adds the entries to the state
      √ converts to immutable
    next
      √ takes the next two entries under vote
      √ puts winner of current vote back to entries
      √ puts both from tied vote back to entries
      √ marks winner when just one entry left
    vote
      √ creates a tally for the voted entry
      √ adds to existing tally for the voted entry

  immutability
    a number
      √ is immutable
    A List
      √ is immutable
    a tree
      √ is immutable

  reducer
    √ handles SET_ENTRIES
    √ handles NEXT
    √ handles VOTE
    √ has an initial state
    √ can be used with reduce

  store
    √ is a Redux store configured with the correct reducer

  17 passing (64ms)


Here's where the old "four-boxes" code lives


You’re still referencing `name` in the docblock and `!db` error. Does it make sense to unregister a database with the database? Doesn’t using its `name` seem easier? Also, what’s up with `db.db`?


@megantuttle I think this is problematic. The simulation time "simtime" as saved by demostration1.slx has a variable time step (the config params show a variable time step in the simulink diagram). I would change this to

(angle(i)-angle(i-1))/(simtime(i)-simtime(i-1))

or [0; diff(angle)./diff(t)]

To avoid issues with assuming a 0.05sec time step.


Por enquanto eu pausar o serviço com a _issue_ #1. Talvez eu volte a mexer nisso após as 22h. Quem quiser colaborar já tem uma base. Como já explicado, `shell` é como o nosso ramo "development".


Oh, interesting! I'm a vim user as well :)

What plugin do you use? I've been using RIV I think it's called and it's definitely gotten my attention how slow it is sometimes. Maybe I'm the one who needs to switch :)


It's very possible that this is why your torque values are weird.


@megantuttle see my other comment above. Same issue here


This change should be removed from your commit


@brchiu : Please refer to [main.c](https://github.com/embedded2016/phonebook/blob/master/main.c) used in phonebook. Simply changing cflags to `-std=gnu99` is preferable.


What is `data`? Where is this variable defined? I'm a little lost.


Shouldn’t need to `bind(this)` to `confirmDestroyed`.


@PetrDlouhy Your change looks good!


Shortlog of HEAD commit is too long.

*GitCommitBear, severity NORMAL, section `commit`.*


This fix permit to make work my network controller: Qualcomm Atheros QCA6174 802.11ac Wireless Network Adapter (rev 32)


I've committed the fix to master, but I haven't tested it. Please let me know if it works for you.


shouldn't it be ```i -> instance.gen(i, standard)``` instead of ```i -> instance.gen(0, standard)```?


The attribute itself isn't a problem. It's that its value is wrong when process substitution is used. As you've shown above, it's correct when `file@some_file.txt` is used so that's my preference.


:+1: hawt.


Fair enough. So please add a FIXME to the `#include`, so it's easier to remember that it should be removed.


This should mention Atom too, and we probably don't want the heading nested under "Developing for Atom".


Thanks much, we will test.

________________________________
From: mprudhom <notifications@github.com>
Sent: Monday, March 7, 2016 2:47 PM
To: glimpseio/GlimpseXML
Cc: Sam Anant
Subject: Re: [GlimpseXML] Can xmlFree on strings retured from xmlNodeGetContent fixes #3 (2318bb4)

I've committed the fix to master, but I haven't tested it. Please let me know if it works for you.

-
Reply to this email directly or view it on GitHub<https://github.com/glimpseio/GlimpseXML/commit/2318bb47db0aee68187900329a51cb5c9d735adf#commitcomment-16544660>.


readme


No, I did mean "rather than". I could have been clearer: once this is merged I'll also open an issue for adding support for per-part content type.


I think that the breakage got introduced because of upgrading the i18n-js gem. I got the same error as you were running into when I just did `bundle update i18n-js` (which takes it from 3.0.0rc11 to 3.0.0rc12). I also note that there's some weirdness going on with that gem; we have a reference to a tarball of rc8 for the i18n-js project in our package.json. Maybe something changed in rc12 that made it incompatible with what we use via npm?

I don't know why we have that in both package.json and a gemfile, in the first place.


Are you suggesting we add to the SDK the same checks that exist on the server side of this API (if any) or are you suggested we add some additional functionality to the server side of this API?


Oh, I did miss something.  You want to retain the reserve in `cur_` with your `swap()`.  `next_` would retain its reserve regardless.  You might want to change the comment so `next_` -> `cur_`.


oh, right. Thanks for pointing.


That sounds like a good compromise. Thanks.


Prueba dde como hacer un commit


Aah easy fix, Flask-Bootstrap


ok THX


How do I override this?


Nice ^^


Took me a long time to debug this missing `break;` :cry:

It didn't matter before because the value types were the same, and the pointers were still assigned.


Line 133 " Each spatial feature in an R spatial object has the same set of associated attributes"  Does this apply to vector spatial objects outside R?  I'd like to make this as broadly applicably as possible to other languages.


@megantuttle sorry for the spam, but I forgot that you interpolated to get everything in terms of "time1" above. Still, When I look at "time1" I see 0.01 as the timestep, not 0.05.

The other thing I'm noticing is that the motors seem to be going all the way to their max angles, so I think this might be contributing to some of the issues you're seeing. I'm going to add a feature that plots the desired angles before doing the animation. Stay tuned for a relevant commit.


2097-EP-8-5 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


Don't placehold me, bro :(


> This was because I created an isBasicAuthHeader method in the abstract class that delegates to the static method.

Sorry, I missed that

> Would you prefer making the BasicAuthHeaderFilter (and its member methods) protected and accessing them directly?

No, what you've done is fine now you've pointed out what I'd missed.


I am just wondering if it would be better (in terms of memory waste) if `sol_oic_cbor` was aware of `sol_buffer`.


For the record, checking strings is a bad way of doing this.

Since we're already entirely cannibalizing and devouring pybitcointools, maybe it's worth changing that exception type to one called NotEnoughFundsException or maybe make select() return None if it fails ?


looks good.


2097-EP-8-5 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


The names in this line's comment are AegisName reference and Transparent_Cloth is AegisName of Fabric.


Wuuzzaaaaa :dancer:


will this also report to Airbrake?


Hmm, that is odd - I have been able to iterate over enumerables multiple times without problems... You mentioned it only seemed to happen in your plug-in, but not in the tests.  Could you try to figure out the exact use in your code, and create a test that mimics that, to see if we can re-create the bug?


`writerOutput` is now unused, right?


@gaelTorrecillas , your changes for sync customers in pos_multi_session work. Could you do a separate branch for that and do a PR ?


Can't run it without the /solu-buddy/ directory in front. Not sure why.


Oh Sweet! Nice job!


I don't see this working. Could you give me an example of a console or consoleQuick prompt in your project.


I also noticed that your loop for i=1:length(motion_des) has the length of simtime, whereas your angular velocities are matched to time1... correct?


Tells user which is invalid.


Thank you! I was pleasantly surprised that you discovered this project.
Feel free to submit a pull request if you so desire! :smile:
You can contact me using the email on my profile page.


Right but in this case the part has no filename. So specifying a filename using `file@` syntax will be just as wrong. The equivalent curl snippet has no `filename` parameter at all, which is what currently seems to be impossible using httpie.


delete


It's not necessary in here if we check input before creating the compound.


LGTM


thanks.  indeed, this needs to be cleaned, _pronto_.

> Does it make sense to unregister a database with the database?  Shouldn't we just use name

sometime i already have a ref and dont need to do a lookup, especially if unregister is just going to do a lookup anyway.  however, `name` sure is convenient too.  let's support both.  no harm no foul.  adds 3 LOC.

>  what’s up with db.db?

pouchy maintains a PouchDB instance as `.db.`  Removal of listeners, however, really should be done inside pouchy destroy, and for the most part, that already happens.  scrapping!

:baseball: thanks


no need to check if the class exists here since it's impossible to reach this code if the proper checks are done earlier on


woops that small hut stuff was not supposed to be pused xD


I think it's because of `⌘ + g` being mapped to "goto next" (try first `⌘ + f` to search for something).

http://blog.bugsnag.com/atom-editor-cheat-sheet


Fixes #10


Nice!  This could be in hspec, but at least we could start a module `tests/Test/Core.hs` for it.


Sorry about that! BTW: is there a reason you prefer spaces over the [Core standards](https://make.wordpress.org/core/handbook/best-practices/coding-standards/)? Have always wondered :)


@t00thpick1 Since updating my mcmmo, it resets the config every two restarts. With a working config pre-update, it is entirely fine. Restarting the server, it loads the config and adds the volume setting. Still working fine. Restarting the server again, the config is reset back to default. :(

```
[15:00:17] [Server thread/ERROR]: Cannot load plugins/mcMMO/config.yml
org.bukkit.configuration.InvalidConfigurationException: while scanning a simple key
 in 'string', line 23, column 5:
        Truncate#
        ^
could not find expected ':'
 in 'string', line 24, column 1:
    #  Settings for Skills
    ^

	at org.bukkit.configuration.file.YamlConfiguration.loadFromString(YamlConfiguration.java:56) ~[patched_null.jar:git-Paper-457]
	at org.bukkit.configuration.file.FileConfiguration.load(FileConfiguration.java:184) ~[patched_null.jar:git-Paper-457]
	at org.bukkit.configuration.file.FileConfiguration.load(FileConfiguration.java:130) ~[patched_null.jar:git-Paper-457]
	at org.bukkit.configuration.file.YamlConfiguration.loadConfiguration(YamlConfiguration.java:179) [patched_null.jar:git-Paper-457]
	at org.bukkit.plugin.java.JavaPlugin.reloadConfig(JavaPlugin.java:187) [patched_null.jar:git-Paper-457]
	at org.bukkit.plugin.java.JavaPlugin.getConfig(JavaPlugin.java:162) [patched_null.jar:git-Paper-457]
	at com.gmail.nossr50.util.LogFilter.<init>(LogFilter.java:13) [mcMMO.jar:?]
	at com.gmail.nossr50.mcMMO.onEnable(mcMMO.java:133) [mcMMO.jar:?]
	at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:291) [patched_null.jar:git-Paper-457]
	at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:332) [patched_null.jar:git-Paper-457]
	at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:407) [patched_null.jar:git-Paper-457]
	at org.bukkit.craftbukkit.v1_9_R1.CraftServer.loadPlugin(CraftServer.java:355) [patched_null.jar:git-Paper-457]
	at org.bukkit.craftbukkit.v1_9_R1.CraftServer.enablePlugins(CraftServer.java:315) [patched_null.jar:git-Paper-457]
	at net.minecraft.server.v1_9_R1.DedicatedServer.init(DedicatedServer.java:206) [patched_null.jar:git-Paper-457]
	at net.minecraft.server.v1_9_R1.MinecraftServer.run(MinecraftServer.java:564) [patched_null.jar:git-Paper-457]
	at java.lang.Thread.run(Thread.java:745) [?:1.8.0_74]
Caused by: org.yaml.snakeyaml.scanner.ScannerException: while scanning a simple key
 in 'string', line 23, column 5:
        Truncate#
        ^
could not find expected ':'
 in 'string', line 24, column 1:
    #  Settings for Skills
    ^

	at org.yaml.snakeyaml.scanner.ScannerImpl.stalePossibleSimpleKeys(ScannerImpl.java:465) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.scanner.ScannerImpl.needMoreTokens(ScannerImpl.java:280) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.scanner.ScannerImpl.checkToken(ScannerImpl.java:225) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.parser.ParserImpl$ParseBlockMappingKey.produce(ParserImpl.java:558) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.parser.ParserImpl.peekEvent(ParserImpl.java:158) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.parser.ParserImpl.checkEvent(ParserImpl.java:143) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeMappingNode(Composer.java:224) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeNode(Composer.java:155) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeMappingNode(Composer.java:229) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeNode(Composer.java:155) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeDocument(Composer.java:122) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.getSingleNode(Composer.java:105) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.constructor.BaseConstructor.getSingleData(BaseConstructor.java:120) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.Yaml.loadFromReader(Yaml.java:450) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.Yaml.load(Yaml.java:369) ~[patched_null.jar:git-Paper-457]
	at org.bukkit.configuration.file.YamlConfiguration.loadFromString(YamlConfiguration.java:54) ~[patched_null.jar:git-Paper-457]
	... 15 more
[15:00:17] [Server thread/ERROR]: Cannot load plugins/mcMMO/config.yml
org.bukkit.configuration.InvalidConfigurationException: while scanning a simple key
 in 'string', line 23, column 5:
        Truncate#
        ^
could not find expected ':'
 in 'string', line 24, column 1:
    #  Settings for Skills
    ^

	at org.bukkit.configuration.file.YamlConfiguration.loadFromString(YamlConfiguration.java:56) ~[patched_null.jar:git-Paper-457]
	at org.bukkit.configuration.file.FileConfiguration.load(FileConfiguration.java:184) ~[patched_null.jar:git-Paper-457]
	at org.bukkit.configuration.file.FileConfiguration.load(FileConfiguration.java:130) ~[patched_null.jar:git-Paper-457]
	at org.bukkit.configuration.file.YamlConfiguration.loadConfiguration(YamlConfiguration.java:179) [patched_null.jar:git-Paper-457]
	at com.gmail.nossr50.config.ConfigLoader.loadFile(ConfigLoader.java:44) [mcMMO.jar:?]
	at com.gmail.nossr50.config.AutoUpdateConfigLoader.loadFile(AutoUpdateConfigLoader.java:26) [mcMMO.jar:?]
	at com.gmail.nossr50.config.ConfigLoader.<init>(ConfigLoader.java:26) [mcMMO.jar:?]
	at com.gmail.nossr50.config.AutoUpdateConfigLoader.<init>(AutoUpdateConfigLoader.java:21) [mcMMO.jar:?]
	at com.gmail.nossr50.config.Config.<init>(Config.java:23) [mcMMO.jar:?]
	at com.gmail.nossr50.config.Config.getInstance(Config.java:29) [mcMMO.jar:?]
	at com.gmail.nossr50.locale.LocaleLoader.initialize(LocaleLoader.java:80) [mcMMO.jar:?]
	at com.gmail.nossr50.locale.LocaleLoader.getString(LocaleLoader.java:33) [mcMMO.jar:?]
	at com.gmail.nossr50.locale.LocaleLoader.getString(LocaleLoader.java:21) [mcMMO.jar:?]
	at com.gmail.nossr50.datatypes.skills.ToolType.<clinit>(ToolType.java:10) [mcMMO.jar:?]
	at com.gmail.nossr50.datatypes.skills.SkillType.<clinit>(SkillType.java:42) [mcMMO.jar:?]
	at com.gmail.nossr50.config.AdvancedConfig.getMaxChance(AdvancedConfig.java:693) [mcMMO.jar:?]
	at com.gmail.nossr50.config.AdvancedConfig.validateKeys(AdvancedConfig.java:48) [mcMMO.jar:?]
	at com.gmail.nossr50.config.ConfigLoader.validate(ConfigLoader.java:62) [mcMMO.jar:?]
	at com.gmail.nossr50.config.AdvancedConfig.<init>(AdvancedConfig.java:22) [mcMMO.jar:?]
	at com.gmail.nossr50.config.AdvancedConfig.getInstance(AdvancedConfig.java:27) [mcMMO.jar:?]
	at com.gmail.nossr50.mcMMO.loadConfigFiles(mcMMO.java:405) [mcMMO.jar:?]
	at com.gmail.nossr50.mcMMO.onEnable(mcMMO.java:145) [mcMMO.jar:?]
	at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:291) [patched_null.jar:git-Paper-457]
	at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:332) [patched_null.jar:git-Paper-457]
	at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:407) [patched_null.jar:git-Paper-457]
	at org.bukkit.craftbukkit.v1_9_R1.CraftServer.loadPlugin(CraftServer.java:355) [patched_null.jar:git-Paper-457]
	at org.bukkit.craftbukkit.v1_9_R1.CraftServer.enablePlugins(CraftServer.java:315) [patched_null.jar:git-Paper-457]
	at net.minecraft.server.v1_9_R1.DedicatedServer.init(DedicatedServer.java:206) [patched_null.jar:git-Paper-457]
	at net.minecraft.server.v1_9_R1.MinecraftServer.run(MinecraftServer.java:564) [patched_null.jar:git-Paper-457]
	at java.lang.Thread.run(Thread.java:745) [?:1.8.0_74]
Caused by: org.yaml.snakeyaml.scanner.ScannerException: while scanning a simple key
 in 'string', line 23, column 5:
        Truncate#
        ^
could not find expected ':'
 in 'string', line 24, column 1:
    #  Settings for Skills
    ^

	at org.yaml.snakeyaml.scanner.ScannerImpl.stalePossibleSimpleKeys(ScannerImpl.java:465) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.scanner.ScannerImpl.needMoreTokens(ScannerImpl.java:280) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.scanner.ScannerImpl.checkToken(ScannerImpl.java:225) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.parser.ParserImpl$ParseBlockMappingKey.produce(ParserImpl.java:558) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.parser.ParserImpl.peekEvent(ParserImpl.java:158) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.parser.ParserImpl.checkEvent(ParserImpl.java:143) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeMappingNode(Composer.java:224) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeNode(Composer.java:155) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeMappingNode(Composer.java:229) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeNode(Composer.java:155) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.composeDocument(Composer.java:122) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.composer.Composer.getSingleNode(Composer.java:105) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.constructor.BaseConstructor.getSingleData(BaseConstructor.java:120) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.Yaml.loadFromReader(Yaml.java:450) ~[patched_null.jar:git-Paper-457]
	at org.yaml.snakeyaml.Yaml.load(Yaml.java:369) ~[patched_null.jar:git-Paper-457]
	at org.bukkit.configuration.file.YamlConfiguration.loadFromString(YamlConfiguration.java:54) ~[patched_null.jar:git-Paper-457]
	... 29 more
```


Yeah! Run.add_image() totally works with runs in paragraphs in headers. I'm using it currently. There are details about how the image parts relate in: `https://github.com/eupharis/python-docx/blob/feature/header/docs/dev/analysis/features/header-part.rst`

Basically you get a `header1.xml.rels` which works exactly the same as `document.xml.rels`


> Do we want to log the warning when the value is provided, or when it's different from the stored one? For me if optvalue != replay_args.get(option, None): would be better.

Actually, from my perspective, user is not aware about what's stored. If user decides to provide a new value in command line and it happens to be the same as in stored args, be silent can confuse the user who is expecting an overwrite msg, regardless what is stored. So I'd prefer to keep the warn when any value is provided.

> Btw question number two, we usually handle everything as if getattr(..., None) is None. The problem I see with this is, you can't disable the option from cmdline, as even --external-runner '' is not None. Maybe we should turn places where it make sense to use bool() instead of is None, what do you think?

Yes, I agree the problem with `if getattr(..., None) is None` is we cannot disable the option. I'm used to think the bool check is more flexible in that respect. And since we are in a plugin here, it makes more sense to me.

> Actually the if optvalue is in-deed incorrect. I'm trying to supply --external-runner '', which is evaluates to False. With this it does not make that much sense, but there are other options which can take "" as a valid input.

As I see, "" is not expected to be a valid input. If it is somewhere, then it should be handled individually.

>  Oh, that causes problems with loaders as it's ad-hoc default... It looks like we need two handlings for this...

Not if I keep the any-command-line-overwrite-recorded approach (and not the distinct-command-line-overwrite-recorded).


The following doc(s) have been deployed to the [ttrowellht](http://ttrowellht-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/htaccess](http://ttrowellht-static-docs.pantheon.io/docs/articles/sites/code/htaccess)
- [docs/articles/sites/code/redirect-incoming-requests](http://ttrowellht-static-docs.pantheon.io/docs/articles/sites/code/redirect-incoming-requests)


I'm not sure if changing the event.getPlayerId() back to what it was should effect sourcePlayer checking, but I fixed the biggest issues, which was just a mistake.


Tried adding search facility also. But I am getting some error related to jquery. Will try it later.


Since the constructor is annotated with `@JsonProperty`, and `parsedUri ` is explicitly set in the constructor, I think this annotation is now either redundant or incorrect.


:+1: , patched


Nit: revert this change


should also be `private` and `final`


On OSX:

```
 WorkspaceD Debug: object.Exception@source/app.d(244): Cannot mix sync and async functions!
----------------
4   workspace-d                         0x0000000106c215f8 void workspaced.app.handleRequest(int, std.json.JSONValue) + 1220
5   workspace-d                         0x0000000106bf43fc _Dmain + 2188
6   workspace-d                         0x0000000106d8be37 D2rt6dmain211_d_run_mainUiPPaPUAAaZiZ6runAllMFZ9__lambda1MFZv + 39
7   workspace-d                         0x0000000106d8bd6f void rt.dmain2._d_run_main(int, char**, extern (C) int function(char[][])*).tryExec(scope void delegate()) + 55
8   workspace-d                         0x0000000106d8bddc void rt.dmain2._d_run_main(int, char**, extern (C) int function(char[][])*).runAll() + 44
9   workspace-d                         0x0000000106d8bd6f void rt.dmain2._d_run_main(int, char**, extern (C) int function(char[][])*).tryExec(scope void delegate()) + 55
10  workspace-d                         0x0000000106d8bcc1 _d_run_main + 497
11  workspace-d                         0x0000000106bf4a3f main + 15
12  libdyld.dylib                       0x00007fff896615ac start + 0
13  ???                                 0x0000000000000000 0x0 + 0
```


The states and regions should be normalized as a ref table.


that is weird!


defined
#introduction takes in an argument and uses that argument in the method body.
#introduction_with_language takes in two arguments and uses both arguments in the method body


Nit: revert


develop (mjwitte) - 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)


Interesting! Ok!


no need for either of these


maybe `docs/**/*.rst` would be a nicer glob as it wouldnt include Makefile and any other helper files there ?


I had thought these were both included, see source: and it shouldn't be necessary anyhow, see the source for the test https://github.com/rails/rails/blob/v5.0.0.beta1/railties/test/isolation/abstract_unit.rb


We probably ? want to ignore docs/API as it's autogenerated when I run sphinx-apidoc ?


please revert. this ensures the minitest gem is used, not the stdlib. I forget if we'll need this when we remove Rails 4.0.


looks like here is where you're using the None-ness of the ends. i'd recommend passing the board itself and returning the whole hand if the board is empty


same as `return self.hand`


GG.
WP.
NORE.


Hi,

there is a problem with this commit about the specifying fe= in the URL.

By default tvheadend considers first adapters as DVB-S(2). and then the other type of adapters.

If for example on the system adapter0 is DVB-T, tvheadend specifies by default fe=X, basically you can get to a situation where nothing can work.

For example if adapter0 is DVB-T and adapter1 is DVB-S, then the SETUP for DVB-s will have fe=0 and for DVB-T will have fe=1 which will lead in both cases to 404 because get_free_adapter will return no_adapter.

Commenting //goto noadapter fixes it, but I agree is not a desirable workaround.

If you have any ideas let me know as I agree tvheadend cannot guess which is the adapter number before the first tune. After that rtcp messages  (or DESCRIBE ) can indicate the adapter number.

Thanks


gg :+1:


Just thinking here. Do we really want this only for docs ? What about documentation comments in coalib ?
That's technically a part of docs too because we use sphinx-apidocs

Opinions ?

PS : `tests` shouldnt be there as well, we have the InvalidLinkBearTest there XD


i think before moves used to be represented as a tuple with the domino and an indication of whether you're playing on the left or right side of the board. i currently don't have an opinion about which representation is better, but will think about it as i continue reading the code


could shorten the code by saying `if left_end in domino or right_end in domino`


Corrected at https://github.com/protegeproject/mapping-master/commit/9a2372c279485277f5e4cbfb8fd243371fc6bd78


Good suggestion - I've added translation support in 839d282

But note that as far as I could tell, this code is not really used - it's only use is in UserProfileForm, but I think it isn't really used....


In general, the usage of the service id instead of service name is fine. Usage of the service name is more reliable and "stable" though. This due to the fact that the service id value is assigned dynamically by the provider / ADS at the run time and may change from run to run or from provider to provider.


Attempt to clarify how to run the code to reproduce the reproduction.


Fix typo in commit message: cursor not coursor.


GitHub
http://i.imgur.com/OdOKUgf.png
gitk
http://i.imgur.com/6kFk4tm.png

Weird


Not sure that you need `border-style: solid` here -- the line underneath the link on hover is being provided right now by `text-decoration: underline`.  Having the `border-style` rule here is fine but it's currently not contributing to any part of how the `<a>` tags are being rendered on the screen.


Added signal strength to payload


The lacking of loop device has been fixed in docker 1.10?


new django-related-admin package https://pypi.python.org/pypi/django-related-admin

@davidhadrianus89 @novikesumaningtyas please run:

* pip install -r requirements.txt


I ended up changing (and committing) the REMOTE_HOST ip address to
52.32.228.220. Could you check if yours is the same? Mines worked as of
Friday night.

On Sun, Mar 6, 2016 at 5:56 PM, Jens Vanderhaeghe <notifications@github.com>
wrote:

> Yeah I'm not fully understanding the purpose of these meta files yet.
>
> Are you able to runt he client at all? Mine just hangs.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/worldofbalance/unity-game-client/commit/8b2d07bd8b567f78e291bb1bf305be79190370b1#commitcomment-16526899>
> .
>

--
Nicu Listana
*LinkedIn
<http://www.linkedin.com/profile/view?id=134425825&locale=en_US&trk=tab_pro>.Twitter
<https://twitter.com/#!/apebeast>.G+
<https://plus.google.com/b/101191476444716618590/101191476444716618590/>*


#22 resolved.


`if(len(moves) == 0)` is equivalent to `if not moves`

also, the whole function is equivalent to `if moves: return ...` since None gets returned by default if nothing else is returned. again, though, returning None is usually not the best idea. i'll be on the lookout for how you can remedy this


Seems you have to `stack build happy` at the project root, not inside `editor/.  Disclaimer, I don't know anything about `stack`.


Resolved #17.


부산대학교 우리말 배움터에 의하면 `그다음`이 맞습니다.


Apparently _someone_ doesn't know powers of two.


Resolved #9


check this out: http://stackoverflow.com/questions/2931985/finding-max-in-python-as-per-some-custom-criterion

you can do the same for the reverse greedy to shorten both of these functions to a single function call


wut

[http://i.imgur.com/8TsuKZm.png](http://i.imgur.com/8TsuKZm.png)


@lsj4401 아닌것 같음 ㅋㅋㅋㅋㅋ 근데 O(n)으로 끝낼 방법이 생각이 안남


I think you should be mentioning about the entry_point for bears. It's a pretty important change to the API.

Should we mention newcomer documentation ?
Should we mention that we moved over to rst ? (Was that there in the last release?)


Plot of convergence rates for different ranks.


More like when does it not?

Honestly this method of flagging the failure value can/should be swapped out for something else at this point, since we're now sending the data in plaintext. I may put on agenda for tomorrow.


You could also make this an `<article>` tag instead of `<div>` here.


the python style convention is to not put parens around an `if` condition

also, `if x is True` is equivalent to `if x`


looks good


Don't forget to add `alt` attributes to your `<img>` tags for better search engine optimization results and compatibility with screen readers.


The ADS always sends the same dictionary, from the local file system, no matter which service one selects and independent of what the provider actually advertises it supports.  If you connect direct to the provider it can send different dictionaries or simply may not have those dictionaries on each and every service.


Thanks for the tip @denzildoyle ;)


you are re-encoding some of the greedy logic here if a double is not found

i would suggest going through the hand looking for doubles and, if no doubles are found, straight up calling the greedy function on the hand. it is technically worse in performance, but performance isn't really an issue here, and it will make your code clearer and more concise


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


Change topic.


Your computer is scrublord

-Tony

On Sun, Mar 6, 2016 at 7:10 PM, vpzomtrrfrt <notifications@github.com>
wrote:

> wut
>
> http://i.imgur.com/8TsuKZm.png
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fictional-pancake/turbo-spork/commit/724d70282936b9722e8ccc5677fef4c7a9e27079#commitcomment-16526986>
> .
>


Fix coming soon!  Should be this week.

On Mon, Mar 7, 2016 at 12:49 PM, RebeccaFarrington <notifications@github.com
> wrote:

> thanks for fixing that @jmansour <https://github.com/jmansour> .
> was there an update on the status of MeshVariable_projection
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/underworldcode/underworld2/commit/130b0168fd34ce032d6a029b7bf1b9c4bbc034cf#commitcomment-16526853>
> .
>


next time publish with `--tag next` instead of tagging it afterwards so greenkeeper isn't triggered ;)


i recommend actually making it random with something like this: https://docs.python.org/2/library/random.html#random.choice

it is true that arbitrarily choosing the first one will probably have the same effect in this case, but as the codebase grows, just choosing the first one might not be good enough. again, this will probably add a small performance hit, but that shouldn't be a problem


Licences added and updated


Fix for #35


@vlj je peut pas utiliser la 3.8 pour le moment, elle n'est disponible ni la https://homebrew.bintray.com/bottles-versions/ ni la http://llvm.org/releases/, il faut attendre encore, qu'elle soit sur les 2 pour ca


read


Do not use underScore as constant names use camelcase style.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18521](https://teamcity.mesosphere.io/viewLog.html?buildId=155480&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:03:04


We shouldn't panic inside a library,  return an error here and handle it at the top level caller.


i think this condition will always be false, since all dominos in a game are unique


5.1后改了目录结构


Where possible, avoid inline styles -- over time, leaving too many inline styles in your HTML can make your pages hard to maintain.

Also, when sizing images, if you set either property on an image (width or height), the browser will automatically scale the other property to maintain the correct aspect ratio; so in this case, you can set the height for the images and the browser will automatically scale for the width property.  Usually, you want to let the browser take care of scaling down the other property to avoid any changes in the ratio the image is rendered as (currently, I think your Facebook icon is getting a little bit squished vertically).


*核心代码*

```
- (void)drawInContext:(CGContextRef)contextRef {
    //    CGFloat originalRadius = _radius;
    CGFloat totalAngle = _endAngle - _startAngle;
    CGRect rect = self.bounds;
    CGPoint cP = CGPointMake(rect.size.width/2, rect.size.height/2);
    CGContextSetLineJoin(contextRef, kCGLineJoinRound);
    CGContextSetFlatness(contextRef, 2.0);
    CGContextSetAllowsAntialiasing(contextRef, true);
    CGContextSetShouldAntialias(contextRef, true);
    CGContextSetInterpolationQuality(contextRef, kCGInterpolationHigh);
    CGContextSetLineWidth(contextRef,2.0f);     //设置线条宽度

    for (int i = 0; i < _total; i++) {
        //计算初始点
        CGFloat x0 = cP.x + cosf(_startAngle + totalAngle*i/_total)*_innerRadius;
        CGFloat y0 = cP.y + sinf(_startAngle + totalAngle*i/_total)*_innerRadius;
        CGContextMoveToPoint(contextRef, x0, y0);
        //计算终点
        CGFloat x = cP.x + cosf(_startAngle + totalAngle*i/_total)*_radius;
        CGFloat y = cP.y + sinf(_startAngle + totalAngle*i/_total)*_radius;
        //画线
        CGContextAddLineToPoint(contextRef, x, y);
        //设置颜色
        CGContextSetStrokeColorWithColor(contextRef, _color.CGColor);
        CGContextSetFillColorWithColor(contextRef, _color.CGColor);
        //设置线条起点和终点的状态 (例如：圆角)
        //CGContextSetLineCap(contextRef, kCGLineCapRound);
        CGContextDrawPath(contextRef, kCGPathFillStroke);
    }
    //在这里你可以 对最后绘制出来的那条已完成的线条所在位置进行自己定制 (例如：绘制一个圆环标示一下)
    for (int i = 0; i < _completed; i++) {
        CGFloat x0 = cP.x + cosf(_startAngle + totalAngle*i/_total)*_innerRadius;
        CGFloat y0 = cP.y + sinf(_startAngle + totalAngle*i/_total)*_innerRadius;
        CGContextMoveToPoint(contextRef, x0, y0);
        CGFloat x = cP.x + cosf(_startAngle + totalAngle*i/_total)*_radius;
        CGFloat y = cP.y + sinf(_startAngle + totalAngle*i/_total)*_radius;
        CGContextAddLineToPoint(contextRef, x, y);
        CGContextSetStrokeColorWithColor(contextRef, _completedColor.CGColor);   //设置颜色
        CGContextSetFillColorWithColor(contextRef, _completedColor.CGColor);
        CGContextDrawPath(contextRef, kCGPathFillStroke);
    }
//    CAShapeLayer *shapeLayer ;
//    [shapeLayer setLineCap:kCALineCapRound];
}
```


this change make syntax error
missing a `;` at line 191 and
may need to add back `}`


I think it would be better to do nothing if there is no tag. "untagged" is very English.


这个跟我的结构比较相似，但第一次相遇后，即有圈了，然后接着做，而不是break出来
Here is the code:

ListNode *detectCycle(ListNode *head) {
    if (head == NULL || head->next == NULL)
        return NULL;

    ListNode *slow  = head;
    ListNode *fast  = head;
    ListNode *entry = head;

    while (fast->next && fast->next->next) {
        slow = slow->next;
        fast = fast->next->next;
        if (slow == fast) {                      // there is a cycle
            while(slow != entry) {               // found the entry location
                slow  = slow->next;
                entry = entry->next;
            }
            return entry;
        }
    }
    return NULL;                                 // there has no cycle
}


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


:heart: linting :+1:


@MircoT -- great work in getting the build to run, and fixing all these 2to3 issues! However, I'm not sure about "converting into a python package." This adds a lot of conceptual overhead: packages, modules, __init__.py, setup.py, multiple new directories. I think that many users of this code will be casual Python programmers for whom all these concepts will be new and confusing. Can you convince me that there is a real benefit to having this package structure? I think it is simpler to have all the files in one directory, with no packages, __init__.py, etc.


This issue is SOLVED. I shouldn't have used `echo` to test value.


seriously. https://github.com/WaspRC/Python-for-humans/commit/d494af8376cb918dfa1af10f85b71a75a62de212


这里是为大神F2开发的手机主题文件，同时也适用于使用coolui6.0系统的其他机型。


Yes. The client seems to work again! Thanks :+1:


wow


or maybe

                if (alignedPawns)
                {
                   score += RookOnPawn * popcount<Max15>(alignedPawns);
                   //look for aligned pawns who are defended by a pawn
                   Bitboard alignedPawnsDefended = alignedPawns & pos.attacks_from<PAWN>(pos.square<PAWN>(Them));
                   //if exists and rook have low mobility, remove the bonus
                   if (!!alignedPawnsDefended && (mob <=2))
                       score -= RookOnPawn * popcount<Max15>(alignedPawnsDefended);
                }


The absence of `require 'action_controller/test_case'` results in `uninitialized constant ActiveModelSerializers::SerializationContext`

The absence of `require 'minitest/autorun'` results in `uninitialized constant Minitest::Mock`


qqq


falta el `es-ES` :P


Each declaration should be on its own line and indented; while browsers don't care about the way this is formatted, our human eyes can scan and read these lines of code more easily when each declaration is on its own line and indented properly.


Hi Zenon,

For the username alone, this would work: new installs would get the new length, and upgrades would have the version from the  longerusername monkey-patch.

And this is exactly what's happening - the accounts/0001_initial migration creates ````accounts.User``` with a username.max_length of 255 characters.

The complication comes from the fact that the auth.User model had evolved independently since the version we had with Django 1.4, introducing changes to the ````email``` and ````last_login```` fields that we do not have in the old model.

The changes in accounts/0003 and accounts/0004 are the relevant parts of the migration chain hanging off auth.User.

And so we need these migrations to apply them to existing databases as part of the migration to Django 1.8.  See the instructions included in the documentation (8c8842e) - which essentially are "fake initial migrations, then apply all subsequent migrations".

Details at https://github.com/REANNZ/djnro/commit/8c8842ed47c1612099d753ff37ce532efad3616c#diff-84eddf2c59a1b9acbe66d6b2ca2ee24cR51

Hope this explains it.

Cheers,
Vlad


Don't forget to use `alt` attributes on your `<img>` tags to increase search engine optimization and compatibility with screen readers.


https://github.com/rails-api/active_model_serializers/blob/8a040052afc0430bef039cc3bc11ff900819a713/test/test_helper.rb#L24

I moved it from there to the test group in Gemfile, betting that I'd need to add it to isolated_test.rb. Seems that everything still runs when it's not in Gemfile nor in test_helper.rb, probably getting it from stdlib. I'll see if this breaks things in older Rails versions.


Finally, been waiting for this fix for *ages*.


I know the test case is from https://phabricator.babeljs.io/T7108,

Do we want to add more cases?

Would an arg in the first async matter `static async method1(a) {`

and also with `...a` as a param?


Good idea, but mind if I do this in a separate pull request? The directories are already represented as groups, and changing them to folder references would increase the surface area changed by this pull request (which is already quite large!).


Warning: Declaration of FOS\RestBundle\Controller\TwigExceptionController::createView($format, Symfony\Component\Debug\Exception\FlattenException $exception, $code, $parameters, Symfony\Component\HttpFoundation\Request $req
  uest, $showException) should be compatible with FOS\RestBundle\Controller\ExceptionController::createView(Exception $exception, $code, array $templateData, Symfony\Component\HttpFoundation\Request $request, $showException)


comment 1


> If the gamepad has no axes, the value variable will be undefined: https://github.com/melonjs/melonJS/blob/0a65a97ef5b4dbd136c7cf429b7746efa1691586/src/input/gamepad.js#L258 I think you want:

> var `value = gamepad.axes[mapped_axis] || 0;`

wouldn't it better to actually "return" at this point if axes[mapped_axis] is undefined ?
also i'm not sure what happen if they are no analog stick at all; is axes undefined or just an empty array ?

a quick test would definitely be great :)


haha cool it got useful :D


Creo que sería `Desmontar`


Interesting question.

I reused the code from auth.User without specifically checking.

The reason for including it was to have the option to also bring the application up with ````auth.User```` as the user model.  This was for testing/troubleshooting purposes, but without the swappable property, all hell would break loose.  My understanding is that the property deactivates the model (pretend it wasn't defined) if the relevant setting points to a different model class - but unfortunately, I don't have any documentation to point to.  Sorry about that.


Just noticed I didn't finish cleaning up Read. I'll fix it tomorrow.


Lest need `clock_gettime` - but only that, ref. #174


ここら辺も三項演算子だなー　短くて済むし。


现在看到啦木


別の条件にするんじゃなくて `record.match` のブロックに入れてください。そうしないとせっかく `record.match` のブロックに入れて、何度も `query.keywords` を指定しなくても済む、を実現した意味がなくなってしまいます。

なので、こんな感じにします。

```ruby
full_text_search = record.match(query.keywords) do |target|
  (target.index('Terms.Posts_title') * 10) |
    (target.index('Terms.Participants_name') * 4) |
    (target.index('Terms.Posts_content') * 2) |
    target.index('Terms.Participants_description)
end
```

ただ、これだと著者のフルネームを指定してもその人の記事が必ずしも上にはきません。著者のフルネームで検索するならその人の記事を探しているだろう可能性がすごく高いのに、です。

なので、著者名に完全一致したらスコアーを高くします。

```ruby
full_text_search = record.match(query.keywords) do |target|
  (target.index('Names.Participants_name') * 20) |
    (target.index('Terms.Posts_title') * 10) |
    (target.index('Terms.Participants_name') * 4) |
    (target.index('Terms.Posts_content') * 2) |
    target.index('Terms.Participants_description)
end
```

そのためには、追加でインデックスが必要なので `groonga/init.rb` に以下の追加します。

```ruby
schema.create_table('Names',
                    type: :patricia_trie,
                    key_type: :short_text,
                    normalizer: 'NormalizerAuto') do |table|
  table.index 'Participants.name'
end
```

（でも、このpull requestでここまでやる？っていう気持ちにはなっています。）


Typo error: must be "canola" instead of "canols"


redirectさせるならflashは結構必須だよ


こいつらは使っていないので追加しないでください。インデックスが増えるとその分だけディスクとメモリーとCPUを使うんです。

ちなみに、 `Posts.title` と `Posts.content` に `TokenMecab` なインデックスを作っているので類似文書検索の精度を上げるためです。類似文書検索しないならいらないんですよ。


Are you referring to new lines 540-544 starting with str<< "from PySide import QtCore, QtGui\n"?

Those lines work the both pyside and PyQt5 with the exception of "from PySide import" that string appears in a huge number of python files and needs fixed in every one of them.  Right now I do a regex replace before I compile to fix them all at once.  That is a temporary solution.  I don't know what the final solution will be.  Do you know if python has an equivalent to #ifdef?

PyQt5 needs the line changed to "from PyQt5 import", which can be done with s/PySide/PyQt5/g

Do you know if PySide2 needs "from PySide2 import" or is it fine as written?


@frederickfogerty - what you suggest is what I'm doing, I think.


これpartial細かく記述しなくてもいけるやつやない？


Do not need to use `display: inline` because you are already using `float: right`.


54 wtf


>コミット時に自動プッシュさせてしまっているせいで
これはやめたほうが良いよ　笑


Hi Zenon,

I wasn't sure about the text editor header, so I also added the encoding header to be sure.

I was getting:

    UnicodeDecodeError: 'ascii' codec can't decode byte 0xce in position 1: ordinal not in range(128)

and that went away after either setting ````leave_locale_alone````, or removing the wrapping on the system locale.  And it also worked with both - I ended up including both in the commit, hoping for this working in most scenarios.

Please let me know if you'd like this changed in any way.

Cheers,
Vlad


Since you are using `float` below, if you leave off specifying the `width`, the nav will align to the right side of the page.


We can change it to '' or not at all. As long as we catch the exception.


`typeof` isn't a function, parentheses aren't needed


Too right you are


ここら辺コンフリクト解決よね？


Add 'vertical-align: top;` to make the sidebar align to the top of the blog container.


これもrender @decksでいけそう


... looking again at the documentation for the [leave_locale_alone option](https://docs.djangoproject.com/en/1.8/howto/custom-management-commands/#django.core.management.BaseCommand.leave_locale_alone), it seems to me appropriate to turn this on for a command that insists on using a non-default locale....


I'm not sure about this one. My gut tells me that "edict" implies monarchy, and monarchy has decreased with time. Do any dictionaries call it archaic?


I think you are using this to align the "read more" links to the right side of the page; you want to be careful when using large margin values like these -- especially in a case like this one since screen sizes are not always the same.  A better way to align this text to the right would be to either use `float` or nest this in a container that is 100% width of the blog post and use `text-align: right`.


Lines 40-43 above should all be indented in one to reflect that they are children of the `<div>`


Don't forget to use the `alt` attribute on `<img>` tags to improve search engine optimization.


Fix bug: the X-scale factor of wavelet UV denoise should be twice as much as Y-scale factor


Lines 50-55 above should all be indented in one to reflect that they are children of the <div>


Hi Ricardo,
What's the purpose for this change?


lines 63-70 should all be indented in to reflect they are children of the `<aside>`


lines 75-77 should all be indented one in


indent


My nick has two m's


Yeah, I think I understand it now.  Thanks so much for explaining!


A simple Trie implementation for Java


lines 92-100 should be indented to reflect that they are children of the `<footer>`


```
osd: osdmonitor: do fast_read sanity check earlier
```
this change does not touch osd at all, better off putting "mon: OSDMonitor:" or simply "mon: " as the prefix instead.


@leninkp podemos agregar entonces un comentario? pondría un `TODO` si jshint no se queja


There is one thing i don't see yet is why Jitter here in the first place the blog is not talking about retries instead about a particular feature in DynamoDB - "Optimistic concurrency control (OCC) is a time-honored way for multiple writers to safely modify a single object without losing writes. "  where it can be optimized for various competing calls. We are not even multi threaded so i don't see why. I think a simple retry is all we need, there is no good reason to add this right now.


Also ListViewAdapter will be deleted later, I am using ActivtyList to extend form it as to hopeful not have as much redundancy


This shouldn't change, the default difficulty (normal) should remain the default.


This empty line should remain.


oh wow! didn't know about that one...


one empty line here after ```end``` please.


`개발자들은`에서 `들`은 부정문이나 복수임을 강조할 때만 사용했습니다. 여기는 없어도 될 것 같습니다.


The test is missing a `-b`. In effect `exit 0` is never reached.


this log line is duplicate, debug left-over?


This needs some fixing. The correct files are picked up by this, but only the first one is run when the `command` is executed.


Linux Kernel은 독음으로 표기하지 않았습니다.


empty lines around this section please.


Typo! Should be: `gamepad.id`


우리말 배움터에 의하면 `변경 점`이 맞습니다.


empty line inserted here?


indentation


@Nevcairiel: Thanks for the pointer; pushed a fix into branch cpb of my FFmpeg copy: https://github.com/gajjanag/FFmpeg/tree/cpb.

Note: codecpar has not been merged yet, and I don't know if both the cpb_props and codecpar->bit_rate should be set. Anyway, I have placed a relevant comment on the line.


https://developer.mozilla.org/en-US/docs/Web/API/Gamepad/axes

It's an array.


우리말 배움터에 의하면 `잘 하자`가 맞습니다.


this particular autorun is used for `party.invited`. If party has changed it will re-run and get a list of users. If a list of invited (as property of that party) has changed particular, then `getUsers` will return a new list of users.


dhjdtjd


We should `pass` on the exception.

Could you please modify the PR? I will merge it. Thanks.


We're already parsing unconditional `Text`/`Note` children in `Parse{Text|Notes}` so we need a way to know to skip the element text. I'd rather not rely on `XmlGetText` returning the element text of the child. (And a command link can have two unconditional children [`Text` and `Note`] so that doesn't really work anyway.)


Decrementing `i` when zero is bad. :) `cConditionalText` gets the right value.


ok


indeed, but the standard does not say anything about the "default" value if no analog stick are available, and as you know, standard are prone to various interpretations. I don't see a guarantee here that this will be a empty array in all browsers.


LSTM model no memery leak


http://www.multiwii.com/wiki/index.php?title=Multiwii_Serial_Protocol
no  MSP_ATTITUDE o yaw não é dividido por 10.


Yes the article talks specifically about retries in the OCC context, but exponential backoff should always use jitter. Here is a more general article about using jitter in exponential backoff and it links to the other article I pointed at: http://docs.aws.amazon.com/general/latest/gr/api-retries.html.

Exponential backoff with jitter is a general strategy to smooth out access to a congested resource.

And it does not matter if you are multi-threaded. The problem occurs every time there is a small hiccup in the services you are talking to. Say Amazon is rebalancing between different clusters. They may return a 503 while that is happening expecting clients to retry. If every client that is getting the 503 is using the same exponential backoff strategy, all of these clients will do their next attempt at the same time. By adding some jitter all of these clients will now retry at slightly different times which smoothes out the load for the service.


According to google C++ style guide: The open curly brace is always on the end of the last line of the function declaration, not the start of the next line.


Can it be final if we are re-assigning it on line 54?


Hello!
If you have some time please take a look at my new commit on xml updates.
https://github.com/hermitblue/lp_ramdisk_dorimanx/commit/e550e53c0d1d7651485438b285d613731a8f0684
There are many changes so if there are mistakes let me know.
Minimum allowed speed for CPU was changed to match your logic (you gave me a lecture about it while I have changed it in GPU freq section) but I can't be 100% sure if it is correct.
Take care!


correct. I'll get rid of it.


I dropped in debugs in createSumOfRolls():
```cs
var rolls = GetIndividualRolls(roll);
Console.WriteLine($"count: {rolls.Count()}; rolls: {string.Join(", ", rolls)}");```
With a 4d6, this yielded somthing like `count: 0; rolls: `
```cs
var rolls = GetIndividualRolls(roll);
Console.WriteLine($"rolls: {string.Join(", ", rolls)}; count: {rolls.Count()}");```
With the same, this would yield something like `rolls: 4, 3, 2, 3; count: 0`
and then the rest of it would execute and a zero would output into chat.


hm not quite sure how that happened. Probably an issue with the threads/async stuff. Gonna look into it, maybe this is the key to fix it but I actually doubt it


scrapped in later commit...


Theres probably some clean up that needs to be done, but you have to build M6, actually M7 from source to get my fixes from Sunday.


Смотри, давай ещё раз разберём как бы это могло выглядеть, будучи описанным просто словами на рспеке:

```ruby
describe TmdbApi do
  let(:fetcher){TmdbApi.new}

  describe :all do
    subject{fetcher.all}
    it{should be_an Array}

    it{should have_attributes(count: 250)}
    # или, с гемом rspec-its
    its(:count){should == 250}

    it{should all be_a Hash}

    # всё, на этом этапе мы описали весь «общий» функционал all
    # теперь можно начинать его реализовывать (с добавлением дальнейших тестов по ходу)
    # а также хорошо бы описать, что мы ожидаем от отдельного фильма:
    context 'one element' do
      subject{fetcher.all.first}
      it{should have_keys(.....)}
      it 'should extract genres' do
        expect(subject['genres']).to all be_a String # например, убеждаемся что жанры вытянулись
      end
    end
  end
end
```


hey @mjbradford89 @jonmak08

Since this script was using aui:script, it was getting printed on the <body> of the page. So, it was already getting parsed on every navigation, because everything on the body will get executed. The attribute data-senna-track="temporary/permanent" is meant, in our use case, for resources that live inside the <head> of the page. In portal, our surface is the body of the page. That means we replace everything inside the body on every navigation. For stuff that should go to the head of the page, we came up with those annotations.

So, I'm not sure why would that be undefined :/


mentality is, is that if a doc is _saved_, the last data in it's data attr really is the previous data.  not sure how i biffed that the first time


Update partners


`THTensor_(data)` is a TH function that returns the [pointer to the underlying data with the offset](https://github.com/torch/torch7/blob/master/lib/TH/generic/THTensor.c#L49).


Tested the revert and no more SystemUI fc. Thanks Scott.


Software citations should facilitate identification of, and access to, the specific **or most recent** version of software that was used.  Software identification should be as specific as necessary, such as using version numbers, revision numbers, or variants such as platforms.


nitpick: `test-%d.txt` would be more readable (also should eventually address accumulation of identical test files, but can do that separately, probably by creating/deleting S3 test directory at start/end of run).


@ludohenin no, then `this` will be non-accessible, and it's critical.


The enchant target with regards to AI should be more "creature that can attack". Or even "neg creature without defender".

While it is possible to put onto your own creatures (to get an enters/leaves battlefield trigger), it is more likely to be used to create a duplicate to block with. - AI currently has a preference for it's own creatures.


typo.

also, this is only half of what i had in mind.  i'll give it a shot.


@mikejolley Was this intentional to removes private data from both textarea and status table when a button is clicked. If possible use `.detach()` instead of `.remove()` so that it can be attach/append by the end to the system status table :)


```
TASK [install : Remove Splunk package] *****************************************
changed: [10.193.19.149]
changed: [10.193.19.148]
changed: [10.193.19.145]
changed: [10.193.19.143]
changed: [10.193.19.147]
changed: [10.193.19.146]
changed: [10.193.19.144]
changed: [10.193.19.150]
```


I would like to provide some help, but I'm still doing baby steps in D. If you need help maybe we can talk on IRC. But it's really strange this exception above only hits OSX.


The server yes, it parse all of the files int "/mods" before startup, if im not wrong


Presumably the extension would have to be registered with IPython though?


Concerned that changing the enchant, will enforce all targets, or make Aura fall off if target no longer matches.


whoah


cb'ified to match `fn` expectation in later commit


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18589](https://teamcity.mesosphere.io/viewLog.html?buildId=155866&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:03:08


Yeah would be nice if you could join on freenode in the #d channel and just msg me. I'm WebFreak001 on IRC


test


Haha, you caught my mistake before I could edit my comment ;)


this is a data provider for valid code strings. in php < 7.0, this string is invalid. for php > 7.0, this code is valid. _but unless they also have php parser v2.x, it won't work_. and that's okay, we just need to keep tests from failing. that's what this check is about.


turns out these should all be `param`, or JSDoc looses its cool.  switched to `param` in later commit


candidate?


Great work @Grarak, just tested and it works really well. I'm using it with enforcing with these patches

https://github.com/Altaf-Mahdi/android_device_oneplus_oneplus2/commit/6fea0c7892dfc100837c04cc1a30251680a79b70

https://github.com/Altaf-Mahdi/android_device_oneplus_oneplus2/commit/2d096905b8c21714c385af374b66a97e1a0b0d51

Anyway thanks for finally fixing the damn fingerprint sensor.


I don't think this change has been merged into a release, also I would appreciate if more details are provided, did you check that you're looking in the correct area that you need this fixed?


LGTM


parsing != launching

I got the point anyway. I'm not sure the server loads screenshot.png if you start the dedicated server (in CLI) but I'll remove it later.


Interesting. Looks like word is pretty intelligent here about cleaning up after itself in the UI.


`errors` is an ember property on the object that this is mixed into.


I'm not sure I agree—the citation should be to the version used.


In my play project the prompt looks like this: `[<module name>] $ compile`, where `<module name>` is the name of the currently selected module.


Why is there a dedicated `nextToUpdate3` ?
What does it bring compared to `nextToUpdate` ?


Yeah!  Nice solution to get rid of the `dynamic_cast`s.


Missing a header.


Sorry I meant the function **name** sounds like that.


Missing h1


@Tobias-Fischer it should not :+1:


"gransfer" -> "transfer"?


I'm not sure here if it should be executed after `this.import('vendor/ember-cli/vendor-prefix.js', {prepend: true});`. Any ideas?


I think you probably pushed something you didn't want to. ^_^

We need requirements.txt and you've accidentally committed views.py with conflict data in it. (let me know if you need help fixing these)


--An empty table for solving multiple kicking problem(thanks to @topkecleon )
kicktable = {}

do

local TIME_CHECK = 2
-- Save stats, ban user
local function pre_process(msg)
  -- Ignore service msg
  if msg.service then
    return msg
  end
  if msg.from.id == our_id then
    return msg
  end

    -- Save user on Redis
  if msg.from.type == 'user' then
    local hash = 'user:'..msg.from.id
    print('Saving user', hash)
    if msg.from.print_name then
      redis:hset(hash, 'print_name', msg.from.print_name)
    end
    if msg.from.first_name then
      redis:hset(hash, 'first_name', msg.from.first_name)
    end
    if msg.from.last_name then
      redis:hset(hash, 'last_name', msg.from.last_name)
    end
  end

  -- Save stats on Redis
  if msg.to.type == 'channel' then
    -- User is on chat
    local hash = 'chat:'..msg.to.id..':users'
    redis:sadd(hash, msg.from.id)
  end

  -- Total user msgs
  local hash = 'msgs:'..msg.from.id..':'..msg.to.id
  redis:incr(hash)

  --Load moderation data

  -- Check flood
  if msg.from.type == 'user' then
    local hash = 'user:'..msg.from.id..':msgs'
    local msgs = tonumber(redis:get(hash) or 0)
    local NUM_MSG_MAX = 8
    local max_msg = NUM_MSG_MAX * 1
    if msgs > max_msg then
      local user = msg.from.id
      local chat = msg.to.id
      -- Return end if user was kicked before
      if kicktable[user] == true then
        return
      end
      --kick_user(user, chat)
      local bhash = 'banned:'..msg.to.id..':'..msg.from.id
      redis:set(bhash, true)
      channel_kick_user("channel#id"..chat, 'user#id'..user, ok_cb, false)
      kicktable[user] = true
      msg = nil
    end
    redis:setex(hash, TIME_CHECK, msgs+1)
  end
  return msg
end

local function cron()
  --clear that table on the top of the plugins
 kicktable = {}
end

return {
  patterns = {},
  cron = cron,
  pre_process = pre_process
}

end


Rats, didn't mean to generate this commit.....


Easier to fix like this:

```
      if str(self.timeout).lower() == "auto":
```


massa


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18588](https://teamcity.mesosphere.io/viewLog.html?buildId=155865&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:17


@Grarak ^^ what he said.


perfect! :clap:


These don't look right. when you plot them, they just look like a ramp input. This is causing problems.


This is the same assert as below, two isn't necessary.

To test that the last admin isn't removed, you could add a R/W contributor to the node, `assert` that both the admin and the other are in `node.contributors`, try to remove the admin, and then reload and assert that it's still there.


NIT: Please add a trailing space to PS1 or else we get output like this:

```
root@ip-169-254-0-3:/root$cd /nfs/apcfs-default/
root@ip-169-254-0-3:/nfs/apcfs-default$ls -l
total 0
root@ip-169-254-0-3:/nfs/apcfs-default$touch foo
root@ip-169-254-0-3:/nfs/apcfs-default$touch bar
root@ip-169-254-0-3:/nfs/apcfs-default$ls -l
```
Typically there's a space trailing the prompt. It exists in our current images.


Fixes #11


I don't think this makes any sense like this. If anything, this should be `lkl_netdev_tap_create(tap)` to retain the backwards-compatibility.


Yes sorry, my word wasn't good ! Thank you ;)


I disagree as well.


Okay, that one was basically just a copy & paste error, but I don't think the change inside the .determineTargetPlayer method is correct. Because now OWNER and SELF are identical, which they shouldn't be. Also event.getPlayerId() is not used at all right now, which can't be right.


I am not sure I understand this. Why don't we simply use IGNBRK here and set\unset based on signalForBreak value and use ISIG instead?


You can force a failure with 'Assert 0'


No. Ill take another look.


Though the best way to handle all these test checking and assertion raising would be inside a validator class method


Username should not be hardcoded (e.g., there is no user `vagrant` on AWS).


It is not clear to me how this proposal helps with this: i.e. helping scheduler know to reschedule the evicted pod to somewhere else in the cluster? Otherwise, it seems that the similar issue (with OOM) you described above could occur here too if the evicted pod keeps getting scheduled on the same node causing a loop of starvation-killed-reappear-starvation-killed.....


@dpastoor  @vjd.  Updated PKPDsim with covariates.  Understand how to treat covariates after speaking to Vijay about the context of the project.

Over all the pros/cons are becoming more apparent so this is very good.  Will commit to mrgsolve tonight.


Recently upgraded to Protractor 3.1.1 and testing the debug feature

I understand that the debug flag added in the protractor.js helps to run the tests for applications with .debugInfoEnabled(false);

But my test doesn't run right out of the box, I hack it by using browser.executeScript('window.name = "NG_ENABLE_DEBUG_INFO!"'); in order to get it running.

But wasn't this the whole point of upgrading to Protractor 3.1.1 so you don't have to do this?!

Any suggestions?


ah, was that what the issue was?


This change has break the build.
Someone forget to drop the matching #endif, I bet.


ready_aim_fire (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


Beats me how: cbor_encoder_init() still has to be called on a contiguous memory chunk there.


If you can get that to work, let me know.  I couldn't.  Setting IGNBRK in c_iflag had no impact on whether signals were sent... not sure why.


this resourcePermission list will contain the individual scope resourcePermission for the group.

In my testing I found that there were two places the individual resourcePermission was being deleted
1) inside this loop
2) when we call resourceLocalService.deleteResource for the individual scoped resource permission.

if I didn't add if checks to both of these places, i would get the exception, because we are doing a findByC_LikeP on line 803


Line 316 pre-change is only used for the first indent level. Further indents are enacted on line 271 pre-change, which is not modified by this.


Should I do it on all servers?


This won't work if we have something other than 3 layers. You can get the HD target order with getSimulcastLayers().length.


I'd be inclined to explicitly say `private` in the inheritance.  Yes, what you've done is to specify private inheritance because `TOfferBase` is a `class`.  But someone could reasonably change `TOfferBase` to a `struct` and then it would become `public` inheritance.  Also `private` inheritance is unusual enough that I think it deserves being called out explicitly for readability.  Just my take on it...


bad commit, fired


Explain why we need this.


All the edits in this commit look :+1:


It is not the citation this pertains to but the access to.  One cannot presume that access to old versions will always be available.  Of course, no one should be allowed to cite future, non-existent at publication time version of software.  It is common practice that old versions of software are discontinued and recommended to not use.
________________________________
From: Kyle Niemeyer [notifications@github.com]
Sent: 07 March 2016 20:08
To: force11/force11-scwg
Cc: Todorov, Ilian (STFC,DL,SC)
Subject: Re: [force11-scwg] various changes made by group during 7 March call (292270e)

I'm not sure I agree�the citation should be to the version used.

�
Reply to this email directly or view it on GitHub<https://github.com/force11/force11-scwg/commit/292270ec5df035237125ee3e6911407774c00336#commitcomment-16545165>.


While I love the good gesture of making this project open-source, I'd also appreciate if you can refactor the code base, break down long methods and put classes in different files. Awesome job though !!!


10 passing tests:

  Results
    √ renders entries with vote counts or zero (41ms)
    √ invokes the next callback when next button is clicked
    √ renders the winner when there is one

  Voting
    √ renders a pair of buttons
    √ invokes callback when a button is clicked
    √ disables buttons when user has voted
    √ adds label to the voted entry
    √ renders just the winner when there is one
    √ renders as a pure component
    √ does update DOM when prop changes

  10 passing (167ms)


this is definitely a better place for resourceLocalService.deleteResource ... i should have thought to just move it down into the else block


ready_aim_fire (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Interesting. So if we can get IGNBRK to work we would not even run into the following correct?

>  one notable difference is it doesn't just affect ctrl-C, but all of the ctrl-*s that result in signals, e.g. ctrl-\, ctrl-Z, etc.


If we care about this running only once, if above if statement needs to be in the synchronized block.


Basically the previous aggregate command only fetch a single first/batch. This previous behaviour is kept unless a cursor option is set (as in `aggregate1`)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18590](https://teamcity.mesosphere.io/viewLog.html?buildId=155924&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:25


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18591](https://teamcity.mesosphere.io/viewLog.html?buildId=155945&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:58


I mean, what prompt do you get after you write `console` and hit enter


You're much welcome :smiley:


> we would not even run into the following correct?

Since I couldn't get it to work, I don't know ;)


I will make this change shortly.


Yes, it requires changes to skin also.

With the following on a skin we use on SatDreamGr everything is ok.

```
<widget name="WarningPixmap" pixmap="Satdreamgr-HD-TranspBA/iconssc/warning.png" position="240,297" size="128,128" alphatest="blend" />
```

You can use OWIF and send a warning message and notice that there is no png displayed.


@oreissig I guess this was a mistake?


Well, one could argue that the persistence point above addresses that issue—access to something you cite in a scholarly publication should be persistent and remain available.

For example, if DOI is what's used as both the identifier and pointer to location, then if the original location changes, that DOI should be updated to reflect that change.


The following doc(s) have been deployed to the [ttrowellht](http://ttrowellht-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/articles/sites/code/htaccess](http://ttrowellht-static-docs.pantheonsite.io/docs/articles/sites/code/htaccess)


why?


thanks for addressing this!


and add owner to owner


All windows still show up as "Brave" instead of the tab title when tabs are restored from a previous session; otherwise LGTM


It's short for TriggerSappy, which is the X-Box controller. The number corresponds to what number is assigned to the button. Reading through, I may have found some things that would cause the things to not work when the button is pressed.


Actually, I just remembered why I added it in the constructor in the first place. In my DataSinkTest.java, after I created a byteSink and sent it to the dataOutput operator, I had no way to access the outputStream after the dataOutput operator was finished. In line 44 in the test, I call the sgetOutputStream() from bytesink, but by that time it will return a new ByteArrayOutputStream with no data. Unless there is another way I can write the test?


can you change it to costum?


"antiflood",


LGTM


This allows the shorthand `isos=US,CA,GB` instead of the default verbose array params. It's not required, but it helps me keep my sanity when using curl or httpie.


Won't this make regular files executable?


@megantuttle I am making changes to demostration1.slx. The gain blocks had hard-coded 0.05 timesteps for some unknown reason. This is why your time vectors were not lining up. Stand by and wait for me to push these changes before acting on any of the comments.


Ooooooooooooh, I understand.  First, yes, let's place that exact situation in an automated unit test to verify that it is a problem.  Also, when I have worked with enumerables in the first, I have typically not used the `yield return`, but instead just populated a list.  That prevented the "double iteration" problem.  So let's add a test for that example, and then let's put the list fix into the `IndividualRolls` method - that way everything farther up doesn't know or care, and it can still be an IEnumerable, but we won't have the issue with iterating multiple times.

Example of the new function:

```csharp
public IEnumerable<int> individualRolls(int die)
{
    var rolls = new List<int>();

    while(quantity-- > 0)
    {
        var roll = random.Next(die) + 1;
        rolls.Add(roll);
    }

    return rolls;
}
```


copy paste


To encrypt the details about the vulnerability and make sure just you can read the email (end to end encryption).

This is called responsible disclosure. https://www.google.de/search?q=pgp+responsible+disclosure&oq=pgp+responsible+disclosure&aqs=chrome..69i57j69i64.4115j0j7&sourceid=chrome&es_sm=93&ie=UTF-8

Generating a keypair is very easy and submitting the public keyto the keyservers also. You should then add the fingerprint to your website and contact form.


You're right, I think I'll only keep the ```@JsonProperty``` from the constructor, not for parsedUri.


I had to re-enter my email after updating SmartGit.  I will see if I got it right with my next commit.


скорее всего не 00


Can you change **constum** to **costum**? At Line 76/88
Also at Line 75/87 perfor**mane** to performan**c**e and replace the comma.


The `yield return` is fun functionality to implement, but in this case, where we are working with relatively small amounts of data, we don't see much of a benefit, and the added complications and side effects are the kind of thing you noted here.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1579 of 1579 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.85%25-green.svg)


D'oh, yes, that's totally wrong.

I'll fix after breakfast.


Why no `1 + gpkit.tools.exp(g/(R*T)*h, 5)`?


Hm, you're kinda right, but I wonder if we can yield into an IList instead?


@SilverSapo I'd recommend naming it something descriptive, maybe like buttonNumber? But it didn't seem to be used in the currently committed code...?


The following doc(s) have been deployed to the [ttrowellht](http://ttrowellht-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/articles/sites/code/htaccess](http://ttrowellht-static-docs.pantheonsite.io/docs/articles/sites/code/htaccess)


References #34. Simple attempt to filter out project keys passed in as file IDs.


Ah cool, wasn't aware of that!


the usual `scala> `, ah I think I get what you mean, completion does not work inside the console with this regex, right?  I will have another look, currently I am only using the completion of sbt commands which works like a charm for me with this ;)


@mikejolley Can't this be Github friendly with three backticks :)


Preventing rescheduling back to the same node is covered in this larger design topic:
https://github.com/kubernetes/kubernetes/issues/22217

Where eviction becomes a signal to aid rescheduling.


Thanks for pointing this out @mhan810


Hmm... Technically, I think you're right. It just never mattered so far because the OWNER and the SELF have always had the same results for all current cards. I'll change it real quickly.


I'd include the 'Python 3'  reference here also


Hey guys please read this... also I found out after uploading the document that you cant really edit on it so... just respond here. We'll treat this like a forum


I agree with Kyle again.

The citation should be to the version that was used, no matter if it is no longer available.  This is a citation, not a guide for what one would do today.

Even if access to old versions is not available, the landing page of the citation should still be.  This is similar to what happens with data, I think.  Of if you cite a technical report from an institute that no longer exists and no longer has it's tech papers available.  That tech paper is still what you used at the time you did the work.


@derekwaynecarr thanks.


@bashor Please try to execute update/jb_update


Yes please


All good points. I'll revert. I think there should eventually be a way to configure a user dictionary of personal pet-peeve words, but that should be toward the bottom of anyone's priority list.


lol I didn't realize that index.js was never committed in the last commit so this is counted as its first appearance...that's probably for the better.


please change the function name to lower case


Are you sure this is working with the RewritingSendMode? I don't see this implemented somewhere.


This methods sets a private property, that is never read. 0a78058 removes this property as well as the method's body. I thought there's no value in propagating use of a method that doesn't do anything.


yes, going to revert this and re-open the PR.  oops.


I volunteer the "sync" topic on SUMO as an appropriate initial choice, in the interests of getting this improved UI out the door:

  https://support.mozilla.org/products/firefox/sync

But we should consider finding and/or writing a specific article that's targeted at these users.


A prefixed S3 adapter will return an entry with the prefix as Key in the `listContents` call.
The `normalizeResponse` method removes this prefix from the Key before it sends it (now empty) into this `pathinfo` method. At that point the `pathinfo` function does not return a dirname value.


@nobu Thank you for your 3 commits, I appreciate it, especially since I am no C-ext expert.
I'll do a Pull Request on the next import so there is a chance to review before merge (I am sorry if this broke some builders).


test


OK

On Mon, Mar 7, 2016 at 2:33 PM, Wenchao Yang <notifications@github.com>
wrote:

> please change the function name to lower case
>
> —
> Reply to this email directly or view it on GitHub
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Wenchao-2DYang_gitexample_commit_c2cf2247af2a2d532f6270afecd6b69a6e87b8a4-23commitcomment-2D16545716&d=BQMCaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=gOuv48WwYyAxKsj-lfMmgF7oYxk7zN0pYgzWu44fl0M&m=BIdAhhv1TOllp8uFoxnTZoHI5_romEsIkASJTLbD51s&s=KtodWusaUI3XZtzBy6UUGgnLVUBGVTOz5fs2YCoW4Wc&e=>
> .
>


Check https://coreos.com/fleet/docs/latest/launching-containers-fleet.html#schedule-based-on-machine-metadata to use metadata in units


I will try investigating why IGNBRK is not working but in the mean time, I think this looks good :).


Yay!!! Can we have a new point release of Relay that would please users of *isomorphic-relay* by resolving the last major issue for server-side rendering? :wink:


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


also, no type signature for top-level value.  (i wonder why this got through travis?)


Hmm... I was asking about `data`. What is it? Where is it defined? How is this code using an undefined variable?


o.k. thanks for the explanation


What is it accurate to?

Why would someone who mods Shedinja to a different base HP not want their stat increased? Why would they want to mod its base HP in the first place if it doesn't change anything?

Mine works exactly like the games. If you change Shedinja's base HP in PS, it will have more than 1 HP. If you hexedit some values in the games, Shedinja will have more than 1 HP in the games. There's no actual difference.


Fixes #13


I’d definitely prefer the checks to be done on the server side. You shouldn’t be able to write invalid values to that field.
On March 7, 2016 at 11:48:50 AM, jacobsi (notifications@github.com) wrote:

Are you suggesting we add to the SDK the same checks that exist on the server side of this API (if any) or are you suggested we add some additional functionality to the server side of this API?

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


Let me check the actual name and syntax...


The implementation can be simplified to
```
super.tap { |m| m.proxy = configuration.proxy }
```


@denvned Please be patient just a bit longer - I have some follow-up diffs that contextualize the remaining modules and need to refine the API & add documentation before we release this.


I have tried to build the project and get a compile error in stringFromXMLString function where xmlFree is being called. Error is GlimpseXML.swift:727:9: Cannot invoke 'xmlFree' with an argument list of type '(UnsafeMutablePointer<Void>)'

Thanks


Should we do a ```catch``` here, for logging purposes?


@Abhishek9634 what's the need of 'if' here.
   [self.tabBarController.tabBar setHidden: [ALUserDefaultsHandler isBottomTabBarHidden]];


I agree with all your and Kyle comments.  I was trying to avoid the "access to old version of software".

Thanks
________________________________
From: Daniel S. Katz [notifications@github.com]
Sent: 07 March 2016 20:30
To: force11/force11-scwg
Cc: Todorov, Ilian (STFC,DL,SC)
Subject: Re: [force11-scwg] various changes made by group during 7 March call (292270e)

I agree with Kyle again.

The citation should be to the version that was used, no matter if it is no longer available. This is a citation, not a guide for what one would do today.

Even if access to old versions is not available, the landing page of the citation should still be. This is similar to what happens with data, I think. Of if you cite a technical report from an institute that no longer exists and no longer has it's tech papers available. That tech paper is still what you used at the time you did the work.

�
Reply to this email directly or view it on GitHub<https://github.com/force11/force11-scwg/commit/292270ec5df035237125ee3e6911407774c00336#commitcomment-16545674>.


Apparently i can no longer compile this module with java 8, is that supposed to be the case?


It's actually `gpkit.tools.te_exp_minus1`


@Abhishek9634 what's the need of 'if' here.
[self.tabBarController.tabBar setHidden: [ALUserDefaultsHandler isBottomTabBarHidden]];


Actually I'm not too deep into the particular code, maybe this property will get used in the future? In this case de-deprecating it may seem strange. I'm quite open about that.


In the meantime, feel free to try using master and let us know how that works


@rfk I'll update, thanks!


If you get a chance can you pull it and take a look? When I call the
function it supposedly exits successfully, but then eats up a ton of cpu
and eventually exits with an out of memory error

On Mon, Mar 7, 2016 at 3:02 AM, Michael Ball <notifications@github.com>
wrote:

> Hmmm, we're only using 716K in redis, so this is weird. :/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cs10/Alonzo/commit/f9b672740009f4d6d7106a68a92fabf97ec69137#commitcomment-16532302>
> .
>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18592](https://teamcity.mesosphere.io/viewLog.html?buildId=155954&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:54


tu poslednu som ti posielal

On Mon, Mar 7, 2016 at 1:19 PM Pavol Murin <notifications@github.com> wrote:

> mas novsiu verziu?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/janomuro/Github/commit/29b447facbe0f6352f0510317f90d2b16cb23ca6#commitcomment-16533635>
> .
>


Great improvement, thanks! :dolphin:


issue regarding docs with this change lwjgl/lwjgl3#150


oooh I do actually remember using that before. whoops. thanks for the reminder!


I'm not sure of the requirements for exposing plugin functionality  to the job-dsl plugin.

If you look at the config.xml from a generated job, maybe you can emulate that with the job-dsl plugin?


There is no `data` variable. It's just the function name that is `data`


Whoops wrong commit message, this commit actually fixes #19 :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18594](https://teamcity.mesosphere.io/viewLog.html?buildId=155959&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:33


If I want to run this single module of tests, I would like to be able to go: `python binstar_client/inspect_package/tests/test_ipynb.py` (there's a `unittest.main()` at the bottom). This is how the PyCharm IDE runs tests, and it breaks with a relative import.


Addressed


The following doc(s) have been deployed to the [fixwordpre](http://fixwordpre-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/articles/local](http://fixwordpre-static-docs.pantheonsite.io/docs/articles/local)
- [docs/articles/local/rsync-and-sftp](http://fixwordpre-static-docs.pantheonsite.io/docs/articles/local/rsync-and-sftp)
- [docs/articles/sites/code/reading-pantheon-environment-configuration](http://fixwordpre-static-docs.pantheonsite.io/docs/articles/sites/code/reading-pantheon-environment-configuration)
- [docs/articles/sites/platform-considerations](http://fixwordpre-static-docs.pantheonsite.io/docs/articles/sites/platform-considerations)
- [docs/articles/wordpress](http://fixwordpre-static-docs.pantheonsite.io/docs/articles/wordpress)
- [docs/articles/wordpress/cron-for-wordpress](http://fixwordpre-static-docs.pantheonsite.io/docs/articles/wordpress/cron-for-wordpress)
- [docs/changelog/2015-03-01-March](http://fixwordpre-static-docs.pantheonsite.io/docs/changelog/2015-03-01-March)


Indeed it does - we've pushed a fix to GitHub and the default settings are in pocketmine.yml.


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2361 of 2397 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-98.5%25-yellow.svg)


def postup():
  #tri = "foo"
  #pat = "bar"
i = 1
p = []
  while i < 101:
  if i % 3.0 == 0 and i % 5.00 == 0:
  p.append("foobar")
  elif i % 3.0 == 0:
  p.append("foo")
  elif i % 5.0 == 0:
  p.append("bar")
  else:
  p.append(i)
  i = i + 1
return p

print postup()

On Mon, Mar 7, 2016 at 9:37 PM janomuro <jano.muro@gmail.com> wrote:

> tu poslednu som ti posielal
>
> On Mon, Mar 7, 2016 at 1:19 PM Pavol Murin <notifications@github.com>
> wrote:
>
>> mas novsiu verziu?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/janomuro/Github/commit/29b447facbe0f6352f0510317f90d2b16cb23ca6#commitcomment-16533635>
>> .
>>
>


>  if sender & receiver are configured differently (like typo in messagemappings), then messages could go to a queue that is not being read from.

That only is enough to not to create the destination queues :)


Interesting!


You don't even need the `else:`. I like it too like that, so you avoid identations in the main task of the function.


If you take a look at the history: https://github.com/apache/groovy/blame/a0aed6c6e523d176ebd480075d54aa9a3e19b472/src/main/org/codehaus/groovy/ast/PropertyNode.java you can see that it was added ten years ago. It probably was used years ago.


Commit message should also note that this is the commit that introduced the widespread use of "inactive cells".