text:
Allow only one sdist to be uploaded per release

This change breaks setuptools (see pypa/setuptools#893).

I'm at the airport, but this change was proposed / accepted as part of PEP 527. If setuptools hardcodes the .zip it could just upload zip files instead of .tar.gz?
Sent from my iPhone

On Dec 18, 2016, at 4:59 PM, Jason R. Coombs notifications@github.com wrote:
This change breaks setuptools (see pypa/setuptools#893).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes, that's what I've done for now. I expect Unix folks will complain that there's no .tar.gz but that's for another day.

authors: dstufft jaraco

repo: pypa/warehouse


text:
Elide Constant Join

Build 2 failed with a segfault:
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007fe76c6758a2, pid=3054, tid=140629833103104
#
# JRE version: Java(TM) SE Runtime Environment (8.0_31-b13) (build 1.8.0_31-b13)
# Java VM: Java HotSpot(TM) 64-Bit Server VM (25.31-b07 mixed mode linux-amd64 compressed oops)
# Problematic frame:
# C  [libc.so.6+0x7e8a2]
#
# Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /home/travis/build/quasar-analytics/quasar/hs_err_pid3054.log
#
# If you would like to submit a bug report, please visit:
#   http://bugreport.java.com/bugreport/crash.jsp
#
./scripts/build: line 79:  3054 Aborted                 (core dumped) "$SBT" -DisIsolatedEnv=${ISOLATED_ENV:=false} test exclusive:test

Build 4 failed in DataServiceSpec - see #1295

Current coverage is 75.41% (diff: 70.58%)

Merging #1344 into master will decrease coverage by 0.07%

@@             master      #1344   diff @@
==========================================
  Files           192        192
  Lines         11459      11465     +6
  Methods       11197      11202     +5
  Messages          0          0
  Branches        119        120     +1
==========================================
- Hits           8650       8646     -4
- Misses         2809       2819    +10
  Partials          0          0

Powered by Codecov. Last update 884e359...503b50d

source: gharchive/pull-request

authors: alissapajer codecov-io

repo: quasar-analytics/quasar

license_type: permissive

license_source: bigquery


text:
Add link to gnucap simulator

Thanks for your contribution!

authors: mulderp rajesh-s

repo: rajesh-s/computer-engineering-resources

license_source: github-api


text:
remove functions deprecated in v0.19.0

i still think these are better, more descriptive names than sequence and traverse. But what are you gonna do. :cow2:

I was happy with the old names, but it's nice to have R.sequence :articulated_lorry: to sequence and R.traverse :articulated_lorry: to traverse.

yep, i agree. we can just point people to the Haskell docs

Yes, I definitely preferred commute, but complying with standards is generally a good thing.

authors: CrossEye buzzdecafe davidchambers

repo: ramda/ramda

license: mit


text:
improve build scripts

LGTM

authors: cjellick deniseschannon

repo: rancher/healthcheck


text:
cloud-config option to set NTP time servers

Confirmed with v0.6.0-rc3 that write_files and restart_services can be used to set NTP time servers in /etc/ntp.conf.

source: gharchive/issue

repo: rancher/os


text:
[2.5] Backport chart version sorting

Backport https://github.com/rancher/rancher/pull/30404

https://github.com/rancher/rancher/issues/29696

repo: rancher/rancher


text:
Added regex workaround so as to allow OCDM shared lib unloading from memory after deactivation.

@neeraj-deshpande as this also resolves the unloading of the SecurityAgent (also uses regex), an we move this to the R2.1.It brings no risk on regression (last famous words :-) )

@neeraj-deshpande as this also resolves the unloading of the SecurityAgent (also uses regex), an we move this to the R2.1.It brings no risk on regression (last famous words :-) )

@pwielders I have created JIRA issue METROL-225.

@pwielders I have created JIRA issue METROL-225.

authors: neeraj-deshpande pwielders

repo: rdkcentral/Thunder


text:
XRE-16605 Add import and create named key APIs for SecApi

 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.yramak601 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@pwielders Could you please review this.

@sebaszm was this funtionality not available?

@sebaszm was this funtionality not available?

@sebaszm and @pwielders  , As per our previous discussion , have made the following changes:
1.Include the ImportNamedKey API logic  as part of the Import API
2.Add the CreateKey API
Could you pls review .Thanks.

@pwielders
If the main crypto interfaces should not be modified , can we introduce these APIs as another vault under the SecAPI backend( Something like VaultRDK).So we could add the API ,  ImportNamedKey()  to be mapped to vault_import inside this VaultRDK without bringing in any modifications in the ICryptography or IVault interface.
The only change would be addition of another cryptography vault id for this , like CRYPTOGRAPHY_VAULT_RDK.
But for the other API , i,e CreateKey ..since this a new API not previously part of the IVault interface , How could we add this without modifying the IVault interface ?

