All imported packages seem to be tested as well

When I run mypy on the following code, I get no errors:
import numpy

However, when I run the pytest-mypy-plugin on:
- case: test
  main: |
    import numpy
    reveal_type(1)  # N: Revealed type is "Literal[1]?"

I get a lot of errors about things that apparently go wrong within pytest and numpy:
E   pytest_mypy_plugins.utils.TypecheckAssertionError: Invalid output:
E   Expected:
E     <45 (diff)
E   Actual:
E     ../../../../../../Users/nanneaben/opt/anaconda3/envs/tmp/lib/python3.8/site-packages/_pytest/_argcomplete:103: error: Cannot find implementation or library stub for module named "argcomplete" (diff)
E     ../../../../../../Users/nanneaben/opt/anaconda3/envs/tmp/lib/python3.8/site-packages/_pytest/_argcomplete:103: error: Cannot find implementation or library stub for module named "argcomplete.completers" (diff)
E     ../../../../../../Users/nanneaben/opt/anaconda3/envs/tmp/lib/python3.8/site-packages/_pytest/_code/code:33: error: Skipping analyzing "pluggy": found module but no type hints or library stubs (diff)
E     ../../../../../../Users/nanneaben/opt/anaconda3/envs/tmp/lib/python3.8/site-packages/_pytest/_code/code:369: error: Signature of "__getitem__" incompatible with supertype "list" (diff)
....
E     ../../../../../../Users/nanneaben/opt/anaconda3/envs/tmp/lib/python3.8/site-packages/numpy/__init__.pyi:995: error: Overloaded function signatures 22 and 51 overlap with incompatible return types (diff)
E     ../../../../../../Users/nanneaben/opt/anaconda3/envs/tmp/lib/python3.8/site-packages/numpy/__init__.pyi:997: error: Overloaded function signatures 23 and 51 overlap with incompatible return types (diff)
E     ../../../../../../Users/nanneaben/opt/anaconda3/envs/tmp/lib/python3.8/site-packages/numpy/__init__.pyi:999: error: Overloaded function signatures 24 and 51 overlap with incompatible return types (diff)
E     ../../../../../../Users/nanneaben/opt/anaconda3/envs/tmp/lib/python3.8/site-packages/numpy/__init__.pyi:1001: error: Overloaded function signatures 25 and 51 overlap with incompatible return types (diff)
E     ../../../../../../Users/nanneaben/opt/anaconda3/envs/tmp/lib/python3.8/site-packages/numpy/__init__.pyi:1003: error: Overloaded function signatures 26 and 51 overlap with incompatible return types (diff)
...

Is this the expected behaviour? Am I doing something wrong? Is it possible to make pytest-mypy-plugin match the behaviour of mypy here?
The above is a simplified example, of course. The real use-case here is that I'd like to use pytest-mypy-plugin to typecheck my package, but my package imports numpy (as well as some other packages that seem to show this behaviour), so currently all tests that import my package result in an error.
Thanks in advance!

See https://github.com/typeddjango/pytest-mypy-plugins/issues/34

Thanks, using pytest --mypy-only-local-stub indeed resolves the issue!


Provide an option to turn off generation of unchecked types

Problem
Feedback from @tsongas

Wondering if you've considered an option to turn off generation of
unchecked types, as a way to make sure we're following best practices
and not getting lazy? Kind of like strict mode in JS....

Idea for Solution
Perhaps some configuration in the Prisma schema
generator js {
  provider  = "prisma-client-js"
  unchecked = false
}

My team would really appreciate this as well, whenever someone misses a field or gets the syntax wrong, our data get casted into unchecked inputs and makes debugging the actual issue a big PITA

same, it would be really useful.
Since our team stays off unchecked queries and when the arguments for a mutation on nested relations are wrong, the error shows as unchecked and highlights the relation fields instead of the actual issues, making it a hassle to debug.

This would be very powerful for our team as well. @janpio, we would be glad to help if you pointed us in the right direction to do so.


Wrong subscription icon

The icon that looks like a filter here:

Should be a key, like in the azure portal:

This is by design.  The button lets you filter which subscriptions you would like to see in the tree view.

Right. But I’m this context the filter icon next to the header Logic Apps isn’t clear that you are filtering subscriptions. It looks like you are filtering Logic Apps.

We are filtering Logic Apps by subscription.


ENH: core-metrics upgrades

Note to reviewer, please "rebase and merge," I would like to preserve the two logical commits represented in this PR.

This PR adds support for running non-phylogenetic core-metrics. Note, this changes the existing behavior of core-metrics, now removing the tree input.
Fixes #112
This PR also adds support for returning the rarefied table in both core-metrics* methods.
Fixes #126

Thanks @jairideout! Your changes looked q2-awesome! I did an interactive rebase to clean up history - if this looks good to you feel free to rebase and merge! 🚲


Importing custom css

