Catch Babel parsing errors

When I'm editing .js files with the Stylelint plugin in VSCode, I get spammed with popup error messages from babylon.
For example:
SyntaxError: Unexpected token, expected ;(50:12) at Parser.pp$5.raise (/home/hugo/projects/liber/liber-app/node_modules/babylon/lib/index.js:4454:13) at Parser.pp.unexpected (/home/hugo/projects/liber/liber-app/node_modules/babylon/lib/index.js:1761:8) at Parser.pp.semicolon (/home/hugo/projects/liber/liber-app/node_modules/babylon/lib/index.js:1742:38) at Parser.pp$1.parseVarStatement (/home/hugo/projects/liber/liber-app/node_modules/babylon/lib/index.js:2170:8) at Parser.p

My .stylelintrc:
{
  "processors": ["stylelint-processor-styled-components"],
  "extends": [
    "stylelint-config-standard",
    "stylelint-config-styled-components"
  ],
  "syntax": "scss"
}

Another user had the same problem here: https://github.com/shinnn/vscode-stylelint/issues/89
According to the author of the plugin, it seems that it's a bug with the processor (hence why I made this issue in this repo). He says it's because this processor does not adhere to the original stylelint behaviour to syntax errors as described here:

stylelint.lint() does not reject the Promise when your CSS contains syntax errors.

I'm completley new to Stylelint and I don't really know how it works, so this is where my research stopped. If anyone has some insight, that would be extremely helpful.

/cc @orta you're using this setup, right?

Yep, and a fail generally works fine for us:

The offsets are usually wrong, but we have a complicated babel setup running typescript first, so we just deal with it

Hmm interesting, so it's gotta be something in your local setup @adrianmcli. I don't know what this means or what we should have to change:

stylelint.lint() does not reject the Promise when your CSS contains syntax errors.

Note: I think the wrong offsets are due to #128 (see failing test in #129), any help getting that fixed appreciated too!

Form the sounds of it, the issue is that an exception is being raised in the babel parser that isn't being caught and returned as an object with the right formatting to say there is an issue. I don't think they ever expect you to be throwing an error, which seems like a reasonable assumption.

@orta
So are you not getting the popups that I am getting? And you are using the same vscode plugin with comparable configuration?
Also, would you have any idea as to why the author of the plugin thinks it's a bug with the processor? I'm just trying to find a solution here.

Nope, never seen popups like that - and yeah, I added the support for that extension: https://github.com/shinnn/vscode-stylelint/pull/82
I think it's this: an exception is being raised in a babel parser in this lib that isn't being caught and then returned as an object with the right formatting to say there is an issue. I don't think they ever expect you to be throwing an error, which seems like a reasonable assumption

Okay @orta so you reckon this issue should for example be renamed to "Errors not being caught correctly" and we should add a .catch statement somehwere or something of the likes?

Yep 👍

For what it's worth I have the same issue. The moment I start editing a JS file I get a deluge of alerts. It makes this plugin unusable unfortunately as my only option is to remove Javascript from stylelint's list of types.


Terragrunt should detect if backend configuration is missing from Terraform code

As of Terraform 0.9, you have to declare a backend in your Terraform code, or remote state won't work:
terraform {
  # The configuration for this backend will be filled in by Terragrunt
  backend "s3" {}
}

This is very easy to miss. See #211, #207, and #183.
If the user specifies a remote_state configuration in their Terragrunt configuration, Terragrunt should scan the user's .tf files and report an error if a backend block is not found. This could be done using an approach similar to how we look for modules: https://github.com/gruntwork-io/terragrunt/blob/master/cli/cli_app.go#L254

Thanks @brikis98 man you just saved my day. 💯

Please include this, soon.  I just unwittingly deleted my local state files thinking that I had properly setup remote state.
Thankfully, I was able to recover via backups.

This should be fixed by #302. New binaries will be in https://github.com/gruntwork-io/terragrunt/releases/tag/v0.13.5 shortly.


Make it possible to construct a Response

add Response::new and Response::with_data
Use the new methods for Response

context: https://github.com/yuniruyuni/MrDamian/commit/6817aa8340e542b9539a18595717926d9f3a5647#r112204261

going to try this with #359 to see that the merge queue is working, this pr should fail when batched with #359

hmm, skipped = success....

solved with #372


Friend can't connect to my server and I can't connect to his

My friend told me he set a server up using this code and got it to work with him in the game, but when he sent me the IP with the port I was unable to connect. So I tried setting it up and got it to work as well with my own server, then he tried connecting to me and it wouldn't work. Is this an issue with us being too far apart or what (I'm in NY and they're in VA).

On the router connected to your internet provider, have you setup port forwarding of TCP 25565? If not, you’ll need forward that to the local IP address of the computer where you’re running Docker.

That worked, thanks


makeSum and number formatting fixed

In the number formatting, a comma is used instead of a period, and a period is replaced by a comma.
Fixed issues with makesum function.


customize checks

Codecov Report

Merging #195 (f7a927e) into main (6e14b59) will increase coverage by 0.07%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #195      +/-   ##
==========================================
+ Coverage   74.30%   74.38%   +0.07%
==========================================
  Files         131      131
  Lines        7481     7502      +21
==========================================
+ Hits         5559     5580      +21
  Misses       1613     1613
  Partials      309      309

Flag
Coverage Δ

unittests
74.38% <100.00%> (+0.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

checker/check-response-success-status-removed.go
82.05% <100.00%> (+6.18%)
:arrow_up:

checker/default_checks.go
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Fixes https://github.com/Tufin/oasdiff/issues/194


scrabble-score: add alphabet completeness test

After the discussion in #509 , here is proposed addition to the canonical data.
Feel free to pick nits!
If we really wish to stick to "proper, scrabble-approved word", I could make multiple cases for
"the"
"quick"
"brown"
"fox"
"jumps"
"over"
"lazy"
"dog"
But in my personal opinion that obscures the intended purpose. The complete alphabet is the shortest way to express the complete alphabet, and thereby makes the intent the clearest. Anything else just muddles the waters (IMHO).

Thanks for the quick review!

LGTM

Thanks!

you're welcome! Thanks back for all the feedback!


vue.js:596 [Vue warn]: Error in beforeDestroy hook: "TypeError: Cannot read property 'filter' of undefined"

https://github.com/KoRiGaN/Vue2Leaflet/blob/bd7709d3b7929f4306e279ce5c5e709d8f49e48a/src/components/LMap.vue#L171
layerToAdd is null on calling removeLayer.
I do not see where layerToAdd is defined

Closing this since the issue is resolved and merged, release coming soon

How can this problem be solved?


Fixes #1208 - by force updating nghttp2-libs

Checklist

[x] Affected Issues have been mentioned in the Closing issues section
[x] Documentation has been written/updated.
[x] Changelog entry has been written

Explain the details for making this change. What existing problem does the pull request solve?
Changelog Entry
Bugfix - Force installing nghttp2-libs as the upstream image shipped an outdated version of the library
Closing issues
Closes #1208

I'd let the tests run trough and check manually if we're on version 1.38.0-r0 after that - ship it!


Update Readme of Jaguar Car Project

I'm always frustrated when I don't see Proper readme so ,I will update the readme in order to ease the idea for everyone to find the new project requests.
I want to update the readme file of Jaguar Car Project

@TusharKesarwani Kindly assign this to me under SSOC 2.O

@TusharKesarwani Kindly assign me this under SSOC 2.O


drivers: sdhc: Cadence SDHC driver bug fixes and other improvements

Internal Clock not getting stable during initialization
SD clock was not enabled
Removed duplicate file.
Removed write to read only registers
Moved function pointer structure from the data section to the RO section
Wrong macro names
Refactoring of lower layer header file.

Signed-off-by: Tanmay Kathpalia<EMAIL_ADDRESS>

@decsny @danieldegrasse Kindly review and provide your valuable feedback.

Some duplicate files were added in 64036 which had been taken care of in this PR.

@roymurlidhar can you take a look at this PR as well? Changes look ok to me but would appreciate a look from someone familiar with the IP itself

@tanmaykathpalia do you know if @roymurlidhar will be able to review these changes? I'd appreciate confirmation from them that everything looks good here, since I am not familiar with the Cadence IP

@tanmaykathpalia do you know if @roymurlidhar will be able to review these changes? I'd appreciate confirmation from them that everything looks good here, since I am not familiar with the Cadence IP
@danieldegrasse I spoke with @roymurlidhar, and he mentioned that he would try to review it tomorrow. Also, he left Intel, so he won't have access to IP documents anymore. Let's wait for tomorrow.

@decsny thanks for your time and approval.

Hi, I have reviewed the changes and it looks fine for me. We can go ahead and approve this PR. Thanks :)

@decsny can you approve again and merge the PR.

i dont have merge rights


Cardinality not mentioned in Service Bus Trigger

This page doesn't mention the cardinality attribute to allow messages to be received in batches.  This was added here.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 1d50399c-0873-e903-35ca-1845c84b6c6c
Version Independent ID: 1a3150b4-5eb9-0ac5-ee2b-d1bb6b6fd63e
Content: Azure Service Bus trigger for Azure Functions
Content Source: articles/azure-functions/functions-bindings-service-bus-trigger.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@SiobhanBaynes Thanks for the feedback. We have assigned the issue to the author for review and will update as appropriate.


IBM-314 as a user I would like some sort of indication that my request is being handled when I request a report so that I know its working

(cherry picked from commit 515d396d20c4dbf276a4bb24bc42f73108c53593)

e2e test?

I couldn't reliably since in test I found that the button goes between enabled and disabled faster than I could assert it happened


docs(id): update CONTRIBUTING-id.md for correcting small grammar mistakes in Bahasa

What does this PR do?
Improve minor grammatical errors in Bahasa
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

@adityarizkyramadhan, @HWinsen, requesting help reviewing PR in Bahasa.


Don't merge empty Rubocop configurations

This is a behavior Rubocop has, Hound should not be any different.
Changes:

Refactor: Move ensure_correct_type to Config::MY_CONFIG#content.
The #parse method should not do more than just parsing.

Resolves #983

One small comment otherwise looks fine to me.


Pagination

Within certain Finders, we don't want the documents per page to be all of them (or 1000). This PR allows us to specify a documents_per_page as part of the content item and have Finder Frontend build the pagination query within the SearchQueryBuilder and render the Previous and Next component. Ticket. Most of the work was completed by @dsingleton. I renamed some things and added tests.
Screenshot:

Tests will go green when https://github.com/alphagov/govuk-content-schemas/pull/91 is merged.

