text:
[SPARK-25299] Replace ShuffleLocation with BlockManagerId

The large problem with https://github.com/palantir/spark/pull/565 is that individual file server failure handling is far from ideal. In that, because we can only pass back a BlockManagerId to a FetchFailedException, we can only tell the scheduler to recompute blocks that a single executor computed. If external file server at hostA is down, and hostA was written to by executors 1, 2, 3 and 4, we need to increment the attemptId 4 times before we can recompute all the lost blocks on HostA.
This tries to achieve failure handling by removing both ShuffleLocation and MapShuffleLocations and relaxing some assumptions about BlockManagerId within the MapOutputTracker:

It makes BlockManagerId optional within the MapOutputTracker.
It also allows execId within BlockManagerId to be null to support the individual file server case. In such instances, the BlockManagerId won't be returned when the RDD asks for preferred host locations on which to run the shuffle.

There are only a few places where BlockManagerId is exposed in the API of MapOutputTracker, so I think making it optional should be ok?
@mccheah @squito @ifilonenko for SA

I haven't actually looked at the code changes yet, but lemme just say what I think the desired end behavior should be.
A fetch failure from shuffle server A should treat all data on server A as missing.  It shouldn't matter what executors the data originally came from.  If the data came from 10 executors, it should not take 10 fetch failures to mark all the data as missing.
Furthermore, if each executor wrote its output to a mix of servers, then the data hosted on other servers should still be considered perfectly valid.
For example, suppose you had 2 shuffles, each with two tasks:

shuffle 1, task 1, runs on exec 1, writes output to server A
shuffle 1, task 2, runs on exec 2, writes output to server B
shuffle 2, task 1, runs on exec 2, writes output to server A
shuffle 2, task 2, runs on exec 3, writes output to server B

a fetch failure from A would mark shuffle 1, task 1 & shuffle 2, task 1 as missing (but leave shuffle 1, task 2 as OK, even though it also came from exec 2)
a fetch failure from B would mark shuffle 1, task 2 & shuffle 2, task 2 as missing (but leave shuffle 2, task 1 as OK, even though it also came from exec 2).
it sounds like you're saying this plus https://github.com/palantir/spark/pull/565 get that behavior?

Yup, that's correct, that's what the tests in DAGSchedulerShufflePluginSuite.scala are trying to demonstrate, albeit only doing so with a single map id.
With https://github.com/palantir/spark/pull/565, the MapShuffleLocation is not integrated into the scheduler at all and cannot be used to remove anything from the MapOutputTracker, so there was no way to mark a remote host as down. In contrast, this new PR removes the concept of a MapShuffleLocation and relying on the shuffle plugin API to use BlockManagerId to pass on shuffle location info, and uses existing logic (with minor tweeks) to remove MapStatuses from the MapOutputTracker

@squito and @vanzin, did you have any further comments on this? i think is probably the change that we'd want to put in the final SPIP

authors: squito yifeih

repo: palantir/spark


text:
Turn websocket to be bidirectional

Done in 46814428aacbec4fe5e05a711c01eccbdfac2c7c

source: gharchive/issue

license: MIT

license_type: permissive

license_source: github-api


text:
added simple tomcat test

Not working yet, let's fix it first.

source: gharchive/pull-request

authors: jscotka pavlix

repo: pavlix/network-testing

license_source: bigquery


text:
Diagnostics - DNS Lookup, also query for IPv6 addresses

Before this pull request can be accepted you must first sign a CLA as described at https://www.pfsense.org/about-pfsense/#cla. Please read for more details.

You might like to:
s/resolve_host_adresses/resolve_host_addresses/
Just for future maintenance ease for English people.

@phil-davis, ok thanks typo fixed
As for the 'missing CLA'.. ive signed that over a year ago or something like that.. should i do again.?

@PiBa-NL You probably singed it when it was set to expire in 1 year. Now when you sign it again it won't expire anymore. Can you please do it?

There are several unnecessary commits here which will clutter up the history and make it harder to cherry-pick so I'm closing this one. @PiBa-NL if you can re-submit with DNS_ANY, we'll get that merged. Thanks!

authors: PiBa-NL cbuechler pfSense-git-updates phil-davis rbgarga

repo: pfsense/pfsense


text:
fully dynamic replies using one function. fixes #296

Coverage increased (+0.03%) to 93.07% when pulling 4cea4b78b681cee4c10c2d4a970e993d073f386f on dynamic_mock into ce9612c36e364ff9ebfcc27dfa52759d5a3a90b5 on master.

Coverage increased (+0.03%) to 93.07% when pulling 4cea4b78b681cee4c10c2d4a970e993d073f386f on dynamic_mock into ce9612c36e364ff9ebfcc27dfa52759d5a3a90b5 on master.

authors: coveralls pgte

repo: pgte/nock


text:
How to commission public artwork (hint: not so big that it requires planning permission)

Also added, and who knew?!?

license: Unlicense


text:
Replace Npm with Yarn

Current coverage is 100% (diff: 100%)

Merging #22 into master will not change coverage

@@           master   #22   diff @@
===================================
  Files           6     6
  Lines          48    48
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits           48    48
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 74b6f6f...90af6d4

authors: codecov-io jenshult

repo: pinchonation/cashier-dot-cloud

license: mit


text:
option for flipping X when generating DP

I agree, I'll change it later. It was a quick workaround.

repo: playcanvas/engine


text:
Do you have an example, running Playwright and Jest for e2e Angular tests but Karma for Unit Tests? I have been trying to make this work for a while, but I have been unsuccessful. Thanks

Duplicate of that https://stackoverflow.com/questions/66085172/does-anyone-have-an-example-for-running-playwright-and-jest-for-e2e-angular-test


text:
Last Week in Pony - October 8, 2017

Bambang Purnomosidi published a post about using latest pony on arch linux: https://medium.com/@bpdp/latest-pony-in-arch-linux-dea6427bd77f


text:
Simplify block creation for all null values

@martint I have updated it.

repo: prestosql/presto


text:
Include heap dumps in logs downloaded from PT Presto containers

Replaces https://github.com/prestosql/presto/pull/5554

authors: losipiuk wendigo


text:
Latest jCloudEvents

LGTM

authors: matzew sjwoodman

repo: project-streamzi/event-gateway


text:
Fix/dependency

Hey, @projectivetech any chance to have it merged?

@projectivetech ping

repo: projectivetech/sidekiq-queue-pause


text:
Remove InputSignal checks in OutputSignal custom equality

https://github.com/projectriff/node-function-invoker/pull/187#issuecomment-616504693

A similar fix has been included in the now-merged https://github.com/projectriff/node-function-invoker/pull/181

repo: projectriff/node-function-invoker


text:
Addressed code review comments; pulled out CQL and QDM calculators

For this comment, please add a TODO

repo: projecttacoma/bonnie


text:
Update quickstart

PHP CS issues, fix and merge yourself

authors: codeliner prolic

repo: prooph/event-sourcing


text:
apache alongside nginx

I guess we need both web servers installed.

Apache + php-fpm
Nginx + the same php-fpm

Nginx will not create the issues with performance, but with it we can easily approach https://github.com/propeoplemd/cibox/issues/74 issue
We can have builds within subfolder, hosted by Apache and subdomain like buildXXX.ci_domain.com for testing nginx + drupal installed not in subfolder.

