drop trap exit

WIP: currentlt kind dump is broken

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-istiodremote_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-istiodremote_istio

integ-telemetry-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-mc_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-istiodremote_istio

integ-telemetry-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-mc_istio

integ-pilot-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-multicluster_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-istiodremote_istio

integ-telemetry-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-mc_istio

integ-pilot-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-multicluster_istio

integ-security-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-security-multicluster_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-istiodremote_istio

integ-telemetry-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-mc_istio

integ-pilot-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-multicluster_istio

integ-security-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-security-multicluster_istio

integ-security-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-security-istiodremote_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-istiodremote_istio

integ-telemetry-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-mc_istio

integ-pilot-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-multicluster_istio

integ-security-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-security-multicluster_istio

integ-security-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-security-istiodremote_istio

integ-pilot-istiodremote-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-istiodremote-mc_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-istiodremote_istio

integ-telemetry-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-mc_istio

integ-pilot-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-multicluster_istio

integ-security-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-security-multicluster_istio

integ-security-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-security-istiodremote_istio

integ-pilot-istiodremote-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-istiodremote-mc_istio

integ-pilot-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-istiodremote_istio

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

/test all

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-istiodremote_istio

integ-telemetry-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-telemetry-mc_istio

integ-pilot-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-multicluster_istio

integ-security-multicluster_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-security-multicluster_istio

integ-security-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-security-istiodremote_istio

integ-pilot-istiodremote-mc_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-istiodremote-mc_istio

integ-pilot-istiodremote_istio
c1c4a930e5869a3eae969303ca1dd7ab8901d2ad
link
true
/test integ-pilot-istiodremote_istio

unit-tests_istio
4e1008d92bb3c24ae7513fa7cfc00629696654e8
link
true
/test unit-tests_istio

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


Sceneview visual glitch, multiplying objects in android

(1)This is link of a  video of third party model viewer app where how model loaded
https://drive.google.com/file/d/1SelSOpWpNfQIGYIquvCqT3yeOHzk_8hJ/view?usp=sharing
(2) This is the link of video where same model is loaded in sceneview in my app
https://drive.google.com/file/d/1sreSZbrUgxSwO9-Nq7uYRjCwT1vL_epQ/view?usp=sharing
(3) I Found this warning in My XML File

(4) This warning in sceneview.kt

Please Help me

Hi!
Can you try running the application on another device or emulator?
The warning about the layout shouldn't affect the application at runtime.

Hi! Can you try running the application on another device or emulator? The warning about the layout shouldn't affect the application at runtime.

As you said I try it in another device but there is no improvement at all,
this is the link of video.
https://user-images.githubusercontent.com/84631869/179397394-8e1a9eeb-5460-4b11-af53-cc76c2ed592b.mp4

Are there any lights in the scene? On the second device the are no traces left from the car lights. So the only issue is that the scene is completely dark.

Are there any lights in the scene? On the second device the are no traces left from the car lights. So the only issue is that the scene is completely dark.

Yes only issue is scene is completely dark

This is my code
(1)MainActivity XML

(2) MainActivity.kt

(3)  MainFragment.xml

(4) MainFragment.kt

(5)  AndroidMainfest.xml

(6) build.gradle

Any Updates @grassydragon

I've just looked at your code. There are no light in the scene since you have commented out the code that sets the environment. The environment is responsible for the ambient light.

I've just looked at your code. There are no light in the scene since you have commented out the code that sets the environment. The environment is responsible for the ambient light.

Hey it works . But why it is not working in first device , even device running on android 7.

Hey @grassydragon  For first device I set sceneview height from match_parent to 200 dp.
Luckily It is not glitiching that much as before. But it still show some glitch
https://user-images.githubusercontent.com/84631869/179450197-17090aba-ad5c-4f33-af90-fd0f9168f5a9.mp4
and I also found that if I kept model so small, Glitch gone itself.

There is still some glitch in sceneview in android 7.0


Finished authentication with DB and fixed conflicts

Describe your changes
Users can login with their credentials.
Fixed conflicts at pages/api/auth/[...nextauth].ts

Environment variables needed to connect to the database.


Release 1.5.2 preparation

Will tag rel1.5.2 and publish the packages on PyPI once this is merged.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 20.238%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
962

Relevant Lines:
4239

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 20.238%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
962

Relevant Lines:
4239

💛  - Coveralls


⚠ Samsung has degraded performance

In 3f8fb0b, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2375 ms

Resolved: Samsung performance has improved in 5a9a194.


useOverlayPosition: is it possible to set maxHeight?

Hi everyone, in useOverlayPosition, maxHeight seems to be calculated automatically to fit the viewport. This is a great feature, and it saved me a lot of time without calculating it manually.
However, is it possible to set maximum max-height somehow? E.g. when I'd want to have a popover with a maximum height 300px, even when there is potentially more vertical space?
I'll be happy for any advice on this.
Edit: I can "hack" it in my popover component by checking if (style.maxHeight > 300) { style.maxHeight = 300; }, but having it as prop would be way more elegant.

@honzatmn I'm unsure what side effects would occur if we add a maxHeight prop to useOverlayPosition, will have to dig into that. In the meantime, I believe your approach of overwriting the style.height returned from useOverlayPosition is appropriate.

@honzatmn I'm unsure what side effects would occur if we add a maxHeight prop to useOverlayPosition, will have to dig into that. In the meantime, I believe your approach of overwriting the style.height returned from useOverlayPosition is appropriate.

@honzatmn Apologies for the delay, finally got around to digging into this. It seems to me that adding this wouldn't be too difficult and doesn't seem to break anything when I play around with it locally. It would involve adding a maxHeight prop to useOverlayPosition and propagating it through calculatePosition to calculatePositionInternal, replacing the maxHeight calculated with the user provided maxHeight depending on whichever one is smaller.
I'll bring it up with the team to see if they agree and proceed from there

@LFDanLu that's okay, congrats on the recent release! 👏 A lot of nice updates there.
Thanks for getting back to this and discussing it with the team.

@honzatmn Discussed this with the team this morning, we were curious about your use case for limiting the max height of your overlays. Is it design specific? Concerns about possible inconsistent overlay sizing between similar components were brought up.

Gotcha, I've tagged our design team on this, will update you when they reply back.


Feature request: Use a git repository as storage backend

Since the data is not very sensitive for most people and can be easily dumped in a file, a git repository can be used as a storage backend. A scheduled process can automatically perform a Bidirectional sync with the repository. This will also serve as automatic backup and version control.

That doesn't sound very useful compared to just using SQLite and doing backups from that (which you can dump into Git if you want). I also don't understand the purpose of the bidirectional sync part. Anyway, not really interested in this. But sounds like you could do something like this as a 3rd party app via the API, or as I mentioned via SQLite backups.


gulp bundle --ship is not working

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[X] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[X] 💥 Google Chrome
[X] 💥 FireFox
[X] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version - Microsoft Edge 107.0.1418.26 (Official build) (64-bit)
SPFx version - 1.14.0
Node.js version - 14.15.0
etc

Describe the bug / error
We have created SPFx solution with webparts and extension using SPFx 1.14.0. When we tried to bundle the package using gulp bundle --ship, we are getting below error.
Error - [webpack] 'dist':
./node_modules/readable-stream/lib/internal/validators.js 140:15
Module parse failed: Identifier directly after number (140:15)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file.
If you look at the documentation of the package - you need to add custom webpack plugin to make it work.
Please check this doc to learn how to extend the out of the box configuration: https://learn.microsoft.com/en-us/sharepoint/dev/spfx/toolchain/extending-webpack-in-build-pipeline


Indentation issue with comment

Hey guys I'm seeing an issue with indentation when an inline comment is present... this has been formated by php-cs-fixer already
class fp_charts extends WP_Widget
{
    public function __construct()
    {
        $widget_details = array(
            'classname' => 'fp_charts',
            'description' => 'Create charts using supplied data'
        );
        parent::__construct('fp_charts', 'FlowPress Charts', $widget_details);
    }

