Using bounds not filtering suggests

Short description
I am using bounds like:
import Geosuggest from 'react-geosuggest';

//...

<Geosuggest
        ref={geoSuggestEl}
        placeholder="placeholder"
        onSuggestSelect={onSuggestSelect}
        autoComplete="off"
        autoActivateFirstSuggest={true}
        country="de"
        ignoreEnter={false}
        bounds={bounds}
 />

where bounds defined as:
const southwest = new google.maps.LatLng(47.270111, 9.530952);
const northeast = new google.maps.LatLng(50.558835, 13.835972);
const bounds = new google.maps.LatLngBounds(southwest, northeast);

Expected results
I expected that by providing the boundary (bavaria), I am just getting suggestions within these
boundary.
Actual results
Instead of just  getting suggestions within these boundary, I also get e.g. "New York" in my suggestions, shouldn't this be filtered out?
Reproduction steps

Use provided code snippets from above
Type in "New"

Hi @bayerlse!
I had a look at your issue, but couldn't reproduce it. With all the information you gave here, I get the following suggestions:

There's no "New York" in the list.
Can you provide a more detailed example or link to a code playground to see this issue in action?


adding URL property to Hook struct

This PR adds the URL property to the Hook struct to map the existing url property returned by the Github API for listing hooks or retrieving them by id.
Addresses #251

You'll want to add a test for this, I believe. Otherwise, :+1: .

Wasn't sure what the precedence was for testing individual props.  It doesn't look like there are tests around any of them specifically, so I added checks for URL into the existing hook list/get tests.  Lemme know if there's an alternate approach you'd prefer.

As Brad notes, we don't have great test coverage for every struct field.  In a number of places, we have a single test that does that, but for the rest of the tests we use minimally populated structs.  The integration tests also provide coverage against the live API for a handful of types.
Anyway, cherry-picked the initial commit as 59f41ca3f75660d005283021d0c00bd0dd0da756. Thanks.


[backport] Fix typo

Backport of #1307

LGTM

Jenkins CI test (for commit 6beb046c384ac912c228dc80f93c45d635864e26) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)


api request

[READ] Guidelines
When filing a feature request please make sure the issue title starts with "FR:".
A good feature request ideally

is either immediately obvious (i.e. "Add Sign in with Apple support"), or
starts with a use case that is not achievable with the existing Firebase API and
includes an API proposal that would make the use case possible. The proposed API
change does not need to be very specific.

Once you've read this section, please delete it and fill out the rest of the template.
Feature proposal

Firebase Component: _____ (Auth, Core, Database, Firestore, Messaging, Storage, etc)

Describe your use case and/or feature request here.

Hey, please read the instructions and fill out the feature request template, then comment here and I'll re-open the issue.


rect_scale of node can't be set to (0,0) anymore.

Godot version:
3.2 stable
OS/device including version:
Win 10
The issue that is breaking my code is:  If you assign a Container node to a rect_scale of (0,0), it now is setting the rect_scale to (0.00001, 0.00001).
This is breaking a large section of conditional checking of code I already have in place
Reproduce:
Create a new project with a Node2D.
Add a Container Node as a child.
Create a script for the Node2D an place in the "_ready()" function the code:
var wewe = Vector2(0, 0)
var tete = str(wewe)
var Test = $Container.rect_scale
var uu = str(Test)
wewe = 100
Step though the code and you should find that tete equals "(0,0)" like expected.
However uu == "(0.00001, 0.00001)"
I don't know when this was introduced, but I just updated to 3.2 stable, and this bug is present.

If the only concern is with checking if the scale is zero, you could always just adjust that code.
rect_scale.x < 0.0001 && rect_scale.y < 0.0001

Or maybe:
Vector2.ZERO.is_equal_approx(rect_scale / 10) # Must divide to land below the tolerance.

This isn't my only concern.
The inspector still says (0,0) even though it is 0.00001,
I set the property to (0,0) and it should become (0,0).  The tween code asks for the pro to be set to (0,0).
From 3.1 and before, we have been able to set this to a solid (0,0).  As well, the docs do not mention that you cannot set the property to (0,0).  Hopefully what ever changed can to switched back

Duplicate of #35081?

Duplicate of #35081.


A LagCompensator Revamp

Current Lagcompensator:
Supports box colliders,
Supports position & scale compensation.
Supports primitive movement systems.
Uses performance intensive methods. (addcomponent, destroy, instantiate, raycast)
Easy to script.
Very easy to implement.
Revamped Lagcompensator:
Supports any 3d collider type, including ragdoll rigs.
Supports position, rotation, and animation compensation.
Supports complex movement systems.
Optimized.
Easy to script.
May need refactoring to implement.
Animation Tracking & Compensation has been added. Note that it doesn't track or compensate for Blend Tree variables, that would be very hard to automate. you would have to manually set these in the compensated animator via script. OR edit the Lagcompensator to implement & interpolate these in Capture3d.
Usage is pretty simple, create dummy player with only trigger colliders. parent said dummy and the actual player under a main player GameObject. (may need refactoring). There's probably be a way to do this without parenting, not sure. Make sure to disable this by default, as it will be enabled when necessary by the server.
then just fill in the "tracked" fields with your actual player, and "compensated" fields with dummy player. the separated position / orientation variables are there for more complex player movement systems.
Animator compensation is optional.
Now you have 2 options for implementation, The first and easier version, await UpdateColliders(). you just put this before you Raycast in your servers Fire function. This also awaits a task inside of the function, not sure if its redundant.
Second and more performant option, UpdateTargetCollider() and DisableColliders() work together. DisableColliders() goes at the end of your servers fire function. Then UpdateTargetColldier gets called if the client hits a player with their local raycast. NOT the server. i think the best way to add this is to add a NetworkConnectionToClient target to your servers fire function, then calling await UpdateTargetColldier(target) before your servers raycast, if the target is not null.

This could work in a similar way as mischa's version, only updating if the client hit the collider on their screen. it would decrease firing overhead, as your not always hitting a player, and thus dont need to update the colliders every time you shoot. however it would require more scripting from the user, as the user would need to add a different command to the clients fire function to update the collider

I suggest using [ServerCallback] for Update()

I suggest using [ServerCallback] for Update()

Wouldn't [Server] be better? Only difference on the docs is that it doesn't throw an error if a client calls the method, but all of the methods shouldn't be called on the clients anyways? I've only ever used [Server] myself, so I'm not sure.

I suggest using [ServerCallback] for Update()

Wouldn't [Server] be better? Only difference on the docs is that it doesn't throw an error if a client calls the method, but all of the methods shouldn't be called on the clients anyways? I've only ever used [Server] myself, so I'm not sure.

Have to use [ServerCallback] for Unity methods because Unity will call them in all contexts.

Have to use [ServerCallback] for Unity methods because Unity will call them in all contexts and we don't need the warning flood.

Ah that makes sense. For some reason I thought the original guy meant swapping [Server] to [ServerCallback] on the Update method I added, and not unity's one.


Complex1 scene animation not exporting correctly

When playing back the skinning animated clips, both ThreeJS and BabylonJS twist the cube incorrectly.
Might be because of the shifted pivot point (incorrect inverse-bind-pose)?

Fixed in v0.9.4


Need to add playback speed control

Hi,  I tried to add speed control by adding the following functions in your YouTubePlayer.
"func getPlaybackRate()" "
"func getAvailablePlaybackRates(completion:   )"
"func setPlaybackRate(suggestedRate:Double)"
The functions "getPlaybackRate" and "getAvailablePlaybackRates" worked fine but "setPlaybackRate" caused the error.
Is there any way to control playback speed?
The error observed by "setPlaybackRate" is as follows.
Error Domain=WKErrorDomain Code=4 "A JavaScript exception occurred" UserInfo={WKJavaScriptExceptionLineNumber=1, WKJavaScriptExceptionMessage=SyntaxError: Unexpected token ':', WKJavaScriptExceptionColumnNumber=0, WKJavaScriptExceptionSourceURL=about:blank, NSLocalizedDescription=A JavaScript exception occurred}
["[YouTubePlayer]", "Error executing javascript"]
Thank you,

Hi @ymatt345 Where you able to achieve this?


Make Circom coprocessor use arguments

The PoC implementation of the Circom coprocessor doesn't take arguments, but we'd want it to.

See
https://github.com/lurk-lab/lurk-rs/blob/3292852ce90722ebe263a0de6d0cc7cb6589de37/examples/circom.rs#L76-L81
https://github.com/lurk-lab/lurk-rs/blob/3292852ce90722ebe263a0de6d0cc7cb6589de37/examples/circom.rs#L83-L92
https://github.com/lurk-lab/lurk-rs/blob/3292852ce90722ebe263a0de6d0cc7cb6589de37/examples/circom.rs#L94-L103

@arthurpaulino is this still active post #799, #756 et al?

I was about to tackle this but it seems that I got confused in lexical terms. When referring to arguments do we mean inputs or parameters?
As the main entrypoint does not allow for parameters I'm guessing we refer to inputs, but I just want to make sure.

@tchataigner we mean the arguments we pass in a Lurk program like foo in (sha256 foo)

Alright so arguments is a lurk term. And, deducing that those arguments are then what we use as inputs in Circom.

@tchataigner yes

Hey everyone! Quick follow-up on this. After some consideration, it seems that we can not really create a 1 to 1 relationship between the awaited circom signals and our lurk arguments. So the information about the arity of a CircomGadget and how to handle them will have to be declarative on the developer side.
#997 will show off how that works for now. I have an update to push by EOD.


powertoys run scrolling with finger

Description of the new feature / enhancement
It would be better if could scroll through the results with fingers after searching for powertoys run.
Now when I try to scroll with a touch, the file just opens.
Scenario when this would be used?
It's uncomfortable because I have to keep changing to mouse while working with touch
Supporting information
No response

/dup #7528


#1036 - Fix duplicates issue when asserting containsExactlyInAnyOrder

Fixes #1036

Thank you very much for this fix!

Thanks for letting me support a great tool!


[Bug]: AM not able to run Prowler after all the installations it is showing prowler is a directory

Steps to Reproduce
./prowler
bash: ./prowler: Is a directory - getting this response
i used this link to clone https://github.com/prowler-cloud/prowler.git
Expected behavior
The prowler tool is used to generate some reports
Actual Result with Screenshots or Logs
it will show the checks and reports
How did you install Prowler?
Cloning the repository from github.com (git clone)
Environment Resource
EC2 Instance (linux)
OS used
Amazon Linux 2
Prowler version
cloned from repository
Pip version
pip 20.2.2 from /usr/lib/python3.7/site-packages/pip (python 3.7)
Context
No response

