test_coordinates.py modified.

However, you can see I have defined a dummy variable m (lines: 141, 153, 161...)
I think I might be able to avoid the definition of that variable which is unused... maybe I'm not doing perfectly those tests (because if I remove the definition of that variable it doesn't work)
Maybe someone can help me


Given that "using cURL" will no longer be a link to that section in the Rackspace Cloud Identity API 2.0 documentation, do we even want to keep this sentence "See using cURL"?  Or is this here as a placeholder to extract that text and add here at a later time?


This should be a post or a put, what are you thinking you dingus


ArchiveFactory.Open(FileInfo fileInfo, Options options) needs this change too.


Good catch, I'll remove shortly


A link to pip's issue tracker, where the problem of package dependency conflicts is discussed, would be helpful here.
You may even summarize the problem in this introduction and define some terms that someone unfamiliar with the project can follow.


111101900_MSHeatPump (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2087 of 2088 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-99.95%25-yellow.svg)


These tests can be extended to exercise more code.  I chose not to so that the point of this PR remains clear.


Couldn't it be 2.13 so users with the dev version can get it?


In this place you must use DependencyInjection;
for example:
private final @NonNull UserService userService;

Please use lombok, and use on controller @RequiredArgsConstructor(onConstructor = @__(@Autowired))


Do we want to have all the ballot info on our site?
Not sure why we would have the pro statement if we did not have the rest of
the info.  Just to be clear: the pro and con statements are statements in
support and against the referendum written by, usually, the people who got
it on the ballot and some random folks who are against it.  Not sure why
you want them here.
On Jan 14, 2016 1:08 PM, "Ben Cipollini" <notifications@github.com> wrote:

> title should be FOR or OPPOSED in most cases. brief and full_text ...
> right, I don't know what they're for, since we have pro_statement. I
> think they should just be removed. sound ok?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bcipolli/opendisclosure-backend/commit/f25ecfd96fa0f70727eacad4de68308d83615c6d#commitcomment-15460972>
> .
>


not post, you service isn't connected with controller,  better name will be create


Your right,

I updated this commit in 17c80a13 to include compiled JS file.


Wouldn't it be better to pass the desired CultureInfo as an option instead of having a flag for one specific culture?


What would you change to set Monday as first day of the week?


:star:


for correct implementation you must use:
java.util.concurrent. CopyOnWriteArrayList


Scratch that, one of the commits this has as a diffbase already upgraded `setup.py`.


I misunderstood, you are correct, how this stands the `.error-string` won't show up unless it had `.required-error`.


I will defer to @adborden for comment on that, as he added these fields to the model to begin.


add ```metadata/``` to the .gitignore file to ignore all the files in the folder


please change the variable definition in line 938. Thanks.


Was anything changed here? I don't see a difference.


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


Service always return new created object,


Fix-allocation-for-feneSolarDelaySeq (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1919 of 2045 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-93.84%25-yellow.svg)


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2042 of 2052 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-99.51%25-yellow.svg)


In fact, other configuration methods or beans should be renamed accordingly in all config files for consistency.


this is a test


should return new created object


Ok


I believe `bool(proto.topDownMode)` will work here. And above you may be able to do `int(self.topDownMode)`


this is still bad as we dont stop the code flow.
Need to set/longjump as in LEST


Remove
```
      else:
         util.writeOut("Cleanup operation aborted")
```
Then,  It looks like an error.


neat :)

you're probably well aware, but there's already addons that do this: https://github.com/Geistyp/ofxPocoDirectoryWatcher (poco uses OS events to find changes)


More importantly, this is the root directory for logs too.


add
```
.project
```
to the .gitignore file to ignore this file


cc @nikhiljindal


Or tell me where the content is left over.


more info: http://docs.spring.io/spring-boot/docs/current-SNAPSHOT/reference/htmlsingle/#boot-features-testing-spring-boot-applications


Durchaus richtig. Hab's direkt mal umgesetzt. Kannst nächstes mal auch gerne direkt eine Issue öffnen :+1:


add here
```
.classpath
.project
.settings/
```
to the .gitignore file to ignore all the files in the folder


TeamCity pull requests :: netty [Build 1038](https://garage.netty.io/teamcity/viewLog.html?buildId=4770&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6372, ignored: 23 Build time: 00:42:09


:-)

I think this broke all links in the file.


This should do it. The unit tests pass. Let me know if you think otherwise, @mbadams5.


cc @naveenoid I migrated the urdf sensors parser to be KDL-indipendent and improve the behavior in case of error.. can you check it?


This looks weird to me. Both `window.location.href` and `^article_permalink^` should be the full page URL. Concatenating those together makes no sense. Plus using window.location.href will break the comments if you have multiple routes that resolve to the same page.


Internal. Used in our one-box simulation environment.
Perhaps this setting shouldn't be documented here.


Use [Connection](https://docs.oracle.com/javase/7/docs/api/java/sql/Connection.html) Interface instead.


but why?!


@DiegoPino to confirm when I perform
```
echo "PREFIX dc: <http://purl.org/dc/elements/1.1/> INSERT DATA { <> dc:description 'Longer description' }" | curl -i -XPATCH -H"content-type: application/sparql-update" --upload-file - "http://localhost:8282/islandora/resource/69f35b0f-6cd4-457a-b31d-7100b1626b2f"
```
I get a ```400 Bad Request```

Same if I make the header ```content-type```, but if I remove that header.
```
echo "PREFIX dc: <http://purl.org/dc/elements/1.1/> INSERT DATA { <> dc:description 'Longer description' }" | curl -i -XPATCH --upload-file - "http://localhost:8282/islandora/resource/69f35b0f-6cd4-457a-b31d-7100b1626b2f"
```
I get the ```204 No Content```

Is that correct behaviour?


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2053 of 2054 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-99.95%25-yellow.svg)


