You have to add some implementation of payload encoding to your test scope.
For example `rhiot-cloudplatform-encoding-json`.

czw., 21.01.2016 o 15:14 użytkownik Greg. A. <notifications@github.com>
napisał:

> ok ,
>
> I have got this kind of error If I really test it :
>
> org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'ioTConnector' defined in io.rhiot.cloudplatform.runtime.spring.CloudPlatform: Unsatisfied dependency expressed through constructor argument with index 0 of type [io.rhiot.cloudplatform.encoding.spi.PayloadEncoding]: : No qualifying bean of type [io.rhiot.cloudplatform.encoding.spi.PayloadEncoding] found for dependency: expected at least 1 bean which qualifies as autowire candidate for this dependency. Dependency annotations: {}; nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type [io.rhiot.cloudplatform.encoding.spi.PayloadEncoding] found for dependency: expected at least 1 bean which qualifies as autowire candidate for this dependency. Dependency annotations: {}
>     at org.springframework.beans.factory.support.ConstructorResolver.createArgumentArray(ConstructorResolver.java:749) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.ConstructorResolver.instantiateUsingFactoryMethod(ConstructorResolver.java:464) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateUsingFactoryMethod(AbstractAutowireCapableBeanFactory.java:1123) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1018) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:510) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:482) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:306) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:230) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:302) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:197) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:772) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:839) ~[spring-context-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:538) ~[spring-context-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:764) ~[spring-boot-1.3.1.RELEASE.jar:1.3.1.RELEASE]
>     at org.springframework.boot.SpringApplication.doRun(SpringApplication.java:357) ~[spring-boot-1.3.1.RELEASE.jar:1.3.1.RELEASE]
>     at org.springframework.boot.SpringApplication.run(SpringApplication.java:305) ~[spring-boot-1.3.1.RELEASE.jar:1.3.1.RELEASE]
>     at io.rhiot.cloudplatform.runtime.spring.CloudPlatform.start(CloudPlatform.java:56) [classes/:na]
>     at io.rhiot.cloudplatform.runtime.spring.test.CloudPlatformTest.before(CloudPlatformTest.java:67) [classes/:na]
>     at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:1.8.0_45]
>     at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[na:1.8.0_45]
>     at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:1.8.0_45]
>     at java.lang.reflect.Method.invoke(Method.java:497) ~[na:1.8.0_45]
>     at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) [junit-4.12.jar:4.12]
>     at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) [junit-4.12.jar:4.12]
>     at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) [junit-4.12.jar:4.12]
>     at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) [junit-4.12.jar:4.12]
>     at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325) [junit-4.12.jar:4.12]
>     at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78) [junit-4.12.jar:4.12]
>     at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57) [junit-4.12.jar:4.12]
>     at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290) [junit-4.12.jar:4.12]
>     at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71) [junit-4.12.jar:4.12]
>     at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288) [junit-4.12.jar:4.12]
>     at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58) [junit-4.12.jar:4.12]
>     at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268) [junit-4.12.jar:4.12]
>     at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) [junit-4.12.jar:4.12]
>     at org.junit.runners.ParentRunner.run(ParentRunner.java:363) [junit-4.12.jar:4.12]
>     at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:86) [.cp/:na]
>     at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38) [.cp/:na]
>     at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:459) [.cp/:na]
>     at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:675) [.cp/:na]
>     at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:382) [.cp/:na]
>     at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:192) [.cp/:na]
> Caused by: org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type [io.rhiot.cloudplatform.encoding.spi.PayloadEncoding] found for dependency: expected at least 1 bean which qualifies as autowire candidate for this dependency. Dependency annotations: {}
>     at org.springframework.beans.factory.support.DefaultListableBeanFactory.raiseNoSuchBeanDefinitionException(DefaultListableBeanFactory.java:1373) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.DefaultListableBeanFactory.doResolveDependency(DefaultListableBeanFactory.java:1119) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.DefaultListableBeanFactory.resolveDependency(DefaultListableBeanFactory.java:1014) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.ConstructorResolver.resolveAutowiredArgument(ConstructorResolver.java:813) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     at org.springframework.beans.factory.support.ConstructorResolver.createArgumentArray(ConstructorResolver.java:741) ~[spring-beans-4.2.4.RELEASE.jar:4.2.4.RELEASE]
>     ... 41 common frames omitted
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rhiot/rhiot/commit/4dc125d268190a1704fe2f33ab3f16096750c21b#commitcomment-15586111>
> .
>
--
Henryk Konsek
http://about.me/hekonsek


Good use of `console.log`


The handles could only be changed post initialization, unless the resizable is destroyed and recreated.

Fixes: #3423


Hey,
Am pus rezolvările pe GIT. Te-ai uitat peste ele?


@teckjon DWMR. Nice work. Lean into it!


:cry:, apparently UTF-8 in a console is a ways off.


Hm, shouldn't this be 0.2.1, @vvarma ?


Yeah actually did you realize that we already support `opt:OPTION` syntax as of 2083f4856fdc4d9e26592658bcb358b1594a3565???? Is anything wrong with that?


Revert most of the following and add:
if (conf.conf_use_percentile > 0.0f)
    conf.conf_threshold = percentile(mesh->get_vertex_confidences(), conf.conf_percentile);


The plugin is BSD-licensed, so you're free to whatever ;)
Not sure why you'd base changes on a sor-of outdated fork though


There are two major changes: (1) Added reactions from GramNegative template so this template is a superset of GramNegative template. This resolved the problem where a gap fill could not produce all of the biomass compounds and ran for hours. Note that gap fill takes slightly longer with this template compared with GramNegative since the set of gap fill reactions is larger. (2) Updated 846 reactions with complexes and changed the type to "conditional". The complexes come from the probabilistic annotation data that includes additional data from KEGG. One minor change is to fix a typo in the BiomassCompounds file so it matches the corresponding GramNegative template file.


`str_starts_with`, `str_ends_with` in lib/base.php


Checkmarx scan results 21 Jan 2016 16:08 UTC +02:00

High
Medium
Low
Info

New
6
1
3
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

75cec2c6a1d6c9c6e9863e9f940a91bd0c8c3e21	@amitchen22


It occurs to me that there are weapons that are [swappable](https://elanthipedia.play.net/mediawiki/index.php/Weaponsmithing_products#Swappable) between attack types. For example, the throwing spike can go between LB and LE.

Does that mean attack type is actually a property of the skill being used?


That is a large PR for a trivial change...


Does this support ES6 out of the box?


mbed seems to be using a file under the same proprietary NXP license
https://github.com/mbedmicro/mbed/blob/master/libraries/mbed/targets/cmsis/TARGET_NXP/TARGET_LPC11XX_11CXX/LPC11xx.h


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/26149/testReport/) for this commit**


jQuery please!


Sphinx is so awesome!! Thanks, Matthias!


Hi Jun-san,

I'm also tweaking build scripts of OMake for windows these days. And found that you are doing similar thing.

Please give some advice. Which platform these PREFIX tweak is targeting? Which of editing .config or calling ./configure are you using to set PREFIX? I have been checking build script for Windows with PowerShell. Should I switch to Command prompt or Cygwin terminal?

By the way, my personal goal is to create msi installer for windows for my use.

Regards,
Nozomi Shibano


OMG THIS IS GETTING MESSED UP LOL


`disable`, nie `diasble`


Da, şi am văzut că nu prea seamănă, dar eu ştiu că sunt mai multe abordări corecte. Nu mă deranjează punctajul, asta ca să eliminăm orice suspiciune, dar chiar nu îmi dau seama de ce nu e bine cum am făcut eu...


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/26150/testReport/) for this commit**


This is interesting functionality but I have sort of been thinking of removing the Advanced search page altogether. It is very rarely used, and people should learn the textual search syntax. Obviously you disagree; can you say a few words why?


This commit makes the verifier complain for the following test:
`./goblint --sets solver slr3 tests/regression/04-mutex/35-trylock_rc.c`


