Yes, I thought so too!

I didn't want to take the time to recreate it, but I'm thinking that either the Word developers gave up trying to figure out a more elegant solution or that possibly the "Link to Previous" can only be turned on or off for all three possible headers and footers at once, at least in the UI.

So this will make it important to do the experimenting to work out whether Word supports inheritance that is severable by header/footer type or whether it barfs if the XML has say an even header defined but no default header in a middle section.


Test classes in fj are usually suffixed  by ```Test``` rather than prefixed.


I'll post a followup, thanks


Cocopods requires that the version should be greater than zero. So if you have your react-native dependency pointing to the master repo it will fail when you `pod install`. Could we set the version to 0.0.1-master?


I've seen Chromium breaking on Debian, I think it will happen in Vivaldi also on some distros. I am still looking into the problem, and a fix will apply to all browsers derived from Chromium.


hey, could you add some explanation to the commit message? This commit is important and the reasoning is also, a commit has to be understandable without looking up the issue


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


thanks for your contribution and welcome btw :)


Is it working? :O


Should I then add some javadoc like this?
```
/**
 * @deprecated not used anymore
 */
```
It's strange that it is a setter-only property anyway.


Thanks, Paul.


Instead of undefined "exit"   use this, to exit normally:    process.exit()


sure man! no need for bower :+1:


this is an odd file. it doesn't seem to be a general-purpose scraper, but it's named `cogs.utils.scraper`
it also uses camelCase


@tyronegrandison can we leave the PIF eagle in the code?  I know this is a CDS site, but that's the PIF easter egg for posterity...


I think you have logic error here. The do block for super, not for the resulting model. How about this
```
super(accoun_name, group, options) do |url|
  url << "&validating=" << validating if validating
end.tap { |m| m.proxy = configuration.proxy }
```


@Aangiix3 I tested it, it doesn't work yet (You can try to login with a 1.9 client, but it is kicked due to a invalid packet size), but that's because it isn't finished... yet


Ah, good to know! So that's probably enough support for images for a start. It sounds like that would support, say, adding a logo to a header or something. We should probably add that to the list of use cases just below, adding a logo image. We can show an example in the documentation somewhere. I'm sure that question will come up once a week as soon as we publish this feature :)


there is a very odd mix of single and double quotes in this method. also, there's no reason to use `__len__()` over `len()` or even just nothing. I'm not sure why you are using `lower()` either
````python
if subreddit.strip():
```


This wasn't exactly what I suggested in my original comment (https://github.com/AppFolioOnboarding/perkins_eidolon/pull/36#discussion_r55168221). With this change, you've lost the `btn btn-danger` look.


The retry loop needs to start a little further up so it picks a different port number each time, right?

Also, should there be a retry loop for the commissaire server as well, since we pick a random port number there too?  (I swear I'm not trying to be difficult!)


Do you think this is enough to warrant a new release?


# **PING, @zudov **


"[...] that  **are** used by coala."


You have completely ignored both:
>Similarly, hardcoding Shedinja makes it easier for mod developers to create Pokemon with a base HP of 1
and
>Someone who mods a Pokemon with base HP 1 should get their HP increased by the stat formula

Yours does not work like the game because of the above. Mine does. Hex editing the game to change Shedinja's HP is equivalent to changing this code. Yours is not equivalent to the game.


Not a huge deal either way. I like spaces a little better because it makes the code a little cleaner on GitHub, and it prevents alignment issues when switching editors (e.g. vim).


26, 27, and 28 can now be on the same line.


putting a `return` above and getting rid of this `else` would reduce the overall indentation. you can do the same thing for the category validity check


Well for almost all cards. I introduced OWNER specifically for Blessing of Wisdom. It draws for the owner of the trigger, which may be different than the owner of the minion. Power Word : Glory uses the same mechanic, but those are the only cards I can think from the top of my head.


add braces {...}


LGTM


This is fine, but in my original comment (https://github.com/AppFolioOnboarding/perkins_eidolon/pull/36#discussion_r55168712), I was actually suggesting asserting the presence of the `can't be blank` error message associated with the URL input.


This part cannot be applied.   We need to have the code there that finds a valid network interface.   On my Mac, if this patch is applied, all the UDP based tests fail.

`java.net.SocketException: Can't assign requested address
	at java.net.PlainDatagramSocketImpl.join(Native Method)
	at java.net.AbstractPlainDatagramSocketImpl.join(AbstractPlainDatagramSocketImpl.java:179)
	at java.net.MulticastSocket.joinGroup(MulticastSocket.java:323)
	at org.apache.cxf.ws.discovery.WSDiscoveryClientTest$1.run(WSDiscoveryClientTest.java:92)`


You are not being difficult :smile:. I'm somewhat distracted by looking at the next task which means it's easier to make mistakes on the first. Updates coming.


Already done: https://github.com/apache/groovy/commit/83891b6dc09be0a40ce1ce7cb2a48f2524ac43e1 but thanks for the offer. :)


I fell foul of this too: start of today plus time until 'now' is just 'now'! Replace with:

```
const time_t end = time(NULL);
```


Shouldn't the label be 'GPlus'?


Why? `has-error` is what this class represents.


Are these the same old ones or with edits?


More params that can now be on same line.


there already is in the "fold", the diff is probably hiding it from you.


Ah, true. I just assumed SELF and OWNER worked exactly the same because it always seemed to.


I don't see a method `envx_proxy` in my documentation for LWP::UserAgent. I see `env_proxy` though.

I'll merge and test/fix as necessary.


fast-forwarding 2.0 to borshop-integration-821-basho/bugfix/riak-2383-alt = 24012f4b


I would be less qualitative, say we have "more than 50 bears", that sounds more impressive :)


Better:

```scala
scientificName.canonized(showRanks = false).getOrElse(lastNameInChecklistTaxonPath)
```


Great, well done :+1:


What is the underscore at end of line for?


ts('Activity Date')


I'm not entirely sure about subclassing ECS. A lot of things happen inside ECS and I feel it's written specifically for Java Mode. I had not put a lot of thought back then into how it could be subclassed and reused in the future. So if it were to be subclasses, the Mode would need to use it carefully.

How about moving createPreprocessor() to the JavaBuild class instead? Preprocessor is something that is more fundamental to the Mode and is used primarily while building the sketch. ECS and ASTGenerator only need the preprocessor to get the list of core and default imports. These import statements are returned by non static functions, so a preprocessor instance is needed.


And maybe not writing "we", be more general-audience-like ^^


There's a subtle difference here: if we specify the explicit offset on these then the time is no longer bound to an actual time zone with DST rules.

If this test is run in different zones (with `ENV['TZ']`) it will fail.


Is shell kinda like Batch or?


hm "powerful"... I would say "famous" or "well known" or something like that, "powerful" is an adjective I would give our bears :)


Any specific reason you are first disconnecting the containers before connecting?  This causes a problem with Windows Containers as we don't support disconnecting running containers from the network currently...


@kytrinyx this branch has been hanging around a while-- can we delete it?  Has it been merged?


And I would do "It" -> "coala-bears", since I have lost track again after 4 sentences what "It" is exactly^^


Just a comment (no need to change anything), this looks like it's going to be a source of subtle bugs: people taking control of the buffer, forgetting to update `used` and complaining that their data is not being sent.


but but


At the moment, there are a few feedstocks which are overriding their run docker build script - my aspiration is for a feedstock to never have to do that, but at the moment, that isn't the case (core linux dependencies).


why not return ResultCode instead of List ? I see below you have different scenarios that the caller may want to handle differently then just throwing Exception for all cases ?


Chicos ya solo falta lo de validacion


Maybe write in brackets "e.g. English" or so? Not that someone thinks this is actually a programming language...


Im still getting Uncaught ReferenceError: jvm is not defined with the latest commits?


Hey, where are you?


@sils1297 since coala gets beta, maybe we want the bears also being beta? I would go for it^^


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

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


I found it easier to do it directly in the controller, since it essentially
serves the same purpose of keeping the view files simple, without having to
write an extra view helper.

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

> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rakeshbhatia/bloccit/commit/511750eb40f517588f33e57ebfd66c2aea487868#commitcomment-16527624>
> .
>


Currently the senders map is not backed by a thread-safe implementation, so accessing it like this might cause problems. This is just a suggestion but maybe we could move this whole method in SimulcastSenderManager.getSimulcastSenderByTargetOrder()?


@travi No problem! Out of curiosity, are you using any other hapi libraries for session management or authentication? Looking to build an Auth0 hapi example but just started using it a few weeks ago.


like on gitter, remove this weird
> The source code will be provided by coala so the bear doesn’t have to care where it comes from or where it goes.

:)


I think you need to add, has_many :band_performers


Is it safe to remove `@InputFiles` and just rely on `candidateClassFilesHash`?


`herein[\w]+`
:smile:

Interesting resource: http://app.aspell.net/lookup-freq?words=hereinabove%0D%0Ahereinafter%0D%0Ahereinbefore%0D%0Ahereinbelow;also=similar;parm=normal


Hmm we need more text than, "A bear is meant to do some analysis on source code" is not that much.


Now your argument for making this API buffer-aware is stronger :) OK, TODO++, it won't be forgotten.


@abhinavagarwal07 We have to return a symbol dict kind of thing


Is that still needed when the manylinux patch is integrated into the docker file?

A different workaround would be to add a `linux_addon_packages` config variable.


> because they are needed in the CI script generation

The only one that is actually needed is the TravisCI one though (the rest are part of the respective APIs and aren't in the circle.yml/appveyor.yml.

The sources and targets is reasonable, but should default to the existing defaults.

recipe_dir is a feature I'm not currently prepared to add to the smithy.

Remembering the point of conda-forge is to foster a *community* of recipe maintainers, I'm not convinced giving the ability to allow users to manage feedstocks is healthy/desirable, and certainly isn't something I'm eager to maintain (I personally have 0 feedstocks of value in my user space - I encourage you to do the same and just *use* conda-forge).


Oui...
edde669559a4725aafab68b2712d727ea57ad70e


@megantuttle should not be motion_des here. Should be accelerations!! you can calculate these using diff() outside of this loop, right after you run the simulation. same can be done with the "alpha" variable, which still needs to be changed from 0.05 timestep (not correct). I changed the simulink in the newest commit to use 0.01 fixed time, which is also what the file used.


A very valid point. I suspect we will end up with a policy stating that you should use the image provided to you, and not depend on other RPMs, at which point, there is no real need for the feature.


Change to deploy/nginx/magpie_api.cong


oh yes totally agree! We're in beta!


Супрематизм какой-то :)