Hi @KD17k, you will have to use Python to execute Prowler python prowler.py. These are the steps when executing Prowler from Github (https://github.com/prowler-cloud/prowler#from-github):
git clone https://github.com/prowler-cloud/prowler
cd prowler
poetry shell
poetry install
python prowler.py -v

Also @KD17k if yo install Prowler with pip you just can run prowler.
./prowler was the way for Prowler v2 and before.

Also @KD17k if yo install Prowler with pip you just can run prowler.
./prowler was the way for Prowler v2 and before.

]$ pip3 install prowler
Defaulting to user installation because normal site-packages is not writeable
ERROR: Could not find a version that satisfies the requirement prowler (from versions: none)
ERROR: No matching distribution found for prowler

poetry shell
poetry instal

[ec2-user@ip-33-33-33 prowtest]$ cd prowler
[ec2-user@ip-33-33-33 prowler]$ poetry shell
bash: poetry: command not found
[ec2-user@ip-33-33-33 prowler]$

Hi @KD17k, you will have to use Python to execute Prowler python prowler.py. These are the steps when executing Prowler from Github (https://github.com/prowler-cloud/prowler#from-github):
git clone https://github.com/prowler-cloud/prowler
cd prowler
poetry shell
poetry install
python prowler.py -v

[ec2-user@ip-33-33-33  prowler]$ python3 prowler.py -v
Traceback (most recent call last):
File "prowler.py", line 5, in
from prowler.main import prowler
File "/prowtest/prowler/prowler/main.py", line 6, in
from prowler.lib.banner import print_banner
File "/prowtest/prowler/prowler/lib/banner.py", line 1, in
from colorama import Fore, Style
ModuleNotFoundError: No module named 'colorama'
[ec2-user@ip-33-33-33  prowler]$

After installing all the modules one by one it will tell no module named Azure (am running this in aws) i don't need that

Previously i created a automation that will do the process now suddenly it is not working and also it is also showing Prowler is a directory

Try to install poetry first with pip install poetry.

Also @KD17k if yo install Prowler with pip you just can run prowler.
./prowler was the way for Prowler v2 and before.

]$ pip3 install prowler Defaulting to user installation because normal site-packages is not writeable ERROR: Could not find a version that satisfies the requirement prowler (from versions: none) ERROR: No matching distribution found for prowler

That seems a problem with Internet connection.

[ec2-user@ip33-33-33 prowler]$ python3 prowler.py -v
Traceback (most recent call last):
File "prowler.py", line 5, in
from prowler.main import prowler
File "/prowtest/prowler/prowler/main.py", line 6, in
from prowler.lib.banner import print_banner
File "/prowtest/prowler/prowler/lib/banner.py", line 1, in
from colorama import Fore, Style
ModuleNotFoundError: No module named 'colorama'
[ec2-user@ip33-33-33 prowler]$ poetry shell
bash: poetry: command not found
[ec2-user@ip-33-33-33prowler]$
i installed poetry and running this command getting the same error am running this from linux 2 Ec2 instance .
and about internet connection error are you mentioning about security rules here ?

Here you have how to install Prowler in Amazon Linux 2 instance: https://docs.prowler.cloud/en/latest/#installation
Please, make sure you have your instance in a public VPC with outbound connection to Internet.

[ec2-user@ip-33-33-33 /]$ ping www.google.com
PING www.google.com (<IP_ADDRESS>3) 56(84) bytes of data.
64 bytes from bl-in-f105.1e100.net (<IP_ADDRESS>3): icmp_seq=1 ttl=52 time=3.45 ms
64 bytes from bl-in-f105.1e100.net (<IP_ADDRESS>3): icmp_seq=2 ttl=52 time=2.50 ms
64 bytes from bl-in-f105.1e100.net (<IP_ADDRESS>3): icmp_seq=3 ttl=52 time=2.41 ms
64 bytes from bl-in-f105.1e100.net (<IP_ADDRESS>3): icmp_seq=4 ttl=52 time=2.37 ms
[ec2-user@ip-33-33-33 prowler]$ aws --version
aws-cli/2.11.14 Python/3.11.3 Linux/5.10.176-157.645.amzn2.x86_64 exe/x86_64.amzn.2 prompt/off
[ec2-user@ip-33-33-33 prowler]$ jq --version
jq-1.5
[ec2-user@ip-33-33-33 prowler]$ git --version
git version 2.39.2
[ec2-user@ip-33-33-33 prowler]$ pip3 --version
pip 20.2.2 from /usr/lib/python3.7/site-packages/pip (python 3.7)
[ec2-user@ip-33-33-33 prowler]$ python3 --version
Python 3.7.16
[ec2-user@ip-33-33-33 prowler]$pip3 install detect-secrets==1.0.3 - installed this as well
[ec2-user@ip-33-33-33 prowler]$ pip3 install poetry - installed this as well
[ec2-user@ip-33-33-33  prowler]$ python3 prowler.py -v
Traceback (most recent call last):
File "prowler.py", line 5, in
from prowler.main import prowler
File "/prowtest/prowler/prowler/main.py", line 6, in
from prowler.lib.banner import print_banner
File "/prowtest/prowler/prowler/lib/banner.py", line 1, in
from colorama import Fore, Style
ModuleNotFoundError: No module named 'colorama'
[ec2-user@ip-10-71-1-61 prowler]$

I installed all the things , am able to ping even i tried to install the the modules finally it is asking me to install Azure modules don't know why

Hi @KD17k, you still have to do:
poetry shell
poetry install

am not able to run this commands inside prowler - altough i installed this [ec2-user@ip-33-33-33 prowler]$ pip3 install poetry - installed this as well
[ec2-user@ip-10-71-1-61 prowler]$ poetry shell
bash: poetry: command not found
[ec2-user@ip-33-33-33  prowler]$ pip3 poetry shell
ERROR: unknown command "poetry"

Thankyou man for helping my issue is resloved


Some files do not show git modification hint

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  1.82.2
OS Version:  macOS 13.4.1

Steps to Reproduce:

Modify the code NOT included in .gitignore
I can find the change in git panel but it doesn't show code area
NOT all files like this, and I can't fix it through restart vs code

Here is the screenshot:

@Zhoucheng133, could you try to reproduce with the latest Insiders release? Thanks!

I find it works on preview version. Thank you and waiting for the stable release!


testing codeku

testing

Hi,
Thank you for your fantastic contribution to the Codeku Cloud and DevOps micro tasks! We truly appreciate your effort. However, we regret to inform you that this particular task cannot be approved due to a few errors. But don’t worry—this is just the beginning! There will be plenty of opportunities in the future, and we’re confident that you’ll succeed next time.
We encourage you to keep trying, and we wish you the very best of luck!
Warm regards,
Team CoDeKu


Comma separator for thousand cannot shown on bar chart data.

Version
4.2.1
Steps to reproduce
{
name: '2019',
type: 'bar',
barGap: 0,
label: labelOption,
data: [200, 2,911]
}
What is expected?
data is in currency, need comma separator for thousand.
What is actually happening?
Data cannot shown, it blank for data = 2,911

2,911 is treated as two numbers in JavaScript, and ECharts can do nothing with it. Please use 2911 instead. You may use formatter to change the display of value, but when in input, it should be a number in the form of 2911.

Noted with thanks.

I'm having the same problem, and I believe my data is in the correct format...
bar chart options:
{
  "title": {
    "text": "Payments By Year-Month"
  },
  "tooltip": {
    "trigger": "axis"
  },
  "grid": {
    "left": "3%",
    "right": "4%",
    "containLabel": true
  },
  "toolbox": {
    "feature": {
      "saveAsImage": true
    }
  },
  "xAxis": {
    "type": "category",
    "axisTick": {
      "alignWithLabel": true
    },
    "data": [
      "2019-07",
      "2019-08",
      "2019-09",
      "2019-10",
      "2019-11",
      "2019-12",
      "2020-01"
    ]
  },
  "yAxis": {
    "type": "value",
    "axisLabel": {
      "formatter": "${value}"
    }
  },
  "series": [
    {
      "type": "bar",
      "data": [
        41700,
        39106.15,
        8322.9,
        52273.65,
        11095.4,
        5049.85,
        37690.22
      ],
      "label": {
        "normal": {
          "show": true,
          "formatter": "${c}"
        }
      }
    }
  ]
}

