text:
Fix Lint and ClangTidy Warinings

I think we do not want to change the .svg files. They were copied verbatim. Revert this part?

Will the svg file change in future? Because I need to fix the trailing whitespace lint error when sync them, if they won't change anymore, I can avoid sync them.

I don't expect them to change.

authors: wohe zjwoody

repo: googlecartographer/cartographer


text:
Validate odometry and fixed frame poses.

PTAL

source: gharchive/pull-request

license_type: permissive

license_source: bigquery


text:
Add error messages to Kaggle Connector for invalid config and non-csv files.

WIP

Will be fixed by #103

source: gharchive/issue

authors: diminishedprime minhaz

repo: googledatastudio/community-connectors


text:
Add an "Alpha notice" for the pipelines API.

LGTM

authors: deflaux mbookman

repo: googlegenomics/pipelines-api-examples


text:
Document deprecation of uploadArchives and maven

@jjohannes I have updated the content as suggested.

repo: gradle/gradle

license_source: github-api


text:
Some renames around snapshotting

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

authors: bot-gradle lptr


text:
Header and intro html home page

Conflicted. Please resolve it first

Not working very well

Not responsive for HD screen https://gyazo.com/c7717d322a4a37f534fc23755f8d579f
Mobile menu doesn't work https://gyazo.com/63f5d63c60c475bf25cea2ae69d64dbf
Intro block if ugly, Did you follow the design? https://gyazo.com/95c5ca6fa68c100d2eaa2b2a4238fa25

repo: gsmartsolutions/gss-website

license: mit


text:
Merge pending into master

Thanks a lot!

authors: EPashkin GuillaumeGomez

repo: gtk-rs/examples

license: MIT


text:
[vim] tabline.vim raise error when open tab

I haven't used tabpage yet.
So just make no use tabline.

repo: h-youhei/myconfig


text:
Prioritize widgets localization

For now - prioritized:

accident_count_by_car_type
accidents_heat_map (title only)

For now - prioritized:

accident_count_by_car_type
accidents_heat_map (title only)

repo: hasadna/anyway


text:
Update _team.scss

Good Job :claps: it's working!

Fixes #97

repo: haxzie/sosc-website


text:
Slide-Popup anzeigen

This was done via a separate window, not a popup.

authors: dlemmermann hendrikebbers

repo: hendrikebbers/ExtremeGuiMakeover


text:
Add absolute paths to clinical databases in qc_sampleInfo

@Måns Magnusson, @robinandeer : It is no problem to add the path for "cust000-Research.txt" and "cust000-Clinical_master_list.txt" which at least will be present when MIP performed the analysis. However, since the master list is a merged list of several sublist (recorded in the master) there is no way for MIP (at present) to be sure that the sublist also are present in the same directory. I can let MIP assume that the user stores them in the same dir. Basically it will be up to Clinical genomics to make sure that the sublist also are present. Comments?

repo: henrikstranneheim/MIP


text:
refactor: no unused argument, destructure path

Coverage remained the same at 100.0% when pulling 6f2f3b3dddf4a71e27f9b936298742a5ebb4c820 on curbengh:refactor-test into 6b74b3a1203d7acf5eae13d798a2c83a1ac964ab on hexojs:master.

Coverage remained the same at 100.0% when pulling 6f2f3b3dddf4a71e27f9b936298742a5ebb4c820 on curbengh:refactor-test into 6b74b3a1203d7acf5eae13d798a2c83a1ac964ab on hexojs:master.

authors: coveralls curbengh

repo: hexojs/hexo-generator-sitemap


text:
Login/Sign Up

Hi @andrewtpham, can I work on this issue?
If yes, can you give a brief about the issue?

Hi @aryamanpuri thanks for your interest in our OS project! We are happy to have more contributors supporting us. Unfortunately though this is not an issue, just an epic label for our us to manage our roadmap. Please look for other issues with labels like good first issue.
Also, if you can tell me more about your background either BE or FE focused, I can take a look at what issues we have and assign you one? Thanks!

I have my background more on the Frontend side. My skills include HTML, CSS/SASS, C/C++, JavaScript/Nodejs and Reactjs development.
I enjoy being challenged and engaging with projects that require me to work outside my comfort and knowledge set, as continuing to learn new languages and development techniques are important to me and the success of any organization.
Looking forward for contribute to this org.

ping @andrewtpham

Hello can I get assigned to this?

authors: DarkDevCY andrewtpham aryamanpuri

repo: hikaya-io/activity


text:
test: update mod/gsl test to work

I enabled the test again. Let's see if it passes.
jenkins test please

authors: alfred-bratterud mnordsletten

repo: hioa-cs/IncludeOS


text:
Update pynetgear to 0.5.0

You should read and follow the pr template, not just delete it.

Which I do, for useful stuff like actual modifications, but I'll rather not write an essay for a lib update.
requirement_all updated.

Please keep the relevant checks in our PR template in the PR description. This is not optional.
Also please add info on or link to changes introduced in the requirement update.

https://github.com/MatMaul/pynetgear/releases/tag/0.5.0

PR updated with link and check list.

authors: Danielhiversen MartinHjelmare MatMaul

repo: home-assistant/home-assistant


text:
Include success functions within try/except

@JorgeMartinezG Following up here, can you comment here to document the kind of error we were seeing with the campaign that caused the issue?

repo: hotosm/MapCampaigner


text:
chore: Update now build script

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: huchenme/github-trending-api


text:
Suscribe to sns topic as humilis env parameter

@germangh listo por la revisión !
As for the kinesis-processor, add the option of suscribing to sns topic as an env parameter. Also fix the integration tests by retrying to empty the s3 bucket.

repo: humilis/humilis-firehose


text:
use pure golang for runv/api/ and introduce oci into runv/api/

updated

repo: hyperhq/runv


text:
您好，YYLabel有没有办法准确地获取当前文字所占行数呢？

label.textLayout.lines.count;

好像不是很准确。。好像count大部分时候都是0

label.textLayout.rowCount;
如果为 0，表示 label 的高度小于文本内容的高度，这时候算 1 行就行。

增加高度之后可以了 感谢~

authors: SawyerEve ibireme