Just make it 20..


Please add a note that this "feature" is not expected to work over streaming transports in general.


> Hex editing the game to change Shedinja's HP is equivalent to changing this code.

Why is it? Why do you care whether or not it's equivalent? What does that have to do with accuracy to the game?

PS isn't a direct copy/paste of the game code, it's a reimplementation in Node. There will be differences. It's fine as long as the difference doesn't affect game mechanics, which it doesn't.


Is isWindows built into Go or should we use the Go preferred version?


Please update this message.


Hex editing the game to change Shedinja's base HP and removing the Shedinja special case is exactly the same as changing Shedinja's base HP in PS. They're equivalent. It's accurate. What's the problem?


That's true!


See at the end of this PR...


Yes, we have to do this to add the container short id as an alias. The order has to be:

1. Create a container, connect it to a single network. If we don't connect it to any networks we won't be able to connect later (I believe).
2. Disconnect it from the networks so that we can reconnect with the proper aliases.
3. Connect to all networks (one at a time), with the correct aliases, which we didn't have in (1) because we didn't have a container id yet.

If networks provided the container short id as a default alias, I believe we could avoid this.


Ok.


I agree that our strictest requirement for mechanics compliance should be faithfulness to Hackmons.

Whether this is a good or bad change, that's debatable; however, the OM community is already confused regarding Shedinja, mainly because mods aren't defined as a finished product, but as the ``"standard game with/without some bans, and a discrete set of additional rules"``, and OM creators prefer saying ``"Shedinja will work like in the games (whatever that means)"``, than saying ``"Shedinja will work according to the rules following [...]"``

We should just adopt an interpretation of the underlying mechanics that best fits our interests. In case of conflict, adding a flag should be preferred imo, as long as it doesn't add much complexity to our data, and if it allows us to unhardcode stuff, all the better.


Be better IMO to just explicitly do the test inline, took me awhile to see this definition.


Hmm, and 2 months ago, this PR was sent to LogBlock: https://github.com/LogBlock/LogBlock/pull/655


Done at https://github.com/xwiki/xwiki-commons/commit/cea134fa85137cfecadbbab2d2aa608e4a18ce79

Thanks.


Hadn't been running at 120Mhz


In this instance, I can live (just about) with spelled (shudder :stuck_out_tongue_winking_eye:), but please don't test me with color vs colour :smile:

http://www.oxforddictionaries.com/definition/english/spell


Sounds good. Agreed that string comparison cost in this context is not a concern ;-)


Hi,

basically everything is working except the encoding of the chunks. The problem is that the way chunks are sent and stored in Minecraft 1.9 changed completely and the changes are not well documented.
I tried out a lot of changes today, but it's still not working. I hope that I can get it working tomorrow :)


Perfect. Thanks @JanSchulz.


@kappaOne Thank you!


Ok. Will update


At some point I'm planning on removing the `asBool(double[] a)` method from `BooleanSimilarity`. It was originally designed as a utility but tripled the run time, whereas the build method handles it in one pass of N.


Fixes #14


is this to lazy
or can we extract a bit of data from the AGD list?


Great, thank you @simon04. I will take a look and merge it this week.


Deleting temp file


hey mnaczenski,
today I ran into the problem the table aliases being still uppercase. Without correction you'll get a "column not in field list" error in mysql


OK. Renamed to makeParamsForTransactionRequest()


