TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 27](https://teamcity.labkey.org/viewLog.html?buildId=308940&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 11 (11 new), passed: 40 Build time: 01:11:23
### Failed tests
```
org.labkey.test.tests.cds.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

org.labkey.test.tests.cds.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

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

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

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

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

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

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

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

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

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

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


develop (energy-plus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Will do, thanks.


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.82](http://build.clear-measure.com/viewLog.html?buildId=7673&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 85, ignored: 25 Build time: 00:04:37


bool as method status.  false can mean more than one thing to the consumer?  what can it mean  & how would you change the signature to better communicate intent?


Sounds good. I won't be back online for another two hours or so.


Right, I'll make this a ctor and remove the extra arguments.


Sorry, it is probably github messing up something. It looks like this to me (it looks correct in the file view):
![space](https://cloud.githubusercontent.com/assets/221874/13620161/fbeab914-e58d-11e5-873b-4e77f4394ead.png)


ObjectMapper instances are fully thread-safe so you could create it only once as a field of SwaggerBasePathRewritingFilter for performance reason.


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


TeamCity Device Info :: Tests [Build 43](http://82.35.147.206:8111/viewLog.html?buildId=219&buildTypeId=DI_Test) is now running


what is the benefit of logging the exception here and returning false?  would there be any benefits to letting the exception bubble up?  what are the pitfalls of catching the generic Exception type?


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.83](http://build.clear-measure.com/viewLog.html?buildId=7676&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


s/||/&&/?


#5536-AirConditionerVariableRefrigerantFlow-with-ThermostatOffsetPriority-crashes-with-blank-Zone-Name-for-Master-Thermostat-Location (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2135 of 2135 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)


Why are you only on for a very short time at approximately the same time every day?
To Alex


Please add a comment why this is needed.


If we're not doing null checks anyway, why not use foreach?


develop (energy-plus) - Win64-Windows-7-VisualStudio-14: Build Failed

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


Right................................................................. >.<


Perhaps add a moderate timeout here


OK I understand what you're saying, but the chances of that happening are so low it's not a priority. HTTPS for the site is something we could add at some point though.


`FE-User storagefolder uid`?


It would be a bit involved to test locally, but this is now on production - see my comment on #511.


TeamCity Device Info :: Tests [Build 43](http://82.35.147.206:8111/viewLog.html?buildId=219&buildTypeId=DI_Test) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:56


感谢，回复的太晚了，抱歉。


I don't think it makes a big difference, I can change it to a foreach loop.


#rubby


Same here.  Timeouts can sometimes stop hangs


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.83](http://build.clear-measure.com/viewLog.html?buildId=7676&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 85, ignored: 25 Build time: 00:03:44


@claudiosmweb
Passei mesmo e não reportei aqui, pois preferi quebrar a cabeça antes de receber uma de suas tantas respostas mal educadas.
Ultimamente tenho acompanhado bastante seu trabalho, que não deixa de ser muito bom, porém, se você der uma olhada em suas respostas, não só aqui, mas também nos fóruns de WP, verá que tem uma tendência a menosprezar os usuários e tratar todo mundo como se fosse burro.

Então é, já paguei só para não precisar interagir com sua falta de educação, o que acaba sendo irônico para que desenvolve plugins dependentes de doações.

Abs,


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.84](http://build.clear-measure.com/viewLog.html?buildId=7678&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


TeamCity Device Info :: Tests [Build 44](http://82.35.147.206:8111/viewLog.html?buildId=220&buildTypeId=DI_Test) is now running


@mostaphaRoudsari I got an IO error with the GH file. Any idea why that might be ?

![3-8-2016 6-33-57 pm](https://cloud.githubusercontent.com/assets/10361848/13620305/a211b270-e55c-11e5-9012-dd0a7f1167d7.png)


This seems inaccurate (same as in the super impl), it returns the length in bytes, not words.


It looks like BuildNode can never return null. Maybe turn this into an assert?


You're right, thanks :)


removed


Style nit - the rest of the if statements in this file use braces on the next line


Wondering whether we could use ApiListing from springfox here in place of a LinkedHashMap.

@cbornet do you know it?


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.84](http://build.clear-measure.com/viewLog.html?buildId=7678&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 85, ignored: 25 Build time: 00:03:47


This feels slightly awkward to me. I am generally a fan of the pattern in Scala of having the default constructor take the essential constituents of the type in question. In this case, a DomainClient has an ElasticSearchClient and it has an alias or an index name. Then auxiliary constructors can be defined in the companion object. That said, we're not using this pattern elsewhere however, so it probably doesn't make sense to make this change in this commit.


Can you please keep boto imports together?


why do you need the grep here?  what other pods would be running in the current project/namespace?


Hahahahaha ! Fiquei sem ar agora também!!Valeu demais pelo comentário !Fiquei supreso quando o vi: "caramba, Tomaz por aqui" hahahaha. A princípio to escrevendo sem me preocupar muito com a estrutura, sentarei com o Paulo para ele propor uma divisão adequada. Minha idéia era seguir aquela estrutura do Maurício e da Millene, mas o Paulo recomendou que não o fizesse pois, para o meu caso, talvez não seja o melhor. Valeu mesmo =) !


I'm not super familiar with this red-black tree implementation, but can n.left be null here?


develop (energy-plus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1021 of 1021 tests passed, 0 test warnings)

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


@tejohnso how would we handle rollbacks if we find an issue in our release?


TeamCity Device Info :: Tests [Build 44](http://82.35.147.206:8111/viewLog.html?buildId=220&buildTypeId=DI_Test) outcome was **FAILURE**
Summary: Tests passed: 35; gradle exception (new); exit code 1 Build time: 00:05:03


Yep, I missed this one too.


The software needs a good rewrite to work with SLI, sadly.


Yes... will fix.


Maybe rename to rtcpListeners?


Dragons indeed! It appears `/app/views/dragons.js` didn't get checked in?


Don't you think the name was more clear before?


You don't need to log the exception message, it is done for you when exception is the last arg:

    log.error("Swagger-docs filter error", e);

Beside logging, should we call context.setResponseStatusCode(xxx) ?


Rename to addRtcpListener?


oh wow, good catch. Guess I forgot to delete this commit when I made this repo public


Just curious, how did you find this @nickhoffman ?


What might be even better is to install them into a private environment.  We do that in fonttools for Travis.  You can use the virtualenv module for that.  I haven't fully explored it myself, but see fonttools/.travis/ for example.

But that's a separate project :)


That's a good point, we absolutely lack the REST API to query that information right now and that's likely causing a bunch of bugs when copying snapshots but also when doing say "lxc copy src:a/snap0 b:blah" where we'd need to copy the snapshot's config, devices, profiles, ... rather than the container's.


I'll add all that stuff to what's returned when we GET a container.


Does Python or `setuptools` offer any kind of abstraction for well-known system directories?  Using `autoconf` for too many years has trained me to reflexively cringe at hard-coded directories like `/etc` where I'm used to seeing `$sysconfdir`.

Not a big deal, just curious.


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.85](http://build.clear-measure.com/viewLog.html?buildId=7684&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


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


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


Yeah, quite agree, also the index pattern could use a slightly more precise name to indicate that this is the naming scheme for the indexes that get aliased.


This is great!


Can we change this to:

```
if (!error && response.statusCode >= 400) {
    if (data) {
        // ...
    } else {
        // ...
    }
}


Below you test whether `users_path` is a tuple or list.  Here it's marked `str` only.


I think errorForLoginViewFailedWithResponse would be a better choice of method name..


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.85](http://build.clear-measure.com/viewLog.html?buildId=7684&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 85, ignored: 25 Build time: 00:03:53


Yeah, I know.  Sorry, I have a habit of not uploading to Github.  But I already fixed that on my local version several days ago.  I disagree with your use of strings.  I would rather do

`int(input("text"))"`

and then check for numbers rather than checking for strings.  If I'm only getting a number input, I like to use integer inputs rather than string inputs.  Also, the reason I had the bug in the first place was because I'm getting too used to Ruby.  Ruby lets you do statements like

`if variable == 1 or 3 or 5` without having to restate the variable.  But I've gotten more used to the Python syntax.  I was also messing around with bitwise operators, but I don't think I'll use them in the future.


I don't believe so, no. I've generally seen system specific items be left to the package manager though occasionally I've seen some people use some custom sensing code in setup.


Do we need a separate method? Can we simply set the MSLoginViewErrorResponseData key to nil in the user info dictionary?


Good catch. Will update.


actually, this commit references issue #55 which must be re-opened


Just to be pedantic, this could also be: `if not hasattr(users_path, '__iter__'):`


dragons.js now committed.


Is this extractor working after all? Safari is now using Kaltura as media provider, and it keeps trying to use Brightcov?


I feel like you can capture the below three lines in a lambda or list comprehension to set the variable:

for example:

```
cloud_watch_alarms = [alarm for alarm in self.conn_cloudwatch.describe_alarms() if self.project in alarm.name and self.env in alarm.name]
```

Then you can iterate through the list cloud_watch_alarms and just disable them.

Let me know if you have any questions.


This will conflict with master branch, I changed it to downloads.sourceforge.net which should round-robin to a mirror.


Some initial problems with this approach:
getVClassURI() only returns one class. Some individuals have multiple class assertions
Creates a partial duplicate of exclusion list made for the Solr index
head.ftl is used for all pages, not just individuals.


@jeffmo @gabelevi this commit broke OSS trunk because npm is set up to install version 0.21.
https://travis-ci.org/facebook/react-native/builds/114618771

I prepared and landing a PR with update to 0.22 but still get Flow errors.
https://circleci.com/gh/facebook/react-native/3663

Do we actually run flow in fbsource?
Could you please do a patch?
I can't do it from home right now, going to sleep :)


lol thats all..


`		        var file = _this.fileArray[i];
			var fileName = file.replace(_this.options.buildPath, '');
			if (fileName.startsWith('/')) {
				fileName = fileName.substr(1);
			}
`
这段我用
`var fileName = file.split('/').pop();`
的方式是不是更加简洁？


hello


If you revert changes made by others, please invest the time to figure out WHY these changes have been made in the first place. This theme breaks the navigation!!!

There's another issue related to adjusting the layout properly. Please leave it this way until someone finds time to do #27 properly. Thanks!


ElemType bestAxisScore = DBL_MAX; // it should work only if ElemType=double

I think it should be fixed. RStarTreeSplit has the same problem.


@sariths can you save the log and post it here. That might give us a better idea.
If it's happening when you try to open the file you can try to lock the solver, open the file and set the Boolean to False.


違和感を感じつつ、許容範囲内かなと放置した部分ですが、折角なので修正を検討します。


These could probably be set with the `FLAG_NOSTORE` flag. Is there value in changing these at runtime? Unlikely. I imagine these only existing in config files. I feel like it'd be weird and/or problematic to change at runtime.


why the default is '1' here? (I thought it was "1024")? Also, if it's a string, should we quote it?

Do you accept single size representation here? clarify in the help text here.


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


What is your top priority in:

   - Rest of GMail API, like users.messages or users.threads ...

Ross

On Mon, Mar 7, 2016 at 10:44 PM, espiman <notifications@github.com> wrote:

> Thanks Ross.
>
> The commnad works perfectly, but in my domain I have more than 30.000
> contacts and take a long time because download set of 25 each time. Can I
> increase thay limit?
>
> Regards.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/taers232c/GAMADV-N/commit/c85351fff48ee8261fe7fec957561b96d4f177f2#commitcomment-16554432>
> .
>

--
Ross Scroggs
ross.scroggs@gmail.com


please use more than 1 as default.


The problem is known to Python devs at least:
https://wiki.python.org/moin/Distutils/Proposals/AutoconfLikeOptions


I'm having a hard time with this and I'm also not sure if my approach is a preferable one.

The idea:
* Validation of input forms happen after button click.
* Validation consists of checking each field of `this.state` inside `<JobForm />`, since changes on individual `<Input />` will update the state of `<JobForm />`
* If there is an invalid form, render a message below the `<Input />`.

My problem:
* Since I am doing the validation inside `<JobForm />`, I need to communicate the presence of an error from `<JobForm />` to `<Input />`


Does this load fine without `https:`? This is an intended format to allow for both http and https loading on different setups.


Using `^` seems good, I'm unsure why I didn't do that to begin with (likely I just changed the number and didn't pay attention to the `~` here)...


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.7-LibC++: OK (390 of 390 tests passed, 0 test warnings)

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


Happy Public Birthday!


Sugar: component callback usage


\o/


many rely on this to decide whether to open the message or not :D
probably we comment out online/offline status icons until we implement that later


the response is:

> mcc=250,mnc=1,lac=32000,cellid=221,aol=5,subscriberState=0,vlrNumber=5555555666


When I was trying this, it does not load without https...

Sent from my iPhone

> On Mar 7, 2016, at 22:00, Ben Zhang <notifications@github.com> wrote:
>
> Does this load fine without https:? This is an intended format to allow for both http and https loading on different setups.
>
> —
> Reply to this email directly or view it on GitHub.
>


Did you try running a webserver? `npm install -g serve && serve`


Should it read simply "Its conventions...", i.e. delete "One of"?


Oh I guess I can try that ...

Sent from my iPhone

> On Mar 7, 2016, at 22:04, Ben Zhang <notifications@github.com> wrote:
>
> Did you try running a webserver? npm install -g serve && serve
>
> —
> Reply to this email directly or view it on GitHub.
>


Sure.


Ruby indentation is 2 spaces :)


r+ @dataviruset this one has been around for awhile.  Should allow different menus for CS:S and CS:GO now


This function gets called by the case pillow's `transform_case_for_elasticsearch()` function. It assumes that the form index has already been populated. But, I'm worried that isn't a safe assumption to make. Do you guys know of a good way to ensure it's populated, or have any suggestions for how best to fall back if its not?
@snopoke @benrudolph


thanks @dmick , will fix.


@hannahfrost I see some other spots in the site where we craft relative links like so:

```markdown
... [March 2016 blog post](/2016/03/07/content_types_news.html) ...
```

Give that one a shot?


agree


Awesome XD let's simulate the production environment as much as possible. I do find the web server easier to work with than opening local files.


Common practice to avoid empty parens unless they're needed for clarity


@PramodSSImmaneni these type of changes should not be made in release branches.


I wasn't sure if this was intentional or not but it was preventing the event from actually firing.


Credit goes to @gh0st for most of this one


will fix, will add waiting for cluster ...


maybe we should use the presence of this to mean Hyper-V containers instead of having the separate HvPartition bool


sure


It also wasn't sending any data whatsoever but now it is.


Coooool

Sent from my iPhone

> On Mar 7, 2016, at 22:06, Ben Zhang <notifications@github.com> wrote:
>
> Awesome XD let's simulate the production environment as much as possible. I do find the web server easier to work with than opening local files.
>
> —
> Reply to this email directly or view it on GitHub.
>


Is this because of the problems cmd escaping and docker build? I really don't like that this has leaked into here...


Hi,

yes that is also an option...

I also had the impression that the numbers are absolute... but the adapter numbers from the client should always match with what is on the server (eg. you have 2 adapters 1 on astra, 1 on hotbird)


ひとまず確実にディレクトリごと削除するようにしました。
もしテストが増えてきて実行が遅いのが気になったら別の方法を考えます。


Please don't do this sort of thing.


what is this?


These changes make this code work for me now... somewhat.

Now my issue is that I'm getting an error: [error: not opened]

This is only happening for the "Data Styles" socket events. "Passthrough" events are fine.


@PramodSSImmaneni @vrozov why is the change not in master?


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


缩进不正确


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (398 of 398 tests passed, 0 test warnings)

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


Very nice!


没有测试代码


Do mention your name here :+1:


Good catch. Sorry forgot to take this out.


应该是这个问题 https://github.com/stomita/ios-imagefile-megapixel


Sorry my bad. The `Commit Access` section at https://swift.org/contributing/#contributing-code said :

> To verify that your commit access works, please make a test commit (for example, change a comment or add a blank line).


Same applies here .


Further update testing reveals no re-occurrence of the bug with this code.


> Regarding your first point, sounds like we'll have to change sbt to put the shared xsbt.api classes in a parent classloader when instantiating the compiler.

That sounds like the cleanest solution if it's realizable, in any case I'll share my approach soon which could be done in sbt itself too.

> For the second problem, we just discussed this the other day, and the idea is to put the api logic in a separate jar, say scala-compiler-extract-api.

Sounds good to me, I guess eventually this could be completely decoupled from sbt so that others may benefit from it.


Mom, i see dead projects...
They whisper to me...


Read


Closes #5.  Closes #4.


すでに`ENV['GROONGA_DATABASE_PATH']`が設定されていたら`groonga_database_path`を使ってくれないんじゃないかと思いました。


Dentro do php na rede.


Thanks :)
You forgot the line at the end though LOL


