Hello @OJ, thanks for your feedback. I was testing using the reverse_tcp as well as the _uuid version and the bind_tcp and all are commented out. I am getting the same result that @mubix in this video https://youtu.be/bkfgPawWlQs?t=5m9s


otvorene viticaste zagrade su uvek u istom redu gde su kljucne reci if, else i elseif


@ovidiusas , if NULL is allowed in "domain", maybe the default constraint should be removed or switch to NULL.


首页


Now tests form other PRs are crashing, can you fix the tests @pablopi  please?
https://travis-ci.org/netbeast/dashboard/builds/114463359


za JS koristimo coding style gde ni jedna otvorena viticasta zagrada ne ide u zaseban red


visak else


1. Copy src/public to src/admin and src/web
2. Write  deploy.php


Just a quick remark. Since that change, my prometheus install consume all the CPU available (200% on my 2 core machine).


对函数进行性能分析和计时统计


Why not destructure to `service()`? :cat:


can't use self, outside a function : )


razmaci izmedju =, +, -, itd...


unack


It's ok that anchor to be kind of String instead of Time?


Probably want to change the `type` too :)


Probably want to change the type too :)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/3302


+git config --global http.cookiefile ~/.gitcookies
 +


unack could you do a proper rebase? we generally never want merge commits :)


TeamCity jonas-requests :: WindowsBuild [Build 17](https://build.neohq.net/viewLog.html?buildId=568479&buildTypeId=JonasHaRequests_WindowsBuild) is now running


Not sure why my git decided to push two separate commits and use the description from an old random commit..

Anyways, this commit is suppose to be combined with: https://github.com/bob7l/HawkReloaded/commit/ba659492fcb7c454b39379c3bbf4b7b172f15399


This is the .desktop of the Cura repository, where cura_app.py is correct. The one you are referring to comes from cura-build, which is specifically set up to handle cura in /opt/cura.


PTAL @ngaut


in the docker image its also just /worker_ui.js


kod ovake postavke za update metoda mora da pretrazuje niz i slicno.
ako bi stavio da je original = angular.copy(task) a $scope.singleTask = task onda u update metodi ne moramo da pretrazujemo niz i svapujemo taskove. original se korisit samo ako user pritisne 'cancel'


Yes, I manage my own nightlies in my PPA. (As @Ghostkeeper started signing his commits with gpg launchpad isn't able to import the git repository anymore. However I made a manual sync last weekend.) I don't see any reason in CMakeLists.txt, why "cura_app.py" should be renamed to "cura". If it is done in any build script, eg. https://github.com/Ultimaker/cura-build, then it should be applied into the CMakeLists.txt.
https://github.com/Ultimaker/Cura/blob/2.1/CMakeLists.txt#L58


`if ()`


I think cura-build installs cura_app.py to /opt and a shell script called `cura` in /usr/bin, so @ckielstra is in the right then. We'll have to check that.


Having a separate `ObserveRelationStore` sounds good, because it is easier to persist (the other open exchanges can be lost on a reboot).
Moving data into the Request is a step back. We originally had everything in the request and it was inefficient and had a lot of loopholes. The overall Exchange is definitely what we want (keep deduplication etc. in mind). What exactly is "heavy" with the Exchange? We could have a `cleanup()` method if there is useless data.
However, I see that there is no need for deduplication of notifications: you just ACK them if there is a relation (RST otherwise) and you filter by the Observe sequence number...


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


Entries are never removed explicitly, so this constitutes a memory leak. (Up to the point where the instance is discarded.)


Yes, I also just found https://github.com/Ultimaker/cura-build/blob/master/cura.desktop


This is a style I often use to "simulate" RSpec's lazily evaluated `subject` or `let` blocks in Test::Unit/MiniTest-style tests. However, I can see that if you're not familiar with it that maybe it's a bit confusing.

One option I considered to address your comment was to move the definition of `#define_question` to the top of the test class. However, I thought that still might not address your concern.

So instead in this [fixup commit][1] I've got rid of the `define_question` method and instead introduced a `setup` block which instantiates the question and assigns it to the `@question` instance variable. I actually prefer this approach, because I think it reduces the level of nesting needed inside each test and makes it clearer what the method-under-test is.

Hopefully it's also a more familiar approach and thus less confusing.

I plan to update the commit note of the original commit once it has been fixed up.

[1]: 47f773e6c8896db4a91f0c0141f6d89473137e72


`while ()`


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

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


Test pull request comment


i guess this line should contain `@provider` instead of `@constant`?


You'll notice that in that video the comment uses a `#` instead of `/*`. This payload should now result in something that does work by default. If you're having issues, can you please show me reproduction steps?


Ja dat komt later wel.

Ik denk dat het logisch is om dat te doen in de constructor van een PrintableLayer.


What about doing a rebase of your branch with the latest from boostrapUI branch instead of a merge?
In this way we avoid merging what is actually already in the branch.


i guess this line should contain `@provider` instead of `@controller`?


@bpot would it be possible to also get a link to https://github.com/zendesk/ruby-kafka from here? It's a pure Ruby client that Kafka 0.9 compatible and includes a Consumer Group implementation.


I added my comments inline. There are several topics that should probably move into issues:
- ExchangeStore to encapsulate(?) the lookup hashmaps
- Separate ObserveStore, which can then be persisted easily
- Optimize Exchange objects (vs going back to more info in the request object)


update


i guess this line should contain `@service` instead of `@controller`?


imo it's not necessary
this solution loses injection context


Ah right that's why the 2.1 desktop file from the installers doesn't work any more! I didn't realise the version in cura-build should be different from the version in Cura. I'll fix that right up. Thanks for this conversation, guys.


Up


up


A typo in my commit :
$class should be ComMycomViewTest_underscoreHtml
    instead of ComMycomViewTestUnderscoreHtml


Right ! Cool :+1:
<sub>(It was a spontaneous reaction -  I should have known better and search first :smiley:)</sub>


额，破坏了精心设计的内聚 :(


Shouldn't the entity and database table have sensible defaults for these so that we don't have to set them when creating new donors?


Not yet. the refraction is still wrong.
I will fix it very soon:)


GraphicsClass


test


Ah god, it's a difference of opinion on linux :/

On mac its like "please give us back our menu"
Not sure what to do with you lot :/

What if I hid the edit/view/playback menu thing?  Would that make it look better?


This can be written like this

```python
text = self._buffer.get_text(*self._buffer.get_bounds(), True)
```


police.come here..


Thanks! I opened an issue https://github.com/mattvh/jekyllthemes/issues/285


Yes, I agree - are we going to change them for this ticket? Was it done for Location?


While install sails via npm, this bug is still exist;


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


Thank you :)


This method is still used:

`PHP Fatal Error: Call to undefined method User::isStaff() in /usr/wikia/slot1/10183/src/extensions/wikia/ProtectSiteJS/ProtectSiteJS.class.php on line 76`


Yes, since you are passing it as an argument at object creation and you don't really need it to be `Time` (it would contain useless information, all you need to know is the current date string-formatted as `YYYY-MM-DD`)


Aha, 'next' was the keyword I was looking for! Much nicer! :)


I can't speak for everyone, linux is all about choice, hence the config options I already added etc. Everyone likes it different. Hiding the menubar is of course a welcome option (can also have a shortcut to toggle it or something).

Personally I much prefer the "non-native" borders, check how pretty it's when maximized:

![screenshot_2016-03-08_10-42-29](https://cloud.githubusercontent.com/assets/2041118/13597761/7e530970-e51a-11e5-8a9d-887d5fe82643.png)


Супер!


Coooool!


A user chosen amount of games*


Not bad!


police.come here.this man


Nice catch!


hi guy .


Why the initialization? Isn't v just a temporary variable to reference times in a dictionary?


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


It is more clear when you write this in PR comment. Here just comment line with '@Test', there is no need to comment all lines of test.


опечатка в "Обобщенный алгоритм выгляди так:"
в конце некоторых предложений отсутствуют точки


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


It wasn't done for location, but I think that all of the location fields were specified in the spreadsheet so there was no need for defaults. I think that if you are having to make a change to provide defaults anyway, we might as well do it correctly now.


Cool, I think this looks more consistent with the current state of the other tests throughout smart answers.


I am a nice comment


+1 for that approach.


DOOR_TYPE_ROOM typo comma " , "


Very good!


Hello.

Sorry for my dark, but it's commit fully killed DMA API and I can't compile my driver which this API.
Is it right?

Thanks


I made a similar change in `ZSTD_validateParams()` which automatically change the min value of hashlog3 when `minmatch != 3`. So it does not cost much memory in this case.

Well, that's another thing that it will be possible to remove with this parameter ...


Yes.I built yavijava-6.0.01b1.jar out of the branch "spbm_client_changes" of atokasema/yavijava and use it with hello_world_yavijava_spbm in one project. Then run the main function in main class in hello_world_yavijava_spbm, and came out the error "com.vmware.vim25.SecurityError".

And I am so numb of how to take the vcSessionCookie in yavijava. Or use the branch "sso_support" to build jar to get ssoToken?


ktu e perdor @NotEmpty


wut?


But in `_offset_anchor` you're using it as Time object, since you call `strftime` function.


> (target.index('Terms.Participants_name') * 4) |

どうも`'Terms.Participants_name'`を指定すると`different index column`エラーになってしまうようです。

```
ArgumentError: different index column: <Terms.Participants_name>: #<Groonga::IndexColumn id: <277>, name: <Terms.Participants_name>, path: </.../daimon-news-multi-tenant/groonga/data/test/test.db.0000115>, domain: <Terms>, range: <Participants>, flags: <WITH_POSITION>, sources: <name>>
```

あとでRroongaの中を見てみます。


ktu je tu e shti me zor me pas parent, per qfar arsye?


Ternary is good for simple statements. This one i hard to read.


Can't we wrap all the settings code into a local static function with three parameters, v, &timeref, name? The all look exactly the same to me.


something missing here?


This is not acceptable. First, it would not work for anyone besides root. Second, `-i` to `sed` is not portable. And third, most importantly, doing something like this is just something we don't do in completions (we also have this rule codified in CONTRIBUTING.md, "Do not write ...").

Isn't there a command line option available that could be used to override the config file silent setting?


kudo qe ki string, nese te vyn me pas vlere duhesh me perdor @NotEmpty


As @SnShine said in a module is a best coding practice use relative paths. If you need some functions of a certain module of the package you can import it like this:

```python
from aimaPy.utils import *
```

But the notebook file have to be in the same directory of the **aimaPy** folder or, if it is inside the **aimaPy**, you can import directly the utils like this:

```python
from utils import *
```
Because the *notebook file* and *utils.py* is in the same directory. This is a behavior of python, iPython notebook will never import relative modules. If you need to access the package from the notebook wherever you are you have to install it in your python VM.

@jeff3456 if I can give you a tip, use [Jupyter](http://jupyter.org/) notebook instead of iPython because it will be his successor. Remember also that for the moment the package is not recorded in PyPy index, so you can directly install with *pip* right now but you can install the package with:

```bash
# change with python3 if you have both VMs and you want to use the 3.x version
python setup.py install
```

Depending on the OS and your configuration should be enough, otherwise if you have like me more python VMs (2.7, 3.4, 3.5 etc.) you can [add a kernel](http://jupyter.readthedocs.org/en/latest/install.html#installing-kernels) like this:

```bash
python3 -m ipykernel install --user
# or if python exec is the 3.x version
python -m ipykernel install --user
# or for a different VM (for example to have an ad hoc environment for testing)
source /path/to/env/bin/activate
python -m ipykernel install --user
```


meh, dont read travis docs or buggt travis?


why is register on `Interface` here but on `IResource` elsewhere?


problemi qe te paraqitet me i than @NotNull 1 property qe osht @Id osht se tani user-i gjith duhet me shkru vleren e id-s edhe kur bon save


En tajunnut puoliakaan mutta ei se mitään. Jos tarvitaan jotain speksiä, kertokaa mihin ja miten sitä minnekin tungetaan. Nimim. Git-SuperUrå


Can this be infinite?


I'd rather say here "batches of N" instead of "steps of N".
Also there is apparently no consistency spacing around concatenation operator in the code base but I personally would stick to `'foo' . 'bar'` instead of `'foo'.'bar'`. But would not insist on it here given there is no consistency.


fast-forwarding riak_ts-develop to borshop-integration-1357-at/flexi_keys = f5430ebc


This is the code I made to handle what you propose:
```
## Display a link to the Sandbox application if it is installed and if the page has not been removed.
## TODO: if this panel is empty, display a message that explain what is its purpose, ie. a place
## where administrators can put some links to help navigate through wiki(s).
#if ($services.extension.installed.getInstalledExtension('org.xwiki.platform:xwiki-platform-sandbox', "wiki:$services.wiki.currentWikiId") && $xwiki.exists('Sandbox.WebHome'))
  * [[$services.localization.render('xe.panels.quicklinks.sandbox')>>Sandbox.WebHome]]
#end
```

However, having such a code in a panel that is mainly designed to be edited by administrators might not be a good idea. If the content looks too technical, the administrator could be afraid to touch it.


Done. See commit: https://github.com/jembi/bsis/commit/d8862357d16d670db504d30e3d97a9bddb367ba4


あぁ、参照型ですか。
私が間違っていました。

Groongaだと`sub_filter`を使うところなんですけど、Rroongaでも書けたかはすぐにでてこないので、元のままでいい気がしてきました。（重みが雑だけど。）


Very good!


No. This issue is brought by the rebase patch, which makes title visible on Mac. See the function of the same name for `AppNSWindow` above.


Can you share your configuration?
Anything unusual in the log output?

On Tue, Mar 8, 2016, 10:24 AM Jean-Daniel <notifications@github.com> wrote:

> Just a quick remark. Since that change, my prometheus install consume all
> the CPU available (200% on my 2 core machine).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/prometheus/prometheus/commit/5b9e85e5560dbeda36fe17332d7db630ddf80788#commitcomment-16556793>
> .
>


I agree with @tbeu here. If you don't use an annotation then it's probably best to simply get rid of it. Rather than havin ambigious or outdated information in it. Btw, noone keeps you from introducing semver for your library incl the meaning.


It is better to use the `isTRUE` function:

    if (isTRUE(compute)) {

or more compact to simply say

    if (compute) {

if you are confident that `compute` will not be susceptible to bogus input by a user.


There you go the rest is pushed aswell @fenix008 ^^


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


Wouldn't it be cleaner to retrieve the commit hash by looking up the tag? Happy to do a commit for this if you want.


Thanks. Yes, these setters badly need tests…


Like!


2 file .jsp này là source code. Cần có license information trên đầu.


This one caused `PHP Warning: substr() expects parameter 1 to be string, array given in /includes/Sanitizer.php on line 635` - reverting


I don't think it changes that much from what was there before in term of complexity. If you think it's an issue we could include another page and we'd move this code to the other page, to make it less visible. Something like:

`````
## Display a Sandbox link if the Sandbox Application is installed
{{include reference="Panels.QuickLinksSandbox"/}}
`````


+1

Love unit tests.


Create array by scanning the input and print it by reverse order.


Another idea is to write an Admin UI for this panel


The condition should rather be something like `$stepSize*$iteration >= $snapshotCount`. With the current condition I got the following output
```
```
which reminded me of Alexey Stakhanov.


no? what is missing here? This is just ES6!


second


qqq


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


You are not using the result of the `try` function so you might as well leave it out. Also, since `exists` returns `TRUE` or `FALSE` only, you can simplify this `if` statement as follows:

    if (!exists('systemId' , envir = .BenchEnv)) {


typo


Can we assume buildDriverResult is always present for a failed build?


There is still an `#include "stdint.h"` in bycython.cpp:313. When I changed that to `def.h` as well, the package built fine and I was able to use it.


@Ghostkeeper The problem with launchpad is that the imported code gets converted from git to bzr, but bzr-fastimport is not able (at least on the launchpad servers - locally it works) to convert the signed git commits into bzr commits. Bzr even has signing support - however, for a reason it does not work.


This comment may no longer be valid here.
Does executing tests from Android Studio invokes `connectedCheck`?


Thanks for the updates Bob.

All but the feature flag are good to go as they are.

The feature flag needs some more consideration. If I understand your code correctly this is what happens now the first time you open a data file in gnucash 2.8.x and newer:

1. The end of file-open will call gnc_account_imap_convert_bayes
2. gnc_account_imap_convert_bayes will convert any old bayes information to use guid
3. gnc_account_imap_convert_bayes will then set the feature flag, regardless of whether or not it really had to convert any bayes information
4. setting that flag will dirty the book, so the user will be asked to save the changes on closing

So in the end a user will always end up with a data file that older versions of gnucash refuse to open (due to the feature flag that will be set unconditionally). This is not what you want. The book should only be marked with the feature flag if it really contains guid based bayes data. Which means you should set the feature flag in the code that actually writes a guid bayes kvp. At first sight there are two locations in your code: gnc_account_imap_add_account_bayes and change_imap_entry.

**Sidenote** the parts in gnc_account_imap_add_account_bayes and change_imap_entry that do the actual writing are very similar. A little refactoring to introduce a gnc_account_add_map_entry similar to gnc_account_delete_map_entry may allow to reduce some code duplication.


This should fail as long as TICS_PER_MS > 2, which is by default 1000. Should we comment or test this condition -- that it works when TICS_PER_MS >=2 ?


Thanks, @crhym3!


Please insert license information into header.


TIOCGWINSZ does not work for 32bit system:

 expected `u32`,
    found `u64` [E0308]
src/size.rs:26         if ioctl(STDOUT_FILENO, TIOCGWINSZ, &mut size as *mut _) == 0 {


err msg: ... from pool
I'm nitpicking : )


ご確認ありがとうございます！

> （重みが雑だけど。）

確かにそうですね。。。TODOにしておきます。


Fixed by https://github.com/blueboy/portalclassic/commit/e74eb7073669242c02d34e008082d3efd3be8bfc
In a nutshell, there was a conflict in the way the new loot system handles sending/releasing loots and the way playerbot was doing it.


Looks pretty good, just watch the indentations...Everywhere in Bitcoin code indentations are 4 characters.


@PierreBesson I made few minor comments above.
My main concern is error handling.
I wrote a unit test, I think it should be included in generated code.

~~~java
package com.mycompany.myapp.gateway.responserewriting;

import com.netflix.zuul.context.RequestContext;
import org.apache.commons.io.IOUtils;
import org.junit.Test;
import org.springframework.mock.web.MockHttpServletRequest;
import org.springframework.mock.web.MockHttpServletResponse;
import springfox.documentation.swagger2.web.Swagger2Controller;

import java.io.InputStream;

import static org.junit.Assert.*;
import static org.mockito.Mockito.mock;
import static springfox.documentation.swagger2.web.Swagger2Controller.DEFAULT_URL;

/**
 * Tests SwaggerBasePathRewritingFilter class.
 */
public class SwaggerBasePathRewritingFilterTest {

    private SwaggerBasePathRewritingFilter filter = new SwaggerBasePathRewritingFilter();

    @Test
    public void shouldFilter_on_default_swagger_url() {

        MockHttpServletRequest request = new MockHttpServletRequest("GET", DEFAULT_URL);
        RequestContext.getCurrentContext().setRequest(request);

        assertTrue(filter.shouldFilter());
    }

    /**
     * Zuul DebugFilter can be triggered by "deug" parameter.
     */
    @Test
    public void shouldFilter_on_default_swagger_url_with_param() {

        MockHttpServletRequest request = new MockHttpServletRequest("GET", DEFAULT_URL);
        request.setParameter("debug", "true");
        RequestContext.getCurrentContext().setRequest(request);

        assertTrue(filter.shouldFilter());
    }

    @Test
    public void shouldNotFilter_on_wrong_url() {

        MockHttpServletRequest request = new MockHttpServletRequest("GET", "/management/info");
        RequestContext.getCurrentContext().setRequest(request);

        assertFalse(filter.shouldFilter());
    }

    @Test
    public void run_on_error_404() {
        MockHttpServletRequest request = new MockHttpServletRequest("GET", DEFAULT_URL);
        RequestContext context = RequestContext.getCurrentContext();
        context.setRequest(request);

        // Simulate an HTTP error 404, this should probably be done differently in Zuul
        // according to SendErrorFilter
        MockHttpServletResponse response = new MockHttpServletResponse();
        response.setStatus(404);
        context.setResponse(response);

        // filter should not fail with an empty body

        filter.run();

        assertEquals(404, context.getResponseStatusCode());
    }

    @Test
    public void run_on_valid_response() throws Exception {
        MockHttpServletRequest request = new MockHttpServletRequest("GET", "/service1" + DEFAULT_URL);
        RequestContext context = RequestContext.getCurrentContext();
        context.setRequest(request);

        MockHttpServletResponse response = new MockHttpServletResponse();
        context.setResponse(response);

        InputStream in = IOUtils.toInputStream("{\"basePath\":\"/\"}");
        context.setResponseDataStream(in);

        filter.run();

        assertEquals("UTF-8", response.getCharacterEncoding());
        assertEquals("{\"basePath\":\"/service1\"}", context.getResponseBody());
    }

}
~~~


my thought is that if this can be run at any time to check a cluster's
health and it can technically be deployed in any project such as 'default'
we should ensure we're checking only for the logging pods.
On Mar 8, 2016 5:40 PM, "Richard Megginson" <notifications@github.com>
wrote:

> why do you need the grep here? what other pods would be running in the
> current project/namespace?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ewolinetz/origin-aggregated-logging/commit/fec4c4a3259b49942db3e3140d6f3e2d3336dfbf#commitcomment-16576613>
> .
>


I like that you added this and it would be cool if we could benefit from this for local development and actually populate the index. (Future work.)


it is passed to sem_init.c
But is it necessary to be public?


[![136 Mini Film Cameras in the Smithsonian Institution History of Photography Collection With Old Style Photoshop Filter](http://api.artsmia.org/images/119071/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119071)


It looks like the `clear` was misleading - it was flawed, because it may not "clear" anything. So I think your suggestion derived from that has the same flaw.

`needs_maintenance = set` may not set the geocache's flag but leave it unchanged, because it already was set, or because there already is a newer log wich reset it (logs often are backdated). The same applies to `needs_maintenance = unset`, which also may leave the flag unchanged in either state. So these option names are misleading.

What the logger does is _state an opinon_ about the state of the geocache, similar to ratings and recommendations. You won't rename the `recommend = true` option into `recommendations = increment`, will you? :)

Besides of that, the set/reset logic for the `caches.needs_maintenance` flag is implementation dependent. I explained to you how it currently works at OC.de because you asked, but this may change in the future or be implemented in another way at another OC site. Therefore I think it should not be reflected or documented anywhere in OKAPI.

Having said this, I think that `yes` (= true), `no` and `null` (= false) would be appropriate and backward-compatible names of these options.

`needs_maintenance_was` then becomes `needed_maintenance` - saying what the user expressed, not what the OC logic did. Though I don't like this past tense, because this information refers to the date of the log. In geocacher's logic, "needed maintenance" means that the cache was in need of maintenance _before he found it_ (which implies that the logger has maintained the cache!) The log comments also are written in present tense, the logger e.g. will write "the logbook is wet" or "the cache urgently needs to be maintained". So I think `needed_maintenance` will rather confuse developers.

(I see there already is a `was_recommended` log entry flag. I condider that confusing, too).


This is probably 3.21.1, according to the description below.


:+1: for added consistency here


Theme default is the same...

Theme dark:
![simulator screen shot mar 8 2016 5 46 42 pm](https://cloud.githubusercontent.com/assets/8075821/13620777/e472ce1a-e557-11e5-8224-76502b00342d.png)

Theme light:
![simulator screen shot mar 8 2016 5 49 13 pm](https://cloud.githubusercontent.com/assets/8075821/13620778/e79792b0-e557-11e5-95c8-4b9a79b875c4.png)


Looks good.


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


I wasn't quite sure how to indent these either. I wondered whether Rubocop might offer some suggestions but it seemed happy enough with this style.

You're correct in that the intention is to move this logic to the calculator so these multiple line conditionals shouldn't be around for too long.


No, por el momento no esta soportado.


This is packaged inside maprfs phat jar, not needed here.


No, but when you made the change initially, it wasn't defined yet in sem_init.  The `EINVAL` response should be the correct way of notifying users.


Not needed.


@adamjkeller I saw that in other examples. However, I felt it is not that easy to understand what is happening, if all the logic is in one line.

Probably, that is pythonic way of doing things.. Let me follow that. thank you


enhanced precision.


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


2 space indents


Ruby automatically returns the last value


The following doc(s) have been deployed to the [1200blogva](http://1200blogva-static-docs.pantheonsite.io/docs) Multidev environment:
- [source/docs/assets/images/bv-details.png](https://github.com/pantheon-systems/documentation/commit/ec69a827ee1b62bc228a5e2cf480f6c3df54e6c2/source/docs/assets/images/bv-details.png)
- [source/docs/assets/images/sftp-connection.png](https://github.com/pantheon-systems/documentation/commit/ec69a827ee1b62bc228a5e2cf480f6c3df54e6c2/source/docs/assets/images/sftp-connection.png)
- [docs/articles/sites/migrate/pantheon-migration-plugin](http://1200blogva-static-docs.pantheonsite.io/docs/articles/sites/migrate/pantheon-migration-plugin)
- [docs/articles/sites/migrate/export-an-existing-wordpress-site](http://1200blogva-static-docs.pantheonsite.io/docs/articles/sites/migrate/export-an-existing-wordpress-site)
- [docs/articles/sites/migrate/migrating-from-wpengine](http://1200blogva-static-docs.pantheonsite.io/docs/articles/sites/migrate/migrating-from-wpengine)


New line at EOF


Ahahaha bien vu !


Will this check work when, for example, we compile the runtime for PPC or MIPS?  Also, I think a fatal error might be a bit overkill.  Could we possibly return `ENOSYS` instead?  That would also be appropriate for non-Intel CPUs.


Correct. Fixing.


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


Ack on volatile loading but we still should flash the board after every successful testing so I don't think it will meaningfully reduce flash wear. On the other hand even the crappiest EEPROM has upwards of 10k cycles so I doubt we will hit that either.

Serialboot might be useful but artiq_flash should gain that functionality then. It will be useful elsewhere too.


Finally!

:+1:


App makes weather file paths absolute [here](https://github.com/NREL/OpenStudio/blob/73d012c9716c8ccf2d6d707fe932b1afd7ae2cbc/openstudiocore/src/openstudio_lib/OSDocument.cpp#L471) and [here](https://github.com/NREL/OpenStudio/blob/a477c2f00aa05e37813d0aedab3ddbaf8d2881be/openstudiocore/src/openstudio_app/OpenStudioApp.cpp#L196)


Thanks for the unit-tests!


@beanstown106 no joke...


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


Simply "Out-of-bounds read in Service Worker Manager" ?


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

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


this broke permissions for luarocks_aliases. fixed in 214e8341bbc8d95f90f1334ce1befe777136fb81


The LocationTabView makes the url relative in [LocationView::onWeatherFileBtnClicked](https://github.com/NREL/OpenStudio/blob/a477c2f00aa05e37813d0aedab3ddbaf8d2881be/openstudiocore/src/openstudio_lib/LocationTabView.cpp#L598), this is inconsistent with the way the weather file is handled when you simply open the osm.  I guess this means the app can handle things both ways?


Hello Venthur,

I've been trying to use mushu for my g.usbamp, but I have the following problems
>>> import libmushu
>>> libmushu.get_available_amps()
['randomamp', 'sinusamp', 'replayamp']
>>> libmushu.get_amp('gusbamp')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Python27\lib\site-packages\libmushu\__init__.py", line 143, in get_amp
    return AmpDecorator(c)
  File "C:\Python27\lib\site-packages\libmushu\ampdecorator.py", line 121, in __init__
    self.amp = ampcls()
  File "C:\Python27\lib\site-packages\libmushu\driver\gtec.py", line 60, in __init__
    device = self.amps[0]
IndexError: list index out of range

The amplifier is not found, why?


Wow, excellent, we need to ask vimgolf to add support for Kakoune so that we can brag directly on their site :smile:


somehow etckeeper decided now was a good time to reorder everything alphabetically... whatever; no functional changes.


srcDir  = [ 'api/src/main/java', 'spi/src/main/java', 'impl/src/main/java', 'ceylon/src/main/java', 'webdav/src/main/java', 'maven/src/main/java' ]
should work


Nice job noticing that 2*cos(45) is just sqrt(2), I forgot to evaluate it. Could you explain "eta" for me? I was just using r/4 < r*2*cos(45).


target the form with id `#city-form` instead of `document` which is your entire page. If you had more than one form in one page, this will definitely mess up.


It might be better to leave the default kept on the server-side.

I haven't checked how the interface looks so leaving this to something generic might make the interface odd.


Good job on the hw. A couple of things to consider: think about how functions can be used in order to make the code modular (more easily reusable). Keep up the good work.


sweet!


really digging the clean code, nice usage of `imageName` var

:+1:


guard is really cool btw:

```
func createPerson() throws -> Person {
        guard let age = age, let name = name
            where name.characters.count > 0 && age.characters.count > 0
            else {
                throw InputError.InputMissing
        }

        guard let ageFormatted = Int(age) else {
            throw InputError.AgeIncorrect
        }

        return Person(name: name, age: ageFormatted)
    }
```


Thank you. What do you think about going one step further and defining the value zero to represent the natural alignment? This would result in the value being zero in the common case irrespective of the access natural alignment, and might further help compression. Since the upper limit is defined to be the natural alignment this would only require an offset-by-one for the non-natural alignment encoding. Also happy to explore this in a follow up PR and to look at the actual compression difference it might make.


This is not 100% the case for multisite drupal install. Is this just for PHP, not explicitly Drupal? @AnoopAlias


This way we keep the syntax the same for all os's


by the way how did you find this project?


Looks good


Why did you duplicate [my PR](https://github.com/scottalanmiller/xenorchestra_installer/pull/1)? ;-P


finally worked


@slavapestov Whatcha think?


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


The only change I would make here is to add a comment saying specifically why we had to set the size of a "background image".


I think this should be a PR :)


@mholt I second this way.


Another way to approach solving the exercise at hand:
There are only a few lines of code that are modifed.

var cities = ['NYC', 'SF', 'LA', 'ATX', 'SYD'];
var cityName;

for (var i = 0; i <cities.length; i++) {
    console.log(cities[i]);

    $('#city-options').append('<option val="' +cities[i]+ '"> '+cities[i]+' </option>')

}

$('#city-options').on('change', function(){ //*why does this work if NYC is pre-selected on page load? Do we need to account for that?
    cityName = $("#city-options").find("option:selected").val();
    console.log( cityName );
    $('body').removeClass();
    if (cityName === 'NYC'){
        $('body').addClass('nyc');
        } else if (cityName === 'SF'){
            $('body').addClass('sf');
        } else if (cityName === 'LA'){
            $('body').addClass('la');
        } else if (cityName === 'ATX'){
            $('body').addClass('austin');
        } else if (cityName === 'SYD'){
            $('body').addClass('sydney');
        }
});


There is a subtle bug here now @jpogran / @glennsarti ... the final flush here could drop output because of the `0.upto(10)` now.  I would change `def drain_pipe(pipe)` to `def drain_pipe(pipe, iterations = 10)` ... and then pass in a larger value at this call site (like 1000 or something).

This is another reason we really want multiple threads / signals.


@TheGrits Finished! Everything should work properly now.


Good job on the hw. One thing to consider: how can functions be further used in order to make the code modular (more reusable). Otherwise keep up the good work.


LGTM, nice job


That code does not affect agent<-->manager communication at all. It is only related to the alerts based on the size of the log file increasing compared to the stored average.

Something else must have broken the agent<->manager communication.


I just use eta as an arbitrarily small number that is still distinguishable from 0. Epsilon is probably a better name for it. It makes a close approximation to a line.

r/4 is more correct mathematically as minute circles would go wrong with my version but actually they will with r/4 due the use of fixed point maths in the 2D subsystem.


Nit: indented too much


Somehow you got an old copy of this block which contains some, but not all of the changes I made in https://github.com/OpenBazaar/OpenBazaar-Client/pull/997

Specifically:
* http://www.reddit.com Should be `https`
* All anchor tags should have a `target="_blank"` attribute.  Written as `target=\"_blank\"` when escaped here as JSON string.

I can fix this later if you want to avoid more commits / squashing, but thought I should mention.


Please consider move the size_to_write and memoryview usage outside of this function. This function should be as primitive as possible and accept a read-only pair of <payload_string and payload_bytes>

This way, we enable the caller to have the ability to share a memoryview object across multiple invocations (by multiple threads) as necessary. Current code would create one memoryview object for every call, which is inefficient and the associated overhead is also counted into the benchmarking time.


Thanks, @rebeccawilliams!


This doesn't work yet. Will need to try modifying a bit more


In the future, here's how you would try to do your plotting with idash @romain-chiap.  You were close!

`dash.add(lambda: plt.plot(self.path[0,:], self.path[1,:]))`


Now code is compiled only for Intel that I didn't use any architecture guard.
If we will port to other architecture, then code review is necessary anyway.
Fatal error ensure that code doesn't continue, because it has no sense = no semaphore support.
I am not sure if POSIX standard handle this situation.


Extends Illuminate\Foundation\Application, achieve self application
Set publicPath, configPath and so on.
Set view ptah by change config/view/php


s/min_writes_allowed/min_writes_required


Initialized twice


It's written so that it supports setting them at runtime.


It's 2016


`(template.pattern)` is the same as `template.pattern`. I assume you meant `(template.pattern,)`.


Very minor: Some of the commit note contains longer lines than the rest of it.


`str` Python3, something something, grumble


This is specific to a particular solver, it should be an argument to the instance


But unrelated to this, you bring up a good point. We increase in the internal_options, but not on stats.c, so the new values would not be taken. Will do a PR to increase them in the code too.


This should be type LabelSelector


I committed this from my desktop computer because it told me that the local repo was 2 commits ahead of the GitHub repo. Doesn't look like anything is messed up.


Ah! Pode dar dicas sim , são bem vindas! Fico feliz pela ajuda!


This shouldn't be a pointer


[Datamining_Project_Spring16_Cancer_Diagnosis.docx](https://github.com/rajkumarthodeti/Cancer_Datamining_Project_Spring_16/files/164236/Datamining_Project_Spring16_Cancer_Diagnosis.docx)
Initial Version of project template


Datamining Project overview intial version


do we have a test for this scenario? if not, we should :)


Hate to bug you again but I think there is another 2.6.0 right above this line.


Configure Xcode to remove white space at the end of a line.


if you are in 3.5:

import typing

and install some library.
Or just learn D like I did!

I thought that was pure **ugly** Perl!


Don't copy-paste comments


Print adds the new line automatically


you probably want to normalize here; but it'd be worth trying to get a test

the problem would be if you have an impl with a header that uses a projection, e.g. `impl<T: Iterator> Bar<T::Item> for T`


(As we discussed on IRC, you ... may want to take the obligations that result from normalizing as "assumptions" as well, though I think they *ought* to be entailed by the "main" obligations you already have, or else the impl is not WF.)


(Modulo region stuff.)


Here is the link to the D website
[D](http:/www.dlang.org)


Neat, I figured it was just a small constant, but I didn't know that the 2D subsystem errored past some point. r < r(2^(1/2)) would work as well, would this prevent the error in an exceptionally small case? Or is it just due to floating point approximation? I really don't understand this language well, but I'd like to begin to grasp it better.


`!(node.layer is BackwardLayer) && !(node.layer is LossLayer)`


Holanda qué talca


Yes, I known about next change therefore first change was inconsistent.
I hope that now it should work.
Do you improvement of code if you feel that has sense.
I did quick code change to prepare integration to CRTL with CPU version checking. It can be improved.


@romain-chiap How does the `while` loop get exited with your current functions?  From the logic, I don't see how this line after the `while` loop, `self.setMotorVelocities(0,0)`, gets executed.


* Forward and backward instances have too much in common, may extract a common Instance class
* Runner has too much duplicate code maybe separate it into ForwardRunner and BackwardRunner and extract a common Runner class


This is a comment on the commit


Haven't I reviewed this already? Why a second round is requested?


Warnings of experimental features is nothing new, it'll be fine.

As for a part cache, it can't happen right now. The plugin is really a huge mess in many ways. particularity in architecture, making it impossible to added. it's a long term plan that would hopefully be aided by a certain library I want to use, but for now, if people want to use it, they'll just have to deal with it.

One way we can speed up the import would be to read that file only once and reference that data. We can't generate mesh for it (part of the issue above), but it would slightly help.

As for location, we'd figure that out, but I almost think the logoed studs were merged into the standard library.

So yes, go ahead and put a PR up. We'll work out all the issues.


Why not just `typealias ParameterDimesions = UInt32`


0.001 seems to fail so as soon as r gets that small it will go wrong anyway. Not really a problem for the things I print.


Check index < getProjects().size() ?


saw approval from JeetKunDoug
at https://github.com/basho/riak_kv/commit/b40628f9573f8e1caec29cfb1287de31f692c145


merging basho/riak_kv/feature/fd/hashtree-update-callback = b40628f9 into borshop-integration-1361-feature/fd/hashtree-update-callback


basho/riak_kv/feature/fd/hashtree-update-callback = b40628f9 merged ok, testing candidate = 1ab5f5fd


AtsPluginProxy.dll is required for those trains which use a Windows-based plugin.
It's a distinct possibility that users who download the nightly builds will be on Windows.....


This is never used


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

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


@DingWentao1234
其实有一个还不错的服务——作业部落~
但是我觉得做的还不够好. 尤其是那个令人无力吐槽的客户端.
关键也没有开源, 所以没法帮助改进.
所以只能自己实现一个. 不知道什么时候能出Demo.
最近还真的蛮忙的.


So is it up to the user to first do `oc project $loggingproject` where `$loggingproject` is the project/namespace which has all of the logging components?


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


Is this `puts` supposed to be here?


do not publish the theme with this change. there is a t.js bug that makes the UI fail for guests browsing.


This would allow you to programmatically render html tags: $('#city-options').append('<option val="' +cities[i]+ '"> '+cities[i]+' </option>')


I'm partial to Julia style dispatching and Mathworks is starting to have more like that style (see e.g. their unittest code - http://www.mathworks.com/help/matlab/ref/matlab.unittest.qualifications.verifiable.verifyequal.html#examples)


Is this something you think would be useful in Xdebug, @derickr? I think @alexmalyshev was just responding to an internal feature request but it would be awesome to get it upstreamed.


Good job on the dropdown as well. Please view the comments for best practices.


@dieris clever work around Matlab's lack of conditional assignment.


Should this have "-DATOMIC_BUILD_2D=1" here? You removed it in the CI script.


Test


This is handy, but it is also a specification change.  @aexbrown - Do you concur that we should allow people to specify scaling factors on units (e.g. "0.026*mm")?


Please line up all the arguments.


Great job on the dropdown as well. I like how you broke down the code into separate functions.


At the end there, there's a fix for previously broken onChange renaming, snuck by the commit message.


remove any unused code :)


Looking good Brit!

Nice job


In our current code, we have always performed aggregation on the raw results and then report aggregated numbers as samples. (e.g., average, count, mean, p5, p50, p90, e.g., "single stream upload throughput p99"). We can't store each and every raw data point as "sample" here, that would be way too many. (e.g., 100 stream with 1000 object per stream would mean 100,000 samples per run)

Please take a look at _AppendPercentilesToResults() see how the rest of the code base does that, it's pretty simple to do the same here.

As to metadata, for each sample point from the same run, they should have exactly the same metadata for one test run: how many streams were used in the test, and what the object_storage_object_sizes parameter is for that run.


Where does the batch size come in? Looks like you wanted to add support for batches but never got around to actually doing it.


I just pushed that fix to master a few minutes ago :)


&#123 and &#125 instead of { }. I think as is it will break javadocs.


good job


add form-inline


There has to be a better commit title.


What did this change actually do? This looks identical to me.


Hey Cos! If you're happy with it, can we commit it?


@patrickelectric verdade, já corrigi, valeu!


Please line up the arguments


If you have this in your code, you'd be repeating the creation of `<option>`s line 8 doesn't need to be there I'm pretty sure!


Built! Use 313


`input` is a bad name if it is also going to be the output


@sils1297 @Makman2 - just a reminder that we probably want a better documentation formatting guideline.


@icehess i think `release_failed_job` will send the job_complete ?


you're not calling the req or res variables in this function. Does js_set require these two variables to be passed in? Are they the only arguments that are passable? @aluttik @linuxjedi


Clean up your codes a bit better. Then this is a beautiful job.

:)


I usually look at the Open Group pages, and for `sem_init`, [it says this about ENOSYS](http://pubs.opengroup.org/onlinepubs/7908799/xsh/sem_init.html):

> The function sem_init() is not supported by this implementation.

which would imply none of the other semaphore functions will work.  However, the wording might be implying that the "implementation" is the runtime library.  It's not clear if "implementation" could also mean the hardware upon which the runtime is being used.

I'd personally prefer if `sem_init` failed gracefully with an `errno` set and a -1 returned (sensible code would check this), but I don't think POSIX is clear on the issue.

Given that it is highly unlikely anyone will use this code on a 386 (since Linux itself no longer supports such CPUs), I suppose it doesn't matter much.


This shouldn't be here


why not inline statement()? It does look complex.


Any reason for doing this ? isnt the default value already 0 ?


Same here.


Tunnel vision. Thanks for another pair of eyes.


Please add documentation


Go to https://github.com/ogmp/ogmp_client and click download zip. Or git clone.


Ingester doesn't need the conf? I can envision situations down the road where this would be useful.


Also, this is never used.


The following condition is always true, isn't it ?
`( (mob ==  0) || (rank_bb(s) & b) || (file_bb(s) & b)) `

Maybe you meant this ?
`( (mob ==  0) || !(rank_bb(s) & b) || !(file_bb(s) & b)) `


These don't appear to be real LaTeX commands, but there is an `\odot` that I think is what you want for `\circledot`: http://mathb.in/53477

The situation with `\parallelogram` is pretty unfortunate. There's no widely interoperable command for it in TeX and friends: http://tex.stackexchange.com/q/136948

Purportedly with `\usepackage{stix}` there is a `\parallelogram`: http://tex.stackexchange.com/questions/136948/parallelogram-symbol#comment413261_136980

But that's not supported by MathJax as [the MathBin above](http://mathb.in/53477) showed. What MathJax does support is a `\unicode{}` extension, with which this symbol is `\unicode{x25B1}`: https://groups.google.com/forum/#!topic/mathjax-users/fAleCwCRwZ8

But KaTeX doesn't support that, and it'd probably be a bad idea for MathQuill to export something it can't import (`\unicode`), so...ideas?

Perhaps best we can do is to add a comment to the `\parallelogram` line being like "Note: not a real LaTeX command but see #552"


Added savgol_filter which requires that scipy is installed.


@rplopes can you submit this as PR to sequelize upstream? I think we lost track of it, but not sure..


remove unused code


No need for this alert anymore


develop (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2362 of 2398 tests passed, 0 test warnings)

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


That makes sense to me
On Mar 8, 2016 6:52 PM, "Richard Megginson" <notifications@github.com>
wrote:

> So is it up to the user to first do oc project $loggingproject where
> $loggingproject is the project/namespace which has all of the logging
> components?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ewolinetz/origin-aggregated-logging/commit/fec4c4a3259b49942db3e3140d6f3e2d3336dfbf#commitcomment-16577741>
> .
>


If measurements showed a sizable difference, then it'd be worth considering.  OTOH, since the preceding byte will be the opcode and opcode determines natural alignment, it's not obvious what the effect will be.


Nvm, I was thrown off by the fact that we used a `constructIngester` method instead of directly calling the constructor.


In old ddf, I see it has field mCreatedTime so I put it here for back compatible. will discuss with @PangZhi and remove it if we don't use anymore.


Sorry, now tagged.
https://github.com/hirak/prestissimo/releases/tag/0.1.15


Statement and Description are not final in the TestRule interface


Currently Spark engine just support for LocalFileSystem, in the future we will support for S3 or other DataSource.


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 30](https://teamcity.labkey.org/viewLog.html?buildId=308980&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 39 (39 new), passed: 12 Build time: 00:32:56
### Failed tests
```
org.labkey.test.tests.cds.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Hey Ted,

I would say the only thing left to do is clean up/ indent your code better. Readability is incredibly important for professional devving.

Also there is no need for two event handlers which were `submit` and `click`. When you have an event handler for your form submission, there is no need to check for the button click event. At this point, you're repeating your code because both event handlers will fire when you click on your `#submit-btn`


Remove the $ sign, it is unnecessary here.


please remove the comments


Are there instances when callback is not set? In my testing with cli, I was hitting issues where callback was undefined.


Just add a  "or coala-html" to the above ? Why do you want to repeat this ?


change issue url to this repo


@sils1297 Which is better ? Adding this here or to the default_coafile ?


not work to super group and not work by replay


You should take some time to understand the attack.  You should know that `Body[t&BodyMask]`, which you use here and in all the other customizations one through four, only helps to improve variable ordering in constraint satisfaction.  It was more secure when the key body was only used in the order of the moving pointer.  Using the key body in the consistent order 0..N means that less of the key needs to be guessed to rule out possible key combinations.  The result is that the key recovery attack will run faster.

Please take some time to understand the attacks.  I'm here to help.


This is a lot of repetitive code.
Why not just import  coala_json.main and use that ?


Getting the user value does not necessarily need to be placed in a function.


Change last phrase to "on Docker Hub" with a link and then remove the two lines


we don't need depend on dlux. please remove.


I think I commented it
The program never exit the while loop
On 9 Mar 2016 08:45, "Ryan Louie" <notifications@github.com> wrote:

> @romain-chiap <https://github.com/romain-chiap> How does the while loop
> get exited with your current functions? From the logic, I don't see how
> this line after the while loop, self.setMotorVelocities(0,0), gets
> executed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/youralien/team_ros_plays_football/commit/8243eb04b9b1069c0bee76c11a39c6a9a10bb441#commitcomment-16577661>
> .
>


please remove blanks.


@Myoldmopar I don't know how you managed it but you have random spaces inserted in the flags line. I'm guessing that's at least part of the problem.


I'm totally going to steal some of this


I think this could simply be an assert

```
assert isinstance(hostname_port, str), "Expecing 'hostname:port' string"
```


remove blanks


Overall good job on the hw. Please view the comments for best practices. One thing to consider is the use of functions, think about when and where they need to be implemented. Keep up the good work.


Hey this would give out about 2000 extra lines ...
The silencing is only done to not show basic stuff. It should still show errors


I think 2000 extra lines makes it more confusing to debug


I don't think we need the dependency


Capitalize Hub


Indentation \ formatting all around


Do we ever call this on something that isn't an etcd authority?

If not, let's just rename this to validate_etcd_authority.  We can then make the logging etcd_authority specific which will be much more clear.


remove


remove the blanks


Closes #34


Replace the website repo and static site with something real


No, it gets angry that he passes a null parameter when it's supposed to be
something else.
On Mar 8, 2016 2:55 PM, "legoboy0215" <notifications@github.com> wrote:

> Uh... NBT?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Falkirks/SimplePortals/commit/ed5cc806d8a5f69927c0241a20cf4bf26d0cfb63#commitcomment-16575652>
> .
>


I have re implemented the lock. Please merge


Experiencing the same issue as @NSDeveloper


Replace with something real


I've wanted to get rid of those CVS ids for a long time.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/3308


Change to a real repo


This is the key bit: rather than ignoring `x`, the lambda now modifies it.


Replace with real repo


Greetings,

I have applied your catch and gave you credit here https://github.com/unixabg/raspi-helpers/commit/02385e5d22f06bbfc22d232a1f8a60cf08e84c47

Thanks.


Replace with image in this repo


Remove this line and replace table below with real images


Cool idea!


Yes. I realize that lol. I am correcting the code.

Thanks :D

2016-03-08 19:04 GMT-06:00 Rocky640 <notifications@github.com>:

> The following condition is always true, isn't it ?
> ( (mob == 0) || (rank_bb(s) & b) || (file_bb(s) & b))
>
> Maybe you meant this ?
> ( (mob == 0) || !(rank_bb(s) & b) || !(file_bb(s) & b))
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fbv-sf/Stockfish/commit/a03dda0e3de89dcef2646f8ce03a08eedf417152#commitcomment-16577967>
> .
>


Replace with repo github.com/docker/community


Pleas use the `#title-logo` style you created.


we shouldn't have to translate these strings as they are lazily translated, right?


it might be a good idea to have an addProperties(Properties) API to load stuff in bulk in @Before methods


lgtm


also, i realize it was like this in the source but shouldn't this response be in english?


Nice, thanks for handling this @72squared


and gunblade also... forgot in description


The differrent in cost of dp and greedy is huge in leetcode, is my time complexity analysis correct?


s/reprensented/represented


这么早就在提交代码啊


replace with image in github.com/docker/community repo


this is pussy shit


Any chance to get this change applied on other similar gems/CVEs?

I see:

```
Name: actionpack
Version: 4.2.6
Advisory: CVE-2015-7576
Criticality: Unknown
URL: https://groups.google.com/forum/#!topic/rubyonrails-security/ANv0HDHEC3k
Title: Timing attack vulnerability in basic authentication in Action Controller.
Solution: upgrade to ~> 5.0.0.beta1.1, ~> 4.2.5.1, ~> 4.1.14.1, ~> 3.2.22.1

Name: actionpack
Version: 4.2.6
Advisory: CVE-2015-7581
Criticality: Unknown
URL: https://groups.google.com/forum/#!topic/rubyonrails-security/dthJ5wL69JE
Title: Object leak vulnerability for wildcard controller routes in Action Pack
Solution: upgrade to ~> 4.2.5.1, ~> 4.1.14.1

Name: actionpack
Version: 4.2.6
Advisory: CVE-2016-0751
Criticality: Unknown
URL: https://groups.google.com/forum/#!topic/rubyonrails-security/9oLY_FCzvoc
Title: Possible Object Leak and Denial of Service attack in Action Pack
Solution: upgrade to ~> 5.0.0.beta1.1, ~> 4.2.5.1, ~> 4.1.14.1, ~> 3.2.22.1

Name: actionview
Version: 4.2.6
Advisory: CVE-2016-0752
Criticality: Unknown
URL: https://groups.google.com/forum/#!topic/rubyonrails-security/335P1DcLG00
Title: Possible Information Leak Vulnerability in Action View
Solution: upgrade to ~> 5.0.0.beta1.1, ~> 4.2.5.1, ~> 4.1.14.1, ~> 3.2.22.1

Name: activemodel
Version: 4.2.6
Advisory: CVE-2016-0753
Criticality: Unknown
URL: https://groups.google.com/forum/#!topic/rubyonrails-security/6jQVC1geukQ
Title: Possible Input Validation Circumvention in Active Model
Solution: upgrade to ~> 5.0.0.beta1.1, ~> 4.2.5.1, ~> 4.1.14.1

Name: activerecord
Version: 4.2.6
Advisory: CVE-2015-7577
Criticality: Unknown
URL: https://groups.google.com/forum/#!topic/rubyonrails-security/cawsWcQ6c8g
Title: Nested attributes rejection proc bypass in Active Record
Solution: upgrade to ~> 5.0.0.beta1.1, ~> 4.2.5.1, ~> 4.1.14.1, ~> 3.2.22.1

Vulnerabilities found!
```


should this be id="dockercompetition" ?


read it


ASP.NET Core, please :smile:


Yes, I caught that, but the specification doesn't say you can have any number other than 1 in there anyway.  So unless we change the spec, that's (probably) technically an invalid unit.  It seems like a reasonable thing to be able to do, but I'd like to hear Andre's perspective.


Uh, and tabs!!!!


no global vars please, at least prefix with _ and add a unit suffix and comments, what's that constant supposed to help with? it's not obvious.

Ditto below for SECS_TO_PICOS


When it's available replace with right URL. Currently it's a subdirectory of community/Docker-Birthday-3


This is still not fixed.

s/show_in_mimimap/show_in_minimap/


Replace with actual url when it's available


Samara Oliveira 201411373
Saulo Gomes 201411823


I'm busy so I only get on at school or on my phone. You will notice that I might be editing more on Wednesdays and Saturdays also.


better than nothing


@HaomingChen please also give a star on my original repo if you are forking it, thanks...


Please bring the logo and title closer, while keeping the same height.


This is where the app is built, so maybe this should be in the next section


wtf


conainers->containers


Btw. If I look on my news list, I can see all the messages you deleted.


@mostaphaRoudsari I am pasting the HTML error file here. It seems like it got stuck at the first instance itself.

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">

  <META HTTP-EQUIV="CONTENT-TYPE" CONTENT="text/html; charset=UTF-8">
 <TITLE>Grasshopper Archive messages</TITLE>
  <META NAME="GENERATOR" CONTENT="GH_IO.dll (0.2.0002)">

Grasshopper Event Log

This document contains the messages that were recorded during the most recent Grasshopper® file read/write.
Whenever a read/write operation fails or behaves unexpectedly, this summary will be compiled and put on display.
If you experience problems saving or opening files, please include this log with any bug-report you file.
You can use the Send... button to mail this report directly, or you can save the log and attach it to a personal email message.
This log contains no personal information beyond what you supply, nor any other information that is not directly related to Grasshopper.

Developer contact

Message log start (chronological):

IO Read error: Unknown data type encountered.


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


```
if (is_string($namespace)) {
    $loader->setPaths($path, $namespace);
} else {
    $loader->addPath($path);
}
```


Shouldn't this be port 5001?


Might want to fix to ` <img src="img/carousel/Mount_Rainier_from_the_Silver_Queen_Peak_Image1.jpg">`


No need to push vottingapp_worker. This doesn't change.


This is 2**32


read it


Yeah, that's okay.  I was just making a point to Zachary that we won't get in trouble just because someone might see our Github accounts.  But that's ironic that your free time is during school.  It's the same way for me, but that just shows that our education system is messed up.  If we could just work and learn stuff at school, then maybe we wouldn't be so busy at home.


the issue is still there after update esp-open-sdk to latest version,


Please keep this first comment line.


The conditional block above switches on whether the iterator has a generic type.
I ran some tests combining enumerator vs enumerable and generic vs non-generic and concluded that the types actually needed in the code (`GenerateMethodImplementations` method) only depend on two cases: enumerator vs enumerable.
So I replaced the wrong condition above.


Typos:
- numberes
-hypehn


aren't all those messages lazily translated (why do we need to call ugettext with them)?


Actually they can go to the URL of the result app/birthday.html and push the button.


The change removed a hidden unicode signature at the start of the file.


Also generally accepted that you put spaces at the end of each line when splitting a string across multiple lines.


I think this will show up in the logs of the result app, so they can pull it from there. Outline that step with docker logs xyz


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


Looks good !


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


Yeah, but that implies learning something new, not gonna happen


@ijh165 Starred


I may need help with the unicode escapes, that's too much work!!!!


formatting unnecessary?


Well, no wonder raulston blocks batch, 'cause ALEX IS ON IT EVERY SECOND!!!!


This is wrong, you changed the order of the parameters in the text but the core still expects the first to be a float and the second to be the player name. I'd post a crash log but it's as easy to reproduce as going in-game and using .mod speed so whatever...

@tkrokli this crash's on you


just 12? :)


Well, I wish we could learn new things at a fast rate at school.  But alas, that will probably not happen until college or late high school.


Welllll, I didn't see [that](https://github.com/energy-plus/EnergyPlus/commit/7b9911cd3bf1eaf5c2e2f04633931f0c1d473e71#commitcomment-16576469) coming...


Please ignore this commit. I had forgotten some changes from merging with latest fetch from master.


Yeah, I fixed those in the next commit, which now says it can't find CMAKE_C_COMPILER or CMAKE_CXX_COMPILER...


I agree. Let's revert this back to title case.


maybe just i + 1 in GST... ?


I actually meant issue #620.


this is my favourite line


The condition block above is incorrect and out-of-sync with the code (method `GenerateMethodImplementations`).
There are only two relevant cases for which types we'll need: either we're generating an IEnumerator or an IEnumerable.


for what that tmp var index_in_qt, why not inlined i+1 ?


shorter comment or delete it


Why not just StringMap<std::vector>?


Yeah, all the courses are DE (Math for me has, and probably always will be boring (not Stats!))
Sociology as I have seen on a list of online courses
COMP SCI <<< PROBABLY ME
Chinese <<< WTF
.
.
.
The list goes on


Style: remove the space after `++`.


Well, AP Phys should be _**fast**_ An AP course designed to be taught in one year, in one semester:
And Lindsey, SHE IS GONNA FLUNK!


yuk


She will either 1.) not be able to join or 2.) drop out quickly.


$(document).ready is an event which fires up when document is ready.

Suppose you have placed your jQuery code in head section and trying to access a dom element (an anchor, an img etc), you will not be able to access it because html is interpreted from top to bottom and your html elements are not present when your jQuery code runs.

To overcome this problem, we place every jQuery/javascript code (which uses DOM) inside $(document).ready function which gets called when all the dom elements can be accessed.

And this is the reason, when you place your jQuery code at the bottom (after all dom elements, just before </body>) , there is no need for $(document).ready

    //No need to put inside $(document).ready
    $(document).on('click','a',function () {
    })


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


@lbrentcarpenter Mine too :p


Can you scale the i2k logo to be 20px less wide?


maybe function? For what you use const ?


Makes sense.


This commit seems wrong, the condition should be `if (nullptr == msInstance)`.
It will cause fatal error every time when program exit.


Vk login


Overall good job on the hw. Interesting way to accomplish the task. Please view the comments for best practices. Keep up the good work.


She can't drop a course in HS


It looks like it is actually fixed except in one case that I can find. It was sending two GET requests to my /api/location endpoint whenever you searched for a location that you were not currently viewing the search results for. Now it seems to only have that behavior when you navigate to the home page and it does the default search. I've only noticed it visually a couple times when one of the responses takes significantly longer than the other, in which case the animation resets before it finishes.

Also, I really don't like how I changed the logic in this commit. Need to work on it some more.


If there is no separator, `find_last_of` returns `npos`, which is 2^32.  We should either handle this or have an assertion.


Yeah you can.  My brother dropped and added several courses throughout the course of high school.


Same comment about `find_last_of`.


Well, probably seeing you in Comp Sci DE in 11th or 12th!


Good job, thank you.


This is just because I didn't want to wait for this test as I didn't modify any related code here. I'll put it back once I'm done with my changes.


Are you taking Comp Sci?


Alex, WHAT IS BLOCKED!
ARE YOU AVOIDING MY QUESTION!!!!!!
RESPOND TO IT!!!


I'm taking a Computer Science and Engineering next year if that's what you mean.


What was the reason for hardcoding?


@juliancoleman From now on please do this work on branch


No, I am taking that course also, I am talking about the DE Comp Sci in 11th or 12th


Well, the CSE course has PYTHON
WOOHOO
WOOHOO
WOOHOO


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


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


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


It wasn't pulling from the env variables properly, even though they're set right in Heroku — I can't explain it.


It is not a listed course option, but when I asked abut the online course, the counselor said that the people who did take it took it as a DE course


Is the call to `#dup` necessary because the `current_state` is frozen at this point?


Overall good job on the hw. A couple of things to consider going forward: consider how functions can be further implemented in order to make the code more reusable. Otherwise keep up the good work.


promise.all == life


好大一堆提交物！


I also asked the counselor if I could take it and she said no.  Then I asked her again, and she said yes.  So I decided to take the class before she changed her mind.


Cant read property 'displayName' of undefined if task not found. :crying_cat_face:


s


Which one, the CSE or DE


The CSE


We should do this in Raptor as well


Is that class definitely happening, or do they know if there will be enough students to form a full class?


I asked her and she said that for the CSE, STEM focus areas are given priorities, hence why I asked of AP CS, so I would be given priority.
It is mainly for stem students


Static import


no hay un :stalin:


Make sure this isn't ever binary


7 people have visited Alex's Stuff Repo: Who are the other 5??????????????


(let me know if we should ask Mike if this is intended behavior or not)


Actually, all these could just be string concat


@vtsvang you should modify the README of the project and send  PR to @manuelvanrijn


@Ram-Z @Inkane This is probably a Bad Idea, because it's basically a global. Is there a better way, or does this whole interface need to be put inside a class?


Should I still take CSE then?


Well, that would be awesome if Me, You, Alex, and Connor are all in the same class:
Yes, you should!


This should be
`"url": "https://github.com/glocalcoop/anp-global-menu.git",`


还要讨论下校验的最佳实践么。。。


This is #2072 not #2027.


Yes


r+


Did Alex not take it last year, or was it even offered last year?


Technically this should use `%n` rather than `\n`


ok


Typo "crete"


Why you think this is good?


Good point. In fact `help` and `default` are not relevant for `ConfigDatasetType`, so I removed them from `DynamicDatasetType`. I also added a rather simplistic test for help to the test for `DatasetArgument` (finding out if the help string got into the argument parser is more complicated than I wanted to attempt).


This should probably go on previous line, it looks weird


Tenemos que solucionar el grave problema de los emojis.


Its only for High Schoolers


* Ability to meaningfully map the  underground tunnels of center city, This will require the ability to support maps with levels.


no dictatorship in this group ++


Or I would have taken it, the closest classes are his and mine
Both by Hall,
one is the do nothing class (IT)
the other is the stem class


holaaaa vengo a flotaar


But Alex is a high schooler.


Just gave this encoding a try and it actually increased the compressed size making it zero, so there does not seem to be much if anything in it. Sorry for the noise.


Oh nevermind.


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/5bed0b6b9f69b5a76f7757cdfe95d1d05642d305


merging graydon/stellar-core/scalability-benchmark-tests = 5bed0b6b into auto


graydon/stellar-core/scalability-benchmark-tests = 5bed0b6b merged ok, testing candidate = 006ced58


TRAHAN!


NOT CHAN!


I've never actually met him, so I didn't realize.  I'm only putting together that he's in our grade.


He is the JoinRajunCraftNow,
the malicious one!


So what do you mean by the Nothing class (IT)?


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 29](https://teamcity.labkey.org/viewLog.html?buildId=308982&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14 (14 new), passed: 37 Build time: 01:15:58
### Failed tests
```
org.labkey.test.tests.cds.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Does that even happen in real life ? ... :(


We do nothing!


AT ALL!


Really?


WE SUCKED AT CODETN, NO PROGRESS!


Nope


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

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

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

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

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

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

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

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

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

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

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

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


@jianqiu what is the plan here? We cannot just uncomment code. This needs to be submitted to community? Have you done so?


@mmcco  No, you got me wrong. These MIN() and MAX() macro can exist without reliance on `sys/param.h`. They're _not_ kernel macros. They're universal; can be used in both kernel and user space apps.
The only problem will be name collision. My #undef solution will do the trick.


I'm glad I didn't take it then.


Because it was so last minute, I switched out of STEM to get in it.
And I am not going to be an engineer (unless software engineering counts)


Instead of this

Implement the Comparable interface in Student class to sort the student Object by Name


YOU THOUGHT OF TAKING IT!!!!!


1915_WeatherFile2 (macumber) - i386-Windows-7-VisualStudio-12: OK (2365 of 2398 tests passed, 0 test warnings)

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


I must find out if Connor is taking  the CSE


The four best currently freshman programmers, in the SAME CLASS!
RAULSTON IS GOING DOWN


No, I never really thought of taking it because I knew that I had to take Spanish II, APHG, Orchestra, and gym.  I didn't actually have to take APHG, but I enjoy the class, so it's worth it.


Well. APHG is SS, so 9th doesn't matter for APHG


SUMMER GYM


@jianqiu where is the test for this? I cannot accept without new or modified tests added here: [json_caller_test.go](https://github.com/maximilien/bosh-softlayer-cpi/blob/master/api/dispatcher/json_caller_test.go)


How does a person take French III as a Freshman (one kid that **was** in my French I class is gonna)


I didn't want to take summer gym since I was already doing cross country while at the same time working out my upper body.


I'll do that. Some folks aren't entirely happy with the `abc` library but it seems a good fit here.


Oh, BTW...worked on the GREEN LINE PAGE.  But it's not close to being done, so I didn't want to draw attention to it


Oh, I forgot that you have that awful busy schedule


Equivalent. Any benefit of one or the other?


I can always just give up more sleep though.


Well, Since of my problems, ICANT! I HAVE TO HAVE A F****** MEDICINE THAT (PRETTY MUCH) FORCES ME TO GO TO SLEEP!!!!!

UGGGGGGHHHHHH


I changed these to 3 forks, 20 warmup and 50 iterations


Do we have an idea of how to have people "upgrade" to this, if they already have been using some version of rddmarc or imap-dmarcts?


Well, it's not as bad as it could be.  You could have a debilitating disease that paralyzed you like Steven Hawking.


@yuhanle
感谢你提供的代码，不过下面这种方式并不能起到你预期的作用。
:smile:

`+bool Baby_hasBackgroundMode () {
 +    NSArray *backgroundModes = [[[NSBundle mainBundle] infoDictionary]objectForKey:@"UIBackgroundModes"];
 +    return [backgroundModes containsObject:@"bluetooth-central"];
 +}

+#if Baby_hasBackgroundMode
  - (void)centralManager:(CBCentralManager *)central willRestoreState:(NSDictionary *)dict{		  - (void)centralManager:(CBCentralManager *)central willRestoreState:(NSDictionary *)dict{

  }		  }
 +#endif
`


I'd love to understand this better, but I don't. What does "mitigated" mean? What's a/this "build in caching system"?


We could rename the columns to remove `purchasing`? Easier to call it in all the code and saves space.


Could move this line to a private method in this and call it before_action. You are calling this in the create method too.


OK, removed from AO proper, will probably release as such Thursday-evening (CET).


ccccccccccccc


should have run a spell check! thanks for spotting it.


Actually why is this different to find the shop than the other one in new?


No problem with it, but I think it's better to protect our intellectual property since the start ;-)


> Provides access to the configuration

Maybe add: i.e. the set of parameters associated with a specific buildout. Or so? (Configuration is a very heavily-used word.)


You could use `withdrawals_params[:amount]` here instead of `params[:withdrawal][:amount]`.


Another advantage is that with maven integration all configuration (presto, tempto) would be stored in presto product tests itself. Currently, we store config for automation product tests in `jenkins-scripts` repo.


Je pense qu'il faut enlever les lignes de ce module car ce module est en add-on et pas dans le core. pour les autres cela roule


done


Call `self.get_...` here so it doesn't create a new instance.


I recommend using "&&" over "or", as "or" effects are a bit harder to predict, as they are not equivalents.

Read here: http://stackoverflow.com/questions/5880962/how-to-destroy-jobs-enqueued-by-resque-workers
and here: http://phrogz.net/programmingruby/language.html#table_18.4


Why use 15.04's builds? We should use the closest possible builds to 16.04... compilers, libraries, userland, etc should be as close as possible.


You should do validations of if the withdrawal could be made here. Only allow to create the record if there is enough balance. That way you ensure record cannot be created without it. Don't do that logic in the controller.


Thanks @khiltunen for the review of this initial metadata schema.


Make sure you have https://github.com/haikarainen/light :)


Why we need these comments?


Referring to LeafPsiElement directly seems weird. I'd just check getElementType() here and change the type of `registerErrorIf Necessary` parameter to `PsiElement`.


use English


Also, this is only temporary, so that the role actually deploys on Ubuntu 16.04 (which isn't even out yet). We will update this and nginx sources.list to proper 16.04 builds once the upstream projects actually have builds of course. And it's not like I'm going to deploy shit into production the day 16.04 drops. ;)


format code


Is there any test that verifiers this branch?


Yes I tested it and it works fine


this is not best way of solving it ;)


[DEBUG] /Users/pier/Library/AndroidNDK/android-ndk-r9c/toolchains/llvm-3.3/prebu
ilt/darwin-x86_64/bin/clang++ -Wl,-soname,libApp.so -shared --sysroot=/Users/pie
r/Library/AndroidNDK/android-ndk-r9c/platforms/android-undefined/arch-arm build/
                                               ^---------------^ WAT?
build/armeabi-v7a/libJavaScriptCore.so /Users/pier/Library/AndroidNDK/android-nd
k-r9c/sources/cxx-stl/gnu-libstdc++/4.8/libs/armeabi-v7a/libgnustl_shared.so -gc
c-toolchain /Users/pier/Library/AndroidNDK/android-ndk-r9c/toolchains/arm-linux-
androideabi-4.8/prebuilt/darwin-x86_64 -no-canonical-prefixes -target armv7-none
-linux-androideabi -Wl,--fix-cortex-a8 -Wl,--no-undefined -Wl,-z,noexecstack -Wl
,-z,relro -Wl,-z,now  -L/Users/pier/Library/AndroidNDK/android-ndk-r9c/platforms
/android-undefined/arch-arm/usr/lib -llog -Lbuild /Users/pier/Library/AndroidNDK
/android-ndk-r9c/sources/cxx-stl/gnu-libstdc++/4.8/libs/armeabi-v7a/libsupc++.a
-lc -lm build/build/armeabi-v7a/HL_com_test_app_AppActivity.o build/build/armeab
i-v7a/HL_android_widget_FrameLayout.o build/build/armeabi-v7a/HL_android_widget_
FrameLayout_LayoutParams.o build/build/armeabi-v7a/HL_android_view_Gravity.o bui
ld/build/armeabi-v7a/HL_android_view_View.o build/build/armeabi-v7a/HL_android_g
raphics_Color.o build/build/armeabi-v7a/HL_android_app_Activity.o build/build/ar
meabi-v7a/HL_android_view_ViewGroup_LayoutParams.o build/build/armeabi-v7a/HL_an
droid_view_MotionEvent.o build/build/armeabi-v7a/HL_android_view_ViewGroup_Margi
nLayoutParams.o build/build/armeabi-v7a/HL_android_os_Bundle.o build/build/armea
bi-v7a/HL_android_content_Context.o build/build/armeabi-v7a/HL_android_view_View
_OnTouchListener.o build/build/armeabi-v7a/HL_android_util_Log.o build/build/arm
eabi-v7a/HL_java_lang_Object.o build/build/armeabi-v7a/HL_java_lang_Boolean.o bu
ild/build/armeabi-v7a/HL_java_lang_Double.o build/build/armeabi-v7a/app.o build/
build/armeabi-v7a/HL_jscode.o -lhyperloop -o build/libApp.so


Please remove this instead of commenting out.


这有什么区别吗


Kategory? really? Let's Dasha find the typo


[This line] (https://github.com/inducer/relate/blob/8214d5d0d5c966e3bd8218053d3c08ec9062dd28/relate/templates/403.html#L36) should also be modified accordingly or we will have to translate twice.


It would be nice if this could be changed by the user in the setting menu. Every use-case is different, some  has interferences and thick walls and need high repetition others might have 40+ LWRF devices they want to turn on/off at the same time and it could cause issues.


Nice


@david (hej david)


@david hejawjejao


@oconnetf tthis PR also modifies xAPI.md. See https://github.com/garemoko/xAPI-Spec/blob/readme/xAPI.md


Sounds good!, But PropertyDescriptor.getReadMethod returns a java.reflect.Method instead of CtMethod ?


fd


super!


Missing `use ApplicationException`


Issue #31


Thanks for the update, I have always worried about the Nokia Version. One more problems is, if I register multiple devices with the same ip address, it is very likely to be blocked, even registering a new number.


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


Cool. Thanks.

-----Original Message-----
From: "Matt Lorimer" <notifications@github.com>
Sent: 3/9/2016 3:26 AM
To: "salesagility/SuiteCRM" <SuiteCRM@noreply.github.com>
Cc: "dodexahedron" <brandon@dodecatec.com>
Subject: Re: [SuiteCRM] Schedulers - global key - Fix issue #748 (004db06)

it allowed this file to be translated using online tools
—
Reply to this email directly or view it on GitHub.


no, regexp should be able to catch new line.


...Aand you broke my project's package.json. It is not a good idea to make such changes unannounced :(


Our Easter Egg Hunt format is mostly about finding answers to questions in datasets (although I agree re the broader definition)


You mean a test that invokes smart enter on a method whose last parameter isn't a lambda?


My own opinion but the adjective lovely is a little condescending


Normally that would be the case, yes, but there's nothing in the spec for const pointers.

Would you prefer:
`char *const decimal` ?


Overall - good blog


Yes, this is the change of commit https://github.com/loup99/WtWSMS/commit/596a73e9596d38a8a7c09e5da0188d5615e84f60


Snap is NOT the SnapDragon Camera genius. It's a painstaking effort by the CM team and the CM Team alone.


The package.json on npm was not uploaded by me, but someone else that grabbed the name.

But in any case, as far as I knew, the uploaded version didn't work. At least it didn't work for me.


![alasql logo](https://cloud.githubusercontent.com/assets/1063454/13632161/9fd9cd08-e5e8-11e5-98cc-dca251e7b427.png)


`getElementType()` is not defined for `PsiElement`


carpeta para la asignatura de servidores de alta prestacion


from_payment_service maybe?


@Downchuck

does the issue still exist ?


Well, it works for me.
And since this change broke my project installation, apparently it *does* use this package.json and this git repository...


Hi Olivier,

Unfortunately it doesn't. I am using it to track some changes in the battery driver that Lenovo did from 4.4 to 5.0 and to 5.1. Specifically:

http://forum.xda-developers.com/thinkpad-tablet/general/yoga-2-830-1050-wakelocks-cause-t3329821

Cheers,
Alex

On Sun, Mar 06, 2016 at 06:29:11AM -0800, Olivier K. wrote:
>Just out of curiosity; does that kernel compile (and boot)?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/alexzeitgeist/YT2/commit/a78393d341a45d179729ff85e47b31198021f45c#commitcomment-16523822


Looks good Brother!


apparently you need to `opam init` first, in the risk of stating the obvious ;)


什么时候发布0.1.6呢？


I've pushed up 2.4.0 .


@floreks hey Sebastian, can you take a look at this. That's what's used to do the cross-compilation build. The idea is that `multidest` should be a gulp stream (something that can be piped) and is put at the end of our build pipes, in order to distribute the built sources to multiple directory,

Now I think this is O.K. conceptually, but to me it seems the logic at:
`outputStream.on('end', () => outputs.forEach((dest) => { dest.end(); }));`

is reversed. `outputStream` here is the "merged stream" representing a single step, made of many `gulp.dest` streams. Now shouldn't we call outputStream.end() when all the `gulp.dest` calls are finished (meaning all the files have been placed in the different cross-arch folders), and not the other way around as it is here?


Java is already included in the tomcat8 image.


*ouch*
Sorry. That came from Pro's PR. I simply squashed/merged the PR without seeing that.


Looks like it is a counter, not an index (as stated in the [msdn documentation](https://msdn.microsoft.com/nl-nl/library/microsoft.teamfoundation.workitemtracking.client.workitem.revision(v=vs.120).aspx))


oui tu as raison


I had to add the owner. In the future the owner should be the loged in user and the field should not be exposed in the form anymore.


Please do not make a separate commit to these type of small changes when submitting a PR


네 그런 이유도 크고도 무엇보다 기존 일반적인 자바스크립트들을 대상으로 이런저런 테스트를 해봤는데요 Nashorn의 안정성이 의외로 떨어지거나 해석이 다른 경우도 있다는 걸 알게 되었습니다. 유구한 세월의 리노가 아직은 좀 더 나은걸까? 라는 생각이 드네요 ㅎ

혹 흥미가 있으실지 모르겠지만  http://ariya.ofilabs.com/2014/03/nashorn-the-new-rhino-on-the-block.html 링크도 봐보시면 나중에 유사상황에서도 이런저런 판단에 도움이 되시지 않을까 생각됩니다. (보시고 흥미로우시면 댓글들도 꼭 보세요)


Seems it's a GitHub usability problem, when I try to create PR in a fork, it shows no difference by default, weird. But once GitHub is saysing `If you need to, you can also compare across forks.` then I click `compare across forks`, pick the base repo and my fork, it starts to show diff.

Anyway, thanks again for your help:)


Are you sure that this commit makes sure that 'Release' is compiling with -O3? I don't see any distinction based on the build type.


Java is already installed in the tomcat8 image.


@deruelle I'm not ready to estimate right now,  I need to analyze other files and look for something similar in stack


yep


yes, it seems error are somewhere in travis settings/setup


This commit fixes #3 by using the GeoServer WAR download with Tomcat instead of the self-contained Jetty version. It also adds support for building the image with GeoServer plugins and adds instructions on setting Tomcat properties such as maximum memory.


Saved the file with utf8 encoding


Also, because of the localization I'm going to need to do more complex stuff in the build pipelines, so I think of leaving out this multidest module. I think I'll use the `async` library to merge multiple streams into 1 gulp task.

The problem I am trying to solve is that we want to build many versions of the source, per translation language (en, ja, de, etc.) and per architecture (amd64, darwin, etc.) => I do not want a separate gulp task for each possible combination.

Instead I am building slightly more complex gulp tasks, which iterate over all combinations and multiple streams are run in one gulp task (one stream per combination of language and architecture). Check the `build-frontend` gulp task in this commit if you can, I'll write some more explanations there...


@aarodude Can you elaborate please, I don't understand your comment.


> you used another one. I can't foresee that.

Having old styles in the code always risks them being found.

I do use S3Duplicate for simple usecases...here I was copying a whole pattern (a whole Model) from another working case...the deduplicate was just 1 small element of that.
I didn't think to update both whilst I was there...it didn't cross my mind as I was focussed on other things & there was no '@ToDo: move to S3Duplicate' note there or anything.
Sorry, but if this is a priority for you then the resourcing of this wholesale migration needs to happen (do it or find a volunteer). Currently it isn't a priority for me


I think this commit, removing `.all` method cause the

> `NoMethodError - undefined method `values' for #<Class:0x007fe9e3d0c918>`

as for my environment, Ruby 2.2.2, Rails 4.2.4
`ActiveRecord::Relation.new.merge(object)` object should be an ActiveRecord::Relation not a model.
Like `User.all.merge(User)` also cause the error.


`getNode().getElementType()`