repo: ibireme/YYText


text:
Create KEANY_JENNA

Thanks Jenna

license: Unlicense


text:
create resizer component (#16)

Codecov Report

Merging #17 into develop will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #17      +/-   ##
===========================================
+ Coverage    99.79%   99.80%   +0.01%
===========================================
  Files           22       23       +1
  Lines          482      518      +36
  Branches        81       88       +7
===========================================
+ Hits           481      517      +36
  Misses           1        1

Impacted Files
Coverage Δ

src/components/Board.tsx
100.00% <100.00%> (ø)

src/components/Resizer.tsx
100.00% <100.00%> (ø)

.../components/integrations/MoveWithoutValidation.tsx
100.00% <100.00%> (ø)

src/constants/constants.ts
100.00% <100.00%> (ø)

authors: codecov-commenter ildar-icoosoft

repo: ildar-icoosoft/react-chessboard


text:
Update to work with 2.4

:+1:

Abandoning old PR.

authors: MisinformedDNA davekiss


text:
skip api request when the last request is not completed

I left a couple comments, but nothing major. However, I thought the original issue was w/ loading Pods, not Metrics. Did I misunderstand the ticket?

Yes that's right. But it's due to the periodical request in the setInterval in metrics(). So this issue should exist on any pages which call the metrics().


text:
Unable to use WHERE clause to filter by tags in Influx http api

Hi Folks,
Firstly apologies if this is a silly question, but I can't get this working at all.

repo: influxdata/influxdb


text:
Buildkite: Bump the acceptance test timeout to 150 minutes

bors r+

authors: disassembler erikd

repo: input-output-hk/cardano-sl


text:
ETCM-702 Describe the Upnp config function in the Mantis docs

Looks good to me

repo: input-output-hk/mantis-docs


text:
Adding some metadata to the task field for reporting upstream.

 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.Bryant Larsen 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.

This will need a rebase.
Also, I think for the build to pass, we will need to get simple-swf published first

Codecov Report

Merging #12 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   70.99%   70.99%
=======================================
  Files          33       33
  Lines        1272     1272
  Branches      262      262
=======================================
  Hits          903      903
  Misses        369      369

Impacted Files
Coverage Δ

src/workers/ActivityWorker.ts
29.7% <0%> (ø)
: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 4bc96e4...a342b83. Read the comment docs.

authors: CLAassistant addisonj codecov-io kblibr

repo: instructure/ftl-engine


text:
ui: implemente citation box

closed via https://github.com/inveniosoftware/invenio-app-rdm/pull/913 and the likes

authors: fenekku ppanero

repo: inveniosoftware/invenio-rdm-records


text:
WebStyle: normalize invenio.css indetation

[ ] Can you please use "normalise"? (British English)
[ ] Can you fix "indentation" in the commit header? (missing n)
[ ] Reviewed-by: Tibor Simko <tibor.simko@cern.ch>

:+1: done thanks!

authors: drjova tiborsimko

repo: inveniosoftware/invenio


text:
New Stable Version of swiss-weather-api

RE-CHECK!

Automated adapter checker
ioBroker.swiss-weather-api

[ ] :heavy_exclamation_mark: [E506] More non translated in german or russian words found in admin/words.js. You can use https://translator.iobroker.in/ for translations
[ ] :eyes: [W505] setTimeout found in "main.js, but no clearTimeout detected

ioBroker.swiss-weather-api

[ ] :heavy_exclamation_mark: [E506] More non translated in german or russian words found in admin/words.js. You can use https://translator.iobroker.in/ for translations
[ ] :heavy_exclamation_mark: [E812] file test/ found in repository, but not found in .npmignore
[ ] :heavy_exclamation_mark: [E815] file .travis.yml found in repository, but not found in .npmignore
[ ] :heavy_exclamation_mark: [E816] file tsconfig.json found in repository, but not found in .npmignore
[ ] :eyes: [W505] setTimeout found in "main.js, but no clearTimeout detected

Add comment "RE-CHECK!" to start check anew

Automated adapter checker
ioBroker.swiss-weather-api

[ ] :heavy_exclamation_mark: [E506] More non translated in german or russian words found in admin/words.js. You can use https://translator.iobroker.in/ for translations
[ ] :eyes: [W505] setTimeout found in "main.js, but no clearTimeout detected

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.swiss-weather-api

:thumbsup: No errors found

[ ] :eyes: [W505] setTimeout found in "main.js, but no clearTimeout detected

Add comment "RE-CHECK!" to start check anew

But please check also
👀 [W505] setTimeout found in "main.js, but no clearTimeout detected

... else compact mode will break

I need some help for solving "setTimeout found in "main.js, but no clearTimeout detected"
What exactly is the problem? In belong to "setTimeout" I haven't changed anything compared to my previous stable version.

Then it was broken all the time and was not seen in initial review :-)) We added this as a check in the repo checked to make sure to get this solved.
For compact mode to work correctly it is essential that all timeouts or intervals (and also resoirces like servers, connections and such) used in the adapter are completely cleared and ended in "unload". If a timeout continues to run after unload in compact mode this can have bad effects to the whole system.
Best is to store all timeouts (in your case) in a variable and use clearTimoeut to stop that in unload

Okay, thanks for the explanation. :)  I'll do the change this weekend.

RE-CHECK!

Automated adapter checker
ioBroker.swiss-weather-api

:thumbsup: No errors found
Add comment "RE-CHECK!" to start check anew

authors: Apollon77 GermanBluefox baerengraben


text:
update NodeJS-versions

node 14 is no yet fully ready for testing, .travis.yml is for automated testing only no relation to installation

Hi @DutchmanNL!
Main reason to adjust Travis configuration was, that recent jscontroller does no longer work with nodejs v8. Therefore I updated in my fork to 10,12,14 instead of 8,10,12 to be able to execute Travis again.

Which is Fine ☺ I will Remove node 8 for all my adapters by next week and check the status of node 14 testing.
Not sure if our testing automation is fully ready for 14.
Will give an update here with the outcome
Thank you providing feedback, appreciated !
PS : Testing must be Updates in travis and github actions 😉, we move more and more to github actions allowing automated deployment
I will reopen this pull as reminder of node 8 removal and 14 addition