Resulting image:
![image](https://user-images.githubusercontent.com/9937583/75121232-55cb1880-564f-11ea-9561-56b142fdea50.png)

the formatter for y-axis works fine, and formatter for bar label does add "$", but no commas in bar label number.

use this:
{
   ...
   series: [
      {
         label: {
           formatter: function(d) { return d.data.value.toLocaleString() }
         }
     }
}

also can use:
           formatter: function(d) { return d.data.value.toString().replace(/\B(?=(\d{3})+(?!\d))/g, ',') }

i am searching how to remove separators, i don't figure how to use formatter parameter to make this

i try this adviced by @ahadihadi but didn't change anything to my chart :s i have no clue how to do this

i am searching how to remove separators, i don't figure how to use formatter parameter to make this
i try this adviced by @ahadihadi but didn't change anything to my chart :s i have no clue how to do this

did you find anything?


route component not loading

This is the error I get when trying to host locally.
client/imports/app/app.routes.ts (4, 31): File 'client/imports/home/home.component.ts' is not a module
This is the home.component.ts
`import { Component } from '@angular/core';
import template from './home.component.html';
@Component({
selector: 'home',
template
})
export class HomeComponent {}`

Maybe you should provide the route.ts code.

import { Route } from '@angular/router';
import { ContactComponent } from '../contact/contact.component';
import { HomeComponent } from '../home/home.component';
export const routes: Route[] = [
{ path: '', component: HomeComponent },
{ path: 'contact', component: ContactComponent }
];

I suppose your path to the home component is wrong?
import { HomeComponent } from '../home/home.component';
You should provide a more thorough explanation including all the relevant code pieces / folder structure if you expect people to be able to help you.

@LelouchLight If you are using WebStorm or any other fine IDE, try to remove the import that causes the issue, and try to re-import it using your IDE auto-complete.
If you are still having issues, please provide a repository with you code.


multiple panels

adding panels don't shrink

to fit

It looks like both the react code and vcs-js code are attempting to resize the canvas. I have opened an issue for vcs-js to determine if there is a workaround for this. If not, we will need to determine which code base should be responsible for managing the canvas size.

Fixed in #177


matches/matches.go:XXX:YY: Error return value of f.SetCellValue is not checked

From a CI run for #74:
##[error]matches/matches.go:595:16: Error return value of `f.SetCellValue` is not checked (errcheck)
	f.SetCellValue(summarySheet, "A1", "Evaluated Files")
	              ^
##[error]matches/matches.go:596:16: Error return value of `f.SetCellValue` is not checked (errcheck)
	f.SetCellValue(summarySheet, "A2", "Sets of files with identical size")
	              ^
##[error]matches/matches.go:597:16: Error return value of `f.SetCellValue` is not checked (errcheck)
	f.SetCellValue(summarySheet, "A3", "Sets of files with identical fingerprint")

Existing problem not related to #74, exposed (evidently) due to the work on #70 or #69.

Note: This is due to an API change in v2. v1 didn't return an error for that method call.

Quick thought:
Create a slice of structs to hold the values used when calling f.SetCellValue and then pass that into a wrapper function that handles processing the struct and returning any errors. I really can't see adding 24 if err additions to the current setup.


Deprecate set_constituent_data in favor of upsert_constituent_data

Hey there. I just made these modifications on the vendored version of this plugin we use on our project, perhaps they will save other users some time. Per the notification email sent out today:

Effective December 14, 2018, one API method -- /set_constituent_data -- will be deprecated and replaced by a new method providing a more stable experience for both API and conventional users. Documentation for this new method is now available at the link below.
Documentation: /upsert_constituent_data
The new /upsert_constituent_data returns deferred results in all cases, unlike the deprecated /set_constituent_data method.
Prior to December 14, 2018, your organization will need to modify all existing integrations between your applications and the BSD Tools that rely on the deprecated /set_constituent_data method, to instead use the new /upsert_constituent_data method.

I'm so glad they are making this change actually.

@mattlima where are you getting these notifications from? I did not know they were making this change, we'd like to subscribe too.

Was sent to a client that passed it on to me. I forwarded to your email.


fix 1802: leverage KEP-1755 to retrieve local image registry host an…

…d port
Fixes #1802, this makes:
$kamel install

work on supporting clusters without additional arguments (tested successfully on k3d), thanks !

Release Note
NONE

As @nicolaferraro said I'm not sure how public the "kube-public" namespace really is:
$kubectl auth can-i get cm  -n kube-public --as system:serviceaccount:camel-k:camel-k-operator
no

That means the operator service account do not have the required permission, but I would not deduce kube-public is not "public" from it. That only means read permission is granted for all users, but not service account. This is for us to add the permission to the operator service account.
One solution is to create a dedicated ClusterRole with:
rules:
- apiGroups: [""]
  resources: ["configmaps"]
  resourceNames: ["local-registry-hosting"]
  verbs: ["get"]

And a RoleBinding projecting that ClusterRole on the kube-public namespace for the operator service account.

Hey @johnpoth unfortunately this has some conflict ongoing now. Would you mind rebase with main?

Thanks @squakez ! Rebased and merging....


Someone interfered (not working properly)

Here, we collect interference cases.

https://addons.mozilla.org/ja/firefox/addon/open-tab-next-right-to-current/reviews/2051890/
Firefox
Addon: Tree Style Tab
Remarks: It is unknown if this can be avoided by setting options.

Chrome でも報告あり（詳細な環境は不明）

francis z 2024/08/03
Used to work but now it no longer works.


Should we only support vagrant + libvirt?

Recent virtualbox on macos breaks with the sandbox setup we currently have (#112 ).
Also, apparently its not guaranteed that virtualbox will ever be ported to arm64 macs.
It looks like libvirt works on macos1 and there are patches floating around for libvirt + macos + arm2 which I hope would land in upstream (if it hasn't). So is virtualbox on the chopping block?

Can we still support bare metal/Docker Compose?

Oops this is very poorly worded, I'll update both the title and body. My intent was to drop virtualbox and only support vagrant + libvirt as the sole virtualization option (leaving baremetal alone).

I am in favor of not dropping vagrant+virtualbox. I use it all the time. one option is to make the VirtualBox dependency version explicit. We can support the current working version(s) and then when newer versions stabilize we update.

@jacobweinstock Have you updated to latest mac os? Or do you use it on linux too?

Yes and no. I have 2 Apple machines. One is using Monterey and one Big Sur. I don't use the Linux sandbox as often. When I do though, I use the Docker Compose or Vagrant and Libvirt.

There are trade-offs to using libvirt on Mac OS over VirtualBox. A significant one, for me, is that I believe libvirt on Mac OS requires homebrew (at least according to the link posted in the initial issue). I don't run homebrew. That would be a required dependency. So we are trading dependencies from Virtualbox to homebrew+libvirt.

Thanks for the extra context. I'm not seeing where libvirt requires homebrew. What package manager do you use on mac?

Thanks for the extra context. I'm not seeing where libvirt requires homebrew.

This link shows the libvirt installation requiring homebrew. I'm unable to find any other articles for how to install libvirt that don't use homebrew or macports.

What package manager do you use on mac?

I don't use a package manager.

Thanks for the extra context. I'm not seeing where libvirt requires homebrew.

This link shows the libvirt installation requiring homebrew. I'm unable to find any other articles for how to install libvirt that don't use homebrew or macports.

There's https://libvirt.org/compiling.html but I haven't tried it on mac so can't say anything about getting the dependencies. The recipes for libvirt for both macports and homebrew look pretty sane so it seems like the usual "ensure you have deps, mason configure, meson build, meson install" I guess.

What package manager do you use on mac?

I don't use a package manager.

I guess this issue/discussion has run its course though. No one seems to want libvirt on mac and you want virtualbox so there's no reason to drop virtualbox in favor of libvirt.


new Tree type  is a breaking api change

Hi there,
I believe the change of the Tree type in https://github.com/purescript-spec/purescript-spec/pull/139 is a breaking change and would probably benefit from a clearer version bump or a note in the github release ?
Cheers,
Jun

To clarify, I've been using a fork of https://github.com/purescript-spec/purescript-spec-mocha and v7.5.1 breaks here.

Hi @jmatsushita, you're absolutely right, it totally slipped my mind that the Tree type is part of the public API.
I added a mention of breaking change to the README and the 7.5.1 release description.
I also patched up purescript-spec-mocha, it is now up to date.
I do disagree about the version however. Yes, the change is breaking, but it is minor, easily rectified, and in a place that is not normally touched by regular consumers. I feel like a minor version bump is enough.

Sounds great thank you @fsoikin !


"Zoom to data" on raster layers produces "layer.getSource(...).getExtent is not a function" error

"Zoom to data" works for the vector layers I've tried, but any time I try to zoom to data on a raster layer, I get the following error in the console log:
TypeError: layer.getSource(...).getExtent is not a function
    at zoomToLayerExtent (MapLoom-1.2.js:166347)
    at zoomToLayerFeatures (MapLoom-1.2.js:166325)
    at Scope.scope.zoomToData (MapLoom-1.2.js:165822)
    at MapLoom-1.2.js:10829
    at MapLoom-1.2.js:19070
    at Scope.$eval (MapLoom-1.2.js:12666)
    at Scope.$apply (MapLoom-1.2.js:12764)
    at HTMLAnchorElement.<anonymous> (MapLoom-1.2.js:19069)
    at HTMLAnchorElement.x.event.dispatch (MapLoom-1.2.js:12)
    at HTMLAnchorElement.y.handle (MapLoom-1.2.js:12)(anonymous function) @ MapLoom-1.2.js:10057

The snippet of code that produces this error appears twice in MapService.js:
layer.getSource().getExtent()

The Source object returned from getSource() does not have a getExtent() function. I'm experimenting with calling layer.getExtent() instead but getting nothing but "undefined" so far on my raster layers.

Fixed!
In case getExtent was deprecated (if older layers or certain sources used getExtent), if getExtent is an available function, it will use that.
Note: Trying to zoom to data with an mbtiles without modifying bounds in GeoServer will zoom to the whole world. This isn't because of MapLoom - and this bug will be filed elsewhere.


Implement IAM API

Carrying over from this issue on the GCN tracker, this is the only API left to expose the IAM API: https://cloud.google.com/compute/docs/access/iam

Hi, please refer to the new library, ^3.0.0.


⚠ ClamAV has degraded performance

In 8aee49d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 92 ms

Resolved: ClamAV performance has improved in 781f978.


Investigate open source models

This project looks amazing!! Unfortunately I cannot self host it without paying for OpenAI models - is there a possibility that you could look into alternative open-source models utilizing ggml and/or llama.cpp? So far I have found LLaVa but I'm not sure if it'd work for this project.

Yes! definitely would like to support LLaVa. Don't think it's going to be as good but worth supporting. Related: https://github.com/abi/screenshot-to-code/issues/15

LLava, CogVLM are worth experimenting with.

There is also
https://crosscodeeval.github.io/

Maybe the image processing can be done by LLaVA v1.5, but the code generation can be passed on DeepSeek Coder?

MayB yes
On Mon 20 Nov, 2023, 2:21 AM Altay Yuzeir, @.***> wrote:

Maybe the image processing can be done by LLaVA v1.5, but the code
generation can be passed on DeepSeek Coder
https://github.com/deepseek-ai/DeepSeek-Coder?
—
Reply to this email directly, view it on GitHub
https://github.com/abi/screenshot-to-code/issues/17#issuecomment-1817973830,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEDN5IZW7V7JPIPNET7JUODYFJWMTAVCNFSM6AAAAAA7QNXDMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJXHE3TGOBTGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

👍
Electricblue reacted via Gmail
https://www.google.com/gmail/about/?utm_source=gmail-in-product&utm_medium=et&utm_campaign=emojireactionemail#app
On Tue 21 Nov, 2023, 10:18 AM Electricblue 0003, @.***>
wrote:

MayB yes
On Mon 20 Nov, 2023, 2:21 AM Altay Yuzeir, @.***>
wrote:

Maybe the image processing can be done by LLaVA v1.5, but the code
generation can be passed on DeepSeek Coder
https://github.com/deepseek-ai/DeepSeek-Coder?
—
Reply to this email directly, view it on GitHub
https://github.com/abi/screenshot-to-code/issues/17#issuecomment-1817973830,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEDN5IZW7V7JPIPNET7JUODYFJWMTAVCNFSM6AAAAAA7QNXDMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJXHE3TGOBTGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

nice! can you share some results? screenshot and clone pairs.
also, you could try running llava 13gb with open router: https://openrouter.ai/models/haotian-liu/llava-13b?tab=stats

Will this work? https://github.com/vikhyat/moondream

we test the effect of cogvlm2 is perfect


DOC/CLN: Revise + clean up index.md and 3 how-tos

Make minor revisions/typo fixes etc in index.md
Revise, add headers in doc/howto/remove-silent-labels-textgrid.md
Clean up doc/howto/convert-generic-seq.md
Make fixes in doc/howto/remove-silent-labels-textgrid.md

Codecov Report
Base: 94.37% // Head: 94.37% // No change to project coverage :thumbsup:

Coverage data is based on head (71672c2) compared to base (c2f657d).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #217   +/-   ##
=======================================
  Coverage   94.37%   94.37%
=======================================
  Files          39       39
  Lines        1459     1459
=======================================
  Hits         1377     1377
  Misses         82       82

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 at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add documentation about the demo example and clean up examples

This is a good issue for someone interested in documentation. It doesn't require any particular knowledge of Vizro but does need an interest in poking through our examples folder in within vizro-core for this repo to see what is there and write about the layout and what the example shows. There's a basic style guide for docs creators here, but we are happy to take any contribution as we can run Vale over it to highlight any tweaks needed to follow the guidlines.
The examples folder in the repo holds the code for the demo.
It isn't very well highlighted in the docs and we need to do this:

Add a section on the home page and a link to a page of text about the demo (mentioned in #378 which will restyle that whole page)
Add the page itself. That page should document the basics of the demo and how to work with it.
Add a section in the examples folder where any contributions of example code for #377 will be added.

can i get assigned with this issue?

@sneha9231 - I think this might be an outdated ticket as the demo is not the same anymore!

@stichbury Is this ticket still relevant?

No, this ticket should now be closed since we've revised the demo to become the gallery - and have a separate set of (internal) tickets to make a few information architecture changes as a result. Apologies @sneha9231 that it's not one you could pick up, but thanks for the offer 🚀


e.ChatMessage.EmoteSet returns wrong emotes

Hi,
whem im using e.ChatMessage.EmoteSet and sending a emote twice:
Username: Kappa Kappa
Im not getting two Kappa
25:0-4,25:0-4 but
25:0-4,6-10
Thats happening with every emote im sending twice in one message.
Much thanks for your help! Will be probably the last thing im going to need help for :)

The way the emote set works is this. Given the following chat message:

You'll get the following emoteset:
28087:15-21/25:5-9,28-32

The string can be initially split by the / symbol. On the left side you have the WutFace data, and on the right side, you have the Kappa data for BOTH Kappa's.
So using: 28087:15-21.
The first number on the left of the : symbol is the emote ID. So this is the ID of WutFace. If you paste that ID into this format: https://static-cdn.jtvnw.net/emoticons/v1/{emote_id}/1.0 you can get the direct link as seen here:  .
The numbers on the right of the : symbol represent the start and end indices. So the WutFace emote starts on character index 15, and ends on character index 21.
The same can be applied to the right side of the /, the side representing the Kappas. The first number, 25 represents the ID of Kappa, so if we plug that ID into the emote link format above, we get  . This time however, since there are two Kappas in my message, we get two ranges. the first set is 5-9 and the second set is 28-32. This means that the first Kappa appears on character index 5, and ends on character index 9, while the second Kappa in the message appears on character index 28 and ends on character index 32.
TL;DR:
The numbers represent the emote ID and the emote start and end indices. I am currently writing code to parse this for the library, and return a list of ChatEmote objects that should make your job much easier.

Alright, looks like we're good. Access EmoteSet like you currently are but now instead of the string you pasted above, it will be an object. You'll want to then access the Emotes property, which is a list of Emote objects you can use. An Emote object has the following properties: Name, StartIndex, EndIndex, Id, and ImageUrl.
For reference, there's an example I added to the Example Application: https://github.com/swiftyspiffy/TwitchLib/blob/master/TwitchLibExample/TwitchLibExample/Form1.cs#L225

That works fawlessly for received messages! Thank you!
What about send messages?  Received event does not trigger when im sending through the twitchlib api. Only OnMessageSent - which does not have the new emote object but the old emoteset string, which is different to the old emoteset string on received messagges :)

OnMessageSent is a custom event I wrote (not sourced from Twitch itself), and it required a bit of hacking to get it to work as expected. Because of this, the event doesn't get the same indexing data that OnMessageRecevied gets. I can look into it, but you might need to do it on your side, as the data simply isn't available through conventional means for sent messages.

Ah okay.
Having issues in bigger channels with the new emotes integration. Getting this message on many new chat messages;
Ausnahme ausgelöst: "System.FormatException" in mscorlib.dll

How fresh is your code you are using? I accidentally had a bug in the code and I pushed a fix 2 hours ago. Can you try pulling the fresh code and give it a go?

The new version fixed it for me! Thanks!


Does not seem to work inside submodules

I tried using lazy_static inside a submodule, but nothing happens, my code seems to be never called.
When putting the call to lazy_static in my main.rs file, it works fine though.
Any idea? I tried that only to make my code cleaner, separated into files (a submodule dedicated to define my "global variables" -- because I have no other way to pass them around)

Hm, strange. The testsuite tests the macro also in a submodule, so that doesn't seem to be an issue.

Yes, I surely did something wrong somewhere, that's what I thought when I saw there was 0 issues here :)
I will try to come up with a simple scenario, see if I can reproduce then.

Ok I managed to reproduce. I am sure I did something wrong, please forgive me if it is the case.
Structure
src/
    main.rs
    app/
        mod.rs
        util/
            mod.rs
            robot.rs
            globals.rs

main.rs
extern crate lazy_static;

mod app;

struct Human;

impl Human
{
	fn new() -> Human { Human }

	fn test(&self)
	{
		println!("I am a human");
	}
}

lazy_static!{
	static ref HUMAN: Human = {
		let mut human = Human::new();
		human
	};
}

fn main()
{
	HUMAN.test();
	ROBOT.test();
}

app/mod.rs
pub mod util;

app/util/mod.rs
pub mod robot;
pub mod globals;

app/util/robot.rs
pub struct Robot;

impl Robot
{
	pub fn new() -> Robot { Robot }

	pub fn test(&self)
	{
		println!("I am a robot");
	}
}

app/util/globals.rs
use lazy_static;

use super::robot::Robot;

lazy_static!{
	pub static ref ROBOT: Robot = {
		let mut robot = Robot::new();
		robot
	};
}

When I comment the line ROBOT.test(); , it works fine, but with a warning:
warning: static item is never used: 'ROBOT'

When I uncomment the line, it fails:
error: unresolved name 'ROBOT'

Any ideas? For now I am creating my "globals" inside main.rs, it works fine, it is just not ultimately clean. But then again, I have only been at Rust for a few hours, so it is very likely that I did not understand something important about macros and modules.
Thank you Marvin,
Renaud

Well, your issue is that ROBOT is defined in a submodule, and thus not visible in your crate root.
Add a use app::util::globals::ROBOT; to your main.rs module.

Ooooh, so that's just it, then? I didn't know I was able to do a use on a
result of a macro. I am sorry... Should have tried that!
Thank you Marvin, I'll change that
Le mar. 22 sept. 2015 à 20:07, Marvin Löbel<EMAIL_ADDRESS>a
écrit :

Well, your issue is that ROBOT is defined in a submodule, and thus not
visible in your crate root.
Add a use app::util::globals::ROBOT; to your main.rs module.
—
Reply to this email directly or view it on GitHub
https://github.com/Kimundi/lazy-static.rs/issues/23#issuecomment-142367655
.

The macro basically just expands to a regular static variable with a special type, and access to things outside of a macro body itself follows regular Rust semantic. :)

That's what slipped out of my mind I presume ^^ Thank you Marvin for your
kind answer, and sorry for the false alarm!
Le mar. 22 sept. 2015 à 20:17, Marvin Löbel<EMAIL_ADDRESS>a
écrit :

The macro basically just expands to a regular static variable with a
special type, and access to things outside of a macro body itself follows
regular Rust semantic. :)
—
Reply to this email directly or view it on GitHub
https://github.com/Kimundi/lazy-static.rs/issues/23#issuecomment-142370011
.

No problem, glad I could help. :)