Right now we have a security addon (not yet ported) about a closing all http services by passwords based on nginx. So it should be easier to adopt nginx as alternative.
We can have both at a time, but there is a requirement for apache as well - we should switch to php-fpm version at first

authors: podarok ygerasimov

repo: propeoplemd/cibox


text:
A failed karma testrun doesn't fail a GH workflow

Closed via c6f9cdf1f6e2f5e0cbffab66e3487c2f7dc06e15


text:
FEATURE: Make Export definitions editable

Edit functionality for export definitions was added

repo: punktDe/form-persistence


text:
Upload new library version to Maven Central Repostiory

Done. Now, we have to wait for Maven Sync.

repo: pwittchen/NetworkEvents


text:
Update Jupyter base-notebook

This pull request is ready to be merged once the continuous integration jobs on the master branch finished.

repo: pyiron/docker-stacks


text:
update

update


text:
Add explicit functional import in init.py

Thanks!

repo: pytorch/audio


text:
Clearer use about sysmerge and ABI changes.

You Rock! And Rule! Thanks :D

repo: qbit/snap

license: isc


text:
Improved Docs

https://hacktoberfest.digitalocean.com/faq

authors: Krinkle nikhilkhetwal


text:
dry-run BuildSize bot

Codecov Report

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

@@          Coverage Diff          @@
##           master     #6   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          60     62    +2
  Branches       14     14
=====================================
+ Hits           60     62    +2

Impacted Files
Coverage Δ

src/ColorPicker.vue
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

authors: codecov-io rkunev

repo: radial-color-picker/vue-color-picker


text:
lb-controller:v0.7.0

LGTM

authors: cjellick deniseschannon

repo: rancher/cattle


text:
Setup User & User Verification Database

User DB Schema is setup and is able to store signup info.


text:
Prevent RTCV Common from interfering with other libraries named Common

I'd much prefer a cleaner approach to this. Our common is scoped out via namespace, so unless you have conflicting usings, you  shouldn't be conflicting. Can you explain the exact issue you're having that resulted in this one? If it's on discord, point me to the discussion

authors: ChrisNonyminus NarryG

repo: redscientistlabs/RTCV


text:
move

currently, the list for selecting how many items to display is at the top right. Move this to the ul where the other dropdown lists are


text:
Update docs

SGTM

repo: resin-io-library/base-images


text:
Move iev cache to same directory as relaton cache

@ronaldtse I've just put the relaton caches in ~/.relaton/cache and ./relaton/cache. Is it OK to put the IEV cache in  ~/.relaton/ievcache and ./relaton/ievcache?

Are the IEV cache entries different from Relaton cache entries? If they are perhaps they shouldn't be in the same directory?

They are different, which is why I am proposing to put them in a sibling subdirectory. But I can just put them in a different directory, like ievcache.
that said, you got me to customise the local case name in document attributes. I think it is better to put both caches under the same directory specified in attributes, than to specify two different directories.

Since these conceptually do very different things and are independent of each other, no compelling reason to do this.

authors: opoudjis ronaldtse

repo: riboseinc/iev


text:
의존성 있는 라이브러리를 관리할 수 있는 방법을 찾아 개선하자.

Maven 으로 의존성 있는 라이브러리 관리

repo: roamstory/roamstory


text:
Add ability to override where to redirect to after login/out

Thanks, @jcmuller. Looks like a good change!
Could you get the build to pass? Also, could you change the names from after_login/logout_redirect_to to after_login/logout_path?

Changed! And fixed. Then Travis complained about too may octokit requests...
On Mon, Aug 10, 2015 at 2:46 PM, Ross Kaffenberger <notifications@github.com

wrote:

Thanks, @jcmuller https://github.com/jcmuller. Looks like a good change!
Could you get the build to pass? Also, could you change the names from
after_login/logout_redirect_to to after_login/logout_path?
—
Reply to this email directly or view it on GitHub
https://github.com/rossta/opensesame/pull/8#issuecomment-129562321.

--
Juan C.

Thanks for pulling, @rossta! Can you push to rubygems?

Done!
On Thu, Aug 13, 2015 at 5:18 PM, Juan C. Muller notifications@github.com
wrote:

Thanks for pulling, @rossta https://github.com/rossta! Can you push to
rubygems?
—
Reply to this email directly or view it on GitHub
https://github.com/rossta/opensesame/pull/8#issuecomment-130847403.

authors: jcmuller rossta

repo: rossta/opensesame


text:
Correct the documentation for the SCB::set_pendst function

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @therealprof (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

authors: PTaylor-FluenTech rust-highfive

repo: rust-embedded/cortex-m


text:
Add license information

MIT Licence?

LICENSE.TXT file and license information in pom.xml files added for MIT license

authors: drharper rsenden

repo: ryancblack/FoDToJIRA


text:
KNL-1400; mail from yahoo not working

I believe this is the approach that the core team agreed to in the last call.
After trying it, I don't think we need "On behalf of," though you can configure it as an option. The advantage of not having text is that it isn't language-dependent.
So the default behavior is to always replace the From with postmaster, and move the original value to reply-to. Exception: addresses using the Sakai server name.
Options:

disable
configure a regex for addresses that won't be replaced
configure text, e.g. "On behalf of {}"
configure a from address other than postmaster

I think I've tried all the options.

When we can find a Java library for DMARC, it can be used rather than (or together with) the regex to control when the replacement happens.

repo: sakaiproject/sakai


text:
[Requirements] Freehand drawing

@manishksmd
I'll develop the feature soon.

Thank you @salgum1114


text:
sanitize debug similar to payload for events.

@notque I believe this code has the side-effect of removing the sensitive keys also from the input parameter 'payload' (since dictionaries are following reference semantics). Maybe the log statement should be moved to a later point in time where these keys are already removed from the payload anyway.
If the log statement stays where it is and the code is change to make res_payload is a deep-copy of payload before removing the sensitive keys, then I would check for the log-level before calling this method because it is not so cheap.

Fair enough, moving the log statement to a later point in time seems reasonable, and removes the need to run the sanitation so early.

btw. the error is also in the code you copied

I don't understand the error you speak of then, because it should be removing those keys from the payload. What negative is occurring?

it should not modify the input, just the output.

that's fair, but as input, it has no actual negative effect to logs, or to the service implementing them. It's just incorrect in that it's modifying the input as well as the output. Is that a fair understanding?

Ah okay, now I understand. It is definitely a serious issue.

IMHO three things should be done:

fix the issue to make sure it does not break things in the future
check the log-level before calling clean_payload
refactor attach_payload so that it uses clean_payload instead of maintaining the same code two times.

authors: jobrs notque

repo: sapcc/openstack-audit-middleware


text:
add: Minimal implementation j.u.c.CompletionStage

The only used method in Akka is whenComplete but if you feel we should add all of the declared methods I can do it ...

Yes, I'm afraid it must be all or nothing for abstract methods. Otherwise other users of that interface could get bogus behavior.
It's OK to motivate the addition of a JDK interface because Akka needs it, but that's no excuse for lowering our standards of quality, which are "if it links, it's correct".

Too much work for so little gain, sorry I'm not able to tackle this at this point.

authors: andreaTP sjrd

repo: scala-js/scala-js


text:
fix:  mFooterLocked always true in some case #716

The wrong way, the problem has been solved.

The wrong way, the problem has been solved.

Sleep ok?
| |
wslwno1
|
|
邮箱：wslwno1@163.com
|
Signature is customized by Netease Mail Master
On 12/07/2018 00:42, 树朾 wrote:
The wrong way, the problem has been solved.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

In my case, LoadStatePresenter will replace SmartRefreshLayout with LoadStateLayout and then cannot reset mFooterLocked because onDetachedFromWindow is called.


text:
Fixing typo on spanish translation

thanks!

Oh, wait, there are some conflicts that need to be addressed on this first before I can merge

Thank you for the improvements @julienmartin

@sdras No more conflicts!
@Gonzalo2310 You're welcome!

Thank you @julienmartin!

authors: Gonzalo2310 julienmartin sdras

repo: sdras/array-explorer


text:
add the list of exit status to help

close

repo: sekiguchi-nagisa/ydsh


text:
Added testing with Node@6

Coverage remained the same at 79.984% when pulling 5396bb021669c31110b965ba46193c1aa32bc46f on shanel262:master into 9d8eeaf2a527db109c61a1b7a4d3feac7f4605eb on senecajs:master.

LGTM. It fixes Node 6 on Seneca 2.x
It still fails on Seneca 1.x but not related to this pr

authors: coveralls mihaidma shanel262

repo: senecajs/seneca-auth


text:
Feature/rework handle import

@diorcety: this should solve your recent issues, can you please try this branch?

Indeed it works. But still one issue if you replace the definition in #1084 by
def aa():
    raise ValueError("ee")

CRITICAL: I am in trouble. Your input file does not seem to match Pythran's constraints...
E: identifier __pythran_imported__bb$ValueError unknown, either because it is an unsupported intrinsic, the input code is faulty, or... pythran is buggy. (bb.py, line 4)

Here the patch:
https://gist.github.com/diorcety/8f2d94fcab93aee5d3d72048774ec51c

I need test cases!

Replace aa by
def aa(a):
    ret = []
    while a > 2:
        ret.append(0)
        a -=1
    return ret

https://gist.github.com/diorcety/970f5d1502258c08919f6407b9f99748

@diorcety better now?

Better, but not perfect :)
https://github.com/diorcety/pythran/tree/rework-handle-import