    //Backend form
  public function form($instance)
  {

Notice the second class declaration after the //backend form the public function form( only gets two spaces assigned in front of it.

php --version
php-cs-fixer --version
cat .php_cs
cat .php_cs.dist

and CLI command that you are using.

Same thing for me

php --version

PHP 5.6.25-1+deb.sury.org~xenial+1 (cli)
Copyright (c) 1997-2016 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2016 Zend Technologies
    with Zend OPcache v7.0.6-dev, Copyright (c) 1999-2016, by Zend Technologies

php-cs-fixer --version

PHP CS Fixer version 1.12.0 by Fabien Potencier and Dariusz Ruminski (ddac737)

cat .php_cs

cat: .php_cs: No such file or directory

cat .php_cs.dist

cat: .php_cs.dist: No such file or directory

this has been formated by php-cs-fixer already

Can you share the code before it was fixed? My guess is the braces-fixer did that. So this would be a FR for an indentation fixer.

It was braces-fixer that fixed it... wish we had the option to toggle
between tabs and spaces.
On Wed, Sep 14, 2016 at 6:43 AM, SpacePossum<EMAIL_ADDRESS>wrote:

this has been formated by php-cs-fixer already
Can you share the code before it was fixed? My guess is the braces-fixer
did that. So this would be a FR for an indentation fixer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/2176#issuecomment-246974213,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATJ4SL332z3UKmC4sIaCy2Gw_LCj5Lfks5qp8_igaJpZM4J4QxG
.

--
Bart Dabek

Co-Founder & CTO of FlowPress Inc.  | Sidekick.pro | WPUniversity.com
647.588.6285  |<EMAIL_ADDRESS>|  @bartdabek

wish we had the option to toggle between tabs and spaces.

This is a recurring request and while I don't need it I would like to see that as well, together with Windows linebreak support. It is just that nobody seems to have find the time to implement/PR it. It is however on the list of some contributors here so it might happen in the (near) future. Let me know if you have time to help out :)

I believe I'm seeing this same issue.
PHP Version: 	PHP 5.5.9-1ubuntu4.19 (cli) (built: Jul 28 2016 19:31:33)
php-cs-fixer version: v2.0.0-alpha, but first tried latest stable release with same effect.
Here are some examples of the resultant code:
    }
    //widget form creation
public function form($instance)
{
    $instance = wp_parse_args((array) $instance, array('title' => ''));

and
    // display widget
    public function widget($args, $instance)
    {
        extract($args);
    // widget options
    //$numposts = $instance['numposts'];
    global $apkm_global;
        global $widget_item_count;
        $widget_item_count = 0;
        $accent_color = getAccentColor($apkm_global);

Let me know if there is more information I can provide to help you solve this problem. I'm not sure what the comment above means by "it was braces-fixer that fixed it". Am I doing something wrong? Thanks.

can you provide an example of the input code which was fixed to the code you provided?

Sure. I just made a very small example file, and I messed up the indentation a little on purpose. Luckily I managed to reproduce the problem fairly quickly. After running just php php-cs-fixer fix myphpfile.php I'm left with the following results:
Input file
<?php
class AppManager_previous_versions extends WP_Widget
{
    // update widget
    public function update($new_instance, $old_instance){
    $instance = $old_instance;
    // Fields
    $instance['numposts'] = strip_tags($new_instance['numposts']);
    $instance['title'] = strip_tags($new_instance['title']);

    return $instance;
    }
}
?>

Output
<?php
class AppManager_previous_versions extends WP_Widget
{
    // update widget
    public function update($new_instance, $old_instance)
    {
        $instance = $old_instance;
    // Fields
    $instance['numposts'] = strip_tags($new_instance['numposts']);
        $instance['title'] = strip_tags($new_instance['title']);

        return $instance;
    }
}

Note the wrong indentation following the // Fields comment. Also it removes the closing ?> php tag, but that might be desired behavior? Thanks for your quick reply!

Also it removes the closing ?> php tag, but that might be desired behavior?

This is done by this fixer:
php_closing_tag [PSR-2]
The closing ?> tag MUST be omitted from files containing only PHP.

you can disable it if you want, but I strongly recommend to keep it. It prevents unintended echo-ing of trailing white space in .php files (it is also a recommendation done in PSR2) .
I'll have a look at your samples, thanks for providing those!
Just FYI, if you want to know which fixer does what on your code run the fixer and add -vvv to the command.

Hey @SpacePossum have you managed to take a look at my example to find out why it indents so funnily?

I did and your report is indeed valid. It is an issue within the braces-fixer. I gave it a try to fix it, but I have no time ATM to do it. I hope someone else has time to pick this up.

Would it help if I told you it seems the built-in PSR2 fixer in PHPStorm seems to do the same thing? Are you using the same codebase or something?

we don't base on any logic from PHPStorm. maybe PHPStorm use our logic under the hood. who knows...

I saw a similar behavior on the Sublime Text built-in re-indent command, if that may help.

Any updates here?

feel free to contribute ;)

With the latest 2.0.0 release, the issue and testcase I described above is still broken. Strangely I discovered if you fix the indentation manually and then run php-cs-fixer, it won't break anything. It's only when you start with a badly indented version that it will mess up - around comments especially.

@SeBsZ thanks for testing and sharing your findings!
There has been no PR's to resolve this so please if anyone has time to work on it it would be very welcome!

I can confirm that in the 2.0.0 the issue is still here.

In Fixer/Basic/BracesFixer.php, line 319, add
                $nestToken = $tokens[$nestIndex];

                /* Fix start */
                if ($nestToken->isComment() && substr($nestToken->getContent(), 0, 2) == '//') {
                    $tokens->ensureWhitespaceAtIndex($nestIndex - 1, 0, rtrim($tokens[$nestIndex - 1]->getContent(), ' ') . $indent . $this->whitespacesConfig->getIndent());
                }
                /* Fix end */

In Fixer/Basic/BracesFixer.php, line 330, replace
                if (1 === $nestLevel && $nestToken->equalsAny(array(';', '}'))) {

by
                if (1 === $nestLevel && $nestToken->equalsAny(array(';', '}', array(T_COMMENT)))) {

@owebia Thanks for the feedback and taking time to look into this. Could you maybe make a PR out of the proposed changes?

The proposed fix by @owebia makes soms tests break for me.
like:

./vendor/phpunit/phpunit/phpunit --filter testPreserveLineAfterControlBrace tests/Fixer/Basic/BracesFixerTest.php

[...]

PhpCsFixer\Tests\Fixer\Basic\BracesFixerTest::testPreserveLineAfterControlBrace with data set #1 ('<?php\nif ($test) { // foo\n   ...o 1;}\n', '<?php\nif ($test) // foo\n {\n  ...o 1;}\n')
Code build on input code must match expected code.
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
<?php
-if ($test) { // foo
+if ($test) {    // foo
echo 1;
}
-if (1 === 1) {//a
+if (1 === 1) {
+    {//a
$a = "b"; /*d*/
}//c
echo $a;
if ($a === 3) /**/
{echo 1;}

as you see "{//a" becomes "{\n    {//a". The opening brace get's duplicated. This produces invalid code.

Fixed, will be released soon. thanks to @julienfalque !


can't hide the fallback Web3 Wallet

When there is no injected provider, Web3Connect used the "fallback" provider which is called "Web3", type "injected", check "isWeb3".
What is the point of this?  In what typical conditions will this fallback succeed?  Seems like it should not be displayed unless it has a chance to succeed.  For me, it is showing up and failing.  It is not even needed because I have Portis and Fortmatic showing (but not WalletConnect, and no extensions are available).
So to get rid of this, I tried supplying disableInjectedProvider and when I do, when the Modal comes up the fallback is not hidden (even though it is "injected"), and there is a console log message, even though I am passing true:
Warning: Failed prop type: The prop `injectedProvider` is marked as required in `Modal`, but its value is `null`.
    in Modal
checkPropTypes.js:20

How can I cause this fallback to not be shown?  Thanks.

The idea of the fallback is for injected providers that haven't been added to the library or that don't support the normal specification. Opera is an example of it.
The problem that you have is that you don't want the injected provider to appear and when you supply disableInjectedProvider it is shown?

The problem that you have is that you don't want the injected provider to appear and when you supply disableInjectedProvider it is shown?

I would prefer that the fallback button not appear if it is only going to fail if the user selects it.  If that is not possible, then I'd like to be able to hide it by setting disableInjectedProvider, say, when I can't find window.web3 or window.ethereum (anticipating on my own that it is going to fail).

There was a bug that didn't hide the injected provider using disableInjectedProvider

Cool, but best would still be if the Web3 button were not be displayed if there is no injected extension to work with.  In that case it is worse than worthless, as it crashes.
What is the dApp expected to do in this situation?  I'm not sure that expecting the dApp to know whether to disable extensions (even if that works now) is a friendly UX.  (disableInjectedProvider: !((window as any).web3 || (window as any).ethereum)) Would be better IMHO that Web3Connect be smarter on its own about when to show the fallback.

I'll open it again and check it out since what you describe should be the default functionality and shouldn't crash.

The problem is: "If you have no web3 injected provider it crashes"? right? @dkent600

@crisgarner

If you have no web3 injected provider it crashes

Yes, last i checked.

I think, this was fixed on the 1.0.0 release

@crisgarner So now I'm curious, when is the fallback provider supposed to be displayed?  I'm having trouble now making it appear. Thanks.

Oh, with Opera, I see it, thanks


App Crashes when picking Videos after using Image Compress

In my App, I have 2 different buttons.
First button: Selects images only and this method uses MediaPicker.compressImage() to compress image.
Second button: To select videos only and this method does not use compressImage() method at all.
This is what I noticed:

When I open the App, upload video first, then it works fine.

But, when I upload an image first and then select upload video, select video and press Done, it always crashes.

In other words, the video picker crashes if I try to upload videos after I upload an image that uses compressImage() method. My video upload does not use compressImage obviously, and if only video is uploaded first straight after launching the app, then its fine. But it crashes if video is picked after picking images.
This is the crash error log:

05-18 18:54:21.785 32003 32408 E AndroidRuntime: FATAL EXCEPTION: pool-1-thread-2
05-18 18:54:21.785 32003 32408 E AndroidRuntime: Process: com.example.examplemobile, PID: 32003
05-18 18:54:21.785 32003 32408 E AndroidRuntime: java.lang.NullPointerException: Attempt to invoke virtual method 'boolean android.graphics.Bitmap.compress(android.graphics.Bitmap$CompressFormat, int, java.io.OutputStream)' on a null object reference
05-18 18:54:21.785 32003 32408 E AndroidRuntime: 	at com.dmc.mediaPickerPlugin.MediaPicker.compressImage(MediaPicker.java:264)
05-18 18:54:21.785 32003 32408 E AndroidRuntime: 	at com.dmc.mediaPickerPlugin.MediaPicker.compressImage(MediaPicker.java:232)
05-18 18:54:21.785 32003 32408 E AndroidRuntime: 	at com.dmc.mediaPickerPlugin.MediaPicker$1.run(MediaPicker.java:161)
05-18 18:54:21.785 32003 32408 E AndroidRuntime: 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
05-18 18:54:21.785 32003 32408 E AndroidRuntime: 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
05-18 18:54:21.785 32003 32408 E AndroidRuntime: 	at java.lang.Thread.run(Thread.java:761)

The Google Play Store crash log shows this:
java.lang.NullPointerException:
  at com.dmc.mediaPickerPlugin.MediaPicker.compressImage (MediaPicker.java:264)
  at com.dmc.mediaPickerPlugin.MediaPicker.compressImage (MediaPicker.java:232)
  at com.dmc.mediaPickerPlugin.MediaPicker$1.run (MediaPicker.java:161)
  at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1133)
  at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:607)
  at java.lang.Thread.run (Thread.java:761)

I did not set the quality parameter in args. I add quality to results[i]. This is how I have it:
function buttonClick(format) {

  if (format == 'video') {
    var args = {
        'selectMode': 102,
        'maxSelectCount': 1,
    };
  }
  else {
    var args = {
        'selectMode': 100,
        'maxSelectCount': 30
    };

    MediaPicker.getMedias(args, function(results) {
        if (results.length) {
          if (format == 'video') {
              uploadUsingFileTransfer(results[0].path);
          }
          else {
            for (var i = 0; i < results.length; i++) {
                results[i].quality = 30; //when the value is 100,return original image
                MediaPicker.compressImage(results[i], function(compressData) {
                    uploadUsingFileTransfer(compressData.path);
                }, function(e) { console.log(e) });
            }
          }
        }
    }, function (error) {
          // console.log('Error: ' + error);
          alert("cannot obtain pictures");
    });

}

If I remove the MediaPicker.compressImage() then I dont have issues selecting videos after photos. But if the above is in place, then the video picker fails after uploading photos before it.

sir i think format ==‘video’   there is a problem with the use in  MediaPicker.getMedias callback。
change this
    MediaPicker.getMedias(args, function(results) {
        if (results.length) {
          if (results.mediaType== 'video') {  // change
              uploadUsingFileTransfer(results[0].path);
          }
          else {

          }
        }
    }, function (error) {
    });

update plugin form git ，I'm over the weekend and can only be repaired this way. I can't test

no worries. Thank you for that. I will give this a try. Cheers! and have a good weekend.

sir i fix the bug in npm

Awesome. Thank you for the quick fix. I will update my app now.
This compressImage feature is very helpful. I was waiting for this so I can start using this in my iOS App as well.
Thanks heaps.


chore(lib): don't require domain by default

Domains have an unwanted overhead on Node.js v12. The overhead is
observable just by requiring domain. To prevent that, we postpone
requiring domain until we are sure it's necessary (in which case it
was already required somewhere else).

Coverage decreased (-0.1%) to 98.346% when pulling 830b004cad47a5249ace40be523328fae173f6f3 on mmarchini:dont-require-domain into e79fdf185d215ff95506d65233b4bbc24970776b on restify:master.

Coverage decreased (-0.1%) to 98.346% when pulling 830b004cad47a5249ace40be523328fae173f6f3 on mmarchini:dont-require-domain into e79fdf185d215ff95506d65233b4bbc24970776b on restify:master.

cc @hekike @DonutEspresso

lgtm!

Thanks! Released as<EMAIL_ADDRESS>


block ads.ad-center.com & associated hosts

look.utnldn.com
look.kfioplkn.com
look.ichlnk.com
look.udncoeln.com
look.fjfiln.com
look.ufinkln.com
hlok.qertewrt.com
look.opskln.com
look.ichlnk.com
look.ufinkln.com
look.utndln.com
look.djfiln.com
look.kfiopkln.com
look.udncoeln.com
goa.camimbu.com
zulu.r867qq.net
ads.ad-center.com
cdn1.ad-center.com
cdn2.ad-center.com
ads4.ad-center.com
ads.ad-center.com
ads-script.ad-center.com
www.ad-center.com
#these are closely associated and or interacting directly with ad-center https://threatcrowd.org/domain.php?domain=pwwysydh.com
pwwysydh.com
piz7ohhujogi.com
prjcq.com
pocofh.com
npmpecd.com
pt21na.com
pwdplz.com
pohs2oom.com
ps4ux.com
jadserve.postrelease.com
docaz.net
prjcq.com
e.serverbid.com
d303resfoavrtt.cloudfront.net
payae8moon9.com
pohs2oom.com
#all ptrs for ads.ad-center.com https://www.robtex.com/dns-lookup/ads.ad-center.com
ec2-46-137-77-217.eu-west-1.compute.amazonaws.com
ec2-46-137-110-83.eu-west-1.compute.amazonaws.com
ec2-54-243-100-198.compute-1.amazonaws.com
ec2-175-41-141-63.ap-southeast-1.compute.amazonaws.com
ec2-175-41-150-101.ap-southeast-1.compute.amazonaws.com
ec2-176-34-182-24.eu-west-1.compute.amazonaws.com
ec2-184-72-41-50.us-west-1.compute.amazonaws.com
ec2-184-169-151-9.us-west-1.compute.amazonaws.com
ec2-184-169-178-22.us-west-1.compute.amazonaws.com
ec2-184-169-182-88.us-west-1.compute.amazonaws.com

utnldn.com = Expired
kfioplkn.com = Expired
fjfiln.com = Expired

http://pwwysydh.com/ sure comes with a spooky link http://www.qfind.net/?__jsfail&domain_js=pwwysydh.com&feed=Mocha&subfeed=Accepted/DesktopRow no httpS

any one who have a closed environment where they can test that link?
I've tested and confirm the above obfuscating domains in the following subject

r867qq.net
camimbu.com
udncoeln.com
kfiopkln.com
djfiln.com
utndln.com
ufinkln.com
ichlnk.com
opskln.com
qertewrt.com
ufinkln.com
udncoeln.com
ichlnk.com
ad-center.com

I would entertain a PR for these.


[bedrock2/Keymgr] Transcribe keymgr.c with Bedrock2

Transcribe Keymgr device driver originally written in C code using Bedrock2 syntax.
The generated C file from the Bedrock2 should pass the unit tests and functional tests of OpenTitan.
Note: This issue is retrospectively created

Finished and tested in #844


layer=-1 not needed when tunnel=building_passage

When editing a highway I can select tunnel from the Structure field and iD adds the tags tunnel=yes and layer=-1. When I change tunnel type to building passage tunnel=yes becomes tunnel=building_passage but the layer=-1 tag is kept.

Is it possible to remove layer=-1 for building passages since they tend to be at layer 0? Users can always add the layer if it isn't at 0. (taginfo shows that only 13% of building passages are combined with a layer tag)

Yuck.. yeah we can do this.. Are there any other special tunnel=* values that work like this?

Not that I'm aware of.


Fixes drag-n-drop problems

Summary
Fixed issue #1434
Fixed issue #1454
Also added util methods for SelectionsState - compareWithSelection(), updateOnDeletingSelection()
Test Plan
Added unit tests.

Can you explain how this relates to #1599? Both of them seem to have some part of a fix for #1454.
#1599 fixes this bug:
https://github.com/facebook/draft-js/issues/1454#issuecomment-339360755
#1603 fixes this bugs:
https://github.com/facebook/draft-js/issues/1454#issuecomment-340973151
https://github.com/facebook/draft-js/issues/1454#issuecomment-348872969 (also described at #1434)

This needs a test plan - can you either add test cases or a video with the bug & fix.
Added unit test. Currently it fails on master (throws exception) and passes after this fix.

I'll explain logic soon..

@niveditc
Cleaned up commit, added some comments. Also see unit test  DraftEditorDragHandler.test.js.
SelectionsCompareResult is result of comparison of 2 selections. I added general purpose method (helper) SelectionState.compareWithSelection().
I use it to compare position in content state with selection to be deleted - at comparePositionWithSelection(), which is used at fixPosOnDeletingSelection(), which is used at SelectionState.updateOnDeletingSelection().
Last one is essential to get correct position in udpated content state after we removed text (dragged off), this position should be used to insert (drop) text then.
Without this DraftModifier.moveText() works incorrect.

We are receiving "Cannot read property 'getCharacterList' of undefined".
Why hasn't this been merged since january?

@thread huh, this fell through the cracks. Just confirmed this is still an issue. I need to read more and get familiar with this though, but I'll have a look if I have a spare minute during the holidays.


Virtual keyboard iPhone 6 viewport height

I have a text input in the bottom navbar that triggers scrolling using $location.hash() and $anchorScroll(). This works perfectly on the Samsung Galaxy S4, but there is odd behavior on the iPhone 6.
When you first tap on the text input, the virtual keyboard opens in front of the bottom navbar. Type a letter and the list is scrolled to show the matching item at the top. Type a few more letters, and the bottom navbar jumps up (to the correct positiion on top of the virtual keyboard), but the matching item is now scrolled to a point above the top of the screen. Close the virtual keyboard, and the matching item is now at the top of the screen.
It seems that the viewport/app-body/app-content/scrollable/scrollable-content height is not adjusted when the virtual keyboard opens, and there might be a race condition or something causing the inconsistent behavior (as additional letters are typed and the screen shifts).
In case it makes any difference, I'm using PhoneGap Build to produce the APK and IPA files that I tested on these devices.
Any ideas for a workaround? Detecting presence of the virtual keyboard, forcing resize of the viewport, etc?

On iOS, the virtual keyboard's presence does not change the viewport height. We need to detect presence of the virtual keyboard (an imperfect solution is to use the "focus" event on the text input) and then adjust the height of "app" accounting for the height of the soft keyboard (maybe something like http://stackoverflow.com/questions/2593139/ipad-web-app-detect-virtual-keyboard-using-javascript-in-safari#17604856).
The "race condition" is because Mobile Angular UI tries to scroll to the focused input to keep it on the screen, but I'm trying to scroll to the search result in the list. If we fix the height of the app, the absolute bottom navbar will always be on the screen and we should not have a problem.
Please help, I don't have an iOS device to test on.

I figured out a workaround. On focus of the input (opening of the virtual keyboard) I change the height of #app from 100% to window.innerHeight px, and on blur I change it back to 100%.


Add configurable stream sorting

Possible options:

Order by relevance (this is the current default)
Order by download size
Order by download time
... (maybe more possibilities?)

The options are configurable here when accessing the search api:
https://github.com/sleeyax/stremio-easynews-addon/blob/726239538ccec2cc61c15d63a7f1cad8320983e3/packages/api/src/api.ts#L29-L34

One suggestion would be if it's possible to sort by HDR thing.
Like priority would be Dolby Vision > HDR > 4k > 1080p

One suggestion would be if it's possible to sort by HDR thing.
Like priority would be Dolby Vision > HDR > 4k > 1080p

Create a new issue for that.


1.3.4 - Windows - Plots and Farming tabs take 30+ Seconds to load with a large number of plots/directories

With 10,628 plots and 168 directories the Farming and Plots tabs take 30+ seconds to load.  This does not include initial startup, this occurs every time the user clicks on one of the tabs.  Other tabs operate lickity split.  Farming works as expected.
Thanks!

We are working on improvements related to this problem
https://github.com/Chia-Network/chia-blockchain-gui/tree/harvesters_pagination

Fixed with https://github.com/Chia-Network/chia-blockchain-gui/pull/713


adjust dockers and configs to luigiless

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/retest

/retest


Rewriting entry function in custom next.config.js or client/npm-dev.js & client/index.js

I want to write some my own logic in client js (client/index.js & client/npm.dev.js). So I am trying to change the MainJS's path in entryies of webpack.js to compile my own version of client js.
Is it possible to do this?
Actually I tried to rewrite the entry function in custom next.config.js. But it never been called.
Thanks!

You can access the full webpack config object with next.config.js and using webpack() function. Here entry is an object and try to log it and change as you need it.

@arunoda Thanks for your response.
But when I try to log the entry object in my next.config.js, I got this:
next.config.js:

Output:

Apparently, it's a function when it's logging. Is there some thing I did wrong?

Actually, it's a async function now.
See: https://github.com/zeit/next.js/blob/master/server/build/webpack.js#L39
Now you need to do something like this:
module.exports = {
  webpack: function(c) {
    const original = c.entry
    c.entry = function () {
      return original()
        .then((entry) => {
          // do something with it
          console.log(entry)
          return entry
        })
    }
    return c
  }
}


electron_bridge: Send unread count to electron app on update.

We should talk with @akashnimare about a few details of the API to standardize on, e.g.:

Do we want -s or _s as the word separator in unread-count?  I generally prefer _s since they translate into variable names better in most programming languages.
How explicit do we want these to be?  I would sorta be tempted to have the thing called total_unread_count for clarity, since we may in the future want to send data structures for others cuts of the data.

@timabbott i did talk with him about this, he seems to be fine with this approach. This is like two code base conflicting, in zulip we use snake like all python project case while CamelCase on zulip-electron like most JavaScript projects do.
I electron app we use max of words ie unread-count for ipc's.  Though the app, and this is handled in one file currently max it could be is directory. We can switch it up now so it's consistent throughout.

Agreed with Tim. For bridge thing, let's go for _s so -

in webapp total_unread_count i.e. _ and then in our electron app -

electron_bridge.on('total_unread_count', count => {
	ipc.send('total-unread-count', count);
});

The _ in electron will indicate that the events are coming from the webapp and that way we can differentiate these kind of events.

Cool, sounds like a plan.  @cPhost let us know when you have a new version ready :)

Updated now as discussed.

I think I'd prefer the check to be if (window.electron_bridge !== undefined) {
just because our style is not to use falsyness in the JS codebase.
Otherwise, lgtm.  @akashnimare can you take a look as well?

now using production ready strict type checking :)

@cPhost can you rebase?  I'd like CI to pass.

OK, I fixed the fact that your conditional was backwards (please test more carefully @cPhost!) and merged this, just doing the one call point that we need in notifications.js.
So we can now work on the Electron side implementation to use this API, and will be able to test it on chat.zulip.org in addition to development environments.  One thing we need to be careful about is making sure our Electron app code still works with older servers that don't have this feature yet (it's possible we should add a flag to server_settings or something related to having support for this, maybe just a version number?).
(I think we should also add something in the realm_icon section of server_events_dispatch.js to notify the app about a new realm icon, using this same protocol; that might be a simpler change to finish 100%).

Old server compatibility wise, we don't need to do something here (for now), since the code on electron side is not being removed. For future, if we need something like this I will definitely make a PR adding a something server_settings as you described.

So we can now work on the Electron side implementation to use this API, and will be able to test it on chat.zulip.org in addition to development environments

I had this PR up https://github.com/zulip/zulip-electron/pull/428, that I and @akashnimare  used to test both PR's!

was backward

Sorry about that! (i didn't have access to my computer that has desktop app setup)


Compare unique records from base to synthetic file to ensure privacy

Over email we've discussed how to ensure privacy between the base and synthetic PUF. @donboyd5 described an example of what to watch out for* which I think boils down to:
The synthetic PUF should have few or zero records which are uniquely traceable to a single real PUF record.
This is related to the privacy concepts of k-anonymity and i-diversity, and the R package sdcMicro may be of some use (it's unclear if a Python equivalent exists). However, I think our use case extends beyond these, since we're comparing a second file to the original. One formalization of the problem is as follows:

For each record in the PUF, identify all combinations of variables by which that record is unique. Let's call these combinations of uniquely identifying variables record-identifiers (made up, there's probably a better name for these). There are considerably more record-identifiers than records, since each record is probably unique by the whole record, as well as some combinations of variables.
Check if any of these record-identifiers exist in the synthetic PUF.

(1) would be very computationally expensive, since it would require looping through all 2^k combinations of variables. So I think whatever checks we come up with will have to be weaker than this, but we can compare to this ideal. For example, we could start with just pairs or triples of variables. We could also consider rounding numerics to the nearest $100 or $1,000 to reduce the number of record-identifiers (though this could also result in more matches to the synthetic file from the remaining record-identifiers).
This is a strict test because it implicitly assumes that each PUF record uniquely identifies a tax unit out of the universe of all tax units, which is probably only true for a small number of tax units. Without information on this I don't know what other assumptions we can make.
* Example from @donboyd5's email:

Suppose we have a record in the PUF that might be an unusual and identifiable real person (even though SOI tries hard not to allow that and my example almost certainly cannot happen). But anyway. And suppose we have a few values such as:
$300,123,000 in wages
MARS==2 -- married joint
XTOT==4 -- 2 kids
-$100,654,000 in business losses
$50,789,000 in charitable contributions
$60,456,000 in state and local taxes
Somewhere there is a headline, "Married New York investment banker gets $300 million bonus, invests it in business and loses $100m, but still gives $50m to the Metropolitan Opera while his 2 kids go hungry." Ok, hokey, but the point is, we know who this person is. Somehow they slipped by the SOI PUF-creation non-disclosure procedures.
Suppose further that if we have any 4 of these 6 tax-return items on a record we could identify this individual (even though other record items might be incorrect).
Suppose that we now create a synthetic PUF and we use CART for these variables and have it set so that when we get to the final branch and all we have are leaves, it randomly selects a leaf from actual leaves. That is, when it is predicting wages, it has (perhaps) sorted all of the high wage people across whatever the predictor variables are and chooses one of the actual wage values rather than a mean or other prediction.
In doing this, synthpop manages to construct a single record that actually has wages of  $300,123,000, business income of  -$100,654,000, charitable contributions of  $50,789,000, and state and local taxes of $60,456,000.
This raises the almost metaphysical question of whether we have a disclosure. By my definition, if we have any 4 of the 6 items above from this particular individual we can identify the individual. Let's come back to this. Maybe we argue that this is not a disclosure. But suppose SOI insists this would be a disclosure. Dan's question (Dan, please weigh in) is, "It is my understanding that each value in the synthetic dataset comes from a value in the PUF. Can we assure Barry that no synthetic record has more than N values from the same taxpayer? What would N be? Can we place restrictions on N?"
Because we used CART with defaults, we are choosing actual leaves rather than predicted means or something else, so his premise is satisfied. The question (if we don't change that default), given the rules of this example, can we assure Barry that no record has more than 3 values from the same taxpayer. And more generally, can we assure Barry that no record has more than 3 (or whatever). In this case, we would fail that test. So, just to elaborate on Dan's question, (1) can we, during the prediction process (a) set that N, and (b) if not, use methods that make it likely that N will be small, at least in areas of the joint distribution where large N is likely to create an identification issue?, and (2) after the fact (after synthesis), can we check for N (seems computationally expensive but variants may be efficient) and take corrective action where needed? What other post-synthesis tests should we be thinking about?
Of course, we can also argue that large N does not necessarily mean we created a real person. Perhaps in my example, while we got 4 money items correct, our fictitious record has MARS==1 and XTOT==1, so in other important ways this record doesn't look at all like the real person - we created a single investment banker with no kids. So perhaps we should take dissimilarity into account as well as similarity.

cc @feenberg

Here's what the TPC synthetic PUF paper says (emphasis mine):
Page 20:

Our sequential regression synthesis methodology also protects against the most serious form of attribute disclosure (Burman et al. 2018). That is, while sampling from the estimated multivariate distribution may produce rare records with unusual combinations of attributes, they are extremely unlikely to be close to actual records. If by chance, synthetic records are close to actual records with rare attributes, the existence of the synthetic records provides virtually no information about whether such a combination actually exists.

That is, similarity to real records is OK because intruders don't know whether it's real. So the general dissimilarity between real and synthetic records creates ambiguity about even similar cases.
Page 30:

The process of generating the synthetic data is designed to be disclosure proof, as discussed. However, we will apply various tests to guarantee that we haven’t inadvertently created disclosure risks. Our tests will be especially sensitive to the risk of attribute disclosure. For example, if only one tax unit has a particular combination of tax forms and schedules, the inclusion of such a return in the synthetic database could constitute evidence that the unique unit had filed a return. We will identify such cases if they exist and address them. For example, we might reduce the number of forms and schedules in the synthetic database until attribute disclosure is impossible.

This proposal seems similar to what I sketched out in https://github.com/donboyd5/synpuf/issues/5#issue-381836036, though the (real) PUF has much more uniqueness than the full universe of records, given it's a sample.


Compatibility with Azure CosmosDB

Hello,
We are looking into an option to use occurrent with Azure CosmosDB witch supports MondoDB API ver. 4.0
Could you please tell why there is a requirement to use MongoDB 4.2?
Will occurrent mongodb event-store implementations fail to work with slightly old api implementation?
Vitaliy

Hi vlev,
Sorry for the late response, I've been on vacation.
If I remember correctly it has to do with MongoDB change streams and resume options (this specifically: https://docs.mongodb.com/manual/changeStreams/#startafter-for-change-streams). But if you're not deleting the entire database or event collection then I think it'll work anyway. It would be great if you could try it out, and tell me if it does work for you. If not, I can probably make it work for 4.0 as well :)
/Johan


Submap Drifting problem

Hello,
I had z position drifting problem but thanks to  @ojura  and his suggestion lidar_imu_correction_part
but I have another issue even after the trajectory are loop closed.
pbstream file.

I thought that if trajectory is loop closed, submap should be projected into the proper position not drifting
somewhere else like on the picture above
Side View_2

Another place has the same issue.
What could be the reason for this problem?
any comments would be appreciated.

Closing for inactivity, we can't invest time to help you with your setup anymore unfortunately.


Onboarding stage

@Philip-Nunoo Hey Philip, I created this new PR with your changes and the current stage branch merged in.
some things need some work still:

the aeButton styles changed with the new components, is the a style without the borders as in the layouts?
on smaller screens the images go above the header
on large (desktop) screens the content is aligned bottom left. it should be centered horizontally at least


Facebook signin not working on deployed environment

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
No response
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
7.6.26
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
N/A
Amplify Categories
auth
Amplify Commands
Not applicable
Describe the bug
Facebook signin is working on localhost but not on hosted environment.
These are the network traces for localhost signin

These are the network traces for hosted env signin

Expected behavior

Reproduction steps
followed steps for amplify FB signin
GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

Are you using Amplify Hosting to host the app?

@swaminator yes

Can somebody please help here.

I found the issue here.
In the rewrites and redirects rules, I had
```
{
"source": "https://.in",
"target": "https://www..in",
"status": "302",
"condition": null
},
I changed the status to 200 and it works now.


Panic while serving source page: "krate.description not found"

[ERROR] docs_rs::web::page::web_page: called ctry!() on an `Err` value: Error { kind: Msg("Failed to render \'crate/source.html\'"), source: Some(Error { kind: Msg("Variable `krate.description` not found in context while rendering \'rustdoc/topbar.html\'"), source: None }) }
note: while attempting to fetch the route Url { generic_url: "http://docs.rs/crate/rustc-ap-syntax/178.0.0/source/fold.rs" }
    0: docs_rs::web::source::source_browser_handler

Hmm, so I found why it's crashing, https://github.com/rust-lang/docs.rs/blob/ba9364b935e2ef1c9b925f4295d42fd03b9587dd/src/web/source.rs#L249-L255 doesn't include a description (it's part of file_list). The weird thing is that it only crates on this crate and not others, I would expect this to be broken everywhere.


Enable previous dates

I would like to enable previous dates, is that possible?

can you provide a jsfiddle of your problem?

I only have access to the Javascript code that creates the datepicker.

$('.date-picker').datepicker({
orientation: "right left",
autoclose: true
});


Shell is stuck loading when using Firefox Incognito mode

When opening https://shell.duckdb.org/ in Firefox Incognito mode the shell gets stuck like in this screenshot:

In regular Firefox window it works fine:

Linux, Firefox 105.0.1

Seems it's actually fixed, thanks again for opening this.
Behaviour now on Firefox incognito, and to some degree situations where indexedDB will fail to initialise, is that a console.warn("") message will inform no history could be loaded/persisted, but the shell will work regularly.
Thanks @ankoh and @domoritz.


Fixed issue with broken include paths on different platforms

fix for the include path

The include paths aren't a platform dependant issue. A user will receive include errors if the user's SDL packages are under a directory (ex; SDL/SDL.h). Ass of now, it is like forcing people all under the same directory vs a general include directory for several libraries (ex: tmxlite/Map.h, SDL/SDL.h, GLAD/glad.h).


[A11y][iOS] - VoiceOver reads first letter separately from other text in multiline textInput on legacy architecture

Description
When using TextInput component with multiline={true}, voiceover announces first symbol separately from the remaining text
Managed to reproduce on RN versions 0.73.5, 0.73.8, 0.74.2
Actual result: First symbol is read separately
Expected result: Whole text is read together
Note: The issue is not reproduced with RCT_NEW_ARCH_ENABLED = 1
Steps to reproduce

Turn on VoiceOver
Enter text
Tap on the input to announce

React Native Version
0.73.8
Affected Platforms
Runtime - iOS
Output of npx react-native info
System:
  OS: macOS 14.2.1
  CPU: (8) arm64 Apple M1
  Memory: 88.33 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.11.1
    path: ~/.nvm/versions/node/v20.11.1/bin/node
  Yarn:
    version: 3.6.4
    path: ~/.nvm/versions/node/v20.11.1/bin/yarn
  npm:
    version: 10.2.4
    path: ~/.nvm/versions/node/v20.11.1/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.14.3
    path: /Users/dev/.rbenv/shims/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK: Not Found
IDEs:
  Android Studio: 2023.2 AI-232.10227.8.2321.11479570
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.10
    path: /usr/bin/javac
  Ruby:
    version: 3.3.0
    path: /Users/dev/.rbenv/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react: Not Found
  react-native: Not Found
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

Stacktrace or Logs
N/A

Reproducer
https://github.com/srrmstk/react-native-multiline-a11y-reproducer
Screenshots and Videos
https://github.com/facebook/react-native/assets/61790890/5c1fa74c-1539-4d71-a56a-1af8f42fd917

Any updates on this issue? @cortinico

Note: The issue is not reproduced with RCT_NEW_ARCH_ENABLED = 1

Nope no updates. Also issues with New Architecture disabled are less likely to be address as now we're hyperfocused on shipping the New Architecture. So if this issue is fixed in NewArch, it's unlikely we'll fix it at this stage.


initial commit of Get-DbaAvailabilityGroup

Initial commit

@ClaudioESSilva do you have an environment to support testing for this command? If so, would you be available to do so this week?
Here is the QA checklist

[ ] Working/useful help content, including link to command on dbatools web site
[ ] All examples work as advertised
[ ] Does not contain template content
[ ] Does not contain excessive/unnecessary amounts of comments
[ ] Works remotely
[ ] Works locally
[ ] Works on lower versions or throws error specifying version not supported
[ ] Works with named instances
[ ] Works with clustered instances
[ ] Handles offline/read only databases
[ ] Supports multiple servers (at the command line or piped from Get-SqlRegisteredServerName)
[ ] No un-handled errors which stop the command working with multiple servers

@ctrlbold I have and I am.

Sorry for the delay @wsmelton, gotta build a new AG cluster to test. But at least I'll finally have AGs in the lab again 💃


Animal Crossing City Folk Issues?

So when editing my Animal Crossing City Folk town, I ran into multiple issues with the editor. The first being that my birthday shows as this
 . Second, when editing my acres, they show up as this?
 I'm not sure if that's supposed to happen. The same happens with my town map. Also,
 how come I can't edit the Weather/Native Fruit? Finally,  none of my villagers appear in the villagers section, which is blank. I can't change my house size either and my roof color.

Some of those features are disabled for various reasons (hence the grayed out bars). Weather & native fruit haven't been researched yet. Same with house size & roof color.
Villagers in City Folk all store their textures and names in the file, and I haven't bothered reverse engineering them yet, which is why it's blank. Before version 1.0.0 releases it'll be done.
As for the acres showing up as unknown that shouldn't happen. Can you attach your save file to a reply here so I can take a look at which acres are wrong?

The acres showing up as unknown images should now be fixed. I forgot to finish dumping all the acre map icons.

Closing this as most of the issues that were shown are now fixed. If you find any more issues, please report them!


AID-plugin

Pull Request Template
Description of Changes
Please provide a brief summary of what you have changed in this pull request. Be clear and concise.
Related Issue
If this pull request is related to an existing issue, please link that issue using the format #issue_number.
BaSyx Configuration for Testing
Describe any specific configuration settings or environment setup required to test these changes effectively. If possible, please provide the BaSyx configuration as .zip file.
AAS Files Used for Testing
Please provide any AAS files that were used in the testing of these changes. Include a brief descriptions of their relevance to the changes being proposed.
Additional Information
Include any additional information or context that could be useful for the reviewer. This could include challenges faced, alternative solutions that were considered, etc.

Please ensure that you have tested your changes thoroughly before submitting the pull request.

Uploading AID-AIMC-example.zip…


EXT: Convert "resource" section to "general"

Description
Changes proposed in this pull request:

Adjusted examples and docs
Resource group is no longer required. Previously, when I wanted to use it with Service it was necessary to set a group as an empty string. Now, if the resource belongs to the "core" (also called legacy) k8s group, the user doesn't have to define the group in the config.
Adjusted the code to use information from the general section instead of resource

Related issue(s)
Resolves #1697

/retest

/retest

/retest

/retest

/retest

/retest


User/jason/identitypermissions

Gives proper permissions app permissions to identity when registering with Azure

This change is

Getting an invalid secret from azure, I believe it's related to bug #158975, will need to investigate


Tweaks a few strings, tweaks some conditionals structures.

So uh the first commit is just the string tweaks. I do want to create a few and delete a few, though, maybe Van will help me with that if he bears with me on Discord.
Really just wording changes, not much. I'll probably describe any further changes on the commits themselves or maybe edit this.
Resolve #937

I'm gonna need maybe Van to change the string used by Faruzan in one of her burst conds to "baleCondName", if it's not too much trouble. Made a separate commit just in case.

So this one now would imply on making two conditionals instead of the current one. The 1st one would use "afterUse.burst" from sheet.json (representing the con's 1st effect), and when activated, would make another conditional appear, which would use the currently-used string, c4ReactCond, representing the con's 2nd effect. Explanations on why are in Discord. Again, separate commit just in case

For this commit to work, current uses of Sayu's "p1Swirl" need to be replaced with sheet.json's "energyRegen" and the "c4Ener" conditional needs to be removed entirely and display its effects plainly.

I'm going to opt to keep Tighnari C4 the same as it is. Less churn for users and it accomplishes the same effect. I will take the string change but not implement the conditional change.


The type of the variable in an or expression is incorrect

Issue Type: Bug
Steps:

Create a ts file;
Add example code:

class Demo {
    test(param: string = 'param') {
        const variable: number = 0;
        if (param !== 'test1' || param !== 'test2') return;

        if (variable !== 0 || variable !== 12) return;
    }
}

Put cursor on the "param" (or "variable") after "|| "

Result:

the popup appears, and the type of "param" (or "variable") is "test1"(or "0")

Expected:
the type of "param" (or "variable") is "string"(or "number")
VS Code version: Code 1.29.1 (bc24f98b5f70467bc689abf41cc5550ca637088e, 2018-11-15T19:13:36.375Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz (4 x 3408)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.96GB (1.28GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

I think this is correct. The second part of the || after !== 0 will only be evaluated when that check doesn't pass, when variable === 0

Yes it seems like the type narrowing is correct here


Update to latest libinjection

It is fine to use sha in our situation. I was just curious if this sha had
something special.
On Wed, 30 Nov 2022, 02:47 Anuraag Agrawal, @.***>
wrote:

Merged #102 https://github.com/corazawaf/coraza-proxy-wasm/pull/102
into main.
—
Reply to this email directly, view it on GitHub
https://github.com/corazawaf/coraza-proxy-wasm/pull/102#event-7917861371,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYARAFVZISYQ2N3QWYALWK2W2PANCNFSM6AAAAAASN6KPW4
.
You are receiving this because your review was requested.Message ID:
@.***>


fmt.Sprint issue with our gc/allocator

The use of fmt.Sprint leads to the failure of certain tests (executed in a Gossamer host), when used in conjunction with our custom garbage collector, which involves invoking the external host allocator. The issue is present with both wasmer and wazero runtime interpreters, allocator is the same in both cases. But, not issues are observed when doing a transfer in a Substrate host.
As a workaround, we don't use the fmt package, instead, we are converting everything as a string and log -
log.Critical("invalid inherent position for extrinsic at index " + strconv.Itoa(int(inherentsAreFirst)))
Example
this works
func (m module) idleAndFinalizeHook(blockNumber sc.U64) error {
	_ = fmt.Sprint("kaboom")
	weight := m.system.Storage.BlockWeight.Get()
...

but this does not
func (m module) idleAndFinalizeHook(blockNumber sc.U64) error {
	weight := m.system.Storage.BlockWeight.Get()
	_ = fmt.Sprint("kaboom")
...

Some host function calls are happening in between the calls. Needs further investigation.

The issue is no longer re-producable.
There was an issue with the old gossamer allocator, however, we recently updated gossamer to include a refactored allocator that more closely resembles the implementation in Substrate.
commit 39ca47fc407253426feeb5694c40b63848b51da2 (HEAD -> development, origin/development, origin/HEAD, debug-extalloc-gc-new-allocator)
Author: Eclésio Junior<EMAIL_ADDRESS>Date:   Tue Nov 14 20:05:14 2023 -0400

    feat(lib/allocator): Refactor `FreeingBumpHeapAllocator` (#3570)

    Co-authored-by: Kanishka<EMAIL_ADDRESS>
Example
This line fails with the old one:
https://github.com/LimeChain/gosemble/blob/e7e1ba5edc4668a0203f0b75fdb8d1ccd17ba6dd/primitives/types/era.go#L138


[DOC] We need a place to see the matrix of vendors and SDK support

Change in the documentation

If you’re open to some feedback on the openfeature.dev site: I can’t figure out where it lists what vendors it integrates with.

^^ Feedback I got from someone, which made me realize we're missing this.

Would something like this work?
https://github.com/open-feature/openfeature.dev/issues/63

I really want some kind of matrix or something like that, where it's made clear at a glance... perhaps with some filtering. If anyone has a UI component as a example I'd be happy yo work on it.

@toddbaert These are what I saw from a search.

https://github.com/Paraboly/react-matrix-table for a more row-oriented array style
https://github.com/bronz3beard/react-matrix has you passing around objects instead of arrays.

I've started to put together a prototype.


Create Announcement “2020-05-deep-dive-test-title”

Automatically generated by Netlify CMS

Deploy preview for dds-mil ready!
Built with commit 1b2c6e5761dee30ad1861c0faefb3713b7466108
https://deploy-preview-128--dds-mil.netlify.app


fix duplicate db conn issues

    at eval (webpack-internal:///(rsc)/./src/app/api/db/db.ts:11:12)
    at (rsc)/./src/app/api/db/db.ts (/Users/brandoncohen/Documents/dev/vacant-lots-proj/.next/server/app/api/generateTiles/[z]/[x]/[y]/route.js:112:1)
    at __webpack_require__ (/Users/brandoncohen/Documents/dev/vacant-lots-proj/.next/server/webpack-runtime.js:33:43)```

this is still happening :/


buildOnSave support in tsconfig.json

As discussed compileOnSave support in tsconfig.json is due to ship with TS 1.8.
However compileOnSave is just for stopping compiling on save as I understand.  Are there plans to support for buildOnSave as well?
My use case is this:
I have a web app which has been built using TypeScript (since TS was 0.9!)  The older part of the codebase works and uses global scope with namespaces and is written in ES5.  The newer part of the codebase is written using ES6 / modules and is compiled using Webpack with ts-loader and babel-loader.
Each part of the code base is driven by a separate tsconfig.json; however - deactivating the Visual Studio build by adding <TypeScriptCompileBlocked>true</TypeScriptCompileBlocked> to the .csproj (see here) is an all or nothing affair.
So if I turn it off I have to manually run tsc against the old code in order that it is generated.  Do-able but clunky.  It'd be nice to be able to control both build and file level compilation from tsconfig.json

A few notes, CompileOnSave (in VS 2015) already compiles all files in your project. We have no plans of supporting a watch mode in VS, tsc --w should be sufficient, as well as existing build tool integration.
Since you have other build steps, i am assuming you have some build system already wired in, so why not just use a watch capability in your build system. like gulp watch for instance?
Also, why not just put "compileOnSave": "true" in your older portion of the project, "compileOnSave": "false" in the newer portion?

Hi @mhegazy,
Are you saying that having compileOnSave set to false will effectively act as a <TypeScriptCompileBlocked>true</TypeScriptCompileBlocked> scoped to the tsconfig.json? So this would block TypeScript compilation triggered as part of a Visual Studio build? If so that's awesome!
For now we've been deactivating all compilarion with <TypeScriptCompileBlocked>true</TypeScriptCompileBlocked> and triggering 2 separate TypeScript compile tasks in our csproj. One (the old stuff) invokes tsc -p Scripts and the other (the new stuff) invokes a gulp task which triggers WebPack and some other bits and pieces.
Rather quirkily we're having to include to nuget TypeScript target in the csproj in order that Visual Studio doesn't come along and re-add the standard TypeScript target to the project and ask us if we'd like to search Definitely Typed for typings.
I'm just looking for VS to be a straight TypeScript editor really. However it's not easy. It feels like you have to "trick" the tooling into doing what you want.
BTW we are using watch modes when we're running in debug (not triggered as part of the vs build though)

PS apologies for typos - this is being written on my phone as I walk to work on a cold winter day. Fingers a bit numb!

Are you saying that having compileOnSave set to false will effectively act as a true scoped to the tsconfig.json? So this would block TypeScript compilation triggered as part of a Visual Studio build? If so that's awesome!

that is not the case. I miss understood your original post. sorry about that. the "onSave" part through me off. i think i understand the issue now. let me reiterate it to confirm, you want to have a way to not build a tsconfig.json when you have multiple tsconfig.json in one project.
if so, i think you can do this by excluding the tsconfig.json from your project. the MSBuild targets enumerate them from the project. we can add a property on the item to "ignore" it, as well.

I'm just looking for VS to be a straight TypeScript editor really. However it's not easy. It feels like you have to "trick" the tooling into doing what you want.

The tools look for a property <TypeScriptEnabled> in your project file. including the default targets sets this property for you. if the property is set, that indicates that this is a "TypeScript project" (some expectations are made here, that you have one build target, and that you are not customizing the build, no external tools etc..); from your description this does not seem to be the case. and if i am correct, you already have your build wired correctly. so what you need is to set this property to false. The language service will treat the files in your project as "loose files", i.e. will forget about the project, since you have tsconfig.json, things should work as expected.
give TypeScriptEnabled a try and let me know how it goes.

Thanks for the input @mhegazy - I'll give it a try tomorrow.

Hi @mhegazy,
Didn't work I'm afraid.  Removing tsconfig.jsons from the csproj and setting <TypeScriptEnabled>false</TypeScriptEnabled> started the project treating the 2 separate codebases as a single TypeScript context and ignored the tsconfig.jsons entirely.
So it looks like it doesn't exist the ability to control overall build from tsconfig.json doesn't exist in Visual Studio at present.
Also, it's necessary to have TypeScript targets in your csproj if you want to use Visual Studio as an TypeScript editor in use cases where there are multiple tsconfig.jsons.  However <TypeScriptCompileBlocked>true</TypeScriptCompileBlocked> will need to be set if you're doing your compilation outside of VS.

Hey @mhegazy,
Since I had a rather tricky time getting my workflow up and running in Visual Studio I decided to write it up in case others are in a similar spot.  It can be found here:  http://blog.johnnyreilly.com/2016/02/visual-studio-tsconfigjson-and-external.html
If I'm offering any bad advice then do feel free to set me straight.  Corrections are always welcome :smile:
For what it's worth I've got a workflow that seems okay.  I think the number of hoops that had to be jumped through is a shame but I'm glad that using Visual Studio for more than vanilla TypeScript scenarios now seems to be possible.  I think it could really benefit from being easier though.

so why not split your project into two projects instead?

They're part of the same app / website. To split into other projects would mean 2 websites and extra complexity. Also, in the end I want to unify the compilation approach and go all in on ES6 / modules. This is a short term measure until time allows.

Although did you mean 2 TypeScript projects rather ASP.Net MVC projects? It's effectively 2 TypeScript projects inside a single ASP.Net project as denoted by the 2 separate tsconfig.jsons

Declining due to lack of demand


https possible?

im trying to run koel via https for various reasons. i tried to follow the installation guide and get the login interface, but when i want to log in nothing happens. checking the console i get a 404 for https://server-ip/api/me
i guess that i have to run php artisan serve to get the api to work? but even then it would only be reachable with http, right?
seems like im missing something here, or is https just plain not possible/wanted?

HTTPS is definitely possible. I use fastcgi with nginx and don't have any issues.

@anonaut I had that issue too when running under a subfolder (e.g. /music). Koel works only on the root folder (I had to create an ad-hoc subdomain to host it).

@daniele-athome Yeah, I read trough other issues that running from a webdir other then root folder is not possible. I already deploy koel in the root webdir of my apache installation.
I don't really understand how the routing to the api works, since there is no api folder in the webdir. should there be one?

No, it's a virtual path handled by Koel internally. It's probably server misconfiguration. If you are using Apache, ensure that your virtual host has AllowOverride enabled. If not using Apache, you should check the URL rewriting rules of your server of choice.

@daniele-athome Thanks very much, i added AllowOverride All and everything worked.


Bootstrap Dropdown implementation

Hi,
Here is implementation of Bootstrap Dropdown with sample activity.
If u have any question or suggestion, I'll be glad to answer or improve this implementation.
Thanks!

Updated branch to be up to date with master.


[BUG][Master] process cannot finish and its status is always running

close #6882 process cannot finish and its status is always running
add 'isStart' check in event handler
add log in event handler.

LGTM


Add Tekton plugin

Description

Adds the Tekton plugin to the Showcase App. The CI tab of the App will display the latest pipeline run visualization if the tekton CI annotation (janus-idp.io/tekton-enabled: 'true') is available.
Updated the topology package version

Which issue(s) does this PR fix

Fixes https://github.com/janus-idp/backstage-showcase/issues/172

PR acceptance criteria
Please make sure that the following steps are complete:

GitHub Actions are completed and successful
Unit Tests are updated and passing
E2E Tests are updated and passing
Documentation is updated if necessary
Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Add ${K8S_CLUSTER_NAME} , ${K8S_CLUSTER_URL} ,  ${K8S_CLUSTER_TOKEN} info
Create a pipeline run in the above cluster and ensure that the resource has the "backstage.io/kubernetes-id= test-backstage" label

Screenshots

We don't need the TEKTON_ENABLED flag as there is no Tekton backend plugin.
Once the code conflict is resolved we can merge it.
Thanks @debsmita1 for the plugin and this PR

We don't need the TEKTON_ENABLED flag as there is no Tekton backend plugin.
Once the code conflict is resolved we can merge it.
Thanks @debsmita1 for the plugin and this PR

I have also updated the Topology plugin package version in this PR. Let me know if that's okay or if I should raise a separate PR ?

I have also updated the Topology plugin package version in this PR. Let me know if that's okay or if I should raise a separate PR ?

I usually prefer to have another PR just in case we need to revert something. But it's fine for this one.


Rigging the roulette

You want to double-check it though. I don't want to break everything FeelsBadMan

Never checking out pull requests FeelsBadMan

I've checked it out, I'm just not sure whether this is something I want though

Oh sure, your bot, your choices.


Zigbee2mqtt wont start, 500 Server Error

I am using HomeassistantOS and installed zigbee2mqtt addon. But it fails to start.
Zigbee2mqtt add-on version :1.22.1-1
entire configuration from the frontend:
external_converters: []
devices: devices.yaml
groups: groups.yaml
homeassistant: true
permit_join: false
mqtt:
  base_topic: zigbee2mqtt
serial:
  port: /dev/ttyUSB0
advanced:
  log_level: warn
  pan_id: 6754
  channel: 11
  network_key:
    - 1
    - 3
    - 5
    - 7
    - 9
    - 11
    - 13
    - 15
    - 0
    - 2
    - 4
    - 6
    - 8
    - 10
    - 12
    - 13
  availability_blocklist: []
  availability_passlist: []
device_options: {}
blocklist: []
passlist: []
queue: {}
frontend:
  port: 8099
experimental: {}
socat:
  enabled: false
  master: pty,raw,echo=0,link=/tmp/ttyZ2M,mode=777
  slave: tcp-listen:8485,keepalive,nodelay,reuseaddr,keepidle=1,keepintvl=1,keepcnt=5
  options: '-d -d'
  log: false

Running platform
Core: core-2021.12.1
Supervisor: supervisor-2021.12.1
Operating System: Home Assistant OS 7.0
Logs:
invalid argument")
2021-12-14 11:20:18 ERROR (MainThread) [homeassistant.components.hassio] Failed to to call /addons/45df7312_zigbee2mqtt/start - 500 Server Error for http+docker://localhost/v1.41/containers/addon_45df7312_zigbee2mqtt/json: Internal Server Error ("readlink /mnt/data/docker/overlay2/core-configurator

I found out that there was an old docker contaioner which hass-supervisor did not recognize since last homeassistant update. So it won't start up with conflict conationer. I ssh using port 22222 for higher permission and using docker command to rename old container. It worked!


Smallfix

Delete whitespace.
Compliance PEP8
Better readablility.
Delete duplicate name in variable.

Awesome!

Coverage decreased (-0.5%) to 48.45% when pulling a11ace385d42a138bf63757716b4a200964632a3 on GeonU:smallfix into bdc0a7dd9b8339d9f608165c7534065b47d2fae4 on Parkayun:master.

Coverage decreased (-0.5%) to 48.45% when pulling a11ace385d42a138bf63757716b4a200964632a3 on GeonU:smallfix into bdc0a7dd9b8339d9f608165c7534065b47d2fae4 on Parkayun:master.


Is there a way to import a project from the base digital logic sim ?

Is there a way to import a project from the base digital logic sim from https://sebastian.itch.io/digital-logic-sim ?
the title say it all

There is now. Last version you couldnt
Let me see how to do it

Ok yeah idk


Swapped biorxiv links out to doi

@jcua1  Thanks for updating - I'm merging your changes to master.


Adds support for auto-publish to npm

Fixes #11

This adds support for a nicer publish flow. Now, when someone changes the version in package.json to X.Y.Z and pushes a commit with the message "Release X.Y.Z, the new Publish action will create a new tag vX.Y.Z and publish the package to the npm registry.

This will automatically publish over latest, so it will get the latest tag. Which I think is right. If we want to do some "next" releases, or "dev" releases, then I think this should actually be a manual thing for now. Having said that, we could consider automatic that process as well, and having a separate action that watches for -dev versions and publishes under a different tag?

This will automatically publish over latest, so it will get the latest tag. Which I think is right. If we want to do some "next" releases, or "dev" releases, then I think this should actually be a manual thing for now. Having said that, we could consider automatic that process as well, and having a separate action that watches for -dev versions and publishes under a different tag?

Making publishing dev, or others, manual seems good for now.  It's been very rare so far, so doing this as is seems great!


Database Development Completed

The development of database is completed based on the current iteration of the data base design.

Database dev competed and merged to the master!


Stepper power not supplied

Pins are set correctly, but like they are not enabled.
It's mega with RAMPS1.4
Motors are controllable over marlin, but not trough moveo_ros
I tried supplying power to Enable pin to the driver separately, seems that is not the issue.
Are endstops necessary?
Do you have any ideas I might have overseen?
Best,
B

What kind of drivers are you using?
It could be that you need to add a line to the arduino code to make a pin (like standby) of your driver actively high or low, instead of floating.
Endstops are not necessary.

Just installed DRV4429 instead of LV8729 which I have been happily using quite some time.
I have some DRV8825, do you think they may help?
Funny enough DM452T's on NEMA23's are powered, but they are getting all the wires from RAMPS X driver slots.
What kind of line code do you suggest, I'll google it from there
Thanks

Doing such a connection on ramps is kinda difficult.
Isn't there other solution like in some 3d printer firmwares ?

may it be related to the fact that there is no /joint_steps_feedback topic created
joint_steps and gripper angle are there on the other hand

I thought that it was the drivers, so I just put 4 brand new drv4288. Didn't help.
Tomorrow I'll try new 8825s
I do control the base joint nema 23s and the servo trough the rviz, rosserial lounched folowed by convert and published gripper angle, all seems well except there is no feedback topic.
Than you a lot

@ghost for the feedback topic, it looks like you have to uncomment the following line in the arduino code https://github.com/jesseweisberg/moveo_ros/blob/master/moveo_moveit/moveo_moveit_arduino/moveo_moveit_arduino.ino#L76
Did you manage to make the stepper motors move? and in that case, what drivers did you use? I have not been able to, using A4988 pololu

Closing due to inactivity.


docs: run how-to guides in CI

Add how-to guides to Run notebooks job and fix existing notebooks.

As with tutorials, cassettes must be updated when HTTP calls in guides change (by running existing script).
Cassettes now total ~62mb over 474 files.
docs/scripts/prepare_notebooks_for_ci.py lists a number of notebooks that do not run (e.g., due to requiring additional infra, slowness, requiring input(), etc.).

PR is hitting threshold of 300 files that engages CI for all packages.
Merging despite failures in airbyte, which looks blocked on pydantic issues.


MasterInitial Commit of Find-DbaDatabaseGrowthEvent

Returns any database AutoGrow events in the Default Trace for every
database on the SQL instance.

Nice work, buddy!


feat: dropdown: add ability to portal

SUMMARY:
feat: dropdown: add ability to portal
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
https://eightfoldai.atlassian.net/browse/ENG-22951
CHANGE TYPE:

[ ] Bugfix Pull Request
[x] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@e9c7617). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #210   +/-   ##
=======================================
  Coverage        ?   86.94%
=======================================
  Files           ?      207
  Lines           ?     6835
  Branches        ?     2265
=======================================
  Hits            ?     5943
  Misses          ?      878
  Partials        ?       14

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 e9c7617...5dce100. Read the comment docs.


ci: publish in secure automated publish

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

Project coverage is 29.33%. Comparing base (d4c6d46) to head (7b3b345).

Additional details and impacted files

@@           Coverage Diff           @@
##              dev     #163   +/-   ##
=======================================
  Coverage   29.33%   29.33%
=======================================
  Files          70       70
  Lines        4305     4305
  Branches       88       88
=======================================
  Hits         1263     1263
  Misses       3042     3042


pcsclite: 1.8.24 -> 1.8.25

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/pcsclite/versions.
meta.description for pcsclite is: '"Middleware to access a smart card using SCard API (PC/SC)"'.

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.8.25 with grep in /nix/store/kd4vpmw5z9g39ygmgc3p7fch02rlpp2s-pcsclite-1.8.25-bin
directory tree listing: https://gist.github.com/a0a44c58fbe99259cb5274f8b1231232
du listing: https://gist.github.com/f1c179f784e50d51e3145386e4750376

Rebuild report (if merged into master) (click to expand)

1951 total rebuild path(s)
671 package rebuild(s)
670 x86_64-linux rebuild(s)
636 i686-linux rebuild(s)
26 x86_64-darwin rebuild(s)
619 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
acsccid
adapta-gtk-theme
afew
akonadi
akregator
almanah
alot
altcoins.aeon
altcoins.wownero
amarok
amarok-kf5
appimagekit
aptly
arc-theme
ark
astroid
atom
atom-beta
balsa
bluedevil
breeze-qt5
browserpass
buildah
calamares
calligra
cantata
cardpeek
ccid
centerim
chrome-gnome-shell
chrome-token-signing
claws-mail
clementine
clementineUnfree
colord-kde
connman
connman-gtk
connman-ncurses
connman_dmenu
connmanui
dar
darwin.security_systemkeychain
debian-devscripts
deja-dup
digikam
dino
distrobuilder
dolphin
dragon
duplicity

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kd4vpmw5z9g39ygmgc3p7fch02rlpp2s-pcsclite-1.8.25-bin \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A pcsclite https://github.com/r-ryantm/nixpkgs/archive/fedc568208fd6c71991956eb06a4565310027b73.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kd4vpmw5z9g39ygmgc3p7fch02rlpp2s-pcsclite-1.8.25-bin
ls -la /nix/store/kd4vpmw5z9g39ygmgc3p7fch02rlpp2s-pcsclite-1.8.25-bin/bin

https://github.com/LudovicRousseau/PCSC/blob/master/ChangeLog

@GrahamcOfBorg build pcsclite


Make pip-installable

It'll be useful to have this package pip-installable (from the local directory) during development. Going to implement the rudimentary backbone for this now, can extend later as we see fit.

Resolved with #34.


disable some unsupported syntax

#3373
disable some syntax when subquery is not in binary predicate in case when clause

Hi @yangzhg , you need to create an issue to describe your problem, better to give an example about the query you try to disable.

Hi @yangzhg , you need to create an issue to describe your problem, better to give an example about the query you try to disable.

done

@yangzhg Please add a UT


[EG] Monthly patch release for June 2022

June patch release for EventGrid

API change check
API changes are not detected in this pull request.


Launch on boot linux

Summary

Updated Linux methods to enable and disable autostart.

Tested .deb and .AppImage on ubuntu 20.04, the autostart works but the --minimized argument is not working.
Also checked #75  and it is working.

Before these changes, it was working on macOS and Windows for this branch launchOnBoot.
After these changes, hopefully, it should also work on Linux.
Tonight, I will test this latest version in all 3 major platforms on my local machines. Will approve after that!


Discussion Board

https://app.leanboard.io/board/576d3701-d48f-4dc4-98f6-937623f30b0e

discussion board


Agree on VCS notifier and logger messages format and content

This is likely sorted by now (mostly thanks to resource bundles), but let's revisit at some point anyway :thinking:

We worked out a reasonable state for now.


[Bug] HTML language does not recognize inline

Reproducible in vscode.dev or in VS Code Desktop?

[x] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Code
monaco.editor.create(document.getElementById('container'), {
	value: `<script type="module">\n\tconsole.log("hello");\n</script>`,
	language: 'html'
});

Actual Behavior
The contents of the script tag are marked as language vs.editor.nullLanguage, precluding highlighting and, upon editing or merely moving the cursor within the tag, causing various callbacks for editor features to throw Unexpected languageId when calling LanguageConfigurationRegistry.getLanguageConfiguration.
Expected Behavior
Inline code is recognized as JavaScript, is syntax highlighted, and editing/navigation does not raise exceptions.
Additional Context
This began causing exceptions in version 0.32.0, with 0.31.1 and prior nothing is raised but the inline code is still similarly marked as vs.editor.nullMode.

I can reproduce. It seems like type="module" tokenizes the content as language module, which does not exist - thus the error. type="html"or type="css" works as expected.

Interestingly, type=module (no quotes) works just fine!

We fixed this in vscode. The next monaco editor version will be released soon.

0.34.0 has this issue also
<script type="text/javascript">
var qd = {};
</script>

Any click inside script tag produce

Same with Markdown, when annotating a code code block with unknown language:
```mermaid
graph TD;
  A-->B;
  A-->C;
  B-->D;
  C-->D;
```

Error:
errors.ts:21 Uncaught Error: Language id "vs.editor.nullLanguage" is not configured nor known

Error: Language id "vs.editor.nullLanguage" is not configured nor known
    at f (languageConfigurationRegistry.ts:130:10)
    at c.getLanguageConfiguration (languageConfigurationRegistry.ts:113:13)
    at u.getLanguageConfiguration (tokenizationTextModelPart.ts:329:45)
    at u.getWordAtPosition (tokenizationTextModelPart.ts:350:9)
    at Ot.getWordAtPosition (textModel.ts:1860:42)
    at F._getWord (wordHighlighter.ts:355:21)
    at F._run (wordHighlighter.ts:373:21)
    at F._onPositionChanged (wordHighlighter.ts:347:8)
    at wordHighlighter.ts:224:9
    at t.invoke (event.ts:607:17)
    at errors.ts:26:12

(reproduced on the https://microsoft.github.io/monaco-editor playground).

reproducible in the monaco editor playground

monaco.editor.create(document.getElementById('container'), {
	value: "```notsupport\nerror when press enter\n```",
	language: 'markdown'
});

solve this problem

monaco.languages.register({id: 'vs.editor.nullLanguage'})
monaco.languages.setLanguageConfiguration('vs.editor.nullLanguage', {})

monaco.editor.create(document.getElementById('container'), {
	value: "```notsupport\nerror when press enter\n```",
	language: 'markdown'
});

Error: Language id "vs.editor.nullLanguage" is not configured nor known

This will be fixed with the next release of the monaco editor.

Inline code is recognized as JavaScript, is syntax highlighted,

This is the remaining feature request of this issue.

Was this fix released?

Tested https://github.com/microsoft/monaco-editor/issues/2962#issuecomment-1301587105 in playground, the error is present in 0.34.1 and gone in 0.35.0.

I have just tested this again:

❌ 0.33.0
✔ 0.34.0


Unnamed Providers

Solves #4. Drops the name field in prvd.new and prvd.Provider in favor of using debug.info(2, "s").

I'm going to close this infavor of pushing as its own release.


Service Community

It's around Idiot Village. Basically they are the ones stuck in lousy jobs but they get to grouse about it together.
Also be sure to have the monkey and Sly acknowledge you once you help one or both of them. SC and IV will come together to take down the MB. Class warfare!


DoNOTmerge: Test new travisfile method

Test new travisfile:
- run test scenarios only on development branch
- do not run scenarios on master
- provide list of scenarios to run in the travisfile (expected: scenario in development + any dependencies)
We have experienced a few paint points with travis tests including:
-waiting on other tests
-getting other tests to pass with reruns
-tests having to be excluded altogether
Usually we are uninterested in our test matrix of 17-odd tests, which have been proven to work at time of latest development. Our code changes often shouldnt affect these.
The only reason I see for keeping things this way is as a way to check that the web dependencies in our roles still work as expected. This can be handled through user issues.

.·´¯`(>▂<)´¯`·.


Integrate Milvus (pymilvus) library

Milvus is a popular open-source vector database. We should add a new vector index to support this project.

Hi @mariosasko，Just search randomly and I found this issue~ I'm the tech lead of Milvus and we are looking forward to integrate milvus together with huggingface datasets.
Any suggestion on how we could start?

Feel free to assign to me and we probably need some guide on it

Hi! For starters, I suggest you take a look at this file: https://github.com/huggingface/datasets/blob/master/src/datasets/search.py, which contains all the code for Faiss/ElasticSearch support. We could set up a Slack channel for additional guidance. Let me know what you prefer.

Hi! For starters, I suggest you take a look at this file: https://github.com/huggingface/datasets/blob/master/src/datasets/search.py, which contains all the code for Faiss/ElasticSearch support. We could set up a Slack channel for additional guidance. Let me know what you prefer.

Sure, we take a look and do some research


Add focus as a ref usage case

Per https://github.com/react-hook-form/react-hook-form/issues/230#issuecomment-522764265

Thanks @dandv