Build apps/andoird_rpc error

Actual behavior
after executing gradle clean build --stacktrace:

Task :app:buildJni FAILED
FAILURE: Build completed with 2 failures.
1: Task failed with an exception.
......
Caused by: java.io.IOException: Cannot run program "ndk-build" (in directory "/home/tvm/tvm/apps/android_rpc/app"): error=2, No such file or directory
at net.rubygrapefruit.platform.internal.DefaultProcessLauncher.start(DefaultProcessLauncher.java:25)
2: Task failed with an exception.
-----------
* What went wrong:
java.lang.StackOverflowError (no error message)
* Exception is:
java.lang.StackOverflowError
......

Environment
windows10 wsl, ubuntu 20.04
TVM version: latest(0.10)
Steps to reproduce
Follow the steps in ''https://tvm.apache.org/docs/install/from_source.html#install-from-source'' , build libtvm.so and libtvm_runtime.so successfully.
Follow the steps in android_rpc/README.md, I have installed gradle and tvm4j. Ndk for wsl linux is downloaded from https://developer.android.google.cn/ndk/downloads/, besides that, I also installed ndk on windows by android studio.
I have already set ANDROID_HOME and set sdk.dir in file local.properties.

android_rpc is tested on CI, please check your installation of ndk. Please post a usage question to the discuss forum.

I have posted a question on forum but got no answer, btw, what is CI?Thanks

I have posted a question on forum but got no answer, btw, what is CI?Thanks

Hi!How did you solve this problem? I also encountered the same problem.


How to catch this error: read ECONNRESET

How to catch this error: read ECONNRESET

Look at #164 for a workaround


Timed aggregator.

Online aggretor over time.
Samples added aggregate (scaled by amount of time a sample has lived for) at the time of adding sample.
Can get aggregate, aggregate duration and average value later.
First use: in scorer to keep track of aggregate bandwidth, average distance from desired layer

looks good 👍


Object with "Distance Fade" enabled does not cast shadows.

Tested versions

4.3

System information
Win 10
Issue description
The wall has Distance Fade enabled. It stops to casting shadows.

https://github.com/godotengine/godot/issues/69329#issuecomment-1330895397

To resolve this, use the Pixel Dither or Object Dither transparency mode for Distance Fade.

Pixel Dither does not resolve this.
Also if this is intended then the tooltip should say that the object will stop casting shadows.

But object still do cast shadows from direct light:

Which makes me think that light camera is affected by same shader and are just to close to object. Which means light camera pass should use different settings/different shader.
Changing distance to very small number solves the shadows but looses its usability meaning ofc:

Another detail:
This is GridMap. Which means that material was assigned to mesh material slot and not to OverrideMaterial Slot of MeshInstance (because GridMap has no Override Material slot, but it should!)

Steps to reproduce
Enable Distance Fade on material and put light close to object.
Minimal reproduction project (MRP)
N/A

Probably could be solved by this:
https://github.com/godotengine/godot-proposals/issues/4443

I think you are on the right track and the problem is that the shader considers the distance to the light instead of the distance to the camera.
See here:
https://github.com/godotengine/godot/blob/76a135926aef1f02f27e4e09093787f2c670956d/scene/resources/material.cpp#L1668
here:
https://github.com/godotengine/godot/blob/76a135926aef1f02f27e4e09093787f2c670956d/scene/resources/material.cpp#L1673
and here:
https://github.com/godotengine/godot/blob/76a135926aef1f02f27e4e09093787f2c670956d/scene/resources/material.cpp#L1689
We solved a similar problem with billboards in https://github.com/godotengine/godot/pull/72638. A similar solution should be applied here

I've solved it with this custom shader for now. But works only for DualParaboloid shadow mapping.