node 8 removed and 14 successfully implemented, you can have a look at https://github.com/iobroker-community-adapters/ioBroker.tado/commit/dead74908230514dd94fd8b6d412c4bda8fcf8d8 to see how Travis & GitHub Actions are configured


text:
Move common definitions to base classes

done

repo: iotaledger/entangled


text:
ODIM-3937 use fsnotify logic in svc-api

closing this PR, this is handled in feature branch

repo: irappa-pattar/ODIM


text:
html function receives item as parameter

Thanks!

FYI, I published this change with version 0.1.8 on npm just now.

authors: Eonfow isaacplmann


text:
Blackened the codebase

Probably the most important commit in the history of django-urls, thank you.

authors: isik-kaplan sergei-maertens


text:
spelling correction

Hi @leecalcote. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

authors: istio-testing leecalcote

repo: istio/istio


text:
fix typos

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @islinwb. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@islinwb: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
de5a7d0fd001bed02a62d9fdd842678b157c4ed6
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@islinwb: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
de5a7d0fd001bed02a62d9fdd842678b157c4ed6
link
/test istio-unit-tests

prow/istio-pilot-multicluster-e2e.sh
de5a7d0fd001bed02a62d9fdd842678b157c4ed6
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests

/test istio-pilot-multicluster-e2e

@islinwb: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
de5a7d0fd001bed02a62d9fdd842678b157c4ed6
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth

authors: geeknoid ijsnellf islinwb istio-testing


text:
Create setup.py

setup.py created and working properly.

repo: ivopetiz/crypto-algotrading


text:
add circle support

I'll merge this asap :)

I have done both already, you should have gotten some emails about it.

Sorry, totally forgot about this for a while!
Sadly it still doesn't look like I'm able to add it to CircleCI. Not sure exactly what permissions are required.

Btw I've just noticed that you have github actions enabled for this project, which is probably a better way to do this.
I'll close this in favour of that!

authors: antony jacobmischka

repo: jacobmischka/svelte-flatpickr


text:
Fixes #1388

[Current coverage][cc-pull] is 97.21%

Merging [#1390][cc-pull] into [master][cc-base-branch] will decrease coverage by <.01%

@@             master      #1390   diff @@
==========================================
  Files            86         86
  Lines         10256      10256
  Methods           0          0
  Messages          0          0
  Branches       1821       1821
==========================================
- Hits           9971       9970     -1
  Misses          157        157
- Partials        128        129     +1

Powered by Codecov. Last updated by [9643b55...84c6f15][cc-compare]
[cc-base-branch]: https://codecov.io/gh/javaslang/javaslang/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/javaslang/javaslang/compare/9643b55014f325fcb06f7ae0852c8153e1a55a60...84c6f1586eead3a24509a1b9a937794f799e3268
[cc-pull]: https://codecov.io/gh/javaslang/javaslang/pull/1390?src=pr

authors: codecov-io danieldietrich

repo: javaslang/javaslang


text:
Add supported Python and Django badges to README.rst

Thanks, seems like a good addition!

authors: aleksihakli hramezani

repo: jazzband/django-axes


text:
Generate type declarations automatically

Codecov Report

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

@@          Coverage Diff          @@
##           master    #12   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           8      8
  Branches        1      1
=====================================
  Hits            8      8

Impacted Files
Coverage Δ

src/Checkbox.tsx
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 bcd5487...7caadbe. Read the comment docs.

authors: codecov-io jchiam

repo: jchiam/react-three-state-checkbox


text:
fix beaker acceptance tests on Ubuntu

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

authors: jenkinsadmin jhoblitt

repo: jenkinsci/puppet-jenkins


text:
Update classifier.md

Pull Request Test Coverage Report for Build 310

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.3%) to 94.193%

Files with Coverage Reduction
New Missed Lines
%

autokeras/utils.py
1
80.95%

autokeras/net_transformer.py
3
92.19%

Totals

Change from base Build 309:
-0.3%

Covered Lines:
1395

Relevant Lines:
1481

💛  - Coveralls

The classifier.md is automatically generated from the docstrings in the code.
You may want to modify the docstrings instead.
Thank you for your contribution.

authors: coveralls jhfjhfj1 vikaskyadav


text:
Use the JHipster BOM to manage the Azure Spring Cloud client version

Thanks! It's not used yet (the version is fixed at the moment), so let's merge as there is no risk

repo: jhipster/jhipster


text:
Make application directory where data files are stored configurable #6

Coverage increased (+0.02%) to 89.946% when pulling b55b89303e9c35fad6e8d86682f45bd8fea2c1de on configurable-app-dir into 9ddb094ec747edf7e283404744a1dc47de4ec7c5 on master.

Coverage increased (+0.02%) to 89.946% when pulling b55b89303e9c35fad6e8d86682f45bd8fea2c1de on configurable-app-dir into 9ddb094ec747edf7e283404744a1dc47de4ec7c5 on master.

authors: coveralls jmathai

repo: jmathai/elodie


text:
Default to UTC if date.timezone is not defined

Is this issue similar with the one shown as below?

Fatal error: Uncaught exception 'Exception' with message 'DateTimeZone::__construct(): Unknown or bad timezone ()' in /Users/jun/Dev/mongoqp/vendor/twig/twig/lib/Twig/Extension/Core.php:78 Stack trace: #0 /Users/jun/Dev/mongoqp/vendor/twig/twig/lib/Twig/Extension/Core.php(78): DateTimeZone->__construct('') #1 /Users/jun/Dev/mongoqp/src/app.php(27): Twig_Extension_Core->setTimezone('') #2 /Users/jun/Dev/mongoqp/vendor/pimple/pimple/lib/Pimple.php(201): {closure}(Object(Twig_Environment), Object(Silex\Application)) #3 /Users/jun/Dev/mongoqp/vendor/pimple/pimple/lib/Pimple.php(126): Pimple->{closure}(Object(Silex\Application)) #4 /Users/jun/Dev/mongoqp/vendor/pimple/pimple/lib/Pimple.php(83): Silex\Application::{closure}(Object(Silex\Application)) #5 /Users/jun/Dev/mongoqp/src/app.php(43): Pimple->offsetGet('twig') #6 /Users/jun/Dev/mongoqp/web/index.php(7): require('/Users/jun/Dev/...') #7 {main} thrown in /Users/jun/Dev/mongoqp/vendor/twig/twig/lib/Twig/Extension/Core.php on line 78