@diorcety go?

(this does not fix your performance issue though)

This fixes #1085, #1084 and #1082!

Seems good 👍

authors: diorcety serge-sans-paille

repo: serge-sans-paille/pythran


text:
Add integration with slack to travis

+1

authors: artem-v myroslavlisniak

repo: servicefabric/servicefabric


text:
Make a error if user attempts to purge more than 100 messages.

exports.run = (client, message, args, tools) => {
// This episode will cover purging messages from a channel.
// First, we need to fetch the amount of messages a user wants to purge, this will be stored in args[0].
if (isNaN(args[0])) return message.channel.send('Please supply a valid amount of messages to purge');
// This checks if args[0] is NOT a number, if not it runs the return statement which sends a message in chat.
// We also need to check if the number is LESS THAN 100, since 100 is the max you can delete at once.
if (args[0] > 100) return message.channel.send('Please supply a number less than 100');
// This checks if args[0] is MORE THAN 100, if it is, it returns and sends a message.
// Now, we can delete the messages
message.channel.bulkDelete(args[0])
.then(messages => message.channel.send(**Successfully deleted \${messages.size}/${args[0]}` messages**`).then(msg => msg.delete({
timeout: 10000
}))) // This sends how many messages they deleted to chat, we also want to delete this message. This deletes the message after 10000 milliseconds.
}

repo: shadowolfyt/RodneyTheDiscordBot


text:
Mothur::parsimony: first working test case

kun je je tabs vervangen door 4 spaties? (kun je geany ook automatish laten doen whenever je m saved btw he)

s/geany/vim/g ;)

:set expandtab oid dan ;P

@shiltemann Shall we do a mini hackathon tomorrow?

yeah, sounds good :)

authors: shiltemann yhoogstrate

repo: shiltemann/tools-iuc


text:
Fix the build issue

The bug is in testinfra nothing can be done here to fix it.

repo: siavashoutadi/ansible-role-git


text:
[debian] Create resource package

Avoid to have dup resources in Python2 and Python3. We can wait until Python2 is no more packaged.

Closing, reopen if needed.

repo: silx-kit/silx


text:
Improve es6 syntax and add Prettier

No thanks :)

Why?

authors: rpellerin sindresorhus

repo: sindresorhus/query-string


text:
Search functionality for patterns

Implemented as of patternplate-client@v0.15

authors: LKuechler marionebl

repo: sinnerschrader/patternplate


text:
sbt 1.2.0

As of now we cannot merge this as this would enforce 1.2.x to all consumers of sbt-api-builder

authors: sirocchj sullis

repo: sirocchj/sbt-api-builder


text:
mmlLs:LifestyleModuleとmml_lifestyleとのマッピング

mml_lifestyle_mapping.xlsx

repo: skoba/mml


text:
AData(Dashboard): Pipe amount of active connected dashboard users

Closing this as not doable/extremely hard to track, also not within my interests.

authors: QuantumlyTangled kyranet

repo: skyra-project/skyra


text:
Add base option to RakeTask

Thanks! Good find!

:+1:

Any chance of pushing a new release with this fix?

Yes, I will release new version pretty soon!

authors: jawshooah jkeiser olleolleolle skywinder

repo: skywinder/github-changelog-generator


text:
App Template: finalise list of default plugins

Here is a list of plugins currently in use in different Snowplow apps, to serve as a starting off point. Any suggestions about what we should remove / add?
"com.eed3si9n" % "sbt-assembly" % "xx"
"com.eed3si9n" % "sbt-buildinfo" % "xx"
"com.typesafe.sbt" % "sbt-native-packager" % "xx"
"org.scalastyle" %% "scalastyle-sbt-plugin" % "xx"
"io.get-coursier" % "sbt-coursier" % "xx"
"com.lucidchart" % "sbt-scalafmt-coursier" % "xx"
"com.lucidchart" % "sbt-scalafmt" % "xx"
"com.localytics" % "sbt-dynamodb" % "xx"
"io.github.davidgregory084" % "sbt-tpolecat" % "xx"
"org.scoverage" % "sbt-scoverage" % "xx"
"org.foundweekends" % "sbt-bintray" % "xx"

sbt-buildinfo or "scalify" settings?

"com.localytics" % "sbt-dynamodb" % "xx" - this one is nice to have, but obviously when necessary
"io.get-coursier" % "sbt-coursier" % "xx" - this one can be excluded. Coursier will be default in SBT 1.3.0
"io.github.davidgregory084" % "sbt-tpolecat" % "xx" - I'd exclude this one as well. It's too easy to add and make the build more opaque

Should we recommend "com.localytics" % "sbt-dynamodb" % "xx" for all projects that use DynamoDB as the preferred tool for tests (over mocks)?

@aldemirenes @benjben @chuwy @colmsnowplow @lukeindykiewicz @oguzhanunlu There are a number of decisions we need to make before we can close this issue; for example the one in the comment directly above. I'll create a new comment for each decision and then we can vote yay or nay with the +1 or -1 reaction.

