--only-recognized-file-types doesn't respect Exclude directives

--only-recognized-file-types was added in https://github.com/rubocop/rubocop/pull/7930.  It lets you (with the default config for example) do something like
rubocop --only-recognized-file-types user.rb company.js

and have Rubocop properly only inspect the .rb file, not the .js file.
However, if you include something from the Exclude rules (like something in tmp/, per the default config):
rubocop --only-recognized-file-types user.rb company.js tmp/not_me.rb

It inspects two files, even though it should only inspect the one "recognized" file.
Background for why this is a problem for us: we'd like to run Rubocop on files currently staged by git.  We can get the list of files to pass to Rubocop with git diff --name-only --staged | xargs.  If these staged files include .js files, they're properly ignored like we'd like.  However, if there are staged changes to files that we've Excluded, they're not ignored even though they should be.

Expected behavior
"Excluded" files shouldn't be considered recognized.
Actual behavior
"Excluded" files are considered recognized and are wrongly checked.
Steps to reproduce the problem
See example commands above.
RuboCop version
1.43.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.24.1, running on ruby 3.0.4) +server [x86_64-darwin21]

I just found the --force-exclusion flag, which seems to solve this when passed in combination with --only-recognized-file-types.
I still think the behavior of --only-recognized-file-types is incorrect, but at least there's a workaround here.
Leaving this open just in case the maintainers want to change the behavior of --only-recognized-file-types in a future major version, but feel free to close it otherwise.


[Accessibility][Info&Relationship-Filter Dialog]: Ensures elements under 'filter dialog' with an ARIA role that require child roles contain them.

User Experience
Users who rely on AT will face difficulty to complete task if there is no child available for element.
Test Environment
Browser: Microsoft Edge Canary Version: 82.0.425.0 (Official build)
OS build: 2004 (19564.1005)
URL: Bot Framework Composer
Prerequisite Here
Repro Steps:

Open the above mentioned URL in EDGE.
Select 'Design Flow' from left navigation.
Now, run the Accessibility Insight tools to find the issue.

Actual:
Ensures elements under 'filter dialog' with an ARIA role that require child roles contain them.
Expected:
Every element with an ARIA role must have child roles contain them.
Note:
This accessibility issue was found using Accessibility Insights for Web 2.14.1 (axe-core 3.3.2), a tool that helps find and fix accessibility issues. Get more information & download this tool at http://aka.ms/AccessibilityInsights.
Element path:
div[data-testid="DialogTreeItemEmitAnEvent"]
Snippet:
<div data-testid="DialogTreeItemEmitAnEvent" class="ms-FocusZone css-117 ms-OverflowSet css-zqjw3r root-176" role="menubar" data-focuszone-id="FocusZone147">
How to fix:
Fix any of the following:
Required ARIA children role not present: menuitem menuitemradio menuitemcheckbox
MAS Impacted: MAS1.3.1
Attachment:

As checked, the issue is resolved. So, HCL is closing the issue.


Add user guide

Codecov Report

Merging #17 (12ff5ec) into master (fef5d7d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #17   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 fef5d7d...12ff5ec. Read the comment docs.


Adding SASS CSS preprocessor

Adding SASS CSS preprocessor via gulp-sass and gulp-autoprefixer
Adding SASS linter with gulp-scss-lint
Updated README.md for required scss_lint Gem dependency for
gulp-sass-lint
Switched project’s CSS to SASS files and addressed SASS linting errors
Update performed with Node v5.10.1 and npm v3.8.3

Also ran with Node v5.11.1 and npm v3.8.6.

This is awesome @lfarran  thank you!
@d3viant0ne Have any thoughts here? (review?)

@lfarran Only thing I'll mention is there's a side of me that opts for LESS over SASS due to fact this brings another dependency on another language, ruby, whereas, LESS is just js.

The reason I chose SASS over LESS was that Bootstrap and Angular Material are both using SASS.  The gulp-sass dependency uses node-sass which doesn't use Ruby.  If Ruby is the concern, removing gulp-scss-lint could fix that.  However, I see value in the SASS linter.

Definitely @lfarran I really like this, lemme see if @d3viant0ne has any comments before merging.

@lfarran can you ensure that your SASS setup here also works with the {N} app?
I believe this would be the thing to use: https://github.com/toddanglin/nativescript-dev-sass

@NathanWalker The SASS setup works for the {N} app with https://github.com/toddanglin/nativescript-dev-sass. To install: cd nativescript, tns install sass.  Is there a good way to script this automatically install, or is it sufficient to add an install note in the README under the NativeScript section?  I'm also debating how the SASS linting should work for the {N} app, combine with the current gulp scss-lint task or make separate?

You should be able to:
cd nativescript
npm i nativescript-dev-sass --save

To ensure it's saved to the {N} app's dependencies, then all should be good to go.
SASS linting I'm unsure of, but it'd be okay to leave linting out of the {N} app for now since some debate around build integrations is still circling.

@NathanWalker My only comment is the same I always make, it limits what others can use which is why I lean towards examples as opposed to sticking framework X into something that is supposed to be generic.
I use scss in most of my projects but it's not included in my boilerplate for client projects at work. sass isn't always the right answer so I prefer to be able to support it without implementing something specific.
The end result of this could be after this merges that you end up with a bunch of less / postcss users that have to rip all this out every upstream update to use their prefered CSS pre / post processor.

@d3viant0ne thank you for that 👍
@lfarran I do agree with Josh's comments, what do you think if we merged this to a branch called sass and document it as an option in the readme?

@NathanWalker That's something I've tried to get into seed upstream. We need to support common integrations better and forks are deficient in that regard.
Incoming pull requests could be vetted against integration branches to find problems before they make master and cause users pain when updating.
So you could do one for SASS, on for Material2 & perhaps one for ng2-bootstrap which should in theory enable us to find issues in PRs or refactoring before they make master.
PRs are opened to develop and merged in, then rebased into the integration branches off of develop. If everything checks out, it's merged down into master with reasonable confidence that people can pick up that change set without breaking projects built on top of the advanced seed.

And if you opt to go that direction, create a branch for Material2 and i'll get to work on it.

Material2 branch:
https://github.com/NathanWalker/angular2-seed-advanced/tree/material2
Sass branch:
https://github.com/NathanWalker/angular2-seed-advanced/tree/sass
ng2-bootstrap:
https://github.com/NathanWalker/angular2-seed-advanced/tree/ng2-bootstrap
👍

@lfarran Once you have this comment addressed:
https://github.com/NathanWalker/angular2-seed-advanced/pull/77#issuecomment-219940263
Just close this PR and reopen to target the sass branch and I'll get it merged. Thank you very much for your efforts!

I like the concept with the branches.
What would the workflow be if - lets say - i want to use the advanced-seed with sass und material? Would i merge all three branches (master, sass, material) into my local branch on every update? Couldn't this get messy with the time?

@metzc - The integration branches are more to aid us in validating changes against common integrations and finding issues with code changes before they land in master and effect users with projects built on top of the advanced-seed.
IMO in a production product, you may or may not fork the advanced-seed directly. Either way, I would suggest only pulling updated form upstream master to  master.
No framework integration is going to meet everyones requirements exactly, these integration branches exist as a guide to users and to simplify validation for maintainers to prevent changes from landing in master that break common integrations. Nobody is suggesting that people build on top of integration branches.

Maybe this is a little overreaching, but would this project be better served as a generator? So that users would be able to pick and choose the technologies that they want to integrate. I know this brings in another set of issues, but it might be better than maintaining a bunch of branches.

@JakePartusch - That idea was already proposed and started by @ludohenin. With offline comp & the cli coming, creating a generator is a lot of work with very little payout at the moment.
The only people that "have" to manage branches are the project maintainers and that is specifically to find problems before they hit master. A generator doesn't help solve that problem.

@d3viant0ne thanks for the info! I didn't know that @ludohenin was working on something similar. Even so, the same argument that is being made for/against sass could be made for/against any other proposed technology addition. How is it decided what gets integrated vs. put in a branch?

IMO the general rule would be technologies what have popular alternatives.
i.e.
Pre/PostCSS vs. SASS vs. Less
Material2 vs. Bootstrap vs. polymer
We have broken common technology integrations in the seed quite a few times and finding out after the fact that a change we made had adverse effects on someone real world projects is the suck.
Running branches for integrations may or may not help but something needs to be done to give people more confidence in stability when pulling changes into their forks.

@NathanWalker Sorry for the delay. I have addressed the comments in #77 (comment), merged the changes to the sass branch, and created PR 82.  I'm closing this PR per your request.


x/tools/cmd/goimports: use of own package name creates broken imports