@yashaswini-rk as discussed in our call, I suggest to create a layer, not part of this pull request, that realizes the key name to id mapping (something we do using our Dictionary plugin), creating a dedicated vault will thinks only more complex. Just move it up.
The CreateKey (@sebaszm ) was indeed an omission which I think can be a usefull addition. @sebaszm to confirm and to review that interface.

Closing this as these changes trying to bring up as  IPersistent interface

authors: CLAassistant pwielders yashaswini-rk

repo: rdkcentral/ThunderClientLibraries


text:
Support Node v6 (new LTS)?

Thanks!

authors: ForsakenHarmony timdorr

repo: reactjs/react-router-redux

license: MIT


text:
[golang] Add meta attribute PRESENCE to generated codecs. Issue #483.

Thanks

repo: real-logic/simple-binary-encoding


text:
Clean up all use of forced unwrapped optionals

fixed (for location issues) by https://github.com/realm-demos/realm-teamwork-MR/pull/18

@dhmspector can we continue this in all parts of the app? Would like this code to be as "Swifty" as possible.

sure thing!

repo: realm-demos/realm-teamwork-MR


text:
大佬 请教一个问题  列表播放我删除某一个数据  如何把这个assetURLs也相应的删除某一个  不然播放不对

你重新赋值assetURLs

authors: renzifeng xuhuaping

repo: renzifeng/ZFPlayer


text:
Add tests

LGTM

authors: Page- abresas

repo: resin-io/node-tunnel


text:
Removed intervals <15 minutes for abandoned carts

Codecov Report

Merging #115 (9f7a678) into master (df8f030) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #115      +/-   ##
============================================
+ Coverage     26.97%   27.02%   +0.04%
  Complexity     2143     2143
============================================
  Files            76       76
  Lines          6851     6839      -12
============================================
  Hits           1848     1848
+ Misses         5003     4991      -12

Impacted Files
Coverage Δ

retailcrm/retailcrm.php
26.58% <ø> (+0.43%)
: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 df8f030...9f7a678. Read the comment docs.

authors: codecov-commenter gleemand

repo: retailcrm/prestashop-module


text:
[karaoketv] Add new extractor

Thanks, merged!

authors: lenaten phihag

license: Unlicense


text:
Add support for HTML

Duplicate #330. Please check existing open/closed issues before posting your own.

authors: andyrichardson leeikjeun

repo: rgbkrk/atom-script


text:
Service binding should support binary bodies

Done.

repo: rhiot/rhiot


text:
Update 6to5 and rename everything 'babel'

https://github.com/richgilbank/ES6-Repl-Chrome-Extension/commit/f4ad56662856f71986c04e87a2cf0dc83f308f78


text:
new approach towards threading.

i have tried it by keeping all the actions in a class and creating its instance in the main class. i guess there are some conceptual errors. please review it once. ref commit: https://github.com/ronz94/Steadfast/commit/ee44e26b90ef0b22b67f3ba73f7f64c6a576aff8

have you referred to the basics threading ? I don't see any proper implementation here.


text:
Fix typo

Looks good!

authors: ddbruce justetz

repo: rpiambulance/slack-poll


text:
lsp-types 0.72

bors r+

authors: kjeremy matklad

repo: rust-analyzer/rust-analyzer


text:
Bump all crates to 0.1.10-pre

I've got a better PR coming.


text:
Switch CI to GitHub Actions

bors try

bors r+

repo: rust-num/num


text:
cHTTP has been renamed to isahc

This seems to work and pulls entire body.
The hyper-client seems to mess up the body by doing some decode/encode behind the back? Haven't figured out yet what exactly happens.

How can we fix apps depending on surf version 1.0.1 with chttp if you can't merge this because is a breaking change? Thanks!

This PR has been merged and is there to stay

Is there any plan to release a new version? I've seen the last one is 1.0.1 and it doest not have the chttp read body fix right? Thanks

authors: CarlosLanderas berkus vorot93

repo: rustasync/surf


text:
Remove unnecessary Result return

Thanks!

repo: rustwasm/wasm-bindgen


text:
Added focus and unfocus event handler to TextEntry

Thanks, PRs merged


text:
i3: support for bar tray_output option

I'm sorry, I barely know how to use GitHub. Commit for ssh modules supposed to be a separate pull requests, but as far as I understand it needs a separate branch.

Welcome to Github and thank you for your contribution!
Yes, you need to create another branch to submit a new pull request (generally, we avoid applying patches from the master branch). If the changes were both on i3 it would probably be ok in one PR but the changes are unrelated.

authors: dsx kalbasit

repo: rycee/home-manager


text:
move typescript from dependencies to devDependencies

@itailv Thank you, your right!