:rocket!


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9: OK (398 of 398 tests passed, 0 test warnings)

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


あ、別に`ENV['GROONGA_DATABASE_PATH']`を参照しているやつはいないから問題ないってことか。
だったら代入もしなくていいんじゃないですかねぃ。


Oh, my bad. One did not expect such strange instructions.


Looks like you are right again.

How about this

                if (alignedPawns)
                {
                   score += RookOnPawn * popcount<Max15>(alignedPawns)*2;
                   //look for aligned pawns who are defended
                   Bitboard alignedPawnsDefended = alignedPawns & ei.attackedBy[Them][ALL_PIECES];
                   //if exists and the rook have low mobility, apply a penalty
                   if (!!alignedPawnsDefended && (mob <= 2))
                       score -= RookOnPawn * popcount<Max15>(alignedPawns);
                }

Can I use the * 2 ?
I am still doing tests. In fact is my last try about this.


感谢搬运！


@jmayer417 Nice styling of the bullets.


Agree, you have 0 -> INFO in severity_map too


削除する前にデータベースを閉じておかないとクラッシュするかも。。。


hehe


こっちも既存のデータベースの後始末をしておかないとクラッシュするかも。。。


@dblock Thanks, will do!

@sunnyrjuneja We are not using this internal API directly, but this change of json error format propagates to the API responses - we handle API errors in a middleware where we rescue `Grape::Exceptions::ValidationErrors`, which has now caused issues as our app was on grape `v0.10.1`.


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9-NoThreads: OK (394 of 397 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-99.24%25-yellow.svg)


Dont you want to use DecodeError here ?


interestingly the build is considered as desktop, not phone_app. Any other suggestion?


This import is not needed


<3 fabs


Unused variable ?


sql?


This function was originally implemented to replace this block of code: https://github.com/benckx/CorsixTH/commit/5b490bff82fc6f79eb5ee96cef2e4f0fdf0f491e#diff-cf76c63aceecdf2aff20f717809b74daL1549

In retrospect I think the code would have been better left as is, and this logic should not have been used elsewhere. Thoughts?


Probably.but according to LP commit ,both text and icon are called here .


Would be easier to read if you did
```
if len(output) != 1:
    self.err("some interesting error message !")
    return []
# Now give the results ...
```


Ya true, it looks nice ^_^


Unnecessary bracket for starting_value


Put brackets instead of backslash (I think its there in our styleguide ?)


Unnecessary bracket - ending_value


Ya, it's faster. I do like the idea of keeping the original URLs however. Would be nice if a precompiled binary was available officially on the interwebs that was compatible with Heroku.


Smaller variable names like `start, end` may help you in making the rest of the code neater


Ok cool


print needs to be removed


This code tunes the basic model to the 4 data points for Lampsar II (no prawns and no agrochemicals) without including snail seasonality. The transmission parameter 'beta' was fit based on the first two time points- Baseline and Follow Up 1.


One script to rule them all


Auto generated from IDE. Sorry, I'll modify that...


2129_radiance_server_perf (rpg777) - Win64-Windows-7-VisualStudio-12: Build Failed

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


@SimonSapin This one has a bug. Don't know where yet, but we end up with this: ```http://user:pass@@example.org/```