We should not use "com.eed3si9n" % "sbt-buildinfo" % "xx" and prefer rather "scalified" settings. For example:
lazy val scalifySettings = Seq(
    sourceGenerators in Compile += Def.task {
      val file = (sourceManaged in Compile).value / "settings.scala"
      IO.write(file, """package com.snowplowanalytics.example.generated
                       |object ProjectSettings {
                       |  val version = "%s"
                       |  val name = "%s"
                       |  val description = "%s"
                       |}
                       |""".stripMargin.format(version.value, name.value, description.value))
      Seq(file)
    }.taskValue

We should not use "io.get-coursier" % "sbt-coursier" % "xx" and instead update the templates to use sbt 1.3.0 where coursier is the default.

We should use "com.typesafe.sbt" % "sbt-native-packager" % "xx" rather than "org.foundweekends" % "sbt-bintray" % "xx".

We should not include "io.github.davidgregory084" % "sbt-tpolecat" % "xx" in the default plugins.

"com.timushev.sbt" % "sbt-updates" % "xx" should be a recommended plugin.

Let's use scalafmt. Both Idea and metals does use it, so there is no need for:
"org.scalastyle" %% "scalastyle-sbt-plugin" % "xx"
"com.lucidchart" % "sbt-scalafmt-coursier" % "xx"
"com.lucidchart" % "sbt-scalafmt" % "xx"

Revolver is also very nice way or running application via sbt, it forks the jvm, so there is no need to ctrl-c and killing by accident the sbt.
link: https://github.com/spray/sbt-revolver
It also supports ~, so it can test/run things continually
addSbtPlugin("io.spray" % "sbt-revolver" % "0.9.1")

Just my two cents. I'm strongly against any plugin that is not required for reproducible build and can be added to a global sbt config. There are plenty of awesome plugins out there that make engineer's life easier and i'm all for using them, but including them into all projects is a whole different story. We'd like to keep the dependency footprint as small as possible, otherwise we introduce potential blockers for next Scala versions, introduce potential binary incompatibilities (through diamon dependencies) in a build, increase build time.
E.g:

"com.localytics" % "sbt-dynamodb" % "xx" - required for DyanamoDB tests, :+1:
"io.get-coursier" % "sbt-coursier" % "xx" - just makes life easier, can be added globally, does not (or should not) affect the build, 👎
"com.typesafe.sbt" % "sbt-native-packager" % "xx" - required to produce an asset, CI/CD depends on it in projects configs, 👍
"org.foundweekends" % "sbt-bintray" % "xx" - I'm not sure how it is a counterpart of sbt-native-packager, IMO its a whole different pluging that is also required if we continue to use bintray, 👍
"com.timushev.sbt" % "sbt-updates" % "xx" - super-cool, but does not affect a build, can be used globally, 👎
etc.

Thanks everyone for pitching in! If I can summarise the discussion so far:
There are many great plugins for sbt that we can be using and we want to retain freedom to pick our favourite tools. However, many of these are just about improving the developer's workflow. They're not strictly speaking required for a project.
We want to keep the templates as lightweight as possible, for the following reasons:

easier to maintain
provides more flexibility
less risk for binary incompatibilities between builds
less risk that a plugin we come to depend on might block upgrades (eg, to a newer version of Scala).

It makes sense then to only include in the templates those plugins that projects will depend on. Separately, we can have a list of useful plugins for sbt global settings in: https://github.com/snowplow-incubator/engineering-resources.
It looks like in general we agree on this list for the app template:
"com.eed3si9n" % "sbt-assembly" % "xx"
"org.scoverage" % "sbt-scoverage" % "xx"
"com.localytics" % "sbt-dynamodb" % "xx" // For DynamoDB tests

There is still some disagreement about "io.github.davidgregory084" % "sbt-tpolecat" % "xx". I've gone back and forth on this personally, but ultimately I think this should be in the list. If it's not, we might have to worry about all those scalacOptions separately. Surely, it's better to rely on an already thought-out and tested approach?
Finally, there's an outstanding question about whether "com.typesafe.sbt" % "sbt-native-packager" % "xx" and "org.foundweekends" % "sbt-bintray" % "xx" do the same thing. Strictly speaking they don't. However, if we think we would continue to use both Bintray and Docker, it might be more consistent to have the same approach to creating and publishing artefacts on both. Is it consistent to create Docker images with native packager and publish them with Travis; but to have a dedicated publishing plugin for Bintray? Maybe we should use native packager to create the artefacts and then Travis (release manager) can take care of publishing them to Bintray?

What about code formatter? The rules are something that we can talk about of course, but once we'll have some common config it will be a lot easier to work on the code that is autoformatted. There will be almost no doubt and no noise on reviews about code format.
The downside is there will be bigger diffs in some cases because of formatting (ignore white spaces should help at least a little).
Scalafmt does a pretty decent job and the project is active I think, so I would recommend this one.

Oh, of course, I missed it in the final tally.
"com.eed3si9n" % "sbt-assembly" % "xx"
"org.scoverage" % "sbt-scoverage" % "xx"
"org.scalameta" % "sbt-scalafmt" % "xx"
"com.localytics" % "sbt-dynamodb" % "xx" // For DynamoDB tests

I share the idea that all the plugins that are there only to make the developer's life easier should be kept in his global sbt settings. Thus I'm also not in favor of using sbt-updates and sbt-revolver.
I like the idea of removing sbt-buildinfo in favor of "scalified" settings, because the result is the same and it removes a dependency.
I like sbt-tpolecat but I'm afraid that this does not get maintained for years. Also, I think that the scalac options can be quite different depend on the different libraries used for a project.
Regarding sbt-bintray, I think that we should remove it and keep only sbt-native-packager. sbt should only be in charge of creating the artifacts, be it a Docker image or a zip, it should not need to know anything about where things are published. So I'm in favor of using sbt-native-packager to package everything and then let Travis publish them wherever it wants.
What about a common plugin to generate the Scaladoc, like sbt-site?

I'm not forcing either sbt-updates, nor sbt-revolver, but I see a benefit for having revolver in each and every project. It's a common way of running it, but maybe it's oversimplified. Maybe this should be done be other things (because of docker starting locally or other things), like run.sh, which would contain everything that is necessary to run the project.
Of course it doesn't apply to libs and spark jobs and things like that.
So let's leave it out.

100% agree with everything @benjben said.
Just one thing about sbt-bintray. When I said that it is not the same as sbt-native-packager, I missed the point that we're talking about apps (not libs) here. Applications can (and should) be published by Travis / release-manager. Whereas lib is a different story, we use Bintray Maven repo and JCenter to publish things to Maven Central, and this is where we need sbt-bintray.

IMO the ./project/plugins.sbt should contain bare minimum that is needed to compile the project. Whatever is not used in order to build the artifact or does not provide a safety net does not belong there. It takes time to be downloaded, cached, loaded and applied. Having a small set of plugins (as listed by @dilyand) to do the job at hand makes a perfect sense.
The other things can be put what stuff we recommend for developers to be using/use ourselves (be it global, or alongside the usual plugins.sbt) in engineering-resources. And there we can put all the bloops, revolvers etc. With that we'd get a pretty nice onboarding doc as well (along the lines of http://transparency.narrative.io)
Wouldn't it be also good to pull a sample CI config into the model so we can possibly set there things like scalamft failed to validate - fail the build on PR/master?

authors: benjben chuwy dilyand lukeindykiewicz peel


text:
Kinesis Elasticsearch Sink: add support for Elasticsearch 2.x

See https://groups.google.com/forum/#!topic/snowplow-user/L8ZteHGwtWg

Blocked by https://github.com/awslabs/amazon-kinesis-connectors/pull/63

Correction: not blocked, bringing forwards

Hey @alexanderdean, this ticket could be solved quite quickly by either:

Doing robust testing of the 0.7.0 release and checking whether the current HTTP client works for 2.x

Building a second binary with an updated dependency on the Jest Elasticsearch client. (This is preferable as their 2.x library does not work necessarily work backwards)

authors: alexanderdean fblundun jbeemster

repo: snowplow/snowplow


text:
samples: some chages related to conf files

+1, looks ok

lgtm too

thanks, merged

authors: bdilly glima otaviobp

repo: solettaproject/soletta


text:
Com 892 fix

C'est normal qu'on ait plus accès qu'au mois en cours ou au mois précédent ?

authors: cmolle sophiemoustard

repo: sophiemoustard/alenvi-webapp


text:
サボってるユーザにだけYoを送るようにする

done


text:
unable to find optional library org.apache.http.legacy

are you trying to compile soulissapp ?

closed with Zozzariello removal

authors: shineangelic sidaga

repo: souliss/soulissapp


text:
Add support for Laravel 5.3

I don't think it'll work on L5.3 currently. The way L5.3 handles Blade directives has changed a bit.
I'd accept a PR that adds L5.3 compatiblity. L5.1, L5.2 and PHP 5 support may be dropped.

@oLDu67 @freekmurze I think this can be closed.

authors: billmn freekmurze oLDu67

repo: spatie/laravel-partialcache


text:
Item couldn't be removed from the Inventory list

Solved

repo: sphere-com/Checkem


text:
Add github action to build app bundle on Mac/Windows/Linux

Oops, sorry - I meant to open this PR against my own fork.
Thanks for putting this together on the hack day, btw. It's really handy to have when trying out new things.


text:
Patch 1

@angelinagerbas Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

I don't think this adds anything. Probably a mistake?

Yep, sorry I am still trying to figure out how to use GitHub!
On Sun, Oct 16, 2016 at 1:16 AM, Dave Syer notifications@github.com wrote:

I don't think this adds anything. Probably a mistake?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spring-guides/gs-maven/pull/14#issuecomment-254029866,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQAuB-yI4Bpzq1HJ_Is9jS_tDP2yuHBgks5q0cFHgaJpZM4KX3DM
.

repo: spring-guides/gs-maven


text:
Navigation HTML Only and Fixes

Manually merged

repo: stationer/Letterhead


text:
Make items sortable by category (armor, head, boots...)

Implemented in 1.4.2 (201907)


text:
fix typo

Nice!

authors: bgoscinski distolma

repo: storeon/solidjs


text:
Update image to 3.0.0-alpha.13

Hello @synth3tk, could you update to the last v3.0.0-alpha.13.0.1, thanks for your PR!

Sure, I'll take care of it in a few hours!

@lucaperret I just updated it to 13.0.1.

repo: strapi/strapi-docker


text:
SB-1289: fix windows build

Thanks for fixing this! :)