I fixed this issue in https://github.com/ryota-murakami/react-typescript-todo-example-2019/commit/cce3ac9b75d447a64e25ee4d9c33e848563fb501

authors: itailv ryota-murakami


text:
Adding this in front of server calls

Thank you, and sorry for the confusion! This changed with the new-style application tests.
I'm in the process of updating Mirage's docs on an AddonDocs site and will have more sections dedicated to docs about testing.

repo: samselikoff/ember-cli-mirage


text:
Update content (internal SharePoint): SDM

@myp192
Did we do this last week?

Not sure if this issue is related to this but all the material was moved to Office 365 and links in the internal SharePoint have been updated.

Forget my previous comment, I understand what it means now. I'll work with @myp192 on this.

authors: gabrielcossette jaysonmc

repo: sara-sabr/ITStrategy


text:
Release/1.6.2

Coverage decreased (-0.02%) to 96.992% when pulling 8fdc3d8624f23106bc34143381861ed4ff4d5cac on release/1.6.2 into f648a462d2babddae0fb6a32bf6aa38f7de6e80f on master.

authors: bgriffith coveralls

repo: sasstools/sass-lint


text:
Added german translation

@schneidmaster
Thank you for adding this so quickly.
Is gitreports.com being actively worked on, or is there a pause at the moment?
Thank you very much for adding this feature to github I find it really necessary!


text:
If using mongoURL, the webpage doesn't show the correct connection url string.

https://github.com/sclorg/nodejs-ex/pull/245

repo: sclorg/nodejs-ex


text:
Add MIT license field to package.json

If you rebase against master (https://github.com/scttnlsn/monq/pull/62) the tests should pass.

authors: saintedlama scttnlsn

repo: scttnlsn/monq


text:
feat(button): add large variant

:tada: This PR is included in version 5.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

authors: Maximilian-R sebopensource

repo: sebgroup/vanilla-pattern-library


text:
remove required email field for identifies

Just deployed this change, let us know if you run into any issues with it!

Excellent

authors: dominicbarnes hjz

repo: segmentio/integration-iterable


text:
The Installation, Predefined steps link under Documentation,Writing a test, Predefined steps gives 404 error

FYI...
Issue is for selenium-cucumber-java-maven-example Project


text:
Investigate threading of WinRT BT adapter

Seems like no synchronization is made and the UI thread is not garantueed.

repo: sharpbrick/powered-up


text:
CHANGES for #1059.

Current coverage is 100% (diff: 100%)

Merging #1073 into master will not change coverage

@@           master   #1073   diff @@
=====================================
  Files          21      21
  Lines        1904    1904
  Methods         0       0
  Messages        0       0
  Branches        0       0
=====================================
  Hits         1904    1904
  Misses          0       0
  Partials        0       0

Powered by Codecov. Last update bb0805d...a208ee5

authors: Lukasa codecov-io


text:
[fix] undefined error color_button

Coverage decreased (-0.02%) to 83.516% when pulling 54d9b3d0f832384201d925feb6a435ae5eccab24 on itowtips:fix-editor into 329788af436fbfce38c9a689cefb842d7bc2f9c8 on shirasagi:master.

authors: coveralls itowtips

repo: shirasagi/shirasagi


text:
[modify] reduce complexities

Coverage decreased (-0.06%) to 79.517% when pulling e6e326ae1dcd56111b8e0e4a71da617de627d92b on sunny4381:features/reduce-complexities into cb90df60a67accc80d00be8437623cf27e05ee7f on shirasagi:master.


text:
Fix the deleteing file path

pls update it, thank u :)


text:
add some preview ( videos, gifs image...)

Ok !!! I am adding some of the screenshots and gifs of Sample apps.

authors: fukemy smrutiranjanrana

repo: smrutiranjanrana/flutter-cookbook


text:
Feature/0.2.0

@jbeemster - this PR is ready for review!

repo: snowplow/snowplow-nodejs-tracker


text:
Feature/dedupe

Missing #2979 @chuwy

The command "cd ../.." exited with 0.
Skipping a deployment with the script provider because the current build is a pull request.
Skipping a deployment with the script provider because the current build is a pull request.

Need to be incredibly careful that we never overwrite a prod Hadoop Enrich or Shred with a new build of a newer version with the wrong version number...

Closing in favor #2981

authors: alexanderdean chuwy

repo: snowplow/snowplow


text:
CI for Node

Waiting on tests
Great! Now that the testing framework is properly configured, we should get a response from it soon. This time, you'll practice reading the logs on your own. Just like before, you can follow along as GitHub Actions runs your job by going to the Actions tab or by clicking on "Details" in the merge box below.
When the tests finish, you'll see a red X ❌ or a green check mark ✔ in the merge box. At that point, you'll have access to logs for the build job and its associated steps.
Step 4: Read an Actions log
By looking at the logs, can you identify which tests failed? To find it, go to one of the failed builds and scrolling through the log. Look for a section that lists all the unit tests. We're looking for the name of the test with an "x".