On a side note, it would be better if you could keep like commits together in the same pull request (I'm looking at #197 and #198). I'm not sure what to change in your git/GitHub workflow. @rcuhljr and @kamarcum have more experience with this and might have suggestions.


The original code constructed a scala.math.BigDecimal regardless of whether the value returned was null.  That apparently worked in Scala before version 2.11, but resulted in an NPE afterwards.  Those lines in particular check to make sure the returned value wasn't null.  The wasNull is mostly there for primitive values, but I imagine my thinking was to use it here to keep things consistent with the _handleNull method.


Please change all `ExtensionRegistry` variable names to `extensionRegistry` (i.e., with a lowercase "e").


oh goodie, a new update! :+1:


why you always attach *.js.map file?

Are they just minified javascript file??


See https://dl.dropboxusercontent.com/u/16276273/pxtest.tar.gz
Check test_api.php to reproduce problem
getGrandTotal() doesnt return an incorrect value, the problem is that PayEx sums upp differently then Magento


PHP7 is so easy to change but need many things to add and replace on a big project :o


rerdgdfgd


Where do I find an option to allow empty password? Right now there is no user at all that I could select from that drop-down menu! Login in a tty is not possible either, so I cannot even get any more information I'm afraid ;)


Mă aşteptam să iau mai mult în sensul că eu credeam că e corect şi aşa cum am făcut eu


You should call `on.exit(unload(test_pkg), add = TRUE)` to clean up and unload the package, do so after loading it the first time at L40.

Also I would suggest you use `withr::with_temp_libpaths()` to set a temporary `libPaths()` rather than rolling your own at L27.


Yes


If we're playing spelling police, could you change it to "Could Not Find Lua Library?" That was my fault.


I think you meant `newStr`


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


@zensh Ah, now I see, thanks! I also thought that's a backslash issue.
Can other symbols have similar problems?


wtf?!


@xi this works but is it correct ?


@guiz26 Did it worked for you?


Each of these are duplicated.


LGTM


That's why PayEx should allow UTF-8 completely @asbjornu


`Result` exists for a reason, you know.


Hi

Does your QS shade transparency still working with latest cm sources? My Seekbar cannot move at all for this option with latest cm sources.


Gah! Not sure how my tests all passed then...


Besides the duplication, looks really good!


should we add this to all others as well? Any chance of being able to do multiplatform?


edaw


No, I don't think they are, actually. One is doing min x and one is doing min y.


```javascript
this.changingTicks = triplets.map(function(triplet) {
    this.cycleDuration += triplet[2];
    return this.cycleDuration;
}, this);
this.changingTicks.unshift(0);

```


Duplication wouldn't cause a problem, it's just inefficient. :dancers:


pub pub?


I think you forgot to re-document after adding `@export`


One of these probably needs to be "height".


Thank you @craigmichaelmartin for that.
But don't you mind to update the link to the correct one:
brave/browser-android
And also change the author to
Brave Software <support@brave.com>


agreed. see change here
https://github.com/pivotalservices/gtils/commit/ca0ed1d536923842725ee721d15d04eac51f7c51


It works because these don't test for `MinX`. ;)


Oh wow! :clap:


Ooops, no, something else is causing me problems


There is `/organizations/join` in https://github.com/jpkrohling/hawkular-accounts/commit/44f6c5763bd4fed9213c99591a2b65d49af541bd#diff-ec4164e2212f9fa4a49e4c20bd96c464R132

I know little about how the endpoints are organized so far an Accounts, but would it not be cleaner to hang these `organizationId` related endpoints under `/organizations`? - i.e. `/organizations/{id}/joinRequests`?

Similarly the for the present `/organizationJoinRequests/` showing the requests of the present user - should it not better be placed under some `/currentUser` endpoint (that we perhaps have)?


On purpose on an older version, or OK to upgrade (and advise) to the latest? (I can file a PR if need be). Thanks


Will do. Happened in earlier merging of your commit.

2016-01-21 15:24 GMT+01:00 Sam Brannen <notifications@github.com>:

> Please change all ExtensionRegistry variable names to extensionRegistry
> (i.e., with a lowercase "e").
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/junit-team/junit5/commit/f250792f7f2a6a40cdec82f3090ce88d99aad152#commitcomment-15586387>
> .
>


Is this filtering covered by a test?


I mean, `Result<-, ErrorReported>`.


The problem is that `loose` generates a less safe mapping since it's possible the names might not be found at runtime.  If you have any ideas of how to make it a safer binding then I'd be happy to change it.


You want to be using `PaperInfo::options()`, `$row->options()` or `$row->option(OPTIONID)`. The latter returns a PaperOptionValue or an array thereof.


`npm i --save-dev tap-parser` ??


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/tests/results/TextRangeTest.py`, line 93.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/results/TextRangeTest.py
+++ b/coalib/tests/results/TextRangeTest.py
@@ -90,7 +90,7 @@

 class TextRangeJoinTest(unittest.TestCase):
-
+
     def testUP(self):
         self.pos = []

```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `coalib/tests/results/TextRangeTest.py`, line 93.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/results/TextRangeTest.py
+++ b/coalib/tests/results/TextRangeTest.py
@@ -90,7 +90,7 @@

 class TextRangeJoinTest(unittest.TestCase):
-
+
     def testUP(self):
         self.pos = []

```


thanks for the hint.
changed to `assertThrows` and moved the `@SuppressWarnings` there


So the goal of this is to take option searches and add columns? Interesting, and useful!, but I would go about this a different way.

Specifically, in PaperSearch, I would change `PaperSearch::_search_options` to add the options to the viewmap.

An initial version would look something like this:
```
            $qt[] = SearchTerm::make_float(["view" => ["(opt:" . $oq[0] . ")"]]);
```

This shows the formula corresponding to an option. It would be a couple-liner.

It would be better to have a OptionPaperColumn, though.


I see a tiny space has appeared


It is removed here https://github.com/mate-desktop/caja/commit/5fe5169a0e91783cc7d606c7170f555e04254544#diff-0c6d11420be4751c3843fbf8b7278b07R44


Thanks for the pull request!


Byłbym zdecydowanie za tym by rozdzielić metody api, od silnika api.
Chodzi o Single Responsibility Principle. Metody podobne do `setApiId()`, `apiRequest()` są spakowane w tej samej klasie co metody do obsługi voucherów `get()`, `create()`, `enable()` itd.
Moim zdaniem to jest pierwszy Improvement dla tej klasy


djvenv


Why is this necessary? There's already code in PaperSearch that handles this kind of thing.


Can has posixct please?


Ideally with `readr::parse_time` which should be quite a bit faster


Re: temporary `libPaths()`: I just followed the examples I saw in other test-* files. I see this pattern, and don't see `with_temp_libpaths()` anywhere else in the devtools directory.


Is this endpoint covered by a test?


Sure thing! Would you like me to also update the name, description, and bug url as well as the repository and author?


Thanks!

And yeah... I figured I was indirectly to blame. That's why I wanted to let you make the change yourself. :wink:


Btw, adding <relativePath /> would be better.


Maybe this social stuff can be a partial as it is the same for the blog posts right?


aha, I see you put it in a partial! The joys of reviewing commit by commit


The diff makes it look a lot worse than it is. I literally added 3 lines (import statement, module level lock, and then I am using the lock using a context manager).

The rest was just indents, and the diff algorithm trying to be clever


One sentence of JavaDoc please. What is JoinRequestNotificationDispatcher good for?


M-am re-uitat acum peste ce ai făcut și într-adevăr, ai făcut o treabă mai bună decât nota pe care ți-am dat-o, dar numai privind în retrospectivă și luând în considerare notele celorlalți.
Mulțumesc că mi-ai atras atenția, noua notă o poți găsi pe git.

PS: doar pentru că e funcțional, nu înseamnă că e și corect.


@danielgustafsson can you summarize any interface change so we can let @mkiyama know about it?


Well, this commit was partially applied somewhere before, maybe during 1.5-1.6 development.
In origin upstream commit there was a gdk_spawn_* function


Yep!  Although not to this branch, we're not using it any more :)


nit: why do you use `i8::MAX as u64 + 1` here and `i8::MIN as u64` in the following lines?


Why did not you use message from line 216?


@danielgustafsson @hlinnaka "prevents WAL logging when COPY is done in the same transaction
that created it." will we get a performance boost for this?  It would be nice to advertise this in Greenplum 5 if its worthy


No specific reason. It's an error


Why do you need this reference value? It is used only in function, you have never used it outside


I think that we can combine these scripts


Re: Am I doing the right thing? (for the view classes): I think you're on the right track, and I'm sure some refactoring could help get the code in a better state. One thing I saw was that the `View::Base` class is the superclass to only one view class, `View::PlayingArea`. If you only have one subclass it really seems like that functionality should be rolled into the superclass until you have a need to have multiple subclasses.

Re: `@app` instance variable: I would leave it like you have it since `@app` never changes, so it should be fine just holding onto it during initialize and referencing it when you need to.

Re: Requiring `View` class: You could just do one `require` for the file containing the `View` class, and inside of that file you can `require` all your other view-related files.


This does not work - `::std::i32::MIN as u64` is `ffffffff8'0000000`.


fix the import, the rest lgtm


OK


112083967_VentilatedSlab (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


I'm going to trust that this looks ok.

How does this `.html.markdown` jazz work anyway? Could we use markdown asterisks to create the titles?


@mattymaloney I prefer that as well. Mind opening up a PR?


@cvagrad86 DWMR This nice. Nice = clear and concise.


I created issued #459 for this.


Where are other possible visibility values defined? Wouldn't an enum be better?


bugfix + test?


Yeah, I definitely prefer blog style as well. I like the idea of a config option.


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


Fixed in https://github.com/duckduckgo/ddg-screen-diff/pull/13


When a payload has a length of 126 Byte, plus the 2 Byte of esp:tail, :encrypt() starts another 128 Byte block and fills it with 128 Byte padding, instead of setting pad_length to 0, thus avoiding to send a redundant 128 Block!

                                        pad_length = 128 -((length +esp_tail_length) %128)
length +esp_tail_length   old   new
0         +2                         128  0
1         +2                         127  127
...
126     +2                             2      2
127     +2                             1      1
128     +2                         128      0
129     +2                         127  127
130     +2                         126  126
...

Tested by raising length of payload in selftest() to 126..130, as well as multiple 128 Byte blocks, and selftest() still passes.


Can you please delete that, since `ThrowableResultOfMethodCallIgnored` is an IntelliJ-specific feature?

IMO, we should only use values supported by Java itself. Anything else can be configured in the respective IDEs (e.g., warnings can be turned off in project settings, etc.).


On purpose, just to show its application for older plugins that may require such configurations.
Thanks!


Ok, mulţumesc şi eu !


Didn't knew the trick with this as an argument


I don't think this is correct. A "still screen" could be detected before the system would ask for a password. As discussed, you need to check for the presence of the password prompt, either with a needle or by checking the output on serial


Why `<br/><br/>`?


I think you got the arguments reversed here.


is this correct? Throw a 404 if the user **has** the required permissions?


@csrwng Why this change? Was the previous deletion not breaking?

@deads2k @smarterclayton how can we more reliably break this file to check that validation fails for broken files?
Do we need more pointed tests for this? @sosiouxme will you be writing in-depth tests for master/node config validation in your diagnostics suite?


This fix is not correct. self.extra_headers is a list of tuples. The test should read:

..... and 'Host not in dict(self.extra_headers):

The list of tuples should be made into a dictionary.


- [ ] `getMember/` の部分は不要です。RESTfulなURL設計では、URL内に振舞い(動詞)は含めないのが原則です。


rank.leRank() é o campeao, entao nao precisa chamar novamente.
se vai ter que fazer int, ele ja devia ter devolvino int


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/26154/testReport/) for this commit**


ui.querJogar() :)
faltou voce testar o jogo depoiis da refatoracao, vc teria pego isso


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/26156/testReport/) for this commit**


