text:
Fix permutei rows/cols + add test

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished. 1249 tests run, 37 skipped, 10 failed.

Refer to this link for build results (access rights to CI server needed):
/job/nd4j_PR/259/Build result: FAILURE[...truncated 3.56 MB...]	at hudson.maven.MavenModuleSetBuild.run(MavenModuleSetBuild.java:544)	at hudson.model.ResourceController.execute(ResourceController.java:98)	at hudson.model.Executor.run(Executor.java:404)Caused by: java.io.FileNotFoundException: https://api.github.com/repos/deeplearning4j/nd4j/statuses/4d27ac04dd69612db93a8235d94eb76dc9579894	at sun.reflect.GeneratedConstructorAccessor450.newInstance(Unknown Source)	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)	at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1890)	at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1885)	at java.security.AccessController.doPrivileged(Native Method)	at sun.net.www.protocol.http.HttpURLConnection.getChainedException(HttpURLConnection.java:1884)	at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1457)	at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1441)	at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:254)	at org.kohsuke.github.Requester.parse(Requester.java:535)	at org.kohsuke.github.Requester._to(Requester.java:262)	... 12 moreCaused by: java.io.FileNotFoundException: https://api.github.com/repos/deeplearning4j/nd4j/statuses/4d27ac04dd69612db93a8235d94eb76dc9579894	at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1836)	at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1441)	at java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:480)	at sun.net.www.protocol.https.HttpsURLConnectionImpl.getResponseCode(HttpsURLConnectionImpl.java:338)	at org.kohsuke.github.Requester.parse(Requester.java:525)	... 13 more

authors: AlexDBlack SusanBot


text:
Push notifications

tring to merge

source: gharchive/pull-request

license: mit

license_type: permissive

license_source: bigquery


text:
optimized code style (organize import)

My mistake

repo: dennis-menge/spring-cloud-gateway

license_source: github-api


text:
敢问你这个思路是怎么想到的呢？

确实实现的有点重，而且有BUG -_-|||

source: gharchive/issue

authors: Joosun ZhaoKaiQiang

repo: dersoncheng/MultipleTheme


text:
Deploy v0.4.0

:shipit:

authors: danielfdsilva vgeorge

repo: developmentseed/gep-explorer

license: MIT


text:
Fix link anchor

Nice catch :ok_hand:

authors: antoineco kmisachenka

repo: deviantony/docker-elk


text:
할일1

이 부분은 이렇게 하면 좋겠다~

authors: Minseo-Gang devyeony

repo: devyeony/spring-react-project


text:
Fix the format for the timeShift function to include the offset.

Do we need to update any unit tests?

Interestingly enough, AFAICT there are no tests for timeShift. I'll add some.

Ping.

authors: dgryski vision-sbm

repo: dgryski/carbonapi


text:
Edit task의 시작일이 저장되지 않습니다

현재는 고치고 새로고침을 한번 해야한다.
추후에 getTaskStartdate RestAPI를 만들고 로딩후 셋팅하고 값을 고칠 때 이벤트가 발생하는 형태로 변경하기.

/api/task 제작완료.

80df97d


text:
users.rust-lang.org / internals.rust-lang.org posts per day

Adding links for now, will scrape later.

Boilerplate: I'm going to refocus this project on rfcbot, since that's the primary usage rusty-dash.com is seeing.

repo: dikaiosune/rust-dashboard


text:
Seed DB with test image data to check that it will populate the faces detected page

Look up MySql Blob file type

repo: dimaostrov/fareServer


text:
integrate flow and prettier

we use eslint and other linters.
flow already integrated

repo: django-stars/frontend-skeleton


text:
Added test for errors when saving timezone-aware times if not supported.

@felixxm Thanks
Just a bit of nit-picking, the commit message title has a period(.) in it. Maybe you would want to remove it.

Just a bit of nit-picking, the commit message title has a period(.) in it. Maybe you would want to remove it.

Why? :thinking: A commit message is not a normal sentence so it should end with period.

Why? thinking A commit message is not a normal sentence so it should end with period.

At most places, what I have read is that the title of the commit message should be treated as a headline and hence generally a period is avoided. As I see, there's nothing of this sort mentioned in the committing guidelines for django and the examples explicitly use the period.
Sorry for the noise.

repo: django/django


text:
Fixed #25004 -- Added an OpenLayers 3 map widget

What's the plan for the old widget? Is it used at all? Should it be deprecated at some point?
Add a convenience import for the new widget in django/contrib/gis/forms/__init__.py?
Docs in docs/ref/contrib/gis/forms-api.txt (+ release notes)?
What's the situation with tests?
GeoModelAdmin.openlayers_url still references 'http://openlayers.org/api/2.13/OpenLayers.js' -- is that a candidate for being updated too (now or later)?

What's the plan for the old widget? Is it used at all? Should it be deprecated at some point?

The old widget is still accurate, especially for those installations that still have OpenLayers 2 custom code.

Add a convenience import for the new widget in django/contrib/gis/forms/init.py?

Sure, will do.

Docs in docs/ref/contrib/gis/forms-api.txt (+ release notes)?

To do.

What's the situation with tests?

Bad :-) We'll see with the forthcoming JS test infrastructure what's will be possible, but map-based tests are generally heavy and network-dependent.

GeoModelAdmin.openlayers_url still references 'http://openlayers.org/api/2.13/OpenLayers.js' -- is that a candidate for being updated too (now or later)?

Frankly, my plan is to kill GeoModelAdmin (or refactor it seriously). We'll see...

Mention me when it will be ready to merge. I'll draw SVG icons for this.

Thanks for the offer, very nice!

Claude, please reopen when you return to it.

Returned to the patch. This time I think we should simply replace the OpenLayers 2 version by the OpenLayers 3 version. It might be a bit more abrupt, but OpenLayers 2 is now aging and obsolete.

Code looks okay to me. I guess some test failures remain and shall we have @elky convert the PNGs to SVG?

I'll be able to covert PNGs on Monday morning. Hope it's fine

As for the JavaScript failures, I'll need help. The tests do pass locally in my browser.

I hate JavaScript debugging...

@treyhunner Maybe you have some advice for debugging these tests that work in the browser but not with npm test?

@timgraham I just realised I can't commit to this PR, so I'll add SVG icons (they already drawn) after this PR merged.

@elky, you can send a PR to Claude's branch if you like.

@elky or just send me the icons. I'll credit you.

I'm not sure what is going on there. The command-line failure isn't very clear about what the actual failure is.
From trial and error, it seems like the problem is with this line (line 73 in OLMapWidget.js):
    var map = new ol.Map({
        target: this.options.map_id,
        layers: [this.options.base_layer],
        view: new ol.View({
            zoom: this.options.default_zoom
        })
    });