Oi...?? Concatenando string dentro do [sprintf()](http://php.net/sprintf)!?!?

![](http://i.giphy.com/U4VXRfcY3zxTi.gif)

ehehehe...


Updated.


I don't know if erroring out is the correct way: what if the socket has no concept of packets? (TCP for example)

I don't have a better suggestion for now, so, just adding a FIXME may be enough.


Good point:)  Will fix it


Good, not mention it.


Hey mate, i reckon we still have issue in this method, if we want to add a publishing page and then set required properties for it. Method will still fail trying to check in page before setting those required properties. any thoughts ?


we decided to use candidate until the next stable is released, instead of deploying just one image to stable now (since we got the new image after the last stable deploy).
@rchalooyala promised he'd remember to change it before the stable release.


Although not documented (it's an internal function), the use of that API should always end in that function ``sol_oic_packet_cbor_close()`` being called anyway, so I don't see that much of an alarm.


This should be `Ram ram`, since we shouldn't depend on the implementation, just the interface


Mon: 3/7
  breakfast: 1/2 chalah bread ft. w/strawberries; coffee w/1% milk;
     shake w/1/3 banana.

   snack:  coffee/shake 1/2 nutrigrain bar;
   lunch:  little toasties w/lobster bisque
Excercise:  35 minutes biking ( 3 trips );  35 minutes walking.
   Dinner:  ???


@bgoodri, it's now up and running for me. Do you have a set of tests you
want to run on it?

There's at least one visible change that I've found: the timing information
printed to screen lost its "#" on each line. I can try to track it down if
you feel like you want to keep the behavior. (I stopped before doing that
already cause I'm a bit tired and hopefully there aren't bigger issues to
tackle first.)

We'll talk tomorrow at the meeting?

On Sun, Feb 21, 2016 at 12:00 PM, Daniel Lee <bearlee@alum.mit.edu> wrote:

> On that branch, the folder doesn't exist:
> StanHeaders/inst/include/lib/cvode_2.8.2
> There's this one: StanHeaders/inst/include/mathlib/lib/cvode_2.8.2
>
> For RStan, we need to pass this include folder to the C++ compiler:
> StanHeaders/inst/include/mathlib/lib/cvode_2.8.2/include
>
> We don't need the CVODE shared library to use everything in the Stan
> language except for integrate_ode_cvode(). If we want that function to be
> available, we'll need to build a shared library for CVODE, but I suggest we
> get the first part working first.
>
> I couldn't track through the includes to the StanHeaders to find out where
> to include the cvode folder in the RStan package.
>
> Want to point me in the right direction?
>
> On Sun, Feb 21, 2016 at 11:55 AM, bgoodri <notifications@github.com>
> wrote:
>
>> That branch should already include the CVODE headers in
>> StanHeaders/inst/include/lib/cvode_2.8.2. Are you saying StanHeaders needs
>> to build the CVODE shared object? That would probably be okay although no
>> longer header-only.
>>
>> On Sun, Feb 21, 2016 at 11:52 AM, Daniel Lee <notifications@github.com>
>> wrote:
>>
>>
>> > Ok, so we have a problem with CVODE.
>> >
>> > Ultimately, we need to include the header files from CVODE + one
>> additional
>> > header file that we added.
>> >
>> > In the math lib, the headers take up 1.9 MB. Do we have the space to
>> > include that? If not, do we need to do a similar thing as BH for CVODE?
>> >
>> > That's just the includes. If we want to use CVODE, we'll need to have
>> the
>> > library built too.
>> >
>> >
>> >
>> > Summary:
>> > How do we include CVODE headers?
>> >
>> >
>> >
>> >
>> > On Fri, Feb 19, 2016 at 9:56 PM, bgoodri <notifications@github.com>
>> wrote:
>> >
>> > > You can do
>> > >
>> > > ./sh_b.sh
>> > >
>> > > or probably just
>> > >
>> > > R CMD build StanHeaders && R CMD INSTALL StanHeaders_*.tar.gz
>> > >
>> > >
>> > >
>> > > On Fri, Feb 19, 2016 at 9:44 PM, Daniel Lee <notifications@github.com
>> >
>> > >
>> > > wrote:
>> > >
>> > > > Can I get a little help? Is the build instructions on the wiki
>> > somewhere?
>> > > >
>> > > > There isn't a makefile in StanHeaders.
>> > > >
>> > > >
>> > > > On Fri, Feb 19, 2016 at 9:28 PM, bgoodri <notifications@github.com>
>> > > wrote:
>> > > >
>> > > > > You need to install StanHeaders_*.tar.gz first but it should be
>> > synced
>> > > > with
>> > > > > the developed branches in prepare_for_2.10.0.
>> > > > >
>> > > > > On Fri, Feb 19, 2016 at 9:26 PM, Daniel Lee <
>> > notifications@github.com>
>> > > > > wrote:
>> > > > >
>> > > > > > @bgoodri, I'm on that branch and it builds just fine on my
>> machine.
>> > > > > >
>> > > > > > After updating my submodules recursively and changing to the
>> rstan
>> > > > > > directory, this is what I got:
>> > > > > >
>> > > > > > ~/dev/rstan/rstan (feature/prepare_for_2.10.0)> make build
>> > > > > > R --vanilla CMD build rstan --md5 --no-build-vignettes
>> --no-manual
>> > > > > > * checking for file ‘rstan/DESCRIPTION’ ... OK
>> > > > > > * preparing ‘rstan’:
>> > > > > > * checking DESCRIPTION meta-information ... OK
>> > > > > > * cleaning src
>> > > > > > * checking for LF line-endings in source and make files
>> > > > > > * checking for empty or unneeded directories
>> > > > > > * looking to see if a ‘data/datalist’ file should be added
>> > > > > > * adding MD5 file
>> > > > > > * building ‘rstan_2.9.1.tar.gz’
>> > > > > >
>> > > > > >
>> > > > > > Do I need to update StanHeaders first or something?
>> > > > > >
>> > > > > > On Fri, Feb 19, 2016 at 5:09 PM, maverickg <
>> > notifications@github.com
>> > > >
>> > > > > > wrote:
>> > > > > >
>> > > > > > > @syclik <https://github.com/syclik> while you are at it, can
>> you
>> > > > take
>> > > > > a
>> > > > > > > look at
>> > > > > > >
>> > > > >
>> > >
>> https://groups.google.com/forum/?fromgroups#!topic/stan-dev/4Gd4cy2xEW4
>> > > > > > >
>> > > > > > > —
>> > > > > > > Reply to this email directly or view it on GitHub
>> > > > > > > <
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16215157
>> > > > > > >
>> > > > > > > .
>> > > > > >
>> > > > > > >
>> > > > > >
>> > > > > > —
>> > > > > > Reply to this email directly or view it on GitHub
>> > > > > > <
>> > > > >
>> > > >
>> > >
>> >
>> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16218321
>> > > > > >
>> > > > > > .
>> > > > >
>> > > > > >
>> > > > >
>> > > > > —
>> > > > > Reply to this email directly or view it on GitHub
>> > > > > <
>> > > >
>> > >
>> >
>> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16218326
>> > > > >
>> > > >
>> > > > > .
>> > > > >
>> > > >
>> > > > —
>> > > > Reply to this email directly or view it on GitHub
>> > > > <
>> > >
>> >
>> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16218386
>> > > >
>> > >
>> > > > .
>> > > >
>> > >
>> > > —
>> > > Reply to this email directly or view it on GitHub
>> > > <
>> >
>> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16218452
>> > >
>> >
>> > > .
>> > >
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16227673
>> >
>>
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/stan-dev/rstan/commit/900c66be59d8029ee242e4d613aac5a8051781bc#commitcomment-16227689>
>> .
>>
>
>


@teschnei ..I didn't notice till now. Certain mobs have multiple special melee, one example being iron giants. Previously I was able to make retail accurate Iron Giants that did all 3 of their melee swing animations by giving them all 3 skills and flagging those as "special" in the database, and setting this to true. They were automatically cycling or randomly using all 3 melee animations (I forget if it was an order or random, but it worked all 3 were being used).

If I have a mob with more than one special melee like that now, do I have to use a math.random() with an if/else tree or something now? I'm just looking to update some old mobs that I haven't shared back because the places they go have pretty much nothing yet.


Also, I don't know if it will work when switched back. I just infinity loop killed myself with a set of secrets somehow. Really confused what happened there...


:cake:


It's not been merged, but I don't really care anymore. I was trying to normalize them so I could script something, but that train has left the station. I'll submit a PR with the actual change I want.


The [Travis build error](https://travis-ci.org/GlotPress/GlotPress-WP/builds/114329186#L183) should be fixed by https://github.com/xwp/wp-dev-lib/pull/151


disregard the db tests stuff. thats still in progress, but not hooked up anywhere to cause errors.


Fair.


Oops.


The code does not comply to PEP8.

*PEP8Bear, severity NORMAL, section `autopep8`.*


Line contains following spacing inconsistencies:
- Trailing whitespaces.

*SpaceConsistencyBear, severity NORMAL, section `python`.*


I think it would be better to do:
```
${optionalString !(features.chromiumos or false) ''
  MEDIA_PCI_SUPPORT y
''
```
instead of adding `?` and then disabling it in other place. This should also prevent rebuilds.


Oh, yeah, now I see it. Eye for an Eye killed me by activating off the damage I did to myself. That's... That's not good.


Fair...


Update to pipe drivers


update to pipe drivers


Bootstrap 4 uses has-danger instead of has-error, but simple_form expects Bootstrap 3


pipe drivers


test


typo


Woot! Go team vim!

I looked at Riv and it seemed cool. But I've never really dug in.

It's actually syntastic that's the slow part. When you have this:
```
let g:syntastic_rst_checkers = ['sphinx']
```
it will recognize `highlight` as a valid directive. But it takes like 2 seconds to run on my machine.

This rst syntax checker is the default:
```
let g:syntastic_rst_checkers = ['rstcheck']
```
it runs basically instantly. so i use it!


Ok, I then wont ask if you are living in the US :-)


we don't want to leave `LOG_TEST`s in, right?


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


This is the only new thing in this commit, I just realised that the previous had a completely bogus bitmask.


If you have alu tboost, why u add touchboost into hotplug?


@gblair This commit breaks when `brew update && brew upgrade` as the openssl version has been bumped.

```
Randalls-MacBook-Pro:~ clark$ /opt/boxen/homebrew/opt/openssl/bin/openssl version
OpenSSL 1.0.2g  1 Mar 2016
Randalls-MacBook-Pro:~ clark$ brew search openssl
homebrew/versions/openssl101                                                              openssl
```


Good catch!


So far I'm just using [`hapi-auth-cookie`](https://github.com/hapijs/hapi-auth-cookie) for the session piece. It's simple and covers my needs so far. I would say thats a good place to start if you're looking to create an example for others to reference.

The other piece that I would love to see covered in an example like that is configuring a CNAME so that the redirect to `auth0.com` doesn't look like a phishing attack to those unfamiliar with auth0, but instead uses my own domain. Configuring the CNAME on my side is simple enough, but I don't find anywhere to set it up the auth0 side in the admin tool. This is a really simple config for things like heroku and GItHub pages, so I would think this would be an important part of the auth0 offering as well.


Go into where you can see your installed packages and click on import-js. There should be settings in there with a text input for this.


Hackmons compliance is pretty low priority, too. But yeah, this particular line is already Hackmons-compliant, so that's not even an issue.

Like, to put it another way: Hex editing the game to change it to Fire Emblem will make it Fire Emblem. Hex editing PS to change it to Fire Emblem will make it not run because ARM machine code isn't valid JavaScript.

Why are you drawing arbitrary equivalences between hex editing the game and editing PS?


I thought there is no need to support JSONLoanAccount as input, LoanAccount is typically used everywhere and JSON version can be used just for internal implementation, But we can of course always add if you think it's needed.


I can re-add this but remove the content of `run_docker_build_matrix.tmpl`. But I would really like to avoid it if this could also be fixed by a new docker file :-)


This can be only `some_text`


Document what scale input (ie, is it a scale in flux space)


Did you really want `Stack`?  `ArrayDeque` is meant to be more performant.


puts "Hello World!"


should I have double quotes around `${odb_linux_compiler_url}` ?


Looks good. r=chrafuse


Yes. Thanks.


y u no use full name? :trollface:


> This can be only:
 \`some_text\`


Code guide lines state if you are just checking against None you should say if scale, i believe


Sunt singurul care iti scrie, dar mai am 5 prieteni care il folosesc
zilnic plus prietena care nu are nici o ideie ce face dar reuseste sa
ajunga la postul Tv pe care eil vrea. Cat ai timp ia si te ocupa de el,
ar fi pacat sa il lasi sa moara.
Sincer cand am vazut ca nu mai e in AUR  m-am panicat ca nu o sa se mai
ocupe nimeni de el, ma bucur ca m-am inselat.
Spor si tinem legatura.

On 07/03/16 17:23, calvarr wrote:
>
> Ma bucur, din păcate nu are cum sa fie totul perfect, și nici nu e
> ocupația mea de baza... :(
> am scris la youtube-dl și au openload în lucru, videomega au zis ca
> este suportat însă la mine nu merge. Oricum ma bucura ca mai folosește
> cineva tvmaxe-cli...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/calvarr/tvmaxe-cli/commit/897fa7bc46cfdbc69af7a51d7717739a09dbbd6f#commitcomment-16537822>.
>


LOL, so it's a simple macro?


`Tuple` is missing its generic parameters


@paulbarber OK, so I guess `EcfWrapper` is only needed to bridge the gap from Java to C? Sorry to bother then. It's too bad we can't do without it altogether.


This line is missing a semicolon.


@rezzon8 you need to use promises here. https://docs.angularjs.org/api/ng/service/$q so it should be something like

`apiservice.login('admin', 'admin').then(function (response) {
}, function (error) {
    console.log(error);
});
`


Ah, good catch.  I'll update the commit message when rebasing to also update the message on the swappable property.


Look at how the wyrms use their flying attack. Check mobskills lists and
their script


Missing semicolon.


Missing semicolon;


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


...again `Stack` or `ArrayDeque`?


OK, I'll rebase and update the commit message.


haha I dunno.. I could replace all entries to use my fullname I guess, but the others are arthurnn.
Maybe I should replace it all. Dunnolol


How come this got removed?


Yes

Sent from my iPhone

> On Mar 7, 2016, at 4:14 PM, Thomas Peters <notifications@github.com> wrote:
>
> we don't want to leave LOG_TESTs in, right?
>
> —
> Reply to this email directly or view it on GitHub.
>


yay


@rezzon8 this will not work, instead set a $scope variable and print that out.


@rezzon8 Use the following url: `.constant("PROJECT_SERVICE_BASE_URI", "http://projectservice.staging.tangentmicroservices.com/api/v1/")`


Expose ``shardsToClean.size()`` as a jmx counter.


Asserting a particular error message when there's a generic one available kind of seems too strongly coupled to the model, especially since simple_form displays the error message from the model validation.


Hmm, I tend to prefer to leave it fail  (that is, no < dep) and have an open bug blocking the tracker: that way, ffmpeg-3 wont be unmasked until it is fixed


@rezzon8 you must return the deferred.promise; otherwise you will not get the data and when the async call returned


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1967 of 2130 tests passed, 185 test warnings)

 * 185 tests had: Table small diffs.
 * 163 tests had: Table big diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.

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


AppVeyor does secure variables now that aren't pre-registered? That is cool.


Yes, ASSIGNED is Mambu's default. (As per MBU-11769: "ASSIGNED -> returns the users assigned to the branchId. Default value if this parameter is missing")


and what about libav then ?


Yeah, thats where I am at..I commented right on the line number in the script. This used to be true/false and now specifies the ID, so am asking if I have to if/else it now.


А на сколько такое расширение оправдано?


Did you look at the code in both drivers? I guess not. Please do before writing comments in github.


@rezzon8 once you get the response you can resolve it. `deferred.resolve(response);`


I like the little trash can :) Without the <button> tag, the proportions of the 'btn btn-danger' span come out different, and the button looks squashed.


Updated version of NDPMMetric.java


+1 for sans-#;  I don't even like it in Jupyter/knitr.


- Nouveau format supporté : Chordpro, qui est maintenant le format recommandé.
- Création de l'outils patatools
- Renommage de `sorted` en `sort`
- Nouvel élément : `setcounter`, `addsongdir`
- LaTeX
    - Renommage du mot-clef `cov` en `cover`
    - Suppression de vcov
- Plein d'améliorations diverses


After reading http://redux.js.org/docs/api/combineReducers.html#notes, I believe it is fair to say that:

1. If `insert` is called with `(present === undefined)`, there has been a violation of the Redux Reducer guidelines.

Therefore, we should most likely remove the `if (present === undefined)` from the `insert` method, or even throw an error.


Isn't the slash at beginning redundant?


I like the little trash can :) Without the button, tag, the proportions of the 'btn btn-danger' span come out different, and the button looks squashed, so I just dispensed with the btn completely.


that is to fix the double initial scan, now first time all blobs time stamp will be smaller than lastSweepCycleStartTime so no notification will happen, at the same time the logpoliing strategy will still do the initial scan


Shell is bash.

I'll make a gulpfile to run at some point I guess :P


@rezzon8 you do not need the variables between [].

This structure is sufficient.

    .controller('MainCtrl', function ($scope, apiservice) {


Ah, I see. Fundamentally, this PR is about conda-build-all. Despite writing it, I don't agree with using it *inside* the CI systems and prefer to be using it as CI matrix items instead (as they currently are) - this gives you a nice overview when debugging issues.


Asserting a particular error message when there's a generic one available kind of seems too strongly coupled to the model, especially since simple_form displays the error message from the model validation.


@rezzon8 should probably be renamed from getMe to getProjects() ?


трябва да се въведат 4 последователни еднакви думи.


fix for test assumption that the polling interval is 2 seconds, for the new strategy it is 10 seconds


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.698.1e0c28909d0b86eba67666c2d2d4a562d10ffedd](http://build.palaso.org/viewLog.html?buildId=69991&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3427, ignored: 126 Build time: 00:06:15


Plese don't forget cancel that


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1057.1e0c28909d0b86eba67666c2d2d4a562d10ffedd](http://build.palaso.org/viewLog.html?buildId=69989&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3407, ignored: 107 Build time: 00:06:51


This is script should be:
\`\`\`ruby
  gem install ruby
\`\`\`
Live some indentation.


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1022 of 1022 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.58%25-green.svg)


Sorry, I can't see the line from my phone. If it's where I think it is, it
gets a mobskill LIST and not a mobskill ID, shuffles that and picks one
mobskill to run.


Any methods that aren't called from outside the class don't need to go in the header file.


No "branchId" is not a typo: CustomViewsService now supports 'by branch ID" filtering for custom views (MBU-7042).  Custom views can return data for the specified branch. if branchID is specified. This applies to GET clients too.


Is this still unfinished? I appreciate the comments letting people know, but can you somehow document the classes themselves as well? That way if someone is just doing a dir() they may be warned as well. If you are specifically moving this over such that you think it will still be useful to the future then by all means. If it was just historic in the hsc code, and is just being pulled over because of a cherry-pick, consider removing it


I'm not in any room right now..


this.cells.length <= (Math.random() * this.gameServer.config.playerMaxCells))
Randomize it a little for more... wel random bot play :P


Thanks man.


Yeah, I tried that before reverting back to boost::dynamic_pointer_cast.
Unfortunately, that didn't fix the issue.

On Mon, Mar 7, 2016 at 2:22 PM, Liam Staskawicz <notifications@github.com>
wrote:

> @locustcox <https://github.com/locustcox> as I mentioned on the PR (#12
> <https://github.com/PX4/sitl_gazebo/pull/12>), i think that is not the
> right fix, since then it will be broken for OS X as well (which was the
> motivation for the PR in the first place) :)
>
> can you try to add #include <memory> to make sure dynamic_pointer_cast
> has been included properly?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PX4/sitl_gazebo/commit/2ce086fc0512b8a0b82320ccb3cfb7ef35bfadd7#commitcomment-16544024>
> .
>


I see. I think `has-error` is a better name (it means what it says). I think you may have made this change in response to @aaronpanch's comment: https://github.com/AppFolioOnboarding/perkins_eidolon/pull/36#discussion_r55249575, but I don't think that's what he was asking you to do.


Porque lo pusiste en el commit si no es necesario para mostrar en time?


OK


`char pwm_pin`? Please use types from `stdint.h`
Is there a reason for this is plain char?
@OriginalNexus


This should be one sentence with
> fields ```peer``` and ```md5``` to ```trusted```


I have added  cmake command in the making your first component doc.


His point was that .has-error looked like it belonged to a particular component, but in Bootstrap it's a top-level class.


You don't have to assert the specific error message content, but you can do what you did here: https://github.com/AppFolioOnboarding/perkins_eidolon/commit/cfbb456b91dab83265ad1fd8ccfe747cb12ed2a0#diff-8807b315bee3099305dbdee3d4f0a4f7R29 to assert that there is an error associated with the field you expect to have an error.


Not works with Nivida Drivers 364.47
>{rsx::thread} RSX: Program destructor invoked!
{rsx::thread} Exception: Assertion failed! Result is C4650338h
(in file Emu\RSX\VK\VKTexture.cpp:162, in function vk::texture::create)
RSX: Program destructor invoked!


On the most places leave some indentation.


Are you saying you talked to @aaronpanch and he suggested this change?


Yes, we should take those out

Sent from my iPhone

> On Mar 7, 2016, at 4:14 PM, Thomas Peters <notifications@github.com> wrote:
>
> we don't want to leave LOG_TESTs in, right?
>
> —
> Reply to this email directly or view it on GitHub.
>


I think this should be an optional feature.


Did you look into using MongoDB's [copyDb](https://docs.mongodb.org/manual/reference/command/copydb/) command? It looks like you would have to run it via `runCommand` on the `admin` database, so there would be trade-offs, but it _might_  work better, (e.g. performance, copying of `system` information (like maybe indices), and matching user expectations better).

Unfortunately there doesn't seem to be any documentation of what it does, and how the result might differ from what you're doing here.

Also, [this SO answer](http://dba.stackexchange.com/questions/86919/i-can-not-copy-database-mongodb) isn't encouraging, and [this MongoDB ticket](https://jira.mongodb.org/browse/SERVER-701) hints at the complications.


TeamCity libpalaso :: palaso-precise64-lfmerge Continuous [Build 2.6.15.1e0c28909d0b86eba67666c2d2d4a562d10ffedd](http://build.palaso.org/viewLog.html?buildId=69990&buildTypeId=PalasoPrecise64lfmergeContinuous) outcome was **SUCCESS**
Summary: Tests passed: 3407, ignored: 107 Build time: 00:05:19


No.  I'm working on the Concord right now.  I think once that is in place, a release can go out.  I just wanted to get it into development in case anyone pushes code.  That way we can handle the conflicts easier.


I think this should be an optional feature


Classes in different files will not work for in this case. Some of those classes eg. ChatListAdapter use outer-class members and methods.
But thanks anyways. The code base will be cleaned up when we have achieved the aim: A functional ChatView library. We'll appreciate contributions and your discussions/feedback.


And note that it does check the return from onmobskillcheck or whatever
it's called now, allowing you to do things like shinryus tail whip. You'll
have to modify existing scripts


Using the assignment context works better with invariant array subtyping.
Did you try this change with the invariant array subtyping? It would be good to add a test to
https://github.com/typetools/checker-framework/tree/master/checker/tests/nullness-invariantarrays


> El mio no funcionaba por lo visto eso es lo que me ha dicho Miguel Ángel y tiene razón pero no se porque ¿?¿?

- que quiere decir "por lo visto"?

- La razón porque no funcionaba:
https://github.com/sos-2016-02/sos-2016-02/commit/1c65b59df8623659376b09dc42834a71f389a61d#commitcomment-16520313


OK, that's a fair point that my suggested change violates the expected semantics of a getter. Even if we want to lazily construct the `OutputStream` the first time `getOutputStream` is called, we should keep the result in an instance variable returned from subsequent calls to `getOutputStream`.


LGTM after you fix up the comment.


I think this can be 7000. I believe unknowns are ending with 0 in most cases.
these errors doesn't need to be incremented in powers of 2 unless there is a computational need for it.


In general, it would be good to add a test for this change...


No, the "i" here is really "dummyIndexToBeIgnored", maybe I should rename it. These generators do not read the size parameter anyway.


No.

The original intent here was to make the error message turn red. I wanted to override the default (Bootstrap 3) .has-error class (which isn't red), but since I'm using Boostrap 4, including that override actually amounts to introducing a new top-level class. But, the Bootstrap 4 .has-danger class turns the erroneous input field red, which was good enough, so I took out .has-error.


Hi Zenon,

In Django 1.7, South was dropped to use Django migrations instead.

Any DB history prior to that gets squashed into a new initial migration in the new system.

In Django 1.8, with the new Django migrations, ````auth.User```` evolved to extend the email address (from 75 to 254) and also adds ````blank=True```` to ````last_login````.  For auth.User, these are captured in stand-alone migrations.

When replacing auth.User with account.User, I first thought I'd create just one initial migration with the final version of the model.

However, this would not work for existing databases.  Because we are at the same time switching from South to Django migrations, we need to fake history on what was done.  Because the tables already exist, we need to run ````./manage.py migrate --fake-initial``` to fake the initial migration - and in some cases, also fake subsequent migrations.  E.g., because the ````edumanage```` package initial state is spread over multiple migration steps, we do it with full:

````
./manage.py migrate --fake-initial --fake edumanage
````

and likewise for account:

````
./manage.py migrate --fake-initial --fake accounts 0002_initial
````

But, here we explicitly state that accounts should have the migration history faked only up to ````0002_initial````.  At this point, we get the original tables created with Django 1.4 and with older versions of email and last_login - and this is when we actually apply the additional two migrations included in this commit:

````
./manage.py migrate --fake-initial
````

- which lets migrate go wild and apply any unapplied migrations (but still faking initial ones).

Hope this explains it - I'll add this explanation to this commit message and rebase+force-push.

Cheers,
Vlad


Is there a possibility that we might want to save 16/32 bit images too? If yes, then it is better to copy the load/save image native functions from arrayfire.


Thanks for the invariant array tip, I'll look into that.
>In general, it would be good to add a test for this change...
And that's why this is in a branch and not pushed to master.  I intend to submit a pull request before merging.


Yes, I knew about the JSONLoanAccountResponse class. Unfortunately, it cannot be used directly to deserialize Mambu's response, as it has somewhat different structure (in Mambu API response the settlement accounts are "inside" the "loanAccount") So the development team suggested extending LoanAccount in the wrapper.
BuI I think it's a good point that it would be much better to return a class that exists in the model already. I will update the method to return JSONLoanAccountResponse.
In a future, JSONLoanAccountExpanded and JSONLoanAccountResponse possibly can be combined by Mambu into a single class to be used everywhere for such responses,
I will also rename the get settlement accounts method as per your other comment.


No problem, I'll rename.


"sed" happens in workdir, (not deploydir) which is unique for every image. otherwise your tar command few lines below would be packing into one tar all possible images.


The conditional on `Me.Method.ReturnType.IsDefinition` didn't make sense to me. As far as I can tell, only `_isEnumerable` is relevant to the types we'll need.
I left it as-is for now.


What is this?


that's my fault, I mistyped and made a commit in a hurry.


Addresses #12


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.707.1e0c28909d0b86eba67666c2d2d4a562d10ffedd](http://build.palaso.org/viewLog.html?buildId=69992&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3407, ignored: 107 Build time: 00:05:30


It's important that we check the options too, especially the context.  It's very tempting to parameterize context, which leads to missing translation at runtime.


Okay.


Hey Andrew,

If I understand this code properly, this list of resourcePermission will include the permissions for each entity inside the group. For example if there were 1000 blog entries you would get one resourcePermission for each. And if I understand properly those should still be deleted.

What you are saying is that it also includes one resourcePermission associated to the Group itself. If that's the case there are two solutions:
1) Avoid deleting that one and only that one
2) Change the finder to avoid returning that one