Testcase:
```
#[test]
fn test_setpass() {
    let mut url = Url::parse("http://example.org").unwrap();
    url.set_username("user");
    url.set_password(Some("pass"));
    assert_eq!(url.as_str(), "http://user:pass@example.org/");
}
```


make "params" optional


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-5: OK (398 of 398 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)


@jeffrymorris

Seems like we could simplify this a bit by making it a read only property with `get { return _beginChangeTrackingCount > 0; }`


If this is true, we need better tests


Weird. Are you using Visual Studio to build? What build configuration?


@slitazcn 我刚反应过来为啥我之前觉得你说的那个不对了。我之前试了一下这个功能来着
这个功能的实际含义应该是“在标题栏只显示文件名”（Show only filename in title bar）
而不是“只在标题栏显示文件名”（filename is only shown in title bar）
这么修改的话会造成误解的。


Aha, this is less annoying.


this is not working yet. very strange


- Changed `prosilver` `thankslist_body.html` by rearranging the columns so the **Been Thanked** column is first after the username column.
-  plus removed the following columns: Rank, Info, Joined and Last Active.
-  Also modified `prosilver` `navbar_header_quick_links_after.html` so when users click on the **Thanks List** link, the list will be sorted by the username that has the most **Been Thanked** value.
-  Also modified `prosilver` `thanks.css` file to include the new width fix for all the columns for the **Thanks List** page."


What is `WINAPI_FAMILY` defined to? `WINAPI_FAMILY_DESKTOP_APP` is `100`. If `WINAPI_FAMILY` is not defined, `<winapifamily.h>` seems to default it to `WINAPI_FAMILY_DESKTOP_APP`. Perhaps VS isn't defining `WINAPI_FAMILY` properly.


The thing about serializers is that you have to define every attributes. So, if you do it like this, the participation will only show `id`. So you can do `ap Participation` in the rails console, and just copy and paste the attributes.


I think something went a little goofy on your merge. I don't see the .eslintrc file in this commit, and it's part of your pull request...


The table layout is broken on GitHub.


On 2016/03/08 5:33, Benoit Daloze wrote:
> I'll do a Pull Request on the next import so there is a chance to review before merge (I am sorry if this broke some builders).

Oops, sorry, I've just pushed argf/read_nonblock_spec.rb before reading your reply.


It's probably more RESTful to add this in `ParticipationsController#index` call. And then add `resources :participations` in `config/routes.rb`


You can float either `left`, `right` or `none`. You can however clear `left`, `right` or `both`. :hammer:


not goofy, just git-fu.


I would assume `SIZE/900000` is probably making it run 900000x faster as it's skipping 900000x more values.


Good job on your assignment, just make note of the correction.


xccvcv


update_travis_toolchain (lefticus) - 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-warnings-yellow.svg)


视图控制器的生命周期。
initWithNibName:(NSString *)nibNameOrNil bundle:(NSBundle *)nibBundleOrNil （视图控制器的初始化）
loadView （加载视图）
viewDidLoad（视图已经加载）
viewWillAppear（视图将要显示）
viewDidAppear（视图已经显示）
viewWillDisappear（视图将要消失）
viewDidDisappear（视图已经消失）
dealloc（视图控制器的销毁）


why is ?= used here secondly this should not be appended, it should just be in real recipes since we are building from git hashes even for the stable recipes.


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

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


this is what I meant by doing git correctly.  I didn't ever touch the .eslintrc, but if I had done git your way, I would have a commit history ref for that file associated with me.

However, I rebased before merging, so my commits were run as if I forked from the current master, instead of the old master, so there *never was* a conflict for the .eslintrc.


Minor typo `Water`


this patch should be placed after the additing of 4.1 kernel recipe patch to maintain buildability


To fix wicket-1.5.2.jar not found error

```
Downloading: http://svn.codehaus.org/grails/trunk/grails-plugins/grails-wicket/tags/RELEASE_1_5_2/grails-wicket-1.5.2.jar.sha1 ...
Download complete.
:: problems summary ::
:::: WARNINGS
		[FAILED     ] org.apache.wicket#wicket;1.5.2!wicket.jar: invalid sha1: expected=<!doctype computed=03b8a546d50e3bdd203f8126753f4b5293205c8d (1512ms)
		[FAILED     ] org.apache.wicket#wicket;1.5.2!wicket.jar: invalid sha1: expected=<!doctype computed=03b8a546d50e3bdd203f8126753f4b5293205c8d (1439ms)
		[FAILED     ] org.apache.wicket#wicket;1.5.2!wicket.jar:  (0ms)
```

What helped me solve the issue was getting a wicket-1.5.14.jar not found error after changing the version in BuildConfig.groovy from wicket-1.5.2. A little digging in the release notes of wicket 1.5.14 led to the root cause documented below

```
Download complete.
:: problems summary ::
:::: WARNINGS
		[FAILED     ] org.apache.wicket#wicket;1.5.14!wicket.jar: invalid sha1: expected=<!doctype computed=03b8a546d50e3bdd203f8126753f4b5293205c8d (6836ms)
		[FAILED     ] org.apache.wicket#wicket;1.5.14!wicket.jar: invalid sha1: expected=<!doctype computed=03b8a546d50e3bdd203f8126753f4b5293205c8d (4067ms)
		[FAILED     ] org.apache.wicket#wicket;1.5.14!wicket.jar:  (0ms)
```

ROOT CAUSE:
The reason is most likely that the "wicket" jar has now been moved under "wicket-core"
See line 1315 of the attached release notes from wicket website :
   * [WICKET-3261] - Create wicket-core Maven module that contains wicket + wicket-util + wicket-request
 	http://archive.apache.org/dist/wicket/1.5.14/CHANGELOG-1.5

FIX:
 just changing
  `compile('org.apache.wicket:wicket:1.5.2') {`
TO
` compile('org.apache.wicket:wicket-core:1.5.2') {`

in **BuildConfig.groovy** fixes the issue.


Hello-

Been trying to get this to work with SealsWithClubs site.

Downloaded and installed fine, detected open tables and was able to gather stats on a couple of players but I could not READ the stats.

Also, once I opened the "Mucked" window I could not figure out how to close it and it covered the majority of the table....

Realized I needed to make the color maps in order to be able to use it on 2, 6 and 9 player tables so downloaded/installed latest NET framework.  Now, Pockermuck crashes as soon as opened.  I uninstalled it, downloaded a fresh copy and reinstalled-same result.

Also, I've been unable to register on your site-seems to be the same link for login and register.  When I attempt to input a username and password and hit create account, I get this:

 Login error
There is no user by the name "Senility". Check your spelling.

I've read every page of your site and probably have missed some simple thing.

I know just enough about computers to know I know nothing.  I realize you're not gonna offer unlimited free support for a free/shareware product, but any pointers you might have to offer I would appreciate greatly!

Thank you,

Dean
iceeyuranus@gmail.com


I will add two other projects :
guice-common
memory-guice


I believe they're taken from the LLVM documentation and that project's conventions. I'd suggest an update: with GitHub's big green merge button, new contributors can easily determine their commit access--no need to muddle history.


I'm sorry I didn't get to this today. I'll take a look tomorrow. I haven't
used stack and nix together yet.
On Mar 7, 2016 3:37 AM, "Ben Ford" <notifications@github.com> wrote:

> Initial nix/stack set up
>
> Not sure what the best way to use stack and nix-shell together is yet.
> @k0001?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cdodev/scintilla/commit/da2001ea2240ea49b3798fb7c1a420349a77d22c
>


Ok
I will use Onami for this, just like MPT tests.
By the way, I find that inheritance is usefull when working with annotations...


I do wish we had a more systematic way of testing when dispatch is going to specialized routines like it should.


Is this a mistake here?  should y be int or double?  Is the int(...) used to round the value?


Just a quick edit


There was no example provided, so the sentence was deleted.


rdcast<int> might help check errors in debug mode.


prototype.captured_images.mainみたいな記述できませんか？


これだと、subの一枚目の写真しかとることができませんよね？


delete


please add Apache License version 2 header


Apache License version 2 header


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - 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)


Another alternative would be pushing to a temporary branch which could be immediately deleted afterwards.


複数の写真をもってこれるようにするために、引数とか使えば解決しそうですね


@matejovo, you shouldn't need the `<choose/>` here. Just using `<text variable="DOI" prefix="doi: "/>` will give the exact same output.


Please check the code Silpa-Flask on development branch to findout what URL is being registered.


sounds good, i will figure out how to change that

On Mon, Mar 7, 2016 at 7:44 PM, Khem Raj <notifications@github.com> wrote:

> this patch should be placed after the additing of 4.1 kernel recipe patch
> to maintain buildability
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dwesterg/meta-altera-1/commit/de8affa660092cb5eda830463b8cafe0e6d1b8ec#commitcomment-16552902>
> .
>


Perhaps add an overload of getAtomCoords(const Point2D &) ?


please change until.h    and until.CPP the loop to while (true) and also in main.CPP.


The blockquote and cite tags would also work here, but p is definitely a big improvement over just a span.