I've not had a proper look yet, but I don't think pagination will stay in sync with (leaving inaccurate pagination UI), which'll be confusing for users. IMO thats a blocker for merging this.

AJAX/pagination aside I think it looks good. Although someone else should review as I did a bit of work on it.

Closing until we are able to do the pagination through AJAX.


Fix documentation of re-exports that have dedicated doc comment

Fixes #65.
Rustdoc's behavior for doc attributes on pub use items is to concatenate the attributes from the pub use with the attributes from the original item. It looks like that is not what was intended in the documentation in postcard.
In this PR I have converted all pub use that had /// doc comments on them, into functions. This way they get to have their own self-contained documentation.
Before:

After:

Closed in favor of #107 as I didn't have permission to amend the PR. Thank you @dtolnay!


flutter tool: PowerShell executable detection added, fixing usage of PS version >=6

PowerShell executable detection added, fixing usage of PS version >=6
Description
The name of the PowerShell executable has changed from version 6 (PS core).
PowerShell version <= 5: PowerShell.exe
PowerShell version >= 6: pwsh.exe
The flutter.bat command line script currently requires the PowerShell executable to be PowerShell.exe. This PR introduces a lookup for an available executable.
Old behavior:
The script fails if PowerShell.exe is not in the PATH, even if pwsh.exe is available:
The system cannot find the path specified

New behavior:

with pwsh.exe and PowerShell.exe on the path, pwsh.exe is used
with only pwsh.exe on the path. pwsh.exe is used
with only PowerShell.exe on the path, PowerShell.exe is used
without pwsh.exe on the path, the script fails with the following error message:

Error: PowerShell executable not found.
       Either pwsh.exe or PowerShell.exe must be in your PATH.

Related Issues
https://github.com/flutter/flutter/issues/53135
https://github.com/flutter/flutter/issues/55491
Tests
I tested manually:

with both pwsh.exe and PowerShell.exe on the path
with only pwsh.exe on the path
with only PowerShell.exe on the path
without either pwsh.exe or PowerShell.exe on the path

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

Change seems reasonable to me, fyi @zanderso @goderbauer


Added NRIC, Organization and HealthWorker class. Made changes to Person class.

Do update the testcases for the Person class, and the PersonBuilder class.
Modification of the PersonBuilder class also has to be done to acocunt for the NRIC field.
Adding NRIC to the Person object requires quite a bit of refactoring of the other classes and tests - which will affect the upcoming features that we will be working on as well. If this is not a priority right now, we can consider implementing the NRIC field in the subclasses first (i.e HealthWorker and Patient)


webp support

GDI+ should support WebP and heic (assuming you paid $1)... but I can't figure out how to get it working.
Notes:

It may have something to do with setting the GDI+ version to 2.
Convert from WIC and back.

Solved with 85c68728ab75861760973e5aec365996a6e8fd73


[Houskeeping] The mailing list archival is broken

Following the upgrade from debian8 to debian9 on edna-site the location of the repos has changed on the server.
Now lurker does no more find its template for generating pages.
I believe the archival is still working, it is just the web rendering of the archive which is broken.

I'll try to do this during the CC

Back again ...
What I did: uninstall+purge lurker, reinstall, copy database files.
http://www.silx.org/lurker/splash/index.en.html


Download page content

Adding a .zip on the URL (like dontfile.com/my-page.zip), Dontfile must download the page content (as .txt) and the file on a zip. We can use the rubyzip gem to achieve this!

Hello @MatheusRich 👋🏻,
Love this project! May I work on this issue? Will send a PR and refer
this issue 🙂

@wilbertliu It's yours! Thank you!

@wilbertliu If you want to take a look, dontpad has this feature too!

@MatheusRich That would be nice, Matheus! But I think the link is incorrect, no? 🤔

It's okay to me :disappointed:. Anyway, its http://www.dontpad.com

Sure, no problem at all! 😉
Sincerely,
Wilbert Liu

On 9 Oct 2018, at 9:52 PM, Matheus Richard<EMAIL_ADDRESS>wrote:
It's okay to me 😞. Anyway, its http://www.dontpad.com
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Closed via #68

Closed via #68


does this function work with meters and polar coordinates?

The current collide-and-slide-2d function has always been used in the context of cartesian coordinates with pixel units.
I need to validate if using meters and polar coordinates are viable with this repo.

Yep, it works fine!


Rebase monitoring to current latest upstream

Rebase rancher-monitoring to bring it up to date with the upstreamn. This will make is easier to make smaller and more manageable rebases in the future.

blocked by https://github.com/rancher/image-mirror/pull/687

Regression test post Rancher-monitoring rebase was successfully completed.
Details :
Rancher : v2.10-head
cluster : v1.30.2+k3s2
charts repo : dev-v2.10
monitoring : 105.1.0-rc1+up61.3.2


No release notes for 1.4

I can't seem to find the release notes for v1.4 on the docs page.
https://cellrank.readthedocs.io/en/stable/release_notes.html

Actually it's on the latest version (not default):
https://cellrank.readthedocs.io/en/latest/release/notes-1.4.0.html

Actually it's on the latest version (not default):

Yes, previously (by design), they were only shown on the index page here.
Will hijack this issue to figure out how to use towncrier to have even the smallest news fragments rendered in the docs.

@adamgayoso, we're in the process of switching to towncrier to automaticaly render our release notes from the set of PRs associated with a particular release. @michalk8 is taking care of that.

closed via #708 - #712 (seems to finally work).


Focus Domain and Strategies Selection

Changes
Allow mobile users to select 1-3 focus domains and then choose up to 4 corresponding strategies.
Adds domains and strategies to firebase
Adds focusDomains and chosenStrategies to userState in firebase
Requirements
Requires: must be up to date with #28.
PR Dependencies
Depends on: #28
Merged Branches and Other Dependencies
Merged branch(es):
Other dependencies:
Subtasks / Progress

[x] Implement UI
[x] Integrate with firebase
[x] Fix bugs
[x] Replace 'key' icon in StrategiesDetailsView with correct icon for domain
[x] Fix bug with strategies card checkbox becoming unchecked when navigating back to ChooseStrategiesView
[x] Refactor? strategies card and iconForDomain()

Preview

https://user-images.githubusercontent.com/71405899/120542141-990e4e00-c39f-11eb-9a48-8c2d35b25abf.mp4
Testing
Manual

@MarkDavies0 I applied the fixes from the git merge method. Can you review changes and briefly test?


Multiple Vulnerabilities

Hello,
Multiple vulnerabilities have been detected in the Docker image of OtterTune Agent of severity Critical and High. Would it be possible to update the folllowing packages to their latest versions or remove them if not used ?

dpkg
libtasn-1.6
glibc
pcre2
openssl
wheel
gnutls28
werkzeug
krb5
ipython
mysql-connector-python
xz-utils
gzip

Thank you in advance,
Best,
L.

Hi L.
Thanks for the suggestion! We have updated our dependencies.
Ruogu


Update VisualChallengePage.xaml

Result
IOS Implementation

Android Implementation:

I would like to see
Frame should have ability to maintain custom left right top bottom corner radius.
What i loved most
Use of Collection View With Visual. It's was awesome moment using for first time. 👍  💯 Thank's Xamarin Team.


fix typo

Hi @anders94 , I love your demo. I find there is tiny typo here. I thought the coinable shouldn't change throughout the chain. Please correct me if I am wrong :P

No, I think that last block can go to Sophia - that was intentional. Also, altering this will alter the hash so we'd also need to pick a good nonce as well.
Thanks for contributing but I'm afraid I have to close this.


CHECKOUT-1234: Expose translations object

What?
Expose a translation object on the checkout page.
Please note that this PR can only be merged after merging and releasing https://github.com/bigcommerce/paper/pull/103.
Why?
So it can be used by UCO to display non-English strings.
Testing / Proof

@bigcommerce/checkout @mcampa @lord2800

We are now using CHANGELOG.md. Please add an entry with your change under draft before merging
## Draft
- <here>

## 1.5.1 (2017-02-07)
- ...

💚


feature request: ignore "fix #NNNN" notes when the referenced issue is tagged as a regression

I don't know if this request belongs here and/or should be generalized in some way to apply to projects other than rust-lang/rust ... but here goes:
A recent bug made me realize that a pretty common process failure we encounter in Rust is the following:

Bug with issue number #NNNN is discovered to affect either the stable or beta channel of Rust.
Bug is tagged as a corresponding regression, and often given high priority.
A fix is developed and a PR posted, against the master branch of the repo, that says Fix #NNNN. The PR is reviewed, accepted.
Bors lands the fix on master, and closes the issue. But the work isn't done, because we still need a beta backport.