I tend to like #2 more but you would need to check for other places using that method to check if they are expecting it to be included.


Sorry about that! I just looked at the email notification and the diff and didn't check that this isn't in master yet.


Thanks for the info @mhan810, I had not noticed.

@pei-jung go ahead and submit the pull to Brian.


using `require` syntax here since it seems to be relative to this file, not relative to wherever you started the script from. `fs.readFile` breaks if you start the server from outside this directory.


I think we can use 3-channels hex also right (0xRRGGBB)? I would recommend using that here. Most people who use hex will be used to that. Maybe add a comment that AA can be appended for alpha channel.


Actually, things started working for me after removing ''

include_options works, but not when I write 'include_options'.

Error: Could not parse for environment production: Syntax error at 'include_options'; expected '}' at [...]


maybe not intentional?


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2003 of 2168 tests passed, 185 test warnings)

 * 165 tests had: Table big diffs.
 * 185 tests had: Table small diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.

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


I notice most `fg_chart` variables here are not const-ified. Are they required to be non-const?


>> because they are needed in the CI script generation
>The only one that is actually needed is the TravisCI one though (the rest are part of the respective APIs and aren't in the circle.yml/appveyor.yml.

My mistake: I included the readme in the "CI Script generation":

* github is used for the URL to the badges
* channels for when you want to add/use your own conda channel
* the two BINSTAR_TOKEN variables are used in their two build scripts (appveyor and travis)

> recipe_dir is a feature I'm not currently prepared to add to the smithy.

That's not the multi-patch here, just to be used in the template. It could be used to change the recipe dir to something else. I added it here to then switch to the better named `recipes` name in the multi patch. It can be removed, would only need to switch all templates back to use `./recipe` directly.

> Remembering the point of conda-forge is to foster a community of recipe maintainers, I'm not convinced giving the ability to allow users to manage feedstocks is healthy/desirable

I don't think it's good to use technical restrictions to force users in one direction.

In my opinion conda smithy is mainly to help people setup conda channels for their personal projects, kind of like the personal ubuntu repos (PPAs). E.g. I suspect that the packages in the conda-forge need to be a lot more stable and polished than I intend to do with my private "packages builder" repo and my own channel... I don't want to see a pandas rc candidate in conda-forge, but it's fine in a `pandas` channel. If this use case isn't working with conda smithy (because I can't change the defaults), then I won't use it.