Is it possible to import custom css?
anywidget lets you set _css, but (as far as I can tell), setting that to a file doesn't seem to do anything?
I wanted to check if isn't supported, or isn't documented (and I'm possibly using it wrong..)

Currently, you can do styling of widgets like mentioned in this walkthrough guide https://widgetti.github.io/ipyreact/lab/?path=full_tutorial.ipynb like this
from IPython.display import HTML , display
from pathlib import Path
css_rules = Path('styles_orange.css').read_text()
display(HTML('<style>' + css_rules + '</style>'))

But yes, a _css option would be amazing, here was a conversation about that: https://github.com/widgetti/ipyreact/pull/3#issuecomment-1520490470

Indeed, this is not supported yet, we'd need to do something like in https://github.com/manzt/anywidget/blame/main/packages/anywidget/src/widget.js
Happy to guide someone through a PR and get this in, but I currently don't have the bandwidth to do it myself.

I do not think that should not be part of the same widget (I disagree with anywidget in that respect), you can also use https://solara.dev/api/style
But we could think about adding a simple Style widget in ipyreact.

If all you care about is bundling the .css of your widget alongside it, you can use the SASS plugin of ESBuild to shove it into the .js file (it works with regular CSS too). Just make sure to import the .css somewhere in your javascript code.
plugins: [sassPlugin({type: 'style'})]

Nice, thanks!

I wrote a bit of a hack that gets me to my first release on top of ipyreact.  I will work on a patch back to ipyreact when I get a chance (weeks out).
const BUCKAROO_STYLE_ID = "buckaroo-style-block";
export const injectBuckarooCSS = (css_text:string) => {
    if(document.getElementById(BUCKAROO_STYLE_ID) !== null) {
        return
    }

	const style = Object.assign(document.createElement("style"), {
		id: BUCKAROO_STYLE_ID,
		type: "text/css"
	});
	style.appendChild(document.createTextNode(css_text));
	document.head.appendChild(style);
}

class IPYBuckaroo(BuckarooWidget, ipyreact.ValueWidget):
    style_block =  traitlets.Unicode(
        Path("./../../packages/buckaroo-js-core/dist/style.css").read_text()).tag(sync=True)
    _esm="""
...

export default function({
    style_block
    }) {
      injectBuckarooCSS(style_block)
      """

My code was adapted from the anywidget css code, and it doesn't work HMR.  I only have a single style block for my widget and it's rather big.  I don't want it reinjected multiple times.  Ideally the CSS would work more like ipyreact.define_module.  For now this is expedient.


Ipv64.net - DNS challenge provider

What provider would you like to see added to NPM?

https://ipv64.net/
Have you checked if a certbot plugin exists?

https://pypi.org/project/ipv64/
or
https://github.com/XonaTheProtogen/certbot-dns-ipv64

Please IPv64.net
https://github.com/acmesh-official/acme.sh/blob/master/dnsapi/dns_ipv64.sh

I believe ipv64.net is supported under the DNSmulti "DNS provider" in Nginx Proxy Manager. Relevant documentation: https://go-acme.github.io/lego/dns/ipv64/

I tried it with DNS-Multi and it doesn´t work. It seems like it needs "go" as a dependency.

Please implement 🙏

Use DNS-Multi, it works fine.
You just need to enter the following:
dns_multi_provider=ipv64 IPV64_API_KEY=[KEY]

Dennis was able to fix a Problem regarding custom Domains on IPv64.
The multiDNS Provider is now working for DNS Challenge Wildcard Domains.
Give it a try

Dennis was able to fix a Problem regarding custom Domains on IPv64.
The multiDNS Provider is now working for DNS Challenge Wildcard Domains.
Give it a try

Worked for me but I think a direct integration would be nice. I needed to install some dependencies and that was try and error to get the right one.


releases/v2.0.0-rc.toml: import and clean up highlights

The first commit imports the highlights from https://github.com/containerd/containerd/releases/tag/v2.0.0-rc.5 to releases/v2.0.0-rc.toml.
The second commit cleans up them.
Fix #10601

These are generated using the labels, I updated https://github.com/containerd/containerd/pull/9096 to add the impact/changelog. We can also add new area/ labels to have them show up, but think we should keep that related to the larger functional areas rather than categories like bug fixes.


Not getting option to save cards using paymentflow

Platform

[ ] Web
[ ] iOS
[X] Android

Describe the bug
I have configured as you did it in your demo, Still i do not get the option to save cards for future payments. even i ran your demo on my android device.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Target sdk 32 and version: 4.1.0

Thanks for issue. In my hand, the option to save the card is visible. It is the part that shows "今後のrdlaboの支払いのために保存する" in Japanese.

For me, it's happening the same. It does not appear as an option.

It may be a customer token issue. Have you tried demo/angular?
https://github.com/capacitor-community/stripe/tree/master/demo/angular

This issue is closed due to lack of activity for a long time. Please start a new Issue if needed.


Delete task resources when creation fails

Indirectly closes #314 and mitigates a heap of different issues; requires manual testing on actual infrastructure. I'll script something later today to deplete my S3 quota.

I was doing the same for #330

Tested on an account with the S3 bucket quota already depleted, and works like a charm. 🎉

While this pull request doesn't address the specific error of #326, it looks like a really good solution for it and many other error–related issues.

“Fixtures”
Manual testing can be performed with the following “fixtures” after setting $region and the credential environment variables to sensible values.


View more information from results of model.evaluate()

When training my model, it's great to see the accuracy, arc, f1 score, and confusion matrix.
It would be really helpful to view more information about the confusion matrix. For example, I would like to look at the items that we're trained incorrectly to see if there's bad data present.
Data:
+--------------+-----------------+-------+
| target_label | predicted_label | count |
+--------------+-----------------+-------+
|           0            |               1             |     36    | <- what items are these?
|           1            |               0             |     58    | <- what items are these?
|           1            |               1              |    49    |
|           0            |              0             |   2230  |
+--------------+-----------------+-------+
Looking at another high-level library, fast.ai offers something where you can see the most incorrect items as a built-in method.
A great way to do this could be to have the model.evaluate return a results object, which could then have an explore method, similar to an SFrame.
results = model.evaluate(test_data)
results.explore() <- Opens a Turi create window where you can view the items with added columns: predicted_label and correct.
Ideally, we would be able to sort these columns by header.

Side note: If there's a way to do this now that maybe is not in the documentation or is not as easy to use, I would love to know about it.

This is hugely helpful, thank you. I definitely think this is something that should be added to the docs.


Use Save-DbaKbUpdate to download SQL Server 2016 SP3

Summarize Functionality
Let's start with a list of all links/files for SQL Server 2016 SP3 and SQL Server 2019 CU12 as shown by Get-DbaKbUpdate:

Difference is, that after the architecture "x64" there is no "_", but the language added with a "-", so this test fails if -All is not used:
https://github.com/sqlcollaborative/dbatools/blob/2d4ac0da63f1845caa76aaf65a535e93d8c197b7/functions/Save-DbaKbUpdate.ps1#L94-L96
So my question is: Do we what to change the code to be able to download the one needed file with the correct language? Or is it just an edge case to download a service pack these days as we only need CUs now?
We could add a new parameter -Language which will be included in the test if given. Problem: The user has to use Get-DbaKbUpdate to see that the filenames include a language. But I think adding this hint to to documentation of the parameter is sufficient.
Is there a command that is similiar or close to what you are looking for?
No
Technical Details
No response

I was able to manually download the SqlServer2016SP3-x64-enu_, but my vote would be that the code handle these cases.  Wondering if they also affect SPs for 2014, 2012, etc.


Add-PublicFolderClientPermission

On the Website Page, the documented permissions for the NonEditingAuthor Role for Add-PublicFolderClientPermission are: CreateItems, DeleteOwnedItems, FolderVisible, ReadItems.
However, in powershell if I connect a session to Exchange in the cloud and type  get-help Add-PublicFolderClientPermission -Detailed, then the documented permissions for the NonEditingAuthor Role for Add-PublicFolderClientPermission returned are: CreateItems, FolderVisible, ReadItems.
In test, the website version seems to reflect what happens, but that's not to say which is actually correct.

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

ID: ba50ae21-6a22-22be-58b5-7c145b394d2b
Version Independent ID: 1cad51a8-c5a6-24db-55b1-bd4a094552c6
Content: Add-PublicFolderClientPermission (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Add-PublicFolderClientPermission.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@273Kelvin, the topics on the web are always going to be the most up-to-date. There's nothing we can do about that.
Those topics are used for help at the command line, but they need to be converted into XML, combined into the Help-* .dll.XML files, checked back into code, and then (for the cloud) distributed to all the datacenters.
There's really nothing we can do about the lag.

Thanks Chris.
On a related matter regarding Add-PublicFolderClientPermission, the webpage documentation for the – AccessRight Parameter says You can specify individual folder permissions or roles, which are combinations of permissions. You can specify multiple permissions and roles separated by commas.
When I use,
$Mypermissions = “CreateItems, FolderVisible”
Add-PublicFolderClientPermission -Identity "\My Public Folder" -User Chris -AccessRights $Mypermissions
Powershell automatically works out that this is the same as the Contributor Role and adds the permissions as “Contributor”
Similarly if I use,
$Mypermissions = “ReadItems”
Add-PublicFolderClientPermission -Identity "\My Public Folder" -User Chris -AccessRights $Mypermissions
Powershell adds the permissions as “ReadItems”
But if I use
$Mypermissions = “ReadItems,CreateItems, FolderVisible”
Add-PublicFolderClientPermission -Identity "\My Public Folder" -User Chris -AccessRights $Mypermissions
Powershell fails, reporting that “Cannot process argument transformation on parameter 'AccessRights'. Cannot convert value "ReadItems CreateItems FolderVisible" to type
"Microsoft.Exchange.Management.StoreTasks.MailboxFolderAccessRight[]”
In experimentation, it seems that PowerShell is

allowing the use of any of the documented Roles, and
allowing a list of multiple individual Rights only where the list equates to one of the pre-defined Roles, and
allowing the use of any single Right (any of which I guess may be classified as a Custom Role)

What I want to set is a Custom Role including the rights “ReadItems, CreateItems, FolderVisible” but it doesn’t work. Ive checked using the O365 Website and can set a user to this Custom Role, so its not as if it’s an Exchange limitation.
Could you please advise if I’m perhaps using the wrong syntax for the multiple Rights?
Regards,
Neil Dubout
From: Chris Davis @.>
Sent: Saturday, 17 July 2021 2:11 AM
To: MicrosoftDocs/office-docs-powershell @.>
Cc: Neil Dubout @.>; Mention @.>
Subject: Re: [MicrosoftDocs/office-docs-powershell] Add-PublicFolderClientPermission (#8063)
@273Kelvinhttps://github.com/273Kelvin, the topics on the web are always going to be the most up-to-date. There's nothing we can do about that.
Those topics are used for help at the command line, but they need to be converted into XML, combined into the Help-* .dll.XML files, checked back into code, and then (for the cloud) distributed to all the datacenters.
There's really nothing we can do about the lag.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/office-docs-powershell/issues/8063#issuecomment-881560098, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AU3TOU62YHXSWDFVYDK3V4TTYBKYFANCNFSM5AOSBCPQ.

@273Kelvin, it looks like a simple formatting issue. Quotes around each value works:
[PS] C:\>$x = "ReadItems","CreateItems","FolderVisible"

[PS] C:\>$x

ReadItems
CreateItems
FolderVisible

Or, the parameter value without quotes also works: -AccessRights ReadItems,CreateItems,FolderVisible.

@273Kelvin Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[API, Log in] User creates new user with invalid email when logging in via 'Postman'

Environment: Postman
Reproducible: Always
Preconditions

Postman is opened.

Steps to reproduce

Select 'POST' method
Set 'https://car-api-project.azurewebsites.net/api/login'
Set request body with incorrect email (e.g.
'{
"name": "Yana",
"surname": "Rudyi",
"email": "rudayana@ry"
}')
Click on 'Send' button
Go to 'Car' DB
Open 'User' table
Check that the user is created

Actual result
New user created.
Expected result
400 Bad Request Status.
Labels to be added
"Bug" ("bug"), Priority ("pri: High")

Closed with #809 #810


Mount extra volume when necessary

All eksa commands run inside docker using the mr-tools image.
When creating management cluster, users only have to specify a spec file,
and the cli will mount tha path that contains the spec as a volume to be
used by the mr-tools image.
With the introduction of workload clusters, users can pass an extra flag (--kubeconfig)
that points to the kubeconfig of a management cluster.
If this kubeconfig is not in the same path as the spec then it is necessary to
mount an extra volume to make it availalbe for the cli.
Fix #581
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Keywords which can automatically close issues and at(@) mentions are not allowed in commit messages.
The list of commits with invalid commit messages:

e5c386d Mount extra volume when necessary

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mdsgabriel
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mdsgabriel]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Keywords which can automatically close issues and at(@) mentions are not allowed in commit messages.
The list of commits with invalid commit messages:

7678461 Mount extra volume when necessary

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jiayiwang7, mdsgabriel
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jiayiwang7,mdsgabriel]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm


 Please change your search criteria and try again.

Description
I have a eks1.17 cluster, which create by terraform-aws-eks cluster, and i want to destroy it, but failed.
only alert error message as below,
Error: Your query returned no results. Please change your search criteria and try again.
can't fixed.
⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Terraform: >= 0.12.0

Provider(s):  v0.13.1

Module: 1.11.3

Reproduction

Steps to reproduce the behavior:
terraform destroy

Code Snippet to Reproduce

Expected behavior

destroy the eks1.17 cluster
Actual behavior

can't destroy it
Terminal Output Screenshot(s)

Additional context

please provide more logs from terraform execution

module.eks.aws_iam_instance_profile.workers_launch_template[6]: Refreshing state... [id=usw2c2-dev-eks202103010813028
module.eks.aws_iam_instance_profile.workers_launch_template[13]: Refreshing state... [id=usw2c2-dev-eks20210906101257
data.aws_eks_cluster_auth.cluster: Refreshing state... [id=2021-11-08 02:33:39.828888755 +0000 UTC]
data.aws_eks_cluster.cluster: Refreshing state... [id=usw2c2-dev-eks]
module.eks.kubernetes_config_map.aws_auth[0]: Refreshing state... [id=kube-system/aws-auth]
Error: Your query returned no results. Please change your search criteria and try again.
Releasing state lock. This may take a few moments...
@daroga0002  no any other logs, except the above info

@daroga0002  thanks for your feedback, and i try your solution, but it also can't be destroy by the same error
module.eks.aws_iam_instance_profile.workers_launch_template[3]: Refreshing state... [id=usw2c2-dev-eks20210301081302864300000010]
module.eks.aws_iam_instance_profile.workers_launch_template[12]: Refreshing state... [id=usw2c2-dev-eks20210906101257411400000003]
Error: Your query returned no results. Please change your search criteria and try again.
Releasing state lock. This may take a few moments..._
and i continue to remove the module.eks.aws_iam_instance_profile.workers_launch_template[12] tf.state, then also can't be destroyed with the same error
I still can't find solution.

closing due to lack of information/reproduction


Introduce simplified hypertext controls for pagination and self-reference

Adding section on simplified hypertext controls for pagination and self-references
Updating HATEOAS section in Pagination to no longer use HAL
Removing other references to HAL from Hypermedia.md

👍

👍

:+1:


Greenkeeper/nsp 3.2.1

Coverage remained the same at 100.0% when pulling b51a13f9e5abfc27f35e128ab0bb2d9a6ff10949 on greenkeeper/nsp-3.2.1 into 5d89a33aad1f77a057c2bf0a9db4c0c247e7c4e7 on master.


[Enhancement] New kubeconfig defaults

This PR introduces some new CLI defaults when it comes to handling kubeconfigs:

k3d create cluster: does not do anything with kubeconfigs
k3d create cluster --update-default-kubeconfig: merges the new cluster's kubeconfig with the default kubeconfig
k3d create cluster --switch-context: as above + set current-context to new cluster's context
k3d get kubeconfig mycluster: writes mycluster's kubeconfig to $HOME/.k3d/kubeconfig-mycluster.yaml and returns the path
k3d get kubeconfig mycluster --merge-default-kubeconfig: merges with the default kubeconfig
k3d get kubeconfig mycluster --merge-default-kubeconfig --switch-context as above + switch current-context
k3d get kubeconfig mycluster --output FILE: write/merge to/with FILE

@RouxAntoine I'd like to have your opinion on this as well, since we discussed this once already :)

What do you think about --switch-context instead of --switch

Sure, that makes sense 👍

k3d get kubeconfig mycluster --merge-default-kubeconfig instead of k3d get kubeconfig mycluster --default-kubeconfig to be consistent with --update-default-kubeconfig

Definitely 👍

Does k3d create cluster could write to write to $HOME/.k3d/kubeconfig-mycluster.yaml ? like k3d get kubeconfig mycluster (i am not entirely sure for this last)

I don't see any big issues with this, but which benefit would it bring? You cannot pipe the output or anything right now.
So the file would be there, but you'll have to find it / make use of it yourself 🤔

yes you're right no benefit for the last :)

I think --switch-context should be the default of create cluster.  This is what most people would expect that after running create cluster the cluster is immediately usable by kubectl.
The idea that k3d get modifies the kube config is not very intuitive.  k3d update kubeconfig would make more sense.
Honestly I still think the CLI should be k3d NOUN VERB not k3d VERB NOUN because where every you put "switch context" or "update kubeconfig" is going to be a bit obscure and hard to find.

FWIW, I agree that k3d get modifying the kubeconfig would not be very intuitive. "get" should be idempotent.
I don't like the idea of k3d NOUN VERB as it breaks the convention from other k8s tools: kubectl get pod, kops get cluster, etc.

I think --switch-context should be the default of create cluster. This is what most people would expect that after running create cluster the cluster is immediately usable by kubectl.

So, what would you expect to happen then?
a) creates a new kubeconfig and updates the KUBECONFIG env var to use it
b) updates the default kubeconfig and switches the context

The idea that k3d get modifies the kube config is not very intuitive. k3d update kubeconfig would make more sense.

That's only the case if you specify --merge-default-kubeconfig, so that's not a problem here, right?

Honestly I still think the CLI should be k3d NOUN VERB not k3d VERB NOUN because where every you put "switch context" or "update kubeconfig" is going to be a bit obscure and hard to find.

While I agree, that NOUN VERB can streamline the syntax a little (i.e. everything related to kubeconfig goes under k3d kubeconfig FOO, etc.), I also agree with @weisjohn , that staying close to the other parts of the Kubernetes ecosystem makes a lot of sense (apart from the large change required to change the syntax.
Anyhow, I'd like to think about it a little more.
Here's how the syntax could look like:
$ k3d kubeconfig fetch <CLUSTER>
$ k3d kubeconfig merge <CLUSTER> <KUBECONFIG>
$ k3d cluster create <NAME>

It's something like
- create
	- cluster
	- node
- start
	- cluster
	- node
- stop
	- cluster
	- node
- delete
	- cluster
	- node
- get
	- cluster
	- node
	- kubeconfig

vs.
k3d
- cluster
	- create
	- start
	- stop
	- delete
	- list
- node
	- create
	- start
	- stop
	- delete
	- list
- kubeconfig
	- fetch
	- merge

Hello,
Just another argument about verb and noun order : verb + noun form could permit to apply verb to multiple noun like kubectl get pod,svc which is not possible with noun + verb

This PR
New Change: --update-default-kubeconfig and --switch-context are now true by default on k3d create cluster.
New Syntax Discussion
Will be continued outside this PR to not block progress further


[BUG]: Unable to update agent since upgrading to v3.218.0

What happened?
Hi, I have a script that automatically downloads the latest agent from the github release page, then prompts all of our deployment and build agents to update using the Azure DevOps API. I should have noticed sooner, but this basically stopped working after the agent updated to v3.218.0 on April 2nd, 2023.
Then, on May 7th, the latest version (v3.220.0) was retrieved and prompted to upgrade -- but this failed. Since then, every week it has been failing to upgrade the agent and so all of my agents are still sitting at v3.218.0.
I've included the full log of the update from May 21st, 2023.
This seems to be a duplicate of the question #4357, but I wanted to report it as a bug and provide further information.
According to the documentation, v3.x still supports Windows Server 2012+, so it's not clear what's wrong as of the 3.x release but something isn't working.
Versions
Azure Pipelines Agent 3.218.0 / Windows Server 2012R2
Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
Azure DevOps Server (Please specify exact version in the textbox below)
Azure DevOps Server Version (if applicable)
Azure DevOps Server 2019.1.2 (Dev17.M153.6)
Operation system
Windows Server 2012R2
Version controll system
TFVC
Relevant log output
[2023-05-22 05:00:08Z INFO RSAEncryptedFileKeyManager] Loading RSA key parameters from file D:\azagent\A1\.credentials_rsaparams
[2023-05-22 05:00:08Z INFO MessageListener] Message '38' received from session 'e0e8ccb5-0b86-4afa-8d37-71048de5b2a5'.
[2023-05-22 05:00:08Z INFO Agent] Refresh message received, kick-off selfupdate background process.
[2023-05-22 05:00:08Z INFO SelfUpdater] Version '3.220.2' of 'agent' package available in server.
[2023-05-22 05:00:08Z INFO SelfUpdater] Current running agent version is 3.218.0
[2023-05-22 05:00:08Z INFO SelfUpdater] An update is available.
[2023-05-22 05:00:08Z INFO Terminal] WRITE LINE: Agent update in progress, do not shutdown agent.
[2023-05-22 05:00:08Z INFO Terminal] WRITE LINE: Downloading 3.220.2 agent
[2023-05-22 05:00:08Z INFO HostContext] Well known directory 'Bin': 'D:\azagent\A1\bin'
[2023-05-22 05:00:08Z INFO HostContext] Well known directory 'Root': 'D:\azagent\A1'
[2023-05-22 05:00:08Z INFO HostContext] Well known directory 'Work': 'D:\azagent\A1\_work'
[2023-05-22 05:00:08Z INFO SelfUpdater] Attempt 1: save latest agent into D:\azagent\A1\_work\_update\agent1.zip.
[2023-05-22 05:00:08Z INFO SelfUpdater] Download agent: begin download
[2023-05-22 05:00:12Z INFO SelfUpdater] Download agent: finished download
[2023-05-22 05:00:12Z INFO SelfUpdater] Skipping checksum validation for offline agent update
[2023-05-22 05:01:11Z INFO SelfUpdater] Finished getting latest agent package at: D:\azagent\A1\_work\_update.
[2023-05-22 05:01:11Z ERR  SelfUpdater] Error occurred while calling WinVerifyTrust: System.InvalidOperationException: Structures containing SafeHandle fields are not allowed in this operation.
   at System.StubHelpers.HandleMarshaler.ConvertSafeHandleToNative(SafeHandle handle, CleanupWorkListElement& cleanupWorkList)
--- End of stack trace from previous location ---
   at System.Runtime.InteropServices.Marshal.StructureToPtr(Object structure, IntPtr ptr, Boolean fDeleteOld)
   at Microsoft.VisualStudio.Services.Agent.Listener.UnsafeNativeMethods.WinVerifyTrust(IntPtr hwnd, Guid pgActionID, WinTrustData pWVTData)
   at Microsoft.VisualStudio.Services.Agent.Listener.InstallerVerifier.VerifyFileAuthenticodeSignatureHelper(String filePath, Tracing trace) in D:\a\1\s\src\Agent.Listener\ValidationHelper\InstallerVerifier.cs:line 159
[2023-05-22 05:01:11Z ERR  SelfUpdater] System.InvalidOperationException: Structures containing SafeHandle fields are not allowed in this operation.
   at System.StubHelpers.HandleMarshaler.ConvertSafeHandleToNative(SafeHandle handle, CleanupWorkListElement& cleanupWorkList)
--- End of stack trace from previous location ---
   at System.Runtime.InteropServices.Marshal.StructureToPtr(Object structure, IntPtr ptr, Boolean fDeleteOld)
   at Microsoft.VisualStudio.Services.Agent.Listener.UnsafeNativeMethods.WinVerifyTrust(IntPtr hwnd, Guid pgActionID, WinTrustData pWVTData)
   at Microsoft.VisualStudio.Services.Agent.Listener.InstallerVerifier.VerifyFileAuthenticodeSignatureHelper(String filePath, Tracing trace) in D:\a\1\s\src\Agent.Listener\ValidationHelper\InstallerVerifier.cs:line 159
   at Microsoft.VisualStudio.Services.Agent.Listener.InstallerVerifier.VerifyFileSignedByMicrosoft(String filePath, Tracing trace, String expectedEKU) in D:\a\1\s\src\Agent.Listener\ValidationHelper\InstallerVerifier.cs:line 32
   at Microsoft.VisualStudio.Services.Agent.Listener.SelfUpdater.VerifyAgentAuthenticode(String agentFolderPath) in D:\a\1\s\src\Agent.Listener\SelfUpdater.cs:line 677
[2023-05-22 05:01:11Z INFO Agent] Ignore agent update exception. System.Exception: Authenticode validation of agent assemblies failed.
   at Microsoft.VisualStudio.Services.Agent.Listener.SelfUpdater.DownloadLatestAgent(CancellationToken token) in D:\a\1\s\src\Agent.Listener\SelfUpdater.cs:line 458
   at Microsoft.VisualStudio.Services.Agent.Listener.SelfUpdater.SelfUpdate(AgentRefreshMessage updateMessage, IJobDispatcher jobDispatcher, Boolean restartInteractiveAgent, CancellationToken token) in D:\a\1\s\src\Agent.Listener\SelfUpdater.cs:line 82
   at Microsoft.VisualStudio.Services.Agent.Listener.Agent.RunAsync(AgentSettings settings, Boolean runOnce) in D:\a\1\s\src\Agent.Listener\Agent.cs:line 346
[2023-05-22 05:01:11Z INFO Agent] Auto update task finished at backend, there is no available agent update needs to apply, continue message queue looping.
[2023-05-22 05:07:39Z WARN VisualStudioServices] Authentication failed with status code 401.
X-TFS-ProcessId: feb4ec40-59b5-495f-99d2-b3c8201f73f2
ActivityId: a39b68b2-a10e-443e-bcb4-831b7487904a
X-TFS-Session: 1f35dad2-5007-4f0e-9d7a-62f600d7318b
X-VSS-E2EID: 3400cd73-2c14-4fa0-8944-71ed9211f444
X-TFS-SoapException: %3C%3Fxml%20version%3D%221.0%22%20encoding%3D%22utf-8%22%3F%3E%3Csoap%3AEnvelope%20xmlns%3Asoap%3D%22http%3A%2F%2Fwww.w3.org%2F2003%2F05%2Fsoap-envelope%22%3E%3Csoap%3ABody%3E%3Csoap%3AFault%3E%3Csoap%3ACode%3E%3Csoap%3AValue%3Esoap%3AReceiver%3C%2Fsoap%3AValue%3E%3Csoap%3ASubcode%3E%3Csoap%3AValue%3EUnauthorizedRequestException%3C%2Fsoap%3AValue%3E%3C%2Fsoap%3ASubcode%3E%3C%2Fsoap%3ACode%3E%3Csoap%3AReason%3E%3Csoap%3AText%20xml%3Alang%3D%22en%22%3ETF400813%3A%20Resource%20not%20available%20for%20anonymous%20access.%20Client%20authentication%20required.%3C%2Fsoap%3AText%3E%3C%2Fsoap%3AReason%3E%3C%2Fsoap%3AFault%3E%3C%2Fsoap%3ABody%3E%3C%2Fsoap%3AEnvelope%3E
X-TFS-ServiceError: TF400813%3A%20Resource%20not%20available%20for%20anonymous%20access.%20Client%20authentication%20required.
WWW-Authenticate: Bearer, Basic realm="http://example.com:8080/tfs/", Negotiate, NTLM
X-Powered-By: ASP.NET
P3P: CP="CAO DSP COR ADMa DEV CONo TELo CUR PSA PSD TAI IVDo OUR SAMi BUS DEM NAV STA UNI COM INT PHY ONL FIN PUR LOC CNT"
Lfs-Authenticate: NTLM
X-Content-Type-Options: nosniff
Date: Mon, 22 May 2023 05:07:39 GMT

[2023-05-22 05:07:39Z INFO RSAEncryptedFileKeyManager] Loading RSA key parameters from file D:\azagent\A1\.credentials_rsaparams
[2023-05-22 05:07:39Z INFO VisualStudioServices] AAD Correlation ID for this token request: Unknown

Same issue here.

Hi there!
We are looking into the issue but since the Windows Server 2012 R2 end-of-life date (October 10, 2023) will come soon, we can't guarantee that we have time to release the fix. And it also means supporting this OS will be stopped for us as well. We highly recommend moving to Windows Server 2016 or newer.
Thanks!

I was able to workaround this issue on Win2012R2 by setting the DISABLE_AUTHENTICODE_VALIDATION knob in the agent .env file as follows and restarting the agent.
DISABLE_AUTHENTICODE_VALIDATION=true
This knob is used to: "Disables authenticode validation for agent package during self update. Set this to any non-empty value to disable."


Addressing Issue #31 by testing Elm Platform semver against supported range

Added semver package from npm to handle validation of Elm Platform semver during generator install process.
Added validateEnv function to check Elm Platform semver before files are installed and throw various exceptions if the version isn't within appropriate bounds.
Fixed a spelling error: 'Skiped installation' -> 'Skipped installation'
Adjusted inconsistent line spacing above writing function.

Cool!

Looks like it failed Travis, but that's just because my code emits an error if elm doesn't appear to be installed

Maybe you can add - npm install -g<EMAIL_ADDRESS>to travis so tests will come back green.

I'm not very experienced with Travis configuration, but the docs make it look like the npm install -g<EMAIL_ADDRESS>should belong to an install hook. Hope I did it right... ;)

Well, I wasn't expecting mocha to be missing...

Great! Travis is up and working again. Let me know what you think.
Anyway, hopefully I can sit back and make some use of this generator soon!

Awesome work, thanks!
Published v1.2.0

No problem. Glad I could save some trouble for others wanting to dive in and try this out!


Clean up demo script

Fairly self-explanatory.  It needs comments and documentation.

Fixed in 425bbc1 !


Auth Code Page - key is not showing only showing #

Describe the bug
https://www.berrycast.com/conversations/2c95c0df-f367-58af-bb64-5b786c9948dd

Design Error , Key is not displayed


x1090 AS TS (Katy) I would like the allow the slide lot numbers for xenium slides to allow the following format 4-7838D, on the section planning page, to allow for accurate recording

just for xenium slides
Digit, hyphen, 4 digits, letter"
Reminder for katy to put patch in for SGP119 slides
Small story

Core change on branch lot_format


Handle import dependencies in Importer

When importing some sets of resources, sometimes importing some resources will allow importing others. This can be handled by either:

Understanding the dependency graph and importing in order.
Retrying a full import as long as some progress is made. That is, if some resources were imported, but not all, try again, in case the successful ones now allow importing some of the previously failed ones.

For now I'm going to do 2 because it's simple and straightforward. If this turns out to be a problem, I'll revisit this and look into doing 1.


KDAB GammaRay package (WIP)

Tracking issue.

Has there been any progress on this, or is anything blocking it?

No, I haven't had time to work on this at all. I would be delighted if you would take it up!

I've picked this up again and should hopefully have some at least minimally working code up soon.

Please ping this issue if you want the nix expression for it, I need to clean it up a little before posting.

I have no idea why I didn't post this here at least; this is how I run gammaray
#With a recent nixpkgs.
#If you run into errors like `Cannot mix incompatible Qt library (version 0x50c06) with this library (version 0x50c07)`, try a --pure shell
{ pkgs ? import <nixpkgs> {}
, fetchFromGitHub ? pkgs.fetchFromGitHub
, needsGLFix ? false #I needed this at some point but it seems fine now?
}:
let

  #Not sure why i need this, probably old nixpkgs thing - i need to parametrize this over qt versions
  nixGL = (import (fetchFromGitHub {
    owner = "guibou";
    repo = "nixGL";
    rev = "04a6b0833fbb46a0f7e83ab477599c5f3eb60564";
    sha256 = "0z1zafkb02sxng91nsx0gclc7n7sv3d5f23gp80s3mc07p22m1k5";
    }) { }).nixGLIntel;

  gammaray' = { mkDerivation, lib, fetchFromGitHub, cmake, qtbase }: mkDerivation {
    name = "Gammaray";
    src = fetchFromGitHub {
      owner = "KDAB";
      repo = "GammaRay";
      rev = "4245ead89ff7cdc613f69d1d6a9192a5b054695e"; #outdated now probably
      sha256 = "1wk6lsfflz00is33kncjh62c8w7c3gzj8898kakh58m5wbid9c9v";
      };
    buildInputs = [ qtbase ];
    nativeBuildInputs = [ cmake ];
    };

  gammaray = pkgs.libsForQt5.callPackage gammaray' {}; #TODO clarify docs
in {
  inherit gammaray nixGL pkgs;
  integrated =
    if needsGLFix
      then
        pkgs.runCommand "gammaray" { buildInputs = [ pkgs.makeWrapper ]; } ''
          makeWrapper ${nixGL}/bin/nixGLIntel $out/bin/gammaray \
            --add-flags "${gammaray}/bin/gammaray"
          ''
      else
        gammaray;
  }

Actually the only thing I seem to be able to get it to do right now is crash applications, not sure whats up with that...

The short story seems to be that it's quite sensitive to having the right version of Qt.

I need to get gamma ray working. I'm going to proceed with what you posted above (or please post anything newer you have) and if I will post my own progress here, and perhaps a PR if it works well.

IIRC it should work, but you have to be careful about the Qt versions of things matching up.

Thank you for responding. I was able to run a bare GammaRay GUI but then learnt that using it with PyQt apps (what I'm writing now) isn't supported so I'm not going to do anything about this.

It might not be supported but does it actually not work? :(
I used it with some haskell stuff I wrote, but the internals of that probabyl work very different from the PyQt stuff.

I looked again after your comment and now I found github.com/KDAB/GammaRay/issues/515 which says at least 1 person have successfully used GammaRay with Python Qt apps. However, for me, when I launch Gamma ray, it shows other Qt applications but not my (running) PyQt application. I then try to "launch" app by putting Python in the executable field and setting working directory and args to directory of the python file and the python file itself. At this point the Launch button is greyed out. I didn't investigate further because I read about it not being supported (until your nuanced point made me look again today).
Thank you for trying to help.

The most likely issue is a Qt version mismatch.

Hi @asdf8dfafjk
In the end, were you able to use GammaRay with Python Qt apps? I'm facing a similar problem. I'm using PySide2 5.15.2 and I build GammaRay with Qt 5.15.2 as well. I'm facing the same issue, my app is not in the list of Qt apps and launch button is greyed out using python executable and the right args.

Closed by https://github.com/NixOS/nixpkgs/pull/221141 & https://github.com/NixOS/nixpkgs/pull/253353, I think.


Skip python 3.10

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@dhirschfeld  @h-vetinari How to skip the Python 3.10 build? I added the ray dependency to autogluon.core as suggested by @Innixma at https://github.com/autogluon/autogluon/issues/612#issuecomment-1407229851. Since ray does not support osx, I had to remove noarch from the recipe. I then added the following to skip Python 3.10. Although the recipe passes all checks, the package is actually not being built and not released to conda-forge. If I don't add the follow line, the recipe fails to build because its dependency autogluon.common requires Python <3.10.
skip: True # [py>=310]

conda_build.exceptions.DependencyNeedsBuildingError: Unsatisfiable dependencies for platform linux-64: {MatchSpec("python[version='>=3.7,<3.10']"), MatchSpec("autogluon.common==0.6.2=pyhd8ed1ab_0")}

I did something similar for the autogluon.vision recipe, and it was able to build the package for Python 3.8 and 3.9.  I am not sure why autogluon.core did not work the same way.

You need to rerender the recipe after changes like removing noarch: python, because the respective CI jobs need to be generated.
@conda-forge-admin, please rerender

note that, due to the presence of https://github.com/conda-forge/autogluon.core-feedstock/pull/6/commits/0b64c9163ce86d282c844349ebe293179ea93663, this is not correct yet (i.e. there are also builds for 3.10) - my intention here was to show the principle.
I'd suggest you git reset --hard b0221723, force-push this branch, and then rerender

@conda-forge-admin, please rerender

note that, due to the presence of 0b64c91, this is not correct yet (i.e. there are also builds for 3.10) - my intention here was to show the principle.
 Thank you very much.


Revert "Refactor logic for pre-processing local upload paths, address…

… symlink bugs (#27960)"
This reverts commit 26f2c9628a001af0ed0a2b41a71a43551bde5ef0.
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

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

General Guidelines and Best Practices

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

Testing Guidelines

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

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


Linux init script recognizes systemd ether dev naming conventions for peripheral/embedded hardware nic devices (As implemented in github.com:systemd/systemd/src/udev/udev-builtin-net_id.c)

Device naming conventions for peripheral/embedded hardware nic devices, as implemented in github.com:systemd/systemd/.../src/udev/udev-builtin-net_id.c


🧑💻: Add GitHub Desktop Contribution Instructions

:red_circle: Title : Add GitHub Desktop Contribution Instructions
:red_circle: Enhancement Aim :
:red_circle: Changes :
I am interested in contributing to the project by adding detailed instructions on how to contribute using GitHub Desktop.
I believe that by providing clear and comprehensive guidance, we can encourage more contributors to participate in the project, especially those who prefer using GitHub Desktop for their contributions.
Could you please assign me the issue related to adding GitHub Desktop contribution instructions under gssoc24? I am excited about the opportunity to contribute to the project and help streamline the contribution process for others.
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Pradnya Gaitonde
What is your participant role? (Mention the Open Source Program name. Eg. GSOC, GSSOC, SSOC, JWOC, etc.) GSSOC24

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

not required.


ID Token not verified from cookie causes RateLimits

Describe the bug
Hi, i'm trying to get the OIDC setup running properly but it seems that there is an issue with the tokens in cookie storage.
Right now the order which cookie is verifies is hardcoded to access_token, id_token.
Using auth0 the access_token can not be used because it contains a different (non-valid) schema but the id_token could be used but only the first one is taken into account.
Nevertheless also when the correct token would be read from the cookies is could not get verified in the userinfo code because https://github.com/weaveworks/weave-gitops/blob/main/pkg/server/auth/server.go#L379 only the HMAC verifier is present.
This result in a call to the userinfo endpoint of the oidc provider everytime.
In general this is going to work. But its creating a lot of unnecessary requests against the oidc-provider and in the case of Auth0 the rate limit is kicking in and make the UI unusable.
So i guess the best would be to add the correct verifiers to the userinfo code so the token from the cookies can be used.
Environment

Weave-Gitops Version: 0.10.1

To Reproduce

Configure oidc with auth0
Click around in the ui or call .../oauth2/userinfo multiple times
You will se 429 in logs and you will be logged out of the ui

Expected behavior
IdToken from cookie is used
Actual Behavior
Everytime userinfo is called a request against the oidc-provider is made
Additional Context (screenshots, logs, etc)

Thank you again @eloo for reporting this issue!
We'll take a look at it shortly.


Demo doesn't work

when I open in chrome,there are a mistake:
Uncaught TypeError: Cannot read property 'addControls' of undefined
at demo.js:61
How to fix it?Thanks.

Hi @fwk1010,
current develop branch, the demo/index.html?
It does work for me. Can you provide some more details what exactly you're doing?

well,there are three examples in demo/index.html.I want to use Editing A Polygon and Editing A Layer Group,but they do not work.
@codeofsumit
Thank you for your prompt reply.

they do work fine for me - are there any errors in the console?

@fwk1010 please run npm run build in the plugin folder and try again.

Please reopen if the issue persists.


-fconstant-string-class error...

Some time after 2018-03-28 GNUstep Base started failing during "configure" with the error message:

Your compiler does not appear to implement the -fconstant-string-class option needed for support of strings.

If I use the "1.9" branch it's fine. Problem only occurs in the main branch.

run into the same problem.     git clone https://github.com/gnustep/libobjc2 --branch 1.9
is really what made it work. Debian9 with clang7 here.

There was a clang bug that prevented this from working with -fobjc-runtime=gnustep-2.0.  This is now fixed in svn head and there's a back-port of the fix to 7.0 under review for the FreeBSD package.


HA Core version 2022.11b1 breaks the integration

The problem
Hi all,
Looks like 2022.11b1 breaks the integration. :(
Got some warnings to show:
Invalid config
The following integrations and platforms could not be set up:

nest_protect.select ([Show logs](http://homeassistant.local:8123/config/logs?filter=nest_protect.select))
Please check your config and [logs](http://homeassistant.local:8123/config/logs).

and
Logger: homeassistant.config_entries
Source: helpers/entity_component.py:176
First occurred: 3:51:10 PM (5 occurrences)
Last logged: 3:52:42 PM

Error unloading entry Nest Protect for select
Error unloading entry Nest Protect for binary_sensor
Error unloading entry Nest Protect for sensor
Error unloading entry Nest Protect for switch
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 498, in async_unload
    result = await component.async_unload_entry(hass, self)
  File "/usr/src/homeassistant/homeassistant/components/select/__init__.py", line 68, in async_unload_entry
    return await component.async_unload_entry(entry)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 176, in async_unload_entry
    raise ValueError("Config entry was never loaded!")
ValueError: Config entry was never loaded!

and
Logger: homeassistant.setup
Source: setup.py:320
First occurred: 3:45:22 PM (1 occurrences)
Last logged: 3:45:22 PM

Unable to prepare setup for platform nest_protect.select: Platform not found (Exception importing custom_components.nest_protect.select).

and

This error originated from a custom integration.

Logger: homeassistant.loader
Source: custom_components/nest_protect/select.py:23
Integration: Nest Protect (documentation, issues)
First occurred: 3:45:21 PM (1 occurrences)
Last logged: 3:45:21 PM

Unexpected exception importing platform custom_components.nest_protect.select
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/loader.py", line 728, in get_platform
    cache[full_name] = self._import_platform(platform_name)
  File "/usr/src/homeassistant/homeassistant/loader.py", line 745, in _import_platform
    return importlib.import_module(f"{self.pkg_path}.{platform_name}")
  File "/usr/local/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/config/custom_components/nest_protect/select.py", line 23, in <module>
    class NestProtectSelectDescription(
  File "/usr/local/lib/python3.10/dataclasses.py", line 1185, in dataclass
    return wrap(cls)
  File "/usr/local/lib/python3.10/dataclasses.py", line 1176, in wrap
    return _process_class(cls, init, repr, eq, order, unsafe_hash,
  File "/usr/local/lib/python3.10/dataclasses.py", line 1025, in _process_class
    _init_fn(all_init_fields,
  File "/usr/local/lib/python3.10/dataclasses.py", line 546, in _init_fn
    raise TypeError(f'non-default argument {f.name!r} '
TypeError: non-default argument 'key' follows default argument

What version of this integration (ha-nest-protect) has the issue?
0.0.38
What version of Home Assistant Core has the issue?
2022.11.b1
Device / Model
Nest Protect Battery
Diagnostics information
No response
Home Assistant log
Logs
```
Copy/paste any log here, between the starting and ending backticks (`)
```

Additional information
No response

see bug 106

see #106

Awesome I'm glad it's already known cheers!


Add: provide configurable parameters for connecting to remote k8s.(#37)

What do these changes do?

Provide configurable parameters for connecting to remote k8s, which strongly relies on the kube_config.new_client_from_config function.
eg:
import graphscope

k8s_client_config = {"config_file": "./kube_config", "context": None, "persist_config": True}

sess = graphscope.session(k8s_client_config=k8s_client_config)

Related issue number

Fixes #37

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @howie6879 You need to sign the CLA first then we could get this one merged.

Hi @howie6879 You need to sign the CLA first then we could get this one merged.

Done!

Codecov Report

Merging #40 (ba5dfc8) into main (dd2a58a) will decrease coverage by 10.97%.
The diff coverage is 66.66%.

@@             Coverage Diff             @@
##             main      #40       +/-   ##
===========================================
- Coverage   77.26%   66.28%   -10.98%
===========================================
  Files          48       47        -1
  Lines        4336     4179      -157
===========================================
- Hits         3350     2770      -580
- Misses        986     1409      +423

Impacted Files
Coverage Δ

python/graphscope/client/session.py
64.62% <66.66%> (-8.39%)
:arrow_down:

python/graphscope/interactive/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/interactive/query.py
0.00% <0.00%> (-94.24%)
:arrow_down:

python/graphscope/deploy/kubernetes/cluster.py
19.86% <0.00%> (-51.52%)
:arrow_down:

python/graphscope/deploy/kubernetes/utils.py
12.14% <0.00%> (-48.60%)
:arrow_down:

...n/graphscope/deploy/kubernetes/resource_builder.py
29.03% <0.00%> (-32.56%)
:arrow_down:

python/graphscope/analytical/app/triangles.py
80.00% <0.00%> (-20.00%)
:arrow_down:

python/graphscope/client/rpc.py
66.66% <0.00%> (-17.04%)
:arrow_down:

python/graphscope/framework/vineyard_object.py
83.33% <0.00%> (-4.17%)
:arrow_down:

... and 13 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 dd2a58a...ba5dfc8. Read the comment docs.

Merged as the failed of app-test when experiment is ON isn't caused by this PR and kubernetes related things works well.
Thank you @howie6879 ！


Disable all map's listeners in Fragment

Issue Type
[X] Question
Version of osmdroid the issue relates to:
6.0.0 - Android API 21 (min), API 23 (target)

Hello everyone !
I need some help regarding whether or not something is possible. (And, if someone has a hint or the solution, feel free to help ;) )
For my project, I need to have like a minimap on the screen, above everything, that I can move/destroy with my finger. Like the video in Youtube app, sorta.
Actually, I manage to create a small fragment inside a FrameLayout that hover above the layout and display a "MapFragment" without UI (which I configure, so, that's the right behavior).
However, I didn't manage to add the proper "drag and drop" listener or touch listener. I have the feeling that the touch listener of the map (the one used to move it) get the action, but not the listener of the FrameLayout.
EDIT : I can move the FrameLayout but only when the fragment is not created inside. When the map is instantiated, its listener is taking over.
Is there a way to disable every touch listener ?
Thank you very much,
Kofee

would it be possible to have a transparent view on top of the map? then you can intercept all touch events

Actually that's what I did yesterday evening.
I created a parent FrameLayout with two FrameLayout inside, one that welcome the map and the other to intercept touch event that I dispatch to parent.
It works well!
However it disable the zoom button, which I would have like to keep. That's why I'm asking this since in the GMaps Map View, there's something to disable UI listeners.

you can use your own zoom buttons that are on top of the transparent view

Does that mean it doesn't have any methods to "kill" the UI controls ?
I will try that, thank you.

yeah i think the answer is no at the moment, feel free to open a pr if you want to implement it.

Thank you for the opportunity but I'm don't have enough skills in Android and Java to contribute to an API like this yet.
And thank you for your time !

Thank you for the opportunity but I'm don't have enough skills in Android and Java to contribute to an API like this yet.
And thank you for your time !


[HOWTO] Keep treemacs open when pressing esc

What I want to achieve
I'd like to keep treemacs open when I press escape (evil-force-normal-state), so that treemacs is only toggled when I explicitly toggle it via +treemacs/toggle.
Is there an easy/good way to do this?
System information

emacs   version    26.2
        features   NOTIFY ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS NS MODULES THREADS
        build      Apr 12, 2019
        buildopts  (--with-ns '--enable-locallisppath=/Library/Application Support/Emacs/${version}/site-lisp:/Library/Application Support/Emacs/site-lisp' --with-modules)
        windowsys  ns
        daemonp    server-running
doom    version    2.0.9
        build      HEAD -> develop, origin/develop, origin/HEAD 23ef6e42c 2019-10-27 11:40:47 -0400
system  type       darwin
        config     x86_64-apple-darwin18.2.0
        shell      /bin/bash
        uname      Darwin 19.0.0 Darwin Kernel Version 19.0.0: Wed Sep 25 20:18:50 PDT 2019; root:xnu-6153.11.26~2/RELEASE_X86_64 x86_64
        path       (/usr/local/opt/ruby/bin/ ~/.composer/vendor/bin/ ~/Code/platform/ ~/.emacs.d/bin/ ~/bin/ /usr/local/bin/ /usr/bin/ /bin/ /usr/sbin/ /sbin/ /Applications/Emacs.app/Contents/MacOS/bin-x86_64-10_14/ /Applications/Emacs.app/Contents/MacOS/libexec-x86_64-10_14/ /Applications/Emacs.app/Contents/MacOS/libexec/)
config  envfile    envvar-file
        elc-files  0
        modules    (:completion (company +auto +childframe) (ivy +fuzzy) tab-complete :ui doom doom-dashboard doom-quit ophints workspaces hl-todo indent-guides modeline nav-flash treemacs (popup +all +defaults) vc-gutter vi-tilde-fringe window-select better-company :editor (evil +everywhere) fold file-templates snippets multiple-cursors rotate-text :emacs dired electric vc :tools (eval +overlay) docker (lookup +docsets) editorconfig flycheck flyspell lsp magit :lang data emacs-lisp go (java +lsp) (javascript +lsp) markdown (org +attach +babel +capture +export +present +protocol) (php +lsp) python rest ruby (scala +lsp) (sh +fish) web :config (default +bindings +smartparens))
        packages   (n/a)
        elpa       (n/a)

Unless you created a new popup rules for treemacs without a :quit nil property, this is already the default behavior. I cannot reproduce it otherwise.

@hlissner can i first of all say you are amazing for this project? We all literally spend hours and hours every day benefiting from your work here, and you responded within 3 mintues! So yeah thanks..
Anyway, you are right, with a fresh install it seems this is the behavior, so I'll have to see what I have conflicting with it.

On second thought, I've been able to reproduce the issue with a vanilla doom installation (with the only addition of treemacs).
Steps to reproduce:

treemacs toggle
focus back in editor buffer
create a split (either vertical or horizontal)
press escape a few times

Doing this will close treemacs, while I would expect treemacs to remain open.
I believe the issue to have something to do with the balance windows function, but I don't know for sure

This should be fixed in 4fe24df. Let me know if that isn't the case and I'll reopen it. Thanks for letting me know!

Better, thanks!


Implement authentication for coordination path and for streams path(Zookeeper)

Also, add new options:

zk.security.login
zk.security.password

Just for all prefix (/tts or other specified) and children.


Automator header is missing from repo

Hello,
Could please add Automator class in the repo
Thanks

Hello,
The automator header is used in the unit test program only. You may install it by using qpm
cd tests/asyncfutureunittests
qpm install


V7 - Setting disableSort on a column does not work.

V7
Describe the bug
Setting disableSort on a column does not work.
In https://github.com/tannerlinsley/react-table/blob/master/src/plugin-hooks/useSortBy.js#L167
    const canSort = accessor
      ? getFirstDefined(
          columnDisableSorting,
          disableSorting === true ? false : undefined,
          true
        )
      : false

If columnDisableSorting is true, canSort gets set to true. canSort should be set to false if columnDisableSorting is true.
I think that should be enough information. However, if you need reproduction steps or a codesandbox let me know.
Thanks for the great work on this project!

A fix for this has been merged to master. Thanks!


Small updates to contributing and include docs

I noticed that we were pulling in the # Contributing title markdown in our {include} statement for the contributing guide. I decided to try using the start-after argument to see if we can remove that content from the MyST site. In the process I also added an easier-to-find link to the reference docs for both {embed} and {include}.
Question: Am I using {include} wrong?
I used the start-after argument similar to how it's used in Sphinx, and assumed it'd work since it is documented. However it's not cutting off the top part of the included page. I wonder if this is a bug or if I'm using it incorrectly. @rowanc1 or @fwkoch do you have guidance?

That is currently only implemented for literal includes! Would welcome an issue to expand support!
Also thinking on what the syntax is for notebooks for a "start after" when it is json...?!


Wrangler fails to publish a folder to pages

Fails with this error, btw, it was working fine before.
Select an account from below:
❯ ****
✘ [ERROR] fetch failed
If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new.
🌎  Uploading... (10/36)

Likely a dupe of #960 , but we'll take a look, thank you!

@GregBrimble I don't think so.
I just found that my issue only happens in slow networks.
i can publish using 4g but not able on ADSL.

I'm kind of sure that it's a network issue
4G upload speed is >= 2 Mbps
ADSL ~= 0.6 Mbps
I repeated the process multiple times on both networks and always get the same results, works on 4G and fails on ADSL.
Thank you

Ah, okay, we'll add retries and a better error message for client disconnections. That'll hopefully help in most cases.

@bardouni my upload speed is around 16 Mbps, but I can not upload files. I'm trying to upload almost 260 files the size of my files is between 4K and 500K.

Is there any way how find out which files fail to fetch? Is there an option to show more debug information?

Hey @martinfojtik
Can you perhaps try<EMAIL_ADDRESS>We rewrote much of this to use bulk uploads instead of uploading a file at a time and I suppose that will affect what you're seeing. Let me know if it fixes it?
Cheers.

This was fixed for me in the latest versions.


openwebrtc demo unavailable now

http://demo.openwebrtc.org:38080/
This demo can't use now, please check
PermissionDeniedError: Only secure origins are allowed (see: https://goo.gl/Y0ZkNV).

This is a known issue. We will be redeploying on HTTPS soon

OK, thks!


Reject Range-Request if those ranges are not strictly in ascending order

Request that ranges are not strictly in ascending order, indicates either a broken client or a deliberate denial-of-service attack.

Yes. This change has been reverted. It is not RFC 9110 compliant.

Konstantin has provided examples of requests out there with these kind of ranges. Rejecting them provides no value to Tomcat, so the change was reverted.


Running extensions editor: Indicate that an extension is running in the web worker extension host

I was migrating an extension to a web extension and was running it on the desktop with the a launch config to run the extension as a web extension (see the web extension guide)
When opening the Running extensions editor I would have expected to see some indication in the editor that the extension is running in the web worker extension host, but the extension showed up as the other extensions. There should be an indication in which extension host this extension is running.

This is what will be rendered:

Not sure about the icon. //cc @misolori

A globe might be better suited:


cdata with url not decode

if you want to provide a example, and the example value is a url, story will start.
if u use the raw url in xml,  can let xml invaild.
        /// <summary>
        /// url
        /// </summary>
        /// <example>https://test.com/a?b=1&c=2</example>
        public string Url { get; set; }

cause the xml ask the & must escape to &amp , coincidentally, in most cases, the url always contains &.
so u must use the cdata to auto escape like this
        /// <summary>
        /// url
        /// </summary>
        /// <example><![CDATA[https://test.com/a?b=1&c=2]]></example>
        public string Url { get; set; }

and then, the ui can't decode &amp back to & , so the url is invaild too.
is there a way to show right url in example ?

well, finally I found a solution to this problem
it it caused by https://github.com/domaindrivendev/Swashbuckle.AspNetCore/blob/a66864f65bac06ad326b4f2ec8fddcc91012a4e8/src/Swashbuckle.AspNetCore.SwaggerGen/XmlComments/XmlCommentsSchemaFilter.cs#L52
the innerXml will get the xml string , and means never unescaped.
so we nedd innerText, but, XPathNavigator does not has the innerText, I found a ToString() method in doc https://docs.microsoft.com/en-us/dotnet/api/system.xml.xpath.xpathnavigator.tostring?view=net-5.0
and it said it is eq to innerText.
so, will u fix it by change all innerXml to ToString() ?

@rozbo - could you create a PR for this change?

Yes， I will do it ASAP

#2184
all test are passed


can you opensource the benchmark code

I can not download the benchmark image, so can you opensource the benchmark code or dockerifle?

@DragonFive What issues did you encounter when pulling the image? We're not considering  open source the benchmark code.


account for multiple possible grub.cfg paths

Account for different grub.txt paths on different distros

/hold. Need to verify parsing on grub2 file is same prior to merging


[ALLUXIO-3314][WIP]Reduce InodeDirectory memory usage

https://alluxio.atlassian.net/browse/ALLUXIO-3314

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/1027/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/1029/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/1030/
Test FAILed.

Merged build finished. Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/1032/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/1034/
Test PASSed.

@witgo Thanks for the improvement, had a minor comment.


HTML TextEdit transforms are slow and can cause timeout

Expected Behavior
Formatting should successfully complete in a reasonable amount of time.
 I found that sometimes the format completes as expected, but will cause the UI to become very sluggish.
Steps to Reproduce
test repo link

Format the Rzls_test_fail.razor file to reproduce the error
Format the Rzls_test_pass.razor file.

With commented section, will fail.
Delete commented section, will pass.

Added a stock asp razor file for control to check against MudBlazor components.

Context (Environment)

Linux, Ubuntu 22.04
NVIM v0.11.0-dev-1417+g487c48ec86
LazyVim V14

Detailed Description
Format timeout seems to happen more often than not. Setting timeout with vim.lsp.buf.format({timeout_ms=10000}) seems to decrease timeout messages. Limited testing seems to indicate that the size of the file has a direct impact on the time required to execute a format. Smaller files will always format in the expected time. Suspected that MudBlazor components may be the issue but the non-MudBlazor control file exhibits the same timeout behavior.

it actually completets in time on my machine, but i think that might just be a raw horsepower thing.
Profiling it shows that the diff generation bit is slow, we may need a better/faster algorithm here

faster, but i think there are more wins. can you try again with the req_log branch @RKWarren1964

Still get timeout error without setting a delay. It is much better though, I was able to consistently run vim.lsp.buf.format with a 3000ms delay. As far as horsepower, my box is built specifically for dev, Sys76 Thelio Ryzen 16 core, 64gb ram. Internet runs consistently at greater than 90mbs. Are there any traps I can setup to capture where I am seeing differences? Honestly not to worried about a 3 sec. delay on save, just happy to have a razor LSP!

ive just pushed a small few minor changes
I even tried writing it in rust, and calling out via ffi, but with long strings its just slow
Maybe i need to think of a better approach

Branch performance test (rudimentary)
@tris203 You probably have already done this, but just a check between your system vs mine. I added a vim.uv.now call before and after the lsp format call in my attach function.

main vs req_log in milliseconds.
tested against the rzls_test_fail.razor page
consecutive calls without reopening the file
AMD Ryzen 9 7950x, 64GiB

main
req_log

5.627
2.001

4.050
2.517

3.640
2.467

3.690
2.147

3.894
2.365

5.727
1.94

There is no question your refactoring is performing much better. The main branch absolutely hammers a couple of cpu cores. The req_log branch is much less intense and load balances across cores better. Granted, limited test, but...
I'll test the same on my work pc tomorrow.

I might have a solution that runs in 1ms. but it fails on the lsp side.
If you wanted to help debug why it doesn't like it in the c# side that would be awesome!
The relevant c# is here:
https://github.com/dotnet/razor/blob/main/src/Razor/src/Microsoft.CodeAnalysis.Razor.Workspaces/Formatting/Passes/FormattingContentValidationPass.cs#L26-L26
and i can provide the inputs if you are willing to have a look

@mparq might be able to help with his lsp middleware logger

@tris203 Catch me up, remove the line from the content?

If you delete the trailing blank line on line 509, so that the file is only 508 lines.
Then the formatting completes in 50ms @RKWarren1964

I can setup and take a look at it, might be a day or two. Do you have a test branch?

I can setup and take a look at it, might be a day or two. Do you have a test branch?

No in your reproduction repo. delete the last line from the Rzls_test_fail.razor file

I just pushed a update to the req_log branch @RKWarren1964
If you can test again with your timer that would be great

I just pushed a update to the req_log branch @RKWarren1964
If you can test again with your timer that would be great

3.097, 3.176, 2.445, 3.198, 3.097

I just pushed a update to the req_log branch @RKWarren1964
If you can test again with your timer that would be great

3.097, 3.176, 2.445, 3.198, 3.097

slower? what?
did you definitely pull the update?

I just pushed a update to the req_log branch @RKWarren1964
If you can test again with your timer that would be great

3.097, 3.176, 2.445, 3.198, 3.097

slower? what?
did you definitely pull the update?
 i am formatting in a few hundred ms

Afraid so, removed blank lines, double checked branch and removed some code I had added to adjust timeout based on line count.

Added the line back in, 2.5 ish. Also, I'm on my work computer behind, different hardware, more network security. I can hit it on my home pc again tonight to get a fair comparison to my previous numbers. All in all, should not be as far from your benchmark as I am in any case.

@tris203 I removed all the blank lines, formatted in 1.4. Subsequent format with only minor changes are in the 200 to 400ms range. I think you are probably there. I know I won't have any heartburn with a slightly longer first format on a large and unformatted file. You are on to something with the blank lines, for sure. Thanks a ton for the optimizations!

lets drive this branch for a few more days, and then i will merge


"Previous" button does nothing on first page

Describe the bug - Décrivez le bogue
The first page of the AIA questionnaire prints a "Previous" button. The "Previous" does not lead anywhere. Remove "Previous"
To Reproduce - Pour reproduire
Steps to reproduce the behavior - Étape pour reproduire le comportement:
Go to page 1 of the AIA Questionnaire
Click on 'Previous'
See error - nothing happens.
Expected behavior
Remove "Previous" to avoid clutter

This was included in a PR.  The first page removes the "Previous" button, and the last page removed the "Next" button. Works well in both languages.  Closing.

This was included in a PR.  The first page removes the "Previous" button, and the last page removed the "Next" button. Works well in both languages.  Closing.


Save image with URL link (Integration with Appwrite.io)

Describe your question
I am building a create/edit form for custom data type using flutter + appwrite. The field is displayed fine but I believe there could be an easier way to get the image file object, path, filename on selection. Below is my code to get the file object. Please let me know if there's a better way to get it.
FormBuilderImagePicker(
                  key: _imageFormKey,
                  name: 'photo',
                  decoration: const InputDecoration(labelText: "Pick Photo"),
                  maxImages: 1,
                  previewAutoSizeWidth: true,
                  fit: BoxFit.cover,
                  onSaved: (val) {
                    try {
                      selectedImg = val?.first;
                      // selectedImg = File(selectedImg.path);
                    } catch (e) {
                      logger.severe(e);
                    }
                  },
                ),

Another issue that I am still facing is to store the file (with filename to appwrite) and add image url to data object.
FormButtons(
                  formKey: _editTestformKey,
                  saveButtonLabel: 'Update',
                  resetButtonLabel: 'Reset',
                  saveAction: () async {
                    String? url;
                    if (selectedImg != null) {
                      logger.info("img: ${selectedImg?.path}");
                      url = await DataLayer.instance
                          .uploadImageToStorage(selectedImg!);
                      logger.info("url: $url");
                    }

                    Test test = Test(
                      name: _editTestformKey.currentState!.value['name'],
                      id: testData.id,
                      url: url,
                    );

                    await DataLayer.instance.updateDocumentById(
                        collectionId: DBCollection.testRef,
                        documentId: test.id,
                        data: test.toMap());
                  },
                ),

Code to upload to Appwrite
Future<String> uploadImageToStorage(XFile file) async {
    List<int> fileBytes = await file.readAsBytes();
    logger.info(fileBytes.length);
    logger.info(file.path);

    final uploadedImage = await _storage.createFile(
      bucketId: imagesBucket,
      fileId: ID.unique(),
      file: InputFile(
          path: file.path,
          bytes: fileBytes,
          filename: file.path
              .split('.')
              .last), //TODO: for mob use this instead- InputFile(path: file.path),
    );
    logger.info(uploadedImage.name);
    logger.info(imageUrl(uploadedImage.$id));
    return imageUrl(uploadedImage.$id);
  }

Data Document Screenshot

Storage screenshot

Although image is uploaded to appwrite but data document isn't updated with url. Can you please suggest if there's a better way to upload an image (with image name) & save its URL. Thanks

This issue question is to complex to resolve here. Please write again your text and separated the questions with more objective view. Then, you can add this question on Discussion section
Thanks


Allow references in documentation

Feature
I've only been using dbt for a day, and it is fantastic!
I searched the documentation and existing issues to see if this feature already exists, but I couldn't find it. If something like this already exists, my apologies, and please close this issue!
Feature description
When documenting a model, I'd like to be able to reference other dbt models. For example:
version: 2

models:
  - name: events
    description: A table containing clickstream events from the marketing website

    columns:
        - name: user_id
          description: The ID of the user who recorded the event. Points to the {{ ref('users') }} table.

Note the sentence Points to the {{ ref('users') }} table. Ideally, this would provide two benefits:

This should produce a hyperlink in the documentation from dbt docs generate.
This acts as a check against referencing a model that has been renamed or removed. If a model is renamed/removed, then ideally these references fail so you know where to update your documentation.

I'm not sure calling this function ref() is the best idea. We might want doc_ref() that allows an argument for the model name like ref, but also maybe another argument like a column name:

Points to the {{ doc_ref('users', 'id') }} column in the {{ doc_ref('users') }} table.

Relation to tests
Clearly you can express this relationship via a test:
tests:
    - relationships:
        to: ref('users')
        field: id

A test isn't quite sufficient for two reasons:

A hyperlink isn't produced in the documentation for tests. (This one could be fixed, probably.)
Sometimes a table points to another table, but there might be e.g. missing users, so the test would fail. That is, there might be some events with a user_id but no corresponding users.id. This is not uncommon in the data warehousing world when pulling/combining data from different sources.

Who will this benefit?
Anyone who uses dbt to document their models would benefit from this. It would make dbt's generated documentation more useful by adding hyperlinks to other models inside documentation and by making sure modifying a model doesn't break any upstream hyperlinks to it.

Hey @jdreaver - thanks for the feature request! We're already tracking ref hyperlinks in the docs context here: https://github.com/fishtown-analytics/dbt/issues/1334
I think @beckjake or @cmcarthur suggested using href, which I really like :)
You've also mentioned the relationships test here -- can you say more about that? We received a report that Relationships tests weren't rendering as hyperlinks recently, but I wasn't able to reproduce that error. When a relationships test is defined, the column should link back to the parent model in the documentation website:

Are you seeing something different?

Thanks for the response @drewbanin!
Ah I must have totally messed up my original test for the foreign key hyperlink feature. Womp womp.
I'll go ahead and follow #1334. Thanks for the heads up!


Update google-cloud-pubsub to 1.112.1

Updates com.google.cloud:google-cloud-pubsub from 1.107.0 to 1.112.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-pubsub" } ]

labels: library-update, semver-minor

Superseded by #214.


ETag is returned with surrounding quotes ?

GetETag() returns an ETag as "\"HASHOFCONTENTS\"". The surrounding quotes are intentional or some minor bug ?

That's set by S3 and is a common field in response. See here
HTTP/1.1 200 OK
x-amz-id-2: Vyaxt7qEbzv34BnSu5hctyyNSlHTYZFMWK4FtzO+iX8JQNyaLdTshL0KxatbaOZt
x-amz-request-id: 6B13C3C5B34AF333
Date: Wed, 20 Feb 2008 22:13:01 +0000

Content-Type: application/xml
Transfer-Encoding: chunked
Connection: close
Server: AmazonS3
<?xml version="1.0" encoding="UTF-8"?>

<CopyObjectResult>
   <LastModified>2008-02-20T22:13:01</LastModified>
   <ETag>"7e9c608af58950deeb370c98608ed097"</ETag>
</CopyObjectResult>


Add default whistle + whistles reorganize

About the PR

Move all whistle sprites into one folder
Add grey whistle in arcade
Add equipped sprites for all whistles
Why / Balance

Just feels good to give crew the whistle that won't get them arrested
Technical details

Media

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

Breaking changes

Changelog

:cl:

add: Grey whistle added to arcade machine prizes.

i dont understand what is the problem
the path is written in the common parent and all icon states is written in that prototypes


Can't download datasets

Hi! Thank you for releasing the code!
When I tried to download datasets with AzCopy, I got this error:
./azcopy copy 'https://datarelease.blob.core.windows.net/metro/datasets/datasets_release/freihand' ~/Downloads --recursive
INFO: Scanning...

failed to perform copy command due to error: Login Credentials missing. No SAS token or OAuth token is present and the resource is not public

I guess the datasets are not publicly available yet?
All the best,
Kailin

Thanks for pointing out it. Let me check the permission issues.

The download instruction has been updated.
To download the files, please use the following command.
cd $REPO_DIR
path/to/azcopy copy 'https://datarelease.blob.core.windows.net/metro/datasets/filename.tar' /path/to/your/folder/filename.tar
tar xvf filename.tar

filename.tar could be Tax-H36m-coco40k-Muco-UP-Mpii.tar, human3.6m.tar, coco_smpl.tar, muco.tar, up3d.tar, mpii.tar, 3dpw.tar, freihand.tar. Please check our document for further details about the directory structure.

Successfully downloaded the datasets. Thanks for your reply!


Possible to make multiple (regex) replacements in a single pass?

I'd like to make multiple regex replacements in a single pass of the file. Is this possible, or do I need to run through the file multiple times? I am making several replacements on a very large file so I'm trying to find the most performant solution. Thanks!

I don't understand your question. Can you elaborate?

@dustinbolton You can connect one stream after another, chaining. A stream only goes one time trough the file.

Thanks @rodrigopolo , assume your understanding is correct, @dustinbolton you might wanna learn more about stream.https://nodejs.org/dist/latest-v5.x/docs/api/stream.html


docs do not explain how to paginate

Checklist

[X] I have looked into the README and have not found a suitable solution or answer.
[X] I have looked into the documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have upgraded to the latest version of this SDK and the issue still persists.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Description
The documentation does not explain how to use pagination.
PS: bonus bug: the link in the PR template to the Auth0 Community forums (https://community.auth0.com/c/sdks/5) 404s
Expectation
A concise explanation of pagination using the go-auth0 sdk, including

a code snippet showing how to do it
a discussion of whether sorting is needed or not
explanation of tradeoffs using List.Next vs page indexes
links from each method returning a List to the explanation

Reproduction
Infinite loop doing the naive thing:
auth0API, _ := management.New(...)
ul, _ := auth0API.User.List(ctx)
for {
	for _, u := range ul.Users {
		// process user
	}

	if !ul.HasNext() {
		break
	}
	ul, _ = auth0API.User.List(ctx, management.From(ul.Next))
}

Auth0 Go SDK version
v1.2.0

Hey @DavidS-om, thanks for the issue!
We have an example of performing page and checkpoint based pagination in our examples file.
Do you think if we moved this to the doc.go file (so that it would be included in the packages godoc) it would be easier to find?

Hey @DavidS-om, thanks for the issue!
We have an example of performing page and checkpoint based pagination in our examples file.
Do you think if we moved this to the doc.go file (so that it would be included in the packages godoc) it would be easier to find?

absolutely. I'm seeing this file the first time now. And - except for my weak copy/paste game - it worked fine first try!

To round this out, we now have pagination examples in the godoc.
Thanks for raising this @DavidS-om!


Add a resource group to delete

In the step "Use exported template",  at the section "Create App Service plan", a new resource group may be created.

@yoichinak : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ShannonLeavitt : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@yoichinak - thank you for your comments.  Exporting template takes place in tutorial 7.  I will create PR to include the information in that tutorial.
#please-close.


system/coredump:upload kernal coredump change

Summary
system/coredump:upload kernal coredump change
Impact
coredump
Testing
./tools/configure.sh sabre-6quad/coredump

@anjiahao1 please fix:
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   boards/arm/imx6/sabre-6quad/configs/coredump/defconfig


Termination in progress exception

Hello,
I am using ManagedEsent in my project. I need to do something on different databases in one process. Think a simple ui which has a button which takes a path from user that includes a valid database file (x.edb) and then do the following:
Create an instance ( with the same name in every call)
Set some system parameters (circularlog, SystemPath, LogFilePath vs)
instance.Init()
BeginSession (using instance)
AttachDatabase  (sample project says that "Redundant Attach calls are safe")
OpenDatabase
OpenTable
BeginTransaction
Some delete, insert operations
CommitTransaction( with CommitTransactionGrbit.None - I also tried other two options)
CloseTable
CloseDatabase
EndSession
Terminate instance ( I tried both JetTerm and JetTerm2 api with different TermGrbit options)
After calling this function 1-4 times successfully, a following call throws
EsentTermInProgressException (Termination in progress) at any line of method. (sometimes deleting a row, sometimes closing table, sometimes closing database).
OS: Windows 7 Enterprise 64-bit
Using .NET 4.5
Is this a known issue or am i missing something ?
Could you please help ? Thank you...

@alikilinc , that isn't a great programming model. It will not scale very well. Both Init and Term can do a LOT of disk I/O.
You should create a single instance, and attach the database once in your service. Only do the DetachDatabase/Term when you shut the service down.
Each request should Begin a session, do the Open's, Begin/Commit the transaction. (For even greater scale you can re-use the sessions, keeping them around in a pool that you only close when you're shutting the service down.)


JDK 9

Tests failing with JDK 9 jigsaw:

BeanELResolverCleanUpTest - see BCEL-110
BeanValidationCleanUpTest
JavaServerFaces2746CleanUpTest
MXBeanNotificationListenersCleanUpTest
LdapPoolManagerInitiatorTest
ReplaceDOMNormalizerSerializerAbortExceptionInitiatorTest
SunGCInitiatorTest

Hi @mjiderhamn,
is this information still accurate?

With 9.0.4, the following test seems to leave a leak behind

BeanELResolverCleanUpTest
BeanValidationCleanUpTest
JavaServerFaces2746CleanUpTest
MXBeanNotificationListenersCleanUpTest
MXBeanNotificationListenersCleanUp_ListenerWrapperTest
MoxyCleanUpTest
SAAJEnvelopeFactoryParserPoolCleanUpTest

Not sure I will spend time on them, considering Java 9 is EOL.

Thanks for taking the time, @mjiderhamn. Absolutely understandable about Java 9. My background was actually about Java 10. I am currently trying out JDK 10 on our Jenkins instances and noticed that our classloader leak integration test is failing consistently. I then found this issue and wanted to test it with JDK 10 so I forked the repo and made a branch that executes against Travis CI with JDK 10 and could reproduce your failures as well: https://travis-ci.org/ArloL/classloader-leak-prevention/jobs/408391560
Do you have any intention of fixing it for JDK 10 or maybe target JDK 11 straight away? I tried looking into MXBeanNotitifcationListenersCleanUpTest and fixing it but stopped after 30 minutes of getting nowhere :man_shrugging:


CAMEL-19883 - Xml route initialization ignores Camel context

fixes https://issues.apache.org/jira/browse/CAMEL-19883
Junit coverage is present.
Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

This PR seems to be duplication of https://issues.apache.org/jira/browse/CAMEL-19766
(I accidentaly started to fix it on older version (not current main branch))
Therefore PR could be closed


Remove references to org.mortbay (jetty)

References to org.mortbay are no longer relevant in Jetty 9+
When the Jetty project moved from codehaus.org to eclipse.org the namespace changed from org.mortbay to org.eclipse.jetty
See: https://github.com/perwendel/spark/blob/2.2/src/main/java/spark/webserver/SparkServer.java#L56

Can't find any references, so this was done at some point.


fix(sea): mem2reg to treat block input references as alias

Description
Problem*
Resolves https://github.com/noir-lang/noir/issues/6443
Summary*
The mem2reg SSA pass eliminates stores that have no use, but as testing with different inliner aggressiveness showed it also eliminated stores into mutable references passed to a function. This is an effect that is expected to be visible outside, after the call. It turns out that PerFuncContext::is_store_alias_used does consider the input reference parameters of the function as stopping conditions for removing a store to an address, but it does so only if there is an alias for the address which appears in the input parameters. This wasn't the case, and the removal went ahead.
I added a method that adds a self-alias to all block parameters if they are reference types and don't already have aliases from the merge of the predecessor blocks.
Additional Context
See investigation in the ticket, e.g. https://github.com/noir-lang/noir/issues/6443#issuecomment-2456767668 shows a minimal failing form.
After merging this into the I only have these tests failing:
failures:
    tests::execution_success::test_eddsa::true_0_expects
    tests::execution_success::test_eddsa::true_i64_min_expects
    tests::execution_success::test_slice_dynamic_index::true_i64_min_expects

Which means it potentially fixes #6440 and #6442 as well.
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Is there any reason this is a draft?

Thanks for checking! I left it was draft just because:

I wanted to quickly run my tests locally on a merge into an "integ" branch that contains the brillig_cow fix and the inlining test matrix, to see if anything else broke
I was looking at the unit tests, wondering if I should add one - I probably should, but if you're okay with it as it is, covered by the test_programs, then I will move on. NB @asterite had the idea of writing a parser for the display format of SSA to make it easier to create tests.

NB @asterite had the idea of writing a parser for the display format of SSA to make it easier to create tests.

I think this would be very helpful tbh. It would be a lot easier for us to move full integration tests into unit tests for individual SSA passes.

Awesome! I was about to starting working on that a couple of times yesterday but wasn't sure if it would be wanted, and also wanted to start working on #6453... and the SSA language isn't that small, though it's definitely much smaller than Noir. I'll start working on this in parallel with other things after I finish #6453. I think it will also help quickly write a test by copying some output you get from --show-ssa, putting it into a test and then further checking things.


ie8 xhr patch method not support

Hi @janhommes
We use o.js in our intranet, all the pc are xp with ie8. And we found a problem,  ie8 XMLHttpRequest not support patch method. Maybe, it should check the browser, if it is ie8, xhr should be activexhttprequest instead of xmlhttprequest.
Thanks
function createCORSRequest(method, url) {
1446             var xhr = null;
1447
1448             //if no window assume node.js
1449             if (typeof window === 'undefined') {
1450                 var Xhr2 = require('xhr2');
1451                 xhr = new Xhr2();
1452             }
1453             else {
1454                 xhr = new XMLHttpRequest();
1455             }

Hi,
sorry I need to test more in IE8 but do not have enough time. Can you please check if we can just switch to the ActiveX as follow?
 //if no window assume node.js
            if (typeof window === 'undefined') {
                var Xhr2 = require('xhr2');
                xhr = new Xhr2();
            }
            else if(typeof window.XMLHttpRequest !== 'undefined') {
                xhr = new XMLHttpRequest();
            }
            else {
                //IE 8 support
                xhr = new ActiveXObject('MSXML2.XMLHTTP.3.0');
            }

Thanks,
Jan

thanks, I will test it today.

@lenoray any update on this? Does it work?

No feedback from @lenoray.


Create service files for all major entities

service system is now working one way

this is untested but everything should be implemented for devices to be able to be updated.
updateDevice is called with the a device -> the function calls the service function to update the server with the new data
-> the function changes the current local data and sends it out to all observers
here we have a small issue where once the server has recieved the new data it will trigger the socket and push an update to all clients including the one that triggered the update. This will result in the data re-rendering twice on that client.

Will stay in review until everything is advanced enough to be tested in action

@Michiii11 did u test this properly? afaik the update functions havent really been tested? Thats why i left a comment about this staying in review..

unlucky

this is fine for now i think.. it will be updated in the future but the system seems to be flexible enough for our needs