I believe that so far, we've been relying on people to either 1. not tag PR's that are fixing such regressions with Fix #NNNN, or 2. re-open the regression issue after the PR lands. But muscle memory is hard.
I think it would be better to make bors itself not close the issue when its tagged as a stable/beta regression if the fix is being merged into the master branch. Instead we should require more human intervention to evaluate whether the issue is truly resolved and should be closed.
(As a special case, we might allow bors to still close such issues when the corresponding PR is actually going into the right branch, i.e. stable or beta. But it wouldn't be the end of the world if that special case did not get implemented, IMO.)

It was pointed out to me that github is the one closing the issues; I misinterpreted the message "bors closed this in #54199 on Sep 26" on the web interface as describing a direct action of bors, but its just a side-effect of bors landing a commit.
It was also suggested that we could still perhaps have bors deal with this for us, by e.g. having bors automatically reopen such issues after github closes them...


Create dataset loader for PhoNER_COVID19

Dataloader name: pho_ner_covid/pho_ner_covid.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?pho_ner_covid

Dataset
pho_ner_covid

Description
A named entity recognition dataset for Vietnamese with 10 newly-defined entity types in the context of the COVID-19 pandemic. Data is extracted from news articles and manually annotated. In total, there are 34 984 entities over 10 027 sentences.

Subsets
word_level, syllabel-level

Languages
vie

License
Unknown (unknown)

Homepage
https://github.com/VinAIResearch/PhoNER_COVID19/tree/main

HF URL
-

Paper URL
https://aclanthology.org/2021.naacl-main.173.pdf

#self-assign

Yep working on this issues got busy with some things but will try to wrap this issue by next week.

Okay then, @bp-high. Feel free to let us know if you need any help!

#self-assign

Hi @bp-high, is there any update on this dataloader? If not, @dovanquyet is ready to take up the issue.

Hi @bp-high, may we know the update on this dataloader issue? It's been 3 weeks since the last poke from the SEACrowd stale-checker, and we might consider unassigning if there's no progress update in the next 24 hours.

#self-assign


Lambda expressions

It seems that using lambda expressions is breaking some things.  For example, if I add this to a class:
Runnable breaking = () -> {};
then it won't be found when using something like reflections.getSubTypesOf(Object.class)

Looks like updating  javassist to version 3.22.0 GA fixes this!
Thanks for the tip from:
https://github.com/ronmamo/reflections/issues/150#issuecomment-348549645


"Poienarii Burchii" is listed in the drop-down menu

Description:
"Poienarii Burchii" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Poienarii Burchii" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Poienarii Burchii, PRAHOVA".


syncing with v0.15.0 of Firecracker

Two changes were required in updating to v0.15.0 in that the first changed some values to be pointers due to allowing them to have default values.
There is now validation that the drive name has to be alphanumeric and underscores. Due to this the root drive name, root-drive, has been renamed to root_drive.
Also fixes a bug where StopVMM was racing in TestMicroVMExecution and would sometimes succeed because the defer would happen last and do nothing with the error. However, if the defer happened before the TestStopVMM, then that's when the error is captured and fails.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Please rebase this so that it tests Firecracker v0.15.0.

TestJailerMicroVMExecution seems to be failing consistently:

=== RUN   TestJailerMicroVMExecution
time="2019-02-26T22:22:24Z" level=debug msg="Called NewMachine()"
time="2019-02-26T22:22:24Z" level=debug msg="Called Machine.Start()"
time="2019-02-26T22:22:24Z" level=debug msg="Marking Machine as Started"
time="2019-02-26T22:22:24Z" level=debug msg="Running handler validate.JailerCfg"
time="2019-02-26T22:22:24Z" level=debug msg="Running handler fcinit.StartVMM"
time="2019-02-26T22:22:24Z" level=info msg="Called startVMM(), setting up a VMM on /tmp/jailer-test221159510/firecracker/b/api.socket"
time="2019-02-26T22:22:24Z" level=debug msg="VMM started socket path is /tmp/jailer-test221159510/firecracker/b/api.socket"
time="2019-02-26T22:22:24Z" level=debug msg="Setting up signal handler"
time="2019-02-26T22:22:27Z" level=warning msg="Failed handler \"fcinit.StartVMM\": Firecracker did not create API socket /tmp/jailer-test221159510/firecracker/b/api.socket: context deadline exceeded"
--- FAIL: TestJailerMicroVMExecution (3.04s)
    machine_test.go:246: Failed to start VMM: Firecracker did not create API socket /tmp/jailer-test221159510/firecracker/b/api.socket: context deadline exceeded

Note that https://github.com/firecracker-microvm/firecracker-go-sdk/issues/73 is making retries more difficult to run in CI.

Fixed


specify completeadd gate

Specify the complete add gate of Kimchi

btw I turned the comment that were already there into spec comment, and added the constraints that I got from printing the LaTeX. I think it could use some love, but there's already some content to build on top of :)


'import_lz_ts.default' is undefined

I have implemented reverse geocoding using latitude and longitude in my react + vite based web app. I got an error when I had just imported getNearestCity() like below:
Uncaught TypeError: Cannot destructure property 'decompressFromURI' of 'import_lz_ts.default' as it is undefined. at offline-geocode-city.js?v=a5f8ead6:947:26

How to solve it sir?

I also have installed lz-ts library but still didn't solved

Hi @Vidi005
okay, let me check. Could you copy & paste how and where you imported offline-geocode-city quickly? Only 4-5 lines of code where and how you use the lib would be enough for me to analyze and reproduce the behaviour better.
Thanks and best,
Aron


Documentation on how to integrate

I would like to integrate wallet-core into my iOS app. Where can I find the documentation on how to integrate it - besides the Cocoapods installation?

We don't have such a documentation yet, and it depends on which feature do you want to implement, you can check out the test cases under swift folder now


Support return-X-after-YK for downloads

The Retry Test API currently supports return-X-after-YK for uploads. To expand testing scenarios in the retry conformance tests, we want to add support in the Retry Test API to return-X-after-YK of downloaded data.

Closing per discussion.


Failed to pull new customized template

There is no detailed docs for customize a template step by step. I made a custom template modeled after java8 template.
I follow the contribution guide to install my own template, an error was encountered:
[root@k8s-master openfaas]#  faas template pull https://github.com/kylin1994/openfaas-springboot-tmp
Fetch templates from repository: https://github.com/kylin1994/openfaas-springboot-tmp at master
2020/05/30 15:11:53 Attempting to expand templates from https://github.com/kylin1994/openfaas-springboot-tmp
Error while fetching templates: can't find templates in: /tmp/openFaasTemplates817545317

Your Environment

Docker version docker version (e.g. Docker 17.0.05 ):

Client: Docker Engine - Community
 Version:           19.03.9
 API version:       1.40
 Go version:        go1.13.10
 Git commit:        9d988398e7
 Built:             Fri May 15 00:25:27 2020
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.9
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.13.10
  Git commit:       9d988398e7
  Built:            Fri May 15 00:24:05 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.2.13
  GitCommit:        7ad184331fa3e55e52b890ea95e65ba581ae3429
 runc:
  Version:          1.0.0-rc10
  GitCommit:        dc9208a3303feef5b3839f4323d9beb36df0a9dd
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

Are you using Docker Swarm or Kubernetes (FaaS-netes)?

Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-16T11:56:40Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-16T11:48:36Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}

faas version:

CLI:
 commit:  f7c29ea19b5df9d7aa87e9c70aacf4d9315da2cd
 version: 0.12.4

Gateway
 uri:     http://<IP_ADDRESS>:31112
 version: 0.18.17
 sha:     18f6c720b50db7da5f9c410f9fd3369ed7aff379
 commit:  Extract a caching function_query type

Provider
 name:          faas-netes
 orchestration: kubernetes
 version:       0.10.5
 sha:           9be50543b372381a505e9e54a1356bb076c8f01f

I was fixed this issue.
The structure of the template project must be like this:
your-templates-repo
└── template
└── your-language


EAS build fails from nx-monorepo due to missing dependencies, yet hook that should fix package.json runs without issues

Build/Submit details page URL
https://expo.dev/accounts/thomas.rudfoss/projects/nx-expo-test/builds/6d89456f-b009-4a08-add9-ac1cb8cabeb0
Summary
Created a new nx monorepo with the official expo plugin. The app runs fine during development, but eas build fails apparently because dependencies cannot be properly installed.
Nx comes with pre and post install hooks that should update the package.json file to include the necessary dependencies, but based on the Read package.json step output this does not seem to take effect even though the pre/post hook steps report no issues.
Managed or bare?
managed
Environment
  expo-env-info 1.2.0 environment info:
    System:
      OS: Windows 11 10.0.22635
    Binaries:
      Node: 20.9.0 - C:\Program Files\nodejs\node.EXE
      Yarn: 1.22.21 - ~\AppData\Roaming\npm\yarn.CMD
      npm: 10.1.0 - C:\Program Files\nodejs\npm.CMD
    IDEs:
      Android Studio: AI-231.9392.1.2311.11076708
    npmPackages:
      @expo/metro-config: ~0.17.3 => 0.17.6
      babel-preset-expo: ~10.0.0 => 10.0.1
      expo: ~50.0.3 => 50.0.11
      react: 18.2.0 => 18.2.0
      react-dom: 18.2.0 => 18.2.0
      react-native: 0.73.2 => 0.73.2
      react-native-web: ~0.19.9 => 0.19.10
    Expo Workflow: managed

Error output
No response
Reproducible demo or steps to reproduce from a blank project
Fully reproducible repo here: https://github.com/rudfoss/nx-expo-test

Issue open on nx: https://github.com/expo/eas-cli/issues/2267

It appears that the .npmrc file is required for the installation on EAS to run correctly. Once I copied the .npmrc file to the nx-expo-test root folder the build works.
This seems to be an issue with the nx-generator so maybe this issue can be closed, but it seems that the build agent output for the Read package.json step should respect that the pre-install hook changed the file which it currently does not seem to do.
.npmrc
strict-peer-dependencies=false
auto-install-peers=true
node-linker=hoisted

apps/nx-expo-test/package.json
Add  && cp .npmrc apps/nx-expo-test to the end of the eas-build-pre-install script so that it is also copied along with the lock file.
{
  "name": "nx-expo-test",
  "version": "0.0.1",
  "private": true,
  "dependencies": {
    "@testing-library/jest-native": "*",
    "@testing-library/react-native": "*",
    "metro-config": "*",
    "react-native": "*",
    "expo": "*",
    "react-native-svg": "*",
    "react-native-web": "*"
  },
  "scripts": {
    "eas-build-pre-install": "cd ../../ && node tools/scripts/eas-build-pre-install.mjs . apps/nx-expo-test && cp pnpm-lock.yaml apps/nx-expo-test && cp .npmrc apps/nx-expo-test",
    "eas-build-post-install": "cd ../../ && node tools/scripts/eas-build-post-install.mjs . apps/nx-expo-test"
  }
}

Hey, I think you are right, the build agent output for the Read package.json step should respect that the pre-install hook changed the file 👍

Should be fixed now

@szdziedzic same is happening for projects generated with nx/react-native preset


Changes for Java 8 and the Buildservers

We properly close the HttpURLConnection between App Engine (the App
Inventor server) and the buildserver infrastructure. If we don’t App
Engine sends RST packets which causes issues for some buildserver
infrastructures.
Also limit project size to 10MB up-front (rather then after an
IOException is thrown). Add reporting to the buildserver status page to
report the number of Java threads.
Change-Id: I934df537dfe9419f5f8effafe80f0a2460091cdd

@ewpatton @halatmit @SusanRatiLane NOTE: This code is currently already in production. Also note, do not merge. Approve the review (if that is your inclination). But let me do the final merge in order to preserve the digital signature on the change.


v0.11.0-beta.0 regression: --enable-certificate-owner-ref=true does not set Secret owner

Describe the bug:
I deployed cert-manager v0.11.0-beta.0, then added the --enable-certificate-owner-ref=true option to the controller arguments. The Secret does not have metadata.ownerReferences set. When I create a Certificate and then delete it, the corresponding Secret is not deleted.
Expected behaviour:
The Secret has metadata.ownerReferences set, and it is deleted with the owning Certificate.
Steps to reproduce the bug:
Install cert-manager v0.11.0-beta.0 and add the --enable-certificate-owner-ref=true option to the controller arguments. Then create a Certificate and delete it again.
Anything else we need to know?:
Environment details::