Another thing: the name says it's a generic command, if it is just for conda-forge, it should *not* be under the conda namespace but something like `conda-forge-admin ...`


Worked like a DOT in icbt2. No idea how it is now, though.
hxxps://www.youtube.com/watch?v=xFXACL00a9E


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

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


(this is just the most recently-added thing which does this; it applies to a lot in this file, especially in Run())

Why is this a 'running' state? It seems to me that these should be "failed" messages, since it would be very weird to see "Running: error pulling tool container ..." vs. "Failed: error pulling tool container ...". I guess this is to ensure there's only one failure message per job, which will generally be "Job exited with a status of ...", and the Running message above that (probably) tells you what actually failed. But that's pretty ugly.


So are the arrows bugged or its the trajectory not showing correctly?


I forgot to say I,ve commented it so maven would not compile it. I see no uses for myself.


Although the splitting of this large method was largely mechanical, there is one purposeful change: I removed this statement `F.CurrentMethod = moveNextMethod` which is useless.
The `OpenXYZ` methods that are executed right after it ignore and overwrite the value of that field.


I dislike "if conditions permit". It's too vague.

It would be more in line with existing Regulations to specify when the Delegate has discretion, e.g. listing typical such conditions.


Still visible... just pointing it out...


There is no managementServer OID in the CHECKPOIN-MIB.


Now i have this
>{rsx::thread} RSX: Program destructor invoked!
{rsx::thread} Exception: Could not find suitable memory type!
(in file g:\git\rpcs3-vulkan\rpcs3\emu\rsx\vk\VKHelpers.h:259, in function vk::memory_block::allocate_from_pool)
RSX: Program destructor invoked!


I'm not the one that brought up hex editing non-sense.


Since we are not 1.0 yet, any thought on reordering this?


You need to merge the latest commits from master (to get the gitignore) before the instructions work.


Oh ok.


I agree with @Claster that this makes no sense, because it states something in clear mathematical terms that we do not intend.

Also, I think it is not wise to introduce a new word like "subset" just to clarify "group".


Non-functional change.


As I've mentioned, I actually like what you had before better. It doesn't matter that `has-error` is no longer a Bootstrap class. I think Aaron was just pointing out that he wanted the `.has-error` style to be moved to another file.


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


Also, it's because of your code that Tier Shift had a broken Shedinja for at least two years.


You can add some more line about installation and add this to `README.md`.
You can add:
`bundle` - for installation of the gems
`bundle exec rake db:create` - for creating the database
`bundle exec rake db:migrate` - for running the migrations


Aesu's are not working yet, but please test other blocks. Explsions will come soon


Something went horribly wrong, really... This is gonna take some time to sort out.


Yes, it is.


My code follows mechanics and is relevant to OMs. Your code does not. As much as you can try to argue, your code does not follow any mechanics. It's just to suit your preference.


If you're going to remain stubborn, may I suggest @Marty-D decide on this.


Couldn't this have been fixed more clearly by just adding an extra close-paren?  If not, please explain what the problem was.


Could we add a possibly unique class here, like `bp-docs-content` or `bp-docs-container`? I was looking at #500, and scoping via `.bp-docs .description` doesn't work because the body tag also has the `bp-docs` class. :)


I'll look into the additional functionality.

gam print contacts should get all of your shared contacts. At my site:

gam print contacts

Getting all Contacts for rdschool.org

Got 25 Contacts...

Got 50 Contacts...

Got 75 Contacts...

Got 100 Contacts...

Got 125 Contacts...

Got 150 Contacts...

Got 175 Contacts...

Got 200 Contacts...

Got 225 Contacts...

Got 250 Contacts...

Got 275 Contacts...

Got 300 Contacts...

Got 325 Contacts...

Got 350 Contacts...

Got 375 Contacts...

Got 400 Contacts...

Got 425 Contacts...

Got 450 Contacts...

Got 475 Contacts...

Got 500 Contacts...

Got 525 Contacts...

Got 550 Contacts...

Got 575 Contacts...

Got 597 Contacts...

User,ContactID,Name,Family Name,Work Email,Work Email Primary,Notes,Given
Name,Additional Name

<597 rows of data>

What do you see?

Ross

On Mon, Mar 7, 2016 at 11:25 AM, espiman <notifications@github.com> wrote:

> Ross, are you planning to add more funcionality:
>
>    - Google+ API
>    - Rest of GMail API, like users.messages or users.threads ...
>    - Google API calendar, move events
>
> In gam print contacts, can I get domain shared contacts more than 25 to
> 25?
>
> Regards
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/taers232c/GAMADV-N/commit/c85351fff48ee8261fe7fec957561b96d4f177f2#commitcomment-16544097>
> .
>

--
Ross Scroggs
ross.scroggs@gmail.com


sdasfarst sef   tudor


Does this mean texture points will (silently) not be textured, or did VTK add support for that in some other way that's in use?

if it's the silent-no-texturing behavior, maybe add an #else assert(false) clause?  I worry that we'll have a lot of these kinds of things and might forget to go back and revisit some of them.


`bash -s stable --rails` => `bash -s stable --ruby --rails`


Please add it to Pods


That file misses new line but I'm gonna commit it anyway.


Yes, this commit switches the build to conda-build-all instead of hardcoding the build matrix in the templates.

Removing this PR will later remove the ability to run multiple recipes.


I finally learned how conditionals work in bash and how they differ from conditionals in sh (which is the case here). :smile:


a fix was already added here
https://github.com/diydrones/MissionPlanner/commit/fec12a5a48f4085d6f704c166779e48ffdc30517


I've been experimenting with adding my own parser and I think this should be `=== -1` and not `!-- -1` ?


I can't figure out where you added this initially... wish the whole PR would load on the actual PR screen. Anyway, it seems incorrect that it's a StatusDockerPullFailed when this isn't for a docker pull (it's for creating the container, instead). I do see there's a StatusDockerCreateFailed, maybe that's the intention?


Have a look at http://apaszke.github.io/torch-internals.html . It will give you some better understanding on how the templating mechanism in TH is done


Better remove that entirely


`sudo -u postgres psql` => \`sudo -u postgres psql\`


Semi-colon


Mostly because the rest of the backend services can't handle multiple Fail state messages and I needed to support the same messages from the previous version of the job execution framework. I'm hoping in the near future to remove this restriction.


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)


Sure, whatever you think is helpful.

On 03/07/2016 03:47 PM, David Cavins wrote:
> Could we add a possibly unique class here, like |bp-docs-content| or
> |bp-docs-container|? I was looking at #500
> <https://github.com/boonebgorges/buddypress-docs/issues/500>, and
> scoping via |.bp-docs .description| doesn't work because the body tag
> also has the |bp-docs| class. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/boonebgorges/buddypress-docs/commit/2318e093f82127d857f834b4d229404cd7c328d1#commitcomment-16547542>.
>


test


The match metadata is only required for astrometry.net.  It's rather a pain to have to carry it around.  Is there a way we can group it in with the matches?


Don't have to use template strings here, you can use singlequotes `'` instead (same for below)