bool is_shadow_pass = (PROJECTION_MATRIX[2][3] == 0.0);
if(!is_shadow_pass)
	{

		// Distance Fade: Pixel Dither
		float fade_distance = length(VERTEX);

		// Use interleaved gradient noise, which is fast but still looks good.
		const vec3 magic = vec3(0.06711056, 0.00583715, 52.9829189);

		float dfmn = distance_fade_min;
		float dfmx = distance_fade_max;
		if(is_shadow_pass){
			 dfmn= 0.0;
			 dfmx= 0.1;
		}

		float fade = clamp(smoothstep(dfmn, dfmx, fade_distance), 0.0, 1.0);
		// Use a hard cap to prevent a few stray pixels from remaining when past the fade-out distance.
		if (fade < 0.001 || fade < fract(magic.z * fract(dot(FRAGCOORD.xy, magic.xy)))) {
			discard;
		}
	}


UnixSockets now on Rust 1.10

Now that this landed on stable (std::os::unix::net)...
Are you moving the crate to @rust-lang-deprecated ?
Are there any features on this crate that didn't land on 1.10-stable?

Good question! This crate is almost identical to the unix::net module right now. I think the only difference is that this crate will accept abstract namespace addresses while the standard library version will not. I expect we'll deprecate the UnixStream, UnixListener, UnixDatagram, etc types in this crate when 1.11 or 1.12 lands.
I imagine this crate will continue to exist for a while longer by adding more functionality than is currently surfaced in the standard library. There's a pending PR to add SEQPACKET support, and there's some other functionality that would be good to support as well like file descriptor transfer and credential transfer.

Good answer!
Can we make the Unix* in this crate re-export the std counterparts and then add the extra functionality through extension traits?

Nursery crates need to build against the last two Rust releases (i.e. 1.9 and 1.10 right now), so we'll have to wait a bit to do that.

Ohh, I didn't know about the 2-back requirement on nursery. Thanks!

I don't see any notion of abstract addresses in std::os::unix::net and I don't see std::os::linux::net at all.
Shall there be SocketAddrExt plus more convenient function to create such addresses?

I expect we'll deprecate the UnixStream, UnixListener, UnixDatagram, etc types in this crate when 1.11 or 1.12 lands.