@hlateman There is not a clear center, you can clear: `left`, right` or `both`


> but if I had done git your way, I would have a commit history ref for that file associated with me.

No, the way you did it, we now have two commits with the same code. That's not quite right, is it? :suspect:


If the company had a rebrand would you run into an issue with this class? Is there a better name with that in mind?


Hey,

Did you find something to fix BT when using these flags?

It isn't working even when you force -Os


This is much improved. The layout of everything could be finessed further but this is a huge improvement over the last commit. Nice work!


good use of this tag


@hlateman great job on the assignment, please note corrections.


Where did that come from? I just pushed a commit that took out some unnecessary imports.


google font


jjjj


Yeah I changed a lot up because I was having an issue with sending two
requests at times. I thought this would fix it but it's still occurring and
I can't quite figure out why.
On Mon, Mar 7, 2016 at 5:06 PM Andrew Conner <notifications@github.com>
wrote:

> Just noticed this. For your specific use case, you're fine, because you
> don't change any data. But in general, if you do a setTimeout and change
> data, Angular will completely miss that you changed anything.
>
> Instead, use $timeout (you can inject it). It's basically the same, but
> runs the $watch cycle, so any data changed will get picked up.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vcooley/fcc-nightlife-app/commit/96b9b218c2f3611bca346030fc0cee66ceb92dac#commitcomment-16550226>
> .
>


Is there any easier method?


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


Seems a strange place to put this


Let me check thought I pushed to master as well. I may have pushed it to my
fork.

On Mon, Mar 7, 2016 at 7:09 PM, Thomas Weise <notifications@github.com>
wrote:

> @PramodSSImmaneni <https://github.com/PramodSSImmaneni> @vrozov
> <https://github.com/vrozov> why is the change not in master?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/incubator-apex-core/commit/5a8ce1a51d86ac66ae9dd9cecc5ff02f6ede6370#commitcomment-16552528>
> .
>


why do we mutate the cost attribute of the concat-ed model here?


Mine might be more compact


It is indeed fairly obvious to figure out if you really have access by the github's green merge button.
I imagined this was some sort of tradition rather than an actual check for commit access.


develop (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (2139 of 2397 tests passed, 0 test warnings)

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


Don't have this resource.


Same issue here.


@IndieBeto I haven't. I didn't get a chance to try applying -Os even. If that's the case I may revert this because I don't see a big gain in performance compared to the Arter's -O3


I'd prefer to use JsonDict, but didn't find a way to get Jackson to deserialize to that instead of LinkedHashMap. Any ideas?


gray로 수정된 파일 확인했고 이대로 Merge 하도록 하겠습니다.


Could also use

```
std::copy(dashes.begin(), dashes.end(), dd);
```
now that there is no math manipulation.


Line 145... The only thing that teens think while playing this game.


Not requires (as its declared in if and else)


The triple backticks needs to be on a separate line


Triple backticks should be on separate line


I need a :beer:. This isn't exactly a problem we need to solve in order to move this PR forward, so let's talk about it elsewhere.

I've now spent most of the time I planned on getting stuff done tonight bickering about differences of opinion.

I designed all of my work so that it would be easy to introduce the opinionated tools/pieces in later commits, so can we please flesh out those opinions in issues... instead of this PR?

THANKS.


why different implementation than dataflow's?


就我來看這邊是 user 不存在吧


Volatile loading of the bitstream + netboot/serialboot of the runtime may be better (faster, does not wear out the flash, known-good configuration can stay in the flash). We can also build the BIOS inside the bitstream, or leave it in the flash since it should not change a lot. But we can keep that for later.


Yep.
I would go for merge this change and getting then #1840 done, then we can do this much simpler:
```
highlighted = highlight_whitespaces(line)
highlighted = highlighted[start_of_line_to_print:]
printer.print(highlighted)
```
imo much better, much more generic, way easier to use.


Any reason for canonical vs absolute?


status 用 message 比較合適


+预期实现功能
1.注册与登录
2.新建Markdown文档
3.在线编辑Markdown文档（需要嵌入一个编辑器）
4.能够新建章节，新建的章节作为一个单独的markdown文件存储
5.能够分享章节（点分享，将章节文件生成一个URl发送给好友）


This is why we have PRs, man. If you want to work as a team, everyone on the team needs to be happy with every line of code.


I think the message should be `"There is an issue with" + self.__class__.__name__ + ". Please file..."`
This seems more generic (Although not important)


Got your idea.
Thanks. :smile:


please Add Apache License version 2


What if this throws exception?


We don't use the `[bearname]` automatically in front of logs from the bear ? I thought we do.


copai de index


for reader.py can you update to use import nutch - it exposes the SequenceReader API using the REST server. @jxihong


Also ... this is a bad log message. If people get this, we'd get comments in our repos saying "coala told me to file a bug here for eslint" with no aditional information.

I think its better to just do `assert len(output) == 1` because that will give a traceback and the default "We have a problem" error message. The user will use `-L DEBUG` and that would give us good debug information


Perhaps afterwards

```
if len(legends) < len(mols):
   raise ValueError("legends have too few entries")
```


Nice recursion! better than my method


icon


I wasn't happy with your PR, but i voted it up, because it got the stuff we needed out there, there's nothing actually *wrong* with it, and because I could open up an issue with any specifics on opinions I had about your approach in an issue.

It goes back to what you said about code iterations, and that I tend to look at things too big at first.  I'm taking the same approach with comments and issues with PR's.


This seems to be the first commit to break compatibility with PHP 5.4. A bit sad and confusing. Sad because it seems to be introducing short array syntax just for the fun of it, without consideration for new dependencies. Confusing because the changelog of doctrine/migrations only reports that version 1.3.0 is removing support for PHP 5.4, when in fact it's already removing it (here) since 1.1.0.
Anyway... if you're trying to run something actually using this code from a Travis-ci machine having only PHP 5.4 on the command line, know that thanks to this commit you have to go way back to version 1.0.0 of doctrine/migrations to be able to use it.


Hi


Why does it include irrelevant lines, like App Engine UrlFetch, Rx, OkHttp, Retrofit? Those are either handled in other proguard configs here (like OkHttp), or just irrelevant (like UrlFetch, which is server-side only).


Wow.

Hey, so a lot of these are not needed.
autoconf, automake, build-essentials, git, openjdk, unzip and _definitely_ already installed.
Im fairly sure the others are also installed.

Also, can you add these to the variable `$deps` for uniformity in installation with apt-get ?


wo shi wo de


Well, let me know if it works for you.

Also Temasek just removed Arter97 commit a month ago, I really wonder why, that is the main reason I am picking these optimizations again.


Why is there a random j here...?


Looks awesome now :)


This looks much cleaner too :)


@gibrahim  Great job on your assignment, you could make the header a little shorter, but good work. :hammer:


This parses the csv file and sorts the data format.
![parsecapture](https://cloud.githubusercontent.com/assets/5950562/13591894/25540554-e4bb-11e5-901b-0d74cc3df5f7.JPG)


Oh and by the way, I've tried to force -O1 and literally every other flag that could work.


What j?

Clayton Lawrence
Computer Science Undergraduate
Georgia Institute of Technology
ca.lawrence03@gmail.com
(770) 733-0336
On Mar 7, 2016 11:18 PM, "Tejasvi (Teju) Nareddy" <notifications@github.com>
wrote:

> Why is there a random j here...?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/git-mad/Bitter/commit/08ae64f2aee02beb9819a1b7821140e63e667bab#commitcomment-16553173>
> .
>


unable to move eye elements to overlap over faces; arcs for smiles not properly formatted either;
check more documentation...


Line 29 of AndroidManifest.xml


Is infer available in a deb package ?
It says they have binary versions for linux - https://github.com/facebook/infer/blob/master/INSTALL.md

Can we use that ? They are much faster to isntall


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-MacOS-10.9-clang: OK (2115 of 2129 tests passed, 1 test warnings)

 * 15 tests had: EIO diffs.
 * 7 tests had: ERR diffs.
 * 14 tests had: ESO big diffs.
 * 2 tests had: Table small diffs.
 * 11 tests had: Table big diffs.
 * 10 tests had: MTR big diffs.
 * 2 tests had: MTR small diffs.
 * 1 test had: ESO small diffs.

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


Correction... completed Explorer Mode in the retirement calc assignment.


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1021 of 1021 tests passed, 0 test warnings)

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


icon


add buy ticket


add ticket icon


add button


Dataflow doesn't upload any JDK jars or the current dir. I guess their class loader doesn't include those.

`This does not recurse to class loader parents stopping it from pulling in resources from the system class loader.` from their JavaDoc.


Just tried this on another server but still defaults to 500px.


Canonical handles differences between `/path/to/dir/.` vs `/path/to/dir`. Exception should be propagated by default so no need to catch and re-throw?


sorry, wrong image type


Yep. Also ran into this. The problem is if you change the prompt to `${debian_chroot:+($debian_chroot)}\u@\h:\w\$ ` (with a space), then the prompt becomes this.

```
root@ip-1-2-3-4$\ ls
```
Yup. The space gets escaped...

If we _don't_ set this variable, then the prompt is this.

```
bash-4.3# ls
```
No escaped slash, but now the normal prompt is gone.

I'd be happy to fix this if anyone else has any ideas! :smile:


never mind, I think I get it


@wmarques I dont think we need to add `templates.js` here as its generated only for prod and would be concat and minified into app.js. we should find a way to tell the concat task to include this file as well during the process


There was a conflict with cherry-pick on 3.3 and master because this change
was already on both those branches, hence the change to bring it to parity
with what was there before.

On Mon, Mar 7, 2016 at 7:07 PM, Thomas Weise <notifications@github.com>
wrote:

> @PramodSSImmaneni <https://github.com/PramodSSImmaneni> these type of
> changes should not be made in release branches.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/incubator-apex-core/commit/6875811115e58550f789267d97f04543682e9e26#commitcomment-16552492>
> .
>


I think a better solution would be to write `PS1` in `/etc/profile`—except I tried that too and it gets overwritten by us at some point. This needs more investigating, but I thought having _something_ out would be better than nothing.


the usage/scope of that variable is very limited, it's hard to go wrong so I prefer to leave `const` out.

... and it will break the beauty of simplicity... just joking.


Yup

On Mon, Mar 7, 2016 at 3:04 PM, brine <notifications@github.com> wrote:

> is this ready to be released?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GeneralCake/FoWTCGOCTGN/commit/f8ffae25bc441f029950ebb2409b2e512fee5625#commitcomment-16549217>
> .
>


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2153 of 2167 tests passed, 1 test warnings)

 * 15 tests had: EIO diffs.
 * 7 tests had: ERR diffs.
 * 14 tests had: ESO big diffs.
 * 2 tests had: Table small diffs.
 * 11 tests had: Table big diffs.
 * 10 tests had: MTR big diffs.
 * 2 tests had: MTR small diffs.
 * 1 test had: ESO small diffs.

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


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6: OK (398 of 398 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)


@ashleylevs put these in a `<p></p>` tag :+1:


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


ありがとうございます！こんな感じでしょうか https://github.com/bm-sms/daimon-news-multi-tenant/commit/8ed84df3712f735c8e9f0261343872b3979a273d


contrib/codeanalysis/runcppcheck.sh: line 9: ../cppcheck-1.66/cppcheck: No such file or directory


What about locking the gem's development dependencies, such as mocha?


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


Oh, I see. You moved these into the gemspec.


確かにInitializerを使わないことにしたので不要でした…。ありがとうございます。


arm64 breaks build with opti flags!


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-LibC++: OK (390 of 390 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)


They wanted whatever the game mechanics was; little did they know PS was wrong.


i gave you all the access rights to publish updates, right?


For sync PouchDB sends requests to the CouchDB root which we need to handle

<img width="455" alt="screen shot 2016-03-07 at 23 36 39" src="https://cloud.githubusercontent.com/assets/39992/13592185/d46de0c6-e4bd-11e5-8cda-8adf2089cc69.png">

@christophwitzko ^^


I think at some point you did but I don't remember or have the info to do
so.

On Mon, Mar 7, 2016 at 8:38 PM, brine <notifications@github.com> wrote:

> i gave you all the access rights to publish updates, right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GeneralCake/FoWTCGOCTGN/commit/f8ffae25bc441f029950ebb2409b2e512fee5625#commitcomment-16553355>
> .
>


The change was made as a separate commit and hence I don't see what it would help with. Please don't make code formatting changes in release branches.


Super weird issue. Going to dive into it now: https://github.com/JeremyEnglert/JointsWP/issues/179


I'm not keen on network and networking structs. Not sure if that mirrors oci though


okay


ok


@JamesTFarrington, so it looks like this is the method that used to set the Runner back to a ready state:


ng


Hi Lanninster,
i sent
Transaction : 9712be14e37f28307874d8507d68e648ddd776c1f8ed12de779e87119aa9895e

Block	401,570 (76 confs):
 The above transaction done to the adress: 3Qnj4QtdD4qtZcP82xyLq2paAEPDgfwezd

And the elastic coins are not appearing git hub repository #


Added mime_types as instance variable since it's needed for resource.


Maybe a note in to comment that this is only an approximation for Windows server containers. Minor nit, not important


@ashleylevs This only need to be declared once in the body declaration, it will inherit down the stylesheet.


Changed "url" to "uri".


Remove cgroups comment


My Phone is ready for flash with 100  battery , @akhilnarang @westcripp @varund7726 @bdogg718k

![shinchan_zpsf7abf8f3](https://cloud.githubusercontent.com/assets/10176463/13592248/5b25c43c-e516-11e5-8558-2644fbacd7ac.gif)


Add comment inline with other fields in other structs


Yep. I just canceled the test.

I will try one more idea but looks invincible this thing.


+1 to ^^


Added mimetypes parameter to be used for resource when worker.get_http_response is called.


@agarwal-sandeep , what is this change about?


Try this: in scala console, type `"foo".s"` then press tab. After a while you should see the completions containing `split`, `startsWith`, `subSequence` and `substring`.