@odadev das sollte die herauslaufenden Wassersäulen lösen


(in fact, I don't see StatusDockerCreateFailed used at all, which seems wrong)


I'm not entirely onboard with using long long


The variable was in there, just hardcoded?


Alright, makes sense.


single-quotes
tab instead of space between `data` and `=`? And at the end. Might want to check your tabs and spaces settings for your editor to make sure only spaces are in the code


Yes we need boomy boom!
On Mar 7, 2016 1:45 PM, "Modmuss50" <notifications@github.com> wrote:

> Aesu's are not working yet, but please test other blocks. Explsions will
> come soon
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TechReborn/TechReborn/commit/f966cdb63201bec28a4904241edba07856068938#commitcomment-16547514>
> .
>


The return type seems at odds with the name of the method. I'd expect false to be returned in error case.


The new method (addJsonObjectForFields()) was needed to support model changes for Disbursement Details in 4.0 (MBU-11223). Some fields (supported and needed for APIs like PATCH loan terms API (firstRepaymentDate and expectedDisbursementDate fields) in the new model were moved to be a level below: to be under the new "disbursementDetails" class member. The existent wrapper code was only extracting applicable fields going trough the "top level" class elements. It was now needed to support getting applicable fields defined within the class members too (to find fields like disbursementDetails.firstRepayementDate)


Use braces even for one-line if statement blocks. also, space between `if` and `(`


semi-colon


develop (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2364 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.62%25-yellow.svg)


spaces before and after `&&`, semi-colon


Explain in the commit message why this was necessary.


Same here!!


Who is that?

On 2016-03-07 1:02 pm, PieCoder314 wrote:

> This was not me. This was done by @RandomName2002 [1] and @TheBedrockWarrior [2]
>
> --
> Reply to this email directly or view it on GitHub [3].

Links:
------
[1] https://github.com/RandomName2002
[2] https://github.com/TheBedrockWarrior
[3]
https://github.com/ORCHARD-Minecraft-Mod-Class/RandomMine/commit/517d42305d9cbda00f52eaa8433131a7be207b23#commitcomment-16543580


spaces before and after `=>`


is this just for logging, or is there some logic missing?


We should avoid allocating closures in these helpers (it's created due to use of type parameter in a lambda).


Lots of fixes!!! :dancer:


The value of 1000 should not be hard-coded. It is either an argument of the method, or a key in ResourceMap (which could serve as the default value of the argument if you want to merge both approaches)


Oh, sorry, yes. The variable is encoded globally - it doesn't change *between* repos. I remember now.


Use parens to remove the need for the backslashes.
One usually breaks after an operator like `and`.


iiiii


yes! ;-)


Je vois que tu as choisi mon code ^^


a TODO: move to resources would be cool, just if a miracle happens and we have enough time


It seems like if this is required in order to have condor transfer files back, we should make it fail out the job if it doesn't work. Seems better to fail out early rather than get support messages later which say "it claimed to run fine, but it didn't actually copy back any files"


Looks Good to me. PRed


Before this pattern comes up too much more often: I just want to point out two things:

1. I really want to find a way to do this more elegantly. Something like #352 may be part of the solution although I need to dig and think more.

2. The pattern above is actually wrong from a compile-safety perspective, because it ends up invoking the signature `Computers.binary(OpEnvironment ops, Class<OP> opType, O out, I1 in1, I2 in2, Object... otherArgs` _even when_ the "out" argument is actually a class placeholder like `IterableInteval.class`. The culprit is using the ternary operator inside the argument specification, rather than an if/else OUTSIDE the method call.

Hopefully my correctness concern (2) above will be moot though once we accomplish (1)...


> I don't think it's good to use technical restrictions to force users in one direction.

You're right. I accept your very well put together argument. I've conflated the "I want to be able to do this in my own area" with the "I want to be able to do this with multiple recipes" and that was a mistake. Thank you for putting me right on that :+1:


@bboe, it doesn't seem like it's working for datacleaner and xrff2csv: When I perform my own local tests, it doesn't prompt me to update.


`UnsafeMutablePointer<Void>` would be more canonical, since it (a) highlights the lack of type information, and (b) keeps you from unsafely using `memory`. `UnsafeMutablePointer<Any>` is a pointer to valid memory with type `Any`, which is a totally reasonable kind of pointer.


Those 5 lines of code were the reason we can't have this addon in the workshop.  That's depressing.


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


It's possible for HTCondor to fail to transfer files, but for the transfers into iRODS to work fine. In other words, this can break and the job can still run successfully as far as the DE is concerned. Condor might not be happy, but as long as the job runs the DE can go merrily on its way.


Please don't cast it when there's no need to.


Good catch, I'll work on a fix.


Ah, okay; I forget those are separate steps. I guess it'd mostly just hurt our debugging then, which would be annoying but not breaking.


YMMV, but I think referencing concrete keywords will be more easily understood


Explain in the commit log why this is necessary.


hello, how do I install on my local server?


Can't this go on one line?  Otherwise, use parens.


/cc @saurabh500 @chcosta
You need to do the same on the 4.0 version of the contract that @saurabh500 forked.


excellent commit message!


Still can't believe this wasn't here before haha


Some of this commit overlaps with previous commits and should be orthogonalised.


I'm not sure if this needs to happen in this PR, but I think we should probably for safety only mount the three of these that aren't the working directory in if we need them, i.e. for backwards-compat. I'm pretty sure the current JEX only does it in that case, and it means there's many fewer things which can screw up the contents of those mounts.

In a related note, could they perhaps be mounted read-only, or do we think that's too dangerous?


Why reduce the limit?  This needs explaining in the commit message.


TeamCity Civic Assembly :: Build all [Build 89](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1680&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


will remove the solution file from the commit, but that should be the right way to not run the initial scan in logpolling strategy


We should probably compare to the argument sz passed in as well with the loop to make sure a bad cert with improper total_length is not going to cause a segfault.


TeamCity Civic Assembly :: Build all [Build 89](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1680&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:00:30


No backslash when in parens.


Nice!


K, will adjust.


good progress, but rpcs3 show fatal error after first image show


Having a check here against idx + length being less than total_size or sz before calling DecodePolicyOID may be good.


& instead of &


Why `new_for_commandline_arg` instead of just `new`?


We should be able to mount them read-only, I'll look into that.

I've been avoiding adding backwards-compat-specific code to road-runner because I *really* want that image to go away, but it might be impossible to avoid.


@goodwinnk, in most cases it works after update, thanks!
It still doesn't work for `FileUtilRt`, maybe `FileUtilRt` doesn't contain debug info.


I ran into this issue last week when flashing for the first time. I didn't grab a radio log, but I did check the Testing > Phone Info settings and saw that my network type was set to WCDMA only. Setting it to LTE/CDMA (I have the trltespr) and bouncing the radio off/on fix this and it stayed fixed after subsequent reboots.


Alignment of return statements is off in this file.


I was able to fix something related to RocksDB thread local storage before:
https://reviews.facebook.net/D43677


That's different behavior!

0.0.0.0 means "all IPv4 addresses on the local machine".


TeamCity Civic Assembly :: Build all [Build 90](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1682&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


I think it makes sense to only mount them as read-only, but I'm not sure if that would break any existing backwards-compat tools. Also, we need to mount `/data2` for tools that need Reference Gemones/Annotations/Sequences.


For some reason I can't build this new demo:
```
6 warnings generated.
E:\Working\Development\Tools\msys\tmp\glfw-6f41e2.o:(.text+0x6688): undefined reference to `file_browser_init'
E:\Working\Development\Tools\msys\tmp\glfw-6f41e2.o:(.text+0x7a95): undefined reference to `file_browser_run'
E:\Working\Development\Tools\msys\tmp\glfw-6f41e2.o:(.text+0x9949): undefined reference to `file_browser_free'
clang.exe: error: linker command failed with exit code 1 (use -v to see invocation)
Makefile:31: recipe for target 'demo.exe' failed
make: *** [demo.exe] Error 1
```


I'm -1 on this commit.


User can change the folders, check project properties


Could be replaced by Bool testResult(pValue > 1.0 - level)


@reedloden PR please? :)


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


:question:


No. Rails 3 is not supported anymore.


I really want to fix this, as it's also breaking ``send-once`` in the ``help-lint``. What's the proper fix?


@jakemac53 FYI... this seems to make Travis green


Also, I have GPS working, though it did take some effort in getting it to work initially. Wasn't seeing any satellites and I had to use GPS Status and GPS Data to force A-GPS data to download.


I failed to remove border from code highlight @daniel-acuna, @bluenex. Any recommendation?


Similar to the start case. Different implementation of `Restore` should be enough.


0.0.0.0 is not "all interfaces" it's "any one interface" right?


Nooooooo capitalization mistake


The lack of indentation here might break parsing


It's all interfaces!

https://serverfault.com/questions/78048/whats-the-difference-between-ip-address-0-0-0-0-and-127-0-0-1


Remote object in windows should work the same even if it doesn't have a mapping to a grpc connection. It would return a scoped client instance.


Here's an example of how to do that without rebuilding Trix: http://codepen.io/javan/pen/EPqzZo


You can bind to all interfaces using 0.0.0.0.


This probably needs to be bumped to 0.14.1 now, right @rafatower?


I'm using a static buffer here because it's hard to initialize a CudaIntTensor on nn. nn doesn't have any of the CUDA bindings, and normally we do input.new() for buffers. However, none of the inputs use CudaInt, so I'm not sure if it's possible to initialize a CudaIntTensor from Lua code on nn.

I provide a clearCudaState function to remove the memory associated with it, while gets called when doing a :reset.


Actually, I thought we only needed to mount these `/usr/localX` and `/data2` directories specifically for the `backwards-compat` container.

For other dockerized tools, if they require Ref. Genomes, then the tool installer adds `/data2` as a `container-volumes` setting specifically for that tool.


@reedloden & should be correct here I believe


FTR 65 out of 662 usages of `Jenkins.getInstance()` were correctly checking for a non-null return value prior to this change, so more than 90% of the callers were implicitly assuming the return value is non-null


I've reverted part of this one. The 1.0.3 release was made before they switched the build macros from mate to gnome. yelp-tools is pulled in as a dependency of gnome-common but not mate-common.


Not on Windows, 0.0.0.0 is meaningless there. So maybe a separate ipv4
make-sockaddr for windows?

On Mon, Mar 7, 2016 at 2:11 PM, John Benediktsson <notifications@github.com>
wrote:

> You can bind to all interfaces using 0.0.0.0.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/factor/factor/commit/2c82dfb51ba6923c2e7dcf3fac5109ae9da528f2#commitcomment-16548053>
> .
>


Yeah, I'm not sure. Arguably, mounting these in at all counts as "backwards-compat-specific code". Perhaps it'd make sense to just have a configuration option which is something like a "mount extra paths" list, which we can just have something higher up stick into the job definition? That way road-runner can know it needs to mount in an arbitrary list of extra system paths, without needing to specifically know the name of the relevant images where it needs to, etc.

I've been considering lately if it'd make sense to create a bunch of images `FROM discoenv/backwards-compat` for either more popular or less popular tools (not sure which) in order to remove the need for those tools to mount things in. I don't know how practical that is.


Here you should use sample.computeMean() and sample.computeStandardDeviationPerComponent() directly, as the substraction of a scalar and the division by a scalar are promoted to there NumericalPoint counterpart as a first step of the computation.


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


Would you mind @msalvadores if I removed your Eclipse IDE files from version control, e.g., .classpath, .project, etc?


ya real confused


Created issue EN-3587 to look into this.


Oh, if system paths can be added as container-volumes settings, can we maybe just run a database migration to add the relevant settings to any tool which uses backwards-compat?


Line contains following spacing inconsistencies:
- Trailing whitespaces.

*SpaceConsistencyBear, severity NORMAL, section `python`.*


Line contains following spacing inconsistencies:
- No newline at EOF.

*SpaceConsistencyBear, severity NORMAL, section `python`.*


The code does not comply to PEP8.

*PEP8Bear, severity NORMAL, section `autopep8`.*


@jvendetti  not at all.


Windows supports bind to all interfaces:

https://msdn.microsoft.com/en-us/library/windows/desktop/ms737550(v=vs.85).aspx


> special

Yessssssssssss.


Comparison is redundant. pWindow should fail if it is NULL.
Also, I'm not sure I entirely get the logic here. If pWindows is null, there is a new window being created. If it is not null then shared window is being returned.
I don't think it is very clear. It is not wise for C-style API to check the contents of the in-out variable. I would suggest having a separate flag.


Just to confirm -- this makes it so that hitting enter after editing the password will log you in right?


@AndreasKl @spencergibb Please consider adding to the note since which version is this change effective.

http://cloud.spring.io/spring-cloud-netflix/spring-cloud-netflix.html already renders this information but there seems to be no release which supports this yet.


Yep. It was something stolen from goog's loginactivity. I tried deleting most irrelevant stuff (they tried autocompleting email forms by asking for contact permission, for example) but this one seemed simple and works


instead of waiting for fixed amounts of time (which will really add up as you make more tests), maybe you should check some of webdriver.io's utility function. For instance [waitForVisible](http://webdriver.io/api/utility/waitForVisible.html)


What is the problem being solved?

Assuming that ``f 0 <inet4>`` means "connect to localhost", "bind to all interfaces" on all platforms?  But it doesn't work on Windows?


Well that's annoying.

Keep your lines to 80 characters they said. Here use this tool they said.

Stupid tools should do this shit automatically. When do we get good tools?

</rant>


Good point.. I'll update it and keep it up to date.


Alignment. I'm assuming block tabs?


Please declare the variables in the most restricted scope and use the const qualifier as soon as it applies to a variable. Here sample, standardSample, sumSelectedResidualsSimulation, sumSquaredResiduaksSimulation and statistic are variables local to the loop, and only sumSelectResidualsSimulation is non-const.


The home page points properly to the correct documentation.

http://projects.spring.io/spring-cloud/


TeamCity Civic Assembly :: Build all [Build 90](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1682&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 71 (7 new), passed: 153 Build time: 00:08:18
### Failed tests
```
Basic Page Categories CRUD Actions: Should Delete an Page category: <no details avaliable>

Venue tests: Stadium type: Should Not Add Stadium if Cancel button pressed: <no details avaliable>

Venue tests: Stadium type: Should Add Stadium: <no details avaliable>

Venue tests: Stadium type: Should Add an Stadium and CONTINUE: <no details avaliable>

Venue tests: Restaurant type: Should Not Add Restaurant if Cancel button pressed: <no details avaliable>

Venue tests: Restaurant type: Should Add Restaurant: <no details avaliable>

Venue tests: Restaurant type: Should Add an Restaurant and CONTINUE: <no details avaliable>

Venue tests: Venue type: Should Not Add Venue if Cancel button pressed: <no details avaliable>

Venue tests: Venue type: Should Add Venue: <no details avaliable>

Venue tests: Venue type: Should Add an Venue and CONTINUE: <no details avaliable>

Venue tests: Venue type: Should Not Edit a Venue: <no details avaliable>

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


It doesn't work on windows.

On Mon, Mar 7, 2016 at 2:15 PM, John Benediktsson <notifications@github.com>
wrote:

> What is the problem being solved?
>
> Assuming that f 0 <inet4> means "connect to localhost", "bind to all
> interfaces" on all platforms? But it doesn't work on Windows?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/factor/factor/commit/2c82dfb51ba6923c2e7dcf3fac5109ae9da528f2#commitcomment-16548156>
> .
>


That's a good point. Maybe we shouldn't mount anything other than the working directory by default, and add these `/usr/localX` and `/data2` directories as `container-volumes` settings on tools using `backwards-compat` ... what @ianmcorvidae said as I was typing this :)


hmc is const


change to:  "Ostro OS image"


Du coup, Angular.js ?


Making it configurable enables users to use their own channels...


Well that is interesting.  in terms of reproducibility I would question citing the “most recent” version of software, if a prior version was used for the analysis.

Tim

> On Mar 7, 2016, at 3:03 PM, iliant <notifications@github.com <mailto:notifications@github.com>> wrote:
>
> 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.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/force11/force11-scwg/commit/292270ec5df035237125ee3e6911407774c00336#commitcomment-16545048>.
>


Should have been http://cloud.spring.io/spring-cloud-netflix/


Right.

> On Mar 7, 2016, at 3:08 PM, Kyle Niemeyer <notifications@github.com> wrote:
>
> 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>.
>


use residualSize / 2 instead


a double or a NumericalScalar?


I would probably want that ``resolve-localhost`` to be the main implementation on all platforms.  I don't think 0.0.0.0 should be "localhost" on any platforms.


Hrmm, usually I prefer these in C code, but I guess having them in the headers isn't a big deal for a library of this size.


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1580 of 1580 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.86%25-green.svg)


@ericstj 4.0 version of SqlClient doesn't depend on `System.Collection.NonGeneric`
https://github.com/dotnet/corefx/blob/master/src/System.Data.SqlClient/ref/4.0.0/project.json

Am I missing something here?


C89 style comments please


And we should revert the change to "127.0.0.1" unless we want ``f 0 <inet4>`` to mean localhost.


Good feature, thanks. Only minor glitches ;-)


Ohhh, sorry! I really missing it :)


In the JDDCP (Data Citation Principles) the same approach is used, understanding that older datasets may be de-accessioned.

> On Mar 7, 2016, at 3:30 PM, Daniel S. Katz <notifications@github.com> wrote:
>
> 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>.
>


Oh cool. I swear I tried this exact change locally for me, but that didn't fix the test! I must have botched something up when testing - let's see what the bots say.

Btw, what about `getU16()` and `getU8()` here? Should they need the same treatment?


The code does not comply to PEP8.

*PEP8Bear, severity NORMAL, section `autopep8`.*


Line contains following spacing inconsistencies:
- Trailing whitespaces.

*SpaceConsistencyBear, severity NORMAL, section `python`.*


It does feel a bit odd to make the memory backend json serializable rather than duplicating it into a json storage backend (that is a subclass of the memory backend). That might require refactoring the memory backend a bit to allow access to internal storage via subclasses.


I wonder, somewhat idly, if it'd make sense to make some of these Status...Failed constants be `error`s in the Go sense, so they can be passed up to `Run` by things like `createStepContainer` more idiomatically. That may be an irrelevant implementation detail at this stage of development though.


what's nice about "big arrow" notation, is that it's basic form is: `(params) => (return value)`. This means, unless you are running multiple command in the function, or making a variable assignment, you can just directly call a function (or in this case return a promise). This would instead be:

```
it('should load the page', () => browser.url('/').getTitle()
  .then(title => expect(title).to.startWith('Amazon.com')));
```

you can start to see how that can make readability far greater. You can also do this trick with e.g. Promise.all:
```
it('should load the page', () => Promise.all([browser.url('/').getTitle(), browser.url('/').getTitle()])
  .then(title => expect(title).to.startWith('Amazon.com')));
```

or multiple then statements:
```
it('should load the page', () => browser.url('/').getTitle()
  .then(title => expect(title).to.startWith('Amazon.com'))
  .then(() => browser.url('/something-else').getTitle())
  .then(title => browser.waitForVisible('oreo cookies'))
);
```

You can apply this rule down the line to all your cases here I think


@dwradcliffe coming momentarily!

@acacheung `&` will work, but it's not valid as per the spec. It's know as a "fragile syntax construct".


Ok. I'll continue tomorrow


Ya puedes mergear


Scratch that, I see this type is not present in 4.0 version.


indentation


we'll have to backport that change to our ckan branch for this to work


Are there any unused imports here?


C89 comments please


So fancy :cocktail: :high_heel: :100:


Since the `cache-id` is now an integer rather than a timestamp, could we test what exactly it is, rather than `not nil`? It seems like that might be a better test, but on the other hand updating the tests would be much more annoying.


This should not be part of the pull request. Granted, the product doesn't exist on bugzilla right now, but if people start using this, I'd be happy to push it to git.gnome.org.


are you stuck 27 years in the past :P


Seriously Mihir. Why did you even choose this word.


Updated.


tidied up some code from html to test.js


alignment and c89 comments


OK, thanks for letting me know.

On Mon, Mar 7, 2016 at 4:10 PM, Katrina Owen <notifications@github.com>
wrote:

> It's not been merged, but I don't really care anymore. I was trying to
> normalize them so I could script something, but that train has left the
> station. I'll submit a PR with the actual change I want.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/exercism/xjavascript/commit/afe96b6f9e288c99a66dd912a95d2bda8cc047ed#commitcomment-16546660>
> .
>

--
Matt Morgan
Director of Technology
Dulcet Software LLC
(734) 926-8142


Just a comma was moved.


Below: function opening brackets should be in new line.


g_error() is also a fatal message type.


The comma after Monday didn't seem needed.


Should this be

```clojure
(is (not= production-cache-id-after-second-update
          production-cache-id-after-third-update))
```

?

or should this be moved up to line 365 and another assertion about the second and third updates added here?


Try catch to prevent crash?
not a real fix though


I noticed an extra space between discussion and It's


There was a space(not mine) and my IDE deleted a tail space.


Any unused imports?


Un-factory this


Was able to remove pan map and just set curr pos


I thought this paragraph could be all about the chat rooms in general + project chat rooms. And the next paragraph could be just about general chat.


ok


I'll delete it.


@tanyxp Was this accidentally deleted? @bmolparia added this in a previous commit, but it was removed by your merge. Was this intended?


This doesn't actually need authData as an arg right? Maybe just call it startDriverActivity or something since it's not actually messing w/ the authenticated user


This separate component to calculate current location worked well.  It's also nice that other components can call updateUserLocation() to reuse the method.


https://github.com/rubygems/rubygems.org/pull/1216 submitted


`c_api_headers` and `cpp_api_headers` copy paste duplication


Todone


Ah, probably fine for now. Although the other commit is adding churn, I don't particularly care that much.


My comments are in parentheses, and I added a comma after `what we're planning to do` and `if anything needs to change`.

I think it might be useful to mention using the project board as a meeting facilitation tool. I also thought it was important to mention it's not really that quick of a call, and it's when significant work often gets added to the board.


Is it something like `$"{{DefaultRid}}: {{}}"`?


`splice` will create a new array, but the contents of the array are the same.

You are indeed creating new `newLists` when ` this.state.lists.slice()` but by doing `newLists[listId].archivedMessages.splice(index, 1)` you are mutating the `archivedMessages` array which still points to the objects in `this.state.`


I'd rather do the database migration (preferably in a future release) than try and add if-then's or switch statements based on the image name (which is what I meant by backwards-compat code).

Having the "mount extra paths" config setting in the job JSON would be nicer than the if-then/switch statement(s), but I think that would either require adding explicit backwards-compat code to the apps service in the short-term, or have a per-tool setting the database that would have to be set for all tools. Otherwise, apps would not know when to add that flag to the job JSON. That seems more evil than adding it here, since updating the apps service to remove that code later would require downtime, while updating road-runner doesn't.

Regarding basing new images on backwards-compat, if we're going through the trouble to make new images for tools, I'd rather we just pull the bandaid off right away by creating new, non-backwards-compat based images for those tools. Adding images based on backwards-compat would just mean more images to fix later when we finally do manage to eliminate it.


Code below was not needed.  The are directly passed from redux and so we can just use self.props.userLocation.lat in render


`...sort_by {|t| [Date.parse(t.date), t.title] }...`


Sgtm


I just removed an apostrophe in `check-in's`


Call to fetch annotations is needed.  We need to trigger is since we just added a new annotation.  It runs only once on component mount otherwise.


Fixing!


shouldn't this be `production-cache-id-after-third-update` because we have that at line 367? or am I just extremely confused?


What is the point of having `Prefix` and `Suffix`? Can't these be included by the `regex`?


Check the const-correctness of the code (ie add const when things are const) and open and close curly brackets the way they are in the remainder of the source code.


@davidromani I set the entities and the test went well.


I left the login for the bot just so it can be used if needed.


:+1: :+1:


I ended up creating a container for it.


The main reason I was just thinking of using backwards-compat as a base is that then we don't have to test nearly as heavily, so it can be done much more quickly. But our general situation of having basically nothing in the way of automated tests for tools makes it hard to migrate things no matter what :(


Since you mentioned email for communication, I thought maybe dropbox was similar for keep track of our project resources internally. A lot of stuff gets saved here and it can act like a private repo for design and data modeling work.


`sponsors.reject(&:current?).sort_by { |s| [s.term_end, s.url] }.reverse`


No. For example, this is supposed to render the following (when on Windows):

```
        "win7-x64": {}
```


This doesn't really do anything. There is no such option in pg-promise as `poolSize`.
If you really want to achieve such effect, you need to do the following:
```js
pgp.pg.defaults.poolSize = 0;
```


@davidromani it's very difficult to set cartservice


this. is not needed.


Please fix my typo, I'm using 3.2.3


Excess space here. :P


@paldepind can you do a 0.3.0 and publish that one then please? :)


