Couple of improvements.

I very rarely use this module but some peps probably use it on daily basis and I would like to give some feedback.

On Module dashboard it would be nice to change the order of impressions table to 'latest first'. If I click on month or year I need the most recent info on top so not to scroll down the page.
Search engine keywords - for removal I have no info here since years and probably no infor will come here anytime soon. :)
Best-selling tab - can we make columns here sortable. At the moment we sort by Sales but Quantity sold other columns could be interesting too.
Available quantities - can we have a switch here for 'all products' vs 'enabled products'? Or simply have this tab to show only currently enabled products. I have bunch of old products that I keep for reference (price, or if i get old stock from somewhere) and this pushes my newest products to the bottom of the page.
I use Cyrillic letters in my URLs so this might be an issue just for me but very long URLs are somehow trunckated in Pages not found tab and to them is added "-admin404".... I can't figure out what is causing this. An example:

Some of these has already been addressed. For others I created separate issues


Files are entered twice when an absolute path is passed on the command-line

When an absolute path is passed to srb, it looks like it can end up entering files twice into the GlobalState, one with the relative path, and one with the absolute path.
You should be able to reproduce by running Sorbet on (e.g.) https://github.com/Shopify/shopify-api-ruby (perhaps the extra --dir . in the sorbet/config is relevant, where paths aren't being de-duplicated across arguments?). Put a print statement in enterFile to show the file path of the argument, and pass $PWD to Sorbet as the directory to type-check (instead of . or not passing anything).

Sorbet automatically reads sorbet/config as command-line options.  For shopify-api-ruby, this file reads:
--dir
.
--ignore=test/rest

so you are effectively invoking Sorbet as sorbet $PWD --dir . --ignore=test/rest, passing the directory twice and getting duplicate paths.


Capmox 0.5.0 unable to access proxmox-credentials secret

What steps did you take and what happened:
I upgraded from Capmox 0.4.0 to 0.5.0
clusterctl upgrade apply     --core cluster-api:v1.7.2     --bootstrap kubeadm:v1.7.2     --control-plane kubeadm:v1.7.2     --ipam in-cluster:v0.1.0     --infrastructure proxmox:v0.5.0
Following the upgrade the Capmox controller logs the following errors:
E0619 13:51:44.068023       1 controller.go:329] "Reconciler error" err="error getting infra provider cluster or control plane object: No credentials found, ProxmoxCluster missing credentialsRef" controller="proxmoxmachine" controllerGroup="infrastructure.cluster.x-k8s.io" controllerKind="ProxmoxMachine" ProxmoxMachine="default/capi-management-v2-control-plane-qjwxf" namespace="default" name="capi-management-v2-control-plane-qjwxf" reconcileID="2ae9413d-cb8e-4425-9860-e870d84adb5a"
I then added created the required secret and added the secret in ProxmoxCluster.spec.credentialsRef.name
apiVersion: v1
stringData:
  secret: ${PROXMOX_SECRET}
  token: ${PROXMOX_TOKEN}
  url: ${PROXMOX_URL}
kind: Secret
metadata:
  name: "${CLUSTER_NAME}-proxmox-credentials"
  labels:
    platform.ionos.com/secret-type: "proxmox-credentials"

The capmox provider cannot then find the secret and is unable to communicate with proxmox:
E0619 13:51:43.567318       1 controller.go:329] "Reconciler error" err="error getting infra provider cluster or control plane object: Unable to initialize ProxmoxClient: failed to get credentials secret: Secret \"mk1-busi-cl-proxmox-credentials\" not found" controller="proxmoxmachine" controllerGroup="infrastructure.cluster.x-k8s.io" controllerKind="ProxmoxMachine" ProxmoxMachine="default/mk1-busi-cl-worker-pzvch" namespace="default" name="mk1-busi-cl-worker-pzvch" reconcileID="de7a6641-3c01-443a-8c5b-02f2cd62797a"
	failed to create scope: Secret "mk1-busi-cl-proxmox-credentials" not found
[14:02:29] k8sadmin@image-k get secret mk1-busi-cl-proxmox-credentials
NAME                              TYPE     DATA   AGE
mk1-busi-cl-proxmox-credentials   Opaque   3      26m

E0619 13:57:18.765794       1 controller.go:329] "Reconciler error" err=<
	failed to create scope: Secret "mk1-busi-cl-proxmox-credentials" not found
	failed to get credentials secret
	github.com/ionos-cloud/cluster-api-provider-proxmox/pkg/scope.(*ClusterScope).setupProxmoxClient
		/workspace/pkg/scope/cluster.go:142
	github.com/ionos-cloud/cluster-api-provider-proxmox/pkg/scope.NewClusterScope
		/workspace/pkg/scope/cluster.go:119
	github.com/ionos-cloud/cluster-api-provider-proxmox/internal/controller.(*ProxmoxClusterReconciler).Reconcile
		/workspace/internal/controller/proxmoxcluster_controller.go:107
	sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
		/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.3/pkg/internal/controller/controller.go:119
	sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
		/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.3/pkg/internal/controller/controller.go:316
	sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
		/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.3/pkg/internal/controller/controller.go:266
	sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
		/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.3/pkg/internal/controller/controller.go:227
	runtime.goexit
		/usr/local/go/src/runtime/asm_amd64.s:1650
	Unable to initialize ProxmoxClient
	github.com/ionos-cloud/cluster-api-provider-proxmox/pkg/scope.NewClusterScope
		/workspace/pkg/scope/cluster.go:121
	github.com/ionos-cloud/cluster-api-provider-proxmox/internal/controller.(*ProxmoxClusterReconciler).Reconcile
		/workspace/internal/controller/proxmoxcluster_controller.go:107
	sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
		/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.3/pkg/internal/controller/controller.go:119
	sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
		/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.3/pkg/internal/controller/controller.go:316
	sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
		/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.3/pkg/internal/controller/controller.go:266
	sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
		/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.3/pkg/internal/controller/controller.go:227
	runtime.goexit
		/usr/local/go/src/runtime/asm_amd64.s:1650
 > controller="proxmoxcluster" controllerGroup="infrastructure.cluster.x-k8s.io" controllerKind="ProxmoxCluster" ProxmoxCluster="default/mk1-busi-cl" namespace="default" name="mk1-busi-cl" reconcileID="19414580-a689-45ae-8798-42eb43157644"
[14:07:42] k8sadmin@image-k get secret mk1-busi-cl-proxmox-credentials
NAME                              TYPE     DATA   AGE
mk1-busi-cl-proxmox-credentials   Opaque   3      31m

What did you expect to happen:
I would have expected to upgrade capmox and then have to apply the required secret and add the required value into ProxmoxCluster.spec.credentialsRef.name
I then would expect the capmox controller to communicate with proxmox.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Cluster-api-provider-proxmox version: 0.5.0
Kubernetes version: (use kubectl version): 1.28.8
OS (e.g. from /etc/os-release):  ubuntu 24.04

@mkamsikad2 thanks for reporting this,
While I tested this before I will go and give it another try to check if upgrading broke the ProxmoxClient.

@mcbenjemaa I have upgraded to Capmox 0.5.1 and have now configured ProxmoxCluster.spec.credentialsRef.namespace
I can now build clusters and perform alpha rollouts. One thing that is still an issue is that all upgraded clusters are in a Failed state.
cluster.cluster.x-k8s.io/capi-management-v2                  Failed        66d
cluster.cluster.x-k8s.io/dev1-uk-busi-cl                     Failed        66d
cluster.cluster.x-k8s.io/mk1-busi-cl                         Provisioned   4m32s
cluster.cluster.x-k8s.io/test1-uk-busi-cl                    Failed        60d