authors: sbespalov steve-todorov

repo: strongbox/strongbox


text:
[BUGFIX] Only add order address to item if its different to cart's address

ack

authors: christianbader turbo-ele

repo: sulu-io/SuluSalesOrderBundle


text:
Develop user feed api.

changes are present in master

repo: suryaprasanna/quora


text:
App 580

@dnsymphony please check.

authors: cmarcondes-daitan rsanchez-daitan

repo: symphonyoss/App-Integrations-Jira


text:
Adding ability to update user avatar

I fixed it...  You have to align powermock and mockito version. Please take a pull.
See:  https://github.com/powermock/powermock/wiki/Mockito#supported-versions

authors: blackrock-engineering ftbb

repo: symphonyoss/symphony-java-client


text:
Credit Atif Aziz for his work

Fixed in ae5f5316629508a6c8a0a4c5731ac411b5afd70d

repo: taspeotis/DeltaCompressionDotNet

license: MS-PL


text:
[TEIIDTOOLS-845] Adding apache commons-logging dependency

On this and all of the other used, but undeclared dependencies, check to see what is in the dependency tree.  In this case it's actually spring-jcl.  Also an explicit version won't be needed - that should be coming from another pom.

authors: shawkins vhalbert

repo: teiid/teiid-syndesis


text:
Branch 174023371

Jenkins, test this please.

repo: tensorflow/tensorflow


text:
fix typo

@noppefoxwolf Please sign the CLA.

Nagging Assignee @martinwicke: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

authors: av8ramit noppefoxwolf tensorflowbutler


text:
Wrong order after drag second column on first position when one pinned to left

Done

repo: terma/angular-small-grid


text:
Deprecate fields that are derived from containerInfo

@kiview yes. I reverted a change to getContainerId because we have some usages of containerId before we query containerInfo.
We will make the field private after the deprecation period, so that only GenericContainer will be able to use it, and others will have to use getContainerId that delegates to containerInfo?.getId()

This was released in 1.11.4 🎉

authors: bsideup rnorth

repo: testcontainers/testcontainers-java


text:
Fix #581: setting field content to null value while updating

Not exactly sure why TravisCI fails.
Any ideas?

The issue is that the password field of the User model is set to "NOT NULL", but this change is allowing null password values to be accepted, which breaks at the DB level.

@fletch3555 @marktopper I was too quick with the fix. It should be correct now :)

authors: fletch3555 marktopper peacemoon

repo: the-control-group/voyager


text:
Remove Project Report issue template and add Daily Activity Report issue template

Fixed by: https://github.com/thenewboston-developers/Contributor-Payments/commit/5b534e2083d1a831cc33ae4a989f64c2b70de43a


text:
I have two questions:1) I train on my dataset for implement 2 classes when last layer sigmoid activate for binary cross entropy ,but the loss didn't convergence, 2) and the output shape is (N, C, 32, 32) when input shape is (N,C,128,128), can I unsample output to mask size（128,128） ？

There are a few reasons for losses not converging. Check if your gradients are misbehaving, if they are you can use gradient clipping to fix it.
torch.nn.functional.interpolate scales tensors for you.

I try gradient clipping but did not work， how initial parameters except resnet layer ？ defaut pytorch？

Be default my refinenet implementation uses torchvision.models.resnet101 for the resnet layer.

Make a plot of the gradients after each batch and see if they are exploding. Also plotting the loss after each batch would help in debugging your issue.

@liuchuanloong Have you solved this problem?

@zhouyuangan  sorry, I just changed another scheme

FYI.
I also encountered the problem for loss not converging when I trained the model for 2 classes, and I deal with it just by decreasing the initial learning rate to 1e-4 or 5e-5.

In my experiments, the learning rate 1e-5 seems too high to ensure stable loss convergence, where after several epochs the DiceLoss would increase to 1.00 and then does not decrease any more. Maybe a much smaller learning rate, e.g., 5e-6, can relieve this problem to some extent.

信件已收到:)

authors: jingyzhang kingyj7 liuchuanloong thomasjpfan zhouyuangan


text:
Add Prophet predictions with multiple regressors in TimeSeriesDataset

take

repo: timeatlas-dev/timeatlas


text:
pull

wrong request

repo: tinywind/spring-boot-streaming


text:
Add whence

when + once

repo: tomekwi/stereo