Kubernetes version (e.g. v1.10.2): v1.15.4
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): DigitalOcean
cert-manager version (e.g. v0.4.0): v0.11.0-beta.0
Install method (e.g. helm or static manifests): static manifests

/kind bug

It appears the new CertificateRequest based Certificate controller implementation does not correctly honour this flag.
/priority critical-urgent
/milestone v0.11

/assign


APK

Hello, I used your resource pack, I would like to ask how to solve this bug
Trying to install Instant Preview apk but reference to InstantPreview.apk is broken.
UnityEngine.Debug:LogError(Object)
Gvr.Internal.InstantPreview:Start() (at Assets/GoogleVR/Scripts/InstantPreview/Internal/InstantPreview.cs:520)

sorry late reply
i found refrence link for this issue
https://developers.google.com/vr/develop/unity/instant-preview
make sure you have same version of unity i use, the make sure you installing the newest version of Java, acquiring newest Android SDK with updates, the problem persists. and verify that USB debugging is enabled on your phone


can't add less as dependency

Do NOT ignore this template or your issue will have a very high chance to be closed without comment.

Describe the bug
I use vite generate a project,Everything is fun after i install less as a dependency.
System Info

required vite version: 1.0.0-rc.4
required Operating System: mac10.15.6
required Node version: 14.10.1
Optional:

npm/yarn version  yarn-1.21.1
Installed vue version (from yarn.lock or package-lock.json) 3.0.0-rc.12
Installed @vue/compiler-sfc version 3.0.0-rc.12

Logs (Optional if provided reproduction)

Run vite or vite build with the --debug flag.
Provide the error log here.

yarn run v1.21.1
warning package.json: No license field
$ vite --debug
vite v1.0.0-rc.4
  vite:config env mode: development +0ms
  vite:config env: {} +1ms
  vite:config config resolved in 3ms +1ms
  vite:resolve (node_module entry) date-fns -> esm/index.js +0ms
  vite:optimize optimizing date-fns (contains relative imports) +0ms
  vite:resolve (node_module entry) less -> ./lib/less-node/index +6ms
  vite:resolve (postfix) /Users/yuli/work/vite-test/node_modules/less/lib/less-node/index -> /Users/yuli/work/vite-test/node_modules/less/lib/less-node/index.js +0ms
  vite:optimize optimizing less (no exports, likely commonjs) +2ms
  vite:resolve (node_module entry) vue -> dist/vue.runtime.esm-bundler.js +2ms
  vite:optimize optimizing vue (imports sub dependencies) +1ms
[vite] Optimizable dependencies detected:
date-fns, less, vue
Pre-bundling them to speed up dev server page load...
(this will be run only when your dependencies have changed)

[vite] Dep optimization failed with error:
Dependency graceful-fs is attempting to import Node built-in module fs.
This will not work in a browser environment.
Imported by: node_modules/graceful-fs/graceful-fs.js

Tip:
Make sure your "dependencies" only include packages that you
intend to use in the browser. If it's a Node.js package, it
should be in "devDependencies".

If you do intend to use this dependency in the browser and the
dependency does not actually use these Node built-ins in the
browser, you can add the dependency (not the built-in) to the
"optimizeDeps.allowNodeBuiltins" option in vite.config.js.

If that results in a runtime error, then unfortunately the
package is not distributed in a web-friendly format. You should
open an issue in its repo, or look for a modern alternative.
error Command failed with exit code 1.

Graceful-fs is a optionalDependency of Less

It's same to sass.

@underfin we have a similar issue but with server side rendering package which actually should be part of dependencies because it will be used in release while devDependencies like linter, less, sass won't. Does it mean that meaning of dependencies shifted a bit for Vite and it should contain only what is going to be on the client side?

If you don't want this behavior, you can add them into exclude inside optimizeDeps.

If you don't want this behavior, you can add them into exclude inside optimizeDeps.

How to support require


bug: Password is encrypted with multiple gpg keys. .gpg-id is ignored.

Summary
gopass does not honour .gpg-id, and tries to use multiple gpg keys to sign inserted passwords.
The same behavior is not present for pass
Steps To Reproduce
create gpg key_1 with password
create gpg key_2 without password
pass init repository with key_1
pass init -p sub_folder with key_2
cat ~/.password-store/.gpg-id shows only key_1
cat ~/.password-store/sub_folder/.gpg-id shows only key_2
pass insert personal/example/first
gpg -d personal/example/first.gpg and then insert wrong passwords. descryption will fail. (Expected)
gopass insert personal/example/second
gpg -d personal/example/second.gpg and then insert wrong passwords. access will be defined for gpg key_1 (Expected), but key_2 will be used as fallback and secret shown anyway (Not expected!)
Expected behavior
I expected gopass to sign my inserted password in not subfolder, only with key_1 and fail on inserting wrong passwords for key_1. But it used as fallback my unprotected key_2
Environment
pass version: v1.7.4
gopass version: gopass 1.15.13-git+HEAD go1.22.5 linux amd64
os version: VERSION="22.04.3 LTS (Jammy Jellyfish)"
installation method: go install github.com/gopasspw/gopass@latest

Additional context
At current moment issue is fixable for me if i delete record
and reinsert with pass insert again.

I'm not sure I follow your report.
You are saying that gopass incorrectly used a sub-keyid as a recipient for a secret outside of that folder?
Can you please provide more details information and steps to reproduce?
Please set GOPASS_DEBUG_LOG=/tmp/gopass.log (or simliar), GOPASS_HOMEDIR=/tmp/gopass and then execute all the steps necessary from the shell where these variables are set. This should allow you to operate in an isolated password store. Please provide the exact commands used and the log file.


⚠ Avatar API Endpoint has degraded performance

In 610aaa7, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1249 ms

Resolved: Avatar API Endpoint performance has improved in 609f5f0 after 1 hour, 25 minutes.

Resolved: Avatar API Endpoint performance has improved in fc90afe after 1 hour, 25 minutes.


Correct KeyError from matplotlib when processing Series yerr

[x] closes #11858
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

pls add a whatsnew note

@jreback Which section would this go in whatsnew?

bug fix

left note on orginal issue, but can you please check if this is still a problem on mpl 1.5.1, 2.x or master?  If so, please make an issue upstream.

whatsnew added

Also see https://github.com/matplotlib/matplotlib/pull/6392

@tacaswell Based on the assessment by @diazona  in https://github.com/pydata/pandas/pull/13114#issuecomment-218006317 , sending ndarray into matplotlib seems like a good modification (in addition to https://github.com/matplotlib/matplotlib/pull/6392 proposed by you)

That fixes the pandas problem, but not the case where
ax.errorbar('x', 'y', yerr='ye', data=df)

This should probably go into pandas for use with non-patched versions of mpl, but in the long run mpl should be able to cope with Series anyplace we can cope with ndarray.

Yep, I've confirmed that a test program like the following still reproduces the KeyError. (The fix I was working on in diazona/pandas@4b04f80c41684ba9ab05ce8c87b17b96fde87290 doesn't help with this case either.) Maybe there's one single place to make a change that would cover all these cases?
import matplotlib.pyplot as plt
import pandas as pd
import numpy as np

df = pd.DataFrame(np.random.uniform(size=(5, 4)),
                  columns=['x', 'y', 'xe', 'ye'],
                  index=[1, 2, 3, 4, 5])
plt.errorbar('x', 'y', xerr='xe', yerr='ye', data=df)

Would this above be worked in a followup issue? Thanks

I think it's worth investigating how to do a more "proper" fix, but I don't have time to get on that right now. Perhaps it is better to just incorporate this commit into the master branch and take care of the other cases later with a separate issue. I guess it's up to the Pandas developers.

@gliptak if you can incorporate @diazona more extensive tests and make the changes I suggested.
You can debug the other case that was brought up and see if its possible to fix as well.

The pr into mpl fixes both issues.

@jreback @diazona The code above directly calls into mpl, so changes to pandas will not offer a fix ...

Build fail seems to be unrelated:
  File "/home/travis/miniconda/envs/pandas/lib/python3.5/imp.py", line 234, in load_module

    return load_source(name, filename, file)

  File "/home/travis/miniconda/envs/pandas/lib/python3.5/imp.py", line 172, in load_source

    module = _load(spec)

  File "/usr/share/google/boto/boto_plugins/compute_auth.py", line 64

    except (urllib2.URLError, urllib2.HTTPError, IOError), e:

                                                         ^

SyntaxError: invalid syntax

yeah go ahead and rebase, fixed a weird boto import issue

I thought we were talking about allowing Pandas to work with older versions of matplotlib that don't have @tacaswell's patch. But if you'd rather say people should simply to upgrade matplotlib to avoid this bug, and leave it at that, I have no objections. Just checking that I'm on the same page as everyone else.

thanks @gliptak
yeh important that pandas fixes this (and of course even moreso for mpl)


Proper logging framework?

In Go's stdin log package, there's no way to define the message level, e.g. debug, info, warning, error. Most of the current logs are for debugging, so they should be hidden by default.

Let's try zerolog.


CPS-???? | Properly burning NFTs / tokens

(rendered version in branch)

Thanks for posting this @matiwinnetou - marked officially as in Draft stage according to the document header & the rudimentary stage of development.  When ready for review it can be marked as such.

@matiwinnetou since today's CIP meeting acknowledged that this proposal is still a bit "thin" - a bit like a feature request for wallets - could you include some use cases for what NFT / token burns would look like, as a way of filling out the proposal and identifying components that might exist beyond the wallets themselves?

I am willing to take this up and have a solution suggestion:
Instead of native scripts, I suggest users normally use Smart Contract tokens. I will try to compile a set of sample codes to emulate native script behaviour (most importantly the signature stuff) or maybe even an automatic tool that converts a native script into a Smart Contract.
This smart contract would then enforce the native script conditions for minting, but always allow burning.
Tutorials for minting/burning NFTs should be rewritten to use this smart contract instead of native tokens to make sure that at least all future tokens (that have no particular reason not to) are compatible
Extended reasoning: there are tokens that DO NOT want to be burned under any circumstances i.e. pool tokens certifying the correctness of a pool. Users should be able to opt-in into having their token burned at any time and it should be as simple as possible for them.

The referenced PR shows a sample script that would compile to only 750 bytes of Smart Contract and cost 0.05 ada to execute, IMO feasible (and much more powerful) as a new default for Cardano Native Tokens