Use [Connection](https://docs.oracle.com/javase/7/docs/api/java/sql/Connection.html) Interface.
And make this class a singleton.


Storage directory for the ufora-store service (internally known as "shared-state").
We plan to get rid of it entirely at some point and replace it with some memory store like redis. But in the meantime it's still being used.


Don't you understand anything @ZombiePower  You can do /give <ign> bucket:8 or bucket:10 ffs


Enables Component Analysis window by default


make this variable private and accessible through a getInstance() method


No idea. Maybe we hit some crazy GFM bug or something.


I think that would be great, but there is the broker issue I mentioned earlier. To be more specific, I am talking about the following line in `Processor` (SocketServer.scala):

`selector.register(connectionId, channel)`

Not sure we can replace `connectionId` with a `Node`, but it's worth investigating.


Directory used by the ufora-worker service (cumulus) as temporary scratch space. Data too large to fit in RAM is temporarily stashed here.


jaja yep i just didn't want more dependencies... this is ghetto enough and it can be 100% disabled with a macro


Hi Julmot,
do you think is possibile to make your script working with different search boxes contemporary.

So for the first one you would have yellow highlight; for the second one blue highlight or similar?


The HTTPS port is no longer in use and can be removed.


Actually, your commit seems to have broke it https://github.com/whitequark/parser/blob/7c7a22169d843bf75c76459784dd70cb4db27d89/README.md


Should the product name be "Dedicated Load Balancer API" ?


@AkamaiDASH This was producing the `dash.all.debug.js` file which the samples rely on.

Do we want to eliminate `dash.all.debug.js` entirely? I could update the samples to use `no-externals` and the `externals` file instead.


I can do it.


If this works, even better would be `if field_type.kind.fields?`

(https://github.com/rmosolgo/graphql-ruby/blob/master/lib/graphql/type_kinds.rb#L17-L18)


Wasn't quite sure what to do with this remaining content...don't think it's useful here since it isn't actionable.


I assume the reason we don't want to do this polyfill style is that we're not expecting to need to do any user testing with older versions of firefox before 45 hits release channel?

If so, might be worth checking with Sevaan / @Standard8 about that assumption...

I don't think we'd want to spend much time on making this a polyfill just for that though, so the decision presumably depends on the cost/benefit ratio here...


Some people say no, but Rob Jackson has it as Dedicated Load Balancer API.

Other suggested Dedicated Load Balancer API


Did you mean to move these somewhere or just delete them?


I think i missed that one - which is the correct fix for this ...:
https://github.com/MrMC/mrmc/blob/mrmc-release_1.2.0/xbmc/Application.cpp#L3373


Removed.


why do we need swap?


Nit: technically, the warning is not printed until `emit` is called, I believe.


This is where we tell the user where his leftover content is.


Great Job!


Nit: same here


https://docs.travis-ci.com/user/ci-environment/
we are on trusty beta


comment for why this is necessary


Is there a reason why not to rename init_kernel_mp init_kernel and implement it in spec/arm/smp/init_kernel.cc? And couldn't init_kernel_mp call kernel() directly like the UP init kernel?


@UniversalMediaServer/developers somewhere in the UMS improvement the ability to open internet page from the tooltip we lost. It is not working now.


I am commenting on a commit.


Sorry for that :(
You got my question right anyway. Thank you for answering this. I hope that there will be a better chance to use the relaxed syntax in future.


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


After |product name| can we add |contract version| , to be clear about the product AND version being offered in EA.


The number of compute threads in ufora-worker.


Did you deploy?


What would you suggest |contract version| should be?


Second thought, cool font!


this is a test for leaving comment


Let's be sure to check that this preserves the existing behavior for headless rendering, which is supposed to return an error immediately if a request fails, rather than waiting to retry. I think at one point this relied on tiles going into the `obsolete` state but it looks like that may no longer be the case.


I fixed this since then


This is pretty good, but you can simplify it a lot with Math.abs. I (or someone else) will show you how to use it at the shop later, if you want.


@valib It doesn't work with JComboBox - use CustomJComboBox instead. But, I don't know if it's such a good idea, as it's close to impossible to actually use the link.


Leaving a comment here for test.


I would prefer to keep the `<arg1> [argz]` in there too, as extensions can also take arguments. Remeber, potentially `switch` won't be the only extension.


i think you did not understand what DUP does. DUP should take the numeric-value from a register or internal var and push it on the stack so that it can be used later, even if the value of the register itself has changed.


Forgot to reference the bug this commit fixes: https://github.com/trotzig/import-js/issues/124


The two PYTHON_GIL_LOOP_THREAD settings are internal and used for diagnostics. I don't think users need to know about them.


@adamjonas Re-added CNAME.


Yea...


Opera Mini renders differently because it is the last Opera browser that uses the "presto" engine they developed themselves before switching to webkit.

Opera 12 is also the last version to use Presto, so you can use that for desktop testing.

Windows http://www.opera.com/download/guide/?os=windows&ver=12.17

OSX http://www.opera.com/download/guide/?os=mac&ver=12.16

Linux http://www.opera.com/download/guide/?os=linux-x86-64&ver=12.16

Or run the apk using Chrome ARC Welder but it tends to crash that way.


Maximum amount of RAM to be used by ufora-worker (cumulus)


Can't imagine why. I use only fenced blocks on dozens of projects. I'll look into this later.


I agree with Rob that the product is Dedicated Load Balancer API, since to me this API is the product described in this documentation, not the physical dedicated load balancer itself.


Aha


Its enough. I think he understood.

> Am 14.01.2016 um 22:55 schrieb Link Usumaki <notifications@github.com>:
>
> Yea...
>
> —
> Reply to this email directly or view it on GitHub.
>


#595 - wrong issue?


@smurfpandey i completely forgot about `sessionStorage` which allows to use storage on per-tab basis. Just pushed the change to master. Just FYI


I will like that.
On Jan 14, 2016 3:54 PM, "Max Haland" <notifications@github.com> wrote:

> This is pretty good, but you can simplify it a lot with Math.abs. I (or
> someone else) will show you how to use it at the shop later, if you want.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/frc-2175/2016SimpleDrivebase/commit/b39680748673301af5dfa9a4f45817d6488dc956#commitcomment-15461962>
> .
>


Implemented ChromeCustomTabs to the WebView example. Fixed duplicated dependency entries in the gradle files. Added multiDex support to build.gradle to fix a non-zero exit on app:dexDebug


empd (nmerket) - x86_64-MacOS-10.9-clang: OK (2045 of 2049 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-99.8%25-yellow.svg)


Internal. This is a switch that controls how cumulus tracks its own memory usage. I'd leave it out.


This is super ugly... Definitely going to come back and find a better / more effective way to accomplish this precompiling of assets.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1959 of 2086 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-93.91%25-yellow.svg)


Flask-admin needs to be initialized oddly for tests to work.


The max amount of RAM used by the ufora-backend service when transferring data between a Ufora cluster and a user connection.


Max size of disk space used by ufora-worker as scratch space.


Maximum number of files created in the ufora-worker scratch space.


I meant literally add "|contract version|" so that it is replaced by what is defined in "load-balancer/api-docs/conf.py", which I see is currently:
|contract version| replace:: 2.0


The lines above look like they're just whitespace changes, and AFAICT they are moving *away* from the PL style guide rather than toward it?


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


To run loop in older versions we'd need to do a lot of work to override the built-in loop (which isn't an add-on in 44 and older). Given the time we're needing to get the add-on into shape, I really don't think we'd be doing that - and even if we did, there's probably already a few other things broken wrt 44.


The frequency in which ufora-worker saves its progress during a computation to a durable store. Zero means never.
This feature is currently disabled and the config value must be set to zero.


Oh I reckon, the only thing is 2.0 is only internal and from what Cesar said today, that can potentially change.


On Wed, Jan 06, 2016 at 02:29:21AM -0800, Lee wrote:
> I dont think so.
> The configure is trying to find gconf-2.0, else it failed.

Please try it out. If it really still fails, I'm mistaken.
firefox' configure supports a --disable-gconf flag that
disables the check you refer to. This flag was missing before,
so it failed to find gconf. Now that I added it, configure
succeeds without gconf around.


Well, it would break with the current behavior (which was not my invention either)
But it would be consistent, if python-mpd2 would its own ConnectionError class, which sets the original ConnectionError as cause.
So people would only need to MPDError to catch all errors of this library and switch to more specific Exceptions on demand.


Max size of a ufora-worker data page (in bytes).


isn't what you do with sling.getService in ecma possible with optional service references in the component (which could then be selected through null checks just as in the script)?

> Am 14.01.2016 um 22:11 schrieb justinedelson <notifications@github.com>:
>
> We can refactor back to a non-dynamic implementation once we retire 6.0 support. I think (but did not test) that the new way (using the ServiceComponentRuntime service) works on 6.1 OOTB. But I don't think it is worth cutting off 6.0 support just for this one thing.
>
> —
> Reply to this email directly or view it on GitHub.
>


Same comment re: whitespace above


Include both the base engine and the mungebits engine -- we shouldn't be replacing it since that's not the intended structure.


Number of threads used to load data from external stores like S3 or HDFS.


My mistake, I meant to use `window.location.host`

I will make this update :+1:


Add favorite color


Why did we delete this?


Please do not reintroduce these comments, Michal removed them, see 55b317c69964991d1204ade1eb3d4c69de120683.


Good job, nice description! The only thing I would add in the last sentence is the word "new" between "provision" and "dedicated" (to make clear and to distinguish from reference to "existing" dedicated hardware load balancers in the previousl sentence).


Thanks!


<img width="1440" alt="screen shot 2016-01-14 at 23 07 44" src="https://cloud.githubusercontent.com/assets/1530049/12339148/ad2c2d7a-bb13-11e5-9ae6-b983bbfe9392.png">


:clap:


But why do you need that at all? Is ^article_permalink^ not resolving to the full URL of the article on your site?


probably should standardize the verbiage.. I see several forms.. with or without , or (C) in different places.. or quotes around the team with dash or not.. few examples:

`(c) Copyright 2007-2016 by "The SABnzbd-team"`
`Copyright (C) 2008-2016, The SABnzbd Team <team@sabnzbd.org>`


Also added a bunch of links!


Ok, just added a parameterless ctor on base class and removed from the ones that doesn't need it.


LOL text by: @cbeducation/the-admins


:bomb:


gj


I believe it _should_ still clear here. This matters in the following sequence:

* Tile loaded for the first time, `parseAllLayers` is called.
* Some layers end up in placementPending.
* Before `placeLayers` gets a chance to run, the tile expires, is reloaded. `parseAllLayers` is called again.


Obsolete. This is only used in a few tests now. Shouldn't be documented.


@Nadahar thank for the reminder. Why it is impossible? It works but the only thing which is problematic is to move the mouse over the tooltip and not to close it. Is there any way how to improve it?


Giannis, please be aware that the agent code split is on the way now, and changes in the agent will at this point not merge cleanly anymore.  You may want to create a new branch off `feature/agent_split` (and before that give that branch some test runs,  possibly ;) ).  Let me know if it would make sense to have a quick call to give you an update on the code structure!

Best, Andre.


Number of threads used for background compilation.


Nice work Dan


Yep. You are right. Just did it.


Fix-allocation-for-feneSolarDelaySeq (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1957 of 2083 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-93.95%25-yellow.svg)


This is the most mission-critical fix I've ever seen.


Yeah! That's good compromise. :+1:


OK, we can omit the contract version then.  But if it does change for public version, then we will need to change it in the "load-balancer/api-docs/conf.py"  (even if it's not reference in the actual text anywhere) right?


No probs @stveep, I've pulled that commit across (different hash because I rebased)


Hi Duke,
do you mean to have two text input elements and highlight the content of each of them with a different color?


this is fine since context should be empty, but I was just thinking `(reset! test-init-context context)`


Thanks for your comments. The last one may require some more thought. I
was trying to explicitly only handle -1 since this way it seems to be
the least risk, but you have a point and we should likely check for
negative instead. The return value is automatically casted to
`ram_addr_t` which is unsigned. So in order to make a (readable) check
for negative values I've changed the data types inside
`microblaze_generic_fdt_init` to signed. I think the impact of this is
tolerable and it seems to be the nicest solution. If you prefer
something different (temporary variable or checking the unsigned value
against LONG_MAX) then let me know.

While thinking about this I've recognized that the source code comment
about `object_property_get_int` is wrong. So I've changed that too.

Am 1/14/2016 um 1:02 PM schrieb Alistair Francis:
>
> Use a less then 0 and remove the cast
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liebharc/qemu/commit/cb94f0957eb0bd4e61275d359a623abd4c7dda11#commitcomment-15460835>.
>


This and the following three settings are used to fine-tune memory limits of different components within ufora-worker. They should rarely be used explicitly.


I'm done reviewing changes.  Good job removing the links and notes refering to the Identity docs (per discussion with Jason) and the other edits you made!


@valib I didn't say it was impossible, but close. What I meant is that it's difficult to get the mouse there without the tooltip closing.

I'm sure it's possible to improve, but I think you'd have to dig pretty deep into Swing. One way would be to move the tooltip so that it always overlapped with the control, but that can be annoying too. Another way would be to delay closing it a little bit, but I guess that could be challenging involving timers, threads and whatnot.


Yeah, that makes sense.

What does it mean for a tile to expire? When does this happen?


@jthelin That is the Plan. This first PR will state that we are only changing the test runner to Xunit while keeping the assertion APIs on MSTest as @jdom suggested however, the first thing we will do after it, is to create another PR updating all the Asserts to use xunit or FluentAssertions... Maybe we should open an issue so people can discuss wether we should move all the asserts do xunit or use FluentAssertions. The important thing is to keep a singular API and not use part on one and part on another.


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


I could go either way on this.  It seems like it would cause a lot of code churn, but it would probably be more consistent with other public APIs.


Can you please create an issue for your question? It doesn't seem to be relative to this commit. Thank you :)


The cumulus object store is a persistent cache of in-progress computations and completed results. It requires a durable store such as S3 or HDFS and is currently disabled entirely in ufora-worker.


I'm new to GitHub, where do I find it? Sorry, for stupid question :P


TeamCity pull requests :: netty [Build 1037](https://garage.netty.io/teamcity/viewLog.html?buildId=4760&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6372, ignored: 23 Build time: 01:22:34


Frequency of garbage collection in the ufora-store service (a.k.a shared-state).


you should remove source "drivers/hw_fac_info/Kconfig" from :  drivers/Kconfig  :+1:


Fix-allocation-for-feneSolarDelaySeq (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


There was a circular dependency most easily resolved by changing how it was included, I wasn't aware of its inclusion via compiler flag


I think someone native in English should correct the text itself, it's a bit hard to understand.


No problem, we've all started at a time. Here you go: https://github.com/julmot/jmHighlight/issues/new


@rob006 are these looking as bad w/o zooming in?


`OnlineFileSource` revalidates resources (most importantly tiles) based on the `Expires` and `Cache-Control` headers in the previous response.


111101900_MSHeatPump (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 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.69%25-green.svg)


Cool. Yes, opening an issues now about which "Assertions" package to use is probably a good idea, then xref from this pr. nj.


+1! :)
Maybe we should implement support for the relaxed syntax in NetBeans and send it as a pull request?


Changed, and adding a test in what will be review 19.  We don't support
cross type encoding yet but when we move scheme out of conversion that'll
change.

On Wed, Jan 13, 2016 at 10:20 PM, Clayton Coleman <ccoleman@redhat.com>
wrote:

> I think so but I'll double check with a test.
>
> On Wed, Jan 13, 2016 at 5:40 PM, Chao Xu <notifications@github.com> wrote:
>
>> Shall it be if (c.encodeVersion == nil && len(overrides)==0) ||
>> isUnversioned?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/kubernetes/kubernetes/commit/08270411316784f6cd8c740225529dfc05430d5a#commitcomment-15436559>
>> .
>>
>
>


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


Code was removed in an earlier commit.


right. i thought that explicitly including context here would help to show that the context was being cleared as expected between stop and init during the restart.


Removed


@Nadahar this is always a problem of us not native speakers. :-)


Tests?


this addon looks nice!

for mipmap checking, you could use `ofTexture::bWantsMipmap`


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


Colin Woodbury <notifications@github.com> writes:

> This change was very sudden and quite confusing.

It is not, I should have done this before to avoid confusion with

original M-x and helm-M-x and find-file vs helm-find-files.

> I really preferred helm's M-x and read-file over the default.

Just bind M-x to helm-M-x and C-x C-f to helm-find-files.

--
Thierry


Ok will do that. The main point on this PR is to unblock the migration to xproj and get free from MSTest team schedule on release of DNX-based MSTest, but we will get test improvements over the wire along the way. :)


Also added load_json() function for the Photoshop Export plugin


It should be enough to build the TextMate target, as this one requires the mate executable.

For Xcode version, we don’t officially support anything older than latest non-beta, as we generally want to be able to use the latest features as soon as they are available.

The failing scm/git tests (when async), there definitely is an issue here, but I can’t figure out what it is :( it must be some sort of race condition. Doing a `./configure` switch to disable (or enable) async testing would likely be a good idea. Ideally nothing should ever fail out of the box, so probably non-async tests should be the default.


I already made this plus duplicate name prevention, you should have just cherry-picked this from me


ah, great. missed that. changed to 20px now.


private : https://github.com/openframeworks/openFrameworks/issues/4732


The commit additionally
1.) https://trac.transmissionbt.com/ticket/4078#comment:4
2.) Adds a configurable by rpc magnet-bad-piece-max setting (settings.json).


The installation via PyPi was causing weird problems in the likes of `ImportError: No module named DistUtilsExtra.command`. Installing version 0.8.5 via URL fixed that. I don't really get why it's located under the 0.7.8 branch, though...


Use [Connection](https://docs.oracle.com/javase/7/docs/api/java/sql/Connection.html) Interface instead.


Bro can you please give me better credits than all other kernel devs because you have just taken 100% of my source I have done a lot of hard work putting those 3.10 and Linux patches commit ike i have given to ak in my kernel thread thanks


Lol, you should really cherry-pick my code instead of reinventing the wheel


boo


Btw, this will return a unicode error if your using foreign chars in the script path.


No Macros Dam ;)


Should be careful using flexbox, it's definitely available in MOST broswers, but not 100%. I think it's fine to use, because it's just IE, but let's try to not depend too much on it.

http://caniuse.com/#feat=flexbox


nw_render

A C++ multithreaded ray tracing renderer, simple and works.

Not as complicated as I would have liked would like to improve it in the future.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2050 of 2051 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-99.95%25-yellow.svg)


The producer should have the logic to instantiate different Factories and each factory should instantiate a single connection


I've restored the code, but reorganized it and made it clear what it's used for.


Bro can you please give me better credits than all other kernel devs because you have just taken 100% of my source I have done a lot of hard work putting those 3.10 and Linux patches commit like i have given to ak in my kernel thread thanks


I agree with Pavel.  This change broke an authentication library I've been using on multiple projects.


yeah, I don't like this EQUAL macro... Useless


