Electric-Service-Refactor (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (2083 of 2090 tests passed, 546 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 1 test had: SHD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


test


Does not seem so, according to angular/angular.js#5695 .
Else, we have to use `class="{condition ? 'classname': ''}"`


There's no brackets here


Indeed. That would have been nice.


and what if I paste www.exoplatform.com  ? will you catch it ?


When no site name, the data['site_name'] variable's value is "false".


github does. and probably also exoplatform.org


Or here


Yeah I just totally saw this by chance...waiting for an NBL run to start I
thought I'd see who had forked meza and saw that your repo was ahead of emw.

On Thu, Feb 18, 2016 at 10:18 AM, Daren Welsh <notifications@github.com>
wrote:

> Thanks. I figured I would test it out on a fork before messing up our real
> meza repo.
>
> On Thu, Feb 18, 2016 at 9:53 AM, James Montalvo <notifications@github.com>
> wrote:
>
> > With travis you'll have to make it build on CentOS. See
> >
> http://stackoverflow.com/questions/32535195/how-to-run-tests-on-centos-7-with-travis-ci
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16179311
> >
> > .
> >
>
>
>
> --
> __________________
> http://enterprisemediawiki.org
> http://mixcloud.com/darenwelsh
> http://www.beatportfolio.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16180100>
> .
>


exopaltform.com works ?


Doh.  Thank you.


pwrite() returns errors in errno. Neds to be "return -errno".


Similar here...


I don't think so.  The result for patterns is compiled on demand (the format value of the pattern), which will be different from anything else.  I was thinking of checking the tags in the pattern, but the result is not the pattern but its format value so the pattern itself is not accessible, I think.


nitpick: I'd just write "return 0" here... (yeah, r should be 0 too, but the processing of fsync()'s return value is already complete above...


I just did that!


break;


less or lesscss?


I'm going to find some small part to play in this project if it kills me, just so I can say I worked on the Apollo Program


@dret we have "must ignore" (from I-JSON) in the interop considerations section and I think that's as good as we'll get in our WG. I think that the idea of a format with no processing considerations is fallacious, but it's not an argument that needs to be won in order to ship this draft.


I didn't understand why it said my fork is ahead, but maybe that "one
commit ahead" is the act of forking. I'm not really sure how to set up the
travis script and I don't know if there's another docker container that's
better for our case. I don't think we need ansible, but I'm just going to
try this out.

On Thu, Feb 18, 2016 at 11:29 AM, James Montalvo <notifications@github.com>
wrote:

> Yeah I just totally saw this by chance...waiting for an NBL run to start I
> thought I'd see who had forked meza and saw that your repo was ahead of
> emw.
>
> On Thu, Feb 18, 2016 at 10:18 AM, Daren Welsh <notifications@github.com>
> wrote:
>
> > Thanks. I figured I would test it out on a fork before messing up our
> real
> > meza repo.
> >
> > On Thu, Feb 18, 2016 at 9:53 AM, James Montalvo <
> notifications@github.com>
> > wrote:
> >
> > > With travis you'll have to make it build on CentOS. See
> > >
> >
> http://stackoverflow.com/questions/32535195/how-to-run-tests-on-centos-7-with-travis-ci
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16179311
> > >
> > > .
> > >
> >
> >
> >
> > --
> > __________________
> > http://enterprisemediawiki.org
> > http://mixcloud.com/darenwelsh
> > http://www.beatportfolio.com
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16180100
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16182440>
> .
>

--
__________________
http://enterprisemediawiki.org
http://mixcloud.com/darenwelsh
http://www.beatportfolio.com


definitively


It appears to be indentation, adding a wrapper div


que bonitu :3


ah! In that case, why not just add the keydown to the original top div?


I was thinking consumers could use this in cases where they're basically filtering on some sort of object or set of properties in the consumer. For example, for his risk+sm filter, Sommer could just do a master:
```
serialize() {
    return {
        risk: this.risk,
        issm: this.issm,
    };
}
```
instead of specifying a serializer for each option


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 266](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=799&buildTypeId=XbimEssentials_XbimEssentials) is now running


A comment explaining what this is about would be good, because it's not obvious. Saying something like "This call is executed in a background thread. It's main purpose is to invoke fsync() without stalling the main process. blababalabla..."


Do you need them?


This is to avoid two identical statements. Of course this block can be rewritten with do .. while loop, like  this:
  do {
    bytes_read_cnt = read_from_socket(fd, rbuf, (size_t)MAX_READ_BUFFER_LENGTH);
    parsed_cmd_length = parse_input(rbuf, cmd, (size_t)MAX_COMMAND_LENGTH);
  } while (bytes_read_cnt > 0 && parsed_cmd_length == 0);

Although this approach is  more elegant, it's rather hard to read. I had to draw a truth table for the expression in parentheses.


Here and elsewhere: the whole project tries to follow PEP8, lines  < 80 chars please. Fill fix that when merging, no need to resend just for that.


It's have been corrected in the following commit.


xxx


Might be helpful to ask for the output of slcli --version


line 3


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 266](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=799&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:35


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


Confirm that this functionality is not available in latest PIL/libtiff..?
If we absolutely must, we should import this *inside* the function itself,
and well-document that it's required. Definitely a bad idea to require
OpenCV globally here... though I'm willing to discuss further and be proven
wrong

On Thu, Feb 18, 2016 at 12:24 PM Alex Eusman <notifications@github.com>
wrote:

> @j6k4m8 <https://github.com/j6k4m8> Unfortunately no, its required for
> 16-bit tiffs to be shape checkable
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/neurodata/ndio/commit/16b9e951d4d5fb4336cfccb052f3285c6cec1e6f#commitcomment-16182323>
> .
>


If there is, we were not taking advantage of it before. `workflow-archiver` used to have explicit logic to disconnect from oracle after every request.


> i just say you don't KNOW genisys and clearsky

I am One of the creators of ClearSky and of course i know genisys. @PeratX joined IM once, and @aodzip showed me his project. i was on the test server (and still have the ip)


My guess is that it wouldn't propagate down when you click on a stock and press a key, since that's part of a different div to where it was put originally


You are welcome ;)


I don't feel a "magic" default sorting is always nice to have. I'd rather have no side effect with filter commands and have a separate --order parameter. Will think about it more


You might want to use @ModelAttribute annotation, which will allow spring to pack params into your entity object automaticly.


Hi @jfmengels!

First off you rock. Thank you for your contributions, I hope this doesn't discourage you.
I think this approach is a better fit for the time being. Generating things from the map file is straight forward and answers the questions of "what's different" the easiest.

I dig what you've started though so I could see that continuing as a side repo until things get ironed out.


You can use lambda here


hmm ok strange, I had checked on master... My bad probably (sorry for the noise then)


Esto ya no se usa, eliminamos rama?


I'm not sure, it might be good for consistency to have `() =>` since that's what was used elsewhere on the original es6 PR


Reference a page or post by name instead of ID ... what's to explain? There are two examples in the commit comments.


It is a performance optimization. I believe HasAttr was fine the way it was. The offending code was nodes() returning an empty array and not setting @nodes to it.


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.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)


Yeah!


Why set public variables in script?


This sentence is bad, change to: "Alerts provide a notification **for** certain issues that **can** occur to a give component."


Ah I see. No idea then. The change seems fine, so just remove the commented code for now.


Fixed.


@PeratX Guess what we did. @aodzip said: "lets wait, when it will be on Genisys or ImagicalMine annd which first"

30 Minutes later you pushed a commit that looks HILARIOUS similar to what we (mostly i) did.


When commiting, only commit files with relevant changes.


I'm mostly indifferent, I'll go with the majority.


"Dieser Server läuft mit **der** Version {%0} {%1} (API: {%2}) Build {%3}" maybe? @robske110


war...


I added in the wrapper div so that keydown would be captured as long as the focus is anywhere on the search. I noticed before that it would only work if you were focused on the input itself, which seemed weird and also kind of hard to find out by just using the app


Seems like this else statement is unnecessary. Does it break if this is removed?


Here, actually. The else statement.


we could use choices= (just like the option just above) to have the option parser code validate that the --test-state is among the possible choices.


Please see https://github.com/PowerDNS/pdns/blob/master/modules/gmysqlbackend/enable-foreign-keys.mysql.sql


It doesn't break it, I guess it's more of just a backup.


Is that necesary? but anyways.


Should we leave these blank for the reporter to fill out?


The code probably should look slightly differently to make this reliable:

```c
for (;;) {
        switch (f->offline_state) {
        case OFFLING_SYNCING:
                if (!_sync_bool_compare_and_swap(&f->offline_state, OFFLINE_SYNCING, OFFLINE_OFFLINING)
                        continue;
                r = journal-file_set_offline_fd(f->fd);
                …
        case OFFLINE_OFFLINING:
                if (!_sync_bool_compare_and_swap(...))
                        continue;
...
}
```
i.e. we should figure out what to do, then swap the state, but if that fails, we need to figure out again what to do, hence we need an inner loop here that tries again if the state of the atomic variable changed between the time we decided what to do and the time we update the variable.

Your code above is racy regarding the following case: let's say the offline state is currently set to OFFLINE_AGAIN. And the first three CAS are run this way. Right before the fourth CAS the state is set to OFFLINE_CANCEL, but this causes the call to fail with -EINVAL. Instead it should just have exited the loop cleanly.