NAME                                                                CLUSTER              READY   ENDPOINT
proxmoxcluster.infrastructure.cluster.x-k8s.io/capi-management-v2   capi-management-v2   true    {"host":"<IP_ADDRESS>","port":6443}
proxmoxcluster.infrastructure.cluster.x-k8s.io/dev1-uk-busi-cl      dev1-uk-busi-cl      true    {"host":"<IP_ADDRESS>","port":6443}
proxmoxcluster.infrastructure.cluster.x-k8s.io/mk1-busi-cl          mk1-busi-cl          true    {"host":"<IP_ADDRESS>","port":6443}
proxmoxcluster.infrastructure.cluster.x-k8s.io/test1-uk-busi-cl     test1-uk-busi-cl     true    {"host":"<IP_ADDRESS>","port":6443}```
I'm not sure how to bring the cluster out of a failed state.

i will check this soon

@mcbenjemaa I am also having this same issue. Thanks @mkamsikad2 for posting.

I couldn't reproduce this,
I upgraded from v0.4 to v0.5.1 and after that, i triggered an upgrade.

@mcbenjemaa It issue was cause by me upgrading from 0.4 to 0.5.0.
Upgrading from 0.4 to 0.5.1 works.
The only outstanding issue is the clusters which are in a failed state. There is no issue with those clusters and they function ok. Unsure how to reconcile the cluster state. Its not a major issue its just cosmetic

for me, the clusters were not in failed state, i don't know what happened in your setup

@mkamsikad2 this fix will be released in v0.6


Update storage-files-redundancy-premium-zrs.md

Added updated list of ZRS regions

@tamram

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Provide implementation of selectCoins API function.

Issue Number
#1136
Overview
This PR:

[x] Provides an implementation of the selectCoins API function.
[x] Provides an integration test to verify that a singleton payment is included in the output.
[x] Provides an integration test to verify that multiple payments are included in the output.
[x] Adds an UnsignedTx type to capture the concept of a set of inputs and outputs that are ready to be signed into a real Tx.

bors r+

bors r-

(cancelling to make sure we pick up the current tagged master for the upcoming release, unfortunately, we can only fetch artifacts from the latest hydra build at the moment, will merge a little later).

bors r+


Added Chinese translation

Translation by @Hakuin123 for Chinese language. Would team Crowdin language for the Chinese YAML file translation for pull request.

Merged in #40

Hey bro, it's great to see that your project now supports Chinese. 🥳 But it seems like I didn't contribute any translations for it (I hadn't even heard of this project before). Could you please check if maybe the wrong user was mentioned? I wouldn't want to take credit for someone else's work. (*ﾟーﾟ)


prometheus调用推送消息到alertmanager的rest请求不能加前缀，前缀配置失效

@withqingshao best issue ever, winner of the year!
title in Chinese and no description :smile:


Add raisin meringue pizza item

adds the item:
10847 raisin meringue pizza 268310979 pizza.gif food q,d 11
raisin meringue pizza 3 1 EPIC 0 0 0 0 Unspaded
Item raisin meringue pizza Effect: "Preemptive Medicine", Effect Duration: 13

Thank you! It looks like this item is randomly generated under a single item id so you should implement it like diabolic pizza

does the buff vary?

since this is randomly generated different items under the same ID this is too complicated for me. I will leave it to someone more experienced to implement


TokenRevocation BadRequest invalid_client

Recent version Version 1.2.0 throws me Error 400: BadRequest invalid_client

This might have been to the recent addition of the revocation of tokens. Do you use reference tokens?

Yes, I am using reference tokens

Ok... I'll see if it makes sense to default to false and add a setting to enable this.

I am using IdentityServer3 (2.5.4)

Ok, fixed on dev:
I changed the default for the token revocation at signout to false. You can enable it with revokeAccessTokenOnSignout on the settings. If the OP requires a secret, then there's now a client_secret property on the settings.

1.2.1-beta.1 pushed to npm. please test and let me know. thanks.

Hi,
I tested 1.2.1-beta.1 and the Default works as expected, setting revokeAccessTokenOnSignout:false explicitly also, but setting revokeAccessTokenOnSignout:true and adding client_secret:'****' does not change the result of my test - same behaviour as with revokeAccessTokenOnSignout:false. The reference token is still valid and IdentityServer does not output any log.

Here the log output of IdentityServer for the test with revocation enabled.
I only checked the validity of the token in our database, I did not try to access a resource - if it is only disabled in IdentityServer memory, then there should be no problem.
2016-10-21 08:06:01 [Information] "End session request validation success"
"{
"SubjectId": "84356c76-a0e8-4673-8e22-8a15a648a494",
"Raw": {}
}"
2016-10-21 08:06:01 [Information] End end session request
2016-10-21 08:06:01 [Information] Redirecting to logout page
2016-10-21 08:06:01 [Information] Logout prompt for subject: "84356c76-a0e8-4673-8e22-8a15a648a494"
2016-10-21 08:06:01 [Information] EnableSignOutPrompt set to true, rendering logout prompt
2016-10-21 08:06:08 [Information] Logout endpoint submitted
2016-10-21 08:06:08 [Information] Logout requested for subject: "84356c76-a0e8-4673-8e22-8a15a648a494"
2016-10-21 08:06:08 [Information] Clearing cookies
2016-10-21 08:06:08 [Information] rendering logged out page
2016-10-21 08:06:08 [Information] End session callback requested

Hmm, I tested this all locally and it was working. For me when I made changes, I was getting cached copies of my app.js -- could that be the case?

Hi,
I looked at it again. I think there is an error on my side or else where.
When I start userManager.signoutRedirect(), I get redirected to the logout page of IdentityServer and I can end the session. But I don't get redirected back - EnablePostSignOutAutoRedirect is activated and the post_logout_redirect_uri is set in the userManager config. Also there is no link to get back. So I have to go back to the main app page manually, and no _userManager.signoutRedirectCallback is triggered. Is the revocation started there?

Hi again,
tracking the calls in the browser showed, that the id_token_hint is not sent any more. Is there a configuration option to enable this again?

If the UserManager has an id_token it should be passed.

Hi,
sorry for my late response, I didn't get noticed.
The UserManager gets the response from identity server and fetches information from the profile endpoint, so user and id_token should be present. Any hint where I could look for further details or a solution?

That info is stored in sessionStore in the browser. Also check the HTTP traffic.

Hi,
now I had the time to check this.
in the LocalStorage, there is the following entry:
{"id_token":"eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsIng1dCI6ImEzck1VZ01Gdjl0UGNsTGE2eUYzekFrZnF1RSIsImtpZCI6ImEzck1VZ01Gdjl0UGNsTGE2eUYzekFrZnF1RSJ9.eyJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjgwODIiLCJhdWQiOiJCMkNDQzcyNy1DQUI2LTQ0M0QtQjEwRi02NEQ2REQ4MUZDMkIiLCJleHAiOjE0Nzg1NDE5NjEsIm5iZiI6MTQ3ODUzODM2MSwibm9uY2UiOiI2YjliZjRmNjM5N2M0ZTJiODgyNTYzNDk0MjI0Y2FmZCIsImlhdCI6MTQ3ODUzODM2MSwiYXRfaGFzaCI6Il85U2tRS1o3NDJhbUZwS1Rfd2p6ZXciLCJzaWQiOiJlY2RhNTRlNDllODNhM2U3MTc0Mzc3ZWNkNjEzNDg1ZCIsInN1YiI6IjY4MDc4ZWU2LTllZWYtNDgzMi05NTJmLTNhOWMxMTcyNjRiYSIsImF1dGhfdGltZSI6MTQ3ODUzODM2MSwiaWRwIjoiaWRzcnYiLCJuYW1lIjoiRHIuIERpZXRlciBDYXJsc29uIiwiaWQiOiI2ODA3OGVlNi05ZWVmLTQ4MzItOTUyZi0zYTljMTE3MjY0YmEiLCJ1aWQiOiI2ODA3OGVlNi05ZWVmLTQ4MzItOTUyZi0zYTljMTE3MjY0YmEiLCJhbXIiOlsicGFzc3dvcmQiXX0.E8XHTdpZcscHYtoIzKee92iDHyvS88bdJs9t87pgKSwovYKrGlVnm0IttZItmN-eGScqqEK-Dgo-nF84ANj_jWmIOMyxSVvgZcCJcvfkiy1nb8n40ZGETuIvGf9KlXKSxpbEYOVpA0qmwK6gKrns8iJ7yTwU7MmG8Rn1PoKlylXmhjQwyJDJI-pXmVIuS-1uCBNYa3B3f16jBgdJEQUnULjuoo6wA3KUf8Exu2-8aYV6wvAAZUF6_jG9NL4J-fG2e-mJQAR3TrA-j9NPdJyLXskny-wh_KS2fZxOUG5rc_ICq8YIQXOqqtiHZzqFTdJd0Bct_6OoVYYEFRBDx_BtLQ","session_state":"jpzrJ_Y8L-yxswwhxfO4S_nngBXtTdVpxyEhYQRld1A.124bc2e37088314145518549bdf59c7c","access_token":"5943b4fea6b7dc659b5aa1d4c23838a4","token_type":"Bearer","scope":"FrontendApi openid profile","profile":{"sid":"ecda54e49e83a3e7174377ecd613485d","sub":"68078ee6-9eef-4832-952f-3a9c117264ba","auth_time":1478538361,"idp":"idsrv","name":"Testuser","id":"68078ee6-9eef-4832-952f-3a9c117264ba","uid":"68078ee6-9eef-4832-952f-3a9c117264ba","amr":["password"]},"expires_at":1478538664}
SessionStorage is empty.
Silent renew works fine, but when I want to log out,
This are the http calls:
endsession	302		:8080/vendor.js:38395	492 B	36 ms
logout?id=0a7356782d5ca33c2a77db7a8daa58dc	200	document	http://localhost:8082/connect/endsession	3.2 KB	27 ms
styles.min.css	200	stylesheet	logout?id=0a7356782d5ca33c2a77db7a8daa58dc:7	(from disk cache)	3 ms
scripts.2.5.0.js	200	script	logout?id=0a7356782d5ca33c2a77db7a8daa58dc:45	(from disk cache)	3 ms
endsessioncallback?sid=ecda54e49e83a3e7174377ecd613485d	200	document	scripts.2.5.0.js:4    910 B	30 ms

Can you enable logging in IdentityServer and see if there's any info there? Also, it seems you're using localStorage (not sessionStorage) for the user -- this means you must have explicitly changed the default. If you're triggering signout from a different instance of the user manager and its settings then it might not know the user data and id_token to send for signout. Last idea -- can you getUser() right before you signout and check if there's an id_token on the user object?

Hi,
I changed the setting to use sessionStorage and ensured that the same instance of the user manager is used. This made no difference.
But your idea with getUser() was the point. In the click event of my Logout Button, the user was removed before the signout redirect was called. Removing the removeUser() call reactivated the submission of the token hint.
Thank you!

Hi @brockallen
I'm trying to figure out how to revoke my reference token when logging out of my site, and am having some trouble finding documentation/examples for this.
You mention above

"You can enable it with revokeAccessTokenOnSignout on the settings. If the OP requires a secret, then there's now a client_secret property on the settings."

I have addedrevokeAccessTokenOnSignout: trueto my settings, and have added revocation_endpoint to the metadata field in the settings as well (CORS is not enabled, so manually put the metadata in.)
I do not have a client_secret property there.
I am getting a 400 Bad Request Invalid Client response.
My question is:
1 - can this be because of missing client_secret? How do I know if my OP requires it? (I'm using IdentityServer3)
2 - is it secure to have the secret directly in my client side code where it can be seen by anyone who looks for it?
Thanks so much for your time!


Provide Eq implementation for Alignment

Hello,
I founds this to be a small problem while digging though some old code of mine. It seems like this used to work, but doesn't anymore.
Sometimes, it might be useful to compare alignments for equality. A bonus would be the ability to compare the horizontal and vertical alignments separately as well.

PR welcome...

Sure, I'll try to send one sometime next week :)


[CS] Use getParameterType for KeyPath as function

Previously we were using getPlainType to match the parameter type against the key path's base type. This gave us the external parameter type, which would be the element type for a variadic parameter.  However the code we generate expects the internal parameter type, which is provided by getParameterType.
Resolves rdar://problem/59445486.

@swift-ci please test


Error with client_credentials type

I setup the client_credentials type and it's generating the access token for me. When I use the "oauth-client" middleware I get this error:
NoActiveAccessTokenException in Authorizer.php line 104:
Tried to access session data without an active access token
but when I use "oauth" middleware it works, how I can fix this?
Also I want to know how I can use the client_scopes, because I created two clients with different scopes but it is ignoring the client scopes.

This is expected behaviour. You have to run both the OAuthMiddleware and OAuthClientOwnerMiddleware if you want to check that the current requests owner is a client.


Add format type to email alert signup

As we want to make Email Alert Signup work even when there are duplicate
topic names (thanks Govdelivery) we want to prepend a format_type
string to the topic name to prevent duplications across different format
on GOV.UK. This commit adds it to the schema and examples.

Open to suggestions if people think format_type is too close to format?

After a chat with several people I'm going to close this and take a different approach.


Show flowchart canvas at the bottom on mobile devices

In order to help the users by seeing their algorithm in a graphical way, show the flowchart at the bottom (half screen). We're keeping the toggle button to hide and show the canvas for convenience.

After reviewing the feedback, we have decided to maintain both the algorithm code and flowchart in full-screen mode, but specifically for mobile resolutions. This decision is aimed at enhancing visibility and user experience on mobile devices, ensuring that details are more clearly visible.
Issue closed.


Have a public API to tell if project isolation is enabled.

Expected Behavior

Have a public API to tell if project isolation is enabled. Similar to https://docs.gradle.org/current/javadoc/org/gradle/StartParameter.html#isConfigurationCacheRequested--
Current Behavior

No public API is available
Context

Some plugin author(like AGP) can select another code path to avoid project isolation issue which is a good thing for the project isolation adoption. Currently we rely on the internal API from StartParameterInternal but it is not ideal because it could break at some point and we have seen this happening in the past(e.g. https://issuetracker.google.com/issues/252848749)

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team and is prioritized by them.

@bingranl, could you describe your use-case when the plugin needs to take a different code path depending on whether Isolated Projects is enabled or not?

@alllex Sure, the use cases for isProjectIsolationEnabled and isConfigurationCacheRequested are similar

Collect data on how many users switch to CC/project isolation for analytics purpose
Surface this info to users to encourage them to switch to CC/project isolation
Disable some incompatible minor features when CC/project isolation enabled to unblock making a third party plugin compatible with CC/project isolation -> Gradle doesn't need to worry about having a different code path. The plugin author is responsible for that.


Error while running example.php

I traced the error. when running example.php it call the keypair in keypair function construct it call calcPulibc() in calcPublic it call Sha3::keccakhash in keccakhash it call keccak in keccak it call selfTest() in self test it call keccak64() in keccak64 this line of code having an error     $temp[$rsiz - 1] = chr($temp[$rsiz - 1] | 0x80);        the $temp[$siz - 1] has no value.


Pre-compiled includes/extends

Would be wonderful if the include/extend resolves in compilation time.
Sometimes is not necessary to load them asynchronously (if the include is mandatory and its not inside a flow control structure like an if).
Or at least to have the option to make them at compilation time :+1:

+1

:+1:

There are no plans for this at this time.


URI decoding of routes no longer works except for select few characters

Describe the bug
When there is %-encoded character in pathname decodable by decodeURI, server sends 200 response for pages and 404 for static files (+page.js, +page.svelte etc.)
Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-s8cge5

Click on goto /%5b
Expected: this is %5b. Received: glitch, this is %5b changes to 500 internal error.

Inside network inspector you can see that GET /%5b/+page.svelte fails with 404.
Logs
Not found: /src/routes/%5b/+page.svelte
Error: Not found: /src/routes/%5b/+page.svelte
    at resolve (file:///home/projects/sveltejs-kit-template-default-s8cge5/node_modules/@sveltejs/kit/src/runtime/server/index.js:281:13)
    at resolve (file:///home/projects/sveltejs-kit-template-default-s8cge5/node_modules/@sveltejs/kit/src/runtime/server/index.js:312:5)
    at Object.handle (file:///home/projects/sveltejs-kit-template-default-s8cge5/node_modules/@sveltejs/kit/src/exports/vite/dev/index.js:360:66)
    at respond (file:///home/projects/sveltejs-kit-template-default-s8cge5/node_modules/@sveltejs/kit/src/runtime/server/index.js:309:40)
    at async eval (file:///home/projects/sveltejs-kit-template-default-s8cge5/node_modules/@sveltejs/kit/src/exports/vite/dev/index.js:430:22)

System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.87
    @sveltejs/kit: next => 1.0.0-next.538
    svelte: ^3.46.0 => 3.52.0
    vite: ^3.1.0 => 3.2.3

Severity
annoyance
Additional Information
No response

Should %-encoded characters, decodable by decodeURI be completely rejected in folder names?

When there is %-encoded character in pathname decodable by decodeURI, server sends 200 response for pages and 404 for static files (+page.js, +page.svelte etc.)

I don't understand the terminology here. Static files generally refer to files that live in static. Pages generally mean +page.svelte. Is it all directories with a %-encoded character that are failing? Can you clarify what cases are working? (i.e. sending a 200)

@benmccann I mean
If directory segment contains uri-decodable character:
When html requested, server returns 200.
When .js/.svelte requested, server returns 404.
I updated project on stackblitz.
There are 2 folders with encoded characters and one [slug].

Folder with %5b. Request to /%5b returns 200 for html and 404 for .js/.svelte. Correct candidate used.
Folder with %40. Request to /%40 returns 200 for all. Incorrect [slug] candidate used.
Request to /@ returns 200 for all, Correct candidate is used.
Folder with [slug]. Requests /%5d returns 200 for all. Correct candidate used.

This is a regression. You used to be able to have a directory named something like %5b. It was broken in https://github.com/sveltejs/kit/commit/b30d2b64e4843a309f1033100a5d9d1e774c5140. Unfortunately that PR is 900 files large, so not exactly easy to pin down...

My guess is this is some kind of bug in write_client_manifest? I'm a bit stumped and would love if you could figure this out

@benmccann I am wonder to check this, but in next week

Actually, I lied. It was broken before that commit. I forgot to update my test to the old routing system and another page started loading making it look like it was working. It's been broken for at least a year. I don't know if it ever worked. We do seem to have some docs for it, but I guess we just tested the very few characters that worked and assumed they all did...


[ui] Numpad numbers are not centered in their container

It works on my machine
(/Phone)
Why does iOS something different...
I can't test it that way...

Maybe you've used some css which is not supported by Safari yet, or works differently on safari 😃
I can look into it later ;) If it's a safari problem, it should also happen on my Mac 😄


Fix TimescaleDB version accepted

Previously, the versions accepted the 2.0.0-rc which created problems
when being auto-upgraded. Fix this for now. But a fuller fix for
issue #329 will be in a separate PR.

Superseded by 328?


Add Nepali locale ne

Description
This PR adds language support for the Nepali language.

Type of Change

[x] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

@googlebot I consent.

@felangel Thanks for the review 🎉


Spaces cause decompressFromUTF16 to return null

The following compresses and decompresses just fine.  As soon as I replace a '&nbsp'; with a space, I receive a null value upon return of decompressFromUTF16.

Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nunc cursus egestas ultricies. Aliquam nulla orci, maximus sed consequat vel, pellentesque non purus. Morbi vitae maximus turpis, vitae congue enim. Vestibulum ut nulla mauris.

I've been testing this out in Chrome, and it took me forever to figure out what was triggering the null return.  I'm compressing a string that I am putting into a JSON object. I am sending that object to the server to be stored. When the JSON object comes back from the server I am pulling the text out and decompressing it. I don't think my AJAX or server side code is affecting this.  But I figured I'd leave my full description here just in case.
I've figured it out.  Looks like some escape characters where being inserted. If anyone else runs into this, I simply JSON.stringified() my string before compressing it and than JSON.parse() my string after I decompressed it to account for escape characters.

LZString operates on a string. Not an object.
LZString.decompressFromUTF16(LZString.compressToUTF16('hello'))
"hello"
LZString.decompressFromUTF16(LZString.compressToUTF16('hel lo'))
"hel lo"
LZString.decompressFromUTF16(LZString.compressToUTF16({a:'b'}))
""


Reduce build warnings

When Cilium repo is located on a directory whose path is not a legal
Go path, go list adds a leading underscore to the output absolute
paths. Remove the leading underscore and the absolute path. This will
not make all make targets to to work (e.g. 'make gofmt' will not work
in this scenario), but a lot of spurious warnings for targets that do
not need Go are eliminated.
Filter out the annoying "warning: directory does not exist" warnings
when building protobufs. A preceding "INFO: From ProtoCompile" lines
still remain, but there are only a handful of them, and they provide
an indication that something was filtered out.
Fixes: #4703

This change is

@joestringer Added SHELL=/bin/bash -o pipefail to envoy/Makefile. This adds the pipefail option to all commands run by the makefile, but this should be OK and has the benefit that the commands themselves remain less verbose.

test-me-please

Confirmed with @joestringer that we can merge this, so I am merging.


Add network flag to args

close #188

Can you document the network flag somewhere? I think we should have all the possible flags in the README, and maybe the demo docs will need to be updated.
I'm not sure if this is the right direction tbh. It feels a bit weird to have conf, datadir and network, but if you like it I'm ok with that

I does not break the process of the DEMO with this branch and DEMO specifies to use 0.2 release
Because we want to enable multiple network setup in a datadir, a flag network was necessary to make explicit wich setup the user want to use or install in a datadir.
We keep the --conf flag in order to have an alternative way to run a setup if it exists or get an error if it does not without launching the installer.


Styles are messed up in mobile app

Has there been any progress for this on mobile?

I have a bunch of different themes on my iPad and found that one of them (Sanctum) displays the TODO list properly (narrow column for checkbox and wide for description). All of the other themes display the buggy version (wide column for the checkbox and narrow for the description). Hopefully this contains a clue for fixing the bug.

Well I found the issue. They added a new mobileEmulation flag you can set and turns out for mobile they add the style
button {
  width: 100%
}

and since the collapse icon and the complete icon are both buttons it messes things up. Fix coming in next release!


python3Packages.tinytag: init at 1.10.1

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build python312Packages.tinytag
I recommend that the commit message starts with python312Packages so that ofborg can automatically start the build.
https://github.com/NixOS/nixpkgs/blob/master/doc/languages-frameworks/python.section.md#contributing-guidelines-contributing-guidelines

Commit names of Python libraries should reflect that they are Python libraries, so write for example python311Packages.numpy: 1.11 -> 1.12. It is highly recommended to specify the current default version to enable automatic build by ofborg.

Result of nixpkgs-review pr 331372 at a5541981 run on aarch64-darwin 1

2 packages built successfully:

python311Packages.tinytag
python312Packages.tinytag


add Bareos 22

Pull Request (PR) description
Add Bareos Version 22 support and the possibility to add the bareos user to other groups.

Is this still relevant? Bareos changed repo configuration and it's now located under https://download.bareos.org/ with no versioning, only current / experimental and next.

Is this still relevant? Bareos changed repo configuration and it's now located under https://download.bareos.org/ with no versioning, only current / experimental and next.

For subscription it's still relevant on https://download.bareos.com/bareos/release/


requiredOptionMarker displayed on ArgGroup when it should not

When having a ArgGroup with multiplicity 0..1, the usage shows options as required, but they should not. Here's an example code (I'm running picocli version 4.6.1):
@Command(requiredOptionMarker = '*')
public class RequiredMarkerDisplayedIncorrectly {

	public static class ExclusiveOptions {

		@Option(names = "--silent",
				description = "Silent mode",
				required = false
				)
		public boolean silent;

		@Option(names = {"-v","--verbose"},
				description = "Verbose mode",
				required = false)
		public boolean verbose;

	}

	@ArgGroup(exclusive = true, multiplicity = "0..1")
	public ExclusiveOptions exclusive;

	public static void main(String[] args) {
		new CommandLine(new RequiredMarkerDisplayedIncorrectly()).usage(System.out);
	}
}

The output is this:
Usage: <main class> [--silent | -v]
*     --silent    Silent/quiet mode
* -v, --verbose   Verbose mode

Interesting, that certainly looks like a bug.
Would you be interested in providing a pull request for this?

I could probably fix a pull req for this, but I'm wondering whether there's a corner case that I could have missed here. Or should the option of 'required' or not come completely from some other place than the ArgGroup annotation?

I'm wondering whether there's a corner case that I could have missed here. Or should the option of 'required' or not come completely from some other place than the ArgGroup annotation?

Sorry, I don't understand the question. I expected the usage help message to look like this:

Usage: <main class> [--silent | -v]
      --silent    Silent/quiet mode
  -v, --verbose   Verbose mode

So, no * in front of the options, otherwise the same. Is that your expectation also?

Yes I expect the same behavior in this case! Just pondering if there's some other case where there's some logic needed to deduce if one or more options should show as being required. But I would assume such logic is solved using the @Option or @Parameter annotation information instead.

Yes, the OptionSpec and PositionalParamSpec model classes have an isRequired method (from memory, away from pc).
If you’d be interested in providing a Pr that’d be great!

So I started digging through the code and found on line 10098 in the ArgGroupSpec that exclusive groups are set to required=true always. When removing this the output instead turns to
Usage: <main class> [[--silent] | [-v]]
      --silent    Silent mode
  -v, --verbose   Verbose mode

Which I guess is correct, as neither of them is actually required so the hard brackets should be included? However, it wasn't that straightforward to set up the project in Eclipse so I cannot run any test suites or verify that this doesn't break anything else (which isn't really included in the contributing info anyways). Do you have any pointers on how to easily set up the project to run tests etc in an efficient way?

Thanks for giving some hints on how to run the tests, makes it much easier to contribute!
And yeah, never thought of using the groups to handle lists of exclusive options, that's why I asked if there might be some case of which I haven't yet considered and thus might break something when solving my issue. My case seems much more trivial than the #871 ticket, and the validation seems to work in my case. Only the rendering of the usage text is the issue here. Not sure if I can contribute more to this, would require quite a bit of time.


Crashes after 3 seconds

As the title says, when I open it the overlay stops responding after around 3-5 seconds.

what do you mean when i try to compile it i get more than 2k error messages


remove SDL2::RAW version requirement

hi there, here's a patch to fix the tests..
wasabi:perl6-sdl2 mason$ AUTHOR_TESTING=1 prove -e "perl6 -Ilib"
t/01-load.t ......... ok
t/99-author-meta.t .. ok
All tests successful.
Files=2, Tests=2,  1 wallclock secs ( 0.03 usr  0.01 sys +  1.62 cusr  0.14 csys =  1.80 CPU)
Result: PASS

:+1:


Upgrade to LTS 19.6 and GHC 9.0.2

This is needed for current Nixpkgs as well

Seems like it was never on hackage. Oh well, it's on there now! :shrug:

Thanks for adding it to Hackage. Would you mind adding it to Stackage Nightly as well? Hopefully that's not too much of a bother.


consider adding is_disconnected method to Receiver type

Sometimes we need to check whether the receiver is disconnected rather than empty, to make sure there will not be any other msg being sent to this receiver.
The semantic of this method is quite clear. For ReceiverFlavor::Array ReceiverFlavor::List ReceiverFlavor::Zero, is_disconnected will return true when corresponding Sender is dropped; for ReceiverFlavor::Tick, is_disconnected will always be false; for ReceiverFlavor::At, is_disconnected will be true after it's fired; for ReceiverFlavor::Never, is_disconnected will always be true.

Duplicate of #586.


chore: Enable Android workflow for the demo app

We will be using this as the source of truth to run the tests on GitHub action on our side. We're running issues there too.


ci: reusable github workflow repo

:spiral_notepad: Description

:white_check_mark: Checklist

[ ] Linked to any related issues in the PR description
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Breaking changes were made

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

Project coverage is 100.00%. Comparing base (0be7ba0) to head (f8b32b7).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #54   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            9         9
  Branches         4         4
=========================================
  Hits             9         9

Flag
Coverage Δ

6.0.x
100.00% <ø> (ø)

7.0.x
?

8.0.x
100.00% <ø> (ø)

ubuntu-latest
100.00% <ø> (ø)

unittests
100.00% <ø> (ø)

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

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


chore: fix publish to use github env vars instead of travis vars

And adds the readme.md back to the root of the repo

Could we actually go ahead and put the readme in the .github folder? I'm keen to keep the root level as minimal as we can.

Could we actually go ahead and put the readme in the .github folder? I'm keen to keep the root level as minimal as we can.

Could we actually go ahead and put the readme in the .github folder? I'm keen to keep the root level as minimal as we can.

For sure!

Could we actually go ahead and put the readme in the .github folder? I'm keen to keep the root level as minimal as we can.

For sure!


Domain name setup for DB

My Environment

ArangoDB Version:     3.7.3

React Native.
When i am trying with Domain name (https://{testsite.me}:8529/
API getting failed. but when i try with ip address it's working
We are using React native(Mobile app) & arangodb foxx framework for API
how to use domain instead of IP
for web application it is working with domain name, but mobile app it's not working with domain but works with ip address.
can you please suggest.

Hi,
if you only just configured the DNS, you may see that it takes some time for the information to spread.
Its also sometimes that mobile providers do special things like proxies etc. for their networks.
In general I'd advise that its a good idea to use foxx services via http forwarding in the webserver under the matching locations.

closing as solved.


The height of the blockchain

Hi guys! Please tell me why the height of the blockchain in the explorer jumps up and down? And it just doesn't grow up?
2023/12/10 20:58

2023/12/10 20:59

2023/12/11 19:18

Maybe I don't understand something about your blockchain technology, but so far it's more like a scam to fool people

Kaspa has no "height" and this is not a blockchain, this technology is known as BlockDAG, which is quite different.  I will discuss this with the community developers who created and maintain the explorer. I agree that these metrics are a bit misleading to the general public as they represent metrics of the local node which prunes data every ~3 days, retaining only the cryptographic proof of the data continuity as well as the resulting cryptographically-bound transaction dataset.
You are welcome to join Discord and ask the community for explanations on further details, or lookup "Bitcoin pruning" on Google as that is the closest to what you are observing (if framed in the context of traditional blockchains like Bitcoin).
If you seek further explanations, please use Discord. Closing this issue.


response from this code is not true but instead is 1

System Information (please complete the following information):
Model Builder or CLI Version: not sure where to find
Visual Studio Version (if applicable): 2019 v16.8.3
When I run the app, it says "1" instead of "True"
On which page of the tutorial did you run into an issue (URL):
https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/consume
Clear description of the problem:
The output doesn't match the tutorial.
To Reproduce
Steps to reproduce the behavior:
Run the project
Expected behavior
I expect the output to match the tutorial
Screenshots
N/A
Additional context
N/A

This is because the label is in format 0/1 instead of True/False. And the code's behavior is expected. We will update our doucments.


fix: cursorWordPartXXX commands cannot handle camel case word with number

Handle digits like lower case ASCII characters when moving cursor within a camel case word.
This PR fixes #146498.

Thank you!


Avoid using the is operator with string literals.

This change is

Current coverage is 82.11% (diff: 100%)

Merging #92 into master will not change coverage

@@             master        #92   diff @@
==========================================
  Files            24         24
  Lines          1991       1991
  Methods           0          0
  Messages          0          0
  Branches        283        283
==========================================
  Hits           1635       1635
  Misses          237        237
  Partials        119        119

Powered by Codecov. Last update c4e20b0...d6b3f2a


Updates to ContentWidth and Bug Fixes

Additions:
Content Width

The content width was relatively simple to make work. The container is full width and the input takes up a proportion of that width. Most elements will work with 0.65 scale but the demo window has custom widths as seen in the demo window.

Layout Demo Section

A new Layout demo section has been added which showcases the content width changes and how to use them.

Tweaks:

Changed the slider transparency. I'm not sure why they were partially transparent. Inline with Dear ImGui, there should be fully opaque.
Changed the content width default value to 65% inline with Dear ImGui.
Input fields have a minimum width of 1 to prevent weird sizing issues.
The text for a drag input was under the drag bar. A new textlabel is placed on top to ensure the text is always on top.

Bug Fixes:

The Combo box did not sink inputs meaning you could move the window around whilst the combo box was open, done by holding down on the box and just moving the mouse. By changing it to a button, the input is now sunk.
Some containers did not properly clip descendants.

Notes:

The changes to content width have broken the style editor. Seeing as UDim and Color input fields seem to be a new focus area, I haven't attempted to fix them. The issue is because the new width of each input is the full screen width which pushes everything else away. This does prevent having elements in the sameline for input fields so this may need to change.
As always, some changes are stylistic and up to preference so change anything which needs it.

Great work, no issues with any of the changes. Im also glad you decided not to fix the style editor, im planning on rewriting the entirety of it, once input widgets are completed.


What does "authoring" mean in this context?

Can you please clarify what "authoring" mean? When is that timeout applied exactly? Are we talking about the preview phase while authoring a query and/or the overall evaluation phase (the popup with checkmarks) when a dataflow is being saved?

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

ID: 3faffe45-9b88-7ca0-4fb9-1eaac7859baa
Version Independent ID: 1781cb1b-0976-62e1-3060-7e3fdcd96225
Content: Power Query Online Usage Limits - Power Query
Content Source: powerquery-docs/power-query-online-limits.md
Service: powerquery
GitHub Login: @cpopell
Microsoft Alias: dougklo

Please ignore: I'm closing and opening this issue to get automation to add tags. #please-close

#please-open

Hi @otravers
Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response.
However, the article has been updated with new information that more fully defines what is meant by authoring.
We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


ARLocation framework: Installation with Carthage

👍

hey @awilliams88 - could I get you update this with the latest bits in this repo? I'd love to get Carthage support working!

Superseded by #112


Enable Student Self-Registration and Admin Student Creation⚡

Description:
Issue:
We want to enhance the security of our web application by adding OTP (One-Time Password) functionality to the signup and signin processes. This will help verify user identities and protect against unauthorized access.
Proposed Solution:
1. Signup Process with OTP:

When a user signs up, generate a random OTP (6 digits).
Send the OTP to the user's provided email or phone number.
Store the OTP and its expiration time in the database.
Ask the user to enter the OTP to verify their identity during registration.

2. Sign-in Process with OTP:

When a user signs in, generate a new OTP.
Send the new OTP to the user's email or phone.
Verify the entered OTP to allow access to the user's account.

Acceptance Criteria:

OTP functionality is implemented for both signup and signin.
Users can successfully complete the signup and signin processes using OTP verification.
Documentation is updated to reflect the new OTP functionality.

@Nishitbaria I would like to try and add this feature ..

Sure you can start Working on this issue . @adityakhattri21

@adityakhattri21 are you still working on it?

I want to work on this feature, could you assign this to me ?

Sure @piyush-gangrade, I'm assigning this issue to you you can start working on it.

@Yash636261, could you please add the gssoc label

@Yash636261 @piyush-gangrade please assign this issue to me under GSSOC24

@Yash636261 , could you please check the PR? If you want to change anything, let me know


[AutoPR cognitiveservices/data-plane/LUIS/Authoring] Updating Authoring for roles

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6384

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/5267


Make images and slides for notebook 01

finished all slides and images for 01, see them here - https://www.learnpytorch.io/01_pytorch_workflow/


Issue with /docs/admin/limitrange/

This is a...

[ ] Feature Request
[X] Bug Report

Problem:
In the command kubectl create -f docs/admin/limitrange/limits.yaml --namespace=limit-example, I don't have this docs/admin/limitrange/limits.yaml file.
Proposed Solution:
Provide the contents of the file, or a valid link to download it.
Page to Update:
http://kubernetes.io/...

The referenced doc has been fixed. A quick search of all 'limits.yaml' file referenced in the text are properly fixed with a valid url.


Question on start-mailserver.sh and SASL_PASSWORD

I've seen that in the code of start-mailserver.sh it has been introduced the use of SASL_PASSWORD env var. However when no passwd is being provided the script outputs a warning:

Loaded '/tmp/postfix/main.cf'
==> Warning: 'SASL_PASSWORD' is not provided. /etc/postfix/sasl_passwd not created.

I find this warning not necessary. Could we leave simply the message without the warning?
Or it has been left there for a specific reason? For the parsing of main.cf the warning has been removed, for example, by the PR #104 .

I agree. If optional, it should be an Info.

Feel free to PR.

Fixed.


game crashes on map loading

Loading into a save/map pulls that error message about memory, except all the digits are zeroes.

got this, this time, weird

nwo it just starts and says "game initialization failed" man im fuckin stumped


Add Channel

Add functionality for adding channel in React Version

In #8 I've added functional to add channels, still need to be fixed. We need change our function for IPC to return channel data via sendSync

No need for sendSync since backend can add channels without frontend request. Closing.


xbox 360 kinect sensor

How can I use this for an xbox 360 kinect?

Hi, sorry for the delay. You can't. You can try, if you find stable drivers for Windows 10 and if you can use the SDK (which is different from Kinect V2).


keep trans and child table collection_id in sync

Describe the bug
A clear and concise description of the encountered error.
I am trying to create a new NMU:Mamm accession and getting an error stating a duplicate key value. However, this is not a duplicate accession. The last one that was used was 2024.02 and trying to create 2024.03
To Reproduce
Steps to reproduce the behavior:

Go to Accession create
Typed the following information shown in the screenshot.
Getting the following error:

Screenshots

Desktop (please complete the following information):

OS: 22631.3593
Chrome
Version 129.0.6668.71

@jrpletch

I think what probably happened is I created the transaction first as an NMU:Mamm accidentally and then when I realized that it was the wrong collection changed it to NMU:Para in the edit accession window.

We have similar problems with keeping track of accession numbers coming from mutiple divisions into the tissue collection, and our solution is to add a suffix to each accession - e.g. 2024.03.Mamm, 2024.03.Para, so it is clear which is which. Not all tools in Arctos show the "NMU:Mamm" prefix, so this adds extra clarity to keep me sane.

So if we run into Josh's situation in the future would it be best to simply delete the faulty accession and create a whole new one?  Or is there a way to move a (complete) accession to a new collection?

I'll fix the bug and make sure everything is in alignment, this should not be a future problem.

We created a new NMU:Para accession, transferred the record over, and then deleted the original NMU:Para 2024.03. We then created a new NMU:Mamm 2024.03 accession with no errors.
Everything worked after we deleted the original NMU: Para 2024.03 accession.

This is widespread, but I think it's also easy to understand: trans.collection_id is primary and should "win" when there's any conflict, the bug involves updating the dependencies, not the primary. I'll stash "old" data and update when this is resolved, and I don't think that needs any further interaction - but please let me know if any of this seems wrong in any way.
Bigger picture,collection_id was denormalized to child tables I believe to address some performance issue in Oracle (I can't find the issue, so possibly/probably pre-GitHub). That benefit does not seem to balance the costs at the moment, and I am tentatively planning to return to a state of normalization by removing collection_id from accn, borrow, and loan.

Pre-normalize data stash:
temp_pre_norm_accn.csv.zip
temp_pre_norm_loan.csv.zip


Make heartbeat track get louder for low health/tense fights

The heartbeat track is loaded as an independent audio track so that it's volume can be independently controlled to signal important moments such as low health. Just turn up the volume when the player's total hearts are below a certain threshhold.

I tried to do this by adding a routine in the player class that checks for low health and changes the volume appropriately via Jukebox.heartbeat.volume. Unfortunately, I can't hear any difference. There must be something I don't understand about volumes and audio in JavaScript... :/

I'll take a look at it, if possible

Yeah, I couldn't get this to work, either. I tried mixing it so the music got quieter as the heart-beat got louder, that didn't sound good at all.


Page scrolling in background

On opened gallery, there should be an option to remove page scrolling. Meaning, a parameter, to set body{overflow:hidden;} on open and remove it on gallery close.
I apologize, if there already is such option.

@aaxc There is a fix. Go to ngGallery.js and

Add document.body.style.overflow = 'hidden'; below scope.opened = true; line
Add document.body.style.overflow = 'auto'; below scope.opened = false; line

I'm not sure if we should add this to ngGallery.
Changing styles of body sounds kind of dangerous to me.
@jkuri What do you think?

@thatisuday Thank you, I didn't ask for a fix, I already added $('body').css({overflow: 'hidden'}); and $('body').css({overflow: 'auto'}); before those lines. I just thought this should also have a parameter like hide-overflow="true" similar to  thumbs-num="3".
And I don't think that it's dangerous, since even bootstrap has this option enabled by default.
PS: Really nice script, is perfect for my project!


failed to build 0.24.20

👋 trying to build the latest release, but run into some build issue. The error log is as below:

error build log
  error[E0599]: no method named `blocking_queue_depth` found for struct `RuntimeMetrics` in the current scope
    --> libsql-server/src/http/admin/mod.rs:99:34
     |
  99 |                     .set(metrics.blocking_queue_depth() as f64);
     |                                  ^^^^^^^^^^^^^^^^^^^^ method not found in `RuntimeMetrics`

  error[E0599]: no method named `injection_queue_depth` found for struct `RuntimeMetrics` in the current scope
     --> libsql-server/src/http/admin/mod.rs:101:34
      |
  101 |                     .set(metrics.injection_queue_depth() as f64);
      |                                  ^^^^^^^^^^^^^^^^^^^^^ method not found in `RuntimeMetrics`

  error[E0599]: no method named `num_blocking_threads` found for struct `RuntimeMetrics` in the current scope
     --> libsql-server/src/http/admin/mod.rs:103:34
      |
  103 |                     .set(metrics.num_blocking_threads() as f64);
      |                                  ^^^^^^^^^^^^^^^^^^^^ method not found in `RuntimeMetrics`

  error[E0599]: no method named `num_idle_blocking_threads` found for struct `RuntimeMetrics` in the current scope
     --> libsql-server/src/http/admin/mod.rs:105:34
      |
  105 |                     .set(metrics.num_idle_blocking_threads() as f64);
      |                                  ^^^^^^^^^^^^^^^^^^^^^^^^^ method not found in `RuntimeMetrics`

  error[E0599]: no method named `io_driver_fd_deregistered_count` found for struct `RuntimeMetrics` in the current scope
     --> libsql-server/src/http/admin/mod.rs:109:39
      |
  109 |                     .absolute(metrics.io_driver_fd_deregistered_count() as u64);
      |                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ method not found in `RuntimeMetrics`

  error[E0599]: no method named `io_driver_fd_registered_count` found for struct `RuntimeMetrics` in the current scope
     --> libsql-server/src/http/admin/mod.rs:111:39
      |
  111 |                     .absolute(metrics.io_driver_fd_registered_count() as u64);
      |                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ method not found in `RuntimeMetrics`

  error[E0599]: no method named `io_driver_ready_count` found for struct `RuntimeMetrics` in the current scope
     --> libsql-server/src/http/admin/mod.rs:113:39
      |
  113 |                     .absolute(metrics.io_driver_ready_count() as u64);
      |                                       ^^^^^^^^^^^^^^^^^^^^^ method not found in `RuntimeMetrics`

  error[E0599]: no method named `remote_schedule_count` found for struct `RuntimeMetrics` in the current scope
     --> libsql-server/src/http/admin/mod.rs:115:39
      |
  115 |                     .absolute(metrics.remote_schedule_count() as u64);
      |                                       ^^^^^^^^^^^^^^^^^^^^^ method not found in `RuntimeMetrics`

  For more information about this error, try `rustc --explain E0599`.
  The following warnings were emitted during compilation:

full build log, https://github.com/Homebrew/homebrew-core/actions/runs/10477409925/job/29018548880?pr=181820
relates to Homebrew/homebrew-core#181820

close via https://github.com/Homebrew/homebrew-core/pull/183257


Create master list of services/ repos to scan

Previously we were using a 'service discovery' that cloned and parsed this information out of the dns repo.  We're now using GitHub Webhooks on registered repositories. We'll need to agree upon a registration process/ how and where to store and maintain this master list, and/or if we'd like to use repo root yaml project files with additional information on service urls, domains, etc that could be used.

With the new graph-updater architecture, this is no longer needed. Leaving this in here for now as we may still want to use the DNS repo for additional registration avenues?

Actually going to close, and create a different issue for this


Modify "LeafSegmentKeyGenerator" class and add some test cases.

Fixes #2919.
Changes proposed in this pull request:

modify "LeafSegmentKeyGenerator" class
add test cases in "LeafSegmentKeyGeneratorIT.java"
add test cases in "LeafSegmentKeyGeneratorTest.java"

Pull Request Test Coverage Report for Build 268

32 of 34 (94.12%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 64.909%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/keygen/LeafSegmentKeyGenerator.java
32
34
94.12%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/keygen/LeafSegmentKeyGenerator.java
1
93.59%

Totals

Change from base Build 291:
0.02%

Covered Lines:
9232

Relevant Lines:
14223

💛  - Coveralls


Support utf-8 .draw()

Drawing method is not working on windows CMD as emojis are not supported

Fixed in 4fc99e816758d651af7681dfbf805ecdb99c9656


build_pip_package broken by removal of keras/api directory

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: n/a
TensorFlow installed from (source or binary): source
TensorFlow version: git HEAD
Python version: 3.6.8
Installed using virtualenv? pip? conda?: n/a
Bazel version (if compiling from source): 3.7.2
GCC/Compiler version (if compiling from source): 10.3.0
CUDA/cuDNN version: n/a
GPU model and memory: n/a

Describe the problem
build_pip_pacakge fails with error output
cp: cannot stat /tmp/tmp.enXUtfEE9u/tensorflow/python/keras/api/_v2/keras/: No such file or directory
Provide the exact sequence of commands / steps that you executed before running into the problem
$ bazel build --config=nonccl //tensorflow/tools/pip_package:build_pip_package --verbose_failures
$ mkdir tensorflow-pkg
$ bazel-bin/tensorflow/tools/pip_package/build_pip_package --cpu --project_name tensorflow_aarch64 ./tensorflow-pkg
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
Seems to be a result of the commit
https://github.com/tensorflow/tensorflow/commit/61754492129f651f23f05b5aed889f376ec35bb8#diff-cebea885ef40f295a311dfe2879c002bba33695e437f859f7ce111645a99c0ec

I think @qlzh727 sent a fix to this.


Granular "unsafeAllow" through NatSpec comments

The latest Solidity allows adding NatSpec comments to any statement. We can use this to disable or enable checks on a more granular level than we do now.

addressed as part of #315


Any way to configure the extension to ignore a .git folder (and dot folders generally)?

Hey!
Really nice extension!
As an additional way of being able to use my scaffolds across different devices, I've decided to make the directory housing my folder templates a Github repository.
The only issue I've encountered is the predictable one by which the extension thinks that the .git folder is a folder template!

Is there any way to configure the extension to ignore the Git directory when globing through the template folder?
TIA!

Hi @danielrosehill ,
currently there's no way to exclude folders from being recognized as templates if they are a direct child of the .fttemplates directory.
I will take a look at this when I find the time. I cannot give an time estimate as I have tons of things to do outside of this project unfortunately.
I'd be thankful for a PR if you want to take a look at this yourself.


Rest API basic authentication always returns 401

What steps will reproduce the problem?
Making api call with basic auth.
Call compositeAuth in Module:
public function behaviors() {
    return [
    ....
        [
            'class' => CompositeAuth::className(),
            'except' => [
                    .....
                ],
            'authMethods' => [
                [
                        'class' => HttpBasicAuth::className(),
                        'auth'  => function ($username, $password) {
                               // do something
                       },
                ],
                [
                      'class' => MyCustomizedAuth::className(),
                ],
            ],
        ],
    ];
}

What is the expected result?
It will authenticate with the right credential - supplied as basic auth in header.
What do you get instead?
Returns 401 with message "Your request was made with invalid credentials."
Additional info

Q
A

Yii version
2.0.46

PHP version
8.1.9

Operating system
Ubuntu 20.04

After some digging, it's caused by this fix - https://github.com/yiisoft/yii2/commit/f72310c398759841a0f8b52e1aba7990086c0d9c
Commenting out the if condition added in this fix, it's authenticated correctly.

This was done to respect only and except options. What is in your except configuration?

'except' => [
    'controller1/index',
    'submodule1/*',
    'submodule2/*',
],

So are those rules preventing the filter to work, which is the reason, or those or not firing here and something is wrong with the CompositeAuth now?

The issue I'm having is not to do with except and only config. The issue is trying to do basic auth but the authentication function wasn't called because of the if statement added by this fix: https://github.com/yiisoft/yii2/blame/f72310c398759841a0f8b52e1aba7990086c0d9c/framework/filters/auth/CompositeAuth.php#L73
The if statement doesn't meet ($this->owner is null), the authenticate function doesn't get called, hence the authentication always fails. If line 73 and 78 commented out, it gets authenticated.
Downgrade to Yii V2.0.45 and it works.

@cwongwpp $this->owner can't be null, but action property($this->owner->action) not defined in Module

Sorry, I don't understand this. What action needs to be defined? This has been working and breaks after upgrading to V2.0.46.

@cwongwpp #19418 is BC break because this changes support attaching only to controllers: yii\base\Module don't have action property.

If using CompositeAuth via config file then $this->owner is an instance of yii\web\Application and it has no action
'as authenticator' => [
    'class' => CompositeAuth::class,
    'authMethods' => [
        ['class' => HttpBearerAuth::class],
    ]
],

@h0rseduck no matter via config or not, Application extends Module

@WinterSilence
Of course.  I just described another case

Sorry, i have the same problem. I was declaring the authenticator behavior into the behaviors function of the module. It is suposed to apply to every controller into that module but as @h0rseduck said $this->owner (in this case) is an instance of yii\web\Application and it don't have action property.
So instead of declaring the behavior into every controller, i came ahead with this temporary solution. Into my module init i do:
Yii::$app->on('beforeAction', fn() => Yii::$app->controller->attachBehavior('authenticator', [
    'class' => CompositeAuth::class,
    'authMethods' => [
        HttpBearerAuth::class,
    ],
]));

Now everything works properly. Please anyone can tell me if this is correct to do? Or should i declare the behavior into every controller instead?

@AleMercadal also, you can do it via config($config['container']['definitions'])

Or should i declare the behavior into every controller instead?

Right way is create your base web controller extends by module controllers:
abstract class BaseController extends Controller
{
  /**
   * @var string[]|true
   */
  public $publicActions = true;

  public function behaviors()
  {
      $behaviors = parent::behaviors();

      if ($this->publicActions !== true) {
          $behaviors['auth'] = [
              'class' => CompositeAuth::class,
              'except' => $this->publicActions,
              'authMethods' => [...]
          ];
      }

      return $behaviors;
  }
}

Perfect! Thanks for your help

thanks for the fix we had the same problem. i hope it will be released soonish

I have Yii 2.0.47 and I'm getting the same response. I have set the behaviors function in my controller like:
class SiteController extends \yii\rest\Controller
{
    public function behaviors()
    {
        $behaviors = parent::behaviors();

        $behaviors['contentNegotiator']['formats']['application/json'] = Response::FORMAT_JSON;
        $behaviors['authenticator'] = [
            'class' => CompositeAuth::class,
            'only' => ['logout'],
            'authMethods' => [
                // ['class' => HttpBasicAuth::class],
                // ['class' => HttpBearerAuth::class],
                ['class' => QueryParamAuth::class, 'tokenParam' => 'access-token'],
            ],
        ];

        return $behaviors;
    }
...
}


Running with MPI crash if len(pset) < MPI processors

Hi,
I'm running a regional model of the Caribbean and I found out that running batches of ~1 month is much faster than a long series. The problem arises because particles are leaving the domain (and deleted), so I start with a few thousand particles but after 5+ years, len(pset) < MPI processors, and Parcel stops with this error:
RuntimeError: Cannot initialize with fewer particles than MPI processors
Any idea how I could automatize this process? Would it be possible to catch this error and decrease the number of MPI processors?

For reference, I ended up using the following to call the executable without MPI when the number of particles is too low.
# use MPI if nb particles > nb proc
if restart_file and use_mpi:
    with xr.open_dataset(join(folder, restart_file)) as df:
        if sum(np.isfinite(df.lon[:,-1].values)) < nb_proc:
            use_mpi = False

Thanks,


Copy block command conflicts with cursorless

Some new commands were introduced to community recently that conflict with the cursorless copy block command. This conflict is unfortunate because there is no simple way to fix this without editing this repository.
Related to this issue as well : https://github.com/cursorless-dev/cursorless/issues/428

Also relevant I think: https://github.com/talonhub/community/issues/1412
Probably implementing the actions proposed by #1412 will fix a lot of these problems. I think it will also improve the fallback functionality in cursorless

https://github.com/talonhub/community/pull/1498
One this had merged you can just with a tall long list file change enable or disable spoken forms without changing community.

Fixed with #https://github.com/talonhub/community/pull/1498

Thank you!

is this fixed on the cursorless side too? I still seem to be running into a conflict

There is no planned change in cursorless.
You need to use a different spoken forms in community and cursorless. Or make sure not both are enabled at thr damer time.

Oh-I thought there was a plan for cursorless to override the same command name. If there isn't a plan to change cursorless, doesn't that mean that the default configuration is still a conflict? (I.e. copy is the same?)

That is a potential future use, but not something needed to solve the conflicts now.
I changed the default in community to paragraph so the conflict is gone.

ah! I see.  yes, that is indeed fixed.  Thank you!


login not working[BUG]

If I try to execute this:
`>>> from instagrapi import Client