⌨ Activity: Tell the bot which test is failing so we can fix it

Navigate to the log output
Identify a name of a failing test
Comment the name of the failing test here

I'll respond when you enter the name of at least one failing test. You can either copy and paste that portion of the log directly, or type the name of the test as a comment.

Process completed with exit code 1.

repo: sophiaallui/github-actions-for-ci


text:
feat: support Coq language

none in the repo's examples, you have a link at hands?
On Tue, Jul 31, 2018 at 5:58 PM, Anatoly Stansler notifications@github.com
wrote:

@anatolystansler commented on this pull request.
In src/main/kotlin/app/extractors/Heuristics.kt
https://github.com/sourcerer-io/sourcerer-app/pull/325#discussion_r206697073
:

@@ -1071,8 +1071,12 @@ val HeuristicsMap = mapOf<String, (String) -> ExtractorInterface?>(
"ux" to { _ ->
CommonExtractor(Lang.XML)
},

"v" to { _ ->

   CommonExtractor(Lang.VERILOG)

"v" to { buf ->

   if (buf.contains("Require")) {

Verilog files can contain "Require" too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sourcerer-io/sourcerer-app/pull/325#pullrequestreview-142162362,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdnqbvsslzG-mbkAtXReaeCkW2KW-Esks5uMNMdgaJpZM4VpebY
.

It definitely can give false positives. I guess it can be replaced on a
stricter regepx, which hopefully won't give false positives. Btw, the
concern is valid for other heuristics we have, and the issue is not so
common, since no complains and all tests pass.
On Wed, Aug 1, 2018 at 8:02 AM, Anatoly Stansler notifications@github.com
wrote:

@anatolystansler commented on this pull request.
In src/main/kotlin/app/extractors/Heuristics.kt
https://github.com/sourcerer-io/sourcerer-app/pull/325#discussion_r206851928
:

@@ -1071,8 +1071,12 @@ val HeuristicsMap = mapOf<String, (String) -> ExtractorInterface?>(
"ux" to { _ ->
CommonExtractor(Lang.XML)
},

"v" to { _ ->

   CommonExtractor(Lang.VERILOG)

"v" to { buf ->

   if (buf.contains("Require")) {

What about "Require" word in comments of verilog?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sourcerer-io/sourcerer-app/pull/325#discussion_r206851928,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdnqf8t_bevbPxYRRvwmaz0kXOOweR7ks5uMZi4gaJpZM4VpebY
.

We have 6 other heuristics that are based on contains function:

"<?hh"
":- module"
"flowop"
"last_client="
"SOURCES"
"HEADERS"

First 4 should be strict enough to not be in comments at least. Probably we can improve last 2, both of them related to QMAKE.
I think "Require" is too generic and probably used in comments often.

It deserves a new issue, please file one if agree. Note, I believe some
regexp should be adjusted as well, since some of them appear generic enough
to trigger false positives.
On Wed, Aug 1, 2018 at 9:31 AM, Anatoly Stansler notifications@github.com
wrote:

We have 6 other heuristics that are based on contains function:

"<?hh"
":- module"
"flowop"
"last_client="
"SOURCES"
"HEADERS"

First 4 should be strict enough to not be in comments at least. Probably
we can improve last 2, both of them related to QMAKE.
I think "Require" is too generic and probably used in comments often.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sourcerer-io/sourcerer-app/pull/325#issuecomment-409575325,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdnqUcAAQijpO9om9kOE5sLCPWmKN-iks5uMa2kgaJpZM4VpebY
.

authors: anatolystansler asurkov

repo: sourcerer-io/sourcerer-app


text:
Update command name for generating conf using utility

Closing this PR as invalid.

repo: splunk/pytest-splunk-addon


text:
Fixed various things that Landscape was complaining about

Coverage decreased (-0.38%) to 76.92% when pulling 448f53e2b56cd4e93c01d02be325598c12f3c3b2 on erikbern/landscape into 1939a06c1b205fcb20ba80223537345333f45c6a on master.

88% -> 90% so I guess minor improvement

Coverage increased (+0.01%) to 77.31% when pulling cd3550589b53ba9bbb2a09f811b2577f8aab070b on erikbern/landscape into 1939a06c1b205fcb20ba80223537345333f45c6a on master.

Coverage decreased (-13.51%) to 63.78% when pulling e606e0fcb8ab1d7f562e280b450707d1658807ac on erikbern/landscape into 1939a06c1b205fcb20ba80223537345333f45c6a on master.

Coverage decreased (-0.11%) to 77.18% when pulling 41e413186a8ca761967d61d3adba233ac65c657f on erikbern/landscape into 1939a06c1b205fcb20ba80223537345333f45c6a on master.

can has merge?

authors: coveralls erikbern

repo: spotify/luigi


text:
remove python dependency

python2 is EOL. Not sure why you need python anyway, it seems to build and test fine.

python2 is EOL. Not sure why you need python anyway, it seems to build and test fine.

Python is needed for building native dependencies in node via node-gyp https://github.com/nodejs/node-gyp#on-unix
Recently the latest sqlite3 prebuilt package was (maybe still is?) not available on s3, causing npm to try to build the dependency natively. I was having trouble getting python3 recognized by tooling to get things to build.
I can see if I can get the python 3 version working.
Should note that python, gcc and make are not included in the final docker image. It's only used during the first stage build.

Python is needed for building native dependencies in node via node-gyp https://github.com/nodejs/node-gyp#on-unix
Recently the latest sqlite3 prebuilt package was (maybe still is?) not available on s3, causing npm to try to build the dependency natively. I was having trouble getting python3 recognized by tooling to get things to build.
I can see if I can get the python 3 version working.
Should note that python, gcc and make are not included in the final docker image. It's only used during the first stage build.

Got python3 working in #937

Got python3 working in #937

authors: eladeyal-intel rickbergfalk

repo: sqlpad/sqlpad


text:
Schema: Allow null if using defaults

Hi @kellerza
good catch! thank you!

the clab files that errored before that change:
name: test
topology:
  defaults:
    kind: srl
    image: srl:latest
  nodes:
    srl1:
    srl2:

authors: hellt kellerza

repo: srl-labs/containerlab


text:
[bindings] Fix Jcom + remove se3 namespace

I also fixed the bindings for exp3 because a unit test was failing on my Macbook (related to #597  and #598)

repo: stack-of-tasks/pinocchio


text:
Wip/jsdoc

Pull Request Test Coverage Report for Build 687

13 of 14 (92.86%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 82.318%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/schema.js
1
2
50.0%

Totals

Change from base Build 686:
-0.004%

Covered Lines:
7020

Relevant Lines:
8295

💛  - Coveralls

authors: coveralls rayslxu

repo: stanford-oval/thingtalk


text:
chore: no more petstore

What's the rationale?

The rationale is the pet store is not a good example of anything so let’s not use it as an example for anything. People generally hate the pet store and I keep getting messages about it.

Yeah, keep the URL around along with some other good URLs. I have a bookmark folder in Firefox!

authors: marcelltoth philsturgeon

repo: stoplightio/elements


text:
Adding missing quote mark

Doh thanks

repo: stripe/ApplePayStubs


text:
型引数->型パラメータ + 一部修正

https://github.com/stzn/the-swift-programming-language-jp/issues/120

repo: stzn/the-swift-programming-language-jp


text:
Store the mime-type and filename. Fixes #215

Will require updates to store this passed metadata to the File metadata resource ultimately.

repo: sul-dlss-labs/taco


text:
add header support in graphql plugin

Fixes #69

repo: suparngp/kotlin-multiplatform-projects


text:
push to live

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

qidonArchiveTest/TOC.md
:white_check_mark:Succeeded

qidonArchiveTest/docfx.json
:white_check_mark:Succeeded

qidonArchiveTest/index.md
:white_check_mark:Succeeded

swatDongArchive0509045211/TOC.md
:white_check_mark:Succeeded

swatDongArchive0509045211/index.md
:white_check_mark:Succeeded

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.


text:
Update CODE_OF_CONDUCT.md

/lgtm

/assign @swx457056

/unassign @swx457056


text:
[OO-12350] Update README for docker overrides

please review @takipi/keymaker

repo: takipi/java-bullshifier


text:
remove all parents/children mentions in framework implementation

LGTM

authors: fengjingchao xiaoyunwu

repo: taskgraph/taskgraph


text:
Added last update date, GBIF terms, and current activities

Thanks, @stanblum. Much appreciated.

repo: tdwg/website


text:
ICON updated

Place icons below the profile image,looks better.

repo: technojam/technojam-frontend


text:
add hswish activation function

derivative(hswish(x)) :: [ relu6(x+3)+ x * _Raw.relu6Grad(x+3) ]  / 6
@dan-zheng does this implementation look correct?

Just a question, should we add lesser known activation functions to the main api base or just maintain them in swift-models for now and think about adding them later? Other than mobilenetv3, I know few models which use this activation function. This case is the same with gelu activation.

@brettkoonce
The gradient looks wrong.
import TensorFlow

@inlinable
@differentiable
public func hswish<T: TensorFlowFloatingPoint>(_ x: Tensor<T>) -> Tensor<T> {
  x * relu6(x+3) / 6.0
}

@inlinable
@derivative(of: hswish)
func _vjpHSwish<T: TensorFlowFloatingPoint>(
  _ x: Tensor<T>
) -> (value: Tensor<T>, pullback: (Tensor<T>) -> Tensor<T>) {
	(
	hswish(x), { v in
		let relu = relu6(v+3)
		let rawRelu = _Raw.relu6Grad(gradients: v, features: x + 3)
		return (relu + rawRelu * v) / 6.0
	})
}

@differentiable
public func hswish2<T: TensorFlowFloatingPoint>(_ x: Tensor<T>) -> Tensor<T> {
  x * relu6(x+3) / 6.0
}

let tensor = Tensor<Float>([-4, -2, 0, 2, 4])

print(gradient(at: tensor) { hswish($0).sum() }) // [0.6666667, 0.8333333, 0.8333333, 0.8333333, 0.6666667]
print(gradient(at: tensor) { hswish2($0).sum() }) // [        0.0, -0.16666667,         0.5,   1.1666667,         1.0]

You should add test.
(Do we need _vjpHSwish in this case, BTW?)

@Shashi456 thought i would add it here first, but if needed can yeah just let it live in swift-models, nbd
@t-ae you are correct, i was doing the math wrong, fixed and updated the test to work properly now! you are also correct, we do not need a vjp pullback in this case (or rather, the relu6 call handles it for us), so removed all of that!  thanks for the demo code!

LGTM, thanks a lot!

It's a bit late, but I think hsigmoid should be renamed to hardSigmoid and hswish to hardSwish.
https://www.tensorflow.org/api_docs/python/tf/keras/activations/hard_sigmoid

For reference, the hard sigmoid function may be defined differently in different places.
https://stackoverflow.com/a/39625795

hsigmoid(x) = min(max((x+3) / 6, 0), 1)
tf.keras.activations.hard_sigmoid(x) = min(max(0.2 * x + 0.5, 0), 1)
The slope value is different.
I found japanese article that describes slope ≈ 0.2 minimizes the mean squared error from sigmoid.
https://qiita.com/hsjoihs/items/88d1569aaef01659bbd5
hsigmoid should be identical to tf.keras.activations.hard_sigmoid.

@t-ae that implementation is the one in the paper "Semantic segmentation of satellite images using a modified cnn with hard-swish activation function."

authors: Shashi456 brettkoonce sgugger t-ae

repo: tensorflow/swift-apis


text:
Updated copyright year to 2021

@googlebot I signed it!

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

@googlebot I signed it!

Thank you for the comment.

repo: tensorflow/tensorflow


text:
Replace more GenericVector and PointerVector by std::vector

I am still looking for a better code pattern as a replacement for PointerVector.

repo: tesseract-ocr/tesseract


text:
Use mb_strlen since text has to be in utf8

:+1: Thanks @ksimka
I have used strlen in other places like in Model\Button and Model\Generic\Element can you change it too ? And also add the requirement of the ext-mbstring extension in the composer.json since this extension is not enabled by default.

Done via #3.

authors: ksimka tgallice

repo: tgallice/fb-messenger-sdk


text:
Configure font-size via query-string

This can be merged, but it's still not fixed for single-number metrics

repo: theodi/bothan


text:
Are you planning BLE Mesh encryption as well?

Sorry, no plans for that unfortunately :(
I do have another project for a full BLE Mesh python stack here: https://bitbucket.org/thomsten/blemesh/
It is very immature, and will probably never be finished, but should be able to extract encryption and decryption from it. Have a look at https://bitbucket.org/thomsten/blemesh/src/master/blemesh/keys.py.
-Thomas


text:
Run full test suite on CI

:balloon:

Thank you @dgalarza!

authors: joshuaogle tysongach

repo: thoughtbot/bourbon


text:
Add OAuth2.Twitter

This exists: https://github.com/thoughtbot/yesod-auth-oauth2/blob/master/Yesod/Auth/OAuth2/Twitter.hs
However, I believe we should remove it:

I couldn't get this working in my project
Yesod.Auth.OAuth works with Twitter
Twitter's documentation claims they only support OAuth 1:

There are two forms of authentication in the new model, both still leveraging OAuth 1.0A.

authors: gabebw pbrisbin


text:
Fixed INLINE typos

Thanks!

authors: hakujin tibbe

repo: tibbe/cassava


text:
trivial README fix

 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.

…ugh, nevermind.

@vthriller thanks for this fix. I assume you didn't want to sign the CLA, or was there another issue?

@erimatnor yep, I simply decided to not spend time on paperwork just to be able to change a couple of characters in README.

authors: CLAassistant erimatnor vthriller


text:
Add support for Liqui.io

@cerko2 Could you fix the conflicts? Should be easy. Then I can pull...

@timmolter They should be resolved now.

repo: timmolter/XChange


text:
ar

Looks like an issue opened by mistake. Please close this @DrJukka

sorry, was looking for something, and apparently had some fingering error :)

authors: DrJukka motin

repo: timothycrosley/isort


text:
Create fr_fr.lang

Thanks!

repo: timrwood/roost


text:
Don't allow empty passwords

Are you still able to create other passports such as Facebook even with this field required in the passport model ?
If I remember well the only way to make sure the password is set and does not break other passports methods, is to change the lifecycle methods from the passport model.

Ah, yep. So it needs to be moved to the local strategy.

repo: tjwebb/sails-auth


text:
Use custom token in .github/workflows/TagBot.yml

Codecov Report

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

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   91.02%   91.02%
=======================================
  Files           3        3
  Lines          78       78
=======================================
  Hits           71       71
  Misses          7        7

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 46a17af...f0730c0. Read the comment docs.

authors: codecov-io tkf


text:
Fix link formatting

Thanks!

authors: mexx tpetricek


text:
build: set otp during release process

Proof it worked: https://github.com/trufflesuite/ganache-cli/runs/687318558?check_suite_focus=true#step:6:186
I'll deprecate the npm release tomorrow.

repo: trufflesuite/ganache-cli


text:
merge necessary features from turms-apm into turms-admin and remove turms-apm (Spring Boot Admin)

Merge turms-apm (Spring Boot Admin) into turms-admin.
Motivation:

Developers don't need to deploy two kinds of independent services while they can be one.
We will have full control the features that we don't need at all and add new features designed for turms

Done. For further enhancements, we will create individual tickets to track.

repo: turms-im/turms


text:
词汇表文件，不知道什么样呢，能否给个demo，或者分享一下，万分感谢

对于不同的数据集建议创建不同的词表。
demo：

authors: ECNUHP tuxchow

repo: tuxchow/ecm


text:
add allowSurrogateChars to support weixin nick_name emoji

@devqin 你自己的包为什么要合到我这里，提错了吧

authors: devqin tvrcgo

repo: tvrcgo/weixin-redpack


text:
less instead of lessc in makefile

Sorry, confusion between less and lessc...

repo: twitter/bootstrap


text:
Added n_steps parameter when calling theano.scan

Ok

authors: elsonidoq zomux

repo: uaca/deepy


text:
Add ProviderBase class

Create branch: issue-8-add-providerbase-class from develop.

Create branch: issue-8-add-providerbase-class from develop.

repo: unity-game-framework/ugf-runtimetools


text:
Refactored to use api text methods

The image/text change looks great, but this also seems to have a bunch of stuff about config changed as well..?

The image/text change looks great, but this also seems to have a bunch of stuff about config changed as well..?

Config doesn't appear to be changed - just factored out and moved to load at a different time.

authors: The-Jonsey andydotxyz okratitan

repo: unix-streamdeck/streamdeckd


text:
Write tests

[ ] App.js
[ ] App.js
[ ] App.js
[ ] App.js

This will be an on-going process


text:
Update 2016-05-01-Ransomware.md

Ho messo dei ritorni a capo per facilitare la lettura in markdown.
Bravo Giuseppe.
Adesso devi accetare la mia pull request, e, se ti vabene, risottporre la tua pull request corretta a me e io la confermo.


text:
colorization stage mem leak fix proposal.

seems ok, merged, ty

authors: andreafioraldi devnexen

repo: vanhauser-thc/AFLplusplus


text:
Log routes

Current coverage is 92.80% (diff: 91.50%)

Merging #3 into master will decrease coverage by 0.21%

@@             master         #3   diff @@
==========================================
  Files            22         24     +2
  Lines           645        751   +106
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            600        697    +97
- Misses           45         54     +9
  Partials          0          0

Powered by Codecov. Last update b5b8dbd...f3149ca

How would you print this from Vapor? Something like this?
print(drop.router)

@tanadeau you'd print like print(drop.router.routes())

Lol wrong "tan"

authors: LoganWright codecov-io tannernelson

repo: vapor/routing


text:
fix typo in README

Thank you @oprogramador, it will be include in the next release 🚀

authors: oprogramador rimiti

repo: visionmedia/supertest


text:
unhexlify: do something with this

Moved to issue https://github.com/voidstarHQ/voidstar/issues/97

repo: voidstarHQ/voidstar


text:
uuid_organization should be allowed to be null

issue closed. PR merged into master

repo: voteamerica/backend


text:
fix(template-explorer): enable cacheHeaders checkbox in module mode

cacheHandlers can be enabled in function mode. The only condition is it cannot be used without prefixIdentifiers: true.

@yyx990803 isn't prefixIdentifiers 'locked' in true state when in module?

Yes, but mode has nothing to do with cacheHandlers. It only has contingency on prefixIdentifiers.

repo: vuejs/vue-next


text:
fix isA*Of exception messages

Could you please rebase this PR?

Could you please rebase this PR?

PR is rebased

PR is rebased

I also changed some error messages. You may have a look again?

I also changed some error messages. You may have a look again?

authors: Nyholm smoench

repo: webmozart/assert


text:
如何设置卡片模式左右展示部分的高度

@Black-Knights   参考  #53

authors: Black-Knights wenchaosong

repo: wenchaosong/Banner


text:
WIP: ROV Logging Example

Pull Request Test Coverage Report for Build 808401010

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

Totals

Change from base Build 807661900:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

authors: coveralls lalmei

repo: whylabs/whylogs


text:
Updated file browser with proper function and variable indentation

Make it use non-gmod dedicated comments and and or

Will start advdupe2 work soon. Finishing up starfall this week.

I finished reviewing and that looks like the only change needed. Can merge after that is fixed.


text:
refactor(keyboard-control): ability to add in runtime new control logic

LGTM

authors: BrooklynKing bodia-uz

repo: wix/playable


text:
what is the name in the param? How to upload a file with params(other data)?

Here's my code.
RNFetchBlob.fetch('POST', `${my_upload_url}` {
        'Content-Type': 'multipart/form-data',
        'Transfer-Encoding': 'Chunked'
      }, [{
        // NOTE: my server requires the 'data' key
        name: 'data',
        filename: '${file_name_of_file_path}',
        type: `${file_type_of_file_path}`
        data: RNFetchBlob.wrap('my path to file')
      }, {
        // NOW: how would I add more params with the same key?
        name: 'data',
        data: queryString.stringify({
          ...some params
        })
      }])
      // listen to upload progress event
      .uploadProgress((written, total) => {
        // NOTE: another weird thing only fires once, with 1
        console.log('uploaded', written / total);

      })
      // successfully uploaded
      .then((response) => {
        console.log('on upload success:', response);
      }).catch((err) => {
        // error here is: Error: unexpected end of stream on Connection
        console.log('on upload error:', err);
      });


text:
Add Country webview to match Android

Closing issue and migrating to new repo.

repo: wordpress-mobile/WordPressCom-Stats-iOS


text:
Messaging

Webservice Teil aus dem Tooling berücksichtigen

Ich schreibe an Joho noch eine Mail bezüglich den Kontrollfragen. Auf Notenstatistik sind alle drauf - ich weiss aber nicht wie aktuell...


text:
Refactoring

Thanks for this, but we're going to decline for the time being; if this convention becomes standard in the rust ecosystem we are happy to revisit, but for the time being the advantages do not clearly outweigh the disadvantages.

authors: adamnemecek cmyr

repo: xi-editor/druid


text:
Code Coverage Workflow

There is nothing to test, so closing coverage


text:
マイルストーンのならびを昇順にする

した

repo: yanqirenshi/SOGH


text:
Fixed syntax error in example

Nice, thanks !

Thanks indeed - I've seen this being copy'n'pasted actually.. :)

authors: Eraldo blueyed jpic

repo: yourlabs/django-autocomplete-light


text:
Support projects that use Symfony 5

Is there something I can do to help move this along?
cc: @samuelwilliams

Is there still anyone maintaining this repository?

Hi @ampaze I will be working on an upgrade to Symfony 6.

@ampaze that is why I have a fork of it.

authors: ampaze xaoseric

repo: yswery/PHP-DNS-SERVER


text:
Update README.md

@debbbbie I intentionally left the "d" out of the backticks since require is a piece of code but the sentence has to sound natural in English. But thanks!


text:
Fix typo for integer (int64) type

Changelog ?

There is no change log from the beginning, should be added later.

OK

authors: Xuanwo pengsrc

repo: yunify/snips


text:
Release/0.3.0

:+1:

authors: jhorstmann lukasniemeier-zalando

repo: zalando-incubator/fahrschein


text:
docs: update .map type for Store

👌👌

authors: dimensi zerobias

repo: zerobias/effector


text:
Upgrade dependencies

track by #9

repo: zeroqn/ophelia


text:
Laravel 8 support

Configuration is publishable and can be changed, but you are right to follow latest Laravel recommendations. Thank you!

authors: elnurxf zgabievi

repo: zgabievi/laravel-promocodes


text:
功能：逗留页面排版更改

finished

repo: zhangwanging/ComeLifeClient


text:
React首屏优化

。

优化前：

别人家的首屏

豆瓣集市
知乎

Babel 编译出来还是 ES 6？难道只能上 polyfill？

[译]高性能React: 3个工具提高你app的速度 - 知乎专栏

repo: zhouzhongyuan/qa


text:
Fix more avatar live-update stuff.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

This looks great, merged, thanks @showell!

authors: showell smarx timabbott

repo: zulip/zulip