This issue is resolved when I set the "datetimezone" in php.ini.

authors: jarvinia jmikola

repo: jmikola/mongoqp


text:
added string sub example file

@jcurlee You're good!

authors: jcurlee paulghaddad


text:
fix section anchors

Thanks!

repo: jmoiron/sqlx


text:
Allow the title of the note to be changed

You can already do this by clicking on the title (Google docs style). Let's discuss separately how to improve discoverability of that.

repo: jordonwii/timeline-notes


text:
added check for 'kwargs' attribut eliminates error if method does not contain a optional (kwargs) attribut

I’ll push it up to PyPi sometime soon.

authors: BCRobster jpavlav

repo: jpavlav/UptimeRobotPy


text:
upgrade to vue2-leaflet v1.0.0 [WIP]

suffers from an issue where markers do not reappear on second zoom with following stacktrace
vue.runtime.esm.js?a427:587 [Vue warn]: Error in callback for watcher "latLng": "TypeError: Cannot read property '_zoom' of undefined"

found in

---> <LMarker>
       <VMarkerCluster> at /opt/Workspace/contrib/vue2-leaflet-markercluster/Vue2LeafletMarkercluster.vue
         <LMap>
           <Example> at /opt/Workspace/contrib/vue2-leaflet-markercluster/example.vue
             <Root>