text:
update status following blocknumber

Pull Request Test Coverage Report for Build 1119

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

Totals

Change from base Build 1118:
0.0%

Covered Lines:
92

Relevant Lines:
103

💛  - Coveralls

repo: tomochain/tomomaster


text:
Lua 5.2 compatibility

Thanks Sam

authors: colesbury soumith

repo: torch/sys


text:
preparing pr

Approved


text:
APT whitelist request for yasm

it looks like travis enterprise is not in sync with the public whitelists

repo: travis-ci/apt-package-whitelist


text:
Restore retention

Thanks; pulling!

repo: treeverse/lakeFS


text:
Update blueprints.yaml

I know it's a very old PR and apologies for that, however this is not sufficient. All you need to make it target _blank is to pass external true


text:
Find a possibility to see Scenario Outline Examples together

maybe I can help this ;)

If you have an idea for this just grab it.

Hi @cjwfly1988 did you start this one already?

Not yet, I don’t have time during this week. Will try next

Started tackling this myself :-)

Closed due to no activity.

repo: trivago/cluecumber-report-plugin


text:
feat: testing first push

@cla-bot recheck

@cla-bot check

authors: skflorin troyronda

repo: trustbloc/agent-sdk


text:
Update Hello World.md

Thx

thx

repo: ttakkku/Hello-World


text:
How to set collapsed" class width dynamically on the basis of number of item total length (length the item )

Same as #24346

repo: twbs/bootstrap


text:
Add bookkeeper kubernetes yaml script.

Going to move this somewhere else.

repo: twitter/heron


text:
Add option to disable update notifications

Update notifications aren't even in Cardinal yet

repo: twizmwazin/CardinalPGM


text:
Added support for Realm 4.3.1

it support Realm 4.3.3 @AlonShahaf
now i am using.

repo: uPhyca/stetho-realm


text:
Update docs and readme

Looks good to me. Thanks for updating documentation!

authors: rarutter tobeycarman

repo: ua-snap/dvm-dos-tem


text:
Cross DC - Reload Workflow History When Branch Token Changes

close in favor of #1456

@yux0 is this still relevant?

repo: uber/cadence


text:
Enika refactoring

Fix merge conflicts, after above notes are fixed we can get this merged in and focus more on the insertion/deletion/update process

authors: TrinaKat enikabiswas

repo: ucladevx/Mappening-Backend


text:
Create the login system

pull request url:https://github.com/ucsb-cs56-webapps/ucsb-cs56-book-sharing-and-exchange/pull/7


text:
Restore Sections nav to working

With 7d10d1e, the section nav is on the right side again, but it still isn't functional.

repo: umbc-hackafe/idiotic-webui


text:
Add examples

@anu0012 Hey! I've created a /example folder, please check it out and create a new PR if you have anything to update.

repo: ummahusla/PotatoCSS


text:
У гостей всегда английский язык

гостей больше нет

authors: evguu undrfined

repo: undrfined/TrappedChest


text:
Search Feed

--DONE--

repo: unjown/devFind


text:
Must functions

Coverage increased (+1.4%) to 91.973% when pulling e9378a511a7638410866fe8223854ae9af34669f on must-functions into 9cd4143b54baf7e621c684e6554dea658a6c732f on master.

Coverage increased (+1.4%) to 91.973% when pulling e9378a511a7638410866fe8223854ae9af34669f on must-functions into 9cd4143b54baf7e621c684e6554dea658a6c732f on master.


text:
Feature/cab 2591

Coverage decreased (-0.7%) to 85.533% when pulling b2c82291602366563e1901fe101c93a661959ec3 on feature/CAB-2591 into 8c4b015c5cbfba0946ff02f5a5a8c724fb28df81 on develop.

Coverage decreased (-0.7%) to 85.533% when pulling b2c82291602366563e1901fe101c93a661959ec3 on feature/CAB-2591 into 8c4b015c5cbfba0946ff02f5a5a8c724fb28df81 on develop.

Coverage decreased (-0.7%) to 85.533% when pulling 0caa0ff82cae1965e35cb9b300afff2983fa5132 on feature/CAB-2591 into 8c4b015c5cbfba0946ff02f5a5a8c724fb28df81 on develop.

Coverage decreased (-0.7%) to 85.533% when pulling 89d251c6b14ae9f3a7111fc5b0bd37b4af6b0f01 on feature/CAB-2591 into 8c4b015c5cbfba0946ff02f5a5a8c724fb28df81 on develop.

authors: agagne coveralls

repo: uw-it-edm/content-services-ui


text:
Various requested fixes

not sure if we want to merge this now, it's a new feature and that reimplements the slideshow urls and would break any links to slideshows currently online

repo: vanda/vam-viewer


text:
Fix few emojis with missing/out-of-order codepoints

Coverage remained the same at 93.125% when pulling b6a3ae011dff5e967f00fc3f827230184a979cd9 on BillyGalbreath:master into 18335af05118b826297b6eb6ce066e97bf6c0c46 on vdurmont:master.

Thanks!

authors: BillyGalbreath coveralls vdurmont

repo: vdurmont/emoji-java


text:
Add recursion support to the def get_files_by_extension, change tests

Add support to the --no-recursion argument.
Which version of puremagic is used? (requirements.txt)

repo: victordomingos/Count-files


text:
Contributing for hacktoberfest 2020

can you please add the label as hacktoberfest-accepted


text:
Install Calibre and point to OneDrive/Library/Books

Installed in 4decd4061, folder redirection is harder.

repo: villasv/WindowsProSetup


text:
Dash

Has all work been added for final review and merge @ddasgupta4 @william-cutler @aliceloukianova @rohankrishnam

repo: visionjo/facerec-bias-bfw


text:
npm init package.json and add node-gyp install dep

Thanks :)

np

authors: guymguym vmolsa

repo: vmolsa/webrtc-native


text:
Add homepage to plugin details

Hi Gergo, can you please fix the file indentation? You changed all spaces to tabs.

I replaced all tabs to spaces. Do you use tabs?

Yes you are right, sorry. PR is merged now.

repo: vojtasvoboda/oc-codelogin-plugin


text:
Allow mapping of dyn-access

Codecov Report

Merging #53 (ae37017) into master (0426a36) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   89.63%   89.75%   +0.12%
==========================================
  Files          15       15
  Lines         984      986       +2
==========================================
+ Hits          882      885       +3
+ Misses        102      101       -1

Impacted Files
Coverage Δ

src/access.rs
93.82% <100.00%> (+0.15%)
:arrow_up:

src/strategy/hybrid.rs
95.45% <0.00%> (+1.51%)
:arrow_up:

authors: codecov-commenter vorner

repo: vorner/arc-swap


text:
Label element doesn't allow to use group underneath

Closed by c0f15bf1f2ee98c3629a554d985c4e4df7dd9742

authors: sebescudie vvvv-dottore

repo: vvvv-dottore/Elementa


text:
Add coverage / build icons to README

Codecov Report

Merging #3 into master will not impact coverage.

@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   92.86%   92.86%
=======================================
  Files          21       21
  Lines         967      967
  Branches      172      172
=======================================
  Hits          898      898
  Misses         20       20
  Partials       49       49

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 52760ea...18e255e. Read the comment docs.

repo: wayfair/brickkit-android


text:
Making unit adjustable in settings menu.

Fixed in 7748536.