Maybe better replace them with pub use then (assuming abstract addresses won't be broken by it)?

Abstract addresses will be broken as they're not supported by the standard library implementation.

So until abstract addresses not supported by std this crate is not obsolete.

Assuming a nonzero number of people actually use abstract addresses, which has never been clear to me.

Like with passing file descriptors over sockets, credentials and SEQPACKET, there may be a loop "nobody uses, let's don't support" <-> "nobody supports it, we won't use it".

By "nonzero number of people", I mean that across all languages, not just Rust. I have not personally heard of it being used anywhere.
The other features you've mentioned would be great to support. There's a stalled PR adding SEQPACKET support, and another one for file descriptor and credential transfer.

Abstract addresses are useful when:

Filesystem is separated (e.g. chroot) or somehow unavailable (e.g. security syscall filter). Or there are problems with clearing stale sockets (abstract ones have no file, hence nothing to clean);
Namespaces are in use. Abstract UNIX sockets are attached to network namespace and regular UNIX sockets are attached to files (mount namespace). Hence they are different from security point of view;
Experimentation and wanting to quickly connect two programs together. Just don't need to think about where to put socket file, give appropriate permissions, clean up later and so on.

For example X11 display server typically listens both regular and abstract sockets, allowing to continue using GUI even inside a chroot.
Workaround for missing abstract sockets is using a pair of socats (socat abstract-listen:qqq unix-connect:qqq & socat unix-listen:qqq2 abstract-connect:qqq), but it is ugly.

Oh great. I don't think there's anything blocking getting them in the standard library implementation on a Linux-specific extension trait.


[RFE] Ability to change the Rancher UI background color to differentiate Dev from Prod environments

Persona: Rancher Operator/Rancher Admin
As a Rancher Operator, I would like to have in the Rancher UI a visual evidence of the environment I am working on, so that I am not running a deployment in a Dev environment, for example.
That's why I would like to have the ability to change the Rancher UI background color to differentiate Dev from Prod environments.
gz#10458

Shouldn't this be in the rancher issue tracker, not k3s?

I think this can be closed in favor of https://github.com/rancher/rancher/issues/27480


Laravel 11 framework config auto-merging is a risky breaking change

Laravel Version
11.16
PHP Version
8.3
Database Driver & Version
No response
Description
Hi. 👋 Unfortunaly, the auto-merging of config files broke our app after upgrading to L11.
In our app we are doing something like this, where expected connection data is used in doStuff():
protected function handleEvent($stage): void
{
    /** @var array<string, array<string, string>> $connections */
    $connections = $this->app['config']->get('database.connections');

    foreach ($connections as $connection) {
        $this->doStuff((object) $connection, $stage);
    }
}

Or something like this, where expected connection data is passed over to shell scripts via a command.
$databaseConfig = Config::get('database');

// output connection credentials to be
// consumed by shell script.

foreach ($databaseConfig['connections'] as $connection) {
    echo "db_name={$connection['database']} ";
    echo "db_user={$connection['username']} ";
    echo "db_host={$connection['host']} ";
    echo "db_port={$connection['port']} ";
    echo "db_password={$connection['password']} ";
    echo PHP_EOL;
}

We keep our config files clean, and don't have any unused connections in there
return [
    'connections' => [

        // e.g. no sqlite here

        'core' => [
            // stuff
        ],
        'geo' => [
            // stuff
        ],
        'statistics' => [
            // stuff
        ],
];

Because of config merging by the framework database.connections is
1: array (size=8)
    sqlite => array (size=8)
    mysql => array (size=15)
    mariadb => array (size=15)
    pgsql => array (size=12)
    sqlsrv => array (size=10)
    core => array (size=16)
    geo => array (size=16)
    statistics => array (size=16)

instead of the following, where we deal with explicitly expected data
1: array (size=8)
    core => array (size=16)
    geo => array (size=16)
    statistics => array (size=16)

The issue with this is that e.g. sqlite does not have all keys we expect, which e.g results in:
Undefined property: stdClass::$xyz

// or

Undefined array key password

Before Laravel 11 our approach was perfectly fine, because we were in control what ends up in the final config and what not. Now, with the framework auto-merging in undesired config values, our apps end up to be blackboxes.
After understanding what is going on I learned about  the undocumented
$app->dontMergeFrameworkConfiguration();

which solved the issue.
However, this let's me think: will we now, since we opted out, be at risk to run in similar issues in the future because the framework could have added new configs, which potentially are used by feature ABC or package XYZ?
Since there is no documented breaking change, I decided to submit this as a bug report to start a discussion.
Personally, and respectfully, I really think the new behaviour is dangerous.
I don't see how either way, opting in and opting out from auto-merging can be safe.
Proposed solutions for L12

Go back to the previous L10 behaviour. Don't merge configs at all.
Don't merge if the config file exists in the app, and commit to not adding new config values in non-major versions.

I guess, at the very least, for now this should be documented as high impact breaking change.
Thanks in advance for your consideration. 🙏
Steps To Reproduce
self-explaing?

Hi @NickSdot, thanks for your thorough issue. Unfortunately there's no action to take on our side as this is simple the new way things work in Laravel v11. Deviating from it is a major change and something that'll severely impact any Laravel v11 app. We added dontMergeFrameworkConfiguration specifically for apps like yours to circumvent this. In general I would recommend discovering a different approach on your part so you don't have to rely on dontMergeFrameworkConfiguration and can just use v11 as intended. Right now we don't have any plans to move from the current way of merging configs sorry.
You're free to attempt a PR to the docs to document dontMergeFrameworkConfiguration if you want 👍

Thanks for your answer @driesvints. Surely, in this particular case, we could be explicit in what connections we want to select. Unfortunally, there is no different, universal approach to discover if the framework can merge in new stuff at any time.
Either we opt-out (in whatever way, or we don't.
I appreciate all your hard word, and am sorry to say it, but Laravel – with this – has built a ticking time bomb.

@NickSdot can't you just create your own config file for this and extract everything from there?

Sure, I can duplicate the database config (plus whatever else unexpectedly explodes at some point) and maintain everything twice.
I could also bind an additional full, unmerged "safe config" to the container, and add my own SafeConfig facade to be save, theoretically. This duplicates all the work the LoadConfiguration bootstrapper does. Just to have a second, safe, unmerged config.
But what good does that really do?
We end up with two configs, one safe and one unsafe.
Depending on which one you use, the problem still exists.
In the end it, unfortunally, still doesn't solve anything.
It's the same as in opting out.

I don't see how either way, opting in and opting out from auto-merging can be safe.

And fwiw, this is the case even if I would directly require specific config files (with the cost of losing caching, and with the cost of losing dot notation etc. pp.).
The problem here is that the issue is not limited to the above case.
The concept itself is a risk and - IMHO - broken.
My main concern is that we no longer know what is happening under the hood. Unexpected things could happen at any time, if there is not at least a committment to not introduce new config values in non-major versions. I saw it today - the hard way.
I don't mean to be disrespectful with the following. But is it really worth hiding a bunch of config files and risking breaking apps (ours just did) when there are classes that show me 176 methods in auto-completion?
This is how Laravel 10 looked like.

This is how a typical root directory of an app looks like (hi laravel.io).

It's config files, it's normal that they are there.
Should the framework really merge in a bunch of (in this case) unused database connections? Is it really cleaner if e.g. Laravel IDEA is daily suggesting me unused, auto-merged config values I don't want and use? I don't know how it is for you. Personally, I look more often at auto-completion than in the config folder.
Taylor said there are no breaking changes.

He, understandable, also cannot think of every potential problem in advance. But here we are. And I think with the new input from this issue, things could, and should, be at least re-considered.
By saying

@NickSdot can't you just create your own config file for this and extract everything from there?

you are basically suggesting that we should stop embracing the framework. The config is one of the most core things of the framework I can think of. Should we all start duplicating our config handling to be sure nothing unexpected happens? Or opt-out to end up having the risk from the opposite direction?
I understand that decisions were taken.
However, this issue proves that the new concept can and does break things.
Not sure if I am really the first one reporting such a case. But I would find it really unfortunate if there would be zero re-consideration after I did report it, @driesvints.
Sorry for the long text. You see I am worried. Not sure if that is still the DX I have known and liked for the last 11 years.
I would appreciate if you at least could re-open this to allow some visibility to gather more opinions. 🙏

I'm sorry you got caught with this @NickSdot but please hear me out. You're currently the only person who reported something like this. Plus it's from a very specific use case by you and you alone. Which brings me to the following:

I don't want to be dismissive by posting a comic but want to illustrate a point. We took careful consideration by moving into the direction we did for the config files of the framework. It could very well be that something specific broke for you for your specific use case. But in the vast overwhelming majority of Laravel apps everything is fine.
Right now we're not considering reverting this direction we took. I also believe there are many creative ways you could do what you want to do without a lot of effort on your side. Again, we're sorry things broke for you. But I'm sure you'll find an easy solution to this.

I want to add that i agree with @NickSdot (so that it doesn't look like he's the only one in the world to have this point of view).
In our team we have always cleaned/removed unused connections/stores from the config files (such as cache, database, filesystems and logging, did i forget any?), just because that since are not using those connections, we don't want any un-used/un-configured/non-working connections to even be available in the system, that has been our way of keeping our systems lean.

Same issue, but with queue.connections!
This auto-merge is breaking a lot of stuff in our project, because of this default unwanted connections from base Laravel configs. And this is hard to search and fix, because there is no any word in Upgrade Guide about it!
This is, definitely, not the correct behavior and not "no breaking changes"

If someone would send in a PR to document dontMergeFrameworkConfiguration in the upgrade guide then that would be very much appreciated 👍

We also just into this problem.
Yeah, we are a bit late to the whole update laravel 10->11 party.
In our usecase we have some kind of healthcheck that tests the accessibility of filesystems/caches/databases based on the configured values.
With the update to laravel 11 we could see that all our services would not update due to healthchecks failing and/or timing out.
Thanks to this issue we quickly found a solution/workaround.
However I also strongly suggest that this behavior should at least be documented.

i made a small package as a workaround:
https://packagist.org/packages/dreadkopp/laravel-configured
use Configured - facade or configured() method


PowerToys bug

ℹ Computer information

PowerToys version: 0.33.1
PowerToy Utility: PowerToys Run
Running PowerToys as Admin: Yes
Windows build number: 20H2 Build 19042.844

📝 Provide detailed reproduction steps (if any)

Open PowerToys Run with Alt + Space

✔ Expected result
Open the search box without bug
❌ Actual result
The search box is blocked and black screen is appeared
📷 Screenshots
``

Please right click on the PowerToys icon in the taskbar on the bottom right. Then choose report bug. A zip gets generated that you can attach to a new comment of this issue

@iliasrafia @Aaron-Junker
it's unlikely that the log file has anything useful for for this particular bug. But let's take a look at the logs anyway.
@iliasrafia
can you try taking a screen capture with https://www.screentogif.com/ or any other screen capturing tool, to show exactly when that black box appears on screen?

Please right click on the PowerToys icon in the taskbar on the bottom right. Then choose report bug. A zip gets generated that you can attach to a new comment of this issue

PowerToysReport_2021-03-11-13-52-53.zip

2021-03-05 23:49:16.1694] [ERROR] [S:\src\modules\launcher\Plugins\Microsoft.Plugin.Program\Programs\ShellLinkHelper.cs::172]
-------------------------- Begin exception --------------------------
Message: Failed to fetch description for C:\Windows\syswow64\odbcad32.exe, Unspecified error
'200' violates enumeration constraint of '100 120 140 160 180'.
Exception full name  : System.Runtime.InteropServices.COMException
Exception message    : Unspecified error
'200' violates enumeration constraint of '100 120 140 160 180'.
Exception stack trace:
at Microsoft.Plugin.Program.Programs.ShellLinkHelper.IShellLinkW.GetDescription(StringBuilder pszName, Int32 cchMaxName)
at Microsoft.Plugin.Program.Programs.ShellLinkHelper.RetrieveTargetPath(String path)
Exception source     :
Exception target site: Void GetDescription(System.Text.StringBuilder, Int32)
Exception HResult    : -2147467259
-------------------------- End exception --------------------------

@Aaron-Junker
that is an expected error that we should not log in the first place.

@iliasrafia is this still reproducible on the latest PowerToys version on your side?
/needinfo


WebpackError: document is not defined

I added this package to Gatsby.js React project.
when running "gatsby develop" CLI, site works fine. script loads in what is needed.
But when trying to create production build, "gatsby build" the build fails because scriptjs tries to execute since it thinks it is in browse environment meanwhile it is not yet. The link below explain the situation better. So if you can update your code to only run when "document" is available would be great.
In the meantime I am using method b) by placing the code in componentDidMount(), which is working for me.
See our docs page on debugging HTML builds for help https://goo.gl/yL9lND
10 |   else this[name] = definition()
11 | })('$script', function () {

12 |   var doc = document
| ^
13 |     , head = doc.getElementsByTagName('head')[0]
14 |     , s = 'string'
15 |     , f = false

WebpackError: document is not defined

script.js:12
~/scriptjs/dist/script.js:12:1

script.js:8 doc
~/scriptjs/dist/script.js:8:1

script.js:11 Object.module.exports.layoutContext
~/scriptjs/dist/script.js:11:2

index.js:5 Object.module.exports.siteMetadata.title
src/layouts/index.js:5:1

index.js:3 Object.exports.__esModule
.cache/layouts/index.js:3:3

sync-requires.js:6 Object.exports.__esModule
.cache/sync-requires.js:6:35

The same error...

I've created a fork with the necessary fix if it helps anyone. https://github.com/JJCLane/script.js/commit/5718da99068600773c3f274f49fa6afcf719369c


docs: typo the documentation for moto sagemaker-runtine does not work

In the documentation for mocking invoke_endpoint (here) we can find the following code:
expected_results = {
    "account_id": "123456789012",  # This is the default - can be omitted
    "region": "us-east-1",  # This is the default - can be omitted
    "results": [
        {
             "Body": "first body",
             "ContentType": "text/xml",
             "InvokedProductionVariant": "prod",
             "CustomAttributes": "my_attr",
         },
        # other results as required
    ],
}
requests.post(
    "http://motoapi.amazonaws.com:5000/moto-api/static/sagemaker/endpoint-results",
    json=expected_results,
)

client = boto3.client("sagemaker", region_name="us-east-1")
details = client.invoke_endpoint(EndpointName="asdf", Body="qwer")

This does not work in our hands. The problematic line being:
requests.post(
    "http://motoapi.amazonaws.com:5000/moto-api/static/sagemaker/endpoint-results",
    json=expected_results,
)

This raises the following error:
TypeError: cannot unpack non-iterable NoneType object

The code crashes here:
def get_response(self, request: Any) -> responses.HTTPResponse:
        """
        Need to override this so we can pass decode_content=False
        """
        if not isinstance(request, Request):
            url = urlparse(request.url)
            if request.body is None:
                body = None
            elif isinstance(request.body, str):
                body = request.body.encode("UTF-8")
            elif hasattr(request.body, "read"):
                body = request.body.read()
            else:
                body = request.body
            req = Request.from_values(
                path="?".join([url.path, url.query]),
                input_stream=BytesIO(body) if body else None,
                content_length=request.headers.get("Content-Length"),
                content_type=request.headers.get("Content-Type"),
                method=request.method,
                base_url=f"{url.scheme}://{url.netloc}",
                headers=[(k, v) for k, v in request.headers.items()],
            )
            request = req
        headers = self.get_headers()

        from moto.moto_api import recorder

        recorder._record_request(request, body)

        result = self.callback(request)
        if isinstance(result, Exception):
            raise result

>       status, r_headers, body = result

Removing the port 5000 solved my error.
I can make a proper PR when I am back home.
Many thanks
Best regards

Hi @fbraza, thanks for letting us know! I've fixed this in the source here: https://github.com/getmoto/moto/commit/40a86b7fb40b1ce0b7e117eb46cb68734ddf65bf
The docs themselves will be updated with the next release.

Many thanks for prompt answer. All the best


improvement suggestion:awaitUser to resolve user immediately when user logged in

Hi,
On line 120 of angular-meteor-auth.js you do an autorun, which will check if the user is logged in.
It looks it always requires an extra cycle to check, even if a user is already logged in.
I will give an example where I run into this issue:
I have set up a $stateChangeStart event handler that checks if certain paths/state are allowed.
On that is NOT allowed is to go to login, when you are already logged in.
So I have the following 2 scenario's:
1 ) I am logged and setting in my dashboard. I state change to /login via ui-sref of ui-router and then the $stateChangeEvent executes. Because I am already logged in, I USED to use
if ( Meteor.user() ) {
      let states: Array<string> = [ 'public.auth.login', 'public.auth.register', 'public.login.recovery' ];
      if ( _.includes( states, toStateName ) ) {

        event.preventDefault();

        this.$state.go( 'private.dashboard.open' );
      }
}

This will force the user to the dashboard instead of going to the login, when we are already logged in.
HOWEVER, this doesn't work if I go to the login page by changing the url in the browser manually.
Then, the Meteor.user() check will be undefined, and the force redirect will not occur.
So, to circumvent this I decided to use the awaitUser construct.
This, works fine, however in the $stateChangeStart handler, it will run 2 times. The first time the user is not resolved (while I am already logged in) and it will try to go to the login page.
Then it runs again and it CAN actually execute the event.preventDefault.
Is it possible, to resolve the promise right away when awaitUser is called so that I do not get 2 cycles in which my UI state flashes from one state to the other?
EDIT: my solution seems to work in my local version of your package.

 $$Auth.$awaitUser = function (validate) {
      var _this = this;

      validate = validate ? this.$bindToContext($Mixer.caller, validate) : function () {
        return true;
      };

      if (!_.isFunction(validate)) {
        throw Error('argument 1 must be a function');
      }

      var deferred = this.$$defer();

     // Resolve promise immediately if user exists
     //
      if ( _this.currentUser ) {
         deferred.resolve( _this.currentUser );
         return deferred.promise;
      }

      // Note the promise is being fulfilled in the next event loop to avoid
      // nested computations, otherwise the outer computation will cancel the
      // inner one once the scope has been destroyed which will lead to subscription
      // failures. Happens mainly after resolving a route.
      var computation = this.autorun(function (computation) {
        if (_this.getReactively('isLoggingIn')) return;
        // Stop computation once a user has logged in
        computation.stop();

        if (!_this.currentUser) return _this.$$afterFlush(deferred.reject, errors.required);

        var isValid = validate(_this.currentUser);
        // Resolve the promise if validation has passed
        if (isValid === true) return _this.$$afterFlush(deferred.resolve, _this.currentUser);

        var error = void 0;

        if (_.isString(isValid) || isValid instanceof Error) {
          error = isValid;
        } else {
          error = errors.forbidden;
        }

        return _this.$$afterFlush(deferred.reject, error);
      });

      var promise = deferred.promise;
      promise.stop = computation.stop.bind(computation);
      return promise;
    };

@Urigo I this something that still has value?

@mattiLeBlanc It is relevant indeed, what I am afraid of is that it would be a breaking change since some implementations might rely on the stop() method we bind to the returned promise, we will need to think of a proper solution that won't break any app.

We could still do something like this
let promise =  deferred.promise;
promise.stop = null;
deferred.resolve( _this.currentUser );
return promise;

Or maybe make promise.stop an empty function?

@mattiLeBlanc closing with the new version, let me know if it works for you

great! Thanks for integrating this. Code looks good, should work.


Create a core package with a copy from ↗ openscd/open-scd-core

Solved with https://github.com/openscd/open-scd/pull/1363


weird problem with avcodec-57.dll

I've read that some people has also problem with it, and it seemed in a comment that the solution was to put the file in a folder with  a short name, but it is weirdest. The same avcodec-57.dll is in a folder in which with your demo it's fine and if I open a new empty app and in the Form create event: loadlibrary returns 0 and getlasterror returns 126, module not found, and obviously it's there. So I really don't know what is happening here.

Maybe the issue is related with the fact that I don't see how the unit UDemoBitmaps2Video has access to the unit UBitmaps2Video, having in mind that the project doesn't include the unit folder to the search path.


Can't download pretrained weights from aws

I completed the compile and I try to download the pretrained weights using download.sh
But ERORR 404: NOT FOUND is occurred.
https://s3.us-east-2.amazonaws.com/flownetdata/weights.tar.gz is not working.
How can I solve it? or get the weights?
Thanks.

Seems that the file has been deleted. I can upload mine to somewhere. This might take around one week since I am not out of campus.

could you please send to my email<EMAIL_ADDRESS>Thank you very much!

Please use this backup


[tests] remove beforeall from a bunch of tests

What this PR does / why we need it:
Remove some BeforeAll usages to make  progress towards removing it completely. 20 more tests are now executed in parallel.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/retest

/retest

/remove-lifecycle stale

/cc @dhiller
rebased this old PR which also helps running more tests in parallel. I think it is ready for a review.

M	tests/storage/storage.go
M	tests/vmi_cloudinit_test.go
M	tests/vmi_configuration_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/vmi_configuration_test.go
Auto-merging tests/vmi_cloudinit_test.go
CONFLICT (content): Merge conflict in tests/vmi_cloudinit_test.go
Auto-merging tests/storage/storage.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 remove beforeall from a bunch of tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

@rmohr FYI ^^


Add posibility to give config from playbook

Would be great to be able to give configuration files from playbook and overwrite the defaults inside the role

Release 2.0.0 #11

Release 2.0.0 #11


Throw if no such index/mapping

Is there a way to make ElasticClient blow up on Index() if there is no such index/mapping? We don't use dynamic mapping, so if anything is trying to index a doc into non-existent index it's always a sign that something is broken.
Here's example:
const string defaultIndex = "must-throw";
var connectionSettings = new ConnectionSettings(new Uri("http://localhost:9200"), defaultIndex);
var elastic = new ElasticClient(connectionSettings);
if (elastic.IndexExists(defaultIndex).Exists)
    elastic.DeleteIndex(defaultIndex);

// must throw something here
elastic.Index(new {name = "Panda"});

https://www.elastic.co/guide/en/elasticsearch/reference/1.6/mapping-dynamic-mapping.html
You can set dynamic: false to silently ignore unmapped fields or dynamic: strict to make elasticsearch return an exception. If you configure the client with .AlwaysThrowOnElasticsearchServerExceptions() the client will also throw a hard exceptions.
The cool thing is that you can make it strict for the root object but reenable dynamic mapping in sub objects in the json grap e.g an explicit metadata field.
in Nest call
.Dynamic(DynamicMappingOption.Strict)

Thanks for your response!
Just for the record, I put these two lines into elasticsearch.yml:
action.auto_create_index: false
index.mapper.dynamic: false

And made NEST client throw on server errors:
var connectionSettings = new ConnectionSettings(new Uri("http://localhost:9200"), defaultIndex).ThrowOnElasticsearchServerExceptions();

And hooray, I have the exception. Exceptions are great. So much love for exceptions when it's third time in a row you debug a problem only to find out it was caused by somebody dropped the mapping and the app silenty recreated it (wrong).


Update requirements section and examples on Add-PnPSiteCollectionAppCatalog command

An open connection to the SharePoint Tenant Administration site is needed when invoking the command. In other case, you will receive a 403 error from the console. I've updated the documentation to clarify the use of the Add-PnPSiteCollectionAppCatalog command.
Type

[ ] Bug Fix
[ ] New Feature
[X] Sample

What is in this Pull Request ?
All changes has been done in Add-PnPSiteCollectionAppCatalog command documentation:

Rename the section "Required permissions" to "Requirements" and update the explanation,

remove the example 1 provided because it was wrong, there is no way to create a site collection app catalog when connecting to that site collection,

update the example 2 and set as "Example 1"

Thanks for your submission @codbugs but should actually not be the case. You should be able to connect to any site collection and on it execute Add-PnPSiteCollectionAppCatalog. Internally it will clone the context to the admin context and then add the site scoped app catalog features to the current site.
Exactly how are you connecting if it doesn't work well for you?

Hi @KoenZomers,
If I do what you said this is what I get:

The context is that I'm creating some stuff and the last thing I want to do is to set the site collection as an app catalog. The command that I run is "Add-PnPSiteCollectionAppCatalog -Site $SiteUrl", and $SiteUrl parameter is the current site collection url. As you can see in the screenshot, I always get the 403 error message.
The only thing that worked was to connect to the SPO tenant administration site, set the site as an app catalog, and, then, connect to the site collection again.
Let me know if you want more information.

To add more information here:

I'm the tenant administrator and that's the account I'm using to run the commands,
I'm the site collection administrator of the targeted site collection,
I'm the site collection administrator of the tenant app catalog,
The tenant app catalog was created 5 or more days ago

Can you share how exactly you connect? The problem likely lies in that.

This is the command: Connect-PnPOnline -Url $SiteCollectionUrl -UseWebLogin

Can you try it with Connect-PnPOnline -Url  -Interactive and see if that makes a difference for you?

Interactive parameter is not supported. I installed the latest version of the PnP Module and running under the PowerShell 5.1 version.

Please try it using PowerShell 7. We're stepping away from PowerShell 5 support in our next release anyway.
https://github.com/PowerShell/PowerShell/releases/tag/v7.1.4

And please ensure you're using PnP.PowerShell. In the above screenshot you're using SharePointPnPPowerShellOnline which we stopped supporting back in December 2020.

In my understanding, I'm using the Microsoft recommended version I take from this link, https://docs.microsoft.com/en-us/powershell/sharepoint/sharepoint-pnp/sharepoint-pnp-cmdlets?view=sharepoint-ps. I believe my PowerShell version is correct for this version of the PnP as well.

I'll look into having that documentation updated. If you don't wish to follow the guidance here to try to understand why it doesn't work for you I'm afraid I can't help you further and we'll need to close this PR.

No worries on that, was my mistake.

Closing this PR because its not applicable to this module.
Please try with the PnP.PowerShell module, the module mentioned in the link, SharePointPnPPowerShellOnline is now deprecated.
If the issue still exists when you use this module, would be happy to accept changes :)


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/soef/ioBroker.samsung