靠還注意到這種事XDDDDDD


:facepalm:


eu nao extrai metodo para essas coisas? (nao lembro)


Why does it matter? Within the IDE it can only be turned off completely.
Since it does not break any other IDEs I'm fine with it.

2016-01-21 15:44 GMT+01:00 Sam Brannen <notifications@github.com>:

> Can you please delete that, since ThrowableResultOfMethodCallIgnored is
> an IntelliJ-specific feature?
>
> IMO, we should only use values supported by Java itself. Anything else can
> be configured in the respective IDEs (e.g., warnings can be turned off in
> project settings, etc.).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/junit-team/junit5/commit/114bc1ec7348cc872cca80a9751805451535cfd9#commitcomment-15586940>
> .
>


yup x) I not yet able to reproduce the pb. But if you found a way to fix it, let me know


Looks Good To Me...


> @csrwng Why this change? Was the previous deletion not breaking?

He changed the lines in the file.  This is fragile and sensitive to location.


Im trying to figure out how to commit changes.


The moment I deleted the shop info, unaware that it was being used on the HireUs page.


This code is OK but it also needs to be applied to umc_show_help() and umc_plugin_web_help(). Otherwise the commands are shown to users who are not allowed to use it.


stream nao tem outra maneira de pegar aleatorio mais simples? pq ta complexo o codigo. se nao, deixa em ArrayList mesmo e pega o elemento random.


Caja have so much deprecations, believe me you'll be happy about every none used 'if', after all is fixed.


What exactly is this line sanitizing?


charAt


I believe the content type for json api should be `application/vnd.api+json`


contemChute(string, char) e nao contemChute(string, string) certo?


Yeah, I was just composing a reply to the thread on gpdb-developers.


Yup, deleting the same line numbers changed the structure of the yaml causing a failure in test-cmd


@athanatos `inconsistent_obj_t` and `shard_info_t` are simple structs which offer the minimal set of functionalities for the librados client to access the necessary information for recovery. while `ShardError` and `ObjectError` are equipped with some helper functions to update the errors flags and to encode.

if we just include `inconsistent_obj_t` here, i will remove ObjectError and ShardError, and add some C style helper functions to create/update/encode `inconsistent_obj_t` and `shard_info_t` here instead. does this make sense to you?


This is so cool.


Coverity scan has flagged up this as as bug.
```
*** CID 121826:  Integer handling issues  (CONSTANT_EXPRESSION_RESULT)
/Source/Urho3D/Graphics/BillboardSet.cpp: 214 in Urho3D::BillboardSet::UpdateGeometry(const Urho3D::FrameInfo &)()
208             UpdateBufferSize();
209
210         if (bufferDirty_ || sortThisFrame_ || vertexBuffer_->IsDataLost())
211             UpdateVertexBuffer(frame);
212
213         // If using camera facing, re-update the rotation for the current view now
>>>     CID 121826:  Integer handling issues  (CONSTANT_EXPRESSION_RESULT)
>>>     The "or" condition "this->faceCameraMode_ != Urho3D::FC_NONE || this->faceCameraMode_ != Urho3D::FC_DIRECTION" will always be true because "this->faceCameraMode_" cannot be equal to two different values at the same time, so it must be not equal to at least one of them.
214         if (faceCameraMode_ != FC_NONE || faceCameraMode_ != FC_DIRECTION)
215         {
216             transforms_[1] = Matrix3x4(Vector3::ZERO, frame.camera_->GetFaceCameraRotation(node_->GetWorldPosition(),
217                 node_->GetWorldRotation(), faceCameraMode_), Vector3::ONE);
218         }
219     }
```


jaChutados nunca sera nulo


I don't understand. This branch changes `getImpl` which is in use since `1.0`. If you don't use `StyleChecker`, you won't get any warnings anymore. This is a fact, versioning won't help here. The only way to avoid this would be above mentioned b), but I think this would be overkill.


I was well aware that ThrowableResultOfMethodCallIgnored is Idea-specific. As Johannes pointed out it can only be turned off globally or on method or class levels (same issue).
If anyone takes offence I am willing to remove it as this specific warning is well contained within this method and not a danger to client code.


eu tinha comentado para usar boolean[] chutes = new boolean[100];
onde chutes['a'] indica se ja chutou a letra a


wtf is wtf?


nao precisa concatenar tudo, pode ir cuspindo direto pro out


pq agora tem esse if?


But since GetLatestGitRepo uses
- cloneRepo
- updateRepo
each on time, I thought I should teardown and init again since I want to test form a clean state. One time the clone should be called and one time the update. If I understand this correctly the init() the TestMain() will only make things clean one time. And after the TestCloneRepo() is called the repo will still exist when TestGetLatestGitRepo() is called, right? So the switching functionallity inside GetLatestGitRepo() will not be tested but only the updateRepo(), which means I should not test GetLatestGitRepo() at all (and get rid of the ugly parameter again) and just test cloneRepo)() and updateRepo() on their own?


Interesting, how did you get round the `tab` vs `<tab>` problem in magit (in particular)?


Possible transitions? Possible consequences?  - May I re-request after being rejected? May I re-request while PENDING?


I don't think so, because we want to know if "was" started, not "is" running. Its conceivable that it could start and then stop so fast that the caller never unblocks.


@americool DWMR (done with my review). You shipped code. You'll be shipping a lot of code! Good.


Does it matter? This works, but I thought it would work my way too?

But if you feel this is clearer, then OK.


Agree


no jogo anterior ja ensinamos array? (no jogo 1). se sim, me parece ficar melhor usar arraylist sim


why are you recurring?


You can't call skip_confirmation! on a collection of users, it has to be on each one.


Good point, it seems it isn't. Will add one.


Coverity scan also flagged up this one as problem.
```
*** CID 121838:  Uninitialized members  (UNINIT_CTOR)
/Source/Urho3D/Graphics/BillboardSet.cpp: 91 in Urho3D::BillboardSet::BillboardSet(Urho3D::Context *)()
85         geometry_->SetIndexBuffer(indexBuffer_);
86
87         batches_.Resize(1);
88         batches_[0].geometry_ = geometry_;
89         batches_[0].geometryType_ = GEOM_BILLBOARD;
90         batches_[0].worldTransform_ = &transforms_[0];
>>>     CID 121838:  Uninitialized members  (UNINIT_CTOR)
>>>     Non-static class member "geometryTypeUpdate_" is not initialized in this constructor nor in any functions that it calls.
91     }
92
93     BillboardSet::~BillboardSet()
94     {
95     }
```


Visibility of what?


Because of how devise works, it's probably trying to send the confirmation emails out when you seed your users, which is probably not what you want. Change this line to:
```
 user = User.new(
```


@ucupumar Could you check the above bugs reported by Coverity Scan. Thanks.


I think there was a problem with AngularJS with nested resources. In any case, I think it's easier to know what to expect: on /organizationJoinRequests , the response payload would be expected to be related to OrganizationJoinRequest .


This should probably method**s**
Also, this might look better as some sort of list (bullet, or numerals/letters) with one per line.


Definitely something we should have in the documentation. It can speed up certain ETL use cases, when you use staging tables, or newly created tables (like partitions).


What is APPLY and PRIVATE? What are the consequences for the bearers to have one or the other? Is one of them considered default?


Then here, add the following lines:

```
user.skip_confirmation!
user.save!
```


Thanks! Only suggestion I would have is to display a message that an invalid command had been entered and display help rather than exiting the interactive mode. Thoughts?


You can remove this.


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 tests passed, 0 test warnings)

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


Why did you take out knockout?


authorship belongs to @beanstown106 on this one folks.......


Same as above...


Again, same thing. Change to:

```
admin = User.new(
```


Never heart of Duplicates warning. What is that?


whoops


Is this an oversight? It bypasses the type check.


Wow, we're more badass that we thought!


The bad request? Or the method as a whole? The method as a whole is being covered by an integration test, the bad request not.


This will never update the CA file on a system that once hat written the file.


:tada:


Sinn der Änderungen hier wird nicht klar. Was ist die Story dazu?


Aquí le falta el `.first` o si no devuelve una lista de objetos `CursoAlumno`.
Debería decir:

```ruby
def es_ayudante
  @curso_alumno = CursoAlumno.where(
    curso_id: params[:curso_id],
    alumno_id: params[:alumno_id]
  ).first

  //...
```

No se si ese sea el problema.


btw, The headers are correct, I checked it, but I didn't test the blocks itself.. I don't even know how will I be able to get the blocks from the data. :P


I realize now I've answered my own question, so I added;

```
  (with-eval-after-load 'magit
    (bb/define-key magit-mode-map
      (kbd "<tab>") 'magit-section-toggle))
```


Added to repo. Remove from PR.


Just pushed this to the repo. Remove from PR.


Quando eu estava testando e não tinha informações no arquivo, ele dava a exception de posições que não existem.


すべての７という数字をGitHubに変更するように修正しました。


This didn't seem to render well... I assume you wanted it as some sort of title for the source block below, but it just rendered on the same line above (as "4. Export the service to the OSGi registry. .Blueprint descriptor example")


`data-` is now obsolete.


:hammer:


检查了一下代码，确实有问题。之前我给URL Scheme前面加了一个字母a，但是代码里面有两个地方没修改。

最新的修改已经修掉了，b772a7b


The `-` is in this case a unary operator, so it shouldn't get a space after it. It looks like you're subtracting `island - extrusion_width / 2`.


Same as above...


Sometimes it could be handy to say whose (username) request was that. One can maintain several accounts.


Yes. But if we avoid this, we run into other issues:

