text:
Minor revisions.

Looks good, thanks!
I notice that in some of these early notebooks the %matplotlib inline cell and a cell of import statements appears at the top of the notebook, whereas in most later notebooks we have moved all the code cells to after the initial introductory markdown cells.  Should we try to make this more consistent?

I notice that in some of these early notebooks the %matplotlib inline cell and a cell of import statements appears at the top of the notebook, whereas in most later notebooks we have moved all the code cells to after the initial introductory markdown cells. Should we try to make this more consistent?

Okay, I've incorporated that change into this PR.  I'm still working through the chapters, so this is still WIP.

In the shallow water chapter, I changed the code so that certain characteristics are plotted by default; this way they show up in the book.  This is important since we discuss how these characteristics show that the solution is unphysical in these two cases.

I'd like to go ahead and merge this now, since other chapters are being edited concurrently and it would be good to get these changes to the copy editors.  Please review.

I made the pdf from this PR before I read through it all the other day, and in general it all looks fine to me, and can be merged as far as I'm concerned.  I did make a few notes of some minor changes in these notebooks, but I can do those as part of my next pass through updating the notebooks.
So if you are done updating for now then once this is merged I'll make a pass in the next couple days.

Sounds good.  I am planning to go through the remaining chapters, but it looks like I won't get to that for a week or so at least.  So go ahead and make a pass.

authors: ketch rjleveque


text:
add interpolation of template variables

Thanks for the PR, much appreciated :).

Just a note, I'm going to separate the two changes.
I'll release interpolation support in version 1.1 .
But the multi ami-ids / multi regions support will go to 2.0 later on since it changes the behaviour of this post processor.
I think the guts of the second change is to handle multi builds, I'll modify your PR a bit to add more flexibility to the output. There are users who do not wish to use region as the output key, others wish to use other field as key like AMI name.

source: gharchive/pull-request

authors: bemehow cliffano

license_type: permissive

license_source: bigquery


text:
Fix stepper issue with redirect not working

Only found one existing place this was used (edit app). Confirmed fixed

authors: nwmac richard-cox

repo: cloudfoundry-incubator/stratos

license_source: github-api


text:
Use BOSH 2.0 syntax in release docs.

Also didn't notice #319 . Please merge #319 first and I will resolve conflicts.

repo: cloudfoundry/docs-bosh


text:
Can genie support contiv?

@woshihaoren, we have a sort of generic mechanism wrt various plugin handling which enables the support for most of the plugins seamlessly, we will have a look into this and get back to u with confirmation, thanks

I tried cni-genie with contiv-cni added to supported cni list and it worked seamlessly.

however i am lost getting flannel cni co-exist. This is in k3s environment and i don't see flannel binary present in /opt/cni/bin directory..

source: gharchive/issue

authors: sushanthakumar vdabbara woshihaoren

repo: cni-genie/CNI-Genie


text:
wrap long lines to make it easier to read in terminal

Trivial, merging.

repo: coderefinery/installation


text:
Test bors

bors ping

@bors ping

bors try

bors r+

license: MIT


text:
Use Java 8 to build release artifacts

What's the rationale?