[ ] [E019] Invalid repository URL: https://github.com/soef/iobroker.samsung. Expected: git+https://github.com/soef/ioBroker.samsung.git
[ ] [E104] No common.titleLang found in io-package.json
[ ] [E114] No adapter are allowed in the repo without admin3 support
[ ] [E150] No common.connectionType found in io-package.json
[ ] [E152] No common.dataSource found in io-package.json
[ ] [E802] No topics found in the repository. Please go to "https://github.com/soef/ioBroker.samsung", press the settings button beside the about title and add some topics.
[ ] [E603] NO "## Changelog" found in README.md
[ ] [E604] No "## License" found in README.md
[ ] [E701] No actual year found in LICENSE. Please add "Copyright (c) 2015-2022 soef<EMAIL_ADDRESS>at the start of LICENSE

I have also found warnings that may be fixed if possible.

[ ] [W113] Adapter should support compact mode

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

This issue was moved to iobroker-community-adapters/ioBroker.samsung-community#10

Do you need help fixing the bugs?

Thanks, that all bugs have been fixed.


Not supporting for Firebase Cloud Messaging API (V1)

As google firebase Disabled Cloud Messaging API (Legacy), this module should support for Firebase Cloud Messaging API (V1)

Please check the description of release 1.3.8, thanks!
https://github.com/kutia-software-company/larafirebase/releases/tag/1.3.8


For #1055: Implement Text by RqPrint

See #1055

@paulodamaso Have a look, please

@rultor merge

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@paulodamaso Done! FYI, the full log is here (took me 11min)

Job gh:yegor256/takes#1056 is not assigned, can't get performer


fix: switched from set() to update()

update() should be favoured over set(). From the documentation:

For example, use [update} when value is passed into the class constructor parameters.

This is likely impacting performance during recomposition.

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[X] Make sure to open a GitHub issue as a bug/feature request before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[X] Ensure the tests and linter pass
[X] Code coverage does not decrease (if any source code was changed)
[X] Appropriate docs were updated (if necessary)

Fixes #547 🦕

Regarding the breaking change: I would not like to release this as a standalone fix/feature and a breaking change. However, we can pack it together with #516 and/or #521, to minimize the amount of major releases, which are always painful.

Sounds fine. Right now the changes are inconsistent. Marker() has a single changed version, while AdvancedMarker() and MarkerComposable() have one changed and one unchanged version each.

Please revert 450d842 prior to merge, it was a bad commit.

This was fixed before the merge, thanks!

@kikoso @dkhawk On my end I see that https://github.com/googlemaps/android-maps-compose/commit/450d842c38ca8ef9fb5938425408f57c3a793bb7 was indeed merged to main and introduced documentation problems.


Displaying incorrect response message on updating the status of products.

Framework Version: 0.1.3
Test Steps:

On changing the status of product it displaying the message that product has been deleted successfully

Not able to check due to issue #523

Framework Version: 0.1.4
Fixed


Autofocus search field

It would be nice to have the search field autofocused

on page load
after pressing the X button to clear the search input

This use case was also described in #6.
As this affects the UI, the implementation has to be done in darekkay/static-marks-app.

Could I work on this issue @darekkay ?

@veerreshr Sure 🚀

Thanks!
@all-contributors please add @veerreshr to maintenance

Released in version 2.2.1.


Use standard C or C++ atomics for memoryviews

Use them in preference to the existing compiler-specific
atomic types.
In both cases this requires the 2011 edition of the standard.
Closes https://github.com/cython/cython/issues/4923

Looks good. I'll wait until after the next alpha release with a merge since I'd like to have a safe 3.0 version out there that supports Python 3.11, and this addition seems somewhat risky.

Looks good. I'll wait until after the next alpha release with a merge since I'd like to have a safe 3.0 version out there that supports Python 3.11, and this addition seems somewhat risky.

Yeah that seems reasonable. I haven't confirmed that it works on MSVC yet which I think is the main risk of breaking something.


Adding link for enterprise docs

Adds a link to the doc switcher for the Enterprise docs.
Also, updates the theme to address the menu width for the long name.

This PR is no longer needed.


Remove version: HeidiSQL.HeidiSQL version 12.6.0

Version is actually 12.6
Microsoft Reviewers: Open in CodeFlow

@stephengillie for your reference I have submitted the fix here https://github.com/microsoft/winget-pkgs/pull/142923

URL: https://www.heidisql.com/installers/HeidiSQL_<IP_ADDRESS>65_Setup.exe
Status Code: 200
(Automated message - build 869)


Automated cherry pick of #8450: host increase cpu mem max size