* If we send mail to people not in your DB ("somebody entered your address in your password reset field, but you're not registered. If you want to register, please do so at X. If not, sorry and just ignore this email.") we could annoy them. It's not so bad if that happens just once, but what if somebody triggers a password reset for Angela Merkel's email address every two seconds?

* If we just silently ignore the error, it would be a bad UI experience for people who thought the registered with one address but actually used another one -- since we don't tell that that the address is unkown and don't take any further action, it's very hard to figure out for them why they never receive the expected reset link.


`message` is type-erased. Do you mean to perform the check at runtime?


I know, I didn't really consider it a concern since the system temp dir is cleared on every reboot anyway and I doubt this file will be updated daily. But we can add a SHA as part of the filename to make sure it creates a new file once the content changes.


Same as above: Sometimes it could be handy to say whose (username) request was that. One can maintain several accounts.


@eugeneia Do you think my https://github.com/eugeneia/snabbswitch/commit/555dc11e84083888b01f42bce737579c4440d522 is worth a new PR?


I updated the instrument types.


Thanks I'll fix it


Sem problemas, pode alterar. Notei que esse código é do @luiz-antonio, confirma com ele.


While here you might as well want to fix line 2252 below (missing opening single quote for ConnectedSource)


No, I wonder why you added this overload to `deliver`. The point of `typed_response_promise` is to statically verify correctness.


laziness like this will bite you in the ass some day :)


I don't ever see any meaning redundancy that could be factored out. Every test is a little different.


Yup it works
I had that issue
Realized that the on preference change listener in settings was missing .pick our settings commit and the fix from settings as well


This pattern kind of weirds me out. When I first read it, I thought: "All tests simply throw `IllegalArgumentException` without asserting anything?" But then I saw that they all test inplace, _followed by_ another map call that should generate the exception.

I guess it is OK—just seems unintuitive to me.


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


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/26160/testReport/) for this commit**


Con los cambios realizados en este commit ya funciona


This is the REST request bean. There's an Enum for the actual entities, and the values are validated inside the endpoint.


1. you can not compress it, so online debugging.
2. want to deploy to github pages


112083967_VentilatedSlab (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (957 of 957 tests passed, 0 test warnings)

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


Ask IntelliJ :) /me removing it


We only want to reset the `loader.state` to 'HAVE_MASTER' if it was in the 'HAVE_MASTER' state before the request that caused the error. Otherwise, this would trigger a `loadedmetadata` each time we switch playlists in response to an error.


what?


Do you have this? This was missing for us
https://github.com/ResurrectionRemix/Resurrection_packages_apps_Settings/commit/5bf221cf1b2c273dcd12d328bf20246b2ca8b3fd


Are you sure that `csswring` does not need the same change?


I'm not sure you need a new function to hold this functionality.


That's odd, because if the caster isn't the target, then it should not enter that section of code. That's why I remove the "m_caster==target" check.

I should have tested it on a mob!

Nuts, so the code does not function the way i interpreted it :(


@ctrueden It has been more of a utility package thus far, but perhaps it is time to get it working in FIJI. I also have a half concocted version that will output vector graphics.


@NohSeho
`*.js.map` files are for `*.ts` since we are using Typescript. I will copy `dist/generator/*.ts` from `generator/src/*.ts` while running `gulp dist`


Needs test(s).


Damn, this wasn't supposed to go on this commit.


Should this not be `smallint` rather than `small_int` ?


I see now: it's because the other javadocs have it as well, so, copy/paste problem on my side. (and the poor IntelliJ took the blame... I should probably stop blaming @vrockai and start blaming IntelliJ for everything).


Eu pensei o mesmo inicialmente, porém o que acontece é o seguinte:

//estava apenas criando referências para String, porém sem nenhuma atribuição
String[] chutes = new String[100];

//quando fazemos isso, são impressos 100 null
for (String chute : chutes) {
	System.out.println(chute);
}


Any tests for this one?


This one has a purpose. For better readability of the javadocs when rendered as HTML. Should I replace it with an empty <p> ?


It has an unfinished string? I don't really understand what's up with line 7 anyway.


This line renders a somewhat weird... May I suggest changing it to something like "CUD operations process `PolicyPlugin`, `AccessPlugin`, and `PreIngestPlugin` instances before execution and `PostIngestPlugin` instances after execution."


This should not be a pointer - it is not optional.


Similar to above.


Should not be a pointer.


could use join method instead


Not sure, going to check now!


would be cool, yes. do you want to do a PR to add something to the sandbox? ideally we find a realistic example - maybe adding a register node only when not logged in?


Should not be omit empty.


Can you try again with latest commit 861b4dd2522d0b


I gave it a shot but couldn't really get into it :( Could you give me a quick heads up on how to use a different php version for composer (on windows)? I've got `php 5.5` in `C:\xampp\php_5.5\`
In `C:\ProgramData\ComposerSetup\bin` I've got my `composer.phar` and a `composer.bat` with this content:

```
@ECHO OFF
php "%~dp0composer.phar" %*
```

I'd be okay with a global version but tbh would prefer to have the option to run it with php 5.5, not by default, e.g. something like `composer55 update module/name` or something like that. Already found a few links on the web but couldn't connect them. :/

Sorry for bothering you with this stuff but I still hope you can somehow point me in the right direction!


You said something about php.ini debug stuff


this is not mentioned on the CHANGELOG yet


@erbenmo, thanks!

I'll get that fixed.


for internal classes, the zend_class_constant is malloc-ed. we need to free it.
```
if (const->ce == ce) {
}
````
so, if two classes share one const, and it was freed before, this read(const->ce) is invalid..


It's due to [here](https://github.com/actor-framework/actor-framework/blob/topic/actor-system/libcaf_core/caf/detail/split_join.hpp#L64). I guess I should just fix the call site.


Why is this necessary? Also I think having methods with so many parameters is generally a bad idea.


you should have a nb_occurrences field instead of is_consumed and search for all possibilities as it was done before: bom_id, picking_type_id (of mo), workcenter, ...  You should not create alerts at forehand.


In this particular case I don't think a message is required since Ctrl-d
effectively closes the input stream (STDIN) on *nix -which is why
Console.ReadLine() returns null.  It's a shortcut that is used often on
*nix to exit apps that use stdin for interaction.

On 01/21/2016 08:55 AM, reidev275 wrote:
>
> Thanks! Only suggestion I would have is to display a message that an
> invalid command had been entered and display help rather than exiting
> the interactive mode. Thoughts?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joncfoo/Fix/commit/f0fd18d148e709cf44af10c0f1950393ff226a14#commitcomment-15587316>.
>


Good point, I haven't thought about it too much. Right now, it can go from anything to anything, which seems wrong.


By the way: yes, you can re-request after being rejected. You can re-request while pending.


Just checking to make sure... Did you mean for all three of these lines to be part of the same paragraph?


Although it was enabled, it did work without the change, but `result.map` for `csswring` is also an instance of `SourceMapGenerator`


@diasdavid more like :night_with_stars:


So this duplicating when writing (it is also very error prone) is not needed.  Please remove this.


The linked example uses an untyped promise, does it not?


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


Please bring back the necessary fields!


Thank you very much for the fix.


ah right good catch


My fault. Gonna fix this. Just a moment, please.


Contribute a function to `util` that does what you want. This doesn't seem to be a good place for a random number generator.


I am generally unhappy with the code organization in this file.

1. Please rename the file to `cascade_hashing.h/cc`
2. Create one `class` called `CascadeHashing`
3. Make `Options` nested struct
4. Make `Projection` (or similar) nested struct
5. Make `GlobalData` and `LocalData` nested structs
6. Don't use structs and suddenly use private fields.
7. Don't use structs and put smarts in it (i.e., move the methods to another place, or make proper classes)


No rush. Thanks for staying with me here. :)


___"It's better to be blamed than to be forgotten."___

vrockai


may be call it wget


done


An instance method for `create` seems a bit counter-intuitive to me. If we decide to follow this road I suggest we make package private all method in `ResourceManager` (as well as other services) that are already available through entity objects.


All these `typedef`s belong in their respective class, or at least in the toplevel `CascadeHashing` class.


Did you mean to add an example of addFive()?, toggleZ() is duplicated below.


+1
but I believe `Collections.unmodifiableCollection(new ArrayList<>(plugins.values()))` is more readable and efficient.
If we return defensive copies of the original list, I also think the unmodifiableCollection wrapper is not needed.

Also see https://github.com/igniterealtime/Openfire/pull/436#issuecomment-162859642
Seems like the ConcurrentHashMap had some purpose (thread-safe iteration).


Hi guys,

The problem with IDE-specific features is that they are just that... _specific_ to a particular IDE.

In this particular case, Eclipse complains about an "Unsupported @SuppressWarnings" declaration, since the value is non-standard. The end result is that users of Eclipse and other IDEs are confronted with warnings that they can't do anything about, and these warnings hinder the user's ability to disambiguate real issues and things perceived to be issues by a different IDE.

Of course, none of the IDEs are perfect with regard to `@SuppressWarnings` support. Thus, with that in mind, the easiest solution for a team that uses different IDEs is simply to avoid IDE-specific features.

So.... Thanks, @mmerdes, for deleting that line! :+1:


Since we have `get` why not adding also `list`?


My own comments convinced me to keep the addition of `verifyDelta()` :smile: So the changes should be simple now:

1. Change this line to ```return !!exports.hotp.verifyDelta(options);```
2. Change `window=0` along with docs and tests

If that's okay with you, I'll make the changes by end of week and submit a pull request.


Why is this inline?


this is never called -  i think a file is missing from this PR something like: components/com_fabrik/views/list/view.tags.php ?


LGTM.


I guess `.toString()` was called when result was stored in asset after plugins run. That's easy to check - if you enable autoprefixer, but disable csswring, it should work too without patch.


I made some mistake in the dps meter, I not sure, but I think the arun's mote hit count problem come from that. Fixing that