Not sure how user is modelled in HCS Apis for TP5. User and domain?


size is not a CSS property. didi you mean `font-size` ? :hammer:


Makokha David Werunga, I used Wanjohi Sammy laptop.


update_travis_toolchain (lefticus) - i386-Windows-10-VisualStudio-14: OK (390 of 390 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)


You mean `tDllAsndServiceId` right?


Yeah this shouldn't be here. We should handle escaping outside of the spec and add comment that the args here are expected to be escaped along with an msdn link


readme.MD


My initial thoughts were that the paper had a fairly interesting topic.  You did a good job of establishing the importance of developing more effective batteries to reduce energy costs associated with greener forms of energy production.  The theme was fairly clear, you are comparing the new metal batteries to more traditional methods of energy storage.
I felt that there were some formatting changes you could make to make the paper more readable, including centering and bolding the section headers.  I found myself looking back through the paper to effectively compare the advantages and disadvantages of the batteries because you organized the paper by battery type instead of allowing for a direct comparison at each point.  I think I would find it easier to look at the advantages and disadvantages of each battery type  if they were immediately following their counterparts analysis.
My extraneous thoughts about clarity incude:
-I would entirely revise and or cut the sentence "Magnesium and antimony because of their low melting points, low cost..."
-I'm not entirely sure who the intended audience is, and while we have covered this exact material in chem recently, it might help to explain some of your chem based terminology more.
-When using figures with more than three 0s, I would consider using parenthesis to make the numbers immediately readable
-The pictures could be more data driven, Im not sure many of them furthered my understanding of the material at hand.
-There are some other small grammar edits and revisions that I would be happy to talk to you about, but I think the paper will undergo enough revisions that it wouldn't be helpful at this stage


Don't think console size should be part of process - doesn't feel like the right place.


We should get away from tty as a term as its not really a Windows thing.


Yes agree with John


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3048 of 3048 tests passed, 0 test warnings)

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


i do believe the 4.1 ltsi kernel is added prior to removal of the 3.10 ltsi
kernel

On Mon, Mar 7, 2016 at 7:55 PM, Dalon Westergreen <dwesterg@gmail.com>
wrote:

> sounds good, i will figure out how to change that
>
> On Mon, Mar 7, 2016 at 7:44 PM, Khem Raj <notifications@github.com> wrote:
>
>> this patch should be placed after the additing of 4.1 kernel recipe patch
>> to maintain buildability
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/dwesterg/meta-altera-1/commit/de8affa660092cb5eda830463b8cafe0e6d1b8ec#commitcomment-16552902>
>> .
>>
>
>


Should this be moved to resources?


@ashleylevs  Good job so far, please note the corrections.


@mostaphaRoudsari Thank you for keeping in the loop. Is the current gh file on repository good to go ?


Lgtm


@khoahp Khoa nên remove commit này đi; khi đó Pull Request #7 sẽ chỉ còn 2 commit, không bị conflict phần license information nữa.
Xem hướng dẫn tại: http://sethrobertson.github.io/GitFixUm/fixup.html
(lần sau tốt nhất nên xử lý cái này trước, sync lại với upstream rồi mới code tiếp)


這邊是 user 是要是 po 文的人 才能刪除


Wonder if we should diverge and use a more generic term like 'storage'? @jstarks.


> They wanted whatever the game mechanics was; little did they know PS was wrong.

The game mechanics say that Shedinja has 1 base HP. If you change its base HP, you're not using the game mechanics anymore. "Whatever the game mechanics were" no longer applies if you're changing game mechanics. Why is this hard to understand?


I think this is where system drive size should b, omit empty


Chính xác là phần này: http://sethrobertson.github.io/GitFixUm/fixup.html#remove_deep
Tra thêm HDSD git cho chắc.
Có thể tạo 1 branch mới làm thử, nếu thấy không ổn thì chỉ việc xóa branch đó đi.


this is a test


OK


Like, seriously, you linked to a post _with instructions for changing Shedinja's base HP_ as your evidence for why Shedinja's base HP can't be changed. It's a game. It's made of code. Code can be changed.


@michpark00  can you link the css, I believe the path is incorrect. :hammer:


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#Z7ILGGb1SViopOb2PMl7OA


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (390 of 390 tests passed, 0 test warnings)

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


I think the spec ahould be forward thinking and allow for multiple ips, v4 and v6, multiple dns servers, gateways and so on. Even if not used. We don't really want to have to change the spec in the future if at all possible. Sandeep, suggestions?


Forwarding the frame and parsing in nmtk will produce a looped event post (nmtk post event for nmtk) which we want to avoid.


thanks


not needed


Its just a tuple earlier - which is fine i guess ?


One for Sandeep. What's the right name?


unnecessary bracket ?


@boennemann @christophwitzko can you recall what the "path" option was used for and were we use it? It didn’t make much sense to me and I couldn’t find a place were we used it, so I removed it. Did we maybe need that for testing `my-first-hoodie` within `hoodie-server` in the past?


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


Great work of documenting all the parameter!


why not, the tuples need to be converted to a list don't they ?


I am wondering which module will handle events send by wapi_registration:publish_sync/1 ? I grep 'reg_sync' in source code of kazoo and kamailio but nothing is found.


This is new style for reference docs.  Just curious about the advantages of this style vs just a standard block quote.  The reason I ask is that @joelfranusic-okta will need to make a template to render all annotations with our new doc approach to generating docs.  I am cool to add new styles, just want to make sure we have a consistent approach to make the templates easier to manage.


How can we make sure the inexperienced user is not confusing this with the sn1a_table variable?


Purely with no justification, I kind of prefer to use 'max'. But this might be just a straight mirror from Linux spec in which case is fine.


`this.template.baseStats['hp'] === 1` and `this.speciesid === 'shedinja'` result in the same thing in standard play. The difference is that the first breaks Averagemons and God knows how many other current/future OMs. The latter does not break anything. That's all I have to say. If you want to revert this change then I can't stop you; but I will have to hardcore it in every mod (currently only Averagemons) that comes along. What happens when I forget to mod it on day? Well players will be using something incorrect. That's on you.


Cool! This is very useful. :+1:

How would people feel about adding an option to kill the current tmux session if no name is supplied (and one exists)? I can't recall if the current session name is (readily) available through Tmuxinator, but if it isn't we could wrap-up `tmux display-message -p '#S'`.

I have a Bash alias that does this and I use it constantly, so yes, this proposal is a bit self-serving. :P


Thanks for your corrections Benoit.


No underscore


And add a comment about the valid range


Why int64 if 0..100?


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 131](https://teamcity.labkey.org/viewLog.html?buildId=308737&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:37


Typo: Lenght instead of Length


//интернетваш нинужон!


Can this compile without -DDEBUG switch?  I want to be able to turn it on/off without having to comment out debug statements.


Is it important that secret is different to secret_key?


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2366 of 2397 tests passed, 0 test warnings)

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


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.7: OK (398 of 398 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)


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-14: OK (390 of 390 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)


lets import model class explicitly. I'd import like `from cd.models import (Concept, ConceptGraph)`


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

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


