🐛 Bug Report: [Android] Switching to any authenticator app while creating a Google OAuth Session breaks the login flow

👟 Reproduction steps
Requirements

Google Account with an activated 2FA (two-factor-authentication) option
An android device which is logged into the same google account
appwrite project with Google OAuth setup

Reproduction

Login into your google account through createOAuth2Session(provider: "google")
You switch to your 2FA application
After confirming your 2FA, the login window is automatically closed but you're not logged in

Further information
Article how to add Google 2FA to your account: https://support.google.com/accounts/answer/185839
I've created a minimal reproduction repository:
https://gitlab.com/metalwings/appwrite-flutter-oauth2-demo
In my case the default "is this you?" screen is automatically shown by android as soon as I try to login into my account.

👍 Expected behavior
Switching to any authenticator app doesn't break the login flow.
Users should be able to switch to other app (in my case the default android auth by google, check my 2FA Code (or press "that's me") and login after I've completed the 2FA flow
👎 Actual Behavior
The login flow is cancelled due to loss of app focus.
An exception is thrown:
E/flutter ( 6554): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(CANCELED, User canceled login, null, null)
E/flutter ( 6554): #0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:652:7)
E/flutter ( 6554): #1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:310:18)
E/flutter ( 6554): <asynchronous suspension>
E/flutter ( 6554): #2      FlutterWebAuth2MethodChannel.authenticate
(package:flutter_web_auth_2_platform_interface/method_channel/method_channel_flutter_web_auth_2.dart:16:7)

🎲 Appwrite version
Version 2.0.x
Appwrite Cloud
SDK Version 11
💻 Operating system
Linux
🧱 Your Environment
Problem can be reproduced in the appwrite cloud version
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Playing around with the android:launchMode in https://gitlab.com/metalwings/appwrite-flutter-oauth2-demo/-/blob/main/android/app/src/main/AndroidManifest.xml?ref_type=heads#L9 as suggest by @stnguyen90 didn't help.

Tested on iOS.
Everything works there, because the sign-in window behavior is different.
Also found a new clue: For some reason, the browser window "Sign in with Google" is closed automatically as soon as I try to switch to another app.
https://github.com/appwrite/sdk-for-flutter/assets/6256062/97e719c2-1f23-40f0-849d-db91f00a3d2d

Workaround:

Initialize the OAuth Session in a separate WebView / CustomTab (with url_launcher for example)
After successful login, trigger the AppWrite OAuth Flow


Update email, phone, url crawling

Hi, What is the purpose of this pull request? Please add a description.
Also, there is a lot of of commented out code, please clean it up before submitting a pull request


weird geometry that floats around

should really consider into parsing bsp data and retrieving the complete main tree, and nothing else

Doesn't seem to be possible without manual level editing. Floating geometry (teleported-to map areas, cameras, etc.) doesn't differ in semantics from "main" part of the map geometry, so it seems there's no easy way to cull it automatically.


[AMBARI-24338] [Log Search UI] Add application routing information to README.md

What changes were proposed in this pull request?
There were request to get information about application routing because there are external links from other applications to Log Search. So that we added the basic Logs Screen routing to the readme.
How was this patch tested?
No need for testing. Unit tests are still success: Executed 268 of 268 SUCCESS (7.558 secs / 7.438 secs)
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3257/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3279/
Test PASSed.


Opposite order of optimizer and lr_scheduler in pytorch 1.1.0+

Hi, when I am training nerf in pytorch 1.13 with cuda 11.7, one warning pops out

UserWarning: Detected call of lr_scheduler.step() before optimizer.step(). In PyTorch 1.1.0 and later, you should call them in the opposite order: optimizer.step() before lr_scheduler.step().  Failure to do this will result in PyTorch skipping the first value of the learning rate schedule.

Do you have any idea if this warning will affect the NeRF training? Thank you!

Can you test #1854 to see if that solves the issue? I'm not seeing the warning locally so I can't confirm that this fixes it.

I cannot reproduce the original warning. I was hoping this would fix the error when running nerfacto in torch 2.0.
#1883


WIP - Send API

Implementation of the Send API
And some helpers

Sorry for leaving this open for so long. This all made it into the gem one way or another, though.


Portfast configuration for access-ports

This is useful for fast convergence in L2 segments.
Should be a best practice.

Part of work done by @aphillipps with PR #98


bugfix: fix vlan interface ensure error

Pull Request Description
Describe what this PR does / why we need it
Fix "invalid memory address" error while creating vlan interface.
Does this pull request fix one issue?

#124
Describe how you did it
Describe how to verify it
Special notes for reviews

Codecov Report

Merging #125 (134c7b8) into main (1d8fa3c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #125   +/-   ##
=======================================
  Coverage   19.41%   19.41%
=======================================
  Files          31       31
  Lines        2509     2509
=======================================
  Hits          487      487
  Misses       1965     1965
  Partials       57       57

Flag
Coverage Δ

unittests
19.41% <ø> (ø)

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

Continue to review full report at Codecov.

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


When onElementClick is defined, user can't navigate in the flow on top of nodes

When defining a onElementClick method on the ReactFlow component, the end user can not navigate on top of existing nodes.
It can be easily reproduced on the following example https://reactflow.dev/examples/interaction/.

Navigate on top of nodes works fine
Click on the : capture onElementClick checkbox
You can not navigate anymore on top of nodes

I guess it is the default behaviour, but it is possible override that ? I would like to have clickable elements in read only mode.
One workaround would be to have a specific keyboard to hold to navigate in the flow, but is that doable ?

Hey @clement-faure
you are right. This is the default behaviour. You could implement your own solution with a keyboard event that toggles elementsSelectable and onElementClick.

Hey @clement-faure
you are right. This is the default behaviour. You could implement your own solution with a keyboard event that toggles elementsSelectable and onElementClick.

Well noted, thank you.


Drush ULI task should inform the user when UID1 is blocked

Describe the bug
At the moment, if UID1 is blocked, then drush will still generate a one time login link, however when visited, it will yield 'access denied' (see the s.o question).
To Reproduce

Block UID1
Generate one time login link
Click it
See error

Expected behavior
Perhaps drush can check to see if UID1 is blocked as a pre-flight command, and if so, it will present the user with a message informing them of such, and refusing to generate a one time login link.

...while at it, perhaps also add a more meaningful error when the user does not have permission to run the task ...here's what's thrown when trying to run the "drush uli" task on master:

...better yet, hide the task in the first place, if the user does not have permission to run it.

Drush ULI task to be rewritten anyway


Cannot run singularity: failed to set securebits

Please fill out the below information:

Your Windows build number:  (Type ver at a Windows Command Prompt)
10.0.18362.207

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)

$ singularity --version
singularity version 3.2.1-1
$ singularity run library://sylabsed/examples/lolcow
ERROR  : Failed to set securebits: Invalid argument

What's wrong / what should be happening instead:

Singularity is trying to set securebits but fails:
    if ( prctl(PR_SET_SECUREBITS, SECBIT_NO_SETUID_FIXUP|SECBIT_NO_SETUID_FIXUP_LOCKED) < 0 ) {
        fatalf("Failed to set securebits: %s\n", strerror(errno));
    }

Related: https://github.com/microsoft/WSL/issues/3641

Are you using WSL2? You can get it from wsl.exe -l -v command.

@Biswa96 No I'm not. I'm not running Windows Insiders. Would be good to clarify if it will ever be supported in WSL 1. If not, then this issue can be closed.

Folks, I have been seeing this issue two weeks now. It looks to me that some read/write error of some sort happened to libdvd-pkg.
All I did to get this error is to run on WLS the update commands like I used to do in Ubuntu:
sudo apt update && sudo apt upgrade -y
I then get this error:
etting up libdvd-pkg (1.4.2-1-1) ...
libdvd-pkg: Checking orig.tar integrity...
/usr/src/libdvd-pkg/libdvdcss_1.4.2.orig.tar.bz2: OK
libdvd-pkg: Unpacking and configuring...
libdvd-pkg: Building the package... (it may take a while)
libdvd-pkg: Build log will be saved to /usr/src/libdvd-pkg/libdvdcss2_1.4.2-1~local_amd64.build
failed to set securebits to 024/0x14
dpkg: error processing package libdvd-pkg (--configure):
installed libdvd-pkg package post-installation script subprocess returned error exit status 1
libdvd-pkg: Checking orig.tar integrity...
/usr/src/libdvd-pkg/libdvdcss_1.4.2.orig.tar.bz2: OK
libdvd-pkg: Unpacking and configuring...
libdvd-pkg: Building the package... (it may take a while)
libdvd-pkg: Build log will be saved to /usr/src/libdvd-pkg/libdvdcss2_1.4.2-1~local_amd64.build
failed to set securebits to 024/0x14
E: Sub-process /usr/bin/dpkg returned an error code (2)
Please tell me what else you need to fix this?
Thx

I'm still seeing this issue using WSL / Ubuntu 20.04.1 LTS (focal).
$ sudo apt-get install libdvd-pkg
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following NEW packages will be installed:
  libdvd-pkg
0 upgraded, 1 newly installed, 0 to remove and 0 not upgraded.
Need to get 0 B/14.9 kB of archives.
After this operation, 80.9 kB of additional disk space will be used.
Preconfiguring packages ...
Selecting previously unselected package libdvd-pkg.
(Reading database ... 99222 files and directories currently installed.)
Preparing to unpack .../libdvd-pkg_1.4.2-1-1_all.deb ...
Unpacking libdvd-pkg (1.4.2-1-1) ...
Setting up libdvd-pkg (1.4.2-1-1) ...
libdvd-pkg: Checking orig.tar integrity...
/usr/src/libdvd-pkg/libdvdcss_1.4.2.orig.tar.bz2: OK
libdvd-pkg: Unpacking and configuring...
libdvd-pkg: Building the package... (it may take a while)
libdvd-pkg: Build log will be saved to /usr/src/libdvd-pkg/libdvdcss2_1.4.2-1~local_amd64.build
failed to set securebits to 024/0x14
dpkg: error processing package libdvd-pkg (--configure):
 installed libdvd-pkg package post-installation script subprocess returned error exit status 1