This method doesn't appear to be used - remove it if it isn't


Oui c'est bon


Would it not be more efficient to store the results of both getNetworkResourceProviderClient and getLoadBalancersOperations in local variables rather than making repeated calls?


Nit "setup" -> "setUp"


Remove JsonBuilder, NetworkHelper, and NetworkSecurityGroup - they don't appear to be used.


112083967_VentilatedSlab (Nigusse) - x86_64-MacOS-10.9-clang: OK (2051 of 2051 tests passed, 0 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)


@Fabii547 Do you have a reference handy?


These two were placed wrongly... surely I'll need to fix offsets for the toilet model :smile
![image](https://cloud.githubusercontent.com/assets/11136108/12484338/31bc10f0-c05a-11e5-93bb-4d566bf32538.png)


Few extra spaces here.


Invalid merge. This changes was committed @microshine


It is actually a possibility to move all these template functions to the `.cc` file and instantiate them for float. As far as I can see it, this is the only type we need.


Отлично!


quite, but the `Dummy` is only ever emitted when an error is reported so the rest of rustc can continue reporting more errors.


TeamCity Civic Assembly :: Build backend + frontend [Build 18](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=411&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


@drobakowski I was wondering why it was decided to remove all other possible attributes (e.g. `text`) and just return the `{Line, Col}` tuple. It is useful to also have the value for the `text` attribute under some circumstances. We sometimes use it for some rules in [`elvis`](http://github.com/inaka/elvis).


谢谢您的提交
...您的支持对我很重要


Problema este ca nu apare in lista de pull requests...


Try setting [this](https://github.com/ianbattersby/dotfiles.emacs/blob/master/spacemacs.symlink#L131) to `t`. :-)


trecho de código traduzido:

def joga(nome)
	palavra_secreta = sorteia_palavra_secreta

	erros = 0
	chutes = []
	pontos_ate_agora = 0

	while erros < 5
		mascara = palavra_mascarada(chutes, palavra_secreta)
		chute = pede_um_chute_valido(chutes, erros, mascara)
		chutes << chute

		chutou_uma_unica_letra = chute.size == 1
		if chutou_uma_unica_letra

			total_encontrado = palavra_secreta.count(chute[0])


Nope, this is taking advantage of PromiseKit's fallback/recovery mechanism. Returning a non-error from a `catch`, results the promise being resolved with the value, which is passed to `then`.


Oh yeah, I don't need that anymore - I think I copied that whole block from somewhere without paying a lot of attention. :blush:
I assume dimagimon didn't mark because these are class variables - if they were local (`es` instead of `cls.es`), they'd be obviously unused, but linters often don't warn against class variables because it's very hard to tell how they might affect behavior (maybe some super class of this does something fancy if `cls.es` is set).


112083967_VentilatedSlab (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2089 of 2089 tests passed, 0 test warnings)

 * 6 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)


:+1: Thanks!


This will still malfunction when disconnected (nil doesnt have the IsNull function), the proper way is `if ply` because GetPlayer will return nil on disconnected playerIDs


Looks like there is a typo here somewhere: every track is scrobbled twice for me.


is this a bug?


I would still put the `cwd` in the default options but I dont see it here.


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


这个翻译成 "堆芯熔体" 比较好。


IOW: DRY up all the handling into one `then` callback which is invoked w/ a download when successful, or `nil` on any error.


securityGroup isn't used it looks like this line of code is not needed.


The buttons are called LB/RB (left/right bumper) and the analogs behind are LT/RT. Actually the thumbs are called LS/RS (left stick, right stick)


you actually don't have to re-query it. `aAssets` will already be correct from line 22.


response variable is not used, remove it


You "break" an old version and don't want to support the code to still support the old version. Then we have to remove it.  Do I forget something?


50 pixels does not seem to be enough to save the ostrich!


:) it is. You see, AM first apply a periodic trigger aura on caster. This is done via effect0. But the victim is chosen via effect1. So, in AM case, the enter condition m_caster==target is allways true before effect1 check. Only after effect1 check the target is effectively known


Pushed. Remove from PR.


File with Magnetocosmics' base information for calculating Rididity cutoff


you can't stop autoplay if you not use onPageScroll at ViewPagerAndroid. and e.nativeEvent.contentOffset is undefined , use my commit solve this problem


Yes ......may be comment written on line 890 is wrong. it should be changed.
and
may be fossology#602 (comment) is not handled in delagent. will check it.


Unindent this and remove the ``else:`` (because the subsequent code depends on the fact that the ``if`` branch doesn't fall through).


Could this be `util.format()`?


can remove `oldData`


- [ ] 少し読みづらい感じがしました。->「以下は、Memberリソースを「概要」と「詳細」の２つのフォーマットで扱う際の実装例である。「概要フォーマット」はMemberリソースの主要項目を、「詳細フォーマット」はMemberリソースの全項目を出力する。」とか、どうでしょうか？


I like this tip. A lot of JS developers are used to writing for browsers that don't support immutable objects without 3rd party libraries. You should mention that Object.freeze() is new in ES5 (so no IE8 support).

For the lack of built-in recursion, you could link to the MDN documentation - their 2nd example shows a deep freeze snippet (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/freeze).


This is also something that does not convince me so much about unifying infos and functional objects. Equality now checks also if `options` are equals but this is not in general what a user might be interested in doing. In my mind it is important to have a class whose content represent only the remote resource and whose comparison gives the same result as of comparing the remote resources (e.g. JSON).


done


Awesome! Ian must learn new variables, Ian must learn new variables, Ian must learn new variables :-)


Any of the main js files that can be added to core I'll move over soon.


A test is coming. As far as I know, this bug was introduced with 3.0.


well... `ty_hint` should always be the type of the expression, but since you can't do anything with a constant array but index into it, we're passing the element type hint so the array elements have the correct type hint. This saves us from creating the `[T; N]` type hint just to take the `T` out again. This was the existing behavior


@jdaguil I am not sure if Jinja will escape the ampersand automatically? or would you need to use an HTML escape?


Should be rssPublicationDat**e** (as defined in `CommentListRssPresenter`). Does this need to be escaped?


I would define this global `var` outside of the component definition at the top


I would also define this global `var` outside of the component definition at the top


hmm yeah i'm trying to best find a way to articulate. i'd buy that the `ExportSchema`s are pretty similar. the current plan as far as i understood it is this:

1. Iterate through all the apps and create `ExportableItems` data structure. This is just data and encode no information about order/display/etc. An `ExportableItems` has many tables and each table has many `ExportItem`s

2. Send that structure down to the browser and render as the Export form page.

3. With the information received from the browser + the `ExportableItems` create the `ExportConfiguration` which contains man `TableConfigurations` and each `TableConfiguration` has many `ExportColumn`s. Each `ExportColumn` has a `ExportItem` (from the `ExportableItems`). The main difference comes in the fact that `ExportColumn` has more information like `display` (the name for the column). Then save the `ExportConfiguration`

My main concern is now that an `ExportConfiguration` is saved, in order to render it again we have to deconstruct (or recreate it)  it into an `ExportableItems` data structure + another sort of data structure to encode the `display` name and other properties such as `selected`.

This is as opposed to creating an `ExportConfiguration` sending it to the browser with some defaults for the display name, selected etc, and then saving that `ExportConfiguration`. It'd be dropping the concept of `ExportableItems`. @NoahCarnahan I think that's what we were talking about yesterday; feel free to add on


Okay, tested in-game...

I left the code where I put it
I put back the check m_caster == target  <-- weird ????
And it works fine

That check looks totally superfluous, but without it, the condition fails... odd, plain odd :-)


It'd be bit manipulation. I'm not sure if I have it correct or backwards right now, either way it's easy.


Thanks!


It's more consistent and explicit to do a ``continue`` here (and unindent the ``else:``).


Well we added it to withr not too long ago, but no reason not to use it now!


Remove unused imports: NetworkResourceProviderService, AddressSpace, VirtualNetwork, ResourceGroupListResult, ResourceHelper, Files, Path, Paths


As fi folosit code-highlight in loc de bold, dar, aparent, `iamntz` este impotriva si prefera **bolduirea** sau *italicul*...


This conf isn't good to merge, can it be made generic or something else be done here?


slice(None) is perhaps more clear


I would define above this line `aAssets = document.querySelector('a-assets');`


Do not use JsonBuilder use Jackson


just this last change:

"from factor vector " -> "from a factor vector "


Why is this fastest, are you assuming a storage order here?


or just s/-person//


vérifie si c'est gagné et fait des choses... peut-être séparer :
 * vérifier (un marqueur dans le journal des quetes par exemple)
 * terminer la quete (retirer les pommes de l'inventaire)


@MCSou why are you spamming and going much off-topic?


I just forgot to implement the `FIXME` and pass a new type hint


- [ ] ↑のコメントに関連し、 出力を切り替えるためのパラメータ名は `type` ではなく、`format` とかの方がよさそう。


3 fois ?
```
Apple apples = new Apple();
apples.addCount(2);
pj.getInventory().removeItem(apples);
```


Переменная i не задана через слово var  и является глобальное переменной (может подкинуть ряд сюрпризов)


I guess we have nowhere to link to from an item. Since this is an optional field in the RSS specification, I suggest we omit it.


done


Any reason this wouldnt be 0? Do we test for that?


add ``" for %r" % (encoded_relpath_u,)``.


Not node connections. I would say that we need to check if the options are correct.


"Only after effect1 check the target is effectively known"

Gonna check that in-game now. From what I have just done (see above), it now works fine.


Same as above but `max_ax = len(axes)-1`


Why create the local resourceGroup variable?


Sorry for not being a bit more specific. trace and debug logs should not be i18n'ed, so leave them as they were


Les titres, les chaines de caractères en règle générale, c'est de l'IHM... du coup, du coté de swing uniquement, non ?


This exception instead should be i18n'ed


Mélange de texte et de donnée... :'(


Yes, gigantic path embedded in coffeescript. Bad idea. How can I fix this?


Yep, that's true for sure, it was either one of them, but not both. Is this fix still valid then, or is there another place I should do `.toString()`?


是熔融。


I think that intentation is missing in the following lines.


I think it worth saying that one must add struct sol_flow_metatype_option_description elements to the vector.


fwiw, you could replace the next several lines with this:

```html
var lazyLoadEntities = Array.prototype.slice.call(document.querySelectorAll('a-entity[lazy-load]');
```


Bien vu !


Use Jackson objectmapper instead.


typo; should be "existence"


TeamCity Civic Assembly :: Build backend + frontend [Build 18](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=411&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 303 (303 new), passed: 12, ignored: 8 Build time: 00:11:02
### Failed tests
```
blogs.api.admin.v1.blogs.tests.BlogTestCase.test_create: <no details avaliable>

blogs.api.admin.v1.blogs.tests.BlogTestCase.test_delete: <no details avaliable>

blogs.api.admin.v1.blogs.tests.BlogTestCase.test_detail: <no details avaliable>

blogs.api.admin.v1.blogs.tests.BlogTestCase.test_get_document_attachments: <no details avaliable>

blogs.api.admin.v1.blogs.tests.BlogTestCase.test_list: <no details avaliable>

blogs.api.admin.v1.blogs.tests.BlogTestCase.test_should_get_related_objects: <no details avaliable>

blogs.api.admin.v1.blogs.tests.BlogTestCase.test_update: <no details avaliable>

blogs.api.admin.v1.blogs.tests.BlogTestCase.test_update_tags_field: <no details avaliable>

blogs.api.admin.v1.blogs.tests.VenueCategoryTestCase.test_create: <no details avaliable>

blogs.api.admin.v1.blogs.tests.VenueCategoryTestCase.test_delete: <no details avaliable>

blogs.api.admin.v1.blogs.tests.VenueCategoryTestCase.test_detail: <no details avaliable>

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


Please revert this, it breaks backwards-compatibility for *users* of CPAchecker. This has nothing to do with backwards-compatibility of the Java API.


do you still need the `setTimeout`?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/26171/testReport/) for this commit**


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/26172/testReport/) for this commit**


It worth to add the same fix at https://github.com/nodeca/mincer/blob/master/lib/mincer/compressors/csswring_compressor.js#L72 and squash all to single commit.


Hmm, now it's not working!

Gonna put it back to the way you had it.

GREMLINS!!!


TeamCity Civic Assembly :: Build backend + frontend [Build 19](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=412&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Bandcamp has been tested, just use the track URL.


Now what do you learn by this boys and girls, besides not to trust me, lol

TEST TEST TEST!!!!


getResourcesGroupsForApp does not appear to be used. Can we delete this method?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1209](http://teamcity.monogame.net/viewLog.html?buildId=13831&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


FYI: you can pass an object  `el.setAttribute('material', {src: 'img.png'})` (since we extend `setAttribute` for components


It removes hyphens, replaces [0-9]/[0-9]/[0-9] with [0-9] [0-9]/[0-9] so parse can handle it, removes \sto\s\d+ and the letters 'dia'.
It's pretty crude, it needs much improvement. these are just things that I've run into cleaning some "dirty data".


You can move this inside the get_options if (line 2278), because if there's no options description I think the node will not have an option symbol.


I'd argue this should use Level::DEBUG otherwise Log::Debug won't work as a quick debugging tool.


@AlvaroR156 great catch! thanks for your contribution and feel free to add anything you think would be a benefit!


Sorry -- forgot to check it in the first time.  Done.


We had a deprecation warning for about 6 months now. You don't think 6 months of deprecation for options is sufficient?


Sys::Drop maybe, Com_Log(Error) calls Com_Error I think


getAllResourceGroups does not appear to be used, can we delete this method?


looks great! thanks


ah right makes sense


:guitar::star:  - refreshing the index is definitely what I wanted here.  With nothing there, the test consistently fails (just tried again).  Refreshing the index works, and I'm much more confident that it will continue working then waiting an arbitrary amount of time.

Also, we don't have a better way to send to elasticsearch in tests, do we?  Something that patches `Document.save()` to also `send_robust()`, or some other kind of  synchronous pillowtop?


@coreyfloyd shouldn't this be logging tabbar taps?


`C:\xampp\php_5.5\php composer.phar`


You should init options vector here. Because if you take a look in the line above, if it fails we will go to exit and we will try to clear the vector. However it will contains bogus values that may lead to segfault.


Ah, it's true we don't want these show up by default though. I guess it makes sense to have common.default use NOTICE but ideally this would use a Logger.


I think we should. Yesterday, I was (mistakenly) doing a dynamic require...like:

```
try {
  const devEnvJSON = require('../../dev-env.json');
  Object.assign(process.env, defaults, devEnvJSON);
} catch (e) {
  Object.assign(process.env, defaults);
}
```

That works on Android, and doesn't work on iOS. It shouldn't work in either place (even though it would be cool if it did) :)


Please elaborate on what you mean. T is actually Vec64f or Vec128f. Do you want to pass float* as T?


Absolutely.  You can chain lots of renderers together too, e.g. a file called `foo.html.markdown.erb` would get processed by ERB, the our Markdown renderer (that might be a bad example as I think they *all* get processed by ERB anyway, but the principle stands).  Of course, chaining lots of renderers together might make the file confusing to read, so use caution.


Delete the comment at the end of this file that has the template sample in it. The sample is out of date and we have a valid sample used as a unit test.


It would be more instructive if title="" and key="" had different names. Suggestion: "body", "canvas" or any other name that suggests a main region.


`
This is the first step towards extracting the listen watcher into a
separate project.`

other projects use the tests in their tests


cc @sirthias @jrudolph @rkuhn
so that's our new Java API has much less overloads and is more helpful in parsing the target etc


Done! 411dd81


looks good


Passed TravisCI. Work done while waiting for my flight to NY. Added more basic functionality and fixed a few missing items.


Good point. I tested this with some 3d examples only, and this version was fastest, followed by last axis last, and middle axis was slowest. Could be even faster with reordering completely, though.


You can have `axes=(1, 2)` in a for a 3d array.


@grosser Gotcha! Thanks.


I couldn't find any usages for this class. Do we need it?


LGTM.


The class is missing the `toString` method.


The second to last semicolon is a syntax error.


i think that was noah's intent. when i was writing i didn't see the advantages of storing as a list at the time, and it was easier to do just by storing string. rethinking now and it maybe be best to store as a list in case we want to use different separators (which we do for the exports)


fixes #5689


hmm, this might be okey, the child class is always dtor first. will have a try


Hey Brian,
I tried a lot _not_ to add a config file. Unfortunately it includes things
like the path of the binary and how its called (path , state directory ,
log directory), the path that the binary runs usually is not in system path
and the other 2 parameters are not easy to get. MAYBE we can get it from ps
, from when it runs but i think that this is not really a good way. the
other way might be to get them from the configuration (postgresql.conf) if
you have any ideas i'm good with trying everything but ..Worst case we
leave this check out.

omnipitr_monitor=/home/vasilis/work/db/omnipitr/bin/omnipitr-monitor
#OmniPITR state diretory
omnipitr_state=/home/vasilis/work/db/omnipitr_data/state
#OmniPITR log directory as described in archive_command
omnipitr_log=/home/vasilis/work/db/omnipitr_data/log/omnipitr-archive-^Y-^m-^d.log

On Thu, Jan 21, 2016 at 5:23 PM, Brian Clapper <notifications@github.com>
wrote:

> This conf isn't good to merge, can it be made generic or something else be
> done here?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vventirozos/nad/commit/f5257b039cd357d98637336482229b7659f40636#commitcomment-15588118>
> .
>

--
Vasilis Ventirozos
OmniTI Computer Consulting Inc.
Database Administrator


The remaining problematic part is basically that this interface no longer works

```
public interface SpanAndEndpoint {

    /**
     * Gets the span to which to add annotations.
     *
     * @return Span to which to add annotations. Can be <code>null</code>. In that case the different submit methods will not
     *         do anything.
     */
    @Nullable
    Span span();
```

Span is not mutable, so providing an interface to modify a span will no longer work.


If we have set a specific environment variable, we can forward requests through a proxy (CORSProxy) that adds very permissive Allow-Origin headers to preflight requests. This is just to ease development (doesn't require setting up an nginx in to act as proxy).

Putting your whole Plone behind an nginx and putting the WMS domain server into `/etc/hosts` to point at localhost doesn't work, because then there is no way to reach the real WMS (since it gets searched on localhost)


Awesome! :+1:


I don't really have any ideas, I just reviewed it for any glaring issues, and this is one.  I can't merge this the way it is.  If the check needs left out until something else can be done then I think that is best for now, get the other ones in and drop this one.


El .json? Esta en https://github.com/latam-airlines/crane/raw/PAAS-558/test/resources/marathon_tasks_response.json


hm... it feels like it will be quite easy to get inconsistent when you forget the equalsIgnoreCase somewhere else. Is the IgnoreCase really necessary?

Would not `Visibility visibility = Visibility.valueOf(request.getVisibility())` serve similarly well with less lines of code?


@paulrobinson  - Only change is line 385:  "3. Upload the JBoss Developer graphic to " should read "3. Upload the Red Hat Developer graphic to "


FYI, this is a good use for the C `cleanup` attribute (DRY up the `CFRelease` logic and ensure it always happens when scope is exited, regardless of exceptions).  it's essentially a C version of Swift's `defer` feature.


I didn't include it in the default, because it doesn't seem like something that should have a global default which could lead to unexpected behaviors if not overridden.  Honestly I'm not sure the best way to handle it.  I suppose you're right it's better to include and then override it where it needs to be overridden.


penis


unused import - delete it


Good catch. This same typo actually shows up in a bunch of times in the vignette. It will be fixed in the next release, probably within a day or two.


Might need a logger here.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1240.e8740b0f45b50cdb78846ee01834712f44006400](http://build.palaso.org/viewLog.html?buildId=68319&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 842, ignored: 7 Build time: 00:06:01


Pero quedaría dependiendo de la instalación del marathon en localhost y en el puerto 8081, y si no esta UP fallaría el test, pensé que la idea del test era hacer la aislada de la instalación del Marathon


I was about to open an issue about this commit but will be very difficult to conform to the contribution guidelines (difficult to provide a reproducible scenario), but I'll try sooner.

Well we noticie that this commit is causing in our apps some problems that we haven't before, basically when setup the application generate the database a existing database is needed and that's because with this commit route_translator is loading the application_helper module and in our application that file preload some content from the database.

So is difficult to see what is happening here but I'll try to summarise here:

1. ActionView::TestCase is required
2. declares ActionView::TestCase::TestController that inherits from ActionController::Base
3. ActionController::Railtie extends ActionController::Base with ActionController::Railties::Helpers
4. So when a klass is inherited make a call to `klass.helper :all`
5. AbstractController::Helpers  executes the `helper :all` call executing `modules_for_helpers` method and that method requires `application_helper` you can see it [here](https://github.com/rails/rails/blob/4-2-stable/actionpack/lib/abstract_controller/helpers.rb#L147-L152)

so my solution can consist on including/requiring this changes on this commit for the test environment of the host rails app. What do you think? should I proceed with the changes in a proper PR


ok, thanks for the feedback


You are leaking cdata->url


agreed , if we really need this i'll try to find a workaround

On Thu, Jan 21, 2016 at 5:40 PM, Brian Clapper <notifications@github.com>
wrote:

> I don't really have any ideas, I just reviewed it for any glaring issues,
> and this is one. I can't merge this the way it is. If the check needs left
> out until something else can be done then I think that is best for now, get
> the other ones in and drop this one.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vventirozos/nad/commit/f5257b039cd357d98637336482229b7659f40636#commitcomment-15588690>
> .
>

--
Vasilis Ventirozos
OmniTI Computer Consulting Inc.
Database Administrator


likewise this would need logger


Word.


We should change this so that the subject line is `bootcamp project ug`


This is making travis fail now which is weird because PR passed travis - the check_clean test is failing here: https://travis-ci.org/intel-iot-devkit/mraa/jobs/103870863

Any ideas?


"clicked" usually infers a mouse event, can this be `_checkTabKeyed`?


Does this mean that column alias can't be (reliably?) used in a group by?


Is there a guarantee that the region contained in a description does not contain spaces of should this method call the overloaded method below?


- *TL;DR* is just the *abstract*
- for *exercises* we need to take a closer look

Please use English.


undetermined


"Are you sure you want to continue?"


Method as a whole. I was not sure I'll be able to check if the tests cover the new stuff sufficiently and therefore I have simply written noted it down.


this method doesn't seem to be used. Delete it if it isn't.


An update, I've made some additional changes to provide more options for
the format (including generating a table of contents). I also had to make
some change in RStudio to accommodate previewing in a temporary directory
(a build with those changes should be available tomorrow).

Note that until the new build is ready when you do a "Knit" within RStudio
you'll actually see a "markdown preview" rather than an html preview.

On Thu, Jan 21, 2016 at 6:03 AM, JJ Allaire <jj.allaire@gmail.com> wrote:

> Great idea re: the html file. I tried this out and there appears to be a
> bug in RStudio that prevents documents generated in the /tmp directory from
> previewing correctly. I'll fix this bug then update github_document as
> suggested.
>
>
> On Wed, Jan 20, 2016 at 4:04 PM, Jennifer (Jenny) Bryan <
> notifications@github.com> wrote:
>
>> That is terribly exciting! Yes it works very well.
>>
>> Does the html file have to be left behind or could it be written into the
>> session temp directory and viewed from there? Then you don't have these
>> lying around your Project, needing to be gitignored.
>>
>> This would be perfect as the default document format in
>> devools::use_readme_rmd().
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/rstudio/rmarkdown/commit/62cc240a17ab655b50e823da39e7bcc5f2dbdbb2#commitcomment-15569462>
>> .
>>
>
>


check if cdata->url is valid.


downgrade?


Thank you for PR
It is test failed on below case
```C++
state["value"]["abc"] = 1;
state["value"]["other"] = state["value"]["abc"];
TEST_CHECK(state("assert(value.other == 1')"));
```
value.other is nil
I think user watns to ”value.other” is 1,but it is changed in reference only.


Don't forget that the pathing is set up to work with the clientlibs directory AEM exports stylesheet to. It's the location of the stylesheet that matters here, the clientlibs folder is not going to be in the root folder. So this path you changed to will be looking in the clientlibs folder for a /content/ folder to go into which is actually 3 directories up as written in the original.


:-1: don't fall back to a string.  instead:

- if `tappedItem == nil`, return early
- if `tappedChild == nil` omit that entry from the notification's `userInfo` dictionary

I'd have to confirm, but i'm highly suspicious that falling back to a string would only cause "unrecognized selector" crashes when we try to use the object keyed under `tappedItem` or `tappedChild`, as strings are not (implicitly) expected as the type.


`event` isn't being used, this will prob trigger a lint warning


where is workaround for removed SvgPathSeg ?


This line takes a string and splits it up into length character "words"
for instance, aaaaabbbbbcccccdddddeeeee to aaaaa bbbbb ccccc ddddd eeeee.
I could loop through a substring or slice and do it myself but this regex seemed simpler after I found it.
Thanks for the suggestion


you can use sol_util_replace_str_if_changed()


With the standard ordering, of course. I didn't completely think this through, just wanted a helper function and saw that the difference in speed is roughly a factor of 2.


В данном случае простой проверки на "object" не достаточно, необходима дополнительная проверка на массив.

Следующие проверки вернут одинаковый результат:
typeof {} // "object"
typeof null // "object"
typeof [] // "object"

Следовательно, если в твою функцию одним из аргументов придет не Массив, а например Объект, то она  сломается.

В JS есть стандартный метод для проверки на массив. Array.isArray, можно использовать его. Он IE10+.


so we need to make this an actual issue, but SpanState in general needs some work. ex currently everything is single-level, and I think we need more a stack model. had a chat w/ @kristofa about it.

To your point in general, OpenTracing has a mutable view of a span.. I was thinking about backing that with a Span.Builder. might be a similar concern here, as well.

We're likely to have some time to hack around on it here, which you might be physically close enough to? :)

https://docs.google.com/document/d/1CWiSc2PaKh9YAg5P5zE7S2q-RkpRv1kqQh_z-HefLng/edit#heading=h.mg12wnw1e6xd


It's something to do with CSRF protection, which I don't fully understand but added that because the Flask-WTForms designer/doc author seems to think I should.  See https://flask-wtf.readthedocs.org/en/latest/csrf.html

Not a great answer I know.


I have since updated it for variable names and readability, but at the time I was getting hung up on naming things and instead focused on the logic. Perhaps a bad habit I need to change.


Good refactor


lol


Thanks for the feedback :+1:


Can you add a reference to the definition of `Offering` in the TOSCA YAML spec, please?


hello,

i need to use the dss code, for a specific implementation...
so..., i have the following question:

is there any particular reason for the validation web service is commented, besides large files problem ?

relatively for the problem of large files, i would suggest  the following solution, transfer all the file to a local folder in web server, but in chunks. After the completion of all file download, you can treat them ...

best regards...


Head and tail in functional languages always mean the first element and the remaining elements. Here that's not the case; tail means the last element. Please rename them to something like ``initial`` and ``last``.


Can you add a reference to Offering defined in the TOSCA YAML spec, please?


@txuki2005 you should really be thanking this guy! http://git.kernel.org/cgit/linux/kernel/git/jaegeuk/f2fs-stable.git/log/?h=linux-3.4.y  I'm just grabbing stuff from there and fixing any merge errors.


This needs to be removed as it's no longer being used


Actually, we should just remove all the manual retains & releases here. The association policy alone is enough to determine ownership semantics, as demonstrated in the examples on the relevant [NSHipster article](http://nshipster.com/associated-objects/).


Please, define the proper indentation here.


has to be a @POST?


FYI: dns-01 is now available on the production server! see: https://community.letsencrypt.org/t/shouldnt-verification-via-dns-record-be-a-priority/604/45


thank you for the docs effort :heart:


an empty string means bad request? should probably throw instead..


Oops. Temporary solution that must be reverted. Thanks!


@stigger are you sure you don't have two instances of NepTunes turned on?


Use PRs :P


@ngaskill - Note: our header ID generator isn't smart enough to handle link syntax in headers. We have to keep the header text very plain. It didn't cause any issues in the knowledge base ... our header ID generator task was just generating duplicate IDs. :-)


TeamCity Civic Assembly :: Build backend + frontend [Build 19](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=412&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 60 (3 new), passed: 1632, ignored: 12 Build time: 00:20:40
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1453391121981: <no details avaliable>

Basic Event Categories CRUD Actions: Should Delete an Event: <no details avaliable>

Basic Events CRUD Actions: Should Delete an Event: <no details avaliable>

Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

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


Do note that this translation is **NOT** done by me, hence welcome to send pull request to correct it. Keep in mind that **this is not the official way to submit localization**, but we will submit it manually soon or later.


without any security on the api this is pretty scary..


This request bean could use the enum too, could it not? http://stackoverflow.com/a/20421494


pps will have a look tomorrow to see if any short cuts come to mind (ie ways to swap the model regardless of future work like stack-based span state)


This line is superfluous as Travis does `npm install; npm test` by default.


I didn't make to get already selected entities, i try from storage and then manually test with some dummy data. Also i look at that EntityReference class to figure out how to use parts of that code for this plugin.


Maybe I will learn it sometimes ;)


И еще одна тема для размышления. По желанию. Чтобы не плодить такое кол-во if-фов и сделать код более читабельным, можно попробовать использовать инструкцию switch.

https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Statements/switch

Например:
var typeA = typeof(a);
var typeB = typeof(b);

switch(typeA) { ...}

Это по желанию)


:+1: I like this so we can check from any inherited child class without having an instance first.


nice! not as far as i'm aware. every time i've seen it it's been a bit [clunky](https://github.com/dimagi/commcare-hq/blob/master/corehq/apps/reports/tests/test_esaccessors.py#L48-58). it's kind of the nature of the fact that we usually to transform the model before sending it to elasticsearch


Thanks Dude, I was having serious issues with aggregate targets in my projects.


Absolutely.


It's up to you, but `karma.conf.js` isn't strictly necessary as that is the default Karma configuration file anyway.


Why so strange logic? Related #2634


Will this change to `mocha` at some point so the Node.js tests can be run on the command line?


in this way, it is always true


Still have to work on the removal of modules from timeslots and rooms.


This change was mainly done because some of the new ChampionMastery methods can return either a 200 response with a proper body, or a 204 response without body, both of which are perfectly successful results.

With our old way of handling requests (with a static Request.execute()) we couldn't reliably parse the result, since a returned null from Gson's JsonParser method could also mean an error.

My first idea was to simply create a new class RequestResponse, which contains a responseCode and a responseBody, and return that RequestResponse instead of only the body.
But with major changes having to be done anyways, I decided to completely rework the Request class, making it instancable, and have it do the entire request, including its preparation and parsing the result. On parse errors, RiotApiExceptions are thrown as always, and if the API responds with 204, we simply return null (the good kind of null!) upon returning the dto.

Since the Request class now does most of the work, the individual APIs are much easier to implement or modify.


no


Hi


void 0 === undefined ?


We only accept those types? What about the other ones?


We considered changing ``fileutil.make_dirs`` to do the ``chmod``, but that won't work because we need an extra argument to indicate the highest directory that we might want to ``chmod``.


OK, I keep forgetting that hash returns NOTHING when key not found.

But your change is not 100% compatible as if the hash happens not to contain 'bind' or 'value' keys, it's still assigned to 'val' variable (while it should error out, imo).  Also, it doesn't work properly when 'value' in hash holds NOTHING.


I need a fix of that broken sickrage, please disable this pip updates and installs until you fix this. Or maybe i will need to give a try of the alternate project of the other developers...


Could you please share on github a simple application that reproduces this issue?

Don't worry about a test  case


Ok, I will just remove so. I detected it was causing a memory leak and supposed the CFBridgingRetain calls were added on purpose for some reason.


@Miccia94 resync force pushing fix now


This also leads to redundant code: we pass a `Blob` object that already has a service reference, but to execute the batch we still need the service reference:
```java
Blob blob = Blob.of(storage, ...);
BatchRequest req = BatchRequest.builder().update(blob);
storage.submit(batch);
```


rtfmmf ;-)


yep. waiting on Amit to do the security.


Should be:  if (nCanVetoStep>1) { ?
All the tests are done on nCanVetoPT


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


Cast to uint8_t here, Sachiel made a commit adding these casts in this type of function.
Some compilers might warn if the cast is not present.


Chuck check ur skype!. And good work gents :)


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/26175/testReport/) for this commit**


ytsearch doesn't work again...


I made some changes to the halo function so you can select to save or just show plots (mostly for my own benefit)

I had to edit your loads function because if you set variables in a function they don't become global. You're apparently meant to avoid forcibly setting global variables so i defined an object (simdata) which you can just select the 0,1,2 element of to get the variables. We could change this to a dict?

anyway hope thats ok? i can't work this damn thing so i don't know how i do the pull request thing??


Electric-Service-Refactor (EnergyArchmage) - x86_64-MacOS-10.9-clang: Build Failed

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


Ok, I will change this part so, thanks for the clarifications.


I've checked this commit, again. `npm install` is ok. No changes required.


looks good to me


Maybe it makes much sense to be a "void" @POST request, since it is an update request?


@zakjan  Awesome, great work!


i changed my username and password... tested here that my git was reconnected to my github account


@vcunat I didn't realize they would retire old releases so quickly! Perhaps we should consider updating the KDE packages in `release-15.09`.


Without `<br/><br/>`, there is a single empty line there, which I find fully acceptable. With `<br/><br/>` there are two empty lines.

Generally `<p>` is the usual JavaDoc paragraph delimiter.


112083967_VentilatedSlab (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1508 of 1508 tests passed, 0 test warnings)

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


I have two separate loops because `concat`ing the arrays creates a new array. I didn't want to allocate a new array with the same values for each batched ground primitive. If we don't care, I can change it to use one array.


Why it's necessary to change tables names and drop indexes?
For me that leads to an error "no column with name 'objectClass'" and differencies between actual schema and annotations/


What this allows? and what will be the purpose/advantages inside the plugin?


This was rather to document the intended semantics, than to ask you to implement any constraints.


Both TB1 and TB2 have columns name TB2_ID.  It seems reasonable to produce an error when using an alias named TB2_ID in the GB clause.

If there was only one column named TB2_ID, what would happen?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1209](http://teamcity.monogame.net/viewLog.html?buildId=13831&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:29:13


This looks like it was added to the wrong commit.


vamos a refactorizar las pruebas, tambien TestCliCmdDeploy está mal hecho.
Los UTs de Crane no deben usar mocks de http, solo mocks del stack-manager.
En el UT de Cli solo queremos comprobar funcionalidad de este archivo y no hacer pruebas de integracion que apunten a go-marathon.


@benlaurie Yup. Thanks for the catch. Fixed in 652fb5a25c89c9574e287a98c97d719e9c8c7eec.


We should setup Travis CI for our github repos, so we can catch stuff like this in the future. I'm going to ask Amy to that as a task to JIRA.


Note Respec takes care of CSS and will update when the TR redesign is deployed. Having additional CSS in the doc that aren't managed by Respec will result in duplication. We should only put in stuff that goes beyond the base styles Respec adds.


Oh, and if you change something in the combined.scss remove it and fix it in the component scss file directory so we can get rid of duplicates. :+1:


- [ ] JSONは「フィールド名」のままでよい気が・・。 [RFC](https://tools.ietf.org/html/rfc7158) 上では、「フィールド」でも「プロパティ」でもないみたいです。「member」と表現されているように見える。 ＋ 今回の対応(`@JsonView`対応)と関係な部分への横並びは別issueで対応した方がよい。


Ooops, bogus commit msg.  Removing {emailcloak=off} from PDF's.


Thank you, Kartik! When v4.2.9 is about to be released?


@bgerstle could you give me an example on how to use this attribute? I never used it, I always have use the Core Foundation way.


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


Thanks

Change closeDownloadsShelf has been merged. For another change, please try to use it in your own settings(`se` to open it).


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1241.f4adac422e8e29637463cd50bb75830103c10eb0](http://build.palaso.org/viewLog.html?buildId=68320&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 842, ignored: 7 Build time: 00:05:57


Don't repeat yourself. (DRY)

It'd be nice to file a GitHub issue to keep track of removing this hack, and to refer to that issue anywhere the hack needs removing. It'd also be nice, for those who can't use GitHub or see git commit messages, to have the issue description (once) in this file.


I guess you have a similar problem to #8289? If not, please open an issue and paste the command you use and the full log.


Hi jstl,

We rewrote the signature webservice to be included as a dependency in other project(s) (SOAP or/and REST). To do the same stuff on the validation webservice, we needed more time.

We are currently working to implement the [latest validation standard](http://www.etsi.org/deliver/etsi_en%5C319100_319199%5C31910201%5C01.00.00_20%5Cen_31910201v010000a.pdf). The next version should be released in March with this new implementation and new validation webservices (REST and SOAP).

I hope this will help.

Kind regards,

Pierrick


Can we add please `"since": "2016-01-21",`?

It will be used soonish


The indentation in this commit seems to be incorrect, it seems you are inserting actual tabs while the original file uses two spaces.


@justlizzay So that weird height that started showing up in the cards, that was this. If something is commented out and you're cleaning up files they should just get deleted. This added it back.


Actually I think it is possible for any contributors to make changes to the areas (not the geometry though). See for instance the history of http://www.camptocamp.org/areas/14431/fr/prealpes-fribourgeoises-bernoises


fgt


Great! :+1:


ack


Is this change really needed? AFAICS, since b is unsigned, this will do zero-extension already, so there is no need for that bitmask? Same for the unsigned int version below.


That would've been to easy :D Already tried something like that but it didn't work. This is the result from your call:

![image](https://cloud.githubusercontent.com/assets/2085721/12485664/96b90954-c05f-11e5-81e4-c4b8f45a4605.png)

I've read that I should put the `composer.phar` inside my `php_5.5` directory and change the path in the `composer.bat`. I copied composer.phar to the directory and now was able to run composer with

`php C:\xampp\php_5.5\composer.phar update module/name --ignore-platform-reqs`

Just removed the space and added a slash for the directory, added the parameter because it's still needed due webserver still running 5.3 - with this only composer is run with 5.5.

No need to edit the .bat file and change something there, just copied the file and changed the call. Got it working, thanks :)


`Visibility visibility = Visibility.valueOf(request.getVisibility())` - i think that care needs to be taken for request.getVisibility() being `null` or something unexpected.


查阅董叔日记，hhhhhhhhaaa :)


Yup, that's a mistake.  Will fix in a moment.


In v5, it's not possible to create new areas, even for moderators.
But maybe having this possibility in the API would indeed be interesting (most likely restricted to some roles).


please remove the period


aha! this much better. thanks!


Electric-Service-Refactor (EnergyArchmage) - 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)