@matiwinnetou I also don't know about ERC-721.
I know about the proposal to implement ERC-20 tokens by @michele-harmonic in https://github.com/cardano-foundation/CIPs/pull/444 . This is different in a sense that it would still make use of Cardano Native Tokens and not simulate them within contract datums.
Further I would suggest that we should not retrospectively change the guarantees made towards token creators (which only increases the potential for insecurity and problems with finite gain, only affecting tokens minted so far). Rather I suggest establishing a new standard that applies to all new tokens with no additional risk and infinite gain (affecting all future tokens).

Nice to see that https://github.com/cardano-foundation/CIPs/pull/479 has finally been presented as a solution for this.

In the past we have discussed a much simpler solution: simply don't require the minting policy script to be run when burning tokens. That is: you can always do it. We considered this right from the beginning in Mary.
The argument for why this is okay is quite simple: if you can send some tokens to an output, you can effectively send those tokens to the bin by sending them to an unspendable output. An unspendable output is more-or-less equivalent to burning tokens as far as the rest of the system is concerned, since they can never interact with anything else ever again. The only difference is they still show up in the UTXO set (and eat the min-ada deposit!).
I have always been a bit uncertain about this proposal but I think it a) achieves the goal very directly, b) is very simple, c) is very easy to implement, and d) seems pretty safe.

In the past we have discussed a much simpler solution: simply don't require the minting policy script to be run when burning tokens. That is: you can always do it. We considered this right from the beginning in Mary.

I'd just like to chime in to say that I've always been a strong supporter of this idea.
The other idea that I really like (which perhaps I should write as a CIP) is @redxaxder's "claim check" idea. Think of a hash function as a way to compress data. If I have a large token bundle in a single UTxO, imagine a transaction that exchanges the bundle for the hash of the bundle, thus allowing the min utxo value to decrease. In this form, the tokens are unusable, but if ever you want them back, we would allow for the inverse action, namely exchanging data for the matching hash. And you could have hashes within hashes, an infinite bad of holding.

And elaborating on a variant of the "claim check" idea that Jared and I discussed: just allow the elision of the "value" part of an output in favour of a hash, in exactly the same way that we do for the script and datum (script are always hashes, datums can be either). The preimage must be provided whenever needed, which means both when creating the output (to check balancing), and when spending the output (same). But crucially the size of the output in the UTXO would then be smaller and constant, so could have a smaller min-utxo requirement.

I found out that there is a problem with this specification in combination with CIP 25:
NFT and FT metadata is tagged by the latest minting transaction of the token. However if every user can burn a token, this is technically  a minting transaction, hence allowing every holder to arbitrarily change the metadata of their FT (for NFTs this is an obvious non-issue). However I would propose to adjust CIP 25 instead of changing any of the approaches here.

thanks @nielstron - Even with https://github.com/cardano-foundation/CIPs/pull/527 submitted I think it would also be a good idea to add this reservation to the CPS here (as you say, without changing any of the recommended approaches)... since we don't know whether or how that requested modification to CIP-0025 will proceed and should have this issue documented in any case.

@matiwinnetou this looks Abandoned so we'll close this soon unless you (or another advocate of this proposal) can post some updates & being carrying this forward.


Add tests for substring searching at substring boundaries

These additional tests validate that searching within a substring honors the substring boundaries, even when those substring boundaries are within the boundaries of a character.

@swift-ci Please test

@swift-ci Please test


addon-knobs radio input