Make this method return type [Connection](https://docs.oracle.com/javase/7/docs/api/java/sql/Connection.html) and call this method getConnection()


It's not just forensics. Sometimes you just want to obtain the input data so that you can remake the product in a different way.


Well, with [this announcement](https://www.microsoft.com/en-us/WindowsForBusiness/End-of-IE-support), only IE11 is the supported IE version. Do we really want/need to support older IE versions?


Great job. Another alternative here was to use an AND statement in the condition.
`if (i % 3 === 0 && i % 5 === 0)`


Off Topic, Do you use ATOM to develop?


In this case this abstract factory can be implemented as an Interface and should have a more descriptive name as AbstractDBConnectionFactory


__API Blueprint syntax error on line undefined__: ETIMEDOUT (Autogenerated message from apiary.io)


What kind of explanation do you need ?


Will test in a few hours. Thanks dan for the commit.


On inspection looks good, Pradeep.


@samdark On ~1100px width screen books are pretty big and I can easily see this artefacts. On width ~990px covers are even bigger and artefacts are more visible. On my phone in a horizontal position it looks really bad.


Can't we just enable all of them? These are only used for numbers.  With these disabled we need to deal with the cases where this suddenly doesn't work to check numbers and figure out that its the one or two that aren't enabled.

It doesn't make sense to me.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (954 of 954 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-26.84%25-green.svg)


MMP actually overrides this partial...

The only difference is in the `tel: true` bits.

Or maybe we can just change it there as well afterwards..


@benpeter in some cases, that would be possible, but not in this case because this is first and foremost about a package not being available. So there is no way to have a runtime dependency on the `org.apache.felix.scr` package once you've updated the SCR bundle. I did try to do this in Java code. What started to work was having an interface with two implementations -- one using Felix SCR and one using the new interface. Then, in the ComponentDisabler, it would try to instantiate one using reflection and, if that failed, go to the other one. It basically worked, but the problem then was that there was no released bundle which had both the old and new interfaces (Felix SCR 1.8.2 has the old, Felix SCR 2.0.0 has the new, which was a bit of a questionable decision IMHO). So even if the code was possible and would work at runtime, I couldn't properly assemble a compile-time class path. I guess I could have used fragment bundles... But the scripting way is (I think) at least somewhat easy to read, something which the reflection code wasn't.


Your comments make sense. So changing it.


Less state, :+1:.


Got the first option. Is faster for now. We can evolve when we got the assertion upgrade.


isn't this yet taken care of in request.vars parsing (parse_post_vars) ?


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


The images should be resized to the size they have when viewed at large screens IMO. IE, the should be 141px × 176px. That will remove the artifacts on smaller screens too.


In my mind, keeping as much context visible as possible is typically a win.  That said, I'm open to specific arguments in specific case.


Indeed, I misread.  Please disregard!


the if statement will always add an error was this your intent?
if  cat_main_id_1 == '1000' ....
 errors.add(:main_id_must_be_valid ....
else
errors.add(:invalid_name_and_identifier ....
end


So at http://docs.cdap.io/cdap/current/en/reference-manual/http-restful-api/metadata.html#http-restful-api-metadata-searching, we document targetType as ``One of app, dataset, program, or stream``. This will break that, right? we should check for both the actual enum constant as well as the serialized form


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


Note: I moved IFloatingFlower (which means IslandType as well) into the API in 1.8, just a heads up. Made more sense there since it's an addon-facing class, so no one needs a hard dep on Botania classes


mmk, so it will take a small amount of shifting around in 1.8. Not that much of a problem, I assume?


You're welcome. Glad to help.

Whenever the rewrite or new release is ready, I'll be sure to test that too.


I merged #3516, so rebase on master and use `notModified` instead.


nit: around


If the function is called `getBoolean` it should default to false, because `Boolean(undefined) === false`. If you want to add a parameter to pass in a different default, that's fine.


@jacmoe for retina these images should be 2x the size. Also on mobile these are taking 100% the screen which could be a lot.

@rob006 can you point to the images which are good? Probably I've screwed everything up when choosing image quality after resizing...


Ok, I see what you mean about the ram_addr_t, you are right there.
Looking at this more, can't you use the Error argument of object_property_get_int()?


Is there a reason this is not in the current version?
I'm trying to address this issue: https://github.com/hapijs/hapi/issues/2294


cool stuff!


nit: call the second parameter ``searchQuery`` too?


Good idea for an unexpected value to cause an exception, but `"true"` and `"false"` should be expected and allowed values, because they're very commonly used (perhaps more commonly than 1 and 0) for booleans and there is no ambiguity as to the user's intent.


nah, not a big problem


nit: ``formattedSearchQuery``


Can we look at this? I'm under the impression that urls need to be relative to the stylesheet — https://developer.mozilla.org/en-US/docs/Web/CSS/url#The_url%28%29_functional_notation

![screen shot 2016-01-14 at 2 45 09 pm](https://cloud.githubusercontent.com/assets/2048170/12340146/e79f72a0-bacd-11e5-81d0-e789702dc677.png)


So at http://docs.cdap.io/cdap/current/en/reference-manual/http-restful-api/metadata.html#http-restful-api-metadata-searching, we document targetType as One of app, dataset, program, or stream. This will break that, right? we should check for both the actual enum constant as well as the serialized form


Directly logging to cerr in the library is not acceptable. Use TraceMessage with ThumbnailerLogLevelError and it will go to cerr.


getBoolean should be moved to `five-bells-shared` -> `Config` class. It should be used to parse any boolean config variable so that we have consistent behavior for that.


Same: Directly logging to cerr in the library is not acceptable. Use TraceMessage with ThumbnailerLogLevelError and it will go to cerr.


empd (nmerket) - x86_64-MacOS-10.9-clang: OK (2045 of 2049 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-99.8%25-yellow.svg)


semicolons missing : )


Looks like I picked a bad time to check out this branch, all I'm getting is

```
File "genhl.ml", line 296, characters 56-72:
Error: This expression has type ?stack:'a list -> ttype -> 'b
       but an expression was expected of type 'c -> 'd
```


I don't think you even need this anymore: just `takeWhile(_.nonEmpty)` or whatever the method on `ByteVector` is called.


Test cases for these would be great if possible.


It looks like `FontStack` is intended to have the invariant that `metrics` and `sdfs` always have the same set of keys. Can we merge those into a single `std::map` and simplify this logic?


Disallow "0" and "1" then? Best if there's only one right way to do it.


@samdark I am trying to say that they should not take 100% of the screen width - the images are not *that* interesting. Could retina images not be handled separately?
Anyway, I will see if I can tweak the responsive code a bit because the columns collapse much too soon. The images could be 2 columns for tablets.


If you look at it on the branch level it makes sense https://github.com/syberia/modeling.sy/compare/make_modeling_work -- there was duplication in the original file.


Why? What's wrong with allowing 1, 0, true and false. They are all totally clear and unambiguous. By insisting on one way you're just making developers' lives more difficult.


I love you)))) thank you ))))


Doesn't mungebits already inherit from base.


No, we don't. Deleting.


Well excuseee me lol


Note: This might be unpopular, but when merging this into 1.8 I retained this the way it was before (dropped this change).

I simply retained it like this: https://github.com/williewillus/Botania/blob/e12163f70c4de11b909d6ed77c66ebc8798777a6/src/main/java/vazkii/botania/common/item/ItemGaiaHead.java

It's very clearly indicated that the following code comes from vanilla, with the relevant edits by Botania clearly marked by comments.

My reasoning for doing this is twofold:

1. It's vanilla code. Inserting exorbitant amounts of documentation for non-Botania code is kinda...pointless?
2. It makes it much harder to update in the future. Whenever I port code that's copypasted from vanilla I simply compare the copypasted version vs vanilla and make the necessary changes, which are usually small, localized tweaks.  With this the code is completely reformatted and much harder to compare and update.

I hope yall understand, thanks!


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


I think the reverse would be the case. If a developer sees "1" sometimes and "true" sometimes, they might get confused about what the right way to do it is. I know I would end up looking it up in the docs to make sure that both are valid => wasted time.


I missed a font change.  I need to add a cache clear on font change.  I will add that tomorrow.


Careful of that `s` -- `CheckboxHelpers_Use...` or `CheckboxHelper_Uses...`.


I like to cite "The Zen of Python": "There should be one-- and preferably only one --obvious way to do it."


282x352 (I think) and then have width=141 and height=176 should work. Retina devices gets the 2x image and the rest the downsampled image. See http://code.tutsplus.com/tutorials/the-right-way-to-retinafy-your-websites--net-31793


C'est pas mal pareille à ce que j'avais fait. ;)


Why do we need the 3.10 hidp driver to pick that commit? The code it changes is present in 3.4 :/


Right, but that is mungebits's declaration that it depends on base, or rather, that it has the convention of `config/routes` and `lib/controllers`. This is orthogonal to the modeling engine's dependency on base. After all, there is no reason why the convention of having `config/routes` and `lib/controllers` should be inherited through mungebits; the job of the mungebits engine should solely be providing the definition of mungebits.


It seems that this fixed an issue, but only when downloading source from github and doing "cargo build --release", but not when using "cargo install racer". When using "cargo install", cargo is downloading and using syntex_syntax version 0.26.0 (most recent). New to rust/cargo, so I don't know if some sort of re-submission of the crate is needed.

Cheers,
Kyle


This is without provider
```
braintree.environment = "<%= payment_method.preferred_environment %>";
```


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


Quick thought:  It's cleaner to use a "+build debug" tag (with a function call for actual logging) instead of leaving in the debug comments -- if you want those for later.

Here's an example:  http://dave.cheney.net/2014/09/28/using-build-to-switch-between-debug-and-release


You're right - I must have misread the docs. Still - to be on the safe side I decided not to touch options.map unless --map is acutally used. Have a look in [source maps branch](https://github.com/code42day/postcss-cli/tree/source-maps).


See f7e385a
<https://github.com/kubernetes/kubernetes/commit/f7e385a88a8dd49a3fd55aeb6693aa705c3751ef>

On Thu, Jan 14, 2016 at 1:31 PM, Chao Xu <notifications@github.com> wrote:

> Hmm...sorry, I'm still confused. Maybe I can ask some specific questions:
>
>    - where does the commented-out code come from?
>    - In your last reply, are you talking about the commented-out code or
>    the old code?
>    - When you say "resetting the GVK", who are you talking about? I don't
>    see any code doing that here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kubernetes/kubernetes/commit/afe534d31c307a0b08133b98f2e1a4239e6e5854#commitcomment-15457339>
> .
>


But why not?
Is it because of OUTPOT?


This prevents the selection of custom login protocols in the client UI of 1.8.0.CR1.


Done.


Does that mean the `i` wasn't needed ?


Hi Jane, I just asked Alan what version to use in GLID-2211, thanks for the review! :)


Sure, makes sense, it's going to take some time to get accustomed to the _cleanup_ feature.


111101900_MSHeatPump (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2055 of 2056 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-99.95%25-yellow.svg)


merge


can we rename the repo too?


Shouldn't the destructor be virtual?


Same here