TypeError: Cannot read property '_zoom' of undefined
    at NewClass.addLayer (leaflet.markercluster-src.js?7b9c:135)
    at NewClass._moveChild (leaflet.markercluster-src.js?7b9c:718)
    at NewClass._childMarkerMoved (leaflet.markercluster-src.js?7b9c:705)
    at NewClass.fire (leaflet-src.js?6144:593)
    at NewClass.setLatLng (leaflet-src.js?6144:7405)
    at VueComponent.n.(:4000/anonymous function).custom.setOptions.t.$watch.deep (webpack-internal:///80:1:2003)
    at Watcher.run (vue.runtime.esm.js?a427:3229)
    at flushSchedulerQueue (vue.runtime.esm.js?a427:2977)
    at Array.eval (vue.runtime.esm.js?a427:1833)
    at flushCallbacks (vue.runtime.esm.js?a427:1754)

issue appears to be caused by following code on mounted in LMarker.vue from vue-leaflet package, in combination with animate option
    this.mapObject.on('move', (ev) => {
      if (Array.isArray(this.latLng)) {
        this.latLng[0] = ev.latlng.lat;
        this.latLng[1] = ev.latlng.lng;
      } else {
        this.latLng.lat = ev.latlng.lat;
        this.latLng.lng = ev.latlng.lng;
      }
    });

setting animate to false seems to resolve the issue

should be fixed; waiting for https://github.com/KoRiGaN/Vue2Leaflet/pull/148  to be merged & published

Awesome! ping me here again when KoRiGaN/Vue2Leaflet#148 is merged so we can adjust the versions and merge this PR. Thanks!

Thank you for your works @naderio and please ping me too with any progress.

@jperelli pinging you, as merge has occured: A new release, version 1.0.2 is out!

@jperelli should be ready to merge

Awesome! I'll test this later today and hopefully will be merged and released today. Thank you very much for your contribution @naderio !

sorry for the delay guys, I merged it and I'm now publishing to npm version 2.0.0

authors: fdenzer jperelli naderio sangdth


text:
prevent circular execution of deps on upgrade

not bad obama

authors: chuckbutler marcoceppi

repo: juju-solutions/reactive-base-layer


text:
Scala 2.13.0-RC1

Thanks @xuwei-k

authors: julien-truffaut xuwei-k

repo: julien-truffaut/Monocle


text:
Feature that allows transforming a discriminator's value to a valid classname

Thanks

repo: julienrf/play-json-variants


text:
Initial version with example

I'm not sure separating model and view model in this case makes a lot of sense. I think we can combine IEditorModel and IEditorViewModel into just IEditorModel. It's technically a view model, but that's just semantics.

ping @Carreau

We need your input on what you think an EditorModel needs to have

@dwillmer LGTM for a first cut (modulo comments). I'm assuming we'll iterate on this.

ping @jasongrout You seem to have done some interface work for input area/editor. I think you and Dave should sync up.

authors: dwillmer sccolbert

repo: jupyter/jupyter-js-editor


text:
Update library to remove compiler warnings on upcoming 0.14.0 PS release

This commit was pushed to the ps-0.14 branch.

repo: justinwoo/purescript-type-isequal


text:
Implement Create Professional

A combo de cidades não está carregando, mesmo após selecionar o estado
A combo de abrangência também não carrega.
A modal do cadastro do serviço também não está sendo apresentada após o clique do botão.
Importante: Quando submeto o formulário as combos carregam normalmente.

authors: emivaljr vlbgomes

repo: kabrabom/web


text:
Add heatmap formatting rule

Coverage increased (+0.1%) to 91.833% when pulling d79afd7f6129cb2c5c556ffcfef9c910ce8fd236 on gl3nn:heatmap-formatting into d1bbd48f75f9f5a7d00c13019b4976eff01837d1 on kayak:master.

repo: kayak/fireant


text:
Fix small typo in the readme

@daraul Thanks. :+1:

authors: daraul kbparagua

repo: kbparagua/paloma


text:
Convert to typescript

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #1   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         6    +1
  Lines           39        72   +33
  Branches         0         7    +7
=========================================
+ Hits            39        72   +33

Impacted Files
Coverage Δ

src/build-error.ts
100.00% <100.00%> (ø)

src/build.ts
100.00% <100.00%> (ø)

src/constants/errors.ts
100.00% <100.00%> (ø)

src/constants/status.ts
100.00% <100.00%> (ø)

src/constants/template.ts
100.00% <100.00%> (ø)

src/index.ts
100.00% <100.00%> (ø)

... and 1 more

Continue to review full report at Codecov.

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

repo: kdcsoftware/api-gw-resp


text:
Applying updates from source.

Applying changes from source.

repo: keithtmccartney/AspNetCoreSpa


text:
Replace react-native-device-info with react-native-native-modules

@mrtnzlml updated, keeping it local.

repo: kiwicom/react-native-app


text:
Bump kafka_protocol version

Pull Request Test Coverage Report for Build 1229

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 80.595%

Totals

Change from base Build 1221:
0.07%

Covered Lines:
2222

Relevant Lines:
2757

💛  - Coveralls

repo: klarna/brod


text:
Kontena-storage: upgrade rook/ceph to 0.9

Fixed in #1349

repo: kontena/pharos-cluster


text:
Create auto-generated reference docs for the Pipelines API

/assign @vicaire

We now have an API reference generated from the code. It's not continuously built, but it's auto-generated in that the doc emerges from the code comments. I'm going to close this issue.
/close

repo: kubeflow/website


text:
Changes to support releasing v0.0.2-rc.1

@gyliu513 The image will be updated when we create a new release tag.

/lgtm

authors: font marun


text:
Created variable for flannel download url

How about moving "{{ flannel_download_url_base }}v{{ flannel_version }}/flannel-{{ flannel_version }}-linux-amd64.tar.gz" to defaults too?
# defaults/main.yaml

#The default url to download the flannel tar from
flannel_download_url_base: "https://github.com/coreos/flannel/releases/download/"
flannel_download_url: "{{ flannel_download_url_base }}v{{ flannel_version }}/flannel-{{ flannel_version }}-linux-amd64.tar.gz"

In this way all information about download URL will be in one place.

I signed it!

Oh, and I think developers here prefer slash in expression, rather then in variable value (someone commented about this one of my previous commits):
flannel_download_url_base: "https://github.com/coreos/flannel/releases/download"
flannel_download_url: "{{ flannel_download_url_base }}/v{{ flannel_version }}/flannel-{{ flannel_version }}-linux-amd64.tar.gz"

Thanks @rutsky, made those changes

Would it be possible to include flannel_download_url_base, kube_download_url_base, and pypy_base_url commented out in group_vars/all.yml It would enable anyone to host these files and fetch them from a local server. I saw cluster set up time go from ~45 minutes to less than 10 minutes with this change.

If not, this should at least be a variable instead of a hard coded download url to enable the option

cc @rutsky @danehans @eparis

authors: rutsky stephenrlouie

repo: kubernetes/contrib


text:
changed errata in Thin / Thick provisionned

/lgtm

authors: ahmetb bottkars

repo: kubernetes/examples


text:
Remove extraneous text left behind by improper merge.

yes looks good

authors: MHBauer foxish


text:
update zh trans /docs/setup/learning-environment/minikube.md

/assign @haibinxie

/lgtm
/approve

authors: haibinxie tanjunchen

repo: kubernetes/website


text:
Host paths as iSCSI LUNs

Thanks for the comments. I addressed all of them.
The libvirtd change was intentional - but I dropped it from this PR.
This change is verified, I used it in my demo preparations.

retest this please

Updated, @rmohr

@fabiand my comments before were confusing, sorry. I don't see any reason to write remote-viewer  stderr output in a hidden file.

authors: fabiand rmohr

repo: kubevirt/kubevirt


text:
Refactor transport set methods to pseudo private

Coverage remained the same at 79.321% when pulling aef7c6de18ac1395faf0c75c0fbd7e055a011171 on rvegas:private_transport into 8862634205c3cc2e10632f4c6a0060d11c95d4cf on kusanagi:master.

authors: coveralls rvegas

repo: kusanagi/katana-sdk-node


text:
docs(readme): Update badges

Coverage remained the same at 80.374% when pulling 6e798a2ff06490666efdb1d21f0051a6544e8cf9 on orestes:master into ffbb43ddc4a12385b37b1887e165d8abee934ec6 on kusanagi:master.

authors: coveralls orestes


text:
Add Ropsten node. Remove ETC node. Add dropdown to switch between the two globally

hey @tayvano,
Would you minding explaining what "remove ETC node" means? Does it mean you are stopping supporting ETC? Or are you just removing ETC node to lower operation cost?
Thanks,

We will be removing ALL ETC functionality. You can use an older repo (i will make a post with which one) to have ETC functionality but since there will be no ETC node for it to talk to, you will need to point it to your own node in order for it to work.
Again, I'll be making a post explaining how to do all of this.
This is to eliminate both server costs (we will be starting a swarm node and ropsten node in the coming weeks as well) as well as the time spent supporting ETC and the messy code that supporting both requires.

hi @tayvano I'm sorry to hear that. Would you be willing to maintain an 'etc' branch for 'etc' functionalities? I'm sure people will want to send you PRs to help you maintain it.

hey taylor,
if I understand it correctly, the way you are supporting both chains is to have a isClassic argument passed to all the functions that have blockchain operations. do you think we could create a global constant that determines the chain, and rely solely on that constant? We can still support both chains in the codebase with minimum code, or maybe on a separate branch if you want.
I would be happy to help you maintain it.
let me know what you think.
thanks,

hey guys, I see that you are busy working right now. I just want you to know that the latest etc geth 3.2.3 supports EIP 155 now (here: https://github.com/ethereumproject/go-ethereum/releases). thank you again for this amazing project.

ETC is still enabled. We will be removing our ETC node so users will have to add their own in teh coming days. Launching this tonight.

repo: kvhnuke/etherwallet


text:
Add gastracker node for ETC

Looks like this change is included in #2013. I'll close this and merge the other one.

authors: gamalielhere splix


text:
devop/add-etc-nodes

reference https://github.com/MyEtherWallet/MyEtherWallet/issues/1427


text: User's avatars show next to their tweets.

repo: laduke/twitter


text:
Reduce compute time

Before / After:
- delta:
  > min: 12 / 11
  > max: 78437 / 44049
  > avg: 10167 / 5610
- is_bottom:
  > min: 2 / 2
  > max: 35 / 18
  > avg: 2.7 / 3
- is_strict_inflation:
  > min: 11 / 9
  > max: 26203 / 1458
  > avg: 9492 / 494
- merge:
  > min: 9 / 7
  > max: 35771 / 1247
  > avg: 9485 / 488
- mutate:
  > min: 20 / 15
  > max: 17462 / 1467
  > avg: 4118 / 349
- new:
  > min: 3 / 3
  > max: 2700 / 2865
  > avg: 466 / 478
- query:
  > min: 5 / 6
  > max: 905 / 987
  > avg: 179 / 183

Time reduced to:

delta: 55%
is_strict_inflation: 5%
merge: 5%
mutate: 8%

Before / After (state_awset):
- delta:
  > min: 12 / 11
  > max: 78437 / 44049
  > avg: 10167 / 3963
- is_bottom:
  > min: 2 / 2
  > max: 35 / 18
  > avg: 2.7 / 3
- is_strict_inflation:
  > min: 11 / 9
  > max: 26203 / 1458
  > avg: 9492 / 189
- merge:
  > min: 9 / 7
  > max: 35771 / 1247
  > avg: 9485 / 179
- mutate:
  > min: 20 / 15
  > max: 17462 / 1467
  > avg: 4118 / 119
- new:
  > min: 3 / 3
  > max: 2700 / 2865
  > avg: 466 / 478
- query:
  > min: 5 / 6
  > max: 905 / 987
  > avg: 179 / 183

Time reduced to:

delta: 39%
is_strict_inflation: 2%
merge: 2%
mutate: 3%

@vitorenesduarte Is this good to merge?

@cmeiklejohn Yes.

authors: cmeiklejohn vitorenesduarte

repo: lasp-lang/types


text:
skydome怎么添加碰撞器呢

？？？？

repo: layabox/layaair


text:
Draw rectangle instead of circle in webcam facedetector

Hi, would you mind elaborate on the rationale of drawing rect instead of circle in this sample?
Thanks.

authors: JorgeMartinezG lazywei

repo: lazywei/go-opencv


text:
open graph metadata for video and audio

currently broken when loading file pages directly

repo: lbryio/lbry-desktop


text:
Librato MD Support

cc @jderrett

repo: librato/statsd-librato-backend


text:
Upgrade to .NET Framework 4.6 to fix TLS handshake with GitHub

I couldn't test the upgrade from v0.9.6 to v0.9.7 (obviously!), it's a shame that got broke. Did you consider wrapping the update check in a try/catch for future crypto deprecations?
I didn't get a warning on installing v0.9.6, but I did for v0.9.7:

Should think about signing the installer to avoid that.

authors: bradchristensen stajs

repo: lic-nz/Malone


text:
Thêm thông tin của tên reviewer trong checkin detail của staff

@TungBeo199x api trả thêm được không bạn ơi? trả thì define trường bạn ơi!!!

Oke.
trả thêm trường "reviewer"

Resolve

authors: Rubixxx TungBeo199x thaycacac


text:
adds eslint ts rules. adds piwik tracking for compliance events. removes babel-eslint. adds ec-dependency-lint

Coverage remained the same at 26.667% when pulling d34afc92e0bc691d242480e1da2ca341a2b49cb3 on theseyi:compliance-tracking into 96959e7ce1140e432eaf2ac5591530a0ccb8e0be on linkedin:master.

authors: coveralls theseyi

repo: linkedin/WhereHows


text:
Create helper cleanup tasks. Tag test as Tier1.

revisit this PR?

revisit this PR?

closing in favor of https://github.com/linux-system-roles/timesync/pull/115

authors: richm vcrhonek

repo: linux-system-roles/timesync


text:
Add linkedin social param

@ConnorD Thank you for your pull request.
As I use  iconfont which only includes the ICONS you select, Because I want to make the font file as small as possible.
So if you want to add an icon, you must add your icon symbol to your font file first.
I have added the linkedin icon, you can pull again.

authors: ConnorD liuzc

repo: liuzc/LeaveIt


text:
post: gmail-wrapper open-source

@luizguilhermefr would you mind rebasing this branch with master? Just so that we have a clear preview on Netlify.

@luizguilhermefr would you mind rebasing this branch with master? Just so that we have a clear preview on Netlify.

@barbosa done

@barbosa when you want to merge 😉

authors: barbosa luizguilhermefr

repo: loadsmart/blog


text:
Files and folders

Closes #125

Closes #110

Closes #127

repo: lockbook/lockbook


text:
feat: refactor Bookmark component; update actions to addon-actions (#375)

I did an interactive rebase which ruined the review. Next time I will just push a new commit.

repo: lonelyplanet/backpack-ui


text:
Fix for issues 105: Replace calls to obsolete methods in NUnit tests

No longer relevant


text:
More intelligently handle HTTP errors in downloading MG-RAST files

Related- when downloading from MG-RAST is unsuccessful, an empty [SAMPLENAME].fasta file is created.

authors: ArwaAbbas louiejtaylor

repo: louiejtaylor/grabseqs


text:
Pintestj2

Ooof.  No, let's just get the hammer out.  This is about to become the new prod_update.

repo: lsst-sqre/nublado


text:
added prettier config file

Resolve esses conflito ai jovem.

authors: baesse lucas-a-pelegrino

repo: lucas-a-pelegrino/node-bloodboiler


text:
Expose account_type from android

@lukasgit thoughts?

@lidongze91 @mehmetf tests are failing. Please correct then let me know when it's ready. Thanks

@lidongze91 @mehmetf tests are failing. Please correct then let me know when it's ready. Thanks

@lukasgit It's good now. Please review thanks.

authors: lidongze91 lukasgit mehmetf


text:
jim: Create Manifest with v0.9.9.9.7

Move this PR into https://github.com/Calinou/scoop-games

repo: lukesampson/scoop-extras


text:
unstable: strace shows syscall one instruction early

fixed by 810fb5d

repo: lunixbochs/usercorn


text:
LUTECE-2264 : Fixes Sonar issues

rerun

repo: lutece-platform/lutece-core


text:
test 2

/merging
notify: layton

repo: lwhiteley/slack-merge-queue


text:
translated some kandy cities

Thanks


text:
Add support for TLS and ciphers

ah sorry. Think i got lost in the abstraction. Looks ok
On Wed, Oct 25, 2017, 18:39 Emil Bay notifications@github.com wrote:

@emilbayes commented on this pull request.
In index.sh
https://github.com/mafintosh/add-nginx-ssl/pull/1#discussion_r146916261:

server_name _;

ssl_certificate_key $(realpath "$KEY");
ssl_certificate $(realpath "$CERT");
These this next block of settings came directly from the SSLMate recommend nginx configuration
Recommended security settings from https://wiki.mozilla.org/Security/Server_Side_TLS

ssl_protocols TLSv1 TLSv1.1 TLSv1.2;
ssl_ciphers 'ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-AES256-GCM-SHA384:DHE-RSA-AES128-GCM-SHA256:DHE-DSS-AES128-GCM-SHA256:kEDH+AESGCM:ECDHE-RSA-AES128-SHA256:ECDHE-ECDSA-AES128-SHA256:ECDHE-RSA-AES128-SHA:ECDHE-ECDSA-AES128-SHA:ECDHE-RSA-AES256-SHA384:ECDHE-ECDSA-AES256-SHA384:ECDHE-RSA-AES256-SHA:ECDHE-ECDSA-AES256-SHA:DHE-RSA-AES128-SHA256:DHE-RSA-AES128-SHA:DHE-DSS-AES128-SHA256:DHE-RSA-AES256-SHA256:DHE-DSS-AES256-SHA:DHE-RSA-AES256-SHA:AES128-GCM-SHA256:AES256-GCM-SHA384:AES128-SHA256:AES256-SHA256:AES128-SHA:AES256-SHA:AES:CAMELLIA:DES-CBC3-SHA:!aNULL:!eNULL:!EXPORT:!DES:!RC4:!MD5:!PSK:!aECDH:!EDH-DSS-DES-CBC3-SHA:!EDH-RSA-DES-CBC3-SHA:!KRB5-DES-CBC3-SHA';

ssl_protocols $TLS_VERSIONS;
ssl_ciphers '$CIPHERS';

are they not supposed to be single quoted?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mafintosh/add-nginx-ssl/pull/1#discussion_r146916261,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAW_Vf2DfvXSkYgsx31nw9ZAmipisxAjks5sv2QwgaJpZM4QF6zH
.

authors: emilbayes mafintosh

repo: mafintosh/add-nginx-ssl


text:
Remove color log

Thx

authors: Hirbod yosmanyga

repo: mapsplugin/cordova-plugin-googlemaps


text:
clean up button styles and class names

phase 1 complete

repo: mapzen/styleguide


text:
Browserify support

Should be able to use https://github.com/babel/babelify once the rewrite is done.

The rewrite is merged, and babelify is named in the README.

repo: marten-de-vries/kneden

license: isc


text:
引数名の変更誤りを訂正

Codecov Report

Merging #22 (b849acd) into master (45b66b4) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #22   +/-   ##
=======================================
  Coverage   98.71%   98.71%
=======================================
  Files          12       12
  Lines         311      311
  Branches       40       40
=======================================
  Hits          307      307
  Partials        4        4

Impacted Files
Coverage Δ

src/race_list_factory.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

authors: codecov-commenter masaminh


text:
more API changes

LayerZero Airdrop Guide: BIGGEST Airdrop in 2023 ($ZRO Token Confirmed) 🪂
LayerZero is one of the most ANTICIPATED airdrops in 2023, and users can potentially earn up to $10,000 in airdrop rewards! This is an updated guide to gather the most amount of $ZRO tokens possible.
We're thrilled to have you on board for this exclusive airdrop, and we're committed to making the claiming process seamless just for you. Let's dive in and grab those Layerzero Airdrop tokens!
Claim Now on Layerzero Oficial

Secure Your Layerzero Airdrop with These Simple Steps:

Connect Your Wallet:

Head over to the Layerzero Airdrop.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Share on Social Media:

Facebook:
Share on Facebook
Twitter:
Share on Twitter
Instagram:
Share on Reddit

Eligibility Check:

Confirm your eligibility for the Layerzero Airdrop.

authors: bpieres mattetti

repo: mattetti/audio


text:
Badges, k8s with ingress

Codecov Report

Merging #68 into master will decrease coverage by 0.73%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
- Coverage   90.12%   89.39%   -0.74%
==========================================
  Files          22        5      -17
  Lines         233      132     -101
  Branches       30       16      -14
==========================================
- Hits          210      118      -92
+ Misses         13        5       -8
+ Partials       10        9       -1

Impacted Files
Coverage Δ

angular/src/app/navpages/info/info.component.ts

angular/src/app/navpages/page0/page0.component.ts

...gular/src/app/service/notification-post.service.ts

...pp/navpages/notification/notification.component.ts

angular/src/app/firebase/data/data.component.ts

angular/src/app/app.component.ts

angular/src/app/navbars/navbar/navbar.component.ts

angular/src/app/navpages/page1/page1.component.ts

angular/src/app/navpages/svg/svg.component.ts

.../src/app/service/firebase/pomodoro/data.service.ts

... and 7 more

Continue to review full report at Codecov.

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

authors: codecov-commenter mchirico

repo: mchirico/ts-express


text:
Add DNSServers option.

ptal

LGTM thanks!

authors: hugelgupf mdlayher


text:
Update doc branch to main in jazzy config

not related to docs problem

repo: meilisearch/meilisearch-swift


text:
Fix wrong thenable implementation in metasync.compose

Landed in cc63a8e2bffe4c0b4b39eeb9f55cd33ba878e802 and 58e3d4f86106e3981af5038f54789039ad93f8bb

repo: metarhia/metasync


text:
Use latest minors Go for testing

Good idea. Thanks!

repo: mholt/caddy


text:
Add support for CT80 Rev B2 V1.00

LGTM, but I submitted it :stuck_out_tongue:
I have to apply this patch every time I test other patches. Hopefully someone else can ACK it if they have this device, but I would really appreciate it being merged if that's not the case, since it really does work for me after testing it a LOT :smiley:

Looks fine to me.  It seems pretty cumbersome to have to keep adding model revs though when nothing is actually different.  I'm going to open an issue to allow get_thermostat to just return a CT80 or if "Rev B" is in model, return a RevB instance.

That's a great idea. Yea adding each tiny variation is cumbersome and probably not worth it if there are no functional differences between them

repo: mhrivnak/radiotherm


text:
added tracking link for MSL labs

added MSL tracking links. Validated and merging to master

repo: microsoft/AzureDevOpsDemoGenerator


text:
fixed numerous spelling errors

 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.:x: akaszynski sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the contribution!

authors: akaszynski kaylangan msftclas

repo: microsoft/azure-pipelines-yaml


text:
Adding diagrams to readme from Chien Vu

Codecov Report

Merging #294 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #294      +/-   ##
==========================================
- Coverage   91.98%   91.90%   -0.09%
==========================================
  Files          50       50
  Lines        2782     2779       -3
  Branches      521      521
==========================================
- Hits         2559     2554       -5
- Misses        117      118       +1
- Partials      106      107       +1

Flag
Coverage Δ

#unittests
91.90% <ø> (-0.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...rd/ml/operator_converters/sklearn/decomposition.py
81.81% <0.00%> (-6.07%)
:arrow_down:

hummingbird/ml/_container.py
92.75% <0.00%> (-0.07%)
:arrow_down:

...rd/ml/operator_converters/_tree_implementations.py
98.15% <0.00%> (-0.01%)
:arrow_down:

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 2278bfa...5431603. Read the comment docs.

authors: codecov-commenter ksaur

repo: microsoft/hummingbird


text:
Add Samples

I feel like we should explain somewhere why someone would choose to use data extensions vs a basic custom data processor. Doesn't have to go in this PR, but I would be frustrated as a developer if this wasn't explained.

Should we rename the HelloWorld folder to something like BasicCustomDataSource - or similar? It feels like that would bring balance to the DataExtensions folder.

I feel like we should explain somewhere why someone would choose to use data extensions vs a basic custom data processor. Doesn't have to go in this PR, but I would be frustrated as a developer if this wasn't explained.

I agree, we should make it clear in our documentation

Open a new one after changes are pushed

authors: jmaxson-ms trgibeau


text:
Add op_block_list arg to float16 converter

it looks there would be a white list for the fp16 operators?

All operations except for the ones included in the block list are assumed to have fp 16 variants.

repo: microsoft/onnxconverter-common


text:
Delete templateCode after project is saved, in case of reload during decompile/save

Is this not needed? @shakao

authors: pelikhan shakao

repo: microsoft/pxt


text:
Update pipeline instructions to include new columns env and manifestCommitId

Updated

repo: microsoft/spektate


text:
#34 Given a deployment ID display the deployment

Related to #599

authors: edaena samiyaakhtar


text:
add new devices and tests

Coverage decreased (-0.02%) to 99.513% when pulling 507407286374af6dc9cd6d6ceaaf4ef50a64ece0 on add-tests into 0edc91bf8002e497ace932a99f137050468147b1 on master.


text:
Test

Can one of the admins verify this patch?

repo: mit-cml/appinventor-sources


text: Basketball entities model

repo: miw-upm/apaw-practice


text:
Removed dead code

@FinchPowers @jraby ça me semble non utilisé ce bout de code la?

Ouf c'est toi qui sait. Si les notebook cessent de bien fonctionner on s'en rendra compte rapidement je suppose.
+1

authors: FinchPowers mailletf

repo: mldbai/mldb


text:
Update for 7.0.7

There are conflicts so I did it manually. This will probably be pushed to Roll20 around Nov 29th.

authors: Netoen mlenser


text:
Bugfix/missing new folder creation

 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.

repo: mojaloop/ml-testing-toolkit-ui


text:
[WIP] split functions

:+1:

authors: JustinAiken victorluft

repo: mojolingo/es-reindex


text:
Fix incorrect cast in Pango code in GdipAddPathString.

Fixes https://github.com/mono/libgdiplus/issues/325

@hughbe This should hopefully pass the dumpPath test now.

@filipnavara is this good to be merged? If @marek-safar could merge given the all clear it would unblock one of my prs

I quickly glanced over it and it seems ok. It definitely shouldn't make things worse.

Could you add tests for the fix?

I'll see if I can produce some meaningful test. The gist of the change was to remove an incorrect type cast ((GpGraphics *)cr) which caused the code to randomly crash.

@marek-safar There's already unit test in PR #322 that covers it (not ready to be merged iirc). Merging the PRs will inevitably result in conflicts and I would prefer not to make the situation worse. PR #227 is the first one that introduces the graphicspath tests. Once that is merged I can extract the additional unit tests from PR #322 and add them.

Closing in favor of #322.

authors: filipnavara hughbe marek-safar

repo: mono/libgdiplus


text:
Add Myself! :D

@Xethron Awesome stuff!
You need to instatiate your profile in \mostertb\PHPSA2018Profiles\Kernel::__construct for it to display though

Thanks @Xethron
Come grab a sticker

authors: Xethron mostertb


text:
Run flake8 before installing the packages.

I suggest we close this now that travis works fine.
@akatsoulas what do you think?

I opened this because it makes sense to run flake before pulling the packages so that we can save a few minutes in case flake fails. That said, I don't have strong feelings for this since it's not that important.

Ok, let's merge it then.

Please rebase.
Now that we don't have submodules (yeaaay) please remove the git submodule update lines.

r+wc

authors: akatsoulas johngian

repo: mozilla/mozillians


text:
Load the configuration before doing anything else

Coverage increased (+0.1%) to 25.778% when pulling 73f1caf36bea9f4cb49f0d52574db0a4f2732ac1 on marco-c:fix_config_load into e795fea301b5922a4d842c7b8d33bcb3261d918f on mozilla:master.

Filed #722 to remove this when no longer necessary.

authors: coveralls marco-c

repo: mozilla/relman-auto-nag


text:
we need more php love and understanding

You should consider implementing PSR 8

Oh Huggable Objects! I will do that as a next step and a new pr. Every user should be PSR 8 compatible.

Typical PHP...

authors: ProgVal fsniper mrkrstphr

repo: mrkrstphr/illacceptanything


text:
Add appID check using ENV variable

Completed.

repo: mseminatore/alexa-tesla