Oh, ya that makes sense. It is because we are checking in the generated files, but we aren't checking in the asset graph. We should talk about this probly.


this. not needed, same as below.


Are you referring to the C# 6 syntax or the HTTP error code?


I'm looking for a more elegant solution than this. Anyone please feel free to chime in!


This does not work with ActivePerl (This is perl 5, version 22, subversion 1 (v5.22.1) built for MSWin32-x64-multi-thread)
An empty output file is produced.


Referring to the usage of "this.", we do not use that with our formatting, it isn't needed.


TLS_CERT_KEY is a typo, TLS_CERT_FILE is the correct variable, missed that one.


Add comments to explain these variables. Too many numbers going around.


Thanks for catching the grammar/ edit type stuff. I started going a little quickly on this one, and wasn't looking as closely at that stuff as I should have.


Yeah, that's a good point, thanks.  I could have them be more exact now.  I'll work that into a later commit.


RIP Ball


Looks good.


It's probably good to do both an assertion for first not= second and second not= third.  I'll work that into a later commit, thanks.


přesunuto do chamberu (má to tak nová verze)


Why is this changing only CPU? All the backends should have the same functionality right?


pouze používá persistenceexception z chamberu


Great. I'll see if others have comments to add here before I make changes. I appreciate the close read!


jelikož tam byl přesunut smart model