I've been naming these like `test_CreateAnnotationSchema_*` 'cos I don't think that lowercasing it helps readability (even though pylint will complain). (Or put the tests for a class in a test class.)


Also fix your git username on school computer. youre showing up as v1604pejo@VFK-PF085U3A.skole.top.no


@PocketKiller no. The copyright is removed.


Can't we shorten this to:

```c
return INT_TO_PTR(journal_file_set_offline_internal(f));
```

i.e. no need to invoke pthread_exit(), in this case just "return" should do the same.


Probably worth talking about the 'null' special value.


I can't fail in the `<<` override because Virus uses this internally during coercion of the collection entries, but I can generate a warning.

https://github.com/solnic/virtus/blob/master/lib/virtus/attribute/collection.rb#L79


Hi.

There is one postrotate useless line (line 8 and 9) that broke logrotate on centos 7

sudo logrotate --force /etc/logrotate.d/varnish
logrotate_script: ligne1: postrotate : commande introuvable


Why are you doing this every time you call `validate`? In the previous iteration you were doing once while cosructing the service


I'm not sure how this works - so perhaps a stupid question: Does this allow to pass multiple parameters?


We should also get the observed issue narrative with relevant log from the sample test


-errno will be


@martijnlaan  What you mean "Typo here. News -> New." ?


For some reason, the error messages don't disappear when a valid value is entered.  Not sure why, but I wonder if there's something going on with angular material and the component model.


needs a similar loop as described above. after all the thread can change the state at any time too, and we need to restart checking if it does.


another option is to have something outside ndio to convert these 16-bit tiffs.
@kunallillaney opine?


Cool. Good luck!

On Thu, Feb 18, 2016 at 11:31 AM, Daren Welsh <notifications@github.com>
wrote:

> I didn't understand why it said my fork is ahead, but maybe that "one
> commit ahead" is the act of forking. I'm not really sure how to set up the
> travis script and I don't know if there's another docker container that's
> better for our case. I don't think we need ansible, but I'm just going to
> try this out.
>
> On Thu, Feb 18, 2016 at 11:29 AM, James Montalvo <notifications@github.com
> >
> wrote:
>
> > Yeah I just totally saw this by chance...waiting for an NBL run to start
> I
> > thought I'd see who had forked meza and saw that your repo was ahead of
> > emw.
> >
> > On Thu, Feb 18, 2016 at 10:18 AM, Daren Welsh <notifications@github.com>
> > wrote:
> >
> > > Thanks. I figured I would test it out on a fork before messing up our
> > real
> > > meza repo.
> > >
> > > On Thu, Feb 18, 2016 at 9:53 AM, James Montalvo <
> > notifications@github.com>
> > > wrote:
> > >
> > > > With travis you'll have to make it build on CentOS. See
> > > >
> > >
> >
> http://stackoverflow.com/questions/32535195/how-to-run-tests-on-centos-7-with-travis-ci
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16179311
> > > >
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > __________________
> > > http://enterprisemediawiki.org
> > > http://mixcloud.com/darenwelsh
> > > http://www.beatportfolio.com
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16180100
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16182440
> >
>
> > .
> >
>
>
>
> --
> __________________
> http://enterprisemediawiki.org
> http://mixcloud.com/darenwelsh
> http://www.beatportfolio.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16182492>
> .
>


Nice! It works :D

I do have some "ideas" regarding your python code, but that is secondary.

Haven't played around too much yet, though...