Fixed in 7748536.

repo: webventil/OpenSport


text:
aa

aa

repo: wechatpy/wechatpy


text:
Echo extension과 연동되도록 수정

임시 디자인이지만 적용합니다.

authors: ikasty wiki-chan-server

repo: wiki-chan/ModernSkylight


text:
Fix y2h.sh typo

Thanks for the fix.

authors: holmboe jonseymour


text:
Provide example for HA singleton service

Currently getting this with 9.0.2
java.lang.IllegalStateException: java.lang.UnsupportedOperationException
	at java.util.Collections$1.remove(Collections.java:4684)
	at org.wildfly.clustering.server.singleton.SingletonService$SingletonDispatcher.getValueRef(SingletonService.java:266)
	at org.wildfly.clustering.server.singleton.SingletonService.getValue(SingletonService.java:225)
	at org.wildfly.clustering.server.singleton.SingletonService.getValue(SingletonService.java:69)
	at org.wildfly.clustering.service.AsynchronousServiceBuilder.getValue(AsynchronousServiceBuilder.java:92)
	at org.jboss.msc.service.ServiceControllerImpl.awaitValue(ServiceControllerImpl.java:1166)
	at org.wildfly.camel.test.singleton.SingletonRouteTest.testSingletonService(SingletonRouteTest.java:138)

CrossRef: https://issues.jboss.org/browse/WFLY-5612

Blocked by #958

Won't fix - see #1045

repo: wildfly-extras/wildfly-camel


text:
Remove common-core use from host-controller module

Core - Full Integration Build 488 is now running using a merge of 069f7857f4766d1f6f2551df630595db330de40e

Windows Build 910 is now running using a merge of 069f7857f4766d1f6f2551df630595db330de40e

Linux Build 1221 is now running using a merge of 069f7857f4766d1f6f2551df630595db330de40e

Windows Build 910 outcome was SUCCESS using a merge of 069f7857f4766d1f6f2551df630595db330de40e
Summary: Tests passed: 3020, ignored: 95 Build time: 0:18:36

Linux Build 1221 outcome was SUCCESS using a merge of 069f7857f4766d1f6f2551df630595db330de40e
Summary: Tests passed: 3020, ignored: 95 Build time: 0:18:57

Core - Full Integration Build 488 outcome was FAILURE using a merge of 069f7857f4766d1f6f2551df630595db330de40e
Summary: Tests failed: 1 (1 new), passed: 2858, ignored: 378 Build time: 0:48:17
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptMultipleClientsTestCase.encryptedAndSignedRequestFromAlice: javax.xml.ws.WebServiceException: Could not send Message.
	at java.net.SocketInputStream.socketRead0(Native Method)
	at java.net.SocketInputStream.read(SocketInputStream.java:152)

authors: bstansberry wildfly-ci

repo: wildfly/wildfly-core


text:
engine: add kubectl exec update mode flag

what's the status on this pr?

Sorry, going to try to get back to it today.

@nicks updated this to use wire, WDYT?

authors: jazzdan nicks

repo: windmilleng/tilt


text:
Update TCL4 branch to sync UVR schema and fix small issues

@punamverma, what command are you using for your codegen?  This, for example, seems to work fine as-is: docker run --rm -v ${PWD}:/local swaggerapi/swagger-codegen-cli generate -i /local/swaggerhub_api.yaml -l java -o /local/out/java

@BenjaminPelletier Codegen generates a new class Uuid, which is not very useful. Whereas, we want to use the existing java.util.UUID which is created by codegen when you use -
type: string
format: uuid

This is a bug in the codegen CLI.  I'll  hack around it by putting the object type definition directly in the field, but repeating the content of a definition like that is generally suboptimal API design.

@BenjaminPelletier Are you able to codegen? I am getting "failed to read resource listing".
[main] INFO io.swagger.parser.Swagger20Parser - reading from https://github.com/wing-aviation/InterUSS-Platform/blob/b2441a0d612ed1f6cd390666b44547456d5fe913/assets/swaggerhub_api.yaml
[main] ERROR io.swagger.parser.SwaggerCompatConverter - failed to read resource listing
com.fasterxml.jackson.core.JsonParseException: Unexpected character ('<' (code 60)): expected a valid value (number, String, array, object, 'true', 'false' or 'null')
at [Source: java.io.StringReader@5f8edcc5; line: 7, column: 2]

The URL you listed is to the GitHub page describing the swaggerhub_api.yaml resource; you need to link to the raw content itself: https://raw.githubusercontent.com/wing-aviation/InterUSS-Platform/b2441a0d612ed1f6cd390666b44547456d5fe913/assets/swaggerhub_api.yaml

Thanks @BenjaminPelletier for catching it. In a hurry, I missed that.
It looks good to me for codegen now.
If @drsmit22 has any comments.

authors: BenjaminPelletier punamverma

repo: wing-aviation/InterUSS-Platform


text:
added possibility to modify base path

Hi @nick83ola,
Could you please update docstrings(include new parameter obj_path)?
BTW, why do you need custom path?

because I am building a custom player in python but I don't want to publish
it to other services in the system
:-)
2016-07-07 6:39 GMT+02:00 Mikhail notifications@github.com:

Hi @nick83ola https://github.com/nick83ola,
Could you please update docstrings(include new parameter obj_path)?
BTW, why do you need custom path?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wistful/pympris/pull/12#issuecomment-230977485, or mute
the thread
https://github.com/notifications/unsubscribe/AAond5dj1JobhSJ-QKEhB8Az5BrqJbTpks5qTIMIgaJpZM4JDoRL
.

--
P.S. Le informazioni trasmesse attraverso la presente comunicazione sono di
esclusiva
spettanza dell'effettivo destinatario. Nel caso in cui le stesse
raggiungessero, per
qualunque motivo, soggetti non interessati, questi ultimi vorranno darne
immediata
notizia al mittente. In ogni caso, eventuali soggetti diversi dai legittimi
destinatari
della presente comunicazione e dei dati contenuti negli allegati, possono
essere
sanzionati ai sensi del T.U. sul trattamento dei dati personali d.lgs.
196/2003, sia ai
sensi dell'art. 616 del Codice Penale che disciplina la violazione del
segreto sulla
corrispondenza.

merged into dev branch

repo: wistful/pympris


text:
version update

Thank you  very much @D0rmouse!

repo: wix/exodus


text:
OSS Sync | Thu Aug 13 12:19:27 UTC 2020

#autorebase

repo: wix/greyhound


text:
To implement publish package

Related design issue: https://github.com/worona/worona-dashboard-layout/issues/6

Created an external repo for this: https://github.com/worona/publish-native-app-extension-worona

repo: worona/worona-dashboard


text:
REGISTRY-683 Fast Track Training Project Commit

Reviewed, Please merge.

Please create a carbon jira for this

Related Carbon jira [1]
[1] https://wso2.org/jira/browse/CARBON-15779

Reviewed, Pease merge.

authors: Nipuni Rajith90 cnapagoda


text:
Merging 2.x to master

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 5 committers have signed the CLA.:white_check_mark: tmkasun:x: harsha89:x: kasvith:x: kavishkafernando:x: malinthaprasanYou have signed the CLA already but the status is still pending? Let us recheck it.