cl = Client()
cl.login(USERNAME, PASSWORD)`

I get this error:
`python prova.py
Status 429: Too many requests
Ignore 429: Continue login
Traceback (most recent call last):
File "C:\Users\carlo\AppData\Local\Programs\Python\Python310\lib\site-packages\instagrapi\mixins\private.py", line 302, in _send_private_request
response.raise_for_status()
File "C:\Users\carlo\AppData\Local\Programs\Python\Python310\lib\site-packages\requests\models.py", line 1021, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://i.instagram.com/api/v1/accounts/login/
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\OFFLINEPROJECTS\BOTS\UploadInstagram3\prova.py", line 26, in
cl.login(ACCOUNT_USERNAME, ACCOUNT_PASSWORD)
File "C:\Users\carlo\AppData\Local\Programs\Python\Python310\lib\site-packages\instagrapi\mixins\auth.py", line 390, in login
logged = self.private_request("accounts/login/", data, login=True)
File "C:\Users\carlo\AppData\Local\Programs\Python\Python310\lib\site-packages\instagrapi\mixins\private.py", line 451, in private_request
raise e
File "C:\Users\carlo\AppData\Local\Programs\Python\Python310\lib\site-packages\instagrapi\mixins\private.py", line 438, in private_request
self._send_private_request(endpoint, **kwargs)
File "C:\Users\carlo\AppData\Local\Programs\Python\Python310\lib\site-packages\instagrapi\mixins\private.py", line 358, in _send_private_request
raise UnknownError(**last_json)
instagrapi.exceptions.UnknownError: You'll need to update Instagram to the latest version before you can use the app.
`
Is this a bug?

Hi, I'm having the same problem. What do I need to do to solve it?

Bad IP or Proxy. Change it

according to the documentation, you can add a delay to mimic the behaviour of a real user:
# adds a random delay between 1 and 3 seconds after each request cl.delay_range = [1, 3]
and you can check if the username exists by sending a get request to www.instagram.com/{username} because the login function always return true if the credentials are false!!


Jupyter Hub implementation

Great extension!
I tried to use it in jupyterhub and it does work very well, but only with the user that has sudo permissions (running the the littlest jupyter hub on ubuntu 18). While the options appear in the launcher of all users, only URLs can be opened, the notebook button is unable to create a notebook (unless is in the user with sudo privileges for which it works flawlessly). Is there any call that requires additional privileges?

Hi, for the case of the notebook launcher, the extension will read the content of the notebook and then send it to the frontend to create the notebook file https://github.com/trungleduc/jupyter_app_launcher/blob/fc23909eb26ff05bb0369b436755d01f7ddc8101/jupyter_app_launcher/utils.py#L28-L36.
Maybe normal users do not have the right to read your notebook file.

I think it was an issue with the path I was using, changing it to the full path fixed it! Thanks for the prompt reply!


tone down Send and Sync stuff

Given we haven't explained traits, it might be too much to try and talk about the differences between Send and Sync.

Fixed in #13


install bats from source on centos

The centos CI systems don't have bats and are not configured for epel, so install bats from source.

closing this one as not valid anymore.


Investigate changes and support for VS Code Docker debugging tooling

I created https://github.com/microsoft/vscode-docker/issues/3808 to investigate support for this tech from their side, but it's entirely possible that we will need to expose new data to enable integration. We should try to do that ahead of the full integration.

Specific requests from the VSCode/VS Docker tools teams:

Add a ProjectCapability (name TBD) to our evaluation so that it's easy to tell if a project is using us or not.
Move definition/inference of the base image to evaluation if feasible - save validation for the targets, but the raw property manipulation could move up. I have worries with this that we'd need to check:

Is the AspNetCore ProjectCapability added during evaluation? that's a valuable input into base image inference
Same question for SelfContained/PublishSelfContained
For the convention-based Labels like source control url/commit SHA, those definitely come from data discovered during targets, so at least some data will be added at target-execution time instead of evaluation

Consider tackling #125 and #146 sooner rather than later.

Closing this because it released last Monday!


Add property to Driver to return driver state

Is your feature request related to a problem? Please describe.
If you fire up zwave-js with zwave-js-server but have an application connect to the zwave-js-server after the "driver ready" and/or "all nodes ready" events have fired, there's not a good way for the application to know the state of the network since it missed those events.
Describe the solution you'd like
My recommendation is for an additional property on the Driver class that returns the state of the network. ZWave-JS core could update that property when it emits the event. That way if an application fires up and misses one of the events it can still easily know what state the network is in.

Off the top of my head:

driver ready?
which nodes ready?
all nodes ready?
interview for which nodes completed?

Off the top of my head:

driver ready?
which nodes ready?
all nodes ready?
interview for which nodes completed?

Important states I can think of from an application standpoint:

is the driver ready
are all nodes queried or are we still querying

Important states I can think of from an application standpoint:

is the driver ready
are all nodes queried or are we still querying

@cgarwood I've opened PR #1396 to add two properties (driver ready, all nodes ready). I think this should be enough for your use case.
You should be able to check whether all queries are done using
const allInterviewsCompleted = [...driver.controller.nodes.values()].every(n => n.interviewStage === InterviewStage.Complete);

@cgarwood I've opened PR #1396 to add two properties (driver ready, all nodes ready). I think this should be enough for your use case.
You should be able to check whether all queries are done using
const allInterviewsCompleted = [...driver.controller.nodes.values()].every(n => n.interviewStage === InterviewStage.Complete);


Does this page need a reference to Mesh?

Given the importance of data sharing to the Mesh approach, does this page need some link or reference to the Mesh model implementation of data sharing?

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

ID: 8164dd01-029b-1782-473a-5e674ea16a1e
Version Independent ID: 3d7bc3c5-2a3c-baea-61d3-6539b59d00b0
Content: Data sharing agreements in data management and analytics - Cloud Adoption Framework
Content Source: docs/scenarios/data-management/govern-data-sharing-agreements.md
Service: cloud-adoption-framework
Sub-service: scenario
GitHub Login: @pietheinstrengholt
Microsoft Alias: pstrengholt

@angusforeman I rewrote the sharing contracts in this page have a look at it https://github.com/MicrosoftDocs/cloud-adoption-framework-pr/blob/collab-data-mesh-v1/docs/scenarios/data-management/architectures/data-contracts.md to see if this address your concerns

#label:"data management and analytics"
#please-close


For ROI connectivity table, freeze the first column and first row

The ROI names can barely seen without backgrounds.

meant for neuPrintExplorer


Odd table shapes issue

I am pre-processing a large number of .docx documents with really oddly shaped tables containing text that has to be extracted verbatim.
As useful python-docx has been in this task, a subset of those documents revealed a tiny little bug in this line.
This PR fixes cases of odd table shapes were the strategy of populating a cell with the value of the previous cell (e.g. in the case of row/cell merges) fails, because there simply has not been a 'previous cell' yet.
Please note, I would be glad to contribute a test case as well but this might take a bit more time, tracking down the exact table (within the XML) that causes the bug and creating an "equivalent" test case.
Hope this helps.

@aanastasiou there was a recent update that addressed the "skipped-cells" condition that is actually a legitimate (although relatively unusual) table state.
If you use Table.rows to get rows and then iterate _Row.cells to get each cell you shouldn't have a problem there.
Depending on your needs for column alignment you may want to use _Row.grid_cols_before and .grid_cols_after to discover the empty leading and trailing cells.
There is also a new _Cell.grid_span property so you can tell how many grid-cells a horizontally-merged cell occupies.
I'm not sure what we'll do with Table._cells. It's possible that collection will be deprecated or perhaps we'll reimplement it based on the new "skipped-cell-aware" code, but for now it is probably better to avoid it in favor of the new methods.

@scanny thank you very much for the prompt response. This was using the latest python-docx from pypi, would this recent update be applied to the version on github rather than pypi? Thanks for the rest of the information, it's good to know for our next code revision.

This change appears in v1.1.2, which is the current PyPI version, released on May 1, 2024:
https://pypi.org/project/python-docx/
https://github.com/python-openxml/python-docx/commit/f4a48b5565a3a09087f541e3ac36a447693927b4

@scanny This is the version that I used (and eventually led me to file this PR)

Show me the client code that isn't working the way you want.

@scanny The PR contains the exact problem that I dealt with (and how), what might take longer is me locating the exact document that causes this behaviour.

@aanastasiou the idea there is not that this problem with table._cells is fixed for your case, but rather that you should no longer need to use table._cells and can use something like (c for row in table.rows for c in row.cells).


Feat(Compiler-Base): Reuse kclvm emitter

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

issue #115
2. What is the scope of this PR (e.g. component or file name):
compiler_base/error/emitter.rs
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[x] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

deleted method apply_style(), Style provides method render_style_to_color_spec() to replace it.
Adapted trait Emitter for Diagnostic and StyledBuffer.
4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 59.579%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
22391

Relevant Lines:
37582

💛  - Coveralls

LGTM


switch between kilometers and miles from language/region

new (non metric)

new (metric)

old

Nice. Also remove points?


brew cask install fastlane error

[~] → brew cask install fastlane
Updating Homebrew...
==> Auto-updated Homebrew!
Updated 1 tap (homebrew/core).
==> New Formulae
golang-migrate

==> Satisfying dependencies
==> Downloading https://fastlane.tools/fastlane.zip
######################################################################## 100.0%
==> No checksum defined for Cask fastlane, skipping verification
==> Installing Cask fastlane
==> Purging files for version latest of Cask fastlane
Error: Failed to quarantine one or more files within /usr/local/Caskroom/fastlane/latest. Here's the reason:
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/test/minitest/test_minitest_benchmark.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/test/minitest/test_minitest_unit.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/test/minitest/test_minitest_reporter.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/test/minitest/test_minitest_mock.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/test/minitest/metametameta.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/test/minitest/test_minitest_spec.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/Manifest.txt'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/Rakefile'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/design_rationale.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/hoe/minitest.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/mock.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/hell.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/pride.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/parallel.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/unit.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/expectations.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/pride_plugin.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/spec.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/assertions.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/test.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/autorun.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest/benchmark.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/lib/minitest.rb'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/README.txt'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/History.txt'
xattr: [Errno 13] Permission denied: '/usr/local/Caskroom/fastlane/latest/fastlane_lib/bundle/lib/ruby/gems/2.2.0/gems/minitest-5.4.3/.autotest'

I'm experiencing the same thing with a fresh install of fastlane.

It looks like this might be related to the homebrew cask bug here: https://github.com/Homebrew/homebrew-cask/issues/51538

Is there any solution to solve this bothering problem? ;D

yeah, if you run brew cask install fastlane --no-quarantine, you should be able to get around it.


Add Reddit post cleaning notebook

Description
Adds a new notebook which shows potential ideas for cleaning news stories from Reddit.
Number of Files changed
1
Questions for Reviewer
Type of Change

[x] New Feature
[ ] Feature Update
[ ] Fix Error

Status

[x] Tested

Description
Updated the HTML grabbing code, also shows better results
Number of Files changed
1
Questions for Reviewer
Type of Change

[ ] New Feature
[ ] Feature Update
[ ] Fix Error

Status

[ ] Tested


Please allow spaces in user name

I would like to use the connectivity analyser against an on-premise Exchange server where the test account has a literal space character in the user name, e.g. "MY-DOMAIN\My Name", which is valid in Windows/AD etc, but the tester won't let me enter the space character. Please allow spaces in the user name.

Thanks for the feedback. I've opened an internal product bug to track this issue as this location is more to track issues with the documentation. We'll consider this for future updates to the tool.


Devfile registry operator release automation

/kind user-story
Which area this user story is related to?

/area registry
/area releng
User Story
As a maintainer of devfile services, I want to have release automation, so that the process of creating registry operator releases are efficient to execute.
To make it efficient for devfile service maintainer to perform, we should have a Makefile rule to execute the steps to start the release process on GitHub. This should allow a similar process as the release script for devfile/api: https://github.com/devfile/api/blob/b4fea572af1a98ca487495857add4007c35f5aea/make-release.sh
Acceptance Criteria

[x] Create new release Makefile rule
[x] Add process to make final release changes

Such as updating the release version number references and correcting missing alm-examples when running make bundle (kubernetes-sigs/kustomize#5042)

[x] Add process to checkout target release branch and commit release changes to that branch
[x] Add process to create PR to merge new release changes into remote target release branch
[x] Test the new rule on a forked repository
[x] Update usage documentation with this new rule

Moving to review as I will be opening a PR but marking it as do not merge. @michael-valdron will be working on a fix for the previous release to registry-operator where a squash merge occurred whereas we need the entire git history. Once that is done this issue should be good to mark as complete and merge.

This issue is currently blocked as it requires changes to the git history of the release branches for registry operator. Currently there is squash commits in the release branches which conflict with the automation as the bot cannot handle merge conflicts without human intervention, which defeats the purpose of the automation. This is occurring because the releases are based off of main but in the release branches it has diverged from main in the git history with these squash commits. In order to prevent this from happening and to allow this issue to become unblocked the git history needs to be altered in the release branches so that it does not diverge from main.
release-v0: https://github.com/devfile/registry-operator/commits/release-v0/
main: https://github.com/devfile/registry-operator/commits/main/

This issue is currently blocked as it requires changes to the git history of the release branches for registry operator. Currently there is squash commits in the release branches which conflict with the automation as the bot cannot handle merge conflicts without human intervention, which defeats the purpose of the automation. This is occurring because the releases are based off of main but in the release branches it has diverged from main in the git history with these squash commits. In order to prevent this from happening and to allow this issue to become unblocked the git history needs to be altered in the release branches so that it does not diverge from main.

@Jdubrick I have talked with @thepetk while discussing the #1605 approach that I think this process should be altered to align with other devfile repos by only using the release branches to backport patches to target major and minor releases which I believe mirrors #1516 you created back in April. The epic #1518 should be completed before this issue can be unblocked.

This should be kept open to complete automation for a quicker release process for our team to perform, suggest devtools-week tag be added to this one.


Lack of avatar causing unknown server error

This happens during onboarding, and it looks like the change to make avatar required did not persist (or isnt live).

Resolved the core issue in #32 but another issue is present in which saving your settings after an avatar is present will instead wipe it.


Old master nodes are not releasing queued tasks back to the DB

We need jobs that will identify that a node is not the master and will release any queued tasks back to the DB and clear the queue. We also need a process that will release tasks owned by a node that may no longer be running.


Unable to create, edit or remove QA Data

Issue:
Can not edit, add, or remove a QA record
Steps to recreate:

Log into BETA
Check out config Glen Lyn (3776) 51
Select Lineraity or Rata as a test type
Attempt to add a record, remove an existing record, or edit a record

The system doe not maintain the changes and displays the following error message: Request failed with status code 500
Other scenarios: When a user adds new Test Data and clicks, Save and Close - the modal closes but no new Test Data is added and no errors/validations are displayed

Linearity Summary, Linearity Injection, and Protocol Gas added

Test Summary, Linearity Summary, and Protocol Gas -- all showing edited values

Linearity Summary and Protocol Gas records removed -- but had to close and re-open Glen Lyn (51) tab to see them removed from the Test Summary


An ability to specify AMI in Provisioner spec

Tell us about your request
It would be convenient if we were able to specify custom AMI in the Provisioner spec instead of creating a lot of custom launch templates just to change the AMI in them.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Currently I should create a lot of launch templates and change only AMI in them.
It is annoying because Karpenter already creates launch templates using taints/labels from the Provisioner spec.
Why can't we add AMI as well?
Thanks!

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

The AMI and the UserData in the EC2 LaunchTemplate are tightly coupled. We'd have to expose both of them in some form in the provisionerSpec to address this.

@suket22
Definitely, UserData is also very important!

Praying for this aswell. Karpenter is the dream come true to get rid of managing infra in IAC completely.

I've opened #1495 which might cover part of the use case for this?

The AMI and the UserData in the EC2 LaunchTemplate are tightly coupled. We'd have to expose both of them in some form in the provisionerSpec to address this

A MVP solution for this might be to only support custom AMIs where there are based on an existing family and retain the bootstrapping logic of that family?

Hi @stevehipwell how are you? Is there an ETA on this FR?

@JohnMops I think there are some changes due, but I'm not sure if they cover this.

Support coming in v0.14


Last-Event-Id Header model

The last-event-id processing for reconnecting clients uses a special HTTP-Request-Header defined at http://www.w3.org/TR/2011/WD-eventsource-20110208/#last-event-id.
akka-sse could provide a parser and renderer for this purpose. (WIP :) )

Okay a full header model seem to be a bit overkill because its a header field with a plain value. Maybe a single alias-directive like def withLastEventId = optionalHeaderValue("Last-Event-ID") would do it

Solfed by # #92 in v.1.7.1. Cheers!


where to set pydev?

when enter debugger mode. occur this hint：
pydev debugger: To debug that process please enable 'Attach to subprocess automatically while debugging?' option in the debugger settings.
so，where is debugger settings? I am using vs2017 community version.

Sorry, some of these pydevd warnings/insructions are not applicable in ptvsd and can be misleading. We'll be suppressing some of these in the future.
Automatically attaching to subprocess is not supported yet. We're tracking this at https://github.com/Microsoft/ptvsd/issues/57

So, how should I debug my code with vs 2017? Or, how do I switch to single process from multi process?
thanks.

Single process is the best option for now.


Support some builtin funcs

Adds recognition of the builtin funcs of the FUNC0 (0 args) and FUNC1 (1 arg) variety.
Does not yet add the list-shaped funcs, because I'm having trouble making list-associative precedence work properly for that. But this is a decent start.

Now force-pushed with your fixes to use lexical precedence instead

We may wanna add a test that int() parses precisely the way we want (the benefit of the way you wrote it over simply having func1 contain an optional($._term)

Ahyes; added.