I am open to suggestions as to how to better handle this. I tried using LockedRect, but that results in **worse** performance than ```dxva2-copy``` on my Intel system for some reason.


The `processReceivedAsnd()` in dllkframe.c at line:2710 checks for a valid registered filter before forwarding the ASnd frame.

So we would have to handle the registration somewhere atleast. Please let me know your suggestions.


Lets move all form `ModelForm` class to a forms.py


Those were very relevant comments :)
Thanks.


okay


引数の順番とその意味の対応を理解するのが難しかったです。
これらの引数は post の属性になるので、Hash で渡すほうが意味がわかりやすかったりしませんかねぃ


My suggestion is we adopt new terms for all API docs to make these use cases more clear.
- **Find** Users (`?q=bob`)
    - Doesn't support paging, intended for pickers, matches against pre-defined attributes)
- List Users with **Filter** (`?filter=firstName eq "bob"`)
    - Filters against live data, always up-to-date, but only supports limited fields
- List Users with **Search** (`?search=firstName eq "bob"`)
    - Searches against replicated data, may be out of date, supports most fields

I think this will help in the long-term if we can switch to something like this and avoid terms like `advanced search`.  If we go this approach we should update all API docs and Postman templates for existing resources to match this terminology.

@joelfranusic-okta @tthompson-okta @cwu-okta @larsjohansen-okta thoughts?


`find_link('current site')` ?


@rossy any ideas?


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.7-LibC++: OK (390 of 390 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)


Make this a bit more verbose and explanatory, something like:

```Hydration is disabled, watering can and hygrometer tools removed```


その通りだと思いました！


Khó hiểu nhỉ??


looks workable.  I wonder if adding a barcode to the label would be much trouble?


one empty line here


This is all good after these last fixups.


I felt bored and nobody else is maintaining this... I am not going to do regular maintenamce, though.-------- Wiadomość oryginalna --------Temat: Re: [CharsetMC] Charset 0.2.3 (bb4db19)Od: Fortist Do: asiekierka/CharsetMC DW: Adrian Siekierka Asie, I thought you quit working on this?

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


on page 23 in  notes


The softpot's "baseline" when nothing is selected is between these values. I think we must test when last was the softpot touched.

I will work on that!!


looks great!


>The game mechanics say that Shedinja has 1 base HP. If you change its base HP, you're not using the game mechanics anymore. "Whatever the game mechanics were" no longer applies if you're changing game mechanics. Why is this hard to understand?

Seriously? -_- We're not changing every game mechanic ever, only a specific one. The rest still apply.

There's literally no point arguing with you because your arguments are dumb.


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1579 of 1579 tests passed, 0 test warnings)

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


I like this proposal. Should we use language to steer users towards search as it is more powerful (has more fields)? And then only use filter in the case where they desire live data? We should also highlight that replicated data is not that old. Once we have actual metrics we can incorporate into the doc; currently we only have an educated guess that it is delayed by a few seconds.


Is dit niet {{ user.email }}?


I think we have attempted to use the term `property` when possible which is consistent with JSON terminology and avoid fields or attributes for API params and in the world of UD we use the term Profile Attributes.  Not saying its always worked out :)   I see the terms fields and attributes used interchangeable in this section.  Do you have a suggestion on what is best going forward?

 For APIs its properties.  I'm sure there is still some cleanup.


> The latter does not break anything.

The latter breaks mods that give Shedinja more HP, and mods that want to give other Pokémon (maybe CAPs) one HP.

> What happens when I forget to mod it on day? Well players will be using something incorrect.

I still don't understand how you're defining "incorrect" here. If you change game mechanics, game mechanics will change. There's no such thing as incorrect.

If Game Freak decides to change Shedinja's base HP, do you think they'll also remove its hardcoded HP as 1? If Game Freak adds another Pokémon with 1 base HP, do you think they'll also add it to the hardcoded list? I would guess "yes" on both counts, but since they haven't done so, neither behavior can be said to be any more correct or incorrect.

I'm sure there are things we hardcode and current gens don't, and plenty more things current gens hardcode that we don't. We don't hardcode them because it makes the code more readable and modding easier. For instance, the game probably hardcodes statuses, while we have a statuses.js that makes it easy for anyone to add their own status.


What is the request?

Terry

Sent from my iPhone

> On Mar 8, 2016, at 12:10 AM, Tony McCormick <notifications@github.com> wrote:
>
> looks workable. I wonder if adding a barcode to the label would be much trouble?
>
> —
> Reply to this email directly or view it on GitHub.
>


Moet dit een loop zijn? We hebben maar 1 rol


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (398 of 398 tests passed, 0 test warnings)

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


Hi,
Would it make more sense to just install fpdf package in the library directory and use that(appears that these pdf label packages were made to work with fpdf)?
-brady


16 tests:

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

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

  reducer
    √ handles SET_STATE
    √ handles SET_STATE with plain JS payload
    √ handles SET_STATE without initial state
    √ handles VOTE by setting hasVoted
    √ does not set hasVoted for VOTE on invalid entry
    √ removes hasVoted on SET_STATE if pair changes

  16 passing (161ms)


html5になってだいたいのタグがblock要素になったはずだけどpタグはまだ厳しいんだね。
あとpタグの制限に関わらずなんだけど、pタグって文章の段落を表すタグだからあまりここで使うのはふさわしく無いね。

ちなみにこのpタグって何に変更するのが相応しいと思う？
あと背景色の指定ってulタグに書くのじゃダメだった？


yeah, I would make the changes


gotcha :+1:


今回の場合、色指定を文字列で指定してもいいだけどあまり一般的な方法ではないので基本数値指定するようにしてください。


I'm calling it good enough!


Yes! This key needs to be different than what we have in test or development ENV. Ideally all the production variables should not be in code and they shouldn't be visible to anyone except the people who have access to the deployed server. Rails community mostly uses [dotenv-rails](https://rubygems.org/gems/dotenv-rails) gem to provide ENV variables to applications... And file (.env) containing ENV variables is not version controlled. Every developer and deployment must have their own .env file.


ここでクラス指定とid指定を組み合わせているのって動作確認目的ってことでOK？
idは一意な値だから他の条件と組み合わせる必要が無いので。


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-14-Debug: OK (390 of 390 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)


It can extend ```container_aware_migration``` to allow getting rid of ```global``` thing below :smile:


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9: OK (398 of 398 tests passed, 0 test warnings)

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


That's not what I asked. Why does "secret" need to be different to "secret_key". The only thing you did in this commit was the split the two. Why?


If the class inherits ```container_aware_migration```, you can use ```$this->container->get()``` to get tge cache service here instead of using `````global```.


ok, will let you know. going to give this another effort tomorrow to see if i can get some machines off the ground. but it's been a less than encouraging process. this again makes me want to somehow come up with a plan to run ansible full stack. maybe as like a nightly build or something


Hmm, didn't know about it, let's fix it https://trello.com/c/RuG63hvG/613-don-t-reuse-other-modules-global-variables


Also adds support for spawn eggs!


thanks!


good catch, I'm sorry...


I think is no longer true and we forced customers to limit=200 via FF.  @cwu-okta can conform


Is it really "hidden" or just "unnamed". Either way should be gettext'ed.
Also might be better to go just under the ESSID detection, just  above the quality image definition


:eyes:


needs a boolean variable in mPlaylist to indicate whether playlist is private or public.


yep


ok


The flag is rest in the edrvcycle.c.

In this implementation, the frames with `pBuffer_p->fTimeTrig = FALSE` will still need the launch time in the descriptor to send the frames as we don't have the second queue.

So, in case a buffer having a valid launch time is used to send another frame with `pBuffer_p->fTimeTrig = FALSE`, the old launch time would may be used instead of the current mac time.
To avoid this problem I reset the launch time after a successful send.


okay


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9-NoThreads: OK (394 of 397 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-99.24%25-yellow.svg)


Yep, I changed it couple of times. First I wanted to create a generic class, but then I switched to stdout/stderr-only as it didn't look useful for other purposes... Let me improve that in v2