@kavishkafernando Can you send these (https://github.com/wso2/product-apim/pull/5131/commits/75dcddccfc45e79fff8a096cb34c3a189875db25 , https://github.com/wso2/product-apim/pull/5131/commits/2e4c3a3eb81c4f229248d6e91a83d202e5c997ba ) changes to the master branch

authors: CLAassistant malinthaprasan tmkasun


text:
In Graph, don't keep track of incoming nodes/edges

Hello @wylee ,
If you have some time to handle my comment, I would like to clarify something about the initial structure of incoming edges. How did that work ? I do not understand how the self._incoming got populated.. I understand that it made use of collections.defaultdict() but not at all how, by creating datas in self._data, the "reverse" was done in self._incoming.
Any cue on that ?
Thanks ;)

repo: wylee/Dijkstar


text:
feat: invalid state on text inputs

:tada: This PR is included in version 5.0.0-next.7 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: xamu-co/style-system


text:
Implemented SWAPDB command

LGTM!. @HeartSaVioR?

LGTM!

Merged into master and 2.10. Thanks @sazzad16 .

authors: HeartSaVioR marcosnils sazzad16

repo: xetorthio/jedis


text:
Add a module ome-golang.org to support golang development under emacs

Hi @xiaohanyu ,
Is this not maintained anymore or just in an inactive period?

Hi, @chuchao333 ,
It's still maintained, but with little effort and time since I'm really busy this year, or as you said, in an inactive period.
Still, thanks for your PR, I'm happy today because of your PR, :-).

repo: xiaohanyu/oh-my-emacs


text:
Cosmetic changes

Thanks!


text:
Should throw err while task content changed

Implemented in https://github.com/yunify/qscamel/commit/6a082f7c3f2b40ffec38933c3e5cdf2def954bdd.

repo: yunify/qscamel


text:
为什么单独设置隐藏了头部就头部尾部一起隐藏失效了。看了set方法header隐藏方法里面把全局隐藏给设置成NO了

isNeedRefresh;//是否需要刷新, 默认YES
isNeedHeadRefresh;//是否隐藏头部刷新
isNeedFootRefresh;//是否隐藏尾部刷新
看下是不是设置成isNeedRefresh了

repo: yuwind/HHRefreshManager


text:
Define images for Python 3.7 standard and Alpine

@all-contributors please add @GArmane for maintenance

authors: GArmane pedrogryzinsky

repo: zrpaplicacoes/python-baseimage


text:
mailmap: Add Git .mailmap for zulipchat.com → zulip.com

@gnprice @showell can you confirm this looks right to you for your emails?

LGTM for my emails.
There are a number of other contributors that have used several email addresses over time, or several forms of their name, and it'd be nice to consolidate some of those too -- doesn't have to be part of this PR, and doesn't have to be comprehensive because that can be a never-ending task, but it'd be nice to cover some of the most numerous.
I think most of the dupes show up on scanning this list:
$ git log --format='%aN <%aE>' | sort | uniq -c

Some that stick out at me are
     13 brockwhittaker <bjwhitta@asu.edu>
      3 Brock Whittaker <bjwhitta@asu.edu>
      6 Brock Whittaker <brockwhittaker@Brocks-MacBook.local>
    733 Brock Whittaker <brock@zulipchat.com>
      5 brockwhittaker <brock@zulipchat.org>
     12 Brock Whittaker <brock@zulipchat.org>
    227 Vishnu Ks <hackerkid@vishnuks.com>
    939 Vishnu Ks <yo@vishnuks.com>
    111 Vishnu KS <yo@vishnuks.com>
     18 Zev Benjamin <zev@dropbox.com>
    636 Zev Benjamin <zev@humbughq.com>
     21 Zev Benjamin <zev@mit.edu>
    428 Zev Benjamin <zev@zulip.com>
     60 Tomasz Kolek <tomasz-kolek@go2.pl>
    145 Tomasz Kolek <tomasz-kolek@o2.pl>
     30 synicalsyntax <15116870+synicalsyntax@users.noreply.github.com>
     49 synicalsyntax <cynthialin1004@gmail.com>
      4 Reid Barton <rbarton@dropbox.com>
     60 Reid Barton <rwbarton@gmail.com>
     38 Reid Barton <rwbarton@humbughq.com>
     39 Luke Faraone <lfaraone@dropbox.com>
    356 Luke Faraone <lfaraone@humbughq.com>
      8 Luke Faraone <lfaraone@mit.edu>
    179 Luke Faraone <lfaraone@zulip.com>

Among them, there are a handful of old zulipchat.com addresses that appear for people no longer active:
$ git log --author zulipchat.com --format='%aN <%aE>' | sort | uniq -c
    733 Brock Whittaker <brock@zulipchat.com>
     99 Jack Zhang <jack@zulipchat.com>
     35 umkay <ukhan@zulipchat.com>

which might be nice to cover in particular so there are no zulipchat.com addresses left in the log.
For a complete list of repeated names, there's commands like
$ git log --format='%<|(20)%aN %aE' | sort | uniq | uniq -d -w20

or for repeated email addresses with different names:
$ git log --format='%<|(50)%aN %aE' | sort | uniq | uniq -d -w50

but that's pushing the limits of what I can get the GNU textutils to do :wink: and I don't see a way to get counts at the same time. There's a few dozen in each of those categories.

(Also useful: git shortlog -es.)
That’d require reaching out to those people for a current email address. If you feel excited about doing that, go ahead.

Merging since I think everything is correct here (Those commits were definitely by Steve and should have their authorship data mapped properly).  For additional changes:

We can collapse all of the Brock entries onto brock@zulipchat.com, his last email when he was contributing, if it's useful.
I'm pretty sure @lfaraone would like his data updated; what current email should be used?

(Also useful: git shortlog -es.)

Ah indeed, thanks!

That’d require reaching out to those people for a current email address.

That would be a nice additional touch, but it's not necessary for most of these cases. Here's a recent example of this sort of cleanup in git.git, at git/git@a35b13fce00bf5644d1ab71f9779466be1c7aa67:
commit a35b13fce00bf5644d1ab71f9779466be1c7aa67
Author: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Date:   Fri Nov 9 03:31:14 2018 -0800

    Update .mailmap

    This patch makes the output of `git shortlog -nse v2.10.0..master`
    duplicate-free by taking/guessing the current and preferred
    addresses for authors that appear with more than one address.

    Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 .mailmap | 13 +++++++++++++
 1 file changed, 13 insertions(+)

Similarly we have a lot of cases where there's a real address and a users.noreply.github.com address, or the same email address with slightly different names. It'd be nice to clean those up so the history is clearer.
I'm not going to go do that right now (and I don't expect you to feel obligated to either), but I'd be glad to see a PR from anyone who feels like doing some portion of that improvement.

Created a https://github.com/zulip/zulip/pull/14987 to handle mine and Brock's  in zulip/zulip.

authors: andersk gnprice hackerkid timabbott

repo: zulip/zulip


text:
drafts.js: Narrow when restoring draft.

Also perhaps that zulipbot notice about cancelled builds is a bug?  Probably not super interesting as a notice.

I think zulipbot notices for Travis updates are just all things that happen to the PR (erroring, cancelling, passing, failing).

@lonerz just wanted to follow up on this one, since I think it's a relatively high-impact improvement :).

@timabbott won't have time to complete it soon. I pushed my most current commit.

@sampritipanda is picking this up.

authors: lonerz timabbott