pro novou verzi classproperty je to nutné navíc to pak nehází error v ide


Good thought. @mghaught ? Thoughts on Dropbox?


add string formatting lab file


Aha! Then your fix works http://ideone.com/DFbx5d :smile_cat:


Should be 4.1.0.0.


why don't you use ```cache_password``` here?


meta pro SmartModel (SmartMeta)


líp se s tím pracuje


oh wait, I see, it's because the environment was expired between the get and set. The naming of these in the "when all environments marked expired between get and set for environment that did not previously exist" test helped a lot. I think this would be less confusing if the `third-update` was renamed to something else.


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


defaultně používáme indexy


@vcunat: nope, just ran into an evaluation error while running nox.


duty free variables


pomocí clean_before_save můžeme vypnout validaci


Got ya !

This change has a big impact on compression levels defined with mml=3 and h3=0,
since these were implicitly depending on `ZSTD_HASHLOG3_MIN` to get a correct detector.


předávám changed fields pro pre a post save


Hm, that is strange. What versions of ruby and puppet are you using?


testy našly drobné bugy


defaultní port pro http je 80 pro https je 443 pro ostatní protokoly musí být nastaven


aby fungovalo v django 1.6 i 1.7 a výše


Is "this" like everywhere or a specific one of the changes?


nový helper


You don't need too add the parentheses.
Is there anything fixed by that patch ?
In theory the driver may put default in gtt if it wants, and put some other formats in vram.
In particular right now more formats than just default are put in vram.


update_travis_toolchain (lefticus) - i386-Windows-10-VisualStudio-14: Build Failed

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


lépe zjistíme že je něco špatně


toto tu chybělo


@whoburg, should I rename this method?


radši ordered dict


It is everywhere where you use:
open STDOUT,">$output";


tady bylo spoustu chyb, které našly testy, naštěstí jsme to nepoužívali


e.g. /data/databases/abook/
You can get abook from regex, but if you want db name to be prod_abook then you need to specify prod_ as prefix.


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-14: Build Failed

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


set je lepší


A side note -

The yacoin-qt-tdm32.pro and the makefile.mingw don't have the ASM usage defines!  The latter doesn't have the leveldb tests either!  I didn't check the other daemon makefiles.


13 tests:

  Results
    √ renders entries with vote counts or zero (91ms)
    √ 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

  reducer
    √ handles SET_STATE
    √ handles SET_STATE with plain JS payload
    √ handles SET_STATE without initial state

  13 passing (273ms)


Ok


2129_radiance_server_perf (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2359 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.41%25-yellow.svg)


> # puppet --version
> 3.8.6
>
> # ruby -v
> ruby 2.0.0p598 (2014-11-13) [x86_64-linux]

On CentOS 7 with puppetlabs repoo for puppet/facter


neptejte se jak to funguje, ukradl jsem to :) ale rozumím tomu


You probably need to wait up to 30 seconds


Maybe I could do something like `production-cache-id-before-marked-expired` and `production-cache-id-after-marked-expired` for these, like I did with the other test, "when an individual environment is marked expired"?  I'm obviously having a terrible time naming these things.  :disappointed:


This is going to get really chatty


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-14-Debug: Build Failed

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


to avoid the variable and find test you can directly use index_set.count(iter->first) => it will return 0 or 1


Oskare promiň dal jsem se na první místo, jelikož jsem to pod sebou registroval do Pypi


is LSOF really the right way to do this?  Don't we already have netcat in the image (it's required for other tools, like port forward)?


When would the pid file be missing but the pods be gone?


přidání odchytávání výjimky jinak nic


No good reason; I can change it.


Thank you! I spent several hours with that.


@ctrueden EcfWrapper is just the boilerplate code between Java and C.  If we want to use JNI to call the native library then we need this.  With other methods I think we might not need this, but I'm partial to using it since it's working and is a bit faster than something like JNA.