Changing ol.js to ol-debug.js (to https://cdnjs.cloudflare.com/ajax/libs/ol3/3.15.1/ol-debug.js) in test.html reveals the slightly more descriptive error message:
>> Message: Died on test #1     at test (file:///django/js_tests/qunit/qunit.js:1340)
>>   }.at file:///django/js_tests/gis/mapwidget.test.js:10: 'undefined' is not a function (evaluating 'function() {
> Actual: nanimationDelayKey_ = undefined;
>> Ex  this.renderFrame_.call(this, Date.now());
```  }.bind(this)')
   Actual: null
>> Expected: undefined

That's in the function this.animationDelay_ on line 66985. I'm guessing that function is called at some point with this set to undefined.
That's as far as I got when digging...

I'm wondering if OpenLayers 3 maps can be tested without a browser context. OL3 tests are based on PhantomJS (https://github.com/openlayers/ol3/blob/master/tasks/test.js).

Maybe there's some way to skip the test if not running in a browser then?

I'm guessing there's a race condition somewhere in those tests that isn't appearing except in the PhantomJS tests.
In the browser, when I click into an individual test and then click the back button on my browser one or more of the gis tests appears to fail. Refreshing the page fixes it.

I pushed a fix for the issue raised by Trey.

I guess you force pushed and forget to incorporate the SVG icons in your local branch.

Oh crap, thanks for noticing.

@treyhunner, it seems like grunt-blanket-qunit isn't maintained anymore? https://www.npmjs.com/package/grunt-blanket-qunit (last update 3 years ago). It might be unrelated to the problem here, but I was looking at trying to update to more recent versions of the test dependencies just to rule out some third-party issue that's been fixed. We're pulling in phantomjs 1.9.20 and the latest is 2.1.
I guess we either merge this without tests for 1.10 or defer it to 1.11. I can't find a quick resolution for the test issue.

Blanket and qunit are the only real requirements here. We may be able to switch away from Grunt and use those together in a more manual fashion so requirements can be upgraded easily.
I think merging without tests might be acceptable for this until we figure out how to resolve the test situation for this section of code (considering JavaScript tests didn't exist at all not too long ago).

I discovered another problem with the default map, which should be in the 4326/WGS84 projection to not require coordinate transformations. Let's not block Django 1.10 because of this.

Claude, please reopen if you return to this.

Testing with recent libs, isn't it Christmas soon :-)

@timgraham Tested too soon? http://djangoci.com/job/pull-requests-javascript/10174/console

Try again, I just delete the node_modules directories on the servers.

buildbot, test this please.

buildbot, test javascript.

\o/

What, what, what :-)

Tests are OK: https://djangoci.com/job/pull-requests-trusty/9998/

authors: claudep elky timgraham treyhunner


text:
Fixed #27199 -- Made AdminIntegerFieldWidget use NumberInput.

buildbot, add to whitelist.

merged in ca9c69a9682141bc102ad6751f2feecc3d0d9d21, thanks!

authors: renskiy timgraham


text:
Fixed 27387 -- Documented addition of required HTML attribute in Field.required docs

merged in 611ef422b173b450b1fc6f7f94eb262961b24e54, thanks!

authors: claudep timgraham


text:
where did group go?

oh, sorry, for version 3 I removed a trimmed down core pull-stream, removing stuff that i didn't seem to use.
I have just republished it as pull-group for you.

sweet thanks I'll give it a try. keep up the amazing work and thanks for the quick reply.

no problem!
On Wed, Mar 2, 2016 at 12:19 PM, David Weinstein notifications@github.com
wrote:

sweet thanks I'll give it a try. keep up the amazing work and thanks for
the quick reply.
—
Reply to this email directly or view it on GitHub
https://github.com/dominictarr/pull-stream/issues/38#issuecomment-190958352
.

authors: dominictarr dweinstein

repo: dominictarr/pull-stream


text:
Fix syntax error.

@dotnet-bot test ci please

repo: dotnet/dotnet-ci


text:
Fix closing button tags in button sizes section

Thanks!

authors: mpolky-gr perrygovier

repo: driftyco/ionic-site


text:
DASHBUILDE-60: Shared JSON marshalling services

This PR depends on:
https://github.com/dashbuilder/dashbuilder/pull/60
It's required due to a new module declaration "dashbuilder-json" into dashbuilder. It shall be merged together with the following dependent PRs:
https://github.com/droolsjbpm/guvnor/pull/172
https://github.com/droolsjbpm/kie-wb-distributions/pull/125
All those PR shall be merged at the same time.

Thanks.
master: e0d368a10ab2f38fd58b4797b762cd2634439b2d

authors: csadilek dgutierr

repo: droolsjbpm/jbpm-console-ng


text:
Workbench - User and group management.

+1 to merge (my comments are not essential and can be rectified later if needed).

@manstis Thanks for the review Mike! Good change to improve my english level as well... :) I have provided a new PR for the fixes. Thanks again!

@romartin You're welcome - I know you don't mind correction of your English so was happy to help.

@manstis hehe nice! My personal teacher... sounds good :P

authors: manstis romartin

repo: droolsjbpm/kie-docs


text:
Fix link to sample shop, improve link to sample events API.

Thanks for spotting this!

authors: dschulten fkleon

repo: dschulten/hydra-java


text:
API options used #cwd, not #path

Nice find, thanks!

@dylang the option withoutDev is removing on next major release of depcheck. Do you get any scenario with it?

authors: dylang lijunle sandro-pasquali

repo: dylang/npm-check


text:
Improve the Role PM layout to Embed system

Also make sure that it says "Dear [username]" instead of "Dear Player"

Doc & Cop actions have been fixed in #63
Up next is the role PMs it self. @ZachieBlade wanna have a go at that?
I'll look into fixing the "Dear Player" my self as that requires some recoding of the role assignment system. Which is planned for Version 2.

repo: dylanpiera/Mafia-Quickplay-DiscordBot


text:
[ui]支持widget自定义指标颜色（3月）== xu

重新规划

authors: fumiwork scottsut


text:
first draft of clickable types ready, still needs tests

Looks great, GJ 👍

authors: edraut gomayonqui

repo: edraut/pulitzer


text:
Top N Complaint/Wards (3)

Done

repo: egovernments/analytics


text:
Update messages.json

:+1:


text:
BatchEuphoria integration into Roddy

get the travis tests to run

authors: dankwart-de vinjana

repo: eilslabs/BatchEuphoria


text:
Remove log option as is not used

:+1:

Thanks for all these great pull requests, I've added you as a collaborator on this repository so feel free to open future pull requests as branches in this repository instead of from your fork :shipit:

authors: kevinsawicki lukeapage

repo: electronjs/windows-installer


text:
enable travis and appveyor

resolved by #11


text:
Endoc 78 azure v2

Visible here: https://es-entando.github.io/next/tutorials/devops/installation/azure-kubernetes-service/azure-install.html

repo: entando/entando-docs


text:
Update repokitteh.star

/rk-bless

fsfsdf

cdsdds

repo: envoyproxy/envoybot


text:
Feature/network rules

HansKristianVikOlsen did manual code review, as he's not discoverable

repo: equinor/sepes-api


text:
chore(editorconfig): added .editorconfig

Nice. =)

authors: ericelliott mastilver

repo: ericelliott/credential


text:
Added the backdrop (created for wide resolution) and few tweaks

@Hariofspades conflicts 😄

authors: Hariofspades esoxjem

repo: esoxjem/Apps-Worth-Making


text:
Test Events with motors

https://trac.cc.jyu.fi/projects/legorobotti/browser/patakett/robotarm/src/org/pkettu/legorobot/RobotArm.java?rev=73


text:
Fix a typo in the changelog

Thanks

authors: tasomaniac vRallev

repo: evernote/android-job


text:
word-count: replace underscore with space in description

Thanks @bordeltabernacle

authors: Insti bordeltabernacle

repo: exercism/x-common


text:
refactor: moved JSON code to AsyncBoom

Ike Plugins (test-keeper)
Thank you @gastaldi for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin.

authors: alien-ike gastaldi


text:
Add link to 2 step guide

👍

authors: KrauseFx ohwutup

repo: fastlane/boarding


text:
Add OneHot layer for generating one hot embedding for zero-based integer features

@fchollet Thanks for reviewing the PR. I wrote about the reason for a dedicated layer in https://github.com/fchollet/keras/issues/3680. Basically, we may want a flexibility in encoding categorical features during network construction.

This the layer is stateless (no weights) it is preferable to use Lambda(K.one_hot) IMO.

Agree @fchollet. Thanks for your comments. At least it is a good experience writing a custom Keras layer, :).

authors: fchollet nhanitvn

repo: fchollet/keras


text:
Support $select in get

Closed via https://github.com/feathersjs/feathers-mongoose/pull/150

authors: daffl ekryski

repo: feathersjs/feathers-mongoose


text:
Support $search

https://github.com/louischatriot/nedb#operators-lt-lte-gt-gte-in-nin-ne-exists-regex

Is the syntax for this going to be something like
service.find({name: {$search: 'zeld', $options'ig'}})

If so, we'll need to make sure we support the logical operators: https://github.com/louischatriot/nedb#logical-operators-or-and-not-where

Moved to https://github.com/feathersjs/feathers/issues/334

authors: daffl ekryski marshallswain

repo: feathersjs/feathers-nedb


text:
HOTFIX #172 Corrigindo pagina html

Pull Request Test Coverage Report for Build 294

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 38.843%

Totals

Change from base Build 292:
0.0%

Covered Lines:
77

Relevant Lines:
190

💛  - Coveralls

authors: coveralls egewarth


text:
update reveal.js to latest master

No objections to this one, but was there any particular functionality you found missing/lacking in the previous submodule state?

No, nothing :-)  Just thought it would be worth updating before embarking on a hefty chunk of dev and test ...

authors: aspiers fghaas

repo: fghaas/presentation-template


text:
upload json files for participiants

Looks sane for me all in all! :)

authors: Schubidu apinnecke

repo: finanzcheck/wheel-of-fortune


text:
feat(viewport): add selectedIndex to output event

:tada: This PR is included in version 6.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

repo: flocasts/flo-angular


text:
Treat guid, email as common fields (no descriptions)

lgtm

authors: mbryzek paololim

repo: flowcommerce/api-lint


text:
Code coverage for flutter drive test is not supported

Any news or alternatives to get code coverage?

Yup, this is sorely needed.

Merging with #7474 (because it has a bit more details)

authors: kaiyuanw paul-hammant sthai256 zoechi

repo: flutter/flutter


text: fix some TextInputType dartdocs

authors: gspencergoog yjbanov


text:
proudly add myself to authors :)

because of #6923

authors: abarth zoechi


text:
Handle missing directories in the flutter doctor IntelliJ check

@cbracken


text:
Add path_provider plugin

@mravn-google could you PTAL at main.dart?

repo: flutter/plugins


text:
Report and alarm on capability not found

@Nauna capability not found - does this occur when the capability does not satisfy its availabilityPredicate?


text:
fix fontTools.misc.arrayTools.Vector() division

Good catch

authors: anthrotype justvanrossum

repo: fonttools/fonttools


text:
GitHub Clubhouse [ch515]

A valid branch name for this Story:
feature/github-clubhouse/ch515
Create & checkout a new branch for this Story:
git checkout -b feature/github-clubhouse/ch515
A valid commit message for this Story:
GitHub Clubhouse [ch515]
Link a single commit to this Story:
git commit -m 'GitHub Clubhouse [ch515]'

repo: freight-trust/clubhouse


text:
Create a base widget fed from hooks

Closed with
https://github.com/frostidaho/qtile/commit/9bcf803a3027bc4857fa770398fafcacc2b32f4c

repo: frostidaho/qtile


text:
Remove unused require statement

Thanks

authors: iwillig niwinz

repo: funcool/catacumba-prone


text:
Close #200: rename Interval to SequenceInterval

Since @ahwagner approved and @larrybabb delegated during his absence, I'm merging.


text:
CI patch

Current coverage is 85.15%

Merging #34 into master will increase coverage by +32.50% as of 9a209ed

@@            master     #34   diff @@
======================================
  Files            5       5
  Stmts          547     559    +12
  Branches         0       0
  Methods          0       0
======================================
+ Hit            288     476   +188
  Partial          0       0
+ Missed         259      83   -176

Review entire Coverage Diff as of 9a209ed

Powered by Codecov. Updated on successful CI builds.


text:
gh-1313-incorrect-options-graphId

Codecov Report

Merging #1389 into develop will decrease coverage by 1.93%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #1389      +/-   ##
=============================================
- Coverage      65.66%   63.73%   -1.94%
+ Complexity      1644     1557      -87
=============================================
  Files            704      554     -150
  Lines          22860    17662    -5198
  Branches        2862     2283     -579
=============================================
- Hits           15012    11257    -3755
+ Misses          6681     5448    -1233
+ Partials        1167      957     -210

Impacted Files
Coverage Δ
Complexity Δ

...on/hdfs/operation/ImportAccumuloKeyValueFiles.java

...ain/java/uk/gov/gchq/gaffer/mapstore/MapStore.java

...bstractImportKeyValuePairRDDToAccumuloHandler.java

...q/gaffer/accumulostore/SingleUseAccumuloStore.java

...y/core/impl/byteEntity/ByteEntityRangeFactory.java

...tore/key/core/impl/CoreKeyBloomFilterIterator.java

...umulo/operation/handler/AbstractGetRDDHandler.java

.../gchq/gaffer/mapstore/impl/GetElementsHandler.java

...gov/gchq/gaffer/accumulostore/utils/ByteUtils.java

...affer/federatedstore/FederatedStoreProperties.java

... and 139 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6183993...411edb3. Read the comment docs.

repo: gchq/Gaffer


text:
gh-693 - added ability to provide complex seed objects in the ui

Codecov Report

Merging #694 into develop will not impact coverage.

@@            Coverage Diff             @@
##             develop     #694   +/-   ##
==========================================
  Coverage      61.15%   61.15%
  Complexity      2634     2634
==========================================
  Files            380      380
  Lines          11743    11743
  Branches        1360     1360
==========================================
  Hits            7181     7181
  Misses          3990     3990
  Partials         572      572

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fbc1a00...077b34a. Read the comment docs.


text:
Add post aggregation and post transform filter functions to python shell

Merged into develop.

repo: gchq/gaffer-tools


text:
Maven assembly doesn't keep 0755 attributes for .sh files when built on macos

Retest after move to Gradle.

I think it should be:
0750 for all dirs
0750 for *.sh files
0640 for all other files

repo: gchq/stroom


text:
fix #1379, support storm tick tuple

+1

spec failed. merge after fixing them

authors: clockfly manuzhang

repo: gearpump/gearpump


text:
Update dependencies

https://github.com/yarnpkg/yarn/issues/4986

Finished.


text:
Move to app deployment

This is not working except for the ingress issues with deck and hook.

@giantswarm/team-ludacris PTAL - I want to merge here soon. Already a lot of changes and having smaller PRs would be nice :D

Going with this - feel free to still leave comments. I will continue with smaller PRs today.

repo: giantswarm/test-infra


text:
Fixed Rocket Icon #37

#37 Issue Solved

repo: girlscript/GirlScript-Summer-of-Code


text:
Add Support for import fetch from 'whatwg-fetch';

It's a polyfill, so you should never need to import it with a name, just like you never need to do import Promise from 'es6-promise'.
But if you insist, it should be doable currently
import * as fetch from 'whatwg-fetch';
import {fetch} from 'whatwg-fetch';

It might be a little cumbersome but something like this could work:
Make a little fetch utility module, that wraps up the whatwg-fetch.
// utils/fetch.js

// For browsers without Promise support
require('es6-promise').polyfill()
import 'whatwg-fetch'

// Can remove this if you would like
function checkStatus (response) {
  console.log('response.status', response.status)
  if (response.status >= 200 && response.status < 300) {
    return response
  }

  const error = new Error(response.statusText)
  error.response = response
  throw error
}

export default function whatwgFetch (url, options) {
  return fetch(url, options)
    .then(checkStatus)
}

Import your fetch utility to use in your module
// myModule.js
import whatwgFetch as fetch from '../utils/fetch'

fetch('/users.html')
  .then(function(response) {
    return response.text()
  }).then(function(body) {
    document.body.innerHTML = body
  })

authors: JosephShering drydenwilliams leonyu

repo: github/fetch


text:
Support Shadows

Added basic dithered shadows in commit https://github.com/gkjohnson/unity-dithered-transparency-shader/commit/32d121dee32da0d641b35e087c29aeb9cb1fa922


text:
GUAC-1467: Document configuring the Guacamole Docker image for LDAP.

In the hopes that this helps, here is a PNG rendering of the HTML output of the new sections:

repo: glyptodon/guacamole-manual


text:
Release 0.2.0

Travis automatic deployment:
https://pr35--safereactcomponents.review.gnosisdev.com

authors: gnosis-info nicosampler

repo: gnosis/safe-react-components


text:
Fix: logic error in gdscript_parser.cpp for-loop-range

Done. and thanks for the review
@vnen

Thanks!

Cherry-picked for 3.2.2.

authors: ThakeeNathees akien-mga

repo: godotengine/godot


text:
Fixed force-cast for UIStoryboardExtensions

oh man, conflicts :(

authors: goktugyil lfarah

repo: goktugyil/EZSwiftExtensions


text:
Get Travis running

Current coverage is 0.00% (diff: 0.00%)

No coverage report found for master at de86b89.

Powered by Codecov. Last update de86b89...1687be2

authors: codecov-io goldingn

repo: goldingn/GPflowR


text:
Avoid invoking 'runNow' multiple times.

Codecov Report

Merging #125 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #125      +/-   ##
==========================================
+ Coverage   92.22%   92.23%   +0.01%
==========================================
  Files          11       11
  Lines        1286     1288       +2
  Branches      259      259
==========================================
+ Hits         1186     1188       +2
  Misses        100      100

Impacted Files
Coverage Δ

lib/bundling.js
95.3% <100%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb57085...2e5afbb. Read the comment docs.

authors: codecov-io jmuk

repo: googleapis/gax-nodejs


text:
Update the Discovery parser

An update pulled from my refactoring work - it adds support for some new features, including array items and enums. Also adds auth detection support, and a new field for methods and schemas called path, which is the fully qualified path to a resource (ex: schemas.foo.properties.bar.properties.baz.
See the tests for details.

Codecov Report

Merging #1246 into master will decrease coverage by 0.03%.
The diff coverage is 84.61%.

@@             Coverage Diff              @@
##             master    #1246      +/-   ##
============================================
- Coverage     86.14%   86.11%   -0.04%
- Complexity     3770     3782      +12
============================================
  Files           380      380
  Lines         14967    15006      +39
  Branches       1666     1680      +14
============================================
+ Hits          12893    12922      +29
- Misses         1624     1630       +6
- Partials        450      454       +4

Impacted Files
Coverage Δ
Complexity Δ

...om/google/api/codegen/discovery/DiscoveryNode.java
90.69% <75%> (+3.19%)
20 <3> (+3)
:arrow_up:

.../java/com/google/api/codegen/discovery/Method.java
90.32% <81.25%> (-9.68%)
8 <1> (+2)

...ava/com/google/api/codegen/discovery/Document.java
90.38% <84.37%> (-9.62%)
11 <4> (+3)

.../java/com/google/api/codegen/discovery/Schema.java
89.39% <86.53%> (-3.2%)
8 <7> (+4)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 79a2750...061478e. Read the comment docs.

LGTM - everything seems reasonable.

authors: codecov-io garrettjonesgoogle saicheems

repo: googleapis/toolkit


text:
start reading some lines

Pull Request Test Coverage Report for Build 34

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build 16:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

authors: coveralls gothraven

repo: gothraven/Get-Next-Line


text:
Adding ability to set custom block name in teamcity report

Could you add this key to the getDefaultConfig() method with its default value so that people can discover - via istanbul help config - that this option exists?
LGTM otherwise. Thanks.

Also, if you are familiar with the teamcity internals, could you help review https://github.com/gotwarlost/istanbul/pull/264 ? Thanks in advance.

I've updated the commit with your comments, I'll check about #264

available in v0.3.20. Thanks!

authors: aryelu gotwarlost

repo: gotwarlost/istanbul


text:
DO NOT MERGE: test dart interop_to_prod tests

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Corrupt JSON data (indicates timeout or crash):
    bm_fullstack_streaming_pump.BM_PumpStreamServerToClient_SockPair__8.opt.new: 1

[microbenchmarks] No significant performance differences

I don't think the interop tests are actually using Dart. You're changing tools/internal_ci/linux/grpc_interop_to*.cfg instead of tools/internal_ci/linux/pull_request/grpc_interop_to*.cfg.

authors: grpc-testing jtattermusch matt-kwong

repo: grpc/grpc


text:
Fix memory leak, fix error.c refcount reporting

LGTM

authors: ctiller dgquintas


text:
tab传参和tab也刷新的机制是通过什么实现的，您能给解释一下吗？

传参是通过controller动态注入的吗？tab刷新是怎么实现的？

传参是动态注入的，刷新是通过销毁旧的scope，重新生成一个新的scope，然后重新执行controller，完成刷新

好的，明白了，谢谢

您好，现在项目里面有2个子系统，每个子系统里面有不同的tab，我怎么在切换子系统后，重新去初始化tab列表，uiTabsProvider.tab()能重复执行吗？还有就是切换子系统的时候清空uiTabs后在打开一个tab，这个时候用uiTabs.closeAll()报错

uiTabsProvider.tab()，是在angular初始化的时候被执行，以后无法重复执行。uiTabs 你是怎么清空的？

直接调用的uiTabs.clsseAll()

你把报错信息发一下

一共打开两个tab，切换子系统的时候调用uiTabs.closeAll()关闭目前的tab，然后切完子系统在open一个当前子系统的tab,子系统之间是通过uirouter切换的，每个路由作为一个子系统

好像是关闭还没完成，就又开始open的时候报错了，在open的时候外面套了一层timeout好像就可以了

有点不能理解为什么出现这个问题，closeAll 是一个同步的方法，即使立即打开新tab，也是得等closeAll 全部执行完，这个问题有时间我再仔细看看什么原因导致的

好，目前我只能先用$timeout(function(){uiTabs.open('新首页')})来解决了

我添加了判断，对于正在加载的tab，不执行销毁操作，你可以再试下

authors: L-Z-Y gusumuzhe

repo: gusumuzhe/angular-ui-tabs


text:
Add french language

Thanks a lot @coco-perso 😄

Hello @coco-perso! Just pinging you since I added a new line to the translations file: https://github.com/hacdias/filemanager/commit/889871ec0a1fac26dee1b3152d0f87e2a7af2c65#diff-fe6ccd8f0bf41f00b9ff5d61b8e0e3e9 😃

authors: coco-perso hacdias

repo: hacdias/filemanager


text:
Update website to point to profile website field instead of github

Thank you so much!! @MatthewStanciu or @maxwofford please merge ASAP

repo: hackclub/scrappy


text:
Add meeting notes from contribution discussion

@zachlatta Comments addressed.

@zachlatta I think the tests broke. Could  I get a  second set of eyes on this?

@zachlatta ping

Sorry about the delay. Merged!

authors: MaxWofford zachlatta

repo: hackedu/meta


text:
我试了一下，是不是本地的avi格式的视频不能播放啊

可以啊

authors: fengxiaobing hacknife

repo: hacknife/iplayer


text:
MANAGE SCHEDULE - displaying of info in profile box

Task done.


text:
Clarify docs/meta to denote changes from upstream

Done!

repo: halyard/puppet-ruby


text:
Port tests to support RSpec 3

Looks good but there is a warning when running the spec. Can you sort the warning then LGTM

Hey. I got carried away after merging a bunch of PRs. I think https://github.com/hassox/warden/pull/123 supersedes this one.

authors: balasankarc hassox

repo: hassox/warden


text:
update(frontend): add compare icon

@fantkolja could you generate a screenshots for the new icon?

authors: aigoncharov fantkolja

repo: hazelcast/frontend-shared


text:
Synchronized to https://github.com/akkadotnet/akka.net/pull/1131

Looking good.

authors: Aaronontheweb kekekeks

repo: helios-io/DedicatedThreadPool


text:
Fix typo in message.go

thanks!

authors: bacongobbler nojnhuh

repo: helm/helm


text:
Update podman section in README with db2

Done
We still need to update the README paragraph with the SQL to create users and DB: https://github.com/hibernate/hibernate-reactive/issues/145

repo: hibernate/hibernate-reactive


text:
Document modular models / dynamic/static performance models

also document the following (as described by @bqpd):
Right now it's not safe to use the same ConstraintSet in multiple models. (this isn't the only case where it's unsafe; fundamentally this is because ConstraintSet.sub modifies in place instead of returning a new ConstraintSet). This isn't new to the refactor.

The above issues are no longer true.

changed title since our best practices have changed

I think this is closed by #953?

yes, that seems right.

authors: bqpd whoburg

repo: hoburg/gpkit


text:
Add some JS tests for useClient

#63

repo: homebaseio/homebase-react


text:
merge with latest

Created mistakingly

repo: huggingface/neuralcoref


text:
Initial Commit

minor comments otherwise, LGTM

repo: hydrator/fast-filter-transform


text:
Fix typos in README.md

Woops sorry. DCO signoff added. 😄

Thanks @pgrimaud, merged!

authors: jt-nti pgrimaud

repo: hyperledger/fabric-chaincode-java


text:
语义角色标注

done

repo: iFLYTEK-OP/websdk-java-speech


text:
Upgrade Rubocop to 0.43

Coverage increased (+0.04%) to 96.807% when pulling d47d3fdd1f5083ce9ff2ad29d25429b1139f6035 on georgeu2000:update-rubocop into 5998269b057f7ec8c7f201b1d2969cae4c9fe2c4 on igrigorik:master.

👍

authors: coveralls georgeu2000 igrigorik


text:
Update chat-app node.js demo app tutorial

Tracking https://github.com/infinyon/fluvio/issues/386

repo: infinyon/fluvio-website


text:
Bencher

Hey @mdubbyap , did you want to add some of these features into this repo?  It'd be great to see your decay stuff.

Sorry, that was a github mistake on my part, still working on things locally.

authors: goller mdubbyap

repo: influxdata/tdigest


text:
typo property name

Wow, for real? Thanks!

repo: infostreams/neural-network


text:
Kubernetes

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1a7960b). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #399   +/-   ##
=========================================
  Coverage          ?   81.31%
=========================================
  Files             ?       72
  Lines             ?     2473
  Branches          ?       69
=========================================
  Hits              ?     2011
  Misses            ?      462
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a7960b...48f71c0. Read the comment docs.

authors: FrancoAra codecov-io

repo: ing-bank/baker


text:
Version 6.0.0

@rhyslbw please feel free to push a commit to this branch.

bors merge

authors: dcoutts erikd

repo: input-output-hk/cardano-db-sync


text:
[GH-362]

This PR shows that the issue was not in the Transformable instance.

repo: input-output-hk/cardano-node


text:
global: use inspire-schemas for countries

Depends on https://github.com/inspirehep/inspire-schemas/pull/397

@harunurhan updated

repo: inspirehep/inspirehep


text:
Ether-flatten prototype

Merging this into master, but this is still WIP.

repo: int-index/ether


text:
refactor: issue wizard loading

LGTM!

repo: interlay/polkabtc-ui


text:
api: fix wrong Loan indexer

@topless @equadon @zzacharo tests will probably fail but this is the fix to the tests in case you need to use it locally until the PR is ready.

I am honestly a bit surprised that tests are passing.

repo: inveniosoftware/invenio-app-ils


text:
Implemented shared_connection_block

Method advanced_connection::advanced_connection_impl::block contains race: two different thread can in the same moment block when m_blocker still doesn't exist, so two blockers can be created and there will be data race for m_blocker assignment. Even if assignment will not cause problems itself, incorrect blocker will be returned to one of the caller threads.

Added spinlock mutex to protect the blocker. There is also potential race in shared_connection_block::block(), but I don't think using single shared_connection_block from multiple threads is a valid use case.

authors: Warboss-rus sergey-shambir

repo: ispringteam/FastSignals


text:
Move TG bot sensitive data to .env file

https://github.com/ita-social-projects/WhatBackend/commit/7ec41cd86a686879f241ae74f24f337067733d0c
issue is partially resolved by this commit

authors: AnnBazille DemetriosKorass

repo: ita-social-projects/WhatBackend


text:
Also delete entrytypes not in the schema when using force

:shipit:

authors: boboldehampsink bvangennep

repo: itmundi/schematic


text:
Make Error.parse bail to Logger if it can't find the correct error.

completed in 0fe9a65

repo: ityonemo/zigler


text:
fix usingnamespace @import problem?

not reproducible with either
    ~Z"""
    pub usingnamespace @import("std");

    /// nif: noop/0
    fn noop() void {
    }
    """

or
    ~Z"""
    usingnamespace @import("std");

    /// nif: noop/0
    fn noop() void {
    }
    """

Its possible that somehow usingnamespace was misused earlier.


text:
IZPACK-1679: Add support for 11

LGTM

@reinhapa Thank you for merging.

authors: akuhtz reinhapa

repo: izpack/izpack


text:
Remove unused imports

@advayDev1 - PTAL

lgtm


text:
Other tools do not yet get started with the right coordinates

The Other tools window is a Google Maps Infowindow. It get automatically centered on the map.
i.e. the TIP of the infowindow get centered.
The window is too big though, and it crosses the top of the screen. This causes automatic panning to bring it back to center.
The same problem exists with Filter window and More Info windows.
There are at least three possible solutions:

less stuff in the window
vertically offset position (by how many pixels?)
custom overlays https://developers.google.com/maps/documentation/javascript/customoverlays

True. I've used technique 2 already for the ?p= query parameter. There's also option 1b, provide less text on smaller screens. Didn't we already do that? I think the other tools window has a full-blown and mobile version.
But "right coordinates" in this issue was not about the screen -- it was about going to the same location in that other tool, automatically. For some tools, I just not had time to program it. For others, we need coordinate conversion first.

I will study how various types of collapsible user interface / parts thereof could be implemented with CSS.I have an idea, but not hands-on experience yet. Expect results this month. Then we can discuss if its applicable here or not.

Maybe we can open another issue for specific popup windows (like the big other tools popup) being too big. This issue was about those other tools not being on the same coordinates.
I think the other tools popup could easily become much smaller, and this problem would go away. E.g., just include the icons (or just name) in the mobile version.

This issue has been resolved by a new UI.
Set psgeoDomainDefault.classicUI=false;
We use a Google Maps custom control at BOTTOM_CENTER
with some nice CSS in css/style.css

authors: jariarkko ralfstrandell

repo: jariarkko/psgeo


text:
give michaelneale access to this

:+1:

👍

I should edit commit message next time, sry

authors: batmat michaelneale oleg-nenashev


text:
Add shampoo optimizer.

Codecov Report

Merging #110 into master will decrease coverage by 0.07%.
The diff coverage is 86.56%.

@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
- Coverage   87.54%   87.47%   -0.08%
==========================================
  Files          15       16       +1
  Lines         779      846      +67
  Branches      183      200      +17
==========================================
+ Hits          682      740      +58
- Misses         53       57       +4
- Partials       44       49       +5

Impacted Files
Coverage Δ

torch_optimizer/shampoo.py
86.36% <86.36%> (ø)

torch_optimizer/__init__.py
84.61% <100.00%> (+0.61%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87d9ccf...f244ff4. Read the comment docs.

authors: codecov-io jettify

repo: jettify/pytorch-optimizer


text:
Fix deprecation

LGTM.
Thanks, @kilianc!

Included in v0.4.1.

authors: kilianc shinnn

repo: jgable/gulp-cache


text:
Support for 'useMaxVersion' jgitver paramter

with jgitver/jgitver#53 I think this PR becomes obsolete. I think we do not need maxVersion property.

authors: McFoggy djarosz

repo: jgitver/gradle-jgitver-plugin


text:
docker部署之后，获取ip很慢，要几秒钟

检查你网络连通吧，程序到数据库 程序到你浏览器客户端

检查你网络连通吧，程序到数据库 程序到你浏览器客户端

我只开了api服务，访问 127.0.0.1:5010/ 都很慢，但是有时候却又秒开的。是因为docker的原因吗？

检查你网络连通吧，程序到数据库 程序到你浏览器客户端

我只开了api服务，访问 127.0.0.1:5010/ 都很慢，但是有时候却又秒开的。是因为docker的原因吗？

中间网络原因，Docker不会导致这样

Run 文件导入错误，调度文件已经修改------------------ 原始邮件 ------------------
发件人: "J_hao104"notifications@github.com
发送时间: 2019年8月15日(星期四) 晚上7:10
收件人: "jhao104/proxy_pool"proxy_pool@noreply.github.com;
抄送: "Subscribed"subscribed@noreply.github.com;
主题: Re: [jhao104/proxy_pool] docker部署之后，获取ip很慢，要几秒钟 (#320)
检查你网络连通吧，程序到数据库 程序到你浏览器客户端
我只开了api服务，访问 127.0.0.1:5010/ 都很慢，但是有时候却又秒开的。是因为docker的原因吗？
中间网络原因，Docker不会导致这样
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


text:
#26 Fix permissions error if permissions passed as concatenated string to…

Sorry I didn't see this soon enough or I would've merged. I pushed out a nearly-identical fix to this problem


text:
Portal counts plugin will show the amount of resonators as well

Is this not useless without the portal details data, which is only available having clicked on each portal individually?

Is this not useless without the portal details data, which is only available having clicked on each portal individually?

@PrinterElf You would think that, but if you zoom in far enough that basic info from portals starts loading (when colors start showing up with higher level portals view for example) it will already contain resonator count info.

@PrinterElf You would think that, but if you zoom in far enough that basic info from portals starts loading (when colors start showing up with higher level portals view for example) it will already contain resonator count info.

could you include a screenshot of what it looks like?

Took a bit longer than I planned, but the again I was busy with actually playing all weekend..

authors: PrinterElf mxxcon schoentoon

license: isc


text:
Update README.md

Good spot, thanks!

authors: donglua jonfinerty

repo: jonfinerty/Once


text:
Fix typo

Thanks for the PR!

authors: freebroccolo texodus

repo: jpmorganchase/perspective


text:
Create 0000-01-02-jsh611.md

Pull request test


text:
Fix spelling

thanks!

authors: juliancwirko khardenstine

repo: juliancwirko/react-s-alert


text:
Resizing Octocat image

@rnetro You have a merge conflict on this PR. Look at the git fetch/git rebase workflow to try to resolve. Ping me with questions as needed.

authors: rnetro willingc

repo: jupytercalpoly/team-spirit


text:
Fix bug in CSetBasic

Thanks! (bit by copy and paste)

Yeah, took me hours to find this.

authors: isuruf jverzani


text:
Fix issue #29

Thanks!

authors: BibMartin jwass

repo: jwass/mplleaflet


text:
v0.2.5

Codecov Report

Merging #104 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #104   +/-   ##
=======================================
  Coverage   65.29%   65.29%
=======================================
  Files          13       13
  Lines         680      680
=======================================
  Hits          444      444
  Misses        236      236

Impacted Files
Coverage Δ

bowtie/__init__.py
100% <100%> (ø)
:white_check_mark:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cbfe0d0...6de0af9. Read the comment docs.

authors: codecov-io jwkvam

repo: jwkvam/bowtie


text:
Add Amazing Marvin

👍  need to give Marvin a try

authors: dariusz-wozniak jyguyomarch

repo: jyguyomarch/awesome-productivity


text:
Different Number of System files and Reference Files Supported?

Yes!

Thanks!


text:
Updated README with Capistrano 3 task

Ready to merge, but you should rebase it from master.

It was easier to create a new pull request ;) #14

repo: kavu/sprockets-webp


text:
#1152 Display start-evaluation labels

are those labels displayed for every event type or just for start evaluation?

are those labels displayed for every event type or just for start evaluation?

The labels are displayed for every event that has a labels property in the payload as a key-value pair.

repo: keptn/keptn


text:
Correct method name typo in README

Thank you for the contribution.

repo: kern/minitest-reporters


text:
Simplify retrieval of application groups

Due to extensive usage of in-clauses, we need to tweak the db url (for mariadb) to include the ?useServerPrepStmts=false  option (see #55 for more details)

repo: khartec/waltz


text:
Merge arrays without duplicates

Thanks @khezen! Do you think you could publish to NPM when you have a chance?

repo: khezen/mergejson


text:
Why can't GDC improve the performance of link prediction? Are there any intuitive and theoretical explanations?

First of, we haven't shown that it absolutely can't. We rather just found in our experiments that it doesn't.
My intuitive explanation for this is that since GDC changes the graph it necessarily loses some link information. This hurts link prediction performance, even if the smoothing done by GDC is beneficial for other tasks.

repo: klicperajo/gdc


text:
Fix the links in DEVELOPMENT.md too

/lgtm

/approve

repo: knative/sample-source


text:
defmt_itm: Fix typo

bors r+

authors: Urhengulas jonas-schievink

repo: knurling-rs/defmt


text:
Fixed http Content-length header was added 2 times

Thanks!

repo: korlibs/korio


text:
Disable iOS OTA Updates and Hide OTA Badge

@ijapija00 is here! :smile:

This commit needs serious cleanup before it can be considered for merge. There's a ton of changes that were marked by git that are irrelevant to the commit. Please keep the diff short and only keep the lines that are relevant.

That's hope he can help with sum 4K device compatible issues
Sent from my iPad

On 27 Jan 2017, at 18:44, Albert Tang notifications@github.com wrote:
@ijapija00 is here! 😄
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@coolstar is also here?
Are you two contributing to this jailbreak? I'd ask if YUCCA Utilities will work with this, but ti's better if I don't.

White space should not be unnecessarily truncated in the same commit as other changes. Keep the commit clean. If you want to do formatting changes, do it in a separate commit and pull request.

license: WTFPL


text:
Add PHP 5.6 and 7 on Travis

ping

pong ;)

"Balle de match !"

"Match nul"

Since "Roland-Garros" is now finished, could we please merge this PR @kriswallsmith? :-)

authors: Soullivaneuh lyrixx

repo: kriswallsmith/spork


text:
Upgrade kind dependency to v0.6.1 to acquire k8s 1.16+

@yue9944882 It seems the kubernetes cluster doesn't reach the readiness, and times out. I suggest you run it locally.

@hectorj2f thanks, sadly i didn't re'pro the failure locally.. let me figure out the gap..

@yue9944882
Why not bump kind to v0.7.0?

closing in favor of #1221

repo: kubernetes-sigs/kubefed


text:
ClusterLoader - API server latency

/assign @shyamjvs

/retest

/lgtm

One more thing (I've said this to you offline, but FTR) is before going further and implementing all the measurements, I think you should first start creating a very plain vanilla version of density (and probably load) test to clusterloader. You can begin by setting up a very simple CI job running a simple test (like create/delete few RCs) and then iterating on it. The job should be defined very similar to the pre-existing clusterloader job. Measurements can be added later at ease.

authors: krzysied shyamjvs

repo: kubernetes/perf-tests


text:
zh-trans: add docs/tasks/administer-cluster/_index.md

/lgtm
/approve

repo: kubernetes/website


text:
Fix Uncaught TypeError: Cannot read property 'query' of undefined

That's weird, I thought this was already there.. Well it is on my computer~ Thanks for the contribution anyway :)

authors: Filsh jenow

repo: kuzzleio/kuzzle-sdk-login-oauth-popup


text:
"Add custom token" should support ENS names in the address field

v3.9.7

Have I ever mentioned how awesome you guys are?

I still cant figure out the get decimals thing though 😒

There's an optional feature of ERC20, function decimals() returns (uint8). I was suggesting calling that on the entered name/address and, if it returns a valid value, using that.

Yeah I got that far....a lot of contracts seem to use name / symbol too so we could check all of them.
The think I'm having trouble with is the actual calling part. I tried to just rework from ENS or contracts page but it didn't work out. I'll try again tomorrow. :)

authors: Arachnid tayvano

repo: kvhnuke/etherwallet


text:
Implement FeaturesRequests/FeaturesReply

This issue depends on the issue #31, since the data of the FeaturesReply message should be stored at somewhere.

authors: beraldoleal diraol

repo: kytos/kyco


text:
Document spec.namespaceSelector

Same comment here I can get some "starter" docs and examples, I can run with it and whip it into shape.

authors: JimBugwadia chipzoller

repo: kyverno/website


text:
Preferences

Additional commentary on this PR coming on https://github.com/lbryio/lbry-desktop/issues/3022 momentarily

Closing in favour of https://github.com/lbryio/lbry-react-native/pull/57

authors: akinwale kauffj

repo: lbryio/lbry-react-native


text:
update LICENSE to 2018

lgtm, thanks

authors: kaykurokawa lbrydocs

repo: lbryio/lbrycrd


text:
small additions and corrections to first four chapters

I got the following error while running make test:
ERROR executing 02_Dependent_Type_Theory.org.32.lean, for in_code_block block starting at 930, produced output:
02_Dependent_Type_Theory.org.32.lean:4:0: warning: imported file uses 'sorry'
02_Dependent_Type_Theory.org.32.lean:6:25: error: unknown identifier 'a'
02_Dependent_Type_Theory.org.32.lean:13:6: error: unknown identifier 'a'
02_Dependent_Type_Theory.org.32.lean:14:6: error: unknown identifier 'a'
c
d

@avigad, could you check it?

Sorry! I ran the test before, fixed the problem, but forgot to commit it. I just did.

@avigad, not a problem. Just merged and closed!

authors: avigad soonhokong

repo: leanprover/tutorial


text:
update for Brandywine School District

Please re-create this pull request.

authors: clintmbishop lyzidiamond

repo: leereilly/swot


text:
Add option for custom timezone

Not going to implement this because of issues when setting process.env.TZ. See the issue at NodeJS. https://github.com/nodejs/node-v0.x-archive/issues/3286. To change the timezone for Magister Calendar, you will have to change the system's timezone.

repo: lesander/magister-calendar


text:
问下 为什么用python做pdf导出,而不是用go做导出呢?

是, 如果有精力可以尝试写这个类库, 我看有个类库https://github.com/janczer/htmlPDF   在做这个, 但是不支持中文,  大致的原理是差不错的

支持的有限，根本没办法导出符合网页看到的样式。
一般HTML转PDF会采用Chrome来处理，但是这个在linux比较复杂。


text:
Adicionar cobertura de testes

Integrar com codecov

repo: limberger/libpythonpro


text:
add map to the site

using leaflet.js
centered on the MHT building, University of Lincoln
with link to openstreetmap, and google.


text:
[TE] detection - support active flag

One question. How will this affect our existing detection configs without the new active flag?

They'll be disabled unless specifically activated. We're already seeing a bunch of unused prototypes lying around

authors: apucher jihaozh

repo: linkedin/pinot


text:
Longview

Coverage remained the same at 71.281% when pulling 7a4eceec88239302cbd5cae6f752256a2f439fe2 on yoshizzle:longview into 7c350da7fb9097a881f402197153503db8d2e242 on linode:master.

Coverage remained the same at 71.375% when pulling c54ad6e82ec11027c284774f78f3c68e3a398279 on yoshizzle:longview into 4a69f5d34023e5b8c81511507191e80fbc72c779 on linode:pending-docs-updates.

Coverage decreased (-0.03%) to 71.348% when pulling 1748fdef24cea246c764c354387a42ad3ff07898 on yoshizzle:longview into 4a69f5d34023e5b8c81511507191e80fbc72c779 on linode:pending-docs-updates.

Coverage decreased (-0.03%) to 71.348% when pulling e293928b1ef876c7c240a185d807464f8d5d40d5 on yoshizzle:longview into 4a69f5d34023e5b8c81511507191e80fbc72c779 on linode:pending-docs-updates.

authors: coveralls yoshizzle

repo: linode/manager


text:
Can't wait for your codes!!

The code for testing is available now, sorry for the delay..

repo: liu-vis/DualResidualNetworks


text:
Fixes 304; improves ShowableTile.

#304

authors: metasim vpipkt

repo: locationtech/rasterframes


text:
App pay is not included

@ahtisham-gsoftconsulting  able to implement this library? Please help me. Not able to run the project

@ahtisham-gsoftconsulting able to implement this library? Please help me. Not able to run the project

What is the issue?

@UsamaAltaf while implementing getting many errors


text:
Update ruby-lumberjack dependency to 0.0.26

LGTM

authors: ph suyograo


text:
FileStorageService: verify path resolves under the store directory

Argh. It looks like the CI is broken for external contributors. I'll fix that tonight 😓

Thank you! :)

authors: loic-sharma tmds

repo: loic-sharma/BaGet


text:
First draft with some basic structure and content definition.

I think i responded to all the open conversations. Feel free to go through and resolved the ones that have been addressed.

authors: patrickingraham tribeiro


text:
Chore: Use zero-installs

 Try yarn (berry) zero install

repo: lucassus/bookshelf


text:
Feature/TK 456 - create a new personal event for myself

Before I finalise this pull request I want to check for any known issues. It appears that new company events are not being stored or filtered properly. I will check the current sprint and backlog. @LaurieCoq @nicmarti are you able to confirm/comment ?

repo: lunatech-labs/lunatech-timekeeper


text:
sockaddr_t parsing needs to be target platform specific

depends on #93

fixed by eed0049ed0cf2295a9ab1130d2a2c90be6432e5e

repo: lunixbochs/usercorn


text:
lxc config unset acontainer aninvalidconfigkey doesn't generate an error

That's because the invalid key never actually hits the server, otherwise it sure would generate an error.
What's happening is that the client pulls the current config, then unsets the key and pushes back the config. If the key doesn't exist to begin with, no errors are raised.
So we can't really raise an "invalid key" error, but we can have the client raise a "unsetting a key that's not set" error.

On Thu, Jan 07, 2016 at 12:49:08PM -0800, Stéphane Graber wrote:

That's because the invalid key never actually hits the server, otherwise it sure would generate an error.
What's happening is that the client pulls the current config, then unsets the key and pushes back the config. If the key doesn't exist to begin with, no errors are raised.
So we can't really raise an "invalid key" error, but we can have the client raise a "unsetting a key that's not set" error.

Yep, when I looked at writing a patch before filing this it's actually
a little messier than that, because of the way it's handled
internally. Perhaps I'll get ambitious on a plane ride soon and try
and de-messy it.

repo: lxc/lxd


text:
Feature people design overhaul

@ttannis @feng-tao merge of UI changes into feature/people. I would rather merge more frequently so the merge conflicts don't pile up. (Had to resolve 10-15 conflicts for this)

It appears that my squash and merge messes up the git history from the base branch so I had to re-merge the same commits again.

repo: lyft/amundsenfrontendlibrary


text:
Release v2.0 to Supermarket?

Yeah, I looked into this briefly the other day. Problem is that I don't have a pem file to authenticate with the Supermarket, as I created this account before that was a thing (you used to be able to upload a zip file through the web interface).
There doesn't seem to be a lot of information on how the supermarket works in general, so I'm either going to need to create another account and add it as a contributor or transfer ownership, or I may need to get in touch with Chef to see what I can do to get a pem file.

You should be able to reset your pem (supermarket uses the same OCID as hosted chef) so if you log into manage.chef.io, you can reset your key, which gives you a new username.pem, etc.
I'll do a little digging. As it happens, I work for Chef, so I can help :)

You can also add me as a contributor to the cookbook via Supermarket if you'd like; my chef username is mattstratton.

Do I need to install ChefDK? We're still on chef 11 internally, we haven't had the time or need to update.
When I run this command, it tells me I'm not authorized:
knife cookbook site share windows_firewall Networking
I've updated my knife.rb file with my chef username and pem, and I've logged into manage.chef.io and supermarket.chef.io through my browser.
Things have changed quite a bit since I've dealt with this stuff, and I have to admit it's a bit frustrating to find good information about how this is all supposed to work. I haven't had a lot of success googling chef supermarket concepts. We also host our own chef instance internally, so I wasn't familiar with manage.chef.io.
Anyway, I'm just going to add you as a contributor since I'm sure you can get this up quickly.

Yeah, one reason stove is recommended is that the knife cookbook site share stuff kind of assumes the monolithic chef-repo model.
I'll just publish it. You can have the bragging rights of announcing it on the chef mailing list once it is done though :)

Published to the supermarket! Whoo!
I'll be submitting a PR that doesn't affect the published version, that just bumps the test-kitchen version since it's now been merged into master.

Thanks, your latest PR has been merged.
By the way, do you want to be added as a collaborator on this project?

Sure, if you don't mind.
Sent from Outlook
On Fri, Mar 27, 2015 at 4:06 PM -0700, "Matt Clifton" notifications@github.com wrote:
Thanks, your latest PR has been merged.
By the way, do you want to be added as a collaborator on this project?
—
Reply to this email directly or view it on GitHub.


text:
added myself

Please create new markdown file with your name e.g aasif1297.md you are not allow to edit the files just copy the data

Added new file as described and pushed to the repo.

Got it, added the required label hacktoberfest-accepted

repo: m-hamzashakeel/Hacktoberfest-Flutter


text:
Bot should @mention the owner when someone else calls dibs

done

repo: macterra/DACbot


text:
Setup Bootstrap

Setup hints on Recipes#Edit like this

Vous pouvez agrandir la fenêtre

Images circle

ideas for nav bar

http://bootsnipp.com/fullscreen/Pb5PX
http://bootsnipp.com/fullscreen/PjQN9
http://bootsnipp.com/fullscreen/2q81r

At last terminated!!!!


text:
Improve performace of EXTRACT operator

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Wamsi can you be more specific about the GPU side integration?

@testhound Need to add functions to gpu codegen QueryEngine/NativeCodegen.cpp, also need to make adjustments to QueryEngine/ExpressionRange.cpp and ParserNode.cpp.
Hope this helps!

Overall, it looks good. But as @wamsiv also mentioned, at least for this code to work on the GPU too, you'll need to add deceleration for all these new functions in the QueryEngine/NativeCodegen.cpp

I made all the requested changes for gpu integration and created a new
pull request.
On 8/7/2018 4:47 PM, Saman Ashkiani wrote:

Overall, it looks good. But as @wamsiv https://github.com/wamsiv
also mentioned, at least for this code to work on the GPU too, you'll
need to add deceleration for all these new functions in the
|QueryEngine/NativeCodegen.cpp|
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mapd/mapd-core/pull/256#issuecomment-411237944,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQKx717MUzZktldIK0bPheSzDqyApRGzks5uOib6gaJpZM4Vuk_k.

Closing this one as well, I think the two PRs were linked @testhound? If not, we can always reopen

authors: CLAassistant randyzwitch sashkiani testhound wamsiv

repo: mapd/mapd-core


text:
fix opencv3.0 flag error

thanks @edgarriba, i'm merging this into master. @oscarlorentzon is making further changes for porting to openCV 3.0  in the opencv3 branch

authors: edgarriba paulinus

repo: mapillary/OpenSfM


text:
Fixed extracting index name from error message.

Hi, it will be awesome if you could get this PR merged! it fixes  #48, #54 and possibly #51 (untested)

HI!! What's missing to prevent this PR to get merged?? It is a good addition to the codebase

this solution is better: https://github.com/matteodelabre/mongoose-beautiful-unique-validation/pull/49#pullrequestreview-336297722

authors: NikitaKrasavtsev katp4 krlozadan unicornist


text:
support different size projects

turn off wishbone and save pins, then can have smaller dies
stack up the projects to make a best fit?

Since most of this code is written in python, I think I could start working on this?
I just wanted to ask if by "support different size projects" you mean supporting arbitrary sizes in um^2, passed in projects.yaml, or would a solution where multiple blocks are joined to be above the minimum area would be enough.

Also, this might not be such a great suggestion, but wouldn't it make sense to have this tool generate layout for the bus that connects those cells together? It seems that there is a lot of space wasted between them.
In any case, if you think that I should start working on either of these, I'd be more than happy to provide a design doc and start development on it

Hey, thanks for the offer of helping!
The number of pins a macro has determines the minimum size. I also want some standardisation, so we are thinking of some options: wishbone, logic analyser, memory interface, gpio. Then you toggle them on or off and get different minimum sizes.
Re the bus, initially I thought the same, but was talked out of it by the openlane devs. You're right there is a lot of wasted space and I feel we can improve on this.
Why don't you join the next work session with me and Paweł. Check the mpw-preparation channel on the discord.

authors: mattvenn zbigos


text:
Missing Windows Build

Don't care about that

authors: maxkofler zimsneexh


text:
Fix typo in fixture definition of Martin JEM K1 Hazer

Hehe, thanks :smiley:

authors: kripton mcallegari

repo: mcallegari/qlcplus


text:
Workaround to allow the etcd scheduler to bind to the agent's IP instead of localhost

was thinking about compat of this PR w/ respect to users not running the scheduler as a mesos task but it should be fine as-is: if $LIBPROCESS_IP is undefined then the etcd-scheduler address-detect logic takes over.

authors: djosborne jdef

repo: mesosphere/etcd-mesos


text:
Adding single instance Redis service.

any updates here @joerg84 ?

repo: mesosphere/universe


text:
Add support for WHERE ANY to SelectBuilder

Landed in https://github.com/metarhia/globalstorage/commit/70f905c5eca3b4d70a8fbd3ef4dc6b03d4715162.

repo: metarhia/globalstorage


text:
[vcpkg] Avoid naming Policheck sensitive term 'Virgin Islands'

Is this an issue, given that this is an actual US territory?

/azp run

Policheck wants "US Virgin Islands" or "British Virgin Islands". The address doesn't appear to exist so I couldn't replace with the correct one and we don't care about the specific value, so I just replaced it.

/azp run

Ah, fair enough, makes sense.

authors: BillyONeal strega-nil

repo: microsoft/vcpkg


text:
Don't async anything

Fixed.

repo: mightyiam/on-text-content-edit


text:
new upstream commit detection

Codecov Report

Merging #19 (55341b3) into main (95de89c) will increase coverage by 2.51%.
The diff coverage is 92.59%.

@@            Coverage Diff             @@
##             main      #19      +/-   ##
==========================================
+ Coverage   89.05%   91.56%   +2.51%
==========================================
  Files          14       14
  Lines         274      320      +46
==========================================
+ Hits          244      293      +49
+ Misses         30       27       -3

Impacted Files
Coverage Δ

src/git_guardrails/command_line.py
73.07% <0.00%> (ø)

src/git_guardrails/validate/__init__.py
89.85% <90.00%> (+6.87%)
:arrow_up:

src/git_guardrails/cli/value_format.py
100.00% <100.00%> (ø)

src/git_guardrails/git_utils.py
100.00% <100.00%> (ø)

src/git_guardrails/validate/options.py
100.00% <100.00%> (ø)

src/git_guardrails/coroutine.py
100.00% <0.00%> (+12.50%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 95de89c...55341b3. Read the comment docs.

authors: codecov-commenter mike-north


text:
flat coverage icon

Coverage remained the same when pulling 00915cb587834c6186b1212ce11fb65ee26a820d on prefer:patch-1 into 2a37ee1aaf3afd7cf8b5e8fc0a8632707fee48f0 on mission-liao:master.

authors: coveralls prefer

repo: mission-liao/pyswagger


text:
[Feature] Subcommand 'watch' to automatically detect changes and recompile

Seems not required/not very useful


text:
Remind users to install numpy if it is not found

:+1: thanks @stefanv!

Looks good!

authors: acrellin bnaul stefanv

repo: mltsp/mltsp


text:
Add a builder for Element creation, and many small documentation changes

Really sorry, but this is a massive PR!

repo: mmarquee/ui-automation


text:
Save transaction id and added english translations

Bedankt George!
Ik heb de Duitse vertalingen toegevoegd en nog een paar andere kleine aanpassingen gemaakt.

authors: lvgunst vanengers

repo: mollie/Prestashop


text:
wrong abbreviation fix for month Maart

Please accept this change. Right now I have to do this .replace('Mrt', 'Mar')

@wernerm can you comment on this, please.

@jmaleonard please follow https://github.com/moment/moment/blob/develop/CONTRIBUTING.md#changing-locale-files
You need to update src/locale/af.js and ser/test/locale/af.js

@jmaleonard ok, CLDR agrees with you

Close in favor of #3272

authors: ichernev jmaleonard

repo: moment/moment


text:
fixed one-time binding problems

👍

Approved !

authors: lihorecoco tunix

repo: monitise-mea/gerrit-dashboard-web


text:
Fixed cause of AccessViolationException

@ddobrev not it should be ok

I'm more worried about the lack of tests here.

@zillemarco @tritao 's right, what is the problem that this fixes and can it be tested?

@tritao @ddobrev I'm working on the tests at the moment. As soon as I have some results I'll tell you but until I'm done I'd leave this PR aside. Strange things happen on my side....

@ddobrev I removed the != nullptr. Now it should be fine

@zillemarco the code is great. Thank you for putting up with me.
Please do me one last favour: rebase your code instead of merging master, and combine all of your work into one commit. You can use rebase and amend, and then push force.

@ddobrev I just fixed it, I was fighting with GitHub Desktop and I'm not used to use git from command line :)

@zillemarco beautiful. Thank you very much.

authors: ddobrev tritao zillemarco

repo: mono/CppSharp


text:
add emoji completion

Anyone have good experience with existing libs?

https://github.com/bevacqua/horsey this one is very small, good and optionated. built on top of other super small and focused libs like https://github.com/bevacqua/fuzzysearch which is awesome.

muan's emoji project will be helpful i think, includes synonyms: https://github.com/muan/emoji/

Hey, I've been working on an emoji-chat project for fun. Here's a link to my repo.  At.js combined with emojify.js is easy enough to implement, and has some good fuzzy matching.

authors: bcomnes beaugunderson mshenfield tunnckoCore

repo: moose-team/friends


text:
Update stack snapshot

Thanks! This seems to fix the problems with building using stack.

authors: mortberg vlopezj

repo: mortberg/cubicaltt


text:
Needs better/more code comments

But, does it?

repo: mrverrall/go-row


text:
Is Material UI components WCAG 2.0 compliant?

@kjain-dev Yes, they should be. #14187 for broader concern.

Can you please confirm its compliance for 2.1 version

Can you please confirm its compliance for 2.1 version

We don't have the bandwidth to do that at the moment. It would be awesome if we could include WCAG compliant example usage on the documentation but this is a stretch goal right now unfortunately.

@eps1lon | @oliviertassinari  What are your release cycles ? I hope we can just share our concerns related to the compliance and you guys can address them in the code.
cc: @kjain-dev

@vaibhavsah https://material-ui.com/versions/#release-frequency

@oliviertassinari Thanks for the link. It seems you have a release due in October. Is it feasible for your team to make the updates relates to WCAG2.1

v5 won't happen in October, more likely Q1 2021. Thanks for that, I will update the reference.

Is it feasible for your team to make the updates relates to WCAG2.1

@vaibhavsah I can't answer your question. Let's switch it to. Could you please audit all our components and http://next.material-ui.com/ and report violations, thanks.

Would love to know is MUI library is WCAG compliant or not.

Would love to know is MUI library is WCAG compliant or not.

I don't think so... look at the opened accessibility issues there:
https://github.com/mui/material-ui/issues?page=1&q=is%3Aissue+label%3Aaccessibility+is%3Aopen

authors: RadhikaTakyar abdermaiza eps1lon kjain-dev oliviertassinari vaibhavsah

repo: mui/material-ui


text:
edit readme

thanks for PR @aguanslamet


text:
Add backend unit tests

I can take a stab at this

Goal should be to have at least one positive and one negative test case for each endpoint.

Hi, i can work on this i have some ideas to apply, or you can add more issues that i can solve during the hacktobertfest

A PR will be accepted as soon as all change requests are implemented.

authors: SpencerHanson n1try williamtoll


text:
Let svlogd rotate logs

Resolves nanobox-io/nanobox-docker-runit#6

repo: nanobox-io/nanobox-hooks-code


text:
Fixes for booting core-image-sato built with musl/clang on dragon board

@ndechesne anything more needed from my side ?

@kraj : https://cgit.freedesktop.org/mesa/drm/commit/?id=739b8886025b31f2f9d09a535c8ff3670bb1d036

@ndechesne bravo

authors: kraj ndechesne

repo: ndechesne/meta-qcom


text:
Implement Webhook package

I'm closing the webhook stuff for now. If it should become important, we can create a new ticket.

authors: dbrower msuhovec

repo: ndlib/bendo


text:
Set collection items output to top level only

Coverage remained the same at 93.01% when pulling f92e8f2d6eda772d718c6311c849e0f9925f3e11 on top-level-items into 768fd3078e04f086e0f2f4a6f26e749ec55444a7 on master.

repo: ndlib/honeycomb


text:
Initial commit

Initial commit

Oops, my bad

repo: nea/Typescript-Starter


text:
Update fork to project master

Sorry, forgot to swap the sources!


text:
Better handle users in different access groups

This isn't urgent and probably doesn't deserve a point release now that Luca has deployed a workaround, but it's good cleanup of an edge case.
Also, can whoever merges this use the "rebase and merge" option? It's accessed from the dropdown attached to the "merge pull request" button and will keep the history cleaner by not adding merge commits.

Looks good to me, LGTM

authors: bearloga neilpquinn

repo: neilpquinn/wmfdata


text:
Consider using Asciidoc Java parser

After discussing with @Mogztter:

parsing multi-line document attributes does not seem to work super well (we tried different things: sometimes lost the newlines, sometimes got unwanted XML entity escaping)
attributes may not be a good fit for what we're trying to do

The suggestion is to use to source listing with a specific role, and it indeed works like a charm.
In short, going from:
:param-name: category
:param-value: Dairy Products
:result-column: product
:expected-result: Geitost
:query: MATCH (p:Product)-[:PART_OF]->(:Category)-[:PARENT*0..]-> +
 (:Category {categoryName:$category}) +
 RETURN p.productName as product +

to:
[source,cypher,role=query-example,param-name=category,param-value=Dairy Products,result-column=product,expected-result=Geitost]
----
MATCH (p:Product)-[:PART_OF]->(:Category)-[:PARENT*0..]->
 (:Category {categoryName:$category})
 RETURN p.productName as product
----

Not only is it easier to parse with AsciidoctorJ, it is also less error-prone to write (e.g.: no more + for newline).

If we go down that route, we'll have to update the instructions here.


text:
Add ratpack test

@hydra1983 Thanks for your pull request. I have merged it and published the result. It looks pretty good comparing with servlet container based frameworks.

Thank you :D

repo: networknt/light-java-example


text:
Fixed: infra agent install overwrites config files

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Noah McGivern seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant noahmmcgivern

repo: newrelic/open-install-library


text:
does it support CUDA9?

no and yes (need some change of code)

repo: nicehash/nheqminer


text:
need more demo to show how this engine can reach to

This is on our Todo list.  Stay tuned :)

boardgame fan!


text:
Add a UDF to convert XML to JSON

Done in #48

repo: nineinchnick/trino-rest


text:
Test webhook

Add comment

Testing 2

Testing3

Testing


text:
add ast parser

Codecov Report

Merging #4 into master will decrease coverage by 6.45%.
The diff coverage is 70.28%.

@@             Coverage Diff              @@
##             master       #4      +/-   ##
============================================
- Coverage     86.06%   79.60%   -6.46%
- Complexity      126      159      +33
============================================
  Files             8       10       +2
  Lines           366      613     +247
  Branches         34       67      +33
============================================
+ Hits            315      488     +173
- Misses           33       82      +49
- Partials         18       43      +25

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/org/nobloat/bare/dsl/AstParser.java
59.21% <59.21%> (ø)
30.00 <30.00> (?)

src/main/java/org/nobloat/bare/dsl/Ast.java
98.41% <98.41%> (ø)
0.00 <0.00> (?)

src/main/java/org/nobloat/bare/dsl/Lexer.java
83.33% <100.00%> (+0.98%)
29.00 <19.00> (+2.00)

src/main/java/org/nobloat/bare/dsl/Scanner.java
85.10% <100.00%> (ø)
15.00 <0.00> (+1.00)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ebee492...b428112. Read the comment docs.

authors: codecov-io oliverselinger

repo: nobloat/bare-jvm


text:
Figure out logging

Closing in lieu of #580

This isn't the same as #580.
We need to define a logger in app_config (it should be able to vary per target) and stop using print statements.

Done yussss

authors: TylerFisher eads onyxfish

repo: nprapps/app-template


text:
Trouble compiling on save with go-plus and atom, not in terminal

Please delete this issue as it was mistakenly put here.

repo: nsf/gocode


text:
[W6.6h][T13-B1] Rattanin Siripornpitak

Please close this PR.


text:
make native NuvlaBox MQTT data FIWARE compliant

won't do, because now MQTT data is equal to Nuvla telemetry, with he same format

won't do, because now MQTT data is equal to Nuvla telemetry, with he same format

repo: nuvlabox/agent


text:
Facilitate automatic preprocessing of ELAN files

Import from ELAN definitely seems important. Development of ELAN seems set to continue in future years. Version 5 (2017) has Interlinearization Mode, a desirable tool for linguists.

authors: alexis-michaud oadams

repo: oadams/persephone


text:
Mediacore replacement

Finished review. Back to @sallakarppinen for follow-up

Thanks for figuring out the CSS issue, @nicolaasmatthijs. I've made some fixes and deployed them to qa2 server.

Finished another round of review. Back to @sallakarppinen

Added some more review. Back to @sallakarppinen

Finished another round of review. Back to @sallakarppinen

Left some final comments. Back to @sallakarppinen

Made some changes (including tidying up repetition as brought up by @nicolaasmatthijs)... Had some weird behaviour on qa2, it seems that using position: unset caused the video to not play properly so I've swapped back to width: 98%.

Looks like the "play" icon for MP3 is a bit out of wack:

There are also issues when you play it with the player UI, and the file only seems to play the first half a second or so of the file then stops:

The following functional issues were identified during the bugbash:

The Processing this file screen is not correctly updated once the video has finished transcoding and requires a refresh of the screen before the video can be seen. We expect a push notification to inform the UI that it has finished processing and update the content preview. This is in place for other types of content already.

The video player needs to align on both ends with the comments widget at all widths. That's why I was pushing back on the 98% width as I can't see how that can ensure the alignment. In the screenshot below, you'll see red lines where the video is expected to align:

@nicolaasmatthijs: For issue 2, could you specify the browser and OS that were used, since I'm  not able to reproduce (on Firefox/Linux Mint)?

@sallakarppinen : It appears to be any browser (Firefox / Chrome / Safari). Maybe try changing the width of your browser window as there's a single "perfect" width where they do roughly align (because of the percentage width).

@nicolaasmatthijs You're right, I can see it if I resize to something other than the 'standard' sizes... I'll try to find a way around it.

Have not been able to fix the resizing, but the first issue (file not refreshing when processed) is fixed by the latest changes to Hilary branch for this task which is deployed on qa2.


text:
Bump Feather version to 4.28.0

Superseded by https://github.com/oblador/react-native-vector-icons/pull/1218

authors: oblador rfbezerra

repo: oblador/react-native-vector-icons


text:
Fix the ci

LGTM 👍

authors: heitorPB jamesbeedy

repo: omnivector-solutions/slurm-charms


text:
Version updated to 2021.1.3

/intelci: run

repo: oneapi-src/oneDAL


text:
Fix PHP-style array element in settings json

Coverage remained the same at 97.88% when pulling 7a502199c822fb8876af9561f6339fd6f421bb16 on ThiefMaster:patch-1 into 59c0182c0760036b37130280cfc71d55f0d594ef on onelogin:master.

authors: ThiefMaster coveralls

repo: onelogin/python-saml


text:
split clockset.sh into oref0-set-device-clocks.sh and oref0-set-system-clock.sh

does not work
edison@Bender1061:~/johnopenaps1$ ~/src/oref0/bin/oref0-set-system-clock.sh     Waiting for ntpd to synchronize... No! ntpd did not synchronize. Restarting ntp (via systemctl): ntp.service. Waiting for ntpd to synchronize... No! ntpd did not synchronize. Fri Dec 31 19:25:05 EST 1999 edison@Bender1061:~/johnopenaps1$

I think the issue has to do with the sort, I tried last night from using the tail -1, to a head -1 and it worked correctly. I believe it's because it's seeing the two number of seconds (since beginning of time) as a string with the first number being the one that it will sort with.
in C I would do a StrtoNum and then just pick the greater of the two numbers and use that. but I know you are more familiar with the bash scripting.

Interesting. What were the two epoch time strings, and how did it sort them lexically?
The bash equivalent of that numeric sort would be sort -g - might be worth a try.

edison@Bender1061:~/johnopenaps3$ (date +%s; date -d ( cat $CLOCK; echo ) | sed 's/"//g' +%s) | sort
1475550880
946684887
edison@Bender1061:~/johnopenaps3$

edison@Bender1061:~/johnopenaps3$ (date +%s; date -d ( cat $CLOCK; echo ) | sed 's/"//g' +%s) | sort -g
946685364
1475550880
edison@Bender1061:~/johnopenaps3$ (date +%s; date -d ( cat $CLOCK; echo ) | sed 's/"//g' +%s) | sort
1475550880
946685372
edison@Bender1061:~/johnopenaps3$
Yeah it looks like the sort -g works

Ah, so y2k is just under 1 billion.  Thanks: I'll update the sort to use -g.

so the command oref0-set-system-clock works perfectly now, but if the command is never called automatically when the enact alias is called.

I made a fix for that in the oref0-setup branch.  Just cherry-picked it over: https://github.com/openaps/oref0/pull/234/commits/5fa9d937bf729eb740ae9500377c58a94c56975b - you'll need to re-run oref0-setup to update your loop accordingly.

So this is good to merge now, right?


text:
Fix bug Vector3 initialization

Thanks!

authors: ennehekma kartikkumar

repo: openastro/atom


text:
Installation CPV_ENV fix

Thanks @dragoonis

authors: chartjes dragoonis

repo: opencfp/opencfp


text:
Bump version for v0.5.0 release

I have a ChangeLog commit (through 6734c7a, #370) here 1 if you want
to put that in before the 0.5.0 commit.

pulled in your ff6e729d62c8e9062ac98fcc8cdb4a5cefa52707 commit
On Tue, Apr 12, 2016 at 1:19 AM, W. Trevor King notifications@github.com
wrote:

I have a ChangeLog commit (through 6734c7a, #370) here 1 if you want
to put that in before the 0.5.0 commit.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/opencontainers/runtime-spec/pull/379#issuecomment-208711366

LGTM

@crosbymichael @vishh PTAL

LGTM

LGTM

@vbatts i'll let you merge and tag

Looks like this missed bumping the example to 0.6.0-dev [1,2].  Maybe
add a note to project.md?

@wking not a strict need.

On Tue, Apr 12, 2016 at 12:22:46PM -0700, Vincent Batts wrote:

@wking not a strict need.

Then I'd rather use a real version number there instead of the current
-dev.  And we'd need to remember to bump when any 0.6.0-specific
content landed in the example.

authors: crosbymichael mrunalp vbatts vishh wking

repo: opencontainers/runtime-spec


text:
Support for replace function

Hello @mad,
I assume that you are using JanusGraph, which is not upgraded to TinkerPop 3.4.0 yet, so Cypher for Gremlin 1.0.0+ will not work there.
Do you need #255 to be merged to Cypher for Gremlin branch that uses TinkerPop 3.3.4?

Hello @dwitry
Thanks, It would be good

Merged to tinkerpop334 branch

authors: dwitry mad

repo: opencypher/cypher-for-gremlin


text:
Precond mechanism subgoaling (ANSNA insight :) )

no, let's improve control and it will come in a cleaner way.

repo: opennars/opennars


text:
Release v0.22.0

released! https://crates.io/crates/k

released! https://crates.io/crates/k

authors: OTL taiki-e

repo: openrr/k


text:
Update: kit

Codecov Report

Merging #135 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #135   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          58     58
  Lines         666    666
=====================================
  Hits          666    666

Impacted Files
Coverage Δ

problems/add-two-numbers/add_two_numbers.go
100% <ø> (ø)
:arrow_up:

...s/merge-two-sorted-lists/merge_two_sorted_lists.go
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d67fa31...9c545db. Read the comment docs.

authors: codecov-io openset

repo: openset/leetcode


text:
Add: Navigation

Codecov Report

Merging #361 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #361   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         127    127
  Lines        1315   1315
=====================================
  Hits         1315   1315

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1567e04...ff843aa. Read the comment docs.


text:
bump login/sync plugin versions for pom repository URL fix

@bparees @adambkaplan @nstielau fyi
after this, I'll need to open an issue on the ART board to get them to pull these new versions into our RPM

Installed plugins:
ace-editor:1.0.1
apache-httpcomponents-client-4-api:4.5.5-3.0
authentication-tokens:1.3
blueocean-autofavorite:1.2.2
blueocean-bitbucket-pipeline:1.10.2
blueocean-commons:1.10.2
blueocean-config:1.10.2
blueocean-core-js:1.10.2
blueocean-dashboard:1.10.2
blueocean-display-url:2.2.0
blueocean-events:1.10.2
blueocean-github-pipeline:1.10.2
blueocean-git-pipeline:1.10.2
blueocean-i18n:1.10.2
blueocean-jira:1.10.2
blueocean:1.10.2
blueocean-jwt:1.10.2
blueocean-personalization:1.10.2
blueocean-pipeline-api-impl:1.10.2
blueocean-pipeline-editor:1.10.2
blueocean-pipeline-scm-api:1.10.2
blueocean-rest-impl:1.10.2
blueocean-rest:1.10.2
blueocean-web:1.10.2
branch-api:2.0.20
cloudbees-bitbucket-branch-source:2.2.15
cloudbees-folder:6.6
conditional-buildstep:1.3.1
config-file-provider:3.5
credentials-binding:1.15
credentials:2.1.19
display-url-api:2.2.0
docker-commons:1.11
docker-workflow:1.14
durable-task:1.26
favorite:2.3.1
git-client:2.7.3
github-api:1.90
github-branch-source:2.3.6
github:1.29.2
github-organization-folder:1.6
git:3.9.3
git-server:1.7
handlebars:1.1
handy-uri-templates-2-api:2.1.6-1.0
htmlpublisher:1.16
jackson2-api:2.8.11.3
jenkins-design-language:1.10.2
jira:3.0.5
job-dsl:1.72
jquery-detached:1.2.1
jsch:0.1.54.1
junit:1.26.1
kubernetes-credentials:0.3.0
kubernetes:1.12.8
lockable-resources:2.5
mailer:1.21
mapdb-api:1.0.9.0
matrix-auth:2.2
matrix-project:1.14
mercurial:2.3
momentjs:1.1
openshift-client:1.0.30
openshift-login:1.0.19
openshift-sync:1.0.38
parameterized-trigger:2.35.2
pipeline-build-step:2.7
pipeline-github-lib:1.0
pipeline-graph-analysis:1.7
pipeline-input-step:2.8
pipeline-milestone-step:1.3.1
pipeline-model-api:1.3.4.1
pipeline-model-declarative-agent:1.1.1
pipeline-model-definition:1.3.4.1
pipeline-model-extensions:1.3.4.1
pipeline-rest-api:2.10
pipeline-stage-step:2.3
pipeline-stage-tags-metadata:1.3.4.1
pipeline-stage-view:2.10
pipeline-utility-steps:2.1.0
plain-credentials:1.4
pubsub-light:1.12
run-condition:0.10
scm-api:2.2.8
script-security:1.56
sse-gateway:1.16
ssh-credentials:1.14
structs:1.17
subversion:2.10.3
token-macro:2.6
variant:1.1
workflow-aggregator:2.6
workflow-api:2.32
workflow-basic-steps:2.11
workflow-cps-global-lib:2.11
workflow-cps:2.65
workflow-durable-task-step:2.26
workflow-job:2.29
workflow-multibranch:2.20
workflow-remote-loader:1.5
workflow-scm-step:2.6
workflow-step-api:2.19
workflow-support:3.2

ART req for RPM update https://jira.coreos.com/browse/ART-671

ART req complete
dist git http://pkgs.devel.redhat.com/cgit/rpms/jenkins-2-plugins/commit/?h=rhaos-4.2-rhel-7&id=4caffd3dd34032cc751e00bdcf74c3d70bc82f72

for reference master PR https://github.com/openshift/jenkins/pull/867 merged 7 days ago

repo: openshift/jenkins


text:
Fix SSL support between master and node(s)

@twiest Please review
@detiber @akram FYI

@jwhonce you'll also want to add something like this: https://github.com/detiber/openshift-online-ansible/blob/enterprise2/roles/base_os/tasks/main.yaml#L14-L20
Otherwise, the cli commands are broken by default with https enabled.

@detiber Did you want me to add that code in this PR, or are you going to create a PR with it?

@jwhonce I can create an additional PR for that. Incoming shortly

authors: detiber jwhonce

repo: openshift/openshift-online-ansible


text:
Add documentation around OnBuild and contextDir

@soltysh @bparees PTAL

lgtm but we need a pull to openshift-docs too i think.

@bparees https://github.com/openshift/openshift-docs/pull/170 (still working on the content)

[merge]

authors: bparees mfojtik

repo: openshift/source-to-image


text:
p5_scrypt.c: fix error check of RAND_bytes() call

Can anyone explain this test failure to me? (see https://travis-ci.org/openssl/openssl/jobs/367508858)
Test Summary Report

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

../../_srcdist/test/recipes/70-test_key_share.t              (Wstat: 15360 Tests: 21 Failed: 0)

  Non-zero exit status: 60

  Parse errors: Bad plan.  You planned 22 tests but ran 21.

Files=146, Tests=1336, 201 wallclock secs ( 5.35 usr  1.48 sys + 76.70 cusr 23.44 csys = 106.97 CPU)

Result: FAIL

make[1]: *** [_tests] Error 1

Yup
ok 21 - Ignore key_share for TLS<=1.2 server

Server command: ../../util/shlib_wrap.sh ../../apps/openssl s_server -max_protocol TLSv1.3 -no_comp -rev -engine ossltest -accept [::1]:0 -cert ../../../_srcdist/apps/server.pem -cert2 ../../../_srcdist/apps/server.pem -naccept 1 -cipher AES128-SHA -ciphersuites TLS_AES_128_GCM_SHA256 -curves X25519

engine "ossltest" set.

Using default temp DH parameters

ACCEPT [::1]:52821

Server responds on [::1]:52821

unable to connect: Operation timed out

# Looks like you planned 22 tests but ran 21.

# Looks like your test exited with 60 just after 21.

Dubious, test returned 60 (wstat 15360, 0x3c00)

Failed 1/22 subtests

We do have that issue going on a little now and then.  @dot-asm  and @bernd-edlinger seem to be working on it

The patch applies cleanly to 1.1.0. Branch 1.0.2 does not seem to be affected.

Merged!
e62fb0d31bdf25854aa2c7cda8e1d03768984ab4 master
69712507e73437553790ccac6f19a9ded996c0cd 1.1.0

authors: levitte mspncp

repo: openssl/openssl


text:
[guides] Update site/en/guides/build/features/jam-detection.md

@jwhui This one is ready for approval/merging

authors: Vyrastas superwhd

repo: openthread/ot-docs


text:
Document how to scope an operator

@dmesser @kevinrizza @dinhxuanvu I have addressed all available comment. Please re-review.

lgtm

authors: awgreene kevinrizza

repo: operator-framework/olm-book


text:
Kubebuilder layout ansible tests, by Fabian

@camilamacedo86 these tests are not for scaffolding but for more advanced integration tests of Ansible + ansible-runner + the kubernetes modules, there's another test (that's not updated by this PR) that tests the scaffolded code

Hi @fabianvf,
I am not too much the point of we have tests in the project that are not executed from the code which is gen via the tool. However, if it is required for we move forward and have the Ansible plugin I am OK with. See that with kbutils we can inject code and manipulating the files in the Go tests. (e.g https://github.com/kubernetes-sigs/kubebuilder/blob/master/test/e2e/v3/e2e_suite.go#L84-L92 and https://github.com/kubernetes-sigs/kubebuilder/blob/master/test/e2e/v3/e2e_suite.go#L124-L158)
Could we track the need to replace this for a generated code via an issue in the repo and a todo in the files? Also, would not be better we put the mock project in a dir named testdata?

I am of the same mindset as @camilamacedo86 is. Putting static scaffolded files in the repo isn't the best. That's what test/test-framework is (was?), and it was generally a source of problems. If we can avoid it, it would be nice.
How much differs in this test project vs a vanilla test project? If it isn't much, could we do as Camila suggests and fully-regenerate it during the e2e using the actual plugin code and then applying the changes?
+1 on putting this in a testdata directory.

This PR can be replaced by : https://github.com/operator-framework/operator-sdk/pull/4011
c/c @asmacdo @fabianvf

@asmacdo 50% of this pr is addressed already with https://github.com/operator-framework/operator-sdk/pull/4011 and for the advanced scenarios, we have the issue;  https://github.com/operator-framework/operator-sdk/issues/4025.
So, since it is outdated and if we close it we still able to use as a helper to achieve the #4025 I think it is fine to close this one. However, feel free to re-open if you think that it should not be close at all.

authors: asmacdo camilamacedo86 fabianvf joelanford

repo: operator-framework/operator-sdk


text:
adding a certificate provider to install ssl certs into the store

Do you have some examples on how to use this? Maybe update the README.md with description, usage and examples.
From reading the code it looks like you are creating a new store for each certificate instead of adding the certificates to an existing store.  Is there a reason for this design?

Yes I'll update the README.md
As for the store, it's adding it to the current store and you are specifying the StoreName. This was obtained from here: http://ss64.com/nt/certutil.html

#212 and this attack the same problem and has some of the same issues.  I'll reconcile these and merge it in manually.

Thanks for taking the time to contribute code!

authors: Tech356 blairham ksubrama

repo: opscode-cookbooks/windows


text:
Added ubuntu8.04-gems bootstrap template.

We plan to create a new project called knife-bootstrap to hold bootstrap templates moving forward. See CHEF-2348 for more details.

authors: btm patcon

repo: opscode/chef


text:
Add maven-badges to README.md

LGTM

repo: opt-tech/chronoscala


text:
#1 couchbase persistency, test suite implementation

@CanerPatir,
Thank you for your great efforts:)

authors: CanerPatir osoykan

repo: osoykan/DDDify


text:
Switch the service name to ruby.

Gonna merge this one, the one from before actually took down NPM and gave us two rubys...

Oh no! That's my bad. Good catch. Fortunately the refactor to bundle these will also make this not a thing in the future.

authors: dlorenc jordan-wright

repo: ossf/package-feeds


text:
Add support to specify a branch

For readability I named the parameter branch, the URL should look like this:
http://localhost:8000/?branch=replace-on-matched-line#/sed?_k=r67fkk


text:
Fix readme

Pull Request Test Coverage Report for Build 567345403

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.018%

Totals

Change from base Build 567341156:
0.0%

Covered Lines:
373

Relevant Lines:
436

💛  - Coveralls

authors: coveralls ota-meshi


text:
Enhanchment: Adding Scenes to Accessories Tab

Same as https://github.com/oznu/homebridge-config-ui-x/issues/84

authors: oznu smartupyourhome

repo: oznu/homebridge-config-ui-x


text:
punkte wieder hinzugefügt (rollback)

should be ok

should be ok

authors: SomeOtherGod dadope

repo: pacmanproject/pacman


text:
Buildpack ordering rfc

Sorry for the reverts, please squash before merging :).

@joshzarrabi this looks great to me. Could you merge it? We'll get separate issues filed to get our builders to conform to this proposal.

@kvedurmu I dont have permission to merge.

Ahh forgot about maintainers :). @paketo-buildpacks/release-engineering could y'all merge this?

authors: joshzarrabi kvedurmu

repo: paketo-buildpacks/builder


text:
(Cheap) Count ranged tombstones

Explaining the metrics for future observers:
In Cassandra 2.x, range tombstones are treated differently than cells, where as in C* 3.x, range tombstones are treated as cells. This is important detail, as the iterators used when counting/viewing cells, do not contain range tombstones, as they're not cells. Thus, we've to wrap the iterator which reads from disk, but then we're only getting tombstones from disk -- leaving any tombstones in a memtable ignored.
So we end up with three different metrics:
(1) RangeTombstonesReadHistogram: Total number of range tombstones read from disk
(2) DroppableRangeTombstonesReadHistogram: Total number of droppable range tombstones read from disk
(3) RangeTombstonesHistogram: Total number of range tombstones, both memory and disk.
Which gives us the following: RangeTombstonesHistogram >= RangeTombstonesReadHistogram >=  DroppableRangeTombstonesReadHistogram
We cannot figure out the number of droppable tombstones for both memory/disk as we'd need to do an O(N) scan (N being number of range tombstones) to determine which are droppable -- there's also no convenient place to put this in the code where we are already performing the scan.

repo: palantir/cassandra