@raphaelpaiva, nesse caso, eu assumo a "culpa", pois, apesar de eu ter copiado e colado o código, tenho mesmo o costume de nem sempre usar o StringBuilder ou o StringBuffer para concatenar Strings. Em casos que envolvem um número maior de concatenações eu até uso, com base no argumento da performance. Mas eu tenho o problema de achar essa sintaxe dos appends verborrágica demais. :-S De modo que, em trechos pequenos, eu acabo fazendo do modo mais simples, que me parece mais straightforward para escrever e ler. (Aliás, que bom que inventaram o autoboxing, apesar as [desvantagens](https://effective-java.com/2010/05/the-advantages-and-traps-of-autoboxing/)) É verdade que fazer do outro modo é uma questão de disciplina e que trechos aparentemente pequenos acabam somados, no fim das contas. Mas não sei se me convenci o suficiente de que substituir o += pelos appends realmente compensa em todas as situações.


descriptionShort only appears for device faults too?  others just use the alertType?


This just replaces push constant use with old fashioned UBOs, it's only used in vertex anyways, so performance is irrelevant.


This is one location which expects all the constraints. For some pages one might not need all the constraints. One option but I'm not sure if it's worthy would be to have another associated `type FormPageConstraints p :: (* -> *) -> Constraint`.


also needs a loop as suggested above.


I think we should try to always cast syscall return codes to (void) if we knowingly ignore them.


Weeeeeeeeeeeeeeee


There is a similar node in Nuke (CurveTool), which can be used to deflicker a video. I made this node before noticing the Nuke node.
I may add more functionalities from CurveTool if needed.


I added code to convert time to UTC before Ian told me dates should be stored in CKAN as naive (i.e. no indication of TZ) UTC datetimes.  I've since updated the massage scripts to convert datetimes to UTC before they are imported, so it's not necessary to convert them again in before_index.


Instead of using the name here, it could be referenced with C# 6.0 syntax of `nameof(RunsBothBRanchesOfTheExperimentAndMatchesExceptions)`.


Thought I'd lost these changes!


Good idea.


I only put this out since these details are very generic. I suppose we could do one for PRs but we hardly get a large enough number of them.


Workflow archiving runs under cron every 15 minutes on lyberservices-prod. It would normally be the only user of this service:

0,15,30,45 * * * * /bin/bash -l -c 'BUNDLE_GEMFILE=/home/lyberadmin/workflow-archiver-job/current/Gemfile ROBOT_ENVIRONMENT=production /usr/local/rvm/wrappers/default/ruby /home/lyberadmin/workflow-archiver-job/current/bin/run_archiver >> /home/lyberadmin/workflow-archiver-job/current/log/crondebug.log 2>&1'


So, did you get your 2x performance increase with these changes?


@laruence I want to see this change (not the revert), don't get me wrong. With my documentation hat on, a heads up wouldn't go amiss.

As for why comment now? I don't watch the php-src PRs, so missed it there and only came now as it was linked in a place I do watch. Sorry if that's not good enough for you.  That said, I did see the original commit (not the revert) email, and inwardly applauded the change.


Yeah, I didn't find what caused them either.


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1536 of 1540 tests passed, 0 test warnings)

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


Option #2 does seems like the easy approach. I suppose it is up to the developer to handle multi-threaded issue such as that.


This commit actually breaks Google IMAP for me..

brynet: unexpected data: 6 OK Success
brynet: fetching error. aborted
brynet: 0 messages processed in 2.226 seconds


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


should be "volatile" really, to tell the compiler to never cache this.


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


It makes them in alphabetical order.


Aren't we missing some ops, here?
```lua
          x - mean(x)
y =  -------------------- * gamma + beta
         std(x) + eps
```


Should be a `DELETE`, not post.  Also, this line should end in a comma.  As it is, your js fails to load because of a syntax error.


Avoid this kind of debug messages


This screwed up some things later on. I don't remember what this was supposed to fix, exactly


I guess `server` was getting confusing, but `control` doesn't seem much better.


Should also remove the item from the page!


I would suggest using our javascript module pattern which assigns behaviour based on a `data-module` pattern. An example module is something that automatically tracks an analytics event based on markup: https://github.com/alphagov/govuk_frontend_toolkit/blob/master/javascripts/govuk/modules/auto-track-event.js

Something similar could be created which listens to submit events.


I just said I don't know what email I used


@NicholasBuse the correct order should be: git fetch && git reset --hard origin/master
To first fetch the new state of origin before applying it onto the local copy.


Please rework this pull request (cherry-pick, squash or whatever...) so that it only contains code that is necessary for the commit.


No, it requires a valid url. As in eXo chat.
Github recognizes www.exoplatform.com but not exopaltform.com.


personal IDE configs should probably be put into user specific settings e.g. settings.xml


Use the `@login_required` decorator


Was this testing something that we need to cover some other way?


looks conceptuallly ok, however the atomic variable usage is broked. CAS usually requires looping around operations, until they succeed.

(An alternative to using atomic vars and CAS is of course to just use a proper mutex, if you don't want to deal with the theoretical complexities CAS brings. Another option could be to simply use a pipe() after all..)


Thanks. I've updated it


:+1: Can I work in a summary ? in order to generate one number. The idea is to merge this benchmark in openbenchamarking.org ( with all the credits to authors ) What do you think ?


thanks for the correction


Should be using AJAX -- you should return a partial with this single todo, added to the page through JS.


More details here: https://github.com/alphagov/govuk_frontend_toolkit/blob/master/docs/javascript.md#modules


@TomHAnderson just tagged 1.3.3 with the patch, thanks for reporting the issue!


typed_function_ordering (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-passing-green.svg)


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


typed_function_ordering (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-passing-green.svg)


Based on our chat, I've changed it in https://github.com/okfn/oki-py/commit/90c6831267b282b4028cde591a267754cecc0973 :+1:


a volume group can have multiple physical volumes.


Увелечить челен без регестрации и смс. Обращаться по телефону


Trying to make that happen! Just need some organization/research up front to make it possible to help out.


Lol, this was meant as a pun ;p


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 158](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=800&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


No tabs please, just spaces.


_Very_ happy about the way this cleans up Server, but I wonder if most of these (everything not involving ResponseOr) actually belong in `core` somewhere. They will be needed by the new REPL, for one thing.

On the other hand, that might be something we want to sort out in a separate PR.


Agreed.


Should it get pulled out of the APP Grape API?  Why would the API need to call it?


Typo: "Set the to the same value" -> "Set the parameters to the same value"


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


So histrionically we have use opencv since it is much better to deal with 16-bit tiffs and @Aeusman was using this(because I suggested it). @j6k4m8 pointed out that we can use libtiff as well which I confirmed is true and here is an example to use http://peterthomasweir.blogspot.com/2015/01/using-pylibtiff-to-read-tiff-files-in.html.
If opencv is a pain, we can switch to libtiff. @Aeusman We can talk more about this in person if you want.


should be sufficient to just add a full memory barrier here (`__sync_synchronize`), and then read the value directly.


* I haved added '_' to hypertargets in order not to confuse with values (this was the case for Sdl.init)
* some headings were not treated correctly by ocamldoc because of superfluous newlines. I removed these newlines.
* some targets were missing from the table of content, cf for instance "pixel_formats" in http://erratique.ch/software/tsdl/doc/Tsdl.Sdl.html
This was mainly due to the `(**/**)` that were added around the 'unsafe_foo_of_ptr' functions. I removed the `(**/**)`.

Now everything seems to be ok


Do you mean within this file? See line 103

Within train.py, it's used in lines 72 & 73


Awesome :+1:  Thanks, Pat!


No AJAX DOM manipulation.  Please finish/re-do this with AJAX!


`figlet` for the win


yes, by using CL format


I have 8 sliders and 8 knobs, so I have plenty of parameters to play with, now I just need to figure out what most of them do (some I get... :p )

Mapped up most of them:
![image](https://cloud.githubusercontent.com/assets/72965/13152876/9cfb5ab0-d671-11e5-8b3d-2315a045ddc1.png)


i think it would make sense to turn this bit into a function of its own. journal_file_close_set(Set *s); or so...


Runs well so far under Linux, 1 year and no issues.


I think this should actually be promoted to an `external_configuration_enabled` top-level configuration, while continuing to be defaulted `false`.


btw, not sure if it's worth it, but you could also turn this into a hashmap indexed by dev_t + inode nr, and then just sync on any journal file sthathave the same dev_t+inode as this file...


This logic will cause models to hang forever if training succeeds.

The status != "ready" and subsequent warning should be a secondary if statement within the first.


Can we make this an error instead of a warning?


Looks good to me


Aplicación de tienda


Im down to switch to libtiff, my concern is the same we ran into, it doesnt work on some systems, and those may be the systems people are using


I think we should include the path we looked at in this error (which might or might not be user supplied).


First column represents the Use Case. Second column represents the individual GUI page that houses that particular use case scenario. Third column represents the particular page number of that GUI screenshot that is found in the Google Drive > Roommate Matching System > Milestone 2 > GUI > Rapidprototype.doc


Same here.


Remaining columns are for future purposes to track changes to these use cases and any additional testing done on them.


Here, I think we want to conditionally modify the component list based on that top-level `external_configuration_enabled` configuration.  So something like:

```ruby
components = [
  TomcatInstance.new(sub_configuration_context(context, 'tomcat')),
  TomcatLifecycleSupport.new(sub_configuration_context(context, 'lifecycle_support')),
  TomcatLoggingSupport.new(sub_configuration_context(context, 'logging_support')),
...
]

components << TomcatExternalConfiguration.new(sub_configuration_context(context, 'external_configuration')) if @context.configuration['external_configuration_enabled']

components
```


I would prefer the new fields not show up at all in the csv, as I don't want anyone to interpret those as valid numbers, when using something like elastic search.  If the fields are not present in the sar data, then they should not exist at all.


Nice! Another awesome, simple improvement that will help users be less confused.


By conditionally adding the component, it would not need to be conditional here.  If the component is run, it should do its thing.


should use the same function suggested abive I guess. journal_file_close_set() or so... the difference is after all only in whether the result is just cleared or also freed.


yeah, i should have squashed this pr.  ignore this first one and look at the last commit.  it's fixed up later on


setListAdapter(new ArrayAdapter<String>(this,


I know this is a WIP, but is there any reason the Landing Page content type is provided by its own feature?


android.R.layout.simple_list_item_1, tests));


Made that a method of TestState (get the state from a string)


It would also remove the need for conditional logic here.  As before, `@droplet.copy_resources` should _always_ be executed.  Then if the external configuration TAR is available it will simple expand over the top of the ones that already exist.  You end up with a precedence of

1. external configuration
2. resources configuration
3. Tomcat default configuration


Could you please add sudo to each command or explain in one area that sudo might be required if user not part of docker group.


Does this and the next line apply to `force-pseudo-nesting`? I don't understand what it means in that context. [Relevant link](https://github.com/sasstools/sass-lint/blob/develop/docs/rules/force-pseudo-nesting.md)


The loop is actually not OK this way. journal_file_close() frees the object after all, and you really shouldn't leave the invalidated pointer in the set. Please remove the object frist from the free and free it then. I.e. something like this:

```
while ((c = set_steal_first(deferred_closes))
        journal_file_close(c);
```


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


Could you please clarify here that release means that this removes the PDF watermark.


Could you please clarify here that draft means that this adds the PDF watermark.


Sorry for being pedantic. I really do appreciate your work. But.. Do you have a test case for this as well? :-)

These changes look a bit fishy to me, as there really are no more data in the stream. Perhaps we should rather catch the EOF and pad the image data outside the stream?

--
Harald K


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


I see, then the bug is in the tests :smile:


I'm curious about that myself.  :^)  I took the change since it acts as I would expect and the other changes were good, but I don't know what specific case was broken before.


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


Maneiro :+1:


@Standard8 Do we need to remove each string property from every local file?  or is this done from translation services?


Em todas as situações certamente não, como diz no segundo link que enviei. No caso de
    String a = "";
    a = a + "b" + "c" + "d" + "outra string";

É muito melhor pra ler. Em questão de performance, é absolutamente igual. Essa construção do + é um açúcar sintático para StringBuffers. Esse código é exatamente a mesma coisa que escrever:

    StringBuffer sb = new StringBuffer("");
    sb.append("b");
    sb.append("c");
    sb.append("d");
    sb.append("outra string");
    String a = sb.toString();

A questão é justamente o caso do loop, que é o caso clássico de quando não utilizar concatenação. Como diz no artigo, a escala é absurda: 9536Mb pra ~600Kb concatenando 100000 caracteres. No nosso caso, com certeza o impacto é menor, mas atente pra escala do desperdício: 6,2*10^-5. Ou seja: com o "esforço" de usar StringBuffers (nesses casos) ganhamos uma economia de memória na casa de 100 mil vezes. Acho que vale a pena, não é mesmo? :p

A minha intenção não foi culpar, mas eu sou ruim nessas coisas, posso ter soado arrogante, desculpa :). O que me chamou mais atenção não foi a questão de usar appends ao invés de StringBuffers (ninguém é obrigado a saber disso), mas sim da regressão. A troca de uma solução boa (apesar de misturar o append com `+`, o que seria uma melhora) por uma pior em termos de performance. Por isso resolvi chamar atenção para esse caso.

Como eu disse, o caso pode ser pequeno, mas a escala é enorme. O mais importante ao atentar pra isso  é criar a cultura de ficar atento à estes detalhes, que podem passar desapercebidos em outros casos com um impacto muito maior.

Acho importante mais pessoas pelo menos lerem esta discussão. Todos são notificados dos comentários?

[]'s


I screwed up the original commit. Info about this feature is [here](https://github.com/TonyGravagno/shortcodes-ultimate/commit/3ec2a052f0eee8672dd05bcfc44ece1b0e1d032f).


Too long, keep lines below 120 characters, please


Too long.


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


Too long


Unable to add this on my website..?


FYI @mehaase – this broke the common mark tests. We should figure out why...


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


Sorry about that. You probably don't see the comments here. I just posted a proper pull request on the gndev repo.


Electric-Service-Refactor (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2083 of 2090 tests passed, 546 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SHD diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


Dont want to decommonize this completely, most is common


same


10 years fort under df-ai. Bards are being slaughtered and then their meat is buried.
Graphics dwarf tiles are being mixed - merchants looks like doctors and babies looks like administrators, etc. And this https://github.com/sv-esk/df-structures/commit/dd6dc08faa0a977115c0a48a015eb9768fc06359 is wrong. It should be not os specific.


@gdetrez Oh, good point.


Предлагаю удалить еще .idea и все файлы из гита. )
Сейчас они под контролем версий и не будут игнорироваться даже если их поместить в .gitignore.
Уйдет проблема слияния, когда участники будут подтягивать изменения из апстрима


I see that other IsXXX defer to the underlying Type unconditionally. Like Type.IsArray above. I think it is not supposed to be null.


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-MacOS-10.9-clang: OK (2111 of 2111 tests passed, 547 test warnings)

 * 547 tests had: AUD diffs.

![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)


Ah, this is probably why @aspidites went with a separate method. We're serializing to JSON, not casting to a string in the template. Sorry for the confusion.


Alors je l'ai fait mais rien n'y change Film Ilimit ou Kepliz ne montre rien alors que leur site marche et que j'ai bien la 0.3.8 et Isengard pour le Kodi :( @TmpName


As a general rules, avoid using ```getApplicationContext()```without a very good reason. In the case of ```Activities```, use ```this``` as the ```Context``` by default.


```this``` instead of ```getApplicationContext()```


But it is strange, the docs lists the password as a mandatory field but not the `login`. But now, if you look [here](https://github.com/ralphbean/bugwarrior/blob/develop/bugwarrior/services/bitbucket.py#L69) the password is only looked-up if there is a login.

So they should eiher both be mandatory or both be optional?


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


update_travis_toolchain (lefticus) - i386-Windows-10-VisualStudio-12: OK (388 of 389 tests passed, 0 test warnings)

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


we should review all time types and calls, some time after jewel maybe


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 158](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=800&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **UNKNOWN**
Summary: Canceled (Exit code 1 (new)) Build time: 00:18:23


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 267](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=801&buildTypeId=XbimEssentials_XbimEssentials) is now running


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


Yeah i'm sort of split. Part of me likes leaving it in so it's more consistently formatted but I could see it being potentially confusing.

Perhaps something like `< your node version>` or just blank back ticks?


@ryneeverett nevermind, I just saw the corresponding PR...


php 7 i cant get it to work plz make bck to php 5


which template?


lgtm


You should prefer using something like:
`
var strConstAst = pair.Item1 as StringConstantExpressionAst;
if (strConstAst != null && ...
`


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, line 32.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -29,7 +29,6 @@
 """.split('\n')

-
 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
     (good_file1,good_file2,),
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, line 35.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -32,5 +32,5 @@

 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
-    (good_file1,good_file2,),
+    (good_file1, good_file2,),
     (bad_file1, bad_file2,))
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, lines 35 to 36.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -32,5 +32,5 @@

 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
-    (good_file1,good_file2,),
-    (bad_file1, bad_file2,))+    (good_file1, good_file2,),
+    (bad_file1, bad_file2,))
```


**Line contains following spacing inconsistencies:
- No newline at EOF.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, line 36.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -33,4 +33,4 @@
 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
     (good_file1,good_file2,),
-    (bad_file1, bad_file2,))+    (bad_file1, bad_file2,))
```


Oh I see this template: https://github.com/stefanofontanelli/elasticsearch/blob/master/Dockerfile.template


(nice to see more c++11 here)


Why not just pass in the DockerExtension object instead of creating a new map?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 267](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=801&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:37


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 268](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=803&buildTypeId=XbimEssentials_XbimEssentials) is now running


ew;  sorry


Pls name it rfbMax (to be in line with the existing naming scheme), otherwise good to merge :-)


Again, prefer a as w/ null check for better performance.


ok


missing a blank line to separate logical blocks


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 268](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=803&buildTypeId=XbimEssentials_XbimEssentials) outcome was **UNKNOWN**
Summary: Canceled (Exit code 1 (new)) Build time: 00:00:46


@gdetrez Nonetheless, it is odd that an `if` statement checks for a required field. There is **a lot** of this kind of cruft in bugwarrior. In spite of this, it works pretty darn well.


Careful putting in big commits like this - it can become very difficult to figure out where bugs started, and if the change needs to be reverted, that can cause total bedlam.

Looking at your map component, it seems like you're hard-coding road positions into the map to begin. Is this just for testing, or were you planning on hard-coding the client for starting board configuration? Either way it looks a little sketchy.

Also, who's Gary Peterson, and why does his name pop up in authorship attribution?


Hm so maybe we should document it in the interface instead?


Hey @wulfsolter - I appreciate this was a month ago now, but I'm part of the Snyk team, and specifically I work on the snyk module (that includes the wizard, patching and so on). I just wanted to ask what the reason was behind removing Snyk from your package? Really just hoping for feedback that can improve the software.

We did find that there was some wizard problems with policies and used your specific package (at 0.29.3) to test and deploy fixes (now live in [1.11.0](https://github.com/Snyk/snyk/releases/tag/v1.11.0)).

Thanks in advance!


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


@dimagi/team-commcare-hq just a heads up for people...it's actually really ambiguous / not great when you put includes inside of compress tags because it's difficult to pick up when something like this happens: https://github.com/dimagi/commcare-hq/pull/10227/files and an if statement is inside a compression block, resulting in an offline generation error.

Basically, rule of thumb, the only template tag allowed in a compress block is `new_static` or `static`. Thanks!


Just to clarify for anyone reading. This was removed but could have been fixed by

```
expect(Exchange.model_name.to_s).to eq "Spree::Exchange"
```

However, there's no reason for us to test the behaviour of ActiveModel


IsFeatureEditable?


Whoops - let me find the test case that demonstrated the issue... in the middle of a few things. I'll report back hopefully soon though.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 159](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=807&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


Meme la 0.3.8 n'est plus a jours pour ces 2 sources, il te faut passer par la mise a jour des sources dans les parametres, normallement ca doit te dire X fichiers mit a jours.

Sinon reste la MAJ bourrin avec les fichiers de la beta et le blocnote.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 159](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=807&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:08


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 160](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=809&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


I have just a customer file from an Epson MFP, but I suppose I could modify a testfile accordingly.

I thought about padding the image outside of the stream, but to be honest just adding black-pixels for CCITT was easier than doing something equivalent for all components/color models outside of the stream :-)


Definitely! How about:

```
- Steps to reproduce the bug
- Output from running the test
```


So.. I'm looking at this part in the 68KUM: (8-6)

![screen shot 2016-02-18 at 19 17 40](https://cloud.githubusercontent.com/assets/350822/13153464/8ee425bc-d674-11e5-942c-b3339482a476.png)

The way I understand this is that when register and the condition is false it takes 4 cycles but otherwise 6 cycles in case it's true. Right?


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 160](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=809&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:01:06


Yes, Syncplicity wrote the C# port. You can find their fork here: https://github.com/syncp/liferay-nativity


This is reverted in another commit, could  you squash this change instead?


Public variables so that you can actually edit them in the editor


Т.е. компакней уже нельязя? :-)


I'm learning that ```rails g controller``` is relatively crappy. This is what I get for blindly following instructions in the checkpoints.


pls add the hooks add the end of the struct to not break ABI.


update_travis_toolchain (lefticus) - Win64-Windows-10-VisualStudio-12: OK (388 of 389 tests passed, 0 test warnings)

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


dfsadf


should this be removed because it is never used?


Musashi seems to set *all* dn operations to 4 cycles (except ```st``` but that just seems to be a bug) but that seems wrong to me.


Oops, thanks! cc @mattrajca


## hfhjf


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1313.b96581e8a32ee59c019081a2eac962370e5757bf](http://build.palaso.org/viewLog.html?buildId=69359&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:12


Доделала согласно замечаниям


@ryneeverett indeed


TeamCity Glyssen :: Glyssen-master Upload Test [Build *.*.1.b96581e8a32ee59c019081a2eac962370e5757bf](http://build.palaso.org/viewLog.html?buildId=69361&buildTypeId=Glyssen_GlyssenMasterUploadTest) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:55


@raphaelpaiva, acho que todos estão recebendo, sim, pelo Slack. A "culpa" foi só um modo de falar mesmo. Não achei o comentário arrogante, não. Ao meu ver, foi super válido!


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


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 161](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=810&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


Like using Extent.Text, this incorrectly relies on the input text.  A user could have written ${null} and your check fails.  Instead, you should use the VariablePath property on the variable expression.  Also, prefer as w/ null check for better performance.


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, line 32.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -29,7 +29,6 @@
 """.split('\n')

-
 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
     (good_file1,good_file2,),
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, line 35.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -32,5 +32,5 @@

 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
-    (good_file1,good_file2,),
+    (good_file1, good_file2,),
     (bad_file1, bad_file2,))
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, lines 35 to 36.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -32,5 +32,5 @@

 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
-    (good_file1,good_file2,),
-    (bad_file1, bad_file2,))+    (good_file1, good_file2,),
+    (bad_file1, bad_file2,))
```


**Line contains following spacing inconsistencies:
- No newline at EOF.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, line 36.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -33,4 +33,4 @@
 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
     (good_file1,good_file2,),
-    (bad_file1, bad_file2,))+    (bad_file1, bad_file2,))
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, line 34.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -31,5 +31,5 @@

 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
-    (good_file1,good_file2,),
+    (good_file1, good_file2,),
     (bad_file1, bad_file2,))
```