Before this PR, we published with Java 9. Although we explicitly select the target as Java 8, some binary incompatibilities sneaked in all of our current releases. So, I think it's safer to just use Java 8 for compiling the release versions.
See the output of this experimental integration of MiMa in the mill build:
09:59:20 ⌁ lefou:~/work/opensource/mill mima+* 130 ± mill -j 0 main.__.mimaReportBinaryIssues
Using experimental parallel evaluator with 16 threads
[#07] [281/306] main.core.compile
[#11] Scanning 79 classfiles for binary compatibility in /home/lefou/work/opensource/mill/out/main/api/compile/dest/classes ...
[#03] [282/306] scalalib.api.localClasspath
[#11]  * static method transferTo(java.io.OutputStream)Long in class mill.api.DummyInputStream does not have a correspondent in current version
   filter with: ProblemFilters.exclude[DirectMissingMethodProblem]("mill.api.DummyInputStream.transferTo")
[#11]  * static method readNBytes(Array[Byte],Int,Int)Int in class mill.api.DummyInputStream does not have a correspondent in current version
   filter with: ProblemFilters.exclude[DirectMissingMethodProblem]("mill.api.DummyInputStream.readNBytes")
[#11]  * static method readAllBytes()Array[Byte] in class mill.api.DummyInputStream does not have a correspondent in current version
   filter with: ProblemFilters.exclude[DirectMissingMethodProblem]("mill.api.DummyInputStream.readAllBytes")

You can see that comparing the HEAD of 0.9.x branch with the 0.9.9 tag, some methods only defined in Java 9 sneak into our API. This happens when I run (and compile) mill with Java 8.

Thanks for the explanation!

Glad that the mill-mima plugin is already creating some value 😍

authors: ice1000 lefou lolgab

repo: com-lihaoyi/mill


text:
typo

Totes didnt mean to do this on your fork I'm just tired and inattentive. Thx0rz for not making fun of how absurd this request for a merge was tho haha

license: mit


text:
Refactor frame

I see a lot of repeater color definitions in the code. Can we replace those by a define with a name of the color?

Why did the jenkins test not run? Can you rebase to latest master so we see CPU usage in CI?

repo: commaai/openpilot


text:
draw lead indicator in 3D

Fixes #2545

Fixes #2545


text:
set up exercise ml basics anew

1c: weiß grad auch nicht mehr, was ich da meinte - ist raus
1d: Formulierungen geändert // mit dem feature space sprichst du ein generelles Problem an - hab mich auch schon öfter gefragt, wie wir mit feature extraction umgehen (am saubersten wäre wohl eine Trennung von in put und feature space? Trennen wir aber aktuell nicht.) Der log hat eh keinen tieferen Sinn hier, dann nehm ich ihn raus :)
1e Groß Theta drin
1g Stimmt, der Zusatz macht keinen Sinn // Transponiertzeichen korrigiert // Hab die Likelihood-Version auf die erste mit Produktzeichen geändert (war mir nicht sicher vorher), aber das sollte die Korrespondenz von MLE und ERM am beteen verdeutlichen // Ja, darauf würde ich in der Lösung eingehen - ist ja üblich, die gesamte Lik anzugeben, sie sollen dann checken, dass der Loss point-wise ist und ich denn im empirischen Risiko eben summiere
1h Richtig, das hätte auch Risiko heißen sollen in g :) // Formulierung angepasst

1d: Formulierungen geändert // mit dem feature space sprichst du ein generelles Problem an - hab mich auch schon öfter gefragt, wie wir mit feature extraction umgehen (am saubersten wäre wohl eine Trennung von in put und feature space? Trennen wir aber aktuell nicht.) Der log hat eh keinen tieferen Sinn hier, dann nehm ich ihn raus :)

=> Habe dazu mal ein Issue aufgemacht: https://github.com/compstat-lmu/lecture_i2ml/issues/658

 I2ML Exercise 1 Solution

authors: lisa-wm ludwig-max


text:
Updated dummy

Changes Unknown when pulling 8205bb1b73e48e4ca44bdeab4baceadc2c882128 on fb-new-dummy into ** on master**.

authors: coveralls frbl

repo: compsy/vsv


text:
rerender with auto

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

authors: conda-forge-linter ocefpaf

repo: conda-forge/pyqt-feedstock


text:
Add missing chars for getopt_long in bwtest and kafka client.

Great, thank you! I had totally missed the single-char options.

repo: confluentinc/bottledwater-pg


text:
update env doc to explain local functionality

 document local Environment in environment section of docs

Looks good but does not cover the fact that variables specified in the docker-compose.yml are required and convox start will block until they are set.

authors: ddollar mattmanning


text:
Merge parse yaml

@dankohn : Seem okay?

Codecov Report

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

@@          Coverage Diff          @@
##           master   #900   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          45     45
  Lines        1473   1473
=====================================
  Hits         1473   1473

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 01832c4...21e9ec0. Read the comment docs.

Shall I merge?

LGTM

I'll merge.

authors: codecov-io dankohn david-a-wheeler jdossett

repo: coreinfrastructure/best-practices-badge


text:
readme: add deprecation notice

/cc @philips

looks mediocre to me

authors: crawford jonboulle

repo: coreos/coreos-cloudinit


text:
mantle: add documentation for mantle

Could use some links to it :)

Content and everything LGTM

@marineam Links to the repository?

links from other docs to here so it is discoverable

Added a link at the bottom of the main SDK doc, modifying-coreos.

LGTM

authors: marineam pbx0 robszumski

repo: coreos/docs


text:
doc: update release

lgtm

repo: coreos/etcd-operator


text:
code quality: use more FUNCTION

done


text:
Stricter glob

This doesn't actually change cmake behaviour because * in cmake glob is recursive anyway.

Hmm, you're right. I should test stuff more. Any idea of how this can be fixed?

Nothing elegant comes to mind. You can add src/foo/*.{cpp,h} and keep src/*.{cpp,h} manual.
Note that I'm using * here as a shell would interpret it, not as cmake would.

Actually, I get different results with GLOB and GLOB_RECURSE. GLOB is not going down subdirectories

You're right, I didn't notice GLOB vs GLOB_RECURSE in the diff.

authors: DaanDeMeyer bstaletic

repo: cquery-project/cquery


text:
Update CLI tutorial to use ISS position use case

:+1:

@autophagy wanna merge?

authors: autophagy mechanomi

repo: crate/crate-tutorials


text:
Added next Docker meets

ack. please squash commits before merge

authors: ChrisChinchilla chaudum

repo: crate/crate-web


text:
Added exception on unsupported uri schema.

please add changelog entry

ack

authors: chaudum pustovit

repo: crate/crate


text:
Add PEERDNS network_interface param/option

#14 submitted.


text:
Pechkin turns to Dashamail

Hello, sorry for late response and thank you for PR. But it seems there is some issue with requiring version.rb, you "DashaMailRb/version", while version.rb seats in "dashamailrb" dir, details available here: https://travis-ci.org/crsde/PechkinRb/builds/559165352#L261, or just bundle console inside project directory.

authors: DashaMail crsde

repo: crsde/PechkinRb


text:
Do not use ThreadLocal $errno because this symbol is GLIBC_PRIVATE.

Please provide background for this claim, what historical glibc versions we're breaking, ...

@ysbaddaden,  when running this script
#!/bin/sh

echo >demo.cr 'puts "errno=#{Errno.value}."'
crystal build demo.cr
./demo
LD_DEBUG=bindings ./demo 2>&1 | grep -i errno

on a system with glibc-2.26, you may get this output:
errno=0.
      6798:     binding file /lib64/libpthread.so.0 [0] to /lib64/libc.so.6 [0]: normal symbol `errno' [GLIBC_PRIVATE]
      6798:     binding file /lib64/libpthread.so.0 [0] to /lib64/libc.so.6 [0]: normal symbol `__h_errno' [GLIBC_PRIVATE]
      6798:     binding file /lib64/librt.so.1 [0] to /lib64/libc.so.6 [0]: normal symbol `errno' [GLIBC_PRIVATE]
      6798:     binding file ./demo [0] to /lib64/libc.so.6 [0]: normal symbol `errno' [GLIBC_PRIVATE]
errno=25.

The important aspect is binding file ./demo [0] to /lib64/libc.so.6 [0]: normal symbol `errno' [GLIBC_PRIVATE]. It shows that the compild output references the errno symbol. However, the symbol errno is considered GLIBC_PRIVATE. As of https://groups.google.com/forum/#!topic/uk.comp.os.linux/PX3rstGtZRQ, this means that the the symbol errno is only for the purpose of communicating with other parts of the GNU C library. This means that referencing the errno symbol from programs or libraries not being part of the GNU C library is an error.
There are standard packaging tools which enforce that binaries to be packaged do not reference symbols they shall not reference. This means that current Crystal-generated binaries which access Errno (including crystal itself) do not pass this check and therefore cannot be packaged.
This pull request fixes that situation.

authors: felixvf ysbaddaden

repo: crystal-lang/crystal


text:
Add NOTICES.txt

@izgeri

The go.mod has 32 included deps, but there's only 5 reflected here - what's the difference?
The difference is that the other dependencies are indirect. I wonder if we should be including indirect licenses too

can you please confirm that none of the indirect-marked deps in go.mod are libraries we manually included, and/or that all indirect-marked deps are tier 2+ dependencies?
based on the go docs it looks like modules labeled indirect are either there because a direct dep is missing a go.mod file itself, so its dependencies are recorded - or because you manually specified a different version of a 2nd tier dep than what a dep requires (if I'm reading it correctly)
I just want to confirm if we're not going to include the indirect deps (which I think is fine) that they are all indeed coming from deps-of-deps, and aren't direct includes / versions that we manually specified / required

@izgeri all the indirect-marked deps are 2nd tier and none where manually specified by us.

authors: doodlesbykumbi izgeri

repo: cyberark/sidecar-injector


text:
this.depth = 0;

Closing because no explanation is provided. (It’s valid JavaScript.)

authors: giorgi-ghviniashvili mbostock

license: isc


text:
p16，1.2.4，RoRep -> PoRep？

已收录：第三版重印修正
谢谢指正


text:
pharo-5.0 moving to spur vm and wget --quiet -O - get.pharo.org/vm | bash will no longer work

tests green

repo: dalehenrich/builderCI


text:
Identity server does not support dnxcore50

IdentityServer4 does


text:
Sort out code coverage

Current coverage is 60.42%

Merging #118 into development will decrease coverage by -3.46% as of bfd77a8

@@            development    #118   diff @@
===========================================
  Files                48      41     -7
  Stmts              3353    2795   -558
  Branches              0       0
  Methods
===========================================
- Hit                2142    1689   -453
  Partial               0       0
+ Missed             1211    1106   -105

Review entire Coverage Diff as of bfd77a8

Powered by Codecov. Updated on successful CI builds.

authors: codecov-io danthorpe

repo: danthorpe/Operations


text:
WORKDIR is necessary to mount local folder

Changes Unknown when pulling d43a88e79a3ed1af0e82b4a11d6fc52b33fc973f on gomex:patch-3 into ** on datasciencebr:master**.

Thanks, @gomex!

authors: coveralls cuducos gomex

repo: datasciencebr/jarbas


text:
Add Scala version to datavec-geo for Jackson version conflict workaround

Build triggered. sha1 is merged.

Build started sha1 is merged.

Refer to this link for build results (access rights to CI server needed):
/job/datavec_PR/61/Build result: FAILURE[...truncated 391.01 KB...]	at hudson.maven.MavenModuleSetBuild.run(MavenModuleSetBuild.java:544)	at hudson.model.ResourceController.execute(ResourceController.java:98)	at hudson.model.Executor.run(Executor.java:405)Caused by: java.io.FileNotFoundException: https://api.github.com/repos/deeplearning4j/DataVec/statuses/342c497050263ad5a9b7a6b6e08403b4b8d43e64	at sun.reflect.GeneratedConstructorAccessor49.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/DataVec/statuses/342c497050263ad5a9b7a6b6e08403b4b8d43e64	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

Build finished. 165 tests run, 4 skipped, 4 failed.

authors: SusanBot saudet


text:
Modify test to illustrate issue #223.

Superseded by #225, thanks a lot for the useful reproduction @sethk !!

authors: deivid-rodriguez sethk

repo: deivid-rodriguez/byebug


text:
navigators组件和list组件不弄一个吗

具体有什么功能，或者有没有类似的案例？

1、就是toolbar导航栏
2、list的话就是包括loadmore事件的高效的列表组件
可以看看有赞vant的list和NavBar

我们会在未来的迭代中讨论是否会添加，也欢迎提一些金融场景相关的业务组件需求。

repo: didi/mand-mobile


text:
DEV: Move assignments from custom fields to a table

@ZogStriP The PR is now in a reviewable state. The only question I have is regarding this feature: https://github.com/discourse/discourse-assign/commit/23aa28e2b4c38995d25bd3b71c95a44469f34f3a That's the only thing that's using topic custom fields.
I kept it as is, since I don't thing adding support for this edge case to the new Assignment model path is worth it. Alternatively I could also just remove it. 😜 Should I keep it?

repo: discourse/discourse-assign


text:
#101 - Added route path and view for 404 message.

Fixes #101

authors: captbaritone denisinla


text:
Configure hot module reloading for devs

Current status:
Separated Client Server:

[x] Hot Module reloading
[x] Typescript

Single Package:

[ ] Hot Module Reloading (50%)
[ ] Typescript

Future work

[ ] Finish the typescript module for single package. (*)
[ ] Discuss whether to use one package or the separated client/server

(*)  Getting the typescript working on the front end shouldn't be too much work (less than a day) but will require some effort and research to ensure that we are doing it properly. The only reason it wasn't before was because of errors that made no sense cause by lack of understanding of typescript.

Thanks for the update... I'm mindful we need to choose the right approach.  This sounds like the approach we'll take for a PoC ... but we may want to revisit if this PoC is adopted.
/cc @hdrury1

authors: diversemix will-byrne

repo: diversemix/malcolm


text:
cookbook-test-libraryをignore

Fixed at #28

repo: dlang-jp/Cookbook

license: Unlicense


text:
Unexpectedly found quarter-final in results_processor

Heritage Classic scoresheet doesn't identify prelim rounds correctly


text:
Updated version to 0.7.1

Thanks! :+1:

authors: tianon vegasbrianc

repo: docker-library/ghost


text:
fix: ==> default: Forcing shutdown of VM...

It seems there is a problem within debian 8 that ssh-server prevents shutdown.
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=751636
http://serverfault.com/questions/706475/ssh-sessions-hang-on-shutdown-reboot

i suggests to use the v1-ubuntu branch, which uses ubuntu 14.04

repo: dominikzogg/vagrant-php


text:
Ignore patterns specified in excludes in docfx.json

@IEvangelist I'm going to close for now in favor of your suggested simpler approach: https://github.com/dotnet/docs-actions/pull/25.

repo: dotnet/docs-actions


text:
Internal users can easily migrate from TLC to ML.NET

Related:
https://github.com/dotnet/machinelearning-modelbuilder/issues/1175

Related:
https://github.com/dotnet/machinelearning-modelbuilder/issues/1175

authors: JakeRadMSFT briacht

repo: dotnet/machinelearning-modelbuilder


text:
Add an basic chunk generator

Solved in #27 .

repo: dotnetGame/MineCase


text:
[Bug] Header file don't check fully, only class will check

This issue was fixed with (https://github.com/dougpuob/cpp-namelint/commit/09a2779baba670a6560fec320740fc3de27beacb) commit.
Add new test sample file Sample07.cpp and Sample07.h. You can run the test with cppnamelint.py bldgtest.

repo: dougpuob/cpp-namelint


text:
[DO NOT MERGE] just for testing

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

ok to test

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

authors: kie-ci psiroky sometestuser1234

repo: droolsjbpm/droolsjbpm-build-bootstrap


text:
Add supervisors to overlord console.

@gianm could you add a screenshot of this in action in the PR please?

It looks like this:

If you don't have any supervisors then the top section doesn't show up.

authors: drcrallen gianm

repo: druid-io/druid


text:
Added parliament support

I had the same errors locally when I tried to push this. I don't know if something broke upstream.  The error is:
File "/home/travis/virtualenv/python3.7.1/lib/python3.7/site-packages/pipenv/vendor/toml/decoder.py", line 187, in loads
    " Reached end of line.", original, i)
toml.decoder.TomlDecodeError: Key name found without value. Reached end of line. (line 18 column 11 char 326)

Only related issue I can find is from 2018 (https://github.com/pypa/pipenv/issues/3092) so I'm guessing it is not an upstream problem if no one else on the internet is hitting it.

repo: duo-labs/cloudmapper


text:
Can't parse column with name: type, note

@liamnv thank you for report the issue, seen this already fixed at latest update

authors: duythinht liamnv

repo: duythinht/dbml-go


text:
Whitelist files for npm and reduce lodash dependency

Nice.


text:
Add Dynamo execution support

Added

repo: eirannejad/Revit-Journal-Maker


text:
Add usernames filter

Out of curiosity, what would be a good use case for this?

@ekmartin for our use case we have multiple teams using the same repos use this and the repos filter to only remind specific team members about their teams prs to specific repos

@markddavidoff: In that case, would it be more useful if you could provide teams and have it filter by that? Team, as in https://github.com/orgs/ORGANIZATION/teams/
TEAMS=team1, team2 -> retrieve team members for team1 and for team2 -> filter PRs by members in those teams

Unfortunately we don't use GitHub teams, so for us that doesn't work. That would be neat to add but i like having usernames separate as well.

@ekmartin Unfortunately we don't use GitHub teams, so for us that doesn't work. That would be neat to add but i like having usernames separate as well.

Note:update doc

@ekmartin added teams support in another PR but i think this is still useful such as for members not in a team that still might make prs that need to show up (architect, infosec, etc)

@ekmartin any chance these features get in?

@markddavidoff: Thanks! Sorry about the late response.

authors: ekmartin markddavidoff

repo: ekmartin/slack-pull-reminder


text:
update some typos

 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.

authors: CLAassistant hutuxian

repo: elasticdeeplearning/edl


text:
Fixed double-quote typo to make the command work when copied/pasted to terminal

Thanks, your PR has been accepted and merged into master branch.  Thanks, it's a surprise to use.

Thanks, your PR has been accepted and merged into master branch.  Thanks, it's a surprise to use.

authors: stiartsly wakqasahmed


text:
Clarifying focusing essence of windowByIndex()

Yep, this is better.

I'm closing this PR due to lack of response over the past year.
@3n-mb, please feel free to open a new PR that addresses the changes that @codebytere suggested. I'd be happy for a fixed PR to be merged.

repo: electron/spectron


text:
Feature/responsive

Closing this. This code provided by @gustavoteodoro was used in the final release version. Thankss!!!

authors: gustavoteodoro luiguild


text:
Create my-slide

Add elsnafyh's file

repo: elsnafyh/github-slideshow


text:
Add some testcases

outdated

repo: emqx/emqx-rule-engine


text:
Python 3 compatibility

Hi
I am interested in adding Python3 compatibility. My boss said I could spend a day working on it, so before I do this, I just want to check that a) you would be interested in accepting a PR and that b) there are no hidden dragons preventing the upgrade.
I am assuming that all I need to do is change to the new exception syntax etc and use some from futures import... as needed?

Hey Lee -- this sounds great! Definitely interested in a PR.
I don't have experience doing 2->3 ports, so can't speak to the work involved, but there shouldn't be any barriers. Explorer relies on 3 packages (Django, Factory Boy, and SQLparse), all of which are Python3 compatible.
Let me know if I can do anything to help. Feel free to ping me (chris@epantry.com) if you have any questions.

repo: epantry/django-sql-explorer


text:
return all munki prefs

Thank you kindly.

authors: clburlison erikng

repo: erikng/munkireport-php


text:
fix: fix providerName type to allow user to extend default providers

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

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

repo: exadel-inc/esl


text:
Remove guard-for-in

Adding label to compensate for work done on main.

repo: exercism/eslint-config-javascript


text:
Expose autoCorrect and autoCapitalize Props

Oh wow! Since #4 was not active I wasn't expecting this to happen. Thanks for your work on this, I'm loving it so far.
Keep up the good work.


text:
[BUG] Analyzed transitive count also includes unknown transitives

https://jira.coreos.com/browse/APPAI-830

authors: arajkumar yzainee


text:
swarm -> thorntail minor refactor

Can one of the admins verify this patch?

ok to test

retest this please

authors: centos-ci pmacko1 tradej


text:
Editxml

[merge]


text:
Add array_min/array_max functions.

I think we can make greatest and least work for arrays.

To my understanding, greatest  and array_max are semantically different function.

reatest treat array as a whole. It will return the maximum array
array_max will return the maximum element in the array.

This is what greatest return with array input.
presto:tiny> select greatest(ARRAY[1,2]);
 _col0
--------
 [1, 2]
(1 row)

This is array_max return
presto:tiny> select array_max(ARRAY[1,2]);
 _col0
--------
2
(1 row)

I can get the correct result using presto-cli from the PrestoServer with this patch.
But the newly added test cases for array_max with the same query failed because of ExpressionCompiler failed to compile the expression. I am not familiar with the ByteCode/compiler part of Presto, could you guys give me some idea about it when you have time.
Here is the Exception of test assertFunction("ARRAY_MAX(ARRAY [1, 2, 3])", BIGINT, 3);
java.lang.RuntimeException: Error compiling "array_max"(ARRAY[1,2,3]): -1
    at com.facebook.presto.operator.scalar.FunctionAssertions.compileFilterProject(FunctionAssertions.java:544)
    at com.facebook.presto.operator.scalar.FunctionAssertions.executeProjectionWithAll(FunctionAssertions.java:260)
    at com.facebook.presto.operator.scalar.FunctionAssertions.selectUniqueValue(FunctionAssertions.java:231)
    at com.facebook.presto.operator.scalar.FunctionAssertions.selectSingleValue(FunctionAssertions.java:226)
    at com.facebook.presto.operator.scalar.FunctionAssertions.assertFunction(FunctionAssertions.java:200)
    at com.facebook.presto.operator.scalar.AbstractTestFunctions.assertFunction(AbstractTestFunctions.java:38)
    at com.facebook.presto.type.TestArrayOperators.testMax(TestArrayOperators.java:301)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:483)
    at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:84)
    at org.testng.internal.Invoker.invokeMethod(Invoker.java:714)
    at org.testng.internal.Invoker.invokeTestMethod(Invoker.java:901)
    at org.testng.internal.Invoker.invokeTestMethods(Invoker.java:1231)
    at org.testng.internal.TestMethodWorker.invokeTestMethods(TestMethodWorker.java:127)
    at org.testng.internal.TestMethodWorker.run(TestMethodWorker.java:111)
    at org.testng.TestRunner.privateRun(TestRunner.java:767)
    at org.testng.TestRunner.run(TestRunner.java:617)
    at org.testng.SuiteRunner.runTest(SuiteRunner.java:334)
    at org.testng.SuiteRunner.runSequentially(SuiteRunner.java:329)
    at org.testng.SuiteRunner.privateRun(SuiteRunner.java:291)
    at org.testng.SuiteRunner.run(SuiteRunner.java:240)
    at org.testng.SuiteRunnerWorker.runSuite(SuiteRunnerWorker.java:52)
    at org.testng.SuiteRunnerWorker.run(SuiteRunnerWorker.java:86)
    at org.testng.TestNG.runSuitesSequentially(TestNG.java:1224)
    at org.testng.TestNG.runSuitesLocally(TestNG.java:1149)
    at org.testng.TestNG.run(TestNG.java:1057)
    at org.testng.remote.RemoteTestNG.run(RemoteTestNG.java:111)
    at org.testng.remote.RemoteTestNG.initAndRun(RemoteTestNG.java:204)
    at org.testng.remote.RemoteTestNG.main(RemoteTestNG.java:175)
    at org.testng.RemoteTestNGStarter.main(RemoteTestNGStarter.java:125)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:483)
    at com.intellij.rt.execution.application.AppMain.main(AppMain.java:140)
Caused by: java.lang.ArrayIndexOutOfBoundsException: -1
    at org.objectweb.asm.Frame.a(Unknown Source)
    at org.objectweb.asm.MethodWriter.visitMaxs(Unknown Source)
    at com.facebook.presto.byteCode.MethodDefinition.visit(MethodDefinition.java:253)
    at com.facebook.presto.byteCode.MethodDefinition.visit(MethodDefinition.java:207)
    at com.facebook.presto.byteCode.ClassDefinition.visit(ClassDefinition.java:158)
    at com.facebook.presto.sql.gen.CompilerUtils.defineClasses(CompilerUtils.java:100)
    at com.facebook.presto.sql.gen.CompilerUtils.defineClass(CompilerUtils.java:74)
    at com.facebook.presto.sql.gen.CompilerUtils.defineClass(CompilerUtils.java:80)
    at com.facebook.presto.sql.gen.ExpressionCompiler.compileProcessor(ExpressionCompiler.java:130)
    at com.facebook.presto.sql.gen.ExpressionCompiler.compileAndInstantiate(ExpressionCompiler.java:93)
    at com.facebook.presto.sql.gen.ExpressionCompiler.access$300(ExpressionCompiler.java:42)
    at com.facebook.presto.sql.gen.ExpressionCompiler$1.load(ExpressionCompiler.java:53)
    at com.facebook.presto.sql.gen.ExpressionCompiler$1.load(ExpressionCompiler.java:48)
    at com.google.common.cache.LocalCache$LoadingValueReference.loadFuture(LocalCache.java:3527)
    at com.google.common.cache.LocalCache$Segment.loadSync(LocalCache.java:2319)
    at com.google.common.cache.LocalCache$Segment.lockedGetOrLoad(LocalCache.java:2282)
    at com.google.common.cache.LocalCache$Segment.get(LocalCache.java:2197)
    at com.google.common.cache.LocalCache.get(LocalCache.java:3937)
    at com.google.common.cache.LocalCache.getOrLoad(LocalCache.java:3941)
    at com.google.common.cache.LocalCache$LocalLoadingCache.get(LocalCache.java:4824)
    at com.google.common.cache.LocalCache$LocalLoadingCache.getUnchecked(LocalCache.java:4830)
    at com.facebook.presto.sql.gen.ExpressionCompiler.compilePageProcessor(ExpressionCompiler.java:87)
    at com.facebook.presto.operator.scalar.FunctionAssertions.compileFilterProject(FunctionAssertions.java:536)
    ... 36 more

I think we should be very careful about making greatest and least work element-wise on arrays. I would find it confusing if greatest(array[1, 2]) - > 2 and greatest(array[3, 4]) - > 4 but greatest(array[1, 2], array[3, 4]) - > array[3, 4]. If we don't want to add array_max, I would be more in favor of looking into scalar subqueries to see if we can have max(unnest(array[1, 2])) - > 2 and evaluates as a scalar

My last comment was in reply to @electrum, btw.

@cberner @electrum

What should happen if the input array is null only?

Fixed the review comments and rebased on current master

Merged. Thanks! (made a minor change, so that nulls in arrays always return null, and empty array always returns null)

Thanks @cberner , I will read the commit to see how do you handle nulls.

So the logic is:

If the array is empty, return null
If there is any null in array, return null
Otherwise, return the min/max element.

authors: cberner electrum haozhun yaoxin226

repo: facebook/presto


text:
First draft of Mongodb plugin

This looks awesome! Very complete docs, too!
Nit: the preferred capitalization, from looking at their website, seems to be "MongoDB"

looks good

I've rebased on 0.123-SNAPSHOT and fixed various typos in the document.

@electrum, apologies if I'm missing something. Is there a merge plan for this?

@gonewandering yes, we're super backlogged processing pull requests, but this is definitely on our list.

@martint, awesome. Thanks. Looking forward

Hi, I am using this as a plugin, with my current version of PrestoDB, and may have found a bug.  When creating a schema for a collection, the connector fails with error whenever the db collection has a field name with a mixture or lower-case and upper-case characters.  If I rename the field in the collection to be all lower-case, the queries then work fine.  This has to be tested with a MongoDB server hosted on a case-sensitive O/S like Linux.  The problem does not happen when I use a MongoDB on OSX, which has a case-insensitive file system.

Can you rebase on latest master?

Merged, thanks!

authors: akshatnair electrum gonewandering martint milespoindexter miniway


text:
Better error message when using refmt without arguments

Closed in favor of #945

repo: facebook/reason


text:
Use detect over select...first

The git tree went crazy...

repo: fastlane/ci


text:
stabilize subtitle upload thing

refactor code


text:
Cell

asdf

repo: fcor/arjs-gestures


text:
Add support for php 7.4

@paulmaxwell @quantum-llama @GaborSuranyi can you take a look at this PR as it's a simple one.

closing due to inactivity


text:
Add tests for the examples

Current coverage is 75.31% (diff: 100%)

Merging #640 into master will increase coverage by 0.36%

Powered by Codecov. Last update f165e43...0cb285a

@vkostyukov updated :+1:

Thanks @BenFradet! Merging this in!

authors: BenFradet codecov-io vkostyukov

repo: finagle/finch


text:
Fix indonesia spec to support 9 digits

Coverage remained the same at 99.398% when pulling c18d70a3e2611e76d4dce8717e15997e93fc23c1 on yamanaltereh:fix_indonesia_spec into 53e6aa2b370623646d05518ef14669d54db15d66 on floere:master.

@yamanaltereh Thank you! 🎉 Released in 2.16.1 🚀

authors: coveralls floere yamanaltereh

repo: floere/phony


text:
Fix style issues introduced by dba29aa0fce684b201c6b03efc73ba607c7b507b

Submitting on red flutter build as the underlying issue of the red-ness has been fixed now.

repo: flutter/flutter


text:
Upgrade to Angular 2.0

👍

authors: albertvolkman travist


text:
add pool for SystemProcess/Views

Not only need pool.
Each time SystemProcess call Views.iterator() will alloc extra a lot memory for for map.keys() and a little memory for anonymous function( hasnext(), next(), holder ) in every loop.
And another issue:
the Views T  is anonymous type ,so in some special platform( e.g. C# ), the generated source codes will slower http://haxe.org/manual/types-structure-performance.html .
Can macro detect these anonymous structs and create new type class and replace them?

I believe the allocation problem is still there but I made a few tests of classes vs anonymous and I cannot find any real difference in terms of performances.

authors: daisuda fponticelli

repo: fponticelli/edge


text:
Updated some links

Thanks!

authors: fredreichbier zort

repo: fredreichbier/genie


text:
User-oriented FAQ for WeChat in uProxy

Want to check on this - last I heard, this module is essentially blocked by server-side changes in wechat. If that is the case, then maybe this issue should be revised to "note that this module is blocked in the main landing README." Thoughts?

as i understand it, it's not 'blocked', it's just a slightly worse user
experience where the signaling messages aren't fully hidden from users.
Arguably the same as github where the gists/comments are visible to users
as well.
On Wed, May 11, 2016 at 2:29 PM, soycode notifications@github.com wrote:

Want to check on this - last I heard, this module is essentially blocked
by server-side changes in wechat. If that is the case, then maybe this
issue should be revised to "note that this module is blocked in the main
landing README." Thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/freedomjs/freedom-social-wechat/issues/8#issuecomment-218595901

Ah yeah, I think I recall the conversation more clearly now, but I also think that uProxy basically considers that a pretty unusable state (since with wechat those messages alert your phone). Do we have any plausible path for fixing that?

Left this in an issue raised in the uproxy issue, but I'll just copy here for completeness.
@soycode, From when I was last looking, Will is right in that it is a usability issue -- the messages can no longer be sent hidden. However, Within the wechat api (last I checked), users don't have to be alerted of messages; i.e. a message can get to a phone, and a user can see the message if they look, but the phone will not vibrate or make any sounds to alert a user that a new/unread message is available in a thread.
The current path is to create a separate thread for uproxy's use with your friend, meaning that all uproxy related information will be isolated into a new "contact" where you won't be alerted of any messages passed through there, while keeping a thread between your original contact uncluttered. Some work has been made in order to accomplish this, but hasn't been taken to completion.

authors: LeMasque soycode willscott

repo: freedomjs/freedom-social-wechat


text:
Testing and basic samples needed

Done


text:
Advise against fastclick use

But there are still a lot of those older browsers in use right? Especially in WebViews on Android for example.

The title of this PR is actually a lot bolder than the content of the PR, which is more about advising users of when not to use fastclick. If I had the time I would do a PR to turn off fastclick for modern browsers

authors: adam-lynch wheresrhys

repo: ftlabs/fastclick


text:
on-boarding redux implementation using wallet_core

This is the implementation of the onboarding process. I would really appreciate if everyone can go over it and suggest improvements/thoughts/etc.

@LiorRabin It looks ok, but as long it's not tied up to UI we can't say how good it fits.

@creatix thanks! I will try and run with your additions and see why the 400 error

@creatix I've fixed the 400 error, you can continue from here

Closing this PR, will open a new branch with #2

authors: LiorRabin leonprou


text:
Release v0.2.3

@merge

!merge

repo: fusionjs/fusion-cli


text:
Upgrade dependencies

!merge

authors: KevinGrandon rtsao

repo: fusionjs/fusion-plugin-react-router


text:
enable velero in tests

For UI and Velero we are using api groups (routes, etc.) that do not exist in kubernetes by default in molecule.

repo: fusor/mig-operator


text:
Publish vs. Register... pick a term and be consistent in CLI and Web

These are the definitions we will use for each term:
Register - add container to Dockstore
Publish - make a registered container public (visible to all users)

The isregistered and ispublic columns of the tool table don't seem to mean what we think they would mean.  ispublic refers to if a repo is public on quay, and is registered is used to see if a container is published on Dockstore (using definition of publish mentioned in my above comment).
I believe the isregistered can be removed completely, and ispublic replaced with ispublished. This will prevent future confusion between how we store tools and the way quay stores repos.
ispublished will just determine if the tool is published.  If a tool is registered on the site then it will be in the DB anyways, so it is a redundant field.

I think we're consistent with this nomenclature now, in user-facing interfaces, if not totally in code.

authors: agduncan94 briandoconnor denis-yuen


text:
Greenkeeper/sequelize 4.20.1

Coverage remained the same at 98.105% when pulling ce1ff6c64874a50652d8c2cac2847fd09ce48fcd on greenkeeper/sequelize-4.20.1 into 32b8055d267111cba38c6eb763a6a81625cbadd4 on master.

authors: coveralls gabrieltanchen

repo: gabrieltanchen/xpns-api


text:
Create README.md

 Create README.md

 Create README.md

authors: esdmorais galberafael

repo: galberafael/javakings


text:
Small grammatical error =)

Thanks for this 👍

authors: garetmckinley rivertam

repo: garetmckinley/inferno-boilerplate


text:
Updating deps

Codecov Report

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

@@           Coverage Diff           @@
##             master     #6   +/-   ##
=======================================
  Coverage       100%   100%
  Complexity        8      8
=======================================
  Files             4      4
  Lines            27     27
=======================================
  Hits             27     27

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 b0736cb...515b3a4. Read the comment docs.

authors: codecov-io gbprod

repo: gbprod/algolia-specification-bundle

license: WTFPL


text:
Merge all internal statistics into 'Volumes', 'Memory' and 'CPU' statistics and drop 'Use%' from volume stats

Need to create a new content pack for the new stats that does all rollups but not for 'type'. Need to add removal of all existing statistics to the migration SQL.

repo: gchq/stroom


text:
Do not get browser session for skipped suite

:+1:

repo: gemini-testing/gemini


text:
Output values of generator network appear to be limited to a very low range, causing a darkened image to be generated

All activation functions are set to relu.

repo: generic-github-user/xkcd-Generator


text:
VGA resources and final mem

Everything allright

repo: germanbravolopez/TFM


text:
fix: Remove duplicate data from sampling context

Can we then remove parent_sampled from within the to_json implementation entirely? I don't think we need to send this to the server at all?

Is it being sent to the server, though? The only place I see us calling to_json() (other than where I call it in start_transaction()) is here: https://github.com/getsentry/sentry-python/blob/cc08a6bed116e09db41c712c20ab63eb0a839e41/sentry_sdk/tracing.py#L537-L541 where it's being called on spans, not including self, meaning it wouldn't get called on a transaction.
EDIT: Yeah, I checked. When I threw a pdb into my transport's _send_envelope method, here's what I got:
(Pdb) pp envelope.items[0].payload.json
{'_meta': {'...'},
 'contexts': {'runtime': {'...'},
              'trace': {'description': None,
                        'op': None,
                        'parent_span_id': None,
                        'span_id': '979d39004e00edcc',
                        'trace_id': '5a4fc60521604bfea8d89162ce497dc4'}},
 'environment': 'dev',
 'event_id': 'fab597c687f74d9d9b47dffc57dc237b',
 'extra': {'secret_word': None,
           'sys.argv': ['/Users/Katie/Documents/Sentry/test-apps/flask/server.py'],
           'things': 'stuff'},
 'platform': 'python',
 'request': {'...'},
 'sdk': {...'},
 'server_name': 'test-server',
 'spans': [],
 'start_timestamp': '2020-11-12T17:50:33.731546Z',
 'tags': {'dogs': 'rule', 'dogtag': 'yes'},
 'timestamp': '2020-11-12T17:50:33.731875Z',
 'transaction': 'dogpark',
 'type': 'transaction',
 'user': {'id': '122112', 'is_silly': True, 'username': 'maiseythedog'}}

So I think we're safe.

Ok right. Do you agree though that we can remove the field from the to_json return value?

Ah, yeah, that would do it, too, at which point the entire PR becomes removing this line: https://github.com/getsentry/sentry-python/blob/c9120691c360336e7ef4767cafa14c7d7a7d8226/sentry_sdk/tracing.py#L561
Works for me.
(Looking at it now, I'm not actually sure why I felt like it needed to be there in the first place... Oh - I know. I was trying too hard to copy the JS implementation, which I now actually wish I could go back and redo more like the way I ended up doing it in Python... Anyway, yeah, let's just take out that line.)

authors: lobsterkatie untitaker

repo: getsentry/sentry-python


text:
Vendor all external deps so app doesn't rely on network

Fixed by 9aef3d8c6f137bf0e1a31c449f05fb12e0078e94 and d4a2b3ae2c882fb15ffcc3ede446f5bfa2b9a756

repo: git-school/visualizing-git


text:
#19 - Add Get contributors method to the repository DAO and service

Looked good!

authors: callicles jacobtardieu

repo: gitlinks/gitrank-web


text:
Fix codev

Codecov Report

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

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   80.66%
=========================================
  Files             ?      357
  Lines             ?    30737
  Branches          ?        0
=========================================
  Hits              ?    24795
  Misses            ?     5942
  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 e60d178...557290b. Read the comment docs.

authors: codecov-commenter glynpu

repo: glynpu/espnet


text:
Add ZMScore cmd

@BenjaminPrieur thanks for the PR. I have left one comment, but otherwise this looks good.

@vmihailenco I did the changes you mentioned

Thanks :+1:

authors: BenjaminPrieur vmihailenco

repo: go-redis/redis


text:
Added link to tutorial for Thread class

Also added to Semaphore and Mutex

Thanks!

authors: Chaosus akien-mga

repo: godotengine/godot


text:
Added commonsense probe

@googlebot I signed it!

@dodgejesse We are now requesting that task authors please include an explicit section in the README.md file called "data source", listing exactly where you got your data from (or explicitly state that you made it up yourself). Many thanks!

@StellaAthena are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

@nayeon7lee are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

The amount of tasks we received this round is tremendous.  With so many tasks to review we have decided to extend the review period to Tuesday, June 29.
Reviewers: if the submitter has made revisions please be sure to notify me with a formal ”accept” or ”does not meet criteria” by Tuesday, June 29  (if you haven't already done so).
this is an automated message

Hello, I just saw the suggested change to update the README.md file with an explicit section about where we got the data. The README.md does say that our "dataset [was] built from ATOMIC (Sap et al., 2019), a commonsense KB", but I'd be happy to add that section to the README.md, as long as it's not too late. I don't see a review (or "accept" or "does not meet criteria") yet?

@dodgejesse This task dis not yet have any reviews (apologies!) so I will do a late review. My main comment is regarding correctness: I don't understand most of the sentences in the task. Here are a few examples of labelled sentences I find confusing:
PersonX inherits from PersonX's mother and as a result, everyone else die
PersonX sees PersonX's sister and as a result, everyone else greets
PersonX holds in PersonY's hand and as a result, everyone else is hurt
PersonX holds in PersonY's hand and as a result, everyone else cries
PersonX takes PersonY's places and as a result, everyone else smile

Regarding the data source section: Yes, please add this section to the readme. This will make it easy for us to find this information when we scan through tasks.

@dodgejesse Hi Jesse, I am meta-reviewing this task. I wanted to follow up and see if you saw Guy's question and if you are still open to revising the task for the benchmark?
Best,
Ethan

Hi, thanks for the feedback! I've added a section to the README.md file which explicitly states where we got the data. (This task is ATOMIC (Sap et al., 2020) that we transformed into a probing task).
About the data quality: ATOMIC is known to be pretty noisy on its own, but the original authors were hesitant to try to filter the annotations too much as it was one of the first large-scale commonsense datasets created (we don't yet have good annotation schemes for commonsense tasks). The examples you point out above are at least somewhat noisy because of the annotations from the Mechanical Turkers for ATOMIC. For example, the first instance you show, "PersonX inherits from PersonX's mother and as a result, everyone else die" is a natural language sentence built from the annotation in ATOMIC {Event: "PersonX inherits from PersonX's mother", Relation: "Others_then", Annotations: {"none", "die", "leave earthly goods behind"}}
You can see this example in ATOMIC here:  https://mosaickg.apps.allenai.org/kg-atomic2020/?l=PersonX inherits from PersonX's mother
Our filtering removed "none" and "leave earthly goods behind", and only kept "die" (as it's a single token, thus appropriate for the cloze-style task). This task can be viewed as a probe for the ATOMIC dataset, which does have some noise; as described in the README, we filtered it in a number of ways (including removing annotations like "none") to make it more grammatical and coherent. We hope the task is still useful at capturing some notion of commonsense.

Hi Jesse,
Digging into the examples a little bit, I empathize with the concern that at present many are ambiguous. For example here is a typical contiguous block:
{"input": "PersonX gives birth to the PersonY and as a result, everyone else is ", "target": ["washed"]},
{"input": "PersonX regards PersonY with eyes and as a result, everyone else ", "target": ["laughs"]},
{"input": "PersonX sees PersonY daughter and as a result, everyone else ", "target": ["waves"]},
{"input": "PersonX amends PersonY act and as a result, everyone else is ", "target": ["scolded"]},

Looking at the ATOMIC graph browser, it looks like though noisy, the full graph structure helps to render examples a bit less ambiguous.
Some examples look like the noise is enhanced by subsetting to a specific relation. i.e.
in the example PersonX gives birth to the PersonY and as a result, everyone else is, the browser results show that the Others then category results are not great, and in particular are not well suited to being rephrased as everyone else is
In other examples, it looks like the restriction to only single word responses enhances the noise. For example in {"input": "PersonX makes PersonY escape and as a result, everyone else ", "target": ["smiles"]}, here, the other multi-word examples (eg Gets away from danger), though not perfect, help strengthen the signal.
If you are up for it, I would suggest the following changes:

Include single and multi word targets, rather than filtering to single word.
Convert the task to multiple choice, rather than generative, where all target's present in the graph give a correct score of 1, and negative examples with score 0 are introduced. These could be taken, for instance, from the targets from other relations.

Additionally, if there was some way to do grammatical sanitization of the results, that would be awesome! For instance there are many examples like:  {"input": "PersonX plays PersonY's heart and as a result, everyone has ", "target": ["marriage"]} -- though I understand this might be a headache and was not done in the initial work.
Thanks again for the task submission!
Best,
Ethan

Hi Jesse,
We are trying to finalize all benchmark tasks. Please address the above comments by Monday Aug 16 to be able to be included. Thanks!

authors: chiafullo dodgejesse ethansdyer guygurari

repo: google/BIG-bench


text:
Google Calender reminder setting not set for attendees

Please give a lot more detail than this, otherwise we'll be unable to help you.
Please:

Show the code you're using
Describe exactly what you've observed vs what you expected to observe

var remind = new Event.RemindersData()
{
UseDefault = false,
Overrides = new EventReminder[] {
new EventReminder() { Method = "popup", Minutes = 2880 },
}
};
i have created a event with reminder before 2 days.
but on the attendees calendar the reminder time shows before 30 minutes

To be honest, that's what I'd expect anyway - a reminder is a personal thing, I wouldn't expect you to be able to change the reminder for any calendar other than your own.
In the resource description it specifically says "Information about the event's reminders for the authenticated user."
To put it another way - have you tried doing this in the Calendar web UI? Are you able to set other people's reminders that way?

No i can't do it from Calendar web UI

Okay, it sounds like this is working as intended then. I'll close this issue now. Please note that this repo is really for the .NET client library specifically; general API questions would be better posted on Stack Overflow.

repo: google/google-api-dotnet-client-samples


text:
Fixed Opening brace issue to fix php 5.4 build issue

This class gets autogenerated. You could add this method to Google\Model, which is the subclass where the property comes from.
Can you explain why you need to access the model data here?

authors: bshaffer thilanga

repo: google/google-api-php-client


text:
Subclasses should provide connection settings for ServiceApiSettings

PTAL

LGTM

authors: garrettjonesgoogle shinfan

repo: googleapis/gax-java


text:
Reimplement ircfmt.Strip() to fix issues with $c

Noice, thanks for the PR! Good fix

authors: A-UNDERSCORE-D DanielOaks

repo: goshuirc/irc-go

license: ISC


text:
Rebaseline GradleInceptionPerformanceTest

Feedback build: https://builds.gradle.org/viewQueued.html?itemId=10730038

repo: gradle/gradle


text:
Enable the NodeJS environment if possible

Access to the NodeJS environment has been deemed impossible via standard means. Experimental means will not be pursued.

repo: grakkit/grakkit


text:
travis ci env

travis ci replaced by github action

travis ci replaced by github action


text:
Release/v2.9.0

Superseded by #98.

authors: jmbtutor jrmykolyn

repo: groupby/storefront


text:
Enroll bm_chttp2_transport & bm_pollset into bigquery pipeline

Merging early: no tests here

repo: grpc/grpc


text:
Fix for cmake test failures

asan c failure due to know issue #10169

5aa4158 does not look like your commit, can you get rid of it?

This fix is for a failure in portability test linux aggregate_tests.run_tests_c++_linux_dbg_native_default_cmake
No issue was created for this.
The asan failure in this PR is due to another known issue #10169

Opened another PR since this had incorrect commit messages.

@MakMukhi and what PR is that?

@MakMukhi, nit:  you could have done git rebase -i upstream/master and force push to this branch to reuse this PR.
What is the number of the new PR?

Guessing https://github.com/grpc/grpc/pull/10869
If it's not solved can we figure out how to rollback?

authors: MakMukhi ctiller dgquintas jtattermusch yang-g


text:
xib 中怎样加载啊？ 作者

和在xib里面 用一般的 view 一样的，具体遇到什么问题了呢？

加载不出来，我是用xib 做的autolayout 但是你给的初始化方法不管用。

demo 直接发我邮箱吧，gsdios@126.com

xib初始化方式已经优化，请下载最新代码测试下

authors: gsdios herwangsong

repo: gsdios/SDCycleScrollView


text:
Update all versions

👍 Thanks

IMHO If you remove version from git dependencies it can even build without error (cairo-rs-sys)

I'll start merging cairo's PR then restart tests in here and we'll go up like this. ;)

Your way will be same errors for other git dependencies.

I merged cairo (I'll restart CI for it later).

But as error be in my way too as it build against gtk with versioned dependencies

Oh damn, we need to merge everything to test! Well, then let's merge...

Hm, I don't see gdk PR but see update branch in your repo

:-.

I confirm that examples now builds locally

I restarted all builds, they seem to succeed as well. :)

authors: EPashkin GuillaumeGomez

repo: gtk-rs/sys


text:
update description of SafeCSS & SafeHTML pom

Close PR due to the fact, that the description is already updated

repo: gwtproject/gwt-safehtml


text:
Add Heroku generated app.json

I would rename the app

authors: anniemeng sandlerben


text:
Install Steward plugin to record meeting logs

:+1:

Closing since we haven't been having formal meetings on IRC.

authors: paked zachlatta

repo: hackclub/meta


text: Add notes from 01.19.16's planning session


text:
Geocoding Addresses

I've got this coded - how do you want to deploy it? Do you want to geocode a batch of addresses up front and serve the results or do you want to geocode live in the front end / API?

Hi! Thanks for commenting. I was testing issue notes with Lynn, and so I'm not the only one looking for a solution to this. It would be good to have a discussion at our Thursday meeting about how to proceed.

I can't come to the meeting but you can hit me up in Slack most of the day.

Thanks!

authors: mckinneybock znmeb


text:
feature allow&disallow comments

Thanks @yantxs :+1:

authors: haxzie yantxs

repo: haxzie/instagram-live-streamer


text:
Update GettingStartedGuide.md

Either we need to update it from time to time, or we just put above a sentence saying something like You need to run an installation command similar to the one shown below, although the exact version of three.js might vary

Could we tell them to run:
$ npm view @here/harp.gl dependencies

authors: CraigBerlin ninok


text:
Hotfix/minor changes

С временем создания всё хорошо.
Когда пытаешься удалить юзера, который уже есть у кого-то в контактах, сервер падает. Ошибка: DETAIL:  Key (id)=(38) is still referenced from table "contacts". Исправь, пожалуйста.
Я смогла добавить пользователям несколько одинаковых контактов. Однако, мы не должны разрешать пользователям так делать. Это тоже нужно исправить.

И мы договорились переносить таски, которые касаются кода, в done тогда, когда они уже в девелопе.

@dziubakseniia

Я не знаю, как это пофиксить. Я готова переасайнить кого-то на этот таск, так как не имею понятия, как должен работать on_delete в таком случае. Я предлагала сделать это с помощью питона, а не базы данных, это бы помогло избежать таких проблем. Сейчас проблема связана с тем, что оно пытается удалить всё, что связано с определенной записью (это и значит on_delete = CASCADE), а это не правильно. Ему надо удалить только запись в контактах, не трогая пользователей.

Вот доказательство того, что оно работает отлично и как надо: https://streamable.com/59f8o . Я показала, что у определенного пользователя есть contact_id = 4 и попыталась создать новый контакт с contact_id = 4. В этом случае, я получила ошибку - 400. Если я могу добавить пользователя, то я получаю статус код - 201.

Без проблем, я перенесу таски в IN PROGRESS


text:
Gets rid of all styling within bootstrap-theme.less

Closing since this is incomplete

repo: hevnly/asset-bazaar


text:
Feat: withProps

Pull Request Test Coverage Report for Build 1076140978

10 of 10 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 94.988%

Totals

Change from base Build 1069291358:
0.2%

Covered Lines:
304

Relevant Lines:
308

💛  - Coveralls

repo: hitechline/reactools


text:
[MTDSA-189] [DM] Change confidence level to 50 as default

👍

authors: dabd danmassie

repo: hmrc/self-assessment-api


text:
Update docs for deployment

Done. PR #63

repo: hotosm/OpenMapKitServer


text:
待选列表、已选列表，全选移动有问题

请提供可复现步骤或demo @Cosen95


text:
Added Aegis Nova Badge

Thanks! Published v0.19.1

Forget to add Devra 2016 and Chapeau 2016 badges.
Shame on me ... Added now and pull request sent.

authors: Witaminer therealklanni

repo: hubot-scripts/hubot-ingress


text:
docs: small typo fixes

Robert to the rescue!

haha
On Tue, 18 May 2021 at 18:08, Mathias Buus @.***> wrote:

Robert to the rescue!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

authors: mafintosh robertkowalski

repo: hypercore-protocol/hyperbee


text:
.torrent 文件名不支持空格

文件名是否加了引号？如：lx download --torrent "torrent file name.torrent"

加上引号果然好了。。。

authors: iambus romoo

repo: iambus/xunlei-lixian


text:
Remove entry /etc/hosts when container stops

Fixed bu #12

authors: Jean-Beru iamluc

repo: iamluc/docker-hostmanager


text:
Create a use-case diagram

Created a use-case diagram in the wiki

Created a use-case diagram in the wiki

repo: ibrahimnamdar/CovidAnalyzer


text:
Update dependencies

are we going to convert this to just be a build time extension?

are we going to convert this to just be a build time extension?

@aruniverse I was thinking about that when updating this yesterday. I think it makes sense to do so. I'll hopefully get to it in the next few days.

authors: aruniverse calebmshafer

repo: imodeljs/extension-sample


text:
tokio-core is deprecated.

Any progress on this?

Just a short note (as I otherwise don't have the time to work on this library): moving away from tokio_core to tokio would have to account for the use of the former in tokio_proto, also deprecated but unfortunately the foundation of ldap3. I don't see a straightforward way to excise tokio_proto without a massive rewrite of this crate's internals.

Yes, I was realizing that last night when I was trying to do the migration myself.

Hi @inejge , I had a go at trying to port to tokio today, and have made some progress. It looks like the blocker is tokio_proto more than anything, for the multiplex tcp connections.
I'm due to release a program that will be deployed at clients in the coming days, so I don't have a choice but to "make it work".
I'm going to try something from the tower ecosystem and see if it works. I'll provide feedback in a few days

Right, the tokio_proto multiplexer is the tricky part, and having it available is what made initial development comparatively straightforward. For your immediate problem, I'd recommend starting a dedicated thread for LDAP connection(s), with its own tokio_core event loop, and communicating with the rest of the system via oneshot/mpsc channels. I believe these can work across tokio flavors.

I found https://github.com/red75prime/tokio-proto/commit/cd1af828791fcd60dc518eb713a53d00e9aacf91, which while not ideal, still allowed me to get the crate to build. I just got it to build now actually.

With the port to Tokio 0.2, this can now be closed.

Thanks @inejge

authors: bbigras inejge masonk nevi-me quadrupleslap


text:
[10.1.x] ISPN-7819 update putAll docs

Integrated into 10.1.x, thanks @oraNod !

authors: oraNod wburns

repo: infinispan/infinispan


text:
upgrade console 0.7.Final

Merged, thanks @karesti

authors: gustavonalle karesti


text:
fix deletions being set as the correct type

Thanks! Can't believe the tests didn't catch that. I'll have to add some better ones.

authors: inodb raylim

repo: inodb/sufam


text:
fix rollbacks

Rebased, squashed, and improved commit message from @kderme .

Still testing this. Will keep on rebasing as things are merged to master.

Have had one crash running this branch (following the chain). I was not able to see the actual error because it kept on retrying and failing and the real error was not in the scrollback.

Ok, this thing crashed again:
[2021-04-29 02:07:11.39 UTC] runDBThread: DbInsertException "Tx: 32"
  ( SqlError {sqlState = "23503"
  , sqlExecStatus = FatalError
  , sqlErrorMsg = "insert or update on table \"tx\" violates foreign key constraint \"tx_block_id_fkey\""
  , sqlErrorDetail = "Key (block_id)=(5899482) is not present in table \"block\"."
  , sqlErrorHint = ""
  })

Something is really wrong in this PR. I highly suspect this is due to trying to implement rollbacks at the source end of the TBQueue instead of the destination end.

Indeed there is a race condition. drainQueue is not an good enough check to access the db from the main thread.

I'm thinking about changing or adding a new lock, but I havent found a good solution yet.

authors: erikd kderme

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


text:
Add data parity tests for epoch details

@oneEdoubleD Would you mind refactoring your tests to follow this pattern?

Yep! I agree, this is better. I'll do it the next time I look at GraphQL :)

authors: ArturWieczorek oneEdoubleD

repo: input-output-hk/cardano-graphql


text:
Use OAuth2 for tool proxy registration

Just wanted to make sure you saw my comment from yesterday morning @rivernate, do we want to merge this before all the features are in Canvas? I know you mentioned adding the security profile and we also need to add support for requesting an access token using the reg key.

I have the canvas changes pending right now. We can test this and those changes together.

authors: rivernate westonkd


text:
burstSize should be configurable (can't be built now)

Done by https://github.com/intel-go/nff-go/commit/bae0b7303f9c188348d98d309b6d3c0410c27437

repo: intel-go/nff-go


text:
installation: testing support for PyPy

Are all our dependencies PyPy friendly already?  If not, we'd be introducing N new unnecessary CI jobs in the testing matrix, which is kind of going against the spirit of https://github.com/inveniosoftware/invenio/issues/3647.
Note that that PyPy testing seems to take 2x more time on Travis when compared to CPython: https://travis-ci.org/inveniosoftware/invenio-config/builds/144410515

Note that that PyPy testing seems to take 2x more time on Travis when compared to CPython

The pip cache was not initialized.

Are all our dependencies PyPy friendly already?

I'm trying to find out by starting with base packages.

Can you repush with allow_failures for Travis CI?

authors: jirikuncar tiborsimko

repo: inveniosoftware/invenio-config


text:
feat(ssr): add ionic angular server

Is it currently working?

authors: MTXDevp adamdbradley

repo: ionic-team/ionic


text:
[REF] debts on paymentmethods rendering

@KolushovAlexandr Displays negative values in debt history in POS after a purchase of Credit.

merged in https://github.com/it-projects-llc/pos-addons/pull/632

authors: KolushovAlexandr Railia

repo: it-projects-llc/pos-addons


text:
Even more regular updates (early Oct) towards new Use Case

Note: I reverted be661b5 with 41c49ef instead of removing via force-push (since Ivan added a commit in between). Better to keep history linear (:


text:
Tree structure: show all organizations in the tree where someone has access to

@robvanmieghem does this relate to the view on the user page, or the structure on the organization detail page?

Either this is related to the user page, in which case its fixed, or it is related to the org page, which is more accurately tracked by #474

authors: LeeSmet robvanmieghem

repo: itsyouonline/identityserver


text:
CI for Node

Sets the current player to be whoever it is not

repo: jamesatkin-pexip/github-actions-for-ci


text:
[#162] Don't cancel possible transitions even though it is the future of one of those that is cancelled

Coverage decreased (-0.3%) to 80.217% when pulling 0c46393b32e9d1ba91f5523f12f75b1f2957cc8c on issue-162-dont-cancel-possible-transitions into f693cdb8fe1e0cfa56eea19a2d3bac89d4725504 on master.

Coverage decreased (-0.3%) to 80.217% when pulling 0c46393b32e9d1ba91f5523f12f75b1f2957cc8c on issue-162-dont-cancel-possible-transitions into f693cdb8fe1e0cfa56eea19a2d3bac89d4725504 on master.

authors: coveralls javrasya

repo: javrasya/django-river


text:
CLOUD-388: remove hidden characters

Merged

repo: jboss-openshift/application-templates


text:
Include LICENSE.rst

Thank you. Merged and released in: https://pypi.org/project/dtool-create/0.23.4/

authors: jan-janssen tjelvar-olsson

repo: jic-dtool/dtool-create


text:
translate to korean 2016-06-13

Thanks!

Thanks!

authors: UYEONG azu


text:
Update to Rust 2018 and replace quickcheck! with #[quickcheck]

I've also bumped the Rust version to 1.34.


text:
Shortcut bugfix, documentation, and update.

:shipit:

authors: afshin sccolbert

repo: jupyter/jupyter-js-plugins


text:
Add fs2

done


text:
Add trans-glossary.md

Hi, @tengqm, please also take a look at this PR to help us standardized the terminology translations.

👍

一开始我觉得 @tengqm “吹毛求疵”，但后来发现 他每份 comment 都很认真详尽，而且带着知识的广度。所以
+1

+1 我赞同 @tengqm 前辈的意见

+1

我把格式调整了一下。

+1

authors: Mr-Linus YouthLab chenrui333 tengqm xichengliudui


text:
Hi. I am new to Android. I just want to use mobile microphone to record wav audio and just find your repository. I ran it & it compiled successfully. But the problem is that the path you have used to store wav file is not actually having this file. Means i could not get the file, if i want to use that file for further processing. Note that i want to use this wav file for speech recognition purpose done in another device running on python. Any help would be more than welcome, as i am new bae.

Sorry but please use stackoverflow to ask question.


text:
Allow CLI to manage repositories in different organizations

https://github.com/kalisio/kdk/commit/c46ca6ca8116ec6b8aae4fc925396c29461cbea9 allowed to provide organization as an argument for workspaces completely under the organization.
Now we need to be able to also specify organization for specific modules in workspaces for projects using repositories from different organizations (e.g. KDK + Weacast).

repo: kalisio/kdk


text:
Changed hostage references to fireant

Coverage remained the same at 96.034% when pulling c1530b091934b9c72f6756d110ea5c6bf93d7cff on wahwynn:master into 488e483469a63161c744ce5d167de5470cc113c7 on kayak:master.

authors: coveralls wahwynn

repo: kayak/fireant


text:
Update index.md

nechces to prilepit k jedny z tehle
The component must distinguish between user and application errors.

The component must validate its parameters; an invalid configuration must result in a user error.	- The component must validate its parameters; an invalid configuration must result in a user error.

?

@odinuv nechci :-) a asi taky nedokážu, aby to neznělo roubovaně a nevedlo mě to stejně ve finále k tomu, že to bude samostatnej bod. Máš nějakej tip, jak to k jakýmukoliv z těch bodů přidat? Každej se podle mě týká něčeho jinýho.

zamyslim se

dal jsem to do 2 odrazky a prijde me to takhle prijde uplne dobry

authors: odinuv ondrejhlavacek

repo: keboola/developers-docs


authors: AmazingTurtle caseychoiniere cnsuxiaomi lloti

repo: kelseyhightower/nocode


text:
Not working on Internet Explorer

I believe you can set the following in your configuration as a workaround.

I tried that but I am getting this error
`
`

It works on my local machine

authors: gunthercox torakiki wirecamp


text:
Swift version please! Thanks!

@haducloc Alright. But have you tried bridge the STPopup.h into your swift project?

@haducloc if you are using Cocoapods to manage your dependencies, it is very easy to integrate an obj-c project to your swift project.

I'm using this via cocoapods in my swift project and it works great

I would love an example using swift. My Obj-C is sub par...

authors: barrettj chenzeyu haducloc kevin0571 m-t-space


text:
track the most recent selected conversation, and send sync events for it

Running locally, CI failure is the stupid deadlock case...

repo: keybase/client


text:
fix search bar margin/padding on mobile

[ ] 🍒

authors: chrisnojima joshblum


text:
Test only master branch

Coverage remained the same when pulling 43c75a4ceee7e3859fcda5ec7987a3cdc6524c7c on only_master into 8f9ae43862efc446b1eca53b83a87f863e557ec9 on master.

authors: coveralls kishikawakatsumi

repo: kishikawakatsumi/UICKeyChainStore


text:
add serviceaccount and rolebinding as genroncilerables

/lgtm
/approve

/retest

/lgtm
/approve

/retest

/retest

authors: evankanderson matzew n3wscott vaikas

repo: knative/pkg


text:
jonsnow

i entered into github

htrdxh

tdhdh

repo: kowshiknandi/github-slideshow


text:
fix IMAGEBUILDER_URL

Crap. Too early. Just noticed filename is hardcoded.

repo: krebscode/minikrebs


text:
Remove superfluous backtick

/assign jimangel

/approve

authors: ryanmcginnis stuartpb

repo: kubernetes/website


text:
Add tensorflow/rust

Thank you


text:
Fix/integration test

:tada: This PR is included in version @kuka-js/serverless-v1.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: kuka-js/kuka


text:
23.10.1

Pull Request Test Coverage Report for Build 1874

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 90.771%

Totals

Change from base Build 1873:
0.0%

Covered Lines:
973

Relevant Lines:
1022

💛  - Coveralls

authors: coveralls huafu

repo: kulshekhar/ts-jest


text:
Fix screenshot URL references

I found a better way of doing this across all of the presets, on the main repo. Will open that shortly.

repo: laravel-frontend-presets/tailwindcss


text:
refactoring regarding call of fm RS_CORR_INSERT?

not sure, closing

repo: larshp/abapGit


text:
fixes typo in flag for creating conda environment

Thanks @jennschiffer!

authors: iros jennschiffer

repo: learntextvis/textkit


text:
Fix typos

Coverage remained the same at 90.0% when pulling 6b151a536af3389a772a7f3577a9d068ccff9926 on fisuda:patch into 34821ee61413ba191ec05005411af68bff2686a0 on lets-fiware:master.

authors: coveralls fisuda

repo: lets-fiware/ngsi-entity-sort-operator


text:
CLOiSim-2.2.0: squash merged from 'develop' into 'main'

it's hard to merge


text:
编辑后的图片失真问题,怎么解决.🙏

#58 麻烦你反馈一下失真的效果，机型、系统；如果可以的话，请提供编辑前的图片来测试。


text:
嵌套在fragment中的Listview的时候 视频进度有的正常显示，有的只会跑几秒就不动，有的完全不会动，但是拖动或者暂停开始进度可以正常显示

感谢你的反馈，我好像也隐隐约约有这个体会，极少的情况会出现，但是不知道他是怎么来的，没法复现，也不知道他是怎么来的

我这里的情况是图片和视频混合显示在同一个iteam中，只单独显示图片或者视频，写法是仿照demo中的那个listdemo，视频来源有用户自己传的，也有来自美拍之类的，测试情况是，大部分自己拍的进度条正常，也有一些进度条不会动，或者走几秒就停止了，有时又突然可以，来源自美拍之类的视频这个概率更大一些，不过没找到什么原因，我这里这个情况很普遍的存在，如果有查询到什么原因，麻烦告知一声，谢谢。

这个难道和和视频源有关系吗

个人感觉是seekbar那块写法有问题，端午改改试试

@UC10D 具体有啥想法吗

我这边发现的问题是，进度条拖动的效果不好，时间现实也有问题，我先改下看看。

seekbar 快进的操作  seekto  放 onStopTrackingTouch 里面

fixed 04f5e1ffe8ec521ce789aa9fefdf21878a612517

最近忙了其它的，上来才看到已经修复的样子，感谢，辛苦了

^_^

authors: UC10D linmomo lipangit

repo: lipangit/JieCaoVideoPlayer


text:
Fixed typo in spelling of 'Bootstrap'

Nice. Thanks.

repo: llaske/sugarizer


text:
Experimental

added folder

repo: lpcproject/LightPayCoin


text:
Minor typo correction

Thanks @seckwei

@seckwei, your fix was actually on the auto-generated file. The source files for the site are under the /app folder, from which all the files under the /site directory are autogenerated.
I've applied your fix to the source file instead, with 8b0e4c8d1b723c612f9be4324f16fa0834a14f2e - thanks.

Sorry, I should have noticed that! Thanks for letting me know 😄

authors: lucono seckwei

repo: lucono/xtypejs


text:
Update available markerColors list

Duplicate #58


text:
Fix link to image uploader gist

:+1:  Been meaning to fix this for ages. Thanks!

authors: contolini ninjabiscuit

repo: madebymany/sir-trevor-js


text:
fixed backend id in quickstart/api section

Thanks!

authors: glkz klizhentas

repo: mailgun/vulcand-docs


text:
Dev

Codecov Report

Merging #313 into master will increase coverage by 0.00%.
The diff coverage is 81.81%.

@@            Coverage Diff            @@
##             master     #313   +/-   ##
=========================================
  Coverage     58.44%   58.44%
- Complexity     2783     2788    +5
=========================================
  Files           336      336
  Lines          7068     7078   +10
=========================================
+ Hits           4131     4137    +6
- Misses         2937     2941    +4

Impacted Files
Coverage Δ
Complexity Δ

src/Entity/Issue.php
81.48% <ø> (ø)
64.00 <0.00> (ø)

src/Entity/Craftsman.php
92.15% <60.00%> (-3.50%)
30.00 <3.00> (+3.00)
:arrow_down:

src/Api/Entity/Edit/Craftsman.php
100.00% <100.00%> (ø)
10.00 <2.00> (+2.00)

src/Api/Transformer/Edit/CraftsmanTransformer.php
100.00% <100.00%> (ø)
2.00 <0.00> (ø)

src/Controller/Api/EditController.php
76.34% <100.00%> (-0.72%)
85.00 <0.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 df9d438...4341b09. Read the comment docs.

authors: codecov-io famoser

repo: mangelio/web


text:
Do not run UWSGi as root

https://stackoverflow.com/questions/15140805/can-only-run-uwsgi-with-root

Update filesystem access permissions for newly uploaded and converted files


text:
Can we create hyperlinks that will help us to navigate within generated pdf?

Wondering if that's supported yet? Thanks.

repo: marcbachmann/node-html-pdf


text:
it would be great to have bower package

Good point
I will create bower package soon.

Hi Nedudi
The bower package is now created.
Install by: bower install marionette-show-animated
Thanks
Marcin


text:
Move to Hugo

Fixes #132


text:
Update documentation and usage of the tokenizer API

What a mess. Sorry about that. Merged as 0c4d5ad and 0236aee

authors: TimothyGu marijnh

repo: marijnh/acorn


text:
changed Travis env to bionic

bionic does not support java 8

repo: martin-kuba/metacentrum-accounting


text:
Update CHANGELOG.md

@esethna @deanwhillier @Willyfrog Can you help merge this as a priority?
The desktop changelog is out of date because this PR is still open: https://github.com/mattermost/desktop/pull/1081.
The desktop changelog in the documentation site is up-to-date and is easier to maintain.

@deanwhillier or @Willyfrog please help merge

does this need cherry picking on 4.3?

@Willyfrog Honestly I don't know if it needs to be cherry-picked or not.

I guess not, but I'd love confirmation. Anyways it should be easy if the need arises.

authors: Willyfrog amyblais esethna

repo: mattermost/desktop


text:
Get Class Edited in Each Commit

@mdfeist Could you describe this issue a bit more? Right now I don't exactly understand what you mean. In fact, if it's what I think, this is already done (in JavaScript, you can iterate by commit, and get the set of types changed in the commit).

Example:
class MyClass {

private MyType t;

}
Currently:
#DECLARE | INSERT | MyType | 1
What I want:
#DECLARE | INSERT | MyType | 1
#EDIT | INSERT | MyClass | 1
Therefore we can see what types are being changed. (Their functionality is changing.) So currently we are only looking at external changes (declarations and invocations). But I want to add internal changes as well.

Ah, yes! This is very important!

authors: eddieantonio mdfeist

repo: mdfeist/TypeV


text:
Changed to v0.1.7

Pull Request Test Coverage Report for Build 12

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-21.4%) to 78.571%

Files with Coverage Reduction
New Missed Lines
%

lib/Voxa-Dashbot.js
6
78.57%

Totals

Change from base Build 10:
-21.4%

Covered Lines:
18

Relevant Lines:
24

💛  - Coveralls

authors: coveralls omenocal

repo: mediarain/voxa-dashbot


text:
Code is never coming???

Could you publish your code please?

apologies for the delay. The paper was published right before my graduation, and I failed to publish the code before leaving the University. After that, I didn't manage to buy a PC and clean the code to publish. Sorry for the procrastination. I'll buy the PC and publish the code this summer.

repo: melonwan/sphereHand


text:
Publishing the config file fix

@the94air Thank you so much, I didn't know that 👍

No problem :+1:

repo: meness/verifi


text:
[KAFKA][KAF-297] Disable JMX and STATSD options

Do not merge this yet.

LGTM

@mbdcos:  Please resolve conflicts and merge.

retest this please

retest this please

authors: gabrielhartmann keithchambers mbdcos

repo: mesosphere/dcos-commons


text:
Multiple ion images

@LachlanStuart this is "ready" except for one test, don't think the filter is changing when I want it to. Will look into it more tomorrow.


text:
Fix LICENSE copyright example

@mergify backport foxy main

repo: micro-ROS/micro-ROS-demos


text:
Added blog Resource

Thank you.

authors: PauloLuan lpinca

repo: microapps/Nodify-Shopify


text:
Publish artifacts syntax

@vtbassmatt - Can you take a look at this one?

authors: ericsciple jtpetty

repo: microsoft/azure-pipelines-agent


text:
Updating ARG tasks with typed-rest-client 1.7.2

All the APIs we use from older typed-rest-client is intact with the newer version.

I think you will need to bump up version of all tasks which depend on azure-arm-rest-v2

authors: bishal-pdMSFT issacnitin

repo: microsoft/azure-pipelines-tasks


text:
restructure to fit the pattern of other talks

Missing session abstract. Do not merge yet.


text:
Fixed javadoc issue

Actually I just realized that this issue will be fixed in your PR 1373, so either remove this PR or remove it from the other PR, otherwise there will be a conflict.

authors: lilgreenbird peterbae

repo: microsoft/mssql-jdbc


text:
[doctest] Update to 2.3.3

Thanks for the update!

repo: microsoft/vcpkg


text:
Update main version

Force merging this since this is just version change.

repo: microsoft/vscode-python


text:
Update Application Gateway doc to use auto-generated self-assigned certificate

@edburns Hi Ed, can you please review this PR? I removed the certificate and update to documentation to let users use auto-generated self-assigned cert.

repo: microsoft/weblogic-on-azure


text:
Rewrite stylesheets as SASS and add asset compilation with GULP

Resolved https://github.com/minecraftforum/widget/commit/a90ef6b808e29ecdc8c913b169762c61495458d2

repo: minecraftforum/widget


text:
Fix a.. typo?

Typoz?! Zere are souzands of typoz in this readme!


text:
Use lerna and cssnext

Travis failed:
$ rake
rake aborted!
No Rakefile found (looking for: rakefile, Rakefile, rakefile.rb, Rakefile.rb)
@xcatliu

@cht8687 Please ignore this. The new version doesn't require ruby or sass anymore.

@cht8687 Fixed travis issue.


text:
Re-enable disabled tests

Thanks for the PR!
But the tests fail when --strictNullChecks is passed as a compiler option. It's not enabled by default in the project's tsconfig.json, because officially we only support an older version of typescript, but the tests were commented out for this reason.
We're also rewriting the typescript declarations for 3.0 from scratch in #4121, so...

authors: chris-erickson ovangle

repo: moment/moment


text:
louder error on missing content-length headers

Changes Unknown when pulling 742624ecbd9b2587417623fb5bc66a269bb6f018 on peterbe:louder-error-on-missing-content-length-headers into ** on mozilla:master**.

authors: coveralls peterbe

repo: mozilla/airmozilla


text:
Upgrade Django to version 1.6.11

@akatsoulas or @comzeradd r?

Coverage remained the same at 93.21% when pulling 4ab24c62647fa26eb2e44c1eacf9e0e6068dcd71 on johngian:playdoh-upgrade into c69ef80e2e50d296e0c649fb9c1f86fdf17dedd6 on mozilla:master.

Coverage remained the same at 93.21% when pulling 4ab24c62647fa26eb2e44c1eacf9e0e6068dcd71 on johngian:playdoh-upgrade into c69ef80e2e50d296e0c649fb9c1f86fdf17dedd6 on mozilla:master.

Tested locally and works fine r+ :coffee:

authors: akatsoulas coveralls johngian

repo: mozilla/mozillians


text:
feat: package update

Coverage increased (+0.08%) to 82.34% when pulling 8b0634307c3e56444f21cde453df5bc28a750aaf on ladisalves:feat/update into f5320847ba0324759ac43876224efb83c720be52 on mpneuried:master.

Coverage increased (+0.08%) to 82.34% when pulling 8b0634307c3e56444f21cde453df5bc28a750aaf on ladisalves:feat/update into f5320847ba0324759ac43876224efb83c720be52 on mpneuried:master.

I make a forked package on https://www.npmjs.com/package/rsmq-worker-ls with updated dependencies. I don't want to fight with outdated CI in this repo...

I make a forked package on https://www.npmjs.com/package/rsmq-worker-ls with updated dependencies. I don't want to fight with outdated CI in this repo...

authors: coveralls ladisalves

repo: mpneuried/rsmq-worker


text:
Added support for argparse, so one could cronjob the script

Probably want to squash and merge, as there were a couple issues with the filename_prefix not working on pytube.


text:
Update metadata

OPS Build status updates of commit d6b09af:
:clock10: Incremental building: average incremental building time is 23 sec(s)

OPS Build status updates of commit d6b09af:
:warning: Validation status: warnings

File
Status
Preview URL
Details

building-blocks/extending-templates/toc.json
:warning:Warning

Details

toc.json
:warning:Warning

Details

bread/toc.json
:warning:Warning

Details

docs/reference-architectures/ai/batch-scoring-databricks.md
:white_check_mark:Succeeded
View

building-blocks/extending-templates/toc.json

[Warning] Error happen when converting building-blocks/extending-templates/toc.json to Pdf. Details: Could not find a part of the path 'T:\bgvm\building-blocks\extending-templates\toc.json'.

toc.json

[Warning] Error happen when converting toc.json to Pdf. Details: Could not find file 'T:\bgvm\toc.json'.

bread/toc.json

[Warning] Error happen when converting bread/toc.json to Pdf. Details: Could not find a part of the path 'T:\bgvm\bread\toc.json'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

authors: MikeWasson VSC-Service-Account

repo: mspnp/architecture-center


text:
Added my name to README.md

I can't resolve the conflict without creating a new pull request, i hope this doesn't matter i am just leaving it as it is.


text:
Update dependencies, code polishing

Coverage remained the same at 100.0% when pulling e29ce8f36975f26777a97e5c5fb0ffab218606d4 on jeffgbutler:master into 6b16765622cf2f21fd4a7dcdcee1da27ebf02d84 on mybatis:master.

authors: coveralls jeffgbutler

repo: mybatis/mybatis-dynamic-sql


text:
Ensure headers["Content-Type"] exists before comparing string.

Might be, with a broken server or with a mockup data...

We were getting "TypeError: undefined is not an object (evaluating 'settings.headers["Content-Type"].match')" when POSTing to a certain site (using the master branch of Casper and phantomjs-2.0.1).
I know phantomjs2 isn't officially supported yet but FYI this patch fixed the issue.

Thanks ;)

authors: corford dtarda mickaelandrieu paazmaya


text:
Add rest-api address to secret

Turns out the REST API does not support detailed ACLs, so we will disable it again. Further research needed.

repo: nais/kafkarator


text:
[#noissue] Fix MethodDescriptor of GrpcStreamListener

Codecov Report

Merging #5913 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #5913      +/-   ##
==========================================
+ Coverage   40.91%   40.91%   +<.01%
==========================================
  Files        2951     2951
  Lines       88802    88802
  Branches    11693    11693
==========================================
+ Hits        36329    36330       +1
  Misses      49313    49313
+ Partials     3160     3159       -1

Impacted Files
Coverage Δ

...vercorp/pinpoint/plugin/grpc/GrpcClientPlugin.java
0% <ø> (ø)
:arrow_up:

...ptor/GrpcStreamListenerResultMethodDescriptor.java
0% <0%> (ø)
:arrow_up:

...nitor/DefaultDataSourceMonitorRegistryService.java
77.77% <0%> (-11.12%)
:arrow_down:

...rc/main/java/com/navercorp/pinpoint/test/Item.java
66.66% <0%> (-3.34%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
82.92% <0%> (-2.44%)
:arrow_down:

...cluster/zookeeper/ZookeeperClusterDataManager.java
52.9% <0%> (-1.94%)
:arrow_down:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
74.16% <0%> (+1.66%)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
80.98% <0%> (+1.76%)
:arrow_up:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
83.58% <0%> (+2.98%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ClientClosePacket.java
66.66% <0%> (+11.11%)
: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 f9e131b...2d63d31. Read the comment docs.

authors: codecov-io jaehong-kim

repo: naver/pinpoint


text:
feat(ncform-common): add the keepEmpty parameter to support empty array

You can refer to the showOneIfEmpty configuration in all the array type widgets

OK. Sorry for not reading the document carefully.

authors: F-loat daniel-dx

repo: ncform/ncform


text:
Put efficient writer into item package

I think this is done?
Closing.

authors: dbrower msuhovec

repo: ndlib/bendo


text:
Converting class with field that have default values results in broken code

The following class
public class Sample {

	private int initialized = 1;

	private Boolean enabled = Boolean.TRUE;

	private String name;

}

is now converted to:
public class Sample {

	private final int initialized;

	private final Boolean enabled;

	private final String name;

	public Sample(int initialized, Boolean enabled, String name) {
		this.initialized = initialized;
		if (enabled != null)
			this.enabled = enabled;
		else
			this.enabled = Boolean.TRUE;
		this.name = name;
	}

	public int getInitialized() {
		return initialized;
	}

	public Boolean getEnabled() {
		return enabled;
	}

	public String getName() {
		return name;
	}

}

repo: nenko-tabakov/dev-tools


text:
Added hex editor extension

Merged, scheduled for 0.10.2 release.

repo: nettsi/hadouken


text:
CI for Node

Initializes with two players
Starts the game with a random player
Sets the current player to be whoever it is not

Initializes with two players
Starts the game with a random player
Sets the current player to be whoever it is not

repo: nickeyzindgi/github-actions-for-ci


text:
Add area for 3rd party plugins

Let's add the Gulp one too: https://github.com/prateekbh/gulp-optimize-js

Thanks for the PR; merging!

:(  @nolanlawson https://github.com/vigneshshanmugam/optimize-js-plugin

@vigneshshanmugam PRs welcome! :)

repo: nolanlawson/optimize-js


text:
Docs: 统一使用nb init创建项目

该问题会在nb-cli新版本中修复，尚未发版

authors: StarHeartHunt yanyongyu


text:
5.4 Party

@slaughter550 ready for review/merge

repo: notebowl/laravel-framework


text:
[T5A2][W10-B1] Le Ba Hieu Giang

@joeleba Please close the PR after reading the comments.

authors: OuyangDanwen joeleba


text:
Will the backend be open source in the future

will be


text:
Update schedule.md

Thanks! Merging

authors: MathewBiddle emiliom

repo: oceanhackweek/ohw-resources


text:
Why is the color of "std::cout" and "std::endl" different?

Although usually used together, they are fundamentally different beasts.
One is a global object and the other one is a IO manipulator (a function).

authors: Alaya-in-Matrix octol

repo: octol/vim-cpp-enhanced-highlight


text:
Use correct types for CSVFile properties

PR build is failing in Data Platform OMAS API though I appeared to build ok....

CTS(inmem) failed with 16 errors. See https://ibm.box.com/s/7mqfgcbhe18lbbwfjrm9cb4b15d52v5h
The errors are EntityNotKnown exceptions in reference copy tests
Will repeat without this PR to see if it's a PR_specific regression, or a current master issue - in which case we may as well merge anyway

Prior to this PR the CTS (inmem) passes with 0 failures. Retesting with PR again.

Something has gone odd in the github - merge .
When the PR was initially opened it built ok. I was also able to do this locally, and experienced the cts failures.
Later in the day I clicked on 'This branch is out-of-date with the base branch' as we often do -- the result was a failed build
I pulled down the PR manually and merged with master. No conflicts detected but the build result is the same locally as seen on the build system - problems in the data platform OMAS.
It looks as if something could be a bit odd with the git history or prior merges.
@mandy-chessell suggest you pull or rebase from master and test locally, then push/force-push. One of us should then retest the CTS - in case the previous failure was a 1-off or an artifact of merge oddities.

Ok. I think what's happened is that commit 8e2627630ca196141beb403faef301e5bc1c75eb made some 'in-flight' changes ie to DataPlatformProperties.java which were pushed to origin, and thus into this PR -- before the PR had been merged, and this commit contains an incomplete (work in progress change)  and isn't buildable.
Need a stable branch for the PR - an option is to create a new branch and then cherry-pick the selected changes in needed for this fix.
once any upstream branch contains the fix I'll cherry-pick for the release

I think we need to separate the PR from the specific commit needed to go into the 1.7 branch.
On this PR - I thought it was merged yesterday and I have continued with the work on the Data Platform OMAS fix, not realising that my intermittant changes were going into the open PR. I will get it into a buildable state again sometime today so it can be merged.
It should be possible to just take the commit for 1.7 and test it on the branch.
The specific commit for 1.7 on its own should not break the CTS.  The change to OMRS fixed the formatting of an error message.  The change to the type of the properties was in the OMAS layer which is not tested by the CTS.

1.7 PR created just from the csv fix commit.
One option - If you need to 'freeze' for a PR, whilst concurrently continuing development at pace :-) , you can create a branch from your development branch, and use that for the PR. cherry-picks can be added for any fixes found during review and added back to the PR branch from dev branch. This allows for review time without blocking dev work. Just one possibility of so many with git!

I ran a test of CTS (inmem) after merge - all runs cleanly. Unsure why the early test failed - perhaps due to work-in-progress or a glitch.

repo: odpi/egeria


text:
DSP memory and hardware access

General idea achieved, will be improved later.

repo: ogamespec/dolwin


text:
Detached nsp from svc mgmt grp creation

Changes are fine.

authors: dpaks njagadish

repo: oneconvergence/group-based-policy


text:
Update cadence language server wasm

Done

Done

repo: onflow/flow-playground


text:
How to train a custom multi-label dataset?

Hi @Kyuuki93 ,
First, you need to convert your dataset to existing dataset format, e.g. VOC format or implement a new Dataset class inherited from BaseDataset. Pls refer to Tutorial 2: Adding New Dataset for more info.
Then, you can write a config as config for VGG.

Hi @Kyuuki93 ,
First, you need to convert your dataset to existing dataset format, e.g. VOC format or implement a new Dataset class inherited from BaseDataset. Pls refer to Tutorial 2: Adding New Dataset for more info.
Then, you can write a config as config for VGG.

Thanks for ur quick reply, I have prepared a toy FileList dataset , the anno_file looks like
001.jpg 0
002.jpg 1
003.jpg 0 1

And then created a TestDataset which modified from mmcls\datasets\multi_label.py,  use mobilenet as backbone,
then I got
TypeError: ImageClassifier: MultiLabelClsHead: __init__() got an unexpeted keywork argument 'num_classes'
TypeError: ImageClassifier: MultiLabelClsHead: __init__() got an unexpeted keywork argument 'in_channels'

Does multi-label task only support vgg for now?

For your dataset, it seems like imagenet format, which is not supported for multi-label classification task. You might need to convert it to VOC format.
For VGG backbone, we put the final fc layer inside the backbone rather than put it in the head. So MultiLabelClsHead does not expect argument num_classes and in_channels. If you want use other backbones, you might need to use MultiLabelLinearClsHead. Thank you.

repo: open-mmlab/mmclassification


text:
chore: Add active_model_serializers, delayed_job, and mongo to release configs

Thanks @dazuma!

authors: dazuma fbogsany

repo: open-telemetry/opentelemetry-ruby


text:
Putting image instead of chart since won't render in RTD

@parenthetic just FYI your beautiful chart works in markdown, but not ReadTheDocs. So, I put the raw file of the chart as a file in this folder so we can update if needed, and just stuck an image in that section so people could see it.

Bummer. Good catch and fix.
On Fri, Sep 29, 2017 at 3:51 PM, Dana Lewis notifications@github.com
wrote:

@parenthetic https://github.com/parenthetic just FYI your beautiful
chart works in markdown, but not ReadTheDocs. So, I put the raw file of
the chart as a file in this folder so we can update if needed
https://github.com/openaps/docs/pull/1060/commits/72619d15914ab1edb5ffa96c0d221535ee92c4ef,
and just stuck an image in that section so people could see it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openaps/docs/pull/1060#issuecomment-333222655, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUVFUZk2WiT6rodlRMizltbpjjxdzfrBks5snUpIgaJpZM4PpJVA
.

authors: danamlewis parenthetic

repo: openaps/docs