When running goimports on a file in a package that (wrongly) uses it's own name as a qualifier, goimports will rewrite the imports to include the code file's own package in the imports. Goimports shows no error and only on compilation will there be an import cycle error.
I propose that Instead of the imports being rewritten, a check be performed to see if the file is in the same package as goimports is trying to add. If this situation is detected, the command should fail and output an error message instead of rewriting.
I'll be working on patching this behavior if this proposal sounds appropriate.
cc: @alandonovan (since I think you're the one to ask)

Brad owns goimports.  The feature seems reasonable though.
cc: @bradfitz, @crawshaw

I wrote goimports once. I wouldn't say I own it. :)

(I unassigned it so anybody can feel free to fix. I don't want to give the impression that I'm actively working on or planning on working on this)

I can take a stab at it, we'll see if it as easy a fix as I hope it is :)

@ScottMansfield, are you working on it? If not, I would like to try to come up with a fix.

@davidrjenni Nope, I never did get around to it. By all means, please go ahead. This bit me again recently, but it turned out to be not as much of a nuisance in the mean time so I forgot about it.

I think this was resolved in #30663. Please comment if you disagree.


Add support for telugu language

This PR adds support for Telugu language in the Cloudstack Ui

@blueorangutan package

@blueorangutan ui

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9956 (QA-JID-482)

@kiranchavala , please resolve the lint issue.

@blueorangutan ui

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9956 (QA-JID-483)

@JoaoJandre @DaanHoogland some PRs targeting 4.20.1 are merged on main - should they go to 4.20 branch?

@rohityadavcloud Considering that they were not targeting 4.19 (a maintenance branch), I assumed the authors wanted their changes to be made on the main branch directly. I'll fix the milestone. If the authors want their changes to be on the 4.20 branch, I volunteer to backport the PRs (https://github.com/apache/cloudstack/pull/9930, https://github.com/apache/cloudstack/pull/9708, https://github.com/apache/cloudstack/pull/9956).


Sphinx "inline" line numbers?

Just a little heads-up:
If and when https://github.com/sphinx-doc/sphinx/pull/7879 is merged, those line number will probably have to be removed from the copied text.

Ah good to know, thanks for the heads up

Can confirm, sphinx-copybutton copies line numbers. It would be really nice to have both line numbers and copy functionality at the same time. Adding span.linenos {user-select: none} helps for manual copying, but not for automatic copying, I'm afraid.

I address this issue in this [Pull Request 138](https://github.com/executablebooks/sphinx-copybutton/pull/138_


import * as Tone from 'tone' produces warnings due to module side effects

In looking at https://github.com/Tonejs/Tone.js/blob/dev/Tone/index.ts#L33, I see this line:
export const Transport = getContext().transport;

If this executes before Tone.start(), it will produce the The AudioContext was not allowed to start. It must be resumed (or created) after a user gesture on the page. warning. Same note with Master, Destination, Listener and Draw.
Yet, there is no way to import without executing these inline getContext() calls.
Everything seems to work anyway, but simply importing Tone produces warnings.
I see getters for most of these, is there a reason those aren't required?
import { getTransport } from 'tone'

Why would people waste their useful efforts into writing such terrible bad practices.

That behavior is in there for backwards compatibility since the singleton Transport object has been like that since the beginning. The intention has always been to move to getTransport(), but the question of when is a little tricky since it would likely break many projects, so it would require a good deal of messaging and some warnings before that happens. But it's also tricky to detect when people use the singleton Transport vs the singleton getter (getTransport()).

What about starting a new branch with side effects and the singleton removed?
Maybe npm i tone.js@pure or something?

Would it be enough to remove the inline Transport call on https://github.com/Tonejs/Tone.js/blob/dev/Tone/index.ts#L33 and release a new major version of Tone.js?


Implement VolumeSnapshotClass for backup and restore

Portrate logic from other analog LVM drivers to this repo
https://github.com/aleofreddi/csi-sanlock-lvm/blob/master/conf/csi-sanlock-lvm-snapshotclass.yaml
https://github.com/openebs/lvm-localpv/blob/master/pkg/driver/controller.go#L462

Hello @Gerrit91. Any updates?

Hi @wirwolf, the maintainers are currently busy and I can't tell you when it fits into the schedule. But the feature is on the wish list. If you want to accelerate the process, we are welcoming PRs and offer to review them.


Skip all but the first test in each exercise.

adjust in-repo builds to remove ignores.
for each test, output tests skipped, passed, and failed.

Implements #25.

This looks good to me, should we wait to merge this in until some help documentation is available to let them know to remove the @Ignore annotations?

I don't think so.  I'll be submitting a PR tomorrow morning that includes those instructions complete with a step-by-step tutorial for hello-world.
Since the other PR is waiting on this one, I'd recommend merging this in.

👍  Awesome, thanks.


Modified test_partial_pairwise_align_glmpca to check against baseline npz created by original implementation

The npz file that was supplying the glmpca distance matrix for this test now also supplies the result pi_BC value to check against (since the two are relevant to the same test, I figured its best to package them together).
The csv file is gone. When trying to do high precision numerical comparisons, its best not to rely on pandas to generate the csv for you, since we cannot be sure if it reduces precision while saving the numbers as strings (though probably not).
The pi_BC values used here were generated on ubuntu using code from commit 52fb6734, and for the test to pass, the norm=True needs to be passed in (which should have been the default for paste2, also the dissimilarity should be glmpca, but this is something I'll bring up in another issue), so we're now assured that we haven't broken paste2 pairwise alignment.
The PR will probably fail on some platforms, though it should pass on ubuntu-latest. If/when it does, we should modify the test to lower the tolerance for the check on these platforms.

@anushka255 - can you check if this test is passing on your local machine? I thought the reason we had to go with coarser tolerance initially was because of macos vs ubuntu. If its failing, then we'll add a macos arm runner (and modify this test to account for macos arm).

looks like macos-latest is on an M1 anyway, so we should be good going forward.
https://docs.github.com/en/actions/using-github-hosted-runners/using-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories


Add rule configuration option to the demo

ESLint has a ton of different rules, I think it would be good to add ability for users to play with different rules on the demo page. For that, we need to expose a way to configure them. It would be even cooler, if we would allow to use plugins, but I don't think that's possible with the current setup.

Plugins are a stretch, but options would be nice.

Correct me if I'm wrong, but currently, there's absolutely no way to do plugins in a browser, right? That feels like an oversight, maybe that should be looked into.

You are correct. It's not an oversight - ESLint is made primarily to be used on the command line. We only create a browserified bundle for demo purposes, but it's in no way meant to have all the functionality available on the command line.

I was thinking more in terms of Orion, but I guess, they are forking ESLint anyways, if they want plugin support, they should be able to add it in.

Yeah, they just need to create a plugin loader and it should work fine.

I started looking into this, and the biggest problem that I see is rule configuration options. We do not have central place that lists all available options for rules, and in order to make this even remotely user friendly, I would need to list config options as something like checkboxes.
I also do not want to maintain options separately from rules themselves, because that means that the page will need to be updated every time there's a change to the rule. I think in general it would be a good idea to add rules options somewhere in the ESLint code. It will help not only with this case, but also will help other people who create editor integrations, and it would be useful for documentation purposes.

I think in the short term it makes sense to just toggle options off or on rather than configuring them completely.
We do have this issue open to figure out how to deal with rule options more sanely: https://github.com/eslint/eslint/issues/967


Add support for NearMobileWallet

Description
Please include a summary of the wallet and how users can install it. Be sure to take a look at our guide on Custom Wallets to understand core concepts when integrating with Wallet Selector.
Summary
In order to onboard customers with our wallet, this PR intends to connect NearMobileWallet with Wallet Selector. Users of the Near Mobile wallet may safely store, manage, and send NEAR and other cryptocurrencies while on the go. Non-custodial mobile wallet that is simple to use.
PR summary
This PR contains the integration with Wallet selector. To achieve it, a package has been added to package.json file (@peersyst/near-mobile-signer) to handle the requests.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Hi @GuillemGarciaDev ,
I try to run this PR, and I can see wallet in list, but when I try to open it, I can't see QR code there.

Hi @GuillemGarciaDev  ,
Thank you for raising this PR.
While trying to login with near mobile wallet we noticed a QR code shown in the modal.
Can you explain how to scan the QR code, on the mobile app there is no place to scan it.

Hi @GuillemGarciaDev , Thank you for raising this PR.
While trying to login with near mobile wallet we noticed a QR code shown in the modal. Can you explain how to scan the QR code, on the mobile app there is no place to scan it.

Hi @erditkurteshiSQA!
We are preparing the version of the wallet with the new functionality. Once it is updated, you will be able to scan the QR using a scan button on the bottom bar.
When the version is uploaded I will notify you so you can test it.
Thanks for your quick reply!

Hi @GuillemGarciaDev ,
I try to run this PR, and I can see wallet in list, but when I try to open it, I can't see QR code there.

Hi @HarisSQA !
When you click on the wallet, does a modal appear or nothing at all?
Thanks for the feedback!

Hi @GuillemGarciaDev ,
I try to run this PR, and I can see wallet in list, but when I try to open it, I can't see QR code there.

Hi @HarisSQA !
When you click on the wallet, does a modal appear or nothing at all?
Thanks for the feedback!

Hi @GuillemGarciaDev ,
I had an issue only with the QR code. Modal was working fine.
Also, I have tested it now, and the QR code is shown in modal.

Hi @GuillemGarciaDev ,
I try to run this PR, and I can see wallet in list, but when I try to open it, I can't see QR code there.

Hi @HarisSQA !
When you click on the wallet, does a modal appear or nothing at all?
Thanks for the feedback!

Hi @GuillemGarciaDev ,
I had an issue only with the QR code. Modal was working fine.
Also, I have tested it now, and the QR code is shown in modal.

Hello!
Sorry for taking so long to reply. @HarisSQA we are trying to replicate your case! Thanks for the feedback.
Possibly tomorrow or the day after you will have the new version of the app available to test the wallet connector with the wallet. @HarisSQA @erditkurteshiSQA
When it's up I'll notify you!
Thanks again!

Hi! @HarisSQA @erditkurteshiSQA
The new version of NEAR Mobile wallet has been uploaded to the stores, so you can now test the PR properly. Once you import an account, you will find a scan button on the bottom bar of the app that will allow you to scan QRs.
I'll keep an eye out for any feedback.
Thanks again

Hi @GuillemGarciaDev ,
We have tested the wallet with the latest release of the app.
Noticed a problem while scanning the qr code.
Take a look at the picture below:

Hi @GuillemGarciaDev ,
We have tested the wallet with the latest release of the app. I tested it with importing two different accounts and also creating one. Noticed a problem while scanning the QR code.
Take a look at the picture below:

Hi @erditkurteshiSQA!
The issue is now fixed! It was a missing env variable. I tested the guest-book locally with the app and it seems to be working fine.
All requested PR changes have been fixed!
Regards, Guillem

Hi @GuillemGarciaDev ,
Thank you for the quick fix.
We were able to log in with iOS and tested it. However, the signMessage feature is not working, and I will provide a picture below of the error:

Also, we noticed that if we close the QR modal while trying to connect with the wallet, we still see the message Connecting to Near Mobile Wallet instead of a different message or an error message indicating that the modal is closed.

On Android, however, on the Play Store, it's still the older version of the wallet, and the login problem still persists.
Play Store app version: 1.3.0
App Store app version: 2.3.0

Hi @HarisSQA !
We have been working on the Android issue and it is already solved. You will be able to test it when it is in Play store. 👍🏻
On the other hand, could you provide me some info about how the signed message is verified? What we do is to verify the signature using the public key of the account.
const { publicKey, signature, accountId } = response;
const sign = new Uint8Array(Buffer.from(signature, "base64"));
const messageHash = new Uint8Array(sha256.digest(`NEP0413:${JSON.stringify({ receiver, message })}`));
const isValidSignature = PublicKey.fromString(publicKey).verify(messageHash, sign);

Thank you for your feedback!

Hi @GuillemGarciaDev ,
Please refer to the example referenced here in the NEP:
https://github.com/near/NEPs/blob/master/neps/nep-0413.md#references
This is how the message needs to be serialized and signed in the wallet:
https://github.com/gagdiez/near-login/blob/main/tests/authentication/wallet.ts#L60
This is how we verify on our examples:
https://github.com/near/wallet-selector/blob/main/examples/react/components/Content.tsx#L289
Looks like you have implemented some kind of an old version of the sign message looking at this line of code:
const messageHash = new Uint8Array(sha256.digest(NEP0413:${JSON.stringify({ receiver, message })}));
The NEP413 does not add a prefix string NEP0413 to the message now it uses a tag (number) there's no receiver either please see the interface:
https://github.com/near/NEPs/blob/master/neps/nep-0413.md#input-interface
Message must be signed with the above structure with params ordered alphabetically for the required params.

Hi @GuillemGarciaDev ,
Please refer to the example referenced here in the NEP: https://github.com/near/NEPs/blob/master/neps/nep-0413.md#references
This is how the message needs to be serialized and signed in the wallet: https://github.com/gagdiez/near-login/blob/main/tests/authentication/wallet.ts#L60
This is how we verify on our examples: https://github.com/near/wallet-selector/blob/main/examples/react/components/Content.tsx#L289
Looks like you have implemented some kind of an old version of the sign message looking at this line of code: const messageHash = new Uint8Array(sha256.digest(NEP0413:${JSON.stringify({ receiver, message })}));
The NEP413 does not add a prefix string NEP0413 to the message now it uses a tag (number) there's no receiver either please see the interface: https://github.com/near/NEPs/blob/master/neps/nep-0413.md#input-interface
Message must be signed with the above structure with params ordered alphabetically for the required params.

There's a new version in the Play Store to try out wallet selector in Android! There's a new update on the signMessage feature and it is now working!

Hi @GuillemGarciaDev ,
We checked the latest iOS and android version and the functionalities are working including signMessage but
there are some issues/improvement that require attention:

All Reject(s) are being ignored, and the promise never resolves.
Open the wallet-selector modal and click on NearMobileWallet to open the QR code modal. However, closing the QR code modal does not trigger the expected rejection of the signIn request.
When signing in, trying signAndSendTransaction(s) and scanning the QR code, the rejection on the wallet does not trigger any action. It should reject the signAndSendTransaction by throwing an error.
Add the ability for local signing for FunctionCall transactions without a deposit. Currently, the wallet creates a Limited Access Key (LAK) but does not utilize it.
Multiple-transactions should be able to be signed just by signing the QR code once.

As a suggestion consider comparing the UX of wallet-selector with Here wallet, since both wallets are mobile wallets.

Hi @erditkurteshiSQA and @HarisSQA it's Adrià from Peersyst team, thanks for all this feedback.
We have almost all the points that you mentioned done, except the 4th one. Is it something that you require to be done before getting approved or we can merge without this point and make an improvement later on? Also could you point us to some links or docs that better explain the local signing?
Thank you!

Hey @AdriaCarrera
The 4th point is being mentioned as an improvement and can be done later on once the other points are working fine.
The idea is to be able to sign FunctionCall transactions that don't have any deposit without confirming it on the Wallet since the FunctionCall Access-key (limited access key LAK) is on the localStorage of when we sign in with NearMobileWallet, this improves the UX.
Looks like you save the LAK in near-mobile-signer:session
For instance, one use case of such a transaction is the ability to send messages in our guest book example without needing confirmation in the wallet when no deposit is attached.
This approach is also implemented in near-api-js as demonstrated here:
https://github.com/near/near-api-js/blob/master/packages/wallet-account/src/wallet_account.ts#L306
In our wallet-connect package, we have a simpler version, as shown in here:
https://github.com/near/wallet-selector/blob/main/packages/wallet-connect/src/lib/wallet-connect.ts#L558-L566
The exact implementation might differ but the core idea is the same, I believe a similar approach can be implemented in the @peersyst/near-mobile-signer package.
If you need any more information or have more questions, feel free to reach out.

Hi @GuillemGarciaDev  @AdriaCarrera ,
Following the recent update on the Near mobile wallet and this PR, we tested it on both iOS and Android.
We are unable to login in both platforms. Please refer to the attached image for the error:

Additionally, kindly remember that the 4th point, mentioned earlier, is considered an improvement and can be addressed once the other points are resolved.
Thank you!

Hi @GuillemGarciaDev @AdriaCarrera ,
Following the recent update on the Near mobile wallet and this PR, we tested it on both iOS and Android. We are unable to login in both platforms. Please refer to the attached image for the error:
Additionally, kindly remember that the 4th point, mentioned earlier, is considered an improvement and can be addressed once the other points are resolved.
Thank you!

Hi @erditkurteshiSQA !!
This issue seems already fixed and you have available a new version with all the improvements (except the 4th one). I hope you find this fixes working as expected!
Thank you again!!

@GuillemGarciaDev
After testing the wallet with the latest version in both platform, all issues have been successfully resolved. We are now ready to merge this PR.
A release is planned for tomorrow, which will include the changes from this PR.
Thank you!

Hi, @GuillemGarciaDev & @AdriaCarrera we have published  a new version of the wallet-selector:
Release notes: https://github.com/near/wallet-selector/releases/tag/v8.7.0
Near Mobile Wallet: https://www.npmjs.com/package/@near-wallet-selector/near-mobile-wallet

Hi, @GuillemGarciaDev & @AdriaCarrera could you please take a look at the issues that need improvement:

The @peersyst/near-mobile-signer has too many unnecessary dependencies:
https://www.npmjs.com/package/@peersyst/near-mobile-signer?activeTab=dependencies
Can you please improve the bundler to include only the necessary dependencies and remove all devDependencies from NPM? - they get added to dApps which is not ideal.

The QR Code of NearMobileWallet does not open correctly when other extensions are installed such as MetaMask, Nightly, Sender etc see below video:

qr-code-issue-near-mobile-wallet.webm

Hi, @GuillemGarciaDev & @AdriaCarrera could you please take a look at the issues that need improvement:

The @peersyst/near-mobile-signer has too many unnecessary dependencies:
https://www.npmjs.com/package/@peersyst/near-mobile-signer?activeTab=dependencies
Can you please improve the bundler to include only the necessary dependencies and remove all devDependencies from NPM? - they get added to dApps which is not ideal.
Passing a callbackUrl for signMessageparams is failing can you take a look at it too?
The QR Code of NearMobileWallet does not open correctly when other extensions are installed such as MetaMask, Nightly, Sender etc see below video:

qr-code-issue-near-mobile-wallet.webm

Hi! @kujtimprenkuSQA we are working to fix these issues.
About callbackUrl issue, since NearMobileWallet is an InjectedWallet there's no reason to use a callbackUrl because the signature action is done in a mobile app.
Should we create a new PR with the new fixes?
Thank you for your feedback!

Hi, @GuillemGarciaDev & @AdriaCarrera could you please take a look at the issues that need improvement:

The @peersyst/near-mobile-signer has too many unnecessary dependencies:
https://www.npmjs.com/package/@peersyst/near-mobile-signer?activeTab=dependencies
Can you please improve the bundler to include only the necessary dependencies and remove all devDependencies from NPM? - they get added to dApps which is not ideal.
Passing a callbackUrl for signMessageparams is failing can you take a look at it too?
The QR Code of NearMobileWallet does not open correctly when other extensions are installed such as MetaMask, Nightly, Sender etc see below video:

qr-code-issue-near-mobile-wallet.webm

Hi! @kujtimprenkuSQA we are working to fix these issues.
About callbackUrl issue, since NearMobileWallet is an InjectedWallet there's no reason to use a callbackUrl because the signature action is done in a mobile app.
Should we create a new PR with the new fixes?
Thank you for your feedback!

Yes, a new PR should be made once the QR Code issue is fixed and also the dependencies for @peersyst/near-mobile-signer


Deactive throw error when property not exists

What problem does this feature solve?
Have the possibility of deactive the throw error in console when I use a property that not exists in data.
When execute in a navigator, the template render stops, even in jsfiddle, that not happen.
And in console throw an error "message2 is not defined".
https://jsfiddle.net/dzpLu0a2/
Some times, I get data from another servers and its structure is dynamic and is practical ignore missing information.
What does the proposed API look like?
Have the possibility of deactive the throw error in console and stops render when I use a property or method that not exists in the object. Maybe like this:
let qwe = new Vue({
    //...
    omitErrorInTemplate: true,
    //...
})

every property used in the template must be defined. You can however access data properties through $data like $data.message2 and that would not throw an error

Thanks!!!


[RFC]: Hot reload dab-config.json

What happened?
For reference: #67
Hot reload
The idea of hot reload allows the direct manipulation of the dab-config.json file without requiring dab start.
Use cases

mode=development allow a simpler workflow so developers can tweak the configuration file.
mode=development allow administrators to modify minimum-log-level to diagnose problems.

Canges
Config: runtime.hot-reload-enabled=true|false default true.
When set to true, depending on mode the hot reload behavior will respond to changes in the dab-config.json file.
CLI: dab start --hot-reload-enabled true
When set to true, depending on mode the hot reload behavior will respond to changes in the dab-config.json file. If the same property is set to a conflicting value in the config file, the CLI value overrides it.
Config: runtime.minimal-logging-level=information|enum default based on mode.
Currently only available as an argument in dab start this allows in-memory modification through hot reload. mode=Production is the primary use case, allowing increased (or decreased) logging during troubleshooting.
Order of precedence
In terms of precedence of value, the last thing that sets the log level wins.

Default value is based on mode

When mode=Production then Information
When mode=Development then Debug

When present in config then ignore the mode-based default.
When present in the CLI then ignore the `config setting, except...
When minimal-logging-level is set by hot reload then ignore current in-mem setting for hot reload value.

##Behavior

config.mode
config.hot-reload
Behavior

Development
True
Auto-restart the engine with each/any config file save.

Production
True
React only to log level setting (or anything else we add later).

-any-
False
Nothing at all.

For production it is important that the engine hot reload and not restarts
For development it is not important that the engine hot reload and just restarts

Some questions
?. Should runtime.hot-reload-enabled also be hot reloadable in production?
?. Can we/should we implement the production version of this first?
?. Is "restart the engine" too drastic? Is there a simpler way to do the development version?
Version
Future
What database are you using?
Azure SQL
What hosting model are you using?
Local (including CLI)
Which API approach are you accessing DAB through?
REST, GraphQL
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

The following behavior is not really hot-reload. While probably way easier to implement, this just saves a dev the few seconds from having to hit ctrl + c and run dab start

For development it is not important that the engine hot reload and just restarts

The following behavior is not really hot-reload. While probably way easier to implement, this just saves a dev the few seconds from having to hit ctrl + c and run dab start

For development it is not important that the engine hot reload and just restarts

I agree, and am wondering, aside from ease of implementation, if there is some benefit to restarting the engine this way, since you could do this yourself so easily.

?. Is "restart the engine" too drastic? Is there a simpler way to do the development version?

Yes, restart engine seems unnecessary. That is NOT same as hot reload, why don't we keep it simple - just hot reload(refresh the engine only for the properties the mode supports the refresh?

?. Can we/should we implement the production version of this first?

It was agreed upon previously that development version would be implemented first.
If the requirement is now changing, we will NOT undo what we have implemented already for the development version but consider pivoting towards completing the production scenario first in future after finishing off whatever is in progress for development

?. Should runtime.hot-reload-enabled also be hot reloadable in production?

Since we are only enabling hot reload of the loglevel in production, I think it should be fine to hot reload the runtime.hot-reload-enabled option itself, since even after enabling this - only loglevel can be reloaded.


fix(tests_suite_test.go): seed the rng, use true random numbers for users, passwords and emails

not strictly necessary for v2.0-alpha1, but the additional randomness would help with concurrent test runs on the same deis cluster

I intended to use one random number for both app, user, and test-user suffixes. This helps to correlate results after a test run. And our past experience shows that having a fake password such as "asdf1234" will be valuable if a run fails and you want to investigate the cluster state interactively.

@mboersma modified to use the same random number for all suffixes

getting this error:
_tests [rand-seed]$ make test-integration
go test -v ./...
# github.com/deis/workflow/_tests_test
./tests_suite_test.go:13: imported and not used: "github.com/onsi/ginkgo/config"
FAIL    github.com/deis/workflow/_tests [build failed]
?       github.com/deis/workflow/_tests/example-go    [no test files]
make: *** [test-integration] Error 2

@michelleN fixed, thanks


Documentation problem

Hello,
I ran into a Illuminate\Database\Eloquent\InvalidCastException: Call to undefined cast [Elegantly\MoneyCast] exception while trying to use your package.
In fact it is just a documentation problem.
In your examples you have written :
use Elegantly\MoneyCast;
The correct use is :
use Elegantly\Money\MoneyCast;
You should update your readme file.
Thanks !

Thx!


Support a REPL for .aipl

Use -i for interactive mode! =) Tab auto-complete supported for the operators.


Release working version

Hi, this is working version of github integration. We are using it for some time in production.

:+1:

@mikz is this still working?  How about releasing with a different gem name? I'm interested in a working integration.

@mikz Awesome. Would you be interested in 'forking' and releasing as an actual gem, so I can use it too? :)   Since the maintainer here seems to be MIA.
Right now I guess I'd have to use it from your branch directly from github, I'd love to have an actual gem release.  Oh, I guess this original may not have been released either! If you wanted to release it, could maybe call it "capistrano-github-releases" or something.
If you really don't want to release and take responsibility for a gem... I guess I might be forced to if I really want to use it.

@jrochkind It has been forked and released 3 years ago. https://github.com/3scale/capistrano-github
https://rubygems.org/gems/capistrano-github/versions/0.1.1 is the latest release.

Ha, awesome, thank you! You are my hero!   This repo is what I got to googling, I hadn't found your forked released version.
Well, now this is here for whoever else finds this googling!
I am not sure who the maintainer of this repo is, to tag them, but it seems like ideally they should just delete this repo now, since this is an abandoned not working version of what you have working and released in a different repo, and finding it googling is just a distraction.

@mikz I notice there's no way to file issues on your repo, as it's a fork. You might want to click the button that has GH treating it as an independent thing instead of a fork of this abandoned repo, so it's got an Issues tab.
If i could file an issue, I'd recommend replacing the README instructions to "Add this line to your application's Gemfile: gem 'capistrano-github', github: '3scale/capistrano-github'", to use the actually releaesd version as would be more standard, gem 'capistrano-github', ">= 0.1.1".  And/or to actually release a 1.0, since it's been working great for you in production for a couple years now.

@jrochkind you can open Issues now. But if you have suggestions for the README, then I'd rather see a PR 😆 It has been very low touch project for years and I don't really plan to do any work there.

okay cool! If I get around to using it, I may give you a PR for tagging it 1.0. Alas, you'd still have to do the rake release to get it to rubygems, can't do that for you (unless you wanna give me rubygems release rights too)

Closing this per https://github.com/capistrano/github/issues/9 in favor of https://github.com/3scale/capistrano-github. Future issues should be sent there.


Disable emmet tab completion in PHP tags

I am trying to disable Emmet inside PHP tags. I found how to activate tab completion in PHP files but it is not really useful because Emmet autocompletion is triggered when I am pressing the tab key instead of PHP completion.
For example, if I am pressing tab after if, it will give me <if></if> which has no sense inside <?php ?> tags.
So I tried to enable completion in PHP files outside the tags with the folowing config:
'atom-text-editor[data-grammar="text html php"]:not([mini]) :not(.php.source)':
    'tab': 'emmet:expand-abbreviation-with-tab'

But it has no effect. Is it any way to do this?
Thanks in advance.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/23463607-disable-emmet-tab-completion-in-php-tags?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github).

You are trying to add new expand abbreviation keybinding instead of removing it. See https://atom.io/docs/v1.0.0/behind-atom-keymaps-in-depth#removing-bindings

Thanks for the link, but that's not exactly what I need here. I don't want to unset the tab binding, because, if I do, I won't have any completion: neither PHP or Emmet.
By default, Emmet tab completion is not activated on PHP files. According to the documentation, I have to put these lines in my keymap file:
'atom-text-editor[data-grammar="text html php"]:not([mini])':
    'tab': 'emmet:expand-abbreviation-with-tab'

I did and it works. The problem is that it overrides PHP completion and I get html tags in my php code, which has no sense.
What I would like is the ability to allow Emmet tab completion for PHP files only outside PHP tags.

Hey! could you fixed it? I have that problem too :/

it just worked for me in the file keymap.cons, changing from:
'atom-text-editor:not([mini])':
  'tab': 'emmet:expand-abbreviation-with-tab'
to
'atom-text-editor:not([mini, php])':
  'tab': 'emmet:expand-abbreviation-with-tab'
Note: you can still use the ctrl-e shortcut (on linux) to force emmet expanding an html component in a php file though.


Lightning Storage doesn't support filenames with spaces

Bug description
Lightning Storage doesn't support filenames with spaces; it crashes with an access error if the user tries to upload a file that contains a space. We need to either add the support or make sure that:

Lightning Storage docs mention this as a limitation.
Framework prints a meaningful error if the user tries to upload a file with spaces in it.

How to reproduce the bug
Try to upload a file with a space in it to the Lightning Drive.

Error messages and logs
No response
Environment

#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow): N/A
#- PyTorch Lightning Version (e.g., 1.5.0): N/A
#- Lightning App Version (e.g., 0.5.2): 0.7.0
#- PyTorch Version (e.g., 1.10): N/A
#- Python version (e.g., 3.9): cloud
#- OS (e.g., Linux): Linux
#- CUDA/cuDNN version: N/A
#- GPU models and configuration: N/A
#- How you installed Lightning(`conda`, `pip`, source): `pip`
#- Running environment of LightningApp (e.g. local, cloud): cloud

More info
No response

I vote for error message. In the cloud, it is not so uncommon to encounter file names in links. It can be inconvenient to work with them if the spaces need to be encoded with %20.

@awaelchli, hi! 👋

It can be inconvenient to work with them if the spaces are encoded as %20.
But should this be a decision enforced by Lightning Storage or something we leave for the Lightning App developers to decide?

In my eyes, Lightning Storage API is intended to be used alongside fs writes and reads primarily to store the app artifacts consistently between app restarts and recreations (and even share it between apps). Why would Lightning Storage API care about spaces in this use case? These files aren't being exposed as links anywhere. And, if the Lightning App developer decides to expose these files as links, they probably should know to avoid spaces, and if they choose not to, it's OK to support them still, IMO.
Of course, we should be converting spaces to-and-from %20 in the framework, and in the web app Artifacts UI, the user should see spaces, not %20, everywhere.
My vote is for adding the whitespace support because it streamlines the development experience instead of hindering it by showing an error.

That makes sense. The storage call could occur anytime in an app after running a long time so an error is not useful. Also because the developer of the app might not wand to handle this themselves for their users.
Feel free to also include such concrete proposals and considerations/learnings into the issue description directly next time.


Kubernetes namespace watches sometimes hang indefinitely

We've observed a bug in the kubernetes namer where name resolution (and requests) hang indefinitely when initializing a watch on a second namespace.
For example, in a test cluster, we launch an application into the prod namespace and do continuous deployment to a pre-prod namespace that may be accessed with an override.
The configuration contains:
namers:
- kind: io.l5d.experimental.k8s
  prefix: /ns
  tlsWithoutValidation: true
  authTokenFile: /var/run/secrets/kubernetes.io/serviceaccount/token

and a base dtab like:
  /iface => /io.l5d.k8s/prod;
  /method => /$/io.buoyant.http.anyMethodPfx/iface ;
  /http/1.1 => /method ;
  ...

Requests are routed to the pre-prod namespace with an override like:
/iface => /io.l5d.k8s/pre-prod | /$/fail;

Initially, we observe requests through prod operating as expected, resolving through the k8s namer:
D 0126 23:45:28.991 THREAD29: k8s initializing service0
...
D 0126 23:45:29.327 THREAD30: k8s ns prod initial state: service0
...
D 0126 23:45:42.955 THREAD30: k8s lookup: /pre-prod/http/service0 /pre-prod/http/service0
D 0126 23:45:42.956 THREAD30: k8s initializing pre-prod

And we see no further k8s logging after this.  Furthermore, because the namer never satisfies this  lookup, the request never succeeds and remains pending until the client disconnects
Something appears to be wrong in the kubernetes namer that prevents this second watch from succeeding.

linkerd-0.2.1-SNAPSHOT confirmed working
testing with linkerd-helium:sig3 (linkerd-0.2.1-SNAPSHOT), in namespace sig, works with prod and  pre-prod:
$ curl -H 'L5d-sample: 0' -H 'Dtab-local: /ns/sig=>/ns/prod;' http://<IP_ADDRESS>
$ curl -H 'L5d-sample: 0' -H 'Dtab-local: /ns/sig=>/ns/pre-prod;' http://<IP_ADDRESS>

$ kubectl --namespace=sig logs sproxy-ix56f -c l5d -f
...
D 0226 05:53:17.541 THREAD31: k8s lookup: /prod/router/web /prod/router/web
D 0226 05:53:17.546 THREAD31: k8s initializing prod
D 0226 05:53:17.546 THREAD31: k8s initializing2 prod
D 0226 05:53:17.546 THREAD31: k8s endpoints Promise@1296927950(state=Transforming(List(<function1>, <function1>),Promise@280773399(state=Transforming(List(<function1>),Promise@1529645848(state=Done(Return(<function1>)))))))
D 0226 05:53:17.577 THREAD31 org.jboss.netty.handler.ssl.SslHandler: [id: 0x967a7988, /<IP_ADDRESS>:39833 => kubernetes.default.cluster.local/<IP_ADDRESS>:443] HANDSHAKEN: TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA
D 0226 05:53:17.588 THREAD31: k8s ns prod initial state: auth, users, sessions, web, projects, sproxy, events, accounts
D 0226 05:53:17.588 THREAD31: k8s ns prod service web found
D 0226 05:53:17.588 THREAD31: k8s ns prod service web port router found + /
I 0226 05:53:17.594 THREAD31: Using web as the TLS common name for /ns/prod/router/web
I 0226 05:53:17.595 THREAD31: Using web as the TLS common name for /ns/prod/router/web
D 0226 05:53:17.622 THREAD28 org.jboss.netty.handler.ssl.SslHandler: [id: 0x5e2f4847, /<IP_ADDRESS>:56384 => /<IP_ADDRESS>:4140] HANDSHAKEN: TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384
...
D 0226 05:53:33.712 THREAD31: k8s lookup: /pre-prod/router/web /pre-prod/router/web
D 0226 05:53:33.737 THREAD31: k8s initializing pre-prod
D 0226 05:53:33.737 THREAD31: k8s initializing2 pre-prod
D 0226 05:53:33.737 THREAD31: k8s endpoints Promise@812482979(state=Transforming(List(<function1>, <function1>),Promise@291983516(state=Transforming(List(<function1>),Promise@1672547676(state=Done(Return(<function1>)))))))
D 0226 05:53:33.763 THREAD30 org.jboss.netty.handler.ssl.SslHandler: [id: 0x43ec29ea, /<IP_ADDRESS>:39864 => kubernetes.default.cluster.local/<IP_ADDRESS>:443] HANDSHAKEN: TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA
D 0226 05:53:33.771 THREAD30: k8s ns pre-prod initial state: auth, users, sessions, web, projects, events, accounts
D 0226 05:53:33.771 THREAD30: k8s ns pre-prod service web found
D 0226 05:53:33.771 THREAD30: k8s ns pre-prod service web port router found + /
I 0226 05:53:33.777 THREAD30: Using web as the TLS common name for /ns/pre-prod/router/web
I 0226 05:53:33.777 THREAD30: Using web as the TLS common name for /ns/pre-prod/router/web
D 0226 05:53:33.814 THREAD31 org.jboss.netty.handler.ssl.SslHandler: [id: 0x299458a2, /<IP_ADDRESS>:54610 => /<IP_ADDRESS>:4140] HANDSHAKEN: TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384

linkerd-0.1.1-SNAPSHOT confirmed failing
testing with linkerd-helium:sig (linkerd-0.1.1-SNAPSHOT), in namespace sig, hanging on pre-prod:
$ curl -H 'L5d-sample: 0' -H 'Dtab-local: /ns/sig=>/ns/pre-prod;' http://<IP_ADDRESS>

$ kubectl --namespace=sig logs sproxy-gmheu -c l5d -f
...
D 0226 03:11:52.455 THREAD29: k8s lookup: /pre-prod/router/web /pre-prod/router/web
D 0226 03:11:52.455 THREAD29: k8s initializing pre-prod
E 0226 03:12:36.760 THREAD29: Exception propagated to the root monitor!
com.twitter.finagle.CancelledRequestException: request cancelled
    at com.twitter.finagle.NoStacktrace(Unknown Source)

E 0226 03:12:36.761 THREAD29: service failure
com.twitter.finagle.CancelledRequestException: request cancelled
    at com.twitter.finagle.NoStacktrace(Unknown Source)

E 0226 03:12:36.761 THREAD29: Exception propagated to the root monitor!
com.twitter.finagle.CancelledRequestException: request cancelled
    at com.twitter.finagle.NoStacktrace(Unknown Source)

D 0226 03:12:36.762 THREAD29: ChannelStatsHandler caught an exception
java.nio.channels.ClosedChannelException
    at org.jboss.netty.channel.socket.nio.AbstractNioWorker.cleanUpWriteBuffer(AbstractNioWorker.java:433)
    at org.jboss.netty.channel.socket.nio.AbstractNioWorker.writeFromUserCode(AbstractNioWorker.java:128)

This issue no longer manifests following https://github.com/BuoyantIO/linkerd/pull/137, though an underlying bug may still exist.

Reported issue to finagle:
https://github.com/twitter/finagle/issues/471

I'm still seeing this issue with 0.3.0. Any tips on the best way to debug this?

@jimmidyson if you run linkerd with the -log.level=DEBUG and -com.twitter.finagle.tracing.debugTrace=true flags, your logs should be a bit more verbose.
If you can grab a snapshot of the stats in on :9990/admin/metrics.json when the process is in this state, that would be helpful, too.
Also, we'll be better able to help if you share your linkerd config (especially your dtab) -- this is a common source of confusion.
Finally, would you mind explaining how this state arises?  What version of kubernetes you're running, etc?
Thanks!

Thanks for the tips. Very possible this is just a misunderstanding about the dtab - have found it slightly confusing so have tried to make it as simple as possible:
admin:
  port: 9990

namers:
- kind: io.l5d.experimental.k8s
  host: localhost
  port: 8443
  tls: true
  tlsWithoutValidation: true
  authTokenFile: /home/jdyson/tmp/token

routers:
- protocol: http
  baseDtab: |
    /http/1.1/GET => /io.l5d.k8s/zipkin/http;
  label: int
  servers:
  - port: 4140
    ip: <IP_ADDRESS>

Running curl -H 'Host: zipkin-web-console' localhost:4140. The service zipkin-web-console does exist in the namespace zipkin with the named port http. It's almost like it's not querying at all.
Here's the (hopefully) relevant part of the logs:
D 0411 21:25:53.631 THREAD19: k8s lookup: /zipkin/http/zipkin-web-console /zipkin/http/zipkin-web-console
D 0411 21:25:53.649 THREAD19: k8s initializing zipkin
0411 21:25:53.652 36e4dfa7003f8c8b.36e4dfa7003f8c8b<:36e4dfa7003f8c8b] BinaryAnnotation(namer.dtab.base,/http/1.1/GET=>/io.l5d.k8s/zipkin/http)
0411 21:25:53.652 36e4dfa7003f8c8b.36e4dfa7003f8c8b<:36e4dfa7003f8c8b] BinaryAnnotation(namer.dtab.local,)
0411 21:25:53.652 36e4dfa7003f8c8b.36e4dfa7003f8c8b<:36e4dfa7003f8c8b] BinaryAnnotation(namer.path,/http/1.1/GET/zipkin-web-console)

Here's the output from :9990/admin/metrics.json: https://gist.github.com/jimmidyson/c05a0e7959b52a5dde50bb2837ea3782
If it does turn out to be config problems, hanging isn't great of course.

Sounds like a proper bug. Reopening and assigning to Oliver.

After some internal discussion, Alex will take a look at this.

Hi @jimmidyson, thanks for the info.  This behavior can happen if linkerd in unable to connect to the k8s API.  We've got a couple of issues tracking making this situation easier to debug including adding better logging (#272) and adding a timeout for name resolution (#273).
In your particular case it looks like you've got the k8s namer configured to talk to localhost.  Do you have the k8s master running on localhost?  Or are you running through a proxy?  In the case of a proxy, it's likely that TLS won't work and you'll need to turn that off in your config.
Can you check if you're able to hit the k8s API manually on localhost:8443 over https?

@adleong I'm running this just for dev, local builds from master & running locally, & I have Kubernetes (well OpenShift actually) running locally too, accessible at https://localhost:8443. TLS works but of course with cert validation disabled (as I tried to do with the config). So yeah curl https://localhost:8443 works fine. I've actually also tried setting up a little proxy to capture requests & see that no requests reach the proxy on localhost. The timeout for name resolution is really important - it would help with quick diagnosis of issues like this but still in production consumers would want to have a quick response if lookups fail, just like DNS failures.
Trying to debug through the namer but being a bit of a n00b to scala it makes it a bit difficult. One thing I have noticed is that the filters don't seem to work properly. I added a quick System.out (yeah, dirty but effective :-b) to both the SetHostFilter & AuthFilter & the SetHostFilter doesn't get called for some reason:
Authfilter Request("GET /api/v1/namespaces/zipkin/endpoints", from <IP_ADDRESS>/<IP_ADDRESS>:0)

When I swap round the order of filtered calls in namer/k8s/src/main/scala/io/l5d/k8s.scala it does actually call the SetHostFilter too:
SetHostFilter Request("GET /api/v1/namespaces/zipkin/endpoints", from <IP_ADDRESS>/<IP_ADDRESS>:0)
Authfilter Request("GET /api/v1/namespaces/zipkin/endpoints", from <IP_ADDRESS>/<IP_ADDRESS>:0)

So it could be something odd in the filter stack going on?

So testing the calls via curl:
curl -k -H "Authorization: Bearer $(cat ~/tmp/token)" https://localhost:8443/api/v1/namespaces/zipkin/endpoints

works fine so I know both the API server URL & token file are correct.

thanks for digging into this, @jimmidyson!
Agreed that timeouts are important, we're adding them soon.  As for the behavior you discovered with the filters: that's very strange.  The behavior you found suggests that something is going wrong in the AuthFilter, but if you take a look at the contents of that filter, it's pretty simple.
We're going to try setting up kuberenetes locally to see if we can reproduce the issue.  In the meantime, I encourage you to keep digging and let us know if you find anything.

Could you also post the full linkerd log when running with -log.level=DEBUG and -com.twitter.finagle.tracing.debugTrace=true and exercising the issue?

I've turned off auth & removed the auth filter... & querying works... I cannot explain that at all as you said AuthFilter is ridiculously simple.

It's this line in AuthFilter that's hanging:
req.headerMap("Authorization") = s"Bearer $token"

Here's the full log.

I've found a potential explanation.  Does your auth token file end with a newline?  Attempting to set a header value with a new line causes an exception.  Unfortunately, that exception is getting swallowed and not logged today, which makes this kind of thing very hard to debug.  We'll definitely fix that in #272.  In the meantime, try stripping out any newlines from your token file and see if that helps.

You've got it - new line removed & works great. Well it's a newline char without a new line (just learnt about the eol option in vim). That is really annoying - sorry for the noise (but yeah something in the logs would be great).

Amazing.

Glad to hear it!  This definitely showcases the need for better error logging.  Stay tuned.

Thanks for your help figuring this one out - now I can get on & play more :)


Display HTML is blank under Widget Builder until it is clicked

Describe the bug
Display HTML is blank under Widget Builder until it is clicked
To Reproduce
Steps to reproduce the behavior:

Go to Widget Builder
Select a Widget and click edit.
Click on Display Tab.
Display HTML is blank

Expected behavior
Display HTML should be populated with saved content.
Screenshots

Work around is to click into the Display HTML field, then it renders.


Scrolling simulation is applied even if the scrolling content dimension is changed.

Hi Team,
I have implemented a scrollable container with horizontal direction. The scrolling simulation is applied when I changed the content dimension of my scrollable container after reached the maxScrollExtend position. Is this a behavior? . If it's a behavior, please let me know that how to restrict it while updating the scrolling content dimension. I need to restrict it for my use case.
Steps to Reproduce

Run the below sample.
Find the code snippets below

class NestedDemo extends StatefulWidget {
  NestedDemo({Key? key}) : super(key: key);

  @override
  _NestedDemoState createState() => _NestedDemoState();
}

class _NestedDemoState extends State<NestedDemo> {
  double width = 500;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: SingleChildScrollView(
        scrollDirection: Axis.horizontal,
        child: Container(
          color: Colors.lightBlue[100],
          height: 350,
          width: width,
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () {
          width = 250.0;
          setState(() {});
        },
      ),
    );
  }
}

https://user-images.githubusercontent.com/60499432/126340395-d7dee34f-f928-465d-b7f6-6288cd96f6e2.mp4
Expected results: Simulation should not be applied.
Actual results: Simulation is applied even if changed the content dimension.

flutter analyze
No issues found! (ran in 80.0s)

flutter doctor -v
[√] Flutter (Channel stable, 2.2.2, on Microsoft Windows [Version 10.0.18363.1440], locale en-US)
    • Flutter version 2.2.2 at C:\src\flutter
    • Framework revision d79295af24 (6 weeks ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\AshokKuvaraja\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Professional 2019 16.4.5)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional
    • Visual Studio Professional 2019 version 16.4.29806.167
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 3.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 34.0.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[√] VS Code, 64-bit edition (version 1.58.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.24.0

[√] Connected device (4 available)
    • Moto C Plus (mobile)         • ZZ2222FDR3    • android-arm    • Android 7.0 (API 24)
    • AOSP on IA Emulator (mobile) • emulator-5554 • android-x86    • Android 9 (API 28) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.18363.1440]
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 91.0.4472.124

• No issues found!

Hi @ashok-kuvaraja,
It is a expected behavior, And what you are trying to achieve is not an issue with Flutter. I would recommend you to ask such questions on community/ forums like StackOverflow or Reddit.
I am going to close this issue as invalid if you disagree feel free to comment and I will reopen it.
Thank you.


Static localization support

Allow static localization of some hardcoded texts

German localization (defaults to English localization)

OK, I prepared a pull request.
I am new to git/github and found no other way to only select the changes for static-localization-support, than to create a new branch and undo the other changes I made to my master branch. I hope it is okay that way?

I'll take a look and see how things look. The only bit that might need adjusting is the modifications made to main.js. That file is generated from a Grunt task so the YAML Front Matter would have to be added there I believe. Because if anyone ever modifies the source _main.js file it will overwrite what you did to main.js.
Will have to test because I'm not sure if inserting YAML into _main.js will break things as far as the minification/uglifier Grunt tasks are concerned.

@yonojoy Had a chance to quickly test this out and its looking really good. Only issue I see is finding a way to include {{ site.data.messages.locales[site.locale].overview }} and the blank YAML Front Matter in _main.js so it can be uglified with grunt uglify or grunt scripts. Those tasks are stripping out the ---. I'll have to poke around and see if there is a way to force it through.
I like what you did with messages.yml, I had no idea you could do the referencing in YAML as you did for en_US and en_UK. Very smart stuff :smile

I don't fully understand how _main.js is generated. But I think replacing Overview with {{ site.data.messages.locales[site.locale].overview }} should not be a problem? Regarding the YAML Front Matter: maybe adding a grunt concat task and an empty YAML Front Matter file might help? However I wont have time to look much deeper into it atm.

That's correct. {{ site.data.messages.locales[site.locale].overview }} replacing Overview works just fine. Looking to find a way to keep the YAML Front Matter intact. The Grunt uglifier tasks strips all of that stuff out. I'm sure there's a way to do it just need to investigate.

Thanks @yonojoy for your work on this. I figured out how to append the blank YAML Front Matter header to _main.js after it was processed. Just had to add a new Grunt task to do it.


Update to ink! 5

Update examples to comply with ink! 5

Due to a missing dependency on uapi, which has not been released with ink yet, there are expected failures for the given contracts: erc1155, call-runtime, multisig, upgradeable-contracts. (8 failures in CI)

Due to a missing dependency on uapi, which has not been released with ink yet, there are expected failures for the given contracts: erc1155, call-runtime, multisig, upgradeable-contracts. (8 failures in CI)

@pgherveou has published ink-pallet-contracts-uapi by now, there'll be a re-release under a different name soon, but could you adapt this PR to the already published crate so that we see CI running?

I need ink! version which is using ink-pallet-contracts-uapi crate, so the option could be to set contracts deps on ink! from master branch., but in this case I need also to apply the change call -> call_builder. WDYT?


Split out a tools/setup from tools/ to decrease clutter

Fixes #708

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

@timabbott please review

Looks great, merged, thanks @umairwaheed !

Thanks


How does the Matter device enable one-click restoration of factory settings?

Test issue(s)
Typically, Matter devices come with an initial QR code and a pairing code used to connect to the user's Controller, and the app the user is using usually has the ability to clear the device information, allowing the user to rescan the initial QR code on the device to re-pair it.
I would like to know how this is done. Is there a specific CLUSTER for such a feature?
Does chip-tool have a command to restore a paired Matter device to its initial state? My current understanding is the chip-tool unpair command, but it only unpairs and does not restore the device to its initial state. I would like to know how I should go about doing this.
Looking forward to your reply sincerely! : )
Platform
No response
Anything else?
No response

There is no cluster command for factory reset.
The only available relevant cluster command is RemoveFabric, which is what chip-tool unpair does.  Some devices may at that point reset themselves; some might not.


Fixed typos: initalization --> initialization

[x] Tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Not applicable

Thanks for the contribution!


support ed25519 address

Background
As of now the toAddress is required to be an EVM address. However, as we move towards new non-EVM ecosystems we would need to allow new types of addresses. The first one is to support public keys on the ed25519 elliptic curve. This would enable Solana addresses.
This change request should serve as a point of discussion and should not be taken as the final solution.
Additionally, it would make sense to automatically display the toAddress field if the toChain does not support the EVM address.

Delayed until further notice.


[catalog-backend] Fan-Out/Nested Location Fetching

Feature Suggestion
Add support for referencing a location which is a list of more locations to fetch (with a max-deplth limit for sanity).
e.g. I submit a location like so:
{
  "type": "github",
  "target": "https://github.com/myorg/services-list/blob/master/services.yaml"
}

Inside the services.yaml file I'd like to list more locations to fetch from, like so:
apiVersion: backstage.io/v1beta1
kind: Location
metadata:
  name: service-a
spec:
  type: github
  target: https://github.com/myorg/service-a/blob/master/meta.yaml
---
apiVersion: backstage.io/v1beta1
kind: Location
metadata:
  name: service-b
spec:
  type: github
  target: https://github.com/myorg/service-b/blob/master/meta.yaml
---
apiVersion: backstage.io/v1beta1
kind: Location
metadata:
  name: service-c
spec:
  type: github
  target: https://github.com/myorg/service-c/blob/master/meta.yaml

I saw https://github.com/spotify/backstage/issues/1013, but that didn't seem like the same idea, so I wanted to post this as a separate feature-request
Possible Implementation
Add support for the LocationReader implementations to read kind: Location from upstream Locations to support mixed content.
Example:
apiVersion: backstage.io/v1beta1
kind: Component
metadata:
  name: my-service
spec:
  type: service
---
apiVersion: backstage.io/v1beta1
kind: Location
metadata:
  name: service-a
spec:
  type: github
  target: https://github.com/myorg/service-a/blob/master/meta.yaml

Context
Our developers currently maintain a set of service meta-data and are generally good at keeping it up-to-date as it lives in their service codebase. If we move it out to a central place so we can reference it in a single location then there is the possibility of it going stale or losing sync with the services themselves.
The other option is to add 300+ locations to the catalog-backend, but that doesn't seem like a good ide (correct me if I'm wrong)

Hi!
We've had on our radar for a while how to most conveniently let people manage a large number of entities. There are a few things we can do.
Firstly, we already support having multiple entities in one file. That can help a bit, but not for the kind of situation you mention.
Then you mention the possibility to have a kind of Location. I have actually considered that as well and I think that will be how we eventually model these locations "internally" - however, maybe it does not help you very much to store them in actual yaml files, since you would then be back to writing one entry per entity anyway.
I've been toying with the idea however of being able to add a kind called EntitiesRoot or similar, which is special in the way that it lies at a root catalog somewhere, and it automatically makes sure that any entities in yaml files in subfolders (down to a certain depth) get automatically managed, without explicitly naming them or pointing them out. That way you would register one single location, and the entire tree below that becomes managed.
Does that sound like it would address your situation nicely?

Hi @freben ,
I think I'd personally prefer the ability to load from multiple repositories, mainly because our component definitions will live in the same repositories as the code, so it's easier on the Devs to maintain.
Perhaps, rather than extending the current GitHub location reader, an additional GithubMultiRepositoryReader could be created, that way the simpler reader doesn't become overly complex?
The second suggestion of reading nested subdirectories would be helpful for some use cases, just not mine :stuck_out_tongue:

Ah right, because you are actually pointing to different repos in your example - sorry.
The good news is that our loaders already have the ability to do recursion. If we add the Location kind, it would work just like your example. Interesting.

@daviddyball This got me fired up so I had to implement it :) See #1276 and check if you think it would work for this use case.

@freben amazing. Just pullsed the branch and tested my use case and it's all working perfectly 👍

Hey this is merged now, but note that i simplified the name to just Location instead of LocationRef if you start using it!


Mast: Watch (?-i:b1b11a6ae95402e1151f600a1815d33b)(?#apnews)

Mast requests the watch of the watch_keyword (?-i:b1b11a6ae95402e1151f600a1815d33b)(?#apnews). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:b1b11a6ae95402e1151f600a1815d33b)(?#apnews) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by NobodyNada in Charcoal HQ


In the docs, do you usually keep it all JavaScript?

Do you want to request a feature or report a bug?
I was reading https://react-redux.js.org/api/hooks
and all of a sudden, there is this line
const result : any = useSelector(selector : Function, equalityFn? : Function)
do you usually keep all code to JavaScript because some React / Redux developers use JavaScript?
What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have dependencies other than React. Paste the link to a CodeSandbox (https://codesandbox.io/s/new) or RN Snack (https://snack.expo.io/) example below:
What is the expected behavior?
Which versions of React, ReactDOM/React Native, Redux, and React Redux are you using?  Which browser and OS are affected by this issue? Did this work in previous versions of React Redux?

We've mostly written our docs as just JS, yes, but for the API references, having some form of type definitions is helpful to clarify what the expected arguments are.

We've mostly written our docs as just JS, yes, but for the API references, having some form of type definitions is helpful to clarify what the expected arguments are.

I think it is good to say what the type is, but having JS and then TS mixed in the docs can be confusing IMO.


Fails on CUDNN_STATUS_EXECUTION_FAILED

This is the same error that was reported by @gsh2014 in #4, but I figure it'd be better to have it as a separate issue. I'm running into the same problem:
Namespace(action_embed_size=128, answer_prune=True, asdl_file='asdl/lang/py3/py3_asdl.simplified.txt', att_vec_size=256, batch_size=10, beam_size=15, clip_grad=5.0, column_att='affine', cuda=True, decay_lr_every_epoch=False, decode_max_time_step=100, decoder_word_dropout=0.0, dev_file='data/conala/dev.var_str_sep.bin', dropout=0.0, embed_size=128, eval_top_pred_only=False, evaluator='conala_evaluator', field_embed_size=64, glorot_init=True, glove_embed_path=None, hidden_size=256, lang='python', load_model=None, log_every=50, lr=0.001, lr_decay=0.5, lr_decay_after_epoch=15, lstm='lstm', max_epoch=50, max_num_trial=5, mode='train', negative_sample_type='best', no_copy=False, no_input_feed=False, no_parent_field_embed=False, no_parent_field_type_embed=True, no_parent_production_embed=True, no_parent_state=False, no_query_vec_to_action_map=False, optimizer='Adam', parser='default_parser', patience=5, primitive_token_label_smoothing=0.0, ptrnet_hidden_dim=32, query_vec_to_action_diff_map=False, readout='identity', reset_optimizer=False, sample_size=5, save_all_models=False, save_decode_to=None, save_to='saved_models/conala/model.sup.conala.lstm.hidden256.embed128.action128.field64.type64.dr0.0.lr0.001.lr_de0.5.lr_da15.beam15.vocab.var_str_sep.src_freq3.code_freq3.bin.train.var_str_sep.bin.glorot.par_state.seed0', seed=0, sql_db_file=None, src_token_label_smoothing=0.0, sup_attention=False, test_file=None, train_file='data/conala/train.var_str_sep.bin', transition_system='python3', type_embed_size=64, uniform_init=None, valid_every_epoch=1, valid_metric='acc', verbose=False, vocab='data/conala/vocab.var_str_sep.src_freq3.code_freq3.bin', word_dropout=0.0)
Traceback (most recent call last):
  File "exp.py", line 251, in <module>
    train(args)
  File "exp.py", line 71, in train
    if args.cuda: model.cuda()
  File "/home/gneubig/anaconda3/envs/py3torch3cuda9/lib/python3.6/site-packages/torch/nn/modules/module.py", line 216, in cuda
    return self._apply(lambda t: t.cuda(device))
  File "/home/gneubig/anaconda3/envs/py3torch3cuda9/lib/python3.6/site-packages/torch/nn/modules/module.py", line 146, in _apply
    module._apply(fn)
  File "/home/gneubig/anaconda3/envs/py3torch3cuda9/lib/python3.6/site-packages/torch/nn/modules/rnn.py", line 123, in _apply
    self.flatten_parameters()
  File "/home/gneubig/anaconda3/envs/py3torch3cuda9/lib/python3.6/site-packages/torch/nn/modules/rnn.py", line 102, in flatten_parameters
    fn.rnn_desc = rnn.init_rnn_descriptor(fn, handle)
  File "/home/gneubig/anaconda3/envs/py3torch3cuda9/lib/python3.6/site-packages/torch/backends/cudnn/rnn.py", line 42, in init_rnn_descriptor
    cudnn.DropoutDescriptor(handle, dropout_p, fn.dropout_seed)
  File "/home/gneubig/anaconda3/envs/py3torch3cuda9/lib/python3.6/site-packages/torch/backends/cudnn/__init__.py", line 207, in __init__
    self._set(dropout, seed)
  File "/home/gneubig/anaconda3/envs/py3torch3cuda9/lib/python3.6/site-packages/torch/backends/cudnn/__init__.py", line 232, in _set
    ctypes.c_ulonglong(seed),
  File "/home/gneubig/anaconda3/envs/py3torch3cuda9/lib/python3.6/site-packages/torch/backends/cudnn/__init__.py", line 283, in check_error
    raise CuDNNError(status)
torch.backends.cudnn.CuDNNError: 8: b'CUDNN_STATUS_EXECUTION_FAILED'

I'm not sure if it's related but, I did find this: https://github.com/pytorch/pytorch/issues/953
That post seemed to indicate it might be an out-of-memory error, so I tried to reduce the batch size and size of the hidden dimensions, but this didn't change anything...
@pcyin: I was able to reproduce the error on ogma, so maybe you'd be able to as well?

FYI: I've found out that the problem was because I'm using a 2080Ti, which fails when you use CUDA less than version 10. The environment suggested by TranX is using CUDA version 9. I started working on fixing this by fixing this issue https://github.com/pcyin/tranX/issues/10 and making a more modern environment, but the newest version of PyTorch doesn't work with tranX, and there are several places that need fixing. Will update when I finish.

Should be fixed by https://github.com/pcyin/tranX/pull/15

Hi, Prof. Neubig. I merged your PR onto my fork manually but there were still some issues.
L 247 From
action_prob_var = torch.cat([torch.cat(action_probs_i).log().sum() for action_probs_i in action_probs])
->
action_prob_var = torch.stack([torch.stack(action_probs_i).log().sum() for action_probs_i in action_probs])
L 459 From
new_hyp_scores = torch.cat([x['new_hyp_score'] for x in new_hyp_meta])
->
new_hyp_scores = torch.stack([x['new_hyp_score'].cuda() for x in new_hyp_meta])


consider making pomodoro bookmarklet show clock or progressbar

a clock or progress bar will double as feedback that the bookmarklet actually ran and as indication of how much time is left
https://github.com/hchiam/learning-js/blob/main/bookmarklets/pomodoro.js

Aside: https://www.nextofwindows.com/windows-trick-how-to-make-your-computer-to-speak-out-time-at-every-hour
'to announce hour on Windows
'https://www.nextofwindows.com/windows-trick-how-to-make-your-computer-to-speak-out-time-at-every-hour

Dim what, voice
 what = "It is " & hour(time) & " O'clock"
 Set voice = CreateObject("sapi.spvoice")
 voice.Speak what


minikube 0.17.1 into 17.03

Update Minikube in 17.03
This is just a cherry-pick

I'm cherry-picking this by hand as -x wasn't used.

Oops, thanks :)


[flutter doctor] Android Studio Electric Eel doesn't show app version

The version name is missing with Android Studio Electric Eel
[✓] Android Studio (version xxxx.x) <- This version name
Android Studio Chipmunk (Working as expected)
[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

Android Studio Electric Eel (Version name is missing)
[✓] Android Studio
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

This also happens with Android Studio Dolphin which is currently in beta.

Android studio showing incompatible dart and flutter plugin. while flutter doctor has no issues.
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.3.2, on macOS 12.5.1 21G83 darwin-x64, locale
en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.3)
[✓] Android Studio (version 2021.2)
[✓] VS Code (version 1.69.2)
[✓] Connected device (2 available)
[✓] HTTP Host Availability
• No issues found!

Hi!
Im trying to add flutter to a native android app in Android Dolphin and its impossible to me. I have followed the tutorial but im not able to launch a flutter screen. I only want to change previous android layouts to flutter designs.
can anyone help me?

Hi! Im trying to add flutter to a native android app in Android Dolphin and its impossible to me. I have followed the tutorial but im not able to launch a flutter screen. I only want to change previous android layouts to flutter designs.
can anyone help me?

It seems you are asking for assistance on personal code. Please see https://flutter.dev/community for resources and ask questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

@TahaTesser  is this still happening?  Is flutter fully compatible with electric eel which is RC?


Multi-time support for exsiting webhooks

Currently, notify doesn't support reusing the same webhook multiple time with different keys/tokens.
For example, if we have the following entry in the config file:-
discord_webhook_url: https://discord.com/api/webhooks/7730624177430/XXXX
discord_username: A
discord: true

discord_webhook_url: https://discord.com/api/webhooks/77306241774308/YYYY
discord_username: B
discord: true

Notify will only post to 2nd discord_webhook_url and the same goes for any other webhook, slack, telegram.
With this support -
For the cases when we want to post the same results on multiple servers/channels, we can simply define the webhook multiple times in config with different values to achieve this.

Related issue - https://github.com/projectdiscovery/notify/issues/40


Network byte compare fix

URI:https://xx.xx.xx.xx/xyz/openbmc_project/network//eth0/attr/MACAddress, method:Get, args:{}
Response code:200, Content:{
  "data": "70:e2:84:14:2a:8",
  "message": "200 OK",
  "status": "ok"
}
Configure Valid MAC And Verify :: Configure valid MAC and verify.     | FAIL |
Parent suite setup failed:
MAC address does not exist.: '70:e2:84:14:2a:08' does not contain '70:e2:84:14:2a:8'

The MAC last nibble is 08 so a string compare won't work here..

In rvewiew https://gerrit.openbmc-project.xyz/#/c/9098/


Could we have a list of string passing into the setup.keymap?

Description
As title. Is it possible to achieve something like this? Or there is some existing way to achieve the same result (entering fullscreen and open the preview). Thanks in advance!
keymap = {
  builtin = {
    ['<F1>']        = { 'toggle-fullscreen', 'toggle-preview' },
    -- ['<F2>']        = 'toggle-preview',
    -- ['<F3>']        = 'toggle-preview-cw',
    -- ['<F4>']        = 'toggle-preview-ccw',
    -- ["<S-down>"]    = "preview-page-down",
    -- ["<S-up>"]      = "preview-page-up",
    -- ["<S-left>"]    = "preview-page-reset",
  }
},

Not possible, what’s the point of this if I may ask?

To @ibhagwan:

[...], what’s the point of this if I may ask?

I recently like the idea of minimal UI before manually calling any functions. So I move fzf-lua to the center of my screen and disable the preview window. When I need to preview a hovered item, I will always call the two actions in sequence: toggle-preview, toggle-fullscreen. But this will cost two keypress. It will be good if I can just use one keymap to complete the task.

I have a similar workflow in my neovim config to review git changes, I open git status, make it full screen and then make the tmux pane also fullscreen using <tmux prefix>-Z, IMHO, this is better achieved using your own customization, I don’t plan on adding this functionality into fzf-lua as it adds unnecessary complexity into the bind logic.
My code if you wish to take a look:
https://github.com/ibhagwan/nvim-lua/blob/6fc689c5f14fe9f0850f47622613ddb07ec51061/lua/plugins/fzf-lua/init.lua#L161-L186

@ibhagwan You're right, I complicated the problem. With your config, I came up with the following solution: (You might know how to improve it)
local NOREF_NOERR_TRUNC = { noremap = true, silent = true, nowait = true }
vim.keymap.set('t', '<F1>', function ()
  if vim.bo.filetype == 'fzf' then
    require('fzf-lua.win').toggle_preview()
    require('fzf-lua.win').toggle_fullscreen()
  end
end, NOREF_NOERR_TRUNC)

Thanks again for your beautiful plugin -- fzf-lua :) This issue can be safely closed now!

Great! will close :-)


Update Vuln parser to generate GraphQL nodes

The vuln parser pkg/ingestor/parser/vuln code is still on old GuacNode/GuacEdge generation. The tests are fully commented out.
It should be generating package/osv/cve/ghsa nodes and certifyVuln/isVulnerability connectors. IngestPredicates need to be added to main/pkg/assembler/assembler.go for vuln ingestion.

I'll start working on this.


Error on Windows Install with pear

Aways when I try to install phpDocumentor 2.8.2 on windows I receive the message: ERROR: unable to unpack, even when I use the argument --nocompress.

Installing on the linux with pear Works fine
Installing the version 2.8.1 on windows works fine

My PHP version is 5.6

Another packages are installed succesfully.
Thanks.

@RodriAndreotti It might be caused by the spaces in "Ambiente de des...". Try to install project in directory without spaces.

Hi @TomasVotruba,
Thanks,
I thought it might be something about it. I just thought it strange that the previous version installed without problems

@RodriAndreotti Have you managed to install successfully?

Yes... the previous version (2.8.1) install without problems.
But the 2.8.2 won't install

@RodriAndreotti Even after my suggestion?

I don't have tested yet, I will do this now.

@RodriAndreotti :+1:

This might also be a duplicate of https://github.com/phpDocumentor/phpDocumentor2/issues/1544; where a file in the package has a colon (:), causing issues on windows

I have a feeling this might have been fixed by #1568.
@RodriAndreotti could you check if the problem still occurs?

Well, on my development machine won't works, see attached file:

I have tried to install on a VM, but the pear documentation sites are offline, and go-pea.phar and pyrus.phar links are inside docs.
In this VM the path to php dir is C;\dev\php, I want to try on this vm because have no space in the path.


ssh session into WSL cannot execute windows binaries

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [Version 10.0.15063]

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
ssh into a WSL  session:

ssh -p 2222 user@host

What's wrong / what should be happening instead:

When I try to invoke a Windows binary such as MSBuild.exe, no error is returned and no output is printed, the process seemingly hangs. I am able to launch MSBuild.exe and other windows binaries when running directly on the bash console.

@samwise - Thanks for posting. Could you provide a bit more information about how you're launching your SSH server?

You may very well be right. I was unable to reproduce the issue today. I will update this issue if find a reliable repro. Sorry for wasting your time.

@samwise - You're definitely not wasting my time, I appreciate all bug reports.  The reason I ask my follow-up question is due to how interop currently works, it's a known limitation that we have plans to address in a future release.

@benhillis I've reproduced a similar issue using gnome-terminal. As soon as the original bash.exe exits, the interop of all processes launched by that bash.exe will no longer work. This is exactly what you suspect.


@conda-forge-admin, please update circle

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I updated the circle-ci deploy key and followed the project.


slim ci state:modified doesn't select a downstream child model when the parent tables changed

Describe the feature
Running the slim ci state:modified doesn't select child models when its parent ephemeral dependencies changed.
I have a model folder that looks like this:
└──  models
    └── model1
        ├── intermediate
        │   └── ephemeral_model.sql
        └──  model1.sql

the child 'model1.sql' depends on the parent 'ephemeral_model.sql'.
model1.sql looks something like this:
model1.sql
select
*
FROM another_table
INNER JOIN {{ ref('ephemeral_model.sql')}} ON ...

After changing 'ephemeral_model.sql' in my dbt repo, 'model1.sql' is not selected (listed or ran) in my ci when running the state:modified command below:
dbt run -m state:modified --stage <compare_target>
The new feature should select all dependent models when using -m state:modified.
Describe alternatives you've considered
To run my dependent child model, I made added an arbitrary new line change somewhere in 'model1.sql' so that my model gets selected, ran, and tested with the parent model changes.
Additional context
I am using dbt-spark, but I believe this applies to all databases.
Who will this benefit?
This will guarantee more robust testing given that child models should be tested when parent models dependencies changed.
Are you interested in contributing this feature?
I am willing to contribute to this.

@mv1742 Thanks for opening, and I totally agree with your reasoning here! It's possible to achieve this behavior using dbt's node selection syntax, in particular, the + graph operator.
In our documented best practices, we recommend using the following selection syntax:
dbt run -m state:modified+ --state path/to/artifacts

I'm going to close this issue, since what you've asked for is 100% possible—and desirable!—in dbt today.


Use canonical collection id for creating collection_url

Tentatively resolves #666

If you go to https://cellxgene.cziscience.com/collections/a0a4b739-6c98-42f9-b476-094f25cad922/ (version uuid) and check the response data, you'll see that id = 7855daa4-3f34-4aae-b65a-87720c02e7cd (collection uuid). This means that using the canonical id should pull in the collection id correctly.


[WIP] switch to alpine

Brings container size from 368.4 MB to ~124.6 MB

http and https both working


Add some fancy badges to the README.md

Codecov Report

Merging #183 (87b75a3) into main (45050c5) will increase coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head 87b75a3 differs from pull request most recent head 30a9f79. Consider uploading reports for the commit 30a9f79 to get more accurate results

@@            Coverage Diff             @@
##             main     #183      +/-   ##
==========================================
+ Coverage   32.07%   32.09%   +0.01%
==========================================
  Files          64       64
  Lines         957      944      -13
  Branches      362      354       -8
==========================================
- Hits          307      303       -4
+ Misses        465      460       -5
+ Partials      185      181       -4

Impacted Files
Coverage Δ

...n/scala/dev/atedeg/mdm/clientorders/dto/DTOs.scala
26.88% <0.00%> (-0.48%)
:arrow_down:

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


Linting for black update

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (a8b1e92) 87.03% compared to head (1e6653b) 87.03%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #93   +/-   ##
=======================================
  Coverage   87.03%   87.03%
=======================================
  Files          18       18
  Lines        1003     1003
=======================================
  Hits          873      873
  Misses        130      130

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Sorry Marcell, I did not see this PR for some reason.
My latest changes to the module include this I believe.

Sorry Marcell, I did not see this PR for some reason.
My latest changes to the module include this I believe.

All good!


Add config to adjust when excel export switches to streaming API.

See issue: #74
See corresponding hoist-react pr: https://github.com/xh/hoist-react/pull/1848
Went with a single config rather than the suggested json config, because I felt that individual configs (we currently only have one) are easier to document with the config note

Great change/pickup tom.  Thanks


QoL tweaks

Could we have the ability to change the first day of the week (Monday/Sunday), and the time format in the timegrid/events to 24h, please?

Great point! Thanks for filing this.

This would be terrific. Do you also have in mind to add a config to limit the hours of the day displayed to a custom setting? (E.g. 08:00-18:00)

I'm working towards atomic issues with one request each, so I'm closing this in favor of #34, #33, and #32 which track each of these requests individually.


How to bring in a token in request header when requesting a form

Good question. I can't submit form fetched from api, because I have no idea where should i inject x-jwt-token.


feat(tutorial): complete step 2 - Hursley

Closes #
tutorial step 2
Changelog
New

new content

Changed

old placeholders

Removed

nothing removed

:+1::skin-tone-2:   Looks good - I have recorded your completion of this step!


Web5-kt and tbdex-kt deprecation tracking checklist

web5-kt and tbdex-kt depreciation checklist in lieu of web5-core-kt and tbdex-core-kt
Deprecation Checklist: Migrating from web5-kt and tbdex-kt to web5-core-kt and tbdex-core-kt
web5-core-kt

[ ] Feature Parity

[x] Credential Schema
[x] Credential Status List
[x] Evidence Field
[ ] PEX

[x] select_credentials
[ ] create_presentation_from_credentials
[x] verifiable presentation

[ ] Test Vectors
[x] Windows build
[x] Decouple Web5Exception from UniFFI codegen
[x] Pass over old SDK for feature completeness sanity check
[ ] README updates
[ ] In-line code documentation

[ ] dokka kt docs

[ ] Artifact publishing
[ ] FOSSA Scanning

[ ] License
[ ] Security

[ ] CodeCov (test coverage)
[ ] CodeQL (Static analysis)
[ ] ** BUGS

[ ] fix [XMLSCHEMA11-2] dates

tbdex-core-kt

[x] Feature Parity
[x] Test Vectors
[x] Windows build
[x] Pass over old SDK for feature completeness sanity check
[x] Remove web5 shim

[x] Add standard TbdexError
[x] Totally decouple from UniFFI code-gen

[ ] README updates
[ ] In-line code documentation

[ ] dokka kt docs

[x] Artifact publishing
[x] FOSSA Scanning

[x] License
[x] Security

[ ] CodeCov (test coverage)
[ ] CodeQL (Static analysis)

web5 windows build completed in #329

VC evidence completed in #331
VC credential schema completed in #332

Status list creds completed in #320

Decouple Web5Exception completed in #334

tbdex windows build complete here https://github.com/TBD54566975/tbdex-rs/pull/114

remove web5 shim completed here https://github.com/TBD54566975/tbdex-rs/pull/115
decouple from uniffi code gen here https://github.com/TBD54566975/tbdex-rs/pull/116

standard TbdexError added here https://github.com/TBD54566975/tbdex-rs/pull/117

datetime fix in #343

create_presentation_from_credentials completed in #347
secp256k1 completed in #350