Bonjour @BenjaminBrahmi accepté


Perhaps this is just cosmetic, but setting the file name to 2016_02_17_01_world.sql looks like "back-dating" the file to yesterday (2016-02-17) since today's date is 2016-02-18. I don't know if this matters much for the project, it is just an observation on my part. :)


they are not doubles they are int arrays, so this test doesn't verify the values.


Perhaps this is just cosmetic, but setting the file name to 2016_02_17_02_world.sql looks like "back-dating" the file to yesterday (2016-02-17) since today's date is 2016-02-18. I don't know if this matters much for the project, it is just an observation on my part. :)


I feel like some bugs reports with come in with the default values from the template. I like `< your node version >` for formatting. If the user does not fill out any of the values, we could request more information.


The following doc(s) have been deployed to the [1322varnis](http://1322varnis-static-docs.pantheon.io/docs) Multidev environment:
- [source/docs/assets/images/varnish.png](https://github.com/pantheon-systems/documentation/commit/2b02cdd532c4bdbbc24316013fec372adb76f157/source/docs/assets/images/varnish.png)


The following doc(s) have been deployed to the [1322varnis](http://1322varnis-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/varnish](http://1322varnis-static-docs.pantheon.io/docs/articles/sites/varnish)
- [source/docs/assets/images/varnish.png](https://github.com/pantheon-systems/documentation/commit/be9cafaa4f1b810cc4f2667798576ffab3c34be9/source/docs/assets/images/varnish.png)


travis tests should catch this sort of thing, not sure why the build didn't get run on the PR that broke this


Some javadoc explaining the class' purpose would help understanding.
Why does this class have a '2' at the name end?


I believe the startup command of the md5%$ is incorrect.


Great. I'll go ahead and update it with `<your node version>` etc.


done


Like many things, it's important to learn to do something with training wheels before you take them off. :) It's definitely worthwhile to learn them.


no default information for any of these?


Nit - Output from running the _example_ test. It will prevent bug reporters from posting their output from their actual test.


fuck thought there had been one commit today p:


Alors j'ai été dans Système > Paramêtre > Addons > Video > VStream > Config > Outils > j'ai clique ou tu ma dis mis a jour source, meme faire des clear cache au cas ou et rien ne change pour les deux XD c'est pas logique :p mais bon je vais tenté ce que tu me dis la un copié collé de la beta sur la normal


`_time_t[0]` is never used.


Typo in javadoc :)


Actually, rethinking this i'm just going to go with a pair of back ticks. It's less typing for those opening issues and still provides a hint at the formatting we want


GreaterEqual since LessThan in the else branch?


Does "shifterState == ShifterState.high || shifterState == ShifterState.low" mean "isDriving()"?

The code does not explain why we checking these things, nor what happens if we don't check these things, just that we do before changing speeds.
Does it mean "change the speeds only if we are in driving", or something else?


(the reason for this change was so there was one source of truth of what deps get included for knockout so tests could assume proper deps)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1314.3c19cad9dcb2a5f230cc791d4e7e06810fa51a1f](http://build.palaso.org/viewLog.html?buildId=69364&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:06


These 4 commits are the result of a botched splitting of one big commit. They are all one-off, but the trunk is in a good state and a few bugs have been fixed.


Can we put this back to the shorter `Admins` instead of the long word-wrapping `Admin Access Management`?


License-Fixes-For-8.5 (jasondegraw) - x86_64-MacOS-10.9-clang: OK (2093 of 2093 tests passed, 3 test warnings)

 * 3 tests had: AUD diffs.

![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)


Duplicated logic with prior method.


@wightwulf1944 Aw, chucks :blush:


Right, so after thinking a bit about this, 'ordering' is actually wrong here. Because we're filtering on the test state, all the series returned have the same test state, the one we asked for. ordering on 'test_state' doesn't actually do anything useful then and worse, the ordering isn't really well defined.

I'll leave the default ordering (-last_updated) shine through with --test-state.

For ordering we just need a separate argument so the user can override it.


no need for a nil check


Doh!  What went wrong?


@eljobe this actually caused some tests to fail. I've gone ahead and removed this so we don't have a "last one wins" type situation here. Basically `useAnt` should take precedence I think.

https://github.com/gradle/gradle/commit/e8e61afb956c3e18174f9b66dcc8351ce05c90a5


I asked why you are setting them in the script.


sam comment about default column - is this just a template table we use for all endpoints, and thus want it to be the same?


these single backticks should go


Вопросы:
* Закоммичены результаты сборки проекта
* Сервер не реагирует на передаваемые параметры командной строки
* Сервер не взаимодействует с другими клиентами /серверами, сообщения не доходят клиентам (bav-С++, mva-python):
```
Server was started on localhost/127.0.0.1:1337 with 1337 threads.
New connection with client address /127.0.0.1:33480 established.
Received message from /127.0.0.1:33480:
kel - 26
```
* Скорее всего после исправления предыдущих недочётов будет падать с _WritePendingException_ у клиентов при нагрузочном тестировании


The **t**ype


sure, fixed


Electric-Service-Refactor (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2086 of 2090 tests passed, 546 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SHD diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


TeamCity Glyssen :: Glyssen-master Upload Test [Build *.*.2.3c19cad9dcb2a5f230cc791d4e7e06810fa51a1f](http://build.palaso.org/viewLog.html?buildId=69363&buildTypeId=Glyssen_GlyssenMasterUploadTest) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:03:38


Cause... defaults. When you add a rigidbody script it does have defaults. Instead of everytime you add the character controller to a character, everything will be 0


:dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer: :dancer:


actually technically it's all variables. if we were to ever add a variable that's not related to one of the events, it would be put there


**HTML element identifier** where


What's up with the 'D'?


The **HTML element identifier**,


-vvvvv output...?

On Thu, Feb 18, 2016 at 09:48:41AM -0800, Bryan Steele wrote:
>    This commit actually breaks Google IMAP for me..
>
>    brynet: unexpected data: 6 OK Success
>    brynet: fetching error. aborted
>    brynet: 0 messages processed in 2.226 seconds
>
>    --
>    Reply to this email directly or [1]view it on GitHub.
>
>    Reverse link: [2]unknown
>
> References
>
>    Visible links
>    1. https://github.com/nicm/fdm/commit/68a95b42701d93fa99ec96876720dfe0f0e07537#commitcomment-16182938
>    2. https://github.com/nicm/fdm/commit/68a95b42701d93fa99ec96876720dfe0f0e07537#commitcomment-16182938


enums are "constants" and Java constant convention is all uppercase.  e.g. LOW, HIGH...
Usually better to have enum as own class, unless only ever used here.


Im not sure what that sentence means


reword pls


this isn't rendered right


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


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


If you activated OAuth API on WP panel turn off it.


just to remind - it can cause problems with incorrect timezone, in case of troubles it is a first place, which should be adjusted.


FYI This update breaks a [Spring Boot test](https://github.com/spring-projects/spring-boot/blob/3352e60631c4bec64cd10e2bf495ec7fcd310427/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/mobile/DeviceResolverAutoConfigurationTests.java#L93) where we use a `MockHttpServletRequest` but don't explicitly set a method.

`getRequestMethod` returns `null`, this one exits then `RequestMappingInfo.getMatchingCondition` returns early also with `null`.

We can update the test to set the method but I thought it was worth raising.


Why not something from version.h?


JUnit will assemble a message like this.  The message we add here is related to "if this test failed, what brief message will help the person analyzing as to why"?


Reworded somewhat


yeah this is badly formatted, if you need help, contact us


as above


Would using assertInstanceVariablesNotNull work?


[answer to HW lecture1.txt](https://github.com/TB4Pb/Lecture1/files/136746/answer.to.HW.lecture1.txt)


this probably needs an else ..() for things like the syndicate hardsuit and chrono suit. Need to test that.


@JohanTan Please review when you can. I took the approach of making the `slack_eng_data` condition an `elif` so that `slack` takes priority. Again, this isn't the most scalable approach but for the time being will allow us to tag different slack channels. Let me know what you think.


So setting publics in script adds them as defaults, but are overwritten by the fields in actual unity?


The following doc(s) have been deployed to the [1322varnis](http://1322varnis-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/varnish](http://1322varnis-static-docs.pantheon.io/docs/articles/sites/varnish)
- [source/docs/assets/images/varnish.png](https://github.com/pantheon-systems/documentation/commit/05c1a1d828950e100c1554823d0ead28fda258b5/source/docs/assets/images/varnish.png)


I have no idea. I must have accidentally left it there when copying


Yes


It is better to communicate with the guy who designs images about Spanish playing cards. As a result, it will not have conflict between card suit and value with actual images' names.


For the sake of having a simple SDK I like that the persist option is exposed. Do we ever see the need to add anymore flags? If so, would it make sense to make this a flag param instead?


Is it possible to test without going through SQL? For example, in TestAllocators.java, we have a bunch of tests doing verifications directly in vectors.


This reads a bit awkward with the line break here. If we really, really want to stay under 80 chars, maybe breaking after the comma would be better. Some other instances of that here.


@lt "You may, as you state on the PR, find it weird that people might rely on a predictable stream of pseudo-random numbers, but they do have their uses."

I'd like to know a use case in which someone would rely on that. Just out of curiosity.


@langsdlc Nice change dude!


TIL. Cool cool then it's all fine.
The other points still stand however


TeamCity Glyssen :: Glyssen-master Upload Test [Build *.*.3.ffe9be3212253f60b6c188f4c4483f738beb16aa](http://build.palaso.org/viewLog.html?buildId=69366&buildTypeId=Glyssen_GlyssenMasterUploadTest) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:03:28


Après une mise a jour manuel depuis la beta à la version officiel c'est tout bon Filmilimit et Kepliz remarche ;)
j'ai pas fait attention si tu l'avais celui la ou si tu pouvais l'utiliser
http://streamay.com/
;)

Merci pour l'aide !


See above that for other variables


Given a target *component* and


adds the **component**


Awww, but it's so lame too make another commit just cause I changed a value and two positions


format this like
  ALTER TABLE xxx
    ADD COLUMN ...
    ADD COLUMN ...
    ADD UNIQUE ...


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


**cdf.dashboard.parameter** ? vat is this?


Totally irrelevant to this PR comment: Why is makeGUID in the core respoke library and not as a separate include?


To make sure I understand, previously if source_permissions was `use*` on windows, we required owner/group/mode to **all** be specified, so there's no point in copying it from the metadata. If `source_permissions` was ignore on windows, then again no point in copying from metadata. And the `devfail` below, ensures we never try to copy. :+1:


Need to follow the pattern of the other ones to error when not found.
Replace this line with:
`final String propertyValue = getStringPropertyValue(propertyName, props);`
and process from there.


Looks like other xxx_map has system_id.  This one doesn't.  Well, we don't have a layer that requires it.  Will leave it up to you.


This is a new warning (relative to what was present before)?


Nullable Types in Csharp

using System;

namespace IntroductionToCsharp
{
    class Program
    {
        static void Main()
        {

            bool? AreYouMajor = true;

            if (AreYouMajor == true)
            {

                Console.WriteLine("User is Major");
            }

            else if (AreYouMajor == false)
            {
                Console.WriteLine("User is not Major");

            }

        }
    }
}


{string}


Обсудили голосом - сделаешь тайпдеф отдельный для массивов?


**add-in** for consistency


Pintér


@profyaffle Any suggestions? Thanks..


> I'd like to know a use case in which someone would rely on that. Just out of curiosity.

Maze generation is one example: if you want "levels" of maze, a simple way to do that is to seed the RNG with a particular number that generates a subjectively "easy" or "difficult" maze. The seeds the correspond to the level numbers, reducing storage of the level information by orders of magnitude.


true -> `true`


we're always quoting the metadata value so I think we can use `"1"` in all cases.


Il est en TODO list ^^
https://github.com/LordVenom/venom-xbmc-addons-beta/issues/86


I added the stacks context menu to app.js, since I don't think it's worth creating a new javascript just for this tiny file...
But I can add a new js file if it's not ok...


learning how to use git :)


@mikesaurio porqué dejar código comentado? si ya no sirve hay que eliminarlo :) al cabos git sabe como quedó hasta ese momento


array using the data


Shall we inhibit the removal of non-empty folders?


In case your code needs a reproducable sequence of pseudo-random numbers, why not just hash your seed? If you need more numbers, hash the resulting hash (and so on). They will never change.

Relying on the predictability of a pseudo-random number generator looks like bad style for me.


If a stylesheet violates a rule marked `2`, does it fail to compile entirely? Some of the items marked `2` here are useful for troubleshooting (I'm looking at color-keywords especially), and I wouldn't want them to completely break the file while in development.


needs quotes


align arguments


typo.


This should never be accessed anywhere else, just in this class. To actually change gears, commands should call the method in PowertrainSubsystem corresponding to the desired gear.


Maybe need a newline before the start of the code block? The HTML rendering on the project website isn't doing code highlighting correctly and showing the lowercase word "java" in the visible content, too.


How would I return a message? Also, how would I handle that message in `run()`?


the href paths may have to be altered a bit


Yup.


Fix #444


Fix spacing


Well
you dont have to
Just check in the files/Lines that you actually meant to change


#[](url)


you don't need to handle anything, read the documentation at http://coala.readthedocs.org/en/latest/API/coalib.bears.html#coalib.bears.Bear.Bear.check_prerequisites


:-)

Qué tal tu vida?

Saludos

On Thu, 18 Feb 2016 03:42:41
-0800, AlejandroP wrote:

> Que tiempos...
>
> --
> Reply to this
email directly or view it on GitHub [1].

Links:
------
[1]
https://github.com/dam1d/Repositorio_01/commit/0bbd0caf602b21bc4442f673b1a49b73674c1a06#commitcomment-16173139


> In case your code needs a reproducable sequence of pseudo-random numbers, why not just hash your seed? If you need more numbers, hash the resulting hash (and so on). They will never change, not even when changing the environment/language/whatsoever.

Wouldn't you then just write your own hash-based PRNG?
PRNGS are standardized, just as hash functions are. Why would the latter be less prone to typos than the former?


How is this? It passes the tests fine, and uses what you said:
`protected int[] getIntArrayPropertyValue(String propertyName,
            Properties props) {

        final String propertyValue =
                getStringPropertyValue(propertyName, props);
        String rawValues = propertyValue;

        rawValues = rawValues.replace("[", "");
        rawValues = rawValues.replace("]", "");
        final String[] splitValues = rawValues.split(",");

        int[] returnValues = new int[splitValues.length];
        for (int i = 0; i < splitValues.length; i++) {
            returnValues[i] = Integer.parseInt(splitValues[i].trim());
        }

        return returnValues;
    }`


Yep, this is now good after the next commit to fix this up.

Thanks very much for doing this, its greatly appreciated.


It doesn't, technically. Why do we prefer quotes?


We don't want to normalize the names of columns, they need to be exactly the same so that we can use them in lmod_proxy and pylmod


Yes.


Just checked the website for the geolint validation ajax, it no longer appears. Far be it for me to overstep my bounds, but it was a great idea. As a side note, thanks for this repo/site it's made my life much easier. You're awesome!


No, the interface stays as it is. It's just that there can be additional arguments in subclasses.


It can do both a list of players or a interface to do the discovery on. The
discovery works well as long as you have them on the same net and broadcast
on the right interface.

/R

On Thu, Feb 18, 2016, 17:58 Alexander Fortin <notifications@github.com>
wrote:

> Hi, is there any particular reason to expect a comma separated list of
> hosts instead of a list of them?
> Also, I'd like to patch this to have the possibility to also include
> players from config instead of discovering them, autodiscovery proved to be
> not reliable for me
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/balloob/home-assistant/commit/543190dfb0b467127841b5960d2c1deaf9eff9ea#commitcomment-16181581>
> .
>


Sorry, what's the difference between flush and finish? Is it reserved for future expansion?


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


Looks like order_scores needs to be called on a Result object...


Thanks for putting this in. I got a bit lost when trying to do this earlier this week. Out of curiosity, where did you find the "correct" values for these?


Yeah, this is something I shared with Jason when we were working together this week: I don't feel I've learned new knowledge of OO programming with this attempt (will be writing about that soon).

In some places, objects are clear: a vehicle, a customer, etc.

Others, the idea of an object becomes abstract. And I have nowhere near the experience (or intelligence, too?) to grasp those yet.

In your equation, I don't see objects. But you're suggesting I should see them... so I just make wild guesses... and I'm not sure that's helping me. Perhaps an object could be the calculation as a whole? The individual operations (here: addition and multiplication)?


...order_scores is not being called on a Result object here.


I did mean to change jumpForce


Replied via email.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1316.369be88128b6289eff32a4986e2ab8a13d86345d](http://build.palaso.org/viewLog.html?buildId=69368&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:21


Ah, now I understand the problem! (Sorry, it's hard to debug when it's working fine for me!)

**TLDR:** Change the locateLib() function inside copy-libs to be:

locateLib() {
  # First, search $LD_LIBRARY_PATH
  declare -a dir_array
  IFS=':' read -ra dir_array <<< "${LD_LIBRARY_PATH}"
  for d in "${dir_array[@]}"; do
    if [ -e "$d/$1" ]; then
      echo "$d/$1"
      return
    fi
  done
  # Second, search library cache
  local path="$(ldconfig -p | grep "$1" | awk '{print $4}')"
  [ "$path" ] || echo "$1 not found." >&2
  echo "$path"
}

**Explanation:**
So it turns out that `ldconfig -p` ignores `$LD_LIBRARY_PATH` and only searches your built-in library cache. You could update this cache to include the Qt "lib" directory like this:

sudo ldconfig /usr/local/qt5/lib

but it's better to fix the locateLib() function so that it does what we expect if $LD_LIBRARY_PATH is specified.

Basically, `$LD_LIBRARY_PATH` does not work like `$PATH`. With `$PATH`, if I specify a binary name without a path then the shell searches the current directory and then `$PATH`, like this:

echo $PATH # shows which directories will be searched for binaries
ls # this runs "/bin/ls"
PATH="" ls # this returns error message "ls: not found" because PATH is blank

Whereas `$LD_LIBRARY_PATH` is actually empty by default. Instead, the location of each library is stored in a cache, and setting "`$LD_LIBRARY_PATH`" simply specifies which directories to look in *before* the cache is checked, like this:

mscore # run MuseScore and search library cache for dependencies
LD_LIBRARY_PATH="/usr/local/qt5/lib" mscore # run MuseScore and search "/usr/local/qt5/lib" and then the cache


TeamCity Glyssen :: Glyssen-master Upload Test [Build *.*.4.369be88128b6289eff32a4986e2ab8a13d86345d](http://build.palaso.org/viewLog.html?buildId=69370&buildTypeId=Glyssen_GlyssenMasterUploadTest) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:47


a.) I didn't know this was a thing, but
b.) I'd like to be able to use it in development, so this follows on my question at the bottom.


i think this is just a test leftover because there's git version checking now so this is completely pointless :V


Once you have decided your MVP - add the user stories for it to your readme.


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


Change browsersync mention with parameter.coffee
https://github.com/dummy-team/dummy/commit/3e0f09d11cb84ba23c4304f6029518a94df8898e#diff-04c6e90faac2675aa89e2176d2eec7d8R56


Right... flush() is supposed to be able to be called frequently, finish()
is just once at the end of a block - giving signal that we'd really like to
get on with other things.

On Thu, Feb 18, 2016 at 10:55 AM Vijay Pai <notifications@github.com> wrote:

> Sorry, what's the difference between flush and finish? Is it reserved for
> future expansion?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ctiller/grpc/commit/311445fd32956e9383823400c82ce3fcd71b2b31#commitcomment-16184685>
> .
>


Not sure what all these empty lines are for. Need to be removed


up to 131ffec42dbcf2039


I would probably make sure this is url path encoded.


This doesn't fix the case where `len(student.grades) == aidx`.


Same here


Should this be a warning? Not so relevant anymore, but I've used duplicate properties for fallback behavior in the past and would like to have that available if necessary.


Comment made on master branch


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.???.{build.vcs.number.1}](http://build.palaso.org/viewLog.html?buildId=69373&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:04


lint 추가하기


How does a class which implements the interface "know" about the additional args...? Those are part of the contract dont they?


Be sure to add in your git workflow

Also - if you are starting  a project from a previous project - always good to clear the commit history. So that this project has it's own starting point


Yes that's a start.
I think also need to check is not a 0 length array... as in property exists but no values.
I suggest a new array assert method for second part.


이 구문 때문에 해당 함수가 로그아웃에 종속적이다.
고민 필요함


hello everyone :)


This MVP is a little ambitious.  Start with something you can accomplish by end of day today - then add on features.


The signature change was deliberate.  The status and length passed are
those that the container actually committed and may differ from a request
object that can be changed after commit by an application.

I know it was a breaking API change, but it was for a reason.

On 18 February 2016 at 17:16, Anders Storsveen <notifications@github.com>
wrote:

> "This change did not lose the response object"
>
> I'm not referring to *this* change. I'm referring to the fact that this
> change *reverts* the signature of the log method that was changed here:
> 34a8da2
> <https://github.com/eclipse/jetty.project/commit/34a8da2ba26a806413f06ab18fdfad535de45cfa>
>
> de45cfa also changed the signature of the logExtended() method on line 184
> in this file. I'm asking if this change should also be reverted so that it
> becomes logExtended(request, response, buf); again.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eclipse/jetty.project/commit/9eb2cb4c0f5f9131f7f855836817c88519da0cfe#commitcomment-16180052>
> .
>

--
Greg Wilkins <gregw@webtide.com> CTO http://webtide.com


HeadNav보다 상위에 있는게 낫지 않을까?
여기에 두게 된 이유는 Container가 변경되는경우 HeadNav가 유지될 수 있을거라 판단하여
HeadNav에 이벤트를 두었음


상위 컴포넌트에서 처리하도록 변경


Worth noting that that third `true` parameter allows you to specify whether the file is public or private. Tried it out while trying to figure out my mistake and ended up removing again in a later commit because it didn't make a difference.


필요없는 부분 제거


Where does the groupID get passed?  Maybe I'm blind.


Better.
No need to do this:
` String rawValues = propertyValue;`
either use rawValues or use propertyValue...


It looks good to me.


so this is an almost perfect commit with a nice message, a minimal change, atomic and revertable :) There's only one thing I'd like to have improved, which is the tense of the shortlog. Concretely that means instead of `Removed`, rather write `Remove` :)


url뒤에 버전정보가 붙는 경우를 파싱하기 위해 추가
정규식 리팩토링하여 해결해야됨


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


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 161](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=810&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:36:59


config로 이동


It is not. And it shouldn't be bumped that often.
It triggers the clientside update notification.


@koraa You're right, you would.
But if PRNGs are standardized, shouldn't we get PHPs PRNG to that standard?
Ok, well, we should keep providing the old one, that's right.


Adding another parameter to the interface without breaking anything isn't possible anyway.

```
`$ php -r 'interface x { function a($a = null); } class y implements x { function a() {} }'`
`PHP Fatal error:  Declaration of y::a() must be compatible with x::a($a = NULL) in Command line code on line 1`


TeamCity Glyssen :: Glyssen-master Upload Test [Build *.*.5.cde4f9003ff626b392ff6f7464e8e8e3c94bd0fa](http://build.palaso.org/viewLog.html?buildId=69372&buildTypeId=Glyssen_GlyssenMasterUploadTest) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:27


This change breaks building on windows for me, I get:

common.mkdir M:\data\boost\boost\tools\auto_index\build\M:\data
The filename, directory name, or volume label syntax is incorrect.

        if not exist "M:\data\boost\boost\tools\auto_index\build\M:\data\\" mkdir "M:\data\boost\boost\tools\auto_index\build\M:\data"

...failed common.mkdir M:\data\boost\boost\tools\auto_index\build\M:\data...
...skipped M:\data\boost\boost\tools\auto_index\build\M:\data\boost for lack of M:\data\boost\boost\tools\auto_index\build\M:\data...
...skipped M:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost for lack of M:\data\boost\boost\tools\auto_index\build\M:\data\boost...
...skipped M:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost\dist for lack of M:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost...

The line:

DIST_DIR = [ path.root $(DIST_DIR) [ path.pwd ] ] ;

Changes DIST_DIR from "M:/data/boost/boost/dist" to "/M:/data/boost/boost/tools/auto_index/build/M:/data/boost/boost/dist".


This can only be done if shopizer is configured to run with shopizer-search-0.04


I tried that, but you can't use the trim methods on a static String

On Thu, Feb 18, 2016 at 1:01 PM Jeff Jensen <notifications@github.com>
wrote:

> Better.
> No need to do this:
> String rawValues = propertyValue;
> either use rawValues or use propertyValue...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/frc-2175/2016RobotCode/commit/de5a54b815afac774ca2e76ef0719c2d948464d8#commitcomment-16184850>
> .
>


Ooh, I like this.


Hi All!

Just want to try my hand at learning how to code.


Looks nice!


I don't see where the groupID is set here either.


Will probably need a ton of help!


Suggestion: `translations.has_key?`


Nit: we begin compiler messages with lower-case letters :)


den hier kann man problemlos mergen


Likewise here and below


suggestion: the asset filename or URL


Cynthia Lin


Nit: I'd probably make this one sentence, like:

"no candidates by the name of `{}` found in your project; maybe you misspelled the name or forgot to import an external crate?"


Can we `allow-element-with-attribute`? I think it helps with readability. See https://github.com/sasstools/sass-lint/blob/develop/docs/rules/no-qualifying-elements.md#allow-element-with-attribute


Do we want to instead call `query.uniqueResult()` and add some constraints to the SampleAdditionalInfo table?


suggestion: hash algorithm (defaults to sha1)


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

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


suggestion: list of URLs from where the asset can be fetched (optional)


can you squash this message and the final one below, so it says something like:

"you may be referring to one of the following items, which would then need to be imported with the `use` keyword:"

(though it feels like that sentence could be made more terse)


thanks. gotta love when you return to a project after 3/4 of a year and hit a bug less than 24hr old :)


Uff, a dependency on persistent. Normally I know about the probs regarding this, but maybe I've learned that after I did this commit.
However, that was removed by @saily in a subsequent commit: https://github.com/collective/pas.plugins.ldap/commit/ecda41dfba168d41f5db766c303fd356615bdadd


Isn't that from https://github.com/sul-dlss/workflow-archiver-job, not `dor-services-app`?


@pke I don't believe it did.


I would actually be interested in further discussion on this. I like the idea of being consistent, though I understand getting everyone on the same page for it could be challenging.


Added name


Since `self.basename` is used across methods, it may be a good idea, for clarity, to add it to the class constructor.


at least you can remember :-)


yep! agree with the change, just the compress block should have also moved
inside the include see https://github.com/dimagi/commcare-hq/pull/10364

On Thu, Feb 18, 2016 at 1:32 PM, Ben <notifications@github.com> wrote:

> (the reason for this change was so there was one source of truth of what
> deps get included for knockout so tests could assume proper deps)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcare-hq/commit/732897a44bc5e57e08515d79e5a9a4adac3d2936#commitcomment-16184111>
> .
>


@NW7US hey, awesome! Want to patch the repo, adding yourself? :) http://www.pm.org/faq/hosting_faq.html#new


using System;

namespace IntroductionToCsharp
{
    class Program
    {
        static void Main()
        {

            int? TicketsOnSale = null;
            int AvailableTickets;
            if (TicketsOnSale == null)
            {
                AvailableTickets = 0;
            }

            else
            {
                AvailableTickets = (int)TicketsOnSale.Value;
            }

            Console.WriteLine("AvailableTickets {0}", AvailableTickets);

        }
    }
}


Ok, I understand on the auto discovery, thanks for the clarification.
https://github.com/balloob/home-assistant/pull/1319 should address my real concern, i.e. the need to provide a comma list separated of hosts in the config


why not just write:

```
return _unlikely_(need != 0 && size > (SIZE_MAX / need));
```

No need for an "if" here, if all we want to do is return a boolean anyway.


Hi Victor, what was necessary to complete this branch for JENKINS-31787 ? I am very interested to get rich-text-plugin support in jobdsl (I'm shocked it's been released for 3 years with no one doing the work previously, thanks for picking it up)


Respected madam,
My name is Abhilash chandran and I'm doing a project on SPIN model checker.I'm searching for a translator to promela.If you share your views about pmGen,then it'll be very helpful for me.Kindly I'm requesting you to send an email to me with the details about installation and use of pmGen(email ID: abhilashchandran333@gmail.com). Thank you


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2149 of 2149 tests passed, 550 test warnings)

 * 550 tests had: AUD diffs.

![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)


I just looked at https://searchworks.stanford.edu and picked a format.

These fixture objects don't really have a "correct" format given that they have titles like "File Item4 - Deathstar Bluprints" (i.e. totally made up).


Should I put that method in TestBase?


You've put this code into the layout and into several views as well. This will result that the messages will display multiple times.

If you really need to display it in multiple templates, consider to extract it into a template


looks good otherwise.


That is good news. You tested with Ubuntu 64 bit? Which version if i may ask? Then i can add it to the documentation :)


One way would be representing battery states with matching icons for them.
Battery states can be represented as enum(new in python) or class objects.


If you look at your console log, you'll see that the characters are being added exponentially. After one loop, it's two, then the next time it adds itself to itself, then it's four then 8 then 16, etc. So, try adding the single character to char two times per loop instead of adding itself to itself.


please keep the indentation consistent.


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


setting lastAccess **after** successful call to delegate.clearWarnings(); is safer because its doc says:
...throws SQLException if a database access error occurs or this method is called on a closed connection

I suspect lastAccess was set somewhere else in close earlier in case of reported 'rare' escape


I'm not convinced that disabling this is the right thing to do as CVS is a slow SCM, so jobs set to poll more frequently than the time it takes to complete a checkout will continually schedule new builds.


How about `presence: true` ? :)


The lines below can be simplified to:

```python
# Some fields will hide relationships, e.g. HideIfRegistration
# ignore related_count for these fields
hidden = fetched_field and fetched_field.get_attribute(value) is None
```

It would be good to add those code comments too; this logic isn't obvious.


Change the order here, because the printf inside SOL_WRN() may also set the errno


If only he would have updated the dependencies in our projects to pas.plugins.ldap to something else than 1.3.0, which contained this nugget :-)


Actually, upon closer inspection, it is. I thought it was testing the generic behavior of a failed `Task` in a response stream (which is now tested in `QuasarResponseSpec`), but it isn't, it's testing that a failure in the middle of the upload stream results in a 500. Though I'm not sure why this needs to run a server to test, I'll see if that bit can be removed.


The contents of `public/system` folder should be added to gitignore. Please don't push your uploaded files in development mode :(


yes, switch is more explicit.


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


we've talked about this, why are you using `id`s?


code indentation


@gangstertthug927 Could you please elaborate what the problem is and what a solution would look like? Thanks!


code indentation in the whole file.

Please check your spaces/tab settings in your editors.


Given the cron job and the other project, I would pull it out of the dor-services-app, unless it is clear that certain app actions require a synchronous workflow-archive in addition to the async background process run by the cron.


Missed this before, but we need spaces, not tabs, for this class. At any rate, best practice is to use one or the other, and not mix them in your file. See also [Line 40-41](https://github.com/Mumbleskates/portfolio/commit/3493da402579c10565593c2203bef98a7b0f15da#diff-e3916365b23480aab1be02e1cf80568eR40-41)


972154-NewSimpleChilledCeiling (RKStrand) - 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)


My thinking was it is in the `server` package and provides a means to control the server. Though I think what I'll do is split it into two services, an `info` service and a control service, the latter of which I'm fine to rename if we have a better one.


It should be set on the object prior to calling join, with using the custom init method or setting it manually


To Do:

-  span color in SDLCnotes should be changed to BLUE -- red on pink background hard to read.
> BLUE didn't look good -- bold RED added id="mySpan" to next update
- look at moving span attribute from SDLCnotes to myStyle.css
> done plus working on adding link to get back to main page

- Update index.html to use HTML5 formatting -- header, span, footer, etc.
-  can I link to footer so it is consistent across all my pages? Is this a good idea?
>  JSP or other packages


I think this is breaking my test suite.
I've updated recently and I'm getting:

```
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x10 pc=0x54d992]

goroutine 1 [running]:
github.com/goamz/goamz/s3/s3test.NewServer(0x0, 0xc20802c018, 0x0, 0x0)
	/home/nuno/go-workspace/src/github.com/goamz/goamz/s3/s3test/server.go:103 +0x42
```

I'm initializing the test server this way: `server, err = s3test.NewServer(nil)`
Like suggested in the docs:
https://github.com/goamz/goamz/blob/7c9a0dfe202af99a0a99ba82cf094af8f5be51c7/s3/s3test/server.go#L45

Should I raise an issue?


I don't know how your branch is set up, but by using gfgtdf's suggestion of placing preferences::load_hotkeys() to replace this deleted line, I was able to restore user hot-keys at the title screen.

It may be that preferences stuff besides hot-keys are still okay, I don't know.


LGTM


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


@jpfowler


Need spaces here, not tabs.


code indentation problems in the whole file


You're right. Some (all?) of this will likely be moved to core for the repl, but I think we should let the repl refactoring force that change.


The following doc(s) have been deployed to the [945httpho](http://945httpho-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal/configuring-settings-php](http://945httpho-static-docs.pantheon.io/docs/articles/drupal/configuring-settings-php)
- [docs/articles/drupal/drupal8](http://945httpho-static-docs.pantheon.io/docs/articles/drupal/drupal8)


this line could be better written as
xxxxx x


We are just step back to this version, because we couldn't find  a bug. But yes we will use just the user.


Они по-старым правилам, самые базовые библиотеки внизу. Хотя их можно переобсудить будет.

> On 18.2.2016, at 17.34, Dmitry Yunitsky <notifications@github.com> wrote:
>
> Группы инклюдов map-indexer-geometry. Я не уверен что их надо сортировать, сейчас проверил в других файлах - там тоже несортировано
>
> —
> Reply to this email directly or view it on GitHub.
>


months


This is pure internet gold. Don’t ever change, PHP team. We need some fun in our lifes.


github관련 라우터로 이동 (passport관련작업 전부 포함)


А не получиться. Я пытался через each провернуть (старый вариант) но он не
заработал

2016-02-18 13:38 GMT+03:00 Vlasov Roman <notifications@github.com>:

> Может вообще заюзать функцию?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aquaratixc/rip/commit/26ca68c34cd50a68f4eb1a472b15e5f5e8a946e5#commitcomment-16171894>
> .
>


This property is being overridden by `.card` class. Check the left align on it to get your descriptions justified. After you get that change made you are :+1:


필요없는 로그 제거 ( 필요한 경우 winston 대체)


i don't studied this method before(


Not sure how I feel about this one, forcing that extra line might affect readability (though if there's less nesting, it might not be as much a problem...) Thoughts from others?


우선 sequelize 마이그레이션은 나중에 고민하자


안쓰는 패키지 제거


this is great! sorry to be a stickler, but can you throw it in the rest of the relevant places? it should at least go in the `"member is new"` section getting called with `new_member: true` and then it should have those two cases for subscription too.


ah gotcha. makes sense


Yeah I considered it. The encoding method was added in this pr because I wanted to make sure the new stuffed was properly encoded, bit none of the previous URLs were being encoded. My thinking is that such a change should be a separate pr


Data Type Conversion in CSharp

using System;

namespace IntroductionToCsharp
{
    class Program
    {
        static void Main()
        {

            int i = 100;

            float f = i;

            Console.WriteLine(f);

        }
    }
}


Well, as long as it does not make any difference to the way the source works, I suppose we will just leave it as it is. Not a big deal, just an observation. I talked Kittnz into renaming the SQL file in his PR merge because he followed your numbers without checking the actual date today. :p


Looks great!


You've made good progress. You'll have the character number straight once you address the comment above. Then adding the proper spaces per line is similar, in that you start with a fixed number ( height - 1 will always give you the first line of spaces ) then subtract one from it each time through the loop.


I am going to murder you.


I understand that this supports the use case when an asset may be hosted at a network filesystem, and even so the user may benefit from caching?

What I mean is, if the URL scheme is *file*, the asset could be directly accessed, instead of going through `download._get_file()` and (possibly) being copied. The final decision on whether this is a feature or an anti-feature, depends on the environment configuration and the user's real goal, but I feel that, by default, we should not try to copy files hosted in "file" schemes, but rather return them directly from the original URL. I'd rather have the system admin setup an FS-Cache than depending on this.

Maybe I mistakenly read the idea/code, so please treat the above comment as "proofreading".


Is there potential for .editorconfig conflicts with this one if we're not careful?


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


:baby_bottle: :checkered_flag:


Oops, sorry, it's just I made a typo in this commit commentary (554f9c2 instead of 544f9c2),and I sometimes refer to it to know what was the last en.txt that I took in account, in order to make a diff with the latest en.txt available.
As I don't know how to edit a commit commentary already pushed, I commented it with the correct SHA ("fixed" is not the right word).


Duplication can be avoided by inheritance.


MODE_ALARMS_ONLY  cannot be resolved to a variable


Yes; one of the many reasons that Atom is garbage.


`if (ref in params) {}` is probably more appropriate here.