Issue:
What I did
Added 'radio' knob to addon-knobs. This is functionally the same as the 'select' knob, however the UI is build with <input type="radio"> so the options are presented in a horizontal list with radio inputs.
An ideal use-case for this feature is a theme switcher - e.g. light and dark theme, etc. This is a superior user experience than the 'select' knob, as the selection can be more easily changed by single mouse clicks or using the arrow keys.
How to test
Is this testable with Jest or Chromatic screenshots?
I have included the Radio.js test file for testing with Jest.
Does this need a new example in the kitchen sink apps?
No.
Does this need an update to the documentation?
Yes. The README has been updated with relevant docs.
If your answer is yes to any of these, please make sure to include it in your PR.
For maintainers only: Please tag your pull request with at least one of the following:
["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]
Example

@astrotim I like it, good addition to the knobs addon for sure!

But we already have an open PR for this : #3894

I see, great minds think alike!
Perhaps you'd want to seize the opportunity to work together on this feature?
@brycemhammond @astrotim what can I do to assist you?

Thanks @ndelangen, I have looked at PR #3894, what a coincidence! A key difference I can see straight up is the different name for the knob itself.
Do you guys want to make a call whether you prefer ‘radio’ or ‘radioButton’?

I prefer radios or radio

TBH, I think we should finish the PR @brycemhammond started since it was the first one.
Also. @nfiniteset and @brycemhammond created an issue even earlier.

Hey @astrotim I hope you don't mind:
I just merged #3894 so this feature is in.
Thank you for your effort, truly appreciated!

No problem at, I'm happy to see it added either way.
Congrats on becoming a member of the Chromatic team, by the way 👍🏼


First public community meeting - Monday 19. April - 7:30 AM UTC

Since we have quite a few people wanting to help out in the development of the library I thought it was in order to hold a public meeting which should clarify the development of the library and how we are to collaborate on it.
The meeting will take place Monday 19. April - 7:30 AM UTC

Zoom link: https://postman.zoom.us/j/89392863142

The meeting will be recorded and uploaded to the AsyncAPI youtube channel to ensure those who cannot join that time can follow along as well. Any decisions will be added to issues in the repository so they can be discussed further if needed.
Attendees
@AceTheCreator
@NektariosFifes
@jonaslagoni
Notes:

Focus on modularity, but keeping in a way that it does not dig us down a rabbit hole.
Focus on CLI more than frontend from the start
We are gonna keep track of the project using GitHub's projects page - https://github.com/asyncapi/fluffy-robot/projects/1
@NektariosFifes are gonna create some more overall issues such as for the CLI, React library (already done), etc.
I am gonna figure out some permission settings in terms of code owners

Did I miss something?
Recording
https://youtu.be/CjVZIyse2k4

I would suggest we add an agenda next time to keep it more structured, what do you guys think?
cc. @AceTheCreator, @NektariosFifes

I would suggest we add an agenda next time to keep it more structured, what do you guys think?
cc. @AceTheCreator, @NektariosFifes
You're absolutely right @jonaslagoni

Here is the recording -> https://youtu.be/CjVZIyse2k4

Here is the recording -> https://youtu.be/CjVZIyse2k4

Thanks @derberg!


fix(gatekeeper/constraints/create): fix attribute assignment error

issue: https://github.com/rancher/dashboard/issues/3431

As per conversation in https://github.com/rancher/dashboard/issues/3431, dashboard in 2.4 is unsupported and we have no plans for releasing a 2.4.x dashboard. Issue is not prevalent in 2.5.8 or 2.6 so no porting required.


sync jupyterlab theme with OS?

Can it sync jupyterlab light/dark theme with OS? When using browser with dark reader extension. It can sync with OS.

@mbektas noted that there's a "Sync JupyterLab theme" option in prefs: https://github.com/jupyterlab/jupyterlab-desktop/issues/543#issuecomment-1355973951
Which theme option do you have selected in the Settings window? Which OS and which JupyterLab Desktop version are you running?

Yes I have that enabled and not working on Windows with auto dark mode installed.  I use latest release jupyterlab-desktop with a remote ssh forwarded session. Does the option only work with local sessions?

@xgdgsc yes, the sync theme is designed to work for locally launched sessions or remote sessions that were launched with --expose-app-in-browser.

Thanks for the tip for --expose-app-in-browser . Would it be better if adding that tip to the settings UI?

It seems only work when reopen the session. Can it be made work with working inside the session , when system switches, do the switch? Just like other apps like vscode/edge can do.

@xgdgsc --expose-app-in-browser is mainly intended for testing launches and used by desktop app as well. It is not suggested for general use cases.
Some setting changes do not really require restarts and can be applied in place, but we didn't get chance to optimize. Something to improve in future.


go/libraries/doltcore/creds: jwk.go: Create new creds storage files with 0600 permissions.

 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.


Security Jetpacks In Suit Storage

About the PR
Security and warden suit storages come with a jetpack
Why / Balance
Stations don't supply security with jetpacks. This simple fix makes sure sec is actually space ready incase someone is doing laps around the station with a gun. Also, HOS has one of their own, so why not their lackeys too? You can call this a mapper issue, but I'm not bugging every mapper to add jetpacks to sec.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:Alekshhh

add: Added jetpacks to security suit storages

This is a bit power creep-y no? All of sec just having the tools to catch up with any antag with a jetpack (which are not trivial to obtain) is a bit much.
Sec isn't supposed to be on par with all threats, they are just security guards and some stuff WILL give them a hard time.
Without mentioning that someone running laps around the station is not really a threat more than it is an annoyance. Their job is to protect the inside of the station, not patrol outside. And HoS can already deal with this IF it ever happens.
I think it would be better if it was a cargo purchase or at most there was a single non-HoS jetpack in the armory for emergencies of some kind.
Warden certainly doesn't need a jetpack since he is meant to stay and guard sec (his hardsuit is in case of atmos issues, not to chase people outside), and if he has to cover for a lack of seccies+HoS and chase someone outside, he can ask for the appropiate gear somewhere else (buy some from cargo, get one from the EVA room, ask HoS or Cap for theirs, etc)

I agree with warden not needing a jetpack
stations however have an average of two suit storages for sec, only maybe two having four, so just having on average two spare jets isn't going to kill some kind of dynamic. space battles are not uncommon, given how many roles have some form of space protection, so the jetpacks are there for teams to take out lingering threats out of the station that can harm the inside of the station.

QM suit is because he handles shuttles
RD is for dealing with artifacts
Cap is because cap gets everything
Only HoS is given one for space combat, altho i agree that 2 per station is not really an issue

I think more than just the HoS should be prepared to go on space battles, but I also don't think every officer should have a jetpack. As you mention, each station has far fewer hardsuit storage units than the number of officers.
To verify the last point, I went through all the maps and counted the suit storage units:

Map
Sec Suit Storage Units

Core
2

Barrantry
1

Aspid
3

Box
3

Kettle
4

Meta
2

Packed
2

Fland
2

Cluster
2

Omega
1

Saltern
2

Origin
2

And so I think this PR is fine once you remove it from the warden. And of course, I would recommend for Kettle to have less security hardsuit storage units.


Deprecate version property

See https://github.com/damccorm/actions-playground/pull/11/checks?check_run_id=212225627 for example output.
Once this is reviewed I'll follow up with the same change in other actions

bitcoin.pdf


0.3.0

Codecov Report

Merging #7 (89da550) into master (2f09824) will increase coverage by 0.63%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
+ Coverage   85.49%   86.12%   +0.63%
==========================================
  Files          14       14
  Lines         655      656       +1
==========================================
+ Hits          560      565       +5
+ Misses         95       91       -4

Impacted Files
Coverage Δ

src/path/mod.rs
82.45% <100.00%> (+0.31%)
:arrow_up:

src/file.rs
82.79% <0.00%> (+2.15%)
:arrow_up:

src/path/temp_path.rs
87.50% <0.00%> (+4.16%)
:arrow_up:

src/error.rs
75.00% <0.00%> (+5.00%)
: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 2f09824...89da550. Read the comment docs.


Set value between 5-100 in arch(x86_64)

Set value between from 50-100 to 5-100 in arch(x86_64)

value lowest from 50 to 5


Document 5.0 only APIs and features

https://github.com/microsoft/reverse-proxy/blob/main/docs/docfx/articles/runtimes.md documents relevant differences between 3.1 and 5.0. This should be updated to include new 5.0 APIs that YARP has exposed publicly. Some examples:
https://github.com/microsoft/reverse-proxy/blob/a1730a8817354ff59c1245a061b01da7a6496551/src/ReverseProxy/Abstractions/ClusterDiscovery/Contract/ProxyHttpClientOptions.cs#L52-L64
Also include that EnableMultipleHttp2Connections has been enabled by default.

Triage: E.g. Telemetry is useless (due to bug in Runtime).

I'd suggest we also add info to this doc for how to build for 3.1, accounting for #1002 and #837.


Airplane Bug

Expected Behaviour
The "This station is" Shows only 1 line/Sentence, and says the next station in 1 sentence
Actual Behaviour
Shows/Says Or Duplicates
Crash Report
No file rlly
Screenshots
Reply
Environment

Minecraft Version: 1.19.3
Installed Mods:MTR 3.2.2 Fabric, Joban Client, fabric api, dynamic Sound Filters, Optifabric

Additional Information
Anything else you want to add?
None

umm can you send a screenshot of editing the station in the railway dashboard ?


Implementation with Large-v3 but with Batching

I saw a large-v3 implementation with faster_whisper (https://github.com/guillaumekln/faster-whisper/issues/547) but it's quite slow.
Large-v3 is very fast with batching as shown here --- https://huggingface.co/openai/whisper-large-v3
Batching speeds up the transcription process by a lot. The only reason I wish to use faster_whisper is cause it provides things like srt, verbose, word level transcription

srt, verbose, word level transcription

u can write code to do those things with transformers interface

srt, verbose, word level transcription

u can write code to do those things with transformers interface

Do you have any resource.that I can use

word timestamp is an option when u call the pipeline
fort srt just retrieve timestamp and format properly

word timestamp is an option when u call the pipeline
fort srt just retrieve timestamp and format properly

No I meant do you have a dedicated function that I can use to convert it? For srt and verbose

see my comment in openai/whisper#654

see my comment in openai/whisper#654

Hey thanks! That sorts out my srt requirement. Now just looking for verbose and word

as i said, word timestamp is an option when call the pipeline

as i said, word timestamp is an option when call the pipeline

I actually got an error on running this --

ask in transformers github repo


ElementMap step support

What do these changes do?

Related issue number

Fixes #2778

Codecov Report

Merging #2942 (cf2cb06) into main (badf3f3) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2942   +/-   ##
=======================================
  Coverage   42.31%   42.31%
=======================================
  Files          99       99
  Lines       10631    10631
=======================================
  Hits         4499     4499
  Misses       6132     6132

Continue to review full report in Codecov by Sentry.

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


Update instruction to utilize a quick template

Provide a quick template for

new folks who wanna quick try statiq
Testers, who need a quick preview of statiq blog or cleanBlog for testing purposes

I've been working on a wholesale revamp of the docs in the api branch and unfortunately this looks like it'll conflict at this point. I'm going to close this PR and revisit once the new site it up and running.
Thanks!

No problem, excited about the revamp though!


Re-factor FontFaceObject.getPathGenerator to use Arrays instead of strings

This is similar to a lot of other code, where we use "Array + join" rather than repeated string concatenation.

/botio test

Thanks!


FormatException thrown when parsing DOCTYPE

I am currently trying to parse this xml file:
<?xml version="1.0"?>
<!DOCTYPE datafile PUBLIC "-//FB Alpha//DTD ROM Management Datafile//EN" "http://www.logiqx.com/Dats/datafile.dtd">

<datafile>
</datafile>

A FormatException is being thrown when parsing the DOCTYPE element.
System.FormatException
  HResult=0x80131537
  Message=Exception of type 'System.FormatException' was thrown.
  Source=U8XmlParser
  StackTrace:
   at U8Xml.XmlParser.<TryParseDocType>g__SkipUntil|17_0(Byte ascii, RawString data, Int32& i)
   at U8Xml.XmlParser.TryParseDocType(RawString data, Int32& i, Boolean hasNode, OptionalNodeList optional, RawStringTable& entities)
   at U8Xml.XmlParser.StartStateMachine(RawString data, CustomList`1 nodes, CustomList`1 attrs, OptionalNodeList optional, RawStringTable& entities)
   at U8Xml.XmlParser.ParseCore(UnmanagedBuffer& utf8Buf, Int32 length)
   at U8Xml.XmlParser.ParseFileCore(String filePath, Encoding encoding)
   at U8Xml.XmlParser.ParseFile(String filePath, Encoding encoding)
   at U8Xml.XmlParser.ParseFile(String filePath)

Tested on U8XmlParser v1.5.0

Thanks for the report.
I have been aware of this issue for some time. It is currently not implemented correctly.
I am reluctant to solve this problem correctly.
In order to resolve DTD placed by URI on external servers, etc., it needs to communicate with them using HttpClient or similar. I am providing this library as a simple xml parser and do not want to add extra library dependencies if at all possible.
It is possible to fix the error by skipping the DTD definition, but it does not solve the underlying problem.
(Especially if the DTD contains definitions of ENTITY, i.e., character aliases, the xml will not be interpreted correctly.)

So is there currently a way to programmatically skip the DTD definition when parsing and specifying it manually instead, or at least determine if it's there and skip it and not throw a FormatException?
Maybe some form of "header pre-parsing" before actually digging down to the real parsing to determine that there is an external DTD?
Because it would be a real shame to have it just fail and throw an exception whilst parsing a valid XML file.

Because it would be a real shame to have it just fail and throw an exception whilst parsing a valid XML file.

Yes, I think so, too. The library should provide a way to skip it so that the exception is not thrown in some way.
Currently, FormatException is always thrown when an external DTD is present.
This would require modifying the library.

Can you suggest a temporary workaround modification I can make to the XmlParser.cs file to somehow skip parsing the DOCTYPE altogether please?  Maybe a change to the method TryParseDocType()?
It would be fantastic if this can somehow be done because it would still make this library usable for me.

I wrote this temporary workaround which should be enough until an official fix is released.
This is my new code that reads the xml file:
bool tryExternalDocType = false;
XmlObject xml = null;
try
{
    XmlParser.ContainsExternalDoctype = false;
    xml = XmlParser.ParseFile(filePath);
}
catch (FormatException)
{
    tryExternalDocType = true;
}

if (tryExternalDocType)
{
    XmlParser.ContainsExternalDoctype = true;
    xml = XmlParser.ParseFile(filePath);
}

Added a public property XmlParser.cs and using it in the TryParseDocType method.
public static bool ContainsExternalDocType { get; set; }

private static bool TryParseDocType(RawString data, ref int i, bool hasNode, OptionalNodeList optional, ref RawStringTable entities)
{
    .
    .
    .

    if(SkipEmpty(data, ref i) == false) { throw NewFormatException(); }

    /** New code: *********/
    if (ContainsExternalDocType)
    {
        SkipUntil((byte)'>', data, ref i);
        return true;
    }
    /***********************/

    var nameStart = i;
    .
    .
    .
}

That code is not good and cannot be added to the library as is, even temporarily.
However, I will provide some alternative implementation. I'm working hard on it now.


Wrong networking settings in local docker-compose

For some reason I can't start project without removing networking settings. And when I was changing docker-compose.yml for the 4th time in one month I thought that may be for local development we don't even need this configurations.
Consider moving network settings to another docker-compose or just removing it from docker-compose.

@Aleserche do you use this settings in local environment?
networks:
  default:
    driver: bridge
    ipam:
      config:
        - subnet: <IP_ADDRESS>/16
          gateway: <IP_ADDRESS>

and static IP binding in container settings.
As far as I remember this was used to be able to pass this IP addresses to throttler and tenant whitelists. But I think we can just pass* for local development and it supposed to work well.


View-only "attributes" without binding in templates

Sometime models/maps object may also contain attributes with static (for example string) data, that is not supposed to change during app lifecycle.
When such attributes are used in stache templates unnesessary bindings are created. I think this can affect perfomance in some way, is it possible to get rid of such exessive bindings in templates? Or it is not woth to bother about it?

Are you sure bindings are setup? I can't think of a mechanism that would do this. Afaik, static values don't call can.__observe.
Sent from my iPhone

On Jul 9, 2015, at 2:50 AM, Alex<EMAIL_ADDRESS>wrote:
Sometime models/maps object may also contain attributes with static (for example string) data, that is not supposed to change during app lifecycle.
When such attributes are used in stache templates unnesessary bindings are created. I think this can affect perfomance in some way, is it possible to get rid of such exessive bindings in templates? Or it is not woth to bother about it?
—
Reply to this email directly or view it on GitHub.

I mean static values not they are not set up as:
map.prop = 1 (vs attr set: `map.attr('prop', 1), but this properties come from server for example, but they are not supposed to change, but they are still are treated as map's attributes and are subjects for change event.

I have the same issue. I'm pulling some data from the server for some sort of logging timeline, and it generates 7K cid's (compared to 2.5K from the rest of the page) while it doesn't have to be live at all. Was wondering the same thing as whitecolor.

You can use the define plugin. type: "*"
Sent from my iPhone

On Jul 12, 2015, at 5:02 AM, Guido Smeets<EMAIL_ADDRESS>wrote:
I have the same issue. I'm pulling some data from the server for some sort of logging timeline, and it generates 7K cid's (compared to 2.5K from the rest of the page) while it doesn't have to be live at all. Was wondering the same thing as whitecolor.
—
Reply to this email directly or view it on GitHub.

var Person = can.Map.extend({

  define: {
     age: {
       type: 'number'
       value: 10
     }
     name: {
        type: '*',
        value: {
           first: '',
           last: ''
        }

     }
  }

})

in this example last and first props of name wont have bindings as name is plain object. but name it self and age still will have bindings. An I want this props to be out of live binding. But I a may want some other props be live binded so I have to use can.Map, not simple object or can.Construct for 'Person' class.

@whitecolor you can't have nested type definitions inside something with type: "*".  So:
name: {
        type: '*',
        value: {
           first: '',
           last: ''
        }

     }

wont work.  The reason is that a map needs to be created for things like default properties to work in the first place.
However, you can setup the object yourself in a setter:
name: {
  type: '*',
  set: function(nameObj){
    nameObj.first = nameObj.first || "";
    ...
    return nameObj;
  }
}

closing b/c "*" should provide a good enough solution for most of these problems.

why this wont work?
     name: {
        type: '*',
        value: {
           first: '',
           last: ''
        }

     }

name will be plain object {fist: '', last:''}
'*' can only prevent live bindings for "nested" properties like in this case name.first, but it can not prevent live binding of name property it self ofr age property that are directly on can.Map instance.

I miss-read your code.  That should work, but that {first: '', last: ''}object would be shared by every instance of that map.
What do you mean by:

An I want this props to be out of live binding. But I a may want some other props be live binded so I have to use can.Map, not simple object or can.Construct for 'Person' class.

what is  "this propos"?

'*' can only prevent live bindings for "nested" properties like in this case {{name.first}}, but it can not prevent live binding of {{name}} property it self or {{age}} property that are directly on can.Map instance.

Yes, that is correct.  But I think that creating a binding of name or age is fine.  Otherwise, don't create a map in the first place if you don't want all 1st level properties to have bindings.
The only other alternative is to create a stache helper that prevents binding:
{{~"age"}}

can.stache.registerHelper("~",function(prop, options){
  return options.context[prop]
})

ok helper would work.

Why not include it in core? Could be useful.

Because live-binding is often less expensive then creating the observables. You aren't probably saving much time with this.
Sent from my iPhone

On Jul 13, 2015, at 2:34 PM, Alex<EMAIL_ADDRESS>wrote:
Why not include it in core? Could be useful.
—
Reply to this email directly or view it on GitHub.


Presence of Single Consumer is forcing other producers exposed under output-bindings property  to be treated as BindableFunctionProxyFactory bean with "functionExist" parameter set to TRUE, even when Streambridge is used instead of Supplier

In one of the microservice , i am having  one Producer, for which StreamBridge is used instead of Supplier and a consumer which is backed up by Consumer Functional interface. Because of the presence of only one consumer , i see the producer which is exposed under spring.cloud.stream.output-bindings application property , is registered as BindableFunctionProxyFactory bean with "functionExist" parameter set to TRUE .  [While debugging i see normalizeFunctionDefinition() from org.springframework.cloud.function.context.catalog.SimpleFunctionRegistry is doing this explicitely when only one consumer is pressent]

I was expecting this to be false since there is no Supplier backing it up for my producer. Now i am confused if its a BUG or any purpose is there behind it .
What to do so that "functionExist" will set to FALSE for the producer in my case since i have only one consumer backed UP by Consumer interface and exposed under spring.cloud.function.definition property.
Please let me know if in this scenario i should avoid output-bindings property.
And what is the real use of output-bindings apart from pre-creating the bindings ? Is it mandatory to use when i go with Streambridge?

@Saumrit
In this project we used named bindings and destinations only without output-bindings. If you are building a 'sink' then you have a consumer function that is bound to an input and it can use StreamBridge to send messages to multiple named bindings. This would be a type of router functionality.
So my questions are this: Are you building a source, processor or sink, that will bind to a producer, function or consumer?

We auto-detect function and create binding if there is only one, which seems to be the case for you.
There are several ways to deal with that if that is not a desired behavior. The simplest one is to set spring.cloud.stream.function.autodetect property to false;

We auto-detect function and create binding if there is only one, which seems to be the case for you. There are several ways to deal with that if that is not a desired behavior. The simplest one is to set spring.cloud.stream.function.autodetect property to false;

Actually in my case i have one Consumer[Backed by java.util.function.Consumer]  and several Producers [No Suppliers are here , i am using Streambridge instead] . Presence of this single consumer is forcing the auto provisioning of all these producer bindings and they are exposed under output-bindings property.
During debug i found org.springframework.cloud.function.context.catalog.SimpleFunctionRegistry.normalizeFunctionDefinition() is responsible for this.  My question is in above scenario , i dont have my own Suppliers in class path , so why this is happening ?

Spring Cloud Function auto-discovered only one (single) bean of type Consumer, Function or Supplier in application context and gave it to s-c-stream

Spring Cloud Function auto-discovered only one (single) bean of type Consumer, Function or Supplier in application context and gave it to s-c-stream
i have no Suppliers for those producer bindings , still this auto-provisioning is happening . I use Streambridge.

Actually in my case i have one Consumer[Backed by java.util.function.Consumer]. Is this not the case?

Actually in my case i have one Consumer[Backed by java.util.function.Consumer]. Is this not the case?

yes , for  event consumer i have one java.util.function.Consumer and for Producers , i use
streambridge.

So you have only one Consumer and because it's only one it is auto-discovered by s-c-function. StreamBridge has nothing to do with it as it is the artifact of s-c-stream

I am completely fine with the Consumer . My Problem is S-c-function is trying auto provisioning for those producer bindings which are exposed under output-bindings .

auto provisioning for those producer bindings which are exposed under output-bindings  - s-c function doesn't know what binding is, so I am lost as to what you are trying to say.
Can you post some code or sample app to reproduce the issue you are repoprting?

My yml looks like this -:
Spring:
cloud:
function:
definition: consumerinput
stream:
output-bindings: hardstopproducer
bindings:
consumerinput-in-0:
group: team.queue.cit.user
contentType: application/json
binder: rabbit
consumerretry:
group: team.queue.cit.user
destination: team.river.retry
contentType: application/json
binder: rabbit
hardstopproducer:
destination: team.river
contentType: application/json
binder: rabbit
issue i am getting -:
message":"Shutdown Signal: channel error; protocol method: #method<channel.close>(reply-code=403, reply-text=ACCESS_REFUSED - configure access to exchange 'hardstopproducer' in vhost 'dev' refused for clinical 'hardstopproducer' In rabbitMQ user , access has been provides such that only this exchanges starting with "team" keyword will be allowed to created.

What is the issue? Why do you care about BindableFunctionProxyFactory? What exactly is not working for you?

I this my question needs to be reframed .
Meantime can you please let me  knw what this expression eligibleFunction.size() == 1 do in the
org.springframework.cloud.function.context.catalog.SimpleFunctionRegistry.normalizeFunctionDefinition()
`String normalizeFunctionDefinition(String functionDefinition) {
        functionDefinition = StringUtils.hasText(functionDefinition) ? functionDefinition.replaceAll(",", "|") : System.getProperty("spring.cloud.function.definition", "");
        Set<String> names = this.getNames((Class)null);
        if (!names.contains(functionDefinition)) {
            List<String> eligibleFunction = (List)names.stream().filter((name) -> {
                return !"functionRouter".equals(name);
            }).filter((name) -> {
                return !"defaultMessageRoutingHandler".equals(name);
            }).collect(Collectors.toList());
            if (eligibleFunction.size() == 1 && !((String)eligibleFunction.get(0)).equals(functionDefinition) && !functionDefinition.contains("|") && !((String)eligibleFunction.get(0)).startsWith("&")) {
                functionDefinition = (String)eligibleFunction.get(0);
            }
        }

        return functionDefinition;
    }`

Because my issue comes in that if block whenever there is only one consumr.

@olegz
Let me requote my issue -:
" I see the producer which is exposed under spring.cloud.stream.output-bindings application property , is registered as BindableFunctionProxyFactory bean with "functionExist" parameter set to TRUE .  I was expecting this to be false since there is no Supplier present in code , instead Streambridge i used."
Because of this in console i see this error eventhough this doesnot stop my app to function .
""Shutdown Signal: channel error; protocol method: #method<channel.close>(reply-code=403, reply-text=ACCESS_REFUSED - configure access to exchange 'hardstopproducer' in vhost 'dev' refused for clinical 'hardstopproducer' In rabbitMQ user , access has been provides such that only this exchanges starting with "team" keyword will be allowed to created."

Please read this section. Specifically However, if you want to pre-create an output binding at the initialization (startup) time you can benefit from spring.cloud.stream.output-bindings property. That should answer your question about  why is trying to create one exchange with the producer binding name i have provided under output-binding property .


Upgrade to H3 core 3.7.2

This version improves memory management when performing polyfill operations

Has 3.7.2 been released yet? As far as I can tell 3.7.1 is the latest version.
But I can see that I've forgotten to release a version with 3.7.1 as well, I'll make sure to do that.

I was looking at the h3-js repo!!! :facepalm:
https://github.com/uber/h3-js/releases

No problem, at least I got around to releasing a version with h3 core 3.7.1 😄


[2.0.0] Behavior when both cacheTime and disposeDelay are specified is unclear.

The new cacheTime and disposeDelay features are really great in a lot of scenarios, and it is clear what they do separately. If cacheTime=10 sec, the provider will be automatically refreshed if read more than 10 seconds after the previous refresh. If disposeDelay=10 sec, the provider will wait 10 seconds after the last listener is removed to dispose itself. This is clear.
Now, if BOTH cacheTime and disposeDelay are set to 10 seconds, which of them will take priority? Will the provider be invalidated/disposed 10 seconds after it is loaded, OR 10 seconds after the last listener is removed?

Ok, thank you, that makes a lot of sense. So if you set both to 10 sec, only the disposeDelay will really take effect, since the removal of listeners will always happen after the refresh, it will always be disposed 10 seconds after the last listener is removed. CacheTime only really makes sense if it is bigger than disposeDelay, am I correct here?

Pretty much, yes.
Generally cacheTime can be set to a few minutes, while disposeDelay is more about a few seconds


Add Conditional Requests (IETF RFC 7232) support in the client

It would be lovely if the hyper client supported https://tools.ietf.org/html/rfc7232.
My downstream usage is in https://github.com/softprops/hubcaps/issues/104.
It looks like there's already a rust implementation in servo: https://github.com/servo/servo/blob/master/components/net/http_cache.rs, if that helps. That also seems to implement https://tools.ietf.org/html/rfc7234 (Caching).

Supporting conditional requests is certainly useful, but I think it may be out of scope for hyper. It should be possible to do this entirely outside of hyper (as shown by Servo's implementation).
It might be cool to see if Servo would be interested in extracting it's cache implementation into a library.


Fix glossary tooltip mismatch

To fix #14226. (I found #12608 is related too)
I modified
https://github.com/kubernetes/website/blob/0fe73a189015e20d826341ed7bd5d214612ecfdc/layouts/shortcodes/glossary_tooltip.html#L8
from not exact match ("%s*") to exact match ("%s.md").

This will affect to whole terms in reference/glossary page.
We can check the current bug in
https://kubernetes.io/docs/reference/glossary/?all=true#term-app-container
If we mouse over to "containers" and "pod", we can check they show wrong tooltips.


MFS ipfs.files.cp arguments

What's the thinking behind the ipfs.files.cp arguments?  Specifically why is the first argument an array?
ipfs.files.cp([sourceFile, destinationFile], callback)

It could be more straightforward as:
ipfs.files.cp(sourceFile, destinationFile, callback)

I suppose maybe you'd want to imitate the unix cp command and support both the above and:
ipfs.files.cp(...sourceFiles, destinationDirectory, callback)

Obviously JavaScript doesn't support left-variadic functions so you'd have to emulate it by interrogating arguments but that's an implementation detail.

👍 on removing the array proposal too.

Great, I'll put a PR together tomorrow.

See #277 for more on this.


Pure localization in 3D case

Hi,
The "pure localization" works properly in 2D cases, however, when I tried to localization the robot in 3D environment, I encountered the following error:
# terminate called after throwing an instance of 'std::out_of_range'
what():  vector::_M_range_check
[cartographer_node-2] process has died [pid 30846, exit code -6, cmd
I don't know where i am wrong or the 3D pure localization is not completely implemented.
I believe I correctly generated the Map and obtained the "pbstream" file as shown below

Pure-localization for 3D is not yet fully implemented. It seems we don't have an issue tracking that, so we'll leave this open as a placeholder for it.

Towards 3D localization: Add trajectory trimming support to 3d

The message is caused by unhandled exception of thread pool;
you can catch this exception to avoid the program be terminated;
but the root cause remains to be investgated;
maybe it will be helpful;
--- a/cartographer/common/thread_pool.cc
+++ b/cartographer/common/thread_pool.cc
@@ -73,7 +73,12 @@ void ThreadPool::DoWork() {
}
}
CHECK(work_item);

try {
work_item();
} catch(std::out_of_range & e) {

   LOG(ERROR)<<e.what()<<__FILE__<<__LINE__;

}
}
}

@gladxu That is a bad idea. It fights the symptom (stuff threw exceptions), but not the cause (it should never get an out of range error).

@SirVer That just avoid the program be terminated by OS;the error information will be logged;and the root cause needed to be find out;
It is a bad idea that leave the exception alone, cause the error information provided by OS is too little;you have to do some work to find out where the exception come from;

I disagree - not terminating the program when an unexpected condition occurred is wrong, since it is very dangerous. So letting the exception crash is better than catching.
Having more detailed information what happened would be nice, I fully agree there :). Unfortunately the exception does not carry a stack trace of the callsite.

I had great experience using Mozilla's rr reversible debugger for debugging hard-to-reproduce errors.

here is the root cause of the exception
cartographer/mapping_3d/scan_matching/rotational_scan_matcher.cc
RotationalScanMatcher::RotationalScanMatcher(
const std::vector<std::pair<Eigen::VectorXf, float>>& histograms_at_angles)
: histogram_(
Eigen::VectorXf::Zero(histograms_at_angles.at(0).first.size())) { //!!access vector.at(0) without vector size check;

Thanks a lot for tracking this down. Seems like PR https://github.com/googlecartographer/cartographer/pull/501 introduced this code. So assigning bug to @wohe.

The implementation of pure localization in 3D had not been finished, so it was not surprising it did not work yet. I sent a PR that fixes the issue you ran into and one remaining issue related to IMU data. I think 3D localization should be supported now. Please let me know if you still have issues.

Wooo, great! The 3D localization is working now. But the map was still updating during the pure localization, is there any way to disable the continuous map updating?
Thx!

In pure localization, trajectory 0 is a static map and is not changed.
Can you please specify what you mean by "continuous map updating", i.e. expected behavior and observed behavior? Thanks.

When you see updates in rviz those are submaps belonging to  trajetory 1, i.e. the fixed length trajectory build from sensor data from the localizing agent. This can't be turned off as we localize by both finding matches between scans from the agent against submaps  from trajectory 0, i.e. the frozen map, as also the other way around.


Add string arguments for Cube methods

🚀 Pull Request
Description
Resolves #3602
I have just started to work on this, and after adding a short test I get 5 failures when testing lib/iris/tests/unit/cube/test_Cube.py, all AssertionErrors similar to:
AssertionError: CML do not match: /home/gcaria/github/iris/lib/iris/tests/results/unit/cube/Cube/xml/checksum_ignores_masked_values.cml

which don't seem to be related to the test I've added. I've followed the instructions for running the tests so I'm not sure what I'm missing.

Hi @gcaria, thanks for the contribution!  I’m not sure about that specific error, but I can see that your branch is a couple of commits behind the master branch.   The most recent commit to master incorporates a switch from Travis-CI to Cirrus-CI to run the tests within the PR.  So I suggest rebasing to pick up that change, and then we can see your test output here.  So something like:
git fetch upstream
git rebase upstream/master
git push -f origin

Depending how and exactly when you created your python environment, it may be also be worth worth re-doing that.  cftime v1.3 was recently released, and caused all sorts of failures within Iris.  We’ve temporarily pinned the cftime version (#3927), but that was also after you created your branch.
So if you rebase, then create a new environment from the source, then run the tests again, you might get a different result.

Thanks @rcomer for the useful feedback !
In addition to changing the cftime version, I've also switched to python3.7 (was using 3.8 before) which was surely part of the problem, at least locally. Now everything works :)
I see that in #3295 the docstring for the remove_cell_measure was also expanded.
Would it be appropriate to have the same also for the modified methods of this PR ?

Hi @gcaria, glad you got the tests working!  I don't think I'm the best person to advise on this specific change as I haven't done anything with this part of the code.  Looks like @stephenworsley has done a lot in this area so hopefully he can advise.

Oh I see this was all documented here, my bad for missing it !

@gcaria Perfect, thanks :+1:


Imrove error handling test coverage

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 15 (80.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+8.5%) to 97.546%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/dispatch-queue.ts
9
12
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
8.5%

Covered Lines:
155

Relevant Lines:
158

💛  - Coveralls


Custom serviceType is 1920, but always use default 1000 ,why?

[{"applicationName":"MyTestPP","serviceType":"TOMCAT","code":1010},{"applicationName":"gaeaServer","serviceType":"STAND_ALONE","code":1000}]

@youwoa
If you've developed a custom plugin for an application, you'll have to implement ApplicationTypeDetector to have the agent detect the service type.
Here is an example for Tomcat.

ok


Unexpected ':'

I have the following bit in .pre-commit-config.yaml
---
repos:
    - repo: https://github.com/jumanjihouse/pre-commit-hook-yamlfmt
      rev: 0.0.11
      hooks:
          - id: yamlfmt
    - repo: https://github.com/PyCQA/flake8
      rev: 3.9.2
      hooks:
          - id: flake8
            args: [--max-line-length=100, --per-file-ignores=python/myrepo/*.py:F401]

However, I'm getting a persistent
while scanning a plain scalar
  in ".pre-commit-config.yaml", line 11, column 43
found unexpected ':'
  in ".pre-commit-config.yaml", line 11, column 81

The syntax that I use in legal, and I think the only syntax. How do I get rid of this error?

Ah sorry, this is coming from https://github.com/pre-commit/pre-commit-hooks


Update tutorial-create-vmss.md

Update wording to make it read better.
It might be a small change but I had to re-read
this sentence multiple times to get it. The way it
was written it seems to indicate that the rules
trigger things but its the metrics that trigger the
rules.

@boykoc : Thanks for your contribution to the Azure documentation! The author, @iainfoulds, has been notified to review your proposed change.


Require "composer-packages" when composer.json is present

Summary
Resolve #16
Use Cases
Require "composer-packages" when composer.json is present
Note: I chose not to include an integration test here, for several reasons:

I suspect the composer-install and/or language family buildpacks will have their own integration tests specific to Composer use cases
The composer and composer-install buildpacks have not yet been released

Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@joshuatcasey this looks great! I will approve and merge when https://github.com/paketo-buildpacks/rfcs/pull/170 is merged in!


Generic optimiser example using Ray Tune

This PR adds an example notebook for a generic optimiser implementation with Ray Tune.
Closes #1091

Thank you Niko,
can you install pre-commit install and submit again
after running
pre-commit run --a


Dependency version: route-recognizer fails on build

As a developer I want pretender's route-recognizer dependency in it's bower.json to upgrade from 0.1.1 to 0.1.9.
Given I install pretender 1.1.0 to an Ember 2.3 project
When I execute Ember build
Then I expect to the build to run without error
Instead I encounter the error described in this thread.

#155

Handled by https://github.com/pretenderjs/pretender/pull/163


Choosing how long the temperature is being set

In the vSMART app it's possible to set how long the thermostat will be on.

Is this also somehow possible in this integration?

Just to make sure I got it right - you're referring to the ability to define how long manual temperature set should be used?
If that's a yes, then there is an option in the vSMART settings, which I plan on exposing in the integration. I still have to confirm the API changing this setting works.
I have a couple more cleanups to do before any new features, but I'll write an update here when I manage to work on this.

Just to make sure I got it right - you're referring to the ability to define how long manual temperature set should be used?

Yes. You understood me right.

If that's a yes, then there is an option in the vSMART settings, which I plan on exposing in the integration. I still have to confirm the API changing this setting works.
I have a couple more cleanups to do before any new features, but I'll write an update here when I manage to work on this.
Thanks a lot


Remove business leads endpoints

Description of change
This removes all business leads endpoints.
The model is kept to avoid any issues with the deployment and will be removed next time.
Checklist

[x] Has a new newsfragment been created? Check changelog/README.rst for instructions
[ ] Have any relevant search models been updated?
[ ] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[ ] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[ ] Has the admin site been updated (for new models, fields etc.)?
[ ] Has the README been updated (if needed)?

Should we remove the factory in datahub/leads/test/factories.py as well?

@reupen One of the cleanup tests relies on it so I can't remove it just yet