dpkg: error: failed to remove my own update file /var/lib/dpkg/updates/0000: No such file or directory
E: Sub-process /usr/bin/dpkg returned an error code (2)

Do you need any other information?

Any update on this issue
INFO:    Starting build...
INFO:    Use cached image
WARNING: failed to set O_CLOEXEC flags on image
FATAL:   While performing build: packer failed to pack: root filesystem extraction failed: extract command failed: WARNING: failed to set O_CLOEXEC flags on image
WARNING: failed to set O_CLOEXEC flags on image
ERROR  : Failed to set securebits: Invalid argument

@therealkenc Hello?

This is still happening for me in version 3.8.1?
~/workspace/sw/singularity-3.8.1$ singularity shell lolcow_latest.sif
WARNING: failed to set O_CLOEXEC flags on image
WARNING: failed to set O_CLOEXEC flags on image
ERROR  : Failed to set securebits: Invalid argument

I am using WSL2. Do I need to update it somehow?

Same here.  How to fix?   Can't install anything or upgrade.  HElp!!

I figured this issue by "sudo".

Still have this problem and none of the solutions shown are helpful

Suggest u update wsl1 to wsl2，use this command: wsl --set-version Ubtuntu-22.04 2. Singularity running di ok on type wsl2.

Hi, I am using windows version 10.0.19045.4170 and WSL
ubuntu version 22.04.3 (jammy)
Following the website https://apptainer.org/docs/admin/main/installation.html#install-from-pre-built-packages, I tried to install apptainer from the source. Following the procedure there, I installed the system dependancies, Installed Go, cloned the repo. While compiling the Apptainer, I was able to execute the commands ./mconfig and cd $(/bin/pwd)/builddr. But when executing make and sudo make install command, it said no target specified and no makefile found.
But I was still able to get apptainer --version as 1.3.0 and added the path to bashrc file. The problem was when testing and checking build configuration. I got the errors, ERROR  : Failed to set securebits: Invalid argument
ERROR  : Child exit with status 1. So I am not able to proceed any further. Can anyone guide me a way through this ?

I tried to update to wsl2 using this command wsl --set-version Ubtuntu-22.04 2 and this is what I get, "There is no distribution with the supplied name.
Error code: Wsl/Service/WSL_E_DISTRO_NOT_FOUND"


Add changes to bootstrap tool for migrating helix resources to Full auto

Changed HelixBootstrapUpgradeUtil class to base class and extended HelixBootstrapUpgradeUtilSemiAuto and HelixBootstrapUpgradeUtilFullAuto from it.
Added method migrateToFullAuto in HelixBootstrapUpgradeUtilFullAuto class to help set up cluster/instance/resource configs for migration to Full Auto.

Codecov Report
Patch coverage has no change and project coverage change: -0.03 :warning:

Comparison is base (fa4e8f5) 14.44% compared to head (c2b3678) 14.42%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2427      +/-   ##
============================================
- Coverage     14.44%   14.42%   -0.03%
- Complexity     2357     2359       +2
============================================
  Files           773      776       +3
  Lines         60726    60835     +109
  Branches       7440     7449       +9
============================================
+ Hits           8771     8774       +3
- Misses        51442    51546     +104
- Partials        513      515       +2

Impacted Files
Coverage Δ

...ava/com/github/ambry/clustermap/RebalanceMode.java
0.00% <0.00%> (ø)

...ub/ambry/clustermap/HelixBootstrapUpgradeUtil.java
0.00% <0.00%> (ø)

.../clustermap/HelixBootstrapUpgradeUtilFullAuto.java
0.00% <0.00%> (ø)

.../clustermap/HelixBootstrapUpgradeUtilSemiAuto.java
0.00% <0.00%> (ø)

...ub/ambry/clustermap/HelixBootstrapUpgradeTool.java
0.00% <0.00%> (ø)

... and 4 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Will put up another change without refactoring existing code.


(docs proposal) add example tool: http-server-pwa and/or pupperender

Hey there, I have made two new modules these days that are related to Puppeteer in some way:

pupperender - an ExpressJs middleware for rendering PWAs to bots using Puppeteer.
http-server-pwa - http-server alike server but for serving and rendering PWA (using pupperender).

Do you think are worth to add within the examples README? cc/ @ebidel

Yea, these are similar to puppetron and rendertron. But the express middleware seems generally useful to others. How about just adding that to examples/README? PR?

Yup, I'll prepare a PR for the middleware. Thank you!


Top users without specific language

When I clicked 'Top users by country' it will sort to top Javascript devs too.

Is this a duplicate of #125?


Document molecule as a deployment tool

Proposal: Why not utilize Molecule in different ways than for ansible role-testing ?
Author: Grzegorz Nowak <@grzegorznowak>
Date: 2019-MM-DD

Status: New
Proposal type: branding, sort of ?

Motivation
Hey guys, I've been using molecule for about 2 years now, and while it initially indeed served me well for only integration/unit testing of Ansible roles I was brewing in-house for our servers, nowadays I'm actually using it for various infrastructure deployments. Including live services.
Bit, by bit, while familiarizing myself with the tool's philosophy and architecture it became clear the way it was built, outgrew the test-only scenario it's author announced it to be solving.
Big kudos for that btw.!
I truly appreciate the ability to define my servers in the language of Ansible roles, tied together with a sexy Molecule yaml and have that deployed/upgraded wherever needed.
I mean it's already clear that molecule can do all this if you know it enough, so why bother making it more obvious ? Well the thing is for a newcomer, who doesn't really understand the full potential of the tool, it will probably just sound too narrow and too complex a tool, to bother starting with.
Where in fact, at least for me, it's the only way I'm using LXC/LXD these days for dev machines, and can't even imagine going back to pre molecule days, and not having my teams' dev clusters provisioned using it's capabilities. I find it way easier and lighter than doing dockerfiles and composing them (yes I know not an objective metric but if works for me, might work for someone else too) .
I know many people will fallback to using dockers anyway, because in theory they are easier to go about... until you realize that in practice they aren't!
With only a few touches of ansible (galaxy), combined with molecule's config one can setup perfectly fitted environment, that is already on steroids with testing/linting/idempotence-checking etc. simply put, molecule'd!
If you guys agree that we should emphasize this side of the project more, and get it into boots of proper orchestration platform (that it already is behind the scenes), I'm happy to provide some not-trivial real-life configs that encapsulate clusters of servers for real-life development. And have them plug into documentation... somewhere fitting.
It's hard to tell if that's even on cards, so would appreciate being enlightened as to why we wouldn't want molecule be on par with tools like Docker Compose or Kubernetes, if we do.
Any thoughts on that lads ?
Problems
Possibly will need to be adding more test scenarios to cover this particular use case
Solution proposal
Basically the solution lies in documentation mostly, as the software is already capable of doing that easily.

That is a good idea. I think that first step would be to pinpoint few such use cases so we can document them and link to a repository that contains them. I am really curious about the subject.

Hey @ssbarnea , I'm more than happy to hear that. Let me shove your way the simplest use case I could find in  my (client's in this case) repos:
https://github.com/spottmedia/rust-wasm-starter
the key bit with setup instructions lie in the README.md under the Provisioning section.
I'm yet to prepare grander scale setup to be uploaded publicly (gotta strip those private bits first), but hopefully this will give you some overview of the approach I'm using on my end. Not the bleeding edge in term of the best configs' foldout, but close to.

Hey @grzegorznowak, nice write up and interesting to hear!
Concretely, I think you could focus on:

As @ssbarnea said, pick your simplest use case
Write a documentation page on it. Perhaps in the FAQ to start?

I would prefer to see a "here is my approach" narrative explanation so that the idea is first clear on what can be done. Then if you want, link to configs and more general nuts and bolts stuff.
A lot of people still struggle with the command line interface and the concepts of Molecule as it is for the purpose of testing roles. I don't think the idea will reach people if you dump configs and complicated setups.
Looking forward to see what you do!

hey @decentral1se, thanks buddy.
Personally I believe in the "simplest use case" only if one can back it up with a proper live-ish solution, to begin with.  And that's not Ops specific of course.
But so then I found molecule super flexible in that regard and easily scalable.
The link I'd given is actually already the simplest I can think of, with a real-dev-live application to back it up, so if you haven't already please have a sneak peek and see what parts might be unclear. Will appreciate any feedback.
I've tested it against a couple of devs my side, and the instructions seemed sufficiently clear to follow. Or at least they claimed they did end up with the dev machine spinned up as an LXD container. Maybe just didn't want to sound noob, I don't know ;)
To me concepts and CMD line is not any harder than other orchestration tools I played with, so that's interesting to hear people are struggling.
I mean, with this exercise, given we're re-purposing a tool for a task that it wasn't built around in the first place, it might feel a bit forceful at certain stages, but really not that much, which I hopefully will demonstrate with follow up samples.
I feel motivated due to you being open to giving the project a wiggle, so cheers boys!

I'm sorry I can't put more time into giving it a look but I'll happily review a PR :+1:

Hey @grzegorznowak, after https://github.com/ansible/molecule/pull/2415 is merged, you could see about adding a "Advanced Use Cases" section and start to provide your configurations and tips on how to use Molecule in this way. Let me know if you have any issues, would be great to see this included from your end!

Thanks for letting me know @decentral1se.
Still mid pretty massive redo of our landscape using exactly the molecule-driven approach, so will have tons of real-life (or as close to as possible) examples to feed on here.
Hopefully may have some spare time this weekend, so let's assume me doing anything in theAdvanced Use Case section around that time at earliest. Cheers!

hey @decentral1se @ssbarnea ,
so let me actually get rolling this with addition of the real-world example repository, describing molecule as a provisioning tool in as tender words as I could come up with:
https://github.com/grzegorznowak/molecule-provisioning-example
I certainly em flawed by using this approach for so long I might not even know where the bits needing explanation lie anymore, so might need some help de-mudding bits there.
The giveaway of this particular write-up might be either copy-paste fragments we will want to use in docs here or just keeping the project stand-alone, and have it linked back.
Probably more happy to do the latter as this setup is more about actual codes and how they tie together, rather than describing what could be potentially done.
As I've mentioned before I didn't want to drift into the simplest use institute as it simply won't show the full benefits of molecule-driven projects, and I know people are more keen to use something when they can just copy-paste-tweak into their requirements.
On a margin, lack of examples is something that I think can be attributed to molecule's rate of adoption by newcomers, as I here had to find some answers myself too (around Goos verifies that I planned to use) and it was eventually a journey back and forth between google/docs/sources, until I finally understood where all bits should fall into.
Let me know how you see it coming all together.
As a side note no. 2, when putting the project up together I've stumbled upon https://blog.codecentric.de/en/2018/12/test-driven-infrastructure-ansible-molecule/
that describes very much the same concept as I'm proposing. Using simpler landscape and more elaborate words with possibly more-newcomer-friendly instructions.
I guess if we just had links to projects like this in docs, it might be enlightening already ?

Hey guys,
in case it's helpful in any way we have bootstrapped another molecule-oriented project for some simple cookie-cut containers:
https://github.com/spottmedia/molecule-cookie-cutter
It started as a by product of us wanting to test cover our in-house Ansible roles and configurations but we figured out it might benefit people (and project itself) more if spread out.
Personally I'm still a big believer in what molecule offers in terms of infra setups and em using it with increasing degree of confidence to deploy more and more complex clusters DEV or LIVE alike.
So hoping people will find this contribution useful too.

Last month I created https://github.com/ansible-community/protogen for the sole purpose of using it as a reference implementation about how others should write and test ansible code.
Sadly I did not had time to work more on it, even if I was able to convince @geerlingguy to join the effort. Maybe others can help on making it a reality?
The goal of the project is not to become a template, at least not a cookiecutter template. Templates are good when you are new and start from scratch by they cannot be used on exiting projects. Existing projects need manual refactoring which can take many months, but when you have a model to follow is much easier to look at it for "inspiration" and to copy paste only the bits that do apply for your case.

I second what you said about templates @ssbarnea . And for that I've released the full-blown cluster setup a few months back to show off some of the advanced molecule techniques that we are using to build our stack. Happy days, looking behind...
Anyway! So I'm all for unifying whatever can be unified, but I won't say it will be easy given plethora of possible options one has here. If I was a newcomer I definitely would like to see some "this is how you might do it" kind of tease, but only to discover I won't be actually following it in short while, with growing confidence and understanding what my actual scenarios are.
I was exploring approach of wrapping creation of molecule configs with Wizards - ie. to create replicas of webservers then plug them into load balancer on our production environment - and I think this could be one of the entry-level applicable approaches, but also one that would become obsolete for many users pretty quickly.
Let me know if you see a field upon which any of this work can be merged with yours, be it by linking back, improving upon or else, and I'd be happy to make it happen. Provided those goals align with the vision of clients/projects that are main catalyst of any work I'm able to share with community eventually.
As a standard, any new Molecule/Ansible code I push is already equipped with end-to-end CI and is able to execute molecule test without a hiccup. OR at least should.


Add support for Adeo LDSENK08 contact and vibration detector

Adding a new quirk to have both contact and vibration sensors from Adeo LDSENK08.
Follow up issue #2249

LGTM, but...
@TheJulianJES I believe that this can also be addressed in ZHA, maybe in a cleaner way?:

https://github.com/home-assistant/core/blob/52a94dd2ac50b9035555e69100db33e968e9365d/homeassistant/components/zha/binary_sensor.py#L159-L175

We can create the 2 new sensors (contact + vibration) matching the models_info. What do you yhink?

Do we having getting the the quirk class working for this in ZHA so not need adding device info for all sensors ?
If yes i think its better using quirk class if its coming more versions that is using the same functions in the future.

Hmm, with the current implementation, the bits were also stripped in the IasZone channel. That async_update method was never actually called.
However, https://github.com/home-assistant/core/pull/89481 cleans this up and and should indeed make it possible.
I'll have a look at it in a bit.

https://github.com/home-assistant/core/pull/89481 should make it possible to "cleanly" implement this in ZHA without creating artificial clusters.
However, with https://github.com/home-assistant/core/pull/91841 merged, the artificial IasZone clusters with different types should now have proper names by default in HA.
... so I guess the quirk would be fine too.


ghidra: Add version 10.0.4-20210928

Ghidra is a software reverse engineering (SRE) framework

When I use Ghidra, Ghidra\Configurations has not undergone any modification, and the content under the directory is defaultTools.
In addition, according to the Upgrade Instructions in the document, the configuration of each project is stored independently.

General Upgrade Instructions

!!!Important!!! Important!!!

      Backup by manually copying the .rep directory and .gpr file from any Ghidra
      project directories to a safe location on your file system.

    New installations of Ghidra will, by default, use the saved profile from a user's most recent
    version of Ghidra. This allows any saved tool configurations to be automatically ported to new
    projects. However, this may also prevent new tool options and features from automatically being
    configured in some cases. To open new tools containing the latest configurations, users should,
    from the Project Manager Window, choose Tools → Default Tools...

    When you open a program that was created using a previous version of Ghidra, you will be
    prompted to upgrade the program before it can be opened. The upgrade will not overwrite your old
    file until you save it. If you save it (to its original file), you will no longer be able to
    open it using an older version of Ghidra. You could, however, choose to perform a
    “Save As” instead, creating a new file and leaving the old version unchanged. Be
    very careful about upgrading shared program files since everyone accessing the file must also
    upgrade their Ghidra installation.

So, I think that Ghidra\Configurations does not need to be stored persistently.

I updated the manifest by myself because the PR is pretty outdated.
The "full version" referred by Ash should be 10.0.4-20210928. This helps us to track minor updates.

/verify


Home page message

Update Home page (page 2) and Join Us (page 3) content as in doc:
https://docs.google.com/document/d/1yeUfEvfvKOZHn9YNmFzD3IoqBPJwjLi3txcKiMam3SE/edit?usp=sharing
Leave on staging for approval.
We will improve the whole story next week together with design and images.

This is what I was thinking:


feature: add docker container for selfhost

Adding docker support
let me now if you like it or need some changes
for run this use:
Docker (recommended)
docker build -t yakgpt:latest .
docker run -d --name yakgpt -p 3000:3000 yakgpt:latest

Alternative using docker compose:
docker-compose up -d

Hey @pabl-o-ce. Thank you for this PR. I think I will merge https://github.com/yakGPT/yakGPT/pull/16 since it has a bit simpler of a Dockerfile, uses a pinned version of node and does not use docker-compose (which I'm not sure why is needd) 😅

well the docker-compose.yml is for proxy nginx but as a example how to make it but no worries then.. if you want I can PR to publish the image to github packages


Rethink ports API

The way ports are defined is confusing and should be redesigned.

Using our recommended HAProxy configuration, users have to think about an app's "local" ports, ports bound on the host, plus any NAT rules done by the containerizer.
Ports may be defined via app.ports or app.container.docker.portMappings.
The app.ports array is overloaded, with bearing on both "local" and host ports depending on the value of the app.requirePorts flag.

@lloesche @BenWhitehead @lingmann @guenter

I like three classes of ports: Host ports, Container ports (Used for port mapping with docker primarily), and Service Ports (Used purely for haproxy-marathon-bridge type setups, deprecated hopefully).
Any port which is given as a zero should get replaced with some port from the offer the task ends up being launched with.
I'd personally like for Service Ports to leave the scope of marathon entirely (I like them either as mesos task labels or magic envrionment variables which a service discovery service can scoop up). If they need to stick around I'd say they are a separate array which specifies which host port should map to a service port.
Let me know if anything in there could use more clarification for what I'd like to see.

@cmaloney I agree with your assessment of the global service ports but we have to continue supporting them for a while.

We have like 4 ways of doing this today, we need to fix it completely in v3, but otherwise out of scope.


[WIP] Generate multicluster Cluster ID by platform discovery with metadata server

Please provide a description for what this PR is for.
Today we ask users to pass ISTIO_META_CLUSTER_ID to istio-proxy, and CLUSTER_ID to pilot to help multicluster feature identify a cluster. This PR enables both istio-proxy and pilot to discover the clusterId from platform env with metadata server.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

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

@elfinhe: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test release-notes_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.

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

Test name
Commit
Details
Rerun command

release-notes_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test release-notes_istio

lint_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test lint_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.

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

Test name
Commit
Details
Rerun command

release-notes_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test release-notes_istio

lint_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test lint_istio

unit-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/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.

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

Test name
Commit
Details
Rerun command

release-notes_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test release-notes_istio

lint_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test lint_istio

unit-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-distroless-k8s-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.

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

Test name
Commit
Details
Rerun command

release-notes_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test release-notes_istio

lint_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test lint_istio

unit-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-distroless-k8s-tests_istio

integ-pilot-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-pilot-k8s-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.

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

Test name
Commit
Details
Rerun command

release-notes_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test release-notes_istio

lint_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test lint_istio

unit-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-distroless-k8s-tests_istio

integ-pilot-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-pilot-k8s-tests_istio

integ-telemetry-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-telemetry-k8s-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.

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

Test name
Commit
Details
Rerun command

release-notes_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test release-notes_istio

lint_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test lint_istio

unit-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-distroless-k8s-tests_istio

integ-pilot-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-pilot-k8s-tests_istio

integ-telemetry-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-security-k8s-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.

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

Test name
Commit
Details
Rerun command

release-notes_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test release-notes_istio

lint_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test lint_istio

unit-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test unit-tests_istio

integ-distroless-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-distroless-k8s-tests_istio

integ-pilot-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-pilot-k8s-tests_istio

integ-telemetry-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-security-k8s-tests_istio

integ-pilot-multicluster-tests_istio
f48002f21dcd5fe13a691efc7c63bb980fea417f
link
/test integ-pilot-multicluster-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.


Individual items of playground=* and area=

Hi,
I would to clarify the usage of leisure=playground and playground=*. As stated in the wiki, it seems possible to tag some individual devices with an area, but there is a very explicit warning when I try to do it.
At the same time, this behaviour is flagged as an error on Osmose, so I opened a similar ticket: https://github.com/osm-fr/osmose-backend/issues/1457 The error is a bit different: Osmose flag an area without certain tags as an error : https://github.com/osm-fr/osmose-backend/blob/master/plugins/TagFix_Area.py#L30

I'm a bit confuse and not sure what it's the appropriate way to tag them, and if it's a mistake on the schema or a personal one.

it is kind of extreme nanomapping, but it does not seem wrong

From my time mapping them (correct it where needed).
playground / equipment
Line and area mapping of playground equipment is possible, but is mainly intended for the playground equipment itself -not the surface around it- and is usually used on structures, to detail the individual parts. It is still uncommon, as this is mostly done when aerial imagery has a high resolution.

Each item of play equipment is tagged as a node, way or area depending on its shape and size

So if seperate (from a structure) I would not tag these as ways or areas.
Playground(s)...

If they are inside a leisure=playground
-- Individual surfaces around each piece of equipment can be mapped as an area area=yes + surface .
-- Individual pieces of equipment can be mapped as a node playground.
If it is a standalone piece of equipment on ie. some grass in a residential area, you can map it as you do in a playground, but without any leisure=playground area. [1]
If they are inside a leisure=schoolyard (my personal choice)
-- Individual surfaces around each piece(s) of equipment can be mapped as an area leisure=playground + surface.
--- You may choose to map these as inners of the schoolyard, but a regular area is also ok.
-- Individual pieces of equipment can be mapped as a node playground.

Sidenotes:

leisure=playground overlapping/inside another leisure=playground is, at least technically, a duplicate (one object, one osm object(/tag))
It is disputed to use playground on leisure=playground to indicate multiple pieces of equipment inside the area; it is unclear where the equipment is located -it is usually visible on imagery-, which does not add to the map.
-- [1] playground to indicate a single piece of equipment is ok, but you could argue leisure=playground (when mapped as area) for this is redundant (and tagging for the renderer) and would have to be ideally named/signed (like with man_made=bridge being disputed).

Thanks for this complete explanation!

Line and area mapping of playground equipment is possible, but is mainly intended for the playground equipment itself

Ok It makes sense. By the way, do you know if there is a way to tag the surface only underneath?

the wrong warning should be fixed with f2629f5

do you know if there is a way to tag the surface only underneath?
Nah, make an area for the general surface around the piece of equipment, as I mentioned. If you want to go as far as tag the surface underneath the seats of the seesaw, yea you can make little areas for that too. Again, might want to make it a multipolygon because it's an area inside another area. But yeah, that's another way of micromapping it further.


Removing error-checking for website URL in TagUI scripts and 2. recognizing about:blank as a valid URL?

Raising this from an user email for sharing, as other users might benefit from this question.

I'm doing a lot of testing of your TagUI functionality now e.g.

reading csv file
testing global variables
processing strings (after reading in the csv file) using different javascript functions

All the above does not require the website yet, especially during testing.
Was wondering if you could

allow us to have a switch that by pass the mandatory inclusion of a website
Or, allow us to put about:blank instead of a website

In this case the testing of scripts like the one you've given below can be much faster without the need to load https://google.com or any other website.
For your consideration, please.

This is definitely something that needs to be changed in TagUI. The error-checking for a website URL is to trigger error before even running, to save user time. However, given TagUI's direction to include automation of desktop applications through visual recognition and possible AHK integration, mandatory error-checking for URL would be irrelevant.
Would probably remove the error-checking and internally set it to about:blank if user never provide a website (to be compatible with existing framework which is based on CasperJS + PhantomJS JavaScript engine).

The 2 commits above will remove the need to start a flow file with comments or URL. When automation flow does not start with comments or URL, TagUI will start off automatically with about:blank.
A further improvement can be hiding the output of visiting about:blank, however that will 1. remove output of real visits to about:blank by users and 2. have a flow without any URL still ending with about:blank and showing that as output as the last URL before flow exits.

Above is the 3rd and final commit for this feature. 'about:blank - ' will be muted from output, eg for desktop automation workflows where no website access to web app is needed.
As the changes for this feature have large technical impact to code, made the commits separately for better tracking and future reference. With this, URL is now not required to start an automation flow.


All failing Smart meter connection requests results in endless connecting retry

When trying to connect to Smart Meter by SoapUI or Demo App.


[ML] ValidationException bug fix

Description
Validation Exceptions were being raised as exceptions when passed through log_and_raise_error. Bug has been fixed.
All SDK Contribution checklist:

[ x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ x] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Creating lsp-jdtls in hand made projects?

Hello.
If I have a project which I create manually without the use of any build tools such as maven or gradle; for each file/java_class that I make lsp-jdtls tell me
 warning balblahblah.java is not on the classpath of project thisandthat_b1d88d99, only syntax errors are reported.
I see the mave projects that works out of the box have .classpath at the root of their projects.
I want to know if there is a command to make that file base on what I have?
Or how could I make that file myself? What is the convention - syntax - style for this file?
Thanks for your help.

Normally it should work out of the box.
Please doublecheck that the first folder in your sidebar (used as workspace folder by the LSP plugin) is the project root and that each Java file is in a subfolder matching its package.
That error often occurs when you open a Java file that is not part of the first sidebar folder.

I just create a blank directory and open it on to sublime.
then start to create a directory structure I need, like:
my_created_dir>>src >>io>>starter>>{model,view,base}
Which I open my_created_dir on to sublime. then each of my . java files has io.starter.{model,view,base} package declaration at the top.
I guess it is a wrong approach?
thanks again 0/

I can reproduce this. It seems that this is a bug in LSP-jdtls.


chore: remove dependency on sdk-core from examples

Remove dependency on sdk-core from examples

Hmm looks like this was added for node version 14 to work
Error:    8:1  error  '@gomomento/sdk-core' should be listed in the project's dependencies. Run 'npm i -S @gomomento/sdk-core' to add it  import/no-extraneous-dependencies
Error:   16:8  error  "@gomomento/sdk-core" is extraneous                                                                                node/no-extraneous-import

Hmm looks like this was added for node version 14 to work
Error:    8:1  error  '@gomomento/sdk-core' should be listed in the project's dependencies. Run 'npm i -S @gomomento/sdk-core' to add it  import/no-extraneous-dependencies
Error:   16:8  error  "@gomomento/sdk-core" is extraneous                                                                                node/no-extraneous-import

I believe that error is telling you about another place where core is explicitly used in an import, I don't think it has to do with the node version. We should fix the imports.

I believe that error is telling you about another place where core is explicitly used in an import, I don't think it has to do with the node version. We should fix the imports.

ah yeah will fix

@bruuuuuuuce

we should also update the package.json and remove the dep

yep i can only successfully do that after a release with the changes in this PR


Separate DebugToolbar from CI_DEBUG

I keep running into scenarios where I'd like CI_DEBUG mode to be on but I don't want the DebugToolbar to render because it adds additional code to the response that I don't want (and can mess up AJAX responses that return HTML). Would you accept a PR for a separate constant (e.g. CI_DEBUG_TOOLBAR) to display the DebugToolbar?

No need. Comment the toolbar out in application/Config/Filters.php:
public $globals = [
		'before' => [
			// 'csrf'
		],
		'after'  => [
			// 'toolbar'
		]
	];

Thanks.


sort: disable clippy::suspicious_open_options on OpenBSD

Avoid error on OpenBSD stable/7.5 with clippy (lint)
suspicious_open_options added in Rust 1.77.0 (1.76 used on OpenBSD 7.5) https://rust-lang.github.io/rust-clippy/master/index.html#/suspicious_open_options

Fix uutils/coreutils#6290
Lint with clippy and build OK on OpenBSD stable/7.5 and FreeBSD 14.0 (Rust version 1.77).

Thanks!

@tertsdiepraam sorry about that :| Yes, I saw your comment (and their answer), and thought that no further discussion was necessary...

Alright, at least we agree on the outcome 😄


Update index.js

Sorry, I have to closing this because it's not clear what the change is about.. Please check the CONTRIBUTING guide for how to make changes to the files in this repository.


when in the vab/sph parts dont have textures when launched from the shortcut.

everything else is fine and i can just build crafts regularly from the regular launcher but not the shortcut.

The readme tells you how to fix it.
Although in KSP 1.7, the fix is not 100% effective.

This is actually something funky with the way the game runs in DirectX 12. Nothing I can do to fix it though, and is unrelated to the KerbalVR mod.


[Bug]: Log timestamps are not synchronized with sensor node(s)

Commit Hash
205e6ef
Steps to Reproduce?

Run Server
Run Sensor Node
Compare timestamp of heartbeat

Observed Behavior
Server timestamps are in UTC, Pi timestamps are sometimes in PST, sometimes in GMT
Desired Behavior
Expect the same timezone
Component
On Box Sensor Node
Relevant Sensor Node log output
2021-11-22 15:25:47,795 - RemoteSensorNode - INFO - Queued Packet E4E_Heartbeat(src=a4a5090c-fc41-47d4-a180-e82cf8bfc899, dest=a4a5090c-fc41-47d4-a180-e82cf8bfc899, timestamp=2021-11-22 15:25:47.795814)
2021-11-22 15:25:47,796 - RemoteSensorNode - INFO - Sending Packet E4E_Heartbeat(src=a4a5090c-fc41-47d4-a180-e82cf8bfc899, dest=a4a5090c-fc41-47d4-a180-e82cf8bfc899, timestamp=2021-11-22 15:25:47.795814)

2021-11-22 07:42:23,793 - OnBoxSensorNode - INFO - Queued Packet E4E_Heartbeat(src=a105a7ea-3eae-11ec-8446-a0afbd87594c, dest=a105a7ea-3eae-11ec-8446-a0afbd87594c, timestamp=2021-11-22 07:42:23.793840)
2021-11-22 07:42:23,794 - OnBoxSensorNode - INFO - Sending Packet E4E_Heartbeat(src=a105a7ea-3eae-11ec-8446-a0afbd87594c, dest=a105a7ea-3eae-11ec-8446-a0afbd87594c, timestamp=2021-11-22 07:42:23.793840)

Relevant Sensor Node configuration
No response
Relevant Server log output
2021-11-22 15:42:23,804 - ClientHandler - INFO - Got heartbeat from a105a7ea-3eae-11ec-8446-a0afbd87594c (Auto-registered device) at 2021-11-22 15:42:23.793000
2021-11-22 15:42:47,812 - ClientHandler - INFO - Got heartbeat from a4a5090c-fc41-47d4-a180-e82cf8bfc899 (Auto-registered device) at 2021-11-22 15:42:47.805000

Relevant Server configuration
No response

[ ] https://github.com/UCSD-E4E/ASM-common/pull/2
[ ] https://github.com/UCSD-E4E/ASM-rpi-node/tree/26-bug-log-timestamps-are-not-synchronized-with-server
[ ] https://github.com/UCSD-E4E/ASM-data-server/tree/logging_timestamps


There are remaining looped DB calls

All events should have a constant number of DB calls

This is addressed by #45


TEST: Merge both #1 and #3

Codecov Report

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage        ?   19.94%
=======================================
  Files           ?        2
  Lines           ?      386
  Branches        ?        0
=======================================
  Hits            ?       77
  Misses          ?      309
  Partials        ?        0


Request: Check if Lucky Egg is set

Hey,
It would be great if you could add the option to check either a Lucky egg is set or not.

Pretty sure the server doesn't tell you from the API whether lucky egg is active or not.

I believe you can find the information within AppliedItems.
// Retrieve the applied items instance
$appliedItems = $inventory->getAppliedItems();

// Retrieve the items within the applied items instance
$items = $appliedItems->getAppliedItems();

var_dump($items);

Oh right... So many files all over the place. XD


Fixed issue #365 and #386; Home page look fixed and navigation bar looks Fixed; "Join Discord" and "Join LinkedIn" now visible; Nav bar multiple issues fixed

Fixed issue #365  and #386
1. The navigation bar is now formatted properly
Old:

New:

2. Nav bar sticky throughout website
Old:

New:

3. Site name in the home page now visible
4."Join Discord" and "Join LinkedIn" is visible
Old (3rd and 4th point)

New

5. About page and nav bar look fixed

I have raised a PR from branch2, a new branch; Please check if it is okay and merge it @Its-Aman-Yadav


Failed to load dynamic library 'editor_api.dll'

Hi @icedman, Thanks for writing this lib. I'm trying to run it on windows. But it shows this error.
[ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: Invalid argument(s): Failed to load dynamic library 'editor_api.dll': error code 126
#0      _open (dart:ffi-patch/ffi_dynamic_library_patch.dart:12:43)
#1      new DynamicLibrary.open (dart:ffi-patch/ffi_dynamic_library_patch.dart:23:12)
#2      FFIBridge.load (package:editor/services/ffi/bridge.dart:34:27)
#3      main (package:editor/main.dart:31:13)
<asynchronous suspension>

the same problem.


workbench & locli:  multi-run analysis

We need to increase numeric stability of our results:

for all runs of the same profile, we need to be able to provide a consolidated summary, with ranges & stddevs

Closing this. If this is still relevant please reopen.


Request scoped DB dataloader

The DB cache should be request scoped. The main purpose of the dataloader is to stop duplicated gathering of the same data within the same request. Storing across requests causes issues with cache invalidation which are not worth tackling right now.
This has been seen in multiple frontend issues, including impactasaurus/app#195.

Achieved using dataloader's ClearCacheOnBatch


Introduce mentions into response format.

We can now find messages that were directed to the user, but we need to introduce those into the format for the content script to parse them.
TODO:

Check the current format for new messages.
Verify the case in-sensitive search works as intended.
Send "current user" with format to native client in all responses.

This will involve modifying the existing message format.

Still not certain that the case insensitive search works, but everything checks out so far.


How to work with PM2

Thank you for your amzing work. My source code is safe now.

here is the answer .
https://github.com/OsamaAbbas/bytenode/issues/12


sizing image

Challenge Size your Images has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 Please ask for help in the FCC help chat


Warn or Error in case of multiple matching manifest list entries

I stumpled upon this when analyzing this problem: https://github.com/docker/distribution/issues/2398
Because of that issue, I generated a manifest list with two workable entries (windows/amd64). It appears that Docker will just choose the first one. We should warn or error if there are multiple matching entries.
.\manifest-tool-windows-amd64.exe inspect friism/manifest-test:latest
Name:   friism/manifest-test:latest (Type: application/vnd.docker.distribution.manifest.list.v2+json)
Digest: sha256:574ac88b2c067533a891f06eec2ddcb94e3ed6898bdfa2941c71b251b2ecfca0
 * Contains 2 manifest references:
1    Mfst Type: application/vnd.docker.distribution.manifest.v2+json
1       Digest: sha256:fc38bbbc6101853405595ec453758e9818da9ec7a08d21c507f0a9f27e3378fc
1  Mfst Length: 528
1     Platform:
1           -      OS: windows
1           - OS Vers: 10.0.16278.1000
1           - OS Feat: []
1           -    Arch: amd64
1           - Variant:
1           - Feature:
1     # Layers: 1
         layer 1: digest = sha256:5dec2cd0b7ba3eacacf3ba0c407d5d60ea56aa3648ab9b171d9a3f6e6b2954ae

2    Mfst Type: application/vnd.docker.distribution.manifest.v2+json
2       Digest: sha256:f976abb02e11954e2b3718eb821301fe56dc0e98fdc542df17c936dbf7643a07
2  Mfst Length: 942
2     Platform:
2           -      OS: windows
2           - OS Vers: 10.0.14393.1358
2           - OS Feat: []
2           -    Arch: amd64
2           - Variant:
2           - Feature:
2     # Layers: 2
         layer 1: digest = sha256:bce2fbc256ea437a87dadac2f69aabd25bed4f56255549090056c1131fad0277
         layer 2: digest = sha256:6330793656b1565cd8b5d5c1e2a736a351dec39f7250daf31eb91e3a76cc872b

cc @estesp

Related discussion: https://github.com/opencontainers/image-spec/issues/588#issuecomment-290580795

Should this be filed under moby core? Ambiguity is perfectly fine from the distribution specification perspective, it is up to the client to decide how to resolve it. We should defer to the OCI specification for any further specification on how an implementor should handle it as noted in the linked discussion.

@dmcgowan shall we close this here?


Merge Feb 14, 2022 - Mar 7, 2022 from develop to main

Merge Feb 14, 2022 - Mar 7, 2022 from develop to main

Codecov Report

Merging #858 (2ac0e05) into main (9178d1b) will decrease coverage by 0.98%.
The diff coverage is 62.65%.

@@            Coverage Diff             @@
##             main     #858      +/-   ##
==========================================
- Coverage   63.29%   62.30%   -0.99%
==========================================
  Files         791      861      +70
  Lines       20010    22966    +2956
  Branches     1952     2344     +392
==========================================
+ Hits        12666    14310    +1644
- Misses       6980     8267    +1287
- Partials      364      389      +25

Impacted Files
Coverage Δ

...ns/jobs/monthly/clean_old_employer_applications.py
100.00% <ø> (ø)

backend/shared/shared/audit_log/tasks.py
100.00% <ø> (ø)

...applications/applicationList/useApplicationList.ts
35.63% <0.00%> (-1.58%)
:arrow_down:

...ations/forms/application/step2/utils/validation.ts
0.00% <0.00%> (ø)

...s/forms/application/step5/ApplicationFormStep5.tsx
0.00% <0.00%> (ø)

...forms/application/step5/useApplicationFormStep5.ts
0.00% <0.00%> (ø)

...s/forms/application/step6/ApplicationFormStep6.tsx
0.00% <0.00%> (ø)

...forms/application/step6/useApplicationFormStep6.ts
0.00% <0.00%> (ø)

...orms/application/stepperActions/StepperActions.tsx
0.00% <ø> (ø)

...omponents/applications/pageContent/PageContent.tsx
0.00% <0.00%> (ø)

... and 267 more

Continue to review full report at Codecov.

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

Merged with the 3 CodeQL high severity security vulnerabilities because they were all only in tests, see test_adfs_login.py file's tests

Made a pull request to fix these:

https://github.com/City-of-Helsinki/yjdh/pull/860


Fixing issue #71

https://github.com/Microsoft/HoloToolkit-Unity/issues/71
The proposed change makes GazeManager.FocusedObject into a public property and introduces several new classes to facilitate sending arbitrary events to dynamic objects via KeywordManager that do not require additional code:

Adds FocusedObjectMessageSender class whose SendMessageToFocusedObject method can be registered as a response in KeywordManager to enable arbitrary messages to be sent to currently focused object.
Adds SelectedObjectMessageSender class whose SendMessageToSelectedObject method can be registered as a response in KeywordManager to add voice commands to select objects and enable arbitrary messages to be sent to currently selected object. Saying "Select Object" selects currently focused object and selection persists even when gazing away. Saying "Clear Selection" resets currently  selected object.
Adds FocusedObjectMessageReceiver class which shows an example on how to handle messages sent by FocusedObjectMessageSender. In this particular example, focused object color is toggled on gaze enter/exit events.
Adds SelectedObjectMessageReceiver class which shows an example on how to handle messages sent by SelectedObjectMessageSender. In this particular example, selected object color is toggled when selecting the object and clearing currently selected object.
Adds FocusedObjectKeywords and SelectedObjectKeywords test scenes created to test above functionality. In the first scene, gazing on an object and saying "Make Smaller" and "Make Bigger" will adjust object size. In the second scene, gazing on an object and saying "Select Object" will persistently select that object for interaction with voice commands, after which the user can also adjust object size with "Make Smaller" and "Make Bigger" voice commands and finally clear currently selected object by saying "Clear Selection".

Changes have been discussed in the original PR:
https://github.com/Microsoft/HoloToolkit-Unity/pull/69
This PR also introduces a high-level concept of policies to control object selection:

Selecting object for interaction with voice commands by simply gazing at it.
Selecting object for interaction with voice commands by gazing at it and issuing an explicit "Select Object" voice command.
Other object selection policies are also possible, e.g. auto-aim or selection hysteresis.

Hi @dbouianov, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@NeerajW made some comments on #69 that pertain here.

Curious, what do you think of our general over reliance on the Unity SendMessage paradigm? Performance impacts?

@NeerajW Personally I don't like the message paradigm because it defines how you interact with something in the implementation instead of at an API level. You can't look at the API of a class and know how to work with it, you have to rely on documentation and I think that's sad.
Regarding performance I'm unsure on the underlying implementation, but I suspect it's slow and relies on reflection. It'd be interesting to run some tests on SendMessage vs a virtual method call on an interface.

@NeerajW @jwittner I've had to create strongly-typed message passing mechanisms for other Unity projects in the past. There is a lot of online debate on the topic of message passing in Unity, and it might be a challenge to introduce such a mechanism into HoloToolkit without making into an opinionated framework. String messages are easy, but not performant and prone to error, but callbacks need a lot of boilerplate when dealing with dynamic objects. Factory classes that register keyword handler callbacks for families of dynamic objects would probably introduce too much boilerplate, while arbitrary messages are nice because they make for very loose coupling between issued keywords and objects handling those keywords.

OK, addressed code review comments, added a lot more documentation and comments, removed some unnecessary files and reverted some files to earlier versions.

while arbitrary messages are nice because they make for very loose coupling between issued keywords and objects handling those keywords.

This was one of the main reasons why we went with this approach.

@NeerajW Are we squashing changes when we merge PRs? If not I think we should recommend a rebase on the history for this change.

OK, I can rebase again.

@dbouianov thank you for all the updates. Look forward to more of them. Merged.


Export main CLI function as API

I would like to incorporate this module in hapijs/lab so that we can easily test every hapijs module with a simple lab command line option. This will remove the need to manually add a dependency and configuration to each module file.

After further review, I don't think this would be enough. I would need a way to bypass the package.json config and provide all the config options directly. If that's of interest, I'm happy to keep working on this PR. But if this is not really a direction you want to support, feel free to close this. For now, I'm going to use some of the ideas in here and re-implement them.


ADD Backed by section

Please add "Backed by" section and include links to the twitter pages of the backers.

close. completed


Cold start dashcam mode full boot

Describe the bug
Woke up this morning started the jeep and got dashcam mode after comma logo loaded.  Not sure if its missing a finger print?
5c77528ab6bf019a|2023-03-06--07-21-29--0 - dashcam mode.
5c77528ab6bf019a|2023-03-06--07-30-46--0 - restarted car no dashcam mode.
Provide a route where the issue occurs
5c77528ab6bf019a|2023-03-06--07-21-29--0
openpilot version
0.9.0
Additional info
No response

missing this  1239: 8, 1256: 8, 1879: 8. will test and let know. Submitted pull request

https://github.com/commaai/openpilot/pull/27649


FQDN's for registry.k8s.io

Is there a list of FQDN's that could be whitelisted so that the image url's of registry.k8s.io from version 4.2.0 can be accessed through a firewall?

No.
https://github.com/kubernetes/registry.k8s.io/issues/127
https://github.com/kubernetes/registry.k8s.io#stability


 All my mods are Verified + Optional and no other tags. I play non-gameplay-changing environment and I want that people with the same mindset to actively join my games. However, they see Modded badge on my server description. I think a big fracture of players ignore my server now, since they do not want modded game or afraid of mods' side-effects, like affecting game save. That is a big problem for me.
I think Mod Loader should follow the same rules as the vanilla one. I've never seen blue Modded tag when using mod.io integration in-game. With Mod Loader and the same list of mods my server implicitly marked as Modded.
I think the proper criteria would be to filter out all Verified + Optional mods in list and if the result is not empty then add the badge, otherwise no badge.

If anything it should be a switch in the mod loader. I'm using a bunch of approved mods that do not affect other people's gameplay in any meaningful way (eg. chat popups for pressing buttons, Inferno heat bug fix, grenade collision fix, removing mineral scatter from TCF)  or clientside approved mods (better flashlight, more flares) and I do not think that it warrants the MODDED tag since the gameplay is very much still vanilla and as we all know, people tend to avoid modded lobbies due to the awful official integration that makes you subscribe to hosts' mods and leave them enabled upon leaving the lobby.

The likely path mint is going to take is likely to be a bit more conservative than suggested:

We (most likely) will try to implement auto-verification, in the same vein as how mod.io does auto-verification. This is a conservative approach, because while some mods may become verified under manual approval, mint or mod.io cannot automatically determine so.
But mint, unlike mod.io, has to deal with .paks sourced from the internet or locally from disk, which means that we want to treat these .paks in the same way as mod.io paks.

It might be a possibility for us to consider also fully respecting manual verified labels from mod.io, even though the same pak uploaded to mod.io when used locally might not be able to be automatically verified.
An edge case might be a approved mod from mod.io for some balance reason that is manually marked as approved but actually can be auto-verified. In this case, mint will likely determine the mod as auto-verified still.

Auto-verification is implemented in PR #124.
If a profile consists of only verified (either auto-verified or manually verified from mod.io) then this is a verified profile, so no [MODDED] tag will be added. Note that in this case, some mods when used locally as .paks may not be able to be auto-verified but could be manually verified on mod.io. We respect the mod.io manual verified labels.

I've recently found another case where it is a problem. Some people start joining my lobbies expecting some heavy gameplay modding. After explaining this is pure server (labeled by mistake) they were discouraged and sometimes leave immediately. So this issue is officially double-edged now.


Citations order, Alphabetical or Chronological

Feature or enhancement
When I am curating the history of a Taxon Name many times I have to move many citations from one Taxon Name ID to the new correct one that I have to create.
I would like to be able to see the list of citations in order. The most useful would be chronological but also, there could be the option to see it in alphabetical order. This way I can Pin all the citations I need and add them to the new ID.
Otherwise, I would love to be able to check on the ones I need to move in a checkbox and use a "Move to" function to select the ID of the correct taxon name. This way I would not need to create all the new citations and also delete all the wrong ones.
Location
Radial annotator by citations in Browse Nomenclature
Screenshot, napkin sketch of interface, or conceptual description

Your role
Curator

This is a case where I need to move 12 citations from a duplicate to the correct Taxon Name.


Will not open to server

Type: Bug
Trying to open server on html, will not open window
VS Code version: Code 1.71.2 (74b1f979648cc44d385a2286793c226e611f59e7, 2022-09-14T21:03:37.738Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6300U CPU @ 2.40GHz (4 x 2496)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
7.88GB (1.40GB free)

Process Argv
--crash-reporter-id 6efb30ce-f652-40e9-aae0-0ec76df4dcc3

Screen Reader
no

VM
0%

Extensions (50)

Extension
Author (truncated)
Version

npm-intellisense
chr
1.4.2

path-intellisense
chr
2.8.1

doxdocgen
csc
1.4.0

vscode-eslint
dba
2.2.6

vscode-npm-script
eg2
0.3.28

code-runner
for
0.11.8

vscode-pull-request-github
Git
0.50.0

rest-client
hum
0.25.1

search-node-modules
jas
1.3.0

better-cpp-syntax
jef
1.15.19

vscode-codemetrics
kis
1.24.0

kite
kit
0.147.0

rainbow-csv
mec
3.1.0

mongodb-vscode
mon
0.9.3

vscode-azurefunctions
ms-
1.8.1

vscode-azureresourcegroups
ms-
0.5.6

python
ms-
2022.14.0

vscode-pylance
ms-
2022.9.40

jupyter
ms-
2022.8.1002431955

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.9

remote-containers
ms-
0.251.0

remote-ssh
ms-
0.84.0

remote-ssh-edit
ms-
0.80.0

remote-wsl
ms-
0.66.3

azure-account
ms-
0.11.1

cmake-tools
ms-
1.12.27

cpptools
ms-
1.12.4

cpptools-extension-pack
ms-
1.3.0

vsliveshare
ms-
1.0.5723

vsliveshare-audio
ms-
0.1.91

debugger-for-chrome
msj
4.13.0

sqltools
mtx
0.25.1

vscode-for-node-js-development-pack
nod
2.0.1

java
red
1.10.0

LiveServer
rit
5.7.9

code-settings-sync
Sha
3.4.3

vscode-zipexplorer
sle
0.3.1

tabnine-vscode
Tab
3.6.15

cmake
twx
0.0.17

vscodeintellicode
Vis
1.2.27

vscode-java-debug
vsc
0.44.0

vscode-java-dependency
vsc
0.21.0

vscode-java-pack
vsc
0.25.2

vscode-java-test
vsc
0.37.1

vscode-maven
vsc
0.39.0

nodejs-extension-pack
wad
0.1.9

jinja
who
0.0.8

vscode-import-cost
wix
3.3.0

JavaScriptSnippets
xab
1.8.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30557514
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
hb751961:30553087
dsvsc012cf:30540253
azure-dev_surveyonecf:30548226
i497e931:30553904
vscccc:30566498
fc301958:30573243

Are you trying to use Live Server to open HTML? If so, you should file an issue in that repo or consider using the previewer that we created :)

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!

It’s working now. I had to clean my disco etc
On Thu, Sep 29, 2022 at 7:48 PM Andrea Mah @.***> wrote:

Are you trying to use Live Server to open HTML? If so, you should file an
issue in that repo https://github.com/ritwickdey/vscode-live-server or
consider using the previewer that we created http://aka.ms/live-preview
:)
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/161992#issuecomment-1262943581,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASR4BD2FTBYH7GPBN4NZARTWAYTGPANCNFSM6AAAAAAQW45WTY
.
You are receiving this because you authored the thread.Message ID:
@.***>


python38Packages.casbin: 1.13.0 -> 1.15.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.casbin is: "An authorization library that supports access control models like ACL, RBAC, ABAC in Python"
meta.homepage for python38Packages.casbin is: "https://github.com/casbin/pycasbin"
meta.changelog for python38Packages.casbin is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.15.0 with grep in /nix/store/rwqd47qgjg0rfqdzl6dkh0vlzxn9gkg3-python3.8-casbin-1.15.0

found 1.15.0 in filename of file in /nix/store/rwqd47qgjg0rfqdzl6dkh0vlzxn9gkg3-python3.8-casbin-1.15.0

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

6 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
3 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
python38Packages.casbin
python39Packages.casbin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rwqd47qgjg0rfqdzl6dkh0vlzxn9gkg3-python3.8-casbin-1.15.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.casbin https://github.com/r-ryantm/nixpkgs/archive/d3cf9c984c574bb821d2aff7e94b2ba18794d45b.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/rwqd47qgjg0rfqdzl6dkh0vlzxn9gkg3-python3.8-casbin-1.15.0
ls -la /nix/store/rwqd47qgjg0rfqdzl6dkh0vlzxn9gkg3-python3.8-casbin-1.15.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python38Packages.casbin
python39Packages.casbin

Maintainer pings
cc @costrouc for testing.

Duplicate of #150397


add environments endpoint

@dborsatto Thanks for the detailed review, I will take a look at the detailed comments in a bit.
For the general idea of using Kotlin in here: I would rather avoid having another huge dependency on the library, just because the code is more verbose then we'd like to have it. Also I am not foreseeing another overhaul like this happening soon™. If this keep being a problem, sure we need to address it, but for now I don't see me adding a lot of configurations soon.
To the requirement of having to set spaceid and environmentid: You don't have to. All methods are available as fetchAll() for using the configuration and fetchAll(spaceId, environmentId) if you want to state or overwrite the configuration explicitly. This is essentially sugar for the user of the library: If you don't want to write space id and environment id on all of the method calls, just use the configuration.


Code: 47. DB::Exception: Unknown column with materialized columns in joins

This bug mentioned before:
https://github.com/ClickHouse/ClickHouse/issues/41964
Now it occurs again in Metabase connecting to Clickhouse.

Proven to be fixed by Analyzer.
This issue is waiting when the Analyzer will be enabled by default.

The driver sets allow_experimental_analyzer=0 currently because of https://github.com/ClickHouse/ClickHouse/issues/64487 (which is now closed); on the previous CH versions, a lot of tests from the Metabase test suite were red because of that.
Re-enabling it could probably help in this case; we need to check if the Metabase test suite will pass with the analyzer enabled.

@AaronLhiu, could you try with https://github.com/ClickHouse/metabase-clickhouse-driver/releases/tag/1.50.3?
NB: keep in mind that https://github.com/ClickHouse/ClickHouse/issues/64487 was fixed only in 24.7.

Yes, it's working for the new driver.
Thanks that fix this issue so quickly.


Adopt new editorOverrideService

The use of overrideOpenEditor is being removed this iteration due to the confusing nature of it and the increasingly duplicated code. It is being replaced with the editorOverrideService which consumers should call registerContributionPoint on (this name will change). I have adopted this for custom editors and notebooks and as more code moves there it becomes harder to maintain both the editor service implementation and this one. So for that reason I need help moving the last use cases of overrideOpenEditor to the new service so that it may be removed.
The following cases are:

[ ] searchEditor.contribution.ts @JacksonKearl
[ ] webviewPanel.contribution.ts @mjbvz (or @jeanp413 if you're feeling up for it since you wrote the original one :p)
[ ] fileAction.ts @bpasero
[ ] preferencesContribution.ts @roblourens

Please let me know if you have any questions. Thanks for the help!

Yeah I think it can easily move, the editor service though cannot because it depends on some browser components.

@lramos15 work in progress for my area is in https://github.com/microsoft/vscode/compare/ben/121668 but I am facing an issue: it looks like we always call replaceEditors when an editor gets overridden and this results in the following experience:

Probably related: https://github.com/microsoft/vscode/issues/121708

@roblourens I have moved the editorOverrideService into common and made many of your suggestions let me know if you have any questions

Would still prefer to nuke the split json editor but I'll pick it up again

Can the EditorInputFactoryFunction return a promise of an IEditorInputWithOptions? The old model allowed for an async override function, which the search editor employed.

I'm going to defer to @bpasero for this one. The idea of not letting it return a promise is we really shouldn't be having IEditorInputs that are async in creation. There are a few issues we hit with CustomEditor and NotebookEditor input factories being async and we want to heavily avoid that pattern.

Yeah best to return sync and leave async stuff to the IEditorInput#resolve method if possible.

One thing I'm not sure about, currently if user settings is open in the split editor, and then I try to open workspace settings, I find that user settings editor and use group.replaceEditors to replace it with a split editor for workspace settings. In the new paradigm, I can only return an editor input for someone else to open. Is there some way to get the same behavior? Would it be safe to close the existing editor and assume that the new one will be opened right away? I think I would be ok with just not having this behavior too. Or I could delete the split json editor.

@roblourens Once you register your contribution if you were to call this.editorService.replaceEditors yourself with an override option of your settings editor ID it would do the same behavior by triggering your contribution. The override service just registers handlers, it doesn't actually trigger the opening, the opening pathways are through their respective editorService methods

I mean that we are trying to do "replaceEditors" when the override contribution is called

I think I'm still confused @roblourens. The contribution is opening agnostic, are you saying you want the settings editor to always replace? There is the option singlePerResource, but I'm not sure if that's what you want as that wouldn't close all settings.json and open your split json editor regardless of grouping. If you know when this flow is being triggered and could call close couldn't you call replaceEditors too? I might be misunderstanding though.

Compare action is adopted via https://github.com/microsoft/vscode/commit/c50ab1e89f665466cc3001a4574f82d2d54d6d49

If you know when this flow is being triggered and could call close couldn't you call replaceEditors too?

I don't think so, the contribution is supposed to return an editor input, and I assume you then call openEditor with it, and I get the sense that the contribution should act like a pure factory, not something that has side effects. Maybe I'm wrong and it's ok to call replaceEditors then return the EditorInput that I just opened.
To explain more, you have <userDataDir>/settings.json already open in the editor, then you click on .vscode/settings.json, and I need to override the second open and replace that first editor with the new one.

Just following up on our offline discussion here. It's a very odd case and something we don't want to support in the new service, so for now just don't do the replace and allow multiple instances.

@JacksonKearl I just came across this code and think it needs a revisit as well as part of this issue:
https://github.com/microsoft/vscode/blob/cb6e7b383bb7a2d76848162bd5897e1a04bd5b9a/src/vs/workbench/contrib/searchEditor/browser/searchEditorInput.ts#L220-L222
It looks like the search editor indicates it matches a file editor input if the resources are the same, but if we keep that code, a user would not be able to open a search in both text editor and search editor.

Might be the cause of this issue then https://github.com/microsoft/vscode/issues/120221

Yeah sounds like it.

Yeah I actually came across that line yesterday, I was unable to see the effects of removing it as it seems editors which have not adopted editorOverrideService can no longer participate in "Reopen With..."

Adopted in search editor, but I still see that when I go to "Reopen With...", the text editor is marked as default, even though my contribution is at ContributedEditorPriority.exclusive level (same holds for ContributedEditorPriority.default), and clicking on a *.code-search file in the explorer opens search editor by default.

@JacksonKearl I will investigate this. Thanks for adopting the service.

Update. I don't see this, but I would also recommend against exclusive as then the user can't set a setting to be anything else even if they want to.

Moved to default but I still see this. Another oddity: if I click the file in the explorer once, it opens in search editor. If I click it again, it reopens in text editor, as I click more and more it keeps alternating, or even opening new editors.

The webviewPanel.contribution.ts override was added to prevent a single webview created by an extension from being opened in two groups at once.
@lramos15 Is this something that we could specify on the editor input instead and let VS Code handle this? The current code doesn't fit the EditorOverrideService very well


Where's the documentation for the WriteCache value ?

Where's the documentation for the WriteCache value ?
Can we push the WriteCache value via GPO or it's not supported and must be applied manually to the master image like the CacheDirectory  value?
It's recommended to move these directories when using Citrix MCSIO or Citrix PVS. Please document this properly with more details.
https://github.com/EUCweb/BIS-F/issues/276
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e794b36e-21f9-5545-cd15-a528f34cbc46
Version Independent ID: ec93ddfb-158f-0305-f615-1776086d34c1
Content: FSLogix Cloud Cache configuration reference - FSLogix
Content Source: fslogix-docs/cloud-cache-configuration-reference.md
Product: fslogix
GitHub Login: @SteveD-MSFT
Microsoft Alias: stdowns

What setting are you referring to?  There is no write cache setting.


Zipper exercise tests with different arity of function toTree

The Zipper exercise tests requires multiple arities of the function toTree:
 31:    expect(zipper.toTree(t1)).toEqual(t1);
and
 43:    expect(zipper.left().right().toTree()).toEqual(t1);
In my opinion, that is not possible to achieve in Javascript, since there is no function overloading technique that I know of.

Are you sure this is a thing? I have a solution for this exercise that passes all of these tests.

@MarPenc I think that t1 is already applied in beforeEach so not required again in the tests. Thanks for spotting. 👍
Although it should not be a blocker. You can pass less number of arguments to functions in JS - it won't throw any error - the argument value will just be undefined. This might be how @LyleCharlesScott did it. Since the parameter is not required, I will merge your PR.


Add optional additional entities to authz request

Added an option to pass in entities to authorization requests that would not overwrite the configuration already stored in memory, but modify it by adding new entities to it.
Should be used if your use case has a large amount of entities that is infeasible to store in memory and you have easy access to all relevant entities when making the authorization request.
Fixes #22
This is my first Rust PR. Open and welcoming to feedback.
Would have added tests if the architecture was set up already. Sadly, for integration tests it is not and unit tests are meaningless here.

Extracted the new behaviour and the existing entities override to a function and added a test.
Attached is console output of a test run. Plenty of warnings unrelated to these changes, but all tests pass.
test_run_output.txt


Setting an image source to a file in the app's directory causes a base.h not found exception

Describe the bug
When the source of an Image control is set to a file in the app's directory, whether it's just "image.png" or "ms-appx:///image.png," the app crashes with a "base.h not found" exception.
Steps to reproduce the bug

Create a new WinUI in Desktop (C#) Project
Add an image to the app's directory
Add an Image control to MainWindow.xaml and set its source to "ImageName.png"
Run the app

Expected behavior
The image should load as normal and not cause an exception.
Screenshots

Version Info

NuGet package version:
3.0.0-preview3.201113.0

Windows app type:

UWP
Win32

No
Yes

Windows 10 version
Saw the problem?

Insider Build (20257)
Yes

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context

Is it possible that your resource path is wrong? I think by default the CS project name is appended to the resource name. If you want to see what exactly the name needs to be you can use the MakePri dump tool in the VS command line to dump the PRI file and see what the resource name is https://docs.microsoft.com/en-us/windows/uwp/app-resources/makepri-exe-command-options

I dumped the PRI from the app and there was nothing there related to the images in the directory. Is that what I should be looking for? I'm not sure what I should be doing with the dumped contents of the PRI file.

Strangely enough, after moving the image in and out of folders, adding and removing the slash at the start of the path, and experimenting with workarounds, the image started working. Even when I set the source to exactly what it was when the issue occurred, it worked fine. Sorry for the inconvinience.


ITestExecutor Cancel does not appear to be called

Description
The Cancel method of an ITestExecutor does not appear to be called
Steps to reproduce
Create an implementation of ITestExecuter
Create a long running test that test executer would execute
Run a the long running test
Hit the cancel button in the text explorer
Expected behavior
Cancel method is invoke
Actual behavior
Nothing
Diagnostic logs
Environment
Visual Studio 2022 Version 17.5.0

I run into the same issue public void Cancel() is not called when pressing the stop button on Test Explorer

This is by-design (or rather: too late to change it now).

I am not, this was done to keep the interactive experience in VS responsive. When abort is pressed everything tears down as fast as possible.

The problem is that if a test hangs for some unknown reason, it must be possible to cancel the test execution. Otherwise, this would mean I would have to close and reopen the IDE to solve the problem, which is very inconvenient.


initial and sloppy draft for rest api spec

This document needs review before I get to implementing the REST API! I want all of your opinions.

Okay, @davidmcclure, I updated the document according to your suggestions, as well as asking a few questions.

@lily-mayfield @davidmcclure reviewing by this side. Tomorrow we will be sending any question or suggestion.


feat: Added show_diff method for <:AbstractVector types

coverage: 73.04% (-15.2%) from 88.222%
when pulling 57e072b838a54973fc676bcdda05d521352d8dc8 on show_nice_vector_diffs
into e3684584b1e03b35519ab82518f04e326859ae2d on main.

Codecov Report
Attention: 23 lines in your changes are missing coverage. Please review.

Comparison is base (e368458) 87.69% compared to head (57e072b) 86.78%.

Files
Patch %
Lines

src/show_diff/common.jl
67.60%
23 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   87.69%   86.78%   -0.92%
==========================================
  Files          15       15
  Lines        1341     1407      +66
==========================================
+ Hits         1176     1221      +45
- Misses        165      186      +21

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


OPENBOT custom script does not work

Dear team,
under the new version of botium my old custom openbot script doesn't work. I suspect that is related to botium-connector-webdriverio 0.2.0 update.
I tried to follow the examples such as snippets/openbot.js but with no good results .
Basically I set 2 text field and then I click on login, js file:
module.exports = (container, browser) => {
return browser
.waitForVisible('#username', 20000).then().setValue('#username', 'myuser')
.waitForVisible('#password', 5000).then().setValue('#password', 'mypsdw')
.waitForVisible('#kc-login', 5000).then().click('#kc-login')
.pause(1000)
}
WebDriver error on startup: TypeError: browser.waitForVisible is not a function -> I tried with waitForDisplayed

You are right, there is a breaking change as we switched to webdriverio version 5, we documented this in the repository: https://github.com/codeforequity-at/botium-connector-webdriverio
You can use the old connector version with webdriver 4 furthermore, but we won´t work on it in future. webdriver 5 has lots of advantages (for example support for shadow dom).

Sorry for my basic understanding of js and node.
I tried in these ways to be compatible with the new version but nope:
1)
module.exports = (container, browser) => {
return browser
.then(() => browser.$('#username').then(uname => uname.waitForDisplayed(20000).setValue('myuser')))
.then(() => browser.$('#password').then(psw => psw.waitForDisplayed(5000).setValue('mypsw')))
.then(() => browser.$('#kc-login').then(ccBtn => ccBtn.waitForDisplayed(5000).then(() => ccBtn.click()))
.then(() => browser.pause(2000))
}

module.exports = async (container, browser) => {
const ccUser = await browser.$('#username')
await ccUser.waitForDisplayed(20000)
await ccUser.setValue('#username', 'myuser')
const ccPsw = await browser.$('#password')
await ccPsw.waitForDisplayed(5000)
await ccPsw.setValue('#password', 'mypsw')
const ccBtn = await browser.$('.kc-login')
await ccBtn.waitForDisplayed(5000)
await ccBtn.click()
console.log('cc-btn clicked')
await browser.pause(1000)
}

?

have you been able to make this work ?

sorry for the late, the procedure is quite long and I was moved in the meantime on another project. So if I'll be reassigned and able to fix I'll update you. Thanks


More tests for docker integration

Fixes #237

Move docker probe code into it's own module
Put PIDTree behind an interface for mocking
Disaggregate dockerTagger into a registry, tagger and reporter
Similarly disaggregate tests
Add mocks for docker container and registry
Add test for docker events.

Note the inclusion of work from #254; this will go away when that is merged.

LGTM, then


Check activity of all projects.

We must remember that we do not leave deprecated projects on the list.
The maximum term of the last activity(issue updates, wiki edits, commits) is 12 months.

Is last commit really the only variable that the projects should be measured by?
Maybe more activities could be included such as last issue update  or wiki edits?

@TrueFurby Of course, not. We'll removed from list only deprecated projects (if there are any.). Which are does not supported by maintainer. Only project without any activity.

@TrueFurby I just edited text of issue. Thanks for asking. So now we can avoid misunderstandings.

I created an issue for Image of javascripts/socket.io is outdated but found it's better to post the issue here.
socket.io has been upgraded to 2.0 few months ago but the package's image has v1.0 on it. We should remove the version part of that image and also add a rule to apply to all packages' image.


add an annotation to disable auto import

The related issue: https://issues.redhat.com/browse/ACM-9212

/lgtm
/approve


Add Plex-Media-Server as source

Feature
I'would love if i could add my plex library as a source of phoniebox.
User perspective
All our locally stored MP3s are held in a Plex-Library and we would like to use them directy instead of copying locally to phoniebox.
Further information
No response

The Plex Media Server offers a DLNA server via which media can be streamed. However, the Phoniebox does not support DLNA streaming by default. You would need to use additional software to add DLNA support. f.e. gmrender-resurrect