Fix-allocation-for-feneSolarDelaySeq (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (951 of 951 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-26.63%25-green.svg)


I am happy to announce that I have become a distrubeter for It works. paulashealthyliving.itworks.com


Ok


:+1: I think the comment is no longer necessary


Yes, agreed. However, the next command (`os.chmod`) needs it. I though it would be more consistent if I used `new_path` (i.e. the full path) in both, `shutil.copy2` and `os.chmod`.


These already exist in the super class, so as the removed comment says they were only here because of simpleframework which we've gotten rid of


Hmm, I've marked in the source code where we land in the
`object_property_get_int` code branches with a comment. It doesn't set
an error. Also confirmed that in a quick test.

`
int64_t object_property_get_int(Object *obj, const char *name,
                                 Error **errp)
{
     QObject *ret = object_property_get_qobject(obj, name, errp);
     QInt *qint;
     int64_t retval;

     if (!ret) {
         return -1;
     }
     qint = qobject_to_qint(ret);
     if (!qint) {
         error_set(errp, QERR_INVALID_PARAMETER_TYPE, name, "int");
         retval = -1;
     } else {
         retval = qint_get_int(qint); // We get here!
     }

     QDECREF(qint);
     return retval;
}
`

Am 1/14/2016 um 2:47 PM schrieb Alistair Francis:
>
> Ok, I see what you mean about the ram_addr_t, you are right there.
> Looking at this more, can't you use the Error argument of
> object_property_get_int()?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liebharc/qemu/commit/cb94f0957eb0bd4e61275d359a623abd4c7dda11#commitcomment-15463005>.
>


This does pass on systems with 64-bit `time_t`. So the comment should be something like "This test is known to fail on systems with 32-bit time_t."


suz i would take more time trying to figure out how 2 cherry-pick than write this small piece of code


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


Is the  Error **errp variable set?


Done


Could you do me a favor and add some comments to the changes that you need to add to make this work??


It is missing a lot of land subtypes (Desert, Lair, Locus, Urza's, Mine, Power-Plant and Tower) and all the planeswalker subtypes (Ajani, Ashiok, Bolas, Chandra, Dack, Daretti, Domri, Elspeth, Freyalise, Garruk, Gideon, Jace, Karn, Kiora, Koth, Liliana, Nahiri, Nissa, Narset, Nixilis, Ral, Sarkhan, Sorin, Tamiyo, Teferi, Tezzeret, Tibalt, Ugin, Venser, Vraska, Xenagos)


We can't have anything synchronous, because it will jam up the Node process when doing the given task.


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


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2048 of 2049 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-99.95%25-yellow.svg)


Neat works, just modify commend with return type for exemplify.


This call is ill-formed, `nullptr` carries no notion of type so none of the `T` above could possibly be deduced.

If you were to replace `T` with say `int`, then `nullptr_t` to `int*` is a _standard conversion_, while the other two are _user-defined conversions_ so `void print(int*)` would be preferred.

If you were to drop this first overload, only then the call would be ambiguous.


Hard to see the differences.


Pretty sure this shouldn't be applyEffectsAfter.


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


This change breaks under Microsoft's ODBC driver for Linux.  When there's a connection error (probably applies to other error types, haven't gotten there yet), we get something like `Prelude.chr: bad argument: 3670064`.  I've narrowed it down to the peekCWString.  On Unix, CWChar is defined as 4 bytes wide, but the output of SqlGetDiagRecW appears to be a 2-byte encoding (either UTF-16 or UCS-2).


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


well i wanted it with a minimal amount of cvars, plus i stored it in sess data for reusability


I fixed it along with [some other stuff](https://github.com/HaxeFoundation/haxe/compare/3b4101296780...564bc0ec29fd). The `@:coreApi` checks were missing on this branch so you might come across some more issues still.


Does anyone know why we have this? I see no where it is used


I was using this way 'cause we need other fields from the fixture and not just the host. But will add the HostedCluster field to keep the standard...


Expand "Single Point of Failure".


s/announcement/announcements/


Note that the C style we use is `...const char *dig...`


It was unused.


@Nadahar is it https://github.com/UniversalMediaServer/UniversalMediaServer/commit/1a0af8ba61d6464b46375d1ddf458bd2cf8e0aeb better?


Wow. I did that on October 21st of last year. It took you a while to see it.


Yes


Fix-allocation-for-feneSolarDelaySeq (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1502 of 1502 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.66%25-green.svg)


s/in/in the/


This switch seems to repeat itself way more than necessary. Why not something more like:

```C
const char *digest_name;
size_t required;
switch (hash_alg) {
case PN_SSL_SHA1:     digest_name = "sha1";     required = 41; break;
case PN_SSL_SHA256: digest_name = "sha256"; required = 65; break;
case PN_SSL_SHA512: digest_name = "sha512"; required = 129; break;
case PN_SSL_MD5:       digest_name = "md5";     required = 33; break;
}
if(fingerprint_length < required) {
    ssl_log_error("Insufficient fingerprint_length %i. fingerprint_length must be %i or above for %s digest\n",
        fingerprint_length, required, digest_name);
    return PN_ERR;
}
```
To me this is better style and more readable to boot.


You missed an App


s/tahoe.cfg file/``tahoe.cfg``/


No it's still pointing to NULL after the function returned. My test:

`
     ...
     Error *err = NULL;
     ram_kernel_base = object_property_get_int(OBJECT(main_mem), "addr",
&err);
     ram_kernel_size = object_property_get_int(OBJECT(main_mem), "size",
&err);

     if (err != NULL) {
         error_report("%s", error_get_pretty(err)); // doesn't get here
     }

     fprintf(stderr, "%p", err); // prints (nil)
     ...
`

Am 1/14/2016 um 3:09 PM schrieb Alistair Francis:
>
> Is the Error **errp variable set?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liebharc/qemu/commit/cb94f0957eb0bd4e61275d359a623abd4c7dda11#commitcomment-15463373>.
>


:+1: sorry I hadn't tested this without remotes! Didn't think of that.


I got it running, but I don't see UBS logo in the header.
Btw, good read:
http://blog.keithcirkel.co.uk/why-we-should-stop-using-grunt/
http://blog.keithcirkel.co.uk/how-to-use-npm-as-a-build-tool/


Careful with `null` here... now the `onFailure` method that checks this result has to check for null. Or the if-logic can be switched so the check becomes `ServiceErrorCode.ERR_NOT_WRITEABLE.toString().equals(serviceError)`


Can these percentage strings vary between languages? Otherwise it would probably be fine to hardcode them?


Originally the intent of latest was to identify the stable version of the API that we intended to write to (poorly named, of course).


Hrm, this was caught in a rebase, let me check.


I think you are just repeating the OpenSSL documentation here - this is not a necessary comment unless something unexpected is happening (I don't think it is).


develop (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1521 of 1521 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)


@jtallant - What is the reasoning behind excluding compiled code from diffs? I've found bugs that were introduced at the compile time and not present in the source. In this configuration we're excluding potentially helpful information by default. I think it would be wise to comment this out and leave it up to the developer on a per-project basis.


Should `function` be in this list?


We dont use this right?


Fixed it


double -> single space


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 122](https://teamcity.labkey.org/viewLog.html?buildId=296784&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 46 (32 new), passed: 242 Build time: 00:49:59
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudies: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyAssaySummary: <no details avaliable>

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


Likewise - You don't need to repeat external function documentation, unless it is of special note.
I'm less certain this is wrong here, but I think the comment emphasis is wrong - the focus should be that the null character does get to the output string, rather than telling us what snprintf() does.


Confusing name; maybe change to ``stripped_furl``.


Right, that can be removed.  Left over from the copy from Nuget.core. I cleaned up the others but not this.  I'll get it in a follow up pr


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


This should probably append the stripped furl.


Unlike https://github.com/dotnet/llilc/pull/1003/files, I don't see update for Windows here.


Remove this, or change it to a logging statement.


OK, thanks. That's what I thought...but wasn't sure. I didn't know if the email had messed up the formatting or if this was an actual test case.


thanks :)


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (2050 of 2051 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-99.95%25-yellow.svg)


faaast :P


Glad to see this topic considered in testing.


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


Hardcoded


Again, more unnecessary doc of someone else's API.

It may be significant to tell us how long we can rely on `name` pointing to the correct thing.


This commit is just a copy & paste of the method from https://github.com/sequelize/sequelize/blob/v3.17.1/lib/dialects/abstract/query.js.  I hate to cherry-pick just the method in this way but cherry-picking just what we wanted with Git was too hard - there were too many changes between our code and the latest Sequelize.


What about `optional<any>`?


Just a find/replace `.model` to `.daoFactory`


@aliceh: Why revert this?  I added a more specific selector since sometimes image selection would fail.  Also note this updated selector assumes the environ you're testing on has commit https://github.com/eucalyptus/eucaconsole/pull/1757/files#diff-4cfe2e9a31eb0ef19129d2f3b22196adR34 (which is also in this branch)


This TODO is good, but hidden because of the previous irrelevant doc.


Shouldn't this be 0A 0000... meaning decimal 10?


Shouldn't this be added to the ``introducers`` file and not to ``tahoe.cfg``, if the latter setting is deprecated?


hahahah no profe ya revise eso ese no soy yo mi cuenta es JuliocCuellar y
mi archivo se llama igual, hahahah y mi mama me dice así cuando se enoja :P

2016-01-13 13:57 GMT-06:00 ffranjavierutez <notifications@github.com>:

> CESAR! El nombre del archivo tiene que ser TU NOMBRE...no "respuesta".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ffranjavierutez/gitdesdeapps3/commit/6dc4550729347cc83c514c0644f7c76b024b3c83#commitcomment-15432964>
> .
>


Should we require the extra "padding" bits in ``hashListBytes`` to be 0 during deserialization?  If so, we could add a test case in addition to the length mutations that flips some of those bits on.


True


Please make it ``BASEDIR/private/introducers``, so that the furls can be kept properly secret.


Why `unsigned char`? It is usual for a regular printable chara string to use `char`


develop (rpg777) - i386-Windows-7-VisualStudio-12: OK (2348 of 2384 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.49%25-yellow.svg)


jenkins couldn't configure on Mirko's machine.
http://128.250.120.238:9080/job/uw2/18

in previous builds, though it eventually failed, jenkins got past the config


LGTM.


For code clarity it might be better to break out the field translation into a separate function.


s/there'is/there is/


Wrap this comment.


Beginning of a new era.


Probably don't want the output to be in px...


We normally only indent 4 chars here.


Testing


There should be a default case to ensure that we don't carry on if the enum value isn't one we recognise.


And again. here you talk about `lastpos` but there is no `lastpos` here (only, I presume, in the OpenSSL doc).


Odd indentation.


makes sense :)

Am Donnerstag, 14. Januar 2016 schrieb Oriol Ferrer Mesià :

> jaja yep i just didn't want more dependencies... this is ghetto enough and
> it can be 100% disabled with a macro
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/armadillu/ofxAutoTexture/commit/b545e059ab353d6853290ab891fa6c04d3ff267b#commitcomment-15461796>
> .
>


This would also be useful under Common imo.


This line is not needed (as above)


This line is useful explanation, but it might be useful to say how long the pointer will actually be good.


What about when the collection contains null?


This is an impressive amount of work, I wish we'd added it more incrementally. Better late than never!


Pre-check latestVersionInRepository to null will allow to make descriptorToInstall not-nullable. This will simplify reasoning about !! in line 118.


I just remembered we need ``explicit`` on all ctor's which *may* be called with exactly one argument. This is one example.


But should i leave it as a `todo`, so it causes an "unexpected success" on systems with bigger than 32-bit `time_t`?

I spent a little while trying to figure out how to tell what size a system's `time_t` is but have not found a way (other than trying to create a post-2038 value and catching the exception). I do know that it is not at all the same question as "is there a 64-bit CPU".


Is this really the best Java has to offer w.r.t. sorting?


Yeah, if we copy the solution Stefan committed, we won't need the `i`.


Oh man - has this bug been here this whole time? This would explain some things.


Is it possible to insert break here?


Mmm el commit me marca de esta cuenta o de quien es esta cuenta?


I've reviewed this change and left a bunch of comments.

I hop they're useful.


It's the best for a copy-based sort.


just returning the promise (w/o calling `done()`) should be sufficient.


Are you sure iOS doesn't expect null for "undefined" strings? This appeared to be the case when I was originally testing the accessory information service. (and this was before this `props` bug above)


LGTM, unless they aren't used elsewhere in which case dead code should be removed. We generally have a serious need for coverage analysis. Added #607 as future task.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1505 of 1505 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.69%25-green.svg)


good catch


While I can see that the variable might conceivably not be initialized, from my understanding that warning would have existed with the lower code block before my commit. I'm not sure what the solution is.


Very cool


Done


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


Addressed in https://github.com/smarterclayton/kubernetes/commit/5cb6bd668a7ec07b0ec9d32513defa86363d1f21

On Thu, Jan 14, 2016 at 4:42 PM, Chao Xu <notifications@github.com> wrote:
> cc @nikhiljindal
>
> —
> Reply to this email directly or view it on GitHub.


No; I want to ensure users of the RCs will have sufficient time to update.


I was considering doing this at one point, but I was so scared about how much lag this could cause, when people spam nonsense commands, I chickened out without testing it.

Back then it was a quite common attack on bukkit servers, no idea if it's still a big issue, I seem to recall spigot added something to negate it at one point (I know it never got merged into craftbukkit).


One of the other things to consider in #2404.


Yes, I do use it.
Fixes all the code style by itself.


Please visit my website paulashealthyliving.its.com


Yeah, that's one thing we're working on currently.  Managing mutable data structures within a compact while maintaining the invariant that pointers never escape (there was a mail about it on the CNF mailing list).


Oh, btw, this isn't the latest version of this branch! In fact, this version of this commit has never existed on any branches on tahoe-lafs/tahoe-lafs but these comments are here and not on leif/tahoe-lafs where this is on a branch. :( the latest branch is https://github.com/tahoe-lafs/tahoe-lafs/compare/master...leif:1973.status-last-rx-rebased.8 which is linked in a comment in #223 (as is this commit, i think, now that I've mentioned #223 here).

I'm not really appreciating github's workflow.


Done.


Good. Also, let's add a follow up which mutates one tree and asserts roots do not equal.


LGTM. Can use more test coverage (as always in this stage).


copy/pasting this comment from an old commit where i just left it... ugh github. now i'll say #223 to mention this comment on the pull request.

Should i leave this test marked as a todo, so it causes an "unexpected success" on systems with bigger than 32-bit time_t?

I spent a little while trying to figure out how to tell what size a system's time_t is but have not found a way (other than trying to create a post-2038 value and catching the exception). I do know that it is not at all the same question as "is there a 64-bit CPU".


LGTM.


It is, by definition, since the base class is virtual.  But you are right, it's better to add the keyword.


interesting


Test for !api.Errors.IsNotFound(err), just means the node was not found and we can move on to next case.


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


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


Also safe to move on to next test case (no node object, get version from next option)


I wonder what's the reason adding this line?? It will overwrite the `contentInset.top` from the top level if top bar's height is bigger than 0.


Ok, can you cast ram_kernel_size to an int and then compare with less then 0? That sounds like the best option to me.


What about just using `pull-right` instead of this class?


不是说在解救者参与过烧烧烧才中出么


How do the *Plain and *Thing tests differ?


What is the purpose of raising an exception in a script?  It's just going to appear in the Jenkins log anyway, isn't it?


Can just use session.Output(cmd)


Eh… Not the best test.  ;-)


/cc @dterror @drdee

This was blocking us from deploying Camus


I like this, but I'd prefer if the icon would use the complete height of the message box. I.e. instead of
``` xml
<ContentPresenter Grid.Row="1"
                  Grid.Column="0"
                  Margin="10,0"
                  HorizontalAlignment="Right"
                  ContentTemplate="{TemplateBinding IconTemplate}"
                  Content="{TemplateBinding Icon}" />
```
I suggest
``` xml
<ContentPresenter Grid.Row="0"
                  Grid.RowSpan="2"
                  Grid.Column="0"
                  Margin="10,0"
                  HorizontalAlignment="Right"
                  ContentTemplate="{TemplateBinding IconTemplate}"
                  Content="{TemplateBinding Icon}" />
```
(the difference is `Grid.Row="0"` and `Grid.RowSpan="2"`.


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


这个条件不对，self.people_str 是所有人的字符串表示。应该用 `self.from_['username'] in self.people` 然后在 `process_update` 里面 set `stake.from_`


Thanks for the review! I'll to try to address all of your comments in the very near future, and write a few web tests.


Ah cool, I took the logs out. If I were to make the parser a package, I'd need to come up with something.


`PLAIN` defines it as a `BasicConfigKey(Set.class,...)` whereas `THING` uses `SetConfigKey` which is smart about looking at FQN-style children (e.g. config key `my.map` will give `{x: 4}` if it sees `my.map.x=4`)


@TomokoKuroki Regarding your bypass, if you'd like me to help improve it's effectiveness, message me via Twitter with details regarding it's execution. We may be able to prolong your disconnection, though I can't do much for you without any background information.

Twitter handle is neet_jn at https://twitter.com/neet_jn.


[![Head of a Philosopher](http://api.artsmia.org/images/72/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=72)


[![View up the Grand Canal toward the Rialto](http://api.artsmia.org/images/1317/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1317)


[![Pope Benedict XIV Presenting the Encyclical "Ex Omnibus" to the Comte de Stainville, Later Duc de Choiseul](http://api.artsmia.org/images/1463/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1463)


[![Still Life](http://api.artsmia.org/images/1945/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1945)


[![Shepherd Boy Pointing at Tobias and the Angel](http://api.artsmia.org/images/1401/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1401)


[![Red-and-blue-laced Suit of Armor from the Kii Tokugawa Family](http://api.artsmia.org/images/108860/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=108860)


Nice!


@trueadm

The isomorphism bug you found is fixed by https://github.com/leeoniya/domvm/commit/dbb6cb2be38a857e1fef431c6b697d66e5fe009e, which merges all adjacent plain value nodes.

As luck would have it, I had to tweak your code to unwrap the child array since it now hit one of the two DSL exceptions as result of moving to JSONML (first child of explicit children array cannot be a text node) , Heh, guess that case was infinitely more common than anticipated ;)

Now works as expected:

```js
domvm.useRaf = false;

function View(vm, person) {
	return {
		render: function() {
			return ["a", {href: "/cows"},
				person.name,
				" 123" + person.age,
				["br"],
				["strong", "bar"],
				person.age,
			];
		}
	};
}

var elem = document.getElementById("foo");

var model = {
	name: 'test',
	age: 5
};

var vm = domvm(View, model);
var html = vm.html();
elem.innerHTML = html;

var vm2 = domvm(View, model);
vm2.attach(elem.firstChild);
model.name = 'test 2';
vm2.redraw();
```


this does not catch the case where the value is a nullable string. In that scenario, valBytes will always be null. Better to do something like:

```
Schema valueFieldSchema = input.getSchema().getField(kvTableConfig.valueField);
if (valueFieldSchema == null) {
... some error handling...
}
Object val = input.get(kvTableConfig.valueField);
if (val != null) {
  Schema.Type valueFieldType = valueFieldSchema.isNullable() ? valueFieldSchema.getNonNullable().getType() : valueFieldSchema.getType();
  if (valueFieldType == Schema.Type.STRING) {
    ...
  } else if (valueFieldType == Schema.Type.BYTES) {
    ...
  }
}

```


> a commit message must not exceed a 60 characters
Only the *subject* of commit message should not exceed 60 characters. The complete message can be of any length.


Indeed. https://github.com/purescript-contrib/purescript-profunctor-lenses looks like a pretty interesting use case for them too, but I think the polymorphic types might get a bit clunky to translate directly to JS ... dunno, maybe I'll have a play with it over the weekend :)


Hi,

I just added this to the main codebase(with a couple integration changes):
https://github.com/openemr/openemr/commit/0e78558de82043d33ee818db1f71b6918a01cae3

thanks for the contribution!
-brady


This fix worked for me.


there is a utility Schema.nullableOf() method you can use


NullableType


there's no schema applied to that file


Link?


Joshua Teel


It's a really fun hobby. thanks :)


Might be worth mentioning that the staged changes + the changes in the last commit will be combined into one single commit. If there are no staged changes it will be equivalent to re-writing the commit message only.


test


this may cause simulator not working by default as the timer does not forward ...


pré-requis et slide avant l'archi ! sinon c'est parfait !

un truc éventuellement que tu peux rajouter c'est un lien vers de la doc officielle si y'en a.


Looks like a stray tab.


@StephenBonikowsky, @mconnew - this change puts us out of consistency with the dir.props in CoreFx. #695 contains the way in which most of the other CoreFx projects handle this situation.

Should we revert this back to "AnyOS"?


kconfig is missing:
drivers/Kconfig:193: can't open file "drivers/hw_fac_info/Kconfig"
/home/desalesouche/cm13.0/kernel/huawei/msm8916-caf/scripts/kconfig/Makefile:105: recipe for target 'g620s_defconfig' failed


We probably still need the TODO. Can you add a ticket (or tickets - automated test and perceptual diff) for that?


誤って削除


Great work dan! What does -1 mean mabye add that in documentation?


oops...my spelling strikes again!


rrrdsdds


oh and that's how you officially do it...I learn something new every day :+1:


Seems a bit pointless...


Don't we want this as an `expect` so the rest of the suite will continue?


Hi ,@heckj  In my opinion, here should be "{{user `atlas_username`}}/{{user `atlas_name`}}" get from the env. Like you define before.Otherwise will coming out this error(Post-processor failed: Error uploading (1246609288 bytes): resource not found) by using their own atlas account.


Good afternoon, dear respective feiyuhug!

I have launched your LeNet-5 firstly in training mode train_net(train_covnet, logfile, 2, [0.005, 0.001], 60000), than after training have been completed (took train_time: 33787.64303088188) I launched test mode on 500 samples (test_net(train_covnet, logfile, 500)) but received only 42 correctly recognized patterns out of 500 (test_correct_rate: 0.084 correct_num  42). Is it something wrong in your Code? Have a forward propagation working correctly? Could you please say where I can find the final version of your LeNet-5, as this version not correctly trains network resulting in recognizing low amount of samples. Thank you in advance!


Sure - a RESTMapping is expected to define what object is desired.  The caller is responsible for setting Resource or Kind if they wish it to be different, and resetting the GVK violates the intent of the method (and prevents callers from specifying their exact kind).


newb


I think so but I'll double check with a test.

On Wed, Jan 13, 2016 at 5:40 PM, Chao Xu <notifications@github.com> wrote:

> Shall it be if (c.encodeVersion == nil && len(overrides)==0) ||
> isUnversioned?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kubernetes/kubernetes/commit/08270411316784f6cd8c740225529dfc05430d5a#commitcomment-15436559>
> .
>


... so today i randomly came across this and ...
How this is even working? If it is..
I don't understand it... it uses reflection?... and somehow gets to client? Then finds the server and sends connect packet?? what? It is not april 1st.
If it is/was working, can you explain to me what is happening? Because a while back i made SLP plugin and if i would send anything other that was not on wiki.vg documentation the connection would close...


Oops. Thanks, Dmitri.


At some point it might make sense for the AHRS to return a blended gyro value based on how it's using the gyros although at the moment it simply averages the first two values as you've done here so it wouldn't make any functional difference.


Is it a better idea to use reference?
e.g. `const BitTorrent::SessionStatus &sessionStatus`


Seems legit. Thanks, Slava!


LGTM


Didn't end up needing it because I just iterate over the metrics vector instead and use that to create the dict.


This code runs but the rake says its not right for some reason?


Can we make this more elaborative? Something like `Git at Kayako`?


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


I'd prefer that the logging be done outside of the individual libraries.  The reason is that dataflash is already dependent upon every other library.  Making Mount dependent upon Dataflash means that it is now also dependent upon every other library.  If we continue doing this we will find that we cannot separate anything from anything else.


@cwillmor, would you remind reviewing my commits in the series up to this one? They'll make more sense in forward order.

Recent series: f8bb54f through bbc14af (this commit)
Previous (already in branch, but possibly useful background): b45a69e, fe00083, ad94542


Not sure if we need to include *everything* given in the link in our post, when we have given the link it-self. Generally speaking, we give a link and quote an excerpt/part from it, isn't it?


Correct @AcadGild


I approve of this commit message!


This `end` is for ending the `array.each` block, so it should be indented one further level.


@doowb there is a setter for this https://github.com/assemble/assemble/blob/e5d6182d1d6b53b49697e3e5bb51f8953d5b3036/index.js#L141


These lines beginning with `counts` should be indented one further level in, because they're within the method.


ありがとうございます。修正しました。


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


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


私もそう思いますので、そのまま「複数の」を頂きました。


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


How to use a ResultSet will be covered some topics ahead. This is not the way.


@mostaphaRoudsari ,
I know I already said this over chat but thank you for catching this!
-Chris


dn


It should accept a Query as parameter.


@jonschlinkert  that's using the "setter"
The issue was the name was coming through as "generate" instead of "assemble" when my assemblefile.js exports a function. This is similar to the logic in `generate` to change the name from "templates" to "generate".


### [Codacy](https://www.codacy.com/app/waldmanl/cubes/commit?cid=24679859) detected an issue:
#### Message: `Missing semicolon.`

#### Occurred on:
+ **Commit**:  294bd57d5548613018f80ac4f27a86b4f626aaed
+ **File**:    [js/GUI.js](https://github.com/Johnsmith0508/cubes/blob/294bd57d5548613018f80ac4f27a86b4f626aaed/js/GUI.js)
+ **LineNum**: [1](https://github.com/Johnsmith0508/cubes/blob/294bd57d5548613018f80ac4f27a86b4f626aaed/js/GUI.js#L1)
+ **Code**:    `if (typeof THREE === "undefined") throw "THREE.JS not found"`

#### Currently on:
+ **Commit**:  b98ba990a51852f922c2ef9a9f37c5f7f095b21e
+ **File**:    [js/GUI.js](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/js/GUI.js)
+ **LineNum**: [1](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/js/GUI.js#L1)


It should not implements Query.


saw approval from shino
at https://github.com/basho/riak_cs/commit/6f5e1357cc0c5ec1bca9c1dc8625326f98c542fc


So after your merge of #695 we don't need this anymore? If so then I would say yes.


merging basho/riak_cs/bugfix/cleanup-manifests = 6f5e1357 into borshop-integration-1280-bugfix/cleanup-manifests


basho/riak_cs/bugfix/cleanup-manifests = 6f5e1357 merged ok, testing candidate = 8018a32b


add javadoc to all interfaces methods.


Sorry if I wasn't clear. We still want Revision in either case.


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


this is not thread safe, is ok for this training but keep an eye on it.


Is `v <= 1` the right check? Less than `1` would be zero, which means an element is not in the array at all.


do not push eclipse config files.


###Please Fix so the code climate looks good ### [Codacy](https://www.codacy.com/app/waldmanl/cubes/commit?cid=24679859) detected an issue:
#### Message: `Strings must use doublequote.`

#### Occurred on:
+ **Commit**:  294bd57d5548613018f80ac4f27a86b4f626aaed
+ **File**:    [js/GUI.js](https://github.com/Johnsmith0508/cubes/blob/294bd57d5548613018f80ac4f27a86b4f626aaed/js/GUI.js)
+ **LineNum**: [26](https://github.com/Johnsmith0508/cubes/blob/294bd57d5548613018f80ac4f27a86b4f626aaed/js/GUI.js#L26)
+ **Code**:    `		canvas.style.border = '3px solid #000';`

#### Currently on:
+ **Commit**:  b98ba990a51852f922c2ef9a9f37c5f7f095b21e
+ **File**:    [js/GUI.js](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/js/GUI.js)
+ **LineNum**: [28](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/js/GUI.js#L28)


Ajuste de estilo del home


This is one funny commit.


### [Codacy](https://www.codacy.com/app/waldmanl/cubes/commit?cid=24679859) detected an issue:
#### Message: `"force" is defined but never used`

#### Occurred on:
+ **Commit**:  95ef7849b4acabe0df97f9941b2eebb8cb29df17
+ **File**:    [js/clientPhysics.js](https://github.com/Johnsmith0508/cubes/blob/95ef7849b4acabe0df97f9941b2eebb8cb29df17/js/clientPhysics.js)
+ **LineNum**: [5](https://github.com/Johnsmith0508/cubes/blob/95ef7849b4acabe0df97f9941b2eebb8cb29df17/js/clientPhysics.js#L5)
+ **Code**:    `var force = {`

#### Currently on:
+ **Commit**:  b98ba990a51852f922c2ef9a9f37c5f7f095b21e
+ **File**:    [js/clientPhysics.js](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/js/clientPhysics.js)
+ **LineNum**: [5](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/js/clientPhysics.js#L5)


oh, right. this is the python bindings...


got many more error after deleting


### [Codacy](https://www.codacy.com/app/waldmanl/cubes/commit?cid=24679859) detected an issue:
#### Message: `"$" is not defined.`

#### Occurred on:
+ **Commit**:  ec4d161cf0dab7cd4ca88a258857cd5bf53698fe
+ **File**:    [js/chatlogic.js](https://github.com/Johnsmith0508/cubes/blob/ec4d161cf0dab7cd4ca88a258857cd5bf53698fe/js/chatlogic.js)
+ **LineNum**: [6](https://github.com/Johnsmith0508/cubes/blob/ec4d161cf0dab7cd4ca88a258857cd5bf53698fe/js/chatlogic.js#L6)
+ **Code**:    `$('#msgIn').on('keyup',function(e)`

#### Currently on:
+ **Commit**:  b98ba990a51852f922c2ef9a9f37c5f7f095b21e
+ **File**:    [js/chatlogic.js](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/js/chatlogic.js)
+ **LineNum**: [11](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/js/chatlogic.js#L11)

     do the thing : /*global $ THREE GUI*/ ect


installed MAJOR version


### [Codacy](https://www.codacy.com/app/waldmanl/cubes/commit?cid=24679859) detected an issue:
#### Message: `Values of 0 shouldn't have units specified.`

#### Occurred on:
+ **Commit**:  cf883a1b2cbe12e90c385728aacbf40d94fb635f
+ **File**:    [stylesheet.css](https://github.com/Johnsmith0508/cubes/blob/cf883a1b2cbe12e90c385728aacbf40d94fb635f/stylesheet.css)
+ **LineNum**: [3](https://github.com/Johnsmith0508/cubes/blob/cf883a1b2cbe12e90c385728aacbf40d94fb635f/stylesheet.css#L3)
+ **Code**:    `right: 0px;`

#### Currently on:
+ **Commit**:  b98ba990a51852f922c2ef9a9f37c5f7f095b21e
+ **File**:    [css/stylesheet.css](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/css/stylesheet.css)
+ **LineNum**: [6](https://github.com/Johnsmith0508/cubes/blob/b98ba990a51852f922c2ef9a9f37c5f7f095b21e/css/stylesheet.css#L6)


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


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1759


https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/
感觉像是在恶搞，因为torvalds主页上没找到这个commit


do you need the `.*`?


same question as above


111639676-UniqueThermostats (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: OK (2347 of 2384 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.45%25-yellow.svg)


generally you would need the .* if you're anchoring the regexp by \A \z to get the WHOLE string.


Can't this just be EqualsLoc?


throw in a test that does not have the beta stuff


I think we can just reuse the regular diagnostic (nonliteral_enum_case_raw_value). That's what they need to correct to anyway.


Er, I apparently forgot both what peekToken does and what I asked you to do earlier. This is exactly right.


チケット番号を間違った。 #301 が正しい。


@cos yes, it should be correct if Jenkins succeeds.

BTW, however, I talked to @radumereuta 8 hours ago regarding this, and at that time, @radumereuta wanted to reorganize negative test scripts, which might not be happy with this. Did you talked to him as well?


Do we know of anything else that will trigger this error? I would guess the reason we didn't diagnose anything here before is because anything else would get its own diagnostic first anyway.


maybe a few tests like this? (adding a method on this class to give you the major_version)
```
subject(:node_package_major_version) { NodePackageVersion.new(package_json).major_version }
```

and why 2 classes in one file -- should be one class per file.


GCS_Common.cpp already logs changes to parameter values (both in ardupilot/master and apparently solo/master) so doesn't this lead to duplicating records in the dataflash?  .. or is this perhaps dealing with the case where the gimbal itself has updated it's copy of the parameter values and this would not be logged otherwise?


Formatting.


also put enums into own files


Looks great, thanks!

Fix up the failing Travis build please.

Fix up the Doxygen comments for stream_params and the new in/out params.


@CTXz Good luck with this bro!


Un placer colaborar con tu proyecto. Sigo a la ordene de colaborar contigo.


The isPassworded variable is set with this line: var isPassworded = serverInfo.passworded !== undefined;

There's really no reason for this check when you already know its a boolean value at this point


Try reset


Total win minus loss margin is given as -0.282%. Total win minus loss margin is the same as the expected value of the game. The house edge is equal to negative one times the expected value of the game. Accordingly, the house edge is given as 0.282%. That means the following.  If the player consistently uses the set of combination-dependent-playing-strategy tables to make decisions on how to play the player's hand, then the house will retain an average of 0.282% of each game wager in the long run with strictly average luck.


Leave a comment:
happy new year 2016
every one!
Kmkamruzzaman.com


I think Samnang initially thought we should nest them, but we never got around to it, I will do it now. `NodePackageVersion` is sort of weird unless it's in the context of a version checker.


Bad replace.


I don't see this anywhere. However, would you prefer this next to the
copyright notice? In effect, this is volunteer work and I am perfectly
fine with just donating the code to NV Access.

On 1/13/2016 6:06 PM, josephsl wrote:
> Hi,
>
> I should have (in 2014 when I worked on #4584)… That’s fine (I believe
> users should deserve greater credit than us, as they are the stars of
> the show). I do know that we may need to update file headers (that’s a
> separate ticket), thereby giving people credit for their contributions.
>
>
>
> From: derekriemer [mailto:notifications@github.com]
> Sent: Wednesday, January 13, 2016 4:49 PM
> To: derekriemer/nvda <nvda@noreply.github.com>
> Cc: josephsl <joseph.lee22590@gmail.com>
> Subject: Re: [nvda] The case sensitivity of the last find is
> remembered when searching using NVDA's find dialog. For Example, if
> Food is searched for with case sensitivity on Food will be found when
> pressing NVDA+f3, but food will not. (33189af)
>
>
>
> Are you giving your self credit there anywhere?
>
> On 1/8/2016 12:33 PM, josephsl wrote:
> > Hi, Take time (the code looks fine to me; in effect, you are extending
> > my own work (#4584)).
> >
> >
> >
> > From: derekriemer [mailto:notifications@github.com]
> > Sent: Friday, January 8, 2016 11:25 AM
> > To: derekriemer/nvda <nvda@noreply.github.com>
> > Cc: josephsl <joseph.lee22590@gmail.com>
> > Subject: Re: [nvda] The case sensitivity of the last find is
> > remembered when searching using NVDA's find dialog. For Example, if
> > Food is searched for with case sensitivity on Food will be found when
> > pressing NVDA+f3, but food will not. (33189af)
> >
> >
> >
> > OK do you mind waiting until tomorrow night until I do this or doing
> > it yourself? I am going to be away from my computer for the night.
> > Because I'll be in the mountains.
> >
> > Sent from my iPhone
> > Please disregard errors as this is a smaller device.
> >
> > > On Jan 8, 2016, at 11:53, josephsl <notifications@github.com> wrote:
> > >
> > > Hi, They would go on the copyright line.
> > >
> > >
> > >
> > > From: derekriemer [mailto:notifications@github.com]
> > > Sent: Friday, January 8, 2016 10:39 AM
> > > To: derekriemer/nvda <nvda@noreply.github.com>
> > > Cc: josephsl <joseph.lee22590@gmail.com>
> > > Subject: Re: [nvda] The case sensitivity of the last find is
> > remembered when searching using NVDA's find dialog. For Example, if
> > Food is searched for with case sensitivity on Food will be found when
> > pressing NVDA+f3, but food will not. (33189af)
> > >
> > >
> > >
> > > Where would the names go?
> > >
> > > On 1/8/2016 10:09 AM, josephsl wrote:
> > > >
> > > > If you want, you could add two names: mine and yours, as both of us
> > > > have worked on this and this is a good time to get recognized for
> > your
> > > > work (just my thoughts).
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > >
> >
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15344152>.
> > > >
> > >
> > > --
> > >
> ------------------------------------------------------------------------
> > >
> > >
> > > Derek Riemer
> > >
> > > * Department of computer science, third year undergraduate student.
> > > * Proud user of the NVDA screen reader.
> > > * Open source enthusiast.
> > > * Member of Bridge Cu
> > > * Avid skiier.
> > >
> > > Websites:
> > > Honors portfolio <http://derekriemer.drupalgardens.com>
> > > Non-proffessional website.
> > <http://derekriemer.pythonanywhere.com/personal>
> > > Awesome little hand built weather app that rocks!
> > > <http://derekriemer.pythonanywhere.com/weather>
> > >
> > > email me at derek.riemer@colorado.edu
> <mailto:derek.riemer@colorado.edu>
> > > Phone: (303) 906-2194
> > >
> > >
> > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> >
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15346027>
>
> > .
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> >
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15346891>
>
> > .
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15347088>.
> >
>
> --
> ------------------------------------------------------------------------
>
>
> Derek Riemer
>
> * Department of computer science, third year undergraduate student.
> * Proud user of the NVDA screen reader.
> * Open source enthusiast.
> * Member of Bridge Cu
> * Avid skiier.
>
> Websites:
> Honors portfolio <http://derekriemer.com>
> Awesome little hand built weather app!
> <http://django.derekriemer.com/weather/>
>
> email me at derek.riemer@colorado.edu <mailto:derek.riemer@colorado.edu>
> Phone: (303) 906-2194
>
>
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15438878>
> .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/derekriemer/nvda/commit/33189af8de2d73ee7bcd44ab21e21d40abedd45d#commitcomment-15439183>.
>

--
------------------------------------------------------------------------

    Derek Riemer

  * Department of computer science, third year undergraduate student.
  * Proud user of the NVDA screen reader.
  * Open source enthusiast.
  * Member of Bridge Cu
  * Avid skiier.

Websites:
Honors portfolio <http://derekriemer.com>
Awesome little hand built weather app!
<http://django.derekriemer.com/weather/>

email me at derek.riemer@colorado.edu <mailto:derek.riemer@colorado.edu>
Phone: (303) 906-2194


I according to your solution about  Revert "Transparent GLSurfaceView background";   But   when Scroll the listview , bottom picture show top left or top right error
![Uploading L5}6{%E[IR~A18L@2F4(40J.png…]()