Correct. You must be a grandfathered org in order to receive page sizes greater than 200 (https://oktawiki.atlassian.net/wiki/display/eng/Federation+Feature+Flags). Otherwise all other orgs are forced to a 200 limit. We should keep some language in there which indicates we can change this limit at any time (to cover the case when we will eventually convert grandfathered orgs over to using the 200 limit.


The interrupt handling needs to be configured for the Tx-Rx Queue pairs and required both Tx and Rx queue to be active.

I avoid handling of the 2nd Rx queue in the interrupt routine.


could be, previously we had multiple global-methods used in job/app/log/output, many were already removed, so let me remove this one too..


Thanks makes sense.


I see


We are getting `NoMethodError: undefined method 'secret_key' for nil:NilClass` error at the Jenkins. I thought that was the problem. But now I've realized it doesn't have the whole context file available to it, since its giving warning about other variables as well.


I see what you're doing there, but mere mortals must not sneak things into `std` namespace.


便利メソッド！assertするときは`assert find_link('current site')`とかでしょうか。
テキストの値が重要ならassert_equalがいいと思うのですが、指定したテキストを含む
リンクが存在するかどうかが重要なら`assert find_link`のほうがいいですかね…


You are trying to create android emulator with system image sys-img-armeabi-v7a-android-22. But as per this link (https://docs.travis-ci.com/user/languages/android#Pre-installed-components), the system image you are using is not available.


helloWorld (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed (2136 of 2395 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-89.19%25-red.svg)


Commenting on commit


No, this was a bug already present in code.


nothing specific, just seemed like we can print labels and barcodes from
that patient popup, so it would be good to print them from appts too?

Tony McCormick, CTO
www.mi-squared.com
Support: 866-735-0897, Direct: 713-574-6709
My Calendar:  http://bit.ly/XznvDo
“I am and always will be the optimist. The hoper of far-flung hopes and the
dreamer of improbable dreams.” — The Doctor, Season 6

On Mon, Mar 7, 2016 at 9:15 PM, Terry Hill <notifications@github.com> wrote:

> What is the request?
>
> Terry
>
> Sent from my iPhone
>
> > On Mar 8, 2016, at 12:10 AM, Tony McCormick <notifications@github.com>
> wrote:
> >
> > looks workable. I wonder if adding a barcode to the label would be much
> trouble?
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/teryhill/openemr/commit/650d0ba343a1ad631ab8166db1d21adeabe5f9a9#commitcomment-16553633>
> .
>

--
Please be aware that e-mail communication can be intercepted in
transmission or misdirected. Please consider communicating any sensitive
information by telephone.  The information contained in this message may be
privileged and confidential. If you are NOT the intended recipient, please
notify the sender immediately with a copy to hipaa-security@mrsb-ltd.com and
destroy this message.


Please include the word "toolset" and location info in the error message so users know where this is coming from. Similar to other toolset error messages: https://github.com/Microsoft/msbuild/blob/a4e0dd5f4b31a6c9acb1bab25ac401c438c3dfac/src/XMakeBuildEngine/Resources/Strings.resx#L609


( ͡° ͜ʖ ͡°)


この指定方法はあまり良くないです。

理由としては、col-md-*系統のクラスは配置を定義するクラスとして使われているので、それに対して見た目を定義してしまうとcss指定がカオスになるからです。
またcol-md-x系統は1から12まである上に、col-xs*- col-sm-* col-lg-*と言った類似要素もあるのでその辺カバーできてないのも微妙です。
bootstrapのみを使って課題を実現できるように修正をお願いします。


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-5: OK (398 of 398 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)


[kmc Q24.txt](https://github.com/rellarusty/kmc-assRep-1/files/162591/kmc.Q24.txt)


[kmc read me ass rep 1.txt](https://github.com/rellarusty/kmc-assRep-1/files/162593/kmc.read.me.ass.rep.1.txt)


This commit was brought into codebase.


It seems that ```check_auth``` never returns an ```empty/false/null``` values (just true/action name or error string instead), so the right part of this condition will never be satisfied.


and what seems to be the problem? It's correct, because this is an stderr output of `parse_known_args`. Take a look at the last line:

```
avocado: error: argument subcommand: invalid choice: 'passtest' (choose from 'exec-path', 'run', 'sysinfo', 'list', 'multiplex', 'plugins', 'config', 'distro')
```


I'll leave you all with this and quit this byzantine discussion:
https://en.wikipedia.org/wiki/Ship_of_Theseus


OK, v2...


cool


+1 the change looks good to me


So we should revert this commit?


is asterisk are required to make it work?

I would prefer to comment by indent also, like:

```sass
.foo
    //.bar
        color: red
    .baz
        color: navy
```

it's identical to this:

```css
.foo {
    /*
    .bar {
        color: red
    }
    */
    .baz {
        color: navy
    }
}
```

It works in that way in Pug (ex-Jade) and I very like this way to comment out whole trees.


I missed it as well :) Its fine to just fix the readme and push to master


This is just for the animated part, though, right? How do you reproduce the two at once? Two searches in quick succession? If I can reproduce it, I'll help figure out what's breaking.


I think would be good to add here a comment to make it clear:

```js
// .pipe(postcss([sorting], { parser: sugarss }))
.pipe(postcss([sorting], { syntax: sugarss }))
```


I think this type of comments is to tricky. But if I will see big support in issue, I will think about implementing it.


可以自定义提交事件的操作


This is not working for me. `id` is always `null`. Have you tried it out? With renameing and creating new Repo/User/Team?


@mholt Maybe we could implement Markdown's own `Include` method?


Sure, will do that when i fix cuda and opencl examples, though i changed code they don't seem to work yet based on what you guys tested.


Sure, now everyone can set the instance of the api.


why do we remove this?


Again, if you say “instead of” I think it's good to show “instead of what exactly”:

```js
// postcss().process(css, { stringifier: sugarss }).then(function (result) {
postcss().process(css, { parser: sugarss }).then(function (result) {
```


So much nicer!


update_travis_toolchain (lefticus) - 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-warnings-yellow.svg)


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

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


git merge conflict here and below


@shehzan10 I don't have a way to test CUDA, OpenCL right now because of the driver issues on my laptop. I didn't want to change this example too(like histgoram) without exactly fixing it.


そうですね。
当初class, idいっぺんに使ってしまいたかったから…程度の意図でした


Revised R script with ML fit added


I guess we can now use the ```$manager->get_actions()``` method here instead?


There is no other way of getting it to do the checking without this. This is the same way unit tests are done on the Attribute.


This did not work for me.


Yes we can do that but let me try once with TCPDF because other two (Address Label and Barcode Label) are working with TCPDF only Chart Label having problem if we can make it work then we could have an option to use single PDF library on whole system


僕の想定では、論文のように
段落>図表>ツリー構造図
とするつもりで、背景色は図表の部分にかかってるという意識でした。

この場合はどうするのが一般的でしょうか？


ごめん、textタグってどういうタグなのか知らないので教えてほしい。
あとここでtextタグを使う意図ってなんでしょう？


I'll leave the decision up to you (if you can do it that would be great, but if you start banging your head against the wall out of frustration, feel free to take the easy way out).


@shehzan10  As in, an additional function that takes only RGB values ? or change this one to move alpha to bits 24-31 , for eg. 0xAARRGGBB ? - this way, even if they don't set AA too specific value, we will be fine - we can check and set to 1 in case of zero.

@pavan @brian what option way it should be in your opinion ?


Looks like we lost the @param namespace docstring here. Put it back please?


No unit tests?


+#include <iostream>
 +using namespace std;
 +
 +struct Weather
 +{
 +    double totalRainfall[12];
 +    double hTemperature[12];
 +    double temperature[12];
 +    double averageTemperature[12];
 +
 +};
 +
 +void calAverageTemp(Weather &W)
 +{

 +    for(int i = 0;i< size;i++)
 +    {
 +        w.averageTemperature[i] =W[i].HTemperature[i]+W[i].LTemperature[i];
	ave[i] = ave[i]/2;
 +
 +
 +    }

 +
 +}
 +
 +int main()
 +{
 +    Weather W;
 +
 +
 +    for(int i = 0;i< 12;i++)
 +    {
 +        cout << "Enter Total Rainfall in " << i  +1 << " month ";
 +        cin >> W.TotalRainfall[i];
 +
 +        cout << " Enter the highest temperature " << i +1 << " month";
 +        cin >> W.HTemperature[i];
 +
 +        cout << " Enter the Lowest temperature " << i +1 << " month";
 +        cin >> W.LTemperature[i];
 +
 +
 +    }

	calAverageTemp(w);
 +
 +
 +
 +}


Not sure what happened, but it stopped working for me. I needed to access the port via http to start the download, which prevented the default behavior.


Okay so I created an issue #2


No its alright. Its good that we have a different secret for Devise. Just make sure the context file is available on the Jenkins.


Don't you mean "Subclasses must override addArgument"? getdatasetType is a member of DataIdArgument.


Okay, let's not bicker. Instead of trying to get you to explain why you prefer `this.speciesid === 'shedinja'`, let me just make my case for `this.template.baseStats['hp'] === 1`:

`this.speciesid === 'shedinja'` and `this.template.baseStats['hp'] === 1` do the exact same thing in Hackmons and Custom Game. They are both valid interpretations of:

```
0803E554                 LDR     R0, =0x12F
0803E556                 CMP     R7, R0
0803E558                 BNE     loc_0803E568    @ Routine to calculate HP.
```

But this is compiled machine code. Who knows what the uncompiled code looks like? Maybe it's not even hardcoded; maybe the uncompiled code just checks a list of Pokémon whose base HP is 1, and if you change Shedinja's base HP in the source, it'll have more than 1 HP. You can't say whether it does or doesn't.

So the only question between `this.speciesid === 'shedinja'` and `this.template.baseStats['hp'] === 1` is: which is better for mods?

I would argue that `this.template.baseStats['hp'] === 1` is better for mods, since it lets you 1. change Shedinja's max HP, and 2. give other pokemon 1 max HP, all just by editing pokedex.js (without needing to edit the engine). To be fair, it makes it so you can't have a regularly-calculated 1-base-HP Pokémon (without editing the engine), but I think this is much less common than wanting to give Shedinja more HP and wanting to give other Pokémon 1 max HP combined. And as always, you can have whichever you want by editing the engine.

Now, you call the default behavior for Averagemons and Tier Shift "wrong", but this is only wrong if you define the stat changes in terms of hexediting the games' compiled data, not if you define the changes in terms of editing the source or in terms of what you'd expect. Which, while you are free to define it that way, but I think it's a stupid way to define it.


updated
https://github.com/AtlasOfLivingAustralia/biocollect/commit/c78cc6746e5bf948f56695d2db078de048430c5e


We can add const.


Is this comment misplaced ? I didn't understand what you were referring to by `this` in the above comment 'rendering this` ?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#DHZd1BAETOK4CuA28ZKmgw


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - Win64-Windows-7-VisualStudio-12: OK (2121 of 2135 tests passed, 1 test warnings)

 * 15 tests had: EIO diffs.
 * 7 tests had: ERR diffs.
 * 14 tests had: ESO big diffs.
 * 2 tests had: Table small diffs.
 * 11 tests had: Table big diffs.
 * 10 tests had: MTR big diffs.
 * 2 tests had: MTR small diffs.
 * 1 test had: ESO small diffs.

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


@jimxj @lstanii-magnet Please review


I just committed the following fixes to the OpenEMR codebase:
1. Your regex fix.
2. Your calendar unbalanced parenthesis fix. Note I needs to revert the variables you changed, because they were not working on testing (when using non-english language, the translated term is supposed to be shown to right of the entry).
Commit with the fixes is here:
https://github.com/openemr/openemr/commit/5ef45151bbd310ed096ddbe0df871e70e6b9b9b8

I have a quick question regarding the above fix. For line 732 in the interface/main/calendar/includes/pnAPI.php script. Did that line need to be changed(I noted it wasn't using the 'e' thingy)?


You are free to open an issue which proves to overflow.


Note that for the Address/Patient/Barcode issue, also note we need to ensure all the options are supported at Administration->PDF:
'Patient Label Type'
'Barcode Label Type'


`{0..3}` のほうが UNIX 的には良いと思います。


カーソルの位置に砲台を置けるようにする #305


That's the type used by libraries (for e.g. glbinding, boost::compute to handle types like OpenGL context when exposed to user). Also, we have been using `long long` for this context handle since the onset of forge and we haven't seen issues related to this so far.

Any particular reason why you don't like it ?


Hi,

when building on arm or power, sqrt in trmv is an unresolved externel

Werner

On 03/07/2016 07:49 PM, Jerome Robert wrote:
>
> @xianyi <https://github.com/xianyi> Thanks for the account but I do
> not get any error when running or building it on your machine.
>
> @wernsaar <https://github.com/wernsaar> May I have a compilation error
> message, so I could try to fix it ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xianyi/OpenBLAS/commit/0bbca5e80392625e1af911cc8730c4928b412750#commitcomment-16543190>.
>


When will be able to upgrade to 1.9?


Hm, that's strange. How do I do that?


@shurcooL My bad for not checking well enough. This can't be `const` because it is modified in the for loop below. I could split it out into a separate const, but for the reasons I listed above I think that is the wrong choice. I've returned it to `var` now.


クラス名は短縮せず、 PacketRegisters のほうが分かりやすいと思います。


Yep, definitely can't get to generate osd config.


em học mót từ google đấy :))


5にもそんなものはなかったので、間違えてました。
p/div を使おうと思います。


I'm surprised that RuboCop doesn't complain about this - I thought that it's not advised to chain method calls on block end (unless its `{}` style).


syn


div要素自体に論理的な意味は無いから、ul要素単体で完結しているならui要素に背景色指定して、ul要素以外にも背景色の範囲に入れたいならdiv要素でその範囲を明示的にしてあげるぐらいの考え方で良いと思います。
他の要素で範囲指定できるのにdiv要素入れるのは冗長かと思います。

あと下記の質問についてはどうかな？
> ちなみにこのpタグって何に変更するのが相応しいと思う？


@Cat7373 soon(tm)


TeamCity Todo :: Build [Build 50](http://109.103.226.38:8512/viewLog.html?buildId=38178&buildTypeId=Todo_Build) is now running


であれば問題無いっす！


Should I have to update gem?

I write in my model scope :leaves, lambda { joins("LEFT JOIN categories AS c ON c.ancestry = CAST(categories.id AS text) OR c.ancestry = categories.ancestry || '/' || categories.id").group("categories.id").having('COUNT(c.id) = 0') }
It give me an error


TeamCity Todo :: Build [Build 50](http://109.103.226.38:8512/viewLog.html?buildId=38178&buildTypeId=Todo_Build) outcome was **SUCCESS**
Summary: Tests passed: 4 Build time: 00:00:41


You don't need to escape `/` when using `%r{}`. I thought it's perfectly fine to use `//` literal when only one `/` is inside the regexp.


Thanks for your efforts 0.0


It should be `[]`, not `()`.


We may want to have a single "SHOW CREATE ..." command that works for views and tables. There's no ambiguity between them, since they share the same namespace.


Why do we consider createdTime to be a universal property of a datasource? Does a database table always have a created time? Why is it important?


Oh, it's fixed in the latter commits.


SOFTWARE


This class doesn't seem to do anything useful yet. How will it evolve in the future?


Did you remember I already implemented TEST_EXIT?  It's a simple
implementation, but probably adequate for unit testing of decent code.
Someone can make a more complex and capable one with TEST_EXIT if they
have the need.

James

On 4 Mar 2016, at 2:12, arstrube wrote:

> I would probably wrap the whole thing in a 1 line macro for myself,
> and you'd ask me to get rid of the macro :D.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cpputest/cpputest/commit/5b1e89b21319eaf780866f5d14f80544361679e1#commitcomment-16494112


.... does this fix anything?


Will fix it.


tl;dr if a game wishes for shedinja to be mechanically accurate it must be coded specifically to do so


I love you @dmmiller. :heart:


Limit visibility and mutability. Unless something really *needs* to be `protected` make it `private` and if it doesn't need to be mutable then make it `final`.


Why do we want this? Leading dot always tells you that this line is method calls continuation and is enabled by default.


To add consistency with the sign-up and login-form, how about changing this to
`col-md-4 col-md-offset-4`


same as above


What is the purpose of this interface?


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


Being picky, but here are some suggestions:

1. This Line Should be in Title Case (like this sentence)
2. `Percentage`


I still don't understand your definition of "mechanically accurate" here? You can't be referring to ORAS, XY, BW2, BW, HGSS, DPP, FRLG, or RSE, because Shedinja has a base HP stat of 1 in all of those games, which PS correctly implements.


`solution`, `elements`


Why do we even have this` <div class="mt40"> </div> `here ? This creates extra padding on the top, and can make things look bad in mobile devices.
To avoid the extra scroll down, just remove this class.


Nah, that is wrong condition you caught there. There needs to be a condition except that it is not what i have written here, the shared window `shrdWnd` should be checked there. It is a typo, I will fix the condition.


Minor nitpicks, rest looks good.


missing a `<td>`? also, I'm sure there is a tracking bug for this, I've seen it many times. I'll add it here if I find it


This looks good! A couple suggestions:
- Implement these two assertions in the base_validator, call them from here with the page-specific locator to make them the page-specific assertions
- import the homepagelocators class and reference those locators, instead of putting strings in this file


Not sure what these extra newlines are for. Can you remove these?


There are cases like users have website created and they have zip file with same website. If they are not interested in updating website then this section is irrelevant for them and hence this check box. Please don't remove this and float your PR against vso-agent-tasks/master


Yes line 732 can go without preg_replace_callback I did not realize it is without "e" modifier

I am reverting this change


<img width="1140" alt="screen shot 2016-03-07 at 10 20 41 pm" src="https://cloud.githubusercontent.com/assets/17078886/13593773/397594f6-e4b3-11e5-9a07-1bdeeb44106e.png">


generally the comment describes what is already written with code below. Instead, can you please write motivation behind this? What is happening in the next process phase?


It doesn't break anything. Working on those changes as we speak so we can unmerge the projects asap @Minecrell


Sounds good to me.


iteration (macumber) - i386-Windows-7-VisualStudio-12: OK (2363 of 2397 tests passed, 0 test warnings)

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


Our variables are unsigned. So the ID integer would max at around 4,294,967,295

That amount of rows is absurd and we do NOT recommend anyone operate a hawkeye instance on that large of a database. I'm not sure if INNODB would even be able to process queries on a table that size (Complete guess).

I might make an initialization prompt later on. So it'll say something like "Would you like to use larger integer types for storing data (Larger overhead/resource usage) [Yes/No]: ".

Thanks for taking an interest with HawkEye though, it's nice to know people are still utilizing it.


@tw4452852 Do you mean the Markdown() method (which is Include() but with Markdown rendering)?

The problem we're trying to solve here is that there's a type called markdown.Data which is a context that is used to render markdown files; it embeds a middleware.Context. But when the Context's Include() is called, that included file can't access .Doc etc. on the markdown.Data type... so the only way I could think of was this kinda ugly circular reference hack.


Mod Idea: "What if the Pokemon in game had less base stats?"
Mod Ideal Implentation: "Reduce Base Stats"
PS Result: "Anyone who hits 1 is now Shedinja."
Expected/Cartridge Result: "Anyone who hits 1 now has 1 base HP"

If one wants to design a mod based on these examples they have to specifically code for them to stay aligned WITH the games. Why you feel so strongly about this deviation is beyond me.


I can't comment directly on the DDF.java file because the diff is too large.
What is making DDF a generic type achieving here?

Also, I'm not convinced that pulling all methods from handlers into the main DDF class is a good idea. It creates a wide (and possibly unwieldy interface). And now there are methods in two places, in the functional group handlers and in the DDF interface itself. Unless we can avoid using the DDFs functional group handlers altogether, this will end up duplicating all the interface methods and create a maintenance burden.

cc @PangZhi @mbbui @steveyan777 @nhanitvn


It doesn't fix anything either :P


Please?


"mechanically accurate" in this case means a simple change reflects a simple change in the game. Like Hackmons' Any Move/Ability. Abilities hard-coded to work with only certain Pokemon only work with those Pokemon both in cartridge and on this sim. Why is Shedinja different?


Acknowledged, Can't fix as this code is obsolete and no longer exists.


No you're right. This has no affect @Aaron1011 :P


That is the style i have been following for inline member functions of class and hence it is consistent in all files.


I think for the replacement we should use '$1' instead of '\\1'?


https://github.com/webpack/webpack/issues/1599#issuecomment-193625057

```js
if (/^\.\.?\//.test(req)) return cb() // fixed √
```


IFileFormat is used to resolve a specify datasource.
For example with HDFSDataSource it could contain JSON or CSV or Parquet
For each type of FileFormat, an engine will design how it resolve or support.


We need apply this https://github.com/crosswalk-project/chromium-crosswalk/commit/0177c23e5e5386bbeaf97dc680afcac541659cc0

before the following patches, the following patch will introducing dependence with ContentReadbackHandler.:
[Android] Add a new listener to listen the first rendered frame]
https://github.com/crosswalk-project/chromium-
crosswalk/commit/a8e4b2f99480b63024c42b14017865f6fc98f316

[Android] Use TextureView as an alternative compositing surface target
https://github.com/crosswalk-project/chromium-crosswalk/commit/84699a70714fa85625c556c80c9b4593458c4edb

[Android] Only initialize ContentReadbackHandler in onNativeLibraryLo…
https://github.com/crosswalk-project/chromium-crosswalk/commit/bcb0bdf8757ebad561c7d28ebb2743111c90bfdc


htop is a good addition since the VM can't be monitored from outside. what's graphicsmagick for?


Thanks, didn't know that.
This line isn't necessary anyway ;>


Do you have a specific case in mind? I can't find any breakage.
About that, I don't understand how the REPL could make the difference between the error from `JSON.parse` and an unterminated object literal (when the message was identical).