Cherry pick of #8450 on release/3.5.
#8450: host increase cpu mem max size

Codecov Report

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

@@              Coverage Diff              @@
##             release/3.5   #8451   +/-   ##
=============================================
  Coverage               ?   3.52%
=============================================
  Files                  ?     304
  Lines                  ?   62594
  Branches               ?       0
=============================================
  Hits                   ?    2205
  Misses                 ?   60236
  Partials               ?     153

Flag
Coverage Δ

#aFlag
3.52% <0.00%> (?)

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 b033b6c...998909c. Read the comment docs.

/lgtm
/approve


Crashes if typing run

Describe the bug
Crashes if typing run. Even without Enter after it.
To Reproduce
Steps to reproduce the behavior:

start app
start typing run and press space
it crashes.

Expected behavior
To show available images to run
Screenshots

Text log*
det@B-503:~  ds
>>> docker runpanic: runtime error: index out of range [0] with length 0

goroutine 1 [running]:
main.imagesSuggestion(0x14743e0, 0xc0005b4000, 0x14f4ae5)
        /Users/det/Downloads/docker-shell/main.go:229 +0x320
main.completer(0xc0005a2004, 0x4, 0x4, 0x400, 0x0, 0x0)
        /Users/det/Downloads/docker-shell/main.go:166 +0x1d0
github.com/c-bata/go-prompt.(*CompletionManager).Update(...)
        /Users/det/.go/pkg/mod/github.com/c-bata/go-prompt@v0.2.3/completion.go:69
github.com/c-bata/go-prompt.(*Prompt).Input(0xc00007e000, 0x0, 0x0)
        /Users/det/.go/pkg/mod/github.com/c-bata/go-prompt@v0.2.3/prompt.go:260 +0x47e
github.com/c-bata/go-prompt.Input(0x14f6168, 0xb, 0x15166e0, 0xc0001dff50, 0x4, 0x4, 0x5, 0x0)
        /Users/det/.go/pkg/mod/github.com/c-bata/go-prompt@v0.2.3/shortcut.go:16 +0xd8
main.main()
        /Users/det/Downloads/docker-shell/main.go:247 +0x299

Version
Client: Docker Engine - Community
 Version:           19.03.8
 API version:       1.40
 Go version:        go1.12.17
 Git commit:        afacb8b
 Built:             Wed Mar 11 01:21:11 2020
 OS/Arch:           darwin/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.8
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.17
  Git commit:       afacb8b
  Built:            Wed Mar 11 01:29:16 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          v1.2.13
  GitCommit:        7ad184331fa3e55e52b890ea95e65ba581ae3429
 runc:
  Version:          1.0.0-rc10
  GitCommit:        dc9208a3303feef5b3839f4323d9beb36df0a9dd
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

Hi @ovitente thank you for reporting an issue. I could not reproduce that issue event if I cut internet connection. Do you still facing with it?
How did you install via go get or brew?

Hi @mstrYoda

Installed via git clone + go build
Just checked current local build where i've had this issue, and cannot reproduce it now.
Cloned this repo again into the new dir, compiled, and everything is fine. Hmm

Possible fix

Restart docker service in mac
Reboot computer

btw: thank you for this took, it simplifies my daily work)

Hi @mstrYoda

Installed via git clone + go build
Just checked current local build where I've had this issue, and can not reproduce it now.
Cloned this repo again into the new dir, compiled, and everything is fine. Hmm
Unfortunately I haven't noticed when it was fixed.

Possible fix

Restart docker service in mac
Reboot computer

btw: thank you for this took, it simplifies my daily work)

Thank you for your time. I'm glad it helps you :)


mouse cursor lose focused when typing in the text fields

As titled, is it just me?
Thanks!

Hi @wey-gu ,
please, can you be more specific? Can you provide a screenshot and/or video of the issue manifesting?

Sorry, I should have done so, and interestingly, I cannot reproduce it now.
Thanks a lot! @ondras


update owners of OpenYurt

According to all maintainers's discussion a while ago, the detail info you can find in the issue #68 , we added inactive members and How inactivity is measured contents in community-membership file. then the owners of OpenYurt have some changes as following:

move the following maintainers into Emeritus Members list, express our deep gratitude for their significant contributions to OpenYurt.

SataQiu
charleszheng44
yixingjia
qclc
Peeknut
DrmagicE
kadisi
wawlian
Andy shi

Delighted to elevate four members from diverse organizations to the status of maintainers. A warm welcome to our new maintainers!

Name
GitHub ID
Affiliation

Chenglong Wang
luckymrwang
Inspur

Zhen Zhao
JameKeal
Sangfor

HanQi Li
LavenderQAQ
Tongji University

BingChang Tang
zyjhtangtang
Alibaba

we also update approvers for each repos and reviewers of OpenYurt. the detail info you can check the OWNERS.md file.

Fixes #68

@gnunu PTAL

/LGTM

/approve


Decrease log level for disruptive actions to "warn"

See also:
https://github.com/SpiderLabs/ModSecurity-nginx/issues/21#issuecomment-264398667

Merged! thanks!


loosen perms on bolt mq

the bolt mq file should only be used for local dev and isn't necessarily
sensitive, don't think 0655 restriction was necessary and the data isn't
likely all that sensitive anyway.
see https://github.com/fnproject/fn/issues/404#issuecomment-377570626

let's say this does:
closes https://github.com/fnproject/fn/issues/779
and if it comes back up then i lied :)


Safari issue when ssr is false

Describe the bug
In a simple SvelteKit skeleton project with a single library added and where SSR is disabled, Safari fails with Unhandled Promise Rejection: ReferenceError: Cannot access uninitialized variable.. Other browsers tested had no issues.
Reproduction
Please refer to https://github.com/cge-taal/sveltekit-safari-ssr-example and run with npm run dev.
In src/routes/+layout.ts, when export const ssr = false is set, Safari fails with Unhandled Promise Rejection: ReferenceError: Cannot access uninitialized variable.
Logs
No response
System Info
macOS, Safari 16.6

Severity
blocking all usage of SvelteKit
Additional Information
No response

Given

https://github.com/sveltejs/kit/issues/2889#issuecomment-1112512195
https://github.com/sveltejs/svelte/issues/5501

It seems to be an old bug in SvelteKit that resurfaced, related to top-level await support on Safari.
This is pretty important for us to have fixed and stable (tested) in a framework we hope to use for our products.
Any idea of a likely fix soon would be much appreciated.
Thanks for all the efforts.

Create a file called src/hooks.client.ts and add this line to it:
import { encodeBase58Address } from "@bitauth/libauth";

The code in src/hooks.client.ts is executed before the app starts. I guess the module you're using (@bitauth/libauth) requires some initialization before the node is rendered or mounted when you use safari.
For more information about hooks: https://kit.svelte.dev/docs/hooks

Thanks @MoonlightOffice , to be honest I did not try to find workarounds yet, because I think it should work by default, as it does in non-Safari browsers. Given the issue is related to top-level await, I did consider trying to wrap all interaction with @bitauth/libauth inside async methods.
I could try your workaround when there is a moment, but I do believe it would be most useful if a framework like SvelteKit makes workarounds unnecessary for users.

I can provide some extra feedback on your suggestion @MoonlightOffice.
It seems to work fine for the example repository I listed above, which is nice.
However, my real project is a pretty large monorepo (turbo) and I'm accessing @bitauth/libauth in a standard library package which I'm importing from my main application.
If I add all the same @bitauth/libauth imports I use in the package, to a src/hooks.client.ts file in my main application and also install @bitauth/libauth as a dependency for the main application, this workaround does not function anymore.

Wrapping @bitauth/libauth usage inside async functions is infeasible, as I need synchronous behaviour for some use-cases.

closing as duplicate of https://github.com/sveltejs/kit/issues/8529
It's currently caused by a bug in Safari that prevents the use of top-level awaits client-side. Libraries (and even SvelteKit has before) refactored to avoid the use of top-level awaits in the browser.


Support Segment Model Training in Shifu

If segment conditions are defined and woe binning is computed, then in training, segment model can be training in parallel with global model together.
This feature can be run in only LR & NN woe/zscore transform.

Done and details in wiki page: https://github.com/ShifuML/shifu/wiki/Segment-Expansion-for-New-Feature-Generation


crashes given a noisy image

If the foreground of the image intersects the center line and the width of the foreground area in a row is large enough, the program will crash because of a numpy-array-length-of-zero problem. We should be able to process any image (whether the result will be nice or not) without crashing.

The typical fix is to increase the --threshold argument, but that shouldn't be necessary.

Interestingly, the crash seems to happen only in --sequence mode.
This crashes
./kiss.py examples/couple_small.jpg -w /share --sequence 10 --in-parallel 1 -d 100 -t 10
Traceback (most recent call last):
  File "./kiss.py", line 267, in <module>
    main()
  File "./kiss.py", line 87, in main
    run_sequence(args)
  File "./kiss.py", line 151, in run_sequence
    for img, step in image_steps:
  File "/home/tad/Downloads/share/crash_kiss/crash_kiss/crash.py", line 48, in iter_crash
    yield center_crash(img.copy(), fg, bounds), max_depth  # deepest crash
  File "/home/tad/Downloads/share/crash_kiss/crash_kiss/crash.py", line 155, in center_crash
    lmov, rmov = mov_crash(crash_data, row_data)
  File "/home/tad/Downloads/share/crash_kiss/crash_kiss/crash.py", line 226, in mov_crash
    irow[center + fg_r_start + rlen + rmov:])
ValueError: could not broadcast input array from shape (449,3) into shape (0,3)

This doesn't
./kiss.py examples/couple_small.jpg -w /share -d 100 -t 10
Increasing the threshold also fixes the problem
./kiss.py examples/couple_small.jpg -w /share --sequence 10 --in-parallel 1 -d 100 -t 50

Revealing the foreground illustrates the problem. The crash happens when the foreground consumes the the entire selection area.
The image below was created with ./kiss.py examples/couple_small.jpg -w /share -d 100 -t 10 -e.


installation: extra dependencies for db drivers

Adds extra requirements groups that proxy dependencies for
Invenio-DB.  (addresses #3566)

Signed-off-by: Jiri Kuncar<EMAIL_ADDRESS>

"proxy dependencies in Invenio-DB"

proxy dependencies for Invenio-DB

"(closes #3565) (addresses #3566)"

Keep the closes (linking to the issue), remove the addresses (to an "unrelated" PR that simply "generated" that issue).

simply "generated" that issue

That's why I have added :wink:

LGTM but please rebase against latest master.


merge from arlac77/template-github-action,arlac77/template-kronos-component

README.md

docs(README): update from template

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1370   +/-   ##
=======================================
  Coverage   76.92%   76.92%
=======================================
  Files           1        1
  Lines          39       39
  Branches        1        1
=======================================
  Hits           30       30
  Misses          9        9

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 